Origins 1.17.1 does not work with optifabric

I'm not sure if it's on purpose so some features of the mod can work but you can't use Optifabric with Origins.

Excerpt from the !optifine command in origins discord
OptiFine is closed-source and handles Minecraft code very invasively. This means it breaks many mods and there's not much mod developers can do about it. From the developer of OptiFabric, the compatibility mod that makes using OptiFine possible on Fabric:

Please note the in the 1.16 (and somewhat 1.15) versions there are a lot of incompatiblities with a great number of mods, this is due to the way optifine changes the vanilla code in increasingly invasive ways. Fixing these crashes isnt easy or fun and takes a lot of time that I dont really have.

Therefore OptiFine won't work with Origins. If you are looking for alternatives, Fabric has a couple of options which you can try, depending on what you use OptiFine for. See here for a comprehensive list:
https://gist.github.com/LambdAurora/1f6a4a99af374ce500f250c6b42e8754

Tl;dr optifine is made is a way where Apace can't really account to make it compatible, the best solution is to use optifine alternatives. The list being here:
https://gist.github.com/LambdAurora/1f6a4a99af374ce500f250c6b42e8754


Add "Colour" to "Color Science" sub-category.

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project open source, and does your project have at least 250 GitHub stars?
[x] Has this project been used in the production of a film or game?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~10 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

This PR adds Colour to the landscape of Open Source software used in the Motion Picture Industry.
Colour is used in many VFX (and Games) studios to perform scientific computations and process digital imaging data. It is also used by various Fortune 500 companies to validate and refine colour transformation pipelines, e.g. Google with Youtube.

The build seems to unfortunately break:
7:52:41 PM: TypeError: Cannot read property 'substring' of undefined
7:52:41 PM:     at substring (/opt/build/repo/node_modules/interactive-landscape/tools/fetchImages.js:154:107)

The real error was just above: No city for a crunchbase entry for colour-science at https://www.crunchbase.com/organization/colour-science

Ahh, you're right about the other build error. We're investigating.

@KelSolaar - looks like the issue is around the crunchbase entry...
No city for a crunchbase entry for colour-science at https://www.crunchbase.com/organization/colour-science

It looks like it's fixed now so maybe a rebuild will get it working. Let me trigger that.

I fixed that issue. There’s a separate issue with the SVG that we’re investigating.

I see there's an exception being thrown which is not handled well, so this itself ends up throwing a different exception.
Commenting out some of the code, the original exception seems to be:
Error in parsing SVG: Unbound namespace prefix: "serif"

Great. Could you please improve the exception handling and manually edit the SVG to work.


