Possible race condition

I believe there is a race condition in coro::ring_buffer between the await_ready and await_suspend methods. The lock is released in await_ready and re-acquired in await_suspend. When a reader and a writer are running on independent threads, this provides a window where say a reader's try_read_locked returns false, releases the lock, but before that read operation re-acquires the lock in the await_suspend method, a writer grabs the lock and adds data to the ring buffer, potentially putting the ring buffer in a full state.
I was able to deadlock the ring buffer with a capacity of 2 with one reader and one writer each running on distinct threads.
I have made modifications to your implementation which enables rescheduling the awaiting reader/writer on the same thread pool from which they performed the await read/write, but I don't think those changes are at play in this bug, but I have not ruled them out.
I'll try to submit a PR for my fix in your original code base.

Awesome, good find and I appreciate the explanation.  I'll await the PR.

Sorry, I will get to this. I've been on paternity leave.

Hey no prob, I appreciate you jumping in, take your time.  Congratulations 🎉

I think we just need to recheck if there is anything in await_suspend() since there is the two function calls?  Unconditionally sleeping I think is the problem since something can be added to the ring buffer like you say between these two calls.

@ryanolson I've got a PR up that I think fixes this, working on adding a test to reproduce on the old code as well.

I don't think my test case is reproducing because the consumer task jumps pools, I'll need to explicitly use std::thread probably to reproduce.

@ryanolson I've managed to reproduce the test on the main branch, and it appears to be passing now, please take a look if you have sometime, thanks.


Plugin Architecture

Use vue plugin system and vuex module system to create a plugin architecture.  Eventually move all modules to plugin architecture, start with auth, data, test, prefs, deploy.

Architecture an initial plugins created.


[BEAM-2885] Maintain In-flight preparation requests

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Add a struct to maintain things that a job that is being prepared (but
has not been run) needs to maintain.
Launch an ArtifactStagerService on local prepare job requests.
Add a basic test that stages some files.
Move InProcessServerFactory to the real Fn Execution jar

R: @lukecwik
CC: @herohde

For the ReferenceRunner, I'm not hugely concerned with the resource usage per job submission. However; it would be preferable to have only a single instance in order to ensure that clients are setting appropriate metadata to stage files in the correct location.

Run Java Gradle Precommit

run java gradle precommit

run java precommit


Linux version installs but does not run

I have installed PFV on Linux  Mint 18.3. When starting photonfileviewer.run nothing happens. (it is marked as executable)

Ok, I will try to install a Linux desktop someware and test it.
Until then, you can run it from the command line: java -jar PhotonFileCheck.jar

@NardJ does this till happen with the new releases?

@X3msnake Extract archive and using "java -jar /home/nard/PhotonFileUtils-OpenGL/plugins/PhotonFileValidator/PhotonFileValidator.jar" it works. I think this should be enough for Linux users.


[MANIFEST] turn on default pool in prod

What happens when your PR merges?

Prefix the title of your PR:

[MANIFEST] or [AUTO-PR] - tag main as a new patch release and deploy to production

What are you changing?

[ ] Releasing a new version of Notify
[x] Changing kubernetes configuration

Provide some background on the changes
Use the default Pinpoint pool in production.
Note that I have enabled shared routes on both pools, so international sending should now work.
$ make diff-production

< AWS_PINPOINT_DEFAULT_POOL_ID=
---
> AWS_PINPOINT_DEFAULT_POOL_ID=pool-1240066b32d34ce896fc233a9c1c02fe

If you are releasing a new version of Notify, what components are you updating

[ ] API
[ ] Admin
[ ] Documentation
[ ] Document download API

Checklist if making changes to Kubernetes:

[x] I know how to get kubectl credentials in case it catches on fire

After merging this PR

[ ] I have verified that the tests / deployment actions succeeded
[ ] I have verified that any affected pods were restarted successfully
[ ] I have verified that I can still log into Notify production
[ ] I have verified that the smoke tests still pass on production
[ ] I have communicated the release in the #notify Slack channel.

conflict 😱  Will redo PR when we want to turn it on...


data does not hit and does not appear on the graph

Dear All,
Please help, data does not hit and does not appear on the graph, first day of chart install and normal hits.

Thank You

It seems you might either be referencing the wrong access log or using an incorrect log/date/time format. Please share the log format you're using and a few lines from your access log, so I can take a closer look.

Hi @allinurl
Thanks for the response,
On the first day, after installation the graphics were normal, but on the second day an error occurred like this, if the error from the format I think the first day will occur the same error.
Bellow screen capture after fresh installed

I'm not quite sure what the error in the screenshot you shared is. Could you let me know which browser and version of GoAccess you're using? Also, please ensure that you have the latest version installed.

hi @csandiar19
How to use GOAccess for output html report? Real-time ou simple LOGFILE process?
I see... for me, you is just truncate, delete or replace "same" file, i.e using same filename, e reprocessing every time.
Please, correct me if I'm wrong!
So, you do... something is being wrong in process. And GOAccess is getting is processing file that was empty yet!


fix: decrease and insanely long timeout

Let us fix that insanity I created please.
do not wait 1200/600 minutes for VM
Signed-off-by: Tibor Dudlák<EMAIL_ADDRESS>

LGTM


feat(android): fix android build

Android compile versions increased
Android dependencies updated
Example fixed. Tested both Android & iOS

:tada: This PR is included in version 1.9.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Do we need a new fingerprint ?

Hello MCU tinkerer
Yesterday I was trying to connect my ESP8266 to My Firebase, without success. This morning
I did the same sketch with my ESP32 and it worked.
That is the fingerprint I use
static const char kFirebaseFingerprint[] =
"B8 4F 40 70 0C 63 90 E0 07 E8 7D BD B4 11 D0 4A EA 9C 90 F6";
Any Clue
Have a good one

Hi,
I had the same issue, but when I switched to use the latest master (96eab42c654f2445fd467d32530a1b09013052e2) it seems to be fixed there.

