Expose StyleSheetPropType and ViewStylePropTypes

This exposes StyleSheetPropType and ViewStylePropTypes as part of the react-native exports.
This allows components to define style propTypes that they may require. It would also creating custom style rules that will allow granularity in what styles are permissible on the custom component.
class CustomView extends React.Component {
  static propTypes = {
    style: StyleSheetPropType(ViewStylePropTypes)
  };
  render() {
    return (
      <View style={this.props.style}>..</View>
    );
  }
}

Inspiration has been taken from how prop types are defined in View and ScrollView. Theoretically more style related prop types can be exposed, but first I'd like to see how this is received.
Current workaround:
class CustomView extends React.Component {
  static propTypes = {
    style: View.propTypes.style
  };
  render() {
    return (
      <View style={this.props.style}>..</View>
    );
  }
}

    style: View.propTypes.style

This is not a workaround, this is how we want people to use this API. We don't want to expose the underlying implementation details as they will probably change in the future.
Also, we need to fix ScrollView to use View.propTypes.style instead of the longer form :)
Is there any use case you have in mind for your app that requires this? I'm going to close this down for now.

Oh, good to know this is the recommended approach, thanks for the feedback :+1:

Is there any use case you have in mind for your app that requires this?

@vjeux
i'm looking for greater strictness in the styles that can be passed to a component. For instance, i would want to specify custom subsets of View.propTypes.style, such as just margins:
const styleShape = require('../../node_modules/react-native/Libraries/StyleSheet/StyleSheetPropType');

const marginsType = styleShape({
  marginTop: pt.number,
  marginRight: pt.number,
  marginBottom: pt.number,
  marginLeft: pt.number
});

MyComponent.propTypes = {
  margins: marginsType.isRequired
}

//--------------------
// usage
const styles = StyleSheet.create({
  margins: {
    marginTop: 10,
    marginBottom: 15
  }
})

<MyComponent margins={styles.margins} />

styles can be part of the component interface, just like any other prop, and i think its good to clarify the component interface as much as possible because it will inform the user on the capabilities and variations i chose to expose.
advise if you see an easier way to do this.

So there's now ViewStylePropTypes, and I get a warning whether I use View.propTypes.style or ViewStylePropTypes
When I use View.propTypes.style, I see the following warning:
Warning: View.propTypes has been deprecated and will be removed in a future version of ReactNative. Use ViewPropTypes instead.
When I use ViewStylePropTypes, I see the following warning:
ExceptionsManager.js:71 Warning: Failed prop type: MyComponent: prop type myComponentProp is invalid; it must be a function, usually from React.PropTypes.
in AutocompleteGhost (at Example.js:25)
in Example (at renderApplication.js:35)
in RCTView (at View.js:133)
in View (at AppContainer.js:98)
in RCTView (at View.js:133)
in View (at AppContainer.js:97)
in AppContainer (at renderApplication.js:34)
So basically, I can't win.


v2024.5.1: Resolving the import of non-"third party" modules

Type: Bug
Pylance v2024.5.1 appears to have a bug related to resolving the import of  classes/modules which are not "third-party". In v2024.4.1 this issue doesn't occur: the same modules whose import could not be resolved using the newer version v2024.5.1 of Pylance, do not present a problem using the older version v2024.4.1.
Extension version: 2025.4.1
VS Code version: Code 1.89.0 (b58957e67ee1e712cebf466b995adf4c5307b2bd, 2024-05-01T02:09:22.859Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
12th Gen Intel(R) Core(TM) i7-12800H (20 x 2803)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
63.67GB (52.96GB free)

Process Argv
--crash-reporter-id 0c90dd25-ab31-4ec1-bec4-ccfef9fc48c3

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vscorecescf:30445987
vscod805:30301674
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
vscaat:30438848
c4g48928:30535728
azure-dev_surveyonecf:30548226
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
h48ei257:31000450
pythontbext0:30879054
accentitlementsc:30995553
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
showvideot:31016892
chatpanelc:31018788
dsvsc021:30996838
724cj586:31013169
fchga941:31038042
pythonprt:31039817
dwnewjupytercf:31039676

might be dupe of https://github.com/microsoft/pylance-release/issues/5836
but that said, it would be nice to have more detail repro steps

@SDiWK, can you please provide repro steps with code, required packages (inc. versions), etc?

@SDiWK, can you please provide repro steps with code, required packages (inc. versions), etc?

@debonte: shared repro code via repository SDiWK/5861
Hope this helps. Thanks and kind regards :-)

@SDiWK github says I dont have access to the repository.

@heejaechang Can't say why you ran into trouble accessing it. Sorry for the inconvenience. I just made the repository public. There doesn't seem to be a point restricting access anyway, in this case.

we are talking about this right? https://github.com/SDiWK/5861/tree/main
it has only 2 files in it
A.py
class A:

    def __init__():
        pass

B.py
from A import A

class B(A):

    def __init__():
        pass

there is only 1 import statement from A import A. so I assume you are referring that import statement?
for me, it works fine.
can you provide us some logs as described in the troubleshooting guide?
I think I need see your settings and env to find out why it didn't work for you.

In my use case I've had the impression that my class (here B) which imports a class A, say, doesn't have access to the methods/members of A, even though it should, at least I expected so.
I think I need to refine my reproduction setting a bit, in order to pinpoint what I meant. If the problem persists, I'll recur with more detailed information as outlined by you. Many thanks for your time so far.

if from A import A didn't work properly (such as pylance couldn't resolve either A), then A in import A will be unknown symbol. so pylance won't be able to provide anything off it such as autocomplete after A.


Extension causes high cpu load

Issue Type: Performance
Extension Name: python
Extension Version: 2019.4.11987
OS Version: Linux x64 4.15.0-47-generic
VSCode version: 1.33.1

:warning: Make sure to attach this file from your home-directory:
:warning:/home/dallery/ms-python.python-unresponsive.cpuprofile.txt
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load
ms-python.python-unresponsive.cpuprofile.txt

Please could you upload the comments of /home/dallery/ms-python.python-unresponsive.cpuprofile.txt

It is already here, but i don't understand where put it, so, it is after "Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load"

You need to upload the file.
Without the file we cannot really do anything.
Closing the issue for now, when this occurs again, please upload the contents of the file and we'll be happy to re-open this issue or just file another issue.
See below (the message you pasted contains the request to upload the file)


Trying to install Micosoft ASP.NET

Problem encountered on https://dotnet.microsoft.com/learn/aspnet/hello-world-tutorial/install
Operating System: windows
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
'dotnet' is not recognized as an internal or external command error
64bit operationg system
:\Users\Dell>dotnet run
'dotnet' is not recognized as an internal or external command,
operable program or batch file.
C:\Users\Dell>

Did you install the DotNet SDK for Windows 64?
After install it you can execute “dotnet” command line without the reported error.
Download here:
https://dotnet.microsoft.com/download

@Nelsonh123 did you try to open a new command prompt after you installed the SDK as detailed in the instructions?

Closing due to the lack of response.


[Android] How use expansion files with react native?

Hi,
Building app with 600+ images. Apk file size is ~450MB. Google Play limits upload up to 100MB, only option is to use apk expension files.
http://developer.android.com/google/play/expansion-files.html
What is best way to use those with react?

Hey and thanks for reporting this!
There's an awesome place to ask question like this one: StackOverflow. It's the best system for Q&A. Many people from the community hang out there and will be able to see your question, you can vote on answers and mark question as answered etc. This lets us keep a list of bug reports and feature requests on github and especially Product Pains (again, with voting which is really nice).
If you think StackOverflow works for you please consider posting there instead and closing this issue.
I'm posting this here because github issues haven't been working very well for us and because StackOverflow is so much better. Thanks for reading! :)


Invalid value for prop css when using @emotion/react with Vite

I reported this error in Storybook's GH, but they redirected me to you guys.
Describe the bug
I couldn't find any information on how to make @emotion/react work in Storybook when using Vite as a bundler in a React application.
I'm getting errors like Invalid value for prop 'css' in <div> tag in almost every story. Even though, @emotion/react is working fine for the webapp itself.
Here's my vite.config.js configuration:
import { defineConfig } from 'vite';
import react from '@vitejs/plugin-react';

export default defineConfig({
  esbuild: {
    jsxFactory: 'jsx',
    jsxInject: `import { jsx } from '@emotion/react'`,
  },
  plugins: [
    react({
      jsxImportSource: '@emotion/react',
      babel: {
        plugins: ['@emotion/babel-plugin'],
      },
    }),
  ],
});

And here's my main.js for Storybook:
const svgrPlugin = require('vite-plugin-svgr');

module.exports = {
  core: {
    builder: 'storybook-builder-vite',
  },
  stories: ['../src/**/*.stories.mdx', '../src/**/*.stories.@(js|jsx|ts|tsx)'],
  addons: ['@storybook/addon-links', '@storybook/addon-essentials'],
  viteFinal: (config, { configType }) => {
    config.define = {
      'window.process': {
        env: {
          NODE_ENV: configType.toLowerCase(),
        },
      },
    };
    return {
      ...config,
      plugins: [
        ...config.plugins,
        svgrPlugin({
          svgrOptions: {
            icon: true,
          },
        }),
      ],
    };
  },
};

To Reproduce
Sorry, I failed with trying to create a reproducible repository.
Versions of packages that I use:
react - 16.14.0
@emotion/react - 11.7.1
@emotion/babel-plugin - 11.7.1
storybook-builder-vite - 0.1.11
vite - 2.7.3
vite-plugin-svgr - 0.6.0
@vitejs/plugin-react - 1.1.3
System
Environment Info:
System:
OS: Linux 5.6 Ubuntu 20.04.3 LTS (Focal Fossa)
CPU: (8) x64 Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz
Binaries:
Node: 14.15.4 - ~/.nvm/versions/node/v14.15.4/bin/node
Yarn: 1.19.1 - /usr/bin/yarn
npm: 6.14.10 - ~/.nvm/versions/node/v14.15.4/bin/npm
Browsers:
Chrome: 95.0.4638.69
Firefox: 94.0
npmPackages:
@storybook/addon-actions: ~6.3.8 => 6.3.12
@storybook/addon-essentials: ~6.3.8 => 6.3.12
@storybook/addon-links: ~6.3.8 => 6.3.12
@storybook/node-logger: ~6.3.8 => 6.3.12
@storybook/react: ~6.3.8 => 6.3.12
Additional context
There are ways how to make @emotion/react work when one is using Webpack as a bundler, but unfortunately I couldn't find anything related to Vite.

