 renders 0 on each ; occurs only in FireFox, only with bootstrap4 and only with class provided in docs

unexpected behavior
The number zero appears on each list-item of ordered lists when viewed in Firefox for Mac with class="list-group-item" applied to the ol tag per "List group" docs for v4.0.
This behavior persists even with addition of a start attribute to the related ol tag (e.g. <ol class="list-group" start="3">).
expected behavior
Each list-item of an ordered list should show a sequential number, starting from 1 unless otherwise specified in a start attribute on the ol tag.
This expected behavior appears when

viewing the reduced test case in Safari and Chrome for Mac, or Firefox for Windows
removing the class="list-group-item" from the ol tag and loading the case in any of the tested browsers (including Firefox) on any of the tested platforms
replacing v4.0 with v3.3.7.

reduced test case
https://codepen.io/onso/pen/XzKgpB
purposes of this reduced test case:

confirm issue occurs only in FireFox, only with bootstrap4 and only with class provided in docs
confirm use case works as expected in Chrome & Safari
confirm use case works as expected with Bootstrap3 in Firefox.

browser / os / user agent info
fails: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:49.0) Gecko/20100101 Firefox/49.0
works as expected: Firefox Versions 55.0.3 & 56.0 on Windows 10
works as expected: Safari Version 11.0 (126<IP_ADDRESS>.7) on Intel Mac OS X 10.12.6
works as expected: Chrome Versions 61.0.3163.100 & 62.0.3202.75 on Intel Mac OS X 10.12.6
--
fyi, the following steps are all completed prior to submission of this bug report:

Before opening an issue:

Search for duplicate or closed issues
Validate using html5.validator.nu
Lint using github.com/twbs/bootlint#in-the-browser
Prepare a reduced test case
Read the contributing guidelines

The latest version of stable Firefox is 56. Testing on Fx56 on Windows 10 seems to render the numbers correctly in that codepen example (even after removing your additional CSS that tries to fix things)

agreed with @patrickhlauke - apparently didn't catch ff version mismatch where it counted!


licensed: init at 4.4.0

Description of changes
This PR adds licensed: https://github.com/github/licensed
licensed scans dependency definitions from various build tools (like cargo, npm, gradle, …), and tries to automatically verify whether they match a configurable list of allowed licenses.
Things of note for this PR:

licensed is in "low maintainance mode" upstream. (Meaning: It's currently useful, but there won't be additional support for newly emerging build systems.) I hope that's not a problem for adding to nixpkgs.
licensed will throw an ugly error message + backtrace if it's run on a directory without the necessary configuration (existing .licsensed.yaml). If you want to check whether this package is actually working, try e.g. running licensed status on a clone of the licensed repo itself.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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. ← I hope…

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

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

1 package built:

licensed


Xamarin Android crashes are missing much of the stack trace

When a Xamarin Android crash was logged in HockeyApp, it would give the full stack trace of the exception. In App Center, much of the stack trace is getting omitted. For example, look at this exception and stack trace:

