Support Polars .with_columns(...) and Polars .select(...)

Polars with_columns is nearly identical to Pandas assign.
E.g. instead of this:
import yfinance as yf
from mintalib.indicators import SMA

# fetch prices (eg with yfinance)
prices = yf.Ticker('AAPL').history('5y')

# convert column and index names to lower case
prices = prices.rename(columns=str.lower).rename_axis(index=str.lower)

# compute and append indicators to prices
result = prices.assign(
    sma50 = SMA(50),
)

You can do this:
import yfinance as yf
from mintalib.indicators import SMA

import polars as pl
from polars import col

# fetch prices (eg with yfinance)
prices = yf.Ticker('AAPL').history('5y')
prices = pl.from_pandas(prices)  # Convert Pandas df to Polars df.

# compute and append indicators to prices
result = prices.with_columns(  # Add columns to dataframe.
    sma50 = col('Close').SMA(50),  # Calculate the 50 day SMA from the daily close values.
)

Hi @ddouglas87,
The closest you can do out of the box I believe is using map_batches.
The following seems to work for me.
result = prices.with_columns(
    sma50 = col('close').map_batches(SMA(50)),
)

There may be a way to create a custom accessor say ta that would allow for a simpler syntax like col('close').ta.sma(50). I am tot sure whether this would be warranted/useful ?

I had no idea .map_batches(...) was a thing.  Very cool!

There may be a way to create a custom accessor ... I am not sure whether this would be warranted/useful ?

Polars has plugins that would let you do that, but I'm not sure if Python plugins can be made or they have to be made in Rust.

If you put this .map_batches example in the README.md it might help boost the popularity of your library.
And thanks for the response!  It's great learning something new and useful.  ^_^

Thanks for the feedback! I added an example with Polars to the README. Cheers


refactor: update V2V3ProjectActivity->Select to use JuiceListbox

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3553

PR #3554

PR #3555

PR #3556

PR #3557

PR #3558   👈

This comment was auto-generated by Graphite.

@wraeth-eth queued this pull request to merge with Graphite.

@wraeth-eth merged this pull request with Graphite.


[4.0] SILGen: Project the referent type of property components in key paths.

Explanation: The compiler would crash when a key path referred to a weak or unowned property.
Scope: Without this fix, key paths won't work with weak/unowned properties.
Issue: SR-5408 | rdar://problem/33215221
Risk: Low, one-line fix isolated to key paths.
Testing: Swift CI

@swift-ci Please test

@slavapestov Does this look OK for 4.0?

That all ought to fall out, since with this implementation the key path is going to use accessors to work with the property. I can add some runtime tests.


(winscp) Wrong release notes link

Expected Behavior
Release notes for the WinSCP package should link to release notes for the version of the package.
Current Behavior
For https://chocolatey.org/packages/winscp/5.11.1 release notes link is https://winscp.net/download/WinSCP-5.9.6-ReadMe.txt
Possible Solution
Update release notes link in the AU script
Steps to Reproduce (for bugs)

See https://chocolatey.org/packages/winscp/5.11.1

Your Environment

Package Version used: 5.11.1

I believe this is related to AU bug that was resolved (#853). It shouldn't happen in the next release. I will just leave this version like this.

Force test looks good: https://gist.github.com/gep13/b733b29e320a8c05bc99cc0bf59cd8d1/6bd782cdc535dd7dab448ee691a605ce75e1ab65#winscpinstall


fix: check for file existence before unlinking

This PR resolves []
Description

Add check for file existence before unlinking to prevent a red screen.
PR Checklist

[ ] I have tested my changes on iOS and Android.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Need help with something? Have a look at our docs, or get in touch with us.

ty!


Add ShowValidationErrorOnKeyboardFocus property to validation popup

Describe the changes you have made to improve this project
Add new dependency property ShowValidationErrorOnKeyboardFocus to the validation popup. Also add an attached property to the ValidationHelper.
With this new properties it's possible to hide the popup globally or for a specific control. The validation can then be shown with the ShowValidationErrorOnMouseOver property.

Closed Issues
Closes #4086

I was planning to do this beginning of next year. Sorry for the delay.
Happy coding
Tim


sampled+evaluated gpt4o, reran plotting notebooks

This PR contains:

[ ] a new test-case for the benchmark

[ ] I hereby confirm that NO LLM-based technology (such as github copilot) was used while writing this benchmark

[ ] new dependencies in requirements.txt

[ ] The environment.yml file was updated using the command conda env export > environment.yml

[x] new generator-functions allowing to sample from other LLMs
[x] new samples (sample_....jsonl files)
[x] new benchmarking results (..._results.jsonl files)
[ ] documentation update
[ ] bug fixes

Related github issue (if relevant): closes #0
Short description:

I sampled+evaluated gpt-4o, which was released this week.
I also tried sampling gemini 1.5, but failed due to rate limits.

How do you think will this influence the benchmark results?

gpt-4o is the new leader in our benchmark (pass@1=0.51+-0.41), it is a little bit better than gpt4 turbo (pass@1=0.47+-0.38), the former leader.

Why do you think it makes sense to merge this PR?

We should include new models when they come up and there is a substantial interest by the community to benchmark them. As gpt-4o is hyped and promised to be better than gpt-4-turbo, it makes sense to include it in our benchmark.

This should not be merged yet, as the paper text wasn't adapted yet.

Adding a note on costs: benchmarking gpt-4o costed $2.73


[MOD] Configurable community "attenuator"

Describe the solution you'd like
Allow users to reduce the number of posts they see from any number of communities without blocking them entirely. My solution is to randomly hide posts from communities based on a per-community user-configurable factor. Since KES doesn't support adding settings fields on the fly (which would be awesome), they'd be configured as a comma separated list.
For example<EMAIL_ADDRESS>0.4<EMAIL_ADDRESS>0.25 would hide 60% of posts from 196 and 75% of posts from memes, just by comparing Math.random() to the factor set by the user. One advantage this has over deranking posts (which I don't think would be possible to implement without modifying kbin itself) is that there would be more variety on each refresh of a user's home page. However, you wouldn't be able to eventually see every post from those communities if you keep scrolling, if they are hidden randomly rather than deranked.
Additional context
https://kbin.social/m/kbinMeta/t/193730/Is-there-a-way-to-derank-certain-magazines-on-user

Ah whoops, just saw that you created that issue, I was in the middle of writing this lol

support adding settings fields on the fly (which would be awesome)

This seems dangerous, when would you use this?

I guess it'd be used when you want a user-definable array of options that they can add rows to. RES's account switcher is a good example of what I'm talking about (but it doesn't necessarily need to be as complex):

Right. It's not...impossible...just a lot of refactoring for potentially small benefit. I feel like we shouldn't drift too much from the core design logic of: manifest is immutable and defined a priori, mods can do whatever but mustn't mess with KES's menu itself. Maybe if it becomes necessary at some point.
Besides, is having a largeish text area that you can keep adding to demonstrably worse than click, add row, select input, type, move mouse again, click, add row, etc.? Other than prettifying it.


Attachments when sending mail do not seem to work like described in the FAQ

So I needed to apply for a job and attach my curriculum.
According to the FAQ in the README of Himalaya I can do this like so:
<#part filename=/path/to/file.pdf><#/part>

So in my application email I did the following at the end of the text:
CV:
<#part filename=/home/lk/Downloads/Lennart_Kiil_CV.pdf><#/part>

The email was sent to the employer. But without the attachment!
Instead the e-mail looked like so:
Mange tak og venlig hilsen,

Lennart

CV:
<#part filename=/home/lk/Downloads/Lennart_Kiil_CV.pdf><#/part>

So the "embedding" is just output as plain text, apparently.
How can I actually send attachments with Himalaya?

Before replying to you, I tested the feature and it works as expected.

So in my application email I did the following at the end of the text:

Which email application did you try it?
The <#part filename=…><#/part> syntax only works inside Himalaya usage, when using one of the following command:

himalaya message write
himalaya message reply
himalaya message forward
himalaya message edit

As described in the FAQ, these commands will spawn a buffer in your main text editor defined in $EDITOR. After edition you will be prompted what to do with this message.

@soywod Yes, when doing it like that, it does work :)
I need to do it programmatically though, like this
himalaya message send "TO<EMAIL_ADDRESS><EMAIL_ADDRESS>filename=/home/lk/Downloads/Lennart_Kiil_CV.pdf><#/part>"

Is that somehow possible?

You can achieve it via the template subcommand:
himalaya template send "TO<EMAIL_ADDRESS><EMAIL_ADDRESS>filename=/home/lk/Downloads/Lennart_Kiil_CV.pdf><#/part>"

The difference between himalaya message {save,send} and himalaya template {save,send} is subtile: the first one saves and sends raw messages whereas second one saves and sends templates (containing MML markup you are using). They are compiled into valid MIME message before sending.

Thank you so much!
Both for the support and for making himalaya.


ELBv2 - Simplify tests

FYI @bpandola - this should make it a bit easier to identify why the flaky ELB test fails

@bblommers Thanks!  I figured it out with the help of the additional output from your commit.  I've got a fix here: https://github.com/spulec/moto/pull/4002


Anti-rules

From @bradeneliason :
2) Nonequivalence relations. It is possible to represent nonequivalence edges on the e-graph – anti-rules. The rules are transformations that are always true. An anti-rule would fire on sets of terms that are always false. For example: even ≠ odd. If ever you reach a state in your e-graph where two terms linked by an anti-rule are in the same e-class, then something has gone wrong and you can stop trying to saturate. I could see this used in a theorem prover to perform proof by contradiction. Perhaps anti-rules can be created with the current implementation of egg/MT.

@bradeneliason added this feature:
failme = @theory begin
    p ≠ ¬p
    :foo == ¬:foo
    :foo => :bazoo
    :bazoo => :wazoo
end

g = EGraph(:foo)
report = saturate!(g, failme)
@test report.reason == :contradiction
@test !(@areequal failme foo wazoo)

I'm impressed with how quickly and responsively you've been implementing ideas. Thanks!
I'll have to play around with this later to see if can generate a proof by contradiction!
If you plan on implementing <, ≤, >, or ≥ relationships in the future, :inequality may not be the best choice. Perhaps :strictinequality or :unequal.


ImportError: cannot import name 'XPath'

