Replace search tool

SearpApi is expensive to use, replace with Duckduckgo.

I can't access api document.
Maybe the duckduckgo API isn't available.
https://www.reddit.com/r/duckduckgo/comments/xm6ire/cannot_find_ddg_instant_answers_api_documentation/

Thanks for your work! We (@serpapi) would love to sponsor your project.
Let us know what do you think.

@ilyazub
Wow, Serpapi team! Thank you for the offer.
I believe that search tools are extremely important for autonomous agents. I would like to consider adding other options not only in terms of cost but also in other factors. What are your team's thoughts on the development of those options?

@miurla Thanks for your kind feedback and considerations!
Please clarify what do you mean by "other factors" and "development of other options".

@ilyazub
I apologize for the lack of clarity.
The original source of BabyAGI (BabyBeeAGI), which is at the core of this project, uses SerpAPI, and I tried to import it as faithfully as possible. Therefore, I don't know how the original code will be improved in the future, but I would like to be as responsive to changes as possible.
In the case of LangChain, there can be multiple search tool options within the Agent's Tools. In the future, as we enhance the tools, there is a non-zero possibility that we will increase the options.
Our project's core is the interface, so we don't prioritize expanding backend options, but we prioritize developers and users, so I consulted with you in advance.
It may be overthinking since the project has just started! If there are no issues with the above points, we would appreciate your sponsorship. Please consider it.
(P.S. I have a SerpAPI account, so it's convenient and useful for running LangChain!)

@miurla, having multiple search tools while being sponsored by SerpApi is totally fine. Let's discuss this via an email (I'll send a message).

@ilyazub
I apologize for the lack of clarity.
The original source of BabyAGI (BabyBeeAGI), which is at the core of this project, uses SerpAPI, and I tried to import it as faithfully as possible. Therefore, I don't know how the original code will be improved in the future, but I would like to be as responsive to changes as possible.
In the case of LangChain, there can be multiple search tool options within the Agent's Tools. In the future, as we enhance the tools, there is a non-zero possibility that we will increase the options. Our project's core is the interface, so we don't prioritize expanding backend options, but we prioritize developers and users, so I consulted with you in advance.
It may be overthinking since the project has just started! If there are no issues with the above points, we would appreciate your sponsorship. Please consider it.
(P.S. I have a SerpAPI account, so it's convenient and useful for running LangChain!)

Could we try using Google's Programmable Search Engine??Hope it gives better search results

@ilyazub
Thank you for your understanding.
I'll reply to your email.

Create a separate issue if search options become necessary.


colors for structs and typedefs

In C lang, is possible use a color for structs and typedefs?

Sorry for my late reply, unfortunately the time to work on my projects is rare.
Could you please post a example code and a screenshot?
The struct and typedef keywords should be supported out of the box.

If you meant the name itself there is currently no specific scope provided for it by the default C/C++ syntax highlighting.

I'll close this issue for now since there is currently no way to style these, feel free to reopen if you meant another element or if you're aware of a supported solution.


Bug: Armor Master (mythic ability) not removing max dexterity bonus from armors

As title, noticed it while using heavy armor.

Are you encumbered? This is the most common issue I see when this is reported. If not can you give me more details of what is happening here.

I was going to report the same thing and that was exactly it.  Forgot how wimpy my dude was.  Thank you.

And I as well had just come here to ask about the same: I had Woljif take it, but it did not seem to allow for a bonus from his +9 Dex Mod, until I realized it was just a matter of shedding encumbrance.
Thanks for making this great mod, Vek!


Refactor tests

A PR for refactoring the cellfinder workflow tests, following the feedback collected on issue #26.
Main features
They address the points from issue #26:

define unit tests
separate integration and unit tests
make workflows and tests more modular:

added a utils.py module
separate general unit tests (e.g., for parser) and cellfinder-specific unit tests (with modules / directories)

defined fixtures in the files with the test definitions

I kept a conftest.py for the fixtures that are shared between unit and integration tests. I also use an additional conftest.py for the fixtures shared between utils unit tests and cellfinder-specific unit tests.

replaced the make_config helper functions by config files that are part of the test data

Other additions

added default config files (under brainglobe_workflows/configs)
renamed benchmarks directory to brainglobe_benchmarks
removed cellfinder subdirectory under brainglobe_workflows
added an entrypoint for the workflow called cellfinder-workflow
benchmark tests marked as skip for now (to focus on the workflow ones)

Comments / questions

I think I went down a parametrisations 🐰 🕳 , maybe some are redundant? Any feedback on this is welcome!
The tests are a bit slow to run, mainly because often I download the data from GIN. Should I look into caching this? Right now I don't because I download the data to a pytest temp dir, which is different for each test run.
I wanted to define fixtures per test module and used multiple conftest.py files to do so. But I'm not sure if that makes any difference compared to the case of having all of the fixtures in one conftest.py file (I think pytest just looks for conftest.py files and loads them all at once). Is there a better way to define fixtures per test module?
For a few commits I replaced the use of arparse by typer, and the code look cleaner and more readable, but at the cost of adding an extra dependency. Chatting to Adam he mentioned in brainglobe we try to keep dependencies lists short, so I reverted back to argparse but let me know if you have any thoughts about it.

These two questions relate to our live discussion of where the workflow data should live, and to a further thought of mine that feels that a lot of the test code is still a bit complicated.

I think I went down a parametrisations 🐰 🕳 , maybe some are redundant? Any feedback on this is welcome!

I think the parametrisations you've done are fine, but I'd say the if statement distinguishing two tests is a minor code smell. Maybe split the default case off into a separate test that doesn't require parametrisation?

The tests are a bit slow to run, mainly because often I download the data from GIN. Should I look into caching this? Right now I don't because I download the data to a pytest temp dir, which is different for each test run.

Yes, I think caching is worth it long-term to improve the development experience. If we move to using paths relative to $HOME as discussed, I would mock Path.home() as done in brainrender-napari instead of using tmp_path which would simplify both the test code and the caching..
The test code would then be something like:
@pytest.mark.parametrize(
    "input_config",
    [
        "input_config_fetch_GIN",
        "input_config_fetch_local",
    ],
)
def test_main_non_default(
):
    """Test main function for setting up and running cellfinder workflow
    Parameters
    ----------
    input_config : Optional[str]
        Path to input config json file
    """
 	# mocking of $HOME in conftest.py takes care that test data and user data are separate
    cfg = main(str(Path.home()/".brainglobe/cellfinder/workflows"/input_config))

    # check output files exist
    assert (cfg.detected_cells_path).is_file()


remove 'kubectl set image' from setup-cluster.sh

Blocked on https://github.com/wolfi-dev/os/pull/2514

The dogfooding config in the downstream repos is now done, and they fail if they have nothing to do, which they don't since they match this previous config.


Support async datasource drivers

How does it look with the support for the async drivers? Jooq has support for it since 3.15 https://github.com/jOOQ/jOOQ/issues/6298
https://blog.jooq.org/tag/async/
My project is currently based on vertx-jooq and I'd like to migrate the whole app to quarkus. My db calls are ~async~ reactive atm and updating to quarkus would result in going back to jdbc. I know that I could still use vertx-jooq, but I would like to use a "from one hand" solution.

jOOQ 3.15 supports R2DBC, which isn't based on the vertx drivers. I do hope that vertx will team up with R2DBC to implement their SPI to create that "from one hand" situation, knowing that reactive != async, though...

Hello @lukaseder, nice to meet you. At first, you're doing a very great job, If I search for something related with jooq, I see in 90% of the cases a very detailed answer from you. So thanks for that! (I also linked the jooq issue, in expectation to get an answer from you.)

jOOQ 3.15 supports R2DBC, which isn't based on the vertx drivers. I do hope that vertx will team up with R2DBC to implement their SPI to create that "from one hand" situation, knowing that reactive != async, though...

I corrected my question. I am using the reactive way, It seems I'm using it a bit wrong :).
Ok, so I hope that vertx will support r2dbc. There are some related issues https://github.com/eclipse-vertx/vertx-sql-client/issues/249 https://github.com/eclipse-vertx/vertx-sql-client/issues/245. But sadly it isn't that active, how I'd like to see.
I'm right, that if I use the JooqCustomContext in this project and configure it to use R2DBC (adding the driver), then I should be able to use the async calls from Jooq?

I'm right, that if I use the JooqCustomContext in this project and configure it to use R2DBC (adding the driver), then I should be able to use the async calls from Jooq?

Probably... Well, let us know if it works :)

I got it working after some drawbacks. I published a repo with the code (Kotlin). I implemented both configuration options mentioned in the docs. I am a bit unsure if it is using the R2DBC driver, but the errors I got were related to the R2DBC driver. Maybe you can have a quick look @lukaseder ?
For Agroal there has to be a JDBC implementation present at runtime. Are NO-OP JDBC drivers existing? I haven't found one.
Without the hard dependency on Agroal it may be working without it.

