MAINT: Do not use cgi-traceback on 3.11+ (deprecated, marked for removal)

cgitb that is used as a fallback to convert traceback to html has been deprecated as part of Python 3.11 and is marked for removal in python 3.13.
As we turn warning into errors in Python 3.11+ this will make the test fail.
This add a conditional to simply wrap the traceback into pre tags when IPython is not available and we are on Python 3.11+.
I would be tempted to already completely remove cgi-tb code for us to not have a difference in behavior before/after 3.11.
This is extracted from #5439 in order to make review/discussion easier.
Type of change

[ ] Bug-fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Maybe if branches should be moved into separated functions for better readability, but I'm ok with this.

It wont help much the else is literally just:
else:
   def format_exc_info(
        info: ExcInfo, as_html: bool, color=None
    ) -> str:
        # avoids printing the array data
        np.set_string_function(
            lambda arr: f'{type(arr)} {arr.shape} {arr.dtype}'
        )
        if as_html:
            tb_text = '<pre>' + tb_text + '</pre>'
        np.set_string_function(None)
        return tb_text

It's just git diff get super confused.

Thanks !


Reading and writing objects

Is there a way to write objects, rather than strings?
I am trying to use this stream as a writable-stream in Bunyan:
https://github.com/trentm/node-bunyan#streams
Bunyan writes objects, and I would like to read them as objects. Is this possible?

No, sorry, objects won't work natively with this module, they're not really in the spirit of streams and buffers. I have no plans to make this module support objects; however, feel free to fork and I'll happily consider any pull request.


cannot use PyO3 and tensorflow at the same time.

