Replace Markaby?

Hi,
Would it be possible to replace Markaby with Parkaby once Camping V2 is released?
It would be a good way to get more attention and feedback for Parkaby...
BTW, I wouldn't mind if you renamed Parkaby "Markaby 2.0". Since the API is identical, everyone would benefit from the speed improvements, and you would benefit from the name recognition of Markaby. The API similarity is engough to claim the filiation, IMO, even though the underpants are quite different.
Cheers,
--Pygy

Done.


Update sigils

It looks great :) Could you please add tests for the new sigils you've added?

This is not directly related to your branch, but just so you know I've tried to split the sigils into their own module to speed up compilation but it turns out it's not trivial because of all the combinators we have to embed inside the sigil. Splitting it over modules becomes a major undertaking and not simply a matter of changing the place of some lines.

@tmbb I just pushed the optimization. It does move some of the logic to helper module but if you want I can abstract the logic via a MFArgs. This PR takes the time to compile this file from 13.5s to 4.8s.

I will go ahead and merge this and address feedback later because I want to try this on the new branch. :D


don't import spark's dep

/run-all-test

/run-all-tests

/run-all-tests

local ci passed. idc's ci has some problems.


Issues while training on other dataset

@XifengGuo ,
Thank you posting this implementation. I was trying to implement this network on distracted driver detection dataset. This dataset has also 10 classes. I changed the load mnist function and converted the images into grayscale images.  Ehile training, the loss output and the decoder_loss output gives value as nan. Can you please shed some light to why this must be happening?
Thank you in advance.
Regards,
Bhagyashree

@bhagyashreerawl Is the image size in your dataset 28 by 28? Have you scaled them to [0,1]?

@XifengGuo
Yes it is 28 by 28. By scaling them, you mean one hot encoding right?
Yes they are one hot encoded.
Regards,
Bhagyashree

@XifengGuo
I was converting them earlier, but there were some errors. Thank you so much.
I have one more question. Can you please explain the difference between capsnet_loss, loss and decoder_loss and what does capsnet accuracy refers to?
I do have an idea about all this. Just want to get a clearer understanding.
Thank you once again for taking out time to give a response.
Regards,
Bhagyashree

capsnet_loss is the classification loss, i.e., margin loss defined by Eq. (4) in the paper. decoder_loss is the reconstruction loss of autoencoder, and loss is the weighted sum of capsnet_loss and decoder_loss.


Help debugging "Unsupported CLR type" exceptions

I've moved on to testing using a model from an existing project.  This is a code-first model, originally reverse-engineered from the database, if that's relevant.  I was able to successfully run queries if I manually added some fields (ie calling GraphQLTypeBuilder.AddField()), but given that we have a fair few models and they're quite large, this isn't ideal.
However, I've encountered issues running a basic example using AddAllFields():
var schema = GraphQL<EQContext>.CreateDefaultSchema(() => new EQContext());
schema.AddScalar(new { year = 0, month = 0, day = 0 }, ymd => new DateTime(ymd.year, ymd.month, ymd.day));
schema.AddType<EQ>().AddAllFields();
schema.AddField("eq", new { id = "" }, (db, args) => db.EQs.FirstOrDefault(e => e.EQ_ID == args.id));
schema.Complete();

Specifically, ExecuteQuery is throwing GraphQL.Parser.ValidationException: Unsupported CLR type EQ.
The EQ model is mostly string, decimal, and nullable DateTime fields, along with a bunch of references to other models.  Any suggestions for what could be causing this, or how to debug it?
Things I've tried:

Adding recursive fields to models in the test suite (for example, in User I added a Supervisor/DirectReports many-one pair of fields), to test if that was an issue
Adding every other model referenced from the model in question (ie, everything in the context) to the schema
Ensuring that DateTime has been added via schema.AddScalar()

I'll move on now, but I'd like to add that this is a super promising project, and I'm happy to help out if I can.

So, this error is being thrown in TypeHandlers.fs because of a type resolution failure. It definitely seems like an error since you're defining the EQ type, but the source might be hard to pinpoint.
A few suggestions:

Paste the full stack trace of the exception here.
Since it seems like you have downloaded the code, place breakpoints on the this.GetMapping() calls and see if you can trace through them for more information.
Comment out the potentially broken properties in EQ (references to other entities, nullable datetimes) until you can isolate the problem. This will likely still fail if it gets to executing against the database, but that should help identify what is causing the library to break.

Thanks, I'll do a bit of digging.  This is the stack-trace for now:
at GraphQL.Parser.TypeHandlerExtensions.ITypeHandler.VariableTypeOf(ITypeHandler this, Type targetType)
   at <StartupCode$GraphQL-Parser>.$TypeHandlers.fields@328-1.GenerateNext(IEnumerable`1& next)
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.MoveNextImpl()
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.System-Collections-IEnumerator-MoveNext()
   at Microsoft.FSharp.Collections.SeqModule.ToList[T](IEnumerable`1 source)
   at <StartupCode$GraphQL-Parser>.$TypeHandlers.fields@326.Invoke(Unit unitVar)
   at System.Lazy`1.CreateValue()
   at System.Lazy`1.LazyInitValue()
   at System.Lazy`1.get_Value()
   at <StartupCode$GraphQL-Parser>.$TypeHandlers.varType@332.Invoke(Unit unitVar)
   at System.Lazy`1.CreateValue()
   at System.Lazy`1.LazyInitValue()
   at System.Lazy`1.get_Value()
   at GraphQL.Net.VariableTypes.VariableTypeOf(Type clrType)
   at GraphQL.Net.SchemaAdapters.SchemaField..ctor(ISchemaQueryType`1 declaringType, GraphQLField field, Schema schema)
   at GraphQL.Net.SchemaAdapters.SchemaType.<>c__DisplayClass2_0.<.ctor>b__1(GraphQLField f)
   at System.Linq.Enumerable.WhereSelectListIterator`2.MoveNext()
   at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable`1 source, Func`2 keySelector, Func`2 elementSelector, IEqualityComparer`1 comparer)
   at GraphQL.Net.SchemaAdapters.SchemaType.<>c__DisplayClass2_0.<.ctor>b__0()
   at System.Lazy`1.CreateValue()
   at System.Lazy`1.LazyInitValue()
   at System.Lazy`1.get_Value()
   at GraphQL.Net.SchemaAdapters.SchemaType.get_Fields()
   at GraphQL.Parser.SchemaResolver.Resolver`1.ResolveFieldSelection(Field pfield, SourceInfo pos)
   at GraphQL.Parser.SchemaResolver.Resolver`1.ResolveSelection(Selection pselection, SourceInfo pos)
   at GraphQL.Parser.SchemaResolver.ResolveSelections@228.GenerateNext(IEnumerable`1& next)
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.MoveNextImpl()
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.System-Collections-IEnumerator-MoveNext()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at Microsoft.FSharp.Collections.SeqModule.ToArray[T](IEnumerable`1 source)
   at GraphQL.Parser.SchemaResolver.Resolver`1.ResolveSelections(IEnumerable`1 pselections)
   at GraphQL.Parser.SchemaResolver.Resolver`1.ResolveFieldSelection(Field pfield, SourceInfo pos)
   at GraphQL.Parser.SchemaResolver.Resolver`1.ResolveSelection(Selection pselection, SourceInfo pos)
   at GraphQL.Parser.SchemaResolver.ResolveSelections@228.GenerateNext(IEnumerable`1& next)
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.MoveNextImpl()
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.System-Collections-IEnumerator-MoveNext()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at Microsoft.FSharp.Collections.SeqModule.ToArray[T](IEnumerable`1 source)
   at GraphQL.Parser.SchemaResolver.Resolver`1.ResolveSelections(IEnumerable`1 pselections)
   at GraphQL.Parser.SchemaResolver.Resolver`1.ResolveOperation(Operation poperation, SourceInfo pos)
   at GraphQL.Parser.SchemaResolver.ResolveOperations@277.GenerateNext(IEnumerable`1& next)
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.MoveNextImpl()
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.System-Collections-IEnumerator-MoveNext()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at Microsoft.FSharp.Collections.SeqModule.ToArray[T](IEnumerable`1 source)
   at GraphQL.Parser.GraphQLDocument`1.Parse(ISchema`1 schema, String source)
   at GraphQL.Net.GraphQL`1.ExecuteQuery(String queryStr)
   at PMRTests.UnitTest1.testGraphQL() in C:\Users\mark_2\Projects\utas_campus_services\PMR\PMRTests\UnitTest1.cs:line 77

Actually, I've at least got it working (and back down to a pretty minimalist working test again) thanks to the this.GetMapping breakpoint tip.  There was a pretty nasty hack buried in a partial class, that I hadn't noticed before.  I'm pretty sure it's no longer used too, which is encouraging (without going into too many details, it looked something like this: public EQFields _fields { get { return new EQFields(this); } }.  I haven't trawled any further, but that's a definite code smell and I'm not surprised it caused problems)

Given that this issue was a personal question and definitely doesn't look like a library bug, I'll close it off.
If you feel it should have handled it, or want more details just for your own understanding, just reopen and I'll work with you on it.  Thanks for your help.

I ran into the same issue when trying to map a field of type Nullable. Nullable works fine and DateTime works fine but Nullable throws the CLR exception. Do we expect that Nullable typed fields should be mappable?

Hi harrisse,
Sorry for the delay! I've been pretty busy recently - I probably won't be able to take a close look at this until Sunday. I definitely expect that Nullable<DateTime> should work.
Can you provide me with a minimal test case to look at on Sunday? A good place to start would be forking the project and using the in-memory tests at:
https://github.com/ckimes89/graphql-net/blob/master/Tests/InMemoryExecutionTests.cs
and
https://github.com/ckimes89/graphql-net/blob/master/Tests/MemContext.cs

Thanks for the quick response. I'm having a hard time reproducing the issue in your test project, so please don't worry about the issue for now, and I'll let you know if I can get a more reproducible test case.

Hey there @ckimes89,
Love the library!
I know this is a pretty old issue, but I just ran into it and found a way to reproduce it.
As you suggested I modified the MemContext by adding a Nullable DateTime property to User:
    public class User
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public bool Active { get; set; }

        public int AccountId { get; set; }
        public Account Account { get; set; }

        public int? NullRefId { get; set; }
        public NullRef NullRef { get; set; }

	public DateTime? NullableDate { get; set; }
    }

This causes the AddAllFields test to run into this exception:
GraphQL.Parser.ValidationException occurred
  HResult=0x80131500
  Message=Unsupported CLR type ``DateTime''
  Source=FSharp.Core
  StackTrace:
   at Microsoft.FSharp.Core.Operators.Raise[T](Exception exn)
   at GraphQL.Parser.Utilities.invalid[a](String msg) in C:\Users\<User>\Documents\graphql-net\GraphQL.Parser\Utilities.fs:line 46
   at GraphQL.Parser.TypeHandlerExtensions.ITypeHandler.VariableTypeOf(ITypeHandler this, Type targetType) in C:\Users\<User>\Documents\graphql-net\GraphQL.Parser\SchemaTools\TypeHandlers.fs:line 59
   at GraphQL.Parser.NullableTypeHandler.GraphQL-Parser-ITypeHandler-GetMapping(Type ty) in C:\Users\<User>\Documents\graphql-net\GraphQL.Parser\SchemaTools\TypeHandlers.fs:line 162
   at GraphQL.Parser.ChainTypeHandler.GraphQL-Parser-ITypeHandler-GetMapping(Type targetType) in C:\Users\<User>\Documents\graphql-net\GraphQL.Parser\SchemaTools\TypeHandlers.fs:line 52
   at GraphQL.Parser.ChainTypeHandler.GraphQL-Parser-ITypeHandler-GetMapping(Type targetType) in C:\Users\<User>\Documents\graphql-net\GraphQL.Parser\SchemaTools\TypeHandlers.fs:line 50
   at GraphQL.Parser.ChainTypeHandler.GraphQL-Parser-ITypeHandler-GetMapping(Type targetType) in C:\Users\<User>\Documents\graphql-net\GraphQL.Parser\SchemaTools\TypeHandlers.fs:line 50
   at GraphQL.Parser.ChainTypeHandler.GraphQL-Parser-ITypeHandler-GetMapping(Type targetType) in C:\Users\<User>\Documents\graphql-net\GraphQL.Parser\SchemaTools\TypeHandlers.fs:line 50
   at GraphQL.Parser.ChainTypeHandler.GraphQL-Parser-ITypeHandler-GetMapping(Type targetType) in C:\Users\<User>\Documents\graphql-net\GraphQL.Parser\SchemaTools\TypeHandlers.fs:line 50
   at GraphQL.Parser.ChainTypeHandler.GraphQL-Parser-ITypeHandler-GetMapping(Type targetType) in C:\Users\<User>\Documents\graphql-net\GraphQL.Parser\SchemaTools\TypeHandlers.fs:line 50
   at GraphQL.Parser.RootTypeHandler.GraphQL-Parser-ITypeHandler-GetMapping(Type targetType) in C:\Users\<User>\Documents\graphql-net\GraphQL.Parser\SchemaTools\TypeHandlers.fs:line 512
   at GraphQL.Net.VariableTypes.VariableTypeOf(Type clrType) in C:\Users\<User>\Documents\graphql-net\GraphQL.Net\VariableTypes.cs:line 50
   at GraphQL.Net.SchemaAdapters.SchemaField..ctor(ISchemaQueryType`1 declaringType, GraphQLField field, Schema schema) in C:\Users\<User>\Documents\graphql-net\GraphQL.Net\SchemaAdapters\SchemaField.cs:line 21
   at GraphQL.Net.SchemaAdapters.SchemaType.<>c__DisplayClass2_0.<.ctor>b__1(GraphQLField f) in C:\Users\<User>\Documents\graphql-net\GraphQL.Net\SchemaAdapters\SchemaType.cs:line 18
   at System.Linq.Enumerable.WhereSelectListIterator`2.MoveNext()
   at System.Linq.Enumerable.Aggregate[TSource,TAccumulate](IEnumerable`1 source, TAccumulate seed, Func`3 func)
   at GraphQL.Net.SchemaAdapters.SchemaType.<>c__DisplayClass2_0.<.ctor>b__0() in C:\Users\<User>\Documents\graphql-net\GraphQL.Net\SchemaAdapters\SchemaType.cs:line 17
   at System.Lazy`1.CreateValue()

I have not yet written any code in F# so debugging this is quite tough for me.
Hope its enough for you to go on :)


