mutt: 1.11.2 -> 1.11.4

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/mutt/versions.
meta.description for mutt is: '"A small but very powerful text-based mail client"'.

Checks done (click to expand)

built on NixOS
/nix/store/fb5qm27k4ib61vsfmbzr11aay5b115bf-mutt-1.11.3/bin/mutt passed the binary check.
/nix/store/fb5qm27k4ib61vsfmbzr11aay5b115bf-mutt-1.11.3/bin/mutt_pgpring passed the binary check.
Warning: no invocation of /nix/store/fb5qm27k4ib61vsfmbzr11aay5b115bf-mutt-1.11.3/bin/pgpewrap had a zero exit code or showed the expected version
/nix/store/fb5qm27k4ib61vsfmbzr11aay5b115bf-mutt-1.11.3/bin/muttbug passed the binary check.
/nix/store/fb5qm27k4ib61vsfmbzr11aay5b115bf-mutt-1.11.3/bin/flea passed the binary check.
Warning: no invocation of /nix/store/fb5qm27k4ib61vsfmbzr11aay5b115bf-mutt-1.11.3/bin/smime_keys had a zero exit code or showed the expected version
4 of 6 passed binary check by having a zero exit code.
0 of 6 passed binary check by having the new version present in output.
found 1.11.3 with grep in /nix/store/fb5qm27k4ib61vsfmbzr11aay5b115bf-mutt-1.11.3
directory tree listing: https://gist.github.com/0cf66112cbdae26fddde9e79c71a1414
du listing: https://gist.github.com/8b8d75e97af7912f96f949c5de47e96b

Rebuild report (if merged into master) (click to expand)

12 total rebuild path(s)
3 package rebuild(s)
3 x86_64-linux rebuild(s)
3 i686-linux rebuild(s)
3 x86_64-darwin rebuild(s)
3 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
grepm
mutt
mutt-with-sidebar

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fb5qm27k4ib61vsfmbzr11aay5b115bf-mutt-1.11.3 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A mutt https://github.com/r-ryantm/nixpkgs/archive/3382e1744b9be6f932308c432562de74979c5b91.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fb5qm27k4ib61vsfmbzr11aay5b115bf-mutt-1.11.3
ls -la /nix/store/fb5qm27k4ib61vsfmbzr11aay5b115bf-mutt-1.11.3/bin

cc @the-kenny @rnhmjoj for testing.

@GrahamcOfBorg test mutt
@timokau feel free to merge unless there are further objections :)

Thanks, for some reason there is a merge conflict now though? Maybe based your changes on an older nixpkgs?

@timokau not sure why this happened, I simply checked this branch out using hub. Now rebased onto a recent master and it appears to be mergable again.

Great, lgtm now :) Thanks!


[master] Mergebot Automated Train PR - 2020-Mar-30-12-30

The branch was created by mergebot.

I had to manually create a pull request from the branch because the mergebot was running into a Exception with Github API
GithubException: 422 {u'documentation_url': u'https://developer.github.com/v3/pulls/#create-a-pull-request', u'message': u'Validation Failed', u'errors': [{u'field': u'head', u'code': u'invalid', u'resource': u'PullRequest'}]}

The EE PR will be created from the bump-ee branch of this.

@mesosphere-mergebot bump-ee

@mesosphere-mergebot sync

@mesosphere-mergebot merge-it


Update the README.md to highlight the purpose of this repo

Currently, the README is still a bit cryptic about the purpose of this project. It should be clear why this project exists and what it is trying to solve and how.
At a glance, the project takes care of two things:

reporting to the community on the release team's shadow program to increase transparency and use that as a basis for improvement.
filtering and converting the data into a readable format (markdown) that can be easily read in a selection process (not walking the excel sheet anymore).

This should be included and expanded in the README.
cc @justaugustus @jeremyrickard
/assign @leonardpahlke

/assign @s04

/assign @liquidsaul

ref tracking issue https://github.com/kubernetes-sigs/release-team-shadow-stats/issues/4

in progress...
first iteration...
see fork https://github.com/leonardpahlke/release-team-shadow-stats/tree/update-readme


[pwm/scoreboard] Added heartbeat, phase and resolution

Added heartbeat mode, phase calculation and resolution to generate_exp_item task.
Signed-off-by: Jason Hoddinett<EMAIL_ADDRESS>

Looks like there is some CI failure:

Error-[IND] Identifier not declared has 1 failures:

Test cover_reg_top has 1 failures.

