Add runme env store snapshot subcommand

The command should print a tabular TUI akin to runme list with data returned from the MonitorEnvStore API.
Example client here: https://github.com/stateful/runme/blob/main/examples/grpc-client/main.go#L122-L145
It'd be great to run it where it exits after the first snapshot is received and alternatively with a --watch / -w flag just continuously refreshes. However, let's focus on building the non-refreshing version first.
PS: Please see the session strategy stuff to infer the sessionID. I believe having RUNME_SESSION in the ENV should work as long as it's being used inside of VS Code.

Done.


fix: do not suppress errors when loading classes

This rescue block was suppressing real errors on our codebase. I had to remove this block on a local fork to be able to see the real cause of a bug that was silently happening.
I noticed that removing the blocks do not break any tests. Looks like we should just remove them?

@andreynering Thanks. good catch. I think it was missed when we converted the constantize to safe_constantize in #297
Can you share the example of what error was being squashed here? (for my own personal knowledge)
I did have concerns with the constantize here since loading the class here has caused race conditions in another codebase. But the added benefits of checking this class outweighed the rare issue.
Do you think you could put together a test here for when we specify an invalid class? (if it is super hard, then just say so and we can punt.

@flavorjones I missed this try/catch when you added the safe_constantize.
I'm assuming that we just missed it and I'd like to merge this change.
But I do have to ask: Did you leave this try/catch for a particular reason?
There is an issue when the inflection for the class is slightly different. (e.g.: class named VM but we try and load Vm) Is this the reason you left it in? Or was it just an oversight? This is a very rare edge case, but wanted to double check.

@kbrock We had a concern included on a given ActiveRecord model, but the content of included do; end block wasn't actually been ran. Ends up there was an error being raised, but that block was suppressing it.


Failed Test case

I am trying to create a PR for #63, but facing this error in the test cases. Can someone guide me on how to resolve this?

Sure - I've recently added these CI tests, which are documented a bit here
If you scroll up in the error log a bit, you'll see this:
Reorder python imports...................................................Failed
- hook id: reorder-python-imports
- exit code: 1
- files were modified by this hook

Reordering imports in augraphy/augmentations/__init__.py
Reordering imports in augraphy/augmentations/strikethrough.py

Add trailing commas......................................................Failed
- hook id: add-trailing-comma
- exit code: 1
- files were modified by this hook

Rewriting augraphy/utilities/figsharedownloader.py
Rewriting augraphy/augmentations/strikethrough.py

black....................................................................Failed
- hook id: black
- files were modified by this hook

reformatted augraphy/augmentations/dirtydrum.py
reformatted augraphy/augmentations/badphotocopy.py
reformatted augraphy/augmentations/lib.py
reformatted augraphy/augmentations/folding.py
reformatted augraphy/augmentations/lowinkline.py
reformatted augraphy/augmentations/pencilscribbles.py
reformatted augraphy/augmentations/strikethrough.py
reformatted augraphy/base/paperfactory.py
reformatted augraphy/utilities/figsharedownloader.py
reformatted augraphy/utilities/imageoverlay.py
reformatted augraphy/utilities/overlaybuilder.py
All done! ✨ 🍰 ✨
11 files reformatted, 35 files left unchanged.

What this means is, the code checkers we run in GitHub Actions thought there were incorrect parts of those files, so they changed them to the correct format. These tools can't automatically fix the files you commit or push, but if you use pre-commit (documented in https://github.com/sparkfish/augraphy/blob/dev/doc/contributing.md) locally, they can fix those files, prevent you from committing incorrectly-formatted code, and then when you commit again, the files you commit will be correctly formatted.
Try installing pre-commit with pip install pre-commit, and then in your project directory run pre-commit run --all-files to make the code checkers reformat everything.
Let me know if you need more help

Looks like you got it


fix typo in README

unline Cabal => unlike Cabal.

Thanks!

No problem. Thanks for hpack, it's awesome.
On Sun, Jun 5, 2016 at 11:25 PM, Sönke Hahn<EMAIL_ADDRESS>wrote:

Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sol/hpack/pull/108#issuecomment-223861332, or mute
the thread
https://github.com/notifications/unsubscribe/AACTpzerkEG5F9e7_av520Q8Qq_VB3q9ks5qI5M2gaJpZM4IuZX-
.


[Bug] Monaco 0.49.0 still ships with TS 5.0.2

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Link
https://microsoft.github.io/monaco-editor/playground.html?source=v0.49.0#XQAAAAIcAwAAAAAAAABBqQkHQ5NjdMjwa-jY7SIQ9S7DNlzs5W-mwj0fe1ZCDRFc9ws9XQE0SJE1jc2VKxhaLFIw9vEWSxW3yscw26dnCAEiAKctgAboa7AaZZDHLgrLqPRv_fRv2RyIokydlTdTYccsu6v6jDrc8jQ8ulrCVzjWBVadhzlM4CndfaRo4TdtOngMwKEugNUWDywuRgUpL9GqimHxLC4ufkji7j8557XjTAHG271EXRi9DODuwGeE5hhOACSVwod70zJSA8sYe4BAeVI3cqK-4pwSWk2Cp1aJF5Vjh3cP4pjhEu6dbemGiVgyvSUPw1o5VHXVKZT-gUzuyQQdv6Ycy755BK1oIhTom2WWhRDVN6CY1FbDBxiNjlSLrZJaa2Rfpia1RllBuwM4A5kV2pPz-0tD9hMePfslJQe21CLQu70hi2Obb-doOE5ifO8P_Gffj0Ucczr8VJ2PbxCM-5v7SsirAsxqx8Hg8KhrGherxxXTrOcLi4wCrbHU4X-ue-5XmkbGKJBcCaA689RdOem9KLNxn7YU1rPnV3JGLGefhNhR81EjNkaIVjCjPoWBl0ekxXsWrkh1Jx5FEYLfWn6UK-uP1_DcxjWAhiRcd5v7anfYZz9li6ceCo9VAnby13mBsejx9wiQ4LkP8nJE4TR-v27IS_W_9r__-s5YCA
Monaco Editor Playground Code
const value = /* set from `myEditor.getModel()`: */ `function getUrls(url: string | URL, names: string[]) {
    if (typeof url === "string") {
        url = new URL(url);
    }

    return names.map(name => {
        url.searchParams.set("name", name)
        //  ~~~~~~~~~~~~
        // error!
        // Property 'searchParams' does not exist on type 'string | URL'.
        // this is fixed in TS 5.4 and shouldn't show an error anymore

        return url.toString();
    });
}
`;

// Hover on each property to see its docs!
const myEditor = monaco.editor.create(document.getElementById("container"), {
	value,
	language: "typescript",
	automaticLayout: true,
});

console.log(monaco.languages.typescript.typescriptVersion);

Reproduction Steps
No response
Actual (Problematic) Behavior

searchParams is still marked as an error although TS 5.4 fixed that behavior (https://devblogs.microsoft.com/typescript/announcing-typescript-5-4/#preserved-narrowing-in-closures-following-last-assignments)
the version logged is 5.0.2

Expected Behavior

no error marked in the code
the version logged is 5.4.x

Additional Context
After #4305 was merged I was under the impression that the next version of Monaco (i.e. 0.49.x) will include the new TS version automatically. Is that not the case?

I'm pretty sure it ships with 5.4, it's just that the default options don't raise the lib option to one that includes that prop: https://github.com/microsoft/monaco-editor/blob/a5e379c985d3263aacb1e811958c4663fea3db67/src/language/typescript/monaco.contribution.ts#L753
The TS playground overrides these options and so doesn't have this problem.

This is a) a compiler feature though, not a feature of an updated library and b) the console log reports 5.0.2 as the shipped version. The same playground code above works with both v0.49.0-dev-20240524 and v0.50.0-dev-20240530 where the error is gone and the console log reports 5.4.5. But it breaks/reverts back to 5.0.2 with both 0.49.0-rc and 0.49.0.

Oh, I didn't notice the console log 😞

@hediet @aiday-mar Did something go wrong in publishing? Looking at<EMAIL_ADDRESS>the published package definitely contains TS 5.0.
  var init_corePublic = __esm({
    "src/compiler/corePublic.ts"() {
      "use strict";
      versionMajorMinor = "5.0";
      version = "5.0.2";
      Comparison = /* @__PURE__ */ ((Comparison3) => {
        Comparison3[Comparison3["LessThan"] = -1] = "LessThan";
        Comparison3[Comparison3["EqualTo"] = 0] = "EqualTo";
        Comparison3[Comparison3["GreaterThan"] = 1] = "GreaterThan";
        return Comparison3;
      })(Comparison || {});
    }
  });

TS 5.4.5 ships now with 0.50.0


Docs for deadletter/expiry and message redelivery

Type of change

Documentation

Description
Docs for deadletter/expiry and message redelivery
Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

Release-0.33 branch was renamed to master: Please re-create PR against master branch.


Modifying user configuration in yaml and applying does not result in changes on the running server

First, thanks for the great project/product. The operator has immensely simplified and streamlined clickhouse deployments.
I am relatively new to the operator and configuration. I had added a user through configuration like so:
...
spec:
  configuration:
    users:
      admin/password: ****
      admin/network/ip:
        - "<IP_ADDRESS>/0"
      admin/access_management: 1
      admin/named_collection_control: 1
      admin/show_named_collections: 1
      admin/show_named_collections_secrets: 1
...

Applying the configuration returns "configured" as expected. However, I never see the clickhouse pod restart and it appears that the changes do not get applied. Deleting and recreating the Instance ends up with the user available and configured.
What is the expected behavior on reconfiguration that leads to file changes? And do I need to do something specific to have those file changes activated (like restarting the pod/clickhouse server(s))?
Apologies if I missed something in the documentation.

this is expected behavior, when changing users settings don't restart clickhouse-server
which clickhouse-server and clickhouse-operator versions do you use?
what return
grep admin -C 20 /etc/clickhouse-server/usersd.d/chop-generated-users.xml
restart rules defined in etc-clickhouse-operator-files configmap
look details here https://github.com/Altinity/clickhouse-operator/blob/master/config/config.yaml#L93


⚠ Bofrost.fr has degraded performance

In bf721af, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 707 ms

Resolved: Bofrost.fr performance has improved in 0a244a7.


Calendar Visual

http://community.powerbi.com/t5/Best-Visual-Contest/Calendar-Visual/cns-p/7249

Andy, is this PR still actual? Could you please update your code based on the latest changes in the repo?


Vmdk findbugs

these are two of the findbugs errors coming out of our jenkins all the time. The author seems to be unavailable so I kicked and screamed at it.
notice that this is noredist code. I did the compile without it. there is one with running.

note that jenkins isn't responding to the PR (so far)

Seems good to me looking at it. Don't know how to test it properly, but code-wise the changes look sane to me.

LGTM, did not test it.

