mailutils: 3.10 -> 1.12

Release notes:

3.11: https://lists.gnu.org/archive/html/info-gnu/2020-12/msg00010.html
3.11.1: https://lists.gnu.org/archive/html/info-gnu/2021-01/msg00003.html
3.12: https://lists.gnu.org/archive/html/info-gnu/2021-02/msg00007.html

Motivation for this change
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

[ ] 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"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 113115 at c14c378b run on x86_64-linux 1

6 packages built:

automysqlbackup
gsmartcontrol
logrotate
mailutils
smartmontools
tlp

2 suggestions:

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/tools/networking/mailutils/default.nix:32:5:
   |
32 |     ./fix-build-mb-len-max.patch
   |     ^

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/tools/networking/mailutils/default.nix:33:5:
   |
33 |     ./path-to-cat.patch
   |     ^

Result of nixpkgs-review pr 113115 at c14c378b run on aarch64-linux 1

6 packages built:

automysqlbackup
gsmartcontrol
logrotate
mailutils
smartmontools
tlp

2 suggestions:

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/tools/networking/mailutils/default.nix:32:5:
   |
32 |     ./fix-build-mb-len-max.patch
   |     ^

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/tools/networking/mailutils/default.nix:33:5:
   |
33 |     ./path-to-cat.patch
   |     ^


Dictionary or JSON as Input for a trained model with source factors

With respect to closed issue https://github.com/awslabs/sockeye/issues/999
can I give a dictionary or JSON file as additional input to the test files, instead of annotating the test files?
I mean, in my case it should be easy to change the target vocabulary.
A dictionary like

{ source|1 : Quelle|2,
test|1 : Test|2
...              }

There's the --json-input flag for sockeye-translate.
Details for where this is parsed are here:
https://github.com/awslabs/sockeye/blob/main/sockeye/inference_pt.py#L241
If I recall correctly the format is:
{'text': 'Test test',
 'factors': ['f1 f1' , 'f2 f2'], }

Is that what you are looking for?

Yes, I was looking for something like this. Thank you!

There's the --json-input flag for sockeye-translate.
Details for where this is parsed are here: https://github.com/awslabs/sockeye/blob/main/sockeye/inference_pt.py#L241
If I recall correctly the format is:
{'text': 'Test test',
 'factors': ['f1 f1' , 'f2 f2'], }

Is that what you are looking for?

Can I also just give as text input the source and target terms with its corresponding source factors or must it be complete text lines?

You must provide the full input as factors are required for all input tokens.


Believe in yourself missing on respec after azata merge

I'm not sure if the azata spell believe in yourself is coded differently than the others, but I recently did a respec and merged my spellbook magus and azata spellbooks during the mythic level up process and I had azata spells up to level 6 (currently caster level 19 after the merge) but was missing this level 3 spell.
I had done a respec with a previous version and the spell showed up as expected.  Let me know if there is anything I can get you to help.  I did try reloading (full game exit), changing areas, etc.

I had done a respec with a previous version and the spell showed up as expected.

Can you clarify this? You mean previous version of my mod? If so, which version?

Also note that one of the discord comment says:

yeah believe in yourself is not technically on the Azata spell list, it is added to the book externally at the end of Act 3 depending on how you played the act

So this is likely very tricky to solve, because the Azata spellbook technically no longer exists once you merge.
I will have to look into if Angel has any similar spells,and if so how is the in game code handling it. If you know any Angel/Lich spells that are similar, let me know so that I can debug.

I think I know what it is based on your notes. When I installed the mod the first time (version 1.1.1), I was already ranked up higher in Azata.  I used the respec mod to reset my mythic levels, then I installed this mod and proceeded to rank up including the merged spellbook which at that time included believe in yourself.  I did the same thing this time, but it was on a save where I was only rank 3 of azata before resetting my mythic rank.  I found the feature in toybox that grants believe in yourself, and added it manually and it was added to my merged spellbook.  It's possible that if I had would have received that believe in yourself feature naturally by the game at a higher mythic rank.  If I get a chance to test that, I'll let you know!

So I can confirm that this spell is added by a feature "AzataForSpellsGoodFeature" which is a story trigger (my first save must have already triggered).  I still haven't come across the exact point in act 3 that triggers it, but what I can confirm is that it appropriately adds the spell to the merged spellbook as expected.  I think we can close this issue as I think the worst case here is that people are confused by the timing of when they receive the spell.  Unless that's something you want to alter, I think everything is working fine.
Thanks for your response and the time spent creating this mod!  Will let you know if I run into anything else!


private/protected member and "this" type for functions

TypeScript Version:  2.32
Code
class MyClass {
  private x: number;
  protected y: number;
  public z: number;
}

function temp(this: MyClass): void {
  this.x // type error
  this.y // type error
  this.z // ok
}

There is no expected / unexpected behaviour here...
When using a context (this) type for a function where the this type is a class one can not access private/protected members.
At first this is reasonable, but when thinking about composition/mixins etc, it is common to access properties on the class, changing internal values of the root mixinto class (which holds state) .
A plugin might want to change internal values that are not exposed to the end user but only to mixins...
I understand the security concerns, so "private" members shouldn't get access, but for protected properties, there is some logic in exposing it.
Working around the issue, while preserving types, is not difficult but does have some runtime cost...
class MyClassB extends MyClass {
  static temp(this: MyClassB): void {
    this.x // type error
    this.y // ok
    this.z // ok
  }
}

now MyClassB.temp can be assigned to a mixin or composed, etc...
Seems like exposing the protected properties also aligns with the workaround, it's doable anyway.
Can you expose protected properties in this function type.

it boils down to what private and protected mean. From the spec

Private property members can be accessed only within their declaring class. Specifically, a private member M declared in a class C can be accessed only within the class body of C.
Protected property members can be accessed only within their declaring class and classes derived from their declaring class, and a protected instance property member must be accessed through an instance of the enclosing class or a subclass thereof. Specifically, a protected member M declared in a class C can be accessed only within the class body of C or the class body of a class derived from C. Furthermore, when a protected instance member M is accessed in a property access E.M within the body of a class D, the type of E is required to be D or a type that directly or indirectly has D as a base type, regardless of type arguments.

not honoring these rules, renders these modifiers meaningless.

I understand the security concerns, so "private" members shouldn't get access, but for protected properties, there is some logic in exposing it.

this is not really about privacy or security. it is just that the whole oop encapsulations do not make much sense with the mixin patterns. i would recommend just using public for these case.

Ok, thanks!


Nested layouts default to zero on size

As can be seen in the attached video, when rerendering a new nested layout, the sub-layout would have a zero of size at first. Upon clicking on the resize handle, the sub-layout would snap back to the expected size. Any ideas why this is happening?
Thanks!
https://github.com/bvaughn/react-resizable-panels/assets/35216312/7c5430b8-c113-430a-a510-27aa3810470a

Need a repro. Can't debug a video.

A code sandbox would work, but a Replay would be even better.

Let me quickly get a repo to replicate this issue. Thanks

I created a new repo and tried replicating this issue. Turns out it's because the video above was using v0.0.55. As soon as I upgraded to v0.0.59, the issue was gone.
Thanks for this amazing library!

Glad to hear! :)


Sliding windows

It would be useful to have a sliding function which one can fold over.
For example, apart:
feature age ~> sliding 2 ~>
would give access to a stream of two values at a time, so the difference in the date could be looked at for example, or how large the differences are.

Had a hack today on trying this with just case expressions, but without higher order functions I don't think it's possible.

I think we can still write meaningful expressions without this, so let's leave this for now

there is a branch that started work on window frames, but was not finished.
this will be worth coming back to eventually, but for now Source is expressive enough to focus on Ivory integration.
https://github.com/ambiata/icicle/tree/topic/window-frames


Windows 7 Does Not Boot

Describe the issue
I was going to use parallels for win7 until I found out that it only does ARM64 emulation. So then I used UTM, hoping it would work, but once I pressed start, this came up: qemu-x86_64-softmmu: -netdev vmnet-macos, mode=shared, id=net0: Device 'vmnet-macos' could not be initialized.
Configuration

UTM Version: 2.4.0 (39)
OS Version: macOS Monterey 12.1 Beta (21C5021h)
Intel or Apple Silicon? Apple Silicon with 16 GB

Crash log
No App Crashes.
Debug log
debug.log
Upload VM
I forgot my Google password :(
Edit: Got an image of the error window, thanks Parelells Toolbox

How did you install UTM on your Mac? I got this error recently when I had my code signing settings wrong in Xcode.

Through the website,

As an immediate workaround you can click the options button for the VM, then click Network tab and change the mode to emulated VLAN.

It worked! Thanks!


help

After I downloaded it, the icon is centered, but I don’t know how to find the running program to set it up. Can anyone help me how to do it?

@Lantern-ty You will need to use the "TaskbarX Configurator.exe"


Add more seed corpus

I generated those using a custom "generate out of thin air" script and wondered if such a configuration mode would be welcome to add to Bitcoin's test/fuzz/test_runner.py ?

Yes, improvements to test_runner are welcome :)


Allow NOMUTEX|READONLY Open Mode

This PR is more for discussion - definitely don't merge in this current state.
I couldn't find a way to open a database in read only mode (so that no locks are ever acquired on the DB, essential for highly concurrent readers), so I've hacked this in.
Any thoughts on how this interface should look for the end users, @mattn?

