Multi-region HA using regional managed instance group

Right now the HA example uses multiple copies of the module which creates a zonal instance group. This could be improved by using a regional manage instance group. I am running into this issue during the creation of modules which would want to use a list of zones. (They create other regional managed instance groups which use the NAT gateways.)
Is there any issue with adding inputs zonal and distribution_policy_zones like terraform-google-managed-instance-group has?
How do we maintain backwards compatibility and ensure that we create zonal target tags when we may not know the zones? It may not be possible to determine the zones for the firewall rules if the distribution_policy_zones are not explicitly set. Even if we could determine the zones chosen at deploy time, do we have a guarantee that they will not change? (When a zone goes down or during a rolling update for example.)
Right now without zone set the google_compute_route is created with the default zone of the region. What needs to happen networking wise in order to make this new regional config be HA? Is just making one routing rule per region sufficient? What about instances in other zones? What if the instance in the default zone fails? What if only non default zones are chosen? (By GCP or manually with distribution_policy_zones.)
Is there some other common use case that breaks this idea? Like do people use separate subnetworks for different zones or something?
Sorry for my ignorance here. I would appreciate some help investigating this because there are a lot of unknown unknowns for me. If we can figure out a solution to zone discovery I'm happy to make a PR for this.

@madmod So, I'm not sure you understand a few things about how this works, so let me explain (I've learned this the hard way).

First, let me start with telling you, the ha-nat-gateway example is fully redundant, resilient, and highly available.  At any time you can go kill all except one, or two of those three instances, and the instance tag plus priority-based routing will ensure that your traffic outbound never stops functioning.  I ran a few soak tests from internal to external and saw zero downtime through random scenarios of killing these instances, or downing their network interface, etc.  Please do your own testing if desired, of course one thing it won't do, it automatically fail over a TCP connection to a secondary host if you're streaming data, as it's not a smart multi-homed border gateway or anything.  You'll need to implement a re-try in your application stack.

If that point alone wasn't enough, by-design there's a reason this module does NOT use the regional managed instance group.  Primarily because each managed instance group uses ONE single fixed static public IP address.  Using a regional managed instance group, you couldn't do this (to my knowledge).  This is also by design, as this allows you to get a known and static list of your NAT IP addresses which you can use for whitelisting purposes.  With a regional deployment, and/or with trying to "autoscale" the NAT, you would lose the static IP nature of this.

With scaling in mind, provided you have a HA deployment (more than one) you can one-at-a-time change the instance type of each of the three NAT gateways.  In Google, changing a managed instance template actually triggers an automatic re-rollout of the new machine type (which is unlike AWS when you change an autoscaling config you have to rollout yourself).  So, all you do is chance one machine type at a time and you can scale up vertically indefinitely.  In my tests, this happens in roughly 1 minute.  On that same token, if you suddenly want more redundancy, you can at any time create a new (fourth, fifth, sixth) NAT gateway module, creating another redundancy and technically adding some scale as well (as new connections outbound would round-robin between similarly prioritized routes).

I hope that makes sense.  I think your topic was raised with the intention to make this "HA" but it's already HA.
Let me then tackle one of your other ideas... which I've already had myself.  Specifically the one mentioning zone auto-discovery and auto-HA nat.  So, I had this same thought... but this wouldn't be possible before literally a week ago when I got a feature merged into this codebase to "disable" this entire module in a variable.  Leveraging that feature, I would suggest creating a new module for this, call it a terraform-google-ha-nat-gateway module that automatically includes 6 copies of this module (6 I think is the current max number of zones in a region), and detects from your current VPC what zones are being used and then deploys a NAT gateway in every one automatically, and then using the module disable on the rest that are unused.  This could be quite handy, might make for a neat module full of modules.  It would be fairly simple to implement, it's something I started trying to conceive, but I realized I needed this module to be disabled to do so, so I focused on that first.  Now that that is upstream, it's now possible to do this.  Feel free!

@andrewfarley Thanks for your top notch reply! That all makes sense and answers my questions. I will implement your suggestion of creating another module which contains multiple copies using your new flag to turn them on and off.

@madmod Please share with me/us when you make this module!  :)  I'd fork/use it  :)

@AndrewFarley what about the case where you have to use a specific IP address and want fail-over across zones?  ie: Run single NAT gateway for IP address X in zone A, but if zone A goes down, come up in zone B with same IP (and routes all still working).


Reduce size of resulting docker image

I refactored the Dockerfile. The problem in your file is that some operations download files and following layers removed the files (after extracting). The problem is that the download creates an own layer which will be part of the whole chain and the download will always waste your space even if you don't see it. See docker history your-image-name for this.
This is kind-of-a-bug in docker. One of the feature requests speaking about this is https://github.com/docker/docker/issues/12169.
Anyway: I transformed your Dockerfile so that all commands are in one single RUN command. The result is a 6.778 GB image with much less layers which doesn't need any modification on your docker installation (AUFS)

Awesome work! thanks


[BEAM-10499] Adds a descriptive toString to SamzaRunner KeyedTimerData

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @xinyuiscool

Change LGTM. Could you rebase?

@aaltay Done.

Merging. @kennknowles @xinyuiscool - if you have additional comments please fix it with a new pr.


Need to format code

The code lacks space in GitFxApp.java, GitFxController.java, GitFxGsonUtil.java and others. Need "Code Indentation"

Not sure what you are reporting here. Can you elaborate. Is it the out of memory issues you are trying to indicate here?

I mean to say that we need "Code Indentation". Sorry for the confusion.

Please go ahead, fix it and raise a pull request.


New version: Git.Git version <IP_ADDRESS>

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 1091
Microsoft Reviewers: Open in CodeFlow

Marking it as draft since it is needed for a new feature in the automation (till 20th May).


*: remove unnecessary configs

Notify and message per tick are not used in Server, we can remove them.
PTAL @BusyJay @overvenus

LGTM


Colorscheme shouldn't override user status-position

I've got my statusline along the top of my screen, I don't think forcing it to the bottom is in the scope of a theme.

Makes absolute sense, I've also thought about only providing layout- and non-color related status options when the @nord_tmux_show_status_content is not set to 0.


Implements append() for Index/MultiIndex

https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.Index.append.html#pandas.Index.append
>>> kidx = ks.Index([10, 5, 0, 5, 10, 5, 0, 10])
>>> kidx
Int64Index([10, 5, 0, 5, 10, 5, 0, 10], dtype='int64')

>>> kidx.append(kidx)
Int64Index([10, 5, 0, 5, 10, 5, 0, 10, 10, 5, 0, 5, 10, 5, 0, 10], dtype='int64')

Support for MiltiIndex
>>> kidx = ks.MultiIndex.from_tuples([('a', 'x'), ('b', 'y')])
>>> kidx  # doctest: +SKIP
MultiIndex([('a', 'x'),
            ('b', 'y')],
           )

>>> kidx.append(kidx)  # doctest: +SKIP
MultiIndex([('a', 'x'),
            ('b', 'y'),
            ('a', 'x'),
            ('b', 'y')],
           )

Codecov Report

Merging #1163 into master will decrease coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1163      +/-   ##
==========================================
- Coverage   95.21%   95.13%   -0.09%
==========================================
  Files          35       35
  Lines        7061     7067       +6
==========================================
  Hits         6723     6723
- Misses        338      344       +6

Impacted Files
Coverage Δ

databricks/koalas/missing/indexes.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/indexes.py
96.42% <100%> (-0.24%)
:arrow_down:

databricks/koalas/__init__.py
82.97% <0%> (-2.13%)
:arrow_down:

databricks/conftest.py
96% <0%> (-2%)
:arrow_down:

databricks/koalas/generic.py
95.85% <0%> (-0.42%)
:arrow_down:

databricks/koalas/groupby.py
91.4% <0%> (-0.2%)
:arrow_down:

databricks/koalas/frame.py
96.95% <0%> (-0.06%)
: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 cad297b...f32a780. Read the comment docs.


Problem running sample code

I ran the sample code "Shows the contents of every playlist owned by a user:" from here:
http://spotipy.readthedocs.io/en/latest/#examples
and got this Terminal sequence:
$ python /Users/me/Documents/test/python/hello_spotipy.py myuseracount

            User authentication requires interaction with your
            web browser. Once you enter your credentials and
            give authorization, you will be redirected to
            a url.  Paste that url you were directed to to
            complete the authorization.

Opening https://accounts.spotify.com/authorize?redirect_uri=<some url> in your browser

Enter the URL you were redirected to:

I've white-listed my redirect_uri, exported id/secret/redirect_uri (double-checked that all changes are in effect), and ran the code as is. But the URL opened in my browser says:
Error
Oops! Something went wrong, please try again or check out our help area.

The same happened after I removed the "remembered" Spotify credential from my browsers (Safari, Chrome).

It's ok to get an error message when the browser opens. What you need to get is the URL itself, copy it and paste it into your terminal.
I'm closing as this issue is old and has been working for others. Feel free to reopen if needed. Cheers


Using entropy_batch_mixing in auto_tune_scvi_model raises TypeError: ufunc 'isnan' not supported for the input types

When I use auto_tune_scvi_model with entropy_batch_mixing as metric_name I get the below error. This does not happen when I use marginal_ll. May be related to https://github.com/YosefLab/scVI/issues/610#issuecomment-595613470. I have 13 batches.
In my log I did not get https://github.com/YosefLab/scVI/blob/54faf80ebadf9294b246b845ac1835278f32a953/scvi/inference/posterior_utils.py#L132 It would have been nice if this was enabled for more batches.
Code: (I did not add full code, e.g. leaving out path variable, imports, data preparation,  ...)
test_mode=False
metric_name="entropy_batch_mixing"

def if_not_test_else(x, y):
    if not test_mode:
        return x
    else:
        return y

n_epochs = if_not_test_else(int(400*20000/data.nb_cells), 1)
max_evals = if_not_test_else(100, 1)
reserve_timeout = if_not_test_else(180, 5)
fmin_timeout = if_not_test_else(300, 10)
use_cuda = True

space = {
            "model_tunable_kwargs": {
                "n_latent": 5 + hp.randint("n_latent", 16),  # [5, 20]
                "n_hidden": hp.choice("n_hidden", [64, 128, 256]),
                "n_layers": 1 + hp.randint("n_layers", 5),
                "dropout_rate": hp.choice("dropout_rate", [0.1, 0.3, 0.5, 0.7]),
                "reconstruction_loss": hp.choice("reconstruction_loss", ["zinb", "nb"]),
                "dispersion": hp.choice("dispersion", ["gene", "gene-batch"])
            },
            "train_func_tunable_kwargs": {
                "lr": hp.choice("lr", [0.01, 0.005, 0.001, 0.0005, 0.0001])
            },
        }

