Extract SceneLoader state and functions from static class to module

In a recent PR, I added overloads of the various SceneLoader static functions that take an options param to make it easier to configure behavior of the loaders. In this PR, I am moving the new functions out of the static class and into module level functions. I'm doing this for two reasons:

Conceptually we want to move away from static classes to improve tree shaking.
Concretely in the alpha viewer I see 75% (7.5kb minified) of the SceneLoader code not being executed.

If we make this change now, then we don't have to deal with back compat later, and I can remove some of the code I previously had to add to distinguish between overloads (since the "legacy" functions are part of the static class, and the new ones are totally separate module level functions).
This change requires moving pretty much all the state and core functionality out of the SceneLoader static class and then having the static class functions just call the module level functions. This cuts the viewer package size down by about 5kb.
I also tacked on a handful of small viewer related fixes.

Reviewer - this PR has made changes to one or more package.json files.

Snapshot stored with reference name:
refs/pull/15396/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15396/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15396/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15396/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15396/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15396/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15396/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15396/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15396/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15396/merge/testResults/webgpuplaywright/index.html

Did not check the sceneLoader as the diff tool is all over the place

Yea sorry, I tried to keep everything in the same order to minimize the diff, but it's still bad. Couldn't think of any way to make it cleaner. :/

The only question is why keeping the class instead of a JS object ?

I wanted to keep the "legacy" stuff separate so I left it in the class. At that point, switching to a JS object doesn't change much.

and if splitting the module could make sense as long as we are not side effect false as I have the feeling that package size will grow out of the viewer for now ?

This shrinks the package size by about 5kb. Since I only use the module level function, the rest of the functions as well as the SceneLoader class are tree shaken away.

Reviewer - this PR has made changes to one or more package.json files.


:memo: [ci skip] add notes on 'apm' path in Windows

Comments came out while discussing issue #11137

