Warning: annotation "kubernetes.io/ingress.class" is deprecated, please use 'spec.ingressClassName' instead

Consider this bash content.
edburns@localhost target % kubectl apply -f ${DIR}/cargotracker/target/openlibertyapplication.yaml
<${DIR}/cargotracker/target/openlibertyapplication.yaml
openlibertyapplication.apps.openliberty.io/cargo-tracker-cluster created
Warning: annotation "kubernetes.io/ingress.class" is deprecated, please use 'spec.ingressClassName' instead
ingress.networking.k8s.io/cargo-tracker-ingress created

This issue asks the assignee to investigate and remedy the code in light of this apparent deprecation warning.

Fix in https://github.com/Azure-Samples/cargotracker-liberty-aks/pull/10/commits/e89dd29a132ca2873f458bdd0dca14271a5652bc


Cannot install

Hello,
I am trying to install it and the problems.
I installed Raspbian
sudo apt-get update
sudo apt-get upgrade
sudo apt-get install python3-dev python3-venv
python3 -m venv env
env/bin/python -m pip install --upgrade pip setuptools
git clone https://github.com/Jerrkawz/GarageQTPi.git

Next, I am trying to run: pip install -r requirements.txt, I receive the following error:
Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'
Storing debug log for failure in /root/.pip/pip.log
After that, I figured out that I have to go to GarageQTPi directory first and run it from that directly. This is the output and error:
pi@Garage-Door:~$ pip install -r GarageQTPi/requirements.txt
Downloading/unpacking paho-mqtt==1.2 (from -r GarageQTPi/requirements.txt (line 1))
Downloading paho-mqtt-1.2.tar.gz (49kB): 49kB downloaded
Running setup.py (path:/tmp/pip-build-xftl86/paho-mqtt/setup.py) egg_info for package paho-mqtt
Downloading/unpacking PyYAML==3.12 (from -r GarageQTPi/requirements.txt (line 2))
Downloading PyYAML-3.12.tar.gz (253kB): 253kB downloaded
Running setup.py (path:/tmp/pip-build-xftl86/PyYAML/setup.py) egg_info for package PyYAML
Installing collected packages: paho-mqtt, PyYAML
Running setup.py install for paho-mqtt
error: could not create '/usr/local/lib/python2.7/dist-packages/paho': Permission denied
Complete output from command /usr/bin/python -c "import setuptools, tokenize;file='/tmp/pip-build-xftl86/paho-mqtt/setup.py';exec(compile(getattr(tokenize, 'open', open)(file).read().replace('\r\n', '\n'), file, 'exec'))" install --record /tmp/pip-tJzjZ4-record/install-record.txt --single-version-externally-managed --compile:
running install
running build
running build_py
creating build
creating build/lib.linux-armv7l-2.7
creating build/lib.linux-armv7l-2.7/paho
copying src/paho/init.py -> build/lib.linux-armv7l-2.7/paho
creating build/lib.linux-armv7l-2.7/paho/mqtt
copying src/paho/mqtt/client.py -> build/lib.linux-armv7l-2.7/paho/mqtt
copying src/paho/mqtt/subscribe.py -> build/lib.linux-armv7l-2.7/paho/mqtt
copying src/paho/mqtt/init.py -> build/lib.linux-armv7l-2.7/paho/mqtt
copying src/paho/mqtt/publish.py -> build/lib.linux-armv7l-2.7/paho/mqtt
running install_lib
creating /usr/local/lib/python2.7/dist-packages/paho
error: could not create '/usr/local/lib/python2.7/dist-packages/paho': Permission denied

Cleaning up...
Command /usr/bin/python -c "import setuptools, tokenize;file='/tmp/pip-build-xftl86/paho-mqtt/setup.py';exec(compile(getattr(tokenize, 'open', open)(file).read().replace('\r\n', '\n'), file, 'exec'))" install --record /tmp/pip-tJzjZ4-record/install-record.txt --single-version-externally-managed --compile failed with error code 1 in /tmp/pip-build-xftl86/paho-mqtt
Storing debug log for failure in /home/pi/.pip/pip.log
Can you help?
Thank you.

You either need to run it within the venv you installed earlier, or sun pip as root "sudo pip..."

sudo worked for me, thank you.


Kernel Crash when global_extent=False

Following the example code here: http://geoviews.org/gallery/bokeh/world_population.html#bokeh-gallery-world-population
I modified for my data. It works fine if global_extent=True, but crashes the kernel when global_extent=False.
gps_gv = gv.Dataset(df_gps, kdims=['raw_theta'])
points_gps = gps_gv.to(gv.Points, ['raw_lon', 'raw_lat'], ['raw_theta'])

tiles = gv.tile_sources.Wikipedia

tiles * points_gps.opts(global_extent=False,width=600, height=600)

Python 3.6, Package versions:
geoviews                  1.6.2                      py_0    pyviz
holoviews                 1.12.1                     py_0    pyviz

Thanks for the issue, sorry I didn't reply earlier. This is likely related to the way you installed gdal, GEOS or one of the other C-dependencies. So without more detail on how you installed things we can't help.


Add missing config params for button 1 (ZDB5100)

Copied from params for button 2, matches what is written here: https://logic-group.com/download/3571/

@blhoward2 looks like this needs some cleanup too.

@kvs We've merged some detailed guidance on adding device files since you pushed this, including a detailed style guide. Please take a looka nd change this file to match the style guide if you'd still like this merged. https://zwave-js.github.io/node-zwave-js/#/


NativeSocket's read() cannot read more all the bytes we want at once

I've encountered this problem on Windows XP, php 5.2.10 - jobs sized more than 4096 bytes and less than 8192 (due to previous bug fix) cannot be read - exception "Pheanstalk_Exception_ClientException: Expected 2 bytes of CRLF after 5000 bytes of data" is throwed.
I get this problem randomly on Debian Lenny, php 5.2.6 - exception is throwed really randomly, but always with job size less than 8192 bytes.
I've forked pheanstalk into http://github.com/SlNPacifist/pheanstalk, wrote a test for this bug (trying to put, peek and delete jobs with size incrementing from 1000 to 16000 with step 1000) (will commit soon).
Expecting getting rid of "read-at-once" optimisation to be the solution.

Hi
I had issue with server hence i restarted the server.
But from the time i am reciving this error Expected 2 bytes of CRLF after 26787 bytes of data
Pheanstalk.php(533): Pheanstalk_Connection->dispatchCommand(Object(Pheanstalk_Command_ReserveCommand))
I have restarted the Beanstalkd and the instance of worker process.
But still the issue.
I am using
Beanstalkd V1.4.6
Pheanstalk V2.0.0


fix(build): change if condition to faill over no credentials endpoint

@bacongobbler can you review?


fix: Fix unreleased stack overflow on statechanged

Do not dispatch statechanged events unless there is an actual change. This fixes a stack overflow in the Cast proxy code triggered by statechanged events.

Fixing the bug was fast.  Figuring out how to write a regression test that fails without the fix... that took significantly longer.  😁


Proper McMMO not supported? (Can It be? :D)

What does your feature do?
allows the full official version of McMMO get support
https://www.spigotmc.org/resources/official-mcmmo-original-author-returns.64348/
How would people use it?
same as the classic McMMO support currently with the plugin.

There is no “proper” version of mcMMO. There is only the overhauled, premium version, and the classic, free version. To use Quests with the overhauled version, you need https://www.spigotmc.org/resources/mcmmo-overhaul-quests-module.69548/

oh oups I say it written on the classic page as "official" and mislabeled it here as proper. I'm using the premium one by the original author ^^ My mistake. So this module will work for all of them or just overhauled? Bc i have only the Official premium version and in the quest editor where it lets me use McMMO features it says "Not Installed"

The module can only be used for the premium version. Once you install the module, look under “Custom objectives”
Sent with GitHawk


Include File size and File type in native path selection logic

Please describe the feature you'd like to see
Currently, we are only using Source and Destination to determine the transfer logic to be executed - default path / native path.
There are issues with this approach

File Types - We are assuming the native transfer is supported for all file types. This is not the case with Snowflake and Bigquey.
File Size - When transferring data from the S3 to Bigquery, we create a Data Transfer Job in Biquery. Which is overkill for the smaller files.

Describe the solution you'd like
When we include the File Type and File Size in the decision to select a native or default path. We can better select the path which can lead to a better user experience and faster transfer.
Acceptance Criteria

[ ] All checks and tests in the CI should pass
[ ] Unit tests (90% code coverage or more, once available)
[ ] Integration tests (if the feature relates to a new database or external service)
[ ] Example DAG
[ ] Docstrings in reStructuredText for each of  methods, classes, functions and module-level attributes (including Example DAG on how it should be used)
[ ] Exception handling in case of errors
[ ] Logging (are we exposing useful information to the user? e.g. source and destination)
[ ] Improve the documentation (README, Sphinx, and any other relevant)
[ ] How to use Guide for the feature (example)

We have implemented 1. (File Types - We are assuming the native transfer is supported for all file types. This is not the case with Snowflake and Bigquey. ) already. We can close this and consider later if this is reported by user.


added travis CI infrastructure

Should only run simple_axpy_test only at the current stage.

Looks good to me!


The program auto close when opning

The program auto close when opning it jsut close i tried opening it with adminstration but nothing

@Piratealgeria That happens if you don't have proper .NET 5.0 Runtime installed. Try downloading "Self-contained" version of the downloader or installing https://dotnet.microsoft.com/en-us/download/dotnet/5.0/runtime


How should datasets be added

I'm stuck in the add data set step and it keeps reporting an error

I believe I remember you should create another folder outside of the train folder.
open_images/train/train/all_images


update "your collective is on its way" email

Now collectives are live by default (except tiers). We need to update the text of the email. We currently say:
Thanks for signing up your GitHub repository: hledger. We are reviewing new requests as fast as we can.
While you wait, check out other open source collectives and see how they are spending the money they have raised.
Stay tuned for updates.
Suggestion:
Your collective is now live (link to collective) and under review. While you wait for approval to start receiving donations, here's a quick guide to make the most out of it.

Make your collective your own. Login and hover over elements to edit them.

Add backers/sponsors to Github/Website. We'll make a PR to your README to add backers & sponsors. Instructions to add your backers to a website here (link).