ParcelBoundaryUtility.GetImageUrl (System.Int32 mapWidth, System.Int32 mapHeight, System.Double boundaryMinLongitude, System.Double boundaryMinLatitude, System.Double boundaryMaxLongitude, System.Double boundaryMaxLatitude, RPR.Mobile.Shared.Maps.ParcelBoundaryColor color, System.Threading.CancellationTokenSource cts)
System.ArgumentOutOfRangeException: StartIndex cannot be less than zero. Parameter name: startIndex
Microsoft.Runtime.CompilerServices
TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task)
Microsoft.Runtime.CompilerServices
TaskAwaiter.HandleNonSuccess (System.Threading.Tasks.Task task)
Microsoft.Runtime.CompilerServices
TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task)
Microsoft.Runtime.CompilerServices
TaskAwaiter`1[TResult].GetResult ()
RPR.Mobile.Shared.Maps
ParcelBoundaryUtility.GetImageUrl (System.Int32 mapWidth, System.Int32 mapHeight, System.Double boundaryMinLongitude, System.Double boundaryMinLatitude, System.Double boundaryMaxLongitude, System.Double boundaryMaxLatitude, RPR.Mobile.Shared.Maps.ParcelBoundaryColor color, System.Threading.CancellationTokenSource cts)
RPR.Mobile.Droid.Adapters.PropertyDetails
LocationDetailsBuilder+<>c__DisplayClass10_0.b__0 ()

Here is the actual code:
public async Task<string> GetImageUrl (int mapWidth, int mapHeight,
			double boundaryMinLongitude, double boundaryMinLatitude, double boundaryMaxLongitude, double boundaryMaxLatitude,
            ParcelBoundaryColor color, CancellationTokenSource cts = default (CancellationTokenSource)) {
            session = await GetSession (cts);
            if (session != null && session.Response != null && session.Response.Results != null && session.Response.Results.Data != null && session.Response.Results.Data.Row != null) {
                var url = String.Format ("http://dc1.parcelstream.com/GetMap.aspx?layers=ss.base.Parcels/Parcels&SLD=$(ACCOUNT_FOLDER)/Styles/parcels/{0}.sld.xml&SS_CANDY={1}&bbox={2},{3},{4},{5}&width={6}&height={7}&resolution=40&antialias=true",
                    color == ParcelBoundaryColor.Purple ? "RPR" : "RPRW", session.Response.Results.Data.Row.Candy, boundaryMinLongitude, boundaryMinLatitude, boundaryMaxLongitude, boundaryMaxLatitude, mapWidth, mapHeight);
                return url;
            } else
                return null;
		}

The exception stack trace indicates that the ArgumentOutOfRangeException is getting thrown within this GetImageUrl function, however it's actually not. The exception is being thrown in the GetSession function:
private async Task<ParcelBoundarySession> GetSession (CancellationTokenSource cts) {
            if (session != null)
                return session;
            else
                return await DataService.Map.GetParcelBoundarySessionAsync (cts != null ? cts.Token : default (CancellationToken));
		}

Furthermore, it's not really getting thrown within this function, it's getting thrown within the DataService.Map.GetParcelBoundarySessionAsync function: (note the json.Substring, which throws an ArgumentOutOfRangeException is the json is in an unexpected format.)
public async Task<ParcelBoundarySession> GetParcelBoundarySessionAsync (CancellationToken cancellationToken = default (CancellationToken)) {
            var json = await GetAsync<string> (new QueryRequest<string> {
                Method = "GetParcelBoundarySessionAsync",
                Url = "http://parcelstream.com/InitSession.aspx?sik=80d1a1fe-ba23-4ba2-9048-9cf2e39b451c&obsId=window&obsSuccessMethod=S14732800994305247486921&obsErrorMethod=E14732800994305247486921&output=json",//http://dc2.parcelstream.com/_T94/initsession.aspx?SIK=_T94/68359280-b55a-447f-bcc7-c41d167156ca-100796_570464359&output=json
                CacheExpiration = TimeSpan.Zero,
                CancellationToken = cancellationToken
            });

            json = !String.IsNullOrEmpty (json) ? json.Substring (json.IndexOf ("{"), json.Length - json.IndexOf ("{") - 1) : "";
            var result = !String.IsNullOrEmpty (json) ? JsonConvert.DeserializeObject<ParcelBoundarySession> (json) : null;
            return result;
		}

So clearly the App Center stack trace is incomplete. It's missing a line for the GetSession function call, and also another line for the GetParcelBoundarySessionAsync function call. It's bad enough that you guys still don't support line numbers in Xamarin crashes, but to log incomplete stack traces on top of that is completely unacceptable. We'll be moving to a different product for crash reporting most likely, as App Center is not a reasonable solution for Xamarin apps.

@justintoth I'll pass this feedback to the service PM. Have you discussed this issue with our support team? What did they tell you?

Hi Justin - can you start a support thread (blue chat bubble in App Center) so our support and engineering team can take a closer look for you? We want to make sure App Center is meeting your expectations. Thanks!


AccessCodeForm - Code input box

Is this the ability to set custom codes? Which is blocked due to missing code validation on seam-connect?

Seam API is considering adding a validation and generation endpoints so client applications do not need to implement that logic.


{CI} replace 2019-07-01 with 2020-06-01

https://github.com/Azure/azure-cli/pull/14233/files The CI in this PR didn't sync with resource's latest change. The api-version in three recording files are not correct.

Did you modify yaml files directly?

Did you modify yaml files directly?

Yes. Let's have a test first. We can discuss whether we should merge it or not.

I rerun it lively and got
knack.util.CLIError: Name clitest.rg3hdo26ksbqcrbem7cwwqll4z2p5pyv26gwhroglddmnbyz5v4prss6snz4laex5aq is not valid.

I rerun it lively and got
knack.util.CLIError: Name clitest.rg3hdo26ksbqcrbem7cwwqll4z2p5pyv26gwhroglddmnbyz5v4prss6snz4laex5aq is not valid.

I met same errors. It's return from web app service. I don't know what's the root cause for it. That's why I open this PR.

agree to the temp fix to unblock CI. Issue https://github.com/Azure/azure-cli/issues/14623 is tracking webapp issue.


Lint application JavaScript using StandardJS

We should lint the application JS (everything excluding vendor specific files) by adding standard to package.json
"scripts": {
    "test": "standard && gulp test"
  }

This was fixed in #89.


Struct std.mmfile.MmFile?

bearophile_hugs reported this on 2013-02-12T17:45:01Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=9501
CC List

issues.dlang (@jmdavis)
qs.il.paperinik

Description
Maybe it's a good idea for std.mmfile.MmFile to become a struct like std.stdio.File, so the file is closed when this struct goes out of scope and the reference count it keeps is zero.

issues.dlang (@jmdavis) commented on 2013-02-12T17:54:43Z
Yes, std.mmfile.MmFile should be a struct. It doesn't use polymorphism and has no need to be a class. But making it a struct would break all code that uses it, so we're arguably stuck with it being the way that it is.

bearophile_hugs commented on 2013-02-12T20:05:43Z
Mostly unrelated, about std.mmfile: I think in the std.mmfile module this line (line 270) should offer a string message too when the file is not found:

errnoEnforce(false);

And I think switches like this one should become "final":

            switch (mode)
            {
            case Mode.read:
...
            default:
                assert(0);
            }

bearophile_hugs commented on 2013-02-12T18:19:08Z
(In reply to comment #1)
> Yes, std.mmfile.MmFile should be a struct. It doesn't use polymorphism and has
> no need to be a class.

And maybe it doesn't have an explicit close() method, I don't know why, maybe for safety in accessing the memory mapped bytes.

> But making it a struct would break all code that uses
> it, so we're arguably stuck with it being the way that it is.

There are always acceptable deprecation paths, if we want them. Like adding the struct to std.mmfile with another name ("RcMmFile"?) and deprecating std.mmfile.MmFile for two years.

OTOH, if FILE was a class (or interface), MmFile could plug in anywhere a FILE is used.  And then a debug/tracing version of a FILE (implemented in a subclass) would also be available, as would one which permits a read-only "FILE" based on string contents.  Thsi last would make some unit tests much tidier, leaning only on polymorphism.


Update Readme to have more details about the project :)

Just my personal fun project which I am thinking of launching for the Flipkart interns

Just my personal fun project which I am thinking of launching for the Flipkart interns

Ohh! Amazing.... I better close this issue in that case :stuck_out_tongue:


<EMAIL_ADDRESS>Configuration config.vue.js does not take effect   in vue3

Version
4.5.7
Environment info
<EMAIL_ADDRESS>Configuration config.vue.js in vue3. For example, outputDir, productionSourceMap does not take effect, but<EMAIL_ADDRESS>does, and multiple machines have verified this problem, and hope to fix it soon

Steps to reproduce
<EMAIL_ADDRESS>Configuration config.vue.js For example, outputDir, productionSourceMap does not take effect, but<EMAIL_ADDRESS>does, and multiple machines have verified this problem, and hope to fix it soon
What is expected?
take effect
What is actually happening?
does not take effect

<EMAIL_ADDRESS>Configuration config.vue.js in vue3 For example, outputDir, productionSourceMap does not take effect, but<EMAIL_ADDRESS>does, and multiple machines have verified this problem, and hope to fix it soon

Thanks for submitting this issue! Due to our limited time, we ask you to include a reproduction link to a minimal full reproduction of your problem (for example in a GitHub repository) so we can find what is causing the issue. Thank you for your understanding!

BTW the config file is vue.config.js not config.vue.js.

BTW the config file is vue.config.js not config.vue.js.

It is vue.config.js, the file name was typed wrong just now, as long as it is vue3, the vue.config.js setting will not take effect

Thanks for submitting this issue! Due to our limited time, we ask you to include a reproduction link to a minimal full reproduction of your problem (for example in a GitHub repository) so we can find what is causing the issue. Thank you for your understanding!

https://github.com/wuyanwen/vue-cli-vue3

There's a typo. module.exports is not module.export.


Failure to logout

The sample project available at https://github.com/stormpath/stormpath-angular2-express-example.git works fine. Restructuring the app to something more realistic introduces a few problems. Restructured project is available at the following repository, with read me explaining the changes (in short: routing, three components routing, dedicated login page).
https://github.com/patricknee/stormpath-angular2-testbed
Logout link fails to consistently log out. I would estimate logout probability at 40% on the first try. Repeatedly hitting the logout link will eventually cause logout.

@patricknee Thanks for taking the time to create a new, restructured project and reporting the issues you found after making these changes! I'll work on fixing these before the end of the year.


Adds support for windowing functions.

This allows us to use analytic functions aka windowing functions in generated select statements.
Closes #1019

Thanks. I made the changes and merged the changes.


Dynamic props are not getting added with Storybook Integration

I am exploring Storybook integration with Code Connect for React components. When I create a .figma.tsx file using figma connect create, all required dynamic props are correctly updated in the Code Connect snippet. However, when I integrate it with Storybook, I only see the component without the respective props. I have provided the working Figma Connect code and the Storybook file that is not displaying the props correctly. Additionally, I have attached a screenshot for reference.
*.figma.tsx
import React from 'react';
import ButtonPrimary from './ButtonPrimary';
import figma from '@figma/code-connect';

/**
 * -- This file was auto-generated by Code Connect --
 * `props` includes a mapping from Figma properties and variants to
 * suggested values. You should update this to match the props of your
 * code component, and update the `example` function to return the
 * code example you'd like to see in Figma
 */

figma.connect(
  ButtonPrimary,
  '##### figma url ######',
  {
    props: {
      text: figma.string('Text'),
      state: figma.enum('state', {
        Default: 'default',
        Hover: 'hover',
        Active: 'active',
      }),
      size: figma.enum('size', {
        Large: 'large',
        Medium: 'medium',
        Small: 'small',
      }),
    },
    example: (props) => (
      <ButtonPrimary text={props.text} state={props.state} size={props.size} />
    ),
  }
);

*.stories.tsx
import type { Meta, StoryObj } from '@storybook/react';
import ButtonPrimary from './ButtonPrimary';
import figma from '@figma/code-connect';
import { within } from '@storybook/testing-library';
import { expect } from '@storybook/jest';

const meta: Meta<typeof ButtonPrimary> = {
  component: ButtonPrimary,
  title: 'ButtonPrimary',
  parameters: {
    design: {
      type: 'figma',
      url: '##### figma url ######',
    },
    props: {
      text: figma.string('Text'),
      state: figma.enum('state', {
        Default: 'default',
        Hover: 'hover',
        Active: 'active',
      }),
      size: figma.enum('size', {
        Large: 'large',
        Medium: 'medium',
        Small: 'small',
      }),
    },
    example: (props) => (
      <ButtonPrimary text={props.text} state={props.state} size={props.size} />
    ),
  },
};
export default meta;
type Story = StoryObj<typeof ButtonPrimary>;

export function ButtonExample({
  text,
  state,
  size,
}: {
  text: string;
  state: 'default' | 'hover' | 'active';
  size: 'small' | 'medium' | 'large';
}) {
  return <ButtonPrimary text={text} state={state} size={size} />
}

export const Primary = {
  parameters: {
    design: {
      type: 'figma',
      url: '##### figma url ######',
      props: {
        text: figma.string('Text'),
        state: figma.enum('state', {
          Default: 'default',
          Hover: 'hover',
          Active: 'active',
        }),
        size: figma.enum('size', {
          Large: 'large',
          Medium: 'medium',
          Small: 'small',
        }),
      },
      example: (props) => (
        <ButtonPrimary
          text={props.text}
          state={props.state}
          size={props.size}
        />
      ),
    },
  },
  args: {
    state: '',
    size: '',
    text: '',
    onClick: '',
  },
};

export const Heading: Story = {
  args: {
    state: '',
    size: '',
    text: '',
    onClick: '',
  },
  play: async ({ canvasElement }) => {
    const canvas = within(canvasElement);
    expect(canvas.getByText(/Welcome to ButtonPrimary!/gi)).toBeTruthy();
  },
};

Additionally, when I try to integrate Figma Code Connect via Storybook for Lit components, it does not recognize Figma Code Connect and throws an error: "No Code Connect files." However, it works fine if I integrate it by creating a .figma.tsx file.

Hey @JagdishThavasi, that's strange, it looks like your example should work. Could you please raise a ticket with our support, mentioning that you were sent there from Github Issues? This way we can securely get access to your Figma file and debug what is going on


testnet network support - 4qoosuxone3zd6t - validator,rpc,api

@valerchegs ,Your Moniker and validator addresses are required, please update the PR with those details

@valerchegs ,Your Moniker and validator addresses are required, please update the PR with those details

@valerchegs ,Your Moniker and validator addresses are required, please update the PR with those details

Hi, my moniker valerchegs and my address akashvaloper1ng5jvlvsz4cfjk9ecgydeh8xgnw0l0cd9q387w

@valerchegs ,Your Moniker and validator addresses are required, please update the PR with those details

Hi, my moniker valerchegs and my address akashvaloper1ng5jvlvsz4cfjk9ecgydeh8xgnw0l0cd9q387w


Allow to access env vars

We should allow to access env vars prefixed with STORYBOOK_ in the client side.
This allows to pass different configuration parameters dynamically.
This should works on both in the build time and dev time.

We've this with the version 2.14.0


Transpose with explicit permutation

After some discussion, we decided that using C and F in the transpose codec can be confusing when used in the codec pipeline. Therefore, we changed the spec to only allow explicit permutations in the order attribute. Do you think zarrita.js could also support that? Thanks!

Certainly! Thanks for pinging me. Has this landed in any other implementation, yet?

Yep, here are a few examples:
https://github.com/scalableminds/zarrita/blob/async/zarrita/codecs.py#L370-L443
https://github.com/google/neuroglancer/tree/master/src/datasource/zarr/codec/transpose
https://github.com/clbarnes/zarr3-rs/blob/main/src/codecs/aa/transpose.rs

Thanks for the reference. I haven't had the chance to take a look but will soon.


Python quickstarts Test/Checks fail (Config API get result)

Expected Behavior
Python tests pass.  LKG in main is:
Valid result with Dapr 1.13.0
Actual Behavior
Using Dapr 1.13.5 and no other changes, I see this new failures on the return of Config API
Failure result with Dapr 1.13.5

@berndverst any ideas?

Two things: you aren't version pinning - so this pulled in either the latest -dev image or the pre lease 1.14 RC1.
Secondly, this is by design - I fixed an a customer reported typing issue where config API had not returned the expected type. You will want to call the ".json()" method on each returned configuration item.

See Python SDK folder, example for config API

https://github.com/dapr/python-sdk/issues/720

Fix avail ^^

fixed by #1051


Remove posix calls in TemporaryFile.swift

This implements a portable version of mkstemps and mkdtemp so that the code also works on Windows. This simplified the code considerably

This looks good to me, and the removeItem that this uses just got implemented on Windows today (https://github.com/apple/swift-corelibs-foundation/pull/2117).
@aciidb0mb3r

@compnerd is this still relevant given your recent work for Windows support?

I don't think that this is relevant any longer.  @a20012251 if you think that this is still relevant, please rebase and reopen the PR.


The table lists region human friendly names, but not the azure region code names.

So, it tells us that Switzerland North is paired with Switzerland West, for example. But it does not provide the Azure region code for Switzerland North - is it switzerlandnorth ? Or anything else?
Is the rule to just concatenate (excluding everything in the brackets) and remove spaces? We need this for scripting.

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

ID: 09a788e4-0130-73f2-d985-6fc3aec77974
Version Independent ID: a9f4ec3f-f0a3-b050-020b-f487c74c5179
Content: Ensure business continuity & disaster recovery using Azure Paired Regions
Content Source: articles/best-practices-availability-paired-regions.md
Service: multiple
GitHub Login: @barbkess
Microsoft Alias: barbkess

@MarkKharitonov Thanks for your question. To retrieve the  azure region codes, you can use below az command in Azure CLI.
az account list-locations -o table
Hope this helps!!

@MarkKharitonov Just following up to check if you need further assistance on this issue.

I have managed, but I still think the documentation should mention real region codes. For me they are more important than the human friendly names

@barbkess  Can you please review this request ?

#assign @anukanade

@MarkKharitonov ,
I'm sorry for the inconvenience, but we do not publish the region code names at this time. Please use:
az account list-locations -o table as mentioned above.
#please-close


[integration-tests] Specify path to the llvm bin dir instead of filec…

…heck itself.
We can infer FileCheck's path from the llvm bin dir so there is no change today.
In the future though this will enable us to use other llvm tools to "poke" at
the produced snapshots for verification purposes.
rdar://39456714

The corresponding PR in the integration tests is:
apple/swift-integration-tests#26

apple/swift-integration-tests#26
@swift-ci smoke test


Add 2016 Candidate Survey to site

extended deadline Friday, Oct 14th

Key:

✅ = responded
❌ = no response
💩 = declined to participate

Responses

Name
District
Status
Notes

Casey Ramos
2
❌
Second round of outreach 9/28 via FB msg<EMAIL_ADDRESS>@ramos4d2

Wesley Faulkner
2
✅
@Austin4Everyone

Delia Garza
2
❌
Second round of outreach 9/28 via<EMAIL_ADDRESS>@dgarzaforD2

Greg Casar
4
✅
@gregcasar

Gonzalo Camacho
4
❌
Second round of outreach 9/28 via<EMAIL_ADDRESS>@gonzalo_camacho

Louis Herrin III
4
❌
Second round of outreach 9/28 via<EMAIL_ADDRESS><EMAIL_ADDRESS>

Jimmy Flannigan
6
✅
@JimmyFlannigan

Don Zimmerman
6
💩
@DonZimmermanATX

Leslie Pool
7
✅
@PoolforAustin

Natalie Gauldin
7
❌
Second round of outreach 9/28 via<EMAIL_ADDRESS>@natalieforatx

Sheri Gallo
10
❌
Second round of outreach 9/28 via<EMAIL_ADDRESS>@sherigallod10

Alison Alter
10
✅
@ALTERforATX

Nicholas Virden
10
💩
Second round of outreach 9/28 via FB msg & @VirdenForAustin, responded to outreach with: As a city council member, I would fully support a government that publishes every possible thing it does online so that the public can see it, as long as it is not clearly illegal to do so. Moreover, this information should be intuitively searchable.    Regarding questions of citizen connectivity, we have libraries that provide internet access for citizens. For housing developers, I think it should be the burden of the city to collect pertinent information, not the burden of the developer to provide the information that they want to analyze as long as it doesn't violate any state or federal laws.    Government needs to be accountable to its citizens. An open government is the way to start.

Rob Walker
10
❌
Second round of outreach 9/28 via FB msg<EMAIL_ADDRESS>@WalkerforAustin

canidate-survey.pdf

Hey team, above is a quick wireframe based on our whiteboard image.
@brainnews can you share the notes you took with us?
A few things to address and iterate on:

[ ] Tabbing
[ ] No response sections// What does it look like when 1-3 people don't respond. How do we show this?
[ ] Emailing the candidates// Allowing for easy engagement

Feel free to iterate on these on your own but do update us so we don't duplicate efforts. Additionally, I'll be at this weekend's hack Saturday.

originial whiteboard sketch

Here are my notes from Tuesday night:
Open Questions:
Are there colors for each district that we can use to differentiate each tab?
How do the email links look/function?
What does it look like when a candidate hasn't responded to a question?
Needs:
Auto-populated email subject line and message body(?)
(For example)
Subject line:
I have a question about your position on [topic]
Body:
Let [Candidate Name] know how you feel about [topic]...
Nice to haves/For next year:
Accordionize the question categories
Who are our users? Create personas from research.

Additional information that will likely change the design
Only 5 races right now:
2,4,6,7,10
We only send the candidate questionnaire to districts that have races.
I'm iterating on our current layout right now.

Iterating on the UI with sketch.
I like this website from Progress texas. Taking some ques from it :eyes:
http://fightbacktx.com/

I iterated on the mockups some. Length
was kind of concerning me with the repetitive "no response" sections. This mockup reflects info for district 10. I didn't do all the text for the respones.
PDF: canidate-survey-v2.pdf
Sketch File: Uploading council-survey-v2.zip…

@VictoriaODell, I had a chance to look over v2 today and I think it looks great! Wonderful work! I have a couple tinsy copy edits I would recommend, but I'm feeling like we should go ahead and turn this high-fidelity mockup into code and iterate from there. Is that ok?
I will start this out over the next couple days but I will be pretty busy through Saturday with work stuff. My goal would be to get it mostly working by next Tuesday's hack night. That way we can get some feedback, maybe get some help, round up final responses, and publish a final version by Friday.
That sound ok?

Sounds good @mateoclarke

https://www.open-austin.org/candidate-questionnaires/2016.html
This is as close as I could get to the mock without spending a bunch more time on small things this weekend.
Some things that could/should still be addressed if there is more time:

[ ] Make the 5 districts tabs size down to work on mobile
[ ] Add the navigable table of contents fixed sidebar section that Vickie and originally designed
[ ] Add hover states to svg icons
[ ] create a new page template so the whole width of the bottom half can have a grey background like Vickie's design called for.


Improve Swagger documentation

Is your feature request related to a problem? Please describe:
the swagger documentation doesn't allow you to try some endpoints because you can't insert required information. For example, See /api/plugins/changelog/{pluginName}
You can't edit the pluginName so you can't issue the command.
Describe the solution you'd like:
These should be editable so that the documentation is more helpful to people learning the API.
Describe alternatives you've considered:

Additional context:
Other endpoints such as /api/server/cached-accessories/{uuid} allow the user to edit

Done in the 4.25.x.

Awesome! Thank you.


setTimeout is not getting cleared when the user sign out

Currently the timeout that is set to check for the refresh token is not getting cleared and keeps on making requests to the /api/auth/user endpoint when the user is signed out.

Fixed in<EMAIL_ADDRESS>


"fallback to vim's indent file " is dangerous on plain text

When editing a plain text, the tab are modified when I save the file, which is very dangers. For most plain texts need not formatting.
Although we can turn this off with config, but I still suggest turn this off by default.

What do you mean with 'the tab are modified'?

My Vim will turn tab into four space when I am writing. I believe a lot of people have this setting.
When I open a plain text containing tab characters, modify nothing, and then save with vim-autoformat. The "fallback to vim's indent file" action will change all the tabs to spaces with no warnings.
Plain texts are not like codes. They have no need to be formatted.
So I suggest to turn the fallback action off by default.

If you have expandtab set to true for plain text, then it makes sense to convert tabs to spaces if you ask vim-autoformat to format you text. If you would type things, vim would convert your inserted tabs to spaces too.
More generally, vim-autoformat is by default configured to do the most sensible thing when :Autoformat is executed, based on your vim settings. If you have more exotic wishes, you can configure things differently.
If you don't want the fallback, then maybe you shouldn't have executed the command in the first place. And when you start tweaking things like adding autocmds, then you are expected to be savvy enough to simply turn off the fallback features.
And I disagree about regular text being inherently different from code in this respect. Even plain text has layout features like e.g. line-width, indentation size, indentation characters.


Fix DPI scaling not working correctly when running under SDL2

Tested on macOS
Closes https://github.com/ppy/osu-framework/issues/4044.
@ben9583 would appreciate confirmation that this fixes the issue for you too.

I'm having some issues; the game only takes up the bottom left of the screen and the cursor in game is mapped to only be in the bottom corner.
video

Are you sure you are testing on this PR? I've made no changes which could cause that.

Yes, I cloned from peppy:fix-dpi-scale; on master, the window is 1/2 the length and height, but works properly. It can be scaled up to the same size as on peppy:fix-dpi-scale and have the game take up the whole screen.

So I'm not really sure how this happened, but the issue seems to have fixed itself... After rebuilding a couple times on the peppy:fix-dpi-scale branch, the resolution is proper and the hover/click events on osu.Framework.Tests is working as expected. I'm not really sure what happened.


build-opencv wording is not clear

Hello @UrielCh,
I'm sharing with you my feedback regarding build-opencv tool.
I feel there are 2 level of build here :

openCV build
opencv4nodejs build

I'm not clear about the meaning of the verbs in the command :

'build/rebuild' :

what is the difference between build and rebuild ?
Is it building only opencv4nodejs ?
Is it building also openCV ? in what condition ?
When opencv4nodejs is already built is it rebuilding it ?
When openCV is already built is it rebuilding it ?

'clean/remove' : what is the difference ? Is it working on opencv4nodejs level ? or openCV level ?
'auto' : Same question

I would be happy to discuss this on slack, i feel there is a romm for improvement here in the documentation and/or architecture

1st point:

Build only build module will not works if called without a previous configure
Rebuild equivalent to clean + configure + build

...
All the command are the same as in node-gyp

Command
Description

help
Shows the help dialog

build
Invokes make/msbuild.exe and builds the native addon

clean
Removes the build directory if it exists

configure
Generates project build files for the current platform

rebuild
Runs clean, configure and build all in a row

install
Installs Node.js header files for the given version

list
Lists the currently installed Node.js header versions

remove
Removes the Node.js header files for the given version

The only change is the addition of the auto command.
auto will try to build a working binding without any options. I use it during the installation phase.
You can open a doc branch on UrielCh/opencv4nodejs
I already grant you write access to this git.

Ok, will try to add some doc, thanks for the explanations

In the new version, I moved all macros inside the cc/native-node-utils directory, I started to add some comments on macro, and I would appreciate some assistance on this part.
Ideally, converting macro to N-API instead of NaN will upgrade the project and remove the need for recompilation.


T/qa 2157/apply black

Description
Checklist

[ ] Added proper unit tests
[ ] Left proper TODO messages for any remaining tasks
[ ] Scanned for web accessibility with aXe, and mitigated or documented
flagged issues

Screenshots/Screen Recordings (if Appropriate)

Resolved merge conflics.
@ManjunathJayanna and @bpierce01 I need both of

There are merge conflicts

@bpierce01 fixed. should be good now.

@craigrbarnes could you please merge this whenever you have a chance?


[e2e]: generic tag main assets slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v18.0.216
Run: https://github.com/slsa-framework/example-package/actions/runs/3879712657
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.tag.main.assets.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.tag.main.assets.slsa3.yml
Trigger: push
Branch: v18.0.216
Date: Tue Jan 10 02:48:35 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/v18.0.217
Run: https://github.com/slsa-framework/example-package/actions/runs/3889172510
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.tag.main.assets.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.tag.main.assets.slsa3.yml
Trigger: push
Branch: v18.0.217
Date: Wed Jan 11 02:51:16 UTC 2023
Tests are passing now. Closing this issue.


Change resources content

Heading: Integrated Resources Tab
Content: Access the right resources, tutorials, and Git guides directly inside GitHub. You’ll always have the help you need at your fingertips. How it works: New to Git? Learn essential commands without leaving the project page, boosting your productivity without needing external searches.

Note: How it works should be on a new line, you can just use /n

I want to solve this issue can you please assign me?


103 is not a standard, this was only a draft.

The 103 status comes from here: https://code.google.com/p/gears/wiki/ResumableHttpRequestsProposal. It has never been in HTTP1.1, it was a draft.
Moreover, the Wikipedia link is not up-to-date.

As of the latest version I have dropped the 103 status code. Thank you.

You're welcome.


Customize GenerateRedirect body

Sometimes it might be nice to customize the body of the GenerateRedirects output documents. Add a RedirectBody metadata value that will be used to replace the stock "This page has moved..." text.

Done and will ship in the next release going out tonight.


Fine-tuning example

Please add an example of using fine-tuning with hifi-gan.

We added Fine-Tuning section to the README. Please refer to it.


Question about selected dates/events

Hey,
I stumbled across this in my attempt to find a library to use for an events type calendar.
Is it possible to use this library in a way where I can feed it a set of dates and have those highlighted differently than the others?

I was able to figure out date decorators...awesome library!


molecule destroy failing with yaml.parser.ParserError

Issue Type

Bug report

Molecule and Ansible details
ansible <IP_ADDRESS>

molecule, version 2.3.1

Molecule installation method (one of):

source
pip

Ansible installation method (one of):

source
pip

Actual Behaviour (Bug report only)
I'm running molecule in 4 scenario's via jenkins job. I observed with "molecule destroy " is failing with yaml parse error while reading state.yml.  This happened to me like 4 times in 110 builds.  here is full stack trace for the error
looks like state.yml is being corrupted while execution, any help would be greate
+ molecule destroy --scenario-name skynet
--> Test matrix

└── prod
    └── destroy

--> Inventory /var/lib/jenkins/workspace/Ansible Molecule Tests/ansible/molecule/prod/.molecule/../../../prod/group_vars linked to /var/lib/jenkins/workspace/Ansible Molecule Tests/ansible/molecule/prod/.molecule/group_vars
--> Scenario: 'prod'
--> Action: 'destroy'
Traceback (most recent call last):
  File "/usr/local/bin/molecule", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/molecule/command/destroy.py", line 107, in destroy
    base.execute_subcommand(scenario.config, term)
  File "/usr/local/lib/python2.7/dist-packages/molecule/command/base.py", line 99, in execute_subcommand
    return command(config).execute()
  File "/usr/local/lib/python2.7/dist-packages/molecule/command/destroy.py", line 62, in execute
    self.prune()
  File "/usr/local/lib/python2.7/dist-packages/molecule/command/base.py", line 65, in prune
    self._config.state.state_file,
  File "/usr/local/lib/python2.7/dist-packages/molecule/config.py", line 197, in state
    return state.State(self)
  File "/usr/local/lib/python2.7/dist-packages/molecule/state.py", line 68, in __init__
    self._data = self._get_data()
  File "/usr/local/lib/python2.7/dist-packages/molecule/state.py", line 120, in _get_data
    return self._load_file()
  File "/usr/local/lib/python2.7/dist-packages/molecule/state.py", line 132, in _load_file
    return util.safe_load_file(self.state_file)
  File "/usr/local/lib/python2.7/dist-packages/molecule/util.py", line 186, in safe_load_file
    return safe_load(stream)
  File "/usr/local/lib/python2.7/dist-packages/molecule/util.py", line 175, in safe_load
    return yaml.safe_load(string) or {}
  File "/usr/local/lib/python2.7/dist-packages/yaml/__init__.py", line 93, in safe_load
    return load(stream, SafeLoader)
  File "/usr/local/lib/python2.7/dist-packages/yaml/__init__.py", line 71, in load
    return loader.get_single_data()
  File "/usr/local/lib/python2.7/dist-packages/yaml/constructor.py", line 37, in get_single_data
    node = self.get_single_node()
  File "/usr/local/lib/python2.7/dist-packages/yaml/composer.py", line 39, in get_single_node
    if not self.check_event(StreamEndEvent):
  File "/usr/local/lib/python2.7/dist-packages/yaml/parser.py", line 98, in check_event
    self.current_event = self.state()
  File "/usr/local/lib/python2.7/dist-packages/yaml/parser.py", line 174, in parse_document_start
    self.peek_token().start_mark)
yaml.parser.ParserError: expected '<document start>', but found '<block mapping start>'
  in "/var/lib/jenkins/workspace/Ansible Molecule Tests/ansible/molecule/prod/.molecule/state.yml", line 4, column 1

Include a minimum complete verifiable example with
output of running molecule --debug.

Please upgrade to a newer version of Molecule.
2.3.1 is old.  2.12.1 is the current release.


New CPIS tutorial need to be published

Hi @thecodester,
Kindly publish this tutorial.
I understand its a last-minute request but PM team wanted it at the earliest.
Appreciate your help.
Regards,
Vikram Kulkarni

Hi @thecodester  and @MichaelCzcz ,
Please my comments inline.
Step 7 -- if you have 1 substep, do not number  --> Numbering removed
Step names should be senetence case -- e.g., "Configure administrator access"
Title of tutorial should be "Onboard to SAP Integration Suite Trial" --> Title name changed
trial is almost always lower case
Step 3 substep 2 -- the image does not start on its own line -- blank line in front of note in step 1, indent note 4 spaces, blank line before picture in substep 2 --Removed blank line
Same thing in Step 4 -- blank lines between all items in a list -- Removed blank lines
Pictures quite small in Step 5 -- really cant see what is happening --> its over all screen, we avoided scrolling option. Hence it look like. End user upon zooming can able to understand


Check flatpak runtime dir first on Linux

This change would allow SlimeVR to work when the server is running inside a flatpak.
This change has SlimeVR look for sockets in $XDG_RUNTIME_DIR/app/dev.slimevr.SlimeVR before other locations, since this is the only place inside $XDG_RUNTIME_DIR SlimeVR-Server can write to when running inside a flatpak.
If Steam is also running inside a flatpak, we can gain access to this folder on the host by running flatpak override --user --filesystem=xdg-run/app/dev.slimevr.SlimeVR:create com.valvesoftware.Steam.

SteamVR beta has sandboxed $XDG_RUNTIME_DIR, I don't think we should keep using it

I see. If you'd like, I can modify it to look in ~/.var/app/$FLATPAK_ID instead

i guess the current idea is to go to XDG_DATA_DIR instead

I see. If you'd like, I can modify it to look in ~/.var/app/$FLATPAK_ID instead.
That's too bad since the runtime dir is technically where it belongs

but you just set your launch argument to this and it magically fixes steamvr:
~/.local/share/Steam/steamapps/common/SteamVR/bin/vrmonitor.sh %command%


Set baseline widths for page layouts

Using new variables from https://github.com/AlaskaAirlines/OrionDesignTokens/issues/9 crate baseline CSS layout width matrix.

Per other updates, this issue is no longer valid.


Submits assessment

I'll finish this later! Spent too much time troubleshooting an error.

1/6
Don't sweat the score! Just going through the motions here. If you have any questions about the contents of the quiz, just grab myself or one of the other instructors, and we'd be happy to help!


Feed shows very old posts and needs refreshing a few times to load more recent ones

Reproducing the issue:

Load the feed after having already been logged in for a few hours
Notice that at first load, it'll be a mix of very old posts, some more recent ones, and then new ones that roll in
If you refresh, more recent posts will be added, but it will still be missing many when comparing to what I see on Snort or Iris

There are some heuristics I use to prune very old posts, but it's pretty error prone due to how nostr works. I'm continually paying attention to this and improving the algorithm, so I'm going to close this issue.


require('path/' + moduleName) fails if module is in moduleName/index.js file

Assume we have this folder structure
src
├── plugins
│   ├── Plugin1
│   │   └── index.js
│   └── Plugin2
│       └── index.js
└── entry.js

entry.js is webpack entry module with code like this
module.exports = function askPlugin(pluginName, question) {
    var plugin = require('./plugins/' + pluginName)
    return plugin.respondTo(question)
}

when packing, webpack takes plugins directory and builds files map like this
var map = {
    "./Plugin1/index": 4,
    "./Plugin1/index.js": 4,
    "./Plugin2/index": 5,
    "./Plugin2/index.js": 5
};

this map is incomplete, because there are missing folder modules ./Plugin1 and ./Plugin2 (in this example). So, correct map should be
var map = {
    "./Plugin1": 4,
    "./Plugin1/index": 4,
    "./Plugin1/index.js": 4,
    "./Plugin2": 5,
    "./Plugin2/index": 5,
    "./Plugin2/index.js": 5
};

because we can call require('./plugins/Plugin1') and it is correct shorthand for require('./plugins/Plugin1/index')

@skyh What's the status of this? Do you want to PR that change of yours?

I don't remember. Please, close this PR.

@skyh What's the status of this? Do you want to PR that change of yours?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub


add signatory_name field in ccla signatures.

Summary
add signatory_name field in ccla signatures.
User Story

As a company employee, I want to know name of cla signatory for project.

Acceptance Criteria
The "done" criteria when this feature or problem is resolved. Such as:

Unit Tests added and running in CI
Functional Tests updated to cover feature, if applicable
Demonstrate the set of capabilities to the product team while the code is running in the STAGING environment.

Tested below scenarios
1. When CLA Manager Designee signs the CLA

2. When CLA Signatory signs the CLA


added sendFile

I wanted to be able to send files with hyper-ts but there was no way to use it natively. I had to write Express code and then use fromRequestHandler to turn it into a hyper-ts middleware.
One can use sendFile it in lieu of send. It will also transition the index from BodyOpen to ResponseEnded. It takes an error handler just like json does, and it will trigger the error handler if one of three things is true:

the path is a relative path (Express, for example, only allows absolute paths, so this is somewhat opinionated); I've also named the parameter in sendFile absolutePath to hint to an end user that the path must be absolute.
the path is to a file, not a directory
the file exists

I updated the ExpressConnection to work with this as well and added some unit tests.

Also I started to add a @since 0.7 but wasn't sure if I should. I figure if the PR is accepted, a maintainer would correct that.

One thing to address: should sendFile attempt to auto-detect mime type and add the appropriate Content-Type header? I have not done this because the most straightforward way to do this would be to add another dependency that does this (there are so many mime types!). I didn't want to force a dependency.
I suppose sendFile could take an optional Content-Type parameter. Might that be the right way to do it? Or should we just expect the user to explicitly chain addHeader('Content-Type', 'application/pdf') for example?


stable/fluent-bit: Add labels to all objects

Also: restrict the Service selector somewhat to include the release
name.
Fixes: #6672
See: https://github.com/kubernetes/charts/issues/6672

/assign @linki
CC @kfox1111 @edsiper

lgtm

/assign @kfox1111

/assign @edsiper

/lgtm

another 0.7 was merged recently. please bump the version again.

/lgtm


About the performance of PSPNet.

Hello, thanks for your perfect work. I have a question about the performance of PSPNet , when i use PSPNet alone  in my own dataset and my own code and trainning with 1/2 samples,  the miou could reach about 68%. But when I change to your code and trainningwith suponly, the miou is only 60% .  Could you please tell me what may be the reason for this.

Hi, thanks for your attention.
How do you set the batch size?


torcx/perform: make it works with enforced SELinux

Revert "internal: enable XattrPrivileged for untar to fix selinux issue

This is required in order to avoid this denial:
Nov 15 09:45:43 localhost audit[688]: AVC avc: denied { associate } for pid=688 comm="torcx-generator" name="docker" dev="tmpfs" ino=2 scontext=system_u:object_r:unlabeled_t:s0 tcontext=system_u:object_r:tmpfs_t:s0 tclass=filesystem permissive=0

After some investigations, we found the following reason: during the unpacking of the Torcx image with XattrPrivileged option, we try to set the xattr on the extracted files. If the security.selinux attribute is not set, it defaults to unlabeled_t which is not able to be associated with the targeted filesystem. This denial prevents Torcx to be ran with SELinux enforced mode -> no docker/containerd.
Which leads to the second commit.

perform: do not remount in RO

Once the image unpacked, the Torcx image is sealed (means remounted as read-only) - the Torcx image is fully unlabeled, so we need to define filecontexts for the most important files in it. Since Torcx runs as a systemd-generator, file contexts are not available during the unpacking so we need to wait for systemd-tmpfile to relabel /run to get the proper labels.
The remount in ro can be performed right after.

The remount in ro can be performed right after.

Do you have a change somewhere that does it?

The removal of readonly mount makes me slightly uncomfortable - would temporarily remounting r/w outside torcx before relabeling work?
As to the second issue - sounds like the tarball contents are not properly labeled?

Do you have a change somewhere that does it?

@krnowak not yet - I was thinking about a systemd mount unit running after systemd-tmpfiles-setup

As to the second issue - sounds like the tarball contents are not properly labeled?

@jepio initial content of the Torcx image is not labelled because SElinux is not enabled in the SDK, so when building the tarball, we can set the --selinux --xattrs option but it won't do anything since files are not initially labelled.
Adding SELinux in the SDK can be complicated, for example with the SDK container, since it's the policy from the host system that will be mounted.

Closing this in favor of Torcx relabeling during the build.


Add a link to Android Testing Library site to README.md

Hi,
I think it would be good to add a link to this site https://google.github.io/android-testing-support-library/ in README.md file.
Sometimes after visit this great repository I need to go to this site to learn something more, but there's no direct link to Android Testinmg Library page. Could you fix it?
Cheers

I've been meaning to do that so I'll do it now, the ATSL site is not very discoverable. Thanks!
https://github.com/googlesamples/android-testing/pull/72

Thanks for a quick aswer


gl_engine: Fix calculation error in path triming

This PR rewrite the PathTrim code and fixed almost all the issue caused by PathTrim inside GL backend.
related issue: #2799

The following cases is correct with this Patch.

birth_stone_logo
fly_in_beaker
frog_vr
cat_loader
happy_holidays
holdanimation
isometric
skullboy
threads
lolo
seawalk
expressions/10151
expressions/world_locations
expressions/jolly_walker // This animation is rendered same as Software raster, but I am not sure it is correct

Please check if I am wrong

there are some issues, but it's a huge improvement !

SW
GL before
GL after

I can not see any changes in :
isometric.json
seawalk.json
skullboy.json
but the rest of the samples look better

Some errors seem to be caused by PathStroke. Can we focus on the logic of PathTrim and merge this commit first?

@RuiwenTang awesome, only 5 left. :)


fix: solved issue #7818

Description
I updated checks to resolve an issue where an incorrect error was being displayed. The changes include:

Removed references to the deprecated MIDDLEWARE_CLASSES
Assigned middlewares to an empty list if the attribute 'MIDDLEWARE' was not found.

Related resources

#7818

Checklist

[x] I have opened this pull request against develop-4
[ ] I have added or modified the tests when changing logic
[x] I have followed the conventional commits guidelines to add meaningful information into the changelog
[x] I have read the contribution guidelines  and I have joined #workgroup-pr-review on Slack to find a “pr review buddy” who is going to review my pull request.

Thanks @fsbraun for the quick PR approval!


adding css method

Couple points from .calc_css_percentile function (I think that I fixed most of them already). The functions looked fine and it worked well, but there was room for improvement

Some of the indentations were incorrect. Should be multiplication of 4 spaces
Some lines were too long. Should be max 80 characters
Avoid loops. Vectorize if possible (faster than loops, and also good practice)
Instead of sapply, use vapply. (Comes from Bioconductor recommendations)
Instead of 1:10, use seq_len(10). (Comes also from Bioc)

@Daenarys8 could you add the suggested updates and make a PR against the SHillman836::adding-css-method branch? Or can @SHillman836 help to finalize this one?

metagenomeSeq is not on cran and that is messing up the checks. Experienced it before?

metagenomeSeq is not on cran and that is messing up the checks. Experienced it before?

Actually it is there but since we ran workflow with devel where metagenomeSeq is currently failing build, gha could not add it. I have put temporal solution to run gha with release. We can restore devel later. It was either this or passing the env _R_CHECK_FORCE_SUGGESTS_ as false  which perhaps is not the best ...

we can ask them?

It seems that metagenomeSeq is only used in testing? We can probably just hard-code those values to testing file. That way we do not need this new dependency.

we can ask them?

Sure. I found an existing issue from March this year and updated it. It seems the package had been inactive for a while.

It seems css and css_fast is already in mia?
@TuomasBorman

It seems css and css_fast is already in mia? @TuomasBorman

It did not include the functionality; it included only "css" as one of the options.  I am not sure how that ended up there, probably I copy-pasted the method-vector there incorrectly


build azcmagent for Ubuntu 24.04

Describe the problem you are experiencing.
there is presently no azcmagent for Ubuntu 24.04 hosts
Describe any possible solutions that you would like to see.
please build and package azcmagent for Ubuntu 24.04 LTS
Additional context.

Upgrading an Ubuntu 22 host with azcmagent - azcmagent still works without issue - just cannot install from repo from a clean install
suspect if i manually installed the jammy/22.04 package it would work

@anthonysomerset - This repo backs up the "packages.microsoft.com" service itself, which handles package delivery. Various teams across Microsoft own the packages themselves and are responsible for the content. Some package publishers have a presence on github themselves; for those publishers, issues should be reported there. Other publishers don't have a github presence; for them, the most certain way of getting an issue resolved would be through Microsoft customer support.
That being said, we've forwarded your request to the folks who own azcmagent and asked them to respond here.

thanks - i did genuinely look for the right place to raise this to and couldn't find it, i am more than happy to report it in the correct place, just need to be guided where

We'll be including Ubuntu 24.04 in a near-future release.

I'm closing this issue, as there's no further action for the packages.microsoft.com team.
@edyoung if you could reply here once the 24.04 package is available, or share a link where customers can go for news/updates on this topic, that would be great :)

azcmagent should be available to download for Ubuntu 24.04 starting version 1.45, which was published last week.


TLS problem :  ERR_TLS_CERT_ALTNAME_INVALID

[2021-05-17 14:17:36] [INFO] Connect close, MQTT.js onClose trigger
[2021-05-17 14:20:53] [INFO] Connect client, MQTT/SSL connection: mqtts://<IP_ADDRESS>:8883
[2021-05-17 14:20:53] [INFO] MQTTX client with ID d405d3a0-b170-49b8-a7ea-5297e60c97ab assigned
[2021-05-17 14:20:53] [ERROR] Connect fail, MQTT.js onError trigger, Error [ERR_TLS_CERT_ALTNAME_INVALID]: Hostname/IP does not match certificate's altnames: IP: <IP_ADDRESS> is not in the cert's list:
at Object.checkServerIdentity (tls.js:261:12)
at TLSSocket.onConnectSecure (_tls_wrap.js:1267:27)
at TLSSocket.emit (events.js:194:13)
at TLSSocket._finishInit (_tls_wrap.js:734:8)
at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:566:12)
[2021-05-17 14:20:53] [INFO] Connect close, MQTT.js onClose trigger

Hi! Thanks for using MQTT X. I think this is not a bug. It should be that ALTNAME is missing from the certificate configuration. Perhaps you can refer to the following link to configure:
https://www.emqx.io/blog/emqx-server-ssl-tls-secure-connection-configuration-guide

thanks a lot

thanks a lot

Hi! Thanks for using MQTT X. I think this is not a bug. It should be that ALTNAME is missing from the certificate configuration. Perhaps you can refer to the following link to configure:
https://www.emqx.io/blog/emqx-server-ssl-tls-secure-connection-configuration-guide

sorry,Link display 404

Hi! Thanks for using MQTT X. I think this is not a bug. It should be that ALTNAME is missing from the certificate configuration. Perhaps you can refer to the following link to configure:
https://www.emqx.io/blog/emqx-server-ssl-tls-secure-connection-configuration-guide

sorry,Link display 404

https://www.emqx.com/en/blog/emqx-server-ssl-tls-secure-connection-configuration-guide

Hi! Thanks for using MQTT X. I think this is not a bug. It should be that ALTNAME is missing from the certificate configuration. Perhaps you can refer to the following link to configure:
https://www.emqx.io/blog/emqx-server-ssl-tls-secure-connection-configuration-guide

sorry,Link display 404

https://www.emqx.com/en/blog/emqx-server-ssl-tls-secure-connection-configuration-guide

thanks


PAT Scope

It would be good to know which scopes are required to interact with each section of the api

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

ID: 31074e0d-62b1-31cf-8d16-f957cfb750f9
Version Independent ID: 1cff91ee-df5a-8828-a658-b8c7b47eb361
Content: Repositories (VSTS Git)
Content Source: docs-ref-autogen/4.1/git/Repositories.yml
Product: devops
GitHub Login: @wnjenkin
Microsoft Alias: whjenkin

@Amaranthos thanks for reporting this. Agreed, this is something we'd like to support. Keeping this open for tracking and adding this item to the backlog.


What kind of crash would be triggered when concurrently access to CGContext

What kind of crash would be triggered when concurrently access to CGContext

It's not that there would be a crash from that, its that concurrent doing heavy memory operations (often associated with CGContext work in TIP) will use so much memory that the available memory can quickly run out yielding any number of random crash from running out of memory including being purged by the OS.

thanks a lot


SideBar - On the right

Hi,
Thanks for your wonderful template. I need to create an additional toggle sidebar on the right side to add some filters. How do I go about doing it?

@sumitdhameja since the code that we implemented is working for only 1 sidebar. You have 2 options:

write new code for the right sidebar and made the changes to the wrapper and main-panel in order for them to support the new sidebar
check this library: https://slideout.js.org/ and see if that helps in adding a right sidebar faster.

Hope this helps.
Best,
Alex


Fix missing links on README closes #64

will fix #64

Coverage remained the same at 96.172% when pulling 1fddba7cb9461e836d80126863ee4d2badf0be3c on issue64 into 1d8d563cb36ef45cf3fa5d04a7dc31c0b039228c on master.

:-)


Akeaveny/82/uwrt drive train ros controllers

This branch contains a ROS Controller for the drive train

Open Loop Position Controller

Subscribes to UWRTOpenLoopPosCmd Twist msg: linear & angular position commands
Outputs: left & right position setpoints

TODO (akeaveny):

Open Loop Position Controller

Look at scaling factor from Xbox Joystick
try a ROS srv call to lock position offset from Xbox Joystick

closes #50

@akeaveny thanks for useful pr description.
For #2, is there any reason u decided to recreate the diff drive controller? did u make any modifications to the package because we were already using the diff drive controller?

@akeaveny thanks for useful pr description.
For #2, is there any reason u decided to recreate the diff drive controller? did u make any modifications to the package because we were already using the diff drive controller?

I deleted the custom diff drive controller as it was a duplicate of the standard ROS controller

superceeded by #128


Fix sLSTM and mLSTM to work with 1 layer

sLSTM and mLSTM have dropout layers with length of 1 less than other layers:
self.dropout_layers = nn.ModuleList([nn.Dropout(dropout) for _ in range(num_layers - 1)])

So if you will try to run models with only one layer, no layers will be applied to input because of this for:
for i, (lstm, dropout, f_gate, i_gate) in enumerate(zip(self.lstms, self.dropout_layers, self.exp_forget_gates, self.exp_input_gates))

How to replace lstm with slstm when num_layers=1


How to force load some html using include plugin?

I have a piece of html in my footer that I am using the include plugin to asynchronously load, but I would like to force load it on the DOMContentLoaded event, not just when the parent div is visible.
Ideally, I'd like to be able to add a class that will force the load.
How can I force load some html using the include plugin?

Nevermind, I found it in the code. I can add the lazypreload and it works exactly how I want it to.
Thanks for your amazing work, it's really some of the most useful code I've ever found.


Added required files #1753

Added html , css , js for quiz_game
and a screenshot of my game
fixed #1753

Thank you @manshasinghal ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Added required files

Hey @manshasinghal,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗

Thank you  @manshasinghal , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Adds documentation for SPIRE SDS integration

Please provide a description for what this PR is for.
Adds Spire SDS Integration Documentation.

[ ] Configuration Infrastructure
[X] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

does this PR have the same content as #1? can't we have the commits on #1 squashed? that way we don't lose track of comments and discussions

@mchurichi great point, I agree that we should be squashing the commits from #1


Add MCAP compression settings

Codecov Report
Patch coverage: 4.16% and project coverage change: -0.20 :warning:

Comparison is base (3e05767) 18.84% compared to head (b20be43) 18.65%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
- Coverage   18.84%   18.65%   -0.20%
==========================================
  Files          23       24       +1
  Lines        2138     2160      +22
  Branches     1136     1155      +19
==========================================
  Hits          403      403
- Misses       1357     1378      +21
- Partials      378      379       +1

Impacted Files
Coverage Δ

...participants/src/cpp/recorder/mcap/McapHandler.cpp
42.06% <0.00%> (-0.23%)
:arrow_down:

ddsrecorder_yaml/src/cpp/recorder/YamlReader.cpp
0.00% <0.00%> (ø)

..._yaml/src/cpp/recorder/YamlReaderConfiguration.cpp
8.66% <0.00%> (-0.14%)
:arrow_down:

ddsrecorder/src/cpp/tool/DdsRecorder.cpp
70.00% <50.00%> (+0.50%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Mediapipe doesn't work as npm package

Hello,
I'm trying to use medipipe as dependency of my npm project.
You can find my project here: https://github.com/esnho/mediapipe-minimal-setup
@mediapipe dependencies can be installed with no issues, the problem is when I try to import Mediapipe in my source.
Visual Studio Code is complaining that the module can't be found, webpack is throwing the same error:
ERROR in ./Index.ts
Module not found: Error: Can't resolve '@mediapipe/camera_utils' in '/Users/lmoroni/code/mediapipe-minimal-setup/src'
 @ ./Index.ts 3:0-49 7:12-18
 @ multi ./Index.ts

ERROR in ./Index.ts
Module not found: Error: Can't resolve '@mediapipe/control_utils' in '/Users/lmoroni/code/mediapipe-minimal-setup/src'
 @ ./Index.ts 4:0-56 8:12-24
 @ multi ./Index.ts

ERROR in ./Index.ts
Module not found: Error: Can't resolve '@mediapipe/drawing_utils' in '/Users/lmoroni/code/mediapipe-minimal-setup/src'
 @ ./Index.ts 5:0-80 9:12-26 10:12-32
 @ multi ./Index.ts

ERROR in ./Index.ts
Module not found: Error: Can't resolve '@mediapipe/face_mesh' in '/Users/lmoroni/code/mediapipe-minimal-setup/src'
 @ ./Index.ts 2:0-48 6:12-20
 @ multi ./Index.ts

To reproduce clone above repository, and follow the README file.
Thanks.

Hi @esnho  Did you verify your import paths ? Have a look over similar issue

Thanks, with the right import resolved the error, I've updated above repository with the solution.
Do you plan to support "root imports"?

@esnho  We will let you know if there are any. Closing this issue as you resolved from the above comments.


catch exceptions in Android Database helper

Fixes a silent crash on Android.
Potential causes:

the app updated but has not been opened yet, so the databases have not been created.
For some reason the app failed to create the database and potentially alerted the user that the upgrade could not be performed.

Ticket Link
https://mattermost.atlassian.net/browse/MM-50005
NONE

Cherry pick is scheduled.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
From github.com:mattermost/mattermost-mobile
   34aef73ac..7ed2e73a9  main       -> upstream/main
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
!!! 'upstream/release-2' not found. The second argument should be something like upstream/release-0.21.
    (In particular, it needs to be a valid, existing remote branch that I can 'git checkout'.)

+++ Returning you to the master branch and cleaning up.

/cherry-pick release-2.0

Cherry pick is scheduled.

/cherry-pick release-2.1


Db 226459 Update HikariCP to 5.1.0 version

Update copyright notices on files you edited, such files should have
Copyright (C) 2024 OneVizion, Inc. All rights reserved.

Done


Remove warning when NumPy slice is used

Context: https://github.com/dmlc/xgboost/issues/6908#issuecomment-834658652
Scikit-learn's grid search CV send sliced NumPy arrays into XGBoost, leading to many warnings of form
Use of np.ndarray subsets (sliced data) is not recommended
because it will generate extra copies and increase
memory consumption. Consider using np.ascontiguousarray to
make the array contiguous.

Users have no way to control the behavior of scikit-learn, so the warnings only serve to annoy the user. For now, just remove the warning.
To really address the increased memory consumption caused by copying sliced arrays, we will need to implement #6908 (Use array interface in NumPy) in order to ingest sliced arrays without making copies.

I think it should be possible to use array interface:  https://github.com/trivialfis/xgboost/tree/numpy-array-interface

@trivialfis Yes, that's possible. But it will be a while until I get around adopting the facilities from in-place predictor into DMatrix.

Feel free to take a look at my branch linked above.  Should be PR ready.

@trivialfis Great! Would you like to file a PR?

Yup, opened a PR https://github.com/dmlc/xgboost/pull/6998 .  Would be great if we can continue discussion on https://github.com/dmlc/xgboost/pull/6968 .

Closing in favor of #6998.


[Bug] Failed to launch the browser process! in the docker

When using n8n docker and installing puppeteer as a community node (https://www.npmjs.com/package/n8n-nodes-puppeteer-extended)
Error shows:
`Error: Failed to launch the browser process! spawn /home/node/.n8n/nodes/node_modules/puppeteer/.local-chromium/linux-1002410/chrome-linux/chrome ENOENT
TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md
at onClose (/home/node/.n8n/nodes/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserRunner.js:286:20)
at ChildProcess.<anonymous> (/home/node/.n8n/nodes/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserRunner.js:280:24)
at ChildProcess.emit (node:events:527:28)
at Process.ChildProcess._handle.onexit (node:internal/child_process:289:12)
at onErrorNT (node:internal/child_process:478:16)
at processTicksAndRejections (node:internal/process/task_queues:83:21)`

As far as i know from n8n community, installing Puppeteer in docker does not install all needed dependencies.
Can you provide a solution for this?

I'm hitting the same problem in Jenkins running the snapshot test. Any idea of this?

see the error is ENOENT which means chrome executable file does not exist.
For that you can install chrome separately using
wget -q -O - https://dl-ssl.google.com/linux/linux_signing_key.pub | apt-key add - && \
sh -c 'echo "deb [arch=amd64] http://dl.google.com/linux/chrome/deb/ stable main" >> /etc/apt/sources.list.d/google-chrome.list' && \
apt-get -y update && \
apt-get install -y \
google-chrome-stable && \
apt-get install -yqq unzip && \
wget -O /tmp/chromedriver.zip http://chromedriver.storage.googleapis.com/`curl -sS chromedriver.storage.googleapis.com/LATEST_RELEASE`/chromedriver_linux64.zip && \
unzip /tmp/chromedriver.zip chromedriver -d /usr/local/bin/

and in your code provide the chrome launch path as
/usr/local/bin/chrome OR /usr/local/bin/chrome-stable not sure whichever is.
puppeteer will launch chrome from that path

@Jazeb
After executing the code what you provided the only thing I have in /usr/local/bin is chromedriver.
On the other hand, I found it in the path /usr/bin/google-chrome and /usr/bin/google-chrome-stable so I tested these two options in the application while both return the same error.
How is the application inside the docker container supposed to access chrome data from outside the container?

Please file a bug with n8n-nodes-puppeteer-extended


Add conversion to std io error and better description

I'm adding bzip2 support for the async-compression crate, c.f. https://github.com/rustasync/async-compression/pull/45, and therefore I need conversion to std::io::Error to reuse  existing generic code.
This PR adds this conversion. And while I was at it, added a more detailed error description.
I would be highly surprised if this breaks anything.
It would be great if this could be merged :smile:

Well, technically it's not needed anymore, since I converted the errors manually where needed, however, I still think having it in bzip2-rs would be nice.

These changes look great to me, thanks!


Eventhubs-support read from and write to Azure eventhubs

Thank you very much for contributing to Apache Flink - we are happy that you want to help us improve Flink. To help the community review your contribution in the best possible way, please go through the checklist below, which will get the contribution into a shape in which it can be best reviewed.
Please understand that we do not do this to make contributions to Flink a hassle. In order to uphold a high standard of quality for code contributions, while at the same time managing a large number of contributions, we need contributors to prepare the contributions well, and give reviewers enough contextual information for the review. Please also understand that contributions that do not follow this guide will take longer to review and thus typically be picked up with lower priority by the community.
Contribution Checklist

Make sure that the pull request corresponds to a JIRA issue. Exceptions are made for typos in JavaDoc or documentation files, which need no JIRA issue.

Name the pull request in the form "[FLINK-1234] [component] Title of the pull request", where FLINK-1234 should be replaced by the actual issue number. Skip component if you are unsure about which is the best component.
Typo fixes that have no associated JIRA issue should be named following this pattern: [hotfix] [docs] Fix typo in event time introduction or [hotfix] [javadocs] Expand JavaDoc for PuncuatedWatermarkGenerator.

Fill out the template below to describe the changes contributed by the pull request. That will give reviewers the context they need to do the review.

Make sure that the change passes the automated tests, i.e., mvn clean verify passes. You can set up Travis CI to do that following this guide.

Each pull request should address only one issue, not mix up code from multiple issues.

Each commit in the pull request has a meaningful commit message (including the JIRA id)

Once all items of the checklist are addressed, remove the above text and this checklist, leaving only the filled out template below.

(The sections below can be removed for hotfixes of typos)
What is the purpose of the change
(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache

Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluser with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Hi @zhuganghuaonnet, thanks a lot for this contribution!
However, taking a step back, is there a JIRA ticket for this contribution?
If not, please make sure that one is opened for it first. Also, for bigger contributions like this, you should make sure that the pull request template is properly filled out to ease the effort for reviewers.

Hi Tai,
I cannot find JIRA related to Azure eventhubs. I will create one later, fill pull request template and send another pull request. Thx.
Regards
Joe

From: Tzu-Li Tai<EMAIL_ADDRESS>Sent: Monday, August 14, 2017 3:48:53 PM
To: apache/flink
Cc: zhuganghuaonnet; Mention
Subject: Re: [apache/flink] Eventhubs-support read from and write to Azure eventhubs (#4535)
Hi @zhuganghuaonnethttps://github.com/zhuganghuaonnet, thanks a lot for this contribution!
However, taking a step back, is there a JIRA ticket for this contribution?
If not, please make sure that one is opened for it first. Also, for bigger contributions like this, you should make sure that the pull request template is properly filled out to ease the effort for reviewers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/apache/flink/pull/4535#issuecomment-322122182, or mute the threadhttps://github.com/notifications/unsubscribe-auth/APWS44gqqy5hT02uGV1Mlb2gSdRRCBEYks5sX_vkgaJpZM4O18go.

@zhuganghuaonnet we should also mention that new connectors are being contributed through the Flink release of Apache Bahir.

@greghogan I don't think there's actually a consensus that all new connectors should strictly go to Apache Bahir. It still depends case by case, and brought to discussion in the mailing lists if necessary.
We have a growing base of Azure users, and I think it would be nice to have better Azure support. Having a connector for their mainly used event source would be a nice addition, IMO. What do you think?

@tzulitai this does sound like a good candidate for inclusion in Flink. Perhaps a separate flink-connectors repo would work better than Apache Bahir.

@greghogan yes, a separate flink-connectors idea has always been a possible approach in the community, but AFAIK we haven't fully committed to it yet. I can continue to shepherd this contribution to be merged to Flink, and move it to flink-connectors that eventually happens.

@zhuganghuaonnet please let me know when you've opened the JIRA, and rebased this pull request onto the latest master. Also please remember to rename the pull request title (you can take a look at the other pull request titles as reference) and fill out the pull request template as necessary :)
Thanks a lot!

@Tzu-Li<EMAIL_ADDRESS>I have created a jira https://issues.apache.org/jira/browse/FLINK-7474. Pls help to review. Thx. :)

From: Tzu-Li Tai<EMAIL_ADDRESS>Sent: Wednesday, August 16, 2017 12:54:30 PM
To: apache/flink
Cc: zhuganghuaonnet; Mention
Subject: Re: [apache/flink] Eventhubs-support read from and write to Azure eventhubs (#4535)
@zhuganghuaonnethttps://github.com/zhuganghuaonnet please let me know when you've opened the JIRA, and rebased this pull request onto the latest master. Also please remember to rename the pull request title (you can take a look at the other pull request titles as reference) and fill out the pull request template as necessary :)
Thanks a lot!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/apache/flink/pull/4535#issuecomment-322666087, or mute the threadhttps://github.com/notifications/unsubscribe-auth/APWS4zlTWZ9F_4zj2t7gk-Te135tJg7fks5sYnYGgaJpZM4O18go.

@zhuganghuaonnet please also rebase this pull request (there are conflicting classes) and change the pull request title.

@tzulitai Already done. I may need add unit test cases. Should I include unit test cases in this pull request or open another one?

@zhuganghuaonnet it doesn't seem like the PR is correctly rebased. Ideally, you should rebase your development branch to the latest master, and the PR only contains for your changes, and not others.
And yes, please include necessary unit tests / integration tests along with the PR!

Hi @zhuganghuaonnet , you should drop others commits and squash your commits.
:beers:


Support strictBuiltinIteratorReturn

This updates the fp-ts build to be forwards compatible with microsoft/TypeScript#58243, which uses a stricter return type for built-in iterators under the new strictBuiltinIteratorReturn option, which is enabled by default when compiling with strict.

I've filed this as a Draft PR in the event there are any other changes to https://github.com/microsoft/TypeScript/pull/58243 that must be addressed. Once that PR has merged, I will mark this PR as "Ready for review".

Thank you @rbuckton 👍


[Key Vault] Add support for pre-backup and pre-restore operations

Description
Resolves https://github.com/Azure/azure-sdk-for-python/issues/35252. This adds client-facing support for pre-backup and pre-restore methods, for checking whether a full backup or full restore operation can be performed.
As a draft, tests are pending.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-keyvault-administration


Feature/optimize tree rendering

Creating the underlying elements to render flat data structures as a psuedo nested format with extremely high performance.

PR #3 has all of these changes and more, since it's now finished.


Add Basic Gaze Detection script

An attempt at #241 . Welcome feedback on this approach. It is rather minimalist, but adaptive and generalized to any game object rather than just the controllers.

There were the following issues with your Pull Request

Commit: fd0313c4d2cd79b7555bb6a34fdecf772812bd32
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

resolves #241

Superseded with https://github.com/thestonefox/VRTK/pull/657


docs: rename pre-configured infrastructure

For some reason, before expanding the category, I had a feeling that this is example confs, with digitalocean-style 'you do this this this and now you have a wordpress instance on overnode across 3 nodes'.
If somebody like me glossed over, they might've ended up setting the infra up themselves, or not at all.
Something more defining, its similar case as generic lost in translation: 'I can see how it is translated to this now that I know the other side, but this isn't quite right'.

Do you have a proposal for a better name?
Maybe pre-configured stacks? This is actually what it is - published template configurations for various containerised applications


added ens script

A script that can read data feeds via ens addresses

code looks good @PatrickAlphaC . Re: the comment that says
// This script only works with --network mainnet'
can we also use this with a local hardhat node running a forked mainnet?

Ah good call, yes we should be able to do a forked mainnet

looks good, just made a suggestion to the comment. If that's accepted then the PR can be merged

Added your bit in, thanks!


Excluding source files from code coverage

I have problems with excluding source files from code coverage.
Here is my file structure:
App.UnitTests -- runsettings.xml,
App                 -- obj, < project files >
I want to exclude from code coverage the obj folder which is located inside the App folder. So I have navigated in the terminal to App.UnitTests folder and executed this command:
dotnet test --collect:"XPlat Code Coverage" --settings runsettings.xml

Here is my runsettings.xml file.
<?xml version="1.0" encoding="utf-8" ?>
<RunSettings>
  <DataCollectionRunSettings>
    <DataCollectors>
      <DataCollector friendlyName="XPlat code coverage">
        <Configuration>
          <Format>cobertura</Format>
          <ExcludeByFile>../obj/**/*,</ExcludeByFile>
        </Configuration>
      </DataCollector>
    </DataCollectors>
  </DataCollectionRunSettings>
</RunSettings>

Unfortunately this is not working. The code coverage is including files in obj folder. Why is this happening? I will be glad for any help.

Can you try to enable verbose logging as explained here and copy/paste the result? https://github.com/tonerdo/coverlet/blob/master/Documentation/Troubleshooting.md#collectors-integration
Is it the project OSS?A repro could be very useful in case.

@MarcoRossignoli
Unfortunately, it is not open source. Thus, I have a question for you.
I don't understand how relative paths work. If I have a configuration like this:
<ExcludeByFile>../MyFile.cs</ExcludeByFile>

What is the starting path for Coverlet when evaluating the relative path? Does Coverlet start from the same directory where runsettings.xml file is located?
Because in my case something is not working with relative paths.

We need to update the guide sorry, the filter is globbing one so you don't need to add .. relative fragment, take a look at sample on code https://github.com/tonerdo/coverlet/blob/master/test/coverlet.core.tests/Instrumentation/InstrumenterTests.cs#L290-L389
We use Matcher take a look at samples here https://docs.microsoft.com/en-us/dotnet/api/microsoft.extensions.filesystemglobbing.matcher?view=dotnet-plat-ext-3.1&viewFallbackFrom=aspnetcore-2.2

@MarcoRossignoli
Ok. Thanks for the response. Yeah, the docs need to be updated because they are very misleading, i.e. here:
https://github.com/tonerdo/coverlet/blob/master/Documentation/VSTestIntegration.md
https://github.com/tonerdo/coverlet/blob/master/Documentation/MSBuildIntegration.md#excluding-from-coverage
And the docs even explicitly says:

Use absolute or relative paths (relative to the project directory)

So since the relative path is not working, right now I am using this filter:
**/obj/**/*

This filter excludes all files in the obj folder from code coverage.

I'll keep this open until doc fix.


Fix the kubectl subcommands links.

Fix the kubectl subcommands links.
Issue: https://github.com/kubernetes/website/issues/10080

cc @zhangxiaoyu-zidif @xiangpengzhao

IMO, this is auto-generated file
/cc @zacharysarah @steveperry-53

@warmchang This PR looks good to me because the current reference doc for kubectl is a mix of materials from different sources. Please disassociate this PR from the issue #10520 because this PR is not fixing that issue.

FYI: https://github.com/kubernetes/website/issues/10080

/lgtm
/approve

@tengqm thank your for reminding, I'll fix the https://github.com/kubernetes/website/issues/10520


Update index.md with the language Beef

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[x] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[x] Content (and my changes are in index.md)

Beef is a language in active development on Github with 1.8k stars at the time this PR was created. There is lots of work being done with this language, primarily in game development. This project should get more visibility, as it fills a role for developers who prefer the syntax of language slike C# and Java, but requires compiled, high-performance applications with manual memory management.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x]  I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x]  I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[x]  Suggesting edits to an existing topic or collection
[ ]  Curating a new topic or collection
[ ]  Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ]  Image (and my file is *.png, square, dimensions 288x288)
[x]  Content (and my changes are in index.md)

Beef is a language in active development on Github with 1.8k stars at the time this PR was created. There is lots of work being done with this language, primarily in game development. This project should get more visibility, as it fills a role for developers who prefer the syntax of language slike C# and Java, but requires compiled, high-performance applications with manual memory management.
Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

yhh

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x]  I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x]  I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[x]  Suggesting edits to an existing topic or collection
[ ]  Curating a new topic or collection
[ ]  Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ]  Image (and my file is *.png, square, dimensions 288x288)
[x]  Content (and my changes are in index.md)

Beef is a language in active development on Github with 1.8k stars at the time this PR was created. There is lots of work being done with this language, primarily in game development. This project should get more visibility, as it fills a role for developers who prefer the syntax of language slike C# and Java, but requires compiled, high-performance applications with manual memory management.
Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


Altair doesn't handle column names with dashes in the name

Altair has no output if the column name has a dash (-) in the name. Try the following:
This will work
xd = pd.DataFrame({'Y': {0: 17.81, 1: 22.35, 2: 25.13, 3: 16.76},
'X': {0: '0-12', 1: '12-36', 2: '36-60', 3: '>60'}})
Chart(xd).mark_bar().encode(
x='X:N',
y='Y:Q',
)
The following won't
xd1 = pd.DataFrame({'Y-10': {0: 17.81, 1: 22.35, 2: 25.13, 3: 16.76},
'X': {0: '0-12', 1: '12-36', 2: '36-60', 3: '>60'}})
Chart(xd1).mark_bar().encode(
x='X:N',
y='Y-10:Q',
)

Thanks for the comment! If you check chart.to_dict(), you'll see that Altair is producing the correct JSON output; the problem is that the vega-lite renderer isn't handling this. I've reported this issue at https://github.com/vega/vega-lite/issues/778


[WIP] Add MvcEndpointDataSource and functional tests

A very WIP MvcEndpointDataSource to start getting feedback.
Many tests fail because

no action selection (use to match the HTTP verb)
no IUrlHelper support
and logging issue https://github.com/aspnet/Routing/pull/540

For HTTP verb matching - it was in the implementation in the diaggregator website but not in what you added to routing. Is that because you want to implement it differently?
Asking because a lot of tests fail because action constraint logic is being skipped (where verb was previously matched) and a lot of tests are failing because of it.

Is there a recommended workflow for when making updates to routing (or other packages) and you want those changes immediately in your local MVC?


fix: check qos 0 counter before looping

Issue #, if available:
When GG is offline, or when spool storage is full, we check through all messages to see if i we can remove any QOS 0 message to accommodate the incoming message. This loop reads through all messages in spooler and is heavy as it reads through database, in case its a disk spooler. This was discovered as we saw high CPU usage in the Disk spooler performance tests which ran spooler tests with QOS 1 messages and at 100 TPS in an unstable network (online <-> offline every 5 mins)
Description of changes:

Initial change: Have a counter to track number of QOS 0 messages in spooler, while removing QOS 0 message to make space, check the counter to see if there is any QOS 0 message.

We can try and optimize this more by maintaining a record of message IDs that map to messages that are QOS0, so that we only loop through those messages.
Why is this change necessary:
How was this change tested:

[ ] Updated or added new unit tests.
[ ] Updated or added new integration tests.
[ ] Updated or added new end-to-end tests.
[ ] If my code makes a remote network call, it was tested with a proxy.

Any additional information or context required to review the change:
Documentation Checklist:

[ ] Updated the README if applicable.

Compatibility Checklist:

[ ] I confirm that the change is backwards compatible.
[ ] Any modification or deletion of public interfaces does not impact other plugin components.
[ ] For external library version updates, I have reviewed its change logs and Nucleus does not consume
any deprecated method or type.

Refer to Compatibility Guidelines for more information.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks, that's definitely a better way.

Rather than counting, how about a simpler (and less error prone) boolean flag. ex. set to true when any message is added to the queue. If the flag is true, then we will attempt to remove qos 0 messages from the queue when the removal method is called. After calling the removal method, set it back to true. Basically just a flag which means "do we need to check for qos 0 because the queue has been added to"

Thanks, that's definitely a better way to do it.

Lgtm, unit test?

Existing unit tests should verify that the overall behavior is not changed, testing just this flag part is not possible unless we change a couple of method return types. Went through the logic once again to confirm that when we do set the flag to false, we do it for a valid reason (check removed all qos0 messages or there weren't any in the first place)

Lgtm, unit test?

Existing unit tests should verify that the overall behavior is not changed, testing just this flag part is not possible unless we change a couple of method return types. Went through the logic once again to confirm that when we do set the flag to false, we do it for a valid reason:
1. Check removed all qos0 messages OR

2. There weren't any in the first place OR

3. We removed some qos0 messages and stopped midway because size issues were solved, in that case we set it back to true immediately after adding the original message (for which we did this removal process)

Can force disconnect then verify number of times getMessageById is called?


Input textfield - Improve styling with no required attribute?

Thank you for your great work.
If I omit the the required attribute then the style of the textfield is permanently valid. The floating effect also then doesn't work. Would it be possible for you to add styling so that textfields can work without the required attribute?
I've seen other libraries use the class="validate" for when a user does want validation styling.
If I change line 23 of form.scss from   &:focus, &:valid { to &:focus { then I no longer have to include the required attribute, although the placeholder label does still float back down so I guess more would be required.

Hey @adam-knights ! I spent a lot of time looking into this when I first made Surface. It actually cannot be done with pure CSS alone whilst maintaining the same styling as the inputs with required. It would need to be done with JS. Surface was kind of just an experiment to see how much could be done with pure CSS and no JS, so I chose not to include any JS solutions.
(Just to clarify, just because it was a semi-experiment doesn't mean you cannot/shouldn't use it in production, I've used it plenty).
If you wanted to add that in to your personal project, simply make a class with all the styling for the required input fields and call it validate like you said. Then in JS target all the validate classes and give them more styling based on whether they're onfocus or not.
Thanks :)


Added settings for vscode

On mac visual studio is very crappy so I am adding this so it would make life easier.

Nice. LGTM!
I am using Rider on Mac, so I never came across the issue, but much appreciated!