text:
JanDeDobbeleer.OhMyPosh version 6.42.1

[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] Have you validated your manifest locally with winget validate --manifest <path>?
[x] Have you tested your manifest locally with winget install --manifest <path>?
[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

I moved to allowing an install for all users as well as the current user meaning there's a popup now asking this context to the user. What's the best way to pass the validation with that change? There's not yet a way to override arguments if I read correctly.

You'll need to add a command line argument to your Inno Setup file that controls whether to install for all users or just the current one (https://stackoverflow.com/questions/3618257/is-it-possible-to-accept-custom-command-line-parameters-with-inno-setup). Then, you can add multiple installer entries to your manifest with different scopes that include said command line argument, like so:
Installers:
- Architecture: x64
  InstallerUrl: https://github.com/JanDeDobbeleer/oh-my-posh/releases/download/v6.42.1/install-amd64.exe
  InstallerSha256: 588963560DA039F79FBF58E93A139A1C4C4681BC870C1C26AE3A74A5D3B387EA
  Scope: user
  InstallerSwitches:
      Custom: /justme
- Architecture: x64
  InstallerUrl: https://github.com/JanDeDobbeleer/oh-my-posh/releases/download/v6.42.1/install-amd64.exe
  InstallerSha256: 588963560DA039F79FBF58E93A139A1C4C4681BC870C1C26AE3A74A5D3B387EA
  Scope: machine
  InstallerSwitches:
     Custom: /allusers
- Architecture: arm64
  InstallerUrl: https://github.com/JanDeDobbeleer/oh-my-posh/releases/download/v6.42.1/install-arm64.exe
  InstallerSha256: B97E075E5885E911A81F42FE18EDE0B240DE6A8C0F42D8797BBDF977F7BA1492
  Scope: user
  InstallerSwitches:
      Custom: /justme
- Architecture: arm64
  InstallerUrl: https://github.com/JanDeDobbeleer/oh-my-posh/releases/download/v6.42.1/install-arm64.exe
  InstallerSha256: B97E075E5885E911A81F42FE18EDE0B240DE6A8C0F42D8797BBDF977F7BA1492
  Scope: machine
  InstallerSwitches:
      Custom: /allusers

(as an example, of course you'll need to setup the switches how you like).

The switches already exist so that should be straightforward. How does that work from a consumer perspective with those changes? As in, how does a user select which one to run?

@JanDeDobbeleer The user can use the --scope argument from the command line (or set a preference in winget settings) to select it. By default, winget will use the user scoped one.
winget install JanDeDobbeleer.OhMyPosh --scope machine installs it for the entire machine for example.

authors: JanDeDobbeleer jedieaston


text:
Update YamlCreate.ps1 with InputObject functionality

Result: Installation Failed

Manifest
Add/Remove Programs

Name
SiYuan

Version
2.0.27

Publisher
Yunnan Liandi Technology Co., Ltd.

ProductCode
c1e77cb7-68e1-5adb-ab29-ecf7379a9944, c1e77cb7-68e1-5adb-ab29-ecf7379a9944

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 2763
Microsoft Reviewers: Open in CodeFlow

@vedantmgoyal2009 I assume this is part of a bug you're working to fix 😊

source: gharchive/pull-request

repo: microsoft/winget-pkgs

license: MIT

license_type: permissive

license_source: github-api


text:
Added Firefox Developer Edition version 104.0b1.

[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] Have you validated your manifest locally with winget validate --manifest <path>?
[x] Have you tested your manifest locally with winget install --manifest <path>?
[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

@winetbot run

@wingetbot run

authors: Trenly jedieaston


text:
WinDBG Preview still locks pdb files

When you debug driver in source mode, WinDBG Preview loads pdb file for that driver, but even if you unload driver, WinDBG locks pdb, and Visual Studio can't rewrite it during driver recompilation. Constantly have to restart WinDBG Preview every time, when driver changes.
I saw that behavior in WinDBG, but, i hope, it can be fixed in WinDBG Preview.

If you add "cache*" to the front of your symbol path, that should fix this.

Thanks, advice works! With cache* option WinDBG Preview stops to lock driver pdb file immediately after driver unloaded in debugging OS.

source: gharchive/issue

repo: microsoftfeedback/WinDbg-Feedback


text:
[Sample Queries] GET list of chat members + GET member in chat

Collection of:
https://github.com/LokiLabs/microsoft-graph-devx-content/pull/5
https://github.com/LokiLabs/microsoft-graph-devx-content/pull/8
Blocked from merging until:
https://github.com/LokiLabs/microsoft-graph-explorer-v4/pull/34
Summary:
Adding two new sample queries for the Teams (beta) chat api calls.

This PR was made against the wrong repo.. my apologies :c


text:
Microsoft.Graph/groups 'owners' property is altered to include user who triggered deployment even when 'owners' is explicitly defined in Bicep

Bicep version
Bicep CLI version 0.29.47 (132ade51bc)
Resource and API version
Microsoft.Graph/groups@v1.0
Auth flow
Interactive (performed manually via Azure Cli e.g. az deployment group create ...
Deployment details
Deployment correlation id: 8793c575-bf90-4842-b4de-8a48f87dcbd6
Graph client request id: 9a3b3ca3-00b4-4c0f-bcb3-aae04cb357f1
Graph request timestamp: 2024-08-28T16:05:24Z
Describe the bug
Attempting to create a new Entra ID Group whilst providing the owners property (as a string[] of valid, pre-existing, Entra ID User ID's) throws the following deployment error:
{
    "status": "Failed",
    "error": {
        "code": "DeploymentFailed",
        "target": "/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/my-resource-group/providers/Microsoft.Resources/deployments/MyDeployment_v1",
        "message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.",
        "details": [
            {
                "code": "",
                "message": "{\"error\":{\"code\":\"BadRequest\",\"target\":\"/resources/myGroup\",\"message\":\"Request contains a property with duplicate values. Graph client request id: 00000000-0000-0000-0000-000000000000. Graph request timestamp: 2024-08-28T16:05:24Z.\"}}"
            }
        ]
    }
}

This appears to be related to the owners property as:

if this group property is commented out, the group is created successfully with the user who is logged in to the Azure Cli to trigger the deployment being listed as the single owner (as suggested would be the case in the docs as noted in 'Additional Context' below)
Once the group already exists, if the owners property is re-added, the deployment runs as expected and the groups owners are updated as expected with the users listed in owners: []

To Reproduce
Using the below template:
resource myGroup 'Microsoft.Graph/groups@v1.0' = {
  displayName: 'My Group'
  uniqueName: 'unique-identifier-for-my-group'
  description: 'A group to evidence the bug'
  securityEnabled: true
  mailEnabled: false
  mailNickname: '-'
  owners: [
    '10000000-0000-0000-0000-000000000000',
    '20000000-0000-0000-0000-000000000000',
    '30000000-0000-0000-0000-000000000000'
  ]
}

replace the owner GUID placeholders with actual Azure Entra ID User ID's from the tenant you intend to deploy to ensuring that one of them is the ID of the user who will be triggering this deployment, then attempt to create this new group. You should see the deployment fail with the error above:

Request contains a property with duplicate values.

Additional context
The Microsoft Graph Bicep > Group documentation states the following regarding the owners property:

The owners of the group. Limited to 100 owners. Nullable. If this property isn't specified when creating a Microsoft 365 group, the calling user is automatically assigned as the group owner

It is my strong suspicion that the 'duplicate value' the deployment error is due to the User ID of the currently logged in user who undertakes the deployment (the "calling user" in the above docs snippet) is added to the owners array at all times not just when the "property isn't specified when creating" as suggested above. e.g. if you ran the above whilst logged in as Entra ID User 20000000-0000-0000-0000-000000000000 you'd end up with the following array:
        "myGroup": {
            "import": "microsoftGraph",
            "type": "Microsoft.Graph/groups@v1.0",
            "properties": {
                "displayName": "My Group",
                "uniqueName": "unique-identifier-for-my-group",
                "description": "A group to evidence the bug",
                "securityEnabled": true,
                "mailEnabled": false,
                "mailNickname": "-",
                "owners": [
                  '10000000-0000-0000-0000-000000000000',
                  '20000000-0000-0000-0000-000000000000',
                  '30000000-0000-0000-0000-000000000000',
                  '20000000-0000-0000-0000-000000000000'
                ]
            }
        }

Note that the resulting ARM template from these failed deployments does not reference any alterations to the Bicep inputs, so this modification of the owners array must be undertaken at a later stage, probably by the Microsoft Graph Bicep Extension

As a slight aside, the mailNickname property for groups is also required at all times, including when mailEnabled: false as per the docs:

The mail alias for the group, unique for Microsoft 365 groups in the organization. Maximum length is 64 characters. This property can contain only characters in the ASCII character set 0 - 127 except the following characters: @ () / [] ' ; : <> , SPACE. Required

As can be seen in my sample above, I am forced to provide this and use a placeholder/workaround value of - as an empty string or a space is not valid.
Surely this property should only be required if mailEnabled: true ?

@HARVS1789UK-VAIIE Yeah - that's a separate issue (mailNickname being a required property on creation for all use cases). It's actually a Microsoft Graph API issue (not an issue with the Graph Bicep extension - although we made the property required in the Bicep type, to mirror the Graph API.
I can raise that as a platform issue with the team that owns the Groups API.
We'll investigate the owners issue separately and come back to you shortly, but thanks for reporting!

@HARVS1789UK-VAIIE Quick investigation on the duplicate values - it looks like this is caused by the Microsoft Graph Groups API - so yes it is adding the current signed-in user to the list of users to add as an owner.  If you add that same user to the owners list, it errors.
Again, this isn't something that the Graph Bicep extension is doing.
I can raise this with the Groups API team - it seems like if the duplicate value is for the signed-in user, they should remove that duplicate value from the list and go ahead with the operation.

Thank you for the investigation and feedback @dkershaw10 and I appreciate your offer to raise both of these points with the Groups API team, I think that would be worthwhile if you would be willing to.
It sounds like outside of the above, the underlying issue(s) here are out of your control and not directly caused by the Graph Bicep extension.
Can I suggest that the one directly actionable outcome from the Graph Bicep side would be to update the owners property description below to make clear that the current user will be added as an owner of the group in all cases (due to the Groups API, as yo've identified):
From the Microsoft Graph Bicep > Group documentation:

The owners of the group. Limited to 100 owners. Nullable. If this property isn't specified when creating a Microsoft 365 group, the calling user is automatically assigned as the group owner

Thanks for the timely response 👍

I've filed bugs with the Groups team.
For the documentation issue - I can ask one of our content writers to make an update to the Microsoft Graph REST API reference docs. The Graph Bicep reference docs source descriptions from there.

@HARVS1789UK-VAIIE The public Graph REST API docs have been updated here - see the owners relationship property description. It links to a new known issue entry for the issue you raised here.
While the Groups team agreed this issue is likely a bug, I have no ETA on a fix or whether they will fix this issue.
Is it OK if we close this issue, or would you like it to remain open to track the possible bug fix?

@dkershaw10 Thanks for the update.
It would likely be worth updating the docs re: owners on the Graph Bicep Groups docs as well to match the Graph API Groups additions, but yes I am happy this can be closed.
For now, I am just excluding myself from my parameterised list of owners whenever it is me running the deployment.

@HARVS1789UK-VAIIE Thanks for the response.  Yes - this will be updated in the Graph Bicep Groups docs, at our next update (as the descriptions from the REST API docs are pulled into the Graph Bicep reference topics).
Thanks for building in a workaround.
It's kind of a shame that the Bicep environment() function doesn't return the calling principal's IDs (like appId, userId etc) - that way they could be excluded no matter who runs the deployment.  That said, the proper fix would be in the Groups API.

repo: microsoftgraph/msgraph-bicep-types


text:
Repo name doesn't indicate this is a sample

Feedback from HackNYC: the messaging around this repo is unclear. The name would seem to indicate that it is a library, but the README indicates that it is a sample.
If it is a library, we should offer a nuget pkg w/ integration instructions, etc.
Can we include something in our namespacing like Graph.Microsoft.Sample?

Renamed and transferred the sample. This should address the potential confusions you mention.

Thank you for following up, @jamescro!

authors: iambmelt jamescro


text:
Table output library inclusion...

Need to pick a table formatting library for CLI output.
Something like: https://docs.rs/cli-table/latest/cli_table/
Implement for tdi me as a test.

PR #23

authors: kelim-msft kitplummer

repo: microsofthackathons/tdi


text:
Deprecated: Synchronous XMLHttpRequest

In Chrome, this is shown in the console:
api.f893cafb33554c8c96437b32fc6934b7.1.1.8.js:14 [Deprecation] Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.
send @ api.f893cafb33554c8c96437b32fc6934b7.1.1.8.js:14

That line 14 then refers to minimized jQuery v3.3.1 code

Same warning in Firefox by the way.

repo: microweber/microweber


text:
Custom Generated Templates

I love that the article command is back, it's such a time saver!
It would be awesome if we could actually supply our own template, what do you guys think?
I'd like to add more to the front matter
Maybe set it in the config file for a article.tt path
https://github.com/middleman/middleman-blog/blob/master/lib/middleman-blog/commands/article.tt

Already there:   blog.new_article_template = 'blank-article'

authors: FearGoidte ericraio

repo: middleman/middleman-blog

license: mit

license_source: bigquery


text:
Adds function ExchangeWithConn

#614 requested that connection reuse be implemented in this library to help improve performance when using TCP/TCP-TLS connections.  This was rejected, fairly, because of the complexity in handling that logic within the library.  This PR would solve the problem without introducing connection management code into the library at all.  Instead it does some minor refactoring in order to expose the ability for the caller to manage connections and pass them in to the library via the ExchangeWithConn function.  The existing Exchange behavior is preserved, so backwards compatibility shouldn't be an issue.

Codecov Report

Merging #1110 into master will increase coverage by 0.00%.
The diff coverage is 77.77%.

@@           Coverage Diff           @@
##           master    #1110   +/-   ##
=======================================
  Coverage   55.59%   55.60%
=======================================
  Files          41       41
  Lines       10067    10068    +1
=======================================
+ Hits         5597     5598    +1
  Misses       3441     3441
  Partials     1029     1029

Impacted Files
Coverage Δ

client.go
70.05% <77.77%> (+0.15%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d128d10...e811691. Read the comment docs.

this looks good, thank you
/lgtm

authors: codecov-io miekg mostfunkyduck

repo: miekg/dns


text:
upgraded .NET to v4.5.1

support GetFunctionPointerForDelegate<Action<T1,T2>>
*small syntax fixes VS2015

Mhm, looks like we had a conflict in the middle, would you mind updating the change

authors: goloskokovic migueldeicaza

repo: migueldeicaza/TensorFlowSharp


text:
Question: difference between AMD64 and X86 64 bit Debian packages

Hi,
I was wondering if you could document the difference between AMD64 and X86 64 bit Debian packages.
At https://github.com/mike-goodwin/owasp-threat-dragon-desktop/releases I can only see
Debian package for AMD64 : threatdragon_1.2.0_amd64.deb
Debian package for X86 64 bit : threatdragon_1.2.0_x86_64.deb

They also have different sizes, but just looking at this I don't know what the difference is and which I should use.

Hello @MichaelGissingNC - it depends on the processor type in your Linux machine. If in doubt do you want to try the Linux AppImage? OWASP-Threat-Dragon-1.2.0.AppImage

@jgadsden What's the difference? Debian only knows amd64 and all I know is that x86_64 usually is the same ISA as amd64.

Hello @MichaelGissingNC - as far as I know the two packages are subtly different according to the type of hardware platform. Can you run lscpu and see what processor type you have?
For example on my machine:
threat-dragon-desktop$ lscpu
Architecture:          x86_64
...

I can modify the release notes to make this more clear

@jgadsden It's x86_64 as well, but in Debian terminology this is amd64 for packages. In all my encounters with architectures in recent years, these two are synonyms.

Ah, I have had different experiences and need to use the correct package on my Trisquel box. I have updated the release notes, does this provide enough clarity in these notes? Happy to apply any changes that you want to see

OK, I had a closer look at the packages, they also differ in installed files, installation target and dependencies.
Can you point me to the build scripts for the packages? Maybe I can figure out what the different packages are supposed to do.

Sure, there are two files that determine this: electron-builder.json and installer-lin.js. If you see any tidy-ups then feel free to raise a pull-request.
Note that Mike's repos have been migrated to https://github.com/OWASP/threat-dragon-desktop and so any pull-request should be done in that repo

authors: MichaelGissingNC jgadsden


text:
Update ember-cli-babel to the latest version for node.js 8

Also specify the version using a version range.
This upgrade makes it nodejs 8 compatible.

Comparable to https://github.com/mike-north/ember-lodash/pull/109 but using a version range instead again.

Is it possible to get this merged? Otherwise we can't use node.js in version 8.

repo: mike-north/ember-lodash


text:
[Feature Request] Set default enum conversion

At the moment RepoDb is using NVARCHAR/TEXT as default conversion for enums and offers the possibility to change these either per enum type via the TypeMapper or per property via the FluentMapper. It also checks the database schema, but this is not possible in all scenarios.
I usually always want my enums to be ints in the database and would like to be able to set a default enum conversion for RepoDb so I don't have to map each enum manually.
I would suggest something like this, that can be called when doing the rest of the RepoDb setup:
RepoDb.TypeMapper.DefaultEnumConversion = EnumConversion.Int;
RepoDb.TypeMapper.DefaultEnumConversion = EnumConversion.Nvarchar;

Using an enum and not bool since more options might be nice and I think it is clearer in this instance too.

Perhaps two or more of the above situations will occur at the same time in the same system.
So I recommend using ConverterFactory like the following
RepoDb.TypeMapper.EnumConverterFactory = new DefaultEnumConverterFactory();

public class DefaultEnumConverterFactory : IEnumConverterFactory
{
	public object CreateFactory(Type enumType) => new StringEnumConverter<TEnum>();
}
public class StringEnumConverter<TEnum> : IEnumConverter<TEnum, string>
{
	 public TEnum Get (string input, ClassProperty property) => ...;
	 public string Set (TEnum input, ClassProperty property) => ...;
}

Then you can customize EnumConverterFactory and IntEnumConverter like follow
RepoDb.TypeMapper.EnumConverterFactory = new CustomEnumConverterFactory();

public class CustomEnumConverterFactory : IEnumConverterFactory
{
	public object CreateFactory(Type enumType) => new IntEnumConverter<TEnum>();
}

do you think it is similar to IPropertyHandler? :smiley:

If switch to thinking with IPropertyHandler, it will roughly look like this
RepoDb.TypeMapper.PropertyHandlerFactory = new DefaultPropertyHandlerFactory();

public class DefaultPropertyHandlerFactory : IPropertyHandlerFactory
{
	public object CreateHandler(Type propType)
	{
		if (propType.IsEnum)
		{
			return new EnumStringPropertyHandler<TEnum>();
		}

		return null;
	}
}
public class EnumStringPropertyHandler<TEnum> : IPropertyHandler<string, TEnum?>
{
	 public TEnum Get (string input, ClassProperty property) => ...;
	 public string Set (TEnum input, ClassProperty property) => ...;
}

Then you can customize PropertyHandlerFactory and EnumIntPropertyHandler like follow
RepoDb.TypeMapper.PropertyHandlerFactory = new CustomPropertyHandlerFactory();

public class CustomPropertyHandlerFactory : DefaultPropertyHandlerFactory, IPropertyHandlerFactory
{
	object IPropertyHandlerFactory.CreateHandler(Type enumType)
	{
		if (..........)  // custome condition
		{
			return new EnumIntPropertyHandler<TEnum>();
		}

		return base.CreateHandler(enumType);
	}
}
public class EnumIntPropertyHandler<TEnum> : IPropertyHandler<int, TEnum?>
{
	 public TEnum Get (int input, ClassProperty property) => ...;
	 public int? Set (TEnum input, ClassProperty property) => ...;
}

Do you think it is similar to #646 ?
:grin:

Gents, as this may requires a change to the compiler for the non-schema-based operations, it is important to take note that RepoDB is intelligent enough if it understands your schema, and to some operations (i.e.: Insert<T>, Merge<T>, etc), RepoDB understands your schema.
It is very unfortunate that RepoDB could not understand the target schema if you are to use the following operations (ExecuteNonQuery, ExecuteScalar and ExecuteReader), as the table and/or model is not defined somewhere.
In my case, as I tested and validated that, you do not need to do any action item at all on this area, up until you required it on the call on the non-schema-based operations mentioned above.
To elaborate further, the details below is a replication process for this scenario. There, I have a Person table that has 2 columns pertained for enum values CustomerType and CustomerStatus. I also have created a dedicated enumeration for each and simply call the schema-based and non-schema-based operations.
Table:
CREATE TABLE [dbo].[Person](
	[Id] [bigint] IDENTITY(1,1) NOT NULL,
	[Name] [nvarchar](32) NOT NULL,
	[Address] [nvarchar](max) NOT NULL,
	[CustomerType] [int] NULL,
	[CustomerStatus] [nvarchar](32) NULL,
	[CreatedDateUtc] [datetime2](7) NOT NULL,
 CONSTRAINT [PK_Person] PRIMARY KEY CLUSTERED
(
	[Id] ASC
)WITH (PAD_INDEX = OFF, STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS = ON, ALLOW_PAGE_LOCKS = ON, OPTIMIZE_FOR_SEQUENTIAL_KEY = OFF) ON [PRIMARY]
) ON [PRIMARY] TEXTIMAGE_ON [PRIMARY]
GO

Enumerations:
namespace EnumPropertyHandlerTest.Enumerations
{
    public enum CustomerStatus
    {
        Active,
        Inactive
    }
}

namespace EnumPropertyHandlerTest.Enumerations
{
    public enum CustomerType
    {
        Direct = 1,
        Indirect = 2,
        Partner = 3
    }
}

Models:
using EnumPropertyHandlerTest.Enumerations;
using System;

namespace EnumPropertyHandlerTest.Models
{
    public class Person
    {
        public long Id { get; set; }
        public string Name { get; set; }
        public string Address { get; set; }
        public CustomerType CustomerType { get; set; }
        public CustomerStatus CustomerStatus { get; set; }
        public DateTime CreatedDateUtc { get; set; }
    }
}

Main:
using EnumPropertyHandlerTest.Enumerations;
using EnumPropertyHandlerTest.Models;
using Microsoft.Data.SqlClient;
using RepoDb;
using System;

namespace EnumPropertyHandlerTest
{
    class Program
    {
        static void Main(string[] args)
        {
            Initialize();
            TestSchemaBased();
            TestNonSchemaBased();
        }

        static void Initialize()
        {
            SqlServerBootstrap.Initialize();
        }

        static void TestSchemaBased()
        {
            using (var connection = new SqlConnection("Server=.;Database=DemoRepoDB;Integrated Security=True;"))
            {
                connection.Truncate<Person>();

                var id = connection.Insert<Person, long>(new Person
                {
                    Name = "John Doe",
                    Address = "New York",
                    CustomerType = CustomerType.Direct,
                    CustomerStatus = CustomerStatus.Active,
                    CreatedDateUtc = DateTime.UtcNow
                });

                var people = connection.QueryAll<Person>();
            }
        }

        static void TestNonSchemaBased()
        {
            using (var connection = new SqlConnection("Server=.;Database=DemoRepoDB;Integrated Security=True;"))
            {
                connection.Truncate<Person>();

                var entity = new
                {
                    Name = "John Doe",
                    Address = "New York",
                    CustomerType = CustomerType.Direct,
                    CustomerStatus = CustomerStatus.Active,
                    CreatedDateUtc = DateTime.UtcNow
                };
                var id = connection.ExecuteScalar<long>("INSERT INTO Person (Name, Address, CustomerType, CustomerStatus) " +
                    "VALUES (@Name, @Address, @CustomerType, @CustomerStatus); SELECT CONVERT(BIGINT, SCOPE_IDENTITY());", entity);

                var people = connection.QueryAll<Person>();
            }
        }
    }
}

You will notice that the issue only happens when we called the ExecuteScalar non-schema based operation.
Microsoft.Data.SqlClient.SqlException: 'Conversion failed when converting the nvarchar value 'Direct' to data type int.'

Therefore, please help me understand if your use-case falls on that operations.

And here, if we are to put the proper PropertyHandler to handle the case of the failing one, then it will be fixed and handled properly.
Note: It is a Type-Level mapping.
PropertyHandler:
using EnumPropertyHandlerTest.Enumerations;
using RepoDb;
using RepoDb.Interfaces;
using System;

namespace EnumPropertyHandlerTest.PropertyHandlers
{
    public class CustomerTypePropertyHandler : IPropertyHandler<int, CustomerType>
    {
        public CustomerType Get(int input, ClassProperty property)
        {
            var enumType = typeof(CustomerType);
            return (CustomerType)Enum.Parse(enumType, Enum.GetName(enumType, input));
        }

        public int Set(CustomerType input, ClassProperty property)
        {
            return Convert.ToInt32(input);
        }
    }
}

Initialize:
static void Initialize()
{
    SqlServerBootstrap.Initialize();

    FluentMapper
        .Type<CustomerType>()
        .DbType(System.Data.DbType.Int32, true);
}

Please let me know if this address your issues.

Here is the actual project I simulated (.NET Core 3.1) - EnumPropertyHandlerTest.zip

I know what you mean
In fact, most of my current situations do use model as the processing object, but it still cannot cover all the scenarios I encountered.
(I am currently trying to transfer a large system from a slightly modified dapper to RepoDb)
In addition. If these enums appear repeatedly in multiple models, these duplications will become frustrating.
I will try to see if I can do something, and if everything goes well, I will create an PR for your reference.
thank you very much.

In addition. If these enums appear repeatedly in multiple models, these duplications will become frustrating.

It will not for as long you use the type level mapping like below, specially you are all using the model based implementations.
FluentMapper
        .Type<CustomerType>()
        .DbType(System.Data.DbType.Int32, true);

I don't think the original suggestion of setting int/nvarchar to handle enums is sufficiently.
The propertyhandler is a good uniform way to describe the mapping of each type in your system whether it be some nested structure or an enum.
When dealing with enums, recall that you can have flagged enums, that you may want to handle separately - I've been in that situation some years back where flags were joined to form new settings. The propertyhandler can encompass this requirements as well.

@kbilsted - true, and thank you for your comments specifically on the bit-wised (flagged enums). However, we still need to have a default conversion of enum when it is being pushed into the DB (not pull/query), specially to those operations that is not entity-based (i.e: ExecuteScalar, ExecuteNonQuery and ExecuteReader), in which there is no way the library to understand which underlying table is being targeted.
Currently, RepoDB handles the enums very well if you call the entity-based operations (i.e: Insert<T>, Merge<T>, ExecuteQuery<T>, etc), but is converting it to TEXT/NVARCHAR data type in database if you are calling the non-entity based operations (mentioned above).
If we are to handle it defaultly to INT, it is as simple as commenting some TEXT/NVARCHAR conversion code from the compiler, but I am hesitant to do that due some scenarios other users already implemented.
To elaborate further, given with this class.
public enum CustomerType
{
	Direct,
	Indirect
};

public class Customer
{
	public long Id { get; set; }
	public string Name { get; set; }
	public CustomerType Type { get; set; }
}

The call below will succeed if the property CustomerType is TEXT/NVARCHAR in the DB.
var entity = new Customer { Name = "John Doe", Type = CustomerType.Direct };
connection.ExecuteScalar("INSERT INTO Customer (Name, CustomerType) VALUES (@Name, @Type)", entity);

But it will throw an exception if the CustomerType property is of type INT. Of course, unless otherwise you cast it like below or specifically add a mapping to that specific property to DbType.Int32k.
var entity = new Customer { Name = "John Doe", Type = (int)CustomerType.Direct };
connection.ExecuteScalar("INSERT INTO Customer (Name, CustomerType) VALUES (@Name, @Type)", entity);

Thoughts?

@SpaceOgre - I really would like to capture you as a contributor to this, since you found the issue and made the recommendations. Would you be able to make the PR for the solutions here? It is very simple as it is already pre-tested by me on our DEV ENV.
As of writing this, I had introduce this line of code already. See below.
public static DbType EnumDefaultDatabaseType { get; set; } = DbType.String;

By default, it is DbType.String so it would not affect any existing functionalities and behaviors.
The actual fixes are very simple, you just need to set the following lines to utilize the newly introduced property from the Converter object.

In the DbCommandExtension, simply change the following lines.

268
340
469
525
594
598

In the compiler class PlainTypeToDbParameters, simply change the following lines.

51

Looking forward for your PR, otherwise, please do let me know if you cannot do so I can push it myself or ask the other guys to contribute to it.
By the way, please do not challenge why it is placed as a static property and not an object-based approach like (object settings) as initially thought at #118. That is on top of our mind. For now, let us keep everything simple and easy to adopt.

this is not the better approach. How do you support all but a few enums are to be int and the rest string? E.g. if you are transitioning your dB..
My advice is to go for a code hook.

@kbilsted the TypeMapper and FluentMapper should still take priority, like it does today. This would just change what the default is.

@mikependon sure I can look into making a PR :)
Will hopefully have time tomorrow.

@kbilsted - this is just an enhacement to an existing default string. Other than that, everything works as it is. I hope your concerns and/or whatever use-cases can be addressed by the customized Property Handlers, specifically the code hook?

@SpaceOgre - what's up? as I am planning to make a new beta release, would be happy to include this one 😁

@mikependon Sorry, got swamped at work yesterday and did not manage to get to it. Will try and do it today :)

@mikependon I have done the code changes but I'm not sure if there are any existing test to update or if I should write new ones where should they be placed?

TBH, I already had pre-tested that on our environment. The default is DbType.String and your changes should not affect that. However, I can also add more Integration Tests that targets the DbType.Int32 or the non-String database type.

authors: SpaceOgre fredliex kbilsted mikependon

repo: mikependon/RepoDB


text:
Added missing CardView dependency

First thank you for this library :)
But i couldn't use it without also adding CardView as an dependency

Thank you very much. You are absolutely right ;)

authors: Foso mikepenz

repo: mikepenz/AboutLibraries


text:
Bouvet Island region/continent

Region and continent are currently defined as South America, but it appears to be closer to Cape Town, South Africa. I do not see a clear region definition for this location in ISO 3166. The CIA describes it as Antarctic Region and Wikipedia describes it as South Atlantic.
https://www.cia.gov/the-world-factbook/countries/bouvet-island/
https://en.wikipedia.org/wiki/Bouvet_Island

Updated based on the two links above

authors: mikeprivette mrudden

repo: mikeprivette/only-flags


text:
QUESTION: Work with Nouveau nvidia drivers

I just want to know does this will able to provide the stats of my nvidia gpu running iwth nouveau drivers the kde system monitor doesn't able to provide me that.

The driver writer folks have to add code for gpuvis to work with it. I don’t believe anyone from nouveau has done this so you’re not going to get any information from that driver in gpuvis. Sorry.

authors: mikesart opensusefan

repo: mikesart/gpuvis


text:
Points incorrectly identified as inside certain shapes

First of all, thank you for writing this helpful piece of software. While trying it out, I noticed it wasn't returning the correct result in certain scenarios. I added these tests to the test.js file and they seem to be incorrectly identifying these points as inside the polygon. I tried a few tweaks to the code, but couldn't get it working. Any guidance for fixing this would be very helpful. Thank you.
  // triangle
  t.equals(inside([[1,1],[2,1],[2,2]], [1,2]), 1) // incorrectly returns -1

  // parallelogram
  t.equals(inside([[1,1],[1,2],[2,3],[2,2]], [1,3]), 1) // incorrectly returns -1

Test output:
TAP version 13
# (anonymous)
not ok 1 should be equal
  ---
    operator: equal
    expected: 1
    actual:   -1
    at: Test.<anonymous> (robust-point-in-polygon/test/test.js:7:5)
  ...
not ok 2 should be equal
  ---
    operator: equal
    expected: 1
    actual:   -1
    at: Test.<anonymous> (robust-point-in-polygon/test/test.js:10:5)

Thanks for the heads up, this is a serious problem.  I will try to solve this tomorrow.

Ok, took me a little longer to get around to it but it should be working now.

authors: christophermanning mikolalysenko

repo: mikolalysenko/robust-point-in-polygon


text:
いいところ診断の結果に優しさを追加したい

以下の結果を追加したい
'{userName}のいいところは優しさです。あなたの優しい雰囲気や立ち振る舞いに多くの人が癒やされています。'

これから対応します。

5a6e062 で対応されました。


text:
enhance: Refine show replica output

This PR refines the output information from show replica command

Output printed grouping by collection
Add collection name information
Add ResourceGroup information
Sort node id in case of comparison

/lgtm

repo: milvus-io/birdwatcher


text:
Add benchmark test logs for IVFPQ

Signed-off-by: yudong.cai yudong.cai@zilliz.com

/kind improvement

repo: milvus-io/knowhere


text:
feat: add custom multi part upload fs

related: #148

We need a test or benchmark to verify if the part size is changed as expected.

Verified with Aliyun the behavior meets expectations. Under a 64MB memory limit, a part size of 32MB was allocated, and two files were written, sized 150KB and 85MB respectively.
150KB file: 1 PUT request.
85MB file: Used multipart upload. There were 2 multipart notifications (start and end) and 85MB / 32MB + 1 = 3 PUT requests for data writing, for a total of 5 requests.
In total, 6 PUT requests were made for the 2 files which is as expected.

/lgtm

authors: shaoting-huang tedxu

repo: milvus-io/milvus-storage


text:
Model Definition Front Strides

Hey team! I'm looking into the model definition here.
Should
 front_strides=[[(1,2), (2,1)], [(1,2), (2,1)],
                [(1,2), (2,1)], [(1,2), (2,1)],
                [(1,1), (2,1)], [(1,2), (2,1)],
                [(1,1), (2,1)], [(1,2), (2,1)]],

be
 front_strides=[[(1,2), (2,1)], [(1,2), (2,1)],
                [(1,2), (2,1)], [(1,2), (2,1)],
                [(1,2), (2,1)], [(1,2), (2,1)],
                [(1,2), (2,1)], [(1,2), (2,1)]],

Or is the former intended and, if so, why?

@daniel-deychakiwsky I can't quite remember now, but there was likely no significant reason.
We reduced dimensions by using strides instead of pooling. Input shape was not square (T x F spectrogram, where T is not F), and continuously using [(1,2), (2,1)] would waste parameters.

authors: daniel-deychakiwsky mimbres

repo: mimbres/neural-audio-fp


text:
not getting Spoticraft to run in minecraft java 1.20

i have installed spoticraft for minecraft java 1.20 and i have done what spoticraft said to do but nothing has happened when i have done that the only thing i can do with the mide now that i have done all that is to open the spoticraft menu which there are no songs in Spoticraft so how do I fix it

Make sure you downloaded the correct 1.20 version: https://modrinth.com/mod/spoticraft


text:
🛑 MineVN Maven & Resource Pack is down

In 24c395e, MineVN Maven & Resource Pack (http://app.minevn.net/pack) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MineVN Maven & Resource Pack is back up in 02fdd33.


text:
Add bootstrap toggle checkbox through javascript

When i use  on jsp page.It shows button the way  it should be shown.But when i add it through javascript he it shows only a checkbox not the toggle button.What is the issue?

I seem to have the same problem. As a response to user input I create dynamic elements as html strings which I add with getelementbyId. e.g.:
strBeoord += '\n';
When i put  directly on my html page it works fine!

Sounds like a DOM-ready condition.
After you dynamically append to the DOM, you must 'attach' the listener to it.
Usually this is done by putting the listener at a higher level in source code, such as document, that exists at DOM-ready time.
ie,
$(document).on('load','.mycheckboxclass', {} ,function(e){
       $(this).bootstrapToggle();
});

Sounds like you need to use a delegated event for the init.  Anything added to the DOM after DOMready will need to be attached in this manner.
https://learn.jquery.com/events/event-delegation/

After adding the checkbox, use the following (you'll need to adjust the selector to select just the new checkbox you added though):
$('input:checkbox').bootstrapToggle();

authors: BasDeBas KrishnaMurariVijay briansol codeconqueror

repo: minhur/bootstrap-toggle


text:
支付成功异步通知里的验签一直失败

查看参数, headers跟文档的一样, 也按照文档配置了平台证书,但是在验证签名这步一直失败.

这是request body 和 header:

框架是Flask.

仔细阅读一下readme里关于回调的部分，最直接的办法就是使用examples.py用native支付直击做一下验证，如果examples.py不能执行成功，检查自己的参数。如果examples.py的回调处理成功，检查一下自己项目代码。

没有新的更新就关闭了。


text:
iOS version

Is the iOS version planned? Technically, this should be possible, since the project is written in React Native.

Yes sometimes next year. Will need to look after some help to get it done.

Great!
I tried to build from the source by myself and I faild. The whole Xcode thing is a total mess :-/

Agree. Some of the libs I use require tweaks to xcode project, needs to be done one by one. No showstoppers, I chose the deps so that they do not have blocking issues on iOS.
Matter of strong will and enough time to kick it until it works.
Had a smart young student helping me spring and summer but he got admitted to fairly serious math uni from this sept and is out of any free time as of now...
Will likely need to look for another one, not sure anybody will just volunteer for a task like this. Not a fan of iOS from dev POV.

Have you thought about introducing Expo to the project or you want to keep it lean?
It is an extra dependency, but simplifies cross-platform development a lot.

Did one previous project using Expo and decided to drop it from this one. Staying lean was one motivation, step out of the pace of mandatory upgrades was the second and did not want to invest time to learn their new build tooling. As well, somehow could not get rid of bundled facebook trackers before, but they may be fixed that one, as they decomposed many libs to separate packages over time.

authors: minibits-cash prusnak


text:
[FE] other labels too far above field - advocate category, "case cracked in", rep order date

advocate category

case cracked in

repo order date

There's so many crazy things happening here to cause this...

authors: Demwunz jsugarman


text:
Readable container urns

The urns for containers which we currently use are generated as some sort of hash and are non-readable eg urn:li:container:5457467658a0d2b51d8c872d28a4683a is nomis_sensitive. What do we think about making these actual names to improve url readability? The obvious counter to this is namespace clashes but if that happens we wouldn't want 2 identically named databases anyway.

https://github.com/ministryofjustice/data-catalogue/pull/320

repo: ministryofjustice/data-catalogue


text:
Travis integration

Initial travis integration

looks cool

authors: colinbruce stephenrichards

repo: ministryofjustice/fr-staffapp


text:
(593) Combine multiple prerequisites

Context
In some cases, a single course has multiple prerequisites with the same name. The current code will only display the last of these in such cases
Changes in this PR

Concatenate prerequisites with the same name
Ensure we are working on the assumption (default) that there can be multiple prerequisites with the same name on a single cause

Screenshots of UI changes
Before

After

Release checklist
Release process documentation
As part of our continuous deployment strategy we must ensure that this work is
ready to be released once merged.
Pre-merge

[ ] There are changes required to the Accredited Programmes API for this change to work...

[ ] ... and they have been released to production already

Post-merge

[x] This adds, extends or meaningfully modifies a feature...

[ ] ... and I have written or updated an end-to-end test for the happy path in the Accredited Programmes E2E repo

[x] This makes new expectations of the API...

[x] ... and I have notified the API developer(s) of changes to the contract tests (Pact), or the API is already compliant

[ ] Manually approve release to preprod
[ ] Manually approve release to prod

 Gather and display multiple prerequisites where appropriate


text:
EMP-374: Added backend service to generate CRM4 Periodical Report

EMP-374: Added backend service to generate CRM4 Periodical Report
What
Link to story
Describe what you did and why.
Checklist
Before you ask people to review this PR:

[x] Tests should be passing: ./gradlew test
[x] Github should not be reporting conflicts; you should have recently run git rebase main.
[x] Avoid mixing whitespace changes with code changes in the same commit. These make diffs harder to read and conflicts more likely.
[x] You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

LGTM

authors: solomonmanik tanovellino


text:
Pud 86 UI 4pactonly

Pud 86 UI PACT blockage work-around

Changes approved on original PR as per: https://github.com/ministryofjustice/manage-recalls-ui/pull/82#pullrequestreview-723362327

repo: ministryofjustice/manage-recalls-ui


text:
Prevent breadcrumbs being displayed on 404 page

@marcincichon Can you review?

@sparkdevelopment go for it

authors: marcincichon sparkdevelopment

repo: ministryofjustice/mojintranet-theme


text:
DDPB-3176 - Run API unit tests in CircleCI

Purpose
This change removes the ECS task for running API unit tests and runs it directly in CircleCI. This is in order to support test measures as part of the pipeline and in a future ticket, we will have the code coverage delta acting as a another gate for merging code.
Fixes DDPB-3176.
Approach
This was a painfully long process of repeated attempts to debug issues arising in Circle CI. Initially, it seemed like an out of memory issue as the Docker container was exiting with a 137 code which typically indicates OOM errors. However, after reducing all memory usage to a minimum and switching to a tiny image to run the tests the issues were still present and on inspecting the logs it turned out it was a permissions issue with the Symfony var/cache directory.
I've tried a number of different owners on the folder ranging from root, 1001, 0 and taking the UID of the user running the docker command but nothing seemed to be appropriate so in the end I just gave the folder 777 access. I was initially hesitant to do this due to security concerns but as it's not in the base API Docker image, and this is just to run tests, I'm fairly relaxed now. If anyone has any concerns then let me know in PR comments 👍
Learning
So many blogs and deep dives on Docker and Symfony permissions issues:
https://stackoverflow.com/questions/34949083/symfony-docker-permission-problems-for-cache-files?noredirect=1&lq=1
https://medium.com/@einenlum/testing-your-app-with-docker-compose-on-circleci-ffd4cc8e2cfa
https://github.com/symfony/cli/issues/183
https://sagikazarmark.hu/blog/containerizing-a-symfony-application/
Checklist

[x] I have performed a self-review of my own code
[x] I have updated documentation (Confluence/ADR/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have approved these changes

Great work!

authors: acsauk-mojd aiesoftware

repo: ministryofjustice/opg-digideps


text:
'dz' counts a lot

Hi,
I'm trying to use the method in a domain which has 19 levels in depth (totally 1000m), and I found it makes a poor inversion result. However,  it comes a much better result after I set my vertical levels as the data used in oceanic example (200 levels for 1000m), which convinced me that 'dz' counts a lot in calculation. But with the improvement comes a fat data array ( [360,200,1700,650] ) that is so hard to run forward. So I’d like to know, from your perspective, what value of 'dz' (or levels in depth) can allows me to reduce the size of data while maintaining a good accuracy?
Thanks a lot!

Hi, are you using evenly distributed vertical grid? Currently xinvert only does his job on evenly distributed grid.
For large array, you need chunks to help you.  Which problem are you looking at?  Usually it is chunked along time.  But also depends on your specific case.

That make sence ! I've tried an evenly distributed vertical grid but with 20 levels (totally 1000m), and it comes a nice reslut. Thank you for the patience. BTW, is 'dz' important in the calculation ? Beacuse it seems that to keep an evenly distributed grid is more important, and both evenly distributed case (200 or 20 levels in totally 1000m) show nice accuracy.

BTW, is 'dz' important in the calculation ?

That's right, dx, dy, dz are all important in the calculations of derivatives along each direction.  If they are not evenly distributed, the results are not correct.  Right now it is not easy to add support to unevenly distributed grids.  But it is possible in a future release.

If xinvert requires equally spaced vertical grid then perhaps an assert statement is in order and perhaps an error w some informative warning to the user if the assertion fails?

Hi @navidcy, that's necessary.  I am thinking about it, and will fix this soon.

oh I'm not sure...
yeah, if ... then raise Exception('...') would also do the job!

This is done at here.  So if a coordinate is non-uniform, it will raise an error now.

authors: FuncUny miniufo navidcy

repo: miniufo/xinvert


text:
🛑 Miko is down

In e2f15a5, Miko (miko.mint.lgbt) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Miko is back up in 43757cf after 13 hours, 37 minutes.

repo: mint-lgbt/status


text:
🛑 Matrix is down

In 16ebdf1, Matrix (https://matrix.mint.lgbt:8448/_matrix/federation/v1/version) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Matrix is back up in b573311 after 21 minutes.


text:
🛑 Matrix is down

In e4ef9fc, Matrix (https://matrix.mint.lgbt:8448/_matrix/federation/v1/version) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Matrix is back up in 08f1ccb after 53 minutes.


text:
🛑 Ralsei is down

In 9331b5c, Ralsei (ralsei.nightcore.monster) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Ralsei is back up in ad3a95e.


text:
Fixed Exception when start the game

Fixed the exception when start the game #85

Fixes #85

Thanks for this! Great help. I get home to see a mystery game patch and the mod broken :P

authors: MiCLeal mipen

repo: mipen/BannerlordTweaks


text:
Type error with decode_bin

When building irmin on NixOS I am getting some type errors:
File "src/irmin/commit.ml", line 493, characters 19-37:
493 |         let n, v = decode_bin buf off in
                         ^^^^^^^^^^^^^^^^^^
Error: This expression has type string but an expression was expected of type
         'a * 'b

I have tried the 3 latest versions of irmin.
I have also tried with ocaml 4.8, 4.9, 4.10, 4.11, 4.12 and 4.13.
Any clues?

It seems your version of repr is incompatible, it should be <0.6.0 to work with these versions of irmin.

It seems your version of repr is incompatible, it should be <0.6.0 to work with these versions of irmin.

I am updating some packages in NixOS... I will check this.

repo: mirage/irmin

license: isc


text:
wip: Dangling parent of commits on which gc is called

based on https://github.com/mirage/irmin/pull/1905

Codecov Report

Merging #1913 (fe556d1) into blocking-dummy-gc (e66bf6e) will increase coverage by 0.99%.
The diff coverage is 97.29%.

@@                  Coverage Diff                  @@
##           blocking-dummy-gc    #1913      +/-   ##
=====================================================
+ Coverage              63.94%   64.93%   +0.99%
=====================================================
  Files                    126      126
  Lines                  14876    14900      +24
=====================================================
+ Hits                    9513     9676     +163
+ Misses                  5363     5224     -139

Impacted Files
Coverage Δ

src/irmin-pack/indexing_strategy.ml
45.45% <ø> (ø)

src/irmin-pack/inode.ml
79.02% <ø> (+0.41%)
:arrow_up:

src/irmin-pack/unix/checks.ml
10.08% <ø> (ø)

src/irmin-pack/unix/snapshot.ml
76.19% <ø> (ø)

src/irmin-pack/unix/traverse_pack_file.ml
56.93% <ø> (ø)

src/irmin-pack/unix/gc.ml
85.47% <96.42%> (+82.40%)
:arrow_up:

src/irmin-pack/pack_value.ml
84.26% <100.00%> (+4.26%)
:arrow_up:

src/irmin-pack/unix/pack_store.ml
79.89% <100.00%> (+2.41%)
:arrow_up:

src/irmin-fs/irmin_fs.ml
81.21% <0.00%> (-1.22%)
:arrow_down:

src/irmin-unix/fs.ml
67.74% <0.00%> (-0.65%)
:arrow_down:

... and 18 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

this is ready for review

Looks good to me.

authors: codecov-commenter icristescu tomjridge


text:
Problem:Popraviti access Gui iz drugih threadova

Baca exception za sada?

Nije nam ovo problem, gledao sam, samo kad se debugguje izadje kao problem, al ako hoces da debugujes mozemo da ih zakomentarisemo ili cu napraviti neki bool koji ugasi sve ako selektujes da debugujes...
Ili npr ako ukucamo ime debugg i passw debugg onda ne ugasimo gui...
Radi inace sa tim sve okej, ne moramo delegate i to


text:
BOOLEAN type not working with prepared statement.

A prepared statement of the following kind
select * from test_boolean_type where c_boolean=?

where ? is bound to string True. It returns incorrect result. It however works if integers 0or1` is supplied.
Tested with version 0.8.2

Some mistake on my side. closing it.

Re-opening since the post updating the description.

Hi there, apologies for the late reply.
A string value cannot be a bool type, this should be a user error.
Use boolean true/false instead of a string value.

authors: mirromutth sumitsum


text:
Error Deploying to Heroku (webpack-bundle-tracker)

When trying to deploy to heroku the webpack-bundler-tracker wasn't installed
I have to run
heroku config:set NPM_CONFIG_PRODUCTION=false
in order to make it works...
Hope it helps.

how could u install nom and node.js on heroku?
I can't do this.
so I meet the problem
Error reading ./webpack-bundle.json. Are you sure webpack has generated the file and the path is correct?

Same problem here.

Same here, subscribing to this issue.

Has anyone manage to solve webpack-bundle.json error?

I think this issue is resolved by now. "Deploy to heroku" button works, however, if you are deploying manually (eg. from CLI) remember to set proper Buildpacks.
Saleor needs heroku-buildpack-nodejs and heroku-buildpack-python to work properly.

I used
npm install webpack-bundle-tracker --save-dev
to create the package and then push it to heroku =)

authors: ErrolLin Mitaanshu-Agarwal Pacu2 alcfernandes chmbrs jordantomax mativs

repo: mirumee/saleor


text:
Product details doesn't show price/variant/add to cart

What I'm trying to achieve
Product details doesn't show price/variant/add to cart
Steps to reproduce the problem
I've followed the docs install instructions
Screenshots

System information
Operating system: ubuntu 18.04
Browser: both firefox & chromium

npm run build-assets --development
made it work

I am getting the same error and doing the npm run build-assets --development is not working .

I have the same problem, have you fixed it yet?

No it's not yet solved
On Sun 18 Nov, 2018, 11:59 AM Võ Đình Nghĩa <notifications@github.com wrote:

I have the same problem, have you fixed it yet?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mirumee/saleor/issues/3274#issuecomment-439671172,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD5z_kihKDWdpP7q9UT3IghaJDsXjgQoks5uwP5fgaJpZM4YYwCl
.

have you tried to run
npm start
Either this or the other command solved the issue for me

Doing the above steps also not fixed the issue .Please see the error in console [.](url

)

also ,Doing repeatedly .npm -i
Getting these warning as well.
npm i
npm WARN babel-loader@7.1.4 requires a peer of babel-core@6 but none is installed. You must install peer dependencies yourself.
npm WARN awesome-typescript-loader@5.2.0 requires a peer of typescript@^2.7 but none is installed. You must install peer dependencies yourself.
npm WARN bootstrap@4.1.2 requires a peer of popper.js@^1.14.3 but none is installed. You must install peer dependencies yourself.

@maarcingebala any  help or suggestion ?

@patrys Its chrome .I just clone and followed the installation instruction .Only the console i am getting error .Is the babel-loader installation is having some issues ?

I've the same problem. Only way to fix it so that the product detail page can load the variant options is to add a variant with Options ="---" to the variant.  Then it will display all the variant options in the product detail page. I think it is a bug.

Does this issue still occur?

Yes it does .  I think it is a bug.
On Dec 11, 2018, at 11:57 AM, Marcin Gębala <notifications@github.commailto:notifications@github.com> wrote:
Does this issue still occur?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/mirumee/saleor/issues/3274#issuecomment-446277715, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArZYDHcbQ4eGjTox9ylWeNoBwgKw6LNQks5u3-P7gaJpZM4YYwCl.

Can you check again with latest master? Also I'd recommend removing node_modules, reinstalling all frontend dependencies and rebuilding static assets. We've updated a bunch of dependencies recently. I'm not able to reproduce it on my machine.

Hi Marcin
thank you for your help.
I've a question regarding the images for Variant.
Is it possible for the product page to show the Variant Picture after it is selected instead of rotating images randomly?
How about more than one Variant selected per product?
ie)   A same style of shirt can have different size, different color and different pictures on the shirt.
Can this be done?
If no, do you mind to point me to which file I can modify to make this functions?
thanks
Sunny

From: Marcin Gębala notifications@github.com
Sent: December 11, 2018 11:59 AM
To: mirumee/saleor
Cc: sw80386; Comment
Subject: Re: [mirumee/saleor] Product details doesn't show price/variant/add to cart (#3274)
Can you check again with latest master? Also I'd recommend removing node_modules, reinstalling all frontend dependencies and rebuilding static assets. We've updated a bunch of dependencies recently. I'm not able to reproduce it on my machine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/mirumee/saleor/issues/3274#issuecomment-446278603, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArZYDLe3bXwVHyaAv4rJ7JXd-3TPf1UEks5u3-SOgaJpZM4YYwCl.

Hi Marcin
Also after I enabled Braintree using sandbox for testing.  I always get this error “unable to process transaction. Please try again in a moment all_” as shown in attached picture.
This is a clean install I am working on and the clean install fixed the variant selection in the product page.  However, I am still getting the same error in Braintree payment checkout.
Any fix for this?
[image1.jpeg]
Thanks again
Sunny
On Dec 11, 2018, at 12:00 PM, Marcin Gębala <notifications@github.commailto:notifications@github.com> wrote:
Can you check again with latest master? Also I'd recommend removing node_modules, reinstalling all frontend dependencies and rebuilding static assets. We've updated a bunch of dependencies recently. I'm not able to reproduce it on my machine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/mirumee/saleor/issues/3274#issuecomment-446278603, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArZYDLe3bXwVHyaAv4rJ7JXd-3TPf1UEks5u3-SOgaJpZM4YYwCl.

Looks like it is solved. Closing for now unless it occurs again.

authors: NyanKiyoshi dinhnghia diramazioni maarcingebala suraj1986 sw80386


text:
Add attributes validation while creating/updating a product's variant

Add attributes validation while creating or updating a product's variant. Previously it was possible to create or update a variant without providing any attributes.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[x] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[x] Changes are mentioned in the changelog

Task linked: [Variants] User shouldn't be able to create new variant without attribute


text:
🛑 Sirius is down

In 7d95420, Sirius (https://sirius.apprentissage.beta.gouv.fr/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Sirius is back up in 12b3245 after 9 minutes.

repo: mission-apprentissage/upptime


text:
Question about creating own objectives to the scoreboards.

Hi!
I have a question about creating own objectives. Is board somehow interfere with the objectives? I'm creating my own objective with criteria health and display type - below name. Unfortunately, when I'm joining to the server, the objective always shows 0, and I can't find a way to check what prevents showing the real value 😕

Hence I have a question about changing the objective value. I found that Board only replaces one, its own objective. But it don't interfere with another ones. Is it possible that board API affects to the other objective values? Below, I paste the code where I'm adding a new objective. I'm executing that method after the setup method from the Board API.
Cheers!
    public static void setScoreboardHealthObjective(Board board) {
        Scoreboard scoreboard = board.getScoreboard();
        if (scoreboard == null) {
            return;
        }

        if (scoreboard.getObjective("healthbar") == null) {
            Objective objective = scoreboard.registerNewObjective("healthbar", "health", ChatHelper.fixColor("&c❤"));
            objective.setDisplaySlot(DisplaySlot.BELOW_NAME);
        }
    }

Some months ago I've tested the plugin base again and it looks like the bug is unrelated to the board library. The bug is inside the MultiLineAPI one.

repo: missionarydev/board


text:
widget_styles don't support bitmap_font

my config.yaml
widget_styles:
  text_default:
    bitmap_font: True
    font_name: CG-pixel

slides:
    testSlide:
    - type: text
      text: TEST

slide_player:
  mode_attract_started: testSlide

result:
kivy: Exception while processing RegisteredHandler(callback=<bound method McConfigPlayer.play_from_trigger of McConfigPlayer.widgets>, priority=1, kwargs={}, key=UUID('1d1a423e-565d-4b9e-a980-b80fb095a1de'), condition=None, blocking_facility=None) for event widgets_play
Traceback (most recent call last):
  File "/home/sysop/mpfdev/mpf/mpf/core/events.py", line 662, in _run_handlers
    result = handler.callback(**merged_kwargs)
  File "/home/sysop/mpfdev/mpf-mc/mpfmc/core/mc_config_player.py", line 56, in play_from_trigger
    self.play(settings=settings, context=context, calling_context=calling_context, priority=priority, **kwargs)
  File "/home/sysop/mpfdev/mpf-mc/mpfmc/config_players/widget_player.py", line 147, in play
    self._action_add(s, instance_dict, widget, context, kwargs)
  File "/home/sysop/mpfdev/mpf-mc/mpfmc/config_players/widget_player.py", line 105, in _action_add
    slide.add_widgets_from_library(name=widget, play_kwargs=play_kwargs, **s)
  File "/home/sysop/mpfdev/mpf-mc/mpfmc/uix/slide.py", line 136, in add_widgets_from_library
    play_kwargs=play_kwargs)
  File "/home/sysop/mpfdev/mpf-mc/mpfmc/uix/widget.py", line 909, in create_widget_objects_from_config
    mc=mc, config=widget, key=this_key, play_kwargs=play_kwargs)
  File "/home/sysop/mpfdev/mpf-mc/mpfmc/widgets/text.py", line 96, in __init__
    self._process_text(self.original_text)
  File "/home/sysop/mpfdev/mpf-mc/mpfmc/widgets/text.py", line 204, in _process_text
    self.update_vars_in_text(text)
  File "/home/sysop/mpfdev/mpf-mc/mpfmc/widgets/text.py", line 243, in update_vars_in_text
    self.update_text(text)
  File "/home/sysop/mpfdev/mpf-mc/mpfmc/widgets/text.py", line 264, in update_text
    self._label.texture_update()
  File "/home/sysop/.pyenv/versions/3.6.6/lib/python3.6/site-packages/kivy/uix/label.py", line 413, in texture_update
    self._label.refresh()
  File "/home/sysop/.pyenv/versions/3.6.6/lib/python3.6/site-packages/kivy/core/text/__init__.py", line 759, in refresh
    self.resolve_font_name()
  File "/home/sysop/.pyenv/versions/3.6.6/lib/python3.6/site-packages/kivy/core/text/__init__.py", line 356, in resolve_font_name
    raise IOError('Label: File %r not found' % fontname)
OSError: Label: File 'CG-pixel.ttf' not found

Note that the slide below displays correctly
slides:
    testSlide:
    - type: text
      font_name: CG-pixel
      bitmap_font: True
      text: TEST

I have the exact same problem.
mfp-mc 0.56.1

authors: gbouthenot yetifrisstlama

repo: missionpinball/mpf-mc


text:
Crash in Shots

INFO : SwitchController : <<<<< switch: s_switch4, State:0 >>>>>
INFO : SwitchController : <<<<< switch: s_switch4, State:1 >>>>>
INFO : Mode.attract : Mode Starting. Priority: 10
INFO : Machine : Shutting down...
INFO : Machine : Shutting down...
ERROR : asyncio : Exception in callback EventManager._done(<Task finishe...scriptable",)>)
handle: <Handle EventManager._done(<Task finishe...scriptable",)>)>
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/asyncio/events.py", line 125, in _run
    self._callback(*self._args)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/mpf/core/events.py", line 530, in _done
    future.result()
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/asyncio/futures.py", line 274, in result
    raise self._exception
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/asyncio/tasks.py", line 239, in _step
    result = coro.send(None)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/mpf/core/events.py", line 462, in _run_handlers_sequential
    handler.callback(queue=queue, **merged_kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/mpf/core/mode.py", line 487, in _control_event_handler
    callback(mode=self)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/mpf/devices/shot.py", line 696, in reset
    self.jump(mode, state=0)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/mpf/devices/shot.py", line 655, in jump
    if state == self.player[self.get_profile_by_key('mode', mode)['settings']['player_variable']]:

From: https://groups.google.com/forum/#!topic/mpf-users/WgK4AOVWEvM

This is also a symptom of #838 since it also happens during fuzzing

fixed in dev

repo: missionpinball/mpf


text:
Is the model ckpt compatible with the ckpt of models in mmdet3d

Hi, I would like to know whether the pretrained models (e.g., pretrained backbone(voxelnet)) trained in OpenPCDet can be load to a model in mmdet3d? Thanks!

I did not have a chance to have a look at the OpenPCDet implementation of the voxelnet backbone, but I could imagine that there must be some conflicts in the model keys. You might need to handle that manually.

got that, thanks~

authors: FrontierBreaker kentang-mit

repo: mit-han-lab/bevfusion


text:
The evaluation of imagenet is not right

Dear author:
I downloaded the pretrained model EfficientNet-B1-224 and run the evaluation code, but the result seems not right, only 10% top1 acc on imagenet, i want to know the reason for this and how to fix it.
Thanks
RitchieAlpha

same problem

authors: RitchieAlpha Ysnower

repo: mit-han-lab/efficientvit


text:
Bump to 0.2.0 to test upgrade feature from admin UI

To test out Discourse admin UI for upgrading.

selfie


text:
Allow TES config to allow url for websocket connect to Twitch Mock CLI

I see that tests have been worked on to monkey patch the connection to use a locally running websocket server.
Can we have a simple addition to the new TES(config) process where config would allow us to provide a URL of a locally running Twitch CLI mock server for eventsubs?
By default, it comes up as ..
ws://127.0.0.1:8080/ws
but in many cases, we have to change the ports. Allowing us to pass that in directly would be ideal.
Thoughts?

released in v1.0.2 🎉

Something to note on local testing
You still must call tes.subscribe to initiate a connection with the Twitch CLI local WebSocket server. This will cause TES to attempt to make a request to the Twitch API's eventsub subscribe endpoint. Even though this will fail (because the WebSocket connection ID will not be valid), you will still be able to trigger mock events from CLI that TES will pick up.

authors: fairbairn mitchwadair

repo: mitchwadair/tesjs


text:
MITMWEB: search, highlight, intercept not working

Steps to reproduce the problem:

On OS X El Capitain, update to latest version pip install --upgrade mitmproxy
start mitmweb, I'm ignoring domains with certificate pinning mitmweb -T --wport 8888  -v --host --anticache -p 8080 --ignore '(69\.171\.\d+\.\d+:443)|(31\.13\.\d+\.\d+:443)|(17\.\d+\.\d+\.\d+:443)|(216\.58\.\d+\.\d+:443)|(104\.69\.\d+\.\d+:443)|(108\.174\.\d+\.\d+:443)'
open browser (I tried both Safari and Chrome) and attempt to search, highlight or intercept streams using the selection boxes at the top of the browser window

What is the expected behavior?
search - shows list of streams matching expression
highlight - highlights matching streams
intercept - intercepts matching streams
What went wrong?
No change in browser window
Any other comments?
This was definitely working in 0.13

Mitmproxy Version: 0.16
Operating System: OSX 10.11.4 (15E65)

Thanks for the report, I believe this is fixed on master already. If you need it ASAP, take a look at https://snapshots.mitmproxy.org. :smiley:

Fixed on 0.17 and tested ok on 0.18

authors: mhils peterglock

repo: mitmproxy/mitmproxy


text:
Temporary failure in name resolution

Proxy giving this error
502 Bad Gateway
ProtocolException('Server connection to ('www.oraculous.in', 80) failed: Error connecting to "www.oraculous.in": [Errno -3] Temporary failure in name resolution',)

mitmproxy version is 4.0
operating mode or command was mitmdump -p 8080
i installed python 3.6 and then mitmproxy on ubuntu 18.04

The error message states that mitmproxy cannot resolve the IP address of www.oraculous.in. That's not a mitmproxy issue, that's a problem with your internet connectivity (or with www.oraculous.in being unavailable in general).

See also: #3203

authors: mhils rajkotraja


text:
window，update version can not see details .

hi:
I sync native version.   phone can use mitm network. but can't see details(request/response/and so on).
environment ： window 7 +cygwin
and log:
Traceback (most recent call last):
File "/home/sanbo/package/mitmproxy/libmproxy/console/init.py", line 500, in run
self.loop.run()
File "/usr/lib/python2.7/site-packages/urwid/main_loop.py", line 278, in run
self._run()
File "/usr/lib/python2.7/site-packages/urwid/main_loop.py", line 375, in _run
self.event_loop.run()
File "/usr/lib/python2.7/site-packages/urwid/main_loop.py", line 678, in run
self._loop()
File "/usr/lib/python2.7/site-packages/urwid/main_loop.py", line 715, in _loop
self._watch_filesfd
File "/usr/lib/python2.7/site-packages/urwid/raw_display.py", line 392, in
event_loop, callback, self.get_available_raw_input())
File "/usr/lib/python2.7/site-packages/urwid/raw_display.py", line 492, in parse_input
callback(processed, processed_codes)
File "/usr/lib/python2.7/site-packages/urwid/main_loop.py", line 399, in _update
self.process_input(keys)
File "/usr/lib/python2.7/site-packages/urwid/main_loop.py", line 499, in process_input
k = self._topmost_widget.keypress(self.screen_size, k)
File "/home/sanbo/package/mitmproxy/libmproxy/console/window.py", line 38, in keypress
k = super(self.class, self).keypress(size, k)
File "/usr/lib/python2.7/site-packages/urwid/container.py", line 1128, in keypress
return self.body.keypress( (maxcol, remaining), key )
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowlist.py", line 380, in keypress
return urwid.ListBox.keypress(self, size, key)
File "/usr/lib/python2.7/site-packages/urwid/listbox.py", line 987, in keypress
key = focus_widget.keypress((maxcol,),key)
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowlist.py", line 248, in keypress
self.master.view_flow(self.flow)
File "/home/sanbo/package/mitmproxy/libmproxy/console/init.py", line 591, in view_flow
flowview.FlowView(self, self.state, flow, tab_offset),
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowview.py", line 134, in init
tab_offset
File "/home/sanbo/package/mitmproxy/libmproxy/console/tabs.py", line 28, in init
self.show()
File "/home/sanbo/package/mitmproxy/libmproxy/console/tabs.py", line 66, in show
body = self.tabs[self.tab_offset]1,
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowview.py", line 156, in view_request
return self.conn_text(self.flow.request)
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowview.py", line 207, in conn_text
msg, body = self.content_view(viewmode, conn)
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowview.py", line 188, in content_view
isinstance(conn, HTTPRequest)
File "/home/sanbo/package/mitmproxy/libmproxy/utils.py", line 111, in get
if args in self.cache:
TypeError: unhashable instance
mitmproxy has crashed!
Please lodge a bug report at:
https://github.com/mitmproxy/mitmproxy
Shutting down...

diff --git a/libmproxy/utils.py b/libmproxy/utils.py
index a29a53f..a6ca55f 100644
--- a/libmproxy/utils.py
+++ b/libmproxy/utils.py
@@ -1,11 +1,8 @@
from future import absolute_import
import os
import datetime
-import urllib
import re
import time
-import functools
-import cgi
import json
@@ -60,49 +57,6 @@ def pretty_json(s):
return json.dumps(p, sort_keys=True, indent=4).split("\n")
-def urldecode(s):

"""

   Takes a urlencoded string and returns a list of (key, value) tuples.

"""
return cgi.parse_qsl(s, keep_blank_values=True)
-def urlencode(s):
"""

   Takes a list of (key, value) tuples and returns a urlencoded string.

"""
s = [tuple(i) for i in s]
return urllib.urlencode(s, False)
-def multipartdecode(hdrs, content):
"""

   Takes a multipart boundary encoded string and returns list of (key, value) tuples.

"""
v = hdrs.get_first("content-type")
if v:

   v = parse_content_type(v)

   if not v:

       return []

   boundary = v[2].get("boundary")

   if not boundary:

       return []

   rx = re.compile(r'\bname="([^"]+)"')

   r = []

   for i in content.split("--" + boundary):

       parts = i.splitlines()

       if len(parts) > 1 and parts[0][0:2] != "--":

           match = rx.search(parts[1])

           if match:

               key = match.group(1)

               value = "".join(parts[3 + parts[2:].index(""):])

               r.append((key, value))

   return r

return []
def pretty_duration(secs):
formatters = [
(100, "{:.0f}s"),
@@ -168,50 +122,6 @@ class LRUCache:
return ret

-def parse_content_type(c):

"""

   A simple parser for content-type values. Returns a (type, subtype,

   parameters) tuple, where type and subtype are strings, and parameters

   is a dict. If the string could not be parsed, return None.

   E.g. the following string:

       text/html; charset=UTF-8

   Returns:

       ("text", "html", {"charset": "UTF-8"})

"""
parts = c.split(";", 1)
ts = parts[0].split("/", 1)
if len(ts) != 2:

   return None

d = {}
if len(parts) == 2:

   for i in parts[1].split(";"):

       clause = i.split("=", 1)

       if len(clause) == 2:

           d[clause[0].strip()] = clause[1].strip()

return ts[0].lower(), ts[1].lower(), d
-def hostport(scheme, host, port):
"""

   Returns the host component, with a port specifcation if needed.

"""
if (port, scheme) in [(80, "http"), (443, "https")]:

   return host

else:

   return "%s:%s" % (host, port)

-def unparse_url(scheme, host, port, path=""):

"""

   Returns a URL string, constructed from the specified compnents.

"""
return "%s://%s%s" % (scheme, hostport(scheme, host, port), path)
def clean_hanging_newline(t):
"""
Many editors will silently add a newline to the final line of a

Looks like you installed mitmproxy from master? We fixed #753 a day ago,
please update.
Cheers,
Max
sanbo notifications@github.com schrieb am Mi., 9. Sep. 2015 08:34:

hi:
I sync native version. phone can use mitm network. but can't see
details(request/response/and so on).
environment ： window 7 +cygwin
and log:
Traceback (most recent call last):
File "/home/sanbo/package/mitmproxy/libmproxy/console/init.py", line
500, in run
self.loop.run()
File "/usr/lib/python2.7/site-packages/urwid/main_loop.py", line 278, in
run
self.
run() File "/usr/lib/python2.7/site-packages/urwid/main_loop.py", line
375, in run self.event_loop.run() File
"/usr/lib/python2.7/site-packages/urwid/main_loop.py", line 678, in run
self._loop() File "/usr/lib/python2.7/site-packages/urwid/main_loop.py",
line 715, in _loop self._watch_filesfd File
"/usr/lib/python2.7/site-packages/urwid/raw_display.py", line 392, in
event_loop, callback, self.get_available_raw_input()) File
"/usr/lib/python2.7/site-packages/urwid/raw_display.py", line 492, in
parse_input callback(processed, processed_codes) File
"/usr/lib/python2.7/site-packages/urwid/main_loop.py", line 399, in _update
self.process_input(keys) File
"/usr/lib/python2.7/site-packages/urwid/main_loop.py", line 499, in
process_input k = self._topmost_widget.keypress(self.screen_size, k) File
"/home/sanbo/package/mitmproxy/libmproxy/console/window.py", line 38, in
keypress k = super(self.__class, self).keypress(size, k)
File "/usr/lib/python2.7/site-packages/urwid/container.py", line 1128, in
keypress
return self.body.keypress( (maxcol, remaining), key )
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowlist.py", line
380, in keypress
return urwid.ListBox.keypress(self, size, key)
File "/usr/lib/python2.7/site-packages/urwid/listbox.py", line 987, in
keypress
key = focus_widget.keypress((maxcol,),key)
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowlist.py", line
248, in keypress
self.master.view_flow(self.flow)
File "/home/sanbo/package/mitmproxy/libmproxy/console/init.py", line
591, in view_flow
flowview.FlowView(self, self.state, flow, tab_offset),
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowview.py", line
134, in init
tab_offset
File "/home/sanbo/package/mitmproxy/libmproxy/console/tabs.py", line 28,
in init
self.show()
File "/home/sanbo/package/mitmproxy/libmproxy/console/tabs.py", line 66,
in show
body = self.tabs[self.tab_offset]1,
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowview.py", line
156, in view_request
return self.conn_text(self.flow.request)
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowview.py", line
207, in conn_text
msg, body = self.content_view(viewmode, conn)
File "/home/sanbo/package/mitmproxy/libmproxy/console/flowview.py", line
188, in content_view
isinstance(conn, HTTPRequest)
File "/home/sanbo/package/mitmproxy/libmproxy/utils.py", line 111, in get
if args in self.cache:
TypeError: unhashable instance
mitmproxy has crashed!
Please lodge a bug report at:
https://github.com/mitmproxy/mitmproxy
Shutting down...
—
Reply to this email directly or view it on GitHub
https://github.com/mitmproxy/mitmproxy/issues/756.

ok, I'll try. thx

authors: hhhaiai mhils


text:
Cc/legacy shortcode fix 2

This builds on #1315 so waiting for that to be reviewed first.
Pre-Flight checklist

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
#1317
What's this PR do?
This improves the LegacyShortcodes plugin for CKEditor. "Legacy" Shortcodes are shortcodes that cannot be edited or created in Studio, i.e., only exist due to the migration of legacy content. Alice did some work so that we could safely edit pages containing such legacy shortcodes. This PR uses the some of the work from #1347  to improve upon this in two ways:

Some legacy shortcodes included shortocdes inside shortcodes, e.g., ... The old approach did not handle this, but the approach from #1315 does
Makes closing shortcodes a bit more flexible. The old code expected {{< /shortcode-name.... But some shortcodes, particularly image-gallery-item, used {{</ shortcode-name, ie with no space between "<" and "/". Hugo documentation uses the first form, but Hugo itself tolerates both forms as closing shrotcode tags.

Both changes can be seen the screenshots below:
Before / After:
 /

How should this be manually tested?
Manually insert some legacy shortcodes into a content page and check that the page (but not the shortcodes themselves) can still be edited safely. The shortcode parameters should not change when editing. Note for testing: the actual argument values don't need to be valid for those shortcodes since studio doesn't use the values.
For example:

Cyclosilicates
--------------
{{< image-gallery id="8d434783-38e4-6c38-89ad-01e859532b7e_nanogallery2" baseUrl="/courses/12-108-structure-of-earth-materials-fall-2004/" >}}
{{< image-gallery-item href="2237ca98fc40c772cac4af3df1e8635f_lab2-17.jpg" data-ngdesc="Beryl: Be3Al2Si6O18. Courtesy of OCW." text="Beryl: Be{{< sub 3 >}}Al{{< sub 2 >}}Si{{< sub 6 >}}O{{< sub 18 >}}." >}}
{{< image-gallery-item href="81ba381fade05dbad91194b41151d575_lab2-18.jpg" data-ngdesc="Tourmaline single crystal and tourmalines in quartz: NaMg3Al5B3Si6O27(OH)4. Courtesy of OCW." text="Tourmaline single crystal and tourmalines in quartz: NaMg{{< sub 3 >}}Al{{< sub 5 >}}B{{< sub 3 >}}Si{{< sub 6 >}}O{{< sub 27 >}}(OH){{< sub 4 >}}." >}}
{{< image-gallery-item href="1d7abc15a62bbec93420b7dcfa6ec674_lab2-19.jpg" data-ngdesc="Radiating acicular tourmaline crystals and tourmaline crystal in quartz: NaMg3Al5B3Si6O27(OH)4." text="Radiating acicular tourmaline crystals and tourmaline crystal in quartz: NaMg{{< sub 3 >}}Al{{< sub 5 >}}B{{< sub 3 >}}Si{{< sub 6 >}}O{{< sub 27 >}}(OH){{< sub 4 >}}." >}}
{{</ image-gallery >}}

Here's a complete list of legacy shortcod examples:
opening: {{< quiz_choice arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /quiz_choice foo="arg1" bar="arg2" >}}

opening: {{< quiz_choices arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /quiz_choices foo="arg1" bar="arg2" >}}

opening: {{< quiz_multiple_choice arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /quiz_multiple_choice foo="arg1" bar="arg2" >}}

opening: {{< quiz_solution arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /quiz_solution foo="arg1" bar="arg2" >}}

opening: {{< resource_file arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /resource_file foo="arg1" bar="arg2" >}}

opening: {{< video-gallery arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /video-gallery foo="arg1" bar="arg2" >}}

opening: {{< youtube arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /youtube foo="arg1" bar="arg2" >}}

opening: {{< anchor arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /anchor foo="arg1" bar="arg2" >}}

opening: {{< approx-students arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /approx-students foo="arg1" bar="arg2" >}}

opening: {{< br arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /br foo="arg1" bar="arg2" >}}

opening: {{< div-with-class arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /div-with-class foo="arg1" bar="arg2" >}}

opening: {{< fullwidth-cell arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /fullwidth-cell foo="arg1" bar="arg2" >}}

opening: {{< h arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /h foo="arg1" bar="arg2" >}}

opening: {{< image-gallery-item arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /image-gallery-item foo="arg1" bar="arg2" >}}

opening: {{< image-gallery arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /image-gallery foo="arg1" bar="arg2" >}}

opening: {{< quote arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /quote foo="arg1" bar="arg2" >}}

opening: {{< simplecast arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /simplecast foo="arg1" bar="arg2" >}}

opening: {{< sub arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /sub foo="arg1" bar="arg2" >}}

opening: {{< sup arg1 "{{< sup 2 >}}" >}} ... and now closing {{< /sup foo="arg1" bar="arg2" >}}

@mbertrand Good catch 👍  Is fixed.

repo: mitodl/ocw-studio


text:
Update posthog env vars

What are the relevant tickets?
https://github.com/mitodl/mitxonline/issues/2116
Description (What does it do?)
Update Posthog environment vars in order to support changes made in https://github.com/mitodl/mitxonline/pull/2211
POSTHOG_API_TOKEN to POSTHOG_PROJECT_API_KEY.
Add POSTHOG_ENABLED=True.

@Ardiea should I keep the existing environment variables and just add the newly named variables?  MITx Online currently depends on the existing variable names, but a future release will depend on the updated variable names.

repo: mitodl/ol-infrastructure


text:
Update Docker deploy to include an overview and tags

Right now the deployment page for our Docker builds is sparse and not particularly helpful. There should be an overview which links to the project README.md at the very least, and tags to aid in discovering the images.
https://hub.docker.com/r/mitre/hipcheck

I do not have permission to assign this issue to myself, but I have started to work on it!

@patrickjcasey you should have permissions now! I've added you to the @mitre/hipcheck-maintainers team :smile:

authors: alilleybrinker patrickjcasey

repo: mitre/hipcheck


text:
Rust 1.35 format changes, allow a clippy lint re assert false

Not sure why this didn't work:
diff --git src/runtime.rs src/runtime.rs
index bdb4d82..9281798 100644
--- src/runtime.rs
+++ src/runtime.rs
@@ -555,6 +555,7 @@ pub fn assert_snapshot(
     }

     if should_fail_in_tests() {
+        #[allow(clippy::assertions_on_constants)]
         assert!(
             false,
             "snapshot assertion for '{}' failed in line {}",

Any there any good first issues for those looking for some rust exposure and interested in expect tests?

I'm also seeing a test failure on master: https://travis-ci.org/max-sixty/insta/jobs/538486028
Though can't repro locally. What could the diff be? Rust versions?

It was the rust version. I added a commit that changed the snapshot so it tests pass on stable.
On nightly, we get this error

Reviewing [1/1]:
 insta (0.8.1)
Snapshot file: tests/snapshots/test_redaction__foo_bar.snap
Snapshot: foo_bar
Source: tests/test_redaction.rs
New: Wed, 29 May 2019 01:05:39 -0400

-old snapshot
+new results
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
Selector::parse(".foo.bar").unwrap()
──────┬─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
    1 ⋮ Selector {
    2 ⋮     selectors: [
    3 ⋮         [
    4 ⋮             Key(
    5 ⋮-                "foo"
    6 ⋮+                "foo",
    7 ⋮             ),
    8 ⋮             Key(
    9 ⋮-                "bar"
   10 ⋮-            )
   11 ⋮-        ]
   12 ⋮-    ]
   13 ⋮+                "bar",
   14 ⋮+            ),
   15 ⋮+        ],
   16 ⋮+    ],
   17 ⋮ }
──────┴─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

@mitsuhiko lmk if that's reasonable / any questions?

Thanks!

If you have any other 'good first issues' I'd be up for helping out @mitsuhiko

This one would be good to fix: #36. Shouldn't be too hard if we change the macros slightly.

authors: max-sixty mitsuhiko

repo: mitsuhiko/insta


text:
Switching docs tweaks

Clearing up a few bits I found difficult to parse whilst reading the docs on comparing with Django templates.

Thanks!

:rainbow: :rainbow: :rainbow:

authors: adamchainz untitaker


text:
drop old ruby support on CI

CI time is too long to wait. I'm happy to reduce test matrix.
ruby 2.3.0 was already EOL.
https://www.ruby-lang.org/en/news/2019/03/31/support-of-ruby-2-3-has-ended/
ruby 2.3 is left bacause of Rails4.x testing.
I think Rails4.x can also drop support from next release.

No code are exist that is not working old ruby version.
But adding required_ruby_version to gem spec is better for compatibility for old ruby users.
https://guides.rubygems.org/specification-reference/#required_ruby_version

Good call. I think we should keep Rails 4.2 testing. Apparently Rails 4.2 supports Ruby 2.4.5.

Sorry, reading between my lines, let's keep Rails 4.2 but drop Ruby 2.3.

@brendon Do you mean drop Rails v4.1 support on this PR?

Yes I think so. 4.2 is the only one currently supported by Rails. I'm happy to keep Rails 4.2 support for a year or two after Rails drops support though as I've been in that position where I've been stuck on a slightly out of date version before.

https://github.com/mixonic/ranked-model/blob/master/ranked-model.gemspec#L16 activerecord dependency should be upgrade to 4.2?

Yes that's a good idea :)

@brendon CI passed!

Choice! :)

repo: mixonic/ranked-model


text:
UIView+MPHelpers.m missing from the framework

The repository has a file named UIView+MPHelpers.m, but it's not in the Xcode project and therefore not compiled as part of the framework. It seems this file is what provides the implementations of the 3 methods declared in MPCategoryHelpers.h:
@interface UIView (MPHelpers)

- (UIImage *)mp_snapshotImage;
- (UIImage *)mp_snapshotForBlur;
- (int)mp_fingerprintVersion;

@end

So the lack of this UIView+MPHelpers.m means these methods will crash when they're called. I don't know under what circumstances they will be called, but I searched the project and there's 2 invocations of mp_snapshotImage (I didn't check the other 2 methods).

Do you have a crash log?

I didn't crash, my coworker did. I'll see if he has one.

The file is in the repo and the project.

No it's not. Are you verifying with CocoaPods? Because that doesn't use the project. Using a fresh checkout of the latest master, the Mixpanel.xcodeproj project does not contain the UIView+MPHelpers.m file.

(this problem arose when using mixpanel via Carthage, which builds the xcodeproj, unlike CocoaPods which uses a different way of specifying the source)

Coworker says he doesn't have a crash log either, he hit the issue with the debugger attached (so no crash log was generated). The workaround we went with was to literally copy UIView+MPHelpers.m into our own project, which fixed the crash.

What version of the SDK are you using?

The tip of master. Our Cartfile is just specifying github "mixpanel/mixpanel-iphone" (we're relying on the Cartfile.resolved to avoid accidental update breakage), and I confirmed by checking out a fresh copy of the repository.

@samgreen Any update on this? Can you at least reopen the ticket? The tip of master is definitely broken in regards to this file not being in the project.

Sure thing, we'll verify this through carthage

thank you!

It seems I'm having the same issue. I update from mixpanel 2.8.1 to 2.8.3 because of the BitCode Support issue and now I have this new issue. It appears that MPCategoryHelpers was removed from 2.8.3

@david6p2 This is a separate issue. It appears you need to run pod update Mixpanel

@kballard I'm so sorry for the confusion. You are absolutely correct and I have resolved the issue locally. We will get a release out ASAP.
Again so sorry for the misunderstanding, I was looking in the wrong place, thanks again for your patience.

Glad to hear it!

This is resolved in v2.9.0. Thanks!

@samgreen I think is the same error. I already did a pod update Mixpanel and it update to 2.9.0 but I keep seeing in the Podfile.lock that Mixpanel/MPCategoryHelpers were removed from version 2.8.2 and above.
-  - Mixpanel (2.8.1):
-    - Mixpanel/Mixpanel (= 2.8.1)
-  - Mixpanel/Mixpanel (2.8.1):
-    - Mixpanel/MPCategoryHelpers
-  - Mixpanel/MPCategoryHelpers (2.8.1)
+  - Mixpanel (2.9.0):
+    - Mixpanel/Mixpanel (= 2.9.0)
+  - Mixpanel/Mixpanel (2.9.0)

And the project won't compile when I try to run it on my device trowing this error
ld: library not found for -lMPCategoryHelpers
clang: error: linker command failed with exit code 1 (use -v to see invocation)

If I run it on the simulator it runs fine.

@david6p2 The podspec was modified back in July to remove the MPCategoryHelpers subspec and stop depending on it. This change was in 2.8.2, and is completely unrelated to this ticket here. You should try ⌘⌥⇧K to Clean Build Folder. Beyond that, if you're still having a problem, open a new ticket.

@kballard is correct with this advice.
@david6p2 In addition to cleaning, I would suggest temporarily removing Mixpanel from your podfile, performing a pod install, then re-adding and performing another pod install.

Or just run git clean -xfd to completely clear your working directory of all untracked files (which, presumably, should include the workspace that cocoapods is generating) and then re-running pod install.

Thanks @kballard and @samgreen. You were correct. It was some weird problem with my Linker flags. I have to remove all my pods, erase the linker flags to them, git clean -xfd and then reinstall all the pods again. Sorry

authors: david6p2 ekimia kballard samgreen

repo: mixpanel/mixpanel-iphone


text:
WoW ! Thank you . this is just what i was looking for

Great work .
I'll kick the tires on it  a little to test it.
I'm really looking forward to collaborate on this
⭐

I'd loved to help write up the installation guide.
I would assume i just drop it in to the components folder just like VC_Framework ?

all plugins are installed in the same manner.
https://doc.4d.com/4Dv17R4/4D/17-R4/Installing-plugins-or-components.300-4085753.en.html
quite similar to components:
https://doc.4d.com/4Dv17R4/4D/17-R4/Overview.300-4085682.en.html
the main difference is the language used (C or 4D)

Awesome,  I'll take it for a spin tomorrow, we're running on v16.

Neat.
I was able to sign a token and verify it on jwt.io
but not  the othe way around. I created the token on the jwt.io and paste it in 4d along with the secret.
I think I'm doing it right,
did I mess up ?

$token:="eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwiaWF0IjoiMTUxNjIzOTAyMiJ9.NTghxvd6D0DCZNZxlLf9U1wSJHxsA-m17OAOwcKKgzA"

$result:=JWT Verify ($token;"your-256-bit-secret")  // => 0

honestly I haven't thoroughly tested the "verify" command since we didn't need it for our project.
you can find my messy code in
https://github.com/miyako/4d-plugin-jwt/blob/master/4DPlugin.cpp

authors: luismasg miyako


text:
Compatibility with templating engines

I use MJML for emails in a workflow that includes nunjucks/twig style templating.
Rendering variables(+contatenation) looks like this:
{{ variable ~ 'words' }}
I noticed that this case generates garbage with mjml:
{{ var ~ '<word' }}
Notice how this looks like an open tag. Not even a complete one. Well, mjml v3.3.0 auto-closes this non-tag somewhere down the line in its generated code. v3.3.5 additionally butchers the code immediately surrounding the malformed tag as well.
Would this be considered a bug? One resolvable on mjml's part? Or would it be left for template users to work around ourselves?

Hi @martixy
Yes this is a known bug sadly in MJML parser, it produces sometimes some random opening tag when encountering chars like </ >/...
It will no longer be an issue in MJML4 because we totally rewrite the parser. In the meantime you should replace every </ > with the ASCII HTML counterpart < >, they will be decoded in the final output and should prevent this behavior.
I'm closing this issue because there's already an issue opened on that #394 :)

authors: iRyusa martixy

repo: mjmlio/mjml