Using http-proxy-middleware with react

src/setupProxy.ts
import express from 'express';
import {createProxyMiddleware} from 'http-proxy-middleware';
const app = express();
app.use('/registration-api/**', createProxyMiddleware({target: 'http://localhost:8710', changeOrigin: true}));
app.listen(3000);
config axios
export const http = axios.create({
baseURL: baseUrl()
});
where baseUrl use for context path
export function baseUrl() {
const applicationUrl = document.head.querySelector('meta[name=application-url]');
const baseUrl = applicationUrl ? applicationUrl.getAttribute('content') : '/';
return baseUrl == null ? '/' : baseUrl;
}
api
export function registration(values: RegistrationFormData): Promise {
return http.post('/registration-api/registration', values);
}
api call result
POST http://localhost:3000/registration-api/registration 404 (Not Found)
What am I doing wrong ?

Have you tried removing the ** from the URL passed into app.use?
app.use('/registration-api', createProxyMiddleware({target: 'http://localhost:8710', changeOrigin: true}));

Yes i tried.  But I seem to have succeeded. I don't know what is the difference
src/setupProxy.js
const {createProxyMiddleware} = require('http-proxy-middleware');
module.exports = function (app) {
app.use(
'/registration-api/**',
createProxyMiddleware({
target: 'http://localhost:8710',
changeOrigin: true,
})
);
...
};

I think /registration-api/** means it has to have ** in it. So /registration-api/**foo, for example.
All you need is /registration-api though and that will match anything starting with /registration-api:

/registration-api/some/thing
/registration-api/other/thing
/registration-api

It's explained in the Express docs if you want more examples - https://expressjs.com/en/4x/api.html#path-examples

I had an issue using CRA 4.0.0 and making setupProxy a TS file. Rename it to a JS file.


[[FIX]] Add CompositionEvent to the list of browser globals

https://www.w3.org/TR/uievents/#events-compositionevents

Merged to master at commit 56515cf5444a06a0f02970536deaac84cf88227e. I had never heard of CompositionEvent before--interesting. Thanks for the patch!


[Event Hubs] Troubleshooting Guide Header Fix

Summary
Adding an additional missing header that I missed earlier.

API change check
API changes are not detected in this pull request.

/check-enforcer evaluate


Add erlang related files to mix.exs

Otherwise this package cannot be compiled when using through hex.

Followed the example from https://github.com/opentelemetry-beam/opentelemetry_telemetry/blob/main/mix.exs#L65


when I  use "docker-compose build " command to build notary, it comes some problems，how could I solve it

Step 4 : RUN go get github.com/mattes/migrate
---> Running in 2d9d1c9a378e
package golang.org/x/net/context: unrecognized import path "golang.org/x/net/context" (https fetch: Get https://golang.org/x/net/context?go-get=1: dial tcp <IP_ADDRESS>:443: i/o timeout)
package gopkg.in/inf.v0: unrecognized import path "gopkg.in/inf.v0" (https fetch: Get https://gopkg.in/inf.v0?go-get=1: dial tcp: lookup gopkg.in on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:49441-><IP_ADDRESS>:53: i/o timeout)

Hi @rusirenye - I have not been able to reproduce this myself, but after searching around it seems like this might be related to the proxy network configuration issues described on https://github.com/golang/go/issues/12810?

@riyazdf  thank you very much.  I try to set network proxy environment variable  in docker file ,then go get github.com/mattes/migrate command works all right. It seems  that golang:1.6.3-alpine images container can't use host network proxy.

@rusirenye great to hear that we've diagnosed the issue!  As you mention, it seems like an issue with golang:1.6.3-alpine and proxy configuration -- I'm inclined to close the issue on notary but if you think that we could somehow mitigate the issue on our side feel free to re-open this issue or file another one.


Enable HDF5 support at caffe for android

Now caffe for android compiles and runs without HDF5
it disable at commit like
https://github.com/DVEfremov/caffe/commit/c9abe3ac397af5729e38a947f52a270c616e4f73
but it's not clear for now why it does not work for android.
In scope of this issue need to investigate and try to enable HDF5 for android and get caffe-time program work

I don't think we should bother with that on Android (at least at this stage) - if I am correct, it's mostly needed to efficiently process huge amount of images (or other artifacts) during training, which we will unlikely to do on Android. Instead we may do small re-training (when having unexpected behavior or misclassification) or do more time-consuming re-training on a server (for example via CK API with web-service for predictive analytics)...

I think this ticket is outdated (we do not need it at this stage), so I close it.


Add HTML support in RMF title

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#4672  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nalcalag and the rest of your teammates on  Graphite


[Extensibility Request] Codeunit 402 "Sales Order Statistics" - Procedure from Local to Global & Protected procedures

Hello, we need the following currently Local procedure to be set as Global. Can you please check this? Thank you!
procedure UpdateHeaderInfo(IndexNo: Integer; var VATAmountLine: Record "VAT Amount Line")

We also need the following currently Local procedure to be set as protected, for the same purpose. Can you please check this one too? Thank you!
protected procedure GetCaptionClass(FieldCaption: Text[100]; ReverseCaption: Boolean): Text[80]

protected procedure VATLinesDrillDown(var VATLinesToDrillDown: Record "VAT Amount Line"; ThisTabAllowsVATEditing: Boolean)

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


Missing license file

@mokagio, thank you for this library! Elm's RemoteData is great.
Would you mind adding a license file?
I would love to use RemoteData in my project.

Thank @nmalancea 🙇♂
I added the license file, how silly of me not to do it from the get go 🤦♂ .
What's your project? 😄


Change Readme file #1225

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

@SauravMukherjee44  and please close this issue after merge issue no #1225


Feb15test2

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

@ZohoSign-integ Thank you for your submission. Our Engineer will review your request.

@Amjed-Ayoub Just remainder on this to review this PR.

@ZohoSign-integ thank you for the reminder and apologies for the delay in process. @sriyen-msft will review your request soon. Thank you for your patience.

@ZohoSign-integ Are you sure you are updating your connector? I can not see any file in the PR.

Have done necessary changes in the PR https://github.com/microsoft/PowerPlatformConnectors/pull/1464 . Please check.

@sriyen-msft Have made necessary changes in the PR https://github.com/microsoft/PowerPlatformConnectors/pull/1464 . Please check it.

@ZohoSign-integ, I am closing this PR as the updated files are uploaded in another PR.


Only check if stdout is a tty.

Mocha only writes to stdout, and users may want to redirect stderr. Mocha ends up with unnecessarily constrained output in this case, since while stderr will be a pipe stdout will still be a tty. This PR changes the code to only check if stdout is a tty instead of checking both stdout and stderr.
In  the master branch try in bash:
bin/mocha 2> >(cat)

Mocha assumes stdout is a pipe and only uses the default 75 character screen width.
If running the same command in this branch, Mocha will correctly detect the screen width.

Thanks!  @paparomeo I'm not sure what bug(s) this fixes.  Can you show how this fix can be used, and what happens before the fix?

@boneskull: in this screencast you can see how in master, when redirecting stderr mocha does not make use of the whole width of the terminal, only using the default 80 columns for non-ttys. Then it switches to my branch and you see it using the whole width of the terminal when redirecting stderr. Hopefully this will demonstrate the issue I describe above. Please let me know if not and if you need further clarification.

@boneskull can this be merged?

I can't speak for @ boneskull re. the explanation, but for my two cents' worth:
It may be helpful to have a test for this, which would

demonstrate what the change fixes in terms of (preferably realistic or at least non-contrived) example usage,
in the codebase for posterity (no need to look up the discussion/recording to see what difference the new logic makes), and
ensure that we don't accidentally change it back without realizing it (regression)


Fix barcode centering

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 20 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.0%) to 31.352%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/EasyPDF/ClassEasyPDF-Barcode.cs
0
20
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.0%

Covered Lines:
305

Relevant Lines:
911

💛  - Coveralls


docs: update prompt usage example

Update the prompt example to reveal the list method

coverage: 95.224% (-1.2%) from 96.465%
when pulling 9cf38b57947e07ad70f7f5b0560d758b02f6a5e0 on 0-prompt-examle-update
into aee2e66823ed9611a335f30c4985cb73aa2cc611 on main.


Error when upgrade Angular 2.0.0-beta.13 to 2.0.0-rc.1

IMPORTANT: This repository's issues are reserved for feature requests and bug reports. Do not submit support requests here.
Steps to reproduce and a minimal demo of the problem
When i upgrade, i see error:

In Commandline
D:/nodejs/sb-ng2-client/node_modules/rxjs/add/operator/map.d.ts(2,16): error TS2435: Ambient modules cannot be nested in other modules or namespaces. D:/nodejs/sb-ng2-client/node_modules/rxjs/add/operator/map.d.ts(2,16): error TS2436: Ambient module declaration cannot specify relative module name. src\directives\auth-router-outlet\auth-router-outlet.ts(20,11): error TS2345: Argument of type 'ElementRef' is not assignable to parameter of type 'ViewContainerRef'. Property 'element' is missing in type 'ElementRef'. src\main.ts(6,9): error TS2305: Module '"D:/nodejs/sb-ng2-client/node_modules/@angular/router-deprecated/index"' has no exported member 'APP_BASE_HREF'. src\main.ts(6,42): error TS2305: Module '"D:/nodejs/sb-ng2-client/node_modules/@angular/router-deprecated/index"' has no exported member 'LocationStrategy'. src\main.ts(6,60): error TS2305: Module '"D:/nodejs/sb-ng2-client/node_modules/@angular/router-deprecated/index"' has no exported member 'PathLocationStrategy'.

+systemjs.config.js:
System.config({     transpiler: 'typescript',     typescriptOptions: {emitDecoratorMetadata: true},     map: {         'app' : 'app',         'rxjs': '/node_modules/rxjs',         '@angular'                    : '/node_modules/@angular',         '@angular/core'                    : '/node_modules/@angular/core',         '@angular/common'                  : '/node_modules/@angular/common',         '@angular/compiler'                : '/node_modules/@angular/compiler',         '@angular/router-deprecated'                  : '/node_modules/@angular/router-deprecated',         '@angular/platform-browser'        : '/node_modules/@angular/platform-browser',         '@angular/platform-browser-dynamic': '/node_modules/@angular/platform-browser-dynamic'       },       packages: {         'app'                              : {main: 'main.ts', defaultExtension: 'ts'},         'rxjs'                             : {main: 'index.js'},         '@angular/core'                    : {main: 'index.js'},         '@angular/common'                  : {main: 'index.js'},         '@angular/compiler'                : {main: 'index.js'},         '@angular/router-deprecated'       : {main: 'index.js'},         '@angular/platform-browser'        : {main: 'index.js'},         '@angular/platform-browser-dynamic': {main: 'index.js'}       } });

config.ts
... { src: ${ASSETS_SRC}/js/systemjs.config.js, inject: 'libs', dest: JS_DEST } ...

package.json:
... "dependencies": {   "angular2": "2.0.0-beta.13",   "angular2-notifications": "0.1.47",   "@angular/common": "2.0.0-rc.1",   "@angular/compiler": "2.0.0-rc.1",   "@angular/core": "2.0.0-rc.1",   "@angular/http": "2.0.0-rc.1",   "@angular/platform-browser": "2.0.0-rc.1",   "@angular/platform-browser-dynamic": "2.0.0-rc.1",   "@angular/router": "2.0.0-rc.1",   "@angular/router-deprecated": "2.0.0-rc.1",   "@angular/upgrade": "2.0.0-rc.1",   "reflect-metadata": "^0.1.3",   "rxjs": "5.0.0-beta.6",   "systemjs": "^0.19.27",   "zone.js": "^0.6.12",   "async": "^1.5.2" } ...

pls tell me, which error and fixed?
Current behavior
Expected/desired behavior
Other information

Hi @lhpkhanh,
on which commit of the seed is your code based on?  The SystemJS configuration of the seed is done in seed.config.ts(https://github.com/mgechev/angular2-seed/blob/master/tools/config/seed.config.ts):
/**
   * The system builder configuration of the application.
   * @type {any}
   */
  SYSTEM_BUILDER_CONFIG: any = {
    defaultJSExtensions: true,
    packageConfigPaths: [
      join(this.PROJECT_ROOT, 'node_modules', '*', 'package.json'),
      join(this.PROJECT_ROOT, 'node_modules', '@angular', '*', 'package.json')
    ],
    paths: {
      [`${this.TMP_DIR}/*`]: `${this.TMP_DIR}/*`,
      '*': 'node_modules/*'
    },
    packages: {
      '@angular/core': {
        main: 'index.js',
        defaultExtension: 'js'
      },
      '@angular/compiler': {
        main: 'index.js',
        defaultExtension: 'js'
      },
      '@angular/common': {
        main: 'index.js',
        defaultExtension: 'js'
      },
      '@angular/http': {
        main: 'index.js',
        defaultExtension: 'js'
      },
      '@angular/platform-browser': {
        main: 'index.js',
        defaultExtension: 'js'
      },
      '@angular/platform-browser-dynamic': {
        main: 'index.js',
        defaultExtension: 'js'
      },
      '@angular/router-deprecated': {
        main: 'index.js',
        defaultExtension: 'js'
      },
      '@angular/router': {
        main: 'index.js',
        defaultExtension: 'js'
      },
      'rxjs': {
        defaultExtension: 'js'
      }
    }
  };

