Is broken with openzeppelin

https://github.com/OpenZeppelin/openzeppelin-solidity
correct instruction is mentioned here:
https://github.com/OpenZeppelin/openzeppelin-solidity/issues/1342
everything is compiled but your extension shows errors with import. So at least how to make it ignoring import directives?

Can you post a screenshot of the warning / problem? Is this due to including truffle's test?

@juanfranblanco

You need to add this to your settings:
    "solidity.packageDefaultDependenciesContractsDirectory": "",
    "solidity.packageDefaultDependenciesDirectory": "node_modules",

Thanks @ajhodges , it was by default  "solidity.packageDefaultDependenciesContractsDirectory": "contracts", . I was fighting to find the right solution 🔨


Fix indentation

@Silvest89 Good catch!! Yaml is always tricky 😅

@mysticaltech especially when there is no proper linting for these template files

Yes, good point! We should definitely add it.

@Silvest89 Still we have an issue. It turns out HCLOUD_INIT_CONFIG cannot be blank, the autoscaler pod fails to start with it blank.

@Silvest89  It does not work when HCLOUD_INIT_CONFIG is removed, (as this is necessary the node to boot correctly and get all set up).
I'm going to remove the check on isUsingLegacyConfig. Will post the PR shortly for your analysis.
Here's the error:

Ah I see, you also include the cloud config but with index. Now I understand better. In that case, you may want to check for yourself please.

Ah we are using the old autoscaler version in the project, it's set as default to v1.27.3. But I'm seeing the new version has not been released yet in the format that we use, via manifest. Or has it? https://github.com/kubernetes/autoscaler/releases

I see no trace of new container released for the latest version in the releases of autoscaler. As we use the manifest, it calls the container directly.
However, maybe if we install via helm, it could work? I will try also search for the updated container in the helm release.

The default tag in their latest helm release even uses an older version that we use.

@Silvest89 All in all, maybe they are waiting for the next v1.28.x to release what your merged? If you can think of a workaround please let me know.


READY FOR MERGE: Added code to automatically configure Guacamole web app with Vagrant box connection details and user logins

It looks like there's a new class here. Could we add spec tests for this? So far most of the tests for this repo are pretty minimal "does it compile," but I think it would be good to get into the practice of adding at least that level of testing to new features.

Sure thing @kjhenner - will do

Looks like all the tests in this module are currently failing @kjhenner. Seems all related to hiera


fix(cheatcodes): inspector call

Motivation
Fixes #8015

Solution
Use call.bytecode_address instead of call.target_address, doing this accounts for delegatecalls

Can we add a test for this?

Can we add a test for this?

Yeah

can we add this repro as a test @yash-atreya ?
https://github.com/foundry-rs/foundry/issues/8015#issuecomment-2142186857

Great! This fixes both of my test cases! Thanks!


I get vac banned in 1 day (12/04/20)

i think that you need to use vmprotect and junk code.
Dont forget to change the junk code every week only to be sure..
i hope my commentary where useful for you

i think that you need to use vmprotect and junk code.

 Thank´s for reading

i think that you need to use vmprotect and junk code.
 Thank´s for reading

i dont think you understand how vac even works, you can use any injector even loadlibraryA works, junkcode will do nothing nor will vmprotect do something

duplicate of #27 / out of scope

duplicate of #27 / out of scope


[CompilerPerf] Fix large ranges

The F# compiler has historically used a 32-bit integer for "position" information and a 64-bit integer for "range" information.   But this means we lose information and can only accurately handle:

file references up to 16384 referenced files (across all projects being analysed by FSharp.Compiler.Service)
line length up to 512
file length up to 65536
intra-file range spans up to 32768

So we have bugs where long lines and/or long files and/or lots-of-files give inaccurate information because we compress the range information into a 64-bit integer.
This PR is a possible fix for this by using 128-bits. This gives limits:

file references up to ~16million referenced files (across all projects being analysed by FSharp.Compiler.Service)
line length up to ~1million
file length up to ~2billion

However I’m concerned about perf implications of using 128bits for range values since these get passed around everywhere. I will test perf on this PR and will compare with other compilers such as Roslyn.
(Quote of the day: "F# file lengths of 640K should be enough for anyone!")

@dsyme, conflicts to resolve.  How do you propose investigating the perf implications you mention?

@dsyme, conflicts to resolve. How do you propose investigating the perf implications you mention?

@KevinRansom Good question.  I ran the compiler perf script a few times and it seems to be slower by around 3-5%, which is kind of what I expected.   BUT, I'm always a bit sceptical of the reliability of the data collection in that compiler perf script....
It's possible we should just do this properly by changing to use the character-offset-from-start-of-file.  That is, however, a more intrusive change....

It's possible we should just do this properly by changing to use the character-offset-from-start-of-file. That is, however, a more intrusive change....

That would be great.

Great work! I don't think you should let the 3% performance drop stop you from merging this, it solves quite a few stability problems already. And for code-generation this is great, because sometimes these files get much larger than 64k lines.
This would solve some really old issues, for instance with source stepping, i.e. #758 from 2015 (I know there were others, but couldn't locate them anymore).
It also has a StackOverflow reference: https://stackoverflow.com/questions/33794570.

@dsyme I'd like to consider this for dev16, as the issue in #758 implies this is the right stability fix, and will also help and other tooling that do things that aren't AST-based

@dsyme, sorry mate resolution required :-(

@dsyme merge issues :-)

After talking with @dsyme about this, the approach here is fundamentally flawed, and packing in more information here will result in compiler and tooling slowdowns that we're hesitant to accept. The right way to store this information is as an offset, calculating range information as needed rather than flowing that larger amount of data everywhere. Such a change is more invasive, but the correct approach.
@dsyme do you want to keep this open?

After talking with @dsyme about this, the approach here is fundamentally flawed, and packing in more information here will result in compiler and tooling slowdowns that we're hesitant to accept.

I do think the slight perf slow down is probably still acceptable given we just have to fix this problem.  I'll bring this up to date and do some more perf checks.

This is ready, I think we should merge it and just be done with the issue.  We have to be correct in these cases and the indications above are that performance is reasonable.

Created an issue to track that we need perf improvements in the future: https://github.com/Microsoft/visualfsharp/issues/5826


Bulk query Repeat/Union/Coalesce/Where Steps

Closing this, I'll open a different issue for RepeatStep.


error happens when running 'vot evaluate --workspace ./ vipt_deep'

Thanks for your excellent work.
When I try to test ViPT on depthtrack dataset using vot-toolkit, error occured.
error 'vot.workspace.WorkspaceException: Experiment stack does not exist' happened when running 'vot evaluate --workspace ./ vipt_deep',
Could you please tell me how to solve this error?

The problem looks like you did not install or successfully install vot-toolkit.

Thanks for your reply. I'v already solved this error.
I found the reason is that there is not votrgbd2021.yaml file in the sequences directory, so that got stack_file==None.
And I'v successfully evaluated ViPT on depthtrack, I've trained the model for 30 epoch with the initial deep_rgbd.yaml. (because I found that the total loss hovered around 0.72 when epoch > 30)
Got the results pr(0.557) re(0.560) f(0.559)
Will I get result more similar to （0.592 0.596 0.594）in paper as I continue training the model with any changes?

Yeah, similar results will be gained if maintaining a similar experimental setup as in our paper.

OK, thanks a lot.
By the way, would you mind to provide the training log files?

No problem, the training logs have been uploaded.
Thanks for your attention in our work~

Thanks for your reply. I'v already solved this error. I found the reason is that there is not votrgbd2021.yaml file in the sequences directory, so that got stack_file==None. And I'v successfully evaluated ViPT on depthtrack, I've trained the model for 30 epoch with the initial deep_rgbd.yaml. (because I found that the total loss hovered around 0.72 when epoch > 30) Got the results pr(0.557) re(0.560) f(0.559) Will I get result more similar to （0.592 0.596 0.594）in paper as I continue training the model with any changes?

hey, I have the same error. Could you please send me the votrgbd2021.yaml？

Thanks for your reply. I'v already solved this error. I found the reason is that there is not votrgbd2021.yaml file in the sequences directory, so that got stack_file==None. And I'v successfully evaluated ViPT on depthtrack, I've trained the model for 30 epoch with the initial deep_rgbd.yaml. (because I found that the total loss hovered around 0.72 when epoch > 30) Got the results pr(0.557) re(0.560) f(0.559) Will I get result more similar to （0.592 0.596 0.594）in paper as I continue training the model with any changes?

hey, I have the same error. Could you please send me the votrgbd2021.yaml？
you can get yaml files on https://github.com/votchallenge/toolkit/tree/master/vot/stack. and I just specify the file path(stack_file) at anaconda\envs\depth38\Lib\site-packages\vot_toolkit-0.5.3-py3.8.egg\vot\workspace_init_.py line 34. I wish it can help you.

Thanks for your reply. I'v already solved this error. I found the reason is that there is not votrgbd2021.yaml file in the sequences directory, so that got stack_file==None. And I'v successfully evaluated ViPT on depthtrack, I've trained the model for 30 epoch with the initial deep_rgbd.yaml. (because I found that the total loss hovered around 0.72 when epoch > 30) Got the results pr(0.557) re(0.560) f(0.559) Will I get result more similar to （0.592 0.596 0.594）in paper as I continue training the model with any changes?

hey, I have the same error. Could you please send me the votrgbd2021.yaml？

you can get yaml files on https://github.com/votchallenge/toolkit/tree/master/vot/stack. and I just specify the file path(stack_file) at anaconda\envs\depth38\Lib\site-packages\vot_toolkit-0.5.3-py3.8.egg\vot\workspace_init_.py line 34. I wish it can help you.

Hi, I met the same problem. And when I specify the file path(stack_file), a new error occured:

Have you met this erorr?

Hi, I met the same problem. And when I specify the file path(stack_file), a new error occured:
Have you met this erorr?

i have not met this error, maybe you can try this, put stack_file in Depthtrack_workspace without specifying it, like ViPT/Depthtrack_workspace/votrgbd2021.yaml.

