Store is initialized empty after updating from angular 2.1 to 2.2

In 2.1, my reducers/index.ts looked like this:
// reducers index.ts
import { compose } from '@ngrx/core/compose';
import { combineReducers, ActionReducer } from '@ngrx/store';
import reducer1 from './reducer1';
import reducer2 from './reducer2';
import reducer3 from './reducer3';
import reducer4 from './reducer4';

export default compose(combineReducers)({
  reducer1,
  reducer2,
  reducer3,
  reducer4
});

After updating to 2.2, I started seeing an issue which I was able to resolve with help from an issue here:  https://github.com/ngrx/store/issues/190
So, I had to update my reducers/index.ts to this to not get any more errors:
import { compose } from '@ngrx/core/compose';
import { combineReducers, ActionReducer } from '@ngrx/store';
import reducer1 from './reducer1';
import reducer2 from './reducer2';
import reducer3 from './reducer3';
import reducer4 from './reducer4';

export function reducers(): ActionReducer<any> {
  return compose(combineReducers)({
    reducer1,
    reducer2,
    reducer3,
    reducer4
  })
};

Unfortunately, now my store is empty and I receive errors like this: TypeError: Cannot read property categories of undefined
So I followed some more advice from that same issues thread, but now my store initializes empty.  Is there something that I am missing?
Thanks

Have a look at this and tell me if it works for you 👍

@bertofer Will let you know ASAP.  Unfortunately away from my dev machine right now, sorry about that.  Thank you for your response though, I really appreciate it.

Whoah!  Awesome!
That fixed it.  Thanks bud!


I don't know why, the images I generated were messy

I don't know why, the images I generated were messy

you need to update comfyui


fixed problem with saving multiple repeaters closes #59 closes #167

This might also solve #37 -- testing now and will merge if all goes well
Thank you!


docs: composition api example

https://github.com/ankurk91/vue-loading-overlay/issues/66#issuecomment-894211288

Can not merge this unless we fix this issue
https://github.com/ankurk91/vue-loading-overlay/issues/88

https://github.com/ankurk91/vue-loading-overlay/pull/90


OpenVEX Specification v0.2.0

This PR updates the OpenVEX specification to v0.2.0, a breaking change that expands the product and vulnerability fields to make them more expressive and better suited for future expansion.
This PR incorporates the changes discussed and approved by the SIG as part of the following OpenVEX Enhancement Proposals:

OPEV-0014: Expansion of the VEX Product Field
OPEV-0015: Expansion of the Vulnerability Field

