Raspberry Pi GPIO switch state not preserved

Home Assistant release with the issue: 0.76.2

Last working Home Assistant release (if known): Not known.
Operating environment (Hass.io/Docker/Windows/etc.): Hass.io

Component/platform: https://www.home-assistant.io/components/switch.rpi_gpio/

Description of problem:
GPIO switch states are not preserved through a restart of HA. This applies to the frontend switch state and actual electrical state of the GPIO.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
- platform: rpi_gpio
  ports:
    12: camera_trigger_front
    13: camera_trigger_rear
    5: led_green
    6: led_red
    11: siren_dnstrs
    7: siren_outside
    9: siren_upstrs
    25: strobe_dnstrs
    8: strobe_outside
    10: strobe_upstrs

Traceback (if applicable):

Additional information:
The switch states are recorded in the history component.

This issue still exists. Please fix.

I’m away for another 2 weeks. Will check when I return.

Closing as I don't use this any more and have no way of checking.


Support the custom configuration of the DMS format of DD-MM-SS.SS

Support FAA format by creating a custom configuration
Background
Supporting this format thru the use of our configuration options of a DMS coordinate will allow this addin to support the FAA format.
https://community.esri.com/thread/207791-faa-oeaaa-latitudelongitude-coordinates
Current State
Input

Output

Expected behavior
All coordinate types should accept all characters to allow users to configure the output format. This can be tested on DDM and DMS which currently are not supporting dashed (-) symbols.

@karlterrey Can you follow up with this user on GeoNet if possible to gather any requirements for this format?

Will do

Customer feedback from Hannah Shepherd at Apex Clean Energy, in VA<EMAIL_ADDRESS>The FAA has a mandated protocol for locations that are electronically filed with them.
Hannah’s organization is a wind developer and files locations of facilities with the FAA using the FAA’s OEAAA E-Filing system. Specifically they have issues with the DMS format needing to be in DD-MM-SS.SS for Lat and Lon of each facilities location. This can sometimes be in batches of 150 – 200.
A second important factor in their workflow is the elevation at each specific point. ESRI’s Extract Values to Points tool helps with that currently.
This would help in reinforcing that our work is non ITAR
_FAA_Template for coord conversion.xlsx

This should be able to be done by configuring the DDM output.

@saip please let me know if the updated content in the issue description clarifies the requirments.

added CT-fixed label since it was developed/delivered in #518.
added working labels for testing and doc

Recently rebuilt CC build 19 and MT build 122 to test

MGRS and USNG are not supporting dashes, or other special characters in Pro 2.3.

Using MT build 126, I confirmed FAA format for custom DMS notation is supported.
I also confirmed, as reported by @BobBooth, the add-in does not support dashes for MGRS and USNG notations.

Added new test for checklist

This issue represents a subset of cases for #347 and #348. This is is satisfied given the initial issue. A new issue needs to be logged as a response to this, as this issue does not account for Northern/Southern or Eastern/Western hemisphere coordinates.

#542 was opened to clarify the proper formatting for FAA coordinates.

@dfoll if the coordinates are not in the proper location, this issue has not been delivered. Reassigning to @CTLocalGovTeam . @saip please work with @dfoll for the guidance.

@lfunkhouser we need to find time to understand this format better, or have connected with someone who already does before any meaningful work can be done

Support for custom FAA format by using the map point tool was delivered. Issue #542 will need to be addressed in a future release to support the input/importing of FAA coordinates.

This issue was reintroduced with fix for #527, https://github.com/Esri/coordinate-conversion-addin-dotnet/issues/527#issuecomment-494955592 PRs #602 , #612 that disallowed "+/-" in formatting - Reopening.

Proposed fix for this is to improve the parsing logic that was put into place to fix issue #527. Details of proposed improvement listed in #527

CC build 145 brought back the ability to use a "-" as a separator between DD-MM or MM-SS.SS when creating custom formatting . A "-" in front of the DD is not allowed to be used when creating custom formatting. When using custom formatting the "-" will still be placed for S and W hemisphere if the "Show - for S/W hemisphere" check box is marked. See #600 for more details.

https://solutionsdev.arcgis.com/defense/what-is-new/ (pending this pr)
https://solutionsdev.arcgis.com/defense/help/coordinate-conversion/resources/june-2019-coordinate-conversion.htm
\...\CoordinateConversion\Dev\148\

verified what's new and release notes on solutionsdev, pdfs in CC build 148 and MT 316

i thought about this. the reason i put new labels on it and listed it again is in case a user has or does encounter with like 3.4

verified doc on MT 317, CC 154, and 3 topics on solutionsstg.


[Question] Sharing databases between dart and native/ffi code

Hi,
I'm just starting to get to know the drift, so I'm sorry if I'm asking the wrong questions.
I need to use the database from the dart and native/ffi code at the same time. And the dart part will only read from the database and the native/ffi will only write to it. Updates (writes) may occur periodically. It would be great if dart could get notifications from native about database updates.
Could you please suggest how I can solve such a problem?

You can generally share the database by using the same file path for it in Dart and the native side.
Unfortunately, reacting to updates is not easily possible. Stream queries are a drift feature not directly supported by sqlite3, and there is no general way to dispatch these updates. The only way would be to use platform channels or receive ports with the native side posting about the tables it's changing. Upon receiving such notification in Dart, you can use notifyUpdates on your database to update stream queries.

Thanks for the quick response.
As I can see now, notification of updates is not so easy to implement. I'll try your receipt or maybe use other synchronization methods.
Regarding the first question. I would kindly appreciate for an example of such a sharing of, for simplicity, an in-memory db. Can an in-memory database be used in this way at all? Must it be open in the single isolate or could it occupy several isolates?

You can't have them share an in-memory database unless they both use the same instance of the sqlite3 library. On some platforms (macOS and iOS) where we link sqlite3 statically, this is always the case. On others (especially Android), sharing an in-memory database only works when the native code runs in the same process and uses the same copy of sqlite3 as package:sqlite3 in Dart. If you can make that work, sharing updates is actually fairly easy as well. If you can't, the best option is using a temporary file which is opened by both your Dart code and the native layer.

We can pass a pointer to an open db from drift/dart to native/ffi code. And then periodically update the db from the native code. Will this scheme work, do you think? Will the updates be visible on the drift/dart side?

Right, that will actually make this a lot easier. This allows you to open a database like this:
  final rawDatabase = sqlite3.open(':memory:'); // or alternatively, a file path
  final sqlite3_struct = rawDatabase.handle; // is a sqlite3* and can be used as such

  final db = MyAppDatabase(NativeDatabase.opened(rawDatabase));

Here, sqlite3_struct is a Pointer<Void> which can be cast to a sqlite3* in C and used as a database. The best thing about sharing the underlying database is that you can listen for updates on most tables:
  rawDatabase.updates.listen((update) {
    db.notifyUpdates({
      TableUpdate(
        update.tableName,
        kind: switch (update.kind) {
          SqliteUpdateKind.insert => UpdateKind.insert,
          SqliteUpdateKind.update => UpdateKind.update,
          SqliteUpdateKind.delete => UpdateKind.delete,
        },
      )
    });
  });

That way, updates made natively should also update stream queries in Dart.

Thank you for a brilliant solution! It's exactly what I was looking for.


Is there anyway I can use style with scoped in scss?

Is there anyway i can use style with scoped in scss?. since I've found that the rules for sass loader allows only css module to be used. However, is there anyway I can at least overwrite some webpack rules of scss from using css module to using any style (scoped or global).

Hey @TimothyPrachasri,
you can modify the webpack config based on https://github.com/vuesion/packages/blob/master/packages/service/src/webpack/config/base.ts in ./vuesion/webpack.config.js.
In your case you have to replace the contents of the custom webpack config with:
/**
 * modify webpack config
 * @param {webpack.Configuration} config - webpack config
 * @param {string} target - build target, can  be "client" or "server"
 * @returns {*}
 */
const isProd = process.env.NODE_ENV === 'production';

module.exports = (config, target) => {
  if (target === 'client' && isProd) {
    config.performance = {
      hints: 'warning',
      maxEntrypointSize: 400000,
      maxAssetSize: 300000,
    };
  }

  config.module.rules[2] = {
    test: /\.scss$/,
    oneOf: [
      {
        resourceQuery: /module/,
        use: [
          'vue-style-loader',
          {
            loader: 'css-loader',
            options: {
              modules: true,
              localIdentName: '[local]_[hash:base64:5]',
            },
          },
          {
            loader: 'postcss-loader',
            options: {
              ident: 'postcss',
              plugins: () => [
                require('autoprefixer')({ browsers: ['last 2 versions', 'ie >= 11'] }),
                require('css-mqpacker')(),
                require('cssnano')({
                  preset: [
                    'default',
                    {
                      discardComments: {
                        removeAll: true,
                      },
                      zindex: false,
                      normalizeWhitespace: isProd,
                    },
                  ],
                }),
              ],
            },
          },
          'sass-loader',
        ],
      },
      {
        use: [
          'vue-style-loader',
          'css-loader',
          {
            loader: 'postcss-loader',
            options: {
              ident: 'postcss',
              plugins: () => [
                require('autoprefixer')({ browsers: ['last 2 versions', 'ie >= 11'] }),
                require('css-mqpacker')(),
                require('cssnano')({
                  preset: [
                    'default',
                    {
                      discardComments: {
                        removeAll: true,
                      },
                      zindex: false,
                      normalizeWhitespace: isProd,
                    },
                  ],
                }),
              ],
            },
          },
          'sass-loader',
        ],
      },
    ],
  };

  return config;
};

Let me know if this helps you.

The solution posted enables both css-modules and global/scoped css, in case you want to use existing components that rely on css-modules.

@TimothyPrachasri did it work?

Right now i’m trying to use css module as a workaround. If i change to
scoped scss, will tell you  asap.
On Sun, 21 Jul 2562 at 00:56, Johannes Werner<EMAIL_ADDRESS>wrote:

@TimothyPrachasri https://github.com/TimothyPrachasri did it work?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vuesion/vuesion/issues/422?email_source=notifications&email_token=AEBUAU24MDUSE22S2P75SGLQANGWFA5CNFSM4IELODYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2NTDUQ#issuecomment-513487314,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEBUAU3HO2LMNSSBOUY46ITQANGWFANCNFSM4IELODYA
.

closed, this feature is now included in @vuesion/service version 3.0.0-alpha.6


GetMatchingProductForID

I am interested in submitting a list of 20 ASINS to the GetMatchingProductForID.
I see a list is supported for "GetMatchingProduct" but is it possible to submit a list of ASINS for "GetMatchingProductForID"?

@mixmasterjt
Please refer to the http://docs.developer.amazonservices.com/en_IN/dev_guide/DG_IfNew.html .
This library only supports which has been provided by the AWS MWS.


Reimplement the uninstall script

Reimplement the uninstall script
Currently, the script deletes all the contents of the current directory (using the ./ notation). We have somewhat naively assumed that the script be run inside the source directory only, thus leaving room for potential risks.
Henceforth, if we were to execute it elsewhere, we would permit the script to carry out the deletion, unintentionally deleting files (thinking that only the files within the wlpr source folder would be deleted) which is, indeed, risky!
We would like to make the uninstall script directory specific, such that we explicitly state that only the contents within the wlpr folder (and the folder itself) shall be deleted, leaving no room for risky and/or unexpected behavior.
Note: we may use the .wlpr.json file to extract the path of the wlpr source.

The uninstall script has been successfully fixed from the develop branch, merged into main: 29d00eaed3c296a1a750365c3da51222d255107c


Fix auto login events assertions

