Goto Definition is off by one

Consider the following code:
open import Data.Nat
                -- cursor jumps to this line
x : ℕ           -- cursor should jump to this line
x = {! !}

y = x           -- cursor is on this x when running GotoDefinition

If using CornelisGotoDefinition on the x in the definition of y, then the cursor jumps not to the line with x : ℕ, but to the empty line above it. This seems to happen independently of what code is around the definition and how far the uses and definitions are apart from each other.
Thanks for the great plugin! <3
I'm currently switching from emacs to nvim, and apart from #153 and #121 this plugin seems to have everything I need. Keep up the great work! :)

Weird. This is definitely a regression, but I can indeed reproduce it :confused:
I'm not actively writing any Agda these days, so it might take some time to tackle this one.

Nice! I just tested it in a fork, and it indeed fixes the bug. :+1:

The vim API has at least four different indexing modes, and it's hard to make sure we pick the right one in each case :(

Yeah, I can imagine... I yet have to find a text editor API, which doesn't makes me want to pull my hair out ;)


docs(readme): use double quotes for Windows compat

According to the readme of cross-env:

Pay special attention to the triple backslash (\\\) before the double quotes (") and the absence of single quotes ('). Both of these conditions have to be met in order to work both on Windows and UNIX.

I’ve experienced this on a Window machine before and can confirm that ' doesn’t work within cmd.exe, while " does. The former breaks command execution with an error.
As a result, attention is needed when adding a test script to package.json, as the double quote needs to be escaped in JSON:
{
  "scripts": {
    "test": "cross-env NODE_OPTIONS=\"--loader ts-node/esm\" jest"
  },
}

Thank you!


Remove version: Anaconda.Anaconda3 version 2023.09-0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://repo.anaconda.com/archive/Anaconda3-2023.09-0-Windows-x86_64.exe
OK
(Automated message - build 911)

Close with reason: Version still present. Closing as requested in other PRs.;


Added Sounds with respect to weather

Different sounds will be played when the weather is clear, rainy, or cold respectively
@MaheraFurniturewala can you please approve these changes :)

Awesome job @sumitshinde2608
need you to resolve the conflicts

@khattakdev I have resolved the conflicts :)


[Feat] support cluster ip in gateway services

Description
This pr adds the cluster to the list of the gateways-supported services. It can be helpful for particular use cases where service addresses are discoverable from the other cluster.

/build

This is a niche feature that can be confusing for most people, plus using the ClusterIP can be tempting and lead to misunderstandings.
Should we document this and advise the user not to use it unless you really know what you are doing?

/test

/rebase test=true


I'd love to be able to selectively run logic in the meta only if the relationship is included.
So I could avoid calling the Posts.count method if posts weren't included.
It would also allow me to use Posts.size and perhaps take advantage of eager loading.

Thanks for your feedback – I do not have time to add new features at the moment but I'll keep this issue open in case someone else wants to tackle it in the mean time.

Ref #101.


react-native 0.48.3 and react-native-swiper - Last slide shown first

Which OS ?
IOS
Version
Which versions are you using:

react-native-swiper v1.5.12 (but it affect every single version of it, tried back to 1.5.4 as well)
react-native v0.48.3

Expected behaviour

The first slide should be first.

Actual behaviour

The last slide showed first.
Awkward pagination display - it shows the state of the first slide, while displaying the last slide.

p.s. If loop={false} is set - everything seems fine.
How to reproduce it>
Just get the repo from
https://github.com/nixoz/swiperDemo
Steps to reproduce

Create mint react-native project
Copy paste first example from react-native-swiper README
react-native run-ios
KABOOM!

UPD: I guess it related with https://github.com/facebook/react-native/issues/15808

Yes it's a regression, not working also on RN 0.46

not working also on RN 0.44

Have you try with loop=false ?

same issue here

The bug is here
if (loop) {
pages.unshift(total - 1 + '')
pages.push('0')
}
pages is modified but index is not updated. So the "current" page change.

Still exists in iOS whatever version.
Can you tell me what is the action in this step to update in Line 329?
https://github.com/leecade/react-native-swiper/blob/f00abfb859ce384be2003cfce3010c3fc0005be1/src/index.js#L324-L330
When I change the code,It works as well as Android
- this.setState(state)
+ Platform.OS !== 'ios'&&this.setState(state)


[BUG] Sorting is not working

Click on the heading of the column. It should sort the board.

Click on the heading of the column. It should sort the board.

Hi, it actually is a not implemented feature. Please feel free to pull a request :)

I would like to do this. can i ? and also @lolimay are you one of the mentor in RocketChat under GSOC?

I would like to do this. can i ? and also @lolimay are you one of the mentor in RocketChat under GSOC?

Great! Go ahead please!

@lolimay are you one of the mentor in RocketChat under GSOC?
Not yet 😜

@lolimay I am interested in doing this can you help me on how to get started?

@lolimay I am interested in doing this can you help me on how to get started?

@Sing-Li @lolimay Can you please review my PR?
Thank you!

@Sing-Li @lolimay Can you please review my PR?
Thank you!


Faster redirects

Is your feature request related to a problem? Please describe.
Redirect links always load all the stylesheets before redirecting
Describe the solution you'd like
I would like to replace the # with a /, and then use a 404.html for redirects. Invalid ones would show the default 404.
Describe alternatives you've considered
The homepage could be at homepage.html and the javascript at index.html could redirect to homepage.html when given an empty hash.
Additional context
Null

I think that we should move the homepage to a different folder and have the main index.html blank from elements and styling. That way we don't lock up the 404 pages and we can keep the same code as before. We can have blank and invalid hashes redirect to the home page.

I think that we should move the homepage to a different folder and have the main index.html blank from elements and styling. That way we don't lock up the 404 pages and we can keep the same code as before. We can have blank and invalid hashes redirect to the home page.

OK. Should I put the new homepage at
/homepage.html or /homepage or /homepage/index.html (/homepage/)

I think we should make it /home/index.html, the URL looks best that way.


[BUG] OpenSearch java high-level rest client code not compatible with elasticsearch code as claimed

Describe the bug
The offcial website: https://opensearch.org/docs/2.9/clients/java-rest-high-level/ says:
"Migrating to the OpenSearch Java high-level REST client
Migrating from the Elasticsearch OSS client to the OpenSearch high-level REST client is as simple as changing your Maven dependency to one that references OpenSearch’s dependency.
Afterward, change all references of org.elasticsearch to org.opensearch, and you’re ready to start submitting requests to your OpenSearch cluster."
But when I try to change elasticsearch java client code to opensearch, I found some classes are moved to different folders.
For example,
import org.elasticsearch.common.xcontent.XContentBuilder; cannot be changed directly because XContentBuilder is moved to
import org.opensearch.core.xcontent.XContentBuilder;
This caused huge uncertainty. And the compatibility is not as expected.
Any idea which classes are moved? Is there any tools, or any smart ways that can help changing my java client codes from elasticsearch to opensearch?
Related component
Clients
To Reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
I can change my elasticsearch java high-level REST client to opensearch javahigh-level REST client easily, as official website claimed.
Additional Details
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):

OS: [e.g. iOS]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@VachaShah - Can you please help update the documentation appropriately? Thanks!

I can help with this if allowed. What should I do?

I can help with this if allowed. What should I do?

Yes, that would be fantastic! We have some information here on contributing: https://github.com/opensearch-project/documentation-website/blob/main/CONTRIBUTING.md


Update deployment.md

Add quotation marks around the -NATSubnetAddress parameter values for New-VMSwitch. Doing so prevents an error from occurring in the PowerShell command line.

Thank you!


revert: stream container logs

Based off of https://github.com/go-vela/community/blob/master/proposals/2021/07-22_log-streaming.md
Related to https://github.com/go-vela/community/issues/367 and https://github.com/go-vela/community/issues/368
Essentially, this is a revert of https://github.com/go-vela/pkg-executor/pull/177
This reverts the changes previously introduced in the v0.10.x release to stream container logs.
An issue was discovered with the current implementation of streaming logs:
https://github.com/go-vela/community/issues/427
In a later PR, we'll look to introduce a different implementation of streaming logs:

https://github.com/go-vela/pkg-executor/tree/feature/log_streaming/opt_one

However, this functionality will be behind a feature flag and be considered opt-in.

Can't we adjust the load balancer settings to allow streaming instead of removing this?  Can we look at how kubernetes handles log streaming, because that works quite well and is also written in go. For my Vela instance in kubernetes, I often use stern to stream the logs from pipeline comedones faster than they show up in the Vela UI.

@cognifloyd I want to make sure we're on the same page before answering your question 👍
The plan of attack is to revert the existing implementation of streaming logs from the server -> worker temporarily.
I'm currently working on finalizing code locally to implement the streaming logic from option 1 in this PR description.
This will still allow streaming logs but won't be susceptible to issues depending on architecture setup with Vela.
i.e. load balancer configuration

Can't we adjust the load balancer settings to allow streaming instead of removing this? Can we look at how kubernetes handles log streaming, because that works quite well and is also written in go. For my Vela instance in kubernetes, I often use stern to stream the logs from pipeline comedones faster than they show up in the Vela UI.

We could certainly attempt to remediate the problem by updating the load balancer.
However, determining what settings to apply to the load balancer may get tricky.
For example, AWS uses a default idle connection timeout of 60s which I'm told is fairly common.
What would make sense to set as the new timeout? 5m? 10m?
And even if we did apply that configuration, there would be still be the potential for a consumer to face issues.
To clarify, I could set the idle connection timeout at 5m.
But, any pipeline that didn't send logs in 5m or less time would face the same issue.
The easiest way I could think to reproduce that issue would be:
version: "1"

steps:
  - name: sleep
    image: alpine:latest
    commands:
      - sleep 301
      - echo foo

Another point to mention is the possibility for admins of Vela to not have full control of the load balancer configuration.
This would mean they would be unable to make the changes necessary for their load balancer.

Closing this in favor of https://github.com/go-vela/worker/pull/232


tool_run_cache is flaky on Windows

See: https://github.com/astral-sh/uv/actions/runs/11847662623/job/33017797151?pr=9136

This seems to be new, I hadn't seen it before.

It's failing a lot: https://github.com/astral-sh/uv/actions/runs/11853487114/job/33034634717

