Create CITATION.cff

👀

Should we also add a section in the README?

added

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.32%. Comparing base (7cf656b) to head (bcb52ae).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1756      +/-   ##
==========================================
- Coverage   69.33%   67.32%   -2.01%
==========================================
  Files         305      305
  Lines       15892    15961      +69
==========================================
- Hits        11018    10746     -272
- Misses       4874     5215     +341

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

"a finetuning library" bestows us with a modest sagaciousness
Think we just need to link to the section in the readme heading (https://github.com/pytorch/torchtune/blob/a8a64ec6a99a6ea2be4fdaf0cd5797b03a2567cf/README.md?plain=1#L10)


Update copyright headers for 2023

Used ./ci/scripts/copyright.py to update all existing copyright headers for 2023.

Dont merge this, need to discuss.

/merge


[Suggestion] Water outputs on machines

I think it would be a nice addition if steam-using machines had an output for water, like a condenser that collects water from the steam used in processes. There could also be a vent for each machine that would drastically reduce or stop water from being collected, so that players who don't want to reuse water have that option. This would make for interesting automation with finite water as people will try to create closed-loop systems, there could possibly be a configurable loss on the water collected too.

Note to self - this could be implemented by converting the existing steam tank to a multifluidtank size locked to 2 fluids - steam in and water out. Implement at base level, for consistency.


Try to fix build on travis

I'm giving it another try (now that the scala api stuff was moved).

log4j-core-its fail to build because:

logging.log4j:log4j-core-its:jar:2.9-SNAPSHOT: Could not find artifact org.apache.logging.log4j:log4j-server:jar:tests:2.9-SNAPSHOT in apache.snapshots (https://repository.apache.org/snapshots)

No idea where log4j-server is hosted and where it is build, but its maven-jar-plugin configuration seems to be missing a <goal>test-jar</goal>.

Now the log4j-server jar just needs to be build and published to the apache snapshot repository.


Fix InputGroupParticipationCommand

Description
Initially, the point of having this command was to make the life of TA easier by giving participation points to everyone. However, if there are absent students, the TA won't be able to use this command. To fix this bug, we now allow the TA to use this command, and instead of throwing an error message, we now give 0 participation points to absent students.
Context
Checklist

[x] I have self-reviewed my changes.
[x] I have added JavaDocs to all relevant methods.
[x] I have updated the documentation:

User Guide Table of Contents, Description and Summary.
Developer Guide, if applicable.

[x] I have ran ./gradlew test or ./gradlew check and all checks pass.
[x] I have updated my project portfolio with what I have contributed.

Check is not passing and on my side i am not too sure if this is a regression but i am getting this
08.11.2023_18.36.06_REC.mp4

Expected:


Haines Borough, Alaska

http://services3.arcgis.com/pMlUMMROURtJLUZt/ArcGIS/rest/services/hainesborough_parcels07312015/FeatureServer/0

Contains no addresses.  There's an address field but it's for owner details, not physical address.

dang. I saw the ADDRESS1 and ADDRESS2 fields and thought it would have something
I'll e-mail them and see
Sorry about that

No problem, good to explore.

received a call - they should be sending the data soon.

received a fgdb - erroring out. emailed back and asked for shapefile from:
Holly Smith
Borough Planner
Haines Borough
103 Third Avenue S.
PO Box 1209
Haines, AK 99827
www.hainesalaska.gov
<EMAIL_ADDRESS>Phone:<PHONE_NUMBER> ext. 23
Fax:<PHONE_NUMBER>

holly is going to send data this week!

https://www.dropbox.com/s/0m0gtlnoablvt72/HainesAddress42017.zip?dl=0

updated ready for pr although it is still in shapefile format.


Use right port on backendRef port match

What type of PR is this?
bug
Which issue does this PR fix:
#241
What does this PR do / Why do we need it:
Interpret backendRef port "correctly", it should be service port, not target port per gwapi documentation.
To do this, we need to figure out what is the right set of endpoint ports only based on the service port. We can deduce this by the following logic:

A service port MUST have a name if there are multiple ports exposed from a service. Therefore, if a port is named, endpoint port is only relevant if it has the same name.
If a service port is unnamed, it MUST be the only port that is exposed from a service. In this case, as long as the service port is matching with backendRef/annotations, we can consider all endpoints valid

This is also commented in the code.
If an issue # is not available please add repro steps and logs from aws-gateway-controller showing the issue:
Testing done on this change:
unit tests and e2e tests.
Automation added to e2e:
Fixed existing e2e test case, I just simplified it into two test scenarios that seems relevant to this use case.
Will this PR introduce any new dependencies?:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:
Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

16 of 21 (76.19%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 39.931%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/gateway/model_build_targetgroup.go
0
2
0.0%

pkg/gateway/model_build_targets.go
16
19
84.21%

Totals

Change from base Build<PHONE_NUMBER>:
0.007%

Covered Lines:
3458

Relevant Lines:
8660

💛  - Coveralls


soc: nxp_rt: Enable INIT_ARCH_HW_AT_BOOT and CACHE_MANAGEMENT

Enable INIT_ARCH_HW_AT_BOOT and CACHE_MANAGEMENT so the cache and other hardware registers are cleared on boot.
Fixes https://github.com/zephyrproject-rtos/zephyr/issues/54731

@mmahadevan108 can you revisit this PR?

I think this is no longer needed after the merge of https://github.com/zephyrproject-rtos/zephyr/pull/67443

closing based on comment from @danieldegrasse


Small fixes for torchao quant

Summary:

quantization should happen before tp (in the future ideally we can just laod quantized checkpoints, and tp can be different depending on specific deployment env)
adding _is_linear check for filter_fn

Test Plan:
python3 -m sglang.bench_one_batch --model meta-llama/Meta-Llama-3-8B --batch-size 1 --input 128 --output 8 --json-model-override-args '{"architectures": ["TorchNativeLlamaForCausalLM"]}' --enable-torch-compile --torchao-config int4wo-64 --tp-size 2

Reviewers:
Subscribers:
Tasks:
Tags:

not sure why the CI is failing, I can get speedup locally

Hi @jerryzh168 May you try this locally
cd test/srt
python3 run_suite.py --suite minimal --range-begin 15 --range-end 30


supports per-invocation table name

@MatteoSp Is it possible to add unit tests? Thanks for your pull request

https://github.com/YahuiWong/Dapper.Contrib.BulkInsert/blob/master/test/Dapper.Contrib.BulkInsert.Test/TestUser.cs
Each table is an entity, and if it is a table with the same table structure and different table names, you can use object-oriented thinking to use a new entity


Add anthos distributions to the user-agent-mode enum

right now the user-agent-mode has 3 supported values: gke, kubectl, unspecified, ref: https://github.com/GoogleCloudPlatform/prometheus-engine/blob/v0.6.1/pkg/export/setup/setup.go#L134
By adding anthos distributions to the enum (e.g. one additional enum item anthos or further breakdowns like anthos-vmware, anthos-baremetal, anthos-aws, anthos-azure, anthos-attached), we'll be able to further distinguish the traffic coming from different type of clusters.

Just notices we had the mode for anthos in an earlier version:  https://github.com/GoogleCloudPlatform/prometheus-engine/blob/v0.4.3/pkg/operator/operator.go#L237
we need to add back those options to keep backward compatibility.


Doesn't seem to compile

;
  }
});

I'm assuming, that at the very least, the

And React is adding the

;
  }
});

Unfortunately it seems to still be happening to me

styled-jsx seems to be called, I added a log to styled-jsx/babel.js and it's showing up in the output. But I'm still getting the error.

yarn build:webpack v0.18.0
$ webpack -p
running babel
Hash: afc2d2da1570a962ac31
Version: webpack 2.1.0-beta.27
Time: 12225ms
        Asset     Size  Chunks             Chunk Names
    bundle.js   797 kB       0  [emitted]  main
bundle.js.map  6.14 MB       0  [emitted]  main
 [675] multi main 88 bytes {0} [built]
    + 675 hidden modules

Webpack Bundle Analyzer saved stats file to /home/mischka/projects/ics-merger/public/js/stats.json
Done in 13.13s.

Please let me know if there's anything I can do to help, thanks!

Now I have the error described in #16

I can confirm that 0.0.5 does not resolve the issue I am having. Let me know if I can give more information to troubleshoot this somehow!

The tag is now compiling for me with 0.0.7, though I seem to be running into a different issue now. Regardless, if someone else confirms that I'll close the issue. Thanks everyone!

@jacobmischka @hzoo 0.0.7 fixed this issue for me as well.

Okay, so it seems to work only under specific conditions.
This transpiles:
/** @jsx h */

import 'skatejs-web-components';
import { Component, define, h } from 'skatejs';

define(class Test extends Component {
  renderCallback () {
    // return

;
  }
});

window.customElements.define('x-app', class extends Component {
  renderCallback () {
    return (

this should be bold

        <Test>this should not be</Test>

    );
  }
});

It seems if I comment out the following line, it works:
return

;

Moving the component to a separate file works.

Having the same issue. If a file contains two component definitions and the bottom one renders a

export default App

DOES get transformed (style tag only in first component)
import React from 'react'

const ComponentLink = () =>

    Hello

const App = () =>

    <ComponentLink />

export default App

DOES get transformed (style tag in both components)
import React from 'react'

const ComponentLink = () =>

    Hello

const App = () =>

    <ComponentLink />

export default App

What about this case, with only in second component, but being rendered after the style tag in the second?
import React from 'react'

const ComponentLink = () =>

    Hello

const App = () =>

    <ComponentLink />

export default App

^ building off of aputinski's test cases, a temporary workaround for this issue is to just put an empty style tag in each component in the file:
import React from 'react'

const ComponentLink = () =>

    Hello

const App = () =>

    <ComponentLink />

export default App

The problem is that the babel plugin works only on the first JSX block it encounters.
If it doesn't find any styles it skips the rest.
@rauchg I have a fix for this however it won't transpile "jsx partials" – in order to support them we'd need to rethink about the architecture of the plugin since there are a lot of possible scenarios.
With the fix you'd get this:
import _JSXStyle from "styled-jsx/style";

const foo = hey
const MyComponent = () => (

      {foo}
      <_JSXStyle css={"span[data-jsx=\"2742264064\"] {color: red;}"} data-jsx={"2742264064"} />

)

i.e. foo won't get the data attribute.
I argue that fixing it partially could be worse than the status-quo but I'd be happy to put a PR together anyway if you think that this is better than nothing.

