ImportError: cannot import name 'patterns'

With Django 1.10 I'm getting the following stacktrace:
Traceback (most recent call last):
  File "manage.py", line 22, in <module>
    execute_from_command_line(sys.argv)
  File "g:\Python\lib\site-packages\django\core\management\__init__.py", line 367, in execute_from_command_line
    utility.execute()
  File "g:\Python\lib\site-packages\django\core\management\__init__.py", line 359, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "g:\Python\lib\site-packages\django\core\management\base.py", line 294, in run_from_argv
    self.execute(*args, **cmd_options)
  File "g:\Python\lib\site-packages\django\core\management\base.py", line 342, in execute
    self.check()
  File "g:\Python\lib\site-packages\django\core\management\base.py", line 374, in check
    include_deployment_checks=include_deployment_checks,
  File "g:\Python\lib\site-packages\django\core\management\base.py", line 361, in _run_checks
    return checks.run_checks(**kwargs)
  File "g:\Python\lib\site-packages\django\core\checks\registry.py", line 81, in run_checks
    new_errors = check(app_configs=app_configs)
  File "g:\Python\lib\site-packages\django\core\checks\urls.py", line 14, in check_url_config
    return check_resolver(resolver)
  File "g:\Python\lib\site-packages\django\core\checks\urls.py", line 24, in check_resolver
    for pattern in resolver.url_patterns:
  File "g:\Python\lib\site-packages\django\utils\functional.py", line 35, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
  File "g:\Python\lib\site-packages\django\urls\resolvers.py", line 313, in url_patterns
    patterns = getattr(self.urlconf_module, "urlpatterns", self.urlconf_module)
  File "g:\Python\lib\site-packages\django\utils\functional.py", line 35, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
  File "g:\Python\lib\site-packages\django\urls\resolvers.py", line 306, in urlconf_module
    return import_module(self.urlconf_name)
  File "g:\Python\lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 986, in _gcd_import
  File "<frozen importlib._bootstrap>", line 969, in _find_and_load
  File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 673, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 665, in exec_module
  File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
  File "g:\Git\Jam\jam_music\urls.py", line 69, in <module>
    url(r'^messages/', include('django_messages.urls')),
  File "g:\Python\lib\site-packages\django\conf\urls\__init__.py", line 50, in include
    urlconf_module = import_module(urlconf_module)
  File "g:\Python\lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 986, in _gcd_import
  File "<frozen importlib._bootstrap>", line 969, in _find_and_load
  File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 673, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 665, in exec_module
  File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
  File "g:\Python\lib\site-packages\django_messages\urls.py", line 1, in <module>
    from django.conf.urls import patterns, url
ImportError: cannot import name 'patterns'

I believe that was deprecated in Django 1.8, have other people modified the source code in order to get this to work?

Please install the current git master. The version on PyPI is outdated and will be replaced with a new release soon.


Strange style of active and inverted sap.m.ObjectStatus control used in sap.ui.table.Column

OpenUI5 version: 1.120.17
Browser/version (+device/version): latest chrome
Any other tested browsers/devices(OK/FAIL): -
URL (minimal example if possible): https://sdk.openui5.org/1.120.17/#/entity/sap.ui.table.Table/sample/sap.ui.table.sample.Basic/code
User/password (if required and possible - do not post any confidential information here): -
Steps to reproduce the problem:

Go to the sample https://sdk.openui5.org/1.120.17/#/entity/sap.ui.table.Table/sample/sap.ui.table.sample.Basic/code
Add column with ObjectStatus control:
<Column width="6rem">  <m:Label text="ObjectStatus" />  <template>           <m:ObjectStatus                 inverted="true"                 active="true"                 state="Warning"                 icon="sap-icon://message-warning"/>       </template> </Column>
See the style of the ObjectStatus

What is the expected result? Style is correct. It works on 1.121.5, 1.122.1, etc.:

What happens instead? Style is broken. It doesn't work on 1.120.17 with long-term maintenance:

Any other information? (attach screenshot if possible) -

Hello @AnastasyaNN ,
Thank you for sharing this finding. I've created an internal incident DINC0205553. The status of the issue will be updated here in GitHub.
Regards,
Gergana

Hello @AnastasyaNN ,
I had internal discussion with the team and since version 1.120 is a long running it has a strict rules for downporting changes and unfortunately the fix of visualisation of ObjectStatus doesn't fulfill those requirements.
Therefore it will not be fixed for this version.
Sorry for the inconvenience.
Best Regards,
Gery


Map frame reproject with source reload

A refactor of https://github.com/DotSpatial/DotSpatial/pull/1252. That PR will be closed and this one will be its replacement. In this PR we provide the new reprojection logic as an overload so that the current logic can be used or the dev can switch to the new one.
Reprojects all layers in the map frame so that they use new projection. Before they are reprojected the layers will get reloaded from the original source if available. This is useful when doing multiple reprojections on a layer as the reprojection will occur on the original vertices and not the reprojected vertices. For certain workflows multiple reprojections will tend to lose the original vertices and result in unexpected coordinates. This method attempts to alleviate the issue. This method also includes drawing layers, although they may lose their original vertices since they don't have a source from where we can reload.
Fixes https://github.com/DotSpatial/DotSpatial/issues/1104 and https://github.com/DotSpatial/DotSpatial/issues/782.
Using ReprojectMapFrame as it's currently implemented in DemoMap:
Load up a WGS84 featureset that straddles the edge of 90N
Reproject to North Pole Stereographic
Reproject back to WGS84

Now we use the overloaded version of ReprojectMapFrame in MapFrameProjectionDialog.ApplyChanges

Checklist

[x] I have included examples or tests
[x] I have updated the change log
[x] I am listed in the CONTRIBUTORS file
[x] I have cleaned up the commit history (use rebase and squash)

Changes proposed in this pull request:

Before reprojection of a layer it is reloaded from the source (if available) so that the original vertices are always used as the main source.
This method also includes drawing layers, although they may lose their original vertices since they don't have a source from where we can reload.

@sindizzy I just had a look at this and can't say I like the way this was done. You basically say IMapFeatureLayer, IMapImageLayer and IMapRasterLayer get reprojected but IMapSelfLoadLayers get ignored. I for one have a lot of IMapSelfLoadLayer and wouldn't be a friend of them not being reprojected, too. ;)
I'd say instead of reloading everything you know about in this central method, I'd create a new overload for Layer.Reproject with a reload parameter. That makes sure whoever uses an IMapSelfLoadLayer can say how to handle the reload process themselves.

@jany-tenaj not sure I follow your observations. If you look at the existing logic here all I have done is enhance it with a data reload done through an overload. The existing method works as it always has, but with the overload
bool doReloadSource = false
it does the same thing as before but if set to true then it reloads the data directly from the source rather than from memory.

The problem is the way you reload the layer. You say that if the layer is a featurelayer reload it like this and if it is an imagelayer reload it like that... But you do not say if it is an imapselfloadlayer reload it like this. They do not get reloaded and would continue to be reprojected without reloading.

True, but what happens to those layers now? I don't see any logic for those types of layers currently in the code.

mapFrame.GetAllLayers() also contains IMapSelfLoadLayer that simply get reprojected like all other layers.

I see what you are saying now. I am doing things with specific layer types and omitting IMapSelfLoadLayer. Let me ask you this...does IMapSelfLoadLayer have a source on the disk? If the data is only in memory then it will not be able to use the original source.

You can do with it what you want. It could habe a file as a source. It could come from a database or be in memory or whatever comes to mind. Its a way to create your own kind of layer that is different from the other layer types we have but have it drawn and so on by the main routine. Thats why I said extend the reproject method and don't try to handle the reloading where you are currently doing it.

I think I understand what you are saying now. If I only have the "reproject with reload" on the mapframe then we lose out on that functionality any other time we want to reproject.
Let me take another look and see how I can refactor my code.