best_trainer, trials = auto_tune_scvi_model(
    gene_dataset=data,
    metric_name=metric_name,
    trainer_specific_kwargs ={'train_size':0.90,'use_cuda':use_cuda,
                              'n_epochs_kl_warmup':None,'n_iter_kl_warmup':int(128*5000/400)},
    train_func_specific_kwargs={"n_epochs": n_epochs},
    space=space,
    max_evals=max_evals,
    save_path=path_out_run,
    use_batches=True,
    # MongoDB is not availiable - cant use paralel
    parallel=False,
    #n_workers_per_gpu=4,
    reserve_timeout=reserve_timeout,
    fmin_timeout=fmin_timeout,
    exp_key="ref",
)

Error
[2020-09-04 08:35:28,380] INFO - scvi.inference.autotune.all | Cleaning up
[2020-09-04 08:35:28,380] DEBUG - scvi.inference.autotune.all | Cleaning up: closing files.
[2020-09-04 08:35:28,380] DEBUG - scvi.inference.autotune.all | Cleaning up: closing queues.
[2020-09-04 08:35:28,380] DEBUG - scvi.inference.autotune.all | Cleaning up: setting cleanup_event and joining threads.
[2020-09-04 08:35:28,380] DEBUG - scvi.inference.autotune.all | Cleaning up: terminating processes.
[2020-09-04 08:35:28,380] DEBUG - scvi.inference.autotune.all | Cleaning up: removing added logging handler.
[2020-09-04 08:35:28,380] DEBUG - scvi.inference.autotune.all | Cleaning up: removing autotune FileHandler.
Traceback (most recent call last):
  File "/storage/groups/ml01/workspace/karin.hrovatin//code/diabetes_analysis/integration/scVI/1_run_ref_hyperopt_script.py", line 88, in <module>
    best_trainer, trials = auto_tune_scvi_model(
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/scvi/inference/autotune.py", line 145, in decorated
    return func(*args, **kwargs)
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/scvi/inference/autotune.py", line 478, in auto_tune_scvi_model
    _ = fmin(
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/hyperopt/fmin.py", line 379, in fmin
    return trials.fmin(
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/hyperopt/base.py", line 630, in fmin
    return fmin(
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/hyperopt/fmin.py", line 407, in fmin
    rval.exhaust()
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/hyperopt/fmin.py", line 262, in exhaust
    self.run(self.max_evals - n_done, block_until_done=self.asynchronous)
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/hyperopt/fmin.py", line 227, in run
    self.serial_evaluate()
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/hyperopt/fmin.py", line 141, in serial_evaluate
    result = self.domain.evaluate(spec, ctrl)
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/hyperopt/base.py", line 844, in evaluate
    rval = self.fn(pyll_rval)
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/scvi/inference/autotune.py", line 167, in decorated
    return func(*args, **kwargs)
  File "/home/icb/karin.hrovatin/miniconda3/envs/rpy2_3/lib/python3.8/site-packages/scvi/inference/autotune.py", line 1439, in _objective_function
    if np.isnan(loss):
TypeError: ufunc 'isnan' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

Versions:

0.6.6

Hello,
Indeed, the entropy batch mixing is not implemented for more than 2 batches. I would also check that the number of batches is properly recorded in the dataset object.
Finally, we are actually removing the ability to use this metric in 1.0, though you should be able to write a custom hyperopt objective, which is explained in our tutorial.

Why are you removing this metric? Is it not good?

So it's not clear that optimizing for good mixing will lead to good preservation of the underlying biology, as the mixing can be optimized by randomly projecting the data. This is why we are removing it; that said, it is still possible to write a custom hyperopt objective doing this.

Thank you.


Use bool type for 'allowed' field in authorization response

What this PR does / why we need it:
To fix the bug in keystone webhook authorization
Which issue this PR fixes:
fixes #56
Special notes for your reviewer:
The k8s authorizer is expecting a bool type value from WebhookAuthorizer response.
Release note:

NONE

/ok-to-test

/assign @Fedosin

/approve

/approve

/lgtm


How to select just Segresnet3D as the network

Hi,
I want to setting only use 'Segresnet3d' as the defualt model instead of all models. Do you know how to set it in the alogs below ?
bundle_generator = BundleGen(
algo_path=result_dir,
algos=used_algorithms,
data_stats_filename=datastats_file,
data_src_cfg_name=input,
)

This is a part of Auto3Dseg module code

Hi @baoqiangmaUMCG , sorry this isn't supported until this PR is approved.
But it isn't impossible to achieve what you want. You need to install the monai in the edit mode with source code. Then find bundle_gen.py and remove all the algorithms other than segresnet.

I think it's been fixed by https://github.com/Project-MONAI/MONAI/commit/aef2ecccd4b4d0a663fb8a256ee36d8b00bdde10

If I understand your question, you can simply use:
AutoRunner(input=input_config, algos = 'segresnet')
#runner.set_num_fold(1)
runner.run()


Options to disable flit.nvim when recording macros?

flit.nvim  enchances the f t command by labeling mutiple matches to jump to. But when I'm recording a macro, if I use f or t command in the macro, sometimes it will mess up the macro behavior and jump to somewhere i didn't intend to. For example, I records a macro of f,h, meaning find next , and move one character left, then i apply this macro to multiple lines. But if these is a label h existing after f, the h will jump to it, making the macro totally unpredictable.

I guess you can add autocommands to unmap f/t on RecordingEnter and re-init flit on RecordingLeave.


[Bug] Handle space in heredoc token.

Closes https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/1869
ping @GrahamCampbell and @gharlan

:+1:

Some of these test cases are invalid php.

There was 1 failure:
1) Symfony\CS\Tests\Fixer\Symfony\HeredocToNowdocFixerTest::testFix with data set #8 ('<?php\n$html = <<<'HTML'\na\nHTM...HTML;\n')
Failed asserting that '
Parse error: syntax error, unexpected ',' in /home/travis/build/FriendsOfPHP/PHP-CS-Fixer/cs_fixer_tmp_Khtpgj on line 6
Errors parsing /home/travis/build/FriendsOfPHP/PHP-CS-Fixer/cs_fixer_tmp_Khtpgj
' is null.
/home/travis/build/FriendsOfPHP/PHP-CS-Fixer/Symfony/CS/Tests/Fixer/AbstractFixerTestBase.php:95
/home/travis/build/FriendsOfPHP/PHP-CS-Fixer/Symfony/CS/Tests/Fixer/Symfony/HeredocToNowdocFixerTest.php:29

I think the fixer should not remove the spaces.

I think the fixer should not remove the spaces.

Not removing the spaces results in a syntax error.

Oh, ok, didn't know that.

Not removing the spaces results in a syntax error.

it is valid: https://3v4l.org/GlPME
So, I still think we should not remove the spaces.

So, I still think we should not remove the spaces.

How come my other test file is getting broken then?

Because the fixer does not handle the spaces correctly.

How come my other test file is getting broken then?

because the opening quote was misplaced;
current wrong:
<<< "    HTML"

PR fixes to:
<<< "HTML"

suggested is:
<<<     "HTML"

input
<<<     HTML

I guess?

suggested is:
<<<     'HTML'

(single quotes)

course :)
updated, I have the feeling the replace can be done more efficient, if so maybe someone can suggest how?

I have the feeling the replace can be done more efficient

I don't think so.
👍 🚢

:+1:

Is this good to merge @keradus?

Yes, please rebase.
I got falsy impression thanks to our stalker ;)

I got falsy impression thanks to our stalker ;)

I wonder if I can get gitHub to actually delete all the mess they've made.

One one or another please rebase and put your thumb for #1903 ;)

rtm :)
dunno why the test don't run atm btw

why based on #1896 ?

I guess I messed up the rebase, wasn't a good weekend for development, lets see

:+1:

Thank you @SpacePossum.


item.abbr too wide completion window does not aligned correctly

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Announcement

[X] I have checked Breaking change announcement.

Minimal reproducible full config
local cmp = require('cmp')

local kind_icons = {
    Text = '󰦨',
    Method = '󰆧',
    Function = '󰊕',
    Constructor = '',
    Field = '󰇽',
    Variable = '󰂡',
    Class = '󰠱',
    Interface = '',
    Module = '',
    Property = '󰜢',
    Unit = '',
    Value = '󰎠',
    Enum = '',
    Keyword = '󰌋',
    Snippet = '',
    Color = '󰏘',
    File = '󰈙',
    Reference = '',
    Folder = '󰉋',
    EnumMember = '',
    Constant = '󰏿',
    Struct = '',
    Event = '',
    Operator = '󰆕',
    TypeParameter = '󰅲',
}

local formatFn = function(_, item)
    local icon = kind_icons[item.kind] or ''
    icon = ' ' .. icon .. ' '

    item.menu = ('   (' .. item.kind .. ')') or ''
    item.kind = icon

    return item
end

require('cmp').setup({
    experimental = {
        ghost_text = true,
    },
    matching = {
        disallow_fuzzy_matching = false,
    },
    completion = {
        keyword_length = 1,
        completeopt = 'menuone,noselect,noinsert,popup,fuzzy',
    },
    snippet = {
        expand = function(args)
            require('luasnip').lsp_expand(args.body) -- For `luasnip` users.
        end,
    },
    mapping = {
        ['<C-p>'] = cmp.mapping.select_prev_item({ behavior = cmp.SelectBehavior.Insert }),
        ['<C-n>'] = cmp.mapping.select_next_item({ behavior = cmp.SelectBehavior.Insert }),
        ['<C-e>'] = cmp.mapping.abort(),
        ['<C-y>'] = cmp.mapping.confirm({ select = true }),
        ['<C-d>'] = cmp.mapping.scroll_docs(3),
        ['<C-u>'] = cmp.mapping.scroll_docs(-3),

        ['<CR>'] = cmp.mapping({
            i = function(fallback)
                if cmp.visible() and cmp.get_active_entry() then
                    cmp.confirm({ select = false })
                else
                    fallback()
                end
            end,
            s = cmp.mapping.confirm({ select = true }),
        }),
    },
    formatting = {
        expandable_indicator = true,
        fields = { 'kind', 'abbr', 'menu' },
        format = formatFn,
    },
    window = {
        completion = {
            col_offset = -3,
            side_padding = 0,
            winhighlight = "Normal:CmpPmenu,CursorLine:CmpSel,Search:None",
            scrollbar = true,
            -- border = { '', '─','', '─',},
            -- border = { "", "", "", "│", "", "", "", "│" },
        },
        documentation = {
            border = {
                { "┌", "CmpDocBorder" },
                { "─", "CmpDocBorder" },
                { "┐", "CmpDocBorder" },
                { "│", "CmpDocBorder" },
                { "┘", "CmpDocBorder" },
                { "─", "CmpDocBorder" },
                { "└", "CmpDocBorder" },
                { "│", "CmpDocBorder" },
            },
            winhighlight = "Normal:CmpDoc",
        },
    },
    sources = cmp.config.sources({
        { name = 'luasnip' },
        { name = 'nvim_lsp', },
        { name = 'buffer' },
        { name = 'path' },
        { name = 'nvim_lsp_signature_help' },
    }, {
    })
})

