Validate Card Metadata

Card Metadata that comes at the top of the readme between the ---s should be validated. For example, languages are a list of strings, not a string, so if someone overwrites card.data['language'] = 'en' it should be automatically updated to ['en']. If they try to set it to a weird type, like a float, they should get a helpful error message.

whoops this one shouldn't have been closed, it has only very slightly begun to be resolved with #5 .

As @adrinjalali suggested, we should add a .validate() fn in the card data. this should mirror the validation done on the hub server side of things (as suggested by @julienc here). bonus points if you get same error messages too to help with debugging.

WDYT of calling a server API to perform the validation? that way it's guaranteed to return the same rules forever (even when they involve)
cc @coyotte508

WDYT of calling a server API to perform the validation? that way it's guaranteed to return the same rules forever (even when they involve)
cc @coyotte508

Agreed, similar to the preupload API call now available to determine if a file should be LFS (implemented in #888)

Actually @nateraw you can call POST /validate-yaml with in the body:

repoType: "model", "dataset" or "space"
repoId: string - does not have to be an existing repo

It returns either a 200 if there are warnings or nothing, and a 400 if there are validation errors.
The return value is a raw string, for now (unless you'd like something more elaborate). Please add the Accept: text/plain header in the request. It will work without for now, but it will help us not break anything later if we make changes. cc @Pierrci

I don't think it's a good idea to require internet while creating a modelcard. As a user that would come as a surprise to me. I'd rather have that logic implemented locally.
When working with the hub, internet requirement is reasonable, but that doesn't seem like something we could ask users for creating a model card.

I don't think it's a good idea to require internet while creating a modelcard.

Maybe we could display a message if lack of internet connection (eg "offline mode - model card metadata validation skipped") while still allowing creation if offline.

I think we should have the validation offline. If on top of that we want to have an online validation kinda thing, we can have a separate method which the user can explicitly call knowing that they're validating against the server. But even with something line pypi, one can run the validation using twine check w/o having to upload anything to pypi.
Model card validation on the server side should be very explicit and opt-in, otherwise users will be leaking information to us most of them w/o realizing they're doing it.

The issue is validation rules will change. (and new values will be accepted for licenses, ...)
On one hand we'd have to continually update the validation on the python side, which we maybe could automate (finding a way for python code & JS code to share logic).
But on the other hand users would need to update their HF library version - or periodically download an updated schema.
All that is possible but much more burdensome to implement IMO.

Then we can have a minimal validation logic in the library which should stay stable, and then when/if users upload to the hub, they'll get a meaningful error message which the library can show.
This comes from the fact that we have certain data which is hub specific in the model card, and the hub's backend does things with it. From a user's perspective, a model card should be a mostly free format and this library should help them create a nice one, rather than this library being a place where we help users create some metadata that the hub understands.
Mixing these two together will keep causing us issues.

I guess it's not clear to me what this does, then. If it's "just" free form text in sections, what does the library actually do? It's more like a template of a markdown file?

I think this discussion is now mixing the metadata vs rest of the content. The original discussion was about the metadata :) In any case, I would suggest the simpler approach and rather than duplicating (potentially changing) server logic in the library, just manage it through a call to the API. This will help avoid unexpected mismatches and will be easier to maintain.
A big goal of this library is to standardize across the library integrations, so I feel it's ok (even intended) if it has some bindings with the Hub. We have things to push to the Hub, I think metadata validation from the server falls under this scope.

The only time we really have to validate is when pushing to the hub, eh? Because that's when things would break.
So would it make sense to just validate via API call before pushing to the hub and provide nice error message if it breaks? It could still be at CardData.validate, but would only be called by the library when pushing to the hub because we can assume they should have internet access for that.


Android build fails

Environment
RN: 0.55.0
Platform: Android
Library version: ^5.6.0
Steps to Reproduce
Trying to build with react-native run-android suddenly does not work
Expected Behavior
Build to work
Actual Behavior
Build fails with an error:
Execution failed for task ':react-native-google-analytics-bridge:compileDebugJavaWithJavac'.
Many errors but all are the same thing, one example would be
error: package com.google.android.gms.tagmanager does not exist
import com.google.android.gms.tagmanager.TagManager;
Same for ContainerHolder & DataLayer.
^
It's like analytics from google play services suddenly stopped working?
Weird thing is I was working on this project whole morning - I go away from keyboard for 5 minutes and now I can't build anymore :/
These are my dependencies:
dependencies {
compile fileTree(dir: "libs", include: ["*.jar"])
compile project(':react-native-touch-id')
compile project(':react-native-svg')
compile project(':react-native-image-crop-picker')
compile project(':react-native-blur')
compile project(':react-native-splash-screen')
compile project(':react-native-restart')
compile project(':react-native-image-crop-picker')
compile project(':react-native-google-analytics-bridge')
compile project(':react-native-device-info')
compile 'com.stripe:stripe-android:5.1.0'
compile 'com.android.support:appcompat-v7:26.0.1'
compile 'com.android.support:support-v4:26.0.1'
compile 'com.android.support:multidex:1.0.1'
compile ('com.pulsatehq.sdk:PulsateSdk:2.10.2') {
exclude group: 'com.google.android.gms'
}
compile 'com.google.android.gms:play-services-location:+'
compile 'com.google.android.gms:play-services-wallet:+'
compile "com.facebook.react:react-native:+"  // From node_modules
}
compileSdkVersion/targetSdkVersion 26
buildToolsVersion '27.0.3'
Any ideas? Thanks

+1 Really annoying.

I'm having the same issue. My build was working previously but it had all sorts of errors today.

node_modules/react-native-google-analytics-bridge/android/src/main/java/com/idehub/GoogleAnalyticsBridge/GoogleAnalyticsBridgePackage.java:8: error: package com.google.android.gms.tagmanager does not exist
import com.google.android.gms.tagmanager.ContainerHolder;

/node_modules/react-native-google-analytics-bridge/android/src/main/java/com/idehub/GoogleAnalyticsBridge/GoogleAnalyticsBridgePackage.java:16: error: cannot find symbol
public GoogleAnalyticsBridgePackage(ContainerHolder containerHolder, String trackingId) {

compileSdkVersion 26
buildToolsVersion '26.0.2'

Same exact issue as posted by @matr1xprogrammer everything was working fine yesterday.

Same issue, started today also
compileSdkVersion 25
buildToolsVersion "25.0.1"

same issue for me , RN 0.44 , dependencies {
compile 'com.facebook.react:react-native:+'
compile 'com.google.android.gms:play-services-analytics:+'
}

This is what happens when you use + as a dependency version. This means "just give we the latest version, I don't care what it is". Apparently com.google.android.gms.tagmanager.TagManager used to be bundled with com.google.android.gms:play-services-analytics until today, when google has released play services 15.0.0, and this class is not there anymore. Now you need to include it as a separate dependency. Or if you substitute + (which now results in 15.0.0) with 12.0.1 (the previous version) everything works.
Seriously, don't use + as a version number, this is very bad practice... 12.0.+ is ok-ish but plain + is not...

@denisk20 Can confirm this is the case.  Changing the build.gradle in react-native-google-analytics-bridge to use 12.0.1 fixed the build issues for me.    Google just released 15.0.0 of play services. https://developers.google.com/android/guides/releases
'''def DEFAULT_GOOGLE_PLAY_SERVICES_VERSION    = "12.0.1" '''

You should be able to add googlePlayServicesVersion=12.0.1 to your android/gradle.properties - just confirming now

A workaround from @leecrossley is the easiest one and should be sufficient for most cases, one (very minor) drawback is that it adds googlePlayServicesVersion to all subprojects that you have. If you want to only add it to react-native-google-analytics-bridge you can do this in build.gradle
subprojects {
    if(name.equals('react-native-google-analytics-bridge')) {
        project.ext.'googlePlayServicesVersion' = '12.0.1'
    }
}

See https://github.com/idehub/react-native-google-analytics-bridge/issues/207#comment-381114538
Yes, I agree targeting + is bad practice. But, setting a specific version has historically had people running to this repo demanding I upgrade versions because some other lib they use did as well. Consequently there has been a ton of conflicting versions used.

I am having the same problem with iOS 🤔

we should move to this https://github.com/react-native-community/react-native-camera/pull/1380

hey @cbrevik,
Indeed, I got two issues as following,
Error:(13, 40) error: package com.idehub.GoogleAnalyticsBridge does not exist
Error:(44, 17) error: cannot find symbol class GoogleAnalyticsBridgePackage
My main build:
repositories {
maven { url 'https://maven.google.com' }
jcenter()
google()
}
dependencies {
classpath 'com.android.tools.build:gradle:3.1.1'
classpath 'com.google.gms:google-services:3.2.1'
}
repositories {
mavenLocal()
jcenter()
maven { url "https://jitpack.io" }
maven {
url 'https://maven.google.com'
}
maven {
url 'http://sdk.uxcam.com/android'
}
maven {
// All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
url "$rootDir/../node_modules/react-native/android"
}
google()
}
and tried above solutions, but didn't work out for me, can you please look into it?

Error:Failed to resolve: :react-native-google-analytics-bridge:
Open File
Error:(179, 13) Failed to resolve: :com.android.support:appcompat-v7
Show in File
Couldn't find any proper solution yet, can anyone help me on this?

the same error started when I build yesterday suddenly my project is like what the hell, it's worse when you are about to leave and you have no idea what just happened

got the solution  just add googlePlayServicesVersion=11.8.0 in your android/gradle.properties worked for me

hey @jigaryadav,
Can you share which distribution version are you using,
distributionUrl=https://services.gradle.org/distributions/gradle-4.?-all.zip?

@MaheshNandam  you must be using another service rather than react-native-google-analytics-bridge

why @jigaryadav,
I had already implemented this library before and its working fine, after updating distributionUrl to 4.4 then this line fails to upgrade compile 'com.android.support:appcompat-v7:26.0.2' due to this 'app getting crash'.

I have used @leecrossley 's solution to build again and it successfully built into my android device. However, it comes out another error: Could not invoke GoogleAnalyticsBridge.trackScreenView. Does anyone meet this error?

googlePlayServicesVersion=12.0.1 didnt work for me.

For me, googlePlayServicesVersion=12.0.1 didnt work.

sometimes the gms service lacks common.api classes and the app crashes, sometimes lacks tagmanager classes and the compile crashes..

Indeed, I faced the following issues - #238, #207, gradle issue, Failed resolution of: Lcom/google/android/gms/internal/zzjb, Google Play Services Dependency Issues, DexArchiveMergerException unable to merge dex, android crash, unable-to-merge-dex-after-upgrading-android-studio-3-0.
Finally below steps works for me,
Updated the following:
Android Studio - 3.1.1
classpath 'com.android.tools.build:gradle: - 3.1.1 (project/app/build)
classpath 'com.google.gms:google-services: - 3.1.1 (project/app/build)
distributionUrl=https://services.gradle.org/distributions/gradle-4.4-all.zip (at gradle-wrapper.properties)
compile 'com.android.support:appcompat-v7:27.0.3'
Added the following lines:
googlePlayServicesVersion=12.0.1 (at gradle.properties)
compile 'com.android.support:multidex:1.0.3' ( dependency at android/app/build )
compile 'com.google.android.gms:play-services-analytics:12.0.1' ( dependency at android/app/build )
compile 'com.google.firebase:firebase-messaging:12.0.1' ( dependency at android/app/build )
compile 'com.google.firebase:firebase-core:12.0.1' ( dependency at android/app/build )
before adding the below lines refer
 allprojects {
       repositories {
       //start here
       configurations.all {
resolutionStrategy.eachDependency { DependencyResolveDetails details ->
    def requested = details.requested
        if (requested.group == 'com.google.android.gms') {
           details.useVersion '12.0.1'
        }
         if (requested.group == 'com.google.firebase') {
            details.useVersion '12.0.1'
          }
        }
      }
     //end
      jcenter()
           maven {
            url "https://maven.google.com"
           }
        }
    }

Hope this helps, cheers 👍.

I have successfully build by doing this
    compile(project(':react-native-google-analytics-bridge')){
        exclude group: 'com.google.android.gms'
    }

    compile ("com.google.android.gms:play-services-analytics:12.0.1") {
        force = true;
    }

but crashes on launch with error Could not invoke GoogleAnalyticsBridge.trackScreenView

@MaheshNandam  worked for me! Thanks!

@MaheshNandam it seems only your solution works for me.. thank you

@MaheshNandam worked for me, also! Thank you

It doesn't work for me at all.... It s a big issue !

buildscript {
    repositories {
        jcenter()
        google()
        mavenLocal()
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:3.0.1'

        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files
    }
}

allprojects {
    repositories {
        configurations.all {
            resolutionStrategy.eachDependency { DependencyResolveDetails details ->
                def requested = details.requested
                if (requested.group == 'com.google.android.gms') {
                    details.useVersion '12.0.1'
                }
            }
        }
        mavenLocal()
        jcenter()
        maven {
            url "https://jitpack.io"
        }
        maven {
            url 'https://maven.google.com'
        }
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url "$rootDir/../node_modules/react-native/android"
        }
    }
}

subprojects {
    afterEvaluate {project ->
        if (project.hasProperty("android")) {
            android {
                compileSdkVersion 26
                buildToolsVersion '26.0.2'
            }
        }
    }
}

add the flowing code to android/build.gradle work for me .
allprojects {
repositories {
//start here
configurations.all {
resolutionStrategy.eachDependency { DependencyResolveDetails details ->
def requested = details.requested
if (requested.group == 'com.google.android.gms') {
details.useVersion '12.0.1'
}
if (requested.group == 'com.google.firebase') {
details.useVersion '12.0.1'
}
}
}
//end
jcenter()
maven {
url "https://maven.google.com"
}
}
}

I'll leave this issue open a while for people too see. A pull request for supporting Play Services 15 would be very welcome. @cbrevik and me have no need for the upgrade right now, and won't be spending our already limited time on it (at least not until we need it in our own apps) 👍

Thank you @qingfengming, much appreciated 💯

Thanks @MaheshNandam !
It worked for me changing the appcompat version to 27.0.2

@MaheshNandam works for me in combination with react-native-onesignal

I ran into this issue as well, just wanted to give a heads up that if you've been using Google Play Services version 10.0.1 (like I have), you can fix this issue without having to upgrade to using Google Play Services version 12.0.1.
In my /android/build.gradle file I added the following:
allprojects {
  repositories {
    configurations.all {
      resolutionStrategy.eachDependency { DependencyResolveDetails details ->
        def requested = details.requested
        if (requested.group == 'com.google.android.gms') {
          details.useVersion '10.0.1'
        }
        if (requested.group == 'com.google.firebase') {
          details.useVersion '10.0.1'
        }
      }
    }

    // Other pre-existing configuration for `allprojects.repositories` here.
  }
}

subprojects {
  if(name.equals('react-native-google-analytics-bridge')) {
    project.ext.'googlePlayServicesVersion' = '10.0.1'
  }
}

And now my app builds again, and appears to work normally when using the app.
Here's some other configuration snippets that may or may not be relevant to getting this solution working.
In /android/app/build.gradle:
android {
  compileSdkVersion 26
  buildToolsVersion "26.0.2"

  defaultConfig {
    minSdkVersion 16
    targetSdkVersion 22
  }
}

dependencies {
    compile(project(':react-native-google-analytics-bridge')){
        exclude group: 'com.google.android.gms'
    }
    compile (project(':react-native-fcm')){
        exclude group: "com.google.firebase"
    }
    compile "com.google.firebase:firebase-core:10.0.1" // This decides your firebase SDK version
    compile fileTree(dir: "libs", include: ["*.jar"])
    compile ('com.android.support:appcompat-v7:26.0.0') {
        exclude group: "com.google.android.gms"
    }
    compile "com.facebook.react:react-native:+"  // From node_modules
    // Base Play Services
    compile ("com.google.android.gms:play-services-base:10.0.1") {
        force = true;
    }
    // Required by react-native-google-analytics-bridge
    compile ("com.google.android.gms:play-services-analytics:10.0.1") {
        force = true;
    }
    // Required by react-native-fcm
    compile ('com.google.firebase:firebase-core:10.0.1') {
        force = true;
    }
    // Required by react-native-fcm
    compile ('com.google.firebase:firebase-messaging:10.0.1') {
        force = true;
    }
}

If you have to use play services 15+ you will need to add:
com.google.android.gms:play-services-tagmanager-v4-impl:15.0.2

I just changed the order of the repositories in the root level build.gradle.
It was like this before
allprojects {
repositories {
    mavenLocal()

    jcenter()

    maven {
        url "$rootDir/../node_modules/react-native/android"
    }

   maven { url 'https://maven.google.com' }

    maven { url "https://jitpack.io" }

}

}
After getting the play services error ,  moved the google maven before the jcenter fixed the issue for me.
allprojects {
repositories {
    mavenLocal()

    maven { url 'https://maven.google.com' }

   jcenter()

    maven {
        url "$rootDir/../node_modules/react-native/android"
    }

    maven { url "https://jitpack.io" }

}

}

Hi guys, this should be fixed in 6.1.0 without any hacks needed.
See release notes for more info.
Would be great if anyone could could confirm that it works for them using this version.
Important: For those already on less than v6 of this library, there are some breaking changes when upgrading to v6+ (if you use ecommerce or custom dimensions). See wiki post for more details.

Following the Manual installation worked for me!

@chenop can you please provide some documentation on how you got it to work?
I currently followed Manual installation and builds without any issues on iOS but fails on Android. The follow error is returned: Compilation failed; see the compiler error output for details. -> Failed to resolve: com.google.android.gms:play-services-gcm:16.0.3 when i try to install it, i get Could not find dependency "com.google.android.gms:play-services-gcm:16.0.3" Thoughts?
@cbrevik

Wiki: https://github.com/idehub/react-native-google-analytics-bridge/wiki/Android-build-problems
Or similar issue: https://github.com/idehub/react-native-google-analytics-bridge/issues/280#issuecomment-420906795
I am locking this thread. For anyone coming here in the future; please search the issues first. This has been answered quite a few times now.


Add support for 24-bit colors with +termtruecolor flag

Enables 24-bit truecolor support in terminal vim. Requires patched vim from ZyX (patch here) and supporting terminal.
Requires the following in vimrc:
if has('termtruecolor')
  let &t_8f="\033[38;2;%ld;%ld;%ldm"
  let &t_8b="\033[48;2;%ld;%ld;%ldm"
  set guicolors
endif

More information about 24-bit truecolor support in terminal environments can be found here

awesome, thanks!


Added link to Ruby Gem by @nileshtrivedi

I added a link to the bigchaindb Ruby Gem by @nileshtrivedi
on the Drivers & Tools page.

Codecov Report

Merging #1737 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1737   +/-   ##
=======================================
  Coverage   98.58%   98.58%
=======================================
  Files          59       59
  Lines        2896     2896
=======================================
  Hits         2855     2855
  Misses         41       41

Maybe also add it to the getting started page cc @kremalicious

@diminator yes, adding a new tool or driver to Get Started page should be your first thought instead of adding to docs. Way more traffic on Get started page then on docs, so much more impact:
Last 30 days, visits to Drivers & Tools docs page: 267
Last 30 days, visits to Get Started page: 3,936

https://github.com/ascribe/bigchain-website/pull/171

The number of page views should not be used as a measure of "impact."
Even if the number of page views is 1, if that one person is the right person, that one page view might have enormous consequences.
In short, not all page views are equal impact.


CreateMachine should allow for user supplied Ids

Prerequisites

[x] Checked that your issue isn't already filed.

Description
Development teams with existing Id schemes want to use REstate Machines, not update their data model to track the auto-generated MachineId, but instead use their existing Ids in Machines.

REstate Package Versions
master
Additional Information

I want tests around existing functionality before adding a new full-stack feature like this.
Related PRs: #19


x86_64-linux-gnu-gcc' failed with exit status 1

I am getting below error while executing setup.sh
compilation terminated.
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1
If I ignore this - will the configurations still work?
----------------------------------------
Command "/usr/bin/python -u -c "import setuptools, tokenize;file='/tmp/pip-build-XV7J3Q/cryptography/                                                                                setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(                                                                                compile(code, file, 'exec'))" install --record /tmp/pip-NlG22t-record/install-record.txt --single-ver                                                                                sion-externally-managed --compile" failed with error code 1 in /tmp/pip-build-XV7J3Q/cryptography/
/usr/local/lib/python2.7/dist-packages/pip-9.0.1-py2.7.egg/pip/_vendor/requests/packages/urllib3/util/ssl                                                                                _.py:122: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from                                                                                configuring SSL appropriately and may cause certain SSL connections to fail. You can upgrade to a newer v                                                                                ersion of Python to solve this. For more information, see https://urllib3.readthedocs.io/en/latest/securi                                                                                ty.html#insecureplatformwarning.
InsecurePlatformWarning

The error you posted says, You can upgrade to a newer version of Python to solve this.
Did the error persist after you did that?


[2.0.0.-alpha] Bug in

Vue.js version
2.0.0.-alpha
Reproduction Link

https://jsfiddle.net/ncLrLh6z/6/
Steps to reproduce
Click 'Get' button.
What is Expected?
no js error
What is actually happening?
[Vue warn]: Error during component render (found in root instance)
console.error('[Vue warn]: ' + msg + (vm ? formatComponentName(vm) : ''));
vue.js (line 2168)
TypeError: value.indexOf is not a function
option.selected = value.indexOf(getValue(option)) > -1;

The error appears because you initialized b as an emtpy tring in your data: b: '', but you use it as an array in v-for: v-for="value in b". Initialize it as an empty array instead, then v-for has nothing to complain about.
Working version wiht b: [] https://jsfiddle.net/Linusborg/ncLrLh6z/9/

Ok, I havbe to correct myself:
The reason is not the iitial value of b (though initializing it as an empty array would be better), the problem is the initial value of make:
Since the <select> has the multiple attribute, make should be an array.
Working jsfiddle: https://jsfiddle.net/Linusborg/ncLrLh6z/12/
So this is still not a bug, technically, but I will re-open the issue to evaluate how to improve this, mabe with a Vue-Warn in dev mode.


Migrate to use the Fixie Voice SDK.

This PR updates app/agent/page.tsx to use the Fixie VoiceSession API instead of the hand-rolled stuff currently in there.
To make this work, I had to adjust the initialization logic to more closely match what we do in HiSanta, since we've changed how the startup is done.
I made the start/stop button the only way to toggle the states, rather than tapping anywhere, just to simplify the code a bit.
Happy to change this up. Thinking it might make sense to do some tweaks to the UI so it's a more useful debugging tool without needing keyboard shortcuts and URL path editing.

Justin is going to do this another way


How to generate continous event stream

I am trying to generate a continous event stream with this library.
I get this to work by using an endpoint which has to be called by the client which looks like:
@app.route('/api/get/statusupdate')
def status_update():
    controller = TestController()
    while True:
        sse.publish(controller.get_status(), type='greeting')
        time.sleep(5)

I am configuring the plugin with this:
app.config["REDIS_URL"] = "redis://localhost"
app.register_blueprint(sse, url_prefix='/stream')

Now I can call /stream in the client to listen to events, but I need to call /api/get/statusupdate first which is a workaround. Is it possible to do this withiout calling /api/get/statusupdate and directly output some data on the event endpoint?

@HR-KSSK Why not just have threads or a background scheduler that calls your method in your flask app periodically?

@HR-KSSK Why not just have threads or a background scheduler that calls your method in your flask app periodically?

Thanks for your reply - that's exactly what I'm doing right now. I think the use-case of this library does not match what I needed to achieve.


refine velaux addon

Velaux addon is an example to tell user how to use context.metadata in addon. So refine addon to use new feature of vela-1.5.0.
Signed-off-by: 楚岳<EMAIL_ADDRESS>
Description of your changes

How has this code been tested?

Checklist

I have:

[ ] Title of the PR starts with type (e.g. [Addon] , [example] or [Doc]).
[ ] Updated/Added any relevant documentation and examples.
[ ] New addon should be put in experimental.
[ ] Update addon should modify the version in metadata.yaml to generate a new version.

Block this PR before we release the 1.5.3.


Increase parallel iterator buffers to improve group by query speed

This increases the result buffers size used by the parallel iterators to allow more concurrency in certain group by queries.
In my test dataset spanning 2yr and ~1000 queries such as:
SELECT mean(value) FROM stress.autogen.cpu WHERE time >= '2015-01-29T00:00:00Z' AND time < now() GROUP BY time(24h)

Latency improved from from 54s to 4.5s on an m4.16xlarge and memory usage (RSS) stayed the same at ~3.8GB.  This instance type has 64 cores which allowed them all to be utilized instead of just 1 pegged at 100%.

@jsternberg Removed the protobuf change.  That was an unintended.


Improve doc formatting of properties

The auto-generated docs are starting to look a bit messy using Sphinx built in property/attribute tags. It is strange that "Required" and "Parameters" are on different lines, then the immutable "attributes" are formatted differently (see screenshot below). I think we need some more customized formatting that doesn't use the builtin property/attribute tags.

Yeah, that is a lot more streamlined and consistent. Nice.

Yeah, that is a lot more streamlined and consistent. Nice.

See #119


API spec fixes

Fix #29, OCF server retrieve steps

@nagineni @poussa  could you please review? We'd need to merge this.

OCF LGTM

API versioning will be added in a different PR, as discussed with @gabrielschulhof

@zolkis I was expecting a different PR with API versioning to master before merging this PR and a new version number with the changes in this PR.

There will be bigger changes than just that, and merging would have been hell.
Still, this API isn't "official" yet, outside of iotivity-node.
Check my fork for what's coming up - comments are welcome there, too.


the performance of detecting face image is very terrible!!!!

I put the RGB face image from CelebA-HQ into demo file, but the result is very terrible so that I can not judge whether it is a face image or not.

This project is for document detection instead of face detection.


Move readme from starter module into it's content module and update accordingly.

spring-cloud-azure-starter-xxx should not have readme or changelog.
Current readme / changelog should move to implementation artifact..
For example: spring-cloud-azure-starter-oauth2-client should not have readme or changelog.
But  spring-security-azure-oauth2-client should have readme and changelog.
All starters should check whether it need to enrich readme in implementation artifact:

[x] #24357
[x] #24384
[x] #24385
[x] #24386
[x] #24387
[x] #24388
[x] #24389
[x] #24390
[x] #24391
[x] #24392

Refer to this PR when move readmes: #23928

Closing this issue, because all sub-issues have been closed.


libgroove cannot be compiled with Visual C++

I try to compile libgroove with Visual C++ 2015 but it can't compile the code because pthread.h (Unix specific library) and stdatomic.h (VC 2015 seems to support atomic only for C++ , but without C specific header) files are missing.
Do you plan to change libgroove to be compilable with VC 2015 (as libsoundio is) in the 5.0 milestone?
Thanks.

Andrew, can you ear me?

Hello, yes it is planned to be able to build libgroove with VC 2015 in the 5.0.0 milestone. Progress has been slow though...

Hello. Great news!
When do you think to be able to release the v5?
Thanks you very much.

Hi,
There is no major commit in libgroove since november 2015.
Do you still work on this project?
Thanks.


HTTP Response Status Exceptions

Hi all,
I would like to throw exceptions for several http response codes (in slim 3). This would be very useful for preventing code duplication.
Example:
if ($id <= 0) throw new BadRequestException('Id must be gt 0');

Is there a way to do this in slim without creating a custom middleware? Is there any existing middleware for this? (I wasn't able to find one).
Thanks right now.
Best Regards
mschop

Not sure what you mean. You have to make a custom exception handler to set the response code appropriately on the response object after it catches the exception.


Proxy within container only works with proxy ip, rather than hostname

Description of problem:
When behind a corporate proxy, http://proxy.company.com:8080/, I can set the
proxy variables for the docker daemon on my host machine. However once in a
container, ubuntu:14.04, attempting to set the proxies with
export http_proxy="http://proxy.company.com:8080/"
export https_proxy="http://proxy.company.com:8080/"
export HTTP_PROXY="http://proxy.company.com:8080/"
export HTTPS_PROXY="http://proxy.company.com:8080/"

or when building a container with
ENV http_proxy 'http://proxy.company.com:8080'
ENV https_proxy 'http://proxy.company.com:8080'
ENV HTTP_PROXY 'http://proxy.company.com:8080'
ENV HTTPS_PROXY 'http://proxy.company.com:8080'

Does not work. This is using the --net=host option with docker run.
I could not figure out how to give the build access to the host network.
The trick I found was to ping proxy.company.com from my host and get the ip
then set the proxies from within my container like so.
ping proxy.company.com
PING proxy.company.com (12.34.567.898) 56(84) bytes of data.
^C

docker run --net=host -t -i ubuntu:14.04 /bin/bash

export http_proxy="http://12.34.567.898:8080/"
export https_proxy="http://12.34.567.898:8080/"
export HTTP_PROXY="http://12.34.567.898:8080/"
export HTTPS_PROXY="http://12.34.567.898:8080/"

docker version:
Client version: 1.7.0
Client API version: 1.19
Go version (client): go1.4.2
Git commit (client): 0baf609
OS/Arch (client): linux/amd64
Server version: 1.7.0
Server API version: 1.19
Go version (server): go1.4.2
Git commit (server): 0baf609
OS/Arch (server): linux/amd64
docker info:
Containers: 11
Images: 26
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 48
Dirperm1 Supported: false
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.13.0-55-generic
Operating System: Ubuntu 14.04.2 LTS
CPUs: 4
Total Memory: 5.718 GiB
Name: Graphics
ID: TFMJ:I23E:WUXS:ZF5X:5NJ5:CVHX:F2JN:XUH5:X7GD:EGLU:4IXQ:UPRP
Http Proxy: http://proxy.company.com:8080/
Https Proxy: http://proxy.company.com:8080/
Username: pdxjohnny
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
uname -a:
Linux Graphics 3.13.0-55-generic #94-Ubuntu SMP Thu Jun 18 00:27:10 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
Environment details (AWS, VirtualBox, physical, etc.):
physical
How reproducible:
Every time
Steps to Reproduce:

Start container
Set proxies
apt-get / ping anything

Actual Results:
apt-get says:
Can't resolve hostname proxy.company.com
when updating
Expected Results / What would work best:
If it would resolve the hostname proxy.company.com and find the ip
of the proxy itself
Additional info:

Can you do DNS lookups in the container? That sounds likely to be your issue.

Yeah, this sounds like a DNS problem, not a problem with Docker, especially Docker itself isn't really doing anything here.
Closing this for now since it really doesn't look like a Docker issue.
Feel free to ping on IRC or docker-user ML for help!


✨Image Backups API endpoint

👋 Hi, first time contributor here.
I've been using Ghost for years and one small feature I've always felt was missing, is an image exporter on self-hosted installations, that could potentially compliment the JSON Content exporter. Of course, one could just ssh onto a server and manually zip + download via sftp or something, but it feels like extra steps to achieve something that one would expect to be in the settings.
I challenged myself to build it, as it would be a great way to familiarise myself with the Ghost codebase and as a gateway to perhaps get more involved in open-source.
Last week I dived right in and have a working alpha prototype up and running, that's ready for PR.

[X] There's a clear use-case for this code change, explained below
[x] Commit message has a short title & references relevant issues (I hope I did)
[X] The build will pass (run yarn test:all and yarn lint)

I added an Export button in the Admin client as well under Labs -> Alpha features.
Added PR for that here https://github.com/TryGhost/Admin/pull/2362

A next iteration of this would be perhaps to make it a multithreaded task, to avoid it timing out when it's a big archive.
I only tested it with around 50mb worths photo content, on an M1 Pro. Most cloud servers aren't that powerful and production entities will have much more content to compress.
And then finally, perhaps an uploader that does the reverse.

Hey there @ronaldlangeveld 👋 I really appreciate your time, effort & enterprise with getting stuck in here.
I think you've touched on the main reason why this feature doesn't exist already: the image folder could be infinite in size and we have to make absolutely certain that a backups endpoint couldn't take a site offline as that rather defeats the object 😬
This means that the code to do this needs to be infallible and non-blocking. We'd probably need to generate the backup in a job & provide a download link in an email, rather than doing this inside an HTTP request. There are talks of restructuring the main importer this way too & it is also how the members importer was built.
There was an extensive discussion of implementing a backup feature into Ghost-CLI here: https://github.com/TryGhost/Ghost-CLI/issues/468 although it died. This discussed the performance benefits and complexities of shelling out to tar on the server.
As a last note, there are additional files that would be needed inside of a backup or archive as Ghost now has folders with media & files as well as a few json/yaml files dotted around, although it depends slightly on what the definition of the backup is.
I don't say any of this to discourage you or dismiss your work and I completely agree it'd be great to have an official and single-step method for generating a ghost backup, however, there's a lot to think about here and I don't want you to spend more time going in a direction that we wouldn't merge.
Hope this gives you some food for thought 😄

Hi @ErisDS
Thank you so much for the detailed feedback, much appreciated. 😊
While building it, I did have a gut feeling (that my curiosity ignored 😅) that there's probably informed reasons by the team for not having a feature like this already - and you've summed it up perfectly.
Was fun building a prototype regardless. 😊

If this was something we could do in a blocking HTTP request then this implementation would be spot on 👌 would just be asking for a test 😬
I'll close this for now. If you get inspiration to build something non-blocking by all means open a fresh PR and I'll take a 👀


Optus Sports -  Australia

Optus Sports epg seems to have disappeared from the Australian EPG.
Anyway to update it?

Could you be more specific about which guide you're talking about?

The epg guide to Optus Sports (from Australia) was on https://iptv-org.github.io/epg/guides/au/ but has seemingly disappeared.

I'm sorry, but I can't help you until it's clear which site we're talking about.


test: improve error output of router_check

Description: Explains what the different values output means and writes to stderr
instead of stdout. This should make the error output appear in the
testlogs.
Risk Level: Low, change to test error reporting
Testing: bazel test //test/...
Docs Changes: n/a
Release Notes: n/a

Updated this PR with your suggested fix, ptal.


Issues with the landscapist-bom package - missing coil3 / cannot use platform()

Hi! I just realized that you also provide a bom immediately tried to use it. Unfortunately, there are currently two issues with it:

The bom is missing landscapist-coil3
The bom artifact is missing <packaging>pom</packaging> and there is also a jar, which is unnecessary.
The default way of importing a bom implementation(platform("com.github.skydoves:landscapist-bom:2.4.4")) does not work here.
You can find an example of a correctly working bom artifact in the coil maven repo

Nevertheless, this is a great library. Thanks for all the great work 🙂

Hey @kroegerama, a new version 2.4.5 has been released and coil3 is available with the bom package. Thank you for raising this issue!


Add limited support for bundled signatures of a later JDK version than currently executing

As we do not yet support forking in the Maven/Gradle task and using Toolkits, it is not always possible to run checks against class files of a later version than the current Gradle/Maven runtime.
The code should in addition to #207 do the following:

print a warning that scanning class files of later version than current runtime may not find all issues
disable unknown signatures silently

Actually this make sno sense, because if you run with a newer verson, the JDK related bundled sigantures are never available.


Kelvin - Add new routes, serialization, validations

Changes Made:

Added routes for Session, Goals, Training, Employee Training, and Leave approval.

Testing:

Verified the features on localhost.

Additional Notes:

Conflicts were encountered during the merge, working on fix.

Looks good, merging


Error in TLS Handshake

Hi Standa, hope you do not mind creating an issue for a quesion I have regarding your pg2parquet tool. I would love to export my data from PSQL to Parquet what your tool offers. However, when trying to use it, I get the following error:
DB connection failed: error performing TLS handshake: The certificate's CN name does not match the passed value. (os error -2146762481)
I have tried all SSL modes but without any results. Do you have any suggestions how to fix this?
Thanks in advance,
Hans

Hi Standa, the issue is not relevant anymore. In the original request, I used the IP number of the PSQL server. However, when using the DNS name, I did not get a TLS error. So all good and sorry for bothering you.

No problem! Yes, you have to use the domain name which is in the servers certificate, otherwise TLS cannot verify that the certificate is authentic


Sample client and generators for built-in queries/languages.

Fixes #137 and #509.

Sample client app.
TopiaryQuery generators for each built-in language/query.

I suppose this paves the way or removing the compile-time configuration of the directory which contains the (installed) query files, doesn't it?

I think this would work nicely for the nickel format use case, except that the location of the languages folder may produce the same issue as #511.

Hmm, I wonder if my sample works only because I use a path dependency, and I happen to have the query files where the compiler expects to find them. I need to investigate.

#511 would only be an issue with external Nix builds, I think.

cargo package --list is our friend. Interestingly, it shows that files like languages.toml and all our sample test files are included, but the query files are not.

OK, I see what's going on. cargo package --list is checking each of our workspace members. But the languages dir is outside of these, so it doesn't get included. I will try adding an explicit include now. Which makes me think we can do the same with that languages.toml from #511, and drop the symlink.

Although it looks like we cannot include files from parent directories, so the symlink route is the way to go anyway:
https://users.rust-lang.org/t/including-files-from-parent-directory-in-package/88969

So the PR should be fine now.

I suppose this paves the way or removing the compile-time configuration of the directory which contains the (installed) query files, doesn't it?

Hmm, not directly. Because Topiary doesn't itself use these convenience generator functions for now. It could, but we already have some more complex logic for looking up query files. We need to keep this, because it allows users to override the built-in queries.
/// Convert a Language into the canonical basename of its query file, under the most appropriate
/// search path. We test 3 different locations for query files, in the following priority order,
/// returning the first that exists:
///
/// 1. Under the `TOPIARY_LANGUAGE_DIR` environment variable at runtime;
/// 2. Under the `TOPIARY_LANGUAGE_DIR` environment variable at build time;
/// 3. Under the `./languages` subdirectory.

Actually, making sure the languages directory is included in the crate, the same way we handle languages.toml, is the real fix of #509. The convenience functions are just for, well, convenience, and will likely get dropped when we support dynamic loading of grammars (#4).
Maybe it's best to drop them right now. I won't object to that.
@vkleen Can you check if this branch helps you, even without using the new convenience functions?

Building with crane and Nix fails with the branch as it is. Crane ends up copying dangling symlinks into the cargo vendor directory. Maybe that is an issue that should be addressed in crane upstream. Once topiary makes its way to crates.io, I imagine it will work fine. As you say, cargo package replaces the symlinks with copies of the actual files for publishing.
Just putting the query files into the package sources for cargo wouldn't help with accessing those files in nickel format, I think. There would still need to be either the helper functions like topiary::nickel() or just a global static string with include_str!() to get the right queries.

Building with crane and Nix fails with the branch as it is. Crane ends up copying dangling symlinks into the cargo vendor directory. Maybe that is an issue that should be addressed in crane upstream. Once topiary makes its way to crates.io, I imagine it will work fine. As you say, cargo package replaces the symlinks with copies of the actual files for publishing.

I was trying to be clever, and switched the direction of the symlinks, so the real files could stay in root. But it sounds like that's what breaking Crane. Please try again now.

Just putting the query files into the package sources for cargo wouldn't help with accessing those files in nickel format, I think. There would still need to be either the helper functions like topiary::nickel() or just a global static string with include_str!() to get the right queries.

I see. Then we'll keep them. But I wonder what the scalable, future solution should be when at some point we consider languages something that is loaded dynamically. @ErinvanderVeen

The nix builds succeed with the switch symlinks 👍

It will sure be interesting to see how we solve this in the future!


调用dispose方法后闪退

平台：windows11-23H2
新开一个页面展示webview，退出界面时调用dispose方法，界面退出后应用会未响应，然后闪退。不调用dispose方法则无问题，此问题必现。关键代码如下：
@override
void dispose() {
_controller.dispose();
super.dispose();
}

同一个问题，关联 #96

Hi, every one. I've merged multi instance and multi window support branch to main. There is a lot of problems might have been resloved. Could you guys help us to verify what we did success and what new problem brought in? @Chihiro23333


Add ExtensionType for uuid and map to parquet logical type

Rationale for this change
It would be nice to better support reading and writing the canonical uuid extension type with the arrow and parquet crate i.e. mapping between the arrow extension type and the parquet logical uuid type.
What changes are included in this PR?
This adds an ExtensionType enum for canonical extension types.
Are there any user-facing changes?
Users can now annotate their logical types with extension types, and for uuid they are propagated via the arrow writer to map to the parquet uuid logical type.
This needs tests and better docs, but I'd like to get some feedback on the approach first, because there are many different ways to implement this.
I quickly tested this change with narrow and those uuid fields (in the parquet file) are now picked up as uuid instead of blob by DuckDB.

Maybe ExtensionType could be a trait to be externally implementable and not limited to canonical extension types?

I haven't had time to work on this, but I'm planning to pick this up later.

Thanks @mbrobbel  -- marking this PR as draft as I think it still has planned but not yet completed work

Thanks for the PR. This seems very useful to support not yet mapped logical types. e.g. json

I updated the PR to define a trait for extension types instead. Ready for another round of feedback.


how can i get curl_getinfo() response array?

i need to get curl_getinfo response info

Please see #1798


symfony 4.3 incomaptible composer

is this bundle really incompatible with symfony 4.3?
otherwise the composer.json is very restrictive and it is not possible to run composer with symfony 4.3.
`Problem 1
- Installation request for borsaco/telegram-bot-api-bundle ^1.0 -> satisfiable by borsaco/telegram-bot-api-bundle[1.0.0].
- borsaco/telegram-bot-api-bundle 1.0.0 requires symfony/framework-bundle ^3.4 -> no matching package found.
Potential causes:

A typo in the package name
The package is not available in a stable-enough version according to your minimum-stability setting
see https://getcomposer.org/doc/04-schema.md#minimum-stability for more details.
It's a private package and you forgot to add a custom repository to find it`

current framework-bundle is 4.3.2

#4

Solved


Rendering scrollableTabView blur TextInput from container

Hi!
I'm making an app and in a Scene, I have a container with a TextInput. When I focus on the TextInput, I change a state which suppose to display a ScrollableTabView. However, when the container re-render to add the ScrollableTabView, the TextInput blur and doing so, the ScrollableTabView disapears....
Why does it remove focus on input?

Could you please make a video?

Any news about a possible fix?

Do you want to say that if you remove scrollableTabView then TextInput doesn't blur? I don't believe that scrollableTabView is guilty in that.

Here's the same component but one has the scrollableTabView and the other one hasn't. The logic is the same. The only difference is that when the scrollable is rendered, it blurs the input.
So yes, when i remove the scrollableTabView, then the TextInput doesn't blur

I'm facing exactly the same issue (only when there's a <ScrollableTabView /> ). Any update/solution for it?

Any Solutions ? Happens to me too


Recreating a deleted repository gives error 500.

Create a repository with a name (let's say "myrepository")
Push something to "myrepository"
Delete "myrepository" on Gogs
Try to create another repository called "myrepository"

It returns 500. I think this happens because the repository folder is not deleted, just the database registry. I tried to rename the folder and could create the repository again.

Thanks your feedback!

What is your Gogs version?
Which database are you using?

Sorry, I forgot these information.
Gogs version: 0.6.15
Windows Server 2012
PostgreSQL 9.4
2015-10-06 15:35 GMT-03:00 无闻<EMAIL_ADDRESS>
Thanks your feedback!

What is your Gogs version?
Which database are you using?

—
Reply to this email directly or view it on GitHub
https://github.com/gogits/gogs/issues/1738#issuecomment-145958136.

--
Andrey Nering
<EMAIL_ADDRESS>Skype: andrey.nering

Thanks for following up!

It returns 500. I think this happens because the repository folder is not deleted, just the database registry.

Can you 100% reproduce this problem by doing the same thing again?
And check if there is any error in the log(as far as I know, Gogs only prints an error if there is something wrong with delete files, could be a permission issue).

You are right: the log show a permission issue on deleting the repository.
Log here: https://gist.github.com/andreynering/99e1db9fc9a6c9a2f5b8
Also, is pull requests supposed to be working already? It is giving me
error 500 on merge, too. But I didn't check the log.
2015-10-06 15:48 GMT-03:00 无闻<EMAIL_ADDRESS>
Thanks for following up!
It returns 500. I think this happens because the repository folder is not
deleted, just the database registry.
Can you 100% reproduce this problem by doing the same thing again?
And check if there is any error in the log(as far as I know, Gogs only
prints an error if there is something wrong with delete files, could be a
permission issue).
—
Reply to this email directly or view it on GitHub
https://github.com/gogits/gogs/issues/1738#issuecomment-145961915.

You are right: the log show a permission issue on deleting the repository.
Log here: https://gist.github.com/andreynering/99e1db9fc9a6c9a2f5b8

Great, but I can't help with permission issue so may be reloaded somewhere that Gogs can have full control of.

Also, is pull requests supposed to be working already? It is giving me
error 500 on merge, too. But I didn't check the log.

It's kind of working, with very limited functionality, I'm still working on it, so things of PR are subject to change.

Thank you for the attention.
I will let you know if I find more information.

No problem :smile:

Any updates?

Sorry, but I still didn't find a solution for this.

@andreynering
I think you already pointed out the problem:

You are right: the log show a permission issue on deleting the repository.

That is something Gogs can't fix itself.

I know what is the problem. I don't know how to fix it. ¯_(ツ)_/¯

I don't know either....
And I was asking:

Can you 100% reproduce this problem by doing the same thing again?

Can you 100% reproduce this problem by doing the same thing again?

Yes, just tried again today on latest release.

@andreynering I still insist that you need to check something like delete permission, maybe you put repos in a admin-level directories and so Gogs can't delete anything unless it is ran as admin.

Tomorrow I will try change the folder permission. I already tried another
Windows user and had the same issue. I'll post here if it work or not.

@andreynering OK, maybe try to put in ~/Downloads directory other than home directory (Gogs default).

I was running Gogs with COMPUTERNAME$ as user.
Tried give everybody "full control" permission on the repositories folder.
No luck.
Then tried run gogs as MYUSER and copied the folder to
MYUSER/Downloads. No luck.
Both printed the same error on log: permission denied.

Can you rename a repository?

Yes.

Please post the complete error message here.

Also, can you help check the repo files are not read-only?

Found something relative... https://github.com/golang/go/issues/9606

Please post the complete error message here for permission denied with latest version you tested.

2015/11/03 08:16:28 [W] delete repository files[D:\xxx\gogs-repositories\andreynering\teste.git]: remove D:\xxx\gogs-repositories\andreynering\teste.git\objects\19\8d5d252f4ac328120d476a5592b94aa13677c8: Access is denied.

Also, can you help check the repo files are not read-only?

Yes, all files are read-only.

Found something relative...

Hmm... Maybe Gogs should write files as 0666?

Yes, all files are read-only.

Combine with golang/go#9606 , we know t he problem now...
But objects files are wrote by git itself, not by Gogs...

I am suffering from this, as well, and it might prevent us from using gogs as our enterprise source control solution. :( I can't tell my supervisor we need a *nix machine for the service -- we only run Windows.

@haliphax too bad... any special hack/command could possibly force delete files on Windows (when run Gogs as admin)?

Sounds like a working workaround?

cmd := exec.Command("cmd.exe", "/C", "rmdir", "/S", "/Q", dirPathWithWindowsSeparators)

I am using it as admin, and yes, using the /s /q flag with rmdir would do the trick.

@haliphax I'll try it. Thanks!

This ever make it in?  Having the same issue myself - I have to manually delete the repo from the working directory.  Just wanted to note another (perhaps more prevalent) usecase.
1.) Fork repo "ABC"
2.) Do work, submit PR ...
3.) Delete Forked repo
4.) Realize you need to make another change
5.) Try to fork repo again -> 500 error