/cc OpenVEX Maintainers: @lumjjb @wagoodman @luhring @rnjudge
/cc OpenVEX SIG OPEV Contributors: @sudo-bmitch @SecurityCRob
Signed-off-by: Adolfo García Veytia (Puerco<EMAIL_ADDRESS>

PR is open implementing these changes in the go-vex libraries:  https://github.com/openvex/go-vex/pull/45

@puerco I've opened up #37. I found several other things to cleanup from the linter, hopefully they are all easy to review since a good number of changes were small things like trailing spaces.


tests: spinlock: Add description and doxygen groups

Add test description, RTM links and doxygen groups
for spinlock tests
Signed-off-by: Spoorthi K<EMAIL_ADDRESS>

Codecov Report

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

@@            Coverage Diff            @@
##             master    #9011   +/-   ##
=========================================
  Coverage          ?   52.29%
=========================================
  Files             ?      197
  Lines             ?    24803
  Branches          ?     5157
=========================================
  Hits              ?    12970
  Misses            ?     9754
  Partials          ?     2079

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 eda3e16...db5d7a0. Read the comment docs.


not to much serious interactive layer

it's about the enter on the numeric pad (logitech g910)
while i hit (interactive layer) it does nothing (in the numeric  section) But the "enter" in letter section interact!
https://www.youtube.com/watch?v=iodcjfe-8Vg
as you will see on this short video!
thank you all!

Windows does not differentiate between Numpad Enter and regular Enter. Not much I can do about it.

but on my lg sofware the numeric enter work :s

LGS's key effects also work with G-keys, and those are 100% not part of Windows. They have access to different techniques than I do. So, they can do more refined effects.

ohh  i see   ;P thank you


getAreaOfPolygon returns a number when I pass an unclosed area

Hi dear reader, When I execute this code IDK why but for some reason for the second one also returns a number though it is not even a closed area. So my question is that if this is an expected behaviour and OK. BTW based on my understanding it should not work like that, I mean it is not even a closed are to begin with:
const geolib = require("geolib");

// 1
console.log(
  geolib.getAreaOfPolygon([
    [7.453635617650258, 51.49320556213869],
    [7.454583481047989, 51.49328893754685],
    [7.454778172179346, 51.49240881084831],
    [7.453832678225655, 51.49231619246726],
    [7.453635617650258, 51.49320556213869],
  ])
);

// 2
console.log(
  geolib.getAreaOfPolygon([
    {
      latitude: -72.2781577,
      longitude: 42.9288483,
    },
    {
      latitude: -72.2864358,
      longitude: 42.928252,
    },
    {
      latitude: -72.2877233,
      longitude: 42.9247012,
    },
    {
      latitude: -72.2817152,
      longitude: 42.9237898,
    },
  ])
);

Here is what I meant by not closed area, as you can see in the uploaded image the starting point - the first point on the right top corner - is not even close to the ending point -- the last point in the bottom right corner:

Here is my work env conf:
geolib: 3.3.4
node: 20.3.1
npm: 9.6.7
OS: Darwin Kernel Version 22.5.0 arm64

OK, I am coming from the future. In this part of code we are closing it https://github.com/manuelbieh/geolib/blob/26ec179e861750623d496a962465e5c399664efd/src/getAreaOfPolygon.ts#L20
Though IMO this is not OK, and also the documentation lacks clarity about these kind of underlying implementations.
I hope someone will just address this GitHub issue


Convenience methods for one-player GUIs

Description
Sometimes, GUIs are only meant to be used by one player. However, since the API is designed to work with multiple players, it can be cumbersome when you know beforehand that only one player should see the GUI at a time.
This could be accomplished by adding convenience methods, like Gui#getViewer() which would return the first viewer in the viewers list (equivalent to gui.getViewers().get(0)).
Another idea is to introduce a ViewerPolicy, such as EVERYONE (the default), FIRST_ONLY (only the first to open it), SINGLE (when a new viewer is added, the old one gets removed).
Challenges

Handling changes of the ViewerPolicy (e.g. switching from EVERYONE to SINGLE)

Current Status
A Gui#getViewer() method was implemented and will be available as of version 2.1.0.


phpfpm static files 404

Can't access to static files, every files are in 404 and served as text/html

Hi, @allrib.
Could you please provide some steps in order to reproduce the issue?
Regards,
Alejandro

Hi @alemorcuq
I followed step by step this tutorial : https://docs.bitnami.com/kubernetes/how-to/deploy-php-application-kubernetes-helm/ .
When I attach a simple css file like that in php file :

My PHP file is 200 OK.
My CSS, JS are in 404 error.

Hi, @allrib,
In which step of the tutorial does it happen? Are you building the docker image again after attaching your files?

@alemorcuq
The problem is at the end of tutorials,
I did docker build . -t  MYUSERNAME/phpfpm-app:0.1.0 in tutorials/phpfpm-k8s/ after attaching my files
and I launched helm install --set mariadb.mariadbRootPassword=DB_ROOTPASSWORD,mariadb.mariadbUser=DB_USERNAME,mariadb.mariadbPassword=DB_USERPASSWORD,mariadb.mariadbDatabase=DB_NAME --name phpfpm .
When I go to http://IP:PORT I get for example version.php 200 OK and test.css, test.js error 404

@alemorcuq
Another problem I think : css, jss files are finally served as text/html. Even though I obtain an error 200 on my statics files, they are interpreted as text/html.
Can you take a look on that ?
Regards,
Alexis

You will have to mount your static files to the /app directory of your nginx container.

@alemorcuq Any idea how to do that?

You will have to mount your static files to the /app directory of your nginx container.

@alemorcuq Any idea how to do that?

How are you deploying Nginx?

helm install --set mariadb.mariadbRootPassword=DB_ROOTPASSWORD,mariadb.mariadbUser=DB_USERNAME,mariadb.mariadbPassword=DB_USERPASSWORD,mariadb.mariadbDatabase=DB_NAME --name phpfpm .
when i enter this command it give me the following error
Error:unknown flag --name

--name is not a valid flag in Helm 3, if you are using Helm 3 you don't need to use this flag, only provide the chart name:

Random name:

▶ helm2 install bitnami/fluentd
▶ helm3 install --generated-name bitnami/fluentd

Defined name:

▶ helm2 install --name myChart bitnami/fluentd
▶ helm3 install myChart bitnami/fluentd


Allow LiveLogger to be initialized without the need of NodejsPackage.Instance.

This is done by moving _diagnosticsOptions initialization away from LiveLogger constructor.
Fixes https://github.com/Microsoft/nodejstools/issues/481.

minor comment and lgtm after that


[fix] Fix golangci-lint tests

Signed-off-by: Kapil Bansal<EMAIL_ADDRESS>Description
This PR fixes #6385
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Tests passed when run with these changes :- https://github.com/meshery/meshery/actions/runs/3388651198/jobs/5630881089

The golang-ci was rollbacked to a previous version because it was causing issues with go v1.18 or below. But we recently upgraded to go v1.19 where v1..47.3 don't work

Hi @acald-creator I doubt if directly using master will be good or not. There are chances that it may break. For now, I am pinning it to v3

@devkapilbansal we like to live dangerously (and reduce maintenance overhead where possible). @acald-creator is a faithful steward of the Meshery project. I encourage you to lean into any guidance that she might offer.

Sure, I will


Problems with figures

Guess I should have created different topics from past issues too to better organize this section - sorry.
@Emmanuelle7 said the figure lengths have been fixed yesterday but I'm trying again the code this morning and I have the same problems with figures 4-7 (content and legend are not good).
By the way, when I (re-)install COVIDreportwritrer, I choose to update All packages.
And when it asks me if I want to install from sources the packages which need compilation, I say no (because I said yes in the past and the rest didn't work afterwards as I recall).
Thanks for your support!
Problems.with.figures.pdf
Log 20200508.pdf

Hi Jason, you seem to be facing the old problem. See here: https://github.com/ISARICDataPlatform/ISARIC-COVID-reports/issues/1#issuecomment-624506672

Oops yes :).
Works much better!
Thanks!!
But the axis titles in Figure 5 are still a bit overlapping.
Figure 5.pdf

@jasonbouziotis thanks for the feedback. This has now been fixed. Do you want to run again? Do remember #1.

Doesn't work for me today.
The old folder is not in my working directory (does it matter if it's somewhere else? - I wouldn't think so..).
I can't even generate the report anymore...
Log 20200513.pdf
I'll try another day with an updated export because we are now updating our REDCap project to reflect the changes in the eCRF that occurred this Sunday.
I wonder if those changes will affect the working of the code.

Sure. Let us know whether the updated CRF solves this.


Which version of net-snmp is depended by this gem on windows xp?

I installed this gem on windows xp, but I tried and failed. such errors occur.
"Function 'snmp_close_sessions' not found in libnetsnmp ffi::NotFoundError"
I have no idea which version of net-snmp  should be installed.

now I resolved this problem. I  compiled net-snmp with netsnmp.def modified by myself.


Logic fix in Map.Digger._getDiggingDirection.

The y-coordinate is now compared with height instead of width. As a micro-optimization, I've also pulled the comparison out of the loop.

Thanks!


error

...AddOns/Journalator/Source/Monitor/LootContainers.lua:120: assertion failed!
Count: 3
Call Stack:
[string "=[C]"]: ?
[string "=[C]"]: in function assert' [string "@Interface/AddOns/Journalator/Source/Monitor/LootContainers.lua"]:120: in function TagLooted'
[string "@Interface/AddOns/Journalator/Source/Monitor/LootContainers.lua"]:22: in function <...AddOns/Journalator/Source/Monitor/LootContainers.lua:16>

Do you know what you were looting when this error happened?

I play in the TW server and make a task in the first picture.

If you get a way to replicate this bug report back, but for now there isn't much I can do.


CIV-6398 Remove CVE

Before creating a pull request make sure that:

[x] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] Screenshot attached for the Evidence of manual testing
[ ] Functional, Smoke and API tests have been run locally
[x] Full gradle build and run tests with coverage have been completed (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-6398
Change description
Remove CVE
Testing branch and instruction
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No


Fails to build with GCC 14

As reported in Debian Bug #1075423 shapely fails to build with GCC 14:
src/ufuncs.c: In function ‘polygonize_func’:
src/ufuncs.c:2205:51: error: passing argument 2 of ‘GEOSPolygonize_r’ from incompatible pointer type [-Wincompatible-pointer-types]
 2205 |     GEOSGeometry* ret_ptr = GEOSPolygonize_r(ctx, geoms, n_geoms);
      |                                                   ^~~~~
      |                                                   |
      |                                                   GEOSGeometry ** {aka struct GEOSGeom_t **}

See also: https://gcc.gnu.org/gcc-14/porting_to.html#incompatible-pointer-types

gcc-14 is the default compiler in Debian unstable now, the use of -Wno-error=incompatible-pointer-types is required until a proper fix for this issue is available.

The problem is that the second argument of GEOSPolygonize_r is defined as const GEOSGeometry * const geoms[], and not GEOSGeometry** geoms (as it is done in some other functions taking an array of geometries, and how we create and pass it in our code).
Not directly sure about the best C way to resolve this.
https://github.com/shapely/shapely/blob/9a8bf73ecf2ef51d684b5866ab5718220b402640/src/ufuncs.c#L2275-L2299

This should already be fixed in main by https://github.com/shapely/shapely/pull/1945, which was merged on 2024-12-05 as<PHONE_NUMBER>0680728dc751dc5888cd02086d908e but never cherry-picked over to maint-2.0.

Good catch, thanks @musicinmybrain. We should indeed ideally have backported that.

Good catch, thanks @musicinmybrain. We should indeed ideally have backported that.

I did that now for the case we would have to do another 2.0.x bug-fix release, we would at least not forget to include it

Given that this is fixed, at least for the upcoming 2.1 release, closing this issue.


Do you support webp picture format?

Do you support webp picture format?

And cache webp picture format


Added degrees and radians as units to Scala reader/writer.

Default unit is radians (better for math).
Allowed variants are "deg", "degree", "degrees", "rad", "radian", "radians" with the 3-character forms taking the short SI prefixes (e.g. "mrad") and the others taking the long form (e.g. "centidegrees").
Documentation is not updated to reflect these as this is a proposal rather than a final decision on how to support angular units.
Note that supporting revolutions would be easy also at least in the Scala code base.

Review by @MichaelCurrie

Are we going to normalize all radians to the [0,2π) interval (or perhaps to [-π,π)?)

@MichaelCurrie - I would say no, for two reasons.

We may have measures of cumulative angle where 41.55π is meaningful.
It doesn't simplify things much since as soon as you start doing any math with radians you have to deal with the circular nature of it.  For instance, -179.999 degrees is very close to 179.999.

@Ichoran ok makes sense, I will not normalize in my code either.
Each line in the code diff summary makes semantic sense to me.  Assuming this is compiling for you, I'm going to merge.


Error while loading shared libraries: /lib/libwasmvm.x86_64.so: invalid ELF header

What is happening?

Section description
Provide as much context as you can. Provide relevant software versions, screenshots, copy & paste error messages and so on.
Give as much context as you can to make it easier for the developers to figure what is happening.

Some new installations are receiving the error that the libwasmvm library is an invalid ELF header. This is usually related to the file not being found.
palomad.service: Failed with result 'exit-code'.
palomad.service: Scheduled restart job, restart counter is at 4.
Stopped Paloma Node.
Started Paloma Node.
/usr/local/bin/palomad: error while loading shared libraries: /lib/libwasmvm.x86_64.so: invalid ELF header
linux version 20.04
How to reproduce?

Section description
Please write detailed steps of what you were doing for this bug to appear.

Run new install of Palomad
What is the expected behaviour?

Section description
If you know, please write down what is the expected behaviour. If you don't know, that's ok. We can have a discussion in comments.

sudo wget -P /usr/lib https://github.com/CosmWasm/wasmvm/raw/main/internal/api/libwasmvm.x86_64.so
should function without issue on new install. All links should function.

OS: Ubuntu 20.04
results: palomad.service: Failed with result 'exit-code'.
palomad.service: Scheduled restart job, restart counter is at 4.
Stopped Paloma Node.
Started Paloma Node.
/usr/local/bin/palomad: error while loading shared libraries: /lib/libwasmvm.x86_64.so: invalid ELF header
Valoper: palomavaloper10y227j9d09pckexy32v2gckerj9a0kcewgf7xy

Closing. Library needs to be in the correct directory.


CI: update cilium-cli to v0.12.10, force deploy connectivity test pods on GKE

See individual commit messages.
For #22368

External workloads workflow is failing with

Error: Unable to create external workload install script: No Major.Minor.Patch elements found

https://github.com/cilium/cilium/actions/runs/3582806205/jobs/6027497979
I assume this is due to https://github.com/cilium/cilium-cli/pull/1238 /cc @brb

GKE workflow passed though: https://github.com/cilium/cilium/actions/runs/3582806210/jobs/6027502665
Re-running to verify that https://github.com/cilium/cilium/issues/22368 is indeed fixed.

Removed the test commit and marking as ready to merge given that reviews are in and all workflows except external workloads passed (namely GKE) passed. Let's disable external workloads workflow for now and re-enable once we switch to a cilium-cli version with a fix.


PathFillType.evenOdd is not working in Web with HTML renderer

It seems like PathFillType.evenOdd doesn't have any effect when we run the app with the HTML renderer.
Running the same app with the CanvasKit renderer, the clipping is working as expected. See below.
This issue #81637 is very similar, but it is focused on images behind the clip area, when actually that doesn't matter, in the code sample of this issue I use a Text component, for example.
Steps to Reproduce

Just execute flutter run -d chrome --web-renderer html on the code sample

Expected results:

Actual results:

Code sample
code_sample.zip
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const HomeView(),
    );
  }
}

class HomeView extends StatelessWidget {
  const HomeView({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Stack(
      children: const [
        Center(child: Text("Hello!!")),
        Positioned.fill(
          child: ClipPath(
            clipper: InvertedClipper(),
            child: ColoredBox(color: Colors.blue),
          ),
        ),
      ],
    );
  }
}

class InvertedClipper extends CustomClipper<Path> {
  const InvertedClipper({Listenable? reclip}) : super(reclip: reclip);

  @override
  Path getClip(Size size) {
    Path path = Path();

    // This code is not working in HTML renderer.
    path.addRect(Rect.fromLTWH((size.width - 200) / 2, (size.height - 100) / 2, 200, 100));
    path.addRect(Rect.fromLTWH(0, 0, size.width, size.height));
    path.fillType = PathFillType.evenOdd;

    return path;
  }