Description
item.abbr too wide completion window does not aligned correctly

Steps to reproduce
config as above and edit a cpp file with configed clangd lsp.
Expected behavior
we can set the width of abbr field and completion window like view.document.max_width.
completion window is correctly aligned
Actual behavior
item.abbr too wide completion window does not aligned correctly
Additional context
No response

The completion windows are not aligned correctly, is this expected behavior? Or is it just something wrong with my configuration?

Hm...
The completion window seems to be starting at an incorrect position because it is too large, trying to display all the completion candidates. To prevent this, I think you need to set a maximum width for the completion window.
https://github.com/hrsh7th/nvim-cmp/issues/980

Thank you. It works for me.


Allow to run on specific namespaces with restricted permission

Describe the bug
On the cluster I used, I don't have access to all namespaces.
Even if I specify my namespaces, it seems it try to get the resources through a cluster-scoped api instead of a namespaced one.
To Reproduce
Steps to reproduce the behavior:

Make sure you don't have access to all namespaces, ex :

$ kubectl get po -A
Error from server (Forbidden): pods is forbidden: User "u-wf3je4hm2h" cannot list resource "pods" in API group "" at the cluster scope

krr simple -n my_namespace
See error

Running Robusta's KRR (Kubernetes Resource Recommender) 1.0.0
Using strategy: Simple
Using formatter: table

[ERROR] Error trying to list pods in cluster k8s-prod: (403)
Reason: Forbidden
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"deployments.apps is forbidden: User \"u-wf3je4hm2h\" cannot list resource \"deployments\" in API group \"apps\" at the cluster
scope","reason":"Forbidden","details":{"group":"apps","kind":"deployments"},"code":403}

Expected behavior
It should be able to get the data if I have access to the specified namespace
Thank for krr, it's awesome :)

Update on this one: @mnbbrown has done a great job, but he did not sign the CLA so we can not merge it
We waited quite a bit, so we will most likely create our own fix

can someone merge the PR of  @mnbbrown  please

@aantn Yes if you cant handle it it will be a pleasure thanks to try it

Hello, I am not sure this is fully fixed,
Launching it today from a clone that includes this supposed fix commit, with command
python krr.py simple --namespace cclexploit-monitoring
I get the error below that shows that cluster scope requests are still made, here it's a get services which I am allowed on kubectl if i stay in my namespace.
ApiException: (403)
Reason: Forbidden
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"services is forbidden: User \"https://xxx/dex#ecourreges\" cannot list
resource \"services\" in API group \"\" at the cluster scope","reason":"Forbidden","details":{"kind":"services"},"code":403}

I got the same issue while listing resources for Services

Encountering the same issue

Does #220 fix this?

Does #220 fix this?

It improves for sure compared to the main branch, now it does display the table of deployments for the namespace correctly, but it still tries to get HPA at cluster scope which is not allowed:
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"horizontalpodautoscalers.autoscaling is forbidden: User
                    \"https://mysite.com/dex#myuser\" cannot list resource \"horizontalpodautoscalers\" in API group \"autoscaling\" at the cluster
                    scope","reason":"Forbidden","details":{"group":"autoscaling","kind":"horizontalpodautoscalers"},"code":403}

Does #220 fix this?

It improves for sure compared to the main branch, now it does display the table of deployments for the namespace correctly, but it still tries to get HPA at cluster scope which is not allowed:
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"horizontalpodautoscalers.autoscaling is forbidden: User
                    \"https://mysite.com/dex#myuser\" cannot list resource \"horizontalpodautoscalers\" in API group \"autoscaling\" at the cluster
                    scope","reason":"Forbidden","details":{"group":"autoscaling","kind":"horizontalpodautoscalers"},"code":403}

HPA detection is still broken on https://github.com/robusta-dev/krr/pull/220 (for now) but everything else should work

Can you try the lastest commit on #220?
I've added a fix for the HPA there.
Thanks for the patience!

Can you try the lastest commit on #220? I've added a fix for the HPA there.
Thanks for the patience!

Good job, it works for me with no error now!

Wonderful, thanks.
In general, we're figuring out how to better support enterprise environments (w/ limited permissions and organizational difficulties in actually getting KRR's recommendations applied after a scan). If you're open to it, would love to chat and get feedback on related areas.


Not Working in vscode version 1.30.1

When I click on go live button in vs code it load the listing directory page but go live button still go live not changed to stop server. and when I do some changes in file and save it again then no live reload.

Same here. Can someone help?
http://myweb:5500/


Horizontal rectangle doesn't work until resizing

This plugin is amazing and it's the unique that worked nice and without errors, but when I use
cropper.MaskPainter = new RectangleMaskPainter() { CornerRadius = 10, Corners = CornersEnum.All, MaskHeight = 0.5, MaskWidth = 1 };
and the loaded image for cropping is 1:1, the crop area is raised also as a square(1:1). It only will turn into a rectangle area if I resize to the minimum as I can, suddenly it becomes the expected rectangle. If I open an vertical image, it works at first shot.
Does any one here have any idea of how to solve that?

"This plugin is amazing and it's the unique that worked nice and without errors, but when I use" - thank you.
i'll check this problem

This plugin is amazing and it's the unique that worked nice and without errors, but when I use
cropper.MaskPainter = new RectangleMaskPainter() { CornerRadius = 10, Corners = CornersEnum.All, MaskHeight = 0.5, MaskWidth = 1 };
and the loaded image for cropping is 1:1, the crop area is raised also as a square(1:1). It only will turn into a rectangle area if I resize to the minimum as I can, suddenly it becomes the expected rectangle. If I open a vertical image, it works at first shot.
Does any one here have any idea of how to solve that?

what version do you have installed?
can you tell me the size of your picture?

This plugin is amazing and it's the unique that worked nice and without errors, but when I use
cropper.MaskPainter = new RectangleMaskPainter() { CornerRadius = 10, Corners = CornersEnum.All, MaskHeight = 0.5, MaskWidth = 1 };
and the loaded image for cropping is 1:1, the crop area is raised also as a square(1:1). It only will turn into a rectangle area if I resize to the minimum as I can, suddenly it becomes the expected rectangle. If I open a vertical image, it works at first shot.
Does any one here have any idea of how to solve that?

what version do you have installed?
can you tell me the size of your picture?
Android or iOS?

Version 1.0.4 on Android.  Size: 800x800 px

Fixed in 1.0.5

Fixed in 1.0.5

That's it! Thank you, now it works!


simplify website, remove guides etc

Removes most of the old content.
@pfrazee want to add the autogen part here? We can either have an index on homepage or link to another index page - either is fine with me.
We may want to wait until there is a DEP to show =).

Okay, added the autogeneration! Pretty simple for now: view site

Nice!
On Sat, Feb 10, 2018 at 11:35 AM, Joe Hand<EMAIL_ADDRESS>wrote:

Okay, added the autogeneration! Pretty simple for now: view site
https://5a7f2cf5fd0efa67466312cd--datprotocol.netlify.com/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datprotocol/website/pull/6#issuecomment-364674497,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNhU5GOYArjnFTwBGO86clUGEdLrYrPks5tTdNEgaJpZM4Rc8ZP
.

Do we only want to display accepted DEPs, not draft? Or some way to distinguish draft vs active on the index page?

I suggest we display drafts and indicate status
On Mon, Feb 12, 2018 at 2:01 PM, Joe Hand<EMAIL_ADDRESS>wrote:

Do we only want to display accepted DEPs, not draft? Or some way to
distinguish draft vs active on the index page?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datprotocol/website/pull/6#issuecomment-365044183,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNhU6rul_daiSwT0Y5nkSXL2Tuz_JFNks5tUJiKgaJpZM4Rc8ZP
.

Added status to index:

Solid
On Mon, Feb 12, 2018 at 2:45 PM, Joe Hand<EMAIL_ADDRESS>wrote:

Added status to index:
[image: screen shot 2018-02-12 at 12 45 11 pm]
https://user-images.githubusercontent.com/684965/36118644-99f99a0e-0ff2-11e8-9c24-618b17191245.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datprotocol/website/pull/6#issuecomment-365056097,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNhU4EYIIxHXBwkYvCbZfwYDrnTpQ5sks5tUKLpgaJpZM4Rc8ZP
.


Pull new docker image

Hello there.
I am running NAV on docker.
I accomplished this by creating  a sandbox PS1 file from the Live NAV 365 business central and using the Windows ISE command to pull in that image.  Works great.
There were some updates to the BC 365 and I want to pull those in to my docker image. I found this link https://blogs.msdn.microsoft.com/freddyk/2018/04/16/which-docker-image-is-the-right-for-you/
but I don't know where to run the commands mentioned in the link.
So I would like to know what is the best way to refresh my NAV running on docker with the latest updates ?
Also question on AL
The following suggestion was made
You should make sure that you're using AL Language that comes as part of the Docker image.
Otherwise you run into the risk of incompatibilities between non-released versions.
How do I check that my AL language is in sync with the  Docker  image one ?

You can know If the AL language is sync with docker when you are able to download symbols in visual studio code.

The docker image displays a download URL for the VSIX which maches the docker image.
You need to ensure that the version number is the same. Things might work for downloading symbols and other things, but there might be things which doesn't work.

Thank you for your reply.
I am a little confused.
Where can I find the download URL on the docker image ??
Which version number do I compare to which  ?
On the AL Language extension I have version 0.16.19969.
Thanks

when you use docker run or new-navcontainer it will display the url.
You can also use docker logs  to re-display the url.

The script will output a number of things when running it.
If you didn't see it - use docker logs  afterwards and you will see the line.
Feel free to post docker logs output here and I will highlight the line.

Ok found it on the output.
Files:  http://MyBizCentralApp:8080/al-0.16.19969.vsix
so the above and my AL extension match which is great.
So my other issue was
There were some updates to the BC 365 and I want to pull those in to my docker image. I found this link https://blogs.msdn.microsoft.com/freddyk/2018/04/16/which-docker-image-is-the-right-for-you/
but I don't know where to run the commands mentioned in the link.
So I would like to know what is the best way to refresh my NAV running on docker with the latest updates to BC 365 ?

