How do I implement the fix for Trusted HTML assignment subtitle bug on YouTube?

The post I'm referring to is this one:
https://github.com/killergerbah/asbplayer/issues/482
I see that a way to solve this bug has been found, but I'm not the most tech savvy person and so I'm confused on how to implement this fix. How do I access the code for the asbplayer  web browser version (chromium) to change the line of code?

 The fix was already released in version 1.4.0. But this release version is not yet visible on the chrome extention store. So you have to manually add it. So download the prerelease version here: https://github.com/killergerbah/asbplayer/releases/tag/v1.4.0
(asbplayer-extension-1.4.0-chromium.zip)
And follow the steps written under Installation on Chromium-based browsers

Unfortunally i still get an error,  but a different one:

Yeah, after following those steps I'm also getting the same new error code.

As I wrote in my report, this problem is repeated with the new version of this plugin that I installed directly from GitHub. Yesterday, my friends and I tried to solve the problem by installing the new version of the plugin, but the error just changed a little, but did not disappear

The same thing, it doesn't work at all, whether it's the new version or the precious one. Will it be even solved?

Yup the 1.4.1 version has solved the issue for me. Thanks a ton!


Bug: Md-Tab-Nav-Bar active link no longer underlined.

Bug, feature request, or proposal:
Bug.
What is the expected behavior?
When a tab routLink is activated, it should display an underline. I would have also thought the text color should be 'white' against a dark background.
What is the current behavior?
No underline is displayed.
What are the steps to reproduce?
See plunker here: http://plnkr.co/edit/wW6TZJ8EiLvTz1FFXPvv?p=preview
What is the use-case or motivation for changing an existing behavior?
It used to work out of the box, I also tried to implement a custom underline previously using this CSS and this no longer works:
.md-tab-link {
  border-bottom-width: 5px;
}

md-toolbar nav {
  border-bottom: 0px;
  color: #FFFFFF !important;
}

.mat-tab-link {
  opacity: 1 !important;
}

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Mac OS X El Capitan
Google chrome
  "dependencies": {
    "@angular/animations": ">=4.0.0 <5.0.0",
    "@angular/common": ">=4.0.0 <5.0.0",
    "@angular/compiler": ">=4.0.0 <5.0.0",
    "@angular/core": ">=4.0.0 <5.0.0",
    "@angular/flex-layout": "v2.0.0-beta.8",
    "@angular/forms": ">=4.0.0 <5.0.0",
    "@angular/http": ">=4.0.0 <5.0.0",
    "@angular/material": "^2.0.0-beta.5",
    "@angular/platform-browser": ">=4.0.0 <5.0.0",
    "@angular/platform-browser-dynamic": ">=4.0.0 <5.0.0",
    "@angular/router": ">=4.0.0 <5.0.0",
    "@ngui/map": "^0.18.0",
    "angular2-jwt": "^0.2.3",
    "apollo-angular": "^0.13.0",
    "apollo-client": "^1.1.2",
    "chart.js": "^2.5.0",
    "core-js": "^2.4.1",
    "graphql": "^0.9.6",
    "graphql-tag": "^2.0.0",
    "graphql-tools": "^0.11.0",
    "ng2-charts": "^1.5.0",
    "rxjs": "^5.3.1",
    "zone.js": "^0.8.8"
  }

See this updated plunk: http://plnkr.co/edit/LSakjsgj0nd6x8mJzBAl?p=preview
A couple of things I noticed:

You were trying to match routerLinkActive against '/home' when the route was '/'
If you want to instead match against '/' and '/contact', you'll have to use exact match options

I know that doesn't solve all the color concerns you have, but should demo why you weren't seeing the link underlined.

@willshowell Thanks for the feedback, I've updated the plunker for it to be more explicit about the problem I'm trying to highlight.
Previously, even if I visited home/about the tab would show the home tab as active. Now this doesn't work. Hopefully the updated plunker illustrates both the color and underline issues more clearly.

Hmm so when the route is /about, are you expecting both the "home" (/) and "about" (/about) tabs to show an underline?  It seems nav-tabs are limited to only one active tab at a time.
If that's not the issue, then it could just be a matter of running a custom route matcher to pass into the tab-link's [active] attribute.
Finally, I'd consider opening a separate proposal about being able to invert the color scheme (for use on top of toolbars) as it seems to be unrelated to the underline/route matching issue.

What i did is used this css to display the underline when my toolbar color is set to color="primary":
nav .mat-ink-bar {   background-color: white; }
But my question is what is the correct approach for when you want to show a Md-Tab-Nav-Bar in a Md-Toolbar. I think if the  Md-Tab-Nav-Bar also had a color attribute that you could set to color="accent" it could work nice.

AFAICT everything is working as intended. The tab-nav doesn't support having multiple active tabs at once (intentional). If you have a more specific bug, feel free to open a new issue.


Example 'RFID-Cloner' prompts warning due to dash in name

Step 1: Describe your environment

OS version: Lubuntu newest version, 18.04 LTS (Bionic Beaver), with kernel: Linux winloos 4.15.0-23-generic #25-Ubuntu SMP Wed May 23 18:02:16 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Arduino IDE version: 2:1.0.5-dfsg2-4.1
MFRC522 Library version: master
Arduino device: Uno
MFRC522 device: N/A

Step 2: Describe the problem
When I successfully install this library, most of the examples are added to the File | Examples menu. Except for RFID-Cloner, of which a warning dialog appears, as it has a dash in its name. Every restart of the Arduino IDE, this warning re-appears.

Affected file(s) or example(s):

None

Steps to reproduce:

sudo apt install arduino
Download this library as a zip, this zip will be called rfid-master.zip
Unzip that zip. A folder rfid-master will be created
Rename that folder to RFID
In the Arduino IDE, click Import Library and select the RFID folder. The first warning about the dash in the name of the RFID-Cloner example appears
When restarting the Arduino ID, the warning appears

It can be fixed by simply renaming the folder RFID-Cloner to RFID_Cloner and its file RFID-Cloner.ino to RFID_Cloner.ino

I volunteer to do a Pull Request with the renamed example folder. Just assign this Issue to me :-)
Observed Results:

What happened?  This could be a description, log output, screenshot, etc.

I get a warning that RFID-Cloner is a name with a dash.
Expected Results:

What did you expect to happen?

That I can start the Arduino IDE without warnings
Relevant Code:
  // TODO(you): code here to reproduce the problem
  // No code is needed

Hi,
your IDE version is 1.0.5 right?
This version has several problems (one you mentioned), so the minimum IDE version was set to 1.6.
It is mentioned in the readme under Compatible IDE.

This library works with Arduino IDE 1.6, older versions are not supported and will cause compiler errors.

Best

I fully agree it is well documented.
I suggest (and volunteer) to do a simple rename, so that older IDE versions don't have this warning. I feel the costs outweigh the benefits, but of course, I do have less understanding of the consequences of such a rename.

It would not compile anyway, because version 1.0.X has no enabled c++11 support.

Yes, you are right.
The goal of documenting it is to prevent other people wasting their time in finding this out.


Gela/extent

Description

Type of change

[ ] Bug fix
[x] New sample implementation
[ ] Sample viewer enhancement
[ ] Other enhancement

Platforms tested on:

[x] Windows
[ ] Android
[ ] Linux
[ ] macOS
[ ] iOS

Checklist

[ ] Runs and compiles on all active platforms
[x] Branch is up to date with the latest main/v.next
[x] All merge conflicts have been resolved
[x] Self-review of changes
[x] There are no warnings related to changes
[x] No unrelated changes have been made to any other code or project files
[x] Code is commented with correct formatting (CTRL+i)
[x] All variable and method names are camel case
[x] There is no leftover commented code
[x] Screenshots are correct size and display in description tab (500px by 500px, platform agnostic)
[x] If adding a new sample, it is added to the sample viewer

@tanneryould Not sure why the checker is failing, the primary qml snippet is listed last in the metadata file as expected.
Please review.


Return to exit priority and exit id calculation schema from RootChain

We should return to the exit priority calculation schema that was used in RootChain contract, as we should favor the solutions that do not change the API,  there is also no apparent reason to stay with the current schema, moreover the current schema costs more gas.
RootChain schema
Exit priority used to be computed with the following schema:
|255                     214|213     160|159                      0|
|---------------------------|-----------|--------------------------|
|    exitable timestamp     | tx pos    |         exit id          |

Current schema
|255                     192|191              64|63    0|
|---------------------------|-------------------|-------|
|    exitable timestamp     |    tx position    | nonce |

The changes of the exit priority does not affect the priority itself, as the exitable timestamp is still the most significant field, but such the new schema changes the API (the old priority differs from the new one givent the same transaction) and the current schema enforces to introduce a mapping from exit priority to exit id, in order to process the exit.
The field nonce is stored in the ExitGameController contract and is updated each enqueuing - which increases the overall cost of startExit transaction.
priority -> Exit mapping in ExitGameController
Having changed the exit priority, we can extract exit id directly from the priority, therefore the mapping (uint256 => Exit) public exits; can be changed as follows: mapping (uint256 => IExitProcessor) public exits, which is also more gas efficient.

Exit priority used to be computed with the following schema:

I think your graph is wrong compare to the code:
return ((getExitableTimestamp(_utxoPos) << 214) | (tx_pos << 160)) | _exitId;

It doesn't have full space for exitId previously actually.
From the code document:
 *   Anatomy of returned value, most significant bits first
     *   42 bits - timestamp (exitable_at); unix timestamp fits into 32 bits
     *   54 bits - blknum * 10^9 + txindex; to represent all utxo for 10 years we need only 54 bits
     *   8 bits - oindex; set to zero for in-flight tx
     *   1 bit - in-flight flag
     *   151 bit - tx hash

we'd need to re-desing for the oindex part as we've bumped the size.

As for the issue of using exitId in processEixts that you've mentioned in slack is separate from this. It is still doable with current mechanism if with want.

I think your graph is wrong compare to the code

Yes, you are right - I made it from my memory - which is apparently poor.

Having analyzed both exit priority and exit id structure from RootChain, we came to a conclusion that there is not enough place for outputs in exit id.
Exit id consists of (in order of most significant bits):

8-bits oindex
1-bit IFE flag
150-bits of tx hash.

oindex has only 8 bits, but it is a value from range 0-10,000; which means that it should have at least 14-bits.
In order to fix this issue, we will add the required 6 bits to oindex by moving ife flag to right and subtracting the 6-bits from tx hash.
The new exit id schema will be as follows:

14-bits oindex
1 bit IFE flag
144-bits tx hash.

