Reason code should be propagated for TASK_FAILED status_update_event messages

In the event stream, it would be nice to distinguish between different failure modes whenever you receive a TASK_FAILED event. For example, knowing when the task failed due to memory constraints or disk constraints.
Right now the message looks like this:
{
  "slaveId": "20151103-152147-1444348170-5050-25743-S22",
  "taskId": "test.536e8db4-99e6-11e5-8a3e-00163e011756",
  "taskStatus": "TASK_FAILED",
  "message": "",
  "appId": "/test",
  "host": "foobar3",
  "ports": [
    31595,
    31596
  ],
  "version": "2015-12-03T17:47:53.585Z",
  "eventType": "status_update_event",
  "timestamp": "2015-12-03T17:51:08.753Z"
}

but ideally we would have an additional "reason" field with the reasons code in mesos.proto -
{
  "slaveId": "20151103-152147-1444348170-5050-25743-S22",
  "taskId": "test.536e8db4-99e6-11e5-8a3e-00163e011756",
  "taskStatus": "TASK_FAILED",
  "message": "",
  "reason": "REASON_CONTAINER_LIMITATION_MEMORY",
  "appId": "/test",
  "host": "foobar3",
  "ports": [
    31595,
    31596
  ],
  "version": "2015-12-03T17:47:53.585Z",
  "eventType": "status_update_event",
  "timestamp": "2015-12-03T17:51:08.753Z"
}

Related: #1306

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2913. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2913. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Avoid the generic and unused namespace Open.

It is a bit annoying to work with IDEs when you type using Op<TAB> it gives you using Open. which is not helpful. You always want using Open.Nat. When using it in @OpenRA it is also conflicting a lot. I suggest to go with OpenNat instead.

I understand what you say. The point is that change will break all the clients and we don't know what projects are using open.nat. Another fact is that I use 'Open' for all my projects (for those that I really care) so, @OpenRA will use alway Open.Nat but other projects could potencially use Open.Nat, Open.WinKeyboardHook or Open.HttpProxy.
I appreciate his kind of recomendations from those, like you, that have real requirements for real world projects but I cannot change the namespace even when it could be useful for some projects.

I agree, it is probably too late already.


Only works when name not followed by a dot

I've got letsdane working (it's great, thanks @buffrr!) and it normally works fine. But when the name has a trailing dot, it doesn't.
Not a major problem, was just curious why this happens. (https://google.com. redirects normally)
Without a trailing dot:

With a trailing dot:

The log:

Thank you for reporting this issue. The Server Name Indication (SNI) must not include a trailing dot per RFC 6066 Section 3. This is unfortunately an issue with the current way handshake TLDs are being used. So when using HTTPS you should remove it.
The behavior of the trailing dot in SNI varies  you can try for example https://cloudflare.com. or https://buffrr.dev. without using any proxy and they will either fail in some browsers or show that the certificate is invalid because the name in the certificate does not include a trailing dot.
The Go standard TLS library follows RFC6066 and does not allow a trailing dot in SNI so unfortunately it's not possible to allow the usage of this (non-standard) behavior via letsdane

Ah, good to know. Thanks again :)


Dockstore CLI Should Let User Specify Cromwell Version

Feature Request
Dockstore CLI always runs the version of Cromwell specified at io.dockstore.client.cli.nested.AbstractEntryClient#CROMWELL_LOCATION
Dockstore should use that value as the default, but should optionally allow the user to specify a different version. Using the latest or an earlier version of Cromwell would then no longer require a new Dockstore CLI build.
While Dockstore CLI has no compile time dependencies on Cromwell, there is still the risk a different version of Cromwell may not work from Dockstore, as, for example, the command-line arguments might change between versions. However, the users could use this feature at their own risk.

The same strategy can be used for Bunny

Configurable in config
$ cat ~/.dockstore/config
token: <censored>
#server-url: http://<IP_ADDRESS>:8080
server-url: https://dockstore.org:8443
use-cache = true
cwlrunner = cwl-runner
bunny-version = 1.0.4-4
cromwell-version = 30.2
$ dockstore tool launch --local-entry Dockstore.wdl --json test.wdl.json
Running with Cromwell 30.2 , Dockstore tests with 29
Creating directories for run of Dockstore launcher in current working directory: /home/dyuen/dockstore_tools/dockstore-tool-md5sum
Provisioning your input files to your local machine
Downloading: ga4ghMd5.inputFile from md5sum.input to: /home/dyuen/dockstore_tools/dockstore-tool-md5sum/cromwell-input/2ffde7e9-98e6-4a17-b19c-0878e6b45025/md5sum.input
Caching file /home/dyuen/dockstore_tools/dockstore-tool-md5sum/cromwell-input/2ffde7e9-98e6-4a17-b19c-0878e6b45025/md5sum.input in cache, hard-linking
Calling out to Cromwell to run your workflow
java -jar /home/dyuen/.dockstore/libraries/cromwell-30.2.jar run /home/dyuen/dockstore_tools/dockstore-tool-md5sum/Dockstore.wdl --inputs /tmp/foo8514513664046447635json

Confirmed to work on staging 1.4.0-beta.3

Confirmed to work on staging 1.4.0-beta.3


Exclude specific days of week from datepicker

Is it possible to use the excludeDates fields to exclude every Sunday from the datepicker?

Use the filterDate prop; see https://hacker0x01.github.io/react-datepicker/#example-11


SSLRead error -9806

when i install brew,error as follow.how i can fix this error?
fatal: unable to access 'https://github.com/Homebrew/brew/': SSLRead() return error -9806
Failed during: git fetch origin master:refs/remotes/origin/master --tags --force --depth=1

You're having issues connection to GitHub. We can't do anything about that, I'm afraid. I'd suggest rerunning the installer (and uninstalling in between if necessary) multiple times until it installs 🆗.


Test a New Template - Documentation

All Submissions:

[YES] Have you followed the guidelines in our Contributing document?
[YES] Have you added an explanation of what your changes do and why you'd like us to include them?
[YES] I have updated the documentation accordingly.
[NA] I have added tests to cover my changes.
[YES] All new and existing tests passed.
[NA] My code follows the code style of this project.
[NA] I ran lint checks locally prior to submission.
[YES] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

What is the current behavior?

Issue Number: #352 (Template Testing)
What is the new behavior?

These docs will walk a user through the steps it takes to create a unit and integration test from scratch using the Cobalt provided test-harness that extends Terratest (A Terraform testing framework writtin in Golang). The walkthrough focuses on automating the Cobalt dev workflow and making automated test a part of that.

Does this introduce a breaking change?

[NO]

Any relevant logs, error output, etc?

(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)
Other information

@TechnicallyWilliams Do we have an estimate when this PR will be merged?


[Dialog/Select] Selection mode is active after choosing an item in Select in Firefox

Bug report
Current Behavior
Following this discussion https://github.com/radix-ui/primitives/discussions/1484#discussioncomment-2993174.
Using Dialog 0.1.8-rc.47 and Select 0.1.2-rc.49 in Firefox 101.0.1, after choosing an item in Select, selection mode is activated and text content in Dialog is highlighted on hover.

Expected behavior
After choosing an item in Select component, text in Dialog is not highlighted on hover.
Reproducible example
CodeSandbox Template
Choose option 1 from the select and hover on the text Very basic modal, it will be on selection mode and highlight the text.

Seeing the same behavior with Dropdown after close

After being puzzled by this for quite a while, I realized that the example implementation adds a style to the SelectItem, which my project was missing. This fixed the bug for me.
 .select__item {     user-select: none;   }