@Tim91084 does this comment work on your system?

@Unknwon sorry for my ignorance, but what should I do with that line?  Do you want me to recompile the gogs source with that change, or just test the command in command prompt?
If recompiling the source, could you point me in the general direction where I would make that change?

@Tim91084 No.. i just want your help to verify that command works on your Windows shell/xyz.

D:\>rmdir test
The directory is not empty.

D:\>rmdir /?
Removes (deletes) a directory.

RMDIR [/S] [/Q] [drive:]path
RD [/S] [/Q] [drive:]path

    /S      Removes all directories and files in the specified directory
            in addition to the directory itself.  Used to remove a directory
            tree.

    /Q      Quiet mode, do not ask if ok to remove a directory tree with /S

D:\>rmdir /S /Q test

The /S /Q args seem to mitigate the issue or removing a directory that is not empty.  If that is the root cause of this issue, it's likely that adding those arguments will fix it.

Can anyone follow this thread help test develop branch as we merged a fix 2570?

@andreynering Any time you could try the fix? https://github.com/golang/go/commit/2ffb3e5d905b5622204d199128dec06cefd57790

@Unknwon
I compiled Gogs with the master branch of Go compiler and without the workaround I did to fix this. I can confirm it is fixed, it can delete read-only files/folders. I recreated the same repository + wiki multiples times without any error (I tried with Go 1.6 right before and it still errored without the workaround).
But we still have to wait to remove the workaround. The fix will be in Go 1.7, and Gogs supports, I think, the 3 lasts Go versions?

