Failed on Fresh Install (Affecting GitHub Actions Run)

First discovered when setting up emscripten from scratch using choco.
Error (likely false alarm)
The failure was due to the git LFS dependency installation. Due to this failure (despite successful installation), choco reported the execution as error, thus failing the entire CI run.

Request
Appreciate remedy so that external CI tools can be uniformly setup with choco.
Reference

https://github.com/corygalyna/AutomataCI/actions/runs/6387879358/job/17337116795
https://community.chocolatey.org/packages/emscripten#discussion

Could you post the full log in the text format?

It's available in: https://github.com/corygalyna/AutomataCI/actions/runs/6387879358/job/17337116795#step:6:1329 under ENV native CI job (caught by GitHub Actions).
Copying it to here:
hocolatey v2.2.2
Installing the following packages:
emscripten
By installing, you accept licenses for the packages.

Progress: Downloading git.install 2.42.0... %
Progress: Downloading git.install 2.42.0... %
Progress: Downloading git.install 2.42.0... %
[ ... removed due to repetitive progress status ... ]
Progress: Downloading git.install 2.42.0... 100%
Progress: Downloading git.install 2.42.0... 100%
Progress: Downloading git.install 2.42.0... 100%

git.install v2.42.0 [Approved]
git.install package files install completed. Performing other installation steps.
Using Git LFS
Installing 64-bit git.install...
ERROR: Running ["C:\ProgramData\chocolatey\lib\git.install\tools\Git-2.42.0-64-bit.exe" /VERYSILENT /SUPPRESSMSGBOXES /NORESTART /NOCANCEL /SP- /LOG /COMPONENTS="icons,assoc,assoc_sh,ext,ext\shellhere,ext\guihere,gitlfs,icons\quicklaunch" ] was not successful. Exit code was '1'. See log for possible error messages.
The install of git.install was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\git.install\tools\chocolateyInstall.ps1'.
 See log for details.

Progress: Downloading git 2.42.0... 92%
Progress: Downloading git 2.42.0... 100%

git v2.42.0 [Approved]
git package files install completed. Performing other installation steps.
 The install of git was successful.
  Software installed to 'C:\ProgramData\chocolatey\lib\git'

Progress: Downloading vcredist2015 14.0.24215.20170201... 76%
Progress: Downloading vcredist2015 14.0.24215.20170201... 100%

vcredist2015 v14.0.24215.20170201 [Approved]
vcredist2015 package files install completed. Performing other installation steps.
 The install of vcredist2015 was successful.
  Software installed to 'C:\ProgramData\chocolatey\lib\vcredist2015'

Progress: Downloading python312 3.12.0... %
Progress: Downloading python312 3.12.0... %
Progress: Downloading python312 3.12.0... %
[ ... removed due to repetitive progress status ... ]
Progress: Downloading python312 3.12.0... 100%
Progress: Downloading python312 3.12.0... 100%
Progress: Downloading python312 3.12.0... 100%

python312 v3.12.0
python312 package files install completed. Performing other installation steps.
Installing 64-bit python312...
python312 has been installed.
Added C:\ProgramData\chocolatey\bin\python3.12.exe shim pointed to 'c:\python312\python.exe'.
Python installed to: 'C:\Python312'
Restricting write permissions to Administrators
  python312 can be automatically uninstalled.
Environment Vars (like PATH) have changed. Close/reopen your shell to
 see the changes (or in powershell/cmd.exe just type `refreshenv`).
 The install of python312 was successful.
  Software installed as 'exe', install location is likely default.

Progress: Downloading python3 3.12.0... 100%

python3 v3.12.0 [Approved]
python3 package files install completed. Performing other installation steps.
 The install of python3 was successful.
  Software installed to 'C:\ProgramData\chocolatey\lib\python3'

Progress: Downloading python 3.12.0... 100%

python v3.12.0 [Approved]
python package files install completed. Performing other installation steps.
 The install of python was successful.
  Software installed to 'C:\ProgramData\chocolatey\lib\python'

Progress: Downloading emscripten 3.1.40... 56%
Progress: Downloading emscripten 3.1.40... 100%

emscripten v3.1.40 [Approved]
emscripten package files install completed. Performing other installation steps.
WARNING: No registry key found based on  'emscripten'
Cloning https://github.com/emscripten-core/emsdk.git
The current working directory is changed to C:\Users\runneradmin\AppData\Local
Cloning into 'emsdk'...
The current working directory is changed to C:\Users\runneradmin\AppData\Local\emsdk
Already up to date.
`update-tags` is not longer needed.  To install the latest tot release just run `install tot`
Installing emsdk 3.1.40
Resolving SDK version '3.1.40' to 'sdk-releases-c3122846bb040798aab975f61008c37eb19476de-64bit'
Installing SDK 'sdk-releases-c3122846bb040798aab975f61008c37eb19476de-64bit'..
Installing tool 'node-16.20.0-64bit'..
Downloading: C:/Users/runneradmin/AppData/Local/emsdk/downloads/node-v16.20.0-win-x64.zip from https://storage.googleapis.com/webassembly/emscripten-releases-builds/deps/node-v16.20.0-win-x64.zip, 28623474 Bytes
 [----------------------------------------------------------------------------]
C:\Users\runneradmin\AppData\Local\emsdk/emsdk.py:329: DeprecationWarning: onerror argument is deprecated, use onexc instead
  shutil.rmtree(d, onerror=remove_readonly_and_try_again)
Unpacking 'C:/Users/runneradmin/AppData/Local/emsdk/downloads/node-v16.20.0-win-x64.zip' to 'C:/Users/runneradmin/AppData/Local/emsdk/node/16.20.0_64bit'
Done installing tool 'node-16.20.0-64bit'.
Installing tool 'python-3.9.2-nuget-64bit'..
Downloading: C:/Users/runneradmin/AppData/Local/emsdk/downloads/python-3.9.2-4-amd64+pywin32.zip from https://storage.googleapis.com/webassembly/emscripten-releases-builds/deps/python-3.9.2-4-amd64+pywin32.zip, 14413267 Bytes
 [----------------------------------------------------------------------------]
Unpacking 'C:/Users/runneradmin/AppData/Local/emsdk/downloads/python-3.9.2-4-amd64+pywin32.zip' to 'C:/Users/runneradmin/AppData/Local/emsdk/python/3.9.2-nuget_64bit'
Done installing tool 'python-3.9.2-nuget-64bit'.
Installing tool 'java-8.152-64bit'..
Downloading: C:/Users/runneradmin/AppData/Local/emsdk/downloads/portable_jre_8_update_152_64bit.zip from https://storage.googleapis.com/webassembly/emscripten-releases-builds/deps/portable_jre_8_update_152_64bit.zip, 69241499 Bytes
 [----------------------------------------------------------------------------]
Unpacking 'C:/Users/runneradmin/AppData/Local/emsdk/downloads/portable_jre_8_update_152_64bit.zip' to 'C:/Users/runneradmin/AppData/Local/emsdk/java/8.152_64bit'
Done installing tool 'java-8.152-64bit'.
Installing tool 'releases-c3122846bb040798aab975f61008c37eb19476de-64bit'..
Downloading: C:/Users/runneradmin/AppData/Local/emsdk/downloads/c3122846bb040798aab975f61008c37eb19476de-wasm-binaries.zip from https://storage.googleapis.com/webassembly/emscripten-releases-builds/win/c3122846bb040798aab975f61008c37eb19476de/wasm-binaries.zip, 425017845 Bytes
 [----------------------------------------------------------------------------]
Unpacking 'C:/Users/runneradmin/AppData/Local/emsdk/downloads/c3122846bb040798aab975f61008c37eb19476de-wasm-binaries.zip' to 'C:/Users/runneradmin/AppData/Local/emsdk/upstream'
Done installing tool 'releases-c3122846bb040798aab975f61008c37eb19476de-64bit'.
Done installing SDK 'sdk-releases-c3122846bb040798aab975f61008c37eb19476de-64bit'.
Activating emsdk 3.1.40
Adding directories to PATH: PATH += C:\Users\runneradmin\AppData\Local\emsdk PATH += C:\Users\runneradmin\AppData\Local\emsdk\upstream\emscripten System.Management.Automation.RemoteException Setting environment variables: PATH = C:\Users\runneradmin\AppData\Local\emsdk;C:\Users\runneradmin\AppData\Local\emsdk\upstream\emscripten;%USERPROFILE%\.dotnet\tools;%USERPROFILE%\.cargo\bin;%USERPROFILE%\AppData\Local\Microsoft\WindowsApps; EMSDK = C:/Users/runneradmin/AppData/Local/emsdk EMSDK_NODE = C:\Users\runneradmin\AppData\Local\emsdk\node\16.20.0_64bit\bin\node.exe EMSDK_PYTHON = C:\Users\runneradmin\AppData\Local\emsdk\python\3.9.2-nuget_64bit\python.exe JAVA_HOME = C:\Users\runneradmin\AppData\Local\emsdk\java\8.152_64bit Failed to read environment variable EMSDK: 'EMSDK' Failed to read environment variable EMSDK_NODE: 'EMSDK_NODE' Failed to read environment variable EMSDK_PYTHON: 'EMSDK_PYTHON'
Ignore the above failure messages about environment variables or import Python Windows extensions.
The installation added the emsdk's bundled Python, Node, and Java to the PATH.
and it set JAVA_HOME to the path of the bundled Java.
If this conflicts with your other development environments, remove these from PATH.
The package is successfully installed in C:\Users\runneradmin\AppData\Local\emsdk
The current working directory is recovered to D:\a\AutomataCI\AutomataCI
Only an exit code of non-zero will fail the package by default. Set
 `--failonstderr` if you want error messages to also fail a script. See
 `choco -h` for details.
Environment Vars (like PATH) have changed. Close/reopen your shell to
 see the changes (or in powershell/cmd.exe just type `refreshenv`).
 The install of emscripten was successful.
  Software install location not explicitly set, it could be in package or
  default install location of installer.

Chocolatey installed 6/7 packages. 1 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Installed:
 - emscripten v3.1.40
 - git v2.42.0
 - python v3.12.0
 - python3 v3.12.0
 - python312 v3.12.0
 - vcredist2015 v14.0.24215.20170201

Failures
 - git.install (exited 1) - Error while running 'C:\ProgramData\chocolatey\lib\git.install\tools\chocolateyInstall.ps1'.
 See log for details.
Error: Process completed with exit code 1.


Generate runs for CIFAR 100

We need some runs with the CIFAR 100 dataset.

I'll adapt the our cifar10 code.

Out of scope for the paper. Hence, moved to icebox.

the code was already updated for cifar100 but it we don't need this for the paper and the project


Encounter Participant Cardinality

