Failed to parse SIP Contact header when Topology Hiding is used on server

If SIP proxy uses Topology Hiding, it encodes topology information in Contact field. And SIP.js reports error while parsing such header.
| sip.parser | error parsing header "Contact"
For example,
SIP/2.0 200 OK
Via: SIP/2.0/WSS k4cqi8bdsk05.invalid;received=0:0:0:0:0:0:0:1;rport=34314;branch=z9hG4bK636618
From: "WSS 205" sip:205@siptest.local;tag=u543f8kcr4
To: sip:*7@siptest.local;tag=as6e067557
Call-ID: sol19a5ctkr7uukq9l6d
CSeq: 9042 INVITE
Server: asterisk
Allow: INVITE, ACK, CANCEL, OPTIONS, BYE, REFER, SUBSCRIBE, NOTIFY, INFO, PUBLISH, MESSAGE
Supported: replaces, timer
Contact: sip:[0:0:0:0:0:0:0:1]:8061;transport=wss;thinfo="U2h5YhUORg5uUBNZW1dIUAQadlZjRlFWUV0DBHJXRGgWBhZdBwxxSWRaTVBWXxgGcF9pXVNUVg=="
Content-Type: application/sdp
Content-Length: 582
v=0
o=root<PHONE_NUMBER><PHONE_NUMBER> IN IP4 <IP_ADDRESS>
s=ipnordic
c=IN IP4 <IP_ADDRESS>
t=0 0
m=audio 10958 UDP/TLS/RTP/SAVPF 8 126
a=rtpmap:8 PCMA/8000
a=rtpmap:126 telephone-event/8000
a=fmtp:126 0-16
a=sendrecv
a=rtcp:10958
a=rtcp-mux
a=setup:active
a=fingerprint:sha-1 88:78:35:6D:55:77:4D:32:35:58:CB:D0:71:CA:76:16:B3:19:26:A7
a=ptime:20
a=ice-ufrag:D4HHbBzi
a=ice-pwd:q9CJDdddddUZbAwRIFdS
a=candidate:eSePKrNDiCsyf08i 1 UDP<PHONE_NUMBER> <IP_ADDRESS> 10958 typ host
a=candidate:h3V6UlbW1JuyzKHE 1 UDP<PHONE_NUMBER> 2a03:3250:8000:1::246 11036 typ host
Or this:
OPTIONS sip:0rjpr7gh@k4cqi8bdsk05.invalid;transport=ws SIP/2.0
Via: SIP/2.0/WSS [0:0:0:0:0:0:0:1]:8061;branch=z9hG4bKe61a.ff611026.0
Max-Forwards: 69
From: "asterisk" sip:asterisk@siptest.local;tag=as63bc007e
To<EMAIL_ADDRESS>Contact: sip:[0:0:0:0:0:0:0:1]:8061;transport=wss;thinfo="U2hDYhUORg4lFCcNEQsVDHYFfFJ9X1FMVFYGGnZTZFJWUlBXITQxAyNSUlpTSQEGalVjUE1QUl8MAXRRYw=="
Call-ID: 0c11b264352643c80a2febf24f4a59eb@<IP_ADDRESS>:5060
CSeq: 102 OPTIONS
User-Agent: ipnordic
Date: Mon, 20 Aug 2018 08:47:49 GMT
Allow: INVITE, ACK, CANCEL, OPTIONS, BYE, REFER, SUBSCRIBE, NOTIFY, INFO, PUBLISH, MESSAGE
Supported: replaces, timer
Content-Length: 0
To Reproduce (if possible)
Try to parse header
Contact: sip:[0:0:0:0:0:0:0:1]:8061;transport=wss;thinfo="U2h5YhUORg5uUBNZW1dIUAQadlZjRlFWUV0DBHJXRGgWBhZdBwxxSWRaTVBWXxgGcF9pXVNUVg=="
Expected behavior
No parsing error.
Environment Information
SIP.js v 0.11.3
Chrome Version 68.0.3440.106
Additional context
About topology hiding:
https://www.opensips.org/Documentation/Tutorials-Topology-Hiding

BTW, I checked RFC 3261,
https://tools.ietf.org/html/rfc3261#section-25.1
Contact        =  ("Contact" / "m" ) HCOLON
( STAR / (contact-param *(COMMA contact-param)))
contact-param  =  (name-addr / addr-spec) *(SEMI contact-params)
contact-params     =  c-p-q / c-p-expires
/ contact-extension
contact-extension  =  generic-param
generic-param  =  token [ EQUAL gen-value ]
gen-value      =  token / host / quoted-string
quoted-string  =  SWS DQUOTE *(qdtext / quoted-pair ) DQUOTE
qdtext         =  LWS / %x21 / %x23-5B / %x5D-7E
/ UTF8-NONASCII
So we get contact-params = contact-extension = generic-param
in generic param 'token' is 'thinfo' and gen-value is quoted-string,
which can be for example "U2h5YhUORg5uUBNZW1dIUAQadlZjRlFWUV0DBHJXRGgWBhZdBwxxSWRaTVBWXxgGcF9pXVNUVg=="
as symbol "=" is %x3D and qdtext can't contain only double quotes (%x22) and backslash (%x5C)

Can you re-insert the message that is causing the problem but make sure to click the "Insert Code" editor button first? I ask because your From header is invalid as it appears in your first message so I assume the GitHub editor is doing something funky.

Yes, sorry, I've not checked
There are examples:
OPTIONS sip:u3sbtgnh@n9g6ke4672g9.invalid;transport=ws SIP/2.0
Via: SIP/2.0/WSS [0:0:0:0:0:0:0:1]:8061;branch=z9hG4bK3737.9716d362.0
Max-Forwards: 69
From: "asterisk" <sip:asterisk@siptest.local>;tag=as07da8976
To<EMAIL_ADDRESS>Contact: <sip:[0:0:0:0:0:0:0:1]:8061;transport=wss;thinfo="U2hDYhUORg4lFCcNEQsVDHYFfFJ9X1FMVFYGGnZTZFJWUlBXITQxAyNSUlpTSQEGalVjUE1QUl8MAXRRYw==">
Call-ID: 1bf9d65b7b40bbb96fad31202e3ef626@<IP_ADDRESS>:5060
CSeq: 102 OPTIONS
User-Agent: asterisk
Date: Mon, 20 Aug 2018 13:44:42 GMT
Allow: INVITE, ACK, CANCEL, OPTIONS, BYE, REFER, SUBSCRIBE, NOTIFY, INFO, PUBLISH, MESSAGE
Supported: replaces, timer
Content-Length: 0

SIP/2.0 200 OK
Via: SIP/2.0/WSS n9g6ke4672g9.invalid;received=0:0:0:0:0:0:0:1;rport=35504;branch=z9hG4bK4611047
From: "WSS 205" <sip:205@siptest.local>;tag=1v0bpcha30
To: <sip:*7@siptest.local>;tag=as48634327
Call-ID: k07f5snb8g6vkfvibu54
CSeq: 2182 INVITE
Server: asterisk
Allow: INVITE, ACK, CANCEL, OPTIONS, BYE, REFER, SUBSCRIBE, NOTIFY, INFO, PUBLISH, MESSAGE
Supported: replaces, timer
Contact: <sip:[0:0:0:0:0:0:0:1]:8061;transport=wss;thinfo="U2h5YhUORg5uUBNZW1dIUAQadlZjRlFWUV0DBHJXRGgWBhZdBwxxSWRaTVBWXxgGcF9pXVNUVg==">
Content-Type: application/sdp
Content-Length: 580

SDP...

Thanks.

You quote the ABNF section of the RFC that deals with Contact parameters but thinfo is actually an attempt at a SIP URI parameter which cannot be a quoted-string. You will need to move the thinfo parameter after the SIP URI, so instead of this:
Contact: <sip:[0:0:0:0:0:0:0:1]:8061;transport=wss;thinfo="U2h5YhUORg5uUBNZW1dIUAQadlZjRlFWUV0DBHJXRGgWBhZdBwxxSWRaTVBWXxgGcF9pXVNUVg==">

It should be this:
Contact: <sip:[0:0:0:0:0:0:0:1]:8061;transport=wss>;thinfo="U2h5YhUORg5uUBNZW1dIUAQadlZjRlFWUV0DBHJXRGgWBhZdBwxxSWRaTVBWXxgGcF9pXVNUVg=="
                                                  ^ Move the > to here

Sean, thank you very much for your explanation.
Yes, this is my mistake in reading spec. Of course, everything inside angle quotes (LAQUOT and RAQUOT) must fit addr-spec. And it can't contain double quotes and equal sign.
I asked OpenSIPS guys if topology hiding behavior matches RFC. When I'll find solution, I can write it here.

@egreenmachine this is not a bug in SIP.js - feel free to close.

Yes, this is bug in OpenSIPS. http://opensips-open-sip-server.1449251.n2.nabble.com/Does-topology-hiding-match-RFC3261-td7611813.html I'll open issue there.

Fixed in OpenSIPS https://github.com/OpenSIPS/opensips/pull/1448


Weird behavior with PMMO

While this fork and Project MMO are installed, leveling up a skill in Project MMO will also level up a skill in Reskillable so long as they both have the same name. At first I thought this was a feature, but I don't see any mention of PMMO on the page, or any options to disable it in the config.
I'm starting to think this might actually be a bug.

That is a feature from PMMO. If it is one of the skills that Reskillable has it adds a level, else it tries to add a level to a CompatSkills skill with the same name. If there is no skill with the same name it throws an error because it does not check if the skill exists. There is also no check for a config, so you can't turn it off.
There is the possibility of creating custom skills with the correct name but it is a bit more complicated. https://docs.blamejared.com/1.12/en/Mods/CompatSkills/Supports/Reskillable/CustomSkills/Custom_Skills
I can't do anything on my side, you have to ask the devs from PMMO.
https://github.com/Caltinor/Project-MMO-2.0/blob/9d03ff5782d1205564d8cad7214965b93a79a3ce/src/main/java/harmonised/pmmo/util/XP.java#L1217-L1226

if( Loader.isModLoaded( "compatskills" ) )
{
	if( !player.world.isRemote )
	{
		if( skill.equals( Skill.MINING.toString() ) || skill.equals( Skill.BUILDING.toString() ) || skill.equals( Skill.FARMING.toString() ) || skill.equals( Skill.AGILITY.toString() ) )
			player.getServer().commandManager.executeCommand( player, "/reskillable incrementskill " + playerName + " reskillable." + skill.toString() + " 1" );
		else
			player.getServer().commandManager.executeCommand( player, "/reskillable incrementskill " + playerName + " compatskills." + skill.toString() + " 1" );
	}
}


defaultValue given on top of user-given arguments for list-type

I have an @Option argument that is of type List<MyType> and I have defined a defaultValue that I thought would be used unless the user given an explicit argument to this option. But instead it seems that the option will have both whatever the user give as well as the default value. To me this is not the intuitive behavior, or do you have any reason behind this? Running the latest 4.7.4 version of picocli.

Thank you for raising this!
Can you provide a small program that demonstrates the issue?