What am I looking for?
lukas@LAPTOP-VH130UJV MINGW64 ~/jOOQ/mcve/quarkus-jooq-r2dbc-example
$ mvn clean install
08:40:51,105 [INFO] Scanning for projects...
08:40:52,321 [INFO]
08:40:52,323 [INFO] ---------------< com.example:quarkus-jooq-r2dbc-example >---------------
08:40:52,327 [INFO] Building quarkus-jooq-r2dbc-example 1.0-SNAPSHOT
08:40:52,330 [INFO] --------------------------------[ jar ]---------------------------------
08:40:53,066 [INFO]
08:40:53,068 [INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ quarkus-jooq-r2dbc-example ---
08:40:53,126 [INFO] Deleting C:\Users\lukas\jOOQ\mcve\quarkus-jooq-r2dbc-example\target
08:40:53,192 [INFO]
08:40:53,195 [INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ quarkus-jooq-r2dbc-example ---
08:40:53,355 [INFO] Using 'UTF-8' encoding to copy filtered resources.
08:40:53,363 [INFO] Copying 2 resources
08:40:53,374 [INFO]
08:40:53,375 [INFO] --- quarkus-maven-plugin:2.5.1.Final:generate-code (default) @ quarkus-jooq-r2dbc-example ---
08:40:54,750 [INFO]
08:40:54,752 [INFO] --- maven-compiler-plugin:3.8.1:compile (default-compile) @ quarkus-jooq-r2dbc-example ---
08:40:54,975 [INFO] Nothing to compile - all classes are up to date
08:40:54,978 [INFO]
08:40:54,979 [INFO] --- kotlin-maven-plugin:1.5.31:compile (compile) @ quarkus-jooq-r2dbc-example ---
08:40:56,012 [INFO] Applied plugin: 'all-open'
08:41:00,701 [INFO]
08:41:00,703 [INFO] --- quarkus-maven-plugin:2.5.1.Final:generate-code-tests (default) @ quarkus-jooq-r2dbc-example ---
08:41:02,060 [INFO]
08:41:02,062 [INFO] --- maven-resources-plugin:2.6:testResources (default-testResources) @ quarkus-jooq-r2dbc-example ---
08:41:02,070 [INFO] Using 'UTF-8' encoding to copy filtered resources.
08:41:02,073 [INFO] skip non existing resourceDirectory C:\Users\lukas\jOOQ\mcve\quarkus-jooq-r2dbc-example\src\test\resources
08:41:02,078 [INFO]
08:41:02,079 [INFO] --- maven-compiler-plugin:3.8.1:testCompile (default-testCompile) @ quarkus-jooq-r2dbc-example ---
08:41:02,090 [INFO] Changes detected - recompiling the module!
08:41:02,098 [INFO]
08:41:02,100 [INFO] --- kotlin-maven-plugin:1.5.31:test-compile (test-compile) @ quarkus-jooq-r2dbc-example ---
08:41:02,114 [INFO] Applied plugin: 'all-open'
08:41:02,834 [INFO]
08:41:02,835 [INFO] --- maven-surefire-plugin:3.0.0-M5:test (default-test) @ quarkus-jooq-r2dbc-example ---
08:41:03,248 [INFO]
08:41:03,249 [INFO] -------------------------------------------------------
08:41:03,252 [INFO]  T E S T S
08:41:03,253 [INFO] -------------------------------------------------------
08:41:04,099 [INFO] Running com.example.ExampleResourceTest
2021-12-08 08:41:09,435 INFO  [io.quarkus] (main) Quarkus 2.5.1.Final on JVM started in 2.715s. Listening on: http://localhost:8081
2021-12-08 08:41:09,437 INFO  [io.quarkus] (main) Profile test activated.
2021-12-08 08:41:09,437 INFO  [io.quarkus] (main) Installed features: [agroal, cdi, jdbc-h2, kotlin, narayana-jta, resteasy-reactive, resteasy-reactive-jackson, smallrye-context-propagation, vertx]
08:41:10,742 [INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 6.615 s - in com.example.ExampleResourceTest
2021-12-08 08:41:10,763 INFO  [io.quarkus] (main) Quarkus stopped in 0.055s
08:41:10,895 [INFO]
08:41:10,897 [INFO] Results:
08:41:10,899 [INFO]
08:41:10,900 [INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
08:41:10,902 [INFO]
08:41:10,908 [INFO]
08:41:10,909 [INFO] --- maven-jar-plugin:2.4:jar (default-jar) @ quarkus-jooq-r2dbc-example ---
08:41:11,075 [INFO] Building jar: C:\Users\lukas\jOOQ\mcve\quarkus-jooq-r2dbc-example\target\quarkus-jooq-r2dbc-example-1.0-SNAPSHOT.jar
08:41:11,099 [INFO]
08:41:11,101 [INFO] --- quarkus-maven-plugin:2.5.1.Final:build (default) @ quarkus-jooq-r2dbc-example ---
08:41:11,640 [INFO] [org.jboss.threads] JBoss Threads version 3.4.2.Final
08:41:12,971 [INFO] [io.quarkus.deployment.QuarkusAugmentor] Quarkus augmentation completed in 1856ms
08:41:12,988 [INFO]
08:41:12,989 [INFO] --- maven-install-plugin:2.4:install (default-install) @ quarkus-jooq-r2dbc-example ---
08:41:13,057 [INFO] Installing C:\Users\lukas\jOOQ\mcve\quarkus-jooq-r2dbc-example\target\quarkus-jooq-r2dbc-example-1.0-SNAPSHOT.jar to C:\Users\lukas\.m2\repository\com\example\quarkus-jooq-r2dbc-example\1.0-SNAPSHOT\quarkus-jooq-r2dbc-example-1.0-SNAPSHOT.jar
08:41:13,070 [INFO] Installing C:\Users\lukas\jOOQ\mcve\quarkus-jooq-r2dbc-example\pom.xml to C:\Users\lukas\.m2\repository\com\example\quarkus-jooq-r2dbc-example\1.0-SNAPSHOT\quarkus-jooq-r2dbc-example-1.0-SNAPSHOT.pom
08:41:13,088 [INFO] ------------------------------------------------------------------------
08:41:13,092 [INFO] BUILD SUCCESS
08:41:13,093 [INFO] ------------------------------------------------------------------------
08:41:13,098 [INFO] Total time:  22.014 s
08:41:13,100 [INFO] Finished at: 2021-12-08T08:41:13+01:00
08:41:13,102 [INFO] ------------------------------------------------------------------------

If I correctly called the reactive way of jooq.
If the correct driver is used at runtime. I haven't found a good way to track that. But I think this is not necessary, I don't want to waste your time.

If I correctly called the reactive way of jooq.

You only wrote one query, I think, and it's this? https://github.com/Dudeplayz/quarkus-jooq-r2dbc-example/blob/master/src/main/kotlin/com/example/JooqResource.kt
What could possibly be wrong about it?

2. If the correct driver is used at runtime. I haven't found a good way to track that.

What made you think your driver usage wasn't correct?

You only wrote one query, I think, and it's this? https://github.com/Dudeplayz/quarkus-jooq-r2dbc-example/blob/master/src/main/kotlin/com/example/JooqResource.kt

Yes, other calls would be analog?

What made you think your driver usage wasn't correct?

My first tries were executed with the wrong driver. But I think this should be fixed. If I use it at runtime I would discover it when I use the H2 as the default JDBC driver and use Postgres as the R2DBC driver.  Also I was using executeAsync at first and not the query builder as publisher.

Yes, other calls would be analog?

Yes. Always use jOOQ queries as Publisher<Integer> or Publisher<R extends Record>

Also I was using executeAsync at first and not the query builder as publisher.

That doesn't get backed by R2DBC yet


Add DataRoaster Operator

DataRoaster Operator is used to controll all the data platform components running on kubernetes.
It consists of mysql server, helm operator and dataroaster server which provides rest api to the client.

https://github.com/cloudcheflabs/dataroaster/pull/10


DOCS-2124: Moving third-party licenses to static hosting for OSS

@ronanc-tigera I'm getting some strange behavior on this one. I'm changing these large license files so that they're just static html in our static directory. I've changed the sidebars to link to the URL. But when you click on the link from the sidebars (in Reference/Attributions), it returns a 404. But then, if you refresh the page, the correct licenses page loads just fine.
Is this something you've seen before? The only thing I can think of at this point is to use the full URL (https://docs.tigera.io/...) and merge, hope that that resolves. Not at all sure why it would.


Admin/ site owner cannot see conversations they are not a party to

Thank you for making this.
It addresses the #1 complaint from my forum members: "no PM function".
As a site owner, I would want the ability to see all private conversations (read only: without editing capability is fine).
Please consider that a site owner is responsible (and probably legally liable) for what goes on in their forum.

I guess you mistake this repository for https://github.com/flagrow/byobu ; I can imagine you would like to have this feature. I would consider such a feature (toggle in admin for instance) but only if we make this somehow clear. Feel free to create a new issue with a possible (non technical) implementation on the correct repository.


Use python2 specific shellbang via /usr/bin/env

Since not all Linux distributions link /usr/bin/python to /usr/bin/python2 it is important to set the python version in the shellbang. Also using /usr/bin/env is more portable than /usr/bin/python

Hey, thanks for submitting this!  Will merge shortly :)


refactor:Provider.tsx

The reason being, in case any changes are done regarding the RPC URLs, then it will be done in one place which is in constants/chain instead of in both constants/chain and Provider.tsx. Personally opinion: Shall we allow users to use their custom RPC URLs given that public RPCs are slow

hey @akp111!
thanks for the contribution! we're using yarn for dependency management in the project, could you remove your package-lock and push this up again?
in terms of letting users override the RPC urls we do allow that already via the ChainRrpc prop on the provider:
https://docs.thirdweb.com/react/react.thirdwebproviderprops#properties

Hey @jnsdls
I have removed the pakcage-lock


Uncaught (in promise) Invalid parameters got this error while using icon

Whenever I am trying to use icon it gives this error.
And I am not able to write text.
Please help ASAP.
Thank you

+1

I guess my issue is related to same this issue.
   includeUI: {
          theme: blackTheme,
           ...
          menu: ['crop', 'flip', 'rotate', 'draw', 'shape', 'icon', 'text', 'mask', 'filter'],
          initMenu: 'icon',
        }

I don't want to init any initMenu option on load. When i don't then initMenu icon option stop working.
When i add menu and initMenu option togather then ``menu icon such as Arrow, Arrow-2,Star-1 etc.. work but with wrong canvas position on load.
Am i missing anything ?. I am using Vue version of this repo.

HI All, I too face this issue.
I able to draw icon if I initialize initMenu: ['icon'] else I could not draw icon from icon set.
@junghwan-park  is this issue fixed?


Remove qpe_hubbard tests

These tests are very large and will cause problems when running. These
tests are now disabled.

def test_whatever(bloq_autotester):
  if bloq_autotester.check_name == 'serialization':
    pytest.skip('too big')

  bloq_autotester(_whatever_example)

def test_whatever(bloq_autotester):
  if bloq_autotester.check_name == 'serialization':
    pytest.skip('too big')

  bloq_autotester(_whatever_example)

I gave this a try using test_qubitization_qpe_bloq_autotester as the "test_whatever" method.  I noticed two issues with this:

test_qubitization_qpe_bloq_autotester is never called from get_bloq_report_card.  The stack goes from get_bloq_report_card -> record_for_bloq_example ->check_bloq_example_serialize->assert_bloq_example_serialize

It looks like this won't prevent this test from running.

The other thing I noticed is that _qubitization_qpe_hubbard_model_large doesn't seem to be called by an autotester.  However, _qubitization_qpe_hubbard_model_small is called by an autotester

get_bloq_report_card is called ad-hoc by the developer outside of the CI flow. Re: (2) you have to "opt in" to including the auto-tests as part of the pytest unit tests, which is run as part of the CI flow

get_bloq_report_card is called ad-hoc by the developer outside of the CI flow. Re: (2) you have to "opt in" to including the auto-tests as part of the pytest unit tests, which is run as part of the CI flow

Gotcha.  Should we address how to remove it in get_bloq_report_card as well?

I don't think these changes are needed any longer.  We can close this PR.


Update Smogon scraping script

https://github.com/Dudly01/pkmn/actions/runs/6676158213/job/18145004910 showed that the Smogon scraping script is broken. Something changed on the website.

It may have been just reformatted.


trigger image rebuild

Signed-off-by: Jonathan Marcantonio<EMAIL_ADDRESS>Previous commit seems to have not been merged and imagescan appears to show new vulnerabilities, should solve both issues.
Addresses:

https://github.com/stolostron/backlog/issues/26566

/lgtm


Creating a composition with an edited video and an image applies the image on top of the video as an overlay instead of displaying the image after the video

Version
Media3 main branch
More version details
No response
Devices that reproduce the issue
Pixel 6
Devices that do not reproduce the issue
No response
Reproducible in the demo app?
Not tested
Reproduction steps
Create a composition with (1) a video with effects and (2) an image. IE: Code below
    val editedMediaItem = EditedMediaItem.Builder(MediaItem.fromUri(originalVideoUri))
        .setEffects(
            Effects(
                listOf(), // audio effects
                listOf(RgbFilter.createGrayscaleFilter()),
            ),
        )
        .build()

    val introImage = EditedMediaItemSequence(
        EditedMediaItem.Builder(
            MediaItem.fromUri("https://developer.android.com/static/images/shared/android-logo-verticallockup_primary.png"),
        )
            .setDurationUs(1_000_000) // Show the image for 1 seconds in the composition
            .setFrameRate(30)
            .build(),
    )

    val transformedVideoFilePath = createNewVideoFilePath(
        context,
        "some-file-path",
    )

    val transformer = Transformer.Builder(context)
        .addListener(transformerListener)
        .build()

   val composition = Composition.Builder(introImage, EditedMediaItemSequence(editedMediaItem))

    transformer.start(composition
            .experimentalSetForceAudioTrack(true)
            .build(),
        transformedVideoFilePath,
    )

Expected result
Output video showing (1) image and then (2) video with added effects
Actual result
Output video shows video with added effects and image as overlay. Video also stops after duration set for image.
Video shows capturing a screen recording, and then displaying the output video with the code snippet provided above.
https://github.com/androidx/media/assets/5016806/15f1c8d7-c96c-4710-8eba-68b219da4b6d
Media
Video can be recorded through the sample app here:
https://github.com/android/socialite/tree/caren/transformer_demo_with_image_added
by going to Chats > Sheep > Camera > Record video
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

realized this was actually a coding error and that val composition = Composition.Builder(introImage, EditedMediaItemSequence(editedMediaItem)) should be val composition = Composition.Builder( EditedMediaItemSequence(introImage, editedMediaItem))


Day44: 'Binary Search Tree to Greater Sum Tree' solution

Binary Search Tree to Greater Sum Tree
https://leetcode.com/problems/binary-search-tree-to-greater-sum-tree/
Given the root of a Binary Search Tree (BST), convert it to a Greater Tree such that every key of the original BST is changed to the original key plus the sum of all keys greater than the original key in BST.

As a reminder, a binary search tree is a tree that satisfies these constraints:

The left subtree of a node contains only nodes with keys less than the node's key.
The right subtree of a node contains only nodes with keys greater than the node's key.
Both the left and right subtrees must also be binary search trees.

Example 1:
Input: root = [4,1,6,0,2,5,7,null,null,null,3,null,null,null,8]
Output: [30,36,21,36,35,26,15,null,null,null,33,null,null,null,8]
Example 2:
Input: root = [0,null,1]
Output: [1,null,1]


Helm deploy scripts

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:

Created a concise helm chart and removed unused sidecar containers and rbacs (e.g provisioner, snapshotter)
Configurable helm knobs: ipaddress list, nfs plugin container images
Drop all capabilities, set allowPrivilegeEscalation to false for the controller containers
Remove /var/lib/kubelet bidirectional mount from controller pod
change controller and node names to use a "-lb" suffix
Fix attacher rbacs

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

none

merging this one. Will address comments in internal repo


Hugo Server works but Hugo (build) has no css or js

When I use hugo server the site looks fine when I use hugo to build the site on public, the css and js don't load and the site looks like a skeleton.
Tried using a baseUrl of "/" as recommended. Same result.

hey there, it doesn't work like that. When you generate hugo you need to specify the baseURL that you want to publish. then after generate the public folder, you need to upload the file there. then it will work.

Figured out what was going on.
This blog explains it:
https://alison.rbind.io/post/2019-03-04-hugo-troubleshooting/
Sent from my mobile

On Aug 20, 2020, at 12:58 AM, Evan<EMAIL_ADDRESS>wrote:

hey there, it doesn't work like that. When you generate hugo you need to specify the baseURL that you want to publish. then after generate the public folder, you need to upload the file there. then it will work.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Basically what happens is that if the baseURL is set to "/" then the css, js and image (img) files look for files with a double "//" in them. This fails and the site looks undressed. To solve this I re-wrote the calls for css, js and img files on the outputted index.html file in site. This fixes the problem.
When you go to output for deployment rather than testing locally set the baseUrl correctly to the website's site.


Turkish Translation

Turkish translation by mugurcagdas

Hey @mugurcagdas, thanks a lot for this translation! 🎉
That said, this is not the right place...
Please see our documentation (in short: create your own repo, then ask for addition on awesome-yourls).
Thanks again!


Everything inside createGlobalStyle is not minified.

This have bugged me for awhile, but i can see that createGlobalStyle is not minimized in the final outcome.
Compressed with Terser, and babel plugin "babel-plugin-styled-components".

@kitten Same problem here, is there any reason why createGlobalStyle is immune to minification?


Clicking on a sound that's playing should stop it

I keep trying to click on the playing sound to remove it from the current playing list but that doesn't work and I need to remove it using the X button. Please make it so if you click on an already playing sound, it will stop playing it.

I was just about to add this back 😛 thanks for the feedback

Fixed by https://github.com/jenius-apps/ambie/commit/7d4a4d9bd70c824497684fd3dc15138e1e750ed8


[SvgIcon] color prop do not recognize  "success" and "warning"

[ ] The issue is present in the latest release.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
The title is pretty self-explanatory. The SvgIcon component uses the default color when setting color="success" and color="warning" (I expect the same applies to info and custom color palette as well).
Expected Behavior 🤔
It should turn into the corresponding color (typically green and yellow respectively).
Steps to Reproduce 🕹
Steps:
Use any icons from @material-ui/icon@next or SvgIcon from @material-ui/core@next
Context 🔦
I can't think of any reasons why error would work on SvgIcon while warning and success do not.
Your Environment 🌎

`npx @material-ui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @material-ui/envinfo` goes here.

  System:
    OS: Windows 10 10.0.18363
  Binaries:
    Node: 14.17.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.5 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 7.14.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 91.0.4472.77
    Edge: Spartan (44.18362.449.0)
  npmPackages:
    @emotion/react: ^11.4.0 => 11.4.0
    @emotion/styled: ^11.3.0 => 11.3.0
    @material-ui/core: ^5.0.0-alpha.35 => 5.0.0-alpha.35
    @material-ui/data-grid: ^4.0.0-alpha.30 => 4.0.0-alpha.30
    @material-ui/icons: ^5.0.0-alpha.35 => 5.0.0-alpha.35
    @material-ui/private-theming:  5.0.0-alpha.35
    @material-ui/styled-engine:  5.0.0-alpha.34
    @material-ui/styles: ^4.11.4 => 4.11.4
    @material-ui/system:  5.0.0-alpha.35
    @material-ui/types:  6.0.1
    @material-ui/unstyled:  5.0.0-alpha.35
    @material-ui/utils:  5.0.0-alpha.35
    @types/react:  17.0.9
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-next), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-next), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

I thought this issue is obvious enough so I didn't provide a code sandbox link. Anyway here it is:
https://codesandbox.io/s/sparkling-bash-c4sok

Thanks for the repro.
<Search color="success" />
<Search color="warning" />
<Search color="info" />

These colors are not supported by default. Hence the TypeScript error. Why do you think these colors are supported by default?

I thought this issue is obvious enough so I didn't provide a code sandbox link.

We have 10-20 issues a day. Everytime an "obvious" issue is opened we still need to spend 5 minutes to reproduce it. That's 50-100 minutes a day on "obvious" issues. Now some of these aren't obvious and we need to be careful not to spend more time.
This is time that can be saved for maintainers which we explicitly ask for in the issue template. Generally, if a maintainer asks something, following that does help us more than making educated guesses. Even if these are reasonable.

What do you mean? Those colors are standard in the theme created using createTheme.
Taken from https://next.material-ui.com/

That's why I'm confused. Why the prop validation allows error while rejects other status/levels? Why?

Those colors are standard in the theme created using createTheme.

That doesn't mean they're available on every component.

Why the prop validation allows error while rejects other status/levels?

We no longer verify valid color tokens since these might be customized. We rely on static type-checking via TypeScript.

Closing in favor of https://github.com/mui-org/material-ui/issues/26192 which should cover this issue as well.


Maps thumbnails

Maps thumbnails
Added thumbnails to maps page
Examples 📸
We go from here:

To here:

 Expand for Help

Have questions about the review or deployment process? View our contributing docs
Need additional help and want to chat in real time? Join our community Discord

By submitting this pull request, you agree to our code of conduct

.deploy to development

Looks like some great performance improvements! Instead of a MacOS-specific script that must be manually run to generate the thumbnails, maybe it would be possible to use Jimp to generate the thumbnails automatically when the website spins up? That would keep everything automated.

In the future, we can definitely look into automating thumbs generation as the site spins up. I just ever used Jimp and don't know how to integrate it in this sort of automation. 🤔

I might be able to take a look tonight and see if I can make it work. Will leave this PR open in the meantime as a reminder.

Because this was forked in your own repo, I couldn't put the changes directly into it, so I made another branch and did the automation there: #186

GW on automation! 🚀
I suppose we can close this?

Yeah I think so. Good idea on using thumbnails to speed up loading!


fix typo in node update docs

fix typo in node update docs.
- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)
Signed-off-by: allencloud<EMAIL_ADDRESS>

