Added CSV to JSON or vice-versa Converter Script

Pull Request Template
script name - csv2json.py
What have you Changed
-> I have added CSV to JSON converter script.
what you changed in the codebase.write here
Issue no.(must) -
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My file is in proper folder
[x] I am following clean code and Documentation and my code is well linted with flake8.
[x] I have added README.md and requirements.txt with my script

If issue was not assigned to you Please don't make a PR. It will marked as invalid.
Join Us on Discord:- https://discord.gg/JfbK3bS
acknowledgement

[x] I have tested the script and distributing it under MIT license.

PLease check for lint errors. Use flake8 guidelines for clean and readable code. Thanks
Happy Hacktober


Key value coding for Lists has to know the specific type

Say I have a realm object:
class TestObject : Object {
    let otherObjects = List<TestObject>()
}

If I want to use key value coding:
let testObject = TestObject()
let aList = testObject.valueForKey("otherObjects")!
aList is List<Object> // false
aList is List<TestObject> // true

I have to know the exact type of that relationship.  This makes certain reflection-based approaches sort of difficult, for instance if I want to blindly walk over an object tree performing some action.
Is there a better way to get the KVC to behave, to get generics to behave, or to walk over an object tree?  I see that I can reflect on the objectSchema and get a string representation of the List'd field... but it's a String and I remember NSClassFromString being kind of a bear.  So if I could get the type in the List from the objectSchema that's another option.
Sorry this is sorta hacky!
Just for reference, is this an effect of List<> generics?  Because with normal arrays in swift this works:
class A: NSObject {}

class B: A {}

class Something: NSObject {
    let array: [B] = [B(), B()]
}

let arr = Something().valueForKey("array")
arr is Array<A> // true
arr is Array<B> // true
arr is Array<Int> // false

Is there a better way to get the KVC to behave, to get generics to behave, or to walk over an object tree?

I think our dynamic API would be better suited for this kind of work. If you can represent the objects you'll be acting on as a query, you could then use Realm.dynamicObjects().
Although we officially support our dynamic API and it is technically "public", we don't generally include it in most of our online documentation because it's really only useful in specific scenarios, but it sounds like yours applies.
If you need something we don't have in our dynamic API, please let us know and we'll gladly consider adding it. For example an Object.dynamicList(listProperty: String) -> List<DynamicObject> method in your case.

Just for reference, is this an effect of List<> generics?

I don't know off the top of my head, but I can dig deeper into this if you'd like.

I did some more digging and it turns out that Array/Set/Dict all have different semantics when it comes to the is operator :rage: .  Their is operator does what looks like a protocol match (IMO the expected behavior), whereas the is operator on generics that we can write does a strict match.  How frustrating!  Hadn't noticed/forgot about it.
I thought that List<> was maybe something closer to a true swift Array.  I would go digging into the Array code to see exactly how that's all working but we'll have to wait until Swift open sources.
I'll take a look into dynamicObjects.  Many thanks!

:+1:

Hey so I took a look at the dynamic methods, and you're absolutely right.  I think there's no way right now to blindly get the List<> off an object using KVC if you don't already know what type of list to expect -- this is because as above none of the is or as? operators can cast to anything but the exact type.  So it'd be great if there were a new dynamic method on object that returned a List<DynamicObject> (or a List<Object>.  Really anything that can be reflected) for a RLMProperty.  The other option is to change KVC behavior to return a List<DynamicObject> or again even List<Object> rather than a List<T>, but this requires unsuspecting users to have to cast their objects as their iterate through a List and maybe it's better to just add the extra method?
Let me know what you think.  This exact method looks like it already exists, but internally. Object.listForProperty().

Alright @tom-sparo, I've made this issue into a feature request. You're welcome to submit a pull request for this feature yourself, as I don't think we'll be tackling this ourselves in the near future.

:+1:  I'll see if I can manage it!


"Fixed" the colors of the Dart files.

Finally I fixed the colors of the C# and Dart files.
PD: Great tutorial! It was really easy to implement everything following it.

Can you please include the before and after screenshots? Thanks!

Here they are:
Dart (Flutter) - Before

Dart (Flutter) - After

C# - Before

C# - After

This looks great, thank you.


Add if statement filter

This PR adds a filter for if statements based on library usage in branches. It does not yet check whether the condition of the if involves library usage (or well, technically it does, but this case will always be false in the current implementation).
I don't reaaaallly like it how this looks, any suggestions for a good filtering pipeline or something are very much appreciated.

Codecov Report

Merging #34 into master will increase coverage by 0.72%.
The diff coverage is 82.14%.

@@             Coverage Diff              @@
##             master      #34      +/-   ##
============================================
+ Coverage     75.74%   76.47%   +0.72%
- Complexity       97      123      +26
============================================
  Files            13       12       -1
  Lines           202      204       +2
  Branches         61       63       +2
============================================
+ Hits            153      156       +3
  Misses           23       23
+ Partials         26       25       -1

Impacted Files
Coverage Δ
Complexity Δ

...schaapi/usagegraphgenerator/UsageGraphGenerator.kt
86.66% <100%> (+0.95%)
0 <0> (ø)
:arrow_down:

...schaapi/usagegraphgenerator/filters/ValueFilter.kt
60.97% <100%> (ø)
30 <0> (ø)
:arrow_down:

...api/usagegraphgenerator/filters/StatementFilter.kt
90.9% <100%> (+9.09%)
9 <1> (+1)
:arrow_up:

...i/usagegraphgenerator/filters/IfStatementFilter.kt
77.27% <77.27%> (ø)
31 <31> (?)

...rc/main/kotlin/org/cafejojo/schaapi/common/Node.kt

.../kotlin/org/cafejojo/schaapi/common/DfsIterator.kt

...api/usagegraphgenerator/ControlFlowGraphCreator.kt
66.66% <0%> (+16.66%)
6% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc0f68e...6105613. Read the comment docs.


还需要windows8.1的sdk

首先, 我要说明我是新手.
我按照install.md文件中的指导, 在windows10上构建.dll, .jar的时候, 出了错误.
经过搜索后, 我下载并安装windows8.1的sdk后, 再进行构建, 就没有错误了.
所以我觉得可能需要windows8.1的sdk
不够我并不确定我的想法.
Google Translation:
First of all, I want to show that I am a novice.
When I followed the instructions in the install.md file, when building .dll, .jar on windows10, something went wrong.
After searching, I downloaded and installed the windows8.1 SDK, and then built it, and there were no errors.
So I think it may need windows8.1 sdk
Not enough I am not sure what I think.

Yes, the Visual Studio Windows SDK is required because you need to compile SCIP on your own. I might change the setup such that one can also use our Windows installer.


Context API: Raise a warning if privacy loss is large

I think the best place to raise the warning is in dp.loss_of, and add another argument to hide warnings (warn=False)?
Threshold based on rule of thumb:

epsilon no greater than 1.
delta no greater than 1e-6
rho no greater than 0.25

Threshold based on large values:

epsilon no greater than 5.
delta no greater than 1e-6
rho no greater than 0.5

I think we should only warn based on the thresholds for large values.
A scrappy script that I haven't prettified:
import opendp.prelude as dp
dp.enable_features("contrib")

context = dp.Context.compositor(
    data=[1] * 100,
    privacy_unit=dp.unit_of(contributions=1),
    privacy_loss=dp.loss_of(epsilon=5., delta=1e-6),
    split_evenly_over=1
)
subcontext = context.query().compositor(split_evenly_over=1, output_measure=dp.zero_concentrated_divergence(T=float)).release()

query = subcontext.query().clamp((0, 1)).sum().gaussian()
m_gauss = dp.m.make_gaussian(dp.atom_domain(T=float), dp.absolute_distance(T=float), query.param())
print(m_gauss.map(1.))
print(dp.c.make_zCDP_to_approxDP(m_gauss).map(1.).epsilon(delta=1e-6))

Use an off the shelf logging framework
these should be INFO:

epsilon no greater than 1.
delta no greater than 1e-6
rho no greater than 0.25

while these are WARNING:

epsilon no greater than 5.
delta no greater than 1e-6
rho no greater than 0.5


Add background task support for OnlineIdAuthenticationProvider

This pull request adds support for OnlineIdAuthenticationProvider in a background task.
Adds CredentialPromptType as an optional constructor argument. Using
CredentialPromptType.DoNotPrompt allows for background task usage.
CredentialPromptType.PromptIfNeeded is still the default.
See the issue for more details: #21

Added the enum.

Thanks a bunch! Will get these merged and publish a new package soon.


test PR docs preview

This is just a test PR

Documentation preview will be available shortly at https://provinzkraut.github.io/docs-preview-deploy/1

Documentation preview will be available shortly at https://provinzkraut.github.io/docs-preview-deploy/1

Documentation preview will be available shortly at https://provinzkraut.github.io/docs-preview-deploy/1


Render new lines in suggest widget details

In reference to the refinement here: https://github.com/microsoft/vscode/issues/109717#issuecomment-719440817
The details view in the suggest widget no longer shows the line breaks that the text authors intended for us to see. I think it should work like the mouse over hover text where it breaks the line up if it gets too long but also shows the actual line breaks in the text. See this example:

Any updates on this? It took 3 years to get the current version we have, I hope it won't take another 3 years to finish this feature.


Move core/routemgmt to apigw repo, modify ansible scrips accordingly

Replaced by: https://github.com/apache/incubator-openwhisk/pull/3056


Smartscreen filter integration

We would like to use the Microsoft smartscreen filters for blocking malicious domains injected to our system. Is there a way I can integrate with the filter to know if a domain is malicious or not.

@nitin-kau This channel is for providing bug or feedback on Docs.microsoft.com. Can you provide a page you refer to?


Cant download Video

