[GoRouter] $extra is null after notifyListeners() forces reload

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Open the app and click on 'Go to extra page'
Click on 'Click to reload router'

This will first open a page that has as custom typed $extra parameter. Everything works as expected, the passed value for this parameter is handed to the new page. But if the router gets forced to reload via notifyListeners(), the $extra will be None, causing the app to raise an exception.
Expected results
Just reload the page with the same value for $extra as before.
Actual results
$extra is None
Code sample
Minimal flutter app to reproduce:
https://github.com/claudius-kienle/flutter-river-pod-bug
Screenshots or Video
No response
Logs
Logs
Restarted application in 3,403ms.
[log] Going to / with extra null
[log] Going to / with extra null
[log] Going to /extra-page with extra ExtraClass(content: This is the content)
[log] Going to /extra-page with extra null

════════ Exception caught by widgets library ═══════════════════════════════════
The following _TypeError was thrown building Builder:
Null check operator used on a null value

The relevant error-causing widget was
MaterialApp
When the exception was thrown, this was the stack
#0      ExtraPageRoute.build
#1      GoRouteData.$route.builder
#2      RouteBuilder._callGoRouteBuilder.<anonymous closure>
#3      Builder.build
#4      StatelessElement.build
#5      ComponentElement.performRebuild
#6      Element.rebuild
#7      StatelessElement.update
#8      Element.updateChild
#9      SingleChildRenderObjectElement.update
#10     Element.updateChild
#11     ComponentElement.performRebuild
#12     Element.rebuild
#13     StatelessElement.update
#14     Element.updateChild
#15     SingleChildRenderObjectElement.update
#16     Element.updateChild
#17     SingleChildRenderObjectElement.update
#18     Element.updateChild
#19     ComponentElement.performRebuild
#20     StatefulElement.performRebuild
#21     Element.rebuild
#22     StatefulElement.update
#23     Element.updateChild
#24     SingleChildRenderObjectElement.update
#25     Element.updateChild
#26     ComponentElement.performRebuild
#27     StatefulElement.performRebuild
#28     Element.rebuild
#29     StatefulElement.update
#30     Element.updateChild
#31     SingleChildRenderObjectElement.update
#32     Element.updateChild
#33     ComponentElement.performRebuild
#34     StatefulElement.performRebuild
#35     Element.rebuild
#36     StatefulElement.update
#37     Element.updateChild
#38     ComponentElement.performRebuild
#39     StatefulElement.performRebuild
#40     Element.rebuild
#41     StatefulElement.update
#42     Element.updateChild
#43     SingleChildRenderObjectElement.update
#44     Element.updateChild
#45     ComponentElement.performRebuild
#46     StatefulElement.performRebuild
#47     Element.rebuild
#48     StatefulElement.update
#49     Element.updateChild
#50     SingleChildRenderObjectElement.update
#51     Element.updateChild
#52     ComponentElement.performRebuild
#53     StatefulElement.performRebuild
#54     Element.rebuild
#55     StatefulElement.update
#56     Element.updateChild
#57     ComponentElement.performRebuild
#58     StatefulElement.performRebuild
#59     Element.rebuild
#60     StatefulElement.update
#61     Element.updateChild
#62     ComponentElement.performRebuild
#63     Element.rebuild
#64     StatelessElement.update
#65     Element.updateChild
#66     ComponentElement.performRebuild
#67     StatefulElement.performRebuild
#68     Element.rebuild
#69     StatefulElement.update
#70     Element.updateChild
#71     SingleChildRenderObjectElement.update
#72     Element.updateChild
#73     ComponentElement.performRebuild
#74     Element.rebuild
#75     ProxyElement.update
#76     _InheritedNotifierElement.update
#77     Element.updateChild
#78     SingleChildRenderObjectElement.update
#79     Element.updateChild
#80     ComponentElement.performRebuild
#81     StatefulElement.performRebuild
#82     Element.rebuild
#83     StatefulElement.update
#84     Element.updateChild
#85     ComponentElement.performRebuild
#86     Element.rebuild
#87     ProxyElement.update
#88     Element.updateChild
#89     ComponentElement.performRebuild
#90     Element.rebuild
#91     ProxyElement.update
#92     Element.updateChild
#93     ComponentElement.performRebuild
#94     StatefulElement.performRebuild
#95     Element.rebuild
#96     StatefulElement.update
#97     Element.updateChild
#98     ComponentElement.performRebuild
#99     Element.rebuild
#100    StatelessElement.update
#101    Element.updateChild
#102    ComponentElement.performRebuild
#103    Element.rebuild
#104    StatelessElement.update
#105    Element.updateChild
#106    SingleChildRenderObjectElement.update
#107    Element.updateChild
#108    ComponentElement.performRebuild
#109    Element.rebuild
#110    ProxyElement.update
#111    Element.updateChild
#112    ComponentElement.performRebuild
#113    Element.rebuild
#114    ProxyElement.update
#115    Element.updateChild
#116    ComponentElement.performRebuild
#117    StatefulElement.performRebuild
#118    Element.rebuild
#119    StatefulElement.update
#120    Element.updateChild
#121    ComponentElement.performRebuild
#122    StatefulElement.performRebuild
#123    Element.rebuild
#124    StatefulElement.update
#125    Element.updateChild
#126    ComponentElement.performRebuild
#127    StatefulElement.performRebuild
#128    Element.rebuild
#129    BuildOwner.buildScope
#130    WidgetsBinding.drawFrame
#131    RendererBinding._handlePersistentFrameCallback
#132    SchedulerBinding._invokeFrameCallback
#133    SchedulerBinding.handleDrawFrame
#134    SchedulerBinding._handleDrawFrame
#135    _invoke (dart:ui/hooks.dart:170:13)
#136    PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:401:5)
#137    _drawFrame (dart:ui/hooks.dart:140:31)
════════════════════════════════════════════════════════════════════════════════

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.7, on Arch Linux 6.5.7-arch1-1, locale C.UTF-8)
    • Flutter version 3.13.7 on channel stable at ~/.development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2f708eb839 (3 weeks ago), 2023-10-09 09:58:08 -0500
    • Engine revision a794cf2681
    • Dart version 3.1.3
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at ~/Android/Sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = ~/Android/Sdk
    • Java binary at: /opt/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /usr/bin/brave

[✓] Linux toolchain - develop for Linux desktop
    • clang version 16.0.6
    • cmake version 3.27.7
    • ninja version 1.11.1
    • pkg-config version 1.8.1

[✓] Android Studio (version 2023.1)
    • Android Studio at /opt/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] Connected device (3 available)
    • Pixel 6a (mobile) • <IP_ADDRESS>:5555 • android-arm64  • Android 14 (API 34)
    • Linux (desktop)   • linux               • linux-x64      • Arch Linux 6.5.7-arch1-1
    • Chrome (web)      • chrome              • web-javascript • Brave Browser <IP_ADDRESS>

[✓] Network resources
    • All expected network resources are available.

• No issues found!```

</details>

Hi @claudius-kienle
Could you narrow down the sample code by only using go_router_builder and ValueNotifier to see if the issue still persists?

Hi @huycozy,
sure! I just simplified the example and removed as many dependencies as possible.
The issue still persists. One thing though is that if I replace the custom class ExtraClass as type for $extra with a standard type, e.g. String, the exception does not occur and the code runs as intended.

Thanks for the update.

One thing I figured out though is that if I replace the custom class ExtraClass as type for $extra with a standard type, e.g. String, the exception does not occur and the code runs as intended.

Yes, it's indeed the cause. You are using a custom object for extra parameter which is not supported yet, please see Type conversions and a known proposal for this at https://github.com/flutter/flutter/issues/110781.
Closing this issue in favor of #110781. Please follow up on it for updates.


[Bug]: Deprecation Warning: Use of fillna(method='ffill') Deprecated in Pandas

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
While executing the code it displayed an error that Series.fillna is deprecated and we need to use obj.ffill or obj.bfill instead

Add ScreenShots
No response
What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Can you assign me to work on this issue?

@sanjay-kv Please assign #396 and #397 also


DJango API Documentation: Initial Setup (peter)

The API: reads the db and serves back JSON.
Configure and research api provider within django.
build api for a test schema
setup against test schema
document process

Review the documentation provided in the Readme and ensure it works on your local machine.
Provide any updates as needed as a pull request to the DEV branch. If you get stuck, just provide that information in the comments.

So far, I've set up postgres to run, and added a model for the Occupation Transitions data, and imported that data into the table. I've been trying to figure out the api that is currently on the master branch (I believe what Sean screen shared last week, but I haven't had time to go through it with him to understand what he did). We are going to zoom so he can fill me in, and then I'll try to get it working for the Occupation Transition data to practice CRUD operations.

Should I be setting up a local postgres database on my machine named 'jobhopperdatabase'? I've followed up to Step 8 in the Readme and am getting a 'database "jobhopperdatabase" does not exist' from manage.py runserver. Thanks!

Peter, I think that you've done everything specified in the bullets. Can you document the process to do everything you've done (with a pull request for any changes to the instructions needed?) and then move this over to in-review so someone can take a look (either me or nathan). Once the review looks good, we can comment here that it has been reviewed.
There are other issues and if you go further down your path, you might overlap with other folks' work and I just want to avoid wasting other folks' time. If the work you want to do is not in the issues list, please feel free to add an issue (or multiple) so we can groom (discuss) them together Tuesday.
Once you get your work to in-review, you should feel free to grab another issue from near the top of the "to do" in the sprint board that you think you can tackle. Thanks

Should I be setting up a local postgres database on my machine named 'jobhopperdatabase'? I've followed up to Step 8 in the Readme and am getting a 'database "jobhopperdatabase" does not exist' from manage.py runserver. Thanks!

Yes, so first you need to set up your postgres server. We originally had it running in sqlite which was a bit easier to work with, but the current version requires that you have postgres set up. You can install it from here https://www.postgresql.org/download/ and you can use something like postico (Sean mentioned it last zoom) as a GUI rather than having to run commands through the terminal. You just have to make sure that the name and port of the database you create on your local postgres server matches the ones in the DATABASES section of the jobhopper/jobhopper/settings.py file. It's towards the bottom of the page. Then, you will have connected them, but there won't be any tables. Once you run the migration commands, it should create tables matching the models created in the jobhopper/jobs/models.py.

Peter, I think that you've done everything specified in the bullets. Can you document the process to do everything you've done (with a pull request for any changes to the instructions needed?) and then move this over to in-review so someone can take a look (either me or nathan). Once the review looks good, we can comment here that it has been reviewed.
There are other issues and if you go further down your path, you might overlap with other folks' work and I just want to avoid wasting other folks' time. If the work you want to do is not in the issues list, please feel free to add an issue (or multiple) so we can groom (discuss) them together Tuesday.
Once you get your work to in-review, you should feel free to grab another issue from near the top of the "to do" in the sprint board that you think you can tackle. Thanks

@jedpittman Sure! I'm at my parents this weekend, but I can do a pull request tomorrow when I can sit down and make sure everything is good to go. So I'll do a pull request for changes to the README, and do you want the updated models.py with the model for the Occupation Transition data, or should I wait on that?

I'd wait on that piece because the data folks need to decide the layouts etc. And I caught up to you guys but let's make sure that the migration commands make sense because when I created an empty db and updated settings, I found that it didn't want to do any migrations for jobs.
Also, it looks like some windows specific instructions on VENV would help. e.g. for my machine:
C:\Users\jedpi\OneDrive\Documents\GitHub\jobhopper-forked\venv\Scripts>activate
(venv) C:\Users\jedpi\OneDrive\Documents\GitHub\jobhopper-forked\venv\Scripts>deactivate
So something that helps people understand that would be useful; the onboarding documentation should not require that the person understand python very much, if at all.

@jedpittman Did it give an error message when you ran the migrations? If so, what did it say? I just made a slight alteration to the models.py, and when I ran the the three commands in the migrations section, this is what I got.
`(projects) (base) petergibb@macbook-pro jobhopper % python manage.py migrate
Operations to perform:
Apply all migrations: admin, auth, contenttypes, jobs, sessions
Running migrations:
No migrations to apply.
Your models have changes that are not yet reflected in a migration, and so won't be applied.
Run 'manage.py makemigrations' to make new migrations, and then re-run 'manage.py migrate' to apply them.
(projects) (base) petergibb@macbook-pro jobhopper % python manage.py makemigrations jobs
Migrations for 'jobs':
jobs/migrations/0005_auto_20200914_1220.py
- Alter field occleaveshare on occupationtransitions
- Alter field pi on occupationtransitions
(projects) (base) petergibb@macbook-pro jobhopper % python manage.py migrate
Operations to perform:
Apply all migrations: admin, auth, contenttypes, jobs, sessions
Running migrations:
Applying jobs.0005_auto_20200914_1220... OK`
Also, for VENV, do you only want to include the windows specific activation/deactivation commands you listed?

Those commands were the only ones that were windows specific. Ill try to to run the process again before Tuesday Peter and if I am still stuck I’ll send you the message so we can get this buttoned up by end of Tuesday night.

On Sep 14, 2020, at 8:24 AM, pgibb96<EMAIL_ADDRESS>wrote:

@jedpittman Did it give an error message when you ran the migrations? If so, what did it say? I just made a slight alteration to the models.py, and when I ran the the three commands in the migrations section, this is what I got.
`(projects) (base) petergibb@macbook-pro jobhopper % python manage.py migrate
Operations to perform:
Apply all migrations: admin, auth, contenttypes, jobs, sessions
Running migrations:
No migrations to apply.
Your models have changes that are not yet reflected in a migration, and so won't be applied.
Run 'manage.py makemigrations' to make new migrations, and then re-run 'manage.py migrate' to apply them.
(projects) (base) petergibb@macbook-pro jobhopper % python manage.py makemigrations jobs
Migrations for 'jobs':
jobs/migrations/0005_auto_20200914_1220.py

