[type: refactor] list all common binary media types.

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #2616 (e1523f9) into master (dd19154) will decrease coverage by 0.00%.
The diff coverage is 95.00%.

@@             Coverage Diff              @@
##             master    #2616      +/-   ##
============================================
- Coverage     61.14%   61.14%   -0.01%
- Complexity     4302     4307       +5
============================================
  Files           658      658
  Lines         18843    18860      +17
  Branches       1673     1671       -2
============================================
+ Hits          11522    11532      +10
- Misses         6389     6393       +4
- Partials        932      935       +3

Impacted Files
Coverage Δ

...a/org/apache/shenyu/common/constant/Constants.java
50.00% <ø> (ø)

...ugin/response/strategy/WebClientMessageWriter.java
80.43% <95.00%> (+18.36%)
:arrow_up:

...apache/shenyu/admin/utils/ShenyuDictH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
73.26% <0.00%> (-7.93%)
:arrow_down:

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 dd19154...e1523f9. Read the comment docs.


Add a table summarizing GTFS mappings

@caa06d9c please could r?

yup, a few bad links :(
now I think it is ready


Fix typo: ads -> adds

Coverage remained the same at 98.174% when pulling 714ac8b3d3b116650e8581551e923039f6ecefc7 on arku:patch-1 into 5c9b3a94aad8482ad8126cd7405a3060cd8e022e on form-data:master.

Thank you, @arku


Improve Adventures game

-Improve the invaders game, to demonstrate transitive closures via the "search" command.
-Update the game to lock the door and place the key in an envelop, which is in a draw, which is in desk.

+1


Can't compile rippled with BOOST_ROOT set

Having followed the installation instructions here:
https://xrpl.org/build-run-rippled-ubuntu.html
On Ubuntu Focal with boost 1.71 compiled, as per instructions, I get the following output:
$ echo $BOOST_ROOT
/home/admin/boost_1_71_0
$ pwd
/home/admin/rippled/build
$ cmake ..
-- The C compiler identification is GNU 9.3.0
-- The CXX compiler identification is GNU 9.3.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found Git: /usr/bin/git (found version "2.25.1")
-- Using 2 cores for ExternalProject builds.
-- Build type not specified - defaulting to Release
-- rippled version: 1.5.0
-- NIH-EP cache path: /home/admin/rippled/.nih_c/unix_makefiles/GNU_9.3.0/Release
-- docker NOT found -- won't be able to build containers for packaging
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
-- Check if compiler accepts -pthread
-- Check if compiler accepts -pthread - yes
-- Found Threads: TRUE
-- BOOST_ROOT: /home/admin/boost_1_71_0
-- BOOST_LIBRARYDIR:
CMake Error at /usr/lib/x86_64-linux-gnu/cmake/Boost-1.71.0/BoostConfig.cmake:117 (find_package):
  Found package configuration file:

    /usr/lib/x86_64-linux-gnu/cmake/boost_chrono-1.71.0/boost_chrono-config.cmake

  but it set boost_chrono_FOUND to FALSE so package "boost_chrono" is
  considered to be NOT FOUND.  Reason given by package:

  No suitable build variant has been found.

  The following variants have been tried and rejected:

  * libboost_chrono.so.1.71.0 (shared, Boost_USE_STATIC_LIBS=ON)

  * libboost_chrono.a (shared runtime, Boost_USE_STATIC_RUNTIME=ON)

Call Stack (most recent call first):
  /usr/lib/x86_64-linux-gnu/cmake/Boost-1.71.0/BoostConfig.cmake:182 (boost_find_component)
  Builds/CMake/deps/FindBoost.cmake:273 (find_package)
  Builds/CMake/deps/Boost.cmake:41 (find_package)
  CMakeLists.txt:44 (include)

-- Configuring incomplete, errors occurred!
See also "/home/admin/rippled/build/CMakeFiles/CMakeOutput.log".
See also "/home/admin/rippled/build/CMakeFiles/CMakeError.log".

Looks like it has something to do with:
/usr/lib/x86_64-linux-gnu/cmake/Boost-1.71.0/BoostConfig.cmake

It seems to me that your build of boost cant find the proper packages. I suggest you re-build boost from scratch and give it a try. This is not a rippled issue - closing issue. Thank you

That was a bit abrupt and unhelpful.

@donovanhide I closed the issue because rippled is passing CI and I have just built it this morning myself. So its not an issue of rippled. However I am happy to see if I can help - can you try following the instructions here: https://github.com/ripple/rippled/blob/develop/Builds/linux/README.md and let me know if it works?

I linked to the more up to date instructions that I followed in the original report above. The problem is to do with the ubuntu version of cmake combined with the ubuntu version of the boost libraries, which is not directly linked to rippled, but it is likely that users of rippled will encounter if they choose to compile on a machine that they have compiled previous versions of rippled and boost on. This obviously isn't the case on a CI box.

If anyone else hits this, blitzing the /usr/lib/x86_64-linux-gnu/cmake/ directory and reinstalling cmake seems to fix it.

The bug exists also in Ubuntu 20.04.1. The bug arise because cmake have conflicts with the multiple installed boost versions.
As @donovanhide recommended I delete /usr/lib/x86_64-linux-gnu/cmake/ lib and rerun cmake command.


excessive memory usage in more

Running it over a test file of about 1G in size results in 15G RSS.
Problems start with slurping the entire thing upfront, I don't know how it manages to amass 15 x memory overhead on top of it.
more from gnu coreutils sits at around 2MB.
How to repro:

generate a file of about 1G: perl -e 'print "meh\n" x (256 * 1024 * 1024)' > testfile
cargo run -r testfile

note that we have been focusing on compatibility, not perf or memory usage for now (even if some programs are already doing better than GNU's)

True, but I also agree that 15x the filesize is a bit excessive 😄

Even 100% would be excessive, consumed memory shouldn't scale linearly with the file size at all.

It's also just extremely slow to load such a file. I'm looking into making this at least a bit better.

note that we have been focusing on compatibility, not perf or memory usage for now (even if some programs are already doing better than GNU's)

For coreutils sensible memory usage is part of correctness, especially so for a tool like more.
For example say I logged in to a production machine with the intent of running more /var/log/log_file_sized_1G_or_more. Suppose the the explosive memory usage problem is fixed, but the entire file is still slurped upfront, resulting in 1G RSS.
If the machine I'm running this on does not have a free 1G we are running into trouble. And chances are things are already going wrong, hence why I'm logging in to poke around (including reading logs) in the first place.
The gnu coreutils variant uses few MB of ram making itself a perfectly viable tool in that scenario.
That is to say this code needs to deal with the file in chunks and at the moment is not fit for use whatsoever. Any work done on it which would be thrown away while making the adjustment is a waste of time, which would be better spent either reworking this or beating other tools to shape in a lasting manner.

Not useful for you does not mean useless for everyone. We've already agreed this is an issue and want to fix it. Your point has been made.


Limit update checks to once per day

In #2893 we made more commands check for updates. Strictly not all of these checks are necessary as the code is unlikely to have changed regularly. Instead we should limit the amount we check for updates to (perhaps) once per day.
Every time we perform an update check, we write the current time into update.json. Every time we run a command. if the current time is less than 24h after this value, don't do any checking. If the current time is 24h ahead of this value, do an update check.
Resolves #2894.

We currently have a repo.version file that holds the current version of the repo (not bacalhau):
cat ~/.bacalhau/repo.version
{"Version":1}

We are now introducing a new update.json, which I still agree is the right thing compared to using config.yaml. I am concerned the repo will become a mess as more states need to be persisted. Does it make sense to create a single file that holds both repo.version, latest update check and all future states we care about?

Does it make sense to create a single file that holds both repo.version, latest update check and all future states we care about?

I'm not sure... this is essentially moving a key-value store implemented on the filesystem into a key-value store implemented in a single file. We'd just be moving "mess on the filesystem" into "mess in a yaml file" and I'm not sure if I think that being conservative on files on the filesystem is all that valuable? What problems do we think this is going to cause as it scales?

OK! Thanks for the comments everyone. I have now properly understood the config system and using it properly has actually made most of the issues around contexts disappear. So thanks for the push back @frrist!

I'm not sure if I think that being conservative on files on the filesystem is all that valuable? What problems do we think this is going to cause as it scales?

Just avoiding a bloated repo. No performance or scale issues, just a user experience issue when they see tens of files in their repo not knowing which ones they should or shouldn't touch. Shouldn't be a blocker for this PR, and we can make a more informed decision when we see more of these configs and states in the future. A slight improvement might be to make the file hidden. update.json -> .update.json, or have a directory inside the repo with all these files. Again we can delay this


Feature/pydantic v2 restore

@timvancann Hi, as I said in https://github.com/godatadriven/pydantic-avro/pull/91 I think those commits have lost in some bad force-push or something like this... Do you want to restore them yourself, or merge this PR?
Contains the relevant PRs:
https://github.com/godatadriven/pydantic-avro/pull/91
https://github.com/godatadriven/pydantic-avro/pull/82
Ignored those PR (since you edit the same files later):
https://github.com/godatadriven/pydantic-avro/pull/83
https://github.com/godatadriven/pydantic-avro/pull/84

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.52% :tada:

Comparison is base (393c50a) 90.59% compared to head (c7aa47a) 91.12%.

: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.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
+ Coverage   90.59%   91.12%   +0.52%
==========================================
  Files           2        2
  Lines         202      214      +12
==========================================
+ Hits          183      195      +12
  Misses         19       19

Files Changed
Coverage Δ

src/pydantic_avro/base.py
98.19% <100.00%> (+0.21%)
:arrow_up:

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

No clue what happened. It wasn't intentional in any case. I'll merge this one again.

Great thanks!
So now we just need to bump another version so it will be published, right?


ARROW-12658: [C++] Bump aws-c-common to v0.5.10

This updates aws-c-common to the latest version to fix build errors with new clang versions

Not sure about the Java fail but this is probably unrelated? Feel free to merge this.

I see the same Java test failure in https://github.com/apache/arrow/pull/10242 so I assume it's unrelated.


Rendering Failed error with plot

Describe the bug
Plot command gives me Rendering Failed error.
Output:
/usr/local/Wolfram/WolframEngine/14.0/SystemFiles/FrontEnd/Binaries/Linux-x86-64/WolframPlayer: error while loading shared libraries: libGL.so.1: cannot open shared object file: No such file or directory
Environment (please complete the following information):

Operating system: Ubuntu 22.04.3
Mathematica/Engine: Engine 14.0
VS Code: code cli 1.86.1 for ubuntu, code 1.86.1 for window, connected via Remotes - SSH extension
Version: 0.1.1

After apt install libgl1-mesa-glx, Execute cell never ends :(

sudo apt-get install libegl1 and solved!


Added M-Code Variant for Fanuc i-Series Controls

Pull Request
Description
Due to the multi-chuck and live tooling nature of these lathes they have unique M and G codes that aren't shared with most other lathes. I've gone ahead and added all the common M codes we use in our shop with these machines. I have begun work on the G code variant list, but it  is incomplete currently.
Issues

Reviewer Notes
This is my first time contributing to a github project, I believe I've done everything correctly but apologies if something was missed.

Checklist

[x] I have followed the guidelines in the Contributing document.
[x] I have followed the Code of Conduct for this project.
[x] My changes follow the coding style of this project.
[x] I have run npm run test-all and it completed without errors.
[x] My changes have been formatted and linted.
[x] I have made any relevant changes to the documentation as needed (include CHANGELOG.md and/or README.md).
[x] My changes have a descriptive commit message with a short title, including links to issues it addresses.
[x] I had fun.

I had filled out the description, but for some reason it cleared it when I posed the request. I added support for M-codes for fanuc i-series lathes, and have begun but not finished a G code list as well.

Thanks! I'll look into this and we can merge it in when its finished if need be.


Array removal is prohibited when minlength is available but adding is not prevented when max length available

Not needed

Not needed


police ask id permisson nested menu changes

the new way you have nested the police menu now provides all players with a police menu option to use the ask id function which is confusing for some players wondering why they are not police but have a police option in their menu with that single option , is there a way to clone this in a commit to have it its own option for anyone not a police job. thanks

police.askid not work? Remove from user group and add to the police group?

the option works fine the point is every player now has a police option in the menu to use the ask id function since it was not cloned to be used by civilians , it is confusing for players who are not police to have a police option to ask for id

https://github.com/ImagicTheCat/vRP/commit/3d0e9b8d26b8aacee7e27193fff6787bb4bf4441


Monospace font

Currently, the font is not a monospaced one. This, unfortunately, makes the app unusable for Markdown notes and code snippets.

Hi @fladd , thanks for your opening issue!
I will check about that!

@kazup01 cancelled funding, $20, of this issue. Visit this issue on Issuehunt


[$250] Receipt fails to upload when smart scanned and typo in error message

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.30-11
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @cead22
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1725645046337449
Action Performed:
1.Launch the app and login
2.Submit an expense by smartscanning
Expected Result:
Receipt uploaded and no typo in error messages
Actual Result:
Receipt fails to upload and No space before Download the file on the error message. No error message displayed desktop (For failed upload)
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021833220161234377162
Upwork Job ID:<PHONE_NUMBER>234377162
Last Price Increase: 2024-09-09
Automatic offers:

allgandalf | Reviewer | 103975547
Nodebrute | Contributor | 103975549

Issue OwnerCurrent Issue Owner: @allgandalf

Proposal
Please re-state the problem that we are trying to solve in this issue.
Receipt fails to upload when smart scanned and typo in error message
What is the root cause of that problem?
We are not adding a space after Localize.translateLocal('iou.error.receiptFailureMessage')
https://github.com/Expensify/App/blob/342b1d51f966a5f9fa9a2f6103c3eb8463f938e0/src/components/DotIndicatorMessage.tsx#L71-L79
What changes do you think we should make in order to solve the problem?
We should add space after Localize.translateLocal('iou.error.receiptFailureMessage')
{`${Localize.translateLocal('iou.error.receiptFailureMessage')}+" "`}

What alternative solutions did you explore? (Optional)

@allgandalf Adding aspace after . in en.ts will add space in every usage of the copy. And it seems we only want to add the space when another translation comes after it. Don't you think it is better to add the space where the copy is followed by another copy or translation  as my proposal suggests?
cc: @luacmartins

I don't know if these proposals make sense. The main issue is the receipt failing to upload. The issue with the typo in the error message is secondary.
If we're only gonna solve the typo, let's log a different issue that only talks about the typo. The main issue to solve is the smartscan upload failing

I know. The receipt uploaded and SmartScanned, and I still got an error in New Dot, which made me think this is a front end issue

No, I just uploaded a receipt and it worked fine. And I don't have repro steps, so I'm gonna close this


Adding Linux & Mac support + readme changes

No noticeable problems detected; approved. Please note that I will eventually be changing the install script path soon as to provide a larger category for ease of use regarding the Windows batch process and other soon-to-be preset scripts.


Dataview generated links that have an ampersand (&) are broken

Normal links parse ampersands correctly. For example [[Updates & About]] goes to the slug updates-and-about as desired. However, links generated by dataview parse to updates-and-amp-about.
Thanks for the amazing plugin :)

Thanks for reporting this @shervinsahba! I've just relased a fix for this in the template. Updating the template should fix the issue. Let me know if you get it working or note 😄

It's working now. Thanks!


tensorboard display nothing after update to v1.2.0

I update to tensorflow 1.2.0 by building from source.
after that, tensorboard show nothing.
Is the tensorboard changed?
I use command:
tensorboard --logdir=log

It seems tensor board disappeared in v1.2.0 ?
how should I install tensor board in v1.2.0

https://github.com/tensorflow/tensorboard

When I install tensorflow compiled from source v1.2.0, it automaticly do bellow:
pip install tensorflow-tensorboard,
This tensorboard seems to display nothing!
However using (https://github.com/tensorflow/tensorboard) this one is OK.
Is it a bug?

@dandelionmane @jart

@cxwx Can you open an issue at github.com/tensorflow/tensorboard with the following information:

What was the output that TensorBoard printed to console?
What was the output that TensorBoard printed to web console? (in chrome, right click on the page + click "inspect")
What browser are you using?
What OS are you using?
What exactly is TensorBoard displaying?

I'm closing this issue since this is the wrong place for TensorBoard issues.


Some problems with load application in ex2

In the following code :
 VectorArrayCoefficient f(dim);
    for (int i = 0; i < dim - 1; i++)
    {
        f.Set(i, new ConstantCoefficient(0.0));
    }
    {
        Vector pull_force(mesh->bdr_attributes.Max());//3
        pull_force = 0.0;
        pull_force(1) = -1.0e-2;
        f.Set(dim - 1, new PWConstCoefficient(pull_force));
    }

In this line of code  pull_force(1) = -1.0e-2    What does the number 1 in pull_force represent . I thought 0,1,2 represented forces in X,Y, and Z, but they didn't.If I need to apply a load in the X,Y, and Z directions, what should I do?

Hi @HFH104864,
The array pull_force is used as an argument for a PWConstCoefficient which is a piecewise-constant coefficient on the boundary attributes. Here 1 referrers to the second boundary attribute (counting from 0).
The X,Y, and Z directions are the indices used in f.Set(. For example the first loop makes all components of the force vector zero, while the following block sets the last one, i.e. Y in 2D, to be 0 on boundary attribute 1 and -0.01 on boundary attribute 2.
Hope this helps,
Tzanio

thank you . it helps me a lot!!


Error In Install

Hi All.
Your connection has timed out
Either your request was invalid or the program hasn't completed your request.
Please notify the server admin

Hi Ahriman, thank you for contacting us. Sounds like you are trying to connect to a rsync server without the right configuration. Did you add the public ssh key to your remote location? Check this page for further information.
If you did, please give us a little bit more information about your setup so we can help you.


Update Documentation

What are you trying to accomplish?
Having struggled to set up the bot on my own, I have uncovered some issues with the documentation in terms of setting up the bot with Docker. This PR aims to fix this, by providing detailed information about the setup and configuration of the bot. This PR also cleans up the grammar throughout the entire README.md.
How are you accomplishing it?
Dramatically increased the detail in the former "Config" section of the Readme. It now covers what each of the configuration options does, as well as where to find the external information needed, in the case of the API token and Client ID.
Additionally, I moved the Docker setup instructions to come before the older manual Ruby setup instructions, since the Docker steps are recommended over the old Ruby steps.
Is there anything reviewers should know?
Not that I can think of 😃

[x] Is it safe to rollback this change if anything goes wrong?

this is great. good job dude. <3

i uhhhh forgot to merge it before closing so did that


fix default options

I've worked over the default options management to make sure the options from the feathers config get always considered. The priority order is:
hook defaults < feathers config < hook (service) options
This PR is supposed to fix https://github.com/idealley/feathers-hooks-rediscache/issues/44
@idealley would you please review and let me know what you think abt it?

Coverage decreased (-0.3%) to 92.683% when pulling 2397ec90bde36ff71885768a18e6db5174cb163f on SassNinja:feature/fix-default-options into a4a6e3164c6d41900d02a38d7a4cf5b4ff41854f on idealley:master.

Thank you for the issue and the patch! I have just added a test and merged. NPM version will be published as soon as travis completes.
Sam


⚠ PrivateHD has degraded performance

In ad5eba7, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 3215 ms

Resolved: PrivateHD performance has improved in e22fd83.


[ISSUE #8103] Fix SimpleSubscriptionData equal

Which Issue(s) This PR Fixes

Fixes #8103
Brief Description
Fix SimpleSubscriptionData equal
How Did You Test This Change?
unit test

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

Project coverage is 42.86%. Comparing base (b39f65e) to head (e8d2e3a).
Report is 14 commits behind head on develop.

Files
Patch %
Lines

.../protocol/subscription/SimpleSubscriptionData.java
50.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8104      +/-   ##
=============================================
- Coverage      42.88%   42.86%   -0.03%
- Complexity     10348    10362      +14
=============================================
  Files           1270     1270
  Lines          88673    88690      +17
  Branches       11398    11399       +1
=============================================
- Hits           38029    38014      -15
- Misses         45950    45981      +31
- Partials        4694     4695       +1

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


Multiple conditions in the head of a condition literal

I'm struggling to understand why the following code gives me 'F'' is unsafe.
progression(O, F, S, R) :- operation(O, F, S, R); F != F', F != S' : operation(_, F', S', R'), R' > R.

It seems that clingo is interpreting F != F' as if it is outside of the conditional literal. Is it not possible to have multiple conditions in the head of a condition literal? Indeed, the documentation seems to imply that it isn't. But then, why does running the following code not give me the same error?
neq(X, Y) :- X != Y, variable(X), variable(Y).
progression(O, F, S, R) :- operation(O, F, S, R); neq(F, F'), neq(F, S') : operation(_, F', S', R'), R' > R.

If you have a variable in the head, or even outside the conditional literal set expression, it takes precedence over the variables in the conditional literal. It bothered me for some time and I believe it is bothering you now.
On Tue, Jun 11, 2024 at 18:21, João Cabral Pinto @.***(mailto:On Tue, Jun 11, 2024 at 18:21, João Cabral Pinto < wrote:

I'm struggling to understand why the following code gives me 'F'' is unsafe.
progression(O, F, S, R) :- operation(O, F, S, R); F != F', F != S' : operation(_, F', S', R'), R' > R.
It seems that clingo is interpreting F != F' as if it is outside of the conditional literal. Is it not possible to have multiple conditions in the head of a condition literal? Indeed, the documentation seems to imply that it doesn't. But then, why does running the following code not give me the same error?
neq(X, Y) :- X != Y, variable(X), variable(Y).
progression(O, F, S, R) :- operation(O, F, S, R); neq(F, F'), neq(F, S') : operation(_, F', S', R'), R' > R.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Remark 3.6 from the guide:
During grounding, the instantiation of global variables takes precedence over non-global ones, that is, the former are instantiated before the latter. As a consequence, variables that occur globally are substituted by terms before a condition is further evaluated. Hence, the names of variables in conditions must be chosen with care, making sure that they do not accidentally match the names of global variables.
On Tue, Jun 11, 2024 at 18:34, Mail @.***(mailto:On Tue, Jun 11, 2024 at 18:34, Mail < wrote:

If you have a variable in the head, or even outside the conditional literal set expression, it takes precedence over the variables in the conditional literal. It bothered me for some time and I believe it is bothering you now.
On Tue, Jun 11, 2024 at 18:21, João Cabral Pinto @.***(mailto:On Tue, Jun 11, 2024 at 18:21, João Cabral Pinto < wrote:

I'm struggling to understand why the following code gives me 'F'' is unsafe.
progression(O, F, S, R) :- operation(O, F, S, R); F != F', F != S' : operation(_, F', S', R'), R' > R.
It seems that clingo is interpreting F != F' as if it is outside of the conditional literal. Is it not possible to have multiple conditions in the head of a condition literal? Indeed, the documentation seems to imply that it doesn't. But then, why does running the following code not give me the same error?
neq(X, Y) :- X != Y, variable(X), variable(Y).
progression(O, F, S, R) :- operation(O, F, S, R); neq(F, F'), neq(F, S') : operation(_, F', S', R'), R' > R.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

@ejgroene Thanks for your response! But I still don't understand how to fix the issue. What would you do?

The first thing I would do is rewrite it so it becomes less ambiguous. To me it is unclear, without looking at the guide, what belongs to the conditional literal and what not.
On Tue, Jun 11, 2024 at 18:40, João Cabral Pinto @.***(mailto:On Tue, Jun 11, 2024 at 18:40, João Cabral Pinto < wrote:

@.***(https://github.com/ejgroene) Thanks for your response! But I still don't understand how to fix the issue. What would you do?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

We cannot help you here without knowing what the program is supposed to mean. If you are beginning to write ASP, I would suggest to completely ignore conditional literals and write your program without.

To get started, you could check out our guide or the book Answer Set Programming by Vladimir Lifschitz. There is also a free draft available.

Hey @rkaminsk, I am quite new but I've already read much of the guide and I'm still unable to fix the issue without resorting to a verbose solution.
My objective with the excerpt I sent is as follows. Given a set of arithmetic operations, such as the one below, I want to create a corresponding progression predicate for every operation that has none of its operands used again in the operations that follow (tested with R' > R).
operation("-", "A", "B", "C"). % A - B = C
operation("+", "C", "D", "E"). % C + D = E
operation("+", "C", "F", "G"). % C + F = G

In this example, I'd want my code to output progression("-", "A", "B", "C"). and progression("+", "C", "F", "G"). The second operation doesn't get a progression predicate because C is used again in the following operation.

I can fix the error by splitting the conditions into multiple condition literals (seen below), but this results in a lot of code repetition. Isn't there a cleaner option?
progression(O, F, S, R) :-
  operation(O, F, S, R);
  F != F' : operation(_, F', S', R'), R' > R;
  F != S' : operation(_, F', S', R'), R' > R;
  S != F' : operation(_, F', S', R'), R' > R;
  S != S' : operation(_, F', S', R'), R' > R.

What about the following?
operation("-", "A", "B", "C"). % A - B = C
operation("+", "C", "D", "E"). % C + D = E
operation("+", "C", "F", "G"). % C + F = G

arg(X,R) :- operation(_,X,_,R).
arg(X,R) :- operation(_,_,X,R).

progression(O, F, S, R) :- operation(O, F, S, R);
    #count { R' : arg(F,R'), R' > R;
             R' : arg(S,R'), R' > R } == 0

Good idea! Thank you.
However, as a noobie, I can't help but wonder why clingo was designed without some of the constructs that are commonplace in most languages, like parenthesis to take precedence or the "or" operator. I feel like I spend most of my time finding ways to go around these missing features. Could you provide some insight on why it is the way it is? Is there maybe any plan to add these things in the future?

However, as a noobie, I can't help but wonder why clingo was designed without some of the constructs that are commonplace in most languages, like parenthesis to take precedence or the "or" operator. I feel like I spend most of my time finding ways to go around these missing features. Could you provide some insight on why it is the way it is? Is there maybe any plan to add these things in the future?

In my experience, one does not need nested Boolean connectives for modeling most tasks. It is possible to encode a lot of problems mainly using normal rules. One design goal of the language has always been to keep the language as simple as possible but expressive (in sense of being able to write compact encodings). It is normal that it takes some time to develop a feeling for encoding problems. It might take a little longer to come up with good ASP code but in the end you get something compact and maybe even nicely readable.
You can even encode your problem just with normal rules:
arg(X,R) :- operation(_,X,_,R).
arg(X,R) :- operation(_,_,X,R).

covered(R) :- arg(X,R), arg(X,R'), R'>R.

progression(O,F,S,R) :- operation(O,F,S,R), not covered(R).

Actually, I managed to code it with just one! I just adapted your first idea a little:
#count { R' : operation(_, F', S', R'), (F; S) = (F'; S'), R' > R } = 0.

Thanks for your input. What I would argue is that adding parenthesis (to define precedence) and the or operator would result in more expressive code without adding any significant complexity or readability issues for the end user. I hope you consider it!


Small SILValue::getOwnershipValue() fixes

This PR contains a few fixes for SILValue::getOwnershipValue() that I discovered as I was cleaning up ownership in SILGen.
Specifically:

thin_to_thick_function should return an owned value even though its input is trivial.
unchecked_bitwise_cast does not return trivial semantics. Instead it has the semantics that:
a. trivial -> trivial => trivial
b. trivial -> non-trivial => unowned
c. non-trivial -> trivial => trivial
d. non-trivial -> non-trivial => forwarding
objc_{existential_,}metatype_to_object is owned. In SIL metatypes are trivial. We only convert them to full objects on request for usage in objective-c. These instructions convert metatypes to objects. Thus they should return the value as owned. The one interesting part here is that one may think that these should return an unowned value (since we are converting from trivial => non-trivial), but since in objective-c land class types are eternal (with a special retain/release that do nothing), we can avoid needing to do that.

rdar://29791263

@swift-ci Please smoke test and merge


Decrease max test task time to 60 seconds

I don't think that there are any tasks that we expect to take this long. Let's find out sooner rather than later if any tasks do take this long.

Actually, it seems that the job doesn't get aborted if it reaches the timeout limit; the job is retried (see documentation comment here: https://github.com/linkyndy/pallets/blob/b93ea47/lib/pallets/configuration.rb#L23-L25 ). (See Travis build log for evidence of this: https://travis-ci.org/github/davidrunger/david_runger/builds/699170675 .) This is not really the behavior that we want.
I guess that I'll just delete this config, then, and leave the default job timeout period of 30 minutes.

I guess that I'll just delete this config, then, and leave the default job timeout period of 30 minutes.

See https://github.com/davidrunger/david_runger/pull/2888


pycflow2dot: init at 0.2.3

Motivation for this change
pretty graphs

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)

/nix/store/b7pzfnwx37z1571vzjjnszvwh9hbnniy-python2.7-pycflow2dot-0.2.3	  358191208

[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

odd, python2.pkgs.pycflow2dot fails in a pure shell
TypeError: decode() argument 1 must be string, not None
seems to come from here (no default locale?)
this issue does not occur in my regular environment (sway)
nor does it in the build environment, which has LANG=C.UTF-8
setting LANG=C.UTF-8 in the pure shell resolves this, not sure how to apply this knowledge to the package though


[Snap-CI] Replace with error message

Hello,
This prepares the work needed to close #1466. There are still discussions around colour in #1449, so I have left out any colour assignments for now. This pull request should therefore probably not be merged until then.
No tests previously existed, so I simply added a "no longer available" test, which will also tick one item from the top table in #1358.
Cheers,
Pyves

Thanks everyone for reviewing. Merged!


Incorrect indent after new line

Type: Bug
I don't really have any steps to reproduce... I was writing a simple python script when, suddenly, whenever I tried pressing enter to go down a line it looks like it went down to the correct indent but instantly went to the beginning of the line.
After running extension bisect it seems that the problem was in Juypter...


Fail tests on flake8 failure

For syntax errors we anyways fail the test, for the other errors we show them as warnings, but if the total count of these exceeds 10 then we throw an error asking user to fix those lint errors

Why only if greater than 10 BTW?

@guysmoilov just some number because you asked in the ticket to have a certain range, so I thought 10 would be a good start. Its the sum by the way. So if the total weight of issues is 10 then it will fail. Should I reduce it to 5 ?

I guess I don't understand how flake8 works, I assumed we can set a "maximum allowed severity level"

Then we can do selective error codes like you did for other flake8 test you are already running

But I have slight preference over testing everything, and not just selected ones. Code is more hygiene that way.


add github action for tests

resolve #11

This is now ready! 🎉

This should be merged before #16


Detect embedded "$id"/"$anchor"/"$dynamicAnchor" as valid "$ref"/"$dynamicRef" targets

This scenario (which is apparently not tested int he JSON Schema test suite) is not currently supported:
{
  "$id": "https://example.com/schemas/root"
  "$defs": {
    "foo": {
      "$id": "foo",
      "type": "string"
    }
  },
  "$ref": "foo"
}

The distinction here is that evaluation never passes through "#/$defs/foo" before it is referenced.  The JSON Schema test suite only tests embedded "$id"s that are evaluated before they are referenced, apparently.
There are essentially two ways to do this:  Walk all applicator and schema location ("$defs") keywords at load time and index the "$id"s, or trigger a scan of loaded schemas upon finding a "$ref" that cannot otherwise be resolved.
I am personally inclined towards scanning up front, which also makes some obscure cases involving embedded "$schema" (which I'll file separately) if jschon wants to support them.  But there are pros and cons either way.

After digging through the code for "$anchor" and "$dynamicAnchor" I am fairly certain they suffer the same problem.  For "$anchor", this is tested by a recently added test case (refRemote.json -> anchor within remote ref -> remote anchor valid) for both 2019-09 and 2020-12 (and presumably draft-next).
@marksparkza I'd propose handling this by:

Defining a SchemaLocationMixin from which both ApplicatorMixin and DefsKeyword would inherit
Document that all keywords that have subschemas, whether they are applied automatically (ApplicatorMixin) or not (DefsKeyword and similar constructs such as probably contentSchema) MUST inherit from SchemaLocationMixin
Require subclasses of SchemaLocationMixin to implement a generator method that returns each subschema
Implement a generator method on JSONSchema that chains all SchemaLocationMixin generator results
Adjust the standard keyword class inheritance accordingly
Use these generators to walk each schema document on loading and populate the catalog with the targets found

This approach could also be the basis for implementing #59, if there is a desire to support it.

BTW I made that proposal up off the top of my head and am not attached to it, I just wanted to offer a starting point.

Having dug around a bit more, it seems your ApplicatorMixin hierarchy overlaps a lot with my SchemaLocationMixin hierarchy idea, as you are using it for DefsKeyword and not for the by-reference applicators.  I'll poke around more and make sure I understand this better.
I might suggest tweaking the class names to more closely align with the usage of "applicator" and "location" in the spec, unless that will be some sort of compatibility problem for the next release.  I'm also happy to stick with your use of "Applicator" if you'd rather.

@marksparkza I've dug around a lot more now, and I think I have a better feel for what you're doing with cataloging schemas and resolving references on-demand.  I'm playing around with some ideas and will post here or open a new issue if that feels more appropriate.
I think your approach can be extended to do a search of not-yet-instantiated subschemas when the catalog cannot find a schema, which could also be initiated up front (perhaps a command-line option so folks can choose their performance trade-off).
Or we could keep things simpler by just always scanning schemas when they are loaded.
There is a difficulty with load-on-demand (regardless of the source), which is that there's no way to detect an embedded schema in a resource that has not been loaded.  This is one reason that I default to pre-loading in memory with other tools (see #53, although the InMemorySource proposal wouldn't solve this, only direct loading into the catalog).
This is one reason why the spec discourages load-on-demand and encourages pre-loading.  I think it is sufficient to document that embedded resources in load-on-demand schemas will not be detected.  That's just a tradeoff users make if they want load-on-demand.
BTW I just want to say that this package is a joy to work with!  Especially as someone coming back to Python having not done anything major with it since the 2.7 days.  It's been a great way to learn what has changed.  I know I'm filing a bunch of stuff and proposing some fairly big changes here, but that's definitely not intended as criticism!

I'm going to close this because I confused myself several different ways.  I think I understand things now, and after doing a bit more work to be sure, I will re-file the correct issue for what is actually going on with the failing test case from the recent additions to the JSON Schema Test Suite.


util: add MountOptionsAdd() to add mount options

Describe what this PR does
There are a few occasions where the mount-options string is modified. This is not always done cleanly and could cause weird issues (fuse cephfs not allowing empty options).
The new util function MountOptionsAdd() takes a string, and adds the option in case it does not exist yet. It makes sure to return a correctly formatted (, separated, none at the start, nor end) string.
Is there anything that requires special attention
Idea while reviewing #1169 that does some more mount option string manipulations and corrections.

@mergifyio rebase

This pull request now has conflicts with the target branch. Could you please resolve conflicts and force push the corrected changes? pray

These have been resolved, please review again.

@humblec PTAL

@humblec, this one is still waiting for your review too


error in "Setup error handling"

Section 4.3 Error Handling – Custom without Rollback
The example does not look correct

At step 3 the order must be present.  It has been committed and we are in the situation "custom without rollback"
I just attended the training.  In the syllabus the example is correct.

Thanks @WimStevens I've made the correction via the above commit.


Freya 364 show funding and events on the right hand margin

Let me know what you think of the design so far. It's inspired by the News section and I didn't want to spice it up too much, but keep it clean and neat.

Final review required

Great work @JanProgrammierung 👏  if @LianeHughes is happy with the text and all, can be merged.


Adding leftover strings to resource file for localization

    L"Install",  // text

do these need to be localized too?

Refers to: preview/MsixCore/msixmgr/InstallUI.cpp:462 in c1cdcd2. [](commit_id = c1cdcd2ca64ddad33241460fc87decb928c539e5, deletion_comment = False)

    L"Cancel",  // text

and this

Refers to: preview/MsixCore/msixmgr/InstallUI.cpp:481 in c1cdcd2. [](commit_id = c1cdcd2ca64ddad33241460fc87decb928c539e5, deletion_comment = False)

    L"Installing app package",

this?

Refers to: preview/MsixCore/msixmgr/InstallUI.cpp:520 in c1cdcd2. [](commit_id = c1cdcd2ca64ddad33241460fc87decb928c539e5, deletion_comment = False)

    L"Reason:",

this?

Refers to: preview/MsixCore/msixmgr/InstallUI.cpp:552 in c1cdcd2. [](commit_id = c1cdcd2ca64ddad33241460fc87decb928c539e5, deletion_comment = False)


Prevent TestNG from adding duplicate instances of the same listener

TestNG Version

Note: only the latest version is supported

Expected behavior
Irrespective of whichever mechanisms that I use to add a listener to TestNG i.e., either via

Service Loader (or)
@Listeners annotation (or)
<listeners> tag
runners: surefire plugin, ant, cli, ...

TestNG must only register one listener instance to it.
Actual behavior
Currently I can inject the same listener as multiple instances via each of the above cited 4 mechanisms, thus causing the same listener's implementation being invoked multiple times.
Is the issue reproductible on runner?

[X] Shell
[X] Maven
[X] Gradle
[X] Ant
[X] Eclipse
[X] IntelliJ
[X] NetBeans

Test case sample

Please, share the test case (as small as possible) which shows the issue

There is no specific sample for this. Issue can be easily recreated by wiring the same listener through all the above mentioned mechanisms.

In fact, it depends on the listener but I can't remember the case I needed something different (@gjuillot do you?).
But first, we need exhaustive tests to check the current behavior to understand what will be the impacts of the change.

@juherr - Thanks for your inputs. I would also be curious to understand a use-case wherein a user requires multiple instances of the same listener to be wired into the same instance of TestNG.
So far I have been experiencing this as a problem (lets say I built a listener which implements IInvokedMethodListener wherein inside its beforeInvocation() a browser instance gets created for every @Test method whenever it sees a @Browser annotation. Duplicate instances of this listener ends up causing multiple browser instances getting created and thus causing unwanted confusions. To circumvent this, I had to build a custom ListenerManager implementation which resorts to having all the listeners register itself to it, and then within every method call of the listener, I end up adding a query to the listener manager to check if the listener method can be invoked on the instance. I kind of feel this is a lot of boiler plate code and it would be easier if TestNG maintained it as a map so that we can add the appropriate edit checks.
Also please note as part of #1230 we are already trying to ensure that atleast with IReporter implementations there are no duplicates (It already used a set and now we have changed it to start using a Map

But first, we need exhaustive tests to check the current behavior to understand what will be the impacts of the change.

Can you please let me know what you are hinting at ? Are you looking for a test which maybe looks something like this ?
@cbeust - It would be great if you could also please help advise on this requirement.

Can you please let me know what you are hinting at ?

I'm looking for a global view of the current listener state (which one is called/instantiated, when/how many times/...).
Tests from https://github.com/cbeust/testng/pull/1093/ are doing a part of the job but I don't know if it is enough.

Ok, you got me! I mismatched things. Go ahead then! 😄

Thanks Julien. I will work on submitting a pull request for this.
Thanks & Regards
Krishnan Mahadevan
On Tue, Nov 8, 2016 at 6:44 PM +0530, "Julien Herr"<EMAIL_ADDRESS>wrote:
Ok, you got me! I mismatched things. Go ahead then! 😄
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


AMBARI-25785 : upgrade jackson-databind to resolve CVEs

What changes were proposed in this pull request?
upgrade jackson-databind to <IP_ADDRESS>
How was this patch tested?
built ambari rpms with the proposed change and tested by installing a cluster with few services

+1, thanks @AnanyaSingh2121


is the model can be downloaded and export to Coreml?

DL Models trained on PowerAI Vision for classification and object detection are not exportable.


UI Not Displaying Correctly

Expected Behavior:
UI Interface displays correctly.
This could just be a config issue but i don't know how to fix this - Thanks
Actual Behavior:

Hi @henryp007,
This is how BookStack would appear if no styles are loaded. I'd guess this is due to one of two things:

During installation the master branch has been cloned instead of the release branch. The master branch does not include compiled CSS or JS content. You can check your version by going to /settings and looking for the version number or you can open the VERSION file at root of the installation.
Something like a browser extension is blocking the request to the CSS/JS files. This could also be due to a mis-configured server setup.

If the issue is not 1 let me know the version you're on and the web server you're using (Apache/Nginx/Something else) and we can go from there.

Closing due to lack of response, Feel free to still comment if you want to respond to the above.

@ssddanbrown
I got the same issue, as explained here.
I assume I need a special configuration for NGINX, though I'm not familiar with advanced PHP setups.

@theAkito Do you have the APP_URL env option set?
Feel free to open a new issue for this since this one is quite old and closed, hence any responses could be missed.


"Internal debugger error"

Which OS: OSX 10.11.6
Which LLDB version (lldb -version): lldb-3<IP_ADDRESS>
Which VSCode version (Help/About): Version 1.9.0-insider (1.9.0-insider)
What is the problem and how did you get there: Trying to step-thru debug a simple C++ application and I keep seeing the following errors over and over in the DEBUG CONSOLE:
Internal debugger error:
Traceback (most recent call last):
  File "./adapter/debugsession.py", line 695, in handle_message
    result = handler(args)
  File "./adapter/debugsession.py", line 477, in DEBUG_evaluate
    return self.evaluate_expr(args, expr)
  File "./adapter/debugsession.py", line 501, in evaluate_expr
    raise Exception('Missing frameId')
Exception: Missing frameId
[10:53:42 debugsession] Internal debugger error:
Traceback (most recent call last):
  File "./adapter/debugsession.py", line 695, in handle_message
    result = handler(args)
  File "./adapter/debugsession.py", line 477, in DEBUG_evaluate
    return self.evaluate_expr(args, expr)
  File "./adapter/debugsession.py", line 501, in evaluate_expr
    raise Exception('Missing frameId')
Exception: Missing frameId

The application's output is not affected but get buried since this message prints many many times.

Oh, hm.  Looks like frameId is now optional.

Fixed in 0.3.2


can support to keep some packages not be renamed?

how to use the keep rule?
for example:
keep com.**
it does not work

Finally, I tried to use the "rules" rule and it succeeded.
for example:
A jar file contains

com.x.A.class
com.x.B.class
com.y.C.class

i want to keep A.class not be renamed，and rename B.class and C.class.
edit the rule.txt,like:
rule com.x.A @0
rule com.** rename.@0

execute command line, and check the output.jar, rename successful.
output jar file contains

com.x.A.class
rename.com.x.B.class
rename.com.y.C.class

The point is writing the keep name rule at first


Have a problem with cake-bnb stake v2

Yesterday, I was stake in cake-bnb (may be I misremember) but I see total staked is 0 .
Here is my transaction, what does it do?
0x32071da305c8ccbdb7c71c89701c774ebe4471fd4760f393e1da3e1d3ec837fc

I was track it and know that it sent to 0x73feaa1ee314f8c655e354234017be2193c9e24e Chief Master
The problem is how i can get it back or I just can't see it temporally(I will see it show in the pancakeswapV.2 farm) ?

I saw new contract on pool cake-bnb its address is 0x0eD7e52944161450477ee417DE9Cd3a859b14fD0.
Now, I don't know what happen to my transaction (0x32071da305c8ccbdb7c71c89701c774ebe4471fd4760f393e1da3e1d3ec837fc).

Seem like chief not cook for me. 😭 😭 😭

Solved it been in discontinue farm section.


Updating from 0.1.6 to 0.1.7 fails: nil pointer evaluating interface {}.queueGroup

Error: UPGRADE FAILED: template: job-executor-service/templates/deployment.yaml:158:26: executing "job-executor-service/templates/deployment.yaml" at <.Values.distributor.config.queueGroup.enabled>: nil pointer evaluating interface {}.queueGroup

Quickfix:
Add --set distributor.config.queueGroup.enabled=true
Long-term fix: See https://github.com/keptn-contrib/prometheus-service/pull/276/commits/8f187f6a7e301bbbc711afa6d3b8d24fe4bae146

This has already been fixed in https://github.com/keptn-contrib/job-executor-service/commit/567b8b8bb615c0af6bb9fded8d4a0c8c678dcf9f, but still needs to be released

I have tested 0.1.8-next.3 and that seems to work fine

Fixed in 0.1.8 release.


Add Secrets dialog retains information when cancelled

Description
When adding a secret to an action, if you open the dialog and enter details. If you accidentally close by clicking the background or actively click "Cancel" once you click "Add Secret". The previously entered details are still present.
Gitea Version
1.21.11
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Mac OS
How are you running Gitea?
command-line
Database
PostgreSQL

Could be seen as working as intented. If you close a unsaved modal accidentially, it's good to have the data back. Any saved data should of course be cleared and I think that is already the case.

Not sure I agree with you on a secrets dialog because the information is sensitive.
I think the solution would be to have an "are you sure" dialog when clicking the backdrop and when clicking cancel it will clear the data and close the dialog.


Update for xxHash 0.7.4

This version is slightly behind the head version of xxHash.  Are you planning to update?

Just checked the source code of xxHash and indeed, dev and master seems to be at 0.7.4.
However, the repo doesn't have a 0.7.4 tag yet, or a 0.7.4 release- until that is finalized I'll wait with implementing any changes lest they're unstable or bugged.

Updated to 0.8.1. Closing.


Video encoder timing test

Test that verifies video encoder timing, which uses VideoEncoder to encode several frames to a video file and verifies the duration using the Video class with a new Duration() function.

I believe that the Windows failure here is somewhat expected, being that av wasn't currently being built.  The integration test needs to be dependent on that component being successfully built.


feat(inputs): adding maxHeap size option

Running into out of memory issues and I think this setting might be useful. See https://spotbugs.readthedocs.io/en/latest/faq.html#q2-spotbugs-is-running-out-of-memory-or-is-taking-a-long-time-to-finish

Thanks for the contribution!
I made my own PR https://github.com/advanced-security/spotbugs-findsecbugs-action/pull/10 containing my suggested change, so I'm closing this now.


Question: Removed AppObservable.bindActivity and AppObservable.bindFragment of RxAndroid 1.0.0

Right now I'm in the process of upgrading to RxAndroid 1.0.1.
How can I get the same behavior as AppObservable.bindActivity and AppObservable.bindFragment as of RxAndroid 0.24.0?
Since AppObservable used the isFinishing method for binding and that one is not available as an ActivityEvent I'm a little bit puzzled as how to I can sustain that behavior.
Is there any equivalent behavior / method?

There is no equivalent. bindActivity and bindFragment are not part of this library because they were ill-conceived and didn't really do what you'd expect. For example you could use it and your Observable might never end if it didn't emit at least one item after finishing.

@dlew What alternatives do we have - for the same behaviour?

You can use RxLifecycle to get automatic closing and should use observeOn(AndroidSchedulers.mainThread()) for the scheduling.


Custom routing results in no auto-redirect to /de or /en

I'm using custom routes in order to translate a variety of contenttypes, but would like my homepage to still redirect according to the language headers.  However, this doesn't seem to be possible once disabling the routing_overrides in the config file.
Details

Translate Version: [ dev-master ]
Install type: [ Extension Repository ]
Bolt Version: [ 3.1.2 ]
PHP version: [ 7.0 ]
Used webserver: [ Apache  ] version [ 2.4.10 ]

Reproduction
Disable routing overrides, and then try to load the root page of your site - it will cause an error page, instead of redirecting to /en or /de.
Problem Origin
There are lines in the file vendor/animal/translate/src/Frontend/LocalizedFrontend.php (lines 33-38) that read:
            $routes = array_merge([
                    'homepageredir' => [
                        'path' => '/',
                        'defaults' => ['_controller' => 'controller.frontend:homepageRedirect']
                    ]
                ], $routes);

This seems to add an entry to the routing.yml file in order to link the root path to the homepageRedirect function (in the same file), which handles the actual redirect.  By disabling routing_override, the controller entry is no longer created.
Solution
To solve this, you must simply create the entry manually.  I originally tried it using the namespace of the extension, which didn't work, but copying the controller line directly from the PHP script did indeed work.
The lines I added to routing.yml are:
homepageredir:
    path:               /
    defaults:           { _controller: 'controller.frontend:homepageRedirect'}

@SahAssar It didn't work quite "out of the box", but I found a solution while actually writing this issue.  Therefore, I'm closing it straight off the bat, but for anyone who needs the information, it's there.

@lswest This is one of those things where we need better (or any) docs... Glad you found a solution, but I'm going to reopen this and tag it to make sure that we get docs for it if you don't mind :)

If you want, I can write up the docs for this particular scenario. I'd just need to know your preferred approach.

@lswest I was planning to create a repo/site with basically a clone of the docs repo but with just info on translate. If you want to write up a .md file and PR it into a docs folder on this repo that'd greatly appreciated. Then we can have that as a start of our docs and then move it into proper docs :)


[Improvment] Pushbullet "Friends"

Currently you can only notify deviced linked to your own account. Is it possible to add a list of friends that should also be notified? For Example if there is something important going on I want to notifie all family memberes?
My Suggestion is an additional (optional) field with email-addresses of friends just like statet here:
https://docs.pushbullet.com/#create-push
If it is already possible to get the iden of a friend via the api please feel free to close the issue ;-)

@cimba007 I will check this and update you.

@cimba007 Added option to send Pushbullet with device idens , emails, channel tags. You can use single of multiple fields.

You are working very fast ;-) With this feature it is easy to notifie everybody on my household if something important happens ;-)

@cimba007 Kindly check this and close if everything works as expected.


Update QML feature branch with recent changes to master.

Accidentally created with wrong HEAD. Will fix momentarily.


Receive history messages with Firebase

Hi creators,
First of all, thank you for awesome framework, it makes my life easier.
Today, i tried to implement Chatto with Firebase and of course, trouble found me, LOL. Following is my issue.
In Firebase i just can receive history messages one by one using their API via .childAdded observer. And because i don't know what is last message so each message came, i have to call delegate chatDataSourceDidUpdate to present to screen, it's very slow, you know. It was a nightmare to me for debugging whole day. I think we can disable animation or something like that, but not sure. That's why i create this issue.
Please help to deal with Firebase. Thank you so much.

I'm also wondering, how to calculate width height for image that returned from internet? i.e: Firebase in my case

@crazyfree I never used Firebase, also I don't know your setup. But i've spent 5 minutes reading documentation and I see than you can get all messages at once by requesting specific path: https://firebase.google.com/docs/database/ios/read-and-write
Please, ask questions about Firebase on forums related to Firebase.

Fang Xia 赞了您的邮件。
Spark by Readdle


critical notification

the agents fails to call back to the empire server with regards to stagers of macro and oneliners category.hence this is the response with regards to python one liners "Traceback (most recent call last):
File "", line 1, in
File "", line 14, in
File "/usr/lib/python2.7/urllib2.py", line 154, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python2.7/urllib2.py", line 429, in open
response = self._open(req, data)
File "/usr/lib/python2.7/urllib2.py", line 447, in _open
'_open', req)
File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 1228, in http_open
return self.do_open(httplib.HTTPConnection, req)
File "/usr/lib/python2.7/urllib2.py", line 1198, in do_open
raise URLError(err)
urllib2.URLError: <urlopen error [Errno 111] Connection refused>"
i hope the developers will look into it. thank you

@eqhaleel Have you done any troubleshooting to eliminate other possibilities for what caused this issue?

@xorrior i only tried ./resetting the frame work

@eqhaleel I would recommend some more troubleshooting.


Add drag/resizing

Would be cool to have a single library that supports manual resizing interactions via click/drag/release in addition to all the cool stuff that dnd-kit already does! Love the modern approach this library is taking.
Wish List:

Resize component dimensions  (example from react-resizable)
Resize split pane dimensions (example from react-split-pane)

Suggest to consider re-resizable for this feature

@nelsieborja do you have an example of how both dnd-kit and re-resizable can work together? Which one should wrap the other?

Curious if anyone has a solution for this.

I needed something similar:
https://github.com/clauderic/dnd-kit/issues/1127#issuecomment-1803046796


All builds fail now after upgrade

I recently upgraded to the most recent version of gitlab-ci and gitlab-ci-runner, however now none of my builds are succeeding. They all instantly fail with execvp: Permission denied. Anyone know how to fix the permissions? I haven't found anything in the logs yet and haven't been able to track down what exactly is failing.

I can't submit a real bug report, because it appears Centos is not a supported OS for Gitlab CI Runner(s), but I have this exact error on 5.0-stable
execvp: Permission denied almost instantly on every build. Googling this error points to make scripts. Thing is, my runners are doing PHP only. I don't have a single make command anywhere.
Logs were no help
2015-02-06 23:17:12 +0000 | Submitting build 1484 to coordinator...ok
2015-02-06 23:17:17 +0000 | Completed build 1484, failed.
2015-02-06 23:17:17 +0000 | Submitting build 1484 to coordinator...aborted

Then I downgraded to 4-0-stable and it all worked again. I tried to look through commits for what broke this, but I couldn't find anything. Thanks for the tip @isuttell . Saved me from spending too much time on this.

@iBotPeaches I am using the latest runner on centos just fine. That's not the issue. I am not sure what's going on but I just wanted to reassure you about centos.

Anybody have any solutions to this? I'm experiencing this right now and don't know how to solve for it.

since GitLab CI 5.0 released you need update runners as well.

I JUST installed Gitlab-ci. I thought I installed the latest runners for Centos 6. If not, where can I find the latest runners?
Sent from my iPhone

On Apr 23, 2015, at 4:24 AM, Valeriy Sizov<EMAIL_ADDRESS>wrote:
since GitLab CI 5.0 released you need update runners as well.
—
Reply to this email directly or view it on GitHub.

I JUST followed the instructions for Centos 6 here: https://github.com/gitlabhq/gitlab-ci-runner
I initially ran with the master branch, same issue. I then swapped to the 5-0-stable branch, same issue.

From experiencing this, just downgrade to 4.0. Everything will work. Just tried latest hoping it was fixed, but no go. Back to 4.0

Awesome. just did the following and now builds are running (thank you):
sudo su gitlab_ci_runner
cd ~/gitlab-ci-runner
git fetch
git checkout 4-0-stable
bundle
exit
sudo service gitlab-ci-runner restart

Can you install recent version of GitLab CI and Runner?

Keep in mind that runner 5.0 and newer will work only with GitLab CI 5.0 and newer.

@csprocket777  4-0-stable is more than one year old. Can you describe what kind of problem you have with runners from https://gitlab.com/gitlab-org/omnibus-gitlab-runner/blob/master/doc/install/downloads.md and what version of GitLab CI do you use?

I am running GitLab CI 7.9.4 6037822
When using the runners from that location, I was getting the following error:
execvp: permission denied

You are getting this error in build trace, right?

yes

Can you send me your job script?

This is run on the 'develop' ref:
bower cache clean
npm install
bower install
if test -d dist; then rm -R dist; fi
if test -d tmp; then rm -R tmp; fi
ember build dist --environment=development

However, I took this out totally, and still received the error. I even removed ALL jobs and still got the error. I don't believe it even got to the point of running the jobs.

I think you meant that you removed all lines from job? Because build can be created based on some job.

Yes

Can you post here full trace then.

And please check logs.

That WAS everything in the trace. the ONLY line. And when I checked the logs, it said only had lines like:
Checking for builds...nothing

Can you try Go runner? https://github.com/ayufan/gitlab-ci-multi-runner
BTW. We are considering to make it official soon.


[V2] Upgrade to react 16

Reasons for making this change
Upgrade to react 16, fixes #1275.
If this is related to existing tickets, include links to them as well.
Checklist

[ ] I'm updating documentation

[ ] I've checked the rendering of the Markdown text I've added

[ ] I'm adding or updating code

[ ] I've added and/or updated tests
[ ] I've updated docs if needed

[ ] I'm adding a new feature

[ ] I've updated the playground with an example use of the feature

@fsteger do you know why these tests may be failing on upgrade to React 16:

ArrayField
List of inputs
should retain existing row keys/ids when adding new row:

ArrayField
List of inputs
should allow inserting anywhere in list:

ArrayField
List of inputs
should retain row keys/ids of remaining rows when a row is removed:

I had a look into this and it probably depends on some deprecated lifecycle methods used. I can rewrite everything to functional components using hooks as I already have some experience doing that on a large scale. Will be done by today and open a PR then.

It is actually more work than expected. I'm still working on that, biggest problem is that keys were used wrong. Seems like react 15 is passing the key prop to the child although it is only for internal use. With react 16 that prop is not accessible.
E.g. this is not accessible here
But getting there.

@zepatrik thanks for the update. What exactly is the change between react 15 and react 16 regarding keys? Is it that you can't pass on a key prop to a child? (that sounds strange, because if the key prop is accessible, then you can pass it on, right...?)

I could not really find a breaking change in reacts release notes but as you can see here (also look into the console) it is not possible to access the key prop (and apparently it was before)

Also static getDerivedStateFromProps() lifecycle is used which was added in react 16.3.0. The tests worked until now because the derived state was not used.

What do you mean by "the derived state was not used"? We did use static getDerivedStateFromProps but in combination with a polyfill from "react-lifecycles-compat". Will probably need to remove the polyfill though for react 16.

well because it appeared to not have been called, the state differed on the react version

Here are some debug information. Logging the nextProps and prevState args of getDerivedStateFromProps in ArrayField
This is also what the first failing test detects.
react 15:

react 16:

Steps to reproduce:

add a console.log statement
open the array example on the playground
click the add button twice

As you can see for some reason the component updates twice, once with an updated state and once with updated props.

tracked that as issue #1427

Very cool !


How long does it take to train the full model? (8 a100)

How long does it take to train the full model? (8 a100)

You can get a good evaluation result after 12h. For the model to fully converge, it takes about 48h.


Add MXNet example.

Add an example code of MXNet under examples/.

Can I take this up?

Merged: https://github.com/pfnet/optuna/pull/359


Html generator functions are platform dependent

Before late changes in html.c, there were attempts at making it platform independent but because of merge conflicts and commit chaos, these were never merged.
Dependency on:
#if __linux__
#include <linux/limits.h>
#else
#include <limits.h>
#endif

was solved with snprintf.
However, html_pathToRoot and html_getPath expect a buffer that has previously been allocated, so it cannot be solved anymore with just snprintf.
@SanderMertens, how stable is this part of the code? Can I start moving stuff around?

This basically refers to the same thing in #148, just now more complicated.


Cannot use random publicId

I haved config use_filename = false but after I uploaded imaged to cloudinary and the publicId I get still my orifinal file name

use_filename wasn't supported before, please try with v3.0.0 and let me know if you still have trouble.

Ok, I will try next time. Thanks for your reply


require is not defined

I'm trying to use this component in a Vue3 project but no matter what, I keep getting this error.
I created a Vue project with Vite, and it works great there. It seems like it only works with Vite?
Uncaught ReferenceError: require is not defined     at eval (winbox.bundle.min.js:2:1)     at ./node_modules/winbox/dist/winbox.bundle.min.js (chunk-vendors.js:2348:1)     at __webpack_require__ (app.js:2854:33)     at fn (app.js:3110:21)     at eval (index.mjs:7:64)     at ./node_modules/vue-winbox/dist/index.mjs (chunk-vendors.js:2315:1)     at __webpack_require__ (app.js:2854:33)     at fn (app.js:3110:21)     at eval (main.js:22:69)     at ./src/main.js (app.js:394:1)
PS: I managed to solve this error by removing 'type: "modules"' from the winbox package.json, but it's a temporary solution. I'd like to know if there is a definitive solution.

Are you using webpack/vue-cli?

Check this out. If none of the comments fixes your issue, you can patch winbox locally using patch-package and remove "type": "modules" in its package.json file.


[CSSolver] NFC: Refactor openGenericRequirements to fix ASAN failure

@swift-ci please ASAN test

Well, its not NFC, because it fixes an ASAN failure :)

@slavapestov It's NFC because it makes it identical to how it used to look when it was a part of openGeneric :)

@swift-ci please smoke test


Uncaught TypeError: Cannot read property 'length' of undefined

Hi, guys.
I use "react-sortable-tree" for visualize Folder Tree and "react-dnd" for copying node from Folder Tree to another component.
My issue for react-sortable-tree: https://github.com/frontend-collective/react-sortable-tree/issues/457
I receive error after dropping node from Folder Tree (SortableTree) to another component (DropTarget):
Uncaught TypeError: Cannot read property 'length' of undefined

Please, help me for fix the problem.

Got same error on end of dragging


Option request: align case alternatives like single-line arguments

For code that looks like:
(option1, option2) match {
  case (Some(Object(_, "property value")), _) |
    (None, None)                                       => 5
  case (None, Some(Object(_, "other property value"))) => 6
  case _                                               => 7
}

It gets rewritten to:
(option1, option2) match {
  case (Some(Object(_, "property value")), _) |
    (None, None) => 5
  case (None, Some(Object(_, "other property value"))) => 6
  case _                                               => 7
}

It would be nice to have the option to have case statements with multi-line LHS to still have their RHS aligned

This issue was moved to scala-ide/scalariform#172


Fix flakiness in the TopicInfo test

🦟 Bug fix
Fixes #510
Summary
The flakiness appears to be a timing issue. The fix here involves running the loop that waits for the input messages in a separate thread before starting the publisher.
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Homebrew CI has run 3 times without any failures:

https://build.osrfoundation.org/job/gz_transport-ci-pr_any-homebrew-amd64/200/
https://build.osrfoundation.org/job/gz_transport-ci-pr_any-homebrew-amd64/201/
https://build.osrfoundation.org/job/gz_transport-ci-pr_any-homebrew-amd64/202/


no such table error flutter

E/Dart    ( 4114): Invalid UTF8 sequence encountered,
E/Dart    ( 4114): (Error Code: FD + idx: 234 )
E/Dart    ( 4114): (Error Code: 64 + idx: 235 )
E/Dart    ( 4114): (Error Code: 22 + idx: 236 )
E/Dart    ( 4114): (Error Code: 20 + idx: 237 )
E/Dart    ( 4114): (Error Code: 3A + idx: 238 )
E/Dart    ( 4114): (Error Code: 20 + idx: 239 )
E/Dart    ( 4114): (Error Code: 69 + idx: 240 )
E/Dart    ( 4114): (Error Code: 64 + idx: 241 )
E/Dart    ( 4114): (Error Code: 2C + idx: 242 )
E/Dart    ( 4114): (Error Code: D + idx: 243 )
E/Dart    ( 4114):
E/Dart    ( 4114): Invalid UTF8 sequence encountered,
E/Dart    ( 4114): (Error Code: FD + idx: 234 )
E/Dart    ( 4114): (Error Code: 64 + idx: 235 )
E/Dart    ( 4114): (Error Code: 22 + idx: 236 )
E/Dart    ( 4114): (Error Code: 20 + idx: 237 )
E/Dart    ( 4114): (Error Code: 3A + idx: 238 )
E/Dart    ( 4114): (Error Code: 20 + idx: 239 )
E/Dart    ( 4114): (Error Code: 69 + idx: 240 )
E/Dart    ( 4114): (Error Code: 64 + idx: 241 )
E/Dart    ( 4114): (Error Code: 2C + idx: 242 )
E/Dart    ( 4114): (Error Code: D + idx: 243 )
E/Dart    ( 4114):
Restarted application in 1.887ms.
E/SQLiteLog( 4114): (1) no such table: personTablo
E/flutter ( 4114): [ERROR:flutter/lib/ui/ui_dart_state.cc(198)] Unhandled Exception: DatabaseException(no such table: personTablo (code 1): , while compiling: SELECT * FROM personTablo) sql 'SELECT * FROM personTablo' args []
E/flutter ( 4114): #0      wrapDatabaseException (package:sqflite/src/exception_impl.dart:11:7)
E/flutter ( 4114): <asynchronous suspension>
E/flutter ( 4114): #1      SqfliteDatabaseMixin.txnRawQuery.<anonymous closure> (package:sqflite_common/src/database_mixin.dart:404:30)
E/flutter ( 4114): <asynchronous suspension>
E/flutter ( 4114): #2      BasicLock.synchronized (package:synchronized/src/basic_lock.dart:33:16)
E/flutter ( 4114): <asynchronous suspension>
E/flutter ( 4114): #3      SqfliteDatabaseMixin.txnSynchronized (package:sqflite_common/src/database_mixin.dart:344:14)
E/flutter ( 4114): <asynchronous suspension>
E/flutter ( 4114): #4      KisilerO.readData (package:database/KisilerO.dart:15:37)
E/flutter ( 4114): <asynchronous suspension>
E/flutter ( 4114): #5      _MyHomePageState.writedData (package:database/main.dart:37:26)
E/flutter ( 4114): <asynchronous suspension>
E/flutter ( 4114):

Did you add this table after your app was already installed? If so, you need to write a migration for it.
During development (if no version has shipped to users yet), you can also just delete your app's data. This should make drift initialize a new database with all the tables on the next run.
I assume this problem was caused by the lack of a migration. If you did write a migration or if the table was there all the time, please let me know and I'll re-open the issue. A good way to check for derivations in the schema is to use verification at runtime on debug builds. If this still happens for you, can you enable that and see if it finds anything?

I don't like sqflite, I don't want to use it


[Event Request] Codeunit 99000854 - OnAfterAllowScheduleIn

Please add a new Publisher Call OnAfterAllowScheduleIn in Procedure AllowScheduleIn of Codeunit 99000854 "Inventory Profile Offsetting"
Add Call of Publisher in procedure AllowScheduleIn
        CanReschedule := CalcDate(TempSKU."Rescheduling Period", TargetDate) >= SupplyInvtProfile."Due Date";
+       OnAfterAllowScheduleIn(SupplyInvtProfile, TargetDate, CanReschedule);

New Publisher:
+    [IntegrationEvent(false, false)]
+    local procedure OnAfterAllowScheduleIn(SupplyInvtProfile: Record "Inventory Profile"; TargetDate: Date; var CanReschedule: Boolean)
+    begin
+    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Shortcut "i" stopped working with new version of Todoist

@mgsloan it looks like the shortcut i  has stopped working with the new version of Todoist. Could you please take a look? Thanks.

Oops, looks like version 66 didn't upload properly for chrome yesterday when I was fixing https://github.com/mgsloan/todoist-shortcuts/issues/95 .  After updating extensions is it fixed now?

It turns out that the reason i stopped working for me was due to Toggl Button: Productivity & Time Tracker. If I remove the Toggl extension, i shortcut starts working.

Ah, thanks for diagnosing the problem!  Looks like the toggl extension is reusing the task_view class for its button, causing this issue.  I've included a fix in version 67

Thank you very much.
From: Michael Sloan<EMAIL_ADDRESS>Sent: Saturday, November 9, 2019 8:33 PM
To: mgsloan/todoist-shortcuts<EMAIL_ADDRESS>Cc: h-chatha<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [mgsloan/todoist-shortcuts] Shortcut "i" stopped working with new version of Todoist (#96)
Ah, thanks for diagnosing the problem! Looks like the toggl extension is reusing the task_view class for its button, causing this issue. I've included a fix in version 67
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/mgsloan/todoist-shortcuts/issues/96?email_source=notifications&email_token=AMNBRVK7OHZH3TXN45T7DE3QS5QEXA5CNFSM4JKR24XKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUTEVA#issuecomment-552153684 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNBRVLAE3AM7UMTDA3EGLTQS5QEXANCNFSM4JKR24XA .  https://github.com/notifications/beacon/AMNBRVJDMTE4UWVRXMYZU2LQS5QEXA5CNFSM4JKR24XKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUTEVA.gif


ReactJS Issue

I am getting the below error on console, I am creating an application on reactJS the first time
index.js:1 Warning: Received true for a non-boolean attribute exact.
If you want to write it to the DOM, pass a string instead: exact="true" or exact={value.toString()}.
in a (created by LinkAnchor)
in LinkAnchor (created by Context.Consumer)
in Link (at footer.js:16)
in p (at footer.js:16)
in div (at footer.js:14)
in div (at footer.js:13)
in div (at footer.js:12)
in footer (at footer.js:11)
in Footer (at home.js:21)
in div (at home.js:13)
in Home (created by Context.Consumer)
in Route (at App.js:18)
in Switch (at App.js:17)
in App (at src/index.js:12)
in Router (created by BrowserRouter)
in BrowserRouter (at src/index.js:11)

See https://github.com/styled-components/styled-components/issues/1198#issuecomment-336628848
You can use other values instead of a boolean or manually filter out the props you don't want passed down:
<StyledLink exact={isExact ? 1 : 0} />

or
const StyledLink = styled(({ exact, ...passThrough }) => <Link {...passThrough} />)``

Closing due to inactivity.
Please don't forget to be a good netizen on GitHub and revisit issues, especially ones that have been resolved and need to be closed, or ones that have received replies ❤
Please refer with usage questions to Stack Overflow, as you're more likely to get quicker and helpful answers there ❤ Repository issues on GitHub are great for that, issues, things that need to be resolved in the library itself, or can't be resolved by anyone but the maintainers. While I (and many others on other repos) are happy to eventually answer usage questions, styled-components is a busy repo and mostly stable and slow moving towards new releases, so you're not likely to get fast responses here


Implement IJ1 macro autocompletion

Continuation of scijava/script-editor#21.

The autocompletion of functions from functions.html is working on my system. However, the items from functions_extd.html are not working. Can others comment on whether they are working? @haesleinhuepf were they confirmed to work on your system before I migrated the patches here?

They are still working on my system.
Try by entering w, hit CTRL (CMD? on mac) + SPACE and check if while... shows up in the pullown.

I get if and while autocompletions on Xubuntu 17.10.

Shall we merge?
Btw.: Thanks for the support!  ❤

Thanks for your efforts, @haesleinhuepf! Thanks for testing, everyone!


chore: add .editorconfig

EditorConfig is an editor agnostic way to configure project-local settings and is supported by many editors and IDEs out of the box.
I've created .editorconfig based on the indentations being used in the existing files. Since many people have indentation settings set to 2 spaces for JSON and Lua, having .editorconfig in the repository would help them contribute without having to manually tweak their editor settings every time.

Also, a lot of JSON files didn't seem to follow the 4 space indentation. Should I take this opportunity to run prettier on all of them?

The formatting issue will be fixed by #512. Once merged, CI can handle it correctly.


Look into save_reactive_coord

We have 2 reaction path files in ISOKANN and one more in VGV
Define an interface
keep an eye out for plotting and saving generically

This is partially adressed already.
Note that we have that ugly factor 10 appearing in the save function, due to the nanometer angstrom disambiguity and the non-scale-invariant reaction path method


dc:creator is ignored in certain feeds

Hi,
I am new to newsbeuter.
I am trying to load news from arxiv (http://export.arxiv.org/rss/math.QA) but it looks like the author field does not get filled by the dc:creator fields. Am I missing something?
Thanks

Hi, according to the RSS 1.0 specification (and my personal experience with the 13 feeds I'm subscribing), the field in question should be dc:creator rather than just creator. For me, this still results in authors not being shown.

Cannot reproduce this problem with latest master after all, author field seems to get filled alright. I think I observed this with 2.10-cffd but it seems to have been solved. Closing pull request. Sorry for the distraction!

No worries; thanks for caring in the first place!


Allow clients list to be translated

On Wed, 01 Jun 2022 08:02:15 -0700
Licaon_Kter @.***> wrote:

I don't remember what I meant back then, but surely not the brands, I
dislike that in general.
Must have been something else... but what? :)

Could it be the descriptions of the clients?
The descriptions were removed, as you and I know.


Scope down unnecessary permissions

What does this PR do?

Tightens permissions for better security

Motivation

My workplace pays me to do devops, and I can use that money to buy donuts. On a more specific level, these changes are needed for our security requirements at iSpot.
More

[X] Yes, I have tested the PR using my local account setup  (Provide any test evidence report under Additional Notes)
[X] Yes, I ran pre-commit run -a with this PR

For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes
Same as https://github.com/aws-ia/terraform-aws-mwaa/pull/23, re-opened from different branch so there is no merge conflicts.

Yeah, these changes were tested. Thanks for the merge!


Test script should reinstall pip depends

"$PIP" install --user --no-cache-dir --upgrade .

You mean like that?  If so, maybe the script should also remove dependencies when running uninstall, which I don't think it does by default.  Removing dependencies as part of the process would remove any dependencies which we later decide to stop using.

I think there's a --force-reinstall

I think there's a --force-reinstall

There is; I just tried it out.  PIP still requires --upgrade, otherwise --force-reinstall does nothing.

You were asking me about the "tags" like [Chore], they come from my Git commit template.  I find having a commit template useful for having stuff like GitHub-Issue: or Closes: lines already in every commit message, so I can just un-comment the metadata I want to use.


terraform example code optimized

Description

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

@ocofaigh please review this too, and let me know if any comments. Thanks

@maheshwarishikha Do you know if there was a specific reason why we did not have tests for the basic, hybrid and complete examples? I think there may of been a reason?

@ocofaigh
Hybrid one - we cant test because of hsm connector. We used one HSM connecotr ID provided by HPCS team for our dev activities.
Basic and fscloud tests the same.

/run pipeline

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


ImageFiltered is displayed incorrectly.

Steps to Reproduce
Run the reproduce code and click the yellow button.
expected result
When the button is clicked, the red diamond will only have an opacity change, no translation.
actual result
When the button is clicked, the red diamond is translated
https://user-images.githubusercontent.com/31977171/190979518-282c6188-db1d-4e24-8313-89bac11c6237.mp4
Reproduce code
import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';
import 'dart:ui' as ui;

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(title: Text('title')),
        body: MyStatefulWidget(),
      ),
    );
  }
}

class MyStatefulWidget extends StatefulWidget {
  const MyStatefulWidget({Key? key}) : super(key: key);

  @override
  State<StatefulWidget> createState() {
    return MyState();
  }
}

class MyState extends State<MyStatefulWidget> {
  bool enableOpacity = false;

  @override
  Widget build(BuildContext context) {
    var content = buildContent();
    if (enableOpacity) {
      content = Opacity(
        opacity: 0.5,
        child: content,
      );
    }

    return Column(
      children: [content, buildButton()],
    );
  }

  InkWell buildButton() {
    return InkWell(
      child: Container(
          width: 200,
          height: 40,
          color: Colors.yellow,
          child: Text(
              'click me to ${enableOpacity ? 'disable' : 'enable'} opacity')),
      onTap: () => setState(() {
        enableOpacity = !enableOpacity;
      }),
    );
  }

  Widget buildContent() {
    return ImageFiltered(
        imageFilter: ui.ImageFilter.matrix(Float64List.fromList([
          1, 0, 0, 0, //
          0.5, 1, 0, 0, //
          0, 0, 1, 0, //
          0, 0, 0, 1, //
        ])),
        child: Container(
          color: Colors.red,
          width: 200,
          height: 200,
        ));
  }
}

cc @jonahwilliams  @flar
flutter doctor -v

[!] Flutter (Channel fix_transform_issue, 3.4.0-19.0.pre.339, on macOS 12.2.1 21D62 darwin-arm64 (Rosetta), locale zh-Hans-CN)
    • Flutter version 3.4.0-19.0.pre.339 on channel fix_transform_issue at /Users/bytedance/Develop/upstream/flutter
    ! Upstream repository https://github.com/ColdPaleLight/flutter.git is not a standard remote.
      Set environment variable "FLUTTER_GIT_URL" to https://github.com/ColdPaleLight/flutter.git to dismiss this error.
    • Framework revision f34836a381 (4 minutes ago), 2022-09-19 15:23:33 +0800
    • Engine revision b22a795011
    • Dart version 2.19.0 (build 2.19.0-214.0.dev)
    • DevTools version 2.17.0
    • Pub download mirror https://dart-pub.byted.org
    • Flutter download mirror https://storage.flutter-io.cn
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/bytedance/Library/Android/sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13F100
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.71.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.48.0

[✓] Connected device (3 available)
    • M2011K2C (mobile) • e717b6a0 • android-arm64  • Android 12 (API 31)
    • macOS (desktop)   • macos    • darwin-arm64   • macOS 12.2.1 21D62 darwin-arm64 (Rosetta)
    • Chrome (web)      • chrome   • web-javascript • Google Chrome 105.0.5195.125

[✓] HTTP Host Availability
    • All required HTTP hosts are available

Issue is reproducible using the code sample provided above.
https://user-images.githubusercontent.com/88313112/191007988-d00ba921-f085-472c-9d1b-210714ccac92.mp4

flutter doctor -v
[✓] Flutter (Channel stable, 3.3.2, on macOS 12.6 21G115 darwin-arm, locale en-GB)
    • Flutter version 3.3.2 on channel stable at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e3c29ec00c (5 days ago), 2022-09-14 08:46:55 -0500
    • Engine revision a4ff2c53d8
    • Dart version 2.18.1
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14A309
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] Android Studio (version 2021.2)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8815526/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.71.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.48.0

[✓] Connected device (3 available)
    • iPhone 14 Pro (mobile) • 4302007A-B132-4B89-887A-C5993FCA3284 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-0 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.6 21G115 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 105.0.5195.125

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 3.4.0-19.0.pre.337, on macOS 12.6 21G115 darwin-arm64, locale en-GB)
    • Flutter version 3.4.0-19.0.pre.337 on channel master at /Users/nexus/dev/sdks/flutters
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 085a187e04 (10 hours ago), 2022-09-18 21:11:34 -0400
    • Engine revision b22a795011
    • Dart version 2.19.0 (build 2.19.0-214.0.dev)
    • DevTools version 2.17.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14A309
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] Android Studio (version 2021.2)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8815526/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.71.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.48.0

[✓] Connected device (3 available)
    • iPhone 14 Pro (mobile) • 4302007A-B132-4B89-887A-C5993FCA3284 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-0 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.6 21G115 darwin-arm64
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 105.0.5195.125

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!


[FEATURE] Add tooltip textLabelColor callback

Expected Behavior
I want to be able to apply data color (bar color) to tooltip label:
tooltips: {
  callbacks: {
    textLabelColor: (tooltipItem, chartInstance) => {
      return chartInstance.data.datasets[tooltipItem.datasetIndex].backgroundColor;
    }
  }
}

Current Behavior
There is a callback called labelColor that can be used to set the color of the box that appears right before the label but not the text.

Happy to look at a PR for this.

@etimberg
https://github.com/chartjs/Chart.js/pull/4199
please check this