I can confirm the solution provided by @llStaraill fixes the issue with Firefox. @andy-hook perhaps we should consider this a functional style (especially given native menus don't allow you to select item text anyway and the action is on pointerup).

perhaps we should consider this a functional style (especially given native menus don't allow you to select item text anyway and the action is on pointerup).

I agree

+1

I tried every proposed solution but my dropdown menu item still triggers a click underneath.


Update readme-facebook.md

It's not node facebook_bot.js

What are you referring to here? The included sample is definitely called facebook_bot.js not facebook-bot.js


Making TwistReceived::ProcessMessage virtual

Use case : I want to write a specific ProcessMessage when a twist is received,  for example in my application, to modify the velocity of the four rotors of a quadrotor.
I therefore thought about using a TwistReceiver  but what is done when a twist is received is actually hard coded in the twist receiver.  Therefore, there is a need to modify what is done by the TwistReceiver::ProcessMessage method.
I see two possibilities :

making TwistReceiver   an abstract class   with ProcessMessage a virtual method  ; And then writting derived classes with their specific ProcessMessage
(the one implemented in this pull request)   Keep your "default" ProcessMessage   but add the virtual keywords to allow overriding it.

Also in the new Message Handling Framework the intended way to implement new Subscribers/Publishers is to directly derive from the Subscriber and Publisher base classes and not to derive further from any derived classes.
Explicitly allowing this will restrict ourselfes too much in the development of our Publishers and Subscribers as any change will have an impact on their children.
See also Message Handling Code Map.
Nevertheless thanks again for this suggestion @jeremyfix and please feel free to setup more complex inheritance structures in your ROS# projects. You might convince me about them in the future.


⚠ Doshipui has degraded performance

In 35a6ec4, Doshipui (https://дошипуй.рф/) experienced degraded performance:

HTTP code: 200
Response time: 11672 ms

Resolved: Doshipui performance has improved in 1c126b0 after 23 minutes.


[BUG] 3.1.2 Beta 3/4 - Crash on ARM64 when run without elevation. (FAST_FAIL_GUARD_ICALL_CHECK_FAILURE)

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions.
[X] This issue is about a bug (if it is not, please use the correct template).

UniGetUI Version
3.1.2 beta 3 & 4 (not tested 1 & 2)
Windows version, edition, and architecture
Windows 11 - ARM64 (Runs x64 arch).
Describe your issue
Starting 3.1.2 beta x as Admin user, but not elevated crashes UniGetUI on ARM64 with FAST_FAIL_GUARD_ICALL_CHECK_FAILURE.
Previous 3.1.1. version did not have this behavior, reproduces over 2 different machines.
Steps to reproduce the issue
Take an ARM64 Win11 machine and upgrade to the 3.2.1. beta train, try to start it without using "Run as Administrator" if you keep an eye on the process (e.g. process explorer) you will notice werfault kicking in as the process is crashing.
(3a28.1024): Security check failure or stack buffer overrun - code c0000409 (first/second chance not available)
Subcode: 0xa FAST_FAIL_GUARD_ICALL_CHECK_FAILURE
ntdll!RtlFailFast2:
00007fff`a3875930 d43e0060 brk         #0xF003

CONTEXT:  (.ecxr)
 x0=000000000000000a   x1=00007fff58a09140   x2=0000000000000000   x3=0000000000000000
 x4=000000574ab8dd68   x5=0000000000000000   x6=0000000000000000   x7=0000000000000000
 x8=0000010000000010   x9=00007fffa38694a8  x10=00007fffa39b3170  x11=00007fffa3875930
x12=0000000000000000  x13=0000000000000000  x14=0000000000000000  x15=000000000000014d
x16=0000000000000007  x17=00000000000000ff  x18=000000574adab000  x19=00007fff58a09140
x20=0000000000000000  x21=0000000000000000  x22=0000000000000000  x23=5555555555555555
x24=000001bcaa31db00  x25=000000574ab8e2d0  x26=000000574ab8e310  x27=000001bca9ee1af0
x28=000000574ab8c740   fp=000000574ab8dbc0   lr=00007fffa38d90d8   sp=000000574ab8dbc0
 pc=00007fffa3875930  psr=00000000 ---- EL0

EXCEPTION_RECORD:  (.exr -1)
ExceptionAddress: 00007fffa3875930 (ntdll!RtlFailFast2)
   ExceptionCode: c0000409 (Security check failure or stack buffer overrun)
  ExceptionFlags: 00000001
NumberParameters: 1
   Parameter[0]:<PHONE_NUMBER>00000a
Subcode: 0xa FAST_FAIL_GUARD_ICALL_CHECK_FAILURE

FAILURE_BUCKET_ID:  FAIL_FAST_GUARD_ICALL_CHECK_FAILURE_NOSOS_c0000409_Microsoft.Internal.FrameworkUdk.System.dll!winrt::impl::capture_to_IWpnPlatform,int_

UniGetUI Log
N/A as crash

Package Managers Logs
N/A as crash

Relevant information
No response
Screenshots and videos
No response

Please test 3.1.1, 3.1.2 beta 1 and 3.1.2 beta 2.
There have been internal library changes that could lead to this issue happening.

3.1.1 works as 3.1.2 beta 1 it starts crashing.

I will investigate

Yes, confirmed


Doesn`t load Sortable.min.js

See my comment https://github.com/kotchuprik/yii2-sortable-widgets/commit/fc77dafe4a04997df0505d54bbaccc83bd0bb446#r51330570

@kratkar hey, thanks. do you have time to do a fork and send me a pull request?

@kotchuprik Hi! Unfortunately, no. I didn't configured github after reinstalling OS.

@kotchuprik Fixed in https://github.com/kotchuprik/yii2-sortable-widgets/pull/29

thank you guys. just released https://github.com/kotchuprik/yii2-sortable-widgets/releases/tag/v2.2.0


Login hint is not working properly

I'm experiencing an odd behavior when dealing with a user that has multiple accounts. It seems the oauth login dialog is remembering the last user that logged in from that location, regardless of the login_hint received.
Here's how you can reproduce it:

Log with an account A in google. Make sure you have only logged the account A (i.e.: try it on incognito mode).
Log out account A.
Go to your google sign-in web implementation. In the oAuth URL complete login_hint param with account B (our wanted account).
Make the request and you will see in the google login form pop-up the Account A completed, prepared to fill the password.

I thought that account B should popup as I was using the login_hint parameter. Am I doing something wrong? Is there anything else needed to make the form start with User B regardless of any previous login history on that session?

HI @cincue,
What library are you using? GSI v1 (gapi.auth) or v2 (gapi.auth2)?

Hi @TMSCH thanks for the reply. I am using gapi.auth2.

Could you provide a snippet of code to reproduce this behavior? I was not able to reproduce the bug using GoogleAuth.signIn().

startApp();

var googleUser = {};
function startApp(){
  gapi.load('auth2', function(){
    auth2 = gapi.auth2.init({
      client_id: 'xxxxxxxx.apps.googleusercontent.com',
      scope: 'email',
      fetch_basic_profile: false,
      login_hint: '${googleLoginHint}'
   });
   attachSignin(document.getElementById('google-signin-button'));
  });
}

function attachSignin(element){
  auth2.attachClickHandler(element, {},
    function(googleUser){
      document.getElementById('id_token').value = googleUser.getAuthResponse().id_token;
      document.getElementById('form').submit();
    },
    function(error) {
      logFailure();
    }
  );
}

Code example from the documentation page.

I am not able to reproduce this issue with your code... What are you passing as login hint? An email address?

Yes, I am passing an email address.
This should happen in the same browser session. Something that I see is that when I logout account A, I get redirected to a login screen where it's already showing me the last logged user (Account A), and asking directly for a password (and that's exactly the same screen I see when attempting to log account B).
Could it be that remembering the last user logged in when there's only one user session takes precedence over the login hint?

Hi @cincue, I think I don't understand properly the method to reproduce the issue, as I'm not able to. When you say "log out" from the account, do you mean signOut() using the library or logging the user out from https://accounts.google.com?

I mean logging the user out from https://accounts.google.com
In short:
1- Log user A in https://accounts.google.com
2- Log out user A from https://accounts.google.com
3- signIn() with login_hint = user B (or use my code above instead)
Make sure you reproduce it in the same session (start a new session, try it on incognito mode)

Thanks for the further information! I was finally able to reproduce the issue, I was doing something wrong before. I'll investigate the issue and come back to you as soon as I have more info!

Great! @TMSCH I hope it can be fixed, let me know any update. Thanks!

Hi @cincue! This issue has been fixed and released. Let me know if that works for you!

Fantastic! @TMSCH It works perfect now! Thanks

Glad it works! Closing this issue.


Where to find the model for the finetune notebook and can we use other models from huggingface spaces?

For the finetuning notebook:

I don't find from where I need to download jmann-small-190k.ckpt from and
as jmann-small-190k.ckpt gives poor quality outputs on hugging face spaces I was curious whether I could use jmann-large-580k directly through the same code. (assuming it's the .bin  file).

Thank you so much for releasing this research effort online!

They are ckpt files
generally the format is: https://model-server.zqevans2.workers.dev/ABBREVIATION-STEPSk.ckpt
like https://model-server.zqevans2.workers.dev/gwf-440k.ckpt
yes you can use the other ckpt files for training. Including ones from larger sized models using the same architecture

They are ckpt files generally the format is: https://model-server.zqevans2.workers.dev/ABBREVIATION-STEPSk.ckpt
like https://model-server.zqevans2.workers.dev/gwf-440k.ckpt
yes you can use the other ckpt files for training. Including ones from larger sized models using the same architecture

So we can train our own ckpt files? I wanted to know if it's possible to share the training file too, not just finetuning
And what specs of machine did you have to train on in order to get the results you did?

Me? I trained on colab free, kaggle and one day on an a100 for the really huge sets.

Me? I trained on colab free, kaggle and one day on an a100 for the really huge sets.

Is there a colab file with the training file (not the fine tuning one) that was used to create the models for the glitch etc model? Or will that not be released?

You would have to ask Zack.
Basially they started from scratch AFAIK and just trained a bunch of glitchy noises.  Not sure what you are after.


s/licence/license/g

I'm assuming the appearances of "LICENCE" are a typo b/c of the actual root folder filename and other occurrences of "license" - fixed all here.

LGTM


shellenv missing - through packagecontrol.io

reloading plugin GoGuru.goGuru
Traceback (most recent call last):
  File "/Applications/Sublime Text.app/Contents/MacOS/sublime_plugin.py", line 120, in reload_plugin
    m.plugin_loaded()
  File "goGuru in /Users/uldericofilho/Library/Application Support/Sublime Text 3/Installed Packages/GoGuru.sublime-package", line 57, in plugin_loaded
  File "goGuru in /Users/uldericofilho/Library/Application Support/Sublime Text 3/Installed Packages/GoGuru.sublime-package", line 43, in load_shellenv
ImportError: No module named 'shellenv'

fixed ! thanks for reporting :)
https://github.com/alvarolm/GoGuru/issues/11

it should be fixed as version 0.1.4, the package control uses 1 hour interval to pull updates from the git repo, so we just have to wait, I have tested anyway on my machine and seems to be fixed.

please feel free to close this one as soon its working for you.


Can't install through Composer

I am trying to install this into a fresh laravel 5 install and am getting this:
vagrant@homestead:~/Code/grow$ sudo composer require intervention/image
Using version ^2.3@dev for intervention/image
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for guzzlehttp/psr7 == <IP_ADDRESS> -> satisfiable by guzzlehttp/psr7[1.0.0].
- intervention/image 2.3.0 requires guzzlehttp/psr7 ~1.1 -> satisfiable by guzzlehttp/psr7[1.1.0].
- intervention/image 2.3.1 requires guzzlehttp/psr7 ~1.1 -> satisfiable by guzzlehttp/psr7[1.1.0].
- intervention/image 2.3.x-dev requires guzzlehttp/psr7 ~1.1 -> satisfiable by guzzlehttp/psr7[1.1.0].
- Conclusion: don't install guzzlehttp/psr7 1.1.0
- Installation request for intervention/image ^2.3@dev -> satisfiable by intervention/image[2.3.0, 2.3.1, 2.3.x-dev].

Hello,
try maybe "intervention/image": "2.3.*"
in composer.json?

First, you need to instal guzzlehttp/psr7.
Second, go to your php.ini file and uncomment this line :
;extension=php_fileinfo.dll      To    extension=php_fileinfo.dll

;extension=php_fileinfo.dll To extension=php_fileinfo.dll
it's worked thanks

ok


[FEATURE REQUEST] Use the pacakge without API Key (bypass Google)

Is this a bug report?
No.
Have you read the Installation Instructions?
Yes.
Environment
"react-native": "0.48.3",
"react": "16.0.0-alpha.12",
"react-native-maps": "^0.20.1",

Steps to Reproduce

Set up a minimal react-native project, follow the installation instruction but skip adding a Google Maps API Key in the android/app/src/main/AndroidManifest.xml

<application>
    <meta-data
    android:name="com.google.android.geo.API_KEY"
    android:value="Your Google maps API Key Here"/>
</application>

Use this https://github.com/react-community/react-native-maps/blob/master/example/examples/CustomTilesLocal.js

or

this code https://github.com/react-community/react-native-maps/blob/master/example/examples/CustomTiles.js

Expected Behavior
When I use <LocalTile /> or <UrlTile /> it should work without supplying a Google Maps API Key because I am not using a Google Maps WMS.
Actual Behavior
The app crashes on start up ans asks for a Google Maps API Key.
Reproducible Demo
Simply remove your Google Maps API Key in any react-native Project and use this code https://github.com/react-community/react-native-maps/blob/master/example/examples/CustomTilesLocal.js instead of your <MapView />.
Feature Request
I am wondering why I need an GoogleMaps API Key on Android, eventhough I am not using Google as a provider, but <LocalTile /> or <UrlTile />?!
That just does not make a lot of sense to me... Why would I need a Google Maps API key when using tiles that are stored locally on the device (<LocalTile>)? Why would I need a Google Maps API key when I am using i.e. MapBox tiles (<UrlTile>)? That limits what can be done with react-native-maps because it puts restrictions on the App that are linked to what can be done for free/paid with the Google Maps API Key.
Can anyone explain why that is? That should not be. It would be better to leave the API Key optional and allow developers to bypass Google.

@Stophface would have been nice if you could respect the issue template, even if it's a question :(

@rborn I added the issue template

Unfortunately it is not possible using GooglePlay libs.
I am working on this on a fork, implementing a (still experimental) new provider using osmdroid (https://github.com/osmdroid/osmdroid). You can check this out on https://github.com/fqborges/react-native-maps.
Caution! It is still a work in progress, some things are not implemented and bugs are expected.

Going to close this, @fqborges thnx for replying

@fqborges hey! How is your fork? Did you succeed with extracting provider?

@Peretz30 I am using it on production and it is working as expected. You can find more info and instructions here: https://github.com/react-community/react-native-maps/pull/2348

Any possibility to use this with expo ?


Update test dependencies

This pull request updates various test dependencies (flake8, pylint, astroid, mock).
New versions of flake8 and pylint detect some new issues (and there are also a couple of false positives) so this PR fixes that as well.

Codecov Report

Merging #1288 into trunk will increase coverage by <.01%.
The diff coverage is 52.32%.

@@            Coverage Diff             @@
##            trunk    #1288      +/-   ##
==========================================
+ Coverage   85.95%   85.95%   +<.01%
==========================================
  Files         359      359
  Lines       73788    73790       +2
  Branches     6693     6693
==========================================
+ Hits        63423    63427       +4
  Misses       7689     7689
+ Partials     2676     2674       -2

Impacted Files
Coverage Δ

libcloud/utils/iso8601.py
86.36% <ø> (ø)
:arrow_up:

libcloud/storage/drivers/s3.py
89.08% <ø> (ø)
:arrow_up:

libcloud/common/cloudstack.py
86.59% <ø> (ø)
:arrow_up:

libcloud/storage/drivers/dummy.py
50.41% <ø> (ø)
:arrow_up:

libcloud/common/digitalocean.py
94.59% <ø> (ø)
:arrow_up:

libcloud/common/nttcis.py
73.79% <ø> (ø)
:arrow_up:

libcloud/compute/deployment.py
97.22% <ø> (ø)
:arrow_up:

libcloud/__init__.py
71.42% <ø> (ø)
:arrow_up:

libcloud/compute/drivers/voxel.py
87.3% <ø> (ø)
:arrow_up:

libcloud/common/azure_arm.py
73.01% <ø> (ø)
:arrow_up:

... and 49 more

Continue to review full report at Codecov.

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


sbctl need replaced with libovsdb

Feature request

Use case
kube-ovn-controller has some sbctl get find set operation， I will try to replace them with libovsdb。

If use libovsdb, I think the node tag in external_ids is redundant, no need to filter by externa_ids node tag anymore.
hi @oilbeater , how do you think about this point?

complete


Add box on MobaXterm for R Markdown setup for Windows

For the R Markdown setup, we might want to add a box for Windows users who have issues with the local X server setup, explaining how to get it to work with MobaXterm (see HackMD from last course).

Fixed in recent PR.


Can't install the package.

At first, I installed the package, it worked fine, created a demo template to try it out, and after some exploration, deleted those files as now I wanted to try it for real. But when I tried to instantiate a new Template, it successfully completed all the steps, except the last one i.e. Running Posthooks. It gave the following error, trying to relocate the deleted files:

After trying a couple of times, I thought maybe reinstalling the package would fix this issue. I was able to successfully remove the package, but when I tried reinstalling it, the same error popped up, and I've no idea how to fix this:

Here's my new Template:
t = Template(;
           dir=".",
           julia=v"1.4",
           plugins=[
               Git(; manifest=true),
               GitHubActions(; x86=true),
               Codecov(),
               Documenter{GitHubActions}(),
               Develop(),
           ],
       )

t("NetMSA")

Can someone guide me?

Nevermind, figured it out. Apparently, instead of manually deleting the directory, I had to remove the package using the remove command in REPL. I didn't know that it added the package into the repository as well.


[5.2] Allows app:name to replace namespace in tests/ folder

Required by https://github.com/laravel/laravel/pull/3560

Please could you send to the new 5.2 branch. ;)

Sure. Done


Nekogram themed icon support

App name: Nekogram
Package name: tw.nekomimi.nekogram

nekogram  It is working now

Hello! Weʼre switching to a new system for icon requests in order to be able to better manage them. From now on, icon requests can be submitted here: https://forms.gle/Fx8vZAiWdW1Tyjo57. If your icon request hasnʼt yet been fulfiled, it has been copied to our new icon request database, meaning no action is required. Consequently, this issue will be closed.


crypto/tls: create certs w/o KeyEncipherment KU for non-RSA keys in generate_cert.go

Summary:
The crypto/tls/generate_cert.go utility should only set the template
x509.Certificate's KeyUsage field to a value with the
x509.KeyUsageKeyEncipherment bits set when the certificate subject
public key is an RSA public key, not an ECDSA or ED25519 public key.
Background:
RFC 5480 describes the usage of ECDSA elliptic curve subject keys with
X509. Unfortunately while Section 3 "Key Usages Bits" indicates which
key usage bits MAY be used with a certificate that indicates
id-ecPublicKey in the SubjectPublicKeyInfo field it doesn't provide
guidance on which usages should not be included (e.g. the
keyEncipherment bit, which is particular to RSA key exchange). The same
problem is present in RFC 8410 Section 5 describing Key Usage Bits for
ED25519 elliptic curve subject keys.
There's an update to RFC 5480 in last call stage within the IETF LAMPS
WG, draft-ietf-lamps-5480-ku-clarifications-00. This update is meant
to clarify the allowed Key Usages extension values for certificates with
ECDSA subject public keys by adding:

If the keyUsage extension is present in a certificate that indicates
id-ecPublicKey as algorithm of AlgorithmIdentifier [RFC2986] in
SubjectPublicKeyInfo, then following values MUST NOT be present:
keyEncipherment; and
dataEncipherment.

I don't believe there is an update for RFC 8410 in the works but I suspect it
will be clarified similarly in the future.
This commit updates generate_cert.go to ensure when the certificate public
key is ECDSA or ED25519 the generated certificate has the
x509.Certificate.KeyUsage field set to a value that doesn't include KU's
specific to RSA. For ECDSA keys this will adhere to the updated RFC 5480
language.
Fixes https://github.com/golang/go/issues/36499

Quick note: This is my first Go PR. I've tried to adhere to the guidance in the CONTRIBUTING.md but would appreciate some extra patience from maintainers if I've made any mistakes


Multiple Application doesn't work

Hey,
when i use a single cert like this:
PUSH_NOTIFICATIONS_SETTINGS = {
	"APNS_CERTIFICATE": os.path.join(BASE_DIR, "y.pem"),
	"APNS_TOPIC": "com.x.y",

the push work properly but i to have two application with this django site, so i change my setting to:
PUSH_NOTIFICATIONS_SETTINGS = {
	# Load and process all PUSH_NOTIFICATIONS_SETTINGS using the AppConfig manager.
	"CONFIG": "push_notifications.conf.AppConfig",

	# collection of all defined applications
	"APPLICATIONS": {
		"pro": {
			# PLATFORM (required) determines what additional settings are required.
			"PLATFORM": "APNS",

			# required APNS setting
			"CERTIFICATE": os.path.join(BASE_DIR, "y.pem"),

			"TOPIC": "com.x.y",
		},
		"lite": {
			# PLATFORM (required) determines what additional settings are required.
			"PLATFORM": "APNS",

			# required APNS setting
			"CERTIFICATE": os.path.join(BASE_DIR, "z.pem"),

			"TOPIC": "com.x.z",
		}
	}
}

when i test it, i get BadDeviceToken but each of application work with single mode.. i set application_id properly and have a lot of test. what can i do?

FIXED!!!
i just add python3 "USE_SANDBOX": True, in every application...
Tnx


Hard bound methods when creating new observer

Bug Report
Current Behavior
The follow code fails because the methods on the subscriber are not hard bound to it.
Reproduction

stackBlitz

const mySpecialTimer = new Observable(({ next, complete }) => {
  let count = 1;
  const rinse = () => setTimeout(() => {
    next(++count);
    if (count < 15) {
      return rinse();
    }
    complete();
  }, 500);
  rinse();
  return () => console.log('done');
}).pipe(take(5)).subscribe(console.log);

Expected behavior
That the code works as expected.  I'm not sure if this is a bug or a feature request thought.
Possible Solution
Hard bound methods on observable

@cartant If possible, yes, as it leads to less unexpected behaviours. I came to this after I ran into trouble doing something like this when creating an observable
    return new Observable<any>(obs => {
      this.sock.on(eventName, obs.next);
      this.sock.on('disconnect',obs.complete)
      return () => {
        console.log('socket cleaned');
      };
    });

I know the workaround, but as I like functional programming, and would prefer things like this to work.
Also, doing it like this is similar on how promises are created.

Yeah, but with promises, the resolve and reject functions are passed as separate parameters.
I'm not sure I agree with the effected behaviour being "unexpected", as the methods have to be explicitly extracted using destructuring - or whatever.
Also, binding the methods to the instance would incur a performance penalty: next, error and complete (and perhaps more, as the API explicitly states that it's a Subscriber and not an Observer that's passed) would all need to be bound - whether used or not. And for no reason other than a stylistic convenience.
Maybe there are some implementation changes in the experimental branch that might be - or could be - more inline with what you are looking for, but I'm only just starting to familiarise myself with the work that's happening there. However, my understanding is that subscribers are still going to be "a thing", so there will still be parts of the API that deal with objects, rather than just functions.

I agree with @cartant here in general, this is neither bug nor unexpected behavior. Subscriber never explicitly said to be context binded and it was not a design goal. It is still possible eventually we could have it, but we don't actively pursue as goal compare to other design decisions core trying to achieve at this moment.

About unexpected, #3229 is an example. But I'm not going to pressure here.
As I said, not sure if this was a bug or a feature request.
All I'm saying is that the developer ergonomics of the subscriber usage would improve when this is implemented. It will enable a more functional way of programming.  With the JS destructuring this will enable a more clean way to create observables.
another example:
const documentClicks$ = new Observable(({ next }) =>
  document.addEventListener('click', next)
);

I think that is a nice an concise style of programming.  (I know about fromEvent, this is a sample on style, not on functionality)

@SanderElias Sure, but at some point it will be necessary to accept that the API involves subscriptions and subscribers and that they are object instances. After all, this will fail:
const source = of(1);
const { unsubscribe } = source.subscribe();
unsubscribe();

Also, new Observable is also a low-level API - as you've noted, users are encouraged to create observables using factory functions, like fromEvent, wherever possible - and binding the methods would add overhead. I've more than a few operator implementations that use new Observable and simply pass the subscriber to an inner obervable's subscribe.

hmm, A factory function would do it alright.  I understand that the (although small) overhead adds up, and will be a chore on creating operators.
It would be cool if such a factory would be part of rxjs. If not, I will create one for myself.
Thanks for the explanation.


MSID: 99711 Version: 1 DOI: 10.1101/2<IP_ADDRESS>5543

MSID: 99711
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>5543
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=99711
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>5543

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/99711v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>5543
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/99711/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Reviews have been posted to the v1 but a v2 was posted shortly afterwards to correct a typo in an author name, which has meant there's only a MECA package for the v2. Moving the reviews over to the v2 and loading the v2 MECA.


Expected release data of v3.2.0

Hi, thank you for providing an amazing editor.
Can you tell me what is the expected date of v3.2.0
I am waiting for https://github.com/nhn/tui.editor/issues/1803
Thanks again :)

@ats1999
There's a lot of other work to do in next minor version. So it's hard to say exactly, and the rough goal is to release it within January next year. Thank you.

Thanks for letting me know :)


The integrated terminal cannot find certain files (commands)

VSCode Version: 1.19.0
OS Version: linux-mint 4.10.0-38-generic

Steps to Reproduce:

cd /usr/bin
ls go

Within integrated terminal:
rakesh@linux-mint /usr/bin $ ls go
ls: cannot access 'go': No such file or directory
rakesh@linux-mint /usr/bin $ ls -ld .
drwxr-xr-x 2 nfsnobody nfsnobody 36864 Dec 31  1969 .
rakesh@linux-mint /usr/bin $

in an external shell:
rakesh@linux-mint /usr/bin $ ls go
go
rakesh@linux-mint /usr/bin $ ls -ld .
drwxr-xr-x 2 root root 69632 Dec 19 21:50 .
rakesh@linux-mint /usr/bin $

Wonder why the id's are diff for the /usr/bin directory under the integrated terminal.

Reproduces without extensions: Yes/No

Do you know what nfsnobody is? My guess is this is something to do with users/permissions and the fact that the VS Code terminal is inheriting the environment from VS Code.

the nfsnobody is the group-id 65534 and actually it is listed as nogroup within the /etc/group file. But I am surprised as to why the integrated terminal can not find the files within the directory when it has the right permissions?
Linux mint installation shows the programs under the startup menu and I execute from there rather than from shell. Do let me know what other information can I provide and would be glad to do so.

@rakesh-k does the same thing happen when you launch from a terminal?

@rakesh-k I had the same problem when I installed vscode from the Software Manager. Instead I used the independent deb file to install and the problem went away.

See https://github.com/Microsoft/vscode/issues/44646, the problem is with the unofficial flathub package. Using the official deb instead as @ishunyu suggests is the fix.


Add  web component for HTTPS Callable Functions

Add <firebase-functions> web component to call HTTPS Callable Functions (https://firebase.google.com/docs/functions/callable) directly from a Firebase app in declarative way.

@tjmonsi @mbleigh @merlinnot PTAL.


Prevent window scrolling in terminal

I'm using MacOS, whenever I scroll in a terminal, the screen scrolls with it. Apologies if this has been answered elsewhere.

As you can see the previous buffer is showing when I scroll up.
NCurses manages to prevent this so I assume it's possible, what am I doing wrong?
Thanks :)

I don't have a MacOSX machine to test it, but if you enter raw mode, this shouldn't happen.
Some scroll movement codes might not be supported (which is a bug) but even then they's come out as Event::Unsupported.
Can you offer a minimal example of the behavior?

Well, none of the examples in termion lock the scroll buffer in any terminal (iTerm, terminal) on any shell I tried (bash, fish, zsh) on MacOS, except on alacritty worked with any shell.
I tried the TUI examples (with termion and rustbox backends) and neither successfully locked the scrolling. I tried my own code and no matter what I try, it won't work.
For reference, all of the golang libs I tried worked fine. ncurses works fine. even python libs and so on I've tried on this exact setup work fine.

Hello, I have the same issue using the gnome-terminal (others not tested). I tried to find how to "lock the scrollbar" as for example vim is doing. But after searching the web for several hours and examining the vim source code I had no success. When using vim, the scrollbar of gnome-terminal is "disabled". Can anybody point me to the right direction what must be implemented to archive this in termion?

It is now fixed, you need to update.

You can try the AlternateScreen: by using an alternate screen, it doesn't carry over the previous history (and doesn't leave anything in the history after it switch back to the previous screen).

