3092 improve spellcheck

improved spellcheck and consistency for spelling by adding "NodeJS and JavaScript" to "accept.txt" and others in "reject.txt"

@utkkkarshhh you need to sign the Plone Contributor Agreement to merge this pull request.
Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

How to add entry in Changelog verifier? Also i've already sent an email with Plone Contributors Agreement to<EMAIL_ADDRESS>

@utkkkarshhh for change log see https://6.docs.plone.org/volto/developer-guidelines/contributing.html#create-a-pull-request
A human will review your PCA and notify you when you are approved within a few business days. Please post a comment here when you are approved. Thank you!

@utkkkarshhh you need to sign the Plone Contributor Agreement to merge this pull request.
Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

@utkkkarshhh you need to sign the Plone Contributor Agreement to merge this pull request.
Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

I've been added as a contributor to the Plone Organisation.

Thank you @utkkkarshhh. I will merge as soon as the jobs all pass.

@utkkkarshhh if you would like to ensure that this change also gets imported into the main documentation, would you please submit a PR in https://github.com/plone/documentation?


Complex object parameter non mandatory with mandatory properties

For the Set-AzDiagnosticSetting cmdlet in Monitor. There are two array parameters.
.Notes
COMPLEX PARAMETER PROPERTIES
To create the parameters described below, construct a hash table containing the appropriate properties. For information on hash tables, run Get-Help about_Hash_Tables.

LOG <ILogSettings[]>: the list of logs settings.
  Enabled <Boolean>: a value indicating whether this log is enabled.
  RetentionPolicyDay <Int32>: the number of days for the retention in days. A value of 0 will retain the events indefinitely.
  RetentionPolicyEnabled <Boolean>: a value indicating whether the retention policy is enabled.
  [Category <String>]: Name of a Diagnostic Log category for a resource type this setting is applied to. To obtain the list of Diagnostic Log categories for a resource, first perform a GET diagnostic settings operation.

METRIC <IMetricSettings[]>: the list of metric settings.
  Enabled <Boolean>: a value indicating whether this category is enabled.
  RetentionPolicyDay <Int32>: the number of days for the retention in days. A value of 0 will retain the events indefinitely.
  RetentionPolicyEnabled <Boolean>: a value indicating whether the retention policy is enabled.
  [Category <String>]: Name of a Diagnostic Metric category for a resource type this setting is applied to. To obtain the list of Diagnostic metric categories for a resource, first perform a GET diagnostic settings operation.
  [TimeGrain <TimeSpan?>]: the timegrain of the metric in ISO8601 format.

RetentionPolicy is a non mandatory parameter of MetricSettings and LogSettings. However, it got expanded and its parameters RetentionPolicyDay and RetentionPolicyEnabled are mandatory.
Is there a way to create an IMetricSettings or ILogSettings object type that do not have both RetentionPolicyDay and RetentionPolicyEnabled? If not, that would be a problem, right?

This would require generating another copy of the parameter with both parameters marked as required. (bumping into parameter set max 32 limit)

The other work around would be to do custom cmdlets to support the different options that were needed and hide the original.

In this case, both RetentionPolicyDay and RetentionPolicyEnabled should be optional instead of mandatory. I think this issue should have been fixed.


Nuxt.js (webpack) Module parse failed: Unexpected token (1:11295)

Expected behaviour
Plyr version v3.6.5 works fine
Actual behaviour
Steps to reproduce
When use plyr v3.6.6 in nuxt,js
Environment

Browser:
Version:
Operating System:
Version:

Console errors (if any)
Module parse failed: Unexpected token (1:11295)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
Link to where the bug is happening
https://github.com/sampotts/plyr/blob/951cccb6b0da4372db05f3696a6b558587b59d12/src/js/utils/style.js#L90

I think problem is in "?"

Likely the reference to window?

I think so.

getting the same problem

Same error with Gatsby.js.

Same issue on my angular project

Anyone had a crack at a fix? I'll try conditionally accessing the window and window.CSS property.

I have the same problem

Can you test v3.6.7?

I think this is not the only issue, after the new update. I got now this error
Module parse failed: Unexpected token (1:9956)

What would help here is either:

Someone determines the token that's causing an issue. I originally assumed it was the reference to window (not available in SSR of course) but that's referenced in loads of other places.
Someone provides a reproducible example - ideally on codesandbox or similar.

@sampotts in my setup, the last patch 32444c9 have worked.

I tested and 3.6.7 works just fine.
My error with 3.6.6. points to plyr.polyfilled.mjs:4656:31, which seems to be the ? in the line mentioned in the opening post, and is now fixed.

Can you test v3.6.7?

Thanks, works fine!


Caliph could do with a better economy

I know the topic of improving certain AI's economy has been raised before but in terms of prioritizing I would suggest the first one that really could do with a boost is Caliph.
In terms of character it would fit that Caliph would be quite strong, but in AI vs AI contests, I have noticed that Caliph is let down by the fact he isn't good enough economically in my opinion. His troops are expensive and he does tend to struggle unless he is in the absolute best economic position on the map and he often takes ages to build up for the next siege (especially on 12 additional troops per siege setting)
To give him a bit more of a chance, I think he should have a few more iron mines or another stone quarry. Even a small increase in his mine/quarry amounts would make a big difference because of his -5 fear factor.

Caliph is fine as he is. He is supposed to be medium difficulty.

Can make an improvement optionally.

Caliph is fine as he is. He is supposed to be medium difficulty.

I think the problem is on maps with no oil available to build pitch rigs on, meaning it costs a lot for him to rebuild his pitch ditch if some gets burnt because of enemy sieges or enemy raiding troops. I think if possible on maps with no pitch supplies, Caliph should have one or two additional iron mines to make up for the lack of pitch supplies.

The best way to push Caliph's economy is to give him lots of Breweries and -5 fear factor. All hops that is send to the stockpile will be dublicated and each good of beer that is produced, will be multiplied again.
You can give him up to 15 Breweries if you want, alternatively you can give him weapon production.

The best way to push Caliph's economy is to give him lots of Breweries and -5 fear factor. All hops that is send to the stockpile will be dublicated and each good of beer that is produced, will be multiplied again.
You can give him up to 15 Breweries if you want, alternatively you can give him weapon production.

You're right of course, but I always found it a bit cheatsy-doodly when the AI produces lances to sell, especially arabs that don't use these buildings in general. It's way more profitable than it has any right to be.

,,I always found it a bit cheatsy-doodly when the AI produces lances to sell, especially arabs that don't use these buildings in general. It's way more profitable than it has any right to be."
I agree with that, so let him build a big number of Breweries =  https://prnt.sc/mzj3j4
(You can give him even more) Then Caliph's economy is okay.

I know you can improve AI economy through editing AIVs, but it would still be nice if Caliph was a bit better economically with his vanilla castle, especially on a map without pitch rigs.

Something interesting with Caliph is that when you give him Fletchers, he will manufacture Crossbows. With a -5 fear factor he can make crossbows pretty quickly. 1 Crossbow sells for 30 gold iirc and can give him a good boost.

To make Caliph a bit stronger, maybe he could send more slaves in sieges, and bring slaves for every siege and not just if he has to dig an enemy moat. He could try and burn enemy buildings with them and even attack walls with them. The slaves could be used as arrow fodder, as a distraction for the enemy so his other siege troops could survive longer.

This is now adjustable through AIC edits.


CVE-2024-0553 - gnutls28 Security Vulnerability found building with Jammy-Curl

Hello,
We are currently getting a high severity security warning (CVE-2024-0553 - gnutls28) from AWS Inspector when building a docker image with the jammy-curl dockerfile. We noticed the file hasn't been rebuilt in over a year. Would this cause any issues with the version of ubuntu and it's dependencies being out of date? If so, what would be the recommended course of action?
Thanks,
Brendan

buildpack-deps:jammy-curl was last updated 13 days ago, https://hub.docker.com/_/buildpack-deps/tags?page=&page_size=&ordering=&name=jammy-curl . Rebuilds are triggered for reasons other than the Dockerfile being updated. Perhaps you have an old version locally?

Ok. Maybe I could be misunderstanding. We are currently seeing versions 3.7.1-5 have the vulnerability. Based on the packages in the docker hub link provided, the package gnutls28 is running on 3.7.3-4.
According to Debian security tracker the issue should be resolved in version 3.7.9-2

Another way to come to the same conclusion:
$ docker run -it --rm --pull=always buildpack-deps:jammy-curl
jammy-curl: Pulling from library/buildpack-deps
Digest: sha256:7779a591a40bfc86721dcbc2469adc533308056e5c6e7d674b89a04af56878f6
Status: Image is up to date for buildpack-deps:jammy-curl
root@bde6ec4edbec:/# apt update -qq
All packages are up to date.
root@bde6ec4edbec:/# apt list --upgradeable
Listing... Done

