Test sensitivity of sample matching to exact values of mismatch probabilities and switch probabilities

Right now, small values of 1e-8 and 1e-20 are used for mismatch and switch probabilities, respectively. A previous preliminary analysis shows that precision set to get HMM path likelihood values can affect results (going from 10 to 20, noticeably). So, even the set values of mismatch and switch probabilities may have an effect. It would be good to check by setting the mismatch and switch probabilities to, say, 1e-6 and 1e-18, respectively, or possibly higher. It should be quick to test it on a handful of sample genomes anyway.

It turns out that MMR is not simply rho / mu. The relationship is more complicated, and it is explained a bit in #20. I did an experiment where I set mutation rate to 1e-8 and 1e-7 and ran the matching using MMR using the right formula. I then compared the imputed alleles from the experiments. I obtained more paths that are identical (in terms of the number of parent node ids correct) then simply maintaining rho / mu. There are, however, still some discrepancies, which may be due to numerical instability.


2604 location accuracy

Add location accuracy filter according to the current resolution.
The addition is made in get_accidents_stats where possible.

@ziv17 can you prod merge dev into this branch? I want to examine in the pr only the relevant changes for this issue

@ziv17 can you prod merge dev into this branch? I want to examine in the pr only the relevant changes for this issue

I rebased on top of upstream/dev.

Codecov Report
Attention: Patch coverage is 88.77551% with 11 lines in your changes are missing coverage. Please review.

Project coverage is 53.52%. Comparing base (400af13) to head (b26f4b2).
Report is 21 commits behind head on dev.

Files
Patch %
Lines

...cident_type_vehicle_type_road_comparison_widget.py
42.85%
4 Missing :warning:

anyway/widgets/widget_utils.py
91.83%
4 Missing :warning:

...dgets/injured_accidents_with_pedestrians_widget.py
0.00%
2 Missing :warning:

...ed_and_injured_count_per_age_group_widget_utils.py
90.90%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2641      +/-   ##
==========================================
+ Coverage   53.22%   53.52%   +0.30%
==========================================
  Files         119      119
  Lines        9924     9956      +32
==========================================
+ Hits         5282     5329      +47
+ Misses       4642     4627      -15

Flag
Coverage Δ

unittests
53.52% <88.77%> (+0.30%)
:arrow_up:

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

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


[modify] codeclimate.yml

https://docs.codeclimate.com/docs/advanced-configuration#analysis-configuration-versions

Coverage increased (+0.1%) to 84.421% when pulling 2ac8baa508152640c0cc91c533af5cb6a3753c88 on fix-codeclimate-yml into 006024bbdf1e8738f7dacb6fb3d56501e3508f26 on master.


Add tests and documents on https://github.com/vim-denops/vim-channel-command-deno

Like tests and documents in https://github.com/lambdalisue/msgpack-rpc-deno

Closed by #5


Need to add support for Initial Infection

After starting a new game, user should be directed to an Infection screen.
The initial infection is unique from normal infections in that:

It doesn't advance the turn counter
It doesn't follow the normal infection rate (I think it's always 9 but I'm not 100% sure?)

My plan is never to enforce infection limits. (The new card we got prevents you from drawing further infections). I also think we can keep the -1 hack and only show you the infection screen if your turn is there.

I also think we can keep the -1 hack and only show you the infection screen if your turn is there.

Fine with me if you're comfortable with that change.

My plan is never to enforce infection limits.

Really? In that case I might want to add something to the infection confirmation prompt that warns you if you're infecting more than the infection limit (just in case you mis-tapped).

(The new card we got prevents you from drawing further infections).

Which card? You mean the hollow men gather?

Oh, I actually have another idea of how to do the -1 hack. NVM. I'll do something cleaner.
The card is the infection upgrade that prevents further draws. So maybe I will only set the upper bound then?

Oh right that. Yeah, I think we should not enforce it but we should add a warning as I described in #60


 Locally, if I built this container and run ls -l /go/pkg/mod, I see something like this:
drwxr-sr-x 3 vscode golang 4096 Oct 25 03:21 cache
drwxr-sr-x 8 vscode golang 4096 Oct 25 03:21 github.com
drwxr-sr-x 3 vscode golang 4096 Oct 25 03:21 golang.org

Notice the vscode user is properly named as the owner. Everything working as expected.
Now, If I do the very same thing w/ the GitHub Action, I see this:
drwxr-sr-x  3 1000 golang 4096 Oct 25 03:44 cache
drwxr-sr-x 22 1000 golang 4096 Oct 25 03:44 github.com
drwxr-sr-x  3 1000 golang 4096 Oct 25 03:44 golang.org

Here the username is 1000, and my runCmd to go test -v ./... results in a bunch of permission denied errors.
  go: writing go.mod cache: open<EMAIL_ADDRESS>permission denied
  go: writing go.mod cache: mkdir /go/pkg/mod/cache/download/github.com/iancoleman: permission denied
  go: writing go.mod cache: open<EMAIL_ADDRESS>permission denied
  go: writing go.mod cache: mkdir /go/pkg/mod/cache/download/github.com/antlr: permission denied
  go: writing go.mod cache: open<EMAIL_ADDRESS>permission denied
  go: writing go.mod cache: open<EMAIL_ADDRESS>permission denied
  go: writing go.mod cache: open<EMAIL_ADDRESS>permission denied
  go: writing go.mod cache: open<EMAIL_ADDRESS>permission denied
  go: writing go.mod cache: open<EMAIL_ADDRESS>permission denied
  go: writing go.mod cache: mkdir /go/pkg/mod/cache/download/github.com/ivanpirog: permission denied

I verified that the container built by this github action believes it is correctly logged in as the vscode user, but this file system seems to not be consistent with what I am producing locally when VSCode builds this container.
Am I doing something wrong? This feels like a bug, it should be otherwise fully reproducible.

Also, in the container locally vscode user is user 1000:
uid=1000(vscode) gid=1000(vscode) groups=1000(vscode),999(golang),998(nvm)

OK the problem was fixed by setting: skipContainerUserIdUpdate to true. That was just a very unclear option at first.


Naming cleanup

Fixes https://github.com/ethereum-optimism/roadmap/issues/285

Contributing Agreement

[x] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

Oh sorry didn't realize this was still a draft!


Jumbotron: Putting heading font size in variables to allow for easier overrides

This was relevant on one of my projects recently. It'd be nice to be able to change the jumbotron heading font-size with a simple variable override.
Thanks!

CC: @mdo

:shipit:


Creating multiline documentation from CLI using \n like --doc line1\nline2 doesn't work

I would like to add documentation using -D --doc arguments on multiple lines using command line.
rebot/robot help says

-D --doc documentation   Set the documentation of the top level suite.
Simple formatting is supported (e.g. bold). If the
documentation contains spaces, it must be quoted.

According to manual, simple formatting includes new line management using \n, but it does not work.
Using
-D "line1\nline2"
results in
line1\nline2
beeing displayed in the documentation instead of
line1
line2

I've tried <br> instead of \n, using an argument file, but behavior is the same.
This was tested with versions 3.2.2 and 4.0.2.
I have not tested all features of "simple formatting"
Aside from fixing what I think is a bug/limitation, having the possibility to include a file formated like the documentation section of the *** settings *** could show some usefullness.
Use case:
I'm merging different files, I would like to have in the documentation field, on separate lines (for readability) links to the individual files.

Simple formatting like --doc "*Example* doc" works in general but using \nclearly doesn't. The reason is that the code explicitly escapes` so \n becomes \\n which is then rendered like \n. Escaping like this feels odd, but because it may have a reason I don't want to change it now in RF 4.0.3. I'll assign this to RF 4.1 scope so that we have more time to investigate this.

I agree reading documentation from an external file would be handy. A simple approach would be making it possible to use --doc example.txt and reading the doc from the example.txt file if it exists. Ought to be easy to implement and should be pretty safe from backwards compatibility point of view as well. It's very unlikely someone would use just a path to an existing file as a documentation. Please submit a separate issue about this.
Notice that the terminal/shell you use may support this functionality as well. For example, with Bash you can use --doc "$( < example.txt)" and the doc will be read from example.txt. Built-in functionality that works regardless the terminal would nevertheless be handy.

All this would be perfect.
Using bash redirection is a nice trick, but we also have to take care about people who don't even understand what we are talking about ;)
Here is the truth : In a future mission I may work on MS environment, so this is also self centered.

