Add option to collect volume metrics from kubeletstats receiver

The kubeletstats receiver optionally collects volume metrics. Add an option to configure the receiver to collect these metrics. This will give us parity with the SA chart.

It is possible to configure any component in the chart. To configure kubeletstats receiver you have the following option:
otelAget:
  config:
    receivers:
      kubeletstats:
        <any kubeletstats config overrides>

Isn't it sufficient? Or do we want volume metrics enabled by default?

@dmitryax - yes, one could do that as well. PTAL https://github.com/signalfx/splunk-otel-collector-chart/pull/112 to add comments with the config required to collect metrics from volumes. Let me know if this is a better approach to take.

@asuresh4 we have usage examples here now https://github.com/signalfx/splunk-otel-collector-chart/blob/main/examples/README.md
Do you still want to add a kubeletstats example or we can close the ticket?

Closing the issue as resolved. @asuresh4 feel free to submit a PR to add a usage examples for volume metrics


Adding function to convert common boolean strings to a boolean type.

This is helpful when you have a truthy env variable and want it converted to true or false.

thanks for your contribution.

released as 1.3.4

Awesome, thanks!


Added alias scaffold for missing update_dependency_members method

My team was recently updating our Nagios cookbook only to find that it was failing on a NoMethodError for method update_dependency_members. Indeed, update_members is defined, but update_dependency_members didn't seem to be.
I didn't see an outright reason for the additional method as the hashes appeared to merge the same either way, but didn't want to clear it out in case it was there for some unspecified edge case or intended future use. An alias seemed to be the best middle ground until a proper solution could be determined.

I'm sorry, but this is not the right solution for this. I never used this feature so I have missed this, but update_dependency_members must call out to the push_dependency method on the object.
But you are looking in the right direction (base class) but I think we need to add something like this:
def update_dependency_members(hash, option, object)
    return if blank?(hash) || hash[option].nil?
    get_members(hash[option]).each do |member|
        push_dependency(Nagios.instance.find(object.new(member)))
    end
end

Please let me know if this works for you.

@sneakycactus did you had a chance to test the above? Just would like to know if this works or that your proposed solution is working for you as expected?

I did get an opportunity, and thank you much for looking into it.
Your solution is certainly in a better direction, and it almost works. When
I test it, it seems to retain everything currently in our
servicedependencies.cfg file, except for the service_description. I was
just working on seeing why that's the case.
I found that I can add a reference into config_options in
servicedependency.rb, and this almost fixes it, but on testing it sets the
service_description value to our data bag id instead of the
service_description name/value that's already there (and matches our
application name).
That's about where I stand at the moment.
On Thu, Mar 26, 2015 at 2:09 PM, Sander Botman<EMAIL_ADDRESS>wrote:

@sneakycactus https://github.com/sneakycactus did you had a chance to
test the above? Just would like to know if this works or that your proposed
solution is working for you as expected?
—
Reply to this email directly or view it on GitHub
https://github.com/tas50/nagios/pull/346#issuecomment-86650929.

That is correct, the data bag import takes the id as the unique name and pushes it into the service_description field and takes that as the unique name within the model. I guess we could search for the service_description within the databag and make that one leading if it exists. But I believe the id was used in the old configuration also, that's why I didn't changed it.
Let me just dive into this very quickly to see if I can make this magic happen...

So it's the recipe _load_databag_config.rb thats filling the configuration model with all the databag information. Here it's already creating the objects with the id of the databag, making that the service_description because that's required by the initialisation of the object and cannot change.
servicedependencies = nagios_bags.get(node['nagios']['servicedependencies_databag'])
servicedependencies.each do |item|
  nagios_servicedependency item['id'] do
    options item
  end
end

So we could change this to something like:
servicedependencies = nagios_bags.get(node['nagios']['servicedependencies_databag'])
servicedependencies.each do |item|
  name = item['service_description'] || item['id']
  nagios_servicedependency name do
    options item
  end
end

Would that work for you?

That worked perfectly, thank you, exactly what I was looking for! With this in place, and along with a few other adjustments on our side, I was finally able to get our cookbook to update cleanly (from version 6.0.2, for reference.)
Updated with modifications as discussed.

I've run several tests both earlier and now, and it is indeed explicitly required.

Well Chris, many thanks for testing and fixing this. Still don't understand 100% why the library update is needed. Could you provide some more info on what's going wrong exactly if we remove it, because I really would like too get my head around this. Cheers sander

Hi Chris,
I have merged your request, I suddenly remembered that the definition output uses this hash to gather the information. Without the line inside the library you probably didn't get the right output within your configuration file.
Tnx again!.

No, thank you!


[🛑 DO NOT MERGE] fix(test): force test failure

Description
Assure 'test' Check Run is always executed and fails/succeeds accordingly to the results of its dependency jobs.

Example commit with all tests passing ✅: c4a2c50e7eb30baa6b3fb54c75e3c31429c591ed
Example commit with tests failing 🔴: 8d93278aa268be6706f97c973a17d3d05cbac53f

Context
When using job.needs, by default only runs when the dependency jobs succeed. If one or more of them fail, the dependent job is skipped

@gr2m @wolfy1339 @timrogers I think I found the issue. I'm surprised this did not appear before.
What do you think about my solution proposal? Do you agree with solving this in the context of https://github.com/octokit/.github/pull/13?

What is your proposed solution? I'm not sure I understand from this PR.

Sounds reasonable to me 🤷🏻

What do you think about my solution proposal? Do you agree with solving this in the context of https://github.com/octokit/.github/pull/13?

This will work for now, until we sort out the reusable workflows

This will work for now, until we sort out the reusable workflows

@wolfy1339
Do you mean you want me to apply this change to all our repositories until https://github.com/octokit/.github/pull/13 is addressed?

This will work for now, until we sort out the reusable workflows

@wolfy1339 Do you mean you want me to apply this change to all our repositories until octokit/.github#13 is addressed?

Yes. That would be great

Here is the discussion we had in @semantic-release, we arrived at the same conclusion, but did not yet implement the change
https://github.com/semantic-release/semantic-release/pull/2229


Update README.md

Makes it clearer in the readme which setup (image_preview.nvim or neo-tree.nvim) needs to be updated.

@loqusion Thanks 👍


"Load" a partial view from a string

Is there any way to load a partial view from a string rather than a .cshtml file on disk?
I'm thinking about having razor stored in a DB that I would like to have rendered as a partial view inside of a main view.
Thanks!

You could register an IFileProvider backed by the db to RazorViewEngineOptions.FileProviders. The RazorViewEngine looks for specific paths from file provider instances. As long as you're able to map the path to a db entry, it should just work.


amp-font is actually not builtin (?)

The amp-font sample claims "amp-font is builtin element and is automatically imported via the AMP runtime." Is this true?

https://www.ampproject.org/docs/reference/components/amp-font says it needs to be imported via <script>.
The sample itself includes the <script> tag.
Opening the playground on the amp-font sample source and removing the <script> results in a validation error.

Closed by #850


discourse: 2.9.0.beta1 -> 2.9.0.beta3

Description of changes
Update discourse to the latest beta. Since 2.9.0.beta2, discourse requires building with Ember CLI, so from now on we have to package the asset building yarn packages.
Todo

[x] Implement yarn package updates in update.py
[x] Update redis option name

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)
[x] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Great! I've now added the missing pieces and updated the plugins, so everything should be done.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/request-for-help-fixing-discourse-in-nixpkgs/18009/9

I'll merge this on Monday unless someone objects.

Haven't had time to check it out but I can do it on Monday.

Okay, I'll hold off until you've completed your testing.


Default values for "Global search" on "history" page

I think that there should be some default value on "Global search" page. From my usual search queries:

"Not compromised" is on, "compromised" is off
Result: "Unexpected" is ON, all others are off
Obtained results: all is ON

The only pending question for me is if we should include Unexpected-only by default.
No. 1 - by default all searches should exclude compromised
No. 3 - I don't understand what you mean

From my point of view, it would be better to use the default result obtained when clicking on the "Reset verdicts section" button.

@ol-damirm  you mean fro No3?

Yes, for No3

I would recommend 1, 3 - yes. 2 - no


Add additional logging provider documentation

Adds documentation for Heroku, AWS Kinesis/Cloudwatch & HTTP.

From a tone of voice, etc point of view I think this is good to merge. Might need @matsimitsu for a technical check.

There are a few inconsistencies, but we have a call in an hour to run through those :)

Is this ready to be merged, @noccor?


issue launching with vimscript init

Awesome plugin! Appreciate it.
I'm giving it a try, but having an issue using vim Plug with a vimscript init file.
If I place the recommended / default lua snippet you've provided to call setup in my init.vim I get the following error.
Error detected while processing VimEnter Autocommands for "*":
E5108: Error executing lua .../plugged/todo-comments.nvim/lua/todo-comments/config.lua:123: attempt to concatenate local 'fg' (a nil value)

I expected the defaults to apply, etc.
any ideas?

Fixed now. This happened for colorschemes that don't set a background and/or foreground for the Normal highlight group.
Can you check if it's ok now?

I had the same issue & now it works fine after the fix you pushed.

ok im getting
Todo: ripgrep failed with code 1
though i have ripgrep installed

Thanks!  I'm closing this as the issue above is resolved.  @danielnehrig  you may want to open a separate issue.


Dependancy down

Hello, I have try to install this application on docker, but the dependancy geo maxmind is down ..

Hello, I have try to install this application on docker, but the dependancy geo maxmind is down ..

I'm afraid we are out of luck with that:
https://blog.maxmind.com/2019/12/18/significant-changes-to-accessing-and-using-geolite2-databases/
Unless the dependency to MaxMind GeoLite DB is removed/fixed, this project is in a dead end.


Added Kailash Satyarthi Children's Foundation

Added to 'Helping Children live a better life"

added after reading issue #2

Looks good to me. 👍
Hope you will enjoy the journey with OpenGenus.


CFE still produces kernel files with kUnevaluatedConstant tags in them

For debugging purposes it would be better if CFE did not generate kernel files with the kUnevaluatedConstant tags in them and instead produced errors for the constants which could not be evaluated by the front end.

In which context are you seeing these unevaluated constants? When running the front end in the Kernel service as part of a JIT compilation, or stand-alone to produce a dill file?
If the front end is given an environment, and it does not encounter any unavailable constructors or variable initializers (as is currently the case sometimes for DDK), it should not produce any unevaluated constants.
We can certainly add an option to the front end to report an error if any constants could not be evaluated.