Any one no the ..esp8266 again not working today..please help me..:-(

@bharat9828 / @PencariJawaban - Looks like another fingerprint update was made (#368). I pulled the latest master (33363f7a8097cf212231a96867ef0fc02cbea59b) which seems to resolve the issue.

For everyone another fingerprint was just made , so look at the commit on 19th April.

It's not working again today February 28, 2020. Does anyone got an updated Fingerprint? Please help

It's not working again today February 28, 2020. Does anyone got an updated Fingerprint? Please help
this is the latest fingerprint id
03 D6 42 23 03 D1 0C 06 73 F7 E2 BD 29 47 13 C3 22 71 37 1B

It's not working again today February 28, 2020. Does anyone got an updated Fingerprint? Please help
this is the latest fingerprint id
03 D6 42 23 03 D1 0C 06 73 F7 E2 BD 29 47 13 C3 22 71 37 1B

Thanks, let me try it

It's not working again today February 28, 2020. Does anyone got an updated Fingerprint? Please help
this is the latest fingerprint id
03 D6 42 23 03 D1 0C 06 73 F7 E2 BD 29 47 13 C3 22 71 37 1B

Wow! Thanks so much It's working again, You save my life, God bless you :)

"03 D6 42 23 03 D1 0C 06 73 F7 E2 BD 29 47 13 C3 22 71 37 1B" I tried this id, it worked 1 day, now  it does not work anymore. Anyone has same problem?

"03 D6 42 23 03 D1 0C 06 73 F7 E2 BD 29 47 13 C3 22 71 37 1B" I tried this id, it worked 1 day, now  it does not work anymore. Anyone has same problem?

It seems still working for me, have you replaced it correctly?

Yes i think so, i do not know what to do, this is for university project and if i did not find any solution => FF

Yes i think so, i do not know what to do, this is for university project and if i did not find any solution => FF

You should try to check your database rules and make them 'allow read, write if true'

"03 D6 42 23 03 D1 0C 06 73 F7 E2 BD 29 47 13 C3 22 71 37 1B" I tried this id, it worked 1 day, now it does not work anymore. Anyone has same problem?

same with me, i tried with this fp on my project. it works for 2 weeks but after that my project doesn't connect to firebase

Yes i think so, i do not know what to do, this is for university project and if i did not find any solution => FF

You should try to check your database rules and make them 'allow read, write if true'

Thank you Boss, It solved my problem Rupesh Missal Sampla Vishey!!!!!!


Cannot stop webpack from caching my custom plugin.

I'm using https://www.npmjs.com/package/babel-relay-plugin to make my custom loader in my project. Recently I ran into some issues with this loader and decided to debug by console.log something in the loader code. It turned out that I can only see the log once (after I remove the entire node_modules folder and npm install again). All the subsequence run produces no log. It seems clear to me that webpack cached something. When I put some syntax errors or even clear the entire loader code. Webpack still successfully bundle the files.
This is my config file:
var webpack = require('webpack');
var ExtractTextPlugin = require('extract-text-webpack-plugin');
var path = require('path');

var isProduction = process.env.NODE_ENV === 'production';

function concat() {
  return [].slice.call(arguments).reduce(function (arr, elems) {
    return elems ? arr.concat(elems) : arr;
  }, []);
}

function getJsLoader() {
  return {
    test: /.js$/,
    exclude: /node_modules/,
    loaders: concat(
      'babel?cacheDirectory=true&presets[]=es2015,presets[]=react,presets[]=stage-0,plugins[]=' + path.join(__dirname, 'babelRelayPlugin'),
      !isProduction && 'eslint'
    )
  };
}

function getCssLoader() {
  var module = {
    test: /\.css$/
  };
  var loaders = concat(
    !isProduction && 'style',
    'css?' + concat(
      'importLoaders=1',
      'localIdentName=[name]--[local]--[hash:base64:5]',
      'modules',
      isProduction && 'sourceMap'
    ).join('&')
  );
  if (isProduction) {
    module.loader = ExtractTextPlugin.extract(loaders.join('!'));
  } else {
    module.loaders = loaders;
  }
  return module;
}

module.exports = {
  entry: {
    app: path.resolve(__dirname, 'client'),
    vendor: [
      'react',
      'react-router'
    ]
  },

  output: {
    path: path.resolve(__dirname, '../build'),
    filename: 'scripts/app.js'
  },

  module: {
    loaders: [
      getJsLoader(),
      getCssLoader()
    ]
  },

  devtool: isProduction ? 'source-map' : 'eval',

  plugins: [
    new webpack.optimize.CommonsChunkPlugin('vendor', 'scripts/vendor.js')
  ].concat(!isProduction ? [] : [
    new ExtractTextPlugin('styles/main.css')
  ]),

  eslint: {
    configFile: path.resolve(__dirname, '../.eslintrc')
  },

  cache: false,
}

And this is my plugin for js loader:
//babelRelayPlugin
var getBabelRelayPlugin = require('babel-relay-plugin');
var schema = require('./schema.json');
module.exports = getBabelRelayPlugin(schema.data);

I run webpack with this webpack -p --progress --config src/webpack.config.js
A day of asking around and studying the webpack source code gave me no clue.
Hope that some one could help me!
PS: this problem seems only occur when the schema.json is big,

It turns out that Babel loader is the one who cache things. Just need to set remove cacheDirectory=true.


BUG: Fix loading of DICOM files with no preamble

The fix consists of two parts:

In DCMTK and GDCM ImageIO there are two duplicate functions called readNoPreambleDicom. Its implementation was faulty, as it did not consider the two bytes that were already read but rejected as it was not a VR (value representation). These two bytes are now reused for the length, which is now correctly read, which confirms that the order of information in the DICOM header is correct. This function was fixed both places
In DCMTK file reader CanReadFile function the loadFile call confirms that the file is valid despite not having a preamble, in which case it returns an EC_FileMetaInfoHeaderMissing value instead of an EC_Normal. This value needs to be accepted as well

This change can be tested on the datasets mentioned in this SlicerRT issue: https://github.com/SlicerRt/SlicerRT/issues/49
The datasets in question are: aw-4.4-foot (CT, scout), corvus-6.2.2-phantom (CT, dose, RT images), xio-4.33.02-phantom-chest, xio-4.60.00-phantom-chest/t50, (CT, dose), xio-4.60.00-* (CT, dose, RT image), xio-4.64-phantom-prostate-beams-* (CT, dose)
Which can be found here: https://pocus.cs.queensu.ca/#collection/5cc8811601d4930406c40465 or here https://github.com/SlicerRt/SlicerRtData/tree/master

PR Checklist

[X] :no_entry_sign: Makes breaking changes
[X] :no_entry_sign: Makes design changes
[X] :no_entry_sign: Adds the License notice to new files.
[X] :no_entry_sign: Adds Python wrapping.
[X] :no_entry_sign: Adds tests and baseline comparison (quantitative).
[X] :no_entry_sign: Adds test data.
[X] :no_entry_sign: Adds Examples to ITKExamples
[X] :no_entry_sign: Adds Documentation.

Refer to the ITK Software Guide for
further development details if necessary.

@hjmjohnson Please hang on, maybe I overrode previous changes. Sorry!

I thought I checked all files, but apparently not. Fixed now!

@cpinter Thank you.  Is there any chance a test could be added?  Is one of these data sets suitable for creating a test of this code?  That would ensure that future changes would also be able to read those files.

@cpinter @pieper @lassoan THANK YOU for investigatting this issue.  I think it is 90% there.  The two outstanding issues that need to be addressed to get this into ITK robustly for the future:

[ ] Reference test to validate correct results into the future (not strictly required, but it sure would be nice so that we don't revisit this issue in the future).
[ ] address concerns about big/little endedness

Thank you! I intend to take care of these tomorrow.

I tried loading both liettle and big endian files, and they were loaded fine.
I'm adding very simple tests that load an 8KB test file with DCMTK and GDCM. What is the proper way to add data files so that it appears in ExternalData?

The tests work, but I still don't know how to add ExternalData. I tried to follow this
https://itk.org/Wiki/ITK/Git/Develop/Data#ExternalData
but the "CMake will move the original file" step did not happen. I cannot see anything in my Kitware Girder (yes, I added the API key). Help would be appreciated, thanks!

The link you looked at is outdated. New content is here:
https://github.com/InsightSoftwareConsortium/ITK/blob/master/Documentation/Data.md

... and a more direct link:
https://github.com/InsightSoftwareConsortium/ITK/blob/master/Documentation/UploadBinaryData.md

The two test can be merged, and have the instantiated IO depend on a command-line argument.

I would rather not spend more time on this than absolutely necessary, as I have several projects waiting for me. The tests cover the necessary cases, so this change would be only cosmetic (and I'm not sure where the common test should go, since I wouldn't put a GDCM test in DCMTK project or vice versa). Please let me know if it's OK like this.

Uh, I didn't realize they need to be in different subprojects. Keep it as-is then.

I added and tested the download of the external data input file (8.3K dcm file) for these tests. I did a clean build and the tests pass. I updated the PR, I think it should be good to go.
Thanks for the fast reviews!

@dzenanz, it would be nice if you could add a big red warning to https://itk.org/Wiki/ITK/Git/Develop/Data#ExternalData that instructions are outdated and add a link to the current instructions.
You could even replace the entire text of the page with links to relevant sections of current documentation because the page right now is not useful but potentially misleading for developers (and old content can be always retrieved from the page's history).

There is a warning on the top but I missed it so probably something bigger and redder would be useful, I agree :)

... replace the entire text of the page with links ...

Done!

Perfect, thank you!

Excellent, thanks a lot!


Yarn build should maybe run the typescript compiler to ensure types are sound

Is your feature request related to a problem? Please describe.
Recently ran in to an issue where production builds were not building with the typescript compiler because of unsound types in stitches.
Describe the solution you'd like
I'm not familiar with the build scripts in .bin but maybe they should run the typescript compiler to make sure types are good.
Additional context
Would avoid issues like #497

Closing in favour of the related issues.


Stretchable column

Hi,
Thanks for this project.
I can't find any way of stretching only a single column to fill the available width while having other columns have a fixed width with custom cell renderers.
I'm looking for the following behavior:
| <---- flex column that will fill in all available width ---> | fixed width column | fixed width column |
The idea is that the fixed width columns will not allow the width to go below their min width (or their intrinsic width) and the flex column use up the available width after the fixed width columns have taken theirs.
Things I've tried:

Setting a weight of 100 on the stretch column and a weight of 1 on the fixed columns, but then the fixed width columns aren't rendered at all.
Setting lower weights doesn't give the desired behavior either - For example, using weight 10 on the stretch column and 1 on the others gets them to render, but the columns that are supposed to be fixed width still expand and contract when the window is resized.
The width parameter of the columns seems to be ignored with columnsFit: true.
I also tried wrapping the columns in Widgets that apply constraints like ConstrainedBox or IntrinsicWidth, but there was no effect.

Specifically in my case I only need 1 flex column, but I suppose this can be generalized into the following:
How to mix any number of flex columns (that behave according to the weight behavior) with fixed-width columns that do not flex?

Hi @ykrasik! Sorry about the delay.
You cannot find it because it does not exist. :grimacing: :disappointed:
Currently, there are only columnsFit with weight or columns with width.
I understand that you would like weight without columnsFit. But I don't know how it would be to have flex and horizontal scroll at the same time. I would have to think. I don't even know if that exists :smile:

width would be the initial size?
flex stop growing when the sum of the widths is equal or greater than the available area?

This can also be achieved, I think, by adding a minWidth to columns. If each column has a minWidth and a low weight and the flex column(s) would have a higher weight and no minWidth, that would behave as expected, at least in this case.

Hi @caduandrade!
Hope I don't come off as rude, but I need this feature for what I'm working on and this ability would affect whether I can use this library or not.
Is this something you would consider implementing any time soon?

Hi @ykrasik ! Sorry for the delay. I'm in a very busy week (after work my brain is off).
I'm still thinking about how it will behave with flex + window resize + manual column resize.

@ykrasik , the simple case (N columns and a single flex) is easier to think of. Horizontal scrolling could appear if a minimum flex column width is less than the available space. But there are still other more complex cases.
Example: table with 3 flex columns and one of them is resized. When the window is resized, the rule for displaying the scroll will also depend on this delta of the new column size and not just on the minimum width.

Hi @caduandrade - here are my observations. Currently, its already possible for a flex column to completely "push out" another flex column by having a much larger weight (for example, 200 vs 1), resulting in the 2nd flex column having 0 width (OK, I'm not actually sure its 0 width, but its not visible).
So maybe, as a suggestion - allow each column to declare a minimum width. When laying out the columns, each column takes up its minimum width first, and then the flex columns are given an opportunity to grow (sharing the available width according to their flex weight). If a flex column doesn't declare a minWidth and all the columns don't fit in the parent, the flex columns are shrunk, all the way to 0 if needed.
And manual column resize would act as... minWidth?

@ykrasik
I'm thinking of keeping it simple by disabling sizing on columns with flex. I think it makes more sense because if it's flex, it should only grow when it can.
The width attribute could already be used to display or not the scroll. Its default value is 100.
I see minWidth as another feature that can limit resizing.
I'm also seeing how to rename weight to flex and reuse that same attribute but as nullable.
Perhaps it would be better to change the columnsFit parameter to a ColumnsBehavior enum with scrollable and fit values.

Thanks for taking the time to work on this, I'll be happy to test it when it's ready. Will stay tuned for updates 🙂

@ykrasik, mission accomplished.
Sorry for the delay. As I said, work was busy and unfortunately, I still don't use Flutter there. So there's only free time left for this open-source project. I hope that changes someday.
If you need anything else, let me know.

Awesome, thanks for this! This is really not a big delay, quite quick in fact :) Will test!

Hi @caduandrade!
It works, thanks! I played around with it a bit. However, first thing I noticed was that resizable columns behave a bit... unnaturally, when combined with a flex column:

I'm not really sure what can be done here, happy to hear your thoughts.
Also, I know I suggested using fixed widths for the non-flex columns, but when it was time to actually use it I found myself wanting the columns to just dynamically size themselves - a "fit" column. If the flex column takes up all available width, a "fit" column would take up the minimum width it can to accommodate its children. This is useful, for example, if a column is a "datetime" column whose format depends on the locale (and is in a row with a flex column). It would be useful to have it sized dynamically according to its content, instead of specifying its width manually to fit the maximum width date format.

Hi @ykrasik!
Video
The width of a flexible column is used as its minimum width. The flex only applies to available space beyond the width used by all columns.
All columns have a default width of 100. In the video, it looks ugly but I believe it is expected. When decreasing column (col2), column (col1) will occupy the available space because it is flexible. When increasing column (col2), it will first decrease column (col1) which is flexible since column (col3) must already be at its minimum size.
Maybe you want to define the order of priority between the columns as well. With that, maybe col3 could decrease before col1. It's turning into a layout :smile: .
Sizing according to its content
It is something very difficult especially in Flutter and even more in this table because only the visible Widgets are in memory. Even if it does, this feature should be off by default. It would probably be too slow as well :disappointed: .
For the same reason, the cell has a fixed height. It is one of the tips to improve performance.
Perhaps the best would be to define a technique that I've used a lot in Swing and JavaFx which is to define a prototype to represent the width. It would be more performant than comparing against all cell data. I would just have to think about how this would be possible in Flutter.

Hi @caduandrade ,
Thanks for your reply. About the resizing - it feels unnatural because the resize handle is on the right, but the column grows in size to the left. I understand why this is happening, just as a user I would expect a resizable column to resize in the direction of the drag if it is possible, or just not resize at all if not possible. I think the solution that would make it feel natural is to have 2 resize zones for each column, on both sides, and only allow resizing in a direction if there is available width in that direction. In the example above, dragging the right handle to the right would do nothing and dragging the left handle (which currently doesn't exist) would allow the column to resize. This is probably not trivial to implement 😊
As for the dynamic column sizing - I understand. The "prototype" solution is actually great, as in this use case all content would be the same width, but the width is not known upfront, it is set at application boot.

The idea of having 2 resize zones seems to be a good one :tada: . I will reopen it for now to remember.

Would it be possible to add a auto-width parameter?
A simple bool, if true, sets the column width to fit the largest cell in this column.

@SuricateTrading, I created issue #101 for this.

Hi @ykrasik!
I'm dealing with multiple issues in version 4 and I intend to change the behavior of grow in it.
Usability gets even worse when there is more than one column. It becomes as if it were a hybrid behavior. When the total width is smaller than the available width, it grows, generating this strange resize for the end user.
I'll deal with it in task #102
Thank you and sorry for the delay.


Add support for schema extraction from a jar

Add options to create a schema based on pojo class in a jar.

this should be ready for review, I will add some tests later.

Added Test cases

ping

added producer consumer test

run cppTests

run cpp tests

run cpp tests


Module1.1 Assignment

Given what you have learned about Git, why would you want to use Git in your classroom?
Streamline student collaboration

Streamline student collaboration


[language] improve account parsing in functional tests

Motivation
New contributor to the project and noticed at TODO in global_config_tests.rs:
#[test]
fn parse_account() {
    for s in &[
        "//! account: alice",
        "//!account: bob",
        "//! account: bob, 100",
        "//!account:alice,",
        // TODO: The following should parse. Fix it.
        // "//!   account :alice,1, 2",
        "//! account: bob, 0, 0",
    ] {
        s.parse::<Entry>().unwrap();
    }

    for s in &["//! account:", "//! account", "//! account: alice, 1, 2, 3"] {
        s.parse::<Entry>().unwrap_err();
    }
}

and am taking a stab are fixing this.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
executing the following:
cd language/functional_tests/
cargo test

@tnowacki awesome, thanks for the heads up. If there are any issues/PR's that are relevant to changes for how comments are parsed, would love to see them -- maybe there's something else I can help with?

@rloomba Your CLA is processed. Thanks for your contribution.


BBcode/Print/HTML etc buttons not visible using Chrome.

As title says, the buttons to actually print the list are not visible when viewed using google Chrome. The site works when using Safari, but who wants to do that.

Window was the wrong size... Whoops...


Incorrect display of emphasis in markdown files

Issue Type
Bug
Description
In markdown files, when multiple words are emphasised using asterisks and separated by commas, VSCode incorrectly displays all subsequent text in that line as emphasised. The effect does not carry through to the live preview.
Steps:

Create a markdown file (e.g. text.md)
Write a line that contains two words using emphasis markdown, separated by a comma (e.g. "Hello I have found some problems, specifically during my use of emphasis in markdown.")

Expected Results:
Only the words 'problems', 'specifically' and 'during' are italicized in the VSCode editor.
Actual Results:
The entire phrase 'specifically during my use of emphasis in markdown.' is italicized, while the phrase 'problems' is not.
Screenshot:

VS Code Info
VS Code version: Code 1.20.0 (c63189deaa8e620f650cc28792b8f5f3363f2c5b, 2018-02-07T17:09:39.780Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4670K CPU @ 3.40GHz (4 x 3392)

Memory (System)
15.89GB (7.95GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

html-snippets
abu
0.2.1

vscode-markdownlint
Dav
0.13.0

vscode-eslint
dba
1.4.5

pgsql
dou
0.1.0

MagicPython
mag
1.0.12

python
ms-
2018.1.0

azure-account
ms-
0.3.0

PowerShell
ms-
1.5.1

vscode-docker
Pet
0.0.24

python
tht
0.2.3

Reproduces without extensions

This issue was moved to Microsoft/vscode-markdown-tm-grammar#1


Some features from the original are not in this version

Can I change the color of wires or has this feature not been added? it is in the original.

Also the cursor guide.

That feature, along with multiple others have yet to be added unfortunately

This feature is being added in the PR #79, so this issue should also be closed

Thanks.


fix(CI/CD): Run functional tests in chrome headless mode instead of using xvfb

We can run tests against chrome in headless mode. We no longer need to install xvfb and start a virtual display to run the tests.
This PR depends on https://github.com/fabric8-ui/fabric8-planner/pull/2369

@jarifibrahim fabric8/fabric8-ui:SNAPSHOT-PR-2370-1 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2370-fabric8-planner.badger.fabric8.io

@jarifibrahim fabric8/fabric8-ui:SNAPSHOT-PR-2370-4 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2370-fabric8-planner.badger.fabric8.io

The run_functional_tests.sh script now has support for

User defined BASE URL
DIRECT CONNECTprotractor flag
Using chrome in HEADLESS_MODE mode

Also, the local web server for serving planner will be started only if BASE_URL is not defined.
To run protractor in directConnect mode (this is much faster than using selenium server)
DIRECT_CONNECT=true npm run test:funcsmoke
To run chrome in headless mode for running tests
HEADLESS_MODE=true npm run test:funcsmoke
To run tests against an already running planner server, set BASE_URL
BASE_URL=http://localhost:8080 npm run test:funcsmoke
If none of the above variables are set, then the npm run test:funcsmoke script will

Start selenium server (this is used for communicating with the browser)
Start planner locally
Run tests against chrome

NOTE - npm run test:funcsmoke will fail in an environment where there is no display (eg: CI/CD). Set HEADLESS_MODE to run tests in such an environment.

nice @jarifibrahim , It will help in isolating npm targets!


What is x in the first (and only) example?

This is example's code:

import anyio
import asyncclick as click

@click.command()
@click.option("--count", default=1, help="Number of greetings.")
@click.option("--name", prompt="Your name", help="The person to greet.")
async def hello(count, name):
    """Simple program that greets NAME for a total of COUNT times."""
    for _ in range(count):
        if x: await anyio.sleep(0.1)
        click.echo(f"Hello, {name}!")

if __name__ == '__main__':
    hello(_anyio_backend="trio")  # or asyncio, or curio

So what is x I wonder?

That should be for x in range…. Will fix, thanks.

That should be for x in range…. Will fix, thanks.

Good!

Good!

Fixed in current main branch (tracking 8.0)


Dev Tools

Implementing the Svelte Dev Tools will enhance the developer experience, i.e. debugging process. https://github.com/RedHatter/svelte-devtools

All you need is to install the extension, there is one for Chrome and one for Firefox. Do not forget to re-open your browser after having installed the extension. :tada:

All you need is to install the extension, there is one for Chrome and one for Firefox. Do not forget to re-open your browser after having installed the extension. :tada:


Bazel Conference announcement blog post

Yes I meant to leave Remote Execution but remove bazel.buildfarm.
On Mon, Sep 25, 2017 at 10:11 AM helenalt<EMAIL_ADDRESS>wrote:

@helenalt commented on this pull request.
In _posts/2017-09-25-conference-2017.md
https://github.com/bazelbuild/bazel-blog/pull/38#discussion_r140711857:

@@ -0,0 +1,21 @@
+---
+layout: posts
+title: Bazel Conference 2017
+---

+Bazel team is pleased to announce our first annual Bazel Conference, focused on the needs of our community, with user stories and feedback, migration talks, roadmap, hands-on and break-out tech sessions with Bazel engineers, contributors and users.
+
+Dates:     November 6 and 7, 2017
+
+Location: Sunnyvale, California
+
+We are inspired by our community engagement in the upcoming conference. We've received a large number of requests for talks and topics for discussion, and have scheduled tech talks by engineers at large and small companies on working with Bazel in wide variety of languages, including Scala, JavaScript and Go. We are looking forward to having 200 engineers from 35 companies across the world share their experience with Bazel.
+
+We are humbled by the commitment to make Bazel even better, and are seeing engineers develop advanced features, such as Remote Execution bazel.buildfarm, and sharing migration tips, tricks and tools. You will hear user stories and tips about iOS migration, and moving TensorFlow and Kubernetes to Bazel. We will also discuss as a community what are the different tools out there that could be open sourced.
Remove just a link to it? I'd like to mention that there will be a talk on
it.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/bazel-blog/pull/38#discussion_r140711857,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADjHfzGhEg6vF2ECvkFNJ51GVYSt-GY7ks5sl2BHgaJpZM4PhBXK
.

(side-note: please update the date before merging)

This PR should be merged on October 1 so that we give about a week in between the blogposts (latest one was on Sep 27).


Move named types to a separate HIR enum and separate namespacing

Split hir::Definition enum variants that define types into a new NamedTypeDefinition enum.

Add Salsa queries for finding a type by its name, (excluding non-type named definitions), and use them where appropriate. This changes some diagnostics from "wrong definition kind" to "no such type".

Remove find_type_system_definition_by_name and find_definition_by_name queries since they mix up the four namespaces of a GraphQL document that should really be separate. Instead its callers should probably use one of:

find_directive_definition_by_name
find_operation_by_name
find_fragment_by_name
find_type_definition_by_name

I've renamed NamedTypeDefinition to TypeDefinition

(The two commits make sense separately, so this could be rebased instead of squashed.)


Feat/20 our values section of website

Descriptions

Link to UI design (if possible):

Provide screenshots or videos that the function is working.

Provide screenshots of no warning, or error in the console.

[x] All unit tests passing and coverage of testing is more than 90% unless special needs.
[x] No merge conflict with the main branch, make sure your code is up to date with main.
[x] Check PR title is aligned with your ticket.
[x] Check your app will run without crashes.
[x] Check if there are hard-coded values.
[x] No //eslint-disable-next-line no-unused-vars, unless otherwise specified.
[x] Check you are using camelCase && PascalCase correctly.
[x] Remove unused comments.

The headers need to be capitalized, the same as the UI design.
The font-weight of the text "Our Values" seems not the same as our UI design.

LGTM. One more thing, now we decide to add the "components/*.tsx" path into jest.config.js file, this would let the jest run testing files under components folder. Therefore, may be including the whole section in one component test file is the better way which keeps our project consistent.


Donut document parsing export to onnx does not work.

System Info
optimum==1.8.8
python==3.11.3
system linux

Who can help?
The donut export does not work with the following commands, does anybody know how to get this running or know about the status.
optimum-cli export onnx -m naver-clova-ix/donut-base-finetuned-cord-v2 donut_cord2_onnx/
...
...
...
Exception: The post-processing of the ONNX export failed. The export can still be performed by passing the option --no-post-process. Detailed error: Unable to merge decoders. Detailed error: Expected
a dynamic shape for the axis zero of onnx::Reshape_1045, found a static shape: 2

optimum-cli export onnx -m naver-clova-ix/donut-base-finetuned-cord-v2 donut_cord2_onnx/ --no-post-process
...
...
...
- last_hidden_state: max diff = 0.0012216567993164062
Validation 1 for the model donut_cord2_onnx/decoder_model.onnx raised: The exported ONNX model does not have the exact same outputs as what is provided in VisionEncoderDecoderOnnxConfig. Difference: onnx::Reshape_1263, onnx::Reshape_1359, onnx::Reshape_1364, onnx::Reshape_1045, onnx::Reshape_1146, onnx::Reshape_1258, onnx::Reshape_1151, onnx::Reshape_1050
onnxruntime.capi.onnxruntime_pybind11_state.InvalidArgument: [ONNXRuntimeError] : 2 : INVALID_ARGUMENT : Invalid Feed Input Name:encoder_hidden_states
An error occured during validation, but the model was saved nonetheless at donut_cord2_onnx. Detailed error: [ONNXRuntimeError] : 2 : INVALID_ARGUMENT : Invalid Feed Input Name:encoder_hidden_states.
```

Changing the task name to image-to-text instead of image-to-text-with-past does seem to run. However, I assume that this task is set specifically. Although, for me it is unclear why it is set to that particular task.

```
optimum-cli export onnx -m naver-clova-ix/donut-base-finetuned-cord-v2 donut_cord2_onnx/ --no-post-process --task image-to-text
Validating ONNX model donut_cord2_onnx/encoder_model.onnx...
        -[✓] ONNX model output names match reference model (last_hidden_state)
        - Validating ONNX Model output "last_hidden_state":
                -[✓] (2, 1200, 1024) matches (2, 1200, 1024)
                -[x] values not close enough, max diff: 0.00121307373046875 (atol: 0.001)
Validating ONNX model donut_cord2_onnx/decoder_model.onnx...
Validation 0 for the model donut_cord2_onnx/encoder_model.onnx raised: The maximum absolute difference between the output of the reference model and the ONNX exported model is not within the set tolerance 0.001:
- last_hidden_state: max diff = 0.00121307373046875
The ONNX export succeeded with the warning: The exported ONNX model does not have the exact same outputs as what is provided in VisionEncoderDecoderOnnxConfig. Difference: onnx::Reshape_1359, onnx::Reshape_1258, onnx::Reshape_1146, onnx::Reshape_1151, onnx::Reshape_1050, onnx::Reshape_1045, onnx::Reshape_1364, onnx::Reshape_1263.
 The exported model was saved at: donut_cord2_onnx
```

### Information

- [X] The official example scripts
- [ ] My own modified scripts

### Tasks

- [ ] An officially supported task in the `examples` folder (such as GLUE/SQuAD, ...)
- [ ] My own task or dataset (give details below)

### Reproduction

optimum-cli export onnx -m naver-clova-ix/donut-base-finetuned-cord-v2 donut_cord2_onnx/

### Expected behavior

export to run correctly and validation report.

I am currently using pytorch==2.0.1+cu117, Ill test the proposed solution you provided from the main branch and update the issue.


Force update of SSH key fingerprint at each connection

If a target is restarted after a new rootfs has been flashed, the SSH key fingerprint check fails and this abort the connection. Since a testing target is something we trust in general, we should find a suitable workaround for that annoying issue.
Since we are using directly SSH, but instead the pxssh provided wrapper, this could be a bit more tricky.
One possible solution is to always remove the current key for the host:
ssh-keygen -R <HOST_IP>

and than send back a "yes" to accept the offered key at the prompt generated for the first connection.
Other approaches are resumed here:
http://linuxcommando.blogspot.co.uk/2008/10/how-to-disable-ssh-host-key-checking.html

This should now be possible using the new implementation in https://github.com/ARM-software/devlib/pull/450.
If this is not the case please re-open the issue.


Is there an online demo?

Online demo can be found in the same company's [UI Kit](https://cloudflare.github.io/cf-ui/#cf-component-modal.
Unfortunately, you'll have to run the examples locally yourself.


Dialog causing unexpected DOM mutation issues

Problem Description
So I'm making this App via Firebase hosting, and use the Dialog component extensively throughout the app. However, the inclusion of a Dialog (even just one)  throws the following error:

As for the JS code for the Dialog implementation, it's something along these lines (excluded the non-relevant components):
<AppCanvas>
  <Dialog
    title="Feedback Form"
    autoDetectWindowHeight={true}
    open={this.state.openDialogFeedback}>
      <FeedbackPop userInfo={this.state.userInfo} onClose={this.onFeedbackCancel} />
  </Dialog>
</AppCanvas>

For the stack, it is these:

I've tried debugging and got this far (as to identify what was causing the issue), but lack the expertise/knowledge to be able to tell how to fix this (relatively new developer)... I also think this would be a problem many people face if making apps out of this.
Thanks a lot in advance for your help, and please let me know if you require more information! (Just didn't want to flood with irrelevant info)
Versions

Material-UI: 0.15.0-alpha.2, 0.15.0-alpha.1, 0.14.0 (haven't tried any earlier)
React: 0.14.8
Browser: Edge, Firefox, Chrome

@nickbae91 I think gitter would a better place to ask questions regarding MUI. Check this out: https://gitter.im/callemall/material-ui


Stopping and starting an application causes the app to not load for a long time

Bug description
Currrently if the user stops and starts back up one of the demo apps when he navigates the to app UI the app will fail to load, even though the app shows as running in the AI Lab list, this situation lasts for about 5 mins until the app heals itself.
https://github.com/user-attachments/assets/3184375a-7277-4325-a481-4b3d6653cd13
Operating system
macOs
Installation Method
from ghcr.io/containers/podman-desktop-extension-ai-lab container image
Version
next (development version)
Steps to reproduce

Start up a fresh instance of one of the demo apps.
Stop the app.
Once it's stopped start it back up again.
Once it reaches RUNNING state open the app window and try to use it.
Notice that most of the UI will fail to load.
Close the app window and wait 5 mins.
Open the app window again and see that now it works.

Relevant log output
No response
Additional context
No response

Hi @cbr7 I'm not able to reproduce the problem.
I have tested on Mac, with two versions AI Lab 1.3.0 and with the main branch, and with Podman Desktop main branch.
If you still can reproduce it, could you please share the exact versions of Podman Desktop and AI Lab, and the information of the underlying container (its state, its logs, etc)?

@feloy will try to reproduce after the release is finished

@feloy I've been able to reproduce it with the latest AI Lab release 1.3.2 using PD 1.14.1 release, for me it happens exactly as described in the initial report, after doing a START -> STOP -> START cycle, upon the second startup the app takes a very long time to initialize, to be clear in the end the app will work, you just have to wait for it an unspecified amount of time, currently I'm noticing around 2-3 minutes of wait time until the app is actually usable.
There is nothing inthe container/pod logs that jumped out at me when I looked.
https://github.com/user-attachments/assets/f3887e1e-527f-44fb-b2aa-8175559a04de

After more tests with @cbr7 (the problem appears on his machine but I cannot reproduce in mine), the problem comes from:

when the app restarts, the app tries to check if the model is available (by requesting the inference server on path /v1/models). The request sometimes takes 1 or 2 minutes to complete.
during this check, the app does not display in the page that it is waiting for the model to be available, which is confusing for the user.

https://github.com/user-attachments/assets/2ae3bf77-6c0d-4e14-893c-25531ec06e39

Not sure what can be the root cause of the problem, opening https://github.com/containers/ai-lab-recipes/issues/812 on AI Lab Recipes to have some feedback from app's developers


Con 3306 20230227 add command line args url apikey

Extends CVC to allow users to pass a Conservator URL and API key.
This will allow Conservator to call Conservator-CLI for an arbitrary user.
File Changes:
FLIR/conservator/cli/cvc.py

Added url, api key parameters to main, clone

FLIR/conservator/config.py
FLIR/conservator/connection.py

Cleaned up some pylint issues

FLIR/conservator/conservator.py

Added static from_config_dict function that will create a Conservator instance using a config created from a supplied dict

docs/usage/cvc_guide.rst

Added note about using cvc with URL, API key parameters

[JIRA Task]

@nbutler77 there was a merge conflict so I pushed a merge-from-main commit to resolve it


Create API calls

There should be some simple, documented REST web API calls that can be integrated into code and a Python module.
e.g.
pqr.pitt.edu/search?name=aspirin
returns a list of InChI keys matching the search
pqr.pitt.edu/api/inchikeys
returns a list of all InChI keys, allowing mirroring.
pqr.pitt.edu/mol2/... sends the appropriate mol2 file (i.e., from /static/..)
pqr.pitt.edu/json/... sends the appropriate json file
This can be expanded for future downloads (i.e., the mopac outputs)
pqr.pitt.edu/aux/...
pqr.pitt.edu/mopout/...

pqr.pitt.edu/data/ already returns JSON

Right, but we:

need more calls, including the ability to write python scripts for searches
get all inchi keys
need documentation on the calls..

Done with commit #32e3483

/api/json/{INCHIKEY}
/api/mol2/{INCHIKEY}
/api/inchikeys

Thanks. Great start. :+1:
We need to also add some sort of /api/search/ calls, e.g. by name, by tag, etc.


Update color tokens

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #63
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.

Update design token version and all other dependencies
Refactor color tokens by isolating the color tokens and creating tier 3 tokens for repeatable rules
Replace hardcoded size values with tokens

Type of change:
Please delete options that are not relevant.

[x] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

Verification

[ ] Background colors look correct visually
[ ] Background tier 2 token matches figma
[ ] All layout styles look correct visually

Figma: https://www.figma.com/design/tSkHw1uJfDHy0Ml0aDp7Io/branch/SWOHKZpz2K78PPL43uvp5z/Auro-color-palette-(Variables)?node-id=4230-13084&m=dev
By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 3.3.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


docs(react): 📝 add new jsx runtime documentation

Pull Request Checklist
🚨 Please review the guidelines for contributing to this repository.

[x] Appropriate label or labels have been added
[ ] ✅ Tests for the changes have been added (for type:fix / type:feat)
[x] 📝 Docs have been reviewed, added or updated if needed (for type:fix / type:feat)
[ ] 🏁 Issue number has been linked if this PR closes one

Description
Please describe your pull request.
Breaking Change?

[ ] 💥 Yes
[x] No

❤ Thank you for contributing!

Codecov Report

Merging #28 (7b1064c) into master (739b610) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #28   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           21        21
  Lines           41        41
=========================================
  Hits            41        41

Impacted Files
Coverage Δ

src/react/rules.ts
100.00% <ø> (ø)

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 739b610...7b1064c. Read the comment docs.


Magic Bow doesnt seem to be working for the past like 9 months

I've tried support multiple times but i still can't seem to fix it. I can't use magic bow while my spell inventory is open. It only works in gamemode creative for some reason too, which is super weird.

What version of the plugin, what version of spigot?
Any other plugins that deal with bows or may be interfering in some way?
I just double-checked on latest Paper and Magic and it seems fine for me.


Could not resolve all dependencies for configuration

A problem occurred configuring project ':app'.

Could not resolve all dependencies for configuration ':app:_debugApk'.
A problem occurred configuring project ':react-native-maps'.
Could not resolve all dependencies for configuration ':react-native-maps:_debugCompileCopy'.
Could not find androidx.appcompat:appcompat:1.0.2.
Required by:
myapp:react-native-maps:unspecified > com.facebook.react:react-native:0.61.1

This is resolved by https://github.com/react-native-community/react-native-maps/pull/3020 which isn't in a release yet.
My workaround is using patch-package with the following patch file: https://github.com/OurVoiceUSA/HelloVoter/blob/14857376632ab5dcb165e34e40f2b7ceb8906e3c/mobile/patches/react-native-maps%2B0.25.0.patch
Hope that helps.


Update package.json

Thank you for your contribution @brianjd!
Merging it right now, I'll release a version in a couple of hours :)


step-jira-number-from-branch 1.0.0

What to do if the build fails?
At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.
New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

New Step
Thank you for the new Step share! The CI check might will fail due to our extended validation engine. Nothing to worry about yet, we will get back to you shortly.

@denys-meloshyn when you are done with the changes please do a fresh re-share

@denys-meloshyn @trapacska It looks like there is another step pending review (https://github.com/bitrise-io/bitrise-steplib/pull/2570) which functionality is somehow superset of this one.

@koral-- should I close it in this case?

You have to decide yourself. If I were you I would try contacting @jimmithy and maybe join the forces.
It looks like your step has more inputs so "merged" steps would the best option from the users' perspective.


Content errases

When i save something in a file the content gets errased and also sometimes when i save something vscode makes a line break in the file.

/needsMoreInfo
Is it reproducible with all extensions disabled?
You can try this with F1 and >Developer: Reload Window With Extensions Disabled
Also where line break is placed? At the end of the file or in other place?


Find Matched Gifts Page Limited to 1000 Opportunities

If a customer has more than 1000 Opportunities returned on the Find Matched Gifts page (MTCH_FindGifts), the page is blocked from loading due to a limit on the number of records that can be looped over in an apex:repeat tag. The error displayed is:
Collection size {some number of 1000}  exceeds maximum size of 1,000.

Limit Information
More information about this Limit:
"Maximum collection items that can be iterated in an iteration component such as apex:pageBlockTable and apex:repeat": 1000
The Limit jumps to 10,000 and you can load over 1000 Opportunities if you add a readOnly=true parameter to the apex:page tag. However, you are then blocked from updating Opportunities. [I verified that this is not a valid fix the way the page is currently written].
Root Cause
The problematic query is in MTCH_FindGifts_CTRL, lines 99-123.
SELECT Id, Name, AccountId, Account.Name, Primary_Contact__c, Primary_Contact__r.Name, Amount, CloseDate,
Matching_Gift__c, Matching_Gift_Account__c, Matching_Gift_Account__r.Matching_Gift_Percent__c,
     (SELECT Id, ContactId FROM OpportunityContactRoles WHERE IsPrimary = true AND IsDeleted = false)
FROM Opportunity
WHERE ((Matching_Gift__c = null AND StageName IN :closedWonStages
     AND AccountId = '0011F00000fFLHeQAO'
     AND (Matching_Gift_Status__c = 'Potential' OR Matching_Gift_Status__c = 'Submitted'))
     OR (Matching_Gift__c = null AND StageName IN :closedWonStages AND Matching_Gift_Account__c = '001xxxxxxxxxxxxxxx'
     AND (Matching_Gift_Status__c = 'Potential' OR Matching_Gift_Status__c = 'Submitted'))OR Matching_Gift__c = '0061F00000ANMOiQAP') AND Id != '006xxxxxxxxxxxxxxx'
ORDER BY Name

Workarounds:
No current workarounds for loading the page. Customers with more than 1000 matching Opportunities will need to use a data loading tool to update the matched Opportunities in a different manner.
Steps to Repeat

Insert an Organization Account; copy the Account Id.
Insert 1010 Opportunities associated with that Account.
Add the Find Matched Gifts button to the Page Layout associated with your Opportunities.
Navigate to one of the newly created Opportunities and click the Find Matched Gifts button. The error will be displayed.
Delete 11 Opportunities.
Navigate to one of your Opportunities and click the Find Matched Gifts button. No error is displayed.

W-7790336

I was able to slightly get around this but temporarily clearing out the "Matching Gift Account" and "Matching Gift Status" fields using Dataloader for the account that had 1000+ opportunities. Additionally, when there are that many opps on the page, run into frequent apex cpu limit errors when trying to select a matching opp.

Please follow this Known Issue on the Trailblazer Community for updates.


Enforcer does not work for dict/list

@type_enforced.Enforcer
def foo(arg: List):
    pass

@type_enforced.Enforcer
def bar(arg: Dict):
    pass

foo([])
#bar({})

these result in
TypeError: (foo): Type mismatch for typed variable `arg`. Expected one of the following `[typing.List]` but got `<class 'list'>` instead.

and
TypeError: (bar): Type mismatch for typed variable `arg`. Expected one of the following `[typing.Dict]` but got `<class 'dict'>` instead.

That's not intended, right?

Thank you for the comment. With this module, you do not need to import from typing. For example, you can simply specify list.
EG:
>>> import type_enforced
>>> @type_enforced.Enforcer
... def foo(arg: list):
...     pass
...
>>> @type_enforced.Enforcer
... def bar(arg: dict):
...     pass
...
>>> foo([])
>>> bar({})
>>> foo({})
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/conmak/development/personal/type_enforced/type_enforced/enforcer.py", line 95, in __call__
    self.__check_type__(assigned_vars.get(key), value, key)
  File "/home/conmak/development/personal/type_enforced/type_enforced/enforcer.py", line 117, in __check_type__
    self.__exception__(
  File "/home/conmak/development/personal/type_enforced/type_enforced/enforcer.py", line 34, in __exception__
    raise TypeError(f"({self.__fn__.__qualname__}): {message}")
TypeError: (foo): Type mismatch for typed variable `arg`. Expected one of the following `[<class 'list'>]` but got `<class 'dict'>` instead.

The only time you need to import from typing is when you want to validate an uninitialized class instance. See here

@cmantas I am assuming that this addresses your issue. Feel free to reopen if this is not the case.


Fetch bug

I tried enabling the fetch path in the smoke test component here - https://github.com/fermyon/StarlingMonkey/blob/main/tests/smoke.js#L135.
But on the await textP line I get the following error:
stdout [1] :: Log: Error: TypeError: Response body object should not be disturbed or locked.

This is currently blocking enabling fetch on ComponentizeJS (https://github.com/bytecodealliance/ComponentizeJS/issues/96).

We have a fix for some issues with fetch here, if anyone needs it urgently:
https://github.com/bytecodealliance/StarlingMonkey/pull/49

Fixed by #52


Abstract recent partner data grid work

Abstracts partner tabs styles, and project & engagement columns to components/ for reuse.
Also synced the partner project's grid to have all the same settings as engagements.
Parts of this were pulled from #1551 and polished.

Good catch, just renamed!


Wrong date of modification for files

Describe the bug
Apologies if this is already reported.
The date of modification is not taking into account the local time set on Windows 10. I live in timezone which is UTC +2  (+1 with Summer Time). The date of modification shows 3 hours ago for files which I just modified
To Reproduce

Change the time zone to UTC +2
Modify a file
Check data of modification of that file in Files UWP

Expected behavior
The Files UWP should use the local time in Windows 10
Desktop (please complete the following information):

OS Version: Windows 10 1909 18363.657
App version: v<IP_ADDRESS>

fixed: #831

There is a potential fix for this issue in the fast ring, sign up here to get early access to updates.

O this is great to hear :D :D :D


oic: Parameter not check on public function

It's de-referencing some parameters without any check in a public function.
Signed-off-by: Flavio Ceolin<EMAIL_ADDRESS>

merged


cmd/compile: Incomplete refactoring after CL that introduced package types?

Looking at the truncfltlit function in gc/const.go I see code like this:
switch t.Etype {
	case types.TFLOAT64:
		d := fv.Float64()
		fv.SetFloat64(d)

	case TFLOAT32:
		d := fv.Float32()
		fv.SetFloat64(d)
}

The fact that we're using TFLOAT64 from types and TFLOAT32 from gc makes me suspect the refactoring done for CL 39855 (cmd/compile: factor out Pkg, Sym, and Type into package types) was somehow incomplete.
cc @griesemer

It does look incomplete. Fortunately, it doesn't matter too much. From the beginning of cmd/compile/internal/gc/type.go:
// convenience constants
const (
	Txxx = types.Txxx

	TINT8    = types.TINT8
	TUINT8   = types.TUINT8
	TINT16   = types.TINT16
	TUINT16  = types.TUINT16
	TINT32   = types.TINT32
	TUINT32  = types.TUINT32
	TINT64   = types.TINT64
	TUINT64  = types.TUINT64
	TINT     = types.TINT
	TUINT    = types.TUINT
	TUINTPTR = types.TUINTPTR

	TCOMPLEX64  = types.TCOMPLEX64
	TCOMPLEX128 = types.TCOMPLEX128

	TFLOAT32 = types.TFLOAT32
	TFLOAT64 = types.TFLOAT64

Indeed - I can't even see the difference anymore... . The factoring is in progress, so feel free to clean up as you see fit.

Thanks. To be sure: "clean up" means switch to the types. constants when possible (for example, always when writing new code); and the long term goal is to delete the dups @josharian pointed out exist in gc?

I started switching to the types constants but everything became incredibly verbose. So for now I'd say use the local constants. The factoring as it is clearly doesn't improve the code, but it makes it more explicit what kind of dependencies exist. Ideally (long run) we'd like to use a more direct representation of each type (one struct per type, rather than a Type struct with extension), and make Type an interface (which is already used in some form by the ssa backend).

Thanks for the explanation.
I'm going to close this issue since a) I fixed the specific inconsistency in truncfltlit in another CL b) it's now clear that the goal of your CL was not to switch everything to the types. constants.


Fix missing mouse_button_down_event in event handler

I noticed no mouse down events getting through to my game, and it turned out the mouse_button_down_event was missing in the miniquad::EventHandlerFree implementation in lib.rs.
This pull request fixes this problem by introducing the missing mouse_button_down_event.

Nice, thanks!


Delay the Masonry redraw by one second

This PR addresses issue #264 raised by @janeatcc (and its duplicate #343 raised by @ahmedkrmn) regarding images overlapping when the option to prevent overscaling is toggled on and off.
The cause of the bug and a rudimentary solution were known for some time but the search for a more elegant solution was unfruitful after nearly 4 weeks. This is an implementation of the same solution.

Developer Certificate of Origin Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors. 1 Letterman Drive Suite D4700 San Francisco, CA, 94129
Everyone is permitted to copy and distribute verbatim copies of this license document, but changing it is not allowed.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or
(b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or
(c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it.
(d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved.

@brenoferreira I've made the changes you suggested.


Switches to latest versions which simplifies steps

Special thanks to @dsyer for https://github.com/spring-projects/spring-boot/pull/8486

cc @llinder @devinsba


provide guidance on implementing the shell snippet

This is to help newer shell users avoid confusion. See #2603

Thanks for the suggestion, but I think it's stating the obvious. If we are to do this, we'll have to repeat the comment in every example in the page.

Thanks for the suggestion, but I think it's stating the obvious. If we are to do this, we'll have to repeat the comment in every example in the page.

Hi. I am very confused about the error, and don't understand what is obvious. How should I fix this problem? I tried adding this to my .zshrc and still getting the same error:
fzf_default_command() {
INITIAL_QUERY=""
RG_PREFIX="rg --column --line-number --no-heading --color=always --smart-case "
export FZF_DEFAULT_COMMAND="$RG_PREFIX '$INITIAL_QUERY'" \
	fzf --bind "change:reload:$RG_PREFIX {q} || true" \
	--ansi --disabled --query "$INITIAL_QUERY" \
	--height=50% --layout=reverse
}
fzf_default_command

@sdondley :arrow_up_small:


[FEATURE]: Define a convention for quotes

Summary
I think it can be great to define a convention for uses of simple and double quotes.
Motivation
Define a convention will uniformize more deeply our code.
Implementation
ESLint embed rules for this.
Checks

[X] I am running the latest version
[X] I checked the documentation and found no answer
[X] I checked to make sure that this issue has not already been filed
[X] I'm reporting the issue to the correct repository

Already handled by prettier. A general note, ESLint is used here as a linter not a formatter, thus we won't enable any formatting rule of ESLint.


Deletion bypasses Windows recycle bin

I've just had to restore a file from my backup copy because I chose the Delete item from the context menu in the Favorite panel (naively believing that it would only remove it from the favourites list).
When I remove local files anywhere else on Visual Studio Code files are sent to recycle bin. It'd be nice that the addon would do the same (as long as the extension API allows it). I'm using Visual Studio Code x64 on Windows 10.

Delete is same as Delete from vscode file explorer.
You have confirmation question Are you sure you want to delete 'filename.ext' popup to confirm.
If you confirm file gets deleted.
Use git so you can discard changes (vscode has built-in support for it).
If you need private repositories use gitlab


Fix footer links hover focus state

I think I unintentionally introduced this when changing the link underline behaviour to only be present on certain footer links...
When a link is focused and a pointer is hovering of it, it should not show an underline.
Before

After

I'm not adding another changelog entry since we already have one for adding the new focus styles: https://github.com/alphagov/govuk-frontend/pull/1321


Fatal Error of 1.15 on Windows 10

In the last couple weeks, we have started to see printers failing to print jobs submitted to our Windows 10 device running Cloud Print Connector with the most recent version.  The W10 device is fully patched.  In troubleshooting, I opened up CMD and started the service using command line string.  Upon the service starting via CMD, a test print job was sent.  The following error message appeared within the CMD window upon receiving the submitted print job.
FATAL Local printing has not been implemented in this version of the Windows connector.
What do I need to adjust so we can resume printing via Cloud Print Connector?

When running the connector outside the service, please make sure you are running the exact command that the service is running. It will be the executable but also a --config-filename with the path of your config file (probably something like c:\ProgramData/Google/...).
You can also try pulling the event logs, see:
https://github.com/google/cloud-print-connector/wiki/Windows-Event-Log-and-Troubleshooting


Improve reboot role

Since https://github.com/ansible/ansible/issues/20493 is present we can't call the reboot role from within another role in a handler.
To overcome this we're going to write the super short reboot login right in the kubic role. In the future move some of Kubic's tasks into bootstrap_os and make the other reboot login dependant on playbook.
This is not as clean as intendended but seems the best avenue to take.

At the moment Kubermetes is no longer provisioned with ansible


Fix ρ_q_tot and ρ_θ for surface conditions

Small MSE-changing PR. Changes how ρ_q_tot and ρ_θ are calculated for the surface conditions to match the new EDMF. This requires passing the thermo state in via the State struct.
I still need to format, but the MSE changes are only small so it should be good to go.

@charleskawczynski surf_params will be totally removed in the surface unification PR, this is meant to be a small bug-fix/MSE changing PR so that the next one doesn't have any regression.

bors r+


Copy edits for Numerical Linear Algebra

@arnavs  Review carefully, merge, and then go through the generated HTML/etc. to make sure there were no regressions in the display/etc.


added chicken tikka masala

Great recipe. Could you convert min to an hour or vice versa for all of the units, I think a standardized time unit is better.

Varying denominations of time is the standard, can't think of a good reason to write 240 minutes instead of 4 hours.
Good recipe though, should merge PR.

Great recipe. Could you convert min to an hour or vice versa for all of the units, I think a standardized time unit is better.

Varying denominations of time is the standard, can't think of a good reason to write 240 minutes instead of 4 hours.
Good recipe though, should merge PR.

That's what I thought as well. I feel like most people would prefer 30 minutes to .5 hours etc.

Great recipe. Could you convert min to an hour or vice versa for all of the units, I think a standardized time unit is better.

Varying denominations of time is the standard, can't think of a good reason to write 240 minutes instead of 4 hours.
Good recipe though, should merge PR.

Actually I just realized that it shouldn't give a total cooking time of 4.25 hours...that is a bit weird. It should be just 4 hours cooking. I'll make the change.


Question: Can I hook into the watch mode - and listen on re-transpile event?

Hi,
I looking for a way to run tstl --watch, and to be able to trigger an action (notify another app that the bundle have changed) each time the code is re-transpiled (after the target bundle is updated).
Is there an API for this?
(I guess I can write an external watch for the target bundle)

Currently we do not have support for this, although there are plans to add an afterEmit plugin hook. For the currently supported hooks, see: https://typescripttolua.github.io/docs/api/plugins

tsc-watch works. You can specify a custom compiler there like typescript-to-lua/dist/tstl and an onSuccess script

This should be supported nowadays by our current plugin system


Updated MAINTAINERS.md format.

Coming from https://github.com/opensearch-project/.github/issues/121, updated MAINTAINERS.md to match opensearch-project recommended format.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@c18efda). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage        ?   62.76%
=======================================
  Files           ?       10
  Lines           ?      658
  Branches        ?      119
=======================================
  Hits            ?      413
  Misses          ?      192
  Partials        ?       53

Flag
Coverage Δ

dashboards-query-workbench
62.76% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Looks like some CI has failed. This change has no code changes, can someone please take a look at that?


resume jobs is not working

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument

Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
azcopy jobs resume 3e993a7d-29cb-914f-490e-72ef49cf32c2 --source-sas="U:*" --destination-sas="xxxxxxxxxxxxxxxxxxxxxxxxxx"
Job 3e993a7d-29cb-914f-490e-72ef49cf32c2 has started
100.0 %, 411636 Done, 295 Failed, 0 Pending, 0 Skipped, 411931 Total,
Job 3e993a7d-29cb-914f-490e-72ef49cf32c2 summary
Elapsed Time (Minutes): 0.1013
Total Number Of Transfers: 411931
Number of Transfers Completed: 411636
Number of Transfers Failed: 295
Number of Transfers Skipped: 0
TotalBytesTransferred:<PHONE_NUMBER>486
Final Job Status: CompletedWithErrors
What problem was encountered?
Resume is not transferring the failed files its showing the same failed ones
How can we reproduce the problem in the simplest way?
Have you found a mitigation/solution?
No

Hi,
After closely looking failed file names these file names has chinese name, will it affect the migration if yes then why?
3e993a7d-29cb-914f-490e-72ef49cf32c2Copy.log:6761920:2019/12/16 11:01:32 ERR: [P#19-T#9850] UPLOADFAILED: %5C%5C?\U:\Gxxxxxxxxxxxx\2. GIS\Indus\xxxxxxxxxxx\fras_aux_e90_kutch_mask.␍簴ꊨ簷&U:* : 000 : Couldn't open source-open
%5C%5C?\U:\xxxxxxxxxxxxxxxxxxxxxxxx\fras_aux_e90_kutch_mask.␍簴ꊨ簷&U:*: The filename, directory name, or volume label syntax is
incorrect.

I noticed this in the command you were running: --source-sas="U:*"
Could you try running without this parameter? It looks like AzCopy has accidentally appended a SAS token to a local source, judging by the &U:* on the end of your file open failure.

This'll hopefully serve as a mitigation to your issue, but it's still technically a usability bug to tackle.

What is the status on this @adrees-msft @JohnRusk

We are waiting to hear your answer to the question Adele asked you, above.


Support non-standard but common keys

From my own investigations, this includes publisher, publisher-url, and title

Actually, I'm going to hold off on these until someone requests them, since I have no idea what they're for.


Listener

How do I listen all event by activity() ?
thank you for great library btw.

Here's one way to go about it. Use your own model and fire you own event when a model gets created:
// in your custom activity model

public static function boot() {
   static::created(function(YourCustomModel $model) {
       event(new YourEventClass($model);
   });
}

im looking for something like this

$events->listen(
    'Spatie\Laravel-activitylog',
    'App\Listeners\ActivityEventSubscriber@onLog'
);


Cannot run pub dataset experiments.

A screenshot of the error occurred is attached herewith.

Hi, apologies for the long delay in posting this.
In case it's still relevant, you can download the following files and store them in data/CUB_200_2011/:

Train list
Test list


[BUG] Slow Tokenizer Message is printing when the Fast Tokenizer may be in use

Issue description:
For Llava 1.5 13b, if you run it with the --tokenizer_mode "auto" flag set, it still prints a message that the slow tokenizer is being used. Llava has an image processor and a text tokenizer. It is not possible (to my knowledge) to set a fast image processor, but you can set a fast image tokenizer from the sentence piece library. Setting the flag above should create that fast tokenizer. However the code still prints this message Using a slow tokenizer. This might cause a significant slowdown. Consider using a fast tokenizer instead. If you look at the code  here since the LlavaTokenizer's text tokenizer is a field, it should actually be:
if not isinstance(tokenizer, PreTrainedTokenizerFast) or (isinstance(tokenizer, LlavaTokenizer) and not isinstance(tokenizer.tokenizer, PreTrainedTokenizerFast)):
Steps to reproduce:
Please list the steps to reproduce the issue, such as:
subprocess.Popen([
        "python", "-m", "lightllm.server.api_server",
        "--host", "<IP_ADDRESS>",
        "--port", "8080",
        "--tp", "1",
        "--max_total_token_num", "20000",
        "--trust_remote_code",
        "--enable_multimodal",
        "--cache_capacity", "1000",
        "--model_dir", MODEL_DIR,
        "--tokenizer_mode", "auto",
        "--max_req_total_len", "6000"])

Expected behavior:
No message should print. OR a message should print showing which fast tokenizer is in use.
Error logging:
INFO 05-07 21:41:31 [tokenizer.py:76] Using a slow tokenizer. This might cause a significant slowdown. Consider using a fast tokenizer instead.

@david-vectorflow thanks， we will fix this。


Updated Readme file with contribution rules

@Hristijan95 and @thesanjeevsharma  - I had a thought after seeing this. Should we have these steps in a separate CONTRIBUTING.md file since that also increases our repo's community profile ?

I was thinking the same. Let's merge this and raise an issue for that.

Yep, we can do that. I raised this PR to get things going. Thanks, @jamessspanggg for fixing all that. :)


Fixed #32962 -- Consolidated string concatenations in BaseDatabaseSchemaEditor.column_sql().

https://code.djangoproject.com/ticket/32962

Thanks, @felixxm! I also went ahead and cleaned the comments and docstring in the method.

@cjerdonek Thanks :+1: Looks great :1st_place_medal:

buildbot, test on oracle.


Feature request/question: and 'ends with' option to chant text search page?

Not sure if this has already been discussed or not, but happy to talk about it if not! I'm curious if it would be possible to add an option for searching for chant texts- currently we have 'contains' and 'begins with'. Possible to add 'ends with'? \
Granted, this is maybe a niche use case for fragments, or a poor digitization, but just a thought...

this is something you can do for chant texts on Cantus Index...(I think we had the same list of search functions on CantusDB at one time and decided to only have it one place? but maybe it wasn't really a conscious decision. Or I'm misremembering.)

It's a very easy thing to add technically, so we can implement whatever the decision is.

Might be nice to have it on the cantusdb end without having to go to Cantus Index to make those searches, is my thinking 😁

Yes!


Query Regarding Permissions

Hi Team, I had some queries, I hope this is the right place to put them up.

How the data(that we are permitting during the installation via look book) is being used?
Can we revoke the permissions from the azure enterprise application after the development is complete. What impact will it do to my existing LP?
If we do manual setup, do we still get all the content updates automatically?

I will follow up with staff and see if any other information is available about what is done with the telemetry data.
After the solution is deployed, the specific security principle used to install the Learning Pathways solution can be revoked. It will not change existing Learning Pathways setup.
Manual setup does not provision the same content as the automated installation; it will provision the solution alone. I would suggest doing a Test install in another tenant to see the difference between the full solution and the Manual solution.

I don't know what "data" you're referring to; can you be more specific.

Answered

Manual provisioning only means that you will not have the provisioned template pages provisioned, but that doesn't affect getting content updates.

Thanks @jellis-thrive and @juliemturner .
@juliemturner,
in the first point, I wanted to understand, about the data that is being accessed via permissions that we give during the setup of learning pathways.

You mean via the lookbook.microsoft.com site... there is no data accessed, it just needs permissions of a very high level to access, install, and deploy the customlearning.sppkg file to the tenant app catalog.  That said we do not write, maintain, or support the provisioning service it is, to us, only a service that can provisioning a custom site template with a solution in this manner.  You can get more information and ask more questions by reaching out to that team in the GitHub repository for the provisioning service: SharePoint/sp-provisioning-service GitHub Issues List.

Get it!. Thanks @juliemturner 😊


[TestProxy] Common Sanitizers

Add suggestions either in discussion or directly onto the branch.
Before we get too crazy we need to make certain we understand the differences in sanitization during playback. Recall that during playback, any requests to the proxy are sanitized prior to matching against the recording. It is very possible that we break playback tests just because recordings don't match up anymore.
Something to be aware of, and something we'll definitely be aware of when rolling out any versions of test-proxy taking these new defaults.
Currently the proxy live-test pipeline currently integration tests against the recorded tests for js test-utils, .net core, python tables, and java core if you need an easy check.

Does the proxy already have a way to deduplicate sanitizers so we don't run all of these twice? If not, I imagine it could compare two sanitizers with hashes of their string parameters

Does the proxy already have a way to deduplicate sanitizers so we don't run all of these twice? If not, I imagine it could compare two sanitizers with hashes of their string parameters

It does not at this time.

Thank you to all the reviewers on this PR: @pvaneck , @JoshLove-msft , @chlowell, @samvaity . Hugely appreciate all your feedback.
go, python, .net, and cpp have shifted onto a proxy version produced by this PR. Only java and js remain to migrate, and they are almost there. Given all that. I'm going to merge this PR, as I have further PRs to assist debugging that should include these changes.

/check-enforcer evaluate


Reject Cause Information

Hi
Is there a way to read the reject cause if the network registration is not successful?
The only option I've seen is to read the information from the URC +CREG, but ubxlib seems to skip this information.

Hi @eeFLis; not at the moment but you're right that there should be, I'll look into it.

Had a quick look and the reason why the reject cause is not passed back is because we set the +CEREG type to be 4; the reject cause is only returned if the +CEREG type is 3 or 5.  We need at least 4, since 3 doesn't return the power-saving parameters, but the problem with 5 (which would return everything) is that we then can't tell the difference between a +CEREG URC and a +CEREG response (see detailed explanation for why) since 5 represents a "registered" state (registered, roaming).
At the moment I can't see a way around this but I will continue thinking...

I see. Without taking a deeper look, can't a distinction be made based on the number of parameters returned?

That might be a possibility, let me see if there's a way to do it.  Will require quite a log of re-jiggering/re-testing, so might be a while.

I've had a look at the logic to make a retrieval of the EMM reject cause work within all of the other dodges and workarounds we have to do for the way registration commands work across all supported module types and I can't make a solution fit into my head.
However, this is for SARA-R5 only and so find here:
https://github.com/u-blox/ubxlib/tree/preview_cell_reject_cause_rmea
...an alternative solution: in uCellNetGetLastEmmRejectCause() we temporarily switch to AT+CEREG=5, ignoring URCs while we do so, read the registration status and then switch back to AT+CEREG=4 and re-read the registration status in the normal way in case it changed while we were "away".
This seems to work, in that it doesn't break anything, however in my testing I have not been able to make SARA-R5 actually return an EMM reject cause.
Do you have a scenario in which there is an EMM reject cause which you could try this code out on?

Thank you I will try to force an reject cause.

It was also not possible for me to create a reject cause.
What's a bit strange is that with mode 4 the module returns +CEREG: 4,3 which means "registration denied".
when the reject cause is read and the module switches to mode 5 +CEREG: 5,0 is returned.

Hmph.  Let me find out how this is meant to work...

OK, turns out that the reject cause isn't captured when in +CEREG mode 4, which is why we don't see it.  However, there is an AT command that I hadn't come across, AT+CEER, which allows network cause values to be read.  I have updated the branch:
https://github.com/u-blox/ubxlib/tree/preview_cell_reject_cause_rmea
...to use this and have confirmed that it does return an error value when an incorrect APN is used, resulting in the network denying service; it is not a very specific error code in this case (19, "ESM failure") but it is, at least, an error code.
Could you give this a try, see if it works for you?

hmm. in our test case it doesn't seem to work.
We get:
+CREG: 2,3

OK
-1: Deny

AT+CEER

+CEER: "EMM cause",0,"No cause information available"

we expect an error " PLMN not allowed".
But with your test case "incorrect APN" we also get the error code (19, "ESM failure").

I wonder if this is something to do with it being a circuit-switched (AT+CREG) reject?  Let me go ask...

Sorry I have cut off too much of the debug output. I have edited the message.

Thanks: can you indicate which network this is on, in case it is relevant?

IIRC "PLMN not allowed" is not a "final" error message, in that, assuming you are running in automatic selection mode (AT+COPS=0) [please confirm], the module should put the PLMN in the forbidden list and continue looking for other PLMNs it can roam onto.  But I might be wrong about this, so I will check.

In our test case we use manual registration mode on network 22802 to force the error
AT+COPS=1,2,"22802"
Since we use the manual registration mode I think that the forbidden PLMN list should not play a role.
But since you mention this I have another question: is there a way to delete the forbidden PLMN list in the ubxlib?
we use the following firmware version:
03.30,A00.01

Since we use the manual registration mode I think that the forbidden PLMN list should not play a role.

Ah, understood, yes, you're correct.

is there a way to delete the forbidden PLMN list in the ubxlib?

No, we haven't added any SIM functionality at all so far, not something anyone had asked for.  Are there other SIM-related things you might need, or just that?  We could certainly start with that.

A way to delete the forbidden PLMN list would be very useful for us.

A way to delete the forbidden PLMN list would be very useful for us.

OK, will add that.

after further testing, i realized that we used an wrong mnc "22812" instead of "22802".
So we tried to register on the wrong network.
This was not noticed because the +CME ERROR returned by the module seems to be ignored in registerNetwork().
The deviceErrorDetected is set to true but the "Wait for registration" loop is still called and only returned after the timeout.
Shouldn't the process be aborted immediately after the ERROR?

Interesting: from the comments in the code (this was written a long time ago), my intention was that the registration outcome should be driven by the URCs, so the bit where the manual registration is carried out, if it returned an error, would result in a URC that echoes that error and hence the "wait for registration" loop below that would exit pretty much immediately, when it saw that error.
Can you paste in here a bit more of the AT sequence so that I can see the pattern?

Here is the AT sequence starting with manual registration:
AT+CGDCONT=1,"IP","shared.m2m.ch"

OK
AT+UAUTHREQ=1,0,"",""

OK
AT+CFUN=1

OK
U_CELL_NET: registering on 22812...
AT+COPS=1,2,"22812"

+CME ERROR: not found

+CREG: 3

+CEREG: 3
AT

OK
AT+CREG?

+CREG: 2,3

OK
-1: Deny
AT+CGREG?

+CGREG: 2,4

OK
-1: OoC
AT+CEREG?

+CEREG: 4,3

OK
-1: Deny
AT+CREG?

+CREG: 2,3

OK
-1: Deny
AT+CGREG?

and after the timeout:
AT+CEREG?

+CEREG: 4,3

OK
-1: Deny
AT+CREG?

+CREG: 2,3

OK
-1: Deny
AT+CGREG?

+CGREG: 2,4

OK
-1: OoC
U_CELL_NET: unable to register with the network, is APN "shared.m2m.ch" correct and is an antenna connected?
AT+CEER

+CEER: "No report available",0,"No cause information available"

OK
AT+CFUN=4

OK

+CREG: 0

+CEREG: 0

+CGREG: 0
U_CELL_NET: connection attempt stopped after 240 second(s).

Hmm, OK, we have to be quite careful with this particular bit of code.
I recall that when I originally wrote it I had the "wait for registration" loop exit if a "deny" came back, since that seemed pretty final, but there were situations, possibly only with automatic network selection, where "deny" would be a transient state and giving up registration at that point would be the wrong thing to do.
So we could exit the "wait for registration" loop if "deny" came back and we were in manual selection, but that is a more complicated solution than simply, as you say, setting errorCode to, say, U_ERROR_COMMON_NOT_FOUND if deviceErrorDetected is true.
For the purposes of this experiment, would you be able to try hacking:
https://github.com/u-blox/ubxlib/blob/cd03192c93ea7fa4edd50168965d24160040f71f/cell/src/u_cell_net.c#L1129
...to be:
            deviceErrorDetected = (deviceError.type != U_AT_CLIENT_DEVICE_ERROR_TYPE_NO_ERROR);
            if (deviceErrorDetected) {
                errorCode = (int32_t) U_ERROR_COMMON_NOT_FOUND;
            }
            uAtClientClearError(atHandle);

...and see if that (a) causes an exit of the the wrong manual network is applied and (b) gets us the desired AT+CEER if the correct manual network is applied?

with the modification
a) causes an exit if the wrong wrong manual network is applied. (manual selection mode)
b) causes an exit and return "EMM cause",15,"No suitable cells in tracking area" if the correct wrong manual network is applied (manual selection mode)
c) do not cause an exit and return "EMM cause",19,"ESM failure" if an incorrect APN is used. (automatic selection mode)

That's good, I think, except that the "correct wrong manual network" (which I guess is MCC/MNC 22802?) is returning "EMM cause",15,"No suitable cells in tracking area" whereas you were expecting it to return "PLMN not allowed" (11)?  How sure are you of that EMM cause?

Yes "correct wrong manual network" is 22802. The EMM cause "PLMN not allowed" was only an assumption.
For me it's fine like this.
Thank you for your great support

Phew, that's good.  The changes discussed here have now passed internal testing, need to get them reviewed and they should land here soonish, will update this issue when done.

@RobMeades
I don't understand the commit https://github.com/u-blox/ubxlib/commit/2018c62239763bafae0e7c35f44be729fabf8550
This has the opposite effect: Active time and periodic TAU were reported correctly before and no longer after the change.

reject type and cause seems to be present but empty +CEREG: 1,"025e","01083403",7,,,"00001000","00111111"

Gah, you're right, I'd misread the AT manual, where it says:

...I'd read it as the reject type and cause not being present for +CEREG type 4, but of course  the two commas are present.  Silly me, will revert.

Commit 2018c62 now reverted by commit https://github.com/u-blox/ubxlib/commit/140134b52047bd8d50d63e615cc1a1c13a8390fa.

Thanks.
Another question about uCellSimFplmnListDelete().
Sometimes reading the length of the FPLMN field on the SIM fails with the CME ERROR: SIM not inserted. (Don't know why as the module can successfully register on the network)
No error is returned either, as it appears that no distinction is made between the various CME ERRORs and U_CELL_CSIM_FPLMN_SIZE_BYTES_2G is returned in all cases.
AT+CRSM=176,28539,0,0,20

+CME ERROR: SIM not inserted
AT+CRSM=214,28539,0,0,12,"FFFFFFFFFFFFFFFFFFFFFFFF"

+CRSM: 144,0

ok. any idea why we get a CME ERROR: SIM not inserted ?

ok. any idea why we get a CME ERROR: SIM not inserted ?

No, it is quite baffling: could you maybe post a slightly longer AT sequence here, including the bit where AT+CIMI and registration succeeds, then I can point it at someone who knows the insides of SARA-R5?

I have moved the deletion of the fplmn lists to a different position in the registration sequence.
So far, the error no longer occur. I will post the AT output here as soon as the error reappears.


Disabling all metadata queries when executing queries in SQL Editor Execution for Greenplum database

Hi, Team,
I find that when executing queries in the SQL Editor, it takes additional time to complete compared with directly executing query,
and I observe that there are several metadata queries are executed along with the queries in the SQL Editor.
Sometimes it take additional 10~30 seconds when executing queries in a database with many objects, which is annoying for users.
And for some systems it is less likely have too many metadata queries to occupy database system resource.
For example, in the above Query Manager screenshot, I execute a query SELECT * FROM pg_class LIMIT 10 in a Greenplum 6.25.1 database (one of PostgreSQL database derivatives), DBeaver executes 9 additional queries.

In the above metadata queries, the below SQL takes 2 seconds to complete.
SELECT i.*,i.indkey as keys,c.relname,c.relnamespace,c.relam,c.reltablespace,tc.relname as tabrelname,dsc.description,pg_catalog.pg_get_expr(i.indpred, i.indrelid) as pred_expr,pg_catalog.pg_get_expr(i.indexprs, i.indrelid, true) as expr,pg_catalog.pg_relation_size(i.indexrelid) as index_rel_size,pg_catalog.pg_stat_get_numscans(i.indexrelid) as index_num_scans
FROM pg_catalog.pg_index i
INNER JOIN pg_catalog.pg_class c ON c.oid=i.indexrelid
INNER JOIN pg_catalog.pg_class tc ON tc.oid=i.indrelid
LEFT OUTER JOIN pg_catalog.pg_description dsc ON i.indexrelid=dsc.objoid
WHERE  i.indrelid=1259 ORDER BY c.relname

It would be great if such metadata queries can be disabled completely.
Best Regards.
Describe the solution you'd like
Provide some options to completely disable metadata query during SQL Editor Query Execution.
Describe alternatives you've considered
Caching these metadata required by SQL Editor when a connection start, and avoiding to repeat these metadata queries whenever executing SQL in the editor.

Hello @CC-Hsu
You can disable this behavior with these settings:

But without key info, it will be hard to edit data values in the result set.

It's been a while since no update here. If the issue is still actual please let me know and provide additional information.


Attempts to fix broken rosetta-cli CI step

Fixes broken CI steps around rosetta

This can be closed, fixed in coinbase/rosetta-cli#313


Stepper

Why not use stepper motor for fast and accurate movement  control

This is not an issue for the software development: feel free to take up this question on the forum


Please add hooks for user scripts into openroad tcl files

Description
I like OpenLane being a good reference flow for the use of OpenROAD etc.
But as OpenLane cannot fit all user purposes, sometimes files need individual changes that users may have to re-apply whenever new versions of OpenLane are available.
I would like to propose adding hooks into the OpenLane tcl scripts such that users do no longer need to locally patch OpenLane, but just need to provide their user scripts that OpenLane will source-in at pre-defined positions.
/cc @proppy
/cc @dhaentz1
Proposal
Please find attached example code that adds the following changes to the OpenROAD tcl files contained inside scripts/openroad:
An environment variable OPENROAD_USER_SCRIPTS points to a directory containing all user scripts. If the variable does not exist, or if the directory does not exist, no user scripts will be read in.
There are four classes of user scripts for the following phases:
setup: These files will be read during setup, right prior to reading-in the data base via read
pre: These files will be read-in right prior to the actual task of a script, i.e. right prior to detailed routing.
post: These scripts will be read-in after the actual task of a script, i.e. prior to reporting and saving results via write.
final: These scripts will be read-in at the very end.
Setup-scripts can be used to modify variables etc to change how data is read in.
Pre-scripts can be used to modify how the actual main task is performed.
Post-scripts can be used to modify or enhance the result of the main task.
Final-scripts can be used to modify the data written to disk or to add reporting etc.
For all four classes, there are two file names supported:

A generic one, i.e. user.setup.tcl, that will be executed in all steps, i.e. groute, droute, pdn etc
A specific one, i.e. user.pdn.setup.tcl, that will be executed only in a specific step, i.e. just pdn.
Import order is: non-specific script first, specific script afterwards

If user script are not available for a certain phase or step, execution continues w/o complaint.
The following code illustrates how user script hook could be added into the script droute.tcl:
source $::env(SCRIPTS_DIR)/openroad/common/io.tcl

execute_user_setup_hook "droute"

read

set_thread_count $::env(ROUTING_CORES)

set min_layer $::env(RT_MIN_LAYER)
if { [info exists ::env(DRT_MIN_LAYER)] } {
    set min_layer $::env(DRT_MIN_LAYER)
}

set max_layer $::env(RT_MAX_LAYER)
if { [info exists ::env(DRT_MAX_LAYER)] } {
    set max_layer $::env(DRT_MAX_LAYER)
}

read_guides $::env(CURRENT_GUIDE)

execute_user_pre_hook "droute"

detailed_route\
    -bottom_routing_layer $min_layer\
    -top_routing_layer $max_layer\
    -output_maze $::env(_tmp_drt_file_prefix)_maze.log\
    -output_drc $::env(_tmp_drt_rpt_prefix).drc\
    -droute_end_iter $::env(DRT_OPT_ITERS)\
    -or_seed 42\
    -verbose 1

execute_user_post_hook "droute"

write

execute_user_final_hook "droute"

The procedures execute_user_*_hook have been defined inside io.tcl.
openroad_scripts_with_user_hooks.zip

It's not a bad idea- but a couple notes:

The rewrite of OpenLane is already designed to let you subclass and replace step scripts entirely as follows: e.g. https://github.com/efabless/openlane2/blob/169378e8b2eba0fe5d177779107a4f16af0d6c1a/openlane/steps/openroad.py#L926
We'd have to commit to an API for these scripts. Currently, we enjoy the flexibility of being able to modify them with prejudice.

I'll bring this up with the team internally, though.