cover_reg_top
Line 1174, in log /azp/agent/_work/1/s/scratch/HEAD/pwm-sim-vcs/cover_reg_top/build.log
  Error-[IND] Identifier not declared
  ../src/lowrisc_dv_pwm_env_0.1/pwm_scoreboard.sv, 303
    Identifier 'LargerB' has not been declared yet. If this error is not
    expected, please check if you have set `default_nettype to none.

It says The second line of a commit message must be empty, as it separates the summary from the long description.
Example in workspace:

Seems like the CI complains about the sign-off names again.
Commit has no Signed-off-by line. The commit message must contain a Signed-off-by line that matches the commit author name and email, indicating agreement to the Contributor License Agreement. See CONTRIBUTING.md for more details. You can use "git commit --signoff" to ask git to add this line for you.

You might need to do a git commit --amend and fix that.

This is where you can see the commit message ::)

sorry again Jason, CI still complains about the signature:
Commit b51c8ffa57a6d70082a04a4d72063da7af1fe04b: The commit author name 'jason-hoddinett' contains no space. Use "git config user.name 'Johnny English'" to set your real name, and update the commit with "git rebase -i " and/or "git commit --amend --signoff --reset-author".  See CONTRIBUTING.md for more details.

Also we since the last two commit kind of serve the same purpose, maybe it is easier to squash them into one commit:
You can do it by this: https://www.internalpointers.com/post/squash-commits-into-one-git
Hope it is not too much trouble. Thank you!


[Product Page] Landing page authoring

https://main--fonterra-fernleaf--hlxsites.hlx.page/en/products
authoring for this needs a fix, products seem to have missing links, and some point to recipes. also, few more products to be added as compared to original site

@asthabh23  This is corrected now. Please validate.

Validated, lgtm!


[plugins] Support private GitHub repositories

Summary
Fixes #1918
Plugins stored in a private GitHub repository fail to pull due to due to a lack of authentication. This PR will add an Authorization: token <GITHUB_TOKEN> header to the request if the GITHUB_TOKEN environment variable is set.
How was it tested?
Added test cases for http.Request with and without environment variable.
Manually tested on my machine:

Added a private GitHub repo to devbox.json "include"
devbox shell: confirmed existing 404 error
GITHUB_TOKEN=xxx devbox shell: shell environment boots correctly

Thanks @mikeland73, my team will really appreciate this. I'll try to incorporate your suggestions in a follow-up PR.


check_cflags iterates over each letter, not each option of the .json file

Take this test:
    {
      "dependency": "check_cflags",
      "type": "cflags",
      "cflags": [
        "-Wno-cpp",
       "-Wno-#warnings"
      ],
      "comment": "append to CFLAGS_NO_SOURCE_WARNINGS",
      "append_to": "CFLAGS_NO_SOURCE_WARNINGS"
    },

config.log shows:
Command: gcc -Werror -Wno-cpp -Wno-#warnings /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmp4w3xl9af.c -o /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmp4w3xl9af.c-bin
exit code: 1
Command output(stderr):
error: unknown warning option '-Wno-cpp' [-Werror,-Wunknown-warning-option]

Compiling source code:

int main(int argc, char **argv){
  return 0;
}
Command: gcc -Werror  - /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpw9fqd2ie.c -o /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpw9fqd2ie.c-bin
exit code: 1
Command output(stderr):
clang: error: -E or -x required when input is from standard input

Compiling source code:

int main(int argc, char **argv){
  return 0;
}
Command: gcc -Werror  W /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpboayr9fu.c -o /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpboayr9fu.c-bin
exit code: 1
Command output(stderr):
clang: error: no such file or directory: 'W'

Compiling source code:

int main(int argc, char **argv){
  return 0;
}
Command: gcc -Werror  n /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpcqptonzx.c -o /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpcqptonzx.c-bin
exit code: 1
Command output(stderr):
clang: error: no such file or directory: 'n'

Compiling source code:

int main(int argc, char **argv){
  return 0;
}
Command: gcc -Werror  o /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpe2x8dkam.c -o /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpe2x8dkam.c-bin
exit code: 1
Command output(stderr):
clang: error: no such file or directory: 'o'

Compiling source code:

int main(int argc, char **argv){
  return 0;
}
Command: gcc -Werror  - /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmp7eyk2rke.c -o /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmp7eyk2rke.c-bin
exit code: 1
Command output(stderr):
clang: error: -E or -x required when input is from standard input

Compiling source code:

int main(int argc, char **argv){
  return 0;
}
Command: gcc -Werror  c /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpw3lfiakk.c -o /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpw3lfiakk.c-bin
exit code: 1
Command output(stderr):
clang: error: no such file or directory: 'c'

Compiling source code:

int main(int argc, char **argv){
  return 0;
}
Command: gcc -Werror  p /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmp03dgbpma.c -o /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmp03dgbpma.c-bin
exit code: 1
Command output(stderr):
clang: error: no such file or directory: 'p'

Compiling source code:

int main(int argc, char **argv){
  return 0;
}
Command: gcc -Werror  p /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpvrbjqled.c -o /var/folders/_q/byf907ss6bb130qvj9fqk2nr0000gq/T/tmpvrbjqled.c-bin
exit code: 1
Command output(stderr):
clang: error: no such file or directory: 'p'

Compiling source code:

int main(int argc, char **argv){
  return 0;
}

Note how it tests each letter after "-Werror"

ok, I looked dependency-resolver.py and it is wrong handling failure. I'll fix and send a patch

See #1344

Fixed by 93210b176e6bd62e8d76748cabdbb2a546c5eac2.


Vim binding support

Issue
Could we get Vim keybindings, similar to the ones in Zsh? Especially missing Esc+k to switch to the last command and also general editing of the current text
Version and model info

Thanks for trying aider and filing this issue. This doc may be helpful:
https://aider.chat/docs/usage/commands.html#vi


trying to create wlan networks

I'm trying to generate a few wifi networks.
In my limited python knowledge i've copied and modified the create_network to:
create_wlan = UnifiAPICall(
    "Create a wifi network",
    "rest/wlanconf",
    json_body_name='settings',
    method="POST",
    )

Now when i try to create a test network i get the following error:
Traceback (most recent call last):
File "<pyshell#188>", line 1, in
client.create_wlan(settings={'name': 'test1200_Office', 'usergroup_id': '60f9488e46e0fb012814562b', 'wlangroup_id': '60f9488e46e0fb012814562c', 'enabled': True, 'hide_ssid': False, 'is_guest': False, 'security': 'wpapsk', 'wpa_mode': 'wpa2', 'wpa_enc': 'ccmp', 'vlan_enabled': True, 'uapsd_enabled': False, 'x_passphrase': 'supergeheimwachtwoord', 'schedule_enabled': False, 'schedule': [], 'vlan': 10})
File "C:\Users\marijn\AppData\Roaming\Python\Python39\site-packages\unificontrol\metaprogram.py", line 125, in wrapper
return instance(client, *a, **kw)
File "C:\Users\marijn\AppData\Roaming\Python\Python39\site-packages\unificontrol\metaprogram.py", line 103, in call
return client._execute(url, self._method, rest_dict, need_login=self._need_login)
File "C:\Users\marijn\AppData\Roaming\Python\Python39\site-packages\unificontrol\unifi.py", line 115, in _execute
raise UnifiTransportError("{}: {}".format(resp.status_code, resp.reason))
unificontrol.exceptions.UnifiTransportError: 400:
Any ideas what i'm doing wrong?
Thanks

I'm a step further, Seems like the value i was missing is the ap_group_ids, which on a new site without wifi networks are not yet in the wlanconf.
I'm now trying to figure out where i can get this value on a new site.

Thanks for looking into implementing new calls. Since Ubiquiti still don't document their API this can be a somewhat tedious process. In general the best option I've found is to open up the the debug tools in your web browser while using the Unifi controller web interface and examine the requests that their JavaScript UI makes.

By looking at the debug tools, there seems to be a way to create a new wlan group.
it's posting to https://controller-ip:10443/v2/api/site/zdypdga4/apgroups
{
"_id": "60f97979a13fe06510c85bb7",
"attr_no_delete": false,
"device_macs": [],
"name": "test"
}

It looks like I need to first create one, however i'm having some difficulties adding another api call which will get the current ap_group_ids, (which can be seen at: https://<IP_ADDRESS>:8443/v2/api/site/rr575ua3/apgroups), and post the new ap group to that url as well.
But i don't really understand how the unifiapicall function works, Would you be able to point me in the right direction?

https://controller-ip:10443/v2/api/site/zdypdga4/apgroups

Just want to thank you for your research. Very helpful link!


Add option for DatePicker to return calendar dates without time and timezone ("2018-10-15")

Is your feature request related to a problem? Please describe.
I'm using the current latest version with date-fns.
In my app, I need DatePicker to pick calendar date - where there is no notion of timezone. Yet by default the value returned by a DatePicker is a datetime including the timezone offset. So for instance, if the user picks today in the europe/paris timezone, it will be passed as "2018-10-14T22:00:00.000Z". Which is annoying because I need to know the user's timezone to know which day they meant.
Describe the solution you'd like
Add an option useCalendarDates?
Describe alternatives you've considered
I couldn't find a simple way to do this. I checked the method exposed in the utils, but they don't discriminate between DatePicker and DateTimePicker. I guess I could hijack the onChange callback.
Additional context
The Python standard library has datetime for specific point in times (timezone-aware and timezone-naive) as well as date objects for calendar days.
I checked #517 which is totally unrelated.

Well this is one way:
<DatePicker ... onChange={d => onChange(format(d, 'yyyy-MM-dd'))} />

You can pass Luxor or moment objects with configured time zone. And picker will return back the same time zone. But if you are using date-fns — it’s impossible

I think this is a valuable feature. You'd need a simple way of picking dates such as birthdays.


add fast_finish

Coverage remained the same at 83.529% when pulling 52903b67430251786787a9fb4a8962d424bd2bd2 on chylli:chylli/fast_finish into 6bde585f4541d17e759437100357d8733d08376d on binary-com:master.


BigQuery table insert documentation is incorrect and insufficient

Environment details

OS: OS X El Capitan
Node.js version: 6.9.x
npm version: 3.10.9
google-cloud-node version: 0.49

Issues

Incorrect Promise API array indexes in documentation
Incomplete table.insert PartialFailureError handling example in Promise API documentation

Steps to reproduce

Follow documentation here and write the following code:

    this.table.insert(rows, { raw: false }).then((data) => {
      let insertErrors = data[0];
      let apiResponse = data[1];
    }

Debug the code and notice that the API Response is in data[0] and that the insertion errors are in data[1]. This is the opposite of what the docs state.

The code makes sense because I don't want to be checking for the apiResponse in index 0 on success and index 1 on failure. 'apiResponse should be in index 0 as it is now and then index 1 will be empty or have the insertion errors if they exist. So this seems to be a docs issue rather than a code issue.
My working code is similar to this:
    this.table.insert(item, { raw: false }).then((data) => {
      let insertErrors = data[1];

      if (insertErrors) {
        logger.info(`insertErrors: ${JSON.stringify(insertErrors)}`);
        // Some rows failed to insert, while others may have succeeded.
        insertErrors.map((insertError) => {
          insertError.errors.map((error) => {
            logger.error(`PartialFailureError: BigQuery insert failed due to: ${JSON.stringify(error)}`);
          });
        });
      }
      return message;
    }).catch((error) => {
      throw new Error(`Error inserting into bigQuery for id: "${item.id}": ${JSON.stringify(error)}`);
    });

Now the callback docs for table.insert mention that you can do something like if (err.name === 'PartialFailureError') but using the Promise API, I never get back any objects with a name field. The apiResponse is of this format:

  "kind": "bigquery#tableDataInsertAllResponse",

And the insertErrors object is of this format:
  [
    {
      "index": 1,
      "errors": [
        {
          "reason": "invalid",
          "location": "name",
          "debugInfo": "generic::invalid_argument: Conversion from int64 to string is unsupported.",
          "message": "Conversion from int64 to string is unsupported."
        }
      ]
    },
    {
      "index": 0,
      "errors": [
        {
          "reason": "stopped"
        }
      ]
    }
  ]

So it's unclear why the docs indicate that you should check against the err.name field.
At minimum, if there is a significant difference for handling the response from the Promise API, then that needs to be documented but currently the docs for using the table.insert Promise API are bare bones and incomplete.
From this comment it looks like the tests are only written against the callback API and not the Promise API.

insertErrors appears to have been abandoned when we switched to the PartialFailureError concept. A PartialFailureError is an error, and it's meant to be handled like any other error in a promise, in the catch block.
table.insert(row)
  .then(function(data) {
    // All rows inserted successfully
    var apiResponse = data[0]
  })
  .catch(function(err) {
    if (err.name === 'PartialFailureError') {
      // Insert partially, or entirely failed
    } else {
      // `err` could be a DNS error, a rate limit error, an auth error, etc.
    }
  })

We will update the docs to show the above use and remove the insertErrors argument that no longer exists.

Interesting. I'm slightly confused as that contradicts what I was told earlier about PartialFailureErrors not being 'real' errors that should be thrown. Though I'm happy that it sounds like the error is being handled in a more Promise-friendly way.
I'm looking forward to reviewing the new docs and hopefully they will eliminate the remaining confusion.

Sorry for the confusion. PR sent in #2153


Fixes for issues not fully closed in backend refactoring [PR #134]

Description
Describe the content of the PR and links to related issues, bugs of features.
Requirements
Before submitting this PR, please make sure:

[ ] The code builds cleanly without new errors or warnings
[ ] The code passes all the existing tests
[ ] If this PR adds a new feature, new tests have been added to test these
new features
[ ] All relevant documentation has been updated or added

Additionally, if this PR contains code authored by new contributors:

[ ] All the authors are covered by a valid contributor assignment agreement,
signed by the employer if needed, provided to ETH Zurich
[ ] The names of all the new contributors have been added to an updated
version of the AUTHORS.rst file included in the PR

The main changes have been already added. The only missing comments from #134 still unadressed are:

in caching.py: "Can you elaborate on why are both  is_cache_info_consistent and is_cache_info_available_and_consistent methods needed? They seem a bit redundant since one looks like a stricter version of the other. Wouldn't it be simpler to define only one method with an extra parameter (e.g., ignore_unavailable, strict_validation, ...) to differentiate between the lenient and the strict mode?"

in backend/base.py: "Why are the two of them needed? I might be completely wrong, but my current understanding of the caching system is that it will ask the backend to generate the extra_cache_info dict for a specific stencil, and then either store it in a file record, or try to match it against the values previously stored there. If that is the case, I think extra_cache_validation_data is not needed since both should always computing the the same information. If extra_cache_validation_data exists only because it should be allow to return without errors even if some information is missing, I would either wrap the call with a try ... except block when called for validation or convert it into a function which accepts some validaton_mode: bool parameter to represent the calling context."

@DropD, could you review the current state of the PR and finish the discussion/implementation of the open issues?

The is_cache_info_consistent and is_cache_info_available_and_consistent methods were split only historically. The main difference is that the former can compare any given cache_info dictionary (you could imagine generating cache_info from one builder and ask another builder to check for consistency) while the latter only checks the current builder's up-to-date generated cache info against it's own stored cache info (if it exists). Since currently only the latter is used, they can be merged.
I merged the two methods and got rid of the cache input parameter since that was not used anywhere. It can easily be reintroduced later if a usecase should arise.

As for the extra_cache_info vs extra_cache_validation_data, this is the current situation:

extra_cache_info generates backend-dependent extra caching info that should be stored but might not be suitable for validation
`extra_cache_validation_data generates additional key-value pairs for validation of cache_info.

I will increase code-reuse by refactoring the latter into an extra_cache_validation_keys property which returns a sequence of keys that can be used to filter the extra_cache_info output for only the data that should be validated.


FIX behaviour MockClientResponse.json() if body is empty

When I have a mock like this:
response:
    body: {string: ''}

I have next error:
venv/lib/python3.6/site-packages/vcr/stubs/aiohttp_stubs/__init__.py:29: in json
    return loads(self._body.decode(encoding))
/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/json/__init__.py:354: in loads
    return _default_decoder.decode(s)
/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/json/decoder.py:339: in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/json/decoder.py:357: in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
E   json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Because it tried to decode empty string.
But in the same time aiohttp behave differently.
https://github.com/aio-libs/aiohttp/blob/master/aiohttp/client_reqrep.py#L1002
This should help.

@StasEvseev can you rebase your branch with master in order to trigger a CI build ?
I'll be happy to merge your changes then !

@arthurHamon2 Thanks for reacting :) I rebased code with master and build has been successful.


Run rustdoc --check in CI to prevent broken intra-doc links

I recently discovered that rustdoc has a --check flag to avoid generating documentation, and just produce warnings/errors. We currently have a number of broken internal doc links, so (after fixing them) it'd be great to automatically prevent more from being added.
Problems
Overall, running this right now might be a bit hacky:

There's no cargo doc --check command (see: https://github.com/rust-lang/cargo/issues/10025, https://github.com/rust-lang/cargo/pull/8859)
rustdoc --check requires nightly (see: https://github.com/rust-lang/rust/pull/78984)
cargo rustdoc ... can only be run for one crate at a time (not too bad; we can just loop through all of them)

Sample command
Currently, the following correctly runs, but requires nightly:
cargo +nightly rustdoc -p pageserver --lib -- -Z unstable-options --check

Notes:

-Z unstable-options is required for --check.
-p pageserver is required because rustdoc expects only a single crate
--lib is required because pageserver has both lib and bin targets, and rustdoc needs just one
(not shown) We have to also have to either pass --deny rustdoc::all or --deny rustdoc::$lint (for each lint) in order to cause rustdoc to exit with a non-zero status code

In general, requiring an extra nightly version in CI might be too much. It's "just" checking the code for dependencies + target crate, so not nearly as much as a full compilation -- it takes ~1.4 minutes on my machine (8 core/16 thread) to run the command above. It's then ~1 second for running on an additional crate (e.g., safekeeper).
In terms of possible nightly issues: I think we can mostly avoid stability issues by pinning to a particular version. We're also not compiling anything, so there's a lot of potential instability avoided there as well. It also might be possible to do something extraordinarily hacky and use nightly features with our stable tools (e.g, as in https://fasterthanli.me/articles/why-is-my-rust-build-so-slow#how-much-time-are-we-spending-on-these-steps). The really hacky version looks like:
RUSTC_BOOTSTRAP=1 cargo rustdoc -p pageserver --lib -- -Z unstable-options --check

It performs an initial run much faster (17.5s).

I have filed #4711 to fix the warnings generated by rustdoc. As explained in that PR, running rustdoc currently requires nightly due to dependencies, but I think after update of opentelemetry, it should be possible to run cargo doc in CI without a --check param and then just not do anything with the result.


Possible to close a message.on?

I'm chaining commands with my bot, and right now I have it only recognizing commands directed to it, but it then prompts the user without recognizing the prefix, however then that means that it will keep listening for all prefixed commands.
(basically I've got a message.on matching everything inside of a message.on only matching mentions of my bot. once the everything connection opens, I can't close it to make it only watch for mentions again)

Hrm, makes sense. Should definitely be an "off" command.


Fixing updateNavigationCurrentMonth with January minDate issue

Coverage remained the same at 68.9% when pulling e799c2fccd50602e3ce135f4aeaa383cd16e5075 on ciandt-dasa:v2.2.4-1 into 77447bfaea137aecf635f281ba32fa282694ac65 on chmln:master.

Hello,
I couldn't quite guess from the changes what issue exactly is being fixed here.
Please provide a description of the issue or a flddle (if possible).

Hi,
The month navigation is displaying the previous arrow when minDate is setted to January month
Find the example at CodePen:
You can also find the fix example at CodePen:

Coverage remained the same at 69.534% when pulling 94a548f7fde4edafc1e9f1b032ae09410e7c58da on ciandt-dasa:v2.2.4-1 into 2cd01fdbbdcff3f83721ffc0b9bfdb6e9b95d53f on chmln:master.

Coverage remained the same at 69.534% when pulling a6d9aa0e2e5eafe4463c08c5849229abe9919996 on ciandt-dasa:v2.2.4-1 into 2cd01fdbbdcff3f83721ffc0b9bfdb6e9b95d53f on chmln:master.


composer failure: "Undefined index: default_environment"

$ composer require yzalis/supervisor-bundle
...
  - Configuring yzalis/supervisor-bundle (>=1.0.5): From auto-generated recipe
Executing script cache:clear [KO]
 [KO]
Script cache:clear returned with error code 1
!!  PHP Warning:  Module 'grpc' already loaded in Unknown on line 0
!!
!!  In YZSupervisorExtension.php line 20:
!!
!!    Notice: Undefined index: default_environment
!!
!!
!!
Script @auto-scripts was called via post-update-cmd```

Fixed. I needed to add the configs prior to running composer.
    default_environment: default
    servers:
        default:
            localhost:
                host: <IP_ADDRESS>
                port: 9001```


Add ability to mark requests as warmup

Hot Chocolate has long had InitializeOnStartup as a mechanism to optimize the performance of initial operations.
It also already has the ability to execute warmup tasks. If you want to initialize caches at startup, by for example performing a dry run of your most important persisted operations, you were kind of out of luck, since there was no easy and builtin way to skip the actual execution of those warmup requests.
This PR adds the ability to mark requests as warmup, so they skip the execution step and bypass things like persisted operations. I've also documented the entire feature in more detail.

This one is looking good. Will merge it tomorrow.


TTL dropdown stays within modal window

This looks a bit weird, IMO:

I think in most applications, the drop-down would behave as if it were outside the modal window.

Yep that's an "issue" right now. You can scroll down outside of the select in the meantime.


Added an option for the format of the monthly view title: 'monthViewTitle'

Also, symbol for Korean is ko, while kr is for the country. I removed and added Korean translation files as so.

@feeva Use $.fn.datepicker.dates[yourLang].titleFormat.
Example: http://bootstrap-datepicker.readthedocs.org/en/master/i18n.html#i18n
@acrobat, @hebbet Close PR, please.


Duplicated class error

Hello,
When I'm compiling influent with maven, this error appears:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.0:compile (default-compile) on project influent-server: Compilation failure: Compilation failure:
[ERROR] /root/influent/influent/influent-server/src/main/java/influent/server/sql/mssql/MSSQLBuilder.java:[26,8] duplicate class: MSSQLBuilder
[ERROR] /root/influent/influent/influent-server/src/main/java/influent/server/spi/StandardSQLBuilderModule.java:[28,33] cannot access influent.server.sql.mssql.MSSQLBuilder
[ERROR] bad source file: /root/influent/influent/influent-server/src/main/java/influent/server/sql/mssql/MSSQLBuilder.java
[ERROR] file does not contain class influent.server.sql.mssql.MSSQLBuilder
[ERROR] Please remove or make sure it appears in the correct subdirectory of the sourcepath.

I followed all the steps from here: http://community.influent.org/community/developer-docs/how-to/installation/
I'm using maven 3.3.9 and Java 1.7.0_101.
Regards,
Joaquín

Solution:
Add line this
package influent.server.sql.mssql;
to MSSQLBuilder.java
Regards,

Thanks! this should now be resolved.


Missing and untranslatable string on Transifex

Description
The string "Since QGIS 3.20 we only ship 64-bit Windows executables." (https://github.com/qgis/QGIS-Website/commit/9f5c3ea540fb13a46b2889de6ddab18154c5d767) is missing in Transifex so it is not possible to translate it.
This is very likely due to https://github.com/qgis/QGIS-Website/issues/1089.

Page URL: https://qgis.org/it/site/forusers/download.html

It seems to me this is fixed now.


Review eiligibility criteria teams exploratory experience

Hi Team,
can you please check and review the eligibility criteria for the teams exploratory experience?
From our experience in support (M365 Subscription and Licensing), the tenant needs to have a subscription (paid or trial) that contains the service plan "Teams" and the subscription needs to be in active status in order for a user (with no license assigned) to be able to start the teams exploratory experience.
If a tenant contains only a paid license like Exchange Online plan 1 (which does not contain Teams), a user is not able to start the teams exploratory experience.
If you need more clarity on this please let me know.
Thanks,
Marta

Detalhes do Documento
⚠ Não edite esta secção. É necessário para a ligação de problemas docs.microsoft.com ➟ GitHub.

ID: 28550025-fde2-9a83-a826-f7e3ccb313db
Version Independent ID: 2a304432-b10c-220f-3453-e19e7a155c67
Content: Manage the Microsoft Teams Exploratory experience - Microsoft Teams
Content Source: Teams/teams-exploratory.md
Service: msteams
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

@MartaRosaGomes Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @MartaRosaGomes, thank you for your feedback.
A Product Manager confirms to us that you don't need a Teams service plan to enable a Teams exploratory license.


napari-cellpose

Hi there cellpose team!
I saw that the last update to the napari cellpose plugin was a while ago and cellpose has had many awesome updates since then. I know one can use the cellpose gui but I was wondering whether there were particular reasons for not yet updating the plugin (just too swamped is also just a valid reason:)). I am a bit time constrained myself, but in some free time can create a fork and try to add the new updates.

we have updated to support cellpose3! we are aiming to add training in napari soon and also the restoration models, please let us know if you have any issues using it over in that repo

@carsen-stringer great! will try it out. Is there some branch already for the added training in case you want some beta testers?

Not yet but soon!


(Dev Branch) Ship Weapons from Compendium don't have damage value

This is a similar issue to #915  which has been fixed.  Ship weapons don't generate a damage value in the chat box until opened and saved again if dragged directly from the compendium.

I believe this has been fixed. I just dragged a ship weapon to a vehicle and it had a non-zero damage and critical.

Not sure of the best way to fix this... it's effecting weapons and vehicle weapons, which I thought we had fixed.
The solution (though not ideal) is to open the weapon's item sheet at least once so that it's damage.adjusted values get calculated and then persisted to the database.

This is broken on newly imported gear as well. I'll take a look at the importer and see if that can help.

I think this can be fixed at the chat layer. It'll be a hack, but given that this version is not going to be maintained, I'm ok with that. I'll work on it after my other PR gets merged since it'll involve editing the same files.


⚠ ClamAV has degraded performance

In 425e515, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 578 ms

Resolved: ClamAV performance has improved in 93e351f after 5 minutes.


Insertion Sort Implementation in JavaScript #3

@divyanshu-rawat Sorry for asking here, but what does it mean by deleting branch? It says akshatbhargava123:patch-1 branch can be safely deleted.

I guess changes are merged into master branch so patch-1 branch is no longer needed !

Okay, thank you. :)
On Mon, Oct 2, 2017 at 3:17 PM, Divyanshu Rawat<EMAIL_ADDRESS>wrote:

I guess changes are merged into master branch so patch-1 branch is no
longer needed !
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/divyanshu-rawat/Basic-JS-Algorithms/pull/26#issuecomment-333488572,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVKB6w0wfXwO0ysv6EQnhEZ5C7yUXcR0ks5soLEngaJpZM4PqZNo
.


Tutorial Issue... cannot define  'SaveChangesAsync' nor  'SaveChanges'

I explicitly followed the example as written, but again VS complains.   I tried all the recommendations that I can find.  But nothing fixes this.  On my personal machine and many past companies, MANY of these tutorials always encounter an issue or 3.
Three of the 7 Error messages connected with the 'SaveChangesAsync'.

"There is no implicit reference conversion from 'TodoDb' to 'Microsoft.EntityFrameworkCore.DbContext'.	ToDoAPI"
CS0311: The type 'TodoDb' cannot be used as type parameter 'TContext' in the generic type or method 'DbContextOptions'
CS1061:'TodoDb' does not contain a definition for 'SaveChangesAsync' and no accessible extension method 'SaveChangesAsync' accepting a first argument of type 'TodoDb' could be found (are you missing a using directive or an assembly reference?)	ToDoAPI

Program_cs.zip

Hello @RangoonPost ... Would you mind re-opening this issue using the This page feedback button and form at the bottom of the English-US topic. Use of the This page feedback form adds metadata to your GitHub issue that cross-links the topic and automatically pings the author.

@guardrex I'm still looking, I havn't found the feedback you are directing me too.  I've looked under the tabs and other links...

It's available via the live (published) doc, not on GH.

Done.  Thank you.


GrantSqlControlToManagedIdentity parameter not available

Please add GrantSqlControlToManagedIdentity to az synapse workspace module.

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

ID: 7a8ce4bb-b234-dc91-cd2c-e5587caf6169
Version Independent ID: 9f08c3b1-5ac8-81d3-02e4-f7111cfe0781
Content: az synapse workspace
Content Source: latest/docs-ref-autogen/ext/synapse/synapse/workspace.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Thanks, we are working on this. Please expect January 2021.

@yonzhan: Is this issue resolved ??

@idear1203 any update for this issue?

Try command group: az synapse sql ad-admin.
cc: @wonner @zesluo


zsa-udev-rules: 2.1.3 -> unstable-2022-10-26

Upstream did changes on the udev rules to fix training in chrome/chromium. With the current stable release the keyboard is not picked up for training. If there is later on a stable release after this date, its safe to udate to that version.
There was some discussion about the udev rules in this PR https://github.com/NixOS/nixpkgs/pull/193912 which wanted to replace all udev rules by a custom file.
https://github.com/zsa/wally/pull/129 is the upstream code  we want to have as udev rules.
As shipping the upstream rules is a maintainer friendlier approach i am bumping the package here.
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1481

Is this something that can be backported to 22.11 stable?

https://github.com/zsa/wally/compare/2.1.3-linux...623a50d0e0b90486e42ad8ad42b0a7313f7a37b3

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/how-to-create-files-in-the-etc-udev-rules-d-directory/11929/16


[ENH] Speed up non forecasting tests

Describe the feature or idea you want to propose
Forecasting testing is slow mostly because a huge number of combinations are tested. Others have slow specific tests, which can probably be sped up by better config. Some may be unavoidable first calls to numba, some may just be CI issues. But all worth investigating.
Describe alternatives you've considered, if relevant
No response
Additional context
No response

I'm closing this now, since the testing has been completely reformed


Review for Missing Documentation - at_commons_flutter (API Reference)

[ ] Right nav bar under Documentation API Reference (samples and examples)
[ ] Do we need a link for documentation?

Please review library links as well (current, samples and examples)
LIBRARIES

[ ] at_common_flutter
[ ] colors
[ ] custom_app_bar
[ ] custom_button
[ ] custom_input_field
[ ] size_config
[ ] text_strings
[ ] text_styles

Hi @sarika01 we need to review this very old ticket for accuracy, potentially close it and divert to the new pub.dev publishing issue. Creating new issue cards today.


New API

To be fair, they did email me, but I also forgot.

Closes #84

This now works, but due to how the API is paid for now, each sensor and field returned is counted against you. I need to adjust this to get a bounding box of your area for the sensors and maybe bust the cache if you're further than like 5km from one or something.

https://github.com/davidwood/node-geopoint/blob/master/geopoint.js#L123-L159 can maybe borrow this MIT open source impl

https://github.com/davidwood/node-geopoint/blob/master/geopoint.js#L123-L159 can maybe borrow this MIT open source impl

I ended up using ChatGPT for this

Imma leave this up in preview for a day so I can see how much my testing cost in terms of tokens and then merge it


Add XtheWiz to Contributors list

Hi I followed your instruction and then I bring me to pull request, please accept :)
Thank you very much