in the script above - you reference microsoft/bcsandbox:us - which references the latest public Business Central Sandbox environment and since you have -alwaysPull, you will refresh the container with the latest image every time you run the script.

OK, appreciate the responses.
The issue that I am having is that some of the things that were supposed to be fixed on a July release of BC 365 are not fixed and I was trying to figure out why I am not seeing those fixes.
One of which was the Search on a new page extension that I created. The page does not display on the search although I have it setup
UsageCategory = Lists;
and all the fields are marked as
ApplicationArea = All;

When you say July release - where did you find that info?
If that is the July Preview - then that is only available as an insider build for people with access to Microsoft Collaborate.

Yes that is what I am referring to.
I did not realize that the July Preview was only available to a certain group of people.
Since I am not in that group, how can get these updates ?
We have quite a few customers that signed up for BC 365 and I have a lot of development to do.
Sorry for bombarding you with all of these questions.
Thanks

https://blogs.msdn.microsoft.com/nav/2018/05/03/al-developer-previews-multiple-releases-and-github/
combined with the blog post from me and this:
http://aka.ms/businesscentralapps
can probably help you.

Here is where I got the info
https://github.com/Microsoft/AL/issues/2977
Towards the middle of the thread the person that answered is Stanislaw Stempin.
Sorry to bombard you with all these issues/questions but we have quite  a few customers that signed up with BC365 and I am working on several extensions.

In case you need access to insider builds, please reach out to<EMAIL_ADDRESS>with your request and they will direct you to sign the necessary documents to get early access.


Added filter support. Added color space conversion.

Added basic support for the following filters:

brightness

contrast

grayscale

hue-rotate

invert(incomplete)

opacity

saturate

sepia

Single filters work quite well overall.
Combining several filters still needs some improvement.
The idea is to draw the image on the canvas and then redraw it with the filters applied.
However, getImageData() gets all the pixels in the area, which means if border-radius has a value, the filters will also apply on the background there.

I would love to see this PR merged. Any updates ?

Could someone resolve this conflicts and merge?????

Can I request at least add basic support grayscale filter?

I think this'll need a rewrite. The library has been refactored into Typescript since this was authored and its likely the flow has changed.
https://github.com/niklasvh/html2canvas/pull/1828


The FLOPS of scalar dot products in Attention has not been considered？

When testing with the TransformerEncoderLayer, the computed FLOPS show a strictly linear relationship with the sequence length ( L ), rather than the theoretical ( L^2 ) relationship expected from Attention calculations. It seems that the computational cost of scalar dot products is not being accounted for at all.

Hello, we tried to solve the issue.
This is what we did:
Added a new function _attention_flops_compute to calculate the FLOPs for attention operations. Modified _linear_flops_compute to use this new function when appropriate. This change will properly account for the quadratic relationship between sequence length and FLOPs in transformer models.
You can review changes in this commit: https://github.com/wwqwq2313/MrYxJ-calculate-flops.pytorch-46/commit/52e1a45fef4e5d81b96affa562d3c97256ae117c.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

Latta AI seeks to solve problems in open source projects as part of its mission to support developers around the world. Learn more about our mission at https://latta.ai/ourmission . If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Add CNAME file to ensure that new publish will not remove custom domain

Adding this CNAME file to the public folder will ensure that the GitHub pages custom domain doesn't get reset on every publish run in Actions.

This is based on step 8 on this instruction page: https://docs.github.com/en/pages/configuring-a-custom-domain-for-your-github-pages-site/managing-a-custom-domain-for-your-github-pages-site


Show models sorted by papularity

It would be helpful to show models sorted by papularity in the model list.

Good idea. I made a pull request #25 . So check it out!

OK. Thanks!


Wallet will not show amount of tokens

The Molly wallet will not show my token transaction after I have not logged into it for a couple of months.
I transferred 11k DAG into my wallet and it shows 0 and it keeps timing out after 5 minutes.
Please advise because I have no where else to put my DAG and it is disheartening to try for days to get it to show my amounts and it stays at 0.

The transaction showed up once I turned off my VPN

The transaction showed up once I turned off my VPN

Please join the support channel if you require any additional help: https://t.me/mollywalletsupport

Please join the support channel if you require any additional help: https://t.me/mollywalletsupport


Add to documentation which processValues are ignored

Sometimes processValues are ignored (e.g., the ones called measurement). This is for legacy reasons. We should document them.
https://github.com/Scarjit/united-manufacturing-hub/blob/7830e9d03a7733782125e8ba879c0e4899537342/golang/cmd/mqtt-to-postgresql/processValueData.go#L207-L210

mqtt-to-postgresql is deprecated.
kafka-to-postgresql's processValue isn't ignoring any messages


Wrong @types/react version when running

when run yarn dev, such error happened:
gitpod /workspace/celo_composer_learn/packages/react-app (main) $ yarn dev
yarn run v1.22.19
$ next dev
ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000
It looks like you're trying to use TypeScript but do not have the required package(s) installed.

Please install @types/react by running:

        yarn add --dev @types/react

If you are not trying to use TypeScript, please remove the tsconfig.json file from your package root (and any TypeScript files in your pages directory).

It's because the @types/react inside package.json is not compatible, need to downgrade the pakcage.
Another error:
react-celo] The `network` prop on CeloProvider has been deprecated, use `defaultNetwork`

If needed, I'm glad to fix such two issues.

Facing same issue

Solved the first issue by downgrading the @types/react. But still facing a second issue.

@alymurtazamemon You can change the network to defaultNetwork and the value to Alfajoes (string). We will update it in the repo too.

It seems the issue has been solved. Closing it.


Mistakes in the German Terms of Service (Nutzungsbedingungen)

Links aren't clickable
"App" is capitalized at "1.
"Corona-Warn-App" sometimes isn't hyphenated ("Corona Warn App")
"Open Source-Lizenzen" should be "Open-Source-Lizenzen" (hyphenated)
Closing brackets are missing at "(transmission risk"
One link still is a placeholder "[...] und Android finden Sie unter [Link]."
There's no dot at the end of the sentences "Sie müssen immer die aktuelle Version der App verwenden", "Sie benötigen aktuelle Versionen von Android bzw. iOS", "Sie benötigen eine Datenverbindung" and "Die App muss auf dem Endgerät laufen und eingeschaltet sein"
"Open Source-Lizenzhinweise" should be "Open-Source-Lizenzhinweise" (hyphenated)
"CWA- Dienste" should be "CWA-Dienste" (no space)
"US Embargos" should be "US-Embargos" (hyphenated)

Also, in my opinion the the sub-headlines should be highlighted in some way to improve readability.
By sub-headlines, I mean things such as "Kein Anspruch auf bestimmte Funktionalität" and "Keine Verfügbarkeitszusage" under section 10.

Text, especially legal text, has to go through the team. Please report it's in #332

@Julian1203 thanks for pointing this out!
Referencing this PR in #332 (thank you @tomjschwanke!), I am closing it here since our text and translation team has to verify the changes first and have them approved as well.
Best,
Marc


Support for rollups on Account when doing Merge Account

As per discussion on Chatter....
"I was testing this out by counting attachments on accounts which worked well.  My question is if there is a way to trigger a recalculation upon merge?  I have a lot of accounts that get merged daily, but the count didn't update after I did a test merge."
"HI @Ben Patterson (Pentair Aquatic Systems) I seem to recall a help document that says that merge operations don't fire triggers. Here is the article: https://developer.salesforce.com/docs/atlas.en-us.apexcode.meta/apexcode/apex_triggers_merge_statements.htm. It seems that a before delete or after delete action would fire the trigger, but I'm not sure how @Andrew Fawcett (FinancialForce.com) has the triggers set up in Declarative Lookup Rollup Summaries Tool (e.g., can the tool capture a before delete action)."
"Hmm, it says no triggers for the related records are fired. That being said, we might be stuck."
"Thanks @[Kent Manning] yeah was just reading that as well, i think though this applies in the context of DLRS to the Parent object, where there is not a Apex Trigger deployed, thats only on the child. I'd need to do some testing to see whats going on here, as reparenting child records should work, i'm wondering if Salesforce is not firing the Apex Trigger on the child in this case of merge, i have some vague recollection this might be the case. Thats not to say we are dead in the water here, i'm up for considering updating the tool to deploy a trigger on the parent to support merge."

Update: I have reproduced this in an Apex Unit test and am looking at a way to resolve.

Just curious if there has been any progress on this - we're running into issues with several fields being out of date due to merges.

Nothing as yet sorry @jongpie. It is marked as a priority enhancement, so when i or another contributor gets a burst of time it will get seen ahead of others.

Ok started to explore this... its not going to be possible to get this to work realtime...
SELF_REFERENCE_FROM_TRIGGER, Object 001b000003O8CFqAAN is currently in a merge operation, therefore a trigger can not update it

Going to try this with Schedule Mode rollups...

Ok, i think i have found a solution that allows the rollup to remain realtime most of the time, and switch dynamically to schedule mode in a merge scenario. This will require the RollupJob class to be scheduled, even if you don't have any Scheduled rollups.

Added in v2.4.1.

I'm running into this issue. We selected the "allow merging" box when we created it.

@ChristiHarris can you please open a new issue and fill in the steps to reproduce etc. I am not quite sure from the above what it is you are saying is related to this issue and I think its best to start afresh. Thank you! 👍


Displaying tags in posts?

Pretty new to Jekyll and Github pages, but after looking around for how to add tags in posts it appears that I should be using tags in the post front matter, similar to this:
layout: post
title: My Title Goes Here
tags: tag1 tag2 tag3

However doing that in Jekyll Now doesn't seem to display the tags.
How can I display the tags a post has or create a block of tags used in the blog site, which would allow easier user navigation?
Thanks,
Marin

You may take a look at my pull request or at my fork of this repository where I implemented a full tag system where all you have to do is exactly what you just did, then the tags will show and will auto-generate pages for every tag.

@Theby Would your PR work with Github Pages, because AFAIK Github Pages does not run any Ruby code from plugins?

It does work, in fact I'm actually using it in my github page www.spoonmangames.cl.
As far as I have read in http://jekyllrb.com/docs/plugins/ you need to generate the pages and links before publishing and the tag plugin that I used does just that. You can see that it works by looking at my blog (sorry it's in spanish but you can click the blog tab, there are a few post)

I checked and you were rigth, I didn't test it live, but what you can do is use the plugin and then copu the _site/tag/ folder outside of _site, everything should work then

