Pearsonr reports p-value of 0 with lists of length 2

My issue is about ...
pearsonr. In the language R, running cor.test on a list of 2 yields an error:
cor.test(c(0.5, 0.25), c(0.25, 0.4))
Error in cor.test.default(c(0.5, 0.25), c(0.25, 0.4)) :
not enough finite observations
But entering the same into pearsonr gives me an ultra-confident result (see below)
Reproducing code example:

from scipy.stats import pearsonr
pearsonr([0.5, 0.25], [0.25, 0.4])

(-1.0, 0.0)

Error message:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  ...

Scipy/Numpy/Python version information:

('1.2.2', '1.16.4', sys.version_info(major=2, minor=7, micro=16, releaselevel='final', serial=0))

But entering the same into pearsonr gives me an ultra-confident result (see below)

This was fixed in SciPy 1.3.  When the input has length 2, pearsonr will report a p-value of 1:
In [1]: import scipy

In [2]: scipy.__version__
Out[2]: '1.3.1'

In [3]: from scipy.stats import pearsonr

In [4]: pearsonr([0.5, 0.25], [0.25, 0.4])
Out[4]: (-1.0, 1.0)

The rationale is given in last paragraph of the "Notes" section of the docstring.

@guhanrv, I am closing the issue, but questions or further discussion are welcome.


Avoid sorting hits twice in search

Currently, we are doing the hits sorting twice.
Once when sorting partial_hits in merge_fruits step (inside, it's top_k_partial_hits is called)
let leaf_search_result = spawn_blocking(move || collector.merge_fruits(leaf_search_results))

Once when sorting hits after docs are returned
hits.sort_by(|hit1, hit2| {
        let value1 = if let Some(partial_hit) = &hit1.partial_hit {
            partial_hit.sorting_field_value
        } else {
            0
        };
        let value2 = if let Some(partial_hit) = &hit2.partial_hit {
            partial_hit.sorting_field_value
        } else {
            0
        };
        // Sort by descending order.
        value2.cmp(&value1)
    });

It is fine as it is...
The second sort usually contains 20 documents.
In most case it is not worth doing anything smart.

Yep, I agree with @fulmicoton  after analysis of this. We cannot get rid of the first sort.
And because we want to optimize fetch document by split, the result gets out of order.
We could merge using Heap but that is still sorting

Yes. So won't fix.


[proof of concept] Revamp filter

Replaced by https://github.com/ParkingReformNetwork/reform-map/pull/452


Redirect root to creator dashboard if it's enabled

We redirect to /feed currently, but in the event it's a creator we should default and redirect to /maintainer instead

Over time I agree, but since core focus is on creators & activation, I think until we see that they are following a lot of others (have content to discover in /feed) it only creates additional clicks for how they're using it today. As a creator, I would want that as my hub and goto feed vs. the other way around.

Hm. Changed my mind here.

No subscriptions, but creator -> Creator dashboard
Has subscriptions -> Feed.

I think Feed could then have additional entry points to the creator dashboard if they're a creator, i.e "Write" in context of the posts to goto writing a new post etc.


Where are the Swift version?

Did not found a reference to the Swift version - the title implies that there is one.

@noiis Unfortunately I have not got around to it yet. Sorry man!

@lukegeiger All right, thanks anyway!


struct.error: unpack requires a buffer of 33 bytes

Hey, it's me again, I encounter this error when trying to read some data from 2018-02-02 :
Downloading 2018-02-02 00:00:00
Parsing IEX_data/20180202_IEXTP1_TOPS1.6.pcap ...

Traceback (most recent call last):
  File "./iex_binary_download.py", line 92, in <module>
    main()
  File "./iex_binary_download.py", line 55, in main
    for message in iex_messages:
  File "/home/tux/market_data_download/env/lib/python3.7/site-packages/IEXTools/IEXparser.py", line 122, in __next__
    return self.get_next_message()
  File "/home/tux/market_data_download/env/lib/python3.7/site-packages/IEXTools/IEXparser.py", line 273, in get_next_message
    self.message_type, self.message_binary
  File "/home/tux/market_data_download/env/lib/python3.7/site-packages/IEXTools/messages.py", line 138, in decode_message
    decoded_msg = struct.unpack(fmt, binary_msg)
struct.error: unpack requires a buffer of 33 bytes

This problem is present with version 0.0.4 and 0.0.5.
Is the TOPS binaries corrupted ?

Very interesting, I downloaded the file, but I couldn't actually decompress it without errors, the decompressing function built into IEXTools returned: EOFError: Compressed file ended before the end-of-stream marker was reached
I also tried decompressing the file using 7-zip and received a similar error: Unexpected end of data.
I suspect the file may be corrupted / incomplete.

Is it possible that some message entries inside the binary are corrupted ?

So I found the problem by looking at the "problematic" message in the file and decoding it manually:
Problematic message

 1  2  3  4  5  6  7  8  9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40
**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|
01 00 03 80 01 00 00 00 00 00 9C 44 28 00 01 00 94 00 19 6D 00 00 00 00 8C AF 7C 02 00 00 00 00 38 E9 1C D3 AC 9F 0F 15
26 00 42 00 38 E9 1C D3 AC 9F 0F 15 41 44 54 20 20 20 20 20 80 0C 00 00 8A F6 01 00 00 00 00 00 1C E6 FA 24 00 00 00 00

TP Header:

 1  2  3  4  5  6  7  8  9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40
**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|
01 00 03 80 01 00 00 00 00 00 9C 44 28 00 01 00 94 00 19 6D 00 00 00 00 8C AF 7C 02 00 00 00 00 38 E9 1C D3 AC 9F 0F 15
v-|r-|-pro-|channel id-|--sess id--|s len|# msg|-----Stream offset-----|-----1st SeqNumber-----|-------Timestamp-------|

Number of messages is 1
The timestamp resolves to 2018-02-02 21:32:44.72225 UTC

Message:

 1  2  3  4  5  6  7  8  9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40
**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|**|
26 00 42 00 38 E9 1C D3 AC 9F 0F 15 41 44 54 20 20 20 20 20 80 0C 00 00 8A F6 01 00 00 00 00 00 1C E6 FA 24 00 00 00 00
-len-|ty|fl|-------Timestamp-------|---------Symbol--------|---Size----|---------Price---------|--------Trade ID-------|

Message length is 0x26 0x00: 38 bytes
Message type is 0x42: Trade Break Message
Sale condition flags is 0x00: ? I'm not too sure how to interpret a value of 0 here
Timestamp 0x38 0xe9 0x1c 0xd3 0xac 0x9f 0x0f 0x15: 2018-02-02 21:32:44.72225 UTC
Symbol 0x41 0x44 0x54 0x20 0x20 0x20 0x20 0x20: 'ADT     '
Size 0x80 0x0c 0x00 0x00: 3200
Price 0x8a 0xf6 0x01 0x00 0x00 0x00 0x00 0x00: $12.8650
Trade ID 0x1c 0xe6 0xfa 0x24 0x00 0x00 0x00 0x00: 620422684

In my messages format in messages.py for the trade break message type I have omitted the size parameter of the message.
https://github.com/vfrazao-ns1/IEXTools/blob/6058305e589b4611265c332c9fe58747643a2902/IEXTools/messages.py#L95
(the decode format should actually be "fmt": "<1sq8siqq")
I'm surprised this didn't come up during testing, but it appears that the trade break message is rather rare. I'll create a quick PR for this and upload a new release to PyPI.

Ok cool, thanks a lot !

Issue resolved by latest PR: https://github.com/vfrazao-ns1/IEXTools/pull/8


segmentation fault in test run after building without bundled proj sources

I encountered an issue for the case when the pyproj module is build without the bundled proj sources. In this case the test run: python -c 'import pyproj;pyproj.test()'
ends with an ugly segmentation fault.
After some digging around I found that this is caused by the fact that my system (Fedora 23) has proj version 4.9.1 which does not have the inverse function for the hammer projection available. The same happens on Fedora rawhide(24) which has proj version 4.9.2
The segmentation fault happens at the call:
projlonlatout = pj_inv(projxyin,self.projpj)
on line 204 of the file _proj.pyx
The root cause is the fact that the inverse function is called through a pointer in the self.projpj object, and the cython code does not check if this function pointer is defined or not (actually I don't even know if this is possible inside cython, maybe a little additional c-code is needed to do this).
Anyway, it would be nice if some safety check coul;d be added to pyproj to let it safely run for older external proj versions. Of course the problem will go away when proj 4.9.3 is released, as mentioned in issue #19 but I have no idea when that will happen.

I have disabled the inverse hammer projection test in the rpm build for fedora, and after this all remaining tests run fine for rawhide (upcoming fedora 24). This one will have proj version 4.9.2
For fedora 23, which has proj version 4.9.1, there are 2 failing tests related to the hammer projection (both for python2 and python3). See the details in this log file:
https://kojipkgs.fedoraproject.org//work/tasks/1374/12551374/build.log
I guess this also is related to the missing inverse hammer in these proj versions.
The output for the failing tests is:
Trying:
p = Proj(proj='hammer') # hammer proj and inverse
Expecting nothing

File "/builddir/build/BUILDROOT/pyproj-<IP_ADDRESS>-1.fc23.x86_64/usr/lib64/python3.4/site-packages/pyproj/init.py", line 324, in pyproj.Proj.new
Failed example:
p = Proj(proj='hammer') # hammer proj and inverse
Exception raised:
Traceback (most recent call last):
File "/usr/lib64/python3.4/doctest.py", line 1318, in __run
compileflags, 1), test.globs)
File "<doctest pyproj.Proj.new[20]>", line 1, in
p = Proj(proj='hammer') # hammer proj and inverse
File "/builddir/build/BUILDROOT/pyproj-<IP_ADDRESS>-1.fc23.x86_64/usr/lib64/python3.4/site-packages/pyproj/init.py", line 355, in new
return _proj.Proj.new(self, projstring)
File "_proj.pyx", line 84, in _proj.Proj.cinit (_proj.c:1156)
raise RuntimeError(pj_strerrno(err))
RuntimeError: b'major axis or radius = 0 or not given'
Trying:
x,y = p(-30,40)
Expecting nothing
ok
Trying:
'x=%12.3f y=%12.3f' % (x,y)
Expecting:
'x=-2711575.083 y= 4395506.619'

File "/builddir/build/BUILDROOT/pyproj-<IP_ADDRESS>-1.fc23.x86_64/usr/lib64/python3.4/site-packages/pyproj/init.py", line 326, in pyproj.Proj.new
Failed example:
'x=%12.3f y=%12.3f' % (x,y)
Expected:
'x=-2711575.083 y= 4395506.619'
Got:
'x=16013975.139 y=19338537.497'

