cmake fortran with yaml fails ctest

$ cmake .. -DPHREEQCRM_BUILD_PYTHON:BOOL=ON -DPHREEQCRM_WITH_YAML_CPP:BOOL=ON -DCMAKE_BUILD_TYPE:STRING=Release -DPHREEQCRM_FORTRAN_TESTING:BOOL=ON
$ cmake --build . -j10
$ ctest
Test project /home/charlton/source/repos/coupled/phreeqcrm-master/_config/phreeqcrm-0.1.0-9067/_cmake
    Start 1: TestCompileAndRun
1/2 Test #1: TestCompileAndRun ................***Exception: SegFault  0.39 sec
    Start 2: SimpleAdvect.py
2/2 Test #2: SimpleAdvect.py ..................   Passed    0.05 sec

50% tests passed, 1 tests failed out of 2

Total Test time (real) =   0.44 sec

The following tests FAILED:
          1 - TestCompileAndRun (SEGFAULT)
Errors while running CTest
Output from these tests are in: /home/charlton/source/repos/coupled/phreeqcrm-master/_config/phreeqcrm-0.1.0-9067/_cmake/Testing/Temporary/LastTest.log
Use "--rerun-failed --output-on-failure" to re-run the failed cases verbosely.

https://github.com/scharlton2/phreeqcrm/tree/default_ctor
https://github.com/scharlton2/phreeqcrm/commit/78f53a3f00fe7d06ab6a306e086a27b561bf6ce9

Closed by PR #47


1.0.11 was last release?

Is 1.0.14 going to be released? Using a git endpoint can be risky if the repo has unresolved issues.

@stevenvachon
I'm a little confused by your question.  The last (and final) released version on this fork was 1.0.15 back in January of this year https://rubygems.org/gems/rails3-jquery-autocomplete/versions
As shown on the README.md I have officially forked over to https://github.com/bigtunacan/rails-jquery-autocomplete and changed the gem name to "rails-jquery-autocomplete"  There have been 3 releases this year.  This was some confusion where people were assuming only Rails 3 was supported and as I move towards Rails 4 and beyond I felt now was as good a time as any to clear that up.

I was confused by thistoo @bigtunacan and @stevenvachon.  The last tagged release in github is 1.0.11, but now that I look over at rubygems, I see 1.0.15, so it looks like it's just the tag that's missing.


#40: Add function call parser to select statement

#40

Coverage increased (+0.03%) to 88.04% when pulling 54e419ca323c3e43172d832c2a2fb4e8d9836101 on Eman-Shaaban:feature/40-add-function-call-to-select into 3976eb0131875673a7f779c53fcd8703516d0c4c on cassandra-scala:master.

Small nitpicks, otherwise LGTM.

Coverage increased (+0.03%) to 88.04% when pulling ad94470413d3673480816bf260b48887cea14c1e on Eman-Shaaban:feature/40-add-function-call-to-select into 3976eb0131875673a7f779c53fcd8703516d0c4c on cassandra-scala:master.


Error when resolving ambiguous class names

I noticed a problem when BeanShell tries to resolve ambiguous class names.  I will have to show a contrived example to demonstrate this in a simple way, but it does affect my application.  My actual actual situation is a bit too complication to show as an example, so please bear with the obvious non-standard naming that I will use to show the problem.
Imagine that I create a class named org.myorg.myapp.base.org.  (I realize that the classname of 'org' is unusual.)
Then I import this class in to the BeanShell interpreter:
import org.myorg.myapp.base.*;
new org();

I also have a class named org.myorg.myapp.Gui.  If I try to invoke the main method from this class like so
org.myorg.myapp.Gui.main(new String[0]);

the BeanShell interpreter will throw an error:
  'org.myorg.myapp.Gui.main(new String[0]); ' : No static field or inner class: myorg of class org.myorg.myapp.base.org

I had a look in the code and the problem is in the name resolving mechanism.  BeanShell will identify that the first part of the name (org) matches to a local known class (org.myorg.myapp.base.org), and will then try to resolve the rest of the name (myorg.myapp.Gui) relative to that class.
I think the problem is at line 292 in Name.java.  I believe that the search for a matching class should be done in reverse order (all parts first, then take away one at a time), allowing for explicitly qualified classes to match before looking at fields within the class.
Any comment on this?  Is this the right change to make, or I am missing something here?

TODO: Need unit tests see #72

Java also won't allow this, it will generate a compilation error:
cannot find symbol
symbol:   variable myorg
location: class org.myorg.myapp.base.org

The only way around this is to avoid the import of the conflicting name.
new org.myorg.myapp.base.org();
org.myorg.myapp.Gui.main(new String[0]);

will work in java and also works in BeanShell.
Trying to find a class backwards will also have the same issue when there is a class with static members identical to a class within a package.
A quick example I can think off now.
class org {
    static Integer val = 1;
}

package org;
class val {
  static Integer valueOf(in val) {}
}

org.val.valueOf(1);

Which valueOf method should we pick now? In java it will be the class named org, but with reverse class detection we will use the class named val.
Since BeanShell also attempts to be a java source interpreter it is best we stick within the java language constraints.
Closed: Won't fix, works as expected.


Bumps Localytics to 4.3.1

Requested by Aaptiv, Localytics is on 4.3.1, and Aaptiv would like to ensure that we are depending on the latest.
iOS Updates (4.0 > 4.3) include:

Fixed an issue that could cause a crash when Places is enabled in the Dashboard but Places integration has not been completed.
Fixed an issue that was caused when file protection is enabled globally for an app.
Fixed a bug that caused places identifiers with quotes to not be handled appropriately.
Fixed a bug where standard events would log transactions with a customer value of 0

.

By analyzing the blame information on this pull request, we identified @none to be a potential reviewer.

I'm wondering if I should suggest that Aaptiv run a pod update since we are currently depending on 4.0, the update should pull in the latest...

Yes they can just declare pod 'Localytics', '~> 4.3.1' to get the latest as well. The problem with us making this change is that everyone using this integration has to now use Localytics 4.3.1. They can't use an older release b/w 4.0 and 4.3.1. We typically depend on the "minimum" version required for the integration to work.


Support fine-grained control over default certificate paths

The XCM_TLS_CERT environment variable may be used to use a non-compile time default directory for the certificate, key, and trust chain files used by the TLS transport.
With XCM_TLS_CERT, the file names (e.g., key.pem) are fixed. In some cases, the files may reside in different directories, or have names which does not correspond to the file name chosen by XCM. In such cases, it would be useful to have a complementary set of environment variables, allowing arbitrary and independent file names (and path) for all three files (e.g., XCM_TLS_CERT_FILE, XCM_TLS_KEY_FILE, and XCM_TLS_TC_FILE).

Using socket attributes is a better option.


Request: Make some packages non internal or expose some needed methods

Hey!
I'm trying to implement my own PubSub logic(like destroy all subscribers when a publisher disconnects, etc) and the problem is some important packages like packet.Packet are internal and the readPacket/writePackage methods are also internal, with that I would have to copy too many files to implement my logic. It makes sense?
Thanks!

Yes that makes sense. These packages are currently internal in order to have smaller API surface. Now that there seems to be some demand, it makes sense to open it up more.

All the internal packages are now public.


feat: Implement RequiredAsterisk component.

fixes #128

Pull Request Test Coverage Report for Build 708

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.8%) to 79.003%

Files with Coverage Reduction
New Missed Lines
%

src/components/Button/content.js
1
60.0%

Totals

Change from base Build 704:
-0.8%

Covered Lines:
419

Relevant Lines:
512

💛  - Coveralls


more performant hostdb locking in interaction methods

Closing this for now, I do not think it's the bottleneck anywhere. I'm also not confident what the consistency model is between the hostdb and the hosttree, I thought you were supposed to have the hostdb locked when manipulating the hosttree. Maybe not, but since this is non-critical and also controvertial, closing for now


Add host filtering endpoint

Add host filtering endpoint similar to https://web.archive.org/web/20240106075954/https://siacentral.com/api/docs/#list-get in Siacentral.  Also support querying the v2/rhp4 versions of fields like contract price if requested by the user.

That was helpful PJ.  V2 along with AcceptContracts are now pointers and optional to specify.


[Security] Workflow coverall.yml is using vulnerable action actions/checkout

The workflow coverall.yml is referencing action actions/checkout using references v1. However this reference is missing the commit a6747255bd19d7a757dbdda8c654a9f84db19839 which may contain fix to the some vulnerability.
The vulnerability fix that is missing by actions version could be related to:
(1) CVE fix
(2) upgrade of vulnerable dependency
(3) fix to secret leak and others.
Please consider to update the reference to the action.

Closing in favor of https://github.com/asyncapi/.github/issues/113. We manage 90% of actions in one repo and then auto-replicate them.


suppress unused fields in params message?

Because of some challenges introduced by the way we use protobufs, it is difficult to create custom messages in VRX 2.0 and above. As a consequence, we rely heavily on the "params" message. A drawback is that this message is very noisy and ros2 topic echo commands produce output that is hard to read without instrumentation.
The situation would be much better if there were an easy way to suppress unused fields in the param message. For example, in the following entry from the range_bearing topic for the acoustic pinger, only 2 fields are really relevant:
- name: range
  value:
    type: 3
    bool_value: false
    integer_value: 0
    double_value: 21.468399037071904
    string_value: ''
    byte_array_value: []
    bool_array_value: []
    integer_array_value: []
    double_array_value: []
    string_array_value: []

If we could display only the name and double_value fields this message would be much more human-friendly. I'm creating this issue as a reminder to check what options might be available.

Closing this per discussion. There is not a currently supported way to suppress unused fields (except by filtering through another program). The plan is to wait for custom message support to improve, then restore our custom messages.


fix: have modal positioning account for toolbar

Before this PR
The IM modal wouldn't open in the right position.
After this PR
The modal takes account of the SFCC top toolbar when settings its position.
Screenshots

Current dependencies on/for this PR:

next

PR #256

PR #258

PR #259

PR #257

PR #267

PR #275

PR #264

PR #274

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: 6cwJaETgMG17vfBY88e2


is there a way to use a picture and not the camera?

Expected Behaviour
Actual Behaviour
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Cordova CLI version and cordova platform version
cordova --version
cordova platforms