@giuseppeg not sure I'm too concerned about jsx partials for now. For jsx partials one can use a different
            bar

      </Provider>
    )
  }

I'll get

The weird part is if I make file dirty (hit space bar or something) and save to trigger rebuild.
It'll work perfectly after that but if I hit refresh button it will error again.
Maybe I doing things wrong? Do I need .babelrc?
Thanks

@katopz yes you need a .babelrc file at the root of your project or similar. See the usage section. If the issue persists you may want to join zeit.chat

I had to add plugins: ['styled-jsx/babel'] to my babel loader query config as well.


FluentSelect loading options from Api call, when return with one entry, can't be selected

🐛 Bug Report
Using FluentSelect and filling options by calling an Api, the Api return with one entry, which is displayed but can't be selected, although the component default to it and visually it's selected but it's never reflected to the @bind-Value even after you try to re-click and re-select it.
💻 Repro or Code Sample
Please note this is a sample code skeleton of my actual code, but not the exact code
@inject IHttpClientFactory _clientFacotry;

@page "/bug"

     <FluentSelect<EMAIL_ADDRESS>Label="Country"
                   Items="countries" OptionText="@(i => i.CountryName)" OptionValue="@(i => i.CountryAlpha2Code)" Required>
     </FluentSelect>
<EMAIL_ADDRESS>     <FluentValidationMessage For="@(() => userRegister.CountryAlpha2Code)" />

@code
{
private HttpClient apiClient = new HttpClient();

private UserRegister userRegister = new UserRegister();
private List<Country> countries = new List<Country>();

protected override async Task OnInitializedAsync()
{
    try
    {
        apiClient = _clientFacotry.CreateClient("namedClientOnProgram.cs");
        countries = await apiClient.GetFromJsonAsync<List<Country>>($"country/all");
    }
    catch (Exception ex)
    {
    }
}

}

🤔 Expected Behavior
either the value should be reflected in userRegister.CountryAlpha2Code
OR
the list should not default after loading or changing the options and the user should be able to select the entry after loading the options.
please note i am able to select and things work if i returned two entries from the Api call instead of one.
😯 Current Behavior
Can't select this single option no matter what i do.
💁 Possible Solution
I am willing to fix this.
🔦 Context
provide a user a way to select from a list of values, without allowing any value outside of this list.
🌍 Your Environment
Tested on MAUI windows and android emulator.
Microsoft.FluentUI.AspNetCore.Components 4.3.1

Please supply us with a working minimal reproduction. Don't expect us to try andcompose an environment ourselves. We have to many things going on to do this. Help us to help you.

@vnbaaij please find the code below, this is a home page of a new MAUI project, just added Microsoft.FluentUI.AspNetCore.Components 4.3.1 with all needed code then modified home page to this, the selectedValue... should be US but a user can't never pick that although the select dose default to it visually. this is not an issue if you have two options but even then the select box default visually to the first value without actually changing bind-value, the value only change if a user a click, with one option not even that is possible.
@page "/"

<FluentSelect @bind-Value="@selectedValue" Label="Test"
            Items="countries" OptionText="@(i => i.CountryName)" OptionValue="@(i => i.CountryAlpha2Code)" Required>
</FluentSelect>

@selectedValue

@code {
    private string selectedValue;
    private List<Country> countries = new List<Country>();

    protected override async Task OnInitializedAsync()
    {
        try
        {
            //Mocking loading the list from API Call
            //await httpClient.GetFromJsonAsync<List<Country>>
            countries.Add(new Country()
            {
                CountryAlpha2Code = "US",
                    CountryName = "United States of America"
            });
        }
        catch (Exception ex)
        {
            //handle errors if any please ignore
        }
    }
}

country class code if needed
public class Country
{
    public string CountryAlpha2Code { get; set; }
    public string CountryName { get; set; }
}

I can't reproduce the described behavior. Pasted your code in the demo site and ran it:

It might be something MAUI specific. There have also been some changes in the FluentSelect code for v4.4 so that might have helped to solve this issue. Can you try with the latest package?
A select always shows the first option as selected without actually reflecting that in the bound value (before any user interaction)

@vnbaaij yes i can confirm that upgrading to v4.4 did enable me to select the single value now thanks!
still as you screen recording indicate when setting the items on the list, it show the first item as if it was selected, while the binding value do not get updated, i would suggest either set the binding value to match the element visual or even better once a list of items are set the element should still show empty selection reflecting the fact that the user yet to make a selection and that binding value still null.

here's an example on how this is leading to a confusing user experience, as the list show a value as a user i would expect then that this value is selected, but the truth is it's just the element visual without the binding value being updated so the form still not valid when submitting.

I might be having a similar issue, though it's hard for me to identify if it's just a "me" problem since I'm binding values in a roundabout way (with a Dictionary<T, U> form model rather than a well-defined object). In my case, I have a FluentSelect component that only has one item. I can select the item and confirm that the value takes (as in the example above); however, when submitting the containing form, the field does not seem to be recognized as having been changed and taken the value:

Does anyone have insights into whether this behavior for single-value select lists is expected within an EditForm?

@alfred-zaki sorry, but that is just the way the component (and also the underlying web component) works. It doesn't matter if there is only one option or more. See how it works with the standard web component here: https://fluent-components.azurewebsites.net/?path=/docs/components-select--select. The first one is shown, looks selected but isn't THE value selected yet. Nothing we can/want to change here.

@thebarrettlo is should not work differently when there's is only one value. Did you try with the latest version? As @alfred-zaki confirmed already if works with that one

Apologies, I had that comment written out yesterday but only submitted it today without refreshing the page and seeing @alfred-zaki's update. I am on version 4.4.1, so it must be the way I'm binding values. After looking through the code for FluentSelect and ListComponentBase, I thought that I might be able to trigger a change event for the underlying components using the Blazor OnAfterRender() hook:
private FluentSelect<Enum> _select; // Reference to FluentSelect component using @ref
private IEnumerable<Enum> EnumValues { get; set; }

...

protected override void OnAfterRender(bool firstRender)
{
    if (firstRender && EnumValues.Any())
    {
        _select.SelectedOptionChanged.InvokeAsync(EnumValues.First());
    }
}

Inspecting _select in the debugger shows FluentSelect.InternalValue and FluentSelect.Value updating from null to one of the enum values; but unfortunately, this still doesn't seem to notify the form/component of the state change.

@thebarrettlo maybe better to create a new issue then (with repro code please)? I'm closing this one as @alfred-zaki confirmed it is resolved.


Fixed #214: Changed the Eureka client version to the latest version

This PR will fix #214 by changing the eureka client version. Currently I updated the version inside gradle.properties and used that variable inside the karyon2-eureka project.

NetflixOSS » karyon » karyon-pull-requests #33 SUCCESS
This pull request looks good

Thanks for spotting this.


This is very out of date

This is very out of date, and doesn't include your new regions like WestUS3.  Please refresh it asap.

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

ID: 311b6efb-2d93-6130-96a9-867fad56bd52
Version Independent ID: a17faabf-04c6-c3da-9a37-a86b0d00cbfd
Content: Azure network round-trip latency statistics
Content Source: articles/networking/azure-network-latency.md
Service: virtual-network
GitHub Login: @nayak-mahesh
Microsoft Alias: kumud

Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

A doc bug for the Azure Latency chart issues regarding out-of-date information has been created.  Since there are multiple issues related to this page, I am currently investigating this issue with the Product team and will be closing all the open issues.
When a resolution is published, I'll report back here.
#please-close


Fix superfluous exception when socket read returns less than the requ…

…ested number of bytes
This problem was observed when using the implementation on Motorola MotoG and MotoE phones. The initial call to read(buffer, 0, length) with a 3-byte request would only return 1 byte. This would result in a WebSocketException and closing of the socket. However, a subsequent call to read() would return the remaining bytes.

Thank you very much for your finding the bug.

I'll release a new version (1.4) soon.

Version 1.4 was released. (nv-websocket-client|1.4|jar)


Data source Updation from Oracle to SQL DB

Hi ,
We have around 50 Tabular Models which have Oracle as the main Data source .
Now we have completely moved to Azure and now we want the Cube to use  Azure SQL Data Warehouse Connection.
In the image attached i can update all the highlighted items using the Scripting option in one go.
Some things i am struggling to update is the kind option inside tabular editor as that option is not present in data source credentials . Another thing i am not able to do is bulk update of all table partitions schema name from DWSSAS (Oracle)  to dbo (SQL ADW)  as can be seen in schema screenshot. Can you help me with any workarounds or scripts .

Regarding the data source credentials, please make sure to update to the latest version 2.11.6. This lets you access all properties of the Credential dictionary like so:
var ds = Model.DataSources.First as StructuredDataSource;
ds.Credential["Kind"] = "SQL";

The partition schema name is more tricky. If “DWSSAS” does not occur anywhere else in the M expression, you can use a simple string replace:
foreach(var p in Model.AllPartitions.OfType<MPartition>())
{
    p.Expression = p.Expression.Replace("DWSSAS", "dbo");
}

@otykier : This works perfectly.
Thanks a lot again almost saved a week's work for us .


Chatdev gets stuck

I ran a task that got stuck  at this point:
💰cost=$0.065786
🔨version_updates=-1
📃num_code_files=2
🏞num_png_files=0
📚num_doc_files=4
📃code_lines=41
📋env_lines=-1
📒manual_lines=-1
🗣num_utterances=28
🤔num_self_reflections=0
❓num_prompt_tokens=10922
❗num_completion_tokens=8255
🌟num_total_tokens=19177
[Execute Detail]
execute SimplePhase:[TestErrorSummary] in ComposedPhase:[Test], cycle 1
Could you suggest what could be the issue?

Can you describe the steps you took which created the issue? What configs did you use, what prompt, etc. Does it seem to consistently get stuck at this point, or is it sporadic?

I think I figured it out. I tried reinstalling the packages from requirement.txt and everything seems to be okay


replace keycloak-config Dockerfile with fabric8's docker maven plugin

In this experiment, I replaced the multi-stage docker build (docker on
top) with a maven configuration that builds the docker file (maven on
top).
Signed-off-by: Lee Surprenant<EMAIL_ADDRESS>

this branch has gone stale.  this could get revisited after upgrading to KeycloakX


Design view does not work after turning on computer for the first time in a month.

Does this occur consistently?
Repro steps:

I followed the install guide, including turning off auto-update.
Come back from vacation.
Logic Apps design view cannot open. I get a very unspecific error.