#210 might solve your issue.

Thanks! I'm closing this issue as the suggested solution resolved it.


react-native-camera front camera not working on ios

Front camera only works sometimes on ios, it seems that it works on the first time the app is opened but then stops? if i reload the js, then it works again. works perfectly on Android
Function to swtich camera:
switchType() {
  let newType;
  const { back, front } = Camera.constants.Type;

  if (this.state.camera.type === back) {
    newType = front;
  } else if (this.state.camera.type === front) {
    newType = back;
  }

  this.setState({
    camera: {
      ...this.state.camera,
      type: newType,
    },
  });
}

Markup:
<Camera
            ref={(cam) => {
              this.camera = cam;
            }}
            style={styles.preview}
            aspect={this.state.camera.aspect}
            captureTarget={this.state.camera.captureTarget}
            type={this.state.camera.type}
            flashMode={this.state.camera.flashMode}
            defaultTouchToFocus
            mirrorImage={false}
            captureQuality="medium"
          />

This happens when using the Navigator, when you push a Route and the pop back to the camera route. Ive seen there is some unmounting problems. Does anyone have a fix for this? on first load after rebooting the phone it works.

I solved it by using this.props.navigator.replace() instead of push and pop, but i like your solution!


1903 and Process isolation

I have a container based on microsoft/dynamics-nav:2016-dk-ltsc2019 that I cannot start.
Its an old container created in 2019 when my windows 10 machine was not updated. I have rolled back the windows updates as fare as I could and windows is now on 1903.  When I start the container now it gives me this error
docker : Error response from daemon: hcsshim::CreateComputeSystem: The container operating system does not match the host operating system.
When I inspect the container I can see that it runs in "process isolation" and I guess that's why I couldn't start it in windows version 1909.
My question is: Is there any way to get this up and running again? I, of course, have c/al code in the container that is not backed up :-(

Ouch.
You can try
docker cp "containername:c:\databases\Cronusdk\Demo Database NAV (9-0)_Data.mdf" "c:\temp\Demo Database NAV (9-0)_Data.mdf"
docker cp "test:c:\databases\Cronusdk\Demo Database NAV (9-0)_Log.ldf" "c:\temp\Demo Database NAV (9-0)_Log.ldf"

from a command prompt to copy the database out. (don't know if this works).
With that, I guess you can attach the database to a SQL Server - open the DB with the classic client and export your objects.

Nice one Freddy that worked like a charm, just the trick I was looking for :-)

A good rule to live by: Never try to keep containers alive - save the script to regenerate the container.
:-)
In case of C/AL - it might be a good idea to have the database outside the container.,

Jep, lesson learned :-)


Login failure does not result in exception being thrown

If login failed with the wrong password, I would expect an exception being thrown, similar to errors. However, the promise resolves and the code carries on.
This is the API response:
{
  login: {
    result: 'Failed',
    reason: {
      code: 'wrongpassword',
      text: 'Incorrect username or password entered. Please try again.'
    }
  }
}

Yes, the login API returns nonstandard errors :/ m3api-botpassword should handle this correctly (since 0.2.0), see https://github.com/lucaswerkmeister/m3api-botpassword/issues/1 / https://github.com/lucaswerkmeister/m3api-botpassword/commit/e4073bfd3e95ff423dedfcf20e5cc0d546fdef3c).
Although the reason in your response looks different… does it depend on the errorformat?

Although the reason in your response looks different… does it depend on the errorformat?

Looks like it does, yeah. I should make the package handle that, at least.

I think this is done now, m3api-botpassword should handle this case correctly. (And I don’t think m3api itself should work around non-standard error formats in specific APIs that are only kept for backwards compatibility reasons.)


Some small changes

@nightism Ugh I just noticed that you did not apply your Format: thing to all commands.
Please follow up on this. Either remove all the Format: things or do it for all.

@pyokagan please check if this branch can be merged so that Sherry can do further revision

Merged in 279715e24dcb8714cd4cf5355d1afde61087ac17


Integration Candidate: 2020-10-20

Describe the contribution
Fix #943
Fix #924
Fix #502
Testing performed
Bundle CI: https://github.com/nasa/cFS/pull/152/checks
Expected behavior changes
PR #946
System(s) tested on
Ubuntu
Additional context
Part of https://github.com/nasa/cFS/pull/152
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
@jphickey

@jphickey we have a problem with the merge from #946,
1980-012-14:03:20.00039 EVS:Call to CFE_ES_GetAppID Failed:RC=0xC4000001
1980-012-14:03:20.00041 EVS:Application Init Failed,RC=0xC4000001

Fixed - this was some sort of merge/rebase issue, the line that sets the ID got moved but the refs before it were using the wrong (reserved) value not the correct (pending) value.


ambuzol makes you unkillable by zombies

Description
If you have ambuzol or ambuzolplus in your system, and in crit, zombie bites seem to heal you.
Normaly ambuzol is supposed to prevent conversions, but it seems to also stop all damage taken when attacked by a zombie while in crit.
Reproduction

Inject ambuzol or ambuzolplus
get crit
have a zombie try to attack you
Videos

With ambuzol in your system:
https://github.com/space-wizards/space-station-14/assets/106507028/0962ad07-32c6-401e-9991-9c9d939c4c14
Same thing with ambuzolplus in your system:
https://github.com/space-wizards/space-station-14/assets/106507028/d78c1a27-69b1-4a87-a3cf-e0d0e9ccc5f5

Additional context
If a zombie attacks you when you have ambuzol in your system you should probally just take damage and die. Being 100% immune to zombie damage if you have it in your system is broken.
 Ive seen a crit person stand back up and run away when getting attacked by zombies when they were in crit.

I was unable to reproduce this with Ambuzol alone. Was the Urist in zombiebug2.mp4 given Ambuzol Plus at some point?

I was unable to reproduce this with Ambuzol alone. Was the Urist in zombiebug2.mp4 given Ambuzol Plus at some point?

Yes i used ambuzol plus on the same urist, after it expired from the body soultion i added the ambuzol to test that as well.


framenavigated doesn't work in firefox

I just need to execute the script when the page is navigated. But in firefox framenavigated doesn't work, even evaluateOnNewDocument  doesn't work. What is the way to detect page navigation in firefox?
import puppeteer from 'puppeteer';

(async () => {
  const browser = await puppeteer.launch({
    product: 'firefox',
    executablePath: "C:\\Program Files\\Mozilla Firefox\\firefox.exe",
    headless: false
  });

  browser.on('targetcreated', async (target) => {
    if (target.type() === 'page') {
      const page = await target.page();

      page.evaluateOnNewDocument

      page.on('framenavigated', async () => {
        page.evaluate(() => { alert(123); });
      });
    }
  });
})();

Try using WebDriver BiDi
  const browser = await puppeteer.launch({
    product: 'firefox',
    headless: false,
    protocol: 'webDriverBiDi'
  });

That seems to work for me. Note that the CDP support in Firefox is limited and it will be superseded by WebDriver BiDi in the future. cc @whimboo

Yes, we will not fix both for the experimental CDP implementation. Use WebDriver BiDi instead and instead of evaluateOnNewDocument you can use script.addPreloadScript.


fix #37

Nice, after the merge I'll be able to delete a workaround in cofree-react-router.

Yeah, sorry, I didn't realise there were a bunch of outstanding PRs until I saw the Aff one. All released now :smile:.


Fix query_mut permitting unsound double-borrows

Fixes #161.
Not a huge fan of how this complicates Fetch, but it fixes the unsoundness and has no overhead in practice.

LGTM

Thanks!


feat: Add validation

feat: Add validation

Why not simply use before hooks to run validation for DTO?

I will take a look at that before hooks.
I created these changes because in @adminjs/mikroork they have Resource.validate where it will run before running the create and update. so I just copy those features here.


Fix LIKE with escapes

Description
Fix LIKE processing for patterns containing escapes

the starts_with / ends_with optimization did not correctly check for
escapes when checking rest of the pattern for being literal or not
the pattern to regexp compiler incorrectly processed \ followed by a
character other than % or _. In PostgreSQL '\x' pattern matches single
'x'.

There are two tests

like_escape_many was generated using PostgreSQL with the code attached
below for verification
like_escape is hand-picked test cases that are more interesting.
Lower cardinality of hand-picked test cases allows for exercising all
scalar/array vs scalar/array combinations.

Test case generation

Show
The below script isn't simples possible, because it was attempted to
generate more test cases by adding padding. Hence e.g.
is_like_without_dangling_escape.  Since this is attached for reference,
should be attached as-is.
#!/usr/bin/env python
import psycopg2

data = r"""
\
\\
\\\
\\\\
a
\a
\\a
%
\%
\\%
%%
\%%
\\%%
_
\_
\\_
__
\__
\\__
abc
a_c
a\bc
a\_c
%abc
\%abc
a\\_c%
""".split('\n')

# Deduplicate, keeping order
data = list(dict.fromkeys(data))

conn = psycopg2.connect(host='localhost', port=5432, user='postgres', password='mysecretpassword')
conn.set_session(autocommit=True)
cursor = conn.cursor()
for r in data:
    try:
        # PostgreSQL verifies dandling escape only sometimes
        cursor.execute(f"SELECT %s LIKE %s", (r, r))
        is_like, = cursor.fetchone()
        has_dandling_escape = False
        pg_pattern = r
    except Exception as e:
        if 'LIKE pattern must not end with escape character' not in str(e):
            raise e
        has_dandling_escape = True
        pg_pattern = r + '\\'

    for l in data:
        # print()
        # print('     '.join(str(v) for v in (l, r, has_dandling_escape, postgres_pattern)))
        cursor.execute(f"SELECT %s LIKE %s", (l, pg_pattern))
        is_like, = cursor.fetchone()
        assert type(is_like) is bool

        if not is_like and has_dandling_escape:
            pattern_without_escaped_dandling_escape = pg_pattern[:-2]
            cursor.execute(f"SELECT %s LIKE %s", (l, pattern_without_escaped_dandling_escape))
            is_like_without_dangling_escape, = cursor.fetchone()
            assert type(is_like_without_dangling_escape) is bool
        else:
            is_like_without_dangling_escape = False
        assert '"' not in l
        assert '"' not in r
        print('(r"%s", r"%s", %s),' % (
            l, r,
            str(is_like).lower(),
            # str(has_dandling_escape).lower(),
            # str(is_like_without_dangling_escape).lower(),
        ))

