Parent class and a Sibling class can be used as an argument that expects a specific class

Bug Report
When extending a parent class, I may overload some functions or properties. As a result, it may be incompatible with the base class since the methods or properties may produce different behaviors. Currently, I can still use both the parent class and sibling class in a method that expects a specific class. If the function accepted the parent class as an argument type, I would understand why it accepts everything else as a possible type.
I don't particularly want to reimplement the entire class and call each of the classes methods just to prevent the parent or sibling classes from being used.
class WholeNumber  {
  add(v: WholeNumber|BigNumber.Value): WholeNumber {
   return  BigNumber.prototype.add.call(this, v);
  }
}

🔎 Search Terms
Extends, Parent, Child, Sibling, Compilation, Function, Argument, Typescript
🕗 Version & Regression Information
I'm not planning on testing every version of typescript. I have tried
typescript: 4.5.4
typescript: next

💻 Code
import BigNumber from "bignumber.js";

class ChildNumber extends BigNumber {
  comparedTo(y: BigNumber.Value, b?: number): number{
    console.log(y);
    console.log(b);
    return 0;
  }
}

class SiblingNumber extends BigNumber {
  comparedTo(y: BigNumber.Value, b?: number): number{
    console.log(y);
    console.log(b);
    return 1;
  }
}

const childOne = new ChildNumber(1);
console.log("cN is Child?", childOne instanceof ChildNumber);       // true
console.log("cN is Sib?", childOne instanceof SiblingNumber);           // false
console.log("cN is Big?", childOne instanceof BigNumber);           // true

const sibOne = new SiblingNumber(1);
console.log("sN is Child?", sibOne instanceof ChildNumber);         // false
console.log("sN is Sib?", sibOne instanceof SiblingNumber);             // true
console.log("sN is Big?", sibOne instanceof BigNumber);             // true

const bigOne = new BigNumber(1);
console.log("bN is Child?", bigOne instanceof ChildNumber);         // false
console.log("bN is Sib?", bigOne instanceof SiblingNumber);             // false
console.log("bN is Big?", bigOne instanceof BigNumber);             // true

function divideByThree(n: ChildNumber): ChildNumber {
  return n.div(3);
}

console.log("cN div 3:", divideByThree(childOne).toString());    // works
console.log("sN div 3:", divideByThree(sibOne).toString())       // Should not work
console.log("bN div 3:", divideByThree(bigOne).toString());      // Should not work

🙁 Actual behavior
BigNumber and DecNumber can be used when a WholeNumber is expected. In some libraries, they may extend a class from another library and setup a custom configuration for that library. If the base class or a sibling class is used in the function this has the potential to cause unexpected results. For example, if it precision was important to a class, one might return 0 another 0.3 and another 0.333. All different values
🙂 Expected behavior
I expect there to be a compilation error preventing the use of BigNumber or DecNumber from being used in the divideByThree function.

These classes are structurally identical, thus interchangeable. See https://github.com/Microsoft/TypeScript/wiki/FAQ#what-is-structural-typing

But like in the example, although it's very simple, the functions compareTo are not the same. This can lead to unexpected results. The typing may be the same with regard to arguments and return functions but if one number rounded down all decimals and the other stored up to 5 decimals, they would be different.
It makes sense for
class Person {
  name: string;
}

class Individual {
  name: string;
}

Because they are effectively the same class. But in my example, the parent, child and sibling classes are different. As a result, you should not be able to use the parent or sibling when the child is expected
This may be "working as intended" but it seems like I have to get hackey in order to prevent unwanted usage. Im under the impression organized, safe code is more typescripts style than hackey work arounds.

This is completely a consequence of structural typing; function assignability can’t be based on anything other than argument and return types because otherwise passing around callbacks wouldn’t work.
That being said, it usually makes more sense for classes to be nominally typed.  Luckily, you can get that by just adding a private property to each class.  Subclasses will still be assignable to superclasses, but that’s expected in OOP.

@fatcerberus Someone else mentioned this and it work partially but was throwing errors. But I believe that may be due to immutability of the library causing it to return an instance of the original class than my extended class. Nonetheless, I appreciate the feedback.


Audit our use of panicking

We panic in the code at many places with expect or unwrap. Ideally we want to return an error instead of panicking unless we are absolutely sure this is the right thing to do. We should maybe write guidelines somewhere about when "this is the right thing to do".

And same for the interface and the tss-esapi crate!


Migrations do not work within applications deployet as snapshots

This issue was originally created by @vadimtsushko here, before being automatically moved:
https://github.com/angel-dart-archive/orm/issues/83