Plugin version
cordova plugin version | grep phonegap-plugin-barcodescanner

Sample Code that illustrates the problem
Logs taken while reproducing problem

Sorry to be a jerk but I'm now closing issues that don't follow the issue submission guidelines. This plugin takes a lot of work to maintain and I can't afford the time required to solicit the required info from everyone.
I'm closing this issue but please feel free to re-open the issue if you provide the requested details from the new issue submission guidelines.
https://github.com/phonegap/phonegap-plugin-push/blob/master/.github/ISSUE_TEMPLATE.md


Updated get_next_id function to avoid errors

Earlier the literal 'int' errors were generated by get_next_id func. I rewrote the function with different logic and now there is no errors in extracting the data

Patched the Error in the Last Update.
It was Because , the Structure of the AuthenticationDetailsDonor works a little Different (Like a Log Book) . We won't be updating that table when new donor function gets triggered.
Additionally , get_next_id_secondary_function generates the id , by looking into the authentication_id field from the donorDetail table.
Check the Main Branch Code Once , it should be working !


px_uploader: multiple baud rates

This doesn't change the default baud rate yet since it's not clear what baud rate will be set.  However it enables easier migration from one version to another that has a different baud rate set.

LGTM

Thanks!


Android - App freezes after reopening it from background after a while

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

Expected Result:
App doesn't freeze and able to view the chat
Actual Result:
App freezes
Action Performed:

Launch the app
Log in with any account
Put app in background
Open the app again

Workaround:
Unknown
Platform:
Where is this issue occurring?
Web
iOS✔
Android ✔
Desktop App
Mobile Web
Version Number: 1.0.67-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

https://user-images.githubusercontent.com/43996225/121615149-374c8480-ca2e-11eb-8e14-b3b3c6fae188.mp4
Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @chiragsalian (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Added the deploy blocker for this one, I'm pretty sure I didn't see this issue on the last regression.

CC @kidroca
I believe this was introduced by the Onyx changes. However, while I have seen this occur it's not reproducible every time for me on 1.0.67.-0.

Unassigning myself since i'm OOO today and cant take a look at it. Posted in engg-chat to hopefully get someone else to help out.

I wonder if this was fixed in https://github.com/Expensify/Expensify.cash/pull/3535/files. Going to see if I can reproduce this on main

Not able to reproduce this on iOS any more.

Yeah, I'm not able to reproduce this on Android either.

Retested this on android and it was not reproducible 🎉 Closing.


Mouse shortcuts

Extracted from Microsoft/vscode#10744

From @DarceyLloyd
Mouse shortcuts
Shortcuts are fine, however would love to get access to the mouse. Eg in phpstorm I use the side mouse buttons (forward and back) with ALT and CTRL combinations for an additional 6 very quick and easy shortcuts that don’t require you to let go of the mouse when coding.
Loving the editor by the way!

That would be really useful!

/duplicate https://github.com/microsoft/vscode/issues/3130


Window Invert Color Utility

Description of the new feature / enhancement
App to invert the colors of a specific window.

Scenario when this would be used?
Invert colors on apps that do now have native dark mode capability.
Supporting information
An app already exists with this functionality - but it would be nice to have similar functionality as a part of the PowerToys suite.
APP: https://www.wintools.info/index.php/easy-window-invert
https://jack-vanlightly.com/blog/2022/12/28/windows-shortcut-to-invert-colours-that-works

/dup #13005


Add serverCommsAddresses value in values.yaml for HA support

Background
We need to allow users to specify multiple server comms addresses to support Octopus Server running in an HA configuration.
[sc-75000]
Results
To maintain backward compatibility, we'll still support the singular form of this variable, but we will also allow users to specify multiple with the new variable.

I've left this in draft because it needs a Tentacle version bump once the related Tentacle PR is merged.


Status Code 401 for Musixmatch

after running the search function in loop for a while it starts printing out this message in the terminal:
[Musixmatch] Got status code 401 for [TRACK_NAME] [ARTIST_NAME]
Although it doesn't seem to be a problem, since the function still returns the lyrics, do you know how to keep this from happening anyway?
In case there was no solution, whould there be a way to avoid printing the message in the terminal?
ps: thanks for making this package, it's really great!

Has nobody found a solution yet? This is really annoying me

same here...might be related to some api-limit

I don't even need a solution for whatever error 401 is, i'd just like an option to make it not get printed to the console


Can I contact you?

Hey, do you have telegram or anywhere I can contact you about a project?

yes
telegram: @AdrianoBarrosMba


Won't let me login

I've checked if I have messed up my Username and Password but, it's correct.I've tested this by logging into Instagram same Username and Password but, for some reason when I attempt at logging into Instagram from this app I'm unable to log in for some reason. Please help,thank you for reading.

I've checked if I have messed up my Username and Password but, it's correct.I've tested this by logging into Instagram same Username and Password but, for some reason when I attempt at logging into Instagram from this app I'm unable to log in for some reason. Please help,thank you for reading.

me too, what did you do?


Resizing window to a smaller size causes DOM elements to overlap

This causes some End-to-End tests to fail when run on a smaller window-size because certain DOM elements (such as buttons) are covered by other DOM elements, thus rendering them unclickable.

I'll take a look to see whether this might have to do with some of the CSS changes I made in #442.

Resolved by specifying window size when running unit test

User has to provide a reasonable window size for rendering the document, closing for now


State label overflows the state width

In certain case and zoom level the state label overflows the state width. I noticed it depends on the label string. I didn't have a good open source example, so I changed the source code to a hardcoded label ABCDEFGHIJKLMNOPQRSTUVWXYZ_abcdefghijklomnopqrstuvxyz in time-graph-state.ts method renderLabel().
It looks like for strings that start with multiple upper case characters followed lower case characters it happens. For other strings, for example all lower case, all upper case or lower case characters followed by upper case it doesn't happen.
It can be reproduced with the example app in this repository or the theia-trace-extension.

I was able to replicate the issue without needing to change the labels of the states. In my example, I have only lowercase letter and some special characters.


Remote validation when submit the submit button value is null

Remote validation when submit the submit button value is null

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually, especially after several years with no activity. It doesn't mean I'm abandoning the project, just that I'm unable to work through 200+ open issues with the little time I can afford to spend on this project.
To the reporter (or anyone else interested in this issue): If you're still affected by the same issue, please consider opening a new issue, with a testpage that demonstrates the issue with a current version of the plugin. Even better, make an attempt to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!


Webhook URLs

Leave examples in Webhook URLs and/or explain in the Readme what they are used for.

Will add once it is in a working state.

ok. Thank you!


Document validation with linkml-validate and navigating the schema with schemaview

@eecavanna - we could possibly use this branch if it made sense to you, to collaborate on useful SchemaView and Validation documentation for NMDC.

Yes! Thank you for bringing this to my attention. I will add a link (here) to the ticket I created about documenting some of this stuff; i.e., https://github.com/microbiomedata/nmdc-schema/issues/2278. Working on this branch works for me. Are you OK with me committing things to it directly (at this stage, where it is still in a draft state)?

yes pls feel free to push w/o PR! :)


replace erros

#236

[x] pkg/util/httpRetriever.go
[x] pkg/util/logConfig.go

Pull Request Test Coverage Report for Build 370859514

0 of 3 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 36.597%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/util/httpRetriever.go
0
1
0.0%

pkg/util/logConfig.go
0
2
0.0%

Totals

Change from base Build 370843109:
0.0%

Covered Lines:
1596

Relevant Lines:
4361

💛  - Coveralls


Fix Int#downto with unsigned int going down to 0.

See issue #6669. When using unsigned integers with the downto method, and the to is equal to 0, the code is an endless loop:
3.downto(0) { |i| p i } # Prints 3, 2, 1, 0
3_u16.downto(0) { |i| p i } # Endless loop printing 3_u16, 2_u16, 1_u16, 0_u16, 65535_u16, ...

Both version of the downto method (with and without a block) are fixed by this patch.

What happens with 3_u16.downto(4_u16).to_a?

Hum. 3_u16.downto(4_u16).to_a does not work. Let me fix the case when it goes up to return an "empty" iterator.

@gmarcais You can probably do something like @done = false unless @from >= @to in the iterator's constructor.

I modified to @done = !(@from >= @to) in the constructor and in rewind.

@gmarcais Would be good to add a spec for a case mentioned by @asterite.

Added spec cases for .to_a.

Please also fix #upto at into max (see https://github.com/crystal-lang/crystal/issues/6669#issuecomment-418900135)

I don't get that error. It does not seem to have anything to do with the current patch:
 1) Semantic: macro allows declaring class with macro expression
       fork: Cannot allocate memory (Errno)

I changed the comment on the patch. Does this have anything to do with it? (Seems unlikely, but I ask nevertheless). Any idea what went wrong?

It does not seem to have anything to do with the current patch

That's correct, nothing to worry about.

@gmarcais Why close?

I did not do that. Not sure what happened. Can it be reopened (I personally cannot apparently)? I was still considering fixing the step method and related.

I added a version of the step method that does not overflow as well.

@gmarcais You probably need to rebase your branch against master

@asterite Yes, I rebased and added the step method. That allowed me to reopen the issue.
I am not yet convinced about my implementation because of the following issue.
The step function is possibly a mess of many different types: self, to, by. They may all be different types. Say self is an Int32 equal to Int32::MAX-2 and to is a UInt32 equal to UInt32::MAX/2 + 2 (which is higher than Int32::MAX), what should the method do? Yield Int32 values up to Int32::MAX? Yield UInt32 values up to to?
One can imagine crazier combination: self is an Int32, by is a Float and to is a Double larger than Int32::MAX.
Should there be some type restrictions?

I don't think it would hurt to just restrict the arguments to the same type as self.

Yeah, it's a crazy combination. Right now you can even do:
1.step(to: 3.5, by: 0.5) do |i|
  puts i # => 1.0, 1.5, ...
end

I'm not sure restricting the types is the way to go, but we could try it.
But I'd try to not involve step in this PR because it will make it harder to decide when and how to merge this. Let's focus on downto, and we can fix upto and step in separate PRs (or, well, upto is probably fine in this PR).