@XtheWiz, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


not compiling on OS X 10.7

unfortunately the latest master is not building on OS X 10.7 due to unrecognized selectors and properties:
NSArray+PureLayout.m:58:13: No known class method for selector 'activateConstraints:'
NSArray+PureLayout.m:78:9: No known class method for selector 'deactivateConstraints:'
NSLayoutConstraint+PureLayout.m:52:18: Property 'active' not found on object of type   'NSLayoutConstraint *'
NSLayoutConstraint+PureLayout.m:119:14: Property 'identifier' not found on object of type 'NSLayoutConstraint *'

Is there any chance to fix this?
Thanks!

Hey Andreas,
Thank you for bringing this up! I don't actually have a machine with OS X 10.7 around to test this, but I think I fixed the issue in master. Can you please try the latest code and tell me how it goes?
If it's working correctly now I will push an update to the pod ASAP.
Thanks!

@azplanlos Any updates on this? I'd like to publish an update to the pod if it is fixed for you.

@azplanlos I would greatly appreciate an update from you, but since I have not heard back in over 2 weeks, I am closing this issue and will assume it has been fixed.


functions in ifdef is ignored when generate ffi bindings with ffigen.

When generate bindings with ffigen, the following function is ignored. How can I fix tell the generator to generate this function?

#if LIBUSB_API_VERSION >= 0x01000107
uvc_error_t uvc_wrap(
    int sys_dev,
    uvc_context_t *context,
    uvc_device_handle_t **devh);
#endif

You can pass defines with compiler-opts with -D<macroname>=<value>.

You can pass defines with compiler-opts with -D<macroname>=<value>.

Thanks, works like a charm!


fix(ci-cd): step to generate changelog

add the missing ci check on prs
GH-172
Description
step to generate changelog
add the missing ci check on prs
Fixes #172
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Intermediate change (work in progress)

Checklist:

[ ] Performed a self-review of my own code
[ ] npm test passes on your machine
[ ] New tests added or existing tests modified to cover all changes
[ ] Code conforms with the style guide
[ ] API Documentation in code was updated
[ ] Any dependent changes have been merged and published in downstream modules

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update to use 1.11.3 of winget-install

Hi again,
Hope all is well,
Is it possible to update this so it uses your latest version of the winget-intune version? (1.11.3).
Thanks,
Alex.

Hi. When I have few time i will.


Data fields for claiming a facility

Overview
Facility owners/managers need to provide additional details when claiming a facility.
Is your feature request related to a problem? Please describe.
Facility owners/managers must provide information in order to verify they are facilities owners and add information to display as part of the facility profile.
Describe the solution you'd like
Possible fields include the options below. Only login name, email address of facility, and official name of LLC or facility registered would be required.
Login Name
Email
Phone number
Contact email (if different from email they’ve registered the account with)
Website
Address of office
Region
Country
Address of facility
Region
Country
Latitude and longitude
Bio - tell us more about your facility
Official name of LLC or company registered
Owner name
Contact person name
Certifications
Type of products they make
Type of production they do on site
Minimum Order quantities
Average lead time
Trade association membership?
Any other information
Additional Context
This list is subject to change based on feedback from the facilities. We would like to show them the field within the context of the wireframes.

@designmatty See list of possible facility related fields in this issue.

Claim fields were implemented and revised
https://github.com/open-apparel-registry/open-apparel-registry/pull/575
https://github.com/open-apparel-registry/open-apparel-registry/pull/650


adding private update/draw functions

sometimes objects need their own update/draw functions to make their animations and whatever work. The functionality is lost if we overwrite those functions for greater customizability. Instead of copying the canonical function body over from unrequited.lua, it would be more desirable to have a private hidden update function along with the objects. For example:
function ent:draw(t, dt)
    ent.cum_dt = ent.cum_dt + dt
    if ent.cum_dt > 1/ent.fps then
        ent.current_frame = (ent.current_frame + 1) % ent.frames
        ent.cum_dt = ent.cum_dt - 1/ent.fp
    end
    --  actual draw calls
end

the draw call had to take in parameters and do calculations inside it's own body. This is so we don't have the calculation in a built-in update function that will inevitably be overwritten. The goal is to be object agnostic when it comes to implementations.
The solution will be to create a built-in private update (or even draw) function that accommodates those builtin functionalities, and it will be called within unrequited:update and/or unrequited:draw.
This will allow the end user to have their own creative freedom to develop their work without having to know the  specifics of this framework.

addressed in PR #14


Android asks for the default launcher after restart.

Whenever i turn off and then on my device (RP2+) it asks if i want to make Daijisho my default launcher, i click on 'always' . If i turn off the device and then turn it on it will ask again for the default launcher. This only happens on 1.4.

Are you able to enter Daijishō app itself and screenshot error from the logger?

here you go

Can you screenshot the part flagged as error (Red one)

Here is an example:

Hmm I enabled debug mode and I couldn't find any error. I Believe the app is not crashing. Is just that is not being saved as the default launcher. Even though I press the "always" button.

Hmm that's weird. Also mentioned that if the app crashed. The default launcher settings will be wiped out.

Does it happen on your device? I'm using it on a RP2+. The setting will be saved until I turn off or restart the device.

Does it happen on your device? I'm using it on a RP2+. The setting will be saved until I turn off or restart the device.

No. I have tested it on my RP2+.

The first time I installed the beta it said that the app was not verified by the playstore and whatever. Maybe the playstore is preventing it from saving it as default? The weird thing is that it stays as the default app until I reset the rp2+

The first time I installed the beta it said that the app was not verified by the playstore and whatever. Maybe the playstore is preventing it from saving it as default? The weird thing is that it stays as the default app until I reset the rp2+

I've never countered this situation. It might be the reason. Thanks for the report bro.

Anytime brother. 👌

The issue seems to haven been solved but know I'm getting this persistent notification.

Okay this means that it is not the timeout causing the issue. But your Android system is limiting background service. New RCs have added foreground roll back to fix this issue.

Awesome dude. I don't mind the notification if that means DAI is working properly :D. Any idea why this didn't happen before 1.4?

Awesome dude. I don't mind the notification if that means DAI is working properly :D. Any idea why this didn't happen before 1.4?

Because the daijishō service is introduced only on 1.4. 👍😬

Oooh cool ,cool 👌


[HWKALERTS-227] make bootstrap script handle shutdowns and restarts

I've reviewed the description of the JIRA and changes proposed.
LGTM merging...


Fixed Issue  #8874

Issue Reference
#8874

