[com.google.fonts/check/name/match_familyname_fullfont] possible false positive with legacy name ID 1/2

Observed behaviour
Consider a family with four styles:

Regular
Bold
Condensed
Bold Condensed

Our goal is for Bold style linking to work (Regular -> Bold and Condensed -> Bold Condensed), even in a legacy Windows context, such as with contemporary PowerPoint.
We accomplished the goal, but FontBakery is not happy about how we did it.
The following table represents our setup:

name ID 1
name ID 2
name ID 4
name ID 16
name ID 17

Font-Regular.ttf
Font
Regular
Font Regular
Font
Regular

Font-Bold.ttf
Font
Bold
Font Bold
Font
Bold

Font-Condensed.ttf
Font Condensed
Regular
Font Condensed
Font
Condensed

Font-BoldCondensed.ttf
Font Condensed
Bold
Font Bold Condensed
Font
Bold Condensed

FontBakery complains about Font-BoldCondensed.ttf:

FAIL On the 'name' table, the full font name 'Font Bold Condensed' does not begin with the font family name 'Font Condensed' in platformID 3, encodingID 1, languageID 1033(0409), and nameID 1. [code: mismatch-font-names]

It's effectively saying name ID 1 should be a prefix of name ID 4.
The spec, however, says this about name ID 4:

Full font name that reflects all family and relevant subfamily descriptors. The full font name is generally a combination of name IDs 1 and 2, or of name IDs 16 and 17, or a similar human-readable variant.

In our case, it is combination of name IDs 16 and 17, but deliberately NOT of 1 and 2.
Expected behaviour
We believe the spec allows what we do, and that the FontBakery FAIL is a false positive.
Resources and exact process needed to replicate
We currently do not readily have a font available that we can share to reproduce this, but I will make one upon request.
Cc. @BlackFoundry, @SolennBor

Hey Just,
Which version of FB are you using? and could you please zip some test fonts (this makes my life easier ;-0).
Nice to hear that your setup works in MS Office. If I don't notice anything strange on other platforms, I'll update the checks.

Hi Marc, thanks!
Here is a subsetted anonimized version of the problem family.
fb-test-fonts-bf.zip
There are a couple of additional fails caused by subsetting, but it reproduces the com.google.fonts/check/name/match_familyname_fullfont fail nicely.

Perfect. I'll take a look next week.
Do you intend to publish this family on Google Fonts? If so, we may have set the stylenames so they're  . They're currently  . We care about axis ordering.

Perfect. I'll take a look next week.

Thanks!

Do you intend to publish this family on Google Fonts?

No.


Where do the grid lines come from?

When I use this module together with Susy I automatically get horizontal lines to guide / debug a vertical rhythm (revealed together with Susy's grid).  Could you point me to how that happens and where the code for it is?
If I include normalize.css instead, the lines disappear.  So it must be coming from here...  Unfortunately GitHub doesn't search forked repos yet.

normalize-scss doesn't make vertical rhythm grid lines.

@orlin: Have you imported normalize before or after you define the susy grids?
susy styles must come after the normalize styles - otherwise they could be overridden.

Thanks for the insight @strarsis! I don't use Susy, so I was hoping someone would provide a fix. :)


Error: Command failed: /usr/local/lib/node_modules/bs-platform/scripts/buildocaml.sh

following https://reasonml.github.io/guide/javascript/quickstart
mac os 10.12.6
npm 3.10.10
node 6.11.1
$ npm install -g bs-platform
/usr/local/bin/bsb -> /usr/local/lib/node_modules/bs-platform/bin/bsb
/usr/local/bin/bsc -> /usr/local/lib/node_modules/bs-platform/bin/bsc
/usr/local/bin/bsrefmt -> /usr/local/lib/node_modules/bs-platform/bin/bsrefmt

><EMAIL_ADDRESS>postinstall /usr/local/lib/node_modules/bs-platform
> node scripts/install.js

Working dir /usr/local/lib/node_modules/bs-platform
ninja binary is copied from pre-distribution
Working dir /usr/local/lib/node_modules/bs-platform/jscomp
/bin/sh: ocamlc.opt: command not found
System-installed OCaml compiler version not found
Building and installing local compiler
dyld: mach-o, but built for simulator (not macOS)
./configure: line 454:  7633 Abort trap: 6           sh ./runtest ansi.c
child_process.js:524
    throw err;
    ^

Error: Command failed: /usr/local/lib/node_modules/bs-platform/scripts/buildocaml.sh
dyld: mach-o, but built for simulator (not macOS)
./configure: line 454:  7633 Abort trap: 6           sh ./runtest ansi.c

cc @bobzhang

@jslz if you're still around, could you try on npm 5.4.2 please?

Ah npm 3. You need to be on 4


add information about @inishchith

Hello everyone, I am Nishchith Shetty ( inishchith ) and I am interested in the project: Support of Source Code Related Metrics

[X] I am adding my information to the list of interested students through this PR.
[X] I've added link to my Microtasks repository

I'll update this thread and the commit with link to the proposal, once i have it ready.
Looking forward to working with you guys!
Thanks

By the way, good job on the sign-off :)

You may link to a google doc that is not finished yet and complete it before the end of the application period.

Yeah. Sure, I'll add them in some time ;)

Is it necessary to send a pull request to governance with my information before the start of the student application process? As I have n't finished my proposal yet, can I wait until my proposal is ready and send everything all at once (during the student application process)?
Though the answer might be obvious, better safe than sorry!
@GeorgLink

Hi @Polaris000,
Good question and I'd be happy to answer it. However, it does not relate to this specific pull request and would better be asked and answered as a separate issue or - even better - on the mailing list. Can you please ask the question in a separate issue or on the mailing list?
Thanks,
Georg

@GeorgLink I've updated the link to my proposal, I'll make sure it gets completed in some time.
Thanks.

@GeorgLink Please do review this. I've added the link to my proposal.
Thanks :)


Add Closest Dessert Cost in python

Related Issue
Closes: #688 issue number that will be closed through this PR
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
I have added the implementation of Closest Desert COst Problem
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
NA
Language used for coding
Mention the language you have used for coding here.
Python
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have documented the code.
[x] New and existing unit tests pass locally with my changes.

@muthuannamalai12 Please add some comments to your code so that beginners can understand it easily.

@muthuannamalai12 Add comments to your code in all your PR.

@Nivedita967 Sorry for not responding For a long time. I had my exams Now I have updated it

@Nivedita967 Sorry for not responding For a long time. I had my exams Now I have updated it

No problem :)


Property hooks for PHP 8.4

Models currently use magic __get and __set methods to access the underlying WP_ class properties. In order to keep the code succinct, we don't transform the property name which means needing to write $user->user_email instead of $user->email.
We can utilize property hooks in PHP 8.4 to allow us to user either user_email or email, we can implement a first/last name getter and setter, and we could even implement validation/escaping logic to the getters and setters, E.g.
class User
{
  public string $email {
    get => $this->user_email;
    set => $this->user_email = $value;
  }
}

class Post
{
  public string $title {
    get => get_the_title($this->object);
    set => $this->post_title = $value;
  }
}

Currently we use traits such as HasTitle with a getter and setter method. I wouldn't recommend using the set property hook to update the database directly as this could fire a lot of updates, however the traits could implement property hooks instead of (or as well as) the methods, and then use the save method to persist the data to the database.

The fill method on models may need to be called on the Navigator model instead of the underlying object in order to use the property hooks correctly. This would allow us to do this:

$user->update([
  'email' => 'new.email@address',
]);

This would lead to a more consistent API where mass assignment and property assignment can use the same keys. It would also allow for the existing property names to be used:

$post = Post::create([
  'title' => 'Hello world',
]);

echo $post->title; // Hello world
echo $post->post_title; // Hello world

$post->title = 'New title';
$post->post_title = 'New title';

$post->save();

$post->update([
  'title' => 'Another title',
]);


Is there a way to retrieve amount of requests enqueued for sending via enqueue() method?

I would like to monitor total amount of requests which we enqueued for sending, is there a mechanism for this?

Use an EventListener https://square.github.io/okhttp/features/events/
Tally them yourself.

The Dispatcher also has a queuedCalls list that you can take the size of.

Thanks.


[AST Builder] Implement drop_table Expr

Issue
Resolve https://github.com/gluesql/gluesql/issues/693
Description
Implement drop table expr at ast_builder

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

42 of 42 (100.0%)  changed or added relevant lines in 2 files are covered.
30 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.08%) to 97.813%

Files with Coverage Reduction
New Missed Lines
%

core/src/executor/aggregate/mod.rs
11
92.2%

test-suite/src/tester.rs
19
88.89%

Totals

Change from base Build<PHONE_NUMBER>:
-0.08%

Covered Lines:
29119

Relevant Lines:
29770

💛  - Coveralls

It looks that we can choose only one rather than supporting multiple options - drop() | drop_table()
How about these names only? table("Foo").drop().build() table("Foo").drop_if_exists().build()

Change to Support one option drop_table() | drop_table_if_exists()


Connection Error with Default Server Installation

I have just installed the server using the automatic installation process. However, when attempting to connect to the server, the proxy console displays the following error:
[20:55:25 INFO]: [connected player] USER (/IP:55232) has connected
[20:55:25 INFO] [vane-velocity]: Connection 'USER ' is connecting to 'server'
[20:55:25 INFO] [vane-velocity]: Disconnecting 'USER ' because they don't have the permission to start server 'server'
[20:55:25 INFO]: [connected player] USER (/IP:55232) has disconnected: Server is offline

This issue occurs while using the default configurations.
Steps to Reproduce:

Install the server using the automatic installation process.
Attempt to connect to the server.

Configuration Details:

Configuration File: proxy/plugins/vane-velocity/config.toml
Setting: allow_anyone is set to true (as per default settings).
Plugins: No additional plugins have been added.

Expected Behavior:
Users should be able to connect to the server without encountering permission-related disconnections when allow_anyone is set to true.
Actual Behavior:
Users are disconnected with the message indicating they lack permission to start the server.
Additional Information:
No modifications have been made to the default configuration.