I'd cast to the self type using self.class.new(to), like like the lhs type rule for operators.

OK, I reverted the step method changes. The current PR only contains the upto and downto methods. I'll create a new PR for step.


question: will you be re-introducing .name?

reference:
https://github.com/jashkenas/coffee-script/issues/2052

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Adding name to the constructor, just like super, would be great for building class hierarchies.  Currently, these issue all say to just create our own property in the instance or prototype.  Yes, that works, but it becomes tedious and isn't DRY, and so more prone to errors.

+1 @drarmstr


Not showing all git submodules

Issue Type: Bug
I have a git repo with 13 submodules. All of these are displayed with git submodule status. Anyway, only 10 of them are displayed within the SOURCE CONTROL PROVIDERS list, together with the master repository on the first line (11 in total).
When I make some changes in one of the invisible repos, the master repository shows there is a change in the submodule - the submodule is listed in the CHANGES list. Anyway, when I click this line, no diff is opened.  The issue is not that the submodules would be scrolled out. They are simply not there.
VS Code version: Code 1.46.1 (cd9ea6488829f560dc949a8b2fb789f3cdc05f5d, 2020-06-17T21:13:08.304Z)
OS version: Linux x64 4.19.0-9-amd64
Remote OS version: Linux x64 4.15.0-106-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (4 x 2592)

GPU Status
2d_canvas: unavailable_softwareflash_3d: disabled_softwareflash_stage3d: disabled_softwareflash_stage3d_baseline: disabled_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: disabled_softwareviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
0, 0, 0

Memory (System)
3.85GB (0.28GB free)

Process Argv
--no-sandbox

Screen Reader
no

VM
100%

DESKTOP_SESSION
/usr/share/xsessions/lxqt

XDG_CURRENT_DESKTOP
LXQt

XDG_SESSION_DESKTOP

XDG_SESSION_TYPE
x11

Item
Value

Remote
SSH: pilatus

OS
Linux x64 4.15.0-106-generic

CPUs
Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz (8 x 2999)

Memory (System)
15.55GB (4.24GB free)

VM
0%

Extensions (18)

Extension
Author (truncated)
Version

doxdocgen
csc
0.6.0

vscode-markdownlint
Dav
0.36.0

gitlens
eam
10.2.2

todo-tree
Gru
0.0.177

kconfig
luv
1.0.3

camelcasenavigation
map
1.1.2

git-graph
mhu
1.24.0

vscode-docker
ms-
1.3.1

remote-containers
ms-
0.122.1

remote-ssh
ms-
0.51.0

remote-ssh-edit
ms-
0.51.0

cmake-tools
ms-
1.4.1

cpptools
ms-
0.28.3

hexeditor
ms-
1.0.1

devicetree
plo
0.1.1

diff
raf
0.0.1

shellcheck
tim
0.9.0

cmake
twx
0.0.17

Please increase the limit using the git.detectSubmodulesLimit setting.

Hi Joao, thank you, that was indeed the problem.
Anyway, I suggest there is a space for improvement. I understand it may be reasonable to have some limit, but how about to show a warning in the SOURCE CONTROL PROVIDERS list that not all submodules are loaded, with a link to the appropriate setting? I admit I did not spend much time with looking for the setting, on the other hand I did spend some time - I searched for provider. But how can I, as a user, know that this is a problem of the git extension so I should be looking for git settings?


_DefaultPub.get tries to read version file but does not hold the lock long enough to do so