Which issue does this PR close?
Fixes https://github.com/apache/arrow-rs/issues/6702
Rationale for this change
Correctness fix for LIKE.
What changes are included in this PR?
Correctness fix for LIKE.
Are there any user-facing changes?
Yes

this is needed for https://github.com/apache/datafusion/issues/13304

@alamb @tustvold please take a look
cc @samuelcolvin @Dandandan @ozankabak

Please can you refrain from repeatedly tagging people on your PRs, we will get to your PRs when we have time, but spamming people needlessly is just rude

Rebased to resolve conflict with https://github.com/apache/arrow-rs/pull/6662

Benchmark results (basically no change in my opinion):
++ critcmp master findepi_fix-like-with-escapes-792c56
group                                              findepi_fix-like-with-escapes-792c56    master
-----                                              ------------------------------------    ------
ilike_utf8 scalar complex                          1.00      2.6±0.07ms        ? ?/sec     1.01      2.7±0.06ms        ? ?/sec
ilike_utf8 scalar contains                         1.03      4.3±0.05ms        ? ?/sec     1.00      4.1±0.05ms        ? ?/sec
ilike_utf8 scalar ends with                        1.00  1202.7±35.59µs        ? ?/sec     1.03  1236.9±23.92µs        ? ?/sec
ilike_utf8 scalar equals                           1.00   747.4±16.29µs        ? ?/sec     1.05   784.2±23.50µs        ? ?/sec
ilike_utf8 scalar starts with                      1.00  1102.1±30.61µs        ? ?/sec     1.02  1127.3±28.58µs        ? ?/sec
ilike_utf8_scalar_dyn dictionary[10] string[4])    1.00     77.6±0.08µs        ? ?/sec     1.00     77.7±0.10µs        ? ?/sec
like_utf8 scalar complex                           1.00  1845.0±21.89µs        ? ?/sec     1.02  1889.4±18.40µs        ? ?/sec
like_utf8 scalar contains                          1.00  1549.6±14.84µs        ? ?/sec     1.01  1570.9±13.61µs        ? ?/sec
like_utf8 scalar ends with                         1.00   410.3±14.51µs        ? ?/sec     1.02   420.4±10.65µs        ? ?/sec
like_utf8 scalar equals                            1.00    127.0±0.52µs        ? ?/sec     1.00    127.0±0.27µs        ? ?/sec
like_utf8 scalar starts with                       1.00   336.7±19.19µs        ? ?/sec     1.01    339.4±5.83µs        ? ?/sec
like_utf8_scalar_dyn dictionary[10] string[4])     1.00     77.5±0.12µs        ? ?/sec     1.00     77.4±0.05µs        ? ?/sec
like_utf8view scalar complex                       1.00    182.4±0.72ms        ? ?/sec     1.00    183.1±0.44ms        ? ?/sec
like_utf8view scalar contains                      1.03    142.5±0.23ms        ? ?/sec     1.00    138.8±0.22ms        ? ?/sec
like_utf8view scalar ends with 13 bytes            1.00     44.3±0.38ms        ? ?/sec     1.08     47.8±0.10ms        ? ?/sec
like_utf8view scalar ends with 4 bytes             1.00     45.0±0.18ms        ? ?/sec     1.09     48.9±0.18ms        ? ?/sec
like_utf8view scalar ends with 6 bytes             1.00     44.8±0.13ms        ? ?/sec     1.08     48.6±0.17ms        ? ?/sec
like_utf8view scalar equals                        1.00     35.8±0.16ms        ? ?/sec     1.01     36.0±0.07ms        ? ?/sec
like_utf8view scalar starts with 13 bytes          1.01     47.8±0.15ms        ? ?/sec     1.00     47.3±0.16ms        ? ?/sec
like_utf8view scalar starts with 4 bytes           1.00     34.3±0.09ms        ? ?/sec     1.00     34.3±0.10ms        ? ?/sec
like_utf8view scalar starts with 6 bytes           1.01     48.2±0.20ms        ? ?/sec     1.00     47.8±0.15ms        ? ?/sec
nilike_utf8 scalar complex                         1.00      2.6±0.06ms        ? ?/sec     1.02      2.7±0.07ms        ? ?/sec
nilike_utf8 scalar contains                        1.03      4.2±0.06ms        ? ?/sec     1.00      4.1±0.05ms        ? ?/sec
nilike_utf8 scalar ends with                       1.00  1189.1±24.87µs        ? ?/sec     1.03  1229.4±22.89µs        ? ?/sec
nilike_utf8 scalar equals                          1.00   755.0±17.17µs        ? ?/sec     1.04   787.2±27.12µs        ? ?/sec
nilike_utf8 scalar starts with                     1.00  1124.4±45.36µs        ? ?/sec     1.01  1136.6±33.68µs        ? ?/sec
nlike_utf8 scalar complex                          1.00  1855.2±39.69µs        ? ?/sec     1.02  1887.3±29.24µs        ? ?/sec
nlike_utf8 scalar contains                         1.00  1548.3±12.27µs        ? ?/sec     1.01  1568.2±12.93µs        ? ?/sec
nlike_utf8 scalar ends with                        1.00   411.8±17.08µs        ? ?/sec     1.02    420.5±7.53µs        ? ?/sec
nlike_utf8 scalar equals                           1.00    126.7±0.14µs        ? ?/sec     1.00    127.0±0.39µs        ? ?/sec
nlike_utf8 scalar starts with                      1.00    328.4±5.42µs        ? ?/sec     1.05   345.2±13.53µs        ? ?/sec
++ popd


Fix missing parenthesis in terraform config

A parenthesis was missing in PR #2553, this makes doom failed to load his config.

:+1: Just ran into this. Workaround: disable terraform in init.el


Pinia state persistence

Apologies for this issue which seeks guidance more than raises code issues.
I have a list of objects in a Pinia store which I am accessing in the component via the standard storeToRefs(). These objects contain a property order which is used to specify the order within the list.
I first filter this list to remove an entry via a computed property like:
const filteredObjectList = computed({
  get() {
    return objectList.value.filter((obj) => {
      return obj.id != 0;
    })
  },
  set(newList) {
    console.log(newList)
  }
})

I have been able to produce the correct drag and drop behaviour via :list="filteredObjectList" item-key="order" and the list reorders properly. However the state does not seem to persist when the list is hidden and then shown again via a v-if clause. It is worth noting as well that the setter is never called.
Would you be able to provide some guidance on this non-persistence issue?

Hello
As stated in the README.md :
To modify your data you need to manually listen to the events and calculate the new position with event.oldIndex and event.newIndex with something like the following:
const moveItemInArray = <T>(array: T[], from: number, to: number) => {
  const item = array.splice(from, 1)[0];
  array.splice(to, 0, item);
};

onEnd(event) { moveItemInArray(store.state.items, event.oldIndex, event.newIndex) }


[RTE] Move rich text editor feature to beta

What

Move rich text editor feature to beta
Why

How Tested

Process & policy checklist

[x] I have updated the project documentation to reflect my changes if necessary.
[x] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

Should this banner be at the top of the page?


Reconsider hiding flex-options in the flex-Merit Order as they are not installed

From https://github.com/quintel/etmodel/issues/3335#issuecomment-621795821:

I think we should revisit the decision to hide flex options with no capacity, but that's a discussion which I'll take elsewhere.

The hiding keeps the list nice and clean but has the annoying side-effect that you have to re-position a technology that you have (temporarily) removed by setting its capacity to zero. This happens quite easily and it turns out to be very hard to remember revisiting the flex MO every time you re-introduce a technology.
Perhaps we can grey-out non-installed technologies instead of hiding them? Or remember their place in the MO from before even when they have been removed so they can re-appear where the user left them?

I prefer greying them out rather than hiding them completely. Doing so...

Allows you to arrange technologies in the flexibility order then set the capacities of the technologies, rather than forcing you to do it the other way around.
Might spur some to investigate and learn more about the options available in the ETM. The default, which contains only "Export" doesn't help the user understand that there are flexibility options they aren't using.

I agree with greying out


[HOLD for payment 2023-03-01] [$1000] Editing header bold text in chat increases vertical gap above the text

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

Action Performed:

Open the app
Open any chat
Send bold text using #
Edit the text

Expected Result:
Vertical gap above the text should not change after edit
Actual Result:
Vertical gap above the text is increased after editing
Workaround:
unknown
Platforms:

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

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

Version Number: 1.2.61-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:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/215596710-75e3d572-746f-4c2a-a4ed-bf90de2c6368.mp4
https://user-images.githubusercontent.com/43996225/215596739-6b89b0fa-b53d-400f-a285-258b4ba4c432.mp4
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1675067869948869
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011432c27e94a6da98
Upwork Job ID:<PHONE_NUMBER>676652032
Last Price Increase: 2023-02-08

@shawnborton can you confirm this is indeed a bug, and I'll get it moved over?

Shawn is OOO, but back this week!

Hm yeah, this seems like a bug! It's a bit strange that there was even a blank line inserted above the #/headline in the first place.

Replicated

Job added to Upwork: https://www.upwork.com/jobs/~011432c27e94a6da98

Current assignee @dylanexpensify is eligible for the External assigner, not assigning anyone new.

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

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

Proposal
Problem
After editing a bold text, we can see vertical gap above the text is increased. The reason is an unexpected line break was added before the edited text.
RCA
This issue happens because when we handle edited text, https://github.com/Expensify/App/blob/a171133dbbda4552366511fc2acedec1816e6137/src/libs/actions/Report.js#L821-L832  we convert HTML to markdown and parser.htmlToMarkdown(htmlWithAutoLinks) adds two redundant end-of-line \n before and after our markdown (check the logic here).https://github.com/Expensify/App/blob/f8d7a3a8e291cd216b26b22e9e8a69c93bb15e9e/src/libs/actions/Report.js#L806-L809
But why there is only a line break before the edited text? Normally, the code for end-of-line \n will be replaced by
 when we convert the markdown back to HTML so we should see a line break before and a line break after the edited text. While investigating I found that the second code for end-of-line \n is removed by chance when we convert the markdown to HTML. https://github.com/Expensify/App/blob/a171133dbbda4552366511fc2acedec1816e6137/src/libs/actions/Report.js#L830-L831