Noticed we already have #3860 about this. Closing this issue as its duplicate.


Switch to ED25519 signing scheme

see #137, part of #127

[x] import keys from cryptoconditions
[x] switch to ED25519

Note: all crypto tests are covered by cryptoconditions
Question: should we be able to configure the signing scheme?

Current coverage is 82.40%

Merging #152 into develop will increase coverage by +82.40% as of 6977646

@@            develop   #152   diff @@
======================================
  Files             0     19    +19
  Stmts             0    858   +858
  Branches          0      0
  Methods           0      0
======================================
+ Hit               0    707   +707
  Partial           0      0
- Missed            0    151   +151

Review entire Coverage Diff as of 6977646

Powered by Codecov. Updated on successful CI builds.

I guess for now we should just stick to one signing scheme

Note that this is a breaking change in the sense that the keypairs need to be regenerated
@vrde any feedback?

:+1:

@diminator Please update the section of the docs where the signature algorithm is stated, and give a link or two to references on Ed25519.
That's in /docs/source/cryptography.md
http://bigchaindb.readthedocs.org/en/develop/cryptography.html


Add support in croutoncycle for multiple displays

Under xiwi copy and paste wasn't working properly because
the second display was showing up as aura_root_x as well as the
normal aura_root_0.  This makes a few changes necessary to handle
any aura_root_ root window as type cros.

I think this is obsolete, since xephyr has been removed.


Create a folder and .jsx files on users dashboard

create a folder, name it users and then add file login.jsx, dashboard.jsx, downloads.jsx, sent.jsx, wishlist.jsx, compare.jsx, classified.jsx, auction.jsx, conversation.jsx, wallet.jsx, earning.jsx, affliate.jsx, support.jsx, manage.jsx.

I'd like to work on this issue.

Great you can go ahead and work on this

id like to work on the dashboard

@ShedrachJonah11  can you pls give us progress feedback on the issue assigned to you. Thank you

id like to work on the dashboard

Thank you for your interest can you pick another issue?

Please check your WhatsApp
On Wed, Nov 9, 2022 at 11:59 AM Kelvin Moses @.***>
wrote:

id like to work on the dashboard
Thank you for your interest can you pick another issue?
—
Reply to this email directly, view it on GitHub
https://github.com/africommerce/africommerce-saas/issues/25#issuecomment-1308575646,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVASJ7QVHSWID7MGSJZUFUTWHN7ZPANCNFSM6AAAAAARLY7674
.
You are receiving this because you authored the thread.Message ID:
@.***>

@ShedrachJonah11  Thank you Sheddy for the help.


update wakuv2 fleet DNS discovery enrtree

We regenerated the enrtree DNS records and need to use the new ones.
@vpavlin, @andy108369, @shimpa1, can you please have a look at this small PR?

LGTM, we will need @andy108369 (or someone from the Akash team) to merge it:)

@andy108369 please help with merging


<EMAIL_ADDRESS>error typed as Unknown