Thanks for your reply. Unfortunately, I tried it but the error still exists. :(

@tdd233 please use the vot-toolkit==0.5.3, nowadays it has been updated for VOTS2023 challenge if you install the latest version.


add 'chkc' to schema check logic

lets just merge this, later it should probably find the folders automatically...

Agree


[WIP] Remove tweedledum from requirements-dev.txt

Summary
Reverts https://github.com/Qiskit/qiskit-terra/pull/9477, now that we've solved both root causes via https://github.com/Qiskit/qiskit-terra/pull/9758 and https://github.com/Qiskit/qiskit-terra/pull/9754.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 19 (0.0%)  changed or added relevant lines in 4 files are covered.
251 unchanged lines in 12 files lost coverage.
Overall coverage decreased (-0.4%) to 84.954%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/circuit/classicalfunction/classical_function_visitor.py
0
3
0.0%

qiskit/circuit/classicalfunction/classicalfunction.py
0
5
0.0%

qiskit/circuit/classicalfunction/utils.py
0
5
0.0%

qiskit/circuit/classicalfunction/boolean_expression.py
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

qiskit/algorithms/amplitude_amplifiers/amplification_problem.py
1
84.72%

qiskit/algorithms/amplitude_amplifiers/amplitude_amplifier.py
1
94.0%

qiskit/visualization/circuit/_utils.py
2
93.26%

qiskit/circuit/classicalfunction/types.py
4
0%

qiskit/circuit/classicalfunction/exceptions.py
8
0%

qiskit/circuit/classicalfunction/classical_element.py
9
0%

qiskit/circuit/classicalfunction/init.py
9
0%

qiskit/circuit/library/phase_oracle.py
16
38.24%

qiskit/circuit/classicalfunction/utils.py
24
0%

qiskit/circuit/classicalfunction/boolean_expression.py
35
0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
67360

Relevant Lines:
79290

💛  - Coveralls


how to improve PDF to HTML conversion

Currently the Semantic Climate project converts PDFs to HTML.
The content is the IPPC Climate report AR6 and we need to improve is markup for further semantic annotation, resuse, and presentation. From a typesetting perspective and freeing us from descructive reliance on PDF (note we can get PDF like results in a non-descructive way using Vivliostyle) - that's me @mrchristian I would like to produce HTMl that could be rendered in Vivliostyle better than this.
The output needs improvement. Currently it contained a number of elements which may not be needed, e.g., page numbers, inline styles, etc.
The objective would be to improve the output with tooling that can integrate with the current workflow.
The suggestion would be to create a way to evaluate the process by collating information on the issue:

Current tooling
Condition of the source PDFs
Problems with outputs
List of parts and markup that we need to retain their integrity
Define what we want in out target outputs
Do we want other output formats for richer markup and other interoperability
List and evaluate tools
Consult experts in the field: pandoc, le-tex, fidus, vivlio, css-rocks, etc

This research can be conducted in a wiki page on the Semantic Climate repository.
Here are sample files:
PDF source - Chapter 8 https://github.com/petermr/semanticClimate/blob/main/ipcc/ar6/wg3/Chapter08/fulltext.pdf
HTML full text - Chapter 8 https://github.com/petermr/semanticClimate/blob/main/ipcc/ar6/wg3/Chapter08/fulltext.html
Tasks

Link to current PDF to HTML tooling.
Consult Single Source Publishing Community https://github.com/singlesourcepub/community/discussions and others: le-tex, pandoc, css rocks?

Thanks very much Simon!
On Sat, Sep 24, 2022 at 2:18 PM Simon Worthington @.***>
wrote:

Currently the Semantic Climate project converts PDFs to HTML.
The output needs improvement. Currently it contained a number of elements
which may not be needed, e.g., page numbers, inline styles, etc.

py4ami code does remove these (by  geometry) but yesterday it stopped
working. This may be due to the pdfminer libraries which I think are
not loaded in Colab in the same way as my machine. The key thing that any
PDF tool must do (maybe include Phillip) are:

preserve character entities
preserve styles (mainly font family)
preserve fill-color (often the only indication of bold)
preserve coordinates (critical for sub/superscripts and math
preserve image-coordinates
(Very few out-of-the box system do all of these)

pdfminer is hacky - it's partially abandonware. pdfplumber is layered on
top. The colour is not transmitted well. I used to use PDFBox (Java) which
is much better. But that's hard for an interactive session. It needs a
person-month from someone who understands this
The objective would be to improve the output with tooling that can

integrate with the current workflow.

The suggestion would be to create a way to evaluate the process by
collating information on the issue:

Current tooling
Condition of the source PDFs
Problems with outputs
List of parts and markup that we need to retain their integrity
Define what we want in out target outputs
Do we want other output formats for richer markup and other
interoperability
List and evaluate tools
Consult experts in the field: pandoc, le-tex, fidus, vivlio,
css-rocks, etc

This research can be conducted in a wiki page on the Semantic Climate
repository.

At the moment the only person who understands the whole PDF process is me.
I am not aware of others who I can collaborate with. So I'd dearly love a
volunteer.

Here are sample files:
PDF source - Chapter 8
https://github.com/petermr/semanticClimate/blob/main/ipcc/ar6/wg3/Chapter08/fulltext.pdf
HTML full text - Chapter 8
https://github.com/petermr/semanticClimate/blob/main/ipcc/ar6/wg3/Chapter08/fulltext.html
Tasks

Link to current tool
Consult Single Source Publishing Community
https://github.com/singlesourcepub/community/discussions and others

—
Reply to this email directly, view it on GitHub
https://github.com/petermr/semanticClimate/issues/10, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFTCS5NM4YGHDBSK6RCVE3V735T3ANCNFSM6AAAAAAQUUTCJU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Peter Murray-Rust
Founder ContentMine.org
and
Reader Emeritus in Molecular Informatics
Dept. Of Chemistry, University of Cambridge, CB2 1EW, UK

Vivlio is now mainstream for our activites


Update Spring Boot

Updates catalog service, identity service, and payment service to Spring Boot 3

@selvasingh fyi - boot 3 upgrade.


How to enable multi-turn in azure bot

Hello everyone my name is Taniguchi.
i've created a QnA Maker, a knowledge base and them associated with a bot.
I've tested but the only thing missing is the multi-turn, by the documentation i ve to add a code in my bot to enable.
what is the code i have to add to enable multi turn ?

@TaniguchiSantos Thank you for your inquiry. Can you please provide a tutorial or any documentation that you are attempting to follow? As it stands, this is a perfect question for the MSDN or Stack Overflow forums.

@TaniguchiSantos Hi, please go ahead to post this question to Stack Overflow forum.


Add GlanceAPI purge db cronJob

Glance provides the command glance-manage db purge [1] to purge all soft deleted records.
This command should be executed periodically to avoid glance database becomes bigger by getting
filled by soft-deleted records.
[1] https://docs.openstack.org/glance/latest/admin/db.html#purging-the-database

Looks good: https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openstack-k8s-operators_glance-operator/325/pull-ci-openstack-k8s-operators-glance-operator-main-glance-operator-build-deploy-tempest/1707712844939661312/artifacts/glance-operator-build-deploy-tempest/openstack-k8s-operators-gather/artifacts/must-gather/quay-io-openstack-k8s-operators-openstack-must-gather-sha256-2404fa785ab34cd4f7cdea105c0236936016c31c84a89475cf2fb42ec6fe71ce/namespaces/openstack/cronjobs/glance-cron.yaml

Some of these jobs will be blocked until https://github.com/openstack-k8s-operators/install_yamls/pull/559 merges

/retest

/test glance-operator-build-deploy-tempest


COBOL identifier with hyphen

It would be quite cool to see if Tabnine can handle COBOL with its non-local identifier definitions...

@GitMensch thanks for bringing this back to the table, I am not sure that the PR will solve what you imagine that it solves. Can you provide an example so we can align expectations?

The expectation for now is only that if tabnine is thrown at COBOL it doesn't split SOME-VAR into two identifiers.
Adding support for COBOL itself would be quite cool and I would be available for questions and sample programs.


What is a motivation?

Hi @mmastrac! Thanks for a cool project, opensource and so on! I am working with a seo a little bit, and once one of my posts goes top in reddit, I see a lot of aggregators like yours link to me.
As a posts author I am only curios of two points:

what is general motivation for creating such aggregator if there are a lot of competitors which do the same?
do you apply some marketing or receive some SEO benefits from listing a links?
Just wondering because there are a lot of opinions like sites with a links might be treated as spammy from Google perspective

@ivictbor I created the project for myself and open-sourced it in case someone else wanted to play with it as well. I've been running it for about 10 years for my own use and a few hundred people also seem to make use of it. I don't get any benefits - in fact I pay out of pocket to run it without ads, but having an ad-free aggregator experience really makes my life better. :)

@mmastrac thanks for the answer! Yepp , makes sense for me, opensource is not always beneficial. I have a couple of same projects which aim is to help people, and maybe get some small amount of back PR in exchange.
Good luck 👍


Having issues running on 1.21 purpur

When starting server i am met with the message
[18:35:35 WARN]: [ShopChest] Server version not officially supported. Version: 'craftbukkit', Mappings : '229d7afc75b70a6c388337687ac4da1f'

....
[18:35:35 WARN]: [ShopChest] Disabling the plugin

what can I do to avoid this. And get the plugin running.
Cheers.

You need to have the latest release of the plugin to support 1.21. You can get it here

Thanks, but that's where I get confused as that is the version I am currently running. Is it something Purpur related.

The error message tells that this Minecraft version is not supported. However, when looking at the code, it clearly is and this should run fine. I will test it on a purpur server but if it doesn't work then it's some kind of black magic.

The plugin loads fine with purpur 1.21-2256-de2e7a7 and the latest ShopChest release. Try it on a fresh server and ensure you got the right jar.


Export RouteContext from library (v6)

It would be nice to gain access to the RouteContext and build some of our own abstractions similar to the new Routes component -- I would prefer using this package over a fork!

I have this use case as well. I have a UI library that is router-agnostic and with several components that support plugging into any router. With v5 I can easily use the context to know if React-Router is available.

I've always considered whatever is on context to be our private API, so just a fair warning that you're going to have to have a very compelling use case if you'd like us to export it. 😅 What's your use case @ajsmth?
What are you using useIsInRouter for @delucis? Shouldn't your components be able to be used with or without the router?
Our context API was never public in v5 @alflennik, that's what the double __ was for in __RouterContext. It's always been for internal use only.

hey there - it would open up other libraries to build their own abstractions similar to useRoutes. i understand the idea behind making it private, i suppose the motivation is that it’s easier to promote a library that builds off of react router internals rather than a new implementation of a router.

@mjackson In my case, if the component is used within a router, it should display some navigation controls and useNavigate, but outside a router it only displays some controls for the current view. For now, I’ve hacked together some alternative logic, but the solution would be cleaner and less brittle if there was a way to check the router context directly.
(I feel like best practices would suggest composing things better and having the router-dependent component in some place where the router is certain to be present, but with context and hooks this kind of deep-nested logic seems more common.)

@ajsmth Can you please say more about why you want to build your own useRoutes?
@delucis It sounds like a simple hook that returns a boolean would work in your case? Like your useIsInRouter hook?

@mjackson For my case, that would be perfect 👍

I have a somewhat valid use case - we have this outdated library that uses homemade Portals (react-toolbox, that we forked a while back for accessibility) to make Dialogs and pop-up components. The problem is that children no longer receive the context, so we have to provide it, something like:
<MyContext.Consumer>
  {
    context => (
      <Dialog>
        <MyContext.Provider value={context}>
          {children}
        </MyContext>
      </Dialog>
    )
  }
</MyContext.Consumer>

I haven't pinpointed how/why it breaks now, but until I can update this library to not break the React API, my workaround is to "bridge" the outer component and the children by forward the contexts. I've had to do it for pretty much every library we use that uses contexts (redux, apollo, i18n, ..).

@rcarton Thanks for explaining your use case. Unfortunately we can't support custom portal implementations. If I were you, I'd try to update to React.createPortal if possible, which should take care of the context forwarding issue.

Sounds good, in the meantime I'm using __RouterContext as a workaround (knowing that if it breaks, it's my fault for doing BadThings™).