Checked and Found that the issue has been fixed, Now the user name is appearing  in the Confirmation mail. Please check the video and screenshot here
Video
https://github.com/bagisto/bagisto/assets/92721837/4920daaa-07c8-42c5-a83a-6ddbb9c3d772


When using json custom mapping, integers are formatted in a wrong way

Description:
When using JSON custom mapping, integers greater than 1000 are formatted in a comma separated way and the resulting JSON is invalid.
Eg :
@map(type='json', @payload("""{"alertId":"{{alertId}}","userId":{{userId}}}""")))
define stream requestStream (alertId String, userId int);

When userId is something less than 1000, it works properly.
But when userId becomes greater than 1000, the output json string from the sink mapper will be something like below, which is an invalid json.
{"alertId":"alert1","userId":1,234}

The correct JSON string should be something like below.
{"alertId":"alert1","userId":1234}

Suggested Labels:
bug, siddhi-map-json
Affected Product Version:
WSO2SP-4.0.0, WSO2SP-4.1.0, WSO2SP-4.2.0
Steps to reproduce:

Create a source with custom JSON mapping enabled like following.

@map(type='json', @payload("""{"alertId":"{{alertId}}","userId":{{userId}}}""")))
define stream requestStream (alertId String, userId int);

Provide a value greater than 1000 for the integer attribute.

@minudika Is this issue fixed already?

Fixed by https://github.com/siddhi-io/siddhi/pull/1431 ...

@mohanvive
Could you patch this issue for WSO2SP-4.3.0?
This is really CRITICAL ISSUE for me.
I don't want to send a message into "event" so I have to use custom mapping.
I couldn't send a message including integer format because it's not valid format.
Please.. I've been waiting to resolve this issue.

@inspiteof
SP 4.3.0 contains Siddhi version 4.2.20 hence Now, I have sent the PR for Siddhi 4.x.x branch as well. You can apply the similar changes to respective Siddhi version, built it and get the update jar.
I am afraid that I cannot give official patches to specific product version through community channel; it is only available to customers who have subscription account/channel.


Repository says there's no release even though there is?

I'm not sure if I'm doing something wrong, but in https://github.com/martinvonz/jj I see releases, but running bin says it doesn't find any. Am I being dumb?

:wave: here's the reason: https://github.com/marcosnils/bin#i-see-releases-on-github-but-bin-does-not-pick-them-up

Ahh, sorry, I saw that but didn't realise they were all prereleases. Thank you.


numpy.core array memory error: unable to allocate X.

When using on PC with window 10 x64.
This code
materialLayer1 = ScatteringMaterial(mu_s=0, mu_a=1, g=0.7, n=1.38)
materialLayer2 = ScatteringMaterial(mu_s=90, mu_a=0.01, g=0.8, n=1.33)
materialLayer3 = ScatteringMaterial(mu_s=1, mu_a=1, g=1, n=1.2)
materialLayer4 = ScatteringMaterial(mu_s=100, mu_a=50, g=1, n=1)

layer1 = Cuboid(a=300, b=300, c=5, position=Vector(0, 0, 0), material=materialLayer1, label="Layer 1")
layer2 = Cuboid(a=300, b=300, c=10, position=Vector(0, 0, 0), material=materialLayer2, label="Layer 2")
layer3 = Cuboid(a=300, b=300, c=5, position=Vector(0, 0, 0), material=materialLayer3, label="Layer 3")
layer4 = Cuboid(a=300, b=300, c=260, position=Vector(0, 0, 0), material=materialLayer4, label="Layer 4")

stack1 = layer1.stack(layer2, "back")
stack2 = stack1.stack(layer3, "back")
stackedTissue = stack2.stack(layer4, "back")

tissue = ScatteringScene([stackedTissue])

logger = EnergyLogger(tissue)
source = PencilPointSource(position=Vector(0, 0, -5), direction=Vector(0, 0, 1), N=500)
source.propagate(tissue, logger)
viewer = Viewer(tissue, source, logger)
viewer.reportStats()
viewer.show3DVolumeSlicer()

Produces this error. Even if this is probably because python is not given enough memory, error should be caught and the source explained so the user can correct.
Traceback (most recent call last):
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\examples\rayscattering\ex05.py", line 39, in <module>
    exampleCode()
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\examples\rayscattering\ex05.py", line 29, in exampleCode
    logger = EnergyLogger(tissue)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\pytissueoptics\rayscattering\energyLogging\energyLogger.py", line 48, in __init__
    self._views = self._viewFactory.build(views)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\pytissueoptics\rayscattering\display\views\viewFactory.py", line 29, in build
    self._setContext(view)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\pytissueoptics\rayscattering\display\views\viewFactory.py", line 104, in _setContext
    view.setContext(limits3D=limits3D, binSize3D=self._defaultBinSize3D)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\pytissueoptics\rayscattering\display\views\view2D.py", line 96, in setContext
    self._dataUV = np.zeros((self._binsU, self._binsV), dtype=np.float32)
numpy.core._exceptions._ArrayMemoryError: Unable to allocate 3.35 GiB for an array with shape (30000, 30000) and data type float32

Reducing layer size launches the simulation, but uncaught memory problems still arise
 layer1 = Cuboid(a=30, b=30, c=5, position=Vector(0, 0, 0), material=materialLayer1, label="Layer 1")
    layer2 = Cuboid(a=30, b=30, c=10, position=Vector(0, 0, 0), material=materialLayer2, label="Layer 2")
    layer3 = Cuboid(a=30, b=30, c=5, position=Vector(0, 0, 0), material=materialLayer3, label="Layer 3")
    layer4 = Cuboid(a=30, b=30, c=260, position=Vector(0, 0, 0), material=materialLayer4, label="Layer 4")

This modification gives
Propagating photons: 100%|██████████| 500/500 [00:02<00:00, 189.30it/s]
WARNING: The volume slicer will require a lot of memory (5632.64 GB). Consider using a larger binSize or tighter limits.
Traceback (most recent call last):
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\examples\rayscattering\ex05.py", line 39, in <module>
    exampleCode()
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\examples\rayscattering\ex05.py", line 35, in exampleCode
    viewer.show3DVolumeSlicer()
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\pytissueoptics\rayscattering\display\viewer.py", line 147, in show3DVolumeSlicer
    hist, _ = np.histogramdd(points[:, 1:], bins=bins, weights=points[:, 0], range=limits)
  File "<__array_function__ internals>", line 200, in histogramdd
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\venv\lib\site-packages\numpy\lib\histograms.py", line 1046, in histogramdd
    hist = np.bincount(xy, weights, minlength=nbin.prod())
  File "<__array_function__ internals>", line 200, in bincount
numpy.core._exceptions._ArrayMemoryError: Unable to allocate 1.84 TiB for an array with shape<PHONE_NUMBER>08,) and data type float64
Report of solid 'Layer 1'
  Absorbance: 99.38% (96.47% of total power)
  Absorbance + Transmittance: 100.0%
    Transmittance at 'Layer 1_front': 0.0%
    Transmittance at 'interface0': 0.6%
Report of solid 'Layer 2'
  Absorbance: 22.30% (0.13% of total power)
  Absorbance + Transmittance: 100.0%
    Transmittance at 'interface0': 77.7%

The first case uses a large tissue of 3x3x3 meters. The logger initializes multiple default 2D views as empty arrays with a default bin size of 100 microns. We could skip this empty initialization when keep3D is left to True, and/or change the default bin size to adjust for scene size.
In the second case, the crash happens when creating the volume slicer (with default bin size of 100 microns -> requiring over 5TB of memory). A warning is first written :

WARNING: The volume slicer will require a lot of memory (5632.64 GB). Consider using a larger binSize or tighter limits.

But we should also catch the OOM error.
In the meantime, both errors should be avoidable by providing a proper defaultBinSize to the EnergyLogger

there's also a third case, when the memory error only occurs when show3D is launched. In this case no warning is issued. Maybe
materialLayer1 = ScatteringMaterial(mu_s=2, mu_a=0.5, g=0.7, n=1.3)
materialLayer2 = ScatteringMaterial(mu_s=5, mu_a=0.8, g=0.8, n=1.4)
materialLayer3 = ScatteringMaterial(mu_s=50, mu_a=2.5, g=0.9, n=1.5)

layer1 = Cuboid(a=100, b=100, c=5, position=Vector(0, 0, 0), material=materialLayer1, label="Layer 1")
layer2 = Cuboid(a=100, b=100, c=5, position=Vector(0, 0, 0), material=materialLayer2, label="Layer 2")
layer3 = Cuboid(a=100, b=100, c=5, position=Vector(0, 0, 0), material=materialLayer3, label="Layer 3")
stack1 = layer1.stack(layer2, "back")
stackedTissue = stack1.stack(layer3, "back")

tissue = ScatteringScene([stackedTissue])

logger = EnergyLogger(tissue, keep3D=False)
source = PencilPointSource(position=Vector(0, 0, -5), direction=Vector(0, 0, 1), N=500000, useHardwareAcceleration=True)
source.propagate(tissue, logger)

viewer = Viewer(tissue, source, logger)
viewer.reportStats()
viewer.show3D()

