ShiftLeft breaking change process on azure-rest-api-specs-pr is confusing and noisy

Per the email thread with subject RE: AKS Breaking change review: Azure/azure-rest-api-specs-pr/pull/11872
Relevant snippet from my email:

Here is my current best guess how things work with my questions to you:
(1)	When a new PR is made against the private spec repo, azure-rest-api-specs-pr, it is assumed that all the previous PRs have been published/released to the public repo, azure-rest-api-specs. Hence if PR 1 added a new API to the private repo and PR 2 made breaking changes to that API in the private repo, it is assumed these are potentially actual breaking changes. @Mike, @Jeffrey is my understanding correct? Is this a good link to read more about this? https://aka.ms/azapi/breakingchanges (I got it from @Jeffrey’s signature guidelines).
(2)	However, ShiftLeft has a flow that violates rule (1). Instead, it treats the private repo as “work in progress”, where one can rapidly iterate on pending changes by pushing multiple PRs to the repo, under the assumption the previous PRs have not yet been published/released. The way it was handled by ShiftLeft is that its PRs were labelled “ShiftLeftApiSpecReview” and “BreakingChange” but not “BreakingChangeReviewRequired”. The idea here is that the PR reviewer would know that if the label “BreakingChangeReviewRequired” is missing and label “ShiftLeftApiSpecReview” is present, it means the “BreakingChange” label is bogus and needs to be ignored. @Matthew Christopher @Ray Chen does this seem correct?
The problem here is that the rule (2) requires memorizing this special case by PR reviewer and may lead to errors where we inadvertently approve breaking change because we though there wasn’t one, but there was. Ideally we want to get rid of this problem, either by moving the ShiftLeft iteration process off the default branches as @Jeffrey suggested, or perhaps have better labelling, like, if the automation detects it is just ShiftLeft changes being iterated on unreleased API, then no breaking changes should be reported by the tooling, at all. No warnings, no labels.

Another instance of the issue "breakin change shall be ignored as it is ShiftLeft":

https://github.com/Azure/azure-rest-api-specs-pr/pull/12179

https://github.com/Azure/azure-rest-api-specs-pr/pull/12179/checks?check_run_id=12929226550


Crashing phonegap app on iOS 8.3

The first call to SCAN results in a query "Allow app to use camera?" then the app crashes.
From that point on, all calls to SCAN result in the app crashing.
This happens on the iPad Air, as well as all of our iPhones (6, 5, 4s).
I'm using version 3.1.2 of phonegap-plugin-barcodescanner.
Here are the other plugins I'm using:
	<gap:plugin name="cordova-plugin-whitelist" 		source="npm" />
	<gap:plugin name="cordova-plugin-statusbar" 		source="npm" />
	<gap:plugin name="cordova-plugin-splashscreen" 		source="npm" />
	<gap:plugin name="cordova-plugin-dialogs" 			source="npm" />
	<gap:plugin name="cordova-plugin-sqlite" 			source="npm" 	version="0.9.0" />
	<gap:plugin name="phonegap-plugin-barcodescanner" 	source="npm" 	version="3.1.2" />
	<gap:plugin name="ionic-plugin-keyboard" 			source="npm"	version="1.0.4" />
	<gap:plugin name="cordova-plugin-datepicker"		source="npm"	version="0.8.0" />
	<gap:plugin name="cordova-plugin-appversion"		source="npm"	version="1.0.0" />
	<gap:plugin name="nl.madebymark.share" 	version="0.1.1" />

I'll dig around a bit and see if I can find which, if any, are conflicting with barcode scanning.
I'll comment on this thread when I find something. Until then, has anyone else run into the app crashing on iOS?

Fixed. I was adding options to the call to .scan, and those options CRASH  the app on iOS devices. Android just ignores them.


How to trigger a pinch zoom (and only that event?)

Hi,
I was looking a lot for an example, but the unique solution I found is this:
		map.getView().on('propertychange', function(e) {
			if (e.key == 'resolution') {
				console.log('Event triggered');
			}
		});

But this is triggering the pan event too.
Please, how could I trigger just only the pinch zoom event?
Thanks a lot!

There is no event that gets triggered on pinch zoom specifically. What are you trying to achieve? I may be missing something, but I don't see why someone would want to know specifically when a user pinch-zoomed.

Hi @ahocevar  :)
I'm developing uNav GPS Navigation, and I would need to control the pinch zoom and the drag events in a different way (for unfocus the view on car).
A hug!

Your code above should work, as long as you do not programmatically change the resolution, e.g. by using ol.View#setZoom()' , ol.View#setResolution()orol.View#fit(). Just drag-panning the map (with one finger on touch devices) will not change the resolution, and the 'change:resolution'event (which you could use instead ofpropertychange`) won't fire.

In my tests pinch the zoom is triggering the drag too :)
A hug!

Your snippet tests for resolution change. The event will be triggered only when zooming, but not when panning using one finger.

@ahocevar Yes :) That is my issue. I would need the zoom event not trigger the pan event :)
My solution by now will be disable the pinch events :P
A hug and thanks for your support!

I edited my above comment to show you how you might be able to combine listening for two different events to determine what the user did.


GetPropertiesOfKeyVersions throws for non-existent key on Managed HSM

The KeyClient.GetPropertiesOfKeyVersions method throws when requesting versions on a non-existent key on Managed HSM because HTTP 404 is returned. Key Vault, in contrast, returns HTTP 200 with an empty response. Arguably in this case, 404 is better, but I'm sure customers will expect a constsitent experience using KeyClient against Key Vault or Managed HSM.
This may be best handled by the service to have consistent behavior, so that any client - in the Azure SDK or not - has consistent behavior. If we have to make the behavior consistent in clients, we'll want to avoid throwing exceptions/errors in some languages where it's expensive to do so (e.g. .NET).

Adding @amitbapat

Ping @amitbapat

From Vick:
We have already fixed the back compatiblity issue for the 404 vs 200 empty key list.
If this is not the observed production behavior in SDK automation, it will be fixed mid way next week.

@vickm has this been resolved?

@vickm has this been resolved?

Resolved.

Resolved.


Second compass ring

What about the possibility to have a second compass ring for the 2nd sensor?

Good idea, however I´d need to find a way to make it work space wise.
As the card is not that configurable in sizing of characters clipping or wrapping might occur pretty quickly with another ring inside the first one.

Maybe program in some limitation if its possible, so when inner ring is used the card can only be sized down to a lower limit ex. 30%(?).  To avoid wrapping go automatically down in font size.  The picture above is from a horizontal stack, with two compass cards,  which making this a 1/4 of full size(?)

I'm not sure if I can detect the sizing, and for now the circle is fixes size. Maybe if I make it relative that would work.

The card does not have to be (become) smaller than 1/2 of a card row, and then maybe keep a fixe size depending if you use on ore two circles. The information for the two sensors can then be moved to the lower corners if that gives more space for data(?)

Good idea, however I´d need to find a way to make it work space wise. As the card is not that configurable in sizing of characters clipping or wrapping might occur pretty quickly with another ring inside the first one.

Hi, would it be possible to get a second entity : its useful for boat: True Wind Speed and True WInd Angle (TWS TWA) and Apparent Wind Speed and Apparent wind Angle (AWS AWA)
Below a GArmin solution, but would ne enough as start just to have second entity


ant-table-placeholder clips the bottom rounded corners of small tables

Version
1.8.0
Environment
any
Reproduction link
https://ng-zorro-antd-start-j4tdbh.stackblitz.io
Steps to reproduce

Create a small table with no data
Look at the bottom corners of the table

What is expected?
The rounded bottom corners of table are not clipped
What is actually happening?
The bottom corners of the ant-table-placeholder are not rounded and therefore clip parts of the rounded corners of the table, making it look broken.
Other?

fixed in https://github.com/ant-design/ant-design/commit/5a1119f37d183efa0eb1a09506f64a79aff5d9f5


build: Use compile with strict mode

Compile with strict mode and fix related errors.

Succeeded by #33


Miner should prioritise transactions with bigger fee

Miners should process transactions with bigger fees first.
Not the actual raw value though - but by percentage.
e.g. send = 1 SHARI - if someone pays 2 SHARI - this is double the send price
e.g create_token = 1000 SHARI - if someone pays 1500 - this is less than double the price
so the send transaction should be prioritised over the create_token one in this case.
(If we did it by raw value the higher fee transactions would always get picked first - e.g. create_token would always be picked over send - which is not what we want)

OK. That's cool.
Then we should detach the transaction pool process from node for the reason https://github.com/SushiChain/SushiChain/issues/140 and fix the cap # of transactions in a block.


test: add and delete .fpkg files

Related to #363

It appears to be impossible to invoke the remJar command since the process puts a lock on any jar-file which is currently included.

It appears to be impossible to invoke the remJar command since the process puts a lock on any jar-file which is currently included.

OK, lets leave it out for now.

We should document - somehow - what is in the binary files. For example, you could create a Foo.jar.txt file with a short description.

Should I merge with the tests failing?

Should I merge with the tests failing?

Yes, fix coming in just a moment


Remove unused TAURI_CHANNEL import

Depending on your tsconfig, unused imports may be compilation errors.

This import is actively used when your exporting Tauri Channels.
You can probably use header to configure your linter to ignore the bindings file as we can't possibly account for all edge cases in people's linting setups.

Thanks for taking a look!
Unfortunately, there's not a way to use header like you describe. Since this isn't a linter error, but a tsconfig option, it can't be turned off for a whole file: see https://stackoverflow.com/q/51385041 . The options are to add a line-comment, like // @ts-ignore, or to turn it off for the whole project. I'd like to avoid turning off noUnusedLocals for my whole project, since I find it to be a valuable rule.
Another option would be to make the TS template smart enough to know which imports were necessary, and only include those ones.
Please let me know if either of these options would be acceptable to you. I'd be happy to write up another MR.

What is the problem with using // @ts-ignore within the header? Really you should be able to trust us to get the types correct within that file.
We could make the template smart enough to know the used imports but it would also add a heap of complexity which would be nice to avoid if we can.

What is the problem with using // @ts-ignore within the header?
@ts-ignore applies only to the next line of code; not the whole file.

It does look like I can do // @ts-nocheck and turn off type errors for the whole file though, so that's a workable solution for me. Thanks for talking it through with me :)


Proposal: flag.failf should return an error with Cause() error