@jany-tenaj ok so I'm looking at the code in DotSpatial.Data\FeatureSet and am testing different approaches. I cant seem to get this to work as every time I reload the source it just doesn't seem like the vertices are being rebuilt.
        public override void Reproject(ProjectionInfo targetProjection) //, bool doReloadSource = false)
        {
            //temp place holder for reloading data from the source before reprojection
            bool doReloadSource = true;

            // reload the data from the source if required and the source is on the disk. since an in-ram source loses its original source data
            // we cannot reload layers that are not on disk. In-ram sources will have to be reprojected as usual and the vertcies may lose their
            // precsiion. One thought is to save the original in-ram source as a seperate DataTable but that is beyond the scope of this mod.
            if (doReloadSource && this.CanReproject && !string.IsNullOrWhiteSpace(Filename))
            {
                //var fs = FeatureSet.OpenFile(Filename);
                //_features = fs.Features;
                //_vertices = fs.Vertex;
                //InitializeVertices();
                //FeaturesFromVertices();

                var res = Open(Filename);
                if (!res.AttributesPopulated)
                {
                    FillAttributes();
                    Vertex = res.Vertex;
                }
            }

            // reproject the points in the FeatureSet
            Projections.Reproject.ReprojectPoints(Vertex, Z, Projection, targetProjection, 0, Vertex.Length / 2);
            if (!IndexMode)
                UpdateCoordinates();


Text selection in explorer preview

Description of the new feature / enhancement
When I any text document is openned in explorer(preview) it is difficult to select section, after tripple click it selects only one line, but i cannot drag and select sections of text. I am not sure if powertoys can solve this problem or is it just windows problem. But anyway it would be really usefull to select text from selection window.
Scenario when this would be used?
When browsing through text files and having to select sections of a text.
Supporting information
No response

Feedback hub would be the better spot for this.  We could override it but that would be a different previewer then.

@Aaron-Junker Monaco would not work for this?

Linked issue / dup'ling #12637

@Aaron-Junker Monaco would not work for this?

@Jay-o-Way Monaco would of course also work for txt files, but it isn't made for it.

Could add in some override feature into it to enable additional formats

Could add in some override feature into it to enable additional formats

Yeah that will be a feature request for sure


cuttlefish Installation failure，help

git clone https://github.com/google/android-cuttlefish
cd android-cuttlefish
tools/buildutils/build_packages.sh
.....
debian/rules build
dh build --with=config-package
dh_update_autotools_config
dh_autoreconf
debian/rules override_dh_auto_build
make[1]: Entering directory '/home/ubuntu/riscv/Cuttlefish/android-cuttlefish/base'
cd cvd && bazel build --linkopt="-Wl,--build-id=sha1" cuttlefish/host/commands/cvd:cvd --spawn_strategy=local
Starting local Bazel server and connecting to it...
WARNING: For repository 'boringssl', the root module requires module version<EMAIL_ADDRESS>but got<EMAIL_ADDRESS>in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'googletest', the root module requires module version<EMAIL_ADDRESS>but got<EMAIL_ADDRESS>in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'protobuf', the root module requires module version<EMAIL_ADDRESS>but got<EMAIL_ADDRESS>in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'zlib', the root module requires module version<EMAIL_ADDRESS>but got<EMAIL_ADDRESS>in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: Couldn't auto load rules or symbols, because no dependency on module/repository 'rules_android' found. This will result in a failure if there's a reference to those rules or symbols.
INFO: Analyzed target //cuttlefish/host/commands/cvd:cvd (139 packages loaded, 3682 targets configured).
ERROR: /home/ubuntu/riscv/Cuttlefish/android-cuttlefish/base/cvd/cuttlefish/host/commands/cvd/BUILD.bazel:51:11: Compiling cuttlefish/host/commands/cvd/cli/parser/cf_flags_validator.cpp failed: (Exit 1): clang failed: error executing CppCompile command (from target //cuttlefish/host/commands/cvd:libcvd) /usr/lib/llvm-14/bin/clang -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -Wunused-but-set-parameter -Wno-free-nonheap-object -fcolor-diagnostics -fno-omit-frame-pointer ... (remaining 192 arguments skipped)
cuttlefish/host/commands/cvd/cli/parser/cf_flags_validator.cpp:19:10: error: module //cuttlefish/host/commands/cvd:libcvd does not depend on a module exporting 'google/protobuf/util/json_util.h'
#include <google/protobuf/util/json_util.h>
^
1 error generated.
Please help answer, thanks

This is an issue with upstream github.com/google/android-cuttlefish. I believe it is introduced by https://github.com/google/android-cuttlefish/pull/815 bringing in a new version of libcurl which implicitly pulls in a new version of protobuf. It is fixed by https://github.com/google/android-cuttlefish/pull/818, but that PR is blocked on submit by our presubmit tests, which are broken by bazel's migration from using WORKSPACE by default to bzlmod by default. The WIP PR https://github.com/google/android-cuttlefish/pull/820 attempts to fix our e2e tests, but I am still working on it.

FYI, https://github.com/google/android-cuttlefish/pull/820 is merged, fixing this issue.

FYI, google/android-cuttlefish#820 is merged, fixing this issue.

Hi, Thank you very much for your reply.


Unable to upgrade all packages at once

Brief description of your issue
When running winget update or winget upgrade it errors with the following
Failed when searching source: msstore
An unexpected error occurred while executing the command:
0x8a150044 : The rest API endpoint is not found.
Logs show
Include:NormalizedNameAndPublisher='googlechromedev'+'google'[Exact]]
2024-12-05 10:18:39.406 [REPO]   Checking match with package id: Google.Chrome.Dev.EXE
2024-12-05 10:18:39.406 [REPO]   Checking match with package id: Google.Chrome.Dev
2024-12-05 10:18:39.484 [REPO] Sending http GET request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/packageManifests/9P7KNL5RWT25?Market=GB
2024-12-05 10:18:39.517 [REPO] Response status: 200
2024-12-05 10:18:39.568 [REPO] Sending http GET request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/packageManifests/9NZSG2H7MS6B?Market=GB
2024-12-05 10:18:39.940 [REPO] Response status: 404
2024-12-05 10:18:39.944 [FAIL] C:__w\1\s\external\pkg\src\AppInstallerCommonCore\HttpClientHelper.cpp(187)\WindowsPackageManager.dll!00007FFAF1067BF1: (caller: 00007FFAF1066F29) Exception(1) tid(9288) 8A150044
2024-12-05 10:18:39.944 [FAIL] C:__w\1\s\external\pkg\src\AppInstallerRepositoryCore\CompositeSource.cpp(1127)\WindowsPackageManager.dll!00007FFAF117420A: (caller: 00007FFAF10AC6F3) LogHr(1) tid(9288) 8A150044     Msg:[C:__w\1\s\external\pkg\src\AppInstallerCommonCore\HttpClientHelper.cpp(187)\WindowsPackageManager.dll!00007FFAF1067BF1: (caller: 00007FFAF1066F29) Exception(1) tid(9288) 8A150044 ]
2024-12-05 10:18:39.944 [REPO] Failed to search source for correlation: msstore
2024-12-05 10:18:39.944 [REPO] Did not find Id [9NZSG2H7MS6B] in tracked source: msstore
I have run a winget source reset --force
and a dism check.
Still have problem.
Steps to reproduce
Run a winget update --all
Expected behavior
That the command updates packages
Actual behavior
Error thrown and update doesn't happen.
You can update individual packages.
Environment
Windows Package Manager (Preview) v1.10.40-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22631.4460
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v<IP_ADDRESS>

Winget Directories
-------------------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled
ProxyCommandLineOptions                   Disabled
DefaultProxy                              Disabled

When I clicked on the links for the REST requests, the first package returned a result, but the second did not. I also performed a WinGet Search for 9NZSG2H7MS6B which yielded a failure. Do you know what package that is?

When I clicked on the links for the REST requests, the first package returned a result, but the second did not. I also performed a WinGet Search for 9NZSG2H7MS6B which yielded a failure. Do you know what package that is?

@denelon - https://apps.microsoft.com/detail/9nzsg2h7ms6b

I ran the following code from elevated powershell and it worked
winget source reset msstore


Backport #24980 to v1.34.x

Backports https://github.com/grpc/grpc/pull/24980 (to fix https://github.com/grpc/grpc/issues/24935).
After this is merged, we will probably need to do a patch-release (C# only is fine if nothing else needs a patch release)

Hi Jan, another backport(#25010) has been merged, will you able to merge this RP by tomorrow? If so, I can do a patch-release this week.

This PR is ready to merge and the test failures are unrelated (this PR only touches C# build and https://github.com/grpc/grpc/pull/24980 has already been merged into master).
But the v1.34.x branch is protected so I need someone with the right permissions to merge this PR. @nicolasnoble @srini100 ?

This PR is ready to merge and the test failures are unrelated (this PR only touches C# build and https://github.com/grpc/grpc/pull/24980 has already been merged into master).
But the v1.34.x branch is protected so I need someone with the right permissions to merge this PR. @nicolasnoble @srini100 ?

1.34.x is now unlocked for the patch.

1.34.x is now unlocked for the patch.


chore(deps): upgrade crypto-js from 4.1.1 to 4.2.0

Description
Upgrades crypto-js following the release of a new security advisory.
Test Plan

[x] Test that account made on a prior version handle the upgrade without issue.

Related issues
N/A
Backwards compatibility
Yes

also, dependabot opens these automatically fyi https://github.com/valora-inc/wallet/pull/4385

hey @MuckT i went ahead and merged #4385 instead to unblock the CI. i see that you added a yarn resolution for the crypto-js version in this PR, which i wasn't sure we wanted (we usually save the resolutions for indirect dependencies only)

I think we must use the resolution as @celo/utils blocks it to 3.x
It will be harmless as it doesn't even use it internally.
Also it's been removed since then.


IS-1248: Allow user to modify cred_def

Signed-off-by: artem.ivanov<EMAIL_ADDRESS>

Case: Private keys for Cred Def have been compromised.
Here is implemented a similar approach as for Verkey rotation (need to call the combination of 3 functions).
Issuing of new a credential even doesn't require writing a transaction to the Ledger (cred def without revocation support). So, Issuer even doesn't know about which credentials have been issued.

Is there documentation being written about this feature where we could add the line, "Warning, rotating the cred_def keys will result in making all credentials issued under the previous cred_def keys unverifiable"

@brentzundel Couldn't verification just compare the date of rotation against the date of credential signing? I am not surprised that we break our current verification algorithm, but I'm surprised if this is a deeper problem that can't be fixed...

I suppose that when it is possible to request a version of a cred_def from a particular point in the past, then my concerns may be moot.

@brentzundel Couldn't verification just compare the date of rotation against the date of credential signing? I am not surprised that we break our current verification algorithm, but I'm surprised if this is a deeper problem that can't be fixed...

If the issuer includes the signing date as an attribute then this should be easy or they would have to include that metadata themselves. The holder would need a way to communicate the cred_def version to the verifier. Is there a way to query the ledger as of a specific datetime? Or would the holder need to know the transaction id when the cred_def was created so as to state "this credential was signed by the key at transaction X."


How to include angular unit test case in TECO

Hi Michal ,
How to include angular unit test case in TECO. could you please help me.

Insert probes into your angular app via instrument2.py ,go to dashboard and create a test session and then run your unit tests. End session when it's done either via dashbord or api. The session will have the coverage report.
Refer to wiki if any questions regarding the process.
Note that the tool was not tested with unit tests but it should work just fine.


Rewrite theme to new light defaults

Changes proposed in this pull request:
Implement new light theme

Can one of the admins verify this patch?

ok to test


Support atlas credentials as part of helm deployment

Currently the atlas userid/password is hardcoded to default.
a) The current vdc chart should manage the require passwords as secrets
b) the atlas-init image should use them
c) The atlas image should set them
See odpi/egeria#555

The VDC environment will need significant re-work to adapt to the many changes made in Egeria to better support
metadata integration. Additionally the helm charts used for the lab & a simple base config have evolved to better support
different types of services, exposing of ports, persistent storage etc.
As such specific incremental changes to the current - now old - charts do not really add value.
As such closing for now


Error in exercise 22.1-4

On step 3 of that given algorithm, the algo seems to add any vertex that's present in the adjacency list of the multigraph to the new adjacency list. I think what it should be doing is, When it encounter any vertex(v) in the adj-list of u, it should check whether there is a edge (v, u), i.e. the opposite edge, and also checks if v and u are not the same. If both the condition is true, than it should add the edge to new Graph.
My approach is based on the assumption:

The mutligraph is implemented using adjList.
Multi edge means (u, v) as well as (v, u), so in the adjList i'm checking for (v, u) whenever i encounter (u, v) edge.

As it is specified that the graph is a multi graph it can be an undirected multigraph or a directed multigraph. Your algorithm would be true in case of undirected multi-graph

Thanks @mhhacker36 @tabshaikh


fix: log.Fatal() and log.Fatalf() will not exit with code 1

log.Fatal() will not exit with code 1
If you execute the following code.
if err!=nil{     log.Faltal(err) }
The expected situation is that the program prints an error and exits. Otherwise, execution with errors may cause the program to crash.
it seems to be missing os.exit(1) after logging.

Don't use this package. It's an internal deprecated package. Use top level logger. Is there any reason you're directly importing this?

Oh sorry, I may have watched an outdated video course.
It is true that it can be replaced by top level logger, Thanks.


Can't convert to Tensorflow Lite with operation Cast

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
TensorFlow installed from (source or binary): binary
TensorFlow version (or github SHA if from source): 1.12.0

Provide the text output from tflite_convert
Traceback (most recent call last):
  File "predict_tflite.py", line 740, in <module>
    evaluatePlanes(args)
  File "predict_tflite.py", line 95, in evaluatePlanes
    predictions = getResults(options)
  File "predict_tflite.py", line 289, in getResults
    pred_dict = getPredictionCustom(options)
  File "predict_tflite.py", line 538, in getPredictionCustom
    tflite_model = converter.convert()
  File "/home/alexfu/PlaneNet/venv/local/lib/python2.7/site-packages/tensorflow/contrib/lite/python/lite.py", line 453, in convert
    **converter_kwargs)
  File "/home/alexfu/PlaneNet/venv/local/lib/python2.7/site-packages/tensorflow/contrib/lite/python/convert.py", line 342, in toco_convert_impl
    input_data.SerializeToString())
  File "/home/alexfu/PlaneNet/venv/local/lib/python2.7/site-packages/tensorflow/contrib/lite/python/convert.py", line 135, in toco_convert_protos
    (stdout, stderr))
