Option to power-off at start

[X] I have checked that a similar feature request does not already exist.

I have a device permanently living next to my computer, and I use the systemd integration from #3101, so basically I always have scrcpy running. Unless I power-off (Alt-P), scrcpy will occupy the CPU all time (#3112). I am looking for an option that automatically hits the power button after connection, similarly to how --turn-screen-off just turns the device screen off. This would allow auto-connection, but without involving the CPU too much until I need to interact.
Thanks!

In fact, unless --no-control is enabled, the server explicitly power the device screen on on start. An option --no-power-on could be added.
Do you want to work on it?

@rom1v this does indeed look like it should be easy to do, but those are famous last words… I haven't done Java in more than 2 decades, and I've never actually developed for Android, I wouldn't even know how to test this… or are there any easy resources I can peruse to get me started?

You need to add a command line option, and propagate it to the server. It's very similar to ccbe370cc52c5cd4315aec0d153b4108e6237498 which adds the --no-cleanup option.
In addition, you should also add the option in bash-completion and zsh-completion scripts, which were written after --no-cleanup.

I implemented it: #3210. Please review/test.

@rom1v Sorry I have not responded earlier. I'm in the process of moving to another country, so there's lot in the air.
I've just tested this, and it seems to work beautifully. Thank you!

FYI, it does not work with a Google Pixel 3 with stock ROM. power_on=false is correctly sent to the server, but the display does power on…

No, if the display is off, then it works. I see where the problem is:

Plug in USB;
Screen powers on;
udev fires scrcpy rule;
scrcpy starts, while display is on.

The Samsung Galaxy S7 and the OnePlus 5T do not turn the display on when USB is connected, so that is the difference. Sorry for the noise.


what is the best way to replace adata.X with another matrix?

Hi anndata team,
I am trying to work with both raw count matrix and normed data. So I hope to do something like anndata.X = anndata.obsm['raw'].copy(). However, this process is super slow.
I tried anndata.X.data = anndata.obsm['raw'].copy() and it does not really work. Do you have any suggestion on this? Thanks!

Is the object on the left hand side a view? If not, I don't see why this would be slow. An example of a profile or some timings could be useful for us to know what you're seeing.
But, this seems fine:
pbmc = sc.datasets.pbmc3k_processed().raw.to_adata()
pbmc.layers["counts"] = sc.datasets.pbmc3k()[pbmc.obs_names, pbmc.var_names].X.copy()

%timeit pbmc.X = pbmc.layers["counts"]
# 2.46 ms ± 10.2 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

oh, by view you meant, if anndata is a subset of another data, by anndata= ad1[:,:], it will be very slow? I think this could be the issue. shall I do anndata= ad1[:,:].copy() in this case?

That could make it slow, but it would be useful to see what your case is and what the timing are.

i took 50k cells x 20k gene from 83k cellsx 20k gene, and performed the assignment. it took more than an hour...

That definitely seems wrong.

Was the assignment target a view? If so, did making a copy help?
Did you get any warnings during this?

That definitely seems wrong.

Was the assignment target a view? If so, did making a copy help?
Did you get any warnings during this?

Yes, it is a view. and making a copy helps.

There is warning when I generate the view but no warning when I assign the matrix to view.X

/home/xiey/miniconda3/envs/scrna/lib/python3.8/site-packages/pandas/core/arrays/categorical.py:2487: FutureWarning: The inplaceparameter in pandas.Categorical.remove_unused_categories is deprecated and will be removed in a future version.   res = method(*args, **kwargs)


--fixed-beam-shape should create separate mosaics

If --fixed-beam-shape is provided the best common resolution to convolve to is not used. In the current form it is one or the other.
In practise if we use this argument then we should create a separate linmos (and sett of per beam) images at this fixed resolution.

Doing this in the multilinmos branch

Addressed by #135


[SPARK-28240][PYTHON] Fix Arrow tests to pass with Python 2.7 and latest PyArrow and Pandas in PySpark