Alter field occleaveshare on occupationtransitions
Alter field pi on occupationtransitions
(projects) (base) petergibb@macbook-pro jobhopper % python manage.py migrate
Operations to perform:
Apply all migrations: admin, auth, contenttypes, jobs, sessions
Running migrations:
Applying jobs.0005_auto_20200914_1220... OK`

Also, for VENV, do you only want to include the windows specific activation/deactivation commands you listed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Did Endy's pull request complete this issue? He added the final step which was to go the api/leads url in order to interact with the API. Is there more to this issue? I'll grab another issue for now

I think this is complete for now. I have been able to follow the readme and I see the leads api working. I'm closing this.


blessc: input file not a .css file

I can't count CSS selectors using blessc count with blessc 3.0.3 with node 0.10.40 on Ubuntu 14.04.3 LTS:

echo 'body { color: green; }' >foo.css
blessc count foo.css
blessc: input file not a .css file

The blessc count command is new to v4, which is currently just an alpha release on npm. Run npm install<EMAIL_ADDRESS>and then give it a try.

I see, didn't know that. Thanks for clarification.


curtail: init at 1.3.0

Description of changes
Added Curtail, a simple GTK image compression utility, as curtail
This is a reopening of PR #173397 due to force pushes. I have tested functionality and localization.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 173520 run on x86_64-linux 1

1 package built:

curtail

I couldn't be able to launch curtail. Fails with
❯ /nix/store/xhjk27j7skafbrwbn6l54vpxqrmqxaf7-curtail-1.3.0/bin/curtail
Traceback (most recent call last):
  File "/nix/store/zm839ag6yaapyivq3h0jzi2kc0a7fivl-python3.9-pygobject-3.42.1/lib/python3.9/site-packages/gi/importer.py", line 142, in load_module
    introspection_module = get_introspection_module(namespace)
  File "/nix/store/zm839ag6yaapyivq3h0jzi2kc0a7fivl-python3.9-pygobject-3.42.1/lib/python3.9/site-packages/gi/module.py", line 257, in get_introspection_module
    module = IntrospectionModule(namespace, version)
  File "/nix/store/zm839ag6yaapyivq3h0jzi2kc0a7fivl-python3.9-pygobject-3.42.1/lib/python3.9/site-packages/gi/module.py", line 109, in __init__
    repository.require(namespace, version)
gi.RepositoryError: Typelib file for namespace 'Pango', version '1.0' not found

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/nix/store/xhjk27j7skafbrwbn6l54vpxqrmqxaf7-curtail-1.3.0/bin/...curtail-wrapped-wrapped-wrapped", line 43, in <module>
    from curtail import main
  File "/nix/store/xhjk27j7skafbrwbn6l54vpxqrmqxaf7-curtail-1.3.0/share/curtail/curtail/main.py", line 23, in <module>
    from gi.repository import Gtk, Gio
  File "/nix/store/zm839ag6yaapyivq3h0jzi2kc0a7fivl-python3.9-pygobject-3.42.1/lib/python3.9/site-packages/gi/importer.py", line 144, in load_module
    raise ImportError(e)
ImportError: Typelib file for namespace 'Pango', version '1.0' not found

I had left out Pango in the build inputs on refactoring. I’ve added it back now.

It still doesn’t work. I seem to have messed up something.

Oh. I had actually removed GTK as a dependency! Thank you for noticing.

Result of nixpkgs-review pr 173520 run on x86_64-linux 1

1 package built:

curtail

Works fine but output has multiple wrappers.
❯ ls -a /nix/store/zxi8vlhvwmjrcziqdc9al9hd3ks6x88q-curtail-1.3.0/bin/
.  ..  curtail  .curtail-wrapped  .curtail-wrapped_  ..curtail-wrapped-wrapped  ...curtail-wrapped-wrapped-wrapped

Is that a big problem? It wraps once for the Python path, once for GApps & GIO, and once for putting the tools into the path.
I could add the tools into the extra arguments variables for one of these, but this seems more idiomatic.
Should I maybe move the two first wrappers into libexec before wrapping?

I guess they are from that. I don’t see the problem with these additional wraps.

Sorry, I bollocksed up the branch with a wrong rebase.

I have reopened this at #174712


elmakers domain is inaccessible

Connection to the elmakers TDL and subdomains keep timing out.

Sorry, had a fire and a power outage! Should be fixed now- I'll leave this open for now, but if I don't hear from you I'll assume all is well :)

Yep. Everything looks back up and operational. Thanks!
On Thu, Sep 3, 2015 at 12:17 PM, Nathan Wolf<EMAIL_ADDRESS>wrote:

Sorry, had a fire and a power outage! Should be fixed now- I'll leave this
open for now, but if I don't hear from you I'll assume all is well :)
—
Reply to this email directly or view it on GitHub
https://github.com/elBukkit/MagicPlugin/issues/2#issuecomment-137500169.

--
Markus OdomSoftware Engineer
Mobile:<PHONE_NUMBER>
Email<EMAIL_ADDRESS>


Fix tests for eslint 3.2

The added a dot at the end of the no undef message. Now the tests fail either for version 2 or 3 :(

Now the tests fail either for version 2 or 3 :(

lol 😓

sigh, we should just drop support for eslint 2 at this point then?

Seems reasonable to me @hzoo.

Since we don't use peerDeps, if we are dropping we should a major for that?

Since we don't use peerDeps, if we are dropping we should a major for that?

Being conservative about it seems good. I'm :+1: to a major bump.


Include mergeBufferGeometries and mergeBufferAttributes methods

Added these two methods from the current BufferGeometryUtils here:
https://github.com/mrdoob/three.js/blob/master/examples/js/BufferGeometryUtils.js

@ivee-tech Can we get this PR approved? We also would like to leverage it form here rather than paste it into our project.

Wonderful! Can you make a new release in npmjs?


PauliFrame initZ! not working properly

I don't know why we haven't noticed this until now, but shouldn't the measurements be half zeros and half ones? I thought for certain this used to work at one point.

The reason this came up was that I was quite skeptical that pauli frames would work to perform the "double syndrome measurement" encoding circuit, thinking mostly about how it would handle entangled states post-measurement . I wanted see if it had correct output for measuring the bell state, but it was all 0s, which lead to this discovery...

I think it is just a minor bug in how you define the sMZ measurement. It needs two arguments, not just one. Implicitly, if you have only one argument it means "measure this qubit and discard the result".
This works: pftrajectories(PauliFrame(20,1,1), [sHadamard(1), sMZ(1,1)]) |> pfmeasurements

We should have a warning when the circuit contain measurement that discard the result (i.e. when we have sMZ(qubit) instead of sMZ(qubit,bit)).

Oh, I feel a bit silly now, but at least it lead to something!


[BUG]  Len of none null, but nullable, buffer, len is written twice

Describe the bug

In case of nullable buffer, if the buffer is not null the client writes the length twice. Once explicitly and once as part of writeBinary
To Reproduce

run erpcgen on this example file:
example.erpc.txt
You will see in the output client side:
        **codec->write(len);**

        if (buff == NULL)
        {
            codec->writeNullFlag(true);
        }
        else
        {
            codec->writeNullFlag(false);
            **codec->writeBinary(len, buff);**
        }

and in server side:
   **codec->read(&len);**

    codec->readNullFlag(&isNull);
    if (!isNull)
    {
        uint32_t lengthTemp_0;
        uint8_t * dataTemp_0;
        **codec->readBinary(&lengthTemp_0, &dataTemp_0);**
        **len = lengthTemp_0;**
        if ((lengthTemp_0 <= 10) && (dataTemp_0 != NULL))
        {
            buff = (uint8_t *) erpc_malloc(10 * sizeof(uint8_t));
            if ((buff == NULL) && (lengthTemp_0 > 0))
            {
                codec->updateStatus(kErpcStatus_MemoryError);
            }
            else
            {
                memcpy(buff, dataTemp_0, lengthTemp_0);
            }
        }
        else
        {
            codec->updateStatus(kErpcStatus_Fail);
        }
    }
    else
    {
        buff = NULL;
    }

Expected behavior

Client side:
    if (buff == NULL)
    {
        codec->writeNullFlag(true);
        **codec->write(len);**
    }
    else
    {
        codec->writeNullFlag(false);
        **codec->writeBinary(len, buff);**
    }

server side:
codec->readNullFlag(&isNull);
   if (!isNull)
   {
       uint32_t lengthTemp_0;
       uint8_t * dataTemp_0;
       **codec->readBinary(&lengthTemp_0, &dataTemp_0);**
       **len = lengthTemp_0;**
       if ((lengthTemp_0 <= 10) && (dataTemp_0 != NULL))
       {
           buff = (uint8_t *) erpc_malloc(10 * sizeof(uint8_t));
           if ((buff == NULL) && (lengthTemp_0 > 0))
           {
               codec->updateStatus(kErpcStatus_MemoryError);
           }
           else
           {
               memcpy(buff, dataTemp_0, lengthTemp_0);
           }
       }
       else
       {
           codec->updateStatus(kErpcStatus_Fail);
       }
   }
   else
   {
       **codec->read(&len);**
       buff = NULL;
   }

Desktop (please complete the following information):

OS:Windows
eRPC Version:1.9.1

Steps you didn't forgot to do

[X] I checked if there is no related issue opened/closed.
[X] I checked that there doesn't exist opened PR which is solving this issue.

Additional context

Changing this behavior in just one side, client or server, will break applications.
Hence I think it can't be part of version 1.10.XX
It probably will need to be as part of 2.0.0
You can add this fix in erpc templates under some flag.

Nice catch @amgross , are you willing to create PR for this bug?

I tried to do it first, but I failed to understand the flow in the templates.

I see what is the issue. The fix has to be done on completely different places. I will provide fix on your branch.

I will provide fix on your branch

Which branch? the PR I made is for different issue.

we can do more complex change next year once i will finish multiple client support

Just reminding my disclaimer at  Additional context here, this is breaking change.

If you see simpler solution let me know

If I will think of such ting I will, currently not.

@amgross Sorry i saw your PR for short time and connect it together.
Definitely it is breaking change as we will need change serialization steps.


Module translations

Few, if any, of the 'Module' sections at the top of these .md Instructions files has the Module title translated.  Most localized repos do this, is it planned to be done to this (and any other DP-420) localized repo?

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

please help provide locfile path. thanks


Implement image tags for web and webapi

Summary
Enable image tags for builds for each environments.  Currently, image builds are getting created with a "latest" tag for all environments. This change will add env specfic tags to each build which will ensure that only required deployments get  refreshed/recreated in the appropriate environment.
Changes

Changed deploy.yaml to pass appropriate image tags to the CI/CD pipeline
Changed build config and deployment config templates for web and api to incorporate relevant image tags

Looks good. We'll have to also do the same for the other services that are not managed by the GH Actions.

Formio
Redash
Camunda
File service

Confirmed with @kkapoor-fresh That there will be a separate PR with the other services once these have been verified working 🙃


add validation-api dependency. #7055

https://github.com/apache/dubbo/issues/7055

Codecov Report

Merging #7056 (7cebe38) into master (47fa631) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #7056      +/-   ##
============================================
- Coverage     59.31%   59.27%   -0.04%
  Complexity      508      508
============================================
  Files          1028     1028
  Lines         41527    41527
  Branches       6038     6038
============================================
- Hits          24631    24617      -14
- Misses        14141    14157      +16
+ Partials       2755     2753       -2

Impacted Files
Coverage Δ
Complexity Δ

...ng/transport/dispatcher/WrappedChannelHandler.java
47.82% <0.00%> (-15.22%)
0.00% <0.00%> (ø%)

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-10.35%)
0.00% <0.00%> (ø%)

.../apache/dubbo/remoting/transport/AbstractPeer.java
63.04% <0.00%> (-4.35%)
0.00% <0.00%> (ø%)

...pache/dubbo/registry/support/AbstractRegistry.java
77.77% <0.00%> (-1.86%)
0.00% <0.00%> (ø%)

...g/apache/dubbo/registry/consul/ConsulRegistry.java
60.00% <0.00%> (-0.59%)
31.00% <0.00%> (ø%)

...apache/dubbo/common/extension/ExtensionLoader.java
75.33% <0.00%> (-0.23%)
0.00% <0.00%> (ø%)

...rg/apache/dubbo/common/timer/HashedWheelTimer.java
63.57% <0.00%> (+0.34%)
0.00% <0.00%> (ø%)

.../remoting/transport/netty4/NettyClientHandler.java
67.79% <0.00%> (+6.77%)
0.00% <0.00%> (ø%)

Continue to review full report at Codecov.

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


x/net/route: TestRouteMessage failures on freebsd-386 builders

--- FAIL: TestRouteMessage (0.00s)
    message_test.go:235: dst|gateway|ifp|ifa|brd (pmtu=0) (inet4 <IP_ADDRESS>) (link 2 <nil> <nil>) (link 2 lo0 <nil>) (inet4 <IP_ADDRESS>) (inet4 <IP_ADDRESS>)
    message_test.go:235: dst|gateway (pmtu=0) (inet4 <IP_ADDRESS>) (link 2 <nil> <nil>)
    message_test.go:223: {0 4 0 0 1160 3 <nil> [(inet6 0000:0000:0000:0000:0000:0000:0000:0001 0) <nil> <nil> <nil> (link 0 <nil> <nil>) (inet6 0000:0000:0000:0000:0000:0000:0000:0000 0) <nil> (inet6 0000:0000:0000:0000:0000:0000:0000:0000 0)] 0 []}: invalid address
FAIL
FAIL	golang.org/x/net/route	0.006s

2019-11-09T02:19:31-daa7c04/freebsd-386-11_2
2019-11-09T02:19:31-daa7c04/freebsd-386-12_0
CC @tklauser @mikioh @bradfitz

Only 386? Always or a flake?

Always or a flake?

Looks like a recurring flake, but getting a better answer to that question is part of my motivation for filing #35515.

2020-10-16T16:51:38-7b1cca2/freebsd-386-11_2
2020-10-09T03:24:41-dbdefad/freebsd-386-11_2
2020-10-06T15:34:59-a7d1128/freebsd-386-12_0
2020-09-30T14:50:03-4acb6c0/freebsd-386-11_2
2020-09-27T03:25:02-5d4f700/freebsd-386-11_2
2020-09-27T03:25:02-5d4f700/freebsd-386-12_0
2020-09-23T18:22:12-328152d/freebsd-386-11_2
2020-09-23T18:22:12-328152d/freebsd-386-12_0
2020-09-04T19:48:48-62affa3/freebsd-386-11_2
2020-09-04T19:48:48-62affa3/freebsd-386-12_0
2020-08-13T13:45:08-3edf25e/freebsd-386-11_2
2020-07-07T03:43:11-ab34263/freebsd-386-11_2
2020-07-07T03:43:11-ab34263/freebsd-386-12_0
2020-06-25T00:16:55-4c52546/freebsd-386-11_2
2020-06-25T00:16:55-4c52546/freebsd-386-12_0
2020-06-02T11:40:24-627f964/freebsd-386-11_2
2020-05-05T04:18:28-1ed2336/freebsd-386-12_0
2020-05-01T05:30:45-e0ff5e5/freebsd-386-11_2
2020-05-01T05:30:45-e0ff5e5/freebsd-386-12_0
2020-04-25T23:01:54-ff2c4b7/freebsd-386-11_2
2020-04-21T23:12:49-e086a09/freebsd-386-11_2
2020-03-24T14:37:07-d3edc99/freebsd-386-11_2
2020-03-01T02:21:30-244492d/freebsd-386-12_0
2020-02-26T12:10:28-0de0cce/freebsd-386-11_2
2020-02-22T12:55:58-5a598a2/freebsd-386-11_2
2019-12-06T10:30:17-1ddd1de/freebsd-386-12_0
2019-12-04T02:50:24-5ee1b9f/freebsd-386-11_2
2019-11-26T23:54:20-ef20fe5/freebsd-386-11_2
2019-11-25T08:49:36-ffdde10/freebsd-386-12_0
2019-11-19T07:31:36-fc4aabc/freebsd-386-11_2
2019-11-19T07:31:36-fc4aabc/freebsd-386-12_0
2019-11-18T18:34:10-d06c31c/freebsd-386-11_2
2019-11-18T18:34:10-d06c31c/freebsd-386-12_0
2019-11-16T16:09:21-f9c8255/freebsd-386-12_0

2021-01-19T19:43:25-5f4716e/freebsd-386-11_2
2021-01-19T19:43:25-5f4716e/freebsd-386-12_0
2020-12-16T05:46:12-986b41b/freebsd-386-11_2
2020-12-09T12:38:23-ac852fb/freebsd-386-11_2
2020-12-09T12:38:23-ac852fb/freebsd-386-12_0
2020-12-02T16:19:06-c7110b5/freebsd-386-11_2
2020-11-09T17:26:40-a11eb1b/freebsd-386-11_2
2020-10-31T05:49:03-ff519b6/freebsd-386-11_2
2020-10-29T22:17:08-28c70e6/freebsd-386-11_2
2020-10-29T05:33:32-8adf50f/freebsd-386-11_2
2020-10-27T13:37:19-8eef523/freebsd-386-12_0
2020-10-22T23:12:55-08b3837/freebsd-386-11_2
2020-10-21T03:54:29-f585440/freebsd-386-11_2
2020-10-21T03:54:29-f585440/freebsd-386-12_0
2020-10-16T16:51:38-7b1cca2/freebsd-386-12_0

Weekly check-in: this needs to be investigated before beta 1.
cc @neild

I have not yet figured out what's going on here, but it's not 1.17 specific. The problem is easily reproducible with -count=1000 and 1.16.

Removing release-blocker, then.

@heschi, I believe the freebsd-12_2-386 builder is new as of the Go 1.17 cycle.
Unless I am mistaken, from #11811 it follows that part of the process of setting up a new builder needs to include either fixing or adding skips for the existing tests that are already failing on that builder.
Since the builder is new for Go 1.17, I believe this issue should still be a 1.17 release-blocker.

Note that the failure rate on the freebsd-12_2-386 builder is qualitatively, if not quantitatively, different from the failure rate on the previous 11_2and current11_4` builders:

I cannot explain why the qualitative appearance of the dashboard does not seem to match the more even distribution of failures in the logs downloaded by fetchlogs. That discrepancy may be worth investigating further, but I don't plan to pursue it at this time.

2021-05-10T12:01:50-4163338/freebsd-386-11_4
2021-05-10T12:01:50-4163338/freebsd-386-12_2
2021-05-10T09:51:57-81045d8/freebsd-386-11_2
2021-05-10T09:51:57-81045d8/freebsd-386-12_2
2021-05-08T05:16:33-16afe75/freebsd-386-12_2
2021-05-05T21:49:59-0714010/freebsd-386-11_4
2021-05-05T21:49:59-0714010/freebsd-386-12_2
2021-05-05T02:47:14-0287a6f/freebsd-386-12_2
2021-05-04T13:21:25-bbd867f/freebsd-386-11_4
2021-05-04T13:21:25-bbd867f/freebsd-386-12_2
2021-05-03T06:03:51-7fd8e65/freebsd-386-11_2
2021-05-03T06:03:51-7fd8e65/freebsd-386-11_4
2021-05-03T06:03:51-7fd8e65/freebsd-386-12_2
2021-05-02T03:00:24-e590880/freebsd-386-12_2
2021-05-01T22:26:12-f8dd838/freebsd-386-11_2
2021-05-01T22:26:12-f8dd838/freebsd-386-12_2
2021-05-01T14:20:56-aec3718/freebsd-386-11_4
2021-05-01T14:20:56-aec3718/freebsd-386-12_2
2021-04-28T14:07:49-89ef3d9/freebsd-386-12_2
2021-04-27T23:12:57-85d9c07/freebsd-386-12_2
2021-04-23T18:45:38-5f58ad6/freebsd-386-11_4
2021-04-23T18:45:38-5f58ad6/freebsd-386-12_2
2021-04-23T17:40:36-e997de6/freebsd-386-12_2
2021-04-21T23:01:15-4e50805/freebsd-386-12_0
2021-04-21T23:01:15-4e50805/freebsd-386-12_2
2021-04-20T21:01:06-798c215/freebsd-386-11_2
2021-04-15T23:10:46-e915ea6/freebsd-386-11_2
2021-04-15T23:10:46-e915ea6/freebsd-386-12_0
2021-04-14T19:42:28-0645797/freebsd-386-11_2
2021-04-05T18:03:19-a5a99cb/freebsd-386-11_2
2021-03-31T21:22:08-0fccb6f/freebsd-386-11_2
2021-03-31T06:09:03-cb1fcc7/freebsd-386-12_0
2021-03-30T21:00:36-cd0ac97/freebsd-386-11_2
2021-03-30T21:00:36-cd0ac97/freebsd-386-12_0
2021-03-26T22:08:43-6ef6e9b/freebsd-386-11_2
2021-03-26T06:03:03-6b15177/freebsd-386-11_2
2021-03-24T20:56:30-d1beb07/freebsd-386-11_2
2021-03-24T20:56:30-d1beb07/freebsd-386-12_0
2021-03-16T09:26:52-d523dce/freebsd-386-11_2
2021-02-26T17:20:49-e18ecbb/freebsd-386-11_2
2021-02-26T17:20:49-e18ecbb/freebsd-386-12_0
2021-02-26T10:14:13-39120d0/freebsd-386-11_2
2021-02-24T08:20:22-3d97a24/freebsd-386-12_0
2021-02-22T17:17:44-9060382/freebsd-386-12_0
2021-02-20T03:31:24-5f55cee/freebsd-386-11_2