(I saw it at least once this week prior to reporting here.)

Me too, but not prior to this week.

I sort of have no idea what could be flaking here.

Although the reason is unknown, the temporary environment used in the first and second uv tool run might be different. While this has little impact on usage due to the rapid environment setup, it can still lead to a certain probability of test failure.
My guess is that this PR(https://github.com/astral-sh/uv/pull/9106) included the problem, as it wasn't there before. However, I can't say for sure since this issue is not always reproducible.


Add CodeQL security scanning

Description
Add CodeQL security scanning.
Motivation
I propose to introduce CodeQL (code scanning) to this project.
This is to prevent security problems hidden in the code.
https://securitylab.github.com/tools/codeql

I've had a look through the default query set, and it flags a few things which aren't issues (like our use of hash functions for features). We also run static analysis tools internally on Tribuo, and I'm not sure that the default set of things for any tool is appropriate to turn on for all Tribuo PRs, as it's going to have a bunch of false positives that we need to disable (similar to the spotbugs excludes we already have). If there is a way of excluding CodeQL alerts which are definitely false positives then I'd be open to enabling this on the Tribuo repo. However before we can integrate any pull requests you'll need to sign the Oracle Contributor Agreement, following the process detailed in our CONTRIBUTING.md (or alternatively let me know the email address you've signed it under if you've already signed it).
If there is something specific that you found using a more detailed CodeQL analysis then please open an issue, or follow our security bug reporting guidelines (depending on the nature of the issue).


Rust 1.72 clippy and format

Overview
A dependabot PR was failing due to updates to Clippy and cargo fmt in Rust 1.72.0. (To be precise, it was 1.71.1 in the dependabot PR, but 1.72.0 got released a few hours ago, so I just upgraded to that)
https://github.com/fossas/broker/pull/122
This PR just fixes those problems.
Acceptance criteria
The linters pass
Testing plan
Risks
None
References
Checklist

[ ] I added tests for this PR's change (or explained in the PR description why tests don't make sense).

LGTM after remaining Clippy lints!

Oops, I missed that one. I just ran cargo clippy and it didn't catch it, but I see the yellow squigglies when I open that specific file.
Fixed!


zipimport rewritten

The built-in from zipimport import zipimporter is rewritten as from zip import zipimporter and classified as third party.

Looks similar to #214


Context propagation for google-gax

This should allow tracing of pubsub, vision, monitoring, spanner,
speech, and nlp.

@kjin Did you get a chance to look through the gax implementation while reviewing this? There are usually many possible patches that could be applied to preserve context. I want to make sure this approach makes the most sense in addition to solving the problem.

/cc @jmuk

I am not familiar with how the wrapping works, but patching like this would be great. GAX also has a logic of retrying or bundling inside of the funcWithAuth promise result, which could destroy the callstack unfortunately.
I am also wondering how refactoring gax could save this situation (in other words, what types of changes on google-gax side can simplify the patch like this). Let me know if you have any proposals.

@jmuk I believe there may be opportunities to refactor gax and simplify things here but I'll have to dig a little further.


MINOR: Include TopicPartition in warning when log cleaner resets dirty offset

Typically this error condition is caused by topic-level configuration issues, so it is useful to include which topic partition was reset for operator use when debugging the root cause.

Thanks for the PR, LGTM.


Cannot Place Conduits at Block Level 256 (last allowed height)

Issue Description:
This occurs in DW20 MC1.10 - version 1.8.1
What happens:
Cannot place energy conduits at block level 256, even though all other items are allowed.
What you expected to happen:
Conduits allowed to be placed at 256 (not above), I have other items there.
Steps to reproduce:
1.Go to block level 256
2. Attempt to place energy conduit
3. Nothing happens
...

Affected Versions (Do not use "latest"):

EnderIO: 1.10.2-3.1.186
EnderCore: 1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: <IP_ADDRESS>4

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

height goes from 0-255...


Switch between airpods, speakerphone and earpiece.

I'm developing a calling application for iOS. Seems Agora recognizes AirPods only if I set GameStreaming scenario.
engine.setAudioProfile(AudioProfile.Default, AudioScenario.GameStreaming)

I don't really understand why it works, because the official documentation regarding audio scenarios is confusing.
The problem here is that it affects the speakerphone volume somehow. If I route playback to the speakerphone while this scenario is activated, I almost cannot hear anything. With the default scenario, it is loud enough.
Could anybody explain how it works and how to make reliable switching between AirPods, speakerphone and earpiece during the call?

https://docs.agora.io/en/Video/set_audio_route_ios?platform=iOS
https://docs.agora.io/en/Interactive Broadcast/faq/system_volume
Are these docs helpful?

I will close this issue, you can reopen it if still exists.


Support for drawing basic blocks

This change allows the visualisation (via a CLI flag) of basic blocks - if the information is available in the BGV file - as seen below:

Still a WIP, but this has been useful for us and I wanted to see your thoughts? @chrisseaton

Ah thank you so much! That's beautiful work and I really appreciate it!
Are you working with Java? I find I don't get useful basic block annotations out of Truffle which is why I've never tried it.
Are you happy with signing the CLA? Do you need any help to do that?

Signed the CLA!
Yep, I'm working with Java - though this graph was generated by native-image, which seems to need an extra -H:+PrintGraphWithSchedule flag to add correct basic block information to the graph.

which seems to need an extra -H:+PrintGraphWithSchedule

We could put that into the docs.

Would you like me to make the suggested changes and get this merged?

I've made the changes mentioned above, but haven't yet had time to write any tests - you mentioned you drafted some tests above?
(Also, I assume #25 needs to be fixed first before merging too?)

I have this working locally, so what I'm going to do is merge it now and then iterate on it show how to add tests and things.

Thanks I released 0.8 so you can use that.

Thanks for merging :)


Add tuya ZigBee flood sensor fingerprint

Tuya ZigBee 3.0 Water Leakage Detector
https://ko.aliexpress.com/item/1005004138696961.html - Type 2

This model was already added to the driver


Transform week view

Clear and concise description of the problem
Hello everyone! Thank you for the great library. Currently, I'm using the week view to display the school schedule, but the business requires some changes in the project. Now, we need to rotate the week view by 90 degrees so that the time (days) is displayed along the horizontal axis, and the list of teachers is displayed along the vertical axis. We also need to preserve the functionality of event creation. Is it possible to achieve this using this library? Thank you in advance for your response.
Suggested solution
I just want to ask
Alternative
No response
Additional context
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Would you like to open a PR for this feature?

[ ] I'm willing to open a PR

@inu255 This is not currently possible with Big Calendar, though we do have this filed as a Feature Request for a future version. We do not have a timeframe for when this might be available.


Spelling and metrics

Master eval.json should have contents: {"AUC": "0.521324"}, hence the output for metrics, should show 0.521324 for master

Thank you! Merged!


Issue with case sensitive MATLAB dir() with Linux system

Hi,
We got the following problem when using Gannet v3.3.0 with MATLAB on a Linux system.
GannetLoad was not able to load our spectra. We also use a Mac Computer and on this computer, with the same Gannet version (3.3.0) and same MATLAB version (R2022a), GannetLoad was able to load the data.
Long story short, it turns out the the dir() function in MATLAB Linux is case sensitive while the dir() function in MATLAB Mac is not.
Indeed, our dicom files are named: '*.IMA' and are not detected with the filter in SiemensDICOMRead.m on the Linux system with the following line:
ima_file_list = dir(fullfile(folder, '*.ima'));

The workaround we used in our Linux system was to change the line above with the following:
ima_file_list = dir(fullfile(folder, '*.IMA'));

Thanks for pointing this out. This bug has also been fixed in the dev version (commit 1e1eb89fb36f041307afdcfd950d700672fdae1d). The code now reads:
[~,~,ext] = fileparts(metabfile);
ima_file_list = dir(fullfile(folder, ['*' ext]));


Fixes & changes to main

Old console read method would throw a NullPointer when developing in an IDE, so I changed the format. Notably, the password is no longer blanked out when you enter it (that is a bit of an issue...).
Along with this change, I also added a prompt for the user's district id as well as updating the whole thing to run more dynamically, so that it uses the district base URL. In theory, this setup should work across Infinite Campuses (pun totally intended).

And a fix for the gitignore to boot

Hm, from an IDE standpoint I do like the change, since it doesn't work in Eclipse at all. I never found the blanked out password to be an issue, but then again that's the standard for how it's done under Linux (for sudo, etc), so it might not be friendly for others. My only gripe is that the password shows while you type it, but the Main class is used for development anyhow so anything interfacing with the API should have a password field.
I'll go ahead and pull for now though, it seems System.In can't actually mask things anyhow without some weird workarounds.

Yup that's what I commented in my PR description. It's fairly unfortunate, but it makes it easier to develop rather than having to export to a JAR each time you want to test stuff.
Another thing, and another goal in time, unit tests.........


Initial PR for a new structure of exceptions.

Summary
Initial PR for a new structure of exceptions.
Checklist

[x] You agree with our CLA
[ ] Included tests (or is not applicable).
[ ] Updated documentation (or is not applicable).
[x] Used pre-commit hooks to format and lint the code.
[ ] Upgrade plan implemented / downwards compatibility implemented

Extensions? You mean exceptions?

Extensions? You mean exceptions?

yes, sorry for the confusion 😉 Before merging I suggest to discuss if the new structure makes sense, and then continue with some more examples before a final merge.

Extensions? You mean exceptions?

yes, sorry for the confusion wink Before merging I suggest to discuss if the new structure makes sense, and then continue with some more examples before a final merge.

No problem ^^. Could you please fix the merge conflict, that the tests can run?

LGTM, but ffs should be renamed to something more descriptive.

We will also need deprecation warnings, otherwise it may instantly break code other people wrote.

At the end before merging into the main branch, I suggest to squash the commits into one and reference the issue you raised in the commit message by adding "(#5)" at the end of the commit message.


Mapping out-of-bounds TEROS data

@stephpenn1 Does this work as a way to display locations of out-of-bounds TEROS sensors? Feedback welcome, but if yes, I'll follow a similar approach for sapflow, Aquatroll, and battery data. Thanks.
Note it pulls from the same data as the home tab badges, so they should always be consistent. Also I shifted to radio buttons up top, as plotting multiple datasets on top of each other would be too much of a hassle and probably unreadable.