ACS CI BVT Run
Sumarry:
Build Number 120
Hypervisor xenserver
NetworkType Advanced
Passed=106
Failed=12
Skipped=4
Link to logs Folder (search by build_no): https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
Failed tests:
Skipped tests:
test_vm_nic_adapter_vmxnet3
test_deploy_vgpu_enabled_vm
test_06_copy_template
test_06_copy_iso
Passed test suits:
integration.smoke.test_deploy_vm_with_userdata.TestDeployVmWithUserData
integration.smoke.test_affinity_groups_projects.TestDeployVmWithAffinityGroup
integration.smoke.test_portable_publicip.TestPortablePublicIPAcquire
integration.smoke.test_over_provisioning.TestUpdateOverProvision
integration.smoke.test_global_settings.TestUpdateConfigWithScope
integration.smoke.test_guest_vlan_range.TestDedicateGuestVlanRange
integration.smoke.test_scale_vm.TestScaleVm
integration.smoke.test_service_offerings.TestCreateServiceOffering
integration.smoke.test_loadbalance.TestLoadBalance
integration.smoke.test_routers.TestRouterServices
integration.smoke.test_reset_vm_on_reboot.TestResetVmOnReboot
integration.smoke.test_snapshots.TestSnapshotRootDisk
integration.smoke.test_deploy_vms_with_varied_deploymentplanners.TestDeployVmWithVariedPlanners
integration.smoke.test_network.TestDeleteAccount
integration.smoke.test_non_contigiousvlan.TestUpdatePhysicalNetwork
integration.smoke.test_deploy_vm_iso.TestDeployVMFromISO
integration.smoke.test_public_ip_range.TestDedicatePublicIPRange
integration.smoke.test_multipleips_per_nic.TestDeployVM
integration.smoke.test_regions.TestRegions
integration.smoke.test_affinity_groups.TestDeployVmWithAffinityGroup
integration.smoke.test_network_acl.TestNetworkACL
integration.smoke.test_pvlan.TestPVLAN
integration.smoke.test_volumes.TestCreateVolume
integration.smoke.test_ssvm.TestSSVMs
integration.smoke.test_nic.TestNic
integration.smoke.test_deploy_vm_root_resize.TestDeployVM
integration.smoke.test_resource_detail.TestResourceDetail
integration.smoke.test_secondary_storage.TestSecStorageServices
integration.smoke.test_vm_life_cycle.TestDeployVM
integration.smoke.test_disk_offerings.TestCreateDiskOffering

+1 for code changes and based on the test report from @bvbharatk.
Since the changes are specific to VMware and the test results are for XS, not sure how to take this forward.

@DaanHoogland can you rebase it once again and push -f
Once CI tested, we should be able to merge this cc @swill
tag:easypr

@rhtyd & @DaanHoogland  I don't have the ability to test this as I have a KVM setup, can one of you post some test results using VMware?

@DaanHoogland I'm seeing a lot of commits, can you rebase against 4.7?

@rhtyd you are right, I rebased against master instead of 4.7, sorry, will close and reopen against master


Consider ES2015 modules and using individual JS files from jsrsasign

Hi,
First of all thanks for a great library. I have trying to learn the concepts of OAuth and OIDC and this library comes very handy.
Currently, I tried using this in Angular-CLI project and checked that the application needs entire JS files from angular-oauth2-oidc (32kb) as well as jsrsasign (85kb). That's fairly big chunk to bring the first page as this library would be needed for authentication..
I see that we use quite minimal parts from JSRSAsign lib.
I checked if jsrsasign is ES2015 friendly.. but it appears that there are no immediate plans to move to ES2015 modules. But as the comments notes.. there are smaller bundles available for consumption.
I think using some libs from here should be sufficient for our usage.
Mostly.. we need below packages only.. (or may be even less)
Crypto
JWS
KeyUtil
Hope that can bring down needs of package size for jsrsasign library..
Similarly, I think right now library publishing has some issues leading to usage of entire library (umd.js). package.json specified module field as   "module": "angular-oauth2-oidc.js", but this js file is not present in package that was downloaded (v3.1.4) May be that's why entire library must get included currently. Fixing this should bring down size for angular-oauth2-oidc library in overall bundle.
Let me know if you need more information..

Thanks for this info. Lets consider this for a future release. Ofc, the bug with the main flag needs to be solved soon.
But I have a quick solution for you:

If you don't set the NullValidationHandler the CLI should tree shake away JSRSAsign. I have not tried it but it should be the case.

You can also create an custom ValidationHandler instead of this. This could use a slimmer library.

If you go this way I would be happy about a contribution, e. g. a pull request.
What do you think?

Sure I will give it a try

Just ran into this myself. Latest Rollup (0.56.3) does not tree shake jsrsasign away.

This is still an issue even with the NullValidationHandler

Anyone has an idea on what to do ?

I've made this here: https://github.com/manfredsteyer/angular-oauth2-oidc/pull/356
Which will fix these issues and reduce dependancies size down to a very tiny amount.
However it comes at the cost of breaking changes (if we go for window.crypto)
An alternative would be to find a lib that supports the same algorithms but with a synchronous API

If you are building with webpack and know that you do not use the functionality that is provided by jsrsasign, you can get rid of the dependency with the help of webpack (in my case webpack 4)
We created a mock for the two modules used from jsrsasign
jsrsasign.js
module.exports.KEYUTIL = {
    getKey: b => null,
};

module.exports.KJUR = {
    jws: {
        JWS: {
            verifyJWT: (...a) => null,
        },
    },
};

Now all you have to do is add the following to your webpack config
{
    resolve: {
        alias: {
            jsrsasign$: 'path/to/your/jsrsasign.js',
        }
    }
}

This will replace the dependency with your own file, reducing the bundle size drastically.
Just ensure that you are not using any functionality that relies on it.

any news for this issue ? Indeed it really heavy in my webpack analyze...

<EMAIL_ADDRESS>BUNDLE SIZE 269.6kB MINIFIED
jsrsasign ｜ 79.6% ｜ ~ 214.45 kB
(self) ｜ 18.4% ｜ ~ 49.56 kB

we are using the NullValidationhandler and with<EMAIL_ADDRESS>jsrsassign still gets its way into the production bundle.

The maintainer chose a different route to fix this in Version 9.x, which does require switching to Code+PKCE flow (the new recommendation for SPA's).
I will close this issue because I think the most honest current way of things is that for Implicit Flow with JWKS, or for versions <9.x, this will not be resolved.
To summarize, your optins for getting rid of jsrassign and its impact on your bundle size:

Use v9.x of the library with Implicit Flow but do not use the JwksValidationHandler and its package (that depends on the jsrasign)
Use v9.x of the library with Code Flow, that does not need JWKS at all


fix: Metrics Controllers Memory Leak

Fixes #3209
Description
New Karpenter pod memory profile is shown bellow. Below is the top 10 blocks of memory for the karpenter controller pod.
      flat  flat%   sum%        cum   cum%
 5633.31kB 21.72% 21.72% 12318.57kB 47.49%  encoding/json.(*decodeState).objectInterface
 4631.90kB 17.86% 39.58%  5149.23kB 19.85%  encoding/json.unquote (inline)
 2644.87kB 10.20% 49.77%  2644.87kB 10.20%  encoding/json.(*Decoder).refill
 1536.51kB  5.92% 55.70%  1536.51kB  5.92%  go.uber.org/zap/zapcore.newCounters (inline)
 1536.23kB  5.92% 61.62%  1536.23kB  5.92%  github.com/aws/aws-sdk-go/aws/endpoints.init
 1536.02kB  5.92% 67.54%  5149.22kB 19.85%  encoding/json.(*decodeState).literalInterface
 1097.69kB  4.23% 71.77%  2133.79kB  8.23%  k8s.io/apimachinery/pkg/runtime.(*Scheme).AddKnownTypeWithName
  600.58kB  2.32% 74.09%   600.58kB  2.32%  github.com/go-playground/validator/v10.init
  532.26kB  2.05% 76.14%   532.26kB  2.05%  github.com/gogo/protobuf/proto.RegisterType
  524.09kB  2.02% 78.16%   524.09kB  2.02%  k8s.io/apimachinery/pkg/conversion.ConversionFuncs.AddUntyped

After running pods for 10 hours, we see a sharpe increase in memory used by the karpenter pod. Below is the top 10 blocks of memory for the karpenter controller pod.
      flat  flat%   sum%        cum   cum%
   37.51MB 23.66% 23.66%    37.51MB 23.66%  github.com/aws/karpenter-core/pkg/controllers/metrics/provisioner.(*Controller).makeLabels (inline)
   22.51MB 14.20% 37.86%    25.51MB 16.09%  github.com/aws/karpenter-core/pkg/controllers/metrics/pod.(*Controller).makeLabels
   16.13MB 10.17% 48.03%    16.13MB 10.17%  strings.(*Builder).grow (inline)
   13.60MB  8.58% 56.61%    13.60MB  8.58%  fmt.Sprintf
   13.50MB  8.52% 65.13%    18.50MB 11.67%  github.com/aws/aws-sdk-go/private/protocol/xml/xmlutil.XMLToStruct
    4.50MB  2.84% 67.97%     4.50MB  2.84%  github.com/aws/aws-sdk-go/private/protocol/xml/xmlutil.(*XMLNode).findNamespaces (inline)
    4.50MB  2.84% 70.80%     5.50MB  3.47%  k8s.io/apimachinery/pkg/apis/meta/v1.(*ObjectMeta).Unmarshal
       4MB  2.52% 73.33%     5.50MB  3.47%  k8s.io/api/core/v1.(*PodSpec).Unmarshal
    3.57MB  2.25% 75.58%     3.57MB  2.25%  sync.(*Map).dirtyLocked
    3.50MB  2.21% 77.79%     3.50MB  2.21%  k8s.io/apimachinery/pkg/util/sets.String.Insert (inline)

We can see that there is lots of data being stored in both metrics provisioner/pod controllers ware there should not be 37.51MB and 25.51MB should be stored respectively. This suggest a memory leak at the location.
github.com/aws/karpenter-core/pkg/controllers/metrics/provisioner.(*Controller).makeLabels
github.com/aws/karpenter-core/pkg/controllers/metrics/pod.(*Controller).makeLabels
How was this change tested?
In a 10 hour period, we see with node/pod churn the memory usage of Karpenter grows in as a factor of time suggest a memory leak.

We also see the memory profile of the karpenter pod,
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 8 (100.0%)  changed or added relevant lines in 2 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 80.605%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/deprovisioning/controller.go
3
77.4%

pkg/controllers/deprovisioning/events/events.go
9
67.21%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
6550

Relevant Lines:
8126

💛  - Coveralls

The Current provisioner/pod metrics dashboard:

The fixed provisioner/pod metrics dashboard:


Tuple must contain at least two elements.

After I referenced pakage, unity reported an error and I couldn't change the code. What do I need to do?Error reported as follows
Library\PackageCache\ai.natml.natml@1.1.16\Runtime\API\Services\Predictor.cs(151,18): error CS8124: Tuple must contain at least two elements.

Hey @damonsmart what version of Unity are you using?

i uesd 2020.3.17

VideoKit requires Unity 2022 LTS or newer. The lowest you can use is 2021.3+.


periodic refresh of api_spec.json

Per @kislyuk , planning on pulling down full swagger spec whenever a release is made. After python bindings are complete, will also refresh python wrapper api whenever releasing. @ttung lmk what you think.

Any time Python bindings are generated, they're pulled down from the hosted api spec. My guess would be that the best way to automate this in the future is to trigger a rebuild and release any time the blue box api spec is updated.


Page crashes when trying to open topo/geojson

I have a geo/topojson file with 18000 polygons, and it is 12Mb large.
I can open the file using mapshaper, arcgis and qgis, but geojson.io always crashes.
Any help is appreciated.
Thanks!

Hello!
Unfortunately this isn't likely to work: geojson.io is a client-side application, limited by the performance of JavaScript and all of its component libraries. Most GeoJSON data is relatively small - smaller than 12mb/18k features, and it works very well for those datasets, but larger datasets are expected to crash. Since this varies with different computers, we don't have a hard limit for how much data it can handle, but this dataset is certainly hitting the limit.

Tom

Hi Tom,
Thank You for the assistance. I'll try to break up the file into few geojson files, and see what happens.
Thanks again!
Djordje


correct the variable in Literal example

Good catch!

Hi @Mearman
In order to merge this PR, we need you to sign the Neo4j CLA. You have more information, document, and steps to sign here: https://neo4j.com/developer/cla/
If you agree to it, please let me know when you have signed 🙂
(The automated message from before is still a work in progress)

Done :)