a couple suggestions

let me first say this is one of the best free leveling bots i have come across. very simple and does the job without error.
im unsure if this could be accomplished due to limitations, but in item shop, it can only purchase the complete item, not smaller items under the completed item's tree. other bots have struggled with this. is this something that could be improved on or is it a limitation of the code?
i had to change the scale of my hud and minimap. i didnt see it in the readme, but i was under the assumption that it could operate without having the change these settings. was it supposed to work as intended without changing hud settings or are there specific instructions on what the hud settings should be set to? thanks

You can make the bot buy any items that you would like. On line 130 in game.py is where the bot attempts to purchase items. You can take the ratio_builder.py script and create your own custom ratios for the bot to attempt to click on in game.
To your second point, I did not even think about hud settings, but all the scaling/settings in game should be the default settings/sizes because the intent of this bot is to level new accounts. I will look into updating the readme to make that clearer.


2/4 move @graphprotocol/nextra-theme to @graphprotocol/docs

Sweet!
Just one thing, and maybe it's just me, but should pages go inside src? Seems a bit strange to me to have them as siblings.

yes, better to put them under src

@benface done here https://github.com/graphprotocol/docs/pull/734


Make logging each received and sent message optional

To speed up sending and receiving, you can now disable logging. Error messages are still printed.
Sending 10k+ packets per node, and printing each one, adds up quickly and results in huge delays in the network.
Just so you know, I am not a go expert, I didn't find an easy way to disable the printing without modifying the module.