I tried uninstalling everything and reinstall: Same issue.
I tried installing the latest version of Azure Functions and the add the VSIX-file: Same issue.
The only thing I have not tried (because I do not know how) is to: "please uninstall them first or make sure that the PATH environment variable points to the specific version here" as there are no values in the PATH variable to indicate any version of Functions.
I just want to get something working for a demo, and I have already uninstalled everything, is there anything else I can do?
Action: azureFunctions.openDesigner
Error type: Error
Error Message: {}
Version: 0.22.1-alpha
OS: win32
OS Release: 10.0.18363
Product: Visual Studio Code
Product Version: 1.48.0
Language: en

Call Stack
Object.<anonymous> extension.bundle.js:164:427722
fulfilled extension.bundle.js:164:424631
processTicksAndRejections task_queues.js:85:5

The official Azure Functions extension does not support this scenario. How did you get the logic apps functionality? I would follow up through those channels as my team isn't directly working on that

Oops! Sorry.
This was in conjunction with a preview of a possible next for Logic Apps. The report was filed directly from VS code, and I thought it got routed to those guys.
From: Eric Jizba<EMAIL_ADDRESS>Sent: den 17 augusti 2020 23:42
To: microsoft/vscode-azurefunctions<EMAIL_ADDRESS>Cc: Mikael Sand<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/vscode-azurefunctions] Design view does not work after turning on computer for the first time in a month. (#2305)
The official Azure Functions extension does not support this scenario. How did you get the logic apps functionality? I would follow up through those channels as my team isn't directly working on that
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode-azurefunctions/issues/2305#issuecomment-675130024, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE7EMJYPT6YPGN6KQZSPXWDSBGP3VANCNFSM4QBVGKKA.


grt creates guides where only a single layer of metal is available, causing shorts in drt

Describe the bug
I think this is a similar issue to https://github.com/The-OpenROAD-Project/OpenROAD/issues/1518 however in this case it's the top layer of metal (Metal5) and there is an obstruction covering Metal4. This test is on GF180.
drt is finishing with one short, seen just to the right of the middle of this screenshot:

The macro has an obstruction on Metal4, and grt has created guides for two wires on Metal5:
_027524_

and
microwatt_0.soc0.processor.execute1_0.divider_0_div_0.d_in_divisor\[22\]

It would be a pity not to use all that free space on Metal5, but on the other hand it seems extremely easy to get stuck with unroutable situations in drt.
Test case:  metal5-grt-drt.tar.gz
Expected Behavior
No DRCs from drt
Environment
Git commit: bdb90cc3f51a1a866234817effb6ac129d9bc0c8
os: Fedora Linux 36 (Workstation Edition)
cmake version 3.24.2
gcc (GCC) 12.2.1 20221121 (Red Hat 12.2.1-4)
clang version 14.0.5 (Fedora 14.0.5-1.fc36)

To Reproduce
See test case above.
Relevant log output
No response
Screenshots
No response
Additional Context
No response

The problem isn't the guides but that drt doesn't follow them as it sees a much shorter path.  One of the shorted nets:

and the other

Perhaps we should increase the out-of-guide cost when in a planar routing area.  Note that the guides are pretty ugly and suggest major detouring due to congestion.

Another option is to remove/cost the wrong way edges higher in the planar regions

Perhaps we should increase the out-of-guide cost when in a planar routing area. Note that the guides are pretty ugly and suggest major detouring due to congestion.

I wonder if we should test an increase to GUIDECOST. If there is little cost for drt to stay on the grt guide, then drt could drastically change the wire length (as it has above) and create hold violations.
A change to GUIDECOST could negatively impact some designs, but I bet in a lot of cases the root cause is the huge derating we often apply to grt (eg 50% by default in ORFS)


Alexey-Ben meeting 2019-02-04

@ashiklom

Do you want to go to April NCAR/NEON/University workshop on ecological forecasting?
UMBS meeting Friday-Saturday -- let's look at Chris's slides

April NCAR/NEON/University workshop on ecological forecasting

Wasn't aware of this. Here's the [link](April NCAR/NEON/University workshop on ecological forecasting) for reference. It's the same week as IALE (my invited presentation in Ft. Collins), but depending on when my talk is, I might be able to attend most of the days anyway. Let's discuss.

UMBS meeting Friday-Saturday -- let's look at Chris's slides

Sounds good. Assuming these are on your drive somewhere? I also started putting together a (very rough) slide about our modeling work. Here's the link. We can discuss, and I can revise over the next few days.
Updates from last week

Generally, not my most productive week 😞
FoRTE

Spent most of my time on outline. Includes brain dump of introduction, and notes on ED modules.
A few hours reading and thinking about further parallelizing ED runs. Some related PEcAn development.

Hector -- Didn't work on

Goals for this week

FoRTE

Look at ED ensemble output (even not all ensembles done). Chances are, there's something wrong with the output, so figure out what and how to fix it.
Some simple benchmarks for running ED -- are ED runs really that slow, or was there something wrong with my setup? Consider whether I need to invest time in speeding things up.
Paper -- Refine introduction, start creating a proper methods section
(Time-permitting) Try short runs of various ED sub-models to see which ones can run without crashing (and what parameters they need, etc.)

Hector

More detailed outline of next steps
Figures and notes (some version of outline above) for Corinne's EPA meeting next week
Background reading, with a focus on how I might build a simple permafrost model


ESPHome does not git pull updated model during re-compile

In trying to experiment with a handful of candidate models and probability cutoffs, I noticed that ESPHome wasn't pulling an update from my model GitHub repo. Rather, it only seems to use the last model and Jason downloaded from the exact URL provided, even going back several merges to main. I'm not sure if that's a microwakeword issue or an ESPHome issue.
Additionally, it would seem that the ESPHome firmware for the ESP32-S3-BOX-3 does not pass through yaml substitutions of probability_cutoff and sliding_window_average_size like it does model. Is there a direct way to substitute these?

Are you doing this via external_components? If so, you need to set the refresh parameter to force an update.

I'm pulling in:
packages:
  esphome.voice-assistant: github://esphome/firmware/wake-word-voice-assistant/esp32-s3-box-3.yaml@main

That yaml uses the micro_wake_word component of ESPHome directly. So the issue seems to be inside there.

I'm not sure what is going on with ESPHome and GitHub. I've been encountering this all day where ESPHome fails to download something from GitHub unless I retry many times. It's not just limited to to mWW models, I'm having issues whenever it tries to install a PlatformIO library, refresh an external component, or download an image (like for the S3 boxes), so I don't believe it is particular to microWakeWord.
The easiest way to adjust the probability_cutoff and sliding_window_average_size would be to copy and paste the entire yaml file into a new device in the ESPHome dashboard. You can then adjust those parameters directly in the copied yaml at the appropriate spot.

there's a "refresh" option for remote packages too: https://esphome.io/guides/configuration-types.html#remote-git-packages

Also you can override substitutions when including a file via the vars option, e.g. https://esphome.io/guides/configuration-types.html#packages-as-templates

But I think this is an ESPHome question. Not a microWakeWord one.

I haven't been having issues the last few days with ESPHome pulling things from GitHub. If you are still encountering this, please open an issue in the ESPHome issues repository.


Can't establish CASE, rs-matter unable to verify ICAC certificate that's missing subject key ID extension

I'm trying to commission a test device utilizing this crate with a SmartThings Hub. It's currently failing to establish CASE, and I've tracked it down to the verification of the initiator ICAC sent by the controller to the device.
This is what I could find in the spec about validating the Sigma3 message:
The responder SHALL verify that the NOC in TBEData3.responderNOC and the ICAC in TBEData3.responderICAC fulfill the following constraints:
a. The Fabric ID SHALL match the Fabric ID matched during processing of the Destination
Identifier after receiving Sigma1.
b. If an ICAC is present, and it contains a Fabric ID in its subject, then it SHALL match the Fab
ricID in the NOC leaf certificate.
c. The certificate chain SHALL chain back to the Trusted Root CA Certificate TrustedRCAC whose
public key was matched during processing of the Destination Identifier after receiving Sig
ma1.
d. All the elements in the certificate chain SHALL respect the Matter Certificate DN Encoding
Rules, including range checks for identifiers such as Fabric ID and Node ID.

The only validation specified for ICAC is that the fabric ID matches that of the NOC leaf certificate, which appears to be done here: https://github.com/project-chip/rs-matter/blob/320d1ec9892a6f32aebf5a2506f18d112cdc3573/rs-matter/src/secure_channel/case.rs#L383
I've verified that that is occurring with log statements. However, just below that, it is failing to add the cert to the verifier. From adding my own logs, I've determined that it is failing here: https://github.com/project-chip/rs-matter/blob/320d1ec9892a6f32aebf5a2506f18d112cdc3573/rs-matter/src/cert/mod.rs#L600
The ICAC has no subject key ID extension, so the function returns an error. Given that the ST Hub is Matter certified I suspect that something is overly strict with this certificate verification. However, this is out of my realm of expertise.
I also logged the ICAC following the parsing from TLV in case that's helpful:
Cert {
    serial_no: OctetStr([67, 38, 73, 198, 26, 31, 20, 101, 57, 46, 16, 143, 77, 160, 128, 161]),
    sign_algo: 1,
    issuer: DistNames {
        dn: [(20, Uint(15512483513290480383))]
    },
    not_before: 718486395,
    not_after: 874187555,
    subject: DistNames {
        dn: [(19, Uint(14440658877384716286))]
    },
    pubkey_algo: 1,
    ec_curve_id: 1,
    pubkey: OctetStr([4, 88, 188, 13, 87, 50, 3, 213, 248, 182, 12, 240, 164, 220, 127, 150, 65, 81, 244, 125, 24, 48, 203, 83, 111, 133, 175, 182, 10, 40, 80, 147, 28, 39, 121, 183, 61, 159, 178, 231, 133, 75, 189, 143, 136, 191, 254, 115, 228, 186, 129, 56, 137, 213, 177, 13, 46, 97, 202, 95, 41, 5, 16, 24, 228]),
    extensions: Extensions {
        basic_const: Some(BasicConstraints { is_ca: true, path: Some(0) }),
        key_usage: None,
        ext_key_usage: None,
        subj_key_id: None,
        auth_key_id: Some(OctetStr([243, 119, 107, 152, 3, 212, 205, 76, 85, 38, 158, 240, 27, 213, 11, 235, 33, 21, 38, 5])),
        future_extensions: None
    },
    signature: OctetStr([70, 43, 150, 195, 194, 170, 43, 125, 91, 213, 210, 221, 175, 131, 131, 85, 22, 247, 213, 18, 101, 189, 30, 134, 20, 226, 217, 145, 41, 225, 181, 150, 28, 200, 52, 237, 218, 195, 144, 209, 205, 73, 88, 114, 139, 216, 85, 170, 63, 238, 164, 69, 35, 69, 39, 87, 211, 234, 57, 98, 19, 43, 13, 0])
}

Hmm, went hunting through the C++ SDK and this seems relevant: https://github.com/project-chip/connectedhomeip/blame/f47ce538dcbb37b3bda40f15cca587f0c01cbe0b/src/credentials/CHIPCert.cpp#L191
Looks like the SKID is not optional in this instance. I wonder what's going on there.

I wrote a small little program that re-parsed the decrypted sigma3 packet above using the functions in this crate and then also parsed the sample ICAC cert in the Matter spec. The parsed sample ICAC matched what was indicated there so I can't see a way that the parsing is incorrect in that area.
I'm puzzled, but inclined to close this for now until I can determine more information that might indicate it's an issue on the device-side (rs-matter implementation). For now I'll be suspicious of the hub.

Ah, one of my colleagues discovered the source of the issue. If you serialize the TLV as JSON, it reveals that the ordering of the keys is not consecutive:
{
  "1": [
    9,
    // snipped
    211,
    13
  ],
  "2": 1,
  "3": [
    {
      "20":<PHONE_NUMBER>776288360
    }
  ],
  "4": 719224012,
  "5": 875388955,
  "6": [
    {
      "19":<PHONE_NUMBER>2623729068
    }
  ],
  "7": 1,
  "8": 1,
  "9": [
    4,
    // snipped
    210,
    137
  ],
  "10": [
    {
      "1": {
        "1": true,
        "2": 0
      }
    },
    {
      "5": [
        250,
       // snipped
        252,
        46
      ]
    },
    {
      "4": [
        115,
        // snipped
        188,
        230
      ]
    },
    {
      "2": 96
    }
  ],
  "11": [
    186,
   // snipped
    223,
    99
  ]
}

In particular, the SKID is contained in 10/5 which comes before 10/4, so it's dropped during the parsing. For lists, the TLV formatting section of the spec simply says:

The meanings of member elements in a list are denoted by their position within the list in conjunction with any associated tag value they may have.

So it seems that TLV lists should be able to be encoded validly in this way, and it seems the C++ SDK agrees since it is able to parse it without issue.

Sorry I didn't get time to do the write-up @ssnover. Agreed on what you said, and further, the spec also says (A.5.1)

The encoded ordering of members in a structure may or may not be important depending on the intent of the sender or the expectations of the receiver. For example, in some situations, senders and receivers may agree on a particular ordering of elements to make encoding and decoding easier.
Where a distinguished ordering of members is required (for example, for the purposes of generating a hash or cryptographic signature of the structure), the members of the structure SHALL be encoded as specified in Canonical Ordering of Tags.

which I think strongly implies that tags should be assumed to be unordered by default.


no failover when master node is down

test with my cluster.
when mysql master pod is down, it failover to other pod successfully.
but when the node which runs master pod is down, it doesn‘t failover.
that is unexpected.
what maybe wrong?
thank you.

Please attach the orchestrator log.


neovim-qt: move to pkgs/by-name

Description of changes
2nd attempt of https://github.com/NixOS/nixpkgs/pull/309074

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 316408 run on x86_64-linux 1

1 package built:

neovim-qt

@AndersonTorres gentle ping :)
NB to potential merger: Dont forget to squash before merge