Solution
We need to remove two redundant \n before we convert markdown to HTML. After that, there will be no unexpected line breaks . We will trim markdownForNewComment as below:
diff --git a/src/libs/actions/Report.js b/src/libs/actions/Report.js
index 7abb23497..2172559dc 100644
--- a/src/libs/actions/Report.js
+++ b/src/libs/actions/Report.js
@@ -825,8 +825,8 @@ function editReportComment(reportID, originalReportAction, textForNewComment) {
     // https://github.com/Expensify/App/issues/9090
     // https://github.com/Expensify/App/issues/13221
     const originalCommentHTML = lodashGet(originalReportAction, 'message[0].html');
-    const markdownForNewComment = handleUserDeletedLinks(textForNewComment, originalCommentHTML);
-
+    let markdownForNewComment = handleUserDeletedLinks(textForNewComment, originalCommentHTML);
+    markdownForNewComment = markdownForNewComment.trim();
     const autolinkFilter = {filterRules: _.filter(_.pluck(parser.rules, 'name'), name => name !== 'autolink')};
     const htmlForNewComment = parser.replace(markdownForNewComment, autolinkFilter);
     const parsedOriginalCommentHTML = parser.replace(parser.htmlToMarkdown(originalCommentHTML), autolinkFilter);

Result
https://user-images.githubusercontent.com/113963320/216362743-1db9f146-12ff-43ae-96c1-36e4ccbff59e.mov

I reviewed @eh2077 and @Tienifr's proposals. Great.
But elsewhere the string edited is laid horizontally, but adding bold puts it vertically.
To fix this, you need to follow the style:
flex-direction: row;
align-items: center;
Applying style eliminates the need to remove two duplicate \n's before converting the markdown to HTML.
All problems can be solved by adding only 2 styles.
thanks.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Vertical gap above the text is increased after editing a h1 tag
What is the root cause of that problem?
In htmlToMarkdown, we're adding new line to both before and after the # , which should not be the case, we should only add new lines if there's non-new-line content before and after the heading.
Why non-new-line? because if before the heading there's already a new line, the #  will be on the next line already and we should not add a (redundant) new line in this case. This complies with how HTML renders the h1 tag.
What changes do you think we should make in order to solve the problem?

In here, we should use a regex that satisfies the requirement we should only add new lines if there's non-new-line content before the heading.
diff --git a/lib/ExpensiMark.js b/lib/ExpensiMark.js
index bac4b81..5622980 100644
--- a/lib/ExpensiMark.js
+++ b/lib/ExpensiMark.js
@@ -229,8 +229,8 @@ export default class ExpensiMark {
             },
             {
                 name: 'heading1',
-                regex: /\s*<(h1)(?:"[^"]*"|'[^']*'|[^'">])*>(.*?)<\/\1>(?![^<]*(<\/pre>|<\/code>))\s*/gi,
-                replacement: '\n# $2\n',
+                regex: /([^\r\n]*?)\s*<(h1)(?:"[^"]*"|'[^']*'|[^'">])*>(.*?)<\/\2>(?![^<]*(<\/pre>|<\/code>))\s*([^\r\n]*)/gi,
+                replacement: (match, g1, g2, g3, g4, g5) => `${g1}${g1 && '\n'}# ${g3}${g5 && '\n'}${g5}`,
             },
             {
                 name: 'listItem',

After this fix, another issue will occur. Let's consider this HTML

heading 1

heading 2
. In here if both h1 blocks matches the regex, it will add 2 new lines in between the blocks, one after heading 1 and one before heading 2. This is because in the match, heading 1 has non-new-line content after it and heading 2 has non-new-line content before it. This is incorrect understanding since after heading 1 is matched, it will add 1 new line after it and heading 2 no longer has non-new-line content before it.
In order to fix this we need to introduce another property into the rule, which can be sequential: true. If the rule is sequential, the regex replacement will be applied once at a time. Example of this in htmlToMarkdown:
if (rule.sequential) {
                while (rule.regex.test(generatedMarkdown)) {
                    generatedMarkdown = generatedMarkdown.replace(rule.regex, rule.replacement);
                }
            } else {
                generatedMarkdown = generatedMarkdown.replace(rule.regex, rule.replacement);
            }

Of course the regex above will need to have the global flag /g removed for this to work.
What alternative solutions did you explore? (Optional)

Proposal
Updated
To fix the following test case mentioned by @tienifr 's in his latest proposal https://github.com/Expensify/App/issues/14668#issuecomment-1423712701

heading 1

heading 2

I'd like to update the solution section of my previous proposal
What changes do you think we should make in order to solve the problem?
I'd like to propose a new solution to improve the regex rule which translates html
 header to markdown. The new solution is to replace
https://github.com/Expensify/expensify-common/blob/668be7533215fc8d683d658da839361664320965/lib/ExpensiMark.js#L233
with
replacement: '[block]# $2[block]',

By replacing
\n# $2\n

with
[block]# $2[block]

we can reuse this function ExpensiMark#replaceBlockWithNewLine to handle line break properly. See
https://github.com/Expensify/expensify-common/blob/668be7533215fc8d683d658da839361664320965/lib/ExpensiMark.js#L521
https://github.com/Expensify/expensify-common/blob/668be7533215fc8d683d658da839361664320965/lib/ExpensiMark.js#L455-L467

/**
* replace [block] with '\n' if :
* 1. We have text before the [block].
* 2. The text does not end with a new line.
* 3. The text does not have quote mark '>' .
* 4. [block] not the last [block] in the string.
*
* @param {String} htmlString
* @returns {String}
*/
replaceBlockWithNewLine(htmlString) {

@eVoloshchak what's your saying on these proposals?

I see a couple of good proposals, reviewing tomorrow

Thanks @eVoloshchak

@danieldoglas @eVoloshchak @dylanexpensify this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

I like @eh2077's proposal, great explanation of the root cause. Both @tienifr's and @eh2077's proposals have a lot in common, but @eh2077 was the first to propose this approach.
We'll need to test more thoroughly during the review (as I couldn't test ALL of the cases), but from my initial testing, this works as expected.
🎀👀🎀 C+ reviewed!
cc: @danieldoglas

Hi @eVoloshchak , thanks for reviewing the proposal.
I think my updated solution will be a better one to address this issue. My original solution can only handle text with only one h1 tag. Can you also have a review on it?

I think my updated solution will be a better one to address this issue. My original solution can only handle text with only one h1 tag. Can you also have a review on it?

Yeah, I tested your updated proposal when reviewing.
I linked the original comment because it has an explanation of the problem, but you're correct, I should include the link to your updated proposal also, thank you.

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

Thanks @danieldoglas!
I applied the job and PR https://github.com/Expensify/expensify-common/pull/502 for expensify-common is ready for review.
cc @eVoloshchak @danieldoglas

@eVoloshchak FYI if you comment in the original PR, I can assign you to review there directly.

hey @eh2077 can you create a new PR updating the expensify-commons library in the app and do the full checklist on App?
The merge commit hash is bf099f71a09bd13cdaef24b30ee63b4d801a686c

Hi @eVoloshchak and @danieldoglas
PR is ready for review, thanks!

Kindly bump @danieldoglas @eVoloshchak

Testing in a couple of hours

Reviewing label has been removed, please complete the "BugZero Checklist".

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

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

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

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

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

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

📣 @MelvinBot! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.

Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

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

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

Regression Test Proposal

Open a chat and send this message
# heading

Edit the sent message and verify it's displayed in one line in editing mode

Edit and change the sent message to
# heading test

and save it

Verify the edited header bold text is still displayed in one line

Do we agree 👍 or 👎

📣 @eh2077! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.

Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0184a9d594d4a0e25a

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

@eVoloshchak can you fill this https://github.com/Expensify/App/issues/14668#issuecomment-1440926211? Thanks!

@dylanexpensify I think we should include this as part of the tests related to the markdown edits.

@eVoloshchak can you fill this #14668 (comment)? Thanks!

Sure, doing it today

The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/expensify-common/pull/468

The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: the PR description mentioned the design doc, which I can't access, I can't verify if it's a regression or was by design at the time

A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: automated regression tests will catch this

Whipping up RT and payments today

@dylanexpensify I think we're being very generalistic in Edit a message with Markdown. the test suggested here will test exactly for the situation where editing a header, so I think it makes sense to update it.

Bump @danieldoglas! 🙇♂

@dylanexpensify I think we're being very generalistic in Edit a message with Markdown. the test suggested here will test exactly for the situation where editing a header, so I think it makes sense to update it.

Hi @dylanexpensify , is this comment from @danieldoglas you're waiting for?

Wow, yes @eh2077 haha, I didn't refresh issue before posting 🤦

RT linked above! Working on payments now!

@eh2077 sent offer!
@eVoloshchak sent invite to apply to job!
@dhanashree-sawant sent invite to apply to job as reporter!

@eh2077 payment sent!
@eVoloshchak offer sent!

@dylanexpensify Thanks and accepted the offer!

@dhanashree-sawant please apply when you can!

Offer sent @dhanashree-sawant !

alrightyyyy we're all settled up!! Thanks everyone!!


Issue with the upcoming Factory Bot Rails due to lacking of lazy load hooks

Hello,
I'm using database_cleaner-active_record together with factory-bot-rails in an application.
The issue is documented here: thoughtbot/factory_bot_rails#426
Here it is a reproducible test case:
# frozen_string_literal: true

require 'bundler/inline'

begin
  gemfile(true) do
    source 'https://rubygems.org'

    gem 'pg'
    gem 'rails'
    # Comment out `git` and `branch` params to use the latest stable version of factory bot and make the test pass
    gem 'factory_bot_rails', git: 'https://github.com/thoughtbot/factory_bot_rails.git', branch: 'main'
    gem 'database_cleaner-active_record'
  end

  require 'active_record'
  require 'factory_bot_rails'
  require 'database_cleaner-active_record'
rescue Gem::LoadError => e
  puts "\nMissing Dependency:\n#{e.backtrace.first} #{e.message}"
rescue LoadError => e
  puts "\nError:\n#{e.backtrace.first} #{e.message}"
  exit 1
end

puts 'Ok'

Error:
~/.rvm/gems/ruby-3.2.2/gems/railties-7.1.1/lib/rails.rb:51:in `configuration': undefined method `config' for nil:NilClass (NoMethodError)

      application.config
                 ^^^^^^^
	from ~/.rvm/gems/ruby-3.2.2/bundler/gems/factory_bot_rails-fd1ca67b43f4/lib/factory_bot_rails/railtie.rb:25:in `block in <class:Railtie>'
	from ~/.rvm/gems/ruby-3.2.2/gems/activesupport-7.1.1/lib/active_support/lazy_load_hooks.rb:97:in `class_eval'
	from ~/.rvm/gems/ruby-3.2.2/gems/activesupport-7.1.1/lib/active_support/lazy_load_hooks.rb:97:in `block in execute_hook'
	from ~/.rvm/gems/ruby-3.2.2/gems/activesupport-7.1.1/lib/active_support/lazy_load_hooks.rb:87:in `with_execution_control'
	from ~/.rvm/gems/ruby-3.2.2/gems/activesupport-7.1.1/lib/active_support/lazy_load_hooks.rb:92:in `execute_hook'
	from ~/.rvm/gems/ruby-3.2.2/gems/activesupport-7.1.1/lib/active_support/lazy_load_hooks.rb:78:in `block in run_load_hooks'
	from ~/.rvm/gems/ruby-3.2.2/gems/activesupport-7.1.1/lib/active_support/lazy_load_hooks.rb:77:in `each'
	from ~/.rvm/gems/ruby-3.2.2/gems/activesupport-7.1.1/lib/active_support/lazy_load_hooks.rb:77:in `run_load_hooks'
	from ~/.rvm/gems/ruby-3.2.2/gems/activerecord-7.1.1/lib/active_record/base.rb:338:in `<module:ActiveRecord>'
	from ~/.rvm/gems/ruby-3.2.2/gems/activerecord-7.1.1/lib/active_record/base.rb:15:in `<top (required)>'
	from <internal:~/.rvm/rubies/ruby-3.2.2/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:38:in `require'
	from <internal:~/.rvm/rubies/ruby-3.2.2/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:38:in `require'
	from ~/.rvm/gems/ruby-3.2.2/gems/database_cleaner-active_record-2.1.0/lib/database_cleaner/active_record/truncation.rb:2:in `<top (required)>'
	from <internal:~/.rvm/rubies/ruby-3.2.2/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:38:in `require'
	from <internal:~/.rvm/rubies/ruby-3.2.2/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:38:in `require'
	from ~/.rvm/gems/ruby-3.2.2/gems/database_cleaner-active_record-2.1.0/lib/database_cleaner/active_record.rb:4:in `<top (required)>'
	from <internal:~/.rvm/rubies/ruby-3.2.2/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:38:in `require'
	from <internal:~/.rvm/rubies/ruby-3.2.2/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:38:in `require'
	from ~/.rvm/gems/ruby-3.2.2/gems/database_cleaner-active_record-2.1.0/lib/database_cleaner-active_record.rb:1:in `<top (required)>'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler/runtime.rb:60:in `require'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler/runtime.rb:60:in `block (2 levels) in require'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler/runtime.rb:55:in `each'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler/runtime.rb:55:in `block in require'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler/runtime.rb:44:in `each'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler/runtime.rb:44:in `require'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler/inline.rb:66:in `block (2 levels) in gemfile'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler/settings.rb:140:in `temporary'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler/inline.rb:51:in `block in gemfile'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler.rb:411:in `block in with_unbundled_env'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler.rb:657:in `with_env'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler.rb:411:in `with_unbundled_env'
	from ~/.rvm/gems/ruby-3.2.2/gems/bundler-2.4.20/lib/bundler/inline.rb:42:in `gemfile'

It is due to the fact that Database Cleaner does not use lazy hooks before using ActiveRecord. You can see a fix to the same issue with another gem here: amoeba-rb/amoeba#115
However, the fix does not appear to be so trivial with Database Cleaner, so this is why I'm opening an issue rather than submitting a PR

Hopefully this will be addressed by Factory Bot Rails itself in thoughtbot/factory_bot_rails#432

Having the same issue here

Please try with the branch used in https://github.com/thoughtbot/factory_bot_rails/pull/432

I think this gem should follow lazy loading like other project:

https://github.com/paper-trail-gem/paper_trail/commit/fc6c5f69410f1b79e617610d522dbedfa4f04f7f
https://github.com/rails/rails/issues/46567


Offer join form #982 [WIP]

This is work in progress PR.
At this point there are form inputs that successfully prepopulate data (fullname, email, phone_no) if the user is logged in.  Currently working on CSS (messed paddings in  and submitting the form.
@juliaanholcer since I kinda work with trials and erros here without much of the big picture knowledge - do we need a service here to work with API @w1stler is currently working on, like e.g. the one which serves the contact administrator form?

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1034   +/-   ##
=======================================
  Coverage   89.46%   89.46%
=======================================
  Files         113      113
  Lines        2667     2667
  Branches       31       31
=======================================
  Hits         2386     2386
  Misses        277      277
  Partials        4        4

Continue to review full report at Codecov.

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

Because of git problems I needed to open this change in another PR. Go to #1066


List view formatting: customCardProps triggered to fast or not at all (MS lists view in Sharepoint list)

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
Declarative list formatting
Developer environment
None
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[X] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details
Chrome 127
Describe the bug / error
customCardProps do not work properly in new list experience.
Moving the mouse over such items, triggers the popup too fast and sometimes not at all. Sometimes the popups do not hide again.
This behaviour was different in the old list and is still different in document libraries.
New List experience:
https://github.com/user-attachments/assets/bd8fa9b4-9292-444c-9406-ac76dbce2573
Document library:
https://github.com/user-attachments/assets/6df51990-c05f-409f-8390-a18cae40e2b3
Steps to reproduce

Create a list
Add a column (e.g. multiple choice in this example)
Add custom json to column with customCardProps (example below)
Move mouse over the customCardProps items

Column formatting example:
`{
  "$schema": "https://developer.microsoft.com/json-schemas/sp/v2/column-formatting.schema.json",
  "elmType": "div",
  "style": {
    "flex-wrap": "wrap",
    "display": "flex"
  },
  "children": [
    {
      "forEach": "__INTERNAL__ in @currentField",
      "elmType": "div",
      "style": {
        "box-sizing": "border-box",
        "padding": "4px 8px 5px 8px",
        "overflow": "hidden",
        "text-overflow": "ellipsis",
        "display": "flex",
        "border-radius": "16px",
        "height": "24px",
        "align-items": "center",
        "white-space": "nowrap",
        "margin": "4px 4px 4px 4px"
      },
      "attributes": {
        "class": {
          "operator": ":",
          "operands": [
            {
              "operator": "==",
              "operands": [
                "[$__INTERNAL__]",
                "Auswahl 1"
              ]
            },
            "sp-css-backgroundColor-BgCoral sp-css-borderColor-CoralFont sp-css-color-CoralFont",
            {
              "operator": ":",
              "operands": [
                {
                  "operator": "==",
                  "operands": [
                    "[$__INTERNAL__]",
                    "Auswahl 2"
                  ]
                },
                "sp-css-backgroundColor-BgDustRose sp-css-borderColor-DustRoseFont sp-css-color-DustRoseFont",
                {
                  "operator": ":",
                  "operands": [
                    {
                      "operator": "==",
                      "operands": [
                        "[$__INTERNAL__]",
                        "Auswahl 3"
                      ]
                    },
                    "sp-css-backgroundColor-BgPeach sp-css-borderColor-PeachFont sp-css-color-PeachFont",
                    {
                      "operator": ":",
                      "operands": [
                        {
                          "operator": "==",
                          "operands": [
                            "[$__INTERNAL__]",
                            "Auswahl 4"
                          ]
                        },
                        "sp-css-backgroundColor-BgGold sp-css-borderColor-GoldFont sp-css-color-GoldFont",
                        {
                          "operator": ":",
                          "operands": [
                            {
                              "operator": "==",
                              "operands": [
                                "[$__INTERNAL__]",
                                "Auswahl 5"
                              ]
                            },
                            "sp-css-backgroundColor-BgMintGreen sp-css-borderColor-MintGreenFont sp-css-color-MintGreenFont",
                            {
                              "operator": ":",
                              "operands": [
                                {
                                  "operator": "==",
                                  "operands": [
                                    "[$__INTERNAL__]",
                                    "Auswahl 6"
                                  ]
                                },
                                "sp-css-backgroundColor-BgCyan sp-css-borderColor-CyanFont sp-css-color-CyanFont",
                                {
                                  "operator": ":",
                                  "operands": [
                                    {
                                      "operator": "==",
                                      "operands": [
                                        "[$__INTERNAL__]",
                                        "Auswahl 7"
                                      ]
                                    },
                                    "sp-css-backgroundColor-BgSage sp-css-borderColor-SageFont sp-field-fontSizeSmall sp-css-color-SageFont",
                                    "sp-field-borderAllRegular sp-field-borderAllSolid sp-css-borderColor-neutralSecondary"
                                  ]
                                }
                              ]
                            }
                          ]
                        }
                      ]
                    }
                  ]
                }
              ]
            }
          ]
        }
      },
      "txtContent": "[$__INTERNAL__]",
      "customCardProps": {
        "formatter": {
          "elmType": "div",
          "style": {
            "padding": "12px"
          },
          "txtContent": "Hallo Welt!"
        },
        "openOnEvent": "hover",
        "directionalHint": "topAutoEdge",
        "isBeakVisible": true
      }
    }
  ]
}

Expected behavior
Popup of customCardProps should only be trigged after a short delay. After leaving the popup with the mouse, it should hide again after a short delay.

Maybe another aspect is that you would not expect that multiple popups appear at the same time and are overlapping each other.

@XueSheng-GIT Thanks for reporting the issue. We will work on this on priority and update you the ETA.

@tpodugu-ms any progress on this matter? The issue still persists and I'm getting more an more complaints because the customCardProps do not work properly.

@tpodugu-ms Sorry to bug you again. Any info to the ETA?
This bug is quite annoying becaus those accidentially triggered customCardProps also overlay the list form so that info is hidden or fields cannot be filled (you first need to close the form, try to get rid of the accidentially triggered customCardProps and open the list form again.

Hi @XueSheng-GIT. Team is actively working on the custom formatter issues. I will check with them and update.


[macos] os sends message saying file open has failed

specs
macbook air 2020
m1
ventura 13.4
error
The open file operation failed., The open file operation failed to connect to the open and save panel service.
reproduce
Run this example code on macos and it will not work, I don't have enough access to different mac's to isolate the issue.
let path = FileDialog::new()
  .set_location("~/")
  .show_open_single_file()
  .unwrap();

println!("{:?}", path);

The output on mac is None.

Confirming this is an issue for me too. The offending snippet for me is pasted below and causes the same effects:
               self.sink.add_to_queue(
                    match FileDialog::new()
                        .set_location("~")
                        .show_open_multiple_file()
                    {
                        Ok(val) => Some(val),
                        Err(_) => None,
                    },
                );

I had the same issue and fixed it by adding a filter to the File Dialog: .add_filter("PNG Image", &["png"]).
Still, it's odd that it fails silently without any compilation or runtime error :(


Correct citation link

I noticed the citation link in the readme pointed to a non-existent page. This fixes it.

Thanks!
I need to do a check of the URLS in the repo in the future.


[web]: http get response null on web and working on android and iOS

Future<http.Response> fetchAlbum() {
  String url = 'http://api.plos.org/search?q=title:DNA';
  return http.get(url);
}

Frame is not available
it return nothing
butt when I run this on android and IOS it work fine

Hi @abdulrehmananwar
can you please provide your flutter doctor -v ,
your flutter run --verbose
and if possible a reproducible minimal code sample.
Thank you

any error in Chrome console? like CORS or other

I have same issue.  When I execute below code in flutter web, it return nothing.
But in flutter iOS, works well
...
RaisedButton(
child: Text('RaisedButton', style: TextStyle(fontSize: 24)),
onPressed: () async{
http.Response response = await http.get('https://m.naver.com/include/grid/panel_DATA.shtml');
dom.Document document = parser.parse(response.body);
List<dom.Element> keywordElements = document.querySelectorAll('.cd_list .cd_item');
List<Map<String, dynamic>> keywords = [];
for (var element in keywordElements) {
dom.Element link = element.querySelector('a');
dom.Element rank = element.querySelector('.cd_num');
dom.Element title = element.querySelector('.cd_tit');
keywords.add({
'rank': rank.text,
'title': title.text,
'url': link.attributes['href']
});
}
print(keywords);
},
),
...

Hi @donmani0307
I'm not sure if the case you are describing match the current issue,
since appears to involve dom manipulation,
I'd recommend to open a separate issue
with your flutter doctor -v ,
your flutter run --verbose
and especially a reproducible minimal code sample.
Thank you

My issue resolved i was using asp.net rest api and it return xml formated data when i have changed rest api response from xml to json fromat its working quite fine on web
Sent from Yahoo Mail on Android
On Wed, 25 Mar 2020 at 14:40, Francecsco<EMAIL_ADDRESS>wrote:
Hi @donmani0307
I'm not sure if the case you are describing match the current issue,
since appears to involve dom manipulation,
I'd recommend to open a separate issue
with your flutter doctor -v ,
your flutter run --verbose
and especially a reproducible minimal code sample.
Thank you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi @abdulrehmananwar
I'm glad to hear that you solved the issue
Thank you for letting me know


Add registry search

Description
refactor image search, using it to add a registry search
Motivation and Context
simplify code
simplify usage
How Has This Been Tested?
manual
Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.

We will probably want to rename this script, as it now reports on images outside of a deployed container (and to remove the "vuln" abbreviation).


Update waas-servicing-channels-windows-10-updates.md, issue 835

issue #835 clarification,
source:

Windows 10 and the “disappearing” SAC-T

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

Dear @nenonix ,
your edits were applied
Thank you

@jaimeo - can you please assist to check, approve and merge the content for this PR based on issue #835?
Thank you.

@jaimeo - I would like to follow up on this. Kindly assist to check, approve and merge the content for this PR based on issue #835.
Thank you.

We will be updating all topics that refer to Semi-Annual Channel (Targeted) for an upcoming release--most references will be removed; however, certain instances must remain for a little while. I have a SME who will need to double-check this particular proposed edit prior to acceptance--I'll ask her to have a quick look at this now.

@jaimeo - I would like to follow up. Do you have any updates on this?

@jaimeo - I would like to follow up on this. Kindly assist to check, approve and merge the content for this PR based on issue #835.
Thank you.


Focus after autoclose

If I activate the autoclose option the cursor is somewhere but not in the input field which has called datepicker.
Datepicker should set the focus to the correct input field after hiding datepicker.

+1
Weird. This doesn't happen if I use keyboard to select a date which kind of saves accessibility. If I use the mouse the focus goes no where.

+1


chore(deps): cleaned up irrelevant dependencies in ui-markdown-editor

Closes #52

Changes

Removed rimraf package
Removed npm-run-all package

Flags

NPM documentation for rimraf https://www.npmjs.com/package/rimraf
NPM documentation for npm-run-all https://www.npmjs.com/package/npm-run-all

Screenshots or Video

Related Issues

Issue #52

Author Checklist

[x] Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
[x] Vital features and changes captured in unit and/or integration tests
[x] Commits messages follow AP format
[x] Extend the documentation, if necessary
[x] Merging to master from fork:branchname
[x] Manual accessibility test performed

[x] Keyboard-only access, including forms
[x] Contrast at least WCAG Level A
[x] Appropriate labels, alt text, and instructions

@irmerk @mttrbrts Any updates?

@irmerk I have rebase the commits. Any updates regarding this PR?

Looking at this now @Cronus1007 , apologies for the delay. Can you rebase to resolve conflicts in packages/ui-markdown-editor/package.json?

@irmerk I have resolved merge conflicts.

Still unsure about the huge addition to package-lock when dependencies are being removed. Please rebase to one commit.

@irmerk I have updated a little bit of ui-contract-editor/Readme.md to resolve the newly linked issue.


Error: Missing cursor in initial fetch response

Error: Missing cursor in initial fetch response

please fix these problems

i look into tiktok web view source, im not found any roomId appear in page code, i think tiktok make some change to prevent any web scrapper to get the roomId from live page.


Call Extension when input string is empty

This will be good to display history of actions when input is focused and empty

Merged in as of #38


15-cognitive-search.md - extra bold tag present

In this doc, in the Task 'Azure Cosmos DB for NoSQL 数据的生成索引器和索引', in the Table for Step 3, there are two extra bold tags at the end:
之前创建的 Azure Cosmos DB for NoSQL 帐户的连接字符串**

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Locfiles to update

Update made and locfile uploaded


Extension doesn't show one's not logged into the OpenShift cluster

If I run oc logout in a terminal, when I open the extension, the current context shows as if I was logged in. Subsequent image deployment fails because anonymous user has no rights.
There should be a way to visually show a user is actually logged in or not.

This is something that can happen at any time anywhere and nothing we can do about it.
Simplest way is to check login status before executing command and if user is not logged in show a warning and request to login.
Even this approach can fail sometimes, because session can expire even in the middle of 'oc new-application' execution.


64 bit version of fix string parse. assign 64 bit value to FIX_INT_FIELD during parse

required to receive large integers like OrderID for instance

@divaykin, Travis CI complains about the following:
lib/proto/fix_message.c:97:9: error: no previous prototype for ‘fix_atoi64’ [-Werror=missing-prototypes]

Can you please fix it up?

done thx

@divaykin, pulled, thanks!


Syntax highlighting issue with Atom 1.6

Tags is not highlighted when JSX inside parentheses:

But highlights if without parentheses:

The issue appears in 0.14.1 and does not appear in 0.14.0.

This commit, besides the README change, is the only change between 0.14.0 and 0.14.1.

I can add that the auto closing is not working anymore since Atom 1.6

A duct tape fix here: #149
Either TextMate grammars are crap in terms of describing nested language constructs, or grammar developers can't grasp the concepts that enable doing it properly.

Hi! This should be fixed in v0.14.2.

Closing for now! Feel free to reopen if issues persists.

@orktes thank you for the more generic fix, which [is here](https://github.com/orktes/atom-react/commit/4a600c83f44cedfcb82840168b1a0d81b89f85f2] in case someone is interested (referencing an issue from a commit really helps).
I can't verify the fix because of a funny bug: I've accidentally installed the facebook/react library instead of this package via apm install<EMAIL_ADDRESS>And I'm still struggling to get rid of it to no avail, it got cached somewhere. For the future, you should really consider changing the published name to something like atom-react in the Atom Package Directory.

True! normally I try to remember do that but only noticed the existence of this issue after pushing. Nowadays it seems to be possible to rename packages in a way that request to the old package get redirected to the new one (upgrades etc.). So not a bad idea! I'll look into it.

@orktes the wrong react got cached in ~/.atom/.apm, resolved after removing that directory.

@sompylasar @orktes Highlighting doesn't work for<EMAIL_ADDRESS>and Atom 1.6.0:

JSX file is considered as Plain Text

Highlighting is broken for<EMAIL_ADDRESS>
JSX file is considered as JavaScript (JSX)

It works for<EMAIL_ADDRESS>
JSX file is considered as JavaScript (JSX)

@ilyapalkin Atom sometimes saves the grammar you previously set for that file (i.e. what the broken<EMAIL_ADDRESS>set for you). All my new JSX files are loaded with the proper grammar in v0.14.2.

@silvenon in version 0.14.1 it sets JavaScript (JSX) and higlighting looks similar to https://github.com/orktes/atom-react/issues/148#issue-141835975:

When I install 0.14.2 there is no JavaScript (JSX) grammar at all.

I have no idea why this would happen, have you tried reinstalling the package?

@silvenon I have the same issue with 0.14.2, reinstalling does not help. With 0.14.2, I can't even select the Javascript (JSX) mode (not available in the modes list). When I downgrade to 0.14.0 or 0.14.1, the Javascript (JSX) mode is available, though the syntax highlighting is broken exactly same as @ilyapalkin showed on screenshot.
Tested on Atom 1.6.0, OS X 10.11.3

This is probably the same issue I encountered. Try removing ~/.atom/.apm and then do a fresh install via GUI.

@sompylasar you're right, thanks, cleaning ~/.atom/.apm/react was sufficient

cleaning ~/.atom/.apm/react was sufficient and solved the issue for me

+1

Some weird stuff is happening again.

Not sure if the import statements along with the class definition should look like that.


feat: Add start to dev command; open to specific demo page

Tooling update

Add the start command to the npm run dev script; update documentation
Add logic to spandx to fire a specific demo page if watching 1 component

Testing instructions

npm run dev -> should compile the entire project, set a watch on all components, and fire the build server at: http://localhost:8000/examples/

npm run dev pfe-accordion -> should compile only pfe-accordion (and it's dependencies), set a watch on only pfe-accordion (and it's dependencies), and fire the build server at: http://localhost:8000/elements/pfe-accordion/demo

npm run dev pfe-accordion pfe-icon -> should compile only pfe-accordion and pfe-icon (and their dependencies), set a watch on only pfe-accordion and pfe-icon (and their dependencies), and fire the build server at: http://localhost:8000/examples/

Ready-for-merge Checklist
Check off items as they are completed.  Feel free to delete items if they are not applicable.

[x] Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
[x] Tests have been updated to cover these changes.
[x] Repository compiles and tests pass.
[x] Changelog updated (not needed for documentation updates).
[x] Documentation (README.md, WHY.md, etc.) updated or added.

Merging
Please squash when merging and ensure your commit message uses conventional commit formatting.
Be sure to share your updates with the<EMAIL_ADDRESS>mailing list!

@mwcz or @kylebuch8 this adds the localhost server to the dev command; this is how to run build, watch, + demo pages (no storybook)


remove unused blueprints

@keyki pls

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1993/


Allow search engine indexing of polyfill bundles

It seems cdn.polyfill.io is blocking Googlebot, at least this is what Google Fetch said to me.
And on my case that caused my website not to be indexed at all.
I found a way around this detecting Googlebot on the client and serving a local polyfill.js for it.
I agree that according to "progressive enhancement" my website should my minimally working even without the polyfills, but I thought it would be good to share the problem here.

Thanks for raising this. I hadn't appreciated that this would cause a problem for sites using the service. Our intention was just to avoid pointlessly indexing polyfill bundles, but if you need them for the site to be read by Google, I think that trumps it.

Thanks for the attention. =)
Well, maybe I'm a rare case... Feel free to judge if it's worth or not to allow search engine indexing.


Fix nightly jobs running CBMC latest on the perf suite

As explained in #2952 , the nightly perf runs started failing over the weekend with this error:
The input was compiled with an old version of goto-cc; please recompile
error: goto-cc exited with status exit status: 1

Changing the build process for CBMC to use make instead of cmake is enough for the error to go away. This action run shows how a successful perf job with the change in this PR.
Resolves #2952
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

I’m sorry, but we’ll need to dig deeper to see how the build system should relate to that error.

Fair enough, I've started bisecting on the Dec 15 commits.

The problem comes from PR https://github.com/diffblue/cbmc/pull/5815, where GOTO_BINARY_VERSION is redefined.
Now, I'm not sure where to go from here. The first question that comes to mind is: why is this error not reproducing with make?

When comparing the outputs of both build systems, I've realized that make doesn't actually produce a build folder with these commands.
To confirm this, I made this change to the workflow, and then triggered the CI run. Both the perf and regresssion (ubuntu-22.04) jobs print the same at the end of the Build CBMC step:
/usr/bin/cbmc
5.95.1 (cbmc-5.95.1)

Therefore, it's likely that these jobs aren't testing with the CBMC we're building, but the one that we install through the Setup Kani Dependencies step.
Still, this doesn't explain why the perf job is failing. If it was picking up version 5.91.1 as the output of the which cbmc and cbmc --version suggest, then it shouldn't be failing. But it's possible that building Kani with --release is helping it pick-up the newest version? That's what I'm testing in this other CI run.

Still, this doesn't explain why the perf job is failing. [...] But it's possible that building Kani with --release is helping it pick-up the newest version? That's what I'm testing in this other CI run.

The perf job is picking up the right version because the path to the CBMC binaries is being added to $GITHUB_PATH. The GA runners will use that variable instead of the usual $PATH, so which is not the most reliable way to determine the cbmc location.
Regardless, I've been able to determine that the CBMC being executed in the regression is not the one we're building, but the one we install with the "setup" action earlier. This becomes clear in this CI run, which is running these changes. In that run, the regression fails because it can't find cbmc, while perf also fails because of the version mismatch problem that we're aware of.
Unfortunately, I haven't been able to make the setup action work with two steps using an exclusive condition (i.e., based on inputs.install_cbmc) to determine if the install_deps.sh should be called with --no-cbmc. This would add some complexity here and there, but I don't think we should be installing cbmc in workflows where we want to test a version built from source. The next step is to get that working.

Apologies for having been very quiet on this. Failing with current CBMC HEAD should be expected at this time as the new major release being prepared for CBMC includes several breaking changes. We should either ignore the job’s failure, or disable it, or prepare a Kani branch and use that branch for our testing.

Failing with current CBMC HEAD should be expected at this time as the new major release being prepared for CBMC includes several breaking changes. We should either ignore the job’s failure, or disable it, or prepare a Kani branch and use that branch for our testing.

Agree, #2952 just seems to be the first breaking change to be noticed.
However, I think we need to at least change the regression jobs to use the CBMC built from source. Closing this PR in favor of #2965 which does exacty that.


Commit

Added a Toggle button #96

@Akshay6890 I am tested covid positive for the 2nd time so I was not able to make a demo video. Thanks for merging my PR

@Akshay6890 Sir I am tested covid positive for the 2nd time so I was not able to make a demo video. Thanks for merging my PR

Hope you are doing good @chazuttu, don't worry about this, your health and safety is a high priority, wishing you a speedy recovery 😃


Consolidate results and Display Test reports once for multiple dbt projects

There can be a scenario of having multiple DBT projects.
It might be beneficial to have single dashboard for all the test runs across multiple separate DBT projects.

It would be useful for me too


Feature/freeze

Description
This feature completes the following:

Freeze Activity UI
Cast Freeze Tx
Event Listener

Notes

Issue / Notion doc (if applicable)

Testing

Screenshots (if applicable)

Here is an image showing ui updates depending if the user has voting power.


Use new plugin interfaces provided by plugin pkg

The use of Client() on v2 plugins is being deprecated so that we can be more flexible on the protocol used for plugins.
This means checking specifically if the plugin implements the Client() *plugins.Client interface for V1 plugins, and for v2 plugins building a the client manually.

CircleCI Failed
go build -tags experimental -o "bin/dnet" ./cmd/dnet
github.com/docker/libnetwork/vendor/github.com/docker/docker/pkg/parsers/kernel
vendor/github.com/docker/docker/pkg/parsers/kernel/kernel_unix.go:21:28: cannot convert uts.Release[:bytes.IndexByte(uts.Release[:], 0)] (type []int8) to type string
vendor/github.com/docker/docker/pkg/parsers/kernel/kernel_unix.go:21:69: cannot use uts.Release[:] (type []int8) as type []byte in argument to bytes.IndexByte
Makefile:27: recipe for target 'build-local' failed

Sure.  I will take care of this.
On Mon, Jun 18, 2018 at 2:24 PM, Flavio Crisciani<EMAIL_ADDRESS>wrote:

@selansen https://github.com/selansen can you carry this one over?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/docker/libnetwork/pull/2174#issuecomment-398149583,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKEKn4g-tvoy2F6_Gq4ALgEsV628aJ7tks5t9_BlgaJpZM4UVYGZ
.

This was replaced by #2199


Commit Lenovo Dock Manager <IP_ADDRESS>

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@philjorgensen

there is an error during winget upgrade... see printscreen.

@philjorgensen

there is an error during winget upgrade... see printscreen.

What version did you upgrade from? I couldn't repro this upgrading from <IP_ADDRESS>.


Implement Oatuh 2

Currently the wrapper only allows connecting using an API key but not OAuth2 credentials.

This is currently on hold for higher priority things, work is done on the branch implement_oauth_2


flif-sys and libflif benchmarks

As was discussed in #45.
I've also regenerated sea_snail.flif from png, as existing file was producing the following warning on flif -d:
CORRUPTION DETECTED: checksums don't match (computed: fb34712f v/s read: 6e52a81d)! (partial file?)

I've also wrote a small CLI tool to check that pure Rust and libflif produce the same decoding result. (it accepts file paths via stdin ad does decoding and comparison in parallel with rayon) I don't think we need it in this repository, but I can add it if you want.


Width fix

windowWidth and windowHeight should be width and height. This is such a fun example!

Ooops, I messed up and included the new pitch detection example is this pull request along with #34. So I'll wait to merge this until after.

We can merge #34 if there are no additional comments

Ok, I merged #34!


Analysis server crashes after increasing min SDK constraint to 2.15

After increasing the minimum SDK constraint of our packages to 2.15, the analysis server crashes after a while. Breaking syntax highlighting, code completion, etc. Things worked fine when the constraint was still >=2.12 in our pubspec.yaml files and >=2.14.0 in pubspec.lock files due to some dependency that required it (but using Flutter 2.8/Dart 2.15).

Neither the commit that introduced ImplicitCallReference nor the one that started producing them seems to have updated AstBinaryWriter for them.

I can reproduce this in a unit test in ResynthesizeAstKeepLinkingTest; the issue here is an implicit call reference (c standing in for c.call) in an argument list in a redirecting constructor invocation in a const constructor.

Fixed by https://github.com/dart-lang/sdk/commit/4b16e02a96512df2f3e615fb929e909cf4294656

Hmm I don't see the commit in 2.16.1. https://github.com/dart-lang/sdk/commits/2.16.1 But I could be wrong.


Create architectural diagram/flowchart of the Matrix Service

https://www.lucidchart.com/invitations/accept/cdb424df-a72f-4391-9549-e83364c7234c


Maintenance: Enable Redis e2e tests

Why is this needed?
Python 3.7 support has been dropped and we can now update the AWS CDK library to a version that supports the CfnServerlessCache constructor.
Which area does this relate to?
Automation
Solution
No response
Acknowledgment

[X] This request meets Powertools for AWS Lambda (Python) Tenets
[ ] Should this be considered in other Powertools for AWS Lambda languages? i.e. Java, TypeScript, and .NET

Closed #3718


Fix network interface ordering

Apply udev rules to network interfaces so they keep their name/number, also after removing/adding (other) interfaces.
Applies to router(/vpc) vm only, not for ssvm/proxy.

All green! https://beta-jenkins.mcc.schubergphilis.com/job/cosmic/job/0020-tracking-repo-build/441/


justMyCode doesn't work for standard library

https://github.com/microsoft/vscode-python-debugger/issues/94
See screen cast. I can't step into the tempfile library.

IM we should not generate the property in the launch.json since true is the default anyways. This allows to override the default via the setting and gives final control in the launch.json

Perhaps I need to wati for a new Python ext release, tested now and no joy.


How does the backup work?

It isn't clean in the readme how the backup works. Is it a mirror? If I mess with the official git repository it will mess in the backup one? Or is it just a tree snapshot?
ps: why do I need to provide credentials for public repositories?

Hi @lucaszanella - backup right now is a git clone for the first backup and then git pull on subsequent backups.
Currently the token is used to access the GitHub/GitLab API. Hence it is needed for public repositories as well.

Added as a FAQ entry in https://github.com/amitsaha/gitbackup/pull/58/files


It is better to make the downloaded items' parent folder selected when clicking 'Show in Folder' next to the 'Download All' activity log

Storage Explorer Version: 1.27.0-dev
Build Number: 20221110.2
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Ventura 13.0 (Apple M1 Pro)
Architecture: ia32/x64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Blob Containers.
Create a blob container -> Upload one blob.
Right click blob -> Download the blob by clicking 'Download -> Download All...'.
After the download complete -> Click 'Show in Folder' next to the activity log.
Check whether the downloaded blob's parent folder is selected.

Expected Experience
It is better to make the downloaded blob's parent folder selected.
Actual Experience
Navigate to the download path.

Fix merged into main.

Verified this issue on the build main 20221116.3. Fixed.


Not updating elemental data objects fields

Elemental blocks which do not contain any extra $db properties other than the extend parents (BaseElement) element do not update AlgoliaUUID or AlgoliaIndexed DB records when running AlgoliaReindexAllJob. This is causing duplicate records in algolia when re-running AlgoliaReindexAllJob.
It cannot find record the record in the DB to update as it is on the parent table Element not the class table therefore,
sprintf('UPDATE %s SET %s WHERE ID = %s', $table, $set, $this->owner->ID) is not updating anything

@kartikpatel95 The logic now checks for the correct table for this field (https://github.com/wilr/silverstripe-algolia/blob/main/src/Extensions/AlgoliaObjectExtension.php#L165). The biggest issue would be if you have that field defined on multiple tables - but in that case just manually remove the duplicate column.


Fix error handling in gcmrecord

@chuckhays PTAL

Cover one more case and test added. PTAL @chuckhays

LGTM

lgtm


Download Button gone after Update BBB 2.3.4 -> 2.3.6

Greetings, thank you for this awesome tool.
My download button is gone since that update.
But also there is no 404 error for the mp4 in the DevTools,
so I think the javascript is not checking for the file anymore.
Running ./bbb-mp4-install.sh again does not help. Any ideas?

@BackfischHunter  restore default index.html mv /var/bigbluebutton/playback/presentation/2.3/index_default.html /var/bigbluebutton/playback/presentation/2.3/index.html
and rerun ./bbb-mp4-install.sh

This didn't work for me because my index_default.html had somehow errors.
I had to  add
</script><script src="/playback/presentation/2.3/download-button.js"></script>
to
/var/bigbluebutton/playback/presentation/2.3/index.html
Thanks for the help nonetheless 👍


Request and Discord

Hello. I was wondering why do we need an Invite for the App rather than providing an apk, ipa, exe file without the need of any Invite? Also could you make a Discord Server for the App, so that more people can join in and give Feedback?

I did not understand it too but, we have to wait.

I did not understand it too but, we have to wait.


AssertionError:  when using BachelierNet

I'm able to get the out-of-the box examples to execute successfully (getting_started and iid) when using the generated data but when using differenty toy datasets I get an AssertionError in the cvxcpy module.
/opt/conda/lib/python3.6/site-packages/cvxpy/cvxcore/python/canonInterface.py in nonzero_csc_matrix(A)
    162     # this function returns (rows, cols) corresponding to nonzero entries in
    163     # A; an entry that is explicitly set to zero is treated as nonzero
--> 164     assert not np.isnan(A.data).any()
    165
    166     # scipy drops rows, cols with explicit zeros; use nan as a sentinel

AssertionError:

Steps to reproduce:

Start with getting_started.ipynb
Replace the data generation logic with loading data as described in this issue  (I've used this as well as larger data sets)
Replace the Network Definition with:

from deepdow.nn import BachelierNet

n_channels = X.shape[1]
lookback = X.shape[2]
n_assets = X.shape[3]
max_weight = 0.5
hidden_size = 32

network = BachelierNet(n_channels, n_assets, hidden_size=hidden_size, max_weight=max_weight)

print(network)

Same error occurs even if reducing channels to 1, increasing number of samples, keeping lookback, gap, horizion small (5,0,1).

I would imagine this error comes out when trying to train the network / do the forward pass, right?
Anyway, it is hard to debug without the actual sample that it failed on. However, looking at the traceback, make sure that there is no NaNs in your data. If that is not helpful, just try to find a single X (of shape (n_samples, n_channels, lookback, n_assets)) such that network(X) gives you the above error and post it here!

Not sure what the issue is/was.  I was able to avoid the issue by re-writing my code.  I also eliminated raw_to_Xy in my data loading.  At this point I've been able to increase my dataset size and run through BachelierNet.  It appears to be functional but I'm not fully understanding some of the nuances of DeepDow yet.  More questions to come.
Thank you again for your patience.

Perfect!


PEP 468 – Preserving the order of **kwargs in a function

https://docs.python.org/3/whatsnew/3.6.html#pep-468-preserving-keyword-argument-order
https://peps.python.org/pep-0468/

This was implemented by having dict preserve order in CPython, and is a feature as of Python 3.7.
There's a nice summary of the implementation in this StackOverflow post:
https://stackoverflow.com/questions/39980323/are-dictionaries-ordered-in-python-3-6
Does it make sense to do something similar by modifying CommonDictionaryStorage?

There's also a nice presentation by Raymond Hettinger about modern dictionaries which explains things at a high level.

Does it make sense to do something similar by modifying CommonDictionaryStorage?

Yes, that is probably the way to go.


Daedalus Preview stuck at 99.45%

Using 6.0.2
https://forum.cardano.org/t/daedalus-stuck-at-replaying-ledger-at-99-45/131669/9

Preview needs a new 10.× cardano-node.
We're in the process of upgrading, but it will take a little more work than usual:

#3229,
#3237,
… and a few others soon.

Can confirm this bug is also happening in the main Daedalus 6.0.2 release as well on Windows 10.

Could this be related to using a symbolic link for the Daedalus Mainnet from C:\Users\MYUSERNAME\AppData\Roaming to D:\ ? @michalrus
A while back, the local state was taking up too much disk space on the default C:\ drive in windows so I created a symbolic link to D:\ drive via cmd.exe like:
cd "%USERPROFILE%\AppData\Roaming"
mklink /D "Daedalus Mainnet" "D:\Daedalus Mainnet"

This has worked great for a while but ever since upgrading to 6.0.2, am now currently stuck at "Replaying ledger from on-disk blockchain 89.80%" infinitely, as shown in the following screenshot.

It's been stuck at 89.80% for over 3 days..
Please advise.
Thanks!

Could this be related to using a symbolic link for the Daedalus Mainnet from C:\Users\MYUSERNAME\AppData\Roaming to D:\ ? @michalrus
A while back, the local state was taking up too much disk space on the default C:\ drive in windows so I created a symbolic link to D:\ drive via cmd.exe like:
cd "%USERPROFILE%\AppData\Roaming"
mklink /D "Daedalus Mainnet" "D:\Daedalus Mainnet"

This has worked great for a while but ever since upgrading to 6.0.2, am now currently stuck at "Replaying ledger from on-disk blockchain 89.80%" infinitely, as shown in the following screenshot.

It's been stuck at 89.80% for over 3 days.. Please advise.
Thanks!

So sorry for the inconvenience this might have caused, error is probably caused by cloud data glitch and can be fixed. For the fastest resolution, please click on the link below to our dedicated support line:
Support: IOHK Help Center
Note: Click on the live chat icon at the bottom corner of the page to start a conversation

Looks like a scam ^
I highly doubt that https://cardano-zendesk[.]web[.]app/ is legit. The domain looks fraudulent as heck lol. Didn't click..
Watch out peoples. Don't click the link. Report web.app domain if ya have time. ✌
The official IOHK Support URL is https://iohk.zendesk.com/hc/en-us


License script fails if untracked files / dirs exist in any source repo

The license script fails if any untracked files exist in the source tree. This has tripped me up today with ctags files, stale gclient-managed files, and gclient-created conflict directories.  At a minimum, the script should ignore things in https://github.com/flutter/engine/blob/master/.gitignore

cc @jason-simmons @Hixie

Dupe of https://github.com/flutter/flutter/issues/8115

Note that ignoring things in .gitignore isn't correct, because it's possible that things in .gitignore end up in the binary and so their copyrights should be included. But we should (and already attempt to) ignore many files we know don't end up there, like ctags files, gclient files, etc. See https://github.com/flutter/engine/blob/master/tools/licenses/lib/main.dart#L62


chore: Clean up tests

Add a few more yoto manager tests following @robertbeal approach.   Corrected variables to match docs as well.

Hey, sorry was a bit busy. Thanks for sorting and adding the manager pattern (makes sense I didn't realise HA requires it)
Might be nice to start keeping API and manager functions separate (just in case a non HA user wants to use the library)


Add cross-file anchor link check (use lychee for markdown link checking)

Fixes #1009
I think... Fixes #332 because lychee has better retry behavior

Going to leave it open, so folks have the chance to look at it after the holidays.