OK great, thank you, I will proceed


Maintainers?

do you need maintainers here as well?

Simon has taken over the project for the most part. @simonihmig would you want another hand?

would you want another hand?

Oh sure! I have been using this as part of ember-bootstrap, and mostly focused on things I needed there. But if other issues arise, having someone to share the responsibility would be awesome!

It is done :)


Added doc on how to reconfigure failed etcd member in CoreOS

/cc @joshix @robszumski
First draft.

Note for the future: can you only rebase at the end so we can see the changes happen per commit?

Could we rename this file to etcd-live-cluster-reconfiguration.md? -- to be shorter and to match the organic convention of file names like etcd-live-http-to-https-migration.md

@joshix done

@kayrus LGTM. Please squash for merge. Thanks!

(decided we could forego the squash in favor of going ahead and getting this in)


Command line app for fetching results from HPCs

We discussed this with @Vafa-Andalibi on Hangouts.

Companion app for cluster_run (which sends the jobs)
The GUI provides the user with metadata for fetching results
User can then fetch simulation results on their local computer (or for demonstration, to jupyter notebook kernel storage inside the Brain Simulation Platform; we will create an example notebook)

Implemented in 5d102b595566bcc691ebe387f4d2e5fa3c89ea08


OpenBSD Cli error: "failed to commit db transaction: function not implemented"

Right now, wakatime-cli is throwing errors in the log file. It is safe to assume this is related to #992, but generates a different error response, and was first reported in a previous issue.
Environment:

OS: OpenBSD
Platform: AMD64
Cli Version: Built from today.

