gdx: Android: Build ELFs to support 16kB page size

Starting from V, android introduces support for 16kB page sizes. The NDK [1] and toolchain [2][3] have been updated to support the larger page size.
Update GDX build rules for android to use a 16kB max page size.
[1] https://github.com/android/ndk/wiki/Changelog-r27#announcements
[2] https://github.com/llvm/llvm-project/pull/70251
[3] https://github.com/llvm/llvm-project/pull/87413

Thank you for bringing this up!
This is a good change, but I think libGDX is not the place that should implement this, but https://github.com/libgdx/gdx-jnigen directly.
You can either open a PR there or I can do it too, however you like.
Also, the docs mention to define __BIONIC_NO_PAGE_SIZE_MACRO, just in case.
They also mention to only set the page size when building for arm64-v8a or x86_64, but I wouldn't know why this is relevant/has any downside for the other targets?

They also mention to only set the page size when building for arm64-v8a or x86_64, but I wouldn't know why this is relevant/has any downside for the other targets?

I think it would just be a waste of space to 16K-align for 32-bit architectures, as there won't be any 32-bit devices running Android 15 (I'd like to end this sentence here but I'll hedge my bets) that have a page size larger than 4K. ARMv9 devices still use the arm64-v8a libs. It shouldn't break them, not that I've tested.

Hi Berstanio,
Thanks for the pointers.

You can either open a PR there or I can do it too, however you like.

TBH I'm not very familiar with the build system here. I'd prefer if you can take a stab at it.

The docs mention to define __BIONIC_NO_PAGE_SIZE_MACRO

You are right we also need to to remove uses of #define PAGE_SIZE 4096

They also mention to only set the page size when building for arm64-v8a or x86_64, but I wouldn't know why this is relevant/has any downside for the other targets?

The reason for this is that only arm64 supports 16kB page size. On Android, x86_64 emulators for studio emulate the 16kB page size, so we need it there as well.
A larger ELF max-page-size would also work on other archs but is not needed since they don't support larger than 4kB page sizes.
The caveat is that the larger segment alignment in the ELFs cause the .so's to be slightly larger due to zero padding. On android this is mitigated by punching holes in the .so's padding when the app is installed. (i.e the disk space usage will not increase although the "apparent" file size will be a bit larger)
Thanks,
Kalesh

Hi Berstanio, Frosty-J,
Do we have an ETA for when a build would be ready with these changes?
Thanks,
Kalesh

Do we have an ETA for when a build would be ready with these changes?

If you mean a ready libGDX snapshot build, it's hard to tell.

I would need to get around implementing the changes in jnigen (hopefully this weekend)
They need to get tested, reviewed and merged into jnigen
A release of jnigen needs to be done
libGDX needs to get updated to the new jnigen version

All of this might take a bit of time.

If you mean a ready libGDX snapshot build, it's hard to tell.

From what I understand, some apps use directly the prebuilts from the libgdx build server.

I would need to get around implementing the changes in jnigen (hopefully this weekend)
They need to get tested, reviewed and merged into jnigen
A release of jnigen needs to be done
libGDX needs to get updated to the new jnigen version

All of this might take a bit of time.

Understood. I'd appreciate if you can keep us posted once these happen.
Thanks,
Kalesh

Do we have an ETA for when a build would be ready with these changes?

If you mean a ready libGDX snapshot build, it's hard to tell.

I would need to get around implementing the changes in jnigen (hopefully this weekend)
They need to get tested, reviewed and merged into jnigen
A release of jnigen needs to be done
libGDX needs to get updated to the new jnigen version

All of this might take a bit of time.

Hi @Berstanio, have yo had a chance to look into this?

Hi @Kalesh-Singh ,
sorry for taking so long, I have opened the jnigen PR: https://github.com/libgdx/gdx-jnigen/pull/73


(Bug) Fullstory session url is not logged in sentry

[x] sentry errors do not have the session url
[x] session url on amplitude should be per event not  a user propery

In the previous task we had to add it to amplitude only. So why we haven't added it to Sentry
They're added both to user profile (getSessionUrl) and to the error events (getCurrentSessionUrl). Non-error events send via fireEvent aren't tagged with session url. Should we remove it from user profile ? And how about non - error events, should we add fs session url to them ?

no just error events both sentry and amplitude

@sirpy As far as I know, the previous ticket related to session URL should be tested on the prod
what the env where it should be tested now?

fullstory is only enabled on prod

@sirpy On the amplitude, session url is displayed
what about sentry, as far as I know, it isn't displayed for next

Checked on NEXTorg V1.1.2
Desktop: Windows 10 // Google Chrome  84.0.4147.135


Fix color definition typos

I found two typos in color definitions which I want to fix with PR #97. Although colors look good at first sight, it may lead to wrongly shown colors when switching icons from a previous color.

@styxlab addressed in release https://github.com/jaketrent/react-social-icons/releases/tag/v5.6.1, thank you


Insecure password hash

The current repo uses SHA-256 hashes for user's passwords here and here. This is insecure, since it's easy to determine simple passwords given their SHA-256 hash: repeated guesses will match the hash eventually, or the hash can even be looked up.
Two ways to eventually improve this could be:

Use a more secure hashing algorithm like Ruby's bcrypt()
Enforce minimum length passwords

While I agree with the notion that SHA-256 is generally insecure, when I first wrote the program I was using bcrypt; however, there were some really yucky dependency issues with windows and osx (with ffi). Since the registration and login signup processes were just temporarily removed, I'll work on fixing the dependency issues and convert the hashes to bcrypt.


Create overview figure panel

Create high level ("graphical abstract") overview of what HTSinfer does, in vector graphics.

Would something like this work?

This is great - thanks a lot @balajtimate!
The only thing I'm not sure about is the tool names in the blue boxes. What do they signify? Do we use kallisto to "infer" the library source, STAR for the read orientation and Cutadapt for the 3' adapter sequences?

And perhaps about the pictograms:

The library type isn't very obvious. Might be better to have something that indicates single vs paired. Not sure what though.
For the 3' adapter sequence, I think that having two lines, at first glance, suggests a double strand (even though it's not) - so perhaps increase the letter sice a bit and then use only one row?

Do we use kallisto to "infer" the library source, STAR for the read orientation and Cutadapt for the 3' adapter sequences?

Yeah, exactly. Technically, STAR is also used for library type inference when the seq_ids are not present, but for the read orientation it's always used, so for the overview I think it's enough to put it there.
If the flow is not obvious, I could put some arrows to indicate it more visibly.

The library type isn't very obvious. Might be better to have something that indicates single vs paired. Not sure what though.

Yeah 😅 Maybe a single and double arrows from the read orientation pictogram, and for that, add another pair of arrows showing the forward read as 2, and reverse as 1?

so perhaps increase the letter sice a bit and then use only one row?

Sure, I'll  change that.

Yes, good idea about the library type pictogram.
I would probably put STAR in as well for the type inference, but maybe with a slightly in a lighter blue shade or with a pair of brackets around it? Just for completion.
Also, how is Cutadapt used for the 3' adapter sequence? I thought Rohan implemented some fancy search algorithm for that. Sure Cutadapt isn't just used for polyA tails?

Here is an updated version:

Great! I think you can just attach the SVG file as well here, then close the issue :)

Here is the SVG:


Deployment doesn't fail when the environment is unavailable

Expected Behaviour
The Github workflow responsible for deploying an updated theme to AaCS fails when the instance is unavailable. It only succeeds when the artifact has been successfully deployed.
Actual Behaviour
If deployment is attempted while the AEM instance is unavailable, the workflow is marked as successfully completed, which could be confusing under certain circumstances.
Under the hood, npx aem-site-theme-builder deploy makes an HTTP request to transfer the zip to AEM. If it receives a non-successful response code, the failure is silently ignored.
Reproduce Scenario (including but not limited to)
Steps to Reproduce

Create a site based on the AEM Site Template
Put it under version control on Github and configure Github Actions to automate deployments
Hibernate the Environment via the Developer Console
Trigger the Build and deploy to Github artifacts workflow
Wait for the Update theme config on AEM with artifact id workflow to kick off and inspect the log

Platform and Version
Site generated using "@adobe/aem-site-theme-builder": "4.0.1"
Sample Code that illustrates the problem
Logs taken while reproducing problem
From the output of the update_theme_config workflow kicked off while the relevant environment was hibernated. AEM responds with HTTP 503 but the workflow succeeds.
Run npx aem-site-theme-builder deploy
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  1269  100  1141  100   128   2616    293 --:--:-- --:--:-- --:--:--  2903
<!DOCTYPE html>

                  <meta charset="utf-8">
                  <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
                  <meta name="description" content="AEM Cloud Service">
                  <title>AEM Cloud Service</title>
                  <link rel="stylesheet" href="https://errors.adobeaemcloud.com/src.status.css">
                  <link rel="icon" href="https://errors.adobeaemcloud.com/favicon.ico">

                  AEM Cloud Service.

The information in the log is quite clear but there's no visual hint at the failure at first. It is only once we've tried to access the environment that we realised it had been hibernated automatically.
.
This is a trivial case in case of hibernation but it could behave in similar ways in case of unexpected problems with the environment and, depending on the development flow, some time-consuming and/or expensive tests could be started in a situation where the failure could be immediately obvious earlier. I think a fail-fast strategy would be a nice improvement here.

The beta flow is no superseded by changes in https://github.com/adobe/aem-site-template-standard/pull/48
When the Quick Site Creation flow will be generally available, the build and deploy pipeline will be through Cloud Manager.