Hi Tobe,
I've just started switching to your ORM in some of my new projects, so far so good. API is quite nice, generally working with it pretty enjoable (admittedly I use it only in simple scenarios yet).
I have small problem though. In my case ORM is embedded in small CLI application, that is deployed as dart snapshot file with dart executable. This applications has several commands delegating actual job to migration_runner.
So when deployed to snapshot migration up command breaks on the line
https://github.com/angel-dart/orm/blob/76016f20624621738748daeec38a3d4407184270/angel_migration_runner/lib/src/util.dart#L13
because with snapshots we do not have any package layout and so on (that's one of the main reasons of using snapshots anyway)
For now I've currently just added a conditional branch for null source file path, using Class name as a name of migration in that case.
I'm not sure that is ideal solution though so I didn't make a PR.
Another, related point is - using dart:mirrors just once in whole ORM effectively prevents us from using new AOT snapshots in deployed applications. Snapshot generation breaks with error about dart:mirrors. I believe ideally we shouldn't use mirrors anymore, they tends to be more and more a liability in all sorts of different scenarios

@thosakwe commented:
Apologies for the late response.
A possible solution is maybe including that information in the generated migration class. It's pretty feasible:
class MyMigration extends Migration {
  @override
  String get sourceHash => 'package:foo/lib/src/models.dart#_MyClass';
}

The only thing is that this would be a breaking change for existing migrations. Perhaps a fallback could use package:stack_trace and some other hackery to figure out the class name:
// pseudocode:
String get sourceHash {
  String file, className;
  var trace = Trace.current();
  file = trace.frames.first.filename;
  className = runtimeType.toString();
  return file + '#' + className;
}


Roboto font not available for matplotlib

... despite the apt-get install ???

Mmm ... the tests show that matplotlib cannot find the font despite the system and matplotlib cache update, but now the proper font appears in the graphs ... WTF !?!

OK, I don't know why it works and I cannot understand now why the graphs are properly using the Roboto font even though matplotlib cannot find it in my test, but nevermind. I am closing this issue ATM, this is not that important.


CO Status shows off

The problem
Hi there! Congratulations for this great job! Just testing out this integration which worked fine out of the box (copying nest_protect to custom_components folder and restarting ha).
I found that CO status shows off in my device, is this correct? Since I cannot see this in nest application I am not sure if this is an expected behaviour, but I think the feedback will be welcomed, screenshot here:

If there is any test or contribution needed just ask, I will very pleased to help with this integration development.
Regards
What version of this integration (ha-nest-protect) has the issue?
0.1.1
What version of Home Assistant Core has the issue?
core-2021.12.9
Device / Model
Topaz-2.7
Home Assistant log
Logs
```
Copy/paste any log here, between the starting and ending backticks (`)
```

Additional information
No response

I think this is the correct state, since it should be 'off' when there is no CO detected.
From 2022.2 (upcoming update) onwards, there is a CO device class for binary sensor, thus perhaps that will show 'clear' as well in the interface.

Great thanks, I forgot to mention that my nest is battery model, just as feedback because I saw you mention it on ha forum.
Thank you again 👍🏻

Hopefully I will be able to work on the real-time updates soon! Until that point, the integration is a bit worthless.

Yep I am aware, I also saw this in the forum, but taking it as a proof of concept I think it went perfect, if you need any help I can contribute with the python code part if needed, just ask and I ping you on ds

Contributions are always welcome! 😉. I am currently spending a lot of time on the Overkiz / Somfy integration in core, thus this currently comes on the second place (if I even have time :D).

Commenting on a closed ticket, but:
The way the CO2 sensor is handled in the HomeBridge integration (where it does show as Clear, instead of Off) is they've used device_class: gas for the sensor.

We are indeed using carbonmonoxide, which has been added to front-end / back-end in upcoming 2022.2 release. -> https://github.com/home-assistant/frontend/pull/10935


when clicking on highlighting element in the tour no action happen

when user is clicking on highlighting element in the tour ..
I am highlighting a box in my web page that contain many button and some action !
unfortunately he click that button/action means that button action opens that same time the tour popover is still their and then i click the next button the tour will keep on running
i can i do ... to stop that action on clicking on highlighting element in the tour
Thankyou

how to restrict when  user click the highlighting element in the tour .????

My fork adds reflexOnly option to step. When true, "next" button is hidden and user can only continue tour by clicking reflex element.
https://github.com/sorich87/bootstrap-tour/pull/710


Remove support for Node 8

Node 8 is officially EOL. Although we weren't initially planning on removing support for it in v3 of the SDK, it seems appropriate to do so since Lambda functions can no longer be created with Node 8 runtimes which will make reproducing some issues impossible.
To encourage customers to keep software up-to-date, unless there is major pushback from the community, we will remove support for Node 8 upon the release of v3 of the SDK. #157 has been updated to reflect this.

Released in 3.0.0.


Can't remove one marker

Hello,
I'm trying to remove one marker an i have done this:
$scope.markers[nm]=null;
delete $scope.markers[nm];
But the marker remain visible, the leaflet countepart seem to be still there.
You have an example for removing all markers, but i just want to remove one.
Any idea?

Probably related to #478

This issue was moved to angular-ui/ui-leaflet#45

I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it.


Synapse workspace auth types

Service account is listed as an authentication type for this connector.  However, when using it there is no option for a service account.  The pattern should be documented

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

ID: 054662fb-66f6-d36d-6ba5-85fde75574b9
Version Independent ID: 054662fb-66f6-d36d-6ba5-85fde75574b9
Content: Power Query Azure Synapse Analytics workspace connector - Power Query
Content Source: powerquery-docs/Connectors/Synapse.md
Service: powerquery
GitHub Login: @miquelladeboer
Microsoft Alias: mideboer

Hi @mikem-MS
Thanks for your feedback. Synapse workspace only accepts an organizational account, and the article has been updated to reflect this.


Fix: Apache.OpenOffice version 4.115.9813

Microsoft Reviewers: Open in CodeFlow
Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-No-Executables


Xaml Islands Win32 MenuFlyoutSubItem crash

This issue has been moved from a ticket on Developer Community.

[severity:It's more difficult to complete my work]
Use any Xaml Islands Win32 sample as base, e.g. Xaml-Islands-Samples
Add a MenuBar with two MenuFlyoutSubItem elements. E.g:
<MenuBar>
	<MenuBarItem Title="MenuBarItem">
		<MenuFlyoutSubItem Text="MenuFlyoutSubItem1">
			<MenuFlyoutItem Text="MenuFlyoutItem1"/>
		</MenuFlyoutSubItem>
		<MenuFlyoutSubItem Text="MenuFlyoutSubItem2">
			<MenuFlyoutItem Text="MenuFlyoutItem2"/>
		</MenuFlyoutSubItem>
	</MenuBarItem>
</MenuBar>

Run the app, open the menu, hover over the first SubItem until the menu animation starts playing, then quickly move to the second SubItem (or just move quickly between the two subitems without giving enough time for the animation to complete). The app will crash with: "Exception thrown at 0x030B450C (Windows.UI.Xaml.dll) in SampleCppApp.exe: 0xC0000005: Access violation reading location 0x00000000."

Original Comments
Feedback Bot on 10/29/2020, 07:16 PM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

I have this issue too.
It is very annoying and is very easy for users to trigger.

I found a workaround: https://github.com/microsoft/microsoft-ui-xaml/issues/3529#issuecomment-827211418

Duplicate of #3529


Aurora dev interested in fixing this properly.

Hey, finally got access to the full aurora source tree, and was planning on doing some bug fixes, and possibly new features. The first thing that should be fixed though is this asset issue. Since you were able to track it down, and dns patch it, any chance you could hit me up on discord, so we can discuss what the actual issue is, would be helpful in tracking down and fixing the issue in the aurora codebase. let me know if you are interested.

Just read this, @GavinDarkglider. Dropped you a fr on Discord. Thanks for the message and your patience, as I haven't been keeping a close enough eye on this project's issues lately.


Updating recipes/diags to use the new interface for fx variables

Is your feature request related to a problem? Please describe.
This is an issue to track and test that the changes in https://github.com/ESMValGroup/ESMValCore/pull/999 do not break anything.
Would you be able to help out?
Yes

[ ] Tested sucessfully recipe_wenzel14jgr.yml preprocessor global_ocean with both the old and new syntax for area_statistics with fx_variables: areacello.
[ ] Update the call of mask_landsea in esmvaltool/cmorizers/obs/cmorize_obs_jma_transcom.py


Add pandoc installation step to installation instructions

While the instruction to install pandoc is stated in the contributing guidelines, it is not stated in the installation from source instructions. Given that the installation instructions include running the tests, for which pandoc is required, an instruction to install pandoc should be explicit in these instructions.

Thanks for opening this, @dc2917 – it is very likely that this was missed earlier. Please feel free to make the necessary changes here!


AotPlugin fails when combined with webpack's multi compiler

Bug Report or Feature Request (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ ] feature request

Versions.
@angular/cli: 1.0.0 (e)
node: 7.7.4
os: darwin x64
@angular/common: 4.0.2
@angular/compiler: 4.0.2
@angular/core: 4.0.2
@angular/forms: 4.0.2
@angular/http: 4.0.2
@angular/platform-browser: 4.0.2
@angular/platform-browser-dynamic: 4.0.2
@angular/router: 4.0.2
@angular/cli: 1.0.0

Repro steps.

Create a new angular-cli project (ng new multi-test)
Eject a webpack config for AOT compilation (ng eject -prod --aot)
Install missing dependencies (npm install)
Edit the config, changing module.exports into an array of copies of the existing configuration, each with a different value in output.path (example file, changes compared to original in lines 21, 23, 51)
Try building (npm run build)

The log given by the failure.
 ERROR in ./src/$$_gendir/app/app.module.ngfactory.ts
    Module not found: Error: Can't resolve '@angular/core/core' in '/Users/daniel/Documents/multi-test/src/$$_gendir/app'
     @ ./src/$$_gendir/app/app.module.ngfactory.ts 17:0-46
     @ ./src/main.ts
     @ multi ./src/main.ts

    ERROR in ./src/$$_gendir/app/app.component.ngfactory.ts
    Module not found: Error: Can't resolve '@angular/core/core' in '/Users/daniel/Documents/multi-test/src/$$_gendir/app'
     @ ./src/$$_gendir/app/app.component.ngfactory.ts 8:0-46
     @ ./src/$$_gendir/app/app.module.ngfactory.ts
     @ ./src/main.ts
     @ multi ./src/main.ts

In the case of my example config file, the first two configs ("pl" and "en") fail due to the above errors, while the last one succeeds and only dist/es is created.
Desired functionality.
I want to build my app for all the languages I have a translation for, leveraging webpack's multi compiler for simultaneous building and the fact that the config is a js file to generate as many config objects in an array as I have xlf files with translations.
Mention any other details that might be useful.
Everything works fine as long as I export an array of one config. As soon as more are added, all but the last one fail.
As far as I can see it's something about the AotPlugin, because with skipCodeGeneration set to true, the errors stop appearing and everything compiles fine.
The workaround I have for now is to call webpack separately for each language, as showcased here.
Thanks in advance for any help on this one.

@Torins, I followed your "Repro steps" and everything is working properly. Could you please retry it and update on the result?
#goodnessSquad

I've re-did exactly my reproduction steps and I still get an error.
Assuming you've used my example file:
Did you get all three subfolders (en, pl, es)?
Did you scroll up for the errors?
Using the multicompiler, my config runs three builds in parallel, which in turn means three "entire" outputs. As I've mentioned, the last build always succeeds, which may lead onto a false sense of the entire config to run properly.
Here is the entire output I've got from a fresh attempt. The result was that only the "es" folder got created, instead of all three

Having the exact same issue. Tried to execute two builds using webpack's multi compiler, and only the last succeeds.
In order to be sure that the issue is AotPlugin related, I didn't use the plugin in the first build, and seems that both builds succeeded in this case. Both build configurations are completely the same, except the output paths.
The error message I'm getting is huge. Basically this is the error, and it repeats for each resource that imports @angular/core:
ERROR in ./app-www/client/dist/www/modules/app/app.module.browser.ngfactory.ts
Module not found: Error: Can't resolve '@angular/core/core' in '/Users/chacha/ironbuzz/git_wa/waterfall/app-www/client/dist/www/modules/app'
resolve '@angular/core/core' in '/Users/chacha/ironbuzz/git_wa/waterfall/app-www/client/dist/www/modules/app'
  Parsed request is a module
  using description file: /Users/chacha/ironbuzz/git_wa/waterfall/package.json (relative path: ./app-www/client/dist/www/modules/app)
    Field 'browser' doesn't contain a valid alias configuration
  after using description file: /Users/chacha/ironbuzz/git_wa/waterfall/package.json (relative path: ./app-www/client/dist/www/modules/app)
    resolve as module
      looking for modules in /Users/chacha/ironbuzz/git_wa/waterfall
        using description file: /Users/chacha/ironbuzz/git_wa/waterfall/package.json (relative path: .)
          Field 'browser' doesn't contain a valid alias configuration
        after using description file: /Users/chacha/ironbuzz/git_wa/waterfall/package.json (relative path: .)
          using description file: /Users/chacha/ironbuzz/git_wa/waterfall/package.json (relative path: ./@angular/core/core)
            as directory
              /Users/chacha/ironbuzz/git_wa/waterfall/@angular/core/core doesn't exist
            no extension
              Field 'browser' doesn't contain a valid alias configuration
              /Users/chacha/ironbuzz/git_wa/waterfall/@angular/core/core doesn't exist
            .ts
              Field 'browser' doesn't contain a valid alias configuration
<EMAIL_ADDRESS>doesn't exist
            .js
              Field 'browser' doesn't contain a valid alias configuration
<EMAIL_ADDRESS>doesn't exist
      looking for modules in /Users/chacha/ironbuzz/git_wa/waterfall/node_modules
        using description file: /Users/chacha/ironbuzz/git_wa/waterfall/package.json (relative path: ./node_modules)
          Field 'browser' doesn't contain a valid alias configuration
        after using description file: /Users/chacha/ironbuzz/git_wa/waterfall/package.json (relative path: ./node_modules)
          using description file<EMAIL_ADDRESS>(relative path: ./core)
            as directory
              /Users/chacha/ironbuzz/git_wa/waterfall/node_modules/@angular/core/core doesn't exist
            no extension
              Field 'browser' doesn't contain a valid alias configuration
              /Users/chacha/ironbuzz/git_wa/waterfall/node_modules/@angular/core/core doesn't exist
            .ts
              Field 'browser' doesn't contain a valid alias configuration
<EMAIL_ADDRESS>doesn't exist
            .js
              Field 'browser' doesn't contain a valid alias configuration
<EMAIL_ADDRESS>doesn't exist
[/Users/chacha/ironbuzz/git_wa/waterfall/@angular/core/core]
[/Users/chacha/ironbuzz/git_wa/waterfall/@angular/core/core]
<EMAIL_ADDRESS><EMAIL_ADDRESS>[/Users/chacha/ironbuzz/git_wa/waterfall/node_modules/@angular/core/core]
[/Users/chacha/ironbuzz/git_wa/waterfall/node_modules/@angular/core/core]
<EMAIL_ADDRESS><EMAIL_ADDRESS> @ ./app-www/client/dist/www/modules/app/app.module.browser.ngfactory.ts 17:0-46
 @ ./app-www/client/dist/main.ts

I don't think @ngtools/webpack supports multi compiler at all, but @hansl can answer this better.

@filipesilva Really? i've been using it with a multi, first one deals with Angular 2 code, and second one with an Electron main.ts file and it works just fine.


Handling of last batch?

Hi, thanks for this very useful library!
How does xbatcher handle the last batch? Does it throw it away or return a reduced batch?

I know this is an old question but I there's a simple answer so I'll go ahead and answer it. Partial batches are currently discarded. Here's a quick demo that shows this behavior:
In [1]: import xbatcher

In [2]: import xarray as xr

In [3]: ds = xr.tutorial.open_dataset('air_temperature')

In [4]: ds['air']
Out[4]:
<xarray.DataArray 'air' (time: 2920, lat: 25, lon: 53)>
[3869000 values with dtype=float32]
Coordinates:
  * lat      (lat) float32 75.0 72.5 70.0 67.5 65.0 ... 25.0 22.5 20.0 17.5 15.0
  * lon      (lon) float32 200.0 202.5 205.0 207.5 ... 322.5 325.0 327.5 330.0
  * time     (time) datetime64[ns] 2013-01-01 ... 2014-12-31T18:00:00
Attributes:
    long_name:     4xDaily Air temperature at sigma level 995
    units:         degK
    precision:     2
    GRIB_id:       11
    GRIB_name:     TMP
    var_desc:      Air temperature
    dataset:       NMC Reanalysis
    level_desc:    Surface
    statistic:     Individual Obs
    parent_stat:   Other
    actual_range:  [185.16 322.1 ]

In [5]: for batch in ds['air'].batch.generator({'time': 100}):
   ...:     pass
   ...:

In [6]: batch
Out[6]:
<xarray.Dataset>
Dimensions:  (sample: 1325, time: 100)
Coordinates:
  * time     (time) datetime64[ns] 2014-12-02 ... 2014-12-26T18:00:00
  * sample   (sample) MultiIndex
  - lat      (sample) float64 75.0 75.0 75.0 75.0 75.0 ... 15.0 15.0 15.0 15.0
  - lon      (sample) float64 200.0 202.5 205.0 207.5 ... 325.0 327.5 330.0
Data variables:
    air      (sample, time) float32 240.7 244.4 257.1 ... 298.7 299.4 299.7


[WFLY-11238] Eliminating ServiceBuilder's deprecated methods usages

Eliminating ServiceBuilder's deprecated addInjection() and addInjectionValue() methods usages.
https://issues.jboss.org/browse/WFLY-11238

For ContextServiceAdd and PersistenceUnitServiceHandler, should these injectors just be dropped and the value passed through the constructor?  The relevant services are not constructed elsewhere (and are not meant to be.)

Done @bstansberry .

@ropalka compile error


Switch network on wallet v0.8.2 requires a restart for the connect approval to work

Description
If you open the wallet and are on e.g. sandbox, mainnet etc, and then switch to fairground, and try to connect to console, the popup in the wallet to choose which wallet to connect to does not appear. You first need to restart the wallet.
MacOS (amd)
Steps to Reproduce

Open wallet v0.8.2 and choose "mainnet" or whichever, restart wallet
Go to https://console.fairground.wtf/ and try to connect by swtiching network to Fairground in wallet
No popup in the wallet appears

Restart the wallet after choosing Fairground, go to console and connect again and it works.

We don't believe this is any longer relevant as we are on version 12.02 and have separated apps by network. Feel free to reopen or create a new ticket if you are still experiencing this


agent.save_model issue!!

Hi,
Again, this is a bug of tensorforce. In docs, agent.save_model has two parameters: path and use_global_step. But in the new version, I have written:
agent.save_model(path=path, use_global_step=False)
Then got:
TypeError: save_model() got an unexpected keyword argument 'use_global_step'
That is not a bug of tensorflow. If I wrote save to "mymodel.ckpt" then I got "mymodel.ckpt". Did anyone really try to call save_model() and load_model() functions??

Please post this in the other issue. In any case, the point is that you are calling a parameter that is only available on the model's method on the agent. We can make this available on the agent too, it was just for internal use thus far.
I will summarise:
The default behaviour is that global step is true. Checkpoint contains the step. In that case, you can only reload by using the correct global step in the file. The checkpoint file will be .ckpt-XXXXX where X is global step.
OR you set global step to false, then the checkpoint file will just be .ckpt, which was currently only available on the model not the agent, so you can trivially use it by calling agent.model.save_model(path=path, use_global_step=False), and we can adjust the API there.


Is this repository abandoned/dead?

If so, could someone put in the readme that it's been abandoned and perhaps Archive it?

Bump


dapr run -f docs

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Read the contribution guide
[x] Commands include options for Linux, MacOS, and Windows within codetabs
[x] New file and folder names are globally unique
[x] Page references use shortcodes instead of markdown or URL links
[x] Images use HTML style and have alternative text
[x] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description

[x] Add doc for dapr run -f under Developing Applications/Local Development
[x] Add Multi App Dapr Run to preview features doc
[x] Add to CLI reference docs?

Issue reference
PR will close: #3047

@hhunter-ms There is PR converting all snake_case in the run template yaml into camelCase as per Dapr standards ....
https://github.com/dapr/cli/pull/1183
Please change the docs also as per that

@hhunter-ms Just a few more suggestions. Thank you for the PR.

@hhunter-ms If possible can we add details on how users can view the started apps using dapr list as well as stop using dapr stop -f . or dapr stop -f dapr.yaml.

Please add --run-file, -f to dapr stop command ref doc as well.

This needs to be added as well @hhunter-ms


Fix audio page mask image in Safari 12

Safari 12 requires CORS headers for cross origin mask images. Use data
url instead.
REDMINE-16141

Coverage remained the same at 99.294% when pulling 3d5ae146b600a5411dd6bacbab1a158417c10635 on tf:inline-mask-image into 18c47bbb1c06e9f7534fc9bc7a24726be1237437 on codevise:master.


Connection pooling issues in sails 1.2.2

Node version:  v8.9.3
Sails version (sails): v1.2.2
ORM hook version (sails-hook-orm): v2.1.1
Grunt hook version (sails-hook-grunt): v1.0.8
DB adapter & version (e.g. sails-mysql@5.55.5<EMAIL_ADDRESS>
Hi I am currently upgrading from sails 0.12 to 1.2.x and stumbled upon and issue where trying to run each test in a transaction.
So the setup previously was to have bootstrap.test.js, now lifecycle.test.js, to have beforeEach/afterEach funcitons run a query to start and end a transaction.
beforeEach(function(done) {
  sails.models.user.query('BEGIN;', function(err) {
    if (err) {
      console.log('BEGIN FAIL: ', err);
    }
    done(err);
  });
});

afterEach(function(done) {
  sails.models.user.query('ROLLBACK;', function(err) {
    if (err) {
      console.log('ROLLBACK FAIL: ', err);
    }
    done(err);
  });
});

Now I've done this:
beforeEach(async (done) => {
  await sails.getDatastore().sendNativeQuery('BEGIN');
  done();
});

afterEach(async (done) => {
  await sails.getDatastore().sendNativeQuery('ROLLBACK;');
  done();
});

I want each and every query to be ran on a single connection from the connection pool and they shall be in a transaction.
But it does not seem to work properly. Nothing is being roll-backed.
Hence I've added this to my datastore configuration:
    default: {
      adapter: 'sails-postgresql',
      host: 'localhost',
      user: 'user',
      password: 'test-password',
      database: 'test_db',
      port: 5432,
      max: 1
    },

Example response from postgres log (log_line_prefix = '<%x/%v>').
NOW  (sails v1.2.2):
06/4LOG:  statement: BEGIN;
04/14LOG:  execute <unnamed>:
04/18LOG:  execute <unnamed>:
06/4LOG:  statement: ROLLBACK;

PREVIOUS (sails v0.12):
03/1696LOG:  statement: BEGIN;
03/1696LOG:  execute <unnamed>:
57033393/1696LOG:  execute <unnamed>:
57033393/1696LOG:  statement: ROLLBACK;

I'll put this on hold, i've just scaled down everything and now it works. Maybe there something wierd in the upgrade. I'll reopen issue if I need help


Need for different types of outputs of the demand model

In the sensitivity analysis, as well as in testing, not all the outputs of the demand model will be needed. Sometimes the file total data is not necessary, sometimes hourly data is not necessary, but monthly and vice versa etc..
I think we could create some standard output configurations. This will be especially effective when generating big chunks of data. like results of 200k simulations..
It would be good to create a class of outputs.
This issue will be solved together with #351

One way to do this is to refactor the methods for writing out the result files to a class, say BuildingResultWriter and TotalsResultWriter. Have the demand_main script use a default value for these - one that writes out the results just like the current version does. Then, you can create alternative subclasses of these writers: a MonthlyBuildingResultsWriter would just write out the monthly results, a DisableTotalsResultWriter would just discard any results, i.e. do nothing (pass).
This is one of the places where following OO principles makes sense. Of course, this could also more elegantly be implemented with functions - write_building_results etc. and store those in a variable. It doesn't really matter which way we choose.

Note, if we are doing the calibration calculations on Euler, we don't really need to do this: We can implement a post-processing step that converts the hourly results into monthly results.
When running on Euler, you have two types of storage: A per-user storage (your $HOME folder) that is persisted between sessions and a per-node storage (a "scratch" folder) that is deleted after a job has been completed.
I envision running each demand calculation in a separate job, that starts with copying a template reference-case from the home folder to the scratch folder, applying the changes, running the demand and then post-processing and collecting the results. The last step would copy the output back to the home folder. Once all the demand simulations have been done, a final step would run the sensitivity analysis and produce the final output.

I created a solution to this with some flags in global variables.
It works nicely in my pc. after merged maybe we can thing on transforming the remake of results_to_cv function of the thermal loads.py to a class as you say? I agree with you that could be the way to go. .
On 21 Oct 2016, at 5:48 pm, Daren Thomas <notifications<EMAIL_ADDRESS>wrote:
One way to do this is to refactor the methods for writing out the result files to a class, say BuildingResultWriter and TotalsResultWriter. Have the demand_main script use a default value for these - one that writes out the results just like the current version does. Then, you can create alternative subclasses of these writers: a MonthlyBuildingResultsWriter would just write out the monthly results, a DisableTotalsResultWriter would just discard any results, i.e. do nothing (pass).
This is one of the places where following OO principles makes sense. Of course, this could also more elegantly be implemented with functions - write_building_results etc. and store those in a variable. It doesn't really matter which way we choose.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/architecture-building-systems/CEAforArcGIS/issues/377#issuecomment-255340242, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIjrgs-aaDcP3kJ3rHFciQ5DN457Ew_3ks5q2IpwgaJpZM4Kc3lP.


DCMAW-8966: generate and publish STS mock JWKS

DCMAW-8966
What changed

Add a script that:

Generates an asymmetric key pair for signing
Saves the private key to the jwks S3 bucket in JWK format
Saves the public key to the  jwks S3 bucket /.well-known directory in JWKS format

Tested in the dev environment
https://github.com/user-attachments/assets/c3f6d1f3-4acc-4e98-97a5-e5d4be8ca485
Public key

Private key

Why did it change

Private key is required for signing the token issued by the STS mock
JWKS (public key) is required for validating the signature on the token issued by the STS mock

Checklists

[x] There is a ticket raised for this PR that is present in the branch name
[x] No PII data logged. See guidance here
[ ] Demo to a BA, TA, and the team.
[x] Update README with any new instructions or tasks

Question: I'm not sure if we want the new directory jwks-helper-script to live inside the sts-mock directory or outside 🤔

inside for now - until we know we might want to use it for more than one application

Please may you delete this public/private keypair if you haven't just to make sure you don't use it for anything :)

Please may you delete this public/private keypair if you haven't just to make sure you don't use it for anything :)

Which key pair?

the one in the evidence :)

Please may you delete this public/private keypair if you haven't just to make sure you don't use it for anything :)

Which key pair?

the one in the evidence :)

Ohhhh! 🤦 - I will delete it from the bucket

thank you! Public repo :)


Ionic serve not working

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:

ionic-app-scripts serve

[13:35:18]  ionic-app-scripts 0.0.45
[13:35:19]  watch started ...
[13:35:19]  build dev started ...
[13:35:19]  clean started ...
[13:35:19]  clean finished in 2 ms
[13:35:19]  copy started ...
[13:35:19]  transpile started ...
events.js:160
throw er; // Unhandled 'error' event
^
Error: listen EADDRINUSE <IP_ADDRESS>:8100
at Object.exports._errnoException (util.js:1026:11)
at exports._exceptionWithHostPort (util.js:1049:20)
at Server._listen2 (net.js:1262:14)
at listen (net.js:1298:10)
at net.js:1397:7
at _combinedTickCallback (internal/process/next_tick.js:77:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
at Module.runMain (module.js:609:11)
at run (bootstrap_node.js:382:7)
at startup (bootstrap_node.js:137:9)
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\T2988\AppData\
Roaming\npm\node_modules\npm\bin\npm-cli.js" "run" "ionic:serve" "--"
npm ERR! node v7.0.0
npm ERR! npm  v4.0.2
npm ERR! code ELIFECYCLE
npm ERR! ionic-conference-app@ ionic:serve: ionic-app-scripts serve
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the ionic-conference-app@ ionic:serve script 'ionic-app-scrip
ts serve'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the ionic-conference-app
package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ionic-app-scripts serve
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs ionic-conference-app
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls ionic-conference-app
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\T2988\Documents\ionic-2\ionic-conference-app\npm-debug.log
There was an error serving your Ionic application: There was an error with the s
pawned command: serve
What behavior are you expecting?
Steps to reproduce:
1.
2.
3.
insert any relevant code between the above and below backticks (```)

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Run ionic info from terminal/cmd prompt: (paste output below)
ordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.3
Ionic CLI Version: 2.1.12
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.45
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 7
Node Version: v7.0.0
Xcode version: Not installed