LGTM


Download chart with different options

Is your feature request related to a problem? Please describe.
Download chart downloads chart with current options.
Example: On a website with dark background, the chart uses white text. When downloaded, the text is also white.
Describe the solution you'd like
Being able to specify options for the chart when downloaded (same structure as chart options where values in the download options overwrite the regular options).
Example: Render chart on a website with dark background with white text and specify the text color in the download options so that the downloaded chart uses black text (or different color scheme, or different font sizes, etc.)
Maybe even add allow to use download links (like print version) that use the download options and the regular links, where the chart is downloaded as currently rendered.

You can make the chart's background dark
chart: {
   background: '#000'
}

When you download with this setting, the chart's background color will be preserved.

I meant like two different versions of the chart in general: One that is rendered on the website and another that is specific for download (or a print version so to speak).
If the chart is rendered to be used on a dark website (maybe a dark mode) the downloadable chart should be readable on white background (to be used in PowerPoint for example).
Right now to achieve a different download chart, the chart has to be rerendered with the download options (different color scheme, dark text, light background, etc) and reverted to the original after  download.

Overriding chart options this way and getting back to the original after download will be difficult (and prone to errors).
Sorry, this cannot be integrated into the core chart library.


Guide deletion policies

Andru has brought up a good point: why should members be able to delete their guides?
I always thought of it as a way for members to remove their garbage guides. Like, if they are just playing around and then realize that their test/fake guide is now stuck on their account and in search results when it has no real information. The issue arises though if a guide is being used by others, then deleted by the creator.
We want to empower members to fix their mistakes, while not giving them too much power such that they can revoke/delete useful content.
I propose this policy idea: Guides can only be deleted if no member has added it to their garden or commented on it (when commenting is a thing)
An alternative idea: Guides can only be deleted if they are less than 1 week old

Yeah, I always saw guides as being semi-permanent, so I'm on board with not making them delete-able.
However, if you make them lock in after a week, how do you phrase that so that it doesn't come off as completely totalitarian?

This is related: 596 (basically deleting an account leaves guides, which is a bug at the moment - though fixed on staging).

We could also have a 'request to be removed' button on every Guide. That's going to have to exist anyways for members to flag inappropriate Guides. Then a community moderator can approve or deny the request based on if it really is an empty/test/inappropriate guide or not.

@roryaronson is this discussion still pending?

Yes, considering that empty guides will probably remain as drafts now we can probably close this discussion. I think we'll need to revisit it when we get to a point where we need a "Flag as Inappropriate" button and policies for when to remove content, but that's probably a ways off.


update plugin to be used with 4.x

Fix #23 #22
This is a huge refactor it still has a few places where it needs polishing but at least it is in a usable state

UP


Refactor some lmesreconcile methods

Refactors createPod other related methods to static functions because I need to create PodSet for Kueue integration using those functions

/cc @yhwang


Add Google code for more unittests