Start tracking expenses. Submit expenses so others can see your costs. It helps to raise money.

Spread the word: if you have a twitter account, you can connect it to your collective and automatically thank your backers. See here how to do it. (link)

Find us on slack.opencollective.com or<EMAIL_ADDRESS>

@piamancini what are the links that you share in #2 and #4 above.

Found link for #4. Here's PR in progress: https://github.com/OpenCollective/opencollective-api/pull/741

@asood123 link for #2 is https://github.com/OpenCollective/OpenCollective/wiki/Website-banner


Add children terms for new "neurodevelopmental disorder" class

A new term was created in #3410 : "neurodevelopmental disorder"
We need to add children to this new class.
Proposal:

SPARQL query: find all terms with the words "neurodevelopmental disorder" in

term label
definition

use ROBOT to update the parentage of these terms

@nicolevasilevsky , do you agree with this plan?

Yes, I love this plan!

see Ada's comment as well: https://github.com/monarch-initiative/mondo/issues/3410#issuecomment-924339251

discussed on Mondo call, sounds great!


DBZ-1517 Skip unknow records while operation header is used

https://issues.jboss.org/browse/DBZ-1517

@jpechane, seems worth and at the same time low-risk enough for including into the final?

@gunnarmorling I belive this is really low risk bug so it could be incorporated.

Applied; thanks, @jpechane!


Cannot update query without specifying a tag. ValidationError: "0" must be a string.

When updating a query I created, I found the following error in the logs:
{ ValidationError: child "tags" fails because ["tags" at position 0 fails because ["0" must be a string]]
    at Object.exports.process (sqlpad/node_modules/joi/lib/errors.js:154:19)
    at _validateWithOptions (sqlpad/node_modules/joi/lib/any.js:601:31)
    at root.validate (sqlpad/node_modules/joi/lib/index.js:105:23)
    at QuerySave [as save] (sqlpad/models/Query.js:56:23)
    at sqlpad/routes/queries.js:148:11
    at sqlpad/models/Query.js:101:12
    at newArguments.(anonymous function) (sqlpad/node_modules/nedb/lib/executor.js:29:17)
    at Cursor.execFn (sqlpad/node_modules/nedb/lib/datastore.js:518:14)
    at callback (sqlpad/node_modules/nedb/lib/cursor.js:126:19)
    at sqlpad/node_modules/nedb/lib/cursor.js:193:12
  isJoi: true,
  name: 'ValidationError',
  details:
   [ { message: '"0" must be a string',
       path: 'tags.0',
       type: 'string.base',
       context: [Object] } ],
  _object:
   Query {
     _id: 'pM2qrGxptUUvsqso',
     name: ‘Query name’,
     tags: [ null ],
     connectionId: 'Z3rtugCVItxwhrZf',
     queryText: ‘query name’,
     chartConfiguration: { chartType: 'line', fields: [Object] },
     createdDate: 2016-07-22T23:19:19.582Z,
     createdBy: ‘email’,
     modifiedDate: 2017-01-11T22:54:53.077Z,
     modifiedBy: ‘email’,
     lastAccessDate: 2017-01-11T22:54:53.077Z },
  annotate: [Function] }

Fix is to add a tag to the query and then save again.

Thanks for the report @dave-elkan - Out of curiosity was this query created in sqlpad v1?
I'm not able to reproduce this in a fresh v2 install. There was a lot of weirdness happening with the old tagging input though, and I know there was some junk data on the backend that went along with it.
I'll have to add some data cleansing for the tags to remove any null/non-string values.

I think you're right there.
I just tried saving a query which I was definitely created in v1 and the error occurs. New queries do not exhibit this behaviour.


Setting any http headers results in immutable issue.

RN Version: 0.42.0
RNFB Version: 0.10.2 (latest)
Issue on: iOS only. (Android working)
NOTE: RNFB version 0.10.0 is working.
Problem:

Code:
this.http_headers = {
  'authorization': 'Bearer ' + this.props.user.token,
  ...Network.device_info_headers
};

RNFetchBlob.config({session: 'images', fileCache: true, appendExt: 'png'})
  .fetch('GET', this._signatureUrl(intervention.user_sign_logo_id), this.http_headers)

@Kaishley , I've fixed this issue. You can now install the package from branch issue-290 and verify if it works 👍

Getting new errors on both devices now.
iOS

Android

Okay, I've just changed the way it process headers, please try again, thanks.

Working on both devices. Cheers.


Autopublish for release branches

I'm curious, how do you plan to handle release branches and publishing from these?

I'm curious, how do you plan to handle release branches and publishing from these?

we already have a setup itself working for publishing versioned docs using branches (using mkdocs and https://github.com/jimporter/mike ), the only thing missing is to run corresponding job in CI to publish from the release branches, right now it's only happening from the master


Improved windowing

This improves Pie.Windowing, moving to SDL-based windowing, as well as moving away from WindowSettings and instead to a WindowBuilder. This also changes how window events are processed. This moves the library more inline with something like winit or sdl2-rs.

This PR doesn't work on systems without SDL already installed (aka Windows) so this PR cannot be merged until either Silk updates the SDL output DLL or SDL.NET becomes a thing

OK the SDL issue has temporarily been resolved as I wrote some SDL bindings for the parts of windowing that need it.

SDL now appears to be fully implemented and now is fully working - all tests I've done have passed. As such, I will now merge this!


discussion: default to being hot

In gitlab by @rdallman on Jul 18, 2017, 13:12
Like Tom Brady's children?
We've discussed this many times but there doesn't seem to be a card for it (I looked). It seems like it would be possible to default to 'hot' regardless of format. Bear with me, but poke holes if there are blatant issues. This takes care of 2 'issues', one being having to use the http format (yes, even with easy to use wrappers) and the other is internally having to treat hot and cold differently (in our server code). Defaulting to 'hot' will still basically entail the user having their code a certain way (ideally, using our wrapper), but we don't really have to treat the two that much differently.
Ultimately, we can run every container as 'hot', it's just that if not coded as such the container will so happen to exit (i.e. if they don't have a for loop / our wrapper). what this means is that as long as we don't queue up work against any container, expecting it to run more things, then that's ok and the container can exit (how it is now), it's just not optimal -- but if the code isn't there to do it, then whatever, not much we can do. It would be possible for us to use docker exec to change the env vars between executions with the default format, and it would be possible to frame STDIN with EOFs, are there other concerns here for default format?
Would this give us a lot of sugar? I think this would give us two things really:

not having to make the distinction whether a function is 'hot' or 'cold' on the server. that is, not having to change the configuration for a route at all (where 'http' => 'hot'). if the code happens to loop, great. if not, nobody cares but things are a little slower. this would make it so if the user just updates their code they don't have to change 'format' on their route.
formats are less important as a distinction for hot/cold. as long as we say we frame 'default' with EOF then we can use it for 'hot'. obviously with wrappers this isn't such a big deal, but it's a little bit 'magic' to have to set your format to 'http' even though your code isn't explicitly doing http parsing (the wrapper hides it) and it's possible right now even to have a "cold" function that talks the 'http' format (just exit instead of looping, we can't prevent this). this also means in the server code we can hit the same execution path for any function invocation (instead of switching on format "type").

possible cons:

it will be hard to tell whether a function is 'hot' or 'cold' based on format alone, which the argument could be made is already the case with 'http' anyway, since the user can just exit (obviously, the question is, why would they? but ignore that, it's only important that it's possible is the point).
we're not really defaulting to 'hot' since the code has to be a certain way, maybe that could be confusing

related: https://gitlab-odx.oracle.com/odx/functions/issues/71

closing for #785 discussion


Build Error -  cannot find symbol DartUiIsolatePlugin

I tried switching from Flutter Isolate to this new package, but get the following error when building the project.
error: cannot find symbol
flutterEngine.getPlugins().add(new com.lib.dart_ui_isolate.dart_ui_isolate.DartUiIsolatePlugin());
^
symbol:   class DartUiIsolatePlugin
location: package com.lib.dart_ui_isolate.dart_ui_isolate

ya i didn't test android yet.
wanna help? open a pr?

ok i'll fix it.
you can copy any missing code from flutter_isolate
The main reason for this package is that it supports FlutterEngineGroup which saves 100s of megabytes of memory.
So this package will be better than flutter_isolate.
The isolates also spawn much faster.


Failures in the report for mixed nodes are reported incorrectly

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

On a cluster(3 etcd, 2 control plane and 3 worker nodes), run scans
Failure is reported for a benchmark for the etcd nodes when checked in the report generated in the API.

"n": [
          "etcd1-ip-addr",
          "etcd2-ip-addr",
          "etcd3-ip-addr"
         ]

The benchmark test is applicable for master and etcd nodes.
Expected Result:
Only etcd nodes should be reported as failure on the UI
Actual Result:

All nodes, master and etcd are reported as failure in the report on the UI.
5 nodes - 3 etcd and 2 control plane nodes are shown failed for this benchmark test

Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head latest commit id-df31802ac
Installation option (single install/HA): single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): custom
Kubernetes version (use kubectl version):

1.16

Per convo between @codyrancher and @leodotcloud, this is a backend issue.

@leodotcloud when it's fixed, let's move this to team/ui for @codyrancher to pick it up.

When the status of the test is "M", the API does not return the set of failed nodes.

This issue is seen when both master and etcd roles are on the same node. Will fix it. @codyrancher you can continue to fix the UI code without waiting for the backend fix by creating a split role cluster.

@leodotcloud Logged a separate issue - https://github.com/rancher/rancher/issues/24540 to track the one mentioned here https://github.com/rancher/rancher/issues/24431#issuecomment-564756298

This was a problem with a single node having multiple roles. Fixed.

Verified on master-head - commit id: 61b78ddf7
On a cluster - 8 nodes, 3 etcd, 3 worker and 2 control planes.

Run CIS Scan.
Expected one test - test1 to fail.
The state of test1: Mixed (expected)
The list of nodes shows both - list of nodes where the test failed (status: Fail) and list of nodes where the test passed (status: Pass)