Thanks @Mearman


Merge H3 integration

#4724

This library is extraordinary slow: less than a million invocations per second:
  11.14%  libm-2.27.so                                    [.] __lroundl
  10.28%  libm-2.27.so                                    [.] __sincos
   8.07%  clickhouse                                      [.] _ijkNormalize
   7.90%  clickhouse                                      [.] _geoToHex2d
   6.71%  clickhouse                                      [.] _faceIjkToH3
   5.68%  clickhouse                                      [.] _upAp7r
   4.51%  clickhouse                                      [.] _unitIjkToDigit
   4.13%  clickhouse                                      [.] _upAp7
   2.24%  [kernel]                                        [k] native_irq_return_iret
   1.92%  clickhouse                                      [.] _hex2dToCoordIJK
   1.59%  clickhouse                                      [.] _downAp7r
   1.55%  libm-2.27.so                                    [.] __ieee754_acos_sse2
   1.25%  [kernel]                                        [k] entry_SYSCALL_64_after_swapgs
   1.16%  clickhouse                                      [.] _downAp7
   1.09%  [kernel]                                        [k] __sigqueue_free
   1.03%  clickhouse                                      [.] _ZN2DB15FunctionGeoToH311executeImplERNS_5BlockERKSt6vectorImSaImEEmm
   1.02%  clickhouse                                      [.] _posAngleRads

I've noticed that the order of coordinates was not consistent with existing functions. I have to change it.
https://stackoverflow.com/questions/7309121/preferred-order-of-writing-latitude-longitude-tuples


Settings revamp

Is your feature request related to a problem? Please describe.
Currently, settings and options are scattered allover the place.
Describe the solution you'd like
Moving towards a more streamlined approach that is very popular.
Settings are split into categories on the left, and the one, you open, shows up on the right-side.

Additional context
The demo provided is just to showcase my idea, the categories are up to you.

Currently, settings and options are scattered allover the place.

No they're not :-)  I don't think, anyway :-P  The options for languages belong with them.  The options for terms aren't actually options, we discussed it and you'd said that these were the import options, but those need to be specified for each import, not in a one-time setting.  But let me know if I've missed something else!
I actually wasn't sure where to put the "Languages" -- so I chose to put it under settings, just because
I feel that this request is really about splitting up the current giant settings page into smaller pages, is that correct?

I'm not sure that splitting it up gains much ... if I were a new user, I'd have to click through each link to see what's what, eg for appearance and behaviour.  I can see benefit in splitting out "backup" and "japanese" though, b/c they're kind of different -- so:

behaviour
backup
japanese