  @override
  bool shouldReclip(CustomClipper<Path> oldClipper) => true;
}

Logs
[✓] Flutter (Channel stable, 3.0.5, on macOS 12.2.1 21D62 darwin-x64, locale es-ES)
    • Flutter version 3.0.5 at /Users/david/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (4 weeks ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/david/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 50.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 69.0.5
    • Dart plugin version 222.3345.108

[✓] VS Code (version 1.69.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.24.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 104.0.5112.79

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hi @AriasBros, thanks for filing the issue.
When I tried removing fillType = PathFillType.evenOdd, the result is still the same. So I think the issue is caused by ClipPath whether it's an Image or a Text widget in there. It looks like two issues are similar and we should only keep one.
Closing as a duplicate, so please follow up on it for further updates. You can leave there your comment and react an upvote to the original comment will help to raise the issue. If you disagree, please write in the comment and I will reopen this. Thank you!

Sorry @huycozy, but I don't understand the reason to remove fillType = PathFillType.evenOdd from the code. If you do that the issue is not fired. Of course, the bug itself can be in ClipPath, or wherever, I don't know. But if you don't use PathFillType.evenOdd you will not be able to see the issue happening.
Below you can see a better images with the actual issue. They are done with this code:
path.addRect(Rect.fromLTWH((size.width - 200) / 2, (size.height - 100) / 2, 200, 100));
path.addRect(Rect.fromLTWH((size.width - 400) / 2, (size.height - 300) / 2, 400, 300));
path.fillType = PathFillType.evenOdd;

As you can see, the ClipPath is doing the clip, but only it is clipping the outer part of the Path, the inner clipping is not being done. And who tells that the inner clipping must be done is fillType = PathFillType.evenOdd. If you don't use fillType with PathFillType.evenOdd you never will see the inner clip (what that is the right thing). I don't know if I am explaining me...
Said this, thanks for your reply @huycozy, I don't mind the issue is closed, that is one of the reasons I linked my original message with that issue. But also I think the title of the original issue is wrong because is giving the understanding that the origin of the issue is to have images in the stack, when that is not the case. The problem seems to be that ClipPath is ignoring the fillType of the Path.
Expected results:

Actual results:

Hi @AriasBros
I reopen the issue for further discussion.

I don't understand the reason to remove fillType = PathFillType.evenOdd from the code.

Because I see that issue #81637 is about ClipPath, I tried removing fillType = PathFillType.evenOdd to see if this issue is still reproducible. And the issue is still reproducible after removing that property, but it seems not like that from your side, right? If so, I'm sorry for the subjective judgment.

But also I think the title of the original issue is wrong because is giving the understanding that the origin of the issue is to have images in the stack, when that is not the case.

Don't worry about this. Once this issue is closed as a duplicate, please write a comment on the issue #81637 about your case. I will also update its title as well to be appropriate with cases. What do you think?

Thanks for your help @huycozy

Because I see that issue #81637 is about ClipPath, I tried removing fillType = PathFillType.evenOdd to see if this issue is still reproducible.

Yes, the issue is about ClipPath... but it seems that it is ignoring the property fillType, so if you remove  that line you are removing the source of the issue and you will not able to replicate it.

And the issue is still reproducible after removing that property, but it seems not like that from your side, right? If so, I'm sorry for the subjective judgment.

This issue that I open and the issue https://github.com/flutter/flutter/issues/81637 are the same in my opinion. But because I am not sure and I am not an expert, I decided to open a new issue to not disturb in the other issue.

Don't worry about this. Once this issue is closed as a duplicate, please write a comment on the issue https://github.com/flutter/flutter/issues/81637 about your case. I will also update its title as well to be appropriate with cases. What do you think?

Ok, I will comment there with a link to this issue.
Again, thanks a lot @huycozy

Hi @AriasBros, I will close this issue as a duplicate, so please follow up on issue #81637 for further updates. Thank you!


Adapt a common make_df() style  in Westeros tutorials

In PR https://github.com/iiasa/message_ix/pull/524 @khaeru mentioned, that a common style of creating dataframes could be introduced for further improvements.

In some cases like here, black was applied but the underlying code idiom was left as-is. The code works and it's fine to merge, but I think a later PR could improve these further to suggest a common style, e.g.:
name = "soft_activity_up"
scen.add_par(
    name,
    make_df(
        name,
        node_loc=country,
        technology="wind_ppl",
        year_act=model_horizon,
        time="year",
        value=[0, 0, 0, 0.01],
        unit="-",
    )
)

or
name = "soft_activity_up"
data = make_df(
    name,
    node_loc=country,
    technology="wind_ppl",
    year_act=model_horizon,
    time="year",
    value=[0, 0, 0, 0.01],
    unit="-",
)
scen.add_par(name, data)

Thanks for the check! If the usage is already consistent, then I agree the issue can be closed.
Per the last example make_df() currently only supports "par" items, not "set" items, so is not usable there in any case. I guess it could be extended to support the subset of "set" items that are represented as data frames (i.e. 1+-dimensional "mapping" sets), but that would be a separate issue.

Since this doesn't actually seem to be an issue as of now, I won't open another one. Once the use case arises that make_df() should support "set" items, we are going to do that anyway.


Fix vendoring of cheet.js

Following on from @zachlatta's comments in #297.

@paked @zachlatta thoughts on serving each of these scripts ourselves? Rawgit has had downtime before.

Also, why are we using https on cheet.js, but not on the other rawgit scripts.

Yeah, I think we should vendorize everything from rawgit or use the copies
on cdnjs.
On Tue, Feb 14, 2017 at 11:05 AM, Max Wofford<EMAIL_ADDRESS>wrote:

Also, why are we using https on cheet.js, but not on the other rawgit
scripts.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hackclub/website/pull/301#issuecomment-279802850, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA8j-GmkxYPbFZOr8S-_gVnfyjTTm1JGks5rcfrwgaJpZM4MA2KF
.

--
Zach Latta
https://zachlatta.com http://zachlatta.com

@paked thoughts?

I'm currently working on this.
From Slack:
@zrl @msw: re vendoring, I’m going to have the dependencies which are on
cdnjs on cdnjs the ones which aren’t i’m going to vendor locally.
On Wed, Feb 15, 2017 at 12:42 AM, Max Wofford<EMAIL_ADDRESS>wrote:

@paked https://github.com/paked thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hackclub/website/pull/301#issuecomment-279949966, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIBJSnTQl6liUB5xceUyzbsYxtysfI9Zks5rcrp4gaJpZM4MA2KF
.

@paked What's the status of this?

Going to close this because I'm cleaning up things that haven't changed for a while. Feel free to re-open if you want!


Improve type of arrayResultHandler

I noticed that currently it's compiled this way in DTS:
declare const arrayResultHandler: ResultHandlerDefinition<any, z.ZodString & {
    _def: z.ZodStringDef & MetaDef<z.ZodString>;
    example: (example: string) => z.ZodString & any;
}>;

After this improvement:
declare const arrayEndpointsFactory: EndpointsFactory<z.ZodArray<any, "many"> & {
    _def: z.ZodArrayDef<any> & MetaDef<z.ZodArray<any, "many">>;
    example: (example: any[]) => z.ZodArray<any, "many"> & any;
}, z.ZodString & {
    _def: z.ZodStringDef & MetaDef<z.ZodString>;
    example: (example: string) => z.ZodString & any;
}, null, {}, string, string>;

I decided to go simpler, since it's not actually so important which type ResultHandler has, since the generators infer actual types programmatically.


Changing confirmation email and reset password URLs with external frontend

Is there any way to change confirmation or reset password URL in emails without reimplementing it like this in GraphQL resolvers?
  def sign_up(_, %{input: input}, _) do
    with {:ok, user, conn} <- Pow.Plug.create_user(conn(), input) do
      send_email_confirmation_email(conn, user)

      {:ok,
       %{
         session: %{
           access_token: conn.private[:api_access_token],
           renewal_token: conn.private[:api_renewal_token]
         },
         user: user
       }}
    else
      {:error, changeset, _conn} -> {:error, changeset}
    end
  end

  def send_reset_password_url(_, %{input: %{email: email}}, _) do
    send_reset_password_email(conn(), email)

    {:ok, %{email: email}}
  end

  defp send_email_confirmation_email(conn, user) do
    token = PowEmailConfirmation.Plug.sign_confirmation_token(conn, user)
    url = "#{System.get_env("FRONTEND_URL")}/auth/confirm-email/#{token}"
    unconfirmed_user = %{user | email: user.unconfirmed_email || user.email}
    email = PowEmailConfirmation.Phoenix.Mailer.email_confirmation(conn, unconfirmed_user, url)

    Pow.Phoenix.Mailer.deliver(conn, email)
  end

  defp send_reset_password_email(conn, email) do
    with {:ok, %{token: token, user: user}, conn} <-
           PowResetPassword.Plug.create_reset_token(conn, %{"email" => email}) do
      url = "#{System.get_env("FRONTEND_URL")}/auth/reset-password/#{token}"
      email = PowResetPassword.Phoenix.Mailer.reset_password(conn, user, url)

      Pow.Phoenix.Mailer.deliver(conn, email)
    end
  end

  defp conn(socket \\ %{}) do
    endpoint = Map.get(socket, :endpoint, ProlingWeb.Endpoint)

    %Plug.Conn{
      private: %{
        phoenix_endpoint: endpoint,
        phoenix_router: ProlingWeb.Router,
        pow_mailer_layout: {ProlingWeb.LayoutView, :email}
      },
      secret_key_base: endpoint.config(:secret_key_base)
    }
    |> Pow.Plug.put_config(config())
  end

  defp config(params \\ []) do
    [plug: ProlingWeb.Plug.APIAuth, otp_app: :proling_web] ++ params
  end

With this code I've got a correct link but user gets authenticated while still has unconfirmed email.
UPD: I've got that returning uncofirmedEmail => nil with create is totally ok.
And my schema
defmodule Proling.Identity.User do
  use Ecto.Schema
  use Arc.Ecto.Schema

  use Pow.Ecto.Schema, password_min_length: 6
  use Pow.Extension.Ecto.Schema, extensions: [PowResetPassword, PowEmailConfirmation]

  import Ecto.Changeset
  import Pow.Ecto.Schema.Changeset, only: [new_password_changeset: 3]

  alias Proling.Uploader.Avatar

  @roles ~w(user employee admin)

  @required_fields ~w(phone email)a

  schema "users" do
    field :avatar, Avatar.Type
    field :first_name, :string
    field :last_name, :string
    field :phone, :string
    field :role, :string, default: "user"

    pow_user_fields()

    timestamps()
  end

  def changeset(user, attrs) do
    user
    |> cast(attrs, @required_fields)
    |> validate_required(@required_fields)
    |> common_changeset(attrs)
  end

  def update_changeset(user, attrs) do
    user
    |> cast(attrs, @required_fields)
    |> validate_required(@required_fields)
    |> common_changeset(attrs)
  end

  def update_role_changeset(user, attrs) do
    user
    |> cast(attrs, [:role])
    |> validate_required([:role])
    |> validate_inclusion(:role, @roles)
  end

  def reset_password_changeset(changeset, attrs) do
    changeset
    |> cast(attrs, [:password])
    |> validate_required([:password])
    |> new_password_changeset(attrs, @pow_config)
  end

  defp common_changeset(changeset, attrs) do
    changeset
    |> validate_phone()
    |> unique_constraint(:phone)
    |> unique_constraint(:email)
    |> maybe_pow_changeset(attrs)
    |> pow_extension_changeset(attrs)
  end

  defp maybe_pow_changeset(changeset, attrs) do
    pow_changeset =
      changeset
      |> pow_user_id_field_changeset(attrs)
      |> maybe_pow_current_password_changeset(attrs)
      |> new_password_changeset(attrs, @pow_config)

    case pow_changeset.changes == %{} do
      true -> changeset
      false -> pow_changeset
    end
  end

  defp maybe_pow_current_password_changeset(%{changes: %{password: _}} = changeset, attrs) do
    changeset |> pow_current_password_changeset(attrs)
  end

  defp maybe_pow_current_password_changeset(changeset, _attrs) do
    changeset
  end

  defp validate_phone(%{valid?: true, changes: %{phone: phone}} = changeset) do
    with {:ok, parsed_phone} <- ExPhoneNumber.parse("+#{phone}", "RU"),
         true <- ExPhoneNumber.is_valid_number?(parsed_phone) do
      changeset
    else
      {:error, _message} -> add_error(changeset, :phone, "has invalid format")
      false -> add_error(changeset, :phone, "is invalid")
    end
  end

  defp validate_phone(changeset), do: changeset
end

You have to block authentication after sign up if the e-mail is unconfirmed, so you need to set it up like this:
  def sign_up(_, %{input: input}, _) do
    case {:ok, user, conn} <- Pow.Plug.create_user(conn(), input) do
       {:ok, _user, conn}         -> maybe_require_confirmation(conn)
       {:error, changeset,  conn} -> {:error, changeset}
     end
   end

  defp maybe_require_confirmation(conn) do
    case PowEmailConfirmation.Plug.email_unconfirmed?(conn) do
      true ->
        {:ok,
         %{
           session: %{
             access_token: conn.private[:api_access_token],
             renewal_token: conn.private[:api_renewal_token]
           },
           user: Pow.Plug.current_user(conn)
         }}
      false ->

      {:error, changeset, _conn} ->
        {:error, changeset}

      false ->
        send_email_confirmation_email(conn)

        Pow.Plug.delete(conn)

        {:error, "requires email confirmation"}
    end
  end

Though if you already know that all users will need email confirmation then you can just bypass the pow auth logic entirely and just create the user directly instead of using Pow.Plug.create_user/2:
  def sign_up(_, %{input: input}, _) do
    with {:ok, user} <- Pow.Ecto.Context.create(input, otp_app: proling_web) do
        send_email_confirmation_email(conn)

       {:ok, %{user: user}}
    else
      {:error, changeset, _conn} -> {:error, changeset}
     end
   end
  end

@danschultzer Thanks a lot for this great explanation. I definitely agreed but the drawback of requiring email confirmation for login like double current password and secure password rules originates a bad UX. Not all systems need to be super secure for UX sake I mean. So I still can't find a way to just change URL for emails if I have external frontend, is it possible with little changes without reimplementing sending email on my own?

I think there is no better way right now to change only change url without building email from scratch so it might be closed right now.

Sorry @vadimshvetsov, got away from this issue.

I definitely agreed but the drawback of requiring email confirmation for login like double current password and secure password rules originates a bad UX. Not all systems need to be super secure for UX sake I mean.

Oh I understand what you were asking for now. Your setup is fine then 😄

So I still can't find a way to just change URL for emails if I have external frontend, is it possible with little changes without reimplementing sending email on my own?

So first, you can change the domain by using the Phoenix URL generator options:
config :my_app, MyAppWeb.Endpoint,
  # ..,
  url: [scheme: "https", host: "example.com", port: 443]

Pow uses the router helpers so with that set up you can instead call e.g. Routes.pow_email_confirmation_confirmation_url(conn, :show, token) rather than building it manually.
You would still need to generate and send the e-mail, as the mail delivery in in the controllers (except for PowEmailConfirmation.Phoenix.ControllerCallbacks.send_confirmation_email/2). I'm planning to refactor some of the mailer logic with https://github.com/danschultzer/pow/issues/469, so might find a way to make this easier.

@danschultzer Thanks again, Dan! It looks like changing Phoenix.Endpoint will change Phoenix url instead pointing my React Frontend app

It looks like changing Phoenix.Endpoint will change Phoenix url instead pointing my React Frontend app

Oh right, I incorrectly assumed you just wanted to replicate the URL with the front-end domain. In that case your setup is probably the best that can be done currently. If you find any opportunities for refactoring Pow to make this easier, please let me know! I'm not sure the refactoring of the e-mail logic I'm planning would help much here.
I would like to move away from requiring conn for the mailer if possible, so maybe it would be good to do both that and a slight change to the logic so you only have to call one method to generate and deliver the email?

@danschultzer I love what Pow offers for Phoenix users but it's not easy to use in GraphQL API which doesn't rely on request as well as use Phoenix. I would happy to help. By the way, when Phoenix will do their auth based on conn Pow might offer auth without conn and it could be killer feature for Absinthe users

By the way, when Phoenix will do their auth based on conn Pow might offer auth without conn and it could be killer feature for Absinthe users

Agreed! I'm planning to let Pow v1.1.0 be the final version of Pow, with hard deprecations for everything I've learned to make all of this much simpler. I've realized that I'm depending too much on conn and probably should do a lot more with just using the Endpoint module (as Phoenix does).


Manhwa freak: HTTP 404 error

Source information
Manhwa freak 1.4.32
Source language
English
Steps to reproduce

Extension not working:
Error: HTTP 404 but website view is working
Expected behavior
Extension should work and give the manhwa list
Actual behavior
Error: HTTP 404
Mihon/Tachiyomi version
0.15.3
Android version
Android 14
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[ ] I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Extension is working again.


x/tools/cmd/ssadump: doesn't compile

The x/tools build is broken across the board:
https://build.golang.org/log/5caa4ff768a880d6f306a9546cb060eb5e592c29 etc:
# golang.org/x/tools/cmd/ssadump
2017/03/14 14:27:21 duplicate symbol golang.org/x/tools/go/ssa.(*BuilderMode).String (types 1 and 1) in main and $WORK/golang.org/x/tools/go/ssa.a(_go_.o)

I think that this was broken by https://golang.org/cl/38139 and I think that the fix is in https://golang.org/cl/38156.

CC @dr2chase @philhofer

Confirmed dupe of #19548.

Closing as dup.


Assets module's AssetId cannot be used as constant

after integrating scale-info, it seems that AssetIdOf cannot be used as pallet::constant anymore because of missing TypeInfo trait bound
https://github.com/paritytech/substrate/blob/master/frame/assets/src/lib.rs#L194-L201

Shall we add TypeInfo trait bound to line

Yes that would be the correct way.


always include newline in resource files

it looks better this way when looking the files from terminal using cat
some parsers even may break without newlines (shell scripts for example!)

@glensc I am in the process of cleaning the PR list :) Can you rebase / fix conflicts for that one so I can merge this ;)
TIA
-SNK

indeed, merge conflict cause from another PR of mine: #46. pushed it over!

LGTM


Timeout handling

Fix #21
Changes an internal API, unpack. The response is now supplied as the return value of a block in order to catch exceptions. Current use case is handling timeout responses.
First commits switches to the new version of unpack, second leverages it to handle timeouts.

Closing for now. Not sure this is the right way to do it.


Doesn't work for me, dockerhub repo is 404

docker: Error response from daemon: pull access denied for hostname, repository does not exist or may require 'docker login': denied: requested access to the resource is denied.
See 'docker run --help'.
'docker login' didn't help

Also page at https://hub.docker.com/r/ravermeister/gitlab/ is gone

could you check again? seems a temporary failure in dockerhub, I could open the page without a login on 02.02.2022 10:00 AM GMT+1

Yes, link works now


petitboot：When the language is Chinese, no Chinese is displayed in petitboot

Petitboot：Language
When the language is Chinese, no Chinese is displayed in petitboot.
Excuse me, how to configure to display Chinese？
I found that there is no locale folder under /usr/share/, how do I generate a locale folder？
thank you.

I see you originally asked here, which is probably the right place. I'll answer there.