<EMAIL_ADDRESS>~/Downloads/Project/bityo/BityoDiscordBot/src  ‹main›
╰─➤  youtube-dl https://www.youtube.com/watch\?v\=EI7j6gNhkJA                                                             2 ↵
[youtube] EI7j6gNhkJA: Downloading webpage
[youtube] EI7j6gNhkJA: Downloading player 1dda5629
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
<EMAIL_ADDRESS>~/Downloads/Project/bityo/BityoDiscordBot/src  ‹main›
╰─➤  youtube-dl --version                                                                                1 ↵
2021.12.17
<EMAIL_ADDRESS>~/Downloads/Project/bityo/BityoDiscordBot/src  ‹main›
╰─➤

How can i fix it?

almost the same issue with windows 11 system:
$youtube-dl https://www.youtube.com/watch?v=29ZQ3TDGgRQ
[youtube] 29ZQ3TDGgRQ: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

just wanted to chime in with the same issue, which I'm seeing basically everywhere I try it (both mac and linux), and every youtube video (and also every twitter link),
as one concrete example, like above:
(base) themachine@Thes-MacBook-Pro youtube-dl % ./youtube-dl "https://www.youtube.com/watch?v=ZREv8V5AU_8"
[youtube] ZREv8V5AU_8: Downloading webpage
ERROR: Unable to extract uploader id; 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.

Just read #30839 as you were (all) asked.


verify resident species in data/wsg_species_presence.csv

Pacific Salmon, Steelhead and Westslope Cutthroat presence per watershed group were verified/QA'ed by CWF for the WCRP process.
For other species (BT/CT/DV/GR/RB), this table simply records whether there is an observation within the watershed group, there has been no QA of observations.

Note also that this table must be highlighted and clearly documented in docs

This is generally being fixed on a case by case basis.


Data not rendering in excel if previous column contain a #

Issue
If a column contained a # somewhere, it would not render the remaining characters and the next columns' data.
To reproduce
Used dataset
        const res = [
          {
            id: "468fe976-c50b-45a3-a7f0-2e7fa21bc3ea",
            brand: "Acute",
            campaign: "Campaign #1",
            round: 1,
            username: "Fairy",
            createdAt: "2023-10-17T06:41:08.474Z",
            visits: 0,
            timeSpend: "0 seconds",
          },
        ];

Used Code
        const parser = new AsyncParser({
          fields: [
            {
              label: "ID",
              value: "id",
            },
            {
              label: "Brand",
              value: "brand",
            },
            {
              label: "Campaign",
              value: "campaign",
            },
            {
              label: "Round",
              value: "round",
            },
            {
              label: "Username",
              value: "username",
            },
            {
              label: "Visits",
              value: "visits",
            },
            {
              label: "Time Spend",
              value: "timeSpend",
            },
            {
              label: "Created On",
              value: "createdAt",
            },
          ],
        });
        const csv = await parser.parse(res).promise();

        reply.header("Content-disposition", "attachment; filename=data.csv");
        reply.header("Content-Type", "text/csv");
        return reply.send(csv);

In my dataset, there is # in the campaign, so the subsequent columns' data won't render.

If I take out the #, then everything renders

Any reason why this is happening?
P.S I'm viewing the CSV in Microsoft Excel
Versions
@json2csv/node: 7.0.3
nodejs: 18

Hi @CodingXD ,
I run your code and the generated CSV is correct:
"ID","Brand","Campaign","Round","Username","Visits","Time Spend","Created On"
"468fe976-c50b-45a3-a7f0-2e7fa21bc3ea","Acute","Campaign #1",1,"Fairy",0,"0 seconds","2023-10-17T06:41:08.474Z"

So, the issue is on the way that excel process CSV.
You can try using the StringExcel formatter to workaround this if all you care is for it to look good on excel.

I tried with StringExcel formatter too, but that also failed.
        const parser = new AsyncParser({
          fields: [
            {
              label: "ID",
              value: "id",
            },
            {
              label: "Brand",
              value: "brand",
            },
            {
              label: "Campaign",
              value: "campaign",
            },
            {
              label: "Round",
              value: "round",
            },
            {
              label: "Username",
              value: "username",
            },
            {
              label: "Visits",
              value: "visits",
            },
            {
              label: "Time Spend",
              value: "timeSpend",
            },
            {
              label: "Created On",
              value: "createdAt",
            },
          ],
          formatters: {
            string: stringExcel,
          },
        });


Mention @username pops up while typing email address

Reproduce:

Try to type an email address like<EMAIL_ADDRESS>after you type @  you get related usernames.

Expected:

It should not show any related usernames while typing email address.

Actual

It shows related usernames while typing email address.

App details:

Android 5.0.46

@zulipbot claim

Should I edit the tests to reflect this change. Currently one test fails which expects the filter to return for for email address.


Document System.exit usage

Documentation for a usecase not to use System.exit.
Resolves #1855

Thank you!
I’m offline until mid next week, so feedback will take a while, thank you for your patience.

Thank you for the contribution!


tests.texlive.binaries: fix test for xpdfopen binaries

Description of changes
Adjust the test for pdfclose, a new binary from texlive.bin.xpdfopen, which apparently does not accepts the same arguments as pdfopen. My bad.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[X] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg build tests.texlive.binaries


Tighten up doc pointers in README

Ready for review @clone1018 @mattbk et al.

Capitalization is inconsistent...

👍


[release-1.5] Set readOnlyRootFilesystem to true for istio-init with Istio CNI is enabled

This is an manual cherry-pick of https://github.com/istio/istio/pull/22701
This differs from the master PR in the manifest files modified, the files modified by make gen, and the Helm chart files (release-1.5 is still shipping the Helm charts unlike master).
Fixes: https://github.com/istio/istio/issues/22695

@rlenglet: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.5
5fa7b637a6907e062dac1076e949f448ab4ae35e
link
/test gencheck_istio_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rlenglet: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.5
950bd98bad98c2838188fa4f28411dc80a51cfb5
link
/test gencheck_istio_release-1.5

unit-tests_istio_release-1.5
950bd98bad98c2838188fa4f28411dc80a51cfb5
link
/test unit-tests_istio_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test gencheck_istio_release-1.5
/test unit-tests_istio_release-1.5

@rlenglet: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.5
950bd98bad98c2838188fa4f28411dc80a51cfb5
link
/test unit-tests_istio_release-1.5

gencheck_istio_release-1.5
950bd98bad98c2838188fa4f28411dc80a51cfb5
link
/test gencheck_istio_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rlenglet: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.5
950bd98bad98c2838188fa4f28411dc80a51cfb5
link
/test gencheck_istio_release-1.5

unit-tests_istio_release-1.5
950bd98bad98c2838188fa4f28411dc80a51cfb5
link
/test unit-tests_istio_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rlenglet Thanks for this PR, this was blocking me from upgrading to 1.5 as I noted in my comment: https://github.com/istio/istio/issues/14327#issuecomment-607459212
Do we know when it will be released into 1.5.2?

The 1.5.2 release date is up to the @istio/release-managers-1-5. I don't know whether a date has been set yet?


SW-4713 Implement CSV export for Participant Project data

Since there are several data sources being stitched together in the view, pass them into the export function instead of re-fetching each source.
There is no backend API for this, as it essentially exports a single row, so we are just crafting the CSV in runtime.
Add a test to make sure double quotes and CSV formatting are correct.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2426  👈
#2425
#2424
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nickgraz and the rest of your teammates on  Graphite

Merge activity

Apr 1, 11:12 AM EDT: @nickgraz started a stack merge that includes this pull request via Graphite.


nushell: 0.80.0 -> 0.81.0

Description of changes
https://www.nushell.sh/blog/2023-06-06-nushell_0_81.html

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 236478 run on aarch64-darwin 1

1 package built:

nushell


Add check for existence of repository/schema for each request

Add check for each request whether the repository for the request exists.
Also activated noFallthroughCasesInSwitch and noImplicitAny TS options, leading to @ts-expect-error for database operations as they always return untypes information.

It looks good to me.
I am not 100% sure if it is best to remove the extra check and spare the // @ts-expect-error TS7053 lines or keep it. It is difficult for me to assess which option is best.

Because we get a lot of untyped data from the Postgres queries, I see no other way that using @ts-expect-error. For the rest of the code, the stricter checking does help.


fix(vue-app): prevent mounting page twice on redirect

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

In spa mode and if in middleware ctx.redirect was call in the client first mount, it will trigger fixPrepatch afterEach hook, and setLayout being called after the app was mounted, which will cause the application v-dom structure to be changed and needs rerender, so the page component will be destroyed and re-created, it looks like app was mounted twice.
And, if redirect was call multiple times, the route won't be correct redirected route, because first render call next callback won't follow vue-router resolve flow.
fix: https://github.com/nuxt/nuxt.js/issues/4757
fix: first load multiple redirect
test: add spa redirect tests
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[x] I have added tests to cover my changes (if not applicable, please state why)
[x] All new and existing tests are passing.

Codecov Report

Merging #5361 into dev will decrease coverage by 0.38%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #5361      +/-   ##
==========================================
- Coverage   96.04%   95.66%   -0.39%
==========================================
  Files          74       74
  Lines        2529     2513      -16
  Branches      640      639       -1
==========================================
- Hits         2429     2404      -25
- Misses         84       92       +8
- Partials       16       17       +1

Impacted Files
Coverage Δ

packages/vue-renderer/src/renderer.js
91.58% <0%> (-4.44%)
:arrow_down:

packages/webpack/src/builder.js
94.94% <0%> (-0.06%)
:arrow_down:

packages/utils/src/locking.js
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 77dcfe6...c7d2f41. Read the comment docs.

Really nice PR, thank you very much @noe132
Waiting for @manniL review :)

Has anybody tested it locally yet?

Merging to land and test on nuxt-edge


multi-sharding key, when insert is excuted, one of the sharding keys is null and throws an exception

hi, i discover a question today

Suggestion: You need to add a judgment that result is not equal to null, and ignore result equal to null

duplicated with #3644


Support network QoS attributes customization through kubernetes annotations

Proposal / RFE
Is your feature request related to a problem?
No
Describe the solution you'd like
In the current kubernetes implementation, we have annotations like "egress-bandwidth" and "ingress-bandwidth", which is used to add tc on hosts and shape the traffic for specific deployments, etc.
        "annotations": {
          "kubernetes.io/egress-bandwidth": "10M",
          "kubernetes.io/ingress-bandwidth": "10M"
        },