Thank you for the recommendation.
I will think about a better way to handle this. The logging mechanism in Go is simplistic.
You can disable logging without changing the module by adding the line
`                log.SetOutput(io.Discard)
My suggestion is more aggressive in that it disables all logging. Your solution adds additional API. I would still want to keep the ability to enable/disable logging by log level.
For now, finish the assignment. Have a nice weekend.


Response data is stripped from certain status codes

I've skimmed through the server code and don't see anything that might be causing this, and when debug is enable the console reports that the response data is there, so I'm thoroughly stumped about the problem.
The example below should take a status code in the request body and echo it back to you using that status code for the response.  For some status codes (e.g. 409 conflict) Postman reports that it gets nothing in the response body, and it's missing all of the headers that should normally be there (Access-Control-Allow-Origin, Content-Length, and Content-Type).
internal class Program
{
    public static void Main(string[] args)
    {
        var server = new Server("localhost", 8080, false, Handler, true);

        Console.ReadKey();
        server.Dispose();
    }

    private static HttpResponse Handler(HttpRequest request)
    {
        var bodyData = request.Data ?? new byte[0];
        var bodyString = Encoding.UTF8.GetString(bodyData);
        int statusCode;
        if (!int.TryParse(bodyString, out statusCode))
            statusCode = 404;

        return new HttpResponse(request, true, statusCode, null, "text/plain", bodyString, true);
    }
}

Hi @mbcrawfo  it looks like this is the behavior anytime a status code is used that isn't accounted for in BuildErrorResponse and SendResponse.  I tried with 200, 201, 400, 404, etc, and they returned the correct headers.  Can you confirm that is the case?  If so, I can add 409 an any other status codes you wish to use that I may have missed.  Thanks!

Hi @mbcrawfo I just committed a fix and also updated NuGet (see https://github.com/jchristn/WatsonWebserver/commit/aeb708198b5c4ef92e97d1c184fe41f72a7adc36) - could you re-open if this doesn't work as expected for you?  Thanks!


In cookiecutter templates, get setting values automatically from .env

Migrated from GitLab: https://gitlab.com/meltano/sdk/-/issues/358
Originally created by @aaronsteers on 2022-04-02 07:54:29

The common convention and best practice today is to use .env files to store sensitive and nonsensitive env vars. The python-dotenv package makes parsing .env context trivial and we should consider adding this to the standard cookiecutter templates.

View 3 previous comments from the original issue on GitLab


chore(Blog): update “2021-10-06-hébergement”

Automatically generated by Netlify CMS

👷 Deploy Preview for lucileetguillaume processing.
🔨 Explore the source changes: 8b0cf0e5dfdcfdd59f794fcd319c6bc21e9e4ca5
🔍 Inspect the deploy log: https://app.netlify.com/sites/lucileetguillaume/deploys/619a5ae57b1b3a0008bd30ac


Detect segment unavailability for queries in broker

Description
Broker maintains a timeline of segments which it builds overtime upon receiving updates from historical server and it uses this timeline to answer queries. Broker isn’t aware of what segments actually exists in the druid system. The result of this gap is incomplete query responses on some occasions.
With this feature the goal is to ensure, if a segment was queryable at one point in time, any future query over that segment would either include that segment or fail.
Design
Broker polls the coordinator periodically to get all the used segments in the system, it merges all the segments that have once been loaded by a historical server into its timeline of segments. This timeline now consists of segments which are available on some historical server and which aren’t available on any server, this information helps the broker identify unavailable segments for the query.
This approach also ensures that any segment which has just been published but not loaded by any historical server doesn’t cause query failure.
Following synchronisation conditions could cause temporary query failure,

If the broker isn’t able to sync its timeline with the coordinator, this would cause broker to be unaware of recently removed segments from the historical

If the broker is behind historical server, sync with coordinator makes it aware of recently loaded segments but the broker would think that they are unavailable

Major changes
Coordinator changes

Add a new column handed_off & handed_off_time in the druid_segments metadata table to represent if a segment has ever been loaded on a historical and the first load time(changes in SQLMetadataConnector)

When the coordinator is notified that a segment has been loaded, set the handed_off column to true

Update DataSourcesSnapshot to maintain diff of the segments from the previous poll.

Add coordinator API  MetadataResource#getChangedSegments to send either full snapshot or delta changes to the broker using the information present in DataSourcesSnapshot

Changed classes: CoordinatorServerView, SqlSegmentsMetadataManager, SqlSegmentsMetadataQuery, MetadataResource, DataSourcesSnapshot

Broker changes

MetadataSegmentView polls the coordinator to fetch the list of all used segments along with their overshadowed and handedOff status, on the very first poll it receives a full snapshot thereafter it receives delta updates.

After the finish of every poll, notify BrokerServerView to update its timeline with all segments that have been handed off

Remove segments that are not used anymore i.e. segments that are not present in the list polled from the coordinator
Add segments that are used and handedOff to the timeline, if they don’t already exist

While handling a query on the broker, lookup the segments required for the query from the timeline. If any of these segments is unavailable, throw an error.

Changed classes: CachingClusteredClient, BrokerServerView, MetadataSegmentView

Upgrade considerations
Usage

druid.sql.planner.detectUnavailbleSegments needs to be set in broker runtime properties
unavailableSegmentsAction query context can be set to allow or fail, accordingly the queries would fail, in either case the unavailable segments will be logged.

Release note

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[x] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

How will this design work when there are used segments but have zero replication factor?

How will this design work when there are used segments but have zero replication factor?

If a segment is created with replication factor of 0, it will never be loaded onto any historical. Consequently, the segment wouldn't be marked as loaded and will not be added to the Broker's timeline. This behaviour is as expected.
If a segment's replication factor is changed to 0 after it has been loaded, the segment will remain in the Broker's timeline with queryable status, resulting in queries finding the segment to be unavailable.
This problem can be fixed by detecting the replication factor change in the Coordinator and thereafter send a change event to the Broker to remove the segment metadata from Broker's timeline.
Now consider the scenario where the segment's replication factor changes to non-zero, and the Broker receives this updated metadata from the Coordinator before the segment is reported as loaded by any Historical, queries in this period will find this segment to be unavailable. Is this behaviour correct? Or should the loaded field of the segment be dynamically adjusted when it is marked as cold?


Do not crash on empty GeoJSON feature collection

Looks fine to me. Go ahead and merge

Thanks for the review, Jesse.


Merge hotfix branch 'hotfix/identity-ibcpatch' back to master

Interesting that it shows my commit in the list.


[BUG] index tag set to the EMRS job run is not correct

What is the bug?
index tag set to the EMRS job run does not include datasource name and schema when those are not explicitly specified in the query.
Example query: CREATE MATERIALIZED VIEW mv1 AS ....
In this case, index tag should be flint_{datasourceName}_{schema}_mv1, but it is set to flint_mv1
How can one reproduce the bug?
Steps to reproduce the behavior:

start async-query with CREATE MATERIALIZED VIEW mv1 AS SELECT ...
check EMR job run

What is the expected behavior?
index tag should look like flint_{datasourceName}_{schema}_mv1
Do you have any screenshots?
If applicable, add screenshots to help explain your problem.
Do you have any additional context?
Add any other context about the problem.

Flint grammar allows this because we can infer the current catalog and database in Spark. Probably we should consider validating and disallowing this in SQL plugin.

Flint grammar allows this because we can infer the current catalog and database in Spark. Probably we should consider validating and disallowing this in SQL plugin.

+1, since Plugin doesn't have the catalog info exposure

expectation is the use full qualified names for create including a DB name


static gas resistance on sensor readings

Not sure if this is the right place for reporting this issue: I'm using BME688 on an Adafruit breakout with the proprietary arduino <bsec.h> library. Temperature, humidity an pressure readings work fine. Only the gas resistance is constant in every reading (12946860 Ohms), and subsequently all other derived/calculated measures (IAQ, CO2 equivalent, breath VOC equivalent) are hanging at their lowes values.
BSEC library version <IP_ADDRESS>
Timestamp [ms], raw temperature [°C], pressure [hPa], raw relative humidity [%], gas [Ohm], IAQ, IAQ accuracy, temperature [°C], relative humidity [%], Static IAQ, CO2 equivalent, breath VOC equivalent
3496, 23.67, 102518.00, 31.25, 12946860.00, 25.00, 0, 23.67, 31.25, 25.00, 500.00, 0.50
6496, 23.68, 102517.00, 31.18, 12946860.00, 25.00, 0, 23.62, 31.29, 25.00, 500.00, 0.50
9496, 23.72, 102517.00, 31.09, 12946860.00, 25.00, 0, 23.66, 31.14, 25.00, 500.00, 0.50
12496, 23.72, 102515.00, 31.00, 12946860.00, 25.00, 0, 23.66, 31.07, 25.00, 500.00, 0.50
15495, 23.73, 102517.00, 30.93, 12946860.00, 25.00, 0, 23.67, 30.99, 25.00, 500.00, 0.50
18495, 23.73, 102517.00, 30.88, 12946860.00, 25.00, 0, 23.67, 30.96, 25.00, 500.00, 0.50

I tested with <Adafruit_BME680.h> library, which works fine and outputs reasonable and changing (slowly increasing) gas resistance values. Am I missing something? Configuration of the heater...?
I used the example script from the Arduino library.
https://github.com/boschsensortec/BSEC-Arduino-library
This repository was not updated since 3 years.
Thank you.

Hi @ratio-x , you are probably using a BME688 with a BME680 library. For the core sensor library, it is recommended to update to Bosch-BME68x-Library and for BSEC, Bosch-BSEC2-Library


Fixing inconsistent SaaS connector integration tests

Purpose
Fixing inconsistencies with the SaaS connector tests for the following connectors:

Hubspot
Segment
Sentry
Stripe

Changes

Fixing bug with filter_results that occurred for row data that is returned from SaaS API responses but not annotated in the dataset
Waiting until data is available for Hubspot, Segment, and Sentry
Updated Stripe access tests since an active subscription on the access identity added a new charge and payment_intent. Cancelled the subscription within Stripe to have consistent data moving forward.

Checklist

[ ] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Good unit test/integration test coverage
[ ] This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
[ ] The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket
Fixes #472

Thanks @galvana these changes are super useful


Catch decimal exception and log as error

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 6 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 89.337%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

custom_components/powercalc/sensors/power.py
3
6
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
2061

Relevant Lines:
2307

💛  - Coveralls


An allocator for this device has already been registered for sharing

This is seen during testing on iOS. Log link: https://github.com/microsoft/onnxruntime-genai/actions/runs/11248015928/job/31272436617
The line that throws exception in ORT is:
https://github.com/microsoft/onnxruntime/blob/c415991c16b02941f678d77e24d75f4837ee87bc/onnxruntime/core/session/environment.cc#L105
It seems to be caused by calling CreateAndRegisterAllocator multiple times.

We only create the globals once. I am not sure how the CreateAndRegisterAllocator is called multiple times. Is there anything different in iOS compared to other platforms?
@RyanUnderhill any ideas how the OrtGlobals constructor could be called multiple times? Is it somehow related to our google test setup?

It is indeed different on iOS, because an adapter named xcgtest.mm is needed to make gtest work in XCTest:
https://github.com/microsoft/onnxruntime-genai/pull/964/files
This changes the init behavior and now OrtGlobals will be created multiple times for each test suite.

Sounds like there's something weird going on with the test wrapper, since globals should only be initialized once.
A single test works fine, it's just that it fails on the second test when it creates the globals a second time and overwrites itself?

@RyanUnderhill yes. To be precise, the static initializer here gets called for the second time:
std::unique_ptr<OrtGlobals>&
GetOrtGlobals() {
  static auto globals = std::make_unique<OrtGlobals>();
  static auto validate = std::make_unique<ValidateShutdown>();  // Must be after the above line so the destructor runs before the above destructor
  return globals;
}

ORT are not happy with it.

if you need global setup and teardown logic for the tests, gtest has a hook for that. I recently updated ORT's iOS unit tests to use it. previously they weren't cleaning things up properly.
https://github.com/microsoft/onnxruntime/blob/a732f7a4b3bcfc33233c50577a7d75d0eddb2dbf/onnxruntime/test/xctest/xcgtest.mm#L98-L110

Thanks @edgchen1 ! I'll give it a try

Sadly due to the lack global initializer in GenAI, I can't balance the calls and bypass the allocator issue. I took a different route and implement the tests in Objective-C, which turns out to be working. I'll close this for now.


Guaranteed Minimum (All Patterns) -- math error

The guaranteed minimum for "All Patterns" (the last row) appears to be taking the maximum of the minimums for the "All Patterns" row. However, it seems to me that the guaranteed minimum should actually be the minimum of the guaranteed minimums for all rows above it.
See below. The guaranteed minimum should be 128 rather than 78.

I have seen this in my own results as well. I agree with your assessment, but I haven't had a chance to look into a fix yet.

This is now fixed :)


Update Gateway API to v1.0.0

Proposed changes
Problems:

NGF does not support v1.0.0 of Gateway API.
NGF does not use the official PartiallyInvalid condition from Gateway API SDK.
Webhook should only be installed when Kubernetes version is <1.25

Solutions:

Update to v1.0.0 of Gateway API
Replace custom PartiallyInvalid condition with the Gateway API condition
Add logic to pipeline and conformance scripts to conditionally install webhook.

Testing:

Ran conformance tests. One expected failure
Ran through examples
Verified install and uninstall instructions

Closes #1173 #1183 #1168
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked that all unit tests pass after adding my changes
[x] I have updated necessary documentation
[x] I have rebased my branch onto main
[x] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

@ADubhlaoich For your awareness, this PR changes some docs that will have to be synced with https://github.com/nginxinc/nginx-gateway-fabric/pull/1226 once merged.

Do we need to specifically disable failed conformance test?

Do we need to specifically disable failed conformance test?

I would assume so, but merging is blocked for other reasons right now, so I can't tell.

@travisamartin  For your awareness, this PR changes some docs that will have to be synced with https://github.com/nginxinc/nginx-gateway-fabric/pull/1226.


Update eslint-plugin-import to version 2.0.0

According to this PR, this new version breaks the build (see build status 755 too). We need to understand why it fails, and how to fix it.

I can take a look into this

Looks like with the update there were missing dependencies, which I have taken care of in PR, problem i'm now seeing is it is linting but its saying previously ok imports are being flagged as bad. Will dig deeper into that issue build status 759

It looks like eslint-config-airbnb is set to<EMAIL_ADDRESS>in its peer dependencies. I'm not entirely familiar with how npm handles version differences between dependencies. Do you know if this can only be resolved once that library has updated its version as well?

Do you know if this can only be resolved once that library has updated its version as well?

Yes, I think so..


Proposal: Stackable objects of same type in the same SnapDropZone

Overview
I'm proposing the addition of new functionality to VRTK_SnapDropZone to accommodate the stacking of identical VRTK_InteractableObject based items.
This will be useful for me in particular in building an RPG style inventory, but I can imagine it being useful in other contexts too (eg. tracking grenade count in an FPS chest-mounted inventory a la Onward)
Code In Progress

Made rough working changes to VRTK_SnapDropZone to tally the number of objects being stacked
Added a (very) rough test to determine whether an interactable object is the same kind already snapped to an SDZ.
Tweaked VRTK_InteractableObject to add a new "stackable" property

Demo

For discussion

Is this functionality generalized enough to merit inclusion into VRTK?
What is the best way to perform interactable object type comparison (tags, subclass comparison, id variables?)
Should all of the proposed functionality, like the item counter, be added to VRTK_SnapDropZone or should it be added to a subclassed prefab instead?
What are the implications for communication between SDZs. Users might want to prevent multiple stacks of the same object, meaning that an SDZ would have to check in with each of the other SDZs in its group to see if they should receive the interactable object instead. Perhaps a VRTK_SnapDropZoneGroup component is worth considering.

I quite like the idea of having a snap drop zone allow multiple items of the same type to be snapped to it and have a counter for them.
I don't think you'd need to extend the interactable object, but also need to be aware that a snap drop zone can have any number of items snapped to it so you could get into a situation where you snap a cube then snap a sphere and in that case the counter shouldn't go up? Perhaps you can only have multi item snapping if the same item is snapped to the drop zone?

That's what I was thinking too. I currently have a rudimentary, easily broken test to determine item type and prevent stacking one type of item on top of another. I've been considering how to improve the test though. In my game, I need to include a large number of items, and I plan on subclassing VRTK_InteractableObject  for each one (potions, weapons, other consumables) in order to manage shared behaviors and differentiate use behaviors. At least in my case, I'm leaning towards adding an "item type" variable to VRTK_InteractableObject that can either be modified in the inspector, or (in my case) hard-coded for each subclassed item. An "item variant" variable might be appropriate too (eg to differentiate between blue potions and purple potions even though they're both potions). Thoughts?

PR with fix: #1547

Moscartong has the inventory basics here: https://github.com/moscartong/VR-Inventory

How's the PR going? Would this feature be merged into the next version?

@moscartong It has merge conflicts and wasn't updated yet to do the requested changes. You can see that on the PR page.

Also, 3.3.0 is now pretty much version locked to only include the new example scenes and the windows mixed reality native support. Anything else is just going to slow down it's release.

@thestonefox Understood. Good news that MS MR support is finally here 👍.
I'll do some workaround on this matter for now. I recently try to use two snap drop zones together to form a super snap drop zone that, in effect, could hold multiple items with the same type, without changing a single line of VRTK code. Could take a look at it if you guys are interested :)
https://github.com/moscartong/Snaaap-Drop-Zone


refac: improve #424

Description
This PR improves #424 in several aspects.

@chokobole For 9034bd3, there is a missing change in tachyon/cc/math/finite_fields/prime_field_unittest.cc!

@chokobole For 9034bd3, there is a missing change in tachyon/cc/math/finite_fields/prime_field_unittest.cc!

In fact, they are required!

Please fix typo in f23741b. DivMode -> DivMod.


##[error]tunneling socket could not be established, statusCode=407

Description:
Were trying to set up a pipeline on a runner behind a corporate proxy. Github Enterprise is running  in the cloud. The Runner has a proxy configured and other actions are running (e.g. actions/checkout) fine. When we try to use the this action, we get the following output:
 Set up job
3s
Download action repository 'actions/setup-java@v2'
Current runner version: '2.272.0'
Runner name: 'si0vmc1985'
Machine name: 'si0vmc1985'
Runner is running behind proxy server 'http://<IP_ADDRESS>:3128/' for all HTTP requests.
Runner is running behind proxy server 'http://<IP_ADDRESS>:3128/' for all HTTPS requests.
Prepare workflow directory
Prepare all required actions
Getting action download info
Download action repository 'actions/checkout@v2'
Download action repository 'actions/setup-java@v2'
Run actions/checkout@v2
1s
Resolved version xxxxxxxxxxxxxxxxxxxxxxxxx
Run actions/checkout@v2
Syncing repository: xxxxxxxxxxxxxxxxxxx
Getting Git version info
Deleting the contents of '/local/home/github-runner/_work/xxxxxxxxxxxxxxxxxxxxxxxxx'
The repository will be downloaded using the GitHub REST API
To create a local Git repository instead, add Git 2.18 or higher to the PATH
Downloading the archive
Writing archive to disk
Extracting the archive
/bin/tar xz -C /local/home/github-runner/_work/xxxxxxxxxxxxxxxxx/62f9ecc6-ab19-4f24-934d-821f9d499392 -f /local/home/github-runner/_work/xxxxxxxxxxxxxxxxxxxx/62f9ecc6-ab19-4f24-934d-821f9d499392.tar.gz
Resolved version xxxxxxxxxxxxxxxxxxxxxxxx
Set up JDK 11
1s
    overwrite-settings: true
Run actions/setup-java@v2
  with:
    java-version: 11
    distribution: adopt
    java-package: jdk
    architecture: x64
    check-latest: false
    server-id: github
    server-username: GITHUB_ACTOR
    server-password: GITHUB_TOKEN
    overwrite-settings: true
Trying to resolve the latest version from remote
(node:30582) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
##[error]tunneling socket could not be established, statusCode=407

The strange thing is, that the configured proxy on that VM is a cntlm that handles the proxy authentication against the corporate proxy and provides a proxy interface  for all apps on the same host without proxy authentication required.
My gutfeeling is that the corporate proxy might block something (the error message tells something like a tunneling, which is something our proxy might block.
Task version:
v2
Platform:

[x] Ubuntu
[ ] macOS
[ ] Windows

Runner type:

[ ] Hosted
[x] Self-hosted

Hello @jensbaitingerbosch. Thank you for your report. Sorry for the long time response. It looks like setup-java can't be used with proxy because of the http-client. Checkout step passes because it does not use http-client. As I know http-client has extra configuration to pass proxy that is why we can track your issue as feature request to add logic for the action to pass proxy.

Hello @jensbaitingerbosch. I'm closing the issue with the description above. Feel free to create new issue as a feature request.


Current Pipeline Running information

fixes #991
Summary of the user interface changes:
https://user-images.githubusercontent.com/61746913/153610323-298fb684-c170-4da7-a547-c486112f3ad2.mp4

The Data Browser won't allow pipelines to be run when you have selected a dataset which currently has a running a pipeline
Within the Annotator when a pipeline is running on the current dataset the system switches to read-only mode and prevents edits and indicates to the user that a pipeline is running
While the pipeline is running in the pipeline dropdown there is a link to currently running pipeline job data
Read-only mode has been updated to disable various UI controls and shortcuts (Adding tracks, deleting tracks, editing tracks) and all of these items
Read Only Mode has it's own indicator below the Dataset Title with some simple hover information
The Dataset Title has been fixed to properly truncate if it is longer than the area provided.
When pipelines finish and you are in the annotator for desktop it will now tell you to reload the dataset

Implementation:

Added to provides useReadOnlyMode which provides a simple boolean return value as to if ReadOnly Mode is Active.  This involved adding this item to a lot of data places to disable specific items.  Lots of small code additions.
Attempting to enter edit mode with right-click will now launch a prompt explaining that you can't edit in read only mode.
ViewerLoader for both web and desktop has been modified to kick a Viewer into readOnlyMode while a pipeline is running.
Web Vuex Jobs has been modified to store some more information including completed jobs and the job ID for returning a link to the proper job log.
Within Web ViewerLoader it watches a getter for completed jobs and will trigger a reload if the current dataset job completes.  It then needs to call an action to remove that job from the joblist so it doesn't trigger any secondary reloads.

Looks like this branch needs a rebase, it's replaying commits from main.

Entering editing mode still works for existing annotations when you right click. You get the dialog, but then you can dismiss it and make changes.  I think it would be better to just completely block the transition into editing mode.
When pipelines fail, you still get the "Pipeline success" message.  It might be out of scope for this PR, but I think we need to tell users their pipe failed.  Is this hard to do?

I'm having difficulty replicating that top one where you can edit after the dialog comes up.
The second one seems doable, just that the implementations will have be different for desktop/web.  Web has the girder job status which I believe I was only using success before, I can easily handle error.  For desktop I might need to redo some more stuff to get it to work.
I can work on those soon.

Wasn't that bad to adding the capability to have the failing conditions.  I grouped cancelled/error as Pipeline Incomplete.  Let me know if you want the wording changed.
I could replicate the dismissal of the dialog when right clicking to get it into editing mode.  Maybe one of my other changes fixed that problem.


[Test generator]: Various improvements + adding specs

What has changed:

New File structure
Tests added
Formatter restructure to not write files, instead implemented an error system to have the same feel as before

The formatter changes has been reverted due to performance concerns


worker_vm_type not respected when providing both vm type and worker_memory

Currently, in the conftest.py, we are creating a cluster that specifies both worker_vm_type and worker_memory,
https://github.com/coiled/coiled-runtime/blob/d53473e6899968d5fb5d1837fa62d18828e7ac72/conftest.py#L74-L89
and this is causing us not to get the instances that we requested for the workers as it seems coiled infers the worker instances based on worker_memory.

I filed the respective issue on coiled about this, as we should get an error or a warning mentioning this. But in the meantime I suggest we remove worker_memory in the Cluster()  call.

Looking at the pricing I see that t3.large are cheaper so maybe we want both the scheduler and the workers to use t3.large . Is there a reason why we would go for m5.large?
m5.large | 2 | 8 GiB | EBS only | $0.10 -> on demand hourly price
t3.large | 2 | 8 GiB | EBS only | $0.08  -> on demand hourly price
Link to pricing aws page: https://www.ec2pricing.net/
cc: @jrbourbeau @hayesgb @ntabris

Nice catch @ncclementi!

I see that t3.large are cheaper so maybe we want both the scheduler and the workers to use t3.large . Is there a reason why we would go for m5.large?

It looks like there's not a huge price difference between these two instance types. Do you know what the default is that Coiled uses?

There is not a huge difference in price, but we are adding more and more tests, and cluster creations, so it wouldn't hurt going for the cheaper one unless there is a good reason not to.
I just launched a cluster without specifying any instance type or worker memory and it looks like the default type is a
t3.medium | 2 | 4 GiB | EBS only | $0.04

Gotcha, thanks for checking. If t3.large or m5.large was the default, then it would be good motivation for choosing the default instance type. But since t3.medium is the default, it's less clear. Maybe we should just go with t3.large since, as you mentioned, it's cheaper. I don't know what other factors differentiate t3.large vs m5.large

Closing this, as https://github.com/coiled/coiled-runtime/pull/159 was merged, not sure why the autoclosing didn't work.


Should not change the line-height when smaller than font-size

It should not change the line-height of the parent container when the computed value is smaller than the computed font-size value.
It seems to me that it does change it but MAYBE i'm wrong (because the code of the projet i'am working on is a little messy and complicated, not my code)

I have same problem


Add a workflow to an add-in Step 8 issue!!!

There is a step which is Drag the two new list instances into the Lists folder. My workflow cannot get the email because of this step! I followed this step to move two workflow list into list folder, but my add-in did not send the email! Later, I moved those list back to initial position. It works!!!

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

ID: 7b8578d8-4703-1ac5-ae2e-f2ebbd85cdbb
Version Independent ID: 075145ec-e01c-00bf-358d-4604fa25daad
Content: Add a workflow to a SharePoint-hosted SharePoint Add-in
Content Source: docs/sp-add-ins/add-a-workflow-to-a-sharepoint-hosted-sharepoint-add-in.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Do you have do some special custom. It is better you can provide the reproduce steps on your side and the configure info such as SharePoint list and SharePoint Framework package.


Updated QueryManager::fetchQuery to force it happening asynchronously, thus fixing #956.

This is a proposal to fix #956. I have to say that this was my first time near TypeScript, and I guess this is much more a proof of concept for the solution than anything else. Hope it helps, though.

@lucasconstantino Thanks for the PR! For modifications to the core of how Apollo Client works, I think it's good to make a design proposal first and discuss it before implementing it.
Since there is already a scheduler that batches queries by certain intervals (sometimes as little as one tick), I think any deduplicating logic should either happen in there, or in the network interface. I'm going to close the PR for the time being until we've had a design discussion. I encourage you to start the discussion with your idea in #956. 🙁


[BUILD] Manually running typed css module loader

Running typed css module loader ahead of the build ensures all the typed styles are present before running the build; thereby ensuring the success of first build itself.

Closing in favour of https://github.com/GoogleChromeLabs/squoosh/pull/350.
@abhijit-paul thanks for taking a swing at this!


Make sure res.send does not override res.statusCode

Consider this piece of code:
service.get('/route/', async (req,res)=> {
     res.statusCode = 250; // random 200-range non-existent code
     res.send('something')
})

Unfortunately, res.send will override the status code and use 200 as a default. This may be a problem with legacy applications which do not pass the status code directly to res.send
While this is not a critical issue per se, since res.send could always be overridden through middleware, I think it is more correct to do this in restana package itself, especially since res.statusCode is 200 by default anyways

Thanks @schamberg97 for catching this issue and also providing a fix. 👏


Domain assertion of observable:displayName is too restrictive

observable:displayName is a property of the facets DigitalAccount, EmailAddress, and WindowsService.  A domain assertion in today's develop head unnecessarily restricts displayName to belonging to WindowsService.
The domain assertion should either be expanded or removed.

I stumbled on this while aligning the CASE example accounts.json with UCO 0.3.0. This is also true for today's state of develop.

This issue will be addressed through implementation of
JIRA OC-68: https://unifiedcyberontology.atlassian.net/browse/OC-68
and CP-13: https://unifiedcyberontology.atlassian.net/wiki/spaces/OC/pages/984678401/UCO+CP-23+Convert+current+property+restrictions+and+domain+assertions+to+SHACL+shapes


does s4cmd accept input from stdout?

Hi
I tried something similar, but not seem working, example
echo "my file"| s4cmd put - s3://mybucket/s4cmd-test/testfile.txt &
it still expects proper source target in this case.

It looks like s4cmd can't do it = (. Just because of this, I still use s3cmd.


Toggle buttons

I am wondering if there is anything built-in to create toggle or group buttons. If not, would be a great addition.
Material design specs

This is something that would definitely be cool to add. There's still a lot on my plate right now and its holiday season for me so I would be open to a Pull Request if anyone wants to take a crack at styling this with checkboxes or radio buttons.
Otherwise I can look into this when I can.

While this would be nice to have, not sure if it would be widely used.


Fix CPU/GPU/SPU errors caused by null entity

Fixes #1994.
Fixes errors caused by a CPU/GPU/SPU being deleted before its queued upload/cache resend/model reset/sound source init happens.

This looks good at first glance. I'd like to look a bit closer and make sure there's not supposed to be some cleanup that needs to happen.

I think it should be fine. Could you change those disgusting timer.Create to timer.Simple though?

Done.

Thanks!


Tracking: Hive pyspec failures

We now run the pyspec hive test suite, the failures will be tracked here:
### `pyspec`
- [ ] https://github.com/paradigmxyz/reth/issues/4811
- [ ] https://github.com/paradigmxyz/reth/issues/4814
- [ ] https://github.com/paradigmxyz/reth/issues/4670
- [ ] https://github.com/paradigmxyz/reth/issues/4815
- [ ] https://github.com/paradigmxyz/reth/issues/4816

closing since we're passing all of them now


Signs image before dock push

Codecov Report

Merging #88 (2724d80) into develop (440a1b6) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 2724d80 differs from pull request most recent head 701e196. Consider uploading reports for the commit 701e196 to get more accurate results

@@            Coverage Diff             @@
##             develop      #88   +/-   ##
==========================================
  Coverage      94.86%   94.86%
  Complexity       174      174
==========================================
  Files             32       32
  Lines            409      409
  Branches          33       33
==========================================
  Hits             388      388
  Misses            14       14
  Partials           7        7

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 440a1b6...701e196. Read the comment docs.


Add back current_conference tag needed for CFP.

@malemburg where was this tag used?

On the CFP and talk voting pages.
We also use such tags on other pages which are maintained in
the CMS.
On 3/10/2020 6:17 PM, umgelurgel wrote:

@malemburg https://github.com/malemburg where was this tag used?

--
Marc-Andre Lemburg
http://www.malemburg.com/

It would be a good idea to leave a note about it in a comment - othwerwise, it is likely someone will try to remove it again at some point, as it's not used in the codebase.


feat: support for Azure quota rule

Issue
Closes https://github.com/validator-labs/validatorctl/issues/225.
Description
Adds support for the rule. Like other rules with potentially repetitive input, users can provide input with a JSON file too.
Extras

Fixes a bug where if a user said they didn't want to enter Azure community image gallery rules, it would force them to anyways.
Fixes a bug where when users were prompted for a permission set for an Azure RBAC rule, the prompt incorrectly called it an IAM (for AWS) rule.

Since you have fresh context, can you please take a look at #29 and either address it as part of this PR or as a follow up?

Sure, I'll work on that next and add it to this PR as an extra bug fix.


Question - parsing to some AST?

Question: @davydotcom do you have any plans for the parsing outcome to be more strongly-typed?
i.e. introduce types for Resources, etc.
Context: I know you can have tons of different resource types and we cannot have object representation hard-coded for any of them. Yet, currently when trying to use the library for parsing my .tf files I end up having:
results.get("resource") which I need to cast to a Map<String, String> and then .get("the_resource_type_I_know") and then I get a Map instance which makes some sense (resource id -> Map of properties).

Good point! I'm primarily using it in a groovy app so haven't had that level of complexity in traversing a map. We could totally introduce another parser that is strongly typed

That being said HCL structure isn't very strongly typed

That being said HCL structure isn't very strongly typed

Yet there are some types, even visible in your code - arrays, attributes, values, etc. You do recognize if values are boolean/string/number, etc.
I see the point of types not being needed in Groovy.
I am wondering how to contribute on this idea. Do you see some way to keep one parser and have this solved? (so that would require the Groovy usage syntax not to become verbose, yet the types returned being different)

I made a first try wrt option 2.1 (see above). By the language spec the top level object in HCL (2) is a ConfigurationFile. Hence, I named the resulting class HCLConfiguration and it should contain a list of HCLAttributes and a list of HCLblocks.
What do you think?

(I know you weren't asking myself, but... )
I think that's the way to go.

I was asking everyone involved in this ticket / in this project - thanks for your support, @mccartney <3.

Ok so been a bit busy but getting back to hcl4j and doing a lot of updates. Some of these ideas are not bad but be aware the long term goal was for hcl4j to also perform runtime operations (which it now can do). It is capable of evaluating functions and variable lookups. It is capable of processing boolean conditional expressions, mathematical operations, etc. Currently this is building out as a Map Structure via the processSymbolPass1 and processSymbolPass2. One idea is to have your classes Extend the LinkedHashMap class so its interchangeable for compatibility with additional type casting methods. Another is to indeed make a Typed Parser that can process a similar runtime. I like either of those approaches. The Lexer and Parser should be nearly FULLY hcl2 syntactically complete.


update bootstrap template to match v4.1.3

update bootstrap template to match v4.1.3

https://getbootstrap.com/docs/4.1

add item_label_class for for radio buttons and check boxes collection

:link: https://github.com/rafaelfranca/simple_form-bootstrap/pull/95

remove unused .form-control-label css class

:link: https://github.com/rafaelfranca/simple_form-bootstrap/pull/97

Thanks 💙


perf: revert offsetParent polyfill

Kinda felt this was going to happen, this is causing poor performance as noted in here: https://github.com/shoelace-style/shoelace/discussions/902#discussioncomment-3607642
Going to revert this for now, unfortunately there's no fast native composedOffsetParent() to use and this polyfill is slow.
Users will have to:

Simply avoid using the DOM layout that causes issues, or;
This PR exports platform from the dom package, so they can spread in the other methods and only rewrite getOffsetParent to include this polyfill themselves without needing patch-package. If this is necessary for React DOM it can also be added

We started noticing performance hits from https://github.com/floating-ui/floating-ui/pull/1835/ as well. 😞

... the default platform object is now exported so you can spread in all the other methods then write your own single getOffsetParent hook that has the polyfill. You might want to add your own custom flags to turn it on/off when it's considered acceptable in a given scenario perf-wise.

Thanks for exposing this, seems like a good middle-ground.
WRT to the lack of composedOffsetParent(), I'll create an issue as suggested in  https://github.com/floating-ui/floating-ui/pull/1835#issuecomment-1207216945 to hopefully get some traction on it.


fix: read full data in mifareClassicReadBlock

This PR contains 2 commits.

The first one is just reformatting the file using default dartfmt.
The second one is the actual fix: mifareBlockLength is the length of the data block. Since response contains error flag in the first byte, we start reading from the 2nd element of the list, so we need to pass mifareBlockLength + 1 as the end parameter to the sublist (the end is exclusive).

Thanks for your submission.


after change Translatable::set_default_locale from EN to LT home page not found

after installing module I have changed default lang to LT Translatable::set_default_locale('lt_LT');
made /?flush but still on homepage with or without /lt/ page not found.

My advice would be to first install the Translatable module and see if that works correctly. I recall you have to set the default language first, before you even install the module, else the existing pages will automatically be set-up as en_US.
After that install the LanguagePrefix module. Note: if you use it out of the box, the URL would be /lt_LT/. Check out the docs on how to change that.

I found one thing that my EN home page had url /en/home and LT had /lt/home-2 , after I changed manualy EN url to /en/home-3 and LT to /lt/home everthing started to work fine. all solves that default lang home page must have home url.

To set all homepage url's to 'home' you need to, in your config.yml (See docs/enindex.md -> enable duplicate url segments):
enable_duplicate_urlsegments: true

Do a ?flush=all, then change your url segment by hand


The system cannot find the file specified

I'm trying to run fvid and after installing all the modules im getting this error.
C:\Users\Yakir Oren\Desktop\fvid>py fvid.py -i ./image.png -e

Traceback (most recent call last):
  File "fvid.py", line 275, in <module>
    make_video(video_file_path, image_sequence)
  File "fvid.py", line 200, in make_video
    ffmpeg.input(frames[0], loop=1, t=1).output(
  File "C:\Users\Yakir Oren\AppData\Local\Programs\Python\Python38-32\lib\site-packages\ffmpeg\_run.py", line 313, in run
    process = run_async(
  File "C:\Users\Yakir Oren\AppData\Local\Programs\Python\Python38-32\lib\site-packages\ffmpeg\_run.py", line 284, in run_async
    return subprocess.Popen(
  File "C:\Users\Yakir Oren\AppData\Local\Programs\Python\Python38-32\lib\subprocess.py", line 854, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "C:\Users\Yakir Oren\AppData\Local\Programs\Python\Python38-32\lib\subprocess.py", line 1307, in _execute_child
    hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] The system cannot find the file specified

It may be because of where your python is running. Try typing the fully qualified path for the input file, eg 'C:\Users\Yakir Oren\Desktop\fvid\image.png'
Python is funny with what it thinks is its root directory.

@YakirOren Yeah, try adding the full file path like @dtaivpp suggested and get back to us.

Hey the problem was on my part, I forgot to install the actual ffmpeg and not just the python lib. 😋


Delegation influences which VS takes effect

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
Here are two configurations, the only difference between them is that there is delegation in vs-0 in the second configuration. However, the route behavior is so different between these two configurations. For the first one, vs-0 takes effect and curl -H"host:www.example.com" http://<IP_ADDRESS>:8080 can get the correct service. For the second, vs-1 takes effect and curl -H"host:www.example.com" http://<IP_ADDRESS>:8080 returns 404.
First
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: vs-0
spec:
  hosts:
    - www.example.com
  http:
    - route:
        - destination:
            host: productpage.default.svc.cluster.local
            port:
              number: 9080
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: vs-1
spec:
  hosts:
    - 'www.example.com'
  http:
    - match:
        - uri:
            prefix: /ratings
      route:
        - destination:
            host: productpage.default.svc.cluster.local
            port:
              number: 9080
---
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: se-0
spec:
  hosts:
    - 'www.example.com'
  ports:
    - number: 8080
      protocol: HTTP
      name: port-1
      targetPort: 9080
  resolution: DNS

Second
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: vs-0
spec:
  hosts:
    - www.example.com
  http:
    - route:
        - destination:
            host: productpage.default.svc.cluster.local
            port:
              number: 9080
    - delegate:
        name: delegate  #only difference
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: vs-1
spec:
  hosts:
    - 'www.example.com'
  http:
    - match:
        - uri:
            prefix: /ratings
      route:
        - destination:
            host: productpage.default.svc.cluster.local
            port:
              number: 9080
---
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: se-0
spec:
  hosts:
    - 'www.example.com'
  ports:
    - number: 8080
      protocol: HTTP
      name: port-1
      targetPort: 9080
  resolution: DNS

I look up the RDS, which proves my opinion on which VS takes effect. In my understanding, which VS takes effect is determined by the host, instead of the route action (delegation here). Is there bug underlying?
route-without-delegate.log
route-with-delegate.log
Version
client version: 1.22-alpha.0554fb92631497bf4d0e7017b85fbaec834d0297
control plane version: 1.22-alpha.0554fb92631497bf4d0e7017b85fbaec834d0297
data plane version: 1.22-alpha.0554fb92631497bf4d0e7017b85fbaec834d0297 (2 proxies)

Client Version: v1.28.2
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.27.3

Additional Information
No response

For the second configuration, here is the delegate VirtualService.
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: delegate
spec:
  http:
    - route:
        - destination:
            host: productpage.default.svc.cluster.local
            port:
              number: 9080

Sorry, I omit VS delegate in the second configuration above, for it doesn't affect the route behavior. It is VS delegate that must have empty hosts instead of vs-0 or vs-1. @wulianglongrd

More complete second configuration.
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: vs-0
spec:
  hosts:
    - www.example.com
  http:
    - route:
        - destination:
            host: productpage.default.svc.cluster.local
            port:
              number: 9080
    - delegate:
        name: delegate
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: delegate
spec:
  http:
    - route:
        - destination:
            host: productpage.default.svc.cluster.local
            port:
              number: 9080
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: vs-1
spec:
  hosts:
    - 'www.example.com'
  http:
    - match:
        - uri:
            prefix: /ratings
      route:
        - destination:
            host: productpage.default.svc.cluster.local
            port:
              number: 9080
---
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: se-0
spec:
  hosts:
    - 'www.example.com'
  ports:
    - number: 8080
      protocol: HTTP
      name: port-1
      targetPort: 9080
  resolution: DNS

It can be set only when Route and Redirect are empty, "it" refers to VS.delegate. see https://istio.io/latest/docs/reference/config/networking/virtual-service/#HTTPRoute
So it looks like vs-0 is incorrect.

In vs-0, route and delegate are in different HTTPRoutes.

Sorry, I misread. Not sure if this is expected behavior, may need to dissect the source code.

On Feb 25, 2024, at 09:55, QiaO @.***> wrote:
In vs-0, route and delegate are in different HTTPRoutes.
—
Reply to this email directly, view it on GitHub https://github.com/istio/istio/issues/49539#issuecomment-1962785607, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNUYGDKZL46O7QCVHDBJHTYVKK23AVCNFSM6AAAAABDYATNL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRSG44DKNRQG4.
You are receiving this because you were mentioned.

Two reasons cause this issue:

VS merge changed order (see #49554)
There are duplicate RDS domains in the VS configuration, which means that only one VS will be applied. However, the order problem (the reason 1) causes the final applied VS to be different. The "First" configuration you provided applies vs-0 (matches /) and the "Second" configuration applies vs-1 (matches /ratings, note that it does not match /, so you will got a 404 ).

Thanks for your answer.
There is one more question. As you say, this packet doesn't match in vs-1 and should be dropped. But even though I turn off the network in my PC, it still returns 404. Is it Istio that replies me with 404?

The outbound proxy (envoy) returns 404 because it cannot match the route, outbound is executed on the client side.

On Feb 26, 2024, at 22:13, QiaO @.***> wrote:
Thanks for your answer.
There is one more question. As you say, this packet doesn't match in vs-1 and should be dropped. But even though I turn off the network in my PC, it still returns 404. Is it Istio that replies me with 404?
—
Reply to this email directly, view it on GitHub https://github.com/istio/istio/issues/49539#issuecomment-1964242897, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNUYGDOIULQ35ERIR7I5C3YVSKA5AVCNFSM6AAAAABDYATNL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRUGI2DEOBZG4.
You are receiving this because you were mentioned.

Is it really proper to return 404? Maybe 503 is more appropriate?

This is a client error, so 4xx is correct.

I see. Thanks.

The hosts must be empty for a delegate VirtualService.

Was getting a 404 NR myself and this solved it. But I haven’t found this documented anywhere, only that Route and Redirect had to be empty here. Are there even more conditions maybe?


Add apps preview

Hi,
Before I do it for each of the sample, i'd like to know if you are interested in my idea : bring a screenshot of each app on the main readme page so that when people look for a sample, they can see it in one page.
Regards

Hi Jeff,
that would be absolutely fine. This works for now, but slightly concerned on how the page will look after >50 samples, but not bad idea in general. Happy to get PR with the references in. I'll close this one to avoid confusion with only two items mentioned in the page.


Avoid using operators for binding arguments

It seems the compiler as of the current master (4852b5d) is no longer able to bind operators as function arguments. If this is intentional, should it be listed in https://github.com/purescript/purescript/issues/2012?

Perhaps yeah, it's implied by "Remove non-aliased operator declarations" but that's probably not obvious.
Thanks!


helm repo add chart-name https://private-repository.company.com --username --password

Description

With the following cmd, you can add easily a protected repository to your charts.
helm repo add chart-name https://private-repository.company.com --username <user>--password <pass>
However, it's not possible through the helm-dashboard. Please consider adding this feature.

Screenshots
No response
Additional information
No response

It's a duplicate of #222


Use new Watson Tone Analyzer API

The API was updated to v3: New endpoint is api/v3/tone?version=2015-02-11

Delivered by #12


swap the order for creating columns through the magic method

if the column is a relation, there is no need to check if it’s a column in the database
this way we don’t need a query if the column is a mutator or relation
eg
$grid->pack_products(); // this is a HasMany relation

Currently it checks first if 'pack_products' is a column, but it should check first if it's a mutation or a relation. This way we skip the redundant call to the database.
BTW
if i use $grid->column('pack_products'); the model doesn't use the ->with('pack_products') for some reason.

Yes you're right, Thanks！


Feat/1289/update helm values

Description
This PR updates the values.yaml file with the content of the development-minimal.yaml file, and the development.yaml with just the startup script
This way we have a default minimal configuration with the option to customize thel deployment using the new development.yaml file.
Fixes #1289
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[x] Deployed on our test server: all pods running without errors

Test Configuration:

Kubernetes Version:
Helm Version:
Hardware:

Checklist:

[ ] I have made corresponding changes to the documentation
[ ] I have made corresponding changes to the changelog (if needed).
[ ] I have made corresponding notices to the upgrade instructions (if needed)

Changelog changes
Upgrade instructions

I need to rebase this on the v0.9.6-prerelease1 branch. A change of the values.yaml and development.yaml can only be executed at the same time with a new version release as it would break otherwise the existing installation process.

Codecov Report

Merging #1291 (d357fcf) into 0.9.6-prerelease1 (7fe6c4b) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@                  Coverage Diff                  @@
##           0.9.6-prerelease1    #1291      +/-   ##
=====================================================
- Coverage              12.42%   12.34%   -0.08%
=====================================================
  Files                     29       29
  Lines                   9185     9185
=====================================================
- Hits                    1141     1134       -7
- Misses                  7984     7989       +5
- Partials                  60       62       +2

Impacted Files
Coverage Δ

golang/internal/exponential_backoff.go
51.72% <0.00%> (-10.35%)
:arrow_down:

golang/internal/ioddfinder.go
64.38% <0.00%> (-2.74%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@JeremyTheocharis Can you re-review this, so i can prepare the 0.9.6-pr1 release ?


Can I have an example with create-react-app?

Can I have an example with create-react-app?
I've tried to implement this lib within create-react-app project but it does not work.
It seems my worker.js has something wrong.

component.js
 <WebWorker url="/worker.js">
    {({ data, error, postMessage }) => {
      if (error) return `Something went wrong: ${error.message}`
      if (data)
        return (

            Received some data:

{JSON.stringify(data, null, 2)}

        )
      return <button onClick={() => postMessage('hello')}>Hello</button>
    }}
  </WebWorker>

https://codesandbox.io/s/ox85ooo31q

https://codesandbox.io/s/ox85ooo31q
https://spectrum.chat/thread/45727df4-6d3b-4392-a771-218c09329333
mention about the solution in those link.

@rainstormza @ghengeveld The sandbox is 404'ing, can we get an updated example?

Here's a basic sandbox with CRA: https://codesandbox.io/s/bold-blackwell-vxxth

Here's a basic sandbox with CRA: https://codesandbox.io/s/bold-blackwell-vxxth
The sandbox is 404'ing

Here's another: https://codesandbox.io/s/react-webworker-cra-example-3fjim

The gist is to put the worker file in /public.


IntelliJ .icls

.. can be easily generated with: https://github.com/JetBrains/colorSchemeTool
Just copy the Visual Studio Code files into vscThemes/, and run bash convert.sh. Files will appear in intellijThemes/

The IntelliJ theme can be found here:
https://github.com/kepano/flexoki/tree/main/intellij


Support Commissioner DUT Test Vectors Generation.

Problem
Need tools to generate test vectors positive and negative for Commissioner DUT
Ticket: #17511
Change overview
Added python script to generate test vectors:
src/tools/chip-cert/gen_com_dut_test_vectors.py
This script currently generates only structural errors in the DAC and PAI certificates.
Next steps will be to generate structural CD error and other validation flow errors.
By default the generated test vectors will be stored here:
credentials/development/commissioner_dut/

/rebase


[moor_ffi] Supports opening Database in read-only mode

Currently Database.open always creates the database. Would it be possible to add a read-only mode?
(I'm opened to work on it If you're ok)

This sounds like a good addition to have, thanks! I'll happily accept contributions, let me know if you need anything.


Sort locstring JSON by key

We should sort the output locstring.json by key to make easier to spot changes.

You are totally right 👍  :D

Oh many we definetely need to do this 👍


pyxdg returns the GitHub mirror

https://github.com/jayvdb/pypidb/issues/40#issuecomment-609167093
https://freedesktop.org/wiki/Software/pyxdg/ says http://cgit.freedesktop.org/xdg/pyxdg/ is the main repository, but https://github.com/takluyver/pyxdg is the mirror.

_get_weighted_best uses count first, which https://freedesktop.org/wiki/Software/pyxdg/ links to a lot, whereas there is only one link to cgit.
https://github.com/takluyver/pyxdg does mention it is a mirror, but doesnt link back to cgit.
We might be able to take advantage of the email<EMAIL_ADDRESS>to prioritise repos under https://cgit.freedesktop.org/xdg/

dbus-python has email<EMAIL_ADDRESS>, and is also hosted on https://cgit.freedesktop.org/dbus/dbus-python/ which is a mirror of https://gitlab.freedesktop.org/dbus/dbus-python , and is in the top 2000-4000
https://gitlab.freedesktop.org/explore?sort=latest_activity_desc&utf8=✓&name=python&sort=latest_activity_desc lists more
https://pypi.org/project/hid-tools/ - direct link to gitlab
https://pypi.org/project/libevdev/ - direct link to gitlab
https://pypi.org/project/xpyb/ - no direct link
I cant find the PyPI name for https://gitlab.freedesktop.org/gstreamer/gst-python or https://gitlab.freedesktop.org/telepathy/telepathy-python
https://pypi.org/search/?q=freedesktop doesnt provide any extra relevant results.


Updates azdataGraph version from 0.0.27 to 0.0.28

This PR updates the azdataGraph version to version 0.0.28.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 41.72%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/book/bookTreeView.ts
2
38.21%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
27742

Relevant Lines:
62043

💛  - Coveralls


Use layout file from DB.

Fixes #294

Hi Orel. The new function def testTelescopeFile(db): has no underline '_' after the 'test'. Will pytest run this test anyways?

This function is not a test, it's a fixture which is called and run when a different function needs to testTelescopeFile. However, I agree it's not a good name cause of this convention. Therefore, I just changed the name to be telescopeTestFile. You can see the changes in https://github.com/gammasim/gammasim-tools/pull/298/commits/7eb3d32180d212ab17aff7c136ebf6779bf902ea


Use yarn to install nodejs dependecies

As suggested by @josephfrazier, this PR replaces npm with yarn. Let's see how this goes

I'm curious to see how this turns out! A couple of thoughts:

We should probably change the appveyor install command as well. I had to use yarn install --ignore-engines to get it working.
The Contributing guide should be updated to use yarn install or just yarn instead of npm install. We might also want to point to https://yarnpkg.com for context.

According to the yarn docs, yarn is already preinstalled on AppVeyor https://yarnpkg.com/lang/en/docs/install-ci/#appveyor-tab
Good shout with the contributing guides! Will update them now

Yarn is preinstalled on AppVeyor, but it doesn't sound like AppVeyor will automatically run it like Travis does.

Oh, it looks like yarn install didn't need --ignore-dependencies after all. The build still failed, but that should be fixed by https://github.com/OctoLinker/browser-extension/pull/296

Unbelievable everything is green ... and blue 😉

Excellent! Now we'll just have to remember to use yarn instead of npm for changing dependencies.


Missing bps

there are some missing bps, let's check what happened and fix it

this issue is solved with the number #1046


IPM-111:Publish HOTA metrics, ID switching and average track length for the tracker

Purpose
What is the purpose of this PR? Added a new metric Average Track Length to multi object tracking evaluation
What is the associated Jira ticket? IPM-111
Approach
How did you go about approaching the Jira acceptance criteria?
What noteworthy changes did you make that reviewers should be aware of? Added new metric
Did you add new packages or change configurations? No
Did you deviate from the original criteria of the Jira ticket? No
TO-DOs

Links to follow-up tickets that should be done after this PR. IPM-136

It would be good to update the README with clear instructions on how to create an environment, download data, and then run the metrics.


Question: Usage of Table 10011 in Report 1306(US) but not in Report 206(US)

Question related to Issue #1452:
Report 206 in financialsUS does no longer use Table 10011 for sales tax calculations (this was modified over a year ago) but it is still used in Report 1306.  Is there a reason for different tax calculation methods in those reports?
This would not be an issue for us if we could extend the Report's dataset and deploy additional layout

Any news on this issue?


How i can set progress colors dynamically?

How i can set progress colors dynamically?

I donot think you can change them dynamically as of now. You can however set them by basically overriding the ones set in the lib.
Here is a detailed answer on stackoverflow
There are ways to hack it, but I wouldn't recommend that. More so the lib is not designed to handle this situation, so you would be writing a lot of the code yourself to make it work, which defeats the purpose of using this lib.
If you think it is required very much I can look into it sometime later, meanwhile you can look at some other lib that actually has the workaround inbuilt and functional.
If you happen to get it to work (before I work on it), please consider sending a PR to update this lib.
Thank you


[BugFix] Fix null pointer error of column upserted

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #6697
Problem Summary(Required) ：

Jump out of current for loop if segment iterator is nullptr. Otherwise, the columns upserted will throw a null pointer error.

@Linkerist bug-fix should be merged in main-branch first, has this bug-fix been merged in main branch?

@Linkerist bug-fix should be merged in main-branch first, has this bug-fix been merged in main branch?

@imay already in main

@Linkerist bug-fix should be merged in main-branch first, has this bug-fix been merged in main branch?

@imay yeah, have been merged already.


Possible invalid C multiline comment

https://github.com/McStasMcXtrace/mccode-antlr/blob/132fb4fa1733f614d190b6bf856548be5293aa2d/mccode_antlr/translators/c_initialise.py#L43
If the string representation of a component Instance contains the characters */ this multiline comment will break, and the resulting C file will not compile.
Either remove this line entirely, or only include the 'basic' positioning/orienting part of the instance definition.
// COMPONENT instance = Type() AT (x, y, z) RELATIVE rel ROTATED (a, b, c) RELATIVE rel
without any GROUP, JUMP, EXTEND, METADATA, etc., information  following.

Fixed by #49


specs: VEP-2420: Getting started with your Data

@antoniivanov  I would love if you could review this again.
I am tempted to remove entirely the sections:  Layer's of abstraction are not clear and Not obvious what properties do .
The will be handled by the source/sink initiative.
WDYT?

@antoniivanov I would love if you could review this again. I am tempted to remove entirely the sections: Layer's of abstraction are not clear and Not obvious what properties do . The will be handled by the source/sink initiative.
WDYT?

Yes. Put them in non-goals explicitly with comment that they would he handled as part of https://github.com/vmware/versatile-data-kit/issues/2422


feat:adding release drafter configuration

Signed-off-by: kushthedude<EMAIL_ADDRESS>Fixes #2359

@wetneb @thadguidry I have renamed and made the config file on the basis of the labels currently getting used in Open Refine.
Now to really take a benefit of the ReleaseDrafter we need to label the PRs accordingly. To facilitate the labelling better : We can have Semantic PR Naming. Once we will have the semantic PR naming, I can make a GitHub bot in a day or two which may label the PR according to its name prefix. With this our complete release drafting process will get automated.
Please tell me how does this sound to you !

I have my own preferences, but @wetneb has been responsible for our last several releases, so I will let him make all the judgement calls for this.  Work with him directly and we should be in great shape.

Now to really take a benefit of the ReleaseDrafter we need to label the PRs accordingly. To facilitate the labelling better : We can have Semantic PR Naming. Once we will have the semantic PR naming, I can make a GitHub bot in a day or two which may label the PR according to its name prefix. With this our complete release drafting process will get automated.

We are already tagging issues, so in theory it should be possible to use these tags without having to duplicate them on the PRs themselves, assuming the PRs correctly use keywords to close issues, no?

Now to really take a benefit of the ReleaseDrafter we need to label the PRs accordingly. To facilitate the labelling better : We can have Semantic PR Naming. Once we will have the semantic PR naming, I can make a GitHub bot in a day or two which may label the PR according to its name prefix. With this our complete release drafting process will get automated.

We are already tagging issues, so in theory it should be possible to use these tags without having to duplicate them on the PRs themselves, assuming the PRs correctly use keywords to close issues, no?
If we can avoid writing our own custom bot for this, it would probably be better (since that avoids maintaining that code).

The releaser drafts the notes on the basis of the pull request since pull request title better signifies the gist of issue getting solved.

Sure, that is obviously fine.
I would be happy to label pull requests too. What I mean is that it should be possible to have a GitHub action which automatically labels pull requests by copying the labels from the issue that the pull request closes, no?

Closing this as stale, no activity for nearly a year. Feel free to reopen once the problem mentioned above is addressed (our PRs are not currently tagged, issues are, so the workflow would not work well as such).

Closing this as stale, no activity for nearly a year. Feel free to reopen once the problem mentioned above is addressed (our PRs are not currently tagged, issues are, so the workflow would not work well as such).


MultiSelection is broken

When using the grid in multi selection mode, it thorws this error when clicking oin the row checkboxes.
(The select all in the header works)

Fix has been released in version 3.0.1


Entrypoint expects DATABASE_URL env var

This is something used on the SODAR Django server but never used here. I think this is a copy/paste error. Will double check to make sure.

Fixed.


Add quotes to "true" value so it is not interpreted as a boolean

Description
The scrape label for namespaces is documented in values with a commented example. This example will not run as-is and results in the newrelic-infrastructure pods crashing on startup.
newrelic.com/scrape: true
This change adds quotes (") to the example. Tested working on a fresh installation.
newrelic.com/scrape: "true"
Type of change

[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] New feature / enhancement (non-breaking change which adds functionality)
[ ] Security fix
[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[ ] Add changelog entry following the contributing guide
[ ] Documentation has been updated
[ ] This change requires changes in testing:

[ ] unit tests
[ ] E2E tests

Hi Max, Do we have a timeline for this PR to be merged into the branch. Please let us know. Thanks Sumit


HL-582 | HL-Backend: Link HL-582 in README.md related to NEXT_PUBLIC_MOCK_FLAG

Description :sparkles:
HL-Backend: Link HL-582 in README.md related to NEXT_PUBLIC_MOCK_FLAG
Issues :bug:
HL-582
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #1652 (8b6d9f7) into develop (00ab1fd) will increase coverage by 28.94%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop    #1652       +/-   ##
============================================
+ Coverage    65.92%   94.87%   +28.94%
============================================
  Files         1050      197      -853
  Lines        29010     8544    -20466
  Branches      2972        0     -2972
============================================
- Hits         19126     8106    -11020
+ Misses        9436      438     -8998
+ Partials       448        0      -448

Impacted Files
Coverage Δ

backend/shared/shared/common/utils.py
51.21% <0.00%> (-48.79%)
:arrow_down:

backend/shared/shared/azure_adfs/views.py
40.00% <0.00%> (-44.00%)
:arrow_down:

backend/shared/shared/common/tests/__init__.py
66.66% <0.00%> (-33.34%)
:arrow_down:

backend/shared/shared/common/lang_test_utils.py
80.00% <0.00%> (-20.00%)
:arrow_down:

backend/shared/shared/common/tests/conftest.py
86.95% <0.00%> (-13.05%)
:arrow_down:

backend/shared/shared/audit_log/audit_logging.py
87.23% <0.00%> (-12.77%)
:arrow_down:

backend/shared/shared/oidc/auth.py
83.05% <0.00%> (-5.09%)
:arrow_down:

frontend/kesaseteli/handler/src/pages/index.tsx

...er/src/hooks/employments/useGetEmploymentErrors.ts

.../applicant/browser-tests/page-model/MainIngress.ts

... and 850 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


how to set the CUDA path to environment variable `CUDA_PATH'