Exit id length
While going deeper into the issue, we saw that we use uint192 to store exit id, while we are actually using and are prepared for 160-bits only.
We decided to tighten the type of exit id to 160 bits only.
The previous type might have been deceiving and error-prone in the future.

@pik694, I peaked into your commit, I am actually lost why we can use only 8 bits for output index and bring back the limitation of 255?
But if that's just a WIP thing, I will wait until the PR then.

We have 256-bits of exit priority, which consists of 3 major parts: exitable timestamp, tx position and exit id.
Now, the lengths of the first two fields are roughly calculated and should be:

exitable timestamp - 42 bits (actually more than we need);
tx pos - 54 bits.

https://github.com/omisego/plasma-contracts/blob/82b4ea7327106837defeb7fb58ad10503ba6c57f/plasma_framework/contracts/src/framework/utils/ExitPriority.sol#L13
The above means that we are left with only 160 bits; but we are free to use each and every bit, if only we guarantee that there will be no collisions between exits.
As the issue is meant to revert to RootChain exit priority and exit id schemas, I took the exit id schema from the previous contract - 8 bits of oindex, 1-bit ife flag, 151-bits tx hash.
As you can see we have only 8 bits for oindex, which means that if we create a tx with at least 256, we will have a collision between the 0th and 256th output.
I must have tightened  the length of exit id as otherwise the 256th output would overwrite tx position.
Please, note that this limitation is simply limitation of the exit id, we are now using. In the future we can provide a new exit id schema in order to increase the limitation.

I may have changed the exit id schema to be compatible with 10,000-outputs transactions. The roughly schema following:

14-bits oindex;
1-bit ife flag;
145-bits tx hash.

I discussed the topic with @pdobacz and we decided to keep it simple for now and use the schema we used to use.

In the future we can provide a new exit id schema in order to increase the limitation.

This is actually something that's not that great using this design. All new exit id schema need to take old existing exit id schema into consideration whether they are gonna collide or not, which adds extra design concern and complexity. But anyway, probably not a huge concern.

I may have changed the exit id schema to be compatible with 10,000-outputs transactions

I just come up with a better (?) idea, I think the following would be nice for SE.
- 1-bit ife flag
- 159 bit left-most 159 bits of outputId

This only drop a bit from the ethereum assumption of address being 160 bits plus outputId is something baked into PlasmaFramework already.

@boolafish Yours is another option, but what is an outputId in case of an IFE?

Just....keep IFE exitId as it is? It is just a hash of txBytes plus a flag now, so probably just
1-bit ife flag
159-bits left-most 159 bits of hash(txBytes)

Please note that nowadays exitId includes oindex, which in case of IFE is not necessary (zeroed), as we process IFEs as a whole tx. In case of a SE, each output from a tx is an another exit, which means that using a tx hash is to little to tell which output is to exit.

Which shows that there is no generic solution that may work out of the box. We can probably move the discussion to an another issue, as I believe that this topic probably is out of the scope of this issue.


Changes to support OAuth1RequestToken deserialisation for SessionStoreCookie

@mmoayyed this patch adds twitter (and oauth1 request token in general) support to the DelegatedSessionCookieManager.  It also makes the SessionStoreCookieSerialiser an overridable @Bean.
Note that the Jackson mixin is currently stored in the @Configuration class for want of a better place to put it, happy to takes suggestions of a more appropriate place for it.

Coverage decreased (-0.04%) to 58.349% when pulling e55c85f1744f76fa9439be6299aaa2e0da264ee3 on sbearcsiro:oauth1-req-token-serialiser into 1a49dd2e0d5b813a49077b4eba9888464b2fc0eb on apereo:master.

This is perfect; Thank you. I do wonder if the mixin can be moved to one of the more core pac4j modules, but it's fine as is today. Lets get this merged once the build turns up green.

BTW, is there an oppurtunity here to write a few test cases preventing this from happening again? It doesn't have to be too comprehensive; even if it's just testing this exact case, it's a good foundation to keep adding more later on and increase coverage (currently at 58% and growing)


braintree-web (3.1.0) breaks JHipster production build

Symptom
After adding braintree-web to a JHipster 3.6.1 project (via bower), the production build no longer works. Compiles, builds, starts w/o errors. Blank screen after hitting main page.
Reproduction
Install the latest versions of Oracle Java 8, Git, Node.js, Yeoman, Bower, Gulp and JHipster:
See https://jhipster.github.io/installation/ for details.
Set up a new application from scratch:
$ mkdir myapp && cd myapp
$ yo jhipster

Confirm defaults.
Set up a production database or simply copy DB settings from dev profile
to src/main/resources/config/application-prod.yml. E.g.
datasource:
    type: com.zaxxer.hikari.HikariDataSource
    url: jdbc:h2:file:./target/h2db/db/myapp;DB_CLOSE_DELAY=-1
    name:
    username: myapp
    password:

Build and start production package:
$ ./mvnw clean package -B -Pprod -DskipTests=true
$ java -jar target/myapp-0.0.1-SNAPSHOT.war

Open http://<IP_ADDRESS>:8080 in Browser. You should see the JHipster welcome
page.
Now add the braintree-web module (I tested 3.1.0) to the project, build
for production and start server:
$ bower install braintree-web --save
$ gulp inject
$ ./mvnw clean package -B -Pprod -DskipTests=true
$ java -jar target/myapp-0.0.1-SNAPSHOT.war

You should now see a rather blank screen.
These error messages appear in the console:

index.js: Uncaught ReferenceError: require is not defined(anonymous function) @ index.js:2
app.module.js: Uncaught ReferenceError: angular is not defined(anonymous function) @ app.module.js:4(anonymous function) @ app.module.js:23

I was not able to pinpoint the cause (as I am fairly new to the build system and Braintree) but I suspect that the gulp configuration is not compatible with braintree-web. As this lib is the only one I encountered problems with, I reported the error here.

This is a duplicate of #194
See my answer here: https://github.com/braintree/braintree-web/issues/194#issuecomment-240422087
In short, the index.js file should not be used with bower. You should instead use the individual files you need. Docs here: https://developers.braintreepayments.com/guides/client-sdk/setup/javascript/v3#install-with-bower
Since this has come up before, we'll discuss as a team if we should do anything different for our Bower release to make integration with Bower easier. Might be as simple as adding that documentation to the README file of this repo.

Thanks for the quick response. It solves my problem.
I'd strongly opt for supporting gulp-inject as this is a real timer saver (when it works...).

Maybe I'm misunderstanding how gulp-inject works, but I don't see why it wouldn't work as is. For instance, if you were building a hosted fields integration:
var gulp = require('gulp');
var inject = require('gulp-inject');

gulp.task('index', function () {
  var target = gulp.src('./src/index.html');
  var sources = gulp.src([
    './path/to/your/apps/files',
    './path/to/bower_components/braintree-web/client.js',
    './path/to/bower_components/braintree-web/hosted-fields.js',
  ], {read: false});

  return target.pipe(inject(sources))
    .pipe(gulp.dest('./src'));
});

You wouldn't want to inject all of braintree-web's js files, because that would increase the time it takes your page to load with code that you would never use.

I use gulp-inject only as part of the JHipster workflow.
Which would be something like
$ bower install braintree-web --save
$ gulp inject

This works for all my 3rd party dependencies. It would be cool if braintree-web supported this, too.
I created a feature request #211 for this.


Android Development Best Coding Practices

@raghunath-r-a can I work on this issue?

@raghunath-r-a Can I work on it ??


Question: how to disable (bypass) data checks in cache?

I see that CPU often checks for instruction data in icache even when it's disabled by SWERV configuration (icache_enable=0). This affects overall CPU memory access performance.
Please, suggest if it's possible to bypass icache.

The core is designed to have ICACHE , ICCM or both. It is not optimized for the case that Icache and ICCM are both not enabled. It will function but not efficiently. In other words , cycles are not taken out when Icache is not enabled.

check for RV_ICACHE_ENABLE define in common_defines.vh if this defined the IC logic will present in design. if you want CPU without IC, you need to use -unset icache_enable switch with swerv.config
(-set icache_enable=0, has no effect) ...


Hello

1st REPO this year!! worth celebrating

Haven't done any appreciable coding in a while, I was beginning to feel the arthritis in my hands.

You have one in the works. LFG!


Migrate field data conversion utilites and EOL software

[ ] Mooring Analysis
[ ] At Sea Analysis
[ ] Tools (like CF conversions and EPIC conversions)
[ ] Common Analysis
[ ] Data Archiving