Bug Description
When importing a module when using PyO3 and tensorflow at the same time, it hangs.
(I couldn't figure out whether the problem was with tensorflow or PyO3)
Steps to Reproduce
use pyo3::prelude::*;
use tensorflow::Tensor;

fn main() {
    let gil = Python::acquire_gil();
    let py = gil.python();

    let sys = py.import("sys").unwrap();
    let _ = py.import("google.cloud.storage").unwrap();
    let version :String = sys.getattr("version").unwrap().extract().unwrap();
    println!("Hello python {}", version );

    let tensor = Tensor::new(&[1,2]).with_values(&[1.0,2.0]).unwrap();
    println!("{:?}", tensor );
}

Running this code causes a segmentation fault when importing google.cloud.storage.
However, if I comment out the last two lines, this code ends normally, even though it is not related to the processing order.
Backtrace
(gdb) bt
#0  0x00007f9f2fde0614 in google::protobuf::python::AddEnumValues(_typeobject*, google::protobuf::EnumDescriptor const*) [clone .isra.0] [clone .constprop.0] ()
   from /opt/conda/lib/python3.7/site-packages/google/protobuf/pyext/_message.cpython-37m-x86_64-linux-gnu.so
#1  0x00007f9f2fde0f02 in google::protobuf::python::InitDescriptor() () from /opt/conda/lib/python3.7/site-packages/google/protobuf/pyext/_message.cpython-37m-x86_64-linux-gnu.so
#2  0x00007f9f2fdd995e in google::protobuf::python::InitProto2MessageModule(_object*) () from /opt/conda/lib/python3.7/site-packages/google/protobuf/pyext/_message.cpython-37m-x86_64-linux-gnu.so
#3  0x00007f9f2fdeb975 in PyInit__message () from /opt/conda/lib/python3.7/site-packages/google/protobuf/pyext/_message.cpython-37m-x86_64-linux-gnu.so
#4  0x00007f9f31d34a93 in _PyImport_LoadDynamicModuleWithSpec () from /lib/x86_64-linux-gnu/libpython3.7m.so.1.0
#5  0x00007f9f31d3500b in ?? () from /lib/x86_64-linux-gnu/libpython3.7m.so.1.0
#6  0x00007f9f31e23f37 in _PyMethodDef_RawFastCallDict () from /lib/x86_64-linux-gnu/libpython3.7m.so.1.0
#7  0x00007f9f31e24a45 in _PyCFunction_FastCallDict () from /lib/x86_64-linux-gnu/libpython3.7m.so.1.0
#8  0x00007f9f31c25700 in _PyEval_EvalFrameDefault () from /lib/x86_64-linux-gnu/libpython3.7m.so.1.0
#9  0x00007f9f31d4d1e6 in _PyEval_EvalCodeWithName () from /lib/x86_64-linux-gnu/libpython3.7m.so.1.0

Your operating system and version
debian 10.11
Your Python version (python --version)
Python 3.7.12
Your Rust version (rustc --version)
rustc 1.56.1 (59eed8a2a 2021-11-01)
Your PyO3 version
0.15.1
How did you install python? Did you use a virtualenv?
use preinstalled image
Additional Info
I created instance on GCP by this command
gcloud compute instances create dl-test-cpu-preemptible
  --image-project="deeplearning-platform-release"
  --image-family="tf2-ent-latest-cpu"
  --maintenance-policy="TERMINATE"
  --preemptible
  --scopes storage-rw

It happens in docker as well.
Source code is here:
https://github.com/tsubakisakura/pyo3tensorflow

What's the google cloud library version?
My wild guess would be that both "google cloud storage" library links to a protobuf library and Tensor with different versions.
Can you try to install protobuf without binaries? i.e, let pypi compile it from source?

As this has not received a follow-up response from OP for three months, I'm going to close this one. @aviramha's theory sounds likely correct.


Setting working directory to ${fileDirname} when opening terminal

Hi,
I might be missing something, but is there a way to set the working directory to the currently opened file when launching R terminal in VSC?
I could not find any workaround in the current setup.
Many thanks!

Hi, as far as I know there is no feature for this in this extension. But a couple of months ago VSCode added a command to open a terminal in the location of the current file: https://code.visualstudio.com/updates/v1_39#_open-new-terminals-with-custom-working-directories
It would probably be possible to create a macro that does this then writes ‘R’ to the terminal to open an R session there.
Someone else might know of a more straightforward way to do this.

Thank you for the hint on macros. I realized that if you start the terminal while having a folder opened, it starts with the working directory set to that folder. So this provides desired functionality, although it requires specific procedure - finding a file in the folder opened in VSC and opening from there rather than directly from the file browser. Applies to Win at least.

I think that’s VSCode’s assumed workflow - that you’ll open a project using ‘Open Folder...’ and then work in that folder.
Sounds like you found what you were after, so that’s good to hear!

@janxmares It sounds like this issue is resolved so I'm going to close it, but if you don't consider it resolved please post and I can look into it further.

Just in case someone else is also stumbling over this question in the distant future: R also provides a method for setting the current working directory from within the R terminal: setwd([dir]) and getwd().
While this doesn't switch the cwd automatically it might still be helpful.
Ref: https://stat.ethz.ch/R-manual/R-devel/library/base/html/getwd.html
Please feel free to delete this comment again if I missunderstood the question or if posting long after the fact is considered inappropriate.

@janxmares Did you ever find a good fix for this? I am experiencing the same issue. Many thanks.

@Fionn-McGlacken Not really, I adopted the workflow mentioned above and I use it ever since.

In case others are looking for an easy solution:
# Install the rprojroot package if not already installed
if (!requireNamespace("rprojroot", quietly = TRUE)) {
  install.packages("rprojroot")
}

# Load the rprojroot package
library(rprojroot)

# Set the working directory
setwd(here::here())

# Check if working directory is set correctly
getwd()

Is there an easy way to set the current directory of the R terminal in VScode to the directory of the file I am currently working on? I know I could use setwd() but changing the path syntax between windows and R is a bit annoying. Is there any easy way to do this? I have cwd set to ${fileDirname} in VScode but it seems like R terminal does not follow those settings. Rstudio has a nice feature when right clicking the title of an open file, there is an option to set the current directory to the path of the opened file.

Thank you for the hint on macros. I realized that if you start the terminal while having a folder opened, it starts with the working directory set to that folder. So this provides desired functionality, although it requires specific procedure - finding a file in the folder opened in VSC and opening from there rather than directly from the file browser. Applies to Win at least.

@janxmares Was there any settings you used to have the R terminal open in the directory of the opened folder? In my vscode, when click to open a new R terminal, radian opens in the project directory even if I have a folder or file  opened.

Not pretty, but a decent workaround is to run setwd(dirname(.vsc$rstudioapi_env$getSourceEditorContext()$path)) at the beginning of your script.


Config for environment vars

It would be better to have envs in a config file, like:
/my-app/dev -> db-url1
/my-app/prod -> db-url2

We have decided to ditch this idea completely and just focus on one database (Postgres).


EIP-681 QR code BSC fails to properly read

Scanning a QR code in the EIP-681 format with the @chain_id included fail to properly read for BSC (BNB and BEP20's), and MATIC. It works for ETH and ERC20's though. Is EIP-681 only supported for Ethereum? If so what would be the correct URI format for a QR code for BSC BNB?

Yeah, currently we only enable eip681 for Ethereum, for bnb chain, please use bip21 and refer to "Send Payment" example on https://developer.trustwallet.com/deeplinking#payment

Odd... Wouldn't it make more sense to support EIP-681 for QR codes for EVM compatible chains (BSC, MATIC, etc.). Rather than BIP-0021 which isn't EVM related at all.... Deeplinking is great for specific payment buttons, but in terms of QR codes, other wallets are supporting EIP-681 for EVM chains so it kind of puts us in between a rock and a hard place, if we support them for other chains the QR code is broken for trust wallet, or we use BIP-0021 it works for trust wallet but not other wallets....

@jjs2484 good point, let me confirm app team, this one is tricky, ethereum: will be treated as EIP681, for other chains there is no well know protocol name like ethereum, or we can handle chain id instead

@ArtemGoryunov can you check if this makes sense? enable eip681 for other evm chains

@jjs2484 Thank you for raising this question. Fixes will be available by end of the week for both planform.


Swift3 syntax compatible with Xcode 8 beta 5

Could you update the .travis.yml file to use osx_image: xcode8 which should make it build there again.

Some minors but otherwise looks good 👍

Well I think cocoapod lint doesn't work with Xcode 8 yet... :(

why am I not surprised… I merged this anyway now, will take a look. Thanks for your help!


Placeholders for the original PR author, and user that made the label or comment

I would like to automatically subscribe to the backport PR by @-mentioning them:

the user who attached the label (or wrote the /backport comment) that triggered the backport PR
the author of the original, non-backport PR

The new pull_description functionality from PR #64 by @korthout is perfect for that, but we would need placeholder variables for those two users.

@nh2 Sounds like a great addition! I'm almost done finishing up a pull request for #29 by adding a new placeholder ${issue_refs}. And I
For the author of the original pull, I think something like ${pull_author} would be fitting.
For the other type of user, I'm not sure. Perhaps ${sender} (matching the GitHub naming) or ${triggered_by} (feels a bit more in the action's domain). Do you have any suggestions or preferences?

For the author of the original pull, I think something like ${pull_author} would be fitting.

Yes sounds good, or ${original_pull_author} if we want to be super specific.

Perhaps ${sender} (matching the GitHub naming) or ${triggered_by}

How about ${backport_trigger_user}?
Also I think the docs of such feature should then explain whether the placeholder already contains the @, or not -- for example, would "triggered by @${backport_trigger_user} @`-mention them?

@nh2 Thanks for the suggestions! I'll have to think a bit about what I like best for naming.
I totally agree with the @ character. Leaving it out is more flexible. I'll make sure the doc is clear on what the placeholder is replaced with.

Hi @nh2. I've added the pull_author placeholder. You can prefix it with an @ character like this: @${pull_author} to mention the user and subscribe them to notifications for the newly created backport pull request.
Note that I have not yet released a new version officially, but you can just refer to latest commit on main: 4c106a22b132961ec31097db8035bb849a401471.
Let me know if you run into any issues.


[Urgent]Problem when upgraded new version.

I have already upgrade new version, and i got a lot of issues. Angular2-select in new version is very smooth, i loved it. But i got 2 problems.

Selected value of new version is array - difference type of old version - so i have to fix all source code. So i got a lot of unexpected problem.
Second, this is very important. Angular2-select in new version can not using HTML5 validate (required).

Thank you.

Thanks for the feedback. A few comments:

To make it easier to manage the issues, report each issue separately with a specific title indicating what the issue is.

Please check if the issue you are reporting is not already reported and/or fixed. If you search all issues for 'array' you will see that your first issue is the same as issue #66, which is already fixed and released in beta.2.

Please note that adding 'Urgent' to the title does not give an issue higher priority or will get it fixed any sooner.

If it is important for you to have stable software, don't use pre-release software. And don't upgrade to a just released beta version for which the documentation clearly states:

Disclaimer: The beta version is a complete rewrite of the alpha version, so new bugs are to be expected. Please do not yet rely on this beta version for production releases.

About your second issue. You can use the select component as form control with the required validator which will keep the form invalid as long as no option is selected.

First issue was already fixed.
For second issue see #83.


Added onMouseLeave event to blur focus

https://github.com/thenewboston-developers/Account-Manager/issues/338 issue

The issue was incorrectly labeled as a 'bug'. Closed because it needs to go through design first


FEAT(SideNav):  add ability to change default expanded state

Closes IBM/carbon-components-react#2145
Added the ability via prop to declare the default expanded state of the sidenav component
Changelog
Changed

Changed SideNav.js
Changed SideNav-test.js
Updated SideNav-test.js.snap

^ Another way to deal with this is creating a stateless version of <SideNav> and make <SideNav> use such stateless version. In this way you can use the stateless version. This is something @emyarod is pushing for.

Thanks @davidicus for the update! Can you rename defaultExpanded prop to isExpanded, given the code has support for subsequent updates?


Invalid write in ucg_builtin_init

Describe the bug
when ucg invokes ucp_init_version(), ucp will allocate ucs_offsetod(ucg_context_t, ucp_ctx) + sizeof(ucp_context_t) bytes space. But in ucg_context_init(), it uses the space after ucs_offsetod(ucg_context_t, ucp_ctx) + sizeof(ucp_context_t) bytes which causes invalid write.
static ucs_status_t ucg_context_init(void *groups_ctx)
{
    ...
   ucg_plan_ctx_h plan_ctx = ctx + 1;
  ...
}

branch is unstable, those bugs is being fixed


(PUP-6259) Fix no_proxy support for File resource

File resources now have the ability to pull from an http/https source. However, the no_proxy variable doesn't seem to be respected. I traced this to behavior related to the Net::HTTP::Proxy object that seems to have been deprecated. Replacing it with Net::HTTP.new seems to work

CLA signed by all contributors.

Well, this was more complicated than I had anticipated 😄  -- the tests fail in repository_spec.rb.
It seems like some module is parsing the port as [port]the_path, which is weird. As far as I can tell, the URI that's being sent is http://fake.comthepath instead of http://fake.com/the_path. I tried changing the paths around, but then VCR complained that it didn't know how to replay the request.
I'm sure someone with more in-depth rspec/codebase knowledge can clue me in on what's going on here. It's also possible that there may need to be two behaviors: the old method for rubies <2.0 and this for rubies >2.0

We still (unfortunately :( ) support Ruby 1.9 in Puppet 4, so if the new code doesn't work there we can't take it (unless there's a compatibility path).
As far as the VCR stuff, that was all done by @ffrank. Felix, if you have the time, do you mind helping out here?

Sure, sounds fun :)
Before we dig into this - @patcable could you update the ticket with information? The file resource should be using the same HTTP client code as the PMT (i.e. puppet module install). Is that afflicted as well?

@ffrank sure thing. I copied that comment over there.
Yes, it would seem that puppet module install breaks with my fix (as evidenced by the tests). So there's something odd about how no_proxy is handled for that code vs the File resource.

This PR has not been updated in some time and is now being closed due to inactivity. Please re-open if this is in error - apologies in advance for any mistake.

Is ruby 1.9 still supported? If not, perhaps this could be merged now?


test for height problem during ace.js refactoring

https://github.com/ether/etherpad-lite/issues/4914 is the main issue
I don't know if there is a better way to detect if something is visible to the user.
This compares the height of the inner iframe which should grow when a lot of lines are added. It's possible that the bug can be triggered without large objects like images or big font just be adding a lot of lines. In this case it would probably make more sense to put the test into core. (However, the original bug report was for large images, so I thought it's still useful here.)
If you think this better should go to core, I can test if the bug is present with a lot of lines.

Plz fix lint errors.  Tnx


Bundle exec foreman starts terminated by SIGKILL

I've managed to setup Huginn up until executing foreman which fails with the following output. Please help I'm doing this on Windows 7 x64. Ran bundle a couple of times, cannot figure out what is going wrong.
C:\Users\User\huginn>bundle exec foreman start
Your Gemfile lists the gem pg (>= 0) more than once.
You should probably keep only one of them.
While it's not a problem now, it could cause errors if you change the version of
just one of them later.
15:27:05 web.1  | started with pid 8976
15:27:05 jobs.1 | started with pid 9096
15:27:52 jobs.1 | C:/Users/User/huginn/lib/agent_runner.rb:52:in trap': unsuppo rted signal SIGQUIT (ArgumentError) 15:27:52 jobs.1 |       from C:/Users/User/huginn/lib/agent_runner.rb:52:in blo
ck in set_traps'
15:27:52 jobs.1 |       from C:/Users/User/huginn/lib/agent_runner.rb:51:in eac h' 15:27:52 jobs.1 |       from C:/Users/User/huginn/lib/agent_runner.rb:51:in set
_traps'
15:27:52 jobs.1 |       from C:/Users/User/huginn/lib/agent_runner.rb:27:in ini tialize' 15:27:52 jobs.1 |       from bin/threaded.rb:5:in new'
15:27:52 jobs.1 |       from bin/threaded.rb:5:in <top (required)>' 15:27:52 jobs.1 |       from C:/Ruby21/lib/ruby/gems/2.1.0/gems/railties-4.2.4/l ib/rails/commands/runner.rb:60:in load'
15:27:52 jobs.1 |       from C:/Ruby21/lib/ruby/gems/2.1.0/gems/railties-4.2.4/l
ib/rails/commands/runner.rb:60:in <top (required)>' 15:27:52 jobs.1 |       from C:/Ruby21/lib/ruby/gems/2.1.0/gems/railties-4.2.4/l ib/rails/commands/commands_tasks.rb:123:in require'
15:27:52 jobs.1 |       from C:/Ruby21/lib/ruby/gems/2.1.0/gems/railties-4.2.4/l
ib/rails/commands/commands_tasks.rb:123:in require_command!' 15:27:52 jobs.1 |       from C:/Ruby21/lib/ruby/gems/2.1.0/gems/railties-4.2.4/l ib/rails/commands/commands_tasks.rb:90:in runner'
15:27:52 jobs.1 |       from C:/Ruby21/lib/ruby/gems/2.1.0/gems/railties-4.2.4/l
ib/rails/commands/commands_tasks.rb:39:in run_command!' 15:27:52 jobs.1 |       from C:/Ruby21/lib/ruby/gems/2.1.0/gems/railties-4.2.4/l ib/rails/commands.rb:17:in <top (required)>'
15:27:52 jobs.1 |       from bin/rails:4:in require' 15:27:52 jobs.1 |       from bin/rails:4:in '
15:27:52 jobs.1 | exited with code 1
15:27:52 system | sending SIGKILL to all processes
15:27:52 system | sending SIGKILL to web.1 at pid 8976
15:27:52 web.1  | exited with code 0
C:\Users\User\huginn>

Hey @amarkhalid, welcome! We recommend using VirtualBox on Windows for running Huginn.  Please see https://github.com/cantino/huginn/blob/master/doc/manual/requirements.md

Closing since we haven't heard back.  Hope it's working for you!


relace the topbar implementation

What type of PR is this?

/kind ui-change

Which issue(s) this PR fixes:

Fixes #1242

@puncsky fixed.

@puncsky Fixed media query error.


Find unused external dependencies

I would like to find unused npm packages

Hello @ild0tt0re,
Thanks for opening a feature request once again!
By "unused npm packages" do you mean packages that are present in the node_modules but not imported anywhere (ghost node_modules) or you were thinking about packages that are imported in the source code but not used in the files they are imported from?

Hello @ild0tt0re,
Thanks for opening a feature request once again!
By "unused npm packages" do you mean packages that are present in the node_modules but not imported anywhere (ghost node_modules) or you were thinking about packages that are imported in the source code but not used in the files they are imported from?

I mean the ghost node_modules

I think it would be useful yeah, but mainly for production dependencies, because the use of devDependencies would be too tedious to track (bundling or linting tools use is not visible from within the files themselves).
So for all production dependencies found in all the package.json files encountered starting from the directory from which the command "skott" was started, Skott will compare the ones used vs the ones not used.
Seems good? @ild0tt0re

I think it would be useful yeah, but mainly for production dependencies, because the use of devDependencies would be too tedious to track (bundling or linting tools use is not visible from within the files themselves).
So for all production dependencies found in all the package.json files encountered starting from the directory from which the command "skott" was started, Skott will compare the ones used vs the ones not used.
Seems good? @ild0tt0re

Yeah seems a good start!

A v0.13.0 was just published including the first version of the feature.
It adds a new --showUnusedDependencies flag which when combined with --trackThirdPartyDependencies will report all unused third-party dependencies.
Note that another flag --manifest can be used to provide a path to any package.json:
skott my-app/src/index.ts --manifest=my-app/package.json --trackThirdPartyDependencies --showUnusedDependencies

Closing this as it's probably good enough on this topic for the V1 to be landed.


VQSR - SNPs

The README references the following which isn't accessible:
https://trello.com/b/pcWKqbXv/stream-b-ngs-variant-calling-illumina

I'm closing this out of date pull request


[Q] [MP] [MHW] Monster & Damage meter widgets

How do I make HunterPie show the parts without focusing/aiming on the monster? Pressing Q to display the parts brings the monster to focus and constantly moves the screen which is making me dizzy.
If a group has a mission to kill one large monster, HunterPie will track the total damage only for the first large monster damaged. It will stop calculating the total damage for the second large monster should a player choose to kill a different monster before completing the objective. This was also causing the damage calculation to stop if different players go to separate monsters to kill separately. How to make HunterPie track total damage for each large monster or total damage for all large monsters damaged during a mission?

How do I make HunterPie show the parts without focusing/aiming on the monster? Pressing Q to display the parts brings the monster to focus and constantly moves the screen which is making me dizzy.

Support for other targeting methods is on its way for next release. As a workaround: change your lockon settings to turn off the focus.

If a group has a mission to kill one large monster, HunterPie will track the total damage only for the first large monster damaged. It will stop calculating the total damage for the second large monster should a player choose to kill a different monster before completing the objective. This was also causing the damage calculation to stop if different players go to separate monsters to kill separately. How to make HunterPie track total damage for each large monster or total damage for all large monsters damaged during a mission?

HunterPie does not calculate the damage, that's done by the game, it only displays the numbers. The game has limitations, it only works on quests, it only tracks damage done by monsters that are in the targets list, if the monster you're hitting is not a quest target then it won't be accounted for. It's not a HunterPie issue, it's a game limitation.


User can specify prior range

Changes discussed in #9

Hi @DeaglanBartlett, thanks again! This is looking good. Don't worry about the tests failing. This is to do with the accuracy of the KL divergence calculation and is a separate issue I need to look at more closely at some point. Occasionally it passes and other times it gets flagged, unfortunately.
Feel free to squash and merge this! :)