You can specify DSN like below.
db, err := sql.Open("sqlite3", file:test.db?immutable=1")

Coverage increased (+0.05%) to 67.819% when pulling 6455f6117979f954d4b44849892d5240013f8309 on fortytw2-graveyard:master into 05548ff55570cdb9ac72ff4a25a3b5e77a6fb7e5 on mattn:master.

Ah, excellent. I was rather confused as to where this was set (L40-45 of sqlite3.go) and to how it interacted with the flags to sqlite_open_v2. Thanks!


error Handling in getCompanyId added

this is important for my case as I have to make sure there are no errors caused by aanalytics2 (even though they come from Adobe not returning the company ID)

added updateCalculatedMetric function and doc on getVirtualReportSuiteComponents

Thanks a lot @loldenburg for this PR.
It is now live with the global library in github, I will push it live with a new version on pypi with the retry parameter.


⚠ AGEG website (prod) has degraded performance

In c08df35, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 5890 ms

Resolved: AGEG website (prod) performance has improved in 36bc1f2.


Interfaces Separation From Implementations

Separate Interfaces from Implementations
Separate Contract Exception from other Implementations
Only Lists and Collection should have Plural "Naming".
"Contracts" Renamed to "Contract"
Improve Code Documentation

Hi @javierpardollama !!
First of all thank you very much for such a good and surprising contribution to our first open source library, i see the different points tried to be improved here, well done, and very nice proposal!! :) I really love this commitment from the first steps
I agree with almost everything except a couple of points, that will be part of our next internal tech event, and/or also part of the contribution guidelines of the current open source project (still not created, sorry :(

We tend to split interfaces in different files, something that is quite good in a lot of different scenarios, however, one point that we will add for the future in new implementations, is to add the interface at the top of the class implementing, when the unique purpose of it is to be injected by the DI container.
We will try to ease the usage of this tools as much as possible, including namespaces. Since the tool purpose is uniquely Domain project, we want to concentrate all implementations in a single namespace, so given that you can use everything related to Domain.Core without specific namespaces distinctions.

Again, thank you very much for taking charge of these proposed improvements, I would like to request to modify the points mentioned above, and upgrade the version of the new alpha packages, which will be published as a release on the 29th.
Best regards


Python version 3.12 not supported for creating azure functions in vs code

Is there any ETA on version 3.12 support?
Have to stay on newer version of python due to internal security policies, will the 3.12 version be supported

Sorry there's nothing we can do at this time, we will add support once the service supports it. Refer to this document for the supported Python versions: https://learn.microsoft.com/en-us/azure/azure-functions/functions-versions?tabs=isolated-process%2Cv4&pivots=programming-language-python#languages

I think that this would be the appropriate repo to file an issue on the Functions service: https://github.com/Azure/azure-functions-host

Also from this blog post it looks like they are already working on supporting Python 3.12.
https://techcommunity.microsoft.com/t5/azure-compute-blog/azure-functions-using-python-2023-in-review/ba-p/4007999


Advice on writing unit tests for irregular whitespace characters

Problem
I wanted to create a sniff that will check the irregular whitespace characters like the one in the ESLint rule.
I wanted to create unit test for it before trying to work on the actual sniff, but I'm not exactly sure how to test it, because even if I drag the characters from the character viewer (on mac), I get things like

<0x0b>

Written in the file, and the only irregular whitespace I can produce on the mac is by typing option+space (without actually it being typed out like above).
So I'd like some advice about how to approach it, and if we want a sniff that checks for irregular white space.
Motivation
The motivation behind the sniff was that I got errors in my php code, and I couldn't see where the errors were coming from, unless I highlighted the code in the editor (Sublime), where I could see that there is not a dot in my space.
So this sniff would detect these irregular white spaces in the code and warn the user not to use them.
To do

Discuss if we want/need such a sniff
Create IrregularWhitespaceSniff that will check these irregular white spaces
Test

Without voicing an opinion about whether or not this should be checked by WPCS, I have a couple of general remarks related to this idea:

For irregular whitespace at the start and end of a file which could lead to the dreaded "headers already send" issue: this will be covered by the Squiz.Whitespace.SuperfluousWhiteSpace sniff once PHPCS 3.3.1 comes out.
See: https://github.com/squizlabs/PHP_CodeSniffer/pull/2040
I'm not sure a sniff will be the best/easiest way to find these. You would need to examine quite a few token types (all text tokens and some of the less common tokens associated with tokenizer issues).
Running a regex for this over each file in a plugin/theme might be more performant, especially as finding these type of whitespaces is reasonably rare anyway.
If a sniff would be decided to be the way to go - I managed to create some test data for the PR mentioned above, by copying and pasting from MS Word to Notepad while making sure the test file encoding was set to UTF-8.
If worse comes to worse, you could even create some test files by letting PHP write the characters you want to test for to a file.

Thanks for the input. I'll start to work on this just in case it gets picked up :)

This hasn't picked up any traction, so we can close it. I'll open a placeholder issue in the PHPCSExtra for this.


AppVeyor & Snyk integration

It is need an email address where to send notifications

    It is need an email address where to send notifications


Icons not available in KDE

Refer the following screenshot:

Note the missing/fallback icons at the buttons on the right side, it should be using the breeze icon or the default icon shipped in the snap.

Should be fixed in https://github.com/Lin-Buo-Ren/audio-recorder-snap/commit/a3945b0d8a1c9468e3794fa5e682b40506c08202 via shipping hardcoded gtk3 settings


GitHub Actions fail for PHP

Need to be fixed.
@wechaty/php @wechaty/grpc

Fixed by

#155

Thanks to @zhangchunsheng !


Add "Commands" tooltip on hover to the list commands Button in Console View

Fixes #138

@kartik-gupta-ij Ran npm audit fix to fix the failing workflow, can you run it again?


MGDOBR-1340: ProcessorService: Restore metrics capture (#1445) - Update kustomization images

This Pull request aims to update the kustomization images for the PR MGDOBR-1340: ProcessorService: Restore metrics capture (#1445)

/deploy dev


[AutoPR reservations/resource-manager] Reservations RP: updating enums to use x-ms-enum

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3234

Codecov Report

Merging #2745 into restapi_auto_reservations/resource-manager will increase coverage by 0.01%.
The diff coverage is 100%.

@@                              Coverage Diff                               @@
##           restapi_auto_reservations/resource-manager    #2745      +/-   ##
==============================================================================
+ Coverage                                       55.84%   55.86%   +0.01%
==============================================================================
  Files                                            7293     7293
  Lines                                          153448   153517      +69
==============================================================================
+ Hits                                            85692    85761      +69
  Misses                                          67756    67756

Impacted Files
Coverage Δ

...e/mgmt/reservations/models/extended_status_info.py
100% <ø> (ø)
:arrow_up:

...gmt/reservations/models/extended_error_info_py3.py
57.14% <ø> (ø)
:arrow_up:

...rvations/azure/mgmt/reservations/models/catalog.py
100% <ø> (ø)
:arrow_up:

...mgmt/reservations/models/reservation_properties.py
100% <ø> (ø)
:arrow_up:

...ions/azure/mgmt/reservations/models/catalog_py3.py
100% <ø> (ø)
:arrow_up:

...mt/reservations/models/extended_status_info_py3.py
100% <ø> (ø)
:arrow_up:

.../reservations/models/reservation_order_response.py
100% <ø> (ø)
:arrow_up:

...vations/azure/mgmt/reservations/models/__init__.py
100% <ø> (ø)
:arrow_up:

...re/mgmt/reservations/models/extended_error_info.py
57.14% <ø> (ø)
:arrow_up:

...e/mgmt/reservations/models/reservation_response.py
100% <100%> (ø)
:arrow_up:

... and 4 more

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 600c8e7...f53aa4a. Read the comment docs.


Use CSS style for colors

This PR is trying to use CSS styles for recurring style attributes (primarily for colors) to 1) reduce SVG size and make it more readable and 2) make colors easily adjustable in the SVG code for all characters that use it. This is WIP, I thought I could get away with using a single set of color styles for both background and foreground, but turns out that setting the stroke color affect the text rendering.

Nice, good idea. When messing around with these thing you really appreciate having good tests :) so please add more if you like

Some possible things to look into:

<g> can be used to inherit attributes and transformations
<use> be be used to duplicate a  <g>

But as you also mentioned i think we should probably stay away from too crazy tricks to keep the svg easily editable. For
example i wonder if the background could be a small embedded png that is stretched? :)

Output looks good, can merge if you want 👍 had some minor code comments but no biggies.
For new features, regressions or some tricky edge cases i usually try to add some kind of test

:rocket:


request help: How to use go-plugin-runner with APISIX Ingress

Issue description
according to this doc: https://apisix.apache.org/zh/docs/ingress-controller/tutorials/how-to-use-go-plugin-runner-in-apisix-ingress/
this step:

apisix pod cant running，error is :
helm epository does not exist or may require authorization: server message: insufficient_scope: xxxxx
change the repository's value not useful
Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long):
your Kubernetes cluster version (output of kubectl version):
Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.3", GitCommit:"816c97ab8cff8a1c72eccca1026f7820e93e0d25", GitTreeState:"clean", BuildDate:"2022-01-25T21:25:17Z", GoVersion:"go1.17.6", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.1", GitCommit:"5e58841cce77d4bc13713ad2b91fa0d961e69192", GitTreeState:"clean", BuildDate:"2021-05-21T23:01:33Z", GoVersion:"go1.16.4", Compiler:"gc", Platform:"linux/amd64"}
WARNING: version difference between client (1.23) and server (1.21) exceeds the supported minor version skew of +/-1
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a):

fix it, it's my fault


Fix incorrect results with cloneWithType for finitetype integer sub types arrays

Purpose

This PR will fix the issues with cloneWithType for finite types of numeric value arrays and the issues with numeric conversions of integer sub types.

Fixes #30301
Fixes #31220
Approach
Samples
Remarks
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

There are some other known issues related to cloneWithType with tuples. Created an issue for it.


CASSANDRA-17656: jvm-dtest upgrade tests run all supported pairs of upgrades between from/to but does not actually test all patches from/to

spoke with @jonmeredith in slack, I will revert back to upgradeFrom(v30).  He pointed out that we really want to test all upgrade paths, but don't have a clear solution on how to do that given CI and JVM Metaspace limitations (testing out POC but likely to punt if not working by EOD)

I gave up on trying to make the tests better... we really need a template system or code generation system where we create 1 class for each path...  given how we currently do this we need *Test.java files for each, so something would need to know how and generate it!  We could do this via a script, but then we need this to work on all environments... we could do this with an annotation processor, but need to make things more complex at build time since annotation processors need to be compiled before the tests compile