@ajsmth Can you please say more about why you want to build your own useRoutes?
@delucis It sounds like a simple hook that returns a boolean would work in your case? Like your useIsInRouter hook?

@mjackson sure - from what I can tell useRoutes only renders the highest ranking route. Sometimes in a native application we would want to hold onto previously rendered routes to preserve there implicit UI (things like scroll position), I'm not sure how I would do that without modifying / customizing useRoutes

Same scenario as @alflennik, UI library that is router-agnostic and on v4. The private __RouterContex API was a workaround, would be good to have it in v6.

For future readers, the request is now being tracked here: https://github.com/ReactTraining/react-router/issues/7545

Sorry I found this issue a little late.
I exposed my use-case here : https://github.com/ReactTraining/react-router/issues/7933
tl;dr : I need the access RouteContext in order to forward the routing context to preserve relative links when using a Slot/Fill approach.


Added details to README's Quick Start

Added additional details to the README's Quick Start section to increase clarity. This section is now self-contained, i.e., a new contributor can have a fully-operational development environment without needing to read any further past this section (although they still should ;)

/lgtm


Update lombok to 1.18.2

Adds support for @Flogger and more.

@thekeenant Thanks. Published to Marketplace


Pallet EVM: Configureable mapping from gas to weight.

This PR makes pallet EVM's mapping from Ethereum gas to Substrate weight, configureable by the runtime author. It also provides a default implementation that is a 1:1 mapping (the previous hard-coded conversion).
This PR also changes pallet EVM's weight annotations so that they consider only the gas used, not the price of that gas. This is a correct change because weight is supposed to quantify compute time. That time is not affected by the price users are paying for it.
A potential TODO item. We should consider a weight refund for when the gas used is less than the gas limit. I'm open to doing that in this PR or in a followup.
cc @tgmichel @xlc

Closing because pallet_evm is not included in Frontier. I'll continue this work there.


[FR] Parallelism using worker threads

Is your feature request related to a problem? Please describe.
Sync code brickes/lags the music temporarily
Describe the solution you'd like
The only solution for this, is to use worker threads (AKA parallelism).
Cons: every function which communicates with the worker thread of music, has to be async.
new Player(
    client,
    {
        workerThreads: true,
    }
)

Describe alternatives you've considered
There's aren't any 😭
Additional Comments
Despacito

Not for now, maybe later - closing for now.


DO NOT MERGE! Mh/input change event test

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/wc-catalogue/blaze-elements/blob/master/docs/CONTRIBUTING.md#commit
[x] There are no linting errors and code is properly formatted (your run before push yarn ts:format && yarn ts:lint:fix)
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x")
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Other information:
This is just showcase for @davidrus. Do not merge

Closes by #151


chore: decouple controller version from package.json and couple to pod image version

Description
The Controller (Server) Version is inaccurate when the image was changed. In reality, the server should get the version based on the image on the pods. This decouples the server version from package.json since it will be deployed in the cluster to ensure the most accurate results.
How to test:

Pull down PR and run npm test
re-tag the pepr:dev image to the ironbank image: docker tag pepr:dev registry1.dso.mil/ironbank/opensource/defenseunicorns/pepr/controller:v0.31.1
import the new image into the cluster: k3d image import registry1.dso.mil/ironbank/opensource/defenseunicorns/pepr/controller:v0.31.1 -c pepr-dev
build the pepr-test-module: npx pepr build && kubectl apply -f dist
Change the images on the controllers:  k set image deploy/pepr-static-test server=registry1.dso.mil/ironbank/opensource/defenseunicorns/pepr/controller:v0.31.1 -n pepr-system &&  k set image deploy/pepr-static-test-watcher watcher=registry1.dso.mil/ironbank/opensource/defenseunicorns/pepr/controller:v0.31.1 -n pepr-system
Get the logs: k logs -n pepr-system deploy/pepr-static-test-watcher | egrep "Pepr Controller"

Related Issue
Fixes #836

Relates to #
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Other (security config, docs update, etc)

Checklist before merging

[x] Test, docs, adr added or updated as needed
[x] Contributor Guide Steps followed

https://github.com/defenseunicorns/pepr-zarf-agent/blob/a26c61dd2f3bc87be10b032156e44cc754433d1b/capabilities/helpers.ts

we are solving this a different way

addressed here: https://repo1.dso.mil/dsop/opensource/defenseunicorns/pepr/controller/-/merge_requests/92/diffs?resolved_conflicts=true


Simplify security groups

💡 Summary
Now that I know that the number of security groups that can be applied to an ENI is not a hard limit, I'd like to create a single cloudwatch_endpoint_client_sg security group, for example.  That will allow me to simplify the corresponding security group rules by dropping the new CIDR-based rule added in #143 and getting rid of all the other SG-based rules.  I will do something similar for the other VPC endpoint security groups.  Instances spun up via the Terraformer instance can then simply add themselves to those security groups as necessary to gain access to the necessary VPC endpoints.
This change will also allow me to get rid of the existing instance type SGs, except for those that actually require an explicit mention.  For example, the Debian desktop SG will probably be rendered unnecessary as it isn't mentioned explicitly outside of these endpoint SGs, but the Teamserver instance will still be becessary because certain other need to be instance types are allowed ingress to the Teamservers.
Motivation and context
"Simplify, simplify, simplify!" - Henry David Thoreau

This was done in #180.


Recreate Core's script_tests data-driven tests

See src/test/data/script_tests.json and src/test/script_tests.cpp.

Reopening to keep migrating tests


Add relANNIS to URN spec & resolver

I just need to know what the relANNIS URNs should look like, and I'll add them to the resolver.