Hi @htjb. Thanks for letting me know this is the cause. I think you'll have to merge the pull request as I don't have write access to the repository

@DeaglanBartlett my bad I have added you as a collaborator, so you should be able to make the merge after accepting that.


Update to IKRPG

Fixing some images, clean up of css.  No HTML or sheet worker changes.
Include the name of the sheet you made changes to in the title.
Changes
Provide a few comments about what you have changed.

If changes fix a bug or resolves a feature request, be sure to link to that issue.
For pull request that change multiple sheets please confirm these changes are intentional for all sheets. This will help us catch unintended submissions.
When updating existing sheets if you are changing attribute names please note what steps you have taken, if any, to assist players in keeping their data.

Additional comments

Greetings @CoalPoweredPuppet
I approved these changes. Could we get those images moved to GitHub in the future? We are slowly working towards requiring all images be hosted in this repo.
Thanks,
Cassie

Thank you.  I will try to figure out the images-on-Github soon.


add health check for hgweb to match api version

this adds a health check to the api so when /api/healthz is called it verifies that hgweb is running the same version, if it's not then the health is degraded.
Todo: this won't work well in the develop envrionment because there's no guarantee that they will be the same version. Should we just disable this health check in develop?
closes #672

It won't work properly in develop, because we only deploy what changed, is that right?
So, either:

A PR changed both and they have the same version
A PR only changes 1, which should indicate that the new version of that is compatible with the old version of the other

So, yeah, disabling it in develop sounds good to me.

Yeah that's right about develop, we only push the changes that were made.

So, it seems to be working, but what do we get out of this?
[lexbox-api] warn: Microsoft.Extensions.Diagnostics.HealthChecks.DefaultHealthCheckService[103]
[lexbox-api]       => TraceId:fd6314267ee0210d42d107a3b5fbc5f7 => ConnectionId:0HN2JQPNPPUSR => RequestPath:/api/healthz RequestId:0HN2JQPNPPUSR:00000001 => Microsoft.Extensions.Diagnostics.HealthChecks.HealthCheckLogScope
[lexbox-api]       Health check hgweb with status Degraded completed after 50.4235ms with message 'api version: 'dockerDev' hg version: 'dockerDev2' mismatch'

I don't see the degraded status anywhere in k8s and it deploys the version regardless of the health being degraded. What do we want to happen?

that's a good point, my thinking was that the deploy script would catch it, but right now it only checks the version header and not the status code to see the response. Need to fix the script to keep trying until the request is healthy


TypeScript definitions

I'd like to use this module in TypeScript, but there is no definition file.
Adding a definition file involves little work and I wrote it already, so I could hack it into my project.
Would a PR for this be accepted? Or should I add a definition to DefinitelyTyped?
Thanks in advance!

@MichielvdVelde I think it's better to add it to DefinitelyTyped. We don't use TypeScript so definitions would not be maintained here.

Thanks for the fast reply. I will add a definition to DefinitelyTyped soon.

Thank you!


adding prefix

Proposing a solution to https://github.com/AnDiChallenge/andi_datasets/issues/29#issue-2290129496
I had an optional prefix argument to the file_nonOverlap_reOrg function.

Hi Maxime, thanks for your PR. I am checking now your changes. Note however that to contribute to the library, you need to do it by applying your changes in the notebooks in nbs > lib_nbs rather than the .py. This library has been generated via nbdev (https://nbdev.fast.ai/), which allows to generate library directly from jupyter notebooks.
Contributing is as follows: 1) change whatever you want in the notebooks; 2) run nbdev_export either running the last cell of the notebooks, which already contain that command or directly in the terminal. This changes the .py you were updating automatically.
I will try to add a contribution description in the Readme to clarify all this.

The issue for this problem has been solved with PR #32.


The role of modules

I have a question that I would like to be answered. What role do these two parts play? There is no introduction to them in the paper.

Thank you so much!

These two modules boost performance by mix-up for data augmentation.

These two modules boost performance by mix-up for data augmentation.

thank you very much!!

These two modules boost performance by mix-up for data augmentation.

Sorry to bother you. Though I have a question about dimensions: k is a (batch size, feature dim) shape tensor whereas the first dimension of c as well as p is batch size. So how could the multiplication happen when batch size was changed and no longer equal to feature dim?


artificially racing Refund and Punish confuses syncer (or just full node?)

node alice
2021-12-04T17:05:44Z INFO  farcaster_node::swapd::runtime] tx Cancel final: 0 confirmations
[2021-12-04T17:05:44Z INFO  farcaster_node::swapd::runtime] Broadcasting Punish tx 3180a9a31eccdc762f0389686ac0ed5a2ed869c5cbee5003443a12bc5886f401
[2021-12-04T17:05:44Z INFO  farcaster_node::syncerd::bitcoin_syncer] successfully broadcasted tx 3180a9a31eccdc762f0389686ac0ed5a2ed869c5cbee5003443a12bc5886f401
[2021-12-04T17:05:54Z INFO  farcaster_node::swapd::runtime] tx Refund on mempool but hasn't been mined
[2021-12-04T17:05:54Z INFO  farcaster_node::swapd::runtime] tx Refund final: 0 confirmations

node bob
[2021-12-04T17:05:40Z INFO  farcaster_node::swapd::runtime] Watching tx AccLock f22b45d8ddb6fd5b776f6e4a0f951b2829d11db7e0050004f0cf7321a9be1575 with id TaskId(7)
[2021-12-04T17:05:42Z INFO  farcaster_node::swapd::runtime] tx AccLock on mempool but hasn't been mined
[2021-12-04T17:05:44Z INFO  farcaster_node::swapd::runtime] tx Cancel on mempool but hasn't been mined
[2021-12-04T17:05:44Z INFO  farcaster_node::swapd::runtime] tx Cancel final: 0 confirmations
[2021-12-04T17:05:44Z INFO  farcaster_node::swapd::runtime] Broadcasting Punish tx 3180a9a31eccdc762f0389686ac0ed5a2ed869c5cbee5003443a12bc5886f401
[2021-12-04T17:05:44Z INFO  farcaster_node::syncerd::bitcoin_syncer] successfully broadcasted tx 3180a9a31eccdc762f0389686ac0ed5a2ed869c5cbee5003443a12bc5886f401
[2021-12-04T17:05:54Z INFO  farcaster_node::swapd::runtime] tx Refund on mempool but hasn't been mined
[2021-12-04T17:05:54Z INFO  farcaster_node::swapd::runtime] tx Refund final: 0 confirmations

I've observed this as well today! I think the syncer is mis-reporting the successful broadcast of a tx. Do debug level logs reveal more?

How is it possible that Alice and Bob try to punish? Does the Broadcasting Punish tx ... log entry mean something else

@h4sh3d you're right, but i dont manage to reproduce that log, so i probably messed up on the editting


[region-isolation] Improve errors around strong transferring and wordsmith main error

This series of commits:

Improves the diagnostics around strong transferring and wordsmiths the main transferring error to be smaller.
Adds a bunch more support in VariableNameInference for inferring more variable names.
Eliminates most of the call site passes self error. Still a little more to do, but this hits most of them.

@swift-ci smoke test


[OTHER] Create new subcategory Hacktoberfest

What would you like to share or ask?
Create a subcategory Hacktoberfest under open-source category. Also, add a highlighted new tag on open-source category.
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[x] I am willing to work on this issue (optional)

Good Morning,
I would like to work on this issue. I estimate it would take about 2 hours to update the list.

Assign to you @RJPalmer. If you have any doubt let me know.

@rajneesh-18
For the second task "add a highlighted new tag on open-source category", is the expectation to have the new tag added whenever a category or subcategory has changed or just a one time instance of the new tag?

@rajneesh-18
For the second task "add a highlighted new tag on open-source category", is the expectation to have the new tag added whenever a category or subcategory has changed or just a one time instance of the new tag?

It's just a one time instance


feat(plugin-chart-pivot-table): support series limit

SUMMARY
Support series limit along with row limit in Pivot Table v2.
Migration benchmark results:
10+: 0.38 s
100+: 0.25 s
1000+: 0.24 s
10000+: 0.36 s
100000+: 0.46 s
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:

After:

TESTING INSTRUCTIONS

Create a Pivot Table v2 chart
Set a series limit and series limit metric
Click run and go to View Query
Verify that the query contains a JOIN subquery ordered by selected series limit metric and limit equal to series limit
Verify that the query limit is equal to Cell limit and is ordereb by series limit metric.
Remove series limit metric and rerun the query
Verify that the query and JOIN subquery are ordered by the first selected metric in Metrics control