For UDS Plus Encounter, the Participant element has a cardinality of 0..1, which means there can be a max of one participant on an encounter.
In US Core Encounter, the Participant element has a cardinality of 0..*, allowing for multiple participants on an encounter.
I do not see any explanation for this change in the UDS+ IG. Is this an intentional change?
In Epic, you can definitely document more than one provider on an encounter.  We request that you update the cardinality of the Participant element to align with US Core and allow more than one participant per encounter resource.

We will make a change to allow for multiple participants (0..*) , so in the upcoming version we will change it.


feat: support overriding TopBar

Adds support for overriding TopBar component. In future we can add support for more component overrides.
import tutorialkit from '@tutorialkit/astro';
import { defineConfig } from 'astro/config';

export default defineConfig({
  integrations: [
    tutorialkit({
      components: {
        TopBar: './CustomTopBar.astro',
      },
    }),
  ],
});

Users may still want to utilize TutorialKit's default <Logo> and <ThemeSwitch>. These are accessible through slots:
// ./CustomTopBar.astro
<nav style="display: flex; align-items: center; gap: 2rem; padding: 1rem;">
  Some content here
  <slot name="logo" />
  and here
  <slot name="theme-switch" />
  here too
  <slot name="login-button" />
  and here!
</nav>

I spent some time trying to set up tests for this kind of features but it seemed a bit bigger task than I expected. I don't think there are any existing tests where this feature could be easily tested..? Let's build good testing setup in separate PR later.
Next I'll look into adding overrides for other components so that rest of the feature requests can be filled. For example the bottom-left corners "Edit this page" link.

Let's build good testing setup in separate PR later.

I totally agree. In particular I think this would be easy to test if we had playwright or puppeteer configured.

I totally agree. In particular I think this would be easy to test if we had playwright or puppeteer configured.

Yep! I had something like this in mind:

import { fileURLToPath } from "node:url";
import { beforeAll, expect, test } from "vitest";
import { type Browser, chromium } from "playwright";
import { dev } from "astro";

let browser: Browser;

beforeAll(async () => {
  // Start server
  const server = await dev({
    configFile: false,
    root: fileURLToPath(new URL("./fixtures", import.meta.url)),
    integrations: [
      tutorialkit({
        components: {
          TopBar: "./src/components/CustomTopBar.astro",
        },
      }),
    ],
  });

  // Open browser
  browser = await chromium.launch({ headless: true });

  // Close server
  return async function cleanup() {
    await server.stop();
  };
});

test("custom top bar is visible", async () => {
  const page = await browser.newPage();
  await page.goto("http://localhost:4321/");

  await page.getByRole("heading", {
    level: 1,
    name: "Custom top bar",
  });
});

But TutorialKit had some issues when used from Astro's Node API and changing root.

Tests on Windows CI were failing but it's fixed now. ✅


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 2.4.11 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[freetype branch] Color Font are Displayed in MonoChrome

hello @pshurgal,
sorry for disturb you, you have created a pull request regarding imgui freetype branch for color font.
for have color font support in ImGuiFontStudio i need first to have the ability to laod and display color font.
but he not work. all glyph are displayed in monochrome.
can you check this branch if you have time, maybe im wrong somewhere?
https://github.com/aiekick/ImGuiFontStudio/tree/freetype
Thanks

Hello, I have just seen you message. Ok, I'l have a look. But first are you sure you are using right font file? As i said in original PR it seems that freetype does not support colors in SVGinOT fonts yet. So you have to check are there CPAL and COLR tables listed inside your font file. You can just open the font file in HEX editor and look which tables are listed in the beginning of the file.

Yep i have checked the font, and there is the tables CPAL and COLR.
i have put a button for switch freetype/imgui rasterizer and activate or not the color.
when i activate color, the smiley was filled in black, and its logic to be filled, sicne this color is layered, but dont understand why no color appearing.
im using the windows font you mention in your PR : C:\Windows\Fonts\seguiemj.ttf

Ok, I'll have a look

see, what i was talking :

you have a windows build here :
https://github.com/aiekick/ImGuiFontStudio/actions/runs/260916146

i though i was just the widget display who was not ok, but when you check the texture, she appears also in two colors :

i found the bug haha. it was not imgui_freetpe :) but my soft
i defined the flag but not pass it to buildatlas.
in fact its weird to have extra_flags in buildatlas func, since flags can be passed directly to  font_config->RasterizerFlags

@aiekick, hello, I have just added a fix into colored glyphs branch which fixes colors and antialiasing https://github.com/pshurgal/imgui/commit/1cc96e735da7367ed8979a288c72123e1ef90017


Add BigQuery dialect

We need the BigQuery dialect to support special syntax such as trailing commas, UNION DISTINCT, identifier detection, etc.
To simplify each PR, this PR includes only dialect support, and we plan to implement other syntax in later PRs.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

36 of 36 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 90.468%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
8286

Relevant Lines:
9159

💛  - Coveralls


Atom 6815 handle for timeouts

Ticket
https://jira.bqsoft.com/browse/ATOM-6815
What this does
Handles for timeout
Why we did this
Today we just blow up instead of handling timeouts like errors
https://developers.braintreepayments.com/reference/general/exceptions/ruby#timeout-exception

@pedromartinez Should be ready for another review.


Update .NET logo