A11y_ML .NET Model Builder_ Create New Experiment_ Label in Name: Visible label and accessible name (Programmatically given name) are not same for "Experiment Name" edit field

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags:
#A11yWCAG2.1; #A11yTCS; #Win32; #DesktopApp; #Feature-ML.NETModelBuilder-Win32-Nov2020; #A11ySev2; #Benchmark; #Visual Studio; #Label in Name; #WCAG2.5.3
Environment Details:
Microsoft Visual Studio 2019 Preview Version 16.8.0 Preview 6.0
Screen Readers:
Narrator; NVDA
Repro Steps:
1.Open Visual Studio Enterprise 2019 Preview
2. Create a new .Net Core Console App (C#)
3. Tab to "Solution Explorer" and right click on the project (Shift + F10) and then from the expanded list Select "Add" and then from the list items select "Machine learning"
4. Tab till "Object Detection" scenario and select the scenario
5. Tab till "Set up work Space" button and press enter
6. Open Accessibility Insights for windows and Inspect name for "Experiment name" control
Actual Result:
Visible label and accessible name (Programmatically given name) are not same for "Experiment Name" edit field
Expected Result:
Visible label and accessible name (Programmatically given name) should be same
User Impact:
Screen Reader users and users who use speech to text converter will face difficulty in using the application if visible name and accessible name are not same
Attachment for Reference

Verified the bug in the latest build and the issue is fixed.
Please find the below attachment for reference
Microsoft Visual Studio Enterprise 2019 Preview
Version 16.9.0 Preview 4.0 [30917.4.main]

#Github tag: #Closed;


[DOC] Docstring style should be made clear in contributing page

Brief Description of Fix
Currently, the docs do not clearly describe the coding standards that the project probably should adhere to.
I believe this should be added to the Contributing page.
For reference, it should look something like this:

We recommend that you write docstrings before you write a test for the function, and that you write the test before writing the function implementation. This is an example of combining doc- and test-driven development together, to clarify what the function is intended to do before actually implementing it. Use the following example to help guide you on how to write your new function docstring properly.
def new_janitor_function(df, param1, param2):
    “””
    One line that describes what the function does.

    Further elaboration on the function. Use one or more paragraphs to do any of the following:
    1. Provide more detail on intended usage
    2. Clearly state assumptions on what the input dataframe should look like.
    3. Identify edge cases that will cause the function to raise an error.
    4. Other things that you want the end-user to know about before using the function.

    Method chaining example:

    .. code-block:: python

        df = df.new_janitor_function(param1, param2)

    Functional example:

    .. code-block:: python

        df = new_janitor_function(df, param1, param2)

    :param param1: A description of param1.
    :param param2: A description of param2.
    :return: A description of the dataframe that is returned.
    “””
    # now put your function implementation here

Relevant Context

Link to documentation page
Link to exact file to be edited

@ericmjl I can work on this issue.

Ok! Thanks @shandou, I will leave it in your hands then :smile:.

This is adjacent to #357, can they be combined?

@hectormz I like that issue as well and I had the exact same questions on Friday when trying to extend a function feature (for example, it was not very clear to me at first about where the test data was loaded). If you and @ericmjl are both comfortable with me taking a stab at it, please combine the two issues.
Can't guarantee to get the skeleton examples right at first try, but I am sure you guys can help me iterate.

@ericmjl I am adding instructions about unit tests but not very certain about the use of custom marks in this particular example:
@pytest.mark.functions
@pytest.mark.test
def test_shuffle(dataframe):
    """
    Test the shuffle function.

    This test checks that the set of indices in the shuffled dataframe are
    identical to the set of indices in the original.
    """
    df = dataframe.shuffle()

    assert set(df.index) == set(dataframe.index)

Just to double-check with you:
My understanding is that having @pytest.mark.function decorator allows us to cherry-pick a subset of tests we want to run.
For example:
# Run all the tests marked with<EMAIL_ADDRESS>$ pytest -v -m functions

# Run all the tests except the ones marked with<EMAIL_ADDRESS>$ pytest -v -m 'not functions'

But it is not clear to me as to what we are doing with the additional mark @pytest.mark.test. When you have a chance, could you help me better understand this? Thanks a lot!! 😃

Also, I can see mypy as one of the dependencies, but I don't see explicit use of it in the repo. Curious about whether type checker is used during CI? And how? Thanks!

@shandou you caught a bug of mine!

what we are doing with the additional mark @pytest.mark.test

I used to use it to isolate just the test that I'm currently hacking on. It must have slipped into the library.
The correct way to isolate just the test that we are writing, borrowing the example above, is to do:
$ pytest -k test_shuffle

This will run only the tests that contain the substring test_shuffle. Usually that is good enough to avoid running the full test suite.

Also, I can see mypy as one of the dependencies, but I don't see explicit use of it in the repo. Curious about whether type checker is used during CI? And how? Thanks!

mypy is used during development. I have a PR open to use mypy in CI, but I'm still not clear what we'd be checking for though. I think we can ignore mypy for the time being.


Support for Xenomai3?

We've recently ported our project to Xenomai3 and we're having problems using g3log.  I think it probably has to do with g3log's use of std::thread and std::mutex instead of the pthread variants.
Do you have a sense of how difficult it would be for me to switch to using all pthread calls?
Thanks.
Will

Good question. If you don't have full c++11 support then you might want to consider using g2log instead.
That one is pretty frequently ported to just using pthreads and posix

This is the reply I gave to another user who asked
I would like to use g3logger but I am stuck in ancient compiler environment,
 the old sun compiler on Solaris 10. This is C++98 and unfortunately this is
not going to change. So I was wondering, would you consider making
g3logger available for C++98 compilers?

My reply was
-->

G3log,  and G3sinks are commonly using some of the latest language features in C++11 and now moving towards C++14.
For older compilers I recommend using g2log instead. If C++11 is not available then it is not a big deal to remove the C++11 features and replacing them for pthread, posix equivalents or similar functionality accessible in other libraries.
You can find g2log here  https://bitbucket.org/KjellKod/g2log
A slightly modified version (more C++11 stuff) can be found here: https://bitbucket.org/KjellKod/g2log-dev
An even more modified version, which relies on boost, and has natively log rotation (which otherwise can be used in g3log + g3sinks) can be found here: https://bitbucket.org/vrecan/g2log-dev
There are several versions of g2log existing without C++11 and instead using Qt, pthread, boost or similar. Unfortunately I do not have any repository that I can point you to since the developers that did that transformation chose to not reveal the repo's location, or kept it private :(
The process is fairly straightforward though. I might have forgotten something but in short details you should look at:
The active.h/cpp is using some C++11 constructs as well as shared_queue.h all these should be replaceable for similar versions in the libraries I mentioned above.
g2future and logworker is using a std::future. To just get going I would suggest to just remove the std::future parts. Comment them out. Once all the rest is working you can always work with enabling them again using whatever favorite or available C++ library that will do the trick for you.
Let me know how it goes,
Cheers
Kjell

Thanks Kjell,
I'll look into this.  We're using gcc, so we do have access to C++11 constructs, but the threading and concurrency appears to need to use the pthread api.

@will2142 I'm leaving this one open for a bit longer. Let me know if you have any questions that I can help you out with.

@will2142 any new status here? Should I close this?

Thanks @KjellKod.
We may have a solution that doesn't require any code changes to g3log.  If we're successful, I'll update with details.

@KjellKod I'm going to close this issue for now.  I'll update you if/when we have a solution.


RuntimeError: output with shape [1, 256, 192] doesn't match the broadcast shape [3, 256, 192]

Getting this error while running train_gmm.py

Namespace(batch_size=16, data_root='data', fine_height=256, fine_width=192, gpu_id='0', grid_size=5, log_freq=100, n_epoch=100, n_worker=16, name='GMM', out_dir='../result', radius=5)
Loading dataset
Building GMM model
Total Parameters: 19057650
Start training GMM
0% 0/100 [00:00<?, ?it/s]Epoch: 0
epoch: 0:   0% 0/1 [00:00<?, ?it/s]
0% 0/100 [00:00<?, ?it/s]
Traceback (most recent call last):
File "train_gmm.py", line 140, in
main()
File "train_gmm.py", line 127, in main
loss = trainer.train(epoch)
File "train_gmm.py", line 33, in train
return self.iteration(epoch, self.dataloader_train)
File "train_gmm.py", line 45, in iteration
for i, _data in data_iter:
File "/usr/local/lib/python3.6/dist-packages/tqdm/std.py", line 1104, in iter
for obj in iterable:
File "/usr/local/lib/python3.6/dist-packages/torch/utils/data/dataloader.py", line 345, in next
data = self._next_data()
File "/usr/local/lib/python3.6/dist-packages/torch/utils/data/dataloader.py", line 856, in _next_data
return self._process_data(data)
File "/usr/local/lib/python3.6/dist-packages/torch/utils/data/dataloader.py", line 881, in _process_data
data.reraise()
File "/usr/local/lib/python3.6/dist-packages/torch/_utils.py", line 395, in reraise
raise self.exc_type(msg)
RuntimeError: Caught RuntimeError in DataLoader worker process 0.
Original Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/torch/utils/data/_utils/worker.py", line 178, in _worker_loop
data = fetcher.fetch(index)
File "/usr/local/lib/python3.6/dist-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/usr/local/lib/python3.6/dist-packages/torch/utils/data/_utils/fetch.py", line 44, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/content/drive/My Drive/viton-gan/viton_gan/dataset.py", line 155, in getitem
data = self._get_item_base(index)
File "/content/drive/My Drive/viton-gan/viton_gan/dataset.py", line 103, in get_item_base
feature_shape_tensor = self.transform(self.downsample(shape_im)) # [-1,1]
File "/usr/local/lib/python3.6/dist-packages/torchvision/transforms/transforms.py", line 61, in call
img = t(img)
File "/usr/local/lib/python3.6/dist-packages/torchvision/transforms/transforms.py", line 166, in call
return F.normalize(tensor, self.mean, self.std, self.inplace)
File "/usr/local/lib/python3.6/dist-packages/torchvision/transforms/functional.py", line 208, in normalize
tensor.sub(mean).div(std)
RuntimeError: output with shape [1, 256, 192] doesn't match the broadcast shape [3, 256, 192]
epoch: 0:   0% 0/1 [00:00<?, ?it/s]

Has this problem been fixed?

I had this problem before but on a different repository. I fixed it by using a specific version of PyTorch. Use pytorch 0.4.1 and torchvision 0.2.1

I had this problem before but on a different repository. I fixed it by using a specific version of PyTorch. Use pytorch 0.4.1 and torchvision 0.2.1


show-filter="false" not working

When setting the show-filter and show-caption options to false, neither of them are effective.
version: "vue-table-component": "^1.7.0",
          <table-component
          :data="trades"
          cache-lifetime= "0"
          sort-by="inputDate.dateFormat"
          sort-order="asc"
          show-caption="false"
          show-filter="false"
          filter-no-results="Sorry no results have been found">
          <table-column show="inputDate.dateFormat" label="Input Date" data-type="date:DD/MM/YYYY"></table-column>
          <table-column show="tradeNo" label="Trade No" data-type="numeric"></table-column>
          <table-column show="description" label="Description" :filterable="false"></table-column>
        </table-component>

also when trying:
import TableComponent from 'vue-table-component';
Vue.use(TableComponent);

TableComponent.settings({
    showFilter: false,
});

Many thanks,
G

ahh I see I had to use :show-filter="false" not show-filter="false"


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/57327

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/60385

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/60899

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/79480

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/103030


Per-neuron vs. population co-bps

Hello!
We recently implemented the co-bps metric described in the NLB paper. In the paper, the bps is described as a per-neuron metric where a neuron's likelihood is normalized and divided by its number of spikes. In our codebase, we therefore compute the bps per neuron and then average the final result (permalink).
We noticed in the nlb_tools repo that the co-bps is computed for all heldout neurons simultaneously which means that the total likelihood is normalized and divided by the total spike count of all heldout neurons (permalink).
We found the per-neuron implementation of the bps leads to very different results than the population bps implementation. The per-neuron bps is more sensitive and leads to more negative values for poorly reconstructed neurons. Also, it seems like the population bps is not affected much by low firing rate neurons.
We were curious your thoughts on these two bps implementations and if you considered doing a per-neuron bps.

Hi! As I recall, we did consider a per-neuron bps measure instead of our population bps measure, and we agree that the per-neuron measure is stricter than the population measure. However, due to the relatively small number of held-out neurons and trials for some datasets, we opted for the population measure on all datasets, hoping it would be a bit less noisy.


cdeco: why group==0 - group==1, not vice versa?

I just want to point out, that defining
$$QE(\tau) = Q_{<0,0>} - Q_{<1,1>}$$
Is very confusing and breaks the usual convention people use, when considering treatment effects:
$$\tau = \mathbb{E}[Y(1)] - \mathbb{E}[Y(0)]$$
https://github.com/bmelly/Stata/blob/b718c3f252b4729c0f04325dc04f4986be2b78ad/cdeco.ado#L200
This leads to code output, which can seem counterintuitive at first, eg consider the following code and output:
E.g. consider the following code and it's output:
cdeco Wages2000 Wages1998, group(black)

---------------------------------------------------------------------------------
                                                   |          P-values
Null-hypothesis                                    | KS-statistic   CMS-statistic
---------------------------------------------------+-----------------------------
Correct specification of the parametric model 0    |      .19            .78
Correct specification of the parametric model 1    |      .33            .41
Differences between the observable distributions   |
    No effect: QE(tau)=0 for all taus              |        0              0
    Constant effect: QE(tau)=QE(0.5) for all taus  |        0              0
    Stochastic dominance: QE(tau)>0 for all taus   |      .91            .91
    Stochastic dominance: QE(tau)<0 for all taus   |        0              0
Effects of characteristics                         |
    No effect: QTE(tau)=0 for all taus             |        0              0
    Constant effect: QE(tau)=QE(0.5) for all taus  |      .01            .01
    Stochastic dominance: QE(tau)>0 for all taus   |      .94            .94
    Stochastic dominance: QE(tau)<0 for all taus   |        0              0
Effects of coefficients                            |
    No effect: QE(tau)=0 for all taus              |        0              0
    Constant effect: QE(tau)=QE(0.5) for all taus  |      .37            .33
    Stochastic dominance: QE(tau)>0 for all taus   |      .91            .91
    Stochastic dominance: QE(tau)<0 for all taus   |        0              0
---------------------------------------------------------------------------------

(Falsely) assuming that the difference in observed distribution is black=1 - black=0 give a (wrong) result that black wages FOSD white wages
Also the convention used in this package is not alligned with other packages in this repository and with the main paper:

It is also not consistent with analogous result from qrprocess package:
qrprocess wages_pcy2000 wages_pcy1998 black, quantiles(0.01(0.01)0.99) functional

------------------------------------------------------------------------------
                                                   |          P-values
Null-hypothesis                              Coef. |   KS-stat.     CMS-stat.
---------------------------------------------------+--------------------------
No effect: beta(tau)=0 for all taus                |
                                    wages_pcy1998  |    0.000         0.000
                                            black  |    0.030         0.000
                                            _cons  |    0.000         0.000
                                       all slopes  |    0.000         0.000
Constant effect: beta(tau)=B for all taus          |
                                    wages_pcy1998  |    0.000         0.000
                                            black  |    0.010         0.070
                                            _cons  |    0.000         0.000
                                       all slopes  |    0.000         0.000
Positive effect: beta(tau)>=0 for all taus         |
                                    wages_pcy1998  |    1.000         1.000
                                            black  |    0.010         0.000
                                            _cons  |    1.000         1.000
                                       all slopes  |    0.040         0.000
Negative effect: beta(tau)<=0 for all taus         |
                                    wages_pcy1998  |    0.000         0.000
                                            black  |    0.960         0.980
                                            _cons  |    0.000         0.000
                                       all slopes  |    0.000         0.000
Location-scale shift                               |
                                    wages_pcy1998  |    0.070         0.230
                                            black  |    0.400         0.270
                                       all slopes  |    0.070         0.220
------------------------------------------------------------------------------

Shouldn't the convention in cdeco package be changed?

I agree with this comment. I will try to find the time to do it in the following weeks. You already get the "correct" results simply by multiplying all the effects by -1 (or by multiplying the outcome by -1). I also need to add an option for the definition of the counterfactual outcome (X from group 0 and coefficients from group 1, or the opposite).


Adding unity schemas

Codecov Report
Patch coverage: 76.71% and project coverage change: -0.44 :warning:

Comparison is base (24be235) 80.88% compared to head (787fa53) 80.45%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   80.88%   80.45%   -0.44%
==========================================
  Files          18       20       +2
  Lines         633      706      +73
==========================================
+ Hits          512      568      +56
- Misses        121      138      +17

Impacted Files
Coverage Δ

...ython/resources/workspace/unity_catalog/schemas.py
64.70% <64.70%> (ø)

...thon/api_client/workspace/unity_catalog/schemas.py
86.48% <86.48%> (ø)

...ython/api_client/workspace/unity_catalog/client.py
100.00% <100.00%> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Problems downloading big packages?

Please see https://github.com/Microsoft/visualfsharp/issues/3253
The package I just had problems with was: StrawberryPerl<IP_ADDRESS>.1
Basically it seems like the NuGet server sends 504 responses on this package, or even stops responding from time to time...

This issue can prevent people from successfully contributing to the F# compiler and tools.  Not sure what the solution here is, but it would be great to get it looked at.

Issue moved to NuGet/Home #5491 via ZenHub


Receiving NPE immediately after '@enduml'

Using Grails 2.4.4 on Windows, runtime ":uml-class-diagram:0.4.4"
Receive the following error after selecting the 'draw' action:
@enduml
Error
|
java.lang.NullPointerException
Error |
at net.sourceforge.plantuml.svek.image.EntityImageClass.(EntityImageClass.java:79)

Hello
Can you dig more into the cause of the error ?  If you're using version 0.4.4, you are using graphviz 8000, which is http://grepcode.com/file/repo1.maven.org$maven2@net.sourceforge.plantuml$plantuml<EMAIL_ADDRESS>The code you refer to is
'''
this.shield = version.useShield() && entity.hasNearDecoration() ? 16 : 0;
'''
which points to a null entity or a null version... Really strange, I can't work it out further.
If you have the UML Script (between those @startuml and @enduml), you can try it out on http://www.plantuml.com/plantuml/form - does the script cause the error?


[BUG]Build Failure in Microsoft.Azure.Management.Monitor.Tests

Describe the bug
useCommonAlertSchema not specified in EmailReceiver constructor.
Exception or Stack Trace
Scenarios\ActionGroupsTests.cs(122,29): error CS7036: There is no argument given that corresponds to the required formal parameter 'useCommonAlertSchema' of 'EmailReceiver.EmailReceiver(string, string,
 bool, ReceiverStatus?)' [C:\Git\azure-sdk-for-net\sdk\monitor\Microsoft.Azure.Management.Monitor\tests\Microsoft.Azure.Management.Monitor.Tests.csproj]