CF and EPIC tools can likely be dropped (a CF to epic routine will probably need to exist to give PJS datastreams she's familiar with... so maybe create an "archive" category?

EPIC <-> CF Time is a needed tool... for historic records and for PJS


[docs] Improve color selector demo

[x] I have followed (at least) the PR section of the contributing guide.

Closes #23909

@oliviertassinari Looks nicer but how about having also the color bar extended?

Never mind, on mobile might not look good.

@oliviertassinari may I commit this version?

I can't see the difference, feel free to polish it.

Sorry, but I can't support this pull request. The code sample is no longer accurate.
This:
{
  palette: {
    primary: {
      main: blue[500],
    },
    secondary: {
      main: pink[A400],
    },
  },
}

is not the same as this:
{
  palette: {
    primary: blue,
    secondary: pink,
  },
}

The layout is also a step backwards. It's currently imperfect (the original PR for this had a nicer layout, but it got "simplified" by an overzealous reviewer, so we have what we have). If you want to make the grid 2 x 8, that's fine, but please adjust the breakpoints so that it stays that way at all screen sizes.

The code sample is no longer accurate.

@mbrookes Ah great point! We also need to update the implementation of the demo to match the displayed code snippet.

The layout is also a step backwards.

The motivation for moving the demo on a second row is to allow the body to be fully visible without a scrollbar. It also to fix the display on smaller screens:

But I agree, the current format is not great.

@mbrookes Ah great point! We also need to update the implementation of the demo to match the displayed code snippet.

Right, users' won't know at a sight that #2196f3 is equivalent to blue[500]
https://github.com/mui-org/material-ui/pull/23926#issuecomment-742546154

Also, there's a bug in which the color picker disappears when inputting the HEX color manually

@oliviertassinari

@mbrookes Ah great point! We also need to update the implementation of the demo to match the displayed code snippet.

Right, users' won't know at a sight that #2196f3 is equivalent to blue[500]
#23926 (comment)

I may be missing your point 🤔

It also to fix the display on smaller screens:

What do you consider broken on smaller screens? The original layout seems to flow nicely to me.

The motivation for moving the demo on a second row is to allow the body to be fully visible without a scrollbar.

Which body? By adding another row you're increasing vertical space not reducing it.
I don't have any scrollbars with 50% larger font-sizes.

I may be missing your point thinking

If we manually input an arbitrary HEX value in the textbox then the code will still say primary: blue[500]. It seems to me we already know if the input color matches a pre-defined color (see check-marks in the color swatch) and we need to use that to adjust the generated code.

@eps1lon cc: @oliviertassinari

What do you consider broken on smaller screens? The original layout seems to flow nicely to me.

Right, it looks fine to me too (smaller width, turns to two rows like the original layout)

The layout is also a step backwards.

The motivation for moving the demo on a second row is to allow the body to be fully visible without a scrollbar. It also to fix the display on smaller screens:

But I agree, the proposal version is not awesome, we could do better.

Agree with @oliviertassinari
This is how the last committed version looks like.

how about adding this breakpoint?
it will maintain the original layout and at the same time fixed the broken layout shown in this comment https://github.com/mui-org/material-ui/pull/23926#issuecomment-742726252
diff with next branch
@@ -210,7 +210,7 @@ function ColorTool(props) {
     const color = state[`${intent}`];

     return (
-      <Grid item xs={12} sm={6} md={4}>
+      <Grid item xs={12} md={4} sm={6}>
@@ -273,11 +275,11 @@ function ColorTool(props) {
   };

   return (
-    <Grid container spacing={5} className={classes.root}>
+    <Grid container spacing={4} className={classes.root}>
       {colorPicker('primary')}
       {colorPicker('secondary')}
-      <Grid item xs={12} sm={6} md={4}>
-        <ColorDemo data={state} />
+      <Grid item xs={12} md={4} sm={12}>
+        <ColorDemo data={state} shades={shades} />
       </Grid>

Also maybe better to leave the color square back to square. If stretching it to the container width, the version I've added at https://github.com/mui-org/material-ui/pull/23926/commits/9d86643976c9950ded874a902a344a7c9d16a185 will look constrained in some screen sizes

docs/src/pages/customization/color/ColorTool.js
@@ -71,11 +71,11 @@ const styles = (theme) => ({
    marginTop: theme.spacing(2),
  },
  colorSquare: {
+    width: 64,
    height: 64,
    display: 'flex',
    justifyContent: 'center',
    alignItems: 'center',
-    flexGrow: 4
  },

@yukinoda No specific preference. We could always reduce a bit the ambition of the changes if we can't find a clear improvement direction. From what I understand, there are, at least two changes that everybody finds valuable:

Reduce the number of spaces for the tab, 2 instead of 4. It helps with the limited space to display the codesnippet
Avoid using a color as is, prefer specifying main: blue[500].

@oliviertassinari

@yukinoda No specific preferences. We could always reduce a bit the ambition of the changes if we can't find a clear improvement direction. From what I understand, there are, at least two changes that everybody finds valuable:

yeah... too many changes might conduct to more changes

Reduce the number of spaces for the tab, 2 instead of 4. It helps with the limited space to display the code snippet.
Avoid using a color HUE as is, prefer specifying main: blue[500].

May I keep working on this branch? Only do the above two changes (remove 2 x 8 color picker view and breakpoint change)

As for the layout itself, it's another discussion. I personally struggle to envision something great with the constraints we have (limited space) and without changing too many things.

Got it.

May I keep working on this branch?

👍 from my end

We also need to update the implementation of the demo to match the displayed code snippet.

The other way around – the changes to the displayed code need to be reverted. It was correct before this PR.

prefer specifying main: blue[500].

No. As mentioned, a color object is preferred so that light and dark are taken from the palette, rather than using a calculated approximation.

As mentioned, a color object is preferred so that light and dark are taken from the palette, rather than using a calculated approximation.

@mbrookes What do you think about moving away from this pattern? It seems harmful. I would propose we remove all mention of it in v5 and remove it in v6. I believe that this initial mapping was done for the very first version of the Material Design specification. Why I'm leaning toward this direction:

Material Design has moved away from this initial mapping of colors https://material.io/design/color/the-color-system.html#color-theme-creation.
I don't understand the value of this API, it seems simpler and more efficient to compute the other colors. It's what we do for most of them and it works great.
The dark & light variants are barely used. Instead, we use a variety of color transformation. Considering the objective to support CSS variables, moving all the color transformation to the theme creation, it seems a lot simpler to kill this option.

@yukinoda I'm closing, at this point, I think that the problem was great to chat about fundamental issues, however, it seems that there are no simpler solutions.
If you want to reopen to change spacing 4 to 2, go ahead :).

The initial color palette is labeled "2014", not a great signal grin.

That is, in fact, a great signal. Stability is a feature, not a bug.

Material Design has moved away from this initial mapping of colors

And yet:
The Material Design palette tool or 2014 Material Design palettes are available to help you select colors.
https://material.io/design/color/the-color-system.html#color-usage-and-palettes
Used in the video under "Distinguish UI elements"
And in the image labelled "This UI uses a primary color and two primary variants."
And in the video under "Dark and light secondary variants"
And in the image labelled "This UI uses a color theme with a primary color, a primary variant, and a secondary color."
And in the section labelled "Color swatches"

I don't understand the value of this API, it seems simpler and more efficient to compute the other colors.

The palette colors don't follow a linear transform of saturation or lightness. If they could be computed, we wouldn't need objects for them. If we were to do anything, it would be to remove support for calculating light and dark from main, and require those to be provided explicitly, if not providing a color object. (That said, the Material Design color tool comes close, it would be interesting to try and figure out what approach it uses).
When you need to create a lighter or darker variation of a color in your palette, don't get clever using CSS preprocessor functions like "lighten" or "darken" to create shades on the fly
https://refactoringui.com/previews/building-your-color-palette/

We will need to have more keys, with, maybe 100, 200, 300, etc

This supports the notion that passing in a color object for primary and secondary is correct, we just need to add the entire thing to the theme, rather than only values for light and dark.

@mbrookes Thanks for sharing the blog post! The key parts my attention stayed on:

"You can't build anything with five hex codes".
"When you need to create a lighter or darker variation of a color in your palette, don't get clever using CSS preprocessor functions like "lighten" or "darken" to create shades on the fly. That's how you end up with 35 slightly different blues that all look the same."
"The darkest shade of a color is usually reserved for text"
"The lightest shade might be used to tint the background of an element"
"you can't rely purely on math to craft the perfect color palette"

It's an interesting perspective. I think that it perfectly surfaces the current usage of the theme in the components. In the current state of things, it seems that we mainly do the following:

We take .main, we apply transformation, opacity, fade, darken, lighten. There is little consistency in which arguments are used.
We ignore .light and .dark most of the time.

This supports the notion that passing in a color object for primary and secondary is correct, we just need to add the entire thing to the theme, rather than only values for light and dark.

Yeah, for me the biggest uncertainty are: 1. will we be able to use the 2014 color palette by default, 2. will it be simple and common enough once we refactor the palette to handle CSS variables?
In the meantime, I think that we can benchmark, using Refactoring UI examples, button primary & alert info.

Primer. They have a lot of CSS variables.

Alert background-color: $blue-100
Alert border-color: rgba($blue-700, 0.2)
Alert color: $text-gray-dark
Button background-color: custom

Evergreen.  has a lot of colors: https://evergreen.segment.com/components/colors.

Alert border-color: success.intent
Button background-color: custom

Bootstrap. they try to make customizing the colors as easy as possible, you can provide a single hex and they will compute the rest

Alert background-color: shift($info, -80%)
Alert border-color: shift($info, -70%)
Alert border-color: shift($info, +40%)
Button background-color: $primary

Chakra-UI.

Alert background-color: $info.100
Alert color: inherit
Button background-color: $primary.500

The next step could be to audit the usage of colors in the components and see opportunities for sharing identical values.


[Autocomplete-Select] Keep all data in options, and allow to retrieve them in onChange

Current behaviour
The autocomplete select component only keep label and value properties in its options:
edit.js#L153: data.forEach(item => {                 options.set(item[keyName], item[labelName]);             });
And only the key is given to the onChange:
edit.js#L210: onChange(key)
As others data properties are only in the store used by the querySearcher, the view has to directly get them here if needed.
Expected behaviour
Keep all data in options: this.setState({options: data}), or keep them duplicated in data: this.setState({data})
And allow to retrieve them in onChange: onChange(key, option)
Versions
Focus-core: 2.1.1
Focus-components: 2.1.7-1

@rodolpheV I won't add a second argument on onChange function.
For input, onChange should take only the value on entry.
Eventually, we could have an option to get the whole value instead of the key.
But before that, for your problem, I don't see any use case. Could you elaborate a bit more ?

Then I'll explain my use of this component:
I have a component that uses the AutocompleteSelect to get a unit:
<AutocompleteSelect
   querySearcher={this.searchUnit}
   onChange={this.onUnitOptionClick}
   keyName='unitId'
   labelName='value'
/>

The method onUnitOptionClick passed as onChange to the AutocompleteSelect:
onUnitOptionClick(unitId) {
    const storeUnitList = planStore.searchUnit();
    const unit = storeUnitList.find(u => u.unitId === unitId);
    return this.addUnitToSelection(unit);
},

And I would like to do:
onUnitOptionClick(unitId, unit) {
    return this.addUnitToSelection(unit);
},

I guess we can add on option wholeValue, and in onChange, return the whole value instead of the id in this case.

@c3dr0x I let you do that one :)


Plot infographic plots for transect and verification plots

What problem does your feature request solve?
Plots such as transect require a user to enter a start lat/lon, end lat/lon of model data to create a transect. It would be helpful if a static (non time varying plot) was created which shows where this transect is on a map (a map projection showing the transect as a line with some padding).
There might be similar uses for these kind of plots, such as in verification tasks to plot the locations of station data being used to calculate profiles, for example.
Loosely defined issue for now, but think there will be other use cases.
Describe the solution you'd like

Describe alternatives you've considered

This sounds useful, as not everyone (such as myself), can intuit lat-lon pairs, especially when transecting between them. Do you have any examples of this being done, such as some example plots you like?
I do wonder if this would be better being in the description, which would require adding some more data types to that, as currently is is just markdown. It would be good to utilise the space however.


FeatureRequest: .soundcloudqueue adding single song via URL

Hello,
Is there a possibility to enqueue just a single song specifying URL?
Thanks

.scq


Create new release with new implementation

Would it be possible to create a new release at commit 6d0fbc769d4bf9489459f26165b9b75ac912f923 ?

Hey @cphyc :). Do you mean like a major version change? If so, what are the benefits of doing so?

That or a tag! I'm using the project as a bower dependency so it's always nicer to have a tag/version number instead of a SHA1.

Okay, I've drafted a new release. Let me know if that helps. This is my first time handling versions and releases on github, so if I do anything weird feel free to call me out :).

Thanks!


Tenant-Type: Production-Scale Tenant

On my Tenant overview there's a property called "Tenant-Type" and the value is "Production-Scale Tenant".
Is there a way to create others types of tenants, like "Testing" or "Development" where there's no SLAs but the MAUs wouldn't be charged?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: fc5e148d-b7cf-92e7-4cc7-a65d20d037b6
Version Independent ID: 2ba6e31b-5899-f35a-0f08-d280110ded13
Content: Frequently asked questions (FAQ) for Azure Active Directory B2C
Content Source: articles/active-directory-b2c/faq.yml
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@Leonardo-Ferreira Thank you for your query . We will investigate and update the thread.

There is only one type of tenant "Production-Scale Tenant". This tenant type property is used a while ago for review tenant. So, it's still there for backward compatibility/support.

#please-close


lifetime: monologue app should not update the skill stack

Version: latest
Platform: all
Subsystem: lifetime

Current we update the skill stack always even though some application is in monologue, and this should be fixed at v7.28.x :)

Monopolization assertion was made here https://github.com/yodaos-project/yodart/blob/master/runtime/lib/app-runtime.js#L428, before cloud stack updaing here
https://github.com/yodaos-project/yodart/blob/master/runtime/lib/app-runtime.js#L450
Do you have any cases that caused the bug?

Can't reproduce any more, just closing this.


fix: improve logging + remove catchup

Catchup was wrong, changes can not be processed in batch.
I have changed the behavior during the typescript migration but now I can delete it and just leave the watch.

Add better logging to understand what is going on

the actual issue is with replicate

Currently nothing, before yes.
It was batching changes, but in a single batch there could be the same package, like multiple update or a create/delete.
Processing them in the wrong order could lead to corrupted data, so in the end the safest way was to process 1:1 which is the same as running the watch.

makes perfect sense!


Crash and Create contraption & Occultism storage deleted! (server)

Minecraft 1.16.5
Forge 36.0.13
Create v0.3e
Occultism 1.0.9
This is on a server. The server crashed. Upon reconnecting the player noticed their Create mining operation was completely deleted and as well as the connected Occultism setup. The player then recounts what is missing here:
"In the picture there should have been about 30 drills with equal number of chassis and a couple of chests that were attached to a pulley. Before the crash I was standing on the setup moving ore from the chest into my occult storage system."

NOTE: This bug report has also been filed on the Create Github issues listing here: https://github.com/Creators-of-Create/Create/issues/1056
Also, here is the crash report: https://pastebin.com/YGJPuy1V
EDIT: Player also commented: "If it helps I was using the storage actuator at the time. Not a stable wormhole, or the remote thing."

@VallenFrostweaver thank you for the report.
Based on the crash report, something was removing blocks from the world (I am not familiar with create, but the log suggests LinearActuatorTileEntity, PulleyTileEntity being involved). It seems in this process the tile entity for the storage actuator was deleted before the block could drop it's items, which caused the crash.
I can prevent said crash by adding a check if the tile entity still exists when the block drops, however the removal of the blocks happens outside my code (presumably by something in create that moves/destroys blocks) so half of the puzzle remains


Broken Symlink in<EMAIL_ADDRESS>npm package

There is a broken symlink after installing<EMAIL_ADDRESS>
The symlink is created in a preinstall, but there is no react directory in ./lib
"preinstall": "ln -s ./lib/react ./react"

Reported by a community member in the forum: https://forum.openzeppelin.com/t/broken-symlink-in-openzeppelin-network-0-4-0-npm-package/2309
Workaround is to uninstall version 0.4.0 and install the previous version 0.3.0:
npm i<EMAIL_ADDRESS>

For some reason TypeScript hasn't compiled React folder into latest build. Fixed now. Starter Kit is updated as well!


QUIC: Default add RVCM to Envoy QUIC client sessions.

Signed-off-by: Renjie Tang<EMAIL_ADDRESS>Commit Message: Default add connection option RVCM to client QUIC connections.
Additional Description: This will truly enable active port migration implemented at https://github.com/envoyproxy/envoy/pull/18103
Risk Level: Low
Testing: Unit tests
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.
Fixes #16311

/assign @danzh2010

Need a main sync to for the tsan failure?

/retest

/assign @RyanTheOptimist

/assign @alyssawilk


Problem with sending Solder data to TechnicPlatform

It is a error I had before Solder 7.2.0 and before.
My TechnicPlatform page for the modpack always shows "Application Error".
@sct told me that the reason is a problem with Solder sending incomplete data:
http://solder.dreadful-adventures.de/api/modpack/radioactive-fortress/1.1.1?include=mods
I mad a new version of the pack as I thought it will fix the problem, but it is still there:
http://solder.dreadful-adventures.de/api/modpack/radioactive-fortress/1.1.2?include=mods
Hope someone can help me!

Now fixed, it was an error in the nginx setup...


obfuscate emails & validate ORCID

[ ] Email address needs to be hidden (high priority)
[ ] ORCID field needs to be limited to 16 characters (without dashes).

Email address needs to be hidden

To give a chance to the user to review the email, at first the email can now be obfuscated/deobfuscated using the new commands encode_email and decode_email

ORCID field needs to be limited to 16 characters

Validation has been added. And to comply with the Expressing the ORCID iD section of the specification, the requirement for dashes has been maintained.
See https://support.orcid.org/hc/en-us/articles/360006897674-Structure-of-the-ORCID-Identifier

Fixed in https://github.com/MorphoCloud/MorphoCloudWorkflow/compare/7fee216...5ef9460


How do you know what window the Metal renderpass descriptor callback is for?

There's nothing passed in to the renderpass descriptor callback. How can it know what to do? It should surely get passed in a void * of the caller's choosing, as presumably supplied as part of the sg_desc.
I'll have a go at making a PR for this, but I reserve the right to soon find out that I've misunderstood what's going on...
Thanks,
--Tom

OK, I think I see - this is #229, and it sounds like it might be more involved than I expect. So I'm not going to do anything about this.
Thanks,
--Tom

Hi Tom, sorry for not responding in time, I was focusing on emulator stuff over the holidays and deliberately ignored sokol stuff ;)
As you already found out, sokol_gfx.h needs "a bit" more work to really work with multiple output windows across all backends, and sokol_app.h would require quite a lot of work.