I'm sorry for the inconvenience, this is due to the in-progress update to 1.21 of some of the components. We had to switch from waterfall to velocity since waterfall is no longer supported, but the relevant setting to allow server starting is not yet released.
You can work around this by downloading a newer vane-velocity.jar file from here and overwrite the existing file in the ´proxy/plugins/` folder

Thanks for the quick reply!, this fixed it!


Support Java Named Modules

It is not important, MiniPlaceholders already supports jigsaw modules through automatic modules, adding full support through manual modules cannot be completed due to the multiple libraries that are yet to be implemented to support them


SIGSEGV with nightly version

Yesterday I ran a rustup update and cargo started crashing at an odd place, I don't remember when was the last time I ran rustup update but stable works properly.  I hadn't changed anything on my system before updating so I'm pretty sure it's something with cargo itself.
This is the stacktrace, which looks really odd:
#0  0x0000555555a7a055 in OPENSSL_ia32_cpuid ()
#1  0x00007ffff60695c3 in OPENSSL_cpuid_setup () from /usr/x86_64-pc-linux-gnu/lib/libcrypto.so.38
#2  0x00007ffff604bfb7 in _init () from /usr/x86_64-pc-linux-gnu/lib/libcrypto.so.38
#3  0x0000000000000001 in ?? ()
#4  0x00007ffff7de90ca in call_init.part () from /lib64/ld-linux-x86-64.so.2
#5  0x00007ffff7de921b in _dl_init () from /lib64/ld-linux-x86-64.so.2
#6  0x00007ffff7ddacda in _dl_start_user () from /lib64/ld-linux-x86-64.so.2
#7  0x0000000000000001 in ?? ()
#8  0x00007fffffffe2ec in ?? ()
#9  0x0000000000000000 in ?? ()

If it could matter, I'm running libressl instead of openssl.

Oh dear this looks quite surprising!
So it looks like you're using your system OpenSSL, which the bundled versions of Cargo aren't supposed to use. There's a current bug in Cargo though which pulls in the system libcurl accidentally, which can transitively include the system OpenSSL.
It may be worth retrying once https://github.com/rust-lang/rust/pull/40507 is merged.

(and the corresponding beta backport (https://github.com/rust-lang/rust/pull/40525))

Ok, that may be related to it (no idea, really, though).
Let's wait until the linkage gets fixed and then see if it's still here.

Not sure how to test this on my own so - does it work for you now ? The linked issues/PRs seem to be in already.

Closing due to inactivity and guessing that the fixes are in now, please reopen if I'm mistaken! thank you!!


fix: osmosis staking fiat deposit/withdraw errors

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

Fixes OSMO staking error as noted by TShifty here.
Risk

None
Testing

Engineering

Operations

From the Osmosis asset page, select a validator from the "Staking" list.
Delegate an arbitrary amount of OSMO, make sure the amount is denominated in fiat.
Ensure that the transaction broadcasts successfully.
Following the first two steps, ensure that withdraw transactions also broadcast successfully.

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #3844   👈

This comment was auto-generated by Graphite.


[HCAP-224] Refactor and transform participants data.

Update participates validation schema, processing and parsing.

Update participates validation schema, processing and parsing.

@fwpushan I removed the old lib and added a new one more optimized. The upload should work now.


Add Git repository URL

To add the link on Node-RED flow library, I added Git repository URL into the package.json file.
After adding the link, Node-RED users can see the code or report an issue on GitHub easily.

I'll close this because #17 is the same change.


Abililty to UPDATE LORAS/MODELS from UI

Feature Idea
I only see a way to update the metadata, but not the actual models.
Other
No response

what


PE-1788 chore: remove z-index property usage from stylesheets

This PR addresses SFCC feedback that asked us not to use the z-index property.
Using the positioning behavior of HTML elements, we can achieve the same effect of the z-index property by modifying the position stack context of certain elements.
This commit changes the position context of the asset grid items so that they can be statically positioned, allowing for the source select dropdown to, when absolutely positioned, render above the asset grid items.
This also made it necessary to change the way that filenames were faded out. Previously, this was done using pseudo-elements and position: absolute. Because of the new position context, this was no longer possible.
Instead, this commit makes use of  linear-gradient for a background-image. We can then clip the text against this gradient, filling the overlap with the color transparent.
The drawback to this approach is cross-browser compatibility is limited. But in the absolute worst case, some users will not see faded-out filenames but still see the modal element stacked above the asset grid items.

Current dependencies on/for this PR:

main

PR #170

PR #171  (this PR)

This comment was auto-generated by Graphite.

@frederickfogerty Looked into this some more. Outside of what I've tried here, I can't think of a way to avoid all the z-indexes currently in use. I kept running into overlap issues.
I think we either use the approach taken here or keep the z-indexes.


Styling of global page is impossible

I would suggest to add a

@NeOMakinG This is by design, Stripe doesn't allow you to do that on the web and neither do we!
You can always propose feature requests to stripe checkout team!

@A-Tokyo I was speaking about the loader global page, we can't reset CSS for example on the global html/body tag, because the only thing we can do is putting style inside the loader block :)

@NeOMakinG I added a htmlContentHead prop, but I think it might cause security issues so might remove later.
Why do you think this is crucial? You can't do it on the web. Why do you need to do it on Native?

@NeOMakinG Thanks for the input!
I added a htmlContentHead prop, but I think it might cause security issues so might remove later.
Why do you think this is crucial? You can't do it on the web. Why do you need to do it on Native? 🤔  Remember that this script can be used maliciously

Why would it cause security issue ? I don't find why.

You can't use script from a website to an iframe (Stripe content is inside an iframe right?)
It's for styling the global page properly, instead of adding negative margin inside your loader component, you are just able to kick default paddings, margins of html element inside the page!

It's not crucial as you've some workaround (but not bests), this is way cleaner don't you think?

I we don't want to give people hands on the initial loading page head, maybe we could at least make it customizable (setting background of the page, padding, flex-box properties...)

@NeOMakinG I get you perspective. I am not sure how it can be used maliciously but I am sure some dedicated hacker can find a way! :D
This library is one of the most sensitive libraries security wise, thus I try to avoid security issues by elimination instead of protection!
I will have a conversation with a few security folks and let you know if anything needs to be updated! So far so good!
Thanks a lot for the proposals! ( :

@NeOMakinG Re customization:
Nice thought! How we do it in our app:
const StripeCheckoutScreen = ({ navigation, route }: Props) => {
  const [isLoading, setIsLoading] = useState(true);
  const { stripeCheckoutComponentProps = {}, navigationTitle } =
    route?.params || {};

  /** set navigation options */
  useNavigationOptions(navigation, {
    title: navigationTitle || 'Stripe Checkout',
  });

  return (
    <View style={styles.flex}>
      {isLoading && <LoadingLayout />}
      <View style={[styles.flex, isLoading && styles.isLoading]}>
        <StripeCheckout
          {...stripeCheckoutComponentProps}
          stripePublicKey={config.STRIPE_PUBLIC_KEY}
          onLoadingComplete={(...args) => {
            setIsLoading(false);
            if (stripeCheckoutComponentProps?.onLoadingComplete) {
              stripeCheckoutComponentProps.onLoadingComplete(...args);
            }
          }}
        />
      </View>
    </View>
  );
};

Would this work for you?

You're welcome. Let's keep in touch if you've some discussion about security, but pretty sure nothing's bad there :)

The way you use your library looks like you are seeing the StripeCheckout component loader, after your own loader right ?

@NeOMakinG Exactly, this way I don't have to bother editing the HTML loading page since it never shows up anyway.

This is a nice approach tbh, but it had some complexity that maybe we could get rid of.
What about including this complexity inside your library directly? Where the htmlContentLoading prop would be a RN component that you can manipulate as you want

So, the stripe payment page would be shown only after the loader you pass as a prop is finished, but on the StripeCheckout component side

@NeOMakinG Re including this complexity inside your library directly It'd be too much work since I render html in the webview not RN.
But I can pass a loadingComponent prop that abstracts the logic I wrote above

But I can pass a loadingComponent prop that abstracts the logic I wrote above

This is what I was saying :)

Would you want to work on it @A-Tokyo or can I contribute on this feature?

@NeOMakinG Awesome job on the PR 💪 https://github.com/A-Tokyo/react-native-stripe-checkout-webview/pull/15 Once it's out of the WIP state I'll review 🚀

Hey @A-Tokyo, I'm so sorry it's not going further for the moment, I'm pretty busy with work stuff and other needs... If you want to go further and continue, do it, otherwise we'll have to wait :(

@NeOMakinG All good! (: Thanks for the awesome work!
It's not urgent, whoever has time first can jump in and finish it 💪


chore(nuxt): mock setInterval for Vue app during SSR

🔗 Linked issue

Resolves #25256
❓ Type of change

[x] 👌 Enhancement (improving an existing functionality like performance)

📚 Description

Throw an error if using setInterval in the Vue app during SSR to avoid memory leaks or getting the node process stuck during nuxt generate.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have added tests (if possible).

I love it! ❤

I love it! ❤

Was my fault 😂


PoS Mempool Struct

Current dependencies on/for this PR:

feature/proof-of-stake

PR #578

PR #563

PR #599

PR #600

PR #601

PR #602

PR #603   👈

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

feature/proof-of-stake

PR #578

PR #563

PR #599

PR #600

PR #601

PR #602

PR #603   👈

This comment was auto-generated by Graphite.


Update media title margin for large screens.

WHAT
Update media title margins for large screens.
WHY
To comply with large screen UI/UX guidelines.
HOW
Set margin between title and artist to 4dp for large screen and  2dp for small screen in MarqueeTextMediaDisplay, TextMediaDisplay.
Also fixed media title alignment in TextMediaDisplay, updated LoadingMediaDisplay, and added separate screenshot tests for large screen in LoadingMediaDisplayTest.
Checklist :clipboard:

[x] Add explicit visibility modifier and explicit return types for public declarations
[x] Run spotless check
[ ] Run tests
[x] Update metalava's signature text files

com.google.android.horologist.screensizes.MediaPlayerTest > screenshot[6] FAILED
    java.lang.AssertionError at Diffing.kt:154

...

com.google.android.horologist.media.ui.MediaPlayerDeviceScreenTest > mediaPlayerScreen FAILED
    java.lang.AssertionError at MediaPlayerDeviceScreenTest.kt:80

...

com.google.android.horologist.media.ui.MediaPlayerScreenTest > mediaPlayerScreen[6] FAILED
    java.lang.AssertionError at MediaPlayerScreenTest.kt:72

...

com.google.android.horologist.media.ui.PodcastPlayerScreenTest > mediaPlayerScreen[2] FAILED
    java.lang.AssertionError at PodcastPlayerScreenTest.kt:69

...

com.google.android.horologist.media.ui.components.MarqueeTextMediaDisplayTest > givenPaintable_thenImageIsDisplayed FAILED
    java.lang.AssertionError at MarqueeTextMediaDisplayTest.kt:54


Use contentOffset to set the starting scroll offset in ReactHorizontalScrollView

Implements contentOffset for horizontal scrollviews in Android [already implemented for ios]. This will allow specifying the position the ScrollView should start at.
A common use case is having a horizontal ListView with pagingEnabled, and having it open to a specified page, without having to wait for it to load and then call scrollTo.
Also see #6849
Usage:
Add contentOffset={{ x: someInitialXValue, y: someInitialYValue }} to your ScrollView props.
Example:

import React, { Component } from 'react';
import {
  AppRegistry,
  StyleSheet,
  Text,
  View,
  ListView,
  Dimensions,
} from 'react-native';

export default class offsettest extends Component {
  constructor() {
    super();
    const ds = new ListView.DataSource({ rowHasChanged: (r1, r2) => r1 !== r2 });
    const { width } = Dimensions.get('window');
    this.state = {
      dataSource: ds.cloneWithRows(['row 1', 'row 2', 'row 3', 'row 4', 'row 5']),
      initialPage: width * 3,
    };
  }

  render() {
    return (
      <View style={styles.container}>
        <Text style={styles.welcome}>
          Android Horizontal ListView ContentOffset Example
        </Text>
        <ListView
          dataSource={this.state.dataSource}
          renderRow={(rowData) => (
            <View style={{flex: 1, backgroundColor: 'red', width: Dimensions.get('window').width, justifyContent: 'center', alignItems: 'center'}}>
              <Text style={{color: 'white'}}>{rowData}</Text>
            </View>
          )}
          contentOffset={{ x: this.state.initialPage, y: 0 }}
          horizontal={true}
          pageSize={1}
          pagingEnabled={true}
        />
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: '#F5FCFF',
  },
  welcome: {
    fontSize: 16,
    textAlign: 'center',
    margin: 10,
  },
});

AppRegistry.registerComponent('offsettest', () => offsettest);

Tests seem to be up and running again, I've rebased to see if they'll pass this time.

To make it work on Android, do you mean I should use this method using ListView instead of ScrollView ? And is this also working for iOS ?

contentOffset should already be working on iOS, this PR brings it to Android too, for both ScrollView and ListView. [And maybe the new FlatList/VirtualizedList if they use the ReactHorizontalScrollView native component]

What about ReactScrollView as well as the horizontal one ?

Shouldn't take much to reproduce the same code there, my initial use case didn't include vertical so I didn't want to make a PR I wouldn't be testing, might do it over the weekend.

I see the commit is submitted at Feb 27, 2017, is it available on any release version now?

@BinLi-OW : This PR hasn't been merged in, so no, it isn't in any of the releases. You could apply the changes locally to have access to it and help test [if you do, please let me know of any bugs that pop up].

I have tested this a bit, and it appears to work. I have merged it into our product, it will be tested more throughout the week by myself and our QA team.

@kkjamie : That's awesome, please let me know if you spot any issues!

@kkjamie, would you please give tips how could I rebuild the react-native-XXX.jar in local storage?
As I thought, I make changes to the java files under node_modules, but it seems I should build out the jar first.

I actually just copied the file from this repo into my repo and renamed it, exposed it by a new Package. I don't want to be forking react-native and worrying about rebuilding, I know somebody at our company has done this. Checkout https://facebook.github.io/react-native/releases/next/docs/android-building-from-source.html

I've just given this a try as I encountered the same problem, and can confirm that the change works as expected.

@BinLi-OW I did it by forking, applying the changes of this commit, then building from source following the link @kkjamie posted above.
@chrisbianca : Thanks for the feedback.

This PR doesn't have a test plan, can you fix that?

@hramos : I'll rebase it on master in the next few days and take a look at handling vertical ScrollViews as well. Any specific test suggestions? Would committing examples be enough?

Hi!
I've tried getting the vertical scroll to behave the same with great success!
I did exactly the same as this patch does but in ReactHorizontalScrollView.java and ReactScrollView.java, dunno if you want to include it in your patch (would make more sense from a commit-perspective I guess) or if I should do this in a separate commit?
Also, a word of warning - we had an included react-native component (seems to be react-native-scrollable-tab-view) that submits a contentOffset with only an x parameter. I think it's this line: https://github.com/skv-headless/react-native-scrollable-tab-view/blob/master/index.js#L155)
This seems to be an illegal state if I understand the data-type PointPropType correctly, but I'm thinking it's a good thing to actually check that both properties are defined before trying to access them (our app crashed before I added validation that both properties are present).

@shergin Any chance to get this merged? Thanks.

@hramos @shergin It's very annoying not having contentOffset on Android. Is there any chance this PR could be reviewed? Thanks.

@todorone it's been closed for over a year. It'd have a better chance if someone opened a new PR and applied the feedback from this thread.


Push gem to RubyGems

Closed via https://github.com/Shopify/maintenance_tasks/pull/284

Closed via https://github.com/Shopify/maintenance_tasks/pull/284

Closed via https://github.com/Shopify/maintenance_tasks/pull/284

Closed via https://github.com/Shopify/maintenance_tasks/pull/284


Build electronic unconference board

based on last year's blog post-http://ncopenpass.com/blog/friday-september-23-unconference-schedule-the-grid/

@jhibbets to provide more details (if needed)

@jtawes Let me know if you have any questions.

[ ] Sessions times are: 1pm, 2pm, 3pm, 4pm.
[ ] Room names: Pender/Dodgeball (80), New Hanover/Red Rover (25), Brunswick (15), Cumberland (15)

Oh, and we talked about opening the pantry at 3pm so we can have a 5th column with Taco bar at 1pm and 2pm, then add room name Onslow (15)

I created a draft post titled "Unconference Grid - 2018". I created the
empty table with the conference rooms as headers. I also set up the post to
show a different version on mobile since the table would be cut off at that
size, so you will see the table on Desktop and Tablet but on mobile it
converts to sections by time.
On Mon, Sep 17, 2018 at 10:05 PM Jason Hibbets<EMAIL_ADDRESS>wrote:

Oh, and we talked about opening the pantry at 3pm so we can have a 5th
column with Taco bar at 1pm and 2pm, then add room name Onslow (15)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1236#issuecomment-422229047,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlJbYD7gY4-SRfzaweY8qwNe-KbcnWnsks5ucFT2gaJpZM4Ws-oM
.

@jtawes looks great, thanks for helping me get it set-up


Enabling multiprotocol (with DMM: BLE+Zigbee) on CC26x2

Hi,
I think it would be great to collect here all our common efforts to activate the DMM feature on the CC2652. As there is also a convenient and not-expensive Launchpad available, putting together the things shall not be super hard.
I've just registered for TI's resources (already have the Launchpad), but if someone already has done some progress, please share it, so we could continue together.

Just my two cents with the little knowledge I have but I feel like using DMM on a zigbee coordinator seems like a terrible idea. The whole zigbee network expects the coordinator to be available 100% of the time, so if the 2652 is talking over BLE right as a zigbee message is sent to it that would probably result in the end device that sent the message dropping out of the network, maybe even the whole network collapsing.

I have the same fear. DMM seems to be suitable for being end device and BLE peripherial.
But what is the right methodology then to be able to use both ZB Coordinator and BT central on the same board? Two separate chips with two separate antennas placed well above the λ/4 distance from each other? Even we do that, results would terrible as those two will fight for the channel, no?

Bluetooth does not fight about channels, but hops frequencies, i.e. will disturb all "fixed frequency" networks (like zigbee &  wifi) only for a short time and will be disturbed by them only for a short time interval.
Not sure if the fear of blocking zigbee messaging with ble functionality is rational or not. Maybe there is some solution for that?
My main question would be what advantage you hope for with adding BLE to the same board? From what I see bluetooth receiver capable of BLE are very common & cheap (and even included in Raspberry PI for example). So I do not really see the need here.

I have some CC2652 + CC2591 frontend, which can be a very decent and stable ZBC, and I also have a custom board without BT.
My intention is to integrate both BT and ZB, and I thought with this single SoC I can do it (even TI confirmed it long time ago, but it was not a precise question from my end: DMM is only for BLE peripherial + ZB*).
You are right though, BLE receivers are common, but then I would need two separate modules (or at least chips), and two separate antennas. I think two separate modules can work on the same antenna, but you have to be an antenna-and-protocol expert to do that.
Now, as my CC2652 module is equipped with CC2591, it increases the efficiency a bit, it also has external antenna connector => it wouldn't be too hard to put my board into a case.
With separate BT module things getting (over)complicated.

By the way, here are TI's resources for the DMM: https://software-dl.ti.com/simplelink/esd/simplelink_cc13x2_26x2_sdk/<IP_ADDRESS>/exports/docs/dmm/dmm_user_guide/html/dmm/performance.html
They state here under limitations: "The DMM has only been tested/characterized with the stack and role combinations covered by the examples provided in the SimpleLink CC13x2 / 26x2 SDK."
And none of their example mentions BLE Central functionality => I assume they have not "tested/characterized" their solution for this.


HTTP REST API /set_switch

Thanks for the great Tutorial!
Is it possible that you add some more options to HTTP Rest API? I am using OpenHAB 2 and could run just the /set_brightness of the API.
Therefore /set_switch would be interesting to switch on and off. Its possible to bring the brightness to 0 but this is not possible by buttons.

Dear @fthmhmt,
extending the API is easy and could be done directly in McLighting.ino.
But set_switch is only a shortcut for alerdy existing API endpoints, I think:
/set_switch=1 is equal to /set_brightness?c=255
and
/set_switch=0 is equal to /set_brightness?c=0
Isn't it possible to use these API calls in OpenHAB?
Best regards
Tobias

Got it working with the settings below
rules/mqtt.rules
rule "Neopixel"
    when
        Item NeoPixel received command
    then
        switch(receivedCommand) {
            case ON  : sendHttpGetRequest("http://192.168.xxx.xxx/set_brightness?p=100")
            case OFF : sendHttpGetRequest("http://192.168.xxx.xxx/set_brightness?p=0")
        }
end

items/mqtt.items
Dimmer NeoPixel_Dimmer "Neopixel Dimmer" <light> ["Lighting"] { http=">[*:POST:http://192.168.xxx.xxx/set_brightness?p=%2$s]" }
sitemaps/mqtt.sitemaps
Frame label="NeoPixel Dimmer" {
        Slider item=NeoPixel_Dimmer switchSupport
        Switch item=NeoPixel_Dimmer
}


MinValue MaxValue extensible

Is there any plans to extend minValue and maxValue properties on tables?

We kindly to ask you to post your enhancements to Microsoft Ideas if it is not already there.


WIP: Progress bar when creating portable app

Resolves #85 when finished.

Closing this in favor of a simpler approach simply stating [Finished x of y]


Update Hub elasticsearch field names and add dev flag for scripts

What changes are proposed in this pull request?

Renamed proto message fields introduced in https://github.com/Alluxio/alluxio/pull/14951

NOTE: renaming fields break back-compat but OK in this case since the fields (and Hub in general) aren't being used yet
keep naming convention consistent with other related fields

added host field to ElasticsearchInfo proto def
added -d dev flag for metricbeat/logstash scripts to allow for connecting to a locally launched Elasticsearch cluster (for dev purposes)

Why are the changes needed?
see section above
Does this PR introduce any user facing changes?
no

failing test
alluxio.client.cli.job.DistributedCommandsStatsTest.testAsyncPersistCancelStats

alluxio-bot, merge this please


Style attribute does not accept a list of style strings

LiveView accepts a list of string for a style attribute, and automatically concatenates them:

Note that LV only joins the strings together with spaces " " (not semi colons ";") so we have to add our own semi-colons above. In our specific instance the styles are for CSS grid and are quite long, so are formatted as above for readability. I've simplified our real example for clarity.
Surface doesn't accept this so an Enum.join() is required to concatenate the strings manually.

 Enum.join(";")} />

Expected behaviour
It would be nice if Surface mimicked the LV behaviour by auto-joining with spaces as follows:

should produce:

Expected behaviour (if Surface was a mind-reader)
It would be amazing if Surface could detect that we're in a style attr and automatically join with semi-colons (;) in which case

should produce:

Your Environment
Surface: v0.7.3
LiveView: v0.17.9
Elixir: v1.13.3-otp-24

Hi @joshprice
Surface does have a bit of magic for certain attributes like class and style.
In the case of the style attribute Surface accepts keyword lists and renders them as expected into a string:

will produce:

Surface also let's you ommit the square brackets when passing keyword lists to attributes:

will also produce:

Could you give this a try and see if it covers your issue?

Perfect thanks! 👌


Collapsed all window on screen after "click".

Hell all, after I call "click" all window on screen is collapsed.
try:
session.Mouse.MouseMove(null, 1038, 404);
session.Mouse.Click(null);
and
Actions action = new Actions(session);
action.Click().Perform();
Result alway same. Can someone help me?

Hi @Schepetkov,
Would it be possible for you to share the WinAppDriver.exe log + your display DPI setting?

Hi @hassanuz C:\Program Files (x86)\Windows Application Driver log.txt is empty. DPI is resolution? (1920 x 1080).

Windows Application Driver listening for requests at: http://<IP_ADDRESS>:4723/
Press ENTER to exit.
POST /session HTTP/1.1
Accept: application/json, image/png
Connection: Keep-Alive
Content-Length: 142
Content-Type: application/json;charset=utf-8
Host: <IP_ADDRESS>:4723
HTTP/1.1 200 OK
Content-Length: 165
Content-Type: application/json
{"sessionId":"EF15320D-F360-4E37-897D-8527C990A3D9","status":0,"value":{"app":"C:\Program Files (x86)\Teslasuit\1.0\bin\Launcher.exe","platformName":"Windows"}}
==========================================
POST /session/EF15320D-F360-4E37-897D-8527C990A3D9/timeouts HTTP/1.1
Accept: application/json, image/png
Content-Length: 31
Content-Type: application/json;charset=utf-8
Host: <IP_ADDRESS>:4723
{"type":"implicit","ms":1500.0}
HTTP/1.1 200 OK
Content-Length: 63
Content-Type: application/json
{"sessionId":"EF15320D-F360-4E37-897D-8527C990A3D9","status":0}
==========================================
POST /session/EF15320D-F360-4E37-897D-8527C990A3D9/element HTTP/1.1
Accept: application/json, image/png
Content-Length: 33
Content-Type: application/json;charset=utf-8
Host: <IP_ADDRESS>:4723
HTTP/1.1 200 OK
Content-Length: 109
Content-Type: application/json
{"sessionId":"EF15320D-F360-4E37-897D-8527C990A3D9","status":0,"value":{"ELEMENT":"42.722348.4.-2147483644"}}
==========================================
POST /session/EF15320D-F360-4E37-897D-8527C990A3D9/moveto HTTP/1.1
Accept: application/json, image/png
Content-Length: 44
Content-Type: application/json;charset=utf-8
Host: <IP_ADDRESS>:4723
HTTP/1.1 200 OK
Content-Length: 63
Content-Type: application/json
{"sessionId":"EF15320D-F360-4E37-897D-8527C990A3D9","status":0}
==========================================
POST /session/EF15320D-F360-4E37-897D-8527C990A3D9/click HTTP/1.1
Accept: application/json, image/png
Content-Length: 12
Content-Type: application/json;charset=utf-8
Host: <IP_ADDRESS>:4723
HTTP/1.1 200 OK
Content-Length: 63
Content-Type: application/json
{"sessionId":"EF15320D-F360-4E37-897D-8527C990A3D9","status":0}
==========================================
DELETE /session/EF15320D-F360-4E37-897D-8527C990A3D9 HTTP/1.1
Accept: application/json, image/png
Host: <IP_ADDRESS>:4723
HTTP/1.1 200 OK
Content-Length: 12
Content-Type: application/json
{"status":0}

I finded bug, when I run first time all work well, but second try collapse all window for fix I need restart WinAppDriver.exe.

Hi @Schepetkov,
Can you answer the following questions:

Are you running multiple-monitors?
To change Display DPI settings, you can right click on Desktop -> Display Settings -> then adjust the "Change the size of text, apps, and other items" percentage to 100%.
Can you share which coordinates you are trying to click?

Thanks

Hi @hassanuz , My DPI settings is 100% percent and I use only one monitor, about my coordinates I use "inspect.exe" and get current coordinate app button what I need click. How I write before if I start first time "WinAppDriver" all work well, but after second run my test when driver try click on the same button all my window is collapsed. How can I check why it hipping?

Hi @Schepetkov,
Can you try sharing your script code? We would like to try and reproduce this on our end.
Thanks

Hi @Schepetkov,
I'm marking this issue as closed. Please feel free to reopen if this persists. Also if you would f eel more comfortable privately sharing your script code (or any repro) please feel free to reach out to me directly via<EMAIL_ADDRESS>Thanks


remove shadow db references

Type issue :( https://discord.com/channels/770287896669978684/1039154072642662410

Not related to this PR, I'm waiting for a solution. Will check that later


[e2e]: go push main config-ldflags slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/8447413082
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.push.main.config-ldflags.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.push.main.config-ldflags.slsa3.yml
Trigger: push
Branch: main
Date: Wed Mar 27 06:13:39 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/8463177486
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.push.main.config-ldflags.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.push.main.config-ldflags.slsa3.yml
Trigger: push
Branch: main
Date: Thu Mar 28 06:12:24 UTC 2024
Tests are passing now. Closing this issue.


Added AUR repo to README

I have created an AUR package. Because on Arch Linux the use of Pip is only possible in a virtual environment. If you try to use pip system/user wide, pip will interfere and expects the package to be installed via Pacman (the package manager of Arch Linux)
Here is the link: https://aur.archlinux.org/packages/python-pytermgui-git

Appreciated! I would rather display this as an additional option inside the first box, just adding it as another line instead of a whole new section for installation. Would that be possible?

Sure i'll do it

Now it should be as you want it to be c:


Create laravel.yml

@uxsl Thanks for your PR!
Currently, there are no tests for this project, so there is no point of running this github action, but I will consider it if we add some tests.


validation for mat-select multi Angular 11?

Hi Team,
Is there any way to set form validation for mat-select multi like minimum one value to select so the enable submit button?
I've no clue to how and where to start with.? could someone help me on this?
How can enable button only after user selecting at least one value
MINIMAL EXAMPLE
onSelectionChange(e){
// how to get checkbox infromation here
}

Thanks

Please take a look here.

Closing since @lekhmanrus already posted the correct answer.


Add ignorechans to iqrm_mask

Add an optional ignorechans argument to iqrm_mask
This is useful when you have a pre-existing list of bad channels.
Votes from these channels will be ignored.

Hi @kmjc,
I had a look and I think that the net effect of your code change is that all channels in ignorechans are forcibly masked, on top of those already masked by IQRM. Therefore, instead of manually editing the votes graph (which is a hack), you could simply create a fixed boolean channel mask from your ignorechans list, and fuse it with the mask returned by iqrm with a logical OR.
From a user perspective, I mean this:

def chanlist_to_mask(chanlist, nchan):
    # TODO: check that no index in chanlist is > nchan - 1
    mask = np.zeros(nchan, dtype=bool)
    mask[chanlist] = True
    return mask

final_mask = iqrm_mask(<params>) | ignorechan_to_mask(ignorechans, nchan)

Is that your actual intent here ? If not, please let me know what you're trying to do. If yes, then I would argue that the above is the cleaner way to achieve the result and that the present MR is unnecessary. I'd still be open to the idea of exposing an extra function iqrm_mask_ignorechans() that does the above in spirit, without touching the iqrm_mask() function itself.

Ah apologies, I did not make my motivations clear!
My aim is to ignore any votes cast by channels in ignorechans.
Our band is particularly plagued by RFI and there are a few large stretches of bad channels. I was finding that iqrm_mask doesn't always behave well around the edges of these areas, or in small sections of good channels which are surrounded by large stretches of bad ones.
Here's a plot showing the difference.

It is a purposefully an extreme case as in the top two plots I didn't fill in the known bad values with anything sensible. But if you do fill them in, what iqrm_mask zaps changes at the borders dependent on what your data look like and what method you use to find a fill value. I was finding the filling scheme then needed a lot of tweaking for different days and pointings.
(I think in this case I could solve the issue by increasing the radius used - it's at 10% of the band - but the stat in these plots happens to be uncommonly flat across our band. Others aren't at all, and the band has a lot of structure so I don't think that would go well)
It's definitely a hack(!), and if this seems like a niche case then no problem, please close the PR, I just thought it might be useful for other people

Oh this is more interesting than I initially thought.

My aim is to ignore any votes cast by channels in ignorechans.

So it probably means that "stat" has a low value in all of those channels ? What does "stat" represent here ? Spectral standard deviation, or something else ?
The main fundamental limitation of IQRM is that it is only as good as the contamination statistic you provide it.
Your case might be relatively common so now I'm curious. Would you mind emailing me the "stat" array you used in that example ? (vmorello, at, gmail, com). In fact, feel free to send other edge cases too. I'd like to understand for myself what's going on internally here.

If you're trying to work around having wide intervals of channels having a low stat value, what you're doing is not a hack and you've got the right idea. That being said, I believe that a more efficient way of achieving the result you're after would be this below. No need to edit the votes graph, and only two lines of code changed in total, just in the "final vote count" loop:
ignorechans = set(ignorechans)  # convert to set for O(1) lookup

for i, casters in votes_received.items():
    for j in casters:
        if j in votes_cast and len(votes_cast[j]) < len(votes_received[i]) and j not in ignorechans:
            mask[i] = True
            break

But then, it's not quite 100% statisying yet, because the values in ignorechans are still included in the estimation of the maximum acceptable "lagged diff" values in outlier_mask() and it's still not as good as truly ignoring them in the whole process. We could come up with something even better.

Looks like we've got ourselves a really good MR now :smile:


[Go SDK] Azure Germany is missing in azcore/cloud

Bug Report
Not all Azure clouds are in the go SDK. AzureGermany is missing here.

SDK version (azcore +v1.0.0 )

output of go version

go version go1.17.9 darwin/amd64

What happened?

Not all Azure clouds are in the go SDK. AzureGermany is missing here.

What did you expect or want to happen?

Be able to get the configuration of AzureGermany as other clouds (AzurePublic, AzureChina, etc..)

How can we reproduce it?

Anything we should know about your environment.

The Azure Germany cloud has closed so we never added it.


Generate identifiers for event documents rather than deriving them

When writing event documents to the underlying store, the ID should be able to be a consistent known format, rather than derived from the commit sequence appended to the commit ID. From a data store perspectives, if someone is using UUIDs, they expect UUIDs everywhere. Or if it's mongo ObjectIds, they should be consistent. This enables data portability with consistent data types for identifiers.
This PR should address the asynchronous support issue from my previous PR.
Thanks @nanov for maintaining this project.

Thank you very much!
In short, iI really don't want to mess with the id logic right now. I also don't see any specific gains from this change and this could possibly lead to some confusion with existing consumers.
Just a few points:

Ids are stored as strings anyways ( for now ), i=I really don't see the necessity of this ( possibly breaking for some consumers ) change right now.

The current implementation as it is, relays on the DB driver to generate ids, this means that those won't be consistent between DBs anyways. ( certainly something that needs to be improved ).

I suggest that you take a look at the position option, this gives each id a unique, auto-incremented position property, which can be used for ordering, or identification.

Ids are stored as strings anyways ( for now ), i=I really don't see the necessity of this ( possibly breaking for some consumers ) change right now.

Obviously, we disagree. It is only because the Ids are currently stored as strings that this code even works. If Ids were stored as any other data type, this code would break down. Two examples off the top of my head: integers and UUIDs. In both cases, this code would be problematic.

The current implementation as it is, relays on the DB driver to generate ids, this means that those won't be consistent between DBs anyways. ( certainly something that needs to be improved ).

If the current implementation did rely on the DB driver to generate ids, I wouldn't have requested this change. The truth is the current implementation relies on the driver-specific class to generate ids EXCEPT for the event id (which is the most visible id because it is the id of the top level item in the data store). That id and that id alone is generated by appending two strings together (a generated id and an integer).

I suggest that you take a look at the position option, this gives each id a unique, auto-incremented position property, which can be used for ordering, or identification.

I'm not sure how another property on the event document (position) will help alleviate my concerns about the id of that document. Interesting functionality, but it doesn't really help me here.
If this were an opt-in that didn't affect existing consumers, would you be more willing to consider it? What if it were an optional way to override line 335 of mongodb.js? Let me explain my exact use case. Our team wants all documents to by identified by a version 4 UUID in our mongo data stores and I'd really rather not have my own version of mongodb.js. I'm already monkey patching the getNewId function. But monkey patching a one line function (getNewId) is a lot more palatable than a 78 line function (addEvents).
Thank you for your consideration and for engaging in this discussion.

Obviously, we disagree. It is only because the Ids are currently stored as strings that this code even works. If Ids were stored as any other data type, this code would break down. Two examples off the top of my head: integers and UUIDs. In both cases, this code would be problematic.

There is a reason why IDs are stored as strings.

If the current implementation did rely on the DB driver to generate ids, I wouldn't have requested this change. The truth is the current implementation relies on the driver-specific class to generate ids EXCEPT for the event id (which is the most visible id because it is the id of the top level item in the data store). That id and that id alone is generated by appending two strings together (a generated id and an integer).

I do get your point here, you are right.

I'm not sure how another property on the event document (position) will help alleviate my concerns about the id of that document. Interesting functionality, but it doesn't really help me here.

My idea was that you could use it as ID, never mind.

If this were an opt-in that didn't affect existing consumers, would you be more willing to consider it? What if it were an optional way to override line 335 of mongodb.js? Let me explain my exact use case. Our team wants all documents to by identified by a version 4 UUID in our mongo data stores and I'd really rather not have my own version of mongodb.js. I'm already monkey patching the getNewId function. But monkey patching a one line function (getNewId) is a lot more palatable than a 78 line function (addEvents).

I have a few concurs, first and main one is that Mongo is just one of the drivers ( I do know for fact a few production projects relaying on Redis stores ), the other one is that at this point I don't want to introduce
a change that is somehow not backward-compatible ( ie. would require some kind of migration from existing users).
Having those in mind, lets try and make those two ( commit and event ) ids ( generators ) configurable and leave the current behavior the default one.
Because of historical reasons the options object should stay a POJO one ( ie without functions in it ), and additional functionality should be added trough helper functions ( something like defineEventIdGenerator, and defineCommitIdGenerator ), this way you could achieve what your after without monkey patching.
I will try find time today to draft something quick.
Thank you for interesting discussion!


Default mail templates: Nearly all do not use fallback chain for sales channel name in subjects and for sender and maybe content

Description
Nearly all mail templates currently use salesChannel.name in subjects and for senders. However, it should be salesChannel.translated.name to correctly use the fallback chain.
Maybe also an issue in the content of the templates but I did not check. Might also be the case for payment method / shipping method names.
Possible Solution
Adjust default mail templates to use language fallback chain for sales channel name.
Not sure, what to do about existing mail templates. Maybe replace salesChannel.name with salesChannel.translated.name or just ignore it.

Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know.
Thank you again for your valuable feedback.


GetUserInfo method

Nice, thanks Bill.
Can you add a Skip line to that test, indicating that it requires an online user to actually pass.  If the skip attribute is there, the build server won't try to run that test.
I think it goes something like:
[Fact(DisplayName = "Can get user info", Skip="Requires an online user to pass")]
public void theTest()
{
}

Added the skip. This test will be hard to get to pass without knowing what users are attached to the auth token.

thanks :+1:


Package structure

Great work on the transpiler! :smile:
This issue relates to the part of the README that says:

I'll work on the readibility and organization of the code, would love suggestions on how/where to do that.

Would rockstar-py be better structured as a package? E.g.:

Move rockstar.py to rockstar/__init__.py
Add a setup.py (setuptools) file, with console script entry point
Add rockstar/__main__.py for running as a module (e.g. python -m rockstar)

If you think this is a good idea, I'd be happy to make these changes and open a PR

I'm not sure if this is related to this repo not being structured as a package, but after pip installing rockstar-py and running rockstar-py, a ModuleNotFoundError is raised:
$ rockstar-py
Traceback (most recent call last):
  File "/home/james/.local/share/virtualenvs/test-wygyIHa6/bin/rockstar-py", line 7, in <module>
    from rockstarpy.command_line import main
  File "/home/james/.local/share/virtualenvs/test-wygyIHa6/lib/python3.6/site-packages/rockstarpy/__init__.py", line 1, in <module>
    from .rockstarpy import rockstar
ModuleNotFoundError: No module named 'rockstarpy.rockstarpy'

jmsv, thanks for the issue! The package wasn't properly organized, that should be fixed now. (You will need to clear pip cache and reinstall the package. I could only work it out on another machine :/ but it's working now haha)

installing with pip lets you use the program in the command line. I'll add this in the README tomorrow morning

Works well now, thanks :+1:


Prev/Total in BTC withdrawal

Hi
Please tell me how to find out the current limit and the balance in BTС for withdrawal of funds from the user through api.
All i found is
https://www.binance.com/assetWithdraw/getWithdrawNum.html
but this is not correct when using api.
Thank.

https://t.me/binance_api_english

and what, is it there?


how to convert uncrypted .ini to crypted .ini

hello, thanks,
please, if i have an existing .ini file, how can i crypt it?
i need a converter.
i read the docs and source code, could not figure it out.
thanks much,
david

Easy one, use read function to read original ini file then use write_encrypted to create New version.
Keep in mind that secure deletion of source file is out of scope.

yes, that was easy one and a quick reply.
fwiw, to me, was not clear that configparser_crypt.read() reads a non-decrypted .ini file
thanks much,
david

thanks much,
overall, very easy to switch from python config to your configparser_crypt
tho i ran into some confusing problems with interpolation errors
that % needs to changed to %%`