looking into org.apache.cassandra.distributed.upgrade.CompactStorageUpgradeTest#compactStoragePagingTest, not sure why it OOMs when it matches the same pattern as other tests...


create User model, migration files and tests

Creating a user model, a static method for creating users.
Made migration files for user.
Wrote tests for user creation.
Deleted joboard application from the first demo since
it is not relevant anymore.

plz separate the removal of the board content from the adding of the user model.
(so you should have 2 commits, one that remove stuff and another one that add new stuff)

also, try to be as extensive as possible with tests, cover all the options. try to think about edge cases

@yuvalk  We split up the PR.

@ormergi
We tried to keep the first commit separate and continue with another branch not dependent on it, but the jobborad app caused some errors.
What should we do in that situation?

@ormergi We tried to keep the first commit separate and continue with another branch not dependent on it, but the jobborad app caused some errors. What should we do in that situation? And can we fix the current situation?

There are few changes on the second commit that related to the first one, maybe its related.
Move the related changes to the first commit and try again.

@ormergi We understand what is the problem but we need help with the git technicalities.

Hello @ormergi these comit have been up on GItHub since sunday and we made correction according to what Edward and Yuval told us, these are too many changes that should have came up at an earlier time. What do you suggest we do taking into account that tomorrow we need to show Demo#2?

@ormergi We opened new PR #24.

Please rebase over the updated master, this PR has conflicts now.

This PR was split up as asked, the reviewing should start at PR create user model

@AshkenYariv its good that you explain why you close the PR, it will be awesome if you also paste the like for the new one.
Also, my code review comments may help you in follwing PR's as well.

Hi @ormergi,
We forgot to paste it.
The following PR is #25.


Prof 1

This time, I add new profiles for all individual langauges. From here, we will start with @antipodite.


chore(controller): bump confd to v0.8

This new warning showed up in the functional test:
deis-controller running...
2015-03-23T19:40:18Z 40cafdfbebf1 confd[84]: WARNING Skipping confd config file.
[2015-03-23 19:40:18 +0000] [86] [INFO] Starting gunicorn 19.3.0

yeah that's due to confd expecting a config file yet not respecting it in recent versions, hence the reliance on flags instead. I'm fixing that up.

@mboersma kelseyhightower/confd#235

FYI because the check script is gone with this PR, we should no longer be seeing "config check failed: exit status 1" any more. We should see more verbose information... Hopefully.

I also filed kelseyhightower/confd#236 to track the --config-file issue/workaround.

This does make the templates a bit uglier, but I gather there's no alternative if we want to catch up with newer confd, which we do. Code LGTM.

Also note kelseyhightower/memkv#4 should help make the "get value or default" calls here a lot cleaner.

Code LGTM.


186420492 Axis Labels

PT Story: https://www.pivotaltracker.com/story/show/186420492
This PR adds editable axis labels to the xy plot.

The axis labels use the new style for all graphs, including those that use the old codap style legend (moth).
Codap style graphs should still display selected attributes on the axis labels, and clicking on them should bring up the dropdown that allows the attribute to be changed, etc.
CLUE style graphs display "axis label" by default in the axis label. Clicking the label will pull up a text box that allows the user to type in another string to display.
The user entered label is persistent.

The PR also addresses some styling issues missed in the styling PR, including adjusting the font size and position of the axis min/max text fields.
It also uses useReadOnlyContext in EditableTileTitle, which touches and simplifies quite a few files.

@tejal-shah When looking at this PR, can you please be careful to check the old style labels (unit=moth)? I tried to make sure I didn't change their functionality at all, but I'm not sure I tested all cases, and it's possible that something unintended changed with them.


Add deprecation notes to tutorials

Remove MilvusDocumentStore as it cannot be run on Colab

I'll close this PR and open a new one


Missing brightness/gamma/contrast setting (Twisted Insurrection)

Twisted Insurrection <IP_ADDRESS>
Is there a way to set gamma/brightness/contrast with cnc-ddraw dll? I tried a few .ini setting but nope - don't work as expected. Maybe there is already a .ini setting, but not yet documented or I was blind?
My workaround: Using DGVoodoo, the brightness slider works there as expected.
Having a "curve Editor" like ability like in Affinity Photo or Photoshop, i.e. a lookuptable as included or separate .ini, would work for me too.

There's now a new shader called "image-adjustment.glsl" included - it will get the job done. Also, latest version can now combine 2 shaders, so you could upscale with one and use image-adjustments as the second shader (rename to same as upscaler and append .pass1)
Not very user friendly yet, but get's the job done. Will try to get a better solution working later
Values that can be changed in "shader-package.zip/shaders/misc/image-adjustment.glsl"

#define ia_overscan_percent_x 0.0  // crop width of image by X%; default is 0.0
#define ia_overscan_percent_y 0.0  // crop height of image by X%; default is 0.0
#define ia_ZOOM 1.0                // zoom factor; default is 1.0
#define ia_XPOS 0.0                // horizontal position modifier; default is 0.0
#define ia_YPOS 0.0                // vertical position modifier; default is 0.0
#define ia_FLIP_HORZ 0.0           // horizontal flip toggle; default is 0.0
#define ia_FLIP_VERT 0.0           // vertical flip toggle; default is 0.0
#define ia_target_gamma 2.2    // the gamma you want the image to have; CRT TVs typically have a gamma of 2.4
#define ia_monitor_gamma 2.2   // gamma setting of your current display; LCD monitors typically have a gamma of 2.2
#define ia_saturation 1.0      // color saturation; default 1.0
#define ia_contrast 1.0        // image contrast; default 1.0
#define ia_luminance 1.0       // image luminance; default 1.0
#define ia_black_level 0.0     // black level; default 0.0
#define ia_bright_boost 0.0    // adds to the total brightness. Negative values decrease it; Use values between 1.0 (totally white) and -1.0 (totally black); default is 0.0
#define ia_R 1.0               // red level; default 1.0
#define ia_G 1.0               // green level; default 1.0
#define ia_B 1.0               // red level; default 1.0
#define ia_TOPMASK 0.0         // mask top of image by X%; default is 0.0
#define ia_BOTMASK 0.0         // mask bottom of image by X%; default is 0.0
#define ia_LMASK 0.0           // mask left of image by X%; default is 0.0
#define ia_RMASK 0.0           // mask right of image by X%; default is 0.0
#define ia_GRAIN_STR 0.0       // grain filter strength; default is 0.0
#define ia_SHARPEN 0.0         // sharpen filter strength; default is 0.0

Latest build:
cnc-ddraw.zip

That is a solution that works for me. Thanks!

Today, since it is a filler day "between holiday-day and weekend" in Germany I had time for more testing.
Everything seems to work perfectly. Luminance gives IMHO the best result, only one thing to change.
If you make it into the menu for the next TS-update: Luminance with brightness and contrast makes sense, only one slider instead of two to "play until it fits". You might add gamma and saturation "for completeness", else the next one will complain :D.
Nod 16 Eagles nest default:

Nod 16 Eagles nest brightness 0.20 contrast 1.4 (or luminance 1.4) is fine for that mission:

The darkest map of all, Nod 10 Chameleon Betrayal default:

Nod 10 Chameleon Betrayal brightness 0.20 contrast 1.4 (or luminance 1.4) makes it better, but not enough for that map:

Nod 10 Chameleon Betrayal brightness 0.33 contrast 1.66 (or luminance 1.66):

For completeness: the "gamma correction" version, which by design washes out the picture, why I prefer luminance:
Nod 10 Chameleon Betrayal monitor_gamma 2.8:

Nod 10 Chameleon Betrayal monitor_gamma 3.2:

Nod 10 Chameleon Betrayal monitor_gamma 3.2 saturation 1.2 to counter the washed out effect:

Looking very good! Yeah, only a few chosen settings will make it into the in-game UI once it's done, having all of these settings in there would be way too confusing


Parent-Child Supported Role Base Access Control Implementation with Entrust

Do we have Parent-Child based Roles management possible with Entrust.
For Example:
I need to assign some permission to a Parent Role then in its child I could have some other permissions. On the bases of User Role I need to restrict his/her access. I don't want duplicate permission assignment to different Roles then assign different roles to User. I want to create a Role that could be a child of some other Role which would finally provide all permission to the user  that Parent has as well as the permission I have assign to that particular Role.
Like we have a Public Role which has access to view a Page for example and we need another role like a Registered Role that need to be a Child of Public Role. All user with Registered Role has to have permission that Public Role has as well as that Registered Group has.
The Functionality I am asking is similar to what PhpRbac provides.
Is it possible to do such parent-child supported role base access control using Entrust?

@psbhanu have you got any solution?
Please let me know because I am also stuck on the same scenario. I want to create parent and child permission. if the parent is assigned the child will automatically be assigned.


Having trouble getting it to work with grails 3