XPath去哪啦？？？看起来你好像移除了xpath？？？

Why was this closed without any resolution? This seems like a valuable problme to fix.

@gaojiuli was XPath removed?


METRON-1510: Update Metron website to include info about github update subscription

Contributor Comments
Update to include links for the 'Issues' mailing list. Have also changed Archives links to point to lists.apache.org, which has a better viewing than the earlier one.
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[ ] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[ ] Have you included steps or a guide to how the change may be verified and tested manually?

[ ] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[ ] Have you written or updated unit tests and or integration tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[ ] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

+1 by inspection.  Thanks @anandsubbu !


chore(share_plus)!: Update multiple dependencies, bump min Flutter to 3.0

Description
Updating a set of dependencies for share_plus as it seems that Dependabot didn't really bump dependencies and was fine with constraints we had specified for the plugin.
We also have a bit lower score on pub.dev due to not having latest versions: https://pub.dev/packages/share_plus/score
Unfortunately, I couldn't bump win32 to 4.x versions as path_provider used in platform interface still uses old win32 version.
This PR is a breaking change because a few dependencies, like cross_file bumped their min Flutter version to 3.0: https://pub.dev/packages/cross_file/changelog
As it wasn't only one dependency that did so, I decided to update our min Flutter version as well.
Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I titled the PR using Conventional Commits.
[x] I did not modify the CHANGELOG.md nor the pubspec.yaml files.
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze) does not report any problems on my PR.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[x] Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
[ ] No, this is not a breaking change.

I wonder if it would be enough to bump the Flutter version and let dependabot take care of the rest. :thinking:

I wonder if it would be enough to bump the Flutter version and let dependabot take care of the rest. 🤔

I think Dependabot will do nothing after reading about some issues today: https://github.com/fluttercommunity/plus_plugins/pull/1593#issuecomment-1454704145

Unfortunately, I couldn't bump win32 to 4.x versions as path_provider used in platform interface still uses old win32 version.

I have a feeling that this is the only version constraint that causes pub.dev to reduce points. This is the only constraint that creates a conflicts if apps try use the latest available package version together with plus plugins. All other version constraints are compatible with the latest package versions.

Alright, I think that since we can't update win32 and meta anyway we won't get score bump, so pushing min version of Flutter might too much at this point. Thus, closing the PR till bumping min Flutter version is justified (for some bug fix or when we can do win32 update).


Add hidden to the defaults, the userDefaults, and the autogeneratd stubs

Summary of the Pull Request
See title. On first launch, profiles.json now looks like:

PR Checklist

[x] Closes #2795
[x] I work here
[n/a] Tests added/passed
[x] Requires documentation to be updated

/azp run

@carlos-zamora eh, I think it's fine just having the description in the documentation. The key's presence there should be enough to clue someone in that the key does something. They can go read the docs if they want.
(also I don't know how to add a comment in jsoncpp code directly. I know it's possible, but that'd probably push this change back a few days)


Actively notfiy a job of an abort request

With PR #1081, we added a feature to mark a currently running job for abortion, but we have to poll the database in the task ourselves to determine whether this mark was set (represented by the aborting status). As discussed in #511, it would be nice if the task was actively notified of an abortion (without the need for intense database polling) using a PostgreSQL listen-notify. We could then raise an asyncio.CancelledError in an async task. It's unclear what to do with a sync task (maybe a callback method like context.listen_abort(cb: Callable?!). It should be optional and triggered by something like job_manager.cancel_job_by_id(33, abort=True, notify=True).
It would be best implemented after the worker refactoring #933, as it must be integrated into the worker class.

It would be best implemented after the worker refactoring https://github.com/procrastinate-org/procrastinate/issues/933

Or be part of the design of #933

If we do #1083, is this worth the complexity ? I won't close this but I think it's interesting to spell out the gains we expect from this before tackling it.

In some cases, it could simplify the abortion process and save the user some code (e.g., no need for another asyncio task that monitors the abort status). On the other hand, it's not a feature that needs to have high priority or be integrated right now as part of the worker refactoring (the refactoring is complicated enough anyway). It's more a nice to have feature and something I have not seen in any other distributed task queue library. So I would say, let us leave this issue open and see what the future brings. Regarding #1083, it's the next thing I'll deal with.

Closed as part of v3


FIX: honor OMP_NUM_THREADS environment variable

Merges #876 into #848 for testing purposes.

This demonstrates that #848 can be merged. It doesn't change outputs beyond what is necessary to restore CI to working order.


Xhamster plugin has stopped working

I believe they have updated the player, any help would be amazing please.
Oh also thanks for such a great bit of proxy software, do you have anywhere to make donations to help support future builds ?

https://github.com/Athlon1600/php-proxy-plugin-bundle/commit/402b9e47870216b780bac76df8941bc2065d8dc3
Should be working now. Try it using the San Francisco server from:
https://unblockvideos.com/
There is no need for donations. Giving credit to this project is more than enough :)


RDEV-2444 added interfaces, which can be used as mocks in the tests.

Interfaces and consumerParams changed to params


Are Lsv2-series NVMe disks covered by Host-based encryption?

The document says that "NVMe disks not encrypted", but what does that mean? I understand that they are not encrypted by default, but do they support Host-based encryption?
Right now it is very unclear if enabling host-based encryption on Lsv2-series NVMes in Azure Kubernetes Service means that those disks are encrypted or not. Please update the documentation so that it explains how this actually works.

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

ID: ade067f1-3f3e-73de-0321-bb8d07147cd3
Version Independent ID: e8dc38c4-0114-b7c2-887f-e1d2810206e8
Content: Server-side encryption of Azure managed disks - Azure Virtual Machines
Content Source: articles/virtual-machines/disk-encryption.md
Service: virtual-machines
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

@grove -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

@grove - They do not support host-based encryption and it is explicitly mentioned in the document.
At this moment, Since there is no doc update required, We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Bzlmod support

Issue type
Build/Install
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
Latest
Custom code
No
OS platform and distribution
Linux
Mobile device
No response
Python version
No response
Bazel version
6
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Please support bzlmod by adding a MODULE.bazel file . See the migration guide.
Standalone code to reproduce the issue
This file currently doesn't exist.

Relevant log output
No response

@mering,
Thank you for the request. Could you please elaborate about the Feature. Also, please specify the Use Cases for this feature. Thank you!

@tilakrayal The current Bazel WORKSPACE approach has several flaws which are fixed by the migration to Bazel modules (e.g. not requiring dependants to specify all transitive dependencies). I can't describe it better with my own words than the Bzlmod user guide.

The team is working on adding bzlmod support, but since this is a big project this will take significant time

Do you have any info on the status of this feature ? Would love to have Bzlmod support to import TensorFlow seamlessly.

Does anyone have a workaround to import and build TensorFlow from a Bzlmod project ? I created a StackOverflow for this matter but haven't go any response yet : https://stackoverflow.com/questions/78413217/import-tensorflow-as-bazel-dependency-for-c-project

Is there any update?👀
This is an important feature, I am willing to contribute my effort if possible


add script that runs prover on sui framework

This draft PR contains a script that runs sui move prove on sui framework Move code. It would be great if we can run this in CI as a sanity check that the changes in a PR don't break sui move prove.
This script is adapted from the script in Move repo that runs diem framework in CI:
https://github.com/move-language/move/blob/main/language/documentation/examples/diem-framework/prove_all.sh
In order for sui move prove to work, we need to install a few dependencies including dotnet, Boogie, Z3, CVC5. We can probably do something similar to what the move repo does in github actions:
https://github.com/move-language/move/blob/main/.github/actions/build-setup/action.yml#L45-L60

@emmazzz Do you think we should include this as part of the cluster tests we are planning on running in continuous environment? Example cargo run --bin sui-cluster-test -- dev-net

@emmazzz Do you think we should include this as part of the cluster tests we are planning on running in continuous environment? Example cargo run --bin sui-cluster-test -- dev-net

The test in this PR is more like unit tests for the Move contracts. I don't think it's necessary to test this in a cluster setting, unless I misunderstood the idea of cluster test.


GCP Director gets stuck on post-start scripts

Hi,
I'm trying to start a bosh director on gcp using the sample manifest provided, and when I run the "bosh create-env" command it completes all the way up to the "post-start scripts" where it times on connecting to the credhub server during the credhub job with the error:
Sending 'get_task' to the agent:
      Agent responded with error: Action Failed get_task: Task 4c2527c0-293b-4b8f-5f77-29f46cd50848 result: 1 of 2 post-start scripts failed. Failed Jobs: credhub. Successful Jobs: uaa.

I also tried deploying the AWS director manifest, which worked fine, but I need this to work through GCP. Are there any certain settings that need to be set for GCP in order for the post-start script to work?
Thanks!

Hello -
Have you checked the deployment logs? An easy way to do that is by running grep -A 10 ERROR /var/vcap/sys/log/credhub/credhub.log on the machine. The errors are often helpful, but there is a deployment troubleshooting guide here that may also be able to help.

Figured it out, was an issue with an old certificate value. Thanks!!


Non-input elements inside FormlyFieldConfig (text and image)

Is it possible to include non-input elements inside FormlyFieldConfig and update them based on some input field?
Example: user inputs a product code and, after fetching the database, some info should be displayed inside the form (ex. description and image).
Formly version: 5.12.6
Thanks.

Is it possible to include non-input elements inside FormlyFieldConfig and update them based on some input field?

yes possible, and there are multiple ways:

template (https://github.com/ngx-formly/ngx-formly/issues/3287#issuecomment-1118815052, https://github.com/ngx-formly/ngx-formly/issues/2057#issuecomment-579724475)
inline type (v6 only) https://github.com/ngx-formly/ngx-formly/issues/2086#issuecomment-584847953
custom type


[otbn] add initial assembly for ECC P-256

Adds the semi-automatically converted library for
ECC with NIST P-256 bit curve.
We will go through the same excercise as with RSA/modexp
and document and optimize this code for OTBN in follow-up
commits.
Signed-off-by: Felix Miller<EMAIL_ADDRESS>

Once we have the calling conventions documented we should really get an end-to-end test for this into the tree (similar to #4071 for RSA) to ensure that refactorings don't break things.

To get CI to pass you need to add a line to util/licence-checker.hjson.

Thanks Philipp, added the line for the license checker.
Re end-to-end tests: tracking this here now #4206


Set security at a resource class level? As @Api(authorizations = ..) did before. (swagger.core.v3)

Q
A

Bug or feature request?
Request

Which Swagger-Core version?
io.swagger.core.v3

Which Java version?
8

Which JAX-RS framework & version?
Glassfish, Jersey, 2.25.1

Previously I could set security on a resource class level (example: @Api(authorizations = ...)). Would it be possible to add this capability with swagger core v3? Instead of having to set it in each @Operation(security = ..).

Hi,
In current snapshot version (io.swagger.core.v3:swagger-jaxrs2:2.0.0-SNAPSHOT, please notice the groupId change) available on sonatype, or in oncoming rc, you can annotate your resource class with one or more @SecurityRequirement to have them added to operation; see https://github.com/swagger-api/swagger-core/blob/2.0/modules/swagger-annotations/src/main/java/io/swagger/v3/oas/annotations/security/SecurityRequirement.java#L26-L34

Fantastic! Thank you :)


feature: prepend_args support for "nvim-lint"

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
If I want to specify extra args for linters, there's no provision for doing that. For conform.nvim, we have prepend_args and LazyVim merges the passed opts with the defaults.
Describe the solution you'd like
As it is with formatters, support for this would be great. For example, since nvim-lint does not have a prepend_args key, we could reserve this key just for LazyVim where the config() function extends the default linter with the passed args.
This would result in a clean config on the user side like this:
  {
    "nvim-lint",
    opts = {
      linters_by_ft = {
        sh = { "shellcheck" },
      },
      linters = {
        shellcheck = {
          prepend_args = {
            "-x",
          },
        },
      },
    },
  },

Describe alternatives you've considered
https://github.com/mfussenegger/nvim-lint/issues/397 discusses this, but when I assigned a function for shellcheck doing the same list extension, it did not work as the function overrides the default linter table.
Additional context
For now, in my config I have the following if condition to implement this feature.
      local lint = require("lint")
      for name, linter in pairs(opts.linters) do
        if type(linter) == "table" and type(lint.linters[name]) == "table" then
          lint.linters[name] = vim.tbl_deep_extend("force", lint.linters[name], linter)

          if type(linter.prepend_args) == "table" then
            vim.list_extend(lint.linters[name].args, linter.prepend_args)
          end
        else
          lint.linters[name] = linter
        end
      end
      lint.linters_by_ft = opts.linters_by_ft

Added!


[ListDivider] getInitialState(): must return an object or null

Uncaught Error: Invariant Violation: ListDivider.getInitialState(): must return an object or null
Introduced in: 0.14.0

Mm, my initial thoughts maybe this is happening because getInitialState() isn't returning an object here.

Separate note:
We should also change index.js to export ListDivider and MenuDivider as Divider so that only you only get the deprecation warning when you import from them directly under lib.

Yes, it's a fairly quick fix to replace occurrences of ListDivider with Divider,  but deprecation warnings shouldn't break things to start with :smile:  It strikes that, especially with the rename / restructure for many components, we need to test old and new ways of importing components in parallel until the old is removed.
I agree on index.js, although hopefully no-one is using that in production until tree-shaking becomes reality!

@mbrookes Totally agree, it's deprecated but it's supposed to work just as it was before. The fix should be easy, I'll submit a PR and test tonight. It could make sense for us to release a 0.14.1 in the near future after we fixed regressions for components that have them.

same
@v0.32.0


Image previews are not displayed correctly.

Describe the bug
Image previews on preview pages are not keeping their aspect ratios. Seems like there's a hard limit for height but not for the width.
To Reproduce
Steps to reproduce the behavior:

Create a new paste,
Upload a picture,
Uploaded picture is distroted on the preview.

Expected behavior
Picture displays and scales properly.
Screenshots
Pardon the spiderman meme. It's the only overly long pic I had on hand 😄.

Desktop 1:

OS: Win10 LTSC
Browser Firefox 115.0.1 and Ungoogled Chromium 114.0.5735.199

Desktop 2:

OS: Fedora Workstation 38
Browser Firefox 115.0 and Ungoogled Chromium 114.0.5735.199

Thanks for reporting this! I will look into it in a bit. Last time I had this issue, it was on my iPad in Safari but haven't seen it since. Should be a pretty easy fix if it's just some CSS issue.

I'm seeing the same issue, aspect ratio is meesed up on the preview.
The other odd thing is when I click on one of the key s that is an image, the first few lines of the image are displayed, then a loooong pause, then the rest of the image previews.

Regarding the slow image preview loading, that's certainly a CPU performance issue on the server as it's serving the image line by line, but this might be improved in the next release because I will change how unencrypted file attachments are served


Fix issue #146

Coverage decreased (-0.3%) to 93.972% when pulling 765cf0d2da66130450a7ca2dacb80a6d7c50c98d on issue-146 into df69a9d6d7987939208a74fdef8a2ec6b36c7b20 on master.


problem using array mapping in flow input/output mapping

Current behavior (how does the issue manifest):
Today we use $.xxx to get the current scope data, for example, we get the current array loop element by using $.elementName.
We are also using $. to get current scope data for the trigger to flow or flow to trigger mapping.
There will be an issue when using array mapping in flow input/output mapping, it's difficult to know $.xxx is getting from array scope or trigger scope.
Expected behavior:
Need a better way to handle array mapping in trigger output mapping or flow output mapping.
Minimal steps to reproduce the problem (not required if feature enhancement):
Please tell us about your environment (Operating system, docker version, browser & web ui version, etc):
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.X.X
Additional information you deem important (e.g. issue happens only occasionally):

Should we come with a way to reference the outer scope within the loop?  $something?  @mellistibco any thoughts?

@fm-tibco @mellistibco Currently we have $loop[loopName].xxxx to get the parent loop(out loop) data.
@foreach($.field.addresses, loopName)

It's better that we can try to use $loop as well.   to get currect loop elements, we can try to use $loop[.] or some name which more specific to current loop.

We would just do $loop for the current loop, and $loop[name] to specify a particular loop... the $loop[.] looks odd . @mellistibco  thoughts?

I tend to think that we should make it as simple as possible for accessing the current loop, so just $loop seems to make sense. If you're explicitly trying to access an outer loop, it's ok to reference it as you both suggested.

thanks, @mellistibco and @fm-tibco, I will go with $loop.xxx for the current loop.


added some generic troubleshooting info

There was an error while validating job d15aab75-8003-4111-9e17-3d9a6e6443d2 Message:Error clonning repositoryDetails:System.ArgumentException: Error clonning repository ---> System.Exception: Error when trying to initialize shared repository ---> LibGit2Sharp.NotFoundException: Branch 434 not found. Available branches:
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetBranch(Repository repo, String name)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass3e.b__3d()
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.<>c__DisplayClass1.b__0()
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction[TResult](Func1 func)    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction(Action action)    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.InitializeSharedRepository(String url, IDictionary2 branches, String sharedRepoPath)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.VerifySharedRepository(String url, IDictionary2 branches, String tempPath)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.VerifySharedRepository(String url, IDictionary2 branches, String tempPath)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository()
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetTreeAsync(String path, Boolean includeSubDirectories, String excludePath, IList1 extensionWhiteList)    at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.ListSourceControlFilesCommand.Do(IDictionary2 context)
at Microsoft.LightweightJobEngine.Core.Language.JobWorkflow1.RunCommands(Guid jobId, IDictionary2 context, CancellationTokenSource cancelSignal)


_process(delta) not called

Godot version 3.0
Windows 10/x64
I defined a class from Node. I use set_process(true) in _ready() constructor. But the method _process() is not called. WTF?
extends Node var idle = 0.0 func _ready(): 	set_process(true) func _process(delta): 	idle += delta 	print(idle)

Can be print instruction dissabled in project settings
Can be Node not added to the scene or to Autoloads
Can be a lot of things, but not a godot bug.  I compile godot from 2.0 and this bug never happened,

Print instruction enabled in project settings. I can check idle from other point.
Node and class added to the scene or to Autoloads
Not working :-(

Upload your project

do you expect it's running in editor without running game?

Is your node in the game tree ? (you used add_child in your game code)

Boys and girls. I cannot post all the project code in which this situation arose. On one side, it contains a very huge and very interesting code that I am not ready to share with the world. On the other hand, 99% of the code is a calculation that does not affect the game cycle, the display, or anything else.
But the fact remains. Apparently this is a bug. But I don’t understand how to cope with it, or how to give a hint to the developers.
Tnx to all

Ваш узел находится в дереве игр? (вы использовали add_child в коде игры)

No, this is a separate class. This class contains several variables that are read from interface elements that are elements of a tree (scene).
But the class itself functions separately from (above) the tree of visual elements. This is a child class of Node.
I made a test project with a similar organization. And it all works. I really don't understand. The difference between a test and a real project is only in the presence of computational procedures and functions that are not directly related either to the physics flow or to other flows. I actually planned from _process (delta) to call triggers initiating calculations.

I will try to consistently kill the functions until I stumble on the source of the problem. If I find such a source, I will describe it here.
Thank you friends for participating

Sorry, but no one can help you if you don't give us code to work with. You could make a separate project reproducing the problem. But as @Ranoller said, I've never came across _process not being called and one would definitely notice that.

But the class itself functions separately from (above) the tree of visual elements. This is a child class of Node.

It sounds to me like that node is not added to the tree anywhere, which in fact is required for processing to work.

...and you don't need to set_process(true) in _ready anymore since 3.0.

No, this is a separate class. This class contains several variables that are read from interface elements that are elements of a tree (scene). But the class itself functions separately from (above) the tree of visual elements. This is a child class of Node.

This is not what I meant. If you create a node but it is not added to the actual SceneTree, like instantiating a node without using add_child, this function will never be called. See https://docs.godotengine.org/en/3.0/getting_started/step_by_step/scene_tree.html

@groud Yes! Solved!
I added a class instance to the root of the tree add_child. All is decided! It all worked!
Thank you so much!

PS Surprisingly another. Here is the test file. In it is the skeleton of my program. In contrast, it has many purely computational operations. And it works. But my code worked only after the announcement "add child" in the root
Test.zip

Here is the test file. In it is the skeleton of my program. In contrast, it has many purely computational operations. And it works. But my code worked only after the announcement "add child" in the root

I am not sure about what this means, but without a reproduction test case it is going to be hard to tell.
Anyway, since the problem is solved I close the issue then :)

@groud Thank you! You saved 6 months of my work


[Icon Request] tPay

Hi,
could you add tPay payment gateway.
The icons and graphics materials are here, in all sort of formats:
https://tpay.com/en/graphic-materials

Any news about this icon?


GitHub action optimizations

name: Pull request
about: CI/CD build updates
Motivate of the pull request

[x] To address an existing issue. If so, please provide a link to the issue.
[ ] Breaking new feature. If so, please decribe details in the description part.

Describe the technical details

What is currently done? (Provide issue link if applicable)
What does this pull request change?

Which part of the code base require a change
In general, modification on existing submodules are not acceptable. You should push changes to upstream.

[ ] VPR
[ ] OpenFPGA libraries
[ ] FPGA-Verilog
[ ] FPGA-Bitstream
[ ] FPGA-SDC
[ ] FPGA-SPICE
[ ] Flow scripts
[ ] Architecture library
[ ] Cell library

Checklist of the pull request

[ ] Require code changes.
[ ] Require new tests to be added
[ ] Require an update on documentation

Impact of the pull request

[ ] Require a change on Quality of Results (QoR)
[ ] Break back-compatibility. If so, please list who may be influenced.

@ganeshgore @srtemp @tangxifan I have no idea what happened, the pull request disappeared. Pushed up changes with merged in from latest master. I've added a new docker workflow that can be manually triggered from the master branch to build the docker images. I've also consolidated the dependency into scripts called from the docker images.

@yashton No worries. Shit happened. I hope that you did not lose any commits.

I see the regression tests are now skipped, which is a concern for me.

Looks good to me. @ganeshgore Can you review this quickly?

Looks good to me. @ganeshgore Can you review this quickly?

@yashton This is really a great implementation I was wondering if we can accommodate following things.

Skipping linux_build whenever the source is unchanged [Important because except Xifan non many developers change source]
Maintaining prebuilt docker image with OpenFPGA, consistent with current master on github packages or docker hub [Required to use OpenFPGA engine in other repositories as is without building again (example: lnis-uofu/SOFA) ]

I made a simple flowchart to detail how it can be implemented, we can discuss if there are any further questions.

@yashton This is really a great implementation I was wondering if we can accommodate following things.

Skipping linux_build whenever the source is unchanged [Important because except Xifan non many developers change source]
Maintaining prebuilt docker image with OpenFPGA, consistent with current master on github packages or docker hub [Required to use OpenFPGA engine in other repositories as is without building again (example: lnis-uofu/SOFA) ]

I made a simple flowchart to detail how it can be implemented, we can discuss if there are any further questions.

This looks awesome, I don't see any problem at this stage now.
Can you add the following lines in Dockerfile.master
ENV OPENFPGA_PATH="/opt/openfpga:${PATH}"
WORKDIR /opt/openfpga

Thanks Ashton.

This looks awesome, I don't see any problem at this stage now.
Can you add the following lines in Dockerfile.master
ENV OPENFPGA_PATH="/opt/openfpga:${PATH}"
WORKDIR /opt/openfpga

Thanks Ashton.

@tangxifan  I think it's good to merge now , you can review from your side and merge.
Note

The existing build.yml is renamed to build_old.yml. This will run new and old regression both in parallel for at-least a week to ensure consistency.
Docs has been added to understand CI/CD flow, it also contains debug information in case of regression failure.

@tangxifan  I think it's good to merge now , you can review from your side and merge.
Note

The existing build.yml is renamed to build_old.yml. This will run new and old regression both in parallel for at-least a week to ensure consistency.
Docs has been added to understand CI/CD flow, it also contains debug information in case of regression failure.

didn't get your first question about ${{matrix.config.name}} can you point to line or file you are referring to.
build_old.yml  is just a temporary file we will remove that  after a week ,
build.yml uses optimized approach and build_old.yml is the one we are using now  (full build for every commit).
It is maintained to ensure consistency.
I will create separate pull request for the doc, its on different branch

didn't get your first question about ${{matrix.config.name}} can you point to line or file you are referring to.
build_old.yml  is just a temporary file we will remove that  after a week ,
build.yml uses optimized approach and build_old.yml is the one we are using now  (full build for every commit).
It is maintained to ensure consistency.
I will create separate pull request for the doc, its on different branch

See the attached image

See the attached image

${{matrix.config.name}} job is a quirk of the matrix generation. If the job does not run because the if clause is false, there is no substitution from the matrix, so the name shows up as the placeholder.

${{matrix.config.name}} job is a quirk of the matrix generation. If the job does not run because the if clause is false, there is no substitution from the matrix, so the name shows up as the placeholder.

O.K. I will merge this PR.

O.K. I will merge this PR.


See if we can do blocked types using an error

See discussion in #394. We might be able to remove this code:
        if deps.iter().any(|tn| self.avoid_generating_type(tn)) {
            return Err(contextualize_error(ConvertError::UnacceptableParam(
                rust_name,
            )));
        }

because it might be that we would naturally generate an error anyway in this case. (Or at least in some way make this neater).

Oops filed in wrong repo! Sorry!


Bugfix: Wrong results for Value Iteration

The value iteration results are wrong as can be seen by the fact that they're supposed to be symmetric.
This change fixes the bug.
Explanation: The line should backpropagate the value at the new state i, i.e. v.v[i] to the old state j, i.e. to v_new[idx][u][z][j].

I guess that would fix #20 as well?

It's been a long time since I've looked at this 🤔
Shouldn't it be a +=, not an =? As mentioned in Issue #20 . Otherwise you're just overwriting v_new[idx][u][z][j] i times?

Oh, yes, sorry, you're of course right, I think it needs both changes?

As you already said, you want to calculate the value for the new state i. Which you do by multiplying the state's value with its observation and transition probabilities. What I believe what you call backpropagation is actaully done later in line 58. The equation (with the +=) is actually correct. Have you tried it out with the bugfix from #20 ?

According to your comment:
# v_i_k * p(z | x_i, u) * p(x_i | u, x_j)
you're computing at the transition from state j to state i. So the value at state i, i.e. v.v[i] should influence v_new[idx][u][z][j], not v_new[idx][u][z][i]. Or in other words: It would be weird if j would only appear once in the entire equation.
Another way to see that somewhere there must be a bug is that the two doors example that you are using is symmetric, but the results you get aren't, so something's clearly wrong. And after changing that line the results are symmetric and seem to make sense.

You're right, sorry it's been a while since I've looked at it. The algrithm is quite different in my fork.

I created a new PR with the extra change for the +=  at PR #24 - let me know if it works @maximilianigl @hlzl


fix(bpdm): add unsupported bpdm identifier #618

Description
This PR contains the changes from https://github.com/eclipse-tractusx/portal-backend/pull/619 plus the required migration
Why
see https://github.com/eclipse-tractusx/portal-backend/pull/619
Issue
#618
Checklist
Please delete options that are not relevant.

[X] I have followed the contributing guidelines
[X] I have performed a self-review of my own code
[X] I have successfully tested my changes locally
[X] I have added tests that prove my changes work
[X] I have checked that new and existing tests pass locally with my changes

Thanks Norbert!
I will quickly approach the FE change description and link it here as well

Thanks Norbert! I will quickly approach the FE change description and link it here as well
We should also enhance country_assigned_identifiers - to ensure that the e2e flow works we need a reference for the DUNS. Or did you already analyzed it?

@jjeroch country_assigned_identifiers must be adjusted, otherwise the new identifier is pretty useless as one cannot post it to registration-endpoint registration/application/{applicationId}/companyDetailsWithAddress
I need your input for that.

obsolete, PR #619 has maintainer-write-access enabled


iOS - Assessment - My Video - if the name of the video is long it merges with the circle

Fixed.

@aleffert pl review

:+1:


Mix3 (perseus) BSOD in newest UEFI release (2210.1)

Prerequisites

[x] Have you read the wiki?
[x] Is your device in the supported list?
[x] Does the device have a Snapdragon SOC which is supported in this repo?

Description
Device: Mix3 (perseus) 8GB/128GB
UEFI version: 2210.1
Windows version: 10 (build 21390)
WOA-Drivers version: 2210.1-fix
Expected behavior: Windows boots normally
Actual behavior: loading a while and then BSOD: INACCESSIBLE_BOOT_DEVICE (also in Windows 11)

Try redeploying drivers

How to redeploy drivers? Same problem after reinstalling Windows and drivers. Use older UEFI can boot but some drivers (such as GPU, Wi-Fi) can't load.

@cmy2008 in dism delete installed drivers and try again to install

@cmy2008 in dism delete installed drivers and try again to install

same problem, both old and new drivers

@

@cmy2008 in dism delete installed drivers and try again to install

same problem, both old and new drivers

do you manage to find fix for this issue?


Fix grammar.

Removed "the" from the sentence: "[...] which divides Events for the you."

Please don't post 1-word fixes. We see your issue and will incorporate that into a bigger PR

Also don't create PRs against master. This would issue a rebuild with our current CI setup and we def. don't need that for Docs grammar fixes


Send a notification email after user password gets changed

Codecov Report

Merging #41 (c434307) into main (a5d649f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main      #41   +/-   ##
=======================================
  Coverage   97.95%   97.96%
=======================================
  Files          80       80
  Lines        3478     3492   +14
=======================================
+ Hits         3407     3421   +14
  Misses         71       71

Files Changed
Coverage Δ

asu/auth/models/user.py
98.08% <100.00%> (+0.04%)
:arrow_up:

asu/auth/serializers/actions.py
100.00% <100.00%> (ø)

asu/utils/messages.py
100.00% <100.00%> (ø)

tests/test_password_reset.py
100.00% <100.00%> (ø)


GitHub pages: Internal server error

Describe the bug
Static github website shows
Internal server error

This error is embeded in index.html suggesting that the pipeline failed to build the website.
See: https://github.com/hspsh/upptime/blob/b4fc9342d593eb5627628bb57a4b95f32f2e142e/index.html
To Reproduce
Create fresh upptime repo from the template.
Expected behavior
Upptime website deployed to GitHub pages
Or in case of configuration failure, error message pointing to the problem.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Here is a log of a failed build job https://github.com/hspsh/upptime/actions/runs/9627948403/job/26555635370

Looking deeper, error at Update template step is non-failing.
To https://github.com/hspsh/upptime
 ! [remote rejected] master -> master (refusing to allow a GitHub App to create or update workflow `.github/workflows/graphs.yml` without `workflows` permission)
error: failed to push some refs to 'https://github.com/hspsh/upptime'
All done!

Yes, it looks like you need the right permissions: https://upptime.js.org/docs/get-started#add-repository-secrets

Yes, it looks like you need the right permissions: https://upptime.js.org/docs/get-started#add-repository-secrets

same issue here, all my permissions have been given but still fail to generate site @AnandChowdhary

https://github.com/PowerfulRichard/PT-Status/actions/runs/9826179829/job/27127323109
> Crawling http://localhost:3000/PT-Status/
TypeError: Cannot read properties of null (reading 'replace')
    at /home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:1058:110
    at Object.$$render (/home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:264:22)
    at /home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:1113:40
    at $$render (/home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:264:22)
    at Object.render (/home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:272:26)
    at /home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:19752:82
    at convertThrownError (/home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:19507:24)
    at detectClientOnlyReferences (/home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:19526:12)
    at /home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:19752:45
    at Generator.next (<anonymous>)
TypeError: Cannot read properties of null (reading 'replace')
    at /home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:1058:110
    at Object.$$render (/home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:264:22)
    at /home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:1113:40
    at $$render (/home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:264:22)
    at Object.render (/home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:272:26)
    at /home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:19752:82
    at convertThrownError (/home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:19507:24)
    at detectClientOnlyReferences (/home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:19526:12)
    at /home/runner/work/PT-Status/PT-Status/site/__sapper__/build/server/server.js:19752:45
    at Generator.next (<anonymous>)
      32 B   (500) index.html
      32 B   (500) service-worker-index.html


NSData rac_dataWithContentsOfURL and NSUserDefaults rac_signalForKey

I need some help here, rac_signalForKey send the value any time NSUserDefaults changed. So the filter should be here, but I don't know how to implement it properly.

Doesn't look like NSUserDefaultsDidChangeNotification includes what changed so that makes this trickier.
What I would do is switch the return value from AnyObject? to NSObject? which should be the case for anything coming out of NSUserDefaults. Then you can append a skipRepeats on the signal and use -[NSObject isEqual:].

Method skipRepeats is almost what needed, except that it doesn't work with Optional. For NSUserDefaults Optional values should be handled somehow.
Implement skipRepeats in other way?

The version of skipRepeats that takes a comparator should work

This implementation unwraps Optional, map { $0.1! }
It looks that there should be one more skipRepeats implementation, that accepts Optionals and returns them.

That's unwrapping the Optional that it already packaged here. In our case T is NSObject? so that line is creating an NSObject?? which is unwrapped back to NSObject? later.

Let me see. It looks that I don't understand something here.

Done. Please check.

This is good for now but will probably need to be updated depending on the outcome of ReactiveCocoa/ReactiveCocoa#1980

It looks like KVO can also be used for specific NSUserDefaults keys which may be a better alternative than global notifications.

I don't think that KVO observing for NSUserDefaults is a good approach if it isn't described in Apple documentation.

By the way I haven't casted NSUserDefaults to NSObject as a return value since objectForKey: returns value of AnyObject? type.

@larryonoff I tagged a new release with these


build constraints exclude all Go files

I get the following while trying to import go-webui/v2 after installing it every way that the readme describes:
package command-line-arguments
        imports github.com/webui-dev/go-webui/v2: build constraints exclude all Go files in C:\Users\trisi\hack\goglitch\go-webui\v2

Trying to give some first-aid I would forward a stack overflow answer.
Can you try again after running:
go clean -modcache

Ref.: https://stackoverflow.com/a/73531307/18392410
To further analyze the problem if it doesn't work, can you please try to run the example relative to the package, and report the result? For this you clone the repository and copy or move the examples directory into the v2/ directory. If you run them here, they will use the relative package.
Like the readme instructcs: https://github.com/webui-dev/go-webui#as-git-clone-in-a-local-directory

I tried the modcache thing before this post
hmm, I think I got something working by enabling cgo, is that a requirement of this project?:
install https://jmeubank.github.io/tdm-gcc/download/
go env -w CGO_ENABLED=1 go build
and then changing the code to:
	w := webui.NewWindow() 	webui.Show(w, "Hello World") 	webui.Wait()
produces no errors but nothing seems to happen, I'm probably doing something wrong

Seems like NewWindow doesnt really do anything, is this because I have webui setup wrong?

ah, I got it working haha

The things I did above + go run .

Glad to hear it works. Yep, you need to run the program :D!
Explicitly enabling cgo is usually not required. But I guess it can be required depending on the setup. If you run into more issues, don't hesitate to reach out! Best with such step by step reproductions like you did then this the issues more clear and easier to help!

Is there a version of examples that uses the new api?

Yes, the current examples in the repository use it. They are made to work with the latest webui version and are not compatible with versions earlier than 2.4.0.
E.g.: https://github.com/webui-dev/go-webui/blob/main/examples/call_go_from_js.go

Oh, for some reason my api looks more like this<EMAIL_ADDRESS>which doesnt work with the example you listed

Oh yeah, the wrapper got a re-write that fixed all known bugs and improved the setup process and the API. The version that you are linking might have a lot of things that do not work right. The readme currently lists to install it via:
go get<EMAIL_ADDRESS>
This will give you the version you see in the example.
Another tag with this new version will be released soon.

First I did this:
go get<EMAIL_ADDRESS>irm https://raw.githubusercontent.com/webui-dev/go-webui/main/setup.ps1 | iex

then I still had to do this

I tried the modcache thing before this post hmm, I think I got something working by enabling cgo, is that a requirement of this project?: install https://jmeubank.github.io/tdm-gcc/download/
go env -w CGO_ENABLED=1
go build

and then changing the code to:
w := webui.NewWindow()
webui.Show(w, "Hello World")
webui.Wait()

produces no errors but nothing seems to happen, I'm probably doing something wrong

but now the example code works as expected :)