Sorry for the late reply, this email got lost somehow in my inbox.
When writing a simplified test-case I found out that it works as expected when using a ITypeConverter but fails when using a IParameterConsumer (i.e. then the value of defaultValue is passed as an additional argument in the stack). See the following test-class:
import java.util.ArrayList;
import java.util.List;
import java.util.Locale;
import java.util.Stack;
import java.util.concurrent.Callable;

import org.junit.Assert;
import org.junit.Test;

import picocli.CommandLine;
import picocli.CommandLine.IParameterConsumer;
import picocli.CommandLine.ITypeConverter;
import picocli.CommandLine.Model.ArgSpec;
import picocli.CommandLine.Model.CommandSpec;
import picocli.CommandLine.Option;

public class PicocliDefaultValueBug {

    // Toy custom class
    public static class MyCustomCls {
        private final String lowerCase;

        public MyCustomCls(String value){
            lowerCase = value.toLowerCase(Locale.ENGLISH);
        }

        // Check equals in the test-case
        public String toString(){
            return lowerCase;
        }
    }

    // Custom converter version
    public static class MyClsConverter implements ITypeConverter<MyCustomCls> {
        @Override
        public MyCustomCls convert(String value) throws Exception {
            return new MyCustomCls(value);
        }
    }

    public static class TestCmdConverter implements Callable<Integer>{

        @Option(names =  {"-f"},
            defaultValue = "defaultArg",
            converter = MyClsConverter.class,
            split = "\\s")
        public List<MyCustomCls> arguments = new ArrayList<>();
        @Override
        public Integer call() throws Exception {
            throw new UnsupportedOperationException("Unimplemented method 'call'");
        }

    }

    // Consumer version
    public static class MyClsConsumer implements IParameterConsumer {

        @Override
        public void consumeParameters(Stack<String> args, ArgSpec argSpec, CommandSpec commandSpec) {
            List<MyCustomCls> list = argSpec.getValue();

            // Simple example, all remaining input are of this type
            while (!args.empty()){
                list.add(new MyCustomCls(args.pop()));
            }
        }

    }

    public static class TestCmdConsumer implements Callable<Integer>{

        @Option(names =  {"-f"},
            defaultValue = "defaultArg",
            parameterConsumer = MyClsConsumer.class,
            split = "\\s")
        public List<MyCustomCls> arguments = new ArrayList<>();
        @Override
        public Integer call() throws Exception {
            throw new UnsupportedOperationException("Unimplemented method 'call'");
        }

    }

    @Test
    public void testShowDefaultAlwaysGiven() throws Exception {

        // Converter implementation - no argument given
        TestCmdConverter program = new TestCmdConverter();
        new CommandLine(program).parseArgs();
        Assert.assertEquals(1, program.arguments.size());
        Assert.assertEquals("defaultarg",program.arguments.get(0).toString());

        // Converter implementation - when giving an expecit argument (works as well)
        program = new TestCmdConverter();
        new CommandLine(program).parseArgs("-f", "explicitArg");
        Assert.assertEquals(1, program.arguments.size());
        Assert.assertEquals("explicitarg",program.arguments.get(0).toString());

        // Test consumer - no argument
        TestCmdConsumer cmd = new TestCmdConsumer();
        new CommandLine(cmd).parseArgs();
        Assert.assertEquals(1, cmd.arguments.size());
        Assert.assertEquals("defaultarg",cmd.arguments.get(0).toString());

        // Consumer with explicit arg
        cmd = new TestCmdConsumer();
        new CommandLine(cmd).parseArgs("-f", "explicitArgument");
        Assert.assertEquals(1, cmd.arguments.size()); // fails, cmd.arguments.size == 2 (explicit argument and "defaultarg")
        Assert.assertEquals("explicitargument", cmd.arguments.get(0).toString());
    }

}

Thank you for the test case!
I'll be traveling for the next 2 weeks and I'll look at this when I get back.
Thank you for raising this!

@StaffanArvidsson  Apologies! I had not marked this ticket as part of the 4.7.5 milestone, and I overlooked it when doing a release earlier today! Darn! 😓
Thank you for raising this!
This was a bug: there is some bookkeeping that was not done for options with a custom IParameterConsumer, and this lack of bookkeeping resulted in the default value being applied even when a user-specified value existed.
Unfortunately I cannot think of a workaround other than not using custom IParameterConsumers for now...

No worries, glad that it might have helped others apart from myself 👍  thanks for the work!


Improve config management

The objective of this PR is to improve how config is handled and reduce clutter in the check.go file.
Signed-off-by: wwwil<EMAIL_ADDRESS>

This is now ready for review and I've made all the changes I had planned. However it's now crashing when I run it with the stubbed data so I need to fix that.

However it's now crashing

Fixed now!