However, adding per host tc is not very ideal considering the traffic we want shape may have many complex source/destination criteria, which involves non-kubernetes networks as well. The expected behavior is to let kubernetes CNI provide a way to customize the QoS related attributes to the kubernetes egress traffic so we could add better policies on core network layer for better traffic prioritization.
Possible Solution
Instead of adding all the complex policies to kubernetes and CNI, one way is to add another new annotation called traffic-priority like the following:
        "annotations": {
          "kubernetes.io/traffic-priority": "best-effort",
        },

The current idea is to use tos field in the ipv4 headers and stamp all the traffics generated by a pod, and create a readable mapping of a name like "best-effort" to the actual tos value 0-7. It should be modifiable when we just deploy this annotation to the pod and traffic of this pod will carry all the tos field specified. In this way, we could add tos based QoS policies on our core network to prioritize certain production traffics.
Rationale
Observed that sometimes dev namespace pods could generate a lot of traffics, which saturate the limited network bandwidth cross regions. However, for non-cross regions traffic, it is okay for those dev namespace pods to use more. Since there is limited information on the traffic egress from kubernetes nodes, nothing much could be done to prioritize the traffic to avoid spamming.

Kindly ping on this thread.

Also, another through is using DSCP mentioned here which should be a better alternative for QoS purposes.

Also, another through is using DSCP mentioned here which should be a better alternative for QoS purposes.

MBOI @borkmann

Hi @lordofire are you planning to use this to deploy DCTCP?

@borkmann Actually I was not aware of DCTCP before you mentioned it here, and it is another interesting case, thanks for bringing it up :) Also, I see what you are coming from on HOL blocking topic, since not appropriate QoS policy in DC could get more HOL blocking situation which further degrades the performance.
However, in my certain use case, we focus more on the QoS for cross-DC long-haul DCI backbone links. Their resources are more scarce and valuable so we would like to have better policies enforced to prioritize only the most important ones. For statically deployed services in VMs it is easy to determine the policies based on the service on the hosts themselves, but it requires CNI's assistance if we would like to add this capability to K8s dynamically. For example, how do we prioritize the traffic from some pods in production compared to those data-intensive traffics if both of them could be dynamically scheduled to different nodes on the same Kubernetes cluster?

check

check

check

Bump.

This feature could help.


cli integration

Hi bro, i know this a branch, but can we try to add the shell functionality to this build? you know that the cli integration is important.

Hey @feraxhp! This project was based on the manual provided at https://github.com/zed-industries/zed/blob/main/docs/src/development/windows.md, and I didn't find any instructions about the shell. I could help with it if you would be able to provide the instructions.

Yeah, i was reading the code, it is no jet implemented for windows, when a update appears i will update.


not producing UF2 output

Your config do not produce UF2 output

🤔
Please post the output of:
git status
git rev-parse HEAD
grep set\(PICO_SDK_VERSION_ $PICO_SDK_PATH/pico_sdk_version.cmake
ls -l *.uf2

I deleted .uf2 files in both build and ps2pico directory as I saw that they are not updated when execute make
they are never build after this
the output is
ceco@ceco-ThinkCentre-M920q:~/git/ps2x2pico$ git status
git rev-parse HEAD
grep set\(PICO_SDK_VERSION_ $PICO_SDK_PATH/pico_sdk_version.cmake
pwd
ls -l *.uf2
On branch main
Your branch is up to date with 'origin/main'.

Changes not staged for commit:
  (use "git add/rm <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	deleted:    ps2x2pico.uf2

no changes added to commit (use "git add" and/or "git commit -a")
b874f5c6dcdb057d60abf4c91afb27174010161d
set(PICO_SDK_VERSION_MAJOR 1)
set(PICO_SDK_VERSION_MINOR 5)
set(PICO_SDK_VERSION_REVISION 1)
#set(PICO_SDK_VERSION_PRE_RELEASE_ID develop)
set(PICO_SDK_VERSION_STRING "${PICO_SDK_VERSION_MAJOR}.${PICO_SDK_VERSION_MINOR}.${PICO_SDK_VERSION_REVISION}")
    set(PICO_SDK_VERSION_STRING "${PICO_SDK_VERSION_STRING}-${PICO_SDK_VERSION_PRE_RELEASE_ID}")
/home/ceco/git/ps2x2pico
ls: cannot access '*.uf2': No such file or directory
ceco@ceco-ThinkCentre-M920q:~/git/ps2x2pico$

Please try this and post the output:
cd build
elf2uf2/elf2uf2 -v ps2x2pico.elf ps2x2pico.uf2

thanks! this did the job :)

Cool, hm but I don't know why cmake/make didn't do that 🤷

I saw you are integrating it into the Neo6502, just ordered one myself! 👍

yes, I was study to add in Neo6502 (where PS2 interface is not necessary)  but the USB keyboards/mouse fits great the FabGL PS2 interface for the further revision of  AgonLight2 ;) PS2 keyboards and mouses are exotic today

PS2 keyboards and mouses are exotic today
True! I ran into the same issue using newer keyboards on my NuXTv2


Rosbag for multiple apollo2.0 machines

Hi all,
I have 2 machines with apollo and apollo platform 2.0 setup.
Machine 1: roscore, dreamview
Machine 2: rosbag play demo_2.0.bag
They are connected together.
I can get data from machine 1 with rosptopic echo command.
But problem is dreamview on machine 1 cannot receive any data.
Thank you

Can you try running ./scripts/dreamview.sh start_fe to see if Dreamview starts correctly on your machines 1? You can take a look at https://github.com/ApolloAuto/apollo/blob/master/docs/howto/how_to_debug_dreamview_start_problem.md for troubleshooting guidance.

Hi SiYangy,
I already check with your command to start dreamview.
The output like this.
I believe dreamview has started well. Because I can load http:local:8888 page and see background image website. But no data or simulation works.
One more thing, if then I run rosbag play from machine 1. Data and simulation work fine.
I don't know if rosbag play has special communication with dreamview beside publish and subscribe message or not?
or does rosbag support play bag for multiple machines?
Thanks so much
@in_dev_docker:/apollo$ cat data/log/dreamview.out 0621 16:29:52.361416  6501 hdmap_util.cc:36] No existing file found in modules/map/data/sunnyvale_big_loop/base_map.bin|base_map.xml|base_map.txt. Fallback to first candidate as default result. E0621 16:29:52.361852  6501 file.h:139] Failed to open file modules/map/data/sunnyvale_big_loop/base_map.bin in binary mode. E0621 16:29:52.361881  6501 file.h:96] Failed to open file modules/map/data/sunnyvale_big_loop/base_map.bin in text mode. E0621 16:29:52.361897  6501 hdmap_util.cc:62] Failed to load HDMap modules/map/data/sunnyvale_big_loop/base_map.bin E0621 16:29:52.361930  6501 hdmap_util.cc:36] No existing file found in modules/map/data/sunnyvale_big_loop/sim_map.bin|sim_map.txt. Fallback to first candidate as default result. E0621 16:29:52.361973  6501 file.h:139] Failed to open file modules/map/data/sunnyvale_big_loop/sim_map.bin in binary mode. E0621 16:29:52.361990  6501 file.h:96] Failed to open file modules/map/data/sunnyvale_big_loop/sim_map.bin in text mode. E0621 16:29:52.362004  6501 hdmap_util.cc:62] Failed to load HDMap modules/map/data/sunnyvale_big_loop/sim_map.bin E0621 16:29:55.620630  6527 file.h:96] Failed to open file modules/map/data/sunnyvale_big_loop/default_end_way_point.txt in text mode. E0621 16:29:59.626375  6527 json_util.cc:70] The json has no such key: lane E0621 16:29:59.626452  6527 json_util.cc:70] The json has no such key: crosswalk E0621 16:29:59.626471  6527 json_util.cc:70] The json has no such key: junction E0621 16:29:59.626488  6527 json_util.cc:70] The json has no such key: signal E0621 16:29:59.626507  6527 json_util.cc:70] The json has no such key: stopSign E0621 16:29:59.626523  6527 json_util.cc:70] The json has no such key: yield E0621 16:29:59.626538  6527 json_util.cc:70] The json has no such key: overlap

I believe this is due to by default ROS uses shared memory as communication channel instead of network. It might be configurable to use network though. @quning78, can you help answer this?

I changed to socket mode then they work fine.
I have some confusion about how come the rostopic echo on machine 1 can get data from machine 2 with share memorry mode in default when dreamview cannot get.
Thanks so much

@levuphuong I have the same issue. Could you tell me how have you changed to socket mode?
Thanks!

You can change mode in /home/tmp/ros/etc/ros/transport_mode.yaml

One possible solution is that both machines should register their IP address in the ROS network. More precisely   machine 1 should append the following command to its .bashrc file:export ROS_IP=< IP address of machine. ——Reply reference comes from the Apollo developer community issue master contest https://apollo.auto/developer/questiondata_cn.html?target=674


Coalesce oAuth token requests

Checking a new oauth token with the provider is currently unconditional. This results in more upstream provider calls than are necessary if more than one API request occurs in a short interval with the new token.
Example: when our web app receives a token error, we will contact Google for a new token, then re-make any failed requests to the scitran API. In practice, this is perhaps ~5 requests made simultaneously with the new token.
The API could coalesce oAuth token provider checks by sharing context between workers such that only one in-flight request for a given token occurs at a time. Expected result is making the correct & minimal request count on the oAuth provider.

Thanks, @kofalt. How urgent is this? @josschne mentioned an issue with rate-limiting. As far as I know, we've never run into this with SDM. Pinging @rentzso confirmation/advice.

I don't think it's very urgent, I mainly wanted to track it as an opportunity. I've been meaning to import GH labels from a successful OSS project I like, and if I had this would be tagged "p:low" :)
@josschne Hit Google's API limit during some testing, but I'm not currently convinced a site will be likely to hit that in production. OTOH, if this reduces upstream oAuth provider calls by a factor of ~5, which seems likely, it's definitely something we should do sooner or later.