NB to potential merger: Dont forget to squash before merge

According to the guidelines, this is part of your job.

thank you !


AgileTs esm is transformed into commonjs instead of using the commonjs files directly

🐛 Bug report
🤖 Current Behavior
AgileTs isn't importable into commonjs
-> esm modules are imported and then transformed into commonjs which causes issues (like see image)

🎯 Expected behavior
AgileTs should be importable into commonjs without problems
📄 Reproducible example
AgileTs documentation
💡 Suggested solution(s)
https://the-guild.dev/blog/support-nodejs-esm
  "exports": {
    "./package.json": "./package.json",
    ".": {
      "types": "./dist/index.d.ts",
      "module": "./dist/esm/index.js",
      "default": "./dist/index.js"
    },
    "./*": {
      "types": "./*.d.ts",
      "module": "./esm/*.js",
      "default": "./*.js"
    }
  },

Specify what export to use in which import scenario.

Has something to do with nasty circular dependencies..


Add @author rule in checkstyle

According to code-conventions, "@ author" tag is not allow in source code.
I notice some PR and some test code still have "@ author" tag, considering we already have checkstyle plugin, I suggest add a new rule to check @ author tag. it can save time to review PR.
Some configuration sample:
https://github.com/danielb987/EmojicodeEditor/blob/master/checkstyle/emojicode_checks.xml#L93

Agree with this proposal.
I think:

We should check and remove present author tag.
The same thing should be done in branch 2.6.x.
:)

@htynkn
I think we should post this proposal to mail list.

@htynkn I suggest running checkstyle and rat plugin during release phase. For daily travis ci, just running mvn test, it needs to be completed ASAP.
Now the ci task takes a bit long time.

An approach I have seen taken in other projects is to remove @author tags but move the names to the appropriate POM section (if they are not already present).

I've enabled checkstyle and rat plugin for default travis CI.

https://github.com/apache/incubator-dubbo/pull/2393


Add note regarding content-type detection when payload is used

Add note to README covering case where Mailman incorrectly detected an email notification with an included payload as binary. Share hotfix of explicitly setting text/plain value for Content-Type header.

May need to add the same note to related projects.


Update the license of the Java language binding

