docker-compose build on master dont work, got some python package error

steps to reproduce: ( followed the readme to run the saleor platform locally with docker)
cd saleor-platform
docker-compose build```

Note: the shared folder was added and docker was given 8GB of memory. This is on mac M1, OS: Ventura

Error:

```Collecting measurement==3.2.0
#0 30.17   Downloading measurement-3.2.0.tar.gz (12 kB)
#0 30.18   Preparing metadata (setup.py): started
#0 35.30   Preparing metadata (setup.py): finished with status 'error'
#0 35.30   error: subprocess-exited-with-error
#0 35.30
#0 35.30   × python setup.py egg_info did not run successfully.
#0 35.30   │ exit code: 1
#0 35.30   ╰─> [14 lines of output]
#0 35.30       Traceback (most recent call last):
#0 35.30         File "<string>", line 2, in <module>
#0 35.30         File "<pip-setuptools-caller>", line 34, in <module>
#0 35.30         File "/tmp/pip-install-y1ihcgko/measurement_bfb6e0bbfc5740df8f743367015d14ce/setup.py", line 4, in <module>
#0 35.30           setup(use_scm_version=True)
#0 35.30         File "/usr/local/lib/python3.9/site-packages/setuptools/__init__.py", line 152, in setup
#0 35.30           _install_setup_requires(attrs)
#0 35.30         File "/usr/local/lib/python3.9/site-packages/setuptools/__init__.py", line 147, in _install_setup_requires
#0 35.30           dist.fetch_build_eggs(dist.setup_requires)
#0 35.30         File "/usr/local/lib/python3.9/site-packages/setuptools/dist.py", line 806, in fetch_build_eggs
#0 35.30           resolved_dists = pkg_resources.working_set.resolve(
#0 35.30         File "/usr/local/lib/python3.9/site-packages/pkg_resources/__init__.py", line 777, in resolve
#0 35.30           raise VersionConflict(dist, req).with_context(dependent_req)
#0 35.30       pkg_resources.ContextualVersionConflict: (sphinxcontrib.applehelp 1.0.3 (/tmp/pip-install-y1ihcgko/measurement_bfb6e0bbfc5740df8f743367015d14ce/.eggs/sphinxcontrib.applehelp-1.0.3-py3.9.egg), Requirement.parse('sphinxcontrib-applehelp'), {'sphinx'})
#0 35.30       [end of output]
#0 35.30
#0 35.30   note: This error originates from a subprocess, and is likely not a problem with pip.
#0 35.30 error: metadata-generation-failed
#0 35.30
#0 35.30 × Encountered error while generating package metadata.
#0 35.30 ╰─> See above for output.
#0 35.30
#0 35.30 note: This is an issue with the package mentioned above, not pip.
#0 35.30 hint: See above for details.```

I have the same problem, saleor-dashboard docker build fails with the error mentioned by @jhsenjaliya above

here is a solution for timebeing until the saleor-platform master gets fixed.

checkout latest release tag for both saleor and saleor-dashboard to 3.10.0

git checkout 3.10.0
cd saleor-dashboard
git checkout 3.10.0

follow the "installation with docker guide. thought the last command throw me following error
 docker-compose run --rm api python3 manage.py createsuperuser

Error:
RuntimeError: Model class django.contrib.auth.models.Permission doesn't declare an explicit app_label and isn't in an application in INSTALLED_APPS.

so just pass the --createssuperuser flag while populating the db
docker-compose run --rm api python3 manage.py populatedb --createsuperuser

and then
docker-compose up worked !

The repository now uses official Docker images so the build step is no longer necessary.


Use GitHub markdown for readme

Amazing tool and I have been using Itsycal for maybe a year or so.
One thing that I notice here is that the readme is using the .txt format whereas now GitHub supports .md files which allow for greater markdown capabilities.
I would love to submit a PR and create such a documentation for Itsycal.

Thank you, I appreciate it. I'm going to leave it the way it is for now because the Readme is so simple and doesn't really need markdown.


Arabic subtitles, not found for the most of the Animes

Hello,
Firstly, thank you for this awesome app. The app is amazing on Android TV.
The problem
For most of the anime, there are no Arabic subtitles. I'm not sure how you acquire subtitles from sub providers. Could you please consider adding OpenSubtitles or any other famous provider to the list of providers on the app?
Thank you.

AnimeTV only support subtitle from source server, if there no Arabic subtitle from source it will only translate from English to Arabic.
As you know not every anime have Arabic subtitle from source, or even it only have hard sub stream. since AnimeTV doesn't manage contents, I can't do anything about it. AnimeTV also will never add regional sources.
Also about open subtitle is duplicate with #87
Will not add it.


feature request - numeric database field types

Hi,
Would it be possible to extend the package to support numeric database field types, similar to how the nova select field already supports numeric field types?
I attempted to use the current package and provided numeric options, and discovered the numeric values were converted to strings.
thanks

Can you please be more specific? I just did some tests and whenever I sent the options from the server as numbers, they stayed as numbers even after saving.

Can you please be more specific? I just did some tests and whenever I sent the options from the server as numbers, they stayed as numbers even after saving.

Closing due to inactivity.


Proposal: Merge Starter Packs and Lists

Lists and starter packs offer the same functionality - adding accounts to a collection - for different purposes. However, their features are fragmented even though they are very similar. Lists can be pinned to the home feed while starter packs cannot, people can be easily added to lists from their profile, while they can't be easily added to starter packs, lists are searchable while starter packs are not.
This proposal argues for the merging of the two. Starter packs would be removed in favor of lists, which would now have an option to "Share as a Starter Pack"
Current starter packs would be converted to lists.
The only obstacle to this is the fact that lists can contain feeds. Two options for this:

Get rid of feeds in current starter packs, as they are used in very few starter packs, and of the starter packs they are used on, they are a relatively unimportant addition.
Allow lists to contain feeds

This proposal will simplify an overcomplicated and confusing lineup of user collection features that are needlessly separated and fragmented. Please let me know your feedback.

I am a fan of this plan. Lists feel native and dynamic, and starter packs bolted on and one-time, but the popularity of starter packs has been a great boost for the community.
But I noticed what I think might be an error in the proposal:

The only obstacle to this is the fact that lists can contain feeds

"that lists cannot contain feeds", right?

Lists are feeds though. Or, nobody has managed to articulate any difference between them.

Lists are feeds though. Or, nobody has managed to articulate any difference between them.

No, they're not really that related. A List is a pre-configured list (sorry) of users, which can be used as a "show me these users' posts" list like lists on Twitter did, or as a block list.
A feed is an interface where a service, external to the main parts of Bluesky, can apply whatever logic it wants to include certain posts.  That might be "posts from these users", where it works roughly like a list does (but maybe "these users" changes somehow), or it can be based on some qualities of the posts themselves (like "popular in your network" or "contains the name of a city" or "is getting ratio'd".)
Feeds are more like "pluggable algorithms", but they also require someone to run the software that implements the algorithm on their own server somewhere.
Starter Packs are a lot like lists because they both are a List (again, sorry) of users, but Starter Packs (unlike Lists) can also refer to Feeds. If they were to merge Starter Packs and Lists, then they'd have to do something about the fact that Lists can't refer to Feeds.

OK thanks for the explanation. So are there starter packs that contain different types of Feeds? Every one that I’ve seen, the Feed was just all the posts by those users.

OK thanks for the explanation. So are there starter packs that contain different types of Feeds? Every one that I’ve seen, the Feed was just all the posts by those users.