@Theby generating the tags upfront and moving _site/tag in the root folder works fine, thanks for that!
However I've noticed when tag index pages are generated the markdown text is not rendered in the resulting pages. Have an idea about this issue?
Here's an example of what I'm referring to:

http://dnaeon.github.io/tag/vpn/

As you can see the Markdown text is not rendered when displaying "tag index" pages.

I'm gonna check on that, It's happening to me too.

A couple of other folks have implemented tags in the past. Check 'em out:

https://github.com/barryclark/jekyll-now/pull/130
https://github.com/barryclark/jekyll-now/pull/186

Cheers,
Barry


Update VIDT contract address in response to KuCoin hack

Please update our contract address as we are preparing a token swap to all holders.
Correct addres is https://etherscan.io/address/0xfef4185594457050cc9c23980d301908fe057bb1.
For evidence ...

https://twitter.com/VIDT_Datalink/status/1311709500404006913
The new contract was deployed by the same account (0x95cc85867d633db0c0abd05f6cff7d45f7ece315)
The new smart contract address is mentioned here https://about.v-id.org/token/
The old contract has been paused and will not function anymore
New tokens will be reissued 1:1 and automatically distributed to the holders of the previous token (to be completed in 4 days)

Added the new BEP20 version of VIDT also ...
Thank you for your time!

Hi @Sjazzz this repository has limitation to not confuse users that in a map can be only 1 token with a given symbol - please update this PR thanks

Thanks xrn, updated contract-map.json and added a distinguishing logo.
Thanks!

@Sjazzz thanks for the updates - as you see tests are failing because contract addresses have to follow EIP-55. Can you fix it, please?

All checks passed now, cheers!

@Sjazzz - thanks I am reviewing it now and seems that there is no case to add the BEP version of the token to this repository. MetaMask is supporting only Ethereum powered tokens. Please remove it because in the Ethereum network this address do not have activity - https://etherscan.io/address/0x3f515f0a8e93F2E2f891ceeB3Db4e62e202d7110

Removed the BEP20 version

https://hotbit.zendesk.com/hc/en-us/articles/360057510593-Announcement-Regarding-Completing-VIDT-Contract-Address-Upgrade-and-Opening-of-Withdrawal-and-Deposit-Functions-of-VIDT-on-Hotbit
https://github.com/V-ID/V-ID-Token


Nodes not drainable with new UI

Rancher Server Setup

Rancher version: 2.6.0
Installation option (Docker install/Helm Chart): docker

Information about the Cluster

Kubernetes version: 1.21.2
Cluster Type (Local/Downstream): AKS

Describe the bug
The new UI doesn't send a request to drain a node. The dialog is shown and then nothing is sent.
To Reproduce

View nodes
Select a node to drain
Set whatever you want in the dialog
Confirm

Result
No request is sent, node is not drained.
Expected Result
Request is sent and the node is drained (as the legacy UI does).

This is apparently fixed from 2.6.2 upwards.


Is IPv6 rotation supported?

A website I want to proxy will be blocked if I access a fixed address multiple times. I want to add a large number of IPv6 to the VPS and then run SniProxy. Does SniProxy support ipv6 rotation? Does it automatically select an ipv6 address every time it is used as a proxy?

both ipv4 and ipv6 rotation is something I would like to take a look at soon.

https://github.com/oliverkirk-sudo/ipv6-pool-tool-socks
Here is a repository for your reference. This is a random egress using the address prefix bound to the network card to provide an ipv6 proxy pool.

the attached repo doesn't work at scale. I'm working on providing upstream ipv6 support. After that, source IP rotation will be added based on a given subnet. There will be some OS level issues that needs to be solved before it can work.

this should be supported in the latest version.


fixing ESP32 demo

https://github.com/littlevgl/lv_micropython
The example is missing
import ILI9341 as ili
import lvgl as lv
in the esp section

Fixed.
Thank you!


Missing LICENSE file

Hi, I'm packaging this project for the Arch Linux User Repository (AUR). Please consider adding a LICENSE file to specify what this project is licensed under. I see it's listed as MIT or Apache in the Cargo.toml, but a license file helps clarify on GitHub.
Thanks for your time!

added


After updating RaspberryMatic <IP_ADDRESS>40316 GUI does not report statuses

Describe the issue you are experiencing
After updating RaspberryMatic <IP_ADDRESS>40316 the if you go to the 'Status and control' -> Devices menu, select an actuator and click on the On/Off buttons, the actuator is works well, but the GUI buttons remain Off state, and the 'Last modified' fields remain empty. 'Last modified' fields of sensor devices also remain empty but the data are actualised.
Describe the behavior you expected
Status of the buttons and the 'Last modified' field are actualised.
Steps to reproduce the issue

Update to RaspberryMatic <IP_ADDRESS>40316
Go to the 'Status and control' -> Devices menu
Select an actuator
Click on the On/Off buttons
...

What is the version this bug report is based on?
<IP_ADDRESS>40316
Which base platform are you running?
rpi3 (RaspberryPi3, ARM64/aarch64)
Which HomeMatic/homematicIP radio module are you using?
RPI-RF-MOD
Anything in the logs that might be useful for us?
-

Additional information
Tested devices:
HM-LC-Sw4-DR-2
HM-LC-Sw1-Pl
HM-ES-PMSw1-Pl-DN-R1

Show a screencast please.

After updating to RaspberryMatic <IP_ADDRESS>40316 the CCU-Jack doesn't started and it generated alarms. After it has removed and CCU has rebooted, the malfunction above leaved off. So it seems that the CCU-Jack blocked something.


update be-rse logo

The gitter avatar from the be-rse could be used as the logo. Ideally this should be hosted on the be-rse.org website, but for now linked via gitter.im. This is ready for review.

You're sure that this url is stable @johanphilips ? Alternatively we can also add the image to this repository, it does not need to be an external url

You'd just need to add the image file to /assets/images and add this path as the avatar

I would +1 to just add the file to the repository, we can't be sure that it will always be available at that url!

made the change to a local asset, which is indeed safer.

thanks a lot @johanphilips :)


Pass manager refactoring

Summary
Qiskit PassManager is strongly tied to the DAGCircuit IR, and input and output type are both limited to the QuantumCircuit:
PassManager[QuantumCircuit, DAGCircuit, QuantumCircuit, BasePass]

which is a special case of
PassManager[InType, IRType, OutType, PassType]

Current limited implementation hinders us from running the pass manager framework on more sophisticated IR (e.g. DAGCircuit doesn't efficiently represent the control flow), and we need to full scratch flow controller for other program types, such as pulse schedule (ScheduleBlock). The primary purpose of this PR is to allow pulse module to use PassManager as a building block of its lowering operation:
PassManager[ScheduleBlock, PulseIR, Schedule, PulseBasePass]

We tried to take this approach in https://github.com/Qiskit/qiskit-terra/pull/4639, and we needed to duplicate the pass manager to manage the inconsistency of object types (eventually the PR was closed). Today, the introduction of the pulse gate leads to high demand for co-optimization of circuit and pulse programs. For example, RZXCalibrationBuilder is well acknowledged in the algorithms such as QAOA, but the compilation of pulse program is currently done by qiskit.pulse.transforms in less scalable way. Having the same building block in circuit and pulse module may allow us to efficiently write quantum program optimization at pulse level.
In addition, Qiskit doesn't support modern OpenPulse code generation, and it still relies on the legacy PulseQobj JSON format. This representation doesn't allow us to fully leverage the capability of hardware, due to missing frame support. Note that other SDKs, such as Amazon Braket, already support this modern format.
Eventually, we want to implement multiple compilers with new pass manager framework. For example,

Code generation for QASM3 backend

PassManager[ScheduleBlock, PulseIR, OpenPulse, PulseBasePass] -> defcal

In-memory data generation for Qiskit Dynamics simulator

PassManager[ScheduleBlock, PulseIR, Signal, PulseBasePass]

and pass manager mechanism allows us to reuse existing passes more efficiently.
Details and comments
Even if we use different IRs, input and output data, the flow controller can be independent of them. With this principle, I newly introduced qiskit.passmanager module with:

FlowController
This is moved from qiskit.transpiler. Note that flow controller is agnostic to pass type. It just controls execution of passes regardless of whatever they are. Previously the flow controller can only provide a custom iterator, but now flow control is fully delegated to the pass runner. This allows us to implement more complicated control flow, that might not be efficiently expressed by an iterator.

BasePassRunner
This is a base class of RunningPassManager. The base class newly implements a generic method _manage_flow that dispatches the pass execution logic depending on the controller type. Thus, we can complete pass control within the runner base class. This flow control logic eventually calls the _do_atomic_pass abstract method that a pass runner subclass must implement. This single pass run method is tailored to the pass type that subclass expects. This runner (subclass) also manages  the type conversion to/from the pass manager IR. This implies input and output type don't need to be consistent, as long as the subclass can provide proper conversion logic across the IR.

BasePassManager
This is a base class of PassManager. This class is responsible for the pass type and input type validation to guarantee the pass runner works with the expected IR. It also takes the responsibility for the thread management for parallerism. This class has an abstract method _create_running_passmanager, and here the pass manager subclass must generate a runner subclass instance, e.g. RunningPassManager, that can handle expected pass manager IR.

TODO

[ ] Discuss details of implementation
[ ] Write unittest for new module
[ ] Write reno

Thanks @taalexander for the assessment of the proposed approach. As you classified, currently I am going with the option 1 in which we allow different representation for circuit and pulse domain, and I think this is the minimum overhead to realize the pulse pass manager which is high priority for experiments team. In addition, we may need to implement different circuit representation in future to manage control flow more efficiently.
In fact, I tend to prefer the option2 (and the extension). I think adding .duration to the circuit operation is reasonable because dynamical decoupling and associated scheduling are in high demand for noisy quantum computers, and such schedulable circuit operation makes implementation of the dynamical decoupling pass cleaner. This Operation[duration, instruction, location] representation can be used in both circuit and pulse domain, i.e. CircuitOperation[Optional[int], CircuitInstruction, List[Qubit]] and PulseInstruction[int, PulseInstruction, List[Channel]]. I think this makes pulse-circuit co-optimization easier, and therefore we can efficiently implement pulse level optimization of circuit. However, this change is too significant and I don't think I can manage this within the single PR.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

413 of 446 (92.6%)  changed or added relevant lines in 12 files are covered.
10 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.03%) to 85.278%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/passmanager/base_pass.py
43
45
95.56%

qiskit/passmanager/propertyset.py
20
22
90.91%

qiskit/passmanager/base_pass_runner.py
53
57
92.98%