Contrib docs

Let's set up the right framework for contributors to succeed.

Per @cheftako - for each person's PRs, we will require approval from a different employer than their own.

/lgtm

/lgtm
/approve

Discussed in meeting with @cheftako @justinsb @a-hilaly


Resize Recyclerview to show all items

I have been using this excellent library, but for life of me I can figure out following.
My RecyclerView is in NestedScrolleView,
I have set setNestedScrollingEnabled to false and setHasFixedSize to true for the recyclerview.
When I add RecyclerView to LinearLayout, I setAutoMeasurable to true for that LinearLayout
However when items are expanded my recycler view doesn't resize to show all items within the adapter. All the elements have attribute layout height wrap_parent
What do I have to do to get RecyclerView to resize to show all items upon expand/collapse events?

Not too familiar with having recyclerviews nested in other scroll views, however it seems like setting setHasFixedSize to true goes against with what you are doing. The documentation for setHasFixedSize says:

RecyclerView's size is not affected by the adapter contents. RecyclerView can still change its size based on other factors (e.g. its parent's size) but this size calculation cannot depend on the size of its children or contents of its adapter (except the number of items in the adapter).
If your use of RecyclerView falls into this category, set this to true. It will allow RecyclerView to avoid invalidating the whole layout when its adapter contents change.

When you expand a parent, this is adding the child items to the adapter. Because of this attribute being set to true it seems the recyclerview does not try to resize itself and therefore doesn't adapt to the change.
Let me know if I understood you correctly!

Yup that was it as soon as I set setHasFixedSize to false - RecyclerView started to expand / collapse as expected. Thank you so much for "prodding" me to look at the documentation again!

Were you able to finally do this ? I am struggling with the same thing...

@swapnilgt it was 2 years ago? They have propably changed their job, and their lifestyle. By now they are most likely vegan and against computers! Come on man!
Anyway I can help you what's up?

@curtismoxam that was two years ago people helped each other a lot more then. Or maybe because it was so long ago they probably don't even have the same job, they probably have a new lifestyle a new car they are probably vegan and don't like computers! Come on man!
Anyway I can help if need be whats up?

@nicparmee i need help i recyclerview doesnt show all items there are 41 but it shows 23 :/

@moudin sorry for the slow reply.
Can you be a bit more specific does it scroll at all?
But for now, try to put the recycleview into a relativelayout with layout_height as wrap content.


development mode issue

Can it be used in development mode with vite? I am running into issues where I am not receiving response from my node.js file to my capacitor app while in development mode also I am using capacitor 6

Yes, everything should work. Does the problem only exist in developer mode or always? And have you already looked at the examples? (The examples use slightly older packages, but it should be possible to simply update them without breaking anything)

//CAPACITOR CONFIG:
{
"appId": "com.beatsy.musicapp",
"appName": "Beatsy",
"bundledWebRuntime": false,
"npmClient": "npm",
"webDir": "www",
"plugins": {
"SplashScreen": {
"launchShowDuration": 0
},"CapacitorNodeJS": {
"nodeDir": "/nodejs"
}
},
"server": {
"url": "http://<IP_ADDRESS>:5173/",
"cleartext": true
},
"cordova": {}
}
//VITE CONFIG
import path from 'path';
import react from '@vitejs/plugin-react';
const SRC_DIR = path.resolve(__dirname, './src');
const PUBLIC_DIR = path.resolve(__dirname, './static');
const BUILD_DIR = path.resolve(__dirname, './www',);
export default async () => {
return  {
plugins: [
react(),
],
root: SRC_DIR,
base: '',
publicDir: PUBLIC_DIR,
build: {
  outDir: BUILD_DIR,
  assetsInlineLimit: 0,
  emptyOutDir: true,
  rollupOptions: {
    treeshake: false,
  },
},
resolve: {
  alias: {
    '@': SRC_DIR,
  },
},
server: {
  host: true,
},

};
}
// static/nodejs/main.js
const { channel } = require('bridge');
// Listens to "msg-from-capacitor" from the Capacitor layer.
channel.addListener('msg-from-capacitor', message => {
console.log('[Node.js] Message from Capacitor: ' + message);
// Sends a message back to the Capacitor layer.
channel.send('msg-from-nodejs', Replying to the message '${message}'., 'And optionally add more arguments.');
});
//CLIENT
import { Page } from "framework7-react"
import { NodeJS } from 'capacitor-nodejs';
import { useEffect, useState } from 'react';
const Downloads = () => {
const [message, setMessage] = useState('Waiting for message...');
useEffect(() => {
// Listen for messages from Node.js
NodeJS.addListener('msg-from-nodejs', event => {
setMessage(event.args[0]); // Store first argument in state
});
// Initialize communication
NodeJS.whenReady().then(() => {
  NodeJS.send({
    eventName: 'msg-from-capacitor',
    args: ['Hello from Downloads component!'],
  });
});

return () => NodeJS.removeAllListeners();

}, []);
return (

Message from Node.js:
{message}

)
}
export default Downloads;
I just see waiting for message in my android studio app