One option would be to move these inverse hammer tests to unittest and skip it the inverse hammer tests when pyproj.test(skip_inv_hammer=true).
That would allow package maintainers to be able to ensure the rest of the functionality stays in the package.  It would retain the useful unittest.

Yes, that would be a nice option (if the plan is to move all tests to the unittests anyway).
A more general concern I think is that it seems possible to have undefined inverse projections in proj. I have no detailed knowledge of the proj  source code, but if more projections exist without their inverse, then this may trigger the same problem. First step would be to add unittests for all projections, and their inverse I think.

It appears 21 projections currently in PROJ.4 have no inverse function.
I've used unittests to test that bug fixes remain fixed.  Unittest does have the ability to skip test if certain conditions aren't right.

Yes, and I see devel version of proj has added a check for NULL pointer of P->inv and P->fwd in the source files pj_inv.c and pj_fwd.c. These checks are present in the bundled c-code but not in proj releases 4.9.1 and 4.9.2. This will cause segmentation faults if the pyproj module is build without bundles sources with these proj versions. I just tested, and for example the 'airy' projection also shows this segmentation fault.

PR #44 was merged, which fixed this issue. The repository no longer has this issue. Closing.


The line after an empty comment is parsed as a comment.

#
echo hello

aah yeah, i've seen this one a bunch of times :+1:

sounds like a bug. thanks.


[Plugin] OpenApi Generator as templater for scaffolder plugin

Summary
For API-First approach, OpenApi generator is a convenient way to bootstrap a project with actual rest endpoints and data model included in code generation in a dynamic way.
The feature can be enhancement to scaffolder-backend plugin or a separate plugin to provide openapi-generator as templater.
https://github.com/OpenAPITools/openapi-generator
https://www.npmjs.com/package/@openapitools/openapi-generator-cli
Context
Most of the templates are static in nature with include HelloWorld kind of examples, which often serve purpose of reference implementation and need to be removed after initial code generation.
OpenApi Generator takes api definition as input and code generation can be made more dynamic and useful for actual usecase and implementation standards.

So with the new version of templates, you are free to make your own actions that run as part of the scaffolding. Sounds to me like it might be interesting to make an action that can run the openapi generator. You can either add that as a custom action of your own, or contribute it to open source. I suppose that the user will want to select from a number of different API definitions then? How do you envision that the user flow would look, should the user supply a URL to an API definition in their version control system when filling out the template form?

Here is an example cli usage
openapi-generator-cli generate -g ruby -i https://raw.githubusercontent.com/OpenAPITools/openapi-generator/master/modules/openapi-generator/src/test/resources/3_0/petstore.yaml -o /var/tmp/ruby-client

Yes, all these options can be supplied by filling out template form.
Indeed, actions are now more appropriate to address such needs.


EXRUN insufficient when variants used

example port split into subpackages:

complete
primary
dev
tools
man

If there are multiple variants (standard and dos) and EXRUN[tools]= example:primary:standard, then example:tools:dos will try to pull in example:primary:standard instead of example:primary:dos subpackage.
ravenadm has to figure this out and link to the proper one.

Release 2.07 is out with this fix.


Password parameter names not aligned for Backup-DbaDbCertificate and Restore-DbaDbCertificate

Backup-DbaDbCertificate has
[-EncryptionPassword] <SecureString>]
[-DecryptionPassword <SecureString>]
whereas Restore-DbaDbCertificate has
[-EncryptionPassword] <SecureString>]
[-SecurePassword] <SecureString>]
In looking at the parameter definitions, it seems they are both used for the same purpose.

-DecryptionPassword
A string value that specifies the system path to decrypt the private key.

-SecurePassword
Secure string used to decrypt the private key.

The parameters in the command are correct and match up, this is simply the documentation does not...at least the way I find the commands right now.

So this one is a bit off in parameters but also the documentation for each does not match up. Restore command has the SecurePassword parameter but the documentation does not present that parameter and shows it as DecryptionPassword. It does have an alias for the later though so technically they do match up, just not directly.


Enum

Do you have enums?
I think it would be useful.

What do enums look like in JS? I’ve tried things like
export const modes = {
  ON: Symbol('ON'),
  OFF: Symbol('OFF'),
};

– but it never seemed particularly easy to use.

I didn't have that in mind when I talked about enum.
Here's the use case:
functionName(param1: String, param2 = 'type1' | 'type2' | 'type3')

This function only accepts theses strings for the second parameter.
Another example would be a function that returns only 0, 1 and -1.

If I correctly understand what @ericelliott said in https://github.com/ericelliott/rtype/issues/26#issuecomment-158695876, then what you propose is already valid syntax.

Then we need an example of that in the union section.

How about something like this?:
You can use unions and the built-in support for literal types to create enum types, meaning that the value must match one of the literal values in the union:
functionName(param1: 'value 1' | 'value 2' | 'value 3') => Any

Feel free to PR.

@ericelliott #34


feat: Replace flask with quart

Closes #42
Summary of Changes

replaces flask with quart
rewrite some tests
added test against crashing
solution should be more scalable as it replaces wsgi with asgi

:tada: This PR is included in version 0.5.0 :tada:
The release is available on:

v0.5.0
GitHub release

Your semantic-release bot :package::rocket:


How can I make the ViewFinderView invalidate faster

I need a moving line in the rectangle :
I override the onDraw() method of ViewFinderView.
But it's not fast enough to be smooth. And I failed to find out how to change the frequency of previewSized() method of "CameraPreview.StateListener".

Are you drawing something based on results from the barcode scanner, or based on something else (e.g. a timer)?
There is currently a 80ms delay between draws for the laser animation. The barcode result points will typically be less frequent than that.
If you're doing significant customizations to the ViewfinderView, it may be the easiest to just copy the entire class. You'll then also need your own verssion of DecoratedBarcodeView, but that is essentially just a FrameLayout that overlays the ViewfinderView on top of the BarcodeView.


[2.5.9] Deprecate helmVersionCommits from v3.Catalog

Backport of:
https://github.com/rancher/rancher/pull/33155
Relevant issue(s):
#32938
#33156
https://github.com/rancher/rancher/issues/33256
This was a clean cherry-pick of the commit on release/v2.5 (2.5.10).

Is this for master or 2.5.9?

Hang on, this is a mess. Opening a new PR @StrongMonkey

Closed in favor of: https://github.com/rancher/rancher/pull/33261


fix: one annotated argument should be a payload if the annotation is not @Payload

Signed-off-by: jpiraguha<EMAIL_ADDRESS>Hey, @joshlong here is small fix
one annotated argument should be a payload if the annotation is not @Payload

@jiraguha Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@jiraguha Thank you for signing the Contributor License Agreement!

Hi - this is awesome! thank you for this.


Update tools-deps to <IP_ADDRESS>9

Another upstream release

Official images PR: https://github.com/docker-library/official-images/pull/8481


Enable missing lints in examples and integration tests

The Rust attributes we use to enable the pedantic/unused_crate_dependency lints must be added to every main.rs and lib.rs in order that they be enabled for the binary/library. (At least until Cargo/Clippy offers a way to specify crate/repo wide lint options.)
These annotations are currently missing from the examples under examples/.
In addition, it seems that since integration tests result in a separate binary, we must add them there too. Doing this would have caught the fact that one of the integration tests returned a Result but yet only ever returned Ok (due to unwrapping all errors):
https://github.com/Malax/libcnb.rs/pull/277#discussion_r796818127

Ugh, I really want Cargo workspace-wide Clippy settings - this is so annoying TBH. :(


Add "authors" array property to publications.

Please add am "authors" array property to publications.
Any properly formatted publication citation should include the author name(s).  I guess this property was left out of this schema because it is assumed that the author is the person whose resume it is on, but often publications can have multiple co-authors, and it should be possible to list them here.

That's a good point. Probably fell through. Thanks for suggestion. Happy to get a PR that makes this possible :)

Thinking further on this. A few questions and would love your insights.
Should we consider books a publication?
Array for authors and a role field similar to projects?
This would lead to something like this:
{
  "$schema": "http://json-schema.org/draft-07/schema#",
  "additionalProperties": false,
  "properties": {
    "publications": {
      "description": "list of publications",
      "items": {
        "additionalProperties": false,
        "properties": {
          "name": {
            "description": "name of the publication",
            "type": "string"
          },
          "publisher": {
            "description": "name of the publisher",
            "type": "string"
          },
          "publicationDate": {
            "description": "date of publication",
            "format": "date",
            "type": "string"
          },
          "roles": {
            "description": "specify your role on this publication",
            "items": {
              "type": "string"
            },
            "type": "array"
          },
          "co-authors": {
            "description": "specify co-authors",
            "items": {
              "type": "string"
            },
            "type": "array"
          },
          "resources": {
            "description": "multiple resources with label",
            "items": {
              "additionalProperties": false,
              "properties": {
                "label": {
                  "description": "label of the resource",
                  "type": "string"
                },
                "url": {
                  "description": "url of the resource",
                  "format": "uri",
                  "type": "string"
                }
              },
              "required": [
                "url",
                "label"
              ],
              "type": "object"
            },
            "type": "array"
          },
          "summary": {
            "description": "short summary of the publication",
            "type": "string"
          },
          "url": {
            "description": "url of the publication",
            "format": "uri",
            "type": "string"
          }
        },
        "required": [
          "name",
          "publisher"
        ],
        "type": "object"
      },
      "type": "array"
    }
  },
  "required": [
    "publications"
  ],
  "title": "Resumic resume Schema",
  "type": "object"
}

With data such as:
{
  "publications": [
    {
      "name": "Learning Resumic",
      "publisher": "Open Source Publisher",
      "publicationDate": "1337-01-01",
      "resources": [
        {
          "label": "link",
          "url": "https://example.com"
        },
        {
          "label": "reviews",
          "url": "https://example.com"
        }
      ],
      "summary": "Awesome book about Resumic",
      "url": "https://example.com/learning-resumic",
      "roles": [
        "reviewer"
      ],
      "co-authors": [
        "Co-author #1",
        "Co-author #2"
      ]
    }
  ]
}

So by default the resume owner is assumed to have authored given publication, but he can add an array of co-authors and additionally add his given role, if he is not the main author.
This is similar to how projects are done.
Happy for other suggestions or feedback on that approach @jaufgang

Legal also has this notion of coAuthor, but also defines the main Author. Will revise and add the Author as well.

One way would be to push towards coRoles, but usually the important things are your own role and the coAuthors not the other editors etc. So probably should be ok with just coAuthor.

Hi! Researcher here :)