Traceback (most recent call last):
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\examples\rayscattering\ex04.py", line 35, in <module>
    exampleCode()
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\examples\rayscattering\ex04.py", line 31, in exampleCode
    viewer.show3D()
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\pytissueoptics\rayscattering\display\viewer.py", line 120, in show3D
    self._addViews(viewsVisibility, viewsSolidLabels, viewsSurfaceLabels, viewsLogScale, viewsColormap)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\pytissueoptics\rayscattering\display\viewer.py", line 223, in _addViews
    self._addView(view, logScale, colormap)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\pytissueoptics\rayscattering\display\viewer.py", line 253, in _addView
    self._viewer3D.addImage(alignedImage, alignedSize, alignedCorner, view.axis, view.displayPosition, colormap)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\pytissueoptics\scene\viewer\mayavi\MayaviViewer.py", line 104, in addImage
    p = mlab.imshow(image, colormap=colormap, interpolate=False,
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\venv\lib\site-packages\mayavi\tools\helper_functions.py", line 38, in the_function
    return pipeline(*args, **kwargs)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\venv\lib\site-packages\mayavi\tools\helper_functions.py", line 83, in __call__
    output = self.__call_internal__(*args, **kwargs)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\venv\lib\site-packages\mayavi\tools\helper_functions.py", line 93, in __call_internal__
    self.source = self._source_function(*args, **kwargs)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\venv\lib\site-packages\mayavi\tools\sources.py", line 1253, in array2d_source
    data_source.reset(scalars=s, mask=mask)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\venv\lib\site-packages\mayavi\tools\sources.py", line 577, in reset
    x, y = np.mgrid[-nx / 2.:nx / 2, -ny / 2.:ny / 2]
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\venv\lib\site-packages\numpy\lib\index_tricks.py", line 174, in __getitem__
    nn = _nx.indices(size, typ)
  File "C:\Users\CAMAVIG2\Documents\GitHub\Python\PyTissueOptics\venv\lib\site-packages\numpy\core\numeric.py", line 1782, in indices
    res = empty((N,)+dimensions, dtype=dtype)
numpy.core._exceptions._ArrayMemoryError: Unable to allocate 37.3 GiB for an array with shape (2, 50000, 50000) and data type float64


How to avoid generating 180 rotated text

I found some generated text orientation is 180 rotated in the Pre-processed 8000 Background Images, Is there any method to avoid generate such text?
for example
original text：16
generate text：9⇂

@shishaozheng can you show me how to avoid generating 180 by edit source code, thanks!

Hi @shishaozheng, could you please share the code, thanks!

     R2d = su.unrotate2d(box.copy())
     # # by lmy make cos(theta > 0)
     if R2d[0][0] < 0:
         R2d[0][0] = -R2d[0][0]
         R2d[1][1] = -R2d[1][1]

For anyone still struggling with mirrored text, the copy of synthgen.py in repo mentioned below seems to fix this. https://github.com/ankush-me/SynthText/issues/266#issuecomment-1180263282


Governance. Vote history. Wrong container for info button.

Description
Wrong container for info button, when it opens it scroll inside a small line, whould be nice to have that container on full available window:
https://github.com/user-attachments/assets/e6d47c7b-9aca-47c3-ad6a-115014e7b3be
Environment
https://github.com/novasamatech/nova-spektr/commit/64f12ff7746cb19024e2df0b6009cec5e6e2cf17

Status verification: ✅


Raid Error Handling

Overview

Wrap the main logic in try/catch to alert Moderators around errors.

LGTM 👍


fix: template_version was empty in manifest

I simply forgot to write it, then forgot to test that I had written it :/

Requesting rubber-stamp re-review with changed base branch.


About the vibration function

Thank you so much for publishing this great package!!!
I appreciate it from the bottom of my heart.
We think it would be even better if the alarm had a vibration function.
For example, it would be greatly appreciated if the following items could be added to achieve this.
final alarmSettings = AlarmSettings(
  // ...
  vibrate: true // this add
);

Hi @tattuu
Thank you so much for your interest in the package !
I totally agree with you. Adding the vibration feature is on my to-do list, and I will start developing it as soon as I finish with the pull request #11 I am currently working on.

What do you think would be the best way to vibrate during the alarm ?
There is this package vibration that allows you to set a duration, amplitude and patterns.

Hi @gdelataillade
Thanks for the reply!
If you don't need to write it in native code and can dependency on flutter, that package is the best choice.
If that package is not available, the implementation method will depend on how far you are willing to go for customizability.
If customizability is to be sacrificed, it may be better to prepare several templates in advance and let the user choose from them.
For example, the enum image is close to the one implemented in this vibration package.
https://pub.dev/packages/flutter_vibrate
// Choose from any of these available methods
enum FeedbackType {
  success,
  error,
  warning,
  selection,
  impact,
  heavy,
  medium,
  light
}

For example
final alarmSettings = AlarmSettings(
  // ...
  vibrationFeedbackType: VibrationFeedbackType.heavy // this add
);

Thanks for your research @tattuu !
I will try to go for customizability as much as possible. An enum can be a good easy-to-use option though !


[Feat] Multi-owner private certificates

Features

[x] Multi-owner private certificates
[x] Support storing OwnershipCommitments off-chain
[x] Allow the Issuer to approve private transfers

Fixes

[x] Re-enable smart contract commitment checks
[x] On-chain isCertificatePrivate check

Chore

[x] Remove BlockchainDataModelEntity and do a tighter integration with Certificates and CertificationRequests

Limitations

Only one private transfer request per certificate can exist at any time

LGTM but there are not a single test for this code. I think this should be tested thoroughly.

Sure, there are old tests which pass - but will add new tests to cover more cases.


KAFKA-3016: phase-1. A local store for join window

@guozhangwang
An implementation of local store for join window. This implementation uses "rolling" of RocksDB instances for timestamp based truncation.

Some minor comment left, otherwise LGTM. Waiting for Jenkins results.


add section in upgrade that will check if ca_serial needs to/can be set

Bug report: [1]
This resolves the issue on an LXD cluster, but it would be good to test on AWS.
My thought process:

Since the issue shows up when upgrading from a previous version [2] where certificate_authority_serial is not set on install, my thought is that we should just check if certificate_authority and certificate_authority_key are set and certificate_authority_serial is not set when an upgrade was triggered. If these three conditions are met (and the file exists), we just set certificate_authority_serial during the upgrade process.
If we still cannot set certificate_authority_serial in the upgrade, then I think we need an additional condition in create_certificate_authority() (line 168).

TODO: formal testing if it is possible to test this fix automatically.
[1] https://bugs.launchpad.net/charm-easyrsa/+bug/1888580
[2] https://github.com/charmed-kubernetes/layer-easyrsa/pull/21/files

Looks reasonable to me.

LGTM

Oops - we missed that this was targeted against the stable branch. Usually we accept work into master first, and then backport to stable if needed. I'll open up a follow-up PR to get this into master.

Merged into master via https://github.com/charmed-kubernetes/layer-easyrsa/pull/29


feat(terraform): use Terraform to create ShardingSphere Proxy Cluster.

Signed-off-by: xuanyuan300<EMAIL_ADDRESS>
Support use Terraform to create ShardingSphere Proxy Cluster.
Type of change:

[ ] Bugfix
[x] New feature provided
[ ] Improve performance
[ ] Backport patches

What this PR does / why we need it:

Pre-submission checklist:

[ ] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?

Codecov Report

Merging #113 (3cc782a) into main (ffb31ea) will increase coverage by 10.87%.
The diff coverage is 76.00%.

@@             Coverage Diff             @@
##             main     #113       +/-   ##
===========================================
+ Coverage   51.49%   62.36%   +10.87%
===========================================
  Files           2        2
  Lines         268      279       +11
===========================================
+ Hits          138      174       +36
+ Misses        128      103       -25
  Partials        2        2

Impacted Files
Coverage Δ

shardingsphere-operator/pkg/reconcile/resource.go
60.07% <76.00%> (+11.66%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


React -native 0.60.5 how to configure react-native-vector-icons

[ ] Review the documentation: https://github.com/oblador/react-native-vector-icons
[ ] Search for existing issues (including closed ones): https://github.com/oblador/react-native-vector-icons/issues

Environment

Description
Describe your issue in detail. Include screenshots if needed.
Demo
You can use https://snack.expo.io/ to create a demo that can help users to better understand your problem.

I have the same question. After upgrading from react-native 59.9 to 60.5, My icons from font-awesome (the normal one, not font-awesome5) are not found anymore.
My own library which I link through react-native.config.js and add with createIconSet(glyphMap, 'myIcons'); do work, but using import Icon from 'react-native-vector-icons/FontAwesome'; does not.
If I try to add the package to getPackages() in MainApplication.java, react-native complains at runtime that the library is linked twice.

Ok, managed to fix it (Android only right now, still need to test iOS).
Make sure there is no linking anywhere (as supposed to be for react-native 60+)
Then only add apply from: "../../node_modules/react-native-vector-icons/fonts.gradle" to android/app/build.gradle

https://github.com/oblador/react-native-vector-icons/issues/1041#issuecomment-515673115

+1

Follow this link to Fix in the IOS https://medium.com/@vimniky/how-to-use-vector-icons-in-your-react-native-project-8212ac6a8f06

Ok, managed to fix it (Android only right now, still need to test iOS).
Make sure there is no linking anywhere (as supposed to be for react-native 60+)
Then only add apply from: "../../node_modules/react-native-vector-icons/fonts.gradle" to android/app/build.gradle

I spent 4 hours trying to figure our this problem I did tried you solutionbefore but dont know why it was not working But its working now thanks a lot. Cheers

https://github.com/oblador/react-native-vector-icons/issues/1073#issuecomment-533681091

Ok, managed to fix it (Android only right now, still need to test iOS).
Make sure there is no linking anywhere (as supposed to be for react-native 60+)
Then only add apply from: "../../node_modules/react-native-vector-icons/fonts.gradle" to android/app/build.gradle

you save my day, thanks!


load graphql schemas from .graphql files

using https://github.com/Swydo/meteor-graphql you can load .graphql files in meteor applications.
this returns directly something that can be used in makeExecutableSchema.
unfortunatly, it does not play nice with vulcan, as there is no api to add these kind of typedefs.
Any way to use graphql files?

Since Vulcan already takes strings and adds them to the schema I think it should work out of the box without that package. Have you tried it?
See http://docs.vulcanjs.org/graphql-schema.html#Custom-Schemas

(Use addGraphQLSchema() btw, the docs are a little bit out of date)

@SachaG yes, i use that currently:
// ...
import MySchema from './MySchema.graphql';

addGraphQLSchema(MySchema.loc.source.body);

But this has some downsides (compared to the normal usage of swydo:graphql):

you can't use # import statements in the graphql file, because MySchema.loc.source.body only contains the raw string (that's identical with just reading the file as string)
you can't have descriptions (as strings) in the graphql file

would be cool beeing able to use
import MySchema from './MySchema.graphql';

addGraphQLSchema(MySchema);

MySchema is an object of the shape
{
"kind": "Document",
"definitions": [
  {
    "kind": "ObjectTypeDefinition",
    "name": {
      "kind": "Name",
      "value": "MyField"
    },
    // ...
    ]
  },
  // ...

so this definition list could maybe be merged with the parsed default schema that vulcan generates

The thing is, I don't know if makeExecutableSchema can accept multiple typeDefs? If not we'd need a way to merge different schema objects together but I don't really know how to do that…

Was at a @formidablelabs workshop recently and they mentioned OK Grows merge schema tool:
https://github.com/okgrow/merge-graphql-schemas

Oh interesting. @macrozone maybe you can give that a try? Here's where the magic happens:
https://github.com/VulcanJS/Vulcan/blob/devel/packages/vulcan-lib/lib/server/apollo_server.js#L204-L226
(The weird formatting is to ensure the schema comes out right when you want to log it out)

Apollo also has a mergeSchema function
On Tue, Mar 13, 2018, 14:24 Sacha Greif<EMAIL_ADDRESS>wrote:

Oh interesting. @macrozone https://github.com/macrozone maybe you can
give that a try? Here's where the magic happens:
https://github.com/VulcanJS/Vulcan/blob/devel/packages/vulcan-lib/lib/server/apollo_server.js#L204-L226
(The weird formatting is to ensure the schema comes out right when you
want to log it out)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/VulcanJS/Vulcan/issues/1926#issuecomment-372663428,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfguWx1qOo0OoyMSIr6_bzfQAF1x0OeVks5td8iEgaJpZM4Sod6c
.

Maybe we can end up with 3 PRs for different solutions :) HAHAHA @MathiasKandelborg wins!

There's also https://github.com/creditkarma/graphql-loader which I think can merge schemas?


Allow runtime-wide TryState checks

Currently the TryState can only be utilized on a per-pallet level by implementing the try_state hook.
This limits the usage tor multi-pallet try_state functionality. Executive could be extended to accept an optional TryState which will then be executed by the try-runtime CLI on a runtime-wide level.
cc @kianenigma

Yup, should be an easy one relatively (cc-ing @gpestana @Ank4n).

Executive could be extended to accept an optional TryState which will then be executed by the try-runtime CLI on a runtime-wide level.

We don't really need to forward this into Executive. We have all pallets as tuple in the runtime and just calling AllPalletsWithSystem::try_state should be enough. Before that we would need to change:
https://github.com/paritytech/substrate/blob/05d11c213b8e097a5368a887b6d055db61ce05db/frame/support/procedural/src/pallet/expand/hooks.rs#L210-L226
to:
		#frame_support::try_runtime_enabled! {
			impl<#type_impl_gen>
				#frame_support::traits::TryState<<T as #frame_system::Config>::BlockNumber>
				for #pallet_ident<#type_use_gen> #where_clause
			{
				fn try_state(
					n: <T as #frame_system::Config>::BlockNumber,
					_s: #frame_support::traits::TryStateSelect
				) -> Result<(), &'static str> {
					#log_try_state
					<
						Self as #frame_support::traits::Hooks<
							<T as #frame_system::Config>::BlockNumber
						>
					>::try_state(n)
				}
			}
		}