I have added an option in https://dart-review.googlesource.com/c/sdk/+/99121. Is this what you need?

I'm marking this as P2 (non-blocking).


Split credsgen in a separate golang module

Is your feature request related to a problem? Please describe.
EiriniX depends on parts of cf-operator code, the critical point is the credsgen package https://github.com/cloudfoundry-incubator/cf-operator/tree/master/pkg/credsgen see https://github.com/SUSE/eirinix/issues/23
Describe the solution you'd like
I'd like to split the credsgen module in a separate package from cf-operator. By looking at the deps of credsgen I couldn't see a valid reason to not do so. The credsgen package doesn't need the operator code as far as I can tell as it is independent (please correct me if I'm wrong) and it can be decoupled easily to be managed separately outside of kubernetes/client-go and controller-runtime bumps.
Describe alternatives you've considered
Duplicate cf-operator code on EiriniX

@mudler since you've got a solution, and we're not planning to make the operator more consumable as a library (at least right now), I'm going to close this


userhost-in-names provides inconsistent replies if unsupported by the server

Version: ZNC 1.7.x-git-536-0268f6d
An example:
CAP LS
:irc.znc.in CAP unknown-nick LS :batch cap-notify echo-message multi-prefix server-time userhost-in-names znc.in/batch znc.in/self-message znc.in/server-time-iso
CAP REQ :userhost-in-names
:irc.znc.in CAP unknown-nick ACK :userhost-in-names
CAP END

PASS _______
NICK captest
USER captest/freenode - - :captest

[welcome message, modes]

On a channel automatically joined the response is partially in the new form, however only for the current user.
<EMAIL_ADDRESS>JOIN :#alexendoo
:holmes.freenode.net 353 captest @ #alexendoo :@Alexendoo <EMAIL_ADDRESS>:holmes.freenode.net 366 captest #alexendoo :End of /NAMES list.

However for a normal use of NAMES #channel it appears to not work
NAMES #alexendoo
:holmes.freenode.net 353 captest @ #alexendoo :captest @Alexendoo
:holmes.freenode.net 366 captest #alexendoo :End of /NAMES list.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29551137-userhost-in-names-provides-inconsistent-replies-if-unsupported-by-the-server?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

Is ZNC doesn't know the hostmask for the nick there isn't really much else it could do. What do you suggest that ZNC does instead @Alexendoo?

I would suggest having it not advertise the capability if it isn't able to provide it, and to use cap-notify if that changes

An alternative solution could be for ZNC to make use of the WHO command to get the missing information before forwarding the NAMES replies on to the client. It's pretty reliable that the WHO command will list all users in the specified channel regardless of user mode +i so long as the issuing user is also in the channel.

ACKing userhost-in-names but omitting the hostmask from some entries in the replies looks like a pretty clear spec violation. The spec says "the full hostmask (nick!user@host) of every user listed". ZNC does not always include the hostmask for every user, therefore it is not following the spec.


Update bug 1450 test v0.2

Supersedes #742

Add maximum version 7.0.0 for previous test for bug-1450 (test was not well-encoded in DER, so new parser will reject it with an according error message).
Add 2 new tests for bug-1450, which were derived from the above one, but properly DER-encoded: one for certificate with incomplete (large) header, and one for incomplete (large) content

ping @victorjulien
https://github.com/OISF/suricata-verify/pull/695 introduced a false result to S-V
cf https://github.com/OISF/suricata/pull/7076#issuecomment-1054131241

Merged in #852


[SR-5579] Make sure we fully validate the 'where' clauses in protocols

We weren't fully validating the where clauses of protocols and their associated types, which could allow errors in those where clauses to be ignored (rather than diagnosed).

Resolves SR-5579.

@swift-ci please smoke test and merge


【BUG REPORT】windows copy file to root directory error

const cpfile = require('cp-file');
const path = require('path')

const src = path.resolve(__dirname, './index.js');
const dest = "D:\\"
cpfile(src, dest, {
 }).on('progress', (process) => console.log(process)).then(done => console.log({done}))
 .catch(e => console.log(e));