Some additional changes of the Tesseract code base make integration of the new tests easier.
As an example, bitvector_test and some more tests were now added to the Tesseract test set.

Welcome addition!!
Thank you @jbreiden  and @stweil .

Some of the new test files which are currently not activated use the Abseil API (https://abseil.io/, https://github.com/abseil/abseil-cpp). We have to decide whether we want to replace those calls, add a minimal implementation to Tesseract or require a full Abseil installation. These tests are affected:
baseapi_test
baseapi_thread_test
fileio_test
imagedata_test
lang_model_test
mastertrainer_test
pango_font_info_test
paragraphs_test
stringrenderer_test
unicharcompress_test

These tests neither depend on the Abseil API nor on unavailable TIFF images, so might be the next candidates to get fixed and added to our test set:
commandlineflags_test
dawg_test
heap_test
indexmapbidi_test
intfeaturemap_test
ligature_table_test
linlsq_test
lstm_recode_test
lstm_squashed_test
networkio_test
normstrngs_test
nthitem_test
params_model_test
qrsequence_test
recodebeam_test
rect_test
scanutils_test
shapetable_test
stats_test
stridemap_test
tablefind_test
tablerecog_test
tabvector_test
tatweel_test
tfile_test
unicharset_test
unichar_test
validate_grapheme_test
validate_indic_test
validate_khmer_test
validate_myanmar_test
validator_test

a full Abseil installation (maybe as a Git submodule, about 10 MB).

Abseil is an open-source collection of C++ code (compliant to C++11) designed to augment the C++ standard library.

We are already using Googletest and test (testdata) as submodules that are required only for testing, so I think it will be OK to add Abseil as a submodule too.

qrsequence_test – needs unknown data type CycleTimer

It is mentioned here:
https://github.com/abseil/abseil-cpp/blob/87a4c07856e7dc69958019d47b2f02ae47746ec0/absl/base/internal/unscaledcycleclock.h
Also check
https://www.google.com/search?q="CycleTimer"+"google"+"apache"+"github"

Try to extract a eng.traindata file that contains the data for the legacy engine.
eng.params_model should be there.
https://github.com/tesseract-ocr/tesseract/blob/5fdaa479da2c/doc/combine_tessdata.1.asc

needs unknown data type Array2D (class?)

Try to replace with GENERIC_2D_ARRAY

https://www.google.com/search?q="StringPrintf"+"google"+"apache"

https://www.google.com/search?q="subprocess.h"+"google"+"apache"

qrsequence_test – needs unknown data type CycleTimer

It is mentioned here:
https://github.com/abseil/abseil-cpp/blob/87a4c07856e7dc69958019d47b2f02ae47746ec0/absl/base/internal/unscaledcycleclock.h
Also check
https://www.google.com/search?q="CycleTimer"+"google"+"apache"+"github"

params_model_test – needs file eng.params_model

Try to extract a eng.traindata file that contains the data for the legacy engine.
eng.params_model should be there.
https://github.com/tesseract-ocr/tesseract/blob/5fdaa479da2c/doc/combine_tessdata.1.asc

needs unknown data type Array2D (class?)

Try to replace with GENERIC_2D_ARRAY

shapetable_test – needs unknown function StringPrintf

https://www.google.com/search?q="StringPrintf"+"google"+"apache"

tatweel_test – needs util/utf8/public/unicodetext.h

https://github.com/tensorflow/models/search?q="unicodetext.h"

dawg_test – needs util/process/subprocess.h

https://www.google.com/search?q="subprocess.h"+"google"+"apache"

I'm just preparing the baseapi_test. It is the first test case which uses the Abseil API, and it also needs CycleTimer and LOG(). I am still struggling with the integration of Abseil into the build process, but hopefully will have finished that soon. Then adding more tests with similar requirements should be possible.

absl::StrFormat is the replacement for StringPrintf.  Tests should work with the LOG lines removed, but the logging library is probably https://github.com/google/glog


import error while using BranchPythonOperator in airflow 1.1.1

Im using 1.1.1 version of airflow on centos and the import stmt for the branchpythonoperator seems to be failing. I can import the PythonOperator. What am i missing ? Do i need to upgrade to a newer version to start seeing the BranchPythonOperator in the airflow.operators package ?

Hi @smnaveen, yes you would need to upgrade to at least 1.2.0, but would recommend you update to latest.
pip install --upgrade airflow with the appropriate extras should be fine. Let me know if you need more info.

Thank you @artwr. I will upgrade as advised and try it out.

Please reopen if it's still a problem in the latest version


Add GCP support

This pull request includes several changes to the Makefile, Go source files, and documentation, primarily focusing on adding new functionality, updating configurations, and improving documentation. The most important changes are grouped by theme below:
Makefile Enhancements:

Added a new image-clean target to remove Docker images.
Updated the clean target to remove additional binaries specified by CONTRIB_BINARIES.

Go Source File Updates:

Introduced the ApplyPriorityLevelConfiguration function in cmd/kperf/commands/utils/helper.go to apply Kubernetes PriorityLevelConfiguration using kubectl.
Updated cmd/kperf/commands/virtualcluster/nodepool.go to call the new ApplyPriorityLevelConfiguration function.

Documentation Improvements:

Added instructions for obtaining KubeConfig for Azure, AWS, and GCP in docs/getting-started.md.
Updated runner group specifications and example commands in docs/getting-started.md to reflect current configurations and image versions. [1] [2]
Revised benchmark scenario descriptions and options in docs/runkperf.md to reflect updated configurations and image versions. [1] [2] [3]

Helm Chart Updates:

Added labels and annotations to Helm templates for FlowSchema in manifests/runnergroup/server/templates/flowcontrol.yaml and manifests/virtualcluster/nodecontrollers/templates/flowcontrol.yaml. [1] [2]

tested
 runkperf -v 3 bench   --kubeconfig $HOME/.kube/config   --runner-image ghcr.io/azure/kperf:
0.1.8   node10_job1_pod100 --total 1000
I1221 21:54:01.124231 2743920 utils.go:287] Fetching apiserver's cores
I1221 21:54:01.124286 2743920 utils.go:388] [CMD] /usr/local/bin/kubectl --kubeconfig /home/ansonqian/.kube/config cluster-info
E1221 21:54:01.301342 2743920 utils.go:322] "failed to get cores" err="failed to get metrics for ip <IP_ADDRESS>: failed to create a new mount namespace: operation not permitted" ip="<IP_ADDRESS>"
I1221 21:54:01.301763 2743920 utils.go:152] "Load Profile" config=<
        count: 1
        loadProfile:
          version: 1
          description: node10-job1-pod100
          spec:
            rate: 10
            total: 1000
            conns: 10
            client: 10
            contentType: json
            disableHTTP2: false
            maxRetries: 0
            requests:
            - shares: 1000
              staleList:
                group: ""
                version: v1
                resource: pods
                namespace: ""
                limit: 0
                seletor: ""
                fieldSelector: ""
            - shares: 100
              quorumList:
                group: ""
                version: v1
                resource: pods
                namespace: ""
                limit: 1000
                seletor: ""
                fieldSelector: ""
            - shares: 100
              quorumList:
                group: ""
                version: v1
                resource: events
                namespace: ""
                limit: 1000
                seletor: ""
                fieldSelector: ""
        nodeAffinity:
          node.kubernetes.io/instance-type:
          - Standard_D16s_v3
          - m4.4xlarge
          - n1-standard-16
 >
I1221 21:54:01.302081 2743920 utils.go:97] "Deploying virtual nodepool" name="node10job1pod100"
I1221 21:54:01.302098 2743920 utils.go:115] "Trying to delete nodepool if necessary" name="node10job1pod100"
I1221 21:54:01.302151 2743920 utils.go:388] [CMD] /usr/local/bin/kperf vc nodepool --kubeconfig=/home/ansonqian/.kube/config delete node10job1pod100
I1221 21:54:01.478761 2743920 utils.go:388] [CMD] /usr/local/bin/kperf vc nodepool --kubeconfig=/home/ansonqian/.kube/config add node10job1pod100 --nodes=100 --cpu=32 --memory=96 --max-pods=110 --affinity=node.kubernetes.io/instance-type=Standard_D8s_v3,m4.2xlarge,n1-standard-8
I1221 21:54:17.626476 2743920 utils.go:236] "Deploying runner group" config="/tmp/temp1227027314"
I1221 21:54:17.626509 2743920 utils.go:240] Deleting existing runner group
I1221 21:54:17.626567 2743920 utils.go:46] Repeat to create job with 3k pods
I1221 21:54:17.626606 2743920 utils.go:388] [CMD] /usr/local/bin/kperf rg --kubeconfig=/home/ansonqian/.kube/config delete
I1221 21:54:17.626730 2743920 utils.go:62] Creating namespace job1pod100
I1221 21:54:17.626827 2743920 utils.go:388] [CMD] /usr/local/bin/kubectl --kubeconfig /home/ansonqian/.kube/config create namespace job1pod100
I1221 21:54:17.793976 2743920 utils.go:388] [CMD] /usr/local/bin/kperf rg --kubeconfig=/home/ansonqian/.kube/config run --runnergroup=file:///tmp/temp1227027314 --runner-image=ghcr.io/azure/kperf:0.1.8 --affinity=node.kubernetes.io/instance-type=Standard_D16s_v3,m4.4xlarge,n1-standard-16 --runner-flowcontrol=workload-low:1000
I1221 21:54:21.173162 2743920 utils.go:251] Waiting runner group
I1221 21:54:21.173270 2743920 utils.go:388] [CMD] /usr/local/bin/kperf rg --kubeconfig=/home/ansonqian/.kube/config result
I1221 21:54:22.784115 2743920 utils.go:388] [CMD] /usr/local/bin/kubectl --kubeconfig /home/ansonqian/.kube/config -n job1pod100 apply -f /tmp/temp821955404
I1221 21:54:23.083833 2743920 utils.go:388] [CMD] /usr/local/bin/kubectl --kubeconfig /home/ansonqian/.kube/config -n job1pod100 wait --for=condition=complete --timeout=15m job/batchjobs
E1221 21:55:21.228889 2743920 utils.go:267] "failed to fetch runner group's result" err=<
        failed to invoke /usr/local/bin/kperf rg --kubeconfig=/home/ansonqian/.kube/config result:
         (output: ): signal: killed
 >