I want to get live changes while running my app live using 'npm run dev'
can I do that?

In the node/main.js from where does the bridge module gets imported? im confused a bit,
here is a breakdown of what I did,
I installed the capacitor/nodejs library
they edited my viteconfig file
followed the steps then
I edited the capacitor config file,
then I did npx cap sync,
then npx cap copy,
then as you can see for live changes I have added the vite development server url inside the capacitor config
then i created a communication system between the server and client
then opened the android project on android studio
then tried to receive message from the nodejs server
BUT it didn't work
So my question is If have to build my react project everytime using npm run build then it would be really hard for development purpose
I'm so confused

ohh okay now I get it, this is kinda hard for live development purpose. Is there a way to create a nodejs server directly from the /android directory on some localhost:PORT and access it from my capacitor based react app?


Howdy on Flatpak or Snap

So the story began when I was trying to get Howdy to work on Fedora Silverblue. It installed fine with layering, but I just couldn't make configurations, since it's a read-only filesystem.
Long story short, is there any difficulties to making a Flatpak or Snap package of Howdy? It would make it available to a lot more users who do not have access to the Ubuntu PPA or the AUR. I've seen feature requests on here wanting Howdy on Solus, Void, Gentoo, Clear Linux, etc. As a Fedora user, I've also had to switch 3 COPRs because of dependency issues.
Thank you all!

I currently have no plans to make one, but i welcome anyone who wants to try their hand in making one

I will try creating a snap package, let's see how it goes.

Howdy is work as a pam module so it is inpossible to bring it to snap/flatpak

CLI programs published by snaps can be "executed" by any user (maybe an asterisk or two here) - it might be possible to snap 95% - and just have to provide the pam hook externally - I'm going to have to have a play.
I'm here because I panicked when I saw the PPA was install pip packages as root (this is a way to end up with a broken base-system).  snaps would provide a better solution to that issue.
Perhaps there is another solution to that issue that doesn't involve snaps though,....

Let me add my voice to those clamoring for a Clear Linux port ... or, even better, a Flatpak! As far as I know, the Flatpak framework is somewhat more flexible than Snap, as there is a way to adjust app permissions, which may not exist in Snap.

Any update/progress?


Manually entering values on Range input is broken

I can reproduce this with consistency on the online demo
Basically, when in Range mode, if you try and type in a date it will not work correctly.

Enter 2/2/2013 into the start date field (start date field is formatted to 02/02/2013)
TAB into the End date field
Enter 3/3/2013 into the End date field
Notice that the start date field now shows 03/03/2

It seems to happen as soon as you add the first digit of the year in the end date.
Additionally, if you try and manually delete the date string (in the start, or end date field), as soon as the year is removed, the date field automatically gets populated with today's date.
Neither of the above issues are present in a regular, singular datepicker, only the range.

Agreed this is an issue for us too.  Workaround at the moment is to specify a startDate of 01/01/1900 (or as appropriate).
http://bootstrap-datepicker.readthedocs.org/en/latest/options.html#startdate

+1. This is still an annoying issue, the workaround isn't that bad, but a fix would still be awesome.

feel free to provide an PR ;)

This seems to be a duplicate of #721

thanks, closing that one.


⚠ SceneTime has degraded performance

In 23abf78, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2458 ms

Resolved: SceneTime performance has improved in 7815fc0 after 12 minutes.


Feature/sds mtls

Issues associated with this PR
Your PR should present related issues you want to solve.
Sign the CLA
Make sure you have signed the CLA
Solutions

support sds mode in mtls

UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases.
And you need to show the ut result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #742 into stable will increase coverage by 1.22%.
The diff coverage is 71.09%.

@@            Coverage Diff             @@
##           stable     #742      +/-   ##
==========================================
+ Coverage   50.29%   51.51%   +1.22%
==========================================
  Files         198      202       +4
  Lines       16939    16976      +37
==========================================
+ Hits         8519     8745     +226
+ Misses       7536     7333     -203
- Partials      884      898      +14

Impacted Files
Coverage Δ

pkg/types/upstream.go
59.37% <ø> (ø)
:arrow_up:

pkg/types/network.go
0% <ø> (ø)
:arrow_up:

pkg/network/connection.go
23.96% <0%> (-0.66%)
:arrow_down:

pkg/xds/conv/convertxds.go
26.73% <0%> (ø)
:arrow_up:

pkg/stream/sofarpc/connpool.go
1.76% <0%> (-0.09%)
:arrow_down:

pkg/types/tls.go
0% <0%> (ø)

pkg/stream/sofarpc/keepalive.go
88.23% <0%> (-5.52%)
:arrow_down:

pkg/stream/http/connpool.go
2.11% <0%> (-0.05%)
:arrow_down:

pkg/api/v2/types.go
71.42% <0%> (-0.94%)
:arrow_down:

pkg/upstream/cluster/host.go
61.9% <100%> (+1.24%)
:arrow_up:

... and 20 more

Continue to review full report at Codecov.

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


bedtools bam2bed race condition

Hey @jinyongyoo @zanglab,
I hope you are having a great day! I just wanted to start this off by saying thank you for creating and maintaining this awesome tool.
Background
I recently noticed an issue while running SICER2. While processing a large batch of samples on our cluster, I noticed that sets of samples with the sample input control were failing.
Here is the error that was received while running SICER2:
Error: Input BED files must have the first six fields. Check input.bed to see if it has the following fields: chrom, chromStart, chromEnd, name, score, and strand.

After checking the bed file for issues, I could not find any problems with it. It had all six fields. What is interesting is that one job with the same input completed successfully. One thing that I noticed is that both jobs started running around the same time. Our cluster uses a network-attached filesystem so multiple concurrent jobs will have access to the same files.
At first, I was confused as to what was happening because we weren't providing a BAM file as input, but then I took a look at what was happening under-the-hood. I see starting here there is some logic to convert a given bam file to bed format:

The issue is that the bed file is not being written to the output directory. It will be written to the same directory as the input bam file. This causes an issue if two treatment samples share the same input control and the jobs run at the same time. This can create a race condition where two processes are trying to write to the same file, causing a possible deadlock.
Possible solution
Would it be possible to write the converted BED file to the output directory? That way a user can create a unique output directory for a ChIP sample to avoid this issue. I can submit a PR to fix this if you want.
Please let me know what you think.
Best regards,
Skyler Kuhn

Hi Skyler,
Thanks for the detailed issue! Yes, you are correct. We will discuss it and see what is the best solution on that. We will update a new version of the package shortly and are happy to include your suggestions.
Best,
Shiyu


Support for multiple orientations.

The context menu is not rendered properly when we change the device (iPad/iPhone) orientation. It always renders the context menu considering the orientation as normal Portrait in all the orientations due to which the context menu is wrongly rendered. It works fine in Portrait orientation but fails to give the same experience in other orientations. We would appreciate if you provide orientation support and help us by fixing this issue on priority. Thank you!

hi any updates on this?


ODC ans Sonatype CSS index results varies

@parimala-kathir See the CPEs listed in the report...
your aar file is identified by DependencyCheck as being
Axis1 1.6.2 as well as
Axis2 version 1.6.2
Whether any of those is a valid CPE for the version-1.6.2.aar file....
That is something only you can tell
BTW dependency check uses more than just OSSIndex - vulnerabilities can also come from NIST NVD.

NIST NVD is listing Apache Axis2 up to (including) 1.6.2 as vulnerable to the CVE
https://nvd.nist.gov/vuln/detail/CVE-2012-5785

The reason that OSSINDEX is not showing any vulnerabilities for the version-1.6.2.aar is because they're a large company that can do detailed linking of CVEs to the individual components of a large framework like Apache Axis2 and the issue of the CVE is in another part of the Axis2 framework.
This project takes the approach to spent the limited resources where they are best spent and therefor does not build the large dataset required to be able to link at the individual subcomponents to the CVEs raised against a framework. Therefor the CVEs of Apache Axis2 framework found in NIST NVD are raised for every component that is part of the framework.


Fix dependency issues for Ruby < 2.2.2

Restrict Rack version for Ruby < 2.2.2
Restrict with_env version for Ruby 1.9.3

Merged upstream


Remove External Dependencies

There are couple of dependencies we can remove by Material Design which can reduce the apk size  -
compile 'com.cocosw:bottomsheet:1.3.0@aar'
compile 'com.nineoldandroids:library:2.4.0'
I will soon be sending PR.

Closed via #210


Use 'latest' Tag for Claude 3.5 Sonnet and Claude 3 Opus Models

Issue

X
Change

Updated the model versioning to allow using the 'latest' tag to automatically get the most recent version. This feature is currently available for Claude 3.5 Sonnet and Claude 3 Opus models for now.
General checklist

[X] There are no breaking changes
[ ] I have added unit and integration tests for my change
[X] I have manually run all the unit and integration tests in the module I have added/changed, and they are all green
[X] I have manually run all the unit and integration tests in the core and main modules, and they are all green

[ ] I have added/updated the documentation
[ ] I have added an example in the examples repo (only for "big" features)
[ ] I have added/updated Spring Boot starter(s) (if applicable)

Checklist for adding new maven module

[ ] I have added my new module in the root pom.xml and langchain4j-bom/pom.xml

Checklist for adding new embedding store integration

[ ] I have added a {NameOfIntegration}EmbeddingStoreIT that extends from either EmbeddingStoreIT or EmbeddingStoreWithFilteringIT
[ ] I have added a {NameOfIntegration}EmbeddingStoreRemovalIT that extends from EmbeddingStoreWithRemovalIT