@andreynering Yeah, we should add a note when Go 1.9/1.10 released?

@Unknwon I think it's a good idea. A comment with a note and the link to the commit.


Basic code for handling movie classifications

Build implementation code for handling movie classifications, including categories, genres, subgenres.
Skip for now: subjects (non-fiction movies), certifications (movie ratings)

Primary commit.
https://github.com/cjcodeproj/medialibrary/commit/37ef564b32431a23b9a14c6c2a280f9474f1213a


Too much magic, no real details

This is just a recipe book that doesn't explain how anything actually works or what to do if you need to change anything. Everything significant is wrapped in extension methods without proper explanations of what they do. I feel like I didn't actually learn anything reading this.

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

ID: 21cb7455-3305-9316-5440-bdc22947a82d
Version Independent ID: 137d4b94-7b26-3911-b22d-42c754a95fc1
Content: Introduction to authentication for Single Page Apps on ASP.NET Core
Content Source: aspnetcore/security/authentication/identity-api-authorization.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @javiercn
Microsoft Alias: riande

Thanks for your comment. We'll take it under consideration when we have the resources to update this doc.


Document that setting node name will remove @s

Closes #33068

Thanks!

Cherry-picked for 3.2.4.


OpenAPI3 emitter is missing paths and components in sub namespace