ADDITIONAL INFORMATION

[x] Has associated issue: closes #16901, #13562
[ ] Required feature flags:
[ ] Changes UI
[x] Includes DB Migration (follow approval process in SIP-59)

[x] Migration is atomic, supports rollback & is backwards-compatible
[x] Confirm DB migration upgrade and downgrade tested
[x] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

CC @rusackas @graceguo-supercat

Can't use custom metrics in sort control.

Can't use custom metrics in sort control.

Thank you @zhaoyongjie, well spotted! Issue fixed. Can you retest?


Upgrade airbrake gem v4.0.0 -> v4.3.8

I changed the constraint in the Gemfile from ~> 4.0.0 to ~> 4.0 and ran the following command:
bundle update --conservative airbrake

My main motivation for doing this is so that airbrake reports ActionController::BadRequest exceptions correctly.
However, I've chosen to upgrade to the latest 4.x version, because that should include other bug fixes without breaking compatibility with errbit. I'm pretty sure this is safe to do, because I can see that a bunch of other GOV.UK apps are using this version of airbrake in production, e.g.

calculators
hmrc-manuals-api
manuals-publisher
publisher
rummager

Closes #122.

@chrislo: Thanks for reviewing.
Rebasing against master and force-pushing in preparation for merging.


Update history extend history message

Some large zones need more large record history msg size because msg(256) was truncated and missing records audit.

Hello,
You need to generate the correspondig alembic script by running flask db migrate into the virtualenv.
Thanks

@vdchuyen I'm cleaning up idle PRs and trying to get merges caught up. Are you planning on addressing the requested change? If not, I'm going to close this PR for now. Thanks!

@AzorianMatt you can close this PR. For sqlite we have to use a little trick to create new table schema and copy around. Maybe I am the only one need this, so I will patch for my own. Thanks anyway.


Implementation of ICA using fixed point iteration

[ ] A. Hyvärinen, "Fast and Robust Fixed-Point Algorithms for Independent Component Analysis," IEEE Trans. on Neural Netw., vol. 10, no. 3, pp. 626-634, 1999.

FastICA has been already implemented in v0.0.0, so I will close this issue.


add hooks support

Once hooks reaches the non-beta branches, react-values should expose themselves as hooks. And eventually we can deprecate the render prop approach.

Just to sketch out the API ideas... we'll want to be able to provide the absolute simplest API for the common case of just tracking some internal state:
import { useBoolean } from 'react-values'

function Toggle() {
  const enabled = useBoolean(false)
  return (
    <Track enabled={enabled.value} onClick={() => enabled.toggle()}>
      <Thumb enabled={enabled.value} />
    </Track>
  )
}
...

But in doing so, we're losing a bit of the niceness that this library provides around automatically handling the "controlled vs. uncontrolled" problem for you. Because for UI components, it's really nice to be able to use these value hooks to avoid having to think about "controlled-ness". But for that, we need to be able to pass defaultValue and value. So we'd introduce a useControllable* variant:
import { useControllableBoolean } from 'react-values'

function Toggle(props) {
  const enabled = useControllableBoolean(props)
  return (
    <Track enabled={enabled.value} onClick={() => enabled.toggle()}>
      <Thumb enabled={enabled.value} />
    </Track>
  )
}

And then finally, we still want to also keep the "connected" concept that is currently available, allowing you to share state across the React tree. To do that, we'd expose createConnected* factories that return React hook functions:
import { createConnectedBoolean } from 'react-values'

const useEnabled = createConnectedBoolean(false)

function Toggle() {
  const enabled = useEnabled()
  return (
    <Track enabled={enabled.value} onClick={() => enabled.toggle()}>
      <Thumb enabled={enabled.value} />
    </Track>
  )
}

You could remove the createConnected* factories and let the user handle the "connected" concept via Context. It's a well documented pattern and removes complexity from react-values. What do you think @ianstormtaylor ?
import React from 'react';
import { useBoolean } from 'react-values';

+ const Context = React.createContext(null);

function Toggle() {
+  const enabled = React.useContext(Context);
  return (
    <Track enabled={enabled.value} onClick={() => enabled.toggle()}>
      <Thumb enabled={enabled.value} />
    </Track>
  )
}

function App() {
  return (
+    <Context.Provider value={useBoolean()}>
      <Toggle />
      <Toggle />
    </Context.Provider>
  );
}


docs(material/theming): rewrite theming guide for @use

This change completely rewrites the theming guide to be more complete, correct, and concise.
Summary of changes:

Explain the concepts of palettes, hues, and themes.
Ensure all content is conceptual and not task-based.
Use the new Sass @use API introduced in #22173
Document usage of the theming system with shadow dom
Document the fact that CSS overrides are strongly discouraged
Be more explicit about which palettes we provide
Consolidate on a single approach to including theme CSS in an app (assuming Angular CLI)

I will send follow-up PRs for the typography, theming-your-components, and customizing-component-styles guides.

Comments addressed- I updated the getting started guide with the correct header link (the prebuilt one is actually still the same).
As for quickly getting started- I think that this is covered already by the getting started guide which lets you just select a prebuilt theme during ng add


fix(server): use pod-name:pod-ip as node identity for kubearmor daemon

to avoid the case, if kubearmor pod recreated for any reason and pod deletion event received after pod added event then node will be removed from current list of nodes/kubearmor-pods if node id is created using node-ip only.
{"level":"warn","ts":"2024-11-21 13:42:55.674541","caller":"log/logger.go:79","msg":"failed to receive an alert (<IP_ADDRESS>:32767) rpc error: code = Unavailable desc = error reading from server: EOF"}
{"level":"info","ts":"2024-11-21 13:42:55.674623","caller":"log/logger.go:54","msg":"Destroyed the client (<IP_ADDRESS>:32767)"}
{"level":"warn","ts":"2024-11-21 13:42:55.674978","caller":"log/logger.go:84","msg":"Failed to call WatchMessages (<IP_ADDRESS>:32767) err=rpc error: code = Unavailable desc = connection error: desc = \"transport: Error while dialing: dial tcp <IP_ADDRESS>:32767: connect: connection refused\"\n"}
{"level":"info","ts":"2024-11-21 13:42:55.874444","caller":"log/logger.go:54","msg":"[addevent] pod kubearmor-bpf-containerd-98c2c-4krn9 has been added"}
{"level":"info","ts":"2024-11-21 13:42:56.273261","caller":"log/logger.go:54","msg":"[deleteEvent] pod kubearmor-bpf-containerd-98c2c-s9bnk has been deleted"}
{"level":"info","ts":"2024-11-21 13:42:56.273329","caller":"log/logger.go:54","msg":"pod with nodeip <IP_ADDRESS> deleted"}

/review @anurag-rajawat


test

@github-actions test

@github-actions publish-pr-on-npm

@github-actions publish-pr-on-np

@github-actions publish-pr-on-npm

@github-actions publish-pr-on-npm

@github-actions publish-pr-on-npm

@github-actions publish-pr-on-npm

@github-actions publish-pr-on-npm

@github-actions publish-pr-on-npm

@github-actions publish-pr-on-npm

@github-actions publish-pr-on-npm

@github-actions test

@github-actions run-benchmark

@github-actions test

@github-actions publish-pr-on-npm

@github-actions run-benchmark

@github-actions run-benchmark

@github-actions run-benchmark

@github-actions run-benchmark

@github-actions run-benchmark

@github-actions run-benchmark

@github-actions run-benchmark

@github-actions run-benchmark

@github-actions run-benchmark

@github-actions run-benchmark

@github-actions run-benchmark


Simple examples for Taipy/Chalk'it

For existing examples, or tutorial examples (not related to dataNodes), provide simple Taipy/Chalk'it illustrations in xprjson format

Done


Deprecated Gradle APIs in use, will break in Gradle 8

They all appear to be from Dokka, as documented in this issue from 2021.
This issue is just a reminder to come back and look at this in advance of Gradle 8.

Well, turns out it's not Dokka, it's Kotlin.  Even as of 1.7.20, with Gradle 8.0 builds coming thick and fast, these still haven't been fixed:
~/D/thrifty ❯❯❯ ./gradlew -s clean check --warning-mode all                                                                                ✘ 1
The TestReport.destinationDir property has been deprecated. This is scheduled to be removed in Gradle 8.0. Please use the destinationDirectory property instead. See https://docs.gradle.org/7.6/dsl/org.gradle.api.tasks.testing.TestReport.html#org.gradle.api.tasks.testing.TestReport:destinationDir for more details.
        at org.gradle.api.tasks.testing.TestReport.setDestinationDir(TestReport.java:92)
        at org.jetbrains.kotlin.gradle.testing.internal.KotlinTestReport_Decorated.setDestinationDir(Unknown Source)
        at org.jetbrains.kotlin.gradle.testing.internal.KotlinTestsRegistry$doGetOrCreateAggregatedTestTask$aggregate$1.invoke(KotlinTestsRegistry.kt:74)
        at org.jetbrains.kotlin.gradle.testing.internal.KotlinTestsRegistry$doGetOrCreateAggregatedTestTask$aggregate$1.invoke(KotlinTestsRegistry.kt:70)
        at org.jetbrains.kotlin.gradle.tasks.TasksProviderKt$sam$org_gradle_api_Action$0.execute(TasksProvider.kt)

Not really sure what we can do other than eat popcorn and watch the show, waiting for JetBrains to fix their stuff.


No Session in Http Context

Through HttpContextBase or UmbracoContext, session is always null. Is there any workaround to get session from Umbraco CMS when using docTypeGridEditor?

Hi @zharonar is this when the previews are rendering, or when the frontend of the website is rendering?

Hi again @zharonar
Just tried adding Session["hello"] = "HELLO"; to my layout view, and then
@Session["hello"]
 to one of my DTGE partials. And that worked fine.

So I suspect your are missing the Session in the previews? Whats your usecase for that?

Hey @skttl , thanks for replying fast! This happens only when I am at the preview, I have a lot of Session uses in my backcode, controllers etc., and when I'm at preview tones of exceptions occurred and I was wondering what was it and how to get session from preview.

Thanks, the session is unavailable in the previews because they are running on an Api Controller, which doesn't have access to the session.
It would be a breaking change to change that, and I don't really see the benefit of having session data in previews.
You can always null check you Session variables and show something else if they don't exist. Or create another view for previewing.


Add oracle icons

https://docs.oracle.com/en-us/iaas/Content/General/Reference/graphicsfordiagrams.htm

