feat: provide kind of Set and Map data types in utils package

Hey, as discussed #317.
I've started to work on a Set implementation. I took the liberty to add more methods to the proxySet object, notably map and filter given how useful they are when working with React. Let me know what you think about it. I've added them as well for Map (not committed yet)
The current Set implementation is not a 1 to 1 mapping. Below are the missing methods. TBH, I don't know if it's worth having them given that I've added map and filter, I don't recall using them often. This is pretty subjective 😆

Set.prototype[@@iterator] (iterator symbol)
Set.prototype.values
Set.prototype.entries

The implementation of Map will follow later.

Thanks for working on this!
valtio strongly prefers providing "normal" JS API.
So, it's strongly suggested to implement all methods and no extra methods.
If it's technically impossible, let me know. (I think we'd throw in such cases.)
And, I'd personally use iterators, quite often. 😁

Thanks for the feedback. I'll stick to the "normal" API then 👌

So, I've managed to be really close to the native Set implementation. I've implemented a test case to ensure that the output of the proxySet always matches with a native Set.
I had to do one modification in the core.

sorry, I changed my mind during the review. hope it's readable. I will think about the vanilla part.

No problem 😄thanks for the detailed review

I will work on the core (vanilla.ts) in a separate PR. So, please wait for it for the integration part.
Meanwhile, I hope you can add more tests, especially for Set items being objects (arrays) and Sets.

No problem. I'll work on adding more tests.

Well, if you ask me, I'd suggest to emulate native Set behavior.
Basically, Set is an iterable, which can't be stringified.
You want to convert an iterable to an array before stringify.
> JSON.stringify(Array.from(new Set([{ id: 1, val: 'hello' }, { id: 2, val: 'world' }])))
'[{"id":1,"val":"hello"},{"id":2,"val":"world"}]'

Now that you mention it. It's wiser, so the end-user knows what to expect. Also, this simplifies the implementation, toJSON will only need to return {}

Don't you even need to implement toJSON?

Don't you even need to implement toJSON?

Indeed it's not needed if data and size are not enumerable
 Object.defineProperties(set, {
    data: {
      writable: true,
      enumerable: false,
    },
    size: {
      enumerable: false,
    },
  })

So I've been playing with #321.
Inside proxySet function, I override canProxy so it always return true.
  const target = proxy(set)
  const handler = unstable_getHandler(target)
  // for testing purpose
  handler.canProxy = () => true

  return target as Set<T>

When I use the proxySet directly as below it works
const state = proxySet([1,2,3])

const App = () => {
  const snapshot = useSnapshot(state);

  return (
      <>
        {/* size updated as expected */}

size: {snapshot.size}

        <button onClick={() => state.add('aValue')}>button</button>
      </>
    )
}

If I use the proxySet inside another "state" this won't because the canProxy only applies to his target.
const state = proxy({
  set: proxySet([1,2,3])
  otherValue: "foo"
})

const App = () => {
  const snapshot = useSnapshot(state);

  return (
      <>
        {/* size not updated*/}

size: {snapshot.set.size}

        <button onClick={() => state.set.add('aValue')}>button</button>
      </>
    )
}

One workaround would be to use useSnapshot on the set itself
const snapshot = useSnapshot(state.set)

I'll keep playing with the API maybe I'm missing something

I don't think you misunderstand. I think I overlooked something...

I have the feeling that the !(x as any)[Symbol.iterator]) is a bit too restrictive as it will prevent objects that implement it to be proxied?

Yeah, now it's technically configurable, I can think about changing the default.

What I overlooked was that, if we already have a proxy, we should skip canProxy. Added a new commit in #321.

Just tried with the new commit this seems to solve the issue! Thanks. I'll focus on the testing part now.

Is it correct we don't even need to use unstable_getHandler in this PR?

Is it correct we don't even need to use unstable_getHandler in this PR?

Indeed. Which is even better 👌


Update UsernamePasswordConfiguration docstrings

Description
Updating the reference docs for UsernamePasswordConfiguration class (azure.ai.ml.entities.UsernamePasswordConfiguration class | Microsoft Learn) to indicate that the username and password should be URL encoded
Refer the following update
Create connections to external data sources (preview) - Azure Machine Learning | Microsoft Learn
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

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


'pastMinKey' can have side effects

~/nimdbx/nimdbx/Cursor.nim(106, 6) Error: 'pastMinKey' can have side effects
> ~/nimdbx/nimdbx/Cursor.nim(106, 43) Hint: 'pastMinKey' calls `.sideEffect` 'compareKey'
>> ~/nimdbx/nimdbx/Cursor.nim(92, 6) Hint: 'compareKey' called by 'pastMinKey'

Indeed.

Hi,
new compiler versions are more strict with func functions.
This is probably the reason why this project doesn't compile and emits the error above. You may just change:
func -> proc for the two funcs: pastMinKey and pastMaxKey
I also added a pull request that fixes the issue.

Fixed as part of #21.
(I’d prefer to leave these as func and fix the bug by making compareKey and mdbx_compare_key (sp?) funcs too, but it’s not a big deal.)


Use golang.org/x/crypto/cryptobyte to unpack DNS records

This substantially simplifies the machinery and state tracking around DNS record unpacking. Gone are the off int arguments! Gone are the manual slice length checks!
Care has been taken to maintain existing behaviour, including quirks, with the only notable intentional change being to UnpackDomainName where we now strictly enforce that pointers point backwards into the message.

holy moly this cleans stuff up.

You know I thought it would, hence the PR, but even I was surprised just how much it improves things.

Would be good to run the coredns testsuite with this version.

Yep that's actually my hold up at the moment. I've been debugging and working through a few scattered CoreDNS test failures.

regarding this errors: I think the est. pattern in this lib is: "bad.. ", with maybe slightly more context than just 'bad SVCB'

Yeah I've gone back and forth on the errors and ended up with something quite different. There may be compatibility concerns, but ultimately I think we need to be far more semantic with our unpacking errors. I've gone and distinguished between failures to unpack the message framing and failures to unpack the record data specifically. Once we're in a length-delimited section, it doesn't make sense to use the same error anymore as we're not dealing with potential truncation but rather just corrupt marshalling.

I've left quite a few TODOs and comments where I've discovered incorrect or problematic behaviour. I've successfully run the CoreDNS test suite against this change now as well. Figuring out why it was failing was quite difficult and led down the RFC rabbit hole and exposed some bugs in this library. I'll aim to address all the TODOs in future PRs.

think this could go in and the fix some fallout and the dyn. update stuff?
(pointer to self should still be checked I guess, but can also be done in followup)


Fix relocation for type

In "hello world" while running CI for Ubuntu in https://github.com/dotnet/corert/pull/425
@jkotas PTAL.

Thanks for fixing it!

Ported fix to https://github.com/dotnet/corert/pull/425 and it looks good for Linux/Windows and OSX CPP as JIT is not there yet!


your badges have no meaningful URLs!

the licence should lead to the licence file
the pypi should lead us to the pypi page

Fixed in https://github.com/leoleonsio/cjdb/commit/975d2ef482fa7668f578d5ee063921fa8d674a9a


W2 d2 updates

W2D2 Linear Systems Production Edits for Consistency:
-Separated out plotting functions, put helper functions right above where they’re used
-Renamed coding/discussion/demos with new order
-Uncommented coding exercises

Forgot to clear all output!


Android engine should be built with frame pointers in profile mode

Running the engine with the Dart VM flag --profile-vm, which enables collecting profile samples in native code, leads to crashes.
@jason-simmons @johnmccutchan

This could be an optional GN argument:
"--profile=vm"

@jason-simmons, @rmacnak-google, is this still an issue?

Closing this per @a-siva


[docs] integration test example

It will really help to add more detail on the integration test procedure at
https://github.com/awslabs/solution-spark-on-aws/blob/main/solutions/swb-reference/README.md#integration-tests
the instructions on editing the .yaml file is not clear. Where do I find the value for each fields?
A screenshot of an example will really help

Hi @tariqha-dev thank you for bringing this to our attention,
We are working on adding more information to the integration test documentation, We'll keep this ticket updated with any news related to the documentation change.
Thanks
Fernando

Hello @tariqha-dev, we have an example file (commented) added to the repository as a reference.
A quick note about the config variables is as follows:
apiBaseUrl: This is the apiUrlOutput that becomes available after running the cdk-deploy command. Value found on console running the deploy command.
Resource creation and extracting values for the following can be found here
envTypeId: The ID assigned to the SC environment type you've imported for the test. Value found
envTypeConfigId: The ID assigned to the SC environment type config linked to the envTypeId above
projectId: The ID of the project linked to the hosting account via index
envType: Name given to the envType (default is sagemakerNotebook)
alreadyTerminateEnvId: ID of a terminated environment
Details of the Cognito User Pool created in your main account by SWB:
awsRegion, userPoolId, clientId (app client ID)
rootUsername: email ID to use for login
rootPasswordParamStorePath: path of the SSM param where password for this user is stored

Hi @tariqha-dev we also have updated the instructions for integration testing here.
I hope it helps
-Fernando


RNAseq new app - tracks don't render

If I select the gtf tracks for viewing in the genome browser, they don't come up.
If I select the bam tracks, they show an error.

In the console:
Failed to load resource: the server responded with a status of 404 (Not Found)
2core.js.uncompressed.js:113064 Uncaught TypeError: Cannot read property 'match' of undefined
    at Object.resolveUrl (core.js.uncompressed.js:113064)
    at Object.resolveUrl (core.js.uncompressed.js:116826)
    at Object.constructor (/js/3.5.9/JBrowse/Store/SeqFeature/GFF3.js:36)
    at new <anonymous> (dojo.js.uncompressed.js:4549)
    at Object.eval (core.js.uncompressed.js:116563)
    at dojo.js.uncompressed.js:3288
    at eval (core.js.uncompressed.js:116610)
    at ea (dojo.js.uncompressed.js:1134)
    at dojo.js.uncompressed.js:829
    at fa (dojo.js.uncompressed.js:1277)