Let's say we have a typespec like this
import "@typespec/rest";
import "@typespec/http";

using TypeSpec.Rest;
using TypeSpec.Http;

model A {
  propA1: string
}

namespace B {
  model A {
    propA2: string;
  }
  interface C {
    op op1(@body body: A): void;
  }
  op op1(@body body: A): void;
}

@route("d")
interface D {
  op op1(@body body: A): void;
}
op op1(@body body: A): void;

The generated openapi3 output is missing the path and the component A defined in namespace B. See playground link
Another point here, if I don't add the @route("d") to interface D, it will report lint errors, but doesn't report inside namespace B, which has similar structure.

This is by design you got to use a service if you want to pickup things in namespace. If you don't have a service it only pick up things right at the root of the global namespace. Otherwise it would mean it would pickup external libraries too.


Change to ordered collections  which uses sample names

For the logging system and some metrics files, the output is not ordered by sample names because the underlying map/set is not a ordered structure. This makes unpredictable results for logging/output, which should be solved for easier development.
Because it could break the compatibility from the user-side point of view (and test data), this is for beta.

Use interfaces in c1cc8fead23697feb172616792e1d189b6d3dd2d as the first step to this

This is trickier than expected, because the implementation relies on the order of several things. Probably a new implementation will be needed.

The best idea is to log and output in the metrics the same order as found in the barcode dictionary, to keep consistency.