(In other words, all packages inside buildpack-deps:jammy-curl are as up-to-date as they can possibly be and a rebuild won't change anything here :heart:)
For Ubuntu packages, you'll want the Ubuntu security tracker instead, which is linked from the header in Debian's: https://ubuntu.com/security/CVE-2024-0553 (fixed in 3.7.3-4ubuntu1.4, in jammy)
https://github.com/docker-library/faq#why-does-my-security-scanner-show-that-an-image-has-cves might also be helpful.


URI encoding for payment URL and QR codes are incorrect

The URI encoding for the payment url and QR code are incorrect. Rather than concatenating using the ? character, the & character should be used instead. I was able to determine this when my wallet couldn't decode the QR code and I'll be submitting a pull request shortly.

Thank you @takinbo 👍


Bugfix for rebit regionest plotting w/ fancy indexing.

The recent improvements to region estimation plotting with rebits (#84 and #86) turn out to have a bug when used from qinfer.tomography.plotting_tools. This PR fixes the bug.

Coverage increased (+0.01%) to 60.333% when pulling 24da153db0b774b1823c6de1d55d4a98dfdd4ced on fix-tomo-plot into 5c1e337923a23f7fffedb717694dab79899eaee8 on master.

As the tests pass, and as this is a minor fix, I'm merging in now.


Change marketplace VHD image naming to include architecture

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

Changed naming convention for marketplace image to include architecture to help recognize images better. The ARM marketplace image already includes architecture in the name, so only needs to be done for AMD.
Change Log

Changed naming convention for marketplace image to include architecture to help recognize images better.

Does this affect the toolchain?

YES
Test Methodology

Pipeline build id: xxxx

The arm marketplace image already includes architecture in the name, so only needs to be done for AMD.

Talked with Jon, and decided to potentially change it in 3.0 reworks


Variable naming docs

This PR takes a first stab at generating some documentation around naming. I'd love as much feedback as possible.

@lemonmade @tzvipm could I get you guys to take another look?
@lemonmade I think we can put this inside React, but will need to break this document into language-specific stuff as this repo grows. https://github.com/Shopify/web-foundation/issues/43

@cartogram  can you update this or close it? It's been here way too long :)

Closing for now


Release Localization 0.1.0

The ticket tracks the initial release of the Localization Plugin.
Release checklist

[x] Change version name to proper release versions in gradle.properties
[x] Publish artifacts to Maven Central
[x] Change version name back to -SNAPSHOT
[x] Update CHANGELOG.md's
[x] Tag releases in GitHub
[x] Close the respective plugin milestones

Post-release

[ ] Update readme files @langsmith
[ ] Update Javadoc @langsmith
[x] Update the constants.js file in android-docs repo @langsmith
[ ] Update the demo app @langsmith

/cc: @mapbox/android

Readme files update pr is here @cammace https://github.com/mapbox/mapbox-plugins-android/pull/303

Work in progress demo app localization example pr is here:  https://github.com/mapbox/mapbox-android-demo/pull/593
Because this is the initial release, there's not really a version # to update in the demo app. I'm going to check off the demo app checkbox in OP

Opened https://github.com/mapbox/android-docs/issues/341 to track adding the Javadoc to the website. Release is finished 🎉


api: upload of individual files

We should support uploading individual files to the audittrail. This helps out when manual intervention happens and could be from the request body or somewhere under ./storage/.
Similar to https://github.com/moov-io/achgateway/issues/24 but over HTTP instead of a CLI.
Examples:
POST /audittrail/:id/files

// Nacha or moov-io/ach JSON formatted body

POST /audittrail/:id/files

{
  "merging": {
    "filepaths": [
      "testing/20210825-135000/uploaded/sha1.ach"
    ]
  }
}

We've accomplished this with a separate shard that performs no merging. That shard can be triggered by itself whenever needed.


Take a look at tus for chunked uploading

https://tus.io/

@mvrhov Thanks!
Super interesting, you could use the tus js client to enable chunk uploading by writing a custom process function.

For someone who want FilePond work with tus, here is an example:
// client
FilePond.setOptions({
      server: {
        process: (fieldName, file, metadata, load, error, progress, abort) => {
          var upload = new tus.Upload(file, {
            endpoint: "http://localhost:3000/uploads/",
            retryDelays: [0, 1000, 3000, 5000],
            metadata: {
              filename: file.name,
              filetype: file.type
            },
            onError: function(err) {
              console.log("Failed because: " + err)
              error(err)
            },
            onProgress: function(bytesUploaded, bytesTotal) {
              progress(true, bytesUploaded, bytesTotal)
            },
            onSuccess: function() {
              load(upload.url.split('/').pop())
            }
          })
          // Start the upload
          upload.start()
          return {
            abort: () => {
              upload.abort()
              abort()
            }
          }
        }
      }
    })

//server
const tus = require('tus-node-server')
const server = new tus.Server()
server.datastore = new tus.FileStore({
  path: '/files'
})

const express = require('express')
const app = express()
app.use(express.static('public'))
const uploadApp = express()
uploadApp.all('*', server.handle.bind(server))
app.use('/uploads', uploadApp)

const host = '<IP_ADDRESS>'
const port = 3000
app.listen(port, host)

Using the scripts above...how do you access the "metadata" property on the server side?

You can also replace upload.start() with this code to allow resume when the there is an issue with the network
upload.findPreviousUploads().then(function (previousUploads) {
// Found previous uploads so we select the first one.
if (previousUploads.length) {
upload.resumeFromPreviousUpload(previousUploads[0])
}

// Start the upload
upload.start()
})


Slate Editor: when hitting return on a bullet point list the cursor goes to the beginning of the item instead of creating a new text block

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

as admin
go to https://6.demo.plone.org/add?type=Document
type "- foo"
hit return
-> cursor goes to the beginning of "foo"
-> no new text block is created

Screenshots

Not reproducible anymore, please verify this @tisto and close this
https://github.com/plone/volto/assets/43907055/2699273c-6d85-41c3-82e9-be301f09bf75

Confirmed, it's working now


Calls to AddHook hang/block

I'm writing a simple splunk log hook, using the example log hooks in the logrus package.
I am doing everything pretty much exactly the same except adding some custom fields but when I call AddHook(myhook), it hangs.
type SplunkHook struct {
    Client      *SplunkClient . // this is just an http.Client with some added fields
    ValidLevels []log.Level
    Myvar1 string
    Myvar2 string
}

func (h *SplunkHook) Fire(e *log.Entry) error {
    Log.Trace("SplunkHook.Fire(): called")
    err := h.Client.Log(e)
    if err != nil {
        Log.Errorf("ERROR: splunk hook failed: %s", err)
        Log.Debugf("SplunkHook.Fire(): event=%+v", e)
        return err
    }
    return nil
}

func (h *SplunkHook) Levels() []log.Level {
    Log.Trace("SplunkHook.Levels(): called")
    if h.Levels() == nil {
        return log.AllLevels
    }
    return h.Levels()
}

I'm adding the hook from within the function called for my run command (using spf13/cobra). Everything works up to the point AddHook is called.
        Log.Trace("calling NewSplunkClient()")
        client := splunk.NewSplunkClient()
        Log.Tracef("SplunkClient=%#v", client)
        Log.Trace("calling NewSplunkHook()")
        hook := splunk.NewSplunkHook(client, splunk.SplunkLevels)
        Log.Tracef("SplunkHook=%#v", hook)
        Log.Debug("adding splunk log hook")
        Log.AddHook(hook)
        Log.Debug("splunk log hook added")  // never get here

Thanks for any help.

Don't you have somehow a stack overflow or a similar error/panic ?
I guess that logging in a hook with the logger to which the hook instance is attached may lead to an infinite recursion.

Yea it ended up being my logrus.Logger calls inside the methods of my splunkclient struct which is included in my Hook struct.
Just my Go noobness showing.
Thanks for the help/


Class 'NodeOnDiskFile' incorrectly implements interface 'File'.

What version of Remix are you using?
Latest
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
Install a new Remix project & run npm run typecheck
npx create-remix@latest
? Where would you like to create your app? ./my-remix-app
? What type of app do you want to create? Just the basics
? Where do you want to deploy? Choose Remix App Server if you're unsure; it's easy to change deployment targets. Remix App Server
? TypeScript or JavaScript? TypeScript
? Do you want me to run `npm install`? Yes
cd ./my-remix-app
npm run typecheck

Expected Behavior
No type-errors would appear
Actual Behavior
node_modules/@remix-run/node/dist/upload/fileUploadHandler.d.ts:47:22 - error TS2420: Class 'NodeOnDiskFile' incorrectly implements interface 'File'.
  Property 'prototype' is missing in type 'NodeOnDiskFile' but required in type 'File'.

47 export declare class NodeOnDiskFile implements File {
                        ~~~~~~~~~~~~~~

  node_modules/typescript/lib/lib.dom.d.ts:2572:5
    2572     prototype: Blob;
             ~~~~~~~~~
    'prototype' is declared here.

Found 1 error in node_modules/@remix-run/node/dist/upload/fileUploadHandler.d.ts:47

This is the same error we got in #6108 & that @dpobel also reported in https://github.com/remix-run/remix/issues/6119#issuecomment-1521288835
The problem seemed to be that NodeOnDiskFile doesn't have a prototype property, but after implementing that (in #6108), we still get

Type 'NodeOnDiskFile' provides no match for the signature 'new (blobParts?: BlobPart[] | undefined, options?: BlobPropertyBag | undefined): Blob'.

Since @jacob-ebey is the creator of the class, maybe he can point me/us into the right direction here
I tried a lot, but I can't seem to find a solution to make this work tbh 🤔

After doing some investigation, I came to the conclusion it couldn't be a problem with typescript itself (even though the error is saying so) as that code never changes 🤷♂
It's always the same code:

typescript/lib/lib.dom.d.ts
interface Blob {
  readonly size: number;
  readonly type: string;
  arrayBuffer(): Promise<ArrayBuffer>;
  slice(start?: number, end?: number, contentType?: string): Blob;
  stream(): ReadableStream<Uint8Array>;
  text(): Promise<string>;
}

declare var Blob: {
  prototype: Blob;
  new (blobParts?: BlobPart[], options?: BlobPropertyBag): Blob;
};

type BlobPart = BufferSource | Blob | string;

interface BlobPropertyBag {
    endings?: EndingType;
    type?: string;
}

type BufferSource = ArrayBufferView | ArrayBuffer;

type EndingType = "native" | "transparent";

interface File extends Blob {
  readonly lastModified: number;
  readonly name: string;
  readonly webkitRelativePath: string;
}

declare var File: {
  prototype: File;
  new (fileBits: BlobPart[], fileName: string, options?: FilePropertyBag): File;
};

interface FilePropertyBag extends BlobPropertyBag {
    lastModified?: number;
}

Using package.json's overrides to specify the installed @types/node version, it seems we only have this problem with having v18 installed.
It seems like @vansergen introduced File in @types/node v18 (see https://github.com/DefinitelyTyped/DefinitelyTyped/pull/64319), which I assume is causing the problem now.
When looking at the implemented types in @types/node, they seem to be compatible to me (but maybe I'm missing some things?), so I don't really know what the actual problem could be here tbh 🤔

@types/node/buffer.d.ts
export interface FileOptions {
  /**
   * One of either `'transparent'` or `'native'`. When set to `'native'`, line endings in string source parts will be
   * converted to the platform native line-ending as specified by `require('node:os').EOL`.
   */
  endings?: "native" | "transparent";
  /** The File content-type. */
  type?: string;
  /** The last modified date of the file. `Default`: Date.now(). */
  lastModified?: number;
}
/**
 * A [`File`](https://developer.mozilla.org/en-US/docs/Web/API/File) provides information about files.
 * @experimental
 * @since v18.13.0
 */
export class File extends Blob {
  constructor(
    sources: Array<BinaryLike | Blob>,
    fileName: string,
    options?: FileOptions
  );
  /**
   * The name of the `File`.
   * @since v18.13.0
   */
  readonly name: string;
  /**
   * The last modified date of the `File`.
   * @since v18.13.0
   */
  readonly lastModified: number;
}

@types/node/crypto.d.ts
type BinaryLike = string | NodeJS.ArrayBufferView;

The issue is it doesn't extend the DOM types directly so it's missing the prototype. @pcattori do you know of a way to trick typescript into adding the prototype to a non-related class?

Doing some more investigation, I landed on https://github.com/DefinitelyTyped/DefinitelyTyped/discussions/65049, which lead me to @pcattori's https://github.com/microsoft/TypeScript/issues/52166
According to that issue, it doesn't seem to be a problem with @node/types, but it seems to be a regression in TS that's failing to recognize the fact that we're doing an implements of the interface instead of the var.
As @pcattori pointed out: things are working as expected with TS 4.8.4 in our codebase (this is also confirmed after I did that in #6108)
When setting TS to v4.8.4 in a template, we still get the same error though 🤔

Did some further investigation and landed on https://github.com/remix-run/remix/issues/4371.
Here @akomm points out (see https://github.com/remix-run/remix/issues/4371#issuecomment-1377481821 & https://github.com/remix-run/remix/issues/4371#issuecomment-1378460377) that it would be a collision between typescript/lib/lib.dom.d.ts & @types/node/buffer.d.ts, as you can see in the TS Playground example he posted.
Basically having types for any built-in Node module is causing this error.

Can we just // @ts-ignore in the remix package on the implements line?

It would be probably better than blocked build or having to entirely disable typecheck on production build. But the types are actually wrong as the prototype isn't properly picked, so writing code that operates on the "wrong" types could lead to unexpected bugs. Just saying. It should be removed once the issue is resolved on the TS side.

@jacob-ebey Adding @ts-ignore on the implements line still throws errors on other lines.

According to #52166:

This changed between TS 4.8.4 and TS 4.9.3. Typechecking passed in 4.8.4, fails in 4.9.3+

Is it an option to roll the TS back to 4.8.4 in the Remix dependencies?

@lpsinger One of the reasons we updated to V5 was to support extends array in tsconfig.json
Reverting to TS v4 would revert that support as well 😕
@jacob-ebey Is there any other solution you can think of besides adding @ts-ignore?
Can't really think of anything else tbh 🤔

Duplicate of #4371

This is now fixed by #6108 and will be released in v2


Search dialogue shows up with "/" even when search.engine = 0(disabled)

Describe the bug
Search dialogue shows up with "/" even when site.Params.search.engine = 0(disabled). And the dialogue is unusable since there is no search engine enabled.

To Reproduce
Steps to reproduce the behavior:

Clone starter-academic
git clone https://github.com/wowchemy/starter-academic && cd starter-academic
Change the search.engine configuration as follows

diff --git a/config/_default/params.toml b/config/_default/params.toml
index 214e096..004bdf2 100644
--- a/config/_default/params.toml
+++ b/config/_default/params.toml
@@ -226,17 +226,17 @@ plugins_js  = []
 ############################
 ## Search
 ############################
 [search]
   # Search provider:
   #   0: No search engine
   #   1: Academic (built-in)
   #   2: Algolia (https://www.algolia.com)
-  engine = 1
+  engine = 0

   # Configuration of Algolia search engine.
   # Paste the values from your Algolia dashboard.
   [search.algolia]
     app_id = ""
     api_key = ""
     index_name = ""
     show_logo = false

Run a local server with hugo server
Press "/" in the page

Expected behavior
Nothing happens ("/" will be ignored).
Technical details:

Link to your GitHub project: N/A
Wowchemy Version (from your go.mod): v0.0.0-20201224144527-9cce661db8d7
Hugo Version (run hugo version): Hugo Static Site Generator v0.79.1/extended linux/amd64 BuildDate: 2020-12-22T14:06:09Z
Browser/OS: Firefox 84 / Ubuntu 20.04 LTS
Wowchemy Template: starter-academic

Fixed by 25e0b0627f0965a221ad9b9e7a9c2ff33c01b657

Fixed by 25e0b0627f0965a221ad9b9e7a9c2ff33c01b657


"copy()" deepcopys "self.root" not "self", leading to unexpected behaviour not in line with documentation

Like the title says, copying e.g. a svgutils.transform.GroupElement returns a lxml.etree._Element, not a new GroupElement. That means that if you try to copy an Element instead of reading it from disk again, the function does not return a copy of the Element, like the documentation says.
Honestly, I don't understand how this could ever be the desired behavior. This way, if you want to include the same SVG twice on two different spots of the figure, you need to pass the result of copy to the appropriate constructor before you can move it.
So I propose that either the documentation should reflect this, or the behaviour should be changed to be in line with what you would naturally expect.

hi @Quwertzuiopp, thanks for the report! I agree that the expected behaviour would be the copy of svgutils class not the lxml base. Please feel free to submit a PR!

Can I take this up (If no one is working on this)?


Refactor utility functions for pheno command into new module

The function order and contents remain unchanged.
Closes #59

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

93 of 93 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 98.107%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
311

Relevant Lines:
317

💛  - Coveralls


ability to avoid duplicate test names

I feel greatest needs a method for avoiding duplicate test names. I personally have 2 scenarios where this would be helpful.

Visually getting a overview of results is difficult when many tests are named the same and only a few of them are failing.

I use TeamCity for continuous integration. I was looking to expanded the CI builds to run some basic regression testing. All of my tests use greatest. greatest support is not integrated into TeamCity but they have a method for reporting test results on the fly. I created a powershell script which interprets greatest print statements and creates the necessary TeamCity messages. Usage looks like mytests.exe -v | greatest2teamcity.ps1. One of the notes in the TeamCity documentation is that it's highly recommend to have unique suite+test names. Currently I solved this problem by keeping a counter of duplicate test names and appending that count amount to the end of the test name when duplicates are detected.

In my use of the greatest test framework I regularly use the following programming paradigms.
RUN_TEST1(MyFirstTest, AN_ENUM_TYPE_A);
RUN_TEST1(MyFirstTest, AN_ENUM_TYPE_B);
RUN_TEST1(MyFirstTest, AN_ENUM_TYPE_C);
RUN_TEST1(MyFirstTest, AN_ENUM_TYPE_D);
RUN_TEST1(MyFirstTest, AN_ENUM_TYPE_E);
RUN_TEST1(MyFirstTest, AN_ENUM_TYPE_F);

--OR--
TEST ASpecialTest(U8BIT u8Resolution)
{
    /*
    Do great things with u8Resolution
    */

    PASS();
}

void ASpecialTest_iterator()
{
    U8BIT u8Resolution;

    for (u8Resolution = 8; u8Resolution < 98; u8Resolution++)
    {
        RUN_TEST1(ASpecialTest, u8Resolution);
    }
}

SUITE(my_suite)
{
    ASpecialTest_iterator();
}

Both of these generate a long list of outputs that look something like
PASS MyFirstTest:  (1 ticks, 0.000 sec)
PASS MyFirstTest:  (1 ticks, 0.000 sec)
FAIL MyFirstTest:  (0 ticks, 0.000 sec)
PASS MyFirstTest:  (1 ticks, 0.000 sec)
PASS MyFirstTest:  (0 ticks, 0.000 sec)
PASS MyFirstTest:  (1 ticks, 0.000 sec)
PASS MyFirstTest:  (0 ticks, 0.000 sec)

One possible solution would be for greatest to have some built in functionality for looping through a range of variables and possible appending the variables value to the end of the test name. This still may not solve this issue for me as some of my "iterator functions" have up to 4 nested for loops.
Another possible solution would be to create additional marcos to the already existing GREATEST_RUN_TEST, GREATEST_IGNORE_TEST, and GREATEST_RUN_TEST1. Three new macros would take in a string argument for an extended name.
For example I have the following in a working example
/* Run a test in the current suite with one void * argument,
 * which can be a pointer to a struct with multiple arguments.
 * Defined with an extended test name */
#define GREATEST_RUN_TEST1_EXT_NAME(TEST, ENV, EXT_NAME)                \
    do {                                                                \
        if (greatest_test_pre(#TEST) == 1) {                            \
            enum greatest_test_res res = GREATEST_SAVE_CONTEXT();       \
            if (res == GREATEST_TEST_RES_PASS) {                        \
                res = TEST(ENV);                                        \
            }                                                           \
            greatest_test_post(#TEST, EXT_NAME, res);                   \
        }                                                               \
    } while (0)

/* Run a test in the current suite with one void * argument,
 * which can be a pointer to a struct with multiple arguments. */
#define GREATEST_RUN_TEST1(TEST, ENV)                                   \
    GREATEST_RUN_TEST1_EXT_NAME(TEST, ENV, "")

I've tried to address this before, but haven't found a solution I'm happy with yet. It becomes even more of an issue once test shuffling is in use, or when the output is being consumed by another system (such as CI).
Unfortunately, snprintf isn't available in C89, and while I could add GREATEST_RUN_TESTx_EXT_NAME for C99, that adds several new macros in combination with the other choices. Most of them would probably be one-line wrappers with a "" or NULL argument for the optional suffix, but there may be an option that avoids adding several more macros.
Taking an optional string argument seems like a better approach, rather than attempting to do any suffix construction in greatest itself (or appending a distinct but generally meaningless counter).

How about adding a function that sets a suffix for the next test run (which is then cleared)? That wouldn't lead to a combinatorial explosion of RUN_TEST_* macros, and would nicely fit in a loop, table iterator, or other construction that repeatedly calls a parametric test.

This will appear as greatest_set_test_suffix(const char *suffix) in the next release.

This feature is working well for me. I deleted my private fork and moved back to the mainline.

Excellent! I find it really useful too, especially combined with -t and -x.


ERR Name to resolved - I'm unable to use dnscrypt

Hello, I'm running an Arch Linux installation and I read and followed the Arch Wiki instructions plus a blog to properly set dnscrypt with unbound.
After doing the proper modifications to my system resolv.conf, ubound.conf etc without incurring in any execution error, when I try to load any website I get the name not resolved error and I'm having some troubles at troubleshooting this because I'm not that expert about the connection scripts.
I'd like any hint to where and what to look for, thanks in advance.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/35504059-err-name-to-resolved-i-m-unable-to-use-dnscrypt?utm_campaign=plugin&utm_content=tracker%2F417785&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F417785&utm_medium=issues&utm_source=github).

Your chosen DNS server supports DNSSEC, right?

I didn't modified the default one which accordingly to arch wiki is dnscrypt.eu-nl, which based on this table supports it.
So TLDR yes :)

Do try others too, maybe that one was not working when you tested or something.

Can you try to just run it without systemd:
$ dnscrypt-proxy -R dnscrypt.org-fr -a <IP_ADDRESS>:5300

And the check if you can resolve names:
hostip -r <IP_ADDRESS>:5300 example.com

?

I tried to do some more troubleshooting and posted whatever I found [here] (https://bbs.archlinux.org/viewtopic.php?pid=1637434#p1637434) (at this point however you don't need to go through all the post).
@jedisct1 running it yelds Unable to retrieve server certificates
[federico@arch-anywhere-federico]: ~>$ dnscrypt-proxy -R dnscrypt.org-fr -a <IP_ADDRESS>:5300
[INFO] + DNS Security Extensions are supported
[INFO] + Namecoin domains can be resolved
[INFO] + Provider supposedly doesn't keep logs
[NOTICE] Starting dnscrypt-proxy 1.6.1
[INFO] Generating a new session key pair
[INFO] Done
[ERROR] Unable to retrieve server certificates
[INFO] Refetching server certificates
[ERROR] Unable to retrieve server certificates
[INFO] Refetching server certificates
[ERROR] Unable to retrieve server certificates
^C

So far I tried also many (not all) other resolvers all of them giving the same result as above:

4armed
dnscrypt.org-fr
dnscrypt.eu-nl
d0wn-de-ns2
ipredator
cs-fr
dnscrypt.eu-dk
cs-uswest
cs-uswest2

I find it really unlikely that all of them are blocked by my ISP.

ovpnto-se instead gave me:

[federico@arch-anywhere-federico]: ~>$ dnscrypt-proxy -R ovpnto-se -a <IP_ADDRESS>:5300
[INFO] - [ovpnto-se] does not support DNS Security Extensions
[INFO] + Namecoin domains can be resolved
[INFO] + Provider supposedly doesn't keep logs
[NOTICE] Starting dnscrypt-proxy 1.6.1
[INFO] Generating a new session key pair
[INFO] Done
[INFO] Server certificate #808464433 received
[INFO] This certificate has expired
[ERROR] No useable certificates found
[INFO] Refetching server certificates

so this is something you might want to fix in the list or contact the resolver to update their certificate, but WOAH one of them answered the call! So I told myself, let's give a shot to one more aaaand d0wn-gr-ns1 WORKS!!
[federico@arch-anywhere-federico]: ~>$ hostip -r <IP_ADDRESS>:5300 google.com
<IP_ADDRESS>

That came unexpected since I started thinking that none of them was going to work =)
At this point I will test all the remaining resolvers in the list but after this experience I would like to highly suggest to improve the dnscrypt service with some kind of test all and maybe order the results by ping. :D

The list of working resolvers is constantly being verified. For example ovpnto-se whose certificate expired has been temporarily removed from the list.
Apparently, your computer or something between your computer and the resolvers, is blocking or intercepting DNS queries. Certificates are retrieved with regular DNS queries, using UDP on port 53.
So, make sure that you don't have something on your computer or router blocking these queries.

Well yes I'm on a restricted network which provides it's own dns server and it's own certificate authority for websites so the blocking happens there for sure.
I was hoping to be able to use dnscrypt to "escape free" on the net in conjunction with a paid VPN maybe.

dnscrypt was not designed to escape corporate policies. Since some servers are accessible and some aren't, maybe they are even explicitly blocking public DNS servers.
The resolvers-check.sh script can be used to automatically check all the resolvers.

Thanks for the infos and the link to the script, it's really useful =)
Guess that the issue can be closed at this point.


fix: height of 100% when in landscape mode

FIX LANDSCAPE MODE
Before

After

Sorry for never responding here 😞 This doesn't seem to be an issue anymore. I'm closing this out. But thanks for contributing!


Fix type of prefix arg in staticfiles_urlpatterns

Using a type of None for the prefix keyword argument is not going to typecheck very well, so it might be a typo here because the default value (rather than the type) is None.

@mkurnikov: Hm, is CI currently broken? It can't find mypy/pytest.


Resolve feature and optional dependencies for workspace as a whole

Currently, a dependency's feature set is determined by what package is getting built, without considering other packages in the workspace. This will likely cause a crate to be compiled more than once with different feature sets.
Instead, we should consider the full workspace when resolving features. We have a few choices:

Assuming all of the features are enabled when resolving/compiling dependencies
This will minimize the possibility we will be building a crate twice, but this can be unwanted behavior sometimes.
Assuming default features for all crate in the workspace
This will be less surprising though it doesn't cover all the feature combinations that are possible.

Note that specifying features in workspace doesn't work anyway ¯\_(ツ)_/¯: #5015
Related: #4463

As a workaround, there is https://crates.io/crates/cargo-hakari to help create a "workspace hack" that enables all of the relevant features for all packages.
We have competing needs

Workspaces that publish their content independently need to be able to have their features tested individually
Workspaces with a product broken into packages need a quick way to move around the product and build pieces without rebuilding dependencies with different features selected

The challenge will be in having a solution that easily let's people navigate between those needs, including when they co-exist within the same workspace.


FIX Confirmation components to respect SS_BASE_URL

Fixes #9061

@robbieaverill sorry, I've rolled back the Director::absoluteURL use. It's not going to work with urls that begin with slash. On the other hand, we cannot rely on having relative URLs in the YML config for the middleware, because then we won't be able to use actual relative URLs there (that would introduce another bug). Sanitising it to suit the peculiar Director::absoluteURL logic doesn't feel right either.

I'd really prefer we don't squash and commit these one commit PRs into framework :\ . It's really weird when you're trying to look at the commit that @dnsl48 authored not appearing any any tag (or branch) because the commit hash has been changed unnecessarily.


fixing serpApi tool, base url and handling results

Fixes #1981

This changes the provider - if you'd like to add this as a separate tool, that could make sense.


Base regex rule style for improved flexibility

Something like this makes it all much easier:
import * as Promise from "bluebird";
import {IValidationRule} from "treacherous";

export class SuperRegexValidationRule implements IValidationRule
{
  constructor(ruleName:string, expression:string, message:string|((value)=>string) ) {
    this.ruleName = ruleName;
    this.expression = expression
    this.message = (typeof message === "function") ? message : (v)=> { return (<string>message); };
  }

  public ruleName = "?";
  public message:((value)=>string);
  public expression = "?";

  public validate(value, regexPattern: RegExp): Promise<boolean>
  {
    if (value === undefined || value === null || value.length == 0)
    { return Promise.resolve(true); }

    var matchesPattern = value.toString().match(this.expression) !== null;
    return Promise.resolve(matchesPattern);
  }

  public getMessage(value, regexPattern) {
    return this.message(value);
  }
}

Called using
ruleRegistry.registerRule(new SuperRegexValidationRule('launchURL',url,value => '${value} is an Invalid launch URL'));
(needs back-ticks in the string parameter - can't escape them in Git editor)
or
ruleRegistry.registerRule(new SuperRegexValidationRule('launchURL',url,'Invalid URL'));

Only problem here is that this rule would need the user to explicitly add it into the rule registry so to even use this rule class you would need to do more than just including treacherous, which makes it a custom step and although this placeholder style rule would be useful for people it may not be useful to all.
So I am in 2 minds if this should be seen as a possible enhancement, so it can be consumed by others if they wanted to have more complex regex rules without having to write custom rules, or if it should just be down to user preference to do the above, as you can already do what you want above in the system via the existing regex objects its just you would put the regex into the options when defining the rule rather than pushing it up to the registry level.
Can can discuss it further but I see that for the majority currently this would not add much benefit as the amount of times people need to do regex the current regex rule would be applicable, and if they have complex regex rules they could easily just create a custom rule like you do above, as once you start to open up blueprint rules like above where do you stop?
Anyway lets chat and see what happens.

Yes, hence the examples at the bottom explaining two ways of adding it with a custom message.
If every rule is responsible for its own message, as it is at the moment, then for the user to add custom re-usable rules with associated messages then they either have to be added to the central registry or you have to add a local registry against the validationGroup.
In this case, in my code I add rules of "domainURL" that does not allow a path on the domain name but requires a trailing slash, and "domainPage" that allows a trailing path. I use the first in one input field, and the other in another three, so in the absence of custom messages per field (and as a possible alternative) this works adequately.
Perhaps a fluent syntax to add the rules would be nice, as this makes adding custom rules a first-class citizen rather than a low-level fallback when you realise the default rules don't suffice.
I think that having a registerRule on the validationGroup that is used in preference to the global rules would be prefereable, as this avoids having to find unique names for every form's field's requirements.
Also, having the option of a method for resolving the name allows more snazzy messages if the dev decides the user needs them, yet adds nothing to the complexity if they don't need it.
The issues with using options to pass in all your parameters are:

You have to duplicate fields such as regex if you have many similar fields (URL matching for example)
You can't change the error message on a per-field basis unless you code that into the rule parameters
The advantage of predefined generic rules is that they save you typing, but the disadvantage is that alterations have to be implemented somehow, and generic options structures get unstructured difficult to follow very quickly.
Perhaps some of this is related to using string for rules. You can't specify parameters nicely when using strings, so at least this rule-builder method allows strong-typing, and an unlimited number of configuration parameters the could be bound to the options in addRule if necessary (have to think about that one)

Given the conversations we have had on the matter and with the ability to add custom message overrides (via #12) what are your current thoughts on this?

The need to do this for custom messaging is now removed, but I see no issue in providing simple helper base classes for when a developer needs to create a custom rule, primarily those that are regex or Promise based.
Even the current email, number, date and iso-date are identical except for the regex expression, so they could be removed and replaced with calls to RegexBaseValidationRule or whatever a good name is in the exposer.
I'd simply suggest there be a helper function to make life easier for people to build their custom rules. The benefit is that a single line in the doco can be used to demonstrate creating a custom rule with a complex regex (which is probably the most common custom-rule usage scenario), rather than having to create new classes where 90% of the codebase is common.

Added to next release.


fix: removed stream methods from connection interface

Moving the discussion from this pull request, PR was originally opened in js-libp2p-interfaces before being moving to this monorepo
the muxers should be ultimately responsible for managing streams, that's why addStream and removeStream methods are being removed from the interface see
Fixes issue #356

It's probably worth rebasing this once #1833 is in as it'll show the full scope of the change across the stack.

Closing in favour of https://github.com/libp2p/js-libp2p/pull/1912


An issue with the tutorial on http://mithril.js.org

I'm new to mithril, so I maybe missing something...
After implementing the 'save' method from the User model in http://mithril.js.org/simple-application.html the result of clicking the Save button is a page refresh.
I tried return false after the call to m.request and that fixed the problem, so I assume the problem relates to preventing the browser's default event handling (?).
But now the function doesn't return the promise, which I guess is not good.
What is the correct way of writing that 'save' method.
btw, I have m.route.prefix("") not sure if that's important..

I think the button should be type=button, not type-submit

Better to bind a preventative handler to the form's submit property IMO, since this is never desired behaviour in this app.

@lhorie Thanks! that worked.
@barneycarroll I also like to bind to the form's submit, but then I can't return a promise (like the tutorial shows for 'load').
Is that not important in some why for mithril?

It should be perfectly possible to prevent form onsubmit and use promises in your model: the return value of event handlers is unconnected to the model.

Thanks.
Created a PR: https://github.com/lhorie/mithril.js/pull/1646

ah turns out the docs are already fixed, but not published to the site.


Question: dzsave "fsync" oddity

I've created a DZI from a 1.6 GB image.  It worked.  It ran in about 2.5 minutes.  Was awesome.  However, once the dzsave finished, I reckon there was a good 30+ second lag before the temporary directory was renamed to blah_files and the blah.dzi file appeared.
Is there something I'm doing wrong?
# Generic code that handles different RAW file formats and extracting 2D slices from
# 3D images but I've replaced variables with the actual values used

# Load the original
original = Vips::Image.rawload(file, 30000, 30000, 2, access: :sequential_unbuffered)

# Re-interpret
original = original.copy(bands: 1, format: :ushort, interpretation: :grey16)

# Extract the slice we want
slice = original.extract_area(0, 0, 30000, 30000)

# Generate the DZI files
extracted.dzsave("blah", suffix: ".png", overlap: 1, tile_size: 510, depth: :onepixel, strip: true)

Totally my fault! So sorry.


Plugin: Battery

Tracks a simple battery plugin. We will write this as a first-party plugin to serve as a sample of how to write a plugin.
It will integrate with to BatteryManager on Android, and UIDevice on iOS

@szakarias with platform_services now being the battery sample, should the sample plugin be geolocation instead?

Alternatively could be accelerometer. Would be good to have something that demonstrates both methods and event streams.

@mit-mit a geolocation sample would be much appreciated!

Yes, we are tracking that in https://github.com/flutter/flutter/issues/121. No promises on when we might get to that though. If you need something fairly simple, you should be able to do that quite quickly with platform channels, see https://flutter.io/platform-channels/

This is now available: https://pub.dartlang.org/packages/battery


Bug: Inconsistent levels when creating Entry from Logger

Example: https://play.golang.org/p/zZODKE5p6SD
Is this a bug?

@iamolegga that does look odd, I added some more output here: https://play.golang.org/p/Xe6QQx1igjo
You can see the entry is actually logging at the right level, ie it doesnt output 'debug' level messages since the root logger is configured to log at 'info' and above... however when you call entry.Level it still outputs level: panic which is incorrect.
logger level: info, entry level: panic
time="2009-11-10T23:00:00Z" level=info msg="I should be printed" foo=bar
logger level: info, entry level: panic

So I'm inclined to say that yes, it is a bug in that entry.Level does not seem to report the actual 'inherited level' that it is logging at.

I believe this isn't fixed yet

not fixed

not fixed

not fixed

not fixed

not fixed

not fixed

not fixed

not fixed

not fixed

@iamolegga The library is already inactive, maybe you should try another solution! Just like me.


Add user_attributes and user_events methods

As per DE-10851 users were ending up with two events sometimes, especially for registration, because a) our eut.create_date is a date getting turned back into a time, so it never matches leanplum's registration time but more importantly b) even if it did match, we would go ahead and post it anyways and dupe the registration event that leanplum creates for users who register on the device.
This isn't that material to our use/analysis in leanplum, but it did confuse some end users.
This PR helps to remedy that by enabling the leanplum push jobs to see if a given event already exists for a given user, in which case they can skip pushing that event to leanplum.
@rfroetscher @sgrgic

:+1:

:sake:


Close idle

Issues associated with this PR
Your PR should present related issues you want to solve.
Sign the CLA
Make sure you have signed the CLA
Solutions

server close connection when read timeout reached the max count
client close (sofarpc) connection if the connection only have heartbeat on it. (do not apply yet)

UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases.
And you need to show the ut result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #721 into stable will increase coverage by 1.16%.
The diff coverage is 66.27%.

@@            Coverage Diff             @@
##           stable     #721      +/-   ##
==========================================
+ Coverage   50.72%   51.89%   +1.16%
==========================================
  Files         193      195       +2
  Lines       16505    16573      +68
==========================================
+ Hits         8372     8600     +228
+ Misses       7292     7095     -197
- Partials      841      878      +37

Impacted Files
Coverage Δ

pkg/types/network.go
0% <ø> (ø)
:arrow_up:

pkg/proxy/proxy.go
8.33% <0%> (+0.33%)
:arrow_up:

pkg/stream/sofarpc/connpool.go
1.86% <0%> (+0.04%)
:arrow_up:

pkg/network/listener.go
49.38% <0%> (+49.38%)
:arrow_up:

pkg/buffer/iobuffer.go
75.31% <0%> (-2.52%)
:arrow_down:

pkg/stream/http/connpool.go
2.18% <0%> (+0.07%)
:arrow_up:

pkg/network/transfer.go
0% <0%> (ø)
:arrow_up:

pkg/server/server.go
0% <0%> (ø)
:arrow_up:

pkg/stream/client.go
0% <0%> (ø)
:arrow_up:

pkg/server/handler.go
37.24% <100%> (+0.48%)
:arrow_up:

... and 12 more

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 0777e6e...e774457. Read the comment docs.


Move eligibility flow into separate json file

I created a separate json file named eligibility-flow.json which anyone can change to edit the question text or answer text.
I tried to make the format flexible.  Right now, all the questions have only two answers, but now it would be easy to add multiple answers if the need arises later on.  I've also left space for "helperText" where we can add definitions to explain the legal terms.
(Everything still looks the same, so I am not adding a screenshot.  This just changed the functionality.)

Here is an example of the format from the new json file.
questionText = question that will be displayed for the user
answerText = words that will be displayed on the buttons
next = which question should the user see next if they click this answer?
helperText = definitions or explanations of confusing wording in the question (currently empty, we need to add it)
   "question0":{
      "questionText":"Do you have a case pending?",
      "answers":[
         {
            "answerText":"Yes",
            "next":"ineligible at this time"
         },
         {
            "answerText":"No",
            "next":"question1"
         }
      ],
      "helperText":[

      ]
   },

   "question1":{
      "questionText":"Are you sealing a conviction or a non-conviction?",
      "answers":[
         {
            "answerText":"Conviction",
            "next":"question2"
         },
         {
            "answerText":"Non-conviction",
            "next":"question5"
         }
      ],
      "helperText":[

      ]
   },

If there is helperText for the question, it now displays below the buttons like this:

I am going to put in the definitions that @jimwebb added in #26

Finally, I added a brief explanation of the formatting of the eligibility-flow.json to the README so that non-technical people can be comfortable working with that file.  If my explanation is not clear, let me know!  Here is what it will look like in the README:

I think I'm done adding commits so unless you have a request, it is ready to merge!

For some reason, the updated site is not working from the github account

It's because of the project name change to "clean-slate"
Try this: http://codefordc.github.io/clean-slate

steven do u think my name change affected his pull?
On Tuesday, May 5, 2015, jmezetin<EMAIL_ADDRESS>wrote:

For some reason, the updated site is not working from the github account
—
Reply to this email directly or view it on GitHub
https://github.com/codefordc/clean-slate/pull/29#issuecomment-99256747.

right. what should i do to fix it?
On Tuesday, May 5, 2015, Briane Knight<EMAIL_ADDRESS>wrote:

steven do u think my name change affected his pull?
On Tuesday, May 5, 2015, jmezetin<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

For some reason, the updated site is not working from the github account
—
Reply to this email directly or view it on GitHub
https://github.com/codefordc/clean-slate/pull/29#issuecomment-99256747.

I think you can just edit the link at the top of the page and it should work.  On my fork of the project I was able to do that; there should be an 'edit' button next to the link.

is the updated site working for you now?
On Tue, May 5, 2015 at 7:11 PM, jmezetin<EMAIL_ADDRESS>wrote:

For some reason, the updated site is not working from the github account
—
Reply to this email directly or view it on GitHub
https://github.com/codefordc/clean-slate/pull/29#issuecomment-99256747.

@brianek Just edited it; this didn't affect the pull, just the description at the top of github.com/codefordc/clean-slate.

thank you!!!
On Tue, May 5, 2015 at 10:35 PM, stvnrlly<EMAIL_ADDRESS>wrote:

@brianek https://github.com/brianek Just edited it; this didn't affect
the pull, just the description at the top of
github.com/codefordc/clean-slate.
—
Reply to this email directly or view it on GitHub
https://github.com/codefordc/clean-slate/pull/29#issuecomment-99298399.

Thanks. I have created a roadmap of the next steps for this project if you are interested. Please note your interest so that i am knowledgeable about who is working on what: https://docs.google.com/presentation/d/1vCgDUQFLVqBx18dExyK01PfNHnLSI6aCoobV-h2Z4X8/edit?usp=sharing


prowgen: support testing on different flavors

SO is tested on different flavors, currently prowgen supports only aws (cluster claim-based).

Identify what we test on different OpenShift flavors in SO
Make changes to prowgen to generate prow configs for different flavors and workflows

With Prowcopy or a similar form of it, I believe we have middle ground between supporting multiple flavors and keeping the CI tooling simple, closing, we can revisit later


Fix version tag environment variable.

Changes

Fix version tag environment variable.
Also format markdown docs.
Fixes #534
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/lgtm


Allow no checkout when adding Worktrees

Added the ability to skip checking out when adding a new worktree.

Closing as per OctopusDeploy/ResearchAndDevelopment#235


SimpliSafe rate limiting causes 403 errors, messing up connectivity and deleting accessories

See #52 .
We need to implement a better way of supporting 403 errors.
Ideally, we would like to:

Monitor for 403 errors
If they do arise, block requests to SimpliSafe for 5 minutes
If it still hasn't been resolved after 5 minutes, wait for 10, then 15, and so on

Finally, if a 403 error occurs upon plugin initialization, do not refresh or delete accessories from HomeKit.

Working good for me:
[2/1/2020, 9:52:06 PM] [Home Alarm] Found 13 cached accessories being configured
[2/1/2020, 9:52:06 PM] Homebridge is running on port 51827.
[2/1/2020, 9:52:06 PM] [Home Alarm] Log in failed due to rate limiting or connectivity, trying again later
[2/1/2020, 9:52:06 PM] [Home Alarm] Refreshing accessories (add and remove: true)
[2/1/2020, 9:52:06 PM] [Home Alarm] Accessory refresh failed due to rate limiting or connectivity
[2/1/2020, 9:53:06 PM] [Home Alarm] Log in attempt failed, still rate limited
Thanks for adding this!


Enable linting etc.

We should enable pre-commit checks and clean the code accordingly.

I want to touch parservasp, but I am afraid of doing it because no pre-commit setting.
As an attempt, I used .pre-commit-config.yaml of aiida-vasp, and pre-commit install && pre-commit run --all-files changed significant amount of lines.
So I thin it is good to introduce pre-commit hook when almost nobody is working on parsevasp. Do you know when is appropriate. I assume only @espenfl is working if it is, but maybe also @zhubonan and @lan496?

I am not currently working on parsevasp. But, if possible, I would like the maintainers to merge this open PR before introducing the linters (to avoid rebasing and re-opening the PR).

I am not currently working on parsevasp. But, if possible, I would like the maintainers to merge this open PR before introducing the linters (to avoid rebasing and re-opening the PR).

Done. Thanks a lot for the contribution and sorry for the delay of merging.

good to introduce pre-commit hook when almost nobody is working on parsevasp.

Yes, absolutely. That was the plan, but I have never gotten to it. If you have the time, please look into it. If it is the same linter it should not really change anything substantial, even though there will be some changes of lines etc. Also we have a pretty decent test coverage.

Thanks for your quick responses.
@espenfl, will you merge your PR #97?
I am going to send a PR of the linted version if it is ready to run pre-commit.

Goodie, will do, but in the middle of fixing the merge conflicts for the aiida-vasp 527 PR. Soon done. But just open a PR and then I will update the stale PR for the sync of the eigenvalues based on that, no problem. Thanks @atztogo :)

Yes, we should use the setup.json data from aiida-core. Since this is a dependency for both aiida-vasp and parsevasp. This is fine, but for parsevasp we need to also let it fly on its own so I suggest that we just inherit what is defined in aiida-core and only override if it is strictly necessary (do not really see any need at this point to pin something in parsevasp).

Thanks for your valuable comments. I will follow .pre-commit-config.yaml in the latest release of aiida-core (v1.6.5). Do you agree with max-line-length = 120 following aiida-core?

Thanks for your valuable comments. I will follow .pre-commit-config.yaml in the latest release of aiida-core (v1.6.5). Do you agree with max-line-length = 120 following aiida-core?

Thanks yourself @atztogo Yes, I have no specific feelings for the max length, except that 80 is too short. Let us go with 120 as well and then we will just update accordingly if there will be changes.


Omnipay v3?

Hi,
Are there any plans to update this gateway to support Omnipay v3?

I am also in need for v3, so i have fork this project and have making the necessary changed.  I have just create a pull request so let's hope it's review and merged into this.  Here's my fork version that support Omnipay v3 https://github.com/ttruong15/omnipay-pin

I merged https://github.com/thephpleague/omnipay-pin/pull/19 so please give it a try

Hi there,
When using v3 I'm getting the following:
requires omnipay/pin ~3.0 -> found omnipay/pin[dev-master, 3.0.x-dev (alias of dev-master)] but it does not match your minimum-stability.

Would it be possible to release a new stable version for 3.x?
Otherwise, is there a workaround so I don't  need to add minimum-stability to dev and prefer-stable to true in my composer.json?
Cheers,
Alby


PHP 5.3 Support.

Please accept this pull request to make the FsUtils PHP 5.3 compliant.
Cheers,
Jérôme


Input dataset

Hi @astorfi
I have some questions about input dataset.

According to the paper, the number of speakers is 511 in the development phase.
But how long is the input audio file per speaker ??

Although there is the function of CMVN preprocessing in input_feature.py, I'm not sure whether CMVN preprocessing is appropriate for the output of speechpy.feature.lmfe function.
Did you use CMVN preprocessing in the experiment of the paper??

Thank you for your work!!

@astorfi Thanks for answering.
I think every 0.81 second audio file result in  (80, 40) feature, and you concatenate 20 features to make (20, 80, 40) feature for development phase, is it right?
I don't know how many (20, 80, 40) features per speaker do you use in the paper .
You use just one (20, 80, 40) feature for one speaker and make the dataset shaped (511, 20, 80, 40) ??
Anyway, I appreciate for your work and kindness.

@ku2482 Yes, that's quite correct.
For the second part, (20, 80, 40) features are fed to the network. "20" is the number of spoken utterances for the speaker. However, there is no restriction on the number of (20, 80, 40) features for any speaker. The rule of thumb is "the more is the better for background model generation". You can use "20" spoken utterances at random for data augmentation (although all needs to belong to the same speaker).

@astorfi Thank you so much!!
I actually solve all my questions and now I can understand your script.
Your work is really great!!
I close this issue, and again, thank you!!


Unable To Debug

I would like to develop within the docker container while being able to setbreak points and inspect in vscode.
I typically accomplish this by passing the --inspect flag for the startup process.
"start": "node --inspect=<IP_ADDRESS>:9222 server.js",

Unfortunately, I'm having a hard time understanding how to apply this concept to this project....
Can someone explain how I can debug strapi in vscode while running in a docker container?

Hello @mikepuglisi, please try to bind ports in the docker-compose file like this tutorial

I am closing this issue because we don’t have any updated/needed information regarding it.
Will reopen it if activity come back.


Docker fails to pull image

Describe the bug
When running the command to pull image, it always throw an EOF in the end.
To Reproduce
Steps to reproduce the behavior:

Start docker in OrbStack
Click start in container tab
Run docker pull node:14.21 in terminal
See 'unexpected EOF' in the end

Expected behavior
Pull image correctly.

Open an issue here: https://github.com/orbstack/orbstack/issues/new/choose

Additional context

vmgr.log from Show logs:
vmgr.log

Did you just restart OrbStack? I don't see any network errors in that log.
The fact that you're using a local proxy stands out:
time="04-11 17:05:15" level=info msg="using proxy: https" host=<IP_ADDRESS> port=1082
time="04-11 17:05:15" level=info msg="using proxy: http" host=<IP_ADDRESS> port=1082

Check your proxy logs. Maybe it's an issue with your proxy? What proxy are you running?
If you run time docker pull node:14.21, how long does it run before failing? It could be a timeout somewhere.

I just added the registry-mirrors in config file, changed the proxy agent and restart OrbStack, it works well, thank you!

Did you find out if the proxy was the issue or not? There could be some sort of bug in our HTTPS proxy support, and if so, it'd be great to get to the bottom of it so we can fix it.

I cannot be certain about the cause of this, but I restored it to original state and it works well again.


Texturerect, Stretch mode tile works with .PNG but not with .tres

Godot version
4.0.3
System information
windows 10
Issue description
In texture rect, stretch more doesn't work when used with .tres atlastexure.

Steps to reproduce
Create texturerect,
Sdd a .tres atlasTrexture to it,
Set the tile mode to tile,
Stretch the texturerect,
Minimal reproduction project
tile bug report.zip

Thanks for the report! Consolidating with https://github.com/godotengine/godot/issues/20472.


CASMNET-1165 Complete adding edge routers to SLS - allow qty adjustment

Summary and Scope
CASMNET-693 included every requirement of CASMNET-1165.  This completes CASMNET-1165 by allowing adjustment of the number of edge routers, for test and development environments.  This only adds optional new flag of --number-of-chn-edge-switches with a default value of 2 (as seen on most normal environments.
Issues and Related PRs

Resolves CASMNET-1165

Testing
Tested by adjusting data using Surtur and Wasp input.  Wasp is an actual use case for this.
Risks and Mitigations
None, this is an override with "normal system" defaults.

The merge conflicts still need to be resolved before this can be reviewed.

Closing this as a much later PR was merged before this and included the fix presented here.


support the helm command can be directly imported externall

To enable external programs to directly manage helm commands, it is recommended that function newRootCmd be exposed, which can be modified to NewRootCmd.

duplicate of https://github.com/helm/helm/pull/10412.


Rejects | in username.

<@kashire|t420> *vomits*
*cricket noises*

<@kashire|t420> vomits
cricket noises

The latest commit fixes this!
Thanks for the report
*cheek pinch*


Latest version broke Symbol Browser

The latest version has completely broken the symbol browser in ST3 where the window is filled with everything
WITHOUT:

Installed:

As you can see with it being installed, it's completely filled the symbol browser with everything single variable or mention of a variable in the current file

Sorry for the issue and my late reply. I added too many things to the symbols list. I'll remove everything else in the symbols list except the mixin and function definition. Does that sound good? Or what other elements would you like to list in the symbols list?

Is it possible to a set a scope as well, as this is affecting non-scss files? Mixins and functions are a good idea though.

This was also reported on the Sublime Text forum:

https://forum.sublimetext.com/t/goto-symbol-not-working/33653 Goto Symbol not working?

The user had to revert Sublime Text to a clean install, and reintroduce his packages until find out this package was causing the problem.

Your preferences selector was the following:
<string>entity.name.tag, entity.other.attribute-name.id, entity.other.attribute-name.class, variable</string>

Any region matching that selector in any view would get added to the symbol list. Notably, each variable occurance (not definition!). By convention, definitions are scoped as entity.name.<something>, e.g. entity.name.function, and you should look into adding that to the symol list. You should also tie the selector to your syntax only by being more explicit, e.g. entity.name.function.sass or source.sass entity.name.

I am really sorry about the issue.

Great thanks,


add wav2vec2_base_ll60k

#434
wav2vec2-base trained on EN with librilight (Reference page)

Added converted checkpoint & sample hidden states to huggingface
Tried model on SUPERB PR task

Thanks @vectominist !


Accuracy always 1 when using LSTM

Hi,
I am modelling a simple text classification task with 3 target labels. I am using word2vec embeddings as input to the LSTM layer.
sequence_size is the size of each sequence. dimsize is the word2vec dimension.
X_train is a 3D tensor of the shape (n_samples, sequence_size, dimsize)
y_train is a simple vector of labels like [0,2,1,0,1,0,...]
model = Sequential()
model.add(LSTM(400, input_shape=(sequence_size, dimsize)))
model.add(Dropout(0.5))
model.add(Dense(1))
model.add(Activation('sigmoid'))
model.compile(loss='binary_crossentropy',
              optimizer='adam')

batch_size = 32
model.fit(X_train, y_train, batch_size=batch_size, nb_epoch=3,
          validation_data=(X_test, y_test), show_accuracy=True)
score, acc = model.evaluate(X_test, y_test,
                            batch_size=batch_size,
                            show_accuracy=True)

But while training this model, I always get accuracy scores of 1 like this:
Epoch 1/3 31168/47226 [==================>...........] - ETA: 2142s - loss: -8.0915 - acc: 1.0000
What could be the issue here?
PS: I am using Keras 0.3.2

I am modelling a simple text classification task with 3 target labels.

The model in your has only one output...

y_train is a simple vector of labels like [0,2,1,0,1,0,...]

y_train has to be >= 2D, e.g. (n_samples, output_dimsize)

Thanks @NasenSpray, that solves it. I didn't find it mentioned somewhere in the docs though, and having used sklearn for long, I just assumed y to be one dimensional.

@ankeshanand，I am doing Chinese text setiment classification（positive and negative）and plan to use word2vec、LSTM 。I want to know what your accuracy  can reach，more than 90%？

I wan to know how do you put your wordvec into embedding level

Can you share your entire code plz? the fixed one


Calling toString on the notification atKey has atsign twice prefixed and appended

Describe the bug
When atKey.toString is called on the key received in the notification callback, the resultant has the duplicate atsign's.
For example, the key looks like @baboon9blue:@baboon9blue:shared_key@81bored7@81bored7 instead of @baboon9blue:shared_key@81bored7.
Steps to reproduce

Send a notification of the type messageType.key from the server
On the at_client SDK, when the notification is received, call toString on the atKey and the resultant has atsign twice prefixed and appended.

Root Cause:
In the update_verb_handler.dart, we prefix and append the sharedWith and sharedBy to the key before sending the notification. So, the key in the notification looks like "@bob:phone@alice"; when logging the key, we have an atKey.toString which will again add sharedWith and sharedBy to the key. Hence the atsign's are shown twice
Expected behavior
When toString is called on the atKey, the resultant should have atSign's prefixed and appended only once.

@gkc and @VJag : Below is the approach to fix this issue. Please review and let me know if there needs a modification in the approach.

In the update_verb_handler.dart and notify_verb_handler.dart, do not concatenate sharedWith and sharedBy atSigns to the key. In the update_verb_handler.dart, introduce a new variable that will hold the fully formed key (sharedWith:key@sharedBy) to insert/fetch the key from the key store.

With the above changes in the notification key store, the AtNotification->notification field will have only a key ('phone') unlike earlier, a fully formed key ('@bob:phone@alice')

In the at_client_sdk, when returning the notification information from the monitor connection, to ensure backward compatibility, if the key starts with  '@' then return the key as is; else concatenate sharedWith and sharedBy to the key and return.

I don't believe this is the right way, as it changes existing behaviour.
Can we come up with a simpler solution?

This changes existing behaviour, so it feels risky. In particular, this bit:

With the above changes in the notification key store, the AtNotification->notification field will have only a key ('phone') unlike earlier, a fully formed key ('@bob:phone@alice')

However clearly it is incorrect for the server to be prepending sharedWith and appending sharedBy to what is already a well-formed AtKey, so we need to do something.
It is strange that we have a clear design / intent inconsistency which has been around for a long time and yet it isn't affecting us except in this one situation ... can we lay out the problem a bit more clearly, explaining end to end (atsign1 client to atsign1 secondary to atsign2 secondary to atsign2 client) what is currently happening (a) for updates (b) for notifys ?

Spent 3 SP finding the root cause of the issue and a fix in PR-49. Moving to PR-50 for the remaining work.

@sitaram-kalluri Can you pick this up in PR56 please?

@sitaram-kalluri Can you pick this up in PR56 please?

@gkc: Sure

The changes are merged to the trunk branch.


ColorPicker: Improve the accuracy of hue slider in OKHSL mode

This PR improve the accuracy of hue slider in OKHSL mode, especially when lightness and saturation are high, by using dynamically generated gradient texture and removing color_okhsl_hue theme property.
Before:

After this PR:

Is it a good design choice that the hue is now affected by the lightness?


Object not set to an instance of an object

Getting "object not set to an instance of an object" error when using RouteRequest with over 25 waypoints.  After testing it seems to be due to using RouteAttributeType.ExcludeItinerary.  Removing that type stops error.
StackTrace	"   at BingMapsRESTToolkit.RouteRequest.<>c__DisplayClass22_0.b__0() in C:\Users\richbrun\Desktop\Code Samples\GitHub\BingMapsRESTToolkit\Source\Requests\RouteRequest.cs:line 443\r\n   at System.Threading.Tasks.Task`1.InnerInvoke()\r\n   at System.Threading.Tasks.Task.Execute()"

Closing as unable to reproduce and a lot has changed since this bug was reported.


Scroll function bug

Hello! I want is when a text box is being focus, I want to scroll down to bottom of the page,
but it seems not working.
    $$('input[type="password"]').focus( function() {
        window.scrollTo(0, document.body.scrollHeight);
    });

does somebody know why?

You need to set scroll on "page-content":
    $$('input[type="password"]').focus( function() {
        var pageContent = $$('input[type="password"]').parents('.page-content');
        pageContent.scrollTop(pageContent[0].scrollHeight)
    });


[impellerc] add shader define for Y coordinate scale

For example, something like FLUTTER_Y_AXIS_SCALE. to be used by shaders to determine opengl vs metal/vulkan y-axis

Today, we are using uniforms passed in to the appropriate shaders to achieve this. For instance, texture_sampler_y_coord_scale in texture_fill.vert.
Not sure how valuable having a shader define would be be since it's hard to image how a flip could be applied without context.

Also, Vulkan applies the y-axis flip using negative viewports from VK_KHR_maintenance1. So Vulkan behaves just like Metal.


Add Facebook Clone

I can add facebook clone made using html and tailwind css. I am participant of GS-SOC 2022. Please allow me.

Hi @Vaib215    you can work on this issue ur project should have a working link and you should add that in the main website too once done with creating it .
Deadline -  5-6 days

I have created the clone. Kindly merge the PR.


Set up release/ branches to support different Cedar releases

Most of the cedar* repositories use branches to track releases, while the main branch is used for development. We should store integration tests for the different Cedar releases as well. (At present, this repository is only set up to track the main branch of cedar and cedar-spec).

Update: this has been done for versions 3.2.x and 3.3.x, but leaving this issue open until we have a branch for the latest 2.x release (2.4.x).


[windows] [bug] Windows 10 yellow exclamation mark (no internet connection)

I'm using dnscrypt-proxy version 2.0.25 (latest) running as a service
using cloudflare dns over https (attached config below).
Windows 10 shows the yellow mark "no internet connection" even if internet connection is present, this is annoying and also prevents me from turning the pc into a wifi hotspot.
This happens both with wifi and ethernet and doesn't happen if i change the system resolver from <IP_ADDRESS> to <IP_ADDRESS> (if i don't use dnscrypt-proxy).
The yellow mark appears immediatly after i plug the ethernet cable.
~Is my configuration wrong or it's a bug of the program?~
The problem is present even with default config.
Tested also over windows 7 and 8.1, only win 10 has the problem, same config.
On win 10 the yellow mark sometimes goes away but after long time
server_names = ['cloudflare']
listen_addresses = ['<IP_ADDRESS>:53']
max_clients = 250
ipv4_servers = true
ipv6_servers = false
dnscrypt_servers = false
doh_servers = true
require_dnssec = true
require_nolog = true
require_nofilter = true
disabled_server_names = []
force_tcp = false
timeout = 2500
keepalive = 30
refused_code_in_responses = false
log_level = 2
log_file = 'dnscrypt-proxy.log'
cert_refresh_delay = 240
fallback_resolver = '<IP_ADDRESS>:53'
ignore_system_dns = true
netprobe_timeout = 0
netprobe_address = "<IP_ADDRESS>:53"
log_files_max_size = 10
log_files_max_age = 7
log_files_max_backups = 1
block_ipv6 = false
cache = true
cache_size = 512
cache_min_ttl = 600
cache_max_ttl = 86400
cache_neg_min_ttl = 60
cache_neg_max_ttl = 600
[query_log]
file = 'query.log'
format = 'tsv'
[nx_log]
file = 'nx.log'
format = 'tsv'
[blacklist]
[ip_blacklist]
[whitelist]
[schedules]
[sources]
[static]
[static.'cloudflare']
stamp = 'sdns://AgcAAAAAAAAABzEuMC4wLjGgENk8mGSlIfMGXMOlIlCcKvq7AVgcrZxtjon911-ep0cg63Ul-I8NlFj4GplQGb_TTLiczclX57DvMV8Q-JdjgRgSZG5zLmNsb3VkZmxhcmUuY29tCi9kbnMtcXVlcnk'
[static.'cloudflare-ipv6']
stamp = 'sdns://AgcAAAAAAAAAGVsyNjA2OjQ3MDA6NDcwMDo6MTExMV06NTOgENk8mGSlIfMGXMOlIlCcKvq7AVgcrZxtjon911-ep0cg63Ul-I8NlFj4GplQGb_TTLiczclX57DvMV8Q-JdjgRgSZG5zLmNsb3VkZmxhcmUuY29tCi9kbnMtcXVlcnk'

for windows 10 I have always used dnscrypt simple which is nothing more than dnscrypt with GUI and I noticed that setting it also showed them the yellow symbol but the navigation went quietly, it was enough for me to disable and activate the lan card to make the yellow symbol disappear

Have you tried start dnscrypt-proxy not as service, just only binary from execution directory?
Have you tried -resolve option (read wiki, pls.)?

Have you tried start dnscrypt-proxy not as service, just only binary from directory where config and binary (better run in cmd)?

Not yet

Have you tried -resolve option (read wiki, pls.)?

No, but i have changed system dns to <IP_ADDRESS> and nslookup/internet works only if the service is started so i think config is fine, i'll try both tomorrow.
@tigernero79
thanks, i'll try that workaround tomorrow but it's not a definitive solution.

Does your ISP support IPv6 ? If not, remove cloudflare-ipv6.

@r4sas
Running dnscrypt-proxy from (admin) cmd leads to the same results (yellow triangle)
According to -resolve output the program is correctly configured (resolver is cloudflare and things doesn't resolve if i stop the service/the exe from cmd)
@uBlock-user
ipv6 is not enabled in the posted config
@tigernero79
Tested disable and re-enable the network card, it worked once in 10 attempts. yellow triangle is almost always there (internet always works)

I meant [static.'cloudflare-ipv6'] from the dnscrypt config file. Disable that or remove it.

it's not used as the only used one are in this list: server_names = ['cloudflare'] as you can see cloudflare-ipv6 is not in the list so it doesn't matter if i left it defined. anwyay the problem exists also with default configuration

I am also facing this problem after win10 v1903 - using Simple DNSCrypt
most of the time due to this problem, apps with Microsoft account (Store, Mail, Weather etc) does not work and reports No Internet
switching back to router DNS or <IP_ADDRESS> removes the exclamation immediately and fixes such problems
have tried with default configs also but no gain

@kamilmirza @tigernero79
i have found how the internet detection is done and how to disable it
Apparently windows connects to this host and check for a file: msftncsi.com/ncsi.txt
It also checks the ip of dns.msftncsi.com and compare to hardcoded value
Workaround: win+r open regedit and go here:
HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Services\NlaSvc\Parameters\Internet
Set EnableActiveProbing to 0 and reboot
some commands that might be useful to the author for debugging
Resolution when dnscrypt-proxy is active:
>ipconfig /flushdns
>nslookup dns.msftncsi.com
Server:  localhost
Address:  <IP_ADDRESS>

Risposta da un server non autorevole:
Nome:    dns.msftncsi.com
Addresses:  fd3e:4f5a:5b81::1
          <IP_ADDRESS>

>dnscrypt-proxy.exe -resolve dns.msftncsi.com
Resolving [dns.msftncsi.com]

Domain exists:  probably not, or blocked by the proxy
Canonical name: dns.msftncsi.com.
IP addresses:   <IP_ADDRESS>
TXT records:    -
Resolver IP:    <IP_ADDRESS>

With dnscrypt-proxy disabled, using assigned dns:
>nslookup dns.msftncsi.com
Server:  dns.google
Address:  <IP_ADDRESS>

Risposta da un server non autorevole:
Nome:    dns.msftncsi.com
Addresses:  fd3e:4f5a:5b81::1
          <IP_ADDRESS>

>dnscrypt-proxy.exe -resolve dns.msftncsi.com
Resolving [dns.msftncsi.com]

Domain exists:  probably not, or blocked by the proxy
Canonical name: dns.msftncsi.com.
IP addresses:   <IP_ADDRESS>
TXT records:    -
Resolver IP:    <IP_ADDRESS>

workaround works perfectly
no exclamation after reboot/login
all apps are working fine now

This is probably same as https://github.com/jedisct1/dnscrypt-proxy/issues/816 (which I'd say shouldn't have been closed). The problem is that you want to forward some local addresses to your internal resolver on modem. But this forwarding as you can see in the bug too doesn't work reliably, which I also reproduced on my system.

Local Group Policy fix from this link works fine
https://support.umbrella.com/hc/en-us/articles/230900948-Umbrella-Roaming-Client-Microsoft-Windows-Limited-Network-Connectivity-Warning-Yellow-Triangle-

May someone describe the problem and workaround on the Wiki?

May someone describe the problem and workaround on the Wiki?

While the workaround seems ok, https://github.com/jedisct1/dnscrypt-proxy/issues/898 shows yet again forwarding is not working as expected. If it did the workaround wouldn't be needed.


Bug fix for Multiple appointments with same mail id

Issue: Doctor is not able to generate different prescriptions for multiple appointments with same mail id, showing a View button instead.
Fix: Changed the code to get the details of consulted patients based on Mail Id, Name and consultation time, rather than using only mail Id.

Pushing changes for bug - "Doctor is not able to generate different prescriptions for multiple appointments with same mail id, showing a View button instead".


Remove usage of glib_floating_reference_guard!()

It's not needed anymore.

👍
Please merge once CI is green.


feat(search): add enpoint create or update temporal representation of entity

you need to rebase it (after the merge of the refactoring this morning).


Conflict with tinymce

Tinymce's toolbar (bold, italic etc buttons) doesn't work properly, cursor moves to initial position and on clicking again leaves new line until refresh the whole app. though I am using in different components. Is there any way to destroy monaco editor when component destroy. Tinymce does same while changing component.?

Fixed with removing editor.on('Change', () => { })


⚠ Bofrost.lu has degraded performance

In fa87b43, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1453 ms

Resolved: Bofrost.lu performance has improved in 10e598e after 17 minutes.


Service gateway

New services added, no risk with previous code

Added network-ddetails component and patch tests networks detail compare are commits for other pr

Added network-ddetails component and patch tests networks detail compare are commits for other pr

Two same commits => Added new model for gateway

Two same commits => Added new model for gateway


WFRP 2e Character Sheet Title

Update the approved.yaml with the edition key.

Attaching a patch.

patch-issue1537.diff.zip

I agree, and the other WFRP games don't follow the same naming convention. I'm going to go ahead and standardize them all as well.

if this is fixed can we close the issue?

yes. I'll close.


compact

A category is compact closed when it is both rigid and symmetric, see an implementation here.

Addressed in https://github.com/discopy/discopy/pull/129


Loading completion script with snippet doesn't work

Describe the bug
Loading completion script for exa with snippets doesn't load the completions for that script.
To Reproduce
Steps to reproduce the behavior.
zinit ice as"completion"
zinit snippet https://github.com/ogham/exa/blob/master/contrib/completions.zsh

Expected behavior
Completions for exa should work when  is pressed after exa  but it doesn't show the possible values.
Screenshots
If applicable, add screenshots to help explain your problem.
Versions:

$ZSH_VERSION == 5.8
Operating system name and version (uname -a): Linux DESKTOP-N1PM98I 4.19.104-microsoft-standard #1 SMP Wed Feb 19 06:37:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Load the file as follows:
zinit as"completion" mv"c* -> _exa" for https://github.com/ogham/exa/blob/master/contrib/completions.zsh

It has to be named with the leading underscore for the as'completion' to work.

That works perfectly. Thanks :)


Implement Exactly method

Have you considered an "Exactly-Where" overload like
Exactly<T>(this IEnumerable<T> collection, Func<T, Boolean> condition, int count)

@JamesFaix I considered such an overload myself, when I wrote a similar method.
I looked at other extension methods and saw that very LINQ methods took more than one argument.
I came to the conclusion that list.Where(condition).Exactly(count) was at least as or even more readable than list.Exactly(condition, count).

This was implemented in #197.


npm start fails on windows

Using git bash on Windows 10, npm start returns:
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>watch script '$(npm bin)/budo src/module.js:dist/tangram.debug.js --port 8000 --cors --live -- -t [ babelify --presets [ es2015 ] ] -t brfs -s Tangram -p [ './build/quine.js' 'tangram.debug.temp.js.map' ] -p [ mapstraction 'dist/tangram.debug.temp.js.map' ]'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the tangram package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     $(npm bin)/budo src/module.js:dist/tangram.debug.js --port 8000 --cors --live -- -t [ babelify --presets [ es2015 ] ] -t brfs -s Tangram -p [ './build/quine.js' 'tangram.debug.temp.js.map' ] -p [ mapstraction 'dist/tangram.debug.temp.js.map' ]

However, copying and pasting that literal command works:
$ $(npm bin)/budo src/module.js:dist/tangram.debug.js --port 8000 --cors --live -- -t [ babelify --presets [ es2015 ] ] -t brfs -s Tangram -p [ './build/quine.js' 'tangram.debug.temp.js.map' ] -p [ mapstraction 'dist/tangram.debug.temp.js.map' ]
[0012] info  Server running at http://<IP_ADDRESS>:8000/ (connect)
[0012] info  LiveReload running on 35729
[0086] 1m 13.6s    1.6MB (browserify)

The problem seems to be the way environment variables are referenced from within unix and windows shells. In this case, when npm start runs it starts a subprocess which uses a Windows shell, which expects the %NAME% pattern, so $(npm bin) can't resolve.
I don't know if there's a way control what kind of shell is used for a subprocesses in a cross-platform script. See https://www.npmjs.com/package/cross-env for more on this.

Fixed by https://github.com/tangrams/tangram/pull/617


ZMQ socket monitoring refactor

Current dependencies on/for this PR:

main

PR #51

PR #52   👈

PR #53

This comment was auto-generated by Graphite.


hi

Welcome to Hacktoberfest 2017!
The goal of this repo is to help beginners who are doing their first pull requests. Feel free to join!
Instruction
try to solve issues.
How to Create a Pull Request

Click on the fork on the top to fork this repo.
Go to your repo where you fork the project.
Hit the clone button on your forked repo and copy the given link.
On your terminal/command prompt, type "git clone [put the link here]".
Add your name in the CONTRIBUTORS.md.
Afterward, on your terminal/command prompt, type "git add CONTRIBUTORS.md".
then type 'git commit -m "[type a message]" '.
Push the commit. then type "git push origin master".
Go back to the original repo.
Hit "new pull request" and compare between forks.
Confirm the pull request and that's it!

Installation
Make sure git is installed.

Hi, :)


Memory and cpu increases when server goes down (socket close)

I have noticed a problem in my code or with the node-ddp-client package. When i have have a node-ddp-client connected to the server, and the server for whatever reason goes down, the cpu usages and the memory of the node script goes up.
Is there anything i could do to change this behavior?
Note: This behavior occurs also when the only thing the script does is to make a connection to the server. and no other code is running
Greetings,
Etienne

Are you saying there is a memory leak, or just that connecting is more resource intensive that listening?

Well i dont know if its a memory leak. I'am not qualified enough to determin this. But maybe somebody could reproduce it, by just creating a node-ddp connection. Then when te connection is  established. Stop the Meteor server, and see if memory consumption of the ddp-client is increasing.  Let it just run for 10, 15 minutes.


Make new plugin command

MarkUpUpDownDown should have a new plugin command to facilitate expanding the ecosystem.

Running markupupdowndown new plugin foo will produce a directory that looks like this:
$ find foo/
foo/
foo/setup.py
foo/README.md
foo/MANIFEST.in
foo/.gitignore
foo/foo
foo/foo/__init__.py

The MANIFEST.in file and setup.py are both constructed such that they will package up any static data, such as found in skel/ directories.
The rest of the files are generic, to provide generic answers to the first few questions involved with building a plugin.


--stop-on-failure doesn't stop

I recently updated to v0.4.3 and noticed that adding the --stop-on-failure flag doesn't stop executing when a scenario fails.

thanks for noting, will need a test for it

Created the simplest test I could think of at https://github.com/omnicolor/godog-stop-on-failure

ah, you misunderstood me :) I've meant that I'll need to write a godog test scenario to prevent regression, cheers ;)


Try to detect unused modules

👍 This is definitely something I need right now. :)

@Phil0xF7 PR welcome :-)


enable custom requirements-uninstall.txt

This change enables to run a custom pip uninstall immediately before pip-install, e.g. to refresh some packages:
Usage

add requirements-uninstall.txt
push

How it works
steps/pip-install will detect requirements-uninstall.txt, pip uninstall all requirements in it (including support for -r inclusion of other requirements). Immediately afterwards, continues with pip install.

The python buildpack already automatically detects packages that have been removed from requirements.txt, and will uninstall them automatically.

Yes, yet that's not the point of this PR. This PR is about reinstalling packages that continue to stay in requirements.txt and thus are not detected by the stale mechanism.
The specific scenario this PR solves is this, which is not at all addressed by the current buildpack.
requirements.txt contains the following
-r requirements-public.txt
-r requirements-custom.txt
In -public, packages from the public repository are installed. In -custom, packages from a private repository are are installed. There are circumstances when you want to uninstall (and possibly reinstall the packages in -custom), e.g. to get a new version during a development/testing deploy. Of course we all know that you're supposed to bump your version when you start development of the new version, but we also all know that this is not realistic in practice.  This PR is making this scenario practicable.
Besides it is a common problem:

http://stackoverflow.com/questions/15046378/manually-remove-python-package-on-heroku
http://stackoverflow.com/questions/8937905/how-to-pip-uninstall-with-virtualenv-on-heroku-cedar-stack

Can you be specific about a use-case that you are solving for? I understand your thought process, but I don't understand how this necessary.