Checklist for changing existing embedding store integration

[ ] I have manually verified that the {NameOfIntegration}EmbeddingStore works correctly with the data persisted using the latest released version of LangChain4j

My bad, that's not recommended in their documentation (only for DEV purpose) :

For convenience during development and testing, we offer ”-latest” aliases for our models (e.g., claude-3-5-sonnet-latest). These aliases automatically point to the most recent snapshot of a given model. While useful for experimentation, we recommend using specific model versions (e.g., claude-3-5-sonnet-20241022) in production applications to ensure consistent behavior. When we release new model snapshots, we’ll migrate the -latest alias to point to the new version (typically within a week of the new release). The -latest alias is subject to the same rate limits and pricing as the underlying model version it references.


Stricter URL matching rules

@match rules path components aren't strict enough to match MB paths in detail. It'd be nice to have some stricter matching of URLs to ensure the scripts only run on the pages that they're supposed to run on.
Rather than predefining the rules in some constants, like we did in #451, it'd be nice to have some sort of generator to which we could pass some options, and the generator then constructs all the @include and @exclude rules we need. That'd allow us to more declaratively (and clearly) specify on which pages these should run (and possibly also automated the testing thereof, #75). Mock up:
includeRuleGenerator({
    edits: true,  // /edit/\d+ pages
    entityEdits: true,  // <entity>/edits, <entity>/open_edits, etc.
    editSearch: true,  // search/edits?...
     }
});

It might be complete overkill though, but hey, MB's URLs are complicated and userscript include rules are too.
What would be useful is a list of possible MB URLs, is there anything like that somewhere around?
Ref: https://github.com/ROpdebee/mb-userscripts/pull/473#discussion_r895151411
Related to #75.

Another reason why a more declarative specification would be interesting would be to allow others to easily get an answer to "On which pages does this thing actually run?", because regex patterns won't answer that question easily.


Travis failing due to missing dependency

Travis is failing to build Finatra due to a missing dependency.
Expected behavior
Travis should be able to resolve all dependencies and build
the project successfully.
Actual behavior
Travis fails while building the project with:
sbt.ResolveException: unresolved dependency: silly#silly-thrift;0.5.0: not found
See: https://travis-ci.org/twitter/finatra/builds/174309040
Steps to reproduce the behavior
Create a pull request (See https://github.com/twitter/finatra/pull/362)

Thanks. @cacoco had just fixed this locally. You can safely remove it in your branch if you need to be unblocked.

This is now resolved.


#170 - added better error checking to the file I/O calls in the LogPr…

…oviderFile::writeLine() method

Coverage decreased (-0.01%) to 70.161% when pulling 89393ee34b2b76a76ef440e4968ddeec57bd9987 on bug/170-fails-silently-if-it-cannot-write-to-log into 0cff81b03a9419c082d67a0c5a3d9be32a571b93 on develop.


add support for XML attributes in SOAP request bodies

Resolves #73 and #38.

Hi @rubysolo
Thanks for your contribution ❤


services/horizon: Use COPY to speed up ClaimableBalanceChangeProcessor

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

What
Use FastBatchInsertBuilder for writing to claimable_balances and claimable_balance_claimants tables. FastBatchInsertBuilder uses ‘COPY' to insert into the db which is faster than using 'INSERT'.
Why
#5086
Known limitations
[TODO or N/A]

As shown in the Ingestion Processor Run Duration graph below, deploying the build with COPY instead of insert to staging significantly reduced the processing time required by the ClaimableBalanceChangeProcessor for the same network activity in claimable balances.


When endpoint creation fails, resources are not cleaned up properly

When you try to create an endpoint and it fails (e.g. due to invalid JSON in $ENDPOINT_RESOURCES), the resources are not properly cleaned up. The next endpoint creation fails with
{"title": "An error occurred during endpoint creation: Conflict"}
Logs from mgt:
INFO:management_api.authenticate.authenticate:Request path: /tenants/default-tenant/endpoints, method POST
ERROR:management_api.utils.errors_handling:Unexpected error occurred: ('Invalid JSON', 'Could not parse JSON body - Extra data: line 1 column 169 (char 168)')
Request: <Request: POST 'https://mgt.imm.net/tenants/default-tenant/endpoints'>  Params: {'tenant_name': 'default-tenant'}
ERROR:management_api.utils.errors_handling:Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/falcon/media/json.py", line 15, in deserialize
    return json.loads(raw.decode('utf-8'))
  File "/usr/lib/python3.7/json/__init__.py", line 348, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.7/json/decoder.py", line 340, in decode
    raise JSONDecodeError("Extra data", s, end)
json.decoder.JSONDecodeError: Extra data: line 1 column 169 (char 168)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/falcon/api.py", line 244, in __call__
    responder(req, resp, **params)
  File "/usr/lib/python3.7/site-packages/falcon/media/validators/jsonschema.py", line 45, in wrapper
    jsonschema.validate(req.media, schema, format_checker=jsonschema.FormatChecker())
  File "/usr/lib/python3.7/site-packages/falcon/request.py", line 965, in media
    self._media = handler.deserialize(raw)
  File "/usr/lib/python3.7/site-packages/falcon/media/json.py", line 19, in deserialize
    'Could not parse JSON body - {0}'.format(err)
falcon.errors.HTTPBadRequest: ('Invalid JSON', 'Could not parse JSON body - Extra data: line 1 column 169 (char 168)')

INFO:management_api.authenticate.authenticate:Checking token: eyJhbGciOiJSUzI1NiIsImtpZCI6ImI2YTU0ZWQzMzdjYjY5N2M4YWFmMmZiMjFhMDYzYTExMjZlMWYwNDkifQ.eyJpc3MiOiJodHRwczovL2RleC5pbW0ubmV0OjQ0My9kZXgiLCJzdWIiOiJDaU5qYmoxMWMyVnlMRzkxUFZCbGIzQnNaU3hrWXoxbGVHRnRjR3hsTEdSalBXOXlaeElFYkdSaGNBIiwiYXVkIjoiZXhhbXBsZS1hcHAiLCJleHAiOjE1NTM2ODg3NDcsImlhdCI6MTU1MzY4NTE0NywiYXRfaGFzaCI6IkpLdlhkdGkyZGtoV0ltV0hWcXRJQ3ciLCJlbWFpbCI6InVzZXJAZXhhbXBsZS5jb20iLCJlbWFpbF92ZXJpZmllZCI6dHJ1ZSwiZ3JvdXBzIjpbImFkbWluIl19.wQ69oWwUFdrdrUnLH2z8yCDSQ62OisreWqmdqLEToFm70hSMQl8NPO-Krmn549f7BJOata8Q60VupKuYmYgZOfRAiXVUSSpokp8cfcr5RBUrpiw-rwdrk8aBKH5P_l_gq1JA74AnUb_fvzWUvICaEUFL1nK9kooC8U1Q01hrAGo-K1-wpRtSfZnvqzU0EnDV8xZr7wdmINmY0dpIv3tGWkxS3M2J5WxnM-_d8qw3-Spl_QHXKwHbuqWHLNxpt6nlAcXW9Sv690JwY0_aWpAvRpbH6hgJpj5Hx3rJ_L9tlzUSAVhj5yrS6SyqSy8ITx2lXUkXncW_6_oTnbaGiRiuOQ
INFO:management_api.authenticate.authenticate:Trying key number : 0
INFO:management_api.authenticate.authenticate:Decoded
INFO:management_api.authenticate.authenticate:Using service account token
INFO:management_api.authenticate.authenticate:Decoded token : {'iss': 'https://dex.imm.net:443/dex', 'sub': 'CiNjbj11c2VyLG91PVBlb3BsZSxkYz1leGFtcGxlLGRjPW9yZxIEbGRhcA', 'aud': 'example-app', 'exp':<PHONE_NUMBER>, 'iat':<PHONE_NUMBER>, 'at_hash': 'JKvXdti2dkhWImWHVqtICw', 'email'<EMAIL_ADDRESS>'email_verified': True, 'groups': ['admin']}
INFO:management_api.authenticate.authenticate:Request path: /tenants/default-tenant/endpoints, method POST
INFO:botocore.vendored.requests.packages.urllib3.connectionpool:Resetting dropped connection: minio.imm.net
INFO:management_api.tenants.tenants_utils:Tenant already exists: True
ERROR:management_api.utils.errors_handling:An error occurred during endpoint creation: (409)
Reason: Conflict
HTTP response headers: HTTPHeaderDict({'Content-Type': 'application/json', 'Date': 'Wed, 27 Mar 2019 12:03:19 GMT', 'Content-Length': '264'})
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"inference-endpoints.ai.intel.com \"endpoint-1\" already exists","reason":"AlreadyExists","details":{"name":"endpoint-1","group":"ai.intel.com","kind":"inference-endpoints"},"code":409}

We try to reproduce this problem, but it works for us.
@poussa are you sure that endpoint-1 was not created earlier on this platform?
We also checked in code, that if JSON is wrong, no resources are created. So there is nothing to be cleaned.

Closing, because we could not reproduce it. Feel free to reopen if it occurs again


local variable 'videoname' referenced before assignment

After exec python3 generate_dataset.py train :
[INFO] Loading data list ... Done!
[INFO] 200 movies are used in train mode
[INFO] Start downloading 200 movies
[INFO] Downloading https://www.youtube.com/watch?v=KeXr_qWVbiU
Traceback (most recent call last):
File "generate_dataset.py", line 171, in
isOK = downloader.Run()
File "generate_dataset.py", line 139, in Run
process(data, 0, videoname, self.output_root)
UnboundLocalError: local variable 'videoname' referenced before assignment

Hi, I encountered the same issue. Did you solve this?

Refer to
https://github.com/cashiwamochi/RealEstate10K_Downloader/pull/9/commits/0512d22c20259609fea4c48f16965bc319ea3a80


Add return type to DummyTestSplFileInfo::getRealPath()

PHP 8.1 emits a deprecation warning if that method does not declare a return type. So I've added it.

Coverage remained the same at 92.853% when pulling 03d61aa6c68cdca27e0dcdf986e7a1802ef4e0d4 on derrabus:bugfix/get-real-path-deprecation into 07430f5aca4874476bf175317cfb6284cd80421a on FriendsOfPHP:master.

Thank you @derrabus.


Add the domain of the University of prince Mugren

@OmarRa15 I am sorry, the domain upm.edu.sa is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


SCC agent: strengthen user and fsgroup