Context
Entur has been gracious enough to transfer their pre-existing Java language binding codebase to MobilityData (https://github.com/MobilityData/gbfs-json-schema/issues/109). Entur codebase was under the license 'EUPL-1.2 with modifications'.
To do
Update the license of the Java language binding (https://central.sonatype.com/artifact/org.mobilitydata/gbfs-java-model) to 'Apache-2.0'.

Before
After

EUPL-1.2 with modifications
Apache-2.0

To be more permissive and consistent with the other MobilityData codebases.

Done in https://github.com/MobilityData/gbfs-json-schema/pull/115


minimal node.js engine

Recently (in 0.16.0 release) the node.js requirement was set at >=18.17.0 in package.json.
Is there any specific reason to set such an higher minimal node.js version?
I'm asking this because in some of my deployments I use Debian's pakaging of node.js, and in bookworm the version that is packaged is 18.13.0 (https://packages.debian.org/bookworm/nodejs).

Hey @JoseGoncalves
This was changed as part of an overhaul to our build system but it doesn't necessarily need to be that high. We can certainly relax the requirement if it's causing you issues.

Hi @HusamIbrahim
AFAIK, the only really "issue" is to receive the following warning when performing a npm i:
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: '>=18.17.0' },
npm WARN EBADENGINE   current: { node: 'v18.13.0', npm: '9.2.0' }
npm WARN EBADENGINE }

Maybe setting to >=18.13.0 would be better to avoid the warning to users that use the nodejs package from Debian 12 (bookworm) or Ubuntu 23.04 (lunar).

Honestly I think we can set it as low as >=16.11.0 to match Vue's if we can. Our other dependencies don't specify a requirement for node versions so if we can go as deep as Vue's that'd be great.


Remove dependency on redis server endpoint

Fixes https://github.com/rollbar/rollbar-gem/issues/724
It looks like the main point of this rspec example is to serialize the object, but it's this single call to Redis#set that requires having a live redis server.

@waltjones the Redis dependency came from this https://github.com/rollbar/rollbar-gem/pull/254. We had problems serializing the web session when there was some TCP connection on the Rails session object. That happened for some customers depending on the storage used for the session.
The .set() operation I think was added in order to start a real TCP connection to the Redis server and then check the change was working correctly. So...I think removing that .set() operation is not safe unless we've fixed the problem in other way or we plan to change the JSON serialization library

Thanks for info @jondeandres . I'm going to work on this and see if it's safe to remove the actual Redis instance.

@jondeandres I found the root cause of the issue. Here is more info:
https://github.com/rollbar/rollbar-gem/issues/724
https://github.com/redis/redis-rb/issues/830
@waltjones this is a great improvement for the specs. However, until the Redis issue is fixed I would rather have your new test and the previous test there so it's easier for us to test if the upcoming solution for the Redis problem works as expected. Does it make sense?

@ArturMoczulski I agree with you on having a TCPSocket test. (Great work on tracking down the root cause by the way.) However, my testing appears to show that the original test case wasn't exercising this condition. item.dump is actually using ActiveSupport JSON, but doesn't hang during serialization as you'd expect.

You mean it doesn't hang when you remove TCPSocket our of the equation? Or does it not hang for you even when serializing TCPSocket?

I brought up this issue on Ruby's issue tracker: https://bugs.ruby-lang.org/issues/15636

I have added back a test case for the Redis issue, independent of having a running redis server. The hard part (for me) was getting the original issue to repro. It turns out the specific ActiveSupport issue only occurs on ActiveSupport 4.1.x - 5.1.x. Also, it wasn't sufficient to use an instance of Redis::Connection::TCPSocket. For some reason, the issue only happens when the socket is wrapped by a Redis::Connection instance.

Thanks to feedback from Ruby maintainers https://bugs.ruby-lang.org/issues/15636#note-1 I understand fully what is the issue here now. to_a on a socket serves a different purpose then converting the instance to an array. Instead, it's supposed to read lines from the IO and split them into lines. Ruby doesn't hand in infinite recursion. It hangs waiting for input. ActiveSupport 4.1.x - 5.1.x uses to_a in this particular case for the wrong reason - obviously, it's an edge case for them.
Not sure what is the best solution for this situation. I'm thinking about temporarily overriding some methods on the ::Socket instance for the purpose of serialization and then reverting when serialization is done.

This is the solution I implemented:
I override ::BasicSocket#as_json for active_support < 5.2 in a scoped manner ensuring that the monkey patch is only used during the serialization and doesn't creep into the user app or rest of Rollbar code: https://github.com/rollbar/rollbar-gem/issues/845
Doing this required implementing scoped plugin loading feature as described here: https://github.com/rollbar/rollbar-gem/issues/846
@waltjones @rokob please review

@waltjones last round of feedback and hopefully this can be merged in

Yes, looks good to merge.


Is there an implementation of this lib in PHP?

Do you know an implementation of this lib in PHP?
To perform intercommunication between JS and PHP.

No, I do not provide any PHP library at the moment.


3D probe summary view

@rossant, is it possible to make scatter plots with 3 dimensions in phy? If so, I have a suggestion for one that could make a really nice probe overview. Here's how it works:
For each spike, compute two things: 1) its peak channel (i.e. channel with largest waveform), and take its PC1 and PC2 from that channel; 2) its position on the probe in the y-dimension, computed as a center of mass of its PC1 projections across channels. Then the plot is: PC1_peakChan vs PC2_peakChan vs yPosition. Coloration could be with a unique color per cluster, or only coloring the selected ones - just as in traceview, basically.
@marius10p this is a sort of "shortcut" version of the t-sne idea we talked about earlier today. Do you think it makes sense?
@rossant let me know if 3D scatters are possible.
If both of you come back positively, I will make a version in matlab to check that it actually looks reasonable and useful (maybe not, but let's see), then I'll come back here with a more detailed description of what exactly to plot if it looks good.

3D plots are not implemented at the moment but I can do it if that's really necessary. The only caveat is that 3D typically requires user interaction which takes more time than just looking at static figures. Or we can animate it by default. Just something to keep in mind.

Ok, I tried this briefly, but sticking to a 2D view for the reasons @rossant has given. Actually it was very simple to make and produces an attractive and useful plot:

I think this is useful because we currently don't have any other way to just "see everything around here".
To get this plot I first estimate the position of each spike, which I do by taking a center of mass of the PC's (first PC only), exact code below. The x-axis is just the value of the first PC on whatever channel is declared as the top channel for that spike.
I guess the only real question is how to handle the large number of points that would be present, like whether there's a way to adaptively add more points as the user zooms in. Or if not, maybe representing the same space as a density plot would be better.
pcFeat = squeeze(pcFeat(:,1,:)); % take first PC only
pcFeat(pcFeat<0) = 0; % some entries are negative, but we don't really want to push the CoM away from there.

% which channels for each spike?
spikeFeatInd = pcFeatInd(spikeTemps+1,:); % +1 is because matlab uses one-indexing

% ycoords of those channels?
spikeFeatYcoords = ycoords(spikeFeatInd+1);

% center of mass is sum(coords.*features)/sum(features)
spikeDepths = sum(spikeFeatYcoords.*pcFeat.^2,2)./sum(pcFeat.^2,2); % using features squared, I just thought it worked better

xData = pcFeat(:,1); % that's it - just take the first channel

can be done as a spike attribute in the dev branch


Running the project with docker compose fails due to migration problems

After running docker compose up the peermetrics-api and peermetrics-postgres containers fail because of non-existing user relation.
Here are the docker logs:

peermetrics-api-1       | Operations to perform:
peermetrics-api-1       |   Apply all migrations: admin, auth, contenttypes, sessions
peermetrics-api-1       | Running migrations:
peermetrics-postgres-1  | 2024-03-27 15:28:16.728 UTC [44] ERROR:  relation "users" does not exist
peermetrics-postgres-1  | 2024-03-27 15:28:16.728 UTC [44] STATEMENT:  ALTER TABLE "django_admin_log" ADD CONSTRAINT "django_admin_log_user_id_c564eba6_fk_users_id" FOREIGN KEY ("user_id") REFERENCES "users" ("id") DEFERRABLE INITIALLY DEFERRED
peermetrics-api-1       | Traceback (most recent call last):
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 84, in _execute
peermetrics-api-1       |     return self.cursor.execute(sql, params)
peermetrics-api-1       | psycopg2.errors.UndefinedTable: relation "users" does not exist
peermetrics-api-1       |
peermetrics-api-1       |
peermetrics-api-1       | The above exception was the direct cause of the following exception:
peermetrics-api-1       |
peermetrics-api-1       | Traceback (most recent call last):
peermetrics-api-1       |   File "manage.py", line 15, in
peermetrics-api-1       |     execute_from_command_line(sys.argv)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/core/management/init.py", line 381, in execute_from_command_line
peermetrics-api-1       |     utility.execute()
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/core/management/init.py", line 375, in execute
peermetrics-api-1       |     self.fetch_command(subcommand).run_from_argv(self.argv)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 323, in run_from_argv
peermetrics-api-1       |     self.execute(*args, **cmd_options)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 364, in execute
peermetrics-api-1       |     output = self.handle(*args, **options)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 83, in wrapped
peermetrics-api-1       |     res = handle_func(*args, **kwargs)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/core/management/commands/migrate.py", line 232, in handle
peermetrics-api-1       |     post_migrate_state = executor.migrate(
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/migrations/executor.py", line 117, in migrate
peermetrics-api-1       |     state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/migrations/executor.py", line 147, in _migrate_all_forwards
peermetrics-api-1       |     state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/migrations/executor.py", line 247, in apply_migration
peermetrics-api-1       |     migration_recorded = True
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/backends/base/schema.py", line 110, in exit
peermetrics-api-1       |     self.execute(sql)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/backends/base/schema.py", line 137, in execute
peermetrics-api-1       |     cursor.execute(sql, params)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 67, in execute
peermetrics-api-1       |     return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 76, in _execute_with_wrappers
peermetrics-api-1       |     return executor(sql, params, many, context)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 84, in _execute
peermetrics-api-1       |     return self.cursor.execute(sql, params)
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/utils.py", line 89, in exit
peermetrics-api-1       |     raise dj_exc_value.with_traceback(traceback) from exc_value
peermetrics-api-1       |   File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 84, in _execute
peermetrics-api-1       |     return self.cursor.execute(sql, params)
peermetrics-api-1       | django.db.utils.ProgrammingError: relation "users" does not exist
peermetrics-api-1       |
peermetrics-api-1 exited with code 1

There might be a simple solution to this but I'm not that familiar with Django and its migration process, and I've not seen any sign that configurations should be modified in order to make it work, so I suppose it's an issue.
Thanks in advance!

That's a good catch @scarecrow98. Looking into it

Dashboard up and running, works like a charm! Thanks for you fix, looking forward to see this thing in action! :)
Closing the issue!


ISPN-11873 Add report failure for RxJava Undelivered Errors

https://issues.redhat.com/browse/ISPN-11873
Reviving https://github.com/infinispan/infinispan/pull/8356, only without the while loop we were discussing in the comments.

Looks like the RxJavaPublisherTest is still failing https://ci.infinispan.org/job/Infinispan/job/PR-9036/1/testReport/junit/org.infinispan.reactive/RxJavaPublisherTest/Undelivered/
The others I am ignoring as they will need to be remedied in another PR.

@wburns I changed the unwrapping of exceptions again to only remove UndeliveredException and it seems to work fine. I'm thinking maybe I should add back the CompletionException check as well, but I'm hesitating because even without it no exception is reported.

Looks good to me. We can iterate upon this if needed.

Integrated into master, thanks @danberindei !


[CAMEL-11946] Invalid artifactId in docs for camel-json-validator

Merged on camel-2.20.x and master. Thanks @tdiesler :-)


Client disconnect when the Mouvet steals food...

Describe the bug
When you get your food stealed by the Mouvet the client disconnect...
To Reproduce
Steps to reproduce the behavior:

Start a server
Connect into
Get food stealed by the Mouvet

Desktop (please complete the following information):

Forge Verison: 1.20.2 48.x.x
OS: All
Dimensional Expansion Versions: Sine the mouvet can steal food

Fixed on this commit.


[BUG] Capitalisation needed for the "verify" button

Describe the bug
The "verify" button would look better if it was labeled as "Verify"
To Reproduce
Steps to reproduce the behavior:

Go to 'https://reach4help-staging-web-client.firebaseapp.com/" under a new Facebook user
Enter a valid phone number
Land on "https://reach4help-staging-web-client.firebaseapp.com/phone/verify"
Check the "verify" button

Expected behavior
A cap "V" would make the button look better
Screenshots

Desktop (please complete the following information):

OS: W10
Browser Chrome
Version: 83.0.4103.61

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.

Verified Fixed v2.1.0


Throwing exeption when it cannot determine the file type.

Methods which return FileType from (MimeTypes.cs) usually have this comment:
	/// <summary>
	/// Read header of a file and depending on the information in the header
	/// return object FileType.
	/// *Return null in case when the file type is not identified.*
	/// Throws Application exception if the file can not be read or does not exist
	/// </summary>

But because of that line https://github.com/clarkis117/Mime-Detective/blob/master/src/Mime-Detective/MimeTypes.cs#L314 this statement about return value isn't true.

This is addressed in 0.0.3,  thanks for raising the issue :)


Allow showing points and list of points as coordinates

Unfortunately this is going to be tricky because there's not already an evaluation there, and the label means we can't just override shouldShowEvaluationForItem: getFooterType prioritizes a label view over the evaluation, so getFooterType would need to be overridden instead.

I'm currently working on this functionality, should be ready pretty soon. Considering merging label and evaluation view footers but that'll be a bit difficult.
(if you have another idea, please lmk! just wanted to avoid duplicate work)

What's your status on this? I'm guessing a bunch of merge conflicts.

Ah this, had some messy stuff but there's a better way to do it. Currently working on other stuff but perhaps will revisit


Memory leak for json encode

#!/usr/bin/env luajit
local json = require "resty.libcjson"
local tbl  = {hello = "world"}

for _ = 1, 100000000 do
    local json_str = json.encode(tbl)
end

Run the above code and observe. The memory usage keeps growing during repeated json encode.
However, json decode works just fine.
The version of luajit and lua-resty-libcjson is:

$ luajit -v
LuaJIT 2.0.2 -- Copyright (C) 2005-2013 Mike Pall. http://luajit.org/
$ luarocks list lua-resty-libcjson
Installed rocks:
lua-resty-libcjson
1.1-1 (installed) - /usr/local/lib/luarocks/rocks

And cJSON is download from http://sourceforge.net/projects/cjson/files/latest/download
This issue also can be reproduced by openresty <IP_ADDRESS>.

Yes, the encode is like total mess with this. On this more up-to-date version there is:
https://github.com/kbranigan/cJSON/commit/dbd0a7098164a2cc118a9f4ef64656f380007ad9

Hi, I added small patch, can you check if it helps, also try out kbranigan/cJSON.

I already checked, and it doesn't help.