Closing by 1d4818884640bfc5be5dcf2060e4e4d06924d0fb


CVAT new UI: fill in black the "occluded" and the "hide" icons

My actions before raising this issue

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

Expected Behaviour

To have an uniform style I'm offering to fill in black all the icons when they are activated
Current Behaviour

The "occluded property" and the "hide" have no black fill.

Occluded doesn't have dark theme in antd


Fix url in expose-external-ip-address.md

Fix url in expose-external-ip-address.md

Hard coding language into the URL is not a practice Kubernetes follows. Closing now. Feel free to open a PR with a solution that doesn't include hard coding the English URL. Thanks!
/close


Add fuzzy searching to the system privacy annotation config UI form

Is your feature request related to a specific problem?
The initial iteration required the user to know what the types were.
Describe the solution you'd like
To simplify the system setup process, especially for new users, we should enable fuzzy search for each of the privacy data types.
Describe alternatives you've considered, if any
A description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

closing, completed via another ticket/stale


Need easy turn off intelisense, checking functions, interaction in documents, eg. by simple icon on progress-bar.

Right now exist often situation where intelisense, checking functions, etc. and words/vars are underlines, exist many other alerts regarding code and doing interaction between code document and VSC. Sometimes are documents where is many errors, alerts and doing problems for VS or OS (both working hard), indicating some notifications to solve problems, etc.. In short.. to much interactions.
By this want suggest easy turn off interaction in documents, eg. by simple icon on progress-bar.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Adds wrapping to tables in docs for improved readability.