stack:
$ node index.js
{ NestedError: Cannot create directory D:\: EPERM: operation not permitted, mkdir 'D:'
at makeDir.catch.error (d:\code\tset\node_modules\cp-file\fs.js:82:8)
Caused By: Error: EPERM: operation not permitted, mkdir 'D:'
nested:
{ Error: EPERM: operation not permitted, mkdir 'D:' errno: -4048, code: 'EPERM', syscall: 'mkdir', path: 'D:\' },
errno: -4048,
code: 'EPERM',
syscall: 'mkdir',
path: 'D:\',
name: 'CpFileError' }
I found that it was 'make-dir' cause error
mkdir
On Windows, using fs.mkdir() on the root directory even with recursion will result in an error:

fs.mkdir('/', { recursive: true }, (err) => {
  // => [Error: EPERM: operation not permitted, mkdir 'C:\']
});

@sindresorhus  help plz !!!

I was having the same issue, fixed it by forking, adding a check and using it from my repo.
if(process.platform==="win32" && path.length===3 && path[1]===':') console.log("skipping root directory")
Yeah, I know, it is ugly but it works

It would be better if we replace
https://github.com/sindresorhus/cp-file/blob/589c637db381f93479031afaa8ab6ba907b7a957/index.js#L15
with
if(!(process.platform === "win32" && /\w:\\/.test(path.resolve(path.dirname(destination), "/))){
	await fs.makeDir(path.dirname(destination), {mode: options.directoryMode});
}


TypeError: client.getNewComments is not a function

Describe the bug
A clear and concise description of what the bug is.
I can't launch bot
(node:23869) UnhandledPromiseRejectionWarning: TypeError: client.getNewComments is not a function
To Reproduce
Steps to reproduce the behavior:

turn on bot
get (node:23869) UnhandledPromiseRejectionWarning: TypeError: client.getNewComments is not a function

Expected behavior
A clear and concise description of what you expected to happen.
Bot to turn on
System Configuration:
Using TypeScript? no
Node Version? latest
Snoostorm Version? latest
Snoowrap Version? latest
Additional context
Add any other context about the problem here.
here is my code
// ----- PACKAGES -----
const { CommentStream, InboxStream, SubmissionStream } = require("snoostorm")
const Snoowrap = require("snoowrap") // Snoowrap is the reddit API
const Snoostorm = require("snoostorm") // Reads comments/posts

// ----- FILE IMPORTS -----
const config = require("./config.json")

// ----- VARIABLES -----

// ----- BOT SETUP -----
const client = config.credentials

const streamOpts = {
    subreddit: config.subreddit,
    results: 25
};

const stream = new CommentStream(streamOpts);

// ----- BOT LISTENER -----

stream.on('comment', (comment) => {
    console.log('New comment: ' + comment);
});```

You need to pass your instance of snoowrap to CommentStream.

Fixed, but now im not getting any logs for comments


Election INF disappeared

After the latest Edmc version Election INF not showing up on latest BGS tally. checked logs plugin logs unable to see progress there also.
Before EDMC update ;
Activity after tick: 2022-12-18 13:20:50 (in-game time)
Hunza
[Talon Order] - .ElectionINF +14; .Expl 82.7K;  <<<<<---- expected to see.
Brokpa
[Brokpa Jet Life PLC] - .INF +2;
HIP 23824
[HIP 23824 Jet Universal Company] - .INF +2;
[HIP 23824 Independents] - .INF +2;
HIP 42539
[Workers Party of HIP 42539] - .INF +2;
Eban
[Official Eban Law Party] - .INF +2;
Ganebucae
[Talon Order] - .INF +17; .Expl 155K;
HIP 37735
[HIP 37735 Crimson Mob] - .INF -2;
Posted at 2022-12-18 22:39:13 (in-game time)                                                   BGS-Tally v2.1.0
Desktop (please complete the following information):

OS: Windows 11. latest update.
Plugin Version [e.g. 2.1.0]
EDMC Version [e.g. 5.7.0]

To be clear, is it the Election INF or the Exploration data that's missing, or both?

Election INF only atm. no issue with exploration data.

I'll need to get a copy of your player journal file to take a look at why the INF has gone missions. Join my server: https://discord.gg/XU6TgyED


update to a recent version of the CLI's DevKit

Describe the bug
https://github.com/angular/components/issues/21045
To Reproduce
Steps to reproduce the behavior:

run ng update @angular/core @angular/cli @angular/material in a project on Angular v10 to update to v11

Expected behavior
No exceptions.
Screenshots
N/A
Desktop (please complete the following information):

OS: macOS
Node Version: 14.15.0
Angular & Angular/CLI Version: 11.0.0 and CLI 11.0.1

Additional context
The deprecated Workspace APIs were removed in v11. It looks like these schematics need some updates to the newest CLI DevKit.

Thanks for reporting @Splaktar. I've update the dependencies and the test sandbox to Angular 11. Let me know if there's still an issue.
The latest package is now published at v 4.0.0


how to collect localhost log(no localhost-access)

I want to collect localhost.log to stderr ，can you help me?
I try to link stderr to localhost.log，but it  is splited multiple file by date

Sorry for the delay!  I think https://github.com/docker-library/tomcat/issues/84 has the information you're looking for. :+1:
In the future, these sorts of questions/requests would be more appropriately posted to the Docker Community Forums, the Docker Community Slack, or Stack Overflow.  Thanks!


Bazel broke when trying bazel build (...) image-retraining

Hello Ritchie,
I use your AMI (Ireland zone) with TensorFlow.When I try image-retraining (https://www.tensorflow.org/tutorials/image_retraining) and especially bazel build tensorflow/examples/image_retraining:retrain
I got an error:
ubuntu@ip-XXX-XX-XX-XX:~/tensorflow$ bazel build tensorflow/examples/image_retraining:retrain
ERROR: /home/ubuntu/tensorflow/tensorflow/core/BUILD:1017:1: no such package '@zlib_archive//': Error downloading [http://zlib.net/zlib-1.2.8.tar.gz] to /home/ubuntu/.cache/bazel/_bazel_ubuntu/ad1e09741bb4109fbc70ef8216b59ee2/external/zlib_archive/zlib-1.2.8.tar.gz: GET returned 404 Not Found and referenced by '//tensorflow/core:lib_internal'.
ERROR: Analysis of target '//tensorflow/examples/image_retraining:retrain' failed; build aborted.
INFO: Elapsed time: 6.759s
Just to let you know. I haven't found a solution for the moment
Regards, mph

Let me look into this, off the hand it seems that it's because of the build of TF. I might have to update it to fix this as I saw a similar bug reported on TF's official repository.

On 22 Feb 2017, at 5:38 AM, Marc-Philippe Huget<EMAIL_ADDRESS>wrote:
Hello Ritchie,
I use your AMI (Ireland zone) with TensorFlow.When I try image-retraining (https://www.tensorflow.org/tutorials/image_retraining) and especially bazel build tensorflow/examples/image_retraining:retrain
I got an error:
ubuntu@ip-XXX-XX-XX-XX:~/tensorflow$ bazel build tensorflow/examples/image_retraining:retrain
ERROR: /home/ubuntu/tensorflow/tensorflow/core/BUILD:1017:1: no such package '@zlib_archive//': Error downloading [http://zlib.net/zlib-1.2.8.tar.gz] to /home/ubuntu/.cache/bazel/_bazel_ubuntu/ad1e09741bb4109fbc70ef8216b59ee2/external/zlib_archive/zlib-1.2.8.tar.gz: GET returned 404 Not Found and referenced by '//tensorflow/core:lib_internal'.
ERROR: Analysis of target '//tensorflow/examples/image_retraining:retrain' failed; build aborted.
INFO: Elapsed time: 6.759s
Just to let you know. I haven't found a solution for the moment
Regards, mph
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@mphuget @ritchieng This is how I resolved mine.
The problem is that http://zlib.net/zlib-1.2.8.tar.gz no longer exist at http://zlib.net.
So I had to manually add the correct link in the workspace.bzl
$ cd tensorflow
$ vi tensorflow/workspace.bzl

change the section on zlib_archive to
native.new_http_archive(
   name = "zlib_archive",
   url = "http://zlib.net/zlib-1.2.11.tar.gz",
   sha256 = "c3e5e9fdd5004dcb542feda5ee4f0ff0744628baf8ed2dd5d66f8ca1197cb1a1",
   strip_prefix = "zlib-1.2.11",
   build_file = str(Label("//:zlib.BUILD")),
 )

This corresponds to the current release is publicly available.
Let me know of any other challenge you faced :)

Thanks Innocent, apparently, that fixed the problem. I tried to upgrade TF as proposed on the Internet but without success.

Thanks @mphuget for raising this issue and @Innarticles for posting a fix. I'll keep this in mind by testing for it when I'm building future versions of TFAMI.


SIMD enhancements

Several miscellaneous simd.h enhancements

addnot wraps the _mm_andnot_ps/_mm_andnot_si128 (with fallback when SSE is not available)

Slight tweaking of madd to be extra sure we get fma instructions with clang (requires direct use of the SSE registers and a compilation flag; gcc seems smarter).

Template SimdSize::size reveals the SIMD vector size of a type, or 1 for any type that isn't one of our simd types. Handy for certain kinds of template metaprogramming where the type might be a simd type or might be a scalar.

Updated after learning that there is a FMA preprocessor symbol after all! And in discussion with Chris, consensus was to be safe for now by telling the compiler not to emit fmadd ops unless explicitly asked to. We can revisit in the future. Also, for symmetry, I added nmsub.

New update:

use OIIO_FMA_ENABLED (triggered by either FMA or AVX2)
try to use actual std::fma(), which is presumably the correct intrinsic, when C++11 is available (I can't find a reliable intrinsic to use without C++11)
more symmetry: add msub and nmadd, so now we have the full complement of madd, nmadd, msub, nmsub.

Without C++11 can't you just use _mm_fmadd_ss ?

I'll try that

Update: use intrinsics directly when available.

OK with this as it ended up?

Yes - sorry, must have forgotten to hit send.
LGTM!


Join node not working any more

On v.1.20.2 joining a node is not working any more.
Syslog shows the following stack trace.
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[567882]: error: error running snapctl: unknown service: "microk8s.daemon-apiservice"
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]: [2021-02-16 17:12:59,916] ERROR in app: Exception on /cluster/api/v1.0/join [POST]
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]: Traceback (most recent call last):
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:   File "/snap/microk8s/2048/lib/python3.5/site-packages/flask/app.py", line 2447, in wsgi_app
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:     response = self.full_dispatch_request()
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:   File "/snap/microk8s/2048/lib/python3.5/site-packages/flask/app.py", line 1952, in full_dispatch_request
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:     rv = self.handle_user_exception(e)
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:   File "/snap/microk8s/2048/lib/python3.5/site-packages/flask/app.py", line 1821, in handle_user_exception
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:     reraise(exc_type, exc_value, tb)
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:   File "/snap/microk8s/2048/lib/python3.5/site-packages/flask/_compat.py", line 39, in reraise
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:     raise value
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:   File "/snap/microk8s/2048/lib/python3.5/site-packages/flask/app.py", line 1950, in full_dispatch_request
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:     rv = self.dispatch_request()
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:   File "/snap/microk8s/2048/lib/python3.5/site-packages/flask/app.py", line 1936, in dispatch_request
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:     return self.view_functions[rule.endpoint](**req.view_args)
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:   File "/snap/microk8s/2048/scripts/cluster/agent.py", line 316, in join_node_etcd
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:     service("restart", "apiservice")
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:   File "/snap/microk8s/2048/scripts/cluster/common/utils.py", line 277, in service
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:     "snapctl {} microk8s.daemon-{}".format(operation, service_name).split()
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:   File "/snap/microk8s/2048/usr/lib/python3.5/subprocess.py", line 581, in check_call
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]:     raise CalledProcessError(retcode, cmd)
Feb 16 17:12:59 pi-k8s-1 microk8s.daemon-cluster-agent[554105]: subprocess.CalledProcessError: Command '['snapctl', 'restart', 'microk8s.daemon-apiservice']' returned non-zero exit status 1

I believe it is a change introduced in this line. Probably should be apiserver instead of apiservice.
https://github.com/ubuntu/microk8s/blame/efde194e67312ab7f701d119a7bbbd183c096739/scripts/cluster/agent.py#L316
Setup is without ha-cluster. Joining node is a fresh snap.

Hi @oleo65 thank you for reporting this. We have reverted the offending change you should be able to add nodes now without problems.

Thank you for your feedback. With the latest update the node joining is working again as expected. 👍

Thanks @oleo65 for confirming  i will close this one then.

After all trial and errors, the only trick that has worked is to update /etc/hosts on each node with at least 2 entries.
 hostname
 hostname
Here, IP of the master node is what is shown post microk8s add-node command.
Also consider opening the firewall to allow the port : where microk8s master process is running (default on the master node).


Nuxt-edge support

Provide Nuxt-edge support

https://github.com/richardeschloss/nuxt-highcharts/tree/feat/nuxt-edge pushed.
Install with:

npm i "git+https://github.com/richardeschloss/nuxt-highcharts.git#feat/nuxt-edge"


JSONP WFS Popups

There is an issue with jsonp popups. They are not responding to clicks when they are nested in ajax call with oneachfeature method.
This is the jsfiddle link to what I am working on.

This is weird.
Can you reproduce this by just creating markers directly?
Maybe add them to a layergroup/featuregroup before adding them to the cluster.

Thanks for the answer Danzel.
I have reproduced it with no clustering and markers' popups were functional with no issues. This is the jsfiddle link.
Yet,upon your advice, when I tried to add them to a layergroup and then add to a cluster, I have ended up with the same issue. Not responding popups. Here is the link to the second result.

Sorry, I meant can you reproduce this without using the jsonp call and L.geoJson?

Sorry for my late answer, yet I have tried but could not succeed to figure out a way to do it without ajax call and L.geojson.

What about L.geojson with some small inline json?


Fix me forgetting to change the version when changing name of the pacakge.

F. Sorry I rushed and forgot to change the package version as well so it wasn't a complete fix :/

Fixup to https://github.com/DougDougGithub/Babagaboosh/pull/33

You were faster than me, but you fricked up with the version apparently :O

You were faster than me, but you fricked up with the version apparently :O

Unfortunately yes. Maybe it would have been smart to spend more time double checking :D

Merged in another PR


Changes needed for Rocky Linux 8+

Tested backward compatible with CentOS 7
Removed the .container suffix, links with that stopped working when we upgraded our host to Rocky Linux 8 with docker-ce 3:26.1.3-1.el8, which was unexpected. Folks will likely need to update their localdev .env files with the SQLALCHEMY_URL  that has just mysql instead of mysql.container as the host name.

Partially addresses https://github.com/freezingsaddles/freezing-web/issues/280 - this is in the middle of the OS upgrade cycle to Rocky Linux 9. This also fixed my localdev environment - I can check http://localhost:8000 now and get freezing-web, especially once I fixed up my local mysql installation's user permissions for the freezing user.


update propagation code with status sync

/hold
Will attempt to add e2e

/unhold
The e2e framework is working and it's currently testing auto import of managedcluster (gitopscluster CR) and part of propagation.
I will keep enhancing it in a new PR.

/lgtm


Use output folder for singler

In my comments on https://github.com/AlexsLemonade/scpca-nf/pull/476, I proposed a bunch of change, and I decided that it would actually be easier for me to just demonstrate and make some of the changes that I was thinking about, so I did that.
The main goal here was to change the output of the singleR process to be a folder with results rather than individual files. This folder also contains the scpca-meta.json file. This solves a couple of problems: rather than having to track a bunch of files separately, we just put the output files we want in the folder,  and then pass that along as a single unit. It also gives us a place to put scpca-meta.json where we won't be overwriting it or having to give it different names depending on the output.
For some reason, maybe my bad memory, I thought we wanted to just pass everything through the processes as simply as possible, so I added the cellassign refs to the singler process inputs & outputs so we can got to the next step without a join. (We could still drop the singleR ref before the cellassign process: it is larger than the others so that might be worth it)
Relatedly, I added a singler_output_ch mostly just to demonstrate how the mixing would go, with adding an empty_file for the skipped singler results. I also chose different names for the branches in anticipation of adding a 3rd branch, has_singler or similar, that would just fetch the existing results.
To make all of that easier, I followed as similar pattern that we used in af-rna.nf and elsewhere, where we had previously had output directories and step skipping: https://github.com/AlexsLemonade/scpca-nf/blob/2b3fb3cf4f39ba08851465de55b0d231e8059e4b/modules/af-rna.nf#L107-L111
Essentially we are now defining the output directories at the start of the subworkflow and adding them into the meta object so we can access them later, both within the process, as I do now, and as we implement step skipping.
Finally, for testing purposes, I changed one of the values in the stub file to be NA, and it did seem to work.

Related, regarding nextflow efficiency, is it maybe more efficient for nextflow to grab files from checkpoint directories (for the forthcoming add_celltypes process where we read annotations back in and add to SCE) vs passing them around as path()s? Or is it about the same? (edit - this is more of a question than a comment!)

I think either way could be about the same... I'm not sure I completely follow, but perhaps we could avoid some extra files being passed in by adding the reference paths to the meta dictionary and then getting them as required for the individual steps.
The thought would be that we could update the following chunk:
      celltype_input_ch = processed_sce_channel
        .map{[it[0]["project_id"]] + it}
        .combine(celltype_ch, by: 0)
        .map{it.drop(1)} // remove extra project ID
        // add celltype_publish_dir to the meta object for later use
        .map{
          it[0].celltype_publish_dir = "${params.checkpoints_dir}/celltype/${it[0].library_id}";
          it[0].singler_dir = "${it[0].celltype_publish_dir}/${it[0].library_id}_singler";
          it[0].cellassign_dir = "${it[0].celltype_publish_dir}/${it[0].library_id}_cellassign";
          it
        }

to also include updating it[0].singler_reference, it[0].cellassign_reference, etc. and then drop those files  from the output that we pass along in the channel. So we would just pass along [meta, proccessed_sce].
Then we could add .map{it.toList() + file(it[0].singler_reference) when creating singler_input_ch, and similar for cellassign_input_ch, which might be a bit simpler, but we would still want to carry along the singler output when going into
the cellassign process. I can pull together a version of that if you think it is something you want to see.

I can pull together a version of that if you think it is something you want to see.

No, it's fine - my question was actually related to this answer you gave me in https://github.com/AlexsLemonade/scpca-nf/pull/477#discussion_r1342674619. I was thinking that why pass around path() values if everything is stored in directories we have saved in meta (as strings!), and the answer is because that's how nextflow rolls.

👍
Just noting that depending on what monsters emerge (im not sorry for writing this) when merging this into my branch, I may close the open PR in #476 and start fresh. In fact now that I write this, that might be the right move anyways since many of those comments in #476 are now deprecated and long PRs are hard to follow!

This seems like the right move. Hang on a bit for one more option.


feat: Tolerate Basic Auth

Just so I can add some simple authentication myself as a dumb middleware in my reverse proxy. Thanks.
Actual:

At least Basic Auth would be great, but allowing integrating OIDC providers (Authelia, Authentik, KeyCloak) would be awesome


Support distinct when it comes to further association chaining

If you do object.association.distinct.other_association, the distinct goes away.
This is probably good for the simple implementation, but we could have a WITH DISTINCT automatically go in there on the second association

Hi @cheerfulstoic.
Is this something I can try to tackle? if so, maybe some code pointers?

Hey @noamyogev84
I'm not currently maintaining the Neo4j.rb gems but I'd be happy to start you in the right direction.  If you have a PR, though, somebody would need to be around to merge / release it ;)
has_n.rb is the main place for a lot of the association logic.
All of that is to support easy creation of QueryProxy objects.  QueryProxy objects represent the generation of higher-level queries in ActiveNode and which wrap the lower level Query objects from the neo4j-core gem
And, of course, I would check out the Query class for the neo4j-core gem


DateTime fields for ObjectAppendingMarker

Hi,
Is there any way of converting Joda's DateTime object to its string representation when we use Markers.appendFields(object) with logstash encoder? I couldn't find a way from documentations.
The default behavior is creating a JSON object with all date properties, something like:
"year":2012,"dayOfMonth":24,"dayOfWeek":1,"era":1,"dayOfYear":359,"centuryOfEra":20,"yearOfEra":2012,"yearOfCentury":12,"weekyear":2012,"monthOfYear":12

Hi @jama707 ,
Yes, you should be able to do this by defining a custom JsonFactoryDecorator as described here.
For example:
import net.logstash.logback.decorate.JsonFactoryDecorator;

public class JodaJsonFactoryDecorator implements JsonFactoryDecorator {
    @Override
    public MappingJsonFactory decorate(MappingJsonFactory factory) {
        factory.getCodec().registerModule(new JodaModule());
        return factory;
    }
}

and then configure the decorator in your configuration like this
<encoder class="net.logstash.logback.encoder.LogstashEncoder">
  <jsonFactoryDecorator class="your.package.JodaJsonFactoryDecorator "/>
</encoder>

I haven't tested this, but it should work.  Let me know if it doesn't.

Thank you, @philsttr ! I didn't know it would be easier with JsonFactoryDecorator, I used JsonGeneratorDecorator which is wrong even though it works.
public class LogstashDateTimeJsonProvider implements JsonGeneratorDecorator {

    public static final JodaModule MODULE = new JodaModule();

    /**
     * Joda DateTime objects are converted into their string representation instead of JSON Object.
     *
     * @param generator - Logstash JsonGenerator with default configurations.
     * @return JsonGenerator enriched with an ability to convert Joda DateTime to String.
     */
    @Override
    public JsonGenerator decorate(JsonGenerator generator) {
        ObjectMapper objectMapper = (ObjectMapper)  factory.getCodec();
        objectMapper.registerModule(MODULE);
        objectMapper.disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS);
        generator.setCodec(objectMapper);
        return generator;
    }
}

Yeah, it'll work with a JsonGeneratorDecorator.  However, the main reason to use JsonFactoryDecorator for this rather than JsonGeneratorDecorator is that the JsonFactoryDecorator will only be called once (during startup), but the JsonGeneratorDecorator is called for every event.  So doing configuration in a JsonFactoryDecorator will be more efficient.


refactor: Use rewrites instead of custom server

This is a first start at refactoring out usage of a custom server.
This relies upon the incoming rewrites NextJs feature, which is still in an experimental phase. By combining the rewrites array with middleware inside _app's getInitialProps, we can achieve the same result as what previously required a custom server.
Note that until NextJs supports middleware for normal routes (not just API routes), we'll need to take this getInitialProps middleware approach.
For rewrites to work, localeSubpaths need to be available both at build time (for NextJs to implement rewrites), and at run time (for next-i18next to implement 302s, etc). To make this as easy as possible, I've dropped localeSubpaths into publicRuntimeConfig in the example.
The future of next-i18next will involve periodic and signficant rewrites as NextJs slowly releases the features which we need to actually support serverless targets in a first class way. Notable examples are normal-route-middlewares and plugin support. This work, however, is a good first start to these incremental refactors.
Links
https://github.com/zeit/next.js/issues/9081
https://github.com/zeit/next.js/issues/9133
https://github.com/zeit/next.js/issues/7208

What could be really useful for me would be to have an example of updated README.md to know exactly how the setup changes and if there are any trade-offs :)

@martpie Sure, I've updated the README now. However, I'm mostly hoping for actual code reviews, as I'm not sure if I've missed any edge cases, especially in the implementation having to do with rewrites and passing the lang as a query param.
Here's how the setup varies from the current next-i18next flow:

No need for a custom server whatsoever
If you want locale subpaths, you need to pass them in via your next.config.js as well

That's it. There shouldn't be any trade-offs, unless I've missed something!

It looks really great, I'll try to go through the changes.
Is there any plan to support getServerSideProps? (or getStaticProps)

@martpie The main drawback at the moment is that rewrites is still experimental on the NextJs side, so we can't really consider this production ready in any sort of way.
As far as the other data fetching methods: at first glance it seems like getServerSideProps is the perfect fit, but _app cannot have getServerSideProps (https://github.com/zeit/next.js/discussions/10874).
And getStaticProps would never work, as we need to initialise an i18n instance based on the request language.
Keep in mind that next-i18next will change dramatically when first-class plugin support lands.

Ah: something I'm not currently sure of, is if importing the middleware in app-with-translation is causing that code to be pulled into the client bundle. That would be something to investigate. We may have to do more env-specific dynamic imports.

Another thing to check is if this rewrites approach is even correct. We need to whitelist the value of the locale subpath, but pass the key into query.lang.

Importing the middleware in the app-with-translation hoc might cause the loading issue you described. With my experiment, which I did some time ago, I have integrated a custom middleware in _document, and the bundle sizes just exploded. Also, google speed insights tells that a large portion of the loading time gets spent interpreting code, but there is not much there. Removing the hoc entirely results in bundle sizes of a few dozen kb max, so I guess everything was pulled to the client. Another thing that made things for me very complicated to understand is the app-with-translation hoc and its internals. In the context of SSR, which is already quite challenging, things like hoisting, possible side effects with the other libraries included made things very complex to go forward. I have not looked at the project next-translate, but what they are proposing is that _app.js, _document.js, and _error.js are not going to be wrapped with the translations context, so it's not possible to translate these files directly. Maybe this is worth some exploring.

@gurkerl83 Not sure if the NextJs tools are detailed enough to be trusted on this. Something like webpack-bundle-analyzer is probably better.
If the NextJs tools are to be trusted, here are the two outputs:
With Custom Server (existing implementation)

With Rewrites (new implementation)

Feel free to give it a try yourself. It looks like there's no difference, however I'm hesitant to take that at face value.

@isaachinman Unrelated to your PR, but I'm considering an alternative approach. See https://github.com/UnlyEd/next-right-now/pull/42#issuecomment-621707188
Maybe you'll learn something from it, in short I intend to use rewrites with SSG, but I won't use _app at all, and I'll use an api endpoint for the cases where the language isn't specified in the url.
I believe my approach will work nicely for SSG, and should also work nicely for SSR (using a getInitialProps in SSR pages to detect the language). It's still very theorical/experimental though, I'm working actively on it.

How work custom routes with rewrites ?
I'm working with diferent paths
Like:

example.com/en/test
example.com/es/prueba

I solver with
  rewrites: async () => [
    ...nextI18NextRewrites(localeSubpaths),
    // EN Routers
    ...englishPaths.map(({ source, destination }) => ({
      source,
      destination,
    })),

    // ES Routers
    ...spanishPaths.map(({ source, destination }) => ({
      source,
      destination,
    })),
  ],

In case someone is helpful

@SantiagoMolinaOrozco What are englishPaths and spanishPaths? I think that is indeed very helpful, and we should add it to the documentation.

sure, i'm goint to share the full example:
routes.js
const spanishPaths = [
  {
    source: "/terminos-y-condiciones",
    destination: "/terms",
  },
  {
    source: "/brokers/:slug",
    destination: "/brokers/[slug]",
  },
  {
    source: "/brokers",
    destination: "/brokers",
  },
];

const englishPaths = [
  {
    source: "/terms-and-conditions",
    destination: "/terms",
  },
  {
    source: "/brokers/:slug",
    destination: "/brokers/[slug]",
  },
  {
    source: "/brokers",
    destination: "/brokers",
  },
];

module.exports = { spanishPaths, englishPaths };

next.config.js
const { spanishPaths, englishPaths } = require("./routes");
const localeSubpaths = { es: "es", en: "en" };

const nextConfig = {
  webpack: (config) => {
    config.node = {
      fs: "empty",
    };
    return config;
  },
  publicRuntimeConfig: { localeSubpaths },
  rewrites: async () => [
    ...nextI18NextRewrites(localeSubpaths),
    // EN Routers
    ...englishPaths.map(({ source, destination }) => ({ source, destination })),

    // ES Routers
    ...spanishPaths.map(({ source, destination }) => ({ source, destination })),
  ],
};

somthing like that, i try to summarize my current configuration, improvement recommendations are received.


Add readme section for good practices

There are certain solutions which allow to use cache for some interesting use cases (e.g. #171). Adding a section to highlight such use cases and potential good way to solve them.

#840


Access Factboxes from Extensions with CurrPage

Describe the request
When trying to access a Function on a Factbox, that is not in the Base App, but in another Extension, it is not possible to access it through CurrPage.[FACTBOX].Page.[FUNCTION NAME].
The Following Error is beign thrown:

The Function "Refresh" is a global Function on the Factbox Page. The Factbox can be accessed with modify in the Layout Section.
Base App Factboxes can be accessed with CurrPage:

Additional context
We have Code that needs to be Updated while the User is on the Page. A normal CurrPage.Update() does not suffice

Unfortunately, this repository is only for Business Central extensibility requests or code contributions towards the 1st party Business Central apps.
For suggestions to the Business Central product group, please create a BC Idea.
If you want to log a support case, please visit:
On premises - https://support.microsoft.com/en-us/supportforbusiness/productselection?sapId=93d37907-ad94-d591-22e9-593cfa09dd3f
Cloud – contact your partner, or if you are a partner open a support incident through Partner Center - https://partner.microsoft.com/en-us/pcv/dashboard/overview


BYOL setup error: undefined symbol: _ZNK6google8protobuf7Message11GetTypeNameEv

Help! I followed the setup steps of byol README exactly, but as soon as I run python -m byol.main_loop \   --experiment_mode='pretrain' \   --worker_mode='train' \   --checkpoint_root='/tmp/byol_checkpoints' \   --pretrain_epochs=40 or python -m byol.main_loop_test I got the following error:
Traceback (most recent call last):
  File "/usr/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/content/byol/main_loop_test.py", line 21, in <module>
    from byol import byol_experiment
  File "/content/byol/byol_experiment.py", line 25, in <module>
    from acme.jax import utils as acme_utils
  File "/usr/local/lib/python3.7/dist-packages/acme/__init__.py", line 35, in <module>
    from acme.environment_loop import EnvironmentLoop
  File "/usr/local/lib/python3.7/dist-packages/acme/environment_loop.py", line 26, in <module>
    from acme.utils import signals
  File "/usr/local/lib/python3.7/dist-packages/acme/utils/signals.py", line 22, in <module>
    import launchpad
  File "/usr/local/lib/python3.7/dist-packages/launchpad/__init__.py", line 36, in <module>
    from launchpad.nodes.courier.node import CourierHandle
  File "/usr/local/lib/python3.7/dist-packages/launchpad/nodes/courier/node.py", line 21, in <module>
    import courier
  File "/usr/local/lib/python3.7/dist-packages/courier/__init__.py", line 26, in <module>
    from courier.python.client import Client  # pytype: disable=import-error
  File "/usr/local/lib/python3.7/dist-packages/courier/python/client.py", line 30, in <module>
    from courier.python import py_client
ImportError: /usr/local/lib/python3.7/dist-packages/courier/python/libserialization_cc_proto.so: undefined symbol: _ZNK6google8protobuf7Message11GetTypeNameEv

I have tried many different python version and OS:

Python 3.9, 3.8, 3.7, 3.6
Linux Ubuntu 20.04 LTS, Ubuntu 18.04.6 LTS (Google Colab), windows10,  Jetson Nano Image jp461 (linux OS nvidia cuda embedded)

There is no any error with the installation process, but as long as the execution of that step (python -m byol.main_loop --...) is to report an error (...undefined symbol: _ZNK6google8protobuf7Message11GetTypeNameEv).
I searched the internet and it seems to be related to protobuf. I tried to update the version or reinstall another version, but because of the dependency of the package, I ended up installing only protobuf version == 3.19.6 (default), protobuf version == 3.19.5
I also tried to compile jax, jaxlib, protobuf by myself (download the source code from their github), but it still didn't work.
I have also tried the following GPU hardware:

Nvidia RTX 3090
Nvidia GTX 1080Ti
Nvidia GTX 1070
Nvidia Jetson Nano 4GB

But all of them are not working. Can anyone please provide me with the software and hardware environment settings that I can be sure will work?

Were you able to find a solution to this?
I am facing the exact same issue.

Is there any update on this?

I was facing a similar issue, with Launchpad.
...
  File "/home/callum/miniconda3/envs/protobuf_issue_test/lib/python3.9/site-packages/launchpad/nodes/courier/node.py", line 21, in <module>
    import courier
  File "/home/callum/miniconda3/envs/protobuf_issue_test/lib/python3.9/site-packages/courier/__init__.py", line 26, in <module>
    from courier.python.client import Client  # pytype: disable=import-error
  File "/home/callum/miniconda3/envs/protobuf_issue_test/lib/python3.9/site-packages/courier/python/client.py", line 30, in <module>
    from courier.python import py_client
ImportError: /home/callum/miniconda3/envs/protobuf_issue_test/lib/python3.9/site-packages/courier/python/libserialization_cc_proto.so: undefined symbol: _ZNK6google8protobuf7Message11GetTypeNameEv

As you can see, there is the same missing symbol: _ZNK6google8protobuf7Message11GetTypeNameEv.
I used unix's library dependency checker on the cited shared object file: (ldd /home/callum/miniconda3/envs/protobuf_issue_test/lib/python3.9/site-packages/courier/python/libserialization_cc_proto.so), as suggested in https://github.com/NVIDIA-AI-IOT/torch2trt/issues/53, which yields the following:
...
        libtensorflow_framework.so.2 => not found
...

I first checked that I had tensorflow installed:
(protobuf_issue_test) ➜  callum-tilbury ✗ pip show tensorflow
Name: tensorflow
Version: 2.11.0
Summary: TensorFlow is an open source machine learning framework for everyone.
...

and did similar things for associated TF packages, tensorflow-datasets, etc. But they were all there already.
I then progressively downgraded tensorflow, eventually to v2.8, pip install tensorflow~=2.8.0, and it fixed the issue. Interestingly, ldd still shows libtensorflow_framework.so.2 as not found.
This approach isn't exactly a solution, and feels more like a hack. But perhaps it's a step in understanding why things are breaking. Hopefully it helps you :)


Unable to add butterknife

I am getting the following error when I am syncing the project after adding dependencies
Error:(27, 0) Could not find method annotationProcessor() for arguments [com.jakewharton:butterknife-compiler:8.7.0] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

The annotationProcessor configuration has been present since Android Gradle plugin 2.2.0. Ensure you are using the latest version which is 2.3.3.


byobu, screen, tmux non-functional in local shell

If I recall, the un-altered lxss environment includes byobu, screen, and tmux.
Regardless, they all either error or no-op (exit without any output) immediately due to incomplete procfs and devfs.

Known issue.  We are working on support.  No ETA at this time.  #40

Ah, touche. Buried. Sorry for the dupe


click on the edge of show more popup triggers onSelectSlot event

Do you want to request a feature or report a bug?
I believe this is a bug
What's the current behavior?
When one has a selectable calendar with onSelectSlot AND popup=true, clicking on the edge of the popup (e.g. the date header of the popup) triggers onSelectSlot.
I've created a sample below, please open the browser console and you will see onSelectSlot being fired when clicking on the edge of the show more popup
Sample: https://codesandbox.io/embed/kind-poincare-ietkd
What's the expected behavior?
Is it possible to prevent this from happening? I'm planning to add a  button on the top right corner of the show more popup so that I will be able to close the popup without having to click outside of the popup. But the current behavior blocks me from doing so and I'm not sure how to overcome it.
Thanks

This is indeed a bug according to me. Thanks for pointing it out.
I will see what I can do to fix it

Any help on this?
I had a look the code but I couldn't understand why this happens.


fix: Fix Camera not stopping when isActive is set to false

What
We forgot to call stop repeating if isActive is set to false again. This fixes the issue.

Changes

Tested on

Related issues

Fixes https://github.com/mrousavy/react-native-vision-camera/issues/2358

I'm facing this issue now.


Can not safely call ReadOnlyDapperRepository.Dispose multiple times

AutoFac expects classes to be able to be disposed of multiple times, so using DapperRepository with AutoFac causes NullReferenceExceptions when AutoFac goes to dispose of objects.  There should be a protection around the call to Connect.Dispose to check for a null Connection to protect against this.

#164

@harshitgindra  Thank you!
Published in 1.16.1


Visual Studio Code 1.58.2 compatible version

Hi All,
Could you please help to download extension compatible with VSC version 1.58.2?
Thanks

You can click on the gear icon in the bottom right for the extension that you're interested in and use the Install Another Version option to find an extension which is compatible with the current version of VS Code that you have installed. As v1.58.2 was released in June 2021 I would also recommend to update your install of VS Code.

This function not available on my VSCode version. I could not update it based on internal corporate rules.


Can't boot on VirtualBox

** RancherOS Version: 1.3.0**
** Where are you running RancherOS? VirtualBox **
See this video, captured by VirtualBox: https://webmshare.com/z4b9j
No idea what's going on, i've tried both chipsets, removed all unnecessary devices and tried both NAT and bridged network...

Turns out 512MB is not enough and results in a hard crash.
While I don't mind the requirement of 1GB; would it be possible to check memory requirement and die in a more friendly way?
(I'm assuming the issue comes from the fact that it's using a ramdrive which obviously requires RAM)

The doc has been updated.


It doesn't work in Unity 6 (6000.0.5f1)

NOTE: Your issue may already be reported! Please search on the issue tracker before creating one.
Describe the bug
It doesn't work in Unity 6 (6000.0.5f1)
To Reproduce
It's not being rendered.
Expected behavior
Should work.
Screenshots

Environment (please complete the following information):

Version 1.0.2
Platform: Windows Editor
Unity version: 6000.0.5f1
Build options: N/A

Additional context
We've used this package fine for a long time - but it just stopped working after unity upgrade. :(

The issue is not reproduced on my environment...

Version 1.0.2
Platform: Windows Editor
Unity version: 6000.0.5f1

BTW, your inspector seems to look different from the default one.
Are you using any other editor extension plugins (like Odin, etc.)?

Thank you for the hint! It was indeed a different package (MyBox) that had a silent conflict. I'll open up a feature request on their GitHub to maybe integrate with this instead because their feature seems less useful. :)


fix(widgets/list): fix line length calculation for selectable lists

The code that outputs the list elements uses the full inner width of its
block, without taking the width of the highlight symbol into
consideration. This allows the elements to overflow the box and draw
over the block's border. To fix that, we need to reduce the target width
for the list elements.

Thanks !


feat: default to legacy environments

While implementing support for service environments, we unearthed a small number of usage issues. While we work those out, we should default to using regular "legacy" environments.

One minor problem here is that we no longer test the default value for legacy-env.
I'll add it to my config validation PR.


DRAFT

WARNING: THIS IS A DRAFT
I want to test if it is possible to reproduce the freeze on Windows when running the Eowyn workflow, when zig fmt --check fails.

What is strange is that the Eowyn workflow fails, but the CI workflow runs correctly (but running the unit tests in windows-latest is taking a lot of time).

but the CI workflow runs correctly

These messages are from the CI: "Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16<EMAIL_ADDRESS>
Do you think it is ok to set job.id.strategy.fail-fast = true?

It's worth a try.

but the CI workflow runs correctly

These messages are from the CI: "Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16<EMAIL_ADDRESS>
Do you think it is ok to set job.id.strategy.fail-fast = true?

It's worth a try.
Update: True? Is this not the default?

You are right, it was a typo.


Rhino: Sending one object does not create "layers"

When sending a single object from Rhino, it seems that the connector does not create a base object with the layer structure. This causes problems down the line, e.g. receiving said commit in grasshopper and "expanding" it - there's nothing to expand.
It's a curious exception, but worth looking into!

I think connectors should handle such case, so it's more of a GH issue than Rhino, no?
From Dynamo, if you're sending 1 single object/element it's not nested at all. Same might happen when using the pyhton/c# apis in a scripting environment....

Tested this and couldn't reproduce - single objects are being sent in a base object with the correct layer container


Ignore case sensitivity of columns for case_sensitive=False

Fixes #1088.
This pr adds a lowercase version of column names to the DaskTable and column container mapping if sql.identifier.case_sensitive=False allowing all columns to be treated as lowercase.
For a table with column name Ab using ab/AB/Ab/aB will all work with the config option.
This PR doesn't enable true case sensitivity when set to True since datafusion implicitly ends up converting all unquoted identifiers to lowercase during logical plan creation (Ref: https://github.com/apache/arrow-datafusion/issues/5626)

Codecov Report

Merging #1090 (00b12e5) into main (5357625) will increase coverage by 0.71%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1090      +/-   ##
==========================================
+ Coverage   80.90%   81.61%   +0.71%
==========================================
  Files          78       78
  Lines        4384     4385       +1
  Branches      786      788       +2
==========================================
+ Hits         3547     3579      +32
+ Misses        666      627      -39
- Partials      171      179       +8

Impacted Files
Coverage Δ

dask_sql/context.py
93.89% <100.00%> (ø)

dask_sql/physical/rel/logical/table_scan.py
88.63% <100.00%> (ø)

... and 4 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

w sql.identifier.case_sensitive=False, this PR solved my problem :)


crossplane: pin to v0.6

Signed-off-by: hasheddan<EMAIL_ADDRESS>
Description of your changes

This PR pins stack-gcp to Crossplane post update of v0.6.0 to use crossplane-runtime  v0.3.0.
Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[x] Updated any relevant documentation, examples, or release notes.
[x] Updated the dependencies in app.yaml to include any new role permissions.

Should we pin exactly 0.6 instead?

@negz unfortunately not because v0.6.0 is not pinned to crossplane-runtime v0.3.0. However, we did not deem that pinning to that v0.3.0 merited a patch release, so we are just opting for using the commit where crossplane was pinned after v0.6.0


iOS:how to use freerdp in my own project

According to /docs/README.ios , I have built a ifreerdp xcode project successly. And I want to use freerdp in my own project, I don't know how to,should I make the src into a .lib or .a something like a library or what? Could anyone nice help me?

@guanyutaba  which version of SDK ?  ios8.1, ios 7.1 or other version ?


[Serialization] Do not assert when inherited type is null

When compiling with allow errors, it's possible to have invalid
inherited types - both null and ErrorType.
Cleaned up the tests a little - moved the majority of
Frontend/allow-errors.swift into separate files in
Serialization/AllowErrors and use split_file.py instead of #defines.
Resolves rdar://78048470

@xymus what are your thoughts on the asserts? Should I add a check for whether compiling with errors or leave as is?

@swift-ci please test

Yeah, I think it would be worth taking into account AllowModuleWithCompilerErrors in the asserts. It can prevent us from introducing future regressions.

Yeah, I think it would be worth taking into account AllowModuleWithCompilerErrors in the asserts. It can prevent us from introducing future unrelated regressions.

While updating this I realised that addTypeRef already has a null type + error check, so adding another would just be duplicating it.


Network gradually becomes unavailable

Current Windows version: 10.0.17763.475
Distribution: Debian
Problem: Seems like ports will become unavailable over time. First I can't view websites, but can ping. Then I can't ping. And finally Windows will see it as no network access. It all happens over a couple of hours. Rebooting Windows will restart the process.
Explanation:
I wanted to use WSL as a webserver running NGINX instead of using Docker. So I uninstalled Docker and had some leftover Hyper-V instances, and the problem occured. So I thought it was Hyper-V that was causing it and I reset Windows fully. Now I tried it on a fresh install and it's happening again.
The only fix that seems to work for me is to uninstall WSL, so I've done that for now.
Im ready to reinstall it again in case more info needs provided.

Sounds like #3951, maybe. See if you've got a large number of processes lingering in RamMap (message). Try upgrading to Insider 18890 if you can, and see if that helps.

That seems to be correct. Did a netstat -apoq and observed BOUND ports from the top down.
Closing


C++ Debugger Error

🐛 Bug Report
I can't seem to get the c++ debugger to work. Terminal does not show any errors but the debugger does not stop on any of the breakpoints I set.
To Reproduce
Clicking the debug button at the bottom of my solution.
Expected behavior
Expect the debugger to stop on the selected break points.
Extension Output
Debug LeetCode window empty
Debug console shows
=thread-group-added,id="i1"
GNU gdb (GDB) 11.2
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "x86_64-w64-mingw32".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<https://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
    <http://www.gnu.org/software/gdb/documentation/>.

For help, type "help".
Type "apropos word" to search for commands related to "word".
Warning: Debuggee TargetArchitecture not detected, assuming x86_64.
=cmd-param-changed,param="pagination",value="off"
[New Thread 10756.0x32b4]
[New Thread 10756.0x30c4]
[New Thread 10756.0x1f80]
[Thread 10756.0x13cc exited with code<PHONE_NUMBER>]
[Thread 10756.0x32b4 exited with code<PHONE_NUMBER>]
[Thread 10756.0x30c4 exited with code<PHONE_NUMBER>]
ERROR: Unable to start debugging. Unexpected GDB output from command "-exec-run". During startup program exited with code 0xc0000139.
The program 'c:\Users\~~~\AppData\Roaming\Code\User\globalStorage\wangtao0101.debug-leetcode\cppproblem844.exe' has exited with code 0 (0x00000000).

Do you slove this problem?Now I face the this bug similar with you.

Unfortunately I didn't.


Generate Not Working

When I click Generate, another tab comes up like it should, and I see the output cards for a split second. But almost immediately the page becomes completely blank except for the text "[object Object]" in the upper-left side of the page. Do you know why this could be, or how I can fix it?
Thanks!

Works for me in Chrome, Firefox, and Safari. Which browser are you using? And have you tried disabling any plugins?

I had not, and what do you know, but the very first plugin I block did the trick. It was Goodblock Ad Blocker. I thought simply turning it off for the website was enough, but clearly it wasn't. It's working just fine now, thank you.


Possible maloja fork support

I sent an email in August about this topic but it might've gone to spam.
I'm considering forking Maloja and adding some of my own features. One of the first features I add will be a new, separate spot for identifiers that all clients can use since it'll be a part of the API standard. There are some more ideas I have in mind which include a "now playing" feature that will require changes on multi-scrobbler's side, and I can pull request those if needed.
I spoke to a couple of other active Maloja users who have said they will use my fork if I make one. But if changes need to be made on multi-scrobbler's side, there will have to be a fork of that too, and it will be tough to maintain both. Ideally, we could add a config flag for your maloja client that enables features from my fork. It seems that including new parameters that my new API changes would take causes warnings from the server, so it would be best to include even just a hidden and off-by-default option to send these parameters alongside requests.

MS already does a bit of "if maloja version is X or Y" due to supporting pre-3.0 versions.
compare-versions handles semver with tags just fine so my recommendation would be to use a tag in your fork's versionstring that MS could use to determine if it should send additional fields in the api call IE versionstring = '3.2.3-duck' or something.


Return usage for all requests

Include number of input and output tokens for all requests: (regular, openai) x (streaming, batch) in the response object.
I know the official openai backend doesn't include such information, but the clients are fine. Also it is a very requested feature.

Hi @ichernev,
Thank you for your PR. Currently, we are trying to keep API server as minimal and encourage folks to use the OpenAI version. If you can limit this PR to just adding streaming support for OpenAI that would be great. We believe there's already support for usage in the non-streaming case.

@simon-mo  code tweaks only openai endpoint now. Edited title & description.


Dalitz plot is flipped along cos(theta) dimension

Model

Data

The hypothesis is that the theta is defined as pi-theta in the analysis code.
It should be possible to flip back by redefining the couplings

\begin{align}
A_{\lambda_0,\lambda_1, \lambda_3}^{j;LS;ls}(m_{12},\cos\theta) =
%
H_{\lambda_0+\lambda_3,\lambda_3}^{LS}\,
d_{\lambda_0+\lambda_3,\lambda_1}^{j}(\theta) \,
H_{\lambda_1}^{ls}
\end{align}

Using properties of Wigner D and Clebsches one finds,
\begin{align}
A_{\lambda_0,\lambda_1, \lambda_3}^{j;LS;ls}(m_{12},-\cos\theta) =
%
(-1)^{l+1/2+\lambda_0+\lambda_3}
A_{\lambda_0, -\lambda_1, \lambda_3}^{j;LS;ls}(m_{12},\cos\theta)
\end{align}

The phase factors with helicity are common for all chains, but (-1)^l has to be applied for chain couplings (weights).

@IlyaSegal I've figured out the transformation, seems as simple as (-1)^l to flip the cos(theta)


(repr "A") is incorrect

Describe the bug
I think the output of (repr "A") is incorrect.
To Reproduce
Run
(repr "A")

Expected behavior
["\"A\""]

Actual behavior
[""A""]

Additional context

MeTTa version: 0.2.1+g8795f59b.d20241129
(repr 'A') or (repr A) is correct, only (repr "A") isn't.

Looks like the issue with string atom formatting in Rust. For example !(foo "\"A\"") (which is not reduced) also is printed as (foo ""A"").

@ngeiswei does it break something or you just reported it to fix at some point?

It doesn't break my code as long as I use MeTTaLog, which behaves correctly.  It's just that I like to be able to switch back and forth between MeTTaLog and HE.

I think a deeper problem is that MeTTa should show the string representation instead of printing it.
For instance
! "A\nB"

should output
["A\nB"]

instead of
["A
B"]

However
!(println! "A\nB")

should output
A
B


allow yaml files of same name in different dirs or file extensions

allow yaml files of same name in different dirs or file extensions

Remove assumption that thing.yaml and thing.yml don't both exist.

Allow same file name in multiple paths, such as

hieradata/idunno/thing.yaml
hieradata/common/thing.yaml

New unit tests:
RSpecPuppetUtils::HieraData::YamlValidator
  #load_data
    with valid yaml
      should load yaml data from files with same name but different dir
      should load yaml data from files with same name but different ext

To accomplish this, we store the subpath (as opposed to just the base name)
in yaml validator data and adjust existing unit tests for the subpath.
The payoff is that we can now validate all files that may
exist in somebody's hieradata yaml tree.

@TomPoulton bump?

Hi @jumanjiman, we're having some issues with the WiFi at work, if it doesn't get fixed today I'll try and find some time tonight to test and merge

thanks @TomPoulton ! this will be a big help for us.

Merged and published. Thanks for this, I remember writing the initial version thinking "I'll come back to this later"!

woot!


Handle empty result from getBalancesInSingleCall

This PR handles empty array result from getBalancesInSingleCall in AssetsContractController. It also fixes the type.

Codecov Report

Merging #120 into develop will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           develop   #120   +/-   ##
======================================
  Coverage      100%   100%
======================================
  Files           21     21
  Lines         1430   1431    +1
  Branches       193    194    +1
======================================
+ Hits          1430   1431    +1

Impacted Files
Coverage Δ

src/assets/AssetsContractController.ts
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Communications Error

@Glitchbone
Hi,
When I try to play the videos without any issue playing in IOS, but not in the android (all android versions). Some videos  comes with this Error message instead of playing them.  I don't have any idea  why does this occur?

Above image shows the error message appear in the simulator or phone.Below image shows the error message in the Android Studio log

Need help ASAP....

:+1:

i have the same issue.
@Aathi did you find any solution ?

@flieks Sorry! Still I have this issue

same issue here, I needed for both IOS and android . but looks like there is no solutions

i have same issue on android( readmi note 4)

@Aathi Did anyone find a solution? Still having this issue...

@FredeHo I couldn't fix the issue at that time so I migrated to https://facebook.github.io/react-native/


Expo managed app?

Hello there,
I have question. Can we use this library with expo managed app?
becuase what I know is that if the package contain native SDK cannot be integrated with expo managed app. And I have project that I'm planning for; will depend on this library so much.
Cheers!

@b0iq Our SDK is supported for integration in Expo-managed apps. Please follow the steps mentioned here:
https://www.100ms.live/docs/react-native/v2/features/integration#expo-setup


policy sets: adding condition to prevent ssa on image-inspector when a new image is detected

Fixes #9302
@moolitayer @simon3z
/cc @cben - Thanks for your help. Can you also review this?

I think it should be possible to run the SmartState on image-inspector if explicitly requested. What we have to block is to scan image-inspector when a new Pod is detected (there's an event for that).

@simon3z I am sorry was not clear. I applied this condition only on the "Analyse incoming container images" policy. This means that it is still possible to initiate ssa manually on image-inspector images.

@miq-bot add_label darga/yes

@miq-bot assign enoodle

Confirmed that this works.  Turns out a bit tricky to reproduce as you only see "image-inspector" if you refresh while a scan is happening.
[----] I, [2016-07-04T16:05:52.131078 #4260:2aee43521988]  INFO -- : MIQ(policy-enforce_policy): Event: [containerimage_created], To: [openshift/origin-sti-builder]
[----] I, [2016-07-04T16:05:52.176932 #4260:2aee43521988]  INFO -- : MIQ(policy-enforce_policy): Resolving policy [Analyse incoming container images]...
[----] I, [2016-07-04T16:05:52.189721 #4260:2aee43521988]  INFO -- : MIQ(condition-eval): Name: Dont scan image-inspectors image, Expression evaluation result: [false]
[----] I, [2016-07-04T16:05:52.216116 #4265:2aee43521988]  INFO -- : MIQ(action-invoke) Invoking action [Initiate SmartState Analysis for Container Image] for successful policy [Analyse incoming container images], event: [Container Image Discovered], entity name: [proj/foobar], entity type: [»Image«], sequence: [1], synchronous? [true]
[----] I, [2016-07-04T16:05:52.216235 #4265:2aee43521988]  INFO -- : MIQ(action_container_image_analyze): Now executing [Initiate SmartState Analysis for Container Image] of Container Image [proj/foobar]

vs
[----] I, [2016-07-04T16:08:56.389427 #4250:2aee43521988]  INFO -- : MIQ(policy-enforce_policy): Event: [containerimage_created], To: [openshift/image-inspector]
[----] I, [2016-07-04T16:08:56.451236 #4250:2aee43521988]  INFO -- : MIQ(policy-enforce_policy): Resolving policy [Analyse incoming container images]...
[----] I, [2016-07-04T16:08:56.460998 #4250:2aee43521988]  INFO -- : MIQ(condition-eval): Name: Dont scan image-inspectors image, Expression evaluation result: [true]

@simon3z why is author == assignee (reviewer)?

@simon3z why is author == assignee (reviewer)?

The answer can be found here.

@moolitayer can you review? Also I don't like the fact that we base this on checking the name (vs labels, etc.). Any better idea?
@enoodle open a BZ for this (so that we can backport it to darga) and please revise the PR.

@simon3z I'm good with this change, just had one naming comment on the condition name.
I also tested and this works as expected. I don't have any ideas better then testing the image name.

I don't have any ideas better then testing the image name.

:sob:

LGTM :+1:
@miq-bot assign chessbyte
@miq-bot add_label blocker

Darga backport details:
commit 4c16d6f09e11c6e6e3d08a1780297a4c9e13d4a2
Author: Oleg Barenboim<EMAIL_ADDRESS>Date:   Thu Aug 4 10:40:19 2016 -0400

    Merge pull request #9484 from enoodle/avoid_ssa_on_image_inspector_image

    policy sets: adding condition to prevent ssa on image-inspector when a new image is detected
    (cherry picked from commit 13022265f1c2326a89429de0858e22089ed20cd3)


rederict callback window not closing on heroku deploy

When I log in with spotify everything works peachy on localhost. However for my heroku deploy I will log in to spotify and the callback.html will appear with an access token in the url string, but the box does not close itself. LocalStorage is updated on the callback.html but not for the main site, index.html

The issue is that in production the callback.html box does not go away by itself after contacting spotify successfully. It works when running on localhost though..

Hmm. I haven't run into that issue before.
In the callback page once it updates the local storage then you could add a window.close().

Now it works. This went away after a few minutes with the heroku deploy. Either that or I was using $cookies instead of localStorage


Adding parameter to specify more master SANs

What this PR does / why we need it: Adding a new parameter to define more SAN's using which API servers can be reached using kubectl.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #2614
Special notes for your reviewer:
If applicable:

[x] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: aku105 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@jackfrancis Updated the variable names and docs to improve readability, as suggested.

@CecileRobertMichon @slack @amanohar  would appreciate a high level review of the intent here and implementation, because the MasterProfile type struct is general purpose. I'm in favor of this addition.

@aku105 Let's add a unit test that proves out the following usage patterns:

nil MasterProfile.SubjectAltNames input produces certs w/ the dnsPrefix only
populated MasterProfile.SubjectAltNames produces certs w/ the dnsPrefix string + the add'l SANs in the input

Hi @jackfrancis I've added an unit test.
Could you please confirm if this is what you were expecting in the unit test?


Update readme to make instructions clearer

Change(s):

Changes to formatting and wording in readme.md

Reason for Change(s):

These changes will make the playbook instructions more clear for users and remove some formatting mistakes.

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

N/A

Hi @manishkumar1991, please provide your comments. Thanks.


Xcode 8 error log

When saving on the new Xcode 8 I'm getting this error log:
[error] error: An observer of NSManagedObjectContextDidSaveNotification illegally threw an exception.  Objects saved = {     inserted = "{(\n)}";     managedObjectContext = "<_PFWeakReference: 0x600000428220>";     updated = "{(\n)}"; } and exception = [<_PFWeakReference 0x600000428220> valueForUndefinedKey:]: this class is not key value coding-compliant for the key @count. with userInfo = {     NSTargetObjectUserInfoKey = "<_PFWeakReference: 0x600000428220>";     NSUnknownUserInfoKey = "@count"; }
Any idea if it has to do with this class: RKManagedObjectStore.m with the function:
static NSSet *RKSetOfManagedObjectIDsFromManagedObjectContextDidSaveNotification(NSNotification *notification) {     NSUInteger count = [[[notification.userInfo allValues] valueForKeyPath:@"@sum.@count"] unsignedIntegerValue];     NSMutableSet *objectIDs = [NSMutableSet setWithCapacity:count];     for (NSSet *objects in [notification.userInfo allValues]) {         [objectIDs unionSet:[objects valueForKey:@"objectID"]];     }     return objectIDs; }
Thanks

I am also getting this issue with the iOS 10 beta.

This is fixed by https://github.com/RestKit/RestKit/pull/2449

Hello, I am using RestKit 0.27 , I'm facing above issue. I have 1200 entry few entry saved into database but after few entry I get above error. Let us know which way i can resolve this.

Same here. Using RestKit 0.27 and facing above issue.

Hey, I also ran into the problem. For me, pinning RestKit to a specific commit helped:
# Fetching a specific dev-version of RestKit due
# to an iOS10 introduced problem in RestKit
# For further information, see
# https://github.com/RestKit/RestKit/issues/2451
pod 'RestKit',
  :git => 'https://github.com/RestKit/RestKit.git',
  :commit => '9cbba9eb1b490c3c5e2873c8fba8e9c0fec1bf07'

thx Jan, that helped, but when can we get this into a release?

Unfortunately, I can't answer that, since I'm not a member of the RestKit team or an active contributor. It's afaik a project they do in their spare time, so I guess there'll be a release when they find the time for it.

This issue is closed but this commit is still not in any official build. This seems like a pretty significant thing and is reproducible in iOS 10. When can we expect a 0.28?

@jasconius I tried to make a release https://github.com/RestKit/RestKit/pull/2466 but travis doesn't agree with me and unfortunately I have no time to fix it.
Anybody's help in making that PR green would be really appreciated.


Abortion probability - Ethiopia

Senegal parameter is based on the percentage of all pregnancies ending in abortion

https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4712915/

Probability for Ethiopia found here: https://doi.org/10.1016/j.amsu.2022.104120
Abortion rate is more commonly published than probability: https://osf.io/6t4eh
Have not come across enough data to produce an age-specific gradient, but there was a recent article on abortion among youth 15–24 in Ethiopia: https://journals.plos.org/plosone/article?id=10.1371/journal.pone.0248228

The Yemane article looks like it is estimating the % of women who've terminated a pregnancy. Am I getting that right? If so, what we really need is % of pregnancies that end in abortion (since we're assessing the probability immediately after conception).

Yes, thank you for noting that. The most recent figure I found was from 2014: https://www.ncbi.nlm.nih.gov/pmc/articles/PMC5568682/
"The abortion ratio in 2014 was 17.6 abortions per 100 live births, indicating that there was approximately one abortion for every six pregnancies that ended in a live birth."

[ ]  add to ethiopia.py script

Issue Closed: PR #156


Fix unbind_external_function

Checklist for this pull request

[x] I have read the guidelines for contributing.
[x] I have checked that my code additions did not fail tests.

Description
External function bindings are stored in a Dictionary which requires "erase" to delete an entry by key. Currently, "remove" is used which is used in Arrays instead.

Nice catch, thanks for the PR!


[mypy] Enforcing typing for some modules

CATEGORY
Choose one

[ ] Bug Fix
[x] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
Adding mypy enforcement for a number of small modules (starting alphabetically):

superset.bin
superset.commands
superset.common
superset.dao
superset.db_engines

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
to: @dpgaspar @villebro

Codecov Report

Merging #9416 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9416   +/-   ##
=======================================
  Coverage   59.01%   59.01%
=======================================
  Files         378      378
  Lines       12162    12162
  Branches     3004     3004
=======================================
  Hits         7177     7177
  Misses       4805     4805
  Partials      180      180

Continue to review full report at Codecov.

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

@dpgaspar are you ok with these changes? Note I added a couple more modules; superset.dao and superset.db_engines, since I authored the PR.

ping @dpgaspar @villebro


Flip111 patch 1

Follow-up on #2232

Coverage decreased (-2.5%) to 91.575% when pulling d86cb4a2e9f284ffd39b4082584f36a7c927e429 on flip111-patch-1 into 8df0dc9002529744fda1fa6306384aaa21935ae7 on master.

Codecov Report

Merging #2234 into master will decrease coverage by 1.82%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2234      +/-   ##
============================================
- Coverage     94.88%   93.05%   -1.83%
+ Complexity     1584     1549      -35
============================================
  Files            50       50
  Lines          3849     3829      -20
============================================
- Hits           3652     3563      -89
- Misses          197      266      +69

Impacted Files
Coverage Δ
Complexity Δ

lib/User.php
93.02% <ø> (ø)
38.00 <0.00> (ø)

lib/Site.php
49.42% <0.00%> (-37.94%)
23.00% <0.00%> (ø%)

lib/Cache/Cleaner.php
66.66% <0.00%> (-28.99%)
6.00% <0.00%> (ø%)

lib/Image/Operation/Resize.php
87.96% <0.00%> (-10.21%)
33.00% <0.00%> (ø%)

lib/Timber.php
88.78% <0.00%> (-7.48%)
34.00% <0.00%> (ø%)

lib/Twig.php
96.72% <0.00%> (-2.78%)
55.00% <0.00%> (-32.00%)

lib/Core.php
84.61% <0.00%> (-2.57%)
24.00% <0.00%> (ø%)

lib/Loader.php
95.62% <0.00%> (-0.63%)
67.00% <0.00%> (ø%)

lib/Post.php
95.17% <0.00%> (-0.59%)
243.00% <0.00%> (-3.00%)

lib/ImageHelper.php
95.69% <0.00%> (-0.36%)
89.00% <0.00%> (ø%)

... and 2 more

Continue to review full report at Codecov.

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

@flip111 thanks so much for your first PR!!! I added some tests on #2234 (and dealt with some random Travis/Composer bugs). I was able to simplify things a bit by moving the empty string to the top in the property declarations.
Thanks as well for the code snippet. Using that I was able to replicate the issue. The way that I was creating fake posts/users before (using WordPress's testing framework) somehow suppressed that error.


S3C-3571 create Gradle project,CI setup,Add Jacoco,Pmd,Spotbugs

Hello saitharunthodupunuru,
My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.
Status report is not available.

Waiting for approval
The following approvals are needed before I can proceed with the merge:

the author

one peer

Peer approvals must include a mandatory approval from @saitharunthodupunuru.

Is it possible to have the coverage percentage reported in the CI logs stdio?

Is it possible to have the coverage report uploaded in our homemade Artifactory(I need to find the right name for that)? e.g: docs-ring-s3c CI → https://github.com/scality/docs-ring-s3c/blob/841e0cf57446f0fa4f8d5c99441eb087e9c8735f/eve/main.yml#L7

Is it possible to have the coverage report uploaded in our homemade Artifactory(I need to find the right name for that)? e.g: docs-ring-s3c CI → https://github.com/scality/docs-ring-s3c/blob/841e0cf57446f0fa4f8d5c99441eb087e9c8735f/eve/main.yml#L7

Added a new commit to add the report files. I will see if I can simplify more to understand which of these files are code coverage and which are lint reports

Is it possible to have the coverage report uploaded in our homemade Artifactory(I need to find the right name for that)? e.g: docs-ring-s3c CI → https://github.com/scality/docs-ring-s3c/blob/841e0cf57446f0fa4f8d5c99441eb087e9c8735f/eve/main.yml#L7

Added a new commit to add the report files. I will see if I can simplify more to understand which of these files are code coverage and which are lint reports

Would've been good if this were split in multiple commits: one to set up the Gradle project, one to set up PMD, one to integrate with Eve,... (or those in some different order).
Also, is APL2 the right license for this project? How was this decided?

Nicolas,
We decided on making this project as open source. I just used APL2 license as many open source projects scality had same license. Please suggest otherwise

Would've been good if this were split in multiple commits: one to set up the Gradle project, one to set up PMD, one to integrate with Eve,... (or those in some different order).
Also, is APL2 the right license for this project? How was this decided?

Nicolas,
We decided on making this project as open source. I just used APL2 license as many open source projects scality had same license. Please suggest otherwise

Is it possible to have the coverage report uploaded in our homemade Artifactory(I need to find the right name for that)? e.g: docs-ring-s3c CI → https://github.com/scality/docs-ring-s3c/blob/841e0cf57446f0fa4f8d5c99441eb087e9c8735f/eve/main.yml#L7

I added a commit for uploading the coverage reports and lint analysis reports and printing same on logs.

Is it possible to have the coverage report uploaded in our homemade Artifactory(I need to find the right name for that)? e.g: docs-ring-s3c CI → https://github.com/scality/docs-ring-s3c/blob/841e0cf57446f0fa4f8d5c99441eb087e9c8735f/eve/main.yml#L7

I added a commit for uploading the coverage reports and lint analysis reports and printing same on logs.

/approve

/approve

In the queue
The changeset has received all authorizations and has been added to the
relevant queue(s). The queue(s) will be merged in the target development
branch(es) as soon as builds have passed.
The changeset will be merged in:

:heavy_check_mark: development/1.0

There is no action required on your side. You will be notified here once
the changeset has been merged. In the unlikely event that the changeset
fails permanently on the queue, a member of the admin team will
contact you to help resolve the matter.
IMPORTANT
Please do not attempt to modify this pull request.

Any commit you add on the source branch will trigger a new cycle after the
current queue is merged.
Any commit you add on one of the integration branches will be lost.

If you need this pull request to be removed from the queue, please contact a
member of the admin team now.
The following options are set: approve

In the queue
The changeset has received all authorizations and has been added to the
relevant queue(s). The queue(s) will be merged in the target development
branch(es) as soon as builds have passed.
The changeset will be merged in:

:heavy_check_mark: development/1.0

There is no action required on your side. You will be notified here once
the changeset has been merged. In the unlikely event that the changeset
fails permanently on the queue, a member of the admin team will
contact you to help resolve the matter.
IMPORTANT
Please do not attempt to modify this pull request.

Any commit you add on the source branch will trigger a new cycle after the
current queue is merged.
Any commit you add on one of the integration branches will be lost.

If you need this pull request to be removed from the queue, please contact a
member of the admin team now.
The following options are set: approve

I have successfully merged the changeset of this pull request
into targetted development branches:

:heavy_check_mark: development/1.0

Please check the status of the associated issue S3C-3571.
Goodbye saitharunthodupunuru.

I have successfully merged the changeset of this pull request
into targetted development branches:

:heavy_check_mark: development/1.0

Please check the status of the associated issue S3C-3571.
Goodbye saitharunthodupunuru.


[UI]: RTK Migration for User Preference

Notes for Reviewers
This PR fixes #9985

[x] GET: Migrate user pref to RTK
[x] POST: Add RTK query for api/user/pref
[x] POST:  Add RTK query for api/user/pref with k8sContext

Screen Recorder
https://github.com/user-attachments/assets/d3cb4dc1-24c7-4cb2-b805-2159333edab1
Signed commits

[x] Yes, I signed my commits.

@senthil-athiban let's discuss this PR during the Meshery dev meeting call, if you would. Please add it as an agenda item to the meeting minutes.

Ready for another round of review.