Notes:
I can provide more information about the error at a later moment.
Logs:
{"caller":"cmd/run.go:135","func":"cmd.Run","level":"debug","message":"command: heartbeat","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","version":"<local-build>"}
{"caller":"cmd/heartbeat/heartbeat.go:77","file":"less","func":"heartbeat.SendHeartbeats","is_write":true,"level":"debug","message":"params: api params: (api key: '<hidden>b6ad', api url: 'https://api.wakatime.com/api/v1', backoff at: '', backoff retries: 0, hostname: 'liberator', key patterns: '[]', plugin: 'wakatime-zsh-plugin/0.2.2', proxy url: '', timeout: 5s, disable ssl verify: false, ssl cert filepath: ''), heartbeat params: (category: 'coding', cursor position: '', entity: 'less', entity type: 'app', num extra heartbeats: 0, guess language: false, is unsaved entity: false, is write: true, language: 'sh', line additions: '', line deletions: '', line number: '', lines in file: '', time:<PHONE_NUMBER>.57854, filter params: (exclude: '[]', exclude unknown project: false, include: '[]', include only with project file: false), project params: (alternate: '', branch alternate: '', map patterns: '[]', override: 'Terminal', git submodules disabled: '[]', git submodule project map: '[]'), sanitize params: (hide branch names: '[]', hide project folder: false, hide file names: '[]', hide project names: '[]', project path override: '')), offline params: (disabled: false, print max: 10, queue file: '', num sync max: 1000), status bar params: (hide categories: false, output: 'text')","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"cmd/api/api.go:40","file":"less","func":"api.newClient","is_write":true,"level":"debug","message":"failed to detect local timezone: name not implemented for 'openbsd'","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/heartbeat/format.go:18","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute heartbeat filepath formatting","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/heartbeat/entity_modify.go:15","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute heartbeat entity modifier","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/filter/filter.go:26","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute heartbeat filtering","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/remote/remote.go:53","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute remote file detection","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/apikey/apikey.go:31","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute api key replacing","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/filestats/filestats.go:24","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute filestats detection","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/language/language.go:25","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute language detection","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/deps/deps.go:38","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute dependency detection","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/project/project.go:143","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute project detection for: less","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/project/filter.go:23","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute project filtering","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/heartbeat/sanitize.go:30","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute heartbeat sanitization","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/remote/remote.go:122","file":"less","func":"heartbeat.initHandleOptions","is_write":true,"level":"debug","message":"execute remote cleanup","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/offline/offline.go:55","file":"less","func":"heartbeat.SendHeartbeats","is_write":true,"level":"debug","message":"execute offline queue with file /home/vassilios/.wakatime.bdb","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/backoff/backoff.go:43","file":"less","func":"heartbeat.SendHeartbeats","is_write":true,"level":"debug","message":"execute heartbeat backoff algorithm","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/api/heartbeat.go:27","file":"less","func":"api.(*Client)","is_write":true,"level":"debug","message":"sending 1 heartbeat(s) to api at https://api.wakatime.com/api/v1/users/current/heartbeats.bulk","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/api/heartbeat.go:52","file":"less","func":"api.(*Client)","is_write":true,"level":"debug","message":"heartbeats: [{\"branch\":\"\",\"category\":\"coding\",\"entity\":\"less\",\"type\":\"app\",\"is_write\":true,\"language\":\"sh\",\"project\":\"Terminal\",\"time\":1706559551.5785441,\"user_agent\":\"wakatime/\\u003clocal-build\\u003e (openbsd-7.4-amd64) go1.21.6 wakatime-zsh-plugin/0.2.2\"}]","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"cmd/heartbeat/heartbeat.go:62","file":"less","func":"heartbeat.Run","is_write":true,"level":"debug","message":"successfully sent heartbeat(s)","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"cmd/api/api.go:40","file":"less","func":"api.newClient","is_write":true,"level":"debug","message":"failed to detect local timezone: name not implemented for 'openbsd'","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"pkg/offline/offline.go:107","file":"less","func":"offlinesync.SyncOfflineActivity","is_write":true,"level":"debug","message":"execute offline sync with file /home/vassilios/.wakatime.bdb","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}
{"caller":"cmd/run.go:341","file":"less","func":"cmd.runCmd","is_write":true,"level":"error","message":"failed to run command: offline sync failed: failed to sync offline heartbeats: failed to fetch heartbeat from offline queue: failed to commit db transaction: function not implemented","now":"2024-01-29T15:19:11-05:00","os/arch":"openbsd/amd64","plugin":"wakatime-zsh-plugin/0.2.2","time":1706559551.5785441,"version":"<local-build>"}

I opened an issue with the code maintainer
https://github.com/etcd-io/bbolt/issues/684

@gandarez That is good! It looks like you have been able to further diagnose the issue, tracing it to a particular model. I greatly appreciate this.

@anoduck I'm looking for building for OpenBSD manually and distribute to you before releasing any fix. I need your help since you have an OpenBSD up and running. Would you join us at Slack?
https://join.slack.com/t/wakatime/shared_invite/zt-2bz7i7u5e-K6JpRWnS0lu6AdS~UsVxoQ

I signed up, and am there.

WooHoo!!!


Add Silo Finance

Revision checks

[x] Used smart contracts were audited.
[x] Your Safe App include a manifest.json file at the root with the required data.
[x] The app can be loaded as a custom Safe App in the Apps section of https://app.safe.global.
[x] The app auto-connects to the Safe.
[x] You are able to trigger and execute one transaction with a Safe.
[x] RPC requests are optimized (not triggering many requests in a very short time period).

Name/Description
Silo Finance App
Type

New addition

Compatible Networks
- Mainnet

Audit documents
[QuantStamp] (https://www.silo.finance/_files/ugd/fd5034_0e99427a343b48bdabf6248b451bbef5.pdf)
[ABDK] (https://www.silo.finance/_files/ugd/fd5034_b73283e612ec4b05867a913e718d3983.pdf)
[CETORA (Formal Verification)] https://www.silo.finance/_files/ugd/fd5034_98066b4c70ea46b5862a00670179f067.pdf
Code for review
[Github] (https://github.com/silo-finance/)
IPFS hash/App URL
https://app.silo.finance
Desired date
No specific date, but soon would be desired. We do have a lot of users who are depositing from Gnosis safe wallets and some are already using our safe app via url, so a natively whitelisted safe app would be ideal.

Hey @dasanra - just a friendly note to let you know about our request to add Silo

Hi @siros-ena !
The App look good!
Could you please share the fronted repo with me? we'd just do a quick sanity check.
Thanks!

Hi @siros-ena !
The App look good!
Could you please share the fronted repo with me? we'd just do a quick sanity check.
Thanks!

Yes, thanks @DaniSomoza - invite sent

Hi @siros-ena !
The code looks good!
Just to mention a very minor thing: the UI is using our old Safe Logo, you can use our new Logo if you want:
https://github.com/safe-global/web-core/blob/dev/public/images/safe-logo-green.png

@DaniSomoza thanks Dani, we will update

@DaniSomoza when would you expect that we would be whitelisted?

Hey guys, QA from safe apps here 😄
The app looks good so it should be all good to be added to the list after some final quick checks from the team 😎 we will notify you here with a message soon.
Cheers!

Sounds good @JagoFigueroa

App is published now

@JagoFigueroa @dasanra can we request 2 updates:

Can we add Arbitrum support (we just launched this week)
Can we update title to "Silo Finance"

Thank you

Hey team! We can change the app title 😄
Regarding the Arbitrum support I gave the app a quick check on this chain but I am seeing a few CORS and RPC errors popping up in the console after a few seconds.

Could you have a look and open a separate ticket for the app addition in that chain as this ticket was closed? Review process should be quick once this is fixed 😎
Cheers!


Merge 2.14.0 into develop

What does this PR do?
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

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

Project coverage is 70.15%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2288      +/-   ##
===========================================
+ Coverage    70.05%   70.15%   +0.11%
===========================================
  Files          732      732
  Lines        27215    27227      +12
  Branches      4577     4580       +3
===========================================
+ Hits         19064    19101      +37
+ Misses        6869     6860       -9
+ Partials      1282     1266      -16

see 27 files with indirect coverage changes


AddFontFromMemoryCompressedTTF Are Crashing.

I've been trying to load fonts directly from memory using AddFontFromMemoryCompressedTTF (), and I haven't been successful, is anyone getting it?
my Code:
fonts.test1 = io.Fonts->AddFontFromMemoryCompressedTTF(BurBank_bold_compressed_data, BurBank_bold_compressed_size, 20.0f, NULL, io.Fonts->GetGlyphRangesCyrillic());
I remember well that a few months ago it was working, I believe that @danielkrupinski  updated his ImGui, and so something changed.

You need to pass in an ImFontConfig with FontDataOwnedByAtlas set to false.
ImFontConfig config;
config.FontDataOwnedByAtlas = false;
fonts->whateverFont = io.Fonts->AddFontFromMemoryCompressedTTF(
    compressed_data,
    compressed_size,
    20.0f, &config,
    io.Fonts->GetGlyphRangesCyrillic()
);

Thanks :)


Package Pretzel as a global tool

Packs Pretzel as a global tool.
relates to #338
@laedit should be ready for review :)

Seems good to me, can you just update appveyor.yml to add Pretzel.Tool in the artifacts and in the nuget deploy?
Since I think dotnet global tools need to be available from nuget, right?

Seems good to me, can you just update appveyor.yml to add Pretzel.Tool in the artifacts and in the nuget deploy?
Since I think dotnet global tools need to be available from nuget, right?

So you mean like so? I wasn't aware that you can specify them in appveyor.yml as well


Add support for negative weights

Currently only positive edge weights are supported in matching graphs, owing to the Dijkstra subroutine (and switching to Bellman-Ford wouldn't help with negative cycles). But negative weights can be supported by flipping edges with negative weights first, before using the updated weights and syndrome to decode.

Note about negative weights also added to the PyMatching paper (will be in the next arxiv version)


[FLINK-20281][table] support consuming cdc stream about window tvf aggregate

What is the purpose of the change
Currently, window aggregation doesn't support to consume a changelog stream, and will throw an exception.
This pr is introduced to support consuming cdc stream about window aggregate.
See more at FLINK-20281 and FLINK-27539
Brief change log
(for example:)

Update the logic about inferring ModifyKind and UpdateKind for window agg node
Introduce a count in window agg, and when there is no data in window, not output the agg result
Add tests for TUMBLE, HOP and CUMULATE window tvf to consume cdc source

Verifying this change
Tests are added for TUMBLE, HOP and CUMULATE window tvf to consume cdc source
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? no need to update doc

Hi, @LadyForest thanks for your patient review.

One suggestion is to split table-runtime and table-planner changes into two smaller commits to ease the review.
Great suggestion! The reason I don't split this big pr into two commits, runtime and planner, is as following:

There are not many changes to the pure runtime part. This pr looks like too big is because the auto-generated files by json plan test, and this part takes up nearly 3000 lines. That means these big files will be added to the planner commits section.
If splitting commits, we may first has a commit with runtime, then planner. But the harness test for runtime operator depends on planner.
But anyway, I appreciate your advice.


Few problems CHBC 1.4

Everything installed well but few errors came up

SYNC on wii u or gamepad  doesn't bring up the menu

Every time exiting game, app, anything that'll bring up the "Wii U Menu screen" autoboots cfw then loops

The 1.4 HBL on the Loadiine HBL site freezes but 1.3 works just fine

Exiting 1.3 HBL or even installing a game with Wupinstaller has the same problem like in problem # 2

If the SYNC button doesn't bring up the menu, or CBHC doesn't work well with certain versions of HBL, then this is a problem for FIX94 to solve - not me.


Orientation EXIF tag incorrectly interpreted from GoPro Hero 7

I'm not sure why this hasn't been a problem with previous GoPro cameras. But I just got a Hero 7 and all the tracks I have uploaded with the camera mounted upside down show up on OpenStreetCam in portrait mode, with the right edge of the image pointing down.
I did some testing with a stationary image and rotated the camera clockwise. The camera detected the rotation and changed the Orientation tag and OpenStreetCam is obviously detecting the orientation tag but mishandling the value. Images taken in a "normal" orientation show up correctly as do images taken when the camera is rotated 90 degrees counter clockwise. But 90 degrees clockwise and 180 degrees are not working. The Windows image viewer, JOSM, GIMP and Mapillary all show the correct orientation.
Here is my user profile right now with the test images plus 3 actual tracks with incorrect rotation:

The four test ones were taken starting with the camera upright (the 4th track in my profile) and then rotated clockwise 90, 180 and 270 degrees. The value of the Orientation tag according to exiftools is 1, 6, 3 and 8.

So it seems like this has been fixed for new uploads.  Should I write a script to go through and rotate them all?

@alexandrui-telenav @mirunac-telenav  will these be re-processed to fix the flipping?


Script running but no data returned

I'm attempting to run some sample scripts as outlined in the examples. I'm not getting any errors but there is no data returning at all.
from sportsipy.mlb.teams import Teams
teams = Teams()
for team in teams:
print(team.name, team.abbreviation)
I expect team names and abbreviations to print but they are not.

Desktop (please complete the following information):

OS: 64-bit Windows 10
Sportsipy Version: 0.6.0

Please help!

I am also getting the same results.

I have the same issues :(

Same issue here. I went to the sportsreference site and it seems to have created other websites that hold the actual information. This project may have just been torpedo'd because of it.

Had to run with in virtual environment with proper dependencies from requirements file.

Ran into this error and found after some investigating I had updated my packages to include pyquery=2.0.0, instead of 1.4.0.  This new version of pyquery requires explicit identifcation of URLs.  (Note: In my personal project I needed the later pyquery for other code I was also running along with sportsipy.)
This commit worked for me to get schedules for various leagues, and I updated the rest of the code where it appeared URL identification was needed as well.
I could probably turn this into a pull request, but I'm not sure this project is being actively updated now.
https://github.com/roclark/sportsipy/commit/bcd92b58ed0d2ce9c6918015e64db62af8fb9627
The specific branch this commit is on is:
https://github.com/davidjkrause/sportsipy/tree/pyquery2
Interested to hear if this helps anyone get by the issues mentioned above.

you fix did the job I can't thank you enough!

Will the solution of @davidjkrause be implemented?

My patch for pyquery2 breaks many existing tests.  Note that many of the current tests already don't pass, but the pyquery change breaks more.
I'm working on fixing all the test cases, and will update my branch.  Some need fixing for the pyquery change, others are broken in base, and others don't fail but will if the locally stored HTML is updated because the source site has change format.
This is all a pretty big effort, but I'm making solid progress on this and plan to get my fork of this repo at least mostly working in the next few days.  I've made many changes to fix issues, but the best approach would be to break the work into separate related patches, rather than just a bulk update.

https://github.com/davidjkrause/sportsipy/tree/pyquery2 is updated with all tests fixed, and a number of other fixes for recent data format changes.
Planning to continue working on my branch / fork to bring in other PRs which fix data issues and have been sitting for a while.


Make it impossible to create IMAP folders

I have searched Dovecot mailing list archives, recommended solution is to setup ACL plugin: https://doc.dovecot.org/configuration_manual/acl/
We need to deny creation of new folders and make a test that tries to create folders like "DeltaChat", "Trash", "Spam", "INBOX/DeltaChat", "INBOX.DeltaChat" etc. and fails every time.

Why do we need to do that?


[SMALLFIX] Cleanup for master block info test

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10855/
Test PASSed.

Merged build finished. Test PASSed.

+1 LGTM


Deploying on aws yields error in resolve_reference

Describe the Bug
Running an initial build on aws (with role switching) yields the following error:
{"target":"localhost","action":"task","object":"terraform::resolve_reference","status":"failure","value":{"_error":{"kind":"JSON::ParserError","msg":"783: unexpected token at ''","details":{"backtrace":["/opt/puppetlabs/bolt/lib/ruby/2.7.0/json/common.rb:156:in `parse'","/opt/puppetlabs/bolt/lib/ruby/2.7.0/json/common.rb:156:in `parse'","/tmp/e3ae0b25-9a2d-46b6-a275-da2c8f1acf41/terraform/tasks/resolve_reference.rb:43:in `load_statefile'","/tmp/e3ae0b25-9a2d-46b6-a275-da2c8f1acf41/terraform/tasks/resolve_reference.rb:19:in `resolve_reference'","/tmp/e3ae0b25-9a2d-46b6-a275-da2c8f1acf41/terraform/tasks/resolve_reference.rb:130:in `task'","/private/tmp/e3ae0b25-9a2d-46b6-a275-da2c8f1acf41/ruby_task_helper/files/task_helper.rb:58:in `run'","/tmp/e3ae0b25-9a2d-46b6-a275-da2c8f1acf41/terraform/tasks/resolve_reference.rb:135:in `<main>'"]}}}}
Error executing plugin terraform from resolve_reference in terraform: 783: unexpected token at ''

I was able to get past this by removing the inventory.yaml file and trying again.
Expected Behavior
The environment is built on aws.
Happy to provide more info if needed. Note that I am using assume-role to switch from the bastion role to the required role.

@jessereynolds I was able to reproduce, thank you for the bug submission.
The scenario which reproduced it for me was running autope post run failure or user initiated kill, specifically while Bolt is running Terraform. When you get into this situation, run ps and check to see if there are any terraform processes running, if yes then kill them. After that, from the autope project directory cd .terraform/aws_pe_arch && terraform destroy. It might say that there is nothing to destroy but will clean up some lock files that were previously left around.
After Terraform finishes the destroy then return to the Bolt project's root and attempt to run autope again. This solved the issue for me.
If you use autope/terraform enough on AWS then you might notice that it "hangs" occasionally. I've found this to be related to the short lifespan of the STS token that is generated when going through the bastion assume-role workflow so I got in the habit of refreshing my credentials before doing anything with Terraform, e.g. re-assuming my role. Overall, we've found more quirks using Terraform with AWS then with GCP.
This is might be a bug more appropriate for puppetlabs/puppetlabs-terraform since the resolve_reference is happening before the autope plan runs so we have a limited ability to work around it in this module.


Former HistroyTrafficProvider remains in next smarts env

BUG REPORT
High Level Description
When I try to initialize second smarts env in one process, the HistoryTrafficProvider from the first smarts env is not removed (even with smarts.destroy()) and will cause vehicles disjoint which makes second smarts env failed.
SMARTS version
develop branch
Steps to reproduce the bug
Just create one more smarts env in https://github.com/huawei-noah/SMARTS/blob/develop/examples/traffic_histories_vehicle_replacement.py and use it to do rest task
Resulting and expected behaviour
HistroyTrafficProvider should be removed when smarts env be destroyed
Error logs and screenshots
the normal providers should be like:

provider num:5
[<smarts.core.agents_provider.AgentPhysicsProvider object at 0x7f44a4318c90>, <smarts.core.agents_provider.DirectControlProvider object at 0x7f44a4318f10>, <smarts.core.agents_provider.MotionPlannerProvider object at 0x7f44a4318fd0>, <smarts.core.agents_provider.TrajectoryInterpolationProvider object at 0x7f44a43300d0>, <smarts.core.traffic_history_provider.TrafficHistoryProvider object at 0x7f95ab69b4d0>]

However, if I then initialize two more smarts env and make sure the first two smarts env are destroyed, the providers of the third env also included ex-provider.

provider num:7
[<smarts.core.agents_provider.AgentPhysicsProvider object at 0x7f95a90fe390>, <smarts.core.agents_provider.DirectControlProvider object at 0x7f95a90fe090>, <smarts.core.agents_provider.MotionPlannerProvid
er object at 0x7f95a90e4ed0>, <smarts.core.agents_provider.TrajectoryInterpolationProvider object at 0x7f95a90e4050>, <smarts.core.traffic_history_provider.TrafficHistoryProvider object at 0x7f95ab69b4d0>
, <smarts.core.traffic_history_provider.TrafficHistoryProvider object at 0x7f95a9118690>, <smarts.core.traffic_history_provider.TrafficHistoryProvider object at 0x7f95a90e4c10>]

The disjoint make ego-agent get done unproperly, the collision event show that it collide with its history-vehicle version.

Hello @CyberSY, I have made a fix for the likely issue: could you try out https://github.com/huawei-noah/SMARTS/pull/1544 or git cherry-pick 2d526bd?

Hello @CyberSY, I have made a fix for the likely issue: could you try out #1544 or git cherry-pick 2d526bd?

@Gamenot It works! Thanks for the quick reply.

Fix is merged.


Testing assertion errors, probably due to quotation marks

I am getting the following errors when running the test suite: https://ppb.chymera.eu/a76304
I assume this is because of quotation mark mismatch. Is this something that should be fixed here?

This particular line "No such command ..." is printed by click itself.
What version of click on which platform are you using?

8.0.1 on Gentoo Linux

I've just noticed that you've changed the test code which leads to this issue.
click uses __repr__ to print that missing command (https://github.com/pallets/click/blob/e8e4b32f5034b8e6df9f4d5ba7ed952da9394a76/src/click/core.py#L1715) which uses single quotes.
You've changed it to double quotes ;)
see
https://github.com/click-contrib/click-didyoumean/blob/master/tests/test_core.py#L40
and yours:

@timofurrer I didn't. In fact, it seems the issue is that you're looking at the live version, not at the most recent release: https://github.com/click-contrib/click-didyoumean/blob/80913ac6907d531346ce98e5902bf3a5707d5bda/tests/test_core.py
It also appears the “most recent release” is rather ancient. Do you have some newer versioning scheme?

I see :)

It also appears the “most recent release” is rather ancient. Do you have some newer versioning scheme?

Yes, the project here can use some love and the latest release is quite out dated.
I suggest that you use it from the master branch for now. Let's see if I can find a few hours this week to bring it up to date :)

@timofurrer Merci vielmohls ;)

A first release 0.1.0 is on PyPI. More updates follow :)
Cheers! 🇨🇭

@timofurrer I tried both the 0.1.0 and 0.2.0 versions you released, and I get the following for both:
 * python3_9: running distutils-r1_run_phase python_test
python3.9 -m pytest -vv -ra -l -Wdefault
=========================================================== test session starts ============================================================
platform linux -- Python 3.9.7, pytest-6.2.5, py-1.10.0, pluggy-1.0.0 -- /usr/bin/python3.9
cachedir: .pytest_cache
rootdir: /var/tmp/portage/dev-python/click-didyoumean-0.2.0/work/click-didyoumean-0.2.0
plugins: pyfakefs-4.5.1, timeout-1.4.2, mock-3.6.1
collecting ... collected 0 items

========================================================== no tests ran in 0.01s ===========================================================

@timofurrer the tests pass from the live cloned repo in any case (they always have, this was just regarding the releases). But it seems the tests aren't included in the PyPI tarball:
chymera@decohost ~/click-didyoumean-0.2.0 $ tree
.
├── LICENSE
├── PKG-INFO
├── pyproject.toml
├── setup.py
└── src
    └── click_didyoumean
        └── __init__.py

2 directories, 5 files

@timofurrer tests are now included and pass as of the 0.3.0 GitHub tarball.


Demo live example doesn't work

I tried to open the web page which shows the demo live.
But it doesn't work.

@laurennlam hello!
Can you please try to open it via Google Chrome browser?

@laurennlam thank you for opening this issue,
Recently we have released MRI-viewer version 2.0. You could try it out.
If you face any problems please open new issue with details and screenshots.
Thank you.


TN-2691 rewire assessment engine view / card html generation

initial take at moving ae card templating into standard template tooling.  Incomplete work - checking in PoC.

Hello @pbugni! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file portal/eproms/views.py:

Line 112:17: E127 continuation line over-indented for visual indent
Line 122:1: W293 blank line contains whitespace
Line 123:5: E265 block comment should start with '# '
Line 126:1: W293 blank line contains whitespace


Fixed Issue #8 - Tags not displaying correctly

Tags were displaying the literal object and not the value of it. Changed the save_article_tags function in the js file to save the value of user input.

Fixes #8 <--- see what I did there?
👍


how to plot your figure

Hi, Thank you for your work and for providing the code. How can we visualize the attention feature map and feature similarity?

Hi, thanks for your attention.
(1) To visualize the attention map (the size of it is (HW)(H*W)) , you can select a position and get it's attention. Add the code like the one shown below in attention module.
# sim_map_div_C size: batch_size*(H*W)*(H*W)
visual = sim_map_div_C[:, position, :]
visual = visual.view(batch_size, 1, *x.size()[2:])

(2) To visualize the feature similarity,  you can select one position of the feature and calculate the cosine similarity. Add the code like the one shown below to the feature you want to visualize.
# feature size: batch_size*C*H*W
point= feature[:,:,h,w].unsqueeze(2).unsqueeze(3)
point = point.expand(feature.size())
visual = torch.cosine_similarity(point,feature,dim=1)

(1)Thank you very much for your answer. I'm still feel confused about question 2. Do I understand correctly. After I use hook to obtain the feature (at this time, the feature size is B C H/8 W/8), then select a point on the feature map and calculate the similarity, and finally interpolate it to the original image size (H W)? Here is the code I used for testing.
(2)Or should we first interpolate the feature map to the size of the original image and then select a point for calculation?
#H W C
img = input.permute(0, 2, 3, 1).cpu().numpy().squeeze()
#B C H' W'
fea = feature[2]
#B C 1 1 -> B C -> B C H' W'
point = fea[:, :, 77, 77].unsqueeze(2).unsqueeze(3)
point = point.expand(fea.size())
#B C H' W' -> B C H W
visual = torch.cosine_similarity(point, fea, dim=1).squeeze()
visual /= visual.max()
visual *= 255
visual = visual.cpu().numpy().astype('uint8')
visual = cv2.resize(visual, (2048, 1024))
heatmap = cv2.applyColorMap(visual, cv2.COLORMAP_JET)
heatmap = cv2.addWeighted(heatmap, 0.7, img.astype('uint8'), 0.3, 0)
cv2.circle(heatmap, (77, 77), 10, (0, 0, 255), 0)
cv2.imwrite('./figure/result/heatmap_' + 'test' heatmap)
cv2.imwrite('./figure/result/raw_'  + 'test', img)
cv2.imshow('heatmap', heatmap)
cv2.waitKey(10000)
atten.remove()

There's no much difference between the two methods in my opinion. I use the first one.


initial home view with a random gene URL

For consistency (and who knows what else; e.g., this might be the reason for #24 as well), the initial pcviz view (using, e.g., BRCA1, picked by chance from the hard-coded list of 4 genes) should probably display (and use/route) the same URL in the browser address bar as if a user typed that gene symbol manually. I.e., currently, it shows up and stays if you refresh the page as:
http://www.pathwaycommons.org/pcviz/

, whereas it should be:
http://www.pathwaycommons.org/pcviz/#neighborhood/BRCA1

Attn: @armish , @onursumer

This was a design choice I made in which I assumed whoever is landing on the PCViz without any query genes doesn't know what gene to search for at first. So I thought it might be a good idea to show a random network up there and in case they refresh the page, let them see new networks for example.
If we set the URL to whatever we are showing randomly on the homepage, it is basically locking the user in, which I don't like that much. Any particular reason to switch behaviour?

Ok, that's no problem. Let's keep as is. :)


Basic building with WIndows for #196

The Problem:
We want to consider support windows for ddev per https://github.com/drud/ddev/issues/196
The Fix:

Add windows target to Makefile (This is actually accomplished in upstream https://github.com/drud/build-tools/pull/25, which will overwrite the minor change here when it comes in)
Add vendoring necessary to build for windows.
Support $DOCKER_HOST and related environment variables, which we may hit in various contexts
Remove the assumption of docker being on localhost and parse $DOCKER_HOST to get it.
Support TLS connection with docker, not just http.
Improve prepLocalSiteDirs() and move it into ddevapp/config.go, which is the only remaining user; run it on config new.
Minor test changes to account for the change in calling prepLocalSiteDirs() (.ddev/data is created earlier)
If sudo is not available, tell people what to do to add the hosts entry.

Although it looks to me like these are all reasonable long-term changes for ddev (not particularly related to windows and our decision on it) I"m not attached to any of them, so if the team wants to leave any of these behind I'll rework.
With this PR you can make windows to build into bin/windows/windows_amd64/ddev.exe
In addition, all 3 binaries (OSX, Linux, Windows) are now pushed into the circle test page:

The Test:
Automation Overview:

Related Issue Link(s):

#196 Decisions and Plan for Linux and WIndows Support
Cross-compile failure on go-winio: https://github.com/Microsoft/go-winio/issues/52

Release/Deployment notes:

This builds for windows. The resultant executable and its operation are a completely different topic.

I think this is ready for review, and that there's really nothing here that is Windows specific. I think it should generally be improvements. This currently runs on Windows using the ddev.exe under cmd and with docker toolbox (regular windows version). It gets messed up if you run in any environment that translates paths.

@rfay Curious. I went to https://circleci.com/gh/drud/ddev/752 and did not see the "artifacts" tab. Does that require being logged in to access?

@rickmanelius do you not see the tab? Please try logging into circle. It's just a matter of authorizing login via github. Sorry I didn't know that was privileged.

@rfay that worked 👍  I hadn't yet explored Circle's user/registration process and I didn't know if this might result in an extra license against an account.

Rebased to current, with one conflict. I think if this runs ok on Windows and passes the tests here it can come in. I'll work on getting windows tests to run and solving shell-out problems in a separate PR.


Break dependency on google::protobuf::UTF8FirstLetterNumBytes()

Remove dependency on the google::protobuf::UTF8FirstLetterNumBytes() to prep for the 22.x upgrade. The UTF8 utilities were removed from protobuf in 22.x.
Since the needed code was very simple I have copied the functions from the protobuf to the validate.h file.

 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.


Erros with env variable

Dear Dev,
there is a bug in this crate in combination with env. variables.
I found out because lazy_static crate failed to compile, they depent on your crate.
However the error is not related to lazy_static, only to env. variables on the system.
Usually i compile with some logging enabled. for that i set the RUST_LOG env. variable.
However i have a invalid configuration of this env, variable.
it is set to show warning, and this seems to changed and now produces a warning.
However, the env. variable does not only affect my compilation of RUST_BACKTRACE=1 RUST_LOG=cargo=warning,server=debug,server-cli=debug,common=debug,region::vol_per=debug,world=debug cargo build. but it seems also to be applied to all underlying processes (AFAIK!) .
So i noticed your crate has a problem parsing the version when the env variable is set. And as far as i understand the code it is calling rustc --version and parsing the output.
However, when these env variables are set. then the output of rustc --version is changed.
And i suspect that this is the point where the is_min_version does no longer works and returns None.
btw, this is an example:
$( cd voxygen && time RUST_BACKTRACE=1 RUST_LOG=cargo=warning,server=debug,server-cli=debug,common=debug,region::vol_per=debug,world=debug rustc --version)
warning: invalid logging spec 'warning', ignoring it
rustc 1.30.0-nightly (63c75d375 2018-09-21)
RUST_BACKTRACE=1 RUST_LOG= rustc --version  0.01s user 0.01s system 98% cpu 0.019 total
See, the warning is outputted first before the version is returned.
Is there a way to fix this problem ? i would suggest either to set RUST_LOG="" while executing the rustc command or maybe parse each line on it's own.
Thank you very much for your time

Cargo likely shouldn't be emitting the warning to stdout. I'd recommend opening an issue on cargo. Nevertheless, I've made the parser resilient to this in v0.1.5, so you should be able to simply run cargo update in your project to have your issue resolved.


Ftrack Integrator may use wrong entity where publish is integrated

Description
Plugin Integrate Ftrack Api use "ftrackEntity" or "ftrackTask" which is collected in plugin Collect Ftrack Api but may not be the final entity for integration to ftrack e.g. during batch publishing or editorial. There may be more than one entity where to publish.
Found out during PR https://github.com/pypeclub/pype/pull/1101
[cuID:PYPE-1150]

@iLLiCiTiT I reckon we can close this considering we havent' ancountered it since it since. Or at least we had no report on it.

I agree we can close this. It is tightly connected to fact that instance does not have information about asset to which it belongs and fallbacks to context data.


ppx antiquotations?

I realize this is not the right place to ask this question, but unfortunately I don't know what the right place is. We're considering moving from camlp4 to ppx for the frenetic project. Currently we use camlp4 to embed our DSL (namely NetKAT) into OCaml, so we can write things like
let p1 = <:netkat< port:=2 >>
let p2 = <:netkat< ip4Dst:=<IP_ADDRESS> >>
let p = <:netkat< switch=1; $p1$ + switch=2; $p2$  >>

Camlp4 will parse the DSL at compile time and replace the code snippets with their less pretty AST versions:
let p1 = Filter(Port, 2)
let p2 = Filter(Ip4Dst, 167772161L)
let p = Union (Seq (Filter(Switch,1), p1), Seq (Filter(Switch,2), p2))

Is ppx powerful enough to support this kind of use case? In particular, can ppx support antiquotations, as in the definition of p above? I have seen that with ppx we could have something like
let%netkat p = ip4Dst=<IP_ADDRESS>

but I couldn't find anything on antiquotations.
Sorry for asking this question here, and thanks in advance for any help or pointers you might be able to provide.

If your concrete syntax compatible with OCaml's, you can indeed just use extension nodes. But an IP address is not valid OCaml syntax, so either you change the syntax e.g. to something like:
  let%netkat p = ip4Dst := IP("<IP_ADDRESS>")

or you need to create you own parser:
  let%netkat p = "ip4Dst := <IP_ADDRESS>"'

(and then you decide on the syntax for "antiquotations")


ANC Subs Visit - Hyperemesis Gravidum - Update to Logic [4h]

Issue
The clinical team found an issue with the original logic. If the patient is over 20 weeks, then she should be evaluated for another diagnosis. So, if <20 weeks, Refer to Hospital for Hyperemesis Gravidum. If >=20 weeks, then Refer to Hospital for Severe Vomiting.
To Do:

Update the diagnosis pop-ups for Hyperemesis Gravidum
Update next steps - these types of screens already exist in e-heza so just reuse those designs

@balagan73 please test on nutrition env


UserWarning: The value passed into gr.Dropdown() is not in the list of choices. Please update the list of choices to include: delete or set allow_custom_value=True.   warnings.warn(

启动webui之后有警告
进入webui 网页，训练方式，训练精度等不可选，没有下拉选项
/hy-tmp/conda/envs/swift_v3/lib/python3.8/site-packages/gradio/components/dropdown.py:173: UserWarning: The value passed into gr.Dropdown() is not in the list of choices. Please update the list of choices to include: delete or set allow_custom_value=True.
  warnings.warn(
/hy-tmp/conda/envs/swift_v3/lib/python3.8/site-packages/gradio/components/dropdown.py:173: UserWarning: The value passed into gr.Dropdown() is not in the list of choices. Please update the list of choices to include: lora or set allow_custom_value=True.
  warnings.warn(
/hy-tmp/conda/envs/swift_v3/lib/python3.8/site-packages/gradio/components/dropdown.py:173: UserWarning: The value passed into gr.Dropdown() is not in the list of choices. Please update the list of choices to include: AUTO or set allow_custom_value=True.
  warnings.warn(
/hy-tmp/conda/envs/swift_v3/lib/python3.8/site-packages/gradio/components/dropdown.py:173: UserWarning: The value passed into gr.Dropdown() is not in the list of choices. Please update the list of choices to include: push_best or set allow_custom_value=True.
  warnings.warn(
/hy-tmp/conda/envs/swift_v3/lib/python3.8/site-packages/gradio/components/dropdown.py:173: UserWarning: The value passed into gr.Dropdown() is not in the list of choices. Please update the list of choices to include: 0 or set allow_custom_value=True.
  warnings.warn(
/hy-tmp/conda/envs/swift_v3/lib/python3.8/site-packages/gradio/components/dropdown.py:173: UserWarning: The value passed into gr.Dropdown() is not in the list of choices. Please update the list of choices to include: nf4 or set allow_custom_value=True.
  warnings.warn(

不能复现，请给下复现步骤，或使用main分支最新代码尝试


the lasted resized example cannot be compiled correctly

the error msg is as followed:

Actually, resize() needs IntoImageView ,but the passed src_image is DynamicImage which does not implement IntoImgeView

You must enable "image" feature to support of image::DynamicImage. This is written before the example in README.md.

my fault. thanks a lot


Add SegNum

https://github.com/Loop3D/map2loop/issues/56

Added test: sampled_geology must have 'segNum' in column names
in https://github.com/Loop3D/map2loop/pull/57/commits/9760cd61748cc85b7403cfd47295119bb3c11270
(sorry, used a different gh account to commit, will pay attention to that in the future)

@lachlangrose updates test as requested initially; let me know if further changes are needed - https://github.com/Loop3D/map2loop/pull/57/commits/7cbf143b84a4c7cb3f34b563d2f9ef3202b0433e


CP4BA storage class should be changed

Hi Team,
This is a reference to the storage class in following file:  modules/cp4ba/templates/cp4a_shared_log_pvc.yaml.tmpl.
The storage class (storageClassName: ibmc-file-gold-gid) is mentioned as static/hard coded.
This storage class is not available in any environment and hence we should change it to a dynamic reference storage class.
Update file reference.
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
name: cp4a-shared-log-pvc
namespace: ${cp4ba_project_name}
labels:
app.kubernetes.io/instance: ibm-dba
app.kubernetes.io/managed-by: ibm-dba
app.kubernetes.io/name: ibm-dba
release: 21.0.1
spec:
accessModes:
- ReadWriteMany
storageClassName: ${storage_class}
resources:
requests:
storage: 100Gi
We can make a variable for storage class i.e., ${storage_class}.

@logicJohn @krook
Could you please review this and update it?

@annumberhocker can you please review this.

@BrahmSingh These terraform repos are no longer supported.  They are available as is.

@annumberhocker , Requesting you to confirm by when above (last week) request of @BrahmSingh will be done. With this we will be able to close current communications. - Thanks in advance.
@logicJohn , @krook , FYI...

@pranavibm The changes have been made

@annumberhocker , Requesting you to confirm by when above (last week) request of @BrahmSingh will be done. With this we will be able to close current communications. - Thanks in advance.
@logicJohn , @krook , FYI...


Count not a general solution in Chipotle, 02_Filtering_&_Sorting

Step 8 proposes the following solution to count how many Veggie Salad Bowl there are.
chipo_salad = chipo[chipo.item_name == "Veggie Salad Bowl"]

len(chipo_salad)

However this doesn't seem like a general solution to the problem. Wouldn't it be better:
chipo[chipo.item_name == "Veggie Salad Bowl"].quantity.sum()
So that quantities get taken into account?

The question is "Step 8. How many times were a Veggie Salad Bowl ordered?" not "Step 8. How many Veggie Salad Bowl were ordered?". Coincidentally they provide the same answer 18, but are different questions.


Error: Could not execute GraphicsMagick/ImageMagick: gm "convert" "/GitHub-Trending/public/debug-politics.svg" "-resize" "96x" "/GitHub-Trending/public/debug-politics.png" this most likely means the gm/convert binaries can't be found

GET https://github.com/showcases - {"page":1}
GET https://github.com/showcases - {"page":2}
GET https://github.com/showcases - {"page":3}
GET https://github.com/showcases - {"page":4}
Error: Could not execute GraphicsMagick/ImageMagick: gm "convert" "/GitHub-Trending/public/debug-politics.svg" "-resize" "96x" "/GitHub-Trending/public/debug-politics.png" this most likely means the gm/convert binaries can't be found
at ChildProcess. (/GitHub-Trending/node_modules/gm/lib/command.js:299:14)
at emitOne (events.js:96:13)
at ChildProcess.emit (events.js:188:7)
at Process.ChildProcess._handle.onexit (internal/child_process.js:213:12)
at onErrorNT (internal/child_process.js:359:16)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)

You need to install GraphicsMagick. This will differ based on what OS you're using but something like sudo apt-get install graphicsmagick would do the trick if you're on linux (and using apt).

Thank you very much, I will try it.

@thedillonb @leichunfeng  I am facing this issue in windows can you help me please,
Thanks.

@Abdul-majid-ashrafi You need to install graphics magick for windows: http://www.graphicsmagick.org/INSTALL-windows.html

@thedillonb Thank you bady

After installing also am getting error..But If i typed gm in E: its showing installed version


[WIP] Added versions in form of helm_releases to the main.tf file

For two of the tools there are not helm entries in the artifact hub

Looks good! Thanks

@pipatth could you test this in oml-dev cluster and merge for us? Rouven won't work on this anymore


Fix analysis quota view leak

This PR fixes a leak related to the quota analysis view. In some scenarios, we found the analysis-control-view is not properly removed when the parent view was removed. This could cause an inconsistent state of this view when the analysis finishes or fails. Basically, it gets stuck in the fetching state.
Context
In analyses-view.js we have a bind for when the analysis form collection is reset or an analysis is removed. When an analysis is done, the source changes, so we reset the analysis form collection with the new data. This follows a complete render of the parent view. But the thread in the actual view is not finished, so when it get back to this point the former view is rendered again, this time in memory because it was removed from the DOM when the parent view was re-rendered, but still listening and rendering subviews.

As far as I know, no news needed.

Tested in staging and working like a charm

When green, I'll merge it 💃 . Good catch man!

Frontend passing, merging 💃 .


[approvaltest] Ignore dynamic fields from sort fields

The approval tests doesn't handle dynamic fields correctly, the fields are sorted before dynamic is handled. Due to this, 2 issues could happen:

If a field is passed as dynamic but it is also in the sort fields then the resulting sort would be indeterministic.
If all the sort fields are identical then the fields are compared using bytes.Compare this will again result in indeterministic sorts if the dynamic fields are randomly generated.

Both these issues are fixed in this PR by rewriting the dynamic fields before sorting and comparing for approvals.
Required to fix the failures in https://github.com/elastic/apm-server/pull/13196

Could this PR be simplified by pulling the rewrite loops out of approveEventDocs and approveFields?

@axw This was my first thought and I spent some time into trying to refactor code as per this but the code I came up with was not optimum and required copying the raw fields and the raw source both in-memory. My previous effort was based on not decoding the data first but I think we might be able to do this if we decode to JSON first and then parse dynamic fields. I will give this a shot.

@axw Fixed the logic to rewrite before sort. PTAL.


created the sam deployer

@MatteoIorio11 looks good in general. However, please address the comments made in the PR, and also add missing trigger logic (see here).

I have added the 'sam build' and the 'sam deploy' inside the make and I have set the "stack-name", "the function name" and the "s3-bucket" defined in the top of the makefile

I have also removed from the "setup.md" the information about the function name, stack name and the s3 bucket because are hard coded inside the makefile

Thank you @pavlovic-ivan for the feedback, for every bug or problem feel free to contact me


Error function, and better error embed.

Creating an Error function, and a better embed for the error.
Old -

New -

Finished -
(4/2/2023) Pending push to main branch!


Allow no enforce sig alg validator

Add noEnforceSigAlg flag to JWTValidator config to allow skipping check on token sig alg when noEnforceSigAlg is configured as true. Added NewConfigurationNoEnforceSigAlg and kept NewConfiguration with noEnforceSigAlg defaulted to true for backwards compatibility.
The check for token sig alg can be safely removed in the following case where:

in a JWKS setting, the kid from the JWK must correspond to one in the JWKS
the secret provider and token secret must correspond

This would allow, in a JWKS setting, to use the same validator to validate requests that may contain JWTs of various sig alg types, corresponding to the the various JWKs with different sig alg types from the JWKS.
See updated test cases to support the pending changes.

Coverage increased (+0.6%) to 89.933% when pulling 89fe52537e99746f25cdad326c55de4383cf94bb on ttjiaa:allow-no-enforce-sigAlg-validator into b3eb4282ad28fd5e22837100dcfd0c8efc5bc7b3 on auth0-community:master.

Looks good to me! Thanks 👍


Support speech_to_speech task in audio_to_audio pipeline

This PR contains the change to enable facebook/textless_sm_cs_en. It's with a separate fairseq task and hub interface. Tests passed on my laptop when I switch TESTABLE_MODELS from facebook/xm_transformer_sm_all-en to facebook/textless_sm_cs_en (Shall we add both for testing going forward?)

Do you mind fixing style with make style  ?

Is there any reason you can't update the model directly on the hub ?
And implement speech_to_text directly for "facebook/xm_transformer_s2ut_800m-es-en-st-asr-bt_h1_2022" ?
I'll approve so we can fix faster in the meantime.
@osanseviero FYI

(1) Good call! Later we could merge S2SHubInterface / S2THubInterface and move the heavy lifting to fairseq side and keep audio-to-audio pipeline generic.
(2) @lpw0 Do you think we could just update cfg in checkpoint? I remember this is just an one-off case
(3) Will Fix make style
Thanks @Narsil! BTW our model loading never finishes, would you mind helping take a look? Described the issue in our slack channel.

Will fix one thing around overrides in 30 mins.


Swipe triggers the onclick event

I am having a imageview with onclick event inside the cards, and the problem, when I click the image view , it works good, but when I swipe the cards, it triggers the onclick event and the swipe is not happened

hmmm please try again with swipedeck 2 and if you have this problem i will fix it there.


feat: Add linting error for assignment expression

Description
This change adds a linting error for direct mutation of widget property like Widget.property = "dsf" and instead suggests to use setter methods.
PR fixes following issue(s)
Fixes #5822
Type of change

New feature (non-breaking change which adds functionality)

Testing

[x] Add jest tests as mentioned in the comments of the tech spec document for this project.
[x] Add test to verify linting error for widget assignment
[x] Add cypress test for autocomplete of more setter methods
[x] Add cypress test for currencyInput setValue

How Has This Been Tested?

[ ] Manual
[ ] Jest
[ ] Cypress

Test Plan
Issues raised during DP testing
Checklist:
Dev activity

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

QA activity:

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

/ok-to-test sha=1d2b02b

/ok-to-test sha=1d2b02b

/ok-to-test sha=1d2b02b

/ok-to-test sha=1d2b02b

/ok-to-test sha=ba7ea2a

/build-deploy-preview env=release skip-tests=true


Will python support be added?

hello, I will switch to the paid version soon, but I couldn't see python support, do you think about adding it?

No, python support won't be added any time soon.


Add support for RabbitMQ

Add a producer and connect to topic for sending to Federation
Add a consumer and connect to topic for receiving from Federation

Topic names are maybe still TBD, but for now I am labeling them based on destination. So "federation" is for any message going to the Federation service and "backend" is for any message going to the Java backend (eg, the service in this repo)

Feel free to reference https://github.com/sublinks/sublinks-federation/pull/28 for the docker-compose.yaml on running RabbitMQ

Branch was merged in closing this issue: 5744955af07a743204a09427abb24a5a67a0dca4


chore(deps): updated graphql-js version to support 0.10.0

Also updated subscriptions-transport-ws version to the latest.

TODO:

[x] Update CHANGELOG.md with your change (include reference to issue & this PR)
[x] Make sure all of the significant new logic is covered by tests
[x] Rebase your changes on master so that they can be merged easily
[x] Make sure all tests and linter rules pass

@DxCx
1 - Done
2 - Why? It thought we want to keep it with the same for query and mutations, but use the ws only for subscriptions (in graphiql)

Yess, this is great! :tada: Can't wait for this to land to try the new subscriptions stuff :blush:

well, you rushed to merge it but this is incomplete:
https://github.com/apollographql/graphql-server/blob/master/package.json#L34
packages will still use 0.9.2 for testing...


Good lando defaults

This is the Symfony version of https://github.com/platformsh-templates/laravel/pull/16

Added to template builder a ways back.

Added to template builder a ways back.


New libvpx version

This PR updates the libvpx binaries to version 1.6.0.
It doesn't address the problem of worsening performance, which is tracked in #86.

I think it's ready for review now.


Refresh Data Prepper content

Description
Refreshes Data Prepper to clarify ingest pipelines and Data Prepper pipelines content and streamline information
Issues Resolved
Closes #6429
Checklist

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dlvenable We're consolidating redundant information across Data Prepper. Will you review this PR at your availability? Changes include deleting pipeline configuration examples in the pipelines.md file and directing reader to the common uses cases and consolidating the component descriptions to one location, their respective pages. Please reach out with any questions. Thank you, Melissa
cc: @hdhalter

Closing to avoid redundant ongoing updates to DP content


"Null" in value column fails on upload

If values in exported files are Null, import fails. Here's a sample file. How do we want to deal with this?
LPG_PK_01_TC_9805_LPGL_20171201_177.lablr.csv.zip

I think we don't support import of null anything (filename, timestamp, value, or label) - if the user has data they want to hide from labeling, just filter out those rows before exporting for trainset


Use field value for finding index since field names may be translated

This fixes array fields not working properly when the names are translated.


Fix jakarta.xml.ws.spi.Provider implementation package

Signed-off-by: Gaurav Gupta<EMAIL_ADDRESS>Seems com.sun.xml.internal.ws.spi.ProviderImpl refactored to com.sun.xml.ws.spi.ProviderImpl earlier but package is not renamed in Provider class.
See the package of ProviderImpl.java.

btw: did you check #90 before creating this?

Hi @lukasj,
I missed that ticket. Thanks for the suggestions, I have updated PR.

What about adding a static overload to Provider that takes the provider class? Then it could be used thusly:
Provider.provider(com.sun.xml.ws.spi.ProviderImpl.class);
As a side note, I believe the com.sun.xml.ws module should export com.sun.xml.ws.spi.


Keras image classifier tutorial issue

Copying and pasting the code from the tutorial into a notebook. Works fine through the "#check the top 5 indices" section. In the next section, "eli5.show_prediction(model, doc)" code results in the following error:
RuntimeError: tf.gradients is not supported when eager execution is enabled. Use tf.GradientTape instead.
Tried adding the following two lines:
import tensorflow as tf
tf.compat.v1.disable_eager_execution()
Error changes to:
ValueError: Tensor Tensor("Mean:0", shape=(None, 1280), dtype=float32) is not an element of this graph.
Help?!

Hi @leszekmp,
I can run the entire notebook if I try it with
import tensorflow as tf
tf.compat.v1.disable_eager_execution()

What is your tensorflow version? I am using tensorflow==2.2.0

I'm also using tensorflow 2.2.0.

Could you share more code that you put into your notebook? I'm not sure what's the issue as the notebook works for me when eager execution is disabled.
I think the error is related to the "weights" variable (mean of all the gradients for each activation map) https://github.com/TeamHG-Memex/eli5/blob/master/eli5/keras/gradcam.py#L120. You can search the notebook for 1280 and you'll see that the model has layers with 1280 activation maps near the end of it.


Update kernelspec environments whenever they're used

For #10255
Force an update to the kernelspec environment whenever possible

Codecov Report

Merging #10324 into master will increase coverage by <.01%.
The diff coverage is 57.69%.

@@            Coverage Diff             @@
##           master   #10324      +/-   ##
==========================================
+ Coverage   61.26%   61.27%   +<.01%
==========================================
  Files         570      570
  Lines       30961    30968       +7
  Branches     4441     4442       +1
==========================================
+ Hits        18969    18975       +6
+ Misses      11004    11001       -3
- Partials      988      992       +4

Impacted Files
Coverage Δ

...ient/datascience/jupyter/kernels/kernelSelector.ts
77.59% <100%> (+1.79%)
:arrow_up:

...lient/datascience/jupyter/kernels/kernelService.ts
55.19% <52.17%> (+2.64%)
:arrow_up:

src/client/common/utils/platform.ts
64.7% <0%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0%> (-0.73%)
:arrow_down:

...c/datascience-ui/react-common/settingsReactSide.ts
22.22% <0%> (ø)
:arrow_up:

src/client/datascience/webViewHost.ts
60.54% <0%> (ø)
:arrow_up:

src/client/datascience/types.ts
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Add flag to avoid setting ports

Some configurations can't handle dynamic port setting, rely on
hard coding ports, and/or don't need a port. In these instances the
use of "Starting X on port " is confusing when the port is not
actually being set for this value.
Add a new --set-ports flag which you can set to False to disable port
setting, and the subsequent logs.

Please resolve conflict

Done!


Add Elkarte to forummatrix.org

Hi, is it possible to add Elkarte to the forummatrix.org, so people can get a better understanding of the matrix features and better decision for comparison, here is the link to use:
https://www.forummatrix.org/static/new-product
I tried to contact people at forummatrix but they require someone from the team behind Elkarte.

IIRC I already tried to submit it few years ago but I'm still waiting an answer.

I did the same ... I had submitted the details a few years back with no response.  Looks like it might have changed hands since then?

Hi, ForumMatrix Maintainer here. We relaunched the site a few months ago and would be happy to have you. What I would need is the email addresses of all users (you need to register first) that shall have maintainer permissions for the entry. Just shoot me a quick mail at<EMAIL_ADDRESS>PS: sorry for not responding a few years back. The site was dormant for quite a while.

@Spuds did you by chance send the email?
ohhh @splitbrain as in dokuwiki! I didn't realize. It has been probably the first open source project I have "contributed" to (inherited a plugin and maintained another or something similar) ages ago. :smiley:

No .... I forgot 😳

@Spuds you remember yet? : )

Spuds, if you want I can do this, it is possible to add more than one user who can maintain the data


Add format weighting ab test

We are going to be running an A/B test to test changes to format
weighting.
The meta tags have been added to main because they do not seem to be
visible to the tests when in the headers. We think this has something
to do with Slimmer. Since these meta tags are only there for a short
time we feel this is ok.
Paired with @suzannehamilton
https://trello.com/c/JJ3mIOP1

@MatMoore: That makes sense. I've updated the names. 👍

I've just moved the A/B meta tag from <body> to <head>. The tests will fail until #318 is merged, because it fixes the header rendering.

Also, the description doesn't reflect the code which is confusing and we should mention the dependancy on https://github.com/alphagov/finder-frontend/pull/318/files being merged


RemoteExecutor class implementation

Acceptance Criteria

[ ] Implement RemoteExecutor according to Design Details section
[ ] Add docstrings to RemoteExecutor class
[ ] Add unit tests for RemoteExecutor class

Design Details (RemoteExecutor Class)
Executors which involve execution on a remote machine inherit from AsyncBaseExecutor.  Note that ultimately the synchronous sibling BaseExecutor will only remain for end-users to write their own executor plugins more easily.

Class attributes:

poll_freq — Number of seconds to wait between polling for task status after it has been submitted
remote_cache — Location where pickled inputs/outputs are placed. In the case of the SSHExecutor, this is a directory on a filesystem; for AWSExecutors (see below) it’s an S3 bucket name.
credentials_file — Location where credentials can be found.  Can be either an RSA public key or cloud credentials, or generally something that will be referenced by a child class’s _validate_credentials definition.

Abstract class methods:

_validate_credentials — Validate user credentials
_upload_task — Uploads the pickled function to the remote cache
submit_task — Invokes the task on the remote backend. Returns some sort of task UUID defined on the remote, e.g., a Slurm Job ID or an AWS task ARN.
get_status — Returns the electron status by querying the remote backend
_poll_task — Polls the remote backend until the status is either COMPLETED or FAILED
_query_result — Retrieves the pickled result from the remote cache
cancel — Sends a cancellation request to the remote backend

Class methods:

run_async_subprocess — Invoke an async subprocess

Current plugins which can inherit from this class:  SSH, Slurm, ECS, Kubernetes, AWSBatch, AWSLambda, Braket, QiskitRuntime

Closing as duplicate of https://github.com/AgnostiqHQ/covalent-staging/issues/578


Populated ApplicationProperties causes error when sending

Hi,
I found that when setting application properties on a message, I was running into a assignment to entry in nil map error.
I've forked the repo and fixed the issue which is just adding one line to initialise the service bus message application properties as below
In sender.go:
func mapMessageToSeriveBus(message *Message) *azservicebus.Message {
	sbMessage := &azservicebus.Message{}
	if message.ApplicationProperties != nil {
		sbMessage.ApplicationProperties = make(map[string]any)
		for k, v := range message.ApplicationProperties {
			sbMessage.ApplicationProperties[k] = v
		}
	}
      ....
}

Thanks in advance!
Example of a message I was using that encountered the error
import { sleep, check } from 'k6';
import { ServiceBus } from 'k6/x/azservicebus';
import { uuidv4 } from 'https://jslib.k6.io/k6-utils/1.4.0/index.js';

export let options = {
	vus: '1',
	duration: '5s',
};

var jsonBody = '{ "Title": "Example" }';

export function setup() {
}

const config = {
    connectionString: "<connectionString>",
    timeout: 30000,
    insecureSkipVerify: true, // the docker servicebus emulator uses a self-signed cert
};

const servicebus = new ServiceBus(config);
const sender = servicebus.createSender('<topic>');
const receiver = servicebus.createSubscriptionReceiver('<topic>', '<subscription>');
const batch_message_count = 2;

export default function () {
    const data = [];

    for(let i = 0; i < batch_message_count; i++) {
        var properties = {
            'Id': uuidv4(),
            'Type': "MessageType"
        };

        data.push({
            applicationProperties: properties,
            contentType: 'application/json',
            subject: 'example',
            bodyAsString: JSON.stringify(jsonBody)
        });
    }

    sender.sendBatchMessages(data);

    sleep(1);

    const messages = receiver.getMessages(batch_message_count);
    check(messages, {
        'Has the expected size': (m) => m.length === 0,
    });
}

export function teardown() {
    receiver.close();
    sender.close();
    servicebus.close();
}

P.S, happy to do this as a PR, but I'm not able to push to the repo

Thank you!


Fix new includes

Codecov Report

Merging #129 (517966a) into foxy (e63896d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             foxy     #129   +/-   ##
=======================================
  Coverage   48.01%   48.01%
=======================================
  Files          39       39
  Lines        1487     1487
  Branches      303      303
=======================================
  Hits          714      714
  Misses        567      567
  Partials      206      206

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 e63896d...517966a. Read the comment docs.


Add clientEndpointType to interface

Description
Add interface to support generating MQTT JWT Token for Azure WebPubSub. Essentially we need to identify if clients want to establish a default WebPubSub connection or an MQTT connection, then we will generate the service URL for them accordingly.
Example Service URL for:
Default Connection: wss://exampleHost.com/client/hubs/exampleHub
MQTT Connection: wss://exampleHost.com/clients/mqtt/hubs/exampleHub
All SDK Contribution checklist:

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

General Guidelines and Best Practices

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

Testing Guidelines

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

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-messaging-webpubsub

Please run a regen via swagger/readme.md
At least, the ServiceVersion class would need a new api-version
https://github.com/Azure/azure-sdk-for-java/blob/4ca954f1abf9165b4589f9f7f7cc351e9b92160b/sdk/webpubsub/azure-messaging-webpubsub/src/main/java/com/azure/messaging/webpubsub/WebPubSubServiceVersion.java