As per discussion on slack, I'm trying to work out how best to get this in. There are a few things in play here and I'd like some input on what we want to do here.
There was a discussion on our call about datagathers being an array so we can have many of the same type (gke, k8s/pods) with different config (clustera, clusterb). This is a potential requirement of the cert-manager package: (internal: https://github.com/jetstack/preflight-private/issues/138#issuecomment-582602850)
While I'd hope to be able to demo the cert-manager with the saas, this might not be an option if we aren't fast enough. This leaves the question, is this PR a requirement of the cert-manager package?
How I understand it, the following datagatherers struct implemented here doesn't support this requirement as it only allows a single config for each type of data gatherer. For reference
type DataGatherersConfig struct {
	GKE   *gke.GKEDataGathererConfig     `mapstructure:"gke,omitempty"`
	AKS   *aks.AKSDataGathererConfig     `mapstructure:"aks,omitempty"`
	EKS   *eks.EKSDataGathererConfig     `mapstructure:"eks,omitempty"`
	Pods  *k8s.PodsDataGathererConfig    `mapstructure:"k8s/pods,omitempty"`
	Local *local.LocalDataGathererConfig `mapstructure:"local,omitempty"`
}

In my agent PR, I implemented the start of a new config format for the agent file: (part of https://github.com/jetstack/preflight/pull/87). See here
The validations for the datagatherer side was implemented in the follow up PR here: https://github.com/jetstack/preflight/pull/95 which is marked as on hold because of this PR.
(We also seem to have a legacy parsing option here. But from what I can tell there are no changes to the config file - only in how it's parsed? not sure if relevant but I was confused to see backwards compatibility without change in the config so I'm checking)
I find myself unsure how best to proceed for the following reasons:

I'm not sure how to use the data gatherer part of this without the rest
It seems this might soon need to change to support the multiple data gatherer requirement.

Sorry for the flapping but thought best to agree before continuing.

As per discussion on slack, I'm trying to work out how best to get this in. There are a few things in play here and I'd like some input on what we want to do here.
There was a discussion on our call about datagathers being an array so we can have many of the same type (gke, k8s/pods) with different config (clustera, clusterb). This is a potential requirement of the cert-manager package: (internal: jetstack/preflight-private#138 (comment))
While I'd hope to be able to demo the cert-manager with the saas, this might not be an option if we aren't fast enough. This leaves the question, is this PR a requirement of the cert-manager package?
How I understand it, the following datagatherers struct implemented here doesn't support this requirement as it only allows a single config for each type of data gatherer. For reference
type DataGatherersConfig struct {
	GKE   *gke.GKEDataGathererConfig     `mapstructure:"gke,omitempty"`
	AKS   *aks.AKSDataGathererConfig     `mapstructure:"aks,omitempty"`
	EKS   *eks.EKSDataGathererConfig     `mapstructure:"eks,omitempty"`
	Pods  *k8s.PodsDataGathererConfig    `mapstructure:"k8s/pods,omitempty"`
	Local *local.LocalDataGathererConfig `mapstructure:"local,omitempty"`
}

In my agent PR, I implemented the start of a new config format for the agent file: (part of #87). See here
The validations for the datagatherer side was implemented in the follow up PR here: #95 which is marked as on hold because of this PR.
(We also seem to have a legacy parsing option here. But from what I can tell there are no changes to the config file - only in how it's parsed? not sure if relevant but I was confused to see backwards compatibility without change in the config so I'm checking)
I find myself unsure how best to proceed for the following reasons:

I'm not sure how to use the data gatherer part of this without the rest
It seems this might soon need to change to support the multiple data gatherer requirement.

Sorry for the flapping but thought best to agree before continuing.

When this PR was initiated, it was just a refactoring. So it wasn't intended to add support for multiple instances of the same data-gatherer.
I'd create a PR that refactors the configuration of the data-gatherers themselves, adding for instance the aks.Config struct, for instance. but keeping the config file as it is. This will unblock #95.
Then as part of #95, I'd create a config that supports multiple instances of data-gatherers.

So it wasn't intended to add support for multiple instances of the same data-gatherer.

Sure, I just wanted to check that it still made sense to use this with that upcoming requirement in mind.
I think I can have a go at this though - thanks.

We are splitting out this PR into chunks.
So far we have merged #100, which creates uniform configuration and constructors for the data-gatherers.
@charlieegan3 and I talked this morning about if it is worth optimising those config structs on the data-gatherers packages for a direct unmarshal of the cmd/check config file. We think it is better to get #95 done and get the agent loading config in a cleaner and direct way and then adapt the config of cmd/agent if necessary.

We have agreed to implement this PR as multiple smaller PRs to make the work more manageable so I am going to close this.


Automagic for Python's script runner?

My setup is as follows:
- /
  |-- src/
  |   |-- __init__.py
  |   |-- script.py
  |   |-- lib.py
  |
  |-- test/
      |-- test_script.py

And I run my script (using PyCharm, which does not really like -m it seems) the script-y way as
python src/script.py

Additionally, my sys.path contains the full path to / and /src so I can go wild with imports. Whether this is kosher or not is beside the point. The thing is that it works. I can import lib into script, and I can import both script and lib into test_script without breaking any qualified names anywhere.
I tried to add the beartype magical hooks everywhere to see if it would make an aging codebase compatible out of the box. In the init. In the file. I also tried with both src.script, src. Yet none of it works.
...Except if I run the script like python -m script from a separate terminal. This is certainly doable but it's enough friction that a bunch of developers with already-configured hotkeys in their editor would just... not use beartype. We're talking decades of accumulated muscle memory here.
So what can I do? Is this a feature request that hasn't been raised before or am I trying to do the impossible?

...heh. Painting outside the lines, huh? At this point, most projects would probably start screeching that you're "doing it wrong," morally bankrupt, and should feel bad.
Thankfully, this is @beartype – the Fightin' Man's Type-checker. We strongly support your rebelliousness here. I myself live in a claptrap shed in the Canadian wilderness. I voluntarily run Gentoo Linux. It don't get more fightin' man. Of course, it's not simply fightin' man ethics; you're also grappling with crippling technical debt and exploding maintenance burden.
Thankfully, I've also tested @beartype against your real-world use case. Congratulations to us! @beartype behaves as expected and fully supports what you want to do.  I tried literally everything. I threw the leaky kitchen sink at friggin' @beartype.  It's broken.

Let @beartype Massage Those Worries Away
So, I replicated your ad-hoc-yet-awesome project structure with random gibberish of my own devising, like so:
# In "src/script.py":
def _hack_sys_path() -> None:
    '''
    Explicitly register all files and subdirectories of the root directory
    containing this top-level ``setup.py`` script to be importable modules and
    packages (respectively) for the remainder of this Python process if this
    directory has yet to be registered.
    '''

    # Avert thy eyes, purist Pythonistas!
    import os, sys

    # Absolute dirname of this directory inspired by this StackOverflow answer:
    #     https://stackoverflow.com/a/8663557/2809027
    setup_dirname = os.path.join(
        os.path.dirname(os.path.realpath(__file__)), '..')

    # If the current PYTHONPATH does *NOT* already contain this directory...
    if setup_dirname not in sys.path:
        # Print this registration.
        print(f'Registering project "{setup_dirname}" for importation...')

        # Append this directory to the current PYTHONPATH.
        sys.path.append(setup_dirname)

# Kludge us up the bomb.
_hack_sys_path()

# Subject this package to beartype-based runtime-static type-checking.
from beartype.claw import beartype_package
beartype_package('src')

# Import type-checked nonsense. Then call type-checked nonsense.
from src.lib import broke_af
print(broke_af())

And then:
# In "src/lib.py":
def broke_af() -> int:
    return "Broke af func ain't so dope af."

That's it. Our src/script.py (A) adds the src package to sys.path, (B) type-checks the src package with @beartype, and (C) imports and calls a type-checked function defined in your src.lib submodule. The type-checking violation from @beartype speaks for itself:
$ python src/script.py  # <-- do it, @beartype. do it for johnny.
Registering project "/home/leycec/tmp/pogo/src/.." for importation...
Traceback (most recent call last):
  File "/home/leycec/tmp/pogo/src/script.py", line 45, in <module>
    print(broke_af())
          ^^^^^^^^^^
  File "<@beartype(src.lib.broke_af) at 0x7fd27c6589a0>", line 24, in broke_af
beartype.roar.BeartypeCallHintReturnViolation: Function src.lib.broke_af() return
"Broke af func ain't so dope af." violates type hint <class 'int'>, as str
"Broke af func ain't so dope af." not instance of int.

We see that our sys.path hack succeeded and that @beartype successfully type-checked your src.lib.broke_af() function by raising a violation when that function returns an unexpected string rather than an expected integer.
No One Asked for This Monologue
The core takeaways here are:

Deferred type-checking. Python import hooks (like beartype.claw.beartype_package()) only apply to subsequently imported modules (like src.lib). They do not apply to the current module (like src/script.py). That's not @beartype's fault; that's just how Python import hooks work, I'm afraid. This means that you'll probably want to refactor most of the contents of your src/script.py script out into a new private submodule (like src.lib or perhaps even a new src.__main__ submodule or whatevah) that you then subsequently import from src/script.py. Your src/script.py script is really masquerading as a src.__init__ submodule. Which brings us to...
Your src.__init__ submodule is mostly useless. This is sad. The whole raison d'être for your src.__init__ submodule is that Python implicitly imports src.__init__ on the first importation of anything from your src package. But you've chosen (wisely, of course) to circumvent Python package standards. So, Python isn't doing that for you. Instead, you have to do all of the stuff you'd normally do in src.__init__ in your src/script.py script. This includes installing @beartype import hooks. Since src.__init__ is only generating confusion here, a reasonable decision would just be to move everything from src.__init__ into src/script.py; lastly, add a banner comment to src.__init__ that that submodule is intentionally empty and everybody should see src/script.py instead. Sadness continues.
beartype_this_package() is also useless. Because you call src/script.py as a script rather than as a module residing in a package, there is no "this package." Ergo, beartype_this_package() fails with an unreadable exception beartype.roar._BeartypeUtilCallFrameException: Callable "__main__.<module>()" package not found (i.e., top-level module or script "__main__" resides outside any parent package). Instead, you have to call beartype_package() with the explicit package name 'src'.

Painting outside the lines is, indeed, fraught with pain. Personally, I'd:

Shift the guts of src/script.py into a new src.__main__ submodule.
Gradually encourage everyone to transition to running your script as python -m src.

That's probably the optimal long-term solution, honestly. Why? It preserves backward compatibility in the short term (good) while also preserving standard Python package semantics in the long term (gooder) and avoiding bothersome issues like this for your team forevermore (goodest).
Thanks for All the Fishy Code
This isn't so much a feature request or outstanding issue as it is an AMA on the intersection of Python packaging, scripting, and import hooks. That's not necessarily a bad thing, of course. This is a fascinating conundrum. But it does kinda lie outside the purview of community standards and norms. Most Pythonistas would regard this sort of thing as a code smell, code debt, and devtool antipattern.
Thanks for the intriguing puzzle, @komodovaran. Let me know if you've got any other questions, concerns, or compulsions to break @beartype over your knee. Until then, let's quietly close this to prevent anyone else from getting any similarly bright ideas. :wink:

You were totally right, @komodovaran! @beartype was communicating poorly with everybody about this. I hang my head and sigh publicly. :face_exhaling:
More importantly, I did something about this. Exception messages raised by the beartype.claw.beartype_this_package() import hook now acknowledge your super-valid use case. They even offer helpful advice on how to rectify this. Behold: an intimidating mountain of words.
$ python3.12 src/script.py
beartype.roar.BeartypeClawHookUnpackagedException: Top-level script
"/home/leycec/tmp/pogo/src/script.py" resides outside package structure.
Consider calling another "beartype.claw" import hook. However, note that only
other modules will be type-checked. "/home/leycec/tmp/pogo/src/script.py" itself
will remain unchecked. All business logic should reside in submodules
subsequently imported by "/home/leycec/tmp/pogo/src/script.py": e.g.,
    # Instead of this at the top of "/home/leycec/tmp/pogo/src/script.py"...
    from beartype.claw import beartype_this_package  # <-- you are here
    beartype_this_package()                          # <-- feels bad

    # ...pass the basename of the "src/" subdirectory explicitly.
    from beartype.claw import beartype_package  # <-- you want to be here
    beartype_package("src")  # <-- feels good

    from src.main_submodule import main_func  # <-- still feels good
    main_func()                   # <-- *GOOD*! "beartype.claw" type-checks this
    some_global: str = 0xFEEDFACE  # <-- *BAD*! "beartype.claw" ignores this
This has been a message from your friendly neighbourhood bear.

You've probably already resolved this on your end – or just abandoned @beartype altogether. Still, @beartype now tries a bit harder to help. Thanks so much for getting us started on this. :sweat_smile:

WOAH. You have just entered the event horizon of typing, @JWCS. Beyond this point of no return, the new guy at work has inserted his white-knuckled fist into his mouth and is now contemplating living under a bridge. Congrats.
Seriously, though. How did you even think of that "special" darkness? If self-importing the script you're currently running isn't the Python Highway to Hell™, I don't know what is. And I know what is. This is like the @beartype codebase all over again.

Because I saw the above solution... and nah fam

:laughing: :smile: :smiling_face_with_tear: :cry: :sob:
I'm tempted to officially document this somewhere. Then I remember that I would be personally responsible for destroying all Python scripting. I slowly back away from the edge. Gibbering whispers can be heard from the yawning abyss below. ...hurk gurgle...

It was pretty easy:

Importing works great
I could import into another main file... but thats an extra file
You can check if you're imported or running by name==main
We're already doing that
Drop a solution, run for the lakes, hope the coworkers never catch on, get paid to do so

For my projects, the ratio of scripts to helper files is 50/50, so I'll probably be rolling this out everywhere 😬
Thankfully the boon of actually having beartype check half of what I'm doing outweighs the small dissonance of style.
But, you speak of event horizons? Truly eldritch implementations? DID HE ASK FOR MORE
There's the potential feasibility that this functionality could be implemented in a
from beartype.claw import beartype_this_executable
beartype_this_executable(__file__, __name__)
With a rough workings like: (stolen from importlib example)
# But, we could further do away with the args by invoking `inspect`
def beartype_this_executable(file_path, module_name):
    spec = importlib.util.spec_from_file_location(module_name, file_path)
    module = importlib.util.module_from_spec(spec)
    module.__name__ = '__main__'
    spec.loader.exec_module(module)
    sys.exit(0)
``

It was pretty easy:

proceeds to list five hard bullet points

Drop a solution, run for the lakes, hope the coworkers never catch on, get paid to do so

"Drop a solution, run for the lakes" <-- quoted for truth
The next time my wife begs me to do something about the rotting food and dish piles, my one-liner is preprepared.

But, you speak of event horizons? Truly eldritch implementations? DID HE ASK FOR MORE

I know not what I speak of! I recant! I dissemble! No, not that! Anything but... the eldritch abominaeeeeeeeeeeeeeee— :skull:

There's the potential feasibility that this functionality could be implemented in a...

Actually, that's ingenious. Let's reopen this as a new feature request to improve the fetid intelligence of beartype_this_package(). In theory, beartype_this_package() should be able to internally incorporate the black magic of beartype_this_executable() by incorporating the diabolical automation you suggested to "further do away with the args by invoking inspect."
beartype_this_package() would then have two internal modes of operation:

The current package-oriented mode.
If that fails, fallback to the script-oriented mode outlined by your beartype_this_executable().

I see no fault with black magic. Let thy minions of darkness arise, Overlord @JWCS! Geh heh heh heh.


OHLC & Candle serie issue

Hello,
It seems that both the OHLC and Candle series suffer from the same rendering issue depending on how many points are displayed. The width of a candle seems to be constant no matter the amount of point shown, but instead of overlapping two candles when there are too many of them, it transform it into a simple bar (high - low)
I think the width of a candle (W) should depend on the width of the plotting area (w) the number of candles (n) and a parameter to allow for controlling the final size (p, with 90% as default, or something around this)
W = w / n * p

11 candles :

12 candles :

So I found the MaxColumnWidth which should allow me to dynamically set it up, but can you tell me how I can get the width of the plot area (i.e. axis, labels, titles, ... excluded)?

I have not investigated this issue, but to answer

can get the width of the plot area (i.e. axis, labels, titles, ... excluded)?

Instead of accessing the pot area, look for DrawMargin

The DrawMargin unit is the same as the MaxColumnWidth?
So basically if I do ToDrawMargin ( value , source , chart , axis ) with the min and max X value currently shown I should get the full extent? I will try it soon (unfortunately i'm lacking time) and post what I found.
Thanks for the answer

Yeah same units.

No, MaxColumnWidth is in Pixels
ToDrawMargin takes chart values returns pixels
FromDrawMargin takes pixels, returns chart values

No, MaxColumnWidth is in Pixels

ToDrawMargin takes chart values returns pixels
So yes. I want pixel to define MaxColumnWidth and I got them from ToDrawMargin. Basically I'll do
(ToDrawMargin(xmax)-ToDrawMargin(xmin)) / (Nxmax - Nxmin) * 95% with N the index
Only thing is that I didn't find an event for Zoom to catch when it changes. I will try through MouseWheel or smthg.

My bad, I deleted than No then.
I have not tested this, I will investigate this issue tomorrow, but as far as I see this seems like a bug, and it should be fixed inside the code of the library.
I am not sure if this is useful, but zooming and panning just changes the axis limits, Axis.MinValue and Axis.MaxValue, this range determines the visible section in the chart.
also here is where the library handles the zooming, it is actually really simple, since the axis is who handles it.
https://github.com/beto-rodriguez/Live-Charts/blob/master/WpfView/Charts/Base/Chart.cs#L846

if you change something in the core code, make that the candle width is dynamic. Financial series may plot thousands of data and do extensive use of zoom/pan

I am working with high performance right now, should be ready next week

You are right, the width of columns was being ignored when the series was crowded, last commit fixes this.


add elliott go cli

accordinng to https://groups.google.com/a/redhat.com/g/aos-team-art/c/JEIYQgTGhQE
I would like to add elliott go cli for advisory parameter

Can you tell me a command that works so that I can test it out? I tried against some of the advisories in 4.12.3 but didn't work.

@ashwindasr parameter updated, it should looks like
$ python3 ./elliott go -a 104601
Following nvrs are built with openshift-golang-builder-container-v1.16.12-202208011952.el8.g58dffad:
ose-etcd-container-v4.12.0-202212121125.p0.g89a451c.assembly.stream
Following nvrs are built with openshift-golang-builder-container-v1.19.4-202301042245.el8.g1832cfe:
atomic-openshift-cluster-autoscaler-container-v4.12.0-202301042354.p0.g6ab8e62.assembly.stream
baremetal-machine-controller-container-v4.12.0-202301042354.p0.g63dcaf1.assembly.stream
cluster-etcd-operator-container-v4.12.0-202301042354.p0.gf24e5ab.assembly.stream
cluster-monitoring-operator-container-v4.12.0-202301050335.p0.gee31d1d.assembly.stream
cluster-network-operator-container-v4.12.0-202301042354.p0.g9eafe23.assembly.stream

Can you tell me a command that works so that I can test it out? I tried against some of the advisories in 4.12.3 but didn't work.

for rpm advisory it should looks like
$ python3 ./elliott go -a 109778
Following nvrs are built with 1.19.4-2.module+el8.8.0+17709+252fe516:
buildah-1.23.4-4.rhaos4.12.el8
conmon-2.1.2-3.rhaos4.12.el8
containernetworking-plugins-1.0.1-6.rhaos4.12.el8
cri-o-1.25.2-6.rhaos4.12.git3c4e50c.el8
openshift-clients-4.12.0-202301312133.p0.gb05f7d4.assembly.stream.el8
runc-1.1.4-2.rhaos4.12.el8
skopeo-1.9.4-2.rhaos4.12.el8

Can you tell me a command that works so that I can test it out? I tried against some of the advisories in 4.12.3 but didn't work.

for rpm advisory it should looks like
$ python3 ./elliott go -a 109778
Following nvrs are built with 1.19.4-2.module+el8.8.0+17709+252fe516:
buildah-1.23.4-4.rhaos4.12.el8
conmon-2.1.2-3.rhaos4.12.el8
containernetworking-plugins-1.0.1-6.rhaos4.12.el8
cri-o-1.25.2-6.rhaos4.12.git3c4e50c.el8
openshift-clients-4.12.0-202301312133.p0.gb05f7d4.assembly.stream.el8
runc-1.1.4-2.rhaos4.12.el8
skopeo-1.9.4-2.rhaos4.12.el8


Draw.io plugin cannot edit diagram in MS Word

Preflight Checklist

x I agree to follow the Code of Conduct that this project adheres to.
x I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
Using the draw.io plugin for Desktop version of Microsoft Word on Windows 10, I could successfully insert a drawio diagram from local directory in a word document. Selecting it and clicking on 'Edit Selected' shows a pop-up error "No diagrams found in the selection". I would like to edit the diagram!
To Reproduce
Steps to reproduce the behavior:

Install draw.io add-in for Microsoft Word. Create a small file using draw.io Desktop version and save it as .drawio file on local disk.
Start new document in Microsoft word. Click on Insert -> 'draw.io' button. The draw.io panel should open on the right. Click on 'Pick Device File'. Select the file created in above step and click on 'Insert'. The image should show up in the word editor.
Click on the image to select it. Then click on 'Insert' -> 'Edit Selected' in draw.io box in ribbon.
See error "No diagrams found in the selection"

Expected behavior
I was expecting the draw.io desktop application (or draw.io website) to open for editing the diagram.
Screenshots
If applicable, add screenshots to help explain your problem.
draw.io version (In the Help->About menu of the draw.io editor):

draw.io version (Desktop 16.1.2)

Desktop (please complete the following information):

OS: Windows 10
Browser NA, I am using Microsoft Word
Version 2016 (.docx file format)

Smartphone (please complete the following information):

Device: NA
OS: NA
Browser NA
NA

Additional context
I tried creating a diagram online on draw.io, saved it to my device, and repeated the steps above. Got the same error popup.

The same issue is mentioned here: https://groups.google.com/g/drawio/c/puLz5Nrzkqg

I can confirm the above mentioned behavior on Windows 10 as well.

I have the same issue on Mac Monterey with Word for Mac 16.61

I'm using Word registered to my company account and attempting to embed a diagram from my personal OneDrive account. Inserting the diagram works fine, but after inserting the diagram nothing else works.
After inserting a diagram in a Word document, selecting it and clicking the Edit Selected toolbar button, the browser opens and I get an authentication prompt asking me to authenticate, (which is a bit annoying since I already authenticated to add the diagram in the first place). Authentication completes ok but then when the browser window tries to load the Draw.IO project it shows an error dialog stating:
API not found
Also selecting the diagram and choosing either Update Selected or Update All displays the dialog saying Updating Document. Please wait..., but the document is never updated and eventually the operation times out with the error
ADD-IN ERROR UI less Add-in command timed out. Please make sure to call event.completed() in your javascript function.

Is there any resolution on this one? The problem probably persists...

I am also running into this issue. I'm running Microsoft® Word for Microsoft 365 MSO (Version 2302 Build 16.0.16130.20378) 64-bit. Would be great if this functionality works instead of having to manually delete and reinsert diagrams

 davidjgraph mentioned this issue  Oct 2, 2023

I am also facing the same issue.
After inserting a diagram in a Word document, selecting it and clicking the Edit Selected toolbar button, getting error "No diagrams found in the selection"
Inserting the diagram by using Pick Device File then attempting to edit flow diagram, Draw.io is throwing error 'No diagrams found in the selection'.

I am facing the same issue. I am also getting the "No diagrams found in the selection" message after I inserted a draw.io file in word and try to edit it.
Also it is not possible to edit a called draw.io file when it is still in the preview window. The button is greyed out. When I press it it says that no app was found even though I installed the desktop app via the Microsoft app store.

This issue appears when adding files from device.

When the file is being added from OneDrive/SharePoint, It does respond to the edit button, but still fails to open the file in the editor.


page cannot be loaded when using lazy loading (blank)

i have tried the sample code but still not work.
<super-tabs id="mainTabs" selectedTabIndex="1" toolbarColor="light" toolbarBackground="dark" indicatorColor="light" badgeColor="light" [config]="{ sideMenu: 'left' }" (tabSelect)="onTabSelect($event)">
<super-tab [root]="page1" title="First page" icon="home" id="homeTab">
<super-tab [root]="page2" title="Second page" icon="pin" id="locationTab">
<super-tab [root]="page3" title="Third page" icon="heart" id="favouritesTab">

This code snippet is not very useful, You should give us a few lines more so that we can help you.

Did you import SuperTabsModule in your tabs module page?

I have the same problem... :-( And i already import SuperTabsModulo... can you help me please?


Fix parquet options builder calls

https://github.com/rapidsai/cudf/pull/9765 removed the enableStrictDecimalType method.
Signed-off-by: Rong Ou<EMAIL_ADDRESS>

build


$wgIframe in LocalSettings.php ignored

Any server key addition in LocalSettings.php like
$wgIframe['server']['example'] = array('scheme' => 'http', 'domain' => 'example.com');
is ignored and can not be used by a later iframe
<iframe key="example" path="" />

True, I talked with the Mediawiki poeple and the "old" and the "new" initialization algorithm working differently. Guess I will find a solution.

does this mean the extension is unusable now?

Uploaded a new version 0.06 which reads the global variable $wgIframe correctly.


Show how to take a DataAccess obj and save as keyless

[Enter feedback here]
What your not showing is how to Create a entry with a keyless from a DTO
EG
//convert the DTO to  a obj that has no key, (keyless)
var obj = _mapper.Map<KeylessDTOTable, KeylessTable>(objDTO);
_dbWaterConsentsSamples.Add(obj);
//Then add and bingo - a error.
//How do I add a keyless obj to EF Core. That's what I and many want to know how
int status = _dbWaterConsentsSamples.SaveChanges();
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: f3ffc114-df51-0be7-8972-6fcead14e011
Version Independent ID: 94c34571-7314-aab1-7898-93ac6d0f3430
Content: Tracking vs. No-Tracking Queries - EF Core
Content Source: entity-framework/core/querying/tracking.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @smitpatel
Microsoft Alias: avickers

This is tracked by https://github.com/dotnet/efcore/issues/29897


diagnostic: fix index out of bounds error in sysinfo

What is changed and how it works?

Issue Number: Close #15006
What's Changed:
Just expand cpus when it's length is insufficient. This prevents sysinfo from force reusing fixed-length cpus which cause crashes.
Related commit: https://github.com/tikv/sysinfo/commit/5a1bcf08816979624ef2ad79cfb896de432a9501

fix index out of bounds error in sysinfo.

* update sysinfo version to the personal branch

Related changes

Need to cherry-pick to the release branch

Check List
Tests

No code in TiKV
All tests in sysinfo have passed

Release note
None

/merge

/merge


Error: Cannot read property 'getDeclaredVariables' of undefined

Hi
Getting this error:
TypeError: Cannot read property 'getDeclaredVariables' of undefined
    at isOnlyUsedForObject (/path/node_modules/eslint-plugin-lodash/lib/rules/prefer-invoke-map.js:31:51)
    at isFunctionMethodCallOfParam (/path/node_modules/eslint-plugin-lodash/lib/rules/prefer-invoke-map.js:39:114)

For this code (based on SourceLocation in func) :
return Promise.all([users(), contacts(operationId)])
        .then(collections => _.map(collections, c => c.update(query, update, options)));

Any idea what's wrong?
Thanks

Hi @udiudi,
I tried running the code (without the return part, since it doesn't make sense otherwise) and it seems to work fine for me.
Can you tell me what ESLint version you're using?

@ganimomer
v3.19.0
Thanks


Add Google Analytics tracker to VMH Generator (and other VMHub pages off main iptc.org site)

This was done a long time ago...


Update to Unity 5.0?

Is it too early to update our unity assets to Unity 5.0? I downloaded Unity 5.0 to get a dev environment setup to test changes and it asked to make a one-way project conversion for 5.0.
Additionally, when attempting to build devscene.unity in unity 5.0, I get the following errors:

Building Solution: OSVR-Unity (Debug)

Building: Assembly-CSharp (Debug)

Performing main compilation...

C:\Windows\Microsoft.NET\Framework\v3.5\csc.exe /noconfig "/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\mscorlib.dll"
 -nostdlib
 "/out:C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Temp\bin\Debug\Assembly-CSharp.dll"
 "/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\System.dll"
 "/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\System.Xml.dll"
 "/r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\v3.5\System.Core.dll"
 "/r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\v3.5\System.Xml.Linq.dll"
 "/r:C:/Program Files/Unity/Editor/Data/Managed/UnityEngine.dll"
 "/r:C:/Program Files/Unity/Editor/Data/Managed/UnityEditor.dll"
 "/r:C:/Program Files/Unity/Editor/Data/UnityExtensions/Unity/GUISystem/UnityEngine.UI.dll"
 "/r:C:/Users/JeremyBell/Desktop/OSVR-Unity/OSVR-Unity/Assets/Plugins/x86/ClientKit.dll"
 /nologo
 /warn:4
 /debug:full
 /optimize-
 "/define:DEBUG;TRACE;UNITY_5_0_0;UNITY_5_0;UNITY_5;ENABLE_LICENSE_RENAME;ENABLE_NEW_BUGREPORTER;ENABLE_2D_PHYSICS;ENABLE_4_6_FEATURES;ENABLE_AUDIO;ENABLE_CACHING;ENABLE_CLOTH;ENABLE_DUCK_TYPING;ENABLE_FRAME_DEBUGGER;ENABLE_GENERICS;ENABLE_HOME_SCREEN;ENABLE_IMAGEEFFECTS;ENABLE_LIGHT_PROBES_LEGACY;ENABLE_MICROPHONE;ENABLE_MULTIPLE_DISPLAYS;ENABLE_NEW_HIERARCHY;ENABLE_PHYSICS;ENABLE_PHYSICS_PHYSX3;ENABLE_PLUGIN_INSPECTOR;ENABLE_SHADOWS;ENABLE_SINGLE_INSTANCE_BUILD_SETTING;ENABLE_SPRITES;ENABLE_TERRAIN;ENABLE_UNITYEVENTS;ENABLE_WEBCAM;ENABLE_WWW;ENABLE_AUDIOMIXER_SUSPEND;ENABLE_NONPRO;INCLUDE_DYNAMIC_GI;INCLUDE_GI;INCLUDE_IL2CPP;PLATFORM_SUPPORTS_MONO;RENDER_SOFTWARE_CURSOR;UNITY_STANDALONE_WIN;UNITY_STANDALONE;ENABLE_SUBSTANCE;ENABLE_TEXTUREID_MAP;ENABLE_RUNTIME_GI;ENABLE_MOVIES;ENABLE_NETWORK;ENABLE_MONO;ENABLE_PROFILER;UNITY_EDITOR;UNITY_EDITOR_64;UNITY_EDITOR_WIN"

 /t:library
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Demo\Scripts\Managers\ColorManager.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Demo\Scripts\Objects\ColorChanger.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Scripts\HandleButtonPress.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Scripts\SampleAnalog.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Scripts\SampleButtonScript.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\ClientKit.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\DLLSearchPathFixer.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\DeviceDescriptor.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\DisplayInterface.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\GetParent.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\InterfaceBase.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\InterfaceCallbacks.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\InterfaceGameObject.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\Math.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\OrientationInterface.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\PoseInterface.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\PositionInterface.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\SimpleJSON.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VREye.cs"
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs"
 "/nowarn:0169"
 /fullpaths
 /utf8output

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Demo\Scripts\Objects\ColorChanger.cs(25,24): error CS1061: 'UnityEngine.Component' does not contain a definition for 'materials' and no extension method 'materials' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VREye.cs(49,24): error CS1061: 'UnityEngine.Component' does not contain a definition for 'CopyFrom' and no extension method 'CopyFrom' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VREye.cs(62,21): error CS0619: 'UnityEngine.Component.camera' is obsolete: 'Property camera has been deprecated. Use GetComponent<Camera>() instead. (UnityUpgradable)'

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VREye.cs(74,32): error CS1061: 'UnityEngine.Component' does not contain a definition for 'rect' and no extension method 'rect' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VREye.cs(77,32): error CS1061: 'UnityEngine.Component' does not contain a definition for 'rect' and no extension method 'rect' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(73,36): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(74,45): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(75,46): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(79,36): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(80,45): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(81,46): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(105,21): error CS1502: The best overloaded method match for 'OSVR.Unity.VREye.MatchCamera(UnityEngine.Camera)' has some invalid arguments

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(105,44): error CS1503: Argument '1': cannot convert from 'UnityEngine.Component' to 'UnityEngine.Camera'

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(105,44): error CS0619: 'UnityEngine.Component.camera' is obsolete: 'Property camera has been deprecated. Use GetComponent<Camera>() instead. (UnityUpgradable)'

c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(145,24): error CS1061: 'UnityEngine.Component' does not contain a definition for 'fieldOfView' and no extension method 'fieldOfView' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)

Build complete -- 15 errors, 0 warnings

Building: Assembly-CSharp-Editor (Debug)

Performing main compilation...

C:\Windows\Microsoft.NET\Framework\v3.5\csc.exe /noconfig "/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\mscorlib.dll"
 -nostdlib
 "/out:C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Temp\bin\Debug\Assembly-CSharp-Editor.dll"
 "/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\System.dll"
 "/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\System.Xml.dll"
 "/r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\v3.5\System.Core.dll"
 "/r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\v3.5\System.Xml.Linq.dll"
 "/r:C:/Program Files/Unity/Editor/Data/Managed/UnityEngine.dll"
 "/r:C:/Program Files/Unity/Editor/Data/Managed/UnityEditor.dll"
 "/r:C:/Program Files/Unity/Editor/Data/UnityExtensions/Unity/GUISystem/UnityEngine.UI.dll"
 "/r:C:/Users/JeremyBell/Desktop/OSVR-Unity/OSVR-Unity/Assets/Plugins/x86/ClientKit.dll"
 "/r:C:/Program Files/Unity/Editor/Data/Managed/UnityEditor.Graphs.dll"
 "/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/androidplayer/UnityEditor.Android.Extensions.dll"
 "/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/iossupport/UnityEditor.iOS.Extensions.dll"
 "/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/wp8support/UnityEditor.WP8.Extensions.dll"
 "/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/metrosupport/UnityEditor.Metro.Extensions.dll"
 "/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/blackberryplayer/UnityEditor.BB10.Extensions.dll"
 "/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/webglsupport/UnityEditor.WebGL.Extensions.dll"
 "/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/linuxstandalonesupport/UnityEditor.LinuxStandalone.Extensions.dll"
 "/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/windowsstandalonesupport/UnityEditor.WindowsStandalone.Extensions.dll"
 "/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/macstandalonesupport/UnityEditor.OSXStandalone.Extensions.dll"
 /nologo
 /warn:4
 /debug:full
 /optimize-
 "/define:DEBUG;TRACE;UNITY_5_0_0;UNITY_5_0;UNITY_5;ENABLE_LICENSE_RENAME;ENABLE_NEW_BUGREPORTER;ENABLE_2D_PHYSICS;ENABLE_4_6_FEATURES;ENABLE_AUDIO;ENABLE_CACHING;ENABLE_CLOTH;ENABLE_DUCK_TYPING;ENABLE_FRAME_DEBUGGER;ENABLE_GENERICS;ENABLE_HOME_SCREEN;ENABLE_IMAGEEFFECTS;ENABLE_LIGHT_PROBES_LEGACY;ENABLE_MICROPHONE;ENABLE_MULTIPLE_DISPLAYS;ENABLE_NEW_HIERARCHY;ENABLE_PHYSICS;ENABLE_PHYSICS_PHYSX3;ENABLE_PLUGIN_INSPECTOR;ENABLE_SHADOWS;ENABLE_SINGLE_INSTANCE_BUILD_SETTING;ENABLE_SPRITES;ENABLE_TERRAIN;ENABLE_UNITYEVENTS;ENABLE_WEBCAM;ENABLE_WWW;ENABLE_AUDIOMIXER_SUSPEND;ENABLE_NONPRO;INCLUDE_DYNAMIC_GI;INCLUDE_GI;INCLUDE_IL2CPP;PLATFORM_SUPPORTS_MONO;RENDER_SOFTWARE_CURSOR;UNITY_STANDALONE_WIN;UNITY_STANDALONE;ENABLE_SUBSTANCE;ENABLE_TEXTUREID_MAP;ENABLE_RUNTIME_GI;ENABLE_MOVIES;ENABLE_NETWORK;ENABLE_MONO;ENABLE_PROFILER;UNITY_EDITOR;UNITY_EDITOR_64;UNITY_EDITOR_WIN"

 /t:library
 "C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\Editor\OSVRUnityBuild.cs"
 "/nowarn:0169"
 /fullpaths
 /utf8output

Build complete -- 0 errors, 0 warnings

---------------------- Done ----------------------

Build: 15 errors, 0 warnings

Looks like most of those errors are due to the fact that Unity 5 removed shortcuts to components (.camera, .materials, etc) and now requires a call to GetComponent(). It's supposed to automatically make those code changes in the upgrade process. Refactoring them manually in the existing Unity4 project wouldn't hurt. Not sure about any other problems that might arise from the upgrade process. I'll be digging into Unity5 more thoroughly tomorrow.

On Mar 8, 2015, at 10:24 AM, Sleepy Daddy Software™<EMAIL_ADDRESS>wrote:
Is it too early to update our unity assets to Unity 5.0? I downloaded Unity 5.0 to get a dev environment setup to test changes and it asked to make a one-way project conversion for 5.0.
Additionally, when attempting to build devscene.unity in unity 5.0, I get the following errors:
Building Solution: OSVR-Unity (Debug)
Building: Assembly-CSharp (Debug)
Performing main compilation...
C:\Windows\Microsoft.NET\Framework\v3.5\csc.exe /noconfig "/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\mscorlib.dll"
-nostdlib
"/out:C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Temp\bin\Debug\Assembly-CSharp.dll"
"/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\System.dll"
"/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\System.Xml.dll"
"/r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\v3.5\System.Core.dll"
"/r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\v3.5\System.Xml.Linq.dll"
"/r:C:/Program Files/Unity/Editor/Data/Managed/UnityEngine.dll"
"/r:C:/Program Files/Unity/Editor/Data/Managed/UnityEditor.dll"
"/r:C:/Program Files/Unity/Editor/Data/UnityExtensions/Unity/GUISystem/UnityEngine.UI.dll"
"/r:C:/Users/JeremyBell/Desktop/OSVR-Unity/OSVR-Unity/Assets/Plugins/x86/ClientKit.dll"
/nologo
/warn:4
/debug:full
/optimize-
"/define:DEBUG;TRACE;UNITY_5_0_0;UNITY_5_0;UNITY_5;ENABLE_LICENSE_RENAME;ENABLE_NEW_BUGREPORTER;ENABLE_2D_PHYSICS;ENABLE_4_6_FEATURES;ENABLE_AUDIO;ENABLE_CACHING;ENABLE_CLOTH;ENABLE_DUCK_TYPING;ENABLE_FRAME_DEBUGGER;ENABLE_GENERICS;ENABLE_HOME_SCREEN;ENABLE_IMAGEEFFECTS;ENABLE_LIGHT_PROBES_LEGACY;ENABLE_MICROPHONE;ENABLE_MULTIPLE_DISPLAYS;ENABLE_NEW_HIERARCHY;ENABLE_PHYSICS;ENABLE_PHYSICS_PHYSX3;ENABLE_PLUGIN_INSPECTOR;ENABLE_SHADOWS;ENABLE_SINGLE_INSTANCE_BUILD_SETTING;ENABLE_SPRITES;ENABLE_TERRAIN;ENABLE_UNITYEVENTS;ENABLE_WEBCAM;ENABLE_WWW;ENABLE_AUDIOMIXER_SUSPEND;ENABLE_NONPRO;INCLUDE_DYNAMIC_GI;INCLUDE_GI;INCLUDE_IL2CPP;PLATFORM_SUPPORTS_MONO;RENDER_SOFTWARE_CURSOR;UNITY_STANDALONE_WIN;UNITY_STANDALONE;ENABLE_SUBSTANCE;ENABLE_TEXTUREID_MAP;ENABLE_RUNTIME_GI;ENABLE_MOVIES;ENABLE_NETWORK;ENABLE_MONO;ENABLE_PROFILER;UNITY_EDITOR;UNITY_EDITOR_64;UNITY_EDITOR_WIN"
/t:library
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Demo\Scripts\Managers\ColorManager.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Demo\Scripts\Objects\ColorChanger.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Scripts\HandleButtonPress.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Scripts\SampleAnalog.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Scripts\SampleButtonScript.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\ClientKit.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\DLLSearchPathFixer.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\DeviceDescriptor.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\DisplayInterface.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\GetParent.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\InterfaceBase.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\InterfaceCallbacks.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\InterfaceGameObject.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\Math.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\OrientationInterface.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\PoseInterface.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\PositionInterface.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\SimpleJSON.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VREye.cs"
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs"
"/nowarn:0169"
/fullpaths
/utf8output
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\Sample\Demo\Scripts\Objects\ColorChanger.cs(25,24): error CS1061: 'UnityEngine.Component' does not contain a definition for 'materials' and no extension method 'materials' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VREye.cs(49,24): error CS1061: 'UnityEngine.Component' does not contain a definition for 'CopyFrom' and no extension method 'CopyFrom' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VREye.cs(62,21): error CS0619: 'UnityEngine.Component.camera' is obsolete: 'Property camera has been deprecated. Use GetComponent() instead. (UnityUpgradable)'
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VREye.cs(74,32): error CS1061: 'UnityEngine.Component' does not contain a definition for 'rect' and no extension method 'rect' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VREye.cs(77,32): error CS1061: 'UnityEngine.Component' does not contain a definition for 'rect' and no extension method 'rect' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(73,36): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(74,45): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(75,46): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(79,36): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(80,45): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(81,46): error CS1061: 'UnityEngine.Component' does not contain a definition for 'enabled' and no extension method 'enabled' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(105,21): error CS1502: The best overloaded method match for 'OSVR.Unity.VREye.MatchCamera(UnityEngine.Camera)' has some invalid arguments
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(105,44): error CS1503: Argument '1': cannot convert from 'UnityEngine.Component' to 'UnityEngine.Camera'
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(105,44): error CS0619: 'UnityEngine.Component.camera' is obsolete: 'Property camera has been deprecated. Use GetComponent() instead. (UnityUpgradable)'
c:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\OSVRUnity\src\VRHead.cs(145,24): error CS1061: 'UnityEngine.Component' does not contain a definition for 'fieldOfView' and no extension method 'fieldOfView' accepting a first argument of type 'UnityEngine.Component' could be found (are you missing a using directive or an assembly reference?)
Build complete -- 15 errors, 0 warnings
Building: Assembly-CSharp-Editor (Debug)
Performing main compilation...
C:\Windows\Microsoft.NET\Framework\v3.5\csc.exe /noconfig "/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\mscorlib.dll"
-nostdlib
"/out:C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Temp\bin\Debug\Assembly-CSharp-Editor.dll"
"/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\System.dll"
"/r:C:\Windows\Microsoft.NET\Framework\v2.0.50727\System.Xml.dll"
"/r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\v3.5\System.Core.dll"
"/r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\v3.5\System.Xml.Linq.dll"
"/r:C:/Program Files/Unity/Editor/Data/Managed/UnityEngine.dll"
"/r:C:/Program Files/Unity/Editor/Data/Managed/UnityEditor.dll"
"/r:C:/Program Files/Unity/Editor/Data/UnityExtensions/Unity/GUISystem/UnityEngine.UI.dll"
"/r:C:/Users/JeremyBell/Desktop/OSVR-Unity/OSVR-Unity/Assets/Plugins/x86/ClientKit.dll"
"/r:C:/Program Files/Unity/Editor/Data/Managed/UnityEditor.Graphs.dll"
"/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/androidplayer/UnityEditor.Android.Extensions.dll"
"/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/iossupport/UnityEditor.iOS.Extensions.dll"
"/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/wp8support/UnityEditor.WP8.Extensions.dll"
"/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/metrosupport/UnityEditor.Metro.Extensions.dll"
"/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/blackberryplayer/UnityEditor.BB10.Extensions.dll"
"/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/webglsupport/UnityEditor.WebGL.Extensions.dll"
"/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/linuxstandalonesupport/UnityEditor.LinuxStandalone.Extensions.dll"
"/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/windowsstandalonesupport/UnityEditor.WindowsStandalone.Extensions.dll"
"/r:C:/Program Files/Unity/Editor/Data/PlaybackEngines/macstandalonesupport/UnityEditor.OSXStandalone.Extensions.dll"
/nologo
/warn:4
/debug:full
/optimize-
"/define:DEBUG;TRACE;UNITY_5_0_0;UNITY_5_0;UNITY_5;ENABLE_LICENSE_RENAME;ENABLE_NEW_BUGREPORTER;ENABLE_2D_PHYSICS;ENABLE_4_6_FEATURES;ENABLE_AUDIO;ENABLE_CACHING;ENABLE_CLOTH;ENABLE_DUCK_TYPING;ENABLE_FRAME_DEBUGGER;ENABLE_GENERICS;ENABLE_HOME_SCREEN;ENABLE_IMAGEEFFECTS;ENABLE_LIGHT_PROBES_LEGACY;ENABLE_MICROPHONE;ENABLE_MULTIPLE_DISPLAYS;ENABLE_NEW_HIERARCHY;ENABLE_PHYSICS;ENABLE_PHYSICS_PHYSX3;ENABLE_PLUGIN_INSPECTOR;ENABLE_SHADOWS;ENABLE_SINGLE_INSTANCE_BUILD_SETTING;ENABLE_SPRITES;ENABLE_TERRAIN;ENABLE_UNITYEVENTS;ENABLE_WEBCAM;ENABLE_WWW;ENABLE_AUDIOMIXER_SUSPEND;ENABLE_NONPRO;INCLUDE_DYNAMIC_GI;INCLUDE_GI;INCLUDE_IL2CPP;PLATFORM_SUPPORTS_MONO;RENDER_SOFTWARE_CURSOR;UNITY_STANDALONE_WIN;UNITY_STANDALONE;ENABLE_SUBSTANCE;ENABLE_TEXTUREID_MAP;ENABLE_RUNTIME_GI;ENABLE_MOVIES;ENABLE_NETWORK;ENABLE_MONO;ENABLE_PROFILER;UNITY_EDITOR;UNITY_EDITOR_64;UNITY_EDITOR_WIN"
/t:library
"C:\Users\JeremyBell\Desktop\OSVR-Unity\OSVR-Unity\Assets\Editor\OSVRUnityBuild.cs"
"/nowarn:0169"
/fullpaths
/utf8output
Build complete -- 0 errors, 0 warnings
---------------------- Done ----------------------
Build: 15 errors, 0 warnings
—
Reply to this email directly or view it on GitHub.

@DuFF14, I assume this is finished now? Do we need to be setting up a separate package build for Unity 5?

This is done. No, I don't think we need a separate build.

Does it help check for build issues, though? (That is, if we do a build but just don't upload it)

The only issue I see currently is if new code is written in Unity 4 that doesn't use GetComponent to access components, and nobody notices. I don't think this error would be caught in an automated package build, but the Unity 5 editor is supposed to catch and fix this issue automatically at any time, not just during package import. If I change "GetComponent." to "camera.", Unity 5 offers to fix it as soon as the file is saved (this is not necessarily the best semantic fix, but it fixes the syntax). If I intentionally leave this error in, then try to export a package from Unity 5, it allows me to do that.
I'm not sure how different the automated package build process will be for Unity 5 (if at all), but it definitely wouldn't hurt to start setting things up for Unity 5. There is a good chance that differences between Unity 4 and 5 become greater as Unity adds VR specific features and optimizations in future releases (for example, if the camera component changes). I'd expect that most VR developers and OSVR contributors will be using Unity 5 very soon if they aren't already.

Well, doesn't it compile the sources in a package build? It opens the project, and I'm pretty sure it compiles the source when it does so, so it would catch that.
This is blocked on getting Unity 5 installed on the CI machines.


Add frameworkStatus to service response

This is required so that frontends can decide whether or not to show a service. Once this change is merged and the buyer frontend is updated to make use of it then the get service endpoint can be changed to always return services regardless of framework status.

:+1:


Is it possible to move a folder from one place to another with Flutter desktop?

Use case
I want to move folder1 from c:\users\jahanzaib to d:
is it possible to do via Flutter desktop? not file but full folder with all files.

Proposal

Please see flutter.dev/community for other resources for questions with your own code; the issue tracker is for bug reports and feature requests.


Checking for file existence

I recommend adding a validation for file existence since I always getting an error about file existence

public enum DiggerError: Int {
case badURL = 9981
case fileIsExist = 9982
case fileInfoError = 9983
case invalidStatusCode = 9984
case diskOutOfSpace = 9985
case downloadCanceled = -999

}
When the file exists, the download task will not be started.


Inconsistent behavior when hour=0 and am/pm is present

Hi there!
First, thanks for this awesome project!  =D
I'm seeing an inconsistent result with dateparser==0.6.0/python==3.5.2 when date_string passed in is at midnight/noon and meridian info is present. See below:
>>> import dateparser

>>> dateparser.parse('2016-12-31 01:10') is None
False

>>> dateparser.parse('2016-12-31 01:10am') is None
False

>>> dateparser.parse('2016-12-31 00:10') is None
False

>>> dateparser.parse('2016-12-31 00:10am') is None
True

Is it an issue? Am I missing something?
Thanks!

Hello @filwaitman , I don't think hour can be zero when am/pm is present? Wikipedia agrees with me.

wow, this is old.  =P
ok @wRAR, it makes sense then. Thanks!


feat: dynamic result type casting

First of all, thank you for creating this great piece of software. It is the best query builder I've used so far in 20 years.
Type casting with $castTo, $narrowType and $assertType is not always efficient. Using them with selectAll or with many columns create a lot of boilerplate. It is also impossible to make dynamic conversions with them. One easy solution is applying the conversion after the query, however it creates unneccessary boilerplate and make it more error prone.
$castUsing method solves all these problems by applying a dynamic conversion in a one-liner, without breaking the call chain. This opens up doors for flexible customizations with less boilerplate.
Example
Convert all string types to CustomString types.
type CustomString = string & { __toString: Function };
type ConvertToCustomString<T> = {
  [K in keyof T]: T[K] extends string | null
    ? (T[K] extends string ? CustomString : CustomString | null)
    : T[K];
};

await db
    .selectFrom('person')
    .select(['first_name', 'last_name'])
    .$castUsing((_) => {
      return _ as unknown as ConvertToCustomString<typeof _>;
    })
    .executeTakeFirstOrThrow()
// returns: { first_name: CustomString, last_name: CustomString | null }

Make all columns nullable without hardcoding all columns in $narrowType.
type Nullable<T> = {
  [K in keyof T]: T[K] | null;
};

const r1 = await db
    .selectFrom('person')
    .selectAll()
    .$castUsing((_) => {
      return _ as unknown as Nullable<typeof _>;
    })
    .executeTakeFirstOrThrow()

Hey 👋
Thank you!
I'm afraid this is a no-go for me.

It inelegantly uses runtime for compile-time.

It is redundant. $call + $castTo + InferResult can achieve the same.

import type { InferResult } from "kysely";

await db
  .selectFrom("person")
  .select(["first_name", "last_name"])
  .$call((qb) =>
    qb.$castTo<
      ConvertToCustomString<InferResult<typeof qb>[number]>
    >(),
  )
  .executeTakeFirstOrThrow();

await db
  .selectFrom("person")
  .selectAll()
  .$call((qb) =>
    qb.$castTo<Nullable<InferResult<typeof qb>[number]>>(),
  )
  .executeTakeFirstOrThrow();

https://kyse.link/2NHp7

Thanks for the alternative, I didn't know it was possible.


New member

Hi. I am having problems creating elastalert in Elasticsearch. When I use the elastalert-create-index command it gives me the error:
root@elk-virtual-machine:~/elastalert# elastalert-create-index
Traceback (most recent call last):
File "/usr/local/bin/elastalert-create-index", line 6, in
from pkg_resources import load_entry_point
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 3251, in
@_call_aside
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 3235, in _call_aside
f(*args, **kwargs)
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 3264, in _initialize_master_working_set
working_set = WorkingSet._build_master()
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 583, in _build_master
ws.require(requires)
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 900, in require
needed = self.resolve(parse_requirements(requirements))
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 786, in resolve
raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'jsonschema>=3.0.2' distribution was not found and is required by elastalert
root@elk-virtual-machine:~/elastalert# python -m pip install urllib3[brotli]
/usr/bin/python: No module named pip

elastalert is not maintained. I'm not sure if the developers are alive, but the project is definitely dead.

Recent changes: As of Elastalert 0.2.0, you must use Python 3.6. Python 2 will not longer be supported.

Doesn't work with python 2.7

You should only use elastalert if you can resolve past issues yourself.

https://github.com/Yelp/elastalert/issues/3178

Hi! Looks like jsonschema>=3.0.2 was not installed during the setup. Try a pip install for the same.

If you don't need help, use elastalert. Use elastalert2 if you need help.


Access Keys

You shouldn't recommend people to use their root key for the bucket upload.
Use IAM users instead

+1

+1


⚠ Avatar API Endpoint has degraded performance

In fd922b0, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 5212 ms

Resolved: Avatar API Endpoint performance has improved in c67034f after 21 minutes.


fix: Update CP4D to 4.6.2

closes: #212
Description of changes:

Update CP4D version

Output of argocd app list command or screenshot of the ArgoCD Application synchronization window showing successful application of changes in this branch.

:tada: This PR is included in version 0.17.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ValueError: Tensor._shape cannot be assigned, use Tensor.set_shape instead.

In import tflib.inception_score
91             o._shape = tf.TensorShape(new_shape)
I find ValueError: Tensor._shape cannot be assigned, use Tensor.set_shape instead.
Does anyone know how to deal with this problem, thank you

modify
O._shape = tf.TensorShape(new_shape)
for
O.set_shape(tf.TensorShape(new_shape))
and
modify
Logits = tf.matmul(tf.squeeze(pool3), w)
for
Logits = tf.matmul(tf.squeeze(pool3, [1,2]), w)
thanks for https://github.com/openai/improved-gan/pull/31

After doing this I am getting error:
AttributeError: 'NoneType' object has no attribute 'value'

Hi you solved the 'NoneType' Error


RTL Support

hi . how to handle rtl for your library ?

how to use react-table-sticky for rtl pages?

how to use rtl for fix 2 columns in react-table-sticky?


Fix building RN Android from source on Windows.

Summary

Running .\gradlew installArchives is currently broken on Windows. This is because .sh scripts were added in the codegen module, which cannot be run by the Command Prompt on Windows. It can be worked around by installing eg. Git Bash., which can be then leveraged using the code modifications in this PR, which include sanitizing mixed Linux-Windows relative paths, and other minor Windows-specific adjustments.
It's required that the user adds a Windows Environment variable storing the path to their bash binary named REACT_WINDOWS_BASH.
Pair-programmed with @davinci26
Changelog

[Android] [Fix] - Fix building React Android on Windows.
Fixes #30271

I'll add a PR to react-native-website documenting the new environment variable and bash requirements.

Did we figure out if this affects those using React Native on Windows in general vs those trying to develop React Native?
I think this would be an okay solution for the latter, but am curious of why we took this approach vs converting the script to node to be xplat. It seems like we might have added more logic to special-case Windows than the size of the script.

@NickGerleman I'm pretty sure this affects people using React Native Android on Windows, but @rectified95 can confirm. We decided the best long-term solution is converting the scripts to Node but that's not something anyone has bandwidth for right now, so we decided to use Git Bash as a short-term workaround to unblock this.

Taking a step back, it looks like the script in question is isolated. It's about 40 lines to run yarn install and move a few files. Is there more complexity or interactions that I'm missing?
It seems like a lot to ask Windows users of RN to manually set an env variables, and adding Windows specific shell logic, to avoid rewriting something of the complexity I'm seeing. RN already has a dependency on shelljs, so it could even be a pretty mechanical translation .

Taking a step back, it looks like the script in question is isolated. It's about 40 lines to run yarn install and move a few files. Is there more complexity or interactions that I'm missing?

My understanding is this codegen stuff involves several shell scripts from the react-native-codegen folder. @rectified95 can you confirm? CC @fkgozali

@lyahdav @NickGerleman
I originally thought this was only affecting people developing on Windows, since this happens while building from source.
But I just realized that cli-generated app was running v63 since I made it before the 64.rc was cut, and the codegen stuff was likely not yet in. I'm gonna create a new app with 64.rc to confirm if it's affected as well, or whether this is purely about building from source.

Confirming that on Windows I am able to run an app generated with this command:
npx react-native init my_app --version 0.64.0-rc.1. This means that the build is broken only when building from source. Given that, I beleive that having developers add an environment variable is in line with is already required of them (adding a few for Android Studio and NDK)
cc @NickGerleman

I think the change is probably okay then. I'm less concerned about Windows developers of RNW since like you mention, there's some setup already, and it's relatively rare.
I thought Gradle only called into a single one of the build scripts. If it does end up just being the single 40 line script effected, it might be worth rewriting it, but agree that it would be a tall ask if there are multiple to do in the short term.

Resolved merge conflict since PR was imported.

Resolved merge conflict since PR was imported.


App Service vs App Service Environment

In the initial diagram above you show a regular app service however in the detail architecture you discuss an App Service Environment.  Is an App Service environment required for PCI compliance?  What is the difference between regular App Service and App Service Environments from a Compliance perspective.   Trying to determine if we require an ASE or if a regular App service is good enough.  Documentation is not clear on differences from a Compliance perspective.

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

ID: e1a534e6-e60c-7cbd-5d74-1d9b5c08857b
Version Independent ID: 7dc4846b-cbfe-4a73-ee9d-806068117648
Content: Azure Security and Compliance Blueprint - PaaS Web Application for PCI DSS
Content Source: articles/security/blueprints/pcidss-paaswa-overview.md
Service: security
GitHub Login: @Meladie
Microsoft Alias: meladie

@mapoitras Thanks for your feedback! We will investigate and update as appropriate.

Hello @mapoitras, I think people just shorting App Service Environment to App Services. This service allows you to isolate your vnet. I would recommend look at this article: https://docs.microsoft.com/en-us/azure/app-service/environment/intro
and watching this video from Ignite: https://azure.microsoft.com/en-us/resources/videos/ignite-2018-what-is-new-in-azure-app-service-networking/

Hi!  Thanks for quick response.  The App Service Environment is
considerably more expensive and complicated to setup.  My client needs to
know the differences between the two environment from a compliance
perspective.  Are regular App services ISO, PCI, FedRamp compliant or is it
only the app service enviroments that are compliant.  The documentation is
really not clear.  It would be usefull to see which controls are manageable
in Regular App services vs App Service Environments.
Marc-Andre
On Wed, May 8, 2019 at 3:48 PM Meladie Espiritu<EMAIL_ADDRESS>wrote:

Hello @mapoitras https://github.com/mapoitras, I think people just
shorting App Service Environment to App Services. This service allows you
to isolate your vnet. I would recommend look at this article:
https://docs.microsoft.com/en-us/azure/app-service/environment/intro
and watching this video from Ignite:
https://azure.microsoft.com/en-us/resources/videos/ignite-2018-what-is-new-in-azure-app-service-networking/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/30821#issuecomment-490625079,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACK7ARBHWCSULTZ27ORPE3PUMVALANCNFSM4HLRUJCA
.

Hi Marc-Andre, first let me better and more specifically answer the questions you have that I can answer. 1. If you go to the first link I provided above there is a Channel 9 video about half way down the page and if you go to 1:50 seconds the Product Owner of App Services provides a slide and walks through the differences in detail. So that will tell you the difference between the App Service, ASE and Azure Stack.

We do not have specific controls mappings between regular App Service vs App Service Environments in this current Blueprint. We are working on a refresh of the Blueprint and associated documentation. I'm happy to reach out to you when the refresh moves to Public Preview to let you know it is available.

Thank you for your feedback.

Hi!  Yes.  I understand the technical difference between the different solutions.  What I'm trying to understand is do I need to deploy a App Service Environment to have ISO, PCI, NIST or FedRamp compliance or are regular App Service environments  compliant with these certifications as well.  Or maybe its App Service Environment for PCI 3.2 but regular App Service for PCI 3.1.  Or Regular App Service are compliance with FedRamp but not with PCI.  That's what I'm trying to figure out.  Its quite clear for most Azure services but for App Service is not clear at all how these certifications apply to regular App Services vs App Service Environments.

Have you looked at the Customer Implementation Matrix for PCI here: https://servicetrust.microsoft.com/ViewPage/PCIBlueprintv3 The services used to meet the requirements are listed there. Most of ASE is about isolating the network.
Also the official PCI site should be able to tell you the differences between 3.1 and 3.2 in the Change History.
From: Marc-Andre Poitras<EMAIL_ADDRESS>Sent: Friday, May 10, 2019 6:53 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Meladie Espiritu<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] App Service vs App Service Environment (#30821)
Hi! Yes. I understand the technical difference between the different solutions. What I'm trying to understand is do I need to deploy a App Service Environment to have ISO, PCI, NIST or FedRamp compliance or are regular App Service environments compliant with these certifications as well. Or maybe its App Service Environment for PCI 3.2 but regular App Service for PCI 3.1. Or Regular App Service are compliance with FedRamp but not with PCI. That's what I'm trying to figure out. Its quite clear for most Azure services but for App Service is not clear at all how these certifications apply to regular App Services vs App Service Environments.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F30821%23issuecomment-491296355&data=01|01|meladie%40microsoft.com|cdeb381d90ac4d9d362b08d6d54ec3fa|72f988bf86f141af91ab2d7cd011db47|1&sdata=WRNyeugsd6TZq5pA4P20IMZaSTamQMnXR1RvFJi%2FUkY%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHDG7PULWOQSG4V7JJE7Z6TPUV42NANCNFSM4HLRUJCA&data=01|01|meladie%40microsoft.com|cdeb381d90ac4d9d362b08d6d54ec3fa|72f988bf86f141af91ab2d7cd011db47|1&sdata=f3NNAd8T1Nmz%2FhX%2BhHewQ%2Fj2nxKF0k6Q3xtGoVybPhY%3D&reserved=0.

Biggest reason to got with ASEv2 vs App Services with vNet integrations is that we need to white-list our outbound IP. You require the ASE to do that since normal App Services can come from a large pool of changing PIPs inside MS Datacenter

@mapoitras - Did you guys ever figure this out? I am having the same questions as I plan our architecture. There is a huge price jump from Web Apps to App Service Environment and I don't feel like we would gain a log from the extra features of ASE for our use case.

Can i use Azure service fabric or AKS instead of ASE?

I think it is clear that a PCI compliant architecture can be build with ASEs.
However it is not clear if similar compliance level can be achieved with other App Service tiers. It would be great to see a PCI compliant reference architecture built with Azure App Service Premium Tier (if possible).
If using ASEs is only option then I think this should also be clearly stated.


[otbn] Fix typo in SVA bindings

This took me ages to figure out. A real "forehead to desk" moment when I finally found the width mismatch error in the VCS build log. Sigh...

The small issues that take ages to find.


Problems building mercurial for packer in sjackman/linuxbrew Docker container.

Please follow the general troubleshooting steps first:

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[x] If you're seeing permission errors tried running sudo chown -R $(whoami) $(brew --prefix)?

I'm trying to touch up the recipe for packer (I have another pull request open which balled a bunch of changes together, I'm redoing them 1 by 1).
I'm working in an sjackman/linuxbrew Docker image.
After touching up the OS value in the formula, brew install packer starts building mercurial, which dies for lack of Python (even though /usr/bin/python in the container is v2.7.6).  Here's the gist-log that the error message suggests.
After I add a dependency on Python, I still have troubles.  The details are in this gist.    In short, when I build install packer it complains about not being able to find ncurses in homebrew/dupes (which I have not yet tapped).  When I brew install mercurial directly, it taps it for me but then the installation fails.
I could use some help understanding what's going wrong.

mercurial/base85.c:13:20: fatal error: Python.h: No such file or directory
 #include <Python.h>

https://gist.github.com/anonymous/557593ffd579dad1f4f7#file-01-make-L18
That's fixed by either brew install python or sudo apt-get install python-dev or sudo yum install python-devel.

Error: No available formula with the name "homebrew/dupes/ncurses"
Please tap it and then try again: brew tap homebrew/dupes

https://gist.github.com/hartzell/c6800ae8e9ee093c1f57#file-install-output-txt-L18
I wasn't able to reproduce this error myself. Have you updated the docker image? docker pull sjackman/linuxbrew.

If mercurial depends on Python (in the sense that you need to provide it somehow) shouldn't it depend on it in the Formula?
Can the formula tell whether a prerequisite has been provided via brew or apt-get/yum/...?  Should it care?

I pulled/updated the container and have a different problem, an issue trying to make a hardlink from zipinfo to unzip.  It suspect that I'm mis-using Docker somehow.  Suggestions welcome.
I've uploaded the commands that I'm running the result I'm seeing into this gist.
I'm not working with the copy of linuxbrew in ~/.linuxbrew, but cloning my own so that I can make changes to push back to the project.  That might be an unsupported model for using the container.

If mercurial depends on Python (in the sense that you need to provide it somehow) shouldn't it depend on it in the Formula?

Yes, probably. I want to avoid installing the Homebrewed python if the system's python can be used, so I've avoided adding a hard "python" dependency to formulas. The Python requirement logic needs to be improved for Linuxbrew.

I'm not able to reproduce that unzip issue. It looks like a Docker issue/bug. Perhaps it's specific to the version of Docker. I'm using Docker version 1.7.1, build 786b29d.

I'm not working with the copy of linuxbrew in ~/.linuxbrew, but cloning my own so that I can make changes to push back to the project. That might be an unsupported model for using the container.

You can add your own remote to ~/.linuxbrew using git remote add hartzell https://github.com/hartzell/linuxbrew.git. Cloning your own is fine too, but I find the remote easier.

The unzip issue is a virtualbox/vboxfs problem, and here.
I'll work on this in another setting.

Thanks! That's helpful.


Run ELT when Airflow not installed

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/762
Originally created by @derek-knox on 2019-07-05 14:07:10

When creating an ELT pipeline schedule, it was assumed that the "Save" kicked off the initial ELT run regardless of Airflow being installed or not. This is not the case.
This issue details the user confusion that currently occurs https://gitlab.com/meltano/meltano/issues/795
Tasks

[x] @derek-knox Add run button when Airflow isn't installed
[x] @derek-knox Communicate ELT is running in UI (job log potential - https://gitlab.com/meltano/meltano/issues/751#note_188829423)
[x] FE -> BE -> FE for ELT running state

View 9 previous comments from the original issue on GitLab


Fix: typos

Fix: typos

Thanks @omahs
@MaciejBaj I have fixed the commit lint but you need to reapprove the PR


Kiwix.KiwixJS.Electron version 1.5.0-E

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This is the Electron version of Kiwix JS that is already in the winget repository. I am the lead developer of this app. The development repository is here: https://github.com/kiwix/kiwix-js-windows.


special version of puredata with only the supported objects

it might be interseting to take a generic version of puredata vanilla and removing all the objects that are not supported by heavy. then it is clear what will not work before you compile

Realistically this wouldn't happen as forking the pure data code would be substantial work.
We're improving the error reporting to show all unsupported objects as it compiles.


Wanted to know if you knew how to integrate google play subscriptions on kivy

Speaking of integrating google play payments( one time or subscriptions based) you might want to take a look at https://github.com/shashi278/IABKivy. It's not a library but a demo showing how to achieve in-app billling in kivy applications.


bump os2 for elemental toolkit v0.1.0

Signed-off-by: Date Huang<EMAIL_ADDRESS>

include in https://github.com/harvester/harvester-installer/pull/375


Automated PR: Cookstyle Changes

Hey!
I ran Cookstyle 7.32.2 against this repo and here are the results.
This repo was selected due to the topics of chef-cookbook
Changes
Issues found and resolved with resources/install.rb

5:1 refactor: Chef/Style/CopyrightCommentFormat - Properly format copyrights header comments (https://docs.chef.io/workstation/cookstyle/chef_style_copyrightcommentformat)

Issues found and resolved with resources/module.rb

5:1 refactor: Chef/Style/CopyrightCommentFormat - Properly format copyrights header comments (https://docs.chef.io/workstation/cookstyle/chef_style_copyrightcommentformat)

Issues found and resolved with resources/state.rb

5:1 refactor: Chef/Style/CopyrightCommentFormat - Properly format copyrights header comments (https://docs.chef.io/workstation/cookstyle/chef_style_copyrightcommentformat)

Released as: 6.1.18


[Off-canvas] Page height not 100%

Hello,
there an issue with height 100% when I use off-canvas. The page is not on height 100%.

To fix it I use:
.off-canvas-wrapper, .off-canvas.position-left, .off-canvas-wrapper-inner  {
height: 100%;
}

This is posing to be more complex than the snippet provided. We're still getting box shadows beneath the body content. Also getting some funky rendering issues when using the height property. `min-height

@Websitescity @tdhartwick I'm closing this issue until we have a reproducible case to test against. Please reopen with the info requested here:
https://github.com/zurb/foundation-sites/blob/develop/.github/ISSUE_TEMPLATE.md


ignore parameter name changes

Codecov Report

Merging #211 (88655af) into main (6ce0ee2) will increase coverage by 0.35%.
The diff coverage is 94.63%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #211      +/-   ##
==========================================
+ Coverage   74.44%   74.80%   +0.35%
==========================================
  Files         133      137       +4
  Lines        7573     7772     +199
==========================================
+ Hits         5638     5814     +176
- Misses       1618     1619       +1
- Partials      317      339      +22

Flag
Coverage Δ

unittests
74.80% <94.63%> (+0.35%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

diff/operations_diff.go
86.95% <85.71%> (+0.11%)
:arrow_up:

diff/path_params_map.go
86.95% <86.95%> (ø)

diff/path_items_diff.go
96.51% <95.00%> (-3.49%)
:arrow_down:

diff/components_diff.go
91.17% <100.00%> (ø)

diff/endpoints_diff.go
83.17% <100.00%> (ø)

diff/method_diff.go
92.59% <100.00%> (+0.09%)
:arrow_up:

diff/modified_paths.go
100.00% <100.00%> (ø)

diff/parameter_diff.go
82.85% <100.00%> (+0.24%)
:arrow_up:

diff/parameters_diff.go
90.19% <100.00%> (+0.47%)
:arrow_up:

diff/path_diff.go
90.76% <100.00%> (+0.44%)
:arrow_up:

... and 1 more

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

Thanks! I'm already using the new version


heart images not changing colour

I am trying the demo code in Xcode 9 beta and Swift 4. It shows the only bordered heart image. Images does not change colour. Can you please guide me on this ?

Got it worked by myself. :)

@unoiatech Can you please post the solution?


hwameistor helm chart schema is not proper

when DRBD is disable, the below two parameter(DRBDStartPort ...etc ) should not appear :
when delete the two parameter in yaml, helm validation pass.

@angel0507 take a look~

update to the lastest version, problem solved


Install the MUI Theme used for the New Manager UI

install @zesty-io/material, import the theme and pass it to the themeprovider that’s wrapping the app. All the theme overrides should apply then.


Update readme

Overview

Checklist

[x] New and updated code has appropriate documentation
[x] New and updated code has new and/or updated testing
[x] Required CI checks are passing
[x] Visual proof for any user facing features like CLI or documentation updates
[x] Linked issues closed with keywords

https://github.com/celestiaorg/optimism/tree/jcs/update-readme#readme