 verbs immediately play when dev phone rings

How to replicate: call the dev phone using a studio flow that uses a  or  verb. When you answer the call, it sounds like the audio has already started during the ring.
First reported by Joe Nash, as follows:

Is it possible that a call TO a Dev Phone is technically getting answered somewhere before the call comes through to the browser? The reason I ask is outbound twilio calls containing just a single  verb seem to end before Dev Phone rings, and if you add more content to those calls, they go through but have clearly already started.

Does this involve answerOnBridge? Documentation
Worst case scenario, we can brute force a fix by including a pause on this, but I think it may be related to multiple devices getting spun up by the webhook context.

I think right here would do the trick testing wise: https://github.com/twilio-labs/dev-phone/blob/37505cbe43bbf641d7dd5bd534d075264615f727/packages/plugin-dev-phone/src/serverless/functions/incoming-call-handler.js#L6

@craigsdennis is a genius, I think it worked. Testing now and if all is well then we can merge in and release a patch on Monday 👌


Legacy stuff — (wiki/G+/community links, multitool/other apks)

Community / Wiki:

community.lightninglauncher.com

@TrianguloY has already started a thread here  :)

The wiki (if I recall correctly) is sometimes fragmented or mirrored in different locations, and oftentimes links are broken on official/community sites, and even the app itself

We should consolidate/import all of the wiki to a Github wiki here in the repo
I do think we should keep the existing documentation - it is actually very thorough and easy to understand!
Scripting API, which I know like the back of my hand by now :)
Google+ links are everywhere in the documentation / official site (which is still kind of amusing)

Legacy APKs

Legacy APKs-20230202T233009Z-001.zip