Hi there,
I am trying to migrate an old grails app to grails 3.  I am using grails 3.0.10.  I was using the rendering plugin in my old app to generate PDFs and have a bunch of PDFs built this way which I would like to keep intact, so I'm trying to get this rendering plugin installed in my grails 3 app.  As suggested, I have added the following line to my build.gradle under the dependencies:
compile "org.grails.plugins:rendering:2.0.0-SNAPSHOT"
This seems to correctly pull this plugin .jar file and the run-app works.  However, when I try to render a gsp as a PDF through my controller I'm getting a NullPointerException that is being thrown by some code in the rendering plugin.  I am including the stack trace below.  Can you please let me know how I could correct this issue?
ERROR org.grails.web.errors.GrailsExceptionResolver - NullPointerException occurred when processing request: [GET] /test/testPDF
Stacktrace follows:
java.lang.reflect.InvocationTargetException: null
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145) ~[na:1.7.0_79]
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615) ~[na:1.7.0_79]
at java.lang.Thread.run(Thread.java:745) [na:1.7.0_79]
Caused by: java.lang.NullPointerException: null
at java.beans.Introspector.getPublicDeclaredMethods(Introspector.java:1281) ~[na:1.7.0_79]
at java.beans.Introspector.getTargetMethodInfo(Introspector.java:1141) ~[na:1.7.0_79]
at java.beans.Introspector.getBeanInfo(Introspector.java:416) ~[na:1.7.0_79]
at java.beans.Introspector.getBeanInfo(Introspector.java:163) ~[na:1.7.0_79]
at grails.plugins.rendering.document.RenderEnvironment.init(RenderEnvironment.groovy:31) ~[rendering-2.0.0-SNAPSHOT.jar:na]
at grails.plugins.rendering.document.RenderEnvironment.with(RenderEnvironment.groovy:68) ~[rendering-2.0.0-SNAPSHOT.jar:na]
at grails.plugins.rendering.document.RenderEnvironment.with(RenderEnvironment.groovy:60) ~[rendering-2.0.0-SNAPSHOT.jar:na]
at grails.plugins.rendering.document.XhtmlDocumentService.generateXhtml(XhtmlDocumentService.groovy:65) ~[rendering-2.0.0-SNAPSHOT.jar:na]
at grails.plugins.rendering.document.XhtmlDocumentService.createDocument(XhtmlDocumentService.groovy:35) ~[rendering-2.0.0-SNAPSHOT.jar:na]
at grails.plugins.rendering.RenderingService.render(RenderingService.groovy:36) ~[rendering-2.0.0-SNAPSHOT.jar:na]
at grails.plugins.rendering.RenderingService.render(RenderingService.groovy:35) ~[rendering-2.0.0-SNAPSHOT.jar:na]
at grails.plugins.rendering.RenderingService.render(RenderingService.groovy:65) ~[rendering-2.0.0-SNAPSHOT.jar:na]
at grails.plugins.rendering.RenderingTrait$Trait$Helper.renderPdf(RenderingTrait.groovy:47) ~[rendering-2.0.0-SNAPSHOT.jar:na]
at com.svp.controller.TestController$_closure1.doCall(TestController.groovy:14) ~[main/:na]
... 3 common frames omitted

You also need this:
runtime "org.springframework:spring-test:4.2.1.RELEASE"
I think this should be included in the build.gradle for this project, but it isn't. So, this is a bug.

Do you have any update on it? Will Grails 3 be supported.

Also just ran into this - not sure why the rendering plugin requires a runtime dependency on a test library.....seems like this is an issue.

I agree. Maybe I will look into why it's necessary later and see if I can
remove that dependency and make a PR.
On Feb 23, 2017 5:22 PM, "Alex White"<EMAIL_ADDRESS>wrote:

Also just ran into this - not sure why the rendering plugin requires a
runtime dependency on a test library.....seems like this is an issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gpc/rendering/issues/27#issuecomment-282141154, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AG7dTqI-r2EngjmW_Tt9TQBzbDgyG4Vtks5rfgbBgaJpZM4GzShN
.


Angular ~1.6 compatibility

Does angular-gauge work with angular ~1.6? Will it soon?

Hi @steven-gruverman ,
I did not test angular-gauge with 1.6 version while developing. I think, v1.6 was developing that time. But you can try it with angular 1.6 and let me know if there is any issue. If you can do this exercise, this would help me to upgrade it promptly.
Else, i will be free around mid December to start looking into it.
Regards
Ashish

Hi Ashish,
Thanks for your reply. I did have a chance to try it with our application – for the features and angularjs-gauge we use there was one small problem that may have had more to do with our css.
Because we are close to a release I don’t have time now to test more or to get it to work – for now we are just going with the Bower dependencies as-is and dealing with the Javascript errors being thrown because other parts of our app have 1.6 as a dependency.
I will let you know if I have a chance to work on this some more and either find issues with 1.6 or not.
Thanks again,
Steve
Steve Gruverman | Application Developer II
IDEXX Laboratories, Inc. | Steven-Gruverman<EMAIL_ADDRESS>Phone: + 1<PHONE_NUMBER>
Mobile: + 1<PHONE_NUMBER>
From: Ashish Chopra<EMAIL_ADDRESS>Sent: Saturday, November 11, 2017 10:49 PM
To: ashish-chopra/angular-gauge<EMAIL_ADDRESS>Cc: Gruverman, Steven<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [ashish-chopra/angular-gauge] Angular ~1.6 compatibility (#42)
Hi @steven-gruvermanhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_steven-2Dgruverman&d=DwMFaQ&c=2do6VJGs3LvEOe4OFFM1bA&r=RmIe1gmnt6yK0mEcsqffLM-n2yERkhj5v2w4_dNuka0&m=0G8mNHrWYBJe1AWEpbcjjiaUzCpFeMqpMQeMqOPjFWI&s=GP-BRtdmRT3SdxbJoZwKJckThuPwUy8wOrtgUZx6rYE&e= ,
I did not test angular-gauge with 1.6 version while developing. I think, v1.6 was developing that time. But you can try it with angular 1.6 and let me know if there is any issue. If you can do this exercise, this would help me to upgrade it promptly.
Else, i will be free around mid December to start looking into it.
Regards
Ashish
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_ashish-2Dchopra_angular-2Dgauge_issues_42-23issuecomment-2D343711461&d=DwMFaQ&c=2do6VJGs3LvEOe4OFFM1bA&r=RmIe1gmnt6yK0mEcsqffLM-n2yERkhj5v2w4_dNuka0&m=0G8mNHrWYBJe1AWEpbcjjiaUzCpFeMqpMQeMqOPjFWI&s=QHozjwn263ZQZvPigGttNyRaOkcQ4z0g1dQ7v8xd5iM&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AM-5FwLjjI-5F5TW65CfLWJh04U8TDvnJ8Quks5s1mqTgaJpZM4QXGN7&d=DwMFaQ&c=2do6VJGs3LvEOe4OFFM1bA&r=RmIe1gmnt6yK0mEcsqffLM-n2yERkhj5v2w4_dNuka0&m=0G8mNHrWYBJe1AWEpbcjjiaUzCpFeMqpMQeMqOPjFWI&s=nR-1dEm8AdJpOyKuBkuM9Q8tr9g-cnFHJRssGQHCLhQ&e=.

I've done limited testing on AngularJS 1.6.5 without issues...
-S

I did some testing with angular 1.6 while testing the pull requests. and no issues found.

Great – thanks for doing that and for letting us know!
Steve Gruverman | Application Developer II
IDEXX Laboratories, Inc. | Steven-Gruverman<EMAIL_ADDRESS>Phone: + 1<PHONE_NUMBER>
Mobile: + 1<PHONE_NUMBER>
From: Ashish Chopra<EMAIL_ADDRESS>Sent: Wednesday, November 15, 2017 11:49 PM
To: ashish-chopra/angular-gauge<EMAIL_ADDRESS>Cc: Gruverman, Steven<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [ashish-chopra/angular-gauge] Angular ~1.6 compatibility (#42)
I did some testing with angular 1.6 while testing the pull requests. and no issues found.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_ashish-2Dchopra_angular-2Dgauge_issues_42-23issuecomment-2D344814945&d=DwMCaQ&c=2do6VJGs3LvEOe4OFFM1bA&r=RmIe1gmnt6yK0mEcsqffLM-n2yERkhj5v2w4_dNuka0&m=8DKPsNVVUJl_NMcmA38Iy2Fc3_ns_uNjxbMStM-e_q8&s=-lKiSjtXKy8qzsBeWQ72URQWAg6FGGvj76uVa9MLskI&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AM-5FwLvglMrCQI0dYBY588nBMlBjnRMzCks5s276zgaJpZM4QXGN7&d=DwMCaQ&c=2do6VJGs3LvEOe4OFFM1bA&r=RmIe1gmnt6yK0mEcsqffLM-n2yERkhj5v2w4_dNuka0&m=8DKPsNVVUJl_NMcmA38Iy2Fc3_ns_uNjxbMStM-e_q8&s=kRMZKGtpIiJhbDi1f1HLxwG_g9qnpQah5XaSFqB22e0&e=.

verified and released v2.2.0 of gauge library.


Java 6/7 Compatibility

Hello!
I was wondering if it would be possible to add compatibility for java 6 or 7, because I wanted to be able to run the launcher on older OSes, thank you :)

I was going to test this as well, but guess I don't need to. It would be nice to have it work with Java 6 like the old launcher did.

When I made this, I wasn't aware people still cared about Java 6. As such, it's not officially supported. Maybe try building it yourself with a java 6 JDK? I've never tried it.

Alright thanks for answering

Can't just give up that easily lol. Will try building it myself

Okay, so I tried building it in Java 6 but couldn't as Maven 3.3.1+ requires JDK 7+ and I'm new to all of this so I don't know how to fix it.

After that I tried building it in Java 7 and this happened.
**[INFO] Scanning for projects...
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[ERROR] Child module C:\Users\luisc\Downloads\olauncher\olauncher\pom.xml of C:\Users\luisc\Downloads\olauncher\pom.xml does not exist @
@
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project dev.figboot:olauncher-parent:0.1.0 (C:\Users\luisc\Downloads\olauncher\pom.xml) has 1 error
[ERROR]     Child module C:\Users\luisc\Downloads\olauncher\olauncher\pom.xml of C:\Users\luisc\Downloads\olauncher\pom.xml does not exist
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
Process finished with exit code 1**

How do you run .sh files on Windows? I've been trying to get a Linux VM up with openjdk6.

So I got it kinda working in Linux, but I need a version of fernflower compatible with Java 6 to get further.

How do you run .sh files on Windows? I've been trying to get a Linux VM up with openjdk6.

I used Intellij IDEA to run them, don't know if I did it right tho

Where would I find a older compiled version of fernflower for Java 6?

https://github.com/fesh0r/fernflower/commit/35259c3e202294d1725666e544faa9ed95fdc272 this commit seems like the commit before they began requiring java 8, seems ancient. Are you sure you can't use the java 8 version of fernflower? It may produce output compatible with java 6 with some modification

Is there a flag or something I can set to make it ignore the wrong Java version?

you will still need to run fernflower with java 8, because a java 6 JRE can't run java 8 class files (the format probably changed)

So decompile with Java 8, then do the rest with Java 6?

yes

I get this even with Java 8
administrator@administrator-VirtualBox:~/olauncher$ ./applyPatches.sh
Cloning into 'olauncher'...
done.
~/olauncher/olauncher ~/olauncher
From ../launcher

[new branch]      master     -> upstream/master
HEAD is now at e5a4852 Initial commit
error: unknown option `no-gpg-sign'
usage: git am [options] [(|)...]
or: git am [options] (--continue | --skip | --abort)
-i, --interactive     run interactively
-3, --3way            allow fall back on 3way merging if needed
-q, --quiet           be quiet
-s, --signoff         add a Signed-off-by line to the commit message
-u, --utf8            recode into utf8 (default)
-k, --keep            pass -k flag to git-mailinfo
--keep-non-patch      pass -b flag to git-mailinfo
--keep-cr             pass --keep-cr flag to git-mailsplit for mbox format
--no-keep-cr          do not pass --keep-cr flag to git-mailsplit independent of am.keepcr
-c, --scissors        strip everything before a scissors line
--whitespace ...      pass it through git-apply
--ignore-space-change
pass it through git-apply
--ignore-whitespace   pass it through git-apply
--directory ...       pass it through git-apply
--exclude ...         pass it through git-apply
--include ...         pass it through git-apply
-C ...                pass it through git-apply
-p ...                pass it through git-apply
--patch-format ...    format the patch(es) are in
--reject              pass it through git-apply
--resolvemsg ...      override error message when patch failure occurs
--continue            continue applying patches after resolving a conflict
-r, --resolved        synonyms for --continue
--skip                skip the current patch
--abort               restore the original branch and abort the patching operation.
--committer-date-is-author-date
lie about committer date
--ignore-date         use current timestamp for author date
--rerere-autoupdate   update the index with reused conflict resolution if possible

~/olauncher
Patches did not apply cleanly to olauncher. Please fix.

So I got further, but maven automatically set my java back to java 8, and I got this error
Downloading: http://repo.maven.apache.org/maven2/org/apache/maven/plugins/maven-clean-plugin/2.5/maven-clean-plugin-2.5.pom
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] olauncher-parent .................................. FAILURE [0.453s]
[INFO] olauncher ......................................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 0.526s
[INFO] Finished at: Wed Feb 22 12:48:12 CST 2023
[INFO] Final Memory: 5M/119M
[INFO] ------------------------------------------------------------------------
[ERROR] Plugin org.apache.maven.plugins:maven-clean-plugin:2.5 or one of its dependencies could not be resolved: Failed to read artifact descriptor for org.apache.maven.plugins:maven-clean-plugin:jar:2.5: Could not transfer artifact org.apache.maven.plugins:maven-clean-plugin:pom:2.5 from/to central (http://repo.maven.apache.org/maven2): Failed to transfer file: http://repo.maven.apache.org/maven2/org/apache/maven/plugins/maven-clean-plugin/2.5/maven-clean-plugin-2.5.pom. Return code is: 501 , ReasonPhrase:Not Implemented. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginResolutionException
administrator@administrator-VirtualBox:~/olauncher$

New VM, copy paste is broken so I'm sending screenshots
Java 6

Java 7

hmm maybe you can run maven with java 8 but set the source/target compatibility to 1.6. That will make it write java 6 class files with a java 8 jdk

How exactly would I do that?

there should be a thing in the pom.xml

I found a target thing and set it to 1.6 but I get this when compiling.
long error.txt

You may be able to downgrade or remove that plugin. All it does is generate the version string for the window title and other stuff. Not particularly necessary.

How exactly would I go about doing that?

edit the pom.xml in the olauncher folder, the plugin configuration should be in there

I replaced it with 4.9.9 and I get this error now
`[INFO] -------------------------------------------------------------
[ERROR] COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] An unknown compilation problem occurred
[INFO] 1 error
[INFO] -------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] olauncher-parent 0.1.0 ............................. SUCCESS [  0.081 s]
[INFO] olauncher 1.7.2_07 ................................. FAILURE [  1.135 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  1.276 s
[INFO] Finished at: 2023-02-26T22:49:03-06:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.0:compile (default-compile) on project olauncher: Compilation failure
[ERROR] An unknown compilation problem occurred
[ERROR]
[ERROR] -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.0:compile (default-compile) on project olauncher: Compilation failure
An unknown compilation problem occurred
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:215)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:156)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:148)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
at org.apache.maven.cli.MavenCli.execute (MavenCli.java:957)
at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:289)
at org.apache.maven.cli.MavenCli.main (MavenCli.java:193)
at sun.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke (Method.java:498)
at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:282)
at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:225)
at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:406)
at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:347)