Wanted to make a point I might not have mentioned yet: This isn’t so much a one-time ask as it is initiating an ongoing relationship. We plan to update the icon set about twice a year for the foreseeable future as we add new services.

There's been no interest in these, apart from Oracle. For low demand icons the best route is to host the set somewhere and build a link that allows users to easily open the set.
However, with no votes in 15 months, this issue is declined.

I just have to say: this was an incredibly bad experience on our side. We approached you wanting to engage and create a relationship. There was practically zero response other than creating this issue. No guidance whatsoever on how to advocate for ourselves was provided or what the basis for any decision would be, and nobody showed the slightest interest in meeting us even part way much less half way. Over a year later, it gets closed with zero discussion.  Is this how you treat AWS?


[9.0.2] Edit page type error

Enable php8 for concreteCMS 9.0.2 and go to Dashboard > Pages&Themes > Page Types click on Basic Details
Generates the following error (see attached screenshot):
Undefined property: Concrete\Core\Page\Type\PublishTarget\Configuration\AllConfiguration::$startingPointPage

When we switch back to PHP7.4, the page works completely correctly again.

Have made the same experience. php 7.4 solves the problem.


Root_Block_Device is not expected here

Description
Getting a Root_Block_Device is not expected here  error message when using an example from this repo

[x ] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]: "4.1.4"
Terraform version: "4.29.0"
Provider version(s): N/A

Reproduction Code [Required]
  root_block_device = [     {       encrypted   = true       volume_type = "gp3"       throughput  = 200       volume_size = 50       tags = {         Name = "my-root-block"       }     },   ]
Steps to reproduce the behavior:
 no
 yes

Expected behavior
Root block device should have been succesfully created
Actual behavior
When copy pasting the example directly, i would get that error
Terminal Output Screenshot(s)

An argument named "root_block_device" is not expected here. Did you mean to
define a block of type "root_block_device"?

Additional context
This is the fix
root_block_device {     volume_size           = "${var.EC2_ROOT_VOLUME_SIZE}"     volume_type           = "${var.EC2_ROOT_VOLUME_TYPE}"     delete_on_termination = "${var.EC2_ROOT_VOLUME_DELETE_ON_TERMINATION}"   }

Can you confirm you're trying to use the terraform-aws-ec2-instance module, and not the aws_instance resource directly from the terraform-provider-aws provider?  The terraform-aws-ec2-instance module expects root_block_device to be a list, as your reproduction code shows, while the aws_instance resource expects it as a nested configuration block.  Your fix is not a list, and since it works for you it makes me think you're not using terraform-aws-ec2-instance.
Here's a simplified reproduction of creating an instance using terraform-aws-ec2-instance that can be run with terraform init && terraform apply that will hopefully help you.
data "aws_ssm_parameter" "ubuntu-jammy-amd64" {
    name = "/aws/service/canonical/ubuntu/server/22.04/stable/current/amd64/hvm/ebs-gp2/ami-id"
}

module "ec2_instance" {
  source  = "terraform-aws-modules/ec2-instance/aws"
  version = "4.1.4"

  name = "t2-root-block"

  ami                    = data.aws_ssm_parameter.ubuntu-jammy-amd64.value
  instance_type          = "t2.micro"
  monitoring             = false

  enable_volume_tags = false
  root_block_device = [
    {
      encrypted   = false
      volume_type = "gp3"
      throughput  = 200
      volume_size = 50
      tags = {
        Name = "my-root-block"
      }
    },
  ]

  tags = {
    Test = "root-block"
  }
}

output "instance_id" {
  value = module.ec2_instance.id
}

❯ terraform apply
Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

Outputs:

instance_id = "i-04b6f31fdb3db51cb"
❯ aws ec2 describe-volumes --filters Name=attachment.instance-id,Values=i-04b6f31fdb3db51cb
{
    "Volumes": [
        {
            "Attachments": [
                {
                    "AttachTime": "2022-09-06T00:06:33+00:00",
                    "Device": "/dev/sda1",
                    "InstanceId": "i-04b6f31fdb3db51cb",
                    "State": "attached",
                    "VolumeId": "vol-059f0dd86d2429d73",
                    "DeleteOnTermination": true
                }
            ],
            "AvailabilityZone": "us-east-1a",
            "CreateTime": "2022-09-06T00:06:33.375000+00:00",
            "Encrypted": false,
            "Size": 50,
            "SnapshotId": "snap-0da515346cb3d642b",
            "State": "in-use",
            "VolumeId": "vol-059f0dd86d2429d73",
            "Iops": 3000,
            "Tags": [
                {
                    "Key": "Name",
                    "Value": "my-root-block"
                }
            ],
            "VolumeType": "gp3",
            "MultiAttachEnabled": false,
            "Throughput": 200
        }
    ]
}


cloudant package : create and delete trigger should should verify creation or deletion

https://github.com/apache/incubator-openwhisk-package-cloudant/blob/master/actions/changesWebAction.js
The functions createTrigger and deleteTrigger should guarantee on return, that the trigger is created or deleted.
Our test code based on this package cannot check the existence of the trigger and we get sometimes errors when the creation or deletion is processed delayed.

We cannot make this guarantee in the current design.  The changesWebAction only creates/deletes the database entry and has not contact with the backend provider.  The trigger feed gets created by the backend provider when it is notified by a cloudant changes feed that a database entry has been created or deleted.

@jasonpet - is there a state in that db record that indicates the feed has been actually created?


can't access wss object

need to access wss object sometimes, such as get all clients and broadcasting, like this:
https://github.com/einaros/ws#server-sending-broadcast-data

Your use case should be possible in v0.2.4.
See example/broadcast.js


feat(proto): implement Interval and Nothing

Ref: #120

Codecov Report

Merging #135 (f193225) into main (a62ef09) will decrease coverage by 0.22%.
The diff coverage is 66.66%.

:exclamation: Current head f193225 differs from pull request most recent head 92595c4. Consider uploading reports for the commit 92595c4 to get more accurate results

@@            Coverage Diff             @@
##             main     #135      +/-   ##
==========================================
- Coverage   76.02%   75.80%   -0.23%
==========================================
  Files         120      122       +2
  Lines        5710     5782      +72
==========================================
+ Hits         4341     4383      +42
- Misses       1164     1189      +25
- Partials      205      210       +5

Impacted Files
Coverage Δ

cht/config.go
57.14% <ø> (ø)

proto/column.go
78.94% <ø> (ø)

proto/col_interval.go
53.12% <53.12%> (ø)

proto/col_nothing.go
76.92% <76.92%> (ø)

proto/col_auto.go
73.49% <78.57%> (+1.03%)
:arrow_up:

client.go
72.82% <0.00%> (-1.05%)
:arrow_down:

query.go
66.10% <0.00%> (-0.64%)
:arrow_down:

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 a62ef09...92595c4. Read the comment docs.


Small suggestion for Chapter 11 fail/assert remarks

Thank you for your highly useful and highly readable book on Ansible.  I have dog-eared my copy extensively!
In Chapter 11 there is a section named "The fail and assert modules".
It currently says:
Both fail and assert, when triggered, will abort the playbook
run, and the only difference is in the simplicity of their usage.

I humbly suggest amending that to something like:
Both fail and assert, when triggered, will abort the playbook
run. The choice between these two is one of style and simplicity.

One subtle difference is that a 'fail' task will be reported as
'skipped' in the final recap statistics when the failure is not
triggered.

If your playbook contains a single 'fail' task that is happily
not triggered, the playbook results will contain:

PLAY RECAP *********************************************************************
default                    : .......(snip)....    skipped=1

If your playbook instead contains a single 'assert' task that is
happily not triggered, you will see "skipped=0" and the
individual task outcome will be:

TASK [my_assert] ****************************************
ok: [default] => {
    "changed": false,
    "msg": "All assertions passed"
}

This subtlety about "skipped" task count can be relevant in scenarios where one must audit and justify tasks that have been skipped.

That's a good point! Didn't even think about that use case, but that would be one case where it's probably best to use assert rather than fail.

I have updated the book with something along the lines of your suggestion. Thanks for the issue!


[KBFS-1398] Allow head replacement in mdJournal

This handles the case where we interrupt an MD put
into the journal. We can safely replace the head
since it hasn't been pushed to the server yet.

I spent some time looking through the code to make sure there's nothing that creates a successor MD and then puts it without holding mdWriterLock the whole time.  If there were, this could cause trouble, but I couldn't find any.  Might be worth double checking. (And maybe documenting above getMDForWriteLocked that the caller must hold the lock until they do a Put?)
I thought I remembered some cases where quota reclamation could cause self-conflicts without a cancelled operation, but maybe that's only when there are two devices involved.
Anyway, modulo all that and the comment, looks good to me.

according to the comments for mdWriterLock:
// 1) mdWriterLock: Any "remote-sync" operation (one which modifies the
//    folder's metadata) must take this lock during the entirety of
//    its operation, to avoid forking the MD.

But did double-check, and added a comment to getMDForWriteLocked.
How would quota reclamation cause self-conflicts without cancelling? That should still be okay, right?

How would quota reclamation cause self-conflicts without cancelling? That should still be okay, right?

Well I thought it might have been that QR didn't make its successor and call put under the same lock, but it seems that it does.  So I think I must have misremembered.
￼


Scans end up blank

after the outage it returned only to get blank scans

no pokemon are appearing

Pokémon Go and PTC servers are all struggling right now.

mines not working either lol...... and i show the servers are running well.......maybe the finally did it ?

I can say that app state is linked to Pokemon Server so just wait for them

troublesome, the phone detects pokemons

hmmmmm lol maybe they finally found a way to block the API?

Nah their servers are still struggling. PTC is down again.

could this be related?
https://twitter.com/nycpokespawn/status/760932605097308160

"Following a supposed API change, we will be down for approximately 24 hours." ??

Its going to be pretty long hours without Pokemap, seems like that.