I like the idea of having a Github wiki will everything that can be archived. It will also allow anyone to contribute and modify. Let me prepare an empty one to start.


[Feature Request]: Datagrid optional hover titles on cells

The problem
Datagrid cells have values show up on hover
I want to have custom tooltip, however then we have two tooltips show up

The solution
cell hover titles could be optional
Design link
No response
Links to other materials
No response
Owner/main maintainer(s)
?
Second/backup maintainer(s)
No response
Product/offering
No response
Business priority
Low Priority = release date is not dependent on fix or not upcoming
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[ ] I checked the current issues for duplicate issues

@mariusd-ibm The default titles have been removed from the cells in #5351 so this should no longer be an issue on your end. Thanks!


SMTP Authentication from IIS Relay causes the user name ignore and pa…

…ss trough password all parameters. Fixed with a method.

Hi,
Can you tell how to reproduce the issue?

Hi,
I was created an SMTP server for tracing the mails from IIS SMTP Server. I saw that, when IIS SMTP send's mail trough that server, the user name which I set on IIS was ignored, and password placed all parameters, such as authenticatedEntity, authorizationEntity and ofc password.
I saw that IIS sends LOGIN AUTH "base64CodedUserName" with capture value "2"  and It was ignored because it's not contains "PLAIN" or something else.
so, I debug it with IIS and placed a method that's makes decode base64 user name.  When "LOGIN" command executed in AuthenticatedCommands class That's handles TempUserName with "LOGIN AUTH base64UserName". So after that SMTP waits password and It's came after Capture Data = 2 and If I set TempUserName before, I ignored that username, and just take Password. If did not set before It's works old behaviour.
I tested that on IIS and other relay and MailKit SMTP Client. All works.
btw. Thank for library. I try to give much more contribution on this package I think.

It's very interesting that IIS uses AUTH LOGIN. It's an obsolete method (that shouldn't be used) and as far as I'm aware it doesn't define passing username as an argument. However if IIS does it this way, it will be implemented in this library of course.
SmtpClient is just a test app that you used to reproduce the issue with IIS, right?

I’m sorry, I should not commit the tester app, If you give me some time I can revert sample app changes :)

I'm trying to understand how it works. So IIS sends username as argument, it's saved as TempUsername, but then we still set CaptureData to 2, so IIS needs to send something before password anyway?
The code works and correct password is received?

So if password is present as an argument of "AUTH LOGIN" shouldn't the library go to captureData = 3 (capturing the password)?
Are you sure the code in PR works with IIS (both user password and email content is properly captured)? If it does then it's even more weird how IIS handles that.

I recently closed a similar issue and PR (#12 and #13), because as far as I know "AUTH LOGIN" doesn't allow that. However if I'm wrong or IIS just ignores the standard and passes username as argument, then it will get implemented.
The implementation in #13 is how I imagine it works - when argument is present we set TempUsername and skip directly to stage 3 (capturing password). That's why I'm asking if you are sure your implementation works.

Actually, I want to be ensured this behaviour, I'll write some unit test that's mock the IIS and standart MailKit SMTP Client. I'll recommit with that tests and behaviours so we can ensure about It. I know the AUTH LOGIN obsolete but also Python Mail Client works like this.
I recently try the System.Net.Mail.SmtpClient that's also not works both version of library, because that client set username and password as NetworkCredential and It's not handled by IAuthFilter and also not placed in ParsedMessage.

Can you send me simple apps using Python Mail Client and System.Net.Mail.SmtpClient? It will be really helpful.
I assume in both cases it will be an easy fix, because entire authentication is quite simple.

Hi,
I try to make some changes and also add Tested SMTP Clients in Readme file. And Improved IAuthFilter, thats can checks now username and password as user and password


Remove core module

This PR completes the node overhaul :).
The first commit changes the compiler module so that it exposes a single function that just wraps serialize::serialize_no_witnesses. Should we just delete the module and move its unit tests to serialize?

Let's move everything from compiler to serialize. The actual compiler part is almost a one liner. (We could rename it to commit and add a type finalization.) The unit tests more or less generalize to WitnessNode as well. At least, if everything works for ConstructNode, it is highly unlikely that something breaks for WitnessNode.

I'll add a commit at the end.

@uncomputable I just rebased this (I needed to because the "remove Witness from dag" commit stopped compiling since #142 added some new uses of dag.)


Set QCSurvey on player object

We want the player to be able to complete the QC survey even if they are not able to join a game. So, the reference to the QC survey can't be on the game object.

Addressed in https://github.com/Watts-Lab/deliberation-empirica/pull/269


Ubuntu 16.14 (Xenial) not allowing to run hello-word with Client.timeout exceeded while awaiting headers

It seems I have installed Docker for ubuntu properly, as I can see the help menu when invoking docker. However, when I run:
sudo docker run hello-world
I get:
Unable to find image 'hello-world:latest' locally
docker: Error response from daemon: Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers).
See 'docker run --help'.
Also, if I try instead a prebuilt docker image for using ROS Indigo on Ubuntu Xenial with
https://github.com/jbohren/rosdocked
Then I get similar error, what can it be? See command and error below:
sudo ./build.sh ros_docker_image
Sending build context to Docker daemon 84.48 kB
Step 1/19 : FROM ros:indigo
Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)
Any hints?

Also, if I try instead a prebuilt docker image for using ROS Indigo on Ubuntu Xenial with
https://github.com/jbohren/rosdocked
Then I get similar error, what can it be? See command and error below:
sudo ./build.sh ros_docker_image
Sending build context to Docker daemon 84.48 kB
Step 1/19 : FROM ros:indigo
Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)

That sounds like the Docker daemon can't get to the internet , so it can't download any images.

Awesome! Glad you got it working.

@NataliaDiaz I know this is not a forum, but could you explain a little bit your solution? thanks in advance!

Under the univ. proxy I had trouble and getting many messages such as (Client.Timeout exceeded while awaiting headers)
Apart from setting the proxy variables in your bash profile such as https_proxy and http_proxy, using sudo -E before every problematic command with such message above, forces to have all env. variables visibles for the proxy too and normally solves the problem.


Uncaught TypeError: Cannot read properties of undefined (reading 'addListener') on Chromium based

Extension doesn't work on chromium based browser.

Chromium support is not my prioriy right now. Not saying this won't come, but for now I have to treat this extension as Firefox only.


Error opening images.

When I click on the images, they don't load me correctly, I thought it was where the link was being obtained from but when I open those links, they don't load correctly, could you help me or something, here I put a screenshot what this looks like:

Could you provide your markdown code?

How about a pseudo code? If all image on your docs does not load correctly, it may be related to the CSS in the mkdocs. Do you use another custom CSS to modify the image tag?

yes, this is the md:
# H1

image

![https://df7vkoxzcj1vg.cloudfront.net/4157d490-56f8-11ed-91e3-000c297ff65d/6f98129d-9646-4f78-9cea-dfccff4907e4]

and here is the css:

  [data-md-color-scheme="slate"] {
    --md-primary-fg-color:        #0056b3;
    --md-primary-fg-color--light: #bbecb7;
    --md-primary-fg-color--dark:  #290390;
    --md-hue: 250;
  }

@IvyMasterDiego Sorry for the late reply. The bug has been fixed in the new version 0.3.2.


Refactor script to use objects

A big refactor is necessary to begin to add more complex features, before the code gets too messy.
I'm working on it and will need feedback on my fork soon.

This was done :) many thanks


Put router behind feature flag

Put router behind feature flag
Added to App.tsx
Put router behind feature flag

#2892  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tommylau523 and the rest of your teammates on  Graphite


Client drops messages longer than 4096 bytes

Hello! I've been using go-sse lately and have been enjoying it a lot. One issue that I came across recently is that messages that are longer than 4096 bytes get dropped by the client. I narrowed this down to an issue with how we parse messages from the event stream.
PR #2 contains a test case that reproduces this bug, along with a fix. Let me know if you have any questions, or need more information!

Thank you for raising the issue and opening the PR! See the PR for code-related discussions.
I know I haven't contributed to the library in a long while, given time issues and that I've shifted my attention to other projects. Nevertheless, I'm still here and able to maintain it again, so if you find any bugs or non-conformities with the specification, have suggestions for new features or find cleaner or more performant ways to implement things internally, please let me know! The goal of go-sse is to be correct, idiomatic and performant.
Feel free to close the issue yourself when the PR is merged, i.e. issue is fixed.


Support empty Query, Mutation, etc.

I have multiple files with each extending the Query type and a base file with an empty Query.
# base.graphqxl

import "things"

type Query

# things.graphqxl

extend type Query {
  getThings: [Thing!]!
}

...

When running the generator I get
Error: Could not parse GraphQXL spec:

 --> 9:1
  |
9 | type Mutation {}
  | ^---
  |
  = expected type_selection_set, implements, directive, or generic
make: *** [Makefile:16: generated/sensorman.gql] Error 1

The recommended way of doing this with GraphQXL would be to instead rely on the already existing composition mechanisms, for example:
# products.graphqxl

type _ProductsQuery {
  product(id: ID!): Product
  productByName(name: String!): Product
  allProducts: [Product!]!
}

# users.graphqxl

type _UsersQuery {
  user(id: ID!): User
  userByName(name: String!): User
  allUsers: [User!]!
}

# main.graphqxl

import 'products'
import 'users'

type Query {
  ..._ProductsQuery
  ..._UsersQuery
}

That way, it's very clear and explicit the order of the fields in the final Query type.
Take a look at this example

Ah that makes perfect sense thank you!


New model structure

Deep Q-learning model (DQN) denoted as f.

Learn:  f( state, action ) = future potential reward
Given state s_i, a possible set of actions A = {a_k}. Use f to evaluate A, and take the best one.

I think reward should be in range -1, 1 instead of 0, 1.. so maybe tanh should be used instead of sigmoid?
https://github.com/kkspeed/chess/blob/master/model_v2.py#L34


Abandon AbstractLink type

All that I need to specify a likelihood is the mapping from f to a distribution, and in many cases the only degree of freedom is the inverse link function: it would be super convenient if I could just write PoissonLikelihood(log1pexp) to try out a softplus invlink, without having to code up a subtype of AbstractLink and having to remember what I have to do to implement it.