Caused by: org.apache.maven.plugin.compiler.CompilationFailureException: Compilation failure
An unknown compilation problem occurred
at org.apache.maven.plugin.compiler.AbstractCompilerMojo.execute (AbstractCompilerMojo.java:1215)
at org.apache.maven.plugin.compiler.CompilerMojo.execute (CompilerMojo.java:188)
at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:137)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:210)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:156)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:148)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
at org.apache.maven.cli.MavenCli.execute (MavenCli.java:957)
at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:289)
at org.apache.maven.cli.MavenCli.main (MavenCli.java:193)
at sun.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke (Method.java:498)
at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:282)
at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:225)
at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:406)
at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:347)

[ERROR]
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :olauncher
`

you need to set both source and target to 1.6 (you will likely have to modify the code to remove java8-only features that may have been used)

How do I know what's Java 8 or not?

what doesn't compile when you set source and target to 1.6

very big error.txt

you're going to have to rectify all of those by replacing them with constructs supported in java 6

I don't know how to do that.

@bigfoot547 won't you be able to do it for us pleaseee?

https://github.com/olauncher/olauncher/tree/java6 try this branch
NOTE: it is unlikely to work - a lot of the libraries are probably compiled for java 8. You may have to find java 6 versions of affected libraries. I cannot test this because I don't have a java 6 jdk.

If you're on Linux, this installs and just works on latest Ubuntu LTS (and probably forks too). Just select Java 6 (or 7) and get the .deb
https://www.azul.com/downloads/
I'll test it myself though to see what happens.

So it compiled without issue. I tried genredist, got an error, but the output jar without genredist works fine, at least with Java 8. For some reason I can't select Java 6 even though it was used to compile in my VM, so I will make a quick XP VM with Java 6 for testing.

Hmm

yeah, I suspected that would be the case. You're going to have to find versions of the libraries used for java 6. It shouldn't be too hard, just not something I have the time to do right now.


Update developer's guide

Ⅰ. Describe what this PR does

Remove go-delve part in Dockerfiles. go-delve may be inaccessible due to some network issue, which may cause a image build error.
Fix typo in how_to_develop.md
Update sample for develop use

Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

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

@@           Coverage Diff           @@
##           master     #135   +/-   ##
=======================================
  Coverage   20.66%   20.66%
=======================================
  Files          16       16
  Lines         329      329
=======================================
  Hits           68       68
  Misses        241      241
  Partials       20       20

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 a13335f...b10ef57. Read the comment docs.


Remove Facebook from docblock

I guess the docblock needs some more changes but I'm not sure.

Current coverage is 84.79% (diff: 100%)

Merging #159 into master will not change coverage

@@             master       #159   diff @@
==========================================
  Files            35         35
  Lines          1276       1276
  Methods         255        255
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1082       1082
  Misses          194        194
  Partials          0          0

Powered by Codecov. Last update 535081c...6f4cad8


chore(roadmap): update features with release versions

While you are at it do you want to tag the other (older) items with versions as well?

Sure!


Crash when insert  clipart from clipboard or file

06 / 14 / 2021 10:58UIThreadException
version <IP_ADDRESS>90 built on 30/05/2021 13:02
Oops, ppInk crashed! Please include the following information if you plan to contact the developers (a copy of the following information is stored in crash.txt in the application folder):
An item with the same key has already been added.
Stack Trace:
at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
at gInk.ImageLister.FromClipB_Click(Object sender, EventArgs e) in D:\gink.git\ppInk\src\ImageLister.cs:line 131
at System.Windows.Forms.Control.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnMouseUp(MouseEventArgs mevent)
at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.ButtonBase.WndProc(Message& m)
at System.Windows.Forms.Button.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

Hi @argostelios,
It looks like you are reporting 2 errors.
The first one (dialog box image)seems to have occured during drawing, most probably after inserting an image, can you confirm my analysis and provide the image you had just inserted
the second one seems to have occured during insertion of the clipboard. Can you confirm and detail how you filled the clipboard.
Thanks

I will try to show you all the steps  from cold start

First   i start the ppink
2)second i make a screen snip  from my windows 10

then i go to ppink and click the clipart ---->  click from clipboard

4)i repeat step 2 and 3 for a second screen snip

at this point everything is working fine,  now im going to show what im doing and the ppink bugs

When i delete the two clipboard that ppink stores witch i don't need them any more

and i repeat step 2 and 3  for a third screen snip i am getting this error

no  i am going to show you what i am doing and i can go around from this bug

i press no when that error occurs on step 5 second picture and the ppink restores the clipboard 1 from step 2 and 3

then i press again from clipboard

and the ppink shows the error from step 5 second picture and i press no
the the ppink paste the second clipboard witch was made from step 4

and from this point everythink is working, i press from clipboard and ppink inserts the third screen snip without error

summary
ppink  as many clipboards you delete in ppink as many  no you have to press  on the error message
looks  likes ppink wants to restore all deleted clipbords before let you insert another one.
i hope so i point out the problem
Thank you

I understand the problem  (reuse of an internal name) this should fix it. Be aware this includes the beta changes in progress (as discussed in other threads)
ppInk64_varFix14a.zip
Can you also describe the first test case (this was very well described 👍 )

it was the same error exactly what i described above  but i just post the crash report that ppink creates for developers i thought that would help.   im not developer no programmer i can't understand a think in that file

Thank you very much   i didn't  realize you fix it  and you send me the app.
that was quick
it works perfect! Thank you


Cft-default

Issue #, if available: #603
Description of changes:
added empty default values for all parameters which didn't have any default
    Default: ''

Description of testing performed to validate your changes (required if pull request includes CloudFormation or source code changes):

one deployment
tested the script ./scripts/deploy-cloudformation-stacks.sh

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Wrong branch used in the PR, will reopen a clean one


Finalise queries for EM to record CPU and Memory readings

The current queries used in scripts are somehow giving the doubled value for CPU and Memory due to the issue with cadvisor which is treating them as 2 different instances while recording the metrics. So prometheus is getting two entries for the CPU and memory recordings for the same pod which are getting added when the scripts use the sum function.
So we need to have a better study for getting exact results using a set of queries to determine the min, max, mean of the CPU and Memory usage.

Here we can see that the query is returning duplicate values for the memory of the same pod (only difference we could see is the id parameter)

so when we do a sum on the above query (How we do it in scripts) we get the addition of the same value which is 2x of the real value.

Changes were added by @bharathappali @kusumachalasani . Closing this issue.


Add owners file for merge control

Pull Request Test Coverage Report for Build 38

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
10 unchanged lines in 3 files lost coverage.
Overall coverage remained the same at 94.937%

Files with Coverage Reduction
New Missed Lines
%

exec_helpers/proc_enums.py
1
99.09%

exec_helpers/subprocess_runner.py
2
89.29%

exec_helpers/_ssh_client_base.py
7
97.91%

Totals

Change from base Build 36:
0.0%

Covered Lines:
825

Relevant Lines:
869

💛  - Coveralls


A higher level API for Bayesian statistical models

The recently developed rstanarm initiated a take-off in the adoption rate of the Bayesian framework by "general" scientists. It implements a high level API to fit all traditional regression models in a Bayesian framework just by adding "stan_" in front of the model's name (stan_glm(), stan_lmer()...). Critically, it includes good default for the priors, so that the end-user can directly fit a model without having to care about these things (which is good from a beginner-friendliness perspective).
I understand that this might not be the scope of Turing. Nevertheless, I was wondering if there were any plans to implement something similar (maybe in another package), allowing to do for instance bayes_glm(y ~ x) and fit a Bayesian regression. Thanks!

This is definitely something that could be supported -- there's probably already some useful work in Turing-Examples that could just be tidied up a bit. I agree that it would be helpful to have non-expert-friendly implementations of these things. One could imagine having a Turing stdlib, which contains a large collection of standard models for which we recommend samplers.

I agree this is a great idea. We haven't made any serious efforts towards this direction in the past. Perhaps we can try to organise some meetups/hackathons to help people implement new models and store them in Turing-Examples? Please feel free to make suggestions!
@DominiqueMakowski Some people are talking about a Turing hackathon on Saturday. Perhaps we can discuss more there?

@yebai Unfortunately I'm not physically attending the conference, but I'll follow what happened there :)

See e.g. https://github.com/TuringLang/TuringGLM.jl


rest: collection serialization

Adds Elasticsearch like envelope of search results.

Adds JSON pretty printing for browser API requests.

Signed-off-by: Lars Holm Nielsen<EMAIL_ADDRESS>

Coverage increased (+0.2%) to 65.007% when pulling e347bd7215ce50283e185604a3047e6eadb7b1ff on lnielsen:ser into 330ea4b6b387ccfcda38cf43dfc9f9abe87caa2c on inveniosoftware:master.


KAFKA-12276: Add the quorum controller code

The quorum controller stores metadata in the KIP-500 metadata log, not in Apache ZooKeeper.  Each controller node is a voter in the metadata quorum.  The leader of the quorum is the active controller, which processes write requests.  The followers are standby controllers, which replay the operations written to the log.  If the active controller goes away, a standby controller can take its place.
Like the ZooKeeper-based controller, the quorum controller is based on an event queue backed by a single-threaded executor.  However, unlike the ZK-based controller, the quorum controller can have multiple operations in flight-- it does not need to wait for one operation to be finished before starting another.  Therefore, calls into the QuorumController return CompleteableFuture objects which are completed with either a result or an error when the operation is done.  The QuorumController will also time out operations that have been sitting on the queue too long without being processed.  In this case, the future is completed with a TimeoutException.
The controller uses timeline data structures to store multiple "versions" of its in-memory state simultaneously.  "Read operations" read only committed state, which is slightly older than the most up-to-date in-memory state.  "Write operations" read and write the latest in-memory state.  However, we can not return a successful result for a write operation until its state has been committed to the log.  Therefore, if a client receives an RPC response, it knows that the requested operation has been performed, and can not be undone by a controller failover.

I think we need to handle preferred leader election in a special way. For example, if the assigned replicas are 1,2,3, isr is 2,3 and the current leader is 3, when doing preferred leader election, we want to keep the leader as 3 instead of changing it to 2.

Hmm, wouldn't we want to switch the leader to 2 in that case, since 2 is more preferred?

Thanks, @junrao ! :)


Creating meshes without textures is a little confusing

Primarily talking about https://github.com/marovira/atlas/blob/master/source/atlas/utils/Mesh.cpp#L228. Here the if statement says if we have textures, but no attrib index for them, than we return. However, if we have no textures and we don't specify an index for textures (we want to render without textures), than we don't hit that code within the if statement and crash on an index out of error. Same applies for the normal check above

I'd suggest just changing it to
if (!mImpl->hasNormals || arrays.size() < 2) and similar for the other, if that's intended behavior. Who knows, it's 3:45 am

I see what the problem is, and you're right in that we could end up with an index out of range error. The issue here is a bit more subtle than this though, and to be honest I hadn't really thought about it. Suppose that we don't have normals, so we don't specify a binding point, but we do have textures (which could contain the normal data) and we want to use them. The question becomes, how do we know we are not supposed to bind the normals but bind the textures? I need to think about this a bit more, but you are right that as it stands it is a bit confusing.


Test ol.format.KML#readNetworkLinks fails on Firefox

Test: "returns an array of network links"
expected 'http://localhost:3000/test/bar/bar.kml' to equal 'bar/bar.kml'

right I noticed the same recently: https://github.com/openlayers/ol3/pull/3171#issuecomment-86548794

This is because node.baseURI seems to use window.location.href on Firefox


Can't install in VS2012

Trying to install the latest version 1.1.2 and get the following error:
The extension could not be installed because the following error occurred:
This extension is not support on the current product.

Yeah, it only works for VS 2015 and above. Sorry.


Add exercise card to DOJO exercises page

This pull request doesn't change any user-facing pages, it only changes the DOJO exercises pages (e.g. /exercises/js0) which currently doesn't have any links to it.
This pull request adds the exercise card to the DOJO exercises page. It currently uses mock data on the frontend and doesn't store the user's answers. It will be connected to the backend and real data in a subsequent pull request.
How to test:

Go to /exercises/js0
Click "SOLVE EXERCISES" button
Answer questions, show and hide solutions, click "SKIP" button, click "PREVIOUS" button, click "Exit" button

The newly added "SOLVED EXERCISES" button on the right side of the /exercises/js0 page:

The first exercise:

The solved second exercise:

This pull request is a part of https://github.com/garageScript/c0d3-app/issues/2253

More margin (32px) between the main content and the nav would be nice

The error is different than the design:

Should be:

Other features such as flagging and the filtering dropdown will be addressed in a subsequent PR?

Other features such as flagging and the filtering dropdown will be addressed in a subsequent PR?

Yeah, I'm just trying to get the features done in small pull requests so they're easy to review. We can include more stuff in later pull requests.

The error is different than the design:

Should be:

With red border and red text.
Actually, this is related to the previous PR and shouldn't be addressed in this PR.

Thanks! Fixed in https://github.com/garageScript/c0d3-app/pull/2326


Strapi docker mongodb atlas

Hello!
I was trying to setup strapi via docker and connect it to mongodb atlas. And looks like approaches to replace mongodb atlas connetion string from docs and video works fine when we launch app via yarn or npm.
But what about docker? I still do not understand how to make it works :(
I could connect to this db via gui or mongoose:
version: '3'
services:

  strapi:
    image: strapi/strapi
    environment:
      DATABASE_CLIENT: mongo
      DATABASE_URI: mongodb+srv://arhe1:<password>@arhetericaclaster-5s8wz.mongodb.net/strapi?retryWrites=true&w=majority
    volumes:
      - ./app:/srv/app
    ports:
      - '1337:1337'

You will need to update your project config to support those env vars. The image doesn't hold anything related to the database connection. When using it, a new project is created with the default config so you will need to configure the strapi app.
Closing the issue.

You've made me confused. Here we have example for mongodb connection in image config...
Btw, could you provide more details how to update this configs for setup via docker? Is this data storing in docker volume?

You need to create a strapi project first. Then configure the database.js config for your usecase. then setup your docker compose scenario. The image we provide is just a basic nodejs image with some dependencies added.

@iKethavel were you able to make this work?, I'm running into the same situation, I assumed the docker image was setup so no prior configuration was needed or if there is some needed it'd be done through the env vars.


bug: Cannot move cursor into middle of contenteditable

Type: bug
Platform: all
If you have a

and enter some text, it is not possible to move the text cursor somewhere into the middle of the text using mouse clicks or tap gestures, only to the very beginning or end of the text.
This can be seen here:
http://codepen.io/anon/pen/vEyrqe
It seems to work if you replace the condition "ele.contentEditable === 'true'" in js/utils/tap.js (line 164 in the current master) by "ele.isContentEditable".

I've just run into this one, and the resolution does appear to make it better for me.

Hello! Are you guys still experiencing this issue with the latest version of ionic?

I've not tested it for a very long time. Sorry.

@morungos No problem! Thanks for using Ionic! This looks like something i will look into on our end. I will be closing this issue for now, but feel free to continue the conversation!

I tested on this: http://codepen.io/anon/pen/vEyrqe
There are still problems

https://github.com/driftyco/ionic/blob/aef490b177406f9bd7d532294abd6c3198357622/js/utils/tap.js#L164
chaning ele.contentEditable === 'true'  to  ele.isContentEditable works

This bug still alive

Still happening.


dub: Updating (1.1.0 -> 1.5.0) and improving

Motivation for this change

Updating 1.1.0 -> 1.5.0
gcc not needed at all as dependency
other D compilers can be used instead of dmd, so it's not good to use it as a runtime dependency.
Adding checkPhase
Improve package description.

Things done

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

[ ] NixOS
[ ] macOS
[x] Linux

[ ] 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 nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

TODO

Need to wait for the final release of dub 1.5.0
Add the interactive-remove.sh unittest because it should work with final 1.5.0 version. (see https://github.com/dlang/dub/issues/1224)
Running the unittests only works with the fixes for dmd from https://github.com/NixOS/nixpkgs/pull/27743 so this PR needs to be merged first.

This is ready to be merged from my side.
It's not building because it needs dmd which is not building on master but a fix for it comes with https://github.com/NixOS/nixpkgs/pull/28635.

What's the status on this now that #28635 is merged?

https://github.com/NixOS/nixpkgs/pull/28635 is merged but there were problems with the build because of the sandboxing.
The new PR which needs to be merged ist https://github.com/NixOS/nixpkgs/pull/29469 now.

https://github.com/NixOS/nixpkgs/pull/29469 is now merged into release-17.09 and master (build already available https://hydra.nixos.org/build/61651969) so this PR is ready to be merged.


typescript 2.1 and moment.d.ts

I'm still having problems with importing moment in a project with es6 modules in typescript.
It is common that node_modules is listed in the exclude compiler options, so moment.d.ts can not be found with this configuration.
I've found that I can get it working, when I copy moment.d.ts to @types/moment/moment.d.ts (@types is not ignored by typescript) and when I replace
export = moment;
with
declare module 'moment'{   export = moment; }
My tsconfig.json is
`
{
"compileOnSave": true,
"compilerOptions": {
    "noImplicitAny": true,
    "noEmitOnError": true,
    "removeComments": false,
    "sourceMap": true,
    "module": "es6",
    "target": "es5",
},
"exclude": [
    "node_modules"
]

}`
Maybe you could consider following the latest guidelines for typescript declaration files and publish the type declarations to the npm repository @types (or definitelytyped, as they are picked up by @types ) and changing the declare statement (which is in line with the way other libraries do it, e.g. jQuery)

Not sure why you're running into these issues - consider filing a bug on the TypeScript repo, since that shouldn't happen.

For people who don't have "node_modules" excluded, and who prefer to continue using an in-repo moment.d.ts file, what worked for me to fix the import error is to replace the export = moment; line in the d.ts file with export default moment;.
This fixes the "module has no default export" error for me in the files which use the module (by means of import moment from "moment";), and provides full intellisense.
In contrast, the declare module 'moment'{ export = moment; } solution in the OP above provides the intellisense, but causes a "declaration or statement expected error" error for me inside the moment.d.ts file itself.
*==========
My TypeScript: 2.1.4
My .tsconfig:
{
    "compilerOptions": {
        "target": "es2015",
        "allowJs": true,
        "jsx": "react",
        "noImplicitAny": false,
        "moduleResolution": "node",
        "watch": true,
        //...
    },
    // ...
}

I found that if in my typescript base object I put
/// <reference path="../../node_modules/moment/moment.d.ts" />
I can reference the moment's d.ts however to get it to work I have to add
export as namespace moment;
at the top of the moment/moment.d.ts file and all compiles and works as expected.

Since at @DanielRosenwasser is on the TypeScript team, and he's saying this should be over there, I'm going to close this.

I think this issue should be reopen. And the line export as namespace moment; should be added to moment.d.ts, as most other @types definitions do (source-map, angular, etc).

And it still is a problem if you include use moment in an external lib.

I can't see this is a typescript problem. Other libraries (e.g. angular) provide a .d.ts file in the @types repository of npm and they seem to work just fine with tsconfig module="es6". Typescripts recommended way of providing typescript definition files is npm@types so why can't moment follow their recommendations ?

Instead of modifying the moment.d.ts directly, we found that defining a moment.shim.d.ts file in the src/ directory fixed the issue with the following code fixed the issue.
import * as moment from "moment";
export as namespace moment;
export = moment;


Added social tab in the home page

I have added the social link for the project where a person can reach out if in case does not recieve any response on the repository. I hope it helps in the communication issue of your project.

@Sarvadnyaawaghad150503
Thanks for your contribution!


Focus Point coordinates lost if eager loaded?

When I eager load a focus point asset the output image no longer crops around the focal point, but if I revert back to lazy loading it works as expected?
In the below code, listingImage is a focuspoint field.
{% set workItemsEager = craft.entries({ 	section: 'work', 	limit: 8, 	with: [ 		'listingImage', 	] }) %}
{% set transformedImages = craft.imager.transformImage(image,[ 	{ width: 500, height: 500, jpegQuality: 60 }, 	{ width: 320, height: 320, jpegQuality: 60 }, 	],{ 		allowUpscale: false, 		mode: 'crop', 		position: image.focusPctX ~ '% ' ~ image.focusPctY ~ '%', 		interlace: true 	}) %}
When outputting a transformed image from this code, the position is ignored. If I remove the "with" lines the focus point is retained.

To clarify, it looks like I still have access to image.focusPctX and image.focusPctY when eager loaded, but the value reported is incorrect, it appears to be centered?

+1 for this issue. Having the same problem when trying to use it on eager loaded elements.

Did anyone come up with a solution for this issue?

@MattWilcox Did you manage to resolve this at all?


Extract provider versions from json file in import-assets

This PR extract provider versions from file instead hardcoded in import-assets utility. It would be better to manage resources in json file. In addition to that, although it is not being used, providerFilter can be used if import-assets is executed directly by user to filter.

cc: @alexander-demichev

@asalkeld could you please have a look at?, thanks.

can you also update the readme?

can you also update the readme?

Done

/lgtm


Build issues on ESP32-C3

I'm unable to use the project as a dependency on an ESP32 based project.
Please see a minimal reproduction at https://github.com/nevi-me/esp32-c3-matter-minimal. The summary is that some of the dependencies don't yet support the esp-idf framework. The esp-rs folks are aware of some of them, as there are forks at https://github.com/esp-rs-compat.

I had created an example that built on ESP32 (https://github.com/kedars/matter-rs/blob/master/examples/esp-onoff-light/Cargo.toml). The crypto part was all nullified, but the build was functional. It was quite an old version of the matter-rs so, but in case it helps your investigations.

@kedars We should probably close this, because the two new branches compile for ESP IDF just fine.

@nevi-me Were you successful in running either of the branches on an ESP32 MCU?


Fix some Liquid warnings

Describe the proposed changes
Wrap some Docker log template directives in {% raw %} because when Liquid sees {{ }} it gets really excited and treats them as mix-ins.
Project version
n/a
Related issue
n/a
Related issue or PR in another project
n/a
Please take a look
@johndmulhausen @joaofnfernandes @londoncalling

I haven't tested to the degree Joao has, but I do know for code blocks you can also use ````liquid` at the start of your code block and it will process curly braces directly.