I1221 21:55:21.229000 2743920 utils.go:388] [CMD] /usr/local/bin/kperf rg --kubeconfig=/home/ansonqian/.kube/config result
I1221 21:56:03.200705 2743920 utils.go:270] "Runner group's result" data=<
        {
          "total": 1000,
          "duration": "1m40.016519689s",
          "errorStats": {
            "unknownErrors": null,
            "netErrors": null,
            "responseCodes": null,
            "http2Errors": {}
          },
          "totalReceivedBytes":<PHONE_NUMBER>,
          "percentileLatencies": [
            [
              0,
              0.024603586
            ],
            [
              0.5,
              0.098262593
            ],
            [
              0.9,
              0.139573687
            ],
            [
              0.95,
              0.162021473
            ],
            [
              0.99,
              0.210638021
            ],
            [
              1,
              0.433002973
            ]
          ],
          "percentileLatenciesByURL": {
            "https://<IP_ADDRESS>:443/api/v1/events?limit=1000\u0026timeout=1m0s": [
              [
                0,
                0.024603586
              ],
              [
                0.5,
                0.030703546
              ],
              [
                0.9,
                0.040356291
              ],
              [
                0.95,
                0.048135445
              ],
              [
                0.99,
                0.074674283
              ],
              [
                1,
                0.074674283
              ]
            ],
            "https://<IP_ADDRESS>:443/api/v1/pods?limit=1000\u0026timeout=1m0s": [
              [
                0,
                0.092264801
              ],
              [
                0.5,
                0.153941227
              ],
              [
                0.9,
                0.196386019
              ],
              [
                0.95,
                0.271004627
              ],
              [
                0.99,
                0.329401482
              ],
              [
                1,
                0.329401482
              ]
            ],
            "https://<IP_ADDRESS>:443/api/v1/pods?resourceVersion=0\u0026timeout=1m0s": [
              [
                0,
                0.05193836
              ],
              [
                0.5,
                0.098406446
              ],
              [
                0.9,
                0.124743188
              ],
              [
                0.95,
                0.136332168
              ],
              [
                0.99,
                0.165001933
              ],
              [
                1,
                0.433002973
              ]
            ]
          }
        }
 >
I1221 21:56:03.201104 2743920 utils.go:277] Deleting runner group
I1221 21:56:03.201175 2743920 utils.go:388] [CMD] /usr/local/bin/kperf rg --kubeconfig=/home/ansonqian/.kube/config delete
E1221 21:56:03.814836 2743920 utils.go:95] "failed to wait" err=<
        failed to invoke /usr/local/bin/kubectl --kubeconfig /home/ansonqian/.kube/config -n job1pod100 wait --for=condition=complete --timeout=15m job/batchjobs:
         (output: ): signal: killed
 > job="workload/100pod.job.yaml"
I1221 21:56:03.814928 2743920 utils.go:388] [CMD] /usr/local/bin/kubectl --kubeconfig /home/ansonqian/.kube/config -n job1pod100 delete -f /tmp/temp821955404
E1221 21:56:03.814946 2743920 utils.go:100] "failed to delete" err=<
        failed to invoke /usr/local/bin/kubectl --kubeconfig /home/ansonqian/.kube/config -n job1pod100 delete -f /tmp/temp821955404:
         (output: ): context canceled
 > job="workload/100pod.job.yaml"
I1221 21:56:08.819143 2743920 utils.go:80] Stop creating job
I1221 21:56:08.819200 2743920 utils.go:69] Cleanup namespace job1pod100
I1221 21:56:08.819273 2743920 utils.go:388] [CMD] /usr/local/bin/kubectl --kubeconfig /home/ansonqian/.kube/config delete namespace job1pod100
I1221 21:56:14.520149 2743920 utils.go:388] [CMD] /usr/local/bin/kperf vc nodepool --kubeconfig=/home/ansonqian/.kube/config delete node10job1pod100
I1221 21:56:15.985835 2743920 utils.go:287] Fetching apiserver's cores
I1221 21:56:15.985910 2743920 utils.go:388] [CMD] /usr/local/bin/kubectl --kubeconfig /home/ansonqian/.kube/config cluster-info
E1221 21:56:16.147100 2743920 utils.go:322] "failed to get cores" err="failed to get metrics for ip <IP_ADDRESS>: failed to create a new mount namespace: operation not permitted" ip="<IP_ADDRESS>"
{
  "description": "\nEnvironment: 100 virtual nodes managed by kwok-controller,\nWorkload: Deploy 1 job with 3,000 pods repeatedly. The parallelism is 100. The interval is 5s",
  "loadSpec": {
    "count": 1,
    "loadProfile": {
      "version": 1,
      "description": "node10-job1-pod100",
      "spec": {
        "rate": 10,
        "total": 1000,
        "conns": 10,
        "client": 10,
        "contentType": "json",
        "disableHTTP2": false,
        "maxRetries": 0,
        "Requests": [
          {
            "shares": 1000,
            "staleList": {
              "group": "",
              "version": "v1",
              "resource": "pods",
              "namespace": "",
              "limit": 0,
              "seletor": "",
              "fieldSelector": ""
            }
          },
          {
            "shares": 100,
            "quorumList": {
              "group": "",
              "version": "v1",
              "resource": "pods",
              "namespace": "",
              "limit": 1000,
              "seletor": "",
              "fieldSelector": ""
            }
          },
          {
            "shares": 100,
            "quorumList": {
              "group": "",
              "version": "v1",
              "resource": "events",
              "namespace": "",
              "limit": 1000,
              "seletor": "",
              "fieldSelector": ""
            }
          }
        ]
      }
    },
    "nodeAffinity": {
      "node.kubernetes.io/instance-type": [
        "Standard_D16s_v3",
        "m4.4xlarge",
        "n1-standard-16"
      ]
    }
  },
  "result": {
    "total": 1000,
    "duration": "1m40.016519689s",
    "errorStats": {
      "unknownErrors": null,
      "netErrors": null,
      "responseCodes": null,
      "http2Errors": {}
    },
    "totalReceivedBytes":<PHONE_NUMBER>,
    "percentileLatencies": [
      [
        0,
        0.024603586
      ],
      [
        0.5,
        0.098262593
      ],
      [
        0.9,
        0.139573687
      ],
      [
        0.95,
        0.162021473
      ],
      [
        0.99,
        0.210638021
      ],
      [
        1,
        0.433002973
      ]
    ],
    "percentileLatenciesByURL": {
      "https://<IP_ADDRESS>:443/api/v1/events?limit=1000\u0026timeout=1m0s": [
        [
          0,
          0.024603586
        ],
        [
          0.5,
          0.030703546
        ],
        [
          0.9,
          0.040356291
        ],
        [
          0.95,
          0.048135445
        ],
        [
          0.99,
          0.074674283
        ],
        [
          1,
          0.074674283
        ]
      ],
      "https://<IP_ADDRESS>:443/api/v1/pods?limit=1000\u0026timeout=1m0s": [
        [
          0,
          0.092264801
        ],
        [
          0.5,
          0.153941227
        ],
        [
          0.9,
          0.196386019
        ],
        [
          0.95,
          0.271004627
        ],
        [
          0.99,
          0.329401482
        ],
        [
          1,
          0.329401482
        ]
      ],
      "https://<IP_ADDRESS>:443/api/v1/pods?resourceVersion=0\u0026timeout=1m0s": [
        [
          0,
          0.05193836
        ],
        [
          0.5,
          0.098406446
        ],
        [
          0.9,
          0.124743188
        ],
        [
          0.95,
          0.136332168
        ],
        [
          0.99,
          0.165001933
        ],
        [
          1,
          0.433002973
        ]
      ]
    }
  },
  "info": {
    "apiserver": {
      "cores": {
        "after": {},
        "before": {}
      }
    }
  }
}```


refactoring code with Assign Multiple Targets

refactoring code with Assign Multiple Targets which is more pythonic, concise, readable and efficient;how do think this change which has practical value？

It a perfectly fine change. That code is not actually executed by anything at the moment, so its not exactly a high priority. Any particular reason for tackling this item?

I think it's a bot, definitely not a hooman. Another pull by them. I like the original more because I find it more readable.

Gonna close this. The PR itself is pointless and the originating account seems to be a bot now.


[5.1] Arr::expand method

If we have an Arr::dot method which flattens array - why don't we have a method which does the opposite?

Fixed

Been shot down a couple times before #4292.


Tie rendering in ScrollScoreLayout

Ties often render improperly in ScrollScoreLayout because TieFactory assumes a page layout, and therefore renders ties as if there were multiple systems. This creates lots of super long ties that cover up the rest of the sheet music. This PR fixes this behavior.

Also fixed octave shift rendering. Same issue

Thanks!


warning: regexp escape sequence `"' is not a known regexp operator

$ gawk -V:
 GNU Awk 4.2.62, API: 2.0 (GNU MPFR 3.1.4, GNU MP 6.1.0)

$ trans -V:
 gawk: /dev/fd/63:176: warning: regexp escape sequence `\"' is not a known regexp operator
 Translate Shell       <IP_ADDRESS>-git:84eaf66

 platform              Linux
 gawk (GNU Awk)        4.2.62
 fribidi (GNU FriBidi) [NOT INSTALLED]
 audio player          mpv
 terminal pager        less
 terminal type         tmux-256color
 user locale           en_US (English)
 home language         en
 source language       auto
 target language       en
 translation engine    google
 proxy                 [NONE]
 user-agent            Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/602.1 (KHTML, like Gecko) Version/8.0 Safari/602.1 Epiphany/3.18.2
 theme                 default
 init file             [NONE]

 Report bugs to:       https://github.com/soimort/translate-shell/issues

When I run any $ trans command, such as:
$ trans -V

Or:
$ trans 'Saluton, Mondo!'

The output begins with the following warning from gawk:
gawk: /dev/fd/63:176: warning: regexp escape sequence `\"' is not a known regexp operator

It seems to be due to this commit in the gawk codebase.

Thanks for the report.
<IP_ADDRESS> was a quite outdated version of translate-shell; a lot of code might have changed since then. Can you please confirm if the warning persists with the newest version <IP_ADDRESS>?