What version of Elysia.JS is running?
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>What platform is your computer?
No response
What steps can reproduce the bug?
Taken the example from the beta docs (https://beta.elysiajs.com/eden/treaty/response.html)
import { treaty } from "@elysiajs/eden";
import { Elysia, t } from "elysia";

const app = new Elysia()
  .post(
    "/user",
    ({ body: { name }, error }) => {
      if (name === "Otto") return error(400, "Bad Request");

      return name;
    },
    {
      body: t.Object({
        name: t.String(),
      }),
    },
  )
  .listen(3000);

const api = treaty<typeof app>("localhost:3000");

// NOTE: async was not in docs but does not affect the error
const submit = async (name: string) => {
  const { data, error } = await api.user.post({
    name,
  });

  // type: string | null
  console.log(data);

// BUG: error is typed as unknown
  if (error)
    switch (error.status) {
      case 400:
        // Error type will be narrow down
        throw error.value;

      default:
        throw error.value;
    }

  // Once error is handle, type will be unwrapped
  // type: string
  return data;
};

What is the expected behavior?
The error variable from eden treaty to be typed.
What do you see instead?
The error variable in submit is typed as unknown.
Additional information
I am not sure if the bug is in elysia or eden.
Edit: Updated to<EMAIL_ADDRESS>and still error is unknown so maybe this is an eden problem?

Seems to be fixed using both<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Thanks!


Don't support for react-router-doom v6

This middleware can't help for react-router v6. Please fix it.

This middleware can't help for react-router v6. Please fix it.

https://github.com/supasate/connected-react-router/issues/543

By the looks of it this package is dead in the water, and it doesn't seem it might actually be updated any time soon. There are a few PR's opened, and no merged since November.
Id suggest at looking for alternatives. I know i will do the same.
PS. Not throwing shade at the owner, i know OSS can be hard sometimes.

i had resolved this issues with this npm package https://github.com/lagunovsky/redux-react-router. It's same with connected react router

Switch to redux-first-history: https://github.com/salvoravida/redux-first-history
support react-router v6 and is issues-free!

@HipSiGa it works on react v18 ?

I think currently they don't provide support in near future. If you still want to use it with react-router@v6. You can make small tweak as follows.
Instead of using the
ConnectedRouter
You can make following default react-router Router component and make changes as follows
import React, { useLayoutEffect, useRef, useState } from 'react'
import { Route, Router, Routes } from 'react-router-dom'

import { createBrowserHistory } from 'history'
import { useDispatch } from 'react-redux'
import { onLocationChanged } from 'connected-react-router'

const history = createBrowserHistory()

export default ({ children }) => {
  const isFirstRender = useRef(true)
  const dispatch = useDispatch()
  const [historyState, setHistoryState] = useState({
    action: history.action,
    location: history.location,
  })

  useLayoutEffect(() => {
    return history.listen(setHistoryState)
  }, [])

  useLayoutEffect(() => {
    if (isFirstRender.current) {
      dispatch(onLocationChanged(history.location, history.action, true))
      isFirstRender.current = false
    }
    return history.listen((arg) => {
      dispatch(onLocationChanged(arg.location, arg.action, false))
    })
  }, [dispatch])

  return (
    <Router
      // ...rest of props required
      location={historyState.location}
      action={historyState.action}
      navigator={history}
    >
      {/* here it lies <Routes> <Route> ..... */}
      {children}
    </Router>
  )
}

Rest of it is same as before.
I hope it is helpful. Thank you.


Dev branch fails compilation

Getting
invalid linkage type for global declaration
i8** @_TWPVSC14UIControlStates9Equatable5UIKit
LLVM ERROR: Broken module found, compilation aborted!

For the following files
The following build commands failed:
        CompileSwiftSources normal arm64 com.apple.xcode.tools.swift.compiler
        CompileSwift normal arm64 /Users/akshit/ios-app/Pods/ChattoAdditions/ChattoAdditions/Source/Input/Photos/PhotosChatInputItem.swift
        CompileSwift normal arm64 /Users/akshit/ios-app/Pods/ChattoAdditions/ChattoAdditions/Source/Input/Text/TextChatInputItem.swift

It's failing with my project but succeeds in ChattoApp. Not sure what more information about my project would be useful to debug the issue. Thanks!
Master branch compiles.

This seems related to https://github.com/badoo/Chatto/issues/187
Can you provide?

Xcode version
Cocoapods/Carthage version
Swift optimization level

Xcode Version 7.3.1 (7D1014)
pod --version :  0.39.0
None [-Onone]

can't reproduce, can you share a project where this happens?

I wish I could @diegosanchezr.
The workaround in https://github.com/badoo/Chatto/issues/187#issuecomment-236364997 you suggested helped.


Getting error while starting the bot

I scanned qr code and deployed in docker.But getting the below error while starting the bot

See https://github.com/yagop/node-telegram-bot-api/issues/319. at node:internal/modules/cjs/loader:1101:14
undefined:1
undefined
^

SyntaxError: Unexpected token u in JSON at position 0
    at JSON.parse (<anonymous>)
    at loadSettings (file:///src/utils.js:11:27)
    at file:///src/index.js:12:18
    at ModuleJob.run (node:internal/modules/esm/module_job:183:25)
    at async Loader.import (node:internal/modules/esm/loader:178:24)
    at async Object.loadESM (node:internal/process/esm_loader:68:5)
    at async handleMainPromise (node:internal/modules/run_main:63:12)

Looks like this error
https://stackoverflow.com/questions/42676564/syntaxerror-unexpected-token-c-in-json-at-position-0

at JSON.parse (<anonymous>)
at loadSettings (file:///src/utils.js:11:27)

The error indicates that the PIPES environment variable is an invalid JSON string. It should be of the format
"PIPES": {
        "Whatsapp Chat":  "telegram channel id"
    }

SyntaxError: Unexpected token u in JSON at position 0

Specifically there is a "u" at the first position of this string
Verify that your PIPES variable is a proper JSON string

><EMAIL_ADDRESS>deploy
> node server.js

Fri, 03 Sep 2021 18:46:50 GMT node-telegram-bot-api deprecated Automatic enabling of cancellation of promises is deprecated.
In the future, you will have to enable it yourself.
See https://github.com/yagop/node-telegram-bot-api/issues/319. at node:internal/modules/cjs/loader:1101:14
undefined:1
undefined
^

SyntaxError: Unexpected token u in JSON at position 0
    at JSON.parse (<anonymous>)
    at loadSettings (file:///src/utils.js:11:27)
    at file:///src/index.js:12:18
    at ModuleJob.run (node:internal/modules/esm/module_job:183:25)
    at async Loader.import (node:internal/modules/esm/loader:178:24)
    at async Object.loadESM (node:internal/process/esm_loader:68:5)
    at async handleMainPromise (node:internal/modules/run_main:63:12)

my example conf
{
    "TOKEN": "1061343725:AAFEPEXkLrDmrPbDUZIYDlhmwBlpJPNKQhQ",
    "PIPES": {
        "Tg File Recieve":  "-597651420"
    }
}

First of all the token is sensitive, don't post it publicly. I've removed it.
Are you using settings.json or environment variables. It seems to me you're using settings.json file
If you're using settings.json, it's not being detected, that's why the program tries to read variables from the environment. Your file's name has to be settings.json and it has to be included in your docker image for this to work.
The JSON looks good to me, verify that you're naming your conf file settings.json and it's included in the docker image.

First of all the token is sensitive, don't post it publicly. I've removed it.
Are you using settings.json or environment variables. It seems to me you're using settings.json file
If you're using settings.json, it's not being detected, that's why the program tries to read variables from the environment. Your file's name has to be settings.json and it has to be included in your docker image for this to work.
The JSON looks good to me, verify that you're naming your conf file settings.json, its in the same directory as index.js, and it's included in the docker image.

Thank you.I checked with whatsapp and whatsapp beta versions.
In whatsapp bridge is working. we can send texts in bothways.when i send an image it onl

First of all the token is sensitive, don't post it publicly. I've removed it.
Are you using settings.json or environment variables. It seems to me you're using settings.json file
If you're using settings.json, it's not being detected, that's why the program tries to read variables from the environment. Your file's name has to be settings.json and it has to be included in your docker image for this to work.
The JSON looks good to me, verify that you're naming your conf file settings.json, its in the same directory as index.js, and it's included in the docker image.

As per my tests,Im concluding these about whatsapp beta-multi device feature

Qr code token never stores,2 or 3 qr codes appearing on npm start.After scanning second or 3rd qr code phone is successfully get connecting.

Session didnt storing.So when we ran using docker it asks again qr codes to scan,same like in the first step

When we did everything we will get connetion succed message in the telegram channel

All messages sending from Whatsapp are recieving in to telegram. But not recieving any single message from telegram to Whatsapp.

These are the issue which i can identify.Please fix it

I already mentioned this issue on venom
https://github.com/orkestral/venom/issues/1116

Qr code token never stores,2 or 3 qr codes appearing on npm start.After scanning second or 3rd qr code phone is successfully get connecting.

I think that's a whatsapp multi device beta issue, as I tested this now and it's working on the stable version.

Session didn't storing.So when we ran using docker it asks again qr codes to scan,same like in the first step

Is a file session data.json being created inside tokens folder. It stores your session info. If it's being created and still you're being asked to login, again it's a whatsapp muti device beta issue.

All messages sending from Whatsapp are recieving in to telegram. But not recieving any single message or media from telegram to Whatsapp.

This is not implemented, as the account used for whatsapp can be banned, which is problematic because I can't even test without risk getting banned.
When this was created was to aid the switch to telegram from whatsapp, this was never meant to be a long term solution.
I have an extra sim, When I get time I'll work on the telegram to whatsapp side too.
Regarding the multi device beta issue, as soon as I get access to that feature I will work on the fixes.


Automate NuGet publish

The NuGet packages should be updated when code is pushed into the master branch.
Edit: It seems like github has this feature now with this workflow: https://github.com/rohith/publish-nuget

Wait until I was able to test this on my other library.


remove crash reason from metadata

Fixes #11

Hello, @benwilson512! This is your first Pull Request that will be reviewed by Ebert, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.

Coverage remained the same at 78.74% when pulling 712452fbd6f2d6b25648bdd9504df3eaee9c469c on benwilson512:master into 3c3c2731772d1d937a61ec8d3d09d9288f4fac8f on Nebo15:master.

Never mind, looks like there's already https://github.com/Nebo15/logger_json/pull/11/files


Values and Variables

Trello card
Implement the technical mechanism adding variables for all currency values on the GIT site through a single centralised config file
Context
Provide a centralised store of important values/dates, which can be maintained in one place
Changes proposed in this pull request
Guidance to review

My test did the following in a review app:

This rendered as:

LGTM 👍

Needs a tech review before merging


removed deprecated method createJSModules

fixes #8
See: https://github.com/facebook/react-native/commit/ce6fb337a146e6f261f2afb564aa19363774a7a8

Thank you!


upload media, make clear it should only be image and pdf

If we're only allowing kaltura for audio and video source, it does not state this on the form under the "upload media" tab

User message added to indicate that only images and pdfs can be uploaded


Correct controller in steam vr home, but not in game (eg the wizards)?

I have been looking for a solution with this issue:
Steam vr home will show the correct quest/rift s controller.
But once I start the wizards (game on steam), the controllers are messed up.
To move or rotate around my position I have to click the joystick in that direction. Any fix out there?
Not all games have this issue.

Try to switch to Vive or Valve Index emulation mode for the controllers

Thank you for the response, I'll try this out.


[Feature Implimented] Fixing Bug in Labels Card Functionality and Implimentation of Labels Page

Pull Request Template
Description:
This pull request resolves a bug in the labels card functionality, fixing the issue where clicking on it opens the wrong page (avatar_page) #217 . Additionally, it includes the implementation of the missing labels page.
Changes Made:

Updated the function call in the labels card to point to the correct page.
Implemented the labels functionality to display the appropriate labels for each item.

Related Issue:
#217
Screenshots/GIFs:
**Updated Function Call : **

Light Mode :

Dark Mode :

Checklist:

[✅] The code follows the project's coding style guidelines.

Please review and merge this PR at your earliest convenience.
Thank you!

@Debajyoti14 No third-party libraries are used. Is this correct, or do I need to make any changes?

@Rhishikesh12 Fix the conflicts and make the PR again

The assignment of this issue should be classified as Level 2, taking into consideration the implementation of a new feature. I implemented the Labels page that was initially missing, so I expected my issue #217 to be assigned a Level 2 tag, considering the added functionality.

Please have a look at tagged message and Solved the Merge Conflicts, Now you can merge.


Migrate from probe-run to probe-rs

Since probe-run is now depreciated https://ferrous-systems.com/blog/probe-run-deprecation/

Solves #100

Anyone able to look at this?

@usbalbin this one is ready to go as well.


Travis "cron" builds run tests with "*" dependencies

"cron" builds will trigger cargo test with all dependency versions
replaced with "*" to check for incompatible dependencies.
Also cron builds will never publish gh-pages
This is somewhat hacky so I am open to suggestions. I guess that I would prefer to remove Cargo.lock altogether and replace dependency versions in Cargo.toml in the repo instead of modifying it on the fly but I see no other possibility to make dependency check only in Travis nightly.
reolves https://github.com/brson/rust-cookbook/issues/89
related to https://github.com/brson/rust-cookbook/issues/75

Thanks! We need rm Cargo.lock as well right? Otherwise it will use the locked dependencies.
The reason I prefer this approach is that we need Travis to play two roles:

Tell us whether PRs are okay to merge
Tell us when code is out of date

If we remove Cargo.lock and put "*" in Cargo.toml, it can no longer fulfill the first one reliably. An outdated example would prevent unrelated PRs from building green.

Done. Honestly I was sure cargo clean removes Cargo.lock but never bothered to check.


xlpython.log file is locked

Hi Eric,
I'm running two separate excel sessions in the same folder, which both use the excelpython functionality. I'm finding that when I try to run some of the python functionality from either session, I have to end the pythonw.exe process or I get a "could not open .....xlpython.log for redirection" error.
Is there a way to avoid this?
Thanks,
Charlie

Please check out xlwings, this shouldn't be an issue any more.


Add tracks legend to the schedule page

resolves #775

Please check the travis build is failing. Thank you.

First of all - it has nothing to do with my delivery.
But I looked into Travis logs anyway - and it looks like someone broke the JSON files that we are using for testing. Specifically the test fails while parsing https://raw.githubusercontent.com/fossasia/open-event/master/sample/FOSSASIA16/speakers - JSON parser throws SyntaxError.
If I load the URL to https://jsonformatter.curiousconcept.com/ I see that the syntax is in fact broken - e.g. at line 1582 the comma is missing after facebook URL:
    {
        "id": 65,
        "name": "Sahil Bajaj",
        "email"<EMAIL_ADDRESS>
        "mobile": null,
        "photo": "/images/speakers/SahilBajaj_65.jpg",
        "organisation": "GitHub/Homebrew/Grumpy Gits",
        "position": null,
        "country": "Scotland",
        "short_biography": "",
        "long_biography": "Sahil is a developer based in Singapore.",
        "website": "http://mikemcquaid.com",
        "twitter": "https://twitter.com/MikeMcQuaid",
        "facebook": "https://www.facebook.com/sahil2912"
        "linkedin": "",
        "sessions": [
            {
                "id": 87,
                "title": "GitHub Workshop"
            }
        ]
    },

Any idea on where these JSONs come from and who can re-generate them?

@alexagr the json that we use can be found in github.com/fossasia/open-event, so have a look there and send a PR if any of the json you find is broken.

@the-dagger just issued PR for open-event - please merge

Merged.

Current coverage is 42.54% (diff: 100%)

Merging #776 into development will not change coverage

@@           development       #776   diff @@
=============================================
  Files                6          6
  Lines              731        731
  Methods             73         73
  Messages             0          0
  Branches           118        118
=============================================
  Hits               311        311
  Misses             420        420
  Partials             0          0

Powered by Codecov. Last update 7146810...1bb0ffa

looks like all is good now

@alexagr Yes, you are right. Thanks for the fixes. Though I do not exactly understand why it did not work. The missing comma was there for some time and it worked before. I also re-added the photo you took out now and it is also working. Thanks for looking into those details!

Both bugs were introduced by commit #340b25 on September 1.
It removed the comma after "facebook" element for "Mike McQuaid, Sahil Bajaj" (id=65) and removed "photo" element for "Herr Flupke" (id=155).

@alexagr Thanks, fixed it. Herr Flupke is back :-) http://fossasia.github.io/open-event-webapp/speakers.html


DynamoDBContext.LoadAsyn: System.InvalidOperationException: Cannot instantiate type System.String[]

I am trying to retrieve from DynamoDB table, this was working fine last week.
when I call LoadAsync I receive System.InvalidOperationException: Cannot instantiate type System.String[].  Please see the sample code and the exception below.
Code Sample:
public class DataContext<T> : DynamoDBContext
{
    private DynamoDBOperationConfig _config;
    private IAmazonDynamoDB _client;

    public DynamoDbContext(IAmazonDynamoDB client, string tableName): base(client)
    {
        _config = new DynamoDBOperationConfig()
        {
            OverrideTableName = tableName,
        };
        _client = client;
    }

    public async Task<T> GetByIdAsync(string id)
    {
        return await base.LoadAsync<T>(id, _config);
    }
}

Table Structure:
public class Item
{
    [DynamoDBHashKey]
    public string Id { get; set; }

    [DynamoDBProperty]
    public string[] Tags { get; set; }

    [DynamoDBProperty]
    public string[] SearchKeys { get; set; }
}

Sample Item:
{
  "Id": {
    "S": "880bd759-99b3-4bf3-af05-029ef1c3999e"
  },
  "SearchKeys": {
    "SS": [
      "key1",
      "key2"
    ]
  },
  "Tags": {
    "SS": [
      "tag1",
      "tag2"
    ]
  }
}

Current Behavior
{System.InvalidOperationException: Cannot instantiate type System.String[]
at Amazon.DynamoDBv2.DataModel.Utils.InstantiateHelper(Type objectType, ITypeInfo[][] validConstructorInputs, Object[] optionalInput) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Services\DynamoDBv2\Custom\DataModel\Utils.cs:line 260
at Amazon.DynamoDBv2.DataModel.Utils.ItemsToCollection(Type targetType, IEnumerable`1 items, Object& result) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Services\DynamoDBv2\Custom\DataModel\Utils.cs:line 129
at Amazon.DynamoDBv2.Converter.TryFromEntry(DynamoDBEntry entry, Type targetType, Object& value) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Services\DynamoDBv2\Custom\Conversion\DynamoDBEntryConversion.cs:line 532
at Amazon.DynamoDBv2.Converter.FromEntry(DynamoDBEntry entry, Type targetType) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Services\DynamoDBv2\Custom\Conversion\DynamoDBEntryConversion.cs:line 508
at Amazon.DynamoDBv2.DataModel.DynamoDBContext.FromDynamoDBEntry(SimplePropertyStorage propertyStorage, DynamoDBEntry entry, DynamoDBFlatConfig flatConfig) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Services\DynamoDBv2\Custom\DataModel\ContextInternal.cs:line 335
at Amazon.DynamoDBv2.DataModel.DynamoDBContext.PopulateInstance(ItemStorage storage, Object instance, DynamoDBFlatConfig flatConfig) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Services\DynamoDBv2\Custom\DataModel\ContextInternal.cs:line 248
at Amazon.DynamoDBv2.DataModel.DynamoDBContext.DocumentToObject(Type objectType, ItemStorage storage, DynamoDBFlatConfig flatConfig) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Services\DynamoDBv2\Custom\DataModel\ContextInternal.cs:line 229
at Amazon.DynamoDBv2.DataModel.DynamoDBContext.DocumentToObject[T](ItemStorage storage, DynamoDBFlatConfig flatConfig) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Services\DynamoDBv2\Custom\DataModel\ContextInternal.cs:line 218
at Amazon.DynamoDBv2.DataModel.DynamoDBContext.LoadHelperAsync[T](Key key, DynamoDBFlatConfig flatConfig, ItemStorageConfig storageConfig, CancellationToken cancellationToken)
.NET Core Info
Output of dotnet --version:
2.1.403
Output of dotnet --info:
.NET Core SDK (reflecting any global.json):
Version:   2.1.403
Commit:    04e15494b6
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.16299
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.1.403\
Host (useful for support):
Version: 2.1.5
Commit:  290303f510
.NET Core SDKs installed:
1.0.0-preview2-003131 [C:\Program Files\dotnet\sdk]
2.1.202 [C:\Program Files\dotnet\sdk]
2.1.402 [C:\Program Files\dotnet\sdk]
2.1.403 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 1.0.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

BTW, I changed string[] to List and it worked.  Not sure what changed but I have used string[] before and it was working until a recent update.

What version of the AWS SDK are you currently using and what were you using before?

Are you still hoping to get support on this?

It will be nice if I don't have to change every string[] to List.
It's a breaking change and I believe it will be useful to fix it.

@salomari and @napalmdaz I haven't been able to identify when string[] might have been accepted, so I will be closing out this issue after a few days.  Other team members were confused about this too because the relevant code hasn't had changes for a while.  If you can identify the version of the SDK that you think was working, please let us know.
You should be able to use lists and sets.

I'm closing this for now.  Please reopen or comment if you can find an example version where string[] was working.


fix: LEAP-515: Fix CommentForm for old UI

annotationStore was not passed to Comments panel there
PR fulfills these requirements

[ ] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[ ] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[ ] Frontend

Describe the reason for change
(link to issue, supportive screenshots etc.)
What does this fix?
(if this is a bug fix)
What is the new behavior?
(if this is a breaking or feature change)
What is the current behavior?
(if this is a breaking or feature change)
What libraries were added/updated?
(list all with version changes)
Does this change affect performance?
(if so describe the impacts positive or negative)
Does this change affect security?
(if so describe the impacts positive or negative)
What alternative approaches were there?
(briefly list any if applicable)
What feature flags were used to cover this change?
(briefly list any if applicable)
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[ ] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Which logical domain(s) does this change affect?
(for bug fixes/features, be as precise as possible. ex. Authentication, Annotation History, Review Stream etc.)

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (28c0e12) 68.31% compared to head (5cb5557) 64.56%.
Report is 3 commits behind head on master.

Files
Patch %
Lines

src/components/App/Grid.js
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1661      +/-   ##
==========================================
- Coverage   68.31%   64.56%   -3.75%
==========================================
  Files         443      443
  Lines       28699    28700       +1
  Branches     7633     7520     -113
==========================================
- Hits        19606    18531    -1075
- Misses       7841    10169    +2328
+ Partials     1252        0    -1252

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


Allow custom error handling on input binding error

I'm using an HTTP-triggered function like this:
public static IActionResult GetAllAvailabilityOnDate(
	[HttpTrigger(AuthorizationLevel.Anonymous, "get", Route = "availability/{datefilter:regex(^\\d{{4}}-\\d{{2}}-\\d{{2}}$)}")] HttpRequest req,
	[Sql("SELECT * FROM dbo.Availability a " +
	"WHERE a.Date = @DateFilter ",
	CommandType = CommandType.Text,
	Parameters = "@DateFilter={datefilter}",
	ConnectionStringSetting = "DatabaseConnectionString")] IEnumerable<MyResult> res,
	string dateFilter,
	ILogger log)
{

This works all great - until a user provides some bogus input like "0000-00-00". In this case the Function errors out with
System.Private.CoreLib: Exception while executing function: GetAllAvailabilityOnDate. Microsoft.Azure.WebJobs.Host: Exception binding parameter 'res'. Core Microsoft SqlClient Data Provider: Conversion failed when converting date and/or time from character string.

The Function immediately returns a 500 result. However, I would like to be able to insert some custom error handling and for example return a 400-Bad Request instead.
Is this something that's already somehow possible? If not, I would like to make this an enhancement request :)

We're facing a similar problem, where we used a Service Bus message property to perform a query. Workarounds exist: you could create another utility function that takes care of all validations and then forwards the message (either through HTTP or another service bus message), but this is highly inconvenient.
More control over bindings would be welcome.

We're facing a similar problem, where we used a Service Bus message property to perform a query. Workarounds exist: you could create another utility function that takes care of all validations and then forwards the message (either through HTTP or another service bus message), but this is highly inconvenient.
More control over bindings would be welcome.


Module Rename

Work has been staged for a module rename to fall into accordance with module naming standards as communicated with Issue #1.  This brings up a challenge:
The PowerShell Gallery works by module name.  A module rename means a new gallery entry.  Microsoft recommends uploading under the new name and then unlisting the versions under the old name.  There is no method of redirection available.  Removing the listings has more to do with trying to get people onto the latest supported version instead of the older code than anything.
I worry a little about abandoning people that may rely upon updating via the console only.  I have plenty of modules where I only update when I see a new version with Find-Module.  To aid on this topic I've considered an update under the old name that would stay listed on the Gallery (for a period of time?) that would add a message to the console advertising the new name/version.
Any thoughts on those steps?

@Rick-2CA  I vote we do a final version of PSServiceNow which throws a write-warning at the beginning of each cmdlet advising that it has been renamed and that users should look to the new module for the latest updates.

That'll work, thank you.
Good luck to you guys with today's events.  Hope everyone around you is safe.

Added a new branch today to do the last update of the current version.  I don't intend on merging it into master, but rather publishing and abandoning the code.  It won't be published until the new version is ready to go so the message can be finalized.
Along with the warning message in the main psm1 I added a line to Set-ServiceNowAuth that is set to $null.  For some reason that one function wasn't triggering the warning, but making it call another function in the main psm1 consistently triggered it.  Mystery to me if anyone knows why it might behave like that.  No other function in the same psm1 behaved that way.

@Sam-Martin - Trying to step closer to making this happen...
I'm proposing calling the current dev version 2.0 and merging it into master.
Once there I'll publish under the new name to the PSGallery and set you as an owner.
I'll then publish the last update of v1 to the PSGallery and hide that option from the gallery search.
Do we want to address any of this in the readme.md?  Do we need a link to "Why there's a version 2" with a summary and the short list of changes that were completed?
Anything else I ought to do that I'm overlooking?

@Rick-2CA that sounds good to me. I would suggest addressing the psservicenow vs servicenow question as a top level item and mentioning v2.0 in that.
Many thanks,
Sam

ReadMe updated in dev branch.  Will make a second pass and likely merge things this week and then work on the PSGallery deployments sometime after that.  I hope...

@Sam-Martin - Was chatting about versioning with a gentleman in the PowerShell Slack channel.  Any sense in taking the current v0.1.13 (published) version and calling the next version a v1 instead of my earlier proposed v2?  The majority of changes are back end and thus don't break the module.  That leaves feature additions and the rename.  The rename isn't breaking the old stuff either, it's just a fresh start with a new name.  What say you?

Hey @Rick-2CA . v1 sounds fine to me! So long as the major number in the semantic versioning gets incremented, I don't mind 😄


Is there a possibility to hide some routes?

Hiding some routes would be great, is there a way to do this?
BTW thank you for this package, it's great!

Nevermind, I hadn't looked at the config file... Sorry ;)

cool


build: adopt black codestyle

On top of #121

Thanks for letting me know about the trailing , trick, it makes black much more usable for me! I went ahead and applied in some other places I though it made sense.

Thanks for letting me know about the trailing , trick,

Yeah, adding them also means that when you add a new entry you only need to change the newline, so one-line diff, instead of the usual 2 (add coma, plus new line content).

FWIW I support anything that makes it so that I can type my code in any syntactically valid way and have it automatically corrected to be in the project style. black or some variation thereof is perfectly acceptable to me.
I personally don't love the 127 character line length, because I generally code with 2 editors / terminal windows side-by-side, which is only ~100 characters with a reasonable font size, and it makes super wide lines annoyingly long. I'd rather either leave the default value or use 80.
That said, it doesn't really matter that much to me.


Debugger display variables in alternate formats

VSCode Version: 1.11.x
OS Version: All

It would be nice if variables viewed in the debugger could display their values in different formats like hex, octal, or binary.

Duplicate of #17668


HeliosClient: Separate sending of requests from creation

@davidxia @rohansingh @mattnworb

+1 for moving the low-level http stuff out of HeliosClient.
With that stuff moved into a RequestDirector, seems like adding a HeliosClient unit test would be a lot easier now :wink:

:+1:


 Currently it always goes to /dev/null

Fixed. I've also removed most README --verbose examples


Change nuspecs to include .ttf in uwp package

This fixes #13 by including the .ttf files in lib\UAP10\Plugin.Iconize.FontAwesome.UWP\Assets\Fonts

Thanks!


[FEATURE REQUEST] cds v8 support

Description
cds v8 has been officially released.
Some notable changes:

@cap-js/cds-types types are no longer re-exported by @sap/cds. The package has to be installed separately.
New odata adapter
ieee754compatible option to always output decimal and int64 data as strings.
express.js is now optional peer dependency

Full Changelog
Deprecated Features
Suggested solution
Release new major version with support for cds v8.

Hi Theo Sun, Are you planning to upgrade your dirver to make it V8 compatible, or not?


Bug from refractoring: max() arg is an empty sequence

....Traceback (most recent call last):
  File "/home/eva/Github/Objective-Revision-Evaluation-Service/ores/features_reverted.py", line 94, in run
    print('\t'.join(str(v) for v in (list(values) + [reverted])))
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 34, in solve_many
    value, cache, history = _solve(dependent, cache)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 95, in _solve
    value = dependent(*values)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/features/feature.py", line 31, in __call__
    value = super().__call__(*args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 20, in __call__
    return self.process(*args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/features/diff.py", line 137, in process_longest_repeated_char_added
    for segment in diff_added_segments
ValueError: max() arg is an empty sequence

Resolved by related pull request.


Fix whitespace removal with escaped joining characters

Fixes https://github.com/jakubpawlowicz/clean-css/issues/735.

:+1: well done @silverwind, merging!


can we use journal?

Quite often my app closes without being able to close the db.  This leaves the db unusable but there is a journal file.  Is there a way to repair the db on the next open using the journal?

Unfortunately, UnQLite does not provide such a function.
(Because it possibly leading to database corruption.)


Add CI fixes to the "docs" branch

Package resolution is handled manually in CI - this assures packages are installed from the git checkout.  The solution was to add an ins toml-doc line to the script used to install the packages to be tested.
I also removed a reference to json, to the best of my knowledge this is shipped in base.

See also #8 .  This would help disambiguate issues such as buggy info.rktd files from  test failures.


Job names should include the image type

https://github.com/YannickRe/azuredevops-buildagents/issues/81

I'm sorry, this took me a while as I had to make the requested changes myself.
Finally got around to it, and now I could merge.

Thanks a lot. I got busy with something else too and it just slipped my mind.


Mount all ServiceAccount imagePullSecrets to allow builds to read the run image

The build's ServiceAccount's imagePullSecrets will be used to pull the builder image but, are unavailable to read the run image which is likely in the same registry.
This also supports multiple builder image pull secrets configured directly on the build resource.
Mounts the imagePullSecrets inside the rebase build pod.
Refactor BuildPod interface to take a "BuildContext"

Resolves: https://github.com/pivotal/kpack/issues/858

Codecov Report

Merging #865 (56f3e9b) into main (7eff976) will increase coverage by 0.27%.
The diff coverage is 97.14%.

@@            Coverage Diff             @@
##             main     #865      +/-   ##
==========================================
+ Coverage   67.68%   67.96%   +0.27%
==========================================
  Files         113      113
  Lines        5019     5044      +25
==========================================
+ Hits         3397     3428      +31
+ Misses       1259     1254       -5
+ Partials      363      362       -1

Impacted Files
Coverage Δ

pkg/buildpod/generator.go
66.15% <87.50%> (+2.82%)
:arrow_up:

pkg/apis/build/v1alpha2/build_pod.go
97.78% <100.00%> (+1.20%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7eff976...56f3e9b. Read the comment docs.


Fix dependency

This has been merged and fixed in PR #62

This seems to be a duplicate of https://github.com/public-awesome/ics721/pull/64/files

Fixed in #67


Clarify whether the application should allow users to paste numbers in scientific notation when using Standard mode or not

Describe the bug
It is not clear if the calculator should allow users to paste expressions with scientific notations (E notation, for example 12e+10) when the calculator is in Standard mode.
Some unit tests and methods show that we should support it:
https://github.com/microsoft/calculator/blob/41e2e975918c6838947e29a046778bd4d57feeee/src/CalculatorUnitTests/CopyPasteManagerTest.cpp#L133
https://github.com/microsoft/calculator/blob/41e2e975918c6838947e29a046778bd4d57feeee/src/CalcViewModel/Common/CopyPasteManager.cpp#L197
while other methods like CopyPasteManager::ExpressionRegExMatch don't accept them when the calculator is in Standard mode, making it impossible to paste a number in scientific notation.
Why we should allow it

the standard mode can display number with this notation, for example if they calculate<PHONE_NUMBER>99*999999999999, so we should probably allow users to copy the result and paste it later.

Why we should not allow it

This mode doesn't have a exp button, meaning the only way to enter a number in scientific mode will be via copy/paste (and via the history).

Steps To Reproduce

launch the Calculator
paste 12e+10
the calculator will display 'INVALID INPUT'

Expected behavior
Currently unknown
Device and Application Information**

OS Build: 10.0.18363.0
Architecture: X64
Application Version: 10.1907.24.0
Region: en-US
Dev Version Installed: True

Requested Assignment
If possible, I would like to fix this.
Bonus
If we allow users to do it, we should also decide if we want to allow them to copy these type of numbers with unit converters.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Despite not having an exp button, I don't think we should be blocking folks unnecessarily from pasting in a valid number, so I think we should support pasting these values in both standard mode and in most converters.
Currency converter is an odd one, though, since that is not a domain where scientific notation is typically used. Are there other converters where scientific notation might not be appropriate?

I agree, we should not block users. I will fix ExpressionRegExMatch to allow these numbers asap.
For converters, I agree, I think we should allow it. I would probably never use scientific notation with the currency converter, but if someone wants to use this notation (💰), it's probably better to allow it and not treat this converter differently.

I did some tests to assess the workload necessary to support:

scientific notations in Standard mode
scientific notations in Unit/Currency converters

Standard mode:
StandardCalculatorViewModel + CalculatorManager already supports this type of numbers, we simply need to modify some regular expressions in CopyPasteManager::ExpressionRegExMatch to support these numbers.
Unit Converter
UnitConverterViewModel and UnitConverter don't support scientific notations. If we want to do it we need:

Add a new command UnitConversionManager::Command::Exp
modify UnitConverter::SendCommand to support exponent.
because the "paste" function creates a sequence of commands, we need to modify many UnitConverter's functions to ignore errors at the moment UnitConverterViewModel sends the command Exp (when users paste 12e21 for example, for a few nano seconds (because commands are sent 1 by 1 (probably something we need to change too)) the number will incorrect (12e) and crashing the application. We need to add some logic to ignore this temporary state.
we need to modify UnitConverter to correctly support backspace with scientific notation.

44e+11 -> 44e+1 -> 44e+0 -> 44, the current implementation is very basic and doesn't support this scenario.

@grochocki  if you are ok with that, I would recommend to fix the issue for the standard mode and create another task for converters if we really want to do it.

Let's fix the issue for standard mode, but leave converters for now. I have not seen any feedback about scientific notation support in converters, so we can revisit later, if necessary.


HDXDSYS-999 Fix logging of HNO warnings and errors

They needed to be batched by country

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

9 of 9 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 93.229%

Files with Coverage Reduction
New Missed Lines
%

src/hapi/pipelines/utilities/logging_helpers.py
1
93.75%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.05%

Covered Lines:
1432

Relevant Lines:
1536

💛  - Coveralls


kprove: pass branching allowed to kore-exec when running proofs

fixes #2224

@ehildenb I don't know kprove very well but I only added this capability to the prove method. Do you think I should add this code to other functions like execute?

We really only need this functionality for proving right now, so I wouldn't bother @SchmErik . So if @radumereuta approves, the changes look good to me!


ICS Support for local calendar

We added support for ICS imports for the local calendar along with tests for the new feature.

This is group 6, we have modified the Local Calendar integration to add ICS support. This was done through adding an optional input field in which the user can paste the link. If the user elects to paste a link, we will go ahead and retrieve the ICS calendar from the link and add it as a calendar to the integration. The user is unable to modify this calendar, they can delete it the same way they would go about deleting a normal calendar.
Tasks finished from our presentation:
TID#00 Read ICS Link from USER.
TID#01 Save ICS file from the LINK.
TID#02 Parse info from ICS file to LocalCalendar.
TID#03 Present ICS calendar to the USER.
TID#05 Prevent the USER from modifying a ICS calendar.
Tasks which are not implemented (this was already out of main scope for the MVP):
TID#04 Periodically refresh ICS calendar.

This PR did not pass the checks. Please address them.

We now have no more duplication issues. The security hotspots are due to http mocking, they are not a "real" security issue. PR should now be mergable.

Also about the CI checks. We did not make any changes in the apple TV integration, where the ruff checks are failing. So not sure why thats happening.


Managing Tenant-Level Resources

Describe the bug
I'm trying to get a grasp of using Azure's API coming from AWS... I can successfully run az login to authenticate to one of my organization's Azure subscriptions then run something like az group list to list the resource-groups associated with that subscription.
I'm running into an issue where I'm trying to manage tenant-level resources.
Example:
I want to manage the secret for a particular App Registration application. I'm internally told to login to the tenant like so:
az login --tenant <insert-tenant-id> --allow-no-subscriptions.
Then when I run az account list -o table I notice that I have successfully authenticated to the tenant specific account.
Name                       CloudName    SubscriptionId    TenantId         State    IsDefault
-------------------------  -----------  ----------------  ----------       -------  -----------
SubscriptionA              AzureCloud   abc-1234<PHONE_NUMBER>0948   Enabled  False
SubscriptionB              AzureCloud   abc-9893<PHONE_NUMBER>8034   Enabled  False
N/A(tenant level account)  AzureCloud   abc-9999<PHONE_NUMBER>9482   Enabled  True

My issue/question arises when I'm trying to use the CLI to query the "tenant". I constantly receive errors like the one below.
Errors:
(SubscriptionNotFound) The subscription 'abc-9999' could not be found.
Code: SubscriptionNotFound
Message: The subscription 'abc-9999' could not be found.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Login to a tenant directory using Azure CLI ( az login --tenant <insert_tenant_id> --no-allow-subscriptions
Attempt to run anything. For example: az group list

Expected Behavior

See the resources associated with the tenant

Environment Summary
macOS-12.6-x86_64-i386-64bit, Darwin 21.6.0
Python 3.10.6
Installer: HOMEBREW

azure-cli 2.40.0

Extensions:
account 0.2.5

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

I'm mainly just looking for some guidance on how to move forward. Thanks for any input.

@jiasli for awareness


why my program have conflict with screen recorder

Hi every body. when I run my exe file and also I want record from my screen but my exe file not work any way. I test three screen recorder and all of them have same result.
my module in my program are .
import requests import json import hmac import hashlib import base64 from urllib.parse import urlencode import time import uuid import datetime import threading from websocket import create_connection import chime import os import sys import socket import base64 from Crypto.Cipher import AES from Crypto.Hash import SHA256 from Crypto import Random
I compile my code with:
nuitka --standalone --assume-yes-for-downloads --remove-output --disable-dll-dependency-cache --plugin-enable=pylint-warnings --plugin-enable=numpy --plugin-enable=pkg-resources python_file.py
when I test my program(exe) without screen recorder it work good and not have any problem. I test two way:
first run exe file then screen recorder and also first run screen recorder then run exe file . this two way don't solve my problem
I install nuitka with pip3 install nuitka
python --version : python 3.10.5
python -m nuitka --version :
1.2
Commercial: None
Python: 3.10.5 (tags/v3.10.5:f377153, Jun  6 2022, 16:14:13) [MSC v.1929 64 bit (AMD64)]
Flavor: Unknown
Executable: C:\Users\hossain\AppData\Local\Programs\Python\Python310\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10

So you want me to guess which import does what and come up with my own program that demonstrates the failure? Please provide a minimal program that demonstrates the issue, or else I doubt that there is anything actionable.

So you want me to guess which import does what and come up with my own program that demonstrates the failure? Please provide a minimal program that demonstrates the issue, or else I doubt that there is anything actionable.

   ` x=1
    while x==1:
        time.sleep(0.05)
        while 1:
            time.sleep(0.05)
            try:
                sp= json.load(open("entry_trader.json"))
                break
            except Exception as error:
                pass
        if (type(sp['stop_trail'])==int) and (type(sp['stopPrice_sl'])==int):
            if (sp['stop_trail']>0) or (sp['stopPrice_sl']>0):
                while 1:
                    time.sleep(0.05)
                    try:
                        json.dump({"status":1}, open('get_ok.json','w'))
                        break
                    except Exception as error:
                        pass
                self.sl_tp=sp
        for i in [*self.alive_positions]:
            self.detail_accuonts[i]['last_time']=str(datetime.datetime.utcnow())

        self.touched_sl=[*{key:value for key,value in self.alive_positions.items() if value['stopPrice_sl']>=self.price}]
        self.touched_st=[*{key:value for key,value in self.alive_positions.items() if value['stop_trail']<=self.price}]

        if (len(self.touched_sl)>0) or (self.sl_tp['stopPrice_sl']>0):

            self.touched_sl=[*self.alive_positions]
            print('touch sl')`

I have problem with this section of my code.
when I change value entry_trader and save my program get that value with json.load
then it arrive to line print('touch sl') it can't show any thing< my mean is that don't work print and my exe locked and do not any work.
I test another program like below and my exe don't have any conflict with screen recorder
``

So, seriously, screenshots of code? I will report back when I am finishing typing it up.

You may want to try 1.2.2, I believe it has a regression fix that may affect the code you posted.


Maven plugin does not longer work for jbake > 2.6.1

This is due to some refactorings within jbake source.

Thanks for the info. I currently have no need for a JBake update, so if you need this, I'd like to ask you to provide a PR. I'd be happy to merge that and push a new release.


initial setup

This PR aims to address:

Adding a userdata script for EC2 startup
Adding a CFT for quick teardowns/deployments
Adding GitHub Actions for pushing changes

Implementing #1 #2


pke

EXCUSE ME ,Do you know ''pke'' ?
Have you compared your model with it

Yes I know "pke", In the paper we used pke implementation of "Multipartite and TopicRank"


SdFat32 support

Hi chegewara,
I've added support of the Bill Greiman's SdFat on SD/SPI card. It uses SdFat32 class.
Please consider merging to your code.

Hi,
thanks for this PR. How this new class is different from sdcard class?

SD.h uses old version of SdFat (Copyright (C) 2009 by William Greiman, Copyright (c) 2010 SparkFun Electronics) and for example doesn't support long file names.
So, using SdFat directly may be preferable in some cases.
That said, I just realized that I was to quick to send the PR, and have not implemented a few features like stop/start, etc. sorry. If you think this switch to SdFat can be useful I will implement them and make another PR.

I dont mind to merge this PR and to have 2 classes, so please let me know when it is ready to merge.

I made some deep changes in MSCCallbacks class in order to be able to override even single callbacks. Also, made SDFat2USB to derive from both MSCusb and MSCCallbacks to simplify its structure.
Fixed onStop, and made some other changes, i.e. new example code to show how callbacks work.
Tried to make MSCCallbacks* defaultCallbacks static, so there is no need to save them in the overloaded MSCCallbacks, but that breaks compiling under Arduino IDE for some reason.
Now, I've been discussing issues with SdFat (older SD.h, Arduino Fork and the new v2.x) with Bill Greiman, as all version suffer from the same problem: https://github.com/greiman/SdFat/issues/349#issuecomment-1188235279 But, apparently, this is due to the SdFat (in any reincarnation) being not thread safe: https://github.com/greiman/SdFat/issues/349#issuecomment-1189061629
Setting CONFIG_DISABLE_HAL_LOCKS in ESP32-S2 cores helps to some degree, but adding fs_changed flag and also adding time delays (not shown in my example) helps even more. Unfortunately, the nature of SdFat makes it inherently unstable if the file system on SD card is used both through USB and directly on ESP32 device.
It will work for my project, as I don't necessarily need to access files both ways at the same time, but this maybe an issue for others.


working on plugin extraction

This is just a start of #112.
@aa755

Everything is now building up until PCUIC.

I am still getting segfault, as illustrated in the comment in the last commit. Is there a way to minimize the use of manually written Obj.magic. Can we generate those occurrences automatically, e.g. via Coq extraction and manually write Obj.magic free glue code?

@gmalecha  I wasn't able to understand run_extractable.ml, but is a fix like this needed at other branches as well?
https://github.com/MetaCoq/metacoq/pull/113/commits/2234271c358f039819ef21b83d310acde792d307#diff-280e8b43ca60f8ce3ad3f730d6945feeL218

The extraction of genLensN now works. The implementation needs a serious cleanup.

@mattam82. Any objections to merging this?

This is a very large PR, so just a quick summary of what happened in it.

There is a new run_extractable.ml file that provides an interpreter for the extracted version of the template monad. It currently lacks support for making inductive types but is able to quote, make definitions, evaluate, etc.
I needed to phase split some of the files so that you don't have to have TemplateCoq loaded in order to run a plugin built with TemplateCoq. This is what produced the new constr_quoter.ml, ast_quoter.ml, quoted.ml files. They are not in the greatest shape right now, but will take a little bit of work to clean up and are mostly orthogonal to the rest of the development.
I broke something about extraction and the current plugins, e.g. checker. The README.md file in plugin-demo explains how extraction works now. There is likely to be a nicer way to implement this, so any suggestions would be welcome. I ended up fixing it by dropping the -open flags since it doesn't use the mlpack file, but if there is a nice way to do that, then that would be better.

Any feedback is welcome.

Ok, it seems very good. However, I can't go in the details because it is complicated.
So if I understand well, there are two ways to run an extractable plugin: either as an old plugin with Run TemplateProgram or by extracting it and then loading it.
A few remarks:

It would be nice to simplify the build process of MetaCoq, even the repository with all the folders becomes very messy! But I don't know how to do this. I neither don't know how to simplify the build of plugin-demo.

You should document in the Readme.md at least where are the examples to look at (plugin-demo and test-suite/extractable.v if I'm correct).

template-coq/test-plugin -> should go in the test-suite directory

Are the Cd stuff really needed in the Extract.v files?

With an operation tmQuoteRecursively : term -> global_declarations we could reimplement the "Template Check" plugin as an extractable plugin.

Say me when you want that I merge. Can you rebase it on top of coq-8.8?

Thanks, @SimonBoulier! I will work on these. For the recursive quoting, I was wondering if it makes more sense to write a dependencies function that would construct a list of all of the (recursive) dependencies of a term. Then you could quote individual pieces as much as you want. Does that seem reasonable?

Ok, it seems very good. However, I can't go in the details because it is complicated.
So if I understand well, there are two ways to run an extractable plugin: either as an old plugin with Run TemplateProgram or by extracting it and then loading it.
This is correct. It seems like a useful property since it lets you debug and test in the Coq REPL and then extract when you have what you want. Ideally they would be identical code paths, but they aren't right now.
A few remarks:

It would be nice to simplify the build process of MetaCoq, even the repository with all the folders becomes very messy! But I don't know how to do this. I neither don't know how to simplify the build of plugin-demo.
I think this would be great, but I'm also not certain. I think I would need a flag that would tell the extraction plugin to not generate code for particular library.
You should document in the Readme.md at least where are the examples to look at (plugin-demo and test-suite/extractable.v if I'm correct).
template-coq/test-plugin -> should go in the test-suite directory
Are the Cd stuff really needed in the Extract.v files?
I really don't like generating ml files into the source directory. It makes it hard for me to browse code.
With an operation tmQuoteRecursively : term -> global_declarations we could reimplement the "Template Check" plugin as an extractable plugin.
Just to be a little bit more clear I opened PR #133 with my current thinking on this.

Say me when you want that I merge. Can you rebase it on top of coq-8.8?

If you do some rebasing, maybe you can do it on top of #128 so that we can merge one after the other ...

I'm really bad at rebasing, so I pushed a new PR #134 . We can merge that and close this one.

Merged in #134


Use SPIFFS for caching timezone?

Is it possible for you to use SPIFFS when caching a timezone instead of EEPROM or NVS?

Why SPIFFS over EEPROM? The preferences APIs seems like it was made just for this.


context.router.transitionTo report: Invariant Violation: Missing "xx" parameter

this.context.router.transitionTo('/yuheng/stat/so:01')   is ok, but  this.context.router.transitionTo('/yuheng/stat/so:DX') is error,  Invariant Violation: Missing "DX" parameter
I use encodeURIComponent consume 'so:DX', then is ok. So, it's a feature or bug ?

@ustccjw I added a test that passes, so I was unable to reproduce this issue. If you can get the test to fail, please do. It would help us nail down the cause of the bug, if there is one.

@mjackson is v0.13.3 ?


Deprecation warning when parsing nightly packages

Description
When running pip with -f https://storage.googleapis.com/jax-releases/jaxlib_nightly_releases.html to install nightlies, I'm getting this deprecation warning:
DEPRECATION: The HTML index page being used (https://storage.googleapis.com/jax-releases/jaxlib_nightly_releases.html) is not a proper HTML 5 document. This is in violation of PEP 503 which requires these pages to be well-formed HTML 5 documents. Please reach out to the owners of this index page, and ask them to update this index page to a valid HTML 5 document. pip 22.2 will enforce this behaviour change. Discussion can be found at https://github.com/pypa/pip/issues/10825

I'm using pip 22.0.2.
Probably best to fix whatever it is angry about soon.
What jax/jaxlib version are you using?
No response
Which accelerator(s) are you using?
No response
Additional system info
No response
NVIDIA GPU info
No response

Thanks for the report – this looks like a duplicate of #5410, but is still something we need to address.

Closing in favor of https://github.com/google/jax/issues/5410


Split Nano 33 BLE & Nano 33 BLE Sense

I think we can then remove ARDUINO_NANO33BLE from variants as well ...
This will ofcourse require a path change across the CMakelists in all the samples as well.

Just tried building both nano ble and sense and both build fine! Without any changes in any cmakelists!
So I guess this PR LGTM

Great! But let me make those changes anyway before merging 👍

Great! But let me make those changes anyway before merging

yes, otherwise the samples may break.


switches are not supported

Hello,
with the new Update 2024.12.2 in Home Assistant Core 2024.12.5 Operating System 14.1 the switches are no longer supported.
I used the following code:
`binary_sensor:

platform: rpi_gpio
sensors:

port: 14
name: "FAN_RPI"
unique_id: "Fan_sensor_port_14"
bouncetime: 80
invert_logic: false
pull_mode: "DOWN"

switch:

platform: rpi_gpio
switches:

port: 14
name: "Fan_RPI Switch"
unique_id: "fan_RPI_switch_port_14"`

Best regards
Martin

You are using the same GPIO port for both switch and sensor. How is it working for you?


Teardown vms

Add ability to tear down only workers.
4 patches:

create remove_dhcp_entries for masters and workers. This removes dhcp entries per host.
Fix dhcp leases cleaning. Some race conditions were sometimes causing cleaning issues. Note that --wait in virsh does not work.
Add support for teardown workers only.
Support non consecutive static ip for nodes. So that we can configure ip as e.g. a.b.c.5, ab.c..7 ... and proper range will be calculated.

Updated to rebase on latest main (e.g. remove duplicate patch as 1 commit in https://github.com/bn222/cluster-deployment-automation/pull/181).
Now 6 patches:
1. Cleanup: cleanup based on feedback on initial PR
2. clusterDeployer: add remove_dhcp_entries function. This removes dhcp entries per host.
3. Avoid deleting/recreating bridge when not needed. This avoids losing existing bridge configuration such as dhcp entries.
4. Cleanup previous ebtables rules before adding new ones. Avoid stale rules in ebtables
5. Add support for teardown workers only.
6. Support non consecutive static ip for nodes. So that we can configure ip as e.g. a.b.c.5, a.b.c.7 ... and proper range will be calculated.


New comments added to comment threads aren't shown

VSCode Version: 1.37.0-insider

Repro steps:

Build and run the comment sample (vscode-extension-samples/comment-sample)
Open a file, click in the gutter to start a comment thread, type some text, and click Create Note to add the first comment to the thread
With the thread still open, again enter text in the text box, and click Reply

Expected result: The second comment should be shown
Actual: The thread only shows the first comment
This can be worked around in the extension by disposing the original comment thread and creating a new one:
function replyNote(reply: vscode.CommentReply) {
    let thread = reply.thread;
    thread.dispose();

    let newComment = new NoteComment(reply.text, vscode.CommentMode.Preview, { name: 'vscode' }, thread);
    _commentController!.createCommentThread(thread.uri, thread.range, [...thread.comments, newComment])
        .collapsibleState = vscode.CommentThreadCollapsibleState.Expanded;
}

Copying @rebornix and @RMacfarlane - since i see you folks have looked at other issues in this area recently.

Hmm, @BenPryor is this the insider’s version from today? I’m having trouble reproducing right now with that and the current version of the comment sample. Can you check the dev console (F1 -> Developer: Toggle Developer Tools) to see if anything gets printed then? Otherwise my hunch would be that there is a timing issue with updating the comments.

Thanks for taking a look.
I'm on the Insiders version from 2019-07-18T14:11:10.247Z. I ran through the repro with the dev console showing, but nothing was logged.

Full version info
Version: 1.37.0-insider (user setup)
Commit: fd5985489982fe14381dd56e6f0b29c53d5f991b
Date: 2019-07-18T14:11:10.247Z
Electron: 4.2.5
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Windows_NT x64 10.0.17763

And, I just updated to the very latest Insiders build (628bdedf671d3230fc76982fae845875a6f6c0da, 2019-07-19T10:33:04.702Z) and the issue looks like it no longer repros. Going to do a bit more testing to be sure, but looks like this may have been fixed.

I pushed a fix with using correct comment id, which should fix the issue properly.