Since the JS error is related to Angular weirdness, let's get this in to stop any weirdness around Liquid/rendering.


[p2p] add config to allow setting TCP_NODELAY on TcpStreams

TCP_NODELAY is not on by default in contrary to Golang, so wanted to open a pr for it :)

Congrats on opening the first PR ❤

Would you mind making this a config (like read_timeout, write_timeout, etc)? I think it is the right approach to not drop the connection if it can't be disabled for some reason but think we should at least log (probably DEBUG)?
I could see a world where some applications could get a lot better performance by using the Nagle algorithm.

If you get bored, I think the same could eventually be done for linger: https://doc.rust-lang.org/std/net/struct.TcpStream.html#method.set_linger

Would you mind making this a config (like read_timeout, write_timeout, etc)? I think it is the right approach to not drop the connection if it can't be disabled for some reason but think we should at least log (probably DEBUG)?
I could see a world where some applications could get a lot better performance by using the Nagle algorithm.

Yeah makes sense, just updated the PR now 👍.

If you get bored, I think the same could eventually be done for linger: https://doc.rust-lang.org/std/net/struct.TcpStream.html#method.set_linger

Sure I can look into this one too in a follow up PR. I believe its value should be set to write_timeout config, right?

Very close!

Sure I can look into this one too in a follow up PR. I believe its value should be set to write_timeout config, right?