When looking for a node js package, it seems to be looking in the wrong place.

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
6.0.0-beta.63
Electron version
18.2.0
Operating system
MacOS 12.3.1 (arm64)
Last known working Electron Forge version
No response
Expected behavior
Find the package and run the nodejs script
Actual behavior
Throws an error in the console.
Error: ENOENT: no such file or directory, open '/Users/nitesh/Downloads/utils/bosgui2/.webpack/grpc/protos/autopilot.proto'
    at Object.openSync (node:fs:585:3)
    at Object.func [as openSync] (node:electron/js2c/asar_bundle:5:1812)
    at Object.readFileSync (node:fs:453:35)
    at Object.e.readFileSync (node:electron/js2c/asar_bundle:5:9160)
    at fetch (/Users/nitesh/Downloads/utils/bosgui2/.webpack/main/index.js:275647:34)
    at Root.load (/Users/nitesh/Downloads/utils/bosgui2/.webpack/main/index.js:275681:13)
    at Root.loadSync (/Users/nitesh/Downloads/utils/bosgui2/.webpack/main/index.js:275722:17)
    at Object.loadProtosWithOptionsSync (/Users/nitesh/Downloads/utils/bosgui2/.webpack/main/index.js:11211:29)
    at loadSync (/Users/nitesh/Downloads/utils/bosgui2/.webpack/main/index.js:11113:31)
    at module.exports (/Users/nitesh/Downloads/utils/bosgui2/.webpack/main/index.js:56281:42) {
  errno: -2,
  syscall: 'open',
  code: 'ENOENT',
  path: '/Users/nitesh/Downloads/utils/bosgui2/.webpack/grpc/protos/autopilot.proto'

Steps to reproduce
N/A
Additional information
Any work around for this?

Yeah it’s the web pack template. Sorry, for the delay in responding.
I will respond with the repro in a couple of days. Traveling now.


Fixed height cell

Put an example for fixed height cells. The requirement is for mixed-signal designs. Get help from Meghna to get SAR_ADC test case and pass this down to Arvind.

Can we move the cell height parameter to the layers.json or somewhere the user has access to change it? I think the cell generator is able to support the requirement of having a specified height while respecting the no. of fins per unit cell mentioned in the netlist.

@kkunal1408 We have fixed most of it in #870.  Do we need "unit_size_nmos" and "unit_size_pmos" values?

@arvuce22 , @meghna We would need this to check if nfin defined in netlist < unit_size_mos. We can rename this to something which would make more sense.

@kkunal1408 @meghna09 Currently, we have four parameters in 'layers.json': "unit_size_nmos", "max_size_nmos", "max_size_pmos", "unit_size_pmos". Instead of these we can have two parameters "unit_size_mos" and "max_size_mos"

@arvuce22  max_size_mos would represent the cell height? What does unit_size_mos imply?

@meghna09 We will remove "cell height", and "max_size_mos" will define maximum number of fins in a transistor (this is a limit from pdk)

@arvuce22 is having cell height and max_size_mos (limit from pdk) more suitable than unit_size_mos? We can discuss this in our meeting.

@arvuce22 to lead discussion on this


introduce husky to guard ramda

Run npm test and lint before git push.

That's probably a good idea.  I do it in all my work projects these days.
Any objections, @ramda/core?

I dislike commit hooks, but I have not committed to this project for a long time so don't mind me.
On an unrelated note, why do we include package-lock.json in the repository? I've never seen a reason for libraries to include such files.

It's pre-push, not pre-commit.
👎 from me – this checks working directory, instead of commits that are about to be pushed.
About lockfile: https://github.com/ramda/ramda/pull/2625#issuecomment-412553820.

👎 from me – this checks working directory, instead of commits that are about to be pushed.

Using lint-staged , we can lint only changed files at commit stage, and only run unit test at push stage.

I wouldn't oppose lint-staged, even tho I'm not sure if it properly handles partial stages.
Naive npm test in pre-push disallows untracked files in source/ and makes no sense when using git push <remote> <branch>. I don't know how to do this properly, I don't think this is the right place.
CI already lints and runs tests, correctly.

Naive npm test in pre-push disallows untracked files in source/ and makes no sense when using git push <remote> <branch>. I don't know how to do this properly, I don't think this is the right place.

You can stash the untracked files before push.
Run lint and unit test locally can find some obvious errors earlier, maybe this can save some time than remote CI.

Hooks are supposed to enhance or ensure valid workflows, not break them.


cmd/go: go install downloads too much

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

$ go version
go version go1.16.2 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/aaron/.cache/go-build"
GOENV="/home/aaron/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/aaron/TESTDOWNLOAD/GOPATH/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/aaron/TESTDOWNLOAD/GOPATH"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.16.2"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build2663290531=/tmp/go-build -gno-record-gcc-switches"

What did you do?
mkdir -p ~/TESTDOWNLOAD/GOPATH
cd ~/TESTDOWNLOAD
export GOPATH=$PWD/GOPATH

go install github.com/volatiletech/sqlboiler/v4@latest

What did you expect to see?
I expected to see only sqlboiler/v4 downloaded installed.
What did you see instead?
I additionally saw it downloading source for versions I was not interested in at all nor appear to be part of the dependency tree.  Perhaps I should add these and commit them as the maintainer of the package.

RE: your last point, apparently it's expected that go mod tidy and go mod download do not result in the same go.sum anymore. See #43994 (and the many linked issues and workarounds people have been applying in other repos...).

cc @bcmills @jayconrod @matloob

Those source downloads are because the go command is looking for a package named at ./v4 within module github.com/volatiletech/sqlboiler. For go get that's maybe not such a big deal because the main module will in general already require the module at the correct version, but for go install I agree that it seems excessive.

Also as a completely separate nit trying go list all without changes in the repo produces the error:
missing go.sum entry for module providing package gopkg.in/check.v1 (imported by gopkg.in/yaml.v2); to add:
	go get -t<EMAIL_ADDRESS>
Running this command changes nothing and the error continues

I'm not able to reproduce that behavior:
sqlboiler$ go list all >/dev/null
missing go.sum entry for module providing package gopkg.in/check.v1 (imported by gopkg.in/yaml.v2); to add:
        go get -t<EMAIL_ADDRESS>
sqlboiler$ go get -t<EMAIL_ADDRESS>
sqlboiler$ go list all >/dev/null

sqlboiler$

However, that sounds like it could be #44749. The fix for that (#44812) just barely missed 1.16.2.


Issue #6646 extend GMP implementation

I made this pull request after the discussion in thread #6646. This solved most of current BigInt implementations performance issues, and using the Benchmarks game pidigits benchmark as a measurement of performance sets this on par with the Rust implentation (within ~5%).
You can now mutate a BigInt variable and reuse a variable for the most common operations add, sub, mul, div, submul, addmul.
The api will look like this:
# mutating a variable comparable to a += b, avoiding new allocations:
a = # some big int
b = # some big int
a.add(b) # a += b

# Reusing a variable, avoiding new allocations:
a = # some big int
b = # some big int
c = # some big int
a.add(b, c) # a = b + c

See the suggested documentation for more information.

Continuing discussion after https://github.com/crystal-lang/crystal/pull/6755#discussion_r219320571 (@Sija)

I'd model the API after Ruby, since IMO it's clear about how these operations work, i.e.:

#add!(other) => a = a + other
.add(result, op1, op2) => result = op1 + op2

I agree for #add! but not for the second, it's unclear to me that it won't add all args and return a new instance with the result.

[...] it's unclear to me that it won't add all args and return a new instance with the result.

You pass a result argument so judging from that, it's clear (at least to me) it's gonna be (re)used for the returned value.
Otherwise signature would be .add(op1, op2).

You pass a result argument

Unless it's a required named argument, this isn't clear at the call site. That's the place where this method needs to distinguish itself from #+.
I guess it could also be #add(op1, op2) but I'm not entirely fond of it, because it looks like adding two summands to self, when it is actually adding two summands and writing the result to self.

@straight-shoota I agree. In addition to having the add methods spread around more by dividing them, there is no president (as far as I know) in other Crystal libraries to have operations that takes a result argument like this. But I agree that #add is not perfect either.
Since a.add(b) => #a += b seems to be the "normal" way to read an add method, can it be a solution to make the a.add(b, c) to a.add!(b, c) to signal that this method deviates from what most will assume that the "normal" add operation does?
Just for reference, I send the link to @asterite 's thoughts from the other thread which also adress this very issue: https://github.com/crystal-lang/crystal/issues/6646#issuecomment-419756459

Suggestion: move BigInt and other types to a shard, so it can be maintained outside of the core team (I think no one in the core team has experience with libgmp).

Do we have any users of Big* in the stdlib?

@RX14 No, not at all

I'm fine with that. I'll close this then if there are no objections.

For the record, I would vouch for #add! as a mutating add method. Even if there is no #add since + can be used for that. I think that it is more clear.


FutureState not work with params

Hi,
I can find example how to do future state with param. if I go /albums/Detail/:id then all ok, but if go to /albums/Detail/123 then it can't find it.
Please advice.
here is my module:
define(['angularAMD', 'angular-ui-router', 'ui-router-extras' ], function () {
var app = angular.module("albumsModule", ['ui.router', 'ct.ui.router.extras']);
var states = [
     {
         stateName: "albumsDetail",
         //urlPrefix: "/albums/:id",
         urlPrefix: "/albums/Detail/:id",
         templateUrl: "app/modules/albums/views/albumsDetail.html",
         type: "requireCtrl",
         //type: "ngload",
         deps: ['modules/albums/services/albumsService'],
         src: "modules/albums/controllers/albumsDetailController",
     },
];

app.config(['$stateProvider', '$futureStateProvider', function ($stateProvider, $futureStateProvider) {

    var loadAndRegisterFutureStates = function () {
        angular.forEach(states, function (fstate) {
            // Register each state returned from $http.get() with $futureStateProvider
            $futureStateProvider.futureState(fstate);
        });
    };

    $futureStateProvider.addResolve(loadAndRegisterFutureStates);
}]);
return { module: app };

});

@ssleptsov, could you please tell me how did you get the value of the :id parameter inside the future state factory?


Installing DATA LAKE CLI on Windows

Is it possible to install the DATA LAKE CLI on Windows?
Is the DATA LAKE CLI
http://docs.awssolutionsbuilder.com/data-lake/cli/cli-getting-started/
the same as the AWS CLI ?
https://docs.aws.amazon.com/cli/latest/userguide/cli-chap-install.html

Data Lake CLI is not the same as Aws Cli!
Description:
The Data Lake Command Line Interface (CLI) is a tool to manage your Data Lake. By downloading and configuring the Data Lake CLI, you can control multiple aspects of your Data Lake from the command line and automate them through scripts.
http://docs.awssolutionsbuilder.com/data-lake/cli/cli-getting-started/

Hi @jlopezvi, please let us know if the above answer is clarified and if you still have questions. If not, can we close this issue

Is it possible to install the DATA LAKE CLI on Windows?

Hi @jlopezvi unfortunately there is no official release for windows at this point in time, but it is Nodejs based command line utility that should work on windows as well. Once you download and unzip the files, you will find an install.sh at the root of the unzipped folder. There are two commands that you would need to execute on the command prompt (executing the install.sh shell script will not work on windows).
cd ./datalake-cli-bundle
npm link
This should install the cli bundle. (Note the pre-requisite is that you have Nodejs installed on your machine)


Mod Menu Bug

Describe your bug here.
I tried to edit the background asset for the week and it came up with this

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No this a built download.

file path seems to be incorrect

file path seems to be incorrect

Ill see if i can edit it without it crashing, ive seen then going on with every text in psych engine


Dark mode enhancement (Recent event1)

I have changed all those necessary things that are required.
@anushbhatia Is there any change required?

it shows the scroll button and change the colour of the heading

Sir, I haven't worked on this. This PR is based on the RECENT EVENT1, not EVENT1.


set up an @net package

This doesn't work as is (no such package '@net472//')... but I think this is some sort of thing on the right path?

ERROR: /home/omsmith/.cache/bazel/_bazel_omsmith/eb8aeb14f366047a0d034e9746595d85/external/net/BUILD.bazel:4:1: no such package '@net472//': The repository '@net472' could not be resolved and referenced by '@net//:System.Linq'
ERROR: /home/omsmith/.cache/bazel/_bazel_omsmith/eb8aeb14f366047a0d034e9746595d85/external/net/BUILD.bazel:11:1: Traceback (most recent call last):
        File "/home/omsmith/.cache/bazel/_bazel_omsmith/eb8aeb14f366047a0d034e9746595d85/external/net/BUILD.bazel", line 11
                nuget_package(name = "net20", package = "Microso...", <2 more arguments>)
        File "/home/omsmith/.cache/bazel/_bazel_omsmith/eb8aeb14f366047a0d034e9746595d85/external/d2l_rules_csharp/csharp/private/rules/nuget.bzl", line 21, in nuget_package
                http_archive(name = name, urls = urls, type = "zi...", <3 more arguments>)
http_archive must be in the WORKSPACE file (used by @net//:net20)

Alright, I think I'm stuck here:
ERROR: /home/omsmith/.cache/bazel/_bazel_omsmith/eb8aeb14f366047a0d034e9746595d85/external/net/BUILD:3:1: no such package '@net472//': The repository '@net472' could not be resolved and referenced by '@net//:System.Linq'
ERROR: Analysis of target '//examples:lib' failed; build aborted: Analysis failed

What we found out IRL: the sub-WORKSPACE isn't getting used. This lines up with some issues in bazelbuild/bazel. @omsmith is going to try moving the nuget_package rules back into csharp_repositories(). We can control visibility via the visibility attrs later.


fix: remove dependency of test utils in none test code

Hey, I just made a Pull Request!
IMHO this is bad practice to use test dependencies in none test code.
Using backend-test-uitls also gives the error now and breaks the current version
[1] /Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:64
[1]     child: jest.fn().mockImplementation(createLoggerMock),
[1]            ^
[1]
[1]
[1] ReferenceError: jest is not defined
[1]     at createLoggerMock (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:64:12)
[1]     at <anonymous> (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:406:7)
[1]     at Object.mock (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:114:18)
[1]     at Object.<anonymous> (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/plugins/tech-insights-backend/src/service/persistence/persistenceContext.ts:40:31)
[1]     at Module._compile (node:internal/modules/cjs/loader:1369:14)
[1]     at Module._compile (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/pirates/lib/index.js:117:24)
[1]     at Module._extensions..js (node:internal/modules/cjs/loader:1427:10)
[1]     at Object.newLoader [as .ts] (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/pirates/lib/index.js:121:7)
[1]     at Module.load (node:internal/modules/cjs/loader:1206:32)
[1]     at Module._load (node:internal/modules/cjs/loader:1022:12)
[1]     at Module.require (node:internal/modules/cjs/loader:1231:19)
[1]     at require (node:internal/modules/helpers:179:18)
[1]     at Object.<anonymous> (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/plugins/tech-insights-backend/src/service/persistence/index.ts:17:38)
[1]     at Module._compile (node:internal/modules/cjs/loader:1369:14)
[1]     at Module._compile (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/pirates/lib/index.js:117:24)
[1]     at Module._extensions..js (node:internal/modules/cjs/loader:1427:10)

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@Xantier please review as current version is broken.

@Xantier right. I can change to minor version and add a breaking change description in the changelog.
Would this be fine?

Sorry, I totally missed this when I reviewed the code change that introduced this bug. :(

FYI @webark

updated PR to announce breaking change and use minor change.


Lesson 10 : Gas estimation failed

Hello, I have this message : ValueError: Gas estimation failed: 'execution reverted' when I want to run my deposit function.
Any help would be greatly appreciated.
My code can be found here: https://github.com/Karlus44/aave_brownie_py
It differs from the Patrick Collin's code in the following points:

I didn't do well with the Kovan faucets in my browser, and at the same time I realised that in the first part of the course I had been trying the https://v3-test.aave.com/ where I could use my rinkeby ether. That's how I decided to keep using Aave V3 and my rinkeby ETH. Now, sadly, there is for now no ETH market in the V3, so right now my tests are only with rinkeby.
In the V3, the name of the function is not "deposit", but "supply" and some other names may be differents. Yet I believe I did things carefully. Anyway, the supply function is the one that gives me an error.
The previous function (approve_erc20) did work well, the transaction is here in rinkeby: 0xb3b9d55928c767f1cd3f4f775e6030ad1457fe4ff2454c397992b18aba8ae8a5

An edit: I've been trying to code the next function: get_borrowable_data, which returns me:
You have 0 worth of ETH deposited. You have 0 worth of ETH borrowed. You can borrow 0 worth of ETH.
That means the program can't read correctly my account.

But the questions remains, why my function get_borrowable_data (and actually getUserAccountData) doesn't work ?

Hello @Karlus44 , I also have the same error, but I have ETH in my account...

@blackyinsol  you mean your get_borrowable_data function detects your ETH in your account ?


Configuration file reading

Hello,
I'm working with this library for my company and I have a trouble with the configuration file:
In my code I have this:
import PyU4V

PyU4V.U4VConn.file_path = 'xxxxx/PyU4V.conf'

...

conn = PyU4V.U4VConn(server_ip=ip, array_id=serial)

But this simple code return me this:
Traceback (most recent call last):
File "./powermax.py", line 596, in <module>
conn = PyU4V.U4VConn(server_ip=ip, array_id=serial)

File "xxx/PyU4V/univmax_conn.py", line 95, in __init__ raise exception.MissingConfigurationException
PyU4V.utils.exception.MissingConfigurationException: PyU4V settings not be loaded, please check file location or univmax_conn input parameters.

After a day of research I find a solution:
PyU4V.univmax_conn.file_path = 'xxxx/PyU4V.conf'
I think it's because the variable has remove from the class to be place on the file or maybe the Documentation is not good.
Documentation link:
https://pyu4v.readthedocs.io/en/latest/configuration.html#alternative-pyu4v-conf-load-using-u4vconn-file-path
PS: I maybe did a mistake during the rewrite of the error.

Hi!
Thanks for highlighting this, I had a look at our docs and the code and you are correct, the docs should indeed read PyU4V.univmax_conn.file_path = 'xxxx/PyU4V.conf' because it is defined outside of the U4VConn() class. I'm so used to having my PyU4V.conf file in ~/.PyU4V that this one went unnoticed. I will get it fixed in the next commit.
Thanks again!
Michael

No problem and thanks for your reply

Hi,
This issue has been resolved in pull request #94, once CI has been run on the changes it will be merged with master and reflected in our online documentation.
Thanks again for highlighting this issue,
Michael


Incorrect counts

The below content displayed talks about 100% recovery which is incorrect count of people recovered.
"944856 people are so far affected in Karnataka by novel coronavirus covid-19. 926664 out of 944856 have recovered."

Hi @MrRahulpd ,
Thanks for reporting the issue and apologies for late reply.
The data has been fixed now and is regularly updated. Please check now.


Make duplicate log suppressing a setting in config

Follow-up to #1431
This replaces the KONG_TEST_ENVIRONMENT environment variable (used by KIC as of #1431) with a setting in config.Config called log-reduce-redundancy (which maps to the --log-reduce-redundancy hidden flag, and the CONTROLLER_LOG_REDUCE_REDUNDANCY environment variable.
Also, moves the setting from the CI job to the place where all the configuration for the controller manager under test is initialized.
Why?

for consistency with all the other settings: https://github.com/Kong/kubernetes-ingress-controller/issues/1182
because the KONG_ prefix is reserved for Gateway; KIC uses CONTROLLER_

Also note that there is a bug:
the integration test configures the logger the following way
https://github.com/Kong/kubernetes-ingress-controller/blob/07ae1516d0ac17efc6afdf9ab5d95c42c31b12a8/railgun/test/integration/suite_test.go#L250-L252
but the logger initialization code in manager/run.go disregards --log-level and --log-format if --log-reduce-redundancy is in use. This has been the case before this PR and this PR preserves preexisting behavior.

This PR basically takes on part of the scope of #1406, just FYI. But since we already have a PR we might as well include it.

I was under impression that #1406 was about cleaning up the integration test. This setting lives in the live production code (and not in tests) and has been added after that issue was created, so we've had a bit of a misunderstanding here.


Amazon Linux package

Package ruby-json-1.4.6-1.el6.x86_64 is obsoleted by rubygem18-json-1.8.2-1.39.amzn1.x86_64 which is already installed
Nothing to do

Not needed on Amazon linux, default install works

Fixed


Fix white texture in the middle of the screen

What happened?

It happened in  Wilderness.  I wanted to play with  NPC  but when pressed on the entrance a white square appeared and does not disappear
Steps to reproduce
go to the game with  NPC  and click on the entrance and that's it
Operating System
Windows 10  (10.0.19044) 64bit
Wallet
Metamask
CPU
AMD Ryzen 7 2700 Eight-Core Processor
GPU
Radeon RX 580 Series 8170MB
RAM
16334MB
Kernel Version
1.0.0-2673696742.commit-535b3fa
Unity Version
desktop-1.0.481-20220714181434.commit-d37208d
Realm
dg
NameTag
Cooper#8a81
Acceptance Criteria

[ ] No white texture shown

If you get out of that scene, does the white square disappear?

no will not disappear ,  only if you completely exit the game

Hello @Cooper8555!
I gave a look at the issue, but it seems that portals are no longer accessible by clicking on them. Now you access them by going through them directly.
I was not able to reproduce the error and get the white square. Is this still happening to you?

no, it's all good
вт, 30 авг. 2022 г., 23:03 Juan Ignacio Molteni @.***>:

Hello @Cooper8555 https://github.com/Cooper8555!
I gave a look at the issue, but it seems that portals are no longer
accessible by clicking on them. Now you access them by going through them
directly.
I was not able to reproduce the error and get the white square. Is this
still happening to you?
—
Reply to this email directly, view it on GitHub
https://github.com/decentraland/explorer-desktop/issues/359#issuecomment-1232109640,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2D4AZPSKYDQC653VVBNTFTV3ZSJVANCNFSM54IQ7YEA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Great, closing this issue. Thank you for your support!


TS Server fatal error:  Debug Failure. False expression: getScriptInfo

❗❗❗ Please fill in the sections below to help us diagnose the issue ❗❗❗
TypeScript Version: 4.7.3
Steps to reproduce crash
1.
2.
3.
TS Server Log
❗Server logging disabled. To help us fix crashes like this, please enable logging by setting:
"typescript.tsserver.log": "verbose"

After enabling this setting, future crash reports will include the server log.
TS Server Error Stack
Server: semantic
Verbose Debug Information: scriptInfo for a file '/Users/xiedejun/code2022/newcity-safety-admin/src/__VLS_types.ts' Path: '/users/xiedejun/code2022/newcity-safety-admin/src/__vls_types.ts' / '/users/xiedejun/code2022/newcity-safety-admin/src/__vls_types.ts' is missing.
Error: Debug Failure. False expression: getScriptInfo
Verbose Debug Information: scriptInfo for a file '/Users/xiedejun/code2022/newcity-safety-admin/src/__VLS_types.ts' Path: '/users/xiedejun/code2022/newcity-safety-admin/src/__vls_types.ts' / '/users/xiedejun/code2022/newcity-safety-admin/src/__vls_types.ts' is missing.
    at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:169403:30
    at Object.map (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:444:29)
    at ConfiguredProject.Project.getScriptInfos (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:169401:27)
    at ProjectService.sendProjectTelemetry (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:172348:66)
    at ConfiguredProject.updateGraph (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:170698:37)
    at ProjectService.createLoadAndUpdateConfiguredProject (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:172423:25)
    at ProjectService.assignProjectToOpenedScriptInfo (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:173508:44)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:173848:97
    at Object.flatMap (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:508:25)
    at ProjectService.applyChangesInOpenFiles (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:173848:41)
    at Session.handlers.ts.Map.ts.getEntries._a.<computed> (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:174983:46)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:177172:88
    at IpcIOSession.Session.executeWithRequestId (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:177163:28)
    at IpcIOSession.Session.executeCommand (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:177172:33)
    at IpcIOSession.Session.onMessage (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:177198:35)
    at process.<anonymous> (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:179833:31)
    at process.emit (node:events:390:28)
    at emit (node:internal/child_process:917:12)
    at processTicksAndRejections (node:internal/process/task_queues:84:21)

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines.
Happy Coding!


Override Template Parameters not working for securestring on V2 (not fixed yet)

#5372
As I can't reopen this issue I would like to make a new one informing all that this is not fixed yet. I added a detailed post to the issue mentioned above here describing how to reproduce the issue we're facing.

@devedse The task is based on node, so PS commands like ConvertTo-SecureString are not recognized in this context, hence the major version change.
We have added this in the release notes:

The task now treats (ConvertTo-SecureString '$(artifactsLocationSasToken)' -AsPlainText -Force) as string where you have expected it to convert $(artifactsLocationSasToken) to a secure string. We have added help texts for secure strings explicitly in the pop up which opens when you click on ... next to Override template parameters.

Ok, to summarize this would be the solution then?:

Yes, that is correct. 👍

Another followup question. Is there a way to expose this issue to VSTS or Azure when the deployment fails? As the only error that you'll get back is a 403 without any context.

You can send your queries to<EMAIL_ADDRESS>the required team would be engaged based on your query eventually.

Sent them an email, let's we'll see an answer soon 😄

I got a response from the<EMAIL_ADDRESS>team with the request to create a UserVoice post:
https://visualstudio.uservoice.com/forums/330519-visual-studio-team-services/suggestions/34745740-deploying-with-arm-msdeploy-doesn-t-give-detaile


Made utils.tokenizePath() path separator agnostic

The extension wasn't working on my Windows computer, and I tracked down the error to an issue with how the path is parsed. Essentially, path.relative returns a string that uses platform-specific separators instead of just forward slash /, which caused the split along the path separator to break. I fixed it by replacing the hard-coded / char with path.sep.
Let me know if this breaks anything on Linux 😄

I actually posted this PR a little preemptively, tests are still hanging when I try to run them... Regardless, I might make another PR that fixes this if possible.

Tests no longer hang, it fully works on my machine


GEODE-5615 Capturing more data in the case of hangs or OOMEs

We are seeing a lot of hangs and OOMEs in DistributedTest all of a sudden. Unfortunately, we are not capturing the call stacks and the hprof files we need to diagnose the issue. These changes should help.

@smgoller @rhoughton-pivot


UNION_DISTINCT on Array of Docs sorts the keys

ArangoDB 3.3.2 AQL
I'm doing APPLY("UNION_DISTINCT", subjectArray) where subjectArray is an array of arrays of documents that look like:
{
id: doc._id,
type: "node",
t: xxxx,
d: KEEP(doc, "Type", "Job"),
c: xxxx,
fc: xxxx
}

After the UNION_DISTINCT the documents look like:
{
"c": xxxx,
"d": {
    "Job": 5,
    "Type": xxxx
},
"fc": xxxx,
"id": "9121.00",
"t": xxxx,
"type": "node"
}

I'm guessing this is an unavoidable side effect of UNION_DISTINCT but it is annoying. It also makes the new SORTED and SORTED_UNIQUE #4157 functions useless on the resulting array since the sort functions seem to depend on the key order for sorting and my sortable 'id' and 't' fields have been pushed to the bottom of the documents.

The SORTED and SORTED_UNIQUE AQL functions should sort the values inside an array. If the values are objects, the objects' keys and values will be taken into account for the comparisons, as described here.
The sort order of attributes inside objects is not defined nor guaranteed nor guaranteed to stay identical in ArangoDB. For ArangoDB {"a":1,"b":2} is the same as {"b":2,"a":1}. This is also in line with the description of "object" from json.org:

An object is an unordered set of name/value pairs.

Keeping the original attribute insertion order for objects in ArangoDB would make things more expensive, and thus we have avoided it from the very beginning.

Thanks @jsteemann


MacOSX ImportError: dlopen(/usr/local/lib/python3.5/site-packages/tensorflow/python/_pywrap_tensorflow.so, 10): Library not loaded: @rpath/libcudart.8.0.dylib

Installed tensorflow for python3.5 on Mac, try the testing code as follows:
import tensorflow as tf

hello = tf.constant("test")
sess = tf.Session()
print(sess.run(hello))

It is working when run it from the terminal. But if I run it from eclipse, will get this exception:
warning: Debugger speedups using cython not found. Run '"/usr/local/bin/python3.5" "/Users/leitian/.p2/pool/plugins/org.python.pydev_<IP_ADDRESS>701191708/pysrc/setup_cython.py" build_ext --inplace' to build.
pydev debugger: starting (pid: 978)
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/site-packages/tensorflow/python/__init__.py", line 49, in <module>
    from tensorflow.python import pywrap_tensorflow
  File "/usr/local/lib/python3.5/site-packages/tensorflow/python/pywrap_tensorflow.py", line 28, in <module>
    _pywrap_tensorflow = swig_import_helper()
  File "/usr/local/lib/python3.5/site-packages/tensorflow/python/pywrap_tensorflow.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow', fp, pathname, description)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/imp.py", line 242, in load_module
    return load_dynamic(name, filename, file)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/imp.py", line 342, in load_dynamic
    return _load(spec)
ImportError: dlopen(/usr/local/lib/python3.5/site-packages/tensorflow/python/_pywrap_tensorflow.so, 10): Library not loaded: @rpath/libcudart.8.0.dylib
  Referenced from: /usr/local/lib/python3.5/site-packages/tensorflow/python/_pywrap_tensorflow.so
  Reason: image not found

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/leitian/.p2/pool/plugins/org.python.pydev_<IP_ADDRESS>701191708/pysrc/pydevd.py", line 1537, in <module>
    globals = debugger.run(setup['file'], None, None, is_module)
  File "/Users/leitian/.p2/pool/plugins/org.python.pydev_<IP_ADDRESS>701191708/pysrc/pydevd.py", line 976, in run
    pydev_imports.execfile(file, globals, locals)  # execute the script
  File "/Users/leitian/.p2/pool/plugins/org.python.pydev_<IP_ADDRESS>701191708/pysrc/_pydev_imps/_pydev_execfile.py", line 25, in execfile
    exec(compile(contents+"\n", file, 'exec'), glob, loc)
  File "/Users/leitian/code/python/eclipse-ws/TestTF/Hello.py", line 1, in <module>
    import tensorflow as tf
  File "/usr/local/lib/python3.5/site-packages/tensorflow/__init__.py", line 24, in <module>
    from tensorflow.python import *
  File "/usr/local/lib/python3.5/site-packages/tensorflow/python/__init__.py", line 60, in <module>
    raise ImportError(msg)
ImportError: Traceback (most recent call last):
  File "/usr/local/lib/python3.5/site-packages/tensorflow/python/__init__.py", line 49, in <module>
    from tensorflow.python import pywrap_tensorflow
  File "/usr/local/lib/python3.5/site-packages/tensorflow/python/pywrap_tensorflow.py", line 28, in <module>
    _pywrap_tensorflow = swig_import_helper()
  File "/usr/local/lib/python3.5/site-packages/tensorflow/python/pywrap_tensorflow.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow', fp, pathname, description)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/imp.py", line 242, in load_module
    return load_dynamic(name, filename, file)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/imp.py", line 342, in load_dynamic
    return _load(spec)
ImportError: dlopen(/usr/local/lib/python3.5/site-packages/tensorflow/python/_pywrap_tensorflow.so, 10): Library not loaded: @rpath/libcudart.8.0.dylib
  Referenced from: /usr/local/lib/python3.5/site-packages/tensorflow/python/_pywrap_tensorflow.so
  Reason: image not found

Error importing tensorflow.  Unless you are using bazel,
you should not try to import tensorflow from its source directory;
please exit the tensorflow source tree, and relaunch your python interpreter
from there.

it sounds like your eclipse is not propagating the ld_library_path/dyld_library_path/path env vars correctly, this sounds like a good q to ask on stackoverflow rather than a bug in tensorflow itself

thanks @yaroslavvb , I put in stackoverflow, but not body answered me yet:
http://stackoverflow.com/questions/41949689/import-tensorflow-failed-in-eclipse
Just have a try here.


⚠ Processors has degraded performance

In 2ff547d, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 93 ms

Resolved: Processors performance has improved in 6d21ef3.


Introduce USDT trace points

Description
Add systemtap into the builder to allow defining USDT, and add one to capture events when a new process was discovered.
Checklist

[ ] Investigated and inspected CI test results
[ ] Updated documentation accordingly

Testing Performed
Local testing during perf evaluation. CI is sufficient.

Closed in favour of #1642


Handle 0x0 slot element

In the SimpleControls class, the size of the slot element is resized to the initAd size dimensions or the dimensions supplied in resizeAd.


[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you successfully run brew tests with your changes locally?

Not only have several maintainers suggested this is a good idea, including myself repeatedly, but at least a handful of users have said they wouldn't hate the idea so let's follow up better late than never.
Obvious test is obvious:

Huh. That failure did not happen locally.

Oh, I see why. Can I wrap it in a simple if OS.mac? or are people going to insist I write a separate macOS compatibility layer for the install command? 🙈.

One comment but otherwise agree with it.

or are people going to insist I write a separate macOS compatibility layer for the install command? 🙈.

Usually yes but in this case you can just pull into diagnostic instead; I don't think we need a separate set of "strict" checks.

Pull this logic into all_development_tools_checks instead to avoid duplication and using OS.mac? here.

Can you clarify this a bit?
I didn't lump the Xcode/CLT checks into all_development_tools_checks because all_development_tools_checks also includes a check as to whether you're running an unsupported version of macOS, so making that section a hard block would stop people running install on 10.5-10.8 and 10.12.
Either that or we throw the blob into all_development_tools_checks but don't make it fatal, retaining the oppo and hope people actually notice it/take it seriously.
I presume I'm misunderstanding how simple you want this, so thought I'd just ask 😅.

I missed the odie difference 👍
Just want to avoid have if OS::Mac.prerelease? or if MacOS. anywhere outside Library/Homebrew/extend/os/mac now. It would be good to instead move as much of this logic as possible to a method in Library/Homebrew/extend/os/mac/diagnostic.rb which could also do the odie and have a no-op in diagnostic.rb. e.g. have a Diagnostic::Checks. check_development_tools method that's a no-op for non-OS X or consider if this could/should live in DevelopmentTools instead.


Content: replace \n\n with \n\n\n to make paragraph breaks appear correctly

ACs: All content is updated so that where there is currently a \n\n to indicate a paragraph break, there will be a \n\n\n.
When these changes are committed to the content repo and not before, notify volunteers to start using \n\n for line breaks and \n\n\n for paragraph breaks.
See https://stackoverflow.com/questions/26228060/use-tr-to-remove-line-breaks-in-multiple-files/26228083#26228083 for a possible approach

We changed our formatting approach, this work is no longer required


Unlicense alternative license?

Maybe we could use The Unlicense so that way anyone can use this template freely

That feels redundant to the CC0 license that already exists to me.

That feels redundant to the CC0 license that already exists to me.

Oh sorry, didn't see that, thank you <3


cypress-configuration breaks with custom registry

I ran into an issue with cypress-configuration when pointing my .npmrc registry value to an internal Nexus registry with strict-ssl=false (also in .npmrc).  When doing this, I got:
npm ERR! code 1
npm ERR! path /home/jenkins/agent/workspace/_612-use-nexus-cache-in-frontend/uiTests/node_modules/@badeball/cypress-configuration/node_modules/esbuild
npm ERR! command failed
npm ERR! command sh -c node install.js
npm ERR! Trying to install "esbuild-linux-64" using npm
npm ERR! Failed to install "esbuild-linux-64" using npm: Command failed: npm install --loglevel=error --prefer-offline --no-audit --progress=false<EMAIL_ADDRESS>npm ERR! npm ERR! code DEPTH_ZERO_SELF_SIGNED_CERT
npm ERR! npm ERR! errno DEPTH_ZERO_SELF_SIGNED_CERT
npm ERR! npm ERR! request to https://<internal.ip.address.here>/nexus/repository/npm-public/esbuild-linux-64 failed, reason: self signed certificate
npm ERR!
npm ERR! npm ERR!  Trying to download "https://registry.npmjs.org/esbuild-linux-64/-/esbuild-linux-64-0.11.23.tgz"
npm ERR! Failed to download "https://registry.npmjs.org/esbuild-linux-64/-/esbuild-linux-64-0.11.23.tgz": read ECONNRESET
npm ERR! Install unsuccessful

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/build/.npm/_logs/2022-08-01T14_12_32_756Z-debug-0.log

After a brief investigation, I stumbled across https://github.com/evanw/esbuild/issues/1485, which leads to https://github.com/evanw/esbuild/pull/1621.
TL;DR: I think the fix may just be a version bump to esbuild.

Confirmed that using:
"overrides": {
    "esbuild@<14": "^0.14.51",
    "esbuild-linux-64@<14": "0.14.51"
  }

in package.json when using a package that uses @badeball/cypress-configuration as a dependency solves this issue.


IsEnabled does not work properly on HamburgerMenuItem

Bug Description
Is not possible to enable or disable a HamburgerMenuItem inside a .xaml file.
If a HamburgerMenuItem contains an IsEnabled attribute, does not matter if it is True or False, binded value or not - it always shows as disabled.
To Reproduce
Based on punker76/code-samples:

Open HamburgerMenuApp.V2
Find a HamburgerMenuGlyphItem in MainWindow.xaml
Add IsEnabled="True"

Expected behavior
Menu item should still be enabled.
Environment(please complete the following information):
Based on punker76/code-samples:

OS: Win10 1803
VS 2017 15.7.3

Based on my own code:

MahApps.Metro updated
OS: Win10 1803
VS 2017 15.7.3
.NET Framework 4.5

Repo
Since it can be simulated on punker76/code-samples, i think it is not needed.
Additional context
Based on Issue #2865, and Pull Request #2921 it is expected to work.

same here. i changed IsEnabled via C# code but if i disabled a menuitem then i cant enable it again using IsEnabled = True :(

I don't think that this is solved as I just cloned the current develop branch and the problem is still there. I'm pretty sure that the problem is caused by this line. In this void, CanExecute is set to true if the Command is null. The problem is, that on creation the Command is null but CanExecute is false which is a wrong state. I will submit a PR that changes this line to  private bool canExecute = true;

Im currently stuck on that IsEnabled thing. Is there any code example of how to enable and disable a menuitem in a hamburger menu?

@Hanebu If you use the v2.0 alpha then this property should work as expected.

Awesome, I will test this right now :-) Thanks!


view - tab_2_dropdown.py

please implement the drop-down bar to use with chart 2, in tab 2

see commit c8381d2, drop downs were created inside tabs.


Multiple scroll bars displaying

[ x] Bug
Since the v3 update scroll bars now display within the page because the length of the web part seems to be fixed. This means any long pages will have two scroll bars, one for the overall page, one in the nested web part. Please can this be resolved so that only one scroll bar displays

The size of the iFrame is set to the size of its internal content in the case of a support.office.com and a custom SharePoint page, however, docs.microsoft.com pages still do not support this setting so the iFrame height is set to 6000px.  In testing, where the iFrame size could be set, the scroll bars don't appear and we haven't found any problems with this, if you could provide more detail we can look into it further.

I have a custom page and playlist in the standard SharePoint subcategory, the white content bit is approx 2300px height. When we had the previous version v2 I remember seeing scroll bars display whilst all the page content was loading, then once fully loaded they would disappear. In v3 they seem to stay, same in both Chrome and Edge, see below.

For info the standard page for Yammer on https://support.office.com/ displays normally with just the one scroll bar but does cut off the last step,

Does the custom page by chance have multiple columns? There is a bug with this because the reported size of the page is incorrect on load until the page in the iFrame has a chance to respond to the width of the frame it's in.  This is an unfortunate side effect of how Microsoft has designed responsiveness into their pages and we are looking into ways we can handle this with as little overhead to the page performance as possible.

So, yes it did have columns, but i have now removed them so the page is sections of text, images and stream video and refreshed custom admin. The updated page pulls through but the double scrolls still remain.


quick fix languages bundle

Codecov Report

Merging #1691 (e27c431) into main (44c5c87) will increase coverage by 0.12%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1691      +/-   ##
==========================================
+ Coverage   48.09%   48.21%   +0.12%
==========================================
  Files          47       47
  Lines        7047     7047
==========================================
+ Hits         3389     3398       +9
+ Misses       3658     3649       -9

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


Can't map Integer to i32 while using Datetime

#2260  Setup
Versions

Rust:1.42.0
Diesel:1.4.4
Database:5.7.27
Operating System:macOS 10.14.6

Feature Flags

diesel = { version = "*", features = ["mysql", "r2d2", "chrono", "bigdecimal"] }

Problem Description
If table struct contains chrono::NaiveDateTime, then the struct can't use i32/i64 data type on other fields.
1. both datetime and i32: Error
error[E0277]: the trait bound `i32: diesel::deserialize::FromSql<diesel::sql_types::Datetime, diesel::mysql::Mysql>` is not satisfied

the trait `diesel::deserialize::FromSql<diesel::sql_types::Integer, diesel::mysql::Mysql>` is not implemented for `chrono::naive::datetime::NaiveDateTime

// src/schema.rs
table! {
    orders (id) {
        id -> Unsigned<Bigint>,
        created_at -> Datetime,
        state -> Integer,
    }
}
// src/models/order.rs
#[derive(Debug, Queryable)]
pub struct Order {
  pub id: u64,
  pub state: i32,
  pub created_at: chrono::NaiveDateTime,
}

pub fn read_orders(conn: &MysqlConnection) -> Vec<Order> {
    use super::super::schema::orders::dsl::*;
    orders.limit(5).load::<Order>(conn).expect("Error")
}

2. only id(i32) and created_at(Datetime): No Errors
This case is similar to examples/mysql/all_about_inserts
// src/schema.rs
table! {
    orders (id) {
        id -> Integer,
        created_at -> Datetime
    }
}
// src/models/order.rs
#[derive(Debug, Queryable)]
pub struct Order {
  pub id: i32,
  pub created_at: chrono::NaiveDateTime,
}

But if I add an extra i32 filed, it would get errors.
3. only id+state(i32): No Errors
// src/schema.rs
table! {
    orders (id) {
        id -> Unsigned<Bigint>,
        //created_at -> Datetime,
        state -> Integer,
    }
}
// src/models/order.rs
#[derive(Debug, Queryable)]
pub struct Order {
  pub id: u64,
  pub state: i32,
  //pub created_at: chrono::NaiveDateTime,
}

4. only Datetime: No Errors
// src/schema.rs
table! {
    orders (id) {
        id -> Unsigned<Bigint>,
        created_at -> Datetime,
        //state -> Integer,
    }
}
// src/models/order.rs
#[derive(Debug, Queryable)]
pub struct Order {
  pub id: u64,
  //pub state: i32,
  pub created_at: chrono::NaiveDateTime,
}

What are you trying to accomplish?
I want to use both i32 and chrono::NaiveDateTime types in my struct.
Checklist

[x] I have already looked over the issue tracker for similar issues.
[x] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)
[x] I have read diesel/examples/mysql

I found the reason.
The fields in shcema and model must in same order.
// Ok
table! {
    orders (id) {
        id -> Unsigned<Bigint>,
        created_at -> Datetime,
        state -> Integer,
    }
}

#[derive(Debug, Queryable)]
pub struct Order {
  pub id: u64,
  pub created_at: chrono::NaiveDateTime,
  pub state: i32,
}

If I switch "pub created_at" line number and "pub state" line number, it would get errors.


ENH: update package version to 1.0.0

Fix https://github.com/conda-forge/gymnasium-feedstock/issues/39 .
Fix https://github.com/conda-forge/gymnasium-feedstock/issues/33 .
Fix https://github.com/conda-forge/gymnasium-feedstock/pull/36 .
Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #39.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #39

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

Use importlib-metadata instead of importlib_metadata
Recipes should usually depend on matplotlib-base as opposed to matplotlib so that runtime environments do not require large packages like qt.
No valid build backend found for Python recipe for package gymnasium using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.

pip check is failing on many os with:
+ pip check
proglog 0.1.9 is not supported on this platform


segmentation fault

Using latest version (1.0.11)
const libXml = require('libxmljs')
const xmlString = `<?xml version="1.0" encoding="UTF-8"?>
<imsx_POXEnvelopeResponse xmlns="http://www.imsglobal.org/services/ltiv1p1/xsd/imsoms_v1p0">
  <imsx_POXHeader>
    <imsx_POXResponseHeaderInfo>
      <imsx_version>V1.0</imsx_version>
      <imsx_messageIdentifier/>
      <imsx_statusInfo>
        <imsx_codeMajor>success</imsx_codeMajor>
        <imsx_severity>status</imsx_severity>
        <imsx_description/>
        <imsx_messageRefIdentifier>ff119fad-1bdd-41af-b908-e7665abc4b28</imsx_messageRefIdentifier>
        <imsx_operationRefIdentifier>replaceResult</imsx_operationRefIdentifier>

      </imsx_statusInfo>
    </imsx_POXResponseHeaderInfo>
  </imsx_POXHeader>
  <imsx_POXBody><replaceResultResponse/></imsx_POXBody>
</imsx_POXEnvelopeResponse>`

const xml = libXml.parseXml(xmlString)

xml.namespaces()

Causes node to exit with a segmentation fault, however adding in a xml.root() before the xml.namespaces() fixes the issue (I think!).
It's not 100% reproducible, but I could do it 8 out of 10 times (just by pasting in the above into the node REPL).

Ah interesting, I'm also seeing it more consistently in the cloud (on Google Cloud Run which I think is eventually back internally by K8?), my Docker image is node:18-alpine which (at the time of reproduction) resolves to 18.19.0-alpine3.19.

Same here... 100% failure resulting in core dump (ubuntu 22.04, AMD/Intel).   Also seeing numerous other oddities (e.g. element.attrs() only enumerating a single attribute when there are multiple)

MIs there any fix available for the above issue?

I am having the same issue. The only thing that fixes it for me is passing true as the first parameter (eg xml.namespaces(true) )

Hello all, I saw that https://github.com/libxmljs/libxmljs/pull/655 was merged a couple of weeks ago. Is this issue now fixed? I pulled the latest version and it's still broken.

Hello all, I saw that #655 was merged a couple of weeks ago. Is this issue now fixed? I pulled the latest version and it's still broken.

It wasn't merged. I closed the pull request. This project no longer appears to be maintained.


404 requests never get back on promisse

Every time I try a email address that don't exists, nether then or catch are fired. It just get stuck on the promise. I'm using v2.person.lookup method.
If I try the curl call I got:
 "status": 404,
  "message": "Searched within last 24 hours. No results found for this Id.",
  "requestId": "fa9a5f16-bbf3-4ac0-abf4-525425bde9f5"

I'm missing something or it's a bug?

Ops. It's my fault. Sorry.


Update http gem dependency to version 3.3

During the usage of this gem I've decided to record a response with VCR and test it with RSpec. However I've encountered a strange issue: the spec took to long to finish and after that I got a request timeout. Outside of VCR it works fine and gets a proper response.
  response:
    status:
      code: 408
      message: Request Timeout

Other people have this issue too, it is described in VCR repository https://github.com/vcr/vcr/issues/702.
I thought that it is related to http.rb gem and updated the dependency to actual version in this PR. It helps me to solve the problem and now twitter gem works like a charm even with VCR.

It's important to mention, that http gem 3.3 has dropped support of old ruby versions.
According to the readme it supports:
Ruby 2.3.x
Ruby 2.4.x
Ruby 2.5.x
JRuby 9.1.x.x


segment control?

Please explain your feature request in a one or two sentences.

wondering if it would be possible to add segment control for the RGBIC devices. Love that option in the Govee app. but within HomeKit it if you touch any of the color options it just lights up the device as that one color etc.

Is your feature request related to a problem? Please describe.

Any particular Govee devices that this relates to?

H702C, H6076, H6046

Anything else?

hi @grizzalorian
you’re not the only one to have requested this!
the issue is that the home app does not have any way of showing some interface for setting different parts of a light strip to different colours

ah fair enough.  I saw another plugin (Govee ultimate) but I could never get it to work. (yours works flawlessly)
Guess I'll continue to use the Govee app to set those for now.

https://github.com/bwp91/homebridge-govee/wiki/Scene%2C-Music%2C-DIY-Modes
Could be useful for using the home app to turn into scenes that you have already made in the govee app?
Appreciate it's not quite what this issue was aimed at, but perhaps some kind of alternative?

what about splitting each segment up into individual home lights? it could be used to at least add some more control? for example, I have the govee desk light strip and have each side of my desk a different color based on the color of light on that slide for better ambiance. It would be handy to be able to do this in homekit if each segment could be set as its own light? I know it's tedious, but the best thing is we have given homekits limitations.


bugfix(TDI-38272) : fix NPE when connection is closed

Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[-] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
NPE when connection is closed
What is the new behavior?
Error message is more informative
Does this PR introduce a breaking change?

[] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:
https://jira.talendforge.org/browse/TDI-38272

Pull request test result: success :star:


Add "manual" type documentation

We should setup Sphinx in a similar way as for Jannovar.

Commit 4c93f77 has some docs, should be fleshed out for v0.2.


relax pip dependencies

Motivation
What was the reasoning behind this change? Please explain the changes briefly.
How to test the behavior?
Show here how to reproduce the new behavior (can be a bug fix or a new feature)

Checklist

[ ] Have you checked our Contributing document?
[ ] Have you ensured the PR description clearly describes the problem and solutions?
[ ] Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
[ ] If this PR fixes an issue, is the first line of the PR description fix #XXX where XXX is the issue number?

@bendichter Looks good, made some minor comments and opened a branched PR for some file naming changes; let me know what you think about loose+lazy vs. strict+full install

Also keep in mind it's on the eventual to-do list to remove modality-specific things like spikeextractors/roiextractors to make the minimal install even lazier since people that want to use the repo for ophys have no need to ever install spikeinterface.
I also just confirmed the sister PR to this one successfully installs on 3.7 as well as 3.9.

Merging once tests finish, then. Thanks for helping with this


Weather Support only available for New Delhi, India

The weather support right now is only available for New Delhi, India at the moment because the code is not fully ready for this functionality. This will be resolved as soon as I find a way to do so😂.

Issue Resolved
The issue with the weather command only working for New Delhi, India has been resolved in the latest v2.0 'Beefing Up'. Please update your code accordingly.


Logging configuration confirmation popup textbox includes a license header

Changing the log level of a class results in a popup text box with an xml text including a license header as follows
Affected version carbon kernel 4.4.15

The logging UI is removed in the 4.5.0 kernel


Removed miri failure.

ccing @roee88 , which actually found the solution to this in https://github.com/apache/arrow-rs/pull/323


Release Google.Cloud.Logging.NLog version 3.3.0

Changes in this release:

Commit 9ed450c: refactor: Extract RenderJsonPayload from BuildLogEntry in GoogleStackdriverTarget
Commit f553511: docs: Add documentation for trace related config options in GoogleStackdriverTarget
Commit d94c27b: docs: Clarify XML doc for GoogleStackdriverTarget trace related options
Commit 0da1f45: test: Add more unit tests around log and trace correlation
Commits a106579 and 858f161: feat: Add support for trace information to be included with log entries

Created release for Google.Cloud.Logging.NLog-3.3.0


[PLATFORM]: runtime bug

Hey there,
I believe this is a new issue, I haven't noticed this happening before.
I have the latest build, Version 2021.3637 (2021.2.3)
I am on Mac running Catalina, 10.15.7
If I now make a typo in one of my image names, say incorrectly calling it "my_image.pngg" instead of "my_image.png",
this now displays an ERROR: Runtime Error in the Console, which doesn't stop running until I fix the typo.
It should definitely cause an error, but in the past it would just display a single error in the Console, and essentially halt the Simulator.
Now it keeps running, so by the time I find and fix the typo, when I come back to the Console, there can be like 10,000 error messages.
[EDIT] This could be Timer related? I just made a empty project with a call to a missing image, and it simply displayed a warning that the image could not be found. So it must be deeper in my code where it is loading in images with a Timer. I'll keep digging.
This certainly could be something on my end, but I don't think there have been any changes in my setup.
Let me know if this doesn't make sense or if you need clarification.
Joe

Well, there's no code to look at, but this is probably the same as this: https://github.com/coronalabs/framework-timer/issues/6.
I've written a fix for this and one other issue and I'll be pushing making a PR for @Shchvova to review either late tonight or tomorrow noon, depending on what time I get home today.

Hi,
how is this fix progressing?
I also found out problem with that bug in versions after fixing pausing timers in their callbacks.
Timer doesn't get removed from the list because callback crashes whole loop so it just repeats every enterFrame and loops infinitely.
Before that bug you just got one runtime error and app continued to work but now the app just stops working.
Can we please get this fix in the next build of corona? We have apps to update but currently can't use any of the corona builds after 14.8.2019 because of this or the pause bug in timers.
Thank you.

We are actively working on this issue.


Register to vote farcaster frame

Links for testing the development frame:

https://debugger.framesjs.org/?url=https%3A%2F%2Fswc-web-git-tay-frame-stand-with-crypto.vercel.app%2Fframes%2Fregister-to-vote
https://warpcast.com/~/developers/frames?url=https%3A%2F%2Fswc-web-git-tay-frame-stand-with-crypto.vercel.app%2Fframes%2Fregister-to-vote


Look to see if there are phase variations

My eyes might be playing tricks on me but are there ellipsoidal variations?

What does an ellipsoidal variation look like?

flux is highest at 0.25 and 0.75 phase.

But I don't see any evidence for that in the folded lightcurve.  Do you?

My eyes were probably fooling me. Can you shrink the y axis to zoom in on the our of transit data

There's a zoom at the bottom of the notebook now:
https://github.com/barentsen/kepler-athenaeum-tutorial/blob/master/how-to-find-a-planet-tutorial.ipynb

I mean zoom further, to 0.002. Maybe what I'm seeing is a secondary eclipse


Possible ambiguous saved scenario history

It is possible to edit a scenario_id that's part of a saved scenario's history and set that scenario_id as the 'active' scenario when loading a saved scenario. In this case the scenario_id will be both in the scenario history as being the active scenario in the returned information.
Once I again update the saved scenario by referring to a new scenario_id, the scenario_id that was already present in the history will not change position in the scenario_id_history list. I would have expected that the order of the scenario_id_history list would at least have changed, making the history a bit ambiguous.
I would expect the API to refuse setting to update an saved scenario with a scenario_id that is already present in the scenario_id_history to prevent this behaviour from happing. You could than revert your scenario to an point in time by setting a copy of the point in time as the active scenario_id.

@noracato is this one to reopen or is this addressed elsewhere?

Yes, it closed itself while I was on holiday 😄

Fix is now live


Has anyone got documentation on running the Teams Recording Bot from amendments to building and testing?

Hey,
Trying to implement a compliance teams bot that just enables recording and transcription automatically when joining a call.
Anyone got any guides or documentation for a novice in Azure and C#?
Any help would be really appreciated
Best wishes,
J

For clarity, do you want to create convenience recordings, or compliance recordings?

Hi mate,
I want to create a bot that 1. joins any meeting that is live / active. 2. Displays a banner about compliance "Calls are recorded blah blah", 3. turns on auto recording options in teams along with the transcribe function automatically and then leaves the meeting.
Does that make sense?

Hey @jelpsIT ,
I understand what you are trying to do. My question was regarding the reason you want to do it?
If you need the recordings for compliance purposes, your bot instance itself needs to subscribe to the audio and video sockets to record that data.
The recordings that are generated using the "Record & Transcribe" feature in the Teams client are called "Convenience Recording", and do not cover what legislation usually requires. Any user in the meeting can stop the recording, for example.
Compliance Policies usually also check the "RequiredDuringCall" option, which will disconnect users under policy, when the bot call leg who is supposed to observe the participant leaves the call.
The record & transcribe feature can only be used by real humans using the Teams client. The person clicking the button becomes owner of the recording and will share it depending on tenant settings to the participants of the meeting.
I think I understand you try to make a shortcut by offloading the recording and storage step to the in-built convenience recording bot from Microsoft, but sadly this won't work.

Hi mate,
Thank you for that clarification.
At the moment we are getting our users to enable recording, which then goes to their onedrive and they then push this to a sharepoint folder. I understand the issues that could arise from that, so ideally I want to create something like this: https://github.com/shawnlien/CustomComplianceRecordingBot/blob/main/Wiki/DeploymentGuide.md
So apologies for misunderstanding, It would be a ComplianceBot, do you know if there is a way for a bot to auto force / trigger the scheduled of the call from our org to enable it via MS graph?
Is that possible?
Essentially, I've setup compliance tags and have setup settings, but the bot deployment and testing is what I am struggling with.
I know the fundamentals of C# from College and Computer Science degree but in terms of MS API's and using Azure deployments I am very new. I have a good understanding of most of it but I'm trying to apply it as simply as possible to essentially just allow a bot to manage users calls so they are auto recorded rather than building a bot that provides a recording (via audio and video sockets) If that makes sense?
If it's not possible then at least I know.
But this looks like something that would be perfect: https://github.com/shawnlien/CustomComplianceRecordingBot/blob/main/Wiki/DeploymentGuide.md as it pushes the recordings to Sharepoint?
Let me know your thoughts and constraints on this?
Appreciate your advice.

The documentation in this repo should be clear enough to get a test environment up and running. I recommend the AKS sample as it is built on AKS and will be far more modern. The classic cloud app (extended support) platform is really old and it's becoming increasingly hard to find relevant resources. Apart from legacy systems, nobody uses these anymore.
All you have to do is implement the Sharepoint part.
I usually try to offer solutions via GitHub issues, but if you continue to have problems that are too general to be answered effectively via GitHub issues, then I can also offer consulting. I've been developing and operating globally distributed production-grade compliance bots for over 2 years + 2 yearsexperience with local-media bots in general. My details are in my profile.


nixos/nixpkgs: fix determination for cross-compiled nixos system

Description of changes
Since the output of lib.systems.elaborate contains functions, an equality check with == does not suffice, lib.systems.equals should be used instead.
Fixes #278001

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

We have three solutions to this problem

Canonicalize all pairs so that if they match, we use only one of the values in both places. Works well for Nixpkgs and its implementation.
Remove the functions.
Don't use ==.

2 is in progress IIRC, but is a slow deprecation. You're doing 3.
1 is a bit more effective though, because it fixes all comparisons of nixpkgs.*Platform.
Could you try add something like this to line 211?
- apply = lib.systems.elaborate;
+ apply = inputBuildPlatform:
+   let elaborated = lib.systems.elaborate inputBuildPlatform;
+   in if lib.systems.equals elaborated cfg.hostPlatform
+     then cfg.hostPlatform  # make identical, so that `==` equality works; see https://github.com/NixOS/nixpkgs/issues/278001
+     else elaborated;

(Perhaps relevant are also https://github.com/NixOS/nixpkgs/pull/237512, https://github.com/NixOS/nixpkgs/pull/254763 and related PRs/issues, if you haven't seen them)
That all said, all other modules shouldn't read from anything other than the pkgs module argument, which is the single most reliable source of truth. Unfortunately we can't enforce that; would be simpler.

@Mindavi

Where did this isCross come from? 😅. It's way too vague like this...

From me. I don't think it is vague.

What about other cross types, like Canadian cross?

Nixpkgs is not a compiler, so Canadian cross does not apply to it. Its configuration only has two platforms: build and host; no target.
Individual packages may well support a Canadian cross, but this isn't about individual packages.

@roberth that makes sense, and is also more forgiving to potential usage of == for comparing these platforms, applied your recommendation


Add domain email.vccs.edu

School Name: Virginia's Community Colleges
Website: http://www.vccs.edu
The domain is: @email.vccs.edu

@khaile This domain is blacklisted because the college allows to register an email address there even if you are not a student of the college. If you are the student of this college, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).


New Sails cli

I propose to create a new Sails CLI and make it a separate module NPM, inherit from Gulp. This will quickly write new features and modules for it.

Hrm, implications on this.  SailsJs creates a quasi-executable that allows us to create new Sails projects in a flash.  Do you really want to manage:

SailsJS versions
SailsJS executables

What happens when you run:
sails new SomeCoolProject
and you get an earlier release because you failed to update the executable or path despite the modules installed. For this to work, it would have to learn the current version first and do all sorts of hand-holding and dependency management for older releases.
My idea, let the SailsJS guys do what they are best (forward progress) with and trust that they give the rock-solid solutions that they have been for forward-thinking development.

CLI is a development tool. In my understanding correct its architecture should not be in production.

@crh3675 thanks for the vote of confidence :)
@listepo I appreciate your point here, and it's something I've considered a lot ever since @colinwren suggested it years ago.  In reality, it just has never been a priority because it hasn't caused any major issues and doesn't really take that much longer to install.  Hooks, however, are a different story...
A big goal for Sails v1 is providing a clean way to install the smallest-possible version of core without hooks.  And you're right, the CLI is absolutely a development tool that doesn't necessarily need to be installed on the remote machine.  However, even then, it's quite light compared to e.g. the orm hook.  So our current plan is to focus on hooks first.  Depending on where we end up on that timeline-wise, maybe we can also plan on breaking out the sails CLI from the core install down the road.

@listepo @crh3675 created the sails-cli and sails-core packages as placeholders (mainly to prevent having to bother isaacs about it later in case anyone tries to homestead the package names -_- )
However, I was also curious which dependencies would actually be pulled out.  The package.json files have a list, but it's definitely incomplete (since right now the CLI requires stuff from the lib/ directory in core).  I'd suggest it's not worth spending any more time exploring that at the moment for the reasons mentioned above.
However, to finish this up, since I can't find more documentation of the approach Colin and I discussed anywhere (I think it may have been mainly over hangouts), here's a summary:
Summary
npm install sails
|_ sails-cli
|_ sails-core

Whether or not these pieces actually end up in different repos (the experience @sgress454 and I have had with generators leads me to believe this might actually be a bad idea) they would be different packages.  This way, both npm install -g sails and npm install sails would continue to do what they've always done, but you could also install sails-core directly (you wouldn't ever install sails-cli directly).
Problems
Looking back, I don't think this would actually be the best solution.  For one thing, consolidating the  the more we can consolidate


Fix include rev flatten

Submission Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
The flattening of rev happening in stan-dev/math#1593 replaces stan/math/rev/mat.hpp with stan/math/rev.hpp, so this is required alongside those changes.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Rok Češnovar
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

It's failing because of stan::math::fill, can't really work out why.


Update author name

Update author name.

Hi @aungoo-msft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (aungoo). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@aungoo-msft : Thanks for your contribution to the Azure documentation! The author, @aungoo-msft, has been notified to review your proposed change.

#sign-off

Closing - please use private repo.


MM-46597:  Unreads button in channel sidebar is missing alt text for screen readers

Summary
Unreads button in channel sidebar is missing alt text for screen readers
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-46597
Related Pull Requests
NONE
Screenshots
https://user-images.githubusercontent.com/102028591/203821723-f16cbb3f-0d0b-489c-b951-03128195a158.mp4
Release Note

Unreads button in channel sidebar is missing alt text for screen readers

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

@tboulis can you please re-review the changes.

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

Thanks @babinderrathi
Tested, looks good to merge.
Verified LHS unreads filter button is read by the screen reader.
E2E report - no PR related failures.

/cherry-pick release-7.6

Cherry pick is scheduled.


Added in set-xterm-title and xterm-title config variables.

Closes #1859

Coverage decreased (-0.01%) to 59.142% when pulling 32337cba6fef19eaee5d9cfed7855ea8fadc289a on aflyingpumpkin:master into 8963a85b7382daeee74043b86b423739a711cde9 on newsboat:master.

Thanks for the PR! Seeing that these new options only set the title at the start of the program, I wonder if the options are necessary at all. One can simply do:
echo "\033]0;TITLE\a" && newsboat

to get the same effect. Are there any downsides to this solution?

The main downside of an approach like that is that the user has to:
A: Setup an alias in order to automatically do this.
B Type out the command in full to do it.
A better solution would probably be to add a way to execute shell commands inside of the newsboat config, as well as while actually inside of newsboat. Maybe using something similar to how vim has !

Creating an alias doesn't sound like the end of the world. It's certainly better for the user to learn about aliases rather than some settings that are unique to Newsboat; the former can be applied to other situations, and the latter can't.
I also don't think that this use-case is a compelling reason to add anything like !. If the problem can be solved outside of Newsboat, let it be solved there.
Thanks for the suggestion and the PR, but I really do think this is outside of the program's scope, so I'm closing this (and the associated issue).


api gateway {parameter_n} doesn't work with Requests Path cache

For some reason I can't enable API Gateway cache when having the Requests Path as {parameter_n}. These params were created by zappa.
Whenever I click on Caching checkbox, AWS gives that error message.
http://g.recordit.co/ShfT242WpM.gif

Hm, not sure about that one. What's the expected behavior? Does it work with a different format?

@Miserlou yea it works normally if I create using different format (manually from UI)
I chatted with AWS support, exported the Swagger json after Zappa, the original Zappa config gives something like this:
"/{parameter_1}": {
      "get": {
        "consumes": [
          "application/json",
          "multipart/form-data",
          "application/x-www-form-urlencoded"
        ],
        "produces": [
          "text/html"
        ],

AWS support said we need to add something like this after it
"parameters": [
          {
            "name": "parameter_1",
            "in": "path",
            "required": true,
            "type": "string"
          }
        ],

I'll play with it later and see if that helps.

I suppose that would go here? https://github.com/Miserlou/Zappa/blob/master/zappa/zappa.py#L663

looks like it, I'll play with it to see if that'll fix it.

We no longer use 'parameter_n'


Added node.js sample for assistant skills

Add sample code for assistant skills implemented for Node.js.
The current problem is that the value returned by assistantSkillTrigger does not seem to be used correctly. This may be a problem with the bind settings or with the extension.

@shibayan , assistant skills feature has also moved to Microsoft repo - https://github.com/Azure/azure-functions-openai-extension. Please migrate this PR or make more contributions directly there.


index: handle computing md5s separately from saving

Avoids overloading save(). upload_odb will also belong here, since it is a caching side-effect of computing hashes.

Codecov Report
Base: 58.82% // Head: 58.35% // Decreases project coverage by -0.46% :warning:

Coverage data is based on head (1f04aae) compared to base (84b7d4b).
Patch coverage: 91.30% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #177      +/-   ##
==========================================
- Coverage   58.82%   58.35%   -0.47%
==========================================
  Files          40       40
  Lines        2499     2519      +20
  Branches      448      369      -79
==========================================
  Hits         1470     1470
- Misses        943      961      +18
- Partials       86       88       +2

Impacted Files
Coverage Δ

src/dvc_data/index.py
82.79% <80.00%> (+0.09%)
:arrow_up:

tests/test_index.py
100.00% <100.00%> (ø)

src/dvc_data/hashfile/build.py
59.67% <0.00%> (-15.33%)
:arrow_down:

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

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


Error Refreshing My Workflows if Only Logged in with Google

Bug Report
In develop branch, if you are only logged in with Google, refreshing your workflows causes an error to appear in UI.

Login with Google (only)
Go to My Workflows page. In case it's relevant, I had no workflows of my own, but I did have a workflow shared with me.
Click the refresh arrow, you get the following:

×
 The webservice encountered an error trying to create/modify.

[HTTP 400] Bad Request: No source control repository token found. Please link at least one source control repository token to your account.

Screenshot:

Hmmm, how did you manage to bypass https://github.com/ga4gh/dockstore/issues/1370#issuecomment-406726202 to test this out anyway?

I am able to log in with a Google account on both my local instance and on staging. I guess #1370 only happens in certain scenarios?

@garyluu were you able to replicate the problem before your fix for it?

Was not able to reproduce, my fix only updates the user profiles upon initial token creation.

It's still happening for me on staging. Is the fix deployed to staging yet?

No, but you can manually update the user profile in the accounts page.  If the problem still persists, then there may be something else that needs to be done.

I went to the Account page, clicked Sync with Google, and now see my profile picture (I didn't before). Just making sure that's what you meant.
After I do that, I still get the error refreshing workflows.

in that case, something else needs to be done.  I think the profile thing only fixes part of the permissions problem but not refresh all it seems.

I'll try to reproduce locally to see if I can catch it in the debugger

I can reproduce locally. I only have a Google token, which is not a source control token, per io.dockstore.webservice.core.TokenType.

Clicking on refresh invokes io.dockstore.webservice.resources.UserResource#refreshWorkflows
Which calls io.dockstore.webservice.resources.WorkflowResource#refreshStubWorkflowsForUser
Which then loops through all of the user's tokens, looking for a source control token.
Since my only token is not a source control token, it throws the exception at io/dockstore/webservice/resources/WorkflowResource.java:239.

Could fix by not throwing an exception, but I don't know if that would cause problems for other scenarios.

To be clear, I was mostly curious how you got past the login issue.
Setting that aside, I think the website could be more clear or helpful (disable refresh if the user has no sources for a refresh via token?) but the error message isn't the worst.

I think we should have a better error message, something along the lines of "error: link to a source control to get automatic workflows"

I still see the Refresh All button, and it is not disabled, when I am logged in with Google only. Clicking it causes the same error as in the description.

are you on develop?

I'm on staging.dockstore.org. Looks like I incorrectly assumed it had been deployed there.

Button disabled  on staging
DockstoreAPI - 1.5.0-beta.4
DockstoreUI - 2.2.0-beta.3


Make Option.ToString return string value of Some or blank

Option<T>.ToString() is currently implemented to return a string representation of Option<T>'s state when ideally it should simply return blank when None or Some(null) and the string representation of the value of Some otherwise. In other words:
        public override string ToString() =>
            hasValue && value != null ? value.ToString() : string.Empty;

Doing so will go a long way to help Option<T> to be easily substituted in code for Nullable<T> and references where one desires. The trouble with the current implementation is that any refactoring towards Option<T> will require one to thoroughly review all code where string formatting is used. For example, suppose the following code:
var d = (DateTime?) new DateTime(2010, 12, 31)
var s = string.Format(new CultureInfo("de-DE"), "{0:D}", d);
Console.WriteLine(s);

This prints:
Freitag, 31. Dezember 2010

However, if you use Option<T> instead:
var d = Option.Some(new DateTime(2010, 12, 31));
var s = string.Format(new CultureInfo("de-DE"), "{0:D}", d);
Console.WriteLine(s);

then output changes to (depending on the thread's default culture) something along the lines of:
Some(31/12/2010 00:00:00)

If you have unit tests then you can catch this sort of issue otherwise your luck will be your friend or your enemy. Either way, the fix looks ugly:
var d = Option.Some(new DateTime(2010, 12, 31));
var s = string.Format(new CultureInfo("de-DE"), "{0:D}",
                      d.Match(v => v, () => default(DateTime?)));
Console.WriteLine(s);

For debugging purposes, one can still rely on DebuggerDisplayAttribute to display the string representation as it is today.

Hi!
I think consistency with nullable is a good idea, and a very reasonable expectation (in retrospect, it seems like the obvious thing to do).
I consider the suggestion approved. It is, however, a breaking change, so I will designate this for version 4.0.0.


Integrate a PR on the bundle?

Hello,
is it possible to integrate https://github.com/boltops-tools/terraspace-bundler/pull/11 for the next realease?
thanks

Released in terraspace-bundler 0.3.4. For immediate use:
cd into-terraspace-project
bundle update terraspace-bundler


Complex numbers example

Adds a Mathematics section with an example that shows how to create complex numbers.
fixes #448

Okay I made the changes as requested. I don't know in how much detail I'm supposed to go with the complex numbers 🤔 since explaining how they work or interact usually takes up an entire section in an advanced mathematics text book (and even then just to cover the basics). So I've shown how they can interact with each other and that the type itself has built-in functions if you want to use the number in mathematical functions.

Love it


Updated TS example to highlight how to output  and run JS from seperate directory

Problem
The simplicity of the existing TypeScript example is enough to get people of the ground.
In practise however most users will want to exclude the JS outputted by the complier from version control. Doing so while outputting .js files next to their .ts makes is possible and included however would also exclude any config or other .js files that might reside in the repository.
This has been raised as a question at least once.
Solution
Update the example so that TypeScripts outputs JS into a bin directory which is then used by next.js by providing the bin directory as a command line argument.
Considerations before merging
Additional complexity in example

This is a great improvement to the TS development experience, but the one thing it's missing before it's perfect* is support for the /static folder. I have small project based off of the TS example, and when trying out your fix, I discovered the /static folder doesn't get carried over into bin like everything else.

@KraigWalker Same issue. Have you found a solution?

I was looking into this today. You can’t configure the static folder in next from what I can see nor is getting  tsc an option.
I’d started looking at configuring somethig in package.json to move the contents of static into the bin.

On 30 Sep 2017, at 16:55, Dillon Raphael<EMAIL_ADDRESS>wrote:
@KraigWalker Same issue. Have you found a solution?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@timneutkens hmm other people having the same issue

This PR seems out of date, could you make a new one against the canary branch, thanks!


Stream Past Data

Hi, I want to know if there is a way to stream past data. Here are the details:

I receive a past data array which is gathered in the backend from date1 to date1. (Both date1 and date2 are past dates)

I want to stream the data array from the beginning to the end.

Thanks a lot!

You can delay the start of streaming using delay property. In this case, set delay to Date.now() - date1.getTime() (assuming date1 is a Date object) and stream the data with the past timestamps. You can preload all the data to chart.data.datasets at a time, but it might be better to use onRefresh() to feed the data periodically if the data size is big.

Sounds good. Thank you!

Inside onRefresh(), I cannot get the service from my component. Any idea about it?
this.mychart.chart.options.plugins.streaming.onRefresh = (chart: any) => {
      chart.data.datasets.forEach(function (dataset: any) {
        this.myService.getDataById(this.id).subscribe(data => {
            dataset.data.push(data);
        });
     });
};

this value In the onRefresh function is not the same as the this value for the function that set onRefresh because it is called by the callback function of setInterval. Here is the solution.

Thanks a lot!


mvcc/reader: scan versions from u64::max to start_ts in get_txn_commit_info

Signed-off-by: youjiali1995<EMAIL_ADDRESS>
What have you changed? (mandatory)
In optimistic transaction, we assume if txn_1.start_ts < txn_2.start_ts, txn_1.commit_ts < txn2.commit_ts, so if we seek a commit record whose start_ts is greater than the seeking start_ts,
we can stop.
But in pessimistic transaction, it's possible a txn with a small start_ts has a greater commit_ts than a txn with a greater start_ts, i.e., txn_1.commit_ts > txn_2.commit_ts > txn_2.start_ts > txn_1.start_ts. So scan versions from the latest to the start_ts to find the commit record.
Please NOTE that:

Do not assume reviewers understand the original issue

What are the type of the changes? (mandatory)
The currently defined types are listed below, please pick one of the types for this PR by removing the others:

Bug fix (change which fixes an issue)

How has this PR been tested? (mandatory)
Unit tests.
Does this PR affect documentation (docs) or release note? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No

@coocood @zhangjinpeng1987 @AndreMouche  PTAL.

LGTM

@AndreMouche PTAL


[Roadmap] Next (major) release

Planned features:

[x] #65
[x] #60
[x] #43
[x] #51
[x] #53
[x] #38
[x] #62
[x] #54
[x] #56

Skip for now:

[x] #44

Misc:

[x] Document the meaning of the schema: the schema describes the arrays not the rust structs
[x] Re-implement maps for the outer struct (tests: test_maps_with_missing_items, into_outer_maps_simple)
[x] Test Lists with i32 offsets
[x] Remove special cased dictionaries in interpreter and use raw buffers directly
[x] Test different order between arrrays and fields (cannot happen: the order of arrays and fields must be consistent)

Release v0.8.0-rc.1


Problem with last item (Testing) on left menu on page https://echo.labstack.com/guide

Can't click on testing item. My browser is google chrome v. 58.0.3029.110. Screen resolution is 1366x768

@akosourov Thanks for reporting. This is fixed now, empty cache and reload.


Create Docker Container only WebApp Folder

I would like to work on it.

@harikishantk Go head

@harikishantk any update !!!

I had finished creating dockerfile and docker-compose.yml files. I'm trying to test by running the container but there is problem with requirements.txt already.

I had finished creating dockerfile and docker-compose.yml files. I'm trying to test by running the container but there is a problem with requirements.txt already.

Do one thing 1st install all dependencies and run the app.py then use this command pip3 freeze > requirements.txt.

@harikishantk update me asap !!

@harikishantk update me asap !!

https://github.com/aryasoni98/AI-Interview/pull/39#issuecomment-997418796


Fix deobfuscator

Coverage decreased (-0.4%) to 73.75% when pulling 53ac3ec5822ab486895d49c62456e02dba43fac4 on NeoSpb:fix_deobfuscator into c923d19bcc6f8ed1dce269aa4d5cf307fb506898 on skylot:master.