I updated my package.json
"scripts": {
"build": "ionic-app-scripts build",
"watch": "ionic-app-scripts watch",
"serve:before": "watch",
"emulate:before": "build",
"deploy:before": "build",
"build:before": "build",
"run:before": "build"
},
It's working,this problem arises from my update ionic2


How to add copy-paste augmentation to train_pipeline?

Hello! I am currently doing object detection using MMDetection and VOC-format dataset. I want to implement a customized data pipeline by adding copy-paste (a data augmentation method) to existing train_pipeline in file voc0712.py, following instructions: https://mmdetection.readthedocs.io/en/latest/tutorials/data_pipeline.html. Codes for copy-paste is here: https://github.com/conradry/copy-paste-aug. But this one seems to be for coco format. For now I have no idea how to implement it correctly.

In MMDetection, the dataset output format is xyxy, which is the same as the input required by the copy-paste-aug library. And you can use scripts to visualize the augmented data.

If you are interested, feel free to submit a PR！

You may have a look at instaboost.

Hello! I am currently doing object detection using MMDetection and VOC-format dataset. I want to implement a customized data pipeline by adding copy-paste (a data augmentation method) to existing train_pipeline in file voc0712.py, following instructions: https://mmdetection.readthedocs.io/en/latest/tutorials/data_pipeline.html. Codes for copy-paste is here: https://github.com/conradry/copy-paste-aug. But this one seems to be for coco format. For now I have no idea how to implement it correctly.