I would create a separate config but set its value to be the same (in the default) 👍 .

I was reading about TCP_NODELAY a tad bit more before merging this...I wonder if it is a better idea to implement this as Option<>?
Like with TCP_LINGER, not setting TCP_NODELAY at all is different than setting it explicitly (will otherwise use OS default behavior).
I think this is less of a concern but I also know that some compile targets don't support this (and might panic or behave unexpectedly). For example, WASI doesn't (not necessarily a goal to work there but also not not a goal lol): https://github.com/rust-lang/rust/blob/19ebdcedf1f5619f8f06c546c7541bb9b54aecda/library/std/src/sys/pal/wasi/net.rs#L150
My view, when in doubt, is make things as configurable as possible with sane defaults. That would mean making this an Option that defaults to None and letting devs choose how to use it.
Curious to hear your thoughts!

Oh I did not know that it was not supported by some targets. My target is always unix and never thought about this 😄. In that case, making it Option makes the most sense indeed. I just converted it to Option and updated the comment. Im also thinking now if this should be a target specific code like it only targets unix and windows?

Oh I did not know that it was not supported by some targets. My target is always unix and never thought about this 😄. In that case, making it Option makes the most sense indeed.

👍  (I think in all common "server" deployments, it is supported...so probably would've been fine)

I just converted it to Option and updated the comment. Im also thinking now if this should be a target specific code like it only targets unix and windows?

I think for now, leaving it as is will be fine. I'd prefer to just let anyone using the crate make that decision for themselves.

Cool. Btw best of luck on your new journey. Im hoping to contribute more by time 🙂


segfault key press

Hi,
I'm still getting similar segfaults from wlc.
#0  0x0000000000418e2a in container_map ()
#1  0x000000000040e891 in cmd_kill ()
#2  0x000000000040b74c in handle_command ()
#3  0x000000000041b966 in  ()
#4  0x000000000041c431 in  ()
#5  0x00007ffff7bb1619 in wlc_keyboard_request_key (state=
WL_KEYBOARD_KEY_STATE_PRESSED, key=16, mods=0x7ffff7dd8f98 <wlc.lto_priv+920>, time=, keyboard=0x7ffff7dd8e68 <wlc.lto_priv+616>)
at /home/ksj/.cache/pacaur/wlc-git/src/wlc/src/compositor/seat/keyboard.c:354
#6  0x00007ffff7bb1619 in seat_handle_key (ev=0x7fffffffe710, seat=0x7ffff7dd8c18 <wlc.lto_priv+24>)
at /home/ksj/.cache/pacaur/wlc-git/src/wlc/src/compositor/seat/seat.c:133
#7  0x00007ffff7bb1619 in input_event.lto_priv.112 (listener=0x7ffff7dd9198 <wlc.lto_priv+1432>, data=0x7fffffffe710)
at /home/ksj/.cache/pacaur/wlc-git/src/wlc/src/compositor/seat/seat.c:198
#8  0x00007ffff7bad35c in wl_signal_emit ()
at /usr/include/wayland-server-core.h:357
#9  0x00007ffff7bad35c in input_event (fd=, mask=, data=0x7ffff7dd8960 <input.lto_priv>)
at /home/ksj/.cache/pacaur/wlc-git/src/wlc/src/session/udev.c:197
#10 0x00007ffff7544d32 in wl_event_loop_dispatch ()
at /usr/lib/libwayland-server.so.0
---Type  to continue, or q  to quit---
#11 0x00007ffff75431da in wl_display_run () at /usr/lib/libwayland-server.so.0
#12 0x00007ffff7bc396c in wlc_run ()
at /home/ksj/.cache/pacaur/wlc-git/src/wlc/src/wlc.c:260
#13 0x000000000040aa1f in main ()
and interesting log file:
sway.txt