This leak is actually pretty bad. In my experience under moderate load (<100 rps), when doing one encode per request this can easily leak 100-150 megs per worker per day.
I had to switch to pure-lua lunajson lib, which doesn't leak.

Yeah, I think the leak is in C-module, and there is little I can do. On OpenResty the bundled Lua cJSON is the preferred lib.

If I change code to this:
function json.encode(value, formatted)
    local j = ffi_gc(json.encval(value), cjson.cJSON_Delete)
    if j == nil then return nil end
    print(collectgarbage("count")*1024)
    collectgarbage()
    collectgarbage()
    return formatted ~= false and ffi_str(cjson.cJSON_Print(j)) or ffi_str(cjson.cJSON_PrintUnformatted(j))
end

I get 46465 printed constantly, but the memory grows. So this might be actually a memory leak in LuaJIT.

Ah, looks like the reply by corsix may be the issue. For instance in my C code I have to do:
	char * json_string;
	json_string = cJSON_Print(root);

	int ret = http_write(c,json_string);
	os_free(json_string);

where root is also a malloc'd cJSON object.

@someburner I just committed this as 1.2 and after some cleanups 1.3 that fixes this issue. Thanks a lot.
@ngo, @jonas0616, will you also check if it is fixed now, if not, please reopen this issue. Thanks for reporting.

@bungle Great work, thanks! My short test (~30 min) shows that the memory leak is gone. I will update once I collect more info overnight.

The long test also doesn't show any signs of leaks

Thanks @ngo for checking and confirming that the fix had the desired effect.


Error: Unable to get unit "webApp" as long as SDK is not initialized.

Describe the bug
Just copy pasted what was in the docs to setup a minimul Webapp
When I call this:
in my App
const webApp = useWebApp();
useEffect(() => {
webApp.ready();
}, [webApp]);
To Reproduce
Steps to reproduce the behavior:
Follow the guide here: https://docs.twa.dev/docs/libraries/twa-js-sdk-react
Expected behavior
A clear and concise description of what you expected to happen.
My main.tsx:
import React from "react";
import ReactDOM from "react-dom/client";

import "./index.css";
import { App } from "./app";

ReactDOM.createRoot(document.getElementById("root")!).render(
  <React.StrictMode>
      <App />
  </React.StrictMode>
);

my app.tsx
const options: InitOptions = {
    acceptScrollbarStyle: true,
    checkCompat: true,
    debug: true
  };
const AppWrapped = withSDK(Root);

export function App() {
  return (
    <SDKProvider initOptions={options}>
      <AppWrapped />
    </SDKProvider>
  );
}

my root.tsx

/**
 * This component is the layer controlling the application display. It displays
 * application in case, the SDK is initialized, displays an error if something
 * went wrong, and a loader if the SDK is warming up.
 */
function Loader({children}: PropsWithChildren<{}>) {
    const {didInit, components, error} = useSDK();

    // There were no calls of SDK's init function. It means, we did not
    // even try to do it.
    if (!didInit) {
      return
SDK init function is not yet called.
;
    }

    // Error occurred during SDK init.
    if (error !== null) {
      return
Something went wrong.
;
    }

    // If components is null, it means, SDK is not ready at the
    // moment and currently initializing. Usually, it takes like
    // several milliseconds or something like that, but we should
    // have this check.
    if (components === null) {
      return
Warming up SDK.
;
    }

    // Safely render application.
    return <>{children}</>;
  }

export function Root({ sdk }) {

  return (
    <Loader>

TEST app

    </Loader>
  )
}

Loader gets stuck at Warming up SDK.
When console logging the sdk:
{components: null, didInit: true, error: null}

What Telegram application are you testing this code in?
Have you tried this code in different Telegram applications?

Please, try opening your app in the different Telegram application. I assume, problem is specific to the Telegram app for macOS only. It is probably a bug in the platform, not in the package, but it seems like I know the solution

As I found out with some other external developer, problem occurs due to the incorrect behaviour of the Telegram app for macOS. The reason is the native application is not responding to web_app_request_viewport method, but it must.
The other important note here, that I could not reproduce this problem using the same native application version. I will release the solution today and will notify you about that.

@twa.js/sdk and @twa.js/sdk-react were updated. Try using the latest versions and let me know if my solution doesn't solve the problem


Review Actions interface

Actions is a powerful interface to define a specific event using Action and usually used internally by event firer. Typically, it backs up method like eventnameAction(Action<Event> action) and change in Actions doesn't affect event firer's API. Though, since it's very useful, it would be nice for user to use it for their own purpose. However, the following points should be considered first.

Interface name
Actions is vague name and is named from jQuery's Callbacks object that was inspiring to write Actions interface. ActionStack, ActionPipeline or EventDefiner, and so on looks better. But more candidate should be considered to fix it.

A way to configure option
I'm not sure which way is better among setter, POJO option, option builder, and so on but clearly it's not needed to change such option in runtime as it defines a specific event. Now POJO option object is used.

Flow control
Considering various events, flow of executing action should be able to be controlled by event firer or listener. Sometimes actions should be executed in a row asynchronously or synchronously and also sometimes firing should be stopped by some condition. Many ways are available: 1. A special class extending ThreadLocal to acquire flow controller 2. A special action to control flow 3. A special method on Actions to control flow. We need to find more flow control cases and determine how to achieve that.

Benchmark Netty and Grizzly
Both frameworks have similar component e.g. Pipeline and Filter. See what functionalities should be provided. Also a process to find proper vibe transport in client may be one of usages using Actions.

It's scheduled in beta.

Interesting issue related to Flow control: https://github.com/Atmosphere/atmosphere/issues/1845#issuecomment-72963639


micorsoftfix

Closes: #305

Title : Microsoft Clone
Name: : Bhavy Zala
Idenitfy yourself: : Gssoc'24

Describe the add-ons or changes you've made 📃
Added new folder Microsoft-Clone with README.md.
Update json file in database.
Update README.md in frontend folder.
How Has This Been Tested? ⚙
Self testing..

Screenshots 📷

Type of change ☑

What sort of change have you made:

[] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[] Code style update (formatting, local variables)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] This change requires a documentation update

Checklist: ☑

[x] My code follows the Contributing Guidelines & Code of Conduct of this project.
[x] This PR does not contain plagiarized content.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] My changes generate no new warnings.

Note to reviewers 📄

@Avdhesh-Varshney Please check this new PR and merge !


Bugs on Basilisk v0.2

Current bugs we are aware.
Same flow as from #3368
Sorted by severity (higher, more critical)

[ ] #3790
[ ] #3138
[ ] #3350
[x] #3395
[x] #3291
[x] #3235
[x] #3423
[x] https://github.com/kodadot/nft-gallery/issues/4000
[x] #3752

closing this since we're tracking rest of the issues (not all of them bugs) in

#3648


[question] what happened to event:type?

In old Hoodie, one was able to do hoodie.store.on('add:type', addTypeHandler), is that still supported?
Same for find[All](type) :)

in current hoodie, we changed the order to type:add a long time ago. In new Hoodie, we now only support hoodie.store(type).on('add', handler)

Thanks!
cc @espy


Setting does not save for the Templates or Scripts locations

when updating the advanced settings to tell it where the Template and Script folders are, it does not save the location and always clears out the specified location, leading to template placeholders to fill with weather data from the last day they were edited.
in the screenshot below, the Template setting had been set to Templates and the Scripts setting had the Scripts folder selected.

Thanks for reporting this. I will have it fixed as soon as possible.
Thanks, William

Fixed in v0.3.0
Not sure how I missed that? It was a simple typo, was missing the last character of the settings name causing it to save to the first excludes folder setting.
Thanks for reporting this, William

Plugin has now been added to the community plugins list. BRAT plugin is no longer required to install/update the plugin.
Thanks for all the input, William


[R-package] [ci] cut nrounds in unit tests

I've been looking at our test logs a lot the few days, and saw some opportunities to cut test time a little. This PR proposes cutting nrounds for a few tests to reduce CI time.

I just rebased this to master to get the changes from #3176 . Will merge when it builds.


Wrong display name and resolution

Display (BOE eDP-2-0x09F9) > 1280x720 @ 240Hz (eDP-2)
My display configuration

You should print either 2560x1440 or 1706x960. The name BOE eDP-2-0x09F9 seems strange too.

On Wayland it grabs the make/model from the compositor, so I'm guessing KDE is returning something wonky. Downloading Manjaro as a "close enough" to test this as my EndeavourOS live boot only has X11
If that's true I can just check for KDE and return the EDID values instead.

Seems to just be affecting laptops, I'm guessing the internal screens mean most manufacturers don't bother with the normal way or something? Either way there's definitely some way to recover it, as my laptop's screen comes out as AU Optronics 0x459D whereas the actual panel name is AUO459D so it's damn close. Will need to look at this tomorrow as I've ran out of time for today however.
Dedicated monitors should remain unaffected.

Would prefer not to clutter my drive with random DE's I'll never use for a project that, being realistic only I seem to use on a daily basis. If you take a look at the README showcases, you'll see that my storage is already quite tight.
Maybe will sacrifice my laptop's storage a bit once I'm not traveling as much as I am right now, but that's gonna have to be a bit later.

BTW on X11 crab reports Unknown Unknown and N/AHz, which is unacceptable.

No refresh rate as I simply couldn't find a way to get it from X11, if you know what to query I'm happy to add it.
As for the make/model, these are fetched from your EDID display descriptors so take a look at if it's set (my laptop doesn't have the model set in EDID for example). Tbf I should likely backup to the ID Product Code but I'll have to investigate that later as I am busy for the first half of today.

Also just had a thought for your incorrect resolution; try setting your scale to 100% and seeing what it comes out as. If it ends up being correct, that tells me where I need to look later today instead of me fiddling with it myself.

No refresh rate as I simply couldn't find a way to get it from X11

So why not hide it from the output, instead of displaying a meaningless and ugly N/AHz? Same as GPU VMEM size.

As for the make/model, these are fetched from your EDID display descriptors so take a look at if it's set (my laptop doesn't have the model set in EDID for example).

They are required field. All EDID data should contain them, including laptops.
The marketing name, however, is not required and most laptop builtin monitors dont have it.

Also just had a thought for your incorrect resolution; try setting your scale to 100% and seeing what it comes out as. If it ends up being correct, that tells me where I need to look later today instead of me fiddling with it myself.