Hello,
Im new using cupy and anaconda in linux. So im afraid to make any error in environment system.
I install cupy using the following and the installation was ok:
source activate testing-env
conda install -n testing-env cupy
Downloading and Extracting Packages
cudnn 7.1.2: ######################################################################## | 100%
cupy 4.0.0: ############################################ | 100%
nccl 1.3.5:############################################################# | 100%
fastrlock 0.3:############################################################ | 100%
Preparing transaction: done
Verifying transaction: done
Executing transaction: done
in my script I test the installation fisrtly:
import cupy as cp
print 'test'
output: test
but when i modify the script to be:
import numpy
import cupy as cp
x = cp.arange(6).reshape(2, 3).astype('f')
print x
i get the following:
(testing-env) [tamouze@cdr352 Python_directory]$ python "/project/6008168/tamouze/Python_directory/ali.py"
/home/tamouze/anaconda2/envs/testing-env/lib/python2.7/site-packages/cupy/creation/ranges.py:57: UserWarning: Please set the CUDA path to environment variable `CUDA_PATH`
  _arange_ufunc(typ(start), typ(step), ret, dtype=dtype)
Traceback (most recent call last):
  File "/project/6008168/tamouze/Python_directory/ali.py", line 3, in <module>
    x = cp.arange(6).reshape(2, 3).astype('f')
  File "/home/tamouze/anaconda2/envs/testing-env/lib/python2.7/site-packages/cupy/creation/ranges.py", line 57, in arange
    _arange_ufunc(typ(start), typ(step), ret, dtype=dtype)
  File "cupy/core/elementwise.pxi", line 826, in cupy.core.core.ufunc.__call__
  File "cupy/util.pyx", line 39, in cupy.util.memoize.decorator.ret
  File "cupy/core/elementwise.pxi", line 625, in cupy.core.core._get_ufunc_kernel
  File "cupy/core/elementwise.pxi", line 33, in cupy.core.core._get_simple_elementwise_kernel
  File "cupy/core/carray.pxi", line 146, in cupy.core.core.compile_with_cache
  File "/home/tamouze/anaconda2/envs/testing-env/lib/python2.7/site-packages/cupy/cuda/compiler.py", line 164, in compile_with_cache
    ptx = compile_using_nvrtc(source, options, arch)
  File "/home/tamouze/anaconda2/envs/testing-env/lib/python2.7/site-packages/cupy/cuda/compiler.py", line 82, in compile_using_nvrtc
    ptx = prog.compile(options)
  File "/home/tamouze/anaconda2/envs/testing-env/lib/python2.7/site-packages/cupy/cuda/compiler.py", line 245, in compile
    raise CompileException(log, self.src, self.name, options)