Did you write your systemjs.config.js yourself or is it a generated file?
Can you try to do a fresh clone of the seed with the latest commit (https://github.com/mgechev/angular2-seed/commit/591a57226bf95bc3d14e443c86b38092dcd6ed73) and try to reproduce it again?
Greetings
Dope

I do it and rebuild, but still error:
-------------CMD-------------------------
$ npm install
npm WARN install Couldn't install optional dependency: Unsupported
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will
fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon
as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained.
Upgrade to<EMAIL_ADDRESS>npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will
fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon
as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program
Files\nodejs\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v5.0.0
npm ERR! npm  v3.3.6
npm ERR! code ECONNRESET
npm ERR! errno ECONNRESET
npm ERR! syscall read
npm ERR! network read ECONNRESET
npm ERR! network This is most likely not a problem with npm itself
npm ERR! network and is related to network connectivity.
npm ERR! network In most cases you are behind a proxy or have bad network
settings.
npm ERR! network
npm ERR! network If you are behind a proxy, please make sure that the
npm ERR! network 'proxy' config is set properly.  See: 'npm help config'
npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program
Files\nodejs\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v5.0.0
npm ERR! npm  v3.3.6
npm ERR! code ECONNRESET
npm ERR! errno ECONNRESET
npm ERR! syscall read
.....
npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program
Files\nodejs\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v5.0.0
npm ERR! npm  v3.3.6
npm ERR! Callback called more than once.
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     D:\nodejs\sb-ng2-client\npm-debug.log
pls help me with error
Thanks!
2016-05-17 18:17 GMT+07:00 Christian Dobert<EMAIL_ADDRESS>
Hi @lhpkhanh https://github.com/lhpkhanh,
on which commit of the seed is your code based on? The SystemJS
configuration of the seed is done in seed.config.ts(
https://github.com/mgechev/angular2-seed/blob/master/tools/config/seed.config.ts
):
/**   * The system builder configuration of the application.   * @type {any}   /
SYSTEM_BUILDER_CONFIG: any = {
defaultJSExtensions: true,
packageConfigPaths: [
join(this.PROJECT_ROOT, 'node_modules', '', 'package.json'),
join(this.PROJECT_ROOT, 'node_modules', '@angular', '', 'package.json')
],
paths: {
[${this.TMP_DIR}/*]: ${this.TMP_DIR}/*,
'': 'node_modules/*'
},
packages: {
'@angular/core': {
main: 'index.js',
defaultExtension: 'js'
},
'@angular/compiler': {
main: 'index.js',
defaultExtension: 'js'
},
'@angular/common': {
main: 'index.js',
defaultExtension: 'js'
},
'@angular/http': {
main: 'index.js',
defaultExtension: 'js'
},
'@angular/platform-browser': {
main: 'index.js',
defaultExtension: 'js'
},
'@angular/platform-browser-dynamic': {
main: 'index.js',
defaultExtension: 'js'
},
'@angular/router-deprecated': {
main: 'index.js',
defaultExtension: 'js'
},
'@angular/router': {
main: 'index.js',
defaultExtension: 'js'
},
'rxjs': {
defaultExtension: 'js'
}
}
};
Did you write your systemjs.config.js yourself or is it a generated file?
Can you try to do a fresh clone of the seed with the latest commit (
591a572
https://github.com/mgechev/angular2-seed/commit/591a57226bf95bc3d14e443c86b38092dcd6ed73)
and try to reproduce it again?
Greetings
Dope
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mgechev/angular2-seed/issues/887#issuecomment-219688770

Hi @lhpkhanh,
just as the log output says: This is most likely not a problem with npm, but rather with a bad network connection:
npm ERR! network read ECONNRESET
npm ERR! network This is most likely not a problem with npm itself
npm ERR! network and is related to network connectivity.
npm ERR! network In most cases you are behind a proxy or have bad network
settings.

This issue is not related to angular2-seed

Error when i run server on CMD:
D:/nodejs/sb-ng2-client/node_modules/rxjs/add/operator/map.d.ts(2,16): error TS2435: Ambient modules cannot be nested in other modules or namespaces. D:/nodejs/sb-ng2-client/node_modules/rxjs/add/operator/map.d.ts(2,16): error TS2436: Ambient module declaration cannot specify relative module name. src\directives\auth-router-outlet\auth-router-outlet.ts(20,11): error TS2345: Argument of type 'ElementRef' is not assignable to parameter of type 'ViewContainerRef'.   Property 'element' is missing in type 'ElementRef'. src\main.ts(6,9): error TS2305: Module '"D:/nodejs/sb-ng2-client/node_modules/@angular/router-deprecated/index"' has no exported member 'APP_BASE_HREF'. src\main.ts(6,42): error TS2305: Module '"D:/nodejs/sb-ng2-client/node_modules/@angular/router-deprecated/index"' has no exported member 'LocationStrategy'. src\main.ts(6,60): error TS2305: Module '"D:/nodejs/sb-ng2-client/node_modules/@angular/router-deprecated/index"' has no exported member 'PathLocationStrategy'. [13:20:54] TypeScript: 6 semantic errors

Hi @lhpkhanh,
from your console output i can see that there are a few issues (incorrect imports etc.).
But without a look at the sourcecode i'm afraid i cannot help you any further.
Also those issues are not related to angular2-seed itself.
This issue should be closed as it is not related to the seed itself.

Closing the issue for now, since it is not a seed issue.


Project status?

Hi Yann, Damien, James!
Your library looks very interesting. Right now I have to use pymatgen for CIF parsing, however it's quite heavyweight and not really a narrow-purpose tool. Do you plan to continue the development of this library? e.g. python bindings would be very desirable.
Cheers,
EB

@blokhin, may I suggest COD CIF parser for your purpose? It's in C language and has bindings for both Perl and Python (see pycodcif on PyPI). I have published a paper (Merkys et al., 2016) comparing it to other general/special purpose CIF parsers.

@merkys Saulius told me once, that COD CIF parser is written in Perl, and so I was hesitant to use it, because I am afraid of Perl :) But if you say it's C, then I'll try, thanks!

@blokhin, we have rewritten the parser in C in order to speed it up and to make it re-usable in other languages.

Hi,
The development of LibCIF is currently stalled for lack of volunteers. We will have an important ESL workshop in February and will decide by then, or before, what to do with it. COD CIF could actually be an interesting alternative if we are able to provide Fortran wrappers on top of its C code.
I will put it as an item for our next ESL monthly meeting and keep you informed here of our discussions.


Feature request: animations based on ratings

What feature do you want to get added on the base engine?
so i had an idea the other day, (based off of this: https://www.youtube.com/watch?v=dObzU7amIBU), and i thought "what if animations were based off of the player's ratings?" like bad would be like a miss animation, and etc.
To test your sight, and reliability, please select the option of what should NOT be requested.


Pre-push hook shows an error

In Windows 10, you can see hook install task generate hook looks like this.
#!/bin/sh
set -e
##### KOTLINTER HOOK START #####
##### KOTLINTER 3.4.4 #####
GRADLEW=C:\Users\rlcks\workspace\freelec\gradlew.bat
if ! $GRADLEW lintKotlin ; then
    echo 1>&2 "\nlintKotlin found problems, running formatKotlin; commit the result and re-push"
    $GRADLEW formatKotlin
    exit 1
fi
##### KOTLINTER HOOK END #####

And this shows error with message below.
.git/hooks/pre-push: line 6: C:Usersrlcksworkspacefreelecgradlew.bat: command not found
\nlintKotlin found problems, running formatKotlin; commit the result and re-push
.git/hooks/pre-push: line 8: C:Usersrlcksworkspacefreelecgradlew.bat: command not found
error: failed to push some refs to 'https://github.com/nkgcp/freelec.git'

It seems like \ in the hook file is recognized as escape character. Changing \ to \\ or / makes it works fine.

Thank you for the error report @nkgcp! Will get it fixed soon.

Whats the status on this - we also ran into it and it would be nice to avoid it for newcomers to the project :-)

Sorry it's slipped through the cracks.
The fastest way would be for someone to submit a PR -- especially someone who works in a windows environment and can quickly verify the fix.
Otherwise I will try to get to it.

Nothing to be sorry for! I can maybe do a PR monday.

Nothing to be sorry about. I am not sure how to verify a fix. I think a possible solution could be altering gradlew.path to gradlew.invariantSeparatorsPath in the gradleCommand val. But not sure how to verify it.

Nothing to be sorry about. I am not sure how to verify a fix. I think a possible solution could be altering gradlew.path to gradlew.invariantSeparatorsPath in the gradleCommand val of GitHookTask.kt. But not sure how to verify it.

On your pull request branch you can bump version, publish to maven local, and then in your test project put mavenLocal() first in the repo list and pull in the version of kotlinter built by your PR. At least that's what I usually do.

Hmm.. Cant seem to make it work.

plugins {
   ...
    id("org.jmailen.kotlinter") version "3.5.0-test" // my test version
}

    repositories {
        mavenLocal()
        mavenCentral()
    }

Cannot find it. But I can find it in my ~.m2/repository folder.

Or the instructions for multi-module in the readme will work:
buildscript {
    repositories {
        mavenLocal()
        ...
    }
    dependencies {
        classpath("org.jmailen.gradle:kotlinter-gradle:3.5.0-test")
    }
}

apply(plugin = "org.jmailen.kotlinter")

@rvplauborg thanks for your idea for solution this problem.
I have issued a pull request along with it. Sorry, if you wanted to do it yourself.
https://github.com/jeremymailen/kotlinter-gradle/pull/218


Some incorrect spelling appeared on the method name.

Hello, I found some spelling errors when I read RocketMq's source code. I think it's necessary to fix it, so I raised this issue and provided a PR to fix it.
Util.computeEclipseTimeMilliseconds must be a error about spelling, according to the meaning of the method name, it must be 'computeElapsedTimeMilliseconds ' or rigght ?
Also  the word 'comput' may be 'compute'.
Thanks.

I reviewed your PR #1307 and it is OK, but if you could rename the computPullFromWhichFilterServer in PullAPIWrapper to computePullFromWhichFilterServer by the way, all the "comput" misspell will be fixed.
Thank you :)


Fix formatting error in kubectl debug release note

A missing newline after the last list item caused an important note about plugins being hidden to be append to an unrelated note about debugging nodes.

Page preview:
https://deploy-preview-25515--kubernetes-io-master-staging.netlify.app/blog/2020/12/08/kubernetes-1-20-release-announcement/#kubectl-debug-graduates-to-beta

Thanks @verb .
/lgtm

/approve


[BEAM-10138] Add Cross-language KinesisWrite external transform

Add KinesisIO.Write for Cross-language usage.
Chronology:

java write
java read
python wrapper
python integration tests

PR for python wrapper: #12235
PR for java external transform for read: #12297
PR for python wrapper integration tests: #12298
R: @TheNeuralBit - if you are no fed up with my changes yet then I'd be grateful for your review :)

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Closing in favor of #12297


Issue while loading Tiff File with load_img()

Hello everyone,
I have a problem while loading a certain tiff image file. The Tiff file dan be downloaded here:
https://www.dropbox.com/s/0lw6i0yc9xmctrn/SOE4630.tif?dl=0
When I try to load it with the command:
from keras.preprocessing.image import load_img
load_img(path_to_tiff)
I get the following error:
------ERROR--------
ValueErrorTraceback (most recent call last)
 in ()
----> 1 load_img('/opt/hdp/share/data/wertermittlung/newData/Ausbau/train/ausbau/GT12113.tif')
/opt/hdp/anaconda2/lib/python2.7/site-packages/keras/preprocessing/image.py in load_img(path, grayscale, target_size, interpolation)
351     else:
352         if img.mode != 'RGB':
--> 353             img = img.convert('RGB')
354     if target_size is not None:
355         width_height_tuple = (target_size[1], target_size[0])
/opt/hdp/anaconda2/lib/python2.7/site-packages/PIL/Image.pyc in convert(self, mode, matrix, dither, palette, colors)
855                 return self.copy()
856
--> 857         self.load()
858
859         if matrix:
/opt/hdp/anaconda2/lib/python2.7/site-packages/PIL/TiffImagePlugin.pyc in load(self)
1009         if self.use_load_libtiff:
1010             return self._load_libtiff()
-> 1011         return super(TiffImageFile, self).load()
1012
1013     def load_end(self):
/opt/hdp/anaconda2/lib/python2.7/site-packages/PIL/ImageFile.pyc in load(self)
202                                       args, self.decoderconfig)
203                 seek(offset)
--> 204                 decoder.setimage(self.im, extents)
205                 if decoder.pulls_fd:
206                     decoder.setfd(self.fp)
ValueError: tile cannot extend outside image
----- ERROR -----
However, I can easily load the image by using the libtiff library. I guess the problem is the shape of the tiff file, which is (x,y,4). When I drop the fourth dimension to get a shape of (x,y,3) I can pass it to Keras. Usually the fourth dimension contains something like an alpha for transparency or other further information. Is there a way to get native support in Keras for such images?
I get a similar error message in Keras with Python 3.
Cheers,
Marcel

Hello,
actually since Keras uses PIL, I guess I should have posted it on the PIL page. However, I found out that I can load the image if I use ImageMagick:
$ convert in.tiff out.tiff
The outout of identify looks similar. Only this has changed:
old:
Properties:
date:create: 2018-05-24T16:32:37+02:00
date:modify: 2018-05-24T16:32:07+02:00
signature: b24dd796543cc9a1be794ed31153c417cf3f32f460fcbc6ca8a7dc30171ceb92
tiff:alpha: unspecified
tiff:endian: lsb
tiff:photometric: RGB
Artifacts:
filename: SOE4630.tif
verbose: true
Tainted: False
Filesize: 705KB
Number pixels: 442K
Pixels per second: 11.04MB
User time: 0.040u
Elapsed time: 0:01.040
Version: ImageMagick 6.9.7-4 Q16 x86_64 20170114 http://www.imagemagick.org
new:
Properties:
date:create: 2018-05-24T17:03:55+02:00
date:modify: 2018-05-24T17:03:55+02:00
signature: b24dd796543cc9a1be794ed31153c417cf3f32f460fcbc6ca8a7dc30171ceb92
tiff:alpha: unassociated
tiff:endian: lsb
tiff:photometric: RGB
tiff:rows-per-strip: 909
Artifacts:
filename: test.tif
verbose: true
Tainted: False
Filesize: 478KB
Number pixels: 442K
Pixels per second: 14.73MB
User time: 0.030u
Elapsed time: 0:01.030
Version: ImageMagick 6.9.7-4 Q16 x86_64 20170114 http://www.imagemagick.org
Maybe it the tiff alpha?
Cheers,
Marcel

I tried loading your example image using PIL and I find that it is not an RGBA image, but an RGBX image, which seems to mean "RGB image with padding" (something that I have not encountered before). Opening the image with some other image manipulation library (I used opencv) and saving it as an RGBA image seems to fix the issue. The convert() function in PIL is not able to to do the job for whatever reason. This suggests that we are looking at a bug in PIL. I suggest you try submitting a bug report to them.
In the meantime, we should probably work around this by giving the user an informative error message if they try to load an RGBX image.

Hello, thanks for your answer. I will open the issue in PIL. BTW, I got the image from a GIS program. Actually, I am not interested in the X dimension. Hence, I used libtiff (which can load the image) as a workaround for a MonkeyPatch.
Cheers,
Marcel

Hm, actually. Keras is using PIL not pillow, right? Do you know the exact github project where I should open the issue?
Thanks in advance and cheers,
Marcel


Log in issues and download issues

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Can you give us more details about the issue you're having?


Higher screenfull version to fix request in chrome 71

Fixes #23

Thank you. Yes I noted problems too and updated some packages without commiting.


Pressing ALT+C trigger Color Picker instead of placing diacritical sign

Pressing ALT+C trigger Color Picker instead of placing diacritical sign. It's very annoying.

Hi! You can change the key binding within the Color Picker settings.
Go to Settings › Color Picker Settings › Trigger key.


engine.Tail() to ignore trailing whitespaces.

In its current form, if an error message has two trailing "\n" instead
of one, an empty line is resulted (see engine/job.go for an example of
such usages).
Skipping all trailing whitespaces will give a better error message.

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:tcnghia/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

Thanks @GordonTheTurtle !
I updated as suggested.

Hello, any update on this PR?

A minor thing, but I'm wondering... if there's a \n at the end of the text then this will not return it.  Should it?  Otherwise "hello" and "hello\n" will look the same - should they?

@duglin "hello\n" and "hello" already look the same before this change.  This change will make "hello\n\n" the same as "hello\n" and "hello".

Understood - I'm just wondering if we need to preserve one \n if its there.  Its a nit.
Either way, I'm LGTM.

ping @jfrazelle

LGTM


layout: Get rid of duplicate pixel size constant

The commit should obviously say font size and not pixel size. Will fix. :P Guess that's what I get for working late.


Cleaning up org teams / owners / owners_aliases permissions

This issue is to track a couple PRs that accomplish the following:

From k/org/blob/master/config/kubernetes/sig-docs/teams.yaml

Remove @xiangpengzhao as a reviewer or approver (welcome back at any time and may reference this issue)
Remove @bradamant3 as a reviewer or approver with write access (welcome back at any time and may reference this issue)
Remove @steveperry-53 as a reviewer or approver with write access (welcome back at any time and may reference this issue)
Remove @jaredbhatti as a reviewer (welcome back at any time and may reference this issue)

From k/website/blob/master/OWNERS

Add @steveperry-53 as commented out emeritus

From k/website/blob/master/OWNERS_ALIASES

Remove @xiangpengzhao as a reviewer or approver (welcome back at any time and may reference this issue)

/priority important-soon
/kind cleanup
/cc @kbarnard10 @irvifa @zacharysarah

@jimangel I'm curious whether we do periodic check for this as part of clean up or this is possible?


setup.go:231: Pruned unused networks: [ibctest-tqjmvsek]
make: *** [e2e-test-interchainswap-basic] Error 1

I see thanks for creating this issue @DedicatedDev, feel free to re-open or open a new issue if you run into any problems with the e2e tests! I'll close this again if you're not having problems anymore.


Do you have a plan for ORC writer?

Thanks for your excellent work.
And do you have a pan for a writer? it's also useful in many cases.

Thank you for your appreciation of our work.
Regarding your inquiry about a writer feature, while it's certainly a valuable addition, we currently lack the resources to implement it. However, we warmly welcome any contributions in this regard. Notably, ORC Writer boasts implementations in various other languages, making the creation of a Rust version relatively straightforward. Additionally, leveraging GPT for code translation could streamline the process.

got it, thanks

I am actually planning to do an implementation of writing (it will be rudimentary and unoptimized at first). We have an existing issue for this where I might post progress as it comes: https://github.com/datafusion-contrib/datafusion-orc/issues/11

I am actually planning to do an implementation of writing (it will be rudimentary and unoptimized at first). We have an existing issue for this where I might post progress as it comes: #11

wonderful, hope we can see it sooner.


Command Line argument on lua

Does it work like this?
expect.ext test.lua arg1 arg2 arg3

Now it is not supported yet. But, I will implement it. Please wait.

Please let me know when it is ready.
-------- Original Message --------
From:HAYAMA_Kaoru<EMAIL_ADDRESS>Sent:Tue, 07 Aug 2018 17:05:36 +0800
To:zetamatta/expect<EMAIL_ADDRESS>Cc:Mobieus Jay tangingw.pas@gmail.com,Author<EMAIL_ADDRESS>Subject:Re: [zetamatta/expect] Command Line argument on lua (#5)

Now it is not supported yet. But, I will implement it. Please wait.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.￼
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/zetamatta/expect","title":"zetamatta/expect","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/zetamatta/expect"}},"updates":{"snippets":[{"icon":"PERSON","message":"@zetamatta in #5: Now it is not supported yet. But, I will implement it. Please wait."}],"action":{"name":"View Issue","url":"https://github.com/zetamatta/expect/issues/5#issuecomment-410987910"}}} [ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/zetamatta/expect/issues/5#issuecomment-410987910", "url": "https://github.com/zetamatta/expect/issues/5#issuecomment-410987910", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } }, { "@type": "MessageCard", "@context": "http://schema.org/extensions", "hideOriginalBody": "false", "originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB", "title": "Re: [zetamatta/expect] Command Line argument on lua (#5)", "sections": [ { "text": "", "activityTitle": "HAYAMA_Kaoru", "activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png", "activitySubtitle": "@zetamatta", "facts": [ ] } ], "potentialAction": [ { "name": "Add a comment", "@type": "ActionCard", "inputs": [ { "isMultiLine": true, "@type": "TextInput", "id": "IssueComment", "isRequired": false } ], "actions": [ { "name": "Comment", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "zetamatta/expect",\n"issueId": 5,\n"IssueComment": "{{IssueComment.value}}"\n}" } ] }, { "name": "Close issue", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "IssueClose",\n"repositoryFullName": "zetamatta/expect",\n"issueId": 5\n}" }, { "targets": [ { "os": "default", "uri": "https://github.com/zetamatta/expect/issues/5#issuecomment-410987910" } ], "@type": "OpenUri", "name": "View on GitHub" }, { "name": "Unsubscribe", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "MuteNotification",\n"threadId": 365364599\n}" } ], "themeColor": "26292E" } ]

In a few days. If you want as soon as possible, I can hurry.

Done. See https://github.com/zetamatta/expect/releases/tag/20180807 to download binary executable.

Thanks for the file. Will test it tomorrow.

Nice feature. I am using this now as well. Just had to look up how string concatenation in lua .. works.


'Sinch/SINLogSeverity.h' file not found

Getting file not found while using use_frameworks! option with CocoaPod, works fine if not using use_frameworks!

I'm getting this error also, any way to fix it ?? Does this mean that i cannot use SinchService with Swift ?

It seems that this repo is no longer maintained.

great xD I'll fork it then and fix the headers. And try to make it work. If it doesn't, i'll use firebase for the notifications.

Any solution or guide for this issue??


SELinux is preventing Everest (Modded Celeste) from functioning

After installing the Olympus flatpak,  setting the installation folder of Celeste,  installing Everest into it and attempting to launch modded Celeste, the game crashes and instead a text editor opens up with the text:
Monocle Engine Error Log
==========================================

01/04/2024 16:55:19
System.ComponentModel.Win32Exception (13): SELinux execheap probe failed! Please ensure Everest has this permission, then try again
   at Celeste.Mod.BOOT.Main(String[] args) in /home/vsts/work/1/s/Celeste.Mod.mm/Mod/Everest/BOOT.cs:line 65

Unfortunately you'll need to either move the location of this executable to a more proper location, set SELinux to permissive, or make a SELinux rule for this. I would recommend trying /usr/local/bin.

I have the same issue, but just on regular Fedora, does anyone how to make a SELinux rule for this?


Modified config generaor to put params in DB

There was an error that params were not added when adding custom sensor configuration.

Thank you for your contribution!
Both params and transform were checked in generate_config, but not checked in insert_to_vehicle_sensor_index
This PR added a safe-check in the function insert_to_vehicle_sensor_index.

@seohwan Can you provide me a little more context about what those rubis files are? The rubis_config_generator looks similar to config_generator except the rubis version uses yaml. Should we keep 2 config generators or should we keep just 1 of them?

Hello,
oh, I just wanted to pull-request only one commits(779bcfc2fdb831fa5ece77b6c92173fe15629bb7)
I'll do the pull-request again.
Sorry for the confusion

No worries. Thanks for the contribution!


Automatic update of redirects page fails

See https://drone.gitea.io/go-gitea/redirects/18
+ /usr/local/bin/docker daemon -g /var/lib/docker
Error authenticating: exit status 1

Solved, added org secrets for the docker authentication.


[wip] Allow compound variants to be extended from a composed component

Description
Allows composed components to inherit and extend compoundVariants (Fixes #431)
TODO

[x] write tests
[x] clean-up code

Feedback welcome.

Finished the feature/fix with tests. Also have this branch running in a medium sized Stitches project, working well and fixing my issue with #431
New to the project, so let me know any feedback on the tests and merge algorithm. I'll squash the commits before merge when it's ready.

@arempe93, this was a well researched PR. I wasn’t sure whether we could revisit how variants were assembled or how it would work, but now with #440 we have a solution that supersedes this. I’m closing this PR, but with great appreciation and mad respect. It also means you noticed the problem and figured out how it worked. I hope you’ll stick around — that’s rare help for OSS.


Magic 8 Ball

For those tough decisions. Could be fun to expand to like 40 or 50 responses
Input: !8 Should I attack the dragon?
Output: Most definitely

Adding help wanted label as a test - if anyone comes across this and wants to go for it, please add it as a new file to the discordlib directory, and add the new file to the array at the top of discordlib/index.js - that will auto-import your code when the bot loads :)


Inconsistent numeric types in certain sequences

The SequenceInterface says that getElement returns "number," but on the other hand, OEISSequenceTemplate fills the cache which produces values for getElement with "BigInt"s. (I am not sure what TypeScript's exact typing rules are for numbers here, so I am unclear on why TypeScript does not report any type error or warning for the code that puts BigInts into a cache declared to contain numbers.)
Anyhow, this mismatch causes some visualizers, e.g. ShiftCompare, to fail on OEIS sequences; they contain code that performs computations (such as the modulo operator) which does not allow mixing of BigInts and numbers.
We need to decide on a policy of how to represent numbers for sequences in Numberscope, and then use it uniformly.

Yes, we should probably discuss this on Friday.  I feel strongly that we should support BigInts, or we will be hobbled (the integers in the OEIS are just big).  But it opens a lot of questions about how to require visualizers to behave.

I guess there is also the question of the numeric type for the input to getElement, i.e. of the index for a sequence. Plain old "number" seems adequate for that, since we generally only analyze thousands of terms in a sequence,, so maybe we should just leave it be as "number". Unless it seems logically simpler/more consistent to map BigInts to BigInts...

We had a productive conversation about this at the meeting today. To summarize the consensus approach:

Sequence objects will handle only integers.
More specifically, Sequence objects might need to do non-integer calculations internally, and a client like a Visualizer might want to do non-integer calculations with the results, but the interfaces will only be in terms of those integers.
To handle the full spectrum of the OEIS, those integers require arbitrary precision.
Hence, all of the Sequence interfaces will be modified to operate in terms of the javascript BigInt datatype, both for input and output (because not infrequently one wants to combine indices and sequence elements in some way).
However, in recognition of point (2.) and the desire to make Visualizer writing as easy as possible, Numberscope will have to provide ways to perform at least common non-integer operations on BigInts, such as sin, cosine, natural log, fractional/negative powers, and probably more. We want these all to be as painless to a Visualizer writer as possible, so ideally the standard context or imports for writing a Visualizer provides the necessary operations (and fails in a very clear way when the operation is not available). Similarly, to make things as easy as possible for creating sequences, we want to make bigint as close to a "native" type for math.js as well.
In pursuing (5.) we should strive to re-use as opposed to re-invent as many wheels as possible. Notably Sage has already solved all of the mathematical problems implicit in (5.)
Finally, the implementations should recognize that computational issues exist also for operations that are strictly integer-to-integer.  Ideally, this would automatically happen in ordinary arithmetic expressions both in TypeScript Visualizer code and in math.js expressions. That may turn out to be unrealistic in one or both of these contexts, in which case we would probably have to settle for providing a powmod(a,k,n) function for that context and prominently advertising it for people to use. There are probably other similar cases, although this is likely the most critical one.

If you have any comments/edits on this plan, please do post them as soon as you have the opportunity. I can commence work on this as soon as my last PR is cleared.

And I guess somewhere in the "user guide," we likely will want a "visualizer writer's guide" with some information about doing calculations with the output of Sequence objects. Should an issue for this be created now, or when we have a clearer idea about what all that would comprise?

Should an issue for this be created now, or when we have a clearer idea about what all that would comprise?

I vote for a placeholder issue.

In practice, point (4) in the resolution turned out to just seem wrong; we far more often use the input to a sequence just as an index and don't combine it with the results. The code will look much more "normal" if we leave this as an ordinary typescript number. So hereby reversing course on that point.

In the first rendition of this (#50), in order to have a way to indicate that an index was not valid for a given Sequence, I had to make the return type of getElement be bigint|undefined. This however will complicate computing with the output of a Sequence. So there is an alternative proposal in which a Sequence would have methods providing the range of valid indices and getElement would always return a bigint. Resolving this can't proceed until we decide between these alternatives, so please see the discussion on them in #48 and provide your thoughts/votes. Thanks!

"Proposal B" in the discussion of #48 has been adopted, so when PR #51 is finally merged, it will resolve this issue.


Separate transaction schema into different schemas for different operations

With oneOf, we may be able to provide more specific schema definitions for each operation (e.g. CREATE, TRANSFER, GENESIS).
There's a few things to think about as to how to actually achieve this breakdown, and if we'd like to commit to it.

@libscott this was handled in #1257, right?

There are separate JSON Schema files for CREATE/GENESIS and TRANSFER transactions, so I'm closing this issue.


feat: new akmods build process

See my PRs in -config, -akmods, -nvidia.

Build process and signing infra based off -nvidia
Currently only builds v4l2loopack, but can be extended with e.g. 'xpadneo'
Builds ublue-akmod-others as a cache
Builds {silverblue,...}-extended images for end users
Build times adjusted to be in line with other repos
Includes ublue-os-akmods-key.spec, which contains, well, the akmods signing key previously found in ublue-os-nvidia-addons.spec
Extends the default justfile (which is now installed in -main) with justfile-akmods

Akmod signing requires the appropriate (the same as in -nvidia!) AKMOD_PRIVKEY in the repo secrets, but I obviously can't do that from here.

@dhoell I want to pickup work on this PR. Not sure if you've noticed, but the related PRs in config, main, and the intermediate PR in nvidia were all merged.
So next is getting this ready to go.
It's still a big chunk of change, but I'm thinking I could pull it into a branch here and work on it. I do have a request, which I'm putting into a change request comment.


Repo still maintained?

Hello. This look like a very promising repo but last update was around 2018. Is this still maintained?

(Sorry for late response.)
Project has been archived.


Windows 10 VirtualBox synced folders don't like Composer

Windows 10, Vagrant 1.8.1, VirtualBox 5.0.20
I get the error:
Failed to download drupal/core from dist: Could not delete web/core/6154c8a03a58e79151b32c248b8ce96d

Full log of the failed task:
==> drupalvm: TASK [Generate Drupal project with composer package.] **************************
==> drupalvm: fatal: [localhost]: FAILED! => {"changed": true, "cmd": ["/usr/bin/composer", "create-project", "drupal-composer/drupal-project:8.x-dev", "/var/www/drupalvm/drupal", "--prefer-dist", "--stability", "dev", "--no-interaction"], "delta": "0:08:23.956900", "end": "2016-06-13 20:09:37.197460", "failed": true, "rc": 1, "start": "2016-06-13 20:01:13.240560", "stderr": "Running composer as root/super user is highly discouraged as packages, plugins and scripts cannot always be trusted
Installing drupal-composer/drupal-project (8.x-dev 544ba79cd95cb4fca599cba586498aaf0faf2885)
  - Installing drupal-composer/drupal-project (8.x-dev 544ba79)
    Loading from cache

Created project in /var/www/drupalvm/drupal
Loading composer repositories with package information
Updating dependencies (including require-dev)
  - Installing drupal-composer/drupal-scaffold (2.0.1)

 Loading from cache

  - Installing cweagans/composer-patches (1.5.0)
    Loading from cache

...

  - Installing symfony/polyfill-apcu (v1.2.0)
    Loading from cache

  - Installing symfony/class-loader (v2.8.7)
    Loading from cache

  - Installing drupal/core (8.1.2)

    Downloading: Connecting...\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 0%
         \b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 5%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 10%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 15%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 20%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 25%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 30%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 35%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 40%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 45%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 50%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b
    Downloading: 55%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 60%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 65%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 70%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 75%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 80%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 85%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 90%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 95%\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b    Downloading: 100%
    Failed to download drupal/core from dist: Could not delete web/core/6154c8a03a58e79151b32c248b8ce96d:
    Now trying to download from source
  - Installing drupal/core (8.1.2)

  [RuntimeException]
  Could not delete web/core/6154c8a03a58e79151b32c248b8ce96d:

create-project [-s|--stability STABILITY] [--prefer-source] [--prefer-dist] [--repository REPOSITORY] [--repository-url REPOSITORY-URL] [--dev] [--no-dev] [--no-custom-installers] [--no-scripts] [--no-progress] [--no-secure-http] [--keep-vcs] [--no-install] [--ignore-platform-reqs] [--] [<package>] [<directory>] [<version>]", "stdout": "", "stdout_lines": [], "warnings": []}

See upstream/related:

https://github.com/drupal-composer/drupal-project/issues/126
http://stackoverflow.com/a/26216954/100134

I'm not sure if I'm hitting the Composer timeout (the task does hang for a long time), or if it's a symlink/similar issue. I don't hit this on Fedora or Mac OS X, so it seems to be a Windows/VirtualBox thing.

Testing with a composer.json build instead of drupal-project, following these instructions—basically set the following variables in config.yml after copying the example composer file to drupal.composer.json:
build_composer_project: false
build_composer: true
drupal_core_path: "{{ drupal_composer_install_dir }}/docroot"

Now I'm hitting https://github.com/ansible/ansible/issues/9526

I'm going to try setting vagrant_synced_folder_default_type to smb to see if that helps. This also requires me to run Cmder/Cygwin64 as administrator, so not ideal, but just testing to see if it works out.

Now I'm hitting a bug in our composer.json :)

Getting the same issue, "Could not delete docroot/core/cifs8b2c" now, at the end of the "Run composer install." task :P
Looks like for Windows, we might need to recommend the makefile approach for now (or any setup that doesn't require Drupal VM building a composer-based codebase itself inside a shared directory), until I can figure out a more stable way of doing Composer stuff in a shared directory.

Testing with the following config (after creating drupal.make.yml from the example):
build_makefile: true
build_composer: false
build_composer_project: false
drupal_core_path: "/var/www/drupalvm/drupal"

The makefile configuration seems to have worked. drupalvm.dev is up and running.

Better title ^^

I'm back to the default configuration. Going to try to see if there's a way to make Composer happy on Windows inside a VirtualBox synced folder.

Hmm. Does it work if you change the location of vendor to somewhere outside of the shared folder?
{
    "name": "custom-project/drupal-vm",
    "description": "",
    "type": "project",
    "license": "MIT",
    "authors": [
        {
            "name": "",
            "role": ""
        }
    ],
    "repositories": [
        {
            "type": "composer",
            "url": "https://packages.drupal.org/8"
        }
    ],
    "require": {
        "composer/installers": "^1.0.20",
        "drupal-composer/drupal-scaffold": "^2.0.1",
        "drupal/core": "~8.0",
        "drupal/devel": "1.0.x-dev"
    },
    "replace": {
        "drupal/drupal": "*"
    },
    "minimum-stability": "dev",
    "prefer-stable": true,
    "config": {
        "vendor-dir": "/home/vagrant/vendor"
    },
    "extra": {
        "installer-paths": {
            "docroot/core": ["type:drupal-core"],
            "docroot/modules/contrib/{$name}": ["type:drupal-module"],
            "docroot/profiles/contrib/{$name}": ["type:drupal-profile"],
            "docroot/themes/contrib/{$name}": ["type:drupal-theme"],
            "drush/contrib/{$name}": ["type:drupal-drush"]
        }
    }
}

This looks highly related to #741 as well (just tagging that reference).

I'm testing using a longer composer timeout (as suggested here), and also for Windows VirtualBox compatibility, I've switched the task that moves composer.json into place (inside build-composer.yml) to two tasks; one to copy it into a non-shared-directory location, then a second to use mv to move the file (because of the upstream Ansible issue on VirtualBox synced dirs and the copy module).

This seems to have worked. (the longer timeout)

Hmm... it worked without the timeout this time, so maybe it's just when things are slow. I'd still like to add some config to make it more reliable/stable of course...

Now trying composer project with:
- name: Generate Drupal project with composer package.
  command: >
    {{ composer_path }} create-project
    {{ drupal_composer_project_package }} {{ drupal_composer_install_dir }}
    {{ drupal_composer_project_options|default('--prefer-dist --no-interaction') }}
  when: not drupal_site_exists
  become: no
  environment:
	COMPOSER_PROCESS_TIMEOUT: 1200

And docs for process-timeout: https://getcomposer.org/doc/06-config.md#process-timeout

It is crazy slow... but it's working. I can't imagine using VirtualBox native synced folders for daily use. Just the 'Install configured modules with drush' task is taking like 30 min :P

Testing, and if it works (even if it's crazy-slow), I'll tag a bugfix release and hopefully solve this plus #741 in one shot.

Works! And was way faster this time, even though I don't have vagrant-cachier installed on the laptop.

Increasing timeout works for me as well on drupalvm  / windows 10

Note: increasing timeout in my composer.json also worked for me

I did not find any solution for this problem.
Is there any?

I did not find any solution for this problem.
Is there any?


feat: use description to encourage appropriate merge/squash/rebase strategy

This PR updates the status check description to be more helpful:

If the PR title is semantic, encourage squashing.
If the title is not semantic but the commits are, encourage merging or rebasing.

Would it make sense to always require a semantic PR title and squash commits?
Often times commits are quite meaningless, like "fix a typo" or "address review feedback".

Would it make sense to always require a semantic PR title and squash commits?

I tend to agree, but I think this bot should be flexible enough to still work in non-squash scenarios.
We could update the bot to detect if the repo only allows squash commits:

In this case we could change the behavior to require a semantic title.


Warn on deleting pinned entries

It would be nice if clipse warned the user before deleting a pinned entry. This would be very useful in cases where I mass-delete entries once I'm done with a specific task and end up deleting pinned items without realizing
And thanks a lot for creating this amazing clipboard utility!

Hi @maydayv7 :wave:
I think that's a reasonable suggestion. Will look into the best way to achieve this and include in the next release if feasible.
Thank you for your suggestion!

@maydayv7  this one too!


Fix Tree-based Navigation Bug

I have encountered a bug where the PresentationState does not turn nil when a View is popped.
This happened during a push transition using Tree-based Navigation like this.
.navigationDestination(
    store: store.scope(state: \.$destination, action: { .destination($0) }),
    state: /MyReducer.Destination.State.screenA,
    action: MyReducer.Destination.Action.screenA
) { store in
    ScreenA(store: store)
}
.navigationDestination(
    store: store.scope(state: \.$destination, action: { .destination($0) }),
    state: /MyReducer.Destination.State.screenB,
    action: MyReducer.Destination.Action.screenB
) { store in
    ScreenB(store: store)
}

Currently, the following code is executed when a View is popped.
self.viewStore.binding(
  get: {
    $0.wrappedValue.flatMap(toDestinationState) != nil
      ? toID($0).map { AnyIdentifiable(Identified($0) { $0 }) }
      : nil
  },
  compactSend: {
    guard
      $0 == nil,
      self.viewStore.wrappedValue != nil,
      id == nil || self.toID(self.viewStore.state) == id
    else { return nil }
    return .dismiss
  }
),

However, when I debugged it, the id was not nil.
So I erased the evaluation of id == nil and it works fine.
Why is it currently evaluating id == nil || self.toID(self.viewStore.state) == id? It seems to me that it should simply return .dismiss when $0 is nil.

@Ryu0118 Any chance you can get a repro for us, or even UI test in the Integration project, so we better understand the fix and catch regressions in the future? These integration tools can be a bit thorny at times, and hopefully we have a regression test that already catches why we have these checks, but we'll see soon!

As @stephencelis mentioned, we do have an integration test in place that shows why this check is here. With your change the integration test suite fails on this assertion:
https://github.com/pointfreeco/swift-composable-architecture/blob/main/Examples/Integration/IntegrationUITests/PresentationTests.swift#L193
It looks like the second alert is not being presented after the first one is dismissed.
In order for us to best understand the problem you are trying to fix, I think it would be best if you could come up with a minimal reproducible example demonstrating the behavior. Then we can see if there is a fix that also maintains the current behavior.

Okay, I will try to write a program that can reproduce it.
And also found that it works correctly in the simulator, but not on the actual device.
In actual device,
self.toID(self.viewStore.state) is
Optional(AnyHashable(ComposableArchitecture.(unknown context at $105d77dc0).NavigationDestinationID(objectIdentifier: ObjectIdentifier(0x0000000105f046c0), enumTag: Optional(1))))

id is
Optional(AnyHashable(ComposableArchitecture.(unknown context at $105d77dc0).NavigationDestinationID(objectIdentifier: ObjectIdentifier(0x0000000105f046c0), enumTag: Optional(0))))

Comparing the two, the enumTag has different values between self.toID(self.viewStore.state) and id.
But In simulator,
self.toID(self.viewStore.state) and id is
Optional(AnyHashable(ComposableArchitecture.(unknown context at $105d77dc0).NavigationDestinationID(objectIdentifier: ObjectIdentifier(0x0000000105f046c0), enumTag: Optional(1))))

So it looks like the case path problem.

@Ryu0118 I believe those test failures are related to why we need the identity check there.

@Ryu0118 I'm going to close this for now. I'd be curious if the new non-reflective case paths fix for you, though! We also will be streamlining these modifiers soon, so potentially even less that can go wrong.


Feature on map loosing hover info after drag/drop, translate or getGeometry().setCoordinates

Using v6.2.1
Project type: GPS tracking vehicles and trains on a map
I have a map I have created with multiple layers defined externally.
var map_large = new ol.Map({             target: 'map',             layers: [layer_railmap, layer_roadmap, layer_moveablestuff, layer_vehicles, layer_trains],             view: map_large_view,             interactions: ol.interaction.defaults({                 dragPan: false,                 altShiftDragRotate: false,                 mouseWheelZoom: false,                 pinchRotate: false,                 doubleClickZoom: false             }),             controls: []         });
I have also added interactions by
map_large_interaction_translate = new ol.interaction.Translate({layers: [layer_moveablestuff,layer_vehicles]}); // allow dragging of moveablestuff map_large.addInteraction(map_large_interaction_translate);
I store my features in an array to allow adding and deleting easily.
my feature definition:
     var vehicles_tomonitor = ["A", "B", "C", "D", "E" ];         let features_vehicles = [];         for (var v = 0; v <= vehicles_tomonitor.length; v++) {             let feature_vehicle = new ol.Feature({                 geometry: new ol.geom.Point(                     ol.proj.fromLonLat([xxxx, xxxx]) // remove for privacy                 ),                 name: vehicles_tomonitor[v],             });             feature_vehicle.setStyle(new ol.style.Style({                 image: new ol.style.Icon({                     anchor: [0.5, 0.5],                     size: [24, 24],                     opacity: 1,                     scale: .8,                     src: '/images/mapping/car-4-24.png'                 }),                 zIndex: v + 90             }));             features_vehicles[vehicles_tomonitor[v]] = feature_vehicle;         }
I receive GPS Lat and Long for trains and vehicles in JSON via a ServerSentEvent process that works well.
I move the feature programmatically by
 let rego = jsondata.rego;                 if (!source_vehicles.hasFeature(features_vehicles[rego])) source_vehicles.addFeature(features_vehicles[rego]);                 features_vehicles[rego].getGeometry().setCoordinates(((ol.proj.fromLonLat([parseFloat(jsondata.lon), parseFloat(jsondata.lat)])))); //  features_vehicles[rego].changed // didn't work
My issue is that once the feature is moved, I sometimes cannot using the event "pointermove" on detect the feature to fire a tooltip or cursor change
 map_large.on('pointermove', function (evt) {                 if (!evt.dragging) {                     var pixel = map_large.getEventPixel(evt.originalEvent);                     var feature = map_large.getFeaturesAtPixel(pixel);                     console.log(feature.length) // proof of concept to see how many features at pixel }}};
I've tried even adding .changed() to the feature after the move but that didn't work either.
If I drag the map and then hover over the lost feature, the feature regains it's info.

Worked out my puzzle, what I didn't say above is that i also had a 2nd map instance sharing layer source.
Features need UID and the source layer had to be unique as well


[/etc/tmpfiles.d/var.conf:19] Duplicate line for path "/var/cache", ignoring.

Describe the bug
Warning message during the system update
 nix-channel --update && nixos-rebuild switch
unpacking channels...
building Nix...
building the system configuration...
activating the configuration...
setting up /etc...
reloading user units for mudrii...
setting up tmpfiles
[/etc/tmpfiles.d/var.conf:19] Duplicate line for path "/var/cache", ignoring.

To Reproduce
Steps to reproduce the behavior:
Run  nix-channel --update && nixos-rebuild switch
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Standard update
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
  /nix/store/5h35s50503spks8gkxdv3kk4szz8vx9g-bash-interactive-4.4-p23-dev
  /nix/store/qghrkvk86f9llfkcr1bxsypqbw1a4qmw-stdenv-linux
copying path '/nix/store/5h35s50503spks8gkxdv3kk4szz8vx9g-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/qghrkvk86f9llfkcr1bxsypqbw1a4qmw-stdenv-linux' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.15, NixOS, 19.09.1977.1dadefec574 (Loris)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.2`
 - channels(root): `"home-manager-19.09, nixos-19.09.1977.1dadefec574, unstable-20.03pre211015.8539d5f48f9"`
 - channels(mudrii): `"home-manager, unstable-20.03pre196836.1c40ee6fc44"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@mudrii can you please upload /etc/tmpfiles.d/var.conf? I'll probably need a list of services you have enabled too, but let's with var.conf.
ping @Mic92 didn't you recently enable upstream tmpfiles rules? While unrelated to this PR directly I am curious if we'll run into more entries like this... :thinking:

@aanderse
After some testing. I find the issue it seems is locale if it is activated in service will throw the error.
cat /etc/tmpfiles.d/var.conf
#  This file is part of systemd.
#
#  systemd is free software; you can redistribute it and/or modify it
#  under the terms of the GNU Lesser General Public License as published by
#  the Free Software Foundation; either version 2.1 of the License, or
#  (at your option) any later version.

Please test: https://github.com/NixOS/nixpkgs/pull/79022


Update conanfile.py

settings.os represent the target system not the build system (https://docs.conan.io/en/latest/systems_cross_building/cross_building.html#gnu-triplet-convention).
Thus we need to check the settings_os_build variable not the os variable.
This will fix the problem that CMake tries to check if the compiler works with wrong parameter (which results in using -rdynamic which may not be supported on mcu compilers)

Seems ok to me.
@amatosov-rbx Do you want to take a look?


Async support for ORMs

Hi, I'm really into apistar as django seems to be out of the async hype train. I see that there are integrations with sql alchemy and django's orm but they don't seem to be supporting async/await things. Could you elaborate more on async/non-async components in docs?

It doesn't relate to Apistar. Django and SqlAlchemy don't support async usage. Actually, it related to DB Driver (psycopg2...).
SqlAlchemy Author explains this problem in http://techspot.zzzeek.org/2015/02/15/asynchronous-python-and-databases/ blog post.
Maybe you can use asynio.run_until_complete for DB queries.

A good option, although it's not an ORM (which can be an advantage) is to use the fast and asynchronous asyncpg driver https://github.com/MagicStack/asyncpg with SQLAlchemy Core integration asyncpgsa https://github.com/CanopyTax/asyncpgsa. We are using in production and it's got strong foundations both the driver and the SA integration. Warning: it's not an ORM, did I say it?


Plan for auto-tuner

Hi all,
After went through paper of Halide,  noticed that hailde has implemented an auto-tuning optimizer, see links here: http://graphics.cs.cmu.edu/projects/halidesched/.
If I'm correct, TVM is pretty much the same as stage 1 of Halide, which was published at the year of 2013,  see links here: http://people.csail.mit.edu/jrk/halide-pldi13.pdf
So, my question is, is there any plan for auto-tuning optimizer? BTW, IMO, in graph IR, we can get more information across kernels, maybe it's a better place to decide the schedule, or make the trade off. Which abstraction is better for the auto-tuning optimizer anyway?
Thanks!

In order to get a good autotuned kernels, there are two part to get the ingredients right:

A large enough space of schedules to cover the state of art techniques in handcrafting kernels,
A auto tuner that can effectively explore the space.

The focus of the project so far has been on part one, because if the schedule space is not good enough,  we never get the comparable performance, however clever the enumeration is. There was a quite a a lot of heavy lifting to close in this part and I think we are getting quite enough ingredient here.
with the scripts API we have, it is pretty straight forward to get some quick auto-tuning scripts for the workloads we care, which is the techniques we use to get some of the optimized kernels.
A more principled auto tuner is on our roadmap

for relation with existing projects like Halide you mentioned

/cc psyhtest

/cc @psyhtest

Thanks, @tqchen , look forward to detailed plan for principled auto-tuner in the future.

Hi, @tqchen , a follow up question, what's the difference between TVM and Halide, if TVM can use Hailide's IR,, why don't we just use the whole Halide directly?

http://docs.tvmlang.org/faq.html#tvm-s-relation-to-other-ir-dsl-projects  answers the difference from existing projects, including Halide.  In short, we specifically focus on deep learning, and optimize for multiple hardware backends.
As I mentioned in the first post, the major challenge is to make the schedule space complete enough to cover the state of art kernels for hardware back-ends we want to support, specifically gpu and other hardwares. The second challenge is to build the dsl representation to cover things we care about in deep learning(e.g. recurrence). The other issues include the ease of deployment and interpolation.
These challenges are not well addressed by existing frameworks(including Halide) and requires rethink and design of the stack as opposed to simply reuse an existing one.
You can also find that the TVM's IR itself is evolving, and we continuously learn new lessons from hand optimization and tuning for various backends.

As for your question in multi-stage scheduling. I think you already answered your own question, in a sense that tvm is intended to be used with graph level IR like nnvm, which gives you graph information and libraries like TOPI will make this problem easier

I think graph IR is a better place to do optimization of auto-multiple-stages, because of you will get more information about the whole pipeline, TOPI is just a temporary and practical solution in involving.
For example,  the main idea in the paper of Halide 2016 is to partition the functions/stages in a large Halide program into groups (subprograms), and independently perform producer-consumer locality and input reuse locality transformations on these groups.
About differences between Halide and TVM,  still confused to me.
1)"Schedule space“ you mentioned above,  what are they Hailde can't have? or will it be hard to extend this on Halide?
2)why TVM cares about the ability of representation, if TVM is a back-end IR of whole stack， that's the question graph engines should care.
Will there be a paper or something to elaborate your points above? I think this might be a often asked question with TVM growing. All of us might wanna know TVM is a new idea or just another wheel.
Thanks!

I think your also raises an valuable point here, in a sense that graph ir is necessary to work together with the low level ir for the workload optimization, that is exactly the approach we are taking right now. With nnvm for the graph level stack and tvm/topi as tensor level stack

figure out problems, and solve them on the

The things we need to do in whole stack

Coarse-grained operators optimization: operators/kernels/stages fusion and layout optimization, which are main issues XLA/NNVM, or other graph engines are trying to improve.
Fine-grained operators optimization: single kernel/across kernels performance,  on which tvm/halide focus.

So differences between halide and tvm,

halide is a tensor type dsl/IR，which don't have coarse-grained operators optimization, which are on graph engines,
it might be very hard or limited to do these coarse-grained operators optimizations on tensor IR
also I guess auto-tuner on graph IR might be simpler and easier than on tensor.

It seems that Tvm itself might  be similar with Halide, but with graph ir, they will be much more powerful.
It makes more sense to me now.
Thanks.

Your comment on the importance of two level stack is good. On the other hand, on the tensor ir level alone also  contains quite a lot of innovations, mainly on the scheduling primitives and language(e.g. recurrence support) to get more effective kernels on gpus and other hardwares, that is what I mean by representation power.

Hi Tianqi,  I've done some research on NNVM, notice that, there is no true IR in NNVM, user can register IR information via attributes, and user implements pass based on these attributes. also I don't see classical compiler related analysis like data flow, control flow, dependence test. Please confirm my understanding is correct first.
If I'm correct on most of part, I think NNVM is a good layer for domain specific optimization, like algebraic transformation in deep learning, which is not covered by traditional compiler technologies.
But it might be hard for NNVM to flow into a true compiler IR like TVM.
Besides, when you look into XLA that has true IR, you will notice that graph related optimization chances (which is orthogonal to  algebraic transformation on NNVM) do exist on this layer, also layer like XLA is
suitable for auto-tuner/auto-scheduler.
IMO, maybe NNVM + something like XLA + TVM is a better IR stack.
Thanks!

@xqdan I totally agree with what you said on graph ir based solution and the original version of nnvm lacks core operator support for that(the comment seems to be deleted). we have been working on such a graph ir based solution, you can checkout https://github.com/dmlc/nnvm for the end to end compiler. Maybe we could move the discussion over to NNVM

that's cool


error building Inveigh 2.0

Hi,
I've tried on my VM and on Appveyor and I get the same issue :
Any ideas ?
Many thanks, and looking fwd to test it.
C:\Program Files\dotnet\sdk\5.0.301\Sdks\Microsoft.NET.Sdk\targets\Microsoft.PackageDependencyResolution.targets(241,5): error NETSDK1004: Assets file 'C:\projects\inveigh\Inveigh\obj\project.assets.json' not found. Run a NuGet package restore to generate this file. [C:\projects\inveigh\Inveigh\Inveigh.csproj]
C:\Program Files\dotnet\sdk\5.0.301\Sdks\Microsoft.NET.Sdk\targets\Microsoft.PackageDependencyResolution.targets(241,5): error NETSDK1004: Assets file 'C:\projects\inveigh\Inveigh\obj\project.assets.json' not found. Run a NuGet package restore to generate this file. [C:\projects\inveigh\Inveigh\Inveigh.csproj]
C:\Program Files\dotnet\sdk\5.0.301\Sdks\Microsoft.NET.Sdk\targets\Microsoft.PackageDependencyResolution.targets(241,5): error NETSDK1004: Assets file 'C:\projects\inveigh\Inveigh\obj\project.assets.json' not found. Run a NuGet package restore to generate this file. [C:\projects\inveigh\Inveigh\Inveigh.csproj]

I'm guessing maybe you don't have .NET 5 installed? You can can try removing 5 from the list of frameworks in the project file.

Looking at the appveyor pre-install defaults, .NET 5 is included with the Windows Server 2019/VS2019 builds.
I haven't had a chance to try to break the compiles. I mainly had issues with 3.5 when working out how to get everything supported in a single project file.

Mhmm unfortunatly it didn't much success yet.
I'll try again in a few days.
Checked installed .net SDK and
dotnet --list-sdks
1.1.14 [C:\Program Files\dotnet\sdk]
2.1.202 [C:\Program Files\dotnet\sdk]
2.1.806 [C:\Program Files\dotnet\sdk]
2.2.402 [C:\Program Files\dotnet\sdk]
3.0.103 [C:\Program Files\dotnet\sdk]
3.1.202 [C:\Program Files\dotnet\sdk]
3.1.410 [C:\Program Files\dotnet\sdk]
5.0.301 [C:\Program Files\dotnet\sdk]

I'm not sure if this will help at all but I took a new Win10 VM and just installed VS 2019 community. I built through the GUI with the version on github. The build throws an error for .net 3.5. The other 2, 4.5 and 5.0 build ok.
dotnet --list-sdks
5.0.301 [C:\Program Files\dotnet\sdk]

This worked for me. Thanks!
Setup  :
win10
Vs2019


Unable to load library on mobile

Describe the bug
Does not work on the mobile chrome browser. I get an error saying Atomics is not defined. Unlike #63, I can load the library in desktop chrome just fine.
Is this not supported/ when can support be expected? I'm also ready to work on this, If someone can point me in the right direction on how to get started.
To Reproduce
Steps to reproduce the behaviour:

Use any example, including the default codepen demo
Open it in chrome mobile
See the error in the logs

Expected behaviour
The library loads just like it does on desktop chrome.
Smartphone (please complete the following information):

Device: OPPO realme one
OS: Android 9 (pie)
Browser: chrome

It might be the fact that the browser you used on mobile doesn't support SharedArrayBuffer, you can check it by using our official website: https://ffmpegwasm.github.io

Guyssss! FFMPEG started working on chrome android from version 11.4 !!!
update version to @ffmpeg/ffmpeg": "^0.11.4

Guyssss! FFMPEG started working on chrome android from version 11.4 !!! update version to @ffmpeg/ffmpeg": "^0.11.4

@Manitej66  it is not working at all i have tested, could you please share your package.json


Update messages.po

Have added translations to Ukraine language.

Hi @smitbmx.
First of all. Thank you for your work. Can you add your translations to our translation service? https://crowdin.com/project/chia-blockchain
We are not able to use translated .po files because it is a one-way (crowdin to repo) process.
Regards,
Zlatko

Sure, thank you!

No, thank you!


Tensorflow, gfx803, hipErrorNoBinaryForGpu: Unable to find code object for all current devices

Environment

Hardware
description

GPU
RX580

CPU
3700X

Software
version

OS
Linux 5.13.10

ROCm
4.3.0

Python
3.9.6

Tensorflow-rocm
2.5.0

What is the expected behavior
-Tensorflow should run correctly.
What actually happens
root@darkbox ~# python -m deeporn.fit
WARNING:root:Limited tf.compat.v2.summary API due to missing TensorBoard installation.
WARNING:root:Limited tf.compat.v2.summary API due to missing TensorBoard installation.
WARNING:root:Limited tf.compat.v2.summary API due to missing TensorBoard installation.
WARNING:root:Limited tf.summary API due to missing TensorBoard installation.
WARNING:deeporn.model:test_run=True
2021-08-21 08:57:36.488884: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libamdhip64.so
/home/yay/.cache/yay/hip-rocclr/src/HIP-rocm-4.3.0/rocclr/hip_code_object.cpp:486: "hipErrorNoBinaryForGpu: Unable to find code object for all current devices!"
fish: Job 1, 'python -m deeporn.fit' terminated by signal SIGABRT (Abort)

How to reproduce

run any tensorflow program

Do I have to recompile tensorflow?

I am not aware that tensorflow-rocm-2.5.0 had been released. I will test it and verify if it support gfx803.
And I have seen you used linux kernel-5.13 and python-3.9. You didn't test under ubuntu-20.04. right?

Thanks for the quick reply! I used Arch Linux.
I'm now trying to recompile tensorflow-rocm 2.5.0 using this AUR building script, which supports gfx803.
https://github.com/rocm-arch/tensorflow-rocm
but encountered some issue:
https://github.com/rocm-arch/tensorflow-rocm/issues/31

I recompiled https://github.com/ROCmSoftwarePlatform/tensorflow-upstream/tree/r2.5-rocm-enhanced , and didn't meet your compiling error.
I guess it may be caused by gcc-11, The gcc used in ubuntu-20.04.2 is gcc-9.

I'm actually using gcc-11. Let me try gcc-9.

Seems only tensorflow-rocm-2.5.0 provided python39 whl.
Maybe you can try python3.8 with tensorflow-rocm-2.4.3 https://pypi.org/project/tensorflow-rocm/2.4.3/#files

OK, after some research, the problem is that TF-2.5.0 referencing an outdated version of ruy. The issue in ruy is fixed in later commit. TF-2.6.0 references a later version of ruy, which solves the problem.
It seems that TF doesn't backport fixes, meaning we can only wait for tensorflow-rocm 2.6.0, or make a patch according to this issue

@xuhuisheng I just realize that you mentioned to patch rocblas (removing library/src/blas3/Tensile/Logic/asm_full/r9nano_*.yaml). Do we still need to do that for rocm4.3.0?

@riaqn
Please see the document for what this patch doing.
https://github.com/xuhuisheng/rocm-build/tree/master/gfx803#rocm-37-broke-on-gfx803

3 months after last posts, I will close this issue, please reopen if there is any updates.


macOS notarization

Major things

Add new options to codesign in all the places we invoke it. The options are --timestamp which adds a secure timestamp to the sig, and --options runtime which turns on the hardened runtime.
Add a new entitlements file for codesign which turns off the parts of the hardened runtime that stop an app from using memory intended for use by a JIT. This is memory marked as both executable and writable. The second option in that file is because Electron (the version we are using at least) doesn't mark the memory allocated as MAP_JIT, so we need to let the OS know we are cool with that.
Generate a new kbfuse.bundle with the new sig options. The binaries for these are just in source control (probably because they are extremely difficult to build from scratch), so luckily all I had to do was regen the sig.
Hack up kbnm to workaround a Go bug that would mark the macOS SDK at 10.7 under most circumstance. The only way I could figure out how to solve this is to force it to link against packages from the rest of the app (I don't know why this works).
Add a new step to package_darwin.sh that submits for notarization and waits for a response before continuing. You can pass NONOTARIZE=1 to build_app.sh (and hence the build bot) to skip it.

cc @chrisnojima


feat: allows ollama usage

User description
Fix https://github.com/Codium-ai/pr-agent/issues/657

Type
enhancement

Description

OLLAMA API Support: The PR introduces support for configuring the OLLAMA API base, allowing for expanded AI model usage within the application. This enhancement aligns with the initial goal of enabling OLLAMA usage, as indicated in the PR title and the linked issue.
Configuration Flexibility: By adding a conditional check for the OLLAMA.API_BASE setting, the application now supports more flexible configuration options, catering to different AI model APIs.
Code Consistency: Minor adjustments were made to maintain code consistency, such as ensuring a newline at the end of the chat_completion method.

Changes walkthrough
Relevant filesEnhancement

litellm_ai_handler.pyAdd OLLAMA API Base Configuration Support

pr_agent/algo/ai_handlers/litellm_ai_handler.py
Added support for OLLAMA API base configuration.  Ensured the API base is set for both Huggingface and OLLAMA configurations.  Maintained the return statement format at the end of the chat_completion method.

+4/-1

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

@gregoryboue
looks good
is the guide here still accurate:
https://pr-agent-docs.codium.ai/usage-guide/additional_configurations/#huggingface
?

@gregoryboue looks good
is the guide here still accurate: https://pr-agent-docs.codium.ai/usage-guide/additional_configurations/#huggingface ?

Yes the guide still accurate

@mrT23 , I did a mistake in my previous answer.
With my PR change, if i do a /review command, then here it's writed : model_type=ModelType.TURBO so in pr_processing._get_all_models the model used is get_settings().config.model_turbo and not get_settings().config.model
So the right configuration to use ollama model locally hosted is :
[config] # in configuration.toml
model = "ollama/llama2"
model_turbo = "ollama/llama2"

If i didn't do that the command uses fallback model and never try with config.model settings.
So is it normal to you ? Is the code must be changed or only documentation ?

/describe
--pr_description.extra_instructions="
For the title, use the format: [type]: [summary]
"
--pr_description.keep_original_user_title=false
--pr_description.final_update_message=false


[Bug] inline tailwind css is causing issues in target app

Bug report
Describe the bug
Tailwind classes used in this library are overriding the styles in the dashboard.
For example, .hidden is being used along with .lg:table-cell in the supabase dashboard, however, this library also contains a .hidden css class, which overrides the .lg:table-cell, hiding it on all screen sizes.

Solution
As a precaution and short term fix, we can move all the inline tailwind classes to use @apply inside a css file, so that all the styles are contained safely inside unique class names.
This has already been done partly already, as we needed to move all the .transition classes into style.css as they were interfering with animations in the dashboard app.

Moving tailwind classes into stylesheet now


primaryKey and relationships

I am having some problems understanding how to use primaryKey's when you have relationships in your realm models. I have three "simple" model objects RealmShow, RealmSeason and RealmEpisode. A Show has many Seasons and a Season has many Episodes. All three models have a primaryKey called "tmdbId" (they're from TheMovieDatabase.org and they're unique).
When I create a single show with one season and one episode, I get a nasty exception that says I can't insert and object with an empty value for the primaryKey 'tmdbId'. It says:
Can't set primary key property 'tmdbId' to existing value ''.'

I don't get what I'm doing wrong.
Here is the function creating the models that gets called:
func wat() {
        realm.beginWriteTransaction()

        let show = RealmShow()
        show.tmdbId = "51231"

        let season = RealmSeason()
        season.tmdbId = "hello wat"

        let episode = RealmEpisode()
        episode.tmdbId = "321"

        season.episodes.addObject(episode)
        show.seasons.addObject(season)

        realm.addObject(show)

        realm.commitWriteTransaction()
    }

Here are the model objects:
class RealmShow: RLMObject {
    // Identifiers
    dynamic var tmdbId = ""
    dynamic var tvdbId = ""
    dynamic var imdbId = ""

    // Data
    dynamic var name = ""
    dynamic var overview = ""
    dynamic var network = ""
    dynamic var country = ""
    dynamic var runtime = 0

    // Images
    dynamic var posterImagePath = ""
    dynamic var bannerImagePath = ""
    dynamic var backdropImagePath = ""

    // Relationships
    dynamic var seasons = RLMArray(objectClassName: RealmSeason.className())

    // State
    dynamic var tracked = false

    class var primaryKey: String {
        return "tmdbId"
    }
}

class RealmSeason: RLMObject {
    // Identifiers
    dynamic var tmdbId = ""

    // Data
    dynamic var number = 0
    dynamic var posterImagePath = ""

    // Relationships
    dynamic var show = RealmShow()
    dynamic var episodes = RLMArray(objectClassName: RealmEpisode.className())

    class var primaryKey: String {
        return "tmdbId"
    }
}

class RealmEpisode: RLMObject {
    // Identifiers
    dynamic var tmdbId = ""
    dynamic var tvdbId = ""
    dynamic var imdbId = ""

    // Data
    dynamic var number = 0
    dynamic var name = ""
    dynamic var overview = ""
    dynamic var firstAired = NSDate(timeIntervalSince1970: 0)

    // Images
    dynamic var stillImagePath = ""

    // Relationships
    dynamic var season = RealmSeason()

    // State
    dynamic var watched = false

    override class func primaryKey() -> String! {
        return "tmdbId"
    }
}

And here is the stack trace:
2014-12-18 15:15:52.351 Televised[19028:545083] *** Terminating app due to uncaught exception 'RLMException', reason: 'Can't set primary key property 'tmdbId' to existing value ''.'
*** First throw call stack:
(
	0   CoreFoundation                      0x000000010db4ef35 __exceptionPreprocess + 165
	1   libobjc.A.dylib                     0x000000010d7e7bb7 objc_exception_throw + 45
	2   Televised                           0x00000001097291f9 _Z13RLMDynamicSetP9RLMObjectP11RLMPropertyP11objc_objectm + 2462
	3   Televised                           0x000000010973a48b _Z19RLMAddObjectToRealmP9RLMObjectP8RLMRealmm + 1533
	4   Televised                           0x000000010972baaa _ZL18RLMAddLinkedObjectP9RLMObjectP8RLMRealmm + 266
	5   Televised                           0x000000010972974d _ZL11RLMSetValueP9RLMObjectmS0_m + 493
	6   Televised                           0x0000000109729020 _Z13RLMDynamicSetP9RLMObjectP11RLMPropertyP11objc_objectm + 1989
	7   Televised                           0x000000010973a48b _Z19RLMAddObjectToRealmP9RLMObjectP8RLMRealmm + 1533
	8   Televised                           0x000000010972baaa _ZL18RLMAddLinkedObjectP9RLMObjectP8RLMRealmm + 266
	9   Televised                           0x0000000109729c3b _ZL11RLMSetValueP9RLMObjectmPU28objcproto17NSFastEnumeration11objc_objectm + 474
	10  Televised                           0x0000000109729035 _Z13RLMDynamicSetP9RLMObjectP11RLMPropertyP11objc_objectm + 2010
	11  Televised                           0x000000010973a48b _Z19RLMAddObjectToRealmP9RLMObjectP8RLMRealmm + 1533
	12  Televised                           0x000000010975b735 -[RLMRealm addObject:] + 23
	13  Televised                           0x0000000109601605 _TFC9Televised11DataManager3watfS0_FT_T_ + 3541
	14  Televised                           0x00000001096006c3 _TFC9Televised11DataManagercfMS0_FT10tmdbClientCS_10TMDBClient5realmCSo8RLMRealm_S0_ + 1171
	15  Televised                           0x00000001096007e8 _TFC9Televised11DataManagerCfMS0_FT10tmdbClientCS_10TMDBClient5realmCSo8RLMRealm_S0_ + 88
	16  Televised                           0x00000001095e0ac5 _TFFC9Televised21ApplicationControllerg11dataManagerCS_11DataManagerU_FT_S1_ + 133
	17  Televised                           0x00000001095e04b0 _TFC9Televised21ApplicationControllerg11dataManagerCS_11DataManager + 448
	18  Televised                           0x00000001095ede65 _TFC9Televised22FeaturedViewController18viewModelsForShowsfS0_FGSaCS_8TMDBShow_GSaCS_17FeaturedViewModel_ + 101
	19  Televised                           0x00000001095f183d _TFFC9Televised22FeaturedViewController17fetchPopularShowsFS0_FT_T_U_FTSbGSqGSaCS_8TMDBShow___T_ + 461
	20  Televised                           0x00000001095f7e8e _TTRXFo_dSboGSqGSaC9Televised8TMDBShow___dT__XFo_dSbiGSqGSaS0____dT__ + 30
	21  Televised                           0x00000001095f94c6 _TFFFC9Televised10TMDBClientP33_3A509457C849A4A5E018A2A8990E3A1014performRequestFS0_U__FTCS_11TMDBRequest12parsingBlockFVS_4JSONQ_10completionFTSbGSqQ___T__CSo20NSURLSessionDataTaskU_FTGSqS2__GSqCSo7NSError__T_U_FT_T_ + 342
	22  Televised                           0x00000001095baa47 _TTRXFo__dT__XFdCb__dT__ + 39
	23  libdispatch.dylib                   0x000000010e497bdf _dispatch_block_async_invoke_and_release + 369
	24  libdispatch.dylib                   0x000000010e4b27f4 _dispatch_client_callout + 8
	25  libdispatch.dylib                   0x000000010e49b8fb _dispatch_main_queue_callback_4CF + 949
	26  CoreFoundation                      0x000000010dab6fe9 __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__ + 9
	27  CoreFoundation                      0x000000010da79eeb __CFRunLoopRun + 2043
	28  CoreFoundation                      0x000000010da79486 CFRunLoopRunSpecific + 470
	29  GraphicsServices                    0x000000010e9c39f0 GSEventRunModal + 161
	30  UIKit                               0x000000010acbb420 UIApplicationMain + 1282
	31  Televised                           0x000000010963bc63 main + 115
	32  libdyld.dylib                       0x000000010e4e7145 start + 1
)

@ksmandersen can you point me to a repo where you fixed this? I have the same issue, but do not understand what the error means.

If you get an exception with the message "Can't set primary key property '...' to existing value ...", it's generally because you're attempting to create a new object with the same primary key value of an existing object. Primary keys are required to be unique, since they represent that object's identity.
If you simply want to update an existing object with a given primary key, use +[RLMObject createOrUpdate:] or Realm().create(_:value:update:).