Noooo my pokemans : (

No footprints and now absolutely nothing, GG shitTic.

My question is after N* saaw soooooo many people leave the game when they took out the foot prints why instead of trying to limit the service why dont they speak with the guys here to team up and sort out their tracker?

The Plural of Pokémon is Pokémon.....Thats why the server dropped.....to many people searching for Pokémons......FACEPALM

found the related thread here.
https://www.reddit.com/r/pokemongodev/comments/4w0jum/all_ptcgoogle_logins_failing_from_api/

Fun while it lasted.  Hopefully someone fingers out a way to make scanners worked again.
Deuces!

https://github.com/NECROBOTIO/NecroBot/commit/35339c3f527e7f934ae5fae6a7c7aa45e246be32
Looks like they changed mapObjects. A fix is on its way.


ENSCORESW-2967: update schema to 96

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
if you change the schema you must patch the test databases as well, see Updating the schema
the PR must not fail unit testing

Description
Using one or more sentences, describe in detail the proposed changes.
Schema is updated for release 96
Use case
Describe the problem. Please provide an example representing the motivation behind the need for having these changes in place.
See above
Benefits
If applicable, describe the advantages the changes will have.
Schema is up to date for release 96
Possible Drawbacks
If applicable, describe any possible undesirable consequence of the changes.
Testing
Have you added/modified unit tests to test the changes?
If so, do the tests pass/fail?
Have you run the entire test suite and no regression was detected?
Yes

Coverage remained the same at 81.365% when pulling 5ddf7bfd36277f20516e0cefd9e3e4a8040f47d9 on feature/schema_update_96 into e3f8508b5bc064d244b2b35c1dbf57793b0659f7 on master.


fix: allow session re-use by re-reading session_id from local storage

Proposed changes:
Re-read session_id from local storage on socket connect and session_confirm to allow re-use of session.
Before, the session_id was only read on widget initialization which means if the connection was lost and then re-established without re-initializing the component the session_id, wasn't reused.
Status (please check what you already did):

[X] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog

@GuillaumeTech @MatthieuJnon any chance you could take a look at this? It's kind of a big deal for the project we are working on right now :pray:


Install fails because nu-plugin and nu-protocol are looking for non-existent local paths

The nu-plugin and nu-protocol dependencies are looked for in a sibling of this crate's directory:
https://github.com/Euphrasiologist/nu_plugin_plot/blob/5d3a7339123f807a97804b7b2264398faec2e68a/Cargo.toml#L11-L12
But the install instructions say nothing about this.

You're totally right. I think this was added from a contributor with a pull request, clearly didn't spot that! Thanks.


Add IntoValue trait to allow borrowed Value

This attempts to close #16 without breaking changes (I think, at least; inference may break, but that's not a SemVer breaking change) by using approach no. 2 detailed in the original issue.
The IntoValue trait looks like this:
pub trait IntoValue<'a> {
    fn into_value(self) -> Cow<'a, Value>;
}

Leaving this as a draft because this seems a bit messy and I think other approaches (including using V: AsRef<Value> instead) should still be considered, if this is even a use case you'd be open to support. Any input would be appreciated.

I think V: AsRef<Value> is cleaner.

I agree 😅

Also now the name render_from doesn't make as much sense, might be worth renaming it 🤔. Possibly something to think about for #15.

That makes sense. I think it would be fine to leave this as a draft until after that discussion is finalized, then maybe either pick up from here or start another PR.
Thank you for your input!

That makes sense. I think it would be fine to leave this as a draft until after that discussion is finalized, then maybe either pick up from here or start another PR.

Up to you, I would be totally fine with doing the AsRef<Value> thing and merging it now already.

Up to you, I would be totally fine with doing the AsRef<Value> thing and merging it now already.

Do you think AsRef<Value> would be better or Borrow<Value>? The former is more flexible but doesn't cover Value (so we must pass &value instead of value as we currently do). The latter is less flexible but has a blanket impl for T.
See: https://doc.rust-lang.org/std/convert/trait.AsRef.html#relation-to-borrow

Do you think AsRef<Value> would be better or Borrow<Value>? The former is more flexible but doesn't cover Value (so we must pass &value instead of value as we currently do). The latter is less flexible but has a blanket impl for T.
See: https://doc.rust-lang.org/std/convert/trait.AsRef.html#relation-to-borrow

I don't think Borrow is appropriate in this context. AsRef makes the most sense to me.

I implemented the AsRef way of doing this in 58b296f


Implemented: logic to fetch Product identification from  Dxp component

Related Issues

#237
Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

Contribution and Currently Important Rules Acceptance

[x] I read and followed contribution rules

Closing this PR in context of the following PR - https://github.com/hotwax/preorder/pull/308/files


Improved tab completion on implant options

This PR is a small, but incredibly satisfying quality of life improvement. Tab completion for implant options only worked if you started typing the option name in the same case. Now you can use either case and it will tab complete.

Github needs more reaction emojis.


Osi grpc communication

Feature Request
Adding new module osi_grpc_communication_module.
Description

The module provides a standardized interface to connect CARLA with third party tools.
Module uses the gRPC (google remote procedure call) to send the groundtruth measurement data of the host_vehicle and the non_player_agents (vehicles) with OSI standardization (Open Simulation Interface) at every frame in the episode.
All the proto_to_python generated files are present with the module. they can be readily used for the compilation and working.
the origional .proto files are also present for the manipulaiton and advancement.
osi_grpc.proto is used to generate the .py version of the request with protoc and a special gRPC plugin

Where has this been tested?

Platform(s):
Windows 10 x64
Python version(s):
Python 3.6
Unreal Engine version(s):
UE 4.18.3
CARLA version(s):
0.8.4

This change is

Hi @mshaheerm,
Please rebase your changes into stable so we can merge your changes only, otherwise we are merging whole 0.8.4 and we don't want that.


Fix sorting of hidden and empty inlines

Without this jQuery ui selector and when dealing with many inline types, i've experienced some problems with reordering inlines. This fixes the problem.

Thank you! I've already merged this (as it seems obviously correct) but some additional explanation would be great -- can you describe the problems you had a bit?

Well, i've experienced some problems when sorting, the blue bar would only occasionally appear on the right place but mostly it would appear on the bottom of the list. When looking at the inspector, i noticed that the invisible forms (class .form-empty) were included in the sorting and were placed above and in between the real instances. Setting an item selector on the jQuery ui sortable method, to exclude "non elements", solved this problem.


Error when updating Xamarin.Android.Support to version 27.0.2

Xamarin.Auth Issue

[ ] nuget version = <IP_ADDRESS>

Steps to reproduce

Try to install on Android project

NU1107: Version conflict detected for Xamarin.Android.Support.Compat. Reference the package directly from the project to resolve this issue.
TestProject.Android -> Xamarin.Android.Support.v7.MediaRouter 27.0.2 -> Xamarin.Android.Support.v7.Palette 27.0.2 -> Xamarin.Android.Support.Compat (= 27.0.2)
TestProject.Android -> TestProject-> Xamarin.Auth <IP_ADDRESS> -> Xamarin.Android.Support.CustomTabs <IP_ADDRESS> -> Xamarin.Android.Support.Compat (= <IP_ADDRESS>).

Same problem here:
Severity	Code	Description	Project	File	Line	Suppression State
Error		Version conflict detected for Xamarin.Android.Support.Compat. Reference the package directly from the project to resolve this issue.
OAuthNativeFlow.Droid -> Xamarin.Android.Support.Design 27.0.2 -> Xamarin.Android.Support.Compat (= 27.0.2)
OAuthNativeFlow.Droid -> Xamarin.Android.Support.v4 <IP_ADDRESS> -> Xamarin.Android.Support.Compat (= <IP_ADDRESS>).			0

Having a similar issue and haven't been able to resolve. Guessing it has something to do with Xamarin.Auth hard dependency on Xamarin.Android.Support.Compat (= <IP_ADDRESS>) and Nuget misunderstanding the dependency chain. But, project builds
``

some issue here.

Same here. Is it possible to backdate the plugins to when Auth was compatible? I've tried rolling back the packages, but they are inter-dependent on their current version and NuGet won't let me roll them back.

I dealt with it some time ago already, could you guys remind me what were
the steps to recreate it please?
On 19 June 2018 at 22:32, ckhericks<EMAIL_ADDRESS>wrote:

Same here. Is it possible to backdate the plugins to when Auth was
compatible? I've tried rolling back the packages, but they are
inter-dependent on their current version and NuGet won't let me roll them
back.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/xamarin/Xamarin.Auth/issues/294#issuecomment-398535537,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKtTIODdPtNBvH92H1Cbcqc0FWWLJtpxks5t-V_ngaJpZM4T342B
.

--
Best Regards,
Gregory Goraj
+45 601 355 79

and errors u see
On 20 June 2018 at 00:17, Greg Goray<EMAIL_ADDRESS>wrote:

I dealt with it some time ago already, could you guys remind me what were
the steps to recreate it please?
On 19 June 2018 at 22:32, ckhericks<EMAIL_ADDRESS>wrote:

Same here. Is it possible to backdate the plugins to when Auth was
compatible? I've tried rolling back the packages, but they are
inter-dependent on their current version and NuGet won't let me roll them
back.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/xamarin/Xamarin.Auth/issues/294#issuecomment-398535537,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKtTIODdPtNBvH92H1Cbcqc0FWWLJtpxks5t-V_ngaJpZM4T342B
.

--
Best Regards,
Gregory Goraj
+45 601 355 79

--
Best Regards,
Gregory Goraj
+45 601 355 79

Literally exactly the same error robsaccone posted above, error mismatch with Xamarin's support extensions. I assume an explicit version dependency is stated somewhere that now crashes with a more recent version released.

I'm using Xamarin.Auth <IP_ADDRESS> with Xamarin.Forms <IP_ADDRESS>0893 though I've tried newer/older forms versions.  Using the Andriod.Support* packages that Forms brings along, in this case <IP_ADDRESS>.
Version conflict detected for Xamarin.Android.Support.Compat. Reference the package directly from the project to resolve this issue.
 GigSteady.Android -> Xamarin.Android.Support.v7.MediaRouter 27.0.2 -> Xamarin.Android.Support.v7.Palette 27.0.2 -> Xamarin.Android.Support.Compat (= 27.0.2)
 GigSteady.Android -> GigSteady -> Xamarin.Auth <IP_ADDRESS> -> Xamarin.Android.Support.CustomTabs <IP_ADDRESS> -> Xamarin.Android.Support.Compat (= <IP_ADDRESS>).

This step worked for me:

Uninstall all your Xamarin.Android.Support.XX and Xamarin.Auth packages
Close your solution (maybe not needed)
Install these packages back:

All your Xamarin.Android.Support.XX <IP_ADDRESS>
Xamarin.Android.Support.CustomTabs <IP_ADDRESS> (important)
Xamarin.Auth 16.0.2

Note: If your Solution contains multiple project, you also need to apply that steps to project that depends on Xamarin.Android.Support

You all should be able to work around this issue if you manually edit the packages.config or your .csproj files to change all of the version numbers for ALL matching Xamarin.Android.Support.xxxxx at the same time.

Installing Xamarin.Android.Support.CustomTabs Works
Thanks!

Where do I send the beer?  Thanks!


Minor Copy Edits

I have made minor copy edits such as spelling and grammar edits to the documents in the "make_an_app" section.

Thanks!


Set the correct script tags for development and production

This seems to be how we do it in other custom elements, see <auto-complete>.

