IronPython

Commented Issue: Problem with CreateEngine and SQL?

ants1591 — Sat, 04 Jul 2009 04:58:30 GMT

Hi, I'm trying to instantiate a python engine in a SQL stored procedure i'm coding through VS2008. However, I continue to receive this error message when the code reaches the asterisked line of the code below:

using System;
using System.Data;
using System.Data.SqlClient;
using System.Data.SqlTypes;
using Microsoft.SqlServer.Server;
/*Other imports here....*/
/*
* Python specific
* */
using IronPython;
using IronPython.Hosting;
using Microsoft.Scripting;
using Microsoft.Scripting.Hosting;

public partial class SqlProcedures
{
[Microsoft.SqlServer.Server.SqlProcedure()]
public static void BrillTagger(SqlString Phrase, out SqlString outPhrase)
{
**PROBLEM** ScriptEngine pyEngine=pyEngine = Python.CreateEngine(runtime.);
}

The error message I receive is below. Any ideas? Thanks =D
Type 'IronPython.Runtime.PythonContext' doesn't provide a suitable public constructor or its implementation is faulty: The type initializer for 'IronPython.Runtime.SysModule' threw an exception.
at Microsoft.Scripting.Utils.ReflectionUtils.CreateInstance[T](Type actualType, Object[] args)
at Microsoft.Scripting.Runtime.LanguageConfiguration.LoadLanguageContext(ScriptDomainManager domainManager, Boolean& alreadyLoaded)
at Microsoft.Scripting.Runtime.DlrConfiguration.LoadLanguageContext(ScriptDomainManager manager, LanguageConfiguration config)
at Microsoft.Scripting.Runtime.DlrConfiguration.TryLoadLanguage(ScriptDomainManager manager, String str, Boolean isExtension, LanguageContext& language)
at Microsoft.Scripting.Runtime.ScriptDomainManager.TryGetLanguage(String languageName, LanguageContext& language)
at Microsoft.Scripting.Hosting.ScriptRuntime.TryGetEngine(String languageName, ScriptEngine& engine)
at Microsoft.Scripting.Hosting.ScriptRuntime.GetEngine(String languageName)
at SqlProcedures.BrillTagger(SqlString Phrase, SqlString& outPhrase)
Comments: ** Comment from web user: ants1591 **

Thanks Rodrigo,
I moved away from this project for a little while, but will want to try your solution.

Source code checked in, #55895

IPyTeam — Fri, 03 Jul 2009 15:52:52 GMT

-------------------------------------------------------------------------------- Changeset Id: 974318 Date: 7/2/2009 4:34:21 PM (dfugate) Extended __clrtype__ metaclass tests some more. (Shelveset: CLRTYPE2;REDMOND\dfugate | SNAP CheckinId: 8842) -------------------------------------------------------------------------------- Changeset Id: 974212 Date: 7/2/2009 3:33:38 PM (dinov) 20174 1 Medium TypeError: Scope is not callable Just removing an unnecessary set of the module back onto the importing module. 18614 1 Medium Some COM errors in Ipy 2 beta 5 Issue appears to be fixed, we have a test for DisplayAlerts, adding test for dir() 20377 1 Medium Tab Completion does not work with Office 2007 Primary Interop Assemblies Issue appears to be fixed (probably w/ the recent dir cleanup), adding test. 22131 1 Medium PythonType.AddSystemInterfaces - COM Interop Issues ?? We need to handle adding when interface mapping returns a null target method. There’s several different spots we need to handle this at. Added an IL-only test case to verify this scenario is working. 21361 1 Medium Cannot write bytes to files 22302 1 Medium help() on a CLR string[] fails with a stack trace 22966 1 Medium Problem with re module/groupindex property, incompatibility with CPython Add filtering for the special named groups we have to insert. 23143 bin(n > System.Int32.MaxValue) broken We’re going through our helpers for string formatting which don’t add the – before the value and don’t expect negative values in general. Assert we have positive values, make the helpers which can’t handle negatives private, and define internal APIs w/ a simple signature which does it all. 3740 0 Medium Trivial: Overloaded properties (indexers) not handled by IronPython IronPython\2.6 This works via the same get_ method that C# supports. Adding test case to verify it keeps working. (Shelveset: StillMoreBugs;REDMOND\dinov | SNAP CheckinId: 8840) -------------------------------------------------------------------------------- Changeset Id: 973686 Date: 7/2/2009 10:43:08 AM (dinov) 3730: Trivial: Add DLLs directory in the default path Following Dave’s proposal in the bug to always add this 19521: 2.6: implement from __future__ import unicode_literals (PEP 3112) Switching to use the Unicode literal parser after this is done 19089: Implement __repr__ for enums Implement an improved repr when we have an exact value 20295 1 Medium property.__doc__ = None throws bad exception Handle the null instance 20250 1 Medium Bad exception when splatted args are None Throw TypeError instead of dereferencing the null pointer 19063 1 Medium max(tuple, none) -> nullpointer exception Already fixed, adding test case 22734 1 Medium nt.system('"...') should work Shouldn’t throw on the imbalanced quotes instead we should imply the end quote 8202 1 Low Trivial: gc.get_debug() returns the wrong value IronPython\2.6 Return an int instead of null 15546 1 Low codecs.charmap_encode throws TypeError for a dictionary with string values 18926 1 Low random module is not thread safe Fixed, adding test case 23251 - type.__clrtype__ shouldn't exist unless the clr or System modules have been imported Mark as PythonHidden, remove filtering of PythonHidden methods (in NameConverter) when defining base helper overloads 23244 - clrtype__ implementations returning None should throw a TypeError Throw ValueError if you return null Also: deleting all of the OldDynamicAction stuff – this also implies updating the default binder so that we no longer flow OldDynamicAction’s in and instead now flow a MemberRequestKind enum instead. Fixing the exception thrown when ... (Shelveset: BugsAndRemoveOldBinders;REDMOND\dinov | SNAP CheckinId: 8835)

Commented Issue: We call clrtype when the type subclass cannot even be initialized

dinov — Fri, 03 Jul 2009 06:41:14 GMT

E:\vslprf\Merlin\Main\Languages\IronPython\Tests>ipyd
IronPython 2.6 Beta 1 DEBUG (2.6.0.10) on .NET 2.0.50727.3053
Type "help", "copyright", "credits" or "license" for more information.
>>> class T(type):
... def __init__(self):
... print "this signature is blatantly incorrect"
... def __clrtype__(self):
... print "__clrtype__ should never get called because __init__ is bad"
... return int
...
>>> class X(object):
... __metaclass__ = T
...
__clrtype__ should never get called because __init__ is bad
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
TypeError: __init__() takes exactly 1 argument (4 given)
>>> ^Z
Comments: ** Comment from web user: dinov **

__clrtype__ gets called during __new__ - when we're actually instantiating the type object. __init__ won't be called until after the type is created so this is expected.

Commented Feature: Implement zlib module

jdhardy — Fri, 03 Jul 2009 00:22:52 GMT

Like the PEP for compressed libraries we should support the zlib module as well.
Comments: ** Comment from web user: jdhardy **

<shameless_plug>
There is a version of zlib for IronPython 2.6 B1 available at http://bitbucket.org/jdhardy/ironpythonzlib/.
</shameless_plug>

Commented Issue: ctypes -- fails with SHGetFileInfo

dinov — Thu, 02 Jul 2009 20:58:42 GMT

#----------------------------------------------------------------------------------------------------
IP VERSION AFFECTED: 2.6 Beta 1
BUILD TYPE: supplied as .msi
FLAGS PASSED TO IPY.EXE: none
OPERATING SYSTEM: 32-bit Vista SP1
CLR VERSION: #NET 3.5 SP1
SCENARIOS AFFECTED: affects the first non-trivial use I had of the functionality
#----------------------------------------------------------------------------------------------------
BRIEF DESCRIPTION:

Calls always fail -- even when they should throw

#----------------------------------------------------------------------------------------------------
REPRODUCTION SNIPPET:

## This unit test set passes for Python 2.6
from ctypes import *
import sys

MAX_PATH = 260
HICON = c_int

class SHFILEINFO(Structure):
_fields_ = [("hIcon", HICON),
("iIcon", c_int),
("dwAttributes", c_uint),
("szDisplayName", c_char * MAX_PATH),
("szTypeName", c_char * 80)]
SHGFI_DISPLAYNAME = 0x000000200
SHGFI_TYPENAME = 0x000000400
SHGFI_ATTRIBUTES = 0x000000800

flags = SHGFI_DISPLAYNAME | SHGFI_TYPENAME | SHGFI_ATTRIBUTES

def exercise(fun):
shfileinfo = SHFILEINFO()
v1 = fun(sys.executable,
0,
byref(shfileinfo),
sizeof(shfileinfo),
flags)
return (v1, shfileinfo.dwAttributes, shfileinfo.szTypeName)

import unittest
class ctypeTest(unittest.TestCase):
def testPositive(self):
self.assertEquals(exercise(windll.shell32.SHGetFileInfo), (1, 1077936503L, 'Application'))

def testNegative(self):
self.assertRaises(ValueError, lambda : exercise(cdll.shell32.SHGetFileInfo))

if __name__ == '__main__':
unittest.main()

#----------------------------------------------------------------------------------------------------
OTHER INFO:
# IPY 2.6 returns

FF
======================================================================
FAIL: testNegative (__main__.ctypeTest)
----------------------------------------------------------------------
Traceback (most recent call last):
File "...test.py", line 35, in testNegative
self.assertRaises(ValueError, lambda : exercise(cdll.shell32.SHGetFileInfo))
AssertionError: ValueError not raised

======================================================================
FAIL: testPositive (__main__.ctypeTest)
----------------------------------------------------------------------
Traceback (most recent call last):
File "...test.py", line 32, in testPositive
self.assertEquals(exercise(windll.shell32.SHGetFileInfo), (1, 1077936503L, 'Application'))
AssertionError: (0, 0, '') != (1, 1077936503L, 'Application')

----------------------------------------------------------------------
Ran 2 tests in 0.250s

FAILED (failures=2)
Comments: ** Comment from web user: dinov **

Part of the problem is str vs bytes (bug #22391). If you change fun(sys.executable) to fun(bytes(sys.executable) the positive test case passes. Alternately you could specify the argument types for windll.shell32.SHGetFileInfo.

Created Issue: core built-in types have some differences in attributes

dinov — Thu, 02 Jul 2009 19:49:21 GMT

Some of this is due to the fact that we have different types for certain things (e.g. wrapper_descriptors show up as built-in functions frequently in IronPython). Other cases we have different types (str vs bytes vs unicode). And some of these are just plain missing or extra functionality.

DESCRIPTION
-------------------------------------------------------
Complete list of module attributes IronPython is still
missing implementations for:
Type: <type 'builtin_function_or_method'> __eq__
Type: <type 'builtin_function_or_method'> __ge__
Type: <type 'builtin_function_or_method'> __gt__
Type: <type 'builtin_function_or_method'> __le__
Type: <type 'builtin_function_or_method'> __lt__
Type: <type 'builtin_function_or_method'> __ne__
Type: <type 'builtin_function_or_method'> __eq__
Type: <type 'builtin_function_or_method'> __ge__
Type: <type 'builtin_function_or_method'> __gt__
Type: <type 'builtin_function_or_method'> __le__
Type: <type 'builtin_function_or_method'> __lt__
Type: <type 'builtin_function_or_method'> __ne__
Type: <type 'builtin_function_or_method'> __eq__
Type: <type 'builtin_function_or_method'> __ge__
Type: <type 'builtin_function_or_method'> __gt__
Type: <type 'builtin_function_or_method'> __le__
Type: <type 'builtin_function_or_method'> __lt__
Type: <type 'builtin_function_or_method'> __ne__
Type: <type 'datetime.datetime'> strptime
Type: <type 'datetime.timedelta'> __rdiv__
Type: <type 'datetime.timedelta'> __rfloordiv__
Type: <type 'dictproxy'> __cmp__
Type: <type 'dictproxy'> __ge__
Type: <type 'dictproxy'> __gt__
Type: <type 'dictproxy'> __le__
Type: <type 'dictproxy'> __lt__
Type: <type 'file'> errors
Type: <type 'file'> readinto
Type: <type 'getset_descriptor'> __delete__
Type: <type 'getset_descriptor'> __name__
Type: <type 'getset_descriptor'> __objclass__
Type: <type 'getset_descriptor'> __set__
Type: <type 'instancemethod'> __cmp__
Type: <type 'method-wrapper'>/Type: <type 'builtin_function_or_method'> __objclass__
Type: <type 'str'>/Type: <type 'bytes'> __getnewargs__
Type: <type 'str'>/Type: <type 'bytes'> __getslice__
Type: <type 'str'>/Type: <type 'bytes'> __mod__
Type: <type 'str'>/Type: <type 'bytes'> __rmod__
Type: <type 'str'>/Type: <type 'bytes'> _formatter_field_name_split
Type: <type 'str'>/Type: <type 'bytes'> _formatter_parser
Type: <type 'str'>/Type: <type 'bytes'> encode
Type: <type 'str'>/Type: <type 'bytes'> format
Type: <type 'type'> __abstractmethods__
Type: <type 'type'> __basicsize__
Type: <type 'type'> __dictoffset__
Type: <type 'type'> __eq__
Type: <type 'type'> __ge__
Type: <type 'type'> __gt__
Type: <type 'type'> __itemsize__
Type: <type 'type'> __le__
Type: <type 'type'> __lt__
Type: <type 'type'> __ne__
Type: <type 'type'> __weakrefoffset__
Type: <type 'wrapper_descriptor'> __call__
Type: <type 'wrapper_descriptor'> __name__
Type: <type 'wrapper_descriptor'> __objclass__

-------------------------------------------------------
Complete list of module attributes that should be removed
from IronPython:
Type: <type 'bool'> __eq__
Type: <type 'bool'> __ne__
Type: <type 'builtin_function_or_method'> __get__
Type: <type 'builtin_function_or_method'> __get__
Type: <type 'builtin_function_or_method'> __get__
Type: <type 'datetime.date'> __getnewargs__
Type: <type 'datetime.datetime'> __getnewargs__
Type: <type 'datetime.timedelta'> __getnewargs__
Type: <type 'instancemethod'> __eq__
Type: <type 'instancemethod'> __ne__
Type: <type 'list'> __radd__
Type: <type 'method-wrapper'>/Type: <type 'builtin_function_or_method'> __get__
Type: <type 'method-wrapper'>/Type: <type 'builtin_function_or_method'> __module__
Type: <type 'method_descriptor'> __cmp__
Type: <type 'str'> __radd__
Type: <type 'str'> isdecimal
Type: <type 'str'> isnumeric
Type: <type 'str'> isunicode
Type: <type 'str'>/Type: <type 'bytes'> __radd__
Type: <type 'str'>/Type: <type 'bytes'> fromhex
Type: <type 'tuple'> __radd__
Type: <type 'type'> __clrtype__
Type: <type 'type'> __dir__
Type: <type 'type'> __getitem__
Type: <type 'unicode'>/Type: <type 'str'> __radd__
Type: <type 'unicode'>/Type: <type 'str'> isunicode
Type: <type 'wrapper_descriptor'>/Type: <type 'method_descriptor'> __cmp__

Created Issue: Classes which define operator + on themselves also get an radd method

dinov — Thu, 02 Jul 2009 19:01:46 GMT

Currently if you have a type like:

public class Foo {
public static Foo operator +(Foo a, Foo b) {
return null;
}
}

We will define both an __add__ and an __radd__ method for the type. We currently need to do this because we don't define all of the overloads for addition for types such as complex. Instead we rely upon the fact that we have conversions from say int->complex and the fact that we've defined __radd__ which takes complex, complex. We then do the conversion and everything works.

Instead we should have explicitly defined operator + (int, complex) which wil turn into an __radd__ method. Then we can update our mapping for reverse operators to check and make sure that the 1st type is not the same as the 2nd type and avoid exposing these __radd__ methods.

This shows up on many types such as array.array, buffer, bytearray, list, etc...

Created Issue: Create a user-friendly, Pythonic way to access .NET custom attributes

dfugate — Thu, 02 Jul 2009 18:21:13 GMT

Assuming there's some attribute, XyzAttribute, on a C# class, Abc, accessible from IronPython; one has to use the following bizarre incantation to read its field values:
#Get 'Abc' in your module somehow
import clr
abc_clrtype = clr.GetClrType(Abc)
attribute_field_value_i_am_looking_for = abc_clrtype.GetCustomAttributes(XyzAttribute, True)[0].WhateverFieldINeed

I'd argue this is needlessly complicated and hardly Pythonic. One should just be able to do something like:
#Get 'Abc' in your module somehow
import clr #to expose .NET members on classes
attribute_field_value_i_am_looking_for = Abc.__clrattrs__["XyzAttribute"][0].WhateverFieldINeed
or even:
attribute_field_value_i_am_looking_for = Abc.XyzAttribute.WhateverFieldINeed

Commented Issue: Incompat: Open existing file with O_CREAT

gz — Thu, 02 Jul 2009 17:30:24 GMT

<Thanks>jdhardy</Thanks>
<Test>Needed</Test>

There is an inconsistency with how Ipy 2.0 RC1 and Python 2.5 handle an existing file opened with O_CREAT:

Python 2.5.2 (r252:60911, Feb 21 2008, 13:11:45) [MSC v.1310 32 bit (Intel)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import os
>>> fd = os.open('test.txt', os.O_WRONLY | os.O_CREAT)
>>>

IronPython 2.0 (2.0.0.0) on .NET 2.0.50727.3053
Type "help", "copyright", "credits" or "license" for more information.
>>> import os
>>> fd = os.open('test.txt', os.O_WRONLY | os.O_CREAT)
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
WindowsError: (17, "[Errno 17] The file 'C:\\Users\\Jeff\\test.txt' already exists.")
>>>
Comments: ** Comment from web user: gz **

Looks trivially fixable at FileModeFromFlags in IronPython_Main/Src/IronPython.Modules/nt.cs

Affects running bazaar under IronPython.

Created Issue: Implement 'mbcs' codec

gz — Thu, 02 Jul 2009 17:25:46 GMT

Used for translating between windows "ansi" strings and unicode.

IronPython 2.6 Beta 1 (2.6.0.10) on .NET 2.0.50727.3082
Type "help", "copyright", "credits" or "license" for more information.
>>> "Test".decode("mbcs")
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
LookupError: unknown encoding: mbcs

Affects running bazaar under IronPython.

Created Issue: A buffer object should be directly writable to a cStringIO object

gz — Thu, 02 Jul 2009 17:25:39 GMT

IronPython 2.6 Beta 1 (2.6.0.10) on .NET 2.0.50727.3082
Type "help", "copyright", "credits" or "license" for more information.
>>> from cStringIO import StringIO
>>> StringIO().write(buffer("Test"))
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
TypeError: expected str, got buffer

Affects running bazaar under IronPython.

Created Issue: Removing a open file should raise an error with errno set to EACCES

gz — Thu, 02 Jul 2009 17:25:28 GMT

Subset of the [workitem:13702] issue.

IronPython 2.6 Beta 1 (2.6.0.10) on .NET 2.0.50727.3082
Type "help", "copyright", "credits" or "license" for more information.
>>> import System, errno, os
>>> f = file("test", "w")
>>> try:
... os.remove("test")
... except OSError, e:
... pass
...
>>> e
WindowsError(16, "[Errno 16] The process cannot access the file 'C:\\test' because it is being used by another process.")
>>> e.clsException
<System.ComponentModel.Win32Exception object at 0x000000000000002C [System.ComponentModel.Win32Exception...]>
>>> e.errno, e.args[0], errno.EACCES, e.clsException.NativeErrorCode
(22, 16, 13, 32)
>>> f.close()

Affects running bazaar under IronPython.

Created Issue: Removing a read-only file should raise an error with errno set to EACCES

gz — Thu, 02 Jul 2009 17:25:17 GMT

Subset of the [workitem:13702] issue.

IronPython 2.6 Beta 1 (2.6.0.10) on .NET 2.0.50727.3082
Type "help", "copyright", "credits" or "license" for more information.
>>> import System, errno, os, stat
>>> file("test", "w").close()
>>> os.chmod("test", stat.S_IREAD)
>>> try:
... os.remove("test")
... except OSError, e:
... pass
...
>>> e.errno == errno.EACCES
False
>>> e
WindowsError(13, "[Errno 13] Access to the path 'C:\\test' is denied.")
>>> e.clsException
<System.ComponentModel.Win32Exception object at 0x000000000000002B [System.ComponentModel.Win32Exception...]>
>>> e.errno, e.args[0], errno.EACCES, e.clsException.NativeErrorCode
(22, 13, 13, 5)

Affects running bazaar under IronPython.

Created Issue: file objects need 'buffering' keyword argument

gz — Thu, 02 Jul 2009 17:25:07 GMT

IronPython 2.6 Beta 1 (2.6.0.10) on .NET 2.0.50727.3082
Type "help", "copyright", "credits" or "license" for more information.
>>> file("test", "w", buffering=1).close()
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
TypeError: __init__() got an unexpected keyword argument 'buffering'

Affects running bazaar under IronPython.

Created Issue: We call clrtype when the type subclass cannot even be initialized

dfugate — Thu, 02 Jul 2009 16:16:34 GMT

Created Issue: Add support for IStructuralComparable for .NET 4.0

dinov — Wed, 01 Jul 2009 23:49:18 GMT

.NET 4.0 defines IStructuralComparable for types which support structure equality. We should define an IEqualityComparer for use with these types and turn our __eq__/__ne__/__hash__ implementations into calls w/ these types. This will give us value equality semantics. We should consider whether or not we want to special case arrays out of this because it would be a breaking change.

Created Issue: IronPython TokenCategorizer missing whitespace and duplicating new lines

harrypierson — Wed, 01 Jul 2009 20:15:10 GMT

Using IronPython 2.6 Beta 1

For example, if you tokenize the string "def foo(a,b):\n return a + b", you get the following tokens: (Category, start, end, length, and the token text)

Keyword 0 3 3 'def'
Identifier 4 7 3 'foo'
Grouping 7 8 1 '('
Identifier 8 9 1 'a'
Delimiter 9 10 1 ','
Identifier 10 11 1 'b'
Grouping 11 12 1 ')'
Delimiter 12 13 1 ':'
WhiteSpace 13 16 3 '\n '
Operator 13 16 3 '\n '
Keyword 16 22 6 'return'
Identifier 23 24 1 'a'
Operator 25 26 1 '+'
Identifier 27 28 1 'b'
WhiteSpace 28 28 0 ''

Note missing whitespace between the first two tokens (first token end at pos 3 and second token starts at pos 4. What about the whitespace between them?)
Also note the newline in the string is tokenized twice - first as WHitespace and then as an operator

Created Issue: ACCESSIBILITY: there's not enough of a color contrast for stderr under -X:ColorfulConsole

dfugate — Wed, 01 Jul 2009 17:51:29 GMT

Using the default background color of cmd.exe (black), the specific red color we choose for stderr (specifically FF0000) under -X:ColorfulConsole doesn't have a high enough color contrast. We need to choose a shade of red that's a bit brighter.

#---------------------------------------------------------------------------
#--Output from "Colour Contrast Analyser" tool (http://www.paciellogroup.com/resources/contrast-analyser.html#download)

Foreground:#FF0000 Background:#000000

The contrast ratio is: 5.3:1

Text passed at Level AA

Text failed at Level AAA

Large text passed at Level AA

Large text passed at Level AAA

SC 1.4.3 Contrast (Minimum): The visual presentation of text and images of text has a contrast ratio of at least 4.5:1, except for the following: (Level AA) Large Print: Large-scale text and images of large-scale text have a contrast ratio of at least 3:1;

SC 1.4.6 Contrast (Enhanced): The visual presentation of text and images of text has a contrast ratio of at least 7:1, except for the following: (Level AAA) Large Print: Large-scale text and images of large-scale text have a contrast ratio of at least 4.5:1

Note: 1. Text or images of text that are part of an inactive user interface component, that are pure decoration, that are incidental text in an image, or that are not visible to anyone, have no minimum contrast requirement. 2. Text that is part of a logo or brand name has no minimum contrast requirement.

Created Issue: ACCESSIBILITY: at high resolutions the ipy icon looks like "IPV"

dfugate — Wed, 01 Jul 2009 17:26:40 GMT

See attachment.

Created Issue: Icons for ipy.exe and ipyw.exe both have 'ipy' image

dfugate — Wed, 01 Jul 2009 17:24:05 GMT

It's a bit confusing that both ipy.exe and ipyw.exe both have the same 'ipy' icon. IMO, this is very low priority fix.