We could still dispatch analytic implementations of expected_loglik etc on typeof(exp) etc, right?

How about creating a GenericLink and have
convert(::Type{AbstractLink}, x::Any) = GenericLink(x)

?

Just so I understand better, what do we gain from defining e.g. ExpLink explicitly (rather than "only" having GenericLink(exp))?

We are able to tell what the corresponding inverse Link is LogLink

An alternative would be to add an API like
inverse_link(::typeof(exp)) = log
inverse_link(::typeof(log)) = exp

inverse_link(::typeof(log1pexp)) = logexpm1
inverse_link(::typeof(logexpm1)) = log1pexp

that would allow us to define
Base.inv(l::GenericLink) = GenericLink(inverse_link(l.f))


⚠ Develop API Endpoint has degraded performance

In f56f8e9, Develop API Endpoint (http://develop.roblox.com/) experienced degraded performance:

HTTP code: 503
Response time: 1104 ms

Resolved: Develop API Endpoint performance has improved in 31442c1 after 29 minutes.


Streaming other sources other than the webcam: OpenCV

Hi!
This is just a placeholder in case anyone can enlighten me on sending media other than the webcam, ideally from frames processed using OpenCV
My guess is that I do not stand alone on this. Sometimes we want to process the webcam image using OpenCV before sending it, so after I have my nice Mat frames in my while loop, I don't know how to plug them to OpenWebRTC in order to send them
Any help is much appreciated ^^
Thanks in advance,
—N

What memory is used to store your OpenCV frames? System memory or graphics memory or something else?

OpenCV stores frames in system memory by default. There are options for hardware acceleration but they're not used so much.

This could be relatively simple solved by https://github.com/EricssonResearch/openwebrtc/issues/221 and using e.g. an appsrc.
Alternatively it might also make sense to have appsrc/appsink-style MediaSources/Renderers, where application can pass in data directly or get data from a callback.

Hey,
@superdump , frames are stored on memory as @richardmitic pointed out
@sdroege , I'm not an expert in this subject, but your alternative sounds good, having a callback where the application can pass data

@ignacioavellino, bear in the mid that in OpenCV the pixel data is not necessarily contiguous in memory. In your app you'd have to either assure that it is contiguous or possibly work some gst_buffer magic to avoid expensive copy operations. It should work though. :)

@sdroege that's part of my plan. Widely-used custom sources/renderers gradually get merged back in. appsrc/appsink are obvious candidates.


Throw exception if CloudblobStore::findKey cannot find any blob.

This can happen if a blob is compacted away between calls to CloudblobStore::findMissingKeys and CloudblobStore::findKey during replication.

Codecov Report

Merging #1573 into master will increase coverage by 0.02%.
The diff coverage is 57.14%.

@@             Coverage Diff              @@
##             master    #1573      +/-   ##
============================================
+ Coverage     73.64%   73.66%   +0.02%
- Complexity     8340     8344       +4
============================================
  Files           584      584
  Lines         45247    45248       +1
  Branches       5717     5718       +1
============================================
+ Hits          33322    33332      +10
+ Misses        10238    10229       -9
  Partials       1687     1687

Impacted Files
Coverage Δ
Complexity Δ

...in/java/com/github/ambry/cloud/CloudBlobStore.java
75.25% <57.14%> (-0.19%)
115.00 <0.00> (ø)

...java/com/github/ambry/store/CompactionManager.java
89.88% <0.00%> (-2.39%)
26.00% <0.00%> (ø%)

.../main/java/com/github/ambry/store/HardDeleter.java
76.06% <0.00%> (-1.43%)
49.00% <0.00%> (-1.00%)

.../java/com/github/ambry/router/DeleteOperation.java
93.75% <0.00%> (-1.39%)
45.00% <0.00%> (-1.00%)

...ain/java/com/github/ambry/router/PutOperation.java
92.32% <0.00%> (-0.33%)
123.00% <0.00%> (ø%)

...in/java/com/github/ambry/store/BlobStoreStats.java
69.76% <0.00%> (-0.22%)
101.00% <0.00%> (ø%)

...rc/main/java/com/github/ambry/store/BlobStore.java
88.09% <0.00%> (+0.15%)
147.00% <0.00%> (+1.00%)

...ava/com/github/ambry/store/BlobStoreCompactor.java
90.14% <0.00%> (+0.28%)
120.00% <0.00%> (ø%)

...src/main/java/com/github/ambry/commons/BlobId.java
93.37% <0.00%> (+0.34%)
73.00% <0.00%> (+1.00%)

...va/com/github/ambry/replication/ReplicaThread.java
85.97% <0.00%> (+0.81%)
113.00% <0.00%> (+1.00%)

... and 4 more

Continue to review full report at Codecov.

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


Update npm package

Hi Alexander,
I am trying to deploy my website using NGL on netlify and the build process fails as my package.json refers to NGL's version from NPM, which predates the Colormaker breaking change name.
Thanks for your help,
Paul

Well, I've found a way to solve this issue by linking directly to git://github.com/arose/ngl.git in package.json

There are more recent version published on npm under the tag next.


Use incluster config fallback for KubernetesProvider

Description
This PR supersedes https://github.com/Parsl/parsl/pull/2503
See https://github.com/kubernetes-client/python/issues/1005 for a good description of the issue.
Changed Behaviour
The KubernetesProvider now falls back to loading config in-cluster if a kube-config file is not found. This allows in-cluster submission of parsl jobs.
Fixes
Fixes # (issue)
Type of change
Choose which options apply, and delete the ones which do not apply.

Bug fix

some exciting new build problem with radical's build/packaging. I tried briefly to recreate it on my laptop and did not have a problem, so I'll dig into it more deeply later in the week - I'm not working on Parsl today.

(very possibly related to build system changes in Python 3.12 which were/are fairly aggressive)

Oof. Thanks so much for providing that context and saving us from spinning our wheels on this! :heart:

@benclifford I've merged in master. Happy to do that again as needed!
Note that I also amended my first commit to this PR to properly credit @manning-ncsa for work contributed in the superseded PR, #2503 .

@trey-stafford if you edit this PR can you enable "Allow maintainers to push"? Or is that only settable when opening the original PR? :thinking:

you did something weird perhaps with rebase that makes the diff look weird - I think I know what, and I think that will fix itself once master gets merged again - which is necessary for putting this PR on master anyway as our tests require the PR be up to date.

I think I see what happened, we'll get it sorted on Monday!

:rocket: Sorry didn't get this done on Monday, but I think we're all fixed up now :)

Can you comment a bit on what the exception reported to a user looks like? Say the user intends to do something with the  config loaded by load_kube_config() but they mess it up in some way that raises an exception... does useful debugging information then get lost because that exception is immediately discarded?

I have a couple meetings coming up to prepare for but we will get back to you shortly!

Would you like that context in the commit message for this commit? 4833fc2f46f700a972bf45e57c94b595825f978f

It doesn't really matter where you write it - if you have something really interesting to say, a comment in the code would be ok too, for example. Just more interested that someone thinks about it because I am always suspicious of too-easily-discarded exceptions.