Yes, I think the issue is still present in <IP_ADDRESS>. I've just re-installed the latest version I could find from github, and the output of trans -V still begins with a warning:
$ trans -V
gawk: /dev/fd/63:177: warning: regexp escape sequence `\"' is not a known regexp operator
Translate Shell       <IP_ADDRESS>-git:b13772d

platform              Linux
gawk (GNU Awk)        4.2.62
fribidi (GNU FriBidi) 0.19.7
audio player          mpv --no-config
terminal pager        less
terminal type         tmux-256color
user locale           en_US.UTF-8 (English)
home language         en
source language       auto
target language       en
translation engine    google
proxy                 [NONE]
user-agent            Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/602.1 (KHTML, like Gecko) Version/8.0 Safari/602.1 Epiphany/3.18.2
theme                 default
init file             [NONE]

Report bugs to:       https://github.com/soimort/translate-shell/issues

Thanks. Now I can reproduce it under the latest git build of gawk, too.

Now that gawk version 5.0.0 is out the same problem happens again, as gawk change the regex routines now using GNULIB's ones.
https://lists.gnu.org/archive/html/info-gnu/2019-04/msg00002.html

Happens on macOS/Homebrew too with version <IP_ADDRESS> and gawk 5.0.0.

same problem

Last version fixed it.
https://github.com/soimort/translate-shell/releases/tag/v<IP_ADDRESS>

i get this error with trans version <IP_ADDRESS> and gawk version 5.0.1


Invstigate starting node from only location/background task triggers

This might only be an issue in the simulator, but there are times when the app is in the background or suspended, and receives a location update. The app status was already 'background', then the trigger occurs, and the app state remains background, so no running of the node is triggered.
This doesn't really follow Apple's documented app lifecycle (figure 1 here where the app can't go from suspended to background), but we should investigate further.

Closing in favor of new information/reports and the fact this will change with the creation of the SDK


Provide sample code for overriding endpoints in GAPIC clients

Currently we have sample code for overriding credentials, e.g.
https://github.com/GoogleCloudPlatform/google-cloud-java/blob/0e4ed46f8dfdc48bac0dba00f4bb50ed7b39b42d/google-cloud-pubsub/src/main/java/com/google/cloud/pubsub/v1/TopicAdminClient.java#L97
This task is to show how to override the endpoint, e.g. by calling setEndpoint:
https://github.com/googleapis/gax-java/blob/6b7f587ca283a7126be9b1e0f49821808dfc5380/gax-grpc/src/main/java/com/google/api/gax/grpc/InstantiatingChannelProvider.java#L255

So... this is a change to the templates in toolkit?
Is changing the sample you linked to from
 *
 * TopicAdminSettings topicAdminSettings =
 *     TopicAdminSettings.defaultBuilder()
 *         .setCredentialsProvider(FixedCredentialsProvider.create(myCredentials))
 *         .build();
 * TopicAdminClient topicAdminClient =
 *     TopicAdminClient.create(topicAdminSettings);
 *

to
 *
 * TopicAdminSettings topicAdminSettings =
 *     TopicAdminSettings.defaultBuilder()
 *         .setCredentialsProvider(FixedCredentialsProvider.create(myCredentials))
 *         .setEndpoint("localhost:8000")
 *         .build();
 * TopicAdminClient topicAdminClient =
 *     TopicAdminClient.create(topicAdminSettings);
 *

enough to resolve this?

That won't work, because the Builder for https://github.com/googleapis/gax-java/blob/master/gax-grpc/src/main/java/com/google/api/gax/grpc/GrpcTransportProvider.java doesn't have a setEndpoint method.
And anyway, since setting credentials and setting endpoints are somewhat independent, I might like to have both as self-contained samples.

This can't be closed until the clients have been regenerated.


feat(stack): make stack created from app template editable EE-1941

Closes https://github.com/portainer/portainer/issues/5748


Add spirv-cfg to dump a GraphViz graph of the CFG

This is experimental, and has not tests.
It's been used to debug validation of structured control flow.

Rebased

Logic LGTM, just some nits.

Rebased, and responded to feedback in 75b4477

Sample output:

+2

Rebased.
+2


internal keyboard stops working completely after "quit karabiner-elements"

the problem is described in the subject line
so, I quit karabiner elements and internal keyboard is dead.
if karabiner-elements are launched again (with the help of mouse), internal keyboard is functional again.
Karabiner-Elements: 12.5.3
OS: High Sierra 10.13.6 (17G6030)
Hardware: MacBook Pro (Retina, 15-inch, Mid 2014)

weird enough, but after full system reboot quitting karabiner elements does not kill the keyboard.
I'll reopen if I manage to reproduce it again.


Add execute

Will allow socketIO to execute all possible RCON commands from remote

I don't get the point of this pull request, the child class will inherit the super class' execute method so why redefine it to what it's already defined as?

Well if you remember I did try to execute it (rcon.execute) however it was not working cz the super class... Therefor I must do this, as you also confirmed it would work like this.
I will double check this to be 200% sure 👍

Well if you remember I did try to execute it (rcon.execute) however it was not working cz the super class... Therefor I must do this, as you also confirmed it would work like this.
I will double check this to be 200% sure 👍

Converting to draft until I'm able to test the stuff we spoke via PM

Going to close this for now.


[WFCORE-5253] Clean up remoting subsystem OSH use of 'operation'

https://issues.redhat.com/browse/WFCORE-5253

@bstansberry thanks


Mutation coverage improvements

Closes: #1020
Installed roave-infection-static-analysis-plugin that checks escaped mutants against static-analysis tools. If they would not pass SA, it kills them. Currently, supports only Psalm and not PHPStan

Notes on changing the infection config:

FunctionCallRemoval - ignored trigger_deprecation. I don't think we want/need to test for that. It is just noise.
global-ignoreSourceCodeByRegex - ignored console output. This would be covered anyways if we ever do #806. Basically, I am claiming there is already another issue opened for that.
CatchBlockRemoval - It will pass infection, and it will even pass roave-infection-static-analysis-plugin that integrates with Psalm, but ut will not pass PHPStan, which will complain about unchecked exception without @throws annotation. So there is another test for that.

Awesome! Thanks


Restore gating for Pre Update 2 for NTVS 1.2

Adds back in gating removed by #1005. This change instead uses the PTVS gating logic.
Tested on RTM machine and on saw gating, plus tested on update 2 machine and saw that it was not gated.
closes #1003

Let's also include a link to http://go.microsoft.com/fwlink/?LinkID=808380

:+1:


Add support for custom server RCON commands

In this PR some changes have been proposed so users, via a new field in the extension settings, can specify custom commands for them to be executed in the server after a successful compilation or upload. This is exactly what's been happening already, but now without the fixed sm plugins refresh.

I would also like to point out that I'm interested in implementing a macro that the extension would replace when executing the command, so the user could do sm plugins reload {plugin} or other variable preffix/suffix, and the current plugin being compiled could be targeted once copied to the server.
The question is, out of ignorance, does the extension have the ability to compile multiple plugins at the same time, for instance when working on a big project? Because if it could, then it would not be as easy as that, and I'd have to think of another way for the user to be able to refresh several plugins at once (the ones he's working in). Maybe I didn't make myself very clear with the last part but I hope my point is not too blurry.

does the extension have the ability to compile multiple plugins at the same time

No, and it will probably never allow that as https://github.com/Sarrus1/sourcepawn-vscode/issues/217 is still stale because of VSCode
You can use this snippet to get the path of the main file, therefore the plugin's name.

Closing to re-open from a feature branch.


chore: suppress unused variable linter false positives

and add a reference to #1214 so that it will be cleaned up when that gets fixed.

I'd already addressed this in #1381. Thanks for the additional references though!


[Refactoring] Import at root to enable vmap monkey-patching

Description

monkey-patches a few more functorch functions
implements tests for nn.Module vmap with tensordicts

Following suggestions from @zou3519 we remove and add a batch dimension when vmapping over a tensordict.
Some commended code shows attempts to register TensorDIct in pytree but that failed + it requires to decompose and recompose the tensordict multiple times, while doing complex operations to keep track of the modified batch size. Besides the fact that it did not work, it also seemed suboptimal from a compute time perspective.

cc @nicolas-dufour
allows for this kind of behaviour
        class MyModule(torch.nn.Module):
            def forward(self, tensordict, tensor):
                a = tensordict["a"]
                return (
                    TensorDict(
                        {"a": a}, tensordict.batch_size, device=tensordict.device
                    ),
                    tensor,
                )

        tensor = torch.randn(3).expand(4, 3)
        tensordict = TensorDict({"a": torch.randn(3, 1)}, [3])
        out = vmap(MyModule(), (None, 0))(tensordict, tensor)

        assert out[0].shape == torch.Size([4, 3])
        assert out[1].shape == torch.Size([4, 3])
        assert out[0]["a"].shape == torch.Size([4, 3, 1])

allowing for nn.Module with arbitrary TensorDict inputs


Monero dev meeting summary

Dev meeting summary by gingeropolous, 20160131
Use of dev branch
First thing discussed is the dev branch. Contributors have fell back into the habbit of merging stuff to master. All hacking should be done onto dev branch.
Apparently this is because of bugs. A new point release to fix the v1/v2 stuck transaction bugs is coming.
The thing holding up general move to development branch is that CZMQ/0MQ hasn't been bundled in source, so its a pain to compile. Everyone agrees to bundle CZMQ/0MQ into dev branch. Fluffypony will get it in the source tree.
There is more talk about minor tweaks that should or shouldn't go into the upcoming point release.
Style guide
In kovri they've been working with a style guide, and fluffypony suggests implementing one in Monero. Basically, this is the way the code is formatted (human formatted).
Its agreed to use the new style with new code, and restyle-as-you-go with old code. Restyling for the sake of restyling is though to be unnecessary. Key points:

we assume from this point on the code is indented like the majority of the code so far - 2 spaces, not tabs, not 4 spaces.

push harder on "code should go in a .cpp not .h"

Collective Code Construction Contract (c4)
As used in 0MQ as described here
The idea is to merge every PR as long as it doesn't break the build. Aim is to avoid PR-hell where everyone comments on a PR for days and weeks and it never gets merged, because its never "perfect." Merge-everything is done so that contributors establish a history, and they can be banned if they are repeat offenders.
In general, the idea is agreed upon. However, there is concern and uncertaintly regarding the roles of the various branches and how the contract applies to which branches. The conversation bleeds into ringCT implementation and timeframe and how the c4 would affect this process. The fact that 0MQ-dev branch is currently unusable is brought up regarding how the c4 would function. It is noted that 0MQ was pushed to dev because the original contributor couldn't continue working on it. Ultimately the 0MQ situation will not happen again.
Ultimately, this issue will be brought up at the next meeting.
Keep hacking around things or dump stuff for things that are easier
This is a lot of meta-code stuff. The thing that was salient to me was the doxygen-compatible approach, which would allow newcomers to the code to become acquainted more efficiently.
Thanks!
Fluffypony extends his gratitude to all contributors, and everyone makes their way to the reception hall for bacon wrapped shrimp, pigs in blankets, and all other sorts of tiny foods that hover on discs.

Done, credited you as the post author: https://getmonero.org/2016/01/31/overview-and-logs-for-the-dev-meeting-held-on-2016-01-31.html


key bindings increaseFontSize and decreaseFontSize are REMOVED from profile when Terminal is restarted

Environment
Windows build number: 10.0.18362.175
Windows Terminal version (if applicable): 0.2.1715.0

Steps to reproduce
Add keybindings for increaseFontSize and decreaseFontSize in the profile settings.
Close and restart Windows Terminal.
Expected behavior
For keybindings to be in the profile, as pre-configured.
Actual behavior
Profile file is reformatted and the bindings for increaseFontSize and decreaseFontSize are lost.

Nonexistent bind commands are deleted, by design. We have workitems out to improve messaging there.

Unfortunately, you can’t just guess at binding names. Sorry about that.


🚧 IMN-812 - Fixing internal token gen claims

Closes IMN-812
WIP 🚧

Closing this: In Scala, the UID is not passed to the token claims but to the context for logging purposes, we agree that we don't need to migrate this feature. See details in JIRA task discussion and here: https://github.com/pagopa/interop-be-backend-for-frontend/blob/b37f24cc27d7690cc9ece8dd215a16437de2f0cf/src/main/scala/it/pagopa/interop/backendforfrontend/api/impl/package.scala#L329-L330


Migrate rm command to cobra

part of #23211
Signed-off-by: Zhang Wei<EMAIL_ADDRESS>

@WeiZhang555 need rebase :(

@LK4D4  Rebased :sunglasses:

LGTM 🐮
but needs a rebase 😝

Rebased, again.

14:44:15 FAIL: docker_cli_rename_test.go:69: DockerSuite.TestRenameInvalidName
14:44:15
14:44:15 docker_cli_rename_test.go:78:
14:44:15     c.Assert(out, checker.Contains, "\"docker rename\" requires exactly 2 argument(s).", check.Commentf("%v", err))
14:44:15 ... obtained string = "Error: Neither old nor new names may be empty\n"
14:44:15 ... substring string = "\"docker rename\" requires exactly 2 argument(s)."
14:44:15 ... exit status 1: Error: Neither old nor new names may be empty
14:44:15

The TestRenameInvalidName should be fixed now, so triggered a rebuild for gccgo and windowsTP5

gccgo still fails, I guess it's not related to this.

darn; guess another job was running, but not visible here, and me triggering it cancelled that job :cry:

LGTM


fix default Bounding Boxes dimensions for DIA data

MS2 bounding box have DDA MS2 defaut BB size instead of MS1 BB size

fix by this commit


Crashing on app launch in iOS 14

Pulse version: 3.5.3
Hello there.
First of all, thank you for creating a very development friendly tool.
In this situation, we weren't calling any Pulse-related methods, and as soon as we launched the app, it crashed with the message below.
App run in the emulator, skipping the fork check.
dyld: lazy symbol binding failed: Symbol not found: _$s10Foundation16AttributedStringVMa
  Referenced from: /Users/go/Library/Developer/CoreSimulator/Devices/F59BC6C7-3B29-43E6-8479-ED10B7D16314/data/Containers/Bundle/Application/C28F1662-B139-4E22-AE75-5192F9F27127/GangnamSister.app/GangnamSister
  Expected in: /usr/lib/swift/libswiftFoundation.dylib

dyld: Symbol not found: _$s10Foundation16AttributedStringVMa
  Referenced from: /Users/go/Library/Developer/CoreSimulator/Devices/F59BC6C7-3B29-43E6-8479-ED10B7D16314/data/Containers/Bundle/Application/C28F1662-B139-4E22-AE75-5192F9F27127/GangnamSister.app/GangnamSister
  Expected in: /usr/lib/swift/libswiftFoundation.dylib

CoreSimulator 857.7 - Device: iPhone 8 Plus (F59BC6C7-3B29-43E6-8479-ED10B7D16314) - Runtime: iOS 14.2 (18B79) - DeviceType: iPhone 8 Plus

I had the same issue with iOS 14.5.
This works fine on iOS 15 and 16.
I downgraded Pulse to 3.4.3 and the app works fine on iOS 14.

duplicate as #171

Thanks for the report. I'll try to look into it soon, but PRs are also welcome.


AMM: Implement the indexer

Indexer Guideline
The AMM consists of two contracts therefore we can separate the indexer into two modules.
Each index should contain functions in the format  index_<event_name>.
E.g.
#[indexer(manifest = "<path/to/file/amm.manifest.yaml>")]
mod amm_index {
    fn index_register_pool_event(event: RegisterPoolEvent, block: BlockData) { ... }
}

#[indexer(manifest = "<path/to/file/amm-exchange.manifest.yaml>")]
mod amm_exchange_index {
    fn index_add_liquidity_event(event: AddLiquidityEvent, block: BlockData) { ... }

   // more events
}

Each type in graphql should have a required field.
Certain event fields would benefit from the @indexed directive.

AMM

RegisterPoolEvent

asset_pair
pool

Exchange

AddLiquidityEvent

added_assets

DepositEvent

deposited_asset

RemoveLiquidityEvent

removed_reserve

SwapEvent

input
output

WithdrawEvent

withdrawn_asset

From the BlockData we should retrieve the block height and add it to each event so that we have a measure of time for when the event took place.

Closing in favor of https://github.com/FuelLabs/sway-applications/issues/605


Convert docs to hub

Need to convert docs / code samples to hub and move there.  During transition we will keep docs in this repository.

doh this was opened in wrong repo


Apple identified issues after updating to 7.4.1

Description
After submitting a build to the app store, Apple let us know about an issue relating to the New Relic framework
Dear Developer,
We identified one or more issues with a recent delivery for your app. Please correct the following issues, then upload again.ITMS-90338: Non-public API usage - The app references non-public selectors in Frameworks/NewRelic.framework/NewRelic: _didReceiveData:. If method names in your source code match the private Apple APIs listed above, altering your method names will help prevent this app from being flagged in future submissions. In addition, note that one or more of the above APIs may be located in a static library that was included with your app. If so, they must be removed. For further information, visit the Technical Support Information at http://developer.apple.com/support/technical/
Best regards,
The App Store Team

Hi @dsuarez-quativa , We will look into this issue with the non-public selector:  Frameworks/NewRelic.framework/NewRelic: _didReceiveData
Is this issue preventing you or your users from using your new build of app in App Store?

@cdillard-NewRelic As of right now, no, because we haven't submitted a new build to be reviewed by Apple yet. We received that response afteruploading a new build to testflight.

@cdillard-NewRelic
In the console, using the latest sdk 7.4.1, the console logs this
2023-02-23 10:47:40.195756-0500 Quativa[62336:953081] API MISUSE: dataTask:didReceiveResponse:completionHandler: completion handler not calle

@dsuarez-quativa We determined there is an issue in iOS agent 7.4.1 that could cause Apple App Store rejection. We fixed the issue in a new release 7.4.2.
Can you please update to the newly released iOS agent 7.4.2? We've fixed the underlying issue in the new release. Thanks!

Hey @sunnyzanchi
There is a bug in the 7.4.2 agent version that causes the API MISUSE log.
The iOS agent team will release an update iOS agent 7.4.3 ASAP fixing this issue.
A workaround for now is for the customer to implement the dataTask:didReceiveResponse:completionHandler code in your apps custom NSURLSessionDataDelegate. ie: Add the below code to your implementation.
Swift
public func urlSession(_ session: URLSession, dataTask: URLSessionDataTask, didReceive response: URLResponse, completionHandler: @escaping (URLSession.ResponseDisposition) -> Void) {
    completionHandler(.allow)
}

Objective-C
- (void)URLSession:(NSURLSession *)session dataTask:(NSURLSessionDataTask *)dataTask didReceiveResponse:(NSURLResponse *)response completionHandler:(void (^)(NSURLSessionResponseDisposition disposition))completionHandler {
    completionHandler(NSURLSessionResponseAllow);
}

@dsuarez-quativa  @koneksa-sunny @ SVKorosteleva
Can you please update to the newly released iOS agent 7.4.3? We've fixed the issue with API MIUSE causing network connection issues in the new version. Thank you.

Installed version 7.4.3 - HTTP requests are not hanging anymore, thank you!


Explicitly enable TLS 1.0, 1.1 and 1.2

For some bizzare reason, TLS 1.2 is not enabled by default in .NET. This results in an error when communicating with the API if you have disabled TLS 1.0 on your web server as per best practices.
Let's Encrypt (Simple Windows ACME Client)

ACME Server: https://acme-v01.api.letsencrypt.org/
Config Folder: C:\[...]\letsencrypt-win-simple\httpsacme-v01.api.letsencrypt.org
Loading Signer from C:\[...]\letsencrypt-win-simple\httpsacme-v01.api.letsencrypt.org\Signer
System.Net.WebException: The request was aborted: Could not create SSL/TLS secure channel.
   at System.Net.HttpWebRequest.GetResponse()
   at ACMESharp.AcmeClient.RequestHttpGet(Uri uri)
   at ACMESharp.AcmeClient.Init()
   at LetsEncrypt.ACME.Simple.Program.Main(String[] args) in [...]

This PR explicitly enables TLS 1.0, 1.1 and 1.2, which are all the protocols that are supported by the API server. As it is an application wide setting, I have added it very early in Main().

It looks good, and didn't break anything that I found.

@ph11 Could you test the latest source and make sure you don't have issues. #219 disables TLS 1.0 and you enabled it here.

Works well for me, the main reason I enabled TLS 1.0 was it is enabled by default in .NET and is supported by https://acme-v01.api.letsencrypt.org/. Its best practice to disable TLS 1.0 where possible, as per #219, so disabling it in the client and preventing it from being used with the Lets Encrypt API is a good move.


Java 10+ Local-Variable Type Inference should be used RSPEC-6212

This is controversial and the Sonar team had to remove it from the default "Sonar Way" profile
https://rules.sonarsource.com/java/tag/java10/RSPEC-6212

This would be straightforward to implement with OpenRewrite, it can exist as a standalone recipe. This leverage UsesJavaVersion in its applicability test.

I'd like to work on this one and would like to clarify a few things.
To be sure, do we want to implement RSPEC-6212 100% Sonar like? I would argue against it because it also suggests using var for primitives, which is not recommended, as you can see in Project Ambers Local Variable Type Inference - G7.

transform local Variables like: MyClass myClass = new MyClass(); to var myClass = new MyClass();
keep null assignments like String line = null
special treatment for generics and the diamond operator List<String> lines = new ArrayList<>() to var lines = new ArrayList<String>(), keep also var list = List.of() in mind

We depend on prior good naming, otherwise some information will be lost. But I think that's out of scope here, right?


(HTTP code 404) unexpected - manifest for elasticsearch:latest not found

Expected behavior
Non 404
Actual behavior
(HTTP code 404) unexpected - manifest for elasticsearch:latest not found
Information about the Issue
Steps to reproduce the behavior

...
...

Dupe of #4171.

same here ...


Local LLMs

Any chance of integrating local LLMs through GPT4ALL, Oogabooga, LocalAI or connect to different model APIs (Claude)?

Thanks and stay tuned (I am AFK for a couple of days) The lowest lying fruit for me would be allowing the user to specify models for the lower level elements. This would allow everything compatible with the Langchain chat API to interoperable.
However your mileage may very as I will not likely have the capability to test them all, as right now, the only local model that I know of that is capable of “agentic behavior” is llama 70b. In terms of online hosted models Claude 2 should be good.

I removed the typing hints for OpenAI llms. While the examples still use them, other models could be used, provided that they implmented chat specific elements. While this doesnt directly address OogaBooga, you should be able to make use of langchain's GPT4All, and other model integrations (see list here ) - of course, your mileage may vary as to how much success you will have, due to the lack of emergent agentic behavior in some of the simpler models.

Closed, however, if this continues to trip people up, I may need to migrate to something other than langchain in the medium/long term.

flowise

Closed, however, if this continues to trip people up, I may need to migrate to something other than langchain in the medium/long term.

flowise


API Changes: AIO / I2C

Task Description
Some changes are required to match Soletta's API design conventions

[ ] use is_busy instead of just busy to highlight the query. So sol_aio_is_busy() instead of sol_aio_busy(). The same applies for sol_i2c_busy().
[ ] sol_i2c_bus_get() -> sol_i2c_get_bus()
[ ] change return type of functions that may fail to return error code instead of boolean false. Example: sol_i2c_set_slave_address()

looks like only the second item is missing, the others seem to be done already


do nothing if the focus is already in dialog

This PR fixes #9293
finally figured out why the focus is moving away, setting the dialog.message will change the focus 😆
the fix is to keep the focus if the it is already inside the dialog.

Are we handling the case where focus is within the messages dialog and it gets closed correctly? i.e. tab to the close button and click enter and then shift+tab - in my testing that seemed to escape outside the dialog.

Yes, I have tested scenario:

Move the keyboard focus to the 'Close' button in the message area.
Press enter to close the message box.
=> The focus will be moved to the first focusable element in the dialog
Press Shift+Tab
=>the focus will go to the cancel button of the dialog


fix: add necessary permissions to example workflow

Why:
Closes #31321
What's being changed (if available, include any code snippets, screenshots, or gifs):
Uses the diff from https://github.com/github/docs/pull/31322/
but adds explicit permissions as well
which allows it to be tested with
workflow_dispatch: if users desire so
Also use github.ref_name instead of the pull_request.number to query the branch
I couldn't get the original to work in local tests
but with ref_name it works flawlessly.
Check off the following:

[ ] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[ ] For content changes, I have completed the self-review checklist.

@corneliusroemer Thanks so much for opening a PR! I'll get this triaged for review ✨

@Mogyuchi made a good point that github.ref_name might not be the right name here, instead one should probably use github.head_ref.

@Mogyuchi made a good point that github.ref_name might not be the right name here, instead one should probably use github.head_ref.

@corneliusroemer
Sorry, I should have just said it straight.
In short, to clear the cache for a pull request, we need to stick to refs/pull/${{ github.event.pull_request.number }}/merge. Alternatively, if we want to clear the branch cache as well, we need to do it like discordjs-japan/om # 90 (pull_request should be replaced by pull_request_target).

…I feel like I need to explain how what is written in restrictions-for-accessing-a-cache affects cache cleanup.
Simply put, for example, if you have a pull request 123 with head(current) branch feature-b, the cache created by the push event will be for feature-b and the cache created by the pull_request event will be for refs/pull/123 /merge.
So, if the cache you want to delete was created in the push event, you need to specify the cache for feature-b, and if it was created in the pull_request event, you need to specify refs/pull/123/merge.

Hello @corneliusroemer and @Mogyuchi - thank you for the suggestions here. We had a chance to dig into this on our team and we agreed that we shouldn't change this example to use pull_request_target vs pull_request. The majority of our docs readers will be using the pull_request trigger and using this as the default aligns with the rest of our documentation as well.
That said, I didn't want to outright close this PR because I think it's a valid change to cover the cross-repository pull request use case.
In the "Events that trigger workflows" article, we do have a warning box about the permissions: "pull_request_target"
Which looks like it has the context someone would need for the cross-repo pull requests.
A few questions for you:

What would you think of keeping pull_request as the triggering event here but adding a note above the example that links to "pull_request_target" and describes the cross-repo pull request use case?
Based on this comment, it sounds like we should keep the permissions lines in this example even if we just use pull_request. Does that sound right?
Should this line remain the same: BRANCH: refs/pull/{% raw %}${{ github.event.pull_request.number }}{% endraw %}/merge

@jc-clark
https://github.com/github/docs/pull/31582#discussion_r1486602743

Therefore, I believe it is more appropriate to make this a separate pull request from https://github.com/github/docs/pull/31322 .

What I wanted to say is that I don't want this pull request to be based on my pull request.

What would you think of keeping pull_request as the triggering event here but adding a note above the example that links to "pull_request_target" and describes the cross-repo pull request use case?

I think we should talk about this in https://github.com/github/docs/pull/31322, so I'll answer there.

Based on https://github.com/github/docs/pull/31582#discussion_r1486602743, it sounds like we should keep the permissions lines in this example even if we just use pull_request. Does that sound right?

Yes. We should specify the permission. https://securitylab.github.com/research/github-actions-building-blocks/#following-the-principle-of-least-privilege

Should this line remain the same: BRANCH: refs/pull/{% raw %}${{ github.event.pull_request.number }}{% endraw %}/merge

Yes. Related pull request: https://github.com/github/docs/pull/23612

Thanks for the additional context @Mogyuchi and @corneliusroemer. We're deciding to make some updates to this article internally to account for using the pull_request_target event.
Our engineers agreed that the example is good as is. So we are going to leave it using the default pull_request event and then add some information about using pull_request_target as an option.
I'm going to close out this PR and our team will merge some improvements to this article soon.


1199282 - ms.topic update

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off Production 1199282

Thanks for catching that @cmcclister !


Pre-Demo Updates

Could you also add a clear filter button on the left-hand side bottom?

We can then mark it as done.


no total and percentage while uploading file on progress

I create file uploader to S3 to host the file and want to use progress bar to let users know that the upload is progressing. But I can't get the percentage or total. How should I do to get the total or percentage so I can use it for progress bar in view section? Thanks.
Below is the code for upload file.
ACTION on CONTROLLER
 'fileUpload': function(req, res, next) {
   var username = req.session.User.username;
   var originalFile = req.file('uploadFile')._files[0].stream.filename;
   var newFileName = username + '-' + originalFile.split(' ').join('-');

   req.file('uploadFile').on('progress', function (event){console.log(event);}).upload({
    adapter: require('skipper-s3'),
    key: 'AKIAJ3QPIM4A42AQITMQ',
    secret: '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',
    bucket: 'prntrprintbucket1',
    ACL: 'public-read',
    saveAs: newFileName
  }, function whenDone(err, filesUploaded) {
    if (err) {
      console.log(err);
      return res.negotiate(err);
    }

    var pt = {
      user: req.session.User.id,
      agency: req.param('agencyId'),
      path: filesUploaded[0].extra.Location,
      filename: filesUploaded[0].filename
    };

    Transaction.create(pt, function transactionCreated(err, trans){

      return res.ok({
        files: filesUploaded,
        textParams: req.params.all(),
        trans: trans,
        username: username,
        session: req.session
      });
    })
  });
}

RETURN in console
{ name: 'file1.docx', written: 17437, total: NaN, percent: 0 }
{ name: 'file1.docx', written: 34874, total: NaN, percent: 0 }

this issue is still happenig today,
i'm using:
skipper 0.8.2
skipper-s3 0.5.9


First presentation

Task Completed

First presentation was completed on the 30th of July 2022

Uploading SHOP SMART [Autosaved].pptx…


fix(AWS::EKS::Nodegroup.Taints): use the correct key for taints

Correct key is not Name it is Key
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-eks-nodegroup-taint.html

Doh! Nice catch. Thanks!

Thanks very much.


unable to bind DYLD_LIBRARY_PATH in order to load psycopg2 under pipenv

Issue description
See also Issue #879
I'm unable to run my app inside pipenv. My app depends on psycopg2, which uses these libraries defined by psql.
/Library/PostgreSQL/11/lib/libssl.1.1.dylib
/Library/PostgreSQL/11/lib/libcrypto.1.1.dylib

Normally, this works fine by defining DYLD_LIBRARY_PATH='/Library/PostgreSQL/11/lib/' before running the app. Under pipenv, however, I'm unable to define this env variable. I suspect MacOS is treating DYLD_LIBRARY_PATH specially, and deleting my definition.
When I define it in .env, the definition appears in pipenv run but not in pipenv shell.
I'm able to run my app (e.g. pipenv run python myapp works ok), but shell doesn't work.
This seems like a likely explanation

It turns out that this is due to a new 'feature' in System Integrity Protection. It will delete DYLD_* variables when launching subprocesses.

Expected result
In .env:
SOME_LIBRARY_PATH='/Library/PostgreSQL/11/lib/'
DYLD_LIBRARY_PATH='/Library/PostgreSQL/11/lib/'

I expect these values to be defined:
$ pipenv shell
(test) $ echo $SOME_LIBRARY_PATH
/Library/PostgreSQL/11/lib/
(test) $ echo $DYLD_LIBRARY_PATH
/Library/PostgreSQL/11/lib/
(test) $

Actual result
The value of DYLD_LIBRARY_PATH is being deleted, presumably by MacOS' System Integrity Protection.
$ pipenv shell
(test) $ echo $SOME_LIBRARY_PATH
/Library/PostgreSQL/11/lib/
(test) $ echo $DYLD_LIBRARY_PATH

(test) $

$ pipenv --support
Pipenv version: '2018.11.26'
Pipenv location: '/usr/local/lib/python3.7/site-packages/pipenv'
Python location: '/usr/local/opt/python/bin/python3.7'
Python installations found:

3.7.5: /Users/sstrassmann/.local/share/virtualenvs/kcl_api-vtPrDRc9/bin/python3
3.7.5: /usr/local/bin/python3
3.7.5: /usr/local/bin/python3.7m
2.7.10: /usr/bin/python
2.7.10: /usr/bin/pythonw
2.7.10: /usr/bin/python2.7

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.7.5',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '18.7.0',
 'platform_system': 'Darwin',
 'platform_version': 'Darwin Kernel Version 18.7.0: Tue Aug 20 16:57:14 PDT '
                     '2019; root:xnu-4903.271.2~2/RELEASE_X86_64',
 'python_full_version': '3.7.5',
 'python_version': '3.7',
 'sys_platform': 'darwin'}

System environment variables:

TERM_PROGRAM
PIP_PYTHON_PATH
TERM
SHELL
TMPDIR
Apple_PubSub_Socket_Render
SOME_LIBRARY_PATH
TERM_PROGRAM_VERSION
TERM_SESSION_ID
USER
SSH_AUTH_SOCK
__CF_USER_TEXT_ENCODING
VIRTUAL_ENV
PIPENV_ACTIVE
PATH
PWD
LANG
XPC_FLAGS
PS1
PYTHONDONTWRITEBYTECODE
XPC_SERVICE_NAME
HOME
SHLVL
LOGNAME
PYTHONPATH
PIP_DISABLE_PIP_VERSION_CHECK
_
PIP_SHIMS_BASE_MODULE
PYTHONFINDER_IGNORE_UNSUPPORTED

Pipenv–specific environment variables:

PIPENV_ACTIVE: 1

Debug–specific environment variables:

PATH: /Users/sstrassmann/.local/share/virtualenvs/kcl_api-vtPrDRc9/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin
SHELL: /bin/bash
LANG: en_US.UTF-8
PWD: /Users/sstrassmann/src/kcl_api
VIRTUAL_ENV: /Users/sstrassmann/.local/share/virtualenvs/kcl_api-vtPrDRc9

Contents of Pipfile ('/Users/sstrassmann/src/kcl_api/Pipfile'):
[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[dev-packages]

[packages]
beautifulsoup4 = "*"
boto3 = "*"
google-api-python-client = "*"
google-auth-httplib2 = "*"
google-auth-oauthlib = "*"
Flask = "*"
PyYAML = "*"

[requires]
python_version = "3.7"

As you quoted the explanation, pipenv run run scripts in the current process while pipenv shell will launch a subprocess, any suggestion about what pipenv can do here?

Perhaps pipenv shell could load definitions from .env in the child instead of the parent?

@straz Oh, that's reasonable

Also, for those of you trying to use psycopg2 on a mac, the correct solution is not to spend your time trying to either link these .dylib files or trying to change the env variable DYLD_LIBRARY_PATH. Instead, add psycopg2-binary to requirements.txt

Is there any action items here for the latest pipenv version?


Only proxy IP is visible in logs

I'm using caddy as a reverse proxy, also CloudFlare DDos protection is used (so generally another proxy).
In docker-compose right now for testing purposes I have this:
   - NGINX_SET_REAL_IP_FROM=<IP_ADDRESS>/0

Caddy config is simple:
support.example.com {
        reverse_proxy localhost:8181
}

Thought even this way, I can only see Cloudflare IPs in the logs.
Docker gets IP from <IP_ADDRESS>/16 range, so this is also unique.
If I remove the NGINX_SET_REAL_IP env, I can only see the docker IP address, with this setting Cloudflare proxy IP, but I never seen the real IP in the logs.
What am I missing, what should be configured?

In nginx logs I can see the correct IP in log lines, like this:
2023-07-18T08:50:42+02:00 <ACTUAL-IP-REDACTED>, <IP_ADDRESS> (Cloudflare proxy IP)

Same for php-fpm log, just appended a localhost proxy to the list:
2023-07-18T08:50:42+0200 <ACTUAL-IP-REDACTED>, <IP_ADDRESS> <IP_ADDRESS>

However on Admin Panel -> Dashboard -> System logs, the IP is incorrect. What can be changed to have this work correctly?