Should we consider books a publication?

Yes!
What I suggest for publications is to go as follows:
{
   "releaseDate":"string",
   "publisher":"string",
   "title":"string",
   "type":"string (or a enumeration)",
   "authors":[
      "string"
   ],
   "link"?:"string",
   "doi"?:"string"
}

If you need feedback on this, do not hesitate!


White border when Terminal is dark while Windows is in Light theme

Environment
Windows build number: Microsoft Windows [Version 10.0.19582.1001]
Windows Terminal version (if applicable): 0.10.761.0

Steps to reproduce

Set Windows theme to Light.
In profiles.json, set "requestedTheme": "dark" and "showTabsInTitlebar": true
Focus on the window.

Expected behavior
No white borders, regardless of Windows theme color. (Like as VSCode)
Actual behavior

Though https://github.com/microsoft/terminal/pull/4817 fixes the white border issue for Dark mode in Windows, the border still presents with the combination of dark theme for Terminal settings and light theme for Windows.

So, we're doing our level best with the system API here. I'm going to keep this one around tracking our switch to the final API when it's out.

This has not been fixed. There should be no white border regardless the theme as mentioned in Expected behavior in this issue


add types to export definitions

Exports type declarations for compatibility with typescript module resolution strategies node16/nodenext and bundler.
Resolves the problem:
Could not find a declaration file for module '@syncedstore/core'. <EMAIL_ADDRESS>implicitly has an 'any' type.
  There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The '@syncedstore/core' library may need to update its package.json or typings.ts(7016)

Should resolve #107 , as the recomended TypeScript (5.x) compiler options for bun.sh appears to be bundler.

Thanks!


Rename topology label to habitat-topology

This:
TopologyLabel = "topology"
Should look like:
HabitatTopologyLabel = "habitat-topology"
and the usage updated in all places throughout the codebase. This keeps the naming of the labels consistent and creates a convention to prefix "habitat-" to internal labels used by the operator.

Now I think we should namespace it in a bit different way, though. Something like habitat.sh/habitat-operator/topology, or habitat-operator.habitat.sh/topology, just like k8s is doing with its statefulset.kubernetes.io/pod-name label.

operator.habitat.sh/topology sounds good.


Users/adityashah/self hosted enoent

Task name: AzureTestPlanV0
Description: Migrate spawn sync executuion
Documentation changes required: (Y/N) N
Added unit tests: (Y/N) N
Attached related issue: (Y/N) N
Checklist:

[ ] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

nit - Please address the nits, the title, description etc needs update

nit - Please address the nits, the title, description etc needs update

Updated


build: bump minimum required CMake version

Update the minimum CMake version to 3.19.  Swift requires 3.19.x for the build, so update the minimum version to match.  This allows us to drop some of the workarounds that we had for older CMake releases.

@swift-ci please test

@swift-ci please smoke test


Print evaluation results to STDOUT rather than to the logger

This makes it easier to save results through IO redirection.

LGTM!


Display week numbers

Got a feature request to be able display week numbers in calendar.

We really would appreciate to have it similar like this.
Thanks

@dmtrKovalenko I'm happy to see you changed your mind 👍 (#521), I was proposing this API in https://github.com/mui-org/material-ui-pickers/issues/1293#issuecomment-538374383:
  /**
   * If `true`, a column is added to show the week numbers of the year.
   *
   * default false
   */
  showWeekNumbers: PropTypes.bool,

Duplicate of #521


crontab location

Hi i am not able to locate the crontab.txt file, can you please share the full path
below is my  docker-compose.yml
volumes:
If you want to perform customizations to the source and have access to it, then uncomment this line - This includes modules

./data:/www/html

Or, if you just want to use Stock Freescout and hold onto persistent files like cache and session use this, one or the other.
#- ./data:/data
If you want to just keep the original source and add additional modules uncomment this line
#- ./modules:/www/html/Modules

./logs/:/www/logs
environment:

root@test:# ls
data db dbbackup docker-compose.yml logs
root@test:#

ps i also tried
root@test:/data# find . -name "crontab.txt*"
if someone can help me would be great
Thank you, I have feeling its something simple, but i am just doing it wrong
Thank you

You can see what's in the crontab by entering crontab -l.   .. Alsonlook in /assets/crown/

root@test:~/data# crontab -l
no crontab for root
did you mean crontab -e ( that is for the host and not the docker container)
I am not able to find the cron, did you mean data/resources/assets - because that is wrong
root@test:~/data/resources/assets# ls
js  sass
root@test:~/data/resources/assets#

Not sure. I write the crontab to /assets/cron.

is the /assets the root folder because according to docker compose,yml

./data:/www/html - it is this one, I love your docker file, just have this one last issue

Thank you

Please dont map anything in /assets other than /assets/custom or /assets/custom-scripts. You may break the crontab that way.

root@test:~# pwd
/root
root@test:~# find . -name "cron*"
./data/vendor/mtdowling/cron-expression
root@test:~# find . -name "assets*"
./data/resources/assets
./data/vendor/rachidlaasri/laravel-installer/src/assets
Do you happen to have the dockercompose.yml file
which has the exposed volume for crontab edit

i am fairly new to docker, i am sure you can tell

I do not expose the volume for Crontab - it is hidden by design.

I understand but how will i access, the crontab of the volume share is hidden or not setup
Please do help

I have no idea what you are trying to do to begin to help?

I want to access the crontab of the docker version ?

https://github.com/tiredofit/docker-alpine#scheduling-options


Provide conversion utility that could create a SIMPLE_CSV format file from the more complete DWCA format download

Assuming that a SIMPLE_CSV format download is a subset of the DWCA download - as per the GBIF download FAQ entry:

CSV: Tab delimited CSV. Only contains the data after GBIF interpretation. No multimedia included. More information about CSV
Darwin Core Archive: The Darwin Core Archive (DwC-A) contains both the original data as publisher provided it and the GBIF interpretation. Links (but not files) to multimedia included. More information about DwC-A

... then it would be good to have a utility that could create a SIMPLE_CSV format file from the larger DWCA.
Rationale: - a user develops a script that needs only minimal data and therefore is designed to operate on SIMPLE_CSV format input. Another user of the script has a pre-existing DWCA format download and wants to use this as input to the script (without having to create another download) - so they need a way to slim down the DWCA to the SIMPLE_CSV set of fields.
Is there a GBIF metadata service that returns the fieldnames used in each of these download formats which could help? If so, pygbif could provide access to this and a column rename mapping (if required).

I believe such thing is outside of the scope of this package. Also there is a Python package to manipulate a DWC Archive python-dwca-reader.
These package are developed to work with the GBIF DWCA downloads.

I believe such thing is outside of the scope of this package. Also there is a Python package to manipulate a DWC Archive python-dwca-reader. These package are developed to work with the GBIF DWCA downloads.

The default download format for occurrences appears to be SIMPLE_CSV:
https://github.com/gbif/pygbif/blob/9590fcf54cc600c00f5e61b5ce60522ff62000c3/pygbif/occurrences/download.py#L69

Indeed, the current default parameter is SIMPLE_CSV, and the supported formats are listed here.
A major challenge when working with a DWCA is that this format is not always consistent. Many times, there isn't a common way to map extra tables to one table. For instance, in the case of a multimedia table, you can expect the id field to be present in a multimedia.txt file. However, it could have a one-to-many relation with an occurrence. In such cases, what would be a common mapping strategy from the multimedia table to a plain table?
I believe that the SIMPLE_CSV format is just focused on share occurrence data without additional information.

My request is about occurrences only and relates only to DWCA downloads originating from GBIF (pygbif is about facilitating access to the GBIF API from Python).
As GBIF is able to represent occurrence data in both DWCA and SIMPLE_CSV formats I'd like to be able to convert from DWCA to SIMPLE_CSV.
Out of interest, are you speaking for GBIF @jmbarrios ?

Out of interest, are you speaking for GBIF @jmbarrios ?

No, I'm not. Also I am not associated with GBIF.

Hi Nicky,
There's an experimental (not necessarily stable, not documented) API for the columns returned in GBIF downloads:

https://api.gbif.org/v1/occurrence/download/describe/simpleCsv
https://api.gbif.org/v1/occurrence/download/describe/dwca

The SIMPLE_CSV format should be a Simple Darwin Core-compatible file, see §6.1 where these files can be shared without a meta.xml description.  It's also a subset of the occurrence file in a DWCA format download, and the column names should be identical — if a CSV reader is referencing columns by name it should work fine with either file.
@CecSve is maintaining pygbif, but is on leave until the end of January.

Thanks @MattBlissett - if / when this becomes stable it might be good to consider making it available from the pygbif library.
I did find two fields in SIMPLE_CSV that are not in DWCA: publishingOrgKey and verbatimScientificNameAuthorship

My mistake, verbatimScientificNameAuthorship should be scientificNameAuthorship from verbatim.txt. publishingOrgKey would need to be retrieved using the API.


Wayland support?

I was looking at whether I could get vscode running on wayland without xwayland (x11 support). It seems this library is required by vscode and uses X11 on linux. Could the linux code be ported to use something like xkbcommon (I'm not sure if this is the right replacement)? How hard do you think it'd be?

@alexdima any updates on this? Wayland is the de facto standard on modern Linux distributions and this would finally allow VSCode to use Wayland natively.

Is this related to #17?

Oh yeah, I meant #21


[JSInterp, YouTube] Support YT player b22ef6e7 etc

Boilerplate: own/yt-dlp code, fix/improvement
Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] Searched the bugtracker for similar pull requests
[x] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense, except for code from yt-dlp for which either this or the below have already been asserted
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
This PR supports YT player b22ef6e7 (probably just in time for it to be withdrawn).
At the same time, the opportunity was taken to improve the alignment of the code and tests with yt-dlp.
Fixes #32842.
Supersedes, closes #32843.
Also included is a previously intended update to PR #32830.
Thx @Grub4k, @bashonly, @seproDev, @pukkandan and possibly others for yt-dlp code included here.

:+1: works for me, thanks!


[clamav] update to 0.105.0

Library name: clamav
New version number: 0.105.0
Other information that may be useful (release notes, etc...)
https://www.clamav.net/downloads

when will merge to master

@SpenserCai Since the port clamav requires rust, and considering that more and more ports are starting to require rust, we are now adding rust to the script to get it.(Maybe a little longer, sorry) :)


Stickers with multiline text

Some stickers have multiple line text:

CLG
Copenhagen wolves
Dreameaters
teses
rez
t0rick

One solution can be array for matching text.