RuntimeError: TOCO failed see console for info.
2019-04-27 02:28:16.893332: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 1517 operators, 2341 arrays (0 quantized)
2019-04-27 02:28:16.938974: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 1517 operators, 2341 arrays (0 quantized)
2019-04-27 02:28:16.939290: F tensorflow/contrib/lite/toco/graph_transformations/resolve_constant_unary.cc:164] Unsupported cast op input type
Aborted (core dumped)

None

Also, please include a link to a GraphDef or the model if possible.
https://mega.nz/#!sjpT2DiQ!Uo-6hxyldmtnPoKk3TTdUHKZADRGy6nIPlmAeVzJs_8
(Actually using DeepLab in this model)
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@futuyao : Did you get chance to have a look on this link. Let us know if that helps. Thanks!

Cast is a supported operation. But in my model, I'm using operation Cast with input type Bool and output type float32. I'm not sure if the casting from input type Bool is supporter. Do you have more information on this?
More details here at line 234 at tensorflow/lite/toco/graph_transformations/resolve_constant_unary.cc:
if (unary_op->type == OperatorType::kCast) {
    for (int i = 0; i < output_buffer_size; i++) {
      float outval = 0.0f;
      if (input_array.buffer->type == ArrayDataType::kFloat) {
        outval = static_cast<float>(
            input_array.GetBuffer<ArrayDataType::kFloat>().data[i]);
      } else if (input_array.buffer->type == ArrayDataType::kUint8) {
        outval = static_cast<float>(
            input_array.GetBuffer<ArrayDataType::kUint8>().data[i]);
      } else if (input_array.buffer->type == ArrayDataType::kInt32) {
        outval = static_cast<float>(
            input_array.GetBuffer<ArrayDataType::kInt32>().data[i]);
      } else if (input_array.buffer->type == ArrayDataType::kInt64) {
        outval = static_cast<float>(
            input_array.GetBuffer<ArrayDataType::kInt64>().data[i]);
      } else {
        LOG(FATAL) << "Unsupported cast op input type";
      }
      output_float_data[i] = outval;
    }

@futuyao  I think cast operation does not support bool input.

@futuyao I think cast operation does not support bool input.

I've made a small change to the converter code, and it should support bool after the fix. See #28315 .

@futuyao Can you submit this PR against master? . Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

@futuyao I think cast operation does not support bool input.

I've made a small change to the converter code, and it should support bool after the fix. See #28315 .

I am facing the same issue while converting model to TFLite. can you please tell how you solved it?

This issue is now fixed on master, try converting with RC 2 toco.


feat(containers-stats): add refresh rate of 1 and 3 seconds

Sometimes user wants to monitor process and CPU usage even more frequently, such as 1s or 3s.

Sorry for the late review @linquize
Thanks :+1:


Confusing error in CLI when a blocking invoke fails

Blocking activations fail with a very confusing error.
Environment details:
any environment
Steps to reproduce the issue:

launch an action that fails with blocking=true

Provide the expected results and outputs:
something like:
error: Invoked the action 'myNamespace/myAction', but it errored:
{
...
}

Provide the actual results and outputs:
stderr: error: Unable to invoke action 'myNamespace/myAction': The connection failed, or timed out. (HTTP status code 502)

Possibly duplicate of #1353 ?

Partly agree, though #1353 seems to be more geared towards the internals? I think this is just a bug in the response handling in the CLI.

Which invoker runtime were you using when this issue was encountered?
I'm pretty sure the CLI encountered a 502 HTTP response code (similar to #1353).  Would you please run the same command with the -d option and include the CLI output.

Yes the CLI encountered a 502. That's what the controller returns for a blocking invocation that fails with an application developer error. It's specified like that in swagger.

In addition to the -d CLI dump, would you also include the action code you used when encountering this issue.  If the code is not nodejs based and is not using a rejected Promise to return error results, then this is a separate issue.

@dubeejw - I was able to recreate this failure with the following action.  The CLI is apparently not handling the error parsing well when the error response is an object.  The HTTP response successfully returns the error object, but the CLI has a parsing bug that results in the error error: Unable to invoke action 'whiskrejectobj': The connection failed, or timed out. (HTTP status code 502)
whiskrejrectobj.js
function main(message) {
    return Promise.reject({
        name: 'myname',
        aflag: true,
        reason: 'failure cause'
    });
}

Thanks for doing my job and reproducing the error @mdeuser :+1:

Fixed by #1405


Update README.md

Ainsley, pls see my edits for the IBM Cloud Catalog

It looks like you updated the main README.md file for the entire repo. I thought you would make edits to docs/ibm-docs/ . Do you want to overwrite the README.md at this time, or have these changes made in docs/ibm-docs/about-safetrace.md ? cc @cankisagun


Tabs: Not preserving the query string when it navigates to the tab anchor

Info
Reported by Kevin Parkinson
If you have a query string in a URL, and on that same page there is a goa-tab component. If that tab is selected, the query string in the URL disappears.
Replication Steps

Load up a page using a URl like http://localhost:4200/tabs?someQueryString=something
Assuming the page has goa-tabs on it, the page won't load with the query string, only the tab ID

Additional Problem
This isn't a user reported problem, just something noticed. If you try to go to a link with a tab-id in it, http://localhost:4200/tabs#tab-1 for example, directly by using the address bar. The page will still load on the first tab, not the tab indicated by in the URL
Acceptance Criteria

If goa-tab is used on a page, it should still respect query string parameters in the URL
Loading a page with a specific tab id in the URL should load the page with that tab selected

Moving back to Ready as the issue has been confirmed and replicated

As discussed in the Dev meeting, we check for the tab indicated in the url (e.g. "/tab-0") one time only.... in the beginning.


Stopline label

How do you label the stopline? And how can I load it?

Thank you for your attention. Due to the lack of data samples, we have not yet released the label of stopline. We have fixed the corresponding readme.


[Transaction Coordinator] Bootstrap pulsar system namespace and create TC assign topic.

Motivation
Introduce pulsar system namespace and init transaction coordinator assign topic when init pulsar cluster.
Broker will bootstrap the system namespace if broker enable transaction coordinator.
Verifying this change
Added new unit tests for system namespace Bootstrap
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

Does this pull request introduce a new feature? (yes)

retest this please


High CPU usage

Current behavior
I recently have done several package upgrades in my project, including webpack, but fork-ts-checker wasn't upgraded. The project uses Electron.
I noticed that after the upgrade, when compilation is started there's a huge CPU spike and the computer freezes for about a minute, before it can operate again. Even the mouse pointer freezes. Disabling ForkTsCheckerWebpackPlugin plugin in webpack config resolves the issue. It happens right after the renderer code has been compiled.
I tried passing logger into the options of the plugin, but it doesn't provide any valuable insight of what's happening.
Expected behavior
Shouldn't freeze.
Steps to reproduce the issue
I don't see any particular reproduction steps, as the issue manifested after one of the package upgrades (there was no code change).
Issue reproduction repository
Unfortunately, my organisation doesn't allow sharing the repository, but I'm happy to run any troubleshooting steps that are necessary to pinpoint the issue.
Environment

fork-ts-checker-webpack-plugin: 6.4.2
typescript: 4.5.2
eslint: 8.3.0
webpack: 5.64.4
os: "Debian GNU/Linux 10 (buster)"

I'd +1 this. After a recent upgrade in our containers, when this plugins starts executing, it uses 100% CPU for ~1 minute.

I rolled back all the changes, but this problem didn't go away. I guess it is not related to the upgrade, after all. I noticed that on Node v14 it is actually much worse than on Node v16. It completely froze my computer, and didn't unfreeze it after 15 minutes.
What I noticed is that disk activity LED on my tower is showing constant activity when this happens.
I tried to run this together with top and iotop. Top shows that most CPU activity is spent on kswapd0, and disk activity didn't show anything helpful, but because the computer becomes very very slow at this moment, maybe these tools just didn't have enough processor time to accumulate the data.

It's probably a file watcher that causes high CPU usage.
You can try to tweak settings: https://www.typescriptlang.org/docs/handbook/configuring-watch.html#configuring-file-watching-using-a-tsconfigjson

I assume that watch settings were the source of the issue. Let me know if tweaking watch settings doesn't help.


python312Packages.pyinstaller: reformat; 6.10.0 -> 6.11.0; fix build

https://pyinstaller.org/en/v6.11.0/CHANGES.html

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

[ ] 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.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

Commenting to note that this is needed for #336191.
Thanks @mweinelt!


Associating dock icons to apps is unreliable

Known to happen with iTerm2, PyCharm, probably other apps
this is probably because it is using string matching instead of some better technique

sometimes forgets previews already there so does not hide them properly


Changed styling for meeting preview participant badges for big numbers

… up to 9999
closes #1410
This may not be the final version since the look of it should first be approved by our Layouter:
Part of the problem was, that the badges (i.e. the blue circles) weren't big enough to fit 4-digit numbers. They'd be displayed as something like '1...', '2...', and so on, which is why such numbers were hidden.
This PR addresses this by making the font size smaller for numbers that are bigger than 999 and raising the maximum limit to 9999 (after which the numbers will still be hidden).
Alternative solutions that could be taken if this new 'layout' isn't approved include:

Showing the bigger numbers in short form (i.e. '1k', '>1k', '>2k', '>3k') (this wouldn't require scaling for numbers <=10k) or
in rounded short form (i.e. 1023 -> '1k', 1469 ->  '1.5k', 1734 ->  '1.7k', 200 -> '2k', 10845 ->'10k', and so on) (this would still require scaling for the point-notation numbers and those above 9999, but would prevent any numbers from being hidden again)

@m-schieder I just pushed a new commit, the test instance should probably be updated


[Snyk] Upgrade eslint-config-standard-with-typescript from 39.0.0 to 39.1.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade eslint-config-standard-with-typescript from 39.0.0 to 39.1.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 2 months ago, on 2023-10-07.

Release notes

Package name: eslint-config-standard-with-typescript

39.1.1 - 2023-10-0739.1.1 (2023-10-07)
Build system / dependencies

renovate shared config (e83cfd8)
unpin actions (18c0988)

Bug fixes

turn off ESLint no-implied-eval (ed8f1d4)

39.1.0 - 2023-09-2339.1.0 (2023-09-23)
Build system / dependencies

add-to-project workflow (dd3ceae)
automergeMinor eslint-config-standard (f531394)
tsconfig.module = nodenext (7590972)

Features

docs: replace npm@<7 with yarn (4dbc4b7), closes #1254

39.0.0 - 2023-08-2739.0.0 (2023-08-27)
⚠ BREAKING CHANGES

deps: minimum @ typescript-eslint packages bumped to v6.4.0

closes #1202
chore

deps: @ typescript-eslint packages bumped to v6.4.0 (1d1ba51)

    from eslint-config-standard-with-typescript GitHub release notes

Commit messages

Package name: eslint-config-standard-with-typescript

dff4053 chore(release): 39.1.1 [skip ci]
0d30679 Merge pull request #1292 from standard/test-equivalents
ed8f1d4 fix: turn off ESLint no-implied-eval
9e0d772 Merge pull request #1290 from standard/renovate/node-20.x
f1b3a89 chore(deps): update dependency @ types/node to v20.8.3
c2b009b Merge pull request #1289 from standard/renovate/eslint-8.x
7af95c7 chore(deps): update dependency eslint to v8.51.0
5284be4 Merge pull request #1288 from standard/renovate/node-20.x
38aebcb chore(deps): update dependency @ types/node to v20.8.2
4ab66cb Merge pull request #1287 from standard/renovate/@ typescript-eslint-packages
c83fb94 chore(deps): update @ typescript-eslint packages to v6.7.4
edde1b9 Merge pull request #1286 from standard/renovate/type-fest-4.x
1f1a855 chore(deps): update dependency type-fest to v4.3.3
5790de4 Merge pull request #1285 from standard/renovate/node-20.x
363f35c chore(deps): update dependency @ types/node to v20.8.0
4728ed9 Merge pull request #1284 from standard/unpin-actions
18c0988 build: unpin actions
1515661 Merge pull request #1283 from standard/renovate-shared
e83cfd8 build: renovate shared config
c582888 Merge pull request #1271 from standard/renovate/actions-checkout-digest
3ca2832 Merge pull request #1267 from standard/renovate/semantic-release-22.x
7a839b5 chore(deps): update dependency @ types/node to v20.7.2 (#1282)
bdeb054 chore(deps): update dependency @ commitlint/cli to v17.7.2 (#1281)
b2fda34 chore(deps): update dependency @ types/node to v20.7.1 (#1280)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (1fbec0b) 77.91% compared to head (f5d99b4) 77.91%.

:exclamation: Current head f5d99b4 differs from pull request most recent head fa9499d. Consider uploading reports for the commit fa9499d to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1948   +/-   ##
=======================================
  Coverage   77.91%   77.91%
=======================================
  Files          39       39
  Lines        3065     3065
  Branches      617      617
=======================================
  Hits         2388     2388
  Misses        484      484
  Partials      193      193

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


Minor typo in User's Guide

The User's Guide document states the anti-symmetry condition for partial ordering to be:

if a <= b and b <= a, then a === c (anti-symmetry)

I'm guessing that this should read:

if a <= b and b <= a, then a === b (anti-symmetry)

BTW, thanks for this fantastic library!

Ah, yes, definitely a typo. Thanks!

Has been corrected, closing.


chore(deps): update dependency cache-loader to v1.2.5

This PR contains the following updates:

Package
Type
Update
Change
References

cache-loader
devDependencies
patch
1.2.2 -> 1.2.5
source

Release Notes

webpack-contrib/cache-loader
v1.2.5
Compare Source
Chores

drop support for webpack < 4 (#51) (2e76d3f)
drop support for node < 6 (#50) (b8225cd)

Features

add cacheContext option (#49) (22d0173)

BREAKING CHANGES

drop support for webpack < 4
drop support for node < 6

1.2.5 (2018-10-31)
Bug Fixes

index: this of stat method is undefined (#47) (5c67ccd)

1.2.4 (2018-10-31)
Bug Fixes

index: fallback to fs if this.fs is undefined (#45) (b84d13e)

1.2.3 (2018-10-30)
Performance Improvements

index: use the compiler's cached fs for stats (this.fs.stat) (#42) (d8c630b)

1.2.2 (2018-02-27)
Performance Improvements

use neo-async instead async (#31) (0851582)

1.2.1 (2018-02-26)
Bug Fixes

package: add webpack >= v4.0.0 (peerDependencies) (#32) (a5e921c)

v1.2.4
Compare Source
Bug Fixes

index: this of stat method is undefined (#47) (5c67ccd)

v1.2.3
Compare Source
Bug Fixes

index: fallback to fs if this.fs is undefined (#45) (b84d13e)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (1.2.5). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


Get fonts

Need to explore which font to use for the labels:
http://fontsup.com/font/microgramma-d-ot-bold-extended.html
The label font is correct, but I need to figure out which variant to use. The current one I'm using ( Microgramma-D-OT-Bold-Extended_30403 ) is a little too bold.
For the LCD, will just use a LCD font I found here:
http://www.dafont.com/lcd-lcd-mono.font

Done.


[Support] After installing and restarting nothing happens

Describe Your Problem:
I installed Cm4 and restarted HomeBridge, in Home aren’t demo devices and there are no logs from this plugin. Indent have a config and I didn’t know if I understood the wiki right that if I don’t have a config demo mode is enabled.
Cmd4 Config:
I don’t have a config

Environment:

Node.js Version: v14.17.3
NPM Version: v8.3.0
Homebridge Version: v1.4.0
homebridge-cmd4 Version: v6.1.2
Operating System: Docker
Process Supervisor: Docker

Hi,
There is a default config with the installed plugin,
Most likely /usr/local/lib/node_modules/homebridge-cmd4/Extras/config.json
Ttyl,
John
On Mon, Feb 14, 2022 at 5:19 PM WaldperlachFabi @.***>
wrote:

Assigned #128 https://github.com/ztalbot2000/homebridge-cmd4/issues/128
to @ztalbot2000 https://github.com/ztalbot2000.
—
Reply to this email directly, view it on GitHub
https://github.com/ztalbot2000/homebridge-cmd4/issues/128#event-6070603646,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSBCX6TCVQONANYDJJSHF3U3F5VNANCNFSM5OMXX72Q
.
You are receiving this because you were assigned.Message ID:
@.***>

Thank you for your help! Is it possible to add all devices in a „Test-Home“ useing a Bridge? Thank you! Fabi

Hi,
That config.json I pointed you to has all the possible devices
Ttyl,
John
On Mon, Feb 14, 2022 at 6:50 PM WaldperlachFabi @.***>
wrote:

Thank you for your help! Is it possible to add all devices in a
„Test-Home“ useing a Bridge? Thank you! Fabi
—
Reply to this email directly, view it on GitHub
https://github.com/ztalbot2000/homebridge-cmd4/issues/128#issuecomment-1039693288,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSBCX3LIMB5S7WZNZWB3W3U3GILTANCNFSM5OMXX72Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi, didn’t mean that 😂. I meant this Feature:

Thank you for your help!

Hi,
It's not looking familiar.  It looks like a homebridge-UI thing.  I'll look.
Ttyl,
John
On Tue, Feb 15, 2022 at 4:24 AM WaldperlachFabi @.***>
wrote:

Hi, didn’t mean that 😂. I meant this Feature:
[image: 6ED638C7-54C6-410D-9EE3-450D5DC0D23C]
https://user-images.githubusercontent.com/80318133/154031988-c72d4cf6-8e3a-46ec-817b-288047207eda.jpeg
Thank you for your help!
—
Reply to this email directly, view it on GitHub
https://github.com/ztalbot2000/homebridge-cmd4/issues/128#issuecomment-1040043089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSBCX6KWEIPDTJ5P7P7BG3U3ILT3ANCNFSM5OMXX72Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yeah normally you can then enable this switch [Picture 1] and then you will be able to add a bridge in HomeKit [Picture 2] with all Devices of the Plugin, but if i klick it on this plugin it says that i have to manually configurate it and i don’t know how to. [Picture 3

]
Thank you for your help!

Hi,
So I finally had some time to sit down and read carefully your issue and now understand your predicament.
Unfortunately you do have to configure Cmd4 manually.  Cmd4 does not have Homebridge-UI configuration.  There is an open issue on it.
I have seriously tried to get Homebridge-UI working with Cmd4.  The issue is that Cmd4 has so many devices and characteristics to work with, that I have found that creating the config.schéma for it is extremely difficult. I do have a private branch for UI, but it is nowhere near completion and the schéma is over 5000 lines!
So for now, you are going to have to work with the documentation available, and many examples.
Remember Cmd4 was created originally to demonstrate what HomeKit can do if you don't have any accessories at all. It morphed into a tool that you can write shell scripts to control devices either not supported by HomeKit, for features you wish to add or devices you have created yourself.
if I can help you further, just reply to this issue, even though it's closed.
ttyl,
John Talbot


Cannot upload SVG

Hi,
I tried creating a Django CMS plugin for uploading SVG images. I followed the instructions under extending Django Filer, but using File as base, not BaseImage. On my local machine (using SQLite and local file storage) it works, but when I try to use the production setup (PostgreSQL + AWS S3), it breaks. In the error log I noticed that the MultiStorageFieldFile uses Thumbnailer which, when trying to save, throws the error StringIO instance has no attribute 'size' (I think it may have to with the fact that I'm using Django 1.8.9 because it breaks in a line self._size = content.size which is no longer in the latest Django version). Because of this, I cannot upload SVG files (nor from my plugin nor from the generic File definition inside filer).
Not actually sure if it's filer that's causing it because of Thumbnailer or if it's Django. Could you shed me some light?

The problem is possibly that gzip is activated for SVG files on S3. So the thumbnailer gets a gzip stream which it can't handle. As a workaround disable gzip for SVG files.

OMG!! That was it! Django-storages had GZIP on and the default config compresses SVGs! And for what I saw, that's new. I just redefined the GZIP setting to exclude SVGs and it worked! Thank you very much!

You can now upload images? and attack them to other tables using a CSV? PLEASE OH GOD PLEASE HELP ME


ISPN-9241 NPE when creating caches via HotRod on non-clustered servers

https://issues.jboss.org/browse/ISPN-9241

Code looks good but CI says no. Let's spin the wheel again.


threading support

I am doing some tests with the @threads macro, in case you are interested:
https://github.com/gdkrmr/MLKernels.jl/tree/threads
For the Gaussian kernel I get around 60% speedup for two threads.

any interest in this? I would be willing to rebase this.

Sure, I'll accept PRs.


Test engine refactor

Hi @jmafoster1 ,
I've marked it ready for review so you can start to look through the big changes here. There are one or two things to discuss in more detail I think?
I'll fix the linting issues over the next couple of working days, but thought it would be good for you to start reviewing now :)


211 restructure configuration format

closes #211

Codecov Report
Base: 77.70% // Head: 77.70% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (6bf15c1) compared to base (a91df4d).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #233   +/-   ##
=======================================
  Coverage   77.70%   77.70%
=======================================
  Files         104      104
  Lines        7939     7935    -4
=======================================
- Hits         6169     6166    -3
+ Misses       1770     1769    -1

Impacted Files
Coverage Δ

src/grid_warehouse.F90
85.34% <ø> (-0.49%)
:arrow_down:

src/dose_rates.F90
85.39% <100.00%> (+1.64%)
:arrow_up:

src/profile_warehouse.F90
89.18% <100.00%> (+0.02%)
:arrow_up:

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

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


crictl exec: add --quiet/-q, --ignore-error/-e and --parallel flags

What type of PR is this?
/kind feature
What this PR does / why we need it:
The flags can be used to further manipulate on the exec behavior. Follow-up on: https://github.com/kubernetes-sigs/cri-tools/pull/1603#issuecomment-2396328240
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

crictl exec: add `--quiet/-q`, `--ignore-error/-e` and `--parallel/-x` flags.

/approve
/lgtm


Missing file/package in import statement misreported as ESM error

Prerequisites

[x] Checked that your issue hasn't already been filed by cross-referencing issues with the faq label
[x] Checked next-gen ES issues and syntax problems by using the same environment and/or transpiler configuration without Mocha to ensure it isn't just a feature that actually isn't supported in the environment in question or a bug in your code.
[x] 'Smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, your usage of Mocha, or Mocha itself
[x] Ensured that there is no discrepancy between the locally and globally installed versions of Mocha. You can find them with: node node_modules/.bin/mocha --version(Local) and mocha --version(Global). We recommend that you not install Mocha globally.

Description
This lead me down the garden path: A bad import statement (where the targeted package doesn't exist) results in mocha claiming the containing file was not interpreted as ESM despite containing an import (and mocha swallows the actual, correct error message).
Error from Mocha:
/…/test.jsx:1
import {
^^^^^^

SyntaxError: Cannot use import statement outside a module

Error from Node.js:
Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@testing-library/jest-dom'
imported from /…/test.jsx

The bug originates from https://github.com/mochajs/mocha/blob/v9.1.3/lib/nodejs/esm-utils.js#L51
Steps to Reproduce

test.jsx
Without installing @testing-library/jest-dom:
import '@testing-library/jest-dom';

NODE_ENV=test NODE_OPTIONS="--loader=./loader.mjs" npx mocha ./test.jsx

(The loader live-transpiles jsx via esbuild and marks the file as esm; it's not especially important for the reproduction but it did help me find the bug in Mocha).
Expected behavior: Mocha reports the correct error (that the targeted package doesn't exist, as node correctly reports).
Actual behavior: Mocha wrongly reports a different, unrelated error.
Reproduces how often: 100%
Versions

The output of mocha --version and node node_modules/.bin/mocha --version: 9.1.3 (also occurs in 8.3.0)
The output of node --version: 17.2.0
Your operating system

name and version: macOS 12.1
architecture (32 or 64-bit): 64 bit

Your shell (e.g., bash, zsh, PowerShell, cmd): bash
Your browser and version (if running browser tests): N/A
Any third-party Mocha-related modules (and their versions):
Any code transpiler (e.g., TypeScript, CoffeeScript, Babel) being used (and its version): esbuild

Additional Information
N/A

I believe the checks for ERR_MODULE_NOT_FOUND and ERR_UNKNOWN_FILE_EXTENSION as they are here are both wrong/outdated:
ERR_MODULE_NOT_FOUND: I see the code comment about the red herring:
https://github.com/mochajs/mocha/blob/28b482472a519b7abaf30a18b8ad709707bfd5a7/lib/nodejs/esm-utils.js#L58-L65
I think ERR_REQUIRE_ESM is not thrown anymore (it's now a SyntaxError). Also, I don't really understand your logic for trying to handle it this way. AFAIK, Node.js (at least for modules) only throws ERR_MODULE_NOT_FOUND when something is actually not found; if that's not true, I think it's probably wrong and should be updated (if it is still the case, could you confirm and provide a stacktrace so I can fix it).
ERR_UNKNOWN_FILE_EXTENSION: require() will interpret any unknown file extension as commonjs, but it shouldn't be blindly applied without first checking whether format is known (for instance, packageJson.type = 'module' or a loader set the format to 'module'). So this condition should be paired with another condition, something like format !== 'module' && err.code === 'ERR_UNKNOWN_FILE_EXTENSION'.

@giltayar could you have a look at this one, please?

Oh, well hello @giltayar haha

@juergba @JakobJingleheimer. I've seen this too! Let me look into it this week (or the week after).

@JakobJingleheimer I tried reproducing the problem, and couldn't. My setup was Node and mocha versions exactly like yours,
$ npm install mocha babel-register-esm # babel-register-esm is a loader of mine that transpiles via babel
$ cat test.mjs
import '@testing-library/jest-dom';
$ npx mocha --loader=babel-register-esm ./test.mjs
(node:3252) ExperimentalWarning: --experimental-loader is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
(node:3252) DeprecationWarning: Obsolete loader hook(s) supplied and will be ignored: getFormat, transformSource

Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@testing-library/jest-dom' imported from /Users/giltayar/code/tmp/mocha-esm-problem/test.mjs
...

So it looks like it's working...? What did I miss here?

Your file extension is .mjs. That's why it's not reproducing for you 😉 Mocha has a specific check just above the code I cited before for .mjs (it returns before hitting the bugged code):
https://github.com/mochajs/mocha/blob/28b482472a519b7abaf30a18b8ad709707bfd5a7/lib/nodejs/esm-utils.js#L44-L46
Ensure your file extension is is not .mjs or .js.
$> NODE_ENV=test NODE_OPTIONS='--loader=./loader.mjs' npx mocha ./test.spec.jsx

/…/test/index.spec.jsx:1
import {
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Object.compileFunction (node:vm:352:18)
    at wrapSafe (node:internal/modules/cjs/loader:1026:15)
    at Module._compile (node:internal/modules/cjs/loader:1061:27)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1149:10)
    at Module.load (node:internal/modules/cjs/loader:975:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:999:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.exports.requireOrImport (/…/test/node_modules/mocha/lib/nodejs/esm-utils.js:56:20)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Object.exports.loadFilesAsync (/…/test/node_modules/mocha/lib/nodejs/esm-utils.js:88:20)
    at async singleRun (/…/test/node_modules/mocha/lib/cli/run-helpers.js:125:3)
    at async Object.exports.handler (/…/test/node_modules/mocha/lib/cli/run.js:374:5)

The contents of ./test/index.spec.jsx that produced the above is:
import 'nonexistent';

@JakobJingleheimer reproduced! The fix is really ugly, as this whole section of code is heuristics in figuring out whether the require threw for "real" reasons or just because it wasn't supposed to be require-ed.
But it's mostly passing the test of time, and I can't think of another way to fallback to require if import fails (for example, if there are require hooks that are needed).
Hmm... maybe we don't really need the require? is it time we got rid of it? AFAIR, require hooks are still called when importi-ng a CJS file.
(just thoughts. Will be fixed one way or another this weekend)

@JakobJingleheimer reproduced! The fix is really ugly, as this whole section of code is heuristics in figuring out whether the require threw for "real" reasons or just because it wasn't supposed to be require-ed.
But it's mostly passing the test of time, and I can't think of another way to fallback to require if import fails (for example, if there are require hooks that are needed).
Hmm... maybe we don't really need the require? is it time we got rid of it? AFAIR, require hooks are still called when importi-ng a CJS file.
(just thoughts. Will be fixed one way or another this weekend)

🤔 can you not trust ERR_MODULE_NOT_FOUND? If it's removed from the 3 conditions currently used to determine whether to try require(), Mocha will throw the true error.
Then in the try/catch for require(), check for a SyntaxError instead of the nonexistent ERR_REQUIRE_ESM.
I just tried both, and it worked for this bug's scenario:
Removing the ERR_MODULE_NOT_FOUND addresses the case where Node has already (correctly) determined the file doesn't exist.
Updating the check for ERR_REQUIRE_ESM → instanceof SyntaxError + selective substring match addresses where there wasn't enough info before to know whether require() is appropriate (so try and see, as Mocha already does), but it turns out, nope, not appropriate.

🤔 can you not trust ERR_MODULE_NOT_FOUND? If it's removed from the 3 conditions currently used to determine whether to try require(), Mocha will throw the true error.
Then in the try/catch for require(), check for a SyntaxError instead of the nonexistent ERR_REQUIRE_ESM.
I just tried both, and it worked for this bug's scenario:
Removing the ERR_MODULE_NOT_FOUND addresses the case where Node has already (correctly) determined the file doesn't exist.
Updating the check for ERR_REQUIRE_ESM → instanceof SyntaxError + selective substring match addresses where there wasn't enough info before to know whether require() is appropriate (so try and see, as Mocha already does), but it turns out, nope, not appropriate.

@JakobJingleheimer unfortunately, I can't. The ERR_MODULE_NOT_FOUND condition is there because you can run mocha ./foo and assume that Mocha will load ./foo.js.
But I fixed the problem and a PR will be coming in a few minutes! (it's a hack, but the whole way of determining whether a file is ESM or CJS is ultimately a hack)


Style Guide: If statements

I know @0xbadb002  loooves having the else part on a new line, but I can't stand it.
	if f == FileMode::Read {
		return "r";
	}
	else if f == FileMode::Write {
		return "w";
	}
	else if f == FileMode::Append {
		return "a";
	}

I'm not particularly bothered if you want to change it, I just prefer it because it's easier to move up and down and write documentation above.

People rarely use else if/ else on a new line and keeping the { on the same line, we should do what is the most common way of styling them.

triggered

hahahaha

This is one of the many holy wars where there isn't really an answer, it's all opinion. I think we should get a style, stick to it and write a formatter a la gofmt.
I'm not particularly bothered about this issue, though I personally prefer it for moving around if/else blocks quickly, commenting them in a clean manner and what not.

Yeah, a formatter is definitely needed. I love Go's gofmt.

My 2 cents: Having the else if on a new line would make it seems as if it was a separate statement, which it isn't.

It's clear that most people would prefer the else/else if on the same line as the brace, I'm okay with this.

I'm okay with this
You better be.

tinted :eyeglasses:


Localization load order

What kind of issue is this? (put 'x' between the square brackets)

[ ] Question. This issue tracker is not the place for questions. If you want to ask how to do
something, or to understand why something isn't working the way you expect it to, use
http://stackoverflow.com/questions/ask .
Provide working code, starting from http://codepen.io/marcandre/pen/jqbzyN?editors=101.
We monitor the tag parsley.js.

[x] Bug report. If you’ve found a bug, you must provide a minimal example in a CodePen,
starting from http://codepen.io/marcandre/pen/jqbzyN?editors=101 .

[ ] Feature Request. Make sure there's no good way to do what you want first;
consider asking on http://stackoverflow.com/questions/ask first.

The documentation mentions that it is possible to load the localization files before loading the parsley library file (http://parsleyjs.org/doc/index.html#psly-installation-localization). However, this results in "Reference Error: Parsley is not defined".
Localization files must be loaded after the library, since they reference the Parsley object. Either the documentation is simply out of date, or this is a bug.

Thanks for bringing this up. Documentation is out of date, sorry...

Fixed. Thanks again


Stateful test found by randomly

To reproduce:
TESTARGS="--randomly-seed=105155306 -vvv" make tests T=tests/unit/api/test_simple.py::TestSimpleDetail

(The T parameter is the narrowest scoping of tests I found that still exhibited the issue to save on a full test run.)
Other seeds that fail the same test:

1843708888
19699634

FAILED tests/unit/api/test_simple.py::TestSimpleDetail::test_with_files_no_serial[text/html-None] - AssertionError: assert equals failed

Failure:

========================================================================= FAILURES =========================================================================
________________________________________________ TestSimpleDetail.test_with_files_no_serial[text/html-None] ________________________________________________

self = <tests.unit.api.test_simple.TestSimpleDetail object at 0xffffa39bc310>, db_request = <pyramid.testing.DummyRequest object at 0xffff9ae9f310>
content_type = 'text/html', renderer_override = None

    @pytest.mark.parametrize(
        "content_type,renderer_override",
        CONTENT_TYPE_PARAMS,
    )
    def test_with_files_no_serial(self, db_request, content_type, renderer_override):
        db_request.accept = content_type
        project = ProjectFactory.create()
        releases = ReleaseFactory.create_batch(3, project=project)
        release_versions = sorted([r.version for r in releases], key=parse)
        files = [
            FileFactory.create(release=r, filename=f"{project.name}-{r.version}.tar.gz")
            for r in releases
        ]
        # let's assert the result is ordered by string comparison of filename
        files = sorted(files, key=lambda key: key.filename)
        urls_iter = (f"/file/{f.filename}" for f in files)
        db_request.matchdict["name"] = project.normalized_name
        db_request.route_url = lambda *a, **kw: next(urls_iter)
        user = UserFactory.create()
        JournalEntryFactory.create(submitted_by=user)

>       assert simple.simple_detail(project, db_request) == {
            "meta": {"_last-serial": 0, "api-version": API_VERSION},
            "name": project.normalized_name,
            "versions": release_versions,
            "files": [
                {
                    "filename": f.filename,
                    "url": f"/file/{f.filename}",
                    "hashes": {"sha256": f.sha256_digest},
                    "requires-python": f.requires_python,
                    "yanked": False,
                    "size": f.size,
                    "upload-time": f.upload_time.isoformat() + "Z",
                    "data-dist-info-metadata": False,
                    "core-metadata": False,
                }
                for f in files
            ],
        }
E       AssertionError: assert equals failed
E         {                                                                      {
E           'files': [                                                             'files': [
E                                                                                {
E                                                                                'core-metadata': False,
E                                                                                'data-dist-info-metadata': False,
E                                                                                'filename': 'LWlHXDYZQIKh-10.0.tar.gz',
E                                                                                'hashes': {
E                                                                                'sha256': 'd3a2990d1fc4c91a16eeb880dbd04f37bf645fb5df36126179
E                                                                                b7dffb692a5fe2',
E                                                                                },
E                                                                                'requires-python': None,
E                                                                                'size': 5823,
E                                                                                'upload-time': '2017-03-01T12:55:14.698742Z',
E                                                                                'url': '/file/LWlHXDYZQIKh-10.0.tar.gz',
E                                                                                'yanked': False,
E                                                                                },
E             {                                                                      {
E               'core-metadata': False,                                                'core-metadata': False,
E               'data-dist-info-metadata': False,                                      'data-dist-info-metadata': False,
E               'filename': 'LWlHXDYZQIKh-8.0.tar.gz',                                 'filename': 'LWlHXDYZQIKh-8.0.tar.gz',
E               'hashes': {                                                            'hashes': {
E                 'sha256': '7b791e0d2c30b12063115d379d6c588758a4147b41b2a3c509          'sha256': '7b791e0d2c30b12063115d379d6c588758a4147b41b2a3c509
E         a38b1c76d955cf',                                                       a38b1c76d955cf',
E               },                                                                     },
E               'requires-python': None,                                               'requires-python': None,
E               'size': 7891,                                                          'size': 7891,
E               'upload-time': '2009-02-12T05:59:26.341814Z',                          'upload-time': '2009-02-12T05:59:26.341814Z',
E               'url': '/file/LWlHXDYZQIKh-10.0.tar.gz',                               'url': '/file/LWlHXDYZQIKh-8.0.tar.gz',
E               'yanked': False,                                                       'yanked': False,
E             },                                                                     },
E             {                                                                      {
E               'core-metadata': False,                                                'core-metadata': False,
E               'data-dist-info-metadata': False,                                      'data-dist-info-metadata': False,
E         ---                                                                    ---
E                 'sha256': '8e4adef422c7671d4e50db09cc893d9e3cc77610429dd68cf8          'sha256': '8e4adef422c7671d4e50db09cc893d9e3cc77610429dd68cf8
E         2728f00c2783b8',                                                       2728f00c2783b8',
E               },                                                                     },
E               'requires-python': None,                                               'requires-python': None,
E               'size': 2104,                                                          'size': 2104,
E               'upload-time': '2019-03-22T08:14:52.990585Z',                          'upload-time': '2019-03-22T08:14:52.990585Z',
E               'url': '/file/LWlHXDYZQIKh-8.0.tar.gz',
E               'yanked': False,
E             },
E             {
E               'core-metadata': False,
E               'data-dist-info-metadata': False,
E               'filename': 'LWlHXDYZQIKh-10.0.tar.gz',
E               'hashes': {
E                 'sha256': 'd3a2990d1fc4c91a16eeb880dbd04f37bf645fb5df36126179
E         b7dffb692a5fe2',
E               },
E               'requires-python': None,
E               'size': 5823,
E               'upload-time': '2017-03-01T12:55:14.698742Z',
E               'url': '/file/LWlHXDYZQIKh-9.0.tar.gz',                                'url': '/file/LWlHXDYZQIKh-9.0.tar.gz',
E               'yanked': False,                                                       'yanked': False,
E             },                                                                     },
E           ],                                                                     ],
E           'meta': {'_last-serial': 0, 'api-version': '1.1'},                     'meta': {'_last-serial': 0, 'api-version': '1.1'},

tests/unit/api/test_simple.py:261: AssertionError

Source test here:
https://github.com/pypi/warehouse/blob/38ba926d2083de4d650340040ec247bb0a2c92bc/tests/unit/api/test_simple.py#L240-L286
However it is likely something about the setup/teardown of other tests is mutating some state that's being left over.

P.S. There's a smattering of statements that don't actually assert, might be worth fixing once this test passes.
Example:
https://github.com/pypi/warehouse/blob/38ba926d2083de4d650340040ec247bb0a2c92bc/tests/unit/api/test_simple.py#L285-L286
Note the lack of a leading assert.

Another seed: --randomly-seed=19699634

Here's another stateful failure with --randomly-seed=3227833760: https://github.com/pypi/warehouse/actions/runs/7957829420/job/21721870412?pr=15364

Here's another one: --randomly-seed=2329742959:
https://github.com/pypi/warehouse/actions/runs/7971390556/job/21834554609?pr=15427

Another one with: --randomly-seed=235656747:
https://github.com/pypi/warehouse/actions/runs/8007534583/job/21871911912?pr=15444

Another one with: --randomly-seed=3916416581:
https://github.com/pypi/warehouse/actions/runs/8009314618/job/21878157012

I've got a PR that fixes all but the last one -<PHONE_NUMBER> it doesn't reproduce for me. @di can you reproduce that one?

I was able to reproduce it locally once, but not every time. It's a weird failure, I would expect Release.classifiers to always be ordered based on: https://github.com/pypi/warehouse/blob/675794be779274b5dee6fcea274f87c131904fc4/warehouse/packaging/models.py#L520-L525
But maybe Classifier.ordering is getting set wrong somehow?

But maybe Classifier.ordering is getting set wrong somehow?

I looked into this, and I think the only time we actually set the value on this column is either during the release-level classifiers sync command, or explicitly in tests. Otherwise, there's no explicit order, so the order is implicit from whomever inserted it first.
I have a patch to have the ClassifierFactory create ordering based on insert sequence, but I don't want to add it to the mix until we either see more failures of this kind and can repro the failure.
diff --git a/tests/common/db/classifiers.py b/tests/common/db/classifiers.py
index 01118c8bd..5339fa7f8 100644
--- a/tests/common/db/classifiers.py
+++ b/tests/common/db/classifiers.py
@@ -10,6 +10,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.

+import factory
+
 from warehouse.classifiers.models import Classifier

 from .base import WarehouseFactory
@@ -18,3 +20,5 @@ from .base import WarehouseFactory
 class ClassifierFactory(WarehouseFactory):
     class Meta:
         model = Classifier
+
+    ordering = factory.Sequence(lambda n: n)

Alternately, instead of testing for an ordered list of classifiers in this test, we could compare the results as a set, so as to determine equivalent content for the resulting classifiers, as test_upload_succeeds_creates_release() is mainly looking to see that the posted data comes back correctly, and the order shouldn't matter to that test.

Well, Release.classifiers should always be an alphabetically ordered list. Perhaps order_by=Classifier.ordering is wrong, and that should be order_by=<something that alphabetizes the Classifier.classifier string>?

This feels like https://github.com/pypi/warehouse/pull/12701 again?

Well, format_classifiers is used in jinja templates only for the classifiers used on a given package. As we're seeing with different seedings of random, sort order can flip if unspecified.
Over here, we use the manually-sorted classifiers from the trove-classifiers source package to declare the order, which also feels a little weird:
https://github.com/pypi/warehouse/blob/68c1db9da8407da648c40323eb803be3d2482f90/warehouse/views.py#L359-L364

This one still seems to be happening, with --randomly-seed=3294924425: https://github.com/pypi/warehouse/actions/runs/8055152986/job/22002003778

Another: --randomly-seed=1697538841: https://github.com/pypi/warehouse/actions/runs/8378481333/job/22943316123

Another: --randomly-seed=1997427500: https://github.com/pypi/warehouse/actions/runs/9859466968/job/27223341094

Another: --randomly-seed=46468882: https://github.com/pypi/warehouse/actions/runs/10359474403/job/28676128926?pr=16205 (this one is Observations related, @miketheman)

Another:
make tests TESTARGS="--randomly-seed=2228510538" T=tests/unit/admin/views/test_ipaddresses.py::TestIpAddressList

Another:
make tests TESTARGS="Using --randomly-seed=326773721" T=tests/unit/admin/views/test_users.py::TestUserEmailSubmit::test_updates_user_emails

https://github.com/pypi/warehouse/actions/runs/12305494574/job/34345230107


Minimal example

Hey team, I've started playing with this amazing tool. Instead of starting from the full-fledged demo, I've been trying to figure out what a minimal solution looks like.
Specifically, I was trying to take the workbench approach and gradually add services / code until I had an MVP. I haven't had much success. It has been a challenge to tease out from the kitchen sink demo what is essential vs is what is nice-to-have.
Are you aware of any more bare-bones examples?
Do you have anywhere or thing you'd suggest I look at to familiarize myself with the essentials of bootstrapping this lib?
I really appreciate you folks putting this powerful tool together! ❤

I'm aware there is a big entry barrier.
It really depends on your needs, it can be as simple as:
import { initialize as initializeMonacoService} from 'vscode/services'
import * as monaco from 'monaco-editor'

const workerLoaders: Partial<Record<string, WorkerLoader>> = {
  editorWorkerService: () => new Worker(new URL('monaco-editor/esm/vs/editor/editor.worker.js', import.meta.url), { type: 'module' })
}

window.MonacoEnvironment = {
  getWorker: function (moduleId, label) {
    const workerFactory = workerLoaders[label]
    if (workerFactory != null) {
      return workerFactory()
    }
    throw new Error(`Unimplemented worker ${label} (${moduleId})`)
  }
}

// Call it once before doing anything else
await initializeMonacoService({})

monaco.editor.create(...)

And you'll have something pretty similar to what you can achieve with the official monaco-editor-core
Now you can have a look at the available service overrides: https://github.com/CodinGame/monaco-vscode-api?tab=readme-ov-file#monaco-standalone-services there are some relationship between them but the graph is quite complex so... the best is probably to use what you think you need to see if it works.
For example:

for syntax highlighting, you'll need the textmate, theme and languages services overrides, them import the relevant default extension
If you want to use the languages features (via LSP or the language workers) or UI components (views or workbench service overrides) I hightly recommend using the model and editor services overrides and using the reference way of creating models

Apart from those, most service overrides are optional and independent, feel free to browser them and cherry-pick what you need

I am not aware of any "bare-bones" examples though, feel free to contribute by adding one, I'll be happy to reference it :)

OK, I'll mark this as closed for the time being. If my explorations produce something usable and minimal, I'll reference this back.
Thanks for the help!

I am also interested in this after having mixed success with the demo.

Let's open this up and work on it :)

Minimal means something different for everyone

Let's try to address this problem then:

I'm aware there is a big entry barrier.

I've already done anything I can think of, do you have any suggestion?

A demo where you isolate out just the editor component.
Then a demo where you add the terminal component.
Then a demo where you add the sidebar component.

Unfortunately, not everything can be achieved

A demo where you isolate out just the editor component.

Yup, basically the stub I've added on my first comment? good idea

Then a demo where you add the terminal component.
Then a demo where you add the sidebar component.

You can JUST take a component and render it. As soon as you take UI parts, you kind of have to take all the layout stuff and most other parts as well


Crane 1.3.0: Unable to save Docker credentials

OS: Mac OS (M1)
Crane version: 1.3.0
Install method: upgraded from 1.2.1 - replaced app in Applications (also tried deleting and installing fresh)
Steps to reproduce:
Launch Crane
Enter Docker credentials
Click save
Click to the local/remote images lists
Click back to Docker credentials - user/pass are empty

 ⎈ dev-eks-kent  kent  Applications  Crane.app  …  MacOS  $  ./Crane                                                                                Mon Dec 20 10:44:32 2021
fork process /Applications/Crane.app/Contents/Resources/app.asar/dist/server.js
/Applications/Crane.app/Contents/Resources/app.asar/node_modules/@sentry/electron/dist/renderer/backend.js:48
            if (window.__SENTRY_IPC__ == undefined) {
            ^

ReferenceError: window is not defined
    at new RendererBackend (/Applications/Crane.app/Contents/Resources/app.asar/node_modules/@sentry/electron/dist/renderer/backend.js:48:13)
    at RendererClient.BaseClient (/Applications/Crane.app/Contents/Resources/app.asar/node_modules/@sentry/core/dist/baseclient.js:52:25)
    at new RendererClient (/Applications/Crane.app/Contents/Resources/app.asar/node_modules/@sentry/electron/dist/renderer/client.js:19:24)
    at Object.initAndBind (/Applications/Crane.app/Contents/Resources/app.asar/node_modules/@sentry/core/dist/sdk.js:18:18)
    at Object.init (/Applications/Crane.app/Contents/Resources/app.asar/node_modules/@sentry/electron/dist/renderer/index.js:41:12)
    at Object.specificInit (/Applications/Crane.app/Contents/Resources/app.asar/node_modules/@sentry/electron/dist/dispatch.js:121:113)
    at Object.init (/Applications/Crane.app/Contents/Resources/app.asar/node_modules/@sentry/electron/dist/sdk.js:52:16)
    at Object.<anonymous> (/Applications/Crane.app/Contents/Resources/app.asar/dist/ServerHandlers.js:24:8)
    at Module._compile (node:internal/modules/cjs/loader:1110:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1140:10)
    ```

pls use 1.2.1 before we fix that

solved, pls upgrade to v1.3.1


Remove datashader's dependency on odo

Datashader currently faces a couple of issues introduced by odo:

Deprecation warnings identified by issue https://github.com/blaze/odo/issues/543
Performance degradation identified by issue #396

I attempted to fix the performance issue in odo, after tracing it to a line which converts all the categorical codes into a list to be passed to datashape's Categorical constructor, however I'm not aware of a straightforward fix that achieves similar performance to the non-categorical case. Furthermore, a new release of odo will be required to realize any of the improvements made to it, and there is no indication of one arriving anytime soon. Given the situation it seems easier to remove odo from datashader altogether, which should achieve a performance boost for the #396 use-case, remove the warnings from https://github.com/blaze/odo/issues/543, and simplify the codebase. So I assign myself to this issue, and hope to open a PR within the next day or so.

This is done, as of PR #428.


[az.json] Modification has been made in Azerbaijani language

Made some changes in Azerbaijani language to improve the quality of semantic translation

For localization please use Transifex:
https://www.transifex.com/traccar/traccar/


write create objects method, fix generate haikus method

I talked with Joan, and we think it'd be best if we close this PR- Noah's computer changed the entire env folder setup and we thought it'd be best if we just copy the code that he added for generating haikus and pasted that into her own branch on a machine here at DBC.


python server running on port 8081 other than 80

python server is documented running on port 80, but python server script actually listens on port 8081

Good catch. I made the update to the python server. It now listens on port 80.
Thanks!


mergeData doesn't return expected results with age

I noticed when using mergeData that I didn't get what I expected for age.
> fm <- dplyr::filter(combined_metadata, age == 40, smoker == "yes", gender == "male")
> e <- filterByDataType(fm, "marker_presence") # reworked "selectWhere"
> filterByDataType
function(metadata, dataType) {
    filteredMetadata <- unique(unlist(metadata$studyName, use.names = FALSE)) %>%
        paste(dataType, sep = ".") %>%
        sapply(curatedMetagenomicData, dryrun = FALSE) %>%
        mergeData()
    filteredMetadata[ , metadata$sampleID]
}
> fm$age
[1] 40 40 40
> fm$subjectID
[1] "SAMEA104142325" "Travel006"      "wHAXPI043593-9"
> colData(e)[colData(e)$subjectID == "SAMEA104142325", ]$age
[1] NA
> colData(e)[colData(e)$subjectID == "wHAXPI043593-9", ]$age
[1] 40

Looking at SAMEA104142325 in JieZ_2017, I see that age has a decimal.
Out of curiosity, I tried
> t <- curatedMetagenomicData("JieZ_2017.marker_presence", dryrun = FALSE) %>% mergeData()

and age isn't available as a column for JieZ_2017.marker_presence but it is for other studies like AsnicarF_2020.marker_presence when I run it through mergeData. Is there a reason for this (and it's expected) or is this a problem?

I think this is the same issue @paolinomanghi documented at https://github.com/waldronlab/curatedMetagenomicDataCLI/issues/2.


Configure V2 to be case insensitive

While V1 is case insensitive, V2 is case sensitive.
I'm not sure if we consider this a bug or a feature. This PR assumes it is a bug and therefore changes the V2 behavior to be case insensitive.
I've also modified OfflineClusterIntegrationTest to run the case insensitivity tests with V2. Given that these tests were testing the case insensitivity but also the fact that V1 ignores the schema, I had to split them:

testCaseInsensitivity and testCaseInsensitivityWithColumnNameContainsTableName test case sensibility
testSchemaIgnoredInV1 and testIgnoresSchemaInColumnNameWithTestNameInV1 test that schema is ignored in V1

In case we consider case sensibility in V2 a feature, feel free to close this PR

@xiangfu0 @Jackie-Jiang

Codecov Report

Merging #11417 (bb3717f) into master (f31ce38) will decrease coverage by 48.42%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #11417       +/-   ##
=============================================
- Coverage     62.91%   14.49%   -48.42%
+ Complexity     1089      201      -888
=============================================
  Files          2301     2301
  Lines        123777   123777
  Branches      18834    18834
=============================================
- Hits          77879    17947    -59932
- Misses        40356   104313    +63957
+ Partials       5542     1517     -4025

Flag
Coverage Δ

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
14.49% <0.00%> (-35.37%)
:arrow_down:

java-17
?

java-20
0.00% <0.00%> (-62.77%)
:arrow_down:

temurin
14.49% <0.00%> (-48.42%)
:arrow_down:

unittests
14.49% <0.00%> (-48.42%)
:arrow_down:

unittests1
?

unittests2
14.49% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

.../java/org/apache/pinot/query/QueryEnvironment.java
0.00% <0.00%> (-90.60%)
:arrow_down:

... and 1471 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Don't publish with unsupported QoS

MQTT <IP_ADDRESS>:
If the Server included a Maximum QoS in its CONNACK response to a Client and it receives a PUBLISH packet with a QoS greater than this, then it uses DISCONNECT with Reason Code 0x9B (QoS not supported) …

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

0 of 0   changed or added relevant lines in 0 files are covered.
85 unchanged lines in 10 files lost coverage.
Overall coverage decreased (-0.05%) to 82.767%

Files with Coverage Reduction
New Missed Lines
%

opt/hostedtoolcache/Python/3.10.15/x64/lib/python3.10/site-packages/mqttproto/async_client.py
4
85.35%

opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/site-packages/mqttproto/async_client.py
4
85.35%

opt/hostedtoolcache/Python/3.13.0/x64/lib/python3.13/site-packages/mqttproto/async_client.py
4
85.35%

opt/hostedtoolcache/Python/3.11.10/x64/lib/python3.11/site-packages/mqttproto/async_client.py
4
85.35%

opt/hostedtoolcache/Python/3.12.7/x64/lib/python3.12/site-packages/mqttproto/async_client.py
4
85.35%

opt/hostedtoolcache/Python/3.13.0/x64/lib/python3.13/site-packages/mqttproto/client_state_machine.py
13
85.45%

opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/site-packages/mqttproto/client_state_machine.py
13
85.45%

opt/hostedtoolcache/Python/3.11.10/x64/lib/python3.11/site-packages/mqttproto/client_state_machine.py
13
85.45%

opt/hostedtoolcache/Python/3.12.7/x64/lib/python3.12/site-packages/mqttproto/client_state_machine.py
13
85.45%

opt/hostedtoolcache/Python/3.10.15/x64/lib/python3.10/site-packages/mqttproto/client_state_machine.py
13
85.45%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.05%

Covered Lines:
6724

Relevant Lines:
8124

💛  - Coveralls

I would be more favorable to a maximum_qos property.

Well your wish is my command. g


Bug: Not Call My Services Static Function In My Modules.

PHP Version
8.1
CodeIgniter4 Version
4.1.9
CodeIgniter4 Installation Method
Manual (zip or tar.gz)
Which operating systems have you tested for this bug?
Linux
Which server did you use?
fpm-fcgi
Database
MySQL 8.0.28
What happened?
Not Call My Service Static Function In My Module.
This issue is caused by "AppServices".
If I modify the source code, it works fine.
/system/Config/BaseService.php In getSharedInstance Static Function.
...
protected static function getSharedInstance(string $key, ...$params)
{
    ...
-   static::$instances[$key] = AppServices::$key(...$params);
+   static::$instances[$key] = static::$key(...$params);
    ...
}
...

After this modification, Services in the "app" folder and "app_my_module" folder work normally.
Steps to Reproduce
app_my_module
├ Config
│　└ Services.php
└ Controllers
My Services File.
<?php
namespace AppMyModule\Config;

use CodeIgniter\Config\BaseService;

class Services extends BaseService
{
    public static function myFunction(bool $getShared = true): myClass
    {
        if ($getShared)
            return static::getSharedInstance('myFunction');
        return new myClass();
    }
}

My Class File.
...
$myF = \AppMyModule\Config\Service::myFunction();
...

Expected Output
TypeError
AppMyModule\Config\Services::myFunction(): Return value must be of type myClass, null returned
Anything else?
No response

@yks118 Thank you for taking time.
But unfortunately I can't reproduce it.
Is something different from yours?
--- a/app/Config/Autoload.php
+++ b/app/Config/Autoload.php
@@ -43,6 +43,7 @@ class Autoload extends AutoloadConfig
     public $psr4 = [
         APP_NAMESPACE => APPPATH, // For custom app namespace
         'Config'      => APPPATH . 'Config',
+        'Blog'        => ROOTPATH . 'Blog',
     ];

     /**

<?php

namespace App\Controllers;

use Blog\MyClass;

class Home extends BaseController
{
    public function index()
    {
        $myClass = new MyClass();
        $myClass->test();
    }
}

<?php

namespace Blog\Config;

use Blog\MyClass;
use CodeIgniter\Config\BaseService;

class Services extends BaseService
{
    public static function myFunction(bool $getShared = true): MyClass
    {
        if ($getShared) {
            return static::getSharedInstance('myFunction');
        }

        return new MyClass();
    }
}

<?php

namespace Blog;

class MyClass
{
    public function test()
    {
        $myF = \Blog\Config\Services::myFunction();
        dd($myF);
    }
}

$ php public/index.php

┌─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
│ $myF                                                                                │
└─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
Blog\MyClass (0) ()
═══════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════
Called from .../Blog/MyClass.php:10 [dd()]

@kenjis
Hi. Thanks for your reply.
I tried again from the beginning, and I wrote the code according to the answer and confirmed that it works normally.
The problem was that I didn't write it in the Autoload.php file.
This is a problem that occurred while trying to automatically recognize a module instead of writing it in Autoload.php whenever a module is added.
After checking the reply you sent, I solved it in another way.
Thanks for your help.
Sorry for wasting your time.

Okay, nobody's perfect.


Add a render prop

Hey!
I have a real life use case where we'd like to be able to provide a custom masked input to flatpickr. To make it work properly I have to forward the ref to the input masking library as well so that it can attach its own handlers. Unfortunately the existing wrap option does not seem to be flexible enough to support this use case (as far as I tried).
I've added an additional render prop that can be used to have more control over the rendering process.

You're crushing it ha, great work!


Document localization

Context

Banano is a component library meant for use in Platanus projects. It has a bunch of components with configuration options to provide a base to begin building fast.
Banano uses vee-validate to provide support input validation. Components show the errorMessage as default value in a slot in input components

Changes
This PR documents localization:

Adds buttonText prop to BnFileInput, which was the only non-validation text that wasn't available as a prop
Adds docs page explaining how to localize error messages texts using yup, and other texts using props

 Localización: agregar documentación.


Daily / Total Traffic vs. 32bit values

(Not really an issue, so feel free to close this if you think it does not belong here.)
In your Grafana Dashboard you display Total Download & Last 24h Download. Which is nice!
Unfortunately, the TotalBytesReceived & TotalBytesSent counters provided by TR-064 (and therefore fritzconnection and your plugin) are 32bit (ui4) values. So they overflow after 4GB.

I checked your dashboard, but you seem to use InfluxDB (and a cumulative_sum?).
Any ideas / suggestion how to do this with Graphite?

Nevermind.
nonNegativeDerivative + integral did the job.