3/data.bam.bai Failed to load resource: the server responded with a status of 404 (Not Found)
2core.js.uncompressed.js:113064 Uncaught TypeError: Cannot read property 'match' of undefined
    at Object.resolveUrl (core.js.uncompressed.js:113064)
    at Object.resolveUrl (core.js.uncompressed.js:116826)
    at Object.constructor (/js/3.5.9/JBrowse/Store/SeqFeature/GFF3.js:36)
    at new <anonymous> (dojo.js.uncompressed.js:4549)
    at Object.eval (core.js.uncompressed.js:116563)
    at dojo.js.uncompressed.js:3288
    at eval (core.js.uncompressed.js:116610)
    at ea (dojo.js.uncompressed.js:1134)
    at dojo.js.uncompressed.js:829
    at fa (dojo.js.uncompressed.js:1277)

The not founds showed /data.bam.bai as the path.

Fixed in https://github.com/PATRIC3/p3_web/pull/741


Add GitHub Actions workflow for handling stale issues and pull requests

Fixes # (provide issue number if applicable)
Feature Summary and Justification
This feature provides ...
References

This is currently for testing and will not action on issues/PR unless manually triggered and dry-mode is explicitly set to false.


This app is not available for any of your devices

Any idea why this app is saying so?
I got a Samsung S10e running android 12

Thank you for raising this issue. I'm not sure why that would be the case, Droid Synth should be available to devices running Android 12. The app is not available on Google Play in all countries/regions, so that may be the reason for you seeing that error message.

Hi yes, that would be it.
Yes I am aware of F-Droid, another open src app that I maintain is published there. It would be good to have this app published there too but I don't have time to do that at the moment.
There's no reason for it to be not available in Israel so I've now added it now. Not sure how long it will take for that change to show up on Play, but maybe check in a few hours or a day and it should be available.

Hey, downloads now.
Though I don't see it showing up as a midi-in input on android. Which is what I was looking for.

@guysoft glad you can see it now.
In regards to midi in, it uses the older style "custom usb" midi apis so you need to plugin your midi controller into your phone before starting droid synth or if you have droid synth running when you plug it in, go to another screen (eg droid synths settings) then back to the main droid synth screen and that should then trigger the OS modal dialog prompting you to give permission for droid synth to access the midi controller.

Ah I see @guysoft . Yes sorry about that, Droid Synth doesn't make use of Androids MIDI API's that would allow that. I have an issue already for working on that #3 but most likely I'll do that work as part of "Droid Synth 2" which I'm going to build as a cross platform app using Flutter.
In the meantime as you are doing Android dev yourself, you may want to have a go at using via JNI the C library I made that extracts the fm synth functionality out of this app, which is what I'm using as the basis of the new app I'm building using Flutter. Unfortunately its not well documented at this point as I've been the only one using it, but essentially the fm synth engine just expects to be initialised and then receive small arrays of bytes containing MIDI messages, even loading the patchsets is via MIDI sysex's. You can look at the source here to get an idea of whats involved doing it via JNI, though I changed a little bit the C API in my library and mine is also using Miniaudio as the audio engine.


Default weather module fix: Hourly shows 4 hours ahead

Hello,
I was running into this issue where the hourly weather reports show 4 hours in advance. Currently, my clock says, 23:40, while my first hourly entry for the weather is 03:00. My calendar is accurate as events show in their proper times. I'm just not sure why this hourly one is being funky. I appreciate your help in advance.
Respectfully,
Jordan

can you show your config?   do you have the lat/lng backwards?

@Oleson-Cybersec

this very old issue which was transfered from the doc repo, we got no feedback since Feb., so closing for now


c asserts not being triggered

I imagine this could be the way I've compiled Proteus, but on several occasions I've noticed that asserts inside the c-code which should have been triggered were not.
In particular I just noticed that the assert in superluWrappersModule.c on line 122 was not triggered in a simulation I'm working on.
I realize this issue is a bit vague but I wanted to post it for future reference.

The macro NDEBUG usually determines whether asserts are enforced.


App hasn't restarted when server came back up

The retrosnub server fell over about 30d ago, but I didn't realise the app hadn't restarted until my mother told she hasn't been getting her messages.
I (well, Simon) started the app manually on the stage.takeyourmeds.co.uk server as user takeyourmeds, and the website can now be seen at stage.takeyour.meds.co.uk
The "takeyourmeds" user does not exist on live.takeyourmeds.co.uk
@lamby Could you clarify:

if the apps are supposed to restart themselves after the server falls over
Which user the live app runs as, and, if the app runs as takeyourmeds on the stage server, why the users are different
What I have misunderstood here

On mobile right now so can't investigate for few hours but in meantime can you clarify "started the app manually"? (Everything should be managed by/via systemd)

Back on my laptop. Not quite sure what happened here. (I see that "manual" meant starting it via a manual call to runserver!)

Which user the live app runs as, and, if the app runs as takeyourmeds on the stage server, why the users are different

Canonically, everything is managed via systemd here:
https://github.com/takeyourmeds/takeyourmeds-web/blob/master/debian/takeyourmeds.takeyourmeds-gunicorn.service#L9
(ie. www-data). There should be no difference between live and stage! (cont)

if the apps are supposed to restart themselves after the server falls over

The apps start themselves upon server restart, yes. In the process of confirming this on stage and live,  this also ensured that things are being run as the systemd-specified user. Maybe there was something stale lying around...? Very odd that it worked for so long…

(FYI I'm using https://updown.io for monitoring; can really recommend it and it costs literally pennies)


Type safety Java Error in eclipse (10 cases)

We should clean this up so a developer can see new issues when changing code.
Description	Resource	Path	Location
Type safety: The method add(Object) belongs to the raw type ArrayList. References to generic type ArrayList should be parameterized	DefaultUDeployClient.java	/udeploy-deployment-collector/src/main/java/com/capitalone/dashboard/collector	line 242
Type safety: The expression of type Map needs unchecked conversion to conform to Map<String,String>	Dashboard.java	/core/src/main/java/com/capitalone/dashboard/model	line 84
Type safety: The expression of type Map needs unchecked conversion to conform to Map<? extends String,? extends String>	PipelineServiceImpl.java	/api/src/main/java/com/capitalone/dashboard/service	line 106
Type safety: The expression of type List needs unchecked conversion to conform to List	PipelineServiceTest.java	/api/src/test/java/com/capitalone/dashboard/service	line 70
Type safety: The expression of type Collection needs unchecked conversion to conform to Collection	CollectorServiceTest.java	/api/src/test/java/com/capitalone/dashboard/service	line 36
Type safety: The constructor HttpEntity(MultiValueMap) belongs to the raw type HttpEntity. References to generic type HttpEntity should be parameterized	DefaultHudsonClientTests.java	/jenkins-build-collector/src/test/java/com/capitalone/dashboard/collector	line 105
Type safety: The constructor HttpEntity(MultiValueMap) belongs to the raw type HttpEntity. References to generic type HttpEntity should be parameterized	DefaultHudsonClientTests.java	/jenkins-build-collector/src/test/java/com/capitalone/dashboard/collector	line 119
Type safety: The constructor HttpEntity(MultiValueMap) belongs to the raw type HttpEntity. References to generic type HttpEntity should be parameterized	DefaultHudsonClientTests.java	/jenkins-build-collector/src/test/java/com/capitalone/dashboard/collector	line 133
Type safety: The constructor HttpEntity(MultiValueMap) belongs to the raw type HttpEntity. References to generic type HttpEntity should be parameterized	DefaultJenkinsClientTest.java	/jenkins-cucumber-test-collector/src/test/java/com/capitalone/dashboard/collector	line 74
Type safety: The constructor HttpEntity(MultiValueMap) belongs to the raw type HttpEntity. References to generic type HttpEntity should be parameterized	DefaultJenkinsClientTest.java	/jenkins-cucumber-test-collector/src/test/java/com/capitalone/dashboard/collector	line 88

There are a few cases left to be addressed:
Type safety: The expression of type Collection needs unchecked conversion to conform to Collection	CollectorServiceTest.java	/api/src/test/java/com/capitalone/dashboard/service	line 36	Java Problem
Type safety: The expression of type Collection needs unchecked conversion to conform to Collection	DefaultSubversionClient.java	/subversion-collector/src/main/java/com/capitalone/dashboard/collector	line 71	Java Problem
Type safety: The method add(Object) belongs to the raw type ArrayList. References to generic type ArrayList should be parameterized	DefaultUDeployClient.java	/udeploy-deployment-collector/src/main/java/com/capitalone/dashboard/collector	line 242	Java Problem
Type safety: Unchecked cast from Object to Map<String,String>	Dashboard.java	/core/src/main/java/com/capitalone/dashboard/model	line 84	Java Problem
Type safety: Unchecked cast from Object to Map<String,String>	PipelineServiceImpl.java	/api/src/main/java/com/capitalone/dashboard/service	line 106	Java Problem


Update README.md with Open in Codeanywhere button

This PR adds "Open in Codeanywhere" badge at the top of Readme file.
Codeanywhere is a cloud-based Integrated Development Environment (IDE) that allows developers to code, learn, build, and collaborate on projects directly from their web browsers. This button opens the Codeanywhere VS Code IDE in the browser, pulls the repo, sets up a container environment and allows the developer to run the code - all of this with a single click and for free.
This removes the local setup hassle and allows developers to start coding and run the app in minutes, straight from the repo page in Github.

你好，你的邮件已收到，稍后将会查看，尽快给你回复。


Skillmap icons off center on small screens

Describe the bug
On a Chromebook screen, the skillmap icons aren't centered

P3 - won't fix.


Error in Add Alert form when upgrading to version 4.0.1

Bug description
When upgrading from version 3.1.3 to 4.0.1, the alert sending configuration form in the Notification method item is validating after adding emails, making it impossible for the alert to be saved.
How to reproduce the bug
1 - Alert & Reports
2 - + Alert
3 -  fill out form
Screenshots/recordings
The ADD button remains disabled even when adding emails:

Superset version
4.0.1
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[X] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

bump

I'm experiencing the same issue, moreover on existing alerts, I cannot update the recipient list from the modal (the only workaround I've found is doing it from the database...)