Killjoy


Chat - Attachment modal and delete workspace modal open with delay

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.73-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail<EMAIL_ADDRESS>Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to chat
Upload an attachment
Click on the attachment
Note that the attachment modal opens with delay
Go to Account settings > Workspaces
Click on 3-dot menu next to workspace
Click Delete workspace
Note that the delete modal opens with delay

Expected Result:
Attachment modal and delete workspace modal will not open with delay
Actual Result:
Attachment modal and delete workspace modal open with delay
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/3f6cf7ce-5c9a-41df-b85d-9b9a92eb852a
View all open jobs on GitHub

Fixed in staging
https://github.com/user-attachments/assets/915bf2bd-c64f-4f9e-bd52-91e7242f483e


Add an option to add ISO WW to left of calendar view

Please select if your request is either something new or an enhancement

[x] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace - VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup - Dendron's Lookup Command
[x] Views - Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema - Dendron Schemas
[ ] Pod - Data import from / export to Dendron
[ ] Publish - External Site Publish
[ ] Markdown - Markdown features, such as syntax support and features

Is your feature request related to a problem? Please describe
My work uses work week a lot, but tracking it is a little difficult.
Describe the solution you'd like
Add an option to display ISO WW to the left of the week in the calendar. Bonus points for clicking on it creating journal.Wnn.m
Describe alternatives you've considered
None
Additional context
Some like this:

Here is how outlook displays it:

Thanks @jeep for the enhancement request. I really like the idea.


Close non-containing overlays when a new overlay is opened

Closes https://github.com/adobe/react-spectrum/issues/1155
Also makes use of shouldCloseOnInteractOutside which reduces the number of times onOpenChange gets called, see https://github.com/adobe/react-spectrum/pull/1510/files#diff-5861ee186d3230691b6ff7f9ee6864ee290647d89e983003e1edfff6fcd6aa5eL225
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue.
[x] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

Added new stories in storybook with two menu/picker/etc in them, check those
🧢 Your Project:

replaced by new behaviors in https://github.com/adobe/react-spectrum/pull/1970


[Bitstamp] Order Type is wrong when there is a very small transaction fill.

This case is triggered by bitstamp rounding issue. If they have very small BTC partial fills, the USD settlement amount returned by the json object is rounded off to ZERO. Example below shows very small BTC fill and the settlement amount become 0.00 instead of 0.0000172472 (btc_usd*btc).
{"usd": "0.00", "btc_usd": 1724.72000000, "order_id": 5863279, "datetime": "2017-05-15 09:43:04", "fee": "0.01", "btc": "-0.00000001", "type": "2", "id": 14830786, "eur": 0.0}
thus, the adaptTradeHistory() checking below will result to BID instead of ASK.
OrderType orderType = t.getCounterAmount().doubleValue() > 0.0 ? OrderType.ASK : OrderType.BID;

I'm on vacation, will reply when i get back...
Sent from my iPhone

On 9. Jun 2017, at 11:46, kisapmata<EMAIL_ADDRESS>wrote:
This case is triggered by bitstamp rounding issue. If they have very small BTC partial fills, the USD settlement amount returned by the json object is rounded off to ZERO. Example below shows very small BTC fill and the settlement amount become 0.00 instead of 0.0000172472 (btc_usd*btc).
{"usd": "0.00", "btc_usd": 1724.72000000, "order_id": 5863279, "datetime": "2017-05-15 09:43:04", "fee": "0.01", "btc": "-0.00000001", "type": "2", "id": 14830786, "eur": 0.0}
thus, the adaptTradeHistory() checking below will result to BID instead of ASK.
OrderType orderType = t.getCounterAmount().doubleValue() > 0.0 ? OrderType.ASK : OrderType.BID;
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Printing from decision tree project template

Hi all (particularly @FayCross)
I was helping someone troubleshoot printing from her decision tree project which by default only prints one page even though there should be many in this particular project.
I can see that \modules\decision\parent_templates\decision\common\css\styles.css has the following:
html, body { 	height:		100%; 	overflow-x:	hidden; }
and that hidden value seems to be what is restricting the printing to a single page.
If I add an optional styles property to the LO and then add the following print css:
@media print {       html, body {     height: 100%;     overflow-x: visible; } }
Then when printing the multiple pages are printed.
So ok to add this to styles.css or is there another solution or any reason not to add this?
Cheers
Ron

It's so long ago that I've looked at this template that I really can't remember why the overflow-x was needed there. I'll change it so that when you click print it temporarily removes that bit of css and then adds it again when after printing

Great thanks @FayCross


Why do we use Incrementalnput in srml_support for storage

get function in srml_support is implemented as such:
https://github.com/paritytech/substrate/blob/a534483d546ac2554160e095772fc6fb3b47da87/srml/support/src/storage/unhashed/mod.rs#L24-L33
it make use of incremental input which is implemented there:
https://github.com/paritytech/substrate/blob/a534483d546ac2554160e095772fc6fb3b47da87/srml/support/src/storage/mod.rs#L30-L42
When implementing Decoding we usually make a lot of call to read, mostly with few bytes, I can't find the reason for those reads to be split, I think we should just do a read_storage and and decode from &[].
Do I miss something ? if no then this can be done for child::get as well.

I once stumbled over this as well and after some thinking found a reason for it, but I don't remember it anymore xD
In general this looks like some try to optimize the storage access, but you are right and that this probably generates to much read accesses. I think you can remove it.