Definitely saw a rate limit error from Google, but it was while testing with invalid oauth tokens (part of how I was testing a refresh scenario without having to wait an hour). Basically if an app hits the API with a bunch of requests with a new/refreshed token, there is a window during which each of them will result in a separate request out to Google. Agree with @kofalt and @gsfr that this is a good idea, but low priority.

@gsfr we never noticed this issue before, but it may be possible that the retry logic on SDM has masked the problem. Is there any retry logic on FDM's API calls? Probably it is not a good long-term solution, but it could be a convenient work-around.

Our client logic is slightly sub-optimal ATM: if a request fails due to invalid token, we get a new token and retry the request. This results in the web app, for example, renewing a token 5 times and sending 5 requests each with a different (new, valid) token.
Ideally, when a token renewal is in-flight, all failed / new requests would hold for that, then all use the same token. Once that client-side bug is fixed, we'd still have the server making redundant requests, which this ticket will address.

Sounds good. Let's draft a solution by the end of the week.

@josschne believes that hello.js is already handling token reveal correctly and only retrieving a single new token.

Cool; that leaves only this ticket to resolve then if I understand correctly

As I stated in #44, this may not a actually be an issue that needs resolving right now. The app should never be sending concurrent requests with non-cached tokens. @josschne is working on ensuring that /users/self requests always happen before further, concurrent requests. Let's see where that leaves us and then reprioritize as needed.
@kofalt If you have thoughts on a possible implementation of this enhancement, please document them here.

closed after discussion with @kofalt


No zoom indicator

With stock Firefox, there's an indicator when you're not zoomed to 100%. I can't seem to find it with the theme applied.

Delete line 172 from urlbar.css!
As someone who set 80% as default zoom, the persistent zoom indicator is annoying to me - hence why I hide it by default here. Recently I tried Arc again and their approach to zoom, displaying our current zoom level just as a tooltip/timed popup, is so much better ahaha


[Superpins] Light theme has too dark boxes?

Hey!
Loving the Superpins you've made for Zen. One thing is kinda buggin me tho, and that's the light theme color:

The boxes are too present in my opinion.
What's your opinion? :)

Hey, thanks for the kind words. Regarding the colors, i completely forgot about light theme when developing the theme and only tested it on dark theme. I've already changed this (since someone else also mentioned this) along with some other things, however the update (pull request) is still waiting for approval by the dev. Here's the pull request if you wanna see what i changed: https://github.com/zen-browser/theme-store/pull/226
Keep in mind that it could take some time before the pr gets approved because only the dev (mauro) can do it and he's quite busy with the browser, but as soon as it gets approved you'll only need to check for updates in the theme store settings page and everything should work

Ah thanks, didn't know you already fixed it! Great work :)

Hey, the update/pr has been merged, the boxes should now be darker in light theme along with some other stuff!


The calldata of the transaction that is not parsed through abi is an object

Try to use the Calldata in the starknetjs 5.14.1 version package to parse and format, but the desired data is not parsed. The data returned by parse and format are both success: 1n }. Can't the calldata be restored to the object through abi?

Please do not delete bug template when posting issue else we need to go back and forward with questions.
Could you give us more context to debug your issue (what type of data Structs, tuples, tuples in tuples in struct...)?
Can you provide a minimal example to reproduce your issue?

Close due to inactivity


Cooperation or any interaction with FSharp.Control.Reactive

FSharp.Control.Reactive is a library that also implements Rx for F#. But it is oriented for the native (.NET, not Fable) runtime and actually is based on System.Reactive.
I like both libraries, but FSharp.Control.AsyncRx has Fable support, whereas FSharp.Control.Reactive has (to be honest) is better supported in general. I think that would be nice if they could somehow cooperate to get a solution that combines the benefits of both options. :slightly_smiling_face:

AsyncRx is related to Rx the same way as AsyncSeq is related to Seq. They are all related to each other

Sync
Async

Push
Rx
AsyncRx

Pull
Seq
AsyncSeq

So all these "collection" types are closely related to each other. In the same way Seq is also better supported than AsyncSeq. It would be nice to keep the impl and docs together but I have no clue of what the process is and who to contact.

I see they have an issue about that. But it's directly related to the .NET implementation, and I have no idea how it can natively transpiled to Fable. Maybe we should take a look at the Fable plugin system?

Yes but the issue is about dotnet/reactive, the C# impl of AsyncRx made by Bart de Smet. This is quite a different implementation than my F# that uses Async instead of Task and tries to be lock free by using mailbox processors (to be Fable compatible)

Yes, I understand the problem. And I suggest to use the Fable plugin system to replace all FSharp.Control.Reactive functions with the corresponding FSharp.Control.AsyncRx functions. So a user can use FSharp.Control.Reactive library (which contains, for example, a Task-oriented code), but in fact the mailbox-based implementation will be used.
Actually all (supported) BCL types works exactly this way in Fable.


[PDI-18459] When the Database connection which is used in the Transfo...

...rmation step is valid,status in pentaho_dilogs .trans_logs table does shows as "running" even though the Transformation has failed. - Unit tests
Also clarified the comment left on the original PR.
Should be merged after https://github.com/pentaho/pentaho-kettle/pull/7523
@pentaho-lmartins @ssamora @ppatricio @bcostahitachivantara

:white_check_mark: Build finished in 31m 10s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3711, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message


model predicts blank for every test sample

Hello,
I tried testing the model 45-best-val-weights.h5 but was consistently getting blank outputs.
I tried testing with my own samples aswell as librispeech samples, but no change in result.
I am using the models-evaluation.ipynb notebook mentioned. No changes to the notebook except for the path inputs.
I am using it using latest Keras 2.0.8 and TF 1.3.0
Can u look into the matter.
Thanks

Data normalization was needed: datagen.reload_norm('860-1000')


https://dotnet.microsoft.com/ SSL certificate mismatch

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Go to https://developer.microsoft.com/zh-cn/
Click on

See error

Expected behavior
https://dotnet.microsoft.com/ should have valid SSL cert
Additional context
> dig dotnet.microsoft.com
;; communications error to <IP_ADDRESS>#53: timed out

; <<>> DiG 9.18.12-0ubuntu<IP_ADDRESS>-Ubuntu <<>> dotnet.microsoft.com
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 59473
;; flags: qr rd ad; QUERY: 1, ANSWER: 6, AUTHORITY: 0, ADDITIONAL: 0
;; WARNING: recursion requested but not available

;; QUESTION SECTION:
;dotnet.microsoft.com.          IN      A

;; ANSWER SECTION:
dotnet.microsoft.com.   0       IN      CNAME   dotnetwebsite.azurefd.net.
dotnetwebsite.azurefd.net. 0    IN      CNAME   firstparty-azurefd-prod.trafficmanager.net.
firstparty-azurefd-prod.trafficmanager.net. 0 IN CNAME shed.dual-low.part-0045.t-0009.t-msedge.net.
shed.dual-low.part-0045.t-0009.t-msedge.net. 0 IN CNAME part-0045.t-0009.t-msedge.net.
part-0045.t-0009.t-msedge.net. 0 IN     A       <IP_ADDRESS>
part-0045.t-0009.t-msedge.net. 0 IN     A       <IP_ADDRESS>

;; Query time: 0 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>) (UDP)
;; WHEN: Wed Dec 20 10:07:38 CST 2023
;; MSG SIZE  rcvd: 424

It seems to be working now, so I'll close this issue. Let us know if it's still an issue for you. cc @mairaw.

No, it still not works.

I can visit the website now, thanks.


How to send monitoring data from Azure Databricks to Azure Monitor using Python notebooks

Hello!
Do you have an example of how to send monitoring data from Azure Databricks to Azure Monitor using Python notebooks?
Best regards,
Cristina

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

ID: 9f728cb2-b203-6b2f-4e3c-9d0e1be2516f
Version Independent ID: 9ed58ead-f4d0-d9d0-01fe-82b11be235a7
Content: Send Azure Databricks application logs to Azure Monitor - Azure Architecture Center
Content Source: docs/databricks-monitoring/application-logs.md
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @Petertaylor9999
Microsoft Alias: pnp

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#200054

Hi @crisansou, even though we do not have a direct sample in python associated with this work, here is related information from Databricks and a sample Python project - https://docs.microsoft.com/en-us/azure/azure-databricks/quickstart-create-databricks-workspace-portal
Now, note that our work leverages the log4 family of technologies to create an additional appender in order to track a set of cluster metrics. Feel free to expand the current set to meet your needs.


How to use this plugin on Chinese phones without Google Play?

Android is China’s most popular mobile operating system, but, paradoxically, smartphones sold in China cannot have Google services pre-installed.

Yes, I'm completely aware of this.
Sorry, this plugin requires apis available only in Play Services.
It will not work for you in China.


Debug using developer tool

Your project looks great! You have put an enormous effort into achieving this. However, if you could improve a few things, it would be bug-free.
index.js : In line 153, you have selected a class that never exist.

Rather, the target has an id.