Given tests on Windows are still broken right now (it's my current task fixing them up) I'm not sure we want to encourage running them just yet.  Recommending we hold off on this for now.

Fair enough. We just should remember to track this together with potential further Windows issues that could arise in #11543.
Side note: out of curiosity, how come a documentation issue is labeled as api ?
Cheers

I've tackled a bunch so far - paths that need normalizing, some font sizings and bufferd-process.

Hi.
Does it makes sense to solve conflicts on this PR?

Go ahead and solve the conflicts. Can you add a note that there are still some Windows tests that don't pass as well?  I'll get it merged in. Thanks!

There you go: please check if that looks fine.


path causes critical failure

running the main branch (love this software by the way, very useful!!)
@@ -156,5 +156,11 @@
 print("Moving downloaded videos to /media/gdrive/solar/cells/masterclips/")
-shutil.move(f'{output_dir}/*.mp4', '/media/gdrive/solar/cells/masterclips/')
+for file_name in os.listdir(output_dir):
+    if file_name.endswith('.mp4'):
+        shutil.move(os.path.join(output_dir, file_name), '/media/gdrive/solar/cells/masterclips/')
+
 print("Video downloading process completed.")

Applying the patch kills the app.  I'm able to recover by asking to build and save the file.
Traceback (most recent call last):
File "/home/noah/.local/bin/gptme", line 8, in
sys.exit(main())
^^^^^^
File "/usr/lib/python3/dist-packages/click/core.py", line 1130, in call
return self.main(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/click/core.py", line 1055, in main
rv = self.invoke(ctx)
^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/click/core.py", line 1404, in invoke
return ctx.invoke(self.callback, **ctx.params)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/click/core.py", line 760, in invoke
return __callback(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/noah/.local/lib/python3.11/site-packages/gptme/cli.py", line 219, in main
for msg in loop(log, no_confirm, model, stream=stream):  # pragma: no cover
File "/home/noah/.local/lib/python3.11/site-packages/gptme/cli.py", line 271, in loop
yield from execute_msg(msg_response, ask=not no_confirm)
File "/home/noah/.local/lib/python3.11/site-packages/gptme/tools/init.py", line 30, in execute_msg
yield from execute_codeblock(codeblock, ask)
File "/home/noah/.local/lib/python3.11/site-packages/gptme/tools/init.py", line 46, in execute_codeblock
yield from execute_patch(f"{codeblock}", fn, ask=ask)
File "/home/noah/.local/lib/python3.11/site-packages/gptme/tools/patch.py", line 95, in execute_patch
apply_file(codeblock, fn)
File "/home/noah/.local/lib/python3.11/site-packages/gptme/tools/patch.py", line 71, in apply_file
assert Path(filename).exists()
AssertionError

I used gptme to find out that it's not modifying the file in the correct path:
Assistant: The contents of `/home/noah/crash.txt` indicate that there was an issue when a patch was applied to a file. The traceback shows that the application crashed at the point where it asserts that a file path exists. This suggests that the file it was trying to patch did not exist or was not found.

The patch in the crash log is attempting to change a line of code that moves `.mp4` files from one directory to another. The original line using `shutil.move` with a wildcard pattern was replaced with a loop that iterates over files in a directory and moves each `.mp4` file individually.

The error `AssertionError` at the end of the traceback indicates that the assertion `Path(filename).exists()` failed, which means the file it was trying to patch was not found at the expected location.

To resolve this issue, you would need to ensure that the file you are trying to patch exists and that the path to the file is correct. If the file does exist, you might need to check the permissions to make sure that the script has the necessary access to read and write to the file.

If you have the file that needs to be patched and want to apply the patch manually, you can do so by editing the file to include the new loop for moving `.mp4` files. If you need assistance with that, I can guide you through the process.                                                                                User:

Ah, yes that should be an easy fix. Thanks for reporting!

Fixed now!
However, do note that the ```patch codeblocks are not meant for the actual diff/patch format, since GPTs can't reliably generate them. Instead they use a git-conflict-marker like syntax, like this:
```patch hello.py
<<<<<<< ORIGINAL
    print("Hello world")
=======
    name = input("What is your name? ")
    print(f"hello {name}")
>>>>>>> UPDATED
```


input Json sending the DataType as string always

Expected Behavior
api called inside the workflow should get the datatype of the input parameters as given in json.
input json as below
{
"candidate_project_key": 1,
"development_phase": "Phase III",
"end_date": "",
"is_archive_only": false,
"start_date": "",
"study_description": "TEST_7OCT21_3",
"study_id": "TEST_11OCT21_3",
"study_title": "TEST_11OCT21_3",
"therapeutic_area_key": 78,
"token": "eyJraWQiOiJsdf1cm9mU1RtNGNwRnFvM3hJVjdVblVZUzVfVXRzcE1JNk1pXzhEY2tPdHFBIiwiYWxnIjoiUlMyNTYifQ.eyJ2ZXIiOjEsImp0aSI6IkFULjNUR1Bic0kxNVFMNFB6NHlIVzdHQTdBZUtCVTUxQXdQTHYsd1bS1jQ3YxOTAiLCJpc3MiOiJodHRwczovL2ludGllbnQtZGV2Lm9rdGFwcmV2aWV3LmNvbS9vYXV0aDIvYXVzcmVxNGJoY1dQdlFna1YwaDciLCJhdWQiOiJpbnRpZW50IiwiaWF0IjoxNjMzOTMzNzU0LCJleHAiOjE2MzM5"
}
Actual Behavior
here even though the json input is having integer, boolean values the api is being passed with all the values as string.
Steps to Reproduce the Problem

create an dummy api which expects integer, boolean and string values as inputs
create an workflow to call the above created api and print the types of the input parameters
run the workflow with a json having boolean, string and interger values

Below is the workflow yaml
main:
params: [run_args]
steps:

step1:
call: http.post
args:
    url: https://internal_application.com/web/clin/api/dataservices/studies
    headers:
      content-type: application/json
      authorization: ${"Bearer "+run_args.token}
    body:
        "study_title": run_args.study_title
        "therapeutic_area_key": run_args.therapeutic_area_key
        "development_phase": run_args.development_phase
        "study_description": run_args.study_description
        "study_id": run_args.study_id
        "candidate_project_key": run_args.candidate_project_key
        "start_date": run_args.start_date
        "end_date": run_args.end_date
        "is_archive_only": run_args.is_archive_only
result: api_response

Specifications
google cloud workflow

Version:
Platform: google cloud platform

In JSON, datetimes are simply strings. They are often ISO 8601 format. With the recently released time.parse, these string dates can be converted to Workflows timestamps (Epoch seconds in a float).


Create a sample appropriate for first-time Deployment Framework users that is easy to get up and running within minutes

Create a sample appropriate for first-time Deployment Framework users that is easy to get up and running within minutes.  The existing sample is too complicated for first-time users because it tries to show many features of the Framework.  Instead, we need a very simple sample app that uses the most basic features to get an app deployed and shows how even a single-assembly solution can be used successfully with the Deployment Framework.
This work item was migrated from CodePlex
CodePlex work item ID: '5685'
Assigned to: 'tfabraham'
Vote count: '0'

[UnknownUser@1/3/2010]
Associated with changeset 33021.

[UnknownUser@1/3/2010]
Associated with changeset 33022.

[UnknownUser@1/4/2010]

[UnknownUser@1/4/2010]
Associated with changeset 33067.

[UnknownUser@1/5/2010]
Associated with changeset 33125.

[UnknownUser@1/10/2010]
Associated with changeset 33250.

[UnknownUser@1/10/2010]
Resolved with changeset 33253.

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]


Cannot edit the "Code is checked in" notification

After settings the global rule "Code is checked in" => send email to "Project Collection Valid Users", that option is grayed out and cannot be disabled

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

ID: c2fdc7bf-dd88-5f39-cfaf-77dca428990e
Version Independent ID: c939066c-c671-7f50-3ba3-9916b0f4cd4c
Content: Manage personal notification settings - Azure DevOps
Content Source: docs/notifications/howto-manage-personal-notifications.md
Product: devops
Technology: devops-collab
GitHub Login: @chcomley
Microsoft Alias: chcomley

Hi @Simone000, thanks for reaching out. I'm looking into this and we'll be in touch. Have a great day!

Hi @Simone000, Following up on this, I'm hoping you found a solution at this point. If you are still having issues, I believe you may need technical help, rather than a doc update. Go here, and then select Technical Support. Best of luck to you!


Implement jsctl clusters cleanup secrets remove-secret-owner-refs

Implement jsctl clusters cleanup secrets remove-secret-owner-refs. This command is to help users prepare their clusters to back up, remove and replace cert-manager CRDs safely.
https://user-images.githubusercontent.com/1774239/201705713-c52b358b-5eb5-4509-920e-fd69f6caf65c.mov

Also noticed that the command errors out if --enable-certificate-owner-ref flag is set to false

Also noticed that the command errors out if --enable-certificate-owner-ref flag is set to false

By this, do you mean that the command interprets it as true (which is what I see and need to fix) or do you see another error?

I've updated the command to prompt before taking action:
$ go run main.go experimental clusters cleanup secrets remove-certificate-owner-refs
Checking for ownerReferences on secrets containing the issued certificates...
jetstack-secure/exmaple-com-tls needs update
Found 1 secrets with ownerReferences to Certificate resources
Would you like to update the owner references of 1 secrets?
> yes
jetstack-secure/exmaple-com-tls updated

I have also addressed the issue with the empty ownerRefs.
Ready for you to have another look now @irbekrm  🙏

Thanks @charlieegan3 , I've ran a couple tests and all looks good to me, I saw that another actor's added owner ref does not get removed this time and that setting the --enable-certificate-owner-ref flag to false does not make the command error
The only thing left should be https://github.com/jetstack/jsctl/pull/62#discussion_r1023918822

Thanks Irbe, I think that this should be all good now! I have rebased but ce15924b7afdb48cd5fea0f9cf1e2a1bb756228f is the only new commit.

Thanks for all the input, much appreciated 🙂


programme overview: data might be missing

shouldn't there be more years?
https://oodikone.cs.helsinki.fi/study-programme/520080

needs more info; backend is currently coded to calculate only 5 years back


better warnings

----- Training u probabilities using random sampling -----

Trained u probabilities using random sampling

Your model is not yet fully trained.

----- Starting EM training session -----

Training the m probabilities of the model by blocking on:
l.first_name = r.first_name and l.surname = r.surname

Parameter estimates will be made for the following comparison(s):
    - dob
    - city
    - email

Parameter estimates cannot be made for the following comparison(s) since they are used in the blocking rules:
    - first_name
    - surname

Iteration 1: Largest change in params was -0.285 in the m_probability of dob, level `Exact match`
Iteration 2: Largest change in params was 0.022 in proportion_of_matches
Iteration 3: Largest change in params was 0.0124 in proportion_of_matches
Iteration 4: Largest change in params was 0.00756 in proportion_of_matches

EM converged after 4 iterations

Your model is not yet fully trained. Missing estimates for:
    - first_name (no m values are trained).
    - surname (no m values are trained).

 -- WARNING --
You have called predict(), but there are some parameter estimates which have neither been estimated or specified in your settings dictionary.  To produce predictions the following untrained trained parameters will use default values.
Comparison: 'first_name':
    m values not fully trained
Comparison: 'surname':
    m values not fully trained

Here's a second example where lots of things go wrong because the data has only a few rows and they're all identical:
----- Training u probabilities using random sampling -----
u probability not trained for first_name - All other comparisons (comparison vector value: 0). This usually means the comparison level was never observed in the training data.
u probability not trained for surname - All other comparisons (comparison vector value: 0). This usually means the comparison level was never observed in the training data.
u probability not trained for dob - All other comparisons (comparison vector value: 0). This usually means the comparison level was never observed in the training data.

Trained u probabilities using random sampling

Your model is not yet fully trained.

----- Starting EM training session -----

Training the m probabilities of the model by blocking on:
l.first_name = r.first_name and l.surname = r.surname

Parameter estimates will be made for the following comparison(s):
    - dob

Parameter estimates cannot be made for the following comparison(s) since they are used in the blocking rules:
    - first_name
    - surname

WARNING:
Level All other comparisons on comparison dob not observed in dataset, unable to train m value
Iteration 1: Largest change in params was 0.05 in the m_probability of dob, level `Exact match`

WARNING:
Level All other comparisons on comparison dob not observed in dataset, unable to train m value
Iteration 2: Largest change in params was -2.22e-16 in the m_probability of dob, level `Exact match`

EM converged after 2 iterations
m probability not trained for dob - All other comparisons (comparison vector value: 0). This usually means the comparison level was never observed in the training data.

Your model is not yet fully trained.

----- Starting EM training session -----

Training the m probabilities of the model by blocking on:
l.dob = r.dob

Parameter estimates will be made for the following comparison(s):
    - first_name
    - surname

Parameter estimates cannot be made for the following comparison(s) since they are used in the blocking rules:
    - dob

WARNING:
Level All other comparisons on comparison first_name not observed in dataset, unable to train m value

WARNING:
Level All other comparisons on comparison surname not observed in dataset, unable to train m value
Iteration 1: Largest change in params was 0.05 in the m_probability of first_name, level `Exact match`

WARNING:
Level All other comparisons on comparison first_name not observed in dataset, unable to train m value

WARNING:
Level All other comparisons on comparison surname not observed in dataset, unable to train m value
Iteration 2: Largest change in params was 0 in the u_probability of first_name, level `Exact match`

EM converged after 2 iterations
m probability not trained for first_name - All other comparisons (comparison vector value: 0). This usually means the comparison level was never observed in the training data.
m probability not trained for surname - All other comparisons (comparison vector value: 0). This usually means the comparison level was never observed in the training data.

Your model is not yet fully trained.

 -- WARNING --
You have called predict(), but there are some parameter estimates which have neither been estimated or specified in your settings dictionary.  To produce predictions the following untrained trained parameters will use default values.


Joining grcpool from boinccmd

Hi,
In headless Linux based boinccmd there is a way to attach to the project.
boinccmd --project_attach "https://www.grcpool.com/" <account_key>
Normally, the account key is achieved with
boinccmd --create_account "https://www.grcpool.com/" <email> <password> <username>
This command returns:
poll status: permanent HTTP error
Registering an account at https://www.grcpool.com/ does not expose the account key. So it's impossible to attach grcpool project to boinc.
How to achieve the account key for GRC?

Resolved with:
sudo boinccmd --acct_mgr attach https://www.grcpool.com/  <username> <password>
sudo boinccmd --acct_mgr sync
Add some projects at https://www.grcpool.com/account/host/XXXX --> save project settings
sudo boinccmd --acct_mgr sync


Bug: First fontWeight applied to whole paragraph instead of specific / section

I'm using react-pdf with nodejs (had to apply the fix of #89 to get css styles to properly work).
For us it all works fine as long as the whole string is marked as bold but as soon as we render a portion of the string as bold eg. Hello, world it all becomes non bold? Same with inline spans etc.
Rendering:
xx results in xx
xx results in xx
The first fontweight is always applied to the whole paragraph
We've also seen that the same issue goes for inline fontFamily changes. However, tags like , <s>, spans with styles... all work fine.
Reproduction
Try rendering the above strings at the latest versions, not sure if its nodejs specific.

Fixed according to the above recommended fix to package.json:

  "resolutions": {
    "@react-pdf/layout": "3.11.5"
  },


WIP: POC: bottom tabs (and "future of terminal UX" toughts)

Summary of the Pull Request
Mocking up a switchable 'titlebarRow' and 'contentRow' in NonClientIslandWindow.cpp
...it was both easier than expected and works better than expected:

References and Relevant Issues

microsoft/microsoft-ui-xaml#7032
#8888
#10000
#11111
#13392

Detailed Description of the Pull Request / Additional comments
This is just a very-very crude POC, just to see if it's possible in the current implementation to place the tabs at the bottom, and mainly as an attempt to induce thought and spark discussion about ideas for streamlining UX around the _quakeMode window, particularly in relation to the settings panel (also considering the clutter of other terminal windows active at the same time, like elevated ones)
Checklists, Todos, etc., ... (as food for thought):

[x] switching the order of the titlebar and the content pane is rather straightforward

[x] window is draggable by the bottom titlebar, window controls work, tab drag and drop works between windows
however, in it's current form it's far from being complete, both functionality/implementation and usability/design wise

[ ] (dependency(?) microsoft/microsoft-ui-xaml#7032

[ ] (bug) tab tear-off sometimes places the bottom-titlebar below visible bounds.

[ ] (to-do) setting UI for displaying the tabs on bottom for the Quake Mode window only (maybe for all terminal windows?)

[ ] (to-discuss) allow the Settings Pane to pop up/over the Quake Mode window (or All terminal windows)
in it's own, possibly centered / default-sized window (in a _settingsMode window, so to speak).
...as currently the wide + narrow shape of the quake mode window hinders the usability of the settings panel embedded in a tab. (see window shapes in the above screenshot at 1920×1080, also considering keeping the current '1-column' design philosophy of the settings panel (which would lead to many of the options not being very visible at first glance, and would leave a lot of horizontal screen estate wasted)

[ ] (to-develop) unifying the elevated and non-elevated terminal processes into one window (using the shield icon per tab)
for example, and if possible, by proxying elevated terminal I/O using named pipes for IPC with a non-elevated monarch?

Thanks for bearing with me, and kudos for an already immensely useful app!

WHOA


Feature request: Add support for NamedTuples in dataloaders

In order to produce self-documenting code, our team has the habit of using NamedTuples instead of plain tuples as the return type of our datasets.
Since they are sub-classes of tuples, they are handled by every PyTorch mechanism we've encountered as if they are tuple, so it all works smoothly.
When using Accelerate, I get an error in send_to_device at line 114. type(tensor)(send_to_device(t, device) for t in tensor) raises a type error because type(tensor) returns my NamedTuple, which cannot use the generator (send_to_device(t, device) for t in tensor) as argument. We would need some way to transform the generator into a type that is accepted by the NamedTuple.

Added support in the PR mentioned above!


Gradle plugin: prevent logging misconfiguration and improve plugin UX

Possible issues

Users may configure logging via gatling configuration closure and yet have their own logback config file. This may lead to confusion.

Scope of changes

If a user puts logback.xml or similar to resources folder, but the same times uses logLevel or logHttp parameters of gatling closure - plugin should

either throw an exception
or - WARN message log should be logged using Gradle loggers
create tests for this feature

I'm in favor of a warning that the plugin logHttp configuration will be overridden by the logback.xml.
The final user may want to add their own task on top of those provided by the plugin. Some will add the logback.xml, some not.
So, in the same project, the default configuration will be that of the logHttp plugin, and some will override this by its own logback.xml
More, it may be difficult to detect if the custom logback.xml is not for logging something else (an additional library to load data, etc.)

Makes sense. Will adjust the descriprion.

FYI this is icorrect statement, a single gatling source folder can not have multiple logback.xml \ logback.groovy

I mentioned a single project, not a single folder.
Single project, single plugin, but different tasks (that can be linked to other folders as well)

What a you talking about ?
The one project has single resource folder where logback.xml \ logback.groovy could be located.
All gatling tasks are sharing the same code base - i.e. there is a single logback.xml \ logback.groovy for all taskk in a one proejct !


Simple click and callback not working. Jasmine error?- Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL.

I try to run a simple request to the server.  The server reply and angular shows the message but the test freezes and i get the error below.
I am new in e2e (testing in general). Am I doing something wrong or it is a bug.
protractor protractor-e2e.js
Starting selenium standalone server...
[launcher] Running 1 instances of WebDriver
Selenium standalone server started at http://<IP_ADDRESS>:54697/wd/hub
Started
A Jasmine spec timed out. Resetting the WebDriver Control Flow.
F

Failures:
1) Registration view should ask for a proper email "The email must be a valid email address."
  Message:
    Error: Timeout - Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL.
  Stack:
    Error: Timeout - Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL.
        at tryOnTimeout (timers.js:224:11)
        at Timer.listOnTimeout (timers.js:198:5)

Ran 1 of 9 specs
1 spec, 1 failure
Finished in 27.718 seconds
Shutting down selenium standalone server.
[launcher] 0 instance(s) of WebDriver still running
[launcher] chrome #01 failed 1 test(s)
[launcher] overall: 1 failed spec(s)
[launcher] Process exited with error code 1

protractor --version
Version 3.2.2
    beforeEach(function() {
        browser.get('#/account/register');
    });
    fit('should ask for a proper email "The email must be a valid email address."', function function_name(argument) {
        var inputname = element(by.model('userData.name')).sendKeys(fulanito.name);
        var inputname = element(by.model('userData.last_name')).sendKeys(fulanito.last_name);
        var inputemail = element(by.model('userData.email')).sendKeys('fulanito@ara');
        var inputpassword = element(by.model('userData.password')).sendKeys(fulanito.password);
        var inputpassword_confirmation = element(by.model('userData.password_confirmation')).sendKeys(fulanito.password);

        element(by.css('button[type="submit"]')).click().then(function() {
            expect(element(by.css('body')).getText())
            .toContain('The email must be a valid email address.');
        });

    });

My config file
exports.config = {
    allScriptsTimeout: 99999,
    // The address of a running selenium server.
    //seleniumAddress: 'http://localhost:4444/wd/hub',
    // Capabilities to be passed to the webdriver instance.
    capabilities: {
        'browserName': 'chrome'
    },
     onPrepare: function() {
        // you can also add properties to globals here
        global.host= 'http://localhost:9000/';
    },
    keepAlive: true, //intesive testing
    baseUrl: 'http://localhost:9000',
    //Stand alone: 9001
    //baseUrl: 'http://localhost:9001',
    framework: 'jasmine',
    // Spec patterns are relative to the current working directly when
    // protractor is called.
    //specs: ['test/e2e/*.js'],
    specs: ['test/e2e/register-spec.js'],
    // Options to be passed to Jasmine-node.
    jasmineNodeOpts: {
        showColors: true,
        defaultTimeoutInterval: 25000,
        isVerbose: true,
        includeStackTrace: true
    }
};

Hi there!
Your question is better suited for StackOverflow or Gitter. Please ask a question there with the 'protractor' tag or post in the [Gitter Channel](https://gitter.im/angular/ protractor) to get help.
That said, the benefit of protractor is that it waits for your application to finish processing the changes you make to it during a test. So your test could be like this:
element(by.css('button[type="submit"]')).click();
expect(element(by.css('body')).getText())
    .toContain('The email must be a valid email address.');

If you have further questions, please follow up either in gitter or stackoverflow. Thanks!

@mgiambalvo
I asked here and not in stackOverflow because I get the Jasmine error in both way (the way you recommend also).
I can see the page has already the alert but Protractor freeze either way. And give me the not so helpfull error. So IMHO or ignorance it is a bug.

Is there a chance you could share your application source? You mentioned an
alert. Is the text you're looking for ('The email must be a valid email
address.') in an alert box, as in window.alert()? If so, you'll need to
look for it in a different way, as described in this StackOverflow question
http://stackoverflow.com/questions/19879631/protractor-get-text-of-an-alert.
Hope this helps!
On Fri, Apr 15, 2016 at 9:45 AM, Hugo Mosh Cardoza<EMAIL_ADDRESS>
wrote:

@mgiambalvo https://github.com/mgiambalvo
I asked here and not in stackOverflow because I get the Jasmine error in
both way (the way you recommend also).
I can see the page has already the alert but Protractor freeze either way.
And give me the not so helpfull error. So IMHO or ignorance it is a bug.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/angular/protractor/issues/3130#issuecomment-210542214

Turns out I was making an error, as explained in stackoverflow: http://stackoverflow.com/questions/36653107/why-i-get-this-error-in-protractor-of-jasmine-timeout-and-how-can-i-test-correct/36702452#36702452
Thank you


KAFKA-10575: Add onRestoreSuspsnded to StateRestoreListener

Add the new API (default impl is empty) to StateRestoreListener.
Update related unit tests

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

This PR is now ready for review @cadonna @lucasbru

Why has this PR 392 commits?

Why has this PR 392 commits?

I created this branch / PR around Oct. 2022, and then I rebased on top of trunk. My local branch is in right place but it seems github cannot recognize the rebasing. That should be okay since when merging it would squash into a single commit anyways.

@mjsax tried to resolve the long commit history but it seems not possible, so I created a new PR (with the same branch name) here: https://github.com/apache/kafka/pull/13179 it's ready for another look.


New Icon/Logo?

Love the app and have been using it more frequently than the actual SickRage web UI. Thanks for the great work!
I wanted to offer if you wanted to use one of the logos I designed for SickRage. I can provide you with the hi-res PNG files with transparencies for you to use. Just let me know if you're interested.
Here's a link to my post on the SR forums w/ the previews:
https://sickrage.tv/forums/forum/main-category/feedback-suggestions/29181-new-logo-suggestions-embedded

Hi @Concept211
Design is really not my strong suit, so any help on that level is always more than welcome. Even I don't like the current ShowsRage logos...
Both of your logos look great, with maybe a preference for the second one. The best for me is to have them in SVG format to include it in the application. Or I can give you a list of sizes the images should be in. Whichever is best for you.
If you want to know how it is used for the moment, you can look here in the drawable-* and mipmap-* folders:

ic_notification.png
ic_launcher.png
ic_launcher_white.png

I am trying to finish a new version for tomorrow, but it can be included in a future version.
I can credit you in the "About" section of the application and the release notes if you want.
Thanks for your contribution :)

Awesome...not a problem. I'll get you the SVG hopefully today if I get a moment. I'll give you both versions so that you can try them and see which looks best.
Btw...I LOVE how you grab a dominant color sample from the show image/graphic and use it for that show's interface color scheme. Brilliant!

Ok, here you go. I uploaded both SVG and hi-res PNG formats for both versions of the logo:
http://jmp.sh/ES7hrfQ
Let me know if they work out for you.
Thanks!

Awesome, thanks!
I will try to integrate them this evening if I have time to show you what it looks like. Otherwise next week or in two weeks.
The dynamic design color is an idea taken from Material Design (the new Android design). I wouldn't have thought of that on my own :P

Awesome...I can't wait to see it! I've been checking for updates within the app last night hoping it was updated but no such luck ;)
I've only seen the UI change colors like that within Google's Messenger app but those colors are completely random. I've never seen any app take a color sample from an image like that. Whoever's idea it is, it's slick!

Here are previews of both logos in the application drawer of Android, compared to the current one:

Now I see that the current logo is really blurry :o
The sharked-TV is still my favorite between the two of your icons. What do you think?
I also use the icon in white for the notification and in the sliding menu. Do you think you can make those icons in white too? Maybe just invert the colors (the white becomes blue and the blue becomes white).
I am trying to submit a new test version in the next hours. It won't include your icon, but I will include them before making the new version public :)

Yeah, I think we can go with the shark/tv version. But the white line inside ended up coming out way too thin. I'll go ahead and revise it as well as give you the white version for the notification drawer. I'll have it to you shortly.
thanks!

Here you go: http://jmp.sh/XgQ0wt3

That was a fast work ! Here are updated previews with the new logos.
It looks really cool. Great job :)

For the colors, I chose a set from here, to stay as close to Material Design guideline as possible. I don't mind using other colors. I just need to check the final result.
In the application, I use #3F51B5 and #303F9F as the main colors. All the colors I use are listed here.
For the Play Store, they ask for the banner of 1024x500. The banner you suggested is fine for me. So I guess you just have to move the text on the right of the logo to fit in the required dimension?
Sadly, I didn't have time to finish the new version today. So it will have to wait next week. At least it will include you new logo.
I really appreciate your contribution here. It really helps ShowsRage looks better! If you have a website/link or something that I can add in the app or here, let me know. And I will gladly promote your work if you want.

Yeah I would use a sample of these values:
#13537F
#09769F
#038EB7
Here's the Google Play banner to spec: http://jmp.sh/6APheDB
Yeah, I'd appreciate the mention :+1:  I'm in the process of redesigning my website but you can link it www.concept211.com
Thanks again!  Looks great so far.

I've been playing with the new colors and logo for the past week, and the result is really nice.
I used #09769F and #13537F for the colors.
Here are a couple of screenshots:

I love it...great job!  Can't wait to update ;)  No offense to the original product, but I feel these colors are easier on the eyes. I just feel the whole "Material" color scheme is a bit too harsh.  I love Google's push for simple/clean designs, but the colors are a little much ;)

I just pushed the changes for the new logo and colors (https://github.com/MGaetan89/ShowsRage/commit/13414a793dbf8375f54e7cec01b0f160dde0a5b1). If you want to change something else, let me know and I will see what I can do.
I added a link to your website in the application and I will credit you in the release note of the next version (next week at the latest).
Thank you a lot for this contribution. It really helped the application look better :)

I'm really happy to be a part of it! So is there a build/.apk anywhere to update manually without having to wait for Google Play to update?

Not for the moment, no. I still have one thing I want to fix before the release.
Hopefully I will managed to do it today, so it will be available later this evening.
It will be available in the beta channel first. So you can subscribe to the beta version if haven't yet.

Sadly, the release is still not for today.
I will have more time next week to finish and publish it.

I will release the next version in a few minutes.
It will be available to the tester soon.

Came out great! Thanks again!

You did most of the work here ;)


Add Google.Cloud.Domains.V1Beta1

Playbook exercise take 2. :)
Release will be opened in a separate PR.

I believe the key has been properly removed from the history now, PTAL.


NIFI-5979 : enhanced ReplaceText processor with "Number of Occurrences" and "Occurrence offset" configurations

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

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

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

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

[x] Is your initial contribution a single, squashed commit?

For code changes:

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

For documentation related changes:

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

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

@bbende Could you please review it..

This PR contains changes related to ReplaceText Enhancement. I've added couple of configurations i.e.Number of Occurrences  and Occurrence Offset as discussed in https://issues.apache.org/jira/browse/NIFI-5979
These new configurations gives more control to the user on selective replacement of text patterns
on a large content which I think is a good feature considering very wide user base for ReplaceText processor.

Hi @pushpavanthar thanks for the PR!
I reviewed the code change, and posted few comments. I haven't tested this improvement in action though.
Would you share some use-cases where the number of occurrences and offset are required?
The reason why I ask that is, on the NIFI-5979 JIRA description, Adam mentioned he wanted to replace only the first line of a CSV file. I feel the proposed changes with this PR looks overkill to realize the requested feature.
Also, if a text contains target values N times. How does a user know the number when they configure this processor? I'm probably missing something, but I can't imagine situations well where the N is a static constraint.
Alternatively, we may be able to add new Evaluation Mode such as The First Line and The Last Line, as I see few other users wanted to replace only header or footer of a text file.
How do you think?

Hi @ijokarumawak, thanks for reviewing.
The requirement in the NIFI-5979 JIRA was just to replace the header values. However, I was thinking of below use-cases

I need to replace all occurrences of a text/regex except for the header and/or footer. If the content has fixed number of rows this can be achieved by using Occurrence Offsets and Number of Occurrences
This might be a specific use-case, but I strongly believe that many might come across such case. Consider a json as payload, which has "cityCode":"NYC" and "city":"NYC", suppose "NYC" is also present in "address" and "description" fields.

{
  "cityCode": "NYC",
 "city": "NYC",
  "address": "200 W 44th St, NYC, NY 10036, USA",
  "description": "The Theater District is the teeming heart of Midtown West, NYC. In the pedestrian plazas of Times Square, costumed characters beckon to energetic crowds under the pulsing lights of towering digital billboards. Locals and tourists gather on the giant red staircase above the TKTS booth, which sells tickets to Broadway shows running at the area’s historic theaters. On busy 42nd Street, chain stores and restaurants abound."

}

But I chose to keep "cityCode":"NYC" and  "city":"NYC" untouched and want to change the rest of the occurrences of "NYC" to "New York City" where I can make use of Occurrence Offsets .

Hi @pushpavanthar thanks for elaborating your use-cases!

If one wants to replace except header and footer, adding evaluation mode would be more easy to configure and can support unfixed number of rows. How about adding Evaluation Mode such as The First Line, The Last Line and Except The First and Last Line?
I feel the second use case may be handled better with UpdateRecord processor. Because it requires schema awareness. With UpdateRecord, you can replace specific element value. Please take a look on the additional details doc for the processor.
https://nifi.apache.org/docs/nifi-docs/components/org.apache.nifi/nifi-standard-nar/1.9.1/org.apache.nifi.processors.standard.UpdateRecord/additionalDetails.html

Hi @ijokarumawak I was not aware of UpdateRecord processor. Yes, second use-case requires schema awareness and UpdateRecord processor makes more sense for such use case.
I'm aligned with you for the first use-case. I'll make changes by adding Evaluation Mode and update my PR.

Hi @ijokarumawak , I've pushed changes as you suggested. Could you please review it?

Thanks @pushpavanthar for the update! I will review it. In the meantime, please add the test text files to Maven rat plugin exclusion configured at pom.xml. Currently, these files cause build failure with -Pcontrib-check maven option.
https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/pom.xml#L381

@ijokarumawak thanks for your time.
I incorporated all your review comments in my latest commit. Kindly review the same.

@ijokarumawak I couldn't agree less on the functional approach ( pushpavanthar#1 ).
I learnt functional concepts and best practices from this PR.
Thank you very much for your patience and time.
Please go ahead and merge.

Thanks @pushpavanthar for your confirmation! (I believe you wanted to say 'I couldn't agree more')
I'm going to squash commits and merge it into master.


Add initial support for Helm dependencies

This implementation supports bumping chart dependencies under Chart.yaml.
It uses Psych to marshal and unmarshal the YAML file so comments are lost.
Updating Chart.lock and archives under charts/ directory will be implemented in future.
This is a draft PR, it will be updated to add tests.

Unless I'm missing something this doesn't update the Chart.lock file?

Unless I'm missing something this doesn't update the Chart.lock file?

Unless I'm missing something this doesn't update the Chart.lock file?

Correct, it will require installing and using helm binary (which can be done in this PR or a separate one).
Also see https://github.com/dependabot/dependabot-core/issues/2237#issuecomment-695829118

Unless I'm missing something this doesn't update the Chart.lock file?

Correct, it will require installing and using helm binary (which can be done in this PR or a separate one).
Also see https://github.com/dependabot/dependabot-core/issues/2237#issuecomment-695829118

Closing this PR as no new ecosystems are accepted https://github.com/dependabot/dependabot-core/issues/2237#issuecomment-767560668

Closing this PR as no new ecosystems are accepted https://github.com/dependabot/dependabot-core/issues/2237#issuecomment-767560668

@siavashs should we reopen this, as new ecosystems are again welcome? https://github.com/dependabot/dependabot-core/commit/0f931f452e9e51a3996a010ca21c90115b7f80cf

Oooh, good find @alexef! What do you say @siavashs?

Hey, good to know that new ecosystems are accepted again.
I might revive this when I find some free time.


Error when implementing an abstract class with a private method

TypeScript Version:  2.0.3
Code
abstract class InterfaceClass {
	private methodToBeImplemented(argOne: string): void {

	}
}

class NormalClass implements InterfaceClass {
	private methodToBeImplemented: (argOne: string) => void
}

Expected behavior:
I should not get an error
Actual behavior:
I get the error
Class 'NormalClass' incorrectly implements interface 'InterfaceClass'.
Types have separate declarations of a private property 'methodToBeImplemented'.

Private methods can't be overridden (they're private!). They also can't be implemented except by their declaring class.

Perhaps you wanted methodToBeImplemented to be itself abstract? You provided an implementation in the base class. If you had marked it abstract, you should get an error that private methods cannot be marked abstract.


tsdb: Do not reuse the same crc32 hash between Chunk() calls

The same crc32 object was called concurrently between chunks. As this
object is only used in that function, create a new one each time the
function is run.
See #6512
Signed-off-by: Julien Pivotto<EMAIL_ADDRESS>

Now to test this...

Now to test it..

It does not fix it (test fails)

I am not sure that it is enough ; now it runs fine if  r.Chunk(chkExp.Ref) is called concurrently with the same chkExp.Ref but not otherwise.

more-complex-than-it-looks

Thanks for starting this up. Looks like we are also reusing buf = [binary.MaxVarintLen32]byte{} This is to reuse the same buffer for calculating the checksum. The easiest would be to just do Sum(nil), but we can consider adding sync.Pool for this. For 5 bytes I think we can + slice header I think it's not worth it.


Filter no use sink when log print

after:50000	     29675 ns/op	    1296 B/op	      18 allocs/op
before:50000	     29636 ns/op	    2161 B/op	      30 allocs/op

who can merge this?


Files that import project references don't get type checked

Current behavior
IDK if this is a ts-loader issue (there is already a conversation at https://github.com/TypeStrong/ts-loader/issues/1158 ) or a fork-ts-checker one.
Problem is CASE A below
CASE A: I have transpileOnly: true in ts-loader options + fork-ts-checker enabled

❌No type errors if the type error is in a non-reference file (eg. a file that imports a reference but doesn't get imported itself)
✔Type errors if the type error is in a reference file (eg. a file that gets imported)

CASE B: I just have fork-ts-checker enabled:

✔Type errors if the type error is in a non-reference file (eg. a file that imports a reference but doesn't get imported itself)
✔Type errors if the type error is in a reference file (eg. a file that gets imported)

It's not clear for me if fork-ts-checker does anything, check also the profiling[1]
Expected behavior
CASE A should emit type errors
Steps to reproduce the issue
Git clone the project ref demo from https://github.com/RyanCavanaugh/project-references-demo add your preferred webpack config and test CASE A1 till B2.
More Resources

[1]fork-ts-checker profiling output for each case: https://github.com/TypeStrong/ts-loader/issues/1158#issuecomment-681058678
type error example: https://github.com/TypeStrong/ts-loader/pull/1173#issuecomment-681040638
benchmarks with different setups: https://github.com/TypeStrong/ts-loader/issues/1157#issuecomment-680674236

Environment

fork-ts-checker-webpack-plugin: 5.1
typescript: 3.9.7
webpack: 4.44.1
os: 18.04

fyi I found the problem: I needed to set context explicitly to the to be compiled project folder within the project references structure. now fork-ts shows also some activity


nixos/tandoor-recipes: revert 'GUNICORN_MEDIA=1'

Unbreak the module.
Description of changes
Quoting from upstream's documentation 1:

Basically everyone recommends not doing this. Please use [a webserver]
to handle media file serving.

Given that this commit broke the module for unrelated reasons, I've decided to just revert it and let downstream users make the choice of easy vs. secure.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[X] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg test tandoor-recipes


"Info" log messages to be shown depending on different log levels

Summary or solution description
Included log_level argument to compile and compile_and_save and --log-level flag to cli compile command to change which message are shown on log. Default log level is set to ERROR
How to Reproduce
from boa3.boa3 import Boa3

Boa3.compile('path/to/smart/contract.py', log_level='INFO')
Boa3.compile_and_save('path/to/smart/contract.py')  # log level is set to ERROR by default
Boa3.compile_and_save('path/to/smart/contract.py', fail_fast='WARN')

 neo3-boa compile path/to/smart/contract.py --log-level INFO

Tests
https://github.com/CityOfZion/neo3-boa/blob/cae5c9ad1c2eb07c1be9e891358eaf3c318e9710/boa3_test/tests/compiler_tests/test_logger.py#L5-L13
Platform:

OS: Windows 10 x64
Python version: Python 3.7+

Task linked: CU-864eejzpx "Info" log messages to be shown depending on different log levels

coverage: 91.722% (+0.08%) from 91.642% when pulling f06748486701c1b2689fb99d867f008c30cf13da on CU-864eejzpx into 817dfee6440a117a08bba0e26e120a528835a406 on development.


x/sys/windows/svc: gs:28h is zero after servicemain returns sometimes

I'm very rarely seeing crashes when using services from c-shared hosted in a .NET process, to the tune of:
Application: demo-client.exe
CoreCLR Version: 5.0.921.35908
.NET Version: 5.0.9
Description: The process was terminated due to an unhandled exception.
Exception Info: exception code c0000005, exception address 00007FF9286BCFC8

Disassembling, that matches:

Apparently our TLS slot has been zeroed by something?

CC @alexbrainman


Add unit tests for cpu check

Unit tests need to be expanded, should be table driven.
See https://github.com/jodh-intel/kata-containers/pull/7#discussion_r958579583 for more comments on tests.

closing in favor of https://github.com/kata-containers/kata-containers/issues/5919


created arshGoyal.txt

Added a little bit text in arshGoyal.txt

Good Work :+1:
Use conventional commits for message


[semantic-sil] Create unmanaged_autorelease_value.

[semantic-sil] Create unmanaged_autorelease_value.
This is an autorelease for use with Builtin.autorelease that does not need to be
balanced as part of the ownership model.
rdar://29791263

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge


Setup a ReadTheDocs documentation

A detailled ReadTheDocs documentation (an example can be found here: Quantil ReadTheDocs - Source Code ) would be helpful to have an overview documentation page for all patternatlas repos

moved away to seperate repository: https://github.com/PatternAtlas/pattern-atlas-readthedocs

moved away to seperate repository: https://github.com/PatternAtlas/pattern-atlas-readthedocs

can be found at: https://pattern-atlas-readthedocs.readthedocs.io/en/latest/


clusternet-hub error log installed by helm

helm install clusternet-hub -n clusternet-system --create-namespace clusternet/clusternet-hub

What happened:
E0301 09:59:42.401091       1 ???:1] failed to transform Manifest : Internal error occurred: unexpected end of JSON input
E0301 09:59:42.400537       1 ???:1] failed to transform Manifest : Internal error occurred: unexpected end of JSON input
E0301 09:59:42.401130       1 ???:1] failed to transform Manifest : Internal error occurred: unexpected end of JSON input
E0301 09:59:42.401130       1 ???:1] failed to transform Manifest : Internal error occurred: unexpected end of JSON input
I0301 09:59:44.130025       1 httplog.go:89] "HTTP" verb="GET" URI="/apis/shadow/v1alpha1" latency="14.325361ms" userAgent="Go-http-client/2.0" srcIP="<IP_ADDRESS>:52624" resp=200
I0301 09:59:44.482383       1 httplog.go:89] "HTTP" verb="GET" URI="/apis/proxies.clusternet.io/v1alpha1" latency="936.159µs" userAgent="Go-http-client/2.0" srcIP="<IP_ADDRESS>:52670" resp=200
E0301 09:59:44.560200       1 ???:1] failed to transform Manifest : Internal error occurred: unexpected end of JSON input

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Clusternet version: v0.7.0

Clusternet-agent version (user clusternet-agent --version=json):
Clusternet-hub version (user clusternet-hub --version=json): v0.7.0

Kubernetes version (use kubectl version): v1.21.5
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Others:

@chestarss Not able to reproduce this. Would you please share more details on this? Thanks.


Factorize and add "isFound" methods

Concerning:
MolComponentImpl>>isNotFoundXXX
MolNotFoundXXX>>isNotFoundXXX

Add:
MolComponentImpl>>isFoundXXX
MolNotFoundXXX>>isFoundXXX

For services, events and parameters.
And propose a Trait to factorize this behavior.

Starting in #150


variables with underscore not found

I have this simple http call
@blue_host="localhost"
@USERNAME="admin"
@PASSWORD="p"

###

POST http://{{blue_host}}/api/token
content-type: application/x-www-form-urlencoded

username={{USERNAME}}&password={{PASSWORD}}

But if I try to run it, I get an error
The variable 'blue_host' was not found in the document or in the environment. Returning the string as received ...

Seems that it doesn't like underscores in variable names..
Changed from blue_host to bluehost and it worked

Is reproducible. Thanks for reporting. Will fix!


New App: Sandfly Security

Name of the app
Sandfly Security (sandflysecurity)
Integration
Integrates with Sandfly Security - Agentless Linux Security (https://www.sandflysecurity.com/)
About
Integrates with the Sandfly Security Server to initiate a Sandfly scan of a specified system in order to track down intruders, malware, an other security breaches.

Repo created at: https://github.com/splunk-soar-connectors/sandflysecurity


Individual Column Searching

I am currently using EFDatatable and I think it is an excellent project. I would like to inquire if you support individaul column searching [(https://datatables.net/examples/api/multi_filter.html).

Assuming you’re using version 1.10 of the Datatables, you would need to install the Search API.
Have a look here:  https://datatables.net/reference/api/column().search()

Hi @tekoscott It is not support yet but it is really good idea. I think we will develop it as soon as possible. Thanks

Hi @ekondur thanks for that. And once again thanks for an excellent project. This is now my goto tool for creating datatables with MVC. Keep up the good work.

Hi @tekoscott I released this issue with release 1.3.0
Please update nuget package and use it :)
Thanks for contribution.


Peak frequency calculation

Have you figured out how to calculate peak frequency?

Group 4 has done it. I'm uncertain whether that approach is the correct one. You can have a look.

There is a function in our repo, file name fft.py


[Program]: Write a R program to convert octal to decimal

Description
Write a R program to convert octal to decimal
Octal is a base-8 number system. It uses only eight digits: 0, 1, 2, 3, 4, 5, 6, and 7. The digits are called octits. An octit is the smallest unit of information in a computer. An octit can be used to represent a single value, such as 0, 1, 2, 3, 4, 5, 6, or 7, or it can be used to represent a single state, such as on or off.
To convert an octal number to decimal, we can use the following algorithm:

Multiply the first octit by 8^0.
Multiply the second octit by 8^1.
Multiply the third octit by 8^2.
Multiply the fourth octit by 8^3.
Repeat steps 3 and 4 until the last octit.
Add the results of all the multiplications.
The decimal number is the sum of all the multiplications.

Input  : 12
Output : 10

How to contribute

Save the solution in program/program/convert-octal-to-decimal/convert_octal_to_decimal.r file

!assign


[GSSoC'23] Wordle Clone

Describe the project you want to add with tech stack
Wordle is a popular word-guessing game where players try to guess a five-letter word by making guesses and receiving feedback on the correctness of their guesses. I'm gonna build it with HTML, CSS, JS, Rapid API, and Dictionary API.
Expected behavior
Players will be presented with a blank word and an input field where they can enter their guesses. After submitting a guess, the game will provide feedback on the correctness of the guess. The feedback may include information on correct letters in the correct positions and correct letters in the wrong positions. The player continues making guesses until they successfully guess the five-letter word or run out of attempts.
Screenshots (optional)

Additional context (optional )
Assign me this project under GSSoC'23.

can u assign me this isssue


Fixes speed issues with iperf3 3.1.3

With a 25Gb Mellanox ConnectX-6 NIC on both sides, I can transfer at ~3.13+ GB/s which is the same as 25Gb using SMB. Using iperf3, I seem to be pretty restricted and am not sure why.
This 3.16 release is better than the 3.13 release though.
IPs: <IP_ADDRESS> and <IP_ADDRESS> on Linux and <IP_ADDRESS> and <IP_ADDRESS> on Windows.  I tripled-checked just to make sure I was seeing the versions correctly.  Seems like transfers to my Windows box as an iperf3 server are much faster than reverse mode.

You are referring to iperf3 version 3.13 and 3.1.3 randomly in above issue. These are 2 different versions. Last official windows binary release was 3.1.3 and that was 7 years ago. Since then there are lot of improvements and bug fixes in iperf3. I assume you meant the 3.1.3 rather than 3.13
You should try running the client with parallel streams using the -P option. And check the row with [SUM]. Something like:
iperf3.exe -c <IP> -P 10
The biggest change from version 3.15 to 3.16 is utilization of multiple threads. Quoted below from official source code release:

Multiple test streams started with -P/--parallel will now be
serviced by different threads. This allows iperf3 to take
advantage of multiple CPU cores on modern processors, and will
generally result in significant throughput increases

I got consistent results at around 18-20 Gb/s for both upload and download (Reverse mode) using parallel streams. I do not know what kind of network setup you have. If it is direct Ethernet transfer or you are using some RAID controller. (May be you need to fine tune your NIC configuration)
I just compile and release the windows binaries from official source code, as it is without any change in the code. They do not release binaries anymore for any platform. Unix based compilation is straight forward. You can check it here in the official source code repo:
https://github.com/esnet/iperf/blob/master/docs/obtaining.rst
If you think there is an issue. You should open it in the official iperf3 source code repo at:
https://github.com/esnet/iperf

My bad. Updated the title to 3.1.3.
My topology for these to machines:
Windows 25Gb Fiber <-> 25Gb port on Switch <-> 25Gb port on Switch <-> TrueNAS SCALE 25Gb Fiber on my own Epyc server hardware that's roughly the same as my PC Ryzen setup.
And there are 2 x SFP28 connectors per machine, but no LAGG or combined adapters right now. All 4 have IPs on the network and are connected to the same switch.
Windows -> Linux
[ ID] Interval           Transfer     Bitrate
[  5]   0.00-10.01  sec  2.89 GBytes  2.48 Gbits/sec                  receiver
[  8]   0.00-10.01  sec  2.89 GBytes  2.48 Gbits/sec                  receiver
[ 10]   0.00-10.01  sec  2.90 GBytes  2.49 Gbits/sec                  receiver
[ 12]   0.00-10.01  sec  2.89 GBytes  2.48 Gbits/sec                  receiver
[ 14]   0.00-10.01  sec  2.88 GBytes  2.47 Gbits/sec                  receiver
[ 16]   0.00-10.01  sec  2.88 GBytes  2.47 Gbits/sec                  receiver
[ 18]   0.00-10.01  sec  2.89 GBytes  2.48 Gbits/sec                  receiver
[ 20]   0.00-10.01  sec  2.90 GBytes  2.49 Gbits/sec                  receiver
[ 22]   0.00-10.01  sec  2.86 GBytes  2.46 Gbits/sec                  receiver
[ 24]   0.00-10.01  sec  2.89 GBytes  2.48 Gbits/sec                  receiver
[SUM]   0.00-10.01  sec  28.9 GBytes  24.8 Gbits/sec                  receiver

Windows -> Linux (Reverse mode)
[ ID] Interval           Transfer     Bitrate         Retr
[  5]   0.00-10.01  sec  2.31 GBytes  1.99 Gbits/sec    0             sender
[  8]   0.00-10.01  sec  2.31 GBytes  1.99 Gbits/sec    0             sender
[ 10]   0.00-10.01  sec  2.31 GBytes  1.99 Gbits/sec    0             sender
[ 12]   0.00-10.01  sec  2.31 GBytes  1.99 Gbits/sec    0             sender
[ 14]   0.00-10.01  sec  2.31 GBytes  1.99 Gbits/sec    0             sender
[ 16]   0.00-10.01  sec  2.31 GBytes  1.99 Gbits/sec    0             sender
[ 18]   0.00-10.01  sec  2.31 GBytes  1.99 Gbits/sec    0             sender
[ 20]   0.00-10.01  sec  2.31 GBytes  1.99 Gbits/sec    0             sender
[ 22]   0.00-10.01  sec  2.31 GBytes  1.99 Gbits/sec    0             sender
[ 24]   0.00-10.01  sec  2.31 GBytes  1.98 Gbits/sec    0             sender
[SUM]   0.00-10.01  sec  23.1 GBytes  19.9 Gbits/sec    0             sender

Linux -> Windows
[ ID] Interval           Transfer     Bitrate
[  5]   0.00-10.00  sec  1.93 GBytes  1.66 Gbits/sec                  receiver
[  8]   0.00-10.00  sec  11.5 GBytes  9.87 Gbits/sec                  receiver
[ 10]   0.00-10.00  sec  1.90 GBytes  1.63 Gbits/sec                  receiver
[ 12]   0.00-10.00  sec  1.95 GBytes  1.67 Gbits/sec                  receiver
[ 14]   0.00-10.00  sec  1.90 GBytes  1.63 Gbits/sec                  receiver
[ 16]   0.00-10.00  sec  1.92 GBytes  1.65 Gbits/sec                  receiver
[ 18]   0.00-10.00  sec  1.94 GBytes  1.66 Gbits/sec                  receiver
[ 20]   0.00-10.00  sec  1.93 GBytes  1.66 Gbits/sec                  receiver
[ 22]   0.00-10.00  sec  1.91 GBytes  1.64 Gbits/sec                  receiver
[ 24]   0.00-10.00  sec  1.95 GBytes  1.68 Gbits/sec                  receiver
[SUM]   0.00-10.00  sec  28.8 GBytes  24.8 Gbits/sec                  receiver

Linux -> Windows (Reverse mode)
[ ID] Interval           Transfer     Bitrate
[  5]   0.00-10.00  sec  3.21 GBytes  2.76 Gbits/sec                  sender
[  8]   0.00-10.00  sec  1.55 GBytes  1.33 Gbits/sec                  sender
[ 10]   0.00-10.00  sec  3.21 GBytes  2.76 Gbits/sec                  sender
[ 12]   0.00-10.00  sec  3.21 GBytes  2.76 Gbits/sec                  sender
[ 14]   0.00-10.00  sec  3.22 GBytes  2.76 Gbits/sec                  sender
[ 16]   0.00-10.00  sec  3.22 GBytes  2.76 Gbits/sec                  sender
[ 18]   0.00-10.00  sec  1.66 GBytes  1.43 Gbits/sec                  sender
[ 20]   0.00-10.00  sec  3.20 GBytes  2.74 Gbits/sec                  sender
[ 22]   0.00-10.00  sec  3.21 GBytes  2.76 Gbits/sec                  sender
[ 24]   0.00-10.00  sec  3.19 GBytes  2.74 Gbits/sec                  sender
[SUM]   0.00-10.00  sec  28.9 GBytes  24.8 Gbits/sec                  sender

"Windows -> Linux (Reverse mode)" is the only one where I don't get the full bandwidth.
Figured you might know something, but if you're just compiling those builds, I guess there's nothing you could do.

Ah, great point! Linux has 3.12:
# iperf3 -v
iperf 3.12 (cJSON 1.7.15)
Linux storeman 6.1.55-production+truenas #2 SMP PREEMPT_DYNAMIC Tue Oct 31 16:07:08 UTC 2023 x86_64
Optional features available: CPU affinity setting, IPv6 flow label, SCTP, TCP congestion algorithm setting, sendfile / zerocopy, socket pacing, authentication, bind to device, support IPv4 don't fragment

20Gb is fine, but why can I do 25Gb from Linux to Windows if Windows is hosting the iperf3 server rather than if Linux is hosting the server with Reverse mode? That's the thing that doesn't make sense. Both are sending data to Windows with iperf3.
Is there a difference of making Windows the server rather than the client?

Possible reason could be different buffer sizes are used. Use -V (--verbose) option in iperf3 command on the client to view buffer size in both cases. Is it same?


Clean up containers/images on Windows CI executor

Intending to extract this to a cron workflow or something we can just run on demand, not something we do on every build (for @bsideup's network bandwidth's sake)...

/azp run

/azp run

/azp run

/azp run


Pretrained checkpoints are not downloadable

Google disk doesn't allow to download pretrained weights files due to too much downloads. Is it possible to upload them to other filestorage?

Today the links work just ok ^)


return 0 in case maxPoints is null in the database

Motivation and Context
Fix sentry issue ARTEMIS-11X
Description

Cannot invoke "java.lang.Integer.doubleValue()" because the return value of "de.tum.in.www1.artemis.domain.exam.Exam.getMaxPoints()" is null.

My solution: check if maxPoints is null and if that's the case return 0

Question: do you know, why this occurs? And could you reproduce the error? I am wondering in which situations this.maxPoints can be undefined? Is it reasonable to call this method in the first place if the points are null? Maybe the fix could also be to disallow calling calculateExamScores in such situations...

Question: do you know, why this occurs? And could you reproduce the error? I am wondering in which situations this.maxPoints can be undefined? Is it reasonable to call this method in the first place if the points are null? Maybe the fix could also be to disallow calling calculateExamScores in such situations...

The crash happens when one wants to access the scores of an exam, that has no maximum points defined.

Should we allow creation of an exam without maximum points or is it a bug, @krusche ?

It probably makes sense to make sure that maxPoints has a value > 0 when the instructor creates the exam, because a couple of other features (e.g. grade scales, and some validations later in the process) depend on it. In case the instructor does not know it, the value could also be changed afterwards.


Sync eng/common directory with azure-sdk-tools for PR 1109

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/1109
See eng/common workflow

/check-enforcer override

/check-enforcer override


nixos/plausible: add package option

Description of changes
I wanted to deploy the fix at  #220246 by referencing the nixpkgs channel directly from the service, but there was no option for that. Hence this PR: I tested it directly on the system where I needed the change.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

I'm sorry, I commited roughly on the same time as you posted your suggestion. I corrected this in the latest commit.

Can you please squash those 3 commits into a single titled nixos/plausible: add package option? :)

It should be ok!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/911

I wanted to deploy the fix at #220246 by referencing the nixpkgs channel directly from the service, but there was no option for that.

Can't you do this?
nixpkgs.overlays = self: super: {
  inherit (import <nixpkgs-unstable> { }) plausible;
};

I don't know much about how overlays work, so I didn't consider using one. I tried to adapt your overlay configuration to my flake with the following:
let
  pkgs = nixpkgs.legacyPackages.${system}.extend (self: super: {
    inherit (import nixpkgs-unstable.legacyPackages.${system} { }) plausible;
  });
in
...

But I kept encountering the issue fixed by #220246, so my attempts did not actually change anything.


optimize for large selections

Now if selected lots of nodes, plugin runs for long

I'll try parallel execution of each node

Can't achieve speed up, because Figma runs in one and only thread. Parallelism could be achieved through web workers, but I can't use them in the project, as they can't see TextNodes to manipulate them.


Update comments/names in the HardFork code

I think the only potentially objectionable content is the "rename defaultSafeZone to noLowerBoundSafeZone" commit. Let me know and I'll drop it.

bors merge


Installing on a Kubernetes Cluster instead on minikube fails

I test a setup of Kubernetes with helm and have chosen to papermerge as my test programm. Why? Because it is a real life application which we might want to use  with Kubernetes.
The value.yaml gives me the hint, that this helm chart was only tested with minikube?
If I try to install after I have set the secrets and disabled ingress in value.yaml, the installation fails.
What I expect to happen
helm install papermerge . -f ./values.yaml

to result in a installation of the pods.
What actually happens
The command returns the following error:
Error: INSTALLATION FAILED: template: papermerge/templates/configmap.yaml:13:35: executing "papermerge/templates/configmap.yaml" at <.Values.global.conf.redis.host>: nil pointer evaluating interface {}.host

@MaKaNu if you are still struggling, you need to provide secrets and other information like the redis host.
worker:
  replicaCount: 1

global:
  conf:
    app:
      django_settings_module: config.settings
      superuser:
        username: "admin"
        email<EMAIL_ADDRESS>      media_dir: "/mount/media"
    ingress:
      enabled: true
      host: papermerge.minikube
    db:
      type: "postgres"
      user: "postgres"
      name: "postgres"
    es:
      enabled: false
    redis:
      host: "<IP_ADDRESS>"
  secrets:
    app:
      secret_key: "abc"
      superuser:
        password: "abc"
    db:
      password: "abc"


when i execute test_hsemotion_package.ipynb, getting an error below.

when i execute test_hsemotion_package.ipynb, getting an error below.
'EfficientNet' object has no attribute 'act1'
Kindly help

The issue may be caused by incompatible version of timm package. Could you kindly verify that you have latest version 0.6.7? If so, please, send me more detailed stack trace of your exception. Also, try to run the code from https://github.com/HSE-asavchenko/face-emotion-recognition/blob/main/src/display_emotions.ipynb which does not need the hsemotion package

Thanks [display_emotions.ipynb is working fine. Closing the issue


💡 [REQUEST] - Add Testing feature in src/pages Folder likewise with Previous added Feature Ones

[x] I am from GSSoC23.
[x] I would like to be assigned this task.

fill [x] to check the boxes.

Section
Description

Concise Description
Add Testing feature in src/pages  likewise added Previous Feature Ones

Solution Proposed
In Every Software Development Life Cycle, Before Production Stage , Conduct Testing due to related Software Requirements. That's why Testing is necessary thing for Development Stage

Alternative Solutions
Describe any alternative solutions you can think of

Additional Context
Without Testing, Software Product does not produce Error/Bug Free Services and Features

Screenshots
[Provide screenshots here]

Code of Conduct
By submitting this issue, you agree to follow our Code of Conduct.

[x] I follow the Contributing Guidelines of this project.
[x] I have checked the current issues for duplicate problems.
[x] I have read the readme of this repository.

@bhartik021 . Why added Bug . This is Feature Related issue

Hi @Astrodevil and @Alkaison . Please Review on this ISSUE as soon as Possible. Thank You !

How will you implement Testing into website deployment using Astro framework? Let us know more.

How will you implement Testing into website deployment using Astro framework? Let us know more.

@Alkaison . I am Telling Testing Category is add along with Other Existed Categories like Computer Networks, Computer Networks etc.
But, I have No Deep Knowledge on Astro Frame work. Opinion is Yours?

Hi @Alkaison . Any Update on this ?

@NarendraKoya999 Go ahead ✅


java uses or overrides a deprecated API.

E:\Dev. Requirement\RD Android\tlb_assistant\tlb_assistant>flutter build apk --release
Initializing gradle...                                              2.4s
Resolving dependencies...                                           7.8s
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
Note: D:\Flutter\flutter.pub-cache\hosted\pub.dartlang.org\firebase_admob-0.8.0+3\android\src\main\java\io\flutter\plugins\firebaseadmob\AdRequestBuilderFactory.java uses or overrid
es a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: D:\Flutter\flutter.pub-cache\hosted\pub.dartlang.org\firebase_messaging-4.0.0+2\android\src\main\java\io\flutter\plugins\firebasemessaging\FlutterFirebaseInstanceIDService.jav
a uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: D:\Flutter\flutter.pub-cache\hosted\pub.dartlang.org\fluttertoast-3.0.3\android\src\main\java\io\github\ponnamkarthik\toast\fluttertoast\FluttertoastPlugin.java uses or overri
des a deprecated API.
Running Gradle task 'assembleRelease'... Done                     149.1s (!)
Built build\app\outputs\apk\release\app-release.apk (9.4MB).
D:\Flutter\flutter\bin\flutter.bat doctor --verbose
[√] Flutter (Channel stable, v1.2.1, on Microsoft Windows [Version 10.0.17134.590], locale en-IN)
• Flutter version 1.2.1 at D:\Flutter\flutter
• Framework revision 8661d8aecd (8 weeks ago), 2019-02-14 19:19:53 -0800
• Engine revision 3757390fa4
• Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)
[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at D:/SDK
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• ANDROID_HOME = D:/SDK
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[√] Android Studio (version 3.3)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 34.0.1
• Dart plugin version 182.5215
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
[!] Connected device
! No devices available
! Doctor found issues in 2 categories.
Process finished with exit code 0

Is this causing an issue? Because that's just a note/warning, we'll need to wait for flutter team update java classes for this plugins.
It doesn't stop you to build the app, at least.

Hi, sorry for the delay, I just want to check if you could solve this problem.

Hi @aarifali
for what I can see the issue is related to a FlutterFire plugin,
please open the issue in the dedicated github.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Add RM parser readme

Coverage remained the same at 51.774% when pulling 84fba0f7fdc435998da880d72f843732a706515a on zimiao/hadoop_resourcemanager_parser_do into af921fac0e5b1f79fa4c17e822fa12a07ba4717c on master.


calling move to page from anywhere produces error

//ScrollViewController has two vc red vc and blue vc
  func movetopage(){
     self.moveToPage(1, animated: true)
   }

//inside datasource
  func viewControllerData() -> [UIViewController] {
    return [redvc, blueVC]
  }

tried calling the function from child vc
Thread 1: Fatal error: Unexpectedly found nil while unwrapping an Optional value
    public func moveToPage(_ index: Int, animated: Bool) {
        let currentIndex = stackPageVC.index(of: currentStackVC)!//error comes here

        var direction: UIPageViewControllerNavigationDirection = .reverse

       if index > currentIndex {
        direction = .forward
    }

    datasource?.changedToPageIndex?(index)
    currentStackVC = stackPageVC[index]

      pageViewController.setViewControllers([currentStackVC], direction: direction, animated: animated, completion: nil)
  }
  }

and tried calling from scrollview controller also inside view did load  and got the same error

class ScrollViewController: EZSwipeController,EZSwipeControllerDataSource {
// let array = [SignUpViewController(),SignInViewController()]
override func setupView() {
    datasource = self
    self.moveToPage(1, animated: true)

}


Kafka stream state stores cannot be found after upgrade to 3.2.5

Describe the issue
The following code works with SCS 3.2.4:
@Service
class TechnischerplatzAdresseHandler(private val interactiveQueryService: InteractiveQueryService) {
    private val technischerplatzAdresseStore: ReadOnlyKeyValueStore<String, TechnischerplatzAdresse>
        get() = interactiveQueryService.getStore("technischerplatzadresse-store")

    suspend fun get(request: ServerRequest) = withContext(Dispatchers.IO) {
        technischerplatzAdresseStore["1234"]?.let {
            ServerResponse.ok().build().awaitSingle()
        } ?: ServerResponse.notFound().buildAndAwait()
    }
}

But it raises an exception with SCS 3.2.5:
Caused by: org.apache.kafka.streams.errors.UnknownStateStoreException: Cannot get state store technischerplatzadresse-store because no such store is registered in the topology.
	at org.apache.kafka.streams.KafkaStreams.store(KafkaStreams.java:1603)
	at org.springframework.cloud.stream.binder.kafka.streams.InteractiveQueryService.getStateStoreFromKafkaStreams(InteractiveQueryService.java:121)
	at org.springframework.cloud.stream.binder.kafka.streams.InteractiveQueryService.lambda$getQueryableStore$0(InteractiveQueryService.java:109)
	at org.springframework.retry.support.RetryTemplate.doExecute(RetryTemplate.java:329)
	at org.springframework.retry.support.RetryTemplate.execute(RetryTemplate.java:209)
	at org.springframework.cloud.stream.binder.kafka.streams.InteractiveQueryService.getQueryableStore(InteractiveQueryService.java:88)
	at at.wrwks.bef.projekt.domain.technischerplatzadresse.TechnischerplatzAdresseHandler$get$2.invokeSuspend(TechnischerplatzAdresseHandler.kt:39)

To Reproduce
Try to load a state store as described
Version of the framework
Spring Cloud 2020.0.4 2021.0.4 with Spring Cloud Stream 3.2.5
Expected behavior
Screenshots
Additional context
With debugging I got down to StreamsMetadataState#isInitialized() where clusterMetadata.topics().isEmpty() == true which it shouldn't be. It is located in kafka-streams:3.1.2 on line 321.
The topology hasn't changed and the store is there.

@andrashatvani Do you mean Spring Cloud version 2021.0.4? Because 2020.0.x is based on Boot 2.4 and Spring Cloud Stream 3.1.x etc. There we do not see the issue. We were able to reproduce what you see on 3.2.5 which looks like some sort of regression issue. We will investigate and get back to you on this. Thanks for reporting.

@sobychacko Yes, I meant 2021.0.4 - corrected in the original description too.

Hi @andrashatvani ,
We have identified the issue and are working on a fix.
In summary, we are relying on the StreamsMetadataState to actually have the metadata. However, as you pointed out above, it thinks it is not initialized (which forces a short circuit empty return). The StreamsMetadataState.topologyMetadata does in fact have the source topics for the store. This could be a bug in KS but even if it were we would not want to wait on that fix. The plan is to adjust the code here and then determine if there is a bug to file in KS. If there is a bug, we would consider re-adjusting this code to use the StreamsMetadataState once fixed.

Hi @andrashatvani
As a workaround you should be able to set the application.server property.
spring.cloud.stream.kafka.streams.binder.configuration.application.server=<server>:<port>

Typically this is only needed in a multi-instance KafkaStreams application but the mechanism we are using to distinguish the state stores requires this to be set in all cases (that is part of the bug).
Let me know if this unblocks you.

@onobc Thanks for the update. As we have dozens of services we'll stick to 2021.0.3.

The issue was reported against 3.2.x and was fixed in main. Re-opening until fix is backported to 3.2.x.


Being able to open links on app to add series easier

Describe why this should be added
On share menu or open app menu there is an option to open link and that would send you to that seri on that site's source

Like in these photos it would be good/foolproof if its both of them
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open (or closed) issue.
[X] I have written a short but informative title.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I have updated the app to version 0.13.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

"Open link with aniyomi" has to be implemented in each respective extension, so make an issue for each extension you want this to happen to

Clarification

Another one


Split up the main README

💡 Summary
Split up the main README into separate files.
Motivation and context
The README is growing large and unwieldy. Splitting it into smaller files will improve readability and maintainability.
Implementation notes
See ScubaGear's README for reference: https://github.com/cisagov/ScubaGear/tree/main
Acceptance criteria

[ ] The README has been split into different files, using a structure similar to ScubaGear's documentation.

Linking #283 since it is a readme related issue


https://cvxpy.org/ security problem

I'm getting "Firefox detected a potential security threat and did not continue to cvxpy.org.  ...  The certificate is only valid for the following names: *.github.com, www.github.com, github.io, github.com, *.github.io, githubusercontent.com, *.githubusercontent.com".

I can replicate. I think this is the issue: https://github.com/hossain-khan/hossainkhan.com/issues/37
My understanding based on this issue is that we can either make https://www.cvxpy.org work or https://cvxpy.org but not both. If you just type in cvxpy.org now it directs to https://www.cvxpy.org

If this is really the case, I would suggest making cvxpy.org work, and forget about www.


Updated: postman-canary 7.1.1-canary01

Newer version of postman-canary (7.11.0-canary01) is now available in master.


feat: Modify Withdrawal Transaction for Unbonded Validators

Description
I added a check to make sure every withdrawal transaction has an amount equal to the stake amount to prevent spam attacks.
Related issue(s)

Fixes #464

@kehiy
Hi, what is the final status of the PR? as long as time goes, the conflicts will increase, it would be better to fix issues and merge PR.
thanks

Hi @amirvalhalla,
We need more research and discussion with @b00f about this.
About conflicts and other stuff don't worry.
We will merge/close it after discussion and researches.
Thanks for reminding us (:


MediumConcurrencyTest should not be flaky

Example: https://github.com/eclipse/rdf4j/actions/runs/4194448129/jobs/7272627465
I have also seen this failing other PRs for totally unrelated changes.
Here is the failing test stacktrace - in case GH workflow logs are subject to retention:
Error:  Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 1.312 s <<< FAILURE! - in org.eclipse.rdf4j.federated.MediumConcurrencyTest
Error:  org.eclipse.rdf4j.federated.MediumConcurrencyTest.queryMix  Time elapsed: 1.269 s  <<< ERROR!
java.util.concurrent.ExecutionException: org.eclipse.rdf4j.sail.SailException: Connection closed before all iterations were closed.
	at java.base/java.util.concurrent.FutureTask.report(FutureTask.java:122)
	at java.base/java.util.concurrent.FutureTask.get(FutureTask.java:205)
	at org.eclipse.rdf4j.federated.MediumConcurrencyTest.lambda$queryMix$0(MediumConcurrencyTest.java:69)
	at org.junit.jupiter.api.AssertTimeout.lambda$assertTimeoutPreemptively$4(AssertTimeout.java:138)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:317)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642)
	at java.base/java.lang.Thread.run(Thread.java:1589)
Caused by: org.eclipse.rdf4j.sail.SailException: Connection closed before all iterations were closed.
	at org.eclipse.rdf4j.sail.helpers.AbstractSailConnection.close(AbstractSailConnection.java:287)
	at org.eclipse.rdf4j.repository.sail.SailRepositoryConnection.close(SailRepositoryConnection.java:248)
	at org.eclipse.rdf4j.federated.evaluation.iterator.CloseDependentConnectionIteration.handleClose(CloseDependentConnectionIteration.java:77)
	at org.eclipse.rdf4j.common.iteration.AbstractCloseableIteration.close(AbstractCloseableIteration.java:52)
	at org.eclipse.rdf4j.federated.evaluation.concurrent.ParallelTaskBase.close(ParallelTaskBase.java:117)
	at org.eclipse.rdf4j.federated.structures.QueryInfo.closeScheduledTasks(QueryInfo.java:260)
	at org.eclipse.rdf4j.federated.structures.QueryInfo.close(QueryInfo.java:223)
	at org.eclipse.rdf4j.federated.evaluation.iterator.StopRemainingExecutionsOnCloseIteration.handleClose(StopRemainingExecutionsOnCloseIteration.java:70)
	at org.eclipse.rdf4j.common.iteration.AbstractCloseableIteration.close(AbstractCloseableIteration.java:52)
	at org.eclipse.rdf4j.common.iteration.Iterations.closeCloseable(Iterations.java:189)
	at org.eclipse.rdf4j.common.iteration.IterationWrapper.handleClose(IterationWrapper.java:127)
	at org.eclipse.rdf4j.sail.helpers.SailBaseIteration.handleClose(SailBaseIteration.java:55)
	at org.eclipse.rdf4j.common.iteration.AbstractCloseableIteration.close(AbstractCloseableIteration.java:52)
	at org.eclipse.rdf4j.common.iteration.Iterations.closeCloseable(Iterations.java:189)
	at org.eclipse.rdf4j.common.iteration.IterationWrapper.handleClose(IterationWrapper.java:127)
	at org.eclipse.rdf4j.common.iteration.AbstractCloseableIteration.close(AbstractCloseableIteration.java:52)
	at org.eclipse.rdf4j.federated.FedXBaseTest.execute(FedXBaseTest.java:99)
	at org.eclipse.rdf4j.federated.MediumConcurrencyTest.lambda$submit$2(MediumConcurrencyTest.java:85)
	... 4 more

Seems related #3317

Related to: https://github.com/eclipse/rdf4j/issues/4242

This is a bug in the federation module where at some point a connection is closed before all open iterations are closed. I've fixed a number of these bugs already and I was really hoping that I can caught everything. The bugs are quite hard to find and fix.


Replacing of header separator with CRLF in multipart

According to RFC1521 (see 7.2.1. Multipart:  The common syntax), header separator must be a CRLF char sequence, so \newline in multipart.clj should be replaced with \r\n (\newline is also platform-specific).
Current implementation leads to a problem with e.g. Apache Commons Multipart. The entire body is interpreted as a header that results in org.apache.commons.fileupload.MultipartStream$MalformedStreamException.

Note that the encapsulation boundary must occur at the beginning of a
line, i.e., following a CRLF, and that the initial CRLF is considered
to be attached to the encapsulation boundary rather than part of the
preceding part.  The boundary must be followed immediately either by
another CRLF and the header fields for the next part, or by two
CRLFs, in which case there are no header fields for the next part
(and it is therefore assumed to be of Content-Type text/plain).

Are both of the bolded cases tested for here? I wasn't quite sure, but thought I'd mention it just in case.

In the current imlpementation we always have at least Content-Disposition and Content-Type headers for each part. Thus, in our case only the first possibility exists, and two CRLFs may only mean that there are no more header fields. The second CRLF is being added here.

Great, I thought that was probably being handled but wanted to mention it just in case. Thanks!


test_lsn_lease_size flaky

https://neon-github-public-dev.s3.amazonaws.com/reports/pr-8291/9811816038/index.html#suites/d70040d097c5c7c438e421e0416d69ce/8b9d1e05e6c02925
AssertionError: assert 312497288 == 312316992 ± 3.3e+04
  comparison failed
  Obtained: 312497288
  Expected: 312316992 ± 3.3e+04

cc @yliang412 do you have any idea or I/we can look into it next week?

Tried to resolve by having a larger threshold in #8282, but I feel this won't fix the root cause.


markup: Handle attribute lists in code fences

Fixes #8278

Note that this particular PR only fixes this when you have CodeFences enabled (default). I'll let the upstream issue track the rest.


Official support and documentation on how to use alongside Devise for APIs

I've been searching for the most secure way to authenticate my users both on my website and on clients using the API on the same Rails application project. Since I'm not going to use a single-page application but a 'normal' Rails application instead I've found Devise to be a good solution for authenticating users on my website.
But when thinking about a secure API I found this gem to be a very secure solution – and dozens of people seem to agree, see the following issues as examples:
https://github.com/lynndylanhurley/devise_token_auth/issues/343, https://github.com/lynndylanhurley/devise_token_auth/issues/36, https://github.com/lynndylanhurley/devise_token_auth/issues/265, https://github.com/lynndylanhurley/devise_token_auth/issues/120
But the problem ist that there seems to be a lack of support or at least no documentation on how to solve this issue correctly. Some of the people linked in the issues above seem to have found a solution for them – so basically it seems to work – but it would be great to be sure that everything is done right and won't break when this gem gets upgrades. So I'm hereby suggesting to introduce official support for this kind of usage – which would make this gem useful in much more contexts with minimal changes. Writing a wiki article may even suffice.
If you maintainers agree I'd love you to just provide a documentation (and changes if any needed) on how to use Devise alongside this gem to use this gem for an API kind of application. Please see the above linked issues as examples on how to it is desired to be used.
Thank you in advance for any help!
Update: Feel free to have a look into my initial approach at this state of my application. The interesting parts should be in app/controllers/api_controller.rb, app/controllers/application_controller.rb and config/routes.rb. Also I feel https://github.com/lynndylanhurley/devise_token_auth/issues/346 should also be clarified to make support complete.

Ok, I've just recognized there's the Can I use this gem alongside standard Devise? section in the README. But I wonder why #343 happened then and why #120 is still open. Also #265 is still around.

Basically, Devise add-on (vs full replacement) support has slowly been creeping in, but it's pretty deficient. If you'd like to approach native Devise support a bit more holistically and straighten out the docs to boot, I'm not going to stop you.  =]
Just need to make sure there's good test coverage in any PR, and that no backwards incompatible changes with token-based support are introduced.

Any ideas on how #265 could be fixed?

Hi there @Dschee    ,
In an effort to cleanup this project and prioritize a bit, we're marking issues that haven't had any activity in a while with a "close-in-7-days" label. If we don't hear from you in about a week, we'll be closing this issue. Obviously feel free to re-open it at any time if it's the right time or this was done in error!
If you are still having the issue (especially if it's a bug report) please refer to our new Issue Template to provide some more details to help us solve it.
Hope all is well.


Tweaking site config

ToC is displayed for 5 heading levels
Feedback section removed from the pages
Bit reorganizing the config file

/assign @adetalhouet @liamfallon

/approve
/lgtm


Windows 7 Aero Glas

On Windows 7 when I use this theme it removes Aero effect. As you can see this is default dark theme on Windows 7

And this is after applying shadowfox. It ads this black bar thing and it looks ugly.

Is it possible to remove it and keep Areo instead of this? It would look much better on Windows 7?

I think this is due to the change made in #198.
I don't have Windows to test on, but the code causing this is the following:
@-moz-document url-prefix(chrome://) {
  @media screen and (-moz-os-version:windows-win7) {
    window {
      background-color: var(--tone-9)!important;
      color: var(--in-content-page-color)!important
    }
  }
}

You could try adding an override in userChrome_customization.css and making the background-color none or transparent to see if that fixes it.

I'm not CSS master I did try and didn't work. Can you paste script with "none" color? Maybe I was doing wrong!

@-moz-document url-prefix(chrome://) {
  @media screen and (-moz-os-version:windows-win7) {
    window {
      background-color: none !important;
    }
  }
}

You could try replacing the word none with transparent as well.  After you add that to the userChrome_customization.css, make sure you re-run the updater script and restart Firefox.

Ok with word "transparent" its working and it looks awesome! Thanks! You should add this to a customization section for people on Windows 7 if they want Areo look back.

Thanks for the update.  I've added the tweak to the wiki:
https://github.com/overdodactyl/ShadowFox/wiki/More-Tweaks#windows-7-aero-theme---transparent-navbar


add: getFormData Date support.

@WildEgo do you want to have a look at the failing tasks + add notes what this is and what it changes/fixes?

@WildEgo do you want to have a look at the failing tasks + add notes what this is and what it changes/fixes?

I would love to but the project that I was using this on was changed to use Inertia for now as it made more sense, if I have some free time I might look at it but at the moment it's not a priority


Add EntityMut::get_mut_by_id_unchecked

Objective

Fixes: #15603

Solution

Add an unsafe get_mut_by_id_unchecked to EntityMut that borrows &self instead of &mut self, thereby allowing access to multiple components simultaneously.

Testing

a unit test function get_mut_by_id_unchecked was added.

This PR does what it says it does, but I would like a confirmation from ECS's SMEs that it's something we want

This is part of our sprawling matrix of "methods to work with components" that was just missed, and should be exposed for end users who are working with scripting.