It would be very nice if flag.failf returned an error with a Cause() so that if you define your own flag.Value-implementing type, you don't lose context on errors you return out of its Set function here.
I would definitely be willing to open a patch if whoever's in charge agrees on the design. :)

I suspect we'd want some overall design & plan for error provenance in the standard library & general recommendations for the community rather than just making changes ad hoc.
Related: #16968 and #15192, both about errors in a general sense.

@bradfitz Thanks for the quick response! (@everyone) Are there any reasons against exposing Cause() error? It would be backwards-compatible-- just an assertable interface.
From skimming the other discussions, I get the sense that the Go team wants to hold off on refactoring any error provenance in the standard library until it's fixed everywhere, but for a standard library of this size, I don't think that's practical. Instead, I suggest analyzing what makes sense on a case-by-case basis (e.g. this issue), incrementally introducing non-breaking improvements, and shaping a "standard" around what proves worthy in Go2.

Are there any reasons against exposing Cause() error?

To clarify, I don't mean we should add something just because it has no downsides-- that's a recipe for creating a mess of a standard library... but I want to focus on what's wrong with this proposal (e.g. doesn't provide enough value to go in stdlib, encourages bad design, not readable, etc.) rather than the error handling everywhere in the standard library.

Are there any reasons against exposing Cause() error?

What do you mean exposing? If this method already exists on the type then there is nothing to do. If the method does not exist, then that's a bigger ask and out of scope for Go 1.8.

The challenge is that this proposal is trying to add a method to a value of type error and the only way for anyone to know about its existence is if they type assert for it. There's a growing precedence in the Go community for Cause() error to be a special method on error, but this proposal is probably not the place to have that discussion.

I was suggesting that the flag package returns a struct that implements non-standard Cause() error method in addition to the normal error interface.

The challenge is that this proposal is trying to add a method to a value of type error and the only way for anyone to know about its existence is if they type assert for it.

It can go in docs as well?

There's a growing precedence in the Go community for Cause() error to be a special method on error, but this proposal is probably not the place to have that discussion.

Agreed that this is not the place to talk about Cause() error as a special method, but I think it could fit well on errors returned by the flag package when it wraps errors from a flag.Value. If the Go team is considering making Cause() error special, we should start by looking at where it can be used (e.g. this one).
Curious- If this is not the right place, what is the right place? I understand that this may be out of scope for Go 1.8, but I'd still like to propose it... maybe for 1.9?