I've also added a dependency on sphinx_rtd_theme in the environment.yml file as
the docs wouldn't build without it.

[X] Closes #1728
[X] Tests added / passed
[X] Passes git diff upstream/master **/*py | flake8 --diff
[X] Fully documented, including whats-new.rst for all changes and api.rst for new API

(I've ticked the last checkbox but I'm not sure if I'm supposed to document anything additional here...)

This looks clean :)
In the smaller window width, the table wrapping seems disabled.
Maybe we can also improve this?
(For the easier review, here are the screenshots)

@fujiisoup I can lower the threshold at which it changes from reflow → horizontal or remove it entirely.

If I remove it, you will have a bad experience at < 500px width as the table clips on the right side and you can't even scroll to the right to see the whole thing. The picture below is at 440px width with reflow.

If I lower the threshold, horizontal scrolling will activate only below 500px width, but the cost is extra vertical space between 767px (the current threshold) and 500px because of excessive wrapping. The picture below is at 500px width with reflow.

Should I with option 2? (The 500px number is somewhat arbitrary, I just guessed it based on a few trials. Of course, any appropriate number between 440 and 500 will do.)

Thanks for putting this together!
I just gave looked at a preview in Chrome. Here's the progression I see, as I shrink the width of the browser window:

Nice wrapping on multiple lines:

Increasingly shrunk right column, but no scrolling yet:

At some critical width, table of contents disappears and table wrapping is again disabled:

My thought: we should definitely have a minimum width for the right column (maybe a few hundred px?). But it would be nice if we had consistent behavior for the API docs table, regardless of whether or not the table of contents fits.

@shoyer Currently that wrap → horizontal threshold is set to 767px in the CSS, which is why your third picture and fujisoup's second picture show the horizontal table.
@media screen and (min-width: 767px) {

Coincidentally (or maybe by design, idk), this 767px threshold is close to the TOC disappearing threshold. I propose lowering the threshold to 500px, or thereabouts. I haven't committed those changes yet; I'm waiting for the green light.
If we make that change, at extremely narrow widths (such as 440px - my first image) it will use a horizontal table (not shown, wrapping shown to illustrate that it looks bad). However, so long as it is readable with wrapping (which kinda' appears to be true at 500px width in the second image), it will wrap.
Sorry if I didn't make this clear; my second picture has no TOC but still has wrapping. It's not a relatively wide window with wrapping and the TOC cropped out.

@theindigamer OK, that makes more sense. I wonder, is there a way to set that cutoff threshold after subtracting the size of the TOC if shown? Or is that too complicated for CSS?

Do you mean something like
if toc.is_shown:
    threshold = foo - toc.width
else:
    threshold = bar
@media screen and (min-width: .threshold.width) {
/* same stuff as in commit */