There are too “ANNIS” objects to reference:

   The search interface with the corpus selected (using the base64 parameter, e.g. A22: https://corpling.uis.georgetown.edu/annis/scriptorium#_c=c2hlbm91dGUuYTIy)

   The relANNIS files on github, if someone wants to import their corpus into their own ANNIS instance (server or local kickstarter): https://github.com/CopticScriptorium/corpora/tree/master/shenoute-a22/shenoute.a22_relANNIS

I’m not sure if we need the second link as a separate thing from the general github link to the repo (then users can just pick what format they want). The first one is essential since it’s ontologically different and has a separate domain too. @ctschroeder what’s your preference?
From: Luke Hollis<EMAIL_ADDRESS>Sent: Tuesday, March 17, 2015 21:27
To: CopticScriptorium/cts
Subject: [cts] Add relANNIS to URN spec & resolver (#40)
I just need to know what the relANNIS URNs should look like, and I'll add them to the resolver.
—
Reply to this email directly or view it on GitHub https://github.com/CopticScriptorium/cts/issues/40 .  https://github.com/notifications/beacon/ACFlWyIOWgF9IRP4iRihiiQQxJFDiFHBks5n2MvbgaJpZM4DwYPS.gif

Hi,
On 3/17/15 7:48 PM, Amir Zeldes wrote:

There are too “ANNIS” objects to reference:

The search interface with the corpus selected (using the base64
parameter, e.g. A22:
https://corpling.uis.georgetown.edu/annis/scriptorium#_c=c2hlbm91dGUuYTIy)

The relANNIS files on github, if someone wants to import their
corpus into their own ANNIS instance (server or local kickstarter):
https://github.com/CopticScriptorium/corpora/tree/master/shenoute-a22/shenoute.a22_relANNIS

Yes, exactly, this is what I meant by the query and the files.  The
buttons to ANNIS on Luke's pages right now go to the search interface.

I’m not sure if we need the second link as a separate thing from the
general github link to the repo (then users can just pick what format
they want). The first one is essential since it’s ontologically
different and has a separate domain too. @ctschroeder what’s your
preference?

I'm going to distinguish between links and URNs:
We definitely need links to the search interface.
We definitely need URNs for the relANNIS files.  Because we should
provide canonical URNs for all of the manifestations of these documents
in their different formats and visualizations, and the URN for, say, AOF
( urn:cts:copticLit:shenoute.AOF )should bring a list of all AOF
instances -- including relANNIS.  If we need to bill this as a separate
thing, that's fine, but we should have resolvable relANNIS URNs, and
those should come up on any list of URNs for their corresponding
documents or classes of documents.  I apologize that these didn't make
it into Bridget's table.  They are basically the same as the PAULA and
TEI URNS -- just with relANNIS instead of PAULA/xml and TEI/xml in the
URNs, and going to
https://github.com/CopticScriptorium/corpora/tree/master/abraham/shenoute.abraham.our.father_relANNIS
instead of
https://github.com/CopticScriptorium/corpora/tree/master/abraham/shenoute.abraham.our.father_TEI.
My question is: do we need a canonical URN for the installation of a
corpus in our ANNIS interface?  (e.g.
urn:cts:copticLit:shenoute.AOF/corpling or
urn:cts:copticLit:shenoute.AOF/ANNIS) which people could cite, and which
would resolve to the relatively stable link
http://corpling.uis.georgetown.edu/annis/scriptorium#_c=c2hlbm91dGUuYWJyYWhhbS5vdXIuZmF0aGVy.
This is DEFINITELY outside the original project parameters.  But a good
question nonetheless, I think.  Not urgent, but something to put on the
longer list.
I think Amir's question is do we need to resolve the relANNIS canonical
URNs to the GitHub link and do we need to foreground that (or even list
it with a button) on the website as we do with TEI? We don't have
buttons for the PAULA XML URNs on Luke's setup right now.  I think
resolving it is a pretty trivial thing, no?  (It's a matter of adding it
to whatever table is underlying these mappings, right?)  Whether we have
a button on the website for every corpus is another question.  Currently
on our corpora page we link only to the "latest" release of the corpora
-- all of it:  PAULA, TEI, relANNIS all in one zipped downloadable
file.  Let me think about whether we need buttons for PAULA and
relANNIS.  But this is minor, to my mind.  We can always add the
buttons.  The URNS are the major thing.
Take care,
Carrie

From: Luke Hollis<EMAIL_ADDRESS>Sent: Tuesday, March 17, 2015 21:27
To: CopticScriptorium/cts
Subject: [cts] Add relANNIS to URN spec & resolver (#40)
I just need to know what the relANNIS URNs should look like, and I'll
add them to the resolver.
—
Reply to this email directly or view it on GitHub
https://github.com/CopticScriptorium/cts/issues/40 .
https://github.com/notifications/beacon/ACFlWyIOWgF9IRP4iRihiiQQxJFDiFHBks5n2MvbgaJpZM4DwYPS.gif
—
Reply to this email directly or view it on GitHub
https://github.com/CopticScriptorium/cts/issues/40#issuecomment-82697985.

--

C. Schroeder
Associate Professor, Religious and Classical Studies
The University of the Pacific
<EMAIL_ADDRESS>http://www.carrieschroeder.com
https://pacific.academia.edu/CarolineSchroeder
Coptic SCRIPTORIUM
http://www.copticscriptorium.org
Blog
http://www.earlymonasticism.org

Added a reIANNIS placeholder to the URN resolver beneath "Cite this document".  Can build on this when we make more decisions in the future.

Updated the reIANNIS and ANNIS UI URNs per consensus specificiations.  Let me how this looks!

yeah again I need to see a list of the URNs. We shouldn't be using doc names for the doc urns--we should be using something different.

reiANNIS is now relANNIS per @balmas's branch and commit: https://github.com/CopticScriptorium/cts/commit/6b14aa8397a187b3c2170dc51678a10cbc54c9b3

looks fine in the paths; need a relANNIS button though

relANNIS button is added to the other links on each document:


UI submodule expects default security group to be present

The UI submodule assumes the AWS default security group is present.
I receive the following error when using the UI submodule.
Error: no matching SecurityGroup found

  with module.metaflow_ui.data.aws_security_group.vpc_default,
  on .terraform/modules/metaflow_ui/modules/ui/data.tf line 5, in data "aws_security_group" "vpc_default":
   5: data "aws_security_group" "vpc_default" {

ui/data.tf line 5:
data "aws_security_group" "vpc_default" {
  name   = "default"
  vpc_id = var.metaflow_vpc_id
}

I believe the module should create its own security group rather than expect the AWS default to exist.

It appears this is never used anywhere in the terraform. I will open a PR to remove this as it is not used.

Makes sense, we should remove those references to default security groups throughout all modules really.

Makes sense, we should remove those references to default security groups throughout all modules really.

This reference in the UI submodule appears to be the only remaining reference from what I see.


Add Warning and Update Help text

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
az k8s-extension
General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

Hi @NarayanThiru, Please write the description of changes which can be perceived by customers into HISTORY.rst. If you want to release a new extension version, please update the version in setup.py as well.

Thanks for your review, @yonzhan !  In this update there is no functionality change.  I am only adding the warning about the response of some of the commands and additional help text for a couple of parameters.  Do you think I still need to update the History.RST?  Looking for your guidance on this.

Updated the history and created  a new version.


⚠ Beulah Revival Network has degraded performance

In 692c52f, Beulah Revival Network (https://beulah.clicki.digital) experienced degraded performance:

HTTP code: 200
Response time: 11354 ms

Resolved: Beulah Revival Network performance has improved in 1f8d8c1.


Fix typo in the spanish documentation

Signed-off-by: jesuslerma<EMAIL_ADDRESS>Description
Fix a typo in the Spanish documentation
This PR fixes #3905
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 04d24094aab5f0607f98c0786aa211ddf4d64a28 at: https://611b017db338d56c6859ba8c--meshery-docs-preview.netlify.app


Bug: No way to close Modals -> Small Modal -> "Signup modal with slider"

no further text

I changed them a bit so now the buttons are visible and you are able to close the modal. However, this is very experimental and as the modals in ionic are not really flexible(you can't style specifically by page, if you apply a style, it applies to all modals on the project), I'm not sure it it's optimal.


Need to find a more graceful way to reference the taskQueue instance.

Based on the current readme, the examples show my best attempt at using this class.  I feel there are some unconventional uses of javascript that I'd like to improve on.

Not sure this is the case anymore.


ZIP files contain no directory entries

Zip files produced by rules_pkg do not contain directory entries. For example, they look like this:
META-INF/MANIFEST.MF
xds/one.xsd
xds/two.xst

Although technically not required by the spec, a convention is to add directory entries to zip files (0 bytes, file name ending with /). A lot of tools seem to follow/implement this convention.
Expected:
META-INF/
META-INF/MANIFEST.MF
xds/
xds/one.xsd
xds/two.xst

Missing directory entries may cause issues with the Java ecosystem, which seems to heavily rely on the directory entries. For example, see this Spring Framework issue:
https://github.com/spring-projects/spring-framework/issues/28230#issuecomment-1324702762

Duplicate of #622.


Resize buffer disk dynamically

Today, a large-enough disk (which means, more than 200GB after compression) can't be exported with format converting. This CL fixed the problem.
Originally: Worker instance use a 200GB for both buffering and OS. Now: It creates a separate buffer disk starting from 200GB and will increase 25GB when there is not enough space during the exporing process.
Detail of the resizing logic:
200GB as initial size.
Check available space every 10s.
If available space is less than 25GB, then add 25GB more.
Unless it has reached the max size. Max size is source disk + 5GB.
Resizing logic is host in a separate shell script "disk_resizing_mon.sh", which runs in background.
OS is hosted in a default size disk, which is 10GB now. It helps to isolate buffer size from OS, because OS size can be different in the future.
This design won't cost too much space - it applies new space only when it's required.
It unblocks >200GB scenario for customers. Ideally it can support the max disk size of google cloud - 64TB. We need test to verify how big it can actually handle. Manual test for 400GB+ works well.
Test result 1: perf regression test. Disk size 198GB.
Originally: 88min Now: 78min. Basically the same level.
Test result 2: large disk (700GB before compression, 493GB after compression) test.
Originally: 100% fail. Now: 100% pass

Addressed comments and changed design of some details

Addressed comments, please have a look again :D

addressed all comments

replaced explicit calling to "bash" by assigning execution right with chmod
added error handling for mkfs/mount to prevent confusion output to enduser.

/unittests

/unittests


OAuth is for authorisation, not authentication

OAuth is for authorisation, not authentication.

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

ID: 535cb7a2-f177-7ba0-bb06-25bc540d9055
Version Independent ID: 5e79865e-3d8d-3f7f-cae4-2670b408870f
Content: OAUTH 2.0 authentication with Microsoft Entra ID - Microsoft Entra
Content Source: articles/active-directory/architecture/auth-oauth2.md
Service: active-directory
Sub-service: fundamentals
GitHub Login: @janicericketts
Microsoft Alias: jricketts

Corrected - see https://github.com/MicrosoftDocs/azure-docs-pr/pull/253599.

#please-close

@janicericketts Thanks for creating PR.


Push docker image to DockerHub and Azure Container Registry

In https://github.com/gomods/athens/pull/1126, we decided to pre-build an image and then docker pull it inside the Athens tests. This repository has that image 🎉
To make the Athens tests in Azure Pipelines as fast as possible, we need to definitely push this image to Azure Container Registry (ACR) so that the pull from the Athens pipeline tests is fast. We also should push to DockerHub so that this image is public just as the athens image itself is public

I am going to take this now

@marpio TL;DR the code for this in azure-pipelines.yml is 3/4 done.
https://github.com/gomods/windows-testing-docker/commit/720c1e98c9c51605ab857d2cc3f85681b32a3b92 adds the docker login and docker push steps.
It looks like I don't have admin access to the Azure pipeline for this project, so I wasn't able to add the credentials for dockerhub or ACR.
I also am not sure if the az CLI is pre-installed on the win1803 VM. If it's not, I'll need to do that so that az acr login --name $(acrName) works.
Let me know what you think!

@arschles
You should have the permissions now.
As for the az cli - I don't think it's installed but it should be as easy as choco install azure-cli


i18n support

it would be good if there's i18n support - especially Japanese because there are many Japanese custom charters

Implemented with #15


Ledger integration issue with extension

luap — Today at 1:09 AM
@jhl  @armaniferrante | Will never DM I think this commit (2 day old) and line: https://github.com/project-serum/spl-token-wallet/commit/699558770eff7d78c1fe3890eb4f83ee8a268309#diff-e8e48145c10ae1bd9815b5774808f98ab2f8830fb440d6c6f4a9621a3af08335R79 that was fixing a regression... introduced a regression for the extension version :stuck_out_tongue: In the context of the extension wallet is null, so it doesn't like the wallet.publicKey
found by @PaulBernier

What is the regression?

Yes

I think the fix here is to go from [wallet, wallet.publicKey, selectedWallet, selectedWallet.publicKey] to [wallet.?publicKey, selectedWallet.?publicKey] or something like that. I can fix this within the next few days.

@jhlx when I try to use my Ledger in sollet i get this error in the console:
DOMException: Unable to reset the device.
Does that have anything to do with this issue?

so the temporary workaround is to connect to the ledger using the sollet.io website version rather than the extension version? just confirming. don't want to end up with lost funds 😅

Correct. Ledger doesnt work with the extension at the moment.


SceneLoadedMessage seems to not work properly sometimes

in this specific situation i cannot get SceneHandler by type in a callback of SceneLoadedMessage, but after restart it works properly
maybe need to revise the call orders of the Traveler's LoadScene method

Fixed in ver. 0.2.4


Cell type to count and link reference-* relationships

If a customer references many bookings, when I view a customer or a list of customers, I want to be able to have a cell that shows:

this customer has three bookings
this customer has no bookings
this customer has five bookings

And when I click on link showing this information, I will go to the bookings view, with the list pre-filtered by the customer
This will require #20

Unnecessary now that bi directional relationships are done


Specify .NET Core 3.1 requirement

Adding the note that if you want to use an existing project for this tutorial, it must target .NET Core 3.1. This tutorial does not work with apps that target .NET 5.0 and higher.

@Cmohan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Cmohan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Updated the article as requested.

#please-merge

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Informal internal style guidance is to avoid "stacking" notes. We'll merge, but you might follow up to revise the consecutive notes. Thanks!

Thank you for the information. I had attempted to add my edit to the existing note to avoid stacking notes and was told that was against formatting guidance (See change thread above)

@scubaninja One option in this scenario is to use bullet points in a note:

[!NOTE]

Note 1
Note 2

Or you might otherwise incorporate the information into the content. Just for future reference or consideration . . . Thanks!

I'm assuming this .NET Core 3.1 note - is in reference to the Azure Dev Ops implementation, since it's part of this tutorial.
Anyone know if the Git Hub version of Azure Static Webaps will support a Blazor WASM app with .NET 5? or would this 3.1 reequipment be the same there too?


cannot round-trip key with odd characters

Trying to write some toml.
[section]
":sys_platform==\"win32\"" = ["sdl2_lib"]

I had several problems. First, had to quote the key (fair enough), second, when quoted incorrectly the toml parser thought the == separated the key and value, third, it wanted to parse "sdl2_lib" as a number or possibly a date.
Please let me know if this is a bug with uiri/toml or if I'm just writing it wrong. Thanks.

This does not round-trip.
toml.loads(toml.dumps({'foo=bar': ['sdl2_lib']}))

If you incorrectly quote the key, the value which is getting parsed is garbage which results in the parser exhausting its options for determining what type of value it got. The last type of value which gets parsed is a date or number, hence the error message which you saw.
Which version of toml are you using? I get:

import toml
a = {'foo=bar': ['sdl2_lib']}
a == toml.loads(toml.dumps(a))
True

Sorry for the delay in my response.

Whichever I installed from pypi at the time
On Jul 4, 2016 22:20, W Pearson<EMAIL_ADDRESS>wrote:If you incorrectly quote the key, the value which is getting parsed is garbage which results in the parser exhausting its options for determining what type of value it got. The last type of value which gets parsed is a date or number, hence the error message which you saw.
Which version of toml are you using? I get:

import toml
a = {'foo=bar': ['sdl2_lib']}
a == toml.loads(toml.dumps(a))
True

Sorry for the delay in my response.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.


Being cited for ShadowingOuterLocalVariable when the variable in question is in a different branch of a conditional statement

Expected behavior
I don't think this should be an offense, but maybe there are reasons that it should be. Figured I should report it regardless.
Actual behavior
Rubocop gives me the following citation:
Test.rb:7:24: W: Shadowing outer local variable - animal.
  puts(animals.find { |animal| animal.length == 5})
                       ^^^^^^

when I ask it to inspect the following code:
# Test.rb

animals = ["dog", "cat", "chicken", "shark"]

if 2 + 2 == 5 # obviously this condition will never be true
  # at this point, animal is still undefined
  animal = "frog"
  puts animal # this line is only here so I don't get cited for useless assignment to animal
else
  # animal is now defined as nil, rather than
  # being undefined or defined as "frog"
  puts(animals.find { |animal| animal.length == 5})
  # Rubocop will cite me here for shadowing, becuase it considers animal to
  # still be defined.
end

Steps to reproduce the problem
To see only the relevant offense, run the line below in a terminal:
rubocop Test.rb --only Lint

RuboCop version
$ rubocop -V
0.48.0 (using Parser <IP_ADDRESS>, running on ruby 2.3.3 x86_64-linux)

Ruby hoists variables such that animal is nil even before entering the if statement.
http://stackoverflow.com/a/12928261/55878

Look at is this way. If you change the name of the block parameter to animal2 for example, you can reference both animal (which is nil and admittedly not much use, but still a defined variable) and animal2 inside the block. But in your example's current state you can only reference the inner animal because it shadows the outer one.


build: make sure nuttx bins generated after both pass1 and pass2 built done

Summary
Create the final NuttX executables after pass1 user libs and pass2 nuttx
libs built finished.
Impact
Testing

The corresponding change needs to be make to tools/Makefile.win too.

LGTM.

The corresponding change needs to be make to tools/Makefile.win too.

It's already there.

tools/Makefile.win also already updated.


get_cells

get cells is currently returning all functions that return a component
how can we check that they are decorated with gf.cell?

gf.kcl.factories should contain all functions decorated with @cell


Some cleanup w/ the move to GitHub

Would this be a good opportunity to convert to markdown? In the past, reStructuredText has been one the most frustrating parts of contributing to onboarding.

I'd be happy with a markdown conversion if someone else automates it.
On Wed, Dec 13, 2017, 7:36 PM Daniel Himmelstein<EMAIL_ADDRESS>wrote:

Would this be a good opportunity to convert to markdown? In the past,
reStructuredText has been one the most frustrating parts of contributing to
onboarding.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/greenelab/onboarding/pull/1#issuecomment-351571108,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAhHs8xe7n3RJO3BBJUV7hnjF4AfFLaEks5tAG2kgaJpZM4RA5V7
.

--
Casey S. Greene, Ph.D.
Assistant Professor
Dept. of Systems Pharmacology and Translational Therapeutics
Perelman School of Medicine
University of Pennsylvania
web: http://www.greenelab.com
phone:<PHONE_NUMBER>

I can run the files through pandoc in a future PR.

Ok - can you pandoc it when you get the chance? I agree that dealing with RST was tough (had the RST docs open yesterday when revising).

Ok - can you pandoc it when you get the chance? I agree that dealing with RST was tough (had the RST docs open yesterday when revising).

Yes. Discussion in #2


Are error comparisons supported?

Does this library currently support comparisons of the error type?
I tried playing around with AllowUnexported and cmpopts.IgnoreUnexported but, can't seem to figure out how to make work.
not-working:
package main

import (
	"errors"

	"github.com/google/go-cmp/cmp"
)

func main() {
	err1 := errors.New("err1")
	err2 := errors.New("err2")
	Match(err1, err2)
}

func Match(expected, result interface{}) {
	test := cmp.Diff(expected, result)
	if test != "" {
		panic(test)
	}
}

Since it may be relevant, here's something I've used when I wanted to compare error (strings):
// equalError reports whether errors a and b are considered equal.
// They're equal if both are nil, or both are not nil and a.Error() == b.Error().
func equalError(a, b error) bool {
	return a == nil && b == nil || a != nil && b != nil && a.Error() == b.Error()
}

Thanks! I'm doing that currently. Just double checking to make sure I wasn't missing something.

Hi @gnahckire, it's intentional that this package does not handle errors by default since there's no sensible policy that works for every use case, and it's the responsibility of the user to define the one that makes sense.
Some simple comparers that you could use:
// equateNilError reports errors to be equal only if both are nil.
equateNilError := cmp.Comparer(func(x, y error) bool {
    return x == nil && y == nil
})

// equateAnyError reports errors to be equal only if both are nil or both are non-nil.
equateAnyError := cmp.Comparer(func(x, y error) bool {
    return (x == nil) == (y == nil)
})

What @shurcool suggests is a possibility, but I want to warn you that the error message provided by most packages is not stable, and something as simple as a grammar change can break your test. If you are simply checking that error messages from your own package match, then this approach is fine.
// equateErrorMessage reports errors to be equal if both are nil or both have the same message.
equateErrorMessage := cmp.Comparer(func(x, y error) bool {
    if x == nil || y == nil {
        return x == nil && y == nil
    }
    return x.Error() == y.Error()
})

Even more advanced, if you know properties about the errors you expect, you can write better equate functions:
// sentinelErrors is a set of common sentinel errors
// (i.e., those that are safe to compare directly using the == operator.
sentinelErrors := map[error]bool{io.EOF: true, io.ErrUnexpectedEOF: true, ...}

equateError := cmp.Comparer(func(x, y error) bool {
    if x == nil || y == nil {
        return x == nil && y == nil
    }

    // Check if x and y are sentinel errors.
    if sentinelErrors[x] || sentinelErrors[y] {
        return x == y
    }

    // Compare error based on their properties.
    switch {
    case os.IsExist(x) || os.IsExist(y):
        return os.IsExist(x) && os.IsExist(y)
    case os.IsPathSeparator(x) || os.IsPathSeparator(y):
        return os.IsPathSeparator(x) && os.IsPathSeparator(y)
    case os.IsPermission(x) || os.IsPermission(y):
        return os.IsPermission(x) && os.IsPermission(y)
    }

    // Default to saying that errors are equal
    return true
})

As you can imagine, the set of possible errors in Go is infinite, so it does not make sense for the cmp package to provide the functionality to handle them. It is the users responsibility to know how to properly compare the errors they are interested in.

Awesome. Thanks!


Use the attribute hasClassScope to select the elements to be displayed

Needed to support Smalltalk

Solved in v0.3.0


Ascendancy fix

Main point of this PR was to fix critical error on Inventory tab due to missing Ascendancy avatars (see first commit), but after noticing tremendous size of main image i just batch compressed all files and saved over 7MB of size which reduced overall size of executable from 9MB to just 2MB.

also removed Talisman league mentions from settings

Very nice, thanks for getting the images together.
I'll merge this as soon as I'm able to do a new release.

I'd recommend squashing the commits which update the images added in this branch (the Ascendancy avatars), so there's only one version in the repo.  Even though git will probably only store the deltas to these files in the .git directory, I'm thinking it'll essentially be like storing both versions of the files, and I wouldn't want to see the repo bloated like that if we could easily avoid it (like it can't be avoided for all the other images modified here, as we'd have to rewrite history).

Well, its pointless, since the size of images with every commit is lower that previous, and only overhead we having here is actually 300 KB (see avatars compression). Main image was introduced way before and we cant rewrite history (bad git practice). On my opinion clear history messages are much more important than 300KB (we are talking before git compression here)

Procurement.1.9.8-RC3.zip
Just in case anyone wants to give it a try


How to skip header on csv files?

Hello,
I can do something like
if $row[16] == "name"
{
return;
}
however was wondering if there was a way to set a skip option in config so it can skip x number of lines
thanks again for an awesome library

For anyone finding this issue through search engines: Since version 1.1.0 (released November 2013) you can skip the first line with $config->setIgnoreHeaderLine(true);


OPA Gatekeeper system chart

Adding basecopy of the OPA Gatekeeper helm v2 chart
Adding builtin ConstraintTemplates
Adding nodeSelector to ensure the workloads never schedule to the Windows node
Add private image registry for airgap case

https://github.com/rancher/rancher/issues/23753
Related PR: https://github.com/rancher/rancher/pull/25578

https://github.com/rancher/rancher/issues/23753

Can we also add in a min Rancher version of 2.4.0-rc1?


Replace EMAIL_FROM with secretary e-mail

Makes sense probably. Alternative would be a no-reply e-mail.

Managed by puppet, so should be an issue there


Clicks fail when put between multiple measure.start and measure.stop

When putting actions under one measure.start/stop block, the actions work fine, but if seperate these actions into two measure.start/stop blocks, the actions in second block fails. From Chrome dev tool can see it's returning error in console.
Script that works:
    await commands.measure.start('Enter new Page');
    await commands.click.byXpath('//h6[contains(text(),"$$$")]');
    await commands.wait.byXpath('//h3[contains(text(),"$$$")]',60000);
    await commands.wait.byPageToComplete();
    //await commands.measure.stop();

    //await commands.measure.start('Get quote');
    await commands.click.byXpath('//button[contains(text(),"$$$")]');
    await commands.wait.byPageToComplete();
    return commands.measure.stop();

Script that fails:
    await commands.measure.start('Enter new Page');
    await commands.click.byXpath('//h6[contains(text(),"$$$")]');
    await commands.wait.byXpath('//h3[contains(text(),"$$$")]',60000);
    await commands.wait.byPageToComplete();
    await commands.measure.stop();

    await commands.measure.start('Get quote');
    // below click will fail
    await commands.click.byXpath('//button[contains(text(),"$$$")]');
    await commands.wait.byPageToComplete();
    return commands.measure.stop();

Error in Chrome console:

Versions(Running on Macbook):
Mac OS: darwin 19.0.0 nodejs:12.18.2 sitespeed.io:13.3.2 browsertime:8.14.0 coach:5.0.1 chrome: 86

Thanks! I tried to create a pull request in Coach but didn't succeed.


[WIP] support gcForest

#331
begin with a gcForest scratch, some details need discussion, suggestion welcome.
the lines with to do means the parts need to be altered.

@wxchan,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@wxchan can u share the example code of using gcForest? tks

@zoey0905 I added an example mnist.py but it works not so well. You can take a look. The dataset can be downloaded from http://www.csie.ntu.edu.tw/~cjlin/libsvmtools/datasets/multiclass.html.

Close for now. See #331 .


Issue with labels in Asciidioc to LaTeX

Hello,
I am running into an issue with asciidoc to convert the following file to LaTeX:
[[table-overview-clocks]]
[cols="9,9"]
|====
|Col A
|Col B

|Foo
|[[ln,link-name]]Bar
|====

<<link-name,the actual link name>>

If you drop the explicit link name in the link and replace it by [[ln]]Bar, the compiler is no longer able to export the data correctly.
Also, the link in <<link-name,the actual link>> is not producing anything useful in the LaTeX file.
Do I miss some point here or is this a bug? How can I help to solve this issue?

Please note that this project is not currently being maintained and is thus looking for a new maintainer.
You are welcome to ask your question in the community chat at https://chat.asciidoctor.org. However, the best outcome you can hope for in this case is a workaround since the code is not likely going to be updated until a new maintainer is found.

@mojavelinux, your comment is only related to the -latex variant or also for the main asciidoctor?

I'm referring to this extension specifically.


Attempting fix for #2565

Contains
Potential fix for #2565
How to test
Look at issue for details on how to reproduce.
This doesn't really work out well, and is probably not a good way to solve this problem. Could use some help on how to fix.
Another thing that can be done is that CROUCHING can be remembered, so that after swimming, the character returns in crouched position (However this will change player's height in water which is undesirable). But this will also solve the second part of #2565

Uh oh, something went wrong with the build. Need to check on that

Can confirm the original bug (at least the crouch-swimming, loading a save seemed OK to me) and that this fixes it, but yeah it doesn't seem like this would be the way to do it :-)
As written this will conflict with #2515 (pinging @coty91!) which overhauls swimming and may be a better architecture to base this fix on top of. It makes me think about movement modes in general and how switching to swimming should probably just reset crouching, yet doing by triggering a button event seems wrong somehow.
Also, interestingly, probably swimming should affect the player's shape anyway - at least for a human a regular swimming stance (while actively moving) is more like a prone stance than standing or crouching, while treading water would be more like standing. Yet a swimming prone would leave the player's eye height the same as treading water, while a landed prone would indeed lower the eye height dramatically. Ahh challenges! :-)
Gets even more fun if you consider other creatures. A crocodile would be prone by nature and that stays unchanged on entering water, whether just floating or actively swimming. And Chessandgo is working on a centaur ...
So maybe crouching and prone should be a subtly different system anyway, yet still reset/change when entering water. Lots of possibilities, but none that are super quick to approach.
In any case I think we need to bake in more flexibility in changing the player's shape during movement, buttons being clicked or not.

@Cervator Yup, this even breaks a test as you can see.
And I can confirm that if you crouch and save and load the save game your standing height turns to half of what it was earlier.
Sending a ButtonEvent was the quickest way around this, since otherwise I would need to @Share LocalPlayerSystem, but is surely not a proper way. It doesn't work smoothly either, I guess there is some lag between the event being sent and height being changed (i.e if you aim for the bottom of the water block in a 1-block deep pool, you might get stuck and would have to jump out). A probably better way would be to change the height before the movement is done, once it is realized that the next step takes you in water.

Let's close this PR for now. I'd get back to this soon :)


Include the "UseCase" Word

It´s a bit confusing when you create an use case because it is not clear if in the name has to include or not the word "UseCase" so it'll be helpful if when you create an use case and the plugin automatically add the word UseCase to it, example if i generate the use case Person, the final result would be a class with name PersonUseCase. This would help us to keep classes naming standard

Fix in release 1.5.2


Focus goes to the Button

I am having a hero card added in web chat control. The focus is automatically set to the first CardAction in button. This is annoying as the user cannot press enter for subsequent chat .
The enter opens skype for business instead.
No auto focus should be set on button in Web Chat Control
heroCard = new HeroCard
{
Title = srchMatUser.DisplayText,
Subtitle = srchMatUser.EntityData.Title + " , " + srchMatUser.EntityData.Department,
Text = srchMatUser.EntityData.MobilePhone,
Images = new List { new CardImage(userImageUrl) },
Buttons =
new List
{
new CardAction(ActionTypes.OpenUrl, "Contact On Skype",
value: "sip:<" + srchMatUser.EntityData.Email + ">")
}
};
message.Attachments.Add(heroCard.ToAttachment());

Hi @punjabimundaaa we have work going on in #630 related to focus. If possible, can you test a local clone of this PR?

@punjabimundaaa can you check if the master is working now?

Closed due to inactivity.


Tweak styling:  use background colors; add line prefixes.

Awesome, looks a ton better already! :D Thank you!


Issue with RTL tables fixed column

Issue with RTL tables fixed column ... scroll and fixed columns not working ... it seems that there is some problem with scrollLeft and negative scrollbar  value...

Hi,
Can you explain this issue more clearly?

Hi,
please check this sample ...

then add   direction to #myDiv and check it again...
#myDiv{   direction:rtl; }
because of negative values ... when the page is Right To Left... nothing happen for columns...


[CNVS Upgrade] Fix inverse classes

This PR removes some inverse classes that shouldn't be present

Conflicts!

@mlunoe Conflicts fixed!

ping @rcorral @MatApple @wavesoft this is one of the worst ones, this should definitely be merged before Thursday's demo

Thanks @jfurrow for the clean up :)


[RFC] Convert lwAFTR over to loops-and-queues style

Instead of processing each packet one by one, this branch converts the lwAFTR to process packets in batches, moving each batch from queue to queue as the internal structure of the lwAFTR determines.
The idea here is that we should write code that plays to the strengths of LuaJIT, and LuaJIT's strength is tight and ideally branchless loops. In this style, we move packets from queue to queue.  Because the packets are alive in only a straight-line code path, LuaJIT's allocation sinking optimization does very well for this style of code.
Unfortunately this patch is currently a regression, relative to the upstream lwAFTR branch, so this is a work-in-progress patch set.
On lwaftr branch:
[wingo@snabb1:~/snabb/src]$ make && for i in `seq 4`; do sudo ./snabb lwaftr bench -D 5  --name test --cpu 2 program/lwaftr/tests/data/icmp_on_fail.conf program/lwaftr/tests/benchdata/ipv4-0550.pcap program/lwaftr/tests/benchdata/ipv6-0550.pcap; done
make: 'snabb' is up to date.
program/lwaftr/tests/data/icmp_on_fail.conf: loading compiled configuration from program/lwaftr/tests/data/icmp_on_fail.o
program/lwaftr/tests/data/icmp_on_fail.conf: compiled configuration is up to date.
Bound main process to NUMA node: 	0
Bound data plane to CPU:	2
Time (s),Decap. MPPS,Decap. Gbps,Encap. MPPS,Encap. Gbps
0.995835,2.468482,10.071408,2.468482,11.651236
1.995899,2.488640,10.153652,2.488640,11.746382
2.995941,2.488694,10.153871,2.488694,11.746635
3.995876,2.488963,10.154971,2.488963,11.747908
program/lwaftr/tests/data/icmp_on_fail.conf: loading compiled configuration from program/lwaftr/tests/data/icmp_on_fail.o
program/lwaftr/tests/data/icmp_on_fail.conf: compiled configuration is up to date.
Bound main process to NUMA node: 	0
Bound data plane to CPU:	2
Time (s),Decap. MPPS,Decap. Gbps,Encap. MPPS,Encap. Gbps
0.996330,3.071272,12.530791,3.071272,14.496405
1.996277,3.039761,12.402225,3.039761,14.347672
2.996215,3.029587,12.360714,3.029587,14.299650
3.996319,3.029085,12.358665,3.029085,14.297279
program/lwaftr/tests/data/icmp_on_fail.conf: loading compiled configuration from program/lwaftr/tests/data/icmp_on_fail.o
program/lwaftr/tests/data/icmp_on_fail.conf: compiled configuration is up to date.
Bound main process to NUMA node: 	0
Bound data plane to CPU:	2
Time (s),Decap. MPPS,Decap. Gbps,Encap. MPPS,Encap. Gbps
0.996449,3.040196,12.403999,3.040196,14.349724
1.996451,2.998793,12.235077,2.998793,14.154304
2.996535,2.998548,12.234076,2.998548,14.153147
3.996536,2.988596,12.193473,2.988596,14.106175
program/lwaftr/tests/data/icmp_on_fail.conf: loading compiled configuration from program/lwaftr/tests/data/icmp_on_fail.o
program/lwaftr/tests/data/icmp_on_fail.conf: compiled configuration is up to date.
Bound main process to NUMA node: 	0
Bound data plane to CPU:	2
Time (s),Decap. MPPS,Decap. Gbps,Encap. MPPS,Encap. Gbps
0.996521,3.019505,12.319579,3.019505,14.252062
1.996427,2.978679,12.153011,2.978679,14.059366
2.996479,2.978245,12.151239,2.978245,14.057315
3.996450,2.978487,12.152226,2.978487,14.058458

Then with this branch:
[wingo@snabb1:~/snabb/src]$ git checkout tight-loops-and-queues && make >/dev/null
[wingo@snabb1:~/snabb/src]$ make && for i in `seq 4`; do sudo ./snabb lwaftr bench -D 5  --name test --cpu 2 program/lwaftr/tests/data/icmp_on_fail.conf program/lwaftr/tests/benchdata/ipv4-0550.pcap program/lwaftr/tests/benchdata/ipv6-0550.pcap; done
make: 'snabb' is up to date.
program/lwaftr/tests/data/icmp_on_fail.conf: loading compiled configuration from program/lwaftr/tests/data/icmp_on_fail.o
program/lwaftr/tests/data/icmp_on_fail.conf: compiled configuration is up to date.
Bound main process to NUMA node: 	0
Bound data plane to CPU:	2
Time (s),Decap. MPPS,Decap. Gbps,Encap. MPPS,Encap. Gbps
0.996315,2.569669,10.484248,2.569669,12.128836
1.996273,2.590910,10.570912,2.590910,12.229095
2.996242,2.550079,10.404322,2.550079,12.036372
3.996266,2.580537,10.528592,2.580537,12.180136
program/lwaftr/tests/data/icmp_on_fail.conf: loading compiled configuration from program/lwaftr/tests/data/icmp_on_fail.o
program/lwaftr/tests/data/icmp_on_fail.conf: compiled configuration is up to date.
Bound main process to NUMA node: 	0
Bound data plane to CPU:	2
Time (s),Decap. MPPS,Decap. Gbps,Encap. MPPS,Encap. Gbps
0.996862,2.598954,10.603734,2.598954,12.267065
1.996779,2.621619,10.696205,2.631820,12.422189
2.996824,2.580484,10.528376,2.570285,12.131745
program/lwaftr/tests/data/icmp_on_fail.conf: loading compiled configuration from program/lwaftr/tests/data/icmp_on_fail.o
program/lwaftr/tests/data/icmp_on_fail.conf: compiled configuration is up to date.
Bound main process to NUMA node: 	0
Bound data plane to CPU:	2
Time (s),Decap. MPPS,Decap. Gbps,Encap. MPPS,Encap. Gbps
0.996817,2.588839,10.562465,2.588839,12.219322
1.996769,2.621527,10.695832,2.621527,12.373609
2.996802,2.570316,10.486888,2.570316,12.131890
3.996713,2.611432,10.654645,2.611432,12.325961
program/lwaftr/tests/data/icmp_on_fail.conf: loading compiled configuration from program/lwaftr/tests/data/icmp_on_fail.o
program/lwaftr/tests/data/icmp_on_fail.conf: compiled configuration is up to date.
Bound main process to NUMA node: 	0
Bound data plane to CPU:	2
Time (s),Decap. MPPS,Decap. Gbps,Encap. MPPS,Encap. Gbps
0.996591,2.538254,10.356076,2.538254,11.980559
1.996658,2.539628,10.361683,2.539628,11.987045
2.996629,2.539874,10.362687,2.539874,11.988207
3.996655,2.539734,10.362114,2.539734,11.987544

So, it seems like a losing strategy.  I don't know why :/
This branch in its beginning also includes the hacks proposed for #922.  Unfortunately, separate apps were also a slowdown!  I had to integrate this functionality inside the lwAFTR to regain perf and I still think I still lost a little bit (though not as much as shown above).  A bit of a disappointment :/

Perf on upstream lwaftr for small packets and setting engine.pull_npackets to 128 so that we evenly fill the lookup streamer:
[wingo@snabb1:~/snabb/src]$ make && sudo BURST_SIZE=128 ./snabb lwaftr bench -D 15  --name test --cpu 2 program/lwaftr/tests/data/icmp_on_fail.conf program/lwaftr/tests/benchdata/ipv{4,6}-0094.pcap
LUA       obj/program/lwaftr/setup_lua.o
GEN       obj/version.lua.gen
LUA       obj/version.lua
LINK      snabb
BINARY    3.5M snabb
pull_npackets	128
program/lwaftr/tests/data/icmp_on_fail.conf: loading compiled configuration from program/lwaftr/tests/data/icmp_on_fail.o
program/lwaftr/tests/data/icmp_on_fail.conf: compiled configuration is up to date.
Bound main process to NUMA node: 	0
pull_npackets	128
Bound data plane to CPU:	2
Time (s),Decap. MPPS,Decap. Gbps,Encap. MPPS,Encap.   My interpretation is that most time is spent in loop bodies and only these lines indicate time spent in loop headers:
  1% TRACE  53               ->loop
  1% TRACE  37               ->loop

So if a benefit of this branch would be making LuaJIT spend more time in loop bodies and not headers, I think we're just not going to get that -- already very little time is spent outside loop bodies.  This traceprof run from this branch seems to bear that out:
[wingo@snabb1:~/snabb/src]$ make && sudo BURST_SIZE=128 ./snabb lwaftr bench -D 15  --name test --cpu 2 program/lwaftr/tests/data/icmp_on_fail.conf program/lwaftr/tests/benchdata/ipv{4,6}-0094.pcap
LUA       obj/apps/lwaftr/lwaftr_lua.o
GEN       obj/version.lua.gen
LUA       obj/version.lua
LINK      snabb
BINARY    3.5M snabb
pull_npackets	128
program/lwaftr/tests/data/icmp_on_fail.conf: loading compiled configuration from program/lwaftr/tests/data/icmp_on_fail.o
program/lwaftr/tests/data/icmp_on_fail.conf: compiled configuration is up to date.
Bound main process to NUMA node: 	0
pull_npackets	128
Bound data plane to CPU:	2
Time (s),Decap. MPPS,Decap. Gbps,Encap. MPPS,Encap.   Could it be that having smaller loops means more loop overhead, and actually we should be looking to increase loop size?  You wouldn't think that... but maybe???

Or, is this test workload pathologically simple?  It's all traffic that doesn't need hairpinning, doesn't include ICMP traffic, and is always matching the binding table.

Or, is this test workload pathologically simple?

Could be that the many smaller loops provide more robustness. If the workload were more complex then you may be exercising more side traces, due to more guards failing and exits being taken, and this may be more disruptive if you have one big loop (one failing guard is a spanner in the works for all processing of that packet.)

Yeah there is just no way this is going to work.  Testing against traffic on our lab machines instead of with the synthetic tester, the current code can do 2.49+2.49 MPPS FDX on a core (two-interface configuration) without dropping on small packets, whereas this branch can only do 2.235+2.235.

Some comments from someone who has been down that route: I believe the general hypothesis to hold true, but  testing this led me to believe me there is a fixpoint where the cost of one more loop outweighs the cost of a branch. More interestingly, I recently removed all the “synthetic” loops and instead split up the app in question into smaller apps. The result was basically the same thing, except “many small loops” arose naturally from the app hierarchy instead of being coded manually case-by-case. (Performance was unchanged.)
So my take would be to avoid things like this branch’s “ilink” and think about larger app networks with smaller apps instead. I suppose Snabb aligns itself naturally with the JIT, but it took me a while to fully embrace the app as a unit of code sized between module and function.

I think of it a bit as an SPMD architecture. Maybe Snabb is a bit like Volta in this regard, only instead of execution masks we have links.

Closing as I have no plan to work on this.


Can you please add a tag to the latest version?

To refer to you project from other projects I would like to have a tag (specified version) of the project.
Can you please attach a tag e.g. v0.1 to 7b30d4a3b37db7eb06cbbfe3c37dd8da58ead050?
Thanks

https://github.com/mintoko/fidl2adoc/releases/tag/v0.1

Thanks.


Testing for likes and comments counters

Features for counters testing

[ ] Added comments counter function for mock testing
[ ] Added likes counter function for mock testing
[ ] Added babel transpiler to import the code into different files
[ ] Modified readme file according to project requirements

Hello @Nasirkhan294 , great work on implementing the tests for the likes and comments. Your project is ready, you can merge it now.


Link to get latest version for installation breaks frequently

As per the documentation, to install delta kusto via command line in Azure DevOps, the url given to get the latest release version is:
$clientVersion=(Invoke-WebRequest https://delta-kusto.azurefd.net/clientVersion/unique?fromClientVersion=0 -UseBasicParsing).Content
However this relies on an azurefrontdoor url which breaks many times causing failures in devops builds.
A more robust way to get the latest version by using github apis is:
$clientVersion=((Invoke-RestMethod -Uri "https://api.github.com/repos/microsoft/delta-kusto/releases/latest").html_url.html_url) | Select-String -Pattern "https://github.com/microsoft/delta-kusto/releases/tag/(.*)").matches.groups[1].value.matches.groups[1].value)

That type of break shouldn't happen anymore.  We moved it to a "better" subscription.
The get-latest of GitHub should also work.  It is limited to only "the latest version" whereas the custom API support latest version, latest minor, latest major.  But if you want the latest-latest, we always publish the latest as "latest" (it's an option on GitHub).


fixes #16 Remove stale msg.Expired

Codecov Report

Merging #31 into master will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
+ Coverage   55.93%   56.02%   +0.09%
==========================================
  Files          43       43
  Lines        5762     5747      -15
==========================================
- Hits         3223     3220       -3
+ Misses       2226     2217       -9
+ Partials      313      310       -3

Impacted Files
Coverage Δ

message.go
80.55% <ø> (+4.36%)
:arrow_up:

transport/ws/ws.go
77.51% <ø> (+0.92%)
:arrow_up:

transport/conn.go
64.86% <ø> (+2.52%)
:arrow_up:

transport/inproc/inproc.go
89.37% <ø> (+1.64%)
:arrow_up:

protocol/xreq/xreq.go
40.78% <0%> (-0.88%)
:arrow_down:

protocol/surveyor/surveyor.go
56.06% <0%> (+0.75%)
: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 1dd7b45...0e219a3. Read the comment docs.


Convert drivers/console/uart_mux.c to new kwork API

Convert drivers/console/uart_mux.c to use new kwork API.
See the following issue for details:
#33104

Hmm, I can take this as it is tied to gsm_mux that I already converted in the same directory.

Hmm, I can take this as it is tied to gsm_mux that I already converted in the same directory.

thanks.


Group property support

license - not the only required property, i need to use group property.
and i need to reload chat if this property changed, but i'm not sure it's posible, maybe You know how to make that?

Hi there!
What exact use case would you like to cover with the group property? For a SPA application our url rules are pretty much always the better choice.

Version 0.0.4 provides optional parameter group, but like @Andarist said

For a SPA application our url rules are pretty much always the better choice.


STLinkV2 USB protocol, stm8flasher use

[X] Programmer/board type: Stlink/v2
[X] Operating system: e.g Linux

Please could you point me in direction of RAW USB STLINKV2 command/data frame.
Tool stm8flash:  https://github.com/vdudouyt/stm8flash
stlinkv2.c: https://github.com/vdudouyt/stm8flash/blob/master/stlinkv2.c
Currently supports only 16bit address and I would like/need to extend it to 24bits, for flash sizes over 32kB
Thank you!
Kind regards

Sorry no support for other tools and repositories, texane/stlink is only dedicated to stm32 controllers. Please submit your question to the stm8flash project.

Thank you on fast respons.
Any suggestions how to sniff this STLINKV2 for SWIM packets as was done for SWD?

Either sniff from a windows virtual machine with VMware (http://esec-lab.sogeti.com/posts/2011/04/06/sniffing-usb-traffic-with-vmware.html) or run windows in a virtualbox virtualmachine on linux and sniff the usb traffic with wireshark.

Thank you,
sniffed USB packets while writing 1byte on 0xFFFF/0x10000 address https://github.com/vdudouyt/stm8flash/issues/36
Now need to make sense of it (first by reading existing code)

Thank you for suggestion. Solved!


xamlstyler 3.2312.0

Summary of the Pull Request
Update xamlstyler to 3.2312.0
References and Relevant Issues
https://github.com/Xavalon/XamlStyler/releases/tag/Release-3.2312.0
changes
This release adds support for Rider 2023.3 (https://github.com/Xavalon/XamlStyler/issues/451).
Validation Steps Performed

We don't use Rider since this project isn't written in C#, so this update is not helpful.

We can skip this version unless there are vulnerabilities in the one we have.
Thanks for working on this @Jvr2022!


[EngSys] upgrade dev dependency dotenv to version ^16.0.0

Most of packages have been updated as well as codegen.  This RP upgrades left-over packages.
Resolves #19217

API change check
API changes are not detected in this pull request.

Build failure is due to api.md change, possibly because of api-extractor upgrades, will fix in another PR

/check-enforcer override


How to disconnect a session ?

Hi,
How can i disconnect a session from a noVNC client ?
I noticed when i'm not disconnecting and move to other page in my application and then trying to reconnect again its getting stuck cause the session is still there.

Thanks for raising this issue @dorroddorrod !
I've recently added v0.2.9
You can use a ref (such as the following) to access the connect() and disconnect() methods.
https://github.com/roerohan/react-vnc/blob/main/src/App.tsx#L8
Here's a sample of how to use the ref.
https://github.com/roerohan/react-vnc/blob/main/src/App.tsx#L40-L53

In #28 , I've also added support for the following methods from RFB
sendCredentials,
sendKey,
sendCtrlAltDel,
focus,
blur,
machineShutDown,
machineReboot,
machineReset,
clipboardPaste


MRTK v2 Bounding Box handles not displayed correctly with scaled objects

Describe the bug
Bounding Box handles are not displayed at the right position when the object it is applied to has a different scale for at least one axis.

Watch the video here: link
Note: in that video, I illustrate the normal behavior with an even scaling and then I change the scale in the properties so you can see both behaviors, including the wrong one.
To reproduce
Steps to reproduce the behavior:

In an empty project: Import MRTK
Add any 3D object
Add a Bounding Box to the object following docs
Change the scale value in the Transform for the 3D object for ONE axis only (for instance, x)
Run the project and move on all axis

You can find an example with that issue here: https://github.com/meulta/MRTK_BoundingBox
(just change the scale of the mountain).
Expected behavior
Uneven scaling of the object should not impact the Bounding Box handle and they should display properly
Screenshots

Your Setup (please complete the following information)

Unity Version 2018.3.12f1
MRTK Version v2.0 RC1 refresh

Target Platform (please complete the following information)

HoloLens 2 (but the issue happens in the player in Unity also)

Editing the object's transform in-editor at runtime while the bounding box is active isn't supported. (Based on your video and scene setup this appears to be what you're doing.) Once the bounding box is active it's the bounding box's job to perform transformations.
We'll make this clearer in the documentation.
Thanks for the detailed bug write-up and example scene.

Understand that, but the issue is also happening if you change scale BEFORE running in the editor. The video was just to explain the issue. This is still more than a documentation issue in my opinion.

Hello, this is still a bug, can you please keep it opened until it is fixed?

Apologies! Finger slipped on mobile while commenting. We'll take another look and try to repro.

I can repro that:

Any initial scale applied to rigRoot's parent, that is the GameObject with the BoundingBox.cs script attached to it, affects the bounding box and produces artifacts as shown in @meulta's screenshot and above. In the example above the Y scale is > 1.
I am not sure this has any influence, but my hierarchy is as follow:

Target object

Manipulator object with non-unit scale, components BoundingBox.cs and BoxCollider, and bounding box overrides Target=parent and Collider=self

This is a common pattern I found very useful to be able to have both the manipulator bounding box and other children (generally a 3D model) all parented together as siblings, and be able to transform the root Target to move them together. I do not want to put the bounding box on the root because I need to deactivate it sometimes, without deactivating the rest of the hierarchy (3D model).

Fixed by #4592


torch._dynamo.variables.higher_order_ops error

when I use flash-attention I get this error. I have no clue what it is. Any guidance at all will be helpful.
[rank2]:[2023-10-09 15:38:00,808] [4/0] torch._dynamo.variables.higher_order_ops: [WARNING] speculate_subgraph: while introspecting the user-defined autograd.Function, we were unable to trace function trampoline_autograd_fwd into a single graph. This means that Dynamo was unable to prove safety for this API and will fall back to eager-mode PyTorch, which could lead to a slowdown.
[rank2]:[2023-10-09 15:38:00,809] [4/0] torch._dynamo.variables.higher_order_ops: [ERROR] call_method UserDefinedObjectVariable(fwd) call [TensorVariable(), TensorVariable(), TensorVariable(), ConstantVariable(NoneType), ConstantVariable(float), ConstantVariable(float), ConstantVariable(bool), ConstantVariable(bool), ConstantVariable(NoneType)] {}
P.S. when I use torch.nn.functional.scaled_dot_product_attention (which my understand is, that it is an older "version" of flash"), I dont get the error.

After a lot of trial and error I found out that this is a conflict with torch.compile(). If anyone has an idea of what may be the problem, I would appreciate it very much.

It looks like an issue that is discussed by the PyTorch team - https://github.com/pytorch/pytorch/issues/93468#issuecomment-1748774273


[WIP] build OpenAPI bundle via GitHub Actions

@jerstlouis @ghobona not sure how we can enable GitHub Actions here?

Nevermind, this will be triggered on any YAML changes.


OCMVerify() on Partial Mocks does not work with selectors implemented by NSObject/NSProxy

	UIView *view = [[UIView alloc] initWithFrame:CGRectZero];
	id mock = [OCMockObject partialMockForObject:view];
	[realObject accessibilityLabel];
	OCMVerify([mock accessibilityLabel]);

reports Method accessibilityLabel was not invoked; but was expected at least once.
This is a very confusing error because it's obviously being called. The problem is that OCMPartialMock does not set itself up to handle any methods implemented by NSObject or NSProxy. All of the accessibility methods are informal protocols (i.e. categories) on NSObject. This can be done by using OCMExpect([mock accessibilityLabel]) and OCMVerifyAll(mock).

Thanks for the merge.


Create /home/zuul directory in job workspace

In EDPM content provider job, the content provider has zuul user and edpm job has zuul-worker user. When content provider returns meta operator git_src data to the EDPM job. It contains /home/zuul user in the path. It breaks the openstack services deployment by saying
/home/zuul/src/github.com/openstack-k8s-operators/openstack-operator
directory not found on EDPM node.

We tried to set '~' for pointing to user's home[1] directory but ansible still resolves to /home/zuul user.
In order to fix that, we are adding a pre-playbook to create a /home/zuul directory and create a symlink to /home/zuul-worker. It will fix the above issue.
[1]. https://github.com/openstack-k8s-operators/dataplane-operator/pull/206
Signed-off-by: Chandan Kumar<EMAIL_ADDRESS>This PR has:

[ ] Appropriate testing (molecule, python unit tests)
[ ] Appropriate documentation (README in the role, main README is up-to-date)

/approve

/lgtm


[DIG-118] [Feature]: Subscribe to Blogs

Is your feature request related to a problem? Please describe.
New Signups automatically get signup for blogs.
Describe the solution you'd like
On the Signup Page, we can add a checkbox (which is ticked by default). This automatically adds to the user to our hashnode blog newsletter.
Alternatives
No response
Additional Context
Explore hashnode yourself - here
Are you interested in contributing?

[ ] I'd like to work on this issue

Code of Conduct

[X] I have used the search function to check if an issue already exists
[X] I agree to follow this project's Code of Conduct

From SyncLinear.com | DIG-118

@pranshugupta54 I would like to work on this issue

i would like to work on this issue as a beginner.

Hey @jaymehta002 @AVtheking, did you setup the repo locally?

Did you explore hashnode?
Join us on Discord for any help.

Yes

@AVtheking, assigned.

@AVtheking any updates?

can you share link to your newsletter blog?

@jaymehta002,

https://blogs.digitomize.com/newsletter
We want them to automatically subscribe to our blogs on creating account (We'll have a checked check-box on signup for newsletter signup - they can un-check)

i need to know that https://blogs.digitomize.com/ this is frontend url or backend url?

I am creating a pr,
check probably it works in live version


EKS Access after deploy cluster

Description
I can't EKS cluster after the last version of the module. In the AWS console appear this warning:
"Your current user or role does not have access to Kubernetes objects on this EKS cluster
This may be due to the current user or role not having Kubernetes RBAC permissions to describe cluster resources or not having an entry in the cluster’s auth config map."
Do I need to modify anything in my code?

Check the example in the README. You need to configure aws-auth
  # aws-auth configmap
  manage_aws_auth_configmap = true

  aws_auth_roles = [
    {
      rolearn  = "arn:aws:iam::66666666666:role/role1"
      username = "role1"
      groups   = ["system:masters"]
    },
  ]

  aws_auth_users = [
    {
      userarn  = "arn:aws:iam::66666666666:user/user1"
      username = "user1"
      groups   = ["system:masters"]
    },
    {
      userarn  = "arn:aws:iam::66666666666:user/user2"
      username = "user2"
      groups   = ["system:masters"]
    },
  ]

what @tutunak is correct - the identity that created the cluster is automatically added to the cluster at time of creation as a systems:master role in the cluster (you won't see this in the aws-auth configmap). If this is not the same identity you log into the console with, you will see the error described. Simply add your role/user to the configmap under the appropriate group(s).