You need to look at fractional scale factor. To reproduce this issue, you simply set the scale factor to 1.5
Fastfetch had all these issues. It took me more than 6 months to find a solution. If you think you are smarter, do it.

try setting your scale to 100% and seeing what it comes out as. If it ends up being correct, that tells me where I need to look later today instead of me fiddling with it myself.

You need to look at fractional scale factor. To reproduce this issue, you simply set the scale factor to 1.5

No refresh rate as I simply couldn't find a way to get it from X11

So why not hide it from the output, instead of displaying a meaningless and ugly N/AHz? Same as GPU VMEM size.

If you take a look at the format;

I would rather have it be an obvious Display (Unknown Unknown) where that placeholder can't be found, opposed to Display () where it could be confusing to a user what's the issue.

As for the make/model, these are fetched from your EDID display descriptors so take a look at if it's set (my laptop doesn't have the model set in EDID for example).

They are both required fields. All EDID data should contain them, including laptops'.
The marketing name, however, is not required and most laptop builtin monitors dont have it.
It was because you failed to find the EDID data.

I only check the the marketing name from the display descriptors which ye I now see is incorrect, hence my comment of "Tbf I should likely backup to the ID Product Code"

try setting your scale to 100% and seeing what it comes out as. If it ends up being correct, that tells me where I need to look later today instead of me fiddling with it myself.

You need to look at fractional scale factor. To reproduce this issue, you simply set the scale factor to 1.5

In that case I'm not sure I can fix this, as from what I can tell my wayland client library wayland_client knows about the the protocol but still returns the scale to me remains as an i32 with no obvious way to get the scale elsewhere?

When setting KDE's scale to 150% it returns 2 to me, which is what gets divided. Likely temporary action I can take is simply to add a config option to disable scaling the display.

fastfetch can detect it; no way you can't

Wrong model/make should be fixed in the next commit I do in a few secs swapping wayland to use edid and using the product id if the marketing name is not available. Fractional scaling will not be fixed. As previously said, the library I'm using seemingly doesn't support it.
I took a look at FastFetch's code, your interacting with the wayland library directly and thus can do what you want, something I can't really do because of Rust. Unless I can upstream the work to the wayland-client bindings (which I have no idea how to do), then it's out the window unfortunately. The only other way I can see to do it is to use zxdg output objects, which are only unstable in the wayland-client crate and thus also out the window.


Copied repos duplicate symlinks

If an s2i build uses the --copy instruction and the source has a symlink, the s2i image produces two full copies of the symlinked file.
Reported in https://github.com/openshift/origin/issues/19809

@wozniakjan here's one you can take a look at.

Currently, the source-to-image doesn't care much about symlinks. Only if they are broken on the src file system, it creates the same symlink on dst file system. In other cases, it treats non-broken symlinks as regular files, opens src file, creates new dst file and copies the content. That's why a symlink on src turns into a regular file on dst
https://github.com/openshift/source-to-image/blob/d1e7db8290f8e8a000a94a503d5bc9babf74d20e/pkg/util/fs/fs.go#L193-L242
There are a couple of scenarios when copying symlinks as symlinks can be tricky

src has a symlink with relative path ../../../abc - this is very likely going to be a completely different file on dst and symlink can be broken
src has a symlink with absolute path - will break
( and hardlinks are probably only going to make this harder to solve, pun intended )

One potential solution I see is to leave it all at the user

add a field, eg. FollowSymlinks, with default to true here
https://github.com/openshift/source-to-image/blob/012c77ca17bc928632232d407e1e0476b7480f63/pkg/api/types.go#L43-L44
when FollowSymlinks is true - behave as implemented currently
when FollowSymlinks is false - if there is a symlink in src filesystem, make a similar symlink in dst

In that sense I'm inclined to keep this code around with your proposed configuration, though I would make the default false.

i'm not clear if you're suggesting the name of the setting should be changed such that a value of false results in the current behavior, or you're suggesting the default behavior should be to not followsymlinks?
I would be on board w/ the former, if we're going to add an option at all.
Also @adambkaplan is the title for this issue correct?  "Copied repos duplicate symlinks"?  That doesn't sound the same as "copied repos turn symlinks into raw files" (it sounds like you're ending up w/ doubled files)

I also think we need to take the usage into account.  We move/copy files around for different reasons. It's possible that in the scenario described in https://github.com/openshift/origin/issues/19809 we should be retaining the file as a symlink, but in the work being done with "--copy" we should not.

@bparees updated the title for clarity.

i'm not clear if you're suggesting the name of the setting should be changed such that a value of false results in the current behavior, or you're suggesting the default behavior should be to not followsymlinks?

I was suggesting the latter, though I wanted your input on this. Perhaps change to KeepSymlinks - false = current deep copy behavior, true = shallow copy as a symlink.

I was suggesting the latter

i'd definitely have concerns about breaking existing behavior.
Adding a flag (such that the value defaults to false and the behavior defaults to the existing behavior) would be ok, but ultimately we'd have to wire it up to the build api so we should be thinking about what that would look like from the start.
Which comes back to my earlier comment:  we copy/move files around for a lot of reasons during a build flow.  are they all impacted by this?  do we want them to all behave the same way or are there reasons why some should behave differently from others?
Is this fundamentally the result of what our tar helper is doing?

we copy/move files around for a lot of reasons during a build flow. are they all impacted by this? do we want them to all behave the same way or are there reasons why some should behave differently from others?

Based on my understanding & analysis, almost all are impacted by this. It's hard for me to find a good use case why we'd need some portion of the build to copy as symlink, and others to deep copy the raw file. Enabling this would also lead to quite a bit of "config clutter".

Is this fundamentally the result of what our tar helper is doing?

It looks like the tar helper is using the standard io.Copy to write files to the tar stream. The symlink deep copy is happening upstream from there (ex: the git clone).
https://github.com/openshift/source-to-image/blob/master/pkg/tar/tar.go#L261

It looks like the tar helper is using the standard io.Copy to write files to the tar stream. The symlink deep copy is happening upstream from there (ex: the git clone).

and also handles symlinks by default
https://github.com/openshift/source-to-image/blob/d2e7686fb703feed3185da07f77aaafb4721459b/pkg/tar/tar.go#L225-L239
It appears to be only fs.go that was causing problems

but ultimately we'd have to wire it up to the build api so we should be thinking about what that would look like from the start.

for testing purposes, I just hardcoded the flag to always keep symlinks https://github.com/wozniakjan/origin/pull/6/files. I can imagine it could make sense as a flag in BuildConfig

i suspect we did this for s2i cli w/ --copy so that if you were building a local dir w/ a symlink reference outside the working dir, things would just work.  When running a build in openshift it makes less sense (since you don't control the host environment, you wouldn't have symlinks that reference locations outside the source dir).
So i think for s2i cli behavior, we probably want to keep the default behavior (deep copy the symlink) so that links to host files (such as /etc/hosts or something) copy in the expected content.
In openshift i think we would want to change the behavior to retain symlinks as symlinks (the original issue that was reported against openshift), but i don't think we want to introduce it as an option.
so i think your starting point for this fix is valid (make it configurable in s2i behavior, retaining the existing behavior as default) and then when we vendor into origin we'll want to set the option differently to actually resolve the original reported issue(https://github.com/openshift/origin/issues/19809)


Backlog: check if PR template was deleted

Scenario: impatient contributor deletes whole PR template message and decides it doesn't apply to them.
Result: PR is decorated with a comment telling user to fill out PR template.
Implementation ideas:
Opt in via feature in .DEREK.yml file pr_template_checking

v1.0 PR body cannot be empty
v2.0 100% of the lines beginning with a heading i.e. # in repo/.github/PULL_REQUEST_TEMPLATE.md must be present in body of PR

@rgee0 @dirkhh @johnharris85 thoughts?

I’d be tempted to also have a switch to enable totally empty PRs to be automatically closed. Default would be just apply a message.

Ye I like the idea (and feature progression). Are you thinking same thing for issues as well?
Also what about the ability to check that a PR references an existing issue / proposal? That ties into the workflow for OpenFaaS that all PRs must have issues / proposals.

@johnharris85 the same thing could apply to issues but for an MVP I'd suggest going after PRs first. I guess this would be valuable to any project that uses a template for PRs.
The issue per PR requirement is a thing, but I think it may be hard to automate so could be revisited later.
If anyone wants to work on a solution as per v1.0 or v2.0 please comment

I was starting to have a look at this, if nobody else is already working on it.

BTW, I missed this earlier... I think v2.0 is a bit too aggressive. It's fairly common for PR templates to have sections that don't really make sense for all PRs... So I'd say that "at least half of the headings need to remain" or something slightly more flexible than that. Extra credits for it being configurable :-)

Bump on this issue and +1 to Dirk's comments I think that makes sense.. Some proportion of headings should be present. Certainly it will pick up the annoying fly-by PRs like:

(no text given)

Alex

@johnharris85 would you like to take a look at this one? I also have a refactoring task that needs doing to make Derek more like an SDK / easier to run in other platforms/apps. #62 (I don't have a good way to contact you so a comment here is the best at the moment, if you'd like to join OpenFaaS Slack we have a #derek channel there)

It seems like we have a kind of dead-lock or lack of interest for this feature. I still have to comment back to people who delete the whole text and submit a PR with no description at all. For that reason I've raised a simpler issue for v1.0 of the feature in #103

Getting started on this now. Hoping for it to be ready for the HacktoberFest.

Did anyone work on this?

Going back to the "Implementation ideas:", I'd suggest that we make a binary decision based upon the following criteria:

A PR template is present in the repo
50% of the lines beginning with # in the template are still in the PR text submitted

A new contributor can start this with a unit test in isolation from Derek's main codebase and PR/webhook handling features, once fully working we can help with integrating it in.

I'm getting started on this now

Version "v1.0" was completed here: https://github.com/alexellis/derek/pull/104
This change/update would be for version "v2.0" where the template is used to validate the incoming PR body.

Aha, yes so I think we need version “1.5” and not quite a “2.0” as the original idea there might have been a bit too strict.


Update Archetypes

Body
The current archetypes need to be changed to reflect the changes made when renaming extensions modules.
StreamPipes Committer
I acknowledge that I am a maintainer/committer of the Apache StreamPipes project.

See discussion #1422 for more context


feat: search box navigation thru results with up/down keys doesn't work (#3028)

Closes: #3028

QA Steps:

Type 'dsa' into searchbox
When products or suggestions are loaded, press up key, check highlights last product in list
Press up key again, check highlights second last product in list
Press down key, check highlights last product in list
Press down key again, check highlights first product in list
Press enter, check navigates to highlighted option and closes searchbox


Cirrus CI

I was testing Android capabilities of Cirrus CI for a blog post and found out that Cirrus CI is approximately 3-4 times faster (4 minutes instead of 13-16 minutels right now).
Here is a Cirrus CI build for this change:

Just wanted to let you know in case you are interested in a new CI. Feel free to close this PR if you are not interested. Didn't want to just throw this work away. 😅
PS in case you'll want to merge this change, install Cirrus CI App (it's free for OSS) on this particular repository as described in Cirrus CI documentation.

I'm no Travis CI fan, but there isn't enough activity on this project where it's (extreme) slowness is a problem. If I was working in this codebase all day, everyday I would not hesitate to switch to something else, but it's a "devil-you-know" situation. Thanks!

@JakeWharton totally makes sense! Thanks for the response!
BTW I also though you'll be impressed with Cirrus CI's built-in HTTP Cache which is compatible with Gradle's Build Cache.


[Form] Inverted styles don't get applied when using inverted loading

Steps
Create form element with the following classes ui inverted loading form
Expected Result
The loader should have the inverted styles
Actual Result
The loader is the standard loader
Version
2.3.1
Testcase
https://jsfiddle.net/ntauf0e8/

Fixed in the community Fork Fomantic-UI
See your adjusted jsfiddle here https://jsfiddle.net/lubber/zyermnpj/


sap.m.Slider does not support vertical slide

OpenUI5 version:
any.
Browser/version (+device/version):
independent
Any other tested browsers/devices(OK/FAIL):
see above
URL (minimal example if possible):
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:

check sap.ui.commons.Slider for property to set vertical slide mode
check sap.m.Slider to find no such option

What is the expected result?
Since sap.ui.commons.Slider is deprecated I would like to have the same options e.g. vertical sliding on the sap.m.Slider.
This would make it useful for scenarios where up and down is the more established control movement (like controlling speeds).
What happens instead?
there is no option to set this at the moment.
Any other information? (attach screenshot if possible)

Hi @comdata,
This seems to be a feature request.
Internally tracked:<PHONE_NUMBER>
Thanks and Kind Regards,
Diana

Hi
There is a plan for the future release to implement this feature.
I'll mark this issue as accepted.
Best regards,
Ivaylo


Drop support for Node < 14

This PR drops node versions before 14. This is a breaking change.
Note that I also updated CI to just use the pinned node version from Volta, and while at it I also updated yarn to the latest version.
See https://github.com/emberjs/ember-qunit/pull/921

ping any chance to get this merged, to continue towards EAI 2?

Ping @rwjblue :)

@locks this should have breaking label, right?


Adds dap support

This is a starting point to add DAP support.
Current status

[x] Test* functions
[ ] Sub test functions with t.Run
[x] Example* functions
[ ] Benchmark functions (it is not supported by the plugin right now, so DAP will not work)
[ ] Testify suites

The status sign is broken right now after a debugging session, I haven't look at why right now.

#12

@florentcigolotti I think your work on this is a very good start. I'd suggest narrowing the scope of this PR though so it isn't blocked by these issues indefinitely. I think factor out stuff like subtests and testify to separate PRs since those aren't even super well-supported at all anyway.
I think the only real blocker here is getting the signs working. I think for those it's important to figure out what the status of each test is and make sure that test results are coming through in the format expected and the status value is correct.

Are there any updates on this PR? Is someone still working on it?

I created new pr, with lesser features.


Promise rejections throw warning

As of v3.2.1, I get the following error in CLI output:
Warning: a promise was rejected with a non-error: [object Undefined]

I suppose it’s related to https://github.com/2createStudio/postcss-sprites/commit/4b897c0e2bf12d40a5905eca73d96d1f5841faec and Bluebird promises handling: http://bluebirdjs.com/docs/warning-explanations.html#warning-a-promise-was-rejected-with-a-non-error

I got same warning.
Warning: a promise was rejected with a non-error: [object Undefined]
    at /Users/username/work/dev/node_modules/postcss-sprites/lib/index.js:176:30
    at /Users/username/work/dev/node_modules/postcss-sprites/lib/index.js:268:11
...

prepareGroupBy is it the same problem?

I'm getting the same issue on version 4.1.1.

It's this line for sure.


Improve inventory storage disconnect

There's a disconnect between inventories.inventories[name] and inventories.inventory(name).items as StateManager changes the underlying data array

I've updated PlayerSave so it now correctly saves the latest, but this is not a great solution and could potentially be exploitable

Room for improvement but good enough for now.


Getting started with monitoring of analysis service

I use a managed analysis services server in my subscription and would like to get started with some monitoring,
What would be the most common and important metrics I should look at when defining alert rules?

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

ID: 33d46c28-4183-1ce1-72fb-5304a7052863
Version Independent ID: ea34e5f4-1c56-8378-e675-673553a38f46
Content: Monitor Azure Analysis Services server metrics
Content Source: articles/analysis-services/analysis-services-monitor.md
Service: azure-analysis-services
GitHub Login: @Minewiskan
Microsoft Alias: owend

@assaf-grth
Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1


As an EN developer, I want to make sure I have access to all PDS repos.

For more information on how to populate this new feature request, see the PDS Wiki on User Story Development:
https://github.com/NASA-PDS/nasa-pds.github.io/wiki/Issue-Tracking#user-story-development
Motivation
...so that I can update / access any of our repositories and source code at any time.
Additional Details
We should make sure all of our repos have the following teams with access:

@pds-software-committers - Write
@pds-software-pmc - Admin
@pdsen-operations - Admin

Additionally, not sure if doing this autonomously is worth the effort at this point. Let's just add this to the template repo READMEs or in a wiki for creating a new repo and call it good.
Acceptance Criteria
Given a member of the @pds-software-committers team has an update to push to a PDS EN repo
When I perform a commit and push to a branch
Then I expect to successfully push to a branch on that repo
Given a member of the @pds-software-pmc or @pdsen-operations team has an update to push to a PDS EN repo
When I perform a commit and push to master on that repo
Then I expect to successfully push to master on that repo

I've updated the wiki page about new repos to refine the advice on team setup. Doing the READMEs for the -java and -python repositories next.

Finito.


Teach "Update package-lock.json" workflow to not upload dev-dependencies on release/v1

Like this: https://github.com/ilammy/setup-nasm/pull/26
node_modules directory is checked in on release/v1 so that the action works. However, it does not need development dependencies there. Tweak the package-lock checking task to avoid installing them for checks (use --only production), which will likely avoid including them into PRs.

Oh, there's even a ticket for this.
Fixed by:

https://github.com/ilammy/setup-nasm/pull/35
https://github.com/ilammy/setup-nasm/pull/36


Rename Agent to Server

Description:
Replace Agent with Server in the Yorkie project.
Initially, I wanted to install Yorkie on the nodes where MongoDB is installed, so I named it Agent.
However, we implemented Agents like normal Servers, and the name of Agent is confusing, so it would be better to rename it as Server.
Why:

Increase the readability of our project.

Can I try this issue?


getReact as a child command should throw a good error if subject is empty

if I accidentally query a non-existent component
cy.getReact('ProductContainer') // does not exist
  .getReact('AProduct')

Then the error thrown is super cryptic

It would be nice to

throw a good error if the subject is empty
auto-retry finding the component

For now I do insert
cy.getReact('ProductContainer')
  .should('not.be.empty')
  .getReact('AProduct')

Hi @bahmutov , Let me know your thought in the below error message. If looks good, I gonna merge the changes.

Looks much better 👍
Sent from my iPhone

On Aug 12, 2020, at 04:04, Abhinaba Ghosh<EMAIL_ADDRESS>wrote:

Hi @bahmutov , Let me know your thought in the below error message. If looks good, I gonna merge the changes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

fixed in v2.0.1


[v9] Theme preset variables having rgba (transparent) will cause Less_Exception_Compiler error or disappears from Customise Skin list

There seems to be an issue with theme preset variables having rgba (transparent) values on V9.0.0.
When you customise a preset, it will either:
1. Throw an error
Less_Exception_Compiler
variable @sidebar-hr-color is undefined in file /home/fiddlrtest/public_html/packages/ds_fiddlr58/themes/fiddlr/css/sidebar_typography.less

Sample styles.xml:
<set name="HORIZONTAL RULE">
  <style name="Main HR" type="color" variable="hr" />
</set>

<set name="SIDEBAR HORIZONTAL RULE">
  <style name="Sidebar HR" type="color" variable="sidebar-hr" />
</set>

Sample preset (preset/sample.less):
// HORIZONTAL RULE
@hr-color: rgba(183, 183,183, 0.16);

// SIDEBAR HORIZONTAL RULE
@sidebar-hr-color: rgba(183, 183,183, 0.16);

If I change the color value to a transparent HEX code (#b7b7b729), the error goes away but the transparency is not read by the color picker.
**2. it will not show on the list of customisable variables under “Customise Skin”. It wil show the set name, but the style won't. **

Sample styles.xml:
<set name="IMAGE">
  <style name="Hover Overlay Color" type="color" variable="image-hover-overlay" />
  <style name="Title Text Color" type="color" variable="image-title" />
</set>

Sample preset (preset/sample.less):
@image-hover-overlay-color:  rgba(255, 101, 95, 0.58);
@image-title-color: #fff;

This may be related to the issue https://github.com/concrete5/concrete5/pull/9230, but  the code base for the changes on that issue is now different on v9.
Steps to reproduce:

Change a preset variable in your theme's preset to rgba (e.g. rgba(255, 101, 95, 0.58))
Go to a page and edit page via cog wheel
Click Design > Customise
Select the preset of the theme used in step 1
Either an error will show (1 above) or the style will not show (2 above)

This has been fixed and will be released in 9.0.1.


Can bind the model in addition to the view

I've tried it out both passing and not passing a model and had no issues either way... Let me know how it looks!

@lipsitzm Thanks for this!  Looks great except for one small thing - the content_view and content_model attributes should follow the convention of being dasherized instead of snake cased.  Can you fix this and push that change out and I will merge?

I've had to merge this in to develop to fix, your commits will exist in history (and I greatly appreciate them) but I'll need to close this PR.


Update commands.js

You would have to change the case 'eat': line to case 'eating': for this to work.

Then I will

I meant for you to do that in addition to, not instead of...
Sorry for miscommunication :S

Oh.