Can we also change the comment to say "demo script" as opposed to "production script"? Perhaps clarify that we are using it for GitHub pages demonstration purposes.


Remove the need for cookies.set

Fixes

Fixes #[issue number] by @[issue author]
Description

This is an exploration of a wrapper composable for the UI store that allows the composables to update the cookies without having to pass the cookies.set.
It makes updating the UI state from the components easy, but adds some complexity to it. I'm also not sure if this won't cause memory issues.
Testing Instructions

Checklist

[ ] My pull request has a descriptive title (not a vague title like Update index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Closing this PR as I found a better solution.


Use a webhook to a Travis CI deployment

This way whenever I submit a pull request to myself, it will automatically run my tests. This way I can display how good my test coverage is, and that the tests pass.

This is going to take to long to figure it out. It'll have to be moved to the next sprint.

This would have been great to have done first thing in the project. It doesn't have as much value now.
Looking back, I would have started with writing an acceptance test that checked if the algorithm was indeed unbeatable. I would have included that in this CI deployment along with the suite of unit tests. And I would have always branched on each individual task that had to be acoomplished submitting PRs against the CI tests. Then I'd have known for certain I was done.
It didn't occur to me that the project as a whole needed to be test driven too. Never experienced that just writing code challenges. Now I know! And boy does it make sense to do that. It would have allowed the project to focus better and probably would have driven cleaner code.
This will be something to do next time.

Finally did it! :tada: It wasn't nearly as hard as I was making it out to be in my head.


[REQ FEATURE] Lower .Net Dependency

Hello @NYAN-x-CAT, love the work.
It is possible to have the stub/client a lower .net dependency? Say 3.5.1 ?
I have been pen testing systems using your RAT. However finding this painful due to the dependencies of .Net 4.0.
I know Win10 supports .net4 by default, but if the target was an earlier system it makes the RAT less reliable.

Didnt read stub source, but maybe few features need 4.0 framework, however you can use nyan's vbs downloader and check for 3.5 or 4.0 then download what u want

@b7001
You can always lower it to 2.0 or 3.5 but you need to do some changes. the project will remain at 4.0 so you will have to change everything on every update.
@UsrTkn I updated the script. it's FUD and fileless.


lmerge/ prune workflow still results in overlapping calls

Just wanted to bring up that running lsort/lmerge/prune pipeline seems to still result in a number of overlapping calls (>50% Reciprocal overlap for about 1000 variant pairs in my filtered call-set). I am curious as to if this behavior is expected.  They don't tend to have similar AF (which is probably why they aren't merged but I am have been taking the set of breakpoints with the highest AF and removing others as my solution for now.
Let me know if you have any thoughts,
Thanks!

I wanted to add in this example as something I wouldn't expect after merging or pruning

this particular variant is found in mostly the same individuals, and overlaps >50%

Hello, I have met the same problem, have you found the answer?


Time required to download 41M data

The source website is very susceptible to multiple API requests at the same time. I was wondering,

What is the exact number of these variables in the settings file while you scraped the data,

CONCURRENT_REQUESTS = ?
# Configure a delay for requests for the same website (default: 0)
# See https://docs.scrapy.org/en/latest/topics/settings.html#download-delay
# See also autothrottle settings and docs
DOWNLOAD_DELAY = ?
# The download delay setting will honor only one of:
CONCURRENT_REQUESTS_PER_DOMAIN = ?
CONCURRENT_REQUESTS_PER_IP = ?

How much time did it take to scrape the full data (41M samples)?
Is there anyway you can share the scrapped data? In that way, the source website won't have to respond to millions of API requests.

The full data is now available: https://huggingface.co/datasets/rahular/varta


More functions for the function animations

We now have two planned types of "function" animations - animations like your math animation or my upcoming  reprocessing animations, animations which rely on selecting some mathematical formula or simple function to get interesting behavior.
People love this [stuff].  We should extract out the animation code into a separate file that's referred to using loady names, and then throw in several dozen more, and then we can say, "100 new waveforms" or something like that.

Not sure what you mean exactly...
Here's the mathfunc stuff, btw:
https://github.com/ManiacalLabs/BiblioPixelAnimations/blob/master/BiblioPixelAnimations/matrix/MathFunc.py
On Thu, Oct 5, 2017 at 5:16 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

We now have two planned types of "function" animations - animations like
your math animation or my upcoming reprocessing animations, animations
which rely on selecting some mathematical formula or simple function to get
interesting behavior.
People love this [stuff]. We should extract out the animation code into a
separate file that's referred to using loady names, and then throw in
several dozen more, and then we can say, "100 new waveforms" or something
like that.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/453, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AA6a6infcby4fO0-15gy9ZFuaQi_fVC4ks5spUcTgaJpZM4Pvtiy
.

Ah, great, thanks!
I'll clarify this once the reprocessing animation is in, so there's a clear example.


Load-Cookie: support passing AMP consent strings

With the current amp-iframe approach, it's not possible to obtain GDPR consent strings and pass them through load-cookie.html. So AMP-originated calls to Prebid-Server's /cookie_sync endpoint will never contain GDPR consent data.
This is obviously a problem -- it causes every /cookie_sync to fail for users in the EEA.
Passing GDPR consent strings through AMP iframes isn't currently possible. As of this writing the AMP PR is in-progress -- https://github.com/ampproject/amphtml/pull/29636 isn't merged.
But even when it is merged, our amp-iframe implementation is going to require work. Instead of the nice simple  approach, we're going to need to be more dynamic because the iframe needs to post a message to get consent data and parse the results.
Here's a proposed approach:

amp-iframes can load a new (optional) version of load-cookie.html (e.g load-cookie-with-consent.html)
This new file takes all the same parameters as load-cookie.html
But it also contains javascript that polls the CMP for results
when it gets results, it parses the consent string and dynamically generates the call to PREBID-SERVER
drop the pixel to PREBID-SERVER

The AMP PR has been merged -- this issue is unblocked.

@klipstein - the I2I you opened on the AMP project - does that mean that the proposed solution above won't work?

@bretg sorry for the confusion. The above solution would work because you now can read the consent-string within <amp-iframe>. I just wanted to refer to this use-case, where standard-postmessage-TCF-2.0 reading would probably be sufficient or is already in place.

Implemented with the load-cookie-with-consent.html file described in https://docs.prebid.org/dev-docs/show-prebid-ads-on-amp-pages.html#user-sync


Docs: h! macro page is missing (404)

Go to the windows-rs docs: https://microsoft.github.io/windows-docs-rs/doc/windows/index.html#macros
Click on the h macro, and get the 404 File Not Found page

Weird - maybe @riverar knows what happened to it?

Strange, will take a look. cargo didn't generate the page.

This seems to work:
cargo doc -p windows --open

yep -- I confirmed cargo doc works fine, but the published github.io pages are missing it.

The docs are generated via cargo doc -p windows --all-features --no-deps --target-dir ... and the h! page doesn't get generated. Might be some sort of bug with the use of all-features or no-deps?

Woot, thanks for the fix Rafael!


krib: install fails, api-server, scheduler and controller manager can't start

Installation of Kubernetes fails when waiting for its base service can't start. Looking in krib-kubeadm.cfg.tmpl it provided extra volumes to controllerManager, scheduler and apiServer.
Krib content-package version is v1.12.0-tip-151-31915dd7430d7119be359e4d684cdab30afda538.
  extraVolumes:
    - name: hyperkube
      hostPath: /k8s/hyperkube
      mountPath: /hyperkube

These three containers fail to start. By removing the extra volumes from the kubeadm config all three services start up as expected.
Are these mount of an external hyperkube binary used in the CentOS installation or just something left when testing?

Closed by #228


installation failed

I'm trying to install ionic  using
npm install -g ionic
but I get this error:

Greetings @elhamsarikhani!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

you must  to  download node.js and cordova first.

Thank you I installed node.js and now it's fine : ]

You're welcome :)

This issue was moved to driftyco/ionic-cli#627


npm install not working with capacitor 5

Hello,
i can't install this plugin with capacitor 5
npm i is installing the v1 version

Sorry, just published 2.0.0


surveys: need to add continuous integration

after writing functions for validation of survey.tsv file in expfactory-python https://github.com/expfactory/expfactory-python/issues/82

Done - since we don't have changes for all surveys, I tested them locally.


Update ingestion process to adapt the new schema

Currently our schema of the extracted data has been changed due to #76. We have a flattened data structure where we store the output_path of the derivation in build_inputs instead of the package itself.
We need to update the ingestion process to adapt the new schema.

Fixed in #82


feat: implement a neo-tree source

I think it could be cool to implement a neo-tree source so you could potentially have a side bar view of all your current tags.
This is not important nor do i believe it to be an imperative feature to be implemented, Just thought it could be an interesting idea so I'm creating this just to leave this as a note.
https://github.com/nvim-neo-tree/example-source

This could be an interesting integration to include. I don't think this will be an priority for now, but would love to look into this later.

I've implemented something that might achieve a similar result, where all your tags sit next to your working buffer. That is, sending scoped tags to the quickfix list. This can also be done directly from the popup menu.
I have found this to be superbly useful in my day-to-day workflow.
Neo-tree integration isn't still off the table, but I thought I'd bring this up as the current alternative.


Brainstorm about uncertainty for semantic segmentation

@Dref360
Really appreciate all the nice work you and your co-developers did on this versatile software library!
I have a question about the uncertainty sampling for semantic segmentation. I've integrated another semantic segmentation network with BAAL, and by stepping through the code blocks, I found something that raised a question:
Assuming the output-shape of the tensor after the Monte-Carlo dropout sampling is:
(1, 19, 1024, 1024, 15)
(batch_size, number_of_classes, image_width, image_height, monte_carlo_iterations)
For what reason does the function singlepass average across the Monte-Carlo samplings: https://github.com/baal-org/baal/blob/e0db2c8c2ede0702eb137a3e2692dc3f07b3522a/baal/active/heuristics/heuristics.py#L46
I'm using the Entropy heuristic and the tensor after singlepass goes to:
(1, 19, 1024, 1024)
Wouldn't be better to keep the output of the Monte-Carlo iterations and track whether the output value after softmax (basically the class label) would change across the 15 iterations? In my opinion an uncertain image would be where the class labels would change across the Monte-Carlo iterations, I guess I would be less interested in measuring the average change in entropy values.
I just wanna brainstorm about what would makes sense as an uncertainty metric for semantic segmentation... (and whether I should build this, or just re-use a default heuristic from BAAL if your experience with that heuristic is better in the context of semantic segmentation)

Hello,
I think what you are describing is BALD, Houlsby et al.. It measures the disagreement between predictions.
We use singlepass on Entropy to create a Bayesian Model Average, this average is often better than a single prediction.
It is a common baseline in industry and in academia.
In practice, BALD is (almost) always better than Entropy

Great! Let us know how it goes please :)
Do you have an idea on how we could make this information more available on our documentation? This muat surely be a common question


