Common misspellings

Adds transformation which corrects spellings of commonly misspelled words.

Hi @csinva
I was assigned as one of your reviewers.
Thanks for this contribution!
I like this transformation.
Here's my general review:
Correctness: No checks (1 workflow awaiting approval)
Interface: I think you have chosen the correct interface(s).
Applicable Tasks & Keywords: I think you have chosen the relevant tasks and keywords.
Specificity: The change is specific to texts with misspellings.
Novelty: This transformation is not yet implemented in NL-Augmenter.
Adding New Libraries: No new libraries appear to be added.
Description: The README is clear and explains what the transformation is aiming to do.
Data and code source: Data and code (with GNU GENERAL PUBLIC LICENSE) sources are referenced in README.
Test Cases: 5 test cases have been added.
Evaluating Robustness: Robustness evaluation has been performed.
Languages other than English: No


Typo fixes that can mislead the reader

Commits contain typo fixes that can mislead the reader. If you feel they are correct, merge them.
I have read this book/manual till dataviz part 2. So fixes till there only. If I read it more later, I will submit another pull request.

I'm also very disinclined to accept any PRs from @santosh after this tweet cc @merwok

I agree that the tweet is in bad taste.  PRs get reviews and thanks, but we’re doing this on our free time.  If santosh shows more goodwill in future PRs, I will gladly review them.

Woo! That's great. You people have been judgemental after a tweet. That's cool.

Yes, people can have reactions after interactions in a PR or tweet.
I closed the PR because the changes are IMO not improvements.  As I said, I will look at other PRs from you without prejudice.

Well, what are plans for Python 3 though? Are we going to adapt it, or not?
If so, are we going to make present code compatible in both 2 and 3 (which I think will be cumbersome for the readers at beginner level)? or just support 3?
There are multiple compatiblity related pulls too, I guess.

@santosh I think how I'd like to approach this is have essentially two versions of each tutorial, one for py2 and one for py3. Then the reader can toggle what s/he wants to learn.

So let's start?

@santosh I'm very strapped for time. I'd happily review pull requests, and I'm sure @merwok as well. Perhaps start with the actual code first?

@econchick What about the pulls already addressing Python 3?

#182
#157

You can see that neither of those actually have separate codebase for the tutorials, it just makes the current tutorial code compatible, which as you said, can be confusing to the reader. It may be a good start to pull those PRs and work off of that.

I didn't understand?


Using the documentation feature for policy assignments

Hello there,
I'm having trouble using the assignment documentation feature to document policy assignments (for both built-in and custom policies). It is able to document initiative assignments without a problem, pulling the correct name, description and effect(s). However, if I add an assignment of definition type policy to the assignment ID in the representativeAssignments block, the result looks like this:

I couldn't find anywhere in the documentation that it is meant to be used only with initiative assignments, so I wanted to confirm if this is working as expected or whether this is an error on my end.
Here is the assignment in question, should it be helpful in debugging this issue.
{
    "nodeName": "parentNode",
    "parameters": {
        "resourceGroupLocation": "canadacentral"
    },
    "definitionEntry": {
        "policyName": "ffb6f416-7bd2-4488-8828-56585fef2be9",
        "friendlyNameToDocumentIfGuid": "Deploy export to Log Analytics workspace for Microsoft Defender for Cloud data"
    },
    "assignment": {
        "Name": "exportDefenderLogs",
        "displayName": "Deploy export to Log Analytics workspace for Microsoft Defender for Cloud data",
        "description": "Enable export to Log Analytics workspace of Microsoft Defender for Cloud data. This policy deploys an export to Log Analytics workspace configuration with your conditions and target workspace on the assigned scope. To deploy this policy on newly created subscriptions, open the Compliance tab, select the relevant non-compliant assignment and create a remediation task."
    },
    "children": [
        {
            "nodeName": "PRD",
            "parameters": {
                "resourceGroupName": "<rg-name>",
                "workspaceResourceId": "/subscriptions/<subscription-id>/resourceGroups/<rg-name>/providers/Microsoft.OperationalInsights/workspaces/<workspace-id>"
            },
            "scope": {
                "prod": [
                    "/providers/Microsoft.Management/managementGroups/<management-group-id>"
                ]
            }
        }
    ]
}

Thanks for the hard work put in this project!

The documentation feature works for initiative assignments only. I should document that caveat. There are no plans to support documentation for Policy  Assignments.


Update resty

" > " is for string comparisons. Use -gt instead. [SC2071]
Prefer [ p ] && [ q ] as [ p -a q ] is not well defined. [SC2166]
I have ignored SC2145
Argument mixes string and array. Use * or separate argument. [SC2145]
[ "$dry" = "yes" ] && echo "curl -sLv $opt \"$dat\" -X $method -b "$cookies/$domain" -c "$cookies/$domain" \"${args2[@]}\" \"${curlopt2[@]}\" \"${curlopt[@]}\" \"$_path$query\"" && return 0

Thanks @extremeshok, I also applied the shellcheck recommandations in the #50 PR, that is to be landing soon.


I can model terminate end events for Camunda Platform 8

Problem you would like to solve
Since Camunda Platform 8.1, terminate end events are supported. I want to model a process that contains a terminate end event.
Currently, I can't add a terminate end event or change the type of an end event to terminate.
Proposed solution
I can add terminate end events to my process. Or, change the type of an end event to terminate.
Alternatives considered
None.
Additional context
Related Zeebe PR: https://github.com/camunda/zeebe/pull/10442

Same issue here. I migrated from camunda modeler 3.4.0 to 5.3.0 and now I can't model terminate end events.

@ghilainm do you model Camunda 8 or Camunda 7 diagrams? If you do modeling only I suggest you to use Camunda 7 diagram type. It works best (largest BPMN compatibility).

@nikku Thx for your answer.
I am using Camunda 8 diagrams :). Didn't know it has any impacts.
Is it the target to have the same BPMN compatibility for Camunda 8 or not?

Is it the target to have the same BPMN compatibility for Camunda 8 or not?

It is, but we're not there yet.


🪸 Update SearchTopcis form to Form component when it's updated

What
SearchTopic uses a html <form> instead of our Form component.
The Form component is based on the aiven core implementation and mirrors their code. It does not take a role or aria-label attribute, which is why for better accessibility we don't use it here right now.
When the aiven core Form component is updated, we can follow with our implementation and update the SearchTopic
see this comment: https://github.com/aiven/klaw/pull/239#discussion_r1032257275

@programmiri is this a duplicate of https://github.com/aiven/klaw/issues/248 ?

I'll close this issue in favour for https://github.com/aiven/klaw/issues/889


Support for Foundry V12.

I spent considerable time trying to figure out the project structure, I'm not a web dev expert but this should make the plugin compatible with V12 as far as I could see. The breaking changes were:

getAudioContext was changed into getAnalyzerContext
Some core labels for the ambient sound panel changed identifiers
The AmbientSound window/application was completely revamped, I did some touch-ups to support the new structure including in the Svelte components. Not as pretty as Foundry's original but it will suffice for now.

Closes #118

I spent considerable time trying to figure out the project structure, I'm not a web dev expert but this should make the plugin compatible with V12 as far as I could see. The breaking changes were:

getAudioContext was changed into getAnalyzerContext
Some core labels for the ambient sound panel changed identifiers
The AmbientSound window/application was completely revamped, I did some touch-ups to support the new structure including in the Svelte components. Not as pretty as Foundry's original but it will suffice for now.

 How do you integrate these changes if they cant be auto updated in Foundry?

I spent considerable time trying to figure out the project structure, I'm not a web dev expert but this should make the plugin compatible with V12 as far as I could see. The breaking changes were:

getAudioContext was changed into getAnalyzerContext
Some core labels for the ambient sound panel changed identifiers
The AmbientSound window/application was completely revamped, I did some touch-ups to support the new structure including in the Svelte components. Not as pretty as Foundry's original but it will suffice for now.

 How do you integrate these changes if they cant be auto updated in Foundry?

Until the maintainer approves the pull requests and publishes an updated version, unfortunately the only option is what @ChunkLightTuna mentioned - that is clone my fork, build it locally and patch your foundry data manually.

Just as an FYI - (after stumbling through trying to figure out what i actually needed to install and build all this)  I have you fix installed and it appears to be working perfectly for me.

Okay, as soon as the CI passes I will merge it and release new version

Mh. Something on jest, possibly due to the changes I had to apply on the passed jquery token from Foundry. I'll have a look

@frondeus The failure was indeed due to how I changed the AmbientSound component, it introduced a regression with respect to the V11 version. I fixed the regression but that still leaves the V12 version uncovered - I would need to change the fixture for that. If you don't mind I'd leave that for whoever takes the maintenance, this was more like a hotfix to not leave the community (...and myself tbqf :D ) without a V12 version.

Yeah ok, merging :)

I didnt test it, but i trust the community. New version has been released.


Adds node ID caching layer to lookup service.

https://groups.google.com/forum/#!topic/rackhd/CiRQhVdkE38
Attempting to improve logging performance by introducing caching to lookup-service. First cache miss results in a DB query, subsequent cache lookups for the same IP or MAC address will wait for the first query to respond unless the there is already a valid entry in the cache.
This should greatly reduce number DB queries by the LookupService.

Uggh. -0.02% code coverage loss.

Two questions:

Is this missing cache refresh/expiration right now?
I assume that independent calls to a cache promise object from different places within the code will chain onto the shared reference to the object, yes? That way we avoid the promise multicast problem?

I think code coverage will always go down if you delete anything~ even if its replaced with updated code~

Correct, it doesn't try to populate the cache, nor does it every clear the cache. It should be trivial memory usage even with 1000s of nodes, but if it is a problem we can easily address it then.

I think the answer is yes. I'll double check with you.