container_map, cmd_kill and handle_command are not wlc functions.


Latest version 3.2.0 exposes some vulnerabilities

Exposed vulnerabilities:

https://www.cve.org/CVERecord?id=CVE-2022-41727
https://www.cve.org/CVERecord?id=CVE-2023-29407
https://www.cve.org/CVERecord?id=CVE-2023-29408

Introduced through:
<EMAIL_ADDRESS>Fixed in:
<EMAIL_ADDRESS>

Thank you for sharing this @alexopenline, we will look at this ASAP.

All three CVEs involve processing an image: decoding a TIFF image.
Even if you are storing TIFF images as events in EventStoreDB, it wouldn’t call the code mentioned in the CVEs.
However a client application that’s decoding a TIFF image from an event could be at risk. In such a case, the onus is on the app developer to use the later version of Go’s TIFF dependency you mention.
We are currently using the October 16 version of this rpc dependency. The  Oct 16, Oct 30 and Nov 6 versions of rpc do not mention this as a security issue.
Thanks.


Return a split Dataset in load_dataset

...instead of a DatasetDict.
# now supported
ds = load_dataset("squad")
ds[0]
for example in ds:
    pass

# still works
ds["train"]
ds["validation"]

# new
ds.splits  # Dict[str, Dataset] | None

# soon to be supported (not in this PR)
ds = load_dataset("dataset_with_no_splits")
ds[0]
for example in ds:
    pass

I implemented Dataset.__getitem__ and IterableDataset.__getitem__ to be able to get a split from a dataset.
The splits are defined by the ds.info.splits dictionary.
Therefore a dataset is a table that optionally has some splits defined in the dataset info. And a split dataset is the concatenation of all its splits.
I made as little breaking changes as possible. Notable breaking changes:

load_dataset("potato").keys() / .items() / .values() / don't work anymore, since we don't return a dict
same for for split_name in load_dataset("potato"), since we now iterate on the examples
..

TODO:

[x] Update push_to_hub
[x] Update save_to_disk/load_from_disk
[ ] check for other breaking changes
[ ] fix existing tests
[ ] add new tests
[ ] docs

This is related to https://github.com/huggingface/datasets/issues/5189, to extend load_dataset to return datasets without splits

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Just noticed that now we have to deal with indexed & split datasets. Two tests are failing because one should be able to get an indexed dataset when accessing the split of a dataset made of indexed splits (right now the index is just trashed)


Fix the preprocessing

Two changes to the way queries are preprocessed:

use a python pool for multiprocessing
do not use pickle to serialize feature modules, pass on the paths instead.

Fixes #74 and #56. Great, many thanks!


Simple DNS provider for testing

Hi,
I wonder if it would be feasible to add a simple DNS resolver to Caddy so that when using it on localhost development it can be used to to hijack where the DNS for a domain resolves to.
Maybe for simplicity of doing this, Consul could be used?
What do you think?

What about just editing the hosts file?


[MRG] Update nteract-on-jupyter to 2.1.3

Updated the version and refroze.

I think this should be uncontroversial so merging.


form page

center the form in the page (it is a desktop page)
see the specification for new fields in the form
https://docs.google.com/document/d/1l0By8a6PK4rIZYMtkzc45goLI6meemnBMcmeyr2DPbA

there is a pb with on field


merge "ManifestDescriptor" and "Descriptor" types

Currently, we have ManifestDescriptor and Descriptor types. The only difference is the inclusion of the platform field. For making generic processors that filter based on platform type, this creates a number of complexities (see https://github.com/docker/containerd/pull/638 for an example of this pattern).
We'll simply need to merge these types in the specification and struct definitions.

@stevvooe @jbouzane Based on the description of this ticket:

We'll simply need to merge these types in the specification and struct definitions.

We need to update both spec and struct definition, however I think #620 only updates the struct definition, we need another PR to update spec (descriptor.md and image-index.md)?

@qianzhangxa oops, for some reason I thought that had already been done, sorry. Feel like putting up a PR?

We need to update both spec and struct definition, however I think #620 only updates the struct definition, we need another PR to update spec (descriptor.md and image-index.md)?

Actually, this was a conscious decision not to. The alternative fields can be ignored outside the context of processing a manifest index. We only merge the Go datatype to make usage easier in practice.

@jonboulle Can we close this or did you want me to add some clarification on the difference between the spec and datatype?

@jonboulle Ping.

I'd still like to see the tweak discussed in #638 - @qianzhangxa @q384566678 either of you feel like taking this on?

Sure. I have submitted #649 for it.

I think it should be explained for the difference. #649 looks good to me.

I think the json file needs to be updated to coincide with the specs-go / v1 file, just as I do here.

@stevvooe does #649 fix this issue?

@stevvooe does #649 fix this issue?

This was already fixed by #620. This was never meant to have bearing on the specification.

right on


Tuning sliders (e.g Position control sensitivity) don´t keep their position, return to left att.

No problem to adjust them, but once you leave the tuning page and return, they are back @ left position.
Furthermore, it would be great, if it would be possible to assign them by choice to an Aux channel (potentiometer), so in flight tuning would be easier.

This is in QGC I assume. This should be a QGC issue.

Thx. Do I have to close this, and place it @ QGC issues, or wait till its sure ?

Tuning sliders are just changing parameters, so nothing to do with firmware directly. You'll need to close this and reopen on the QGC side.


make common expression alias human-readable

Which issue does this PR close?

Closes #10280.
Thanks @JasonLi-cn for pointing out the relevant code snippet and the detailed description, saved me some time ❤
Rationale for this change

Making the plans human-readable when common subexpression elimination is applied (check example in ticket and edited tests in this PR).
What changes are included in this PR?

Changing alias_symbol from curr_expr_identifier + sub_expr_identifier to #{curr_expr_identifier}, making it more readable.
Also, since we no longer need the sub_expr for the alias, we can remove the entire visit stack + visit record enum.
Are these changes tested?

Changed existing tests to reflect the change.
Are there any user-facing changes?

Readable plans when CSE is applied

I don't think removing the accumulation and a couple of clones will increase perf, but let's see
/benchmark

Thanks for the review @alamb.
I was trying to implement it duckdb style with #0, #1, etc ... But as I'm not super familiar with the optimizer, I failed to do so, maybe someone can help.
The biggest problem I found was that the rule is applied twice, meaning that we need a shared/global mapping of expr -> id between rule calls.

Per the discussion on https://github.com/apache/datafusion/pull/10396, we merged that one first and now this one needs to be rebased / resolved. Marking it as draft until we can do that
I believe @MohamedAbdeen21  plans to do it this weekend (per https://github.com/apache/datafusion/pull/10396#issuecomment-2099130452)

At this point I think we'd better revert this, close it and revisit it after #10413. @alamb @peter-toth

Revert PR: https://github.com/apache/datafusion/pull/10436


Ikkle Hydra

Hydra drop rate is 1/3000, you must have forgot a 0 and put 1/300. :)

lol, right you are. I just fixed the plugin, and I'm going to submit it to the hub right now.
Thank you for bringing this to my attention :)


Added wildcard ignore for .keystore files

Keystore files with an extension of .keystore are also valid and commonly used.

@Orbyt any links to StackOverflow discussions or other documentation about this change?

@shiftkey JKS is generally used to denote "Java Key Store", but the extension used is largely redundant. When generating keystores, some developers opt for the .jks extension, but it is also very common to simply use a .keystore extension. Additionally, the default debug keystore generated by the Android SDK tools is named debug.keystore.


WIP: set tmpfs volume rootcontext correctly

For #6238

@thockin @markturansky @pweil- @erictune Looking for feedback on this factoring; still need to test a couple permutations, add unit tests, and add e2e for tmpfs test.

LGTM overall

LGTM

Pulled out my initial stab at the e2e tests into #6362

Could you rebase? Thanks!

@yujuhong I'm pretty sure that I will close this out, actually.  I'm waiting for feedback on #6379 which I expect to obviate this.  I will close or rebase in the next 6 hours or so.

@pmorie shall we close this now?

@rjnagal Yep


Trades publisher

This adds a minimal trades publisher that listens for trades on a websocket API and publishes events to Ensign.

very cool @pdeziel ! The code is well written and I was able to run this on my machine!

Awesome to hear, thanks for running it!