The failing test reads routing information from an AF_ROUTE socket and parses it.
The information read from the socket does not match the expected format. The test passes most of the time, because most of the time the parser doesn't realize there's a problem.
You can easily reproduce this on the freebsd-386-11_2 builder with go test ./route -run=RouteMessage/2 -count=1000 -failfast.
Here are the bytes provided to ParseRIB for an example failure:
rt_msghdr
48010504 02000000 45003000 33000000 5f080000 03000000 00000000 00000000
00000000 00000000 00000000 00000000 00400000 00000000 00000000 00000000
00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000
00000000 00000000 00000000 00000000 00000000 00000000 01000000 00000000
00000000 00000000 00000000 00000000 00000000 00000000

sockaddr
                                                      1c1c0000 00000000
00000000 00000000 00000000 00000001 00000000

unknown data
                                             ffffffff 36120200 18000000
00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000
00000000 00000000 00000000 00000000 38120200 18030000 6c6f3000 00000000
00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000
00000000 00000000

sockaddr
                  1c1c0000 00000000 00000000 00000000 00000000 00000001
00000000

unknown data
         ffffffff

This data is expected to be an rt_msghdr struct, followed by an array of sockaddrs. The number of sockaddrs is specified in the header--2 in this case.
There are what appear to be two IPv6 sockaddrs in the data here, each consisting of 28 bytes starting with 1c1c.
There is also a chunk of data between the sockaddrs--everything from the ffffffff to the next 1c1c. We parse the first sockaddr, and then attempt to parse the ffffffff as a sockaddr and fail. The times the test passes, the chunk of data contains some different data which parses correctly.
I've poked through the source code for FreeBSD utilities which parse this same data, and none of them seem to handle this case. For example the code here looks pretty similar to that in x/net/route:
https://github.com/freebsd/freebsd-src/blob/main/usr.bin/netstat/route.c#L317
I'm very puzzled by what's going on here. This test is definitely pointing out a real problem. It is flaky only because the "successful" runs aren't detecting that they're parsing invalid data.

I think you are misreading the data slightly.  The number of addresses is not 2, but is determined by the number of bits set in the rtm_addrs field.  In this case the value 0x33 indicates that there are four addresses.  The first is an AF_INET6 address, the next two are AF_LINK addresses, and the last one is another AF_INET6 address.  Viewed this way we can see that the 0xffffffff values appear after the AF_INET6 addresses, which strongly suggests that the address lengths are being rounded up to a multiple of 8 bytes, in this case rounding from 0x1c to 0x20.  That makes me suspect that we are running a 386 program on an amd64 kernel.  And that makes me suspect that https://golang.org/cl/139577 was mistaken, or perhaps that it simply doesn't apply to whatever system we are running.  See what happens if you revert that CL.

You're right, I was misreading it.
If I'm following the FreeBSD code correctly, after this change, sockaddrs in routing messages returned by the sysctl syscall with the SCTL_MASK32 flag set are 32-bit aligned. The test here is reading routing messages from an AF_ROUTE routing socket. Empirically, this returns 64-bit aligned sockaddrs.
We do also make a sysctl syscall elsewhere in this test, which is also returning 64-bit aligned sockaddrs.
I'm not clear on who sets the SCTL_MASK32 flag (userspace? kernel?), but so far as I can tell all the routing messages we're reading are 64-bit aligned.


Fix hot module main flag for hot module replacement

What kind of change does this PR introduce?
Bugfix for HMR

Did you add tests for your changes?

Yes, added a test case which reproduces the issue if the fix is not in place. The test case requires using CommonsChunkPlugin and multiple entries, but there could be a simpler test case.
Summary

When a main module is loaded after a module which is already loaded, the main module should still be marked as a main module, but the current logic causes a main module to be marked a not a main module in this case, which causes issues with HMR. When an incorrectly marked main module is replaced, the current HMR logic will process the update and mark the module as accepted even though it was not.
To fix this issue, this change handles that case by making sure that hotMainModule is always true after a require from hotCreateRequire. hotMainModule should be set to true in require from hotCreateRequire in the case where hotCreateModule (which sets hotCreateModule to true) is not called. hotCreateModule is not called when the requested module is already loaded.
Reported corresponding issue: https://github.com/webpack/webpack/issues/4621
Does this PR introduce a breaking change?
No, unless there are some people who rely on unexpected behavior of HMR accepting modules with no accept handlers.

Thanks


Variable Download URLs

Helpful for an org

Thanks for this. Do you mind providing a bit more information about what problem you're trying to solve?

@benbalter
check this https://github.com/uMatrix-Rules
the gh page is hosted on https://github.com/uMatrix-Rules/uMatrix-Rules.github.io
And I want to provide download URLs toward files from the rest of repos


Not showing all icons on Tech Reborn mod on Fabric

What does this mean?

Closing for no response.


Refactor api --> http

Looks good.

@jmarin, looks like you need to resolve a merge conflict before this can be merged.

That's better.  Merging...


Migrate to Eslint

This PR has config for Eslint and Prettier. Also added format script to automatically format files.

This is still WIP tho. Although I mapped all typescript related rules, still there are missing rules from eslint like cyclomatic-complexity.


Multiclusterapp Roles changes

Types PR: https://github.com/rancher/types/pull/751
https://github.com/rancher/rancher/issues/18521
https://github.com/rancher/rancher/issues/18494
https://github.com/rancher/rancher/issues/18495

LGTM


[ISSUE-222][FOLLOWUP] GetBlacklist/GetBlacklistResponse should replace null value with empty list

What changes were proposed in this pull request?
Why are the changes needed?

What are the items that need reviewer attention?
Related issues.
Related pull requests.
How was this patch tested?
/cc @related-reviewer
/assign @main-reviewer

ping @waitinfuture


Spark Program on sbt console keeps running

steps
I've the following build file.
name := "SparkScalaCourse"

version := "0.1"

scalaVersion := "2.12.17"

libraryDependencies ++= Seq(
  "org.apache.spark" %% "spark-core" % "3.0.0",
  "org.apache.spark" %% "spark-sql" % "3.0.0",
  "org.apache.spark" %% "spark-mllib" % "3.0.0",
  "org.apache.spark" %% "spark-streaming" % "3.0.0",
  "org.twitter4j" % "twitter4j-core" % "4.0.4",
  "org.twitter4j" % "twitter4j-stream" % "4.0.4"
)

and a simple code snippet
import org.apache.spark._
val sc = new SparkContext("local[1]", "SimpleProg")
val nums = sc.parallelize(List(1, 2, 3, 4));
println(nums.reduce((a, b) => a - b))

running the code in sbt console doesn't produce result. I'm using sbt version 1.8.2

problem
The above works fine when running the code in the main function in an object in a scala file. However if I open sbt console and run the code show above then the executor just keeps running and is not able to finish the task.
Also if I put the above code in a function(in sbt console) and then execute the function, it works fine.