Hi, have you successfully implementing copy-paste in mmdetection? If so, can you provide the code or some advices? Thanks!


stuck on Downloading Metadata...

stuck on Downloading Metadata...

use magnet links

@gunysa1 also see #20
@r12habh please close this. It is a duplicate of #20


[OP] Negative axis, begin, end and end=None for slice_axis

Support negative axis, begin, end and end=None for "slice_axis"
Should be continuing https://github.com/dmlc/mxnet/pull/3921

@piiswrong @sbodenstein I plan to merge this in.

@sxjscience: thanks for following up on this! (and apologies for disappearing)
LGTM


Publish Docker

Publish the application as Docker Image

So the end user can run the command directly with like 1-2 or several lines
No need to set JAVA_HOME or other settings

Parameterize the config.properties and MySQL password

https://www.tutorialspoint.com/how-to-run-sql-script-using-jdbc

Build the Docker Image

sudo docker build -t fuiny/mavendb .

Show current Docker images and containers

sudo docker images

Run the Docker

sudo docker run -it --rm fuiny/mavendb

Provide a docker-compose.yml also

Support Apple Silicon


API Status "error"

As subjected after sign on

Bug Report Dump (Auto-generated)

Version 1.27.2<PHONE_NUMBER>)
Device: iPhone15,2 (iOS 17.0)
TestFlight: false

As subjected after sign on
Bug Report Dump (Auto-generated)
Version 1.27.2<PHONE_NUMBER>)
Device: iPhone15,2 (iOS 17.0)
TestFlight: false


Rework auto-scrolling options

This changes two things, the first thing it does is that now whenever a
keybinding sends an escape sequence, the viewport is automatically
scrolled to the bottom.
This is enabled by default and fixes #1187.
The second thing is automatic scrolling when a command writes to the
terminal. So when running a command like sleep 3; ls -lah, alacritty
will scroll to the bottom once the output is sent, even if the viewport
is currently not at the bottom of the scrollback.
Because this can have an impact on performance, and is not enabled by
default in terminals like iTerm or Termite (VTE), it is an opt-in
setting in the config.

This looks great; thanks!


Use taxonomy query instead of meta query.

Summary
Requires #5636 #5635 to merged. But use taxonomy query instead of meta query.
Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

Fixes #

#5635 and #5636 have been merged, so this needs a rebase

Tests have been added.


Profiles: profile display in tribes - Cross-platform App

Service involved:
Cross-platform app
Goal:
Create a display for a user's tribe-specific profile information, which is returned from the tribes-server when someone clicks on their username in a tribe.
Method:

Profile information will be returned here: tribes.sphinx.chat/person/uuid/{uuid}
It should be displayed according to Milan's design below:

*for getting the badge icons using the badge IDs: https://liquid.sphinx.chat/list

@soup20  We built a first version of the tribe member profile on android app and it will be included on next release, but as the designs didn't match the information coming on the response json, I just decided what to show on the profile for now until we defined them.
Right now we are showing: Image, Name, Description, Price to meet, coding languages, posts, twitter and GitHub accounts.
Screenshot below

Most of the items on the design are not retrieved from the people server (including badges)
You can check the current json respose on this example:
https://people.sphinx.chat/person/uuid/cd9dm5ua5fdtsj2c2m7g


Add CI for examples/.tool-versions are already installable in asdf-plugins

Because of asdf-nodejs dependents node-build. Latest versions can not be installed by asdf-nodejs
And I would integrate faster automergers for all bot PRs, .tool-versions check is the last one of my worry.

Cons
https://github.com/asdf-vm/actions/tree/171cc3570019f3a69ba10ae269ab46990936fa9f does not support git-url yet. So non registered https://github.com/asdf-vm/asdf-plugins/tree/master/plugins plugins will not work

Works as https://github.com/kachick/renovate-config-asdf/actions/runs/3354907278/jobs/5558777814#step:5:19 🎉
Invalid version of golang will be blocked with new CI 😤

Covering node-build looks nothing with current renovate apps 🤔
https://github.com/nodenv/node-build/tree/98f674e2ff844328351be1550fcd68cbb1263659/share/node-build
https://github.com/renovatebot/renovate/issues/4674
https://github.com/nodenv/node-build/pull/756 is a way to resolve it 👀


Trouble using Witness object

After resolving the ssl certificate problem on all of my nodes and successfully being able to connect to them using python-peerplays, I am getting an exception:
peerplays.exceptions.AccountDoesNotExistsException: baxters-sports-witness

This exact, simple code works in python-bitshares, but not in python-peerplays. The only differences I can see is that in bitshares my witness account name has no dashes in it whereas in peerplays it does, and the length of the name is much longer in the failure case. I added the active key for the witness account to the SQLite wallet after pulling it from my active witness wallet using "dump_private_keys". I also verified the key is correct by comparing to what I saved in my key vault.
API = PeerPlays(API_NODES, nobroadcast=False)
print(ACNT)     <--- prints correct account name, including dashes
status = Witness(ACNT)   <--- this call fails
print(status['signing_key'])
exit(-10)

I also tried this to eliminate issues with private key, length and the dash as possible causes:
# account = Account("taconator\x2Dwitness")
# account = Account("roelandp")
account = Account("init0")
print(account.balances)  <-- it printed "[4,359.99946 PPY]"

exit(-10)

All failed except init0. I then tried to verify the chainId using:
print(API.info())

and it printed everything except the chainId (which is 6b6b5f0ce7a36d323768e534f3edb41c6d6332a541a95725b98e28d140850134):
{'recently_missed_count': 0, 'current_witness': '1.6.12', 'last_budget_time': '2018-03-29T23:00:00', 'witness_budget': 321175, 'head_block_number': 8463816, 'id': '2.1.0', 'random': 'e6a71bc39291540dfc3c2011d68144ab8d4561f7', 'time': '2018-03-29T23:38:00', 'recent_slots_filled': '340282366920938463463374607431768211455', 'dynamic_flags': 0, 'next_maintenance_time': '2018-03-30T00:00:00', 'head_block_id': '008125c8681c2396a9a146ba3958d11714fd9b7a', 'last_irreversible_block_num': 8463803, 'current_aslot': 8512624, 'accounts_registered_this_interval': 0}
If I'm doing something wrong (I'm now rather doubtful I am) it is mighty subtle.

Still not working for me after upgrading:
...
Installing collected packages: treelib, peerplays
Found existing installation: peerplays 0.1.15
Uninstalling peerplays-0.1.15:
Successfully uninstalled peerplays-0.1.15
Successfully installed peerplays-0.1.16 treelib-1.5.1
(pyppy) admin@ppy003 $ ./testPPYconn.py
Traceback (most recent call last):
  File "./testPPYconn.py", line 45, in <module>
    status = Witness(ACNT)
  File "/home/admin/.virtualenvs/pyppy/lib/python3.5/site-packages/peerplays/blockchainobject.py", line 96, in __init__
    self.refresh()
  File "/home/admin/.virtualenvs/pyppy/lib/python3.5/site-packages/peerplays/witness.py", line 27, in refresh
    self.identifier, peerplays_instance=self.peerplays)
  File "/home/admin/.virtualenvs/pyppy/lib/python3.5/site-packages/peerplays/account.py", line 51, in     __init__
    peerplays_instance=peerplays_instance,
  File "/home/admin/.virtualenvs/pyppy/lib/python3.5/site-packages/peerplays/blockchainobject.py", l   line 96, in __init__
    self.refresh()
  File "/home/admin/.virtualenvs/pyppy/lib/python3.5/site-packages/peerplays/account.py", line 64, in     refresh
    raise AccountDoesNotExistsException(self.identifier)
peerplays.exceptions.AccountDoesNotExistsException: baxters-sports-witness