lv_port_pc_eclipse V9 build failure

function 'Tvg_Result tvg_engine_init(Tvg_Engine, unsigned int)' definition is marked dllimportGCC
'Tvg_Result tvg_engine_init(Tvg_Engine, unsigned int)' redeclared without dllimport attribute: previous dllimport ignored [-Wattributes]GCC
Tvg_Result tvg_engine_init(Tvg_Engine engine_method, unsigned int threads)
Engine API
Initializes TVG engines. TVG requires the running-engine environment. TVG runs its own task-scheduler for parallelizing rendering tasks efficiently. You can indicate the number of threads, the count of which is designated threads. In the initialization step, TVG will generate/spawn the threads as set by threads count. tvg_engine_init(TVG_ENGINE_SW, 0); //Initialize software renderer and use the main thread only

Parameters:
engine_method – The engine types to initialize. This is relative to the Canvas types, in which it will be used. For multiple backends bitwise operation is allowed. - TVG_ENGINE_SW: CPU rasterizer - TVG_ENGINE_GL: OpenGL rasterizer (not supported yet)
threads – The number of additional threads used to perform rendering. Zero indicates only the main thread is to be used.

Returns:
Tvg_Result enumeration.

Return values:
TVG_RESULT_SUCCESS Succeed.
TVG_RESULT_FAILED_ALLOCATION An internal error possibly with memory allocation.
TVG_RESULT_INVALID_ARGUMENT Unknown engine type.
TVG_RESULT_NOT_SUPPORTED Unsupported engine type.
TVG_RESULT_UNKNOWN Other error.

Note:
The Initializer keeps track of the number of times it was called. Threads count is fixed at the first init() call.

I am using the version in the folder，

Low version (requires lv_drivers folder) can successfully build

Please tell me what I should do?

I also encountered this problem

Does it help if you add #define TVG_BUILD 1 here?

I solved this problem by changing mingw64 to cygwin, and then I compiled and installed SDL2, but I couldn't use debug because gcc under win is missing ASAN related libraries

I solved this problem by changing mingw64 to cygwin, and then I compiled and installed SDL2, but I couldn't use debug because gcc under win is missing ASAN related libraries

Feel free to disable ASAN. It's just optional.

Thank you！

I had the same problem, what's the meaning of changing mingw64 to cygwin @wxlinus .   I worded in windows platform and use mingw and vscode， I am not get used to use codeblock even thoug it's recommended in windows platfrom

Should we ask questions in this project?

Should we ask questions in this project?

Oh yes, as it's VS code please open a new issue in https://github.com/lvgl/lv_port_pc_vscode

Adding a sentence with a red line may solve your problem


Support grpc for transformer

What this PR does / why we need it:
Make gRPC predictor call work out of the box in the transformer.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1796
Special notes for your reviewer:
apiVersion: serving.kserve.io/v1beta1
kind: InferenceService
metadata:
  name: torch-transformer
spec:
  predictor:
    triton:
      storageUri: gs://kfserving-examples/models/torchscript
      runtimeVersion: 20.10-py3
      env:
      - name: OMP_NUM_THREADS
        value: "1"
      ports:
      - name: h2c
        protocol: TCP
        containerPort: 9000
      args:
      - --log-verbose=1
  transformer:
    containers:
    - image: yuzisun/grpc-image-transformer:latest
      name: kfserving-container
      command:
      - "python"
      - "-m"
      - "model"
      args:
      - --model_name
      - cifar10
      - --protocol
      - grpc-v2

Release note:

/ok-to-test

@pvaneck Can you help take a look?

/lgtm

/approve


[Bug Report] The check box can't be checked when all of it's child nodes has disabled

Element UI version
1.4.2
OS/Browsers version
win7 / chrome 57
Vue version
2.4.2
Reproduction Link
https://jsfiddle.net/PaDaker/7gwh2eyd/
Steps to reproduce
Click the check box of node 3-1-1 after disable all child nodes of node 3-1-1.
What is Expected?
When check the node '3-1-1',the check box will be checked.
What is actually happening?
When check the node '3-1-1',the check box won't be checked.

This is intended. See #4767


Bundled packages don't match required versions

Installing<EMAIL_ADDRESS>yields the following warning:
npm WARN<EMAIL_ADDRESS>had bundled packages that do not match the required version(s). They have been replaced with non-bundled versions.

This is was introduced in c5aec41a2743c6f4918be29e2120f155c219340a, which bumped the version in package.json without also updating package-lock.json. Therefore, the version that was bundled (according to package-lock.json) does not match the dependency declared in package.json.
This can be fixed by regenerating package-lock.json.

Should be resolved in v1.2.2


nodes endpoint: search against a node and report n_metapaths

WIP

As of https://github.com/greenelab/hetmech-backend/pull/51/commits/eb045c88ca74baf6aa8ea8e2699862d86f455141, the following output is returned for http://localhost:8000/v1/nodes/?count_metapaths_to=5:
        {
            "id": 4,
            "metapath_count": 1,
            "identifier": "GO:0045833",
            "identifier_type": "str",
            "name": "negative regulation of lipid metabolic process",
            "properties": {
                "url": "http://purl.obolibrary.org/obo/GO_0045833",
                "source": "Gene Ontology",
                "license": "CC BY 4.0"
            },
            "metanode": "Biological Process"
        },

When count_metapaths_to is not specified, then metapath_count is blank. I would prefer if it were removed entirely in that case and will discuss with @dongbohu. Thanks @dongbohu for the help to get thus far.

@dhimmel, if you change NodeSerializer class into:
class NodeSerializer(serializers.ModelSerializer):
    def __init__(self, *args, **kwargs):
        super(NodeSerializer, self).__init__(*args, **kwargs)
        request = self.context.get("request")
        if request and request.query_params.get('count_metapaths_to'):
            self.fields.append('metapath_count')

    class Meta:
        model = Node
        exclude = 'metapath_count'
... ...

See whether metapath_count field will disappear when count_metapaths_to is not specified.

We also need to look into sorting the results by metapath_count and potentially other factors.