Another thing that comes to my mind now that I know about the version. It might have been necessary to explicitly enable CGO with that version. You can check if it's still required after you have made the update.


Update templates

[x] src to code
[x] add parameters to parameter files to simplify the getting started experience

Branch simple_template is ready to go. Please look at the changes in that branch.

done


Results  from attempted run on Apple M1

Vanilla:
lawik@m1mini example % arch                                      arm64
lawik@m1mini example % mix release                               Release example_cli_app-0.1.0 already exists. Overwrite? [Yn] y
* assembling example_cli_app-0.1.0 on MIX_ENV=dev
* skipping runtime configuration (config/runtime.exs not found)

17:19:16.758 [info]  Burrito build target is: :native

17:19:17.255 [info]  Build working dir: /var/folders/fj/m1gn99tx05v1h8tdwtkkhgvr0000gn/T/burrito_build_36CCA0C1B4461889

17:19:17.256 [info]  Patching shell script for release...

17:19:17.259 [info]  Generating wrapper metadata file...

17:19:17.481 [error] Burrito failed to wrap up your app! Check the logs for more information.
** (exit) 1
    (burrito 0.4.0) lib/burrito.ex:219: Burrito.do_wrap/5
    (elixir 1.12.3) lib/enum.ex:930: Enum."-each/2-lists^foreach/1-0-"/2
    (burrito 0.4.0) lib/burrito.ex:34: Burrito.wrap/1
    (mix 1.12.3) lib/mix/tasks/release.ex:1047: Mix.Tasks.Release.run_steps/1
    (mix 1.12.3) lib/mix/task.ex:394: anonymous fn/3 in Mix.Task.run_task/3