or something else? I think arithmetic should be doable easily using the calc() function. For the conditional, I think we have to modify the HTML too (I don't know much about CSS so I'm simply following StackOverflow). However, we could just copy the theme's conditional statement as an approximate solution.
I've been looking at the CSS and the value of the sidebar's width is set using wy-nav-side.width which is 300px right now (line 4126 in sphinx_rtd_theme.css). If I understand correctly, the logic it is using the make the toc disappear is in lines 4251 - 4260.
4251 @media screen and (max-width: 768px) {
     [...]
      .wy-nav-side {
       left:-300px
4260  }
     [...]

I suppose this just moves the sidebar outside the display area (it has fixed width). My guess is that we can't refer to this 768px value directly because it isn't saved in any attribute, so we will need to copy the whole @media screen and (max-width: 768px) wherever we want to make that conditional check and hope it doesn't break in the future.

We switched to a different theme in #4835 so this PR might not be necessary anymore. Should we close it? cc @andersy005

Yes, I think this PR can be closed

let's close this, then. Thanks for the effort, @typesanitizer.


Do we need the get_data class?

The line:
d <- get_data(proj_metadata_key ==1)
Returns an object of class popler, get_data, and data.frame.
I do not think we need a class for get_data - but there is something I might be missing!

Having a get_data class helps a lot for the summary functions of dictionary and browse.
There is a function called rebrowse() that takes an object of popler class as an input and returns the equivalent browse() call with trim=FALSE, full_tbl=TRUE, but it needs to do different things depending on whether the popler object is a get_data object or a browse object.
By making separate classes, we can create a rebrowse function with different methods (i.e., rebrowse.get_data() and rebrowse.browse(). This makes it easier to change code (and debug) each method independently. It also shouldn't interfere with anything, since most functions will not have methods for browse or get_data classes.
We could get rid of the classes, but I'm not sure we gain much and we'd need to re-write the rebrowse function using something a bit hackier... I was originally looking for column names that were only returned by either browse() or get_data().

Sorry, I wrote that first sentence without thinking. rebrowse() is used in the summary function for browse (what used to be called "fancy browse") and the citation function.

I love it!
Issue closed!


Put error spans deep on nested object literals

And include an additional location pointing back at the deeply nested property in the target type it is being assigned to. Additionally, like in classes which are incompatible with a base type, object and array literals will now flag all locations which are incompatible with the target instead of just one.
Fixes #25030 partially (for all instances involving expressions where we're tracking which properties have issues).
This elaboration technique can cause the order we cache relationships in to change (since a no-errors check is done first to see if elaboration should be attempted, whose result may be used to help fastpath other relationship checks done later), so certain existing elaborations (those in this commit - things involving an initial pass Maybe result or excess prop, mostly) change.

As a follow-up to today's error sync, let's use checkExpressionForMutableLocation

@DanielRosenwasser Done. Also fixed the enum and elaboration changes like I said I aught to be able to. Wanna review?

@DanielRosenwasser I've synced with master, hopefully addressed your reviews, and added a few more tests for good measure. Have anything more you'd like to see?

Can you also add a test case similar to what you already had, but which uses a contextual indexed access type?

Ahhh.... to what are you referring?

Looks like you do have a case - however, as we discussed offline, the computed property message is a little confusing given that it's not clear it you're erroring on the name.

@DanielRosenwasser Added; what do you think about the new error?


[PSM Interop] Legacy tests: fix xDS test client build

https://github.com/grpc/grpc/pull/33699 incorrectly changed the legacy builds to not just use the test driver from the master, but also to build from it. This PR fixes the issue, and also updates the python job to work use the driver from master.

[ ] grpc/core/master/linux/grpc_xds_v3
[ ] grpc/core/master/linux/grpc_xds_k8s_lb_python


fix(live-preview-react): prevents duplicative subscriptions

Description
Prevents the subscribe from being fired more than once in @payloadcms/live-preview-react.

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] Existing test suite passes locally with my changes

This is not the proper fix. Another PR incoming.


Fixing the extra layout inflations

Layout should only be inflated if the passed in convertView is null. If the view is recycled, layout doesn't need to be inflated. Removing the extra inflation will improve performance.

Hi @ashishmody, thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla
Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!


Global Css and missing files when start dev enviroment

Describe the bug
It's seem like some module of  codemirror (tiddlywiki) have conflict with nextjs css. Beside that, next cannot found files for / in build manifest
To Reproduce
Steps to reproduce the behavior:

Clone project
Run yarn
See error

Expected behavior
When navigate to localhost:3000 you should see error Global CSS cannot be imported... then a black screen appeared.
Screenshots

Info (please complete the following information):

OS [e.g. macOS, Linux, Windows, iOS]:
Browser [e.g. Chrome, Safari, Firefox]:
Carbon URL [e.g. carbon.now.sh?bg=pink]:

Same here, if you visit using chrome you could see the error

./node_modules/codemirror/mode/tiddlywiki/tiddlywiki.css Global CSS cannot be imported from files other than your Custom <App>. Due to the Global nature of stylesheets, and to avoid conflicts, Please move all first-party global CSS imports to pages/_app.js. Or convert the import to Component-Level CSS (CSS Modules). Read more: https://nextjs.org/docs/messages/css-global


Comparison with similar tool "SymWin"

SymWin does the same thing as PowerAccent and originally made by @mjvh80 in 2018. Here's my fork with a couple of minor updates and binaries.
I'm not a professional C# developer, but there might be some ideas in the code for you. It currently supports every key on the keyboard, for which many accents and special symbols, approximately by usage for me (English/French/Spanish/Portuguese speaker) on an "average" of a UK (mine) and US (standard) keyboard. It uses CAPSLOCK as the activator key, using multiple presses of the character to type the accent.

Having tested your implementation, it's definitely more responsive (the menus on SymWin have poor performance, no idea why).
I'm still not sure whether I préfer CAPSLÔCK as the äçcent key or space.
SymWin has the same problem with guessing caret position in browsers, but I do prefer its slightly-higher-than-center default position.
SymWin uses less RAM initially (usage grows with time/accents loaded)

Did not know about this tool. Will see what it is able to do :)
I've also got some improvement on my side ongoing to get a lot more possibility with choosing countries.
CAPSLOCK is a bad idea in PowerAccent, because it use to Uppercase the typed letters.
@Darthagnon slightly-higher-than-center can be an improvement if you wish.
About RAM, I know PowerAccent take a bit, it is because of the technology I am using (WPF). But it is the best I can use for now to have something that works. And so it is not my priority for now.


cdc: fix memory quota overflow

What problem does this PR solve?
Problem Summary: Freeing sink quota more than it actual allocated causes overflow, and blocks all messages from TiKV to TiCDC.
Related changes

Need to cherry-pick to the release branch

Check List
Tests

Unit test

Release note
This is not released yet.
None

/run-all-tests

/merge

/merge


UI/X to assist crowdsourcing de-dupe/record matchin

Description
We are dealing with large amounts of data, and making associations between a variety of different records.
In some cases, we can make exact match association automatically, based on, for example, unique identifiers for trials.
In other cases, we can make decent guesses - strong enough to actually declare an association - but we may be wrong.
In yet other cases, we may have no way in code to associate different records, but a domain expert could see the same records and make an association.
We want to design a user interaction to facilitate the following:

Validate existing matches
Create new matches

Underneath, much of our record matching efforts in OpenTrials should be related to or merged into the generic codebase of http://github.com/pudo/nomenklatura - especially any user interface work we do to make this a friendly and intuitive process. So, while we are striving her to meet specific OpenTrials use cases (Example: it recently become apparent that there are different paths to guide a user along for record matching, dependant on the domain expertise, and possibly age, of the potential matcher), let's keep in mind the wider applicability of this work. ( cc @pudo )
Tasks

[ ] TBD

closed in favour of https://github.com/opentrials/opentrials/issues/51


CVE-2017-6006 Symphony 2.6.11 has XSS in publish/articles/new/ via the Body field.

Dear Developers,
Did you fix this vulnerability?
PoC :
https://www.yiwang6.cn/xss.docx

@Nassz Please check your emails.

Re: http://www.cve.mitre.org/cgi-bin/cvename.cgi?name=2017-6006
Re: https://nvd.nist.gov/vuln/detail/CVE-2017-6006

After proper review, the CVE has been rejected. False alarm.

Great to hear that. Thanks for caring!

@nitriques can you summarize why it was rejected? false positive, no valid exploit vector, etc?

It was a false positive; something on the lines of including a <script>
tag in a text-area field in the backend (logged-in author/user) allows XSS
on the front-end.
That's not a Symphony CMS XSS exploit; we have clients who need to include
third-party tools which require unfortunately XSS, if you have un-trusted
users and don't want to allow script tags you can easily eliminate them
with XSLT - so it's a non-issue.
@nitriques correct me if I'm wrong.
Jonathan Mifsud
Founder and Lead Developer
T +356 9983 0394 <+35699830394> E<EMAIL_ADDRESS>W maze.digital
On Sat, Apr 1, 2017 at 1:30 AM, Jericho<EMAIL_ADDRESS>wrote:

@nitriques https://github.com/nitriques can you summarize why it was
rejected? false positive, no valid exploit vector, etc?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/symphonycms/symphony-2/issues/2651#issuecomment-290863199,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0efw2J5ALOrR1U6lLWIqEoGmGlJamGks5rrYyWgaJpZM4Ms-Zh
.

Great, appreciate the details!

@jonmifsud is right ! And if you leave the XSRF protection active, XSS can't be injected from another site.


Pass optional fileId param to addFile on server

We have a use case where it would be helpful to set the _id prop when calling addFile from the server. Hopefully this does the job. Thanks for a great package.

+1

@dr-dimitru Thanks for your comments. Good suggestions. I've made the changes you've requested. Hopefully that does it. Thanks again for all your work on this.


TypeError when changing route and when the previous state childview element isn't on the DOM anymore

AngularJS   v1.2.26
ui-router      v0.2.11
See this jsfiddle http://jsfiddle.net/heiso/Lxx3vL42/ using ui-router v0.2.11(context and bug) and this same jsfiddle http://jsfiddle.net/heiso/Ldedhuw8/ using ui-router v0.2.10 (same context, no bug).
Context (4 states):

otherwise

onEnter -> $state.go('home');

auth

abstract
mainView template:
login page

auth.index

childView template:
login form

home

Steps :
Firsts, go to auth.index state then go to otherwise state, ui-router attempt to get the previous state element -> TypeError: Cannot read property 'mainView@' of null

https://gist.github.com/eddiemonge/f6a58169c2846731a1eb


Issue in identifying package

Description:
In the below sample code, the io package is not identified correctly.
function printStack() {
    foreach i in 0..sp {
        io:print(i + " ");
    }
}

Affected Versions:
0.973.0

This issue occurred due to outdated grammar. Grammar is updated in #8975.


What is the normal val_loss range?
And how to evaluation loss size?
here is setting:
Step 1 : fine-turn on last layer

epochs=50
batch_size = 32
optimizer = Adam, lr=1e-3