@dhimmel, here is a revised NodeSerializer to give error response when count_metapaths_to is not a number:
class NodeViewSet(ReadOnlyModelViewSet):
    """
    Return nodes in the network that match the search term (sometimes partially).
    Use `count_metapaths_to=node_id` to return non-null values for metapath_counts.
    """
    http_method_names = ['get']
    serializer_class = NodeSerializer
    filter_backends = (filters.SearchFilter, )
    # See the following page for "search" implementation and other filter options:
    # https://www.django-rest-framework.org/api-guide/filtering/
    search_fields = ('identifier', 'metanode__identifier', 'name')

    def get_serializer_context(self):
        """
        Add metapath_counts to context if "count_metapaths_to" was specified.
        https://stackoverflow.com/a/52859696/4651668
        """
        context = super().get_serializer_context()
        search_against = context['request'].query_params.get('count_metapaths_to')
        if search_against is None:
            return context
        try:
            search_against = int(search_against)
        except ValueError:
            return context

        from dj_hetmech_app.utils.paths import get_metapath_counts_for_node
        context['metapath_counts'] = get_metapath_counts_for_node(search_against)
        return context

    def get_queryset(self):
        """Optionally restricts the returned nodes to a given list of
        metanode abbreviations by filtering against a comma-separated
        `metanodes` query parameter in the URL.
        """

        queryset = Node.objects.all()
        if 'count_metapaths_to' in self.request.query_params:
            if not self.request.query_params.get('count_metapaths_to').isdigit():
                return None

        metanodes_str = self.request.query_params.get('metanodes', None)
        if metanodes_str is not None:`
            metanodes = metanodes_str.split(',')
            queryset = queryset.filter(metanode__abbreviation__in=metanodes)

        return queryset

    def list(self, request):
        queryset = self.get_queryset()
        if queryset is None:
            return Response(
                {'error': 'count_metapaths_to is not a valid number'},
                status=status.HTTP_400_BAD_REQUEST
            )

        data = NodeSerializer(get_queryset, many=True).data
        context = {
            "data" : data,
            "message" : "Contents returned successfully",
            "success" : True
        }
        return Response(context, status=status.HTTP_200_OK)

Try it and see whether this is what you want.

About sorting the results, why not let the frontend handle it? It will be just a few lines of extra code.

The method in https://github.com/greenelab/hetmech-backend/pull/51#issuecomment-493149816 destroys pagination causing a massive JSON response for queries like http://localhost:8000/v1/nodes/?count_metapaths_to=34000.

About sorting the results, why not let the frontend handle it?

Because the frontend receives only a small subset of all the matches when there are many matches due to pagination. Sorting therefore should happen on backend.

@dongbohu this is now ready for review.
http://localhost:8000/v1/count-metapaths-to/999/ returns results like:
{
    "query-node": 999,
    "count": 2841,
    "max-nodes": 50,
    "results": [
        {
            "id": 38340,
            "identifier": "DB01234",
            "identifier_type": "str",
            "name": "Dexamethasone",
            "properties": {
                "url": "http://www.drugbank.ca/drugs/DB01234",
                "inchi": "InChI=1S/C22H29FO5/c1-12-8-16-15-5-4-13-9-14(25)6-7-19(13,2)21(15,23)17(26)10-20(16,3)22(12,28)18(27)11-24/h6-7,9,12,15-17,24,26,28H,4-5,8,10-11H2,1-3H3/t12-,15+,16+,17+,19+,20+,21+,22+/m1/s1",
                "source": "DrugBank",
                "license": "CC BY-NC 4.0",
                "inchikey": "InChIKey=UREBDLICKHMUKA-CXSFZGCWSA-N"
            },
            "metanode": "Compound",
            "metapath_count": 8
        },
        {
            "id": 43519,
            "identifier": "DB01047",
            "identifier_type": "str",
            "name": "Fluocinonide",
            "properties": {
                "url": "http://www.drugbank.ca/drugs/DB01047",
                "inchi": "InChI=1S/C26H32F2O7/c1-13(29)33-12-20(32)26-21(34-22(2,3)35-26)10-15-16-9-18(27)17-8-14(30)6-7-23(17,4)25(16,28)19(31)11-24(15,26)5/h6-8,15-16,18-19,21,31H,9-12H2,1-5H3/t15-,16-,18-,19-,21+,23-,24-,25-,26+/m0/s1",
                "source": "DrugBank",
                "license": "CC BY-NC 4.0",
                "inchikey": "InChIKey=WJOHZNCJWYWUJD-IUGZLZTKSA-N"
            },
            "metanode": "Compound",
            "metapath_count": 8
        },


'double_initialize' test is failing

failures:

---- double_initialize stdout ----
	thread 'double_initialize' panicked at 'init failed when run a second time: InitializedProject { location: "/var/folders/1w/7c6mj3rn59nbknpt834zrqd00000gp/T/create_root_readme.PWdYSBG9wHGj" }

Seems like these commits contradict each other:

https://github.com/steveklabnik/doxidize/commit/66444541a11a32d916c213bcd75e5f5cab8379a9
https://github.com/steveklabnik/doxidize/commit/fe23012f965e566b72e6a9427f52f7220580b0ce

Maybe that test should assert that an error is returned the second time init is called? What's the expected behavior here?

test commented out in https://github.com/steveklabnik/doxidize/pull/89


Update PPP

Codecov Report

Merging #256 (105caf9) into master (06b30db) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #256   +/-   ##
=========================================
  Coverage     71.13%   71.13%
  Complexity      927      927
=========================================
  Files           145      145
  Lines          3066     3066
  Branches        367      367
=========================================
  Hits           2181     2181
  Misses          769      769
  Partials        116      116

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Custom Vertex Buffer Serialization

Closes https://github.com/BabylonJS/ThePirateCove/issues/547
With these changes, when serializing a mesh/scene, there are two new top level objects, buffers and vertexBuffers. Both are mappings from id (which was added to both as GUID) to serialized objects. On the geometry itself, instead of saving the buffer data, it saves the ID of the corresponding vertex buffer. This allows shared Buffers and VertexBuffers to be serialized/parsed only once.
Playground ID: #KSBUDT#12

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14749/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14749/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14749/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14749/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14749/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14749/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14749/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14749/merge#BCU1XR#0

I would vote for the latter as well as this is not critical (not blocking anybody for now).
Let s only address it once.

EDIT: Can a Buffer be shared between geometries? Because if so, then we have to serialize them in a level above the geometries...

Indeed, that's what I was about to comment... In glTF, for eg, it's possible to have big vertex buffers with mesh data (positions, normals, etc) pointing all to the same buffer but with different offsets.
So, the Geometry should only serialize a pointer to a buffer, and we should add a proper serializing / parsing method on the Buffer class.

Visualization tests for webgl1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14749/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl2 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14749/merge/testResults/webgl2/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl2 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14749/merge/testResults/webgl2/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14749/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Experimental playwright test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14749/merge/testResults/webgl2playwright/index.html

@carolhmj will you finish this one or should we park it for later ?

@sebavan I can take a look at it whenever there is time 🙂

closing for now, let s reopen when we ll have more time


Tree shaking with ramda or lodash-es

with fuse.js
plugins: [
  UglifyJSPlugin({
    mangle: { toplevel: true },
    compress: { side_effects: false, pure_getters: true }
  }),
  QuantumPlugin({
    treeshake: true,
    uglify: true
   })
]

// other config is copied from examples/quantum-tree-shaking.

Compare the two samples below.
import { pathOr } from 'ramda'
console.log(pathOr);

app.js around 90+kb
import pathOr from 'ramda/es/pathOr'
console.log(pathOr);

app.js around 7.x kb
Comparing two output files without uglify
  // ramda/es/index.js
  $fsx.f[2] = function(module,exports){
  Object.defineProperty(exports, '__esModule', { value: true });
  var F_1 = $fsx.r(3);
  exports.F = F_1.default;
  var T_1 = $fsx.r(7);
  exports.T = T_1.default;
  var __1 = $fsx.r(8);
  exports.__ = __1.default;
  var add_1 = $fsx.r(9);
  exports.add = add_1.default;
  // ... many other exports

  // ramda/es/index.js
  $fsx.f[2] = function(module,exports){
  Object.defineProperty(exports, '__esModule', { value: true });
  var pathOr_1 = $fsx.r(249);
  exports.pathOr = pathOr_1.default;
  }
  // other files

It seems that shaking happens and uglify-js didn't remove the code which should be.
I pass some testing option to uglify-js through UglifyJSPlugin of fuse-box, but nothing changes..
lodash-es has almost same output. Any config I have missed?

@LucasIcarus thanks for bringing this up. It's all my list to fix it. Treeshaking doesn't work with default exports ATM. I made this limitation due to the abuse of default usage back in the days.
Please be patient ;-)

@nchanged  Thanks for the quick reply.
Looking forward to the release of the new version.


Salary mode

I think we should add new mode (switch maybe) "Salary mode". The difference would be showing how much you lose/gain based on an hourly rate from config (new field) and diff of time, in place of time diff.

Not in the current milestone. Negatively motivates to work for money, not for the goal


Socket service is not connected if keep iOS app in background for a while

Hello. When we test our app with chip-tool on darwin, if we keep app on backgroud for a while then make it to the front, the socket service can never connected again, and matter framework will loop log print "Socket is not connected", see as the picture. This issue will make us can not pair any matter device unless we restart the app.
So I want to know how to fix this?

We use iPhone12 running on iOS 15.4

This is a known limitation of the current iOS CHIPTool.  See also https://github.com/project-chip/connectedhomeip/issues/584

您好。你的邮件我已经收到，谢谢！

And right now the fix is "restart the app".

Now this issue will call iOS App OOM crash,  how can I close socket service when the app on backgroup?

Darwin Issue Review: Not critical for 1.0, moving to 1.1

Dup of: https://github.com/project-chip/connectedhomeip/issues/584


Image sprite rotation leaves does not fill enclosing rectangle on Android 5.1.1

See thread with attached aia file
https://groups.google.com/forum/#!category-topic/mitappinventortest/specific-programming-issue/EDMn85XAlg0

Thanks @halatmit, I will look on it.

Adding  java rotatedDrawable.setAlpha(254) just before java rotatedDrawable.draw(canvas) around line 140 of the ImageSprite.java component seems to "fix the bug".
I don't really understand why.
There must be a nicer solution.

@jisqyv
Momoumar,
That sounds like the right fix, but I don't see the pull request in our public appinventor/sources repo.
I don't understand  why it's not there, since it's apparently in your fork.

pull request #403

handled in pull reuest #408


Update mayors (SC, NY, IA, MI, PA, MA)

I'm helping out resistbot by loading some people updates.
There's both a retirement (David-J-Narkewicz-1602c8e0-d2d4-48ed-9c49-5e9c09cad12d.yml and an retirement (Matthew-Wismer-d0118143-7e94-438b-9f44-499f61797fdd.yml) in this batch; hopefully I handled them correctly.

whoops, I was going by the schema.md which doesn't currently reflect the OSEP #6 changes from contact_details to offices, fixed now


Modifying Methods of ActivityLogger.php without touching it's code

Is it possible to set up my own class that extends ActivityLogger and use it?
I want to add a few things (for example logging IP address in separate database column), but don't want to change the code of the package, so I can still update it.

Currently there's no way of customizing the ActivityLogger itself.
You have two options to add the IP address to a logged item.

You can leverage custom properties.

You could create an observer on the Activity model and let it add the IP address to the row whenever an Activity model is created.


python3Packages.ducc0: 0.32.0 -> 0.33.0

Description of changes
Version update from 0.32.0 to 0.33.0

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 281105 run on x86_64-linux 1

4 packages built:

python311Packages.ducc0
python311Packages.ducc0.dist
python312Packages.ducc0
python312Packages.ducc0.dist


fix: #1512

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[x] Yes. Issue #1512
[ ] No

The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
I've made my tests by running and publishing to a local Verdaccio.
How the package looks like in node_modules:

An extract of my code (made with an old Vue CLI and Vue 2):

Hovering the invoke function displays the types as expected:

Result browser-side:

Question: is there ever a reason for us to ship the TS?

Question: is there ever a reason for us to ship the TS?

Not really, that's why we only have type definitions. :)

But lets merge and release


Certain games not showing up.

Just realized "Killer 7" does not show up in my library even though its named correctly and placed correctly.
The only game I have noticed so far. Not sure what could be the reason

Just set this up and despite having named them all correctly, none show up. The Wii part worked flawelessly, so not sure what is wrong.

I was in the same boat a few days ago. There must be a game that is incorrectly named. Be careful of spaces in between names. Example: "SoulCalibur" instead of "Soul Calibur"  etc

Thanks for your reply, I rechecked everything. Didn't change anything, but they loaded just fine, except The Legend of Zelda Ocarina of Time (Bonus Disc). Maybe it will show up later, I don't know. But the rest is working now :)

Has there been any progress with fixing certain ISOs not showing up? I have made a copy of the GamesList.txt with only working ISOs in it and have tried to add to it paying close attention to spelling. Despite this about 1/3 of my games still aren't showing up. Any advice?

Do you have the best match mode turned on? I'm really not sure about what's
causing this.
On Tue, Apr 28, 2020 at 2:33 AM Cryator<EMAIL_ADDRESS>wrote:

Has there been any progress with fixing certain ISOs not showing up? I
have made a copy of the GamesList.txt with only working ISOs in it and have
tried to add to it paying close attention to spelling. Despite this about
1/3 of my games still aren't showing up. Any advice?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JTNDev/galaxy-integration-gc/issues/9#issuecomment-620409282,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ5LEJ3B673RYPZYKB5V2CTROZ2DDANCNFSM4J2WMT4Q
.


Bug in tokenizer for Tibetian Language

At the moment you cant use tibetian language tokenizer. It gives the error message:
TypeError: "module" object is not callable
The error is thrown here in sentence_split.py:
    elif split_algo == "bodnlp":
        logger.info(f" - Tibetan NLTK sentence splitter applied to '{lang}'")
        from botok.tokenizers import sentencetokenizer as bod_sent_tok

We have released the new version 2.1.0 last week, could you check again ?


Cannot install

Current Behavior:
There are these errors after running npm i :
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/lerna
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/cz-lerna-changelog
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Steps To Reproduce:

Download the code via github
Run npm i

Environment:

OS: Windows 10 Enterprise v1909
Node: 16.14.2
npm: 8.6.0

This issue is resolved here https://github.com/DhiWise/dhiwise-nodejs/issues/10. check it out.


400 Using login endpoint

I followed the request structure as specified to send a text with the venmo otp, however when sending a request to
POST https://api.venmo.com/v1/oauth/access_token?client_id=1
I see:
{
    "error": {
        "message": "Your code has expired. Sign in again and we'll send you a new code.",
        "code": 81110,
        "links": null,
        "title": "Error"
    }
}

Is there an error in the documentation?

The authentication process should happen quickly, otherwise the otp-sercret or the otp will expire. Here either your otp has expired or otp-sercret

The issue was a problem in postman. The request returns 200 now.


in raw_decode - raise JSONDecodeError("Expecting value", s, err.value) from None

Describe the bug
When inserting a {} or a dict(), the row gets inserted but error is thrown
in execute     return APIResponse.from_http_request_response(r)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File ".../Personal/Coding/python1/.venv/lib/python3.11/site-packages/postgrest/base_request_builder.py", line 154, in from_http_request_response     data = request_response.json()            ^^^^^^^^^^^^^^^^^^^^^^^   File "../Personal/Coding/python1/.venv/lib/python3.11/site-packages/httpx/_models.py", line 756, in json     return jsonlib.loads(self.text, **kwargs)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File<EMAIL_ADDRESS>line 346, in loads     return _default_decoder.decode(s)            ^^^^^^^^^^^^^^^^^^^^^^^^^^   File<EMAIL_ADDRESS>line 337, in decode     obj, end = self.raw_decode(s, idx=_w(s, 0).end())                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File<EMAIL_ADDRESS>line 355, in raw_decode     raise JSONDecodeError("Expecting value", s, err.value) from None json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
To Reproduce
Python 3.11
Any table, any dictionary, just insert.

Came here for the same error, didn't noticed it gets inserted in the database, for the moment switched to http requests for inserting

Same issue here

Hey team,
This looks like a postgrest-py issue, aware of the issue and will look into it once a slot frees up. Most probably tomorrow evening

If you need any help, more details let me know :) Happy to help

Will do, thanks!

P.S. I had similar issues when doing a select query so for now I have completely ditched the supabase client and switched to psycopg3 (which is extremely slow) and I think I will move to asyncpg (which claims to be 3x faster than psycopg)

FYI this breaks the examples on the supabase.py Readme


[WEBPLAT-641] Update to v4.0.3

This pulls in the changes from Create React App 4.0.3.
Notable changes

It includes the change from https://github.com/facebook/create-react-app/pull/10170, which introduces a DISABLE_ESLINT_PLUGIN environment variable. That makes one of our customizations (disabling the ESLint plugin from https://github.com/EverlongProject/create-react-app/pull/29/files#r527210700) unnecessary.

I dropped the customization when resolving merge conflicts, and intend to switch consumers to use DISABLE_ESLINT_PLUGIN when updating them to this version.

It also pulls in https://github.com/facebook/create-react-app/pull/8986, which introduces a BUILD_PATH environment variable that can be used to customize the build output location. This could be helpful for simplifying the league-web CI config (by removing the need to manually rename the build output folder). I will followup on that separately in league-web.
updates various dependencies which might help with dependency vulnerability analyses

~I didn't see where the eslint was reenabled though.~ Never mind I found it. Looks good.

Yeah it was done as part of the merge commit, which in retrospect was a bad idea!


Ajout de champ verrouiller par Wikidata

Description
Bonjour,
J'aimerai savoir si vous pouvez mettre aussi la "population", le "postal_code" en synchronisation avec Wikidata.
Merci d'avance, cordialement.
Screenshots
No response

See #5566

Are you asking whether it’s OK to set population and postal_code based on what you see in Wikidata? There’s some discussion of that on the wiki. The short answer is that Wikidata considers its contents to be in the public domain, as with all individual facts, but many in the OSM community believe the project should apply a stricter standard just in case. In any case, Wikidata got its population and postal code data from somewhere else, so you could check if that other source has more restrictions that make it suitable for inclusion in OSM.
#5566 is just about making the UI a little more informative, but not about adding or changing any tags. Occasionally people have floated the idea of making it easier to copy facts from Wikidata items to OSM via iD, like in #4382, but there doesn’t seem to be much appetite for it.


Find All References does not find references in modules within same workspace

VSCode Version:  1.33.1
OS Version: Windows 10

Steps to Reproduce:

git clone https://github.com/hediet/typescript-find-references-in-workspace
Open lib/src/index.ts, find all references to helloWorld
Open client/src/index.ts permanently
Open lib/src/index.ts, find all references to helloWorld

Does this issue occur when all extensions are disabled?: Yes
Expected outcome:
The references found after step 2 and 4 are the same, they both find the reference in client/src/index.ts.
Actual outcome:
Vscode only finds the reference in client if any file of the client projected is opened. This is very unsatisfying when renaming elements.
Basically, #48474 is not fixed. I don't know whether this is a TypeScript or VSCode isssue.
It also happens if references are specified in tsconfig.json, either from lib to client or vice versa.

Oops. The duplicate detection actually was helpful this time.


Usage with GitHub-enterprise GitHub instance

Hello,
In order to use the provider with a GitHub-enterprise instance it's required to set base_url property, but I don't retrieve the properties into to "providerconfigs.github.upbound.io" CRD.
Could you add an example to explain how to make it ?

I can confirm that adding the base_url property into the credentials JSON object works.

@sebastienbrunkyndryl  i am going to close it, but feel free to reopen if this issue is not resolved.


Don't run in production env

Great package 👏👏👏
Is there a way to run schematics on during development, when config('app.env') != 'production'?

Thanks @mtvbrianking,
I think you could do this in a couple of ways:
Only require it in development: composer require mtolhuys/laravel-schematics --dev
You could alternatively provide a Middleware to the route through your config/schematics.php file.
    'middleware' => ['dev-env-or-something-similar'],

And then handle the check, block and/or redirect through the middleware.

I had required in --dev but I could still access the routes in production, I am just worried about users stumbling across the database schematics.
Let me write up a middleware to validate and restrict access based on the app environment.

The middleware option works just fine. I am leaving the steps here it could help someone else.
Create middleware
<?php

namespace App\Http\Middleware;

use Closure;

class NotInProduction
{
    public function handle($request, Closure $next)
    {
        if (config('app.env') == 'production') {
            abort(403, "Don't run in production.");
        }

        return $next($request);
    }
}

Register middleware: app\Http\Kernel.php
<?php

namespace App\Http;

use Illuminate\Foundation\Http\Kernel as HttpKernel;

class Kernel extends HttpKernel
{
    // ...

    protected $routeMiddleware = [
        // ...

        'dev' => \App\Http\Middleware\NotInProduction::class,
    ];
}

Protect routes; config/schematics
<?php

return [
    // ...

    'middleware' => 'dev',
];

You may need to clear the cache for the change to be effected.
php artisan config:clear

Thanks for sharing @mtvbrianking !


Stop using /dev/null for dummy IO (removing all unsafes)

This PR replaces /dev/null with io::empty() and io::sink() so that unsafe is no longer needed.

Thanks a million.
Contributions from people who are familiar with Rust are quite helpful :)


Hunger meter

Walking decreases his hunger meter by 1. When 0 dies and respawns. Eating replenishes the meter.
Need to refactor code to allow for death / rebirth.

This is done.


karma-coverage

Is it possible to use karma-systemjs in combination with karma-coverage? Is there an example of a working configuration?

I've never tried it before, though I don't see any reason why it shouldn't work.
The preprocessor should instrument the code before SystemJS loads it, and then it just needs to hook into karma.start() to generate the report.
I'll give it a test now and see what I can find.

Got it working: https://github.com/rolaveric/angular-seed/commit/cb9337145d8a4873ddc9ae0486111b1b6e9427df
The trick was getting karma-coverage to use a different instrumenter that supports ES6.
Thankfully I found this commit: https://github.com/douglasduteil/karma-coverage/commit/626e7b0c5a36fba948f6be349020e62794a79377
However, the helpful README information that tells you how to use a different instrumenter is now missing from the master README file - that's what made this situation more confusing than it had to be.
I'm going to add a note to karma-systemjs's README about how to use it with karma-coverage.


Remove the 404 page and use the Redirects feature instead

Is your feature request related to a problem? Please describe.
dogehouse/kibbeh/src/pages/404.tsx is redundant since we don't have a 404 page
it is a simple redirect. We should instead use the Redirects feature built into the Next by modifying
next.config.js This should significantly speed up redirects and send a 301 redirect response code to browsers.
module.exports = {
  ...,
  async redirects() {
    return [
      {
        source: '/404',
        destination: '/',
        permanent: true,
      },
    ]
  },
}

@overlisted @amitojsingh366

can you do it yourself

can you do it yourself

@overlisted I could do it myself but it will be a while due to my day job.
And I also wanted feedback/validation on this approach to the 404 redirect.

And I also wanted feedback/validation on this approach to the 404 redirect

I'm not a Next.js professional, but it seems reasonable to me

@overlisted Can I take this?

@overlisted Can I take this?

go ahead

@chalkedgoose i tried it and unfortunately it seems like you cannot turn a 404 to a 301. it will have to go through pages/404.tsx and then redirected to /dash

and looking at it again i don't see why we would want to turn a 404 into a 301

and looking at it again i don't see why we would want to turn a 404 into a 301

@amitojsingh366 The reason you'd want to do it is that your essentially utilizing the page as a 301 redirect anyways. There's just no status code and a client side redirect.


HDDS-5878. support chinese key in om

What changes were proposed in this pull request?
support chinese key in om
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5878
How was this patch tested?
ut

@ChenSammi PATL, thanks!


double NaN as default value can massivly expand off heap mutable dictionary

Hi,
It seems that using NaN as a default value for Double in schemas can lead to much more calls to the BaseOffHeapMutableDictionary.expand that required.
And because a multiplication factor is apply at each call. It can quickly try to allocate a huge amount of off heap memory
I suspect a hazardous conversion of NaN Double to null somewhere that prevent the DoubleOffHeapMutableDictionary.compare to achieve its goal correctly
I wasn't able to get much more details, but maybe be it will be enough to ring a bell to someone knowing this part of Pinot.

Good catch. The problem is from DoubleOffHeapMutableDictionary.equalsValueAt() which will always return false when input value is NaN.
We can fix it by comparing the Double.doubleToLongBits() (bytes level same), but I'm not sure if allowing NaN can cause other problems (it might break the binary search as well). Another option is to prevent NaN to be ingested (not allowing it as default value, and also replace it with default value when seeing it in the input data)

Maybe a quick fix would be to disable NaN.
But from my standpoint, to be able to use NaN would be a better solution as it is not semantically equivalent to Infinities, null or zero. It is a very common value used in mathematical and finance domains.
The only other place where we have issue with NaN so far is in the where filters. We have to do something like : cast(num_col as STRING) = 'NaN'.
num_col = 'NaN', num_col != 'NaN' run, but doesn't filter the records correctly

I think this bug can be very dangerous. Maybe waiting for a good NaN management, we could reject schemas in case on NaN default...?

I feel we should just reject NaN for now. Dictionary binary search might also fail.
To reject it, we can add a check in FieldSpec.getDefaultNullValue(), and in the record reader, replace NaN with the default value. @jvenant Do you want to help contribute this?
cc @snleee

Not sure the PR actually fix the issue. It's more a protection than a fix

It is auto-closed by merging #11661. Reopened it since we haven't addressed the issue completely

MySQL: not allow NaN
Postgres: allow NaN
official documentation states, In order to allow numeric values to be sorted and used in tree-based indexes, PostgreSQL treats NaN values as equal, and greater than all non-NaN values.
Some similar issue with -0.0 and 0.0 (This would also mess up the binary search/sorting)
https://jonisalonen.com/2013/positive-and-negative-zeros-and-mysql/

To simplify the handling, as a short term solution, I'd propose:

Not allowing NaN and replace it with default null value when ingested
Replacing -0.0 with 0.0

We may consider following PostgreSQL behavior as long term solution

@Jackie-Jiang sounds good to me.

@Jackie-Jiang since now also the second bugfix is merged: can this issue be closed or do we keep it open till PostgreSQL behavior is implemented?


JBPM-5325 Replaced Java comments with SQL comments in .sql files.

Replaced Java comments with SQL comments.
Master PR: https://github.com/droolsjbpm/jbpm/pull/599

+1


feat(Storage): Enable Autoclass V2

See go/ac2-1-sdk-request for more details.

The feature has been allow listed only for particular testing projects in each language as can be seen here.
I'll look into this matter as to how we are supposed to proceed with the release. Till then I'll mark this as do not merge

GA date is Nov 3rd, and we plan to release libraries in the week of. I'll signal the go ahead once we're ready to release. Thanks for working on this!


annotate full proto name

feat: add full proto name to generated structs
doc: document new behavior
doc: fix
Update README.md

didn't realise I was pushing to the other PR as well 😅


Docker image not found on private registry

Hello guys,
I've just setted my private docker registry https://docker-registry.impero.me/v2/.
Manually from docker it works fine, but when I use it in the Empire stack config, and I try to deploy an image, for example
bin $ ./emp deploy docker-registry.impero.me/impero/nginx

I get
Pulling repository impero/nginx
error: Error: image impero/nginx:latest not found

Can  someone help me out? Also I don't understand how to setup the login with GitHub. But this is another topic so I may create a dedicated issue.

Hey @JacopoDaeli - this appears to be a duplicate of: https://github.com/remind101/empire/issues/624
This is definitely something I hope we'll be able to fix soon.


Update README, env file and docstring

Updated README, environment file, and docstring

Looks good to me!


Can't click on "apply now" in https://storj.io/careers.html#job-1

Clicking on it does nothing.

solved


404 Errors with Native Federation

Hello,
I'm trying to implement Native Federation in my Angular project, but I'm encountering the following errors:

These errors occur when attempting to load dynamic modules using es-module-shims.
Expected Behavior
The modules should load correctly, and the project should run without issues.
Actual Behavior
The above modules cannot be loaded, resulting in 404 errors.
Steps I Have Taken
Webpack Configuration: I checked the webpack.config.js file to ensure the output settings are correct.
Module Availability: I verified that the relevant modules are correctly defined in the remoteEntry.js file.
Path Check: I inspected the requested URLs in the browser developer tools' network tab.
Updates: I updated Angular and related packages.
Clearing Cache: I cleaned the project and rebuilt it.

My guess is that you have "@angular-architects/native-federation" in your dependencies, while you have shareAll in federation.config.js, so it cannot build shared modules, because "angular-architects/native-federation" depends on nodejs modules.

@CavidH, it is really necessary for us that you provide a reproducible example. You can try out the advice from @thefirst24, but if it doesn't work, we need that Stackblitz example.

https://stackblitz.com/edit/stackblitz-starters-jrwi3t?file=src%2Fbootstrap.ts

@CavidH, it is really necessary for us that you provide a reproducible example. You can try out the advice from @thefirst24, but if it doesn't work, we need that Stackblitz example.

https://stackblitz.com/edit/stackblitz-starters-jrwi3t?file=src%2Fbootstrap.ts

Is thre any errors in build process?
My guess is that you have "@angular-architects/native-federation" in your dependencies, while you have shareAll in federation.config.js, so it cannot build shared modules, because "angular-architects/native-federation" depends on nodejs modules.
Try put "angular-architects/native-federation" into devDependencies section in package.json

I tried it, but it didn't work.

@CavidH , I've managed to exclude some libs which are not needed. I've also removed the es-module-shims from polyfills.ts.
If it is required we might have to install as an explicit dependency.
The app is starting but failing because of other reasons (hope they are not related to the missing esm shim...)
Please take a look: https://stackblitz.com/edit/stackblitz-starters-rb9dwf?file=src%2Fbootstrap.ts

@CavidH , I've managed to exclude some libs which are not needed.
The app is starting but we are still get an error because of the echarts library.
Please take a look: https://stackblitz.com/edit/stackblitz-starters-rb9dwf?file=src%2Fbootstrap.ts

After the changes you made locally, I tried to run the project again, but it still didn't work. Even if this project succeeds, I'm hesitant because it could jeopardize the project's sustainability and continuity in the future

I continued to play witht it as well. What I realized is that a lot of libraries should go to devDependencies. Otherwise you have a nodejs library which is not ready for a browser and esbuild fails because of that.
I do understand your concerns, but we are using here real web standards and if you clean up the dependencies a little bit, it will work.
By saying that, I am going to close this issue. As always, if you have further questions, feel free to re-open it.

Thank you

@CavidH, that being said we are working on automizing that part. So we want to filter out those libraries which are used for in the app and which ones are not in ESM.


chore(infra): Share some properties across workspace

I noticed version bumping touches many places, this is an attempt to alleviate that a bit.

@dcvz This is a great idea! I guess you should update with main and use 0.12.0 instead of 0.11.0, the flaky tests are supposed to be gone.

@nathanielsimard updated!


this is a test project

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


Potentially unnecessary &mut self on methods.

I noticed that the code compiles fine if mutating methods [MultiMap]Table::{insert,remove} etc are changed to only require &self. Is there a reason they're currently requiring &mut self?

Yes, it would result in race conditions if multiple threads modified the same table at the same time

Yep, makes sense. Could it work if it required &self but marked !Sync? This would make it more ergonomic for single thread use.

No, that would break the iterators returned from methods like range()
Sent from my phone
On Wed, Nov 8, 2023, 5:19 PM Andy Yu @.***> wrote:

Yep, makes sense. Could it work if it required &self but marked !Sync?
This would make it more ergonomic for single thread use.
—
Reply to this email directly, view it on GitHub
https://github.com/cberner/redb/issues/717#issuecomment-1803013197, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGNXQH2OZGJMATZHMRC4LDYDQVQTAVCNFSM6AAAAAA7CROIWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBTGAYTGMJZG4
.
You are receiving this because you commented.Message ID:
@.***>


"Zed Mono Extended" or "Iosevka Extended" don't work

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.90.0
OS Version: Sonoma 14.5

Steps to Reproduce:

Change font to "Zed Mono Extended" or "Iosevka Extended"
See that it falls back to the next font.

Notes:

The issue doesn't replicate in other text editors like Sublime.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.90.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Looks like duplicate of #214390 see https://github.com/microsoft/vscode/issues/214390#issuecomment-2154822715 and check if it works

Closing as a duplicate. Feel free to comment on or reopen the issue if it still shows up after the 1.90.1.


Discord.exe not found [Bug]

I have tried moving better discord to different places in my folder, but it does not seem to work every time. It just keeps repeatedly keep saying "discord.exe not found". Please help me on this issue..
I download the file. It goes to my downloads folder. I move it to desktop. I click on better discord. I then go to agree then i click the first top button that says download. I also then go to "install to stable" it doesnt let me check it off as it says "discord.exe not found".
Here, I expected to happen was to find "discord.exe" to let me install it. It doesn't let me install.
Stable
Picture:

You can manually browse to your Discord installation using Browse. If the installer does not find it, that means you either installed in an unusual location or your environment variables. For both cases you will need to manually browse to your installation.

And which way to indicate?


What does nightscout_id mean?

I've had nightscout for 8 years now, and it gets hosted on node.js Platform as a Service hosts. I've had george.azurewebsites.com, then george.herokuapps.com, and now george.up.railway.app . I've never come across the term "nightscout_id" tho. What is it referring to? I'm coming up empty on google too https://www.google.com/search?q=Nightscout+ID

Oh, derp. The code itself has a comment explaining it.
XXX in https://XXX.herokuapp.com/api/v1/entries.json
Ok nevermind. Heroku is dropping free support, so I guess this tidbyt won't work on other hosts.
Think it would be hard to get https://george.up.railway.app/api/v1/entries.json to work?

I think I figured out where to put it


Vue & vite prod error with near-api-js

I am trying to preview a built application using latest near-api-js, when previewing, it gives me the below error

Can anyone help :)

Can you please provide some more context on this? Under what conditions did you see this error? Could you maybe add some code snippets?

I'm going to close this issue. Feel free to open again if you are still facing this issue :).


Card scroll

Create scroll effect for cards.

https://webdesign.tutsplus.com/how-to-build-a-simple-carousel-with-vanilla-javascript--cms-41734t


Get Azure login GitHub action working

Goal
Create a new github action that logs into Azure, and prints out something simple to show that it is working.
Context
Getting Azure Github Actions working for this project will significantly improve our ability to automate testing on Azure. This issue is the first step towards that goal. See the Azure/login action.
@jkislin is assisting us getting the new credentials, and into the repo.
Required features

Create a new workflow file that logs into Azure
Add the necessary secrets to this repo's actions secrets

Specifications

[x] Create the new workflow file, ideally as small and simple as possible
[ ] Receive new secrets
[ ] Put new secrets in the repo's actions secrets.

Out of scope

Anything beyond just getting logged in

Related documents

An example of using the Azure login for the VAP WSL
The examples on the Azure/login page.

Ref: https://github.com/cdcent/cfa-multisignal-renewal/issues/60
author: @natemcintosh

@natemcintosh, this article is very relevant for GHA + Azure: https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions#understanding-the-risk-of-script-injections.

This is also key, @natemcintosh: https://msrc.microsoft.com/blog/2023/11/microsoft-guidance-regarding-credentials-leaked-to-github-actions-logs-through-azure-cli/


Add support for performance evaluation via a Wrapper

This change adds a gym wrapper for collecting max rewards and max floors
across a set of seeds provided by the user.  Includes an update to the
README and an example of usage which mimics OTC evaluation.

Oh, whoops -- I had already fixed a couple of these things and not pushed the changes.  Thanks for making the fixes.  I will make the reset parameters change and review.

Rebased and added reset parameters.  PTAL.


Documentation/structure valid JSON

Hi,
Page:
Upload Result; https://<IP_ADDRESS>/upload
Issue description:
I tired to upload my fist file, but I was not sure what exactly to upload, I could not find any information about the structure the JSON should have.
Expected:
Have a help command/link about where I can find the required JSON structure needed for each benchmark selection.
Optional:
After I uploaded, the page returned me an error:

It would be nice to have a tip about where is the mistake

Do we have any kind of specific prescribed structure, or structure you'd like us to use? Currently we're referencing the example provided to us by Valentin by email a few months ago (see https://github.com/EOSC-synergy/eosc-perf/blob/master/app/controller/config/result_template.json).
Currently the JSONValidator checks if the entire json data-tree/all keys match the example json, but a lot of it seems like extra noise. Currently the only used fields are user_args.num_gpus and evaluation.result.average_examples_per_sec in the diagram page, so unless you want us to require anything else, I believe these are the only fields we'd require (e.g. what will be written in the help text).

That is good, I see you have simplified it a lot.

Is it configurable on the code?
Would it be difficult to do so?

In some cases I undestand not all the fields are mandatory, for example a computer might not have any GPU. It would be nice display in this area what are you matchin in the controller to verify if the file is correct:

That way I could see which fields are optional and what is failing in my .json.
I see there is a field "Select benchmark". I suppose this is the type of ".json" that would match.
(Correct me if I am wrong)
Is therefore "Add benchmark" to create new types of accepted ".json" to be comparede later?

I see there is a field "Select benchmark". I suppose this is the type of ".json" that would match.
(Correct me if I am wrong)

Is therefore "Add benchmark" to create new types of accepted ".json" to be comparede later?

Currently there is only one global template used for every submitted result. We could change it so it uses a template that is stored with the benchmark info, then the user would have to submit a template when adding a new benchmark, along with what fields to use for the speedup graph.

In general, yes, there can be different benchmarks with completely different fields. For example, if one tests I/O of a system, it is very different from testing GPU performance. Therefore:

test that uploaded document is JSON (must)
check that in the uploaded document, necessary fields are present -> depends on a particular benchmark.

Now, for very first testing, we may relax requirements to only 1), i.e. that it is JSON.

Next, is 2) : that necessary fields are present. It should not be the exact match!
For instance, example I sent by email, does not pass your current check and rejected with the same "Failed to submit result: no valid result JSON"

OK, maybe it failed because my document did not have "evaluation.average_examples_per_sec" but "training.result.average_examples_per_sec", which is in fact, more important :-)

OK, maybe it failed because my document did not have "evaluation.average_examples_per_sec" but "training.result.average_examples_per_sec", which is in fact, more important :-)

Sorry, when I flew through it for what parameters to use, I just assumed the training section was just input parameters, and the end result was in evaluation, totally missing that there's two seperate average_examples_per_sec. 🤔

2. check that in the uploaded document, necessary fields are present -> depends on a particular benchmark.

We can of course make the template field for new benchmarks optional. Same thing for the speedup fields; however if the field is empty, speedup graph will be unavailable for that kind of benchmark.

Update: no, tried another document with "evaluation.result.average_examples_per_sec", still "Failed to submit result: no valid result JSON"

I do believe the validator checks if the set of keys is exactly the same, not a subset, so that's a bug I'll file seperately.

Since 4f52fdd the uploaded result must not have the exact same keys as the template. The keys in the template must only be a subset of the uploaded result's keys. I'll work on more informative error messages during the next week.

Since 4f52fdd the uploaded result's keys must only be a subset of the template's keys. I'll work on more informative error messages during the next week.

Since 867a5f0 the missing key that made the upload fail is included in the error message.

I am still getting the error:
Failed to submit result: Uploaded Json misses the following (sub-)keys: {'num_gpus'}
Even if the field "" is set.
Is it mandatory to have always the field 'num_gpus'? Or depends on the selected benchmark? I am using "someone/cpu_pi" which in theory should not be GPU but CPU dependent.

The num_gpus key must be a sub-key of 'training' (https://github.com/EOSC-synergy/eosc-perf/blob/master/app/controller/config/result_template.json#L6), I think the full path to the missing key should be displayed if it's missing as to not cause confusion.
Currently, following the template, there must always be a num_gpus field, and this is what the speedup chart uses. Given the diagram only reads out a 'core_count' and 'score', we could, however, add a 'x-field' and a 'y-field' to the benchmark entry to specify what field should be used for the diagrams (if any), which would define the fields to be used for the chart axes.

We should now have an error message that documents which keys are missing. Do we intend to add a visible example of the required keys to the upload page, or do we think the error message is sufficient? @mlbehr

I think for real world usage an error message is sufficient. Normally you would upload a real benchmark result which should contain all required fields. If one is missing, something went wrong, so I think an error message suffices.

That does sound reasonable. I think if you're trying to upload random jsons and fill in random fields you're doing something wrong. Given real results come out of benchmarks however, I think such an example should be given to the code guidelines page, so that developers creating new benchmarks know what to output (#39). I'll mention it in the other issue and close this one for now.


zetacore : btc decode address causes panic

btcutil.DecodeAddress, panics for some combinations of address and params.
Solution: Write a wrapper function to handle the panic

Fixed in https://github.com/zeta-chain/node/pull/1528


netframework issue

[Enter feedback here]

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

ID: fd3a8483-a56e-7f3d-8c25-1d8b18943342
Version Independent ID: 3b8e451b-95b5-3d26-c7ec-160fef01d85e
Content: Troubleshoot installation issues with the App Installer file - MSIX
Content Source: msix-src/app-installer/troubleshoot-appinstaller-issues.md
Product: msix
GitHub Login: @dianmsft
Microsoft Alias: diahar

Please provide additional information of issue you are facing.


[BitwiseCopyable] Infer and check constraint.

When the BitwiseCopyable experimental feature is enabled, infer types to conform to _BitwiseCopyable.  The _BitwiseCopyable inference broadly follows the approach taken to infer Sendable.
(1) Special types are conformed:

function types if trivial
metatypes
builtin types if trivial

(2) TheTupleType is conditionally conformed.
(3) Nominal types are conformed if:

non-public or public+fixed-layout
enum or struct (non-class)
every field conforms to _BitwiseCopyable

Additionally, check that nominal types which are explicitly conformed to _BitwiseCopyable satisfy the latter two conditions of (3).
For a public, non-fixed-layout type to conform to _BitwiseCopyable, the user must conform the type explicitly.
Finally, verify that conformances correspond to TypeLowering's notion of triviality to the appropriate extent:

if a type isn't trivial, it doesn't conform to _BitwiseCopyable unless it's an archetype
if a type is trivial, it conforms to _BitwiseCopyable unless some field in its layout doesn't conform to _BitwiseCopyable, which is only permitted under certain circumstances (the type has generic parameters, the type is public non-fixed-layout, the type is a reference but has ReferenceStorage::Unmanaged)

@swift-ci please test

@swift-ci please test source compatibility

@swift-ci please test

@swift-ci please test source compatibility

@swift-ci please build toolchain

@swift-ci please test

@swift-ci please test

@swift-ci please test source compatibility

@swift-ci please build toolchain

@swift-ci please test macos platform

@swift-ci please test source compatibility

@swift-ci please build toolchain macos platform


Add option to run tests asynchronously

We're already running tests sort of pseudo-asynchronously when time limits are enabled, by running the test function in an asynchronous subprocess and waiting for it to finish. It would be cool to extend that to launch multiple processes simultaneously (preferably with a defined thread limit) and then map the exit codes to tests names and output results on the main thread. I tried this a while ago and failed, so it might not be possible to do that since communication between processes is difficult, but i recently did something similar in PHP using socket pairs, so want to reinvestigate.


automatic publish docs with deployment

Repo should include docs as part of deployment and add as part of gh-pages

Should we also start using sphinx format for doc strings? If so we might want to do a big pass through existing doc strings to migrate.


Add syntax highlighting to docs built from markdown

Github-flavour markdown can use syntax highlighting with a code after three backticks, e.g
```html
<p class="code">syntax highlighting</p>
```

resulting in:

syntax highlighting

We should enable this syntax highlighting in our own built documentation

This has been done for the majority of markdown files.


Notion won't open after customisation --> Following FAQ does not solve problem

describe the bug
Notion won't open after running customiser.py - have followed advice on running cleaner.py and rebooting, and then re-running customiser.py - issue persists. Notion is not running (checked task manager)
expected behaviour
This issue should be resolved by following the FAQ guidance.
screenshots

platform
Manjaro Linux 5.7.15-1 -> AUR notion-app version 1:2.0.7-3.2

Can you try the fixes from https://github.com/dragonwocky/notion-enhancer/issues/41 / https://github.com/dragonwocky/notion-enhancer/issues/20? The linux installer has some faults that should be fixed in the next update.

Hi, so sorry. I hadn't see that fix. I will close this report
#41 worked for me


Restrict allowed characters in library objects name

Restrict the allowed characters set for objects name property to [A-Za-z0-9\-_\.].

Shipped in 2b988d3eaf805c7bb18f764b5d72e6a08f1d3d96.


nao_dcm_robot: 0.0.5-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository nao_dcm_robot to 0.0.5-0:

upstream repository: https://github.com/ros-naoqi/nao_dcm_robot.git
release repository: https://github.com/ros-naoqi/nao_dcm_robot-release.git
distro file: jade/distribution.yaml
bloom version: 0.6.1
previous version for package: 0.0.4-0

nao_dcm_bringup
* adding robot_ip parameter in nao_dcm_H25_bringup_position.launch
* fixing nao_dcm_H25_bringup_position.launch
* Contributors: Natalia Lyubova

Jade is eol


ebrochure fails in website gallery, showing some of the front matter

Fixed. The demo file itself had redundant front matter


Fixes 1:N relation for DV

✅ No public API change.

✅ No public API change.

✅ No public API change.


ImportError: No module named queue

from queue import PriorityQueue

ImportError: No module named queue

python2 does not have queue， I try python3, the problem solved.

@WXB506
i also meet this question.
but i m not using python3.
how to solved it?
thanks.
@jhfjhfj1


Add onClick prop to NotificationStack definition

Hello! :wave:
This template enables you to open an issue that is valuable for you, the project, and the GitHub community. :two_hearts:
Before submitting an issue, please:

Agree the Code of Conduct.
Read the Contributing Guide.

If you are new to open source, check out this 38 minute course on how to contribute to open source on GitHub. It's free! :smile:

Before submitting a pull request, please make sure the following is done:

Add tests for new functionality or bug fixes, following the red/green/refactor method
Ensure all tests pass: npm test
Update and/or add documentation

In order to merge your Pull Request, tests and documentation are required.

This is released as<EMAIL_ADDRESS>Thanks!

Thanks @pburtchaell ! Any chance that you can publish the new version to npm?

Sure, sorry for the delay. I thought it was published... let me check!

Okay @sh3nan1gans, it should be released now. I'm sorry for the confusion there. Enjoy!

@pburtchaell - I'm having issues with installing the latest release. Looks like an issue with the generate-lockfile script. Here's a repro repo - https://github.com/sh3nan1gans/test-install-react-notification

Okay, thanks. I'll take a look.

The changes in PR #145 should fix this because I removed the generate-lockfile script in that PR.

Would you mind installing that branch (PR #145), @sh3nan1gans, and letting me know if you get the error? The branch is rebased from master and includes the commits from this PR.

You can install it with:
npm install git://github.com/pburtchaell/react-notification.git#pburtchaell-parcel

@pburtchaell - That branch worked perfectly 👍

Great! I will get that merged and published. 😄

<EMAIL_ADDRESS>is now deprecated due to that installation error. The latest release is<EMAIL_ADDRESS>which includes the changes from this PR #149 and from PR #145. Enjoy!

Also, thanks for your contribution and your help debugging! 💯

Awesome. Thank you! And thanks for this great library


Allow using sRGB as default color space on load

I have recently encountered an issue when loading a png-file using the function LoadFromWICMemory() in DirectXTex.h / DirectXTexWIC.cpp. It seems that the png-file does not explicitly specify the color format. LoadFromWICMemory() therefore seems to assume a linear RGB format, while all tools I have tested seem to assume sRGB as a default.
It would be very useful to have a flag in order to switch the default format (i.e., behavior when there is no explicit color format given) to sRGB instead of linear RGB.
As a workaround, I added support for the WIC_FLAGS_FORCE_SRGB flag by adding the following code in the function DecodeMetadata(): if (flags & WIC_FLAGS_FORCE_SRGB) sRGB = true;
which might also be useful, but is of course a different behavior since it would ignore an explicitly set linear format.

Every PNG written by WIC has sRGB in it by default, so I'm assuming you are talking about non-Windows tools, correct?
So far I have:

WIC_FLAGS_FORCE_LINEAR which assumes the format is linear no matter what the metadata says
WIC_FLAGS_FORCE_SRGB which assumes the format is sRGB no matter what the metadata says (although this only matters for DXGI formats that have SRGB equivalents)
WIC_FLAGS_IGNORE_SRGB which ignores any metadata in the file, which because DXGI has only a few explicitly sRGB formats defined means it defaults to 'standard' without the _SRGB.

Can you attach an example PNG that lacks any sRGB/gamma information?

I have not created the PNG myself, so I am not sure which tool was used to write it, but it may very well have been a non-Windows tool. I have attached the image file, thanks for looking into it!
To give a little bit more context, loading image files is handled in different ways in our code base. One code path (purely in C++) loads the file "as is" in form of a binary buffer and hands it over to LoadFromWICMemory() (this is the one that "fails" in that it creates a linear RGB texture). We also have a code path used in a different application where the file is first loaded using C#'s Bitmap class, then saved to a buffer that is handed over to the C++ part, again using LoadFromWICMemory() - here, it seems that C# explicitly sets the sRGB information so that the texture is created correctly with the sRGB format.
Regarding the WIC_FLAGS_FORCE_SRGB flag, as far as I have seen this is ignored by LoadFromWICMemory() which is why as a test I added it in DecodeMetaData() (but the preferrable option would be to not force sRGB as we would like to still support linear image data).

See pull request

Works great, thanks for the super quick improvement!


400 bad request exception due to use of blob uri instead of name

[Enter feedback here]
CloudBlobContainer blobContainer = blobClient.GetContainerReference(destBlobURI);
This statement does not work as the GetContainerReference method should contain the name of container. Although , it does not give exception, but when the
blob.UploadFromStream(stream);
is invoked , it raises a 400 bad request exception.
The solution is to use the name of container instead of the full uri.

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

ID: d953c822-ec4e-f50c-569e-3561d76927e0
Version Independent ID: 9771b40c-add6-623f-5793-3742bfac99de
Content: Upload files into a Media Services account using .NET
Content Source: articles/media-services/previous/media-services-dotnet-upload-files.md
Service: media-services
GitHub Login: @Juliako
Microsoft Alias: juliako

#please-close


[WIP] Research jammed adb commands (aka adb jam)

On Detox runs on CI (both OSS and internal projects') we occasionally see that adb commands get stuck "forever", midway through runs of severals suites on the available devices (parallel-execution). When that happens, the running test becomes frozen and eventually fails due to the global test time-out definition. Most often we see that the app install itself is the one that hangs, but that's not consistent.
From some discussions with associated devs at Google, the main culprit here is adb. We've been given by them tools to investigate that, and we need to apply them for the sake of research and hopefully, help them fix whichever issues we find, thus substantially decrease stability issues.

Fortunately, we haven't seen this happening in a long time -- esp. since we switched to more powerful macs on our CI (i.e. since buildkite).


apps: separate the ops and user grafana dashboards

What this PR does / why we need it: it will allow us to control what grafana dashboards from the grafana-ops chart will be visible to the user
Which issue this PR fixes (use the format fixes #<issue number>(, fixes #<issue_number>, ...) to automatically close the issue when PR gets merged): fixes #12
Special notes for reviewer:

only dashboards with "labelKey" values set to 1 will be visible to the user. By default all kube-prometheus-stack dashboards. An ops dashboard can be visible to the user if you set "user_visible" to true in grafana-ops/values.yaml
the grafana-ops chart template was modified to automatically load the dashboards defined in the values.yaml file
we will have one ConfigMap for each dashboard
added the prometheus-sc datasource in user-grafana so they will have access to all metrics.

Before:

After:

Checklist:

[x] Added relevant notes to WIP-CHANGELOG.md
[x] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[x] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

Chart checklist (pick exactly one):

[ ] I upgraded no Chart.
[x] I upgraded a Chart and determined that no migration steps are needed.
[ ] I upgraded a Chart and added migration steps.

Pipeline config (if applicable)
If you change some config options (e.g. add/rename variable or change the default value) you may need to update the config used by the pipeline in pipeline/config.

Looking really good, nice work!
Although would it be possible to set labelValue to user instead of 1? Seems a bit more descriptive.

unfortunately not really possible.
I used "1" because this is the value hardcoded in kube-prometheus-stack dashboards.
This way they will see this dashboards by default.

Looking really good, nice work!
Although would it be possible to set labelValue to user instead of 1? Seems a bit more descriptive.

unfortunately not really possible.
I used "1" because this is the value hardcoded in kube-prometheus-stack dashboards.
This way they will see this dashboards by default.

Right, well then it makes sense.

Looks nice!
I just want to note that with this we are giving the users access to all metrics from the service cluster, since we often (sometimes?) give users editor access in grafana. Then they can use whatever metrics they want.
I'm ok with this, but just want to make sure that others are also ok with it.

We discussed it in an arch meeting and I also highlighted it in the issue #12. If needed I can ask for a confirmation.


withCount(), having() and paginate() together not working

Laravel Version: 5.7.21#.#.#
PHP Version: 7.1.12#.#.#
Database Driver & Version: phpMyAdmin Version : 4.7.4

Description:
I am developing a application in which I have to put condition on count column by using having() function but it's not working. While same query working in mySql with same version.
lluminate\Database\QueryException: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'upvotes_count' in 'having clause' (SQL: select count(*) as aggregate from posthavingupvotes_count = 1) in file D:\xampp\htdocs\hubl\vendor\laravel\framework\src\Illuminate\Database\Connection.php on line 664
Steps To Reproduce:
1) Not Working
$posts = Post::withCount('upvotes')
->having('upvotes_count', '>', 5)
->paginate(6);
2) This will work
$posts = Post::withCount('upvotes')
->having('upvotes_count', '>', 5)
->get();

Yeah the Same issue happens with me.
Please do some needful action.

This is a long-standing issue and not trivial to fix. Have you looked at the various workarounds?

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? If so feel free to reply and we'll try to have a look and re-open this issue.
Thanks!

Same problem. Upgrading to 5.8 did not help

This problem still persist with Laravel Framework 5.8.21

Can we reopen this? Encountering the same bug using 5.8.35

having wont work on aggregated selects properly. If you want to filter you need to use where
Post::query()
    ->withCount([
        'upvotes' => function (Builder $builder) use (&$query) {
            $query = $builder;
        },
    ])
    ->setBindings($query->getBindings(), 'where')
    ->whereRaw("({$query->toSql()}) > ?", 1)
    ->paginate(5);

@dkulyk nice workaround, thanks!

While workaround exists the bug is still not fixed

While workaround exists the bug is still not fixed

Has the issue not been solved yet? using laravel 9.* and the problem still exists.

withCount() works perfectly for me when I use having instead for where.


Initial commit

INFO: api-lint: Using api-lint version: 1.1.4
INFO: api-lint: https://dev.folio.org/guides/api-lint/
INFO: api-lint: Assessing API description files: OAS
INFO: api-lint: Processing OAS 3.0 file: src/main/resources/swagger.api/queryTool.yaml
ERROR: api-lint:
---- Summary:
parsingResult.conforms: false
parsingResult.results.length: 1
Violation: YAML map expected
--------
validationResult.conforms: true
validationResult.results.length: 0
--------

---- parsingResult:
Model: file://./src/main/resources/swagger.api/queryTool.yaml
Conforms: false
Number of results: 1

Level: Violation

- Constraint: http://a.ml/vocabularies/amf/core#syaml-error
  Message: YAML map expected
  Severity: Violation
  Target:
  Property:
  Range: [(8,21)-(8,132)]
  Location: file://./src/main/resources/swagger.api/schemas/entityDataType.json

---- validationResult:
ModelId: file://./src/main/resources/swagger.api/queryTool.yaml
Profile: OAS 3.0
Conforms: true
Number of results: 0

ERROR: api-lint: There were processing errors. See list above.

Note: When those ERRORs are fixed, then any INFO and WARNING messages will be gone too. See Jenkins "Artifacts" tab for ApiLintReport.

@TarasSpashchenko -- You merged this without attending to the errors reported by api-lint. So now every subsequent use will get those errors, and the generated API docs will be broken.


Moment get('day') is not correct.

calling .get('day') on this date returns 2 not 10.

Day gets the day of the week, which is in fact Tuesday (2). Use .date(). http://momentjs.com/docs/#/get-set/date/
In addition, be very cautious of the data that dumps to the console. You are looking at internal properties and they may not be what you expect: http://momentjs.com/guides/#/lib-concepts/internal-properties/

Ty. Writing to console was solely for testing client side. I'm concerned that the current moment object which is of date 2016-10-05 does not provide a property implementation for day. I would think following the C# DateTime object would be preferable. Anyways just a suggestion.

I guess I'm not sure what you mean. .day() will give you day of week, .date() will give you date of the month. This is different than BCL datetime in C#, but it's the same as the built in JavaScript Date object :
new Date().getDay()
2
new Date().getDate()
10

Doesn't it make more sense for a JavaScript library to mirror JavaScript Date than C# DateTime?
If you are saying that you want it to work like moment().date (as a property instead of a function) - we would have to use a getter: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/get
We can't do that until we do a major version release, because we would have to drop support for IE 8.

Ideally an interface where a moment object had properties for year, month, day, hour, etc. and utc, would be preferable (out of the box understanding). Please, remove support for IE 8... IE 11 support as an internal web developer is literally the most frustrating thing I have ever dealt with. Thankfully there is babel. Cheers!


Add user lists to groups

When I add a user to a group, I think it would be more convenient to choose a user from the list of existing users instead of typing an email of each user I want to add.

Added in version 1.2.0


Multitask tutorial

We need a tutorial with a training model in a multi-task dataset.
@rsomani95 We can use the retinanet model you modified in this tutorial, what do you think?

Not relevant anymore.


Raw data being read but dropped

Raw data is being read in actual TDMS files, but dropped from the final output file.  Probably in the handling of ObjDict or not being preserved before emptying of ObjDict.  Passes reference file tests.

Is there a work around for this? I can read in all of the channels and meta data, and have modified a few functions to get it to read the file, but it doesn't seem to store the data as this bug points out. At least not to all of them.


DoorBird Chime

Has anyone else had issues with the DoorBird Chime not working in this setup? Does it need to be setup as a separate HTTP call?

Works fine for me.  There’s really no configuration to add DoorBird Chime into this setup, it hangs off the DoorBird app as an accessory as it did before.  No change.

On 26 Jul 2020, at 21:09, jbk1988<EMAIL_ADDRESS>wrote:
Has anyone else had issues with the DoorBird Chime not working in this setup? Does it need to be setup as a separate HTTP call?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/brownad/homebridge-doorbird/issues/75, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABUK4CZ7YYOU76M6UVMR5S3R5SEQZANCNFSM4PIENFSA.

Likewise - I have several Chime devices. The Chimes are configured to speak with the Doorbird. There's no interaction model for them with this plugin. I'd validate that the chimes are working correctly with Doorbird, independent of this plugin.