@ds-cr Thanks for the report!
Likely this is related to Spark not closing some threads. In the previous versions of sbt, we used to shutdown the thread group forcibly to close loose ends, but that technique became incompatible to JDK 17 so we probably stopped doing that since https://github.com/sbt/sbt/pull/6665.
Allowing console task to fork (https://github.com/sbt/sbt/issues/1918) likely would solve these type of issues.

The issue mentioned (#1918) is still open right?

https://stackoverflow.com/questions/76028146/running-spark-on-sbt-console
https://stackoverflow.com/questions/76047885/does-sparkcontext-have-to-execute-in-main-only


Quarks on Raspberry PI

I've been thinking about picking up a Raspberry Pi 3 specifically to try running Quarks on it.   Has anyone tried running Quarks on one of the earlier versions?  Did you have any issues?
Given the utility of Quarks for edge devices, a Raspberry Quark Pi would seem like a really logical dessert. :)

Yup! I've used Quarks on the Raspberry Pi 2B with success. The Raspberry Pi3 looks like a great piece of hardware. If you try it, let us know how it handles!

I ran the demo shown in this presentation on the older Raspberry Pi B (car OBD-II data, K-Means clustering, connection to Streams through Watson IoT Platform):
https://ibm.box.com/s/j50ktov45r28cyj6lkeu6habzmpwapdp
The only requirement is to have a JVM, and now Raspbian includes a Java 8 vm by default.

A Raspberry Pi Zero would be interesting to try ...

Although I was originally thinking about the new 3 for a more performance demanding application, I agree about the zero for a super low cost edge or embedded device.  Since William already verified that Quarks can work (I'm assuming on Rasbian) now I just need to find some place that has some in stock outside of bundles.

I've heard someone has run it successfully on a Raspberry Pi 3.


First shot at benchmarks

I think it would be good if we had a suite of benchmarks to run. Ideally we would a global benchmark per task (i.e. one for regression, one for binary classification, one for multi-class classification). In each benchmark we want to run a set of estimators against a dataset and compare their scores, running time, memory consumption, etc. I'm not sure if we should compare our models to those of scikit-learn in these benchmarks or we should do this separately.

Done!


Make wrk2 arm compatible

Trying to compile wrk2 on arm, generates following error:
lj_arch.h:279:2: error: #error "No target architecture defined"
 #error "No target architecture defined"

Upgraded LuaJIT to 2.1 to make wrk2 arm compatible.
Also added few minor changes to hdr_histogram.c script.c wrk.c to make it compile

Wanted to followup on this PR to get wrk2 compiled on aarch64

Wanted to followup on this PR. Happy to address any feedbacks/reviews

Wanted to followup on this PR, would love to see wrk2 supporting graviton

I've since moved away from this. I solved my issue by simply creating a docker container and doing things from the inside so I can't really provide any further feedback.
Best of luck 🙏

Hi,
who can review and merge this patch?
With this patch I can build and use wrk2 on arm64.
Thanks!

thank you

found this guy https://github.com/AppsFlyer/wrk3

any hope to get this reviewed and merged?

For those, who would like to run wrk2 on Apple M1, please refer to https://github.com/outsinre/homebrew-wrk2.

Team, Can we please merge this PR to make wrk2 compatible with arm64.

Checking again... Can this patch be merged? Thanks.

I switched to oha - https://github.com/hatoo/oha
recommended


Add Pibulsongkram Rajabhat university

the university official website URL:  https://reg.psru.ac.th/reg2018/
an URL of a page on the official website where the university offers an Computer Science related long-term (>1 year) course: http://cs.psru.ac.th/
a URL of a page or some other proof : https://email.psru.ac.th/search.php

@Loga-Thrim I am sorry, I have to close the request without merging, because you made a mistake in a request and requested to merge too many files, most of them do not relate to the organization which you would like to add to this repository. Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!


6586 postnews api an authorized user can likes its own comments and replies

GreenCity PR
Issue Link 📋
6586
Changed
added a check if current user is an owner of a comment. if so, comment will not be liked
added a check if current user is an owner of a comment. if so, comment will not be disliked
Code is up-to-date with the develop branch.
You've successfully built and run the tests locally.
JIRA/ Github Issue number & title in PR title (ISSUE-6586: Ticket title)
This template filled (above this section).
Sonar's report does not contain bugs, vulnerabilities, security issues, code smells ar duplication
NEED_REVIEW and READY_FOR_REVIEW labels are added.
All files reviewed before sending to reviewers

Fix issue


Test against SqlClient/SqlServer running on Linux

On either EF C.I. or ASP.NET C.I.

A new configuration EFCore -> EntityFramework (Ubuntu) has been added in efci. We have our first successful run already 😄

Should we change milestone? :trollface:


Android Exception

Hey 👋,
I can't run the app when I add super_context_menu 0.1.0 to my pubspec.yaml
Compiler says:
FAILURE: Build failed with an exception.
* Where:
Script 'C:\Users\emrec\AppData\Local\Pub\Cache\hosted\pub.dev\super_native_extensions-0.4.0\cargokit\gradle\plugin.gradle' line: 145

* What went wrong:
A problem occurred configuring project ':super_native_extensions'.
> Failed to notify project evaluation listener.
3
   > NDK is not installed

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 3s
Exception: Gradle task assembleDebug failed with exit code 1

Any ideas?

https://developer.android.com/studio/projects/install-ndk


Markup cannot be imported from Flask since Flask 3.0.0

There is a problem with importing Markup from Flask 3.0.0, thus flask-mdeditor no longer works.
from init.py
from flask import current_app, Markup, request, Blueprint, url_for, render_template_string, render_template, request

Markup has been removed from Flask.
The workaround, for now seems to downgrade Flask to its 2.3.3 version.

I noticed that too.  I was looking for a GKEditor alternative.

Looking at the init.py file you just need to remove Markup from the flask import line, then "pip install MarkupSafe", and add "from markupsafe import Markup".  Works fine after that.

@redredkrovy, thank you for your interest! Currently, it seems that the package is not compatible with Flask 3.0. I greatly welcome you to submit a PR to implement this functionality. In the meantime, I will also try to work on implementing this feature in the coming days.


[BUG] Proxy should be shutdown when user choose Puma as background server and it's not installed

Describe the bug
As described in the title.
To Reproduce
without Puma installed

$ bundle exec toycol server -u puma

then LoadError occurs and error message will display, but proxy server will be still running

$ bundle exec toycol client "REQUEST MESSAGE" in other terminal

then Errno::ECONNREFUSED Connection refused - connect(2) occurs

Expected behavior
Proxy server should be shutdown when user choose Puma as background server and it's not installed.

Resolved by https://github.com/shioimm/toycol/pull/19


Question: boilerplate and predictability

Hi, after a few months of working with redux, I got an idea that I would like to share.
I built a small function which produces an object containing action types and action creators, where the action creators names are the action types in camelCase, and the action arguments are defined as array of keys.
A simple example:
export const counter = createActions({
  COUNTER_INCREMENT: [],
  COUNTER_DECREMENT: [],
  COUNTER_ADD      : ["value"]
});

now the content of counter is the following object
{

  actionTypes: {

    COUNTER_INCREMENT : "COUNTER_INCREMENT",
    COUNTER_DECREMENT : "COUNTER_DECREMENT",
    COUNTER_ADD       : "COUNTER_ADD"

  },

  actions    : {

    counterIncrement: () => ({
      type: 'COUNTER_INCREMENT'
    }),

    counterDecrement: () => ({
      type: 'COUNTER_DECREMENT'
    }),

    counterAdd      : (value) => ({
      type : 'COUNTER_ADD',
      value: value
    })

  }
}

so i can use it in a reducer:
case counter.actionTypes.COUNTER_INCREMENT:
  ...
case: counter.actionTypes.COUNTER_DECREMENT:
  ...
case: counter.actionTypes.COUNTER_ADD:
  ...

and in a component
dispatch( counter.actions.counterIncrement() );
dispatch( counter.actions.counterDecrement() );
dispatch( counter.actions.counterAdd(5) );

This approach drastically reduces the redux boilerplate,
but the question is, it also breaks the predictable definition of redux?

Nope, that's fine. There's actually already a package that does this for you: https://github.com/redux-utilities/redux-actions

Or, even better, our official Redux Starter Kit package.


asp.net core 6 app - Access Denied

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
d34fcwfeedlxri
Region
us-east-1
Amplify Hosting feature
Access control
Describe the bug
When I open the app url https://prod.d34fcwfeedlxri.amplifyapp.com, it gives access denied error.
Expected behavior
it should open the app.
Reproduction steps

I uploaded a zip file containing build output of asp.net core 6 site.
Deployment succeeded.
Got this url  - https://prod.d34fcwfeedlxri.amplifyapp.com/
When I go to this irl, it gives me access denied error

AccessDenied
Access Denied
M0CF5RXCMMNDJFK8
MVAVlpF6umLRgKepze1oY313yHucX7IlXkwHWrzOc8b2VhI8OQNq0YE04/el0KHWu3cJ7twyOBc=

Build Settings
No response
Additional information
No response

Hi @paresh-bhatewra 👋🏽 it looks like you removed your app so I'm not able to further investigate. Did you navigate into the build output folder and zip the contents? If you just zip the folder itself you can run into this error. Please see our Manual Deploy documentation for more details on this error: https://docs.aws.amazon.com/amplify/latest/userguide/manual-deploys.html

Hey,
I had gone through that article before raising the ticket, and I confirm that I created the zip in correct way as mentioned. I have again deployed the same zip with same error.
https://prod.d2ethzgr24y9e0.amplifyapp.com//
Let me know what I am doing wrong.

@hloriana - any update?

@paresh-bhatewra 👋 , thanks for reaching out. The 404 error can occur if the index.html file is not present or is nested under a directory in the zip file. To get started with deploying .NET apps on Amplify Hosting, refer to this blog post: https://aws.amazon.com/blogs/devops/deploy-net-blazor-webassembly-application-to-aws-amplify/.


WIP: Remove svd2swift ir

SVD2Swift previously leveraged a simplified IR to avoid directly
operating on SVD types. This allowed it to perform mutations to the
underlying data structure when converting from SVD types to IR types.
The new version of SVD2Swift directly operates on SVD types so that a
common core set of API can be used by SVD2LLDB and SVDExplorer instead
of needing an IR.
This commit does the following things to achieve the goal of removing
the IR.

SVD2Swift export functions are moved from IR types to SVD types.
An inflate function is added to handle derived-from relationships.
Common Swift extensions in SVD2Swift are moved to MMIOUtilities.

The end result is a simpler implementation of SVD2Swift and fixes for
SVD2LLDB to walk through derived-from relationships.
A future improvement should be made so that inflation is lazy and
request-based so the various tools do not need to inflate the entire
tree if the user only interacts with a subset of the tree.

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test


Simplify engine (breaks things)

This is the first change in many to simplify the use of the shade engine.
This change probably breaks things.

LGTM, made only one comment.


Export a model CODETR to onnx

Hello ,
I am trying to export Co-DETR/projects/configs/co_dino/co_dino_5scale_swin_large_16e_o365tococo.py to onnx using the file tools/deployment/pytorch2onnx.py, but when I run the file it tells me that KeyError: 'CoDETR is not in the models registry', does the project does not support the export for this specific model or there is a solution to this?
Sincerely,
Marouane

I have the same issue

Thank you, I have done the import as you did, but I also found the same error AttributeError: 'CoDINOHead' object has no attribute 'input_proj' . I am trying to use MMdeploy as @TempleX98 suggested in https://github.com/Sense-X/Co-DETR/issues/26#issuecomment-1707752118

@MarouaneMja
Did you success？
I have the same needs to deploy co-detr to any  type, ts, onnx or openvino.

I actually used mmdeploy to convert the model to onnx, I did not succeed using this project

@MarouaneMja
Thank you so much for your answer.
Why you did not succeed?

I actually used mmdeploy to convert the model to onnx, I did not succeed using this project

Did you success?

No, I did not success.

Has anyone overcome this problem?


C ABI: passing a struct after numerous other args works wrong

The following function definition generates wrong calling code in x86_64 ABI:
lib Lib
  struct Vec
    x : Int64
    y : Int64
  end

  fun foo(Int64, Int64, Int64, Int64, Int64, Vec)
end

Lib.foo(1, 2, 3, 4, 5, Lib::Vec.new(x: 6, y: 7))

The C code sees this data:
void foo(long a, long b, long c, long d, long e, Vec v) {
    printf("%ld %ld %ld %ld %ld %ld %ld\n", a, b, c, d, e, v.x, v.y);
    // 1 2 3 4 5 7<PHONE_NUMBER>55776
}

The last two should be 6 7, not 7 <pointer to v>. And that's just the basic case, I'm sure you could get it to crash instead.

Compare the calling code generated by Crystal:
call void @foo(i64 1, i64 2, i64 3, i64 4, i64 5, { i64, i64 } %9)

to an analogous call generated from C by Clang (thanks, @jhass):
%struct.Vec = type { i64, i64 }
[...]
call void @foo(i64 1, i64 2, i64 3, i64 4, i64 5, %struct.Vec* byval(%struct.Vec) align 8 %1)

As I pointed out that this starts happening only beyond a certain number of arguments, @jhass found this very likely lead – that there can be "not enough registers".

Indeed, if we remove one of the integer args, the behaviors converge (without any change on Crystal's part):
call void @foo(i64 1, i64 2, i64 3, i64 4, { i64, i64 } %9)

vs Clang:
call void @foo(i64 1, i64 2, i64 3, i64 4, i64 %5, i64 %7)

This leads us to the conclusion that Crystal probably just doesn't have any code handling the "overflow" arguments, but it should...

After I ported the code from Rust, Rust kept changing and tweaking things. We need to update the code to reflect the latest Rust (or any other) language. If anyone is up for a challenge, then please do so!

There's another issue I already reported where I mention the ABI isn't working well in some cases, so this is almost a duplicate.

I found this issue while fixing something for Win64 ABI (#9520). I added a spec for the previously failing case, and it turned out that it fails on x86_64 too! Presumably for completely different reasons.
So if someone wants to try fixing this, they can enable that spec and see.
Also some pointers in chat.

@asterite I see, that would probably be https://github.com/crystal-lang/crystal/issues/4890.
Well, at least I think my report is better isolated and with some further digging on the root cause.

An interesting tidbit is how cgo deals with this: https://github.com/golang/go/blob/master/src/runtime/cgocall.go#L5

@jhass tl;dr? :-)
Or what's the relevant part?

No relevant part  and no TL;DR, just interesting  that Google's solution to the whole C ABI problemm is "yeah nope, let's avoid re-implementing that and reuse  the C compiler's implementation if possible in any way". Also interesting read on how they did so.

I'm just going completely crazy over how complicated this is.
These two are impenetrable by now:

Rust https://github.com/rust-lang/rust/tree/master/compiler/rustc_target/src/abi/call

Clang https://github.com/llvm/llvm-project/blob/main/clang/lib/CodeGen/TargetInfo.cpp

A library that wants to generalize this but is incomplete:
https://github.com/scrossuk/llvm-abi

A discussion to move this ABI code out of Clang, but can't find recent progress:
https://lists.llvm.org/pipermail/llvm-dev/2020-June/thread.html#142055

Swift seems to straight up include Clang's code:
https://github.com/apple/swift/search?q=targetinfo

Python seems to be using some ancient C library and it's working fine?
https://github.com/python/cpython/blob/63298930fb531ba2bb4f23bc3b915dbf1e17e9e1/Modules/_ctypes/libffi_osx/x86/x86-ffi64.c#L35
Could be interesting but there's no LLVM here so you'd need to bridge the gap.

Go was mentioned. That looks to have big runtime overhead.

Nim obviously just compiles to C.

But, there's a new kid on the block!
https://github.com/odin-lang/Odin/blob/master/src/llvm_abi.cpp
The only other example that one could hope to understand.
No idea how it's being written. There are similarities to the older Rust code, but I found at least some parts of it that definitely never were in Rust.

Overall, that Rust code may still be the best bet, but I just can't do it. We need someone familiar with Rust and some LLVM.
For short-term gains, the Odin sample can be useful, but from what I see it's not much farther along than Crystal.

Oh, another lead:

Julia
https://github.com/JuliaLang/julia/blob/master/src/abi_x86_64.cpp
sadly seems to be intermixed with details about Julia language

There's a copyright pointing towards being based on "LDC – the LLVM D compiler" but I couldn't find the trace

I think the Rust code is probably the safest bet, as you mention. The code we have right now was ported from Rust by me so I can read it. However, I don't have time.

I'd like to note that libffi is definitely not a new library, but it's far from unmaintained and is used by quite a few language FFI implementations. In other words, it will still be around for quite a while and shouldn't have major issues.


Coverage broken: codeclimate-batch issue

See https://github.com/grosser/codeclimate_batch/issues/7

Fixed by https://github.com/thredded/db_text_search/commit/057bba6615b190cd0396e03188e362c6422174f0


missing data

Hi,
Thanks for your great work!
When I was trying your work, I noticed that the download link of the 12-view images from MVCNN is not valid anymore, is there any other way to obtain these images?
Cheers,
Yucheng

Hi, thanks for your interest in our work and sorry for the late reply.
I've uploaded the dataset to Tsinghua Cloud and updated the download link. Thanks for pointing out the invalidity of the previous link.


Add Collection: Cowrie Honeypot

This collection adds the ability to discover Code Execution Attempts within the Cowrie Honeypot.

Hey thank you for your PR
So the first issue I see is we already have a parser for cowrie connection logs https://github.com/crowdsecurity/hub/blob/4ddce084e6bc313033cbfb2753d38f4569a0ced5/parsers/s01-parse/crowdsecurity/cowrie-logs.yaml#L4
And it uses the same key I would suggest merging these into the same parser file and crowdsecurity one has existed longer so that would be the target


Golang 1.4 support

Any chance this could be updated to work with golang 1.4? Currently having issues resulting from the URL.EscapedPath() function not existing.

The addition of EscapedPath fixes some issues that s3gof3r encountered with the url package in previous versions of go so I would prefer not to remove it. See https://golang.org/pkg/net/url/#URL
I would accept a pull request that works around this issue, with tests demonstrating that url paths are escaped correctly.

Closing this as I'm not planning to support go versions earlier than 1.5. It should be possible to deploy a static binary in most cases so the go compiler version is not a production dependency.


redirect_logging disabled?

https://github.com/DenizenScript/Denizen-For-Bukkit/edit/master/plugin/src/main/java/net/aufdemrand/denizen/utilities/debugging/LogInterceptor.java

Yes.
As I said on Discord, use - adjust system redirect_logging instead of the server one, due to the 'server' one being non-functional

config: https://one.denizenscript.com/haste/51120
script: https://one.denizenscript.com/haste/51122
Nothing happens...

Please record debug (/denizen debug -r, run test script, /denizen submit) and post the link it gives you.

https://one.denizenscript.com/paste/51125

It's just the
- adjust system redirect_logging:true

one, not the others.
The world event is just on console output:
I tested it locally, it works fine.
If anything, it's more likely you're having trouble with the announce command (it uses the Bukkit broadcast system, which has a required permission key)


Class names with dots are not accepted

Environment

name
version

IDEA version
IU2020.1.3

Luanalysis version
1.0.1-IDEA201

OS
Windows 10

What are the steps to reproduce this issue?
Define two classes with dots:
--- @class a.x

--- @class a.y

What happens?
I get Duplicate type found in the file: ... error, i.e. both classes are treated as one
What were you expecting to happen?
Classes are independent
Any other comments?
Works in EmmyLua

Oops, I'm blind... there already was #5


The core Pulse.Lib.Core.vprop is now an affine H2.heap -> prop, where vprop : Type u#3
(We also have an invariant store in PulseCore.Memory, storing slprop as invariants, this is mostly unchanged, except for a universe level increase, and some other cleanup, notably, directly using PulseCore.MonotonicStateMonad in a universe-polymorphic way, rather than using FStar.MSTTotal, which fixes the state to be in universe 1.)
We also have two functions

down : vprop -> small_slprop and
up: small_slprop -> vprop, where small_slprop : Type u#2
And a predicate is_small (p:vprop) = up (down p) == p
That is, vprop validating is_small can be coerced to small_slprops without loss of information, and then coerced back.

Most familiar predicates are small, e.g, emp, pure, pts_to, ghost_pts_to etc.
The connectives ** and exists*  form small vprops if their arguments are also small.
But, we now also have Pulse.Lib.BigReference.pts_to (#a:Type u#2) (x:ref a) (#p:perm) (v:a) : vprop that allows storing terms in universe 2 in the heap, including, notably small_slprop.
An example PulseExample.UseBigRef shows how it works, storing list of closures packed with small vprops denoting their pre and postconditions.
Note, we also have Pulse.Lib.BigGhostReference, for a ghost variant of a reference to store a universe 2 term.
I plan to add one more level to the heap, this one to turn any slprop into a resourceful invariant---so, expect one more universe bump.

Note, this commit gives a sense of the changes incurred in the Pulse checker and libraries for a universe bump:
https://github.com/FStarLang/pulse/pull/19/commits/e8c7e6dd03aa9345ea2aec8c629c34dc557a74c3
It's relatively small. If F* had a way to declare top-level universe constants, e.g., let universe mem_level = u#3, then we could possibly isolate the edit to just changing this constant.


Clear .logs and .evfevent directory instead of deleting

@edmundreinhardt I am unable to consistently reproduce the errors we were seeing even after following the same steps shared by Jesse. Not sure yet if this change will resolve the issue.

@ThePrez Once we get this change in and create a release, let me know if you see this bug again. Hopefully this change fixed it as the bug is not consistently reproducible on my end.


Editor auto go back to the top

When I type in the bottom of .mmd file, it will auto go back to the top of the interface.
I wonder it's a bug or the own problem of mine.

Hi @yzy1996, I couldn't reproduce this exactly although I did find a similar problem where the editing side of the preview scrolled up partially when editing the bottom. This is happening because the preview is shorter than the content and it tries to keep the side by side. Thanks for reporting this, we'll look into possible solutions.
When you were editing the bottom of the file would the editor or preview side go back to to the very top?

@yzy1996 Do you have long lines in your .mmd file?
When using very long strings vscode does not sync edit and preview scrolling.
The issue in vscode
https://github.com/microsoft/vscode/issues/119578

@yzy1996 as a temporary fix I found turning off the following VS code setting fixes the problem while still keeping the preview scrolled to the editor:

You can disable it from the vscode settings page: https://code.visualstudio.com/docs/getstarted/settings

Thanks for the quick response! I really love this project. Thank you guys again!
@montanaflynn This scroll setting does fix my problem.  This exactly happens when I type many multi-line block mode mathematics, so it wants to sync the editor scrolled to the preview.
That's a couple of settings, you can also turn off 'Scroll Preview With Editor'.
@OlgaRedozubova Thanks but I did not have long lines.


line continuation

replaced line-continuation backslashes
please see commit messages for details
since this was a bit of a stupefying task, careful review would be appreciated

I carefully read the diff.  All tests pass after applying the patch.  We don't have a doc example tester, but in reading each diff hunk, it appears correct.  Overall, this pull request looks great to me!

After discussing this with @mitsuhiko et al. on IRC,
"""
http://docs.python.org/howto/doanddont.html#using-backslash-to-continue-statements
"""
"""
[trailing whitespace] raises a SyntaxError with an obvious explanation
"""
"""
so far everything i wrote is very consistently using backslash escaping in werkzeug, jinja and flask as well as all extensions
"""
"""
if i would write everything from scratch i might consider changing that style. but now i value consistency more
"""


chore: update ZooKeeper up alerting

Fixes https://github.com/canonical/zookeeper-operator/issues/164

I was expecting this PR to implement the three alerts here, possibly including the suggstion from @nishant  about the level (for which I don't have much opinions against having it as "CRITICAL" if ManSol prefers so)

Please @ the full username, I have no affiliation w this repo

I was syncing with @marcoppenheimer that has been discussing with @acsgn and my understanding is that the current PR should already fix the concerns or issue raised by ManSol.
@sbparke Could you confirm that this and this address your comments and that we can move forward with the merge, and finalize the release?

LGTM, thanks for the update!


Not working with majorgeeks.com since yesterday

Not working with majorgeeks.com since yesterday.
Thanks A LOT for this extension!!

@Adriano0312 sorry to hear you are having trouble with this extension.
Can you be more specific about what's not working?

I'm not sure what part of their site is blocking paste. Their login & signup forms for their forums do not, nor do the comment forms on the rest of the site. If you can be more specific, I can look into further.


Invalid user token even when correct user token is passed!

Describe the bug
I'm working on a dataset which comprises other datasets on the hub.
URL: https://huggingface.co/datasets/open-asr-leaderboard/datasets-test-only
Note: Some of the sub-datasets in this metadataset require explicit access.
All the other datasets work fine, except, common_voice.
Steps to reproduce the bug
https://github.com/Vaibhavs10/scratchpad/blob/main/cv_datasets_bug_repro.ipynb
Expected behavior
It should work if the provided access token is valid (as it does for all the other datasets)
Environment info
datasets version -> 2.14.4

This is because download_config.use_auth_token is deprecated - you should use download_config.token instead


Wrong widths of events in some cases

Bug
What's the current behavior?
If events are:
[
  {
    title: "Event 1 title",
    start: new Date(2019, 0, 0, 8, 0),
    end: new Date(2019, 0, 0, 8, 25)
  },
  {
    title: "Event 2 title",
    start: new Date(2019, 0, 0, 8, 25),
    end: new Date(2019, 0, 0, 8, 55)
  }
]

I see wrong widths of events:

Example:
https://codesandbox.io/s/0pw408l7rl
react 16.5.2
react-dom 16.5.2
moment 2.22.1
react-big-calendar 0.20.3
What's the expected behavior?
Events should have 100% width of the relevant cell.
If events are:
[
  {
    title: "Event 1 title",
    start: new Date(2019, 0, 0, 8, 0),
    end: new Date(2019, 0, 0, 8, 30)
  },
  {
    title: "Event 2 title",
    start: new Date(2019, 0, 0, 8, 30),
    end: new Date(2019, 0, 0, 9, 0)
  }
]

everything works fine:

@arecvlohe do you know if this is still an issue, I think this has something to do with how events are chunked up when laid out? I thought it was fixed tho

@jquense I just checked out the code sandbox and updated the rbc dep to the latest and it was still laying out incorrectly. That's weird. I adjusted the times slightly and it still doesn't go away.

I have this same issue. Events that are 2 minutes apart are considered to be on the same line which causes 'incorrect' rendering.
example:

I've gotten around it by overriding the left and width properties of .rbc-event but it's not an ideal solution.


Don't sort directory fingerprint tuples

The code seems to run fine without sorting, though there doesn't seem to be much difference in the benchmarked times.
Closes #19.

strange, if anything it might have only made it slower... worth profiling using py-spy or adding some log messages to see where time is actually spent -- the goal is to have it spent on IO with as little of CPU overhead as possible.

Codecov Report

Merging #20 (2e3a78c) into master (14f4263) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #20   +/-   ##
=======================================
  Coverage   91.20%   91.20%
=======================================
  Files           3        3
  Lines         341      341
  Branches       30       30
=======================================
  Hits          311      311
  Misses         17       17
  Partials       13       13

Impacted Files
Coverage Δ

src/fscacher/cache.py
90.98% <100.00%> (ø)

Continue to review full report at Codecov.

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

@yarikoptic I can generate graphs using py-spy, but I have no idea what I'm looking at or what's to blame for performance.  Should I just upload the output somewhere so that you can look at it?

This PR is moot as of #71.


fontbakery-fix-vertical-metrics.py crash

$ git clone git@github.com:davelab6/bootstrap-winjs.git ;
$ cd bootstrap-winjs/src/fonts ;
$ fontbakery-fix-vertical-metrics.py *ttf ;
... ok :) ...
$ fontbakery-fix-vertical-metrics.py --autofix *ttf ;
Traceback (most recent call last):
  File "/usr/local/bin/fontbakery-fix-vertical-metrics.py", line 5, in <module>
    pkg_resources.run_script('fontbakery==0.0.10', 'fontbakery-fix-vertical-metrics.py')
  File "build/bdist.macosx-10.9-x86_64/egg/pkg_resources/__init__.py", line 723, in run_script

  File "build/bdist.macosx-10.9-x86_64/egg/pkg_resources/__init__.py", line 1636, in run_script

  File "/usr/local/lib/python2.7/site-packages/fontbakery-0.0.10-py2.7.egg/EGG-INFO/scripts/fontbakery-fix-vertical-metrics.py", line 128, in <module>
    fixer.apply(ymin, ymax)
  File "/usr/local/lib/python2.7/site-packages/fontbakery-0.0.10-py2.7.egg/bakery_cli/fixers.py", line 72, in apply
    self.save()
  File "/usr/local/lib/python2.7/site-packages/fontbakery-0.0.10-py2.7.egg/bakery_cli/fixers.py", line 62, in save
    self.font.save(self.fixfont_path)
  File "build/bdist.macosx-10.9-intel/egg/fontTools/ttLib/__init__.py", line 214, in save

  File "build/bdist.macosx-10.9-intel/egg/fontTools/ttLib/__init__.py", line 629, in _writeTable

  File "build/bdist.macosx-10.9-intel/egg/fontTools/ttLib/__init__.py", line 642, in getTableData

  File "build/bdist.macosx-10.9-intel/egg/fontTools/ttLib/tables/_c_m_a_p.py", line 66, in compile
  File "build/bdist.macosx-10.9-intel/egg/fontTools/ttLib/tables/_c_m_a_p.py", line 865, in compile
IndexError: list index out of range
$

Not sure that this is fontbakery issue, but may be I am using fontTools wrong.
#!/usr/bin/env python
# coding: utf-8
from fontTools.ttLib import TTFont

def main():
	ttfont = TTFont('winjs-symbols.ttf')
	ttfont['hhea'].ascent = 1178
	# ttfont['OS/2'].usWinAscent = int(1178)
	# ttfont['OS/2'].sTypoAscender = int(1178)
	ttfont.save('1.ttf')

main()

If you run this script with font then you will get no errors but
#!/usr/bin/env python
# coding: utf-8
from fontTools.ttLib import TTFont

def main():
	ttfont = TTFont('winjs-symbols.ttf')
	ttfont['hhea'].ascent = 1178
	ttfont['OS/2'].usWinAscent = int(1178)
	# ttfont['OS/2'].sTypoAscender = int(1178)
	ttfont.save('1.ttf')

main()

This code is raising exception. May be someone know what's wrong with it? @behdad @davelab6

What exception do you get exactly? I don't see anything wrong with the latter snippet (apart from int(1178) which is redundant)

yeah, int is reduntant, dont worry with it :)
  File "vmet.py", line 13, in <module>
    main()
  File "vmet.py", line 11, in main
    ttfont.save('1.ttf')
  File "/usr/local/lib/python2.7/dist-packages/FontTools/fontTools/ttLib/__init__.py", line 214, in save
    self._writeTable(tag, writer, done)
  File "/usr/local/lib/python2.7/dist-packages/FontTools/fontTools/ttLib/__init__.py", line 629, in _writeTable
    tabledata = self.getTableData(tag)
  File "/usr/local/lib/python2.7/dist-packages/FontTools/fontTools/ttLib/__init__.py", line 642, in getTableData
    return self.tables[tag].compile(self)
  File "/usr/local/lib/python2.7/dist-packages/FontTools/fontTools/ttLib/tables/_c_m_a_p.py", line 66, in compile
    chunk = table.compile(ttFont)
  File "/usr/local/lib/python2.7/dist-packages/FontTools/fontTools/ttLib/tables/_c_m_a_p.py", line 865, in compile
    firstCode = codes[0]
IndexError: list index out of range

Compiling cmap doesn't have anything to do with vertical metrics...
Did you say the first snippet where the usWinAscent line is commented out doesn't raise the exception?
Does it occur with other fonts as well?

Does not happen with another fonts. Strange, why that error is raised?

Compiling cmap doesn't have anything to do with vertical metrics...

So in this case we have to create those rules in cmap. Is that right, @davelab6 ?

It happens maybe because the cmap subtable format 6 is empty and so it raises index error.
Are you using the latest HEAD from behdad's fontTools?

Are you using the latest HEAD from behdad's fontTools?
yes

How cmap table related to vertical metrics?

the reason the cmap table is being re-compiled after you modify the OS/2 table has nothing to do with vertical metrics. It's because fontTools updates the OS/2 usFirstCharIndex and usLastCharIndex values, and to do that it first needs to decompile the cmap table to get all the character codes.
There seems to be an issue with the way the cmap subtable format 6 is compiled. I'll have a look and let you know.

it should be fixed with https://github.com/behdad/fonttools/commit/e05db5c75f8be0a9364a10e754059ff95527b920
Please let me know if it works now.

@anthrotype :elephant:

@anthrotype it works. @davelab6 please check :)


helloworld-knative to 0.1.5

Promote helloworld-knative to version 0.1.5

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


python3Packages.prox-tv: ignore known MacOS failure

Description of changes
prox-tv seems to intermittently fail on darwin. this issue is known upstream
I'm a bit stuck there, as I don't have access to a darwin system :(
I've at least tested it does not break on linux
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Make sure to run the hooks for the check phase.


Show ignored / no-config in list-services

list-services (and any API/UI that derives from it) ought to show which things

are both running and configured
are running but not configured

and given a flag,

are running or configured but ignored

What does "configured" mean here?

What does "configured" mean here?

In the git repo. #962 effectively resolves this, because it marks services (controllers, now) as read-only if they are in the cluster but not in the repo (and a couple of other reasons).


Block size should be limited

It will probably take some time reach to a level of network activity for this to matter

Also see, https://github.com/bosagora/agora/pull/2416#issuecomment-906494622


Fix Lmu computation

I think there was a mix up in the calculation of the boundedness and derivative's Lmu. I changed it to match the published paper. It does not affect the results since self.mean is set to be 0, but if a different prior was chosen, results might be affected.

Yes, you are correct :) Thanks!


Parkinsons Vision Based Pose Estimation

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title : Parkinsons Vision Based Pose Estimation
:red_circle: Aim : The goal of this project was to use features derived from videos of Parkinson's assessment to predict the severity of parkinsonism and dyskinesia based on clinical rating scales.
:red_circle: Dataset : https://www.kaggle.com/datasets/limi44/parkinsons-visionbased-pose-estimation-dataset
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Email ID :
Participant ID (if applicable):
Approach for this Project :
What is your participant role? (Mention the Open Source program)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Hey, @abhisheks008 I would love to work on this issue,
Please find all the details, mentioned for the issue to be assigned.
Full name : Aditya Goutam
GitHub Profile Link : https://github.com/adityagoutam23
Email ID<EMAIL_ADDRESS>Approaches for this Project :-
1. Using coco pose iterator, with Part Affinity Fields, openCV and torch.
2. Using mass RCNN for human detection then using openCV and other Deep Learning techniques to use given        coordinates in data set for detecting the Parkinson's disease.
What is your participant role: contributor at SSOC'22

Issue assigned to you @adityagoutam23

@abhisheks008 I have started working on the project and currently exploring the dataset and associated features with it as given in the Jupiter file along with the dataset. I am also going through the basics of JSON as it is required for the understanding of dataset. As my exams are starting this week, so I would not be able to work on this project till 24th of September. After that I will resume my current task and finish the project as soon as possible. I kindly request you to please grant me some time for my exams and hereby apologise for the delay in the project submission.

First of all thanks for the update. You can start working on it after Sep 24, but make sure you complete the project within the deadline for this open source program, which is Oct 1, 2022. Any contributions beyond the deadline will not be counted by the SSOC team.
All the best for your exams.
@adityagoutam23

Thank you @abhisheks008, will definitely finish the project by 30th September.

@adityagoutam23 updates please.

@abhisheks008 just finished with exams, will continue the project from today onwards.

Hi! Can I be assigned this? I was thinking of trying out GANs on it.

Full name : Madhuria Rudra
GitHub Profile Link : https://github.com/Ria-R
Email ID<EMAIL_ADDRESS>Participant ID (if applicable):
Approach for this Project : GANs
What is your participant role? (Mention the Open Source program) SWOC 23

Issue assigned to you @Ria-R.
Go ahead!


feat: add tools page

fixes #646

Can you add some screen shots

Can you add some screen shots

Sure, I've kept it simple for now


Debug console doesn't show stderr output

follow steps in https://github.com/Microsoft/vscode/issues/2231
debug the program

You see:

It worked when debugging the second time. Might be a timing issue. One thing to note from my experiences with child processes: when 'exit' is received not all output from a child process is processed. You need to listen on 'close' which is sent when the streams (stdout and stderr) are flushed and read.

If it is the issue that Dirk is suspecting this should be done in the adapter I believe. Forwarding to @weinand

I have seen this as well, in the very first run stderr is not shown, then it worked fine.

@dbaeumer since mono-debug is relying on sdb, I have no access to the streams that sdb is opening to mono or to the mono process itself. However, I've added a small delay in the debug adapter's shutdown code to increase the likelihood that sdb has enough time to read all the data from mono and pass it to the adapter.

Verified that it is still not working. According to Andre not fixable.

Can only be fixed in sdb.


Cannot Install webpack-config-stream from registry

> npm install webpack-config-stream
npm WARN package.json<EMAIL_ADDRESS>crypto is also the name of a node core module.
npm WARN package.json<EMAIL_ADDRESS>fs is also the name of a node core module.
npm WARN package.json<EMAIL_ADDRESS>No description
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm ERR! Error: shasum check failed for /var/folders/wq/6qz3ygjx0ld0p51zxrtqlq8c0000gp/T/npm-85582-49hsdKen/registry.npmjs.org/webpack-config/-/webpack-config-1.1.0.tgz
npm ERR! Expected: ca2f3233d176c4bbe3b11f2f0687341b4c089ade
npm ERR! Actual:   100626b7aa29637ac99cffbf3f1354e6ca562a7e
npm ERR! From:     https://registry.npmjs.org/webpack-config/-/webpack-config-1.1.0.tgz
npm ERR!     at /Users/eudisduran/.nvm/v0.10.38/lib/node_modules/npm/node_modules/sha/index.js:38:8
npm ERR!     at ReadStream.<anonymous> (/Users/eudisduran/.nvm/v0.10.38/lib/node_modules/npm/node_modules/sha/index.js:85:7)
npm ERR!     at ReadStream.emit (events.js:117:20)
npm ERR!     at _stream_readable.js:944:16
npm ERR!     at process._tickCallback (node.js:448:13)
npm ERR! If you need help, you may report this *entire* log,
npm ERR! including the npm and node versions, at:
npm ERR!     <http://github.com/npm/npm/issues>

npm ERR! System Darwin 15.0.0
npm ERR! command "/Users/eudisduran/.nvm/v0.10.38/bin/node" "/Users/eudisduran/.nvm/v0.10.38/bin/npm" "install" "webpack-config-stream"
npm ERR! cwd /Users/eudisduran/github/airtime/boba
npm ERR! node -v v0.10.38
npm ERR! npm -v 1.4.28
npm ERR! not ok code 0

@eudisd I don't have any ideas why, I will try to publish to npm again.

I'm seeing the same issue as well... (on OSX)

Same here.
    "webpack": "^1.12.2",
    "webpack-config": "^1.1.0"

@eudisd @jowcy @adalgleish I'm working on that.


Fresh install error

On php artisan migrate -
  [Doctrine\DBAL\Schema\SchemaException]
  There is no column with name 'url' on table 'notifications'.

Which Version are you using? And have you tired to clear the Composer Packages, the Cache? And which tables, columns are already created? In general it is working :/

closed due to inactivity


Cannot rebuild project after Template 10 update

I had created a project with Template 10 a couple of months ago which was working fine.  Today, after making some changes to my code, when I went ahead to build the project I noticed that Template 10 was updated to the latest version, so now as a result I can no longer build my project.
I think users should have a better control over this to prevent any updates to something that is already working fine and that could potentially break your code.
The rule should be: if it's not broken, don't fix it, or at least, always ask the user before updating.

The rule should be: if it's not broken, don't fix it, or at least, always ask the user before updating.

Wow! that is a bit unkind if not rude. You are getting free service here and put proper arrangements in place to make use of this free service. When you download updates, test your app in isolation before changing your working system. Template 10 is in constant development and it's up to users to adapt to the developmental change, not the other way round. Besides, there is enough info on this website about the updates and what more do you expect? Do you expect @JerryNixon to ask YOU personally before making changes?

It's okay. I checked in some broken bits. My fault.


Cordova Plugin Support instructions incomplete

I've fixed some of the instructions for setting up cordova plugin here:
https://github.com/crosswalk-project/crosswalk-ios/wiki/Cordova-Plugin-Support
However, I still am not able to get the full Cordova plugin setup working. I've read through the code, and it seems that XWalkCordovaExtension.js injects the code from plugins/org.apache.cordova.device/www/device.js as a script in the global scope. Since it is in the global scope, the variables such as  require and module are undefined and thus will throw an error on loading the plugin. That code is meant to be enclosed as such via cordova-cli builder:
cordova.define('cordova/device', function(require, exports, module) {
    // code goes here ...
});

However, I still cannot load the plugin. What is the issue?

@kicktheken Thanks for trying Crosswalk Project for iOS and your feedback, that is truly appreciated! I did miss some important steps in the wiki, I'll update it ASAP with more detailed and precise steps.
For the issue that I can see for now, the additional steps you may need to follow are:

After cloned the crosswalk-ios project, you'll need to run git submodule update --init --recursive to retrieve the submodules which needed by Cordova project;
Create a workspace and import XWalkView.xcodeproj, Cordova.xcodeproj and DeviceDemo.xcodeproj into it;
After imported native and html5 codes from 'cordova-plugin-device', build target 'XWalkView' firstly and 'Cordova' secondly, then drag the generated 'XWalkView.framework' and 'Cordova.framework' into 'General' -> 'Embedded Binaries' in DeviceDemo's configure, then build 'DeviceDemo' target.

We have two Cordova plugin demos in Demos/Cordova/ directory, you may try them to test the Cordova plugin support.
Anything you need to know or issues you've found, feel free to let us know. Thanks very much for your help.

Hi @kicktheken , I've updated the Cordova Plugin Support Wiki with a more detailed process to create a demo, you may check it out. And feel free to let us know if there's any issue in it or any further suggestions. Thanks

Thanks! I'll try it out and let you know.


⚠ Bofrost.lu has degraded performance

In a32710b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1315 ms

Resolved: Bofrost.lu performance has improved in 90ab44d after 8 minutes.


Dispose live kernels in tests after we're done

In our remote tests we start kernels for almost every single test, hence the number of live kernels keeps growing.
& it takes a while to fetch these and build the list of controllers.
Additionally we have a lot of these that just ends up complicating the logs (as these old kernels are no longer required in subsequent tests)

Codecov Report

Merging #11548 (fcc463b) into main (8f80258) will increase coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #11548   +/-   ##
======================================
  Coverage     63%      63%
======================================
  Files        477      477
  Lines      34075    34075
  Branches    5528     5528
======================================
+ Hits       21510    21523   +13
+ Misses     10505    10497    -8
+ Partials    2060     2055    -5

Impacted Files
Coverage Δ

src/platform/common/cancellation.ts
59% <0%> (-6%)
:arrow_down:

src/platform/common/process/baseDaemon.node.ts
50% <0%> (-4%)
:arrow_down:

...rc/notebooks/controllers/controllerRegistration.ts
86% <0%> (-3%)
:arrow_down:

src/platform/common/utils/localize.ts
76% <0%> (+<1%)
:arrow_up:

.../notebooks/controllers/vscodeNotebookController.ts
82% <0%> (+<1%)
:arrow_up:

src/kernels/kernel.ts
83% <0%> (+1%)
:arrow_up:

src/notebooks/notebookCommandListener.ts
54% <0%> (+1%)
:arrow_up:

src/kernels/execution/kernelExecution.ts
72% <0%> (+1%)
:arrow_up:

src/kernels/helpers.ts
68% <0%> (+1%)
:arrow_up:

src/kernels/kernelDependencyService.node.ts
85% <0%> (+1%)
:arrow_up:

... and 2 more


updating sample jax-rs readme file

removing old readme and adding new readme.md of jax-rs for AS 6.0.0

Closing this PR since this file already exists.


feat: Kotlin SDK development principle

First version of Kotlin SDK development principles, outlining future improvements for the ably-java SDK and upcoming JVM SDKs for new products.

Thanks for adding me as a reviewer, will take a look once free 👍


[FEATURE] Page size selector for pagination

Describe how this feature will help us improve
It is very common that users want to change page size on the paginated list.
Describe the solution you'd considered
A concise description of your preferred solution. Things to address include:

building page size select component base on SfSelect
add the possibility/modifier for select to change the direction of the dropdown list (on the top)

@martaradziszewska IMO it should be SfSelect, outside SfPagination component.

@pspaczek you're right, I changed the description :)

Since you requested for this feature, I am referring to you directly :) This component can be done based on SfSelect. You can pass there your page size options and invoke a method on value changed - I think you should have everything that you need to create such a version. This looks exactly like SfSelect so there is no need to create another component. If you need we can prepare some story showing the example of using :) Please let me know


Lack of Type Hints in Code

Summary
There's a lack of type hints in the code which makes the code less readable.
Requirement:
To increase Code Quality , add type hints to the code to make the code more readable and debuggable!
Example :
#/app/auth.py
#python3.9
...
def login(user: User_dict, Authorize: AuthJWT = Depends()):
    hasher = Hasher()
...

here hasher = Hasher() can be changed into hasher:Hasher = Hasher()

Hey there, i would be happy to contribute in this project. can you assign me to this issue?

@donheshanthaka Surely!


filter *.dex files by build type

closing this, same as https://github.com/NativeScript/android-runtime/pull/616