I created a fresh virtual env, downloaded a zip of the develop branch, unzipped it and changed to the python-peerplays-develop folder. I installed with pip3 install . It behaves identical to the previous version (works for the init0 account but no others that I've tried):
(tst-ppy) admin@ppy003 $ ./testPPYconn.py
A wallet exists but the password won't unlock it.
Create a new wallet with the password you entered (y/n)?y
Are you REALLY sure?y
Done.

No wallet exists! Creating one now...
[998,359.99769 PPY]  #<--- This shows init0 balance

(tst-ppy) admin@ppy003 $ ./testPPYconn.py
Traceback (most recent call last):
  File "./testPPYconn.py", line 216, in <module>
    account = Account("roelandp")
  File "/home/admin/.virtualenvs/tst-ppy/lib/python3.5/site-packages/peerplays/account.py", line 51, in __init__
    peerplays_instance=peerplays_instance,
  File "/home/admin/.virtualenvs/tst-ppy/lib/python3.5/site-packages/peerplays/blockchainobject.py", line 96, in __init__
    self.refresh()
  File "/home/admin/.virtualenvs/tst-ppy/lib/python3.5/site-packages/peerplays/account.py", line 64, in refresh
    raise AccountDoesNotExistsException(self.identifier)
peerplays.exceptions.AccountDoesNotExistsException: roelandp

Are you sure all your dependencies are declared in the setup? I'm Running python version 3.5.3 on
$ uname -a
Linux ppy003 4.9.0-6-amd64 #1 SMP Debian 4.9.82-1+deb9u3 (2018-03-02) x86_64 GNU/Linux
I now suspect there is a Debian issue. This is Debian 9 'Stretch'. I had to create a symbolic link to readline.so.6 --> readline.so.7, but even with that I can't get the witness_node to compile, so the platform apparently has issues.
If it weren't for the fact people have reported docker issues on other platforms, I'd blame by docker problem on the OS as well.
I dunno what the $^%#^ is missing / screwed up, but I highly suspect the OS now, especially since xeroc has looked at this and can't duplicate the issue.
Seems real odd it works for some accounts but not others, but with this extensive list of Stretch issues I have serious doubts about Debian's latest OS release. See the notes about it here: https://www.debian.org/releases/stretch/arm64/release-notes/ch-information.en.html

Baxter is the public testnet, not main net.
Make sure you connect to a node that talks to the public testnet.
From your quote, I see init0 has PPY. Those don't exit on baxter! There they core token is called PPYTEST (IIRC)

Why do you assume I want to connect to Baxter? The witness name I'm using is on main net, and the api nodes are as well.


Green Screen Android System Webview

Hello. There are complaints from users about the appearance of a green screen during playback. The problem is on newer versions of Android. We managed to find out that this happens after updating the Android System Webview. Removing Android System Webview updates solves the problem. But Android System Webview is updated quite often and the problem arises again. Tell me what can be done?

Hi, I don't have a good answer to your question, the issue doesn't seem to be with the library but with Webview.
How Webview is used in the library is open source, in case you figure more about this and/or have changes to suggest let us know.


[WIP] Creating community page

We could include some suggestions from Matt Rocklin's blog post, which provides an excellent outline on how to file a good issue: http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports or we could just link to it.

Linking to it seems sufficient to me.

Looks like we both added the link at the same time!  I've removed that an am happy to merge it now.

I'll just check it for look and feel.

LGTM - feel free to merge.


simplify fs code:

@coderanger said: shell_out('mount', '-o', "remount,#{fields[3]}", mount, returns: [MOUNT_EX_FAIL]).error? to simplify code

I think the code is simplified now?

shell_out('mount', '-o', "remount,#{fields[3]}", mount, returns: [MOUNT_EX_FAIL]).error?
will treat a return code of 0 or FAIL as an indication that the file system is frozen, that test is unfortunately incorrect. A zero return should indicate that the file system is not frozen.   On some of the tested systems a condition of busy o returns a code of 1 instead of FAIL which would indicate that the file system is frozen.

@MarkGibbons I didn't get a chance to test it. I am not sure the changes in the last 3 years, I would close it.

I'm adding some missing tests and cleaning up a little, it's a small change. Just dumped some notes here in case I get interrupted.

Fixed by #88


Is this implementation correct?

Hello,
Thank you for your full python implementation version of the ROUGE metric.
But I found a weird thing when I use your ROUGE evaluator.
To the best of my knowledge, by the definition, ROUGE-L has the property that its value is less than or equal to unigram F-measure.
However, in the example result of the README, ROUGE-L is higher than ROUGE-1 in all the measures. (and the same thing happens when I use it in my project.)
I think there may be some error in your implementation.
Can you check it for me? or maybe we can find it together.
Thank you

Hi,
Well, unit test ensures that values obtained via the python code is similar that the ones produces by the perl script. If the perl script is producing higher ROUGE-L, well it should be correct. Could you do the test with the pyrouge for example ?
Don't forget that ROUGE-L is not just the LCS of a text but depends on the LCS of each sentence.

Please open this problem
I have test my summary with rouge(https://pypi.org/project/rouge/) and rouge-score and they give my rouge-l < r1 which is fair enough.
While the py-rouge makes the rouge-l higher than r-1 which is not correct in my case.
I think there might be some bugs.
pyrouge :  38.91 | 16.64 | 39.58
rouge :  37.71 | 16.16  | 36.54
rouge-score :  38.55 | 16.56 | 26.45


how add menu

hi again , how add menu or bottom navigation in or button to redirect to other activity ?

Currently, there is no such functionality exposed by the library. I'll think of adding this as a feature soon.
Till then, you can import the library module to your project and edit these activities directly:
hari.allagi.MenuListActivity and
hari.allagi.ScrollableMenuActivity


06-03

There are no files added, check again all the git commands, status, add, commit, push.. and then add a PR


Full Anisotropy

At the moment, MeSigmaIDeriv, MfRhoDeriv, MfRhoDeriv will not take fully anisotropic media - this actually needs to be taken care of inside of getFaceInnerProductDeriv and getEdgeInnerProductDeriv within simpeg, but I am creating the issue here too so it is documented that the fully anisotropic inversion won't be working yet

This issue was moved to simpeg/simpeg#146


typo correction, on to non

as per the user report #6660  , so I corrected word on to non

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@chuckedmonson PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


docs: update README [skip ci]

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

v4.7.0
GitHub release

Your semantic-release bot :package::rocket:


KeyringLocked error when trying to publish (on a system with keyring and an unrelated KDE app instaled)

Thank you for Flit! I finally made a package with it, and found it very easy and polished!
Except for one issue.
On my system, keyring (and so flit) defaults to using the KDE Wallet. This is not intentional: I do some KDE application installed that requires kwallet libraries, but I don't seem to have any kwallet configuration UI to go along with it, and after some cursory research I don't want to trust kwallet to save my passwords.
Running flit publish looks like this for me:
$ flit publish
Found 17 files tracked in git                                                              I-flit.sdist
Writing generated setup.py                                                                 I-flit.sdist
Built sdist: dist/....tar.gz                                                          I-flit_core.sdist
Copying package file(s) from /tmp/tmpwwtybui_/.../....py                              I-flit_core.wheel
Writing metadata files                                                                I-flit_core.wheel
Writing the record of files                                                           I-flit_core.wheel
Built wheel: dist/...-py3-none-any.whl                                                I-flit_core.wheel
Using repository at https://upload.pypi.org/legacy/                                       I-flit.upload

(click Cancel)
Traceback (most recent call last):
  File "/usr/bin/flit", line 8, in <module>
    sys.exit(main())
  File "/usr/lib/python3.10/site-packages/flit/__init__.py", line 163, in main
    main(args.ini_file, repository, formats=set(args.format or []),
  File "/usr/lib/python3.10/site-packages/flit/upload.py", line 261, in main
    do_upload(built.wheel.file, built.wheel.builder.metadata, repo_name)
  File "/usr/lib/python3.10/site-packages/flit/upload.py", line 243, in do_upload
    repo = get_repository(repo_name)
  File "/usr/lib/python3.10/site-packages/flit/upload.py", line 121, in get_repository
    repo['password'] = get_password(repo, prefer_env=(name is None))
  File "/usr/lib/python3.10/site-packages/flit/upload.py", line 151, in get_password
    stored_pw = keyring.get_password(repo['url'], repo['username'])
  File "/usr/lib/python3.10/site-packages/keyring/core.py", line 55, in get_password
    return get_keyring().get_password(service_name, username)
  File "/usr/lib/python3.10/site-packages/keyring/backends/chainer.py", line 51, in get_password
    password = keyring.get_password(service, username)
  File "/usr/lib/python3.10/site-packages/keyring/backends/kwallet.py", line 112, in get_password
    raise KeyringLocked("Failed to unlock the keyring!")
keyring.errors.KeyringLocked: Failed to unlock the keyring!

Flit documentation mentions it uses keyring library, whose README lists many options on how to configure it from code (i.e. what Flit can do, but as a user I can't), before mentioning I can run keyring --disable or set PYTHON_KEYRING_BACKEND=keyring.backends.null.Keyring. That works for me, but the experience isn't smooth.
Perhaps Flit could handle the error more gracefully, or point to the end-user-facing keyring documentation?
It looks like this error in keyring, which best fixed in KDE (1, 2), but that's now way outside what I wanted to do with Flit. Reporting this seems like the best I can do to help with the time I have.

I think presenting a better error message in this is definitely reasonable and a worthwhile enhancement. @takluyver do you have any preferences between making the message self-contained vs linking to the existing keyring documentation?

Thanks! I agree there are probably better ways we can present this to the user. We already handle the case where keyring finds no password by falling back to an interactive prompt, so a simple option is to do the same on this error, probably along with a message saying that keyring access was denied.
It's annoying that kwallet prompts to create a keyring even when we're just looking for an existing password, but I don't think there's anything we can easily do about that. :confused:
Just to mention a few possible workarounds (not least for other people getting this error and finding this issue):

If you put your username and password (or an API token) in a ~/.pypirc file, Flit will use that in preference to keyring. Obviously putting secrets in a plain file isn't great either, but it is simple. :shrug:
Likewise, you can pass credentials in environment variables FLIT_USERNAME and FLIT_PASSWORD.
If you're happy with the idea of letting keyring store your password/tokens but don't like kwallet, you can install Gnome keyring, or another program implementing the secret service API (e.g. Keepass XC) - keyring will prefer these if you're not in a KDE session - or you can install another keyring backend.

I've also thought about making an interface to manage your PyPI credentials as a separate program, but I don't think it's possible on the generic keyring API - there's no way to list secrets, only query for a specific one. I could probably do it on the SecretStorage API, but then it's only useful for a subset of Linux users (including me, so I may get round to this one day anyway).


Detect idiom: packing a 32-bit int from 2 16-bit ints

NumericT<32> int32From2Int16(NumericT<16> lo, NumericT<16> hi){
	return lo | (hi << 16);
}

Is this some kind of compiler idiom?

IDK. May be generated (for unaligned access) and may be not. But it is surely widely used when coded manually.

Do you have some input on where it occurs?

It is a basic block for higher-level idioms. It is they that occur.

What do we produce at the moment?

Sometimes bit operations, sometimes integer arithmetics with the same effect. Sometimes they are mixed: i.e. bit shifting and + or * and |.

What do you suggest we should produce? (Do I guess right that you propose a function call?)

Yeah. Add an (possibly inline) function and use it every time the idiom occurs.


Update for JuMP 0.21

Closes #9

Codecov Report

Merging #10 into master will increase coverage by 0.79%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
+ Coverage   96.26%   97.05%   +0.79%
==========================================
  Files           1        1
  Lines         107      102       -5
==========================================
- Hits          103       99       -4
+ Misses          4        3       -1

Impacted Files
Coverage Δ

src/TravelingSalesmanExact.jl
97.05% <100%> (+0.79%)
:arrow_up:

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 299f087...79ff1a8. Read the comment docs.


can you release one new version

hello,
last version is 3 years ago, can you release one new version, our company not allowed use old version open source code.

random constraint to have, but here you go

thank you!


rolling-update: detect init system properly

Previous use of the stat module didn't work as expected so using the
linux stat command now.
Closes: #1286
Signed-off-by: Sébastien Han<EMAIL_ADDRESS>

Actually, should we be using ansible_service_mgr here instead?  It looks like it was added in https://github.com/ansible/ansible/commit/99d775a0c12ff16c53aa2a19860d9293a2ef5b3c


Insert Title Here

Describe the issue in detail here. Attach any relevant screenshots or files. The section below was automatically generated. Remove any information that you do not wish to include in the report.

Debug Data

Property
Value

Name
"Wolfram/Chatbook"

Version
"1.4.0"

ReleaseID
"00db891fc2d1dd712145aa46464ded3b71633f75"

EvaluationEnvironment
"Session"

FrontEndVersion
"Desktop: 13.3 para Linux x86 (64-bit) (July 24, 2023)"

KernelVersion
"13.3.1 for Linux x86 (64-bit) (July 24, 2023)"

SystemID
"Linux-x86-64"

Notebooks
True

DynamicEvaluation
True

SynchronousEvaluation
False

TaskEvaluation
False

Settings

Property
Value

MainTabViewApproxWidth
600

TabViewApproxWidth
FE`Evaluate[FEPrivate`If[FEPrivate`SameQ[FEPrivate`$OperatingSystem, "Windows"], 580, 660]]

HighlightColor
None

Stack Data
throwFailure
trimURL

Failure Data
<|
  "Evaluation" :> Wolfram`Chatbook`Services`getServiceModelList["PaLM", <|"Service" -> "PaLM", "ServiceConnectionRequest" -> "Chat", "PreprocessingFunction" -> Identity, "SupportedParameters" -> {"Model", "Temperature", "TotalProbabilityCutoff", "TopProbabilities"}, "ParameterMap" -> {}, "PostprocessingFunction" -> Identity, "ProcessResultQ" -> True, "ModelList" :> Enclose[ServiceExecute[Confirm[Wolfram`LLMFunctions`APIs`Common`ConnectToService["PaLM"]], "ChatModelList"]], "Icon" -> RawBoxes[TemplateBox[{}, "Serv...
  "Arguments" -> {}
  "Stack" :> Wolfram`Chatbook`Common`Private`$stack$83988
|>

Issue

This should be fixed in more recent versions of the Chatbook paclet. Evaluate the following to get the latest version:
PacletInstall["Wolfram/Chatbook", UpdatePacletSites -> True]


Fetching Athena S3 output CSV instead of iterating Queryresult will boost loading data

Feature Suggestion
Is your feature request related to a problem? Please describe.
Fetching billing data for multiple days/months via AWS Athena is extremely slow. The reason is that Athena QueryResult API is only fetching a maximum of 1000 records per invocation. It just scales enough for monthly grouping but not for daily.
Describe the solution you'd like
I found this post https://stackoverflow.com/a/64031488, which suggest fetching the S3 output CSV directly, instead of iterating over QueryResult API.
Additional context
https://stackoverflow.com/a/64031488

Hello Johannes, thanks for the new issue! Are you comfortable at all working in TS/JS? If so, would you want to start working on this implementation and we could help collaborate? Otherwise, this sounds like a great idea so we could add it to our backlog.

Honestly, I'm good at copy/paste but not in generating new code in TS/JS ☺
So, add it to the backlog, please.

Sounds good, thank you!


Latest version has not been published to docker

It seem like a few versions are missing from the dockerhub repository.
https://hub.docker.com/r/lycheeverse/lychee/tags?page=1&name=0.10

Guess that's caused by a race-condition with the release pipeline right now.
I think I can run the pipeline again for an existing tag and should work.

Triggered https://github.com/lycheeverse/lychee/actions/runs/3501392131. Let's see how it goes.
Thanks for the heads up.

Latest version is now on dockerhub, seems fixed to me ! Thanks !
Unsure if you want to do the same with v0.10.2, the version seem missing as well.

Yeah I might as well. Don't think there will be any negative side-effects.

Seems like v0.10.2 is the only one that's missing, so I triggered https://github.com/lycheeverse/lychee/actions/runs/3504163799 for that. I might have to set back latest to v0.10.3 after that.

Done. Thanks for the report.


Feedback Submission - docs - Installation - /v3/getting-started/installation

What browsers are you seeing the problem on?
Firefox, Chrome, Safari, Microsoft Edge, Other (please write below)
Feedback
On the installation page of the docs, under the “Test Your Setup” section, there is a typo in the warning/info box:
“ For more details on why these dependancies [should be -> dependencies] are used, ...”

Thanks! Fixed in #464
Next time I encourage you to submit a PR to help make our docs better, and add you to the contributor list :sunglasses:


data partitioning - Data equivalence classes + coverage based measures

Introduce data equivalence classes,
and measures based on them.
The measurement can be depth based. For example, given the data type
conditional_int:= absent | present { value: int}

with equivalence by constructors,
and the equivalence classes equivalence <0, ==0, >0 for int.
All equivalences covered with depth 1:
absent , present (0)
All equivalences covered with depth 2:
absent , present (-1), present (0), present (1)

related to #406


Allow regexp as ignored patterns

Thank's for a great piece of software!
I would like to propose a feature where also regular expressions are allowed as the ignore rules. This behavior would be consistent with diff -I ignored-regexp ....

Allow regex, or provide support for regex?
the system does not yet support it, so allowing it, wont do anything :)

Hi!
Right, initially I thought that this was as complex as adding a single flag into diff.js. But probably, at the back-end, this still can be done by changing the following loop? https://github.com/dgtlmoon/changedetection.io/blob/444510c9caebeed0638d58caced5802974afe789/backend/fetch_site_status.py#L25-L27

Yeah, I think the system can also tell to switch to regex mode, if the ignore text line starts with a /

this is merged


New package: NealArundale.AisDecoder version 3.5.149

What's changed

Resolve https://github.com/microsoft/winget-pkgs/issues/115737

Checks

[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?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

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

Microsoft Reviewers: Open in CodeFlow

This package gives a warning when starting:

I haven't used this application before. I also meet this warning when launched in Windows Sandbox.
I think it's expected. Press "OK" the application is still launch-able. But MAYBE all configs in the application will discard while quitting. According to the warning, after user first save and overwrite their new configurations to default.ini, this problem will be fixed.

We can call this a normal package operation.


"{"data":"0_SUCCESS: CLIENT_READY"} Stuck

Hello
I tried a few month ago and it was working flawlessly but when I format my windows and install it again, I cannot get past this message.
I have the extension installed correctly.
I tried everything restart pc, chrome, disabling and re-enabling.

Hi ModernLobby, if you are getting 0_SUCCESS: CLIENT_READY, then that means the extension did successfully connect to your local Discord application. The extension usually works if it gets past this step, but if it's not working on your end, the only thing I can think of on your end is that some setting might be off on Discord.
Can you play a video, open devtools again, and show me what the log is saying?

I got it working dunno why I close the cmd prompt by pressing ENTER.
And now it's showing I tried for 1 hours but nothing.
Resolved

Nice!


Add files via upload

Review updates for IQR Simulated Gamma distribution of length of stay

This is brilliant! Nice work simulating what a LOS inpatient distribution would actually look like if we pulled real data!


Alma implementation for availability#index

availability GET      /availability(.:format)                   availability#index {:format=>:json}
https://github.com/pulibrary/marc_liberation/blob/master/app/controllers/availability_controller.rb
This endpoint is used for the Orangelight show page and search results and in the browse lists.
pulibrary/orangelight#1895 has links to documentation which may be relevant
There are a lot of integration points here, and a lot of different cases in the data

[ ] Support ?id=

[x] Implement copy_number
[ ] Implement on_reserve - Blocked by https://github.com/tulibraries/alma_rb/issues/62
[ ] Implement patron_group_charged - Blocked by process needing determined in https://github.com/pulibrary/marc_liberation/issues/903
[ ] Implement "status" (big ticket) - Blocked by https://github.com/pulibrary/marc_liberation/issues/937
[ ] Implement "status_label" (big ticket) - Blocked by https://github.com/pulibrary/marc_liberation/issues/937
[x] Implement "label"
[x] Implement location.

Should NOT be 852b, this is old info from Voyager that will not be updated. Pull from the first item in the holding.

[ ] Support ?ids=
[ ] Support ?mfhd=
[ ] Support ?mfhd_serial=
[x] Support ?barcodes=
[x] Support ?scsb_id=

Barcodes/SCSB use SCSB, should work.

@mzelesky @christinach I'm trying to do on_reserve, I see the comments on https://github.com/pulibrary/marc_liberation/issues/641 but it's not clear to me what on_reserve: Y means. Does that mean it's in a course reserve?

@mzelesky @christinach I'm trying to do on_reserve, I see the comments on #641 but it's not clear to me what on_reserve: Y means. Does that mean it's in a course reserve?

Yes

Description copied here from pulibrary/alma_experimentation#30 that may be helpful:
Availability logic currently is in VoyagerHelpers. https://github.com/pulibrary/voyager_helpers/blob/37b62ad83ed2f1af6185e54534d901e23e5fdf30/lib/voyager_helpers/liberator.rb#L229
In Alma, PO lines for items can be returned in the Publishing jobs. Based on the location of a holding, Bibdata should retrieve the order status of an attached PO line. If it is an electronic resource, order status does not need to be retrieved. There are likely other criteria to look at as well.

https://github.com/pulibrary/marc_liberation/issues/629#issuecomment-696933001 gives info about Alma's availability values which may also be relevant

https://github.com/pulibrary/marc_liberation/issues/629#issuecomment-696933001 gives info about Alma's availability values which may also be relevant

@hackartisan For the availability?mfhd=
Fields we do not use, but are currently present: id, item_sequence_number, due_date, patron_group_charged, enum, label(?), location(?). (I put question marks, because I do not see them at first glance, but they seem like things we might use...)
Endpoint the fields we use are: barcode, location, copy_number, status, on_reserve, item_type, pickup_location_id, pickup_location_code, enum_display, chron, status_label


Fixed all public full Adonis bypasses

These would still be detected by Adonises remote detector but I guess it's a good idea to immediatly kick them.
Fixes:

You cannot make a new line in kick messages anymore?!!
Why the "\n" characters?


PC and Turtlebot4 communication issue

Hello all!
I have some issues with the communication of PC and Turtlebot4.
For example, when I do the ros2 topic list on PC, it doesn't show the topics of create3 while when I go to the turtlebot4 by ssh from PC it shows all the topics of turtlebot4 and create3. As shown in Fig below

Turtlebot4 and PC communicate at the rate of 5GHz while create3 works at the rate of 2.4GHz.
I want to implement SLAM and NAV2 but am not able to do that, so please help me to resolve this issue.
thank you

@Kazimbalti To debug this,

Ensure that create3 is connected to 2.4Ghz Wifi and Rpi to the 5.8 Ghz Wifi band.
Check if Domain ID is same across create3, RPi4 and your PC.
Check if your terminal is sourced properly, i.e source /opt/ros/distro/setup.bash -> I suspect this in your case

thank you @ipa-rar for your response and guidance.
I already did everything you mentioned:

create3 is connected with 2.4GHz and RPI is connected with 5GHz band
Domain ID of creat3=RPI=PC = 0
I properly sourced it, on both PC and on RPI4

Still, I am not able to get the topics on my PC, please guide me, is there anything I am missing? thank you

@Kazimbalti can you please clarify the folowing information:

OS / Containerization info:
ROS Distro:
Built from source or installed?:
Package version:
Create3 Firmware version:
Networking configuration (simple discovery or discovery server):
RMW Implementation (CycloneDDS or FastDDS):

Also, are you connecting your system to a dedicated router using the default functionality? are there any firewalls that could be present, blocking communication.
Finally, is this issue consistent even over restarting the robot and your PC and the ros2 daemon on the pi (ros2 daemon stop; ros2 daemon start)?

OS / Containerization info: Ubuntu 22.04.3
ROS Distro: Humble
Built from source or installed: Source
Package version: H.2.2
Create3 Firmware version: H.2.2
Networking configuration: simple discovery
RMW Implementation: FastDDS

Yes, I have connected the RPI with 5GHz while the create3 is connected with 2.4GHz.

In connect to Robot to Wi-Fi it doesn't show IP address but from the application when I send commands it works like restart etc,

There are no firewalls present/active,

I restarted the PC and turtlebot4 many times,
This is my ~/.baschrc file

I have been working on it for two weeks, I even tried on Galactic, and same issue there as well.
When I will give any commands or command velocity from RPI/ubuntu, it works fine but when I give any commands from PC it doesn't respond because it doesn't get any topics from create3. Please help me resolve these issues, I have two turtlebot4 and both have the same issue. I want to implement SLAM, NAV2 etc,
thank you

Your create3 is not connected to the wifi. You are seeing some topics show up with no messages because the diagnostics on the raspberry pi are subscribing to those topics. You are connecting to the create3 webserver through the pi as well.
In order to fix the create3 wifi connection, please hold the • and •• buttons for a few seconds until you get the spinning blue / cyan light. and then either connect to the create3 webserver using the AP that shows up or through the raspberry pi and reconfigure the wifi connection. The connect tab should show you a drop down of available networks and the SSID should be filled in on that page after it connects successfully. If you are unsuccessful following this please provide the create3 log.

This issue is being closed due to inactivity. If you are still experiencing the issue, feel free to reopen this ticket when you are ready to continue the troubleshooting process.


Changelog Entry

Added entry for the work in this branch.
@apotterri

Hmmm. I'm thinking we should really rev the gem, rather than just marking it WIP. 5.0.0-beta.3, maybe?


StreamPipeReader should return the buffered data even after the Stream is done

Describe the bug
The StreamPipeReader should return the buffered data from the stream even after the underlying Stream has been drained.
To Reproduce
using System;
using System.Buffers;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using Microsoft.AspNetCore.Builder;
using Microsoft.AspNetCore.Hosting;
using Microsoft.AspNetCore.Http;
using Microsoft.Extensions.DependencyInjection;

namespace WebApplication101
{
    public class Startup
    {
        // This method gets called by the runtime. Use this method to add services to the container.
        // For more information on how to configure your application, visit https://go.microsoft.com/fwlink/?LinkID=398940
        public void ConfigureServices(IServiceCollection services)
        {
        }

        // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
        public void Configure(IApplicationBuilder app, IHostingEnvironment env)
        {
            if (env.IsDevelopment())
            {
                app.UseDeveloperExceptionPage();
            }

            app.Use(async (context, next) =>
            {
                if (context.Request.ContentLength is null)
                {
                    await next();
                    return;
                }

                await context.Request.BufferAsync();

                await next();
            });

            app.UseRouting(routes =>
            {
                routes.MapGet("/", async context =>
                {
                    await context.Response.WriteAsync("Hello World!");
                });

                routes.MapPost("/api/values", async context =>
                {
                    var result = await context.Request.BodyPipe.ReadAsync();
                    var buffer = result.Buffer;

                    var value = Encoding.UTF8.GetString(buffer.ToArray());

                    await context.Response.WriteAsync(value);
                });
            });
        }
    }

    public static class RequestBufferingExtensions
    {
        public static async Task BufferAsync(this HttpRequest request)
        {
            // Buffer the entire request body into the pipe
            while (true)
            {
                var result = await request.BodyPipe.ReadAsync(request.HttpContext.RequestAborted);
                var buffer = result.Buffer;

                if (result.IsCompleted)
                {
                    break;
                }

                request.BodyPipe.AdvanceTo(buffer.Start, buffer.End);
            }
        }
    }
}

The above code tries to buffer all of the pipe's contents middleware before the post body is processed.
Expected behavior
I expected to get the entire post body in /api/values, instead I get a 0 byte read.
Screenshots

Notice the buffered bytes is non-zero but the buffer returned is empty.
Additional context
latest build

cc @jkotalik

I'll take this one.


[Bug] Activating transparency in BD causes window snapping to break in Windows 10

Before Reporting

[X] I found no existing issues matching my bug
[X] My issue is not caused by a plugin or theme

Describe the Bug
Activating "window transparency" in BD and reloading Discord causes the window snapping feature to stop working, as seen in the video attached below.

BD version: Core v1.9.7 (all themes and plugins switched OFF)
Discord: Stable 256231 (96a39c5)
OS: Windows 10 64-Bit (10.0.19045)

Thanks
Expected Behavior
For window snapping to work like usual
Steps To Reproduce

Switch "Enable Transparency" option ON under BetterDiscord settings
Restart Discord
After it restarts, try snapping the Discord window to any of the monitors edges to see that it no longer functions

Operating System
Windows
Discord Version
Stable
BetterDiscord Version
v1.9.7
Anything else?
https://github.com/BetterDiscord/BetterDiscord/assets/143902524/9e983bbe-deda-4a16-abc3-6baa973c16ee
(The only reason for moving the Discord window to my other monitor after reloading was in order to navigate back to the settings menu, in order to hide my friends list and server list – I didn't change anything during that time)

This is a problem with the way Electron handles transparency - it is not specific to BD.

I see, thank you for the response.

I see, thank you for the response.

I see, thank you for the response.


Add image for fedora 40

i have a system running fedora 40. the documentation says that support for fedora is wip, and the supported versions are from 34-37. since it is not feasible for me to install ubuntu, i decided to compile sysbox from source. copied the Dockerfile for 37, added libseccomp-static in the dnf install step, and sysbox seems to be running just fine on fedora 40.
happy to update the doc as well if needed.

happy to update the doc as well if needed

No need to update the docs yet; support for Fedora (and friends) is WIP because we don't have a package installer (RPM) for those distros yet. But otherwise Sysbox should work fine in them.


Automate Deployment Process using GitHub CI

Talking with @znarf about the deployment process I had the following idea. 💡
Currently the deployment process involves a number of manual steps as per, https://github.com/opencollective/opencollective-frontend/blob/33535f88e79921d89daa8cf0e6c0d0e3346f076c/docs/deployment.md. And we have to wait until the process runs in the local terminal. I would suggest automating it with a GitHub workflow. So my suggestion is as follows. The person who wants to deploy will just create a release tag on GitHub and a workflow will run on the release tag and we will configure all secrets for the deployment in GitHub secrets. So for example if the release is 2.0.0-staging it will deploy to staging, if it's 2.0.0-prod it will deploy to prod and so on. Let me know what you think or if this is useful to have. 😄
cc: @opencollective/core-developers

The pain is not really to type npm run deply:staging and wait. One problem is that the app needs to be built at deploy time.
It could be interesting to see what we could do to have 1-3 minutes deployments, instead of the current 15 minutes or so.

The pain is not really to type npm run deply:staging and wait. The problem is more that the app needs to be built at deploy time.
It could be interesting to see what we could do to have 1-3 minutes deployments, instead of the current 15 minutes or so.

@znarf : Thinking about this; I think we should try caching on Heroku for nextjs build as per recommended here; https://nextjs.org/docs/messages/no-cache#heroku. Have we tried this before; if not I can enable it and we could give it a try. 😄

Not sure we could do anything about this at this point; therefore I am closing this. 🤔


Bug : Issues on Component > Data table > Action modal issue

Action modal should get closed once click outside the modal and it should open in left side


Update import on nextjs-pages index

Chore
Update the import to auth-helper-nextjs for the clarity of the example
Describe the chore
Modify index.tsx import from:
import {
Session,
createPagesBrowserClient,
createPagesServerClient
} from<EMAIL_ADDRESS>to
import {
Session,
createPagesBrowserClient,
createPagesServerClient
} from '@supabase/auth-helpers-nextjs';
Additional context
This should provide added clarity and consistency in the example provided.

Can you point me to where you've found this example code please?

examples/nextjs-pages/pages/index.tsx


Failed to activate the northem-light-atom-syntax theme

This error occurs when the following steps are taken:

Install northem-light-atom-syntax theme 0.4.0
Try to change to this syntax theme from the Atom settings as usual

Atom Version: 1.0.5
System: Microsoft Windows 10 Pro
Thrown From: northem-light-atom-syntax package, v0.4.0
Stack Trace
Failed to activate the northem-light-atom-syntax theme
At variable @dirty-white is undefined in C:\Users\pauli\.atom\packages\northem-light-atom-syntax\styles\colors.less:130:34

LessError: variable @dirty-white is undefined
  at C:\Users\pauli\.atom\packages\northem-light-atom-syntax\styles\colors.less:130:34

Commands
     -0:11.6.0 settings-view:uninstall-themes (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "northem-light-atom-ui",
      "northem-light-atom-syntax"
    ]
  }
}

Installed Packages
# User
color-picker, v2.0.11
file-icons, v1.6.3
linter, v1.3.4
linter-jshint, v1.1.5
northem-light-atom-ui, v0.1.0

# Dev
No dev packages

Thanks for reporting! :
Released a fixed apm package update 0.4.1 :smile:

Great stuff. It`s after all my favorite Atom theme. Keep up the good work.


cluster-autoscaler

What this PR does / why we need it
Issue: https://github.com/giantswarm/roadmap/issues/2927
Prepare CR's to allow managing MachinePools by cluster-autoscaler.
Cluster-autoscaler watches ASG's on two tags

sigs.k8s.io/cluster-api-provider-aws/cluster/CLUSTER_ID <- already an ASG tag
k8s.io/cluster-autoscaler/enabled: "true" <- needs to be created by capa-controller because it's not present yet.

I tried to make an overview how it works:

capa-controller will only adjust the min and max size for MachinePools in case cluster owner modifies AWSMachinePool CR and updating the status of the current replicas
cluster-autoscaler will take ownership of scaling in and out according to it's calculation in the workload cluster.

Once released we can manage scaling for MachinePools by cluster-autoscaler.
Checklist

[x] Update changelog in CHANGELOG.md.

Trigger e2e tests

/run cluster-test-suites

/run cluster-test-suites

Same here: I don't know why but the cluster (CAPA Upgrade Suite) cannot be deleted in time but I doubt it has something to do with cluster-autoscaler

/run cluster-test-suites

/run cluster-test-suites


Make the KB a REST service

Making the KB a REST service, exposing views through its API. The source should be kept in efficient data storage (maybe couchdb or mongodb?)

Done in MAPE-knowledge-base-service


When the tab title is in Chinese, the Form Tab is not separated correctly.

Package
filament/filament
Package Version
V3.2.0
Laravel Version
v11
Livewire Version
No response
PHP Version
PHP 8.3
Problem description
When I have two tabs in the form, if both tabs have Chinese titles, then their contents are merged into one, and both tabs are in a selected state.

Expected behavior
When using tabs, Chinese characters should be handled correctly.
Steps to reproduce
Tabs\Tab::make(__('基本信息'))->schema([
TextInput::make('title')
                            ->label(__('Post Title'))
                            ->required()
                            ->maxLength(255)
                            ->live(onBlur: true)]),
Tabs\Tab::make(__('(E)扩展信息'))->schema([
                        Placeholder::make(__('扩展信息')),

                        Hidden::make('user_id')
                            ->default(auth()->user()->id)
                            ->required(),

                        Hidden::make('post_type')
                            ->default('post')
                            ->required()])

When I use the above code, the error occurs. If I change it to the following code, the result is correct.
Tabs\Tab::make(__('Title '))->schema([
TextInput::make('title')
                            ->label(__('Post Title'))
                            ->required()
                            ->maxLength(255)
                            ->live(onBlur: true)]),
Tabs\Tab::make(__('Other Info '))->schema([
                        Placeholder::make(__('扩展信息')),

                        Hidden::make('user_id')
                            ->default(auth()->user()->id)
                            ->required(),

                        Hidden::make('post_type')
                            ->default('post')
                            ->required()])

Reproduction repository (issue will be closed if this is not valid)
https://github.com/cityisempty/filament-issue
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

The issue comes from
forms/src/Components/Tabs/Tab.php
infolists/src/Components/Tabs/Tab.php

    final public function __construct(string $label)
    {
        $this->label($label);
        $this->id(Str::slug($label));
    }

Str::slug($title, $separator = '-', $language = 'en', $dictionary = ['@' => 'at'])

The language parameter defaults to en, and when encountering Chinese characters, it results in an empty string.
I recommend modifying it to:
$this->id(Str::slug(Str::transliterate($label, strict: true)));


Enable UNIT_gz_TEST on Windows

🦟 Bug fix
Summary
https://github.com/gazebosim/gz-plugin/pull/63 fixed installing ruby files in windows, but UNIT_gz_TEST was still not running.
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

It seems like the UNIT_gz_TEST is not running on windows even with this fix. @Voldivh do you mind creating an issue in gz-plugin?


WIP: Nginx mii new it

This PR is to add Nginx ingress controller functionality in new integration test and use Nginx to access the sample apps in the model-in-image domain.

This is so far the best test result from Jenkins:
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-model-in-image-tests/590/testReport/oracle.weblogic.kubernetes/

In this run, https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-model-in-image-tests-10/, ItMiiDomain passed. However ItSimpleNginxValidation did not run since NGINX pod stuck in the pending state and BeforeAll failed.

got a clean Jenkins job finally:
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-model-in-image-tests-10/15/testReport/oracle.weblogic.kubernetes/


Add NIM flag logic

Add NIM flag logic
Description
This will get the nim flag state of removed or managed from params.env and add to the environment variables for odh-model-controller to be utilize
How Has This Been Tested?
Unit tests

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

@spolti do you know if I need to change something for the image tag failure?

/ok-to-test


Do not change default replication size.

This PR removes the command to change the replication size to 1. Now that we can have OSDs backed by loop files, this setting doesn't make sense anymore.

Closed, as the branch is locked.


fixed parsing ttl from json message

Fixes #153

Thanks @aanodin - great catch and valuable contribution 👍

Do you need a new release by the way? Then I would need to update to latest Zeebe versions. Please ping me in https://github.com/camunda-community-hub/kafka-connect-zeebe/issues/156 if you would need such a release


There is case that does not close pooled connections

MyBatis version
3.4.1
Database vendor and version
MySQL Oracle community edition version 5.7.13
Test case or example project
https://github.com/mberta/mybatis-test-mysql-aborted-connection
Steps to reproduce
Run the example project
Expected result
Actual result
The example project executes a dummy SELECT 1 statement against a mysql server and it generates an error message in mysql server log like this:
2016-11-03T11:30:44.894384Z 2558213 [Note] Aborted connection 2558213 to db: 'COPERNICO' user: 'copernicodba' host: '<IP_ADDRESS>' (Got an error reading communication packets)

Don't know if this is an issue or not, but it seems to me that mybatis shouldn't generate an 'aborted connection' as it does in the example project I've attached.
I should add that changing datasource type from POOLED to UNPOOLED makes the problem disappear.

@mberta Thanks for reporting !!
I've confirmed that does not close a pooled connection in this case.
The PooledDataSource close pooled connections at the finalize method. However in this case, the finalize method does not called. As result, pooled connections has been aborted during shutdown a JVM.
I will proposed a solution to prevent this issue as workaround.
Please close pooled connection calling the PooledDataSource#forceCloseAll method on your application shutdown phase as follow:
((PooledDataSource)sqlSessionFactory.getConfiguration().getEnvironment().getDataSource()).forceCloseAll();

We will consider a best solution to prevent this issue for next release.

@kazuki43zoo the problem seems not related to the shutdown of the JVM. In the snippet of code I've posted, if you call pingDb() method multiple times you should notice a message in mysql server log for every call of the pingDb() method.

Hi @mberta,
Sorry, i don't understand your comment.
In my understanding, your application should be closed pooled connections before shutdown of the JVM. However, current implementation of the PooledDataSource does not close pooled connections automatically. I think this implementation is root cause of this issue.
I think this issue can be prevented adding following code (This solution is workaround):
    private void pingDb() throws IOException {
        SqlSessionFactory sqlSessionFactory = getSqlSessionFactory();

        try (SqlSession session = sqlSessionFactory.openSession()) {
            Mapper mapper = session.getMapper(TestMySqlAbortedConnection.Mapper.class);
            mapper.pingDb();
        } finally {
            // Add following code
            ((PooledDataSource)sqlSessionFactory.getConfiguration().getEnvironment().getDataSource()).forceCloseAll();
        }

    }

or
    private SqlSessionFactory getSqlSessionFactory() throws IOException {
        InputStream inputStream = Resources.getResourceAsStream("mybatis-config.xml");
        SqlSessionFactory sqlSessionFactory =
                new SqlSessionFactoryBuilder().build(inputStream, System.getProperties());

        Configuration configuration = sqlSessionFactory.getConfiguration();
        configuration.addMapper(Mapper.class);

        // Add following code
        Runtime.getRuntime().addShutdownHook(new Thread() {
            @Override
            public void run() {
                ((PooledDataSource)sqlSessionFactory.getConfiguration().getEnvironment().getDataSource()).forceCloseAll();
            }
        });

        return sqlSessionFactory;
    }

@kazuki43zoo, you're perfectly right! I was misunderstanding your answer

@kazuki43zoo does this bug affect more than just 3.4.1?

@Quantas Probably yes, this issue affect all versions.

Hi, @kazuki43zoo is the bug fixed?


Allow users to select form labels

Reviewers: By approving this Pull Request you are approving the code change, as well as its deployment and mitigation plans.
Please read this description carefully. If you feel there is anything unclear or missing, please ask for updates.
What are you trying to accomplish?
I’ve received some internal reports that not being able to select and copy labels in settings is frustrating. This seems to be a behaviour limited to PVC and isn’t consistent with PRC.
This PR removes the user-select: none rule applied to .FormControl-label to allow selection.
Screenshots
Example of the current issue and inconsistency:
https://github.com/primer/view_components/assets/6905903/5e6010e5-a225-4ac0-9391-49137cda6007
Risk Assessment

[x] Low risk the change is small, highly observable, and easily rolled back.
[ ] Medium risk changes that are isolated, reduced in scope or could impact few users. The change will not impact library availability.
[ ] High risk changes are those that could impact customers and SLOs, low or no test coverage, low observability, or slow to rollback.

While it’s possible this was added for a Very Good Reason, I scoured the git history fpr PCSS and PVC and couldn’t find where it was introduced (only where the whole file was added here).
There’s a small chance that removing this makes it easier for users to accidentally select or focus a form element when they only mean to select its label, but I think that’s unlikely and a worthwhile tradeoff. That this rule doesn’t exist in PRC further indicates it might not be that important to us.
Accessibility

No new axe scan violation - This change does not introduce any new axe scan violations.

Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Added/updated previews (Lookbook)
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

From the preview deployment, maybe this is what we were trying to prevent? Seems unlikely someone would be spamming clicks like this though.
https://github.com/primer/view_components/assets/6905903/bb8b83d4-22c0-4f77-ae8d-1c4036e9544e
https://github.com/primer/view_components/assets/6905903/904eb7c1-84cf-4b5e-97ec-765cb652ef31