To Reproduce
Remove <ExcludeFromBuild>true</ExcludeFromBuild> and
<ExcludeFromTest>true</ExcludeFromTest> from Microsoft.Azure.Management.monitor.Tests
Attempt build using msbuild build.proj /t:Runtests /p:Scope=monitor -v:m
Expected behavior
Build Fails

Build was failing before repo restructure.

Fixed: https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/monitor/Microsoft.Azure.Management.Monitor/tests/Microsoft.Azure.Management.Monitor.Tests.csproj#L12


aegisub: update OpenAL build for new Darwin SDK

aegisub has the option to be built with or without OpenAL. On Darwin, this was done by explicitly importing the OpenAL framework from the SDK (and openal on other platforms).
The new Darwin setup imports the whole SDK, including OpenAL. So we need to explicitly exclude it when building without openal support.
Based on this review comment by @tjni
Darwin migration Tracker: #354146
IMPORTANT
Merge #355564 first.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Please put the formatting in a separate commit.

I had to move the patch into the upstream.


fix(controllers) expand TypeMeta population

What this PR does / why we need it:
Expands TypeMeta population (see earlier work). Predicting where we'll need this isn't always easy, so just stuffing it in every controller to be safe.
Add TypeMeta population to controller reference predicates.
Use a scheme-based helper to populate TypeMeta from a runtime.object instead of local GVK helpers or pre-defined objects.
Adds a test to confirm credentials update properly.
Which issue this PR fixes:
The added test breaks without the predicate changes. Updating a credential Secret (and presumably other referenced Secrets) after its initial load would not modify Kong configuration.
We add referent Secrets directly to the store from reference updaters, This populates the Secret initially, but future updates need to run through the Secret reconciler. The reconciler predicate was not building the reference key properly without TypeMeta and would never reconcile Secrets with references.
Special notes for your reviewer:
Ideally we could centralize TypeMeta injection, but I don't think we can (barring an upstream fix). I initially tried adding it to store.Add(), but this doesn't work for anything that might need it in the reconciler. AFAIK there's nowhere in controller-runtime that'd let us add a global preprocessing step. Failing that, we can add it in either resource-specific functions or the key generator. The latter is more universal (assuming we don't end up with custom key formats for other types) if a bit cumbersome due to a new error return.
Trying to load manager configuration outside the manager results in a package import loop that I'm not particularly inclined to untangle. Removing feature gates for the scheme builder should be fine, since simply being aware of types doesn't do anything on its own.
This came out of research on https://github.com/Kong/kubernetes-ingress-controller/issues/4672, but I'm still not fully sure why outdated CRDs trigger this. I suspect that the issue is somewhere in the reference builder store update since the predicate was never working, and that updater is seemingly the only way into the store that bypasses the reconciler.
Sadly, my reproduction steps (disable CRD installation in the test harness, helm install wat kong/kong --version 2.13.1; helm delete to get the old CRDs, install consumer group CRD and GWAPI) stopped reproducing the issue after I located this second store insert path, so I wasn't able to explore it fully.
PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[ ] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

https://github.com/Kong/kubernetes-ingress-controller/actions/runs/6399242484/attempts/1?pr=4767 failed, but looks like maybe a flake. Doesn't look like it should be particular to router flavor.
Re-running but not staying around to confirm, :crossed_fingers:
If it works, this is ready for review.

I'm not exactly sure I understand the "why" of this PR 🤔
I can see 2 paragraphs in the PR description

This came out of research on https://github.com/Kong/kubernetes-ingress-controller/issues/4672, but I'm still not fully sure why outdated CRDs trigger this. I suspect that the issue is somewhere in the reference builder store update since the predicate was never working, and that updater is seemingly the only way into the store that bypasses the reconciler.

Sadly, my reproduction steps (disable CRD installation in the test harness, helm install wat kong/kong --version 2.13.1; helm delete to get the old CRDs, install consumer group CRD and GWAPI) stopped reproducing the issue after I located this second store insert path, so I wasn't able to explore it fully.

But those don't really explain why would we need this change. Can we elaborate more on the "why" in this PR? Would it be a big effort to work towards the reproduction of the issue that gets fixed here?

But those don't really explain why would we need this change. Can we elaborate more on the "why" in this PR? Would it be a big effort to work towards the reproduction of the issue that gets fixed here?

It's in the "What this fixes". If you created a Secret with a credential, it would load into the store initially, but the controller wouldn't actually track updates to it. If you attempted to change a basic-auth password or similar, your old password would continue being valid, and your new one would not work. The test changes reliably fail on existing code.
There was a separate issue where having old CRDs would prevent the initial load also. This is the part that I can't replicate reliably. It's odd, and I'd like to understand it, but it has a known fix that you should do anyway (run with the current CRDs). Absent reliable repro steps that unfortunately remains a curiousity.


Delete a rapid review (not a CoC violation)

Likely we will have to deal with users who submitted the review as a mistake. We as admin need a way to delete content that is not then labled as CoC violation.
@sballesteros does this require development separate from what we already have?

Right now this can be done in 2 steps:

admin report the review (as a normal user would do)and specify a report reason
from the moderation panel admin moderate the review and specify the moderation reason

So deleting content is not automatically recorded as violation of CoC in the database?

No we record a moderationReason. The value is currently plain text so could be anything. We could introduce discrete options if you prefer.

Closing in favor of #59


chef-server-ctl doesn't show admin status

Right now it is impossible to see which user has admin rights to a given organisation.
This was originally reported as https://github.com/chef/chef-manage-issues/issues/50 by @runningman84. It has been moved here since chef-server-ctl falls under the chef server and not manage.

Thanks for moving this over @mmzyk. We'll have to think about this one.  An admin user of that org should be able to do the following:

Install knife-acl
knife actor map
knife group show admins

To see all members of the admin group.  I agree that it seems like a common administrative task that you might want to do with chef-server-ctl, but I think we'll need to put a little though into how we avoid recreating tons of knife commands in chef-server-ctl.

@stevendanna @mmzyk My initial thoughts are that anything requiring pivotal should be chef-server-ctl.  Everything else should probably just be a knife command run from the workstation using the user key.

@ryancragun @mmzyk As the primary maintainer of knife-opc and knife-acl, I can confidently tell you that the plan is as follows:

Create knife commands in core Chef for /all/ API operations, even those requiring pivtoal.
Provide chef-server-ctl analogs to those commands (using the Chef libraries) for commands requiring pivotal.


this is undefined

Hello. When I use "this" in callbackOnInit, this is undefined.
I used webpack with babel. Code after compile:
var _this = this;

$(document).ready(function () {
  var choices = new Choices('select', {
    searchEnabled: false,
    itemSelectText: '',
    callbackOnInit: console.log(_this)
  });
});

You need to provide callbackOnInit with a function, you are providing it the result of a console log which is always undefined


lucid bumbles

I am trying to use vcpkg for a protobuf c++ library. That seems okay so far.
But I also use protoc in my build process, that is the protobuf compiler.
I had a custom FindProtobuf that had some functions that did the compilation with protoc. It declared a function called protobuf_generate_cpp. This is missing when I use:
set(CMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake)
I have this in my CMakeLists.txt:
set(CMAKE_MODULE_PATH "${PROJECT_SOURCE_DIR}/scripts/cmake" ${CMAKE_MODULE_PATH})
which defines the function but it doesn't take effect when I use
CMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake
It used to take effect but it doesn't now that I'm using vcpkg.
I see that there is a file in the vcpkg installed directory
vcpkg/installed/x64-windows/share/protobuf/protobuf-module.cmake
with the comment:
This file contains backwards compatibility patches for various legacy functions and variables
It seems to support the functionality I want.
How do I get my cmake scripts to see it and use it?
thanks.

Now, you can use the following code to use it:
    find_package(protobuf CONFIG REQUIRED)
    target_link_libraries(main PRIVATE protobuf::libprotoc protobuf::libprotobuf protobuf::libprotobuf-lite)


Add make to gnu-utility commands

Please be sure to check out our contributing guidelines
before submitting your pull request.
Fixes
Adds a make utility to wrap the gmake command provided by brew install make

Looks good to me! Thanks for your contribution!


Bash-Email script

This is a simple post-build script for a bash email, along with a readme.md

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

This is great, can you please incorporate the README in the script itself as comments, it's more on point in case people miss it.

And add an entry to the table of content in the root README.


Added Static Code Analysis

Static Code Analysis
This PR does the following:

Adds static code analysis (MISRA C)
Added static code analysis to CI pipeline
Added lots of documentation

Self Checklist

[x] I have tested that the new behavior works
[x] I have formatted the code with make format

Closed because of scope creep, reformatting into a new PR


Error: unknown metric function

loaded_model stored under cache(redis) whenever required trying to get that loaded_model (Keras Instance) But gives error.

Hi @saifxd7-
Can you provide the code for reproducing issue for helping to resolve it?


Usermod

16x16 WS2812B panel animations: Usermod for WLED - Plasma, Snake Game, Sprites and more
Based on https://github.com/s-marley/ESP32Matrix
See video: https://www.youtube.com/watch?v=9BVHkfE_R_8&feature=youtu.be
FIXME: Draft as it based on codegen-html branch, which is not yet merged into WLED.

Are you on Discord? We have forked WLED and added Audio Reactive functionality. We're looking to incorporate matrix support and would love to chat with you!


Android: NetworkCallbackConnectivityReceiver does not report "none" status correctly

Environment
React Native Environment Info:
    System:
      OS: macOS 10.14.3
      CPU: (12) x64 Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
      Memory: 3.03 GB / 32.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 10.13.0 - ~/.nvm/versions/node/v10.13.0/bin/node
      Yarn: 1.15.2 - /usr/local/bin/yarn
      npm: 6.4.1 - ~/.nvm/versions/node/v10.13.0/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 23, 25, 26, 27, 28
        Build Tools: 25.0.2, 27.0.3, 28.0.2, 28.0.3
        System Images: android-28 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom
    IDEs:
      Android Studio: 3.1 AI-173.4907809
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.8.3 => 16.8.3
      react-native: 0.59.2 => 0.59.2
    npmGlobalPackages:
      react-native-cli: 2.0.1

Platforms
Android
Versions

Android: 24+ (N and above)
iOS: NA
react-native-netinfo: Any
react-native: Any
react: Any

Description
The NetworkCallbackConnectivityReceiver class does not correctly handle the none case. It should set connectionType to CONNECTION_TYPE_NONE when mNetworkCapabilities is null. This will match the behaviour of iOS and older versions of Android.
Reproducible Demo
https://github.com/matt-oakes/net-info-test

@matt-oakes Made this change and it works well with manual testing. Any plans for real tests of the Android implementation here? Feels wrong to submit a test-free PR. :)

@powdahound Thanks for that! Feel free to submit a PR. I agree that it's strange to not have tests, but it's currently pretty hard to test this with the way that Detox is built. I have some ideas for the future on how to improve that (basically allow direct access to the adb command to allow you to do anything you like with the device).


Clean Debug Messsage

Remove CW that was put for debugging purposes which no longer useful and should not be in live version anyway

Fix v3.0.8


HTTPS with custom domain

is it possible to use custom domain with https?

Hi izzuddinrazi,
Not by default, but you can do it by using CDN services like Cloudflare or Google CLOUD CDN.

As alimuzzaman said it's not possible with the tools offers in Google Colud Console (https://cloud.google.com/storage/docs/request-endpoints#cname), you also need to setup a load balancer or use a service like Coludflare.

i am using this plugin here
https://guidetour.in/media-test/
media does not show on the page but WP editor show media. Right click on page "open media on new page" then show media here "https://cdn.guidetour.in/2023/06/e1757ec8-parking-bg.ecaf4b09-1024x276.png"
Now Plugin setting i am using "Ephemeral"
Please give me a solution how to solve this (same issue on "Stateless" mode


Add container image env vars in deployment manifest

Operator marketplaces require providing a way to set the related image used by the operator in the operator's deployment manifest as per https://github.com/zachncst/om-env-example/blob/master/README.md . In the examples, default fallback image is still kept and not removed from the operator.
This change creates a new package internal/pkg/image and moves all the default container image constants to this packages. It also defines env vars for the default container images. The default image constants will now be used as a fallback when the env var image defaults are not set.
With this, the container images used in StorageOS deployment can be set in two different places. The operator's deployment manifest can be used to set the default images. But these defaults can be overridden by StorageOSCluster CR spec.images property. When none of these are set, the default fallback images are used.
Since kube-scheduler image is dynamically selected, if it is set in the env var or StorageOSCluster CR, that will be used, else, a dynamic image will be used based on the k8s version.
The latest CSV files have empty values for those image env vars at the moment. When the release-gen.sh script is run for the next release generation, they will be filled with the appropriate image env var values.

Updated release-gen.sh to remove the default images.


Add more scripts/chars to extra_chars_needed, including LGC chars.

Sorry, got git into an odd state and don't have the git-fu to clean it up.

Thanks to @moyogo for showing me how I can use branch renaming to clean this up.

@roozbeh ping?

@ jungshik, reassigned to you since roozbeh is too busy.
So far this is just a data change.  The main use of this data in nototools is by lint, it generates a list of unicode characters that need to be in the font cmap, based on the script(s) of the font. By default lint assumes all characters with that primary script or with that script in their script extensions belong in the font (except for Nastaliq/Aran, which is special-cased).  This misses common characters, as well as cases where characters from other blocks are used with a script but script extensions does not include it.  When common punctuation is used with a script I want to list it if it might at all interact typographically (e.g. kerning, positioning, or stylistic changes), this should help ensure more focus on these characters to see if they do interact.  To determine what is needed I'm looking at the unicode L2 history of the script, our existing fonts, and some other data of various degrees of reliability like cldr exemplars and web pages, it's kind of ad-hoc.

About Coptic Epact in Coptic, @roozbehp explicitly didn't want this.  He thinks they should go into a symbols font.  Some comments he made in an email thread when I was asking about Coptic EPACT and Arabic Mathematical characters:
"Coptic Epact numbers and Arabic Mathematical characters don't need different weights. So they can both be just in a single-weight symbols font."
"All in all, the Arabic fonts themselves don't need either the Arabic Mathematical characters or the Coptic Epact numbers."
"Please don't put them in Noto Sans Coptic. That would make them very hard to find. IIRC, these numbers don't have much of a relation with the Coptic script, but with the writing of the Copts in the Arabic script."

Thanks a lot for the reply.  LGTM, then.


Translated new strings for rig

Added translations for:

Inspector.Rig.CleanupBoneColliders
Inspector.Rig.GenerateDebugVisuals
Inspector.Rig.ClearDebugVisuals

(this comment will be updated if more changes will be made before this pull request's merge/closing)

Thank you for the new translations!

Thank you for the new translations!


Version should not be hardcoded from the code

Version is set by appveyor in appveyor.xml. Version is always is using ASBmajor.ASBminor.BuildNumber. BuildNumber is managed by appveyor to uniquely identify each build. Locally build versions are always <IP_ADDRESS>
This exact issue #71 was fixed before and not it has been re-introduced in commit 7dd0063463d40213b1686fc0afcdc79a77202a94 (line and line).
@paolosalvatori please see above

:white_check_mark: Build ServiceBusExplorer 3.4.42 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/759f2d8c9d by @SeanFeldman)


Check that forward HMM matches edges in ARG on export

We should have exact agreement for all those exported here.

Yeah, I think there is scope for both an "export-only-really-consistent" recombination nodes, and an "export-comprehensive breakpoint-info" function.


CacheBehaviour probably not increasing speed

Hi,
according to these articles
Custom == operator, should we keep it? Unity Blog
Checking for null references in Unity
you can speed null checks in CacheBehaviour with something like this (I use this extension):
public static bool IsNull(this UnityEngine.Object gameObject)
{
#if UNITY_EDITOR
             return gameObject == null;
#else
             return (object) gameObject == null;
#endif
}

You're probably right, but I'm not sure it's worth putting in any trickery to speed it up. At some point my true recommendation is for scripts to simply cache fields for the things they need and then all of this is a non-issue. CacheBehaviour was a quick attempt at making this easier or getting easy wins on large code bases, but it really isn't the right way to be dealing with components in Unity, IMO.


missing validation for email format

adding a function to validate the email format before submission to ensure users provide a correct email address.

please describe more

Specific email format<EMAIL_ADDRESS>If the mail is not in correct format pop up message will be displaced.

Thank you for assigning me. I will let you know the update soon.


⚠ Samsung has degraded performance

In aab2e32, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2824 ms

Resolved: Samsung performance has improved in 9f788b0.


Create Label to Link Arrow

Dear Team,
Appreciate your advice regarding the below query:
1.Please, advise how we can add labels to the linking arrow between components?
2.How to save the design as XML or JSON structure. So we can load later.

Hello. Thank you for your interest in the project.

There is no such functionality at the moment.
You can do it yourself by editing the "VueLink.vue" file and adding text output to the svg generator and calculating the position you need.
You will also need to transfer text, I think it is better to do this in the "Line" object.
And of course, you will need to store the text somewhere on the scene, so that you can save/load the scene while saving the text (if the text is dynamic).

When creating a "VueBlocksContainer" you pass a "scene" object. There it is possible to specify the "sync" modifier for a two-way communication, so the "scene" object will be updated every time it changes and can be saved in any format and loaded in the future (for example, using standard JSON methods).

<VueBlocksContainer
...
:scene.sync="scene"
...
/>```

@ghostiam
Thanks for the quick reply and support. Really appreciate it. We are studying the code and update you on the progress. Once success we can send you a pull request.


Add one component per file lint rule

This is a good idea, let's not forget to circle back to this:
https://github.com/blockstack/stacks-wallet-web/pull/1776

@kyranjamie it seems like this is unnecessary and can be closed?


Improve instructional text for API tokens

Based on feedback collected in user testing:
On the API token confirmation page, we should:

Show the username/password instructions on the API token confirmation page
Show an example .pypirc file, and explain where to put them for "entire account" tokens vs "scoped to project" tokens

On the manage project > settings page:

Add a link back to the API token creation page

FYI @justinmayer

Example help text for "whole account" (user scoped) tokens:

For tokens scoped to a specific project, what would the .pypirc file look like and where would it be located?
I assume that users can create a .pypirc file in their project root directory, but I could not find this documented anywhere.
@theacodes @di @dstufft - any ideas?

For tokens scoped to a specific project, what would the .pypirc file look like and where would it be located?

Project-scoped tokens look and are configured identically to user-scoped tokens, so the .pypirc file should look and be placed identically. The major distinction between the two will probably be in whether people use .pypirc frequently for project-scoped tokens, given that they're mostly useful for CI/CD and platforms like Travis provide separate configuration mechanisms.

Just to be clear -> if I created a project-scoped token, then I would add the details of this token into a .pypirc file at $HOME/.pypirc.  Effectively, it means that I can only have one token on my local machine at any time?
E.g. I cannot create .pypirc files that correspond to each project locally (either by naming them differently, or by placing them in the project root)?

Effectively, it means that I can only have one token on my local machine at any time?

Ah, not exactly. You would effectively have one "default" token. If you wanted to have multiple project-scoped tokens, you could do something like this:
# either your user-scoped token or your default project-scoped token
[pypi]
username = __token__
password = pypi-blah

# some other token
[whatever]
username = __token__
password = pypi-blarg

And then do twine --repository whatever to switch to the whatever token.

Thanks for your feedback @woodruffw.  I'll update the text based on your feedback and we can go from there :)

Ok, so second attempt:
User scoped token instructions

Project scoped token instructions

@woodruffw can you please review the text?
I'm thinking that this information is maybe too long for this page and could be moved to the help page instead - but I'd like to get the details nailed down first :)

Contractors on the OTF-funded work need to deprioritize work on the security features in order to ensure we complete the accessibility and internationalization work by the end of the month. Therefore, while this would be great to resolve and I recognize that Nicole may still complete it, I'm removing it from the milestone.


[Agent] Don't support MondoDB 3.2.2 client

我目前的MongoDb 驱动是3.2.2，我之前询问过新版本已经支持 2.x.x +  3.x.x+ 了。不过在实际使用当中，存在驱动包版本错误，如下：

Is there a plan to supported mongodb 3.2.2?

You could try to contribute one. The plugin supports come from the whole community. Even we want, I am not sure we have enough resources to test all.

OK, i will try


updated cosmos readme with perf and troubleshooting guide

updated cosmos readme with perf and troubleshooting guide.
Take a look at the final version here : https://github.com/kushagraThapar/azure-sdk-for-java/blob/cosmos_update_readme/sdk/cosmos/azure-cosmos/README.md#troubleshooting

/check-enforcer override


Is there any secure storage or secure local db for react-native?

I need a secure storage, or local db, or a best encryption library for react-native.
Because I want to store some keys, password inside my app.
It should be store in a secure place, here already AsyncStorage is exists, but it is not secure to save keys or ids.

AsyncStorage does not provide encryption layer. You'll have to do it yourself and then store the hashed values.
You can try with https://github.com/brix/crypto-js.

@Krizzu : I have the similar question. From Android perspective, AsyncStorage seems to be using SQLite out of the box, wondering if we can choose to use SQLCipher instead of SQLite. Or is there any plan to encrypt SQLite at rest?

There's a plan to add secure Storage for Async Storage v2.
Current version does not include any encryption selection.


Reading a file throws error or hangs

With the following code:
<?php
require "vendor/autoload.php";

$loop = \React\EventLoop\Factory::create();
$filesystem = \React\Filesystem\Filesystem::create($loop);
$loaded = [];

// I have "big_file_5.txt" and "big_file_7.txt".
// Both with about 6 MB.
foreach (['5', '7'] as $num) {
    $filename = __DIR__.'/resources/big_file_'.$num.'.txt';

    $file = $filesystem->file($filename);
    $file->getContents()->then(
        function ($content) use ($filename, &$loaded) {
            echo "Got contents of $filename\n";
            $loaded[] = $content;
        },
        function ($error) use ($filename, &$loaded) {
            echo "Error when reading $filename\n";
            echo $error;
        }
    );
}

$loop->addPeriodicTimer(1, function () use (&$loaded) {
    echo count($loaded)." loaded files\n";
    echo "tick-tack\n";
});

echo "Running. Press CTRL+C to exit\n";
$loop->run();

The output should be something along the lines of:
$ php main.php
Running. Press CTRL+C to exit
Got contents of <project_path>/resources/big_file_7.txt
Got contents of <project_path>/resources/big_file_5.txt
2 loaded files
tick-tack
2 loaded files
tick-tack
2 loaded files
tick-tack
2 loaded files
tick-tack
^C

But sometimes the Eio throws an unknown error
$ php main.php
Running. Press CTRL+C to exit
Error when reading <project_path>/resources/big_file_7.txt
exception 'React\Filesystem\Eio\RuntimeException' with message 'Unknown error calling "eio_open"' in <project_path>/vendor/react/filesystem/src/EioAdapter.php:280
Stack trace:
#0 <project_path>/vendor/react/filesystem/src/EioAdapter.php(254): React\Filesystem\EioAdapter->executeDelayedCall('eio_open', Array, -1, Object(React\Promise\Deferred))
#1 [internal function]: React\Filesystem\EioAdapter->React\Filesystem\{closure}(Object(React\EventLoop\LibEventLoop))
#2 <project_path>/vendor/react/event-loop/Tick/FutureTickQueue.php(46): call_user_func(Object(Closure), Object(React\EventLoop\LibEventLoop))
#3 <project_path>/vendor/react/event-loop/LibEventLoop.php(202): React\EventLoop\Tick\FutureTickQueue->tick-tack()
#4 <project_path>/main.php(32): React\EventLoop\LibEventLoop->run()
#5 {main}Got contents of <project_path>/resources/big_file_5.txt
1 loaded files
tick-tack
1 loaded files
tick-tack
^C

Or one or both reads hangs forever
Running. Press CTRL+C to exit
Got contents of <project_path>/resources/big_file_7.txt
1 loaded files
tick-tack
1 loaded files
tick-tack
1 loaded files
tick-tack
1 loaded files
tick-tack
1 loaded files
tick-tack
1 loaded files
tick-tack
1 loaded files
tick-tack
1 loaded files
tick-tack
1 loaded files
tick-tack
1 loaded files
tick-tack
1 loaded files
tick-tack
... (forever)

Every time I run the script I got one of the results randomly(?). I'm running Ubuntu 14.04.1 LTS.
@WyriHaximus, do you have an idea of what may be causing the problem?

Hmm interesting, I'll take a look at his, I've seen odd behavior with reading multiple files at the same time before but never tried to nail it down before because it occurred randomly. I'll give this code a shot locally and see what happens :+1: .

Hello @WyriHaximus !
I have same issue with file reading, maybe you need to add other listener to the 'end' event on stream in FileInterface::getContents function. Like this solution:
    public function getContents()
    {
        return $this->open('r')->then(function ($stream) {
            /** @var ReadableStreamInterface $stream */
            $stream->once('end', function () use ($stream) { $stream->close(); });
            return Stream\buffer($stream)->always(function () {
                $this->close();
            });
        });
    }

IIRC I fixed this locally but was hunting down some other errors popping up. I'll try to work on this during the weekend.

This can be also an issue from React\Promise\Stream::buffer function, there is no handling of 'end' event and that is strange ^_^

FYI my WIP changes: https://github.com/reactphp/filesystem/compare/master...WyriHaximus-labs:fix-test-chown-uid

Got a bit further, everything runs without issues on PHP 5 and 7 locally. So I'm currently debugging permission issues on travis

Nearly fixed all issues: https://travis-ci.org/WyriHaximus-labs/filesystem/builds/378239247
Once done I'll file a bunch of PR's for all fixes :tada:

Gave up and marked 5.4 as allowed failure. Will start creating PR's for all other fixes tomorrow

Right so PR #23, #24, #25, #26, #27, #28, #29, #30, and #31 filed :tada:


Identify where further docs on zkApp transaction API should live

In my judgement, this is the most important open doc issue we have currently, and I want to pursue it before testnet launch.
Here's a proposal:

We add a new explainer section, maybe titled "Transactions, account updates and proofs", at the bottom of "How to write a zkApp"
We give succinct descriptions of what transactions and account updates are in Mina/snarkyjs
We introduce the Mina.transaction API, in generality -- i.e. without reference to any specific flow like "interacting with Auro wallet via Mina provider" or "running a unit test with Jest". (These specialized use cases should live in later sections IMO.)
We explain how account updates are typically created under the hood by more high-level methods, and we give a concrete example: creating a payment with the send API
We explain how calling a zkApp creates an account update under the hood (which needs a proof for authorization)
We show how to use the Mina.transaction output to create a proof, and how you have to compile the SmartContract first
We hint at how the finished transaction would be used in a web app -- passed to the wallet via Mina provider and signed there. (When it exists, we would link to the Mina provider docs here.) We also explain how a transaction can be signed and sent with snarkyjs alone (hint at use cases: testing, server-side/automated interaction with Mina)

Further thing that came up:
we should document how to fetch accounts and explain the caching that we do in some places. Mina.fetchAccount vs Mina.getAccount, zkApp.x.fetch() maybe

More stuff that came up: it should be documented that the transaction code runs multiple times, so using mutated global state as method inputs is not supported


Implement C4Lock

This was tested on a Yale Deadbolt Lock with a Device Name 'Door Lock' within the Director. There was another device that had commands to control the lock as well, so this might vary between user..

@BrycesonKimura I've fixed the black formatting errors, so hopefully any future commits will pass CI. Please take a look at the changes I requested?


Drop the deprecated request packages

Re-write parts of a few test apps to make proper use of request without the need for a separate set of packages.
This is only a partial dropping of the deprecated packages as the main one is the logger-config usage of it. I have another set of changes ready to swap that over to use itwin-client's wrapper around request.

@MichaelBelousov @wgoehrig mind reviewing?

@Mergifyio update

@Mergifyio update


java example cyrillic issue

after authorization (parameters.systemLanguageCode = "ru";) in java console i`ve wrote "sm 825663194 кириллица" but message was generated with
content = MessageText {
text = FormattedText {
text = ":8@8;;8F0"
entities = Array[0] {
}
and reccived message is not "кириллица" but ":8@8;;8F0"
chat names with cyrillic simbols looks fine
how may i fix it?

#520 was vary halpful for my country, but why official telegram client works fine without proxies ...... i`m from russia

The problem is very likely to be in the way symbols are read from the console and not in the TDLib itself. You need to check that in the console settings an encoding which is able to represent cyrillic symbols is chosen. You also need to check that Charset.defaultCharset() is the same encoding that is used in the console and if it isn't then pass explicitly correct Charset to InputStreamReader.
Could you specify, which exactly console application is used and its encoding, and which charset is default on your system?

thanks a lot!! i have added to "netbeans_default_options"  "-J-Dfile.encoding=UTF-8" in my netbeans config file.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in ad3d8be after 5 minutes.


[Enhancement] Support cast string to int

Why I'm doing:
What I'm doing:
Fixes #52473
StarRocks> select cast ('123.4' as INT) ;
+-----------------------+
I CAST ('123.4' AS INT) |
+-----------------------+
|                   123 |
+-----------------------+

What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@Seaven @LiShuMing PTAL

https://github.com/StarRocks/starrocks/pull/53267/files this pr simpliy handle this case already


Getting issue while rendering table component after data updation

I am getting below error while i am rendering my componet 2nd time , at intial run its working fine. please help

My data n everything is proper....i have one trigger which will update the data i every 5 min...on senod updated data its falling n showing me above error


New package: BetaDispersion v0.1.1

Registering package: BetaDispersion
Repository: https://github.com/EvoArt/BetaDispersion.jl
Created by: @EvoArt
Version: v0.1.1
Commit: 926a2982641fa0a4aac9f0ea0642b57178e6949e
Reviewed by: @EvoArt
Reference: https://github.com/EvoArt/BetaDispersion.jl/commit/926a2982641fa0a4aac9f0ea0642b57178e6949e#commitcomment-55830565

I no longer wish to register the package in it's current format. Need to fix some bugs following a name change


Multiple result sets - "Object must implement IConvertible."

Please ignore for now. Sorry. I think it has to do with a column of type TimeStamp


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 95bef33:
:clock10: Pending: waiting for processors (36 builds ahead of you)
:warning: Docs Build is busy, currently there are 36 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 95bef33:
:clock10: Full build: the average full build time is 7 min(s) 43 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 95bef33:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Improve Dapr side car failure logs in debug level

This issue is to track the improvement in Dapr side car logs in debug level.
In this mode, expectation is to add more granular details in the failure case in addition to the error code but only in the debug mode.

Closing this , using the #1720


Docplex is not compatible with Python 3.9

Information

Qiskit Optimization version: NA
Python version: Python 3.9.1
Operating system: macOS 11.2

What is the current behavior?
The latest Docplex 2.20.204 does not work with Python 3.9
Related to #24
Steps to reproduce the problem
from docplex.mp.model import Model
a = Model()
x = a.binary_var()
print(a.add_constraint(x <= 1))

Python 3.8
x1 <= 1

Python 3.9
Traceback (most recent call last):
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/environment.py", line 290, in get_cplex_module
    import cplex  #@UnresolvedImport
ModuleNotFoundError: No module named 'cplex'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/ima/tasks/1_2021/qiskit/aqua/tmp/c.py", line 2, in <module>
    a = Model()
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/model.py", line 417, in __init__
    self._environment = self._make_environment()
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/model.py", line 181, in _make_environment
    env = Environment.get_default_env()
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/environment.py", line 427, in get_default_env
    Environment._default_env = Environment.make_new_configured_env()
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/environment.py", line 422, in make_new_configured_env
    return Environment(start_auto_configure=True)
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/environment.py", line 84, in __init__
    self.auto_configure(logger=logger)
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/environment.py", line 220, in auto_configure
    self.check_cplex(logger=logger)
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/environment.py", line 328, in check_cplex
    cplex = self.get_cplex_module(logger=logger)
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/environment.py", line 314, in get_cplex_module
    cplex = load_cplex_from_cos_root(loc) if loc else None
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/environment.py", line 286, in load_cplex_from_cos_root
    return load_cplex(full_path, version=version)
  File "/Users/ima/envs/dev39/lib/python3.9/site-packages/docplex/mp/environment.py", line 261, in load_cplex
    raise FileNotFoundError("Could not load module from %s" % module_location)
FileNotFoundError: Could not load module from /Applications/CPLEX_Studio201/cplex/python/3.9/x86-64osx/cplex/__init__.py

What is the expected behavior?
Suggested solutions
We should configure setup.py and requirements.txt so that Python 3.9 does not install docplex.

I merge this issue with #24


Added total hours of sun and remaining sun hours

im not sure that localization is accurate (just put lines in the google translator).

Thanks for the PR! I can see that there's many code styles changes (adding semicolon, double quotes, etc.), would you mind change those back?

I'm thinking, that for localization, it's probably better to just add the words in English rather than translator. In my experience, the traslations are not the most accurate, so probably the best option is to leave that part for people that speak the language and don't mind to open a PR/issue with the translations


Fanoutsearch for .net core

Make LIKQ work for .net core, also a typo error for code generating.

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

For .new core version, both Trinity and LIKQ project should add Serializer.Linq as dependencies, as it is serializing, it will checking if any assembly is depend on it.
Also there still an issue that, on non-embed mode, the StorageTable of CloudStorage is empty, i have to call  Global.CloudStorage.Open(false) on client now.

close it as it is the .net core implement but not .net standard.

Created a new .net core project for Trinity.Core, so that reference of Serialize.Linq will not fallback to .net framework 4.6 version.

@yatli Thanks for you comment that remind me, now both projects are netstandard.

Looks good to me now, thanks @chaosddp!


GEODE-9456, GEODE-9452: Authentication Expiration

Created the AuthenticationExpirationException
Created the test SecurityManager to expire users at a definite time
add serialization conversion to support older clients
Create tests to verify behavior for current and older clients

I think we still need javadocs on the new org.apache.geode.security.AuthenticationExpiredException since it's user-visible. Then I can add my approval.

Oh, must have been distracted by adding the javadoc to the SecurityManager. Done now!

Changing the base branch to a feature branch


Adding atlas orm

Atlas.ORM : A data mapper implementation for your persistence model in PHP.

@VarCI-bot hope I explained it well :1st_place_medal:

Closing for later PR https://github.com/ziadoz/awesome-php/pull/881 is merged.


splitting init and tracking

Can you split initialization and update steps?

Well, they have been split (e.g. if frame == start_frame, the tracker is initialized, else update).

Does it mean anything outside frame!=start belongs init? Probably no. Which variables should be kept across updates and inits?
Is not it better to have a class with members and functions instead of spaghetti main?

I already converted it to the class, but having almost identical init and update stages seems weird. Subwindow extractor also crashes. I can share mine but it is not much different than you told, reference implementation would be cute.

I feel very sorry for the long main because of my poor coding ability. Welcome your advanced version sincerely~

I am not saying your ability is poor, I am saying main is poor.
What are you doing is appreciated and there is room for improvement. Do you want to improve?

It's my pleasure if you can share your version but If you ask it for money, i would say sorry.

Correcting: you are poor, sorry.


feat: add Space cover support

Summary
Closes: https://github.com/snapshot-labs/sx-ui/issues/580
Depends on: https://github.com/snapshot-labs/stamp/pull/56
Depends on: https://github.com/pulls
Test plan

Use local stamp from PR and change URL in Stamp.vue
Change goerli API URL to https://api.studio.thegraph.com/query/41343/sekhmet-sx-goerli/version/latest
Disable all other networks.
You can see cover photo there.
You can add cover photos to your own spaces.

Screenshots

Further comments
We might want to lift size limit on pineapple too, some images will fail to upload now.

Looks great! I tried to upload a cover but somehow it doesn't get saved on your subgraph. Also any idea why the tooltip moved from over to under on Space overview?

Actually my cover got indexed on the subgraph after few minutes.

I noticed something that can be improved, the message "Failed to upload image." and others notifications in general should appear over the modal and not under.


saving cubes with masked cell measures

Hey guys, how's it going? We're hitting an issue with saving cubes that have masked cell measures attached to the main variable, see https://github.com/ESMValGroup/ESMValCore/issues/1917 - I don't really understand the issue with this, so long as one supplies the saver with a valid fill_value (like we do, the infamous 1e20) - is this something that's a bit legacy or you're still afraid of it? Cheers muchly :beer:

@valeriupredoi this originally stemmed from a desire to align as-close-as-possible to CF. But from discussion today we can't see a practical reason why cell measures shouldn't be masked.
So we'll leave this open as a future development task. If this is a priority for you please let us know.

we can't see a practical reason why cell measures shouldn't be masked.

Oh dear, I think it was me said that.
Now I looked at the code, where this is specifically dis-allowed for cell-measures only
This test was added as a specific exclusion in this PR
-- though the code has since been refactored somewhat, so it's not that obvious.
The argument given is "We can't save masked points properly, as we don't maintain a fill_value"
I guess the danger envisaged is that we might have masked data with a specific fill-value set on the array, and we don't support any way of making that the fill-value actually recorded in the netcdf file, so it could generate missing-point collisions.
We can avoid it for cube data (only), by specifying fill_value(s) in the save operation.
Altogether It's a pretty theoretical idea of a potential problem : points masked in the array will be saved as default-missing-value, but there might also be unmasked points with that value :  Highly improbable, except for integer data.
And this code is supposed to detect and (at least) warn about that occurrence.
So, I really don't agree with this argument -- and especially as it should apply equally to aux-coords, and ancillary variables (added much more recently), and those we don't disallow.
So I think if all agree we could quickly make a change to just remove this check
Any contrary opinions ?  speak now !

cheers gents! Not a priority nor anything stringent for us, just thought I'd raise a flag or two - if you guys decide to keep it we'd be finding a workaround, and in fact, one of @bouweandela 's current PRs (in ESMValCore) is inadvertently fixing this :+1:

cheers gents! Not a priority nor anything stringent for us, just thought I'd raise a flag or two - if you guys decide to keep it we'd be finding a workaround, and in fact, one of @bouweandela 's current PRs (in ESMValCore) is inadvertently fixing this +1

Nevermind the second part of my statement - that was a bug, that is now fixed, so we're back at hitting this. It'd be cool if you guys removed the raise as @pp-mo mentions, or both the check and raise :grin:

hey guys - we're still hitting this with iris=3.4.1 - not sure why you folks closed the issue 😕

Hi @valeriupredoi, this feature was first available in v3.5.0rc0. See 897a7cbb9547d76aedb5b60deef7fbc8a61f9240:

ahaa cheers @trexfeathers - let me update then, thanks for the heads up, I jumped the (version) gun

with iris=3.5 all goes smoother than a Ferrari's paint work, many thanks @trexfeathers and the iris gang :beer:


Make exceptions for certain columns?

So I have a column that I don't want to appear in the exported Excel file (this column contains buttons/links that only work on the web) -- Maybe mark cells to exclude with a certain CSS class or something. Like
my buttons are here
or something of the sort.

I'll be adding filter functions for rows and columns. Not designed how it will work yet.

Implemented on #88
https://github.com/jmaister/excellentexport/tree/3.3.0


Memoize: Debugging stats investigations

Problem
The cache debugging stats provided via Jason's AppInfo file, show a couple of issues related to memoization:

Actual
Desired
Action

Some of the memoized functions lack identifiers
Each function should have id to be distinguishable
Add monitoringName property to inlined functions passed to memoize

Memoized functions that have 100% cache hit ratio have avgFnTime equal to zero
avgFnTime should be equal to at least first function call time
Register first function call and add it as starting point of avgFnTime

getEmojiUnicode cache retrieval tends to be slower than actual function call
Retrieval should be faster than the actual function call
Analyze memoization for getEmojiUnicode

getLocaleDigits seems to not be used at all
The function may be underused, possibly due to a specific flow Jason performed
Reason about adding memoize stats data to analytics in order to have more, varying test samples

getUnreadReportsForUnreadIndicator retrieval/execution times are way above expected (8ms/22ms)
Retrieval/execution should be faster
Investigate the function

freezeScreenWithLazyLoading creates many unnamed empty cache entries
Cache entries should be limited to one per function
Check how we can cache it effectively and add monitoringName

getItemHeightMemoized lacks id
It should be properly referenced
Add monitoringName

cacheRetrievalTime measures the time for the cache itself but not for the entire memoize overhead
We should also track entire memoize helper overhead
Add memoizeFnTime as a new mark or change cacheRetrievalTime to be measured from function start

After trying to follow Jason's flow and exploring different parts of the app, I've added a few adjustments:

getLocaleDigits caches properly but it was not used in Jason's flow
on my end, getEmojisUnicode cache retrieval was faster than the function. Let's keep it and wait until we have access to broader data

Update avgFnTime calculation
Memoized functions with a 100% cache hit ratio have avgFnTime equal to zero, while it should be equal to at least the first function call time.

After investigating, the first call is registered properly.
What is happening is that monitoring was enabled after initial function call was made, so the function call timing could not be registered and all subsequent calls were intercepted by the cache, so we ended up with zero value for fnTime.

Action points from the table status update:

✅ Add debug names
✅ avgFnTime (context)
✅ getEmojiUnicode (context)
✅ ❓ getLocaleDigits - Discuss adding memoize stats to the analytics
❓ getUnreadReportsForUnreadIndicator - Can we drop caching in the runtime?
✅ freezeScreenWithLazyLoading - It adds duplicates to cache but still memoizes component properly, so let's keep it
✅ getItemHeightMemoized
⌛ cacheRetrievalTime

Affirmative!