qiskit/transpiler/runningpassmanager.py
53
59
89.83%

qiskit/passmanager/base_passmanager.py
103
111
92.79%

qiskit/passmanager/flow_controller.py
94
105
89.52%

Files with Coverage Reduction
New Missed Lines
%

qiskit/extensions/quantum_initializer/squ.py
2
80.0%

qiskit/transpiler/fencedobjs.py
4
81.48%

src/sabre_swap/layer.rs
4
97.32%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
67383

Relevant Lines:
79016

💛  - Coveralls


Error: terminal filter named envoy.filters.network.golang of type envoy.filters.network.golang must be the last filter in a network filter chain.

Title:
Error: terminal filter named envoy.filters.network.golang of type envoy.filters.network.golang must be the last filter in a network filter chain.
Description:
Seen on contrib-dev, but also on all v1.30+ releases that admit the golang filter.
It seems that the golang network filter implementation is hardcoded to be a terminal filter https://github.com/envoyproxy/envoy/blob/63d4fb0c442d6362bcf5101fc2a5279aeaef31bb/contrib/golang/filters/network/source/config.h#L42, while the HTTP golang filter can be placed anywhere in the chain.
Repro steps:
Add any network filter in a chain after envoy.filters.network.golang

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Config:

Include the config used to configure Envoy.

# envoy demo with golang extension enabled
static_resources:
  listeners:
  - name: listener_0
    address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 10000
    filter_chains:
    - filters:
      - name: envoy.filters.network.golang
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.filters.network.golang.v3alpha.Config
          is_terminal_filter: false
          library_id: simple
          library_path: "/lib/simple.so"
          plugin_name: simple
          plugin_config:
            "@type": type.googleapis.com/xds.type.v3.TypedStruct
            value:
              echo_server_addr: echo_service
      - name: envoy.filters.network.tcp_proxy
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.filters.network.tcp_proxy.v3.TcpProxy
          stat_prefix: destination
          cluster: cluster_0
  clusters:
  - name: plainText
    type: ORIGINAL_DST
    lb_policy: CLUSTER_PROVIDED
  - name: cluster_0
    connect_timeout: 30s
    type: LOGICAL_DNS
    dns_lookup_family: V4_ONLY
    load_assignment:
      cluster_name: cluster_0
      endpoints:
        - lb_endpoints:
            - endpoint:
                address:
                  socket_address:
                    address: localhost
                    port_value: 5000

Logs:
golang-network-proxy-1         | [2024-10-25 08:55:50.756][1][critical][main] [source/server/server.cc:412] error initializing config '  /etc/envoy.yaml': Error: terminal filter named envoy.filters.network.golang of type envoy.filters.network.golang must be the last filter in a network filter chain.
golang-network-proxy-1         | [2024-10-25 08:55:50.756][1][info][main] [source/server/server.cc:1050] exiting
golang-network-proxy-1         | Error: terminal filter named envoy.filters.network.golang of type envoy.filters.network.golang must be the last filter in a network filter chain.

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Call Stack:

If the Envoy binary is crashing, a call stack is required.
Please refer to the Bazel Stack trace documentation.

It's a known TODO, but we do not have much time to implement it yet.
Please see: https://github.com/envoyproxy/envoy/issues/25906#issuecomment-1615117709


Order of authors

Clearly this is Pedersen et al., but what about the et al?
I don't personally care about this and suspect the rest of you don't either, but we will have to put an author order down at some point prior to submission.
Two proposals:

After Eric, we go alphabetically by family name (so Pedersen, Miller, Ross, Simpson), or

After Eric, we go in random order to be determined by Eric as necessary by running
 sample(c('Miller','Ross','Simpson'))

Any other offers/thoughts on this?
I'm entirely happy with either of the orderings proposed above or any other suggestions you have.

I am happy with either or an even more convoluted random allocation scheme 😄 🎲

We'll hash everyone's name, and sort alphabetically by hash!
Seriously, though, I'm happy with either option, and would be honoured to pull the handle on the RNG to sample names if that's what everyone thinks is fair.

Pull the handle, Eric!  Don't forget to set.seed() for reproducible
revisions ;)
On Wed, Sep 12, 2018 at 1:42 PM Eric Pedersen<EMAIL_ADDRESS>wrote:

We'll hash everyone's name, and sort alphabetically by hash!
Seriously, though, I'm happy with either option, and would be honoured to
pull the handle on the RNG to sample names if that's what everyone thinks
is fair.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/noamross/mixed-effect-gams/issues/28#issuecomment-420736240,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAi5aJDs78W1Ait_LdIGntoloAdtxkLKks5uaUdsgaJpZM4WkBIR
.

All right! I ran:
set.seed(11) #the first palindromic prime!
sample(c('Miller','Ross','Simpson'))

And the order is:
Pedersen, Miller,  Simpson, Ross


Merge release/5.0.0 back to dev

are we removing this yubikit-related code from dev? @melissaahn @gitaumoses4

are we removing this yubikit-related code from dev? @melissaahn @gitaumoses4

I don't think we want to remove this.

Added the YubiKit changes back in.

I'm going to revert the commits that added the YubiKit changes back in. Going to keep YubiKit features separate from Dev until August.


US608063: Java schema object builder enhancements

https://internal.almoctane.com/ui/entity-navigation?p=131002/6001&entityType=work_item&id=608063

A developer build has not yet been created for this branch. Click here to go ahead and create the build...

CI Build Link:
http://sou-jenkins2.swinfra.net:8080/job/FileAnalysisSuite/job/FileAnalysisSuite~adaptersdk-schema~US608063-enhance~CI


Receive page design alignment

This is a redesign of the' receive page' that normalizes the font and elements alignment

The 'receive page' has been updated to align the QR code and the address horizontally and to hide the 'generate new address' button in a dropdown

Just noticed a display issue when there are multiple accounts in wallet. The account drop down appears cut off. Something z-index can prolly fix.

Also, @raedah once said these dropdowns are too wide and can/should be reduced to simply fit the content adequately.

Another issue with multiple accounts, this time when you select a different account to generate new address. Ideally, as soon as a different account is selected, the currently displayed address and qr code should be cleared so a user does not mistake it for the newly generated address.
In the following screenshot, the "whew" account doesn't really exist (has a wrong account number) and there was an error trying to generate the address. The error could not be displayed.

updated


Add inspect override to Connection

