Invalid Column Name

Executing an operation to get and "Tasks" entity from DB throws an "System.Data.SqlClient.SqlException". Task entity containd a fileld (idBedroom) representing an FK to "Bedrooms entity. The relationship between them is one to many.
The entities are defined bellow:
public partial class Tasks
{
    public Task() { }
    public int IdTask { get; set; }
    public int IdBedroom { get; set; }
    ....................
    public Bedrooms IdBedroomNavigation { get; set; }
}

public partial class Bedrooms
{
        public Bedrooms()
        {
               Tasks = new HashSet<Tasks>();
        }
        public int IdBedroom { get; set; }
        ...........................
        public ICollection<Tasks> Tasks { get; set; }
}

The code defining Tasks entity by modelBuilder in OnModelCreating function is:
modelBuilder.Entity<Tasks>(entity =>
{
         entity.Property(e => e.IdTask).HasColumnName("idTask");
         entity.Property(e => e.IdBedroom).HasColumnName("idBedroom");
         entity.HasOne(d => d.IdBedroomNavigation)
                  .WithMany(p => p.Tasks)
                  .HasForeignKey(d => d.IdBedroom)
                  .HasConstraintName("FK_Tasks_Bedrooms");
          .......................
}

Produced T-SQL query by EF is:
SELECT [t].[idTask], [t].[BedroomsIdBedroom]
FROM [Tasks] AS [t]

The exception is:
Exception message: "Invalid column name 'BedroomsIdBedroom'".

Stack trace:    at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at System.Data.SqlClient.SqlDataReader.TryConsumeMetaData()
   at System.Data.SqlClient.SqlDataReader.get_MetaData()
   at System.Data.SqlClient.SqlCommand.FinishExecuteReader(SqlDataReader ds, RunBehavior runBehavior, String resetOptionsString)
   at System.Data.SqlClient.SqlCommand.RunExecuteReaderTds(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, Boolean async, Int32 timeout, Task& task, Boolean asyncWrite, SqlDataReader ds)
   at System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, TaskCompletionSource`1 completion, Int32 timeout, Task& task, Boolean asyncWrite, String method)
   at System.Data.SqlClient.SqlCommand.ExecuteReader(CommandBehavior behavior)
   at System.Data.SqlClient.SqlCommand.ExecuteDbDataReader(CommandBehavior behavior)
   at System.Data.Common.DbCommand.ExecuteReader()
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.Execute(IRelationalConnection connection, DbCommandMethod executeMethod, IReadOnlyDictionary`2 parameterValues)
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.ExecuteReader(IRelationalConnection connection, IReadOnlyDictionary`2 parameterValues)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable`1.Enumerator.BufferlessMoveNext(Boolean buffer)
   at Microsoft.EntityFrameworkCore.Storage.Internal.SqlServerExecutionStrategy.Execute[TState,TResult](TState state, Func`3 operation, Func`3 verifySucceeded)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable`1.Enumerator.MoveNext()
   at System.Linq.Enumerable.SelectEnumerableIterator`2.MoveNext()
   at System.Linq.Enumerable.TryGetFirst[TSource](IEnumerable`1 source, Boolean& found)
   at lambda_method(Closure , QueryContext )
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.<>c__DisplayClass17_0`1.<CompileQueryCore>b__0(QueryContext qc)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.Execute[TResult](Expression query)
   at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryProvider.Execute[TResult](Expression expression)
   at System.Linq.Queryable.FirstOrDefault[TSource](IQueryable`1 source)
   at HRPv3.Data.Services.SharedRepository.GetTask_ById(Int32 idTask, Boolean nonCancelled) in C:\Users\matei.nenciu.INTRANET\Documents\Visual Studio 2017\Projects\HRPv3.2.API\HRPv3.2.Data\Services\SharedComponents\TaskShared.cs:line 1668

Steps to reproduce
Scaffolded my DB then execute the code :
this.Context.Tasks.Where(...);

Can someone tells me why EF generates column name "IdBedroom" as "BedroomIdBedroom" ? Am I configuring something wrong?
I have other colums representing FK's but only this one is generated wrong.
I also removed the connection between Bedrooms and Tasks and it works like a charm.
Further technical details
EF Core version: 2.0.0.
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10 latest update
IDE: Visual Studio 2017 version 15.3.4

I tried reproducing above based on data provided. First error I hit was the entityTypes don't have PK configured. Once I set PKs, it works correctly for me.
Can you please modify following sample code to demonstrate issue you are seeing.
using System;
using System.Collections.Generic;
using System.ComponentModel.DataAnnotations;
using System.Linq;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Logging;

namespace EFSampleApp
{
    public class Program
    {
        public static void Main(string[] args)
        {
            using (var db = new MyContext())
            {
                // Recreate database
                db.Database.EnsureDeleted();
                db.Database.EnsureCreated();

                // Seed database

                db.SaveChanges();
            }

            using (var db = new MyContext())
            {
                // Run queries
                var query = db.Tasks.ToList();
            }

            Console.WriteLine("Program finished.");
        }
    }

    public class MyContext : DbContext
    {
        // Declare DBSets
        public DbSet<Tasks> Tasks { get; set; }

        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        {
            // Select 1 provider
            optionsBuilder
                .UseSqlServer(@"Server=(localdb)\mssqllocaldb;Database=_ModelApp;Trusted_Connection=True;Connect Timeout=5;ConnectRetryCount=0")
                //.UseSqlite("filename=_modelApp.db")
                //.UseInMemoryDatabase(databaseName: "_modelApp")
                .EnableSensitiveDataLogging()
                .UseLoggerFactory(new LoggerFactory().AddConsole());
        }

        protected override void OnModelCreating(ModelBuilder modelBuilder)
        {
            // Configure model
            modelBuilder.Entity<Tasks>(entity =>
            {
                entity.Property(e => e.IdTask).HasColumnName("idTask");
                entity.Property(e => e.IdBedroom).HasColumnName("idBedroom");
                entity.HasOne(d => d.IdBedroomNavigation)
                    .WithMany(p => p.Tasks)
                    .HasForeignKey(d => d.IdBedroom)
                    .HasConstraintName("FK_Tasks_Bedrooms");
            });

        }
    }

    public partial class Tasks
    {
        [Key]
        public int IdTask { get; set; }
        public int IdBedroom { get; set; }
        public Bedrooms IdBedroomNavigation { get; set; }
    }

    public partial class Bedrooms
    {
        public Bedrooms()
        {
            Tasks = new HashSet<Tasks>();
        }
        [Key]
        public int IdBedroom { get; set; }
        public ICollection<Tasks> Tasks { get; set; }
    }

}

I tested your code and worked. I'm investigating today why my full code doesn't work.

Ok so I made another program and I found the problem: my Bedroom model is composed with 2 partial Bedroom classes:

public partial class Bedroom that contains the fields described in my first post.
another public partial class Bedroom that contains some additional fields and methods.

I found the code causing me the invalid column problem:
public partial class Bedrooms
    {
       .................
        public IOrderedEnumerable<Tasks> OrderedTasks
        {
            get
            {
                if (IsEmergency)
                    return this.Tasks
                        .OrderBy(t => t.IdStationStartNavigation.Name)
                        .ThenBy(t => t.DateActualStart);
                else
                    return this.Tasks.OrderBy(t => t.DateActualStart);
            }
        }
      ........................
     }

When I comment this code it works like a charm. Any idea what is wrong with it?

@MateiNenciu - With that code in other partial class, basically you are defining 2 navigations in Bedrooms pointing to Tasks. Hence tries to create 2 relationships. Since you have configured one of them explicitly the other one will be configured by conventions and creates a shadow property to represent the FK. (in this case it is named entityType + PK name = Bedrooms + IdBedrooms).  Looking at the code it seems like the intent of OrderedTasks is not to have 2 relationships but just provide related data ordered according to business logic. You can achieve that by putting [NotMapped] attribute on OrderedTasks navigation property. That way EF Core will ignore that navigation and will not try to create relationship associated with it.
Feel free to re-open if you still face issues after using NotMapped attribute.

It worked thak you for explanation!

While NotMapped is a valid work-around, this is still a bug. Creating and requesting shadow properties is not expected behavior. It isn't like we can actually access and use these new properties.
Beyond that, the error message is very misleading because nothing about it indicated what EF is doing.

This behavior will also occur if you have a class called BedroomView with a the property public ICollection<Tasks> Tasks { get; set; }.
In that case every query that touches Tasks will break, even if  BedroomView is not involved in said query.

@Grauenwolf - NotMapped is not work-around, it is correct way to configure. Every navigation in your class defines a relationship in database. If user specifies 2 navigations then there will be 2 relationships.
This issue only happens when you have a navigation like property for business logic but not part of the model or the example you gave, defined an entity with navigation but not wanting the relationship. When you define your domain model which is not same as your database model, you are going to run into issues. Unwanted navigations being treated as relationship is just one of the ways. You could have CamleCase properties in domain and lowercase column names in database. Or int stored as strings in database. There are many different ways, when database model does not match domain model exactly. And there is no way to find out what was incorrect in user code which caused the error. If an existing database is being targeted then its user's responsibility to map your domain model to your database model correctly. Any failures to do so can cause runtime failures.
Verifying that both models are same is cost associated with it. Most importantly, we don't want to block users from using certain features which are not supported yet. e.g. An entityType can be mapped to a view instead of table. Certainly saving data will not work but query would just work fine. That allows using views which returns ad-hoc types. If EF Core went to stricter restrictions then it will be impossible to use such things till supported.


Additional condition appears when setting up rule with OR function

Hi @KalliamvakosKon ,
When we are setting up the conditions using OR, if we have created the first statement before dragging in the OR function, then the first statement will re-appear. So, instead of getting (condition A or condition B), we will end up with (condition A and (condition A or condition B)) should we forget to remove the condition A afterward. Please see the attached video.
or_function_bug.MP4.zip

Thanks @winnerng I reported the issue now to our team. Will close this one for now.


#585 - Sign-in: Need to update bundle id in google.plist to support latest changes in the bundle name - Implemented

#585 implemented.
Disabled HockeyApp version check for iOS Simulator.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Vadim-Bulavin sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


[release/1.2] Windows:NewDirectIOFromFIFOSet

Cherry pick for https://github.com/containerd/containerd/commit/90caf6f6a7356756c2595b7695af7b12b87cdf2e to the 1.2 branch.
Refs: https://github.com/containerd/containerd/issues/3011

FAIL: TestContentClient/CommitErrorState  -> this test was failed, looks fail on window platform

@ehazlett looks like some more commits were requested after you opened this PR; I went ahead and picked the other ones (plus some more; see my comment on that PR); https://github.com/containerd/containerd/pull/3013

Closing in favor of https://github.com/containerd/containerd/pull/3013


Would adding a callback be considered useful ?

Hi,
I forked to add (mainly for myself) the possibility to add a callback to the fitvids call, taking the newly created container as an argument.
Would you consider this use-specific, or would you be interested in a PR (in this case, I'd take care of adding tests for this) ?
Have a nice day.

Hi! I'm definitely not opposed to it.
Do you mind sharing an example of how you would use the return value? I just want to understand what you're trying to do.

In some Vue app I'm developing, a container can contain layered absolute-positioned divs, sometimes above a static one, sometimes not, and in those cases, I need to retrieve the padding value to also use it on the parent container.
But I'm calling fitvids on specific elements to avoid a global treatment of video elements, so, in this use case, the return value serves my purpose.
I was also wondering if I could switch to a global fitvids() call, given I'm passing the callback an array looking like this mock :
[
  {
    el,
    cssValue: 'xx,x%',
    aspectRatio: 0.xxx,
  },
  ...
]

To further re-use those values.
To be fair, this situation came from an undiagnosed need for videos when this app was drafted, and a lack of time & budget to refactor before the first delivery. (So, it would help me expand on a hack.)

Hmm. I'm not sure I follow. You want to get the paddingTop of each of the wrappers (ie. the number defined here)? Why do you need to use it on the parent container?
 It might make it more clear. (Sorry for the confusion!)

I'm using some full-width, 8/5 aspect-ratio containers that can contain a mix of those items :

Two picture containers that are both a 4/5 ratio, side-to-side, that combined have a fixed ratio of 8/5
A 8/5 full-width "cover" picture that is hiding everything below, absolutely positioned
A video, either directly under this cover in case there are no picture containers, either revealable with a "show video" button.

In the case there's a "cover" and a video, but no picture containers, I use a padding hack to attain this 8/5 ratio, even there are no statically positioned items inside this container. When the cover fades out, in case the video has an aspect-ratio smaller in height than 8/5, the margins below the container get too high, so I'm retrieving the padding value used by the video to correct my container's padding.
I should provide you a fiddle. Sadly, we have no time to rework the markup for a last-minute edge case, so I'm just hooking into the video reveal step to get its fitvids-provided padding value from the DOM. To clean this a little bit, I'd like to get a return value from fitvids.
But I'm starting to think this is really too use-case-specific to merge a PR.
Anyway, thanks for your answers and your work, have a nice day !

Yeah, I think this is a bit too specific of a case for this library.
I'm totally happy to change the API, and make/accept PRs, but I just want to make sure they're broadly applicable first.
Thanks for reaching out to ask though, and feel free to re-open if you find this case cropping up more and more!


chore: Rename Actions, Refactor Actions

Codecov Report

Merging #302 (b08d1c4) into main (e7bd916) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main     #302   +/-   ##
=======================================
  Coverage   43.60%   43.60%
=======================================
  Files          19       19
  Lines        2128     2128
=======================================
  Hits          928      928
  Misses       1200     1200

Flag
Coverage Δ

unittests
43.60% <ø> (ø)

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


Added transformation for pushdown and corrected the bug https://cdap.…

The transformation logic is to set all the values of the properties that are created in WindowsAggregationDefinition of CDAP. Added transformation logic for Windows-aggregation repo and unit tests for generating aggregation definition .As part of testing the code identified and created the bug https://cdap.atlassian.net/browse/CDAP-19670 and added fix for the bug in the PR.
For the existing code, unit tests are to be created. Jira https://cdap.atlassian.net/browse/CDAP-19671 as part of tech debt

There are a lot of unnecessary whitespace diffs in the PR. Can you look into them and undo the diffs wherever they are not intentional? They are causing indentation and readability issues

Taken care

LGTM apart from the Long vs String variable typing and a minor nit.

Will take care during refactoring of code after all pushdowns are done


create sample for amp-list

Please assign this to me

Implemented.

tip: you can automatically close issues via commit messages by adding Closes #27 in your commit message. https://help.github.com/articles/closing-issues-via-commit-messages/


Change hard-coding to dynamic list

https://github.com/InformaticsGenomicMedicine/DraftCoreDataModel/blob/79450932a44ee677816f9c02e6f5660ea03e5ec1/src/core_variant.py#L95C83-L95C83
Replace the hard-coded list in the error message with a run-time expansion of the allowed values array. This will facilitate maintenance by ensuring the content is always in sync and doesn't need to be changed in more than 1 place.
Please change the linked code and then search for any other similar instances that need to be updated.

Fixed throughout the module:
https://github.com/InformaticsGenomicMedicine/DraftCoreDataModel/blob/9165f8bba932e2e838e4a60a1c884fa505aff5dc/src/core_variant.py#L130

Reviewed with @rrfreimuth.


Fix #484: Remove dataset from entity

Also:

Dropping dataset and kind from Entity constructor.
Dropping Entity.from_path factory.
Removing Connection.dataset() quasi-factory.
helpers.entity_from_protobuf() no longer accepts / needs dataset.
Making Entity.key a data attribute instead of a getter/setter method.
Removing Entity.dataset() (since no longer stored).
Making Entity.kind() and Entity.exclude_from_indexes() into @property's.
Removing Entity._must_dataset (no longer needed).
Adding optional connection argument to Entity.save() and Entity.reload().
Making Entity.save() and Entity.reload() return nothing.
Updating regression/datastore.py for changes.

Fix #484
NOTE: Has #483 as diffbase.

Coverage remained the same when pulling 0dbf10c01001acb5d7788e5d86ed50653fe2f412 on dhermes:remove-dataset-from-entity into b79d8624950b3b205de20f527488dde7aa8c8cd8 on GoogleCloudPlatform:master.

@tseaver I am re-working this now, having just merged #483.
Can you take a stab at #485?

@tseaver Rebase completed. PTAL.

Coverage remained the same when pulling 1dab8922e76e082dd52de5d7210c4dfaf516060e on dhermes:remove-dataset-from-entity into a02a73ebd32e6e31ffbea006d70f0744213e189f on GoogleCloudPlatform:master.

Looks OK.  No impact on regression tests?

The impact is the code looks nicer every time we throw something away :+1:
LGTY?

Yes -- I just didn't see any changes in the regression side.

They're there, at the very bottom.

We have regression failures right now.

Where? If you're referring to build 808, that was a transitive failure (we should add retries so flakes stop occurring so often).

Like so:
======================================================================
FAIL: test_empty_kind (datastore.TestDatastoreSave)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/tseaver/projects/agendaless/Google/src/gcloud-python/regression/datastore.py", line 141, in test_empty_kind
    self.assertEqual(posts, [])
AssertionError: Lists differ: [<Entity[{'kind': u'Post', 'id': 123456789[522 chars]se}>] != []

First list contains 2 additional elements.
First extra element 0:
<Entity[{'kind': u'Post', 'id': 123456789L}] {u'title': u'How to make the perfect pizza in your grill', u'publishedAt': datetime.datetime(2001, 1, 1, 0, 0, tzinfo=<UTC>), u'wordCount': 400L, u'author': u'Silvano', u'rating': 5.0, u'tags': [u'pizza', u'grill'], u'isDraft': False}>

+ []
- [<Entity[{'kind': u'Post', 'id': 123456789L}] {u'title': u'How to make the perfect pizza in your grill', u'publishedAt': datetime.datetime(2001, 1, 1, 0, 0, tzinfo=<UTC>), u'wordCount': 400L, u'author': u'Silvano', u'rating': 5.0, u'tags': [u'pizza', u'grill'], u'isDraft': False}>,
-  <Entity[{'kind': u'Post', 'id':<PHONE_NUMBER>270848L}] {u'title': u'How to make the perfect homemade pasta', u'publishedAt': datetime.datetime(2001, 1, 1, 0, 0, tzinfo=<UTC>), u'wordCount': 450L, u'author': u'Silva

The test is test_empty_kind, so you have some entities from a previously failed test that didn't get deleted.
I made clear_datastore for cases like that but filed #453 when I realized I couldn't keep up with it as the API changed massively.

Ugh:
$ .tox/regression/bin/python regression/clear_datastore.py
This command will remove all entities for the following kinds:
- Character
- Company
- Kind
- Person
- Post
Is this OK [y/n]? y
Traceback (most recent call last):
File "regression/clear_datastore.py", line 107, in <module>
    remove_all_entities()
File "regression/clear_datastore.py", line 103, in remove_all_entities
    remove_kind(dataset, kind)
File "regression/clear_datastore.py", line 66, in remove_kind
    with dataset.transaction():
AttributeError: 'Dataset' object has no attribute 'transaction'

Yeah, I will fix the util after we get stable again. In the meantime:
from gcloud import datastore
from gcloud.datastore.query import Query

datastore.set_default_connection()
datastore.set_default_dataset(dataset_id='foo')

q = Query(kind='Post')

for post in q.fetch():
    post.key.delete()


how to query CallStack by transactionId in pinpoint-web

in pinpoint-web  which page can query CallStack by transactionId?

There is no page that can display information only with TransactionId.
This is because TransactionId alone can not display the desired information completely.
thanks.

Unfortunately, We do not plan now.
It would be really nice if you developed and request it to us.
thanks.


[FIX] fix optional passing of Oak TFs to be truly optional, necessary…

… for multi-robot/multi-sensor systems
Overview
Author: Holly Dinkel
Issue
Issue link (if present): #472
Issue description: Launching camera.launch in a multi-robot/multi-sensor system leads to system failures
Changes
ROS distro: Noetic
List of changes: camera.launch
Testing
Hardware used: ABB IRB120 robot + Oak-D Pro
Visuals from testing
These are documented in the Issue (#472).

Thanks for the PR!

@Serafadam DO NOT MAKE A NOETIC RELEASE WITHTOUT FIXING THIS!
No, this can't be right! This solution fixes one problem but creates another.
The only thing this PR did was invert the condition whether to include urdf.launch or not. The group additions is irrelevant.
With the inverted condition, we have now:
pass_tf_args_as_params:=false:

urdf.launch is not included
camera_i_publish_tf_from_calibration is false, so the driver also doesn't publish TF.
Thus nobody publishes the TF.

pass_tf_args_as_params:=true:

urdf.launch is included, sets /robot_description and publishes TF via the running robot_state_publisher.
camera_i_publish_tf_from_calibration is true, so <camera_name>/robot_description is set and TF is published by the driver
The static transforms are published two times, once nominal, once calibrated. What gets used by TF clients is random, depending on which message they get last.

I think a proper solution would be to invert this line:
<param name="$(arg name)/camera_i_publish_tf_from_calibration" value="$(arg pass_tf_args_as_params)"/>

If done, this would be the result:
pass_tf_args_as_params:=false:
TF is published by driver, <camera_name>/robot_description is set.
pass_tf_args_as_params:=true:
TF is published by urdf.launch, /robot_description is set.
Summed up, combination of this PR and the proposed changes would just invert the meaning of pass_tf_args_as_params, as now (in the version released into Noetic), passing pass_tf_args_as_params:=true actually publishes the calibrated transforms, which is non-intuitive based on the name of the argument.
I think the problem in #472 was that urdf.launch overwrites the global /robot_description parameter. It would be better if it would set the driver-private parameter instead, the same way the driver binary does: <camera_name>/robot_description. This might require fixing a few distributed rviz files if there are some, but would result in a consistent experience.
Also, the robot_state_publisher would need a remap from robot_description to ~robot_description.

Hi @peci1 @hollydinkel
Proposed changes are available in this branch, it would be a great help if you could test and verify them, they should be merged and released this week if there are no issues.

Thanks @Serafadam , the commit fixes the issue. However, I'd still prefer to invert the logic of pass_tf_args_as_params. I still think the currently implemented behavior does exactly the opposite than one would say based on the name of the argument.

@peci1 How about changing it to publish_tf_from_calibration as an argument, is that more explicit?

Yeah, that sound better ;)


virtualenvwrapper plugin: activate only works on second cd

I'm trying to use the virtuelenvwrapper plugin. I'm running into the situation that the first time I cd into a virtualenv, I get the error
∞ ➜  ~  cd Code/geodas
prepending /home2/hilboll/.virtualenvs/geodas-dev/bin to PATH
discarding /home2/hilboll/.virtualenvs/geodas-dev/bin from PATH
Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home2/hilboll/.local/lib/python2.7/site-packages/virtualenvwrapper/hook_loader.py", line 8, in <module>
    import inspect
  File "/usr/lib/python2.7/inspect.py", line 35, in <module>
    import string
  File "/usr/lib/python2.7/string.py", line 83, in <module>
    import re as _re
  File "/home2/hilboll/.virtualenvs/geodas-dev/lib/python2.7/re.py", line 105, in <module>
    import sre_compile
  File "/home2/hilboll/.virtualenvs/geodas-dev/lib/python2.7/sre_compile.py", line 14, in <module>
    import sre_parse
  File "/home2/hilboll/.virtualenvs/geodas-dev/lib/python2.7/sre_parse.py", line 17, in <module>
    from sre_constants import *
  File "/home2/hilboll/.virtualenvs/geodas-dev/lib/python2.7/sre_constants.py", line 18, in <module>
    from _sre import MAXREPEAT
ImportError: cannot import name MAXREPEAT
virtualenvwrapper.sh: There was a problem running the initialization hooks.

If Python could not import the module virtualenvwrapper.hook_loader,
check that virtualenv has been installed for
VIRTUALENVWRAPPER_PYTHON=/home2/hilboll/.virtualenvs/geodas-dev/bin/python and that PATH is
set properly.
ERROR: Environment '/home2/hilboll/.virtualenvs/geodas-dev' does not exist. Create it with 'mkvirtualenv /home2/hilboll/.virtualenvs/geodas-dev'.

However, when I then do cd .. && cd my_directory, everything works as expected (no errors).  Any idea what might be going on?

It seems the workon function crashed and got you into this error you see in line 243. It's a matter of following the execution trail upwards to see what might be the cause. It seems there is a bit of explanation in this issue: virtualenvwrapper#40
If this is still happening let me know and I'll see what I can do to help you.

Any update on this issue?

Thanks, @mcornella, and sorry for the belated reply.  I switched machines in the meantime and the problem has somehow gone away ...


Requesting for a pinout - Node MCU ESP32S

Thanks for this lovely library. Requesting for a pinout for the Node MCU ESP32S which seems to have its pin outs labelled incorrectly.
Cheers

is it this one : https://esphome.io/devices/nodemcu_esp32.html


initial changes for Gantt-like feature

for my bachelor thesis I will implement a Gantt-like view for processes. This should be as independent as possible from the rest of the project.
integrating the new component on the same level as the current modeler seems to be the best way to me as integrating it into the modeler might bring user experience benefits but would require much deeper integration into existing code. this should allow most of my work to be done inside the BPMNTimeline component without interfering with other developments.
for the usage of the .env variable I went with how I understood it for other features in the project.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Make codelabs compatible with --incompatible_load_{java,proto}_rules_from_bzl

Friendly ping?
(Also adding @meteorcloudy who has landed changes in this repo recently)


Add details on "Non-usable authentication methods"

Hello,
We received multiple requests to explain what are "Non-usable authentication methods" under Authentication methods for each user in Azure portal.
Can you please add more details what are "Non-usable authentication methods" on this public article or an explanation that if an administrator required re-register MFA for a user their current authentication methods will be under "Non-usable authentication methods" until the method is deleted or registered again?

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

ID: 51eadd8e-b820-be21-9cb3-b52f334fb94d
Version Independent ID: fe358aa5-5bb6-b8f0-8ab7-ef181dc8af42
Content: Manage authentication methods for Azure AD Multi-Factor Authentication - Microsoft Entra
Content Source: articles/active-directory/authentication/howto-mfa-userdevicesettings.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

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

@Justinha
Can you please check and add your comments on this doc update request as applicable.

@larasawalha90
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly

@ManoharLakkoju-MSFT Thank you for the reply and appreciate your help in making this clear to our customer

sorry for delay and thanks for raising this. I created an internal work item for SMEs to review and update the topic as needed. #please-close

Can we get an update on this? This is impacting people's ability to log in.

Ask the user if they have passcode disabled on their iPhone, this worked for me.

@larasawalha90 @MrTechGadget @moogs37 @ImNotIT The reason why a method is unusable is listed under Detail in the admin center. If you don't see a reason listed, please contact Microsoft support. For more info, please see https://learn.microsoft.com/en-us/azure/active-directory/authentication/concept-authentication-methods#usable-and-non-usable-methods

Thank you, I see you added this verbiage in June which does not address the
issue, which is that there is no explanation for what a non-usable state
is, how their authenticator app that has been usable for months or years
suddenly became "non-usable", and no instructions for how a user can
restore their authenticator app to a "usable" state.
Appreciate the continued lack of good support for these products /s Please
notice that the website is called learn.microsoft.com. It isn't
unreasonable for people to expect good documentation.
On Thu, Oct 5, 2023 at 9:54 AM Justin Hall @.***> wrote:

@larasawalha90 https://github.com/larasawalha90 @MrTechGadget
https://github.com/MrTechGadget @moogs37 https://github.com/moogs37
@ImNotIT https://github.com/ImNotIT The reason why a method is unusable
is listed under Detail in the admin center. If you don't see a reason
listed, please contact Microsoft support. For more info, please see
https://learn.microsoft.com/en-us/azure/active-directory/authentication/concept-authentication-methods#usable-and-non-usable-methods
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/109068#issuecomment-1748949548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AECUW5IDTYW4TIGKEWDSM5TX523YXAVCNFSM6AAAAAAXVJVZGCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBYHE2DSNJUHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

We are implementing MFA in our company and from entra ID we see:
1.6k Users registered by authentication methiod BUT 868 "Users capable of Azure multifactor authenticantion"
it means that half of registered users for a reason arent able to can use MFA !
It is very important to be able identify the list of those users to manage, mitigate and understand the source of the problem.
Please, implement the opportunity to extract these users.
thanks


Start Building 'README'

I thought it might be helpful to get your demo & instructions started in the 'README.md' for easy viewing by visitors to your repo.

Thanks!

Any updates here for the 'README'? How is progress on the build?
Have a great holiday!


#668: Update about page for version 3.6.1

This closes cessda/cessda.cdc.versions#668

Isn't the correct version 3.6.0?
https://github.com/cessda/cessda.cdc.versions/issues/665

No, because 3.6.0 has already been released


Wrong link when searching for "FAQ"

When "FAQ" is searched I get the link to https://electronjs.org/docs/tutorial/faq which is 404, but the valid link should be https://electronjs.org/docs/faq

Thanks for reporting this, @micnic.

This issue is also valid for "Glossary" and I guess for some other inputs.


Enhance contrib guide technologies section with beginner friendly containers and k8s resources

Goal
As a beginner contributor, I want to have a list of resources that i can check so that I can learn the basics about containers and Kubernetes.
Details
This issue is a result of the research done in #2710.
The technologies contrib guide page should be enhanced by adding new resources to the list or changing the list.
The list should have a new section at the top for beginners with beginner resource links about containers and kubernetes.
The list already has some beginner material as well that can be pulled into the new section to make it easier to find.
DoD

new beginner section added to list
beginner section filled with beginner-friendly material
beginner material from other sections is moved to the beginner section

I would like to work on this but i have a query:

As the k8 information pr has been merged so do i need to work on that or just containers ?

Hi @BalaadityaPatanjali you could move "Understanding the basic of Kubernetes" under the new beginner section and yes then find a good resource for the containers part.

Hi @BalaadityaPatanjali you could move "Understanding the basic of Kubernetes" under the new beginner section and yes then find a good resource for the containers part.

Surely ! I will work on this.

Containers
Containers are a basic prerequisite in the path of learning kubernetes and a good understanding
of containers will help a lot for working with and contributing to Keptn.

Understand the Basics of Containers

Docker's Official Getting Started Guide
Docker for beginner

Container Architecture

Understanding Docker Architecture

Container Concepts

Docker Images and Containers Explained
Container Security Explained
Container Orchestration Explained

Useful Tools

Docker Compose Overview
Docker Compose Tutorial
Docker Swarm

@RealAnna I have made the section and i have added the credible sources as well.


Create kipu-rc.txt

Please add this domain in yours lib.

@abdurayimov You might miss my previous comment. Please react.

@abdurayimov You might miss my previous comment. Please react.

Thanks for reviewing!

@abdurayimov Pull request merged. Thank you!


PCI-2572 Pin pip and python

pin pip version in Pipfile and install the dependencies with their correct versions in Docker image.
For example in newer version of pip the following function signatures are changed which breaks the in and check commands.

pip._internal.operations.prepare.unpack_url - 1 more required parameter: verbosity
pip._internal.models.target_python.TargetPython(): platform changed to platforms and abi to abis.
Additionally, it is now required to set the defaults values for:
options.platform = None
options.abi = None

https://github.com/Pix4D/concourse-pypi-resource/blob/67c7d902ed0b0429f3bc68a86ef8870b2216aa39/pypi_resource/pipio.py#L54
I think these changes deserve a separate PR and are probably not the only ones.

pin the Python version to 3.9 in Dockerfile and ensure its in sync with the version configured in Pipfile
updates/corrects the readme where needed.
stop running the python2 tests since setuptools version does not support it

Closes: https://github.com/cf-platform-eng/concourse-pypi-resource/issues/84
This is a draft PR. I will open the real one upstream after the first review pass from our team.

Question: should I also add mboldt as reviewer to this PR or only after I open the PR against the main branch in the upstream repository?

While trying to run integration tests (both local nexus and public pypi) I found a lot of issues (existing one from master). Please hold with review until I ping here again. Sorry for the noise.

@Pix4D/integration ready for review


Qwen2: can not run with the latest Qwen2 models

I am trying to run Qwen2 example with the latest Qwen2 0.5B model, but get a Error: cannot find tensor lm_head.weight.
https://huggingface.co/Qwen/Qwen2-0.5B

Ah thanks for reporting this, indeed there is a small difference in the new model architecture, I'm minting #2257 to support this and you should be able to use the new model with  --model 2-0.5b.

@LaurentMazare Thank you very much, it’s now running. However, the results from the 0.5B model are a bit strange, which might be due to other issues.

7b is incorrect


1157

Modified #1157 PR

Deploy preview for landscape ready!
Built with commit cc022a6243ab5d1afd17aeaedee5eb7cff125e96
https://deploy-preview-1159--landscape.netlify.com

LGTM, @dankohn please check


Remove duplicate elements from sorted Array

Given an integer array, nums sorted in non-decreasing order, remove the duplicates in place such that each unique element appears only once.
If there are k elements after removing the duplicates, then the first k elements of nums should hold the final result.
Return array containing all distinct elements after placing the final result in the first k slots of nums.
Example :
Input:
N = 3
Array = {1, 2, 2}
Output: 1 2
Explanation: After removing all the duplicates only two elements i.e., 1 and 2 will remain.

@Shashwat-Sahu I have done PR with complexity O(n). Please review it!

Ok @riyasingh775 saw that, done it !


Version 0.11.3

This pull-request bumps the ghost to version 0.11.3
Please keep the repo updated with latest.

Why no merge this ?

Because OpenShift does not support Node.js higher than 0.10, and Ghost >0.11.2 does not support anything lower than Node 4.*
This PR could break the build.

mmmm, okay i understand, i use the branch of @dkbhadeshiya and works fine for me , exists any petition for openshift upgrade Node.js version ?

http://ingenieroperales-jesusisrael.rhcloud.com/

@ripper2hl if you find such petition, please write its URL here for us. We'll go and subscribe it!
What version of the Node.js is working on your site?

Even though Ghost requires version 4.x+, it works fine. I guess Ghost does not use any 4.X+ specific Node APIs which can break the build. And keeping in mind that I've been using it on my own blog as well as production blogs as well. I think it's ready to be merged.
Any Suggestions are appreciated.
Even if the Ghost > 0.11.2 does not support older versions of node, this could still be updated to 0.11.2 seeing that it's stuck on 0.7.5 for almost a year now. @MunGell

@dkbhadeshiya It's the cool news! Thank you.

Openshift uses nodejs container from https://github.com/sclorg/s2i-nodejs-container
I guess by opening a pull request at that repository with latest LTS release of NodeJS, which can allow us to use latest LTS release of NodeJS.

@dkbhadeshiya excellent idea. Do you sure that OpenShift will update its container when the s2i-nodejs-container will be updated?

@23W Not so sure as I see there is Node version 4.0 available in s2i-nodejs-container and not available in OpenShift. So it's upto them if they want to include it or not. And I see that openshift v2 will eventually be replaced by v3 with containers. And they provide v4 in images built with v3. If they provide latest LTS on v3 while releasing, it will be great.

Could we move this to a newer version please. It is a shame that the quick-start is 0.7.5 but the upstream is now 1.6.2.


Reflect repo being moved in readme

According to this comment: https://github.com/jaswope/rack-reverse-proxy/pull/51#issuecomment-118666996
@jaswope This is supposed to be last PR to this repo :)

Haha, thanks.


I hope to get the version of CUDA for Pytorch.

I always encounter compilation errors related to the CUDA version during use. I want to install the corresponding Pytorch version. Can I get the CUDA version in INSTALL.md, thank you!

https://github.com/open-mmlab/mmdetection/blob/master/MODEL_ZOO.md
see the software environment.

Thanks! @Curry1201


Failed to compile example

I tried to run the Usage code, and got an error:
Module not found: Can't resolve 'classnames' in 'C:......*PROJECT_NAME*\node_modules\material-survey\components\USRegionQuestion'

Thanks @michaelozery . I'm guessing we need to add classnames to the package.json.
Can you trying running npm install classnames to see if it fixes it?

Yup that worked. Thank you


Make Session loading (Deserialize) faster

From @sdrapkin on Wednesday, May 10, 2017 8:43:05 AM
DistributedSession.cs adds deserialized property values in a loop to a Dictionary<EncodedKey, byte[]> _store.
This causes continuous dictionary resizing (which uses prime-number capacities). However, this resizing is wasteful, since Deserialize() already knows the required capacity (expectedEntries).
_store should be initialised with a known capacity inside Deserialize(), instead if being initialised with an unknown capacity inside ctor.
Copied from original issue: aspnet/Session#163

From @muratg on Tuesday, September 26, 2017 3:49:35 PM
@sdrapkin Would you be interested in sending a PR?

From @sdrapkin on Tuesday, September 26, 2017 7:15:21 PM
@muratg No (didn't sign CLA, and the change is trivial).

We won't be fixing this.


Shift tenant to aggregateId part of streamId

While the exact layout of a streamId is pretty arbitrary, in general it works better to have the tenant to the right of the '-' in the streamId as the default category projection treats anything to the left of the first '-' as the category (while this is configurable, this is the default, so Pit Of Success principles apply)

(Not adding the following goldplating validation logic, which I have implemented and used with success elsewhere in the interest of not drowning the reader in trivia)
// Generates a stream id in the canonical `{category}-{aggregateId}` format
public static string ToStreamId(Type streamType, object aggregateId, object? tenantId = null)
{
    var tenantPrefix = tenantId == null ? $"{tenantId}_"  : "";

    if (tenantId != null)
        CheckNoEmbeddedDashes(nameof(tenantId), tenantId.ToString()!);
    var category = ToStreamPrefix(streamType);
    CheckNoEmbeddedDashes(nameof(streamType), category);

    // (Out-of-the box, the category projection treats anything before a `-` separator as the category name)
    // For this reason, we place the "{tenantId}_" bit (if present) on the right hand side of the '-'
    return $"{category}-{tenantPrefix}{aggregateId}";

    static void CheckNoEmbeddedDashes(string argName, string value)
    {
        if (value.IndexOf('-') != -1)
            throw new ArgumentException("Stream name components may not include embedded '-' characters.", argName);
    }
}


[stableswap]: Add scaling factor tests

Background
Scaling factors currently have a basic test skeleton commented out in the keeper tests that needs to be moved to our pool_test.go and filled with test cases for scaling/descaling pool assets
Suggested Design

Move commented tests from keeper tests to pool_test.go
Add thorough test cases (even/uneven pools, even/uneven scaling factors, min and max scaling factors, overflow/underflow, panic catching etc.)

Acceptance Criteria

All new and existing tests pass

@hieuvubk I saw that you self-assigned - I was working on this alongside a few other scaling factor changes so will unassign you for now (I hope you weren't too far into it!).
In general I will probably be picking up (or already have significant progress for) the stableswap-related issues I create unless I tag them as 'Good First Issue' :)

Okay got it, thank you!


⚠ File access has degraded performance

In 246bb69, File access (https://fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 2039 ms

Resolved: File access performance has improved in 517721f after 177 days, 18 hours, 53 minutes.


Training from scratch

Hi,
I am trying to train your model from scratch. I dont exactly get it how you have fed the data to the model. Can you please provide the steps ?
Thanks!

I wrote a simple guide on README : https://github.com/ildoonet/tf-openpose#training
Let me know after you try.

Hi,
I think my question was confusing. let me rephrase, '/data/public/rw/coco-pose-estimation-lmdb/' How did you download/generate data at this path ??
Will surely keep you in loop once I begin with my training.
Thanks!

@ildoonet Can you share a model that you have trained on coco? Then i can evaluate it and train it on coco on the basis of  you model.

@nsbendre
I updated README.
$ wget -nc --directory-prefix=lmdb_trainVal/ 		http://posefs1.perception.cs.cmu.edu/Users/ZheCao/lmdb_trainVal/data.mdb
$ wget -nc --directory-prefix=lmdb_trainVal/ 		http://posefs1.perception.cs.cmu.edu/Users/ZheCao/lmdb_trainVal/lock.mdb

@felixfuu I will but no at the moment. It will take few weeks before I release them.

@ildoonet  Thank you ~


The Homework from<PHONE_NUMBER>

The homework is under 'examples' !

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[Bug]: Incorrect version and new version message

v0.5.0-beta.2
Windows 11
PowerShell 7.3.2
When running the latest beta I am warned that a new version is available, but shows me the version that I am currently on.

As an aside, we should also change the URL in the message to use the https://aka.ms/graph/proxy/download link.

This should be fixed by #199. Let's confirm if that's indeed the case

Seems to be working as expected in v0.5


⚠ Processors has degraded performance

In 16ced37, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 82 ms

Resolved: Processors performance has improved in 2ff4a0f.


Allocate EVP_PBE_CTL with OPENSSL_zalloc.

Fixes openssl#18598.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged to master and 3.0 branches. Thank you.


fix: add accept header

Fixes #3962

I think we probably need this code here as well: https://github.com/microsoft/botbuilder-js/blob/main/libraries/botframework-connector/src/auth/connectorFactoryImpl.ts

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.0002%) to 84.543%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.0002%

Covered Lines:
19655

Relevant Lines:
22023

💛  - Coveralls


scripts: ci: check_compliance fails custom projects file that contain DOS line endings

The problem:
We have projects files for Bluetooth Launch Studio and PTS application here.  Compliance check fails on bls (launch studio project file) files because of line endings which is required by the launch studio, it is not possible to import the file back to launch studio if we change the line endings.
Suggested solution:
Bypass compliance check for bls files.

How is this an issue for the Zephyr Project? Sounds more like an issue for your downstream project/application?

More detail:
This file is the project file that we use in Bluetooth Launch Studio to add/delete/update ICS for Zephyr Bluetooth Controller/Host/Mesh qualification.
Bluetooth Launch Studio web app only accepts files with DOS line endings, so storing this particular file with DOS line ending is mandatory-ish.
The current compliance check is not possible to pass for this file.

This issue actually prevents this PR to be merged: https://github.com/zephyrproject-rtos/zephyr/pull/68975
Explanation: https://github.com/zephyrproject-rtos/zephyr/pull/68975#issuecomment-1996839934

@m-alperen-sener, @PavelVPV Thank you for explaining this.

This is also an issue for certain Bluetooth configuration files that needs to have DOS line endings to work with testing software

this was fixed in 420ffdd8a88

this was fixed in 420ffdd

@nashif
Can you elaborate on how reducing the size of the error report helps with DOS line endings? Won't CI still fail?

@nashif
Can you elaborate on how reducing the size of the error report helps with DOS line endings? Won't CI still fail?

CI will fail, of course, but this is not the point or problem. the problem was that CI was getting stuck with 10s of thousands of checkpatch violation and processing them one by one, now we do not do that and fail with showing the errors directly and without trying to inline them on github.
DOS line endings is still a rule violation and we need to fail on it and apply exceptions where applicable.
So I am not sure to be honest what the original enahncement is about? We always have false positives that we override. The commit above was for a similar issue, do not remember which PR it was, there the compliance checks was getting stuck because of the thousands of DOS line endings.

Thanks. I agree that the compliance not finishing was an issue, but this issue was specifically for DOS line endings, which is still an issue regardless.
I do not consider https://github.com/zephyrproject-rtos/zephyr/pull/77283 a fix for this issue, and we should reopen, as the issue with DOS line ending still exist and ideally we would have some way of disregarding that check for specific files.

The issue still exists: https://github.com/zephyrproject-rtos/zephyr/pull/78400


[LESSON] - Tags, ids, classes

Closing for now.
When we get to the CSSS module we'll get back to this lesson.


how to roam around in a room unable to zoom upto the mark

how to use three js for indoor scene  - as it is getting restricted zoom in certain areas how to i zoom to pass that surface area .

If you are unable to zoom past a certain point, you probably need to pan the camera to a different location with a better vantage point. With a mouse you can do this by dragging the mouse while holding the right mouse button.

Just checking in, we you able to resolve this issue?

Closing for now, please let me know if you're still having trouble with this.


Build fails with HEAD (13b901bf1c5) on macos

Clone the repo;
cd into the repo;
make -j;
Fails with the following message:

[ 55%] Building passes/sat/sim.o
frontends/rtlil/rtlil_parser.y:48.9-18: syntax error, unexpected identifier, expecting string
make: *** [frontends/rtlil/rtlil_parser.tab.cc] Error 1
make: *** Waiting for unfinished jobs....

Seen that one before, you need to get the compilation process to use a modern version of bison.

Yep. Just figured out how crazy ancient Bison is on macOS.

In case this gets anyone else, the fix for a macOS build environment is trivial:

Yosys also comes with a Brewfile - if you have brew, the recommended way to set up the environment for compiling is:
$ brew tap Homebrew/bundle && brew bundle


Metric separator configuration

It might not be a real issue for some people, but using it with InfluxDB creates series like host/cpu-0/cpu-idle.
The '/' separator is a bit annoying here, considering that a lot of graphing software (like Grafana) are using the "Graphite standard" which is the dot.
I know that configuration can happen in another place, but do you think we could make it configurable so that the '/' could be change to another character (or string)?

haha, I think the '.' separator is annoying because hostnames contain dots (https://github.com/graphite-project/graphite-web/issues/191). But I'll see what I can do about being able to change the separator

Le 11/02/2015 18:10, Kimo R a écrit :

haha, I think the '.' separator is annoying because hostnames contain
dots (graphite-project/graphite-web#191
https://github.com/graphite-project/graphite-web/issues/191). But
I'll see what I can do about being able to change the separator
—
Reply to this email directly or view it on GitHub
https://github.com/kimor79/gollectd/issues/3#issuecomment-73920613.

Indeed, that's why the write_graphite plugin of collectd has an
EscapeCharacter parameter, defaulted to '_'
However, in the binary protocol, it is obviously not escaped.
Thanks for having a look at this!


Dynamic Programming: Palindrome Partitioning

@avinash201199  Please assign this issue to me for contributing to hacktober fest.

@NitikaGupta16
All the best :)
follow me for more such repo to contribute

Please assign me this issue for contributing to hacktober fest.


Keep Glyphs custom parameter does not work

After doing some testing on files that use the Keep Glyphs custom parameter (Noto Sans Devanagari, Noto Sans Bengali, Noto Sans Kannada, Noto Sans Malayalam) it was clear that nothing at all was happening. I see a couple problems so far.
font_project.py
276    def save_otfs(self,
277                  ufos,
278                  ttf=False,
279                  is_instance=False,
280                  interpolatable=False,
281                  use_afdko=False,
282                  autohint=None,
283                  subset=None,
   ...
406        for font, ufo in zip(fonts, ufos):
407            if subset is None:
408                export_key = GLYPHS_PREFIX + 'Glyphs.Export'
409                subset = ((GLYPHS_PREFIX + 'Keep Glyphs') in ufo.lib or
410                          any(glyph.lib.get(export_key, True) is False
411                              for glyph in ufo))
412            if subset:
413                self.subset_otf_from_ufo(otf_path, ufo)
   ...
446        keep_glyphs_list = ufo.lib.get(GLYPHS_PREFIX + 'Keep Glyphs')

subset is initialized to None so the first time through the for loop it is None, but after that it gets set to either True or False and any subsequent iteration doesn't necessarily get to the subset code. Each ufo needs to be checked for the presence of Keep Glyphs because it could be different for each one. In Noto Sans Bengali all glyphs are set to export so any(glyph.lib.get(export_key, True) is False for glyph in ufo) is False and so is (GLYPHS_PREFIX + 'Keep Glyphs') in ufo.lib because GLYPHS_PREFIX + 'Keep Glyphs' is never set in the ufo in the first place.
I adjusted things locally so that everything works, but when it does work we run into another big problem: https://github.com/googlei18n/glyphsLib/issues/205
If Keep Glyphs works and removes the encoded glyphs from the UI instances (because those are replaced with UI specific glyphs) then those codepoints need to be assigned to the UI specific glyphs. Otherwise they just show up as .notdef. That's way worse than the current behavior which is to completely miss the Keep Glyphs which leaves the non-UI glyphs in the UI instances.

@punchcutter thanks for debugging this. The first is definitely a bug. It should reset the subset variable to None and evaluate the custom parameters of each ufo independently. I'll fix that.
I'll also work on adding support for Reencode Glyphs in glyphsLib. I think the best place for this is in the apply_instance_data function that fontmake calls after mutatormath has generated the UFO instances. The custom parameters should already be dumped in the designspace document's lib element. We just need to apply them.

Ouch.. I just found another issue which related to this.
In glyphsLib 2.2.1 (used by fontmake 1.4), the "Keep Glyphs" instance custom parameter was (used to be) exported with the following UFO lib key:
"com.schriftgestaltung.Keep Glyphs"

That's how fontmake itself expects it to be when it goes to subset_otf_from_ufo.
However, in the latest glyphsLib 2.3.0, the same "Keep Glyphs" custom parameter is now being exported using this mouthful:
"com.schriftgestaltung.customParameter.InstanceDescriptorAsGSInstance.Keep Glyphs"

Therefore fontmake cannot see it...
The glyphsLib.builder.custom_params was completely rewritten by @belluzj to support going in both directions (glyphs -> UFOs, and viceversa):
https://github.com/googlei18n/glyphsLib/compare/v2.2.1...master#diff-b0b07578ca1cbb8c00dd528d6b044934
I personally don't like that long name (let alone the space between the words, but that's not the point).
The problem is, prefixing it with com.schriftgestaltung.*, suggests that this comes from Glyphs.app the app itself, whereas it really is a private key which is understood and meant to be consumed only by glyphsLib, ie. the python library. So, perhaps something like com.github.googlei18n.glyphsLib.* would be more appropriate in this and possibly other cases, I don't know.
It's one thing to use com.schriftgestaltung.* when Glyphs.app's built-in UFO exporter uses that prefix for the same kind of data. But if glyphsLib is storing new stuff in the UFO lib for its own use, it should not pretend that those come from Glyphs.app.
Btw, it's not just fontmake that needs to handle the new lib key prefix scheme from glyphsLib 2.3, but also ufo2ft, eg. here:
https://github.com/googlei18n/ufo2ft/blob/dbb107c8093bb8824c7e26fa69d240d3accdb548/Lib/ufo2ft/constants.py#L8

Anyway, for now I'll change fontmake to check both the old short name, and the the new long one.
I hope one day we get the chance to clean up all this technical debt..

I think glyphsLib should only do Glyphs.app to master UFOs+designspace and not Glyphs.app to master UFOs. That way instance specific parameters can be stored in the designspace instance lib, and be repricated when instance UFOs are generated. We could then get rid of com.schriftgestaltung.customParameter.InstanceDescriptorAsGSInstance.* keys and only have com.schriftgestaltung.* keys.

"com.schriftgestaltung.customParameter.InstanceDescriptorAsGSInstance.Keep Glyphs"

Yes, this is a stupidly long name. I'm going to explain how that came to be, so that a better solution can be found:

for font & master custom parameters, I added "GSFont" or "GSFontMaster" inside the name, because both font and master custom parameters end up in the same UFO lib. When we go in the other direction, we can know from the name where the custom parameter should go.

this is happening here as well for instance custom parameters ("InstanceDescriptorAsGSInstance"), but it is unnecessary because those custom parameters are not meant to "go back" to glyphs, since they are added to a UFO that has been freshly interpolated by fontmake/mutatorMath and that will be discarded when we get a TTF, and also because if this customParameter from the GSInstance is in conflict with one that came e.g. from the GSFont, then the instance value should overwrite (or be appended?) to the value from the GSFont.
even in the case of GSFont vs GSFontMaster, we could do:

GSFont <-> designspace lib
GSFontMaster <-> UFO lib
GSInstance <-> designspace instance lib
and thus there is a one-to-one mapping and no need for that part of the key name.

This part is added here:
https://github.com/googlei18n/glyphsLib/blob/b35f4abe16df4d49f687833dbf841b9d7b0ec281/Lib/glyphsLib/builder/custom_params.py#L77-L79

the key starts by "com.schriftgestaltung.customParameter" because I wanted to be sure of what is a custom parameter and what is not. A key like "com.github.googlei18n.glyphsLib.custom" would be fine as well. The key prefix is here:
https://github.com/googlei18n/glyphsLib/blob/b35f4abe16df4d49f687833dbf841b9d7b0ec281/Lib/glyphsLib/builder/custom_params.py#L66


Command Side Validation Issue

Timeframe
Today was a outage with ZITADEL.ch which affected our customers for 40 minutes starting from 2021-06-27 09:56 CEST
https://status.zitadel.ch/incidents/49172
More information to come, soon.

Hey @livio-a @adlerhurst please link the corresponding PR as soon as it is ready


slight adjustment to the landing page

Docs Build status updates of commit 496608a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/index.yml
:white_check_mark:Succeeded
View (qsharp-preview)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix for client.ExecuteWorkflow bug

ExecuteWorkflow failed if Options.ID was empty.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Coverage increased (+0.06%) to 64.749% when pulling cc3a092ade4ca35a4263e823db238d573700f583 on mfateev:id_fix into 824f00639d1c9523a43dd215b476342283d97d15 on uber-go:master.


lucihttp has a bug？

If the textarea has multiple lines and the last line is not a blank line but a single character, the textarea value will not be obtained.
https://github.com/jow-/lucihttp/issues/2
https://github.com/openwrt/luci/issues/2737

Duplicate of https://github.com/jow-/lucihttp/issues/2


fix commit txn check txn status (#40064)

pick https://github.com/apache/doris/pull/40064

run buildall


Enable InputMutator to pick variables from outer scopes when mutating end-of-block instructions

Review comments addressed should be good to go.

Review comments addressed and testcases added, should be good to go here!
Note: I had to change the visibility on randVarInternal for some of the tests


Create Fedora 40 Final Compose

After Fedora has signed off on the release and declared Go! we need to do a final compose switch the branch from devel to stable

PR's changing streams
Ensure the final release deliverables are signed with the correct key
Ensure the release is copied to the staging location for the mirrors

Associated PR's:
https://pagure.io/fedora-iot/ostree/c/c1eb922887dcb3301b73a8d5f058340eaef5c6b1?branch=f40
https://pagure.io/fedora-iot/pungi-iot/c/8c2e1a884f900d91760d3f0b2fc3f1c071cb60e7?branch=f40
Sign the deliverables:
https://pagure.io/releng/issue/12074
Sync'd - https://dl.fedoraproject.org/pub/alt/iot/40/IoT/
Enable updates:
https://pagure.io/fedora-iot/pungi-iot/c/220559ddd960f9f6e6108f11324f0a17e07a2079?branch=f40


Automated cherry pick of #4574: fix: 子网删除忽略弹性网卡

Cherry pick of #4574 on release/2.14.
#4574: fix: 子网删除忽略弹性网卡

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.14@4fc8467). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.14   #4576   +/-   ##
==============================================
  Coverage                ?   6.83%
==============================================
  Files                   ?     522
  Lines                   ?   88859
  Branches                ?       0
==============================================
  Hits                    ?    6076
  Misses                  ?   82276
  Partials                ?     507

Flag
Coverage Δ

#aFlag
6.83% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/networks.go
0.53% <0%> (ø)

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 4fc8467...c119f61. Read the comment docs.

/lgtm

/approve


Termux installation failed

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hey, @MrDavlatJan
Provide more details please, or I won't be able to help you. Moreover, please fill the form above, because I literally can't understand what you want from me. I provide such forms because I want to help with issues and solve the exact problem instead of trying to understand what you meant by this long title. So please, follow these simple community rules and I will be happy to help you. Otherwise I am closing this issue because of the lack of details.
Regards!

Hey, @MrDavlatJan
Provide more details please, or I won't be able to help you. Moreover, please fill the form above, because I literally can't understand what you want from me. I provide such forms because I want to help with issues and solve the exact problem instead of trying to understand what you meant by this long title. So please, follow these simple community rules and I will be happy to help you. Otherwise I am closing this issue because of the lack of details.
Regards!

Can you send me an email to<EMAIL_ADDRESS>, I'll send you screenshots

@MrDavlatJan No, I can't. You can attach screenshots and details here.

@MrDavlatJan No, I can't, sorry. You can attach screenshots and details here.

@MrDavlatJan As far as I understand you need to install cmake. I guess you can do this via pkg install cmake.

@MrDavlatJan It can be, because error says that you are missing cmake installation. Install it and send me screenshot.

@MrDavlatJan It can be, because error says that you are missing cmake installation. Install it and send me screenshot.

I'm still failing sir, i installed cmake but, no luck:(

Then I guess problem is in your environment (Termux). Closing this issue due to that it is not related to Ghost Framework.
I suggest you to contact with Termux support.

Then I guess problem is in your environment (Termux). Closing this issue due to that it is not related to Ghost Framework.
I suggest you to contact with Termux support.

Gotchu, thanks


BaseMultiActionAgent is not exported by "langchain/agents"

As the title suggest, i was trying to import BaseMultiActionAgent but it seems like it's not exported
const { BaseMultiActionAgent } = require("langchain/agents");

Is any reason for that, or it's just an error?

Just an oversight, please open a PR if you're able to!

Nvm I got it


find sum of gp series in python

Why:

Closes #4986
What's being changed:

Merged python file for finding the sum of the gp series.


vscode-server-linux-x64.tar.gz  decompression failed

VSCode Version:
OS Version:

Steps to Reproduce:

wget --tries=1 --connect-timeout=7 --dns-timeout=7 -nv -O vscode-server.tar.gz https://update.code.visualstudio.com/commit:cd9ea6488829f560dc949a8b2fb789f3cdc05f5d/server-linux-x64/stable
tar -zxvf vscode-server.tar.gz

Does this issue occur when all extensions are disabled?: Yes/No

I tried again today and still reported the same error

Is anyone responsible for solving this problem? I cannot connect to my remote server through vscode now.

@ #100586
My problem is solved. Because the storage size of my home path is limited by the system administrator, the storage is full.


Only generate schema on initial request

To improve performance of the middleware the schema generation should only occur on initial request at which point the schema should either be cached to file or more likely memory.

Hi, check #187. I`m start implementing this.
PS: pls, use issue template in future)

As mentioned there is issues with the templates where they are putting them in the wrong project hence the manual creation.
Please reopen this issue as what I am asking for is not addressed by the mentioned issue, I want it to work as it is now except generation only occurs on initial request at which point it is cached.


ROCm 1.6-148: Vega FE power usage relativly high after Ubuntu 16.04 boot

I noticed via a wattmeter that after booting into X server and  i3 window manager my PC power usage is about 95 watt. GPU temperature is about 70-80 degrees Celsius. After executing clinfo (nothing else) power usage drops to ~ 70 watt and temperature to  ~ 50C.
Is it possible to monitor what the GPU is doing? It seems that current radeontop  does not work (only shows 0%).

in \opt\rocm\bin you find ROCm-SMI you can see your temp and power level.  When we run headless
This is on MI25 card,
====================    ROCm System Management Interface    ====================
GPU  DID    Temp     AvgPwr   SCLK     MCLK     Fan      Perf    OverDrive  ECC
3   6860   24.0c    7.0W     852Mhz   167Mhz   0.0%     auto      0%       N/A
1   6860   23.0c    4.0W     852Mhz   167Mhz   0.0%     auto      0%       N/A
4   6860   26.0c    5.0W     852Mhz   167Mhz   0.0%     auto      0%       N/A
2   6860   25.0c    4.0W     852Mhz   167Mhz   0.0%     auto      0%       N/A
GPU[0] 		: PowerPlay not enabled - Cannot get supported clocks
GPU[0] 		: PowerPlay not enabled - Cannot get supported clocks
0   2000   N/A      N/A      N/A      N/A      None%              N/A      N/A
====================           End of ROCm SMI Log          ====================

Ok, thank you! Tried this today after switching my computer on. While temperature was around 50 from the start, power consumption is about 20W and after I execute clinfo drops to 4W.
Could it be that the card is not fully initialized by the driver after booting?
====================    ROCm System Management Interface    ====================
GPU  DID    Temp     AvgPwr   SCLK     MCLK     Fan      Perf    OverDrive  ECC
0   6863   53.0c    20.0W    852Mhz   945Mhz   13.73%   auto      0%       N/A
====================           End of ROCm SMI Log          ====================
chris@beastieboys:~$ clinfo
Number of platforms                               1
Platform Name                                   AMD Accelerated Parallel Processing
Platform Vendor                                 Advanced Micro Devices, Inc.
Platform Version                                OpenCL 2.0 AMD-APP (2450.0)
Platform Profile                                FULL_PROFILE
Platform Extensions                             cl_khr_icd cl_amd_event_callback cl_amd_offline_devices
Platform Extensions function suffix             AMD
Platform Name                                   AMD Accelerated Parallel Processing
Number of devices                                 1
Device Name                                     gfx900
Device Vendor                                   Advanced Micro Devices, Inc.
Device Vendor ID                                0x1002
Device Version                                  OpenCL 1.2
Driver Version                                  1.1 (HSA,LC)
Device OpenCL C Version                         OpenCL C 2.0
..........
====================    ROCm System Management Interface    ====================
GPU  DID    Temp     AvgPwr   SCLK     MCLK     Fan      Perf    OverDrive  ECC
0   6863   52.0c    4.0W     852Mhz   945Mhz   13.73%   auto      0%       N/A
====================           End of ROCm SMI Log          ====================

Can you check the Airflow in your system, looks like you need more CFM across the chip.

Increased the front chassis fan to full speed from silent mode (which sits a little bit below the horizontal Vega card) and managed to get temps down to a minimum of 45C. Looking at
http://www.tomshardware.com/reviews/amd-radeon-vega-frontier-edition-16gb,5128-11.html
45C seems near the minimum of Vega FE in idle mode?
Problem with higher power consumption after boot remains:
================================================================================
 GPU  DID    Temp     AvgPwr   SCLK     MCLK     Fan      Perf    OverDrive  ECC
  0   6863   51.0c    27.0W    852Mhz   945Mhz   20.0%    auto      0%       N/A
================================================================================
====================           End of ROCm SMI Log          ====================
chris@beastieboys:~$ clinfo > /dev/null 2>&1
chris@beastieboys:~$ sudo /opt/rocm/bin/rocm-smi
====================    ROCm System Management Interface    ====================
================================================================================
 GPU  DID    Temp     AvgPwr   SCLK     MCLK     Fan      Perf    OverDrive  ECC
  0   6863   47.0c    7.0W     852Mhz   945Mhz   13.73%   auto      0%       N/A
================================================================================
====================           End of ROCm SMI Log          ====================

I noticed that compared to your MI25 cards my MCLK does not go down to 167Mhz but stays at 945Mhz. Could that be a reason for the higher temp?

When doing hipCaffe cifar10 example temps go up to 95C:
Could it be that CPU throttling is not working correctly?
./examples/cifar10/train_full.sh
====================    ROCm System Management Interface    ====================
================================================================================
GPU  DID    Temp     AvgPwr   SCLK     MCLK     Fan      Perf    OverDrive  ECC
 0   6863   94.0c    195.0W   1528Mhz  945Mhz   40.0%    auto      0%       N/A
================================================================================
====================           End of ROCm SMI Log          ====================

HIPinfo runs a kernel.  I need to dump the voltage cuves on the VBIOS.
Greg

Ok, I have now installed fancontrol package (https://wiki.archlinux.org/index.php/Fan_speed_control)  to manually control fan speed with following /etc/fancontrol config:
# Configuration file generated by pwmconfig, changes will be lost
INTERVAL=5
DEVPATH=hwmon0=devices/pci0000:00/0000:00:01.0/0000:01:00.0/0000:02:00.0/0000:03:00.0
DEVNAME=hwmon0=amdgpu
FCTEMPS=hwmon0/pwm1=hwmon0/temp1_input
FCFANS= hwmon0/pwm1=hwmon0/fan1_input
MINTEMP=hwmon0/pwm1=25
MAXTEMP=hwmon0/pwm1=90
MINSTART=hwmon0/pwm1=20
MINSTOP=hwmon0/pwm1=5

with that I get max. temps of 77C when testing with ./examples/cifar10/train_full.sh (max. 81% fan speed) and during idle about 39C:
====================    ROCm System Management Interface    ====================
================================================================================
 GPU  DID    Temp     AvgPwr   SCLK     MCLK     Fan      Perf    OverDrive  ECC
  0   6863   39.0c    5.0W     852Mhz   945Mhz   27.84%   auto      0%       N/A
================================================================================
====================           End of ROCm SMI Log          ====================

Idle temp. would be probably lower when MCLK scales like GPU frequency or this only possible on MI25 cards?

This is much better
Greg


NEXT Classic USB gamepad not recognizing all the keys

Searched existing issues, may be related to #12696
Godot version:

3.0 RC1 (also happens in alpha 2)
OS/device including version:

Windows 8.1 x64
Issue description:

Gamepad API doesn't seem to handle all the possible key inputs in 3.0. Works well in 2.1.4.
On some controllers, only buttons 0,1,10 and 11 work. Axis are not affected.
Tested with several USB controllers. Bug does not affect all controllers.
(Affects snes and sega megadrive Datafrog imitation and possibly other controllers)
Steps to reproduce:
Testing all the keys in a gamepad affected by the bug.
Minimal reproduction project:

Testing an affected controller in the Joypads demo in Godot 3.0

This has been fixed in https://github.com/gabomdq/SDL_GameControllerDB/commit/8ca95ea6cfcf9e1295c4c91a65fccc060b8ebb3d (my PR got accepted), so it will be fixed in the next SDL_GameControllerDB database sync.

Was this synced?

Not recently AFAIK.


Pixels coordinates rounded

Hi everybody,
I get this message on one of my map :
Projection.LonLat.js:20 Uncaught TypeError: Cannot read property 'lng' of undefined
After long investigation, I found that, in my case, it is due to the combination of 2 plugins : Leaflet.ArrowHead and Leaflet.PolylineOffset :
PolylineOffset has a 'bug' : if a segment is reduced to a point, it returns an empty geometry and then ArrowHead try to compute the angle of the empty segment to create the triangle arrow, which create, at the end, this error message ...
In fact, it is impossible for PolylineOffset to offset a segment that is an point ... (but it could have created an offset segment-point at the same location instead of empty geometry ... to avoid problem)
So I tried to understand why my segment was reduced to a point, and I discovered that latLngToLayerPoint() function use .round()
Why ? Why screen pixels coordinates should be integer ? As a matter of fact, some algorithms need to work on screen coordinates (like ArrowHead) and need floating point précision. For instance, even if there was not the PolylineOffset buf (empty geom), ArrowHead create bad arrow on a segment when the zoom is far because latLngToLayerPoint() transforms the segment to a point ! (and then ArrowHead use a fixed angle of 180° ...)
So my question is : why rounding pixels coordinates ? There is no gain (and it slows down latLngToLayerPoint() a little bit ...) and it can create issues with algorithms/plugins working directly on pixels coordinates.
I tried to "hack" this with :
L.Map.include({
	latLngToLayerPoint: function (latlng) {
		var projectedPoint = this.project(L.latLng(latlng)); // no round()
		return projectedPoint._subtract(this.getPixelOrigin());
	},
});

And now everything is working well.
But I wonder if the fact to not round coordinates, can create another bugs ?

So my question is : why rounding pixels coordinates ?

Because subpixel offsets leads to bugs like https://github.com/Leaflet/Leaflet/pull/6240 https://github.com/Leaflet/Leaflet/issues/6069 and https://github.com/Leaflet/Leaflet/issues/3297


Dense option for Rust

For pokedex.json:
extern crate serde_json;

#[derive(Serialize, Deserialize)]
pub struct Pokedex {
    pokemon: Vec<Pokemon>,
}

#[derive(Serialize, Deserialize)]
pub struct Pokemon {
    id: i64,
    num: String,
    name: String,
    img: String,
    #[serde(rename = "type")]
    purple_type: Vec<String>,
    height: String,
    weight: String,
    candy: String,
    candy_count: Option<i64>,
    egg: Egg,
    spawn_chance: f64,
    avg_spawns: f64,
    spawn_time: String,
    multipliers: Option<Vec<f64>>,
    weaknesses: Vec<Weakness>,
    next_evolution: Option<Vec<Evolution>>,
    prev_evolution: Option<Vec<Evolution>>,
}

#[derive(Serialize, Deserialize)]
pub struct Evolution {
    num: String,
    name: String,
}

#[derive(Serialize, Deserialize)]
pub enum Egg {
    #[serde(rename = "Not in Eggs")]
    NotInEggs,
    #[serde(rename = "Omanyte Candy")]
    OmanyteCandy,
    #[serde(rename = "10 km")]
    The10Km,
    #[serde(rename = "2 km")]
    The2Km,
    #[serde(rename = "5 km")]
    The5Km,
}

#[derive(Serialize, Deserialize)]
pub enum Weakness {
    Bug,
    Dark,
    Dragon,
    Electric,
    Fairy,
    Fighting,
    Fire,
    Flying,
    Ghost,
    Grass,
    Ground,
    Ice,
    Poison,
    Psychic,
    Rock,
    Steel,
    Water,
}

See #516

Would you provide the output that this produces?

@Phrohdoh sample output added

Does this need / do we have tests that can be included in this PR?

@Phrohdoh the test fixture will test the dense option on combinations.json.
Overall, our test harness could use some work in case you're interested in contributing! Right now there is not much clear control over what samples get tested and with what options.


Wait for DOM ready before initialising pre existing elements

There was inconsistent behaviour when this occurred in the  script tag, due to the debounce providing a timeout.
This caused problems in IE (sometimes execution was slow enough that by the time the timeout was complete, the DOM was still not ready.)

It would be a good idea to use DOMContentLoaded. Things to keep in mind that this PR should take into account:

Every call to skate() should still debounce DOM initialisation so consecutive calls should only trigger a single DOM traversal after the last call is made.
When using DOMContentLoaded, it should traverse the DOM once. Calling skate() before it is fired should not traverse the DOM because when it is fired, components will be initialised. Calling skate() after should traverse the DOM because it would have already been fired and would not trigger again.

For number 2, that probably means there will need to be some sort of flag that it can check before it attempts to traverse the DOM.


[improve][doc] highlight important notice

Some important notices are not highlighted, they are buried in the running texts.

We can make it more prominent by using tip, so that they are in green background.

This PR only fixes some occurrences as an example.
Anyone would like to update all occurences? Thank you

I can help with this.

please assigned it to me ~

@azhsmesos thanks! Assigned. Looking forward to your contribution!

收到！
| |
11
|
|
@.***
|
签名由网易邮箱大师定制
On 06/6/2022 @.***> wrote：
@azhsmesos thanks! Assigned. Looking forward to your contribution!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

I'd like to take this up this docs issue if it is available.

I'd like to take up this docs issue if no one is working on it.

@hpa16 thanks! You can ping me to review your PR.

@Anonymitaet I have updated the PR. Can you please assign this issue to me and merge the PR? Thanks!

@hpa16 assigned. Feel free to ping me to merge your PR after all checks pass.

@Anonymitaet the checks have passed, you can merge it now. Thanks for the quick turnaround.


[opencl] Update, add Utility lib

Includes CMake config changes. The vendor-neutral ICD loader is now OpenCL::OpenCL.

Thanks for the update and improvements!


Did you train the model with some datasets like ImageNet-1k?

If you have, could you share the result of the experiments?
Anyway, thank you for your sharing the clear implementation!

Hi @tucan9389
I haven't trained on ImageNet, please refer to this, but I might try to.

i'll be waiting for your result 😁 if i make some result later, i'll share it also. thank you.

Sure, welcome.


why not code

@ErikaLiu  Can you share the code please!?

where's the code?

@ErikaLiu Could you share me your code, thanks!

Sharing the code shows us you did what you proposed in the paper. Otherwise, your work is not true. Kindly, share the code. Thanks.


Pre-commit hook should check for ESLint

Many contributors push code that fails our CircleCI checks because they are not forced to lint their code at any point in the Git process.
We should:

Update Husky to check for linting errors
Create a pre-commit hook for the above

I would be happy to take a crack at this one if it is not yet claimed.

I already made the pr, thanks though @BrianWisniewski

Closed with #165


tc-angular-chartjs x axis labels

Hi,
I there a way to "combine" the x axis labels when it to crowded?
for example when I'm having only 10 days on x axis it look nice but when it's 100 days I can't see anything.
I can see that it's doable on the original chart.js examples, but I'm not sure it doable in the directive.
Please take a look a the example.
https://developers.google.com/chart/interactive/docs/gallery/linechart
screen shot:
http://prntscr.com/77delm

im not sure on this. i think this is more of a chart.js related problem/feature.


Multiline method call with separate line for each param cop

Is your feature request related to a problem? Please describe.
Multiline method call is waiting for a custom cop to be written. Details here - #39
Finished with this style (separate line for each param) - https://github.com/datarockets/datarockets-style/issues/39#issuecomment-511474175
Additional context.
Should be discussed in details. Do we want it to be restricted?

Additional context.
Can have conflicts with other metrics (like class or method size).

@AleksSenkou I'm not going to work on this story in nearest future. Would you like to discuss and resolve it? Or can we set postponed label and return to this issue later?

Set postponed


Move WiFi MAC path prop to vendor namespace

Ever since PRODUCT_COMPATIBLE_PROPERTY was introduced,
vendor binaries are supposed to be using ro.vendor
namespace props.

this needs changes in all device trees *

update it !

if you can freeze current macaddrsetup for <master I'll do it.

that's not that simple since it backtracks for all Android versions

the compat has to be added here


App Request: NAPS2

NAPS2 is a scanner application i use to scan documents and correct them before i export it to a PDF file. You can get the latest version here: https://github.com/cyanfish/naps2/releases

Added


ML 8.19: GAN(D)

Description 📜
Documentation of GAN.
Fixes #7519

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[x] Machine Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

@abhisheks008  Sorry, I am unable to get you, could you elaborate more what changes are to be done. Thank you

As per the recommendation of the GWoC Team, we have created a generalized template for all the contributions (Audio, Video and Documentations) to maintain the professionalism of this event and the contributors. In your case, you have done the documentation of a topic and for that you need to maintain the documentation template, check out here, https://github.com/girlscript/winter-of-contributing/blob/Machine_Learning/Machine_Learning/Demo/ML_%23_%23_Documentation Template_(D).md
Hope you get it!
@Yugal41735

Sorry for pinging u again, but it would be great if u could point out the
errors as I am unable to point out my errors
On Sat, 13 Nov 2021, 18:28 Abhishek Sharma, @.***>
wrote:

As per the recommendation of the GWoC Team, we have created a generalized
template for all the contributions (Audio, Video and Documentations) to
maintain the professionalism of this event and the contributors. In your
case, you have done the documentation of a topic and for that you need to
maintain the documentation template, check out here,
https://github.com/girlscript/winter-of-contributing/blob/Machine_Learning/Machine_Learning/Demo/ML_%23_%23_Documentation Template_(D).md
Hope you get it!
@Yugal41735 https://github.com/Yugal41735
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/pull/7830#issuecomment-968064414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARZOLN4BWZIXED3ALNIEWO3ULZOI7ANCNFSM5H6S7ZTA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@abhisheks008  Sorry for pinging u again, but it would be great if u could point out the
errors as I am unable to point out errors in the documentation

## Title : Name of the Topic
🔴 Type of Content : Documentation
🔴 Domain : Machine Learning
🔴 Module : Name of the module for every week

Add these points before starting your documentation. This is called following any template, there is no rocket science behind it!
@Yugal41735

@abhisheks008 ohh, I dont know because in previous documentation that I made in Ml does not ask me do something like this and I also have seen other documentation in the ml but no one added has added anything that u have mentioned above. Thanku for letting me know.

@Yugal41735 If someone is not following the guidelines, that doesn't mean that you also will not be following the same. The guidelines are guidelines, you've to follow them, otherwise nothing will happen, just your PR will not be merged. As simple as that.

First of all, I haven't said anything like "why should I follow the guidelines".
Secondly, since you said there is no rocket science, that why I told you since I have seen nobody added that.
Third, we learn things from other people's work.
And if any way I have offended you by asking questions, then I am sorry for that.

Fine enough, complete your another assigned issue by EOD @Yugal41735


remove __getitem__ from Mongoengine models

__getitem__ is typed to return Any since we don't have a way to derive overloads based off the class properties.
Instead of __getitem__ users can use getattr if they really want to access properties dynamically.

Fixed by #45. 0.13.0 has this change


Allow hyphens in target names

https://github.com/nuke-build/nuke/issues/270

Hi @Rixium and thanks for diving in. I already did some work around that in https://github.com/nuke-build/common/commit/1efbd012ab0b75a73283fc8519a5eb75e9c3e1c3. As noted in the issue, the global tool also needs some update. I think your changes are just the other part of it that was missing. You can either include those changes in your commit, or I can see if I get to take another look at it. Two test cases that are definitely missing is that for a target InstallService the input INSTALLS gets completed to INSTALLSERVICE and Inst gets completed to Install-Service. While for parameters I prefer all-lower-case, for targets it looks nicer to have them in Pascal-Case, but when skipped the dashes, it must fully omit them in the completion (also a requirement for powershell completion to actually work).

@matkoch I have merged your branch to mine, though there were some issues when I did that.
I believe they are all fixed.
I also changed a few things that may or may not be relevant, but one of the tests were failing:

"NuGetSource" would return "Nu-Get-Source" rather than "Nuget-Source". I don't believe this was intended.

I'm not too sure what you mean by the tests, can you clarify?
Should these be added as test cases to CompletionUtilityTests.cs?

@Rixium sorry for the delay. There are a few things I have to look at / refactor first.


Convert formatting type from .clang-format to Google format with custom modifications

Replaces the .clang-format used by clang-format to a command line
This would give us greater control over the style guide that we are choosing, as we stay as close to it as possible rather than maintaining our own style guide, and at the end, gives us similar style results.
Example execution of clang-format:
clang-format ... --style Google --style "{ColumnLimit: 100}"
Example execution of clang-tidy:
clang-tidy --format-style=Google --style "{ColumnLimit: 100}" ...

Holding on this issue, taking a look at microsoft's .clang-format file


Translation to pt_BR

Hello!
I've been using i3weather with wttr.in. I'm native in pt-br. I could translate it!
Thanks for all the work in wttr
att Luis T

I could help with this translation.

@ltsuemitsu @TupaNegreiros
Thank you guys! Have you checked pt.wttr.in? Is the difference to bt-br big?

Brazilian Portuguese is very different from the original Portuguese. Take a look at https://en.wikipedia.org/wiki/Brazilian_Portuguese
Also, much more people speaks pt-br than pt-pt.

@TupaNegreiros Very interesting; I did not know it; I thought it is some like US-EN vs BR-EN.
Could you please add the translations here?
https://github.com/chubin/wttr.in/tree/master/share/translations
Just make a fork/commit/pull request.
If you need help, just say

Created files pt-br.txt and pt-br-help.txt

@TupaNegreiros   Thank you!
Please check also this:
"Previsão do tempo para:"
"Manhã",  "Meio-dia", "Tarde",    "Noite"

Is this correct?
Please check the live version:
https://pt-br.wttr.in/
If everything is fine, the final step is this:
https://github.com/chubin/wttr.in/blob/master/lib/translations.py
and this:
https://github.com/chubin/wttr.in/blob/master/share/translation.txt

Those texts are correct. Subdomain pt-br.wttr.in is working fine.
Created a pull request with updated translations.py and translation.txt.
I put pt-br as "full translation", is it correct?
Thank you!

I can contribute to this, I've noticed pt-br has a lot of cut out text from the weather prediction due to portuguese being too extensive. What's your idea on this one, shorten text, abbreviations?
The problem lies in text order, since it will list possibility first, but not of what due to language construction.
Example image of today's weather listing a lot of possibilities (rain?)
Also chuveiro something means shower as in literal shower head object.
Correct translation of Light Rain Shower would be something like Pancada de Chuva Leve
I'd love to help!

Hey guys, testing this out and most things are translated but for some reason "Clear" is not? I mean, the correct translation would be "Limpo" but it doesn't seem to be implemented, how can I go about fixing it?

Hey guys, testing this out and most things are translated but for some reason "Clear" is not? I mean, the correct translation would be "Limpo" but it doesn't seem to be implemented, how can I go about fixing it?

Partly Cloudy isn't translating either.


Fix equality comparison of scalars with expressions

cc @mrocklin
This is one potential approach that we could. We would have to remove every other binary op from Expr as well, since, 2 < df.expr is still True.
The alternative would be to raise for bool(Expr), which might make more sense, although we couldn't use Expressions in Sets or as dictionary keys anymore. bool(DaskDataFrame) raises as well

I don't have a strong opinion here yet.  Do you?

No not a strong opinion at this moment.
I think making bool(Expr) raise gives us more stability guarantees in the future but it will be annoying to deal with in the beginning

I'm fine with having bool(expr) raise.  I guess this lets us continue to use binary relational operators.
Are there cases today where we want to use expressions in sets and dicts?  That's probably relevant to help make this decision.

There are cases where we are using them but I think that's not a good idea there. expr._name is a better choice in most cases I think

#307


Update SDK and tinygo dependencies.

Description
Updates the Golang SDK version and the wapc-guest-tinygo dependencies.

I looked briefly into that, there are a couple of things going on. I can help if you want

@jvanz I pushed some small fixes. There are other things that have to be addressed, the whole project has to start using easyjson for its Settings type.
Again, feel free to ping me if you need help

Closing, this is now fixed on main