Step 2 : fine-turn on all layer

epochs=50
batch_size = 8
optimizer = Adam, lr=1e-4

and here my loss value traing on training on BDD dataset:
the last value is: 62.07, Although this value looks big, it still work!
and it take 6 day to training!!!

my training is not finished, but the last val_loss is about 30.0 after 5 epochs(i stop the training process because i got a new machine with higher performance GPU :) )
parameters : 50 epochs, 8 batch_size, optimizer and lr are default.
i will use my new machine to train my data set, update the comment soon.

@sherlockchou86
Look like your training val_loss can be very low!!!
I think the bdd dataset ground truth is too difficult to found, that cause my training loss too big.

I trained on VOC dataset and got around 16 on val_loss after 120 epoch.
The trained model can achieve 77% mAP if I lower confidence threshold to 0.005.

I trained a model with 1 class, got loss of 24. The performance is significantly lower than what I get using tensorflow object detection...
A same image using yolo/tensorflow object detection on the SAME training set (OID)
The detection threshold is 0.0 (showing everything it detected)
yolo:

tensorflow:

@kwea123 , i want to train on single class to detect person more accurately, for which i changed the class.txt to person and then am i supposed to change the .cfg before converting to get .h5
The change such as the filter and class number. ?

50左右 效果很差


wifi setup screen in card layout should contain next/skip buttons within the card itself

needs more changes (probably wider changes) to settings app in order to fix this UI issue.

can be closed with: https://github.com/GrapheneOS/os-issue-tracker/issues/3263

yes. closing now.


Example should not depend on other packages

Hi, I just looked into the example and it seems to utilize rearch, another one of your packages.
I am not really in the liberty to judge how your repository should work, but it might put off people to using this package, especially while using a undocumented one.

I originally started to write the example in vanilla Flutter but quickly realized how much of a pain it is. Quickly switched it to Riverpod + hooks for a bit, but those have their own problems.
Didn't actually expect anyone to take too much of a look at the example here yet (mimir is still relatively new), but you can find all of the mimir-specific code in the first ~50 lines of code in main.dart. Not much is going on there--essentially just opening an index and then triggering a searchStream on it:
final instance = await Mimir.defaultInstance;
final index = instance.getIndex('movies');

// Add all the documents asynchronously.
await rootBundle
    .loadString('assets/tmdb_movies.json')
    .then((l) => json.decode(l) as List)
    .then((l) => l.cast<Map<String, dynamic>>())
    .then(index.setDocuments);

// ...

index.searchStream(query: query.$1)

That's about it; the rest is just UI and glue, which is out of the scope of what mimir does.

especially while using a undocumented one.

Absolutely agree, but rearch is also currently pre-release (it's the topic of my master's thesis) and I frankly didn't expect anyone to look at it yet. Essentially the only reason it's published right now is to reserve the name and also so my thesis advisor can more easily play around with it. I did jump the gun and convert this package to use rearch before formal release, but that is mostly because:

I wanted the new gif that you can see in the README, and that required a rewrite (the old version with riverpod was not as clean)
The old version gave me issues in integration tests from time to time (that's not Riverpod's fault though, just a poorly-written integration test that I rewrote alongside the example app)

All that being said, I have full documentation coming out for it at some point within the next couple months when I release rearch, so this won't be an issue for too much longer.
Closing this as I don't plan on rewriting the example, but thank you for the feedback!

More power to you, as your the author of the package

Riverpod + hooks for a bit, but those have their own problems.

Can you elaborate on this? Just wanted to know what some of the pains were

Can you elaborate on this? Just wanted to know what some of the pains were

I just made the rearch Dart implementation public (it's still pre-release though); and there is a section in the README detailing some of the issues I've found with Riverpod and hooks.


any easy example to get latest price of exchange for example ?

Hi,
Is there any examples for this aweose lib please ?
Thank you !

reference: https://github.com/nntaoli-project/download_binance_kline_history/blob/master/main.go

https://github.com/nntaoli-project/goex/wiki

@nntaoli Thank you sir !


Update DataChain worker liveness probe

This updates liveness probe command for DVCx worker k8s deployment.
Should be merged after https://github.com/iterative/studio/pull/9174 will be merged and released.

Waiting this PR to be merged and deployed to dev and prod: https://github.com/iterative/studio/pull/9247

@dreadatour what is the status of this?

folks @dreadatour and @iterative/platform - what is the status here?

folks @dreadatour and @iterative/platform - what is the status here?

It looks like https://github.com/iterative/studio/pull/9174 mentioned on https://github.com/iterative/helm-charts/pull/325#issue-2136284740 was released 4+ months (!) ago; after https://github.com/iterative/helm-charts/pull/438 we no longer need this liveness probe.
Still, we should have some liveness probe, maybe making sure that the Celery workers are storing heartbeats into Redis or something that ensures us the pod is healthy and ready to ingest new tasks.

So there's nothing wrong with this pull request and we can merge it, but /app/bin/liveness-probe.sh will have to be redesigned first.

there was some code duplication around SA liveness probes and tests on the Studio side that we also should cleanup along the way

Maybe https://github.com/iterative/studio/pull/10307#discussion1683252817 can provide better readiness/liveness probes for DataChain that anything else, if it gets merged.

Stale; opened https://github.com/iterative/helm-charts/issues/504 to track this


[Bug]: Parser of Environment Variables

Description
When I set value 0 on a Environment Variables, is parsed to "".
Minimal Reproduction (if possible, example repository)
Set Environment Variables 0 a console on your code.
Exception or Error
No response
Version
v4.0.0-beta.239

This will be fixed in the upcoming version.


Fix Netlify Redirect

Netlify is redirecting all .com to .app, this is affecting the demo by making every link redirect, you can test by navigating pages on the current demo.
Also, feel free to add me as a maintainer to this theme if you need some help with approving good pull requests, etc.

This issue also affects the mozilla observatory rating which is now only 35


Editing projects sometimes auto approves

I was able to create a project, not answer the questions, got to dash, edit, resubmit and then it was approved

Cannot reproduce anymore closing


As non-DataCurator user, method changes not noticed

I created a new user with full permissions, and loaded RowanSample4.
Changes to existing methods, adding new methods do not show up in changes and do not get written to disk.  An added class does show up in changes and gets a new file created on write (but no methods are there).
Reported in HR 9824

The method is being changed in the class, but the loaded method is not updated...

Interestingly enough the following executed in a workspace, does work:
RowanSample1Test
	rwCompileMethod:
'testFailure

	self assert: RowanSample1 new foo = ''car''
'
	category: 'tests'

... and this is pretty much the code that should be execute by Jadeite???

... okay this is a known bug: issue #177, #178, and #205. The following expression needs to be evaluated in the context of a user using Jadeite:
GsPackagePolicy current enable.
UserGlobals at: #rowanCompile put: true.
System commit

We automatically do this for DataCurator and SystemUser when installing Rowan ... I suppose we could do this for AllUsers when we install --- what do you think @LisaAlmarode ...

Has to be done for all users... otherwise other users can't do stuff to methods, which appears to be a requirement.

Yeah, but it does not fix the problem if users are added after Rowan has been installed ... and the release  version of Jadeite (pyrite) will not have this requirement ...

I'm inclined to have the user chime in on this one --- they are the only ones who will see it ...

Maybe a login hook or something.  This is something that no one is going to remember since users are not often created; and it causes a hard-to-see problem... one does not want to take chances with code getting lost.

Yes but we have a captive audience ... this is a short term problem that might be solved by a one-time doit on the customer end ... which is why I want their opinion ... this in no way is a long term workaround (unless you call 4 weeks long term:)

Why didn't you think you should do it for all users on rowan load, then?
Anyway... you asked and you have my opinion...  I like safe code.

A login hook would require a new release of Jadite and that is not something that will happen in the next week ... Eric should be spending his time working on the release version ... I think it is likely to be a onetime patch for the customer --- if we ask/remind them ... they can add the three lines to their installation script and make sure that the development suers are covered ...

give me a call and discuss this ... we can dispose of this in two minutes ...

Here's what I think: let's not get too cute about making sure all users get this code change.
The real goal is to make sure that if you do something like compile a method and write the project, the change definitely positively is in fact written to disk and won't get mysteriously lost if you reload.
So... make it an error to compile, if say, rowanCompile isn't true, or whatever the correct tests is.  That way you'd know something was wrong and have a chance to find the magic incantation to satisfy Rowan.

... nice try, but the reason that the global is to explicitly allow someone to compile a non-Rowan method ... if it was simple, it would be done by now ... let's talk it over on phone or when we're both in office

With the completion of #364, we'll be ready to remove #rowanCompile from Jadeite, the Rowan install scripts and docs


dartanalyzer fails in batch mode in checkouts with a space in the path

On Windows and linux, if there is a space in the path of a dart checkout, running tools/test.py -cdartanalyzer -rnone  will show most tests failing, with the error
File not found: [absolute path of checkout, truncated at first space]
Other failures found (from the test infrastructure) when running in --nobatch mode are being fixed, but this issue is due to the batch mode of dartanalyzer (--client flag) reading the input file name from stdin, and parsing it by splitting it on spaces.  Even enclosing the file name in quotes, when the test runner puts it on stdin of the dartanalyzer (java) running in batch mode, does not succeed - it is still truncated at the space.  The analyzer needs to do a cleverer job of parsing the batch input, and then the test runner needs to be modified too.
This probably needs to be fixed in the Dart version of the analyzer too.  I believe floitch is working on a similar issue for dart2js.

I believe this is no longer an issue?


Revert "Explicitly set the metadata generator's OS X deployment target"

Merge with: https://github.com/NativeScript/ios-metadata-generator/pull/28

run ci