Improve user experience in IRB and elsewhere by hiding the details of Google API Client, the inspect output of which is incredibly verbose.
[closes #208]

Do these inspect overrides need nodoc?

Yes, inspect is a public method but not something we should document as part of the public API. I think we should nodoc it.


Fix Parallel transform in the presence of metadata

This is a hotfix for the Parallel transform. I started playing with more complex pipelines and realized that we didn't have this case covered in our tests. In a future PR we should consider introducing a dummy table type with metadata in our test suite.

Codecov Report

Merging #122 (bcd4a85) into master (d4fa63c) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #122      +/-   ##
==========================================
+ Coverage   94.22%   94.31%   +0.08%
==========================================
  Files          27       27
  Lines         797      809      +12
==========================================
+ Hits          751      763      +12
  Misses         46       46

Impacted Files
Coverage Δ

src/transforms/parallel.jl
95.06% <100.00%> (+0.85%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Change db migrations from jobs to init containers

From comment https://github.com/huggingface/datasets-server/pull/810#discussion_r1105871837

there are so many past jobs that we don't need anymore that we could delete them periodically. Manually or with a cron job. Otherwise, the migration jobs take forever, and the helm update timeouts and fails

@severo sugesstion:

Migrate from jobs to init containers to ensure the migration has time to finish without making Helm upgrade timeout

A possible implementation is described in https://andrewlock.net/deploying-asp-net-core-applications-to-kubernetes-part-7-running-database-migrations


install.sh fails on macOS

I am trying to use your install.sh script on mac and linux. Works great on linux, however on macos I get a failure that unary operator is expected in line 128 and 133. Do you have any hints on what could be wrong here?

I think I found the fix. Using 'expr str : regex' instead of 'expr match str regex' seems to have fixed it while still working on linux. I'll do a pull request.

#2


Allow SerializesModels to optionally restore models excluded by global scope (SoftDeletes, for example)

Currently, if you want to queue jobs with deleted models (or anything else restricted by default via a global scope), you cannot use SerializesModels. This change adds a method to Illuminate/Queue/SerializesAndRestoresModelIdentifiers called restoresModelsWithoutScopes(), which returns false by default, but can be overridden by classes that use it to change this behavior.
Now you can do:
class DoSomethingWithDeletedUsers extends Job implements ShouldQueue
{
    use SerializesModels;

    public function __construct(User $user)
    {
        $user->delete();
        $this->user = $user;
    }

    public function handle()
    {
        $this->user->trashed(); // true
    }

    protected function restoresModelsWithoutScopes()
    {
        return true;
    }
}

By having restoresModelsWithoutScopes() return true, the getRestoredPropertyValue() method will call newQueryWithoutScopes() instead of newQuery().

Personally it seems like it should always be without scopes and not even be configurable. If you queued the model instance surely that means you want it to be available.

That's what I thought at first, but I can think of some cases where that's not true. For example, say you queued an email to a user to be delivered in 3 hours and then deleted the user for some reason 2 hours later—in that case you probably wouldn't want the job to run.
I could see a strong argument for defaulting restoresModelsWithoutScopes() to return true, though, because I think you're right in most cases.

Yeah, I can't decide there. I mean that sounds fairly edge case and could be mitigated by checking if the user has been deleted or not before sending the email. I'm still of the opinion it should be without scopes by default.

I'll update tonight making no scopes the default. I don't think it hurts much to make it configurable, especially because it's a pretty big pain to modify this particular behavior via inheritance/composition.

Looks like a nice idea. :+1:

Done.

I tweaked this a bit. If you want scopes you can just override the getQueryForModelRestoration method. There isn't really a need for an option then.

Oh, good call. That's certainly simpler is we assume that most people won't want to change this.


Unexpected extra space in multi-line tuple struct with pub

pub struct MyService(
    pub  dependency::Multi<
        Arc<dyn dependency::ServiceWithDebugInfo + Send + Sync>,
        Arc<dyn dependency::ServiceWithDebugInfo + Send + Sync>,
    >,
);

I realize this is kind of a terrible type to begin with, but the extra space after the pub for the field was unexpected. If there's a reason for it, though, please close this!

Duplicate of https://github.com/rust-lang/rustfmt/issues/5703, https://github.com/rust-lang/rustfmt/issues/5525, https://github.com/rust-lang/rustfmt/issues/5997

🤦 I searched "tuple struct space" and "tuple struct pub". Thanks.


Support ETag based feed polling

Track ETag http response header and use it in requests

Version 1.0 does this


Dependency mirroring RFC

I have dependency mirroring and mirror replication as requirements for my intended use case. I have some ideas for how to hack around it short term but would like to understand how mirroring (referenced in https://github.com/apple/pkl/issues/153) is intended to work long term so I can limit rework later.
My guess is that there will be an argument to pkl project resolve like --mirror pkg.pkl-lang.org=example.com/pkl (supporting multiple mirrors) which would act as a replacer pattern for package manifests when pulling into the cache.
My current thought for working around it is to mirror the raw dependency (in an OCI repo, https://github.com/apple/pkl/issues/354) and for each dependency of my project

download the dependency explicitly and unpack the zip
run a string replace based on the mirror, e.g. pkg.pkl-lang.org -> example.com/pkl from above, in the manifest (and assume for right now that all dependencies are in the manifest rather than inlined in an import in source)
generate a PklProject file for the dependency based on the updated manifest and a defined folder structure
recurse through its dependencies

Then use all resolved dependencies as local dependencies.
I'll also have to mirror all transitive dependencies on the inbound side but that's workable.
It would be a big help to understand if this is wildly different from the planned mirror support or if I'm over-engineering it (which wouldn't shock me).
Thanks!

I know you already said mirroring is a requirement, but: would HTTP proxying be good enough for you?
To support mirroring, we are considering rewrite rules. This is hand-wavey, but, possibly would work by adding something to the Pkl settings file. Something like:
groovy
// ~/.pkl/settings.pkl

packageRewriteRules {
  [#"http://pkg.pkl-lang.org/.*/(\w+)#] = "package://my.company.com/$1/$1"
}

But, we wouldn't implement this if HTTP proxying is good enough.

HTTP proxy is not good enough, unfortunately


"Checkout Pull Request" does not always put me in review mode

Play around with the PR extension and checkout some branches
After like 5 minutes of work, open the PULL REQUESTS view and click on some PR and "Checkout Pull Request"
Nothing happens 🐛

If I reload VS Code and try again all is good.
Here are the output logs of the Pull Request extension.
[Info] Review> switch to Pull Request #169839 - start
[Info] PullRequestGitHelper> Branch origin/lszomoru/git-remotes-fix doesn't exist on local disk yet.
[Info] PullRequestGitHelper> Fetch tracked branch refs/remotes/origin/lszomoru/git-remotes-fix
[Info] PullRequestGitHelper> associate lszomoru/git-remotes-fix with Pull Request #169839
[Info] Unshallowing failed: Failed to execute git {
  "exitCode": 1,
  "gitCommand": "pull",
  "stdout": "",
  "stderr": "fatal: --unshallow on a complete repository does not make sense\n"
}.
[Info] Falling back to git pull.
[Info] Pull after failed unshallow still failed: Failed to execute git {
  "exitCode": 1,
  "gitCommand": "pull",
  "stdout": "",
  "stderr": "fatal: --unshallow on a complete repository does not make sense\n"
}
[Info] Review> Review> switch to Pull Request #169839 - done
[Info] Review> Queuing additional validate state

I'm not able to reproduce this, but it sounds like you may be losing git events. If you see this again can you go to the SCM view and hit the refresh button there?

Ok. I will keep my eyes open.


Use TS InferredProject, ConfiguredProject instead of language service API

If project depend on some large packages (lot of .d.ts files), language service API spend lot of time to resolve auto import from node_modules packages.
TS implement cache logic in InferredProject, ConfiguredProject over language service API to avoid performance issue. Not sure is possible to use InferredProject, ConfiguredProject instead of language service API in volar. If can, should be able to fully achieve vanilla TS level performance.
https://github.com/microsoft/TypeScript/blob/009dd487a6543d16c8e403ea055ac0683c6bc5d3/src/server/project.ts#L258

In 0.30.0, you can config "typescript.suggest.autoImports": false to skip calculate auto import for now.

@johnsoncodehk  In the Settings. The json?

@jahnli in VSCode settings / .vscode/settings.json.

Is that right ?

Just tried reference TS internal code to implement auto-import cache logic for language service API, but it may invalid at any time with TS version update, has happened once in the past (https://github.com/johnsoncodehk/volar/commit/15e744f335fcbdb4db2e42a2778970d597b92ae0).


Declaring a var inside a loop with a let of the same name gives no error

for (let v; ; ) { var v; }
for (let v in []) { var v; }
for (let v of []) { var v; }

These all should be errors per the ES6 spec sections <IP_ADDRESS> and <IP_ADDRESS>:
It is a Syntax Error if any element of the BoundNames of LexicalDeclaration also occurs in the
VarDeclaredNames of Statement.

The var scope is not the for statement, it is the enclosing function. so i do not think this is an error

That's true, but the spec still says it's an error.

@bterlson is this really intended to be an error?

Yes, generally speaking it is an error for a var decl to hoist past a lexical decl.

@bterlson, does that include blocks as well?
{
   let x;
   {
      var x;
   }
}

@JsonFreeman Yes, believe this is covered by the static semantics in 13.1.1.

Thanks @Andarist!

Closing 4-digit issues feels good 🤣


feat(kit): add itemsHidden prop to tui-input-time

Fixes #3519

@waterplea could you review please?


ARTEMIS-1122 ActiveMQJAASSecurityManager class loading issue

The ActiveMQJAASSecurityManager class uses LoginContext to validate
users and roles. LoginContext loads LoginModule classes defined in
the configuration (login.config) using current thread's context
classloader.
Normally this wouldn't be a problem but when a caller thread comes
from JMX (for example a client calls QueueControl.sendMessage() via
JMX) the caller thread has a different context class loader.
This will cause the LoginContext to fail to load the LoginModule
class (e.g. PropertiesLoginModule) and the validation will fail
even if correct credentials are supplied.

@jbertram Hi Justin can you review this pls?
Thanks


Running project with release config fails

Description
Running react-native project with release config via Xcode fails and gets stuck to the screen below. There is no error or stack trace to find the reason behind it. Everything works smoothly in debug mode.

Reproduction Steps

change the run config to release via product/scheme in Xcode
run on any iOS simulator

React Native version: 0.44.2, react: 16.0.0-alpha.6
Platform: iOS, Android
Development Operating System: macOS Sierrra 10.12.5
Build tools:Xcode 8.3.3

I feel this is an issue with bundle not getting attached to production build.
I get the following error on running react-native build:
error: option '--entry-file' missing

This is an issue with bundling.
react-native bundle --platform ios --dev false --entry-file index.ios.js --bundle-output iOS/main.jsbundle gives the following error:
SyntaxError: Unterminated string constant

Bundling with --dev true works as expected.

@veris-amoghbanta Did you solve it? Running into the same issue...

Wasn't able to pin point the exact cause, but it was some native dependency that caused the bundler to fail silently. So I switched to a new project and installed and linked the exact compatible dependency versions by hand and it seemed to have worked.
Most probably, it was either react-native-elements or react-native-router-flux that was causing the issue in my case.


[fix]Core dump in SegmentIterator when there is no predicate

Proposed changes
Issue Number: close #10453
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

MySQL [test]> select count(*) from test;
+----------+
| count(*) |
+----------+
|    60017 |
+----------+
1 row in set (0.181 sec)

MySQL [test]> select count(*) from test;
+----------+
| count(*) |
+----------+
|    45150 |
+----------+
1 row in set (0.036 sec)

Hi, I do test and get wrong result


linux/arm not available

[x ] I have read the contribution documentation for this project.
[X ] I agree to follow the code of conduct that this project follows, as appropriate.
[X ] I have searched the issue tracker for an issue that matches the one I want to file, without success.

I'm wanting to build for linux/arm which the documentation suggests is possible on 12.1.0 but it isn't an available option when i execute

Console output when you run electron-packager with the environment variable DEBUG=electron-packager. Please include the stack trace if one exists.
$ electron-packager . --appname=wsp_kiosk  --platform=linux --arch=arm64  --version=2.0.0
Unsupported arch arm64; must be one of: ia32, x64

What command line arguments are you passing? Alternatively, if you are using the API, what
parameters are you passing to the packager() function?
 --appname=wsp_kiosk  --platform=linux --arch=arm64  --version=2.0.0

Using electron-quickstart I'm trying to package my app for arm but it isn't working.

Can you make sure what version of Electron Packager you're running? if electron-packager --version doesn't print out version numbers, it's too old.

Also, --appname is not a valid parameter.

Strange, when I run npm list it shows the version as 12.1.0, but electron-packager --version returns nothing. looks like a dependency issue on my end thanks

I just realized that you're using --version to specify the Electron version. This hasn't been the way to specify that parameter since it was deprecated in 8.5.0 and replaced in version 9. You need to use --electron-version. Check out --help for details.

Yeah, thanks again, uninstalled/installed the package and it's all working correctly now. Thanks again, obviously i've been using this for a couple years and appreciate what ya'll do!


getDerivedStateFromProps not updating the layout properly

getDerivedStateFromProps seems to not update the layout property, when changing the layout property with the newest React version seemingly nothing happens. I have debugged it and it seems like nextProps.layout and prevState.propLayout are always identical in the getDerivedStateFromProps hook. This might be due to some recent bug fixes by the react team: https://reactjs.org/blog/2018/06/07/you-probably-dont-need-derived-state.html
Pull request will follow, please have a look.

We checked this bug on all versions from 16.4 to the current one. We did not meet anything that could disrupt the grid, if you change prop "layout". If you have more information, then share it with us so that we can more accurately reproduce the error.

hey, thanks for checking it out, please check this codesandbox
https://codesandbox.io/s/wizardly-hugle-s7x0h?fontsize=14
After 3 seconds the static flag is set to false and a new state is set, therefore the GridLayout receives a new layout props. However, due to the faulty check for layout change, the new layout is not loaded and the 'a' in the example remains static. The same code run with the changes I made in my pull request results in the layout updating after 3 seconds and therefore the 'a' becomes non static correctly.
Let me know if it's somewhat unclear.
Expected behaviour: After 3 seconds the item 'a' in the grid layout becomes non static.
Actual behaviour: nothing happens, even though the state is updated and the new layout is passed as a prop to the GridLayout

@Lagily, hey, your example not exact, because array copied only, and not copied items from array which are using by link, so nextProps.layout and prevProps.propsLayout is equal in getDerivedStateFromProps.
Fixed example: https://codesandbox.io/s/quirky-roentgen-jc7i1
Me it seems, this not library problem :slightly_smiling_face:


Dependency warnings blog post

Coverage: 93.958%. Remained the same when pulling 103ecd57c7e2f460ff833ff9c85d89615f4481c5 on cody-dependency-warning-blog-post into 589d5cca5692bdffe56e7f4701abd405187f9b21 on main.


(prerelease) unequipping scrying lens returns grid to normal size despite having another

if you hold and equip a scrying lens at the same time, and remove one of them, the grid goes back to the normal size. unequipping and equipping it again/ stopping holding going back to holding fixes it.

(Disclaimer: this was said without looking at the code nor trying the pre-release) Perhaps the mod should use vanilla's attribute system and define a custom attribute, rather than doing it based on inventory events (or whatever exactly it's doing right now). If nothing else it would be more extensible. Just make sure 2 lenses can't combine their attribute modifier or I suppose you would end up with the opposite problem of this.

going to test if this is since it is now attribute based


audio creation gives an empty(261 bytes) mp3

when using this CLI cmd:
python3 -m podcastfy.client --transcript /Users/Andreas/deep_dive/data/transcripts/transcript_2dcea9b591e340bda5778e323822e7ad.txt -tts openai -l --conversation-config conversation_config.yml
also with -tts edge, same result
it ends with an empty mp3 , i added my openai key to a .env file and this in config.yml:
openai:
default_voices:
question: "echo"
answer: "shimmer"
model: "tts-1-hd"
ao
Please help

Hi Thársis,
I figured it out , the transcript appears to need ... and ...  tags
After I added those to my handcrafted dialog file it started to work
Still I wonder why some of my fully, i.e. taking from an url, did not create those tags in the transcript
Cheers
Andreas

Edge is the most limited model available. Consider using more advanced models.
As of today, our recommendation is to use tts_model = geminimulti, our multispeaker model available in v0.4.0.

All sample audio in README have been updated to use the new TTS Model. Added some longform podcasts too.

I've updated python notebook describing longform podcast + new Google TTS model work:

https://github.com/souzatharsis/podcastfy/blob/main/podcastfy.ipynb

Hi ThársisThanks for the explanation and recommendationI have a Gemini API key now, will see it’s benefits!RegardsAndreasmy email: @.*** 16 Nov 2024, at 20:33, Tharsis Souza @.***> wrote:
Edge is the most limited model available. Consider using more advanced models.
As of today, our recommendation is to use tts_model = geminimulti, our multispeaker model available in v0.4.0.
All sample audio in README have been updated to use the new TTS Model. Added some longform podcasts too.
I've updated python notebook describing longform podcast + new Google TTS model work:
https://github.com/souzatharsis/podcastfy/blob/main/podcastfy.ipynb
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


makemkv should use a different standard storage folder

Hello,
I recently saw a user in a support forum who had issues because he saved all his movies in the standard directory. As the snap updated, according to him hundreds of gigabytes were copied to the new revision. So this essentially reduced his available disk space a lot and he found it out, but I believe a lot of users won't and this can cause serious trouble for them. So maybe something like /home/$USER/movies should be used as a standard directory instead of $HOME/snap/makemkv/revision.
Best regards ;)

Thanks for the report. I think the commit I just added will fix this. It is building now and will be available in the edge channel of the Snap Store soon.


fix Gem Load Error

There was an error while trying to load the gem 'yard-cucumber'.
Gem Load Error is: uninitialized constant Cucumber::Gherkin::AstBuilder
fixes burtlo/yard-cucumber#72

A solid, clear change.


Should we use cloned typeComposer to keep the original one immutable?

According to: https://github.com/nodkz/graphql-compose-connection/blob/master/src/composeWithConnection.js#L30
We may change to
newTypeComposer = typeComposer.clone(newTypeName);
newTypeComposer.addResolver(resolver);
return newTypeComposer;

Clone on Connection will copy only the top type and resolvers (it will not go to deeper types Edges, Node) https://github.com/nodkz/graphql-compose/blob/master/src/typeComposer.js#L258
The main purpose of clone method to clone single type for adding/removing some fields.
Widely used in copy filters and output records in the graphql-compose-mongoose. Resolvers are cloned for tests, right now. And I can not imagine good business logic needs for resolver cloning.
What is your main aim (business logic) to do clone?
Immutability with types is something strange for me right now, but maybe I don't understand something.

Thanks for the response.
There is no specific business logic for cloning. At the first sight of composeWithConnection, I think that is a decorator for making a higher-order TypeComposer.

composeWithConnection() add one resolver connection to initial TypeComposer and creates proper OutputType for it. In other words, just extend existed TypeComposer.
On this week we will finish tests with relations between types in our big project, and I'll update https://graphql-compose-mongoose.herokuapp.com/ with a more complex example. Right now it almost has stable syntax.
// managers.js
import mongoose from 'mongoose';
import { GraphQLString } from 'graphql';
import { CrmCompanyTC } from 'app/_schema/crm/company';
import composeWithMongoose from 'graphql-compose-mongoose';
import composeWithRelay from 'graphql-compose-relay';

const CrmManagerSchema = new mongoose.Schema({
  name: String,
  lastname: String,
  middlename: String,
  phones: [String],
  email: String,
  tags: [String],
});

CrmManagerSchema.virtual('fullname').get(function () {
  return `${this.lastname} ${this.name} ${this.middlename}`;
});

export const CrmManager = mongoose.model('CrmManager', CrmManagerSchema);
export const CrmManagerTC = composeWithRelay(composeWithMongoose(CrmManager));

CrmManagerTC.addField('fullname', {
  type: GraphQLString,
  resolve: source => source.fullname,
  projection: {
    lastname: true,
    name: true,
    middlename: true,
  },
});

CrmManagerTC.addRelation(
  'companies',
  () => ({
    resolver: CrmCompanyTC.getResolver('connection'),
    args: {
      filter: (source) => ({ managersIds: `${source._id}` }),
    },
    projection: { _id: true },
  })
);

// company.js

import composeWithMongoose from 'graphql-compose-mongoose';
import composeWithRelay from 'graphql-compose-relay';
import mongoose from 'mongoose';
import { CrmManagerTC } from './managers';

const CrmCompanySchema = new mongoose.Schema({
  name: String,
  phones: [String],
  address: String,
  email: String,
  tags: [String],
  additionally: String,
  access: String,
  personsIds: [String],
  managersIds: [String],
});

export const CrmCompany = mongoose.model('CrmCompany', CrmCompanySchema);
export const CrmCompanyTC = composeWithRelay(composeWithMongoose(CrmCompany));

CrmCompanyTC.addRelation(
  'managers',
  () => ({
    resolver: CrmManagerTC.getResolver('findByIds'),
    args: {
      _ids: (source) => source.managersIds,
    },
    projection: { managersIds: true },
  })
);

Both files import each other and make cross references via the anonymous functions. That allows solve hoisting problem with undefined.


What was the reason for the seemingly arbitrary default value of historySize?

I use the same value for constants in my code, don't know why. It might be to do with a TV show I watched a long time ago.


Update Project.toml

Pull Request Test Coverage Report for Build 611552224

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.3%) to 48.45%

Totals

Change from base Build 611397581:
2.3%

Covered Lines:
2078

Relevant Lines:
4289

💛  - Coveralls


Release of podaacpy version-1.7.0

Lots of new features have been addressed since the last release and I think this is a perfect time to release a new version.

@lewismc

@Omkar20895 good idea. I think we should document the l2ss class
https://github.com/nasa/podaacpy/blob/master/podaac/l2ss.py
Thank you guys for adding this... great work man.

Nice @Omkar20895 I'll push a release now.

Thank you very much! :) @lewismc


Release 1.5.0 does not pass unit tests

+ go test -buildmode pie -compiler gc -ldflags ' -X github.com/optiopay/kafka/version=1.5.0 -extldflags '\''-Wl,-z,relro  '\'''
--- FAIL: TestTLSConnection (0.01s)
        connection_test.go:776: could not conect to test server: x509: certificate has expired or is not yet valid
FAIL
exit status 1
FAIL    github.com/optiopay/kafka       6.508s

Release unit tests should always work long term so they can be run when a security issue forces the update of the golang compiler and existing code needs checking for regressions.
(also there’s a typo in the error message)

You are right, thank you!  It was fixed in https://github.com/optiopay/kafka/pull/91

Seems to be fixed, and v1.5.0 is now quite old. Closing.


[BUG] Excepted Rule not Being Ignored

Describe the bug
When running the tool, the rules excluded per the scanning config file are still being run by the tool against the specified .bicep file.
Expected behavior
When referenced, the config file with the exclusion list should force the Template Analyzer to exclude the listed rules. For example, if we want to exclude Rule Number AZR-000125, Azure.KeyVault.PurgeProtect, we should be able to add the rule number or name in the "exclusions" list and not have that rule run against the .bicep file being scanned.
Reproduction Steps
Whether run as a local installation or as a Conan package from our production pipeline, the config file with the exclusion list does not exclude the listed rules. For example, if we want to exclude Rule Number AZR-000125, Azure.KeyVault.PurgeProtect, whether we use the rule number, name, or both, the rules are still run against the .bicep file, as is seen in the output. Attached is our testing config file for reference.
ta-config.json
Environment

Ubuntu 20.04
Template Analyzer Version 0.5.2
Run both from the CLI in Ubunu's Visual Studio Code (x64 1.84.1) and Ubuntu's built-in Terminal

@securityboxtc Thanks for describing your issue. Looking at your config fule, I want to point out that unfortunately the filters in the configuration only applies to the JSON based rules (start with TA-) and not the PSRule based rules (AZR-). Also, your config file itself is not valid JSON.

Good morning, @nonik0.  Apologies for the late response; I was on extended leave.
Thanks for the response and clarifying those points, especially about the two types of rules, PS and JSON.
Since the tool stops running when a file has been judged as noncompliant with an individual rule, is there any way to have the tool generate a report of all rule violations? If not, is that option or the option to also exclude PS-based rules on the future roadmap for this tool?
Thank you!

Hi @securityboxtc, can you clarify what you mean by this?

... the tool stops running when a file has been judged as noncompliant with an individual rule

The tool will report all rules that a template doesn't comply with, not just the first rule violation.


fix: updating coral tag label to word wrap

The tag labels currently expand and cover any nearby field, changing them to break into multiple lines.
Description

Related Issue

https://jira.corp.adobe.com/browse/CQ-4324579
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.

@Pareesh  please review, Thanks!


Fix django urls in template

e.g. in index.html:
{% url 'core_version' %}
{% url 'user_settings' %}
{% url 'password_change' %}
Check other templates!

@emanuelschuetze is this still open?

There are still two old django links in index.html:
{% url 'user_settings' %}
{% url 'password_change' %}

Already fixed. Closed.