_DefaultPub.get has this code:
    final File currentVersion = _fileSystem.file(
      _fileSystem.path.join(Cache.flutterRoot!, 'version'));
    // ...
    if (/*...*?
        lastVersion.readAsStringSync() == currentVersion.readAsStringSync()) {
      // ...

That code has a race condition. If that command runs in parallel with another flutter command, there's no guarantee that the currentVersion file will still be around.
We should not release the lockfile until after this test is done.
cc @christopherfujino @jmagman

@Hixie thanks for root causing this one!


Update Android.gitignore

Reasons for making this change:
The libraries folder makes change lists too cluttered and is generated during a gradle build
Links to documentation supporting these rule changes:
https://intellij-support.jetbrains.com/hc/en-us/articles/206544839-How-to-manage-projects-under-Version-Control-Systems

@AdrianVovk thanks!


Doesn't work on Chrome 72

Maybe due to some problems with storage functionalities.
Reporter:
https://twitter.com/Yuichiro_S/status/1078235803887722497
options.js:36874 Uncaught (in promise) TypeError: Illegal invocation: Function must be called on an object of type StorageArea
    at Promise (options.js:36874)
    at new Promise (<anonymous>)
    at doAsync (options.js:36863)
    at Object.get (options.js:36835)
    at Object.pickOut (options.js:36843)
    at Main._callee$ (options.js:38835)
    at tryCatch (options.js:199)
    at Generator.invoke [as _invoke] (options.js:426)
    at Generator.prototype.(anonymous function) [as next] (chrome-extension://dojehnlfnmmhkgahnfnjeeemhleflhfc/options/options.js:252:21)
    at asyncGeneratorStep (options.js:38662)

Memo:
I made sure this error didn't happen when Chrome was v71. When this issue was reported and I fixed it, Chrome was Beta version yet(72.0.3626.28).
(Since this issue is referred from several repositories I added this memo)


Android - Workspaces - The icon of the deleted offline workspace is broken

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

Action Performed:

Open the App
Log in with any account
Go to Settings -> Workspaces -> Open any workspace
Kill off the internet connection
Delete the workspace

Expected Result:
The icon of the deleted offline workspace should be square's form
Actual Result:
The icon of the deleted offline workspace is broken
Workaround:
Unknown
Platforms:

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

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/93399543/223285065-2ab984bb-19fc-4478-ab66-f9e0c9be095c.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Triggered auto assignment to @marcaaron (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

This feels pretty edge case to me and imo not a deploy blocker. Considering it's reproducible on production I'm going to remove the blocker and we can just fix this when we fix it.

Triggered auto assignment to @jliexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Very odd - I tried on Staging .79 and couldn't reproduce on my Pixel 3a:

I also specifically tried deleting the blue icon, and couldn't reproduce:

What phone are you using @kbecciv ?

Closing, unable to reproduce on v .80-1

Ah sorry @marcaaron - didn't realise you commented. I can't reproduce this on staging or production!

I'm on v1.2.87-0 and it looks like it's happening now:

Job added to Upwork: https://www.upwork.com/jobs/~01f8699d8d61ac8a4c

Triggered auto assignment to @sophiepintoraetz (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @s77rt (External)

Triggered auto assignment to @dangrous (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Uhh wtf, I'm in Bug Zero. Unassigning Sophie.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The workspace icon when deleted while offline show another circle behind it.
What is the root cause of that problem?
The issue is similar to this issue https://github.com/Expensify/App/issues/15283 where on Android, the opacity from parent is "inherited" to the child. More explanation here https://github.com/Expensify/App/issues/15283#issuecomment-1439413441
What changes do you think we should make in order to solve the problem?
We can simply apply the same solution from the proposal, but I think we can avoid it because there is another problem I see.
So, the circle behind it is a default background color (green) for the old workspace icon. Our new workspace icon already have it's own bg color, so we don't need the circle bg color anymore.
https://github.com/Expensify/App/blob/c803b5ea434967cf959a0a34c2178d0037069933/src/components/Avatar.js#L88
https://github.com/Expensify/App/blob/c803b5ea434967cf959a0a34c2178d0037069933/src/pages/workspace/WorkspacesListPage.js#L126
We can simply remove the iconStyle

@bernhardoj Thanks for the proposal. Your RCA is correct, we are applying a redundant style where its effects are only visible on Android due to how opacity styles are applied differently on Android compared to other platforms. The fix of removing that redundant style makes sense to me :+1:.
:ribbon: :eyes: :ribbon: C+ reviewed
cc @dangrous

@getusha Thanks for the proposal. Your RCA is partly correct but I don't think we want to apply styles.popoverMenuIconEmphasized style for any case. Your first attached screenshot seems referencing to the old workspaces UI and I don't think we want to revert to that either.

📣 @bernhardoj You have been assigned to this job by @dangrous!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

Hired @s77rt and @bernhardoj

PR is ready for review.

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.2.89-0 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/16330

If no regressions arise, payment will be issued on 2023-04-03. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@s77rt / @dangrous] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@s77rt / @dangrous] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@s77rt / @dangrous] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@jliexpensify] Determine if we should create a regression test for this bug.
[ ] [@s77rt] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@jliexpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

The PR that introduced the bug has been identified: https://github.com/Expensify/App/pull/15280
The offending PR has been commented on: https://github.com/Expensify/App/pull/15280#issuecomment-1485471299
A discussion in #expensify-bugs has been started: https://expensify.slack.com/archives/C049HHMV9SM/p1679935587721169

Regression Test Proposal

Login to App
Go to Settings > Workspaces
Create a new workspace if you don't have one
Open any workspace
Turn off the internet
Delete the workspace
Verify the deleted workspace avatar does not have a circle behind it

Test looks good to me @s77rt . Responded in the chat thread, as well. Thanks for handling that!

Looks good, paying now

Paid (+ bonuses), and closing now


need feh cron for switching desktop background on schedule

https://wiki.archlinux.org/index.php/Feh
https://wiki.archlinux.org/index.php/Systemd/Timers

ideas:

feh cron for switching desktop background on schedule

ideas:

feh cron for switching desktop background on schedule

ideas:

feh cron for switching desktop background on schedule


feat: updated manifest paths for my CloudWorld '23 sessions

@jasperan - Could you share and update the PR with the WMS ID of the workshop?

certainly @anooshapilli the changes are pertaining to two WMS IDS: 11228 and 11226

@jasperan - Just following up on this PR.

Yes @anooshapilli that reference will be removed, the index.html found in ocw23-tenancy will be the new production URL (I believe I did make this change in WMS as well).

@jasperan - I Updated the WMS brown button URL with the link - https://oracle-livelabs.github.io/analytics-ai/league-of-legends-data-extraction/ocw23-tenancy/index.html
Could you create a new workshops folder under league-of-legends-data-extraction folder, move these two folders into that and create a new PR?


Bug 2181323: Select 'Size' automatically if 'Default InstanceType' chosen

📝 Description
Fixes:
https://bugzilla.redhat.com/show_bug.cgi?id=2181323
Select the 1st of the options in Size dropdown if already some Default InstanceType was chosen. Prevent missing 'instancetype.kubevirt.io/default-instancetype' label in the created bootable volume's yaml in case of not choosing Size in Add volume modal, because the correctly set instanceType label requires both parts defined. It does not make sense to only choose Default InstanceType without size.
With this change, also align with the actual behavior in Edit modal accessible in Bootable volumes list. Save one user's click when automatically selecting the 1st option from sizes, if they already choose something from Default InstanceType dropdown. They still can choose some other option if they want.
🎥 Demo
Before:
it's possible to add a bootable volume with only Default InstanceType chosen, without Size => label missing:
https://user-images.githubusercontent.com/13417815/228647283-92f62f18-fc9f-4524-b006-46cc709e35a1.mp4
After:
'Size' automatically set if 'Default InstanceType' chosen by the user => label correctly set:
https://user-images.githubusercontent.com/13417815/228647292-db2be1ab-f980-404a-9d0f-1b6d3863eef5.mp4

@avivtur @metalice @pcbailey @upalatucci @vojtechszocs please review


fixed aliases being returned in unmapped_index_fields

Signed-off-by: Petar Dzepina<EMAIL_ADDRESS>Description
GetMappingsView API was returning aliases in "unmapped_index_fields" due to bug in field type filtering.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

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

Codecov Report

Merging #147 (1b04c86) into main (68231e9) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #147      +/-   ##
============================================
- Coverage     39.19%   39.17%   -0.03%
+ Complexity      891      890       -1
============================================
  Files           176      176
  Lines          6702     6699       -3
  Branches        815      815
============================================
- Hits           2627     2624       -3
+ Misses         3834     3833       -1
- Partials        241      242       +1

Impacted Files
Coverage Δ

...ch/securityanalytics/mapper/MappingsTraverser.java
82.92% <100.00%> (+1.32%)
:arrow_up:

.../opensearch/securityanalytics/util/IndexUtils.java
60.46% <0.00%> (-0.90%)
:arrow_down:

...g/opensearch/securityanalytics/model/Detector.java
69.89% <0.00%> (-0.70%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fixed sensitivity of swipe gestures

-> only swipe when swipe vector is within the coordinates diagonals.
This is useful if you have scrollable containers on your carousel "pages" that need to be scrollable in y-direction themselves. It was very hard to scroll without initiating a swipe on the jcarousel.

Thanks for your help


Videostream changegable size and position

As I don't have enough space above the octo mirror module it would be great to have an option to select wether the streaming window should be placed above, under, left or right of the normal informations. Also it would be nice to select the size of the streaming area.

Also have this request.  Size of the streaming video is very large, be nice to halve its size!

A little related to this, my stream us upside down because that was the best way to mount my webcam. Octoprint has a way to flip or rotate the webcam stream. It would be nice to do this also.

Very interested in being able to resize the stream, it takes up way to much space on the screen, was hoping to have it as a small screen on the side or top corner.

@BadenMorgan If you are interested, I forked this repo and added a few changes. Still a work in progress though the stream is handled differently and can be rotated, scaled and paused.

I too have this request, but will be taking a look at the forked repo above for more versatility.

I too have this request, but will be taking a look at the forked repo above for more versatility.

HELLO I NEED TO ROTATE MY CAMERA 180 IF ANYONE KNOWS THE FILE TO CHANGE THAT WOULD BE GREAT

Your edit custom.css in config MagicMirror directory and add
.octomirror-module img {     max-width: 300px;     width: 300px; }


halo2-ecc: ec_double helper doesn't work for secp256r1

halo2-ecc/src/ecc/mod.rs helper ec_double helper only work for secp25611 , not work for p256, need to be updated as following:
if current point is P (x, y)
then
lambda = (3* x ^2  + a)  / 2y (mod p)
x_3 = lambda^2 - 2x (mod p)
y_3 = lambda(x_3 -x)  (mod p)
the point(x_3, y_3) is the addition result.

better to make modification compatible to both two curves instead of creating new helper special use for r1 curve.


exception in 2.0.1

In the 2.0.1 version on Craft 4.12.1 - the main plugin file throws an exception on the handleException method -- PHP Warning ‘yii\base\ErrorException’ with message ‘Attempt to read property “sentry” on null’ in /var/www/html/vendor/statikbe/craft-sentry/src/Sentry.php:74
Editing the main plugin class method handleException to call the main class with self::$plugin->handleException($exception); instead of Sentry::$plugin->sentry->handleException($exception); fixes this issue, but since your main and develop branches are both on 5.0.0 I don't know how to submit a PR for 2.0.1 which is used on Craft 4.

Ran into this issue - if the edit can be made to the Craft 4.0 version (heads up that the README title needs to be updated on that branch too ... it says its for the 3.0 version) that would be awesome to ensure we have no future issues.
With Craft 4 still being an active release it makes sense to patch :)

Yes, makes sense to patch - I'll have a look and try to get a fix out asap!

Editing the main plugin class method handleException to call the main class with self::$plugin->handleException($exception); instead of Sentry::$plugin->sentry->handleException($exception); fixes this issue,

@justinholtweb that might remove the error but it will break the plugin I think. The initial code  (Sentry::getInstance()->sentry->handleException($exception);) refers to a handleException function on the SentryService, where your change (self::$plugin->handleException($exception);) refers back to the function you're in at that point, creating a loop...

Hey @justinholtweb @cherrykoda, I pushed an update with some changes related to this (2.1.1) but couldn't reproduce the issue myself. We recently started adding the plugin to all are builds and haven't seen this error there either...
Can you try the latest release and see if that changes anything?

@janhenckens has 2.1.1 been released? I don't see it in releases and can't install that version locally.

@janhenckens Installing 2.1.0 results in the following error:
Exception 'Error' with message 'Call to a member function getSettings() on null'
in /var/www/html/vendor/statikbe/craft-sentry/src/Sentry.php:76
Stack trace:
#0 /var/www/html/modules/sitemodule/src/Sitemodule.php(85): statikbe\sentry\Sentry::handleException(Object(yii\base\InvalidConfigException))
#1 [internal function]: modules\sitemodule\Sitemodule->__construct('sitemodule', Object(craft\console\Application), Array)
#2 /var/www/html/vendor/yiisoft/yii2/di/Container.php(411): ReflectionClass->newInstanceArgs(Array)
#3 /var/www/html/vendor/yiisoft/yii2/di/Container.php(170): yii\di\Container->build('modules\sitemod...', Array, Array)
#4 /var/www/html/vendor/yiisoft/yii2/BaseYii.php(365): yii\di\Container->get('modules\sitemod...', Array, Array)
#5 /var/www/html/vendor/yiisoft/yii2/base/Module.php(445): yii\BaseYii::createObject(Array, Array)
#6 /var/www/html/vendor/yiisoft/yii2/base/Application.php(313): yii\base\Module->getModule('sitemodule')
#7 /var/www/html/vendor/craftcms/cms/src/console/Application.php(66): yii\base\Application->bootstrap()
#8 /var/www/html/vendor/yiisoft/yii2/base/Application.php(271): craft\console\Application->bootstrap()
#9 /var/www/html/vendor/yiisoft/yii2/console/Application.php(124): yii\base\Application->init()
#10 /var/www/html/vendor/craftcms/cms/src/console/Application.php(52): yii\console\Application->init()
#11 /var/www/html/vendor/yiisoft/yii2/base/BaseObject.php(109): craft\console\Application->init()
#12 /var/www/html/vendor/yiisoft/yii2/base/Application.php(204): yii\base\BaseObject->__construct(Array)
#13 /var/www/html/vendor/yiisoft/yii2/console/Application.php(89): yii\base\Application->__construct(Array)
#14 [internal function]: yii\console\Application->__construct(Array)
#15 /var/www/html/vendor/yiisoft/yii2/di/Container.php(419): ReflectionClass->newInstanceArgs(Array)
#16 /var/www/html/vendor/yiisoft/yii2/di/Container.php(170): yii\di\Container->build('craft\console\A...', Array, Array)
#17 /var/www/html/vendor/yiisoft/yii2/BaseYii.php(365): yii\di\Container->get('craft\console\A...', Array, Array)
#18 /var/www/html/vendor/craftcms/cms/src/Craft.php(59): yii\BaseYii::createObject(Array, Array)
#19 /var/www/html/vendor/craftcms/cms/bootstrap/bootstrap.php(260): Craft::createObject(Array)
#20 /var/www/html/vendor/craftcms/cms/bootstrap/console.php(42): require('/var/www/html/v...')
#21 /var/www/html/craft(12): require('/var/www/html/v...')
#22 {main}`

While I am seeing a 2.1.1 tag here on github, its not findable in composer.

Problem 1
- Root composer.json requires statikbe/craft-sentry 2.1.1 (exact version match: 2.1.1 or <IP_ADDRESS>), found statikbe/craft-sentry[dev-master, dev-craft4, dev-develop, 1.0.0, ..., 1.1.0, 2.0.0-beta.1, 2.0.0, 2.0.1, 2.1.0, 5.0.0-alpha.1, 5.0.0, 5.1.0] but it does not match the constraint.

I did make the edit manually (the one on 2.1.1 compared to 2.1.0) with no effect to the outcome. The System is on:

PHP 8.2.15
Maria DB 11.1.2
Craft CMS Pro 4.12.1
Craft Commerce 4.6.11
Sentry.io Plugin 2.1.0

Other Notable Plugins:

Blitz
Craft Commerce
Element API
Imager X
And others...

Forgot to push 2.1.1, sorry about that - should be available now!
Are you seeing the error when an exception happens or on any page?

@janhenckens this is on throwing an exception, and 2.1.1 is still not installing via composer (sorry!)

Hey @justinholtweb @cherrykoda, I was any from the office for a couple of weeks and sort of forgot about this issue - if this is still happening in your project(s), would you be open to sharing the composer.json, composer.lock & database so we can try to reproduce it on our end?


Fix bug when consumer is started from the UI the second time.

Fix bug when consumer is started from the UI the second time.
Signed-off-by: azerr<EMAIL_ADDRESS>

This PR fixes this usecase:

start a consumer from the UI (by selecting a topic)
close the consumer view editor
start a consumer from the UI (by selecting the same topic), it opens the consumer view editor without displaying a dialog.

Thanks @angelozerr!


Adding more property mediators cause unexpected behaviours  description

Description:
If we try to add a new property mediator in between a property mediator and a Create operation from the Salesforcerest connector which are in the mentioned order, it will add more mediators, as shown below.

Suggested Labels:

Suggested Assignees:

Affected Product Version:
v0.9.24071508
OS, DB, other environment details and versions:
Steps to reproduce:

Create a new project.
Create a Rest API.
Add a resource.
Select the resource and add Create operation from Salesforcerest connector.
Add a property on top of the Create operation.
Add a new property mediator between the previously added property mediator and the Create operation.

Related Issues:

The latest pre-release v0.9.24072507 has resolved the issue. Please reopen if the problem persists.


Livestreamrequest throwing errors

Hi,
actually I recognized in the intro that issues due to snapshot and livestream are known and not unusual, but I still want to give a try. Snapshot is working and saving the file to my iobroker path. But the livestreamrequest is throwing severall issues.
Just attached the error log of iobroker. Hopefully it helps.
iobroker ring log.txt
Cheers Jan

okay fixed it by installing ffmpeg. Now having no issues been thrown but unfortunately still no file is created. Any idea?


distrobox (podman containers) can fail to run after an rpm-ostree upgrade

What happened?
Multiple users have reported in discord that after doing an rpm-ostree upgrade (or rebase), they reboot, and then their distrobox cannot be entered.
For me it looked like:
$ distrobox enter fedora
Container fedora is not running.
Starting container fedora
run this command to follow along:

 podman logs -f fedora

Error: unable to start container "79ac894fa239b849a92715a0b196861f645d1351eb5722af135740a006dc6aa1": crun: setrlimit `RLIMIT_NPROC`: Operation not permitted: OCI permission denied

"@scotty" reported: "I just upgraded to 38.20230509.0 and I amgetting this now. No rebase, just rpm-ostree upgrade."
Error: unable to start container "<container>": crun: setrlimit `RLIMIT_NPROC`: Operation not permitted: OCI permission denied

https://discord.com/channels/1072614816579063828/1074422586894712912/1105698916797792308
Relevant log output
$ rpm-ostree ex history
NOTICE: Experimental commands are subject to change.
BootTimestamp: 2023-05-09T20:12:37Z (7h ago)
CreateTimestamp: 2023-05-10T00:29:58Z (3h 13min ago)

  ostree-unverified-image:docker://ghcr.io/ublue-os/silverblue-main:38
                   Digest: sha256:974e6c2dc125ec79986b3d52afe91ab1370cb97582b9c8802554e35a22237561
                  Version: 38.20230509.0 (2023-05-10T00:28:31Z)
          LayeredPackages: langpacks-en pass

BootTimestamp: 2023-05-09T17:21:30Z (10h ago)
CreateTimestamp: 2023-04-29T01:24:16Z (1 week 4 days ago)
CreateCommand: upgrade --trigger-automatic-update-policy

  ostree-unverified-image:docker://ghcr.io/ublue-os/silverblue-main:38
                   Digest: sha256:f0636f5a97d4133fc90278dc6f0608a209afd6a235971c4765fc0eddc9f1fbec
                  Version: 38.20230428.0 (2023-04-29T01:23:40Z)
          LayeredPackages: langpacks-en pass

BootTimestamp: 2023-04-26T17:20:05Z (1 week 6 days ago)
CreateTimestamp: 2023-04-27T00:19:36Z (1 week 6 days ago)
CreateCommand: install pass

  ostree-unverified-image:docker://ghcr.io/ublue-os/silverblue-main:38
                   Digest: sha256:4bc0858e0f37bf40057d707bf61d5b06189148ab93a26fca0a1176a9ccdd9a9a
                  Version: 38.20230421.0 (2023-04-27T00:09:35Z)
          LayeredPackages: langpacks-en pass

I joined the discussion on discord and have been following along with the conversation since this happened to me. Im up north and away from my ublue machine, but once im back home im willing to run some experiments and trials to help out as much as i can

I believe this is the issue: https://github.com/containers/podman/issues/18555

I believe this is the issue: containers/podman#18555

based on this issue, I imagine that we started seeing this problem intermittently after Fedora upgraded to podman 4.5.0

The other issue is related to using kube, however a very similar issue happens without using kube. Upstream issue:
containers/podman#18714

It seems that exporting the container to an image and reimporting it to a new container allows you to fix existing containers without having to nuke them entirely. This is only useful for those that don't have a reproducible build process (e.g. you're using toolbox/distrobox and you've manually installed packages).
Distrobox provides instructions on how to do this here:
https://github.com/89luca89/distrobox/blob/main/docs/useful_tips.md#container-save-and-restore
I'm not sure how you'd approach this with toolbox unfortunately.

I think you should be able to do the exact same thing, you just snapshot the container into an image an then you use that as the origin image for a new toolbox container


How can I fix System.Numerics.Vectors conflict?

I am currently using CSharpMath 0.3.0 in my Xamarin.Forms application, because when I am trying to update it to 0.5.1, there is a System.Numerics.Vector conflict, and the LaTeX won't render. I don't know if these 2 things are related, but regardless can I do anything about this to fix it (to get it to render the LaTeX)?

I seem to have the same issue. Would be nice to know the fix to this.

I have also noticed that it makes the place for the render but not showing up. It's the basic Xamarin template with the 0.5.1. CSharpMath.

If anyone is wondering, if you are using CSharpMath above 0.4.0 you also need to add SkiaSharp as a nugget package to render LaTeX


Update troubleshoot-vm-by-use-nested-virtualization.md

Added changes to lines 77 and  91 to include Generation 2 while creating Hyper-V server.
if the wrong generation is selected the boot device will not be recognized and will be a different issue.

@chakra146 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@simonxjx : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Change available locations to regions for Product Package

This pull request is to close issue #126.
The list of changes:

renamed availableLocations to regions as availableLocations are deprecated
added a missing 'exec' in the contribution guide

I don't know if it's better to rename :available_locations to :available_regions... Maybe it will break someone's code, but as no-one has reported that before, probably it won't break anything. :smile:
Anyway, the regions format is different than availableLocations. So you cannot avoid changes in your tools/applications which use this gem.

Coverage increased (+0.06%) to 99.821% when pulling 7836e2d13f5ceeb4c606cdd7907d3710be1ffb59 on ArtsiomMusin:fix-available-locations-for-product-package into 03f75fdd15f15e6cbf16df27478d31df8293c8a8 on softlayer:master.


This is a pull to merge my code for mentors and Nader's results

Thank you for contributing! To help the review process, please provide the following:
Proposal
This is a pull to merge my code for mentors and naders results. Almost all of this was reviewed already, just need to merge code and did one change in Results - changed main div className to container to make page less wide
Related
Provide links to any associated pull requests or issues. You can use #123 to link to a PR or issue in this repository, or user/repo#123 to link to other repositories.
Checklist

[ ] I have read the code of conduct and contributing guide
[ ] I have made this pull request to the master branch
[ ] I have run all of the automated validation using npm run ship
[ ] I have added myself to the "contributors" list in the package.json (or do not want to)

I mention it in recommendations on top of the repo about what those changes are

And not a problem to change tomorrow 20 am it's not merged


Author Update

updated author details

@hectorkambow

I believe this PR was trying to "re-add" the files.
When files already exist in the repo - you can edit them instead of re-adding them.
That is where the merge conflict is coming from.


Inform the game editors about how to put rich text

Taso,
can we put a popup for the Unity rich text annotation allowed in the text editors?
This is bold text
This is italic text
<size=50>This large text</size>
<color=green>This is green text</color>
<color=#00ffffff>This is custom color text</color>

combinations are also valid e.g.
<size=50><color=green>This is bold, italic, large, green text</size></color>
Details in: https://docs.unity3d.com/Manual/StyledText.html

Let's freeze this for now

Putting tags was not considered good idea from curators. They are not font of it. They will provide a template for POI-Text. It is addressed in another issue to transform this template into poi-text unity template.


PHPLIB-1144: Add unit tests on Explainable::getCommandDocument()

Fix PHPLIB-1144

Add let and comment options to explain delete.
Remove explain option for Aggregate, it is incompatible with the explain command.

Adding a unit test will full options for each implementation of Explainable::getCommandDocument:

Count ✅
Delete ✅
DeleteMany covered by Delete
DeleteOne covered by Delete
Distinct ✅
EstimatedDocumentCount covered by Count
Find ✅
FindAndModify ✅
FindOne covered by Find
FindOneAndDelete ✅ + covered by FindAndModify
FindOneAndReplace ✅ + covered by FindAndModify
FindOneAndUpdate ✅ + covered by FindAndModify
Update ✅
UpdateMany covered by Update
UpdateOne covered by Update

These tests allow us to review exactly which options are sent to the explain command.
The comparison of ReadConcern values will become stronger with PHPC-2234.

CS and Psalm checks OK. The failing tests are unrelated, due to https://github.com/mongodb/mongo-php-driver/commit/358fd2fc174faed00ce5c9f91f645a35da1fcbe9


[async] StreamTransformer handleData with async

Hi all,
it would be nice if StreamTransformer's handleData
could handle async function;
here is a code sample
Thank you

There are two possible behaviors for an async handleData:

Pause the original stream while handling the data, or
allow output events to occur in a different order than the corresponding input events.

Both makes sense, but for a transformer, the former is probably more likely.
It's also the one which is directly implementable using an async function:
class AsyncStreamTransformer<T, R> extends StreamTransformer<R> {
  final FutureOr<R> Function(T event) _handleEvent;
  AsyncStreamTransformer(FutureOr<R> handleEvent(T event)) : _handleEvent = handleEvent;
  Stream<R> bind(Stream<T> source) async {
     for (var event in source) yield await _handleEvent(event);
  }
}

(or, if you don't need to expose the stream transformer, you can just use asyncMap with the event handler).
I'm not particularly fond of of the StreamTransformer API, but adding a StreamTransformer.fromAsyncHandlers constructor seems like a small thing.


Clean up current "Build" step

Will do. I deleted what I could from GitHub including this comment from
you, in order not to draw attention to it. But I cannot delete the PR
itself, that does not seem to be possible in GitHub.
On Tue, Nov 21, 2023 at 1:56 PM Thasso Griebel @.***>
wrote:

@.**** requested changes on this pull request.
In .github/workflows/publish.yml
https://github.com/castlabs/prestoplay-react-components/pull/17#discussion_r1400465205
:

 # TODO remove the --dry-run

 - name: Publish to NPM

   run: npm publish --dry-run

 - name: Add NPM Release Job summary

   run: |

     echo '### NPM Release' >> $GITHUB_STEP_SUMMARY

     echo "Released version ${{ env.VERSION }} of<EMAIL_ADDRESS>>> $GITHUB_STEP_SUMMARY

 - name: Build Storybook

   run: npm run build-storybook

 - name: Assume AWS role

   uses<EMAIL_ADDRESS>

   with:

     role-to-assume: arn:aws:iam::068037490145:role/prestoplay-web-web-uploader

@fingerartur https://github.com/fingerartur I got some feedback from
@amphied https://github.com/amphied on this one and re recommended to
move this to a secret or at least a variable
—
Reply to this email directly, view it on GitHub
https://github.com/castlabs/prestoplay-react-components/pull/17#pullrequestreview-1741803522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFYSKJB4Z55HQICPWE7SUB3YFSJIJAVCNFSM6AAAAAA7SXALOGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONBRHAYDGNJSGI
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

--
Artur Finger  |  Javascript Software Developer, PRESTOplay Team
@.***
[image: castLabs]
castlabs.com  |  @castlabs https://twitter.com/castlabs  |  LinkedIn
https://www.linkedin.com/company/castlabs
castLabs GmbH, Wilhelmine-Gemberg-Weg 5-7, 10179 Berlin, Germany | Managing
Directors: Michael Stattmann, Martin Tews | Amtsgericht
Berlin-Charlottenburg HRB 101555 B


regexp: wrong match found without capturing parens

http://play.golang.org/p/n5XbC7kTDt
package main

import (
	"fmt"
	"regexp"
)

func main() {
	s := "abc"

	rePass := regexp.MustCompile("(a.*?c)|a.*?b")
	reFail := regexp.MustCompile("a.*?c|a.*?b")

	fmt.Printf("%v\n", rePass.FindStringIndex(s))
	// Writes "[0 3]"
	fmt.Printf("%v\n", reFail.FindStringIndex(s))
	// Writes "[0 2]"
}

The overall match found should not depend on capturing parens. In this case, the a.?c branch of the alternative should take priority over the a.?b branch in both expressions, so both should match [0 3] not [0 2].
This happens in C++ RE2 as well.

Summarised from commentary elsewhere:
The problem is caused by factoring of common prefixes in alternations. Complex subexpressions (e.g. involving quantifiers) aren't safe to factor because that collapses their distinct paths through the automaton.
The fix that I've proposed for RE2 is very simple. I'll port it to Go with @rsc's blessing.


fix: set renderer within an effect to avoid updating the state during a render

Fixes https://github.com/decentraland/sdk/issues/664

Integrated in https://github.com/decentraland/js-sdk-toolchain/pull/479


style: add icon for qq and netease

style: beautify search bar
chore: up deps
inspired by #899

I'm not sure it's legal to use platform icon (as trademark) in our product. Let's do some research for that.

I'm not sure it's legal to use platform icon (as trademark) in our product. Let's do some research for that.

I've never thought about that. But as long as we're following license, I think we should be fine. (etc.
https://iconarchive.com/show/papirus-apps-icons-by-papirus-team/netease-cloud-music-icon.html)


Delete diagrams/logo.webp

Removing extraneous .webp image
Description of changes:

logo.webp appears to be an image not related to POSSUM, and this PR proposes its deletion.

Description of testing:
Please make sure the following changes have been made before creating a pull request.
Update Description

[ ] Change tested on actual device
[ ] Changes tested for simulator
[ ] Existing device functionality is working fine
[ ] Unit Tests Written for Code Changes, and Verified that Coverage is 100% for Modified Files(with the exception of ATM devices)

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


Add REAML 2022

This add the REAML2022 Summer School about Resource-Aware Machine Learning in Dortmund.
The 6th International Summer School on Resource-Aware Machine Learning (REAML 2022) provides lectures on the latest research in machine learning, typically with a twist on resource consumption and how these can be reduced.
The summer school is organised by the collaborative research center SFB 876 and the artificial intelligence group at TU Dortmund University.

Thank you for your contribution!


224: Ellipsis hint to overflowing text

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

@elaineo Done.


[consistency] Document and reduce number of duplicated types

Looks like we have lots of duplicitous types:
MoveResourceType === MoveStructType, though one has docs another doesn’t
There’s a representation of a MoveModuleId and a ModuleId class, and a StructType which is a ModuleId plus a struct name
There’s also a representation of a FunctionId  which uses the StructType
It may be good to consolidate these in some way
Additionally, it would be good to document each accordingly.

Additionally, it would be good to document each accordingly.

Do we want to document them in code?


Suggestion: Can we have an option to just check the current minimum price of an item?

Hey there, can we have an option to just check the current minimum price of an item?
As in, just check on Universalis, and grab the cheapest price of an item on sale on my world?
For example, if Universalis says the cheapest "Weathered Earrings" on my world is 2940, just grab that number and show it to the user?
This feels like what me, as a seller, needs to know the most. Since for most of my random items, I just want to sell them at the lowest price possible.
Thanks!

I did consider this originally...but I didn't add it yet as universalis data is not usually reliable enough for this given it's crowd-sourced. Hopefully this will change as more people contribute.

This is now available on testing.


Set custom label and key

I had some trouble-shootings with my key-value option. My structure differs from your suggested [{key: "", label: ""}] structure. That's why I integrated two new props in order to specify your own label and key of the passed object array:

getOptionKey(value: T): string
getOptionLabel(value: T): string

T is the value of the option array. I'm looking forward to hear your answer.
Cheers
Stefan

Yeah, good idea, it might be handy. Code changes are merged now :)


Not working after update

Hi,
Ive just updated my Raspberry Pi, and Homebridge is no longer working. This is what I get if itty to run it:
pi@raspberrypi:~ $ homebridge
/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/node_modules/mdns/lib/dns_sd.js:35
throw ex;
^
Error: Module version mismatch. Expected 48, got 46.
at Error (native)
at Object.Module._extensions..node (module.js:583:18)
at Module.load (module.js:473:32)
at tryModuleLoad (module.js:432:12)
at Function.Module._load (module.js:424:3)
at Module.require (module.js:483:17)
at require (internal/module.js:20:19)
at Object. (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/node_modules/mdns/lib/dns_sd.js:24:20)
at Module._compile (module.js:556:32)
at Object.Module._extensions..js (module.js:565:10)
Any ideas?

+1  Same issue here on macOS 10.11.6 running NodeJS 6.6.0. Updated earlier today and Homebridge no longer runs. Terminal error below:
/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/node_modules/mdns/lib/dns_sd.js:35
    throw ex;
    ^

Error: Module version mismatch. Expected 48, got 46.
    at Error (native)
    at Object.Module._extensions..node (module.js:583:18)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/node_modules/mdns/lib/dns_sd.js:24:20)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)

Try the following solution:
sudo npm install -g --unsafe-perm homebridge hap-nodejs node-gyp
cd /usr/local/lib/node_modules/homebridge/
sudo npm install --unsafe-perm bignum
cd /usr/local/lib/node_modules/hap-nodejs/node_modules/mdns
sudo node-gyp BUILDTYPE=Release rebuild

Done all that. Still get the same result

Same problem here after updating my raspberry (apt-get dist-upgrade). homebridge refuse to start with error :
/usr/lib/node_modules/hap-nodejs/node_modules/mdns/lib/dns_sd.js:35
throw ex;
^
Error: /usr/lib/arm-linux-gnueabihf/libavahi-client.so.3: symbol __stackchk_guard, version GLIBC_2.4 not defined in file ld-linux-armhf.so.3 with link time reference
at Error (native)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at Object. (/usr/lib/node_modules/hap-nodejs/node_modules/mdns/lib/dns_sd.js:24:20)
at Module._compile (module.js:434:26)
at Object.Module._extensions..js (module.js:452:10)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
I tried to uninstall/reinstall homebridge, but same result. I tried with another version of nodejs (5.19 instead of 6.6.0) and same error.
I also tried to reinstall with
sudo npm install -g --unsafe-perm homebridge hap-nodejs node-gyp
cd /usr/local/lib/node_modules/homebridge/
sudo npm install --unsafe-perm bignum
cd /usr/local/lib/node_modules/hap-nodejs/node_modules/mdns
sudo node-gyp BUILDTYPE=Release rebuild
but also same error.
What else can I try ?

+1, watching. Thanks for any advice.

I had this problem but managed to fix it with the following steps:

uninstalled Homebridge and all installed plugins
reinstalled, renamed the bridge in config.json to something different

Et voila, that was it. All working perfectly.

Updated nodejs installed in /local/use/bin and homebridge in /opt/local/lib/node_modules/homebridge/
Is that part of my problem? Nodejs is not my area of expertise.

Same problem on macOS Sierra.

Same problem here on a Raspberry Pi Zero.

I'm back in order after uninstalling and reinstalling homebridge

None of this worked for me.
Reverting back to the latest lts nodejs did.
sudo apt-get purge nodejs npm
sudo apt-get install -y nodejs
credit

Yeah I figured that's what I had to do as well
Thanks
On Oct 17, 2016, at 2:58 PM, b3nk3 <notifications<EMAIL_ADDRESS>wrote:
None of this worked for me.
Reverting back to the latest lts nodejs did.
sudo apt-get purge nodejs npm
sudo apt-get install -y nodejs
credithttp://askubuntu.com/questions/594656/how-to-install-the-latest-versions-of-nodejs-and-npm-for-ubuntu-14-04-lts

You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/nfarina/homebridge/issues/780#issuecomment-254315538, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALiHO8yL938JFRxr2y_eaZxLPbDClM7aks5q09NGgaJpZM4KBtUO.

Hi im getting this error now. All solutions posted here did not work:
`/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/node_modules/mdns/lib/dns_sd.js:35
throw ex;
^
Error: Module did not self-register.
at Error (native)
at Object.Module._extensions..node (module.js:597:18)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/node_modules/mdns/lib/dns_sd.js:24:20)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)`

@bhaap this is not related to this issue, as it was a version mismatch, hence the solution did not work for you.

Oh sorry. Will create a new issue

@b3nk3 this worked for me too!


spellcheck

Codecov Report

Merging #21 (7ad612c) into main (ca7dec3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   99.62%   99.62%
=======================================
  Files           4        4
  Lines         269      269
=======================================
  Hits          268      268
  Misses          1        1

Files
Coverage Δ

src/BoxCox.jl
100.00% <ø> (ø)


player bullet is too slow

Player bullet is much slower than arcade version.
 See example:
https://youtu.be/MU4psw3ccUI

Yep! I need to do some mame comparisons....

Now made twice as fast. Added extra hit detection to cover increased delta.


[FLAPI-2040] Fix Duffel Payments class method

Add missing self being passed. Instance methods should take a "self" parameter.

Hahah
Ah yes, that's a better solution.


chore(readme): remove unneeded dependency

This closes #13

Since it's my fault the coverage dropped (earlier PR), I'd be more than happy to fix it, however I'm not familiar with testing react components at all. If you could help me with the test setup that would be greatly appreciated.


Resolving the placeholders(with default value) of XML BeanDefinition in the Configuration Class

The previous one #6543 has been closed by @mercyblitz with comments "fixed in 2.7.8". However, this issue still exist in both 2.7.8 & master.
[X ] I have searched the issues of this repository and believe that this is not a duplicate.
[X ] I have checked the FAQ of this repository and believe that this is not a duplicate.
Environment
Dubbo version: 2.7.7 & master
Operating System version: linux
Java version: JDK8
Steps to reproduce this issue
update https://github.com/apache/dubbo/blob/master/dubbo-config/dubbo-config-spring/src/test/resources/org/apache/dubbo/config/spring/demo-provider.xml
from
<dubbo:protocol name="${dubbo.protocol.name}" port="${dubbo.protocol.port}"/>
to
<dubbo:protocol name="${dubbo.protocol.name}" port="${dubbo.protocol.port:8080}"/>
run testProviderXml() in https://github.com/apache/dubbo/blob/master/dubbo-config/dubbo-config-spring/src/test/java/org/apache/dubbo/config/spring/schema/DubboNamespaceHandlerTest.java
Expected Result
UT should pass
Actual Result
UT fail as flow:
java.lang.AssertionError:
Expected: is <20813>
but: was <8080>
Expected :is <20813>
Actual :<8080>
This issue may related with previous issue or change:
#6035
#6079

@shenyyi Please check it agian in dubbo 3.0-preview
Refactored the initialization process of ReferenceBean in Dubbo 3.0, the main purpose is to solve the problem of property injection failure caused by early initialization.
The related pr: https://github.com/apache/dubbo/pull/7383


Object API -- Interaction with maliput::api::RoadGeometry

A recent use case was raised in which object / regions of the Inertial Frame space are defined as polygons and many queries are required to interact with the RoadGeometry. Find them below:
Object definition: it could be defined as a 2D polygon or as a 3D bounding box with a pose in the Inertial Frame.
Queries:

Distance to a Lane center.
Which Lane the object is on?
All Lanes the object collides with.
Given 2 Objects, which is the distance in the Lane Frame? Which is the tracing path between them?

Definition of done

Object class definition with polygon construction support.
ObjectBook class which allows to query objects by their Id.
Implementation of the tracing path in Lane Frame between two objects.
Support for fast spatial queries that allow efficiently solving:

Which Lane the object is on?
All Lanes the object collides with.

Any further discussion can be issued at https://github.com/maliput/maliput_object repo


Invalid Handler Specified

Hi, from the last update I'm not able to use the integration. No datas, as the title I always see that error when I try to add the integration...
Invalid Handler Specified.

Reading the log files I've found this entry:
2024-12-19 15:57:21.288 ERROR (MainThread) [homeassistant.config_entries] Error occurred loading flow for integration fusion_solar: No module named 'homeassistant.core_config'
Seeing commit history I see that from homeassistant.core import HomeAssistant, Config was turned into from homeassistant.core import HomeAssistant from homeassistant.core_config import Config by  Commit b704cb5, "Merge pull request #205 from tijsverkoyen/202-Config-deprecation".
Bringing it back it seems to work properly:

Which version of Home Assistant are you using?

core 2024.9.3

You should update Home Assistant. The current release is 2024.12


feat(sumneko_lua): update to v3.3.1

Updated the build command with the instructions from the source repo. Successfully built and tested locally (OSX arm64).

Awesome! Thank you for the changes!
We can merge this for now once we feel confident with #79 we will update to that later.

Due to my delay on this - I've gone ahead and actually merged #79 to get the latest version out now. We can iterate over this to improve what we need. Thank you for the changes!


MM-40218 - start trial from team edition, do upgrade and request license

Summary
This PR adds changes in the licensing page Start Trial banner, so when in the team edition scenario (no enterprise binary available), the user can directly upgrade the server and start the trial with a single button click.
Ticket Link
https://mattermost.atlassian.net/browse/MM-40218
Screenshots
Normal Upgrade:
https://user-images.githubusercontent.com/10082627/146788666-4a985a6d-bfc8-4104-9cd4-75f5a8f08c8e.mp4
Upgrade and request trial:
https://user-images.githubusercontent.com/10082627/146789344-913a5c94-1646-4fd6-a769-ef7aa02233b6.mp4
Release Note
This change will allow admins to upgrade the server to the enterprise version and request the trial license with a single button click enhancing product adoption by an improved user experience.

@marianunez @nickmisasi @annelieseklein the doNotMerge label is set because I haven't been able to fully test this feature, but I would like to receive early feedback about the code implementation and the design. Thanks!

@annelieseklein Did we confirm with legal that not including the Enterprise Edition license text on the page (but instead having it accessible via a popup modal) is okay?
(The primary use case is GitLab Omnibus which ships with Mattermost Team Edition, but has the option to upgrade to Mattermost Enterprise Edition and start a trial. The Enterprise Edition license text will need to be easily accessible to these admins)

@jasonblais @marianunez confirmed with legal this is fine!

@annelieseklein Great 🙌

@stevemudie to test this, you will need a specific scenario required for doing the upgrade from team to enterprise. Let me know when you are ready to test and I can enable you a server for performing this tests.

@pablovelezvidal the new v6.3 branch is ready and this should be cherry picked there after merge.

/cherry-pick release-6.3

Cherry pick is scheduled.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
From github.com:mattermost/mattermost-webapp
   3e6009b5f..3cc5bb7e6  focalboard-compliance-export -> upstream/focalboard-compliance-export
   dda68425f..3f9d6da4c  master      -> upstream/master
   5221d6deb..6a5ae55c5  release-6.3 -> upstream/release-6.3
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-mattermost-webapp-#9454-upstream-release-6.3-1640129491
Switched to a new branch 'automated-cherry-pick-of-mattermost-webapp-#9454-upstream-release-6.3-1640129491'
Branch 'automated-cherry-pick-of-mattermost-webapp-#9454-upstream-release-6.3-1640129491' set up to track remote branch 'release-6.3' from 'upstream'.

+++ About to attempt cherry pick of PR #9454 with merge commit 3f9d6da4c71783f0fcfa30c9940cf2d931cf8140.

error: could not apply 3f9d6da4c... MM-40218 - start trial from team edition, do upgrade and request license (#9454)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

+++ Conflicts detected:

UU components/admin_console/license_settings/license_settings.tsx
Aborting.

+++ Aborting in-progress git cherry-pick.

+++ Returning you to the master branch and cleaning up.

@pablovelezvidal Seems that this needs a manual cherry-pick to v6.3.

@pablovelezvidal Seems that this needs a manual cherry-pick to v6.3.

ok! on it.

@pablovelezvidal Seems that this needs a manual cherry-pick to v6.3.

done @amyblais https://github.com/mattermost/mattermost-webapp/tree/release-6.3


About --adv and "PGD" in generation part

When I am replicating your code, I found that for the training of generator, if I train incrementally with --rot then --crop then --flip and do poisoning, the ASR is high and stable, but once I continue to train generator with --adv the final result's ASR drops a lot. And I found the log of training generator shows that the Loss_H is high (around 10^3 ~ 10^4 all the time, for all other settings Loss_H is much lower).
So my question is that is --adv necessary? or should I change the default parameters? (I keep all of them except changing --rot --crop --flip for incremental training)
And also I see this:
advdir = "/public/zhangjie/ECCV_2020/backdoor/data_processing_zk/data/cifar10_vgg19_totensor_norm_LR0.01/adv_PGD-8/test"
When I generate the poisoned samples, I cannot find this folder and cannot understand why you use PGD (since I find no clue in the paper), so I remove that part in my experiments.
Is this PGD step important to make ASR high? (And how did you do that? can you provide the script)

First, sorry for late feedback due to the overwhelmed academic duties.
Clarification: incremental learning means using --rot; then --rot --flip, wherein --rot shall be preserved for maintaining robustness for the previous augmentation.
Besides, using adv loss will induce a trade-off between visual quality and the ASR. Thus, you can decrease the weight of adv loss for maintaining ASR.
BTW, PGD is not specific selected and just due to common use for classification tasks. We did not try other options. You can try it yourself.
Last, Thanks for your interest on our work.

does this adv_dir mean the images generated from other attacks compared with it like badnets? as in your "test" folder there are other attacks files exist.

or you mean we have to create this folder  in which images with PGD attack are included??

adv_folder is used for the adaptive defense, which dose not belong to default setting, thus, just ignore it.

ok. thank you for your response


Better icons for JOptionPane

Icons are from FontAwesome.
Error and info colors taken from com.bulenkov.darcula.util.DarculaColors class, the other two are my personal choice.
HiDPI variants provided.
Preview:

Accidentally deleted branch in my fork after merging it into my fork master

Thanks again.


[FEATURE REQUEST] Adding support for all other formats

I'd want to use this plugin with format tscn. Actually it's possible but (in my case) trees doesn't appears in game mods. But in scene mode are visible. So I am stick to .mesh. Will you add more formats to use? It would be really helpful.

A note to self and any others who want .gltf support:
I hear you. Will be in one of the next updates (whenever that would be...)
It's quite a pain on the implementation side, cause the whole point is mesh optimization. It works only if using the "mesh" field because it's literally how it's done on the GPU level. And it doesn't quite work with gltf, since gltf may contain several meshes, in different positions and possibly parented to each other.
This creates a lot of permutations of how to stuff information from gltf into the Multimesh based optimization.
I'll need to figure it out somehow.
If you don't need optimization, you can already use gltf in the "spawned spatial" field to simply draw with scenes (unless there is a file extension restriction, in which cases just make an inherited scene with .tscn extension). But again, no rendering optimization. Just a whole lotta nodes.


Coverage causes test to fail with pexpect and pdb.set_trace.

Hi,
First of all, thanks for the great library!
Describe the bug
I have a test that spawns a child process with pexpect and runs my application. Then, it checks whether some output is displayed and interacts with the program in the child process. The test looks like this.
def test_pdb_interaction_capturing_simple(tmp_path):
    source = """
    import pdb
    def task_1():
        i = 0
        print("hello17")
        pdb.set_trace()
        i == 1
        assert 0
    """
    tmp_path.joinpath("task_module.py").write_text(textwrap.dedent(source))

    child = pexpect.spawn(f"pytask {tmp_path.as_posix()}")
    child.expect(r"task_1\(\)")
    child.expect("i == 1")  # The error happens here.
    ...

You can think of the task function defined in the string as a test function from pytest. Like pytest, my program would also stop at pdb.set_trace and allow the user to debug the function.
When I run the test with the --cov option from pytest-cov and coverage==6.4.1, the test fails after the statement with pdb.set_trace on Ubuntu and macOS. It seems like the content i == 1 is not displayed in the child process, but it is hard to tell since pexpect is hard to debug.
The test succeeds when I remove the --cov option. The test is also OK on Ubuntu and MacOS with coverage<=6.4.0.
To Reproduce
I reproduced and fixed the error by pinning coverage in https://github.com/pytask-dev/pytask/pull/283.

Create a conda environment from environment.yml. (If it helps reproduce the bug, I can add a requirement.txt.)
Run the failing tests with pytest tests/test_debugging.py --cov=src.
The tests will fail on Linux and macOS with Python 3.7-10 and coverage==6.4.1.

Expected behavior
The tests should run fine as before.
Additional context
Since my package is a fork of pytest and very similar regarding this feature, I would not be surprised if they encounter the same issue. I saw some failing jobs in their PRs, which seem to have the same problem: https://github.com/pytest-dev/pytest/runs/6852766697?check_suite_focus=true#step:6:201.

Hmm, this is because of https://github.com/nedbat/coveragepy/commit/4b592fe30a7ebf871ba9a3d883ec955dae198c15#diff-68a483552b5cf650f43e371e28ef86ba13f3958e93bc1587c3cb7afad0397f24R517, which was done to get a slight performance improvement.  I guess there isn't a way to keep it?

CPython has an open issue to set the flag when using bdb:  https://github.com/python/cpython/issues/80675

I guess there isn't a way to keep it?

Maybe this question was rhetorical 😅, but if not, I want to mention that I cannot judge whether the changes justify breaking some code -- probably not many people are affected?. Of course, it would be more convenient for me if you would revert the patch. Still, the alternatives are only minor nuisances one can quickly work around (pinning coverage, verifying it locally, etc.).
Would the changes to CPython be backported to 3.7 and the other versions, or would it only affect 3.10 and 3.11?


Updating the final project a bit

Made a bonus about functions
Added more detail about requirements in the example

Any thoughts about the final project @cansavvy  and @avahoffman ?


Add Typescript definition

This PR contains:

[ ] bugfix
[ ] feature
[ ] refactor
[ ] tests
[ ] documentation
[x] metadata

Breaking Changes?

[ ] yes
[x] no

If yes, please describe the breakage.
Please Describe Your Changes
Added the index.d.ts declaration file for the interface described under API in the README.md
Should resolve issue #94

🎉  thanks!


Asynchronous HTTP client

Are there any plans on implementing an asynchronous HTTP client, as right now the client can be a real bottleneck in a web application if requests aren't moved into the background?

@tbarbugli @tschellenbach ^^^

@kennym right now it should be fairly easy to move add and follow from the request to an async queue (eg. resque)

@tbarbugli yes, that sounds perfect. How would you do that with the Rails integration though? The only way I'd see that would be possible is by moving model creation/deletion into an async queue which is totally infeasible. We need a better way. Any ideas?

or are you talking about an async queue integration within this library? I think we could write an API to support popular ones like resque or sidekiq.

model creation/deletion would stay in sync, it would be the feed manager to
run in the background via resque/sidekiq.
On Thu, Dec 10, 2015 at 9:40 PM Kenny Meyer<EMAIL_ADDRESS>wrote:

or are you talking about an async queue integration within this library? I
think we could write an API to support popular ones like resque or sidekiq.
—
Reply to this email directly or view it on GitHub
https://github.com/GetStream/stream-ruby/issues/35#issuecomment-163743385
.

@tbarbugli ah, yes! I agree with your approach. Already working a PR ;-)


docs(itk-python): example of file format conversion via wasm io packages

Also simplify the native example so it is easy to see the
correspondance.

@neurolabusc please take a look 🙏

@thewtex thanks for your quick work. These new examples work well, and I am now able to open and save meshes in many formats (e.g. obj, stl). So I think the documentation is fine. It does expose an issue with meshes saved in the popular vtk format - it can load and save some meshes in the popular vtk format (e.g. the cow.vtk). However, it fails on any mesh created with 3DSlicer. I tried several different meshes of different formats that loaded fine in slicer and the slicer output vtk meshes would be read fine by other tools (Slicer, NiiVue, Surfice). However, while meshread does not complain opening these vtk format meshes, it fails whenever your try to save it, regardless of the format (iwm.cbor, vtk, obj, stl):
>>> mesh = meshread('gourd.vtk')
>>> meshwrite(mesh, 'gourd2.iwm.cbor')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm_mesh_io/write_mesh.py", line 39, in meshwrite
    return write_mesh(mesh, serialized_mesh, information_only=information_only, use_compression=use_compression)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm_mesh_io/write_mesh.py", line 30, in write_mesh
    func(mesh, serialized_mesh, information_only=information_only, use_compression=use_compression)
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm_mesh_io_wasi/write_mesh.py", line 54, in write_mesh
    could_write = io(mesh, serialized_mesh, information_only=information_only, use_compression=use_compression)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm_mesh_io_wasi/wasm_write_mesh.py", line 82, in wasm_write_mesh
    outputs = _pipeline.run(args, pipeline_outputs, pipeline_inputs)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm/pipeline.py", line 354, in run
    return_code = ri.delayed_start()
                  ^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm/pipeline.py", line 148, in delayed_start
    return_code = func(self._store)
                  ^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/wasmtime/_func.py", line 93, in __call__
    with enter_wasm(store) as trap:
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/contextlib.py", line 144, in __exit__
    next(self.gen)
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/wasmtime/_func.py", line 265, in enter_wasm
    raise trap_obj
wasmtime._trap.Trap: error while executing at wasm backtrace:
    0: 0x515db - <unknown>!<wasm function 669>
    1: 0xa2d99 - <unknown>!<wasm function 1227>
note: using the `WASMTIME_BACKTRACE_DETAILS=1` environment variable may show more debugging information

Caused by:
    wasm trap: wasm `unreachable` instruction executed
>>> meshwrite(mesh, 'gourd2.stl')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm_mesh_io/write_mesh.py", line 39, in meshwrite
    return write_mesh(mesh, serialized_mesh, information_only=information_only, use_compression=use_compression)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm_mesh_io/write_mesh.py", line 30, in write_mesh
    func(mesh, serialized_mesh, information_only=information_only, use_compression=use_compression)
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm_mesh_io_wasi/write_mesh.py", line 54, in write_mesh
    could_write = io(mesh, serialized_mesh, information_only=information_only, use_compression=use_compression)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm_mesh_io_wasi/stl_write_mesh.py", line 82, in stl_write_mesh
    outputs = _pipeline.run(args, pipeline_outputs, pipeline_inputs)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm/pipeline.py", line 354, in run
    return_code = ri.delayed_start()
                  ^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/itkwasm/pipeline.py", line 148, in delayed_start
    return_code = func(self._store)
                  ^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/wasmtime/_func.py", line 93, in __call__
    with enter_wasm(store) as trap:
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/contextlib.py", line 144, in __exit__
    next(self.gen)
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/wasmtime/_func.py", line 265, in enter_wasm
    raise trap_obj
wasmtime._trap.Trap: error while executing at wasm backtrace:
    0: 0x5199c - <unknown>!<wasm function 673>
    1: 0xa6af5 - <unknown>!<wasm function 1253>
note: using the `WASMTIME_BACKTRACE_DETAILS=1` environment variable may show more debugging information

Caused by:
    wasm trap: wasm `unreachable` instruction executed

@neurolabusc, thank you for the review and testing!

However, it fails on any mesh created with 3DSlicer. I tried several different meshes of different formats that loaded fine in slicer and the slicer output vtk meshes would be read fine by other tools (Slicer, NiiVue, Surfice). However, while meshread does not complain opening these vtk format meshes, it fails whenever your try to save it, regardless of the format (iwm.cbor, vtk, obj, stl):

WIP fix for this is here:

https://github.com/InsightSoftwareConsortium/ITK/issues/4858
https://github.com/InsightSoftwareConsortium/ITK-Wasm/pull/1245


Chevron Icon not loading

When running monaco inside vscode the chevron icons are not loaded. If I run the app outside they load without a problem.

monaco-editor version: 0.22.3
Browser: NA
OS: NA
Playground code that reproduces the issue:

I found the problem. The fonts tff for the chevron are not loaded inside vscode. How can I make webpack add those fonts inline, instead of separate files