the .NET logo on the Ecosystem page is linked directly , but it should follow the format of all other logos  (stored in this folder.
You can see the directly linked image is also a 404

I want to contribute.

I want to contribute.

@AdityaAB4 Thank you for the interest in contributing. I assigned the issue to you. Please dln't hesitate if you need any helo or have any questions.

Thank you, Sir, for assigning me this issue.
I have cloned the repo and ran the npm install command but it is
throwing an error at node-sass installation. Could you please help me
with this?
Screenshots of the errors are attached.
On Tue, 29 Nov 2022 at 15:42, Aaron Junker @.***> wrote:

I want to contribute.
@AdityaAB4 https://github.com/AdityaAB4 Thank you for the interest in
contributing. I assigned the issue to you. Please dln't hesitate if you
need any help or have any questions.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/opensource.microsoft.com/issues/251#issuecomment-1330392218,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCGU4WSIH7OPGIIQ4WTPPTWKXJJVANCNFSM6AAAAAASN5DSS4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you, Sir, for assigning me this issue.
I have cloned the repo and ran the npm install command but it is
throwing an error at node-sass installation. Could you please help me
with this?
Screenshots of the errors are attached.

Hi @AdityaAB4 Files attached to mail are bot displayee in GitHub. Could you please attach the file over the website?
And can you also try to build the docker image?

Getting this error on running npm install

I tried it on codespace but it is giving me the same error.

@AdityaAB4 Building the docker image in the codespace should work.

Thanks, I'll check it out.
On Tue, 29 Nov 2022 at 21:47, Aaron Junker @.***> wrote:

@AdityaAB4 https://github.com/AdityaAB4 Building the docker image in
the codespace should work.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/opensource.microsoft.com/issues/251#issuecomment-1330898811,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCGU4TIVO2IONY46ZHABTDWKYUAJANCNFSM6AAAAAASN5DSS4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Gulp error:

Gulp error:

cc: @emmairwin

The error is solved

For running the project I have added webrick by using command bundle add webrick
That changed the following files:
Gemfile

Gemfile.lock

For solving the error I changed the externalLogo field's value to "https://old.dotnetfoundation.org/img/dotNetFoundationHorizontal.svg"

Sure, I will do that.
On Wed, 30 Nov 2022 at 22:01, Aaron Junker @.***> wrote:

For running the project I have added webrick by using command bundle add
webrick
That changed the following files: Gemfile [image: image]
https://user-images.githubusercontent.com/63203954/204773071-843326e3-011e-4638-b124-15c94e9a4bc3.png
Gemfile.lock [image: image]
https://user-images.githubusercontent.com/63203954/204773282-9af922ff-c792-48c4-b321-7cc5f36f440e.png
For solving the error I changed the externalLogo field's value to "
https://old.dotnetfoundation.org/img/dotNetFoundationHorizontal.svg"
[image: image]
https://user-images.githubusercontent.com/63203954/204773893-fedd45fc-b5fe-4810-8b92-8a7bae73ab40.png
Thank you. But unfortunately the issue asked to store the image in this
folder
https://github.com/microsoft/opensource.microsoft.com/tree/main/assets/images/projects
and the reference it. Could you please correct that?
And when you're finished you can open a pull request.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/opensource.microsoft.com/issues/251#issuecomment-1332433189,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCGU4T6QUOB2ICENTAX3GTWK56PZANCNFSM6AAAAAASN5DSS4
.
You are receiving this because you were mentioned.Message ID:
@.***>

I have created a branch name dotNetlogoissue but after adding the command git push origin dotNelogoissue I got the following error:

I have created a branch name dotNetlogoissue but after adding the command git push origin dotNelogoissue I got the following error:

Hi @AdityaAB4 You have to fork the repository first.
I would recommend you to read this

Ok, I'll check it out.
On Thu, 1 Dec 2022, 1:33 am Aaron Junker, @.***> wrote:

I have created a branch name dotNetlogoissue but after adding the command git
push origin dotNelogoissue I got the following error:
[image: image]
https://user-images.githubusercontent.com/63203954/204874568-c6be449a-1b38-4072-b38b-f3bf6ca8b0f4.png
Hi @AdityaAB4 https://github.com/AdityaAB4 You have to fork the
repository first.
I would recommend you to read this
https://opensource.guide/how-to-contribute/#opening-a-pull-request
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/opensource.microsoft.com/issues/251#issuecomment-1332675959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCGU4VECIHIW5TZ4GFU4PTWK6XJJANCNFSM6AAAAAASN5DSS4
.
You are receiving this because you were mentioned.Message ID:
@.***>

I have forked the repo and did the changes as required but still getting the same issue; while pushing the code.

I have forked the repo and did the changes as required but still getting the same issue; while pushing the code.

Hmm. Weird.
Can you try having a look at this: https://docs.github.com/en/get-started/getting-started-with-git/caching-your-github-credentials-in-git

I have opened a PR.

[image: image.png]
Could you please take a look at this Aaron?
On Wed, 21 Dec 2022 at 07:32, github-actions[bot] @.***>
wrote:

Closed #251
https://github.com/microsoft/opensource.microsoft.com/issues/251 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/opensource.microsoft.com/issues/251#event-8083119383,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCGU4X25AZM5ORGJAQCX23WOJQLDANCNFSM6AAAAAASN5DSS4
.
You are receiving this because you were mentioned.Message ID:
<microsoft/opensource.microsoft.com/issue/251/issue_event/8083119383@
github.com>

[image: image.png]
Could you please take a look at this Aaron?
On Wed, 21 Dec 2022 at 07:32, github-actions[bot] @.***>
wrote:

Closed #251
https://github.com/microsoft/opensource.microsoft.com/issues/251 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/opensource.microsoft.com/issues/251#event-8083119383,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCGU4X25AZM5ORGJAQCX23WOJQLDANCNFSM6AAAAAASN5DSS4
.
You are receiving this because you were mentioned.Message ID:
<microsoft/opensource.microsoft.com/issue/251/issue_event/8083119383@
github.com>

@AdityaAB4  I can't see the image you attached, you have to upload it over the GitHub interface. Mail attachemebts don't work

Thanks again for fixing this @AdityaAB4. The change is live in production now.

Glad I could help.

Message ID: <microsoft/opensource.microsoft.com/issues/251/1374531372@
github.com>

the .NET logo on the Ecosystem page is linked directly , but it should follow the format of all other logos  (stored in this folder.
You can see the directly linked image is also a 404

can see the directly linked image is also a 404

Tree/main/assets/images/projects).


Integration Specs

In this PR
n + 1

Make sure that the N+1 problem is solved when fetching all posts and their comments for a user.
Integration specs

Use Capybara to write integration tests for each view in your project.

User index page:

I can see the username of all other users.
I can see the profile picture for each user.
I can see the number of posts each user has written.
When I click on a user, I am redirected to that user's show page.

user show page:

I can see the user's profile picture.
I can see the user's username.
I can see the number of posts the user has written.
I can see the user's bio.

User post index page:

I can see the user's profile picture.
I can see the user's username.
I can see the number of posts the user has written.
I can see a post's title.
I can see some of the post's body.
I can see the first comments on a post.

Post show page:

I can see the post's title.
I can see who wrote the post.
I can see how many comments it has.
I can see how many likes it has.
I can see the post body.
I can see the username of each commenter.
I can see the comment each commentor left.
I can see how many comments a post has.
I can see how many likes a post has.
I can see a section for pagination if there are more posts than fit on the view.
When I click on a post, it redirects me to that post's show page.
I can see the user's first 3 posts.
I can see a button that lets me view all of a user's posts.
When I click a user's post, it redirects me to that post's show page.
When I click to see all posts, it redirects me to the user's post's index page.

Hi @white3d & @evans22j,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that doesn't have a first review present.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi @white3d & @evans22j,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that doesn't have a first review present.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.

Hi @arslanbisharat,
Kindly elaborate on this statement, "You have submitted a project, that doesn't have a first review present." I don't understand what you mean? Thank you.


feat(dynamic-forms): allow elements to be disabled via configuration

Description

adding a disabled property to disable properties on the fly without the
need of accessing the underlying form
What's included?

new disabled property in the configuration
updated Demo and README

Test Steps

[ ] npm run serve
[ ] Go to http://localhost:4200/#/components/dynamic-forms
[ ] Play with form build to see how the disable state affects different components

General Tests for Every PR

[ ] npm run serve:prod still works.
[ ] npm run tslint passes.
[ ] npm run stylelint passes.
[ ] npm test passes and code coverage is not lower.
[ ] npm run build:lib still works.

closes https://github.com/Teradata/covalent/issues/1126

I don't like the how the disabled slider looks/works.  It's not grayed out, but the dot grays out when you click on it.  Title should be grayed out too.
Don't like the slider toggle.  Title should be grayed out and larger font to match the others.
File upload button is still active even though element is disabled.


Whitepaper "Note on infectious period" might be missing a Bayes' rule conversion

The "Note on infectious period: Contact's symptoms" section of the whitepaper states:

...only about 9% of transmissions from people who eventually show symptoms occur more than 3 days before those symptoms appear.... Additionally, only about 6% of transmissions come from people who won't ever show symptoms.
These two facts combined imply that, if you interacted with someone more than 3 days ago, and that person has not yet shown symptoms, then you should be about 7 times more confident than you were before that they did not transmit COVID to you: 0.94*0.09 + 0.06 = 0.14 ≈ 1/7.

I think this is missing a Bayes’ rule conversion:  if "6% of transmissions come from people who won't ever show symptoms", then 0.06 is the proportion of transmissions that come from asymptomatic patients, not the the proportion of asymptomatic patients who transmit the disease. In other words, p(asymptomatic COVID|transmission happened) = 0.06, and we want to find p(transmission happened|asymptomatic COVID).
The quoted paragraph above also states that 9% of transmissions from eventually-symptomatic patients occur when those patients are still pre-symptomatic. But, looking at the source paper, I think 0.09 is actually is the expected number of transmissions from pre-symptomatic individuals. The total R0 is 2.0, of which 0.9 is accounted for by pre-symptomatic patients:

(I think this would that 0.9/2.0 = 45% of transmissions from eventually-symptomatic patients occur when the patients are still pre-symptomatic. But, again, we want to figure out p(transmission|pre-symptomatic COVID), not p(pre-symptomatic COVID|transmission).

This needs an overhaul anyway to incorporate Byambasuren et al
(via @beshaya: https://jammi.utpjournals.press/doi/10.3138/jammi-2020-0030 is a December meta-study of asymptomatic spread. They found that 17% of cases are asymptomatic and asymptomatic individuals were 42% as likely to transmit.)

cc @oremanj (I know you don't have time right now, but I think there's sufficient information now that these mitigations need adjustment!)


fix(docs): remove link-checker from CI

It had too many false positives, so can't be a blocking step in CI. However, it was still super useful so we'll just need to manually run it every now and then.
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Merging through smoke test failure


Visible scrolling when trying to start calendar at a certain date #263

(Required) Version Number:
JTAppleCalendar 8.0.2
Description
Absolutely love JTAppleCalendar. Using it in production apps for about two years now. But, I recently ran into an issue. I am having the same issue as Issue #263.
I want the calendar to be on a selected date when it first appears, so I do calendarView.scrollToDate(selectedDate) at viewDidLoad().
When I go to the calendar's view controller, I can see the starting month flash very quickly, and then the selected month shows up (even if I use animateScroll = false). This is very quick in the simulator, but very apparent on my iPhone 6 (running iOS 12.4.9). For example, you can clearly see September 2020 show up before the calendar changes over to show January 2021.
Steps To Reproduce
Here is my code:
let startDate = Date(timeIntervalSince1970: FamilyData.setupCompletedDate)
let endDate = Date()

override func viewDidLoad() {
    super.viewDidLoad()

    setupUI()
}

// ---------------
// Collection View
// ---------------

func configureCalendar(_ calendar: JTACMonthView) -> ConfigurationParameters {

    // These are the dates the user can select from when choosing due dates
    // end date will always be at the end of their budget period
    // start date depends on which calendar they're looking at.
    // If they're looking at the first payment calendar, then the first selectable date should be the first day of their budget
    // BUT... if they're looking at the last payment calendar, then the first selectable date should be AFTER the first due date

    return ConfigurationParameters(startDate: startDate,
                                   endDate: endDate,
                                   calendar: FamilyData.calendar(),
                                   generateInDates: .forAllMonths,
                                   generateOutDates: .tillEndOfRow)
}

func configureCell(view: JTACDayCell?, cellState: CellState) {

    guard let cell = view as? DateCell else { return }

    cell.dateLabel.text = cellState.text

    handleCellTextColor(cell: cell,
                        cellState: cellState)

    handleCellSelected(cell: cell,
                       cellState: cellState)
}

func handleCellSelected(cell: DateCell, cellState: CellState) {

    // round corner of BG, whether it's showing or not
    cell.selectedView.layer.cornerRadius = cell.selectedView.layer.bounds.height / 2

    if cellState.date < startDate || cellState.date > endDate {
        cell.dateLabel.textColor = .mpGray40
        cell.selectedView.isHidden = true
        cell.isUserInteractionEnabled = false

    } else if cellState.isSelected {
        cell.dateLabel.textColor = .white
        cell.selectedView.backgroundColor = .mpBlue
        cell.selectedView.isHidden = false
        cell.isUserInteractionEnabled = true

    } else {
        cell.dateLabel.textColor = .black
        cell.selectedView.isHidden = true
        cell.isUserInteractionEnabled = true
    }

    if FamilyData.calendar().isDate(cellState.date, inSameDayAs: Date()) {
        if !cellState.isSelected {
            cell.dateLabel.textColor = .mpBlue
            cell.selectedView.isHidden = true
        }
    }
}

func handleCellTextColor(cell: DateCell, cellState: CellState) {

    if cellState.dateBelongsTo == .thisMonth {
        cell.isHidden = false

    } else {
        cell.isHidden = true
    }
}

func calendar(_ calendar: JTACMonthView, cellForItemAt date: Date, cellState: CellState, indexPath: IndexPath) -> JTACDayCell {

    let cell = calendar.dequeueReusableJTAppleCell(withReuseIdentifier: "dateCell", for: indexPath) as! DateCell

    self.calendar(calendar,
                  willDisplay: cell,
                  forItemAt: date,
                  cellState: cellState,
                  indexPath: indexPath)

    return cell
}

func calendar(_ calendar: JTACMonthView, willDisplay cell: JTACDayCell, forItemAt date: Date, cellState: CellState, indexPath: IndexPath) {

    configureCell(view: cell,
                  cellState: cellState)
}

func calendar(_ calendar: JTACMonthView, headerViewForDateRange range: (start: Date, end: Date), at indexPath: IndexPath) -> JTACMonthReusableView {

    let header = calendar.dequeueReusableJTAppleSupplementaryView(withReuseIdentifier: "DateHeader", for: indexPath) as! DateHeader
    header.monthTitle.text = DateFormatter().mpMonth(from: range.start)
    return header
}

func calendarSizeForMonths(_ calendar: JTACMonthView?) -> MonthSize? {
    return MonthSize(defaultSize: 50)
}

func calendar(_ calendar: JTACMonthView, didSelectDate date: Date, cell: JTACDayCell?, cellState: CellState, indexPath: IndexPath) {

    configureCell(view: cell,
                  cellState: cellState)

    // When a user selects a day, they get the start of day but not the time.
    // So, we'll transpose the current day's time onto the selected day.
    // This will ensure unique dates for transactions.
    let dateOffset = date.days(from: Date())

    selectedDate = FamilyData.calendar().date(byAdding: .day, value: dateOffset, to: Date())!

    calendar.reloadData()
}

func calendar(_ calendar: JTACMonthView, didDeselectDate date: Date, cell: JTACDayCell?, cellState: CellState, indexPath: IndexPath) {

    configureCell(view: cell,
                  cellState: cellState)
}

// ---------
// Functions
// ---------

func setupUI() {

    // Calendar views
    calendarView.ibCalendarDelegate = self
    calendarView.ibCalendarDataSource = self

    calendarView.scrollDirection = .horizontal
    calendarView.scrollingMode = .nonStopToSection(withResistance: 0.5)
    calendarView.showsHorizontalScrollIndicator = false

    // If user already had a date selected and is returning to calendar picker, then automatically scroll to their selected date
    // Otherwise, default is today
    calendarView.scrollToDate(selectedDate, animateScroll: false)
    calendarView.selectDates([selectedDate])
}`

Expected Behavior
As you can see, my code is pretty boilerplate. Not a whole lot of customization going on. But when I load the view, I expect it to go right to the selected date, not flash the initial month first.
Additional Context
I've tried putting the scrollToDate code in viewWillAppear as well as viewDidLayoutSubviews, but to no avail. I'm using Xcode 11.3.1, if that helps.

Will have to take a look over the weekend.

Will have to take a look over the weekend.

Thanks!!
On Thu, Jan 14, 2021 at 3:34 PM PatchTheCode<EMAIL_ADDRESS>wrote:

Thanks for your support.
Will have to take a look over the weekend.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/patchthecode/JTAppleCalendar/issues/1304#issuecomment-760545106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHD5UIGMELHUNNJ72YMRM43SZ55RZANCNFSM4WAOI6BQ
.

Thanks!!
On Thu, Jan 14, 2021 at 3:34 PM PatchTheCode<EMAIL_ADDRESS>wrote:

Thanks for your support.
Will have to take a look over the weekend.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/patchthecode/JTAppleCalendar/issues/1304#issuecomment-760545106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHD5UIGMELHUNNJ72YMRM43SZ55RZANCNFSM4WAOI6BQ
.

Hi. Any progress on this?

@vitaliichernysh i think the fastest way to heklp me resolve this is to provide a minimal app which recreates this issue.
Edge case bugs are very hard to find based on code. Often, its one tiny setting based on your app.
So can you please provide an empty app for me which recreates this issue? No, extra designs or company codes etc.
Just a minimal calendar that recreates this issue.


[api] Adds DeferredTranslatorFactory to let serving.properties take effect

Change-Id: I656ec0a0f67176bb0de85763c04e6690db7148a6
Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report

Merging #1868 (d5a1df7) into master (bb5073f) will decrease coverage by 2.32%.
The diff coverage is 65.14%.

@@             Coverage Diff              @@
##             master    #1868      +/-   ##
============================================
- Coverage     72.08%   69.76%   -2.33%
- Complexity     5126     5553     +427
============================================
  Files           473      528      +55
  Lines         21970    24495    +2525
  Branches       2351     2670     +319
============================================
+ Hits          15838    17088    +1250
- Misses         4925     6097    +1172
- Partials       1207     1310     +103

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../cv/translator/SemanticSegmentationTranslator.java
0.00% <0.00%> (ø)

.../cv/translator/StyleTransferTranslatorFactory.java
40.00% <ø> (ø)

... and 424 more

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

Will this breaks the Tensor-in, Tensor-out operation?


Start doesn't work

The start button doesn't work
I have java (the server work in terminal), screen and i set the server to 2048MB
Help please :/

When i do : php cli-run.php action=start server=survie
PHP Notice:  Undefined index: jar in /usr/share/nginx/html/panel/inc/lib.php on line 304

I fond the error :
{"user":"survie","pass":"X","role":"admin","home":"/usr/share/nginx/html/panel/survie","ram":2048,"port":25565}
"jar" is not here
However :


Use the last job as the leader

Should help against build timeouts when number of jobs is very big

The travis failure is caused by flake8 itself.

The number (-1) probably should be in configuration. Always using last may break with matrix.fast_finish if the "after all" code should be run for failures too.
What do you think?


output filename collision

Hello,
I have some trouble using trunk serve
2024-03-13T14:20:05.638423Z  INFO 🚀 Starting trunk 0.19.1
2024-03-13T14:20:06.289681Z  INFO 📦 starting build
warning: output filename collision.
The bin target `my_project` in package `my_project v0.1.0 (/home/n4n5/Documents/github/proj311-test)` has the same output filename as the lib target `my_project` in package `my_project v0.1.0 (/home/n4n5/Documents/github/proj311-test)`.
Colliding filename is: /home/n4n5/Documents/github/proj311-test/target/wasm32-unknown-unknown/debug/my_project.wasm
The targets should have unique names.
Consider changing their names to be unique or compiling them separately.
This may become a hard error in the future; see <https://github.com/rust-lang/cargo/issues/6313>.
warning: output filename collision.
The bin target `my_project` in package `my_project v0.1.0 (/home/n4n5/Documents/github/proj311-test)` has the same output filename as the lib target `my_project` in package `my_project v0.1.0 (/home/n4n5/Documents/github/proj311-test)`.
Colliding filename is: /home/n4n5/Documents/github/proj311-test/target/wasm32-unknown-unknown/debug/my_project.wasm.dwp
The targets should have unique names.
Consider changing their names to be unique or compiling them separately.
This may become a hard error in the future; see <https://github.com/rust-lang/cargo/issues/6313>.
    Finished dev [unoptimized + debuginfo] target(s) in 0.25s
2024-03-13T14:20:07.474165Z ERROR ❌ error
error from build pipeline

Caused by:
    0: HTML build pipeline failed (1 errors), showing first
    1: error from asset pipeline
    2: running cargo build
    3: found more than one target artifact: ["my_project", "my_project"]:
        * consider adding `<link data-trunk rel="rust" data-bin={bin} />` to the index.html to build only the specified binary
        * or adding `<link data-trunk rel="rust" data-target-name={artifact} />` to select the specific artifact by name
2024-03-13T14:20:07.474228Z  INFO 📡 serving static assets at -> /
2024-03-13T14:20:07.474661Z  INFO 📡 server listening at:
2024-03-13T14:20:07.474667Z  INFO     🏠 http://<IP_ADDRESS>:8080/
2024-03-13T14:20:07.474672Z  INFO     🏠 http://[::1]:8080/

Do you have any idea where the problem could come from?
Thanks

That's more of a cargo problem then, right?

I think yes, closing it !
Thanks for trunk


Fix tests by running cargo build a second time in tests

This is a fairly ugly workaround, but it works for now. And I am
tired of getting daily "Travis failure" mails :P
Fixes #51

Thanks for fixing this up @LukasKalbertodt! I've been very distracted with other things lately so hadn't gotten around to looking at it.


Split up angular-ui into angular-ui-router and angular-ui-sortable

Hi @michelsalib @jjeffery @santialbo @panuhorsmalahti @basarat @mattyork @thetallweeks @ofirgeller @gregwebs @cmc19 @vvakame @Bartvds
I was wondering if it might be a good idea to break out the angular-ui typings into angular-ui-router and angular-ui-sortable?  UI-Bootstrap appears to already be following this convention and given the distinct nature of the various angular-ui projects I think it's the more sensible approach.
Would anyone have any objections?

Seems a good idea, I like one-to-one relationships between typings and a bower/npm packages.

Right by me also

Thanks - I might give it a go then.  @santialbo I'll try and keep the 1-2-1 relationship as you suggest (I too :heart: bower)

+1 from me


review/revamp cyberware

Original concept:
https://github.com/breakinblocks/nutsandbolts/issues/135

Done what I can with config changes, Infernal Mobs, and spawner blacklisting. To finish up balancing, Cyberware needs a more fitting later-game recipe for:
Surgery Chamber
Cyberware Charger
Scanner
Engineering Table
Radio Beacon
Radio Tower Components
Radio Kit
Optionally create a recipe for the Robosurgeon which is normally found only in Nether Fortress loot.


Tags is being listed in side menu items for post pages

Describe the bug
Tags is showing up as a side menu item when on a post page.
To Reproduce
Steps to reproduce the behaviour:

Go to dev site
Navigate to views page
Click on any posts
Observe side menu to see error

Expected behaviour
Tags should not be listed in the side menu.
Screenshots

Additional context
N/A

@uttaraghodke & @lliskovoi, the issue described above can be seen on any post page such as this one.
This is because the "Tags" title was changed recently from h3 to h2 due to the fix to this issue. Now the side menu picks up the "Tags" heading.
Personally, I think it is a good feature rather than a bug that the "Tags" title shows up on the side menu so users can click on it to jump to the tags section.
What do you think? Is it OK to have "Tags" title displayed on the side menu?

@cindyli I think it seems fine too me. It is not a problem if Tags appears in the side menu.

Closed as to @uttaraghodke's suggestion above.


tests: drivers: build_all: ethernet: Fix build and improve coverage

This PR aims at cleaning up the build_all test for ethernet drivers.

can I request that the two ST commits are squashed together and the other 3 ethernet test commits are squashed together

can I request that the two ST commits are squashed together and the other 3 ethernet test commits are squashed together

I've squashed the ST commits, and the tests rework. The NET_TEST should be kept separate IMO.


Add make audit

Add an audit command as part of #4264 .
Test by running make audit, regardless of a cargo-audit version being installed. It will install (or update to) the latest, then run audit.
A follow up PR will add this to our CI builds in the SRE repo.
Test locally
make audit

You can test with or without cargo-audit installed.

\retry

/retry

/rebuild

This failed in our CI because openssl headers weren't installed: https://internal.pingcap.net/idc-jenkins/blue/organizations/jenkins/tikv_ghpr_test/detail/tikv_ghpr_test/2030/pipeline/
:(


ibm_db_sa:  decfloat support

From @GoogleCodeExporter on March 16, 2015 6:50
At time of writing, there is no field mapping in ibm_db_sa for the DB2 DECFLOAT
type, so they just come back as strings.  decimal.Decimal is the appropriate
python-side type to map them to, as discussed -
https://groups.google.com/d/msg/ibm_db/mjjocQ-rHEw/OWv3h3zLZI4J

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 14 Mar 2014 at 2:53
Copied from original issue: ibmdb/python-ibmdb#148

From @GoogleCodeExporter on March 16, 2015 6:50
(wonder if it would be possible and a good idea to actually warn if
decimal.getcontext().prec <34 too, it defaults to 28 but can be raised. It's
only relevant if python-side arithmetic is then performed I suppose, a created
Decimal may have more precision than the current context prec, so maybe a
mention of that issue in the documentation would be sufficient, or even
unnecessary since it's just how python decimal is and one might expect a dev to
know that...)

Original comment by<EMAIL_ADDRESS>on 14 Mar 2014 at 2:54

Closing issue as attached issue is closed.
Feel free to report a new one in case this still needs to be addressed.


sync: up to v0.11.1

Details are here https://github.com/golang/tools/releases/tag/v0.11.1.

fixed https://github.com/kamilsk/go-tools/releases/tag/v0.11.1


[Package Request]: Game Editor

How can we help?
I would like someone else to build the manifest.
Please read and ensure the following

[X] The installer meets the above requirements

Please provide the following information
Download Page Url: http://www.game-editor.com/static/Download.html
Publisher: Makslane Rodrigues
Package Name: Game Editor
Description: Game Editor is the open source game design software that gives you the power to create the games of your dreams in a simple interface and a basic version of the C language, and unlike other game creation tools it gives you the chance to get and change the source code of the game creator and design and develop 2D games for personal computers as well as mobile devices.
Package Version: Multiple - see below
Installer URL: BETA version (1.4.1 Beta) AND Stable 1.4.0

Sorry, this package is released as a compressed archive of loose executables. winget.exe does not support this kind of package at the moment.


Scaling

Think about scaling up because git clone <repo> takes a long time.
Ideas:

split the repository (git submodules) into sub-repositories by year.
...

I think I found a solution through the git submodule. Remote repositories can be integrated into LearningPath.


New Bootswatch themes

A collection of new skins based on the themes here. Screenshots included and switcher updated. Also added searchable tags for the showcase. :smile:

This is fabulous work!


Fix query unrecoverable panic

Issue description
When calling Interpreter.QueryContext(...) the prolog query is executed in another goroutine, as this is a different goroutine than the caller's one, if a panic occurs executing the query (i.e. typically executing a predicate) we can't recover from the caller's goroutine.
Proposed solution
The proposed fix moves the panic occurring in the querying goroutine to the Solutions.Next() call to allow the caller to recover.
The implementation simply recover any panic in the querying goroutine, sets the recovered error value to the Solutions structure to propagate the panic in the Next() method, if any.
Remarks
I tried to find the least invasive way possible, but I may not have seen the consequences of that choice, either in design or implementation. Please let me know if you have a more elegant way to handle this, I'd be happy to discuss it.
Thanks!

Hi! Thank you for the high quality PR!
I think custom Go predicates should recover panics and turn them into prolog exceptions so that the caller can catch/3 them. This kind of treatments is already done in length/2 and functor/3 when make([]Term, n) is called with an absurdly large n.
	i.Register0(engine.NewAtom("do_not_call"), func(_ *engine.VM, k engine.Cont, env *engine.Env) (p *engine.Promise) {
		defer func() {
			if r := recover(); r != nil {
				p = engine.Error(fmt.Errorf("panic: %v", r))
			}
		}()
		panic("told you")
	})

?- catch(do_not_call, X, true).
X = error(system_error,'panic: told you').

As for unforeseeable panics, we can do this kind of treatments in Predicate0.call() and (*Promise).Force() as a backup.

Hi! Thank you for your reactivity!

I think custom Go predicates should recover panics and turn them into prolog exceptions so that the caller can catch/3 them.

I agree with you custom Predicate should be panic free and I like the idea to use catch/3. However, even with provided predicate the panic free can't be ensured by design. Typically the Consult returning a delayed Promise, if built on top of a panicking file system is hard to make safe.

As for unforeseeable panics, we can do this kind of treatments in Predicate0.call() and (*Promise).Force() as a backup.

I thought about setting the recover in (*Promise).Force() but as it returns an error the panic any value needed to be wrapped in a dedicated error type, for instance:
type PanickedError struct {
    Value any
}

func (e PanickedError) Error() string {...}

I think introducing a new error type for this may be too much, what do you think ?

Typically the Consult returning a delayed Promise, if built on top of a panicking file system is hard to make safe.

Could you elaborate on that? I've never thought of a file system panicking so if that's the case, we should recover it in consult/1 and convert it to a dedicated prolog exception.
Regarding the error type, I don't think PanickedError is too much as long as it's unexported (so, panickedError?). I also think fmt.Errorf("panic: %v", r) will suffice.
Anyways, I want to leave the option of re-panicking to the users of this library.

Could you elaborate on that? I've never thought of a file system panicking so if that's the case, we should recover it in consult/1 and convert it to a dedicated prolog exception.

Could you elaborate on that? I've never thought of a file system panicking so if that's the case, we should recover it in consult/1 and convert it to a dedicated prolog exception.

That was just an example, a possibility to illustrate that it is difficult to guarantee panic free.

Regarding the error type, I don't think PanickedError is too much as long as it's unexported (so, panickedError?). I also think fmt.Errorf("panic: %v", r) will suffice.

I can implement it that way If you prefer, to be noted that the panic value won't be accessible by users of the library.

That was just an example, a possibility to illustrate that it is difficult to guarantee panic free.

I see. Yes, there're many possibilities to panic and it's a good example other than predicates. Thank you for elaborating on that!

I can implement it that way If you prefer, to be noted that the panic value won't be accessible by users of the library.

Do you think the panic value should be accessible? If not abused, panics are for when the program cannot continue. There's little that can be done with it. So, oftentimes the panic value is simply a string.

Do you think the panic value should be accessible? If not abused, panics are for when the program cannot continue. There's little that can be done with it. So, oftentimes the panic value is simply a string.

To give a more concrete example here is the original motivation, we use this library at OKP4 in a constraint environment on which we must ensure the execution of Prolog programs doesn't exceeds a computation limit, and in order to be strict regarding such requirement the panic seems suitable to provide fail fast behavior.
The possibility to recover the panic value was just under the idea to provide the most open approach to give users as much freedom as possible.
Regarding the alternative you proposed https://github.com/ichiban/prolog/pull/290, I actually like the idea to convert it to a Prolog exception, it enhance the possibilities of panic handling, allowing to manage it either directly in Prolog or as a fail fast mechanism.
I think we may close this one in favor of https://github.com/ichiban/prolog/pull/290 :)

Okay, let's proceed with #290 then! Thank you very much for the contribution!


Option to disallow overpay

A small suggestion:
If possible it would be nice to have an option within the config file that would allow the bot owners to allow/disallow overpay on the bot trades.

Hasn't it been removed completely a while ago?

I wouldn't say it has, I already had 2-3 players that overpaid for an item and I had to manually look up the logs/screens and refund so I think it is still possible for them to overpay.

To prevent errors?

Human errors yes :P

Now that I think about it. When you say overpay, do you mean that people are offering items for your items? @Senakhtenre

Now that I think about it. When you say overpay, do you mean that people are offering items for your items? @Senakhtenre

When someone offers me more for an item than the value I've asked for. Like I ask for 40ref and I'm given 42ref.

Added in v2 https://github.com/Nicklason/tf2-automatic/commit/6aadaf6c5410df68fad139535fcc5ae9e9719205


Add node architecture to the apt repository

Pull Request (PR) description
Add architecture of node in apt repository to remove a warning message on Ubuntu 18.04.

This Pull Request (PR) fixes the following issues
Fixes #80

Hey @Dan33l,
I did the changes you requested.
I hope this is fine :)
Cheers

LGTM. Can you squash ?

Sorry @Dan33l :)
Am I supposed to be able to do it ? I don’t see where is the merge squash button.

Yes you can.  Or you can try it for this PR.
[1] https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History

Hi @meskaya . In fact you did a merge.
But no problem. I will do it. And so i will be commiter and you will be author.


Owssvr readme

Hi Bert,
Added owssvr guidance document.
Thanks,
Raj

Hi @rajashekarusa, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Thx Raja.


Add autocomplete page. Closes #34

//@warpech edit: For issue: https://github.com/StarcounterSamples/KitchenSink/issues/34

Please let me know what's the best approach to de-duplicating code in backend

You can provide handlers in parent using the Action type. For example: https://github.com/StarcounterSamples/People/search?utf8=✓&q=ConfirmAction

Reviewed, assigning back for some fixes of CSS selectors and potential usage of Action

It seems that hooking blur event works only in chrome. I'll some other way tomorrow

@warpech please take a look

The Autocomplete Breadcrumb example does not work if you don't visit the BreadcrumbsPage first.
In style.css there is a redundant } character at line 122.
In the Places autocomplete, please add something for every letter of the alphabet. Otherwise it is hard to guess what values to type.
For both autocompletes, please add some summary: "Your current selection is: _______"

Why would I need a summary

@warpech

I propose to change this line:
if (Db.SQL<long>("SELECT count (p) from Simplified.Ring3.Product p WHERE name = ?", name).First() == 0) {

to this:
if (Db.SQL<Product>("SELECT p FROM Simplified.Ring3.Product p WHERE Name = ? FETCH ?", name, 1).First == null) {

because that's a pattern we use in other places.
Other than that, it's good to merge!

Looks good to merge!


Circleci project setup

@douglas-j-bothwell Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Docs: Archive/Cleanup list of releases

Description

List will grow very large, fairly quickly. Maybe we only show the last few versions
and have an archive in the repo.

we decided to let the list grow.


Gene List API for fast-updating results

Our Keyword Search has a great, fast, lightweight auto-updating results bar, directly underneath it. It displays in text format, the amount of documents and genes found with the inputted search query.
Our Gene List Search simply states how many genes were inputted, but does not have an actively updating count like the Keyword Search. Once you hit search though, beside the Search button, will state how many linked and unlinked genes were found. This is good, and works fine, but because of the MultiSpecies functionality, if users were to have Wheat selected top left, and searched for Zea Mays gene lists (like ZM00001EB038010, with keywords), KnetMiner returns the following string: "In total No user genes found. Showing keyword-centric results. In total 35625 were found. (7.09 seconds).". This string is often missed, as the user still gets results and may quickly scroll down (or have the camera auto-move down, even), and definitely confuses the user because he/she is no longer getting the expected output, yet it's outputting something.
Two things led to this problem being more pronounced now than previously:

We moved where the errors/ text box appears to now be beside the search button.
We now have multispecies functionality and the user can easily "break" the app by clicking Example 3 (Wheat) with Zea Mays.

I propose two ideas:

Move the live-updating query-result down, just below the search button and make it global. It then consideres all three search types.
Add a Gene-List Search result text (same as KeyWord Search one, with live updating text-based result reporting) just below the gene-list search.

I think going with the first option would be best and the least confusing from a user's perspective. But technically we could word it differently and display both even. Almost like a running total and a grand total.
@jojicunnunni can work on the API
@manlikleo on the frontend implementation

Review and find the points where these searches are made in KnetMiner.
See if this could be an entry point:

We want to know how many genes inputted by the user has a match, combined with inputted keywords.
Something else: The offline initialiser is an ondex component and that also needs to be reviewed - config has been updated and it needs to be made compatible with the new way that KnetMiner gets its configuration.

This is on hold

Migrated to https://github.com/KnetMiner/knetminer/issues/13


Cotary Create: Create DAOForm object for deployment.

In this step we going Install/Deploy the DAO Form Object
https://www.figma.com/proto/t77rlBAupEeqIYHFBBpqrl/Playground?scaling=scale-down&node-id=1192%3A3164

Step 4 in ui v1 was in part used to update the daoCreator's state, flipping isMigrating between true and false. Could someone let me know how this translates in v2? @dOrgJelli @mrrobot16 @cbrzn
Also, what changes are to be made to the Migrator? It's currently still class based which I imagine is temporary but it would be useful to know if there is anything functional I should keep in mind while implementing it into this step.

To my understanding there is not anything new functionally speaking. Regards being class we gonna convert that into a Hook i suppose. I am not what extra functionality we gonna need.

Sir! Sorry for the late replay, here's what we need:

Step 4 in ui v1 was in part used to update the daoCreator's state, flipping isMigrating between true and false. Could someone let me know how this translates in v2? @dOrgJelli @mrrobot16 @cbrzn

It's the same logic, since when the user clicks "Deploy" we'd need to have that state too, and let the user know that the migration of the DAO is being done.
I'm not really sure yet how we need to handle the events of the migration (not sure how the creation - configuration process works) here's basically what will happen when the users click "Deploy". (@dOrgJelli if you can give us maybe some example code to understand this would be awesome!)

Also, what changes are to be made to the Migrator? It's currently still class based which I imagine is temporary but it would be useful to know if there is anything functional I should keep in mind while implementing it into this step.

We need to improve the UI of this component, it needs to be like the mock up - The style library that we are using now is MDBReact and not Material UI. Also make it a functional component like you said
@Flash-Git


401k calculator

Non working proof of concept.

+1

+1

+1

@DanielDiTommaso , you got the two "+1"s so you're ready to merge!


pip-sync broken with editables

pip-sync is broken with editables: It will switch between installing and uninstalling them in consecutive installations:
vagrant@v-ston ~/t/envtest> cat requirements.txt
-e git+https://github.com/Xof/django-pglocks.git@f5df93e#egg=django-pglocks

vagrant@v-ston ~/t/envtest> virtualenv -p python3 .
Running virtualenv with interpreter /usr/bin/python3
Using base prefix '/usr'
New python executable in ./bin/python3
Also creating executable in ./bin/python
Installing setuptools, pip, wheel...done.

vagrant@v-ston ~/t/envtest> . bin/activate.fish

(envtest)vagrant@v-ston ~/t/envtest> pip install pip-tools
Collecting pip-tools
  Using cached pip_tools-1.3.0-py2.py3-none-any.whl
Collecting first (from pip-tools)
  Using cached first-2.0.1-py2.py3-none-any.whl
Collecting six (from pip-tools)
  Using cached six-1.10.0-py2.py3-none-any.whl
Collecting click>=6 (from pip-tools)
  Using cached click-6.2-py2.py3-none-any.whl
Installing collected packages: first, six, click, pip-tools
Successfully installed click-6.2 first-2.0.1 pip-tools-1.3.0 six-1.10.0

(envtest)vagrant@v-ston ~/t/envtest> pip-sync
Collecting django-pglocks from git+https://github.com/Xof/django-pglocks.git@f5df93e#egg=django-pglocks
  Cloning https://github.com/Xof/django-pglocks.git (to f5df93e) to /tmp/pip-build-ete6jfh2/django-pglocks
  Could not find a tag or branch 'f5df93e', assuming commit.
Installing collected packages: django-pglocks
  Running setup.py install for django-pglocks
Successfully installed django-pglocks-1.0.2

(envtest)vagrant@v-ston ~/t/envtest> pip-sync
Uninstalling django-pglocks-1.0.2:
  Successfully uninstalled django-pglocks-1.0.2
Requirement already satisfied (use --upgrade to upgrade): django-pglocks from git+https://github.com/Xof/django-pglocks.git@f5df93e#egg=django-pglocks in ./lib/python3.4/site
-packages

(envtest)vagrant@v-ston ~/t/envtest> python
Python 3.4.3 (default, Oct 14 2015, 20:28:29)
[GCC 4.8.4] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import django_pglocks
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: No module named 'django_pglocks'

+1, I have to run it twice every time since upgrading to 1.3 (I was on 1.1.6 before).

This has been addressed in 1.4.1. It was a weird bug, but fixed in 8f64896e11a1d34a27b4f4c5d12bed04336a7c75. Thanks for finding/reporting!


CLI tool for migrating webpack 1 config file to webpack 2

Do you want to request a feature or report a bug?
technically not feature for webpack but an independent cli tool
What is the current behavior?
we need to update each webpack.config.js manually
What is the expected behavior?
something like eslint --fix but webpack-migrate webpack.config.js would make it webpack 2 compatible.
If this is a feature request, what is motivation or use case for changing the behavior?
it hurts if we have to migrate many webpack 1 projects

Could you re-open this at webpack-cli repo? Thanks.

Issue is now open in the webpack-cli repo: webpack/webpack-cli#6


CommunityToolkit docs for EventStore integration

Summary
CommunityToolkit docs for EventStore integration
Integration PR: https://github.com/CommunityToolkit/Aspire/pull/277

Internal previews

📄 File
🔗 Preview link

docs/community-toolkit/hosting-eventstore.md
.NET Aspire Community Toolkit EventStore integration

We also need this to be added to the toc.yml.

We also need this to be added to the toc.yml.

Thanks @IEvangelist , I just pushed the suggested changes.


maxPagesToFetch bug

Ok, this is a bit more subtle.
The config option is maxPagesToFetch. However, as currently implemented its semantics would be better described as maxPagesToSchedule.
Consider the following use case:

I want to crawl 20 pages for a given site, but queue up everything else
I can decide later to continue crawl the site, based on the queue made in stage 1.
This fails because once the queue has 20 pages in it, no more pages are added to it.

This patch fixes the semantic of the config option so that pages are added to the queue, but queue consumption stops once maxPagesToFetch has been exceeded. This is also useful if you are inserting pages into the queue with higher priorities.

In the changes I don't see any use to the maxPagesToSchedule

To be clear, I didn't name any code called maxPagesToSchedule. However, maxPagesToSchedule could be added in schedule() and scheduleAll(). But I agree with your premise that such semantics (i.e. the status quo) is not actually useful.


Create 0000-01-02-stepin.md

created a branch, created a file and made a commit, and opened a pull
request


Add configuration option for custom pattern for Tailwind color palette

There is currently a hardcoded Regex that matches the default Tailwind v3 color theme names:
https://github.com/dillonkearns/elm-review-html-to-elm/blob/e2ed0f06e5e571ae8bd7bff42cde6e89da3c2b34/src/TailwindClass.elm#L124
However, this will not catch colors from a custom Tailwind configuration if they don't follow the same pattern. One solution to support this would be to add an option in the configuration where the user can pass in a custom Regex string that will be used instead of the default when provided.

See also discussion in #21


Client Refactoring: Pull out multiplayer position logic

Specifically, refactor client.on('players', function(others) { out of src/client.js into a file concerned with multiplayer stuff.

Completed in 2eab1cc88058fcd7c45609311fda45b09da14b7d


Understanding pre-training and fine-tuning

I am confused about what these two steps actually do to the model. I would have assumed that pre-training is unsupervised (i.e. no labels) and, thus, the only thing that can be 'learned' is the embedding representations of all tokens. You can then use this pre-trained model (which is an 'empty' model but with pretrained vector representations of tokens) to actually train the whole mode.
However, from the things that I read that is not the case. It's not only the embeddings that are pretrained, but also the whole model. Is that right? How can that be, without labels?
For context: I want to train a Bert model on a new language and I am trying to figure out which steps I have to do to get there. Can I load custom embeddings, and do I then still need to pretrain the rest of the model (but how, which labels?).
I know that my question is quite general, but the difference between pre-training and fine-tuning is not clear to me. Pre-training should be 'done once per language', but then what does the model actually learn? It it simply a seq-to-seq pre-training?

Maybe a good introduction on the topic are the various writings of Sebastian Ruder:

http://ruder.io/transfer-learning/
https://thegradient.pub/nlp-imagenet/
the ULMFiT paper: http://nlp.fast.ai/classification/2018/05/15/introducting-ulmfit.html


Update the uxSchedulerSuspended after prvCheckForRunStateChange

Description

Update the uxSchedulerSuspended after the prvCheckForRunStateChange to prevent race condition in fromISR APIs

Consider the following sequence:

Core A tries to suspend the scheduler and are requested to yield ( Task lock is held )
Core B enters the critical section From ISR ( ISR lock is held ) and reading the uxSchedulerSuspended

The original implementation is to update the uxSchedulerSuspended without ISR lock.
FromISR version API can still read the uxSchedulerSuspended when task is updating the variable.
The problem in xTaskRemoveFromEventList

After uxSchedulerSuspended is udpated to 0, the task can be put to the ready list directly.
Before uxSchedulerSuspended is updated to 0, the task will be put to the pending ready list and added back to the ready list when scheduler is resumed. However, if the task is requested to yield. The task in pending ready list will be blocked. This can be reproduced easily with proper delay.

            uxSchedulerSuspended = 0U;

            if( uxPrevCriticalNesting > 0U )
            {
                pxThisTCB->uxCriticalNesting = 0U;
                portRELEASE_ISR_LOCK();
                portRELEASE_TASK_LOCK();
            }
            else
            {
                /* uxPrevSchedulerSuspended must be 1 */
                portRELEASE_TASK_LOCK();
            }

The fix in ab074f54e. This ensure that the uxSchedulerSuspended is always 1 in this scenario since portGET_ISR_LOCK ensure the order, which is the before case in previous section and is not expected.
                if( uxPrevCriticalNesting > 0U )
                {
                    portSET_CRITICAL_NESTING_COUNT( 0U );
                }
                else
                {
                    /* Adding proper delay here to increase the probability ISR get the lock
                     * first to reproduce the problem. In that case, uxSchedulerSuspended is 1
                     * in the from ISR API. */
                    portGET_ISR_LOCK();
                    uxSchedulerSuspended = 0U;
                }

                portRELEASE_ISR_LOCK();
                portRELEASE_TASK_LOCK();

The suggested change in the PR. The uxSchedulerSuspended is update only when the task is not requested to yield. This prevent the race condition.
            /* uxSchedulerSuspended is increased after prvCheckForRunStateChange. The
             * purpose is to prevent fromISR API read the uxSchedulerSuspended if the
             * task is requested to yield. */
            if( uxSchedulerSuspended == 0U )
            {
                if( portGET_CRITICAL_NESTING_COUNT() == 0U )
                {
                    prvCheckForRunStateChange();
                }
                else
                {
                    mtCOVERAGE_TEST_MARKER();
                }
            }
            else
            {
                mtCOVERAGE_TEST_MARKER();
            }

            portGET_ISR_LOCK();

            /* The scheduler is suspended if uxSchedulerSuspended is non-zero. An increment
             * is used to allow calls to vTaskSuspendAll() to nest. */
            ++uxSchedulerSuspended;
            portRELEASE_ISR_LOCK();

            portCLEAR_INTERRUPT_MASK( ulState );

Test Steps

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The problem ( the before case )

The suggested fix in this PR

This problem can be observed when running demos on XMOS AICORE.
Without this patch, there will be the following error
xrun: Program received signal ET_ECALL, Application exception.
      [Switching to tile[0] core[6]]
      prvSyncTask (pvParameters=0x8) at ../../../../Common/Minimal\EventGroupsDemo.c:304

      304               configASSERT( ( uxReturned & ebALL_SYNC_BITS ) != ebALL_SYNC_BITS );
      Current language:  auto; currently minimal
make: *** [run] Error 125


Accessing the invalid memory address

Occurs when the user enters after starting the server.
Occurs if the user name is Korean.

I fixed a problem about string length recently.
could you check it again?
It's hard to check on my system

I will close this issue. you can reopen it if the problem is not fixed


TBS East

Source
https://www.ontvtonight.com/guide/listings/channel/69046986/tbs-superstation-east.html
Notes
This TBS guide is wrong. I am watching is actually American Dad but your guide schedule said Happy Glimore. Wrong schedule. The link above is correct.

Closed in favor #1310


Where is the documentation for the Tapable.plugin deprecation?

(node:63533) DeprecationWarning: Tapable.plugin is deprecated. Use new API on .hooks instead

I'm trying to find docs for this use case:
const compiler = webpack(webpackConfig);
compiler.plugin('done', function(stat) {
});

How to I rewrite compiler.plugin('done'…) with the new Webpack API?

Ok, found the docs: https://webpack.js.org/api/compilation-hooks/
So, I should me doing:
    compiler.hooks.done.tap('some-string', function(stat) {
      console.log('first build complete');
    });

Is some-string a key of my choosing, perhaps to allow me to unregister the hook later?

It looks like you just deleted our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please either update the issue with the template and reopen, or open a new issue.


2 dockerfile issues

This is a draft.
I'm trying to make this project usable again but it still don't work.
Please freeze your version !!!

https://github.com/mansheej/data_diet/issues/2


Why is there no sample code?

Sample code should be required for all documentation. Sample code is the best type of documentation and it is missing most of the time now a days. :(

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

ID: 1be36c1a-bf94-fd12-7010-4ca4dc38d419
Version Independent ID: 1d043ff3-d5fa-1e6f-cf26-223f89155e5d
Content: DataLakeDirectoryClient.CreateIfNotExists(PathHttpHeaders, IDictionary<String,String>, String, String, CancellationToken) Method (Azure.Storage.Files.DataLake) - Azure for .NET Developers
Content Source: xml/Azure.Storage.Files.DataLake/DataLakeDirectoryClient.xml
Service: azure
GitHub Login: @CamSoper
Microsoft Alias: casoper

@fvalencia12 thanks for reporting this -- are you looking for a specific sample for that API?

@fvalencia12 thanks for reporting this -- are you looking for a specific sample for that API?

Yes, it is good to have documentation, but using that method is confusing to me and needed a sample of how to use it. It took me a bit to find some sample online in how to use it. It would help a lot if we can find samples so we know how to use the code. Something like this:
public override bool OnStart()
{
ServicePointManager.DefaultConnectionLimit = 12;
        var dbConnString = CloudConfigurationManager.GetSetting("PhotoGalleryDbConnectionString");
        db = new PhotoGalleryContext(dbConnString);

        var storageAccount = CloudStorageAccount.Parse(RoleEnvironment.GetConfigurationSettingValue("StorageConnectionString"));

        Trace.TraceInformation("Creating images blob container");
        var blobClient = storageAccount.CreateCloudBlobClient();
        imagesBlobContainer = blobClient.GetContainerReference("images");
        if (imagesBlobContainer.CreateIfNotExists())
        {

            imagesBlobContainer.SetPermissions(new BlobContainerPermissions { PublicAccess = BlobContainerPublicAccessType.Blob });
        }

        Trace.TraceInformation("Creating images queue");
        CloudQueueClient queueClient = storageAccount.CreateCloudQueueClient();
        imagesQueue = queueClient.GetQueueReference("images");
        imagesQueue.CreateIfNotExists();

        Trace.TraceInformation("Storage initialized");
        return base.OnStart();
    }

Fabian Valencia
Information Technology
William Blair
The William Blair Building | 150 North Riverside Plaza, Chicago, Illinois 60606
Direct: +1-312-364-5132
fvalencia<EMAIL_ADDRESS>| williamblair.comhttp://www.williamblair.com/
Facebookhttps://www.facebook.com/WilliamBlairCo/ | LinkedInhttps://www.linkedin.com/company/166939 | YouTubehttps://www.youtube.com/channel/UC7CPnCaTlZcLxOF7WFwY0uw | Twitterhttps://twitter.com/WilliamBlair
From: Alex Ghiondea<EMAIL_ADDRESS>Sent: Wednesday, January 13, 2021 10:56 AM
To: Azure/azure-sdk-for-net<EMAIL_ADDRESS>Cc: Valencia, Fabian<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-net] Why is there no sample code? (#17937)
@fvalencia12https://github.com/fvalencia12 thanks for reporting this -- are you looking for a specific sample for that API?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-sdk-for-net/issues/17937#issuecomment-759581818, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQZG2MZIRQZOJSPVVN2MRM3SZXGDDANCNFSM4WA7MBMA.

Yes, it is good to have documentation, but using that method is confusing to me and needed a sample of how to use it. It took me a bit to find some sample online in how to use it. It would help a lot if we can find samples so we know how to use the code. Something like this:
public override bool OnStart()
{
ServicePointManager.DefaultConnectionLimit = 12;
        var dbConnString = CloudConfigurationManager.GetSetting("PhotoGalleryDbConnectionString");
        db = new PhotoGalleryContext(dbConnString);

        var storageAccount = CloudStorageAccount.Parse(RoleEnvironment.GetConfigurationSettingValue("StorageConnectionString"));

        Trace.TraceInformation("Creating images blob container");
        var blobClient = storageAccount.CreateCloudBlobClient();
        imagesBlobContainer = blobClient.GetContainerReference("images");
        if (imagesBlobContainer.CreateIfNotExists())
        {

            imagesBlobContainer.SetPermissions(new BlobContainerPermissions { PublicAccess = BlobContainerPublicAccessType.Blob });
        }

        Trace.TraceInformation("Creating images queue");
        CloudQueueClient queueClient = storageAccount.CreateCloudQueueClient();
        imagesQueue = queueClient.GetQueueReference("images");
        imagesQueue.CreateIfNotExists();

        Trace.TraceInformation("Storage initialized");
        return base.OnStart();
    }

Fabian Valencia
Information Technology
William Blair
The William Blair Building | 150 North Riverside Plaza, Chicago, Illinois 60606
Direct: +1-312-364-5132
fvalencia<EMAIL_ADDRESS>| williamblair.comhttp://www.williamblair.com/
Facebookhttps://www.facebook.com/WilliamBlairCo/ | LinkedInhttps://www.linkedin.com/company/166939 | YouTubehttps://www.youtube.com/channel/UC7CPnCaTlZcLxOF7WFwY0uw | Twitterhttps://twitter.com/WilliamBlair
From: Alex Ghiondea<EMAIL_ADDRESS>Sent: Wednesday, January 13, 2021 10:56 AM
To: Azure/azure-sdk-for-net<EMAIL_ADDRESS>Cc: Valencia, Fabian<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-net] Why is there no sample code? (#17937)
@fvalencia12https://github.com/fvalencia12 thanks for reporting this -- are you looking for a specific sample for that API?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-sdk-for-net/issues/17937#issuecomment-759581818, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQZG2MZIRQZOJSPVVN2MRM3SZXGDDANCNFSM4WA7MBMA.

@fvalencia12 thanks for the detailed response.
While we don't have samples directly in the reference documentation, we do have some samples here. Would you be interested in contributing a sample there?

@fvalencia12 thanks for the detailed response.
While we don't have samples directly in the reference documentation, we do have some samples here. Would you be interested in contributing a sample there?

@fvalencia12 The code snippet you provided uses the v11 library instead of the v12 library. The v11 library is deprecated, therefore we cannot create newer samples with this library.
If you're interested in using our v12 library we have many places with documentation and simpler samples as well.
Azure.Storage.Blobs
Azure.Storage.Files.Shares
Azure.Storage.Files.DataLake
Azure.Storage.Queues
Was their a specific example you were looking for? What API are you looking to use?

@fvalencia12 The code snippet you provided uses the v11 library instead of the v12 library. The v11 library is deprecated, therefore we cannot create newer samples with this library.
If you're interested in using our v12 library we have many places with documentation and simpler samples as well.
Azure.Storage.Blobs
Azure.Storage.Files.Shares
Azure.Storage.Files.DataLake
Azure.Storage.Queues
Was their a specific example you were looking for? What API are you looking to use?

Please reopen if there are any further questions.

Please reopen if there are any further questions.


Add tox-gh to conda-forge

Hi there,
Thanks for developing this plugin. Very helpful!
It would be great to have tox-gh on Conda-forge. Would this be something that the tox developers would be amenable to? I can open a draft PR if that's of interest.
Best

This can be done without our involvement. You are welcome to add it to conda-forge yourself. Thanks!

Sounds good! For reference: https://github.com/conda-forge/staged-recipes/pull/27581


fix(GlobalHeader): a11y fixes for resources & catalog drop downs

Overview

hides decorative right arrow icon from screen readers on links (in resources dropdown, the arrow next to 'view all docs' and 'view all topics'; in the catalog dropdown it's next to 'take our quiz')
hides decorative box svg from catalog dropdown (next to "dont know where to start" in catalog dropdown)

PR Checklist

[ ] Related to designs:
[x] Related to JIRA ticket: DISC-898, DISC-900
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change
[x] This PR includes testing instructions tests for the code change
[x] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

testing in just monolith should be enough
monolith PR has screenshots as well

Visit preview env
Navigate global header w/ voice over
Open the resources menu and confirm that view all docs and view all topics are not read w/ the right arrow symbol
Open the catalog menu and confirm that the img next to 'take our quiz' is not read, and neither is the right arrow symbol
Approve!

🚀 Styleguide deploy preview ready!https://634f592442ffdf2e2fe6f6e1--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.214-alpha.c3af97.0@codecademy/gamut-labs@33.6.5-alpha.c3af97.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.217-alpha.96cf38.0@codecademy/gamut-labs@33.6.8-alpha.96cf38.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6352a321f4766908ba0bd961--gamut-preview.netlify.appDeploy Logs


simplify gather->shapeof into shapeof->gather for 0D indices

@GlebKazantaev please review and estimate how much time will it take to rewrite internal OV transformations preserving names to make this merge happen.

@jane-intel @GlebKazantaev this PR has topk unittest and @diyessi has a fix for it, we are testing this as MR on OV repo.


When creating a new submission, check that user does not already have one

Make configurable?

Now implicit.


Fix performance regression results from selection intersection in runtime-filter exploration-phase.

What type of PR is this：

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

Which issues of this PR fixes ：

Fixes #
This PR https://github.com/StarRocks/starrocks/pull/4934 causes the performance regression on branch-2.2
Problem Summary(Required) ：

vector::assign(n, 1) is needless and time-consuming so remove it, and make the for-loop of bloom-filter probing more tight.
the performance test as follows.
+===================================+========+===================+================+
| query                             | 2.2.1  | 2.2.2(regression) | this PR(2.2.2) |
+===================================+========+===================+================+
| join.18_ssb_inner.q4.q_18_4_4.sql | 0.224s | 0.350s            | 0.223s         |
| join.19_ssb_left.q4.q_19_4_4.sql  | 0.234s | 0.336s            | 0.219s         |
+-----------------------------------+--------+-------------------+----------------+

run starrocks_be_unittest

@Mergifyio backport branch-2.3


[REQUEST] Add sub-beats to main .beats time value

Hello and thanks for this handy extension - so far is the only one which can display Internet time.
Two sites https://beats.wiki/ and http://www.swatchclock.com/ feature beats "seconds" or sub-beats, centibeats as wikipedia article describes these - it would be cool to have these provided by extension as well - if it's of course possible to do so.

Modern desktops don’t display seconds to reduce the number of screen updates. It’s an energy saving feature. Knowing the background, do you still want this feature? How much precision would you need? .beats is mostly useful today as a universal ordination time within teams. Do you really need sub-beat precision to arrange your meetings?

I come here to suggest a feature that would enhance this extension a bit more since it already includes quite large variety of displaying preferences and the last thing I want is to being lectured on "energy saving". If that's too much to ask for then forgive me for bothering you guys and your precious time. If you care so much about wasting energy then don't reply to issues you find pointless.
Thanks for nothing


lvh.me should be considered?

Since lvh.me points to localhost

I'm not making specific exceptions here as I'm using the https://github.com/publicsuffix/list as it it.


Suggestion: Use monaco code editor in examples gallery

https://github.com/Microsoft/monaco-editor
With the expected soon arrival of TypeScript typings, regl example editor could really benefit from all the features offered by Monaco editor, such as

autocompletion,
type inference,
inline API documentation.

We could even build a custom syntax definition for syntax coloring of GLSL in frag and vert fields, kinda like JS/CSS embedded in HTML.

Cool, this issue should probably be moved over to the regl-site repo where it can get fixed.

OK, closing in favor of https://github.com/freeman-lab/regl-site/issues/13


[SPARK-32163][SQL][3.0] Nested pruning should work even with cosmetic variations

What changes were proposed in this pull request?

This patch proposes to deal with cosmetic variations when processing nested column extractors in NestedColumnAliasing. Currently if cosmetic variations are in the nested column extractors, the query is not optimized.
This backports #28988 to branch-3.0.
Why are the changes needed?

If the expressions extracting nested fields have cosmetic variations like qualifier difference, currently nested column pruning cannot work well.
For example, two attributes which are semantically the same, are referred in a query, but the nested column extractors of them are treated differently when we deal with nested column pruning.
Does this PR introduce any user-facing change?

Yes, fixing a bug in nested column pruning.
How was this patch tested?

Unit test.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/29856/
Test PASSed.

Test build #125245 has started for PR 29027 at commit 5e4a420.

Thanks for the backport, @viirya .

Merged to branch-3.0. Thank you, @viirya and @maropu .
All UTs (including R) already passed in the current running Jenkins.

Test build #125245 has finished for PR 29027 at commit 5e4a420.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/125245/
Test PASSed.

Thanks @dongjoon-hyun @maropu


Widgets keep showing 'Loading content...' instead of 'No pull requests available`

Dev Home version
0.1901.687.0
Dev Home Azure Extension version
0.1300.624.0
Windows build number
10.0.26120.0
Other software
No response
Steps to reproduce the bug
I've added three widgets, one 'My Pull Requests' and two 'Pull Requests'.
These widgets worked fine the first time that I added them.
However, over the next few days, I noticed that the widgets that would logically not show any data (there were no PR's for that specific repo for example) would just show loading content... all the time.
I would expect these widgets to read something like No data to be shown or No data available.
I do recall seeing a message similar to this once or twice on the widgets.
Here is a screenshot of what I mean (added white boxes over some parts to anonymize them)

Expected result
When a widget on the dashboard would not contain any data (because the repository doesn't currently have any PRs open) I would expect that widget to show something to the extend of Currently no open Pull Requests available.
Actual result
The widgets keep showing Loading content...
Included System Information
No response

After a quick look at the code, I would say this line (in AzurePullRequestsRepositoryWidget) might be a bit incorrect:
itemsData.Add("is_loading_data", DataState != WidgetDataState.Unknown);
Because the WidgetDataState enum has the following possible states:

So if the state becomes Okay the widget would still show loading content...


docs(storybook): fix broken contribution guide link

As the storybook docs import from README.md, the link refers to './CONTRIBUTING.md' which works on github but on Storybook it redirects to 'https://baklava.design/CONTRIBUTING.md' which naturally results in 404.
Only working solution I could come up without creating the need to update both files seperately is to use absolute paths rather than relative, and have both github and storybook links avaliable
Fixes #568

Hi @cagatay-usta. Thanks for the PR.
We use Readme.md as the main welcome content in our repository, and that link works well there. But we also include those md files to our documentation, then problem occurs. Ideally, this link should go to the Contribution guideline in our documentation: https://baklava.design/?path=/docs/documentation-contributing-baklava-contribution-guideline--page But we also have one more environment for our docs for beta versions: https://next.baklava.design I'm not completely sure what is the best way of handling this.
Maybe we should consider preprocessing of those links before publishing our docs. 🤔

There is another suboptimal workaround. Keeping github link absolute and storybook link relative,
As storybook MDX docs recommends:
[Go to specific documentation page](?path=/docs/some--id)
For more information please check our [Contribution Guideline document on GitHub](https://github.com/Trendyol/baklava/blob/next/CONTRIBUTING.md), or [on Storybook](?path=/docs/documentation-contributing-baklava-contribution-guideline--documentation).

I tested it locally and both links work on storybook using the above version and it should work on next.baklava.design too, however it's still suboptimal as if one clicks the storybook link on github it wont direct them to storybook page. Still an improvement to current situation though, all links on baklava.design and next.baklava.design will work, and github will still link to github. Only the link to storybook guideline link on github wont work.
I  also experimented with options.overrides on markdawn component here welcome.stories.mdx :
import { Meta, Markdown } from '@storybook/blocks';
import ReadMe from '../README.md?raw';

<Meta title="Documentation/Welcome" />

<Markdown>{ReadMe}</Markdown>

but the problem is storybook passes it's own options down to markdown-to-jsx library (I think), as when I try to pass my options to override tags, it messes up with storybook's and codeblocks etc. wont display properly. Someone with more experince may find a way to do it properly so I will link the related docs for the library: https://github.com/probablyup/markdown-to-jsx#optionsoverrides---override-any-html-tags-representation


update 2 photon session schema

Updating the 2-photon session schema.
Updated Schema will roughly follow this outline:

Ophys_Session_type: (str) "2-Photon"

Metadata: (object)

Session id
Mouse id
Rig id
User id

Subject (object):

nested subject schema

ophys_rig (object):

nested ophys_rig schema

Data streams

eye tracking video
body tracking video
face tracking video
wheel encoder
stimulus
physio

field of view:

nested field of view schema

behavior task

lick detection
water/rewards delivery
task parameters
task performance

Can we break these into smaller pieces? I think it'll be easier to get them aligned.
eg. Several of these data streams are already modeled for the ephys session and we can bring those over. It's very important to us to keep our schemas for different modalities as closely aligned as possible. (so far our ophys users haven't needed them, which is why they aren't here yet)

yes, I was thinking that we would be making a schema for each of the datastreams

@saskiad can you please point me to where the streams are already modeled? I've been looking here:
https://github.com/AllenNeuralDynamics/aind-data-schema/blob/main/src/aind_data_schema/ephys/ephys_session.py
From what I can tell you're referring to this stream model and implemented in the session schema here
These look a little more generic but if there is something more specific I'd love to take a look

Many of these are modeled in the device schema:
https://github.com/AllenNeuralDynamics/aind-data-schema/blob/main/src/aind_data_schema/device.py

closed with PR #266


Add MFA info to UserRecord

Discussion
Issue: https://github.com/firebase/firebase-admin-go/issues/421
This PR implements a small part of Multi Factor Authentication (MFA). I do realize this isn't a full implementation, unfortunately I don't have time for that. This PR only implements unmarshalling the MFA info from the REST response into the UserRecord struct.
For structure and naming within the UserRecord, I've decided to be consistent with the Node.js client library, which already has this implemented: https://firebase.google.com/docs/reference/admin/node/admin.auth.UserRecord
Testing

I've updated the existing tests to also incorporate MFA. On the existing testUser, I've added MFA info. Next to that, I've introduced a test user without MFA. In the list endpoints, I replaced every third user with a non MFA user, so the list tests both users with and without MFA info available. All unit tests should be OK.
Unfortunately, I wasn't able to get the integration tests running locally, so I'm not sure whether I did break something there. I did manage to test the code in an actual project with a small local setup and verified that it works.

API Changes

This PR does not change the public API. The REST API already returns the mfaInfo section. This PR merely adds the code to pick it up into Golang structs.

Thanks @jasperkuperus for the PR. It looks mostly good. I've suggested some changes to the public API.
@bojeil-google how do you feel about just releasing the changes to UserRecord without the support for updating MFA settings?

I think it's fine to do so, as long as we keep track of the missing functionality (updating MFA settings). Ultimately, we want to make sure we have feature parity across all languages.

API review process for this has been initiated. Should have some updates by next week.

This API has been approved with some minor changes. The change being removal of the MultiFactorID type and the related Phone constant (we will use FactorID string instead).
@jasperkuperus would you like to push the change to this PR? If you're busy I can merge this as-is, and implement the required changes on top of that.

@jasperkuperus would you like to push the change to this PR? If you're busy I can merge this as-is, and implement the required changes on top of that.

@hiranya911  It's a small change, but I have a rather busy week for me. If you could be so kind to merge it and implement the change on top of that, it'd be of great help, thanks! 😊