deajan, yes, that was me at stackoverflow,
fwiw, when i first posted there, i tried to upvote but cannot yet.
i just tried again, still cannot upvote, sorry about that
"Thanks for the feedback! You need at least 15 reputation to cast a vote, but your feedback has been recorded."

here is the output of my python program,  tried to read RclonePassword from the .ini file
the trouble text from .ini is RclonePassword=+oy%#"An2EBKc3Gb@
if i change that to RclonePassword=+oy%%#"An2EBKc3Gb@, then no errors
and here is the output from python.
DoEnd: ERROR: GetKey - Traceback (most recent call last):
  File "C:\data\rclone\common.py", line 167, in GetKey
    Value = Common.config[Section][TheKey]
  File "D:\Program Files\Python39\lib\configparser.py", line 1255, in __getitem__
    return self._parser.get(self._name, key)
  File "D:\Program Files\Python39\lib\configparser.py", line 799, in get
    return self._interpolation.before_get(self, section, option, value,
  File "D:\Program Files\Python39\lib\configparser.py", line 395, in before_get
    self._interpolate_some(parser, option, L, value, section, defaults, 1)
  File "D:\Program Files\Python39\lib\configparser.py", line 442, in _interpolate_some
    raise InterpolationSyntaxError(
configparser.InterpolationSyntaxError: '%' must be followed by '%' or '(', found: '%#"An2EBKc3Gb@'

Okay, that's actually a ConfigParser error that ConfigParserCrypt just inherits.
By default, ConfigParser uses interpolation.
You can disable that behavior in order to use 'raw' chars by using
config_file = ConfigParserCrypt(interpolation=None)

With the above, the '%' char will not be interpreted.
Just added an interpolation test in https://github.com/netinvent/configparser_crypt/commit/e46240f8dbc209c64c6ed7f2f33854b9b403ec60

good, that  is not an issue with your code.
i will have to educate myself about interpolation.
i am good now,
david


fix it

the unlock all pets hack makes me logout:and when i do nothing happens

When you put in the hack look on the top of your screen and it should say something about how much pets you want. After putting the amount of pets you want, it should say that you have to log back in again. And then you log back in and check your pets and done!


recent room list takes up most iframe

on narrow iframes, the new "recently visited" list of chatrooms takes up most of the space. Would it be possible to hide it when loaded through the SDK?

Hi Michele,
We have already removed the room list on the embedded version. This will be live on the next deploy.
-- Svein, appear.in

awesome, thanks!
On Jun 25, 2016, Svein Willassen<EMAIL_ADDRESS>wrote:

Hi Michele,
We have already removed the room list on the embedded version. This
will be live on the next deploy.
-- Svein, appear.in

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/appearin/appearin-sdk/issues/15#issuecomment-228562811

--
Sent with K-@ Mail - the evolution of emailing.


Add logger to the different functions

Add logger to some of the different functions to inform the user about the behavior of the preprocessing method


Add a like button in the songs card

On the Music Hub, there is no 'like' button present on any of the song cards. Use local storage to store liked songs in the memory. Here is a picture showing where you should implement the 'like' button.
In the playlist Page

In the search Page

In the home page

I would like to work on this issue.

I would like to work on this issue.

Sure, please proceed @SonalGuptaa
Happy Coding...!!


Adding rollback tagging step in github actions for uat and testnet

Description
Adding rollback tagging step in github actions for uat and testnet.
This will make it so the rollback image is more easily identifiable if a deployment needs to be rolled back.
Contributing Agreement

[x] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

Merged without approval because @tynes approved the exact same change in the go-ethereum repo.


feature/DEVOPS-30 to main

rainsing PR as we fixed creating github teams errors part of issues #1

https://github.com/devwithkrishna/create-github-teams/actions/workflows/create_github_teams.yml
refer this workflow.


:hammer: Replace object.update(fn) by util.convert(fn)

fix #39

Codecov Report

Merging #40 into master will not change coverage.
The diff coverage is 100%.

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


Feature/python3.8

Description
Followed the same directory structure as in master branch .
added split processing function in feature/python3.8 branch
Multithreading enabled in python 3 .
Fixes #59  (issue)
Type of change
converted python 2 file to python 3
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] Any dependent changes have been merged and published in downstream modules
[x] I have added all the three reviewers: @vinitshahdeo, @kashish121 & @ishika1727

Please review this @vinitshahdeo , @Kashish121 and @ishika1727

@Kashish121  I corrected everything please have a look

@Kashish121 please tell me how to proceed


removed bulkSetItems from public interface because the use case would…

… be rare, and placed it into the constructor where it would be far more common

Codecov Report

Merging #44 (82a0841) into master (8bf2edf) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              master       #44   +/-   ##
===========================================
  Coverage     100.00%   100.00%
- Complexity        62        63    +1
===========================================
  Files             14        14
  Lines            140       142    +2
===========================================
+ Hits             140       142    +2

Impacted Files
Coverage Δ
Complexity Δ

src/KeyToValueArray.php
100.00% <100.00%> (ø)
17.00 <4.00> (+1.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 8bf2edf...82a0841. Read the comment docs.


write_sdf missing delay values for the typical corner

The sdf file output from write_sdf doesn't have delay values for the typical corner. It only has the minimum and the maximum corner. For example, this path only has the max and min delays
   (IOPATH A X (0.069::0.128) (0.069::0.132))
I tried to explicitly define the corners and read the liberty with the -corner typ option, but it didn't help.  I am attaching a test case to reproduce the issue.
spm_test_case.zip

sdf does not require all three corners to be present and two is quite typical.  What is the simulator issue?

The issue is that the skywater-pdk has a placeholder for the three corners in the specify blocks, so the simulator expects to see three corners in the sdf file as well.

There is no relationship between process corner names and the position a delay is written in an sdf triple.
The min/max values written by opensta are specifically designed to be compatible with static timing analysis
and because sdf only supports 3 values they are the min/max values for a specific process corner.
I added an option -include_typ that inserts the meaningless average of min and max just to make the confused
simulator happy.


Fix for opening browser in Windows.

Running snowpack dev in my Windows system does not open the browser. Instead it treats the url as the Title of the new console window where the command is run. This fix prepends an empty parameter when the environment is Windows so that this one gets treated like the Title of the window and the second one as the actual url that needs to be open.
Fixes #326
Sorry, didn't have time to write a test so please review carefully.

LGTM, thank you!


Enhance ScubaGear AAD report by clarifying how conditional access policy exclusions work

💡 Summary
This issue is meant to improve how we communicate conditional access policy user/group exclusions to users in ScubaGear. We received feedback from a user in another issue that they did not understand why their configuration was failing the policy check for AAD 1.1 even though they were following the configuration instructions. The user was not aware of config file CA policy exclusions.
Currently the AAD report has a note at the top of the page that mentions CA policy exclusions, but it does not appear to be designed to help users setup exclusions via the config file when they need to, therefore this issue is focused on helping the users do so.
Motivation and context
Users can get clear instructions on how exclusions work and how to set them up so that they pass the policy checks, assuming their configuration is compliant with Scuba.
Implementation notes

[ ] Determine how the message at the top of the AAD report should be modified and adjust it. Screenshot of the current message for reference is below. Add a link to the ScubaGear README exclusions section so that users can follow the steps to setup their config file with exclusions when needed.
[ ] Augment the README exclusions sections to make it clear that exclusions should only be used when an organization has performed a formal risk acceptance of the excluded accounts via their risk management process.
[ ] Perform the HTML update that Mitchel described in his comment attached to this issue.

We have a nested  <p> tag ("Note: Conditional Access (CA).." text) within an <h4> tag here which fails guideline 2.4.6. If we keep this text then the <p> should be removed and the <h4> should be modified.

@dagarwal-mitre Please replace the existing Note paragraph at the top of the AAD baseline report with the revised paragraph below:
Note: Several of the policies in this baseline allow you to define specific user accounts that will be excluded from the assessment checks via a configuration file. Please reference this section in the README file if you need instructions on setting up exclusions. These types of account exclusions must only be used if they are approved within an organization's security risk acceptance process. Exclusions can introduce grave risks to your system.


SSHException: Error reading SSH protocol banner debug

Every now and then an ssh operation in one of our tests throws an exception such as:
Traceback (most recent call last):
File "/usr/share/chroma-manager/tests/integration/shared_storage_configuration/test_lnet_functionality.py", line 15, in setUp
super(TestLNetFunctionality, self).setUp()
File "/usr/share/chroma-manager/tests/integration/core/api_testcase_with_test_reset.py", line 137, in setUp
self.reset_cluster()
File "/usr/share/chroma-manager/tests/integration/core/api_testcase_with_test_reset.py", line 626, in reset_cluster
self.remote_operations.unmount_clients()
File "/usr/share/chroma-manager/tests/integration/core/remote_operations.py", line 921, in unmount_clients
'mount')
File "/usr/share/chroma-manager/tests/integration/core/remote_operations.py", line 375, in _ssh_address
ssh.connect(address, **args)
File "/usr/lib/python2.7/site-packages/paramiko/client.py", line 338, in connect
t.start_client()
File "/usr/lib/python2.7/site-packages/paramiko/transport.py", line 492, in start_client
raise e
SSHException: Error reading SSH protocol banner

It would be good to get some idea of why this is happening.
So let's first gather some diagnostics at the time that this is occurring. We might discover that simply trying again will be successful but it would be interesting to see current state before simply assuming
that is the solution.

It's difficult to correlate times exactly due to the lack of time-stamping in the console logs but the few instances of this that I have investigated have LU-9568 happening on the same node as this failure so this failure is entirely expected if it is happening while LU-9568 is happening.

Appears to no longer be an issue.


PRODUCT_BUNDLE_IDENTIFIER should be user specifiable

Description
We'd better provide a user-specifiable string for the package's bundler ID (CFBundleIdentifier).
Those (like me) who migrate Framework to Swift Package might want to make this field consistent, especially for private packages that share the same prefix in reverse-DNS format.
Expected behavior
Typically, we use a reverse-DNS format for bundle ID strings. PRODUCT_BUNDLE_IDENTIFIER should be user specifiable, like using product.bundleID instead of product.name if the bundlerID property exists.
Btw., I noticed there's a commit that mentioned this TODO when @bhargavg set this setting field, but it seems abandoned now. Any reason here? Apologies if I missed anything that was discussed before.
"[Xcodeproj] Make module name as bundle identifier"
Actual behavior
Currently, the CFBundleIdentifier uses PRODUCT_BUNDLE_IDENTIFIER:
https://github.com/apple/swift-package-manager/blob/f05e1fa98631217ca96c63dfe7e74903cdd0c412/IntegrationTests/Fixtures/BinaryTargets/Inputs/SwiftFramework/SwiftFramework/Info.plist#L10
And PRODUCT_BUNDLE_IDENTIFIER uses product.name now:
https://github.com/apple/swift-package-manager/blob/f05e1fa98631217ca96c63dfe7e74903cdd0c412/Sources/XCBuildSupport/PIFBuilder.swift#L416
i.e., the product name and bundle id are sharing the same string.
Steps to reproduce
No response
Swift Package Manager version/commit hash
5.7
Swift & OS version (output of swift --version && uname -a)
swift-driver version: 1.62.15 Apple Swift version 5.7.2 (swiftlang-<IP_ADDRESS>.5 clang-14<IP_ADDRESS>)
Target: x86_64-apple-macosx13.0
Darwin MacBook-Pro.local 22.3.0 Darwin Kernel Version 22.3.0: Mon Jan 30 20:42:11 PST 2023; root:xnu-8792.81.3~2/RELEASE_X86_64 x86_64

I think this is a reasonable request, but we would need the ability to set Xcode/Apple-specific settings in order to implement this since frameworks are an Apple concept.


fixed the issue: dropdown granite models are editable

fixed the issue: dropdown granite models are editable

Thanks @lavanyaj3


Core::Credentials::TokenCredential implementation for public clients

Query/Question
Is there/or will there be an implementation of the Core::Credentials::TokenCredential class for public clients? Where one does not need to provide a client certificate or client secret. Similar to the .NET PublicClientApplication class or Python PublicClientApplication  class. Or is there a way to achieve this using azure-sdk-for-cpp?
Why is this not a Bug or a feature Request?
Not a bug: I'm not running code with expectations different than the outcome. Not a feature request: I'm not suggesting to implement something. I'm simply asking a question.
Setup (please complete the following information if applicable):

OS: Windows
IDE : Visual Studio
Version of the Library used: commit 593464cedaf21b00d7b6fd1467f25ab9284ebe1b azure-core_1.6.0, azure-identity_1.3.0-beta.2

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

Thank you!

Hi @benstadlbauer, can you please provide more details, how would you want to use that type of credential? How would you like the code to look like? If you have a code snippet in other language, that's good too.
PublicClientApplication is from MSAL library and not Azure SDK, and there is no publicly available C++ MSAL at the moment. But we may be able figure something out, depending on your scenario.

Hey @antkmsft, I'm very new to this library and I wasn't really sure how to formulate the question. I also was not aware, that the MSAL library is something different than the Azure SDK, but I am still not sure how these two relate. According to this stackoverflow post, which refers to this github issue and this other github issue, it seems that MSAL is a building block for AzureSDK.
What I was looking for initially was something like
auto cred = Azure::Identity::PublicClientCredential(getTenantId(), getClientId());
auto tokens = cred.get(getScopes());

which automatically opens up a browser window to sign-in, similar to Get Azure AD tokens by using the MSAL Python library. I'm also not sure how the tokens would look like.
The reason why I ask is I have to implement the authentication process for accessing a resource via a REST API, where the authentication follows a federated authentication. In the end I need to attach a cookie to my requests (among a certificate but I'm not sure about that either), so I guess the variable tokens would contain this cookie somehow. I already have implemented a process using cpprestsdk, but the way I get the cookie now is I recreated the authentication protocol, containing of several HTTP GET and POST requests, each of them returning information for the subsequent ones. This was very tedious.
Conclusion: Apparently I would need a MSAL library for C++, which does not exist (yet).
I don't know enough about these processes (yet) to propose a way how to implement something in this library which could to a federated authentication process interactively (opening a browser window).

@benstadlbauer, you are right, if MSAL is available, it can be a dependency for an azure-identity, and then the azure-identity would wrap around some of its functionality and provide support for the scenarios such as interactive log-on with browser window. As I said, unfortunately, there isn't MSAL for C++ at the moment.
So, to your question, there is currently no interactive log on with UI support in the library.
We'll see what we can do, maybe we could provide some class with a virtual function, or with some callback, that gets called and it gives you a URL to display, or an HTML that you can render in the UI, but that won't happen in the next few months at least.

@antkmsft thank you for your information. Closing the issue now.


Update Output Comparison for pytest

This PR changes the result comparison method from iree-run-module comparison to numpy based comparison of iree_outputs and golden_outputs

Progress on https://github.com/openxla/iree/issues/16674

Alternate approach to keep using expected output added here https://github.com/nod-ai/SHARK-TestSuite/pull/212

Closing in favor of https://github.com/nod-ai/SHARK-TestSuite/pull/212


Is there a way to pass initial value inside the render function ? I pass 'value' property to Field

Help Required
Is there a way to pass initial value inside the render function ? I pass 'value' property to Field.

Good question, solution would also help me :)

Something like this?
`import React, { Component } from 'react';
import { change } from 'redux-form';
class ReduxFormInputContainer extends Component {
componentDidMount() {
const { defaultValue, meta } = this.props;
if (defaultValue === undefined || meta.initial !== undefined || meta.dirty)
return;
const {
meta: { form, dispatch },
input: { name }
} = this.props;
dispatch(change(form, name, defaultValue));
}
render() {
const { defaultValue, component: Compo, ...fieldProps } = this.props;
return <Compo {...fieldProps} />;
}
}
function reduxFormInputContainer(component) {
return function(props) {
return <ReduxFormInputContainer {...props} component={component} />;
};
}
export default reduxFormInputContainer;
`

Am I right in thinking that would set dirty === true?  If so, that's no good, 'cause it breaks enableReinitialize && keepDirtyOnReinitialize.

@vishnukalakota You could pass initialValues to reduxForm HOC or as props to wrapped component.  So, if you want to pass initial values in render method, it needs to be in the render method of who's rendering the wrapped component (https://redux-form.com/8.2.2/docs/api/reduxform.md/#config-properties)

I would still like to be able to specify a default on a field, though...


🤖 Remove obsolete maintainers.json file

The maintainers.json file has been deprecated and no longer has any use.
This PR removes this maintainers.json file.
Tracking
https://github.com/exercism/exercism/issues/6094

Closing this as it is a duplicate of https://github.com/exercism/go/pull/1928


playground: synchronize editor library with tour

for unknown reasons the playground and the go tour use completely separate editor libraries. The tour one is much more advanced leveraging http://codemirror.net.
Ideally we would use a single script to power both and simply embed one in the other. To as much of a degree as possible sharing code between them would result in better features and less maintenance.
Somewhat dependent on https://github.com/golang/tour/issues/146

Hi, maybe considering merging https://goplay.space/ into the official playground would be a good improvement for all gophers? Thank you.

I've not seen this before. It looks quite impressive.
On Sat, Jun 10, 2017 at 6:38 AM Florin Pățan<EMAIL_ADDRESS>wrote:

Hi, maybe considering merging https://goplay.space/ into the official
playground would be a good improvement for all gophers? Thank you.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/18723#issuecomment-307557218, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAKlZITP_8lE1Baaa1llW-5IcsmtdO8Xks5sCnIxgaJpZM4LovYG
.

CodeMirror is an enormous library and we wouldn’t use 99% of the features. I’d rather move the tour to a simpler editor implementation.

Okay. Adding the whole library just for the sake of syntax highlighting is a no go. Closing.


A list of objects automatically add an item even if all of the fields are empty

        schema={
            'type': 'array',
            'items': {
                'type': 'dict',
                'keys': {
                    'foo': {
                        'type': 'string',
                    },
                    'bar': {
                        'type': 'string',
                    },
                },
            },
        },

if you edit an instance that has [] in database, even if you don't touch any of the fields, it automatically stores
[{"foo": "", "bar": ""}]

While I understand that's how django manages empty string fields, setting them to an empty string instead of null, it doesn't make sense for it to happen in cases like this. It's causing us problems because then our frontend/api tries to return this stuff and it shows an empty item.
I'm not sure how to handle this case, but here's three suggestions:

If min_items is 0, don't show an empty item by default, but if a user clicks on add item and doesn't add anything the issue still happens, which is not really the expected behavior.
A new option for dicts to only be stored if any of the values is not null (nested objects are a problem)
Let the user handle it by extending the field

Okay, if the field is initially empty and is unedited, I will add some checks to prevent adding empty items.

but if a user clicks on add item and doesn't add anything the issue still happens, which is not really the expected behavior.

You can't really know if the user intends to keep an empty item or not. So, if the user manually adds an empty item, that should be kept in the data.

A new option for dicts to only be stored if any of the values is not null (nested objects are a problem)

I'll provide some validation hooks for this. Before saving the data, your custom validation functions will be called where you can remove non-empty dict keys.
I will try and include this in the next release (maybe next week).

You can't really know if the user intends to keep an empty item or not. So, if the user manually adds an empty item, that should be kept in the data.

That's fair, it's a weird case were both storing as empty or not storing could be interpreted as valid.

I will try and include this in the next release (maybe next week).

Thank you so much.

@alexgmin Hi, I'm working on this right now. I'm looking for some suggestions. If the array is unedited, should I save an empty array ([ ]) in the db or keep the field completely empty (null)?

I think [] makes more sense too, but what if the field has a default? Shouldn't it be that?

Ah, thanks for reminding me that.

So the only change you're making is making it empty in min items isn't set, not setting it to the default if it's empty?
What you suggest is feasible, although I'm not a fan of overriding the save method for this type of things, since the save method isn't called in bulk updates. What I've been currently doing for this type of custom things, like https://github.com/bhch/django-jsonform/issues/18 is creating a custom field and overriding the get_prep_value or the pre_save methods
class ActionTimestampsJSONFieldWithSchema(JSONFieldWithSchema):
    # Remove values that are empty strings
    def pre_save(self, model_instance, add):
        value = super().pre_save(model_instance, add

        for timestamp_key in value.copy().keys():
            if value[timestamp_key] == '':
                del value[timestamp_key]

        return value

I also have a similar case in which I want an object to be null if none of boolean keys inside are true, because that makes querying that field much simpler.
If the min items issue is resolved I can work with the rest.

So the only change you're making is making it empty in min items isn't set, not setting it to the default if it's empty?

Yes, if a default is provided it will use that value instead of being blank.
I'll also add a pre_save_hook parameter for your use case. If you have a better suggestion, do let me know.
I'll try to release the new version by tomorrow.

A pre_save_hookwould allow me to remove all of of my overrided fields, so I would be extremely grateful.

I've released v2.10.0 which fixes this.
Please upgrade and test out the new version.

@bhch Perfect, will do. I also sent you an email about a private issue, could you check it?


Blocking psock_tcp_recvfrom waits for all requested data.

Summary
The function recvfrom for TCP operates in two stages: first it reads any available data in the read-ahead buffer, and then blocks waiting for the rest of the data.
The offending line here caused the recvfrom to return less data than the requested amount, which is wrong.
Two cases exist:
If the socket is in blocking mode, it can read from the read-ahead buffer, but it should also wait for the specified time till all data have been received.
If the socket is in non-blocking mode, then it should return only what is available in the read-ahead buffers, and not wait (block).
(This is correctly handled by an if case above).
Essentially, the socket was always behaving as if it was set in non-blocking mode.
This PR fixes this, making the socket to actually respect the time-out, and return the correct data size.
Impact
TCP should behave better now, but I don't expect any impact on existing applications.
If some code was working correctly with the bug in place, then I guess it will continue to work correctly.
Testing
This is tested on a proprietary server on an STM32F4 board.
Before the fix, it was impossible for the clients to correctly communicate with the server.
After the fix, all communication seem very stable (at least in regard to this aspect of TCP).

@fjpanag the change may block the recv indefinitely when caller use INFINITE timeout. The internet has many discussion about this special behavior, here is example:
https://stackoverflow.com/questions/2295737/case-when-blocking-recv-returns-less-than-requested-bytes
Does the server write from scratch?

@xiaoxiang781216 The fact that recv can return less bytes than requested, does not mean that it should.
At least in my case, all data have been received by the device, in a single TCP frame, but only part of them is in the read-ahead buffer. The data is smaller than the MTU and the available buffers.

the change may block the recv indefinitely when caller use INFINITE timeout.

This is the expected behavior when you use an infinite timeout, and it is unrelated to the contents of the read-ahead buffer.
In fact, if data are unavailable, then surely the code will block (even with the old implementation), since the buffers will also be empty!
The proposed change allows the user to fine-tune the socket to their needs, choosing between blocking or non-blocking and selecting a proper timeout. I don't see any problems, or how it can harm existing applications. Quite the contrary, you now have more options.
It also simplifies the user code, as now I have to re-do what the socket does (i.e. loop with recv() till all data are received or till timeout).
Do you see any problems / disadvantages with this PR?

Maybe the correct approach would be to implement MSG_WAITALL in flags.
I will try this, and update the PR.

Maybe the correct approach would be to implement MSG_WAITALL in flags.

Yes, this is right direction to go.
Both receive what available and return immediately or receive all required size is useful in different case. Since POSIX define the default behavior is first one, other strategy has to be indicated through flag.

I will try this, and update the PR.

Closing in favour of #7286.


the error message as follow

python demo/image_demo.py demo/demo.jpg rtmdet_tiny_8xb32-300e_coco.py --weights rtmdet_tiny_8xb32-300e_coco_20220902_112414-78e30dcc.pth --device cuda
Loads checkpoint by local backend from path: rtmdet_tiny_8xb32-300e_coco_20220902_112414-78e30dcc.pth
The model and loaded state dict do not match exactly
unexpected key in source state_dict: data_preprocessor.mean, data_preprocessor.std
03/07 15:54:04 - mmengine - WARNING - Failed to search registry with scope "mmdet" in the "function" registry tree. As a workaround, the current "function" registry in "mmengine" is used to build instance. This may cause unexpected failure when running the built modules. Please check whether "mmdet" is a correct scope, or whether the registry is initialized.
/home/gf/.local/lib/python3.8/site-packages/mmengine/visualization/visualizer.py:196: UserWarning: Failed to add <class 'mmengine.visualization.vis_backend.LocalVisBackend'>, please provide the save_dir argument.
warnings.warn(f'Failed to add {vis_backend.class}, '
/home/gf/.local/lib/python3.8/site-packages/torch/functional.py:507: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at
../aten/src/ATen/native/TensorShape.cpp:3549.)
return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined]
Inference ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Traceback (most recent call last):
File "demo/image_demo.py", line 192, in
main()
File "demo/image_demo.py", line 184, in main
inferencer(**call_args)
File "/home/gf/mmdetection-main/mmdet/apis/det_inferencer.py", line 403, in call
preds = self.forward(data, **forward_kwargs)
File "/home/gf/.local/lib/python3.8/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/gf/.local/lib/python3.8/site-packages/mmengine/infer/infer.py", line 296, in forward
return self.model.test_step(inputs)
File "/home/gf/.local/lib/python3.8/site-packages/mmengine/model/base_model/base_model.py", line 145, in test_step
return self._run_forward(data, mode='predict')  # type: ignore
File "/home/gf/.local/lib/python3.8/site-packages/mmengine/model/base_model/base_model.py", line 361, in _run_forward
results = self(**data, mode=mode)
File "/home/gf/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/home/gf/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
return forward_call(*args, **kwargs)
File "/home/gf/mmdetection-main/mmdet/models/detectors/base.py", line 94, in forward
return self.predict(inputs, data_samples)
File "/home/gf/mmdetection-main/mmdet/models/detectors/single_stage.py", line 110, in predict
results_list = self.bbox_head.predict(
File "/home/gf/mmdetection-main/mmdet/models/dense_heads/base_dense_head.py", line 197, in predict
predictions = self.predict_by_feat(
File "/home/gf/mmdetection-main/mmdet/models/dense_heads/base_dense_head.py", line 279, in predict_by_feat
results = self._predict_by_feat_single(
File "/home/gf/mmdetection-main/mmdet/models/dense_heads/base_dense_head.py", line 423, in _predict_by_feat_single
return self._bbox_post_process(
File "/home/gf/mmdetection-main/mmdet/models/dense_heads/base_dense_head.py", line 485, in bbox_post_process
det_bboxes, keep_idxs = batched_nms(bboxes, results.scores,
File "/home/gf/.local/lib/python3.8/site-packages/mmcv/ops/nms.py", line 319, in batched_nms
dets, keep = nms_op(boxes_for_nms[mask], scores[mask], **nms_cfg)
File "/home/gf/.local/lib/python3.8/site-packages/mmengine/utils/misc.py", line 395, in new_func
output = old_func(*args, **kwargs)
File "/home/gf/.local/lib/python3.8/site-packages/mmcv/ops/nms.py", line 127, in nms
inds = NMSop.apply(boxes, scores, iou_threshold, offset, score_threshold,
File "/home/gf/.local/lib/python3.8/site-packages/torch/autograd/function.py", line 553, in apply
return super().apply(*args, **kwargs)  # type: ignore[misc]
File "/home/gf/.local/lib/python3.8/site-packages/mmcv/ops/nms.py", line 27, in forward
inds = ext_module.nms(
RuntimeError: Unknown layout

i have same problem

I also encountered this issue, and i solved it by matching torch, cuda and mmcv version correctly.
first, you should check on https://mmcv.readthedocs.io/en/latest/get_started/installation.html to see torch, cuda dependency for mmcv

then,you shoud go to https://pytorch.org/get-started/previous-versions/, and then select a proper install command
for other steps, just follow the GET STARTED guide.

I also encountered this issue, and i solved it by matching torch, cuda and mmcv version correctly. first, you should check on https://mmcv.readthedocs.io/en/latest/get_started/installation.html to see torch, cuda dependency for mmcv  then,you shoud go to https://pytorch.org/get-started/previous-versions/, and then select a proper install command for other steps, just follow the GET STARTED guide.

It doesn't work at all，i tried torch2.2.0，torch2.3.0，torch2.4.0,,pip install mmcv==2.0.0rc4,,2.2.0 and 2.1.0,，，finally i compiled by source code,it works will when i run this command: python .dev_scripts/check_installation.py,,but it still remind me that registry error or save_dir or something wrong,,,,i tried mim with pip,with minconda,,,,,feel so tired...thinking about giving up...


StorageCache lock is slowing down RPC requests

Scenario
In frontier, we have the eth_getLogs with allows to query big range of blocks (we allow 2000 per request) for their ethereum logs.
In the case of Ethereum, it uses the client storage to retrieve the block 1 by 1 (see https://github.com/paritytech/frontier/blob/master/client/rpc/src/overrides/schema_v1_override.rs#L59)
When multiple request to eth_getLogs comes together, it uses up to 4 threads (or up to --rpc-http-threads).
The observation
(on a 32 cores computer)

No matter how many cpu and rpc-http-threads, we are always limited to a certain CPU load (2.5 in our case), so with 4 threads, it would use 4 CPUs @ 60% utilization each, but with 10 it would use 10 CPU @ 25% each.
Using parity-db boosted a bit the performance but there is still a bottleneck somewhere
Changing the --state-cache-size value has no impact

Those logs shows the behavior when requesting those blocks

The issue
When accessing the state_at for a given block, substrate relies on a StorageCache (https://github.com/paritytech/substrate/blob/master/client/db/src/lib.rs#L2311) which works using a RW lock (https://github.com/paritytech/substrate/blob/master/client/db/src/storage_cache.rs#L540).
This lock creates a bottleneck for all the threads trying to access the storage.
This is confirmed by the perf analysis of the request:

The solution

The possibility to remove the StorageCache for the node (in the case of RPC nodes it makes more sense), or to only have the StorageCache used for the last 10 blocks (the one mostly used) would work.
Disabling the StorageCache (by commenting the fn storage(... function logic to only do Ok(self.state.storage(key)?)) showed great improvement in performances, using up to 90% of the cpus for each of the 4 rpc threads used (there are still some other bottleneck preventing to scale to 100% but I'll investigate later)

If you have another good solution in mind, I'd be happy to discuss it

(cc @bkchr and @arkpar )

@crystalin Could you check if this commit improves things?
It's a stop gap until we come up with a better solution,

@arkpar , I did test it (cherry-pick over polkadot-v0.9.6) and it worked great too (similar results with when I disabled the cache logic). Splitting the read/write is a good strategy in that case


[DOC] Linkis architecture docs are not standardized and unified, some even missing

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Problem Description
For example:

The directory structure and document nameing are not unified;
The doc structure and chapters are not unified, some are too sketchy;
Some modules' architecture docs are missing, for example in MGS we modified Eureka's service discovery mechanism, but not documented.
CN:
https://github.com/apache/incubator-linkis-website/tree/master/i18n/zh-CN/docusaurus-plugin-content-docs/current/architecture
EN:
https://github.com/apache/incubator-linkis-website/tree/master/docs/architecture

Description
No response
Use case
No response
solutions
No response
Anything else
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

[WeOpen Star]I would like to help

Hello, I 'm interested in this issue. Can I try it?