@bkchr I am not sure why you are against this. The main goal is to enable a try-state check that will look into multiple pallets. The only way to achieve this would be through doing it at the runtime (probably via executive)

So we can already write and execute TryState checks in Executive? Then we can close this issue?


BUG Sort out the colour updates for the measurement station lines

We have some code that is responsible for updating the colours of the lines in the measurement charts. In the SingleCity.tsx page we have a bunch of code setting site colorus, which calls updateColors() etc.
When running locally this code appears to work fine, and you can see the colours changing on the loaded graphs (easier to see with more stations, i.e. Tokyo). The lines are made darker and more prominent as well as changing colour.
On the deployed version we do not see this, and the default eCharts colours are still shown. We also receive a JS error in the browser console, which may be related.
We should do either

fix the error so the colour change also happens on the deployed version,
delete the code and leave the eCharts colours as they are on our deployed version too.

We will fix!
Hopefully whichever we choose will sort out the error occurring on the browser (seen in Chrome at least)
Example of the issue :

Ideas to test:

check prod issue in screenshot is resolved
automation for one city site - cryptographically hash station name, small graph with one station, assert colour or if not possible, screenshot test would work

Charter here  -> https://github.com/ECMWFCode4Earth/vAirify/wiki/Charter-25:-Line-Colours


FIX - Add Auth Rules

add auth rules to allow different groups different priviledges. eevry user is by default added to the Everyone group. Make default groups:

Admin - CRUD
Publisher - RU
Editor - RC
Everyone - R

Amplify example of everyone group


Fix autogenerated macro examples

This PR:

makes the capitaliseComponentName() function available to all Nunjucks templates and macros
uses this function in showDefaultAndVariants.njk for autogenerating the macro examples

LGTM. Suggest rebasing those commits into one if possible. 👍


RuntimeError: DataChannelNccl does not support barrier

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
RuntimeError: DataChannelNccl does not support barrier.
Reproduction

What command or script did you run?

./tools/dist_train.sh configs/masktrack_rcnn_r50_fpn_1x_youtubevos.py 4

Did you make any modifications on the code or config? Did you understand what you have modified?
I understand.
What dataset did you use?
Youtube Vos.
Environment

OS: Ubuntu 16.04.6
GCC: 5.4.0
PyTorch version: 0.4.1
How you installed PyTorch: pip
GPU model: TITAN Xp
CUDA and CUDNN: 8.0
[optional] Other information that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
  File "./tools/train.py", line 90, in <module>
    main()
  File "./tools/train.py", line 77, in main
    cfg.model, train_cfg=cfg.train_cfg, test_cfg=cfg.test_cfg)
  File "/home/earhian/anaconda3/lib/python3.6/site-packages/mmdet/models/builder.py", line 51, in build_detector
    return build(cfg, DETECTORS, dict(train_cfg=train_cfg, test_cfg=test_cfg))
  File "/home/earhian/anaconda3/lib/python3.6/site-packages/mmdet/models/builder.py", line 31, in build
    return _build_module(cfg, registry, default_args)
  File "/home/earhian/anaconda3/lib/python3.6/site-packages/mmdet/models/builder.py", line 23, in _build_module
    return obj_type(**args)
  File "/home/earhian/anaconda3/lib/python3.6/site-packages/mmdet/models/detectors/mask_rcnn.py", line 31, in __init__
    pretrained=pretrained)
  File "/home/earhian/anaconda3/lib/python3.6/site-packages/mmdet/models/detectors/two_stage.py", line 54, in __init__
    self.init_weights(pretrained=pretrained)
  File "/home/earhian/anaconda3/lib/python3.6/site-packages/mmdet/models/detectors/two_stage.py", line 62, in init_weights
    self.backbone.init_weights(pretrained=pretrained)
  File "/home/earhian/anaconda3/lib/python3.6/site-packages/mmdet/models/backbones/resnet.py", line 349, in init_weights
    load_checkpoint(self, pretrained, strict=False, logger=logger)
  File "/home/earhian/anaconda3/lib/python3.6/site-packages/mmcv-0.2.10-py3.6-linux-x86_64.egg/mmcv/runner/checkpoint.py", line 137, in load_checkpoint
    checkpoint = load_url_dist(model_urls[model_name])
  File "/home/earhian/anaconda3/lib/python3.6/site-packages/mmcv-0.2.10-py3.6-linux-x86_64.egg/mmcv/runner/checkpoint.py", line 101, in load_url_dist
    torch.distributed.barrier()
  File "/home/earhian/anaconda3/lib/python3.6/site-packages/torch/distributed/__init__.py", line 513, in barrier
    return torch._C._dist_barrier(group)
RuntimeError: DataChannelNccl does not support barrier

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

Please refer to the second item of the checklist. 0.4.1 is no longer officially supported and maintained.
Also I found some related issue: https://github.com/pytorch/pytorch/issues/17848

As stated in the documentation, PyTorch 1.1+ and CUDA 9.0+ are required.


iptables panic leading to CI test failures

https://storage.googleapis.com/istio-prow/logs/integ-distroless-k8s-tests_istio_postsubmit/1792/artifacts/security-1561ecc9de344180b05696/TestReachability/reachability-1-80906-state952705050/vm-v1-d4468d46d-9nwsj-istio-proxy.log
https://storage.googleapis.com/istio-prow/logs/integ-distroless-k8s-tests_istio_postsubmit/1791/artifacts/security-4cf7a47365f5425d8bcde4/TestReachability/reachability-1-20457-state367595562/vm-v1-795d595484-hlbnk-istio-proxy.log

iptables-restore --noflush /tmp/iptables-rules-1594751504941469528.txt993273918
Writing following contents to rules file:  /tmp/ip6tables-rules-1594751504945812290.txt096504453
* nat
-N ISTIO_REDIRECT
-A ISTIO_REDIRECT -p udp -j REDIRECT --to-ports 15013
COMMIT

ip6tables-restore --noflush /tmp/ip6tables-rules-1594751504945812290.txt096504453
ip6tables-restore v1.6.1: ip6tables-restore: unable to initialize table 'nat'

Somehow the NAT tables is not created as expected.

@lambdai the  test failed 2x in a row with no apparent related configs. Maybe the actual nodes the test runs on changed?

I have the same feeling: It's likely the node/docker env causing it
This 4 liner ip6tables rules are absolute reasonable. My machine could restore the 4 liner or 2 rules perfectly.

https://prow.istio.io/view/gcs/istio-prow/logs/integ-distroless-k8s-tests_istio_postsubmit/1794 passed after this one. Lets keep an eye on this and see if any pattern comes up

fixed by #25590 I think


Cant put lights in full brightness after change to some color and come back to temperature

The result is that the brightness is 100% in the app but the lamp is only 50%

Hello @cpvbruno,
Unfourtunately this is a issue with the firmware after an update from Yeelight, so there is really nothing we can do about it:
https://forum.yeelight.com/t/topic/8594
https://forum.yeelight.com/t/topic/11026


dataLabelPosition option for Pie charts

Category

Enhancement

Version
Please specify what version of the library you are using: [   3.3.1     ]
Please specify what version(s) of PowerPoint you are targeting: [   360    ]
Expected / Desired Behavior / Question
When creating a Pie to have the dataLabelPosition option available like Powerpoint has, so we can position the labels for example outside the pie with outEnd.

The Chart API docs show this as an optoin for bar charts, but it should work with pie charts according to what i see in the source code.
https://github.com/gitbrent/PptxGenJS/blob/506af72a7a0e2d8694ca012d374521d7d1c7de6c/src/gen-charts.ts#L1457

@gitbrent  The line you mentioned has no effect. To work properly it should be replaced with:
strXml += "<c:dLblPos val="" + (opts.dataLabelPosition || 'inEnd') + ""/>";

Fixed via Pull #938


False Positive Pandas

In the version of pandas:2.0.3 is giving possitive to the CVE-2020-13091 which is applicable up to 1.0.3.
https://nvd.nist.gov/vuln/detail/CVE-2020-13091#range-5039694
PyPI::pandas:2.0.3 CVE-2020-13091 9.8

@Franco0700  This is likely coming from an data issue at the security vulnerability provider. Which security advisories provider are you using?  For a list of providers also https://github.com/oss-review-toolkit/ort#nexusiq.

Any update @Franco0700?

Let me check with Lucia.

Closed as part of backlog grooming. Feel free to comment if you would like to contribute to this.


Fixing coreos docs for proper operation.

Couldn't get heapster to properly work on coreos until I made the following changes.

Can one of the admins verify this patch?

I signed it!
On Tue, Jun 2, 2015 at 2:55 AM, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project, in which case you'll need to
sign a Contributor License Agreement (CLA).
[image: :memo:] Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/heapster/pull/304#issuecomment-107831182
.