(If this issue is about changing the menu structure, as implied in the discord conversation, that's a separate issue ... but I'm not inclined to tinker too much with it at the moment as the current stuff works even if it's not great. :-) )

What about putting all the Backup stuff under the backup category in settings, viewing, downloading, and creating backups are under a separate thingie.

Re "Backup stuff under the backup category in settings" - do you mean remove the backup menu item?
The original intent of this ticket was to clarify the settings only.  It appears that there should be three separate settings "tabs": backup, settings, and Japanese.
Actually -- I don't know why I just thought of this now -- those Japanese settings would only be present on the Japanese language page ... that should be a separate issue.
But back to the original issue: "Currently, settings and options are scattered all over the place." -- I still don't understand this statement, and perhaps am missing the point.  The issue and discord discussions appear to combine a few ideas.  I currently don't feel a pressing need to make changes -- things may be clunky but they work -- but I probably have blinders on given my "creator's bias".  I feel we could spend a fair amount of time talking about menu item placement, but at this point I'd rather not focus on that, given the existing backlog and functional things that are missing. :-)  It might be faster to hash this out on Discord instead of here.
Happy to get more feedback and wait for some clearer requests -- but of course everything goes into the backlog to get sorted out as it can be done.  Thanks!

These are some notes, to 'narrow' the scope of the issue, and make it more clear
This issue is concerned with UI/UX improvement based on my own preferences, so it's a low priority anyway
My Issue could be broken into two steps (if you want, I can split the second one to a separate issue)
Step one: (intermediate priority)

Splitting the current settings page into sub-categories, with a static panel on the lift, and the selected item rendered on the left. As shown in the illustration.

Behavior

Appearance

Backup

Japanese

Step two: (low priority, but better UX IMO)

Adding more categories to the settings panel, to make settings aligned with most modern settings panels.
The next are some suggestions.

Behavior (move the backup list, and creating a manual backup here)

Appearance

Backup

Japanese

Languages (eliminate the submenu under settings on home screen, and make clicking on settings just takes the user to the settings page)

About (Version, Docs, Contact info) -statistics, shouldn't be under the about section.

I'm liking your idea @shetozz. Only thing I would suggest is changing 'Japanese' to 'Parsers' or something more generic for when other parsers are added. I was thinking about consolidating all the 'About' stuff into one menu, but it's even better as another section in the settings.


Add more error handling

In some cases of errors it’s not clear what happened. To bring more details to fails it’s necessary to add handling of retrieved parameters and handle some negative cases in framework

done in f02fcbe


[Feature Request] Add support for tagging part-of-speech

The jieba-rs support tagging part-of-speech by tag function, could you export it to this library?

Can you give me some examples to show the use cases of this tag function?

Can you give me some examples to show the use cases of this tag function?

Such as searching and language processing, part-of-speech tagging is significant in these scenes.
With part-of-speech, it is possible to do different post-processing according to the POS in language processing. POS could also provide better searching accuracy as nouns are much more related to the theme than adjectives, adverbs, and other POSs.

@KiritaniAyaka Could you try this pr and check if it is what you want? https://github.com/fengkx/jieba-wasm/pull/12#issuecomment-2401626172

@fengkx
Yep, and it works great. Thank you for your efforts. 😆


Typo in the Demo Example

There was a typo in the example code of this section:
https://github.com/OneDrive/onedrive-sdk-android/blob/master/docs/items.md#delete-an-item
On the second line, a pair of parentheses are missing. Instead of
final ICallback<Item> callback = new ICallback<Item> {
It should be
final ICallback<Item> callback = new ICallback<Item>() {
Thank you :)

Also, it seems that it needs to be <Void> instead of <Item> to get this code compile.


Add lifecycle hooks to run middleware

At the moment, middleware can only intercept and modify outgoing requests in capture mode. This is inadequate if someone wants to mask sensitive information in both request and response when capturing.

It makes sense to extend Hoverfly as there are probably other use-cases, but having to do masking  in middleware seems like a leaky solution for larger teams as it has to be applied in real-time. Developers will forget and policy would be difficult to apply.
We could do with more use-cases on the general topic of masking.

After reading this GitHub issue: https://github.com/arquillian/arquillian-organization/issues/10 I get a feeling that it is not necessary a middleware feature
Problem
Request/response pair could contain sensitive data from capturing live endpoints. Storing them as plain text in Github is a bad idea. Manually removing them is error-prone and time-consuming.
Use case
The original use case from @lordofthejars goes like this:

In Capture mode, hoverfly should be able to send the original request and receive the original response, but the exported simulation data should have the sensitive data masked.
Switching to simulate mode and using the modified data shouldn't break the original tests.

Implications

One should be able to define which fields in the request and response to obfuscate
Data masking should be done before a request/response pair is stored.
If data masking is performed by middleware, Capture mode should support middleware execution similar to  Modify mode, but also be able to preserve the original request when sending to target service.
Sensitive data in the request can be replaced by GlobMatcher
Sensitive data in the response can be encrypted (but requires decryption in Simulate mode, and encryption key from the user) or replaced by random value (it might break the tests that assert an exact value on this field in Simulate mode)

The change of title reflects that we won't be adding explicit support for data masking in Hoverfly itself. Lifecycle hooks - yes. Data masking - no.

The current middleware does not support request modification in simulate mode, and not support response modification in the capture mode. The way it works is probably to tackle problems such as:

A delay middleware is only intended to be used for SIMULATE mode, and switching to CAPTURE mode should not introduce any latency.
A middleware which modifies requests is supposed to be used for CAPTURE mode, and you don't want to apply it again when Hoverfly is switched to SIMULATE mode, otherwise the requests could be modified twice.

Making middleware to apply on both request and response for simulate/capture/spy mode is desirable for some use cases but has the potential to break backward compatibility.

What I propose is that middleware should behave as it is, but you can pass an additional flag to force it to run at a certain point regardless of the mode. For example,
$ hoverctl middleware --binary python --script intercept_requests.py --hooks "PRE_REQUEST_HANDLE"

Allow middleware to be applied before request matching in simulate mode.
Available hooks could be:

PRE_REQUEST_HANDLE   (before hoverfly processes the requests, i.e. before matching in simulate mode, or before saving the requests in capture mode)
POST_REQUEST_HANDLE (after the request is saved in capture mode, but before it's forwarded to the destination.)
PRE_RESPONSE_HANDLE (before hoverfly saved the response for capture mode)
POST_RESPONSE_HANDLE (after hoverfly saved the response for capture mode)

Looks good to me, except you've got PRE_REQUEST_HANDLE, etc when I assume you meant PRE_REQUEST_HOOK.
Two questions:

Can the same middleware run multiple times if more than one hook is specified?
Can the middleware get or derive any context information to tell it what sort of hook it is?

I assume the answer is No to both these questions, but it's worth checking.

The suffix_HANDLE in the name just provides clarity that the hook is before/after Hoverfly has processed the request/response.
For question 1, yes. You can pass comma seperated list of hooks, and invoke the same middleware at different hooks.
For question 2, maybe no initially, but it's possible to pass context information into the middleware.

Ok. Q1 is good. but wrt Q2. I think that then compromises the feature.
What if the lifecycle stage could be passed in metadata as that shouldn't affect existing middleware, or to be 100% sure, pass it if --hooks is stated?

Currently, we do not need this feature anymore because we bypassed this information into fake information manually. Of course, it might still be interesting to have this feature but let's say that it is not urgent for our use case.

@tommysitu if we need to build this feature, then I would like to take this up. You can assign this to me. I can start working on the same.

@tommysitu  you can assign this to me. I have started working on the same.


MINIFI-357 fixing PutFile bug that caused all writes to fail

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Thanks for catching this. I or someone else will work on https://issues.apache.org/jira/browse/MINIFI-359 to help avoid this in future changes.


Add KPTx token to superfluid assets

@msoni89 I just noticed that there's a typo, every file is KPR instead of KPT
please can you make changes? KPT = Kitty Party Token, not sure how R got in there


CPv3- Add Access Control to Smart Contract

Solves Issue #42

Addonly_owner, grant_minter_role, revoke_minter_role, grant_offsetter_role, revoke_offsetter_role in contracts/project.cairo
Add the OpenZeppelin RBAC Component to the Project contract
Add Owner validation to set_absorptions and set_project_carbon in components/absorber/carbon_handler.cairo
Add Owner validation to set_public_sale_open, set_unit_price, set_min_money_amount_per_tx in components/minter/mint.cairo
Add Minter validation to mint and batch_mint in contracts/project.cairo
Add Offsetter validation to offset and batch_offset in contracts/project.cairo
Refactor existing tests to pass with the RBAC Component
Add tests to ensure proper behaviour of the modified functions with the RBAC Component

Thank you for the feedback. I'll start working on the suggested changes ASAP and will mention you once everything is modified.

@julienbrs all the requested changes have been addressed.
Please review again and let me know if anything else is necessary or if I skipped something.


Wrong reference

Build Process
Describe how you build polybar, list the exact commands you are using:
git clone --recursive https://github.com/polybar/polybar
If you build polybar directly from this repository:

Output of git describe --tags:

3.4.0-55-g85721d3
List exactly where you are building from and which version you are building.
my home folder; git (commit 85721d3d109e29decc3fc1d4185ae63d5afa20d6)
Build Log
Post everything that is output to the terminal while building polybar below. This HAS to include the output of the cmake and make commands, if you used them.
Cloning into 'polybar'...
remote: Enumerating objects: 16, done.
remote: Counting objects: 100% (16/16), done.
remote: Compressing objects: 100% (14/14), done.
remote: Total 14170 (delta 2), reused 7 (delta 0), pack-reused 14154
Receiving objects: 100% (14170/14170), 7.28 MiB | 6.62 MiB/s, done.
Resolving deltas: 100% (10514/10514), done.
Submodule 'lib/i3ipcpp' (https://github.com/polybar/i3ipcpp) registered for path 'lib/i3ipcpp'
Submodule 'lib/xpp' (https://github.com/polybar/xpp) registered for path 'lib/xpp'
Cloning into '/home/us/polybar/lib/i3ipcpp'...
remote: Enumerating objects: 762, done.
remote: Total 762 (delta 0), reused 0 (delta 0), pack-reused 762
Receiving objects: 100% (762/762), 299.42 KiB | 985.00 KiB/s, done.
Resolving deltas: 100% (243/243), done.
Cloning into '/home/us/polybar/lib/xpp'...
remote: Enumerating objects: 53, done.
remote: Counting objects: 100% (53/53), done.
remote: Compressing objects: 100% (35/35), done.
remote: Total 2660 (delta 28), reused 22 (delta 17), pack-reused 2607
Receiving objects: 100% (2660/2660), 494.69 KiB | 1.26 MiB/s, done.
Resolving deltas: 100% (1843/1843), done.
Submodule path 'lib/i3ipcpp': checked out '21ce9060ac7c502225fdbd2f200b1cbdd8eca08d'
remote: Enumerating objects: 1, done.
remote: Counting objects: 100% (1/1), done.
remote: Total 1 (delta 0), reused 1 (delta 0), pack-reused 0
Unpacking objects: 100% (1/1), done.
From https://github.com/polybar/xpp
 * branch            f70dd8bb50944ab64ab902d9d8ab555599249dc1 -> FETCH_HEAD
Submodule path 'lib/xpp': checked out 'f70dd8bb50944ab64ab902d9d8ab555599249dc1'

Environment:

Distro (with Version):

Gentoo (Head commit of repository gentoo: f73c1995cbc9ba20295b2d6c47107620ff81ae85)
Additional context
Add any other context that you think is necessary about the problem here.
Technically this won't stop anybody from building polybar form git... but it fails while used in an ebuild on Gentoo (it fails to fetch the referenced repo)

I am surely missing something but I can't spot the error you are reporting.

I am surely missing something but I can't spot the error you are reporting.

From https://github.com/polybar/xpp
 * branch            f70dd8bb50944ab64ab902d9d8ab555599249dc1 -> FETCH_HEAD
Submodule path 'lib/xpp': checked out 'f70dd8bb50944ab64ab902d9d8ab555599249dc1'

f70dd8bb50944ab64ab902d9d8ab555599249dc1 doesn't exist

Strange that git was even able to check it out then. What is inside the lib/xpp folder when this happens?

That's strange because, this hash doesn't exist in my repo but it exists on github.
On my repo, the hash is 8c019e6d7fefd2468791bc1cbf90d68ff7c1ba33

Okay, I just found the problem, this hash is indeed the wrong one, this hash refers to the hash of @jeremyong but not to the hash of our repo. We should update that.

i went to lib/xpp and did git checkout master - it reverts to "8c019e6d7fefd2468791bc1cbf90d68ff7c1ba33 (HEAD -> master, origin/master, origin/HEAD)"
before that it says "f70dd8bb50944ab64ab902d9d8ab555599249dc1 (HEAD)"

Okay, I just found the problem, this hash is indeed the wrong one, this hash refers to the hash of @jeremyong but not to the hash of our repo. We should update that.

Thanks for cooperation

Yup that was my fault (should have paid more attention to the .gitmodule file). Thanks for addressing it and sorry :(

No problems ;)


Improve doc comment style/approach

This PR includes change from PR #137 (to help find invalid doc comments / prove we're making progress). Please review that PR first.
Currently, the rbd package is the "messiest" with copy-n-pastes of the c function prototype as many of the doc comments. Instead I went with the following formats:
// FuncName does something.
func FuncName() {
}

For code that does not act as a wrapper around a C function, didn't need a reference to a c function or simply doesn't need the extra detail.
and
// FuncName does something.
//
// Implements:
//  int rbd_foo_bar(int i, uint64_t bloop);
func FuncName(i int, bloop uint64) {
}

For code that references a particular function from the c api. I removed unused / invalid references if they existed.
This has the nice side-effect of both being searchable from a Go code POV and a C API POV (search for the c-function name in page). It also formats decently in the "godoc" style.
This is not comprehensive but fairly representative at this point. If we like this style, I'll continue updating the files. Otherwise we can bikeshed it a bit and then I'll go back and update this changes. I just don't want to go too far yet. :-)

Aside: I have some plans regarding a tool to compare the api from ceph with our code. The "Implmements:" line can also help act as a keyword for this tool I'm thinking of to help mark explicit coverage vs. implicit coverage. This isn't super relevant until I put out a prototype of what I'm thinking of bit I figured I would throw it out here anyway as encouragement to accept the style I'm proposing. :-)

@nixpanic lemme know if you agree with my responses. I don't want to sit on this one too long as this kind of churn creates a lot of busy work conflicts :-)


Use a streamlined way to trigger component dismiss

Till this MR:

plugins that support dismiss action by clicking on a data-bs-dismiss trigger, can be dismissed ONLY if the trigger is inside the plugin html.

Alert, makes the difference.  Alert on  data-bs-dismiss click,  tries to parse a data-bs-target element and in case in null, it fallback on the standard way, that assumes the trigger is inside the plugin html

This MR:
introduces the enableDismissTrigger helper function, that streamlines the functionality, using the idea of Alert component.
When 'dismiss' is clicked, it searches for getElementFromSelector(event.target) || this.closest(.${Plugin.NAME}) .
Result:

The dismiss trigger can be either inside the component, or with the use of data-bs-target it can be  anywhere in doc
the dismiss trigger will respect the disable state and will handle preventDefault in case it is an  anchor

The idea came from #33324
closes #33324
Can replace #33403

How about you define enableDismissTrigger on the base-component? That way you should be able to use it like Plugin.enableDismissTrigger(callback).

I don't know if it is ok to use it as static, as it will leave open the possibility to be called outside the code, causing fault usages

https://github.com/twbs/bootstrap/blob/34218dab70d07a0421e4492620329011aafdabb6/js/src/modal.js#L144-L146
do we still need this here?

As we still support anchors with href triggers, I think yes.

https://github.com/twbs/bootstrap/blob/34218dab70d07a0421e4492620329011aafdabb6/js/src/alert.js#L49
https://github.com/twbs/bootstrap/blob/34218dab70d07a0421e4492620329011aafdabb6/js/src/alert.js#L61-L63
This is covered by enableDismissTrigger too, isn't it?

Is better to merge #33402 before this MR, because existing alert code, does some 'magic',  and makes me nervous, using other code
After #33402 is merged, we won't have to solve this riddle

https://github.com/twbs/bootstrap/blob/34218dab70d07a0421e4492620329011aafdabb6/js/src/modal.js#L144-L146
do we still need this here?

As we still support anchors with href triggers, I think yes.

But this is done by enableDismissTrigger already?

coy
On Wed, Jun 2, 2021 at 9:03 PM GeoSot @.***> wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/twbs/bootstrap/pull/34170
Commit Summary

use a streamlined way to trigger component dismiss

File Changes

M js/src/alert.js
https://github.com/twbs/bootstrap/pull/34170/files#diff-8ef356b85a832c9ab47b2d778e7c4729aa0f4f62cc6a2bc337d4099863700f2b
(18)
M js/src/modal.js
https://github.com/twbs/bootstrap/pull/34170/files#diff-e5510179795c61da7183e9dda93b479d7b1ab9c7d261b57c46b92e3d94745c66
(6)
M js/src/offcanvas.js
https://github.com/twbs/bootstrap/pull/34170/files#diff-b2b03351168f5f589b5bb6a52667bae837fc0956b6afbc32bf9d474050d48eb7
(6)
M js/src/toast.js
https://github.com/twbs/bootstrap/pull/34170/files#diff-46174b4a0fa36e17acefc0510870537f7358857c875e8e6b0e31cff6edd824a6
(7)
A js/src/util/plugin-functions.js
https://github.com/twbs/bootstrap/pull/34170/files#diff-6bc9afeb916ad527a4f8b6061f66e991e61307372d690d50442d467e4a3dddd3
(32)

Patch Links:

https://github.com/twbs/bootstrap/pull/34170.patch
https://github.com/twbs/bootstrap/pull/34170.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/pull/34170, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHORFC5HZDF7FRAA7MBHBC3TQY23JANCNFSM456ZWHAQ
.


Make googlefonts optional

Thanks for this great tool!
Is it possible to add an option to disable google font? There are some places in the world hard to access google service :crying_cat_face:

I created a custom index.html.erb to overwrite the default one.

Yes, that works. Generally we would welcome some refactoring of that ERB into helpers and what not to make it easier to customize.


add code that violates eslint rules

This will check if branch protections and CI for repository is working correctly.

Successfully tests actions and permissions set up for the main branch.


Docker registry token defaults to "A"

brew doctor output
bash-3.2$ brew doctor
Your system is ready to brew.

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
bash-3.2$ brew config
HOMEBREW_VERSION: >=4.1.0 (shallow or no git repository)
ORIGIN: (none)
HEAD: (none)
Last commit: never
Core tap JSON: 14 Feb 20:32 UTC
Core cask tap JSON: 14 Feb 20:33 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_ARTIFACT_DOMAIN: https://myrepository.com/artifactory/homebrew-remote
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 8
HOMEBREW_NO_AUTO_UPDATE: set
Homebrew Ruby: 3.1.4 => /opt/homebrew/Library/Homebrew/vendor/portable-ruby/3.1.4/bin/ruby
CPU: octa-core 64-bit arm_firestorm_icestorm
Clang: 15.0.0 build 1500
Git: 2.43.2 => /opt/homebrew/bin/git
Curl: 8.1.2 => /usr/bin/curl
macOS: 13.6-arm64
CLT: <IP_ADDRESS>.1.1700200546
Xcode: 15.2
Rosetta 2: false

What were you trying to do (and why)?
Connect to a private repository using anonymous access
What happened (include all command output)?
The default Authorization: Bearer token sent is QQ== ("A"). As a result anonymous access fails.

bash-3.2$ brew install openjdk@17 --verbose
==> Downloading https://myrepository.com/artifactory/homebrew-remote/v2/homebrew/core/openjdk/17/manifests/17.0.9
Already downloaded<EMAIL_ADDRESS>==> Fetching openjdk@17
==> Downloading https://myrepository.com/artifactory/homebrew-remote/v2/homebrew/core/openjdk/17/blobs/sha256:41eda5749bf9032865a1168a0306a5b089f3098433dc1b95eebe82bedb98f023
/usr/bin/env /opt/homebrew/Library/Homebrew/shims/shared/curl --disable --cookie /dev/null --globoff --show-error --user-agent Homebrew/4.X.Y\ (Macintosh;\ arm64\ Mac\ OS\ X\ 13.6)\ curl/8.1.2 --header Accept-Language:\ en --fail --retry 3 --header Authorization:\ Bearer\ QQ== --remote-time --output<EMAIL_ADDRESS>--location https://myrepository.com/artifactory/homebrew-remote/v2/homebrew/core/openjdk/17/blobs/sha256:41eda5749bf9032865a1168a0306a5b089f3098433dc1b95eebe82bedb98f023
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0   101    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 401
Error: openjdk@17: Failed to download resource "openjdk@17"
Download failed: https://myrepository.com/artifactory/homebrew-remote/v2/homebrew/core/openjdk/17/blobs/sha256:41eda5749bf9032865a1168a0306a5b089f3098433dc1b95eebe82bedb98f023

What did you expect to happen?
The request does not attempt to send an auth header because there is no token setup. Why is the default token "A"?
Step-by-step reproduction instructions (by running brew commands)
1. Configure brew to use a private repository using HOMEBREW_ARTIFACT_DOMAIN
2. Attempt to install a package
3. The install fails since an invalid auth header is sent

Why is the default token "A"?

Because GitHub Packages is the default registry and it requires an authentication header (even though the value is bogus).
It can be customised with HOMEBREW_DOCKER_REGISTRY_TOKEN however (or HOMEBREW_DOCKER_REGISTRY_BASIC_AUTH_TOKEN for Basic auth instead of Bearer)

Unfortunately it seems those environment variables are not respected if set to nothing:

bash-3.2$ export HOMEBREW_DOCKER_REGISTRY_TOKEN=
bash-3.2$ export HOMEBREW_DOCKER_REGISTRY_BASIC_AUTH_TOKEN=
bash-3.2$ env | sort
. . .
HOME=/Users/homebrew
HOMEBREW_DOCKER_REGISTRY_BASIC_AUTH_TOKEN=
HOMEBREW_DOCKER_REGISTRY_TOKEN=
. . .
bash-3.2$ brew install openjdk@17 --verbose
==> Downloading https://myrepository.com/artifactory/homebrew-remote/v2/homebrew/core/openjdk/17/manifests/17.0.9
/usr/bin/env /opt/homebrew/Library/Homebrew/shims/shared/curl --disable --cookie /dev/null --globoff --show-error --user-agent Homebrew/4.X.Y\ (Macintosh;\ arm64\ Mac\ OS\ X\ 13.6)\ curl/8.1.2 --header Accept-Language:\ en --fail --retry 3 --header Accept:\ application/vnd.oci.image.index.v1+json --header Authorization:\ Bearer\ QQ== --remote-time --output<EMAIL_ADDRESS>--location https://myrepository.com/artifactory/homebrew-remote/v2/homebrew/core/openjdk/17/manifests/17.0.9
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0   101    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 401

Unfortunately it seems those environment variables are not respected if set to nothing:

Correct.
To ensure we've understood correctly: you have a private Artifactory but it does not have any authentication?

There is authentication but we allow anonymous access on repository mirrors (remote repositories) on our private Artifactory instance. Hence homebrew-remote which mirrors ghcr.io does not require authentication.

We'll review a PR to fix this.

A potential quick fix for this might be to set HOMEBREW_GITHUB_PACKAGES_AUTH="Bearer QQ==" only when both:

HOMEBREW_ARTIFACT_DOMAIN isn't set
HOMEBREW_BOTTLE_DOMAIN isn't OR is set to the defaults (`` and https://ghcr.io/v2/homebrew/core respectively)

https://github.com/Homebrew/brew/blob/a570e4da19131eb9e7263f392e863cbaf3325884/Library/Homebrew/brew.sh#L955-L963
One blind spot: Is sending QQ== something unique to ghcr.io, or does any Docker registry require a token and the clients just know to use this Base64'd A if otherwise unset?

Is sending QQ== something unique to ghcr.io

I think this is the case.

A potential quick fix

@colindean can you open a PR? Thanks.

Passing on this for now in favour of a PR.

@MikeMcQuaid do you have a link to the PR?

I think he means someone should raise a PR, not that one already exists.

Exactly, thanks @gromgit.

If that's the case why close this? It's still an issue.

@MarkRx It's unclear anyone cares about it except you. Homebrew has tens of millions of users and we can't keep them all open indefinitely or our issue tracker becomes unusable, sorry. We will happily review a PR if you create one to fix your own problem.


ng-token-auth is not storing the cookie on sign in

I dont know what's happening, the app make the request to the api and this return the user data properly, but the cookies to store the data aren't created. I tried running the app in chrome and firefox but doesn't work. Here's my code:  https://gist.github.com/jsvelasquezv/1284ec388355cc9ff7ba

I solve my problem with this http://stackoverflow.com/questions/30145395/ng-token-auth-not-persisting


Dataset preprocessing

Dear authors,
Thanks for the amazing work! May I ask if the dataset preprocessing codes are included in the repo? I dived into the codes and didn't find one.
Thanks,
Yuanqi

Hi Yuanqi,
Thank you for the kind words and interest! You can find most of the preprocessing scripts in data_scripts/ (as described in README). The GNPS dataset was extracted from the CANOPUS paper more or less in the format that we link to re-download. NIST was parsed as described here. Let me know if there's anything else you're looking for.
Best,
Sam

Thanks so much for your prompt response! Just another quick question, for GNPS, I believe you also filtered 1,500 Da and adduct types. Just double check if the downloaded GNPS from the link already got filtered or it was explicitly filtered out in the code?
Thanks again!

Should already be filtered!

Thanks!

Dear Sam, sorry for bother you again, I downloaded NIST following this repo https://github.com/Roestlab/massformer, but your repo (https://github.com/samgoldman97/nist-parser) seems to read the data from the NIST SDF file, may I know how the SDF is obtained from the original source?
Thanks again!

Hi Yuanqi,
If I recall correctly, the SDF was one of the files i was able to extract using the Massformer's instructions directly-- I don't recall taking any additional steps beyond what was described in MassFormer. I used a windows machine to extract it not currently accessible. If you're not able to do this, I would recommend parsing the MSP directly.
Sam

Thanks for your prompt reply! I will play with their code and see how to bridge the two!

I managed to process from the MSP to obtain the data, just two quick questions: (1) for PRECURSOR M/Z, is this the mass for the parent molecule or parent molecule + ion mass? (2) is it possible to include [M-H]- as one adduct type?
For the second question, I feel it is possible to just extend the list where it is used, correct me if I were wrong!
Thanks again for your help and great work!

also spot one thing ms-pred/src/ms_pred/common/chem_utils.py, ion2mass, should it be minus the mass of hydrogen since its minus H2O together
"[M-H2O+H]+": -ELEMENT_TO_MASS["O"] + ELEMENT_TO_MASS["H"] - ELECTRON_MASS,
"[M+H-H2O]+": -ELEMENT_TO_MASS["O"] + ELEMENT_TO_MASS["H"] - ELECTRON_MASS,

Thanks for the catch-- will leave this as is for now with a comment in the readme and make sure to fix as part of revision process (I don't want to mess up pre-trained models by shifting this now)

Glad to help! Sorry if this question is missed:
I managed to process from the MSP to obtain the data, just two quick questions: (1) for PRECURSOR M/Z, is this the mass for the parent molecule or parent molecule + ion mass? (2) is it possible to include [M-H]- as one adduct type?
For the second question, I feel it is possible to just extend the list where it is used, correct me if I were wrong!
Thanks again for your help and great work!

No problem to help at all. Also, feel free to email me rather than going back and forth on this issue-- I'm happy to contribute in a more active guidance role or help you think through next steps, as we are not planning to actively develop new forward models outside of a simple extension of adding a few additional covariates into our model (instrument type + collision energy, etc.).
Re. precursor M/Z, usually this refers to the observed mass over charge at the MS1 level, which is the parent molecule mass + ion mass. I would do a few manual checks for yourself (you can use the utility functions chem_utils if you already have the package common.mass_from_smi(SMILES) + common.ion2mass[ADDUCT]). One thing to look out for is that in some of these standards libraries like NIST, they give you theoretical mass rather than observed mass with error. This is less relevant for forward prediction, but matters for something like learning to predict chem formula from spectra (https://www.nature.com/articles/s41592-023-01850-x).
Lastly, regarding negative mode-- this will likely work if you want negative mode spectra as well. There may be some nuances to how molecules fragment, but since it's a learned model anyways it may not matter.

Thanks so much for the detailed explanations! Yeah, I will send an email and happy to learn more from you!

Corrected ion2mass in latest runs / pull for camera ready! Thanks again for flagging.


[BUG]: "repctl cluster inject --use-sa" doesn't work for Kubernetes 1.24 and above

Bug Description
Since Kubernetes 1.24, the default service account won't be generated automatically, unless feature gate LegacyServiceAccountTokenNoAutoGeneration is disabled. https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.24.md
Because of this change, "repctl cluster inject --use-sa" doesn't work for Kubernetes 1.24 and later version.
Logs
[root@k8s-n1 csm-replication]# repctl cluster inject --use-sa
[2022-09-06 21:11:16]  INFO Generating config maps from existing service accounts
[2022-09-06 21:11:16]  INFO Generating config for service account in site-n1 cluster
Compiling kubeconfig for "default"                [  OK  ]
Extracting secret                                 [  OK  ]
Extracting token                                  Error from server (NotFound): secrets "" not found
[ FAIL ]
[2022-09-06 21:11:16]  INFO
[2022-09-06 21:11:16]  INFO Generating config for service account in site-n2 cluster
Compiling kubeconfig for "default"                [  OK  ]
Extracting secret                                 [  OK  ]
Extracting token                                  Error from server (NotFound): secrets "" not found
[ FAIL ]
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
Run "repctl cluster inject --use-sa" on Kubernetes 1.24 or later version.
Expected Behavior
"repctl cluster inject --use-sa" still able to process.
CSM Driver(s)
CSM 1.3
Installation Type
No response
Container Storage Modules Enabled
csm-replication
Container Orchestrator
Kubernetes 1.24
Operating System
CentOS7

@team,
The request is to :

Update the documentation to create a proper account
Patch to repctl cluster inject --use-sa to self generate the accounts

We can not ask the user to change a feature gate.

In the upcoming release, installing csm-replication on the clusters using the installation script will solve the ./repctl cluster inject --use-sa issue.
In a future release, the above issue will be addressed via the installation of the replication controller using repctl path (step 3).


Download Error

Checklist

[ ] I'm asking a question
[ ] I've looked through the README and FAQ for similar questions
[ ] I've searched the bugtracker for similar questions including closed ones

Question

WRITE QUESTION HERE
Hello guys,
I am not able to download a m3u8 file cause of this error (HTTP Error 403: Forbidden (caused by HTTPError()) . Any suggestions?
Thank you in advance!

You don't expect an answer without any useful information in your question, or? Please read (and check) the checklist and add all requested information, especially the verbose log.

Sorry man, my english is not so good.
I used to download normal m3u8 files but since some days i cannot.
The link only works with VLC media player but due to the error i wrote
youtube-dl is not able to do it.
If you can help me i can send you a link to try yourself.
Il giorno mer 29 set 2021 alle ore 03:14 MuenchenSued <
@.***> ha scritto:

You don't expect an answer without any useful information in your
question, or? Please read (and check) the checklist and add all requested
information, especially the verbose log.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ytdl-org/youtube-dl/issues/30029#issuecomment-929903484,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO4FEHNEFKRYRW4KTGVL6GTUEK4F5ANCNFSM5E6QTWGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Without a link and your verbose log (youtube-dl --verbose https://.....m3u8) nobody could help.

youtube-dl https://skyfire.vimeocdn.com/1633041688-0x84e8fa0c2b62d408055a1a62beccdd0f9469ae09/589619037/video/2816841829,2816841824,2816841821,2816841560,2816841554/subtitles/13915260-English-eng/master.m3u8
[generic] master: Requesting header
WARNING: Could not send HEAD request to https://skyfire.vimeocdn.com/1633041688-0x84e8fa0c2b62d408055a1a62beccdd0f9469ae09/589619037/video/2816841829,2816841824,2816841821,2816841560,2816841554/subtitles/13915260-English-eng/master.m3u8: HTTP Error 403: Forbidden
[generic] master: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
I always was able to download this kind of link before of 3 days, but now i can't.

These look like temporary links that might be extracted from some page. There may be cookies required or the links may have expired. Where did the links come from?

Yes it's a temporary link, it changes every 15 minutes i guess.
I don't understand if required cookies what i have to do eventually ....

I don't understand if required cookies what i have to do eventually ....

It's all in the Readme:
How do I pass cookies to youtube-dl?

i never understand that. And i am sorry. I try a lot of times, i don't understand how should be the command line :-(

Let's go step by step:

install a suitable browser extension to export cookies, if required (it is for FF and Chrome, although an expert user should be able to use the browser's development tools to extract cookies)
play the target media in the browser
save the cookies to a file using the extension installed in the first step, or a native browser function if available
use your yt-dl command with the addition of the --cookies option whose value should be the filename from the previous step.

All the background information that you need to achieve this is in the Fine Manual, especially the linked section that you've never understood.

Actually I try following those steps, but the result was not good.
But maybe i made some mistakes.
I put the txt file in the same folder where youtube-dl is, and the command
line should be like this?: youtube-dl --cookies/file name of the
cookies.txt "
https://skyfire.vimeocdn.com/1633041688-0x84e8fa0c2b62d408055a1a62beccdd0f9469ae09/589619037/video/2816841829,2816841824,2816841821,2816841560,2816841554/subtitles/13915260-English-eng/master.m3u8
"
Tell me if I am wrong....!
Il giorno ven 1 ott 2021 alle ore 12:35 dirkf @.***> ha
scritto:

Let's go step by step:

install a suitable browser extension to export cookies, if required
(it is for FF and Chrome, although an expert user should be able to use the
browser's development tools to extract cookies)
play the target media in the browser
save the cookies to a file using the extension installed in the
first step, or a native browser function if available
use your yt-dl command with the addition of the --cookies option
whose value should be the filename from the previous step.

All the background information that you need to achieve this is in the
Fine Manual, especially the linked section that you've never understood.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ytdl-org/youtube-dl/issues/30029#issuecomment-932380577,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO4FEHJF37BRGLFMTEIZ4KTUEXPO3ANCNFSM5E6QTWGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

If the cookie file is cookies.txt:
youtube-dl --cookies cookies.txt "https://skyfire.vimeocdn.com/1633041688-0x84e8fa0c2b62d408055a1a62beccdd0f9469ae09/589619037/video/2816841829,2816841824,2816841821,2816841560,2816841554/subtitles/13915260-English-eng/master.m3u8"

Put the cookie file in the current working directory from which you're running yt-dl (or use its full path, like c:\Users\Brugni\cookies.txt, or /home/brugni/cookies.txt). In Windows with cmd shell, use the cd command to show it, or in a POSIX shell, use pwd (and change the "" in the command line to '').

[generic] master: Requesting header
WARNING: Could not send HEAD request to https://skyfire.vimeocdn.com/1733242677-0x4c5e0f639bb1aa823a587745abae5aa0d3ff2834/617857814/video/2868449456,2868449455,2868449445,2868448059,2868448052/master.m3u8: HTTP Error 403: Forbidden
[generic] master: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

I can get the file!!!!
Thank you , thank you a lot for your support and help!!!!!!


Be able to monitor performance

Looked into native postgres monitoring, and don't think it will track historical query data (so I can compare length of execution for the same query over time). It has a lot of low level information, including index usage, so could be useful for other things but maybe not for monitoring changes in query performance as I do a migration.

tracking and aggregating time from the client is pretty easy, can just do that for now.
Other possibilities for the future might include aggregating performance statistics on the server with a filter, and exposing as a spring actuator health/metrics endpoint.


DBZ-3936 Upgrade MySQL example image to 8.0

https://issues.redhat.com/browse/DBZ-3936

@jpechane, if we change this image to 8.0, do we need any changes in CI/testing set-up? Not that we end up testing twice with 8.0 but not at all with 5.7.

There is no relation to CI - MySQL example images are not used for testing.

Ok, thanks for clarifying, @jpechane. Applied. Thanks, @ani-sha!


add workflow to cleanup action caches

building off of recommendations here: https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#managing-caches

https://github.com/NationalSecurityAgency/emissary/pull/816


Warning: Frame scripts cannot be added to empty keyframes.

Hi,
I'm using this.stop() from my timeline in an empty layer. I get an error on export from Adobe Animate:
Warning: Frame scripts cannot be added to empty keyframes. (Layer: 0)
From a desktop web browser, the this.stop() calls work. When I load my Pixi Animate animation in an Android Web View, however, the this.stop() calls are ignored. What would be the best practice for adding a script to a MovieClip?
To prevent the MovieClips from reseting to the first frame upon completion, I added the following. It still does not allow me to stop the timeline from the MovieClip's timeline.
mc.autoReset = false;

Any help would be tremendous. Thanks!

Yes, you cannot add frame scripts on empty keyframes. It's a limitation of the SDK for Adobe Animate, these frame scripts are ignored. You need to add something on the frame, even if it's an invisible graphic.
Try console.log in your frame script to see if it's being called. PixiAnimate by default drops frames on lower performing devices, which might be causing the frame to be skipped. I'm working on a new major release which will make frame-dropping opt-in instead of automatically used.
As a workaround, I'd strongly suggest you use PIXI.animate.Animator API http://jiborobot.github.io/pixi-animate/PIXI.animate.Animator.html
This will allow you to play sequences of your animation by spanning between frame numbers or frame labels. Much easier to control your choreography this way than frame labels.


Sending error response: 500 getaddrinfo EAI_AGAIN d.docs.live.net d.docs.live.net:443

I am unable to get this working. I copied everything over from my old NAS and it fails to connect with the following message

and loads of lines of this in the Docker log
2024/11/25 22:02:17 | stdout | 2024-11-25T21:02:17.936Z proxy:error [16891942/3924] Sending error response: 500 getaddrinfo EAI_AGAIN d.docs.live.net d.docs.live.net:443

I figured it out, I forgot to allow the Docker's network to my firewall.


How to compile with PostCSS

I used yarn to add fontsource to my project. My project is Jekyll, so static site generator. I notice a node_module entry for fontsource. I have a build script that builds my CSS based on using Tailwind. My build script has a single import for fontsource such as:
@import "@fontsource/cooper-hewitt";

This generates an error in my build due to the css not being found. The build expects source to be in /assets/css so the mistake is in trying to find /assets/css/files/cooper-hewitt/index.css etc.
There has to be a way for me to import without having this issue. I solved it by manually copying the /files directory from node_modules to /assets/css. Is that really the secret sauce? Manual copying of files? I am not sure why the PostCSS builder phase fails to look in the node_modules, but that seems to be the problem? I have the latest PostCSS and PostCSS-import packages.
Any hot tips on how to manually build with fontsource ?

I'm not too familiar with PostCSS, but I believe it's not built like a bundler such as Webpack that can process things in node_modules.
#133 is a future solution for you, however, it is something that someone would need to pick up since I personally don't have plans on taking on that workload.

I'll close this as #133 really is the most ideal solution for this if anyone ever chooses to pick that up.


Invalid PID when trying to kill BackgroundCommand

I have started a background task as follows:
self._process = self._target.background(f"poller -l {','.join([str(x) for x in self._files.keys()])} {' '.join(self._files.values())}", as_root=self._as_root)

When I then try to run:
self._process.cancel()

I see the following error:
Command '/home/appa/devlib-target/bin/busybox kill -15 -329893' returned non-zero exit status 1.
OUTPUT: kill: can't kill pid -329893: No such process

Checking on the device I can see that the PID does exist:
root      329893  0.0  0.0   2060   500 ?        S    10:44   0:00 sh -c sh -c 'export PATH=/home/appa/devlib-target/bin:$PATH && export LC_ALL=C && poller -l 0 /sys/devices/system/cpu/cpu0/cpufreq/scaling_cur_freq'
root      329894  0.0  0.0   2060   472 ?        S    10:44   0:00 sh -c export PATH=/home/appa/devlib-target/bin:$PATH && export LC_ALL=C && poller -l 0 /sys/devices/system/cpu/cpu0/cpufreq/scaling_cur_freq
root      329895  0.0  0.0    936     4 ?        S    10:44   0:00 poller -l 0 /sys/devices/system/cpu/cpu0/cpufreq/scaling_cur_freq
appa      330021  0.0  0.0   8576   648 pts/0    S+   10:45   0:00 grep --color=auto poller

~It seems the issue is the preceding -, if I run the command manually without it (/home/appa/devlib-target/bin/busybox kill -15 329893) the process is killed as expected~

It seems that when it is being wrapped by sudo, the PID returned is not the PGID.
$ ps -efj | grep 373747
UID          PID    PPID    PGID     SID  C STIME TTY          TIME CMD
root      373747  370822  373747  373747  0 11:43 ?        00:00:00 sudo -k -p   -S -- sh -c printf '%s ' $$; exec sh -c 'sh -c '"'"'export PATH=/home/appa/devlib-target/bin:$PATH && export LC_ALL=C && poller -l 0 /sys/devices/system/cpu/cpu0/cpufreq/scaling_cur_freq'"'"'  '
root      373748  373747  373747  373747  0 11:43 ?        00:00:00 sh -c sh -c 'export PATH=/home/appa/devlib-target/bin:$PATH && export LC_ALL=C && poller -l 0 /sys/devices/system/cpu/cpu0/cpufreq/scaling_cur_freq'
root      373749  373748  373747  373747  0 11:43 ?        00:00:00 sh -c export PATH=/home/appa/devlib-target/bin:$PATH && export LC_ALL=C && poller -l 0 /sys/devices/system/cpu/cpu0/cpufreq/scaling_cur_freq
root      373750  373749  373747  373747  0 11:43 ?        00:00:00 poller -l 0 /sys/devices/system/cpu/cpu0/cpufreq/scaling_cur_freq
appa      373769  352164  373768  352164  0 11:43 pts/0    00:00:00 grep --color=auto 373747

The PID sent to the kill command in this case was 373748, but you can see the PGID is actually 373747 which is the the outer sudo call

Interesting, thanks for reporting.
In case someone has the same idea, this PR https://github.com/ARM-software/devlib/pull/689 does not provide a workaround, since SIGTERM, SIGQUIT and SIGKILL are still handled in the same way as currently.


Tests does not run as expected with WebAssembly

I'm trying to write some unit tests in typescript for a WebAssembly (wasm) lib written in rust. I'm developing on Windows 10.
I've a sample project here. To compile the rust part, you will need to install rust and wasm-pack.
The test file looks like this:

console.log("before import");
import * as wasm from '../target/pkg'

console.log("after import");
describe("test", function () {
    it("should be ok", function () {
        console.log("test 1");
        expect(wasm.get_color()).to.be("#F5D742");
    }),
    it("should be also ok", function () {
        console.log("test 2");
        expect(1).to.be(1);
    })
})
console.log("end of file");

There are two tests, a wasm one and a javascript one. Both should be ok.
Case 1: large .wasm file
So I ran the tests with npm run test, I got this output in terminal:
TOTAL: 0 SUCCESS

No tests ran as there was no console log from my code.
By checking Chrome Dev tool, Network tab, I noticed the .wasm file is not fully downloaded. Chrome says 69.6KB was downloaded but there should be 15MB.
Case 2: large .wasm file in Karma Debug page
So I clicked the "DEBUG" button, things are getting better, I've this in Chrome console log:
Skipped 0 tests
test.spec.ts:6 before import
test.spec.ts:8 after import
test.spec.ts:19 end of file

The .wasm file is successfully downloaded, all 15MB of 15MB, but still no tests ran.
Case 3: small .wasm file
Then I removed the get_some_large_string function in lib.rs file, this time the compiled .wasm file size is 58KB.
I got logs like this:
LOG: 'before import'
LOG: 'after import'
LOG: 'end of file'
TOTAL: 0 SUCCESS

By refresh the page in Chrome for several times, I got this log:
Chrome 102.0.5005.63 (Windows 10) LOG: 'before import'

Chrome 102.0.5005.63 (Windows 10) LOG: 'after import'

Chrome 102.0.5005.63 (Windows 10) LOG: 'end of file'

LOG: 'test 1'
LOG: 'test 2'
Chrome 102.0.5005.63 (Windows 10): Executed 2 of 0 SUCCESS (0.002 secs / 0 secs)
TOTAL: 2 SUCCESS

It's now working!
I think the problem has something to do with WebAssembly download and compilation must happen asynchronous, there is a line asyncWebAssembly: true in karma.conf.js which is required.
Please kindly check out this issue. Thanks.

I've also tried:
console.log("before import");
import * as wasm from '../target/pkg'

console.log("after import");
describe("test", function () {
    it("should be ok", function () {
        console.log("test 1");
        expect((async wasm).get_color()).to.be("#F5D742");
    }),
    it("should be also ok", function () {
        console.log("test 2");
        expect(1).to.be(1);
    })
})
console.log("end of file");

The result is the same

I've also tried:

console.log("before import");
import('../target/pkg').then((wasm) => {
    console.log("after import");
    describe("test", function () {
        it("should be ok", function () {
            console.log("test 1");
            expect(wasm.get_color()).to.be("#F5D742");
        }),
        it("should be also ok", function () {
            console.log("test 2");
            expect(1).to.be(1);
        })
    })
 });

console.log("end of file");

The result is
Chrome 102.0.5005.63 (Windows 10) LOG: 'before import'

Chrome 102.0.5005.63 (Windows 10) LOG: 'end of file'

Chrome 102.0.5005.63 (Windows 10): Executed 0 of 0 SUCCESS (0.001 secs / 0 secs)
TOTAL: 0 SUCCESS

This works:

console.log("before import");

let wasm : typeof import('../target/pkg');

describe("test", function () {
    before(async function () {
        wasm = await import('../target/pkg');
        console.log("after import");
    });
    it("should be ok", function () {
        console.log("test 1");
        expect(wasm.get_color()).to.be("#F5D742");
    }),
    it("should be also ok", function () {
        console.log("test 2");
        expect(1).to.be(1);
    })
})

console.log("end of file");

I am facing the issue with angular but in my case I am using automerge and karma. Automerge uses webassembly


Problem with duplicate folders

Hello,
I have a problem with folders duplicates in suggestions
As in the picture below:

Can you tell me why this is happening? Can anything be done about it?
Tested on:
Version: 1.42.0-insider
Commit: cba31b819ea99836daf51cee56299348e9c6d20b
Date: 2020-01-08T05:39:25.874Z
Electron: 6.1.6
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.18363

On some file types vscode also provides suggestions for path autocompletions.
To avoid duplicates you can use the path-autocomplete.ignoredFilesPattern to disable path autocomplete on those file types.
You can also review the discussions here:
https://github.com/ionutvmi/path-autocomplete/issues/61
https://github.com/ionutvmi/path-autocomplete/issues/56


Recently used dirs

I often switch between two dirs lately, where I have different project. They're not listed anywhere, so I always have to browse to the last one when switching again. This functionality would be much appreciated (I'd even donate a bit in exchange, you can tell me what EDIT: just remember I'm in a quite poor part of the world).

Hello @bitoolean ,
Thank you for your suggestion, we will look at it how we could implement it.
Best Regards,
Jonathan


Added first version of NFT drop banner

Co-authored-by: Dominic heil<EMAIL_ADDRESS>Co-authored-by: Berinike Tech<EMAIL_ADDRESS>

Some images:

Looks cool!
Seems like we still have PR review problems.
For frontend stuff you could usually add me but I'm honestly rather glad that this time I didn't need to bother so much with reviewing PRs. This React JavaScript stuff is streneous. I usually find a lot of readability and software engineering "problems" for myself but I just won't mind violations against our review guidelines anymore.

And I hope you used an image which is not copyrighted XD .

Ah well, you didn't include "reserved" and the "view" button (and the "retract" button what I expected however). What is the reason?


The history of queries seems to slow down the app

Hi, first of all thank you for your amazing work!
I noticed that the history of queries doesn't seem to have a limit (or a high one). The state.json file that stores the state of the app is currently ~8mb in my case, and the last time I opened the app it took around 1gb of my memory - which is a lot!
Is is possible to put a reasonable limit to the history (I think that less than 100 is good), or even better make it configurable?
FYI, I'm on Windows 10 and using the latest version (1.1)

@Schunk Hi, thanks for pointing out! Will certainly look to improve this experience by maintaining a low memory profile. For now you can clear the history by right clicking the list, not convenient, i know. Will be up for next release.


I106 enable read kerchunk

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes issue #106

[x] Tests for the changes have been added (for bug fixes / features)
[x] Documentation has been added / updated (for bug fixes / features)
[x] HISTORY.rst has been updated (with summary of main changes)
[x] I have added my relevant user information to AUTHORS.md

What kind of change does this PR introduce?:

Support for reading kerchunk files via:
roocs_utils/xarray_utils/xarray_utils.py::open_xr_dataset(...)

Does this PR introduce a breaking change?:

No breaking changes.
New dependencies listed in requirements.txt are: fsspec, zarr, zstandard.

Other information:

Hi @cehbrecht, this PR allows the open_xr_dataset(...) function to read kerchunk files. We are building a version of daops on top of it. Please accept this PR if you are happy with it. Thanks


Fix pulling exact commits for external repos

It turns out we were not pulling the specific git commit for our external repos that are not at a version number. This fixes that as well as the Sundials test to reflect the new file structure.
Resolves https://github.com/LLNL/serac/issues/239 .

LGTM


When the token of a selloffer no longer is available, it isn't removed from active selloffers

        // If, for some reason, the token is not approved anymore (transfer or
        // sale on another market place for instance), we remove the sell order
        // and throw
        if (token.getApproved(tokenId) != address(this)) {
            delete (activeSellOffers[tokenId]);
            // Broadcast offer withdrawal
            emit SellOfferWithdrawn(tokenId, seller);
            // Revert
            revert("Invalid sell offer");
        }

Hey, I am pretty sure that the event will never fire and the selloffer will not be deleted when you revert here.
I guess an alternative would be to send back the value that was sent and remove the revert and put the rest of the code in the purchase method in an else block?

https://github.com/benber86/nft_royalties_market/blob/main/contracts/Marketplace.sol

Thanks for catching this!


Issue regarding multi-geo support.

If I'm correct, this isn't exactly true for multi-geo environments:
"Because there are no remote APIs for creating custom properties in the user profile service, this step must be performed manually for each of the tenants where custom user profile properties are needed (this only needs to be done once per tenant)."
In a multi-geo environment, you need to do this for each of your geo locations in your tenant.
Please correct me if I'm wrong or update the content to reflect this.

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

ID: 8fb0dffd-d314-6e01-be9a-f72d852227e3
Version Independent ID: e48220e4-412d-aee0-ce37-86400efb6cc3
Content: Bulk update custom user profile properties for SharePoint Online
Content Source: docs/solution-guidance/Bulk-user-profile-update-api-for-sharepoint-online.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Correct, this would need to be done in each tenant for multi-geo.


[IDEA] Chapter 18: Basic DQN CartPole

Hi Aurelien,
Thank you for your book and amount of efforts you put down. This is book #1 I've ever read.
I've a suggestion about basic-DQN training in Chapter 18 - CartPole example.
In the book-code you are using very small learning rate 1e-3 and number of trainings is 600-50=550. IMO 550 gradient applications is not enough for RL-model with that learning rate. Even though graph in the book look awesome , I've not been able to reproduce it with same hyperparameters. As a possibility code might be specific to seedings.
I'd propose 2 changes to book code:

increase learning rate (5e-3 or even 1e-2, like in PolicyGrad). Seems that CartPole  is the "hello world" of RL and quite tolerate to high LR
On each episode run multiple Learning Steps (this is partially matches DeepMind '13 Fixed Q-Learning targets paper, by running learning at each step)

    if episode > 50:
        for _ in range(50):
            training_step(batch_size)

Side effects. having same model for learning and evaluation still kind of chasing your own tail, but it will converge/oscillate independently of seeding.
On the other side I totally understand willingness to show how RL-model could be trained with small amount of compute, but it might bring a lot of confusion to audience. If you'll set proper expectations from the get go it could remove many questions like: "my model is not converging", "what seed I should use".......
Though if model is tabular it completely different game.

Hi Ivan,
Thanks for your very kind words and your interesting suggestion. I'm currently writing the 3rd edition of the book (at chapter 13), but as soon as I reach chapter 18, I'll give your suggestion a try, it sounds like a great idea. 🙏

Sum of undiscounted rewards per episode.


[edn/test] Add FIFO reset to prevent overflow

This commit adds a FIFO reset to prevent the generate FIFO from overflowing in the edn_disable_auto_req_mode test.
This only concerns xcelium.
This PR resolves #20300

Thanks @rswarbrick !


[Feature]: f, F, t and T motions for vi mode

What would you like to see added?
Part of #96
Support f<motion> family of keybindings in vi mode implementation. These keys take in a character and will seek to it. See vim's documentation on how these motions work for implementation details.
Implementation suggestions

Add support for these motions in the vi mode parser, which can be found in shrs_vi crate
Implement the actual behavior in crates/shrs_line/src/vi.rs

See also the merged issue #181 as a reference

blocked by #350


Add uib prefixes in accordance to migration guide

Migrate deprecated angular-ui-bootstrap usage to new uib-prefixed
versions. Most of the changes apply to dropdowns, modals and tooltips.
Also remove classes and aria attributes that are added during link phase
for dropdown directives.
Tested against latest ManageIQ Service UI and was not able to locate any
issues.
https://github.com/angular-ui/bootstrap/wiki/Migration-guide-for-prefixes

Hi @jjlangholtz, funny we were just discussing this week how/when to upgrade to latest angular-ui-bootstrap.  Thanks for starting this effort! :+1:
However, I don't see where you actually updated the ui-bootstrap version in bower.json ?
Thanks,

Dave

@dtaylor113 I believe the version was already bumped in a recent PR https://github.com/patternfly/angular-patternfly/blob/master/bower.json#L43, I was thinking that it could actually be bumped to the latest version 2.2.0 but I believe that would be a breaking change for clients of angular-patternfly and should probably involve a version bump here as well.

Hi @jjlangholtz, you are correct, we did just update to angular-bootstrap "0.14.x".  The migration guide says the new prefixes were introduced in 0.14, so it seems like release 0.14 works with and without the 'uib-' prefixes?
Thanks,

Dave

@dtaylor113 so it looks for the 0.14.x series both versions are supported, but usage of the non-prefixed versions shows deprecation warnings in the console. For versions 1.x and above the non-prefixed versions are not supported.

Hi @jjlangholtz, thanks for the uib prefix info.
Cloned this branch, unit tests all passed, but saw the following issues when reviewing the ngdoc examples:

pfCard - Timeframe Filters: The "Last n Days" dropdowns do not open.  Here is a working example
pfCard - Trends: The "Last 15 Days" dropdown doesn't open.

Thanks,

Dave


For some IFSC code, city, state and district are not coming.

For example: DEUT0796DEL
Here city, state and district is not coming in response. It should have address details if it has a valid IFSC ideally. Can you please look into this.
Thanks.

This should automatically get corrected in the next release.

Great.
Thanks for the update. When is the next release expected? Also would there be any impact due to merger of Vijaya, Dena and Baroda Banks?


[HOLD | EXAMPLE] Initial stubs for Vendor Load Plugin

This is a draft branch to provide an example of extending the airflow UI with a flask based plugin.

no longer need this example, and it's still findable as an example even if not merged


values_string

https://humanitec.slack.com/archives/CFLTYLGVB/p1721408939283169
https://humanitec.atlassian.net/browse/DOCS-719
Based on this: https://github.com/humanitec/terraform-provider-humanitec/commit/0b6a10cfc386302386a730ef712e197dd10af6a5.

Ready for your review, thanks!

Ready for your review, thanks!

Thanks so much for the initiative! I found some more occurrences of secrets which IMO must be secrets_string and pushed a commit accordingly.

I found some more occurrences of secrets which IMO must be secrets_string and pushed a commit accordingly.
Oh yeah, really true, good catch!


Don't update sphinx document into gh-pages

gh-pages branch use docfx's document now.

close via #61


Multiple submissions

How do we prevent people from overfitting on the test data by performing many submissions?
Maybe the leaderboard should initially only show the performance on the training data and only after submission is close show the performance on the real data?

We agreed to only show training results until the end.


Bad error message when using CPU tensors

Hi and thank you for the nice project,
I know CPU ops are not supported, but I believe the error messages can be improved, especially when trying to store to CPU tensors.
Here is a toy kernel that copies one tensor to another:

import torch
import triton.language as tl
import triton

@triton.jit
def _copy(
  from_ptr,
  to_ptr,
  N,
  **meta
):
    pid = tl.program_id(0)
    offsets = pid * meta['BLOCK'] + tl.arange(0, meta['BLOCK'])
    values = tl.load(from_ptr + offsets, mask=offsets < N)
    tl.store(to_ptr + offsets, values, mask=offsets < N)

N = 1024
grid = lambda meta: (triton.cdiv(N, meta['BLOCK']), )

feats_from = torch.randn(N, device='cuda')
feats_to = torch.empty(N, device='cuda')
cpu_feats_from = torch.randn(N, device='cpu')
cpu_feats_to = torch.empty(N, device='cpu')

When we call it with 2 CUDA tensors, it goes fine as expected:
_copy[grid](feats_from, feats_to, N, BLOCK=1024)
torch.cuda.synchronize()
# success

When we call it with the tensor to be loaded on CPU, we get:
_copy[grid](cpu_feats_from, feats_to, N, BLOCK=1024)
torch.cuda.synchronize()
# ValueError: Expected a torch.device with a specified index or an integer, but got:None

Which hints that something is wrong with the device. Maybe the message could be more clear and print the actual device of the tensor and that it is expected to be 'cuda'.
However, when we call it with the tensor to be stored on CPU, we get:
_copy[grid](feats_from, cpu_feats_to, N, BLOCK=1024)
torch.cuda.synchronize()
# RuntimeError: CUDA error: an illegal memory access was encountered

This error may also happen when the indices offsets are wrong, so users may look into the wrong place when debugging.

You are totally right. This was fixed in PR #167  :)


Compilattion problems with mongo-cxx-driver on Ubuntu

Hello. Thank you for your work on vcpkg. I have a problem with building my app with a vcpkg dependency (package name is mongo-cxx-driver). Please take a look at details since it can be a bug in the package manager.
In file included from /usr/src/vcpkg/installed/x64-linux/include/bsoncxx/document/element.hpp:20,
                 from /usr/src/vcpkg/installed/x64-linux/include/bsoncxx/array/element.hpp:20,
                 from /usr/src/vcpkg/installed/x64-linux/include/bsoncxx/array/view.hpp:21,
                 from /usr/src/vcpkg/installed/x64-linux/include/bsoncxx/array/value.hpp:20,
                 from /usr/src/vcpkg/installed/x64-linux/include/bsoncxx/builder/core.hpp:21,
                 from /usr/src/vcpkg/installed/x64-linux/include/bsoncxx/builder/stream/document.hpp:17,
                 from /usr/src/skymp3/src/main/mongo_test.cpp:3:
/usr/src/vcpkg/installed/x64-linux/include/bsoncxx/stdx/string_view.hpp:24:10: fatal error: bsoncxx/third_party/mnmlstc/core/string.hpp: No such file or directory
 #include <bsoncxx/third_party/mnmlstc/core/string.hpp>

In string_view.hpp we see these lines:
#if defined(BSONCXX_POLY_USE_MNMLSTC)

#if defined(MONGO_CXX_DRIVER_COMPILING) || defined(BSONCXX_POLY_USE_SYSTEM_MNMLSTC)
#include <core/string.hpp>
#else
#include <bsoncxx/third_party/mnmlstc/core/string.hpp>
#endif

bsoncxx/third_party folder mentioned here doesn't exist:
>> ls /usr/src/vcpkg/installed/x64-linux/include/bsoncxx

array
builder
config
decimal128.hpp
document
enums
exception
json.hpp
oid.hpp
stdx
string
types
types.hpp
util
validate.hpp
view_or_value.hpp

From include/bsoncxx/config/config.hpp:
#define BSONCXX_POLY_USE_MNMLSTC

I just installed mnmlstc/core manually as a quick and dirty solution:
git clone https://github.com/mnmlstc/core.git
mkdir vcpkg/installed/x64-linux/include/bsoncxx/third_party
cp -r core/include vcpkg/installed/x64-linux/include/bsoncxx/third_party

It seems to work fine, but I think vcpkg should learn to install mnmlstc/core automatically.

This issue has been fixed. So I am closing this issue for now.


🤐 Use adm-zip to unzip templates, not unzipper

This should address https://github.com/myst-templates/docx_default/issues/1
Same change we made here: https://github.com/curvenote/jats/pull/19

Also this switches site template download from git clone to admzip. Before we needed a clone in order to pick up submodules. Now, though, the templates are bundled nicely and can simply be downloaded. Hopefully fixes: https://github.com/executablebooks/mystmd/issues/209
Note - it would be good to update the myst-templates API to have zip urls for site templates, not git, but for now, we construct zip from git.

@fwkoch can you confirm this also fixes the site clone, I am not seeing the code for that.

Oops! You're right, added it now.

This has broken site templates, they are not unzipped correctly and are nested into a folder.


LogP calculations for 50k dataset

LogP calculations for the 50k dataset.  Same order as original SMILES file.

What's the origin of these? Are these measured logPs based on looking up the SMILES? Or cLogP from rdkit? Something else?

Sorry, I should have specified.  The LogP values were generated using ChemAxon's "generatemd" program from this repository's "smiles_50k.h5" file.

What would be really great is if you could alter the smiles_50k.h5 file to include a clogp column with this data indexed to the right rows. Then it would work with the --property_column parameter on the preprocessing script and the rest of the tooling here.

I am curious about if it will be an issue to include data generated from ChemAxon. I mean any licensed issue.