@rolandpoulter any reason not to use a the lrucache library with some options for timeout and decay of the cache over time - seems like that would have a smaller overall memory footprint, where this could conceivably grow memory usage without bound under pathological testing conditions (which we've experienced before)

@heckj Nope, I didn't know about it. It should be pretty simple to change it to use lru-cache instead of a plain Object.
I'll update my PR soon.

@benbp @heckj ^^

:+1:

:+1:

@rolandpoulter :
The caching is good, I just wondering doesn't the mongodb have such caching capability? if so, in Mongodb or Waterline, we can have a common cache layer for all query.
And could you explain how you sync your cache with real DB data?

@yyscamper MongoDB (like any database system) includes some caching, this is intended to cache before we make a request to mongoDB so that we don't need to incur the cost of that call every time we're using the lookup service.
LRU is specifically set to expire out data, and this code is set to attempt the cache first, and load on a cache miss, with a limited set of values. there's no explicit sync otherwise, so this does introduce the possibility that the DB can be updated and the cache will return a previous value that's not strictly correct. In the lookup service, the strict atomicity isn't necessary and the eventual consistency is quite acceptable.

Fixed whitespace.

@heckj @benbp I think this is ready to be merged.

:+1: from me

@benbp lru-cache was not being used before, so upgrading the version should not cause anything to break. Just double checked that.

:+1:


cannot convert call result type '(_)->Void' to expected '(()->Void)!'

At compilation produces 76 errors of type:  cannot convert call result type '(_)->Void' to expected '(()->Void)!'. iOS 11.

Could you give us more details? What are you doing to get this errors?
As long as pod lib lint --allow-warnings on our pods returns no errors, things are right in our side.

I'm create project, add Podfile:
source 'https://github.com/CocoaPods/Specs.git'
platform :ios, '11.0'
use_frameworks!
def pods
pod 'SwiftMatrixSDK'
end
target 'VistaChat' do
pods
end
Then i go to еру terminal and prescribe pod install.
Then i open .xcworkspace, compile a project and get this errors.

I'm getting this error in the method "@nonobjc @discardableResult func resetPaginationAroundInitialEvent(withLimit limit: UInt, completion: @escaping (_ response: MXResponse) -> Void) -> MXHTTPOperation {
return __resetPaginationAroundInitialEvent(withLimit: limit, success: currySuccess(completion), failure: curryFailure(completion))" for  currySuccess(completion).

I am also getting the same issue. Any updates on the solution??

SwiftMatrixSDK doesn't work for swift 4.
My solution was to use the swift 3.2 version.
I did this by selecting pods in the Xcode project explorer, SwiftMatrixSDK in the target, selecting 'Build Settings' and then choosing Swift 3.2 in the 'Swift Compiler - Language'

SwiftMatrixSDK doesn't work for swift 4.

True. This is the problem.
pod lib lint --allow-warnings works when running in the sdk folder because we have a .swift-version file pointing to swift 3.2.
If someone is strong enough in swift to convert the currySuccess template method in swift 4, the help will be much appreciated.


AdvancedControlBase.get_control_inject() takes 5 positional arguments but 6 were given

ComfyUI Error Report
Error Details

Node ID: 108
Node Type: KSampler
Exception Type: TypeError
Exception Message: AdvancedControlBase.get_control_inject() takes 5 positional arguments but 6 were given

Stack Trace
BUG01.txt

Need to roll back the version, issue solved

You don't need to rollbakc, you need to update Advanced-Controlnet.
https://github.com/Kosinkadink/ComfyUI-Advanced-ControlNet/issues/202
I made the code changes in ComfyUI that make require get_control to take an extra input, and I naturally pushed those changes up in my nodes within 3 minutes of the PR getting merged.

There are no incompatibilities between latest ComfyUI, Advanced-ControlNet, and AnimateDiff-Evolved. You just need to make sure they are updated and you're good to go.


Doc - Notify user of second device using SA F9.

Provide notice for extraneous queries from SA F9,
When extraneous queries, not sent by the SAE J1939-84 software are observed:

Print a time stamped message in the log:
HH:MM:SS.SSSS ccccccF9 [n] XX XX XX XX XX XX XX
WARN: Unexpected Service Tool Message from SA F9 observed. Test results uncertain. False failures are possible.
Add a pop up message for the UI
Unexpected Service Tool Query from SA F9 observed. Test results uncertain. False failures are possible.
Should a 2nd Query appear while this Pop-up is displayed, a 2nd Popup will not be displayed.
This popup shall not interfere with the user interface pop ups, and allow the user to respond to them.

After II.B add the following footnote.
[xx] Implementors are encouraged to detect overlapping queries created by on-board devices using the test tool’s F9h address. Notices to the user through the implementor’s user interface, and messages in the test log of these occurrences are recommended.

I suggest that we word this as an invalid test, not a warning.  A second tool makes the test invalid; all DS transport protocol will be effected.  It's not just overlapping queries.

Joe reported that Bob collected this log fragment today 5/28.
Start Test 1.26 - Data stream support verification
Expecting the following messages:
...
Step <IP_ADDRESS>.a - Reading bus for 20 seconds
WARN: 6.1.26 - Unexpected Service Tool Message from SA 0xF9 observed. Test results uncertain. False failures are possible
ABORT: User cancelled testing at Part 1 Step 26
WARN: <IP_ADDRESS>.f - N.5 SPN 84 provided by more than one ECU
End Test 1.26 - Data stream support verification

Footnote 2 inserted into 24June as built draft as: "Implementors are encouraged to detect overlapping queries created by on-board devices using the test tool’s F9h address. Notices to the user through the implementor’s user interface, and messages in the test log of these occurrences are recommended."


Additional output fields from package.json

Would it be possible to allow the consumer to define additional fields that they would like extracted from package.json and added into the output JSON? For instance, I'm looking to get the homepage field. I think with a simple addition to the options and reflection in the LicenseMetaAggregator this can be accomplished.
I can also help with the implementation and PR if you'd prefer.

@nima-ap Sounds like a helpful functionality. I’d be happy if you could provide a PR! 😄


Scrollbar inconsistently handles clicking on scrollbar region (not thumb)

Describe the bug
From https://github.com/microsoft/terminal/issues/18063

Typically when mouse click the scrollbar(not the thumb), hold the mouse button down. The scrollbar will:
page up, , page up, page up, page up, page up,...
Then thumb will stop at the mouse pointer position.
Instead, there are 3 random behaviors.

Page up/down once and stop.
Thumb not stop. And reach the top/bottom position.
Thumb move across the mouse pointer position.

https://github.com/user-attachments/assets/12989340-50c2-4ff1-a14c-82d142296614
Steps to reproduce the bug

Go to a page with a long scroll region in the WinUI 2 gallery (I used ListView)
Click and hold on the scrollbar region (not the thumb)

Expected behavior
We should be consistent. I'd expect it to scroll to the pointer position.
Screenshots
No response
NuGet package version
None
Windows version
No response
Additional context
No response

Does this bug also repro in WinUI3? This may not meet the bar for WinUI2 servicing.

Yup! Just tried it in WinUI 3 controls gallery (All samples, then made the window very narrow).


BUG: Text on 'Subscribe' button becomes invisible on hover

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When hovering over the 'Subscribe' button in the footer, the text becomes invisible due to the hover effect applied to the button. This makes it difficult for users to read the text.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned!
Make sure to attach before & after screenshots in your PR. Also, star the repo before working on it. While creating your PR, ensure you pull all existing changes beforehand to avoid conflicts.
Happy open-source contributing!


xplat: use xmm for float args

Previously xplat LowerCall args were limited to REG_INT_ARG... Adding XMM* support and a test case to make sure we don't break.
This PR also improves performance on xplat jetstream
Disabling couple of additional wasm tests on xplat with a note.
edit: added another test case

@dotnet-bot test Windows ci_slow_x64_debug please

@jianchun Thanks for the review. Made some additional improvements.


Turn off A2DP

How can I turn off the A2DP to connect to the WiFi?
void A2DPOn(bool BlnOn) {
	if (BlnOn == true) {
		IniA2DP();
	}
	else
	{
		a2dp_sink.end();
		Serial.println("Stopping A2DP");
	}
}

a2dp_sink.end(); should work. However I would recommend to reboot if you want to have the full memory available again!

The memory is not the issue. It doesn't seem that the a2dp_sink.end(); is enough to connect to my broker.

Did you just try to start Wifi first with your own code?

That's what I do with WiFi ini. I didn't connect to the WiFi first.

Try to activate the ESP32 logging. Maybe you get some more info there...

Seems that after enabling the A2DP and disabling, I can not load the certificate correctly.

[  7730][E][ssl_client.cpp:37] _handle_error(): [start_ssl_client():276]: (-9984) X509 - Certificate verification failed, e.g. CRL, CA or signature check failed

I would guess that there is not enough memory....

You can try to release additional memory with a2dp_sink.end(true); with the effect that a2dp can not be started any more

I just did and it is connected now. sadly I can not start the a2dp anymore without restarting it. So my idea is nothing worth anymore. I use PSRAM and I know the PSRAM is unacessible for DMA so encryption functions (like those, used in SSL) are not working. I think I am stuck on this...

Maybe disable PSRAM on code for a2dp... Do you know how to do this?

No, I never tried. Anyhow I don't think that this is an error in my library: A2DP has nothing to do with Mqtt.
Did you try to search for issues in the ESP32 Arduino or I suggest to open an issue with the MQTT library provider

I did not put the question on the Arduino side. I know they are working an new designs that are more compatible with encryption. What I did is test my esp32 further to see how it is reacting on certain settings.

Arduino IDE 1.8.19
Pins 21,22
SDK: ESP Arduino 2.0.4
Board ESP32 DevKit Wrover IE
I2C device PCF8574
ESP32 chip with 2 CPU cores, WiFi/BT/BLE, silicon revision 3, 4MB external flash
Chip id: D4BD9710
Model number: ESP32-D0WDQ5

PSRAM OFF -> boot on A2DP -> a2dp_sink.end();  -> WiFi : WiFi can not connect
PSRAM ON -> boot on A2DP -> a2dp_sink.end();  -> WiFi on -> MQTT can not connect
Conclusion: With PSRAM I was able to connect to the WiFi without clearing all the memory of A2DP (a2dp_sink.end();) , but I couldn't connect to the secure MQTT. I believe it is possible if the MQTT is not HTTPS based website. So for HTTP it could equally work with PSRAM.
Because I have in my case a secure server it is easier for me to restart the esp. If there is no device connected to the A2DP I wait for 1 minute and turnoff the a2dp_sink.end(true); initialize the WiFi connect the MQTT and see if there are any messages and the restart.

I agree: restarting is the cleanest solution and you don't have to worry that certain memory does not get released...


Upgrade Mattermost.app to v1.1.1

Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Updating the :checkpoint value in appcast is required. You can find it with curl --silent --compressed "{{appcast_url}}" | sed 's|<pubDate>[^<]*</pubDate>||g' | shasum --algorithm 256 | awk '{ print $1 }'.
Alternatively, you may wish to use cask-repair. It can at once update the cask’s version (according to what you give it) taking care of updating both the cask’s sha256 and the appcast’s :checkpoint, while still giving you the chance to perform other manual fixes.
You can easily install it with brew install vitorgalvao/tiny-scripts/cask-repair. cask-repair --help will show you some options, but as a whole the script will guide you through the update process.

using the curl option give me always a checkpoint error: cannot use the sha256 for an empty string
Instead, I used the cask-repair tool :+1:


Kubectl commands work even after deleting an imported cluster from the dashboard

Expected vs actual behavior
Even after I do a delete from the dashboard, and if kubectl is already open, I'm still able to perform operations on the cluster for a few minutes. Only after a while it starts throwing error. (image attached)
Steps to reproduce the bug

Import a cluster to Paralus
Open web kubectl (prompt) & execute some command i.e. kubectl get pods
Delete the cluster from the UI, but clicking the delete cluster. Ensure the web-kubectl screen is still open
After the cluster has been deleted from the UI, execute another command kubectl create ns testns followed by a 'kubectl get ns' - you'll observe that the namespace is created as well as listing command works even though the cluster is deleted from the UI.

Are you using the latest version of the project?
NAME     	NAMESPACE	REVISION	UPDATED                                	STATUS  	CHART     	APP VERSION
myrelease	paralus  	1       	2022-10-14 19:08:59.321602061 +0530 IST	deployed	ztka-0.1.7	v0.1.6

What is your environment setup? Please tell us your cloud provider, operating system, and include the output of kubectl version --output=yaml and helm version. Any other information that you have, eg. logs and custom values, is highly appreciated!
clientVersion:
  buildDate: "2022-05-03T13:46:05Z"
  compiler: gc
  gitCommit: 4ce5a8954017644c5420bae81d72b09b735c21f0
  gitTreeState: clean
  gitVersion: v1.24.0
  goVersion: go1.18.1
  major: "1"
  minor: "24"
  platform: linux/amd64
kustomizeVersion: v4.5.4
serverVersion:
  buildDate: "2022-08-17T18:47:37Z"
  compiler: gc
  gitCommit: 95ee5ab382d64cfe6c28967f36b53970b8374491
  gitTreeState: clean
  gitVersion: v1.24.4
  goVersion: go1.18.5
  major: "1"
  minor: "24"
  platform: linux/amd64

Screenshot

Suggestions
Can we do something about it? Like maybe running a kubectl delete ns paralus-system in the background as soon as the cluster is deleted from the UI.

[x] I've described the bug, included steps to reproduce it, and included my environment setup with all customizations.
[x] I'm using the latest version of the project.

Please aasign it me -

I am able to replicate it, need to check if kubectl delete ns paralus-system is right (or) make the UI readonly


Fix missing CrashTypeId in minidump post

When we moved to a zipped upload via pre-signed url we broke the ability to specify which crashTypeId is associated with the minidump.

This was fixed by a backend change.


uv run can install two versions of the same package (and forces upgrades)

I was trying out different versions of pytest with a pytest plugin to see when it was broken. I mentally was thinking of uv run as the Python launcher for UNIX, a way to avoid activating a venv (py instead of .venv/bin/python). It seems to do some installing automatically - including three strange things:
$ uv venv
$ uv pip install -e .
$ uv pip install 'pytest==7.0.*'
$ uv run pytest
warning: `uv run` is experimental and may change without warning.
Resolved 5 packages in 8ms
   Built pytest-github-actions-annotate-failures @ file:///Users/henryschreiner/git/software/pytest-github-actions-annotate-failures                                                                    Downloaded 1 package in 1.25s
Installed 2 packages in 33ms
 - pytest==7.0.1
 + pytest==8.2.0
 - pytest-github-actions-annotate-failures==0.2.0 (from file:///Users/henryschreiner/git/software/pytest-github-actions-annotate-failures)
 + pytest-github-actions-annotate-failures==0.2.0 (from file:///Users/henryschreiner/git/software/pytest-github-actions-annotate-failures)
... pytest output ...

At this point, I hadn't noticed it had forcibly upgraded the pytest I was testing to the latest version, so I thought 7.0.x worked. (Some else in parallel found it was 7.4 that broke us).
Then the next line caught me by surprise, it uninstalls two pytests:
$ uv pip install 'pytest==6.*'
Resolved 7 packages in 145ms
Downloaded 2 packages in 68ms
Installed 2 packages in 6ms
 - pytest==7.0.1
 - pytest==8.2.0
 + pytest==6.2.5
 + toml==0.10.2

And finally, the final uv run actually works just fine, and doesn't upgrade pytest:
$ uv run pytest
... pytest output ...

Thanks. To be clear, I wouldn't recommend uv run for "real" use yet but I do appreciate bug reports. Are you able to reproduce the double-uninstall reliably?

Same sequence does do it again:
$ uv venv
$ uv pip install -e .
$ uv pip install 'pytest==7.0.*'
$ uv run pytest
$ uv pip list
Package                                 Version
--------------------------------------- -------
attrs                                   23.2.0
iniconfig                               2.0.0
packaging                               24.0
pluggy                                  1.5.0
py                                      1.11.0
pytest                                  7.0.1
pytest                                  8.2.0
pytest-github-actions-annotate-failures 0.2.0
tomli                                   2.0.1

Thanks!

@charliermarsh this looks like it might be because we use EmptySitePackages in resolve instead of passing the site_packages we grabbed earlier.

Thanks @zanieb.

To be clear, I wouldn't recommend uv run for "real" use yet

If it were finished, where would the fun be in me using it? ;)


DEPS.xwalk: Roll v8-crosswalk(f08c04f->108d89f)

Fix debug build failure in runtime about XWALK-3517 and XWALK-3780

thanks!


DROOLS-1399 Comparison of DRL changes should not consider spaces duri…

…ng incremental compilation

	Test Result (2 failures / +2)
org.drools.compiler.integrationtests.KieContainerTest.testIncrementalCompilationSynchronization
org.drools.compiler.integrationtests.TimerAndCalendarTest.testHaltAfterSomeTimeThenRestartButNoLongerHolding

Logged to 8ball-report.
Jenkins, test again :8ball:

@tarilabs the overall idea seems fine, just fix the handling of single quoted strings and it should be good.

looks good now. +1


Modified unique constraint in user_poi_note schema so that only 1 not…

…e can exists for each user + POI combo

 Database schema modification


Pantheon: Backport fixes for Epiphany 44 and a crash fix

Description of changes
When granite is presented, new version of libadwaita no longer handles the style, this makes epiphany 44 looks very broken. We are likely going to land 44 faster than elementary I think. This PR picks up the fixes.
Also backport a crash fix (another one!) for elementary-mail.

Which tab are we at right now?

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

@ofborg build pantheon.elementary-gtk-theme pantheon.elementary-mail
@ofborg test pantheon


⚠ BCR Bot has degraded performance

In 5f79536, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 866 ms

Resolved: BCR Bot performance has improved in edcece4.


Get Started with SAP API Management

Group URL: https://developers.sap.com/group.cp-apim-code-1.html
Hello Team, I am unable to complete the mission "https://developers.sap.com/group.cp-apim-code-1.html", for the last tutorial i.e., "Consume the API Proxy in SAP Web IDE", because of 2 reasons:

Unable to set the destination "Web_IDE_to_APIM_Designtime", because in the trial cloud foundry account, we do not have Authentication type as ApptoAppSSO. If I want try in Neo account, it is not even available for us in trial version.
We do not have WebIDE Fullstack now for Cloud foundry, we just have Business Application Studio. I could not find anywhere about how to setup to connect API Proxy with Business Application Studio.

Can you please recheck this Tutorial and help complete the mission? As a next step, I would like to try to consume Business Rules API as well inside this API Management. If you have any information about that, please share it too.
Thanks in advance

do we have any update please?

looks like there is no use of commenting anything here

don't get discouraged, the more specific your issue is the more chances you have it will be addressed. give the assignee some time as they may not have all the information that you are after. i think a lot of Neo services are in the process of being rewritten for the cloud foundry, something that has started about a year ago, but obviously has not been fully transitioned yet.

Dear All,
Kindly give the engineering teams some time to work on the backlog of integrating SAP WEB IDE now know as Business Application Studio to APIM. This is the transition for the services too when it comes from moving from NEO to CF. The integration points are being worked upon therefore this tutorial is decommissioned as of today. I hope this helps! We will enable it as soon as the integration is developed and tested.
I hope this helps!

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Nuget updates

Summary of the Pull Request
updated sql lite, oledb, mahapps, contract sdk, WIL and UWP.  This shoudl also help reduce our file size on CI too

References

PR Checklist

[x] Applies to #3041
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

There's a WinUI update (2.4 something) that will resolve some NumberBox (visual) bugs.


Implement Placeholder Text for MdInput as per material.io guideline

Bug, feature request, or proposal:
Feature Request
What is the expected behavior?
Able to add hint / helper text alongside label
What is the current behavior?
It is not possible to have a label AND helper text (or actual input placeholder) together.

What are the steps to reproduce?
API for this feature does not currently exist
What is the use-case or motivation for changing an existing behavior?
Follow official material guideline
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
All
Is there anything else we should know?

closing as duplicate of #6194


[W5][T12-1] Zhang Shaoqian

Pretty printing in the help command
Before, the help command prints everything without proper indentation so it looks very messy. Now indentations are added for the "parameter" and "example" lines.

Useful changes. The printing is more clear now. The coding style is also coherent with the rest of the files.

It looks really nice now! Do you think it might be useful to have a function which takes in a number and returns the number of indentations accordingly?

Good enhancement! This will help in printing clearly. Maybe we can use this to create user profile later on and print the user details in a proper profile number.


Add deployment job to CI

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Blocked by https://github.com/CosmWasm/cosmwasm-plus/issues/30

Also blocked by https://github.com/CosmWasm/rust-optimizer/issues/18

I closed #30 and https://github.com/CosmWasm/rust-optimizer/pull/19 is ready for review (to fix the build errors)

I updated this to rust-optimizer:0.10.0. Let's see if it works now

Note cw1-whitelist is super small, this is due to the fact the "library" feature is activated from the cw1-subkeys import, which affects the normal cw1_whitelist.wasm build as well. Since we don't use this contract, I can live with it, but time to revisit the build process a bit

Oops, I need to add a github token here. I will add it and try re-running the build


Expose the encapsulated jsrsasign libraries

To mitigate the open-redirect security flaw that sends ID tokens on the location's hash to other domains, I need to implement Request Signing in the browser. This involves generating temporary keypairs and signing tokens in the request, using the following jsrasign library calls:
KEYUTIL.generateKeypair(...)
KJUR.jws.JWS.sign(...)

Unfortunately, the oidc-client embeds and encapsulates these libraries. This means that on top of the 360KB of oidc-client.min.js, this requires another 253KB of jsrsasign-all-min.js, which is ridiculous. Can the jsrsasign libraries please be exposed?

To mitigate the open-redirect security flaw that sends ID tokens on the location's hash to other domains,

I'm not sure what this means.

Can the jsrsasign libraries please be exposed?

Perhaps. I can have a look.

I know this is stale and I never got to it, but I think webcrypto should be used instead these days. Sorry.

I know this is stale and I never got to it, but I think webcrypto should be used instead these days. Sorry.


Change descrepencies in DX8 sound upgrade

See r70.  apparently there is a cut and paste error, go figure.
Imported from trac ticket #57,  created by Imago on 01-13-2010 at 04:08, last modified: 01-09-2011 at 19:57

also see #52
Trac comment by Imago on 01-13-2010 at 07:09

it's long gone
Trac comment by Imago on 07-05-2010 at 07:20

Milestone R5.1 deleted
Trac comment by pkk on 01-09-2011 at 19:57


[Flake] generate message boundary benchmark timeout

//google/cloud/storage:internal_generate_message_boundary_benchmark     TIMEOUT in 302.2s
https://source.cloud.google.com/results/invocations/427d799d-60ec-47dc-9810-5af5684860d9

This benchmark has joined the choir eternal.


made websocket client use java.net instead of java.nio

These changes should help to resolve:
#88, #155, #167, #175, #177, #187
I tested briefly on my local machine but before merging further testing has to be done.

The default java proxy's can now be used
ssl should no longer make trouble on android
the code is simpler

Hi, the changes in master allowed us to do WSS through a http-proxy with Connect. Sample code:
	public static void main( String[] args ) throws Exception {
		URI serverUri = new URI( "wss://ourapp.herokuapp.com:443/ws1" );
		WebSocketChatClient chatclient = new WebSocketChatClient( serverUri );

		Proxy proxy = new Proxy( Proxy.Type.HTTP, new InetSocketAddress( "proxy.corporate.com", 2128) );

		Socket proxySocket = new Socket(proxy);
		String host = serverUri.getHost();
		int port = serverUri.getPort();
		proxySocket.connect(new InetSocketAddress(host, port));

		SSLContext sslContext = null;
		sslContext = SSLContext.getInstance( "TLS" );
		sslContext.init( null, null, null );

		SSLSocketFactory factory = sslContext.getSocketFactory();
		chatclient.setSocket( factory.createSocket(proxySocket, host, port, true) );
		chatclient.connectBlocking();

		BufferedReader reader = new BufferedReader( new InputStreamReader( System.in ) );
		while ( true ) {
			String line = reader.readLine();
			if( line.equals( "close" ) ) {
				chatclient.close();
			} else {
				chatclient.send("{\"msg\":\"" + line + "\"}");
			}
		}
	}

This wasn't possible in the previous 1.3.0 release. Would you consider a new release incorporating these changes?

RE: @shuckc's comment, anyone know if it works for Android or just in (desktop) Java? For Java, there seems to be a platform limitation that was fixed a while back (in Java 8?). But not aware of anything similar for Android, there's no mention of the issue or whether there is or isn't HTTP proxy support for sockets in Android. Just asking as it seems we've encountered the mentioned Java limitation error using this library with HTTP proxy specified on Android.


Android/Windows: cannot find official tabs

cannot find Radiohead Creep official tab in Windows app (and Android)

but can in browser

Official scores aren't supported at this time.

ok, tnx

just for the info I dont have pro+ subscription (I did have)
but I'm still able to download pdf, midi, mp3
but "Individual parts" and mscz fails with ipfs error

additional question:
Do You think it's in principle possible to implement download of

mscz
individual midi parts (that's what I actually would like to have)
without having subscription?

First off, you are using an outdated version of the script. We have a new repo at https://github.com/LibreScore/dl-musescore.

Not yet, for reasons we cannot explain yet
No, because the website requires user-specific authentication to download it for official scores, if you have that authentication you can use the same url that we use in the script.


Add option to limit number of items stored in ephemeral driver.

This makes it possible for the ephemeral driver to use bounded amounts of memory.
It is useful if you want to "cache the cache" in an Ephemeral/Other composite stack, but limit memory consumption, or if you want to use the Ephemeral driver directly in scenarios where the number of unique Items is occasionally very large.

Coverage decreased (-0.07%) to 95.394% when pulling b326caa48a323c87ecd372989ed2ea49b220d656 on mbaynton:EphemeralMaxItems into df07131d3cf016796ee9fa91449d2addb594dc3b on tedious:master.


SignalR2Kusto

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
Example: I am trying to do [...] but [...]

I am trying to log messages for analytics to Kusto (Azure Data Explorer/Fabric KQL Database) but there's no native connector.

Describe the solution you'd like
A clear and concise description of what you want to happen. Include any alternative solutions you've considered.

Ideally a low-code way for WebPubService to register a client app, auth and stream messages to a destination.

Alternatively, I've created an azure function webjob that leverages SignalR Client SDK and Kusto bindings to run continuously on a timer. Pls see work-in-progress sample: https://github.com/hfleitas/signalrtokusto

Additional context
Add any other context or screenshots about the feature request here.

Consider the following architecture but my Data Store is Kusto for analytics instead of Azure Tables.

Currently we suggest using Azure Function output binding for WebPubSub or SignalR to do this.


Git extension shell

Does this issue occur when all extensions are disabled?: No

Version: 1.90.2
Commit: 5437499feb04f7a586f677b155b039bc2b3669eb
Date: 2024-06-18T22:54:35.075Z
Electron: 29.4.0
ElectronBuildId: 9728852
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Darwin arm64 23.3.0
Steps to Reproduce:

add echo $SHELL && exit 1 in git pre-commit hook
After commiting in GUI i can see in output that bash shell is used

How can I change it to zsh so that all git commands by extension will run in zsh?

same question on SO https://stackoverflow.com/questions/78204185/how-to-configure-the-shell-that-vscode-extensions-use

Sorry for not getting back to you on this until now. When you perform a source control operation VS Could would spawn a child process and execute git.exe. This child process "inherits" the environment variables from VS Code and in turn VS Code inherits the environment variables from where it was launched. What is the reason for using zsh for running the git commands? Is it merely the environment variables?

I wanna use zsh to run scripts and get env vars from .zshrc

Could you try opening a Terminal that uses zsh and launch code from there? Thanks!


OnTap of GestureDetector is not working in OverflowBox

I've just wanted to put a big map in an OverflowBox, but it seems like you can't detect touches in an Overflow-Box?
I've tried the example on Android & Web
class _MyHomePageState extends State<MyHomePage> {
  Province _pressedProvince;

  double mx = 0;
  double my = 0;
  double zoom = 0.5;
  @override
  Widget build(BuildContext context) {
    /// Calculate the center point of the SVG map,
    /// use parent widget for width/heigth.

    return Scaffold(
      appBar: AppBar(title: Text('Clickable SVG map of The WORLD')),
      body: SafeArea(
          child: Container(
        color: Colors.purple,
        child: GestureDetector(
          behavior: HitTestBehavior.translucent,
          onVerticalDragUpdate: (details) {
            mx += details.delta.dx;
            my += details.delta.dy;
            print("Y $my");
            setState(() {});
          },
          child: GestureDetector(
            behavior: HitTestBehavior.translucent,
            onScaleUpdate: (details) {
              zoom *= details.scale;
            },
            child: GestureDetector(
              behavior: HitTestBehavior.translucent,
              onHorizontalDragUpdate: (details) {
                mx += details.delta.dx;
                my += details.delta.dy;
                print("X $mx");
                setState(() {});
              },
              child: Transform.translate(
                offset: Offset(mx, my),
                child: Transform.scale(
                    alignment: FractionalOffset.center,
                    scale: zoom,
                    child: Container(
                      width: 2000,
                      height: 1000,
                      child: OverflowBox(
                        alignment: Alignment.topLeft,
                        minWidth: 2000,
                        minHeight: 1000,
                        maxWidth: 2000,
                        maxHeight: 1000,
                        child: Container(
                            height: 1000,
                            width: 2000,
                            child: FlatButton(
                              child: Text("TEST"),
                              color: Colors.green,
                              onPressed: () {},
                            )),
                      ),
                    )),
              ),
            ),
          ),
        ),
      )),
    );
  }

In this Code I've only got a small box in which Touch-Events can occure. I've created a Picture which shows the problem:

The Dart-Debug-Paint shows a small "OverflowBox"-Rectangle in which the touches work, but they should work everywhere - since the overflowbox is as big as the button in it. Or not?

There's an easy fix - swap the Overflow-Box with the Transformations - but I still don't get why the problem exists.

Tried with current master 1.20.0-8.0.pre.57.
I can see opposite situation - taps are detected out of the small OverflowBox (if I add onTap: () { print('tap'); }, to GestureDetector), but not inside

For performance reasons a widget (or to be more precise the backing render object) can only detect hits within the bounds of its parent. The OverflowBox allows it's child to draw outside of the bounds of the OverflowBox, but hit tests can only be detected within its bounds. This is working as intended.


[PASS] Add VerifyMemory pass and test cases (#410)

This PR includes VerifyMemory pass to check if memory accesses are legal, along with some test cases.
As described in issue #410, in the case that tgt is cuda, if workload is not bind with threads, CPU code is generated that tries to access GPU memory, which is illegal.
This pass performs verification by inserting AssertStmt before ProducerConsumer, where memory access occurs outside of thread env (i.e. CPU code). The AssertStmt checks if device_type is CPU. If not, then CPU code is detected to access GPU memory, and a fatal is thrown.

Good work!


Added services to start and stop the camera

Please ensure the above guidelines for contributing are met.
Fixes Issue: #85
Changes proposed in this pull request:

Added service "camera/set_power < true/false > " to turn camera on/off.
Added service "camera/force_power < true/false > " to turn camera on/off regardless of subscribers.
Added service "camera/is_powered" to check whether camera is powered on or off.
Added unit tests for the newly added services
Updated readme

@mdhorn @reaganlo updated code based on review comments.

👍  Approved

Approved


Sticker support

Message objects can have an array of stickers associated with them.
This will require a new class (Sticker) and an additional field (#stickers) on Message.

Do we use #stickers or #sticker_items (following the api)? I'm contemplating adding this so I can remove those pesky stickers more easily :)


devenv up fails with error

Describe the bug
When I try executing devenv up I get the following error.
It should start a postgres and redis process.
To reproduce
https://gist.github.com/MatzHilven/4201608fc91df95d2a852e3c6d386bb7
• Building processes ...
• Using Cachix: devenv
✔ Building processes in 1.2s.
• Starting processes ...• Building shell ...
✔ Building shell in 1.0s.
✨ devenv 1.0.2 is out of date. Please update to 1.0.3: https://devenv.sh/getting-started/#installation
panic: exec: "infocmp": executable file not found in $PATH

goroutine 13 [running]:
github.com/f1bonacc1/process-compose/src/tui.SetupTui({0xe5d8f0?, 0xc000662320?}, {0xc0000a1f80, 0x2, 0x0?})
        github.com/f1bonacc1/process-compose/src/tui/view.go:482 +0x156
github.com/f1bonacc1/process-compose/src/cmd.startTui({0xe5d8f0, 0xc000662320})
        github.com/f1bonacc1/process-compose/src/cmd/project_runner.go:92 +0x179
created by github.com/f1bonacc1/process-compose/src/cmd.runTui in goroutine 1
        github.com/f1bonacc1/process-compose/src/cmd/project_runner.go:76 +0x65

Version
devenv 1.0.2 (x86_64-linux)

Does it help if you add pkgs.ncurses to `packages?

Does it help if you add pkgs.ncurses to `packages?
Getting the same error

  packages = [
    pkgs.openssl
    pkgs.libyaml
    pkgs.ncurses
];

I installed ncurses globally and it's now returning the following error:
panic: exit status 1

goroutine 37 [running]:
github.com/f1bonacc1/process-compose/src/tui.SetupTui({0xe5d8f0?, 0xc0004055e0?}, {0xc00026ff80, 0x2, 0x0?})
        github.com/f1bonacc1/process-compose/src/tui/view.go:482 +0x156
github.com/f1bonacc1/process-compose/src/cmd.startTui({0xe5d8f0, 0xc0004055e0})
        github.com/f1bonacc1/process-compose/src/cmd/project_runner.go:92 +0x179
created by github.com/f1bonacc1/process-compose/src/cmd.runTui in goroutine 1
        github.com/f1bonacc1/process-compose/src/cmd/project_runner.go:76 +0x65

can you paste devenv info

`❯ devenv info
env

ADVISORY_LOCK: false
BUNDLE_PATH: /home/matz/Documents/Coding/Ruby/Backend-devenv/.devenv/state/.bundle
DEVENV_DOTFILE: /home/matz/Documents/Coding/Ruby/Backend-devenv/.devenv
DEVENV_PROFILE: /nix/store/985rwc0igk9gqlc19wng8annywahvyfb-devenv-profile
DEVENV_ROOT: /home/matz/Documents/Coding/Ruby/Backend-devenv
DEVENV_RUNTIME: /tmp/devenv-1721d98
DEVENV_STATE: /home/matz/Documents/Coding/Ruby/Backend-devenv/.devenv/state
GEM_HOME: /home/matz/Documents/Coding/Ruby/Backend-devenv/.devenv/state/.bundle/ruby/3.3.0
PGDATA: /home/matz/Documents/Coding/Ruby/Backend-devenv/.devenv/state/postgres
PGHOST: /tmp/devenv-1721d98/postgres
PGPORT: 5432
PREPARED_STATEMENTS: false
REDISDATA: /home/matz/Documents/Coding/Ruby/Backend-devenv/.devenv/state/redis
REDIS_URL: redis://<IP_ADDRESS>:6379

packages

process-compose-0.88.0
redis-7.2.4
postgresql-15.6
start-postgres
bundler-2.5.6
ruby-3.3.0
stdenv-linux
gnumake-4.4.1
clang-wrapper-17.0.6
ccls-0.20230717
gcc-wrapper-13.2.0
pkg-config-wrapper-0.29.2
openssl-3.0.13
libyaml-0.2.5
pkg-config-wrapper-0.29.2

processes

postgres: exec /nix/store/ayqjps0b2abvvz5v293fi3i8baqf0wmy-postgres
redis: exec /nix/store/rzxalzqsgjs65k1qxqhbs4qyy1w2vqjy-redis
`

Please try to define TERM:
export TERM=xterm-256color
And run again.

Please try to define TERM: export TERM=xterm-256color And run again.
Same error, tried exporting & setting via devenv ( env.TERM="xterm-256color";)

Hi, I've been having the same issue. It used to work fine with the older v0.6.3 version of devenv.
I believe this is something to do with the generation of the config file and a regression with the new detach option. Before devenv up in a non-interactive context would work fine, but seems like either that or the automatic detection of the tty is causing issues.

I've seen this in our tests as well, if we nail it down I'm happy to fix it!

@MatzHilven, I noticed you have clean: enabled: true in your devenv.yaml.  Does the issue persist if you disable that? Or if you set clean: keep: [ "PATH", "TERM"]?

@sandydoo Nope, I've never had that key set.
Also, I do not seem to be facing this issue anymore since the v1.0.4 release, I do believe there was some modification to the process compose argument handling. I did have to set the following keys to get it to work. Not setting the unix-socket causes an error too (which maybe should be another issue)
  env.PC_DISABLE_TUI = "1";
  env.PC_TUI_ENABLED = "0";
  process.process-compose.tui = false;
  process.process-compose.unix-socket = "/tmp/process-compose.sock";
  process.process-compose.port = 9999;

@MatzHilven, I noticed you have clean: enabled: true in your devenv.yaml. Does the issue persist if you disable that? Or if you set clean: keep: [ "PATH", "TERM"]?
@autolisis, do happen to use the clean option in your devenv.yaml by any chance?

It seems like the clean option was te culprit for me! After removing it, my postgres & redis server work fine

I went through the latest tests and it seems like it's not happening anymore.

We'll need to modify the docs to explain the why the clean option is probably not what you want. It currently removes all env variables, which is an easy way bork the shell. Ideally it would be smart about the vars it removes, but this is tricky to get right.


@conda-forge-admin, please add user @xylar

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @xylar in conda-forge/google-cloud-texttospeech-feedstock#23.


Usage of the CurrReport.Skip method in report extensions

1. Describe the bug
The usage of the CurrReport.Skip method in the OnBeforeAfterGetRecord trigger doesn't influence on the execution of the OnAfterGetRecord trigger of the corresponding data item.
Could you please check whether it is expected behavior (limitation for the report extensions objects) or not (it is a bug)?
2. To Reproduce
Steps to reproduce the behavior:

Create a new app with the following report extension:

reportextension 50100 "TEST Get Item Ledger Entries" extends "Get Item Ledger Entries"
{
    dataset
    {
        modify("Item Ledger Entry")
        {
            trigger OnBeforeAfterGetRecord()
            begin
                CurrReport.Skip();
            end;
        }
    }
}

Add the following breakpoints:

in the first line of the OnBeforeAfterGetRecord trigger (in the report extension)
in the first line of the OnAfterGetRecord trigger of the "Item Ledger Entry" data item (in the original report)

Publish and debug an app.
Run report 594 (Get Item Ledger Entries).

3. Expected behavior
The OnAfterGetRecord trigger of the "Item Ledger Entry" data item is not executed.
4. Actual behavior
The OnAfterGetRecord trigger of the "Item Ledger Entry" data item is executed.
5. Versions:

AL Language: v9.0.615906
Visual Studio Code: 1.67.1
Business Central: 20.1 BE
List of Visual Studio Code extensions that you have installed: unrelated

#6949

Do you have any new information on this?

This is currently out of scope for this github repository, so we're not actively looking at this. If it's urgent or blocking, I suggest contacting customer support through partner source.

Still facing the same on BC23.0
Created incident report<PHONE_NUMBER>002985

Update from MS support team:

... they agreed to create a bug, we created a bug (495100), which you shall track in upcoming release.

@fvet do you know if this is fixed in BC24?

I've looked up the current status of the bug internally, and it's been closed as By Design. Documentation has been updated around this.
The reason why this is By Design is that many reports bind to Integer table, and then have trigger logic in OnAfterGetRecord to advance a temporary record/query/etc.. These reports would break if the OnAfterGetRecord  was skipped. The skip does correctly make the data not be included in the final dataset even if the trigger is still called.

Thanks for the detailed answer.
This makes it more complicated to deal with Report Extensions, but at least it's documented now.
We have some copied reports where we don't want to print some lines in a report an therefore don't want to include those lines in the total amount.
If we were able to skip the On AfterGetRecord Code with this then we could easily move from copied reports to report extensions.
The workaround would be to substract the values from the line on the OnAfterOnAfterGetRecord.
Furthermore the value of the variable FirstValueHasBeenOutput would be wrong if the trigger is executed, but the line is ommitted in the output.

One solution would be to introduce a skipTriggers param for CurrReport.Skip()  so it's like CurrReport.Skip(true) where the default value is false.  That would be a new feature request, but if it's a blocker for deduplicating reports and moving customizations to extensions, then I think that's a reasonable argument.
Raise it on BCIdeas?

I've created an idea.
https://experience.dynamics.com/ideas/idea/?ideaid=6552bd01-b653-ef11-b4ac-0022484d1652
I've search again in the documentation for a note on the current behaviour but I couldn't find it.

It's been added as a note to: https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/methods-auto/report/reportinstance-skip-method#remarks


Add build tool for Fletcher OPAE projects

Add a build tool that provides a simple interface to create, build, simulate and synthesize Fletcher OPAE projects.

The tools provided by OPAE work great. No need to build something separate in the context of this platform.


change locking order to avoid deadlock

too heavy-handed?

I am pretty worried about this, since it spreads state and logic to more places instead of just removing the call-outs being done from GalvanStateInterlock (since it is bogus that we have that making external calls, anyway, when it is just a state interlock) but we can proceed if it addresses the problem and seems stable.
Worst case scenario, we can re-approach this in a different way.


add new dockerfile for terraform

Adds a docker file for terraform.

very similar to the consul docker file with relevant updates
uses latest version of alpine
builds latest version of terraform

Builds and runs without issues.

Why build from src?
Can I suggest something more like:
https://github.com/Daviey/docker-terraform/blob/master/Dockerfile
Ta

I followed the same approach as with the current consul image. @Daviey

thanks!


Support wasm32-unknown-unknown

Now I see only support for emscripten, is there any plan for supporting target wasm32-unknown-unknown?
https://github.com/alexcrichton/cc-rs/blob/9e57a2b0e922a96be5d28b1adf6e5fc41d5d762c/src/lib.rs#L1696

Ideally we would support this! The target, however, doesn't really have a C compiler because there's no real standard "sysroot" (currently) to actually compile C code against. The recently released clang 8.0 compiler though is the only one we'd also be compatible with on wasm32-unknown-unknown, so we could at least add that and rely on users to set the right CFLAGS to compile with the right headers/sysroot/etc

I'll be happy to use clang 8 for it.

I'm facing a similar problem here https://github.com/servo/rust-stb-image/issues/94 but I have no clue where the wasm32 sysroot lives. I'm on CLANG 8.0.0 so I suppose it actually supports compiling to wasm32 but without the correct sysroot it can't find stdio.h and such headers.


Update authors list

including participants to June 2018 meeting?

Lets merge the set of people at the two workshops, and write something at top about this being an effort over two workshops with these participants.

This is solved by #43, if someone would like to merge.


[Improvement-16647][UI][Workflow Instance]remove deprecated workflow execution states

Purpose of the pull request
close #16647

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
Pull Request Notice
Pull Request Notice
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

To to target specific issue from PR, we recommend use keyword fix: #issue_id or close: #issue_id or closes: #issue_id in your PR describe(not title, just desc). It would not only connect issue to PR but also close issue automatically when PR is be closed.

To to target specific issue from PR, we recommend use keyword fix: #issue_id or close: #issue_id or closes: #issue_id in your PR describe(not title, just desc). It would not only connect issue to PR but also close issue automatically when PR is be closed.

Like this?

Like this?

Yes.


[SIMPLE][NOP] use underscore for markdown horizontal separators

@dmitrig01
We use * * *, --- and ___ all over the place. This PR consolidate them under one ring.

:+1:

Oh forgot to mention that there is no visible diff.


Light client: trait impls

This PR will have implementations of the traits created in #31 and a few tests.

Note that @yihuang added and impl for lite::Commit which I've merged (and then slightly modified) in #63.

@ebuchman Can we merge this soon?

@Liamsi I'm unfortunately out of commission this week so will leave it to you. Thanks @yihuang for your contributions, much appreciated!!

OK, thanks. I'll merge in a bunch of (reviewed) changes from #63 into lite_impl. Will add more tests and if everything looks OK, merge all this into your branch bucky/lite. Think we should still do another review on bucky/lite -> master. Then we have a proper (skipping) light client to use.

Actually, no. Will merge your intial traits PR #31 to master and base the follow-up PRs (this one #36 and #63) to master too.

Thanks for the review @brapse and @ancazamfir! I would also like to merge this as is. A few tests are in a follow-up PR #63 which I didn't merge in here as this PR would grow quite large.


GH1292: Roundhouse Improvements

Made changes as discussed in the ticket.

@gep13 Thanks for changing the title, something I was going to do this morning.

LGTM!

Relates to #1292

@phillipsj I have just rebased your branch against the latest develop.  Assuming all the CI's pass, I will merge this into develop.
Thanks!


about dynamic shape

This is a great project. I have some questions about the "dynamic shape" in the code. I refer to your relevant code and modify it in the original "depth-anything-v2" to export the onnx model, but I get the depth output is always fixed (1,140,140). Why is this? Hope to get your reply, thank you.

Hi @njfugit, thank you for your interest in Depth-Anything-ONNX.
I'm not sure what modifications you made, but the 140 placeholder probably comes from this line:
https://github.com/fabio-sim/Depth-Anything-ONNX/blob/9abf814d22ac06f50fa96c9c632c30430be8a828/dynamo.py#L145
If you pass -b 0 -h 0 -w 0 to the script, it should apply dynamic shapes with these lines:
https://github.com/fabio-sim/Depth-Anything-ONNX/blob/9abf814d22ac06f50fa96c9c632c30430be8a828/dynamo.py#L136-L142

Hi @njfugit, thank you for your interest in Depth-Anything-ONNX.
I'm not sure what modifications you made, but the 140 placeholder probably comes from this line:
https://github.com/fabio-sim/Depth-Anything-ONNX/blob/9abf814d22ac06f50fa96c9c632c30430be8a828/dynamo.py#L145
If you pass -b 0 -h 0 -w 0 to the script, it should apply dynamic shapes with these lines:
https://github.com/fabio-sim/Depth-Anything-ONNX/blob/9abf814d22ac06f50fa96c9c632c30430be8a828/dynamo.py#L136-L142

Thanks for your reply, I understand now. The 140x140 here is just an artificially set resolution.


hasOne() hasMany() relation not working with Dbref column

Laravel-mongodb Version: 4.4
PHP Version: 8.1
Database Driver & Version: 1.19

That's exact. Laravel and Eloquent is an opinionated framework: the relations use a single id field because that's how it's implemented with SQL databases. We can't change this behavior while extending the Eloquent classes.


Inaccurate statement regarding links in codepen.io in build-a-tribute-page video

Right around 3:00, Quincy says that codepen.io won't follow links without the target attribute equaling _blank, but in my current codepen (http://codepen.io/tmowad/full/WwOZwJ/) this doesn't seem to be required.
No biggie, but something to change, if its universal in all browsers, if you re-record the video.
I have Chrome Version 49.0.2623.87 (64-bit) -- on Mac OS X Version 10.11.4 (15E65), in case this matters.

In your code pen your link to the biography does indeed work. But for me in the chrome browser, your link to your FCC profile does not load. What Quincy is advising is using the target="_blank" method to a get around this issue.

Maybe we can have a video or lesson on this, to make it more clear.

I think you only need the target="_blank" to work around sites that don't allow iframing.

@tmowad @jbull328
https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/Known-Issues-With-Codepen

Closing this issue. Feel free to reopen if this is still an issue for you, thanks.


feat(travis): switch to codecov for code coverage

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6cee262). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1165   +/-   ##
=========================================
  Coverage          ?   69.68%
=========================================
  Files             ?       29
  Lines             ?     1039
  Branches          ?        0
=========================================
  Hits              ?      724
  Misses            ?      315
  Partials          ?        0

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 6cee262...5b0a5a2. Read the comment docs.

@patrick96 much more informative than coveralls IMO. I'm probably gonna turn off the PR comments.

Looks good. Do you think we should have the test code be part of the coverage?


ec2-net-utils SRPM depends on upstart

AWS publishes the ec2-net-utils SRPM. Will need to massage it to make it systemd-compatible.

Per note from @lorengordon, check out ec2-utils

Appears to have been obsoleted by upstream maintainers


#113 Add saves file to list of local saves on clients as well.

Could you please add cloud / local preference support? (SaveLoader.GetCloudSavesDefault)

Could you please add cloud / local preference support? (SaveLoader.GetCloudSavesDefault)

Done


[BEAM-14143] Renames ExternalPythonTransform to PythonExternalTransform

This makes the Java API more consistent with the naming of the corresponding Python API 'JavaExternalTransform':
https://github.com/apache/beam/blob/adfa113a6402b76571d746f8357879bd66fff4d7/sdks/python/apache_beam/transforms/external.py#L292

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @ihji
CC: @robertwb

Run Java PreCommit

Thanks.


2.5

Hey ,
So I broke my ripper going to docker. I guess so I figured I go to your latest but it seems running python3 main.py is giving the following. Am I missing something? Seem like Tom is up and running.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/opt/arm/arm/ripper/main.py", line 516, in
utils.check_db_version(cfg['INSTALLPATH'], cfg['DBFILE'])
File "/opt/arm/arm/ripper/utils.py", line 1136, in check_db_version
flask_migrate.upgrade(mig_dir)
File "/usr/local/lib/python3.8/dist-packages/flask_migrate/init.py", line 96, in wrapped
f(*args, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/flask_migrate/init.py", line 271, in upgrade
command.upgrade(config, revision, sql=sql, tag=tag)
File "/usr/local/lib/python3.8/dist-packages/alembic/command.py", line 298, in upgrade
script.run_env()
File "/usr/local/lib/python3.8/dist-packages/alembic/script/base.py", line 489, in run_env
util.load_python_file(self.dir, "env.py")
File "/usr/local/lib/python3.8/dist-packages/alembic/util/pyfiles.py", line 98, in load_python_file
module = load_module_py(module_id, path)
File "/usr/local/lib/python3.8/dist-packages/alembic/util/compat.py", line 184, in load_module_py
spec.loader.exec_module(module)
File "", line 783, in exec_module
File "", line 219, in _call_with_frames_removed
File "/opt/arm/arm/migrations/env.py", line 88, in
run_migrations_online()
File "/opt/arm/arm/migrations/env.py", line 81, in run_migrations_online
context.run_migrations()
File "", line 8, in run_migrations
File "/usr/local/lib/python3.8/dist-packages/alembic/runtime/environment.py", line 846, in run_migrations
self.get_context().run_migrations(**kw)
File "/usr/local/lib/python3.8/dist-packages/alembic/runtime/migration.py", line 522, in run_migrations
step.migration_fn(**kw)
File "/opt/arm/arm/migrations/versions/9cae4aa05dd7_create_settingsui_table.py", line 19, in upgrade
op.add_column('job',
File "", line 8, in add_column
File "", line 3, in add_column
File "/usr/local/lib/python3.8/dist-packages/alembic/operations/ops.py", line 1929, in add_column
return operations.invoke(op)
File "/usr/local/lib/python3.8/dist-packages/alembic/operations/base.py", line 373, in invoke
return fn(self, operation)
File "/usr/local/lib/python3.8/dist-packages/alembic/operations/toimpl.py", line 132, in add_column
operations.impl.add_column(table_name, column, schema=schema, **kw)
File "/usr/local/lib/python3.8/dist-packages/alembic/ddl/impl.py", line 238, in add_column
self._exec(base.AddColumn(table_name, column, schema=schema))
File "/usr/local/lib/python3.8/dist-packages/alembic/ddl/impl.py", line 141, in _exec
return conn.execute(construct, *multiparams, **params)
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1011, in execute
return meth(self, multiparams, params)
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/sql/ddl.py", line 72, in _execute_on_connection
return connection._execute_ddl(self, multiparams, params)
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1068, in _execute_ddl
ret = self._execute_context(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1316, in _execute_context
self.handle_dbapi_exception(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1510, in handle_dbapi_exception
util.raise(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/util/compat.py", line 182, in raise
raise exception
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1276, in _execute_context
self.dialect.do_execute(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/default.py", line 593, in do_execute
cursor.execute(statement, parameters)
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) duplicate column name: path
[SQL: ALTER TABLE job ADD COLUMN path VARCHAR]
(Background on this error at: http://sqlalche.me/e/13/e3q8)
root@zero:/home/arm# python3 /opt/arm/arm/ripper/main.py -d sr2
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1276, in _execute_context
self.dialect.do_execute(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/default.py", line 593, in do_execute
cursor.execute(statement, parameters)
sqlite3.OperationalError: duplicate column name: path
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/opt/arm/arm/ripper/main.py", line 516, in
utils.check_db_version(cfg['INSTALLPATH'], cfg['DBFILE'])
File "/opt/arm/arm/ripper/utils.py", line 1136, in check_db_version
flask_migrate.upgrade(mig_dir)
File "/usr/local/lib/python3.8/dist-packages/flask_migrate/init.py", line 96, in wrapped
f(*args, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/flask_migrate/init.py", line 271, in upgrade
command.upgrade(config, revision, sql=sql, tag=tag)
File "/usr/local/lib/python3.8/dist-packages/alembic/command.py", line 298, in upgrade
script.run_env()
File "/usr/local/lib/python3.8/dist-packages/alembic/script/base.py", line 489, in run_env
util.load_python_file(self.dir, "env.py")
File "/usr/local/lib/python3.8/dist-packages/alembic/util/pyfiles.py", line 98, in load_python_file
module = load_module_py(module_id, path)
File "/usr/local/lib/python3.8/dist-packages/alembic/util/compat.py", line 184, in load_module_py
spec.loader.exec_module(module)
File "", line 783, in exec_module
File "", line 219, in _call_with_frames_removed
File "/opt/arm/arm/migrations/env.py", line 88, in
run_migrations_online()
File "/opt/arm/arm/migrations/env.py", line 81, in run_migrations_online
context.run_migrations()
File "", line 8, in run_migrations
File "/usr/local/lib/python3.8/dist-packages/alembic/runtime/environment.py", line 846, in run_migrations
self.get_context().run_migrations(**kw)
File "/usr/local/lib/python3.8/dist-packages/alembic/runtime/migration.py", line 522, in run_migrations
step.migration_fn(**kw)
File "/opt/arm/arm/migrations/versions/9cae4aa05dd7_create_settingsui_table.py", line 19, in upgrade
op.add_column('job',
File "", line 8, in add_column
File "", line 3, in add_column
File "/usr/local/lib/python3.8/dist-packages/alembic/operations/ops.py", line 1929, in add_column
return operations.invoke(op)
File "/usr/local/lib/python3.8/dist-packages/alembic/operations/base.py", line 373, in invoke
return fn(self, operation)
File "/usr/local/lib/python3.8/dist-packages/alembic/operations/toimpl.py", line 132, in add_column
operations.impl.add_column(table_name, column, schema=schema, **kw)
File "/usr/local/lib/python3.8/dist-packages/alembic/ddl/impl.py", line 238, in add_column
self._exec(base.AddColumn(table_name, column, schema=schema))
File "/usr/local/lib/python3.8/dist-packages/alembic/ddl/impl.py", line 141, in _exec
return conn.execute(construct, *multiparams, **params)
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1011, in execute
return meth(self, multiparams, params)
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/sql/ddl.py", line 72, in _execute_on_connection
return connection._execute_ddl(self, multiparams, params)
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1068, in _execute_ddl
ret = self._execute_context(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1316, in _execute_context
self.handle_dbapi_exception(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1510, in handle_dbapi_exception
util.raise(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/util/compat.py", line 182, in raise
raise exception
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1276, in _execute_context
self.dialect.do_execute(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/default.py", line 593, in do_execute
cursor.execute(statement, parameters)
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) duplicate column name: path
[SQL: ALTER TABLE job ADD COLUMN path VARCHAR]
(Background on this error at: http://sqlalche.me/e/13/e3q8)
root@zero:/home/arm# python3 /opt/arm/arm/ripper/main.py -d sr2
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1276, in _execute_context
self.dialect.do_execute(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/default.py", line 593, in do_execute
cursor.execute(statement, parameters)
sqlite3.OperationalError: duplicate column name: path
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/opt/arm/arm/ripper/main.py", line 516, in
utils.check_db_version(cfg['INSTALLPATH'], cfg['DBFILE'])
File "/opt/arm/arm/ripper/utils.py", line 1136, in check_db_version
flask_migrate.upgrade(mig_dir)
File "/usr/local/lib/python3.8/dist-packages/flask_migrate/init.py", line 96, in wrapped
f(*args, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/flask_migrate/init.py", line 271, in upgrade
command.upgrade(config, revision, sql=sql, tag=tag)
File "/usr/local/lib/python3.8/dist-packages/alembic/command.py", line 298, in upgrade
script.run_env()
File "/usr/local/lib/python3.8/dist-packages/alembic/script/base.py", line 489, in run_env
util.load_python_file(self.dir, "env.py")
File "/usr/local/lib/python3.8/dist-packages/alembic/util/pyfiles.py", line 98, in load_python_file
module = load_module_py(module_id, path)
File "/usr/local/lib/python3.8/dist-packages/alembic/util/compat.py", line 184, in load_module_py
spec.loader.exec_module(module)
File "", line 783, in exec_module
File "", line 219, in _call_with_frames_removed
File "/opt/arm/arm/migrations/env.py", line 88, in
run_migrations_online()
File "/opt/arm/arm/migrations/env.py", line 81, in run_migrations_online
context.run_migrations()
File "", line 8, in run_migrations
File "/usr/local/lib/python3.8/dist-packages/alembic/runtime/environment.py", line 846, in run_migrations
self.get_context().run_migrations(**kw)
File "/usr/local/lib/python3.8/dist-packages/alembic/runtime/migration.py", line 522, in run_migrations
step.migration_fn(**kw)
File "/opt/arm/arm/migrations/versions/9cae4aa05dd7_create_settingsui_table.py", line 19, in upgrade
op.add_column('job',
File "", line 8, in add_column
File "", line 3, in add_column
File "/usr/local/lib/python3.8/dist-packages/alembic/operations/ops.py", line 1929, in add_column
return operations.invoke(op)
File "/usr/local/lib/python3.8/dist-packages/alembic/operations/base.py", line 373, in invoke
return fn(self, operation)
File "/usr/local/lib/python3.8/dist-packages/alembic/operations/toimpl.py", line 132, in add_column
operations.impl.add_column(table_name, column, schema=schema, **kw)
File "/usr/local/lib/python3.8/dist-packages/alembic/ddl/impl.py", line 238, in add_column
self._exec(base.AddColumn(table_name, column, schema=schema))
File "/usr/local/lib/python3.8/dist-packages/alembic/ddl/impl.py", line 141, in _exec
return conn.execute(construct, *multiparams, **params)
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1011, in execute
return meth(self, multiparams, params)
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/sql/ddl.py", line 72, in _execute_on_connection
return connection._execute_ddl(self, multiparams, params)
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1068, in _execute_ddl
ret = self._execute_context(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1316, in _execute_context
self.handle_dbapi_exception(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1510, in handle_dbapi_exception
util.raise(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/util/compat.py", line 182, in raise
raise exception
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/base.py", line 1276, in _execute_context
self.dialect.do_execute(
File "/usr/local/lib/python3.8/dist-packages/sqlalchemy/engine/default.py", line 593, in do_execute
cursor.execute(statement, parameters)
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) duplicate column name: path

This isn't your fault, this is mine. Ive made changes to the database recently that aren't yet fully upgrade compatible.
The problem is the previous upgrade path added a new column 'path'  to the job table and added a new table 'ui_settings' but the update doesnt handle partial updates.
To solve this the easiest way is to delete the db file and start again, but if you want to keep previous jobs you will need to edit the database manually and remove the 'path' column and the ui_settings table and then re-run ARM

I'm having similar upgrade issues. I just want to know how to start with a new database. I have tried deleting the /home/arm/db directory, but it comes back and I get the same error.

without digging deeper it seems the culprit is this line
utils.py
On the surface it seems to be calling an upgrade function to create the new db?
That function is passed the migration directory, so it seems it assumes the lowest versino for the db.
I will keep looking through the code.

I fixed it. I had to edit arm/migrations/versions/6dfe7244b18e_.py and change the table name from job to config.

I fixed it. I had to edit arm/migrations/versions/6dfe7244b18e_.py and change the table name from job to config.
EDIT: my error was different than ops though.

Dont worry, op's error was the same type of issue (bad upgrade pathing)

sweet up and running well sorta but its burning thanks to everyone for your help

Glad its working. Thanks for all your helping testing out ARM!


Integrate SIGcomLincs API

[ ] Determine what data is accessible via API
[ ] Determine how API will be used within the shiny app.

View at https://maayanlab.cloud/sigcom-lincs/#/

@Poojas241 can we find a list of data types and post here?

API in shiny app and R
https://community.rstudio.com/t/best-practice-for-using-external-api-in-a-shiny-app/35018
https://www.r-bloggers.com/2020/05/fetch-api-results-from-the-browser-and-send-them-to-shiny/#:~:text=You can write API calls using the fetch,%3D "get_rand_beer ()")%2C as we will see below).

@MariaJose501 begin integration of this api and explore it.

For sigcom lincs:
We'll integrate:

[ ] Validation of genes
[ ] GTEx Tissue-Specific Age Group Signatures
[ ] LINCS L1000 Chemical Perturbations (2021)
[ ] LINCS L1000 Ligand Perturbations (2021)
[ ] LINCS L1000 Chemical Perturbations Consensus Signatures
[ ] SIgnature search by gene, disease


What is the correct setup to initialise CocoaLumberjack now for all versions of iOS up to iOS11

All the current documentation for CocoaLumberjack that i've found still recommends using the same loggers I've used since using it a few years ago, namely
DDLog.add(DDTTYLogger.sharedInstance) // TTY = Xcode console DDLog.add(DDASLLogger.sharedInstance) // ASL = Apple System Logs
I happened to notice in my current app that I was getting two logs output for every log I called, and did some searching to find the following discussions
https://github.com/CocoaLumberjack/CocoaLumberjack/issues/685
https://github.com/CocoaLumberjack/CocoaLumberjack/issues/765
https://github.com/CocoaLumberjack/CocoaLumberjack/issues/858
These are all quite old but had good advice that it was the two loggers that may cause issues, though there seems to be some debate as to what version of iOS you're using, whether its in the simulator etc. For what its worth I was seeing the double logging on the simulator in my case.
I've commented out the TTY logger and am left with just
DDLog.add(DDASLLogger.sharedInstance)
for now, which has fixed my double logging in the simulator targeting iOS11.
The last issue referenced above talks about ASL Depreciation warnings, which I've not noticed but i guess suggests I've gone for the wrong option. Beyond that within that issue someone recommends actually using

I recommend you use DDOSLogger instead of DDASLLogger with iOS 10+, macOS 10.12+, tvOS 10.0+, or watchOS 3.0+. You can use it from CocoaLumberjack 3.2.0.

Which sounds interesting again, though seemingly with caveats on versions of Cocoalumberjack and iOS.
I guess I'm looking for an authoritative answer to be put in the documentation, or at least some notes towards the fact that the current advice doesn't seem to be correct anymore. Perhaps the solution is complex but having a suggestion or guidance on the recommended approach for this would be great to see.
Basically, what is the right way to configure CocoaLumberjack right now?
Thanks for your time
Cheers

While still waiting for somebody authoritative, will try to help as much as I can.

DDOSLogger works around os_log available since iOS 10.0, which supersedes ASL (Apple System Logger) and the Syslog APIs (https://developer.apple.com/documentation/os/logging). Modern logging system (os_log-based) manages not only console logging, but storing logs in memory buffers, wiping them and so on (that's why logs were duplicated when using with TTY).

CocoaLumberjack's implementation is smth like: if @available(iOS 10.0, ...) os_log(), so it should be used since iOS 10.0 (it will do nothing on lower versions - no logging, no warnings about it).

DDASLLogger worked with ASL, using asl_... C-functions. Deprecated since 10.0 (as mentioned above).

DDTTYLogger logged to stderr using writev(STDERR_FILENO, ...). It was some sort of classic logging from the world of ANSI-C and one of its main "killer" features was console log coloring (after https://github.com/robbiehanson/XcodeColors plugin installed). For now - there is no more plugins since Xcode 8 and so there are problems with console coloring (see Issues for that library and similar ones, the only thing I found is attempt to use emoji instead of colors - https://github.com/hemangshah/printer).

Finally, I think you should do something like:
if @available(iOS 10.0, ...) {
  DDLog.add(DDOSLogger.sharedInstance)
} else {
  DDLog.add(DDTTYLogger.sharedInstance)
  DDLog.add(DDASLLogger.sharedInstance)
}

Hi guys and thanks for the interest you take in this project.
This issue was long due a response.
As @kpower said, the recommended way would be to just use the DDOSLogger which is available on all platforms.
We also need to update our documentation about this, since it's not clear anymore. I will leave this issue open until we get to it.

I think the current documentation is better about this subject (DDOSLogger vs DDTTYLogger + DDASLLogger). Closing. Feel free to comment / propose any other improvements to docs/examples

Based on this conversation and others I've ended up with the following initalisation code (objc). (My app supports iOS 9.3+)
// setup CocoaLumberjack loggers
    if (@available(iOS 10.0, *)) {
        [DDLog addLogger:[DDOSLogger sharedInstance]]; // Modern logging works on iOS 10+
    } else {
        [DDLog addLogger:[DDASLLogger sharedInstance]]; // ASL = Apple System Logs
#if !TARGET_IPHONE_SIMULATOR
        // When running on the simulator ASL logs are forwarded to the TTY logger however when run on a device they're not
        [DDLog addLogger:[DDTTYLogger sharedInstance]]; // TTY = Xcode console
#endif
    }


Document/clarify whether TextDocumentContentProviders can send onDidChange only for open files

It's not clear to me from the API docs whether a TextDocumentContentProviders can selectively send onDidChange only if the document is open. Eg. after didCloseTextDocument or if a document is not in workspace.textDocuments, can an extension/server stop sending these messages?
My feeling is that the answer is yes (because otherwise there could be a lot of unnecessary spam), but it relies on VS Code not caching the contents of any documents that are "closed" and I'd like to be certain of this (and that it is the intention, and not only the current behaviour) before relying on that.
The docs for this field are here:
https://github.com/microsoft/vscode/blob/545eb1dabec2ccca84500f47ebb740c4c7170d95/src/vscode-dts/vscode.d.ts#L1805

You don't need to care. You can send events any time and we should be doing the right thing

@jrieken I was trying to minimize the number of events being sent if they don't do anything. Eg., if a user opens then closes a generated file and it is changed, can I skip sending the event and know that if VS Code tries to re-open the document later, it will first fetch the latest content?
Otherwise it seems like we might end up sending a lot of unnecessary change events as the user is modifying files if their changes affect generated files.

@jrieken are you able to answer the above? I'm can send events on every change, but if they're unnecessary it's just more traffic and processing VS Code is doing as the user is typing. I also hope to add support for TextDocumentContentProviders to LSP, so being explicit on the rules would be better.


style(NavLinks): Improved NavLinks animation and scale

Summary
Improved NavLinks animation and scale
Before:

After:

Change Type

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

Checklist

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] I have commented in any complex areas of my code
[x] I have made pertinent documentation changes
[x] My changes do not introduce new warnings
[x] I have written tests demonstrating that my changes are effective or that my feature works
[x] Local unit tests pass with my changes
[x] Any changes dependent on mine have been merged and published in downstream modules.

it would be better if your after image showed the same picture/name, as I can't really tell the change

it would be better if your after image showed the same picture/name, as I can't really tell the change

Done!


Feat/onboarding touch ups

fix ios bottom padding - also with keyboard in view
make textfield underline white instead of gradient
on invalid. pin entry, smaller screens scroll down to feedback msg

Messed something up while rebasing. Will create new PR


Extract --pex-repository resolving from resolver.

The resolver module is huge and the code for resolution from a PEX had
nothing in common except the result type which is now extracted as well.

This is all pure moves.


Add hipMallocPitch and hipMemcpy2D

Solved issue #23

I'm not very familiar with catch2 framework currently. I would try to cover the test for it soon.

@ueqri thank you ever so much for the contribution! LGTM, although I'm starting to wonder if, as a follow-up, we shouldn't simply issue a memcpy for the entire area of an n-dimensional copy (e.g. w bytes * h rows, in the 2D case), since it is dense - a tale for another day!

I'm not very familiar with catch2 framework currently. I would try to cover the test for it soon.

What I would do here is just port one of the existing HIP tests which deal with the functions you added.

Thank you for your kind reviews! :grinning:


request: custom command requirements and actions

Is your feature request related to a problem? Please describe.
Skyra's custom commands don't allow actions to happen, which is very limiting.
Describe the solution you'd like
Use Sapphire's options:

--required-roles=Creator,Patreons
--ignored-roles=Muted
--required-channels=welcome
--ignored-channels=*
--add-roles="Advanced User"
--remove-roles="Novice User,Egg"
--remove-message
--remove-reply-after=15s
--send-at=#channel (inhibit if the channel is deleted, but not delete, so it can be edited to set a new one)

We can also implement meta.* properties, such as meta.add-roles, to access to their properties.
Describe alternatives you've considered
Modifying the tags library to support those natively.
Additional context
#201 in #suggestions.
#207 in #suggestions.

There can be many things and they are demanded also..

Custom Command Reply Message Auto Del
Command Used Auto Del.
Ignored Roles.
Ignored Channels.
All of the features should be available on Dashboard if possible.


[CI] Test results have the wrong timestamp resulting in build failures

When doing incremental builds, it looks like Jenkins has some logic around the timestamp for test results files. And fail if they are too old.
Currently we implemented a work-around in the Jenkinsfile: https://github.com/realm/realm-kotlin/pull/531#discussion_r746357497
But we should consider if this logic truly belongs in the Gradle files instead. We should re-evaluate this when moving to GHA.

Fixed


Fix pass in MonadWriter instance

Previous version would accumulate the entire log unchanged, and then
append a modified version at the end; the new one removes the old log
entries by replacing them with mempty.
Here's an example (any execWriter from mtl):
execWriter $ runEffect $ pass $ tell [1] >> pure ((), (2:))

Previously, this would evaluate to [1,2,1] (old log + modified old
log); now, it evaluates to [2,1], same as when using just WriterT.

Ah, good catch!  Yes, I agree that your version is the correct behavior

No credit to me, I picked the idea from implementation in free.

This is now up on Hackage as pipes-4.3.6


Update data-flow-expression-functions.md

@kromerm - Could you review the proposed updates?  Thanks.

checked

@kromerm, are you signing off? Also, can you fix the conflict? Thanks.

@nimoolen, @kromerm, can either of you resolve the merge conflict?

@nimoolen  Do you know how to resolve the merge conflicts? I can approve, but I have no idea what needs to be done here.

@kromerm @nimoolen The article has been updated substantially since the pull request was opened. It looks like the majority of the updates in the PR are now reflected in the live article. We'll close this PR. Thanks.


Fix docs on return file provider string

I mainly wanted to do my first "pull request" ever to learn how it works.
Fixed docs explaining Android FileProvider
@Override
public String getFileProviderAuthority() {
return BuildConfig.APPLICATION_ID + ".provider";
}

can u write your android/app/build.gradle?

@bakystefan What does build.gradle have to do with a documentation issue?
Phishing for API keys ?

@esutton looks good!
could you please resolve conflicts?

@jgcmarins I merged.  I added changes intended to help clarify.  I hope this may helps others.
Thank you for sharing your work!

Thank you @esutton


how does it render pdf thumbnail preview?

Provide a description of requested docs changes
I want to know and learn and implement on my project, please help!

Hi @Praskand,
You can take a look at https://github.com/microsoft/PowerToys/blob/main/src/modules/previewpane/PdfThumbnailProvider/PdfThumbnailProvider.cs. Most of the logic is there.
Additionally, you need to implement IThumbnailProvider interface and register the class as thumbnail provider. You can check it out here https://learn.microsoft.com/en-us/windows/win32/shell/building-thumbnail-providers?source=recommendations

Thanks > **stefansjfw ** c !


utf-8 codec cant decode 0xe2 in position 8

                                                              1 ↵

╭─p@MacBook-Pro ~/playzon/podcastfy
╰─➤  python -m podcastfy.client --file ./file.pdf
An error occurred: 'utf-8' codec can't decode byte 0xe2 in position 10: invalid continuation byte
any thoughts?
thank you

please try again. Audio from pdf had a bug that has been fixed now.

I had a similar issue with Chinese characters and asked o1-mini for a fix - that works - (I'm no coder)
Looking back I think it was in client.py
if transcript_file:
logger.info(f"Using transcript file: {transcript_file}")
with open(transcript_file, "r", encoding="utf-8") as file:
qa_content = file.read()

The PDF file I tried could be processed with API (hf.space demo), but not cli.
An error occurred: 'utf-8' codec can't decode byte 0xe2 in position 10: invalid continuation byte

having the same issue with a document created from google docs and saved as pdf


do not use CAS

Is there something wrong with CAS. A 2017 Microsoft article is all about using CAS. I'm confused.

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

ID: 5f89dddf-e9b5-b7c6-9ecc-a67b94f4412a
Version Independent ID: d44fe46d-fce9-c72e-d035-a0c9c7f09353
Content: Secure coding guidelines for .NET
Content Source: docs/standard/security/secure-coding-guidelines.md
Product: dotnet
GitHub Login: @mairaw
Microsoft Alias: mairaw

@madamakis if you're talking about topics such as https://docs.microsoft.com/en-us/dotnet/framework/misc/code-access-security, they have been updated recently to have that big red warning about the dangers of using CAS. So the guidance about not using CAS for enforcement of varying levels of trust is correct in this topic. For this reason, I'm closing this issue. But please let me know if there's something I missed.


Typo on =open message

🐛 Bug Report
Included screenshot change baby impling jar to baby impling as that is the correct command and consistent with the other implings.

Looks like this has been fixed between the issue being opened and today.

Closing as completed.