Any chance of a fix coming out for this modal?

This was reported in https://github.com/apache/superset/issues/28149 and fixed in https://github.com/apache/superset/pull/28409.  Sorry for the inconvenience, this pull request should be included in the next version (both 4.0.2 and 4.1.0, unclear which will come out first).  Feel free to test the master branch version of Superset to confirm that it is indeed fixed.


Unity2018.2输出AssetBundle包报错

untiy升级为2018后 ,调用BuildPipeline.BuildAssetBundles   会报下错
Assets/ToLua/Generate/UnityEngine_LightWrap.cs(1091,20): error CS1061: Type UnityEngine.Light' does not contain a definition for shadowRadius' and no extension method shadowRadius' of type UnityEngine.Light' could be found. Are you missing an assembly reference?
Assets/ToLua/Generate/UnityEngine_QualitySettingsWrap.cs(1077,32): error CS0200: Property or indexer UnityEngine.QualitySettinAssets/ToLua/Generate/UnityEngine_TextureWrap.cs(817,8): error CS1061: Type UnityEngine.Texture' does not contain a definition for imageContentsHash' and no extension method imageContentsHash' of type `UnityEngine.Texture' could be found. Are you missing an assembly reference?gs.streamingMipmapsRenderersPerFrame' cannot be assigned to (it is read-only)

升级tolua


key problem

My cmd commanod i use: vsd save wholelink/manifest.mpd -o name.mp4 --key f0328832f032f032f032f0328832f032:b4fa767b544197379af4fdf8eda3a61b

CMD output
Key f0328832f032f032f032f0328832f032:6f87daefaedbe78e35f7bdfbf5a7f87dd7fc79d6b76bad5b
Downloaded stream successfully
Processing audio stream language: ?, codecs: mp4a.40.2, bandwidth: 126.95 KiB/s, channels: 2
Downloading stream to vsd_audio_manifest_mpd.mp4
Key f0328832f032f032f032f0328832f032:6f87daefaedbe78e35f7bdfbf5a7f87dd7fc79d6b76bad5b
Downloaded stream successfully

the correct decrytion key is=  f0328832f032f032f032f0328832f032:b4fa767b544197379af4fdf8eda3a61b

So why then in cmd it gets changed?
i'm on the latest build.
Any fix for this?

Ok, I will look into this.


[4.4-overfitting] which model is better? (original model vs L2-regularized model)

This example illustrates the impact of L2 regulation. I can see that the L2-regularied model gets overfitting slowly comparing with the original model. But the peak of validation loss of L2-regularied model is greater than that of the original model. Thus, for this example, the original model is still better than L2-regulaized model, right?

I see that the validation loss profile for the L2 regularized model peaks at around 0.4 in the figure while the standard model peaks at 0.8. The validation loss axis (y axis) is in positive units. From the plot you can also tell that both models start overfitting at 2.5 epochs but the standard model clearly performs poorly (far steeper gradient) after that compared to the regularized model. This must mean the regularized model performs far better.


Feat - Utilise .NET Auth required_scopes

Utilise required_scopes on OAuth requests to reduce chance of user unselecting and then getting an error message.
Disables the ability to deselect the requested scopes.

:tada: This PR is included in version 6.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 6.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added dynamic language switching

Hey,
I needed the ability to change the language of the captcha dynamically. I've added the ability to register language="en|fr|es|etc." attribute on the "no-captcha" tag
<no-captcha expired-callback="expiredCallback"
            g-recaptcha-response="gRecaptchaResponse"
            control="captchaControl"
            language="{{language}}"></no-captcha>

I am by no mean an angular expert so feel free to modify the code to fit yours.
Thanks.

Thanks @running-coder!
It seems to work well, but tests fail. Can you add some tests?

Closing as this is quite old and has no updates


Coordinates for J.Modifier locations

In order to more conveniently add or replace J.Modifier's on VariableDeclarations, MethodDeclarations, ClassDeclarations, etc., it'd be convenient to have access to make changes to J.Modifier's using Coordinates (similar to being able to addAnnotation)
EDIT including more specific examples

We can't use mv.getModifiers().get(0).getCoordinates().before() in this case because if the list of Modifiers is empty, we get an exception-- and "VariableDeclarations" doesn't have an accessor to add/replace the coordinates of modifiers outright.
Specific example from https://github.com/openrewrite/rewrite/pull/298.
mv = mv.withModifiers(
    ListUtils.concat(mv.getModifiers(), new J.Modifier(Tree.randomId(), Space.EMPTY, Markers.EMPTY, J.Modifier.Type.Final))
);

Additionally, from https://github.com/openrewrite/rewrite/blob/71c6e2541387bbc031640adb0708e41902239cdf/rewrite-java/src/main/java/org/openrewrite/java/cleanup/HideUtilityClassConstructorVisitor.java#L125-L135

I agree this would be an improvement. But I suspect there just isn't that much more of interest to do with modifiers than what we already have. We already have recipes that reorder and change modifiers. Their implementations could be simplified if we had this functionality but I don't see that alone as sufficient argument to prioritize it.


Turn ParsingError exceptions into Sphinx warnings

I have some code blocks that intentionally contain syntactically incorrect Python code (https://nbsphinx.readthedocs.io/en/0.8.7/allow-errors.html).
Right now, when using the sphinx_codeautolink extension, I'm getting an extension error.
This is of course very good in most cases, because it allows us to detect mistakes.
However, I would like to be able to ignore this in some cases.
This would be possible if those errors would be turned into Sphinx warnings.
And if those warnings have a type and subtype, they can be ignored while other warnings would still be active.
I think this would be consistent with similar errors like e.g. illegal reStructuredText syntax, which don't stop Sphinx processing and produce warnings instead.
BTW, warnings can easily be turned into errors with the -Wflag.

This seems reasonable on the surface, but I have a counter offer: how would you feel about ignoring those blocks explicitly with ..autolink-skip::? That way ignoring such errors wouldn't mask unwanted failures either.
Do you have other use cases in mind that would benefit from this? I guess one benefit would be that we can go through all the examples and issue warnings instead of stopping at the first and raising.

how would you feel about ignoring those blocks explicitly with ..autolink-skip::?

Yes, when using reST source files that would be a very good solution.
However, I'm talking about using Jupyter Notebooks as source files, in which case I would have to come up with a mechanism to insert those .. autolink-skip:: directives programmatically, e.g. via cell tags or cell metadata.
This would certainly be doable, but IMHO that's too complicated (both to implement and to use) to be worth it.

That way you wouldn't mask unwanted failures either.

Yeah, that would be a disadvantage of ignoring the whole class of warnings, but I would probably be willing to accept that in my case.
It would be a trade-off that users have to decide on their own.

Do you have other use cases in mind that would benefit from this?

Not really. But it seems natural, since parsing errors in Sphinx typically raise warnings (if I'm not mistaken).

In my concrete case I've just removed the offending cells from the notebook (https://github.com/spatialaudio/nbsphinx/pull/616), so this isn't urgent for me.
But still, I think it would generally make sense to use warnings for parsing error.

Fair enough, and I'm warming up to the idea of having warnings regardless of the Jupyter use case 👍 That'd leave ignoring warnings as the user's responsibility while also providing the info.
Implementation considerations remain, like how do we provide the current information for warnings. We could just propagate an exception as we do currently and convert it to a warning, but then we can't parse further examples in the same document. So we should probably pass all the necessary info into the parser and generate warnings there.

Sounds good!
I agree that exceptions should not be used for warnings.
But you can probably use node.document.reporter.warning('blah blah', base_node=node)?
This would already automatically include the path to the source file and the line number (if available) in the warning message, is there any other info needed?
You could probably even include the offending code snippet with  node.document.reporter.warning('blah blah', node, base_node=node).
I don't know whether any of this is an official API, but it seems to work ...

I realized today that node.document.reporter.warning() is a feature of docutils and it doesn't support Sphinx's type and subtype.
However, that's no problem, we don't have to use it, we can just use the standard Sphinx way of creating warnings, something like this:
from sphinx.util.logging import getLogger

...

logger = getLogger(__name__)

...

            logger.warning('blah blah', location=node, type='codeautolink', subtype='syntax_error')

Thanks for digging! I'll have a look next week 👌🏿

Yeah, now having implemented the warnings, using location=node seems like the most elegant way to go!


Bad adaptability to small and vertical screens

Issues

Centering stops working
Repeating background caused by being anchored to width
Projects grid should adapt in size to avoid going off screen

Preview on mobile

Partially resolved by a9c8b3a


React native support

Hello,
would you consider React native support for this library? It would be really amazing not only for me :-).
Thank you,
Krystof

Hello, React native support is not planned at the moment.


⚠ CenterDevice has degraded performance

In 7b0168b, CenterDevice (https://auth.centerdevice.de/login) experienced degraded performance:

HTTP code: 200
Response time: 2095 ms

Resolved: CenterDevice performance has improved in b1d2503.


[nodejs] 14.nlp-with-dispatch

@cleemullins; @yochay - Fixes based on your feedback is in. This PR is ready for another round of review/ merge.

@cleemullins  - thanks for another round of feedback! I've updated the PR with fixes.

const MainDialog = require('./dialogs/mainDialog');

Other comments (such as on line 15) are sentences. Consider making this a valid sentence as well.

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:13 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

const { BotConfiguration } = require('botframework-config');

spelling. configuration

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:16 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

});