Reading/writing from/into CRIO socket is admitted by DAC (via groupID) and by MAC(thanks to privileged container). Therefore no need to let the agent image set any uid or fsgroup.

/retest

/test e2e-gcp

"e2e-gcp" post steps failed: "e2e-gcp" pod "e2e-gcp-gather-gcp-console" failed: the pod ci-op-mwx9xry6/e2e-gcp-gather-gcp-console was deleted without completing after 1m10s

/retest

/label docs-approved
/label px-approved
Updates to stricken scc permissions attached to the agent. Docs and px review not required.

/lgtm

/assign @sunilcio

/label qe-approved


Fix minor typos in sone methods

This file had a small typo where instead of key, it was kye. This PR fixes this without any other modifications.

hey @isfedorov - since this is my first contribution apparently there's a need for an approval.
This is just a minor typo fix so it could be a quick-win :)

These aren't typos: https://github.com/php-gnupg/php-gnupg/blob/master/gnupg.c#L252 :)
You might want to submit a PR into the extension repository instead.


Recover funds

My raspiblitz system doesn't want to start lnd anymore after rebooting. It looks like it the harddrive is corrupted. How can I recover the funds? Some funds are in channels and some in the wallet.

Hello jetlagv1,
Rootzoll write some helpfull informations about this state of the raspiblitz.
https://github.com/rootzoll/raspiblitz#recover-your-coins-from-a-failing-raspiblitz
hope that helps you!

Hi CryptoSigmunt. Thank you for posting the link but it doesn't seem to take me anywhere. I'm not that familiar with github and I'm using it on my phone right now but the link only takes me to the open issues page. Can you point me a little further?

Then it's a browser issue... it should bring you to the bottom of the first page where Rootzoll discribes how to recover funds.
Headline is "Recover your Coins from a failing RaspiBlitz" ;)

"It looks like it the hard drive is corrupted." why do you think this?  You should post debug log output so we can see the errors. Try the terminal command
sudo journalctl -f -u lnd

I'm thinking there is nothing specific about Raspiblitz with respect to wallets and keys.  It's running Bitcoin Core under the hood. So you might get better results searching and posting this question on Bitcoin core discussion sites. I don't know the answer off the top of my head.  See: https://bitcoin.org/en/community

Seed is not working on other bip39 Wallets like electrum

so I recovered funds on new setup today with lnd folder from previous setup on a new setup. When I check funds directly via raspiblitz menu I am getting this error but when I connected via zapwallet reckless can access all funds and all channels were recovered. So if someone needs help if not finding funds after recovery just connect via zapwallet reckless and it should show up. If you need any help ping me.
./00mainMenu.sh: line 419: lnbalance: command not found
Press ENTER to return to main menu.

The ./00mainMenu.sh: line 419: lnbalance: command not found error is not critical an will be fixed in next release.
@walletofsatoshi can you describe more in detail on how you managed the recover with "zapwallet reckless" - is that a special version of zap wallet?

No I am connecting zap wallet over internet . I and ran few commands from terminal for command not found from earlier similar issue which fixed it.
The balance was not showing in raspiblitz since it was giving error with command not found issue but was showing on my zapwallet.

@walletofsatoshi ah ok got it, thx.


fix for issue#172- property of OrderViewModel.cs lowercase

Updated OrderViewModel.cs properties names as per C# convention.
Issue - (https://github.com/dotnet/eShop/issues/172)

Updated OrderViewModel.cs properties names as per C# convention. Issue - (#172)

@microsoft-github-policy-service rerun

@microsoft-github-policy-service agree

@microsoft-github-policy-service rerun


Model and table structure mismatch?

For example:
// TrackedTime represents a time that was spent for a specific issue.
type TrackedTime struct {
	...
	Created     time.Time        `xorm:"-"`
	CreatedUnix int64            `xorm:"created"`
}

In database (MySQL / SQLite):
CREATE TABLE `tracked_time` (
	...
	`created_unix` bigint DEFAULT NULL,
);

IIRC CreatedUnix int64            `xorm:"created"` means that the column name should be created, is it an undocumented/documented behavior, or bug, or I missed something?

Learnt from discussion:
created is a XORM keyword, not a column name.
So the column name is CreatedUnix => created_unix


Update Http3Frame.cc - fix Use-of-uninitialized-value error

Fuzzing reveals a use-of-uninitialized-pointer-field case
https://oss-fuzz.com/testcase-detail/6159793621368832
I think this should fix the problem

[approve ci autest]

Cherry-picked to v10.0.x


Provide solution files for the example instances: small_example and large_example

It would be awesome to test against solutions for the small_example.json and large_example.json .

This is a great idea. However I just made those examples up. Will likely have to change the instances a bit in order to provide optimal solutions. Thanks for the issue report!


subarray of length two

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Given an array A[ ] of N elements. In one operation, you can select any subarray of length 2 and multiply all
elements of that subarray by -1. You can do any number of operations (possibly zero). Your task is to maximize
the sum of all elements of all subarrays of A[ ].
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
issue #3664

Do review @HarshCasper


Uses FactoryGirl in HostOpenstackInfra spec

This changes usage of HostServiceGroupOpenstack.create to
FactoryGirl.create in spec for HostOpenstackInfra.

:+1: Thanks!


Update Configuration.cs

Fix dapr should use http instead of https.
#2119

I saw there is already a pull request contains the problem. Closed.


test comment

Added test comment

Excellent comment, however, we do not need this.

Yes, I agree


Feature request: Optional auto start hook

Would be great to be able to provide a function to an auto started tour that takes an introjs instance as a parameter. That would open up the use of the whole intro.js API on auto started tours. The function could be provided a Component method.

implemeted


can I  call or use python library with flutter

I  was wondering if I can call or use some python library with flutter to handle business logic .
In other word I want flutter to handle the UI and for python fir business logic without using http request to the server

Currently you would need to make it available on your native platform (Java, Kotlin, Objective-C, Swift) and from there make it available to Flutter using https://flutter.io/docs/development/packages-and-plugins/developing-packages
https://github.com/flutter/flutter/issues/7053 might help as well, but I don't know if Python apps can expose its API to make it usable that way.

Can we call/use flutter engine directly by other c-serie languages(python etc.)?

@Shu-Ji sorry, can't help you here. I'd suggest you ask on StackOverflow. You might get a quicker response there.

is there an active support for integrating python with flutter, compatible with Dart2?

You cannot run python on mobile devices (it requires an interpreter), nor on the web. Its possible that a flutter for desktop application could callout to python in a plugin, but by design that is not a concern of the flutter framework

http://www.srplab.info/flutter/2018/07/29/flutter-and-python.html


Quality Control

Description
Improved logging to catch potential heisenbugs and efficiency checks surrounding state commits.
Issues:
closes #37
closes #36
Resolutions

[x] Refactoring unittests to improve test encapsulation
[x] Coverage exclude rules
[x] Improve logging in state module
[x] Improve state commit efficiency by checking asynchronous event queue.

Codecov Report

Merging #39 (039ac5f) into master (372f63e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #39   +/-   ##
=======================================
  Coverage   99.49%   99.50%
=======================================
  Files           7        7
  Lines         197      200    +3
  Branches       28       27    -1
=======================================
+ Hits          196      199    +3
- Misses          0        1    +1
+ Partials        1        0    -1

Impacted Files
Coverage Δ

myosin/models/base.py
100.00% <ø> (+3.03%)
:arrow_up:

myosin/state/state.py
100.00% <100.00%> (ø)

myosin/utils/funcs.py
75.00% <0.00%> (-25.00%)
:arrow_down:

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 372f63e...039ac5f. Read the comment docs.


summarization lab

1.3 description was a bit confusing for people in terms of the part about the census tracks - maybe could just say what was the largest population? - they got confused about the single row phrasing

also maybe good to add a hint to 1.4 about having unique column names for the summarize table - some people were using the same name and therefore only seeing one column in the output

also maybe good to add a hint to 1.4 about having unique column names for the summarize table - some people were using the same name and therefore only seeing one column in the output (and reminding them that it is whatever they want to call it)

although it was a cool lesson - so maybe could add a question about trying it without unique names  and have them observe what happens and why

[ ] Typo in 2.3 - should be CaliforniaCounty not county


Bring back fontbakery-fix-vertical-metrics.py

It seems there are tools from the last 0.0.15 release still missing :(
I needed to use fontbakery-fix-vertical-metrics.py and found it missing, but I'd like to ensure all tools are available.
https://github.com/googlefonts/fontbakery/blob/7145ce54594fb8b8de8f65e94bae5be76bdf07e3/tools/fontbakery-fix-vertical-metrics.py

old script was here:
https://github.com/googlefonts/fontbakery/blob/v0.0.15/tools/fontbakery-fix-vertical-metrics.py
it gives you command line params for setting specific values explicitely. I can quickly bring it back to master as is.
Aditionally it has got an autofix block of code that depends on the VFixer class. I'll have to look at this class more carefully later.

VMet and VMetFixer classes are here:
https://github.com/googlefonts/fontbakery/blob/v0.0.15/bakery_cli/fixers.py#L691-L739

Hey Felipe,
I'm currently in the middle of writing a vertical metrics checker for Glyphsapp. This is based on 2 new  schemes we discussed in Kalapi's thread. It should be loose enough to incoporate into FB. I'll let you know when I upload.
Cheers,
Marc

nice :-)

Sure, the logic is duplicated, but I still need the old tools that work
'surgically' on a few issues! :)

Reopening this.
Had a request from Edd Harrington at Colophon. He used to use the autofix option, https://github.com/googlefonts/fontbakery/blob/master/bin/fontbakery-fix-vertical-metrics.py#L256-L257

Thanks Marc. I haven't been checking this but yes, I want all the old tools
to be available. See any others missing?
On Aug 4, 2017 9:47 AM, "Marc Foley"<EMAIL_ADDRESS>wrote:

Reopening this.
Had a request from Edd Harrington at Colophon. He used to use the autofix
option, https://github.com/googlefonts/fontbakery/blob/
master/bin/fontbakery-fix-vertical-metrics.py#L256-L257
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontbakery/issues/897#issuecomment-320253223,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP9y_lUZvXOLZZITVw_0xMzKBKFKuthks5sUyDUgaJpZM4JZ_MS
.

All scripts are now guaranteed to be available.
I'll keep this issue open since it still requires bringing back the --autofix argument to the fix-vertical font-metrics script.

(1) View vmetrics data:

(2) Run the vmetrics fontbakery test (com.google.fonts/test/040) and notice that the data has bad values as reported by all the FAILs:

(3) Then run the fix-vertical-metrics subcommand with --autofix
(4) And re-run test/040 to make sure we get all green PASS results :-D

Hey, @m4rc1e ! Could you please ping Edd Harrington at Colophon and tell him that we got the --autofix option back so that he can use it again ?
cheers!

I've done so
On Oct 18, 2017 3:52 PM, "Felipe Corrêa da Silva Sanches" <
<EMAIL_ADDRESS>wrote:

Hey, @m4rc1e https://github.com/m4rc1e ! Could you please ping Edd
Harrington at Colophon and tell him that we got the --autofix option back
so that he can use it again ?
cheers!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontbakery/issues/897#issuecomment-337708529,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP9y1yvfcimYez3Q_OETjXzrwBiQP7tks5stlcagaJpZM4JZ_MS
.

Thanks, @davelab6 !


Added minimal Jenkins CI support to the packager.

This currently only works automatically with jenkins 2.0 pipeline builds since jenkins just started setting the BRANCH_NAME env var.  The variable name used to be different depending on which version control was used.

I've published 0.2.14 to pypi with this feature.


Bottom sheet not opening from DialogFragment

Hi,
First of all thanks for the awesome library.
I'm facing two issues.
1.) BottomSheet is not opening at all. This is my code : https://gist.github.com/ShivamDev31/6526827e6fe531870310279ce5d41a19
I've a dialog and 2 radio buttons, on click on one of them I'm trying to open BottomSheet but its not opening at all, When I changed the type From BottomSheetFragment to DialogFragment then it showed a dialog.
Please have a look at the code and let me know if I'm doing anything wrong, followed the sample properly.
2.) Other issue is BottomSheetLayout is the root layout of dialog_booking and its not wrapping it height, I tried everything possible but its opening full screen. Even I hardcoded its height even then its opening full screen.

Hi @ShivamDev31. I'm happy this library is useful for you!
This issue tracker is for issues with the library, not for help debugging your own project. Your best bet for that is probably StackOverflow, or a similar website. Your first question doesn't seem to be a bug in the library, but since I looked at it: I think the first issue might be because of this line. It looks like the bottomsheet fragment is being added to the Activity instead of the fragment that has the BottomSheetLayout. Try getChildFragmentManager() instead of getActivity().getSupportFragmentManager(). Your sample code is incomplete, so I'm guessing here.
For your second point: BottomSheetLayout extends FrameLayout, and it doesn't override any of the layout code, so I don't think your bug is in this library. Dialog sizes can be complicated, so it might be better to avoid trying to put a lot of layout code in a dialog. Either way, I think you're better off on StackOverflow for this question too.


New version: Alibaba.AlibabaCloudCLI version 3.0.196

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335187 2024-01-04 01:44:58.402 [FAIL] Installer failed security check. Url: https://github.com/aliyun/aliyun-cli/releases/download/v3.0.196/aliyun-cli-windows-3.0.196-amd64.zip Result: 0x80004005

(Automated response - build 516.)

Hi @SpecterShell,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 516.)

/AzurePipelines run

Automatic Validation ended with:

Installation failed with exit code -1978335187 2024-01-05 11:34:18.602 [FAIL] Installer failed security check. Url: https://github.com/aliyun/aliyun-cli/releases/download/v3.0.196/aliyun-cli-windows-3.0.196-amd64.zip Result: 0x80004005

(Automated response - build 530.)

Hi @SpecterShell,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 530.)


how to increase the max memory  and sga ?

my server's memory is 16G,but the oracle container only use 1.5G memory,
i can't increase the memory ,and i'm very depressed.
thanks

No, it's not possible with Oracle 11g "Express" which only allows 1g memory, 11g storage and 1 core cpu.

oh,shit,fuck oracle.
thanks for your answer.


feat: Allows parallel benchmarks

Previously it was not possible due to cache interference.
Note that I have kept til cache to the base cache dir for datasets and metrics. Though it might be cleaner to nest them into their own folders so instead of e.g. having .scandeval_cache/f1 you have ``.scandeval_cache/metrics/f1`.
Fixes #59
Edit: The tests fail, but it seems like it is an issue with the CI?

Also, as you might have seen, I've opened a PR into your branch which fixes the CI bug:
https://github.com/KennethEnevoldsen/ScandEval/pull/1

Perfect - I have merged it in. Everything should work as intended now.


Add name selector to apstra_datacenter_system data source

This PR introduces the name attribute to the apstra_datacenter_system data source:
data "apstra_datacenter_system" "foo" {
  blueprint_id = "1c0c455f-6029-4696-9cda-5e7fde1d7ff6"
# id = "N_n5hK0BcEQc5KFfHoE"
  name = "apstra_esi_001_sys001"
}

Previously only id could be used to select a system node.

closes #166


fix: skip number in native c

This fixed #507

Codecov Report

Merging #520 (a029fe3) into main (6c2a723) will decrease coverage by 0.58%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #520      +/-   ##
==========================================
- Coverage   73.79%   73.22%   -0.58%
==========================================
  Files          63       58       -5
  Lines        8288     8021     -267
==========================================
- Hits         6116     5873     -243
+ Misses       1847     1836      -11
+ Partials      325      312      -13

Files Changed
Coverage Δ

internal/native/avx/native_amd64.go
95.65% <ø> (ø)

internal/native/avx2/native_amd64.go
95.65% <ø> (ø)

internal/native/sse/native_amd64.go
95.65% <ø> (ø)

... and 5 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


chore(main): release 0.1.32

:robot: I have created a release beep boop
0.1.32 (2023-12-04)
Features

Connections: import and re-export connection classes in openhexa.sdk (#93) (2e3db63)
delete pipeline (#91) (549f73d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/BLSQ/openhexa-sdk-python/releases/tag/0.1.32 :sunflower:


added message object to context

Checklist
This pull request...

[ ] Fixes a bug
[x] Adds new functionality
[ ] Updates documentation and/or examples
[ ] Adds tests
[ ] Is a breaking change -- existing code written for this library may need to be rewritten to work after these changes.

Description
This pull requests adds a message object to the context when the incoming interaction is a component interaction.

Thank you so much for submitting this PR! I've looked through and verified the changes on my machine, everything looks good.
I'm going to push a few commits to add some docs and a short example, but other than that, looks good to merge! I'll push out a new release with this functionality sometime tonight.

Released as v1.2.2, you can upgrade using pip3 install --upgrade Flask-Discord-Interactions. Thanks for contributing! :+1:


Serde qs

Replaced using serde_urlencoded in form

Thanks for the PR! However, I'm not inclined to make this change. My previous comment here explains why: https://github.com/seanmonstar/reqwest/issues/1079#issuecomment-741206109

That makes a lot of sense, I was trying to find a simple approach but
didn't think of using bodies and hadn't found this comment. Thanks for
pointing it out.
On Sun, Jun 23, 2024 at 5:50 AM Sean McArthur @.***>
wrote:

Thanks for the PR! However, I'm not inclined to make this change. My
previous comment here explains why: #1079 (comment)
https://github.com/seanmonstar/reqwest/issues/1079#issuecomment-741206109
—
Reply to this email directly, view it on GitHub
https://github.com/seanmonstar/reqwest/pull/2330#issuecomment-2184976824,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFIKVQ2Y4RVL6E34RFSUYKDZI277ZAVCNFSM6AAAAABJYAD4ZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBUHE3TMOBSGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


How to force redirect to https

This is my .htaccess file
RewriteEngine On
RewriteCond %{REQUEST_FILENAME} !-f
RewriteRule . index.php [L]

How can i force to use https instead of just http?
I already search google and not yet get any solution. Please help me. Thanks.

Please try this:
RewriteEngine On
RewriteCond %{HTTPS} off
RewriteRule ^(.*)$ https://%{HTTP_HOST}%{REQUEST_URI} [L,R=301]

RewriteCond %{REQUEST_FILENAME} !-f
RewriteRule . index.php [L]

Thanks @ErikThiart, it works like a charm!


Record items are loaded back into a default device

Now that a device is required when initializing NN modules (see  https://github.com/tracel-ai/burn/pull/1081), we should do the same when initializing with a record. However, this is not done (forgotten to be refactored), see: https://github.com/tracel-ai/burn/blob/main/burn-core/src/record/tensor.rs#L88-L134

We should fix it before the upcoming release.
CCing @nathanielsimard , @louisfd , @kpot


Is it posible combine Instagram API login and this API?

Hello! I want to create an app where users will log in through Instagram Auth. Is it possible to combine these two methods of auth?
I attempted to use the token provided by Instagram after a successful login, but it did not work with the Threads API. Perhaps there is a different approach I should take. Would anyone be able to assist me with this? Thank you.

I have the same question. To make sophisticated app it is required to use token based auth. If there any work around

@prashantkumar23 I have conducted research on the matter and attempted to merge the two API types. However, it appears that the token provided by the Instagram API upon login is incompatible with the Threads API. I remain optimistic that I may receive guidance from someone more knowledgeable on the subject.

Likely not possible until OAuth is made available from Meta.


Update e2e tests to have default value for BROKER_CLASS

Fixes
When trying to run the e2e tests by following the guide provided in DEVELOPMENT.md, I was shown the following error: ERROR: Broker class is not defined. Specify it with 'BROKER_CLASS' env var.

Proposed Changes

Set a default value for BROKER_CLASS if it is undefined when running the e2e tests.

/ok-to-test

/cc @aliok

/test unit-tests

/retest-required

/retest-required

/retest-required

/retest-required


Swift port just published.

I have ported the library to Swift(*) and have used it in a few of my iOS apps. Recently, I thought I would make the code available and would appreciate it if you could link to it in your README.
My repo is at https://github.com/profburke/WanaKanaSwift/
Also, if possible, can it be linked from https://wanakana.com/ -- I notice the Python port is listed there, so it may be you don't have access to edit that site.
Regards,
Matt
(*)To be completely honest, it's not so much a port as I have wrapped your JavaScript code using Apple's JavaScript Core library. I'm working on a complete re-write in Swift, but not quite finished yet.

Thanks for contributing Matt! 🥇
I've updated the readme and website to include your port - let us know if you finish the re-write and I'll update it again.

Thanks for contributing Matt! 🥇
I've updated the readme and website to include your port - let us know if you finish the re-write and I'll update it again.


Feature idea: intelligently parse and import Google Takeout from Google Photos

For example, dealing with how Google Takeout annoyingly gives two copies of every photo that's in an album and in your main library, and not ending up with duplicates.
Google Takeout also provides .jpg.json files that could be parsed for additional information.

Cool idea. I'll look into it!