What changes were proposed in this pull request?
In Python 2.7 with latest PyArrow and Pandas, the error message seems a bit different with Python 3. This PR simply fixes the test.
======================================================================
FAIL: test_createDataFrame_with_incorrect_schema (pyspark.sql.tests.test_arrow.ArrowTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/.../spark/python/pyspark/sql/tests/test_arrow.py", line 275, in test_createDataFrame_with_incorrect_schema
    self.spark.createDataFrame(pdf, schema=wrong_schema)
AssertionError: "integer.*required.*got.*str" does not match "('Exception thrown when converting pandas.Series (object) to Arrow Array (int32). It can be caused by overflows or other unsafe conversions warned by Arrow. Arrow safe type check can be disabled by using SQL config `spark.sql.execution.pandas.arrowSafeTypeConversion`.', ArrowTypeError('an integer is required',))"

======================================================================
FAIL: test_createDataFrame_with_incorrect_schema (pyspark.sql.tests.test_arrow.EncryptionArrowTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/.../spark/python/pyspark/sql/tests/test_arrow.py", line 275, in test_createDataFrame_with_incorrect_schema
    self.spark.createDataFrame(pdf, schema=wrong_schema)
AssertionError: "integer.*required.*got.*str" does not match "('Exception thrown when converting pandas.Series (object) to Arrow Array (int32). It can be caused by overflows or other unsafe conversions warned by Arrow. Arrow safe type check can be disabled by using SQL config `spark.sql.execution.pandas.arrowSafeTypeConversion`.', ArrowTypeError('an integer is required',))"

How was this patch tested?
Manually tested.
cd python
./run-tests --python-executables=python --modules pyspark-sql

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12343/
Test PASSed.

Test build #107173 has started for PR 25042 at commit 8e16d94.

Test build #107173 has finished for PR 25042 at commit 8e16d94.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107173/
Test PASSed.

I'm going to merge this. Trivial and I'm pretty confident.

Merged to master.

+1, late LGTM.


[Feature] AWS Authentication/Signing

Overview

Insomnia Version: 5.5.2
Operating System:  macOS 10.12.5
Summary:  Would like to be able to do use AWS IAM credentials and make signed requests to AWS services

Other Notes
The node.js request module has built-in support for this and it comes in handy. I don't know all the details but the end user provides AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY (for non-MFA accounts) and the timestamp and some headers are factored into a digest that gets calculated. MFA-enabled accounts are a bit trickier and also require AWS_SESSION_TOKEN.
In the mean time, I have worked around this by using a node.js script and the request package.

Hey @focusaurus. The signing seems like a straightforward process.
AWS has some docs (along with Python source code examples) here: http://docs.aws.amazon.com/general/latest/gr/sigv4_changes.html
Would you be interested in taking a shot at implementing this? If not, I should be able to get around to it, eventually.

OK taking a crack at this. Almost certainly will need some commits from you to get into mergable shape but I'll see how much I can get working locally basing code off existing patterns for AUTH_NTLM.

Have the UI rendering OK and just got my first 200 OK back from AWS so getting the basics in place is looking promising so far.

@focusaurus that's awesome! Nice work.
I don't mind throwing some of my own commits at this, as long as you can get the hard part (request signing algorithm) working. It's probably also a good idea to put this into its own function with some basic tests, if you haven't done that already.

FYI the signing algorithm there's the aws4 package on npm which I have integrated and working nicely. We don't need to implement the guts of the signing from scratch, just get the options and credentials ready to pass to aws4. No tests yet but will look at that next. The code structure follows the pattern in place for NTLM.

Oh nice. May not be worth writing tests if the aws4 package handles all the hard parts 👍


feat: Add flag to allow overriding of port

Solves #371
Added a new port flag to override the default value of '3000'. Do I have to deal with checking if the port number is valid, or will BrowserSync handle it for us? (for example, setting port to something not between 1 and 65535, or using alphabets?)

@Ashvith10 I would prefer for us to validate our arguments and emit our own errors. Packages that script against us may want to parse error messages, and I'd prefer to expose as few implementation details to cli users as possible.


ConnectionError: HTTPConnectionPool

Hello,
I have a problem, I can connect my database influxdb with jupyter notebook.
any ideas ?
regards
SR
ConnectionError: HTTPConnectionPool(host='http', port=80): Max retries exceeded with url: //xxx.xxx.xxx.xxx:8086/query?q=SELECT++*++FROM++"p1"+++++&db=myhome (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x10b6094a8>: Failed to establish a new connection: [Errno 8] nodename nor servname provided, or not known',))

Can you paste the client code that set up the connection? From that output it looks as though you're not connecting to the right hostname.

`import argparse
import pandas as pd
%matplotlib inline
from influxdb import DataFrameClient
#host='localhost'
host='my_url_VM'
port=8086   # port of InfluxDB http API
"""Instantiate the connection to the InfluxDB client."""
user = 'root'
password = 'root'
dbname = 'home_assistant'
client = DataFrameClient(host, port, user, password, dbname)
fields = """ * """
measurement = """ "°C" """
filters = """WHERE "value" < 18.5 """
query = """SELECT * FROM "°C" WHERE "value" < 18.5"""
query = """SELECT {} FROM {} {} """.format(fields, measurement, filters)
response = client.query(query)`

I got the same issue but not solved yet


nat is no longer in the arch user repository

I've received this email on 10/27/2020, 10:46 PM UTC:
Foxboron [1] deleted nat-git [2].

You will no longer receive notifications about this package.

[1] https://aur.archlinux.org/account/Foxboron/
[2] https://aur.archlinux.org/pkgbase/nat-git/

I don't know the reason for this but I think the installation instructions for arch can be removed for now until another maintainer chooses to support this program.

https://github.com/willdoescode/nat/commit/9f5a1cf773b2b6b291a9d14ba8f92ee7b87002c9
I am not sure why it's not working but for now I will just remove the installation instructions.


Bluetooth

We have UART, we have no screen... Sounds like bluetooth to me?

For hardware see https://github.com/rusefi/rusefi-hardware/issues/58


Update chain.json to v2.0.2-rc1 release

Can we update the versions array as well, please?

I've assumed that "v2.0.1-rc6" -> "v2.0.2-rc1" is non consensus-breaking.

I've assumed that "v2.0.1-rc6" -> "v2.0.2-rc1" is non consensus-breaking.

Yes is not consensus-breaking.
I'm going to update the versions array.
All the releases have not binary, is it a problem?

I'm going to update the versions array. All the releases have not binary, is it a problem?

having binaries for the latest patch of each version object is a nice-to-have. If it's missing here, anyone needing that would just have to dig elsewhere for it.

I'm going to update the versions array. All the releases have not binary, is it a problem?

having binaries for the latest patch of each version object is a nice-to-have. If it's missing here, anyone needing that would just have to dig elsewhere for it.

Please hold until Monday, I will update it again with the last barberry upgrade and binary's history as su said

Should this be closed out for https://github.com/cosmos/chain-registry/pull/2417?

Should this be closed out for #2417?

No, it's different, this is DEVNET and 2417 is for MAINNET
Thanks 🙏

Should this be closed out for #2417?

No, it's different, this is DEVNET and 2417 is for MAINNET Thanks 🙏

Ah, yes, you're right. Thanks for clarifying. Will this need any additional update? or is it good to merge now?

Should this be closed out for #2417?

No, it's different, this is DEVNET and 2417 is for MAINNET Thanks 🙏

Ah, yes, you're right. Thanks for clarifying. Will this need any additional update? or is it good to merge now?

Yes, you can merge it; I also have updated the names setting the right Plan Names


Linux Build Fails Due to Recent ghoul/opengl/uniformcache.h Changes

OpenSpace linux build (gcc13, CMAKE_CXX_STANDARD=20) fails in ghoul with error:
In file included from /var/lib/jenkins/Desktop/OpenSpace/ext/ghoul/include/ghoul/font/fontrenderer.h:30,
                 from /var/lib/jenkins/Desktop/OpenSpace/ext/ghoul/src/font/fontrenderer.cpp:26:
/var/lib/jenkins/Desktop/OpenSpace/ext/ghoul/include/ghoul/opengl/uniformcache.h:94:44: error: static data member ‘ghoul::fontrendering::FontRenderer::<unnamed struct>::UniformNames’ in unnamed class [-fpermissive]
   94 |         static inline constexpr std::array UniformNames = { #v1, #v2, #v3, #v4, #v5 };

It is unclear why this fails but none of those before it, which are identical except for the number of function parameters.

I merged in https://github.com/OpenSpace/OpenSpace/pull/3217 that fixes the issue


Code Analysis Fixes 1

A small set of fixes for code analysis rules

avoid_types_on_closure_parameters
prefer_asserts_with_message
unnecessary_await_in_return

You will probably want to squash this one.
@kevmoo

@MelbourneDeveloper – super excited you want to help here.
Adding lints is likely not the best use of time, though.


Google Play Store rejects app because of this package

Google Play Store rejects app because some dependencies of this package.
This is an excerpt of error message:
Issue with your app
We found that your app is using Tencent TBS SDK which contains code to download executable code (e.g. dex, JAR, .so files) from a source other than Google Play.
If necessary, you can consult your SDK provider(s) for further information.
About the Device and Network Abuse policy
An app distributed via Google Play may not modify, replace, or update itself using any method other than Google Play's update mechanism. Likewise, an app may not download executable code (e.g. dex, JAR, .so files) from a source other than Google Play. This restriction does not apply to code that runs in a virtual machine or an interpreter where either provides indirect access to Android APIs (such as JavaScript in a webview or browser).
Apps or third-party code (e.g., SDKs) with interpreted languages (JavaScript, Python, Lua, etc.) loaded at run time (e.g., not packaged with the app) must not allow potential violations of Google Play policies.

Yeah i have the same issue, but there is nothing that you can do, because webview use TENCENT TBS SDK, i hope a fix to that.

Hello, I updated the webview some days ago, are you sure you have latest version?

updated the webview some days ago

@Mixpeal  Could you please mention this commit so we can know what changes has been made?

I am also getting the same issue. I have removed this package and app is approved. Please fix this issue


picolibc: sigevent missing fields

Describe the bug
In picolibc (both the module and SDK headers, fwict), struct sigevent is missing the sigev_notify_function and sigev_notify_attributes fields.
This ticket was opened to track the upstream bug and ensure the fix makes its way into Zephyr's picolibc module and the SDK prior to LTS release.
https://github.com/picolibc/picolibc/issues/754
Please also mention any information which could help others to understand
the problem you're facing:

What target platform are you using? All
What have you tried to diagnose or workaround this issue? Found bugs while running the testsuite
Is this a regression? Yes. Please see the picolibc bug.
...

To Reproduce
Steps to reproduce the behavior:

move signal.h to the minimal libc from posix
build just about any POSIX test suite
See error about missing fields.

Expected behavior
Tests should pass. Assigning a value to sigev_notify_function or sigev_notify_attributes should pass
Impact
It's a showstopper for fixing additional bugs in Zephyr or for using POSIX features from picolibc.
Logs and console output
Environment (please complete the following information):

OS: all
Toolchain (e.g Zephyr SDK, ...): all
Commit SHA or Version used: main

Additional context
https://github.com/picolibc/picolibc/issues/754

@keith-packard - wondering if it's best to assign this one to you. What do you think?

Do we need this in LTS? If so, I'll need to do an upstream picolibc release and then we'll need an SDK release and a module update. Is all of that feasible?

I could do a picolibc point release that just addresses that issue, but I'd rather do a full release as there are a few bug fixes which should get included.

synopsis has been running a closed test suite and finding some bugs.

@keith-packard - are you able to confirm this is fixed with recent revisions of picolibc? I think it was likely fixed in 1.8.7 and 1.8.8

Yup, this was fixed in picolibc 1.8.7. As 1.8.8 has been merged to Zephyr, I think this issue can be closed?


Fix credentials detection + syntax errors

I just got around to actually setting this up for myself, and found syntax errors 😬
Anyway, I managed to fix them before anyone could raise an issue about it~
Please feel free to test this pull request before merging it to make sure everything works as intended.

Looks solid to me! Thank you so much for looking into this and making some great code. I really appreciate it!


Add a chrome extension

It'd be nice to be able to easily install a chrome extension with this UI.

I haven't done much work with browser extensions TBH. Sounds like it would be neat though. Any interest in contributing? :)

Yup, the debugger is coming along.
Soon we'll have a standalone electron app that will be able to debug chrome + firefox and will support plugins.

That's sounds cool! I look forward to seeing it. :smile:


chore: enable errname and errorlint and goprintffuncname

Related issue(s)

Fixes #656
Fixes #657
Fixes #659

@b00f you can merge it now.


model attributes of WIMPEnergySpectrum not loaded in correctly using configuration files

Context: I was trying to simulate some fd.SR1WIMPSource events while loading the model parameters in via the configuration file functionality (PR 140).
df = fd.SR1WIMPSource(config=config_path).simulate(100)

Specifically, I wanted to simulate some fd.SR1WIMPSource events between 30 keV - 40 keV instead of the default 0.7 - 50 keV so I had the following line in the config file
energy_edges                            = np.geomspace(30., 40, 100)

However, I got some simulated events with energy outside 30 keV - 40 keV.
Problem: The model attributes of WIMPEnergySpectrum were not loaded in correctly from configuration files.
The configurations are loaded in in source.py which is called after the energy spectrum has been computed in energy_spectrum.py.
Solution:
It is possible to create a specific class that inherits from fd.SR1WIMPSource and this somehow manages to overwrite the model attributes like mw and energy_edges.
class my_wimp(fd.SR1WIMPSource):
    mw = 50
    energy_edges = np.geomspace(30., 40., 100)

However, being able to load in all model attributes of the sources via a single configuration file is still a better solution in terms of keeping track of the configurations used for fits.

Good catch! The problem is that the WIMPsource computes and sets new attributes (energies and energy_hist) on initialization/setup, and does not recompute them when the original attributes (energy_edges, mw, etc.) are changed.
I think the fix would be to have BlockModelSource override set_defaults to call each Block.setup() again after Source.set_defaults. The one complication would be that the length of array columns like energy_edges might change as well, so we'd either have to disallow that (so you probably still have to do subclassing) or make the handling of the array_columns field more complex.
I was working on a fix when I realized the NEST sources currently use copy-paste-modified energy spectra -- so any changes would have to be there as well, and there is a big PR for that open already. If this is not a big roadblock, I would wait until that is merged and we've had a change to reduce some of the duplication.


Add nitpick configuration

Set nitpick to ignore some directories that are known to not meet psr-2 (specifically, top level namespace).

Thanks @jwalton512!


feat: [M3-7345] – Unrecommended Linode configuration indicators on VPC Detail page

Description 📝
Assigning a linode to a VPC but not having the VPC interface be primary results in situations where network connectivity issues can arise (see ticket for more details). This PR adds warning indicators on the VPC Detail page when such configurations exist: a warning icon at the SubnetLinodeRow level
Changes  🔄

Interactive tooltip w/ warning icon added to SubnetLinodeRow when a linode has an unrecommended configuration
hasUnrecommendedConfiguration utility function
Test updates

Preview 📷

Warning icon

How to test 🧪
Prerequisites

An account with proper VPC tags
A VPC with:

a subnet within it
at least one linode assigned to it, with the linode having:

a public interface in eth0, designated as the primary interface (you can do this in the Linode Edit Config dialog)
a VPC interface in eth1

Verification steps
With the prereqs described above, observe:

The warning icon being present for the linode with an unrecommended configuration

If you go to the linode with an unrecommended configuration and adjust it by making the VPC interface (eth1) the primary interface, returning to the VPC Detail page should show that the warning icon is no longer present for that linode.
As an Author I have considered 🤔

[x] 👀 Doing a self review
[x] ❔ Our contribution guidelines
[x] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[x] 🔐 Removing all sensitive information from the code and PR description
[x] 🚩 Using a feature flag to protect the release
[x] 👣 Providing comprehensive reproduction steps
[x] 📑 Providing or updating our documentation
[x] 🕛 Scheduling a pair reviewing session
[x] 📱 Providing mobile support
[x] ♿  Providing accessibility support

Should the warning icon and dismissible banner supposed to show up for all linode statuses, including when the linode is inactive?
Also, when I switch the linode from a good config profile one to an unrecommended one, I need to reload the page in order to get the icon and banner to show up (when the linode is booting/active. If the linode's inactive, then the banner/icon don't show up after a page reload) -- see video:
Screen going white/loading is me reloading the page. (also deleting this linode now)

Should the warning icon and dismissible banner show up for all linode statuses, including when the linode is inactive, or just when it's booting/online?

Confirmed with UX last Friday that we don't want these indicators displayed when the linode is inactive 👍

When I switch the linode from a good config profile one to an unrecommended one, I need to reload the page in order to get the icon and banner to show up (when the linode is booting/active. If the linode's inactive, then the banner/icon don't show up after a page reload) -- see video:

What was happening here is: the linode is off in order to change the config --> config is changed, query gets refetched and active shows as false for the interfaces because the linode is off --> turn the linode back on, query doesn't get refetched. This should be fixed now

thanks for confirming @dwiley-akamai! Still getting the warning indicator to show up for inactive linodes. This happens if the linode had the warning indicator while online -- it stays after the linode is shut down. Might need to invalidate config queries as the linode is shutting down as well then?

Regarding the warning icon status, how does it work for linodes with multiple configurations? Is it a similar situation to the Reboot Needed status, where it might get super confusing for linodes w multiple configs? 👀

It's a similar situation unfortunately: I created a linode with an unrecommended config profile and a recommended config profile, assigning the former to one subnet and the latter to another subnet. I booted it with the unrecommended config profile and the icon displays for both of the linode's appearances in the table:

@dwiley-akamai thanks for confirming! Is UX good with this behavior? Otherwise maybe you could pass the subnet ID into the hasUnrecommendedConfiguration method and then use that to determine that whether or not the warning indicator should be displayed?

@dwiley-akamai thanks for confirming! Is UX good with this behavior? Otherwise maybe you could pass the subnet ID into the hasUnrecommendedConfiguration method and then use that to determine that whether or not the warning indicator should be displayed?

What did you have in mind with the subnet ID? I'm not sure just the subnet ID is enough on its own. I wrote this out and the logic seems to work:

Revised hasUnrecommendedConfiguration() and new helper function
export const hasUnrecommendedConfiguration = (
  configs: Config[],
  subnet: Subnet | undefined
) => {
  const subnetAssignedLinodeData = subnet?.linodes;

  for (const config of configs) {
    const configInterfaces = config.interfaces;

    /*
     If there is a VPC interface marked as active but not primary, we want to display a
     message re: it not being a recommended configuration.

     Rationale: when the VPC interface is not the primary interface, it can communicate
     to other VMs within the same subnet, but not to VMs in a different subnet
     within the same VPC.
    */

    return configInterfaces.some(
      (_interface) =>
        _interface.active &&
        _interface.purpose === 'vpc' &&
        !_interface.primary &&
        interfaceInSubnet(_interface.id, subnetAssignedLinodeData)
    );
  }

  return false;
};

export const interfaceInSubnet = (
  interfaceId: number,
  subnetAssignedLinodeData: SubnetAssignedLinodeData[] | undefined
) => {
  return subnetAssignedLinodeData?.some((subnetAssignedLinodeDatum) => {
    const assignedInterfaces = subnetAssignedLinodeDatum.interfaces;
    return assignedInterfaces.some(
      (assignedInterface) =>
        assignedInterface.id === interfaceId && assignedInterface.active
    );
  });
};

In the scenario I described in my previous comment, this logic has the warning displayed in the subnet for the linode whose VPC interface has the unrecommended config, and not in the other subnet (same linode but whose VPC interface has the recommended config).

@dwiley-akamai I wasn't clear, apologies. I meant pass in the subnet_id **in addition to the configs you were already passing in, which would allow you do do what you wrote above ^, to check if an interface was associated with the subnet_id you passed in. Glad to hear that works ^!

@dwiley-akamai Wednesday before Thanksgiving shenanigans 😅 - I just looked over what you wrote above again, and realized you had a different approach. I had more in mind that you could compare the interface.subnet_id to the passed in subnet_id, since interfaces should also have a subnet_id field if they're assigned to a vpc. You might not need the helper function interfaceInSubnet if so - you could compare directly in at line 53 if _interface.subnet_id === subnet_id
Will be re-reviewing this shortly!

@coliu-akamai Thanks for the thorough testing and review -- with the most recent changes I think the expected indicators should be displayed

Ah - one small thing I was thinking about in terms of accessibility for the warning indicator: is there a way to use the keyboard to click on the Configuration Profiles link that appears in the tooltip? I'm able to use the keyboard to get the tooltip to show up and have the contents be read in the voiceover, but couldn't figure out how to click on the link

Created M3-7486 for investigating the accessibility question


Add support for refunding payments instead of items.

Right now, we only support refunding items. However, it might at some point be useful to be able to refund an entire payment. This could leave some items in a slightly weird state (for example, if a payment only partially covered a particular item) - but I don't know that there's an elegant resolution for that.

This is actually what we do know anyway. Hah.


An error with numba0.43

Hi @lmcinnes. I am new to Python and running UMAP-learn via an interface from R package Seurat. It gives the following error related to Numba, any suggestion? Thank you!
macOS10.12.6; python 2.7; numba 0.43.0
Error in py_call_impl(callable, dots$args, dots$keywords)  TypingError: Failed in nopython mode pipeline (step: nopython frontend) Unknown attribute 'shape' of type none
File "../../../../../../Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/umap/umap_.py", line 88:
def smooth_knn_dist(distances, k, n_iter=64, local_connectivity=1.0, bandwidth=1.0):
    <source elided>
    target = np.log2(k) * bandwidth
    rho = np.zeros(distances.shape[0])
 ^
[1] During: typing of get attribute at /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/umap/umap_.py (88)
File "../../../../../../Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/umap/umap_.py", line 88:
def smooth_knn_dist(distances, k, n_iter=64, local_connectivity=1.0, bandwidth=1.0):
<source elided>
    target = np.log2(k) * bandwidth
    rho = np.zeros(distances.shape[0])
     ^

This is not usually a problem with Numba itself but instead often caused by
the use of unsupported features or an i

This seems to be specific to numba 0.43, and is potentially an issue with numba itself. The only solution may be to roll back to an earlier version of numba for now.

Similar problem that I just wanted to bring to your attention. I switched from python 2.7 to python 3, and have started getting the error in the attached image. I am still able to get coordinates and plot them, so it technically still works. But it's still disconcerting....

I suspect the error is fine, it means it failed to manage to parallelise that function, but it still computed everything just fine.


sbt 0.13.15 -> 0.13.17

no special reason, just good to be on the latest.
(well, 1.1.1 would be the latest, but sbt-dotty doesn't seem to be available for sbt 1 yet)

SBT 1 support is ongoing work in a branch but it's indeed not trivial. 👍

Thanks!


Contributions to SkillShow Repository

Hello Everyone!
SkillShow is a new repository where we can contribute to advance our skills. Although this repo has been excluded from Hacktoberfest 2024, we still welcome contributions!
Guidelines:
Please follow the existing guidelines in the repository.
Do not create any additional rules.
Link to the project: SkillShow
Looking forward to everyone’s contributions!

Hi there!
I am the Mentor of this project .
Although this repo is not part of Hacktoberfest 2024, we have some great alternatives for you. Welcome to SkillShow, a repository focused on skill enhancement!
Guidelines:
Please adhere to the existing guidelines.
No additional rules, thanks!
Check it out here: SkillShow.
Note: Start by creating an issue. Once it’s assigned to you, feel free to open a pull request (PR). Directly opening a PR will lead to an invalid label.
Looking forward to your contributions!


[WIP] Add a patch to ActiveRecord::Migration for tracking replicated migrations

This will fix https://bugzilla.redhat.com/show_bug.cgi?id=1601015 by only allowing the global region to execute a particular migration if it is sure that migration was successfully executed and replicated from all of the remote regions.
This ensures that we will never "miss" a schema state and get ourselves into a situation where we cannot continue replicating changes from a remote region.
Requires: https://github.com/ManageIQ/manageiq-schema/pull/266
Alternative to: https://github.com/ManageIQ/manageiq-schema/pull/264

Think this one is ready for review as well. The specs will go green once https://github.com/ManageIQ/manageiq-schema/pull/266 is merged. I'm in the process of testing this out on live systems.

So while testing this on a live system I discovered that enabling and disabling the subscription doesn't actually work from within the migration context because it takes a lock on the pglogical.local_node relation from within the migration transaction. That conflicts with the process that wants to start the replication process back up which also wants to take that lock.
We effectively deadlock waiting for a change to replicate. One option is to just wait without bouncing the subscription, but I'd like to find something that won't impact migration time so much. Will investigate more tomorrow.

Resolved the deadlock by using a separate dummy model connection to do the disable and enable calls for the subscription. Was able to see this work in a for-real upgrade so this should be good for final review.

Rails/ApplicationRecord - Models should subclass ApplicationRecord.

So I originally had this, but encountered issues with load order. I was getting things like ArRegion not being defined when running rake environment because we were defining a model within lib/extensions. So since ActiveRecord::Base did the job I went with that instead.
Don't want reviewers to think this wasn't a conscious decision.

All of my requests are code structure, but the code itself looks good to me :+1:.  Before merging, how does this affect documentation?

how does this affect documentation?

We need lots of documentation for this :laughing:
I think this might be a change from what was traditionally done back with rubyrep. Plus now that global vs remote ordering hasn't mattered for some time I think people have developed their own assumptions about how to orchestrate a migration across multiple regions.
This patch actually enforces that all the remote regions are migrated before the global will migrate. So telling people that this is the case in documentation would probably be good.
My one open issue with this is logging/output to the console during migrations. If someone does migrate the global region first, what do we log or print? Right now we're pausing without output which probably isn't a good idea alone.

Right now we're pausing without output which probably isn't a good idea alone.

:+1: Completely agree that we need output
Maybe call something like this in the loop for logging:
def log_and_print(message)
  if @last_message == message
    print "."
  else
    puts
    print message
  end
  logger.info(message)
  @last_message = message
end

Expected log output something like:
[---] 2018-09-20T01:02:01 INFO Waiting on remote region 9 to migrate to<PHONE_NUMBER>3245abcdefg
[---] 2018-09-20T01:02:02 INFO Waiting on remote region 9 to migrate to<PHONE_NUMBER>3245abcdefg
[---] 2018-09-20T01:02:03 INFO Waiting on remote region 9 to migrate to<PHONE_NUMBER>3245abcdefg

Expected terminal output something like:
Waiting on remote region 3 to migrate to<PHONE_NUMBER>3245abcdefg....
Waiting on remote region 9 to migrate to<PHONE_NUMBER>3245abcdefg..........

Where do we keep upgrade docs?

I'm not sure manageiq has upgrade docs really.
I'll update https://bugzilla.redhat.com/show_bug.cgi?id=1610095 with some information about this at the least.

Adding WIP real quick while I re-test a full upgrade with the new changes.

Annnnd it still works :tada: Removing WIP.

I thought I commented about the MiqRegion thing but I don't see my comment anywhere.  In any case, I don't like the use of the MiqRegion model since this is migration code, where we usually state "no models".  I'd be fine with a stub model, but I think everything in the PR can be done with plain SQL..even the array stuff.

Hammer backport details:
$ git log -1
commit 62980873f9c9baa4f0a24fbcee3adc88de0a8c9b
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Wed Sep 26 16:15:39 2018 -0400

    Merge pull request #17919 from carbonin/replication_migration_extension

    Add a patch to ActiveRecord::Migration for tracking replicated migrations

    (cherry picked from commit cc2043f32d08cd5c273a8866a14f55cef11a1a87)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1601015


react-bootstrap table pagination for historical data

Table pagination required for historical data, in case of overflow in frontend

Reviewed and looks okay!


Add video trimming quick action

Context
Adds quick action menu for the Video trimming feature (currently behind a feature flag).

Summary

Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

Enable Video trimming feature flag.
Add a video element
Verify it has the Trim quick action which enters the Trim mode.
Set the video as background.
Verify that now as well has the Trim quick action which enters the Trim mode.

[ ] This is a non-user-facing change and requires no QA

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[ ] I have tested this code to the best of my abilities
[ ] I have verified accessibility to the best of my abilities (docs)
[ ] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This code is covered by automated tests (unit, integration, and/or e2e) to verify it works as intended (docs)
[ ] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #8878

@barklund Can you do @googlebot I consent. here? It got messed up a bit with the merges.

@barklund Can you please reply to the CLA bot so it stops spamming us? :)

@googlebot I consent.


feat (Data/Polynomial/RingDivision): add lemmas for substraction and negation

Adds two lemmas that show that negation and subtraction commute with polynomial division.

thanks!

maintainer merge


Automatically run Biber in AUCTeX

For some reason, although AUCTeX is generally able to figure out which commands it should run and when, it can't handle running Biber and I have to do it manually. This should be fixed.

Fixed by setting LaTeX-biblatex-use-Biber to a non-nil value.

Apparently not fixed anymore; I don't know what changed, but this appears to still be an issue.

I haven't used Biber in years and nobody else has complained about this, so closing for now as stale.


core dump in macosx using big model

(Brief description of the problem in no more than 2 sentences.)
My cpp program sometimes core dump  in libmxnet.so when the model is as large as 200M bytes;
no core dump with small model.
Environment info (Required)
imac osx 10.13.6
Build info (Required if built from source)
git diff make/config.mk
@@ -82,7 +82,7 @@ USE_NCCL_PATH = NONE
whether use opencv during compilation
you can disable it, however, you will not able to use
imbin iterator
-USE_OPENCV = 1
+USE_OPENCV = 0
#whether use libjpeg-turbo for image decode without OpenCV wrapper
USE_LIBJPEG_TURBO = 0
@@ -90,7 +90,7 @@ USE_LIBJPEG_TURBO = 0
USE_LIBJPEG_TURBO_PATH = NONE
use openmp for parallelization
-USE_OPENMP = 1
+USE_OPENMP = 0
Error Message:
(Paste the complete error message, including stack trace.)
lldb main -c /cores/core.97762
(lldb) target create "main" --core "/cores/core.97762"
Traceback (most recent call last):
File "", line 1, in
File<EMAIL_ADDRESS>line 52, in
import weakref
File<EMAIL_ADDRESS>line 14, in
from _weakref import (
ImportError: cannot import name _remove_dead_weakref
Core file '/cores/core.97762' (x86_64) was loaded.
(lldb) bt
warning: could not execute support code to read Objective-C class data in the process. This may reduce the quality of type information available.

thread #1, stop reason = signal SIGSTOP

frame #0: 0x00007fff63e7da16 libsystem_kernel.dylib__psynch_cvwait + 10 frame #1: 0x00007fff64046589 libsystem_pthread.dylib_pthread_cond_wait + 732
frame #2: 0x00007fff61c81cb0 libc++.1.dylibstd::__1::condition_variable::wait(std::__1::unique_lock<std::__1::mutex>&) + 18 frame #3: 0x000000010d6bc364 libmxnet.somxnet::engine::ThreadedEngine::WaitForVar(mxnet::engine::Var*) + 596
frame #4: 0x000000010d7cd49a libmxnet.somxnet::NDArray::SyncCopyToCPU(void*, unsigned long) const + 954 frame #5: 0x000000010d6ad0d4 libmxnet.soMXPredGetOutput + 340
frame #6: 0x000000010c1cac30 mainInfer(pred_hnd=0x00007fcba2f00000, image_data=size=1, data=size=1) at face_predict.cpp:296 frame #7: 0x000000010c120e99 mainprocess_camera(model_path="../models/ncnn", camera=0x00007ffee3af5170, output_folder="./output/<IP_ADDRESS>", mainThread=true) at main.cpp:278
frame #8: 0x000000010c125f42 mainmain(argc=4, argv=0x00007ffee3af57b0) at main.cpp:484 frame #9: 0x00007fff63d2d015 libdyld.dylibstart + 1
(lldb) thread list
Process 0 stopped

thread #1: tid = 0x0000, 0x00007fff63e7da16 libsystem_kernel.dylib__psynch_cvwait + 10, stop reason = signal SIGSTOP thread #2: tid = 0x0001, 0x00007fff63e7da16 libsystem_kernel.dylib__psynch_cvwait + 10, stop reason = signal SIGSTOP
thread #3: tid = 0x0002, 0x00007fff63e7da16 libsystem_kernel.dylib__psynch_cvwait + 10, stop reason = signal SIGSTOP thread #4: tid = 0x0003, 0x00007fff63e7da16 libsystem_kernel.dylib__psynch_cvwait + 10, stop reason = signal SIGSTOP
thread #5: tid = 0x0004, 0x00007fff63e7da16 libsystem_kernel.dylib__psynch_cvwait + 10, stop reason = signal SIGSTOP thread #6: tid = 0x0005, 0x000000010c589a4a libmxnet.sovoid mxnet::op::BatchNormForwardImpl<mshadow::cpu, float, float>(mshadow::Streammshadow::cpu*, mxnet::OpContext const&, mxnet::op::BatchNormParam const&, std::__1::vector<mxnet::TBlob, std::__1::allocatormxnet::TBlob > const&, std::__1::vector<mxnet::OpReqType, std::__1::allocatormxnet::OpReqType > const&, std::__1::vector<mxnet::TBlob, std::__1::allocatormxnet::TBlob > const&, std::__1::vector<mxnet::TBlob, std::__1::allocatormxnet::TBlob > const&) + 1002, stop reason = signal SIGSTOP
thread #7: tid = 0x0006, 0x00007fff63e7da16 libsystem_kernel.dylib__psynch_cvwait + 10, stop reason = signal SIGSTOP thread #8: tid = 0x0007, 0x00007fff63e7da16 libsystem_kernel.dylib__psynch_cvwait + 10, stop reason = signal SIGSTOP
thread #9: tid = 0x0008, 0x00007fff63e7da16 libsystem_kernel.dylib__psynch_cvwait + 10, stop reason = signal SIGSTOP thread #10: tid = 0x0009, 0x00007fff63e7da16 libsystem_kernel.dylib__psynch_cvwait + 10, stop reason = signal SIGSTOP
thread #11: tid = 0x000a, 0x00007fff63e7e28a libsystem_kernel.dylib__workq_kernreturn + 10, stop reason = signal SIGSTOP thread #12: tid = 0x000b, 0x00007fff63e7e28a libsystem_kernel.dylib__workq_kernreturn + 10, stop reason = signal SIGSTOP
thread #13: tid = 0x000c, 0x00007fff63e7e28a libsystem_kernel.dylib`__workq_kernreturn + 10, stop reason = signal SIGSTOP

Minimum reproducible example
There is no obvious condition which cause the core dump.
I do manuelly send a sigstop signal to my main program, then main stop as usual.
I'm curious that there is no segment fault or abort or some other signal but a sigstop when the core dump occurs.
At first I compile the mxnet master branch. Then I switch a release tag '1.2.1.rc1', same thing happens.

Maybe the model size is not relevant. The model vision is more likely the trigger.
coredump model:
/Users/load/code/python/model/model-r100-gg/model-symbol.json ... 287521 bytes
/Users/load/code/python/model/model-r100-gg/model-0000.params ... 260958682 bytes
[11:34:04] src/nnvm/legacy_json_util.cc:209: Loading symbol saved by previous version v1.0.0. Attempting to upgrade...
[11:34:04] src/nnvm/legacy_json_util.cc:217: Symbol successfully upgraded!

This core dump issue causes by the bug from other code. But I do find some core dump bug which can be repeat very easily: create a infer handle by load a model, then the thread exit immediately. Then core dump happens, something like this:
  // Create Predictor   MXPredCreate(static_cast<const char*>(json_data.GetBuffer()),                static_cast<const char*>(param_data.GetBuffer()),                static_cast<int>(param_data.GetLength()),                dev_type,                dev_id,                num_input_nodes,                input_keys,                input_shape_indptr,                input_shape_data,                &pred_hnd);   assert(pred_hnd);   exit(0);
However, if insert a sleep() statement before exit(), the issue doesn't exist.


Add JMH benchmarks for decoding MVT and MLT into in-memory representation

Add benchmarks for the decoding performance of OpenMapTiles schema based tiles into the MVT and MLT in-memory representations. First Java benchmarks are really promising, with an order of magnitude improvement in the decoding performance in the current benchmarks (but has to be double checked with different libraries in particiular with the original Js lib for example):
Benchmark                         Mode  Cnt   Score   Error  Units
MLT decodin Z10  avgt    5   0,346 ± 0,042  ms/op
MVT decoding Z10   avgt    5  11,075 ± 2,051  ms/op
MLT decoding Z12 avgt    5   0,480 ± 0,052  ms/op
MVT decoding Z12  avgt    5   4,905 ± 0,324  ms/op
MLT decoding Z14 avgt    5   1,811 ± 0,270  ms/op
MVT decoding Z14  avgt    5  39,027 ± 1,670  ms/op
MLT decoding Z2   avgt    5   0,910 ± 0,114  ms/op
MVT decoding Z2   avgt    5  11,990 ± 0,793  ms/op
MLT decoding Z4  avgt    5   1,284 ± 0,152  ms/op
MVT decoding Z4   avgt    5  24,963 ± 2,668  ms/op
MLT decoding Z5  avgt    5   1,380 ± 0,178  ms/op
MVT decoding Z5   avgt    5  18,798 ± 1,704  ms/op
MLT decoding Z7   avgt    5   1,082 ± 0,141  ms/op
MVT decoding Z7   avgt    5  22,721 ± 2,873  ms/op
MLT decoding Z9   avgt    5   0,822 ± 0,095  ms/op
MVT decodingZ9   avgt    5  12,141 ± 1,266  ms/op

Yes avg is the average time. There are different metrics to benchmark in JMH for example also the througput see https://jenkov.com/tutorials/java-performance/jmh.html#jmh-benchmark-modes. Cnt is the number of benchmark iterations

Thanks! Good to merge when you are ready.

@mactrem or @springmeyer I guess its ok to merge - whichever one has more time to shepherd it forward?

@nyurik i add some additional benchmarks for the missing zom levels and do some minor improvements of the benchmarking then we can merge it

@springmeyer in the macOS tests there is some tiny tile size increase (25 bytes). The regeneration of the fixture tiles for the test is not working on my machine, so maybe we can have a look at. I will now merge it anyway to move forwared and try to fix the broken Ci later with your help

@mactrem sounds good, PR to fix the CI is up at https://github.com/maplibre/maplibre-tile-spec/pull/138


CNF-12792:nto: add configmaps/finalizers under operator Role

What this PR does / why we need it:
NTO is watching for the ConfigMap that encpasulets the PerformanceProfile and creates different components based on the given profile.
We should block the PerformanceProfile deletion as long as the dependent components exist.
For that NTO needs access the finalizers of the ConfigMap.
We should also update the control plane operator since
it's its responsibility to update NTO's role.
Which issue(s) this PR fixes
Fixes # CNF-11559

This needs to be added to the control-plane-operator role as well.
failed to update control plane: failed to reconcile cluster node tuning operator: failed to reconcile cluster node tuning operator role: roles.rbac.authorization.k8s.io \"cluster-node-tuning-operator\" is forbidden: user \"system:serviceaccount:e2e-clusters-5526g-example-tm6z7:control-plane-operator\" (groups=[\"system:serviceaccounts\" \"system:serviceaccounts:e2e-clusters-5526g-example-tm6z7\" \"system:authenticated\"]) is attempting to grant RBAC permissions not currently held:\n{APIGroups:[\"\"], Resources:[\"configmaps/finalizers\"], Verbs:[\"*\"]}"

This needs to be added to the control-plane-operator role as well.
failed to update control plane: failed to reconcile cluster node tuning operator: failed to reconcile cluster node tuning operator role: roles.rbac.authorization.k8s.io \"cluster-node-tuning-operator\" is forbidden: user \"system:serviceaccount:e2e-clusters-5526g-example-tm6z7:control-plane-operator\" (groups=[\"system:serviceaccounts\" \"system:serviceaccounts:e2e-clusters-5526g-example-tm6z7\" \"system:authenticated\"]) is attempting to grant RBAC permissions not currently held:\n{APIGroups:[\"\"], Resources:[\"configmaps/finalizers\"], Verbs:[\"*\"]}"

Thank you @sjenning updated as requested PTAL

/lgtm
/retest

I added the permissions to the control plane operator but the error persist:
    hypershift_framework.go:395: ReconciliationSucceeded, Reason: ReconciliationError, Message: failed to reconcile control plane operator: failed to reconcile controlplane operator role: roles.rbac.authorization.k8s.io "control-plane-operator" is forbidden: user "system:serviceaccount:hypershift:operator" (groups=["system:serviceaccounts" "system:serviceaccounts:hypershift" "system:authenticated"]) is attempting to grant RBAC permissions not currently held:
        {APIGroups:[""], Resources:["configmaps/finalizers"], Verbs:["*"]}

I added the permissions to the control plane operator but the error persist:
    hypershift_framework.go:395: ReconciliationSucceeded, Reason: ReconciliationError, Message: failed to reconcile control plane operator: failed to reconcile controlplane operator role: roles.rbac.authorization.k8s.io "control-plane-operator" is forbidden: user "system:serviceaccount:hypershift:operator" (groups=["system:serviceaccounts" "system:serviceaccounts:hypershift" "system:authenticated"]) is attempting to grant RBAC permissions not currently held:
        {APIGroups:[""], Resources:["configmaps/finalizers"], Verbs:["*"]}

@sjenning Could you please have take a look? it seems like the hypershift operator needs some additional permissions as well but I couldn't find the relevant files

/retest

/retest

/lgtm
/retest

/retest

The failure doesn't seems related to this PR and also reproduced on other PRs here
error: timed out waiting for the condition on hostedclusters/400b93ce3e99541261a6-mgmt
{"component":"entrypoint","error":"wrapped process failed: exit status 1","file":"sigs.k8s.io/prow/pkg/entrypoint/run.go:84","func":"sigs.k8s.io/prow/pkg/entrypoint.Options.internalRun","level":"error","msg":"Error executing test process","severity":"error","time":"2024-05-15T08:15:19Z"}
error: failed to execute wrapped command: exit status 1
---
Link to step on registry info site: https://steps.ci.openshift.org/reference/create-management-cluster
Link to job on registry info site: https://steps.ci.openshift.org/job?org=openshift&repo=hypershift&branch=main&test=e2e-azure, "e2e-azure" post steps failed: "e2e-azure" pod "e2e-azure-destroy-management-cluster" failed: could not watch pod: the pod ci-op-k9wjbwri/e2e-azure-destroy-management-cluster failed after 40m11s (failed containers: test): ContainerFailed one or more containers exited
Container test exited with code 1, reason Error
---
enshift/hypershift/cmd/cluster/core.waitForRestOfFinalizers
	/hypershift/cmd/cluster/core/destroy.go:198
github.com/openshift/hypershift/cmd/cluster/core.DestroyCluster
	/hypershift/cmd/cluster/core/destroy.go:119
github.com/openshift/hypershift/cmd/cluster/azure.DestroyCluster
	/hypershift/cmd/cluster/azure/destroy.go:71
github.com/openshift/hypershift/cmd/cluster/azure.NewDestroyCommand.func1
	/hypershift/cmd/cluster/azure/destroy.go:41
github.com/spf13/cobra.(*Command).execute
	/hypershift/vendor/github.com/spf13/cobra/command.go:920
github.com/spf13/cobra.(*Command).ExecuteC
	/hypershift/vendor/github.com/spf13/cobra/command.go:1044
github.com/spf13/cobra.(*Command).Execute
	/hypershift/vendor/github.com/spf13/cobra/command.go:968
github.com/spf13/cobra.(*Command).ExecuteContext
	/hypershift/vendor/github.com/spf13/cobra/command.go:961
main.main
	/hypershift/main.go:70

/approve

/retest-required

/jira refresh

/retest-required

/retest-required

/retest-required

/test e2e-kubevirt-aws-ovn

/test e2e-kubevirt-aws-ovn

/retest

@openshift/openshift-team-hypershift

/hold cancel


Viewports+Cameras in ViewportContainers not rendering in editor

Godot version:
Godot 3.0.3
OS/device including version:
Ubuntu 18.04 64bit on AMD Cayman (HD 6950) and Mesa 18.0.0-rc5.
Intel® Core™ i5-4570 CPU @ 3.20GHz × 4
Issue description:
Some viewports+cameras in viewport containers don't seem to render in editor (even though they render ingame). Duplicating viewport containers along with the viewport and camera will result in the duplicate to render correctly but the first one still does not render. Restarting Godot and reopening the project will result in the old viewports to render correctly but any new ones will have the same problem.

Steps to reproduce:

Create new scene with spatial node as root
Add Viewport container + Viewport + Camera.
Set size of Viewport to anything, 500x500 will do fine.
Look into 2D view and see how the viewport is invisible in editor.
Duplicate Viewport Container along with its children and see how the duplicate renders correctly in 2D view.
Play the scene and see how both viewports render correctly ingame.
Restart Godot and reopen the project/scene and see both viewports rendering correctly in editor
Do steps 2 to 4 again and see how the new viewport does not render correctly in editor also.

@Calinou I think this sort of bug was the reason we started asking people to set viewports as local to scene.

A minimal reproduction project doesn't really help given the fact that this bug disappears the moment you load the project. (see step 7 above)
I have tested this in 3.2.1 and 3.2.2beta4. This bug is present in both versions.

Seems to be fixed in v3.5.2.stable.official [170ba337a] and v4.0.stable.official [92bee43ad].


TAU-virkistys: avoin vs ei avoin

if vain avoin selected, nothing is shown. this differs from the current situation in prod

Few overall fixes regarding this particular issue:
Invalid fetching of Course Realisation Organisations
Probably the old code was working before for fetching the extra organisations, but after TAU changes something had changed and the logic was invalid.
Fixed by altering mapping logic:
https://github.com/UniversityOfHelsinkiCS/palaute/commit/16fdbe4a622cc8b03d12fc0da90521b7d62da348
Invalid date range fixes
There were issues when some of the summaries had no time ranges at all, which caused errors to the users. Not directly related to this issue but still fixed at the same time:
https://github.com/UniversityOfHelsinkiCS/palaute/commit/db56e872d01ac6c3c1f5c424f16ec6b7707858d7
Invalid summaries array given to the sumSummaries function
We were passing an filtered array of groupSummaries to the sumSummaries function:
https://github.com/UniversityOfHelsinkiCS/palaute/blob/1c9024bde4488d361545df45db745875a8067f67/src/server/services/summary/getOrganisationSummary.js#L80
This cause undeterministic behaviour based on what would be the first groupSummary gotten from the DB.
Fixed by just giving the function sumSummaries the whole cu.groupSummaries:
https://github.com/UniversityOfHelsinkiCS/palaute/commit/fdc470864503175cfe97a4068718f409c3bfbf57


mysql-community-client and mysql-community-server missing requirements

Description
The mysql.sh provisioning script specifies packages to install.
mysql_debs=(
        mysql-common*
        mysql-community-client-plugins*
        mysql-community-client*
        mysql-client*
        mysql-community-server*
        mysql-server*
        libmysqlclient21*
        libmysqlclient-dev*
    )

When running this provisioning script, I'm getting errors about missing dependencies

mysql-community-client-core  as a dependency of mysql-community-client
mysql-community-server-core as a dependency of mysql-community-server

I modified the script to install client-core and re-ran, only to run into the server-core dependency.  I'm testing adding server core as well, so my new package list is as follows:
mysql_debs=(
        mysql-common*
        mysql-community-client-plugins*
        mysql-community-client-core*
        mysql-community-client*
        mysql-client*
        mysql-community-server-core*
        mysql-community-server*
        mysql-server*
        libmysqlclient21*
        libmysqlclient-dev*
    )

I don't see any changes in the repository history that would explain this, so I hesitate on making a PR with the change because I'm not sure why it started happening.
Virtual environments affected

[ ] Ubuntu 18.04
[X] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
I use this library as a submodule in another project that I have for provisioning Hyper-V build agents on a local server, and my last successful agent build was 9/29/2021.  I will update this bug if my above local change succeeds.
Is it regression?
No response
Expected behavior
mysql.sh should provision without error on Ubuntu 20.04
Actual behavior
mysql.sh is unable to run based on missing dependencies.
Repro steps

Provision Ubuntu 20.04 machine - In my case, I am provisioning a Hyper-V VM with Ubuntu 20.04.3 iso
Execute the mysql.sh as part of provisioning.

Here is the log for the mysql.sh provisioning - Note that this is log is AFTER I added mysql-community-client-core to the package list, but before i added mysql-community-server-core.
==> hyperv-iso.autogenerated_1: Provisioning with shell script: templates/buildagents/lib/virtual-environments/images/linux/scripts/installers/mysql.sh
hyperv-iso.autogenerated_1: /tmp /home/matt
hyperv-iso.autogenerated_1: Reading package lists...
hyperv-iso.autogenerated_1: Building dependency tree...
hyperv-iso.autogenerated_1: Reading state information...
hyperv-iso.autogenerated_1: libaio1 is already the newest version (0.3.112-5).
hyperv-iso.autogenerated_1: libaio1 set to manually installed.
hyperv-iso.autogenerated_1: The following packages were automatically installed and are no longer required:
hyperv-iso.autogenerated_1:   liblldb-10 liblldb-12
hyperv-iso.autogenerated_1: Use 'sudo apt autoremove' to remove them.
hyperv-iso.autogenerated_1: The following NEW packages will be installed:
hyperv-iso.autogenerated_1:   libmecab2
hyperv-iso.autogenerated_1: 0 upgraded, 1 newly installed, 0 to remove and 13 not upgraded.
hyperv-iso.autogenerated_1: Need to get 233 kB of archives.
hyperv-iso.autogenerated_1: After this operation, 1,865 kB of additional disk space will be used.
hyperv-iso.autogenerated_1: Get:1 http://us.archive.ubuntu.com/ubuntu focal/main amd64 libmecab2 amd64 0.996-10build1 [233 kB]
hyperv-iso.autogenerated_1: Fetched 233 kB in 0s (1,025 kB/s)
hyperv-iso.autogenerated_1: Selecting previously unselected package libmecab2:amd64.
hyperv-iso.autogenerated_1: (Reading database ... 217216 files and directories currently installed.)
hyperv-iso.autogenerated_1: Preparing to unpack .../libmecab2_0.996-10build1_amd64.deb ...
hyperv-iso.autogenerated_1: Unpacking libmecab2:amd64 (0.996-10build1) ...
hyperv-iso.autogenerated_1: Setting up libmecab2:amd64 (0.996-10build1) ...
hyperv-iso.autogenerated_1: Processing triggers for libc-bin (2.31-0ubuntu9.2) ...
hyperv-iso.autogenerated_1: /tmp/mysql /tmp /home/matt
hyperv-iso.autogenerated_1: (Reading database ... 217222 files and directories currently installed.)
hyperv-iso.autogenerated_1: Preparing to unpack mysql-common_8.0.26-1ubuntu20.04_amd64.deb ...
hyperv-iso.autogenerated_1: Unpacking mysql-common (8.0.26-1ubuntu20.04) over (5.8+1.0.5ubuntu2) ...
hyperv-iso.autogenerated_1: Setting up mysql-common (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Installing new version of config file /etc/mysql/conf.d/mysql.cnf ...
hyperv-iso.autogenerated_1: Installing new version of config file /etc/mysql/my.cnf.fallback ...
hyperv-iso.autogenerated_1: Selecting previously unselected package mysql-community-client-plugins.
hyperv-iso.autogenerated_1: (Reading database ... 217223 files and directories currently installed.)
hyperv-iso.autogenerated_1: Preparing to unpack mysql-community-client-plugins_8.0.26-1ubuntu20.04_amd64.deb ...
hyperv-iso.autogenerated_1: Unpacking mysql-community-client-plugins (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Setting up mysql-community-client-plugins (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Selecting previously unselected package mysql-community-client-core.
hyperv-iso.autogenerated_1: (Reading database ... 217232 files and directories currently installed.)
hyperv-iso.autogenerated_1: Preparing to unpack mysql-community-client-core_8.0.26-1ubuntu20.04_amd64.deb ...
hyperv-iso.autogenerated_1: Unpacking mysql-community-client-core (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Setting up mysql-community-client-core (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Processing triggers for man-db (2.9.1-1) ...
hyperv-iso.autogenerated_1: Selecting previously unselected package mysql-community-client.
hyperv-iso.autogenerated_1: (Reading database ... 217244 files and directories currently installed.)
hyperv-iso.autogenerated_1: Preparing to unpack mysql-community-client_8.0.26-1ubuntu20.04_amd64.deb ...
hyperv-iso.autogenerated_1: Unpacking mysql-community-client (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Setting up mysql-community-client (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Processing triggers for man-db (2.9.1-1) ...
hyperv-iso.autogenerated_1: (Reading database ... 217293 files and directories currently installed.)
hyperv-iso.autogenerated_1: Preparing to unpack mysql-community-client-core_8.0.26-1ubuntu20.04_amd64.deb ...
hyperv-iso.autogenerated_1: Unpacking mysql-community-client-core (8.0.26-1ubuntu20.04) over (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Setting up mysql-community-client-core (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Processing triggers for man-db (2.9.1-1) ...
hyperv-iso.autogenerated_1: (Reading database ... 217293 files and directories currently installed.)
hyperv-iso.autogenerated_1: Preparing to unpack mysql-community-client-plugins_8.0.26-1ubuntu20.04_amd64.deb ...
hyperv-iso.autogenerated_1: Unpacking mysql-community-client-plugins (8.0.26-1ubuntu20.04) over (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Setting up mysql-community-client-plugins (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Selecting previously unselected package mysql-client.
hyperv-iso.autogenerated_1: (Reading database ... 217293 files and directories currently installed.)
hyperv-iso.autogenerated_1: Preparing to unpack mysql-client_8.0.26-1ubuntu20.04_amd64.deb ...
hyperv-iso.autogenerated_1: Unpacking mysql-client (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Setting up mysql-client (8.0.26-1ubuntu20.04) ...
hyperv-iso.autogenerated_1: Selecting previously unselected package mysql-community-server.
hyperv-iso.autogenerated_1: (Reading database ... 217299 files and directories currently installed.)
hyperv-iso.autogenerated_1: Preparing to unpack mysql-community-server_8.0.26-1ubuntu20.04_amd64.deb ...
hyperv-iso.autogenerated_1: Unpacking mysql-community-server (8.0.26-1ubuntu20.04) ...
==> hyperv-iso.autogenerated_1: dpkg: dependency problems prevent configuration of mysql-community-server:
==> hyperv-iso.autogenerated_1:  mysql-community-server depends on mysql-community-server-core (= 8.0.26-1ubuntu20.04); however:
==> hyperv-iso.autogenerated_1:   Package mysql-community-server-core is not installed.
==> hyperv-iso.autogenerated_1:
==> hyperv-iso.autogenerated_1: dpkg: error processing package mysql-community-server (--install):
==> hyperv-iso.autogenerated_1:  dependency problems - leaving unconfigured
==> hyperv-iso.autogenerated_1: Errors were encountered while processing:
==> hyperv-iso.autogenerated_1:  mysql-community-server
==> hyperv-iso.autogenerated_1: Provisioning step had errors: Running the cleanup provisioner, if present...

Hello @spyder007!
Thanks, we will take a look.

@mikhailkoliada For what it's worth, my updated package list in mysql.sh seems to have worked:
mysql_debs=(
        mysql-common*
        mysql-community-client-plugins*
        mysql-community-client-core*
        mysql-community-client*
        mysql-client*
        mysql-community-server-core*
        mysql-community-server*
        mysql-server*
        libmysqlclient21*
        libmysqlclient-dev*
    )

@spyder007 Hi!
Well, the main problem is that apt guessing a deptree if mysql was previously installed, no matter how it was done.
Could you please confirm that the current state of the main branch works for you?

@mikhailkoliada the new mysql provisioning step in main works fine.  Thanks for the quick response.

mysql-client is still broken. I'm using https://github.com/linux-training-group-1/asset-registry/blob/bb5731dc2e4659eba587209b5b04b1222ced9bd2/.github/workflows/ci.yml#L50


fix(kernels::grisubal): remove #[allow(unused)] from the module

remove the lint exception
fix warnings accordingly
call detect_orientation_issue after generating the Geometry2, in the main kernel function

Scope

[x] Code

Type of change

[x] Fix

Codecov Report
Attention: Patch coverage is 54.54545% with 5 lines in your changes missing coverage. Please review.

Project coverage is 81.78%. Comparing base (ddd74f4) to head (ab6cb3a).

Files
Patch %
Lines

honeycomb-kernels/src/grisubal/kernel.rs
66.66%
2 Missing :warning:

honeycomb-kernels/src/grisubal/mod.rs
0.00%
2 Missing :warning:

honeycomb-kernels/src/grisubal/model.rs
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #141      +/-   ##
==========================================
+ Coverage   81.74%   81.78%   +0.03%
==========================================
  Files          31       31
  Lines        4010     4007       -3
==========================================
- Hits         3278     3277       -1
+ Misses        732      730       -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


POST_SCRIPT always says filesystem bit not executable.

Maybe I'm doing something wrong but when using the POST_SCRIPT option, I am simply turing to either curl or wget to a URL (healthchecks.io) to keep a monitor on my backups to make sure they actually fire off nightly.
It seems that it is not working via either of these three methods
POST_SCRIPT: '/home/xxx/backup-healthcheck.sh'
Which is just a .sh file to do the curl command or wget command, same as options below.
I have tried this with and without sudo.
The permissions of the file are  0755 - so everyone has execute access.
Direct commands in the post script also do not seem to work.
POST_SCRIPT: 'curl -m 10 --retry 5 https://xxx'
POST_SCRIPT: 'wget https://xxx -T 10 -t 5 -O /dev/null'

Everything ends up failing with
[db-backup] Can't execute POST_SCRIPT environment variable '##########' as its filesystem bit is not executible!
How can I get this to run when backups kick off? Thank you!

Make sure your script is set to executible chmod +x (yourscript). I used to perform this automatically but stopped for security purposes.

Make sure your script is set to executible chmod +x (yourscript). I used to perform this automatically but stopped for security purposes.

@tiredofit do the .sh for sure was +X'd in this.
What about direct commands too, like curl or wget, they are also failing, is that expected? I would assume curl and wget are executable already but I am admittedly not super familiar with *nix at all, including permissions

You should be able to execute curl or wget from inside a script without issue - there are no barriers with that.
In a future release I can add an environment variable to set it executable before starting. I'm assuming you are using Windows and this could be the root cause of this.  Alternatively to bind mounting a directory on your system to /assets/custom-scripts you could probably mount a named volume, which could resolve that issue?

You should be able to execute curl or wget from inside a script without issue - there are no barriers with that.
In a future release I can add an environment variable to set it executable before starting. I'm assuming you are using Windows and this could be the root cause of this. Alternatively to bind mounting a directory on your system to /assets/custom-scripts you could probably mount a named volume, which could resolve that issue?

Hi @tiredofit  - so the device that's running this is an Ubutnu Server 20.04.4 LTS server running it as part of a docker-compose.yml file - My apologies for not including that in the original bug report.
So it is running on linux right now and the
POST_SCRIPT: 'curl -m 10 --retry 5 https://xxx'
unfortuantely did not work, and I would have thought that would work directly without issues.
I'll try mounting a bind mount and re-testing w/ my script there and making sure it's +x'd properly. I'll report back ;)

Ok it's working now - part of this was just my own stupidity, thank you for reminding me I need to have the script within a mounted volume. Whoops ;)
I was first trying to curl directly within the POST_SCRIPT variable which wasn't working then when I made my script I never mounted it.  My config is as follow if this ever helps anyone else searching here.
backups:
image: tiredofit/db-backup
container_name: backups
volumes:
- /home/xxx/backups:/backup
- /home/xxx/scripts:/scripts
restart: unless-stopped
links:
- db
environment:
TZ: 'America/Detroit'
DB_TYPE: 'pgsql'
DB_HOST: 'db'
DB_NAME: 'xxx_xxx'
DB_USER: 'postgres'
DB_PASS: 'postgres'
EXTRA_OPTS: '--schema=public --blobs'
COMPRESSION: 'GZ'
DB_DUMP_FREQ: '360'
DB_DUMP_BEGIN: '+10'
DB_CLEANUP_TIME: '10080'
POST_SCRIPT: '/scripts/backup_complete.sh'

Glad you are sorted out! Will address your other issue soon - Just getting ready for a trip and will resolve in transit.


Please clear the "Please consider the alternative route, if you want encrypt such factories." process. it's difficult to follow.

Please consider the alternative route, if you want encrypt such factories.
https://docs.microsoft.com/en-us/azure/data-factory/enable-customer-managed-key#during-factory-creation-in-azure-portal
Blow statement was so confusing for the customer (SR<PHONE_NUMBER>04429
121033026004429.pdf
) , me and my ADF team on CXP. We have created steps (Instructions for customer) in my lab and tested with customer as well and its working. Customer have provided the feedback to improve the document on the chat and call.
To encrypt the factory, Data Factory needs to first retrieve customer-managed key from Key Vault. Since factory deployment is still in progress, Managed Service Identity (MSI) isn't available yet to authenticate with Key Vault. As such, to use this approach, customer needs to assign a user-assigned managed identity (UA-MI) to data factory. We will assume the roles defined in the UA-MI and authenticate with Key Vault.
Instructions for customer
Please follow the below step for the "alternative route" as describe in the documentation and steps test in the lab, below in red highlighted is the confusing and complex part, which was simplified with step and screen shots.  I have attached the pdf as well.
https://docs.microsoft.com/en-us/azure/data-factory/enable-customer-managed-key#during-factory-creation-in-azure-portal>
Create Manage identity, Key vault (Access policy , Key ) and data factory.
During factory creation in Azure portal
This section walks through steps to add customer managed key encryption in Azure portal, during factory deployment.
To encrypt the factory, Data Factory needs to first retrieve customer-managed key from Key Vault. Since factory deployment is still in progress, Managed Service Identity (MSI) isn't available yet to authenticate with Key Vault. As such, to use this approach, customer needs to assign a user-assigned managed identity (UA-MI) to data factory. We will assume the roles defined in the UA-MI and authenticate with Key Vault.
Note: Manage Identity, Key value and data factory all need to create on the same region. This are the pre-step to data factory creation.
•	Create Manage identity: Managed identity types

•	Review +create
•	Create a Key Vault.

•	Add Policy in the key vault, and select roles Get, list, unwrap key and wrap key

•	Make sure you save the Access Policy.

•	Add the Key to the Key vault:

•	go the Key properties and Select and copy key identifier URL.

•	Create Data factories now
https://docs.microsoft.com/en-us/azure/data-factory/enable-customer-managed-key#during-factory-creation-in-azure-portal
This section walks through steps to add customer managed key encryption in Azure portal, during factory deployment.
To encrypt the factory, Data Factory needs to first retrieve customer-managed key from Key Vault. Since factory deployment is still in progress, Managed Service Identity (MSI) isn't available yet to authenticate with Key Vault. As such, to use this approach, customer needs to assign a user-assigned managed identity (UA-MI) to data factory. We will assume the roles defined in the UA-MI and authenticate with Key Vault.
To learn more about user-assigned managed identity, see Managed identity types and Role assignment for user assigned managed identity.

Make sure that User-assigned Managed Identity (UA-MI) has Get, Unwrap Key and Wrap Key permissions to Key Vault
Under Advanced tab, check the box for Enable encryption using a customer managed key

Sample from my lab:

Provide the url for the customer managed key stored in Key Vault
Select an appropriate user assigned managed identity to authenticate with Key Vault
Continue with factory deployment

Hiten Bhavsar
Customer Engineer II |   Azure CXP
Email<EMAIL_ADDRESS>|   Office: +1<PHONE_NUMBER>
Working hours: Mon - Fri 9:00 AM to 5:00 PM Eastern Time   (Atlanta)

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

ID: 8571ca06-129a-58b7-92be-dc680acb835f
Version Independent ID: 9bd22daf-eb91-b509-0f84-4d5e96863c96
Content: Encrypt Azure Data Factory with customer-managed key - Azure Data Factory
Content Source: articles/data-factory/enable-customer-managed-key.md
Service: data-factory
GitHub Login: @chez-charlie
Microsoft Alias: chez

All the steps in the PDF:
121033026004429.pdf

@HitenBhavsarMSFT
Thanks for the feedback! We have assigned the issue to the content author to investigate this and update the document as appropriate.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


#1757 Fix focus outline color for selected activities & restart

Focus color has been fixed for selected activities and the restart conversation button

Coverage remained the same at 65.77% when pulling 13aacd66d0798aec2a897b6942fc609576bd8284 on corinagum/1757 into e434a70da29c5086e7b4eb2a8b5011c3f8baac14 on master.


The punycode module is deprecated

DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.

I investigated this a bit more today, since it seems that it is fixed in yarn:
https://github.com/yarnpkg/yarn/issues/9013
I upgraded yarn, but still saw the warning; however running with
❯ NODE_OPTIONS='--trace-deprecation' yarn dev --no-open
showed that the error was now coming from
node_modules/markdown-it/lib/index.js:14:20
fixing it then showed the same issue coming from a transitive dependency psl (jsdom#tough-cookie#psl):
node_modules/psl/index.js:5:16
fixing that, and the warning was gone.
To sum up, three of our dependencies will need an upgrade: yarn, markdown-it, and psl.
The fix in both cases was to transform require("punycode") into require("punycode/"); I haven't tried to understand what it did or why.

Just an FYI, I get the "punycode" warning when pulling the official node.js docker image (v21, Debian "bookworm"). I tried updating yarn before creating the observable project, but that didn't seem to help.


[WIP] Investigate block broadcast

After mining a block, broadcast the block not the announcement.
edit: branch created of #1845, will clean this when merged.

Close, use another PR for this.


Installation failure (Windows)

Hi,
I ran into an installation failure when trying to do a pip install in Windows. The messages from the cmd terminal are below. Any advice?
pip install pygfried
Collecting pygfried
  Using cached pygfried-0.2.0.tar.gz (168 kB)
  Preparing metadata (setup.py) ... done
Building wheels for collected packages: pygfried
  Building wheel for pygfried (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py bdist_wheel did not run successfully.
  │ exit code: 1
  ╰─> [5 lines of output]
      running bdist_wheel
      running build
      running build_ext
      $ GOPATH='C:\Users\----\AppData\Local\Temp\1\tmpdbqnsh53' go get -d
      error: [WinError 2] The system cannot find the file specified
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for pygfried
  Running setup.py clean for pygfried
Failed to build pygfried
Installing collected packages: pygfried
  Running setup.py install for pygfried ... error
  error: subprocess-exited-with-error

  × Running setup.py install for pygfried did not run successfully.
  │ exit code: 1
  ╰─> [5 lines of output]
      running install
      running build
      running build_ext
      $ GOPATH='C:\Users\----\AppData\Local\Temp\1\tmpy_311sv5' go get -d
      error: [WinError 2] The system cannot find the file specified
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: legacy-install-failure

× Encountered error while trying to install package.
╰─> pygfried

note: This is an issue with the package mentioned above, not pip.
hint: See above for output from the failure.

Thanks!

Hi @raszel. I apologize for not getting back to you sooner, somehow I missed the notification.
I never published the win32 distribution (see https://pypi.org/project/pygfried/#files). We would need a Windows machine, e.g. we could build and publish from a win32 environment in CI. I don't really have time but if you do I'd be happy to accept a pull request that includes a CI workflow in it. Locally, you may have some success if you install the Go compiler, but that's far from ideal.

Thanks @sevein for the update! I think those would be beyond my skills at the moment, but I appreciate the suggestions!


[TypeChecker][Relay] Issue with return type not being properly inferred.

cc @hypercubestart after I land #6274 there is an issue where omitting a return type now causes inference to fail, when you work on mutual recursion can you look into this?
 // (@jroesch): if we leave off the return type this doesn't work
/*
 * Computes the sum of a list of integer scalars.
 */
 // (@jroesch): if we leave off the return type this doesn't work
def @sum(%xs: List[Tensor[(), int32]]) -> int32 {
  let %add_f = fn(%x: Tensor[(), int32], %y: Tensor[(), int32]) -> Tensor[(), int32] {
    %x + %y
  };
  @foldl(%add_f, 0, %xs)
}

/*
 * Computes the size of a tree.
 */
def @size[A](%t: Tree[A]) -> Tensor[(), int32] {
  match(%t) {
    Rose(_, %sub_trees) => {
      1 + @sum(@map(@size, %sub_trees))
    },
  }
}

https://github.com/jroesch/tvm/tree/tyck-err

This also type checks:
/*
 * Reverses a list.
 */
def @rev[A](%xs: List[A]) -> List[A] {
  @foldl(@flip(fn (%h: A, %t: List[A]) { Cons(%h, %t) }), Nil, %xs)
}

This one doesn't type check.
/*
 * Reverses a list.
 */
def @rev[A](%xs: List[A]) -> List[A] {
  @foldl(@flip(Cons), Nil, %xs)
}

@jroesch please update to see if the issue can be closed


Cannot set property 'canvas' of undefined

this is undefined when I try to create a chart.
I'm wondering if ChartJS should stop relying on this being defined.

Nevermind, I was using it wrong...


Add files via upload

Add our edu domain, please.

Please fork SWOT repository, copy file.zip in the root folder of your forked SWOT repository. Then unzip file.zip there with no changes to the directory structure. Then make a new pull request. If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message. I have to close this request without merging. Thank you for your understanding.


Keyboard events sample in winui3

Description
This PR contains a sample that showcases the implementation of the below keyboard events in WinUI3 Windows App SDK sample app.

KeyUp
KeyDown
CharacterReceived
KeyboardAccelerators

Windows App SDK samples are integrated with the docs samples browser. All feature samples should include metadata in the sample README to integrate with the docs sample browser.
For reference:
https://github.com/microsoft/WindowsAppSDK-Samples/tree/main/Samples/Widgets
https://github.com/microsoft/WindowsAppSDK-Samples/blob/main/Samples/Islands/README.md

The guidelines recommend having samples for both C# and C++ at a minimum.
On a side note - There is not a hard requirement to cover all the other combinations (e.g. having all UI frameworks, having both Unpackaged and Packaged samples). However, if there is something unique about the specific component, the samples should highlight and showcase these scenarios.

Add the appropriate copyright notice and license text to the headers.
For C#/C++ files (*h, *.cpp, *.cs, *.idl):
// Copyright (c) Microsoft Corporation.
// Licensed under the MIT License.

For XAML files (*.xaml):
<!-- Copyright (c) Microsoft Corporation.
 Licensed under the MIT License. -->

/azp run SamplesCI-Input

/azp run SamplesCI-Input

Has this step about "Onboarding Samples CI Pipeline for new feature" mentioned in the docs been taken care of?
I can see the .yml file is already present. SamplesCI-Input.yml
Kindly check if there are any pipeline changes required for this as well.


ExternalSeqRecordParser model and ExternalRecordParser model run error，why？

Hello，I am using TLA+ to implement trace verification, and found that you have provided two interfaces ExternalSeqRecordParser and ExternalRecordParser, but I am running these two interfaces and report an error, what is the reason for this, and I look forward to your reply.
ExternalRecordParser
connect.log
a : 42, b : true
a : -1, b : false
a : 0, b : true

ExternalRecordParser
connect.log
a : 42
b : true

Thank you for reporting the errors. I am currently traveling so it will be a few days before I can investigate. I look forward to resolving the issues!

What version of the tools are you using? There is a jar file provided in the repo. Everything should work if you run it with that version.

Aha! It must be related to this


AP Exam Week 1 - Proctored MCQ Week 5

[x] test results and corrections

Task | Grade | Correction |
Quiz5 | ✔ | Amazing job! I knew you could do it. Thank you for taking my comments from last time to heart, it means a lot to me to seething monumental progress in your growth for taking Computer Science Principles. Again, so proud of you and keep up the good work. But to keep in mind, I would definitely review more material so that you have the same conditioning for the actual AP test. |


SAK-43303 Lessons: On Settings, Ownership list should appear ordered …

…and similar as Assignments

LGTM!


Hygieia-stash authentication problem

Hi,
I tried to connect Hygieia to Bitbucket and therefore I used the stash-collector.
I always get this error:
GET request for "myURL.com" resulted in 401 (Unauthorized); invoking error handler
I tried to hardcode my login-values in the code but it dont work.
Can somebody help? Is there a possibility to provide basic authentication with the stash collector ?

Hi.
I wonder if ... did someone solve this problem?

They use some key for login . I also wasnt able to make it work and did a small workaround for this . Add the below code in your class DefaultBitbucketServerClient.java after line no 96.
if (decryptedPassword.equals("")) {
        decryptedPassword = "Yourpassword";
        repo.setUserId("YourUsername");
      }

But ofcourse not a nice way . Better understand how the key works.

See #664. There currently is not a way to set this up in the UI.

@Matschick Functionality merged in https://github.com/capitalone/Hygieia/pull/835
Refer to the "Encryption for private repos" section in the UI module readme.


Read-only service account role

Would be helpful to have read-only privileges to DB for support.

We should do this by adding a read-only replica, for with we can manage permissions separately. Replica can be deployed in different region, and should be used by Metabase and other non-production workloads. The only reason we don't have it yet is the cost-saving before we really need it (because it will add 50-100% db cluster costs).


Loop issue with AutoGPT

Hello.
The loop issue continues even after the updates. The model creates every single time another chat now. I will search for a solution myself, but in the meantime i will leave this issue open if someone knows how to solve this.

Ciao.
Il problema del loop continua anche dopo gli aggiornamenti. Il modello crea ogni volta un'altra chat ora. Cercherò io stesso una soluzione, ma nel frattempo lascerò aperto questo issue se qualcuno sa come risolverlo.

I saw this method in the README, but i didn't try that yet and i don't really like it. I feel like it's too "manual" if you know what I mean. I will find other ways and I'll let you know. I would like if it will be able to start a new chat and then associate the id automatically with that chat.
If anyone else have some good option let us know.

Lascerò l'issue aperto per eventuali aggiornamenti :)


Crash reports are ignored without error if Fabric is not initialized

Problem
I managed to produce a situation where I had a crash very soon after the launch of the app when Fabric wasn't fully initialized yet. In such case, a call to FlutterCrashlytics().reportCrash() returns successfully even though it doesn't actually report the crash.
The problem is the else branch here:
https://github.com/kiwi-bop/flutter_crashlytics/blob/e14aa46b1f71f6b68a38ab8a100e1dcf89cd240e/android/src/main/java/com/kiwi/fluttercrashlytics/FlutterCrashlyticsPlugin.java#L36-L40
As can be seen, it always returns succesully, even when Fabric is not initialized.
Possible solution
Instead of calling result.success(null), do something like this so that the caller can retry later if they want to:
if (Fabric.isInitialized()) {
  onInitialisedMethodCall(methodCall, result);
} else {
  result.error('CRASHLYTICS_NOT_FULLY_INITIALIZED_YET');
}

Hi @apaatsio, we can't return an error, the reason we return a success is because if you ask permission to retrieve data to your user (as you must do in Europe because of the law) user can say no so you'll not call initialized to not have it enabled. But you don't want to all your report the throw an exception after that as it's expected.
What I don't understand is how can you have a crash before initializing fabric ? that's the first thing you should do and await the initialized call to be sure it's initialized after that.

I'm using await.
I believe Fabric.with() is asynchronous under the hood and that's causing the issue.
I added logging like this:
if (methodCall.method.equals("initialize")) {
    Fabric.with(context, new Crashlytics(), new CrashlyticsNdk());
    Log.d("FlutterCrashlytics", "Fabric.isInitialized:" + Fabric.isInitialized());
    result.success(null);
}

and it prints false. So, Fabric is not initialized instantly after running Fabric.with().

Shit, dawn Fabric ^^
Are you able  to reproduce on both Android and iOS ? or just one platform ?
I need to think of a way to fixing this...

@jaumard I have experienced this only on Android since I don't have an Apple device to test on.

From the doc:
public static boolean isInitialized()
Returns true when all kits have finished asynchronous initialization.

so problem is only on Android as iOS API doesn't have this isInitialized method ^^
So maybe the fix is just not to use this method and have a boolean into the class instead.
What do you think ? Can you test that ?


Add feature toggles to web client

Add a new feature toggle, UNIFIED_API, which defaults to false.
A constant is now exposed on every component for every defined
feature toggle. Components can access the new toggle with
if (this.UNIFIED_API) {...} or in templates with v-if="UNIFIED_API".
Add two functions to the global scope, enable and disable.
Typing enable('UNIFIED_API') into the web console will enable the
feature toggle for the rest of the session, while
disable('UNIFIED_API') will disable it.
Toggles are listed in web/src/featureToggle.js. For development,
they can be temporarily enabled/disabled there.

Also, a minor linting change was applied to DownloadDialog.
You can see this in action on the netlify preview. Just open the web console and type enable('UNIFIED_API') and a console log should report the successful toggling.

Would we want this feature available to the client in production? What's the main/immediate use case for this?

I'm working on adding the ability to log into django instead of logging into girder. When you enable('UNIFIED_API'), the login form will switch from using the GWC auth to a different component that is aimed at the django auth endpoints (which I am also working on adding). In development you can also achieve that by editing featureToggle.js, but for branch previews, it's useful for anyone to be able to see what the changes look like.
This feature would be available in production, but it's completely undocumented and hard to find. I think it's more useful for us to use toggles in production than it is scary to have end users somehow figure them out and see our toggle-protected code.
As an aside, I figured out you can make a bookmark with the content javascript:enable('UNIFIED_API') to switch it on with the push of a button.


Remove the "Broadcast your kills" setting

I'M NOT SAYING THAT KILLS SHOULDN'T BE BROADCAST!
When Maxopoly added kill broadcasts, he also added a setting allowing you to disable broadcasts your kills. When I asked him about this he said that the feature was "very intentional" to allow for assassinations. I think players should be able to disable receiving kill notifications if they aren't bothered about that kind of information, but I do think, as I did then, that allowing people to disable their own kill broadcasts gives a massive advantage not only to assassins, but to client mods like interpol (or other pvp mods) which would effectively reimplement kill broadcasts. I just think kill broadcasts should be enabled for everyone.

I'm closing this out as something that we won't fix. It could be revisited in the future, but it's not within our capacity to work on right now.


How can I used it in obsidian after installation ?

basically title.

it isn't released yet so you can't. (you can still install it if you know how to convert the .ts into the right main.js plugin stuff but that isn't really helpful too because most of the query stuff is still missing)

it isn't released yet so you can't. (you can still install it if you know how to convert the .ts into the right main.js plugin stuff but that isn't really helpful too because most of the query stuff is still missing)

I have successfully compiled and installed the plugin, but I am unsure of how to utilize it. Unfortunately, I cannot locate the corresponding command in the command palette.
Would you kindly provide me with instructions on how to use this plugin, even if it is just for a partial feature? Your guidance would be greatly appreciated.

@kickTiger see the comment above. Currently, the plugin only does behind-the-scenes work and there is no way yet to interact with the plugin. In short: There is not functionality that you can use yet.

@mProjectsCode Thank you for your response. I installed it and saw the settings interface, I thought it already had a UI to use.

I manually installed version 0.16 today and tried it out, but I couldn't find a successful way to run DataCore.


Update dependencies

Fix updated lint
Update travis

/cc @boygirl

Tested locally lank'd with corresponding PR in victory-chart. Tests pass and everything is working as expected. Approved


token-cli: Don't sign with owner when creating an associated token account

Problem
The create-account always signs with the owner argument when present but this should not be done when creating associated token accounts.
  I'm fine with this as is if we need it now, I'd like to fix it upstream if it can wait, though.
The meat of the changes are here, but I still need to slog through the CLI plumbing and we'll need new crates.
The spl-token CLI changes are then just
diff --git a/token/cli/src/main.rs b/token/cli/src/main.rs
index e4607b1..c4337a7 100644
--- a/token/cli/src/main.rs
+++ b/token/cli/src/main.rs
@@ -1983,13 +1983,14 @@ fn main() {
                     )?;
                 }

+                let signers = signer_info.signers_for_message(&message);
                 let mut transaction = Transaction::new_unsigned(message);

                 if config.sign_only {
-                    transaction.try_partial_sign(&signer_info.signers, recent_blockhash)?;
+                    transaction.try_partial_sign(&signers, recent_blockhash)?;
                     println!("{}", return_signers(&transaction, &OutputFormat::Display)?);
                 } else {
-                    transaction.try_sign(&signer_info.signers, recent_blockhash)?;
+                    transaction.try_sign(&signers, recent_blockhash)?;
                     let signature = config
                         .rpc_client
                         .send_and_confirm_transaction_with_spinner(&transaction)?;

I'd prefer to get this out now since the bug is causing problems for users, your upstream changes look great though!


Include information on tsdb status api to help estimate memory usage.

Proposal
Probably everyone working with Prometheus already cross this amazing post from robust perception.
It is useful to at least give us one idea of how much memory a prometheus instance may require to ingest data based on their cardinality.
Based on this I would like to suggest the addition of the following information to the TSDB Status API.

Average Labels Per Time Series
Number of Unique Label Pairs
Average Bytes per Label Pair

cc @bboreham WDYT?

Generally more information is good, but I’m afraid Brian’s post is misleading. Unique labels are all stored as separate copies per series, and label pairs are never treated as a unit.
I spoke about memory consumption at the last PromCon: https://promcon.io/2022-munich/talks/why-is-it-so-big-analysing-the-m/
See also https://github.com/prometheus/prometheus/issues/11530

Hello from the bug-scrub!
The TSDB Status page now gives more information, and the specific items requested in this issue are not high-value, so we will close this.
Personally I think a broader view of memory - how much is used by scraping, service discovery, queries, tsdb, etc. - would be very useful, but that would be a different issue.


Increase Dapr logging verbosity from app -> sidecar

As it stands today, there is very little logging from the dapr sidecar once initialization is complete. I would like to propose that we increase the the logging for calls between the application and its local sidecar instance (specifically runtime errors that may be occurring such as ERR_STATE_STORE_NOT_CONFIGURED). Since this is a runtime issue that is not going to correct itself automatically, it should be surfaced in not only the caller's log but also in the sidecar log.
I would like to propose the following:

All non-successful calls from application to local dapr sidecar be logged in the sidecar logs
In the case of Invoke calls, a sidecar log message should only be produced if there is an issue in calling the local sidecar (e.g., the path is incorrectly formatted). A log message should not be produced if the target of the invocation (remote sidecar) fails, but instead this error message should be passed back to the caller.

/cc @yaron2 @amanbha @msfussell

@lynn-orrell
Thanks for the feedbacks. When we worked on structured logging, we had considered log enrichment like you mentioned. However, the logs you mentioned can be duplicated with traces so that we did not improve it. But I agree the point that not all customer would enable trace and log is more beneficial during the debugging in production scenario.
Instead of additional duplicated logic, we may be able to leverage OT stdout exporter to export the trace to log.
cc/ @shalabhms

This issue is to add granular logging in case of failures in existing debug level.

Created a separate issue https://github.com/dapr/dapr/issues/1881 for log exporter , which is closed.