So, instead of dot (.), you can use hash (#).
Also, on the mobile menu,

On selecting menu-item About it is not scrolling to the respective section on the page.

On selecting menu-item Contact it is not scrolling to the respective section on the page.

Thanks for your suggestion. The bug has been removed. 😉


Support new base images

Support patch versions of torch by not stripping the torch version to major/minor
Add new compatibility matrix for torch 2.4.0 and CUDA 12.4.x and cuDNN
Exclude torchaudio from pip installs like torchvision since the new base images should include the torchaudio package

This will introduce a race condition, since building the base images relies on this matrix, and until that is run these base images will be invalid, so it should be done in close connection to the generation of those base images.

Will separate https://github.com/replicate/cog/pull/1890/commits/43f0d662e9f17201fcd275e64aa80130e570f5c9 into a separate PR

Making https://github.com/replicate/cog/pull/1890/commits/46375e3f40f343891f8a338ea1a58f33daed0c30 into a separate PR.
https://github.com/replicate/cog/pull/1893

Isolating the torch patch version support in https://github.com/replicate/cog/pull/1894


alias highlighted in green no matter the configuration

Operating system:     Ubuntu 16.04.5 LTS
Terminal emulator:    rxvt-unicode v9.22
Terminal multiplexer: tmux 2.8
$TERM:              tmux-256color
Shell:                zsh 5.6.2-dev-0 (x86_64-pc-linux-gnu)

This commit seems to have broken the syntax highlighting of aliases in my shell.
They are green no matter what I assign to ZSH_HIGHLIGHT_STYLES[alias].
At the moment, I can reproduce with this minimal zshrc:
alias ls='ls --color=auto'
[[ -f ${HOME}/.zsh/plugins/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh ]] && \
. "${HOME}/.zsh/plugins/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh"
ZSH_HIGHLIGHT_HIGHLIGHTERS=(main)
typeset -A ZSH_HIGHLIGHT_STYLES
ZSH_HIGHLIGHT_STYLES[alias]='fg=red'

The ls alias is highlighted in green, even though ZSH_HIGHLIGHT_STYLES[alias] should make it red.
I don't think there's anything wrong with the syntax of the command, since this one works fine:
ZSH_HIGHLIGHT_STYLES[single-quoted-argument]='fg=red'

It highlights a single quoted argument in red.
I had the same issue yesterday, but after commenting out some code/checking out old commits and getting back to master, the issue disappeared. Today, after updating the plugin again, the issue is back, so I don't know how reliable is the previous MWE.

That commit made it so that alias are highlighted as alias and whatever the first word of the alias is (in your case command). https://github.com/phy1729/zsh-syntax-highlighting/tree/highlight-alias-once changes that to just alias (unless the alias is insane or a loop in which case it also gets unknown-token). I have no strong opinion either way.

Thank you very much for your comment.
I really liked the old behavior because it allowed me to easily distinguish a builtin command from and an alias, which I could expand by pressing M-e:
__expand_aliases() {
  emulate -L zsh
  unset 'functions[___expand_aliases]'
  functions[___expand_aliases]=$BUFFER
  (($+functions[___expand_aliases])) &&
    BUFFER=${functions[___expand_aliases]#$'\t'} &&
    CURSOR=$#BUFFER
}
zle -N __expand_aliases
bindkey '\ee' __expand_aliases

I've tested your fork and your branch highlight-alias-once, and it works exactly as before, thank you.
It would be nice if there was an option to get back the old behavior, otherwise I'll try to get used to the new one.
Feel free to close the issue if you disagree.

(I commented earlier via email; I assume that'll appear eventually so I won't repost it. In the meantime..)
Let's try to nail down a behaviour spec for aliases.
How about:

By default, an alias is highlighted as alias, regardless of what it expands to.
Nevertheless, an alias that expands to something that would be highlighted as unknown-token, such as alias foo=/no/such/command and even alias foo='ls && /no/such/command, would itself be highlighted as unknown-token. (New behaviour)
If someone ever wants to highlight aliases differently depending on their expansion, that should be done by adding one style for the alias to $region_highlight.  For example, instead of adding "1 3 alias" "1 3 command", one should add just "1 3 alias-to-command" (using the syntax of test expectations).

Which implies that the test expectations of alias-to-dir and alias-comment2 are correct as they stood before the recent changes.

+1 It seems like a sensible spec, I would like it.

Thanks for the review! You're welcome to join us on IRC if you like :)

I would default to erring on the side of backwards compatibility.
@phy1729 What's the benefit of the new behaviour?  Can it be implemented differently, for example, by highlighting it as command then alias (opposite order of now), or as alias-command which would auto-fallback to alias for compatibility?  I'm particularly fond of the latter since it'll make it much easier for theme authors (aka, users, currently) to set the styles.
lacygoill wrote on Mon, 22 Oct 2018 06:15 -0700:

Thank you very much for your comment.
I really liked the old behavior because it allowed me to easily
distinguish a builtin command from and an alias, which I could expand by
pressing M-e:
__expand_aliases() {
  emulate -L zsh
  unset 'functions[___expand_aliases]'
  functions[___expand_aliases]=$BUFFER
  (($+functions[___expand_aliases])) &&
    BUFFER=${functions[___expand_aliases]#$'\t'} &&
    CURSOR=$#BUFFER
}
zle -N __expand_aliases
bindkey '\ee' __expand_aliases

I've tested your fork and your branch highlight-alias-once, and it
works exactly as before, thank you.
It would be nice if there was an option to get back the old behavior,
otherwise I'll try to get used to the new one.
Feel free to close the issue if you disagree.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/zsh-users/zsh-syntax-highlighting/issues/565#issuecomment-431832360

@danielshahaf Branch updated to be either alias or unknown-token. Found a few bugs as well. If you like I can make a PR out of it.
@lacygoill I forgot earlier, but thank you for the clear, minimal reproducer. It makes things much easier.

Sure, may be easier, and numbers are cheap. In alias-to-dir I think you removed the wrong line?
(Will look closer later.)


(Webpack > 4.3.10) ReferenceError: document is not defined

Bug report
What is the current behavior?
Before 4.31.0, webpack never referenced document when it is not defined. After this release, there is code somewhere which assumes a global document is available. This makes it so server side rendering does not work in our application.
If the current behavior is a bug, please provide the steps to reproduce.
Using webpack > 4.31.0, compile a simple app for the web target, then attempt to run in an environment without document.
What is the expected behavior?
Webpack should always check to see if document is defined before attempting to access it.
Other relevant information:
webpack version: any version ><EMAIL_ADDRESS>Node.js version: v10.15.0
Operating System: Linux
Stack Trace
MiniRacer::RuntimeError:
  ReferenceError: document is not defined
JavaScript at length (/webpack:/__lib.[name]/webpack/bootstrap:48:1)
JavaScript at Array.forEach (<anonymous>)
JavaScript at r (/webpack:/__lib.[name]/webpack/bootstrap:41:1)
JavaScript at Array.apply (/webpack:/__lib.[name]/webpack/bootstrap:23:1)
JavaScript at Array.Object (/webpack:/__lib.[name]/webpack/bootstrap:18:1)
JavaScript at Array.Object (/webpack:/__lib.[name]/webpack/bootstrap:18:1)
JavaScript at Array.Object (/webpack:/__lib.[name]/webpack/bootstrap:18:1)
JavaScript at Array.Object (/webpack:/__lib.[name]/webpack/bootstrap:18:1)
JavaScript at Array.Object (/webpack:/__lib.[name]/webpack/bootstrap:18:1)
JavaScript at Array.Object (/webpack:/__lib.[name]/webpack/bootstrap:18:1)

Please create minimum reproducible test repo

@Borming https://github.com/webpack/webpack/issues/9394#issuecomment-510186922


Display Virtual Desktop Name on the Taskbar

Description of the new feature / enhancement
Display the name of the current virtual desktop somewhere on the taskbar.
Scenario when this would be used?
I use several virtual desktops to separate personal and different contract workspaces. Since many of these use similar windows (chrome, code, notepad), it's difficult to know which I'm on when switching through them via ctrl-win-arrow. Having that information appear in the taskbar would be a godsend.
Supporting information
No response

/dup #12246


[BugFix] Remove reserve operations to avoid unnecessary memory allocation

ChunkHelper::new_chunk_pooled Remove reserve operations to avoid unnecessary memory allocation

What type of PR is this:

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

Does this PR entail a change in behavior?

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

If yes, please specify the type of change:

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

Checklist:

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

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

Bugfix cherry-pick branch check:

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

[x] 3.2
[x] 3.1
[x] 3.0
[x] 2.5

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhengxingmao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


⚠ Bofrost.fr has degraded performance

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

HTTP code: 200
Response time: 961 ms

Resolved: Bofrost.fr performance has improved in 9e611d1.


Get SVG as text so I can use it on the server, without a DOM

This is an awesome tool, but I'd like to be able to use it on the server without a DOM. I don't see anything stopping this from working outside the browser, and SVGs are just text. It would be nice to just be able to do:
var svg = Diagram.parse("A->B: Does something").getSVG( { theme: "hand" } );

I don't think Raphaeljs that serves as the rendering engine can work outside a browser. You should check diagrams

Let's see if the move to snapsvg will help (if/when.)

Ok, I've rolled out v2.0 based on Snap.svg. I haven't looked to see if that would work purely outside a browser, but I suspect it will (with perhaps some minor changes).


Uninstall plugin if post-install hook fails, i.e. dependencies can't be installed

[1] Plugins with failed post-install, post-update routines should be removed or disabled.
[2] There should also be a way to emit a post-install failure in the Plugin Installation window.
i.e.
function! SyntasticPostInstall(info)
  "install jshint
  !npm install jshint -g

  "check jshint is installed
  if(system('command -v jshint') == '')
    echoerr 'jshint must be installed in order for syntastic to work with js'
    "remove plugin
    !rm -rf ~/dotfiles/.vim/bundle/syntastic
    "emit some failure message here
  endif
endfunction

Plug 'scrooloose/syntastic', {
    \ 'do' :  function('SyntasticPostInstall'}

I guess this would work...
function! SyntasticPostInstall(info)
  "install jshint
  "!npm install jshint -g

  "check jshint is installed
  if(system('command -v jshint') == '')
    "remove plugin
    !rm -rf ~/dotfiles/.vim/bundle/syntastic
    throw 'Warning: jshint not installed. Plugin removed.'
  else
    echom 'jshint found'
  endif

  echom 'End postinstall'
endfunction

Plug 'scrooloose/syntastic', {
    \ 'do' :  function('SyntasticPostInstall')}


DEPRECATION: Sidekiq exception handlers now take three arguments

worker | 2023-10-04T10:02:00.510Z pid=253522 tid=4yc6 INFO: DEPRECATION: Sidekiq exception handlers now take three arguments, see #<Proc:0x00007f07be656108 /home/devanil/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/rollbar-3.4.0/lib/rollbar/plugins/sidekiq.rb:15>
worker | 2023-10-04T10:02:00.510Z pid=253522 tid=4y76 ERROR: Error fetching job: Waited 3.0 seconds
      config.error_handlers << proc do |e, context|
        Rollbar::Sidekiq.handle_exception(context, e)
      end

I'm always surprised by the lack of Ruby support for this product we pay for.. when simple maintenance tasks like this grow stale, I begin to wonder if we should switch to something else
@waltjones is Ruby still supported?

Hey @devaniljr @bf4, We scheduled working on the Ruby SDK after Thanksgiving. Our Ruby dev is currently headed down on another crucial initiative, but I want to assure you we still remember this one. I was informed that the Sidekiq update will probably require a breaking change; hence expect a major version bump.


Fix inline recognition

Right now, if the struct is like the following:
type Foo struct {
    Bar Bar `bson:",inline,omitempty"`
}

it's not detected, because it looked for a struct tag equal to ",inline" when in reality it can contain omitempty as well.
This PR corrects this behaviour and makes the recognition of "inline" fields work properly with all cases.

Can you provide a test?

Done @mcuadros

Current coverage is 44.60% (diff: 100%)

Merging #29 into master will not change coverage

@@             master        #29   diff @@
==========================================
  Files            16         16
  Lines          1094       1094
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            488        488
  Misses          556        556
  Partials         50         50

Powered by Codecov. Last update b5fc309...96b76a9


#5424: Delegated sfpu reciprocal calls to gs submodule functions

Ticket
(https://github.com/tenstorrent/tt-metal/issues/5424)
Problem description
api and submodule for gs had own copies of their functions to compute reciprocal.
What's changed
Delegated the function calls to the ones in submodule. Did not ignore and the flag for saving a register as it seems some might be using it to save some registers in some code which requires a lot of assumptions. But I set them as false by default as otherwise it will slow down performance
Checklist

[x] Post commit CI passes
[ ] Model regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

The post commit tests before pointing the tt_llk_gs branch to main.


Gradle wrapper missing in distribution

Distribution is generated without gradle-wrapper.jar.

@ChuckTheSheep I am unable to reproduce this issue, try running assembleDist again and see if the issue persists.

The issue still persists for me.

Did you try running gradlew clean assembleDist and verifying that there is no wrapper produced?

I only ran gradlew assembleDist this morning, but I just ran that command and this is the contents of the build zip created.

I guess this doesn't matter that much since users are not suppose to be building their own distributions anyway, but I am still curious as to why this is happening. Can you confirm that you are checked out on release/2.0 branch in the repository and that you are using the right directory?
Also, we are using assembleDist shell script to create distributions (you can find it in run configurations) so you don't need to run ls-git-ignore and then assembleDist via Gradle. Thanks for helping to track this issue down.

I was actually on the previous version. The problem is fixed with 2.0.

Resolved by 17def301730b03f0805906062631e607b20715ae


Indenting Nokogiri HTML

Hi there! Love this gem!
I noticed that the Nokogiri pretty printer doesn't always indent HTML as nicely as it could. The problem seems to be that Nokogiri's #to_xml method behaves slightly differently from the #to_xhtml method.
Examples:
[4] pry(main)> puts Nokogiri::HTML("
whee").to_xml(indent: 2);
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">

whee

vs.
[5] pry(main)> puts Nokogiri::HTML("
whee").to_xhtml(indent: 2);
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">

        whee

The second one looks a lot nicer to me -- it's especially nice when the HTML document has very deep nesting.
What say you? :)

https://github.com/awesome-print/awesome_print/blob/master/lib/awesome_print/ext/nokogiri.rb


Unable to use Kubelogin with service principal login flow

Hi! I'm trying to use the Service principal login flow to login to my AAD enabled connected cluster to enable non interactive login for Azure/aks-set-context action. However, I've been getting this error:
server response object: [{
  "metadata": {},
  "status": "Failure",
  "message": "the server has asked for the client to provide credentials",
  "reason": "Unauthorized",
  "details": {
    "causes": [
      {
        "reason": "UnexpectedServerResponse",
        "message": "Security token validation failed."
      }
    ]
  },
  "code": 401
}]

Kubelogin works fine with interactive flow, but doesn't seem to work with service principals.
Also, I could not find any executables in the latest release. So, I used the makefile in the source code to get kubelogin.exe. Could that be a reason for this issue? Or is there anything that I am missing?

@shigupt202 have we discussed offline? Service Principal login support is determined in the backend. Since the current graph API call to get /getMemberGroups doesn't support Service Principal, the only way to support SPN is to configure Guard to skip Graph API call when there is no overage claim (e.g. your SP is in less than 250 AAD groups)
https://github.com/appscode/guard/blob/master/docs/guides/authenticator/azure.md#skip-graph-api-call-when-overage-indicator-is-not-present

@weinong I don't think we have. But, thanks for the clarification. I tried this by passing --azure.graph-call-on-overage-claim=true while configuring guard on my api server and it worked!


#1570 Add Examples Support

Support ApiResponse examples.
Summary
Transforms @Example.value() (i.e. @ExampleProperty[]) into a List<Example> that then will be transformed into a Map<String, Object> that then will be used to populate the examples map in io.swagger.models.Response
Details

Added a List<Example> to ResponseMessage
Added a small block of code in SwaggerResponseMessageReader (where an ApiResponse is transformed into a ResponseMessage) that will transform an ExampleProperty into an Example and then add it to the ResponseMessage using the field above.
Transform the List<Example> to a Map<String, Object> in ServiceModelToSwagger2Mapper, and populate the Response.examples with it.

Test

Added additional UT to ResponseMessageSpec, ResponseMessageBuilderSpec and ServiceModelToSwagger2MapperSpec to cover newly added functionality.
Added a ResponseExampleTestController that will be used by SwaggerResponseMessageReaderSpec to test examples support.
Added UT for new ExamplesMapper class.
Added a new endpoint in FeatureDemonstrationService (not sure if this is the right place), to test the contract using the contract tests.

Hi @dilipkrish, any news regarding this PR (or Q1nt's PR)?
Cheers!

@jasouyris thanks for the PR! 🤘

It's a pleasure! Thank you for all your time and work in this project, I don't know what'll do without it! 🤘


Change $type to $kind and move unit tests to declarative

Change $type to $kind
Add Json annotations to all properties
Add declarative form of  TestFlow (TestScript) which allows us to convert TestFlow unit tests to declarative .test.dialog files
Replace TestFlow() unit tests with .test.dialog declarative tets

Pull Request Test Coverage Report for Build 91842

0 of 0   changed or added relevant lines in 0 files are covered.
730 unchanged lines in 81 files lost coverage.
Overall coverage decreased (-1.0%) to 75.561%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Adapters/Microsoft.Bot.Builder.Adapters.Facebook/FacebookAdapter.cs
1
94.03%

/libraries/Adapters/Microsoft.Bot.Builder.Adapters.Facebook/FacebookAdapterOptions.cs
1
92.31%

/libraries/integration/Microsoft.Bot.Builder.Integration.ApplicationInsights.Core/ApplicationBuilderExtensions.cs
1
75.0%

/libraries/Microsoft.Bot.Builder.AI.LUIS/LuisApplication.cs
1
96.77%

/libraries/Microsoft.Bot.Builder.AI.LUIS/V3/LuisApplication.cs
1
97.37%

/libraries/Microsoft.Bot.Builder.AI.LUIS/V3/LuisUtil.cs
1
95.24%

/libraries/Microsoft.Bot.Builder/BotStateSet.cs
1
85.71%

/libraries/Microsoft.Bot.Builder.Dialogs/Debugging/SourceRange.cs
1
45.45%

/libraries/Microsoft.Bot.Builder.Dialogs/DialogContainer.cs
1
75.0%

/libraries/Microsoft.Bot.Builder.Dialogs/Prompts/PromptCultureModels.cs
1
96.88%

Totals

Change from base Build 91739:
-1.0%

Covered Lines:
8212

Relevant Lines:
10868

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.ApplicationInsights.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.AspNet.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Testing.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Streaming.dll compared against version 4.6.3


[BUILD] Bump actions/setup-python from v4 to v5

:mag: Description
Issue References 🔗

As described.
Describe Your Solution 🔧

setup-python  Github Action v5 release note: https://github.com/actions/setup-python/releases/tag/v5.0.0. Update node version runtime for the actions from node16 to node20.

Types of changes :bookmark:

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklists
📝 Author Self Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] This patch was not authored or co-authored using Generative Tooling

📝 Committer Pre-Merge Checklist

[ ] Pull request title is okay.
[ ] No license issues.
[ ] Milestone correctly set?
[ ] Test coverage is ok
[ ] Assignees are selected.
[ ] Minimum number of approvals
[ ] No changes are requested

Be nice. Be informative.

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

Comparison is base (79b24a7) 61.42% compared to head (f1703d6) 61.42%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5838      +/-   ##
============================================
- Coverage     61.42%   61.42%   -0.01%
  Complexity       23       23
============================================
  Files           608      608
  Lines         36094    36094
  Branches       4952     4952
============================================
- Hits          22172    22170       -2
  Misses        11524    11524
- Partials       2398     2400       +2

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

Thanks, merged to master.


Upgrade JDBC driver to Jackson 2.16.0 due to 2.15.0 being vulnerable to Denial of Service (CVE-2023-35116)

Current driver build uses Jackson 2.15.0
Please review and rebuild with Jackson 2.16.0. and update public page at
https://docs.aws.amazon.com/redshift/latest/mgmt/jdbc20-download-driver.html
CVE-2023-35116:
jackson-databind through 2.15.2 allows attackers to cause a denial of service or other unspecified impact via a crafted object that uses cyclic dependencies.
Fixed in 2.16.0

Thanks for bringing this to our attention @dqmdev . Will take a look and possibly include in the next release of the driver.

fixed in version <IP_ADDRESS>


emoji to high

thats it
emoji to high

Can't reproduce, can you share your device and browser info?

Can't reproduce either on latest Firefox, Chrome and Safari on MacOS 11.3

I also can't reproduce. Please provide proper information i.e. device information or steps to reproduce, otherwise I suggest to close this issue.

Can't reproduce

please provide more info

@abdpro300 Could you please tell us what browser and the version of it that you are using?

This just seems extremely stale, considering we switched to a completely new design.


Grep functionality breaks with relative test paths

I have my tests setup so a "helper" object will run through my "elements" directory and find test files, and will build an array to pass to WCT.loadSuites(). It works fine except until now it was passing in relative paths ("../elements/element1/tests/element1-test.html") relative to the test runner html file.
The testing seemed to work fine, but the grep functionality would return nothing whenever any link on the page was clicked.
I have since changed my helper script to build absolute paths and it works, however I'm wondering...
Is this a bug in WCT, or is this something that simply needs a note in the readme (Use full paths here, not relative...)?

Yeah, I'd like to treat this as a bug; it's definitely surprising, and should be fixable. Might be a while before we get around to it though

Is there a work around for it?


Unknown field type: 'V'

Some files created with FoxPro 9 can not be opened with dbfread.DBF("file.dbf")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "~/dbfread/dbf.py", line 120, in __init__
    self._check_headers()
  File "~/dbfread/dbf.py", line 259, in _check_headers
    raise ValueError('Unknown field type: {!r}'.format(field.type))
ValueError: Unknown field type: 'V'

Seems to be DBF format slightly updated
https://msdn.microsoft.com/en-us/library/st4a0s68(VS.80).aspx

dbfread should definitely support this. Do you have an example file I can
look at?

if I copy parseC in FieldParser.py and rename the copy parseV the error goes away and it doesn't seem to break anything.

Ah, that's great news. Thanks for letting me know.
I'll add support for this in the next release (which should be out any day
now).

I've added support for V fields to my develop branch.
@bobintetley I just added this to the parser class:
parseV = parseC

Should work for now.

Not sure if you can comment on closed issues. Reopening just in case.

I just released 2.0.6 with support for V fields.


Warn users on signup

Could be rewritten/styled more nicely, what do you think?
Also, should we put it on the login form as well?

Current coverage is 98.50% (diff: 100%)

Merging #178 into master will not change coverage

@@             master       #178   diff @@
==========================================
  Files           100        100
  Lines          1472       1472
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1450       1450
  Misses           22         22
  Partials          0          0

Powered by Codecov. Last update 2fae381...25ebc2a

should be a little bit more prettier now and yep, i think it should be on the login form as well, so that the people keep it in their mind.
Probably a hint how to contribute would still be great!

What about a link to our github repo with the github logo?
And a hint to https://github.com/yunity/foodsaving-frontend/blob/master/CONTRIBUTE.md ?

an github icon/link should be definitely always somewhere, but i think more about normal non-dev users having bugs or suggestions.
=> There is now the address<EMAIL_ADDRESS>which gets forwarded into slack.


Error when removing steps from material stepper with ngIf when the selectedIndex is greater than the updated stepper length

Bug, feature request, or proposal:
bug
Description
I am making a material stepper where the subsequent steps change dynamically depending on which radio button option a user selects in their current step. When the user goes back and changes a selection, which then reduces the number of subsequent steps from what it was before, I am experiencing an error. More specifically, this error occurs when the number of newly rendered steps is less than the selectedIndex of the previously rendered steps.
When the new steps are rendered, they are all closed (none are set as the selectedIndex). When I click on one of the steps I get this error:
ERROR TypeError: Cannot set property 'interacted' of undefined

When you click on a step it is calling this function which changes the selectedIndex of the stepper:
set selectedIndex(index) {
        const newIndex = coerceNumberProperty(index);
        if (this.steps && this._steps) {
            // Ensure that the index can't be out of bounds.
            if ((newIndex < 0 || newIndex > this.steps.length - 1) &&
                (typeof ngDevMode === 'undefined' || ngDevMode)) {
                throw Error('cdkStepper: Cannot assign out-of-bounds value to `selectedIndex`.');
            }
            if (this._selectedIndex != newIndex && !this._anyControlsInvalidOrPending(newIndex) &&
                (newIndex >= this._selectedIndex || this.steps.toArray()[newIndex].editable)) {
                this._updateSelectedItemIndex(index);
            }
        }
        else {
            this._selectedIndex = newIndex;
        }
    }

In my case, we are hitting the line that calls this function:
_anyControlsInvalidOrPending(index) {
        const steps = this.steps.toArray();
        steps[this._selectedIndex].interacted = true;
        if (this._linear && index >= 0) {
            return steps.slice(0, index).some(step => {
                const control = step.stepControl;
                const isIncomplete = control ? (control.invalid || control.pending || !step.interacted) : !step.completed;
                return isIncomplete && !step.optional && !step._completedOverride;
            });
        }
        return false;
    }

This line is causing the error:
steps[this._selectedIndex].interacted = true;

At this point, this._selectedIndex is the index we are moving away from, since it hasn't been set to the new value yet. So it is trying to set the interacted property of the previous step (the one you are leaving), but in this case that step no longer exists in the DOM due to the ngIf and, therefore, this line is causing an undefined error and crashing.
Seems like there should be a way to account for this situation where the previously selected step no longer exists. Maybe just an undefined check that only updates the interacted property if the step still exists in the DOM?

I believe this is the same issue as https://github.com/angular/components/issues/20735 which has a pending PR.


[ncp] add service spinel bindings

I will begin working on this. Please assign to me.
Firstly, a couple of questions:

The most suitable Spinel property? SPINEL_PROP_THREAD_SERVICE = SPINEL_PROP_THREAD__BEGIN + 15?
Should adding/removing a property over spinel trigger otServerRegister() or should this be done separately (like the CLI)?
We want the ability to add, remove, list all services & list all leader services?

With reference to the above, I have decided to go with adding a new group of spinel properties
    SPINEL_PROP_SERVER__BEGIN = 0x90,

    SPINEL_PROP_SERVER_SERVICES = SPINEL_PROP_SERVER__BEGIN + 0,

    SPINEL_PROP_SERVER_ALLOW_LOCAL_DATA_CHANGE = SPINEL_PROP_SERVER__BEGIN + 1,

    SPINEL_PROP_SERVER__END = 0xA0,

@josephn123 can you outline a bit how you're going to implement this, please?
Generally it should be possible to register a number of services, providing:
a) enterprise number
b) service data (sequence of bytes)
c) server data (sequence of bytes)
d) stable flag
for each service.
Theoretically it should be possible to register a number of servers (multiple server data) for single service, though such flexibility will probably not be necessary in most use cases.
I'm not sure how the spinel props you listed above map to those data.

And - it should be possible to read:
a) whether service is successfully registered or not (i.e. confirmed by leader)
b) what aloc was assigned
The above can be read from network data, though it would be nice to not force user to parse it manually.