Motivation
Fix wrong assertions in new user login events tests
Changes

Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] Relevant label (run-parametric-scenario, run-profiling-scenario...) are presents
[ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team

Your Pr is doing the same things as: https://github.com/DataDog/system-tests/pull/2568/files

Yep, I'll close mine


{ServiceBus} az servicebus: fix typos in help docs

Description
Fix typos
Testing Guide
N/A
History Notes
N/A

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

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

ServiceBus


Add more models for broader testing

[ ] Alpafold
[ ] GPT-J
[ ] Top-K HuggingFace Models

Perhaps a project is better for tracking this since it involves many parallel threads, each of which also may need to be solved as multiple tasks?

there is one already, we ideally want to make the board public.

The board is now made public 🎆


Capstone Project.

Description

Metrics Webapp is an application that provides information on the various crypto platforms.
Use React components.
Use React props.
Use React Router.
Connect React and Redux.
Write integration tests with a React testing library.
Use styles in a React app.

Screenshots

Thank you for the feedback


(PUP-6641) Fix Service integration test for Windows 10

Previously the integration test for the Service provider for Windows used the
lmhosts service as a test fixture.  However in Windows 10 the start type for
this service changed to Manual (Triggered) which now fails the integration test.
This commit changes the service used to DHCP which is the DHCP Client service,
which is always Automatic and exists in all of the supported Windows Puppet
platforms.
Note that this platform is not tested for integration tests but can easily be tested locally via VMPooler

:+1:

CLA signed by all contributors.

@glennsarti I think I must be missing it - I don't see in the test(s) where we actually require the service's startup type to be automatic?

The actual test is below;
https://github.com/glennsarti/puppet/blob/bcee048502c530c64d43c7e5cd953204286aa058/spec/integration/provider/service/windows_spec.rb#L34
Manual services come through as manual not true or false

Ah I see. It looks like the Windows service provider's enabled? method is actually intended to return three possible values, true, false, and manual so perhaps instead that test would be better updated to expect one of those possible values?
(https://github.com/puppetlabs/puppet/blob/master/lib/puppet/provider/service/windows.rb#L52)

I must admit, I don't really see the points of those tests.  They do not make me fell confident that the code is behaving correctly.

Yeah I can I see that. Perhaps the idea was just a double check that the contract/signature of the method isn't accidentally changed unintentionally, but it seems like possibly a better test would be to set the service status before hand or even just stub the underlying service status query (aka to return Win32::Service.get_start_type(Win32::Service::SERVICE_DEMAND_START)) and then ensure that the enabled? method actually returns the right thing..

After investigating the spec tests test whether specific values from FFI map to the correct provider values.  This integration test is merely a smoke test against a real Windows operating system i.e. Does calling the actual Windows OS generate a valid value (not necessarily a correct value).

👍


I'm having an issue with the Meetup API...

as instructed, I've add this snippet to my page
to display and work this code: RSVP

when I press it I get this error:
Invalid Authorization request
If you were directed here by another application, contact its developer.
I see the code works here but not on site.  www.itac.org

you need to make sure the redirect_url for the selected oauth consumer matches the url you are hosting the page on

Hey @softprops, I'm trying to make authorization request using OAuth2.0 and have followed the API carefully. However, I'm not able to get the code from this request https://secure.meetup.com/oauth2/authorize. And the problem is, I think, that I couldn't set the redirect_uri to http://localhost:5000 when I created the consumer application. Do you already have an idea of how to proceed?


ADB Root Requirement (su alternative?)

For btsnoop-extcap,
My device is currently rooted but running a production build and cannot run "adb root". Would an implementation be possible that uses su prefixed commands instead?
Thanks!

Sure, it makes sense to add that.
I don't have a device in that configuration to test against, can you confirm whether the below commands work?
adb shell su cat /data/misc/bluetooth/logs/btsnoop_hci.log > /tmp/btsnoop.log
wireshark /tmp/btsnoop.log

The following works on my system:
adb shell su -c 'cat /data/misc/bluetooth/logs/btsnoop_hci.log' > btsnoop.log
Wireshark.exe btsnoop.log

Currently, I have just swapped the cmd args within shell to the following and it is working (after modifying the root checks):
cmd.args(["-s", serial, "shell", "su", "-c", &("'" + command + "'")]);

I also had to add character returns to the mode matching, not sure if that is related:
match output.stdout.as_slice() {
    b"full\n" => Ok(BtsnoopLogMode::Full),
    b"full\r\n" => Ok(BtsnoopLogMode::Full),
    b"filtered\n" => Ok(BtsnoopLogMode::Filtered),
    b"filtered\r\n" => Ok(BtsnoopLogMode::Filtered),
    b"disabled\n" => Ok(BtsnoopLogMode::Disabled),
    b"disabled\r\n" => Ok(BtsnoopLogMode::Disabled),
    _ => Err(anyhow!("Unknown BTsnoop log mode")),
}

Thanks for looking into the issues. I think I have fixed the issues you described in the rooted branch https://github.com/mauricelam/btsnoop-rs/tree/rooted, and I am now setting up on Windows to verify.


Support conda 4.4+ (for SGE)

Backport of PR ( https://github.com/nanshe-org/docker_nanshe_workflow/pull/80 ) for SGE.
Refresh the activation/deactivation procedure to be compatible with conda 4.4+.

The failure here is not reproducible outside of CI. Also the fact it doesn't show up on the sge branch (only here) is weirder still. 😕

Given this can't be reproduced locally and we can't seem to login to the CircleCI worker using ssh, simply raising issue ( https://github.com/nanshe-org/docker_nanshe_workflow/issues/86 ) and pushing ahead. It's not a great place to be in. However if this builds on Docker Hub, this will unblock testing against DRMAA locally.


Allow returning a workqueue somehow associated with an endpoint

Build on #7209
Will build upon this to get delayed writes going

Subsumed by #6737


RoadPosition and RelativeRoadPosition Implementation

Description
In this PR i have changed

RelativeDistanceCondition: longitudinal, lateral distance calculation. But I tested this for only roads with left lanes and right lanes, but not sure about junctions.

RelativeRoadPosition and RoadPosition (new implementation): This is tested and work fine for me. If you find any cases where it does not work, let me know, I will fix it.

If anything is not clear in the code let me know, I will try to explain.
I am preparing diagrams for use case and will share it asap.

Where has this been tested?

Platform(s): Ubuntu 20.04
Python version(s):  3.8
Unreal Engine version(s): -
CARLA version: 0.9.11

Possible Drawbacks
None, I don't see any conflicts with existing code.
Regards,
Sagar GV

This change is

Hi, I have made the requested the changes.


Aggregate phenotype probabilities across well

In this pr, the phenotype probabilities both the shuffled and final models are aggregated by well. I may decide to refactor this code to improve runtime. Otherwise please let me know if you have any suggestions.

Hey @gwaybio, it looks like the tracked columns should be included now. If there is another column I should include, just let me know. Otherwise, I will go ahead and merge this pr.

Thanks @gwaybio, merging now!


Prevent default commands from modifying readonly TextEditors

The readonly property prevents users from interacting with a TextEditor by typing characters, but other default commands (such as core:backspace, core:paste, core:transpose...) still modify the buffer.
If external packages attempt to modify a readonly TextEditor, let's allow this, but require them to pass an option to indicate that they explicitly intend to, and throw an Error otherwise:
editor.setReadOnly(true)
editor.insertText('okay', {bypassReadOnly: true}) // Works
editor.insertText('oops') // Throws an Error

Fixes #16736.

Why throw errors? Why don't just ignore the input?

That's the other option, yeah. I waffled a bit but I wrote this to throw an error because it seemed like the more informative thing to do rather than fail by silently doing nothing. Also, read-only editors are a fairly recent addition.

The packages that use read-only editors don't have control over the packages that alter editors globally.

That's a really good point.
Maybe I should change it to silently ignore by default, but throw in spec or dev mode? That may be a nice alternative to give package developers a chance to notice the failure case, but not give package users a bad experience.

Noticed, this change impact is big for vim-mode-plus.
I noticed this issue at most vim-mode-plus's operator got error.
Before #16294, vim-mode-plus have been using editor.component.setInputEnabled(false) to achieve normal-mode which allow h, j, k, l to move cursor instead of inserting h, j, k, l chars.
But after #16294, setting editor.component.setInputEnabled(false) is now equivalent to calling editor.setReadOnly(true).
Now, this PR, we forced to set {bypassReadOnly: true} for every mutation command if we want to mutate readOnly editor.
vim-mode-plus does lots of mutation in normal-mode through operators such as p, P, surround, upper-case(g U) and lower-case(g u)...
After this PR, all operators now need to add {bypassReadOnly: true} options when calling text mutation function.
But doing so is still not perfect.
Since

Before this PR, normal mutation commands or package(which is completely not related to vim-mode-plus) was still being able to change text in normal-mode(readOnly editor) which was useful for some workflow(user can change text without explicitly entering insert-mode(readOnly=false).
But after this PR all these commands no longer work unless each package author add bypassReadOnly: true option.

I understand this is what you wanted to do for some feature like GitHub package's readOnly editor, but for vim-mode-plus's point of view, I want disable all this bypassRedOnly: true enforcement.
I just want to allow any mutations without specifying bypassRedOnly: true.
Since I want normal text transformation package is usable in vim-mode-plus's normal-mode without asking each package author to set bypassReadOnly: true options.
I want editor's textInput being ignored on normal-mode but still want to allow mutation without explicitly specifying bypassRedOnly: true for vim-mode-plus enabled editor.
I want something like editor.setDefaultBypassReadOnly(true) and want to call it for vim-mode-plus enable editor to revive previous behavior.
What do you think?

@smashwilson , @maxbrunsfeld give me some comment, what do you think?
This is very critical breaking change for vim-mode-plus and other package was used with vim-mode-plus.

Hey @t9md, it was my fault that there's been a delay in getting back to you about this. We discussed it in a team meeting yesterday and I was supposed to summarize the thoughts for @smashwilson and got side-tracked. I've done so now and we're talking over what the best approach is and we'll let you know what we figure out :+1:

Thanks for your help @lee-dohm and am glad to hear your team discussed about it!

Btw one idea for this is to introduce soft-read-only mode which basically prevent insertion of text through key typing but can change text without special bypass instruction.
Which is enabled by editor.setReadOnly(true, {soft: true}) something like that.

Btw one idea for this is to introduce soft-read-only mode which basically prevent insertion of text through key typing but can change text without special bypass instruction.

👍 This matches what I was thinking. I feel like the name "read-only" doesn't describe what you actually want to accomplish - you want to take responsibility for keyboard input, but you still want the editor to be modifiable.
I'll file a new issue to track this.


I ran a library on OpenCV 3.2.0

I compiled the library using cmake specifying the path to opencv 3.2.0. At the first compilation, I got an error saying that only opencv 2.4.x is supported. After that, removing a few lines in the condition of cmakefiletxt the library was compiled and I was able to collect it in the Visual Studio 17
In order to run the sample code, it was necessary to change the work with the camera very little, because at startup there were always black windows.
I checked all the algorithms. All work. So 3.20 can be used with this library

I can confirm successful compilation of the library with OpenCV 3.2.0 (repository) on Linux Kubuntu 18.04. The example code required some corrections -- after that all the algorithms worked.

Hi @vakikrin @piotr-szczypinski , thanks for the update!
I merged a pull request that remove OpenCV 3 restriction, and fixed some issues.
Could you validate if this new version is working well for you? Thanks in advance

Hello, from 2021. I'm using (or at least tryiing to) this lbp library project for my Thesis, but it seems i cant get it to work. Could someone assist me, my email is<EMAIL_ADDRESS>and im in dire need of help...

Hello @nanadaime ,
I developed this library together with @carolinepacheco .
How can we help you?

Hello sir,
I'm trying to use your library for my thesis, and I have to apply different
lbp variants on images stored in a folder. The idea behind it, is to take
for example, the DRIVE dataset and apply different lbp variants on it, on
all 40 tif images, and then feed them on a pre-trained neural network, and
observe it;s performance, based on the different lbp algorithms used. Until
today, i haven;t been successful in making the changes in the code, so that
your library modifies more than one picture, and that's where I need your
help...
Sincerely,
Bill Kouris, from Greece...
Στις Τετ, 29 Σεπ 2021 στις 11:59 μ.μ., ο/η Andrews Cordolino Sobral <
@.***> έγραψε:

Hello @nanadaime https://github.com/nanadaime ,
I developed this library together with @carolinepacheco
https://github.com/carolinepacheco .
How can we help you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/carolinepacheco/lbp-library/issues/5#issuecomment-930538497,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALPPCEPUFWDLXQATTVT2KRTUEN4Z7ANCNFSM4FKVSEHQ
.


Toggl Mac app crashing when a running timer is changed

💻 Environment
Platform: macOS
OS Version: OS X 10.14.3
Toggl Version: 7.4.330
🐞 Actual behavior
A user reported that his desktop app is crashing every time they try editing a running timer. I asked them to remove local data and gave them a link for a fresh install of the latest stable version and the issue persists. They sent a crash report, you can find it HERE.
💯 Expected behavior
They should be able to edit a running timer without crashing the app.
Intercom case:<PHONE_NUMBER>1

From what I see in the crash log,
There were 511 execute calls on this method https://github.com/toggl/toggldesktop/blob/4f44d7d8374890d474f0cf5972451f6db2c918c6/src/toggl_api_private.cc#L147
Technically, this func will call recursively to all nodes of Workspace, Tag and Client list. Thus, I assume that the use has large number of tags and during manually deleting those pointers, it crashed somewhere. Or worst situation that there was a loop recursive, so the app is terminated by the OS.
IMO, I suggest that we could use some sort of smart pointer, such as std::shared_ptr, so the list will be automatically released when it's out of the scope.

This could be related to the race condition issues I've discovered last week as well. I'll add it to the library overhaul and check it out in detail tomorrow.


Add pure op-based data types

Left some feedback, but looks good overall.

Please see changes in the API in #33 and incorporate them into this work.


Increase robustness to hang-ups when shutting down arbiter

Noticing some senarios where the core application has exited, but cleaning up connections hangs up somewhere.
It seems to be related to restarted connections

Added logging to connection manager here: https://github.com/vkottler/runtimepy/pull/173.
That should at least clarify whether or not the problem is related to that entity.

Waiting to get more runtime with this to see if intermittent unit testing hanging still occurs.


Adding support for end to end SSL and SSL policy in application gateways

@azuresdkci add to whitelist

cc @DeepakRajendranMsft - please sign off

thanks @vinatara !


[Rel #1] index SQLA hybrid properties

This allows to specify SQLA hybrid_property (proxied and managed
computed field) in __searchable__ list to be indexed. Computed nature
of this property limits underlying schema field to be TEXT and
sorting by it has no sense. See test for example.

I refactored test suite as well to make it more modular should more tests be added.

Thank you for your pr.It looks great,but hybrid_property could be other column type,such as INTEGER or DATETIME,I think it's incorrent that all  hybrid_property are set  as TEXT.
Add an optional constant to allow some users to customize schema  may be is a better way.

I don't see any way to specify type for hybrid_property, at class level it uses db.func() that works with underlying database data types and the result is implementation dependent and the exact type is undefined when creating schema - it is defined later, at instance level. TEXT seems to be safe common denominator, and in other cases __whoosh_schema__ attribute needs to be specified for Model subclass.
Anyway, I will try to annotate these properties with something that may be usable as type hint.

well, you are right,I will check it and add to master branch later.


test: Automated tests for AForce App Migration

Description
Implements the automated script validation for Migration scenarios of AForce Application via stubbing.

Export, Import app & verify

Fixes #11723
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Tested on Fat container - release
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: test/AForceAuto

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
55.82 (-0.01)
37.16 (0)
35.21 (0)
56.14 (0)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

/ok-to-test sha=6140e54

/ok-to-test sha=8fcfc25

/ok-to-test sha=ac9e01f

/ok-to-test sha=387ba6b

/ok-to-test sha=d3acdb8

/ok-to-test sha=b1cd8a8

/ok-to-test sha=bbcb046

/ok-to-test sha=fb01544

/ok-to-test sha=8321bcd

/ok-to-test sha=0c6afc4

/ok-to-test sha=0e4d847

/ok-to-test sha=827dddb

/ok-to-test sha=dccd26b

/ok-to-test sha=b469f0d

/ok-to-test sha=89f6c19

/ok-to-test sha=b97f25e

/ok-to-test sha=451da24

/ok-to-test sha=bf8b5a4

/ok-to-test sha=d803377

/ok-to-test sha=c54455f

/ok-to-test sha=2cc1433

/ok-to-test sha=934f392

/ok-to-test sha=934f392


[Request] Add Material Symbols outlined

Derp, I just realized I couldn't replace my Material Symbols icon font because I'm using the outline version. Can you add those, along with the different symbol weights?
Something like:
import { IconHelp } from '@iconify-prerendered/vue-material-symbols/400/outlined'

This set containse outlined version of all icons


⚠ Samsung has degraded performance

In 4aed159, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2657 ms

Resolved: Samsung performance has improved in a5b9d30.


Add cmake install

I have added cmake install commands to do a clean install of the library.
I have not used Conan before, but I think it can call cmake to do the installation.

I tested this out, but found a couple of issues:

the generated config file is named MoreConceptsTargetsConfig.cmake, but it should just be MoreConceptsConfig.cmake (now I have to do find_package(MoreConceptsTargets)).
the target path CMAKE_INSTALL_PREFIX/lib/cmake/ for the .cmake files does not seem to be the right one, as the config is not found by default when I do system installation, but must set -DCMAKE_PREFIX_PATH=/usr/local/lib in the consumer project.

Installation with no additional arguments makes find_package(MoreConcepts) work now.

Installation with no additional arguments makes find_package(MoreConcepts) work now.

Had to revert as it broke the build.
https://github.com/MiSo1289/more_concepts/runs/8295548432
Create a new PR if you have a fix (or I'll try to look at it later).


fix: update l10n files and fix typo in en-US.json

Description
Another round of l10n updates here, plus fixing a typo I accidentally introduced by a mistaken manual merge.
Task Item
#minor

Coverage decreased (-0.005%) to 55.689% when pulling 1e586121c0d8e4d90674ccc90aa5c767a9976ea5 on beyackle/l10n_2020-10-19 into 55fea4cc5632dbaf96908ed6b1de88ee6fa2e913 on main.


Error installing Mollie payment gateway package

I got two webshops where I can't install the Latest Mollie payment gateway. Any ideas what the problem could be? And if it is a problem related to the latest C5CS version, is it possible/easy to downgrade without losing products / data?
View the problem:
https://github.com/concrete5-community-store/community_store_mollie/issues/3#issue-641984198
Works: (C5: 8.5.2   C5CS:2.1.10   Molli:0.0.8)
Does not work: (C5: 8.5.3  C5CS: 2.2.7  Mollie: 1.0.0)
Does not work: (C5: 8.5.2  C5CS: 2.2.2  Mollie: 1.0.0)
Also tried to install old version of mollie (0.0.8) (wanted to know if old version maybe work). Gives me an error:
An exception occurred while executing 'DROP INDEX IDX_C0DB918CECC024CA ON molStoreOrderTransactions': SQLSTATE[HY000]: General error: 1553 Cannot drop index 'IDX_C0DB918CECC024CA': needed in a foreign key constraint

I don't use Mollie myself, but I'll give this a test today myself and see what I can find.


Add more design tools

Add Adobe XD, Marvel, and uxpin to design tools

Thanks for your contribution!!! 👏👏👏


Does this library support multi-bots?

I'm developing an application with multi bots running at the same time. I've noticed that this library stores all states in the db tables without indicating the Bot associated with it, for e.g. the 'Conversation' table.
Is this bot compatible with multi-bots? what happens the same command issued to different bots by the same user?

You can use different hook.php and same codebase. Just change SQL connection information info in hook.php for a bot.

Maybe it comes to the point that I've 1000s of bots or bots on demand. Its not a good idea to use a separate database for each bot.

@ahmadazizi You can have a single database with different tables, just set the table prefix when enabling the MySQL connection ($telegram->enableMySql($mysql_credentials, $BOT_NAME . '_');).
Because the framework is very static at the moment and only 1 bot can run in parallel within the same PHP process, you can only use multiple bots if you run multiple instances of the code. Hopefully this will change in the future.
@jacklul What do you think about having the bot linked in the tables too to allow multiple bots use the same tables? We could add a new table called bots and just store bot_name and an ID (and possible open up the world of bot settings here!)

Adding multibots suport would I believe change db schema for all tables, alternatively we could have second DB handler build for multibots and primary one for one bot only. This could open up a road for supporting different DB types.
Also, why not changing $BOT_NAME to $BOT_USERNAME ? Makes more sense and can prevent confusion!

Thanks @noplanman, this approach would be fine for just a few bots but is absolutely an unnormal database scheme; just suppose a platform that lets users to add their own bot API KEY and they want each user receive updates with their own bot because of telegram limits or other considerations.
I really believe that this framework is strong enough to be used in large-scale enterprise applications but the lack of multibot support is the only drawback.
@jacklul I wont believe this could be so complicated. Just adding the BOT_USERNAME to all tables and considering this in queries would be enough. For simplicity I would use part of bot API KEY as BOT_USERNAME

For me I've to say thats absolutely is possible to run multiple bots without any changes to the code with the same instances. You dont neet do double hook.php or something else, you only need some config arrays and a unique get parameter in the url. You even can handle the messages per bot with only one extra database table where you store the botid with the message ids which he has received. There is absolutely no need to change the entire library for that. My bots can even handle an unlimited number of different bot subscriptions for the same user without any problem. You need for sure some lines of own code and some extra database tables, but it's working perfectly.

@tchibomann can you give some code samples how you've implemented this?

@ahmadazizi I've build this on a very simple basis, I have to recode this because I currently have this splitted into multiple hooks. This was my first way for using this script for multiple bots. But I did it with the intention to simplify this into one file. Maybe I have next week some time to finalize and show what I mean.

@tchibomann so that will be great and so appreciated if you show us how you did.

Just adding the BOT_USERNAME to all tables

@ahmadazizi I don't think that's a good idea, as there would be a whole bunch of redundant data. Better to have a separate table called bots, which have an ID, name, maybe some settings too. That is also easier if someone chooses to rename their bot but want to keep the data.
@tchibomann Looking forward to learn from your solution 👍

I don't think that's a good idea, as there would be a whole bunch of redundant data.

@noplanman  You're right, I was looking at it as some temporary data tables.
Are you planning to implement this feature? and if yes does it involve major changes in schema or it would be a quick enhancement update?

@noplanman @ahmadazizi that was my idea, reep away redundances. So my idea was a separate table for bots and a link-table for bots and messages, but last may be inefficient, needs unnecessary more space. Better is a new column for the bot id in all necessary tables.
But I'll show my solution in some days :)

You can create also database on demand if you really need to create on the fly new bots. Is the easiest thing. Is to much complicated add a field with the botname


Terminal ignores $SHELL var

Hi,
When using Jupyterhub + sudospawner, $SHELL var looks ignored. I have users set with bash and shell launched is sh.

Closing this issue as it's been inactive for several months. Please feel free to open a new issue at the jupyter/help repo or reopen this issue. Thanks! :sunflower:


Jetty dependency update

Jetty 8 is End of Life and they recommend to use Jetty 9.

In the commits referred above I have made the necessary adaptations to update to Jetty 9.
All tests pass locally, but I still need to make it JDK 7 compliant to avoid your Travis CI from failing.

Ok, there is no other way but to use JDK 8.
Seeing that JDK 7 is no longer maintained and that Oracle recommends to update to JDK 8, I was wondering whether you could consider to update the JDK used by your Travis CI.


permission xxxxx set to undefined in context global

Could you tell me why I would see the following?
lp user f098c794-f41a-4edd-a536-b638b26b748e permission checkinherits jobs
[LP] smmmadden has permission jobs set to undefined in context global. (inherited from self)
I have two groups (default and owneradmin).  OwnerAdmin inherits from default.
I'm using LuckPerms-Bukkit-4.0.18.jar on Spigot Server 1.12.2.

nevermind - I figured out that this means it isn't set to true or false and possibly not set at all.  The exact behavior of the permission/command may or may not work depending on the developers "default" value when not set which could be true or false.

On Bukkit, it also depends on whether the user is opped - but yes, you're quite right. :)


Violation link not found when deleting crew member

When deleting a crew member from a report, removeItemClicked in CrewView.swift fails to find the violation that's referencing that crew member

 Raise new issues resulting from using/testing the apps


Airflow dev

Hello @JINO-ROHIT! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file dags/dag_pipeline.py:

Line 17:1: E302 expected 2 blank lines, found 1
Line 17:80: E501 line too long (88 > 79 characters)
Line 27:80: E501 line too long (84 > 79 characters)
Line 33:1: E302 expected 2 blank lines, found 1
Line 48:80: E501 line too long (93 > 79 characters)
Line 50:80: E501 line too long (83 > 79 characters)
Line 53:10: E201 whitespace after '('
Line 57:80: E501 line too long (90 > 79 characters)
Line 79:80: E501 line too long (88 > 79 characters)
Line 101:80: E501 line too long (103 > 79 characters)


Fix external border rendering and rename BorderRadiusWorld to BorderRadiusRelative

Fixes #54
Fixes #52
Adds tests for BorderPlacement=1 to assert correct rendering.

There's one more thing I need to fix in this before merging. Didn't realize when I hit the "ready for review" button. The changes I made cause text to render larger than specified when there's a border placed outside the shape. I'll fix it then comment when it's ready for review.

Update: This PR is now ready for review.

Thanks for the review, I'll take care of these points.


[FLINK-1687] [streaming] Syncing streaming source API with batch source API

It is important to keep the streaming and the batch user facing API synchronised in regard of the source and sink functions (even if the inner workings are different).
Most of the source functions are done here. Because this is not a high priority issue, sink functions will be done later on.

Would it make sense to add a API completeness checker between the streaming and batch java API (similar to the java/scala API completeness checker?) The whitelist is probably a big bigger in that case, but somebody changing something somewhere would at least notice that the APIs are out of sync.
Does this change actually break with the current (0.8.x) methods? or is it just adding new methods (it seems you're relocated some methods)

Ah, it would also be good to rebase this PR to the current master to get some feedback from travis

I completeness checking between the streaming and batch java API sounds to be a great idea. I wouldn't force syncing in any other way, because there can be source methods that are meaningless in the streaming or in the batch context.

Only new methods are added. There are two versions of generateSequence: a new parallel one, as in the batch API, but the old, sequential one is also useful for testing.
The new methods were reordered to follow a more logical ordering.

Okay, cool.
Let us know when the pull request is ready for another round of reviews.

Hey Peti,
I added a refactor commit that reworked the way types are handled in the sources a little bit.
https://github.com/apache/flink/commit/6df1dd2cc848d0a691a98309a3bb760f9a998673
Can you please rebase on that one?
It should make things nicer.
Gyula

@gyfora
Of, course, no problem. It will make the source API cleaner.
Peter

The change looks good in theory but there are some details that need some reworking.

Thank you Robert. Unfortunately, I didn't have much time this week to
review the code. As soon as I'm done with the storm compatibility examples,
I will be able to fix it up.
2015-05-20 11:25 GMT+02:00 Robert Metzger<EMAIL_ADDRESS>
The change looks good in theory but there are some details that need some
reworking.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/flink/pull/521#issuecomment-103822246.

Started working on this: rebased after Aljoscha's change, and found some minor issues - fixing those.


remove KUBEVAL_BUILDER_IMAGE from generic defaults

The defaults file is intended for generic default values that can be
shared for Dockerfiles across multiple functions. In this case
KUBEVAL_BUILDER_IMAGE is specific to the kubeval function and can simply
be defined in the dockerfile for that function.

Can I get a review on this?


Authorising entire path nodes

MATCH (result_person2:`Person`)
  WHERE (result_person2.uuid = $result_person2_uuid)
  WITH result_person2
  OPTIONAL MATCH `role_path`=(result_person2)-[`role_rel`:`role`]->(`role`)
  WHERE (`role`:`Role`)
  WITH
    result_person2,
    apoc.agg.slice([`role_rel`, `role`] , 0, 1000) AS `role_collection`
  OPTIONAL MATCH `role.persons_path`=(result_person2)-[`role_rel`:`role`]->(`role`)<-[`role.persons_rel`:`role`]-(`role.persons`)
  WHERE
    (role.persons.active = $role_persons_active) AND
    (`role`:`Role`) AND
    (`role.persons`:`Person`) AND
    (`role` in [i IN `role_collection` | i[1]])
  WITH
    result_person2,
    collect([`role.persons_rel`, `role.persons`] ) AS `role.persons_collection`,
    `role_collection` AS `role_collection`
  OPTIONAL MATCH `role.persons.parent_path`=(result_person2)-[`role_rel`:`role`]->(`role`)<-[`role.persons_rel`:`role`]-(`role.persons`)-[`role.persons.parent_rel`:`parent`]->(`role.persons.parent`)
  WHERE
    (`role`:`Role`) AND
    (`role.persons`:`Person`) AND
    (`role.persons.parent`:`Person`) AND
    (`role` in [i IN `role_collection` | i[1]]) AND
    (`role.persons` in [i IN `role.persons_collection` | i[1]])
  WITH
    result_person2,
    collect([`role.persons.parent_rel`, `role.persons.parent`] ) AS `role.persons.parent_collection`,
    `role_collection` AS `role_collection`,
    `role.persons_collection` AS `role.persons_collection`
  RETURN
    result_person2,
    [`role_collection`,`role.persons_collection`,`role.persons.parent_collection`]

Codecov Report
Base: 82.83% // Head: 91.54% // Increases project coverage by +8.70% :tada:

Coverage data is based on head (ea15e8f) compared to base (ecc69a1).
Patch coverage: 88.88% of modified lines in pull request are covered.

:exclamation: Current head ea15e8f differs from pull request most recent head e5c2152. Consider uploading reports for the commit e5c2152 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
+ Coverage   82.83%   91.54%   +8.70%
==========================================
  Files          10       10
  Lines         268      272       +4
==========================================
+ Hits          222      249      +27
+ Misses         46       23      -23

Impacted Files
Coverage Δ

...y/query_proxy_eager_loading/scope_eager_loading.rb
93.10% <50.00%> (+43.10%)
:arrow_up:

lib/active_graph_extensions.rb
100.00% <100.00%> (ø)

...extensions/node/query/query_proxy_eager_loading.rb
94.25% <100.00%> (+11.49%)
:arrow_up:

.../query_proxy_eager_loading/association_limiting.rb
90.24% <100.00%> (+2.43%)
:arrow_up:

...e/query/query_proxy_eager_loading/enhanced_tree.rb
100.00% <100.00%> (ø)

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

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


Analog is closed

Would it be possible to make it so that it says that Analog is closed (so that it takes vacations and exam periods into account)?

Sure, we could add a list of closed days. However I don't know if there is a "smarter" way since the dates vary every year

Sure, we could add a list of closed days. However I don't know if there is a "smarter" way since the dates vary every year

I will try to ask AnalogIO if a smarter way can be implemented

any news on this?

Unfortunately, I am not getting a response. But since the dates vary every year as Philip pointed out, it seems to be a lot of extra work to keep updating the dates often.


IN PROGRESS: Remove dummy webpage

Clicking on a person card will now simply show the default page, instead of loading the dummy page.
This addresses #129.

Coverage remained the same at 86.974% when pulling e170745adbf8ae18cb2534c41fa446d29ca6ff18 on zioul123:Webpage into 6a6503d9c2ee77e95017d23e79e36745361e0981 on CS2103-AY1819S1-W13-1:master.


Variables (extracted from a discriminated union object) lose narrowed types after being exported

🔎 Search Terms
discriminate union export destructure destructuring
🕗 Version & Regression Information

This is the behavior in every version I tried (including the nightly one), and I reviewed the FAQ for entries about (export, destructure, discriminate, union).

⏯ Playground Link
https://www.typescriptlang.org/play/?#code/GYVwdgxgLglg9mABAQwhApgBygZSgJxjAHMB5MAGwE8AKAfQA8AuRAZwKOIEpEBvRAL4AoUJFgIUaLFAByIChQCCYACYBVVemBF0K8tXrNEYeRUQAfROBVadKnv2Ejw0eEgC2IKCGQKqAUTBkACMKXQAFZBh8AB4AFQA+GgA3XxB0FjiuFl4hRER0INDdFgJ0gBo8xFSKdJYZBDkFELD4hMqBCz4qwpaSxGBfVnRK-Jq641Mu61swXQ7u-JhgRBoTBUQAXm3qtPQLSxntOZUtnfH0Byr8-HRvfCR+XuKVFkGKYfLd2ozJjYEANxVTroD77XL5G53EAPPgFIphV6IMojb7pQRA-LCJwQBDsOHPREsfxfC4sABqgi2iE83l81ECfRUkWiNAARKw4O59uxCCQ0eg2VwgUJlqt-Fd8uSAYgAPSyxAAC3Qt0QMCgatYbA4JC+cAA1lVUBhsHg+WRKLRycKhCCwYtENK5Qrlar1Zq-mZDppjvNEAajVJsE0lKoNDZfXpLTRrUCnOgGJg4PgNbiwPingi-RcqZsaV4fH5GS8Wfh2ZzudrzQKhUJRSsaITdJKBTL5UqVft3TAtbzOF11l6rD67F8AO7oaIqQMm3A6i0GC7C52IfwAJTXpDXtoK9ohrZXrq7GqGcA9ff5lkH0xHJxn0hDynUt90+loS7bCvXm+3wiAA
💻 Code
function acceptStringOnly(_x: string) { }
function acceptNullAndUndefinedOnly(_x: null | undefined) { }

function mutuallyEnabledPair<T>(value: T): {
  enabled: true,
  value: NonNullable<T>,
} | {
  enabled: false,
  value: null | undefined,
} {
  if (null === value || undefined === value) {
    return { enabled: false, value: null };
  } else {
    return { enabled: true, value };
  }
}

const { enabled: E, value: V } = mutuallyEnabledPair("some string value");

if (E) {
  V; // here it is string, ok
  acceptStringOnly(V);
} else {
  V; // here it is null | undefined, ok
  acceptNullAndUndefinedOnly(V);
}

export const { enabled, value } = mutuallyEnabledPair("some string value")

if (enabled) {
  value; // here it is string | null | undefined, weird
  acceptStringOnly(value); // ERROR
} else {
  value; // here it also is string | null | undefined
  acceptNullAndUndefinedOnly(value); // ERROR
}

🙁 Actual behavior
When variables are destructured from an object of a discriminated union type and they are exported, those variables lose their narrowed types defined by the discriminated union type. Meanwhile, if the variables are NOT exported, their narrowed types will be preserved, which is the expected behavior, whether the variables are exported or not.
🙂 Expected behavior
Preserve narrowed types for variables that are:

Destructured from an object of a discriminated union type
Exported.

Additional information about the issue
I guess this issue may be more or less related to this one.

Simplified repro:
declare function mutuallyEnabledPair(): {
  discriminator: true,
  value: string,
} | {
  discriminator: false,
  value: null | undefined,
}

export const { discriminator, value } = mutuallyEnabledPair()

if (discriminator) {
  value;
}

In the function getCandidateDiscriminantPropertyAccess, symbol.valueDeclaration is undefined in the export version & isn't undefined without the export.
This leads to the narrowing using the discriminant to not work.


[SDL 0274] add preferred FPS to VideoStreamingCapability

This change resolves issue #222 : [SDL 0274] add preferred FPS to VideoStreamingCapability.

@shiniwat please tag me when this is ready for review

@jordynmackool,  this PR is ready for review since last March. Thanks.


[UnitaryHack] Added Rotosolve optimizer

PR adds the Rotosolve optimizer from the paper from Ostaszewski et al. to Lambeq.
Addresses #85.

Hi @Shiro-Raven. Please check the initial github issue. I've created an example notebook to test your optimiser.

@Shiro-Raven Hi, please use the notebook provided by @Thommy257 in the issue to check your optimiser, so we can proceed and assign the task to you.
Note that currently one test is failing, but it's not relevant to your PR -- we'll have a look at it at some point.

@Shiro-Raven we've fixed the test error, so if you merge those changes into your PR all the tests should now pass.

@Shiro-Raven Please address any remaining comments by tomorrow 13/6, last day of the hackathon.

Just to clarify here, hackers have until June 20th to fully finalize contributions to be eligible for a unitaryHACK bounty. The June 13th deadline was for submitting pull requests.


Proposal: Add IsReadOnly property to NumberBox

Proposal: Add IsReadOnly property to NumberBox

Summary

The TextBox control has IsReadonly but the NumberBox doesn't.
Rationale

There are cases when you want to switch if the NumberBox is read-only or not. We can make the NumberBox read-only with the IsEnabled property but it makes the look grayed out and makes the value unselectable thus unenabled for copying the value to clipboard.

Scope

Important Notes

Open Questions

This was already proposed in #2204 three years ago with no progress yet. There is a workaround using the visual tree but it does not work correctly as shown in #8752.

@danielgruethling Thanks for pointing it out. Closing this as duplicate.


Print values inside fused attention kernel

I am trying to debug my fused attention code and would like to print some intermediate values such as the ptrs as well as the values loaded from the pointers. Is there a way to do this? Thank you!
code:
    off_k = (
        off_hz * stride_qh + offs_n[:, None] * stride_kn + offs_d[None, :] * stride_kk
    )
    k_ptrs = K + off_k
    # loop over k, v and update accumulator
    for start_n in range(0, N_CTX, BLOCK_N):
        start_n = tl.multiple_of(start_n, BLOCK_N)
        # -- load k and mask ----
        k = tl.load(k_ptrs + start_n * stride_kn)

Currently there's no way to print values from a kernel, but you can copy the values to a temporary buffer and print it out on the CPU.
I will work on the print function later.

How can there not be a print function that works?
I'm trying to debug a memory corruption CUDA problem and can't find a way to get at the offset from out_ptr0 being written to.
There is a print function but it can't handle a variable!?
@Jokeren How do I do the copy to a buffer idea?  I can pass an additional debug out value to triton_mm to store data into but can't figure out how to declare it properly nor how to copy a variable's value into it.
torch.optimize results in CUDA error: an illegal memory access was encountered
and I believe it comes from:
tl.store(out_ptr0 + (xindex + tl.zeros(mask.shape, tl.int32)), tmp1, mask)

That's a print function now. Will improve it a bit likely this week.
Try tl.printf("", <variable name) for now

https://github.com/openai/triton/pull/1143

Try tl.printf("", <variable name) for now

It sometime works and sometimes doesn't and I don't know why.  When I call the ?jit? ?kernel? function named triton_mm, in a simple run harness, the prints I added work.  However, when the same triton_mm function is called from a complex app they don't print anything.  However, 'print("string")' does work before and after my tl.printf() attempts.
Knowing the values of some indexes used to offset into the out_ptr0 Tensor for the tl.store() function is needed to debug a corruption problem.

It sometime works and sometimes doesn't and I don't know why

Because the printf buffer isn't configured correct at this moment.

Knowing the values of some indexes used to offset into the out_ptr0 Tensor for the tl.store() function is needed to debug a corruption problem.

Sure

@aifartist After more considerations, I think it's not something triton should control.
The attribute is attached to each cuda context but not cuda function, so the triton compiler is not supposed to change the property.
We could expose an interface for external users, but there's already existing python wrappers you could use.
https://nvidia.github.io/cuda-python/module/cuda.html#cuda.cuda.cuCtxSetLimit
See CU_LIMIT_PRINTF_FIFO_SIZE

It sometime works and sometimes doesn't and I don't know why.

What I was trying to answer is for this question. If you want to know more details about why it doesn't work, you can attach you code here. Otherwise, we won't be able to explain.
It likely your question is out of the "triton" scope, but we can still try to figure it out.


Replacing the old concept of 'patch' by 'pull request' or 'contribution'

Replacing the old concept of 'patch' by 'pull request' or 'contribution'
This work is part of effort done in the Sprints in DjangoCon Europe in Vigo with Daniele Procida @evildmp
Task picked from the task list in https://docs.google.com/spreadsheets/d/16UTGwtAoOwznc46cszbwAHU9xbukXnnpwG-faE94Rw8/edit?gid=0#gid=0
Discussed this subject with Natalia @nessita

@nessita I agree with your corrections, I addressed all of them in eccf56b7f15e2a315a61ff013132388850cdd79d

@nessita as dicussed, I'm extending a bit the scope fixing also this page: https://docs.djangoproject.com/en/5.0/internals/contributing/writing-code/submitting-patches/


 A large repertoire of T-cell receptors with different specificity is needed to provide protection against a wide range of pathogens. This new task aims to predict the binding affinity given a pair of TCR sequence and epitope sequence.
Describe the solution you'd like
from tdc.multi_pred import TCR_Epitope_Binding
data = TCR_Epitope_Binding(name = 'Weber',  path = './data')

Additional context
Reference: https://academic.oup.com/bioinformatics/article/37/Supplement_1/i237/6319659
Data folder: https://ibm.ent.box.com/v/titan-dataset

Hi Anna! Thanks for offering to contribute! Sounds great. I just uploaded to dataverse and it has ID: 5790963 You can find the dataset here: https://dataverse.harvard.edu/file.xhtml?fileId=5790963&version=58.0
Also here are some helpful guides on how to contribute a new task and a dataset:
Contribution guide: https://github.com/mims-harvard/TDC/blob/main/CONTRIBUTE.md
An example new task: https://github.com/mims-harvard/TDC/commit/322bddc88acf9617a1fc191d17b8f6b775f7fa8c
Let me know if you have any question!


Add PHP 8.0 to github actions

Also update PHPUnit require to include 8.5 and 9.4

PullRequest blocked until next version of symfony-validation-shorthand is released with PHP 8 support
123inkt/symfony-validation-shorthand#26


Release v0.3.1 messed up the website's functionality

https://github.com/Skarlso/crd-to-sample-yaml/pull/45/files
This made the accordion no longer work. 🤔

No idea what's wrong. Once I have some time, I'll have to compare the static output of both and see what's the difference.

Found it. I introduced the wrong value for the ID value.

Fixed with https://github.com/Skarlso/crd-to-sample-yaml/pull/56


feat: Official support for Python 3.13 and drop support for Python 3.8

Codecov Report
Attention: Patch coverage is 97.10145% with 2 lines in your changes missing coverage. Please review.

Project coverage is 88.60%. Comparing base (f18a7c8) to head (ace8b3e).

Files with missing lines
Patch %
Lines

strawberry_django/mutations/resolvers.py
50.00%
1 Missing :warning:

strawberry_django/test/client.py
66.66%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #643      +/-   ##
==========================================
- Coverage   88.93%   88.60%   -0.33%
==========================================
  Files          41       41
  Lines        3596     3606      +10
==========================================
- Hits         3198     3195       -3
- Misses        398      411      +13

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


Updating Haiku platform for Godot 3.x

Someone had to say this sooner or later, but I think Haiku should go, whatever little of it exists anyways.
It's a extremely niche OS, and the Godot files of it has seen little to no maintenance for it. I don't know the OS can even open the editor at this point.

For the reference, I'm moving the Haiku platform port to a dedicated repo (and thus this issue with it): https://github.com/godotengine/godot-haiku-platform
This removes the maintenance burden of having to propagate any porting changes to platform/haiku, without really being able to test if they compile or work, and it should make it easier to have the Haiku port following its own delayed release schedule to match changes in Godot code.
@CodeforEvolution I'm still very interested in the work you're doing to fix the Haiku port for 3.2. It should now be PR'ed to https://github.com/godotengine/godot-haiku-platform, and once we're happy with the code we can redo the 3.2.x tags to point to the working commit for Haiku.
This code could then be used together with the matching Godot code to package an official Haiku port and templates.

@akien-mga I understand the decision, and thank you for hosting this repository! (I actually had to recently rebase the Haiku code due to formatting changes made across the engine code, etc) Otherwise, I'm getting pretty close to having a good, working product, and should have a pull request for 3.2 (hopefully) soon! At this point, I'm trying to improve the engine's performance on Haiku.

Good afternoon @akien-mga! I apologize that I have been inactive for quite some time, though I feel I just needed a bit of a break before continuing. The only problem is that I realized my patches not only cover stuff in the "platform/haiku" folder, but also other buildsystem files, files to do with the OS_Unix class, etc. Should I try to separate these changes into separate commits and create a pull request over at the main Godot repository for everything that does not apply to the "platform/haiku" folder? Or is there something else I should try?

If that's the changes from https://github.com/godotengine/godot/compare/3.2...CodeforEvolution:haikuFixup3.2, I guess that's fine, we can get them merged one way or another in the main Godot repository.
Some of them might be avoidable, but we can discuss them directly in the relevant godotengine/godot pull request(s).

That sounds good! I'll make sure to explain the reasoning behind some changes in more detail (Haiku is a whole different beast after all), and of course, I'll also clean up my mess of commits. (I have a lot of squashing to do!)
In other news from the Haiku community:
One of the Haiku community members added syntax highlighting for GDScript to one of our native coding editors: https://github.com/KapiX/Koder/pull/125
In addition, this community member is also currently designing a Haiku-like vector icon for Godot. (We don't use SVGs or PNGs for our icons. Instead, we have a special "Haiku Vector Icon Format": https://en.wikipedia.org/wiki/Haiku_Vector_Icon_Format).

Good day @akien-mga! I have one other question as I wrap things up. With this pull request, I have implemented lots of new platform specific features to the Haiku backend. However, I was wondering if, besides the simple command line tests, if there was a test suite for testing the functionality of “operating system specific classes” such as the OS , AudioDriver, and MidiDriver classes. Or, should I just try to search around the Godot template project library for example projects?

Hey @akien-mga. It's the first day of September, and I believe that I won't be able to submit a finalized pull request with the little time that I have... I really appreciate the patience you have given to me and the Haiku community, and I thank you for that! I don't believe this is the end of the road however. I believe someone else from the Haiku community could pickup my code and put the finishing touches on it. I may even comeback one day to assist, though I can't predict anything.
For those interested, my code base is currently based upon a few commits after the 3.2.2 release and is functional, though slow and a bit buggy. There are currently performance problems (even with the LLVM rasterizer from MESA, I'm achieving only 11 FPS on average), a few crashes (there's a small crash in the MIDIDriver, and there are other issues with how Haiku handles multi-threading), and some platform specific features aren't handled correctly (window handling specifically).
What I have been able to accomplish is here (all those crazy commits will need to be squashed together by the way): https://github.com/CodeforEvolution/godot/tree/haikuFixup3.2
I apologize to other Haiku community members, I know it isn't too new for someone to pickup a big project, get going, and then suddenly disappear. I will at least try to offer help when someone tags me, though I can't guarantee that I'll have time to answer.
With that, I hope you all have a good evening!

What's the status on this?
Any plans for 4.0?
Or getting it merged to main repo.


Added some --help description text, and revised some hint text.

Workflow in development mode, Sync file and Clean Sync file are detailed in --help.

Time to update to PyPI 😄

I have drawn a flowchart.

You may be busy with other things~
Check it out if you have free time 😉

Thanks for doing this nice job! I've also checked the flowchart and it seems fine!
Would you please add flowchart to readme.md in dev mode instruction?

Which typography would be suitable, perhaps portrait?

Thanks! I think the landscape one is more compact and better to be viewed in desktop.

Done~

Thanks :)

@AmirHmZz
Discord link


log eye v.s. link when direct to a record page

Consider adding a new url parameter to pass to the record page e.g. paction=view|link or plink, pclick
Extend the pcid to add this extra info makes the string different from cid which makes the join during the analysis harder.

@RFSH Did we do this in the log already? If so, can you comment and close this issue?

No, we haven't implemented this.

All the links in chaise are generated by the appLink function. This includes that all the buttons (create, edit, view) as well as links that are generated based on the pseudo-column type (or the markdown display settings).
We can make modifications in different places to pass different query parameters. For example for the view button (eye icon) in a tabular format, I can pass this extra query parameter. But I don't see the benefit of adding the query parameter to the rest of the links as well. I think we limit the change for now and just add a specific query parameter to the view button.

This has been done as part of the #2001 PR. I added a section called "Find number of clicks on the view button in recordset app" to provide more information.
@hongsudt Can you please verify and close this issue?

Closing this issue due to inactivity.


chore: exclude some jobs in draft mode unless a label is set

What does this PR do?
exclude some jobs in draft mode unless a label is set
Screenshot / video of UI

What issues does this PR fix or reference?
fixes https://github.com/podman-desktop/podman-desktop/issues/9111
How to test this PR?

[ ] Tests are covering the bug fix or the new feature

All the tests in this PR marked as Skipped are not reassuring me

@axel7083 you should not be so scared. I added a new event ready_for_review so going from draft to review is triggering the checks

@feloy provided https://github.com/podman-desktop/podman-desktop/pull/9965


added main for jspm compat

https://github.com/google/lovefield/issues/80

LGTM. However, I need you to sign the contributor license agreement form as instructed in https://github.com/google/lovefield/blob/master/CONTRIBUTING.md before I can merge your pull request. Please let me know when you have it signed. Thanks!

ok you should have it.
On Wed, Sep 16, 2015 at 10:06 AM, Arthur Hsu<EMAIL_ADDRESS>wrote:

LGTM. However, I need you to sign the contributor license agreement form
as instructed in
https://github.com/google/lovefield/blob/master/CONTRIBUTING.md before I
can merge your pull request. Please let me know when you have it signed.
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/google/lovefield/pull/86#issuecomment-140805846.

Merged c315be3969bc905313b702a8fb3525e1ee8b74e1


Maya: "Selected Lights" for reviews is invalid.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior:
Reviews have the option to use "Selected Lights".
Expected Behavior:
Reviews should not have the option to use "Selected Lights".
This is because there should not be any concept of selection when publishing because it makes publishes not reproducible.
There could be an argument for having an option called "Included Lights", which would only use the lights within the review objectset, but think that is a feature request if needed by production.
Version
3.15.3
What platform you are running OpenPype on?
Windows
Steps To Reproduce:

Create review in Maya and set Display Lights to selected.
Publish.

Is there any more labels you wish to add?

[X] I have searched labels and added any

Relevant log output:
No response
Additional context:
No response
[cuID:OP-5571]

Correction: selected option does actually work when publishing.

Reviews should not have the option to use "Selected Lights".
This is because there should not be any concept of selection when publishing because it makes publishes not reproducible.
There could be an argument for having an option called "Included Lights", which would only use the lights within the review objectset, but think that is a feature request if needed by production.

It would make sense however if the use selection feature uses lights that are in the review object set? So that you could explicitly say, "use these lights only"?


Add stubs for non linux original destination funcs

A small QoL improvement to appease IDEs

/ok-to-test


GitHub User Analysis

Is your feature request related to a problem? Please describe.
The current problem is that users lack a comprehensive and visual way to analyze GitHub profiles. Users often find it frustrating to manually collect and interpret data from various profile elements to gain insights.
Describe the solution you'd like
I would like to develop a feature that automatically analyzes a GitHub user profile, generates a visual chart of the user's activity and contributions, and displays the following information at the bottom of the chart:
Total followers
Total following
Username
Total stars
Describe alternatives you've considered
I have considered alternatives such as:
Manual analysis of profile data
Using GitHub’s API to fetch data and manually creating charts with libraries like Chart.js or D3.js
Integrating third-party analytics tools
Additional context
The feature aims to streamline the process of GitHub profile analysis by providing a clear, visual summary in one place. This will enhance user experience and make it easier to understand profile metrics at a glance.
What problem is this feature trying to solve?
The feature addresses the problem of manually gathering and interpreting GitHub profile data. It aims to simplify the process and provide users with a quick, visual summary of important profile statistics.
How do we know when the feature is complete?
The feature will be considered complete when:
The GitHub user profile data is accurately fetched and analyzed.
A chart is generated that visually represents the user’s activity and contributions.
The total followers, following, username, and stars are correctly displayed at the bottom of the chart.
The feature is thoroughly tested and free of bugs.
Feel free to adjust any details based on your specific needs and the scope of your project!
Video Solution
https://github.com/user-attachments/assets/d9504001-a77f-4780-a16c-8953a37913be

Not accepting the new Issue and PRs as the deadline is 7pm IST today @ankit071105

Not accepting the new Issue and PRs as the deadline is 7pm IST today @ankit071105

Okay afterwards you will assign me ? @Sulagna-Dutta-Roy


prefer already loaded core functions to searching for 4x api functions

i understand this is not perfect, as there are reasons to override some 3x functions with 4x implementations, but traversing every gem and module in LOADPATH in search for include.rb is just not acceptable.
is there a class/object somewhere that can tell me a list of defined core functions to prefer that instead of all defined 3x functions?
i'm happy to implement any suggestions. this patch is about 20% performance boost and i wouldn't like to give it up.

Ping @hlindberg

CLA signed by all contributors.

4.x functions must always win over 3.x functions or things will go very bad. Changing the load order is not an option.

The proposed fix would break the specification and we just cannot do it this way. In 4.x the loaders cache misses. I am therefore going to close this PR.


Validate function parameter types

Functions inside of the BBL files can contain type information about the parameters.
Add a validation that checks if the actual type is matching with the expected type.

See #179

We decided to close this issue, because we a same a certain effort and because there are other important tasks at the moment.


Create an email alias for CommComm?

Currently, the CTC, TSC, and a few others have an email alias on the iojs.org domain that is used for a handful of purposes: sharing confidential information, reminding people to vote/review things, etc.
I think it would be worthwhile to create an alias for CommComm too that emails everyone here. It has also recently come to my attention that creating alias on the official nodejs.com domain is pretty straightforward (we used to be under the impression it was complicated).
What does everyone else think?

+1

I'd be ok with an email being added.

(we used to be under the impression it was complicated).

It's not difficult IIRC but we cannot automatically maintain it, so it is easiest to alias it to the io.js one which can do that automatically from the repo.

+1

Thanks for that insight @Fishrock123. +1 to implementing this.

+1

It's not difficult IIRC but we cannot automatically maintain it, so it is easiest to alias it to the io.js one which can do that automatically from the repo.

Ahh, good to know. Thanks for the insight @Fishrock123!

@nodejs/community-committee can you let me know which email address you would like to use for this distribution list?
Please note that this email address will be publicly visible, so be sure to use one you don't mind being publicly visible in the README.md, similar to how the TSC does it.
If you would prefer to not have your email address listed publicly, please let me know and we'll see what else we can come up with.

To clarify: listing it on the readme is a separate thing from aliasing, but the emails an alias forwards to is publicly visible on https://github.com/nodejs/email/blob/master/iojs.org/aliases.json.

Related: https://github.com/nodejs/community-committee/issues/90

add me as<EMAIL_ADDRESS>

Could you please use<EMAIL_ADDRESS>for me 😄

use my non-microsoft one, tracy has it.

<EMAIL_ADDRESS>please and thank you.

my email is<EMAIL_ADDRESS>

@Fishrock123 who admins the @iojs.org lists(if you aren't one)? It'd probably be helpful if I have direct contact. I don't want to mess this up 😅

@hackygolucky No single person admins the @iojs.org lists, the alias just needs to be added with the emails to https://github.com/nodejs/email/blob/master/iojs.org/aliases.json (via a PR).
Setting up the @nodejs.org -> @iojs.org redirect is a separate thing. I am not sure how that is done, I think Mikeal was probably the last to touch that.

<EMAIL_ADDRESS>please.

@nodejs/community-committee I want to be clear, from our last meeting, that emails provided here will be published in a public repo. Working on this today!

https://github.com/nodejs/email/pull/46


python312Packages.mhcflurry: 2.1.1 -> 2.1.3

Description of changes
Update this package in its brokenness.
Changelog: https://github.com/openvax/mhcflurry/releases/tag/v2.1.3

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 343599 run on x86_64-linux 1

2 packages marked as broken and skipped:

python311Packages.mhcflurry
python311Packages.mhcflurry.dist


[WiP] Simplify Superset's permissions

A new feature in FAB 2.1 allows us to assign permission names.
This allows us to greatly simplify the security model in Superset, by
redefining the atomicity of permissions, and effectively aggregating
sets of permissions to individual ones.
FAB docs about this feature
https://flask-appbuilder.readthedocs.io/en/latest/security.html#permission-customization
Note that this requires running commands:
flask fab security-converge

These commands will reassign and merge old permissions to new ones appropriately.
TODO

[ ] further aggregate and cleanup perms
[ ] out-of-scope for this PR, migrate individual views, and assign perms that match related ModelViews perms
[ ] break down the main views/core.py module into smaller modules, assign proper perms

Be careful Superset should not use flask fab security-cleanup, since it will delete all permissions that are not associated with any view (all datasource access, all database access, etc...)

Well noted @dpgaspar, I overrode with noop in SupersetSecurityManager

I'm soo looking forward to this, It will make life much easier

@mistercrunch ,
Was wondering if as mentioned in , https://flask-appbuilder.readthedocs.io/en/latest/security.html#permission-customization
is FAB_UPDATE_PERMS = False flag missed or it's not required ?


Multiple custom mappings

💬 Question
How can I have multiple custom mappings and change them during runtime?
For example, imagine I have two themes (theme1, theme2) that differ in both color and font family.
The font family can only be modified via custom mapping, so when changing from theme1 to theme2 I need to select the appropriate custom mapping.
I use the metro config as described in the docs, but it only allows a single custom mapping
UI Kitten and Eva version

Package
Version

@eva-design/eva
2.0.0

@ui-kitten/components
5.0.0

Hi @schiller-manuel 👋
There is no way to change mapping during the app runtime unfortunately.
As a possible workaround, I'd suggest defining font styles as new variables in theme, giving them the same name in theme1 and theme2. After that, you should also rely on those variables with UI Kitten Text component by customizing mapping. That will do the trick.

Thanks @artyorsh that did the trick!


Track - Page is not scrolled to the bottom when navigate back from transaction thread

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

Version Number: 9.0.29-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4919148&group_by=cases:section_id&group_id=309130&group_order=asc
Issue reported by:  Applause - Internal Team
Action Performed:
Precondition:
User have more than 3 track expenses in Self-DM
User should select Do nothing from the actionable whisper

Go to Self-DM chat> Track> Complete the flow
Actionable whisper will appear
Click on expense from step 1 to open transaction thread
Click on header (you) to go back to Self-DM chat
Take a look under expense

Expected Result:
Page should be scrolled to the bottom and actionable whisper should be visible
Actual Result:
Page is not scrolled to the bottom and actionable whisper is not visible until manually scroll the page
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/b84cb8e9-b1ba-4e22-9b98-5f56b80e26bb
View all open jobs on GitHub

@joekaufmanexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Hm, I don't think this is really a bug. If you ignore the actionable whisper and click into any of the expenses above it, and then click back out of them, we are using our default behavior of landing with the expense you previously clicked into in focus.
It doesn't really make sense to me that we'd add some custom logic to this and ignore the default behavior, just because the actionable whisper is there, since the customer chose not to action it. They can still scroll to see it.

Closing as I don't think this is a bug, but LMK if anyone disagrees.


Cannot get device ring settings: No such device

Hi, I have this problem after upgrade to v0.20.2-GIT.
OS name: GNU/Linux Ubuntu 16.04.3 LTS
Elasticsearch version: 5.6.0
Moloch build using easybutton-build.sh
Elasticsearch and moloch in the same server
I think the problem is from molochcapture. First time when I try upgrade manually from git, this error is show up multiple times and then molochcapture is down. Then I try to make it with clean installation (delete all existing moloch and any nodejs installation) and whenever I run for the first time, the error show up and still the same but on second try, I can start molochcapture

What is your interface setting?

My interface is enp0s3 and I already put it the same interface at /data/moloch/etc/config.ini

Okay, it works using this configuration
https://github.com/aol/moloch/wiki/FAQ#Network_Card_Config
But I believe I had done this configuration before the upgrade version.
Thank you


x86_64: Detect Intel chips from target-cpu

Implement an approach mentioned in https://github.com/taiki-e/portable-atomic/pull/16#discussion_r900695941.
A potential concern is that some users may be using target-cpu=haswell in target-cpu=x86-64-v3-like sense.

A potential concern is that some users may be using target-cpu=haswell in target-cpu=x86-64-v3-like sense.

Given this concern, I don't think this can ever be merged.
I think it would be reasonable to add a target feature (in our case probably cfg) as discussed in https://gcc.gnu.org/pipermail/gcc-patches/2022-November/606039.html.


MS Teams Bot Login Issue: Successful Authentication but 'Unable to reach app' Error

Details:
I'm working on a Microsoft Teams Bot and encountering an issue. Even though the login is successful, I keep seeing the error message "Unable to reach app. Please try again."
When I use Node.js with the Microsoft Bot Framework to display the OAuthPrompt, the Authentication Sign In is correctly displayed. The login process does succeed and returns the correct token. However, an error message is displayed on the Login card itself, stating "Unable to reach app. Please try again."
Configuration Steps Taken:

I set up the App Registration.
In the Azure AD's Manage section, I configured the following:

Authentication
Certificates & secrets
API permissions

I also configured the Azure bot, particularly the OAuth Connection Settings. Testing the connection was successful and returned a token.
I then created an app in the Microsoft Teams App using the Developer Portal. I added the basic configuration, app features (bot), single sign-in, and domain section.

Issue:
After publishing and testing, the sign-in bot appears as expected. However, when I click on it, I encounter the "Unable to reach app" error. This issue is exclusive to the Teams app. When testing with the bot emulator and Azure bot (test in web chat), everything works correctly. Additionally, when I tried the Teams app in a browser and inspected the network tab, it threw a 502 bad gateway error.

Has anyone encountered a similar issue or can provide insights into resolving this? Any guidance would be greatly appreciated.

Hi @kajasherif, thanks for reporting this issue.
Since the Authentication works and the issue is exclusive to Teams channel, could you please open an issue in OfficeDev/Microsoft-Teams-Samples repository for a more targeted resolution?
https://github.com/OfficeDev/Microsoft-Teams-Samples/tree/main/samples
Please make sure to additionally share your Teams app manifest with them as well.


Migrate from AssertJ to AssertK

AssertK is a fluent assertion library for Kotlin, which is designed with Kotlin idioms. It would be nice to migrate to that.
https://github.com/willowtreeapps/assertk

Currently I see no benefits of migrating ktlint from Assertj to Assertk. Most of the tests are based on ktlint's internal dsl for validating lint/format issues. That dsl is quite complex and might be simplified when using Assertk. I will mark the issue as parked, but if someone else would like to pick it up I am okay with that.


Bug report: Unable to install jaspersoft-studio - It seems the App source is not there

Remember to follow the pre bug report guide beforehand. Failure to do so might get your issue closed.
Description of issue
Installation of jaspersoft-studio fails, the app ist not found.
It seems to me that the app name in the cask is not correct:
Currently:
   app "Jaspersoft Studio #{version}.final/Jaspersoft Studio.app"
Should be:
   app "TIBCO Jaspersoft Studio #{version}.final/Jaspersoft Studio.app"
Output of `brew cask install jaspersoft-studio --verbose`
==> Downloading https://downloads.sourceforge.net/jasperstudio/TIBCOJaspersoftStudio-6.3.1.final-mac-x86_64.dmg
Already downloaded: /Users/nicolas/Library/Caches/Homebrew/Cask/jaspersoft-studio--6.3.1.dmg
==> Verifying checksum for Cask jaspersoft-studio
Error: It seems the App source is not there: '/usr/local/Caskroom/jaspersoft-studio/6.3.1/Jaspersoft Studio 6.3.1.final/Jaspersoft Studio.app'
Error: nothing to install
Error: Kernel.exit

Output of `brew doctor`
[paste output here]

Output of `brew cask doctor`
==> macOS Release:
10.12.2
==> Hardware Architecture:
intel-64
==> Ruby Version:
2.0.0-p648
==> Ruby Path:
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
==> Homebrew Version:
Homebrew 1.1.6
Homebrew/homebrew-core (git revision 24f8; last commit 2017-01-08)
==> Homebrew Executable Path:
/usr/local/bin/brew
==> Homebrew Cellar Path:
/usr/local/Cellar
==> Homebrew Repository Path:
/usr/local/Homebrew
==> Homebrew Origin:
https://github.com/Homebrew/brew.git
==> Homebrew-Cask Version:
Homebrew-Cask 1.1.6
caskroom/homebrew-cask (git revision 471f6; last commit 2017-01-08)
==> Homebrew-Cask Install Location:
<NONE>
==> Homebrew-Cask Staging Location:
/usr/local/Caskroom
==> Homebrew-Cask Cached Downloads:
/Users/nicolas/Library/Caches/Homebrew/Cask
1 files, 363.3M (warning: run "brew cask cleanup")
==> Homebrew-Cask Default Tap Path:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask
==> Homebrew-Cask Alternate Cask Taps:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-fonts
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-versions
==> Homebrew-Cask Default Tap Cask Count:
3508
==> Contents of $LOAD_PATH:
/usr/local/Homebrew/Library/Homebrew/cask/lib
/usr/local/Homebrew/Library/Homebrew
/Library/Ruby/Site/2.0.0
/Library/Ruby/Site/2.0.0/x86_64-darwin16
/Library/Ruby/Site/2.0.0/universal-darwin16
/Library/Ruby/Site
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/x86_64-darwin16
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/universal-darwin16
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/x86_64-darwin16
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/universal-darwin16
==> Contents of $RUBYLIB Environment Variable:
<NONE>
==> Contents of $RUBYOPT Environment Variable:
<NONE>
==> Contents of $RUBYPATH Environment Variable:
<NONE>
==> Contents of $RBENV_VERSION Environment Variable:
<NONE>
==> Contents of $CHRUBY_VERSION Environment Variable:
<NONE>
==> Contents of $GEM_HOME Environment Variable:
<NONE>
==> Contents of $GEM_PATH Environment Variable:
<NONE>
==> Contents of $BUNDLE_PATH Environment Variable:
<NONE>
==> Contents of $PATH Environment Variable:
PATH="/usr/local/sbin:/Users/nicolas/.nvm/versions/node/v5.10.1/bin:/usr/local/bin:/Users/nicolas/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/myApp:/opt/X11/bin:/opt/smartrsa/tools/usr/local/bin:/opt/jboss/jboss/bin:/usr/local/Homebrew/Library/Taps/homebrew/homebrew-boneyard/cmd:/usr/local/Homebrew/Library/Homebrew/cask/cmd:/usr/local/Homebrew/Library/Homebrew/shims/scm"
==> Contents of $SHELL Environment Variable:
SHELL="/bin/bash"
==> Contents of Locale Environment Variables:
LANG="en_US.UTF-8"
LC_ALL="en_US.UTF-8"

https://github.com/caskroom/homebrew-cask/pull/28747

Thanks for reporting the issue.