Using AlternateScreen not only solved the scrolling problem, but also issues I hadn't thought about like keeping the old history. Thank you very much!


Update flow emulator to v0.58.0

@m-Peter @turbolent It looks like the code coverage changed with the latest emulator version - is this OK?

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

Comparison is base (aeeb2f0) 39.80% compared to head (c4f3b82) 39.80%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1278   +/-   ##
=======================================
  Coverage   39.80%   39.80%
=======================================
  Files          39       39
  Lines        2143     2143
=======================================
  Hits          853      853
  Misses       1197     1197
  Partials       93       93

Flag
Coverage Δ

unittests
39.80% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@jribbink This is fine, this usually happens when versions of flow-emulator/flow-go are bumped (e.g. https://github.com/onflow/flow-cli/commit/11bfd481e2f6f8de3586c97c9bf8e3cbdfd369e8). Most likely there is a change in the bootstrapped transactions/scripts that are run, which results in this coverage change. Nothing to worry though.

oh looks like there are some duplicate prs from dependabot that where just created.


Question: is it possible to deploy Bot to Windows containers?

As the documentation says, application-hosted media bot can be deployed only to: VMs, Cloud Service, or Service Fabric.
https://docs.microsoft.com/en-us/microsoftteams/platform/bots/calls-and-meetings/requirements-considerations-application-hosted-media-bots#application-hosted-media-bot-development-requires-cnet-and-windows-server
But what about Windows containers? Is it possible to deploy there?

@adoronin
This is something we are just starting to test, running an application-hosted media bot within a Windows Container using Azure Kubernetes Service. It looks promising; hopefully we will publish an updated sample showing this.
The Windows Container must be running a full Windows Server operating system (not Server Core), and must have Server Media Foundation installed.
The underlying VM must have an instance-level public IP address (IL-PIP).
There must be unique TCP ports configured on the load balancer (e.g. via VMSS NAT port forwarding) which map to each specific VM/node instance.

@ssulzer
Thank you for the detailed answer! We'll try to implement that

@ssulzer any updates? from what I can tell its supposed to work in AKS, but I'm not sure how to do NAT port forwarding properly in AKS? I dont think its possible?

We are looking into this, to see if we can recommend something - it is certainly a challenge.

In VMSS used in SF and AKS, you can create an inbound nat pool to do port mapping, but this assumes that only one pod is deployed in a VM. Also, I haven't found a way to do it during AKS deployment, so the following steps are one of workarounds after deployment to enable the port forwarding.
// Create inbound nat pool
az network lb inbound-nat-pool create --backend-port [backend_port_num] --frontend-port-range-start [starting_port_num] --frontend-port-range-end [ending_port_num] --lb-name [loadbalancer_name such as "kubernetes"] --name [name] --protocol Tcp --resource-group [resource_group_name] --frontend-ip-name [name at Azure portal -> resource_group -> lb -> Frontend IP configuration]
// Assign nat pool to VMSS
az vmss update -g [resource_group_name] --name [aks_pool_name] --set virtualMachineProfile.networkProfile.networkInterfaceConfigurations[0].ipConfigurations[0].loadBalancerInboundNatPools="[{"id": "/subscriptions/[your_subscription_id]/resourceGroups/[resource_group_name]/providers/Microsoft.Network/loadBalancers/kubernetes/inboundNatPools/[inbound_nat_pool_name]"}]"
// update vmss
az vmss update-instances --instance-ids "*" --no-wait -n [aks_pool_name] -g [resource_group_name]

Thank you. We've seen the AKS sample and will discover that.

sorry, I'm not sure what you are talking about @adoronin

@4c74356b41
I'm talking about that sample:
https://github.com/microsoftgraph/microsoft-graph-comms-samples/tree/master/Samples/V1.0Samples/AksSamples/teams-recording-bot

mhm, i dont see any AKS specific configuration out there? like natting mentioned in the comment above (all the funky not supported az vmss stuff). do they use nginx ingress to publish it? I'm not sure whats going on there, sample is called AksSampes, but the readme talks about anything BUT AKS

@4c74356b41
They described AKS config here:
https://github.com/microsoftgraph/microsoft-graph-comms-samples/blob/master/Samples/V1.0Samples/AksSamples/teams-recording-bot/docs/deploy/aks.md

I’m working o migration from Cloud Services to VMSS, hopefully someone of you can have a tip for me.
In VMSS architecture it seems that proper NAT  inbound pool and port mapping is absolutely a key feature, since external machines shall be able to communicate to specific VM instance directly.
As @dongkyunn wrote before  it this thread, I faced the problem:
“how to pass Public IP port to the VM instance, or retrieve it on the VM once it’s provisioned”.
VM instance itself doesn’t know too much about surrounding environment.
I think this could be workaround by a reverse proxy, but this complicates the architecture.
Another option would be going straight to AKS.


EthicalAd: specific placement for Antora

Based on https://github.com/readthedocs/addons/pull/473

I also added the placement for mdBook as well:


Propose API for [onWill|onDid][Create|Delete|Rename]

This PR is for https://github.com/microsoft/vscode/issues/43768. It adds a new onWillRunOperation-event to IFileService, removes the corresponding experiment from the text file service and exposes corresponding events in the API.
Open questions

[ ] Should we enforce time limit for the onWill-events, like onWillSaveTextDocument, or should we just show a status bar message?
[ ] Should we disable bad listeners (slow, throwing exceptions...)
[ ] Should the onWill-events allow to do stuff, like sending off a workspace edits etc
[ ] Using vscode.workspace.fs.delete etc will trigger this event which IMO is very correct (an extension might run this as part as a command) but needs documentation

fyi @bpasero for changes in the file service

@jrieken some feedback:

the onWillRunOperation makes sense, then I would rename onAfterOperation to onDidRunOperation to be clear
we have to be careful about what extensions can do when participating: e.g. if they are able to modify the document and leave it dirty before the operation starts, we will see a ton of issues that the respective ITextFileService methods (move/copy/del) helps with

if they are able to modify the document and leave it dirty before the operation starts,

Yeah, that's the whole point of this API, e.g before renaming foo.java you need to update tweak its class definition... I have moved all of this to the file service so that we really catch all cases, e.g vscode.workspace.fs.delete etc. Tho, only rename allows to do things in API ways, collect workspace edits, all other don't allow things yet (e.g onWillCreate could be used to populate a file). Not sure how to proceed... It seems that we miss some sources of events or that we are short on some features...

Jus to clarify what happens today when you move a dirty file:

ITextFileService#move() must be used if you care about dirty state
we find out that the file is dirty
we use our backup service to create a backup of the dirty file at the target
we mark the file as non-dirty so that the editor is happy
we move it
we restore it from that backup

Somewhere in there is the IFileService#move operation. If an extension modifies the file via text edits, the backup we made would be stale and wrongly restored.
I think in general, I would:

only allow edits if ITextFileService is involved

PS: I am also not sure what this means for custom editors that potentially also have dirty state that is not under the control of the workbench...

Moving things to the text service is easy, it just means that those events aren't fire when using the fs-api. Unless, we wire some of the fs-call through the text file service... Not sure what that means but the textfile service likely doesn't care if non-textfile uris pass through it.
For now, I will move the will-events to the text file service. Likely also adding did-events on the text file service so that both events always come in pairs

Maybe if extensions must have API to call move/copy/delete these methods should be exposed on a text document and then use the text file service. This would make it clear that the operations are considering dirty states and possibly more UI state of the document to carry over.

@bpasero another challenge is that some extension would like to get bulk events, e.g when move 3 files in the explorer these should be one event listing those 3 files, not three events. This is to prevent expensive computations that need to be done. Extensions could solve this with debouncing but that's more fragile esp since we really know what's happening. Thought?

Yeah I think that would require us to expose bulk methods on the text file service, or just accept an array of resources for move/copy/delete.

@bpasero now it is really confusing as it seems that delete from the explorer doesn't go through the text file service but hits the file service directly. Is that the design? It seems to use the text file service to revert things but doesn't seem to use it for delete, in fact bulk edit seems to be the only user of TextFileService#delete.

I have the feeling that those event might be best living in the file service and that we tackle the move-and-dirty case pragmatic, e.g by saving before applying an edit from extensions

@jrieken that is correct, the explorer is doing something custom as you can see starting from here:
https://github.com/microsoft/vscode/blob/7b038f120726ac66bfb54452dc0bb9b1e7d6a4ca/src/vs/workbench/contrib/files/browser/fileActions.ts#L160
It will show a dialog asking to revert any files with dirty state. I think that makes sense to have on that layer because the bulk edit for sure does not want to bring up a dialog like that when running.
However, if we introduce the events on the level of ITextFileService, then I agree the explorer must call into that method 👍

I have the feeling that those event might be best living in the file service and that we tackle the move-and-dirty case pragmatic, e.g by saving before applying an edit from extensions

This will cause a weird flow of things:

someone calls ITextFileService#move() with a dirty file
a backup is made to the target to preserve the dirty contents
we use the file service to move
a move participant makes an edit and so we apply the edits save the file (that cannot happen from the file service because it has no knowledge about dirty files)
the file is restored in the editor and we have a backup so we restore it
the file is correctly still dirty, but we lost all the edits

Why can we not do the edits on the model and keep the editor dirty?

Hm, maybe this could work:

we move the events to file service as you suggest
the text file service actually registers as listener to these events to participate in the move/copy/delete operation
we remove the move/copy/delete methods from text file service and everyone uses the file service

I am not sure just relying on the event would be enough for the text file service to do everything it does today, but it is something to investigate.

Hm, that's an interesting thought. I am back and forth to where those events should live. If we move them into the file service we won't missing anything but also nothing can go unnoticed, e.g creating a log-file or deleting a temp-folder. The text-file service seems like a good abstraction of "user initiated" actions and would offer a clear boundary between the low-level and high-level world. Plus, it knows how to handle dirty move etc... Undecided.
And then custom editors are likely to have an impact on the text file service, e.g it might have to morph into a working copy service. Let's discuss Monday and in person


Realsense R4xx series camera support

Hi,
Any news on the release of librealsense for R4xx series cameras?
I have the hardware but no SDK at the moment. I already reached out to my contact at Intel but don't have a response yet.
Regards

Required Info

Camera Model
R410

Firmware Version

Operating System & Version

Kernel Version (Linux Only)

Build System

Do you have some information about when will it be available for purchase? at least approximate?

Hi,
The latest SDK 2.0 supporting RS4xx Series is available for Download on Intel® RealSense™ SDK 2.0

Hi @nlyubova, the RealSense D400 series of cameras will be available for purchase in October: https://software.intel.com/en-us/realsense/400-series. The librealsense branch that supports it is not final yet but will be final at launch of the product.
Jesus
Intel Customer Support

@jlgarcia75 thank you for the information! Looking forward to try it!


[zig] Unable to sign in and start desktop environment

Hello,
After the changes implemented in 0.12.0, I was finally able to get the zig rewrite of ly to compile.
I set up the systemd service using the recommended method (zig build installsystemd) and made some changes to /etc/ly/config.ini (enable an animation, set a clock format, and change the masking text from * to -).
However, upon a reboot, a few things happened:

I can see ly. This is great!
There is a stray cursor flashing in the top left of the screen maybe 30 columns over and 20 rows down (but I didn't count). This only goes away when I hit an arrow key to move to the password field or change the selected desktop environment.
No animation is displayed, which is less than ideal - and the reason I moved away from the main branch (see #592). In addition, the other changes I made to the config file are not visible either.
Finally, when I actually attempt to enter a password, I am greeted with the following...
exception (no address available)
??:?:?: -x73e4ee88709d in ??? (libc.so.6)
Unwind information for 'libc.so.0x73e4ee88709d' was not available, trace may be incomplete
... displayed atop the ly prompt -- so, I am unable to get to sway or i3 via ly, and I have to switch to a different tty and start it manually.

Additional info:
uname: Linux my_fqdn 6.8.9-arch1-1 #1 SMP PREEMPT_DYNAMIC Thu, 02 May 2024 17:49:46 +0000 x86_64 GNU/Linux
latest commit hash in my clone of ly: 629749b3832aaec86929794a527c108e5c1c29d1
init system: systemd
desktop environments tested: i3 and sway
If there are any other questions I can answer or things I can test, let me know. Thanks!

I can't seem to reproduce the exception you're getting and I can successfully login on an arch vm into sway.
Could you possibly list some steps that leads to the exception occurring so I can reproduce it?
My latest pr should fix the config issue u were having.

I can't seem to reproduce the exception you're getting and I can successfully login on an arch vm into sway. Could you possibly list some steps that leads to the exception occurring so I can reproduce it?
My latest pr should fix the config issue u were having.

Hi there,
Thanks for getting back so quickly. I'll be able to be more specific on Monday when I'm back at my PC, but here are the steps I took:

Clone ly
Checkout zig branch
Verify the latest version is pulled
zig build
zig build installsystemd
Enable ly.service
reboot

Nothing fancy/that I think should cause this, but I was then unable to sign in with the error mentioned above.
While I won't be at my machine physically for a few days, if there are any logs/details I can grab/share remotely let me know and I will.

My latest PR "should" allow you to login.
Let me know if the issue still occurs when you can.

Thanks!
There's a chance I'll be at my computer tomorrow, if I am I will check and let you know -- else Monday.

@Kawaii-Ash I am now successfully seeing my config changes applied to my login screen.
When I attempt to log in, the more verbose error is gone.
However, there is no evidence of a sway session even attempting to start, and I am told "logged out" in the space above the username field.
In addition, the cursor in the corner is still there when I do not have an animation applied (see attachment).
https://github.com/fairyglade/ly/assets/150633550/7eef552a-0222-4769-b9cd-339cd3e5921f

https://github.com/fairyglade/ly/assets/150633550/2a97e02f-24be-4a58-bdc9-bc604cb53420
... and here is a video of my attempt to sign in to an unconfigured ly session and start sway.

... and here is a video of my attempt to sign in to an unconfigured ly session and start sway.

It seems like Sway is crashing here (though I may be wrong). Just to confirm, can you launch Sway from a TTY?

... and here is a video of my attempt to sign in to an unconfigured ly session and start sway.

It seems like Sway is crashing here (though I may be wrong). Just to confirm, are you able you launch Sway from a TTY?

Yes, I have no issue opening sway from a tty. I've also verified that ly is looking in the right directory for my .desktop file.

I could reproduce this in an arch vm only when I didn't have polkit installed.

That could be it, I'm not sure if I have polkit installed on this system or not. I will check this evening.
I don't seem to see polkit as a dependency listed in README, fyi -- but I'll look into this asap.

Alternatively, adding my user to the "seat" group and restarting seatd also allows me to launch sway as an alternative to installing polkit.
But without doing either, I have the same issue starting sway from a tty when I'm not root. But apparently you have no issue opening sway from a tty.
Other than that, I'm not really sure what the issue is.

It looks like I have polkit installed and running already. Adding myself to the seat group and restarting seatd + rebooting didn't seem to affect anything either. And yet, I am still able to start sway just fine from a tty.

If you redirect stderr to a file, like:
sudo zig-out/bin/ly 2> o.txt
Then try to login to sway, does the file contain anything?

Ah yes, wlroots-based WMs like labwc and sway have issues when polkit and D-Bus aren't installed. Check that the latter is installed (it should be, but just to make sure).
If that's OK, I recommend redirecting and checking stderr like Ash suggested. If it says anything about the former 2 dependencies (or if it doesn't say anything useful), try prepending this command to the Exec line of your Sway desktop file:
dbus-run-session

I have both polkit and dbus installed and running.
On attempting to redirect errors to a file, running ly and trying to start sway, no errors are generated, both when I use dbus-run-session and when I do not.

That's interesting... so it doesn't work with dbus-run-session either? Sometimes quite wrong here then. How did you end up installing Arch+ly+Sway, so that we could try to reproduce this behavior as perfectly as possible? @amberjennings

I installed Arch from the official iso, looks like this system was set up in August 2023 so the 2023.08.01 image. I keep my packages up to date, though, so I'm not sure that's super relevant. Sway would've just been installed via Pacman, and ly via this repository and zig build/zig build installsystemd. Not sure how helpful this is but let me know if you have more questions.

Could you edit your config file and change wayland_cmd = /etc/ly/wsetup.sh to just wayland_cmd = to see if it makes any difference?

This seems to have done it! I do use tcsh and I'm not sure how extensively that block is tested. I notice that it's not checking for ~/.cshrc alongside /etc/csh.login and ~/.login though I don't know if that means much.
But in the meantime, this issue is resolved. Thank you both again!

I guess maybe this should stay open until the repository is updated to fix this. But my personal situation is resolved.

@amberjennings We're both unable to reproduce your issue with TCSH changed as the default shell for all users. Do you have a custom TCSH configuration perhaps? And, just to make sure (but I can probably guess the result), what's the output of echo $SHELL in a TTY?

@amberjennings We're both unable to reproduce your issue with TCSH changed as the default shell for all users. Do you have a custom TCSH configuration perhaps? And, just to make sure (but I can probably guess the result), what's the output of echo $SHELL in a TTY?

amber@shruikan ~ % echo $SHELL
/bin/tcsh
amber@shruikan ~ % cat .cshrc
alias ls  		ls --color
alias l   		ls -al
alias ll  		ls -l
alias g   		git
alias em  		emacs
alias sget		spotdl --bitrate 320k download

setenv	EDITOR	emacs

if (`uname` == "Linux") then
 setenv PATH  "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/home/amber/.local/bin"
endif

if (`uname` == "Darwin") then
 setenv PATH  "/opt/homebrew/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/Users/amber/.local/bin"
endif

if (-e ~/.hushmotd) then
	rm -f ~/.hushmotd
endif

set     red="%{\033[1;31m%}"
set   green="%{\033[0;32m%}"
set  yellow="%{\033[1;33m%}"
set    blue="%{\033[1;34m%}"
set magenta="%{\033[1;35m%}"
set    cyan="%{\033[1;36m%}"
set   white="%{\033[0;37m%}"
set     end="%{\033[0m%}"

set prompt="${blue}%n${white}@${blue}%m ${cyan}%~ ${white}%%${end} "
unset red green yellow blue magenta cyan yellow white end
umask 077

@amberjennings Well that's awkward, I'm still unable to reproduce the bug. I think it might be easier for you to try to mess with the wsetup.sh script, in particular with the TCSH configuration part, to maybe find what's wrong (and perhaps we'll be able to better deduce what's wrong later on).

Yeah, I can do that. No promises on timeline as I am quite busy but if you want to assign to me I'll do what I can.

Yeah, I can do that. No promises on timeline as I am quite busy but if you want to assign to me I'll do what I can.

No pressure! I feel like assigning yourself to an issue you reported is kind of odd though... but I'll edit the title of the issue to reflect its "pending" status.

I'm attempting to find the source referred to in wsetup.sh (kde-workspace/kdm/kfrontend/genkdmconf.c) and it looks like the kde-workspace repository was shuttered 9 years ago in favor of plasma-workspace, and even then I cannot locate that file. I realize that this is a part of the original C version of ly and not put in place by you, but also, if telling it not to run at all makes the program work, with some more testing, could it be phased out entirely?

Yeah, I can see the use case, I'm just not sure why it's entirely preventing me from starting a desktop session. Just running the script on its own works fine (because all its doing is sourcing environment variables that already exist, aiui). The xsetup script is definitely more integral.
/etc/csh.login has some interesting stuff happening:
if ( -o /dev/$tty && ${?prompt} ) then
    # Console
    if ( ! ${?TERM} )           setenv TERM linux
    if ( "$TERM" == "unknown" ) setenv TERM linux
    # No tset available on SlackWare
    if ( -x "`which stty`" ) stty sane cr0 pass8 dec
    if ( -x "`which tset`" ) tset -I -Q
    unsetenv TERMCAP
    settc km yes
endif

/etc/csh.cshrc seems mostly uninteresting, but it exists as well.
When I get some time I will try modifying wsetup.sh and doing some more testing.


Add missing Qwen2-VL monkey patch test

Summary
Add a monkeypatch test for Qwen2-VL

Testing Done

Hardware Type:
[x] run make test to ensure correctness
[x] run make checkstyle to ensure code style
[x] run make test-convergence to ensure convergence

All tests passing, except for the gemma2 tests that are failing on main


Using Jersey declarative linking does not appear to be working.

Helidon Version: 2.2.1
Helidon MP
JDK version: OpenJDK Runtime Environment 18.9 (build 11.0.5+10)
OS: Ubuntu 20.10

We want to see if we can build hypertext-driven APIs using Helidon. There is experimental/Beta support for this available in Jersey, by declarative linking: https://eclipse-ee4j.github.io/jersey.github.io/documentation/latest/declarative-linking.html
Because Helidon is using Jersey under the covers, this probably should work.
What we are seeing is that although the DeclarativeLinkingFeature gets properly registered via en @Provider annotated class, is that links aren't actually emitted from the payload.
Other observations: it took some time for use to get the application booted with the correct set of libraries. Santiago helped us out by figuring out what we actually need, which is:

declarative linking 2.33
jakarta.el-api 3.0.3
jakarta.el 3.0.3

With respect to the jakarta libs, these names have been changed apparently but the packages internally have not. This is a good thing as Weld's BeanManagerImpl would not have coped with that in acquiring the ELResolver.
Easiest way to reproduce this is to create a Pojo that is exposed through a resource, and then add an @ProvideLink on that @GET operation.

As an aside registering the DeclarativeLinkingFeature will result in the following warning being emitted on System.err:
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.glassfish.jersey.linking.FieldDescriptor$1 (file:/home/jvissers/.m2/repository/org/glassfish/jersey/ext/jersey-declarative-linking/2.33/jersey-declarative-linking-2.33.jar) to field java.time.LocalDateTime.date
WARNING: Please consider reporting this to the maintainers of org.glassfish.jersey.linking.FieldDescriptor$1
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
{"createdBy":"TODO","createdDateTime":"2021-03-03T16:47:03.119759","status":"running","uuid":"a3cc3bcb-390f-4d2b-85ab-071ac892a8c4"}

I misunderstood the declarative linking feature, this is not a Helidon issue.


[Bug]: Terraform instructions do not set environment variable EKS_CLUSTER_NAME

Installation method
Own AWS account
What happened?
While following the instructions to deploy in my own environment with Terraform, Step 2 "un the following commands to deploy the workshop environment" fails to deploy due to errors with var.cluster_name.  If you set the environment variable, EKS_CLUSTER_NAME, and then run the command it works fine.
What did you expect to happen?
Terraform to deploy without error
How can we reproduce it?
Download the terraform files and run the following per the instructions:
terraform init
terraform apply -var="cluster_name=$EKS_CLUSTER_NAME" -auto-approve
Anything else we need to know?
No response
EKS version
any

Fixed by #795


how to setting up a Secure Web Socket?

Earlier in the drupal7 website, I have configured Push notifications using PHP WebSockets.
Now we have implemented https for our server and we are not able to get the push notifications and real-time functions on the websites.
The following error message is shown on the console.

Mixed Content: The page at 'https://xxxxxx.xx' was loaded over HTTPS, but attempted to connect to the insecure WebSocket endpoint 'ws://xxxxxx.xx:15003/echobot'. This request has been blocked; this endpoint must be available over WSS.

We have real-time jobs configured on the website related to the status of the data and time period.
Referenced many related threads including New feature discussion: TLS #41
Please give any suggestion.
Thanks in advance

change ws://
to wss://

don't work. In ws:// work, wss:// don't.

can this do response wss from github pages?


AVRO-1973: Upgrade netty to the latest 3.x release

Upgrading netty version to 3.10.6 caused a test failure. The root cause was that from 3.6.0 netty does not interrupt worker threads therefore, server2.close() would hang until blockingSimpleImpl.releaseRunPermit() is invoked. Could not find any way to call an interrupt on the related workers therefore, modified the test.
Also fixed an issue in the same test that an assert would never fail a test if simply invoked in another thread.

Thanks for the review, @benmccann.
@busbey, as you have reviewed my other change related to version upgrades, could you please check this one as well?


HBase auth fix

Once we started using Geomesa with Kerberos-secured HBase on different runtimes (regular APIs in docker, Flink, Spark on Databricks) we faced a number of issues that prevented Geomesa to work at all. The issues are:

While running on managed env such as Databricks there is already a current user present on the workers. And simply running HBaseConnectionPool.configureSecurity which does UserGroupInformation.loginUserFromKeytab does not set the current user as logged-in user and current authorization basically does nothing. There are 2 ways to make a fix:

The most simple which affects less code which I applied. Switch from UserGroupInformation.getCurrentUser to UserGroupInformation.getLoginUser which always would used logged-in user.
The best way, yet will require lots of refactoring. The connection to the HBase should be created with an explicit user via ConnectionFactory.createConnection(conf, user). In case there is any nested code that relies on the current user (for example, hadoop hbase data format) is called - that should be wrapped with user.doAs { ... }

HBaseIndexAdapter leaks on long-running jobs. There is a known HBase defect and passing the explicit thread pool and closing it manually solves the issue.
While writing into HBase that is a good idea to control the flush timeout, otherwise, we cannot guarantee the latency of writing into HBase.
While working with Spark we need to handle hadoop configuration changes as well as authorization in every mapPartition . Note, that explicit reauth is not expensive cause  the user will be statically cached on every node anyway. Also for some reason a piece of code with token auth never worked for us (I am not even sure how it was supposed to be working)
While ingesting spark DataFrame data into Geomesa if there is an existing Geomesa schema we should prefer it if compatible and not try to force Spark inferred schema which might be slightly different yet compatible and will cause the crash.

Note, those changes were already working in Geomesa running on Kerberos-secured Azure HD Insights HBase for more than a year on different workloads. We were adding more and more fixes as we faced different situations. I believe the code might be improved a lot yet that will require rewriting a bunch of original code and a new cycle of E2E tests.

Thanks for the fixes!

lgtm - @aheyne do you have any reservations about these changes?

looks like the Scala 2.11 build is dying on the lambda here, which 2.11 doesn't convert into a Runnable like 2.12 does:
/home/runner/work/geomesa/geomesa/geomesa-utils/src/test/scala/org/locationtech/geomesa/utils/concurrent/ExitingExecutorTest.scala:26: overloaded method value submit with alternatives:
Error:    (x$1: Runnable)java.util.concurrent.Future[_] <and>
Error:    [T](x$1: Runnable, x$2: T)java.util.concurrent.Future[T] <and>
Error:    [T](x$1: java.util.concurrent.Callable[T])java.util.concurrent.Future[T]
Error:   cannot be applied to (() => String)
Error:          val second = pool.submit(() => Thread.currentThread().getName).get()

lgtm - @aheyne do you have any reservations about these changes?

Looks good to me. I don't think the auth bits will effect anything we're doing. Would need to test to know for sure but I'm comfortable merging this.


new sources

I'm about to do a release of jose that doesn't fail here, you can mark it as failing for now and I'll open a PR when I have a release

https://github.com/ocaml/opam-repository/pull/18588

wanna push it to this branch?

Sure


Unable to Install Mage and Its Dependencies Without Internet Connection

Is your feature request related to a problem? Please describe.
I am encountering issues while attempting to install Mage and its dependencies in an environment without internet access. The installation process seems to be dependent on online resources, and I need guidance on how to perform an offline installation.
Steps to Reproduce:
Attempted installation of Mage and its dependencies in an environment without internet access.
Encountered errors related to missing dependencies or inability to fetch required packages.
Expected Behavior:
Mage and its dependencies should be installable in an offline environment without relying on internet resources.

I have installed Python 3.10
Downloaded Mage source code from Git
there is a file requirements.txt
downloaded all libraries using pip also using PyPi
Installed libraries which are required as it is in Requirements.txt
Installing Mage via Pip
Still got exceptions

@matrixstone great to see your achievement can you please let me know how you installed these libraries that are in requirement.txt??

Hi @ZaighamRaza I download all the packages in requirements.txt with this command: python -m pip download -r requirements.txt --prefer-binary -d download_packages Also there is another requirements.txt file under mage-ai/mage_integrations/requirements.txt requirements.txt. Then I install it offline with pip install command but specify the param --find-links with your folder containing all downloaded packages download_packages

Hi @matrixstone i am trying with this command pip install --no-index -e . --no-build-isolation --find-links=C:\Users\VWDemo\Downloads\MageOfflineInstaller\download_packages -r requirements.txt  and i am getting this error ERROR: File "setup.py" or "setup.cfg" not found. Directory cannot be installed in editable mode: C:\Windows\system32

@matrixstone  if i execute this command pip install --no-index -e . --no-build-isolation --find-links=C:\Users\VWDemo\Downloads\MageOfflineInstaller\download_packages -r requirements.txt in Mage github directory i am getting this error ERROR: Command errored out with exit status 1:      command: 'C:\Users\VWDemo\AppData\Local\Programs\Python\Python310\python.exe' 'C:\Users\VWDemo\AppData\Local\Programs\Python\Python310\lib\site-packages\pip\_vendor\pep517\in_process\_in_process.py' prepare_metadata_for_build_wheel 'C:\Users\VWDemo\AppData\Local\Temp\tmpc_bn1lfe'          cwd: C:\Users\VWDemo\Downloads\MageOfflineInstaller\mage-ai-release-0.9.64     Complete output (14 lines):     running dist_info     creating C:\Users\VWDemo\AppData\Local\Temp\pip-modern-metadata-twd9fnah\mage_ai.egg-info     writing C:\Users\VWDemo\AppData\Local\Temp\pip-modern-metadata-twd9fnah\mage_ai.egg-info\PKG-INFO     writing dependency_links to C:\Users\VWDemo\AppData\Local\Temp\pip-modern-metadata-twd9fnah\mage_ai.egg-info\dependency_links.txt     writing entry points to C:\Users\VWDemo\AppData\Local\Temp\pip-modern-metadata-twd9fnah\mage_ai.egg-info\entry_points.txt     writing requirements to C:\Users\VWDemo\AppData\Local\Temp\pip-modern-metadata-twd9fnah\mage_ai.egg-info\requires.txt     writing top-level names to C:\Users\VWDemo\AppData\Local\Temp\pip-modern-metadata-twd9fnah\mage_ai.egg-info\top_level.txt     writing manifest file 'C:\Users\VWDemo\AppData\Local\Temp\pip-modern-metadata-twd9fnah\mage_ai.egg-info\SOURCES.txt'     reading manifest file 'C:\Users\VWDemo\AppData\Local\Temp\pip-modern-metadata-twd9fnah\mage_ai.egg-info\SOURCES.txt'     reading manifest template 'MANIFEST.in'     adding license file 'LICENSE'     writing manifest file 'C:\Users\VWDemo\AppData\Local\Temp\pip-modern-metadata-twd9fnah\mage_ai.egg-info\SOURCES.txt'     creating 'C:\Users\VWDemo\AppData\Local\Temp\pip-modern-metadata-twd9fnah\mage_ai.dist-info'     error: invalid command 'bdist_wheel'     ---------------------------------------- WARNING: Discarding file:///C:/Users/VWDemo/Downloads/MageOfflineInstaller/mage-ai-release-0.9.64. Command errored out with exit status 1: 'C:\Users\VWDemo\AppData\Local\Programs\Python\Python310\python.exe' 'C:\Users\VWDemo\AppData\Local\Programs\Python\Python310\lib\site-packages\pip\_vendor\pep517\in_process\_in_process.py' prepare_metadata_for_build_wheel 'C:\Users\VWDemo\AppData\Local\Temp\tmpc_bn1lfe' Check the logs for full command output. ERROR: Command errored out with exit status 1: 'C:\Users\VWDemo\AppData\Local\Programs\Python\Python310\python.exe' 'C:\Users\VWDemo\AppData\Local\Programs\Python\Python310\lib\site-packages\pip\_vendor\pep517\in_process\_in_process.py' prepare_metadata_for_build_wheel 'C:\Users\VWDemo\AppData\Local\Temp\tmpc_bn1lfe' Check the logs for full command output.


Is it literally a stack?

Expressive middleware is a [] XD

Sorry, what do you mean by "expressive"? But yes, once composed all middleware used in Koa is expected to be a flat array.

Sorry, what do you mean by "expressive"? But yes, once composed all middleware used in Koa is expected to be a flat array.

not a fat array XD


feat: update app to new version, add database variabls

Adds PSC and private IP functionality for Cloud SQL.
Updates the application based on upstream generative AI retrieval app

Reminder: You already know this, but just a heads-up to introduce a new sic-jss-x tag (instead of using the existing sic-jss tag) since this is a breaking change. More about "incompatible changes" at go/jss-authoring-guide/update-a-solution.


Lua error in om.lua / healing.lua

Im getting this error sometimes, rotation continue to execute correctly (Im not a healer though) :
2x [string "C:\ToolZ\FH\Lua\Object.lua"]:18: The GUID does not belong to an existing object.
[C]: in function `ObjectPosition'
[string "C:\ToolZ\FH\Lua\Object.lua"]:18: in function <[string "C:\ToolZ\FH\Lua\Object.lua"]:16>
(tail call): ?
NerdPack\system\om.lua:37: in function `Get'
NerdPack\system\healing.lua:65: in function <NerdPack\system\healing.lua:64>
SharedXML\C_TimerAugment.lua:16: in function <SharedXML\C_TimerAugment.lua:14>

Locals:

There is a check for UnitExists(Obj.key) but it isn't enough, it seems not to like elseif/else.
Also, I don't know if it's related to the ObjectManager but since you change how it works a few time ago, my WoW randomly crash (error 132) when entering combat.
~2 crash on a 6 hours session (World Quest/Daily heroic dunjeon etc...).

this should be fixed now, closing


When parallel processing the sessioninfo is stored in a file by multiple processes

When parallel processing "sessioninfo" is written to a file by parallel processes:
results/QC/sessioninfo_part4.RData and results/QC/sessioninfo_part5.RData, in the case of g.part4 and g.part5 respectively.
Possible solution:
First check whether the file exist and whether it was created in the last (X hours) and only overwrites or create it if this is not the case.

I have now updated the code (as of version 1.8-2) to store the sessioninfo at the end, but only if the the existing file is older than 2 hours + a random number of seconds (max 1 hour) to minimize the chance that two parallel process want to write the file at the same time.


OTP verification resend mail issue and UI design Fixed

Please check and verified and merged in master

merged in master branch


Split provisioning app?

When discussing #283, the following idea emerged:

Have one central provisioning app which can ONLY provision new projects
Extract the functionality to create components within the project to a specialised, per-project provisioning app (potentially part of the webhook proxy). That way, the authentication to create components can be done via OpenShift, which already solves the auth/permission problems.

my thoughts ... the prov app only calls services (that in the future sit within a project / ocp namespace) so aside the rights to create a bb repo (based on the current users' rights) everything else is done by rundeck (and going forward jenkins)..
So why would we need the split?
Just thinking out loud here

@clemensutschig Let us talk this over next week in person :)

hi! indeed this topic was also raised (with positive voting) in https://github.com/opendevstack/ods-provisioning-app/issues/179
I also agree on it, so to go on the direction of having a provided web/GUI (provisioning-app) that uses the provisioning API (provisioning-api) which is in turn used for any other store-like service, as stated in the aforementioned issue.
If adding too complexity due to having (maybe) another repo/piece, we could integrate the provisioning-app into the provisioning-api, being served (static files/content) through it.
Main reasons are also:

to use a front-end web application framework
better testable and maintainable
can reduce complexity of current implementation
not using jquery or plain javascript, so to having it typed and more secure
easy to add new functionalities from an app level point of view

@gerardcl The idea is not about splitting backend and frontend, but splitting the provisioning app into two apps. One to create projects, and one to create components.


Update cf-expandables to Atomic Design

As part of the v4 atomic design update, convert the appropriate bits of cf-expandables to the atomic design principles. Component should be broken out into appropriate atoms, molecules, and organisms.
How to Build an Atomic Component
Atomic Design Workflow
Making Sense of Atomic Design
Where to find existing Atomic Work

Atomic PRs
v1 atoms
v1 molecules
v1 organisms

CFGov Refresh Code:

https://github.com/cfpb/cfgov-refresh/blob/flapjack/cfgov/unprocessed/css/organisms/expandable.less
https://github.com/cfpb/cfgov-refresh/blob/flapjack/cfgov/unprocessed/css/organisms/expandable-group.less
https://github.com/cfpb/cfgov-refresh/blob/flapjack/cfgov/unprocessed/js/organisms/Expandable.js
https://github.com/cfpb/cfgov-refresh/blob/flapjack/cfgov/unprocessed/js/organisms/ExpandableGroup.js

@mistergone how are the expandables going?

@jimmynotjim - Good! I got feedback on my work on the 7th, but I was out most of this week (including the holiday), so I haven't done additional work since that feedback session.

The main code is merged but I just realized it's not in the sandbox. @mistergone do you have time to make sure https://github.com/cfpb/capital-framework-sandbox/tree/atomicjs-exp is all ready to go and open a PR?

Sandbox PR is open and available for review.


Removing firmware version mechanism

Until now, Ragger was keeping track of firmware version for two usages:

being able to properly start Speculos (--model nanos --sdk 2.1.0)
keeping track of graphical object positions in Stax

The first need is now resolved for quite some time as the version is embedded into the application ELF files, hence Speculos figures it out alone.
The second one is not aligned with Ledger policy which for now only supports the latest firmware version. Maintaining the current mechanism for just this purpose is probably not worth it (and would be marginally useful, as it is expected that graphical movement in Stax should be quite rare).
So this PR removes the version mechanism, then simplify the Firmware class, which is now a simple enumerate with some fancy properties on top.
Not sure on the version bump. It should be pretty invisible for those who rely on the ragger.conftest module, but anyone crafting custom Firmware will be impacted. As it could be a breaking change for them, I defaulted on a major version but :shrug:

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

Comparison is base (7902e07) 75.54% compared to head (de1b30a) 75.58%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #138      +/-   ##
==========================================
+ Coverage   75.54%   75.58%   +0.04%
==========================================
  Files          35       35
  Lines        1660     1663       +3
==========================================
+ Hits         1254     1257       +3
  Misses        406      406

Impacted Files
Coverage Δ

src/ragger/conftest/base_conftest.py
73.88% <ø> (ø)

src/ragger/firmware/versions.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Moved to a minor bump.
The change is hardly breaking, and would only impact rare users (only myself probably).


how to  'sum my data'

hello Alexander Prokhorov,
I like YaLinqo and it make my work more efficient.
But it's difficult  when I  'sum my data'
The array is as follows, I want to group by catId and  sum up quantity
$products = array(
    array('name' => 'Keyboard',    'catId' => 'hw', 'quantity' =>  10, 'id' => 1),
    array('name' => 'Mouse',       'catId' => 'hw', 'quantity' =>  20, 'id' => 2),
    array('name' => 'Monitor',     'catId' => 'hw', 'quantity' =>   0, 'id' => 3),
    array('name' => 'Joystick',    'catId' => 'hw', 'quantity' =>  15, 'id' => 4),
    array('name' => 'CPU',         'catId' => 'hw', 'quantity' =>  15, 'id' => 5),
    array('name' => 'Motherboard', 'catId' => 'hw', 'quantity' =>  11, 'id' => 6),
    array('name' => 'Windows',     'catId' => 'os', 'quantity' => 666, 'id' => 7),
    array('name' => 'Linux',       'catId' => 'os', 'quantity' => 666, 'id' => 8),
    array('name' => 'Mac',         'catId' => 'os', 'quantity' => 666, 'id' => 9),
);

new array like this
$products = array(
    'hw' => [
        ‘catId’ => ‘hw’,
        ‘quantity_sum’ => ‘xxx’,
    ],
    'os'=>[
        ‘catId’ => ‘os’,
        ‘quantity_sum’ => ‘xxx’,
    ]
);

what should i do?

Sorry for not replying in time. I hope my answer is useful for somebody.
The method groupBy is just toLookup paired with select, so it has a lot of arguments and arguments of anonymous functions in its arguments are plentiful as well (and let's not forget separate handling of keys and values in PHP iterators), which can be confusing. For reference, here's simplified code of groupBy:
function groupBy ($keySelector, $valueSelector, $resultSelectorValue, $resultSelectorKey)
{
    return self::from($this->toLookup($keySelector, $valueSelector))
        ->select($resultSelectorValue, $resultSelectorKey);
}

The simplest solution with minimum number of calls is this:
$productGroups = Enumerable::from($products)
    ->groupBy(
        function($product) { return $product['catId']; },
        null,
        function($products, $catId) {
            return [
                'catId' => $catId,
                'totalQuantity' => Enumerable::from($products)->sum(
                    function($p) { return $p['quantity']; }
                )
            ];
        }
    );
var_dump($productGroups->toArray());

Here, we group products by category by specifying key, but skip transforming products until we get to the select stage of groupBy when the whole list of products is available. There we use sum on the array of products and return a category value.
The null passed to valueSelector means "return what was passed": function($product) { return $product; } (interpretation of default arguments is in the reference documentation).
If the groupBy call with 3 arguments is confusing, you can use something like this instead:
$productGroups = Enumerable::from($products)
    ->groupBy(
        function($product) { return $product['catId']; }
    )
    // here each element is `catId => productArray`
    ->select(
        function($products, $catId) {
            return [
                'catId' => $catId,
                'totalQuantity' => Enumerable::from($products)->sum(
                    function($p) { return $p['quantity']; }
                )
            ];
        }
    );
var_dump($productGroups->toArray());

This is essentially the same as the previous solution, though more functon calls are performed.


[PackageLoading] Execute manifest parsing within a sandbox

-- rdar://problem/31107213

@swift-ci please test

Haven't tried running a build with sandbox, but build directory is overridable so we would need to use the parameter thing for it, right?

Yeah, or just correct the .sb files on the fly.

@swift-ci please test and merge

@swift-ci please test and merge

If the sandbox profile is generated, then the -p option could be used to pass the sandbox profile on the command line, so that a temporary file wouldn't even be needed.  I see this done a fair amount in build logs.

@swift-ci please test

@swift-ci please test

That is some seriously great response time, given that I only came up with the proof of concept (which lead to rdar://problem/31107213) three days ago. Kudos! 👍


Firefox/chrome inconsistency

Hi @mikolalysenko!
I’ve noticed that firefox and chrome handle scrolling speed perceptually differently, at least in windows.
I’ll try to figure out why.

Thanks!


Substratest: avoid using string to define tuple status

It is better to use global variables or custom type for this status. It improves code readability, code maintainability and is less error prone.

Fixed in #30


when viewing some messages it crash

Hi thanks you for fixing the boolean problem.
Im happy to test because i went this to be my to go app for servicebuss messages. And I like this is in blazor. i learning it. maybe i can contribute to fix problems in the future
In some topics when i try to view the messages from the messages list i get a red banner and say An error has occurred, the application no longer responds. Reload
but other topics work to view messages without problem.
I have checked both topics, the one that work and the one that crash in the windows service bus explorer and both work. I just want to check that it was not an issue with the topics. to rule that out

Here is the message of the console:

The errors are not saying much since the error handling is not complete - it just basically states there was an unhandled exception and the application crashed. Can you maybe check pinpoint a not working message and check in the classic Service Bus Explorer what Application Properties do you have there? It may be the application property type is not supported property. In the meantime I will add global exception handler that at least logs to console the exception details.

Please install version 0.4.3 - this one will print detailed error message. Please then then paste this and the types of application properties as mentioned above.

I have installed 0.4.3
and i get the same error but not more logging from the console

the screenshot for the same message below minus sensitive information . here is from windows Servicebus explorer:

i can see the message well and do everything normal.
if you need me to look something else i am available

I suspect in Message Custom Properties there is a type that is not supported. Can you please tell me which types are used there? I see for sure string, some numeric one (which one exactly)?

I have downloaded 0.4.5
here is the screenshot

In the windows service bus in the red square can you see the Custom Properties and their values.
I dont if it is because source = null    makes the error that it cant make and instansce of the object?

Ok thank's for the feedback. Yes that would be for sure this and I'm able now to reproduce the error. I will look into it ASAP. Didn't think about the nullability to be honest.

Ok thank's for the feedback. Yes that would be for sure this and I'm able now to reproduce the error. I will look into it ASAP. Didn't think about the nullability to be honest.

Is not easy to think about everything, usually is when things dont work that we find the problem. And i hapy to give feedback this will be good for both of us. I test it and report back. so in the end it will be an more Awsome app that already is. and a totally cross plattform app

anoher discovery...i think this was related to a another issue about the size of the panel to the left sio the text dont get squished

If remove the css max-width: 444px
then the panel look really nice

Pushed fix for the null messages. For the other finding I will create a separate bug.


ImportError after upgrading from 5.7.0 to 6.1.2

Jun 27 10:15:02 homeassistant python[797]:     from .server import MatterServer
Jun 27 10:15:02 homeassistant python[797]:   File "/srv/homeassistant/lib/python3.12/site-packages/matter_server/server/server.py", line 22, in <module>
Jun 27 10:15:02 homeassistant python[797]:     from ..common.helpers.api import APICommandHandler, api_command
Jun 27 10:15:02 homeassistant python[797]:   File "/srv/homeassistant/lib/python3.12/site-packages/matter_server/common/helpers/api.py", line 10, in <module>
Jun 27 10:15:02 homeassistant python[797]:     from matter_server.common.helpers.util import parse_value
Jun 27 10:15:02 homeassistant python[797]:   File "/srv/homeassistant/lib/python3.12/site-packages/matter_server/common/helpers/util.py", line 28, in <module>
Jun 27 10:15:02 homeassistant python[797]:     from chip.clusters.Types import Nullable
Jun 27 10:15:02 homeassistant python[797]:   File "/srv/homeassistant/lib/python3.12/site-packages/chip/clusters/__init__.py", line 26, in <module>
Jun 27 10:15:02 homeassistant python[797]:     from .Objects import (AccessControl, AccountLogin, Actions, ActivatedCarbonFilterMonitoring, AdministratorCommissioning, AirQuality,
Jun 27 10:15:02 homeassistant python[797]: ImportError: cannot import name 'Scenes' from 'chip.clusters.Objects' (/srv/homeassistant/lib/python3.12/site-packages/chip/clusters/Objects.py)

running alongside homeassistant core 2024.6.4
was working fine before the pip3 install --upgrade
upgrade output:
Installing collected packages: home-assistant-chip-clusters, python-matter-server
  Attempting uninstall: home-assistant-chip-clusters
    Found existing installation: home-assistant-chip-clusters 2024.2.1
    Uninstalling home-assistant-chip-clusters-2024.2.1:
      Successfully uninstalled home-assistant-chip-clusters-2024.2.1
  Attempting uninstall: python-matter-server
    Found existing installation: python-matter-server 5.7.0
    Uninstalling python-matter-server-5.7.0:
      Successfully uninstalled python-matter-server-5.7.0
Successfully installed home-assistant-chip-clusters-2024.5.2 python-matter-server-6.1.2

For posterity, I managed to get it working by doing a removal and then reinstall of python-matter-server, homeassistant-chip-core, and homeassistant-chip-clusters
Seems like something in the upgrade process was broken. Sadly I can't repro.


[INT-1227] Fix all outstanding eslint and jest warnings

This PR fixes all outstanding console warnings, errors and eslint warnings which do not fail the build, but create unnecessary noise. For the console statements that are not easy to fix, but have valid reasons to ignore I setup jest.setup.js to ignore them and document why.
[pach@pachyderm jupyter-extension]$ npm run test

><EMAIL_ADDRESS>test
> jest

 PASS  src/plugins/help/__tests__/help.test.ts
 PASS  src/plugins/mount/__tests__/mount.test.tsx
 PASS  src/plugins/telemetry/__tests__/telemetry.test.tsx
 PASS  src/plugins/mount/components/Datum/__tests__/Datum.test.tsx
 PASS  src/plugins/mount/components/Pipeline/__tests__/Pipeline.test.tsx
 PASS  src/plugins/mount/components/SortableList/__tests__/SortableList.test.tsx
 PASS  src/plugins/examples/__test__/examples.test.tsx
 PASS  src/plugins/mount/components/FullPageError/__tests__/FullPageError.test.tsx
 PASS  src/plugins/mount/components/Config/__tests__/Config.test.tsx

Test Suites: 9 passed, 9 total
Tests:       41 passed, 41 total
Snapshots:   2 passed, 2 total
Time:        5.728 s
Ran all test suites.
[pach@pachyderm jupyter-extension]$ npm run eslint

><EMAIL_ADDRESS>eslint
> eslint . --ext .ts,.tsx --fix

@bbonenfant yeah good idea! let me look into changing the configuration to fail instead of just warn 🤔