actually implementing it the correct make the CI failing:
apply this diff on master for instance:
diff --git a/srml/support/src/storage/unhashed/mod.rs b/srml/support/src/storage/unhashed/mod.rs
index 77e6c2b37..573cf33ce 100644
--- a/srml/support/src/storage/unhashed/mod.rs
+++ b/srml/support/src/storage/unhashed/mod.rs
@@ -23,12 +23,12 @@ pub mod generator;

 /// Return the value of the item in storage under `key`, or `None` if there is no explicit entry.
 pub fn get<T: Decode + Sized>(key: &[u8]) -> Option<T> {
-       runtime_io::read_storage(key, &mut [0; 0][..], 0).map(|_| {
-               let mut input = IncrementalInput {
-                       key,
-                       pos: 0,
-               };
-               Decode::decode(&mut input).expect("storage is not null, therefore must be a valid type")
+       runtime_io::read_storage(key, &mut [0; 0][..], 0).map(|len| {
+               // TODO TODO: the +1024 works but this a bug no?
+               // let mut buffer = sr_std::vec![0; len+1024];
+               let mut buffer = sr_std::vec![0; len];
+               runtime_io::read_storage(key, &mut buffer, 0).expect("storage is not null");
+               Decode::decode(&mut &buffer[..]).expect("storage is not null, therefore must be a valid type")
        })
 }

it is on the key :extrinsic_index


RPMB secure storage

Enable eMMC RPMB support and OPTee driver in u-boot so that secure storage could be usable in u-boot.
Also add the RPMB key paring support in u-boot.

Change in V2:

Fix the symbolic links in secure-boot-otp-provisioning
Make PRODUCT_GENERATION global to reuse it in OPTee for warning PG1 devices on RPMB setup.
Integrate edk2 to isar-cip-core.
Integrate edk2 StandaloneMM to OPTee.
Enable StandaloneMM managed UEFI variable operation in u-boot.
Make optee-rpmb command default to u-boot config, since optee build is controlled, expose this command would not bring risks.
Rewords u-boot README regarding RPMB key provisioning.

Let me finish the isar-cip-core review and integration of Sven's patches. We don't want to rebase again here (or worse) but rather consume an upstream-accepted revision.

Let me finish the isar-cip-core review and integration of Sven's patches. We don't want to rebase again here (or worse) but rather consume an upstream-accepted revision.

Fine, schedule-wise I will continue work on this branch for the further topics, such as Key enrolling. To me, the EDK2 part of Sven's patch seems acceptable except the git clone takes too long and occupies too much space. So once there is a final state of the upstream patch I will update here accordingly, since that part takes only one commit in my patch set, it should not produce too many troubles by then.

Change in V3:

Disable the REE FS for secure storage in optee, RPMB based secure storage is mandatory.
Add optee-client package, preparing for UEFI key enrolling under linux user-land.
Rename download tarball name for optee-os for avoiding name collision.

UEFI key enrolling is on the way.

This PR is partially obsoleted by #426, the rest will be obsoleted by upcoming disk encryption integration.


Docker Daemon Shutdown Issue

Description
Docker Daemon failed to restart
Logs
CloudWatchLog Groups/aws/elasticbeanstalk/freeswitch-production/var/log/ecs/ecs-agent.logi-0bb7863548a55efa5
2017-11-27T22:55:30Z [WARN] Error with docker; stopping container freeswitch(dwilkie/freeswitch-rayo:somleng.ews1294.info) (STOPPED->STOPPED) for task awseb-freeswitch-production-gejeu9jqvp:200 arn:aws:ecs:ap-southeast-1:314673618878:task/e8e97f26-d819-4322-9ee8-305ae384dcc6, Status: (STOPPED->STOPPED) Containers: [freeswitch (STOPPED->STOPPED),]: API error (500): devmapper: Thin Pool has 0 free data blocks which is less than minimum required 2428 free data blocks. Create more free space in thin pool or use dm.min_free_space option to change behavior

http://docs.aws.amazon.com/AmazonECS/latest/developerguide/CannotCreateContainerError.html
https://stackoverflow.com/questions/26753087/docker-how-to-analyze-a-containers-disk-usage
Look in container
$ docker exec -it <container-id> /bin/bash

From inside docker container:
$ find / -xdev -type f -size +10M

Inspect disk usage of container:
$ docker system df -v

Todo

Have alarm trigger autoscaling action
Solve underlying problem. FreeSWITCH logs should be rotated

Done

Create Alarm.

Alarm triggers when Memory utilization of docker daemon <= 0 for one period

https://www.pivotaltracker.com/story/show/154896504


fix typo in migrating nb

In the migrating notebook, the instruction had a typo, nbdev_migrate_directives was changed to nbdev_migrate

I am using 2.0.7 from the fastai conda channel, and the command is still nbdev_migrate_directives.  Perhaps consider mentioning both commands in the documentation?


Exclude LongMethod in tests by default.

I don't feel like this rule should be on for tests. Often a longer more descriptive test is better than a short comprehensive one that's not understandable.

What for do you need more than tweenty lines of code in your tests? oO

When setting up a few things for a particular test. I wouldn't say it's that uncommon. If we wouldn't use the file system for testing, we'd execute it almost every single time.

I also don't quite like this setting. I can live with a slightly higher threshold for the test package, but I wouldn't want to disable this check for test methods completely.

Makes sense. So should we close this then?

I would say so.


Make report tasks cache relocatable

Thanks for making the task cacheable as part of 0.4.4. Now I'm seeing that inputs are using absolute path, so I'm getting cache misses when running from different directories (see sshot below).
It can easily be fixed by specifying PathSensitive(PathSensitivity.RELATIVE) on relevant inputs.

Implemented in 0.5.0-RC.


module 'pyphetools.creation' has no attribute 'create_hpoa_from_phenopacketsIndividual'

Using the latest version of pyphetools leads to this error
(...)
from pyphetools.creation import *
(...)
AttributeError: module 'pyphetools.creation' has no attribute 'create_hpoa_from_phenopacketsIndividual'

The name of the function is simply create_hpoa_from_phenopackets, and searching for create_hpoa_from_phenopacketsIndividualin the python files does not find anything. However, we do find this
grep -R 'create_hpoa_from_phenopacketsIndividual' src
Binary file src/pyphetools/creation/__pycache__/__init__.cpython-38.pyc matches
Binary file src/pyphetools/creation/__pycache__/__init__.cpython-311.pyc matches

This must be coming from here
https://github.com/monarch-initiative/pyphetools/blob/allinone/src/pyphetools/creation/import_template.py
but I cannot figure out what the issue is.

fixed


[MLLIB][SPARK-5502] User guide for isotonic regression

User guide for isotonic regression added to docs/mllib-regression.md including code examples for Scala and Java.

Test build #27297 has started for   PR 4536 at commit 94cdffb.

This patch merges cleanly.

Test build #27298 has started for   PR 4536 at commit ad98881.

This patch merges cleanly.

Test build #27299 has started for   PR 4536 at commit 504b5c3.

This patch merges cleanly.

Test build #27300 has started for   PR 4536 at commit 7d8136e.

This patch merges cleanly.

https://issues.apache.org/jira/browse/SPARK-5502 mentions the guide should be in docs/mllib-regression.md. I had to create the file. Don't we want to use existing mllib-classification-regression.md instead?

@zapletal-martin My bad. It should be mllib-classification-regression.md.

Test build #27297 has finished for   PR 4536 at commit 94cdffb.

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

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

Test build #27298 has finished for   PR 4536 at commit ad98881.

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

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

Test build #27300 has finished for   PR 4536 at commit 7d8136e.

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

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

Test build #27299 has finished for   PR 4536 at commit 504b5c3.

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

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

Test build #27352 has started for   PR 4536 at commit 80bd4c3.

This patch merges cleanly.

Test build #27354 has started for   PR 4536 at commit 67fe773.

This patch merges cleanly.

Test build #27352 has finished for   PR 4536 at commit 80bd4c3.

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

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

Test build #27354 has finished for   PR 4536 at commit 67fe773.

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

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

@zapletal-martin Could you merge the user guide to mllib-classification-regression.md?

@mengxr can you please expand on that? I am not sure I understand your request.
Isotonic regression belongs to regression category. Documentation for that category is in mllib-classification-regression.md. It however only contains links to other documentation pages and no user guides. The existing links lead to linear models, decision trees and naive bayes. Isotonic regression does not fit into any of those which is the reason why I created a new page.

@zapletal-martin You are right. Let's keep the current layout.

LGTM. Merged into master and branch-1.3. Thanks!


controller restart when GameServerSet redeploy to cluster

Background
We encountered an issue while configuring Hostport network mode using OKG's GameServerSet.
First of all, the GameServerSet has been deployed in the cluster, and the pod has been started, and its status is Running.
Then I deleted it by running "kubectl delete" on the GameServerSet. At this time, pod's status is Terminating.
While the pod has not finished exiting, I re-applied this GameServerSet. However, the newly created pod failed to obtain the Hostport.
Later, I deleted this GameServerSet again, waited for the pod to completely exit, and then re-applied this GameServerSet. The pod was able to obtain the Hostport information correctly.
Upon investigation, I found that the Kruise-game controller panicked due to this operation, which caused a restart. I suspect that the issue was caused by this operation as a whole.
Deployment file
apiVersion: game.kruise.io/v1alpha1
kind: GameServerSet
metadata:
  name: trunk
spec:
  replicas: 1
  updateStrategy:
    rollingUpdate:
      podUpdatePolicy: InPlaceIfPossible
  network:
    networkType: Kubernetes-HostPort
    networkConf:
    - name: ContainerPorts
      value: "container1:5000/TCP"
  gameServerTemplate:
    spec:
      containers:
        - image: container1-image
          imagePullPolicy: IfNotPresent
          name: container1
          env:
          - name: KRUISE_CONTAINER_PRIORITY
            value: "2"
          volumeMounts:
            - name: network
              mountPath: /opt/network
        - image: container2-image
          imagePullPolicy: IfNotPresent
          name: container2
          env:
          - name: KRUISE_CONTAINER_PRIORITY
            value: "1"
      volumes:
      - name: network
        downwardAPI:
          items:
          - path: "annotations"
            fieldRef:
              fieldPath: metadata.annotations['game.kruise.io/network-status']

    volumeClaimTemplates:
      - metadata:
          name: db-storage
        spec:
          accessModes: ["ReadWriteOnce"]
          storageClassName: "cfs"
          resources:
            requests:
              storage: 10Gi

Controller logs
1.686739246445438e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/validate-v1alpha1-gss", "UID": "e5c1cf00-f3d3-46ae-b86a-9b5b8ad537a6", "kind": "game.kruise.io/v1alpha1, Kind=GameServerSet", "resource": {"group":"game.kruise.io","version":"v1alpha1","resource":"gameserversets"}}
1.686739246445752e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/validate-v1alpha1-gss", "code": 200, "reason": "pass validating", "UID": "e5c1cf00-f3d3-46ae-b86a-9b5b8ad537a6", "allowed": true}
1.6867392465119815e+09	DEBUG	events	Normal	{"object": {"kind":"GameServerSet","namespace":"default","name":"a4","uid":"587e3fb3-cc3a-4b12-aa3b-6254ff0d7875","apiVersion":"game.kruise.io/v1alpha1","resourceVersion":"33261501457"}, "reason": "CreateWorkload", "message": "created Advanced StatefulSet"}
1.6867392465414512e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "a8cf9c48-0ab7-4366-978a-885777aee37f", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392465423882e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "a8cf9c48-0ab7-4366-978a-885777aee37f", "allowed": true}
1.6867392466240625e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "3d27baa4-34d7-4681-b3c3-42ac7bee3ea2", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392466250088e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "3d27baa4-34d7-4681-b3c3-42ac7bee3ea2", "allowed": true}
1.686739246684605e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "82bb9c68-6182-44ad-9874-d5845f78af91", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.686739246685527e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "82bb9c68-6182-44ad-9874-d5845f78af91", "allowed": true}
1.6867392467381902e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "413f0e56-ed28-46b2-a6c6-b7b183adbec2", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.686739246739107e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "413f0e56-ed28-46b2-a6c6-b7b183adbec2", "allowed": true}
1.686739246776154e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "2a41ab6a-97d3-4c68-847b-450fcebd1752", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392467770834e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "2a41ab6a-97d3-4c68-847b-450fcebd1752", "allowed": true}
1.6867392468166428e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "1da9233f-f440-4845-84b4-b807ea56279e", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392468175259e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "1da9233f-f440-4845-84b4-b807ea56279e", "allowed": true}
1.6867392470155354e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "a23451b2-dd25-4628-995f-56ca2470e466", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392470164003e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "a23451b2-dd25-4628-995f-56ca2470e466", "allowed": true}
1.6867392473775764e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "964ff6a1-5588-4aca-a1a5-1959576b29eb", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392473784983e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "964ff6a1-5588-4aca-a1a5-1959576b29eb", "allowed": true}
1.6867392481780772e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "54e1e9c6-5769-48e3-86ec-b06390cb19f8", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392481789427e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "54e1e9c6-5769-48e3-86ec-b06390cb19f8", "allowed": true}
1.6867392495441675e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "96ac2a82-7933-4894-8c27-0764d4069dd1", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392495450299e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "96ac2a82-7933-4894-8c27-0764d4069dd1", "allowed": true}
1.6867392521424189e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "0d92f93c-0e2e-4a55-ad51-e8425ce19de4", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392521432865e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "0d92f93c-0e2e-4a55-ad51-e8425ce19de4", "allowed": true}
1.68673925730318e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "16658c6b-f5e2-4976-b9c7-eb08e8d4bdb1", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392573040788e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "16658c6b-f5e2-4976-b9c7-eb08e8d4bdb1", "allowed": true}
1.6867392675877454e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "f7c72efe-8657-4122-97c4-7ebc8c530a3c", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
1.6867392675886924e+09	DEBUG	controller-runtime.webhook.webhooks	wrote response	{"webhook": "/mutate-v1-pod", "code": 200, "reason": "", "UID": "f7c72efe-8657-4122-97c4-7ebc8c530a3c", "allowed": true}
1.6867392744261086e+09	DEBUG	controller-runtime.webhook.webhooks	received request	{"webhook": "/mutate-v1-pod", "UID": "dedc8191-1383-4fd4-a151-ada8b4a73caf", "kind": "/v1, Kind=Pod", "resource": {"group":"","version":"v1","resource":"pods"}}
panic: runtime error: index out of range [-1]

goroutine 1790 [running]:
github.com/openkruise/kruise-game/cloudprovider/kubernetes.(*HostPortPlugin).deAllocate(0xc0003cf8b0, {0xc0008025d0, 0x1, 0x1a1172e?}, {0xc00099acd0, 0xc})
	/workspace/cloudprovider/kubernetes/hostPort.go:251 +0x169
github.com/openkruise/kruise-game/cloudprovider/kubernetes.(*HostPortPlugin).OnPodDeleted(0xc0003cf8b0, {0x7000000000000?, 0xc00077a060?}, 0xc000700800, {0x0?, 0x0?})
	/workspace/cloudprovider/kubernetes/hostPort.go:175 +0x14a
github.com/openkruise/kruise-game/pkg/webhook.(*PodMutatingHandler).Handle.func1()
	/workspace/pkg/webhook/mutating_pod.go:81 +0xdf
created by github.com/openkruise/kruise-game/pkg/webhook.(*PodMutatingHandler).Handle
	/workspace/pkg/webhook/mutating_pod.go:72 +0x37d