@nopik I've edited the above comment with updated spinel props and documentation.
Essentially these just call the otServer* APIs listed in server_api.cpp

Resolved by #3659 and https://github.com/openthread/wpantund/pull/386.


synchronize in pool.py

Hello, GNNAutoScale is an interesting and solid work, and thanks for your hard work.
However, when running "synchronize = torch.ops.torch_geometric_autoscale.synchronize", it reports an error.
How to deal with this problem?

Did you run python setup.py install before executing the scripts?

Did you run python setup.py install before executing the scripts?

Yes, and when running small_benchmark/main.py, it shows that a circular import happens.

Can you try to run the script not within the root folder, but within small_benchmark?
cd small_benchmark
python main.py

Can you try to run the script not within the root folder, but within small_benchmark?
cd small_benchmark
python main.py

Yep that's the way I run it.

This is weird. What's the installation log of
python setup.py clean && python setup.py develop


[TASK] Allow ExternalModels to return string data

Issue Description
Is your feature request related to a problem? Please describe.
Sometimes it would be useful for an ExternalModel to return some string data.
Currently this results in:
(    0.00 sec) EXTERNAL MODEL           : DEBUG           -> Unsupported type found. Available ones are: 'float', 'bool', 'int', 'ndarray', 'c1darray', 'float16', 'float32', 'float64', 'float128', 'int16', 'int32', 'int64', 'bool8', 'numpy.float', 'numpy.bool', 'numpy.int', 'numpy.ndarray', 'numpy.c1darray', 'numpy.float16', 'numpy.float32', 'numpy.float64', 'numpy.float128', 'numpy.int16', 'numpy.int32', 'numpy.int64', 'numpy.bool8'
(    0.00 sec) EXTERNAL MODEL           : DEBUG           -> variable code has an unsupported type -> str