cupy.cuda.compiler.CompileException: /home/tamouze/anaconda2/envs/testing-env/lib/python2.7/site-packages/cupy/core/include/cupy/carray.cuh(10): catastrophic error: cannot open source file "cuda_fp16.h"

1 catastrophic error detected in the compilation of "/tmp/tmplsiNNY/kern.cu".
Compilation terminated.

My environment includes the following package:
# packages in environment at /home/tamouze/anaconda2/envs/testing-env:
#
# Name                    Version                   Build  Channel
ca-certificates           2018.03.07                    0
certifi                   2018.4.16                py27_0
cudatoolkit               9.0                  h13b8566_0
cudnn                     7.1.2                 cuda9.0_0
cupy                      4.0.0            py27he3f55b6_0
fastrlock                 0.3              py27hf484d3e_0
intel-openmp              2018.0.0                      8
libedit                   3.1                  heed3624_0
libffi                    3.2.1                hd88cf55_4
libgcc-ng                 7.2.0                hdf63c60_3
libgfortran-ng            7.2.0                hdf63c60_3
libstdcxx-ng              7.2.0                hdf63c60_3
mkl                       2018.0.2                      1
mkl_fft                   1.0.1            py27h3010b51_0
mkl_random                1.0.1            py27h629b387_0
nccl                      1.3.5                 cuda9.0_0
ncurses                   6.0                  h9df7e31_2
numpy                     1.14.2           py27hdbf6ddf_1
openssl                   1.0.2o               h20670df_0
pip                       9.0.3                    py27_0
python                    2.7.14              h1571d57_31
readline                  7.0                  ha6073c6_4
setuptools                39.0.1                   py27_0
six                       1.11.0           py27h5f960f1_1
sqlite                    3.23.1               he433501_0
tk                        8.6.7                hc745277_3
wheel                     0.31.0                   py27_0
zlib                      1.2.11               ha838bed_2