Here's the thing. HostPort Plugin allocate & deallocate hostPort for pod by webhook mechanism. When pod add event is created, plugin allocate, and when pod delete event is created, plugin deallocate.
However, there will be multiple delete events when pod is terminating, so that plugin has to use a map called isAllocated to record whether the pod's hostPort number has been allocated to avoid repeating deallocation.
The problem is that the key of map isAllocated is {pod namespace}/{pod name}, which cause the situation you mentioned. The whole process would be like this:

old pod deallocate, isAllocated is set to false.
new pod allocate, isAllocated will be changed from false to true.
old pod deallocate again because isAllocated is true.

We can find that old pod deallocate more than once.

This problem will be fixed in the next version.


Is there a way to create entity mapping automatically?

Is there a way to automatically capture the new ID of an entity spawned from the snapshot? Or I need a mechanism to match a new entity ID to the old one manually, like in the pipeline example, where they're matched by the position
upd
What I need to achieve:
I have a tilemap, a tile storage stores IDs of tile entities that belong to it. After I load tiles and the storage from the save file all tiles have new IDs so tile storage now contains invalid entity IDs. I wasn't able to figure out how to do that yet. The only idea is, like in the pipeline example, to manually map entity IDs based on some tile props. Is that the only way?

Just in case someone bumps into this question: it is actually created automatically. If you need to map entities referenced inside a component, you need to implement MapEntities trait for it and the rest will be done for you.
Simple example
use bevy::ecs::entity::MapEntities;
// this thing is needed for #[reflect(Component, MapEntities)]
use bevy::ecs::reflect::ReflectMapEntities;
use bevy::prelude::*;

#[derive(Component, Reflect)]
#[reflect(Component, MapEntities)]
pub(crate) struct Inventory {
    entities: Vec<Entity>,
}

impl MapEntities for Inventory {
    fn map_entities<M: EntityMapper>(&mut self, entity_mapper: &mut M) {
        self.entities = self.entities
            .iter()
            .map(|entity| entity_mapper.map_entity(*entity))
            .collect();
    }
}


Publish on Bower

Hi Michael, would you consider publishing js-toast on Bower so that it can be used proper in Bower pipelines?
Thanks for teh lib btw!

Never mind. Issue turned out to be something else. I made some changes to the library so it could be imported whether in an npm or window or amd environment (took some notes from HammerJS) Would you consider this for js-toast? Happy to send in a PR in that case.
How we're using it in production: https://github.com/ihatehandles/js-toast/blob/master/toast.js

Hi Sam, your fork looks very nice! However, Quantum.js does not support modules yet, so I cannot pull your changes just for Toast.js yet. I will definitely consider it after I figure out how to sort out Quantum's module handling. Thank you :+1:

Nice, didn't know about Quantum.js - will definitely check it out.
Will wait on the module support (however you will implement it) - please ping me when it's ready.
How I wired it up into Webpack via Bower:

Alias in webpack.config.json
Regular Bower install
Imported and used

Cheers!

@ihatehandles
Hi Sam, Toast can be used as a module now.
Using toast.min.js:
const Toast = require('./toast');
Toast(...);

Using toast.mod.js, the require() function is in window, so it can be required directly in the browser:
const Toast = require('iqwerty-toast');
Toast(...)

Hopefully this is useful to you! Thanks :smile:


Updated copyright to 2016

Sorry, but this change is completely unnecessary.

Sorry, but this change is completely unnecessary.


Keycloak 120

Adding Keycloak 1.2.0, done for AGPUSH-1398
Please test and review, @abstractj @cvasilak

@matzew I have other things on my plate and I don't think it requires a security analysis, more testing. Feel free to ask others to review.

@secondsun mind to take a look ?

One thing I just noticed, clicking on the "account manager" link of our UPS console, I am redirected to the account manager, but on the SERVER console, I see:
08:55:49,958 ERROR [org.keycloak.freemarker.ExtendingThemeManager] (default task-20) Failed to find ACCOUNT theme patternfly, using built-in themes
08:55:49,987 ERROR [org.keycloak.freemarker.ExtendingThemeManager] (default task-28) Failed to find ACCOUNT theme patternfly, using built-in themes

@sebastienblanc @stianst any ideas?

@matzew maybe before looking further we should rebase so that we have the latest account styling changes from @andresgalante  ( https://github.com/aerogear/aerogear-unifiedpush-server/commit/08b34a3d8b8effe6d4e8b27a62aa9b7d011961eb )

well, it's rebased :-)

@sebastienblanc you mind to test ?

:+1:


[5.9] Slightly simplified user provider

Might cause backward compatibility issues in projects overriding the retrieveById function.
Makes extending EloquentUserProvider easier as only the retrieveById function must be reimplmeneted. The retrieveByToken has a common behavior.

I am closing this pull request because it lacks sufficient explanation, tests, or both. It is difficult for us to merge pull requests without these things because the change may introduce breaking changes to the framework.
Feel free to re-submit your change with a thorough explanation of the feature and tests - integration tests are preferred over unit tests. Please include it's benefit to end users; the reasons it does not break any existing features; how it makes building web applications easier, etc.
Thanks!


Added a Parent Edit tab

Hi,
Any news for this?

Can you check the conflicts and remove the f***ing lines.. :-D

Closing this for inactivity.


py shortcut icon still broken

Details
I have just installed ida.diaphora.vm 3.2.0 and the  icon of the .py file looks broken:

@emtuls hadn't this been addressed in https://github.com/mandiant/VM-Packages/issues/675?
We are planing to change how diaphora is installed (without shortcut): https://github.com/mandiant/VM-Packages/issues/994, so this issue is about ensuring that the .py shortcut look nice and not about fixing diaphora specifically. But I noticed the issue while testing diaphora.

Strange...I just tried it on a fresh setup and it looks fine for me? 🤔

Ah, so I was mistaken. It appeared to work because of my vscode I had installed.
So, this issue is because we opted to go the route of requiring tools that were python based to use python as the executablePath (which will make python the Target for the shortcut), which will in turn make python the icon due to iconLocation being set to executablePath: https://github.com/mandiant/VM-Packages/blob/main/packages/common.vm/tools/vm.common/vm.common.psm1#L228
The downside that this had was that we don't default all Python files to use the Python icon as a shortcut if they are something like ida.diaphora, since it's just a python plugin and we weren't planning to execute it directly with Python.
The other downside is that we have to call VM-Install-Shortcut directly, like such: https://github.com/mandiant/VM-Packages/blob/main/packages/didier-stevens-beta.vm/tools/chocolateyinstall.ps1#L22-L25
An alternative we could do, that I think I may have suggested somewhere, would be to add inside of VM-Install-From-Zip, a check for if the $executableName has an extension of .py, and then we could do:
if ([System.IO.Path]::GetExtension($executableName) -eq ".py") {
    $executablePath = (Get-Command python).Source
    $filePath = Join-Path $toolDir "$toolName.py"
    $arguments = $filePath + $arguments
    $consoleApp = $true
    VM-Install-Shortcut -toolName $toolName -category $category -executablePath $executablePath -consoleApp $consoleApp -arguments $arguments
}

I've just tested that and it would work, if you want to go that route.

Thanks for all the research @emtuls! I do not like the idea of complicating VM-Install-From-Zip just because of one package, specially because diaphora is a very special package (and IDA plugin which we are not installing as a plugin). I think we should focus on https://github.com/mandiant/VM-Packages/issues/994 and close this issue.

Sounds good!
Just a heads up, if I'm not mistaken, if we start to get more python tools, we will be forced to use VM-Install-Shortcut directly unless we choose to do something like https://github.com/mandiant/VM-Packages/pull/1011 in the future. 🙂


Use original client IP when checking ACLs

https://www.pivotaltracker.com/story/show/179659713
Previously we used "client.ip" to control access to Staging and
Integration, but this won't work for GOV.UK Alerts, since the IP
gets modified due to its use of the shielding feature 1. This
moves setting the existing Fastly-Client-IP header earlier so that
we can reuse it as a reliable means of checking who's allowed in.
Disclaimer: I haven't tested this, since I don't have access to do
so. I'm fairly confident the header will always be set, so I don't
expect any of the subsequent uses of it to fail.

Closing as this isn't something we're actively working on.


request: Update to typedoc 0.26

Is there an existing issue or pull request for this?

[X] I have searched the existing issues and pull requests

Feature description
Please update typedoc to the latest version (0.25.13 => 0.26.2). We cannot update typedoc in our app because it will show an error "TypeDoc has been loaded multiple times".
Desired solution
Update to latest version of typedoc.
Alternatives considered
Adding an npm override works but is ugly.
Additional context
No response

I will be updating to latest typedoc later today. There were a lot of breaking changes so I need to go through and fix them all.

#249


APKEditor can't handle tags in string resources

Describe the bug
If you decompile with APKEditor and build again, tag processing, which was originally good, does not work.
Please refer to the picture for related information

To Reproduce
Steps to reproduce the behavior:

Used version 1.2.5
Operating system macOS 13.4.1
Command

decompile: java -jar ~/APKEditor-1.2.5.jar d -i kakaotalk.apk -o kakaotalk
compile: java -jar ~/APKEditor-1.2.5.jar b -i kakaotalk -o kakaotalk.apk

Log/Stacktrace
Nothing.
Used apk file
Attach/share your apk
Additional context
Related classes are the same as xc.f, and you can refer to the label_multi_photo_guide resource.

Check the latest update

It's working! thanks


How to visualize the motion in scene on the HumanML3D benchmark

Hi, thanks for the great work and amazing code!
I have a small question: How to visualize the motion in scene on the HumanML3D benchmark? I have tried the visualize_h3d.py and found that it can only visualize joints. Looking forward to your reply~

Hi, I follow the HumanML3D to convert the skeleton into SMPL mesh; please refer to https://github.com/EricGuo5513/HumanML3D/issues/1.
For all results in the paper, I first save the results as .obj/.ply files and then import them into Blender for manual rendering.

Thanks for your reply!
Maybe I didn't phrase it very clear. I know how to convert the skeleton into SMPL mesh and I want to visualize the motion together with the scene (interaction with the scene) on the HumanML3D benchmark, just like on the HUMANISE betchmark:

But the visualization on the HumanML3D benchmark does not include the scene:

Looking forward to your reply~

I see. But, the HumanML3D benchmark doesn't include the 3D scene. In the experiment on the HumanML3D benchmark, our work just augments the HumanML3D motion sequences with a ground without other objects, so we didn't visualize it.

Okay, thanks for your reply again!


[libra framework] fix specs in p2p script and beyond

Motivation
This PR

Fixes specs and turns verify back on for peer_to_peer_with_metadata transaction script.
Adds some global invariants in LibraAccount needed by the transaction scripts peer_to_peer_with_metadata and rotate_authentication_key.

Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
cargo test

@bors-libra land


Question: why is there no entity.findById(id) ?

When I look up an Object, why do I need to pass the class?
Book book = Book.findById(Book.class, 1);
could be wrapped in
Book book = Book.findById(1);
automatically? When calling on Book, Book knows its a Book? :)
See https://developer.jboss.org/wiki/GenericDataAccessObjects?_sscc=t
how they get the persistenceClass within the Constructor, which later can be used as T in findById
Best,
Andreas