Describe the solution you'd like
Extend ExternalModel to support str

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or task?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[x] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

checklist is good.


"this.$http.interceptors" doesnt work.

Hi there!
I'm trying to set interceptors on a Vue component but it doesn't work.
The requests are made on router loaded templates.
The provided example is a root component, an App entry point.
I do not get any errors, it simply does nothing...
Am I doing something wrong?
<style></style>
<template>
    <router-view></router-view>
</template>
<script>
    export default {
        ready(){
            this.$http.interceptors.push(
                {
                     request(req){
                        console.log('Intercepted REQ:', req);
                        return req;
                    },
                    response(res){
                        console.log('Intercepted RES:', res)
                        return res;
                    }
                }
            )
        }
    }
</script>

Found a solution:
<style></style>
<template>
    <router-view></router-view>
</template>
<script>
    import Vue from 'vue';
    export default {
        ready(){
            Vue.http.interceptors.push(
                {
                     request(req){
                        console.log('Intercepted REQ:', req);
                        return req;
                    },
                    response(res){
                        console.log('Intercepted RES:', res)
                        return res;
                    }
                }
            )
        }
    }
</script>


[BUG]

🙁 Actual behavior
No longer required.
🙂 Expected behaviour
No longer required.
🔢 Steps to Reproduce the Problem
No response

Issue Fixed.


macOS error UtimesNano: operation not permitted on sudo restic restore

Output of restic version
restic 0.16.2 compiled with go1.21.3 on darwin/amd64
What backend/service did you use to store the repository?
An S3 bucket on Kubernetes hosted on the internet, but within the EU.
Problem description / Steps to reproduce
I'm using (Intel) macOS Sonoma patched to
$ sw_vers
ProductName:	macOS
ProductVersion:	14.0
BuildVersion:	23A344