Incorrect Bot Name.

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:29 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

const BOT_CONFIGURATION = 'echobot-with-counter';

extra space before "." at the end of the sentence.

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:45 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

let botConfig;

Ends in a ".", yet doesn't have a capital "R".

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:35 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

// A bot requires a sate store to priciest it dialog and user state between messages

Add a "." at the end.

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:57 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

const memoryStorage = new MemoryStorage();

grammer. "priciest"?

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:58 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

// const { CosmosDbStorage } = require('botbuilder-azure');

Remove references to Table Storage.

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:62 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

// const cosmosConfig = botConfig.findServiceByNameOrId(STORAGE_CONFIGURATION);

Casing on Cosmos

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:64 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

// create conversation state with in-memory storage provider.

Ends in a ".". Needs a capital "C".

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:70 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

// register conversation state as a middleware. The ConversationState middleware automatically reads and writes conversation sate

Cleanup casing / punctuation.

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:73 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

server.post('/api/messages', (req, res) => {

needs a "." at the end.

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:80 in 7a3a801. [](commit_id = 7a3a801c4d384132776bfc6f2ddd4e6587105f7e, deletion_comment = False)

    // route to main dialog.

Casing. "Route".

Refers to: javascript_nodejs/2.echobot-with-counter/index.js:82 in 83e9b1e. [](commit_id = 83e9b1ea3bb26b602eda8a3a586766079b0396cc, deletion_comment = False)


Extension causes high cpu load

Issue Type: Performance
Extension Name: azurerm-vscode-tools
Extension Version: 0.15.0
OS Version: Windows_NT x64 10.0.19042
VSCode version: 1.54.3

:warning: Make sure to attach this file from your home-directory:
:warning:c:\Users\brobi\AppData\Local\Temp\msazurermtools.azurerm-vscode-tools-unresponsive.cpuprofile.txt
Find more details here: https://github.com/microsoft/vscode/wiki/Explain-extension-causes-high-cpu-load

@brobinson234 I can't debug the problem unless you attach the cpuprofile file as instructed above.


Useless Errors

React Native is an amazing development platform don't get me wrong...but the LogBox and debugging is almost completely useless and amounts to putting console statements everywhere. i.e. console.log('hey, here...1');
Seriously it's one of the worst debugging experiences ever. The call stack is worthless and the errors are so vague and don't show where it happened, etc.

Can you provide more details and suggestions for improvement?
Do you use remote JS debugging? https://reactnative.dev/docs/debugging

Here is a simple example that shows in a nutshell all that is wrong with the errors in general. It's like it was designed by an internal React Native developer to figure out internal issues but is practically useless for end developers who need to see their own code. Same with the call stack. It's like 50 internal React Native functions instead of showing my code it always shows 1000 lines of React Native code.

I feel your pain. Today I'm trying to debug where this error comes from and no luck. I have around 100 places where it might be and it's very frustrating to check them all.

Thanks for the issue, and I agree I have not had a good experience with the errors. This issue would be better to open at the metro repo https://github.com/facebook/metro.


TFLu: Fix Ethos-U build issue

The issue is that a binary need to be built twice, since it
depends on recursive_find. The driver is not fully downloaded when
recursive_find is called.
The solution proposal is to call the download script
immediately via the make's shell function.
Also add missing exit to error case in download script.

Thanks for contributing to TensorFlow Lite Micro.
To keep this process moving along, we'd like to make sure that you have completed the items on this list:
Read the contributing guidelines for TensorFlow Lite Micro
Created a TF Lite Micro Github issue
Linked to the issue from the PR description
We would like to have a discussion on the Github issue first to determine the best path forward, and then proceed to the PR review.

I'll let @petewarden review this since he is most familiar with the Makefile logic and its repercussions.

Looks good from my side.

@petewarden Ping for review


Can't add merge paths switch to template section

Expected Behavior
We have a RKE Kubernetes cluster, using Calico as network plugin and BIRD as Calico backend.
I can't find the way to add 'merge paths switch 2' to the template section in bird.conf.
This is needed to 'see' our two paths for HA.
Current Behavior
Unable to add a required command to the template section.
Possible Solution
How to interact with confd and the bird.conf
Context
It blocks us from having multiple paths on our Kubernetes network while they are configured and working on the host.
Your Environment

Calico version: 3.4.0 (Docker container in Kubernetes)
Orchestrator version (e.g. kubernetes, mesos, rkt): Rancher RKE
Operating System and version: CentOS 7.6
Link to your project (optional):

Autogenerated bird.conf (redacted)
`
Generated by confd
include "bird_aggr.cfg";
include "bird_ipam.cfg";
router id REDACTED;
Configure synchronization between routing tables and kernel.
protocol kernel {
learn;             # Learn all alien routes from the kernel
persist;           # Don't remove routes on bird shutdown
scan time 2;       # Scan kernel routing table every 2 seconds
import all;
export filter calico_kernel_programming; # Default is export none
graceful restart;  # Turn on graceful restart to reduce potential flaps in
# routes when reloading BIRD configuration.  With a full
# automatic mesh, there is no way to prevent BGP from
# flapping since multiple nodes update their BGP
# configuration at the same time, GR is not guaranteed to
# work correctly in this scenario.
}
Watch interface up/down events.
protocol device {
debug { states };
scan time 2;    # Scan interfaces every 2 seconds
}
protocol direct {
debug { states };
interface -"cali*", ""; # Exclude cali but include everything else.
}
Template for all BGP clients
template bgp bgp_template {
debug { states };
description "Connection to BGP peer";
local as REDACTED;
multihop;
gateway recursive; # This should be the default, but just in case.
import all;        # Import all routes, since we don't know what the upstream
# topology is and therefore have to trust the ToR/RR.
export filter calico_export_to_bgp_peers;  # Only want to export routes for workloads.
source address REDACTED;  # The local address we use for the TCP connection
add paths on;
graceful restart;  # See comment in kernel section about graceful restart.
connect delay time 2;
connect retry time 5;
error wait time 5,30;
}
------------- Node-to-node mesh -------------
Node-to-node mesh disabled
------------- Global peers -------------
No global peers configured.
------------- Node-specific peers -------------
For peer /host/REDACTED/peer_v4/REDACTED
protocol bgp Node_REDACTED from bgp_template {
neighbor REDACTED as REDACTED;
}
For peer /host/REDACTED/peer_v4/REDACTED
protocol bgp Node_REDACTED from bgp_template {
neighbor REDACTED as REDACTED;
}
`
Desired config (only the template section)
`
Template for all BGP clients
template bgp bgp_template {
debug { states };
description "Connection to BGP peer";
local as REDACTED;
multihop;
merge paths switch 2
gateway recursive; # This should be the default, but just in case.
import all;        # Import all routes, since we don't know what the upstream
# topology is and therefore have to trust the ToR/RR.
export filter calico_export_to_bgp_peers;  # Only want to export routes for workloads.
source address REDACTED;  # The local address we use for the TCP connection
add paths on;
graceful restart;  # See comment in kernel section about graceful restart.
connect delay time 2;
connect retry time 5;
error wait time 5,30;
}
`

Just trying to understand what you're trying to do here.. is this a new template or just a configuration that you'd like to add to an existing one?

Hi Rafael,
Thanks for your reply. What I was looking for is modifying an existing
template, the one for bird.cfg. Due to our network setup I needed to add a
couple of lines to the 'kernel' section and I couldn't find a 'clean' way
to do it.
We are using RKE and things like the asNumber I can update but nothing in
the kernel section AFAIK.
For now I'm using a 'workaround' by just putting all the templates in a
Kubernetes configMap and mounting that one in the template directory from
within the daemonSet.
If there is a way I would appreciate if you could point me in the right
direction or towards the documentation.
Kind regards,
Eric Van Steenbergen
E-mail<EMAIL_ADDRESS><EMAIL_ADDRESS>Skype: ericvs2014
LinkedIn: Eric Van Steenbergen
http://es.linkedin.com/pub/eric-van-steenbergen/b/8a4/51b
On Tue, Jul 16, 2019 at 5:10 AM Rafael Vanoni<EMAIL_ADDRESS>wrote:

Just trying to understand what you're trying to do here.. is this a new
template or just a configuration that you'd like to add to an existing one?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/projectcalico/calico/issues/2711?email_source=notifications&email_token=AACCAP34EQTTI27JDQHTLRLP7TRTFA5CNFSM4H7A24J2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ67KEQ#issuecomment-511571218,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACCAPZUZTL67H5HPPSX2OLP7TRTFANCNFSM4H7A24JQ
.

There isn't a supported way for modifying the bird templates currently, I think what you are doing is the best option at this time for modifying them.
If you would like to open a feature request issue or even better submit a PR to make it configurable through other means that would be great.


Revert "Manually close tree while devicelab staging is failing (#74434)"

This reverts commit 444c954f439bb5dce0e9c130c7a37a424ccb5927.
Should NOT be landed until the framework prod_builders.json file has been updated with the mac/android devicelab tasks.
part of (but does not alone fix) https://github.com/flutter/flutter/issues/74431

landing on red flutter-build to fix tree

landing on red flutter-build to fix tree


fix: update similar templates component cards

Description
Goal
To use the new FixedHeightTemplate and BuildingBlock components in the similar templates section of the platform.
Refactor
Removed react-masonry-css and used css grid in implementing similar templates list
Added a width: 100%; style declaration to the Wrapper component in TemplateView.tsx to maintain full width
PR fixes following issue(s)
Fixes #29983
Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

Testing
How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Summary by CodeRabbit

Refactor

Updated the rendering logic in the SimilarTemplates component for enhanced conditional display of elements based on template features.
Added a width: 100%; style declaration to the Wrapper component in TemplateView.tsx.

/build-deploy-preview

/ok-to-test<EMAIL_ADDRESS>

@jacquesikot  Also, I have made changes for pointing to prod server url.
We should be able to check if everything looks good on prod as well.

@jacquesikot Should we have some unit tests for this?


Find existing research on reticles in AR and VR

@rafai perhaps you can work with @poplockanddrop on this. I found this oculus rift best practices document with relevant info. http://static.oculus.com/sdk-downloads/documents/Oculus_Best_Practices_Guide_0.5.0.pdf
Targeting reticles are an excellent illustration of adapting old paradigms to VR. While a reticle is critical for accurate aiming, simply pasting it over the scene at a fixed depth plane will not yield the reticle behavior players expect in a game. If the reticle appears at a depth plane different from where the eyes are converged, it is perceived as a double image. In order for the targeting reticle to work the same way it does in traditional video games, it must be drawn directly onto the object it is targeting on screen, presumably where the user’s eyes are converged when aiming. The reticle itself can be a fixed size that appears bigger or smaller with distance, or you can program it to maintain an absolute size to the user; this is largely an aesthetic decision for the designer. This simply goes to show that some old paradigms can be ported over to VR, but not without careful modification and design for the demands of the new medium.```

Going to close this, there doesn't seem to be any activity. We can reopen if necessary.

Very cool. Also read through Google design principles for VR: http://www.google.com/design/spec-vr/designing-for-google-cardboard/a-new-dimension.html

Any take aways?


cmd/compile: sync/atomic.SwapPointer arguments escape for the wrong reason

In the implementation of sync/atomic.SwapPointer (which is in runtime/atomic_pointer.go), there's nothing that causes the "new" argument to escape, even though it needs to (because you could be swapping it in to a global). The compiler agrees:
$ go build -gcflags "-m -m" -a runtime |& grep sync_atomic_SwapPointer
runtime/atomic_pointer.go:58: sync_atomic_SwapPointer ptr does not escape
runtime/atomic_pointer.go:58: sync_atomic_SwapPointer new does not escape

However, you can't actually tickle this. For example, in principle the following program should sneak the address of stack variable y into a global variable:
package main

import (
    "sync/atomic"
    "unsafe"
)

var x unsafe.Pointer

func main() {
    var y int
    z := unsafe.Pointer(&y)
    atomic.SwapPointer(&x, z)
}

But it doesn't!
$ go build -gcflags "-m -m" x.go
# command-line-arguments
./aesc.go:13: &x escapes to heap
./aesc.go:13:   from &x (passed to function[unknown]) at ./aesc.go:13
./aesc.go:11: moved to heap: y
./aesc.go:12: &y escapes to heap
./aesc.go:12:   from z (assigned) at ./aesc.go:12
./aesc.go:12:   from z (passed to function[unknown]) at ./aesc.go:12

This happens to work right now, but for the wrong reason: because SwapPointer is defined in the runtime and we use a "go:linkname" comment to expose it from sync/atomic, there's no escape information at all when the test program calls SwapPointer (hence the "passed to function[unknown]"), so the compiler conservatively assumes everything escapes. (In fact, the first argument doesn't have to escape and we have annotations in the runtime to that effect, but they never make it through.)
This all seems very fragile.
/cc @dr2chase @randall77

Fragile, but not broken.

Still fragile, but still not broken.

Still fragile, but still not broken with newescape too.


Disclosure icons and header line number alignment needs some work

As the title suggests - there seems to be some issue with alignment of headers, disclosure icons, and line numbers. Can that be fixed? Thanks!

I use a monospace font for editing, but the alignment is the same with other types of fonts as well.


Fix stream not being flushed

Currently this transport only sends the first log to Rollbar. All subsequent logs entirely bypass this transport. This happens because callback is never called because the cb is not bound to the current this.
This regression was introduced in cd6225966 when cb was converted from an arrow function (implicit binding) to a function expression (unbound).

Thanks for the PR. Published v3.0.2


Using TypeConverters for extended Messaging capabilities

Feature description
I'm using a layer of abstraction on top of your library, and as I'm writing some features for my abstraction, I have an idea for an improvement. I could possibly even help with the request, but I'd like to know if you see any issues with this or don't want it implemented.
In Message, you can only transfer certain of the basic, almost "primitive types", such as Guid, Int32, Int64, String, Decimal, etc.
What about incorporating TypeConverters? That way, if a person passes through an object and dictates how it should be converted to and from another type, then it will automatically know how to parse it.
The use-cases I can see it for include if I want to pass in and retrieve a complex dto, but I'm fine serializing it with Newtonsoft, and if I want to send and receive an enum value.
Feature in action
        /// <summary>
        /// The <see cref="TypeConverter"/> for Notification so that Artemis Library doesn't need to import the Newtonsoft Package.
        /// </summary>
        private class NotificationTypeConverter : TypeConverter
        {
            public override Object ConvertFrom(ITypeDescriptorContext context, CultureInfo culture, Object value)
            {
                if (value is string str)
                {
                    return JsonConvert.DeserializeObject<Notification>(str);
                }

                return base.ConvertFrom(context, culture, value);
            }

            public override Object ConvertTo(ITypeDescriptorContext context, CultureInfo culture, Object value, Type destinationType)
            {
                if (destinationType == typeof(string) && value is Notification notification)
                {
                    return JsonConvert.SerializeObject(notification);
                }

                return base.ConvertTo(context, culture, value, destinationType);
            }
        }

    [TypeConverter(typeof(NotificationTypeConverter))]
    public class Notification
    {
        public Guid UserId { get; set; }

        public Decimal Value1 { get; set; }

        public DateTime RequestDateTime { get; } = DateTime.Now;
        private class NotificationTypeConverter {.../*see above snippet*/}
   }

Or, for the case of enums, or classes that you don't have access to assign the attribute TypeConverterAttribute to, you would do something like:
TypeDescriptor.AddAttributes(typeof(MyEnum), new TypeConverterAttribute(typeof(EnumerationTypeConverter<MyEnum>))));

Describe alternatives you've considered
More layers of abstraction are always possible, that's why this is, I'm sure lower on the priority queue.
Additional context
Like I said above, I don't think it'd be too hard to accomplish. In the Message.cs file, in private static RestrictedDescribed GetBodySection(object body), you'd add a reference to some bit of code, and then in the public T GetBody<T>() code, or in InnerMessage.Body Getter, you'd add another reference to the bit of code. And that bit of code would look something like this:
        var body = message.GetBody<string>();
        var converter = TypeDescriptor.GetConverter(typeof(T));
        if (body == null || !converter.CanConvertFrom(typeof(string)))
        {
            return default;
        }

        return (T?)converter.ConvertFromInvariantString(body);

Thanks. I haven't looked into it. I'll look into it! I read this and forgot to close it, I apologize.


Linux isn't same as Debian/Ubuntu

In README.md#installing-mono, "Linux" installation guides is using apt-get, while it's only Debian's and Ubuntu's package manager, not all Linux distros have apt.

Feel free to provide a pull request to fix this.


recursive mode  default is off

i want to set mode is on .but everytime i have to press 'r' . i want to set default is on .
what should i do ??

Add 'let g:EasyGrepRecursive=1' to your .vimrc
On Sun, Jun 21, 2015 at 4:44 AM daben1990<EMAIL_ADDRESS>wrote:

i want to set mode is on .but everytime i have to press 'r' . i want to
set default is on .
what should i do ??
—
Reply to this email directly or view it on GitHub
https://github.com/dkprice/vim-easygrep/issues/21.

thx


: Timeline component

we would like to build timeline block for Awes.io.

Technical requirements:

component for user picture
integrate models as we have integrated in
button show more, will load additional items from endpoint and render on the page
datatime we print as it is

┆Attachments: Screenshot 2019-11-27 at 10.26.37.png

➤ Ilya Kitayev commented:
Usage example

<AwTimelineBuilder :collection="collection">
    <template #default="{ title, subtitle, author: { fullName, avatar }, createdAt }">

{{ title }}

{{ subtitle }}

        <AwAvatar :src="avatar.src" :name="fullName" /> {{ fullName }}

{{ someConvertMethod(createdAt) }}

    </template>
</AwTimelineBuilder>

➤ Артем Ломако commented:
https://git.awescode.com/awes-io/ui/merge_requests/6/diffs ( https://git.awescode.com/awes-io/ui/merge_requests/6/diffs )


[JavaScript] NPM Package 3.0.0

As far as I understood, the JavaScript implementation in this repository is supposed to replace this NPM package, which is already released (version 2.12.2) and accessible from here.
Now I'm wondering:

Is this implementation production ready? (cucumber-js is still using this old no-longer-supported JS parser)
When do you plan release of Gherkin NPM package 3.0.0?
Where can I find it's public API documentation?

Looking at src code (especially test suite) it seems that some bits are missing to make it even a release candidate. Anyway, it would be nice to have a reliable status of this library in README.md so we can push release forward.

It's fairly stable and feature complete, but some minor parts of the API might change before we cut the first release
Hopefully before Sept 2015, but no promises made
There is no documentation beyond what's in the README.

Can you be more expicit about what you think is missing?
What are you planning to use the library for? What are you making?

Sorry, I made my judgement based on the numer of unit tests run and reported here. Not sayin' that something is missing or does not work though.
I'm working on FeatureBook which is a specification editor/browser for non technical folks.
Another project I'm interested in is ez-bdd. Essentially, this one is a command line tool that transforms a Gherkin source file into a Jasmine spec scaffold.
In both cases we need a reliable Gherkin parser accessible from NPM repo.

3.0.0 is released to NPM!

Great! Thank you :dancers:


abi_stable::sabi_trait triggering repr_transparent_external_private_fields warning with latest nightly rustc

After upgrading my rustc to rustc 1.65.0-nightly (2befdefdd 2022-08-06), following warning appeared and I think it should be fixed.
   |
60 | #[abi_stable::sabi_trait]
   | ^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: `#[warn(repr_transparent_external_private_fields)]` on by default
   = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
   = note: for more information, see issue #78586 <https://github.com/rust-lang/rust/issues/78586>
   = note: this struct contains `UnsafeIgnoredType<SyncSend>`, which contains private fields, and makes it not a breaking change to become non-zero-sized in the future.
   = note: this warning originates in the attribute macro `abi_stable::sabi_trait` (in Nightly builds, run with -Z macro-backtrace for more info)

https://github.com/rust-lang/rust/issues/78586

Note this is now also present in 1.64.0 (stable).

@rodrimati1992 To what degree does abi_stable depend on this behavior? Can this warning be fixed in a non-breaking change to stable_abi?


Change chapter for Assessment on load

Right now, it passes a prop to ControlBarProps for display, but nothing happens to the actual context.

Yup


Add sharing

It's impossible to share the game when it is installed.  It should be possible to easily share the URL of the game.
Some options:

Add a share button to the home screen and the win screen
Add a general share button to the entire UI (bottom left or right or something like that)
Share the current state of a game with some one (encode the game level into the URL)

yeah original design had that in About page. We should add them.


Add For component

Description
Used to loop over an array and render a component for each item.

https://www.chakra-ui.com/docs/components/for

Problem Statement/Justification
<For each={["One", "Two", "Three"]}>
  {(item, index) =>
{item}
}
</For>

Proposed Solution or API
No response
Alternatives
No response
Are you willing to participate in implementing this feature and create a pull request include the implementation?
Yes
Additional Information
No response

Fixed #3301


Make _singleLineVerticalPadding a changeable parameter of SnackBar

@joseph-maher
Can you provide description for this feature request ? Check this issue template.

@darshankawar
The _singleLineVerticalPadding is what surrounds the content widget within a snackbar, which is currently set to 14.0, and presently there is no way to change it, hence the request

@darshankawar
Can I handle this ticket? I currently have a PR for SnackBar. #77514


Web Compliance portal found accessibility issues on adaptivecards.io

https://webcompliance.trafficmanager.net/ScanResultsApi/GetAccessibilityScanConsolidatedReport?siteId=a9e04ff4-f488-e711-80c3-0004ffc0ff2c (internal link)

This isn't actually affecting the accessibility score (at least, not directly). So I'm retagging this issue a bit.


Installation bug - ERROR: Cannot bind argument to parameter 'Path' because it is null.

Describe the bug and expected behavior
Installation fails on first reboot AFTER debloat.vm runs. Specific error is:
C:\Windows\system32>Call PowerShell -NoProfile -ExecutionPolicy bypass -command "Import-Module 'C:\ProgramData\boxstarter\Boxstarter.Bootstrapper\boxstarter.bootstrapper.psd1';Invoke-Boxstarter -RebootOk -NoPassword:$False"
Boxstarter Version <IP_ADDRESS>
(c) 2018 Chocolatey Software, Inc, 2012 - 2018 Matt Wrock. https://boxstarter.org

Boxstarter: Disabling Automatic Updates from Windows Update
++ Boxstarter starting Calling Chocolatey to install installer.vm. This may take several minutes to complete...
Chocolatey v2.2.2
Installing the following packages:
installer.vm
By installing, you accept licenses for the packages.
Progress: Downloading installer.vm <IP_ADDRESS>31009... 100%

installer.vm v<IP_ADDRESS>31009 (forced)
installer.vm package files install completed. Performing other installation steps.
 *** LOADING BOXSTARTER ***
Boxstarter: *** HELLO WORLD ***
Cannot find path 'C:\ProgramData\_VM\packages.xml' because it does not exist.
2023/10/13 19:06:34 [installer.vm] vm.common.psm1 [+] ERROR : [ERR] Cannot bind argument to parameter 'Path' because it is null.
At C:\ProgramData\chocolatey\lib\installer.vm\tools\chocolateyinstall.ps1:9 char:36
+     $configXml = [xml](Get-Content $configPath)
+                                    ~~~~~~~~~~~
ERROR: Cannot bind argument to parameter 'Path' because it is null.
The install of installer.vm was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\installer.vm\tools\chocolateyinstall.ps1'.
 See log for details.

Chocolatey installed 0/1 packages. 1 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Failures
 - installer.vm (exited -1) - Error while running 'C:\ProgramData\chocolatey\lib\installer.vm\tools\chocolateyinstall.ps1'.
 See log for details.
++ Boxstarter finished Calling Chocolatey to install installer.vm. This may take several minutes to complete... 00:00:15.0862313
True
Boxstarter: Restore Automatic Updates from Windows Update
Boxstarter: Enabling UAC
Type ENTER to exit:

To Reproduce
Steps to reproduce the behavior:

Install script using .\install.ps1 -cli <password> in an Administrator Powershell session.

Screenshots

Version

OS: Windows 10 22H2 Build 19045.2965

Additional context
To repeat, this happens right after the first reboot, AFTER debloat.vm runs.
I believe this is due to this merge - https://github.com/mandiant/commando-vm/pull/306 which renamed packages.xml to config.xml which does exist in the C:\ProgramData\_VM\

I've noticed the issue is that packages.xml is not present in _VM folder. Part of the script checks which profile you selected and copies proper config accordingly. After an error occurred, I've copied manually (in my case) .\Profiles\Default.xml to C:\ProgramData\_VM\packages.xml and that did the trick.
I might have missed setting ProfileName/ProfilePath when executing command, hence the issue. Would have to go with fresh install to test that. (snapshot, eh?)


Revert "Update readme icon for education"

Reverts github-community/community#18044

.patch

First, try to host your logo and then add it to Github.
You may hot just by raising an issue and uploading your logo on GitHub.

Languages and Tools:

Ayzat

Ну что творится да

Well, what's going on

？？？？

Alexalucky:revert-18044-education-icon-update

Languages and Tools:

Thanks to everyone

@BLINGBOARD


OMOP CDM docs

Work in progress.

@Iain-S I will create new issues for the new suggested PRs and work on them.


defining V(x): defining common function forms and creating ways for the user to describe which function they want to use and define parameter values

are we still wanting to do this as we have a completely generalised form for user defined potentials

are we still wanting to do this as we have a completely generalised form for user defined potentials

I think it's still worth adding the option for some preset potentials, since the user defined potentials now require the user to input using numpy

are we still wanting to do this as we have a completely generalised form for user defined potentials

I think it's still worth adding the option for some preset potentials, since the user defined potentials now require the user to input using numpy

I edited the part of the code where you input v(x) to allow for a preset option, which I arbitrarily defined x^2 but can be easily amended in future to allow editing of parameters w/in preset form (like selecting order of polynomial, whether or not it is polynomial or trigonometric, etc)
the definition of V is as follows:

#defining V
def V():

    print('how would you like to input potential?')
    print('---')
    print('1. preset format')
    print('2. custom')
    porc = int(input('enter your preference:'))

    if porc == 1:
        vdef = x_array**2
        return(vdef)

    elif porc == 2:
        String = str(input("Please input potential using Python.numpy formatting."))
        command = """def f(x):
            return """ + String
        exec(command, globals())

        return f(x_array)

    else:
        print('please enter only either 1 or 2 to indicate your preference')
        #how do i stop the function in this case?

are we still wanting to do this as we have a completely generalised form for user defined potentials

I think it's still worth adding the option for some preset potentials, since the user defined potentials now require the user to input using numpy

I edited the part of the code where you input v(x) to allow for a preset option, which I arbitrarily defined x^2 but can be easily amended in future to allow editing of parameters w/in preset form (like selecting order of polynomial, whether or not it is polynomial or trigonometric, etc)
the definition of V is as follows:

#defining V
def V():

    print('how would you like to input potential?')
    print('---')
    print('1. preset format')
    print('2. custom')
    porc = int(input('enter your preference:'))

    if porc == 1:
        vdef = x_array**2
        return(vdef)

    elif porc == 2:
        String = str(input("Please input potential using Python.numpy formatting."))
        command = """def f(x):
            return """ + String
        exec(command, globals())

        return f(x_array)

    else:
        print('please enter only either 1 or 2 to indicate your preference')
        #how do i stop the function in this case?

However, there is the issue here of the user inputting an option other than 1 or 2, which just returns a default error. It would be useful to be able to give a customised error with a message like that which I put in the 'else' statement

are we still wanting to do this as we have a completely generalised form for user defined potentials

I think it's still worth adding the option for some preset potentials, since the user defined potentials now require the user to input using numpy

I edited the part of the code where you input v(x) to allow for a preset option, which I arbitrarily defined x^2 but can be easily amended in future to allow editing of parameters w/in preset form (like selecting order of polynomial, whether or not it is polynomial or trigonometric, etc)
the definition of V is as follows:

#defining V
def V():

    print('how would you like to input potential?')
    print('---')
    print('1. preset format')
    print('2. custom')
    porc = int(input('enter your preference:'))

    if porc == 1:
        vdef = x_array**2
        return(vdef)

    elif porc == 2:
        String = str(input("Please input potential using Python.numpy formatting."))
        command = """def f(x):
            return """ + String
        exec(command, globals())

        return f(x_array)

    else:
        print('please enter only either 1 or 2 to indicate your preference')
        #how do i stop the function in this case?

However, there is the issue here of the user inputting an option other than 1 or 2, which just returns a default error. It would be useful to be able to give a customised error with a message like that which I put in the 'else' statement

added a default V value which should act as a good enough failsafe


Update String property field only when text has changed

Avoids resetting the cursor position when the inspector updates while editing a string property.
Fixes #42488

Thanks!

Thanks!

Cherry-picked for 3.2.4.

Cherry-picked for 3.2.4.


running gazelle breaks BUILD file containing gomock rules

Let me qualify "breaks":
At master (dd57d5599d64ebde72aff1c91df41ef32baece51):
Test everything works:
$ bazel test //...
INFO: Analyzed 14 targets (64 packages loaded, 7078 targets configured).
INFO: Found 13 targets and 1 test target...
INFO: Elapsed time: 4.534s, Critical Path: 3.45s
INFO: 75 processes: 14 internal, 61 linux-sandbox.
INFO: Build completed successfully, 75 total actions
//tests:go_default_test                                                  PASSED in 0.1s

Executed 1 out of 1 test: 1 test passes.
INFO: Build completed successfully, 75 total actions

As does, of course:
$ bazel test //tests:go_default_test
INFO: Analyzed target //tests:go_default_test (0 packages loaded, 0 targets configured).
INFO: Found 1 test target...
Target //tests:go_default_test up-to-date:
  bazel-bin/tests/go_default_test_/go_default_test
INFO: Elapsed time: 0.084s, Critical Path: 0.00s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
//tests:go_default_test                                         (cached) PASSED in 0.1s

Executed 0 out of 1 test: 1 test passes.

INFO: Found 1 target...
Target //:gazelle up-to-date:
  bazel-bin/gazelle-runner.bash
  bazel-bin/gazelle
INFO: Elapsed time: 0.244s, Critical Path: 0.01s
INFO: 1 process: 1 internal.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   tests/BUILD.bazel

no changes added to commit (use "git add" and/or "git commit -a")

$ git diff
diff --git a/tests/BUILD.bazel b/tests/BUILD.bazel
index 9dad7cd..017bf1b 100644
--- a/tests/BUILD.bazel
+++ b/tests/BUILD.bazel
@@ -46,8 +46,8 @@ go_test(
     srcs = [
         "gomock_test.go",
         ":helloer_reflect_mock",
-        ":helloer_source_mock",
         ":helloer_reflect_mock_with_mock_names",
+        ":helloer_source_mock",
     ],
     args = [
         "-srcWithCopyright=$(location :helloer_mock_with_copyright)",
@@ -59,7 +59,25 @@ go_test(
     ],
     deps = [
         "@com_github_golang_mock//gomock:go_default_library",
-        "@io_bazel_rules_go//go/tools/bazel:go_default_library",
         "@com_github_google_go_cmp//cmp:go_default_library",
+        "@io_bazel_rules_go//go/tools/bazel:go_default_library",
+    ],
+)
+
+go_library(
+    name = "go_default_library",
+    srcs = [
+        "hello_mock.go",
+        "hello_mock_renamed.go",
+        "helloer_copyright.go",
+        "helloer_source_mock.go",
     ],
+    importpath = "github.com/jmhodges/bazel_gomock/tests",
+    visibility = ["//visibility:private"],
+)
+
+go_binary(
+    name = "tests",
+    embed = [":go_default_library"],
+    visibility = ["//visibility:public"],
 )

Now, while bazel test //tests: succeeds:
$ bazel test //tests:go_default_test
INFO: Analyzed target //tests:go_default_test (2 packages loaded, 22 targets configured).
INFO: Found 1 test target...
Target //tests:go_default_test up-to-date:
  bazel-bin/tests/go_default_test_/go_default_test
INFO: Elapsed time: 0.831s, Critical Path: 0.70s
INFO: 7 processes: 1 internal, 6 linux-sandbox.
INFO: Build completed successfully, 7 total actions
//tests:go_default_test                                                  PASSED in 0.1s

Executed 1 out of 1 test: 1 test passes.
INFO: Build completed successfully, 7 total actions

The additional stuff that gazelle put (that it should not have put) in the BUILD file causes the global tests to fail:
$ bazel test //...
INFO: Analyzed 16 targets (27 packages loaded, 301 targets configured).
INFO: Found 15 targets and 1 test target...
ERROR: /home/ji/bazel_gomock/tests/BUILD.bazel:67:11: GoCompilePkg tests/go_default_library.a failed (Exit 1): builder failed: error executing command bazel-out/host/bin/external/go_sdk/builder compilepkg -sdk external/go_sdk -installsuffix linux_amd64 -src bazel-out/k8-fastbuild/bin/tests/hello_mock.go -src ... (remaining 17 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox builder failed: error executing command bazel-out/host/bin/external/go_sdk/builder compilepkg -sdk external/go_sdk -installsuffix linux_amd64 -src bazel-out/k8-fastbuild/bin/tests/hello_mock.go -src ... (remaining 17 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
compilepkg: missing strict dependencies:
        /home/ji/.cache/bazel/_bazel_ji/7a27ddd7c8683af35cb30bc7964830b9/sandbox/linux-sandbox/35/execroot/com_github_jmhodges_bazel_gomock/bazel-out/k8-fastbuild/bin/tests/hello_mock.go: import of "github.com/golang/mock/gomock"
        /home/ji/.cache/bazel/_bazel_ji/7a27ddd7c8683af35cb30bc7964830b9/sandbox/linux-sandbox/35/execroot/com_github_jmhodges_bazel_gomock/bazel-out/k8-fastbuild/bin/tests/hello_mock_renamed.go: import of "github.com/golang/mock/gomock"
        /home/ji/.cache/bazel/_bazel_ji/7a27ddd7c8683af35cb30bc7964830b9/sandbox/linux-sandbox/35/execroot/com_github_jmhodges_bazel_gomock/bazel-out/k8-fastbuild/bin/tests/helloer_copyright.go: import of "github.com/golang/mock/gomock"
        /home/ji/.cache/bazel/_bazel_ji/7a27ddd7c8683af35cb30bc7964830b9/sandbox/linux-sandbox/35/execroot/com_github_jmhodges_bazel_gomock/bazel-out/k8-fastbuild/bin/tests/helloer_source_mock.go: import of "github.com/golang/mock/gomock"
No dependencies were provided.
Check that imports in Go sources match importpath attributes in deps.
INFO: Elapsed time: 0.599s, Critical Path: 0.14s
INFO: 7 processes: 5 internal, 2 linux-sandbox.
FAILED: Build did NOT complete successfully
//tests:go_default_test                                               NO STATUS

FAILED: Build did NOT complete successfully

If you don't want gazelle to modify build files you can use keep directives


new(scripts): support InRelease file creation for publish-deb script.

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
Newest Debian/Ubuntu versions look for this file instead of Release + Release.asc, hitting lots of 404 (and a retry on the old format).
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

build(scripts): publish deb repo has now a InRelease file

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: FedeDP
To complete the pull request process, please assign leogr after the PR has been reviewed.
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[WIP] Fixing crash issue if multitasking is turned on

Working on the same fix that I applied to OpenVR to prevent crashing when Godot is in multi threading mode.

This is now seriously crashing because we are calling OpenGL functions in initialise while we moved initialising GLAD to our process loop.
Need to possibly move building our texture chain to the process function as well.

This may no longer be necesairy after the changes I did in #22


Update Command Line 'Usage' for pe-reports

🗣 Description
Proposed in issue - https://github.com/cisagov/pe-reports/issues/54#issuecomment-926128289 to simplify command-line inputs and build continuity to both the pe-reports and pe-mailer module usage.
💭 Motivation and context
Implementation of the following 'Usage' would be applied. Updates are made specifically to pe-reports module.
Reports
Usage:
  pe-reports [--pe-report-dir=DIRECTORY] [--db-creds-file=FILENAME] [--log-level=LEVEL]

Arguments:
  -r --pe-report-dir=DIRECTORY  Directory containing the pe-reports output.
  -c --db-creds-file=FILENAME   A YAML file containing the Cyber
                                Hygiene database credentials.
                                [default: /secrets/database_creds.yml]
Options:
  -h --help                     Show this message.
  -v --version                  Show version information.
  --log-level=LEVEL             If specified, then the log level will be set to
                                the specified value.  Valid values are "debug", "info",
                                "warning", "error", and "critical". [default: info]

Mailer
Usage:
  pe-mailer [--pe-report-dir=DIRECTORY] [--db-creds-file=FILENAME] [--log-level=LEVEL]

Arguments:
  -r --pe-report-dir=DIRECTORY  Directory containing the pe-reports output.
  -c --db-creds-file=FILENAME   A YAML file containing the Cyber
                                Hygiene database credentials.
                                [default: /secrets/database_creds.yml]
Options:
  -h --help                     Show this message.
  -v --version                  Show version information.
  -s --summary-to=EMAILS        A comma-separated list of email addresses
                                to which the summary statistics should be
                                sent at the end of the run.  If not
                                specified then no summary will be sent.
  -t --test_emails=EMAILS       A comma-separated list of email addresses
                                to which to test email send process. If not
                                specified then no test will be sent.
  -l --log-level=LEVEL          If specified, then the log level will be set to
                                the specified value.  Valid values are "debug", "info",
                                "warning", "error", and "critical". [default: info]

So essentially to run these in sync the following cmd-lines could go:
% <Connect to CyHy DB>
% pe-reports -r /output_dir -c /connect_cyhy.yml
% <Load AWS SES Profile>
% pe-mailer -r /output_dir -c /connect_cyhy.yml

🧪 Testing
The following processes must run successfully
Install: pip install -e .
Tests: pytest /tests
Additionally pass all pre-commits
pre-commit run --all-files
% <Connect to CyHy DB>
% pe-reports -r /output_dir -c /connect_cyhy.yml
% <Load AWS SES Profile>
% pe-mailer -r /output_dir -c /connect_cyhy.yml

📷 Screenshots (if appropriate)

✅ Checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.987%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
174

Relevant Lines:
217

💛  - Coveralls

Currently out of scope for next 2 sprints. A new solution may be occur due to change in priorities and code architecture.
https://github.com/cisagov/pe-reports/issues/93


Update Mypy tests for new Mypy output

@hynek Think we can merge this now?

After the 3.6 one

this gives us a conflict now because there's no setup.py anymore 🤓

@hynek Any idea why tox keeps installing an old version of Mypy?

Hi @Tinche, is it likely due to this?
https://github.com/python-attrs/attrs/blob/46053d703d99a6e7419fd291cbf546340690d69d/pyproject.toml#L45-L47

Yeah, it should be getting the new version, but it isn't for some reason.

It really looks like tox is installing the tests extras from PyPI.
Tox runs this line in the setup:
python -m pip install --no-compile 'attrs[tests]' 'coverage>=5.3' 'importlib_metadata; python_version < "3.8"'
To test it out I changed the tox env to:
[testenv:py3{7,10,11}]
extras = cov
# Python 3.6+ has a number of compile-time warnings on invalid string escapes.
# PYTHONWARNINGS=d and --no-compile below make them visible during the Tox run.
install_command = python -m pip install --no-compile {opts} {packages}
setenv =
    HATCH_BUILD_HOOK_ENABLE_AUTORUN=1
    COVERAGE_PROCESS_START={toxinidir}/pyproject.toml
    PYTHONWARNINGS=d
commands =
    python -m pip install .[tests]  # <================================= added this
    coverage run -m pytest {posargs:-n auto}

and got 1338 passed, 9 skipped, 1 xfailed in 28.35s

Per my observations, tox4 is not ready to be deployed everywhere blindly. I've had to keep it below v4 in a number of envs and will be evaluating removing the upper bound on the case by case basis, paying close attention at the bugs being fixed meanwhile...

Per my observations, tox4 is not ready to be deployed everywhere blindly. I've had to keep it below v4 in a number of envs and will be evaluating removing the upper bound on the case by case basis, paying close attention at the bugs being fixed meanwhile...

weird… wann try pinning 'tox<4' in ci Tin?

Cool, that fixed it.
Do we want to let Tox maintainers know?

Cool, that fixed it.
Do we want to let Tox maintainers know?

yeah let's be good citizens


write from POSTGRES to Google Sheets

@RicSegundo I pretty much copied this excellent video: https://www.youtube.com/watch?v=4ssigWmExak&ab_channel=LearnGoogleSpreadsheets
There is also gspread which is a wrapper around the sheets API, but I prefer to directly use the API like in the video.
Things that need to be improved:

there is a creds.json file in the repo that the API reads to connect to the sheet. I tried to add it as a JSON to github secrets but that threw an error in the deploy workflow (couldn't be passed as an environment variable). Also I recall reading somewhere that JSON should not be passed as a secret.
there's no column names in google sheets after writing.

In the cred.json file there is an email address that one must give editor access to in the google sheets

I'll check asap, in principle this weekend!

https://stackoverflow.com/questions/56277661/is-it-possible-to-store-a-json-file-to-an-env-variable-with-dotenv
https://stackoverflow.com/questions/71544103/how-can-we-store-a-json-credential-to-env-variable-in-python

@RicSegundo I solved 2 of the 3 "areas for improvement" listed above:

pass the json through github secrets as base64, and then decode back to json on the VM
write the column names to google sheets

Hope you can support on the last bullet (the general design of the code). Also need to add type hinting...

I have removed sonar-project.properties files and replaced it with .sonarcloud.properties. sonar-project.properties is only used for the CI approach but we're using the automatic approach (recommended)

@RicSegundo, the code works perfectly, but there is a mypy issue that I cannot resolve.
It complains about this line:
super()._DataBaseInteraction__db_connect()
This SO question explains the syntax:
https://stackoverflow.com/a/20261595/5392289

@RicSegundo it should be protected. I've fixed it. You can approve.
https://stackoverflow.com/a/12784107/5392289


HTTPError: 400 Client Error: Computer Vision for TextRecognition

[HTTPError: 400 Client Error: Bad Request for url: https://westcentralus.api.cognitive.microsoft.com/vision/v2.0/recognizeText?mode=Handwritten
The code same as above just changed the key for my account.]
I tried accessing URL directly it gives 404 error.

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

ID: e4c8bd02-16d6-d043-4f2c-a337a70954f5
Version Independent ID: 8fa84a6d-5c1e-3562-43aa-019f968edd0f
Content: Computer Vision Python quickstart handwritten text - Microsoft Cognitive Services
Content Source: articles/cognitive-services/Computer-vision/QuickStarts/python-hand-text.md
Service: cognitive-services
GitHub Login: @noellelacharite
Microsoft Alias: nolachar

@devenderA17010 Thanks for the feedback! We are currently investigating and will update you shortly.

#11078

@devenderA17010 Hi, I have created a work item for this bug, it should be fixed soon. Thanks for your feedback.We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

Hi
This issue seems to have not been resolved. I am getting the following error..
"HTTPError: 400 Client Error: Bad Request for url: https://westcentralus.api.cognitive.microsoft.com/vision/v2.0/recognizeText?mode=Handwritten"
Appreciate if you could help in solving the above issue

Hi
I have the same issue.
400 Client Error: Bad Request for url: https://westcentralus.api.cognitive.microsoft.com/vision/v1.0/analyze?visualFeatures=Categories%2CDescription%2CColor
Is there any solutions now? Thank you.

Something happened to the recognizeText endpoint today:
https://westeurope.api.cognitive.microsoft.com/vision/v2.0/recognizeText?mode=Printed
This worked yesterday. What's going on??

Is there a workaround?? Urgent!

It's working again now, with no changes on my side. What changed on the Azure side if anything?

Turned out to be an outage on the Azure side

@jtlz2 @vashish76 @Lindahe0707 @oninross Hi guys, thank you for your feedback. Could you please open a new ticket if you are still suffering from this issue? The original post is too old. Sorry for all inconvenience and thank you for your post here again.


UI fails to notice successful finish of a second task

Reproduce by creating (not running yet!) two tasks, e.g. using the Books file. Then start the first task, wait few seconds and start the second one.
When the first task finishes, the UI correctly reports that. But when the second one does finish, the UI fails to notice it and still displays the "in progress" icon.
I have checked the requests made by the UI, and it seems it is sending requests for the state of the first task, instead of the second one.

Done; paging branch.


www.chibilinks.com

Can Chibi-links be considered? Thought it would be cool to set up a super lightweight url shortener with some ahem... interesting looks :). Yeah sorry if it hurts your eyes

I am afraid, this website does not satisfy rule 5.


Update Aptana cask for version 3.6.1

The download is now on github.com instead of aptana.com.

Thank you for the contribution. It needed some fixes, so they were made in c6972a90f95dccf530c90dce3a9e5102e6f3c92e. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

No problem. Thanks for your improvements.


Update README.md

Added my bot that uses your API

Thanks for using my API!

@barreeeiroo @martincarrera the link (GO) has a problem, fix it please.


add configuration for preload script.

Can you please add configuration for preload script to bundle it with the app, and if you can make it in typescript it will be amazing.

Hi! If I understand you correctly, you mean you want to use context isolation.
This project supported preload script already and you can write it in typescript. Exactly, this project using another electron-run to run electron application. And electron-run support preload script.
You can write a preload.ts and save it beside your entry point script (in this starter, it's src/main/index.ts), and append --preload preload.ts to your dev command in the package.json.
Check more detail at electron-run docs

The link above is down, unable to open it.

The link above is down, unable to open it.

I can visit it (the site is deployed on GitHub Pages), or you can try https://github.com/jctaoo/electron-run/tree/main/docs#options---preload-file

@jctaoo
Thank you for the replay and unfortunately it didn't work.
here's a screenshot showing what I did.

@jctaoo
Thank you for the replay and unfortunately it didn't work.
here's a screenshot showing what I did.

You should include preload option in your webPreferences when you create BrowserWindow, just like this:
const win = new BrowserWindow({
    width: 800,
    height: 600,
    webPreferences: {
      nodeIntegration: false,
      contextIsolation: true,
      preload: join(__dirname, "preload.js")
    },
    show: false,
  }).once("ready-to-show", () => {
    win.show();
  });

@jctaoo  Thanks it worked now.
do I have to do the same for the build command in package.json ?

@jctaoo  Thank You So Much, keep up the good work sir !


Move close handling to framehandler instead of closehandler

This is because the closehandler doesn't get information about the close code/reason, whereas the close frame will have that info in the framehandler.

We will want to handle gateway ws close codes differently, which means that we need to have some logic in the framehandler to catch close frames.

See also: https://github.com/eclipse/vert.x/issues/2615

Looks like we won't get the nice closing until v.x 4.0 :( https://github.com/eclipse-vertx/vert.x/issues/2615#issuecomment-430569338

Going to just close this now, as v.x 4.0 isn't any time soon, and close handling is done correctly now as of 891808d


Update configure-exchange-server-for-hybrid-modern-authentication.md

This edit is in combination with PR #3330 - modifying the command lines further down and point out that there is the possibility of having the EvoSts server still named without a GUID, depending on when the HCW was run. Re-running the HCW won't update the AuthServer name so these instructions still need to cover both scenarios.

@dstrome -- more Exchange modern auth that needs to be reviewed.


feat(tutorial): complete step 1

Closes #
Completed step 1. My location is Ann Arbor, MI.
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

👍 Looks good - 🎉 You are one step closer to completion of the Carbon React tutorial and earning the digital credential.