We pushed another change that we feel will help prevent a really confusing user experience if both local config and incluster config fail to load. We used an ExceptionGroup to handle this, but perhaps the better solution is to only re-raise the local config exception with a note that we tried both loading local and incluster config. The ExceptionGroup looks like this when raised:
Traceback (most recent call last):
  File "/home/mfisher/Projects/qgreenland-net/parsl-exploration/src/parsl/parsl/providers/kubernetes/kube.py", line 109, in __init__
    config.load_kube_config()
  File "/home/mfisher/.miniconda3/envs/parsl-exploration/lib/python3.12/site-packages/kubernetes/config/kube_config.py", line 815, in load_kube_config
    loader = _get_kube_config_loader(
             ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mfisher/.miniconda3/envs/parsl-exploration/lib/python3.12/site-packages/kubernetes/config/kube_config.py", line 772, in _get_kube_config_loader
    raise ConfigException(
kubernetes.config.config_exception.ConfigException: Invalid kube-config file. No configuration found.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/mfisher/Projects/qgreenland-net/parsl-exploration/src/parsl/parsl/providers/kubernetes/kube.py", line 123, in __init__
    config.load_incluster_config()
  File "/home/mfisher/.miniconda3/envs/parsl-exploration/lib/python3.12/site-packages/kubernetes/config/incluster_config.py", line 121, in load_incluster_config
    try_refresh_token=try_refresh_token).load_and_set(client_configuration)
                                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mfisher/.miniconda3/envs/parsl-exploration/lib/python3.12/site-packages/kubernetes/config/incluster_config.py", line 54, in load_and_set
    self._load_config()
  File "/home/mfisher/.miniconda3/envs/parsl-exploration/lib/python3.12/site-packages/kubernetes/config/incluster_config.py", line 62, in _load_config
    raise ConfigException("Service host/port is not set.")
kubernetes.config.config_exception.ConfigException: Service host/port is not set.

During handling of the above exception, another exception occurred:

  + Exception Group Traceback (most recent call last):
  |   File "/home/mfisher/Projects/qgreenland-net/parsl-exploration/run.py", line 104, in <module>
  |     config = get_parsl_config()
  |              ^^^^^^^^^^^^^^^^^^
  |   File "/home/mfisher/Projects/qgreenland-net/parsl-exploration/run.py", line 49, in get_parsl_config
  |     provider=KubernetesProvider(
  |              ^^^^^^^^^^^^^^^^^^^
  |   File "/home/mfisher/Projects/qgreenland-net/parsl-exploration/src/parsl/parsl/providers/kubernetes/kube.py", line 125, in __init__
  |     raise ExceptionGroup(
  | ExceptionGroup: Encountered errors loading kubernetes cluster configuration (2 sub-exceptions)
  +-+---------------- 1 ----------------
    | Traceback (most recent call last):
    |   File "/home/mfisher/Projects/qgreenland-net/parsl-exploration/src/parsl/parsl/providers/kubernetes/kube.py", line 109, in __init__
    |     config.load_kube_config()
    |   File "/home/mfisher/.miniconda3/envs/parsl-exploration/lib/python3.12/site-packages/kubernetes/config/kube_config.py", line 815, in load_kube_config
    |     loader = _get_kube_config_loader(
    |              ^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/mfisher/.miniconda3/envs/parsl-exploration/lib/python3.12/site-packages/kubernetes/config/kube_config.py", line 772, in _get_kube_config_loader
    |     raise ConfigException(
    | kubernetes.config.config_exception.ConfigException: Invalid kube-config file. No configuration found.
    +---------------- 2 ----------------
    | Traceback (most recent call last):
    |   File "/home/mfisher/Projects/qgreenland-net/parsl-exploration/src/parsl/parsl/providers/kubernetes/kube.py", line 123, in __init__
    |     config.load_incluster_config()
    |   File "/home/mfisher/.miniconda3/envs/parsl-exploration/lib/python3.12/site-packages/kubernetes/config/incluster_config.py", line 121, in load_incluster_config
    |     try_refresh_token=try_refresh_token).load_and_set(client_configuration)
    |                                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/mfisher/.miniconda3/envs/parsl-exploration/lib/python3.12/site-packages/kubernetes/config/incluster_config.py", line 54, in load_and_set
    |     self._load_config()
    |   File "/home/mfisher/.miniconda3/envs/parsl-exploration/lib/python3.12/site-packages/kubernetes/config/incluster_config.py", line 62, in _load_config
    |     raise ConfigException("Service host/port is not set.")
    | kubernetes.config.config_exception.ConfigException: Service host/port is not set.
    +------------------------------------

And as we're tying this @trey-stafford called out that parsl needs to support 3.10, and ExceptionGroups are new in 3.11.

@benclifford we think we've got things ready to go!

ok cool. you still need to do something to let me update this branch, to fit in our merge process

wrt your comment on using Python 3.11 features, in recent years our informal policy has been to support the same Python versions as are listed here as "bugfix" or "security", so under that model, no 3.11 features until near the end of 2026 - https://devguide.python.org/versions/
We have a low-key project going on to understand our usage a bit better, and my gut feeling is that with people using things like conda more and being less tied to system-level Python, we could probably drop support earlier (and so access newer features like ExceptionGroups earlier) - tagging @NishchayKarle who is working on that usage project to help him understand this use case for that work.

ok cool. you still need to do something to let me update this branch, to fit in our merge process

Shoot. I think @trey-stafford has to do that, and he may not be available until Monday. I don't have an edit button even though the PR is from a repo I have write access to. As ugly as it is, I can create another fork and open a new PR and ensure the "allow maintainers to change" checkbox is ticket if you'd like to get this merged sooner than that.

@mfisher87 no super rush to mess around to get it merged

ok cool. you still need to do something to let me update this branch, to fit in our merge process

Shoot. I think @trey-stafford has to do that, and he may not be available until Monday. I don't have an edit button even though the PR is from a repo I have write access to. As ugly as it is, I can create another fork and open a new PR and ensure the "allow maintainers to change" checkbox is ticket if you'd like to get this merged sooner than that.
Trey, when you get to it, this checkbox is towards the bottom of the right sidebar:

This option is not available. I think it's because this is an org-forked repo. This documentation suggests it is possible with "user-owned forks".
Maybe a new fork is the right path?

Ok, I transferred the repo to my personal account and was able to enable edits and access by maintainers! @benclifford hopefully that does the trick!

Did it turn out that the setting was not available because we forked to an org?
In retrospect, that makes sense, as checking that box would give maintainers of other projects access to org-wide secrets! :exploding_head:

ok that worked, so this will be in the next Parsl release happening in the next few hours. Thanks!

👋 Hi @trey-stafford and @mfisher87! My name is Sophie and I'm Parsl's community manager.
We're trying to broadly understand how people are using Parsl and would appreciate it if either/both of you would fill out our quick survey: https://bit.ly/parsl-use-survey. Thanks in advance!

Hey @sophie-bui thanks for asking for our input! I just filled it out.


Dart does not throw a compile error for non-function type alias with non-well-bounded type in body

Dart SDK version: 2.12.0-259.0.dev (dev) (Mon Jan 25 20:00:51 2021 -0800) on "windows_x64"
Here is a simple source code example which throws a compile error with analyzer and passes without errors or warnings with dart:
class A<T extends A<T>> {}

typedef B1<X> = A<X>;
typedef B2<X extends A<dynamic>> = A<X>;
typedef B3<X extends A<Never>> = A<X>;
typedef B4<X extends A<void>> = A<X>;

main() {}

Sample output is:
$> dart --enable-experiment=nonfunction-type-aliases test.dart

$> dartanalyzer --enable-experiment=nonfunction-type-aliases test.dart
Analyzing test.dart...
  error - 'X' doesn't extend 'A<X>'. - test.dart:3:19 - type_argument_not_matching_bounds
  error - 'X' doesn't extend 'A<X>'. - test.dart:4:38 - type_argument_not_matching_bounds
  error - 'X' doesn't extend 'A<X>'. - test.dart:6:35 - type_argument_not_matching_bounds
3 errors found.

Seems like dart should throw an error here too.
See also issue https://github.com/dart-lang/co19/issues/991.

At the current HEAD I see compile-time errors for B1, B2, and B4:
$ tools/sdks/dart-sdk/bin/dart pkg/front_end/tool/fasta.dart compile --enable-experiment=nonfunction-type-aliases -o /dev/null /tmp/asdf.dart
Running: /Users/dmitryas/dev/dart_secondary/sdk/sdk/bin/dart --enable-asserts /Users/dmitryas/dev/dart_secondary/sdk/pkg/front_end/tool/_fasta/compile.dart --enable-experiment=nonfunction-type-aliases -o /dev/null /tmp/asdf.dart
/tmp/asdf.dart:3:17: Error: Type argument 'X' doesn't conform to the bound 'A<T>' of the type variable 'T' on 'A'.
 - 'A' is from '/tmp/asdf.dart'.
Try changing type arguments so that they conform to the bounds.
typedef B1<X> = A<X>;
                ^
/tmp/asdf.dart:1:9: Context: This is the type variable whose bound isn't conformed to.
class A<T extends A<T>> {}
        ^
/tmp/asdf.dart:4:36: Error: Type argument 'X' doesn't conform to the bound 'A<T>' of the type variable 'T' on 'A'.
 - 'A' is from '/tmp/asdf.dart'.
Try changing type arguments so that they conform to the bounds.
typedef B2<X extends A<dynamic>> = A<X>;
                                   ^
/tmp/asdf.dart:1:9: Context: This is the type variable whose bound isn't conformed to.
class A<T extends A<T>> {}
        ^
/tmp/asdf.dart:6:33: Error: Type argument 'X' doesn't conform to the bound 'A<T>' of the type variable 'T' on 'A'.
 - 'A' is from '/tmp/asdf.dart'.
Try changing type arguments so that they conform to the bounds.
typedef B4<X extends A<void>> = A<X>;
                                ^
/tmp/asdf.dart:1:9: Context: This is the type variable whose bound isn't conformed to.
class A<T extends A<T>> {}
        ^

It looks like B3 should not be a compile-time error because X extends A<Never> conforms to the bound of T.  @eernstg WDYT?

Agreed, errors at B1, B2, B4, no error at B3.


Sign up button and sign in button needs to align

Hello @VaibhavArora314 please assign this issue to me under gssoc 24

the website is not properly loading on your machine, try clearing cache or use a different browser/incognito mode

the website is not properly loading on your machine, try clearing cache or use a different browser/incognito mode

Ok checking


Confusing button on Threat Actors page

The "New Indicator" button may be confusing, as the "new object" page supports more types than just indicators. So when you're on, say, "Threat Actors" and see a "New Indicator" button, that may tell the user something other than what's intended.

Good point. Think we should rename "New Indicator" to "New Object" on all the pages, or just on the Threat Actor page?

I was thinking "New Object" in the template is probably the easiest way to go.

This issue should be fixed in 97d46f791145b84da6a381426c11fb6e077014a3


Black screen when xrdp-sesman process is non root - SOLVED

Solved:
Having xrdp, xrdp-sesman fully functional in an unprivileged container.
Solutions :
chown -R user /etc/xrdp/ directory
and modify logs path in /etc/xrdp.ini and /etc/sesman.ini to /home/user/.
After that, xrdp --nodaemon can be initiated as normal as for xrdp-sesman initiate it like this:
DISPLAY=:10 xrdp-sesman --nodaemon
Only issue would be what happens when display changes for some reason. The issue can be closed.

Wow !! So much thanks  ! Works for me in docker too :)


Prevent selenium-standalone-v2.44 and bigger

At the moment, selenium-standalone-v2.44 is unstable with PhantomJS. See: https://github.com/webdriverio/webdriverio/issues/342

This on is obsolete by now .. sorry @dmoreno for not responding. I will update to v2.45

No sorry. Good news! :)


Add appProtocol to service

Zitadel requires http2 as protocol to work properly. Whenever Istio is used, Istio determines the protocol to forward the traffic from the service specification. Either the name needs to be prefixed with http2 or the appProtocol needs to be set to http2. If this is not set Istio uses TCP to forward the traffic and Zitadel does not work properly.
Isitio reference: protocol-selection

@eliobischof do you have this on the radar?

@eliobischof do you have this on the radar?

Yes, I have


Refactors sidelining to use the new _try_specifizing_pnad method

Refactors sidelining code in preparation for non 1:1 sidelining implementation merge!
@ronuchit - I couldn't figure out how to nicely include the new test I want to. Could you fix up test_side_predicate_learning.py if you have some time tomorrow? I unfortunately have a bunch of other things to do, but could look into this more tomorrow evening ish.

Supercloud confirms that repeated_nextto is still fine. Also, both repeated_nextto_single_option and repeated_nextto_painting give the error:
AssertionError: 1:1 assumption violated
which is exactly what we expect.
However, there are at least two weirdnesses that need to be investigated:
This leads to an assertion failing, which I thought should never fail: python src/main.py --env repeated_nextto_painting --approach nsrt_learning --side_predicate_learner backchaining --seed 502 --num_train_tasks 50 --rnt_painting_num_objs_train [1] --rnt_painting_num_objs_test [1]
This leads to harmlessness failing, which I thought should not happen in the 1-object painting case since the machine problem doesn't occur: python src/main.py --env repeated_nextto_painting --approach nsrt_learning --side_predicate_learner backchaining --seed 502 --num_train_tasks 50 --rnt_painting_num_objs_train [1] --rnt_painting_num_objs_test [1] --painting_initial_holding_prob 0.0
cc @NishanthJKumar


Login Token problem

Forge CLI Version
1.0.8
PHP Version
8.2.7
Description
After put
forge login
later put TOKEN, give this error
An unexpected error occured. Please report this issue here:
https://github.com/laravel/forge-cli/issues/new/choose

Forge CLI Version: 1.8.0
PHP Version: 8.2.7
Operating System: WINNT
Error Message: cURL error 60: SSL certificate problem: unable to get local issuer certificate (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for https://fo
rge.laravel.com/api/v1/servers.

Steps To Reproduce
forge login

TOKEN

It seem you missing correct SSL certificate locally and this prevents the CLI client from communicating to Forge API, you need to fix your local PHP using similar solution to https://stackoverflow.com/a/34883260


ASG6: Introducing Unit Tests;

Issue #, if available:
ASG6
Description of changes:
@shashankj-aws @AWSJackson Here is a proof of concept for the unit tests. Will work on more if they are acceptable.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Moved to PR https://github.com/aws/amazon-gamelift-plugin-unity/pull/107


Steam not counting hours

Godot version:

v3.1.stable.official
Issue description:

Steam is counting hours spent in Godot only if you are in the Project Manager. Meaning while you are editing a projects, hours will not get counted.
What I suspect the issue is, is that the Godot executable exits, when you select the projects and starts again. Instead you could implement it so it makes a subprocess of itself. So with this solution, I do not think Steam specific binaries would need to be implemented.
Useful tool (for Windows): Process Explorer (can be downloaded from Microsoft's website).
Steps to reproduce:

Download Godot from Steam.
Open Godot.
Open a project.

Duplicate of #18233


In a hello world example:
package main

import (
	"fmt"
	_ "github.com/lib/pq"
)

func main() {
	fmt.Println("vim-go")
}

I ran:
dep init -v

What did you expect to see?
github.com/lib/pq vendored like any other dependency
What did you see instead?
A confusing failure. Not a git repository, but enough of a git repository to see the remote branches?
$ dep init -v
Root project is "stub/hello"
 1 transitively valid internal packages
 1 external packages imported from 1 projects
(0)   ✓ select (root)
(1)	? attempt github.com/lib/pq with 1 pkgs; 6 versions to try
(1)	    try github.com/lib/pq@master
(1)	✗   unable to update repository: fatal: Not a git repository (or any of the parent directories): .git
(1)	    try github.com/lib/pq@1895-fix
(1)	✗   unable to update repository: fatal: Not a git repository (or any of the parent directories): .git
(1)	    try github.com/lib/pq@499
(1)	✗   unable to update repository: fatal: Not a git repository (or any of the parent directories): .git
(1)	    try<EMAIL_ADDRESS>(1)	✗   unable to update repository: fatal: Not a git repository (or any of the parent directories): .git
(1)	    try github.com/lib/pq@pgpass
(1)	✗   unable to update repository: fatal: Not a git repository (or any of the parent directories): .git
(1)	    try<EMAIL_ADDRESS>(1)	✗   unable to update repository: fatal: Not a git repository (or any of the parent directories): .git
(1)	  ← no more versions of github.com/lib/pq to try; begin backtrack
  ✗ solving failed

Solver wall times by segment:
     b-source-exists: 6.321992483s
         b-list-pkgs:   43.45328ms
            new-atom:     328.36µs
             satisfy:    304.534µs
         select-root:    283.162µs
  b-deduce-proj-root:     58.775µs
     b-list-versions:     25.679µs
               other:      7.237µs

  TOTAL: 6.36645351s

No versions of github.com/lib/pq met constraints:
	master: unable to update repository: fatal: Not a git repository (or any of the parent directories): .git

	1895-fix: unable to update repository: fatal: Not a git repository (or any of the parent directories): .git

	499: unable to update repository: fatal: Not a git repository (or any of the parent directories): .git

	go1.0: unable to update repository: fatal: Not a git repository (or any of the parent directories): .git

	pgpass: unable to update repository: fatal: Not a git repository (or any of the parent directories): .git

	go1.0-cutoff: unable to update repository: fatal: Not a git repository (or any of the parent directories): .git

Cleaning out $GOPATH (removing $GOPATH/pkg and everything in $GOPATH/src that isn't mine) got it working. Something being used as a cache in here must have been bogus.

Thanks for reporting. I think we had similar issues before dep v0.1.0, which was fixed before the release.
@stub42 can you check your version of dep? (git describe --tags in $GOPATH/src/github.com/golang/dep directory)

I have been testing trunk, so probably 77df563b6a79bfe62932d91ed37e587c68878a56 .
govendor and 'go get' where the other tools that might have pulled things in.

No submodules or containers involved. Per https://github.com/golang/dep/issues/790#issuecomment-311010012 vendoring with dep worked again after I cleared out $GOPATH/pkg, so I'm assuming something non-standard ended up in here (like an empty directory where dep expected a git repo). Any evidence has been destroyed, so maybe just close this issue unless recovery can be done (ignore or remove the broken local copy and use the remote repo).

yeah, this is most likely a case of failure to terminate subprocesses gracefully on interrupt. it's a known issue - we just went for a lowest-common-denominator that would work cross platform, but it's too harsh, using os.Process.Kill(), which doesn't give e.g. git a chance to clean up after itself.


Panel control

Hello, the IP number appears, and when you type it in the browser to display the control panel.. a message appears: Connection rejected

Hello, add your IP first
sample https://your ip address


Rename the http module to helpers

to prevent conflicts with the built-in Python http library (fixes #1323)

Yes, this will probably break backward compatibility and it's not something we desire, but eventually we'll have to deal with these kind of changes to fix something as simple as a conflict with Python's own stdlib.

Could we target this change for the december release and be very clear about the breaking change in the announcement?


[Bug]: czkawka-gui v8.0 released 2 weeks ago not picked up

Prerequisites

[X] I have written a descriptive issue title.
[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.
[X] I have verified that I am using the latest version of Scoop and corresponding bucket.

Package Name
czkawka-gui
Expected/Current Behaviour
The latest version available through the extras bucket is 7.0.0. it should be 8.0.0 - the current release.
Steps to Reproduce
scoop update extras/czkawka-gui
czkawka-gui: 7.0.0 (latest version)
Latest versions for all apps are installed! For more information try 'scoop status'

Possible Solution
Manifest likely just needs updating, but I noticed this happened before for this specific project, so perhaps there is a larger issue for this package? Please see https://github.com/ScoopInstaller/Extras/issues/12408
Scoop and Buckets Version
Issue present in: https://github.com/ScoopInstaller/Extras/blob/master/bucket/czkawka-gui.json

Scoop Config
Issue present in: https://github.com/ScoopInstaller/Extras/blob/master/bucket/czkawka-gui.json

PowerShell Version
Issue present in: https://github.com/ScoopInstaller/Extras/blob/master/bucket/czkawka-gui.json

Additional Softwares
No response

@aliesbelik Thanks Alieksandr!


Set default completion token on ssl_stream

async methods

Ah yeah, it's been move constructible since 1.74 and move assignable since 1.77. And scatter-gather got into 1.74 as well.
Replacing boost::beast::ssl_stream with boost::asio::ssl::stream looks good in my code, PR to deprecate?

PR to deprecate, yes please

Scatter-gather, NICE !!!


Custom URL(JSON) option does not exist in Yomichan's Firefox version.

I am unable to get the add on to work in the Firefox version of Yomichan because the JSON option does not exist in the settings. Is there a way to work around this issue?

The firefox addon hasn't been updated since June 5, 2020.
You would have to sideload a newer version from the yomichan github

Sorry for spaming your notifications. I moved back to using chrome because I tried using Firefox to reduce my browser's cpu consumption but it did not work. In the future if anyone wants to try using Firefox they can perhaps try the step above I guess.


Add DataNodes usages Metrics in JMX Collector - EAGLE-513

https://issues.apache.org/jira/browse/EAGLE-513

Pls. clear the commit history and recreate another PR.


Better handling of switching training backends mid training

Can you elaborate more on why it should be a warning instead of an error? I think this will now cause errors later when torch.distributed attempts to do dist functions on gpu tensors, since the gloo backend is not supported there.
Originally posted by @ravi-mosaicml in https://github.com/mosaicml/composer/issues/277#issuecomment-1024253935

Closing for now since @hanlint and @ravi-mosaicml both confirmed that pytorch's destory_process_group does not properly free ports. Once this is addressed, we can re-evaluate. We currently hard-error if a new backend is requested after distributed has already been configured.


Serve raw files with the correct content types

Resolves #8152.
This is an alternative implementation to the existing PR for this issue.

I think this should be optional and default off for the sake of security. See https://github.com/go-gitea/gitea/issues/8152#issuecomment-814351404.

Thanks @silverwind.  PR updated.

See https://github.com/go-gitea/gitea/issues/8152#issuecomment-819724255 for a better suggestion.

While I can see merit in your suggestion, the changes in this PR could be explicitly enabled by users via a configuration option (as you originally suggested), with a warning stating the potential security implications.  It would also coexist with custom content types when they are implemented (i.e., check custom content types before detecting a file's content type).
Closing because the scope of the solution has grown beyond my interest.


Add cypress test for comments

In Volto 5.9.0 comments are broken without it being noticed right away. So it would be great to have a cypress test for them.
I already created a setregistry function which is usable to globally enable comments in the test setup: https://github.com/plone/volto/pull/1481

We need to:

[ ] Enable comments globally
[ ] Enable comments on createContent(..., allowDiscussion=True) (this should set allowDiscussion = True on the REST API)