Can you please help on how to set the cuda variable?

Thank you for using CuPy.
Currently, to use CuPy package from Anaconda with CUDA 9.0 or later, you need to install CUDA Toolkit on your host. (conda package cudatoolkit==9.0 does not contain cuda_fp16.h, which is required for CuPy.)
Then, specify the CUDA Toolkit installation directory (e.g., /usr/local/cuda-9.0) to CUDA_PATH.

@kmaehashi
thank you for you help
when i type  echo $CUDA_PATH  i get nothing. So as im working on an ssh server (compute canada) some modules are available so if i type before runing my code, module load cuda/9.0.176 and then write  echo $CUDA_PATH /cvmfs/soft.computecanada.ca/easybuild/software/2017/avx2/Compiler/gcc5.4/cuda/9.0.176
can i use this one? or i want to install as you suggest?

@kmaehashi i wanna tried but how set up the variable?

@kmaehashi i can confirm it is working. Thank you so much


Add Autocomplete to Docs Site

Would be great to add Autocomplete to docs site.

Making a note that I'll address this in the next week.

Closing since we are moving to storybook docs

 Add Autocomplete to Docs Site


omiagent using 100% cpu

see this issue for details.  Was referred here by that parent issue.

the issue has been fixed.


FE-72: Fix lefthand menu with separate scrollbar.