[Avatar] Opacity is processed at NaN

Description
The opacity code available at https://github.com/adobe/spectrum-css/blob/master/components/avatar/skin.css#L14-L18 renders to opacity: NaN; in the published package.
Steps to reproduce

yarn add @spectrum-css/avatar
cd node_modules/@spectrum-css/avatar/dist
vim index-vars.css

Expected behavior

Screenshots

Environment

Spectrum CSS version:
Browser(s) and OS(s):

Additional context

Oh lawd. Nice catch, looks like a PostCSS function issue. Will investigate.


Java sharding tutorial instructions require updating

Couple of micro changes needed to the Java sharding tutorial:

the @Package annotation is missing the publish=true field
the jarFile value in the @Package annotation does not match the Jar created by the Maven POM

Thanks if you have a PR, I'd love to see it (otherwise I'll update it myself)

Hi @brendandburns, no need to do it yourself as PR #113 has already been merged.

Closing, since the PR w/ the fix is merged.


[Grid] Improve support for nested grid

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
The nested grid container is not full width, there is unnecessary space to the right.
Expected Behavior 🤔
The nested grid container is full width with no unnecessary space.
Live Example 🕹

https://codesandbox.io/s/material-nested-grid-bug-lyeg5?file=/demo.tsx
Your Environment 🌎

Tech
Version

Material-UI
v4.11.0 (latest)

React
v17.0.1

Browser
Google Chrome v86.0.4240.198 (Official Build) (64-bit)

TypeScript
v4.0.5

<Grid item container> doesn't always work, especially with spacing. You can create an intermediary DOM node: https://codesandbox.io/s/material-nested-grid-bug-forked-lbbq5?file=/demo.tsx
import React from "react";
import { makeStyles, createStyles, Theme } from "@material-ui/core/styles";
import Paper from "@material-ui/core/Paper";
import Grid from "@material-ui/core/Grid";

const useStyles = makeStyles((theme: Theme) =>
  createStyles({
    paperCenter: {
      padding: theme.spacing(1),
      textAlign: "center"
    },
    paperRight: {
      padding: theme.spacing(1),
      textAlign: "right"
    }
  })
);

export default function NestedGrid() {
  const classes = useStyles();

  return (

      <Grid container spacing={2}>
        <Grid item xs={12}>
          <Paper className={classes.paperRight}>Unnecessary space ↓</Paper>
        </Grid>
        <Grid item xs={12}>
          <Grid container spacing={2}>
            <Grid item xs={4}>
              <Paper className={classes.paperCenter}>item</Paper>
            </Grid>
            <Grid item xs={4}>
              <Paper className={classes.paperCenter}>item</Paper>
            </Grid>
            <Grid item xs={4}>
              <Paper className={classes.paperCenter}>item</Paper>
            </Grid>
          </Grid>
        </Grid>
        <Grid item xs={12}>
          <Paper className={classes.paperRight}>Unnecessary space ↑</Paper>
        </Grid>
      </Grid>

  );
}

However, it doesn't seem to work well either in https://next.material-ui.com/components/grid/#nested-grid.

Hopefully it will be fixed in v5.

My bad, I have overlooked the issue. It's actually the demo of the documentation that is wrong. We have had this visual regression test for it, for 3 years, it works:

I would suggest this diff in the documentation:
diff --git a/docs/src/pages/components/grid/NestedGrid.tsx b/docs/src/pages/components/grid/NestedGrid.tsx
index 864ed53077..356ddaa65e 100644
--- a/docs/src/pages/components/grid/NestedGrid.tsx
+++ b/docs/src/pages/components/grid/NestedGrid.tsx
@@ -38,13 +38,13 @@ export default function NestedGrid() {
   return (

       <Grid container spacing={1}>
-        <Grid container item xs={12} spacing={3}>
+        <Grid container item spacing={3}>
           <FormRow />
         </Grid>
-        <Grid container item xs={12} spacing={3}>
+        <Grid container item spacing={3}>
           <FormRow />
         </Grid>
-        <Grid container item xs={12} spacing={3}>
+        <Grid container item spacing={3}>
           <FormRow />
         </Grid>
       </Grid>

https://codesandbox.io/s/material-nested-grid-bug-forked-w8s76?file=/demo.tsx
@buitrbao222 do you want to work on a pull request? :)

Kinda surprised that more people haven't run into this issue. This my first time using Material UI and I've already hit this.
My issue was really difficult to even identify what I was doing wrong. In my codesandbox, change both sm={6}s to just sm. I thought to myself "surely these are the same thing in this context", but turns out not.
IMO <Grid container item /> it should at least be removed from the docs since it creates unwanted bugs depending on the combination of props passed.

@k-funk What do you think about this change in the documentation? I have seen many developers successfully use it. Flexbox were designed to support it.
diff --git a/docs/src/pages/components/grid/grid.md b/docs/src/pages/components/grid/grid.md
index 716d47bb89..76b90a459e 100644
--- a/docs/src/pages/components/grid/grid.md
+++ b/docs/src/pages/components/grid/grid.md
@@ -89,6 +89,12 @@ https://www.w3.org/TR/css-flexbox-1/#box-model

 {{"demo": "pages/components/grid/NestedGrid.js", "bg": true}}

+❌ Defining an explicit width to a Grid element that is flex container and item at the same time can lead to unexpected behavior, avoid doing it:
+
+```jsx
+<Grid container item xs={12}>
+```
+
 ## Limitations

 ### Negative margin

Actually, we can even add a runtime warning and document the limitation under http://next.material-ui.com/components/grid/#limitations.

@k-funk What do you think about this change in the documentation? I have seen many developers successfully use it. Flexbox was designed to support it.
diff --git a/docs/src/pages/components/grid/grid.md b/docs/src/pages/components/grid/grid.md
index 716d47bb89..76b90a459e 100644
--- a/docs/src/pages/components/grid/grid.md
+++ b/docs/src/pages/components/grid/grid.md
@@ -89,6 +89,12 @@ https://www.w3.org/TR/css-flexbox-1/#box-model

 {{"demo": "pages/components/grid/NestedGrid.js", "bg": true}}

+❌ Defining an explicit width to a Grid element that is flex container, flex item and has spacing at the same time lead to unexpected behavior, avoid doing it:
+
+```jsx
+<Grid spacing={1} container item xs={12}>
+```
+
+If you need to do such, remove one of the props.

 ## Limitations

 ### Negative margin

Actually, we can even add a runtime warning and document the limitation under http://next.material-ui.com/components/grid/#limitations.

Maybe both?
For myself (and I assume a lot of others), I don't read the docs in their entirety when first trying out a new library. I'm first trying to discover "will this work for me? is it easy to work with? do the docs describe real-work problems?". I try out the pieces that I need, so I appreciate when caveats are noted inline or next to the same line as the first example usage, such as:
<Grid container spacing={1}>
  // DON'T  include explicit widths in combination to a Grid element that is flex container. See "Limitations" section.
  <Grid container item xs spacing={3}>
    <FormRow />
  </Grid>
  <Grid container item xs spacing={3}>
    <FormRow />
  </Grid>
  <Grid container item xs spacing={3}>
    <FormRow />
  </Grid>
</Grid>

Additionally, typescript should be able to prevent people from doing this in the first place, which might guide people back to the docs.

My bad, I have overlooked the issue. It's actually the demo of the documentation that is wrong. We have had this visual regression test for it, it has been here for 3 years, it works:

I would suggest this diff in the documentation:
diff --git a/docs/src/pages/components/grid/NestedGrid.tsx b/docs/src/pages/components/grid/NestedGrid.tsx
index 864ed53077..356ddaa65e 100644
--- a/docs/src/pages/components/grid/NestedGrid.tsx
+++ b/docs/src/pages/components/grid/NestedGrid.tsx
@@ -38,13 +38,13 @@ export default function NestedGrid() {
   return (

       <Grid container spacing={1}>
-        <Grid container item xs={12} spacing={3}>
+        <Grid container item spacing={3}>
           <FormRow />
         </Grid>
-        <Grid container item xs={12} spacing={3}>
+        <Grid container item spacing={3}>
           <FormRow />
         </Grid>
-        <Grid container item xs={12} spacing={3}>
+        <Grid container item spacing={3}>
           <FormRow />
         </Grid>
       </Grid>

https://codesandbox.io/s/material-nested-grid-bug-forked-w8s76?file=/demo.tsx
@buitrbao222 do you want to work on a pull request? :)

If you don't mind me asking, how would I make a pull request? I've never contributed to an open source project before. I thought I could fork the codesandbox example then make a pull request from it but it is not linked to any GitHub repository.


Support for SVG namespace

I have added a comment to #1671 but I think this deserves a new issue. AngularDart doesn't currently support using components inside an SVG because the elements are inserted with the wrong namespace. This is possible in the Typescript version of Angular by prefixing the SVG elements with the namespace e.g. <svg:circle /> or by using a .svg file as the component template. I hope this will be supported in AngularDart soon.

Just a note that one of the blockers for this is the direct use of HtmlElement for a component's root element:
https://github.com/dart-lang/angular/blob/706c1b634443cfe2ba60b6ab37e2a536078ed00a/angular/lib/src/core/linker/views/component_view.dart#L44-L45
To support SVG, we'd need to relax this to Element since an SVG element is not an HtmlElement. Last time this came up, there were performance concerns about doing this:

SvgElement is not an HtmlElement, so there are a lot of code paths that could turn polymorphic since we wouldn't be able to type them strictly HtmlElement.


Always return 0 balance

i try to use bitcoinLib with rpc.blockchain.info , it's always return 0 as balance

Can you point us to the exact URL of the REST API?

rpc.blockchain.info, is an rpc endpoint from blockchain.info, i can't afford downloading the full block to my system, so i need to use an rpc to a public node

I only see https://blockchain.info/api. From there, where should I go in order to see the API docs for it?

https://blockchain.info/es/api/json_rpc_api
On Tue, Mar 6, 2018 at 9:35 AM, Nikos Baxevanis<EMAIL_ADDRESS>wrote:

I only see https://blockchain.info/api. From there, where should I go in
order to see the API docs for it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GeorgeKimionis/BitcoinLib/issues/72#issuecomment-370703935,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANlYrSiouauTwclo_CLznBopTf7LvfUpks5tbko3gaJpZM4SbOl2
.

have you look it up?

Closing this as non-reproducible. Feel free to reopen it, add comments to it, but most importantly, provide some examples to it.