@hyperbolic2346: Thanks for the PR! The coreos documentation does require some updates. I have opened #305 to fix the docs. Can you check if the steps in #305 work for you?

Added fleet files with updated versions from @vishh .

@cadvisorJenkinsBot: ok to test.

@hyperbolic2346: LGTM. Needs a rebase though.

LGTM


How to save the Embedding RecommenderNet Model

After building the model
https://keras.io/examples/structured_data/collaborative_filtering_movielens/
model = RecommenderNet(num_users, num_movies, EMBEDDING_SIZE)
model.compile(
    loss=tf.keras.losses.BinaryCrossentropy(), optimizer=keras.optimizers.Adam(lr=0.001)
)
history = model.fit(
    x=x_train,
    y=y_train,
    batch_size=64,
    epochs=5,
    verbose=1,
    validation_data=(x_val, y_val),
)

How to save the model
model.save('model.h5py')
tf.keras.models.save_model(model, overwrite=True, include_optimizer=True, save_format='h5')
tf.keras.models.save_model(model, overwrite=True, include_optimizer=True, save_format='tf')

Tried these, all throws
NotImplementedError: Saving the model to HDF5 format requires the model to be a Functional model or a Sequential model. It does not work for subclassed models, because such models are defined via the body of a Python method, which isn't safely serializable. Consider saving to the Tensorflow SavedModel format (by setting save_format="tf") or using `save_weights`.

using tf == 1.14.0, Thanks

I upgraded to tf==2.2.0, save_model and load_model worked
but after loading, I get valueError
    ValueError: Python inputs incompatible with input_signature:
      inputs: (
        Tensor("IteratorGetNext:0", shape=(None, 2), dtype=int32))
      input_signature: (
        TensorSpec(shape=(None, 2), dtype=tf.int64, name='input_1'))

May I know how to fix this please, I am looking for this solution asap, Thanks

I tried saving model as
tf.keras.models.save_model(model,'./saved_model',
                           overwrite=True,
                           include_optimizer=True,
                           signatures=model.call.get_concrete_function(
                               tf.TensorSpec(shape=(None, 2), dtype=tf.int64, name='input_1')),
                           save_format='tf')

Still not working

Actually recreating the model with
keras.models.load_model('path_to_my_model')

didn't work for me
First we have to save_weights from the built model
model.save_weights('model_weights', save_format='tf')

Then
we have to initiate a new instance for the subclass Model then compile and train_on_batch with one record and load_weights of built model
loaded_model = RecommenderNet(num_users, num_movies, EMBEDDING_SIZE)
loaded_model.compile(loss=tf.keras.losses.BinaryCrossentropy(), optimizer=keras.optimizers.Adam(lr=0.001))
loaded_model.train_on_batch(x_train[:1], y_train[:1])
loaded_model.load_weights('model_weights')

This work perfectly in TensorFlow==2.2.0


upgrade typer to 0.4.1

@Tobi-De can you update the version in pyproject.toml file to 0.6.1

@Tobi-De can you update the version in pyproject.toml file to 0.6.1

ok


Support numeric values from ops-metric-client cli

stg - #2134
cc @twiest

@twiest @tiwillia bot issue?

[test]

@kwoodson The error in the CI for this PR looks like the same error we investigated a week or two ago where pylint was missing from the oso-host-monitoring image.

@tiwillia, I don't think anyone has picked this up.  Pylint is still missing.

@tiwillia @kwoodson @twiest any way to skip the tests and merge?
This commit already passed CI in stg.

@zgalor I'm okay with skipping testing here, the testing passed in stg and the prod-specific parent check did pass here. I certainly don't have final say though, you'll want to confirm with @kwoodson or @twiest

@twiest @kwoodson ?

@zgalor, @tiwillia, @twiest, I'm uncomfortable merging this without actually testing it in our container.

[test]

[test]

[test]

[test]


Add integration with Jenkins CI

Jenkins plugin Warnings NG can read checkstyle reports about found issues. Is it possible to add such reporting format for BartyCrouch to get reports integrated in Jenkins CI build results?
this format is used for example in Swiftlint project

@nysander Thanks for this suggestion!
Could you please link to a documentation and an example of what the output should look like?
We could solve this by simply adding a new case named something like jenkins (or the format name) to the OutputFormatTarget enum here. Then, by adding a new option here similar to the already existing --xcode-output there could be something like --jenkins-output that you would be using when you need it on the CI.
Having outlined the way of how this could be tackled, feel free to post a PR with the changes needed to get this running sooner. Otherwise you might need to wait a little longer until I have time to look into this.

I do have almost any knowledge about this 😉 but I have found that swiftlint uses checkstyle report format which is parsable via this plugin without much problems. I have yesterday rewrited this parser for IBLinter project and made PR to test it on live jenkins if merged as I have no knowledge how to force jenkins use it without cocoapods.

IBLinter and SwiftLint uses --reporter ... format for different exports. format name should be checkstyle as this is XML format to which we must conform here - this format is not Jenkins' only format. Here you have example XML from IBLinter exported error:
test.xml.zip
reporter from IBLinter rewrites from SwiftLint one by myself: https://github.com/nysander/IBLinter/blob/checkstyle/Sources/IBLinterKit/Reporters/CheckstyleReporter.swift
reporter from SwiftLint: https://github.com/realm/SwiftLint/blob/master/Source/SwiftLintFramework/Reporters/CheckstyleReporter.swift

@nysander Sorry for getting back to you so late. I think a different output format can be implemented quite easily, just add a new option like we already have the -x option for Xcode compatible output, then add a case in this enum and implement the output file parser. I'd love to review and merge a PR adding this feature. :)

thanks for feedback :) I will look at it but cannot tell you the timeframe now unfortunately


update to version 1.1.11

Fixed problem with latest GDAL

@jakirkham this error seems similar, but slightly different (not the upload) from https://github.com/conda-forge/conda-forge-build-setup-feedstock/issues/60. Any thoughts?

Not sure about this one either unfortunately. 😕 😞 Though I have seen a few others with the same issue. I don't see anything in conda-build 2.1.7 that I think would fix it, but it may be worth retry with that version just in case. In any event, I think an issue should be opened with conda-build upstream about this.
xref: https://github.com/conda-forge/sunpy-feedstock/pull/21
xref: https://github.com/conda-forge/pyqt-feedstock/pull/16

Raised the AppVeyor failure in issue ( https://github.com/conda/conda-build/issues/1853 ).

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

That worked @jakirkham !


Modify pyproject.toml: Fix sdist didn't contain any files

I noticed that .tar.gz file on PyPI was almost empty. It didn't had any files except the 3-4 files. I think the source distribution should contain all the source files.


[RFC] claws-mail: enable PGP plugin by default

I think this is generally desired and overriding causes rebuilding of claws.
/cc @fpletz & @khumba

:+1: from me. I think it's good practice to have gnupg available by default. Encrypt all the things. :smile:
Also, the claws-mail Windows installer ships gnupg by default and asks you to generate a PGP key if you don't have one. By having gnupg support available by default we would follow the recommendations from upstream.


What do these various implementations mean?

What do these various implementations mean?
forward_core = forward_core_share_ssm
forward_core = forward_core_share_a
forward_core = forward_corev1
forward_core = forward_corev2
forward_core = forward_corev0
if  forward_core_share_ssm the orignal  s6 in mamba?

The forward_corev0 is the original s6 in mamba, and all the checkpoints we provided are based on it.

In forward_corev0, I wonder whether we can use the selective_scan_fn in the code you provided in the alter.. package?

Yes for now. In previous code, our selective_scan_fn is slightly different in inputs with mamba_ssm (we delete the z). But now, we use our own written  SelectiveScan which fits for either selective_scan_cuda or selective_scan_cuda_core.
We've tested in check_vssm1_equals_vssm(forward_type="v0"), that the code of new version acts the same with the previous one, so feel free to use it.


await till database is clean

I have a problem and is that the clean database script just returns the promise and the caller doesn't wait for it
it should look like:
beforeEach(async() => {
  await cleanDatabase();
});

I'm not using sequelizejs though, I adapted it to knex but I just checked sequelizejs documentation and it tructate returns a promise indeed.

The way it is right now the beforeEach should wait for the promise to be resolved since I return it from the function (https://github.com/talyssonoc/node-api-boilerplate/blob/master/test/support/cleanDatabase.js#L4). It is similar to returning a promise from an it, Mocha waits for it to be resolved.  awaiting a promise inside an async function works exactly the same as returning a promise from a normal function.


Areas in .net core 3 which is using endpoints

In .net core 3, endpoints are used to for routing. Could you update the docs.

Can you create an issue from the doc page? That will connect the doc to the issue. Go to the bottom of the page and select the Content feedback issue.


Issue running madge globally, excluding node_modules dir

I get this error when attempting this command
$ sudo madge --exclude ./node_modules --image graph.png .

https://www.dropbox.com/s/n4xd3k9xywgdtyp/Screenshot 2015-11-11 12.36.10.png?dl=0
the error is "Error: spawn dot ENOENT"
even with sudo I get the ENOENT, it is an "unhandled error event", please advise thanks

The call to --image graph.png seems to cause the problem, it does not seem to be the --exclude node_modules call...

sorry, my fault, I didn't have graphviz installed, my bad

my only advice, perhaps an error message saying that graphviz isn't installed, but otherwise just close this thanks

Which OS are you using?

Mac OSX 10.9x - I think it's not a big deal - just didn't know that graphviz wasn't instead.
I used
$ brew install graphviz
and it was all good after that


[HUDI-3469] Refactor HoodieTestDataGenerator to provide for reproducible Builds

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
Refactoring HoodieTestDataGenerator to provide for reproducible Builds: currently, HoodieTestDataGenerator relies on static state which make its state shared across all of the tests making data generation dependent on the order of execution.
Instead we should properly abstract HoodieTestDataGenerator to hold all of the state w/in individual instances so that individual Tests can:

Create they own isolated instance (which won't be affected by other Tests)
Pass "seed" value to DataGenerator to init its PRNG w/ it, so that it always produces the same (pseudo-)random sequence (for a given seed)
Be certain that all of the data produced by DataGenerator will be 100% reproducible w/ the same seed (meaning that all of the DataGenerator operations w/in it only rely on such internal PRNG and don't rely on any external sources, such as UUID.randomUUID(), System.currentTimeMillis(), etc)

Brief change log
See above
Verify this pull request
This pull request is already covered by existing tests, such as (please describe tests).
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure


[REQ]  : Keep last position

Hi,
I'd like Fritter to be able to keep the last position for when coming back to the app start from bottom (older) to up (newer) Tweets.
Thx,

Closing as a duplicate of #57. Please follow that issue for updates!