with System Integrity Protection (SIP) enabled
$ csrutil status
System Integrity Protection status: enabled.

and iTerm2 (Build 3.4.22). I found this hint that enabled SIP could cause restic problems, the suggested solution I found was to use macOS full diskaccess. That's why I configured macOS full diskaccess for
/System/Applications/Utilities/Terminal.app
/Applications/iTerm.app
/usr/local/bin/restic
/usr/local/Cellar/restic/0.16.2/bin/restic

iTerm2 runs my bash that interacts with restic.
Initialize the repository:
export sudoPW="secret";
export RESTIC_PASSWORD="secret";
export AWS_ACCESS_KEY_ID="secret";
export AWS_SECRET_ACCESS_KEY="secret";
export UserName="$(stat -f%Su /dev/console)";
export RealNameSpaced="$(dscl . -read /Users/$UserName RealName | cut -d: -f2 | sed -e 's/^[ \t]*//' | grep -v "^$" | awk '{print tolower($0)}')";
export RealNameDotted=${RealNameSpaced// /.};
export S3_Bucket_MyBase_URL="s3:https://s3.eu-de.cloud-object-storage.appdomain.cloud/backup-${RealNameDotted}";
export local_iPad_Backup="/Users/$USER/Library/Application Support/MobileSync/Backup/";
restic -r "$S3_Bucket_MyBase_URL/iPad_backup" init;

Backup:
echo $sudoPW | sudo -p "" -S -E bash -c 'restic --repo "$S3_Bucket_MyBase_URL/iPad_backup" backup "$local_iPad_Backup"

Restore:
echo $sudoPW | sudo -p "" -S -E bash -c 'restic -r $S3_Bucket_MyBase_URL/iPad_backup --verbose=2 restore latest --target "/"'

Restic executed with sudo has an error upon restoring my restic iPad backup. Restics error message doesn't help me to find the reason for the error. The options --verbose and --verbose=2 doesn't change restics restore output to be more detailed.
Expected behavior
Upon restics restore I'd expect restic to reproduce all data exactly as they were prior the backup. The options --verbose and --verbose=2 should change restics output to be more detailed.
Actual behavior
$ echo $sudoPW | sudo -p "" -S -E bash -c 'restic -r $S3_Bucket_MyBase_URL/iPad_backup --verbose=2 restore latest --target "/"'
repository 535a9ac9 opened (version 2, compression level auto)
[0:00] 0.00%  0 / 1 index files loaded
restoring <Snapshot 0ea37a4c of [/Users/username/Library/Application Support/MobileSync/Backup] at 2023-11-07 13:48:49.171048 +0100 CET by root@hostname> to /
ignoring error for /Users: UtimesNano: operation not permitted
Summary: Restored 3377 / 3378 files/dirs (583.808 MiB / 583.808 MiB) in 0:33
Fatal: There were 1 errors

The ouput stays the same if I don't use --verbose or --verbose=2.
$ echo $sudoPW | sudo -p "" -S -E bash -c 'restic -r $S3_Bucket_MyBase_URL/iPad_backup restore latest --target "/"'
repository 535a9ac9 opened (version 2, compression level auto)
[0:00] 0.00%  0 / 1 index files loaded
restoring <Snapshot 0ea37a4c of [/Users/username/Library/Application Support/MobileSync/Backup] at 2023-11-07 13:48:49.171048 +0100 CET by root@hostname> to /
ignoring error for /Users: UtimesNano: operation not permitted
Summary: Restored 3377 / 3378 files/dirs (583.808 MiB / 583.808 MiB) in 0:33
Fatal: There were 1 errors

Do you have any idea what may have caused this?
If I backup my iPad (iOS patched to latest, not jailbreaked) with the macOS Finder function to my MacBook that I run restic from. MacOS always uses the fixed path
/Users/$USER/Library/Application Support/MobileSync/Backup/

I think something in this path could interfere with the enabled SIP. Disabling SIP is not a solution for me, because I have to obey a rule to leave SIP enabled.
Did restic help you today? Did it make you happy in any way?
I'm new to restic, but until now I had more problems using restic than restic solved for me. But I would love to use restic instead of rsync for backups. 🥲

I found here, that macOSs SIP does not protect /Users, then maybe the path to the iPad backup isn't the issue here?

I'd say you are generally speaking not supposed to fiddle with /Users on macOS.
To not do that, it'd be better to restore the data to your ~ and move the relevant directories from it into their real place.
Or you can use the <snapshot>:<subfolder> syntax described here to restore only the relevant part of the backed up path to where you want it, e.g. (not tested) $ echo $sudoPW | sudo -p "" -S -E bash -c 'restic -r $S3_Bucket_MyBase_URL/iPad_backup restore --target ~ latest:/Users/$USER'. Presuming the expansion works as expected, otherwise try with "$HOME" instead of ~.
The latter should restore only the /Library/Application Support/MobileSync/Backup part of the backed up path /Users/username/Library/Application Support/MobileSync/Backup into/under your ~. Try it and see it if works!

Thank you very much for your reply, @rawtaz. I will try your suggestions. Where did you get from that SIP protects /Users, I can't find that here.

I had success recovering my iPad backup without restic showing errors. Because the Target is inside me home directory I executed the restore as my user, simply without sudo.
restic -r $S3_Bucket_MyBase_URL/iPad_backup --verbose=2 restore latest --target "/"
But I'm still confused /Users is not a directory that is listed to be protected by SIP and in a restore command verbose doesn't do anything. Who can shed some light on this?

Atm I'm uncertain if the following works with enabled SIP. Does anybody know? I'll continue testing this next week. 🙂
mkdir $TMPDIR/iPad_backup
restic -r $S3_Bucket_MyBase_URL/iPad_backup restore latest --target "$TMPDIR/iPad_backup"
echo $sudoPW | sudo -p "" -S -E bash -c 'mv -f $TMPDIR/iPad_backup/*.* /'

I don't think there's anything that can be done on restic's side about this. Ideally, don't restore to / which is rather dangerous anyways.


how did you get recoil values?

how did you manage to get the exact values for recoil (specs.json)?
my only guess currently would be view angles.

Hi @nullify0000 , yes I have used +cl_showpos 1 to see in-game angles and then converted them to mouse pixels.


Add Responsive Dynamic Form

Add Responsive Dynamic Form
Fixes #3
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Screenshots (if appropriate):
For the confirmation message.
https://github.com/user-attachments/assets/275fabdb-b76a-4cc7-8bb5-f3e37fea9c38
Checklist:

[x] I have registered myself at Contrihub website.
[x] My code follows the code style of this project.
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

The website logo is not showing have u did it separately?


replace xml/html files with asciidoc files for rendering on github

Since moving the styleguides from googlecode to github, the one written in xml have become unreadable. Those written in html have always been very bad to edit and fix.
This PR applies the transformation suggested by me in #36 (1st comment).

on the other hand, asciidoc does not serve as input to pandoc, not has github whitelisted the jekyll asciidoc plugin, so markdown is probably a better choice.

What happened to this change? Would not the use of Asciidoc instead of HTML be much better for future maintenance. You could build both HTML and PDF from Asciidoc.
We use the Google Java Code Standard, with some minor adjustment (indentation for instance). It would be much easier for use to copy the standard if it where written in Asciidoc.
We could however use Pandoc to produce Asciidoc from the HTML, but it does not produce a 100% correct output.


Add warning and screen negative norms in SC-NEVPT2

It turns out that even with deterministic solvers, some of the perturber states have negative norm.
Now all the contributions with negative norms are screened and a warning is printed.
Here is an output of CAS(2,2) case for system including two Cu2+ with cc-pvdz basis
CASCI E = -3851.42285404094  E(CI) = -1.21547445991519  S^2 = 0.0000000
Density matrix diagonal elements [1.08742974 0.91257026]
Inactive and virtual orbitals were pseudo-canonicalized for NEVPT
Sr    (-1)',   E = -0.00003725711411
Si    (+1)',   E = -0.00002053164168
Sijrs (0)  ,   E = -2.29929845036311

WARN: 26075 perturber states have negative norm
Largest negative norm is -0.00032381144271730663
Sum of negative norms is -0.021714265535472073

Sijr  (+1) ,   E = -0.10135929403462

WARN: 66729 perturber states have negative norm
Largest negative norm is -0.00032158452641913254
Sum of negative norms is -0.04182706487968975

Srsi  (-1) ,   E = -0.16495261735336
Srs   (-2) ,   E = -0.00006119692458
Sij   (+2) ,   E = -0.00004437366205
Sir   (0)' ,   E = -0.02011240527878
Nevpt2 Energy = -2.585886126372277
:: NEVPT2 for sing -3854.008740

But even C2 example gives negative norms.
This PR is only the first step to the actual fix. The negative norms show up for Sijr(+1) Srsi(-1), which seem to be prone to numerical instabilities. @sunqm , do you have an idea?

Good to have this check. I don't understand why this happens. Is there a theoretical reason or a signal of bug in the program? Is this observed in other NEVPT2 implementations?

The norm has to be positive, but the way it is evaluated in the code
involves lots of addition and subtraction. Someone should really check
against DALTON or Molpro ...
On Fri, Oct 30, 2020 at 8:19 PM Qiming Sun<EMAIL_ADDRESS>wrote:

Good to have this check. I don't understand why this happens. Is there a
theoretical reason or a signal of bug in the program? Is this observed in
other NEVPT2 implementations?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/pull/737#issuecomment-719875899, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN5NRDFCTLCEOHXDHRIG6TSNN653ANCNFSM4SQRAGUA
.


TypeError: encode() missing 1 required positional argument: 'texts'

hello, I always get this error
TypeError: encode() missing 1 required positional argument: 'texts'
on this line:
vecs = en.encode(texts=['hello aman', 'how are you?'])
I don't understand why even though texts isn't missing ?

@IS5882 Hi, First of all thank you for raising the issue, here we got the issue and fixed it in the latest release 1.0.0
just upgrade the current package following below command
pip install embedding-as-service==1.0.0
Hope this will resolve the issue.
Thanks Again