Yeah, if you go through the process of creating a starter pack here, you can see that you can use it to recommend both Users and Feeds to people. In practice, though, everyone seems to be using them for Users.
(Keep in mind that the "Posts" tab at the top of a starter pack isn't a Feed, it's just a feature of Starter Packs. You can't add that to your Feeds, for instance.)
Interestingly, the announcement blog post listed Feeds first!

See further discussion in #5970


Updated README to include codespaces details

Add feedback link in readme file.

Hi friend,s who r you


If I activate dynamic crosshair the whole UI moves up

same as title, but its just the UI moving, including inventories, the pause menus, and any menu acessible while in a world, exept for replay mod ui elements. all the areas you click on are in the same place, its just the textures that move. also im runnung an enormous amount of mods, but none which modify the UI other than sodium and reeses sodium options. EDIT: it doesnt move up, it moves relative to the crosshair. There is an inaccessible option in the code that turns the mod off if you are in a gui, but it isnt in the menu or the config files. implementing that might fix it?

slight change post some further investigation, the ui doesnt move up, it moves relative to where the crosshair is on your screen

This bug is really annoying. I reproduced it without any issues. The dynamic crosshair is a very nice and cool feature, but because of this bug it's very annoying. I hope it gets fixed sooner.

Here's a video demonstrating the bug. Running it on 1.20.1. I am positive that this is not a compatibility issue with another mod. There's no other mod that could be causing this.
https://github.com/xTracr/RealCamera/assets/85520957/a2e6c744-d0dc-4fbc-83c1-7d889a683f4e

I was able to remove this issue by adding a few more mods to my configuration. It is not really a fix for the bug. This must be an incompatibility with another mod I had. I will make a breakdown soon and update you with the fix I got.
If you can't wait - probably the mod called bedrockify. I am saying probably as because this is only a speculation I have currently. I installed a few mods alongside Real Camera and it magically fixed itself, but yeah, I can't say it's completely fixed, but rather less visible or impactful. I think that the bedrockify mod changes some UI stuff like an animation when opening your UI and that basically removes the weird offset, but you can still see it if you move your camera before you open a UI element, you will see a really thin outline at the border of the screen where it shifted a little then it goes back in place. In most cases you don't notice it or it's not there at all, the bug is still there you just don't see it anymore. (or at least if you don't pay attention to your side view you won't see it). Until I confirm which mod breaks it and that bedrockify helps you a little this should stay as a speculation or a possible community fix. I hope I get down to which mod breaks it, it might take a while though since I use more than 90 mods at a time (and yes all of them are client side only).


Add static type checking to configuration override API implementation.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/29/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1490/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1596/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/161/ for details.

I don't get how this is better than the previous version. That was purely compile time and this compares the global typeid constants at runtime. It doesn't seem better.

I think the implementation is significantly simpler, avoids more advanced template techniques, and eliminates a type. It is also more easily expandable in that the converters can check for any type easily, rather than depending on the set of values in the enum.

I like the elimination of that additional enum type, but I don't know enough about the details in C++11 here to know what the impact are on performance.

Using typeid on built in types and variables of built in types compiles to a single constant load instruction. Comparison is not as efficient, however. That does a pointer check first and if not successful does a name string compare which in our particular case only checks the first character.

@SolidWallOfCode If you fix the merge conflicts now, I'll +1 this against my better judgement :-).

I guess. I don't really think this is all that much better than the template logic, except that it's easier to understand for non template people.

Intel CC build failed! See https://ci.trafficserver.apache.org/job/icc-github/48/ for details.

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/freebsd-github/1615/ for details.

Linux build failed! See https://ci.trafficserver.apache.org/job/linux-github/1510/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/180/ for details.

Gah, what a tarpit. The conflict resolution collided with another override update and that caused the regression test failure. Should be fixed now.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1620/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1515/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/53/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/185/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/RAT-github/7/ for details.


Support configuration to explicitly use javax.ws.rs.core.Response return type

This is needed, for example, when the response code needs to be customized. Resteasy returns HTTP 200 from POST methods by default, but user may want to return HTTP 201 instead.

This feature already exists.  You can set a custom return type on the MediaType object in the OpenAPI definition using the x-codegen-returnType vendor extension.  It might look something like this in the OpenAPI:
    "get": {
        "responses": {
            "200": {
                "content": {
                    "application/json": {
                        "schema": {
                            "$ref": "#/components/schemas/Widget"
                        },
                        "x-codegen-returnType": "jakarta.ws.rs.core.Response"
                    }
                },
                "description": "Successful response - returns a single `Widget`."
            }
        },
        "operationId": "getWidget",
        "summary": "Get a Widget",
        "description": "Gets the details of a single instance of a `Widget`."
    }

This feature already exists.  You can set a custom return type on the MediaType object in the OpenAPI definition using the x-codegen-returnType vendor extension.  It might look something like this in the OpenAPI:
    "get": {
        "responses": {
            "200": {
                "content": {
                    "application/json": {
                        "schema": {
                            "$ref": "#/components/schemas/Widget"
                        },
                        "x-codegen-returnType": "jakarta.ws.rs.core.Response"
                    }
                },
                "description": "Successful response - returns a single `Widget`."
            }
        },
        "operationId": "getWidget",
        "summary": "Get a Widget",
        "description": "Gets the details of a single instance of a `Widget`."
    }


GPR_LIKELY annotations for client_channel

Client channel GPR_LIKELY annotations based on @markdroth 's recommendations in #15044

****************************************************************

libgrpc.so

     VM SIZE                                                             FILE SIZE
 ++++++++++++++ GROWING                                               ++++++++++++++
  +0.4%    +112 src/core/ext/filters/client_channel/client_channel.cc    +112  +0.4%
      +7.4%     +73 pick_callback_start_locked                                +73  +7.4%
      +3.4%     +27 pick_after_resolver_result_done_locked                    +27  +3.4%
      +6.3%     +22 add_retriable_send_initial_metadata_op                    +22  +6.3%
      +2.8%     +16 cc_destroy_call_elem                                      +16  +2.8%
      +1.1%      +9 pick_done                                                  +9  +1.1%
      +2.3%      +8 recv_message_ready                                         +8  +2.3%
      +2.3%      +8 recv_initial_metadata_ready                                +8  +2.3%
      +0.1%      +3 on_complete                                                +3  +0.1%
      +0.2%      +1 [Unmapped]                                                 +1  +0.2%

 -------------- SHRINKING                                             --------------
  -0.0%      -8 [None]                                                   -872  -0.0%

  +0.0%    +104 TOTAL                                                    -760  -0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[microbenchmarks] Performance differences noted:
Benchmark                                                                              cpu_time    real_time
-------------------------------------------------------------------------------------  ----------  -----------
BM_StreamingPingPongMsgs<MinInProcess, NoOpMutator, NoOpMutator>/262144                +7%         +7%
BM_StreamingPingPongWithCoalescingApi<InProcess, NoOpMutator, NoOpMutator>/262144/2/0  +7%         +7%

I'd be curious to know how this impacts our benchmarks.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

src/core/ext/filters/client_channel/client_channel.cc, line 2992 at r1 (raw file):

  }
  // Handle cancellation.
  if (batch->cancel_stream) {

This can use GPR_UNLIKELY().

src/core/ext/filters/client_channel/client_channel.cc, line 3071 at r1 (raw file):

  calld->owning_call = args->call_stack;
  calld->call_combiner = args->call_combiner;
  if (chand->deadline_checking_enabled) {

This can use GPR_LIKELY().

Comments from Reviewable

****************************************************************

libgrpc.so

     VM SIZE                                                             FILE SIZE
 ++++++++++++++ GROWING                                               ++++++++++++++
  +0.0%     +16 [None]                                                +2.95Ki  +0.0%
  +0.3%     +96 src/core/ext/filters/client_channel/client_channel.cc     +96  +0.3%
      +6.8%     +67 pick_callback_start_locked                                +67  +6.8%
      +3.4%     +27 pick_after_resolver_result_done_locked                    +27  +3.4%
      +6.3%     +22 add_retriable_send_initial_metadata_op                    +22  +6.3%
      +1.1%      +9 pick_done                                                  +9  +1.1%
      +2.3%      +8 recv_message_ready                                         +8  +2.3%
      +2.3%      +8 recv_initial_metadata_ready                                +8  +2.3%
      +1.6%      +7 [Unmapped]                                                 +7  +1.6%
      +0.1%      +2 cc_start_transport_stream_op_batch                         +2  +0.1%

  +0.0%    +112 TOTAL                                                 +3.05Ki  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Added the other recommendations too except for one. (Not sure if they should be made though, because this means consciously saying that doing things like a single rpc, cancel_batch and disabling deadlines is meant to be slow.

I've also run benchmark runs on this. I should have numbers in about 3 hours.

[microbenchmarks] No significant performance differences

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

src/core/ext/filters/client_channel/client_channel.cc, line 3106 at r1 (raw file):
Previously, yashykt (Yash Tibrewal) wrote…
Some values can be empty, for example, for non-traced, insecure calls

I think you're right on this one -- I would expect the security context to be non-null in the common case, but the others are probably not used as often.

src/core/ext/filters/client_channel/client_channel.cc, line 2955 at r3 (raw file):

      return;
    }
    if (GPR_LIKELY(!chand->started_resolving)) {

This should be GPR_UNLIKELY().  It will only be true when the channel is idle.

Comments from Reviewable

These were the results from -
On HEAD
+--------+---------+---------+----------------------+----------------------+
|        |  l_50   |  l_99   |  server_qps_per_cpu  |  client_qps_per_cpu  |
+========+=========+=========+======================+======================+
| AVG    | 63.5064 | 126.705 |       41661.9        |       38646.3        |
+--------+---------+---------+----------------------+----------------------+
| STDEV  | 5.52187 | 8.22963 |         2105         |       3188.86        |
+--------+---------+---------+----------------------+----------------------+
| MEDIAN | 64.1037 | 127.629 |        41400         |       38380.4        |
+--------+---------+---------+----------------------+----------------------+

With GPR_LIKELY change
+--------+---------+---------+----------------------+----------------------+
|        |  l_50   |  l_99   |  server_qps_per_cpu  |  client_qps_per_cpu  |
+========+=========+=========+======================+======================+
| AVG    | 62.8346 | 125.397 |       41505.9        |       39647.9        |
+--------+---------+---------+----------------------+----------------------+
| STDEV  | 5.43443 | 8.2653  |        3150.9        |       2884.49        |
+--------+---------+---------+----------------------+----------------------+
| MEDIAN | 62.3697 | 127.039 |       41657.1        |       39726.4        |
+--------+---------+---------+----------------------+----------------------+

****************************************************************

libgrpc.so

     VM SIZE                                                             FILE SIZE
 ++++++++++++++ GROWING                                               ++++++++++++++
  [ = ]       0 [None]                                                +3.07Ki  +0.0%
  +0.3%     +80 src/core/ext/filters/client_channel/client_channel.cc     +80  +0.3%
      +6.8%     +67 pick_callback_start_locked                                +67  +6.8%
      +3.4%     +27 pick_after_resolver_result_done_locked                    +27  +3.4%
      +6.3%     +22 add_retriable_send_initial_metadata_op                    +22  +6.3%
      +1.1%      +9 pick_done                                                  +9  +1.1%
      +2.3%      +8 recv_message_ready                                         +8  +2.3%
      +2.3%      +8 recv_initial_metadata_ready                                +8  +2.3%
      +0.1%      +2 cc_start_transport_stream_op_batch                         +2  +0.1%

  +0.0%     +80 TOTAL                                                 +3.15Ki  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] Performance differences noted:
Benchmark                                                               cpu_time    real_time
----------------------------------------------------------------------  ----------  -----------
BM_StreamingPingPongMsgs<MinInProcess, NoOpMutator, NoOpMutator>/32768  -4%         -4%

Looks like this does yield a noticable improvement.  Thanks for doing this!

Reviewed 1 of 1 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Known issues : #15526, #14673, #14240


Add procps to wasmcloud Docker image

Add procps to the Docker image so that it installs kill as a binary
instead of just a shell builtin.
Partial fix for #344.

Whoops, always forget about that. Done.


Rename project to symfony-docker-edition

I was thinking if we should rename the repository name to symfony-docker-edition. Thoughts?

I'm not sure this is a Symfony Docker Edition, this is just a Symfony app with a Docker configuration.
This configuration can be move to any project, even if is Symfony or not.

While I agree with you, I'd like to point out that this project started with the idea of being Symfony specific, therefore we would tweak every possible service to be fully optimised for the Symfony environment. However, if we would support multiple frameworks, the code base would change significantly, as well as some configuration such as virtual host and so on.


Figure out code coverage

Some features to consider:

Use the emoji with an umbrella ☂ in it
Be reusable by other repos

May require some action variables like send_to_codecov: true/false to allow user to control if the code should be uploaded to the default code coverage website.
Should not fail if deployment to codecov fails due to codecov not existing
Configurable failure percentages

Resolved by #589


Build markup changes

@simonmcmanus This PR addresses your comments on the earlier PR
https://github.com/lnug/lnug.github.io/pull/90#issuecomment-166265479
I've restored the unknown venue. It hadn't struck me that we'd need this placeholder venue again, but you are right.
I've amended the new venue information to match the style of the stackoverflow venue details. The venue is now shown in the same manner as that venue. I've chosen an appropriate zoom level for the map.
I've made an intermediate refactoring of the build process to store the venues in a json data file. It doesn't make use of any parameters or config info, etc, but it's tidier than storing the data with the code, as you say.
@simonmcmanus @maxwellito - with the css styles as they are now, the marker is hidden by the ticket. I experimented with changing padding on the map-holder class for wide & thin maps, which did allow the marker to be seen. However, I was reluctant to change any css styles without knowing more of the context. Is this how we do this? I couldn't see another mechanism for it.
Feel free either to give me some guidance, or add some appropriate changes to this PR to bring the markers out from under the ticket.

Thanks for these tweaks @jkbits1 -
I've pushed a couple of tweaks to the venue structure on this branch.
I think you just need to tweak your lat/long to get the pointer showing in the right place.  Im just having a play now.

Ive updated the lat/long for the pointers and pushed. No reasons why the offsets cant be automatically calculated but I did it manually this time.
The linting is also passing. Not sure why CircleCi is not picking up my commits.

I've also put back in the tickets button.
It currently links to a 404 but I guess we should be able to launch the Jan tickets now.

Your tweaks to the PR look good, and the marker appears in the correct place now.
I probably won't get the tito done for a day or two, as it needs the new venue set up and I like to make sure all that is correct before launching but it doesn't hurt to have the tickets sign on there.
Ok, I see Oliver is definitely scheduled - there's is always a lot of activity in speakers, which is why I kept the changes just to the venue before.


HL-562: Fix session expiration error

See details here: https://helsinkisolutionoffice.atlassian.net/browse/HL-562
This is hard to reproduce and probably concerns only local environment because user has changed to anonymous user after session has expired. The situation happens at least when the flag DISABLE_AUTHENTICATION is 1.
Description :sparkles:
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

This would take into account DISABLE_AUTHENTICATION=True and do the AnonymousUser removal more cleanly using is_authenticated:
diff --git a/backend/benefit/applications/api/v1/serializers.py b/backend/benefit/applications/api/v1/serializers.py
index d4466404..7c2428e4 100644
--- a/backend/benefit/applications/api/v1/serializers.py
+++ b/backend/benefit/applications/api/v1/serializers.py
@@ -95,7 +95,10 @@ class AttachmentField(FileField):

         url_pattern_name = "v1:applicant-application-download-attachment"
         request = self.context.get("request")
-        if request and get_request_user_from_context(self).is_handler():
+        user = get_request_user_from_context(self)
+        if settings.DISABLE_AUTHENTICATION or (
+            user.is_authenticated and user.is_handler()
+        ):
             url_pattern_name = "v1:handler-application-download-attachment"

         path = reverse(

Sorry, my bad, didn't take the user is None case into account.
So user.is_authenticated and user.is_handler() → user and user.is_authenticated and user.is_handler() would take care of that too.

Codecov Report

Merging #1410 (ff6a58b) into develop (ddd8b91) will increase coverage by 40.40%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##           develop    #1410       +/-   ##
============================================
+ Coverage    54.41%   94.82%   +40.40%
============================================
  Files          824      188      -636
  Lines        19927     8233    -11694
  Branches      2755        0     -2755
============================================
- Hits         10844     7807     -3037
+ Misses        8687      426     -8261
+ Partials       396        0      -396

Impacted Files
Coverage Δ

backend/benefit/applications/api/v1/serializers.py
93.60% <100.00%> (ø)

backend/shared/shared/azure_adfs/views.py
40.00% <0.00%> (-44.00%)
:arrow_down:

backend/shared/shared/common/tests/__init__.py
66.66% <0.00%> (-33.34%)
:arrow_down:

backend/shared/shared/common/lang_test_utils.py
80.00% <0.00%> (-20.00%)
:arrow_down:

backend/shared/shared/common/tests/conftest.py
86.95% <0.00%> (-13.05%)
:arrow_down:

backend/shared/shared/audit_log/audit_logging.py
87.23% <0.00%> (-12.77%)
:arrow_down:

backend/shared/shared/oidc/auth.py
81.48% <0.00%> (-5.56%)
:arrow_down:

...end/kesaseteli/applications/tests/test_mock_vtj.py

frontend/tet/admin/.testcaferc.js

frontend/shared/src/components/table/utils.tsx

... and 928 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix some C++11 issues

This is an assortment of bug fixes and improvements for our optional C++11 support.
It's mostly only relevant for those who were compiling by passing the -DCXX11 option manually.
Previously, that was the only way to compile with C++11, and it only affected clang (not gcc).
But in addition to fixing some bugs and warnings, this PR also improves the logic in the CMake
configuration files.  Previously if you had too recent a version of boost ( > v1.64) and the -DCXX11
option was not passed, it would just fail.  With the new behavior, it will auto-enable CXX11 support
if the version of Boost requires it.  It could still fail, if the compiler doesn't support C++11, but this
is no worse than the previous behavior.  If the option is passed manually and the detected Boost is
too old, it will still fail as before... but with a hint to upgrade Boost before trying again.
Also, if boost is not detected, the version it automatically downloads will now depend on whether
-DCXX11 was passed.
This has still only been tested on OSX platform, but the option is now available for other platforms as
well (in case it works).
Critical bug fixes:
- Use bernoulli_distribution from boost::, not stl:: or boost::TR1

The TR1 sub namespace no longer exists in recent versions of boost.
Previously, this was fixed by using the C++11 STL version if it's
available, otherwise falling back on TR1.  But that didn't work
(caused a crash) due to a slight incompatibility between the C++11 and
the boost version.  The boost version still exists, just not under TR1--and
we already use it elsewhere in the codebase, so this should be
the cleanest fix.

- Fix memory-corruption crash in std::domain_error(), by copying formatted
  string to memory allocated safely via postgres with palloc()

Non-critical fixes/improvements:
- Convert ptr_fun() to function() to get rid of C++11 deprecation warning

- src/dbal/Reference_proto.hpp:  Fix an error in the template definition
  of class Ref.  The default value for IsMutable should be the value true
  or false, not the type bool.  The only reason this wasn't generating
  compiler errors is that IsMutable happens to be passed explicitly in
  every place in the code where the template is instantiated--so the
  default was ignored.

- Hide global boost symbols from external libraries
  We were already doing this for all STL symbols--for the same reasons,
  doing it for boost as well should make MADlib more robust when
  integrating with 3rd party libraries.

another comment


Adaptive favicon

This PR makes the favicon white in dark mode.
https://web.dev/articles/building/an-adaptive-favicon

It looks like the changes don't display effectively due to the presence of the .ico icon.

At least in Firefox it doesn't work:

But GitHub favicon changes, so there must be a way to do that...

At least in Firefox it doesn't work

I'm guessing this is because the head also links favicon.ico.
<link href="/favicon.svg" rel="icon" type="image/svg+xml">
<link sizes="16x16 32x32" href="/favicon.ico" rel="icon">

yes, but the svg icon is declared before. In the network panel I see the svg file is downloaded

yes, but the svg icon is declared before. In the network panel I see the svg file is downloaded

That's weird, I'll try removing favicon.ico and what happens.

Works like a charm. Thanks!


gh-850: bugfix/small resources

closes #850
makes sure weaviate runs with 3GB of memory
verified locally and with a n1-standard-1 vm on GCE
used the verification script manually on the machine mentioned above to import 200 vertices over 60 classes and make sure everything still works
updated the docs

Awesome @etiennedi, gonna use this right away.


Add panels for showing caption

What?

Add panel for showing current caption
Add panel for selecting scene

Why?

現在時刻に合わせてキャプションを表示したいから
シーンの選択を簡単に行いたいから

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: WatanabeToshimitsu
The full list of commands accepted by this bot can be found 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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: WatanabeToshimitsu, yusukefs
The full list of commands accepted by this bot can be found 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


When modifying firearms, switching items causes the mouse cursor to pop out

Describe the bug
When modifying firearms, switching items causes the mouse cursor to pop out
Reproducibility

[X] I reproduced this issue with as few other mods as possible installed.
[ ] I am unable to reproduce this issue consistently.

Screenshots and/or videos
No response
Logs
no logs
Debug Logs
No response
Crash Reports
No response
MWC Version
0.1.3
Forge Version
1.12.2 forge <IP_ADDRESS>60
Red Core Version
0.6 dev 3
MixinBooter Version
9.1
Java Version
1.8
Java Virtual Machine
Hotspot
Operating System
windows 11
GPU Vendor
NVIDIA
Code of Conduct

[X] I agree to follow this project's Code of Conduct

https://github.com/Cubed-Development/Modern-Warfare-Cubed/assets/130232431/daea4357-52c2-47c2-8b83-d85d901ebab7

During firearm modification, can it be set so that when switching items, the modification mode is automatically exited? Or can the mouse cursor automatically revert to its normal appearance when switching items?


Add tests and Github Action

This is draft PR that adds some test for the normal index.
The status of the tests is as follows:
Pass:

NumpyIndex - passes, but requires overwrite=True, not sure why
TorchIndex - this is skipped due to lack of GPU on GHA
MemoryIndex - passes

Fail:

FaissHnsw - this returns -1 as the first docid, creating an error. https://github.com/facebookresearch/faiss/issues/244 discusses why this might happen
FaissFlat - this returns no results

For Github actions, its a little disappointing that the Python dependencies setup is longer than the unit tests!

In the process of re-working the tests:

replacing the deprecated indexes with FlexIndex
testing model encoding and scoring directly


[DO NOT COMMIT] Add mobile auth redirect to custom URI scheme (zulip://)

This is just for reference while implementing the correct functionality. I used this locally for testing.

@neerajwahi thanks for your work on this!  I've created a timabbott/mobile-otp branch that cleans this up, adds tests, and requires that the client provide a one-time password.  See #4640.


Causing Exception for three parentheses (I Guess)

[04:10:36|INFO|input.SqlReader:109] Parsing query sessions by file from dir: data/yelp
net.sf.jsqlparser.JSQLParserException
	at net.sf.jsqlparser.parser.CCJSqlParserUtil.parseStatements(CCJSqlParserUtil.java:121)
Caused by: net.sf.jsqlparser.parser.ParseException: Encountered " "(" "( "" at line 1, column 265.
Was expecting one of:
    "SELECT" ...
    "SELECT" ...
    "SELECT" ...
    "SELECT" ...
    "SELECT" ...
    "SELECT" ...
    "SELECT" ...

The SQL statement is:
select n.nspname, c.relname, a.attname, a.atttypid, t.typname, a.attnum, a.attlen, a.atttypmod, a.attnotnull, c.relhasrules, c.relkind, c.oid, pg_get_expr(d.adbin, d.adrelid), case t.typtype when 'd' then t.typbasetype else 0 end, t.typtypmod, c.relhasoids from (((pg_catalog.pg_class c inner join pg_catalog.pg_namespace n on n.oid = c.relnamespace and c.relname = 'business' and n.nspname = 'public') inner join pg_catalog.pg_attribute a on (not a.attisdropped) and a.attnum > 0 and a.attrelid = c.oid) inner join pg_catalog.pg_type t on t.oid = a.atttypid) left outer join pg_attrdef d on a.atthasdef and d.adrelid = a.attrelid and d.adnum = a.attnum order by n.nspname, c.relname, attnum
Using version 0.9.4
This query works fine in postgres.

Regardless of what we choose to do in the short term for QueryPredict, we should definitely file this bug against JSQLParser as well.

i did already.
On 28 Dec 2016 11:01, "Tony Ohmann"<EMAIL_ADDRESS>wrote:

Regardless of what we choose to do in the short term for QueryPredict, we
should definitely file this bug against JSQLParser as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JSQLParser/JSqlParser/issues/375#issuecomment-269497361,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATFGKEs48IbPo-7WHSe_E26jUX7zpKlZks5rMofbgaJpZM4LWJJR
.

The problem was this expression
(not a.attisdropped)
I added this condition and now your SQL is parsed.

Thanks for updating.


Run uncrustify before applying complexity

The GNU application "complexity" analyses source files and gives a complexity score to each of the procedures / functions found. A score of 9 or lower is considered good.
But unfortunately, "complexity" stops parsing a source file as soon as it sees a function that is indented either with spaces or with tabs:
    int test_2( int value )
    {
        if( value > 0 )
        {
            printf( "test_2: value = %d\n", value );
        }
    }

It seems to help if the source code is reformatted to remove indentation. I added a new config file for uncrustify called tools/uncrustify.complexity.cfg, which is used by a script called ".github\workflows\ci.yml".
Test Steps
Add the function here above to a source file and check it with complexity.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

It seems like there is some problematic formatting and we are trying to work-around it by applying formatting before running complexity. Is it possible to fix the formatting so that this work around is not needed?

@aggarg wrote:

It seems like there is some problematic formatting and we are trying to work-around it by applying formatting before running complexity. Is it possible to fix the formatting so that this work around is not needed?

Gaurav, I don't think that the FreeRTOS formatting is problematic. The GNU tool complexity is failing and should be repaired.
complexity --scores --threshold=0 --horrid-threshold=10 timers.c
No procedures were scored

The source is indented with a global #if:
#if ( configUSE_TIMERS == 1 )
    /* Source code */
#endif


Specify the type of configuration file in the kube-proxy and kube-scheduler --config flag help doc

Specify the type of configuration file in the help doc for --config flag to avoid confusion between the various Kubernetes configurations files.
It is easier to figure out which file it is when dealing with individual tools, but it might be confusing when reading about them in the larger K8s context, in which case it helps to explicitly mention the type of the configuration file.

Thanks for the contribution. However, these two files are generated from upstream source code. Please propose upstream fix instead.

Thanks for the contribution. However, these two files are generated from upstream source code. Please propose upstream fix instead.

/close


Questions: Clarifying the use of FP8 for Training

@tocean @wkcn
In line with the investigation in https://github.com/NVIDIA/TransformerEngine/issues/424, it would be great to get the insights from the team at microsoft for using FP8 in aspects of training besides matmul.
Questions
1. Performance
The repo only mention training accuracy and memory savings. However, the kernels may not be very optimized and majority is implemented in Torch. I guess that performance is still unexplored.
2. Weight Update

is the weight update applied while the backward pass is running (on-the-fly)? Or is it applied after the entire backward pass is complete?

Seems that one can get memory savings from on-the-fly
Is there a possibility to use CUDA graphs API to efficiently schedule the weight updates concurrently even if it does not saturate the GPU? Furthermore, one should be able to batch uneven-sized weight update into a single kernel invocation based on an outstanding_weight_updates_bytes threshold.

3. More Accurate Scaling Factors
Is there a way to maintain more accurate amax by estimating:

For e.g. naive SGD case:

scaling_factor_weights_t = amax_weights_t-1 + amax_grad_t - this is an accurate upper bound (no necessity of apriori knowledge)
amax_weights_t = max(abs(weights_t)) - this is only used for the next iteration

For Adam optimizer:

Utilizing e5m2 might be able to help with dynamic range for v (same dynamic range as FP16).
Storing sqrt_v rather than v may help the precision. Update rule: see appendix

Intuition: sqrt will reduce the dynamic range of bits by half (2^16 -> 2^8, 2^-16 -> 2^-8). Hence we perform sqrt in fp32/fp16 and quantize that as fp8, thus preserving the dynamic range

A more rigorous analysis is needed here.

If it is possible to better estimate scaling_factor_weights_t then it may be possible to use more of the dynamic range. Hence, storing the weights as FP8 (rather than FP16 as in the MS-AMP repo) might be possible.

Since Adam optimizer is momentum-based, the effect of deviation of amax on a per-batch basis is more bounded.

4. Adaptive Precision
Has it been explored using lower precision (FP8) at high learning rate (at earlier epochs) and higher precision (e.g. FP32, FP16) at lower learning rate (at later epochs)?
Appendix
Update Rule for sqrt_v_fp8
scaling_factor = amax_sqrt_v_prev / 448. # 2^8 * (1 + 3/4) - use more of fp8e5m2 dynamic range. margin = 7

v_fp32 = pow2(sqrt_v_fp8.to(dtype.fp32) * scaling_factor)
v_new = beta_2 * v_fp32 + (1 - beta_2) * grad_sq
sqrt_v_fp8 = (sqrt(v_new) / scaling_factor).to(dtype.fp8e5m2)

# end of loop
amax_sqrt_v_new = sqrt(max(v_new))

Notes:

If amax_sqrt_v_fp8 = 448.0, then the scaling factor is 1. This is captured in margin bits: https://github.com/Azure/MS-AMP/blob/aed29d6533e8ff86686fdb6fafa6d0b720e9e5f6/msamp/common/tensor/meta.py#L39

Hi @jon-chuang , I am sorry for late reply.
Thanks for your attention to our work!
1. Performance

The repo only mention training accuracy and memory savings. However, the kernels may not be very optimized and majority is implemented in Torch. I guess that performance is still unexplored.

Yes. Our first step is to apply FP8 format as much as possible to reduce memory footprint while maintaining accuracy, and the second step is to optimize the performance in MS-AMP. MS-AMP can be combined with the TransformerEngine to invoke optimized operators in TE. (Related PR: https://github.com/Azure/MS-AMP/pull/98)
2. Weight Update

is the weight update applied while the backward pass is running (on-the-fly)? Or is it applied after the entire backward pass is complete?

a) Seems that one can get memory savings from on-the-fly
b) Is there a possibility to use CUDA graphs API to efficiently schedule the weight updates concurrently even if it does not saturate the GPU? Furthermore, one should be able to batch uneven-sized weight update into a single kernel invocation based on an outstanding_weight_updates_bytes threshold.

a) It is applied after the entire backward pass is complete. The FP8 weights are updated in the optimizer. (https://github.com/Azure/MS-AMP/blob/main/msamp/optim/adamw.py#L193).
b) Good idea. I had tried using an additional CUDA stream for weight update, but it did not achieve the dessired acceleration, probably due to my implementation not being optimal : ) However, I still believe that it is effective to schedule weight updates concurrently, since weight update does not affect the calculation of backpropagation.
It is available to update multiple FP8 weights in a single CUDA kernel, but it is notable that the FP8 tensor with a scaling factor should be treated as a whole. The maximum value of the entire tensor should be computed before quantization a high-precision tensor to a FP8 tensor.
3. More Accurate Scaling Factors

Is there a way to maintain more accurate amax by estimating:

For e.g. naive SGD case:

scaling_factor_weights_t = amax_weights_t-1 + amax_grad_t - this is an accurate upper bound (no necessity of apriori knowledge)
amax_weights_t = max(abs(weights_t)) - this is only used for the next iteration

For Adam optimizer:

Utilizing e5m2 might be able to help with dynamic range for v (same dynamic range as FP16).
Storing sqrt_v rather than v may help the precision. Update rule: see appendix

Intuition: sqrt will reduce the dynamic range of bits by half (2^16 -> 2^8, 2^-16 -> 2^-8). Hence we perform sqrt in fp32/fp16 and quantize that as fp8, thus preserving the dynamic range

A more rigorous analysis is needed here.

If it is possible to better estimate scaling_factor_weights_t then it may be possible to use more of the dynamic range. Hence, storing the weights as FP8 (rather than FP16 as in the MS-AMP repo) might be possible.

Since Adam optimizer is momentum-based, the effect of deviation of amax on a per-batch basis is more bounded.

Accurate amax
In MS-AMP, the update of weights contains two stages. In the first stage, we pre-compute the updated high-precision weight and obtain the amax. In the second stage, we re-compute the updated high-precision weight and quantize it into FP8 with a scaling factor. So it's an accurate amax but it increases the computational overhead. I agree that using amax_weights_t = max(abs(weights_t)) for the next iteration is an effective approach.

Storing the weights as FP8
In MS-AMP, the master weight is FP16 with a scaling factor, and the weight is FP8 with a scaling factor. The choice of FP16 as the master weight is primarily for precision rather than range.

Utilizing sqrt(value) and e5m2
I believe it is a good approach and worth trying.
Sqrt can reduce the dynamic range, but it can not handle the issue of low precision.

4. Adaptive Precision

Has it been explored using lower precision (FP8) at high learning rate (at earlier epochs) and higher precision (e.g. FP32, FP16) at lower learning rate (at later epochs)?

No. This approach requires preserving enough memory in earlier epochs to store high-precision weights in later stages, which may not be as efficient as using high-precision weights and low-bit computations.

Close this issue since there are no activities more than 9 months


Support $foo/bar in command position: parameter expansions with pasted constant strings

Issue sweep: two separate concerns here:

Don't make the command word red when we don't know whether or not it'll execute successfully or not.
Support ${foo}/bar in command position where bar is a constant string.

We have #695 for (1) so I'm repurposing this issue to track (2).  Patches welcome.

Edge case to consider: highlighting $prefix/bin/zsh in command position when the variable $prefix is unset or empty and /bin/zsh exists.
I wonder if we can highlight $prefix as an elision (comment) and the remainder of the word in green.  This would (1) differ from how that would be highlighted if $prefix were non-empty; (2) correctly indicate that the command-line is valid.

sorry if it's the wrong place to ask. my variable $foo $bar are not highlighted, and i see no option to enable/change it in the tweak section. i mean i would like a option if the variable exist ZSH_HIGHLIGHT_STYLES[variable] for example and ZSH_HIGHLIGHT_STYLES[variable-unset] if the variable doesn't exist. it's preferable that i create a new issue ? i didn't find one related. thanks

@mirsella Create a new issue, please, and I'll reply there.

The new issue is #739.  I'll delete these three messages shortly since they're copied on the other issue.

Maybe you can look at https://github.com/zdharma-continuum/fast-syntax-highlighting for reference? It seems that the plugin handles this case pretty well.


Add ignore files feature

I don't want to stylize build_runner generated files, because when they are generated again they are unnecessarily counted as modified files by git.
Would you consider adding a flag or config file to list files and globs to ignore?

Would you consider adding a flag or config file to list files and globs to ignore?

That sounds like a nice feature to add. Do you have a preference or a favorite glob style or favorite command you would like to emulate? I ask because there are an unlimited number of ways that this could be implemented, and if you are familiar with something it might be best to start there.

I would say that using the same syntax used in analysis_options.yaml would be good because flutter developers would be already accustomed to it.
This is what they use: https://pub.dev/packages/glob#syntax
Maybe creating something like flutter-stylezer.yaml would be even better, with something like this inside:
exclude:
  - "**/*.g.dart"
  - "**/*.gr.dart"
  - "**/*.freezed.dart"

@Maatteogekko - please try out the v0.1.4 release and see if this implements what you asked for:
https://github.com/gmlewis/go-flutter-stylizer/releases/tag/v0.1.4

Works like a charm! 💯
Thank you for the nice addition in so little time. Keep up the great work! 💪🏻


Convert ValuesController return types to ActionResult

Fixes https://github.com/aspnet/templating/issues/339

Adding a few reviewers who might be interested: @jbagga @DamianEdwards @rynowak @pranavkm .

@jbagga - putting on your plate for final review + merge.

@cartermp - what would be the syntax for this, if any, in F#? Here's the F# ValuesController:
https://github.com/aspnet/templating/blob/dev/src/Microsoft.DotNet.Web.ProjectTemplates/content/WebApi-FSharp/Controllers/ValuesController.fs#L14-L20

Was this change made more as a consistency thing, or is there a different reason?
I don't think there should be any change required for F# here.
Reason: The return types inferred by the compiler will still be string [] and string, but that gets correctly dealt with by ASP.NET anyways. Working directly with ActionResult<'T> in F# is a bit cumbersome because we don't do implicit conversions like C# does. That means that something like this:
public async Task<ActionResult> Register(RegisterViewModel model)
{
    if (someCondition)
    {
        return RedirectToAction(foo, "bar", "wheee");
    }

    return SomethingElse();
}

Is a bit more annoying to encode because you must ensure that SomethingElse() either have the same return type as RedirectoToAction or cast to the appropriate base type or interface. F# folks are also far less likely to build fully-fledged MVC-style apps which utilize this heavily, so I wouldn't say it's important for F# unless there's a technical reason to make everything be ActionResult<'T> based.

@cartermp having that explicit return type is optional, but it helps MVC do a few things:

It can provide better metadata about the structure of the response (e.g. what allowed response types there are)
It allows the developer to either return the data type (e.g. string[]), or to return an HTTP status code such as a 404 Not Found

So it's very much a recommended pattern now, and I think even F# users would benefit from it (though the syntax might be less than ideal in F#).

@Eilon I see. In that case, the F# syntax update would be simple:

    [<HttpGet>]
    member this.Get() =
        let values = [|"value1"; "value2"|]
        ActionResult<string []>(values)

    [<HttpGet("{id}")>]
    member this.Get(id:int) =
        let value = "value"
        ActionResult<string>(value)

Those could be one line for bikeshedding purposes, but I think it's better to just have it like this.
I also noticed that I get "Connection is not secure" errors when testing with dotnet run:

With these tools:
.NET Command Line Tools (2.1.300-preview1-008174)

Product Information:
 Version:            2.1.300-preview1-008174
 Commit SHA-1 hash:  b8df89a54f

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.13
 OS Platform: Darwin
 RID:         osx.10.12-x64
 Base Path:   /usr/local/share/dotnet/sdk/2.1.300-preview1-008174/

Microsoft .NET Core Shared Framework Host

  Version  : 2.1.0-preview1-26216-03
  Build    : f2c3216183d20416568a4bbf5bb7d153e826f153

Could that be issue in the recent updates to use HTTPS? The template is updated and the code looks like this:
    member this.Configure(app: IApplicationBuilder, env: IHostingEnvironment) =
        if (env.IsDevelopment()) then
            app.UseDeveloperExceptionPage() |> ignore
        else
            app.UseHsts() |> ignore

        app.UseHttpsRedirection() |> ignore
        app.UseMvc() |> ignore

Which looks correct based on what the C# template is doing.

@cartermp thanks for the updated code, I think we should use that. @scottaddie - can you update the F# template code for consistency with C#?
@cartermp - the error you're seeing is expected if you haven't enabled the HTTPS dev cert. When you run "dotnet" for the first time it should have given you instructions to enable the dev cert. You need to run dotnet dev-certs https --trust, and we have more info on this blog post: https://blogs.msdn.microsoft.com/webdev/2018/02/27/asp-net-core-2-1-https-improvements/

Merging now. Thanks @scottaddie

Happy to help! Thanks to everyone for reviewing.


Pick up 0.2.2 release of gitops-cli

0.2.2. release of gitops-cli has been released
Request it be picked up as it has security fixes in

@kvijai82 can we get this in kabanero 0.11 please? Thanks, Carolyn!

Please see https://github.com/rhd-gitops-example/services/releases/tag/v0.2.2. Thanks!

Yes we will try to get this in as soon as possible.
On Fri, Jun 26, 2020 at 9:53 AM Anne Michalak<EMAIL_ADDRESS>wrote:

Please see
https://github.com/rhd-gitops-example/services/releases/tag/v0.2.2.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kabanero-io/kabanero-pipelines/issues/432#issuecomment-650192217,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBUMTMBWZX6242ZUDBSNCTRYSR7NANCNFSM4OIHQZSA
.


conformance: enable GatewayClassObservedGenerationBump

Signed-off-by: bitliu<EMAIL_ADDRESS>

Codecov Report

Merging #896 (4b5153c) into main (15fb1a5) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #896      +/-   ##
==========================================
+ Coverage   63.54%   63.55%   +0.01%
==========================================
  Files          53       53
  Lines        7485     7485
==========================================
+ Hits         4756     4757       +1
+ Misses       2434     2432       -2
- Partials      295      296       +1

Impacted Files
Coverage Δ

internal/provider/kubernetes/helpers.go
77.86% <0.00%> (-2.46%)
:arrow_down:

internal/provider/kubernetes/controller.go
47.76% <0.00%> (+0.51%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@danehans from this test https://github.com/kubernetes-sigs/gateway-api/blob/release-0.6/conformance/tests/gatewayclass-observed-generation-bump.go it looks like its updating the description of the same GatewayClass, not a new one (NamespacedName used is same)

The GC name is different, so I think it's created a new one.
https://github.com/kubernetes-sigs/gateway-api/blob/main/conformance/tests/gatewayclass-observed-generation-bump.yaml
@arkodg

ah thanks for the clarification @Xunzhuo
looking at https://github.com/envoyproxy/gateway/pull/861/, it updates status for GCs with matching controller name but not accepted, will that fix this test ?

@arkodg I might not, see: https://github.com/kubernetes-sigs/gateway-api/blob/release-0.6/conformance/tests/gatewayclass-observed-generation-bump.go#L65. It cannot process util GC is accepted.

See: https://kubernetes.slack.com/archives/CR0H13KGA/p1673599220709819

@Xunzhuo thanks for discussing this issue with the Gateway API community. I have confirmed that EG is handling observedGeneration properly for GCs with a matching controllerName. I performed the following to reproduce:

Install latest EG: kubectl apply -f https://github.com/envoyproxy/gateway/releases/download/latest/install.yaml
Install latest quickstart: kubectl apply -f https://github.com/envoyproxy/gateway/releases/download/latest/quickstart.yaml
Confirm quickstart resources are reporting the expected status conditions.
Create a 2nd GC, eg2, with a controllerName that matches EG:apiVersion: gateway.networking.k8s.io/v1beta1
kind: GatewayClass
metadata:
  name: eg2
spec:
  controllerName: gateway.envoyproxy.io/gatewayclass-controller

Verify the 2nd GC reports the expected "Accepted=False" status condition and an observedGeneration that matches spec.generation, e.g. 1.
Update the 2nd GC to bump spec.generation:apiVersion: gateway.networking.k8s.io/v1beta1
kind: GatewayClass
metadata:
  name: eg2
spec:
  controllerName: gateway.envoyproxy.io/gatewayclass-controller
  parametersRef:
    group: config.gateway.envoyproxy.io
    kind: EnvoyProxy
    namespace: envoy-gateway-system
    name: eg

Verify the 2nd GC, eg2, still surfaces the Accepted=False status condition and the observedGeneration matches spec.generation, e.g. 2.

Instead of changing the GC name as done by this PR, I suggest that we comment-out this test with a link to https://github.com/kubernetes-sigs/gateway-api/issues/1650. Thoughts @arkodg @skriss @youngnick @AliceProxy @zirain ?

yah @danehans I agree with commenting out this extended test (GatewayClassObservedGenerationBump) . In the initial design, the project decided to support a single accepted GatewayClass to not have to deal with multi tenancy (multiple accepted GatewayClass) within a single process (users should instead create a new Envoy Gateway controller).
The design can def be changed in the future, but imho we will incur more complexity and community cycles without adding much value prop if we take on this design change to make above test pass

based on @youngnick 's suggestion in https://kubernetes.slack.com/archives/CR0H13KGA/p1673836949189379?thread_ts=1673599220.709819&cid=CR0H13KGA EG should be able to support the test if the conformance test is edited to enforce an Accepted condition rather than a Accpeted=True condition .

based on @youngnick 's suggestion in https://kubernetes.slack.com/archives/CR0H13KGA/p1673836949189379?thread_ts=1673599220.709819&cid=CR0H13KGA EG should be able to support the test if the conformance test is edited to enforce an Accepted condition rather than a Accpeted=True condition .

raised https://github.com/kubernetes-sigs/gateway-api/pull/1655 to address this.

Waiting for gw api v0.6.1.

can confirm via https://github.com/envoyproxy/gateway/pull/780 that this test has been fixed, we should use this PR to insert GatewayClassObservedGenerationBump into the list of conformance tests that ned to be run

@Xunzhuo now that gw api has been bumped to v0.6.1, can you please rebase and remove draft?


Add support for changing layout

Is this everything for theming? It functions but I'm not so sure this is absolutely everything from forem. I was able to successfully create a theme for BS3, and this was the process to apply the theme:
Add following to application.css
/*
*= require mongoid_forums-bootstrap
*/

Add following to application.js
//= require mongoid_forums-bootstrap

Change the following for the mongoid_forums.rb initializer:
MongoidForums.layout = "mongoid_forums-bootstrap/bootstrap"
MongoidForums.admin_layout = "mongoid_forums-bootstrap/bootstrap-admin"

Is this the same as radar/forem

:+1:

Wait, can't a user just generate views and theme them?

@AustinLMayes yeah, radar/forem seems to have some built in method that makes it 'easy' to theme which I'm trying to add (however I don't fully understand it). As far as I can see, it kind of just depends on you setting the layout for each controller through the config. Then, because it's a Rails engine, it looks like the stylesheets and javascript all become available for use, and by requiring it in your app you get a theme.
I'm hoping someone more familiar can explain the process, and confirm if this works the same way. For now I'll leave it unmerged.

@skipperguy12 that way makes the user edit CSS and not actual html content which is kinda pointless IMO, just let the users generate views and customize them on their own.

@AustinLMayes that's already allowed, I guess I'll leave this open because as you're saying it's almost pointless. This will be left open, if anyone wishes to come and explain it's purpose, if this is correct, and why it should be merged.

@skipperguy12 ok


Fix host fuction -> import mapping code

The code to map host functions to imports during the instantiation process doesn't quite work. We've gotten away with it because it works well enough for simple use cases: https://github.com/dylibso/chicory/blob/main/runtime/src/main/java/com/dylibso/chicory/runtime/Module.java#L181
In short, we should be iterating over the imports and trying to satisfy those and not the other way around. Also the host functions need to be ordered in the way that the imports expect. index of the import is essentially it's function id as the imports come first in the function list.
I'll tackle this one, I have an idea how it should work.

I think this closes the issue: https://github.com/dylibso/chicory/pull/95


People are not recognized with Base 3 in combination with this integration.

People are not recognized with Base 3 in combination with this integration.
I always used the Base 2 with doorbell dual. Doorbell Dual has its own facial recognition. When recognizing a person, it checked that that person is also at home in Home Assistant and controls the door lock to open the door.
That works perfectly. But now I have replaced the Base 2 and now use a Base 3. Because Base 3 has its own person recognition, it is switched off from the doorbell and you only see 'unknown' with this integration.
That's because Base 3 has its own facial recognition and we can't indulge that with this integration.
How can we get that option available again?

you can check the logs if add-on is sending the person name information to integration. If not, you need to work with bropat to help him to implement this.

please share your debug logs following readme over here, I will take a look at details


Test io_uring_prep_renameat

Need to create basic test for io_uring_prep_renameat

https://github.com/axboe/liburing/issues/1090


make PKGBUILD more generic by using make install

Go back to an older version because of bugs

Thanks, I'll be looking into why all the issues. I'm thinking something to do with not resetting the state fields?


Animations

Create animations and transitions on webpage

This is the link to the readme file that contains the live link for my project
Readme-file


Logging performance to a file

Is your feature request related to a problem? Please describe.
The need to log performance into a file
Describe the solution you'd like
Directly log into a file in the private gpt
Describe alternatives you've considered
log other data into other files (sources and the rest)
Additional context
We need to present the performance

The analysis of performance is done on the branch of logging for this repo


api: use traffic_direction over operation_name if specified

Signed-off-by: Kuat Yessenov<EMAIL_ADDRESS>Description: Use the listener-level field for the tracing direction over the per-filter field. Unfortunately, the per filter did not provide an "unspecified" default, so this appears to be the right approach to deprecate the per-filter field with minimal impact.
Risk Level: low (uses a newly introduce field traffic_direction)
Testing: unit test
Docs Changes: proto docs
Release Notes:

Can you also add the ability to log the traffic direction in the access log please? [if its not already there].

Sure, but I think adding an access log extractor should be separate from this. I think it's under change, too, with Lizan's TCP access log work.

/azp run

Added a deprecation notice.
Contributes to #7751

/retest
(seems like a flake)

@kyessenov After a lot of trial and error I've worked out that the deprecation warning is only triggered by:
          tracing:
            operation_name: egress

If you specify the operation_name as ingress there is no deprecation warning.
          tracing:
            operation_name: ingress

Is this a bug? Should I raise a separate issue?

My guess is that this is due to the unfortunate choice of "ingress" enum value as the default value. The default values are omitted on the wire in protobuf so deprecation warning is not triggered.

Another issue I think it related to this is might Lightstep tracing stopped working when I implemented traffic_direction. Looks like Zipkin stopped working in our work environment also. Need to dig further and will report back


NIFI-9824 - Address WARN log message in ProcessorAuditor on action Ru…

…nOnce
Description of PR
Properly handle audit of processor action "Run Once".  Class ProcessorAuditor generates an audit record for this action, where processorAction.setOperation() needs to be called to specify the operation.  If the operation is unspecified, an NPE is thrown, and the operation is not audited.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[X] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Thanks for addressing this issue @greyp9. It seems like this change should cover the particular issue, but it also looks like the ScheduledState has the following additional values that may need to be considered:

STARTING
STOPPING
RUN_ONCE

I'm also not sure whether the previous scheduled state of STOPPED always implies the RunOnce operation. That seems to be the case in common operation. Do you think those other states should be considered as part of this change?

Hmm.  It's true these additional values are not considered.  My testing zeroed in on the "Run Once" case, and this change  was sufficient to address it.  Maybe we could revisit if we encounter a different case in the future?
Another option would be to add an } else {, but not sure if that makes things clearer, as there isn't an enum Operation that would be appropriate.


Ensure hashval is reset in observer if match fails.

I hit an issue whereby my contract calls and origination attempts would never return. After investigation it turns out the root cause is that the monitor re-uses a hash object when checking operations in each block, but the reset of the hash object was placed such that it only got called if there was success on the first operation hash (subsequent hashes would fail because the hashval variable hadn't been reset).

Darn, good catch! Thanks.


Reach api

hitting plants endpoint and rendering it in json
need to hide api key still
need to serve up search api endpoint and hit that instead perhaps
Also need to add image url for each plant as well as endpoints for each plants recipes etc

Looks like you're building things in React  :) I wish I had more meaningful things to contribute, but I see where you're going with this, and it looks good.

Thanks Adrienne!  There's not much there yet.. More juicy pr's to come!

Your api calls are so fetch. Cool that there is a figaro package. React looks good!

Thanks July!  Actually switched from the fetch api back to jQuery.  Steve was saying that fetch will most likely be the standard for making ajax calls in 2 years but right now its not supported on mobile.


Working on M1 Macs?

Hi - curious if this is working on M1 Macs:
I'm unable to get any rendering on my:

It works when installing through go install github.com/fogleman/primitive@latest and running from terminal. Running from Mac App is also broken on M1 Max for me.

Looks like the mac app is still broken, canvas stays white. (M1, monterey)

Can confirm mac app still broken. Tried on MacBook Pro M1 Ventura, and also MacBook Pro Intel Ventura.

Hey! I saw the app is broken too on my M2, but there are no fail logs, just a blank canvas.
Any chance of getting a working GUI release for M1/M2?
Thanks!


[lifelong learning]: S3 storage support

Add docs and code comment
S3 storage support
Reduce parameters for initial
work with #85

Signed-off-by: JoeyHwong<EMAIL_ADDRESS>

/close


[FEATURE] Add network capability

Description:

Send necessary player data to server
Apply game logic on server
Send game data to players

Created the basic server and client handling.
The server handles multiple clients.

The server logic now runs in a multiprocessing Process.
Data flow from and to the server requires 2 multiprocessing Queue, one for sending data to clients and another to get client data.
Next step is to create a class to inteface game logic and the server process.

Client has been refactored to use a wrapper around multiprocessing.Pipe and use states.


ClassifierOutputStep [wip]

Signed-off-by: Andrii Tuzhykov<EMAIL_ADDRESS>

@atuzhykov can you also add some reference documentation in the ClassifierOutputStep using @Schema(description = "..."). You'll have the dependencies set with you if you merge from master. It will help out in auto generating swagger documentation for our clients in multiple languages.
Examples are here for how to use the schema: https://github.com/KonduitAI/konduit-serving/blob/master/konduit-serving-pipeline/src/main/java/ai/konduit/serving/pipeline/impl/step/logging/LoggingPipelineStep.java#L34-L35
https://github.com/KonduitAI/konduit-serving/blob/master/konduit-serving-pipeline/src/main/java/ai/konduit/serving/pipeline/impl/step/logging/LoggingPipelineStep.java#L38-L40
So, basically one @Schema annotation for the main class and then one for each configurable field.


"Sărulești" is listed in the drop-down menu

Description:
"Sărulești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Sărulești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Sărulești, CĂLĂRAȘI (Sărulești)".


🧹 Monorepo Cleanup: Phase 2

Some items left from our initial cleanup at #2296 plus any other things that may come up meanwhile

[x] #2407
[ ]  Uniform target frameworks (when available) or at least minimise the amount of versions we use.
[ ]  Do a solution-wide reformat to ensure all code complies with .editorconfig in root. THIS SHOULD BE DONE PER CONNECTOR, 1 AT A TIME, to allow each owner to choose when is the best moment to do this.
[x]  Use Central Package Management and lock files (this + CircleCI caching should speed up restoring time considerably) https://learn.microsoft.com/en-us/nuget/consume-packages/central-package-management
[x] Codeowners (potential for more specific entries in the CODEOWNERS file. @JR-Morgan mentioned a few places in core where he'd like to be pinged for changes. And I'm sure there are parts of Core/Objects that some people would like the opportunity to give feedback on changes)
[x] Potentially #2409 Will be part of the CI rehaul

Central package management Isn't really what we need. It could help on some very specific cases, but in general we don't share enough dependencies between all packages to make it worth it.

For the "Uniform Target Frameworks" item, the ideal solution would be for each converter to target the same framework as its connector. @JR-Morgan has brought up in the past that, although this currently works, it could be introducing some difficult to track bugs that we're not aware of yet.
Moving from targeting netstandard2.0 to their respective frameworks shouldn't break, as we're already doing so for Rhino7 due to targeting issues when compiling.

Closing this ticket as we are moving to Jira to reduce noise. You can access it there.


Dynamic Replication

#https://github.com/spring-cloud/spring-cloud-netflix/issues/2071
As mentioned in the issue above, would there be any issue in modifying code so that eureka nodes can pick up new nodes that are not in the config?

Without this feature it's impossible to scale eureka


Expose FakeTimers.timers: [TimerID]

Do you want to request a feature or report a bug?
feature
What is the current behavior?
FakeTimers doesn't expose _timers making hard to test if a timer id is present (not cleared).
What is the expected behavior?
I was looking for a way to test the non-existence of a "timeout id" after a clearTimeout but I believe that the "timers" ids are not exposed.
With sinon this would be done like this:
describe('when unmounted', () => {
    it('clears timeout', () => {
      const clock = sinon.useFakeTimers()
      const timeout = wrapper.instance().timeout
      expect(timeout).toBeDefined()
      wrapper.unmount()
      expect(clock.timers).to.not.include.keys(timeout.id)
    })
})

What would you think of exposing it?

I believe this could be addressed with this PR: https://github.com/facebook/jest/issues/5165. cc @SimenB

We still don't expose the underlying timers. we might want to expose the underlying clock though, not sure

I think it wouldn't harm to return it. But it might be better idea to do it as a followup, when we're sure Lolex is here to stay.

We keep track of the timers today, so potentially, we could expose it: https://github.com/facebook/jest/blob/74892f0f6a0abd6a4e9d5a7053fdcd22e4f6d23c/packages/jest-util/src/fake_timers.js#L80


Add support for the XXX comment prefix

Using XXX as a tag has a long history in Unix and is used in Java code to flag code that's bad even though it works. It's listed as an example tag on Wikipedia. I also happen to work on a codebase that uses it as a FIXME tag, so I have my own reasons for wanting to see it in my Nova sidebar 😉

Thanks @flother! The extension has been updated with your changes.


Read reVUE json from reVUE-data repo

Read reVUE from https://github.com/knowledgesystems/reVUE-data/blob/main/VUEs_confirmed.json instead of local json file

The test MyVariantInfoIntegrationTest.testMyVariantInfoAnnotationLargePOST has been failed for a while, I'll fix it in a separate PR.


The servo is set to 180 degrees, but it comes up 135 degrees

I have the servo servo angle set to 100% (180°) on my program, but when running in the real thing, the simulator shows 135° (and the real thing is really only 135° as well).When I press the button again, the angle goes back to normal

https://makecode.microbit.org/_e8fRPs4atah4

is this a firmware issue? The simulator shows 180

what is the correct procedure to repro this issue?

please attach a jacdac trace

https://github.com/microsoft/pxt-jacdac/pull/101

Fix deployed, please test.

Yes，It's fixed. Thank you.


Customize error messages

Is there currently a way to override the hard-coded error messages in the serializers.py file?
Some options are:

Subclassing the LoginSerializer in our code.

Submit a PR to allow for customizable messages.

What are your thoughts?

Yeah also looking at this.  Any thoughts on if we can help with a PR?

@amyschools @snewcomer Thanks for the suggestions. Added PR. With this, you should be able to override specific error message inside __init__ method. Please lmk if it looks good or needs more work. Thanks again.


Cosmetic - Swap order of ovary table and fecundity table (14 and 15)

Can we swap order of ovary table (14) and fecundity table (15) so that the fecundity table does not split leaving 1 line alone on the next page? The other option is reduce the font size of the fecundity table but then it will be smaller than the other tables around it.

I kind of think that we can remove Table 14 because we have tv maturity which is not captured in this table.

That would fix the cosmetic issue as well. If there are no objections, I'll do that

Does that mean remove this figure as well?

There is a TODO right where these are referenced.. here:
https://github.com/pacific-hake/hake-assessment/blob/e6fbce23e41ac5de3e7ad90a5f99ee3df8497249/doc/008-data.rmd#L48
So that part would have to be re-written/removed but I wasn't sure exactly what you were going to do there @kellijohnson-NOAA

I made a few preliminary figures for the maturity at age. The first, is the difference between the ogive we have been using, the same analysis but done without Canadian data, and Eric's NULL model that does not have time or space (i.e., basically trying to replicate Melissa Head's work). I am not sure if this needs to go in the document or just in our presentation. The second figure is of time-varying maturity-at-age but I decided to use lines instead of ridges or coloured tiles.
1.

2.

As always, all comments are welcome. Though, I do know that I cut off the labels when I saved them but I am too lazy to fix it right now. FYI these data are in the package in maturity_estimates_df if you want to play around with them.

My thoughts:
For the document: I'm thinking that we need to show the time-varying maturity figure (your 2nd figure Kelli) because that is what is being used in the base assessment model, but I think we need to add what is being used during the historical period (pre-2009; unless that is what the 2009 line itself represents).  That way we show a complete time series picture.  If we then replace our old 'North'/'South' maturity figure we then lose sample sizes, but if we keep the sample size table then we are still covered at least in terms of total samples, we still won't show samples by year but that will be in the appendix (correct Kelli?).  In fact, this new analysis is using many more samples than the maturity analysis used in previous assessments (and thus shown as sample sizes in that north/south figure).
For the SRG presentation: I think we can show the figure in the document (suggestion above) and also your first figure Kelli, with the null model, and maybe add again there our previously used ogive for comparison. As a reminder, Canada data was too sparse for t.v. maturity, but it was not for stationary maturity, correct? (i.e., that's why the first figure includes adding Canada).

Agree with Aaron. New one certainly shows the changes at age-3 nicely. (Presume tick marks can get added, little hard to hone in on age-3 without them). Thanks for doing this.

@aaronmberger-nwfsc great comment about equilibrium maturity, I am assuming this comment also pertains to forecasted years as well. I can add those lines. Then regarding the sample sizes, they are all in the appendix. But, I think we should still remove Table 14 and change the sample sizes in Table 15 to be what was used not everything that was collected because some of those do not have functional maturity and thus cannot be used for any analysis.

Fixed the tables and now, I am working on the figures

Thanks to @cgrandin this is done. Sorry the code that I pushed last night was such a mess. And, thanks for iteratively working with me to get something that readers will actually be able to see the differences in the ogives. Way to go @cgrandin 🥇 .


If Ite2000 doesn't want to accept this pr then we will publish on intellij marketplace

https://github.com/lte2000/intellibot/pull/25

Don't forget to update the documentation / metadata to clarify that this is a fork, and not supported by millennial media, mtrubs , lte2000 ...


eslint installing errors

17:15 $ npm install
npm WARN package.json<EMAIL_ADDRESS>No README data
npm ERR! Linux 4.2.0-23-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! node v4.2.4
npm ERR! npm  v2.14.12
npm ERR! code EPEERINVALID

npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants babel-eslint@*

npm ERR! Please include the following file with any support request:
npm ERR!     /home/brad/rails_projects/propertysite/client/npm-debug.log

Not doing anything fancy.  Wasn't running into this problem a few weeks ago.  If I'm doing something wrong let me know.
Also slightly curious that running rails g install without -L for linting still gives me the linting setup.

@wyaeld we don't have a package called react-webpack-rails-tutorial.
I suggest you work your way through the tutorial for v2 and let me know if you have issues.

@wyaeld We're also using babel 6, fyi. Please see what the generators create: https://github.com/shakacode/react_on_rails-generator-results

That's the default package name coming from the generator, I just haven't changed yet.
This is all react_on_rails v2.0.1, installed an hour ago

Here is the complete client/package.json
{
  "name": "react-webpack-rails-tutorial",
  "version": "1.1.0",
  "description": "Built using the react_on_rails generator.",
  "main": "server.js",
  "engines": {
    "node": "5.0.0",
    "npm": "3.5.0"
  },
  "repository": {
    "type": "git",
    "url": "https://github.com/shakacode/react-webpack-rails-tutorial.git"
  },
  "keywords": [
    "react",
    "tutorial",
    "comment",
    "example"
  ],
  "author": "justin808",
  "license": "MIT",
  "bugs": {
    "url": "https://github.com/shakacode/react-webpack-rails-tutorial/issues"
  },
  "homepage": "https://github.com/shakacode/react-webpack-rails-tutorial",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "start": "node server.js",
    "build:client": "NODE_ENV=production webpack --config webpack.client.rails.config.js",
    "build:server": "NODE_ENV=production webpack --config webpack.server.rails.config.js",
    "build:dev:client": "webpack -w --config webpack.client.rails.config.js",
    "build:dev:server": "webpack -w --config webpack.server.rails.config.js",
    "lint": "npm run eslint && npm run jscs",
    "eslint": "eslint --ext .js,.jsx .",
    "jscs": "jscs --verbose ."
  },
  "dependencies": {
    "babel": "^6.3.26",
    "babel-cli": "^6.3.17",
    "babel-core": "^6.3.26",
    "babel-loader": "^6.2.0",
    "babel-plugin-syntax-decorators": "^6.3.13",
    "babel-polyfill": "^6.3.14",
    "babel-preset-es2015": "^6.3.13",
    "babel-preset-react": "^6.3.13",
    "babel-preset-stage-0": "^6.3.13",
    "es5-shim": "^4.4.1",
    "expose-loader": "^0.7.1",
    "immutable": "^3.7.5",
    "imports-loader": "^0.6.5",
    "jquery": "^2.1.4",
    "jquery-ujs": "^1.1.0-1",
    "loader-utils": "^0.2.11",
    "lodash": "^3.10.1",
    "mirror-creator": "0.0.1",
    "react": "^0.14.3",
    "react-bootstrap": "^0.28.1",
    "react-dom": "^0.14.3",
    "react-on-rails": "2.0.1",
    "react-redux": "^4.0.5",
    "redux": "^3.0.5",
    "redux-promise": "^0.5.0",
    "redux-thunk": "^1.0.2",
    "webpack": "^1.12.8"
  },
  "devDependencies": {
    "babel-eslint": "^5.0.0-beta6",
    "babel-plugin-react-transform": "^2.0.0",
    "bootstrap-sass": "^3.3.5",
    "bootstrap-sass-loader": "^1.0.9",
    "css-loader": "^0.23.0",
    "eslint": "^1.10.3",
    "eslint-config-airbnb": "^2.0.0",
    "eslint-config-shakacode": "^0.0.1",
    "eslint-plugin-react": "^3.11.3",
    "express": "^4.13.3",
    "file-loader": "^0.8.4",
    "jade": "^1.11.0",
    "jscs": "^2.8.0",
    "node-sass": "^3.4.2",
    "react-transform-hmr": "^1.0.1",
    "sass-loader": "^3.1.1",
    "style-loader": "^0.13.0",
    "url-loader": "^0.5.6",
    "webpack-dev-server": "^1.12.1"
  }
}

nevermind, I've been using node_4x distributions from nodesource, and the npm version was the problem.  Updating to 5x resolved it.  Up to you whether that's work a note somewhere for the next poor guy

Sure. Thanks @wyaeld. I'll add that to readme.

Looks good.
On 16 January 2016 at 18:35, Justin Gordon<EMAIL_ADDRESS>wrote:

[image: 2016-01-15_21-34-28]
https://cloud.githubusercontent.com/assets/1118459/12370704/d1961cd6-bbcf-11e5-9983-363f3ba17421.png
—
Reply to this email directly or view it on GitHub
https://github.com/shakacode/react_on_rails/issues/207#issuecomment-172160232
.

@wyaeld BTW -- npm is separate from node. Maybe you just needed to update npm? that's why I didn't mandate 5.0 in the package.json.

In the production environments I work in, adding another package manager
like nvm is a little more frowned upon.  The nodesource PPAs work, but they
also bundle things together.  At least that way we should be able to expect
the versions work, and the distributions are also very Dockerfile-friendly.
Current nodesource 5x gets you node v5.4.1 and npm 3.3.12
I tried updating npm locally, but wasn't having much luck with it, user
error :-(
On 16 January 2016 at 18:41, Justin Gordon<EMAIL_ADDRESS>wrote:

@wyaeld https://github.com/wyaeld BTW -- npm is separate from node.
Maybe you just needed to update npm? that's why I didn't mandate 5.0 in the
package.json.
—
Reply to this email directly or view it on GitHub
https://github.com/shakacode/react_on_rails/issues/207#issuecomment-172160812
.

FWIW I'm on node v4.3.0 and had the same error until I upgraded npm (to 3.7.1).


feat: combination of more than 2 streams

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.96%. Comparing base (bc7a6c2) to head (2aaec44).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #72      +/-   ##
==========================================
- Coverage   86.21%   85.96%   -0.25%
==========================================
  Files          68       66       -2
  Lines         885      841      -44
  Branches      141      132       -9
==========================================
- Hits          763      723      -40
+ Misses         74       73       -1
+ Partials       48       45       -3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


How to color elements of latex matrices

Hi, I wonder how to set a color to individual elements in a matrix.
matrix2 = TexMobject(r"\begin{pmatrix} 4 \\ 3 \end{pmatrix}")
I would like to colour the number 4 in red.
When I break down the TexMobject for the matrix into tuples in order to run
matrix2[x].set_color(COLOR) I get an error.
Can somebody explain to me how I can split a matrix in Tex format.
Greatly appreciate it!

You have to indentify each element with:
# You can use this without split it
matrix2 = TexMobject(r"\begin{pmatrix} 4 \\ 3 \end{pmatrix}")
matrix2[0].set_color(RED)
matrix2[1].set_color(ORANGE)

When I do:
matrix2 = TexMobject(r"\begin{pmatrix} 4 \\ 3 \end{pmatrix}") matrix2[0].set_color(RED)
The whole matrix is coloured in RED. I only want the 4 to be coloured in RED.

Try matrix2[0][1]

Thanks, very clever

Don't forget close the issue


The placeholder in search bar is not readable properly in neighborhood view for small screen.

In the search bar in neighborhood view , there is a placeholder which says "Search in Neighborhood". This is not readable when the screen size is reduced to (min-width: 410px) and (max-width: 1023px).
This issue looks like this.

I have completed the solution for this.

It's just a matter of visibility, it's not a bug.

Yes that's not a bug but I thought it was an improvement to be done. That
is actually resolved now. So should push it from dev branch or not?
On Tue, Feb 28, 2023, 02:41 Lionel Laské @.***> wrote:

It's just a matter of visibility, it's not a bug.
—
Reply to this email directly, view it on GitHub
https://github.com/llaske/sugarizer/issues/1207#issuecomment-1447095578,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVAAZN3YZT4NVURTIQK57PDWZUKBTANCNFSM6AAAAAAVJKO254
.
You are receiving this because you authored the thread.Message ID:
@.***>

No