Overview
JIRA: https://r3-cev.atlassian.net/browse/FE-72
Issue: left hand index menu in docs should not scroll up and down in sync with the main content window. When the doc page is long, as the user scrolls, eventually the left hand index menu can no longer be seen at all.
Fix: There should be a separate scroll window for the left menu, so that when several items are expanded, the user can scroll the menu independently of the the main content window.
Screenshot

Hi @zhengmar3, I published a preview of this PR here  for review.
Something funny seems to be happening with the footer when you scroll to the bottom. It's especially strange on pages where the left-hand menu is long. For example: https://nadine.preview.docs.r3.com/en/platform/corda/5.0-beta/developing/development-tutorials/build-basic-cordapp/basic-cordapp-state.html

Fix: There should be a separate scroll window for the left menu, so that when several items are expanded, the user can scroll the menu independently of the the main content window.

@nadinequinr3
Ok, let me try to fix it. Could be very tricky to fix due to hugo while keep the left-band bar fixed on its position.

@nadinequinr3
You can have a look.
I sort of have a fix for this, although it's not perfect. Cos for most docs site with table of content on the left-side, it seems to not having big footer like ours.

@nadinequinr3 You can have a look. I sort of have a fix for this, although it's not perfect. Cos for most docs site with table of content on the left-side, it seems to not having big footer like ours.

Thanks @zhengmar3. It's definitely better than it was. @colin-hillis can you take a look to see if you're happy?