Unless and until this becomes an accepted soft standard for errors, we should probably not start adding it to the standard library throughout, and we also shouldn't add it piecemeal. Like @bradfitz said above, the situation is far from settled.
Declining until that changes. (And if it does change, and we decide that this is something we want everywhere, we'll probably just do it, without needing to reopen this proposal.)

@rsc I understand your concern about adding this piecemeal. My proposal is that the Go standard library should return a struct that satisfies Error() string and Cause() error in all cases where an error that the user (e.g. me writing Go code) is re-thrown. One example of this is flag.FlagSet.Parse([]string) error re-throwing the error returned by Set(string) error from a custom, user-provided flag.Value.
What does the Go team think of this conceptually? I understand that, in the end, this is out of my control, but what's missing / how can I help convince the Go team that this is important?

Without this feature, stdlib expects users to either

inspect the result of Error() string (eww!); or
store the original error from Set() (and similar methods) in some other place... which eliminates the purpose of the error return value (could as well be a pass/fail bool)

I understand the general problem. We should be sure of how we want to solve that before we start editing package flag. Maybe 'Cause() error' is not the solution. Until we know the solution, we shouldn't add it to package flag.

@rsc Got it.

Maybe 'Cause() error' is not the solution. Until we know the solution, we shouldn't add it to package flag.

Where does discussion about that take place? What are next steps?

To expand, I think it's important to have a place for open discussion about error handling in the standard library since the Go team recognizes the general problem but isn't sure what the right solution is yet. As I mentioned here, one exists for generics, which is in a similar boat, except almost certainly requires large language changes and breaking changes in general. I was hoping this issue would serve that purpose, but I'd be happy to open another one (or a mailing list thread, etc.)

Maybe 'Cause() error' is not the solution. Until we know the solution, we shouldn't add it to package flag.

^ The aforementioned issue would be a good place for pros/cons of Cause() error and other solutions, as it's still unclear to me what's wrong with the proposed solution :(

I don't think we're ready to have a conversation about errors right now. It's on my list for next year, hopefully before the summer.

Alrighty


Add covariate and delineation info to the manuscript

Add covariate and delineation info to the manuscript

I added in my section on the covariate data. I also included a bit on the delineation at the end of the previous section though I wasn't sure if that's where you wanted it. Take a look when you get a chance and let me know if you want me to make any changes, add more or less detail, etc.


Prototype Pollution in JSON5 via Parse Method

@ShashwatAgrawal20 do have idea what this means or how to fix it?

https://github.com/advisories/GHSA-9c47-m6qq-7p4h is the link to a general advisory for this issue.
If you npm uninstall firebase and npm uninstall firebase-tools, this app will be left with 10 high severity vulnerabilities. This one of them. You can run npm audit to see the vulnerabilities. I always brake the code when I try to fix these things.
Please make sure you install the right dependencies after you investigate this issue. Run npm install<EMAIL_ADDRESS>to install a specific version.
These are the apps current dependencies:
"dependencies": {
    "@emotion/react": "^11.8.1",
    "@emotion/styled": "^11.8.1",
    "@material-ui/core": "^4.12.3",
    "@material-ui/icons": "^4.11.2",
    "@mui/icons-material": "^5.4.2",
    "@mui/material": "^5.11.1",
    "@reduxjs/toolkit": "^1.7.2",
    "@testing-library/jest-dom": "^5.16.2",
    "@testing-library/react": "^12.1.3",
    "@testing-library/user-event": "^14.4.3",
    "bootstrap": "^5.2.3",
    "node-fetch": "^3.2.10",
    "nth-check": "^2.1.1",
    "react": "^17.0.2",
    "react-bootstrap": "^2.7.0",
    "react-dom": "^17.0.2",
    "react-icons": "^4.7.1",
    "react-player": "^2.9.0",
    "react-redux": "^7.2.6",
    "react-router": "^6.6.1",
    "react-router-dom": "^5.2.0",
    "react-scripts": "5.0.0",
    "react-timeago": "^6.2.1",
    "redux": "^4.1.2",
    "redux-thunk": "^2.4.1",
    "rellax": "^1.12.1",
    "styled-components": "^5.3.3",
    "web-vitals": "^2.1.4"
},

@ShashwatAgrawal20 do have idea what this means or how to fix it?

This vulnerability is patched in json5 version 2.2.2 and later.
https://github.com/json5/json5/security/advisories/GHSA-9c47-m6qq-7p4h
I think updating the package is required

Ship, I'm just starting to be mindful that translators do not work on photos. The github bot could not fix it because of other dependencies. When I try to fix it, it takes me 3 hours to get the app to run again. Are able to look at the bot alerts under this repos security tab.
And Hey, Happy New Year!! and Congratulations on another semester down!!!


Removal of PersistentTensor breaks custom ops (Horovod)

The recent commit https://github.com/tensorflow/tensorflow/commit/f8153ae87a88586ac1c364116208cfb144c9b64a#diff-b35354f62819de66aaa049a9498cccc261c108a7c488d39e04882110bdee65b5 removed the PersistentTensor from the TensorFlow C++ API. For Horovod, we rely on this functionality to allocate the "fusion buffer" that packs multiple smaller tensors into a single buffer during allreduce here which ultimately calls into here.
The docs recommend using Tensor with allocate_temp instead, but this does not seem like a viable workaround. My understanding is these tensors will not survive past the life of a single Op. Other frameworks including PyTorch and MXNet provide similar mechanisms for allocating long-lived tensor buffers, it seems reasonable TensorFlow should continue to do the same. Or is there a workaround I am missing?
Thanks.
cc @reedwm @romerojosh @DEKHTIARJonathan

@reedwm @sanjoy @pkanwar23 could you help addressing the issue or advising how we can work around the issue ?µ
@nluehr for viz

Hello Travis,
Tensors allocated with allocate_temp will survive past the life of a single Op. This call really needs to be renamed just "allocate"  or allocate_tensor for consistency.
Persistent tensor implementation that was just removed, and you are using, was a wrapper around Tensor class and allocate_temp.
It is completely safe to do the flowing update to your code:
FROM:
PersistentTensor ptensor_;
Tensor* second_tensor = nullptr;
OP_REQUIRES_OK(context, context->allocate_persistent(
value, shape, &ptensor_, &second_tensor));
const Tensor& tensor = *ptensor_.AccessTensor(context);
TO
Tensor tensor;
OP_REQUIRES_OK(
context, context->allocate_temp(value, shape, &tensor));

Thanks @gromovkg, that seems to have worked! I think what through me off about allocate_temp was this section of the docs that reads: "Tensor must not be used after kernel construction is complete." Can you clarify at which point the tensor can no longer be safely used?

@tgaddair You are welcome!
Tensor doesn't directly store the tensor contents. Tensor has a pointer to a TensorBuffer, which holds the tensot's contents and is refcounted. When a Tensor is copied, it doesn't copy the tensor contents but just copies the TensorBuffer pointer and increases the refcount. When the TensorBuffer's refcount reaches zero, it is destructed. In practice, usually only tensors reference TensorBuffers so when the last Tensor referencing a TensorBuffer is destructed, the TensorBuffer is also destructed.

I see, thanks for clarifying! I'll go ahead and close this issue then since it sounds like this is what we're looking for.


multiple errors with websocket

Hi,
I love this package, thank you so much.
I am running into two issues with the websocket connection.
This is my code:
async def main(loop):
    async def deal_msg(msg):
        if msg['topic'] == '/market/ticker:all':
            my_ticker_callback(Platform.KUCOIN, [msg])

    client = WsToken()
    ws_client = await KucoinWsClient.create(loop, client, deal_msg, private=False)
    await ws_client.subscribe('/market/ticker:all')
    while True:
        await asyncio.sleep(30)

def handler():
    while True:
        try:
            loop = asyncio.get_event_loop()
            loop.run_until_complete(main(loop))
        except Exception as e:
            logger.exception(f"Websocket failed")
            sleep(0.1)

When I use the code above, I run into this error every short while:
<Task finished name='Task-4' coro=<ConnectWebsocket._run() done, defined at ...\site-packages\kucoin\websocket\websocket.py:33> exception=ConnectionClosedError(None, None, None)> got an exception no close frame received or sent
_reconnect over.

When I change the sleep to 30, I get this every short while:
<Task finished name='Task-4' coro=<ConnectWebsocket._run() done, defined at ...\site-packages\kucoin\websocket\websocket.py:33> exception=ConnectionClosedError(None, None, None)> got an exception sent 1011 (unexpected error) keepalive ping timeout; no close frame received
_reconnect over.

Any idea how this can be solved?

You don't need the second while True loop. You can put that whole try-except block under  if name == 'main' and run it like that.

Because run_until_complete is a method that runs the event loop until a Future is done and returns the Future's result or raises its exception, you don't need to add a while True loop. Please refer to this and modify your code accordingly.
example_default_ws_public.py


[NXP][k32w1] Remove the need for extra DAC private key conversion binary

The application will check at initialization whether the DAC private key has been
converted or not and convert it if needed. There is no need for an extra binary.

fast track platform specific changes

Thank you


Update TypeScript version

Update TypeScript to v4.5. Also enforce VS Code to use the workspace version of TypeScript, so we don't get discrepancies between workspace and tests.

Related to #277
Object index signatures accessed via array notation must be checked for undefined as of TS 4.4
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-4-4.html#symbol-and-template-string-pattern-index-signatures


[feature] Add support for sftp protocol

Would be nice to support enable-libssh. Note that it requires libssh package.

Oh yeah that's a good idea, you're interested ?
https://github.com/jrottenberg/ffmpeg#contribute


Sorting provokes scroll bug on data change

Describe the bug (required)
Without sorting, scroll doesn't change when I update my data. With sorting, it does.
Provide an example via Codesandbox! (required)
https://codesandbox.io/s/gifted-frog-b53oli?file=/src/App.js:905-1008
Steps To Reproduce (required)
Steps to reproduce the behavior:

Click on click me button in order to load data
Scroll down
Click on click me button again
As data changes, it scrolls top, this is the problem
Remove sortBy / initalState by commenting lines 60 to 66
Repeat steps 2 to 3
It doesn't scroll top anymore

I tried memoizing the initial sortBy and passing as prop with no luck
Expected behavior (Recommended)
Should not scroll when data changes
Desktop (please complete the following information):

OS: Windows 10
Browser : Chrome + Firefox
Chrome 99 + FF 98

I'm not sure why this is happening TBH. But... the issue is completely gone in v8 :) https://codesandbox.io/s/magical-banach-nfumi6?file=/src/App.js

The table is not sorted in your example, so I'm not sure the issue is gone 😮
Thanks for the quick reply anyway 👍

I’ll add sorting to my example, but even so I’ll get to the bottom of the v7 issue.

Did you have a chance to look at the issue yet ? Thanks in advance.

I'm not sure why this is happening in v7. Feel free to keep investigating, but I'm going to close this as I don't have bandwidth to dive into deep v7 bugs right now.


chore: v1.4.2

Bumps version to v1.4.2. Tried:
npm version 1.4.1
git push --follow-tags

But I don't have access to push directly to main, so it failed to push the version bump. However this already released a 1.4.1 version since the tag was pushed. This time I'm opening this PR to bump the version and only then create a tag based on latest main, with the correct version.

Gonna go the other way and fix 1.4.1 instead!


Pivoting data to create new columns

In the section that begins "In this example I pivoted my data to create new columns..." - After pivoting the data, I had to delete and then recreate the relationship between "Customer Table" and "Device" and then I had to close the pbix file and re-open it in order to fix the "A field in "Explain By" is not uniquely related..." error.  It's not immediately obvious that users have to go through these steps.

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

ID: 4200dd45-15ab-ee4a-9818-10cc81475f64
Version Independent ID: 0b96ac28-e2a2-41c9-eff0-efd864fc0529
Content: Key influencers visualizations tutorial - Power BI
Content Source: powerbi-docs/visuals/power-bi-visualization-influencers.md
Service: powerbi
GitHub Login: @mihart
Microsoft Alias: mihart

Thank you for the feedback! I will add the point about fixing the relationship as I agree it is not intuitive. The other point you mentioned is a caching bug and should be fixed in the next release of the Desktop.

Thanks for the comment and update.

#please-close @mgblythe


Packaging /app to asar file (when npm run package?)

https://github.com/electron/electron/blob/master/docs/tutorial/application-packaging.md
That have 17000+ files in /dist/tofino-win32-x64/resources/app, it is a horrible effects for end users, although we may have no plans.
In command line way: asar p app app.asar
Then, renamed or removed the app dir.

This depends on #564

Fixed by #838


Do not cast the result of malloc

https://stackoverflow.com/a/605858/3005749, also expanded the definition of heap

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

This looks good to me.  Thank you for your contribution!


Chart series not overriding original one

If I do chart addSeries arearange that answers me a good instance of HighstockSeriesArearange. However if you inspect the result of chart series you will see the instance in there is a "a HighstockSeries ({""type"":""arearange""})" instead of a HighstockSeriesArearange.
The problem is that the methods like column, line , etc are NOT replacing the original HighstockSeries instance created by addSeries.
My workaround is this:
	series := HighchartsSeries new column.
	chart series add: series.

The protocol has changed since v12, so this should no longer happen as described. I'll close this and refer anyone interested to check the examples in HighchartsDemo


Is there any progress for supporting Cordova Android?

Hi,
I want to ask you is there any progress for supporting cordova android?
Thanks

Actually, one of your helpful fellow Cordova developers has recently registered an updated version of the plugin that now supports both iOS and Android: http://plugins.cordova.io/#/package/com.keepe.cardio


Nested JSON Layouts configured in NLog.config

Hi,
I'm trying to output my exception detail in a JSON structure with the config being done in the NLog.config.
At the moment the, in the example below, the exception is being rendered as a string.
I had a look at https://github.com/NLog/NLog/wiki/JsonLayout#advanced-examples which shows that it can be done, and an example of doing it using the API, and it does mention 'This is also possible with the XML config.' but there is no example and I cannot get it working.
If you could provide me with an example of Nested JSON Layouts configured in the NLog.config that would be great.
Many Thanks,
Lach
NLog version: 4.2.3
Platform: .Net 4.6.2
Current NLog config
    <target name="file" xsi:type="AsyncWrapper" queueLimit="5000" overflowAction="Discard">
      <target xsi:type="File" fileName="${logFilePath}web.txt"
              maxArchiveFiles="100"
              archiveNumbering="Sequence"
              archiveAboveSize="${archiveFileAboveBytes}"
              archiveFileName="${logFilePath}archive/web-{#####}.txt">
        <layout xsi:type="JsonLayout">
          <attribute name="datetime" layout="${iso8601dateformat}" />
          <attribute name="level" layout="${level:uppercase=true}" />
          <attribute name="logger" layout="${logger}" />
          <attribute name="thread" layout="${threadid}" />
          <attribute name="machine" layout="${machinename}" />
          <attribute name="message" layout="${message}" />

          <attribute name="exception" layout="${onexception:${exception:maxInnerExceptionLevel=5:innerFormat=tostring:format=tostring}}" />
        </layout>
      </target>
    </target>

Hi,
I have to agree that I also don't know how to configure this exactly from the XML config. We have to test this first.

Hi, I have edit the wiki: https://github.com/NLog/NLog/wiki/JsonLayout
It is working as can be seen in https://github.com/NLog/NLog/pull/1309

Thanks!


Fix smtp and unique email

Fix SMTP Bug
Student email must be unique

@recluze Check email for new SMTP settings.


Adding times manualy

As a user I want to add to the existing task a time record manualy.
Adding a time record should be enough but in the perfect scenario adding time record and date would be great.
The record should appear on the records list with indicator that it has been added maualny.

now it is possibel due to #33 but in history we should see all own work items from YT


How to deep filter

How do you deep filter the returned select?
Prisma Models
model User {
  id        String     @id @default(cuid())
  email     String
  profile   Profile?
}

model Profile {
  /// The id matching the user id
  id         String        @id
  user       User          @relation(fields: [id], references: [id])
  firstName  String
  lastName   String
}

GraphQL Query
query Users {
  users {
    email
    profile {
      firstName
      lastName
      name
    }
  }
}

Resolvers
t.list.nonNull.field("users", {
  type: "User",
  resolve: async (_, args, ctx, info) => {
    const { select } = new PrismaSelect(info).value;

    return ctx.prisma.user.findMany({
      select,
    });
  }
});

// Select Returns
{
  select: {
    email: true,
    profile: {
      select: {
        firstName: true,
        lastName: true,
        name: true   // This return an error since it's not present in the model.
      }
    }
  }
}

Is there a way to force PrismaSelect to filter out the whole select (deep nested select as well) of every fields that is not present in the Prisma's DMMF?

It's already filter any field not in the model
Did you generate the prisma client in a custom path or it's in @prisma/client

It's in @prisma/client.
Could it be possible that these lines are the issue?
https://github.com/paljs/prisma-tools/blob/1abadc6f2ddfdcf1e14761993d986b5f57f95353/packages/plugins/src/select.ts#L78-L80
Shouldn't dataModel actually be datamodel?


0.5 Release

[x] Cut release on Github and tag
[x] Pip package

https://github.com/kubeflow/fairing/releases/tag/v0.5.1

https://pypi.org/project/fairing/


SQLLite option does persist

It either does not persist, or does not show up toggled when reloading a project.

I think the problem is that no boolean values persist to run configurations. When Share was enabled the output was
<component name="ProjectRunConfigurationManager">
  <configuration default="false" name="Testing" type="DrupalRunTestsConfigurationType" factoryName="Drupal Run Tests" concurrency="7" test_group="1" test_group_extra="commerce" test_types="PHPUnit-Unit">
    <method />
  </configuration>
</component>

No boolean properties would export.

Actually, not sure.
    public static class Settings implements PhpRunConfigurationSettings {
        private String myHost = "localhost";
        private int myPort = 80;
        private String myDocumentRoot;
        private boolean myUseRouterScript = false;

The built in runner uses boolean.


Update broken links on examples doc

PR Type

Docs Pull Request

Fixes broken links on https://molecule.readthedocs.io/en/stable/examples.html

Fix for failing CI is @ https://github.com/ansible/molecule/issues/2349#issuecomment-538970935. It needs some eyes to decide on a correct solution. Take a look if you have time.


Example to param does not show up

I have been trying to add example to param but is does not show up in the swagger file.
params do
        requires :order_id, type: String,
                 documentation: {
                   type: 'String',
                   desc: 'Your unique order id of this order. (max 255 characters)',
                   example: '1234',
                   param_type: 'body'
                 }
end

But the swagger file only shows this:
...
"definitions":{
   "postOrders":{
      "type":"object",
      "properties":{
         "order_id":{
            "type":"string",
            "description":"Your unique order id of this order. (max 255 characters)"
         }
      }
   }
}
...

Any clues, workarounds or fixes?

Hi, we are running into the same issue. The GrapeSwagger::DocMethods::ParseParams class simply does not copy over the example from the definition.
As workaround you can load the following code:
module MyApp
  module AddExampleToBodyParam
    module ClassMethods
      private

      def document_description(settings)
        super
        @parsed_param[:example] = settings[:example] if settings[:example]
      end
    end

    def self.prepended(base)
      class << base
        prepend ClassMethods
      end
    end
  end
end

module GrapeSwagger
  module DocMethods
    class ParseParams
      prepend MyApp::AddExampleToGrape
    end
  end
end

I am also getting the same issue @lvonk can you create a PR to fix this issue ?


Fix when scheduler on machine 1 is no longer master but continues to schedule, even after scheduler on machine 2 becomes master

Current coverage is 36.49% (diff: 0.00%)

Merging #233 into develop will decrease coverage by <.01%

@@            develop       #233   diff @@
==========================================
  Files           260        260
  Lines         14177      14180     +3
  Methods           0          0
  Messages          0          0
  Branches       2380       2381     +1
==========================================
  Hits           5175       5175
- Misses         8281       8284     +3
  Partials        721        721

Powered by Codecov. Last update d83339e...60ce7a6


Matterbridge uses Mattermost usernames instead of nicknames

Is your feature request related to a problem? Please describe.
Matterbridge uses Mattermost usernames instead of nicknames which is, depending on the user or nickname policy, sometimes confusing for people in the other chats.
Describe the solution you'd like
It would be cool if matterbridge uses the mattermost nicknames which can be changed easily through the users if necessary. Also the username could be the login name from different oauth providers.
Describe alternatives you've considered
Additional context
I tried implementing it myself, but I'm not familiar with 1.) Go and 2.) Mattermost internals, so I could not successfully change the username to nickname.
func (m *MMClient) GetUserName(userId string) string {
	user := m.GetUser(userId)
	if user != nil {
		return user.Nickname // This should be correct, based on https://github.com/mattermost/mattermost-server/blob/master/model/user.go
	}
	return ""
}

I can provide a PR if someone can point me into the right direction and the change is welcome.

Thanks for reporting and looking into this!
It needed some more changes, so that when a nick or username is changed, it's also updated.

Thanks for implementing it. Now I definitely have to take a look into that commit. Tried so many things, but it didn't worked for me. :D


Animate values inside a calc() expression

Is it possible to animate values inside calc expressions? For example, something like this:
width: calc(300px + ${progress} * (100% - 300px));

You should use interpolate.  If you can't get something in particular to work, let us know.
On Apr 26, 2016, at 2:30 AM, Maël Nison <notifications<EMAIL_ADDRESS>wrote:
Is it possible to animate values inside calc expressions? For example, something like this:
width: calc(300px + ${progress} * (100% - 300px));
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHubhttps://github.com/animatedjs/animated/issues/29

I've successfully used interpolate to remap simple values from the [0;1] range to [0.9;1.0], but I'm not sure how I can use it to remap arbitrary values in the middle of a string (as in the example above). The reason I could use a calc construct is that interpolate works fine if the property value only deals with a single unit, but not that much if there's more that two units involved (as in the example above, where both px and % are used in the same expression).

That might be a bad idea, but have you considered using tagged literals for this purpose? For example, using interpolate, it could be as simple as this:
render() {
    let style = { width: Animated.tpl `calc(300px + ${progress} * (100% - 300px))` };
    return <Animated.div style={style}>Hello</Animated.div>
}

It would also fix the #31 issue in the process, since crafting the job of crafting the transform would be left to the user instead of the library (which is probably more performant anyway):
render() {
    let style = { transform: Animated.tpl `scaleX(${this.state.progress}) rotateX(${this.state.progress})` };
    return <Animated.div style={style}>Hello</Animated.div>
}

As final perk, it would also opens new CSS property to be animated (such as filters).

Oh I see.  That does seem pretty interesting. What do you think, @vjeux?

The API may need to be a bit more restrictive than that, but I like the concept.

This should already be supported today. You can write the following:
width: this.progress.interpolate({
  inputRange: [0, 1],
  outputRange: [
    'calc(300px + 0 * (100% - 300px))',
    'calc(300px + 1 * (100% - 300px))',
  ]
})

But, I really like your proposal where you can specify where each component is. Right now the solution above relies on extracting things that look like numbers from a string which is not ideal

I'd be down to accept if you can implement it. I don't think it has much value inside of React Native since we are doing everything possible to avoid strings as value but for the web I can see the use case.
Animated.template`calc(300px + ${progress} * (100% - 300px))`

You can look at AnimatedTransform as a base to implement it: https://github.com/animatedjs/animated/blob/master/src/AnimatedTransform.js

@vjeux Sure, I've just opened a PR with a tentative implementation

Just wanted to say thank you for this 🙏 I was able to get some pretty cool animations working quickly with the template feature 🎉

Sadly this is only supported on web :( need to port this back to native.


Thoughts on land modeling philosophy

Land models, in the context of climate, are mostly needed to inform land-atmosphere exchanges (source or sink) of water (evaporation and transpiration), energy (latent and sensible heat flux, shortwave and longwave radiation) and greenhouse gases (e.g., CO2, CH4, N2O).
Water and energy fluxes are important at short time scale (hour, days) whereas greenhouse gas exchange are important at larger time scale (sink or source over decades: e.g., how much carbon will be in the atmospheric pool vs. land pools such as plant biomass or soil carbon).
Transpiration is coupled to photosynthesis, as both are regulated by the conductance of gas through leaves opening (stomata), the more stomata are open, the more transpiration and photosynthesis can occur from air exchange between the insides of leaves and the atmosphere.
Modeling photosynthesis and transpiration can be excellent when we prescribe structural information such as leaf area index (the area of leaves per area of ground), as we have good theory and data to inform parameterisation.
While change in pools (i.e., carbon pool in plant biomass) over long time scale is equal to the integration of short time scale fluxes, it remains very challenging to model the evolution of vegetation structure (biomass, leaf area index, roots, etc.), as there is less theory and less data, and great complexity in processes happening at microscales.
Current model attempt to capture this complexity, leading to heavily parameterized models with no observation to inform those parameters, especially their spatio-temporal variability. This leads to large disagreement between models - large uncertainties.
Similarly, change in soil carbon is very complex and difficult to parameterize, as there is almost no observation of change in soil carbon. Nevertheless, models are becoming more and more complex, with no parameterization framework (because no observation to constrain the parameters), and no spatiotemporal generalization.
Despite decades of research, land models perform poorly when it comes to change in carbon pools (plant biomass, soil carbon), which is unavoidable - we have no observations to constrain models anyway. More complexity will always remain a dead end as long as there is no data. While above-ground observation is increasing from remote sensing (e.g., SIF, NDVI, LiDAR), the period of data is often too short relative to climate models, and below-ground remain unobservable (change in roots depth and density, change in soil carbon).
I think ClimaLand should aim for intelligent but as simple as possible carbon pool models, for now, until we have more data. Such models could be informed by optimality framework, simple processes (mass conservation - growth, litter fall, transport ...), and empirical relationship informed by data and parameterisation (for example, growth in transient and equilibrium at steady state of known vegetation structure in given environmental condition - mean temperature, mean precipitation, their distribution, soil properties, etc.). We should avoid over-complicated models, even if eventually we can add them to inform synthesis, using our modular framework.
Note: I plan to edit this text, and I am happy to discuss more.

Additional thoughts:
Let's think of human (easier to relate) mass balance, simple logics.
Let's call mass M (kg)
The change in mass of any human is:
dM/dt ~ (Ein - Eout)
where E is calories.
Ein is the calories in food ingested during a period dt.
Eout is the calories used, by maintenance metabolism + activity metabolism.
Maintenance metabolism is a function of mass, weighted by muscle mass and fat mass. (higher for muscle)
For an full grown adult, change in mass is either muscular or fat. For a growing person, change in mass can also be structural (bones).
When scaling to the total mass of a population, we sum all individuals.
Population mass may change from same individuals changing mass, and/or from population change (less or more individuals)
We could make a simple parameterization that population change is a function of access to food and water, and some mortality rate.
In land models, we are doing this for two main carbon pools: vegetation carbon mass and soil carbon mass.
Similarly, for vegetation, we do
dM/dt ~ (Ein - Eout)
where Ein is photosynthesis and Eout is autotrophic respiration.
In dynamic carbon models, eventually, Ein = Eout and we are at equilibrium. We do this during spinoff.
Then, in the transient modeling where atmospheric CO2 increase and T increase, and moisture change, Ein is no longer equal to Eout, and equilibrium mass will be re-adjusted.
How do we account for growing vs mature trees?
How do we differentiate "trees getting bigger" vs. "more individuals"?
How do we model mortality (e.g., stochastic fire, drought or carbon starvation mortality, insect disturbance mortality, ...)
What data can inform the model?
likewise for soil carbon, in and out: how do we approach this?

On another note - trees can invest the carbon gained from net photosynthesis (NPP) to grow structures (stem, leaves, roots), or store it in their sap as sugar for future use (e.g., to face a drought). It is currently not well known how plants manage their carbon resources, but it clearly matters a lot in time of adaptation like climate change.


Add "description" field to ECSAttributeMap

This will be an optional field to be used to explain to the player, in short plain English, what the effects of the card are. This will be left empty if the card has no special effects. It can be identical to ECSAttribute.FLAVOR as far as I'm concerned, unless there is good reason to do it differently, of which I cannot think of.

I'm not sure if this is required at the moment, as I have managed to dynamically generate the description?

I agree.


beforeSend is now using the current tags when sending crash dumps from previous sessions

[x] Review the documentation: https://docs.sentry.io/platforms/javascript/electron/
[x] Search for existing issues: https://github.com/getsentry/sentry-electron/issues
[x] Use the latest release: https://github.com/getsentry/sentry-electron/releases
[x] Provide a link to the affected event from your Sentry account

Versions + Platform

[x] SDK version -<EMAIL_ADDRESS>[x] Electron version -<EMAIL_ADDRESS>[x] Platform - macOS

Description
When a crash occurs, it may not be sent to Sentry until the app restarts. In that situation, it used to send the previous session's tags/context to be sent with the event.
How it used to work in 4.0.2 (expected result):

Set a session-specific tag, e.g.:

setTag('session', myAppSession);

App crashes
Restart app, with a new myAppSession value
Inspect event in beforeSend:

beforeSend: (event) => {
  if (myAppSession !== event.tags.myAppSession) {
    console.log('this event was on a different session');
  }
}

How it works in 4.1.0 (actual result):
event.tags.myAppSession now is the same as the current session's. I assume the breadcrumbs may be the same as well?

The only this that could have caused this regression is the change to using a fully async store. I'll do some testing and see if I can pin down the cause.


Benchmark tests for view generation and config generation

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #2067
Engineer Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the FHIRCore app to verify my change fixes the issue and/or does not break the app
[ ] I have checked that this PR does NOT introduce breaking changes that require an update to Content and/or Configs? If it does add a sample here or a link to exactly what changes need to be made to the content.

Code Reviewer Checklist

[ ] I have verified Unit tests have been written for any new feature(s) and edge cases
[ ] I have verified any strings visible on UI components are in the strings.xml file
[ ] I have verifed the CHANGELOG.md file has any notable changes to the codebase
[ ] I have verified the solution has been implemented in a configurable and generic way for reuseable components
[ ] I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

To run the macrobench instrumentation tests run, open an emulator/device,API 28 (Android 9) and below, and run the following command in terminal in the directory fhircore/android
./gradlew :quest:assembleOpensrpDebug --stacktrace && \
adb install quest/build/outputs/apk/opensrp/debug/quest-opensrp-debug.apk && \
./gradlew :macrobenchmark:connectedBenchmarkAndroidTest && \
cp macrobenchmark/build/outputs/connected_android_test_additional_output/benchmark/connected/*/org.smartregister.opensrp.quest.macrobenchmark-benchmarkData.json macrobenchmark/benchmark-results.json

It is recommended to run these tests on an Android 28 and below devices. To run with higher APIs, an update on the benchmarking library will be required among other updates


Sharing search url with page number is broken

Plus I've found another issue, when I share someone's URL https://beta.kodadot.xyz/rmrk/gallery/?search=nft&page=10 it actually pushes me on page 1, rewriting the 10th page to first. If you can fix it with this that would be superior! :)
Originally posted by @yangwao in https://github.com/kodadot/nft-gallery/issues/2513#issuecomment-1072998406

Hey, I can fix it!


Make error message case consistent

I noticed that almost all error messages began with a capital letter—and I realise this is a very trivial change (so, feel free to close)—so I thought I'd spend the minute to make the changes. 😅

https://github.com/golang/go/wiki/Errors says:

Error strings should not start with a capital letter because they'll often be prefixed before printing:

So this PR should not be merged.

Good point! So the consistency change should be the other way around. I'll do an update.

Done. :) Let me know if I've missed anything—I grep'd for errorf, errors.new, and just straight error and went through the results. I left all the testing Errorf calls.

LGTM

Landed in release branch for 4.0.0.
Thank you


Bugfix/texture fallback

This PR addresses an abort that can happen when running demos using the Bistro data.

Full file paths for some very long texture names could exceed OS maximum of 255 bytes, e.g. a full system path to ... /3D-Graphics-Rendering-Cookbook/data/out_textures/deps__src__bistro__exterior______proptextures__street__paris_bistroexteriorspotlight_01__paris_bistroexteriorspotlight_01_diff.png__rescaled.png can exceed 255 bytes in UTF-8.

Those files would not be written during the Bistro data conversion step (a Chapter 7 demo), and subsequently not able to be loaded in Chapter 10 demos.

This PR does not prevent very long filenames being created.

In the event that any texture asset is missing, a checkerboard image is created in its place and used instead.

This may be helpful to diagnose other texture file and path issues that arise with reader set-ups.

A warning message is printed to stderr to help users diagnose the problem.

The previous assert( img ) was removed, but since this also had the virtue of checking for memory allocation issues, a separate memory allocation check is added here, with an error message and a similar abort().

Example of output, where missing textures produce a warning, and are recovered from:

anton:~/projects/3D-Graphics-Rendering-Cookbook[master]$ ./Ch10_SampleGL01_CullingCPU
WARNING: could not load image `data/out_textures/deps__src__bistro__exterior______proptextures__street__paris_bistroexteriorspotlight_01__paris_bistroexteriorspotlight_01_diff.png__rescaled.png`, using a fallback.
WARNING: could not load image `data/out_textures/deps__src__bistro__exterior______proptextures__street__paris_bistroexteriorspotlight_01__paris_bistroexteriorspotlightglass_01_diff.png__rescaled.png`, using a fallback.

Testing

Tested all of the Chapter 10 demos on Ubuntu, and a random selection of the other demos to make sure there was no regression introduced. I was able to move around the scene and interact/view correctly the demos that were not loading before.
I have not testing this on Windows.

@capnramses how about Vulkan?


Eureka: client does not unregister on application shutdown (stays with status DOWN)

This issue is an attempt at reopening https://github.com/spring-cloud/spring-cloud-netflix/issues/1543
The problem is still there in Dalston.SR1.
Although the code has been refactored since this issue is created, the same root causes are still present: the replication of the DOWN status hits the server after the unregistration.
There is not much that can be done to have 100% guarantee: the Eureka client itself does not take guarantee there is no in-flight requests when unregister is invoked.
To (temporarily) solve that issue, we used to register a custom Jersey ClientFilter in the Eureka client to intercept outgoing requests. When it is time to shutdown the client, our filter blocks all requests except the unregister. If needed I can share the workaround.

Closing this so we can continue to use #1543

So I can keep posting on the old issue even if closed ?
Did not know. Sorry about the noise caused by this one.

https://stackoverflow.com/questions/53955613/working-with-eureka-clients-programmatically-issue-completed-shut-down-of-disc- please guide here


Enable the ESLint no-useless-escape rule (PR 12551 follow-up)

Note that a number of these cases are covered by existing unit-tests, and a few others only matter for the development/build scripts.
Furthermore, I've also tried to the best of my ability to test each case manually to hopefully further reduce the likelihood of this patch introducing any bugs.
Please find additional details about the ESLint rule at https://eslint.org/docs/rules/no-useless-escape

/botio unittest

Nice improvement!


Account 20175 doesnt exist

Module: 00
Lab/Demo: 00
Task: 00
Step: 00
Description of issue
Module 3
exercise 3
step 1
there is a reference to Account 20175 that doesn't exist, or has not been created in previous modules
Repro steps:

Labs have been refreshed in January 2021

MOC Labs Review confirms this issue


Suggestion: Show line number in ambr, when stepping through the matches

Suggestion: Show line number in ambr, when stepping through the matches.

ambr supports --column option like below:
$ ambr --column version version2
./src/ambr.rs: 23:#[structopt(raw(long_version = "option_env!(\"LONG_VERSION\").unwrap_or(env!(\"CARGO_PKG_VERSION\"))"))]

Ah thanks, I didn't that, I'll try it.
But it would be useful to show the line numbers by default to at least give a little context.

I added config support.
If there is ~/.ambr.toml like below, showing line number becomes default.
column = true


addUserDB - Is it possible to achieve this from the client?

Hi,
My application will allow an administrator to create users and allow access to the same database. With this in mind, I need the functionality of addUserDB so the new user can be associated with the current database.
Is it possible to achieve this from the client?

I'm afraid not. This is controlled on the backend side only.
You'll have to create an API yourself to do that

No problem, as I learned more about this API I came to that conclusion as well. Thanks.


When you "tab" into select2, you have to click the input, before you can type anything

If you go from one input to a select2 input (using TAB) nothing happens when you type. You have to manually select the input with your mouse in order to type anything. Using version 4.0.0

This should be fixed on the latest master, and will be part of the upcoming 4.0.1 release.


Make it run on Windows

The package was designed to work with Linux, so I changed a few lines to make Camunda start on my Windows machine.

Detects OS and use spawn() to start a Windows shell running Camunda
Introduced a new test parameter to specify Camunda Version to test with (hardcoded 7.8 was to old)

Test is running fine on my Windows machine.

Thanks for your PR. I like the way you support multiple Camunda versions / extract the relevant Tomcat server directory.
Overall however, the implementation does not take a few key things into account:

we separate the Camunda RUN directory from the server directory for the reason that it's state can be easily cleaned up on stop
camunda start does not block; it does so however in your implementation
we must sufficiently test the stuff, otherwise I'm not able to maintain it anymore

I've implemented Windows support and arbitrary Camunda version support via https://github.com/nikku/run-camunda/pull/2, incorporating the points mentioned above.

Released as v1.3.0; thanks again for your contribution.


get_tree().quit() on IOS doesn't close the game

Godot version:

3.2.3-stable
OS/device including version:

IOS Iphone 6
Issue description:

get_tree().quit()  on IOS doesn't close the game
Steps to reproduce:
try to call get_tree().quit() on IOS
Minimal reproduction project:

See also #24049. This is likely a design decision on iOS.

Reopening, as this still needs to be better documented.


Working on update docs, and a few small rizzo enhancements

Customizes are docs output a bit.
Also adds better docs to Component.
Move component to a class.
Improve the ComponentRegistry.


[deps] add scorecard info in external dep table

Signed-off-by: Asra Ali<EMAIL_ADDRESS>
Commit Message: Adds scorecard evaluation into the external dependency documentation.
Additional Description:

OSSF Scorecards (https://github.com/ossf/scorecard) is a project that can be used to evaluate health metrics for dependencies.
We currently use it in evaluating new dependencies: https://github.com/envoyproxy/envoy/pull/14191
Now we can add scorecard info in our external dependency documentation during CI doc generation because OSSF Scorecard now publishes 100k repos' scorecard data, so we don't need to run Scorecards with an auth token to generate the results https://github.com/ossf/scorecard#public-data

Risk Level: Low
Testing: Generated docs
Docs Changes: This is it.
[Optional Fixes #Issue] Related to https://github.com/envoyproxy/envoy/issues/10471
I'm still working on formatting the output nicely inside the table, rendering newlines through sphinx in the csv table text has been a pain, but I think I will figure it out soon.

lgtm /deps

/lgtm deps

docs are rendered here
https://storage.googleapis.com/envoy-pr/17206/docs/index.html
eg, relevant page is here
https://storage.googleapis.com/envoy-pr/d2c8f79/docs/intro/arch_overview/security/external_deps.html

@asraa it would be good to have some explanation or a link to get some context to the scorecard info

Yeah, looking at https://storage.googleapis.com/envoy-pr/d2c8f79/docs/intro/arch_overview/security/external_deps.html, I think the table is not making things super clear yet. We should have links from each criteria or a tool tip to what it is about. Also, the formatting needs some work. Maybe some color coding, table-in-table, whatever creative ideas folks have to make this easier for a human to parse as they scan down.

(FYI this PR is now 14 days stale)

We should have links from each criteria or a tool tip to what it is about.

What if instead of printing out the results (which I still can't figure out how to format properly through sphinx), I linked to the deps.dev links? e.g. https://deps.dev/go/github.com%2Fgrpc%2Fgrpc which embed the scorecard a lot nicer than I could?

We should have links from each criteria or a tool tip to what it is about.

What if instead of printing out the results (which I still can't figure out how to format properly through sphinx), I linked to the deps.dev links? e.g. https://deps.dev/go/github.com%2Fgrpc%2Fgrpc which embed the scorecard a lot nicer than I could?

I think this could wort too.

/wait


[DISCUSSION] Some config values should be optional

I was able to use the plugin with the latest stable version of GLPI (it looks OK with v4 RC4, but I'd need to do more tests) and Google as an Identity Provider. Here are some feedback:

Some IdP doesn't support logout scenario (it is the case of Google SAML), in this case I think you should delegate the logout to GLPI (it would simply clear the session) if the logout URL is empty.
You should make optional the SP's key and certificate fields. Because many IdP doesn't require a signed response (although it is a good practice). In your future doc ;) you might suggest this page from OneLogin that helps to generate a valid x509 certificate and key: https://www.samltool.com/self_signed_certs.php

I made the Idp cert, key and slo fields optional but experienced some issues with the empty slo field.   It appears that logout.php redirects back to index.php where I'm hooking the authentication process.  This kicks off another SSO request to the Idp and since it never received a logout request it responds successfully.  The end result is that the user is redirected back to GLPI and logged in.

The latest update addresses the SLO issue.  Let me know if this resolves for you.

I have tested it and it works perfectly.
Just for your information, Auth0 offers a free account for testing with something else than Microsoft AD.


Add support for Arrow SoCKit

board is alive and blinks!

https://twitter.com/hansfbaier/status/1460822834263904256


Is there a way to download pre-trained networks to skip computationally intensive steps?

I am just following the steps on the README file and was wondering if there is any way to make the setup faster for a new user. Maybe we can delegate that to Travis and make it generate downloadable files for later use?

Once I've fixed the instability of predictions (different initial weights -> different predictions on specific peptides), then we could put HDF files of all the models on dropbox (or hammerlab.org)

@timodonnell @hammer Is Dropbox a good enough solution for now?

I'm imagining we would have versioned model files (e.g. /mhcflurry/0.1/ALLELE-NAME.{json, hdf})

Hmm, we don't have a lab Dropbox account that I know about. Maybe stick them on the hammerlab.org Google Drive and put a link to the files on our website?

@hammer I just meant my personal dropbox, which does seem sloppy/unsustainable. How do I access our Google drive? And can we download from it programmatically when installing mhcflurry?

Do we know what the file size limit is for throwing these files into the PyPi upload?

@iskandr just log in with your<EMAIL_ADDRESS>address to access Google Drive. How big are the files though?

Another idea for how to get the model parameters to end users: we could use Sage Synapse, like Joel Dudley did for his IP Map paper: https://www.synapse.org/#!Synapse:syn4877787/files. From what I can tell, there are no limits on the size of the data we can upload.

Ah looks like the best way to get model parameters to users is to attach the HDF5 files to a release: https://help.github.com/articles/distributing-large-binaries/

This is now implemented


Multisig OOBI is missing end role authorization messages after member joins

There is a bug when a group is created with a signing threshold that is less than 100%. For example, consider a group with 3 members and a signing threshold of 2.
Affected versions

main as of 2024-10-11
0.2.0-dev3
Probably before as well

Steps to reproduce

Member 1 and 2 creates group consisting of member 1, 2, and 3
Member 1 and 2 authorizes the end role for the agent of member 1
Generate OOBI and verify that the oobi contains the /end/role/add message
Member 3 creates the group
Generate OOBI and verify that the oobi contains the /end/role/add message

Expected result
The /end/role/add message can be found in the oobi response in both step 3 and 5
Actual result
The /end/role/add message is missing from the oobi in step 5.
Notes
A reproduction is available here: https://github.com/nordlei/vlei-sandbox/blob/main/src/issues/multisig-join-late-no-endroles.test.ts
This happens because all three group members are using the same KERIA instance and currently, when KERIA responds to an OOBI request, it reads this information from the last agent that joined the group. In this case, that agent does not yet have the end role information.

This is where the agent is loaded for the oobi request: https://github.com/WebOfTrust/keria/blob/56534bff7bb7a9efabb64e846f96cb1f084cf837/src/keria/end/ending.py#L87
It will return the agent that was the last one to call "incept" on the agency for the group. I was able to resolve this issue by requiring the eid parameter on the OOBI endpoint and use that to retrieve the agent instead. In other words, KERIA would require you to specify which agent to load the OOBI from.


Build with the latest Go 1.11 version

Go 1.11.0 is very old and lots of bugs have been fixed until 1.11.13.
https://jira.mesosphere.com/browse/COPS-5314

@mesosphere-ci run all integration tests


Keep relevant files synced with the most recent version in the upstream repo.

Done with git cherry-pick to pick selective changes from upstream repo without affecting Expo integration.

I will merge this tomorrow.  I need to also push the changes to the NPM package and can’t do that from my phone.


feat(core): Add stream data return and reception

add file: utils/steam_pipe.py
add demo: samples/flask_web_api.py
Other core code modifications, Add and use the StreamPipe class at night
Add flask library to requirements

Features

xx
yy

Feature Docs

Influence
Not affecting other functions
Result

Download this MD file here

Other

Codecov Report
Attention: Patch coverage is 62.96296% with 20 lines in your changes are missing coverage. Please review.

Project coverage is 81.20%. Comparing base (6434503) to head (7706b88).

Files
Patch %
Lines

metagpt/utils/stream_pipe.py
57.69%
11 Missing :warning:

metagpt/roles/tutorial_assistant.py
66.66%
2 Missing :warning:

metagpt/actions/action.py
75.00%
1 Missing :warning:

metagpt/provider/anthropic_api.py
50.00%
1 Missing :warning:

metagpt/provider/dashscope_api.py
50.00%
1 Missing :warning:

metagpt/provider/google_gemini_api.py
50.00%
1 Missing :warning:

metagpt/provider/ollama_api.py
50.00%
1 Missing :warning:

metagpt/provider/openai_api.py
50.00%
1 Missing :warning:

metagpt/provider/qianfan_api.py
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1118      +/-   ##
==========================================
- Coverage   81.28%   81.20%   -0.08%
==========================================
  Files         248      249       +1
  Lines       13895    13946      +51
==========================================
+ Hits        11294    11325      +31
- Misses       2601     2621      +20

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

Awesome! Let’s start Review now!

@2495165664 . This is a good example, but the introduction of stream_pipe has intruded upon the original implementation. We can optimize it. With the help of metagpt.logs.set_llm_stream_logfunc and contextvars.ContextVar, we can achieve the same functionality without modifying any code under the metagpt directory.
Update the metagpt/utils/stream_pipe.py:
...
from contextvars import ContextVar
from metagpt.logs import set_llm_stream_logfunc

stream_pipe_var: ContextVar[StreamPipe] = ContextVar("stream_pipe")

def stream_pipe_log(content):
    stream_pipe_var.get().set_message(content)

set_llm_stream_logfunc(stream_pipe_log)

def write_tutorial(message):
    async def main(idea, stream_pipe):
        stream_pipe_var.set(stream_pipe)
        role = TutorialAssistant()
        await role.run(idea)
    ...
```.

Hello, there are still some CRs that need to be modified. After modification, they can be merged.

lgtm

@2495165664 there is a conflict
<<<<<<< main
    "web-api": ["flask==3.0.2"],
=======
    "android_assistant": ["pyshine==0.0.9", "opencv-python==<IP_ADDRESS>"],
>>>>>>> main

you should keep web-api and android_assistant both.

@2495165664 @shenchucheng Due to the presence of human intervention in metagpt, how can it be paused during the streaming print process on the web end to receive input from humans?


Run clang-extdef-mapping on the .ast files instead of the source

I just wanted to give you guys a heads up that I just landed: https://reviews.llvm.org/D128704 which makes it possible to run clang-extdef-mapping on the ast files directly instead of having to re-parse the source.
This gives a nice 3x speed-up on our codebase. You can see the numbers in the review above.
This will be included in clang 15.x - but you can detect if it's possible by parsing clang-extdef-mapping --help and it will mention ast files there when my patch is running.

This is a great addition. Thanks!
We will add this support for this in CodeChecker.
On Tue, Jul 5, 2022 at 1:49 PM Tobias Hieta @.***>
wrote:

I just wanted to give you guys a heads up that I just landed:
https://reviews.llvm.org/D128704 which makes it possible to run
clang-extdef-mapping on the ast files directly instead of having to
re-parse the source.
This gives a nice 3x speed-up on our codebase. You can see the numbers in
the review above.
This will be included in clang 15.x - but you can detect if it's possible
by parsing clang-extdef-mapping --help and it will mention ast files there
when my patch is running.
—
Reply to this email directly, view it on GitHub
https://github.com/Ericsson/codechecker/issues/3703, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB5IV2MC542YMF75ILE3GVLVSQONPANCNFSM52V6B2AQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Intune Deployment using line-of-business app instead windows app (Win32)

Why is it necessary to convert the MSI file to the .intunewin format for rollout? Intune can also install the MSI using line-of-business app.
What are the differences in using those procedures?

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

ID: 2a7b356e-3dbd-7f59-845e-a92e05865a52
Version Independent ID: 711c5497-373f-662c-7fae-ba60f046e50c
Content: Microsoft Surface Dock Firmware Update - Surface
Content Source: devices/surface/surface-dock-firmware-update.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@ekalweit converting the MSI to .intunewin format gives you the ability (which many have been asking for) to package the application (or more than one) together with command line installation and uninstall commands, but also dependencies.
"App dependencies are applications that must be installed before your Win32 app can be installed. You can require that other apps are installed as dependencies. Specifically, the device must install the dependent app(s) before it installs the Win32 app."
If we are talking about just one MSI, sure you can go the LOB app road, but in that case you need to take care of the whole management lifecycle.

@ekalweit - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by VLG17 so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


Checkup ? Question

Hey, i wanted to ask you some questions.
so the question is i wanted to use your search engine to index a website using sitemap.xml ( index and crawl the whole content from the website) this way it will be easier to pinpoint the engine on what pages it needs to search on. it would be much more easier to find content you are looking for.
because I followed your Read.me file but each time Doodle crawl through a website I find out that it only saves the page title and the website description. eg. Hackernew website. ( when I index and search for a keyword the result is almost the same( description)  but the URL is present and the title is not.
eg. when I search for Malware
the result present is
title:  Malware Strains Targeting Python and JavaScript Developers
description: The Hacker News is the most trusted and popular cybersecurity publication for information security professionals seeking breaking news, actionable insights
https://thehackernews.com/2022/12/malware-strains-targeting-python-and.html
see the description uses the main website description instead of the blog page.
am not sure if am missing something.

@RedWilly I'm having to make updates, but from what I found this code was copied from another repo.
I would really like it to be from the original author to continue with the improvements.
Original repo: https://github.com/phucvo0709/Clone-Google-Search-Engine

@pedrolaxe
I initially developed Doogle from Reece Kenney's course Google search engine clone. I suspect the Clone-Google-Search-Engine repo you provided was built using his course too, as I can see object-oriented PHP and PDO references similar to Doogle.
No copying of repos occurred.

@RedWilly
I never thought about using sitemap.xml to crawl the website.
Doogle crawls and inserts database entries using links using the insertLink($url, $title, $description, $keywords) and images using insertImage($url, $src, $alt, $title) functions respectively.
As of v1.1.2-beta all crawling functionality is contained within crawl.php and classes/DomDocumentParser.php.
Regarding the link you provided, I am not able to replicate your issue (see image below).
I am running Doogle v1.1.2-beta and PHP 8.1.


TiKV panic 'error: Corruption: L6 has overlapping ranges'

Bug Report

What version of TiKV are you using?
v4.0.1
Steps to reproduce
Unknown
What did you expect?
TiKV runs properly
What did happened?
TiKV panics:
[2020/07/03 12:24:11.015 +00:00] [FATAL] [lib.rs:481] ["rocksdb background error. db: kv, reason: compaction, error: Corruption: L6 have overlapping ranges '7A7480000000000112FFBC5F698000000000FF0000040380000000FF000E1xxxx......' seq:0, type:1 vs. '7A7480000000000112FFBC5F698000000000FF0000040380000000FF000E0xxxx....' seq:0, type:1"] [backtrace="stack backtrace:\n   0: tikv_util::set_panic_hook::{{closure}}\n             at components/tikv_util/src/lib.rs:480\n   1: std::panicking::rust_panic_with_hook\n             at src/libstd/panicking.rs:475\n   2: rust_begin_unwind\n             at src/libstd/panicking.rs:375\n   3: std::panicking::begin_panic_fmt\n             at src/libstd/panicking.rs:326\n   4: <engine_rocks::event_listener::RocksEventListener as rocksdb::event_listener::EventListener>::on_background_error\n             at components/engine_rocks/src/event_listener.rs:66\n   5: rocksdb::event_listener::on_background_error\n             at /rust/git/checkouts/rust-rocksdb-a9a28e74c6ead8ef/d472363/src/event_listener.rs:254\n   6: _ZN24crocksdb_eventlistener_t17OnBackgroundErrorEN7rocksdb21BackgroundErrorReasonEPNS0_6StatusE\n             at crocksdb/c.cc:2140\n   7: _ZN7rocksdb12EventHelpers23NotifyOnBackgroundErrorERKSt6vectorISt10shared_ptrINS_13EventListenerEESaIS4_EENS_21BackgroundErrorReasonEPNS_6StatusEPNS_17InstrumentedMutexEPb\n             at /rust/git/checkouts/rust-rocksdb-a9a28e74c6ead8ef/d472363/librocksdb_sys/rocksdb/db/event_helpers.cc:53\n   8: _ZN7rocksdb12ErrorHandler10SetBGErrorERKNS_6StatusENS_21BackgroundErrorReasonE\n             at /rust/git/checkouts/rust-rocksdb-a9a28e74c6ead8ef/d472363/librocksdb_sys/rocksdb/db/error_handler.cc:220\n   9: _ZN7rocksdb6DBImpl20BackgroundCompactionEPbPNS_10JobContextEPNS_9LogBufferEPNS0_19PrepickedCompactionENS_3Env8PriorityE\n             at /rust/git/checkouts/rust-rocksdb-a9a28e74c6ead8ef/d472363/librocksdb_sys/rocksdb/db/db_impl/db_impl_compaction_flush.cc:2797\n  10: _ZN7rocksdb6DBImpl24BackgroundCallCompactionEPNS0_19PrepickedCompactionENS_3Env8PriorityE\n             at /rust/git/checkouts/rust-rocksdb-a9a28e74c6ead8ef/d472363/librocksdb_sys/rocksdb/db/db_impl/db_impl_compaction_flush.cc:2317\n  11: _ZN7rocksdb6DBImpl16BGWorkCompactionEPv\n             at /rust/git/checkouts/rust-rocksdb-a9a28e74c6ead8ef/d472363/librocksdb_sys/rocksdb/db/db_impl/db_impl_compaction_flush.cc:2092\n  12: _ZN7rocksdb14ThreadPoolImpl4Impl8BGThreadEm\n             at /rust/git/checkouts/rust-rocksdb-a9a28e74c6ead8ef/d472363/librocksdb_sys/rocksdb/util/threadpool_imp.cc:266\n  13: _ZN7rocksdb14ThreadPoolImpl4Impl15BGThreadWrapperEPv\n             at /rust/git/checkouts/rust-rocksdb-a9a28e74c6ead8ef/d472363/librocksdb_sys/rocksdb/util/threadpool_imp.cc:307\n  14: execute_native_thread_routine\n  15: start_thread\n  16: clone\n"] [location=components/engine_rocks/src/event_listener.rs:66] [thread_name=<unnamed>]

The key in the panic log cannot be found in TiKV's logs about ingesting sst. Neither can it be found in RocksDB's logs and manifests.
After deleting the panic mark file and restarting the TiKV node, it will soon panic again, printing the same key in the log.
cc @Little-Wallace @yiwu-arbug @zhangjinpeng1987

@MyonKeminta is the instance still exists?
@Little-Wallace is this the issue you mentioned?

@MyonKeminta The tikv node was stopped and the data and logs still exists.

The above error is caused by RocksDB detected unordered SST files after a L5->L6 compaction. The two unordered SSTs are generated by the same compaction. After investigating the remaining DB, we find that one of the compaction input SST is unordered internally. The problematic SST is generated as a result of multiple compactions, and the intermediate result is gone, so we are not able investigate further this time. In the sequence of compaction that leads to the problematic SST there's also ingested SSTs participated in it, so we cannot rule out if one of the ingested file is unordered.
Followup will be to add logic to fail compaction once unordered result is generated, so that next time we reproduce the problem we can examine the data and see how it could happen.

@yiwu-arbug Thank you :)

Got another reproduction from another user POC test. Following up.

The issue, at least the last reproduction, is due to RocksDB block cache cache key conflict, causing compaction reading wrong block content from file. https://github.com/facebook/rocksdb/issues/7405#issuecomment-694595587

Adding compaction and read path consistency check (by @Connor1996): https://github.com/tikv/rocksdb/pull/195

known affected operation systems:
AWS, ubuntu 20.04LTE, kernal 5.4.0
AWS, ubuntu 18.04LTE ,kernel 5.3.0
GCP, Debian 10, kernel, kernel 4.19.0
known unaffected operation systems:
IDC, CentOS 8.0.1905, kernal 4.18.0
GCP, Debian 9, kernal 4.9.0
The test:
run the program https://gist.github.com/ajkr/2eac6fe4d918d0c8819e9656ec4eab41.  being a new file on a ext4 file system,  can be 10. Check if the second number of each line is sequential or random. The operation system is affected if the number is random.

https://github.com/tikv/rocksdb/pull/205 Generate uniqueID based on db instance and sst file number instead of inode number

@Connor1996 Can you help also cherry-pick the change to disable force-consistency-checks and the change to check key ordering?

@Connor1996 Can you help also cherry-pick the change to disable force-consistency-checks and the change to check key ordering?

okay

kernel 4.18.20-2.el7.x86_64  The same thing happened，error： L5 have overlapping ranges

kernel 4.18.20-2.el7.x86_64 The same thing happened，error： L5 have overlapping ranges

What's your linux distribution and its version, and what's the file system you use? Just want to keep a record. Also can you run the test program in this comment and report the result? https://github.com/tikv/tikv/issues/8243#issuecomment-708774344


Template will not depoy

The following error occurs when this is deployed in US East.
I have deleted all openAI resources and purged them.  Still get this error.  There is zero information I have found that is helpful in resolving this on the internet.
Another operation is being performed on the parent resource '/subscriptions/xxx-xxx-xxx-xxx-xxx/resourceGroups/CosmosRAG1/providers/Microsoft.CognitiveServices/accounts/jeffmoore-openai'. Please try again later. (Code: RequestConflict)

It doesn't matter what region I deploy in I get the same error as listed above.
Interestingly if I hard code a different supported region then I get an error stating that S0 (standard) is not a valid option, yet I can create that very resource manually in the region I've specified, eg eastus2

This was fixed by modifying the ARM template to have the models deploy one at a time by putting a dependsOn in the template.


Idea for webpage capture/bookmark

I've been experimenting with org-capture together with Firefox bookmarks lately. By using a Firefox addon and org-protocol, links can easily be send to Emacs. In Emacs, org-capture is opened and I can save the links to a file called "links.org" in my org directory. I've configured Emacs in such a way that "links.org" will automatically be exported to links.html which I've "bookmarked" in Firefox. Furthermore, a package called org-board  saves the html+css (using wget) file of my captured webpage, so that I'm able to view my bookmarked website without an internet connection. I've implemented the following code for this system:
(require 'org-protocol)

(def-package! org-board
  :commands
  org-board-open
  org-board-archive
  :config
  (setq org-board-capture-file "links.org")
  (setq org-board-default-browser 'firefox)
  )

(defun do-org-board-dl-hook ()
     (when (equal (buffer-name)
             (concat "CAPTURE-" org-board-capture-file))
       (org-board-archive)
       (save-buffer)
       (org-html-export-to-html)
       ))
(add-hook 'org-capture-before-finalize-hook 'do-org-board-dl-hook)

(setq org-capture-templates `(
	("p" "Protocol" entry (file+headline ,(concat org-directory "notes.org") "Inbox")
        "* %^{Title}\nSource: %u, %:link \n #+BEGIN_QUOTE\n%i\n#+END_QUOTE\n\n\n%?")
	("L" "Protocol Link" entry (file+headline ,(concat org-directory "links.org") "Links")
        "* %?[[%:link][%:description]] \nCaptured On: %U")
    ("c" "capture through org protocol" entry
             (file+headline ,org-board-capture-file "Links")
             "* %?[[%:link][%:description]] \n:PROPERTIES:\n:URL: %:link\n:END:\n\n Captured On: %U")
))

However, I failed to get this to work your conform coding conventions, but that's mainly a result of my lack of Elisp knowledge. In either case, if you like this implementation, let me know :)

I like the idea. I'll look into this more, closer to the new years. I have big plans for the org modules I need to finish before adding to its features.

af4ca97 introduces org-protocol support, which I'll build better org-board/org-capture support onto later. This is a WIP, but for the time being, I'll consider this issue "resolved".


Feature Request: Merchant History

This is probably too much to ask for a stateless server but it may be worth tracking each merchant's reported history so users can track stats like Wei card frequency on their server. Probably requires some sqlite or postgres server to work optimally but would be a cool feature.

It's not actually stateless, it's all stored in a database. That's how the account and profile system can work.

seems like this was recently added a few days ago


Error.

ERROR in ./client/app/index.js
Module build failed (from ./node_modules/babel-loader/lib/index.js):
SyntaxError: C:/Users/Hp/Desktop/tafara/Hospital-Management-System/client/app/index.js: Unexpected token (26:4)

Hi , developer does this repository ([Hospital-Management-System] working

i'm facing problem


CI doesn't like targeting Bevy's main branch

Git dependencies seem messed up. See https://github.com/bevyengine/bevy_editor_prototypes/actions/runs/10999742035/job/30540587660?pr=24, from #24.
Paging @BD103, @mockersf and @tim-blackbird, who may have ideas on how to fix this.

That looks like a simple 404 (not found) error code. It likely was just a spontaneous error, let me try re-running CI.

It was a small error in the manifest. Comment


fix the dropdown menu width logic

This PR fixes #19826
Currently the width calculation doesn't take the padding/font size into the consideration, as a result, the calculated size is not accurate.
before:

after:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.004%) to 42.241%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/services/connection/browser/connectionDialogWidget.ts
1
71.43%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
28174

Relevant Lines:
62333

💛  - Coveralls


ord needs recovery.

When I press ctrl-c only once, the shell closes correctly, but json-api will also be closed. How can I keep it running correctly?

When you CTRL-C that closes down the server and with that also the json-api. If you want to keep it running keep it running in a shell window or start a tmux session. If you are on Linux you can use systemd to run ord for you. See our service files in /deploy/ord.service


Vulcanize breaks dom-repeat in OSX and iOS Safari

The presence of <template is="dom-repeat"> (even with no data input from items) causes the following error in Safari: TypeError: Attempted to assign to readonly property.
I'd be happy to throw together some kind of example if there's a tool you recommend.

A reproduction would be great.

The issue mentioned is caused by Polymer on Safari in strict mode as mentioned in https://github.com/Polymer/polymer/issues/2581. That bug is on their roadmap with a proposed solution of "use strictify[ing] all Polymer source."

Thanks @davidlwatsonjr! Even though it's a bug in core, I'm going to keep this open so that other can find it if they need too. Thanks for the report @jshado1!

critical for a year? :(


[feature-request] support dark/light mode

Hello, I tried your plugin and it unexpectedly works well with the reading mode. However, there is one issue. Is it possible to add support for a light/dark mode by using two different system settings?
Additionally, I have a suggestion. It would be convenient if there was an option in the settings to annotate words. In my particular scenario, I am studying English and I want to highlight words that I don't understand or consider important.
To achieve this functionality with the current plugin, some things are needed. Firstly, a batch highlight feature (a suggested method is to include a selection box when highlighting words, allowing users to choose a color scheme). Secondly, the addition of a management library for highlighted words. It would be inconvenient if the only way to manage the words is through the settings panel, especially when there are many words. Lastly, it would be great to have color management, allowing users to choose from several preset color schemes.
These are my suggestions. I understand that the amount of work involved may be significant, so feel free to choose what you think is best. Thank you for your efforts.

Hi @YIRU69, thanks for your ideas!
I only use obsidian in dark mode, so I did not check this before. How do you think such a settings should look like? Do you know any other plugins that handle this light/dark mode well?
I think I do not fully understand your second suggestion. What exactly is your workflow to use the plugin?
I could think of a batch highlight like separating keywords using a delimiter. So instead of creating a keyword entry for each word (eg. tree, leaf, root), one can simply write tree:leaf:root to highlight them all in the same color.

Hi, for the second suggestion. Please refer to the functions of this plugin. (https://github.com/Quorafind/Obsidian-Mini-Toolbar) What I have in mind is the ability to select words, make annotations, and finally be able to generate a word count of the annotations.
If it use a delimiter, it break the markdown style and the origin text.
For the first suggest, you can set up two color schemes: one for light mode and one for dark mode. If you think it's unnecessary, you don't have to bother with it. I don't think it's very important either, but there is one issue. When switching between light and dark modes, we need a color scheme that is compatible with both.


Design of HTTP client

Basic points:

support for all HTTP methods
works in browser and node
wraps response/request data in both success and error case
supports typed request/response
integrates with a generic tracing instrumentation
websocket?

Any ideas both on the API and on the features are open for discussion!

Further points made by slack conversations:

avoid wrapping libraries that are promise-based (will slow down everything due to promise performance)
avoid wrapping any library with big footprint

Ideally build upon the platform api (xmlhttprequest in browser & http/https in node)


Forgotten class InetAddress in the description of enum.

There was forgotten class InetAddress in the description of enums in autocomplete documentation.

Codecov Report

Merging #177 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #177   +/-   ##
=========================================
  Coverage     91.11%   91.11%
  Complexity      100      100
=========================================
  Files             2        2
  Lines          1609     1609
  Branches        404      404
=========================================
  Hits           1466     1466
  Misses           46       46
  Partials         97       97

Continue to review full report at Codecov.

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

Oops! Good catch! Thank you very much!


Handlebars 2.0.0 suddenly broken in Ember app

I'm using handlebars 2.0.0 as part of my Ember app's build process. I had to delete my node_modules directory. After an npm install, handlebars suddenly started producing garbage. Example:
data.buffer.push(this.escapeExpression,(,(helpers.partial || (depth0 && depth0.partial) || helpers.helperMissing).call(depth0,"main_menu",{"name":"partial","hash":{},"hashTypes":{},"hashContexts":{},"types":["StringLiteral"],"contexts":[depth0],"data":data}),));

I hunted down an old handlebars directory from backup and copied it over the one in my node_modules directory. Suddenly, things started working again:
data.buffer.push(escapeExpression(((helpers.partial || (depth0 && depth0.partial) || helperMissing).call(depth0, "main_menu", {"name":"partial","hash":{},"hashTypes":{},"hashContexts":{},"types":["STRING"],"contexts":[depth0],"data":data}))));

I checked and both versions are labeled "2.0.0" inside package.json.
I have no idea what happened here. Is it possible npm version got corrupted? Or was it updated without a version bump?

Handlebars was not repushed in npm to my knowledge and I believe I'm the only one with rights to push to it on npm. Have you diff -red the two directories?

The problem has disappeared.
In the meantime, I have deleted npm's local cache. That was probably what fixed it. Although how the thing got corrupted in such a way to produce garbled output instead of errors is anyone's guess.
As I didn't keep the broken copy, there's probably no way to figure out what had happened.


Sphinxcontrib-versioning does not like changes in docs/config.py

See https://app.circleci.com/jobs/github/poldracklab/smriprep/2059/parallel-runs/0/steps/0-108
/cc @rwblair

I believe this is because the github link package is not listed in the docs requirements, and its code only exists on the branch in the PR. sphinx-versioning checks out other branches from the github remote and uses the PR branch's conf.py.

Yup, that is what I thought. Shouldn't it just skip old branches because they are already generated?

Ugh, I don't know whether it would fail with old tags, but I can see it is failing for master.
Should we not generate master in the build?

It might be just as easy as using __file__ as reference when adding the sphinxext folder to sys.path.

Fixed via c6371b5ffdb3cb7a7ded0a8935036597f0f3f60d

I'm confused what was the path that ended up getting added with dirname as opposed to abspath?


RHOAIENG-15743: fix(odh-nbc): trim ca-bundle certData to avoid logging scary messages

https://issues.redhat.com/browse/RHOAIENG-15743

https://github.com/opendatahub-io/kubeflow/pull/455

(cherry picked from commit 3cc53abd298dccad96ba872603645b1a430ebb95)

/lgtm

/approve
@atheo89 can you please merge? We don't have prow/tide active here. Could be it's so because of https://redhat-internal.slack.com/archives/CHY2E1BL4/p1731659879988309?thread_ts=1731637664.069799&cid=CHY2E1BL4?