With rosetta?
lawik@m1mini example % arch -x86_64 mix release                  Release example_cli_app-0.1.0 already exists. Overwrite? [Yn] y
* assembling example_cli_app-0.1.0 on MIX_ENV=dev
* skipping runtime configuration (config/runtime.exs not found)

17:22:30.892 [info]  Burrito build target is: :native

17:22:31.382 [info]  Build working dir: /var/folders/fj/m1gn99tx05v1h8tdwtkkhgvr0000gn/T/burrito_build_9566FAB46C2CEF47

17:22:31.383 [info]  Patching shell script for release...

17:22:31.393 [info]  Generating wrapper metadata file...

17:22:32.119 [error] Burrito failed to wrap up your app! Check the logs for more information.
** (exit) 1
    (burrito 0.4.0) lib/burrito.ex:219: Burrito.do_wrap/5
    (elixir 1.12.3) lib/enum.ex:930: Enum."-each/2-lists^foreach/1-0-"/2
    (burrito 0.4.0) lib/burrito.ex:34: Burrito.wrap/1
    (mix 1.12.3) lib/mix/tasks/release.ex:1047: Mix.Tasks.Release.run_steps/1
    (mix 1.12.3) lib/mix/task.ex:394: anonymous fn/3 in Mix.Task.run_task/3
    (mix 1.12.3) lib/mix/cli.ex:84: Mix.CLI.run_task/2

Hope that is useful, let me know if I should try anything else :)

Quick bump on this I have acquired a new Macbook with an M1, so I'll have a better time testing this stuff!
Additionally I believe I have the flags to compile apple silicon erlang in our erlang-builder repo -- hopefully I can get that up soon!

I was just curious about the state of this. I'd like to try out burrito to wrap up a CLI tool I wrote using Ratatouille, but it would need to run on (at least) M1 and Intel Macs.
I'm happy to contribute to fixes (within my abilities), but thought I'd ask if there was any known issue or workaround first.

@sax hello!
Since we just refactored the build pipeline that should be fairly doable now. I don't think we offer a ton of pre-built ERTS release for aarch64 Darwin but that's something I want to resolve sooner rather than later...
But I think the following process could get you what you want:

Ensure you're using the latest release of Burrito from the main branch
Unlock our dependency on Zig 0.9.0 (which is there for stability, but their nightly builds have fixed a Mach-O linker issue for cross-compiling NIFs)
Build a custom ERTS for Intel or aarch64 (whichever your build machine isn't) the README has a section on this!
Configure your Burrito targets so they look something like this:

...
  # If you are building on an Intel machine, provide an ERTS compiled for aarch64 like below
  # flop this around if you are building on aarch64 targeting Intel
  targets: [
    macos: [os: :darwin, cpu: :x86_64],
    macos_m1: [os: :darwin, cpu: :aarch64, local_erts: "/path/to/darwin_aarch64_erts.tar.gz"],
  ]
...

I haven't tested this, but in theory it should get you the ability to produce M1 and Intel compatible binaries

Awesome, thank you! I haven't yet tried the cross-compile, but after a few tries I got an M1 build working.
One suggestion: I had to do a few iterations before I found all the things I needed to do. Maybe it would be helpful to add the following to the README?

Brew packages
asdf zig plugin
Overriding zig version in deps/burrito

I could do a pull request for those. I'm going to try cross-compiling to x86_64 first, though.

Oh... another thought is that if zig is already in the path (ie, from an application's .tool-versions), to use that instead of the version specified in deps. Maybe print out a warning when this happens, so it's less surprising if tragedy ensues?

Just tested to build for aarch64 on my M1 Pro and had no problems :-)


The bootstrap addon example is not showing fontawesome icons

The bootstrap addon example is not showing fontawesome icons:
https://stackblitz.com/angular/rydpaqlrgav?file=src%2Fapp%2Fapp.component.html
My Environment:

Angular version:9.1.0
Formly version: 5.5.15

Stackblitz example environment

Angular version:7.2.16
Formly version: 5.5.15

Thank you

The fontawesome style is not included, we'll fix that!
Thanks!


Updated installation instruction about Sass/SCSS support

The goal of this PR is to update installation instruction about --tags extended when CGO_ENABLED=0 as it shows such no straightforward error:
$ go install --tags extended
../../go/gopath/pkg/mod/github.com/bep/golibsass@v0.6.0/libsass/transpiler.go:15:2: build constraints exclude all Go files in<EMAIL_ADDRESS>
This is because CGO_ENABLED=0 on my linux machine (go1.14.2 linux/adm64 - but previous versions of go and hugo had this problem as well)
I don't know if this is related to this issue as I haven't tested on ARM: https://github.com/gohugoio/hugo/issues/6991

 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.


Add documentation to new API

Adding documentation to the new API.  Fixing warnings from Doxygen and Sphinx.
Builds on work from #7.

Too many things have changed, so this PR is not relevant anymore.


docs: fix src link in the readme

:tada: This PR is included in version 1.1.0-test.1 :tada:
The release is available on:

npm package (@test dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error when play local path isLocal

Hello ,
i have a problem with isLocal ,,
this is a message give it to me in console
flutter: audioPlayer error : AVPlayerItemStatus.failed
and this is a path of my audio
/Users/mypc/Library/Developer/CoreSimulator/Devices/10098BA7-9F7B-4FE5-85B9-B7F1C43FD5CF/data/Containers/Data/Application/A5505A39-7200-4A3E-9ABD-7300938CC354/Documents/650.mp3
and i used function in an example of this plugin
  Future _loadFile() async {     final dir = await getApplicationDocumentsDirectory();     final full_file=('${dir.path}/$songid.mp3');     final file = File(full_file);     if (await file.exists()) {       setState(() {         localFilePath = file.path;       });     }   }
localFilePath =  /Users/mypc/Library/Developer/CoreSimulator/Devices/10098BA7-9F7B-4FE5-85B9-B7F1C43FD5CF/data/Containers/Data/Application/A5505A39-7200-4A3E-9ABD-7300938CC354/Documents/650.mp3
the local of file is true and i founded in the folder

it's work with me .


Typo in SQL keyword: ESCEPTION

(Original issue 853 created by roskakori on 2013-02-21T14:56:06.023897+00:00)
There seems to be a typo in pygments.lexers.sql.SqlLexer:
#!python
r'EXCEPT|ESCEPTION|EXCLUDING|EXCLUSIVE|EXEC|EXECUTE|EXISTING|'

Presumably ESCEPTION should read EXCEPTION.

(Original issue 853 last updated on 2013-05-19T09:48:47.197346+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by birkenfeld on 2013-05-19T09:48:47.190948+00:00)
Closes #853: fix typo in SQL lexer.
→ <<cset 1087ccc1af73>>


Imports/types not resolved in if TYPE_CHECKING blocks

Given the following code:
from typing import TYPE_CHECKING

if TYPE_CHECKING:
    import some_module

def some_fn(arg: "some_module.SomeType") -> None:
    var = arg.some_instancevar

There is no type shown on hover over var and Ctrl+click (Go to Definition) does not work for arg.some_instancevar.
I guess this is because the imports inside TYPE_CHECKING are not actually loaded/resolved?
Extension information:
Microsoft Python Language Server version <IP_ADDRESS>
Name: Python
Id: ms-python.python
Description: Linting, Debugging (multi-threaded, remote), Intellisense, code formatting, refactoring, unit tests, snippets, and more.
Version: 2019.5.18875
Publisher: Microsoft
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=ms-python.python

The typing module is implemented within the language server, and I don't think we do anything with TYPE_CHECKING (and likely never will), but regardless goto def should work there. I think goto def working is probably unrelated to TYPE_CHECKING itself. Do you have a test project that reproduces this (past this short code example)?

Can't really reproduce it in a simple project... but it looks like this has more to do with the fact that those type definitions are from .pyi files rather than the conditional import. Let me try to boil it down.

Okay, I think I got it. It's related to the types being declared as strings, which is one requirement for using the if TYPE_CHECKING import style. Let me file another bug.


GUVNOR-2338: Copy/Rename modal doesn't disappear when used from Project Explorer links

Depends on: https://github.com/uberfire/uberfire-extensions/pull/174

Other than the change to use the Presenter's hide() method I think this is much clearer to follow than the flag you had before. Thanks for making the changes... once you switch to using the Presenter I'll OK the merge.

+1 to merge.

master: c60f46b576bde63f0d1afbcdb7fb30f373189d48


[WIP] initial edits to clarify on-prem install/run

Signed-off-by: JENNIFER RONDEAU<EMAIL_ADDRESS>Brief start. TODOs/Qs:

Would it help to have a single topic (maybe an addition to Nolan's PR in flight, #723?) that lists all storage types, including local types? (Pull out list from intro para to restic topic maybe?) We emphasize cloud storage types so much (I understand why) that it skews the focus.
Create a new topic that addresses on-prem specifically? I'm not sure this is really needed, but I doubt that the current little bits that I've added are sufficient either.
Address issue #558 to help with this one?

Addresses issue #332

@Bradamant3 @nrb @skriss @carlisia @wwitzel3 would it be possible to get this updated & merged today?

if @Bradamant3 isn't available to push this thru, I could take it over, but unlikely to be able to do it today

Sorry, @nrb just called my attention to these comments. I can't get to it today (need to end early), but should be able to do it first thing in the morning.

@Bradamant3 and @skriss Any updates for this?

Working on it now, should have something up in a couple hours

Edits are continued in PR #880, closing this PR.


ip-range should accept comma separated list of IPs

Description
This is more or less a question. I haven't seen any docs that describe the accepted format of the ip-range. Can this accept a comma seperated list? I have a block of IPs which have some being used in the middle. I would like to use an ip range like so:
{"dev":"<IP_ADDRESS>-<IP_ADDRESS>,<IP_ADDRESS>-<IP_ADDRESS>"}

If this is already possible that would be great to know!

Created [CONTCNTR-3279] for internal tracking.


Update Dockerfile to include file utility

Apparently paperclip needs it to check the filetype.
Footprint is 2 extra packages: file and libmagic1 for some reason so should be reasonable.
I've built the image ok, but please, test :)

I'm a member of GoogleCloudPlatform and Google orgs.

@Temikus Check that you used your @google.com email address as your commit address, and that your @google.com email address is associated with your github account. (Alternately, if you didn't use your @google.com email address as the committer, then you do need to sign the SLA with the email address that you used.)

@dazuma Sorry, my bad. Rebased.


fix: random_drop dropping on top of the ant

This should fix the case when ant drops a block at its position, then spins around stuck inside.

Thank you! The intent behind this looks correct, but we should update the other use of *position to *ahead_position:
let target_element_entity = grid.elements().element_entity(*position);
The intent with this check is a poor person's safeguard. It's trying to make sure that the Air at the location we're going to Drop is still there when the Drop Command gets applied. If another ant were to race against this code and drop their Element first then the target_element_entity check would fail.

🚢


[noetic][Windows] Accommodate different drives for --root

This is a follow-up fix for #1070
In cases of that CMAKE_INSTALL_PREFIX resides in the different drive from the source code, the previous fix will result in the Python module to be installed at the same drive where the source code sits. For example, if ROS source code is checked out on C: drive and cmake install into D:\opt\instsall, the Python module will be mistakenly installed to c:\opt\install\.
This pull request is to take the target drive into consideration. And hope this can be cherry-picked into kinectic-devel.

@dirk-thomas This is ready for review and merge. Thanks for your feedback on the drive name thing. Problem do manifest when the source workspace is on different drives from the install space.

Thanks for the patch.
That was exactly what I was concerned about in the previous patch: https://github.com/ros/catkin/pull/1070#discussion_r393912628

Cherry-picked to kinetic-devel: 39bbd52cb69c567f915a372906216e6c9ac8b226.


Add operator section to user guide

Which issue does this PR close?

Closes #7681
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Several functions added in this PR:

Functions already listed in documentation but not implemented:

not
eq
not_eq
gt
gt_eq
lt
lt_eq

Additional arithmetic functions introduced in this PR:

add
sub
mul
div
r#mod
neg

Few more questions:

These implementations and tests already exist somewhere. Do these still have to add test code here?
mod is a reserved keyword in Rust. The reminder function named here to be able consistant with SQL function naming, but does it ok to datafusion?

Methods added to Expr in this PR:

Methods already listed in documentation but not implemented:

bitwise_and
bitwise_or
bitwise_xor
bitwise_shift_left
bitwise_shift_right
not

Additional arithmetic methods introduced in this PR:

As same as arithmetic functions listed above

I found that there are multiple implementations to do the same thing, e.g., bitwise operations on https://github.com/apache/arrow-datafusion/blob/a64f36d8fbce35cb7146271fb5d9ab98a9f094e6/datafusion/expr/src/expr_fn.rs#L172-L215 and https://github.com/apache/arrow-datafusion/blob/a64f36d8fbce35cb7146271fb5d9ab98a9f094e6/datafusion/expr/src/operator.rs#L313-L355, that should/is identical to each other. Do we have to merge these implementations and place tests in the same place?

Folder links does not work in the document. Change all file links to GitHub, which would be easier to browse in this commit 046e020a1b7eab4c84aee6da0716efc92694e16d

Thanks @ongchi for the PR, I left some minor comments

Doc is definitely looks more solid and consistent now, thanks @ongchi
For the new methods introduced for Expr trait we may need to cover them by unit tests.
Another thing comes to my mind if we support both syntaxes, we should prob have some notification when new operator is introduced.
It might be a common trait that will require during compilation for another syntax to be implemented

Another thing comes to my mind if we support both syntaxes, we should prob have some notification when new operator is introduced. It might be a common trait that will require during compilation for another syntax to be implemented

I think we should reorganize test cases in datafusion-expr and make some improvements by creating a test module. Currently, these tests do not cover all use cases. The tests may follow the order/structure from expression API documentation, which would make it easier to identify which parts needs to be improve.

Converting to draft as this PR is not waiting on feedback anymore. Please mark it ready to review when ready.
I do realize when writing documentation, often deficiencies in the existing APIs become clearer
One suggestion I had was to split out the changes to the documentation from the changes to how the traits work.

I think we should reorganize test cases in datafusion-expr and make some improvements by creating a test module. Currently, these tests do not cover all use cases. The tests may follow the order/structure from expression API documentation, which would make it easier to identify which parts needs to be improve.

I agree -- @ongchi could you please file a follow on ticket describing this work? Perhaps it would be a good first step in updating how the traits worked: consolidate all the existing tests we do have, so the scope of the "to do" work becomes clear.

Converting to draft as this PR is not waiting on feedback anymore. Please mark it ready to review when ready.
I do realize when writing documentation, often deficiencies in the existing APIs become clearer
One suggestion I had was to split out the changes to the documentation from the changes to how the traits work.

I think we should reorganize test cases in datafusion-expr and make some improvements by creating a test module. Currently, these tests do not cover all use cases. The tests may follow the order/structure from expression API documentation, which would make it easier to identify which parts needs to be improve.

I agree -- @ongchi could you please file a follow on ticket describing this work? Perhaps it would be a good first step in updating how the traits worked: consolidate all the existing tests we do have, so the scope of the "to do" work becomes clear.

Sure, I'm happy to do that.
I'll focus on correctness of the documentation in this PR with minimal code changes. Any further changes can depend on other users' input.


Expanded yes command and fixed case-sensitive issues

Can now type in "y"/"ye"/"yes" to confirm delete/clear
Commands are now case-insensitive

Closes #153 and #160

Codecov Report

Merging #169 into master will decrease coverage by 0.16%.
The diff coverage is 25.00%.

@@             Coverage Diff              @@
##             master     #169      +/-   ##
============================================
- Coverage     75.53%   75.37%   -0.17%
- Complexity      565      566       +1
============================================
  Files            93       93
  Lines          1713     1718       +5
  Branches        174      173       -1
============================================
+ Hits           1294     1295       +1
- Misses          338      343       +5
+ Partials         81       80       -1

Impacted Files
Coverage Δ
Complexity Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...rc/main/java/seedu/address/logic/LogicManager.java
41.86% <25.00%> (+3.22%)
6.00 <1.00> (+1.00)

...java/seedu/address/logic/parser/GradPadParser.java
60.00% <25.00%> (-15.00%)
14.00 <0.00> (ø)

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 c4fbcba...55067ab. Read the comment docs.


Replace Era with AnyCardanoEra in cardano-testnet

This will remove the maintenance burden of updating Era to match AnyCardanoEra

This is more nuanced than simply replacing Era with AnyCardanoEra as such I am closing this PR whilst we work on structurally changing cardano-testnet to facilitate the use of AnyCardanoEra.


Bump llvm-sys to version 150

Signed-off-by: Michal Rostecki<EMAIL_ADDRESS>

Adding this for the record https://discord.com/channels/855676609003651072/855676609003651075/1017130100099121182


Created githash saving and display functionality

This PR completes GH issue https://github.com/ManageIQ/manageiq-ui-service/issues/999

Screenshot of hash in action

@miq-bot add_label enhancement
@miq-bot add_label fine/no


Appreciation + potentially unrelated bug

I am on macOS10.12 and build it with --release and running it from iTerm2 with ./target/release/alacritty .
The problem that I have is that the typed command shadows over to its output.

The same issue appears with Terminal.app, however it doesn't appear with iTerm2.app
Any help would be appreciated.
Also, damn this is awesome. :-) I'm going to start using alacritty from now on (even though it would be nice to get rid of the shadowing).
Furthermore, how can I bind alacritty to a desktop shortcut that I can open from spotlight?

I've had the same problem of the command shadowing the output before, when I was using zsh and had set $TERM to screen-256color.  Either setting it to xterm-256color or running inside of tmux solved the problem for me.

Also, damn this is awesome

Thanks!

Furthermore, how can I bind alacritty to a desktop shortcut that I can open from spotlight?

Need to build an Alacritty.app. We'll be publishing these at some point once the project matures.

Any help would be appreciated.

This does sound like a potential TERM issue. Going to close this as such, but feel free to correct me if this turns out to be an Alacritty issue.


Why the audio_data_rate and video_data_rate is always not set in SrsAvcAacCodec class

Hi @winlinvip , I found a bug. The audio_data_rate and video_data_rate is always 0 in srs_app_source.cpp and srs_app_source.cpp
The result log:
44B video sh,  codec(7, profile=Baseline, level=3, 640x768, 0kbps, 0fps, 0s)
7B audio sh, codec(10, profile=LC, 1channels, 0kbps, 22050HZ), flv(16bits, 2channels, 22050HZ)

It is always  0kbps. And after I search all source code with video_data_rate or audio_data_rate, I am not found where to set it.
Why not set it?

It should be parsed from sps/pps. But the actual kbps can be calc by server. That is, the kbps in sps/pps is encoding settings, not the real data.

So you use class Kbps to calc it? The http api is SrsKbps::get_send_kbps_30s().
But it will delay at least 30s.

Yep, it's ok.

How about use the stream info of SrsSource::on_meta_data(SrsCommonMessage* msg, SrsOnMetaDataPacket* metadata)

数据准确度优先级是：

metadata：这个是最不准确的。
sps和pps：编码器设置的，比metadata要好。
cacluate：服务器自己算出来的最准确。


Updating to the new versions OCPP1.6 schemas

OCA has released new versions of the OCPP1.6 schemas together
with erratas.
This is the latest version.
Fixes #57 (adding Celsius with correct spelling) and more.
Signed-off-by: Anders Darander<EMAIL_ADDRESS>

Thanks for the comment.
I've updated the commit to omit the id lines.

For some reason the builds on CirceCi haven't been run. I will try to fix that. If tests pass I'm happy to merge!

I've added Celsius to StopTransaction.json, OCA is notified, but I rather not wait for them anyway.
I've also updated the commit message with info on when the new schema files were released and which modifications we've done to them.

@darander I fixed CircleCI's settings and now the test suite has ran. Unfortunately the build fails, can you fix it?

Fixed the build.
Thanks for getting the CircleCI working again!

Many thanks for your help!

Thanks for your help with my contribution!


Resurrected backup function

Coverage decreased (-0.3%) to 85.094% when pulling 22636bef7b570eb2185aed2ed6cbb91b628ecb32 on hengyu95:master into e554219cc4baaabef8d44f0a9c39466a77bd80e2 on CS2103AUG2017-W14-B2:master.

Coverage decreased (-0.3%) to 85.094% when pulling fdde23bd382e02d11cc9b2a00b6202ac89f4eafc on hengyu95:master into 274ea083b2fc336a2e11c1c38cb97689c259029c on CS2103AUG2017-W14-B2:master.


Android: star rating widget selected state color not appearing

We have implemented a star rating widget with the following configuration:

This works as expected on iOS however on Anrdoid (simulator and physical device) when the user taps the star the selected state color does not appear. The rating is recorded however the user does no see this and it appears if nothing has happened.
I've reproduced this on version 4.0.0 and also on 3.2.0. Please let me know if I can provide any additional info

Hi @mikrich , thanks for reaching out. I've taken a look at this issue here and attempted to reproduce in both a native Android test app, and a React Native Android test app. I am using the 4.0.0 version of the SDK. So far I have not been able to reproduce, as the tapped stars are showing the selected rating.

There may be something different about how I am testing from the configuration you are using, though. At this point, I'd recommend emailing in to<EMAIL_ADDRESS>or filling out a contact form at https://docs.appcues.com/contact-us. This way we can gather more details about your account and the flow you are using, and do a deeper inspection to see if there are any issues in how this flow is set up or the SDK is interpreting it.
Thanks for using Appcues, and if you have any more details you'd like to share right here in the meantime, please feel free. I'll keep this open and monitor it.

Thanks for getting back to me. It seems like re-creating the flow has fixed the issue for now, will keep an eye our for it and let you know if it happens again.


[CWS] refactor a bit the span ptracer code

What does this PR do?

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Fixes #82359 for recovery build

This PR fixes #82359
@aeschli https://github.com/microsoft/vscode/commit/479563e361f3face96b36220e7fe75384377e17f was a dangerous change because while JSON.parse would throw on parse errors, our custom json.parse doesn't and while its documentation reads On invalid input, the parser tries to be as fault tolerant as possible, but still return a result. it seems that there are cases in which the result it returns is undefined, which is what is happening here. The code happily continues and assumes manifest is an object.

Looks good, I have also tested and verified this indeed fixes the issue -> approved


Does not launch on ubuntu based distributions

Describe the bug
The .deb and .appimage don't open
Platform (please complete the following information):

OS: ubuntu based distributions
App version  v3.1.0

Additional context
Terminal Output:
gb-studio
No language pack for user setting, falling back to en
Add a language pack by making the file src/lang/en-US.json
[151221:0401/183732.976965:FATAL:gpu_data_manager_impl_private.cc(1034)] The display compositor is frequently crashing. Goodbye.
Trace/breakpoint trap (core dumped)
"gb-studio --disable-gpu" does not work either

Try to add --no-sandbox as argument.

@gonzoMD Thanks for the hint! That worked for me.

same problem and same solution "gb-studo --no-sandbox" ubutnu studio on ubuntu 23.04


rbmviz charts are not updating side panel

When a site on chart is selected, it is no longer updating the side-panel.
Need to trace previous work to see what's causing the error.

Hey @mattroumaya, I wanted to try to do more to isolate the issue. Basically, the following is not always triggering properly:
snapshot$lCharts[["kri0001"]]$scatterJS
snapshot$lCharts[["kri0001"]]$scatterJS$x$bHideDropdown = TRUE
snapshot$lCharts[["kri0001"]]$scatterJS

This is true in the Viewer as it is the Shiny session.
As far as I can tell, this is not an R version issue nor a Shiny version issue, but it does appear to be related to a package. I haven't isolated the package yet, as the usual offenders (shiny, shinyjs, htmltools, htmlwidgets) doesn't trigger any change.
However, if you install the gsmApp package and agree to install the updates for every package, this issue does goes away.

Closed via #107


XMLHttpRequest support

Is there any interest in adding support to connect via XHR? The main advantage is that it automatically uses system settings for proxies, Instead of having to set everything manually.
Back when this repo wasn't being maintained (a year ago), My team branched off to merge pending PRs and to add XHR support. Now that this repo is alive again we would be eager to abandon our fork if XHR support made it into this project.

Sure, a pull request is welcome.

@Mr-Wallet has this been done yet?

This problem isn't isolated to node-githib - API packages in general don't have XHR support. To minimize how many independent forks we maintain we've been looking at the possibility of dropping our XHR dependency instead of making a PR for this issue, but exploring that option has been on hold for now in favor of other priorities for our project.
We didn't think anyone wanted XHR support except for us, but If you also have a need for it then I could try to find time to investigate exactly what a PR would entail, since we're about a year behind on our fork. Let me know.

Given the lack of +1's on this I think we'll just leave it out for now.

We're really interested in doing this now because we need to modernize our API client - are you still open to one, despite the lack of enthusiasm? We'd rather not have to maintain our own fork if we can help it.

We are using fetch now internally. But we plan to provide a hook API with the next release in which you can replace the request logic with your own, stay tuned :)


webapp ssh handles ASEs

https://github.com/Azure/azure-cli/pull/9098 without the ASE test

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

 [cli] webapp ssh ASE


Unable to delete btrfs container with deleted storage

(Moved from https://github.com/lxc/lxc/issues/2654)

Required information

Distribution: Ubuntu
Distribution version: 18.04.1
The output of "lxc info":

config: {}
api_extensions:
- storage_zfs_remove_snapshots
- container_host_shutdown_timeout
- container_stop_priority
- container_syscall_filtering
- auth_pki
- container_last_used_at
- etag
- patch
- usb_devices
- https_allowed_credentials
- image_compression_algorithm
- directory_manipulation
- container_cpu_time
- storage_zfs_use_refquota
- storage_lvm_mount_options
- network
- profile_usedby
- container_push
- container_exec_recording
- certificate_update
- container_exec_signal_handling
- gpu_devices
- container_image_properties
- migration_progress
- id_map
- network_firewall_filtering
- network_routes
- storage
- file_delete
- file_append
- network_dhcp_expiry
- storage_lvm_vg_rename
- storage_lvm_thinpool_rename
- network_vlan
- image_create_aliases
- container_stateless_copy
- container_only_migration
- storage_zfs_clone_copy
- unix_device_rename
- storage_lvm_use_thinpool
- storage_rsync_bwlimit
- network_vxlan_interface
- storage_btrfs_mount_options
- entity_description
- image_force_refresh
- storage_lvm_lv_resizing
- id_map_base
- file_symlinks
- container_push_target
- network_vlan_physical
- storage_images_delete
- container_edit_metadata
- container_snapshot_stateful_migration
- storage_driver_ceph
- storage_ceph_user_name
- resource_limits
- storage_volatile_initial_source
- storage_ceph_force_osd_reuse
- storage_block_filesystem_btrfs
- resources
- kernel_limits
- storage_api_volume_rename
- macaroon_authentication
- network_sriov
- console
- restrict_devlxd
- migration_pre_copy
- infiniband
- maas_network
- devlxd_events
- proxy
- network_dhcp_gateway
- file_get_symlink
- network_leases
- unix_device_hotplug
- storage_api_local_volume_handling
- operation_description
- clustering
- event_lifecycle
- storage_api_remote_volume_handling
- nvidia_runtime
- container_mount_propagation
- container_backup
- devlxd_images
- container_local_cross_pool_handling
- proxy_unix
- proxy_udp
- clustering_join
- proxy_tcp_udp_multi_port_handling
- network_state
- proxy_unix_dac_properties
- container_protection_delete
- unix_priv_drop
- pprof_http
- proxy_haproxy_protocol
- network_hwaddr
- proxy_nat
- network_nat_order
- container_full
- candid_authentication
- backup_compression
- candid_config
api_status: stable
api_version: "1.0"
auth: trusted
public: false
auth_methods:
- tls
environment:
  addresses: []
  architectures:
  - x86_64
  - i686
  certificate: |
    -----BEGIN CERTIFICATE-----
    [OUTPUT TRUNCATED]
    -----END CERTIFICATE-----
  certificate_fingerprint: [OUTPUT TRUNCATED]
  driver: lxc
  driver_version: 3.0.2
  kernel: Linux
  kernel_architecture: x86_64
  kernel_version: 4.15.0-34-generic
  server: lxd
  server_pid: 3818
  server_version: "3.5"
  storage: ""
  storage_version: ""
  server_clustered: false
  server_name: Notebook

Issue description
There was a btrfs storage pool named local and with source /var/lib/lxd/disks/local.img. It used to work perfectly fine, until the file /var/lib/lxd/disks/local.img was deleted outside of LXC (cannot be recovered). It is used by 5 containers, and the storage pool and the containers appear in the output of lxc storage list and lxc list, respectively. Now, none of the storage pool local or its associated containers can be deleted in LXC.
Executing lxc delete container1 (container1 is a container in the missing local storage pool) outputs Error: no such file or directory. Using the --force flag does not change the output. There doesn't seem to be any way to remove container1 from LXC. This, I believe, is a bug or is at least undesirable behavior. Perhaps the only real negative effect is that it limits the namespace for new containers and storage pools.
Desired behaviour
I think that, perhaps through a --delete_from_db flag, LXC could offer an option to remove any references to a container/storage pool from the LXC database.
Steps to reproduce

Create a btrfs storage pool with an image file as a source.
Create a container in that storage pool.
Delete the source image file outside of LXC, for example using bash.
Attempt to delete the container or storage pool in LXC.

Information to attach
Executing  lxc storage delete local outputs: Error: storage pool "local" has volumes attached to it.
Executing lxc storage volume list local outputs:
+-----------+-----------+-------------+---------+
|   TYPE    |   NAME    | DESCRIPTION | USED BY |
+-----------+-----------+-------------+---------+
| container | container1   |             | 1       |
+-----------+-----------+-------------+---------+
| container | container2    |             | 1       |
+-----------+-----------+-------------+---------+
| container | container3 |             | 1       |
+-----------+-----------+-------------+---------+
| container | container4    |             | 1       |
+-----------+-----------+-------------+---------+
| container | container5  |             | 1       |
+-----------+-----------+-------------+---------+

The container storage volumes cannot be deleted outside of lxc delete: executing lxc storage volume delete local container/container1 outputs Error: storage volumes of type "container" cannot be deleted with the storage api.

Hmm, so normally I'd expect lxc delete container1 to succeed, we have code that should have LXD move on if the storage volume is gone. I guess there's a bug in the btrfs driver somehow, will have to reproduce the issue and fix it.

I couldn't reproduce the exact behavior above but I could reproduce some issues which I've now fixed in my branch.
Note that this will still not allow you to just do lxc delete container1 when you're missing the btrfs storage pool, you will need at a minimum to slot in an empty pool so that LXD moves on:

truncate -s 1G /var/lib/lxd/disks/local.img
mkfs.btrfs /var/lib/lxd/disks/local.img

I considered changing our checks to not require this part, but the problem with doing so is that this exact same behavior may happen during boot on systems that use encrypted or external storage and we wouldn't want a user to think they've deleted something to then have it show up on disk a minute later causing all kind of annoying conflicts.
I believe our current behavior of allowing an empty storage pool to be put in place of the existing one is sufficient and will generally match with what would happen should a drive go bad and be replaced or a loop file be corrupted. It lets you repair (potentially loosing some data) or just outright replace the partition/file with another one, then try to use whatever was on there and anything which doesn't behave can be deleted from LXD just fine.

The code from @stgraber's last post, with some minor changes, solved my problem. My full code for deleting container1 from local was:
lxd_dir=/var/lib/lxd
pool=local
container=container1

truncate -s 1G $lxd_dir/disks/$pool.img
mkfs.btrfs $lxd_dir/disks/$pool.img

mkdir $lxd_dir/storage-pools/$pool
mount $lxd_dir/disks/$pool.img $lxd_dir/storage-pools/$pool
mkdir $lxd_dir/storage-pools/$pool/$container

lxc delete $container

After deleting all containers from local, the pool can be deleted with
lxc storage delete $pool

umount $lxd_dir/storage-pools/$pool
rm $lxd_dir/storage-pools/$pool

I faced another minor bug. When container1 and local were still in the database, I ran lxc launch container1 -s different_pool, which resulted in an Already exists error, but container1 would mistakenly appear in the output of lxc storage volume list different_pool even though it wasn't created. Running lxc launch container1 -s different_pool after removing container1 from local fixes the problem: container1 no longer appears from lxc storage volume list different_pool, and the name container1 is reusable again.

@mlaradji Ah, I think the extra steps above likely wouldn't have been needed if you had restarted LXD after creating the new btrfs loop file.
As for the already exists and leftover volume issue, I believe I've fixed that in master already a few days ago, we were basically missing a bunch of volume deletion code on error.