(findById is a static method defined and implemented in SugarRecord class).. In a static method, we always get the class as the one where it's present. So, we never get Book's class, infact we get the SugarRecord class. That's the reason, we're having to pass the Book.class as a parameter.
I'd love to remove the class parameter, but haven't found a more viable solution yet.

Hello!
The solution is in https://developer.jboss.org/wiki/GenericDataAccessObjects?_sscc=t
You can detect the Type t upon instance creation via
public SugarRecord(){
this.pclass = (Class) ((ParameterizedType) getClass()
.getGenericSuperclass()).getActualTypeArguments()[0];  //thsi gives Book Type in "Book"
}
then:
public T findById(long id) {
T entity = (T) findById(this.pclass, id);
    return entity;  // a Book!
}

Ah, just realized that its of course a static call..

You can always overload the method in your entity passing the class statically, like this:
public class Book extends SugarRecord<Book> {
 public static <T> T findById(Long id) {
    findById(Book.class, id);
}

Improves readability with no significant downside.

@anti43, is there anything else? I'd like to close this to clean up the issues list.


How much wait for the new source

It's been so long since you announced , plz do it fast , it's the only thing lacking in your app , if you add some better sources ISTG the will become just netflix

deceptions said that he wants a bug free app before adding a new site (idk if thats still the plan). If you guys are gonna report bugs please follow the bug template to keep questions to its minimum. You can join Kuro s community/discord server to ask for more info and help

deceptions said that he wants a bug free app before adding a new site (idk if thats still the plan). If you guys are gonna report bugs please follow the bug template to keep questions to its minimum. You can join Kuro s community/discord server to ask for more info and help

Yeah but watching naruto is difficult sometimes (quality on some eps is 360p)


Unable to add action buttons to empty state in view.

The list/card/table views have a parameter to set the configuration of an empty state view, but do not have a parameter to set the action buttons for the configured empty state view.

Merged


Improve registry namespace usage

From my understanding, this fixes #1286.

It will, yes.

One other thing - should the namespace also be lower-cased?  Right now you're lowercasing the value but not the namespace; the vanilla ResourceLocation lowercases both.  I don't think that that code would be used externally (so it wouldn't really matter too much) but it's worth considering.

I lowercase the entire string if it includes the namespace: https://github.com/SpongePowered/SpongeCommon/pull/1552/files#diff-fe71dfd1d29d9150251c9f6ca2af9064R41

Right, but what about passing an uppercase namespace parameter to namespacedId?  I doubt it would happen in practice (I'd assume that the parameter wouldn't ever be changable from user code), but it's a case to consider.

Moving to #1554


Add parameter to customize the number of cinderAPI processes

@ASBishop one interesting thing here is that we have apiTimeout as top level parameter, while I think it's specific to CinderAPI.
I'd like to consolidate those parameters into a substruct (httpdTuning or something like that), scoped at CinderAPITemplate level, but this might be part of a follow up patch to keep the focus on the goal of this change.

form httpd tuning, check this keystone PR https://github.com/openstack-k8s-operators/keystone-operator/pull/500 . we should do the same here


[FLINK-22197][tests] Shutdown cluster before collecting checkpoints

What is the purpose of the change
UnalignedCheckpointRescaleITCase iterates over the checkpoints to return a completed one.
Currently, during the iteration the cluster isn't shut down so it can concurrently remove any files read.
With this change, cluster is shut down first and then checkpoint is collected.
Verifying this change
This change is a trivial rework without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? no

Build failure is unrelated (FLINK-22085).


How to use odoo enterprise addons with doodba

I want to add enterprise addons for my testing environment and I am getting error message.
This is my repos.yaml.
# See https://github.com/Tecnativa/doodba#optodoocustomsrcreposyaml
./odoo:
  defaults:
    # Shallow repositories ($DEPTH_DEFAULT=1) are faster & thinner
    # You may need a bigger depth when merging PRs (use $DEPTH_MERGE
    # for a sane value of 100 commits)
    depth: $DEPTH_DEFAULT
  remotes:
    ocb: https://github.com/OCA/OCB.git
    odoo: https://github.com/odoo/odoo.git
    openupgrade: https://github.com/OCA/OpenUpgrade.git
  target: ocb $ODOO_VERSION
  merges:
    - ocb $ODOO_VERSION
    # Example of a merge of the PR with the number <PR>
    # - oca refs/pull/<PR>/head
./private:
  defaults:
    depth: $DEPTH_DEFAULT
  remotes:
    enterprise: https://github.com/odoo/enterprise.git
  target: enterprise $ODOO_VERSION
  merges:
    - enterprise $ODOO_VERSION

This is my configuration for /odoo/custom/ssh directory.
This is config file.
Host github.com
  HostName github.com
  IdentityFile ~/.ssh/id_rsa
  StrictHostKeyChecking no

I updated id_rsa and id_rsa.pub.
Then, I run ssh-keyscan -t rsa github.com >> ~/.ssh/known_hosts for the known_hosts.
After that, I run sudo docker-compose -f test.yaml up -d and I got the error like below image.

@qrtl

I think the SSH should be put on the host machine that is doing the git aggregation, in the user that performs it through Docker. I don't use enteprise since long, but I remember to use a deploy token as something that can be embedded in the repos.yaml so no extra config is needed.

@pedrobaeza  I tried to use github token for enterprise repo like this.
./odoo:
  defaults:
    # Shallow repositories ($DEPTH_DEFAULT=1) are faster & thinner
    # You may need a bigger depth when merging PRs (use $DEPTH_MERGE
    # for a sane value of 100 commits)
    depth: $DEPTH_DEFAULT
  remotes:
    ocb: https://github.com/OCA/OCB.git
    odoo: https://github.com/odoo/odoo.git
    openupgrade: https://github.com/OCA/OpenUpgrade.git
  target: ocb $ODOO_VERSION
  merges:
    - ocb $ODOO_VERSION
    # Example of a merge of the PR with the number <PR>
    # - oca refs/pull/<PR>/head
./private:
  defaults:
    depth: $DEPTH_DEFAULT
  remotes:
    origin<EMAIL_ADDRESS>  merges:
    - origin $ODOO_VERSION

But I still get the error.
64.86 From https://github.com/OCA/OCB
64.87  * branch              18.0       -> FETCH_HEAD
64.87  * [new branch]        18.0       -> ocb/18.0
64.88 (INFO) [08:39:33] git_aggregator.repo  odoo   Checking repo status
65.69 (INFO) [08:39:34] git_aggregator.repo  odoo   Reset branch to ocb 18.0
66.98 (INFO) [08:39:36] git_aggregator.repo  odoo   Execute shell after commands
66.98 (INFO) [08:39:36] git_aggregator.repo  odoo   End aggregation of /opt/odoo/custom/src/odoo
66.98 (INFO) [08:39:36] git_aggregator.repo  private   Start aggregation of /opt/odoo/custom/src/private
66.98 (INFO) [08:39:36] git_aggregator.repo  private   Switch to branch _git_aggregated
66.99 fatal: not a git repository (or any of the parent directories): .git
66.99 (ERROR) [08:39:36] git_aggregator.repo  private   /opt/odoo/custom/src/private> error calling ['git', 'checkout', '-B', '_git_aggregated']
67.00 Traceback (most recent call last):
67.00   File "/usr/local/lib/python3.10/site-packages/git_aggregator/main.py", line 230, in aggregate_repo
67.00     repo.aggregate()
67.00   File "/usr/local/lib/python3.10/site-packages/git_aggregator/repo.py", line 180, in aggregate
67.00     self._switch_to_branch(self.target['branch'])
67.00   File "/usr/local/lib/python3.10/site-packages/git_aggregator/repo.py", line 298, in _switch_to_branch
67.00     self.log_call(['git', 'checkout', '-B', branch_name], cwd=self.cwd)
67.00   File "/usr/local/lib/python3.10/site-packages/git_aggregator/repo.py", line 160, in log_call
67.00     ret = callwith(cmd, **kw)
67.00   File "/usr/local/lib/python3.10/subprocess.py", line 369, in check_call
67.00     raise CalledProcessError(retcode, cmd)
67.00 subprocess.CalledProcessError: Command '['git', 'checkout', '-B', '_git_aggregated']' returned non-zero exit status 128.
67.04 Traceback (most recent call last):
67.05   File "/usr/local/bin/autoaggregate", line 170, in <module>
67.05     aggregate(REPOS_YAML)
67.05   File "/usr/local/bin/autoaggregate", line 72, in aggregate
67.05     check_call(
67.05   File "/usr/local/lib/python3.10/subprocess.py", line 369, in check_call
67.05     raise CalledProcessError(retcode, cmd)
67.05 subprocess.CalledProcessError: Command '['gitaggregate', '--expand-env', '--config', '/opt/odoo/custom/src/repos.yaml', '--log-level', 'INFO', '--jobs', '1', 'aggregate']' returned non-zero exit status 1.
67.07 Traceback (most recent call last):
67.08   File "/opt/odoo/common/build", line 30, in <module>
67.08     subprocess.check_call(command)
67.08   File "/usr/local/lib/python3.10/subprocess.py", line 369, in check_call
67.08     raise CalledProcessError(retcode, cmd)
67.08 subprocess.CalledProcessError: Command '/opt/odoo/common/build.d/100-repos-aggregate' returned non-zero exit status 1.

I thought private folder is for the private repos and I tried to use this for enterprise addons. Am I missing something about the workflow?

./private is reserved for local addons (customer custom code).

Have you tried just using ./enterprise as your folder prefix?

@chienandalu I did and there is no issue when I run sudo docker-compose -f test.yaml up -d but I don't see my enterprise addons inside directory. And what I thought is even it is exist in the directory, do we have to update odoo.conf to add enterprise addon?
lin@localhost:~/doodba$ sudo docker exec -it 887302afa32f /bin/bash
odoo@doodbatesting:/opt/odoo$ ls
auto  common  custom  requirements.txt
odoo@doodbatesting:/opt/odoo$ cd custom/
odoo@doodbatesting:/opt/odoo/custom$ ls
build.d  conf.d  dependencies  entrypoint.d  src  ssh
odoo@doodbatesting:/opt/odoo/custom$ cd src/
odoo@doodbatesting:/opt/odoo/custom/src$ ls
addons.yaml  odoo  private  repos.yaml

My repos.yaml.
# See https://github.com/Tecnativa/doodba#optodoocustomsrcreposyaml
./odoo:
  defaults:
    # Shallow repositories ($DEPTH_DEFAULT=1) are faster & thinner
    # You may need a bigger depth when merging PRs (use $DEPTH_MERGE
    # for a sane value of 100 commits)
    depth: $DEPTH_DEFAULT
  remotes:
    ocb: https://github.com/OCA/OCB.git
    odoo: https://github.com/odoo/odoo.git
    openupgrade: https://github.com/OCA/OpenUpgrade.git
  target: ocb $ODOO_VERSION
  merges:
    - ocb $ODOO_VERSION
    # Example of a merge of the PR with the number <PR>
    # - oca refs/pull/<PR>/head
./enterprise:
  defaults:
    depth: $DEPTH_DEFAULT
  remotes:
    origin<EMAIL_ADDRESS>  merges:
    - origin $ODOO_VERSION

Why are you trying with test.yaml instead of devel?
About the syntax with deploy tokens, are you sure the URL is right?
One thing is sure, you need to put other repo name (like ./enterprise). ./private can't be used.

Why are you trying with test.yaml instead of devel?

I am using for staging purpose.

One thing is sure, you need to put other repo name (like ./enterprise). ./private can't be used.

I can run successfully with enterprise by using ./enterprise in repos.yaml and adding enterprise: ['*'] in addons.yaml.

OK, then that was the problem.

@chienandalu @pedrobaeza Thanks for the help!

Can I ask another question in here?
I was not very clear about the intention of /odoo/custom/src/private. My understanding was this is for the customer custom addons and the addons under private folder will be linked automatically with odoo/addons as per this explanation from addons.yaml.

Addons under private and odoo/addons are linked automatically unless you specify them.

Then, I found

Recommendation: use repos.yaml for everything except for private, and ignore in your .gitignore and .dockerignore files every folder here except private

I am confusing how I should add my customer custom addons. Should I add like below using addons.yaml?
# Custom repositories
ENV:
  DEFAULT_REPO_PATTERN: https://github.com/qrtl/{}.git
  ODOO_VERSION: 16.0
some-repo:['*']

Or
Do I have to clone manually inside private folder?
Am I missing something about private folder?

You don't need to declare the private addons in the addons.yaml or repos.yaml. Just put their files in the private folder.

@chienandalu
I created my doodba project.
Then, put my customer addons like below.
/odoo/custom/src/private/my-private-repo
  ├── module_1/
      ├── __manifest__.py
      ├── __init__.py
      ├── models/
      ├── views/
  ├── module_2/
      ├── __manifest__.py
      ├── __init__.py
      ├── models/
      ├── views/
  ├── module_3/
      ├── __manifest__.py
      ├── __init__.py
      ├── models/
      ├── views/

Then, I run sudo docker-compose -f test.yaml up -d but I don't see my modules in the odoo instance's apps list.
I also checked inside docker /opt/odoo/custom/src/private and I can see my addons.
And I am curious about why does doodba put odoo code inside the docker.
If we put odoo code inside the docker,

We need to get inside the Docker container to inspect the Odoo code
Docker image gets larger than otherwise
Odoo code update process gets a bit cumbersome as we need to update the Docker image

Modules must be directly inside private, without subfolders. And please use discussions for these questions.


can't work in *.vue file

I found the plugin not watching *.vue suffix, It would be great if you could support it.Thanks

Still doesn't


Dependency issue

Hi,
can you check why
    "@types/express": "4.17.8",
    "@vue/test-utils": "^2.4.1",

are dependencies and not dev-dependencies? This is causing a bit of overhead in package size.
Thanks
Yves

Maybe I was wrong. I'll check it


meanings of dense features/embedding tables

Hi, thanks for giving useful and interesting works.
While doing my study regarding recommendation systems (especially DLRM), it was very hard for me to clearly understand the meanings of a dense feature and embedding tables. I searched the web to find out their meaning, but there are few materials about them. Would you mind if you help me to understand the meaning of the input dense features and embedding tables? So, my questions are as follows.

(a-1) As far as I know, embedding tables hold some categorical data. For example, embedding tables 0/1/2 indicate movie/music/car, respectively. Is it right?
Otherwise, (a-2) does each table indicate a category such as a non-specific category which humans cannot recognize in a specific word such as movie/music/car?
Moreover, (b-1) assuming that each embedding table holds ‘n’(=the number of rows in each embedding table) entries, do the entries indicate specific movie titles? For example, in the embedding table 0 which holds embedding vectors regarding movies, entry(=row) 0 holds an embedding vector about movie ‘Parasite’, entry 1 holds an embedding vector about movie ‘Harry Potter’, and entry 2 holds an embedding vector about movie ‘Star Wars’, and so on. Is it right?
Otherwise, (b-2) do the entries only illustrate categories of the movies (not the specific titles) such as Fantasy, Action, Comedy, and so on? Or, do they have any other meanings?

Furthermore, do dense features illustrate non-sparse user data such as birthday, nationality, and etc (in contrary to the sparse indices which illustrate sparse user data) and the characteristic data of the input bench? (for example, if we use an advertisement model, part of the dense features illustrates the characteristics of each single advertisement.)

(a-1) Assuming that embedding table 0 is related to the movie category (if question 1-b-1 is right) and the number of movies/music is getting larger because a lot of movies/music are released each week, does the row/size of the embedding tables also get larger? For example, the number of rows of the embedding table 0 (related to the movie) is 1000 in this week because 1000 movies have been released in this week. If 50 movies are released next week, the number of rows of the embedding table 0 becomes 1050 in the next week. Is it right?
(a-2) Otherwise, is the #row/size of the embedding tables fixed although many movies/music are released over and over?

Thank you in advance!

Let me try to answer your questions below:

I usually think of a sparse feature as an index (or a set of indices), that correspond to an item. It is often helpful to embed this index/item into real d-dimensional space R^d because it provides an abstract representation of the item. Notice that the representation is learned during training. The collection of such embedding representations (vectors) is the embedding table.

I usually think of a dense feature as a floating point number that can represent an arbitrary input. You can feed anything to it.

The embedding tables can change over time. However, DLRM provides a mechanism for capping their size --max-ind-range=<number> which caps the size and mods all indices into the given range.

You will find further examples in the following issue https://github.com/facebookresearch/dlrm/issues/22
Also, please take a look at my talk "Training Deep Learning Recommendation Models" at the PeRSonAI Tutorial at ASPLOS 2020. It should give you additional background and references about the subject.
https://personal-tutorial.com/personal-at-asplos-2020/

Thank you for your kind reply.
I have one more question.
Let’s assume that embedding table 0 has 1000 entries now, but the number of the entries becomes 1050 in a few days.
In this case, a) are all the weights in the embedding table 0 fine-tuned during the re-training process? or b) are all the weights trained from scratch during the time?
Thank you so much!

There are many different approaches that work better in different circumstances. I think that we have addressed some of these questions in in section 2 of the "Bandana: Using Non-volatile Memory for Storing Deep Learning Models" paper https://arxiv.org/abs/1811.05922. Please take a look at it.

Thank you for your quick response.
Your comments have been very helpful to me!

It seems that this issue is resolved. Closing.


Added easybook.io integration

Hi,
This pull request adds support for easybook.io.
You can test the integration by signup on easybook web app: https://app.easybook.io/register

Hey,
Thanks for contributing!
However, I'm having trouble finding the button in easybook. I have a feeling easybook may show a newer design for newly-created accounts as it looks a bit different from your screenshot.
Here's what my "Edit Organization" screen looks like.
Also, please run make lint and fix any linting errors :)

Hey @adrihein , we decided to clear up old PRs to make way for some big refactoring that is upcoming. If you wish to continue work on this PR please feel free to reopen or add an issue for adding easybook.io support!


Always reflect kube-root-ca.crt configmap in offloaded namespaces

Description
This PR modifies the kubelet to always reflect the kube-root-ca.crt ConfigMap in offloaded namespaces.
The ConfigMap must always be reflected for the offloaded pods to work, even if using a restricting policy as the AllowList .
How Has This Been Tested?

[x] locally
[x] e2e tests
[x] unit tests

/rebase test=true

/merge


Acid Claws (melee cantrip) is getting ranged attack penalty

[edit] The ranged attack penalty appears inconsistent.  I had be a non-disadvantage in another fight with my first attack.

As this is a melee range cantrip, should it have disadvantage?

Fixed on dev. Thanks.


is it possible to use MPT-30B-instruct with the MPT integration?

Hi everyone. First of all, thank you for adding MPT as a supported model.
I was wondering how flexible it is to add mpt-30B-instruct to be used for inference, instead of vanilla MPT-30b. Do you have any guidelines to help with that?

@paulovasconcellos-hotmart I have tested the mpt-30B-instruct on a few medical benchmarks and I experience no issue. Just follow the recommended prompt formatting:
def format_prompt(instruction):
    template = "Below is an instruction that describes a task. Write a response that appropriately completes the request.\n\n###Instruction\n{instruction}\n\n### Response\n"
    return template.format(instruction=instruction)

example = "Tell me a funny joke.\nDon't make it too funny though."
fmt_ex = format_prompt(instruction=example)


What's the meaning of the self.rotate_deform_attn?

Here is some code in the file dn_ars_rotated_transformer.py.It ensures the reference_angle_input in [-pi/4,pi/4],is it necessary? Abd why should do this?
if self.rotate_deform_attn:
    index = (reference_angle_input > math.pi / 4).view(-1) | \
            (reference_angle_input < -math.pi / 4).view(-1)
    bs, q_num, layer_num, points_num = reference_points_input.shape
    reference_points_input = reference_points_input.view(bs * q_num, layer_num, points_num).contiguous()
    reference_points_input[index, :, 2], reference_points_input[index, :, 3] = \
        reference_points_input[index, :, 3], reference_points_input[index, :, 2]
    reference_points_input = reference_points_input.view(bs, q_num, layer_num, points_num).contiguous()
    reference_angle_input = (reference_angle_input + math.pi / 4 - reference_angle_input * 1e-10) % \
                            (math.pi / 2) - math.pi / 4 + reference_angle_input * 1e-10

The initial bias of sampling offsets are located in eight directions of the square, so the suitable range would be [-pi/4, pi/4]. We have also tested that adding this constraint results in better effects.