"Error: composition is only allowed when selector is single", when trying to import react-toolbox components

Here is the console error which pops whenever I try to import a component from react-toolbox (eg import { AppBar, Checkbox, IconButton } from 'react-toolbox';
I tried digging a bit, and I think that I could fix that if it wern't for Happypack, which makes it harder for me to understand.
Any idea how to fix it? I did not touch a single webpack/other config file!

reference: https://github.com/react-toolbox/react-toolbox/issues/1359

Thanks for answering.
I've tried all solutions there, and in linked answers, and none worked. I'm on the feature/apollo branch, and just tried fixing it again from a fresh clone, did not work for me. I would be so grateful if someone helped out on this, (ideally clone as well and try to fix it ?) I really need to make this work :(

Could you make a sample branch so I could take a look on it?

Sure! Head over to https://github.com/PDS42/react-toolbox-issue and have a blast :)
To make things clear, I did manage to import some components without the build failing, such as import Button from 'react-toolbox/lib/button/Button', but 1) not all of them work for me and 2) I'm having trouble using similar libraries, so it must be related to something I'm doing wrong. But I really can't see it!  Thanks for taking the time anyways

@PDS42 sorry for late reply, to solve the issue there are some steps:

Implement css modules in your css-loader, because react-toolbox using that.
add postcss-loader to handle css import, also because react-toolbox will need that to compile their styles
Also, implement those loaders mentioned above for your compiled server bundle

As your reference to implement react-toolbox completely in this boilerplate, you could compare your current setup with my branch.
Note: don't use latest postcss-loader version, it is still buggy and will make react-toolbox component buggy, use postcss-loader v1.3.3

@diondirza I cloned your branch, and did the following:
=> created a new Menu folder in /shared/components/Menu
=>created new file index.js, with the following example from react-toolbox/components/Layout:
`import React, { Component } from 'react';
import { AppBar, Checkbox, IconButton } from 'react-toolbox';
import { Layout, NavDrawer, Panel, Sidebar } from 'react-toolbox';
class Menu extends Component {
constructor(props) {
super(props);
this.state = {
drawerActive: false,
drawerPinned: false,
sidebarPinned: false
};
}
toggleDrawerActive = () => {
    this.setState({ drawerActive: !this.state.drawerActive });
};

toggleDrawerPinned = () => {
    this.setState({ drawerPinned: !this.state.drawerPinned });
}

toggleSidebar = () => {
    this.setState({ sidebarPinned: !this.state.sidebarPinned });
};

render() {
    return (
        <Layout>
            <NavDrawer active={this.state.drawerActive}
                pinned={this.state.drawerPinned} permanentAt='xxxl'>

                    Navigation, account switcher, etc. go here.

            </NavDrawer>
            <Panel>
                <AppBar leftIcon='menu'/>

Main Content

Main content goes here.

                    <Checkbox label='Pin drawer' checked={this.state.drawerPinned} onChange={this.toggleDrawerPinned} />
                    <Checkbox label='Show sidebar' checked={this.state.sidebarPinned} onChange={this.toggleSidebar} />

            </Panel>
            <Sidebar pinned={ this.state.sidebarPinned } width={ 5 }>

<IconButton icon='close'/>

Supplemental content goes here.

            </Sidebar>
        </Layout>
    );
}

}
export default Menu;`
I only added a constructor and this.state=... instead of state = ....
=> added the import Menu from '../Menu'; line in DemoApp's index.js.
It throws me (which I already got and fixed on my own project, so no big deal.)
=> I commented the problematic functions to see if react-toolbox components alone still rendered, got
This is literally all I did, I truly do not understand.
As for the steps, I'm not comfortable enough with the environment to complete them on my own. I tried digging in configFactory and related config files, but I could not figure a solution out.
Do you think this could come from my OSX environment somehow?

No it is not. 1st error just change your babel preset to use stage 0. 2nd error its related on your css loader config.   The error really obvious to tell us that your css loader not implement css modules rules and postcss loader yet, therefore it can't process the css file because the file have composition syntax that normal css loader don't understand. See commit history to fully understand how to setup css loader in the right way.  You can see it clearly.

this [commit] (https://github.com/diondirza/react-universally/commit/d7e75b08f415c0bda9f383958d08a18a08d377da) should tell you clearly. There is also setting in config/value.js to be added for react toolbox to be excluded being process in server.

Thank you very much for all your help. I'll try this out tomorrow, but I finally understood what you meant with the commit! Much love, I'll let you know how it went but I'm pretty sure I can fix it now.


Add ThrottleAsyncSequence & AnyAsyncSequence

This PR adds ThrottleAsyncSequence & AnyAsyncSequence<Element> which allows to represent AsyncSequence in simmilar way as in Combine AnyPublisher without need to specify Concrete type of sequence.

Ah i see that you have added AnyAsyncSequenceable two days ago 🤓 So only ThrottleAsyncSequence make sense to add

Hey! Amazing thankyou! ThrottleAsyncSequence looks great!
Haha yer, I added AnyAsyncSequenceable and also AnyThrowingAsyncSequenceable for those sequences that can throw.
Imma just on holiday atm, but once back I'll have a proper look and merge in!

Hey! Amazing thankyou! ThrottleAsyncSequence looks great!
Haha yer, I added AnyAsyncSequenceable and also AnyThrowingAsyncSequenceable for those sequences that can throw.
Imma just on holiday atm, but once back I'll have a proper look and merge in!

Sure np. take a rest!

Closing this one


Revert "updated requirements.txt file to fix werkzeug bug"

Reverts PromotionsSquad/promotions#54

I should not have reverted, apologies, closing the pull request.


Passing data back to parent with Actions.pop?

It is easy to pass data through a push as seen in the example:
Actions.login({data:"Custom data", title:'Custom title' })
How should it be done when popping back to the parent?

The parent can just pass a callback function through props.

you can use Actions.refresh({data:"Custom data", title:'Custom title' }) immediately after Actions.pop() and this will do the trick. Refresh will be called on the parent

Actions.pop( {refresh: {variable: value} }) will do


AudioPluginHost implementation in Kotlin

We need AudioPluginHost implementation (not to be confused with JUCE AudioPluginHost) that host app developers can easily use without native hassle. Also useful for proof of concept implementation.
It does not have to provide fully-featured hosting implementation, and can be done partial migration from AAPHostSample step by step.
Design notes:

The host works as a client to AudioPluginServices. Services are apps, so one service handles multiple binder connections from multiple clients (hosts). But connections are (should be) made every time client requests. It is possible that many many channels can be connected between the same pair of a service and a client.
Right now there is no native API that can start and manage service connection as there is no way to query available AudioPluginServices. It will be handled via Kotlin AudioPluginHostHelper API.
The entire process of connecting to a service is based on asynchronous callback (Activity.onServiceConnected()). While the plugin framework API is synchronous, the actual implementation likely have to be asynchronous. Or maybe the API needs to transform into aynchronous style. The trends are towards asynchronous (e.g. Web MIDI API, Android MIDI API).

The API has been there for a while, but never actually used. Now it's going to be used by aaphostsample as of 8095959, but there is some buffer sharing issue between host and plugin, which results in bogus buffer passing. aapbarebonesample results in unexpected audio output, which means, parameters are not involved.

It can be closed as aaphostsample is based on Kotlin code now.


Access physical hostname in the k8s launched docker container entrypoint script

Hi,
Is there a way I can get hold of the underlying physical hostname within the docker container entrypoint script soon after it is launched? k8s doesn't seem to set any env variables to access this. Currently I use a hacky way of mounting /etc/hostname within the container and read the hostname from there. Also does /pods endpoint work since the pod may not be entirely setup at this stage.
Thanks!

The only way you can know the hostname is to query the Kubernetes API, for
now. In general, we'll say you should not ask that question - it leads to
couplings that you don't really want.
Once a pod is accepted by the API it is available through the API.
On Jul 28, 2015 9:28 AM, "Sanjana J Bhat"<EMAIL_ADDRESS>wrote:

Hi,
Is there a way I can get hold of the underlying physical hostname within
the docker container entrypoint script soon after it is launched? k8s
doesn't seem to set any env variables to access this. Currently I use a
hacky way of mounting /etc/hostname within the container and read the
hostname from there. Also does /pods endpoint work since the pod may not be
entirely setup at this stage.
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11932.

Please re-open if that doesn't answer your question.

I agree that using the hostname programmatically is probably a bad idea, but it can be valuable to annotate telemetry emitted from a container (metrics/logs) with a physical hostname, to facilitate identification of host-level issues. I'd be in favor of this feature.


Add hard shadows and make it more dark

You don't like the edges of shadows being soft, right? Okay I made them a bit sharper. The problem is that the way we express shadows can't take account of distance between objects and the ground, but that's what determines the shapness of shadows as explained in this page: https://blog.demofox.org/2017/07/01/why-are-some-shadows-soft-and-other-shadows-hard/
As for darkness I took a careful balance of it and I'm not going to change it for now. In the future when the game provides a way to configure shaders in game, I will definitely make a settings screen to change various aspects of my shaders. Please bear with the status quo until then.

A screenshot of the development version:


Not able to view files with a ~ (tilde) in their name.

My team and I are unable to view files with tilde's in the file name. We have a hack solution for ourselves for now, but I doubt that this is a proper fix for the problem.
The tilde gets escaped and comes out as "~" instead of "~".
We've added a line to part of the Symfony framework to do a str_replace to swap it out, but obviously that is out of scope for this project, but was a quick hack to fix our small issue.

Confirmed, it's some kind of bug in Symphony. @Anialation81 what part of Symphony / Silex did you narrow it down to?

Related to #419 once the escaping issue is fixed these 2 can be closed.


AJ-1548: Connect to prometheus rsocket proxy

Ticket AJ-1548

Enabled in bees
Disabled everywhere else
Can be enabled locally with spring-profile local-promprox

there's a hacky demo dashboard I threw together this morning just to show a basic example of this working this morning here


Create a distinction between old style race and new style race. More …

Let's try again :).
I pulled from your last changes, so there should not be conflict anymore ;).
I added more tests. Some are noted with "### Following tests work after this PR" some with "###Following tests don't work with this PR, more code is required". You can try with your present code to be convinced.
Target of this pull request is to be able to read the classifications for "new style race" like Itzulia Women. To avoid making too much changes at a time, this PR is only the first step. It populates standings. An object Standing() is created, to be able to access the results using obj.results_table. Without this object the behavior would be different for "old style race" and "new style race" (one would be accessed with res.standings["gc"] and the other with res.standings["gc".results_table --> bad).
Part2 of the PR will be to resolve the
 r=r_2023.results(classification_num=2).results_table
    assert len(r) == 18
    assert r['Rider'].iloc[0] == 'Wyllie Ella'
    assert r['Time'].iloc[0] == "10:04:05"

Presently, the API will call
https://firstcycling.com/race.php?r=14244&y=2023&e=&l=2
But as the parameter "l" is not supported it redirect to https://firstcycling.com/race.php?r=14244&y=2023&e= and we get the gc instead of the youth classification.
This will be solved by the use of standings.

Thank you very much @psemdel ! I adapted your changes into a separate PR in #29 since I think it's a slightly easier solution. Feel free to review if you think there is anything missing there. If not, I'll go ahead and merge so we can move on to the next fix 😄

Hum, I suppose some of your solutions are helping for sure. But for new stage races, we still have nothing in standings. And the l=01 / 02 will not work. What is your solution for that?
As mentioned, some part of the code are also preparation for the PR2.

I confirm that your change to headers and trs work also.

@psemdel sorry for the delay. I have merged my change now.

Ok thanks, I need to give it some more 20 min, to be sure what is the right way :)

I have still some progresses to do with git :( . I tested the new status and wanted to push a small correction, but I get permission denied.
In test_race, I need to change the last 2 lines (basque)
assert len(results_2023.standings['youth'].results_table) == 26
assert results_2023.standings['youth'].results_table['Rider'].iloc[0] == 'McNulty Brandon'

Note the results_table. I didn't introduce those for fun. I wanted to have the same way to handle standings and results. Otherwise, we will always need a condition to check if the other classification are in a results_table or without.
With your code with new races: r_2023.results(classification_num=2).results_table will return the gc, because the url redirect to the gc. So my idea was to use standings[] to handle that (and it works quite well). However here there is a "results_table", that's why I put it that way.

Thanks! I am on vacation for the next two weeks or so, but I will get back to your PR when I come back. If you want to set up other PRs for the other improvements, feel free to go ahead. I would just encourage you to make the PRs as small and straight-forward as possible, so it will be easy for me to understand and review.
Thanks for all your help!

I let you finish this one, and then I will post more. The version I use has many changes in between ^^.

 It was not my intention.

If you would like, feel free to revert the recent changes and force-push. Just make sure you have them saved on another branch somewhere :)

@psemdel will you have a chance to revert the recent changes or should I try to get these fixes through myself?


[Question] How to update the gradient of vit?

Question
When I set vit's attribute requires_grad to True, when loss performs gradient backpass, vit's grad is still None. I think this is because the function truncates the gradient postback when organizing text and images as input. Can someone tell me how to update the gradient of vit? Thank you!

I have the same question, after set ViT parameters grad  to True and the parameters  weren't updated : (

I have the same question, after set ViT parameters grad to True and the parameters weren't updated : (

I have solved it! You can delete the decorator decorator @torch.no_grad() of forward method in clip_encoder.py.


Slovenian - New Language Request

Enter the jw.org url to access this language:
https://www.jw.org/sl/
Please provide translations for the following terms in your language:
Congregation: občina
Assignment: dodelitev
Publisher: oznanjevalec
Meeting: shod
Midweek meeting: shod med tednom
Weekend meeting: shod ob koncu tedna
Circuit overseer: okrajni nadzornik
Student (of a meeting): učenec
Assistant (of a student): sogovornik
Elder: starešina
Ministerial servant: strežni služabnik
Ministry: oznanjevanje
Service: služba
Report: poročilo
Meeting Schedule: razpored shodov
Schedule: razpored
Pioneer application: prošnja za pionirsko službo
Auxiliary pioneer: pomožni pionir
Regular pioneer: redni pionir
Speaker: govornik
Talk: govor
Memorial: spominska slovesnost
Chairman: predsedujoči
Prayer: molitev
Reader: bralec
Meeting workbook: delovni zvezek
Governing Body Update (video): Poročilo Vodstvenega organa (videoposnetek)
Organizational Accomplishments (video): Organizacijski dosežki (videoposnetek)
Branch office: podružnični urad
Bethel: betel
Territory: področje
Do you feel fluent to be a proofreader of the language? If yes, please provide your Crowdin username:
Yes, I can proofread into Slovenian.  My Crowdin username is brobic74.

Thank you @brobic74 for the new language request and your willingness to contribute! Slovenian language is already available on Crowdin, and the AI pre-translations are ready.
You have received the proofreader role invitation, so once you're ready, feel free to start. In case of any questions, don't hesitate to ask :)
@rhahao I've also created the Notifications and releases project as well. Could you sync the Kingdom Songs + set up the EPUB parser in addition?


chore: Updated another set of third party urls

Description
Updated third party URLs.
Fixes [Issue URL](https://github.com/appsmithorg/appsmith/issues/36437)
Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!IMPORTANT]
🟣 🟣 🟣 Your tests are running.
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/11119054397
Commit: d92353308918a8ef3501530d03e5f2369ae2df98
Workflow: PR Automation test suite
Tags: @tag.All
Spec: ``
Tue, 01 Oct 2024 05:13:02 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

Summary by CodeRabbit
Summary by CodeRabbit

Bug Fixes

Updated URL parameters in tests for improved navigation and data accuracy.
Changed avatar URLs in test data to ensure consistent image sourcing.
Modified URLs in XSS vulnerability tests to use local mock APIs.
Corrected expected outputs for URL and date formats in the Table Widget tests.

Chores

Modified test cases to utilize local mock APIs for enhanced testing reliability.
Updated image URLs in fixture data to reflect new local sources.
Adjusted default images for widgets to ensure consistent visual representation.
Enhanced fetch tests to ensure proper handling of requests and responses.
Improved interactivity in modal widget tests with updated actions and dynamic data binding.

/ci-test-limit-count run_count=12

Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/11119069401. Cypress dashboard url: Click here! All Cypress tests have passed 🎉🎉🎉
***** Repeat Run Summary ***** Total Tests with repeat: 36 Total Passed: 24 Total Failed: 12 Total Skipped: 0 *****************************

Please ignore this failure as changes reverted for this file


Need an example of user login auth page ( Basic authentication login)

My project need a user login auth page, since different user will assgin them diffferent permissions.
Please give us more examples of user login auth.

@thierrymoudiki


chore(deps): upgrade to v2.2.0-mocha

Upgrade to https://github.com/celestiaorg/celestia-app/releases/tag/v2.2.0-mocha
v2.2.0-arabica has baked on Arabica since Thursday and no new issues reported.

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

Project coverage is 45.42%. Comparing base (8ee3b73) to head (f0011e7).
Report is 1 commits behind head on release/v0.16.1.

Additional details and impacted files
@@                 Coverage Diff                 @@
##           release/v0.16.1    #3786      +/-   ##
===================================================
- Coverage            46.55%   45.42%   -1.14%
===================================================
  Files                  314      281      -33
  Lines                17999    16075    -1924
===================================================
- Hits                  8380     7302    -1078
+ Misses                8597     7934     -663
+ Partials              1022      839     -183

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


Fix GetItemQueryIterator typo (perhaps due to API change) in exception message

Description
When this exception is thrown, it refers to GetItemsQueryIterator but it should be GetItemQueryIterator:
Unhandled Exception: System.NotSupportedException: To execute LINQ query please set allowSynchronousQueryExecution true or use GetItemsQueryIterator to execute asynchronously
   at Microsoft.Azure.Cosmos.Linq.CosmosLinqQuery`1.GetEnumerator()+MoveNext()
   at System.Collections.Generic.List`1.AddEnumerable(IEnumerable`1 enumerable)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Created in the wrong fork.


Add a new App named Papillon

Papillon is a IPFS based static blog system

Thanks, @terasum. I'm not sure that this work qualifies - there are spelling errors in the README, it's not clear how to contribute, there are no examples. I think this needs a bit more polish. Thank you, though! Feel free to fix up Papillon and resubmit it later.

Thanks, @RechardLitt. I'll fix up those problems, this project was completed in haste, so it is not perfect. I'll do some work on it, thank you for your suggestions!


必须放在if name == 'main'里面才能运行

你好 我直接运行
from capture_metric.capture import CAPTURE
refs = {
'example_0': [
"The image depicts a busy city street with cars running in the foreground, including a red car and a white truck. The street is surrounded by green trees. In the backgound of the image, modern edifices and a clock tower stand under a clear blue sky. ",
"The image depicts a busy city street with cars running in the foreground, including a red car and a white truck. The street is surrounded by green trees. In the backgound of the image, modern edifices and a clock tower stand under a clear blue sky. "
],
}
preds = {
'example_0': [
"The image shows a red car, a white truck and other automobiles running on a city road. Pedestrians are walking on the side. Tall buildings can be seen under a clear blue sky."
]
}
evaluator = CAPTURE()
score = evaluator.compute_score(refs, preds)
print(f"CAPTURE score: {score}")
但是出现了下面的错误：
RuntimeError:
An attempt has been made to start a new process before the
current process has finished its bootstrapping phase.
    This probably means that you are not using fork to start your
    child processes and you have forgotten to use the proper idiom
    in the main module:

        if __name__ == '__main__':
            freeze_support()
            ...

    The "freeze_support()" line can be omitted if the program
    is not going to be frozen to produce an executable.

不知道是否有可以解决的方法呢？

你好 我直接运行 from capture_metric.capture import CAPTURE refs = { 'example_0': [ "The image depicts a busy city street with cars running in the foreground, including a red car and a white truck. The street is surrounded by green trees. In the backgound of the image, modern edifices and a clock tower stand under a clear blue sky. ", "The image depicts a busy city street with cars running in the foreground, including a red car and a white truck. The street is surrounded by green trees. In the backgound of the image, modern edifices and a clock tower stand under a clear blue sky. " ], } preds = { 'example_0': [ "The image shows a red car, a white truck and other automobiles running on a city road. Pedestrians are walking on the side. Tall buildings can be seen under a clear blue sky." ] }
evaluator = CAPTURE() score = evaluator.compute_score(refs, preds) print(f"CAPTURE score: {score}") 但是出现了下面的错误： RuntimeError: An attempt has been made to start a new process before the current process has finished its bootstrapping phase.
    This probably means that you are not using fork to start your
    child processes and you have forgotten to use the proper idiom
    in the main module:

        if __name__ == '__main__':
            freeze_support()
            ...

    The "freeze_support()" line can be omitted if the program
    is not going to be frozen to produce an executable.

不知道是否有可以解决的方法呢？

请问解决了吗？想借鉴一下

只能在自己的主程序里面加一个if name == 'main'，然后把这个capture import进来

你好 我直接运行 from capture_metric.capture import CAPTURE refs = { 'example_0': [ "The image depicts a busy city street with cars running in the foreground, including a red car and a white truck. The street is surrounded by green trees. In the backgound of the image, modern edifices and a clock tower stand under a clear blue sky. ", "The image depicts a busy city street with cars running in the foreground, including a red car and a white truck. The street is surrounded by green trees. In the backgound of the image, modern edifices and a clock tower stand under a clear blue sky. " ], } preds = { 'example_0': [ "The image shows a red car, a white truck and other automobiles running on a city road. Pedestrians are walking on the side. Tall buildings can be seen under a clear blue sky." ] }
evaluator = CAPTURE() score = evaluator.compute_score(refs, preds) print(f"CAPTURE score: {score}") 但是出现了下面的错误： RuntimeError: An attempt has been made to start a new process before the current process has finished its bootstrapping phase.
    This probably means that you are not using fork to start your
    child processes and you have forgotten to use the proper idiom
    in the main module:

        if __name__ == '__main__':
            freeze_support()
            ...

    The "freeze_support()" line can be omitted if the program
    is not going to be frozen to produce an executable.

不知道是否有可以解决的方法呢？

请问解决了吗？想借鉴一下

只能在自己的主程序里面加一个if name == 'main'，然后把这个capture import进来

你好 我直接运行 from capture_metric.capture import CAPTURE refs = { 'example_0': [ "The image depicts a busy city street with cars running in the foreground, including a red car and a white truck. The street is surrounded by green trees. In the backgound of the image, modern edifices and a clock tower stand under a clear blue sky. ", "The image depicts a busy city street with cars running in the foreground, including a red car and a white truck. The street is surrounded by green trees. In the backgound of the image, modern edifices and a clock tower stand under a clear blue sky. " ], } preds = { 'example_0': [ "The image shows a red car, a white truck and other automobiles running on a city road. Pedestrians are walking on the side. Tall buildings can be seen under a clear blue sky." ] }
evaluator = CAPTURE() score = evaluator.compute_score(refs, preds) print(f"CAPTURE score: {score}") 但是出现了下面的错误： RuntimeError: An attempt has been made to start a new process before the current process has finished its bootstrapping phase.
    This probably means that you are not using fork to start your
    child processes and you have forgotten to use the proper idiom
    in the main module:

        if __name__ == '__main__':
            freeze_support()
            ...

    The "freeze_support()" line can be omitted if the program
    is not going to be frozen to produce an executable.

不知道是否有可以解决的方法呢？

请问解决了吗？想借鉴一下

只能在自己的主程序里面加一个if name == 'main'，然后把这个capture import进来

好滴


Update auth.php

Fix misspelled word "As" to "Has".

Thanks @AitorMatxi, Will merge for next patch release.
Note: Crowdin is our primary method of managing translations, it's often quicker and easier to make changes there:
https://crowdin.com/project/bookstack


consider Maven reactor in kogito-quarkus-test-list

This PR attempts at fixing Maven reactor related issues with kogito-quarkus-test-list.
Goal is to

enable dynamic "subscription" of integration-test modules, so that only those which are built as part of the reactor are included into the resulting XML test list.
Adhere to excludeITModules profile, not to build (and then release) kogito-quarkus-test-list when the integration-test modules were excluded.

Implemented as

Each applicable integration-test module need to activate rpkgtests-maven-plugin execution for itself by adding           <plugin> <!-- this way module defines itself as included in kogito-quarkus-test-list artifact -->
              <groupId>org.l2x6.rpkgtests</groupId>
              <artifactId>rpkgtests-maven-plugin</artifactId>
          </plugin>

This writes a simple test list into temporary location
In kogito-quarkus-test-list module, merge these temporary XML files, using XSLT transformation and produce the final artifact.
Attach the resulting XML file to the released artifact.

How to retest this PR or trigger a specific build:

Run all builds
Please add comment: Jenkins retest this

Run (or rerun) specific test(s)
Please add comment: Jenkins (re)run [runtimes|optaplanner|apps|examples] tests

Quarkus LTS checks
Please add comment: Jenkins run LTS

Run (or rerun) LTS specific test(s)
Please add comment: Jenkins (re)run [runtimes|optaplanner|apps|examples] LTS

Native checks
Please add comment: Jenkins run native

Run (or rerun) native specific test(s)
Please add comment: Jenkins (re)run [runtimes|optaplanner|apps|examples] native

Full Kogito testing (with cloud images and operator BDD testing)
Please add comment: Jenkins run BDD
This check should be used only if a big change is done as it takes time to run, need resources and one full BDD tests check can be done at a time ...

Waiting for build outcomes before turning to regular PR. Reactor order will be decisive point (the kogito-quarkus-test-list needs to run after all integration-test modules - ideally as the last one).

Does not work properly, kogito-quarkus-test-list is getting into test-list itself. I will add tests for the test-list as part of this (should be quick).

Moved back to draft. The changes in rpkgtests-maven-plugin https://github.com/ppalaga/rpkgtests-maven-plugin/pull/14 was merged and version 0.11.0 released. This will simplify configuration, I will adapt this PR once it's available in maven central.

Closing this one and will create a new PR, to limit scope of reviews (new config should be much simpler).

Superseded by #1584


Make credential-context resolvable

Description
As Operator or Enablement Service Provider for a CX-0018 compliant service,
I want to read the expanded JSON-LD of VCs,
so that I can reliably process and understand their content.
Acceptance Criteria

[ ] https://w3id.org/catenax/credentials/v1.0.0 is resolvable to a JSON-LD context.
[ ] The versioning between the context and the namespace is aligned.

Additional Information
Currently, the samples' @context in the credentials-folder rely on the https://w3id.org/catenax/credentials/v1.0.0 context. It currently resolves to the default path specified in the redirects. The credential-context included in this repository is already in v1.1 so I'm unsure whether it's an appropriate target. Its content would be affected if issue #25 is accepted with further implications for versioning.
I'd be willing to contribute if there's an aligned concept.

@jimmarino - is this on your roadmap? If no, I could pick this up. EDC doesn't need it but I think there's value in it.

@arnoweiss If you could pick this up it would be much appreciated.

Created a PR in the https://w3id.org/catenax/ namespace for the credentials [1] and one to register https://w3id.org/tractusx and populate it with a redirect to the policy-jsonld-context [2].
[1] https://github.com/perma-id/w3id.org/pull/4059
[2] https://github.com/perma-id/w3id.org/pull/4060

Hey @arnoweiss  thanks!
Just a small comment for the links. I would use the published on in GH pages,
since they will be updated automatically in when changed and also GH pages preserve the content type, some libraries complain about the raw.* of GH since the content type i think it's plain/text
You can find the published one here:
Policy Context
https://eclipse-tractusx.github.io/tractusx-profiles/cx/context/policy.context.json
Credentials Context
https://eclipse-tractusx.github.io/tractusx-profiles/cx/context/credentials.context.json

Thanks, I wasn't aware that exists.
As the documents you linked mirror main, it may become an issue that they are mutable. Fetching the context from remote may yield varying results. I think it's good enough for now but this may become an issue further down the road.

Thanks @arnoweiss
LGTM!


[Dropdown] Simple pointing dropdown closes before trying to hover on the dropdown item

I am using Semantic-UI-React. I'm trying to create a simple pointing dropdown and because of the hover nature of the simple dropdown, the pointing dropdown menu closes before one can even move to menu area.

Hi, please provide a test case as per our contributing guidelines; you can fork this JSFiddle.

Please check the fiddle for example.
http://jsfiddle.net/3o5sx34f/1/

It seems this can be solved by using CSS's transition delay (https://css-tricks.com/almanac/properties/t/transition-delay/). I'll check this this evening.

@ardamose123 I'm trying to do the same.

@ardamose123 Any luck?

@itsashis4u Nope, transition-delay doesn't work as I expected.

Any update on this?

@Banandrew Any update?

@itsashis4u No. I can’t answer it, because I’m not maintaining the project.

This problem is driving me nuts :)

Fixed. See http://jsfiddle.net/pjga38wv/


File copy fix

url unescape file paths, otherwise space and other symbols break file copy.

May I suggest to add a unitary test for these new public functions ?

@hardening done. also rewritten the encoding/decoding to work with all unicode characters.


Support for Soft Hyphens, a first step to better indexing for languages like German

In the following thread the problem is discussed that in German words are often a composition of simpler words. For example „Hochspannungsnetzgerät“ is a composition of „Hochspannung“ (High voltage) and „Netzgerät“ (power supply). If I search for „Netzgerät“ with pagefind it currently does not find „Hochspannungsnetzgerät“ though the term is included and semantically it most certainly is a kind of „Netzgerät“, so the user would expect to find it.
There seems to be no easy solution to that problem. But a first step would be to add optional support for soft hyphen characters. Pagefind should treat the soft hyphen as a word boundary. This would enable the generators of the static html to include this hints for pagefind in the page.

👋
This can be added, and no need for it to be optional. Pagefind already indexes multiple words for a given location when required — e.g. when it encounters a word source_text it will index source_text, source, and text at the given location. I can add the soft hyphen to this list which will roll it into the same handling.


Fix memory leak in case of realloc failure

Fixes #101.

Coverage decreased (-0.5%) to 80.193% when pulling f7eea6295742839bb819d1ee6b18ee0a4f64dac3 on fix-101 into 1b5073419fdde0197c6d85c60cccc9af75340487 on master.

Coverage decreased (-0.5%) to 80.193% when pulling f7eea6295742839bb819d1ee6b18ee0a4f64dac3 on fix-101 into 1b5073419fdde0197c6d85c60cccc9af75340487 on master.


'.NET Thank You!' website contribution counting

Announcing .NET 5.0 blog post has a .NET 5.0 Contributors link.
.NET 5 gets splits into many versions on this site. It would be nice to have a total number per contributor that indicates contributions since 3.1.
I also think the numbers of this website may be wrong. For example, for each .NET 5 RC/preview/final it shows exactly 2 contributions for me.
cc @richlander @leecow @spboyer @omajid

Thanks, Tom. That contributor page is definitely a work in progress. I don't know that we'll include pre-5.0 contributions owing to the repo consolidation effort that took place.
For the 5.0 release, this contributor list may be a little more accurate.
/cc @spboyer

Cross-referencing to the source repo: https://github.com/spboyer/dotnetthanks/issues/4

Updated data loader, smoke tests look good.

I'm not so sure the data is accurate right now. Examining Kevin Jones for each release:

v5.0.0: 0
v5.0.0-rc.2: 0
v5.0.0-rc.1: 4
v5.0.0-preview.8: 4
v5.0.0-preview.7: 0
v5.0.0-preview.6: 5
v5.0.0-preview.5: 1
v5.0.0-preview.4: 0
v5.0.0-preview.3: 2
v5.0.0-preview.2: 0

Add it up and we get 16. In the linked issue with the contributor list I have 51. GitHub agrees that 51 looks like the right number. https://github.com/dotnet/runtime/pulls?q=is%3Apr+author%3Avcsjones+created%3A2019-09-24..2020-08-26+is%3Amerged
Automating this a bit with jq:
cat ~/Desktop/core.json | jq -c '[.[].Contributors[] | select(.Name == "Kevin Jones") | .Count] | add' #17
cat ~/Desktop/core.json | jq -c '[.[].Contributors[] | select(.Name == "Tom Deseyn") | .Count] | add' #5

Mine came out to 17 because I had one in v5.0.1, so the number agrees. Tom has 5 but the contributor docs show 22.

Cc @richlander

@spboyer has been working on improving that data. I also think it could be fun to add a search by the person's name so it's easier to check yourself.
An update was pushed this morning. Can you let us know if that looks accurate now?

@mairaw

Can you let us know if that looks accurate now?

Well, I see it was updated for .NET 5.0.3. I had one servicing PR merged for 5.0.3 in https://github.com/dotnet/runtime/pull/46713 and I don't see myself in the 5.0.3 notes.
Looking further in to 5.0.0 for myself:
v5.0.0-preview.2: 0
v5.0.0-preview.3: 0
v5.0.0-preview.4: 0
v5.0.0-preview.5: 5
v5.0.0-preview.6: 6
v5.0.0-preview.7: 3
v5.0.0-preview.8: 10
v5.0.0-rc.1: 9
v5.0.0-rc.2: 0
v5.0.0: 0

That totals up to 33. That's closer but still seems fairly off. The counts that do exist also seem a bit off, but that could be my own faulty memory. I don't feel like I did 10 for preview 8 and I feel like I at least did a few in preview 2, 3, and 4.
Looking at @tmds' numbers:
v5.0.0-preview.2: 0
v5.0.0-preview.3: 1
v5.0.0-preview.4: 0
v5.0.0-preview.5: 3
v5.0.0-preview.6: 2
v5.0.0-preview.7: 6
v5.0.0-preview.8: 2
v5.0.0-rc.1: 1
v5.0.0-rc.2: 0
v5.0.0: 0

That comes out to 15. The contributor docs say 22, but I'd leave it to @tmds to say what the right number would be.

The number is more correct, but why is it not the same? Are these websites counting something different?
I'd prefer to see the aggregated count for .NET x.y, and not have it split across patch and preview releases.

Yes, we have plans to consolidate the data under major.minor. @spboyer can comment on the difference.

The most recent issue solved here was the contribution paging in the API was not implemented correctly.
moving forward these are plans to consolidate the releases to Major (maybe major.minor) and roll up the counts.

@spboyer feel free to close this issue when you know it is solved.

@spboyer do you have an idea of the time frame to do the rollup view? Specifically do you plan to enable anything like this for previews -- so eg for Preview 6 we could have a page for everyone that contributed since 5.0 GA up to that point?
if not I guess this can wait until 6.0 GA

I endorse the idea here. There is some UI work to determine the best way to present it all. @mairaw

@danmoseley @richlander is there a document that list the contributors for .NET 6 release?

@tmds I do not know of one, but it would be easy to create one (a list of github handles at least). I already did the query in order to get the number of contributors and contributions for the launch events.
do you think this would be valuable, until someone works on thanks.dot.net? we could add it as a markdown to this repo. cc @richlander

For the previous releases there is also a contributor-list.md that is published as part of the core release-notes (https://github.com/dotnet/core/tree/main/release-notes).
It's a nice way to give visibility to the contributors of an open-source project.

I didn't actually realize that (and I agree). @rbhanda @leecow were you planning on generating another? If not I can give it a shot.
my inclination is to --

omit the count of PR's (as it doesn't correlate well with impact)
continue to include both Microsoft and community folks on an equal basis

is that reasonable?

I have a PR ready to update https://dotnet.microsoft.com/thanks

omit the count of PR's (as it doesn't correlate well with impact)

I don't mind this number. It's common to see it on contributions lists.
Even if it does not reflect the impact, it is well defined what the number represents.

continue to include both Microsoft and community folks on an equal basis

For the contributor-list that gets published in the https://github.com/dotnet/core/tree/main/release-notes, definitely.
For the Thank you website, my preference is also to include both (https://github.com/dotnet/website-thanks-data/issues/11#issuecomment-967734697).

I assume this has been fixed, so I'll close it.
As I put in my initial comment, I think it would also be an improvement if contributions were aggregated per major release. It would significantly reduce the number of links/buttons on the thank you site.

Yes, aggregating per major/minor is on our backlog.
We'll have to continue to validate if the data is correct. If we have recent examples, we can look into it if you find any discrepancies.


[model] Add new field to Webhook

There is now a new a field in webhook structure called type that describes the type of the webhook and has two possible values 1, 2 which is Incoming or Channel Follower relevant commit

The funny thing is the example payload they included does not include this field


Multiple alert events created

Observed on our test infrastructure, alert set to raise event if wind speed < 2 m/s and close it automatically when it becomes inactive. It seems that four different events have been created at the same time then closed sequentially.

Linked to replication, the 2 instances check the alert at the same time and raise events catched internally => 2 objects are created. The event is also sent to each other instance through feathers-sync => 2 more objects are created.

As a fix, we added a unicity constraint on the alert linked to the event. However, it has been reverted as it prevented to create multiple events not linked to an alert because in this case the unicity constraint was applied to a null alert value.
We now have to find a new way to manage this.

Partial indexing could probably resolve the issue with the unicity constraint on null values.


html5shim should use a scheme-relative URL

When deployed to a SSL server, the stock Elkarte theme causes security warnings due to its insecure link to the html5shim. It should use a scheme-relative URL.
My personal preference would be to avoid linking to a googlecode resource, but this change at least addresses the security issue.

Yep, makes sense, thanks!
memo: to backport to 1.0.3 as well.


[WIP][Enhancement] Upgrade Spark to 3.4.0 due to CVE-2023-22946

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #22223
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:disappointed: fail : 0 / 2 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/transaction/PublishVersionDaemon.java
0
2
00.00%
[141, 275]

[FE PR Coverage Check]
:disappointed: fail : 0 / 2 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/transaction/PublishVersionDaemon.java
0
2
00.00%
[141, 275]

[FE PR Coverage Check]
:disappointed: fail : 0 / 2 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/transaction/PublishVersionDaemon.java
0
2
00.00%
[141, 275]

[FE PR Coverage Check]
:disappointed: fail : 0 / 2 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/transaction/PublishVersionDaemon.java
0
2
00.00%
[139, 275]

This security repair may have a relatively large impact on daily. If you find that it affects but cannot be completely repaired, you can directly revert it.

[FE PR Coverage Check]
:disappointed: fail : 0 / 2 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/transaction/PublishVersionDaemon.java
0
2
00.00%
[147, 281]


M33 – an One-on-One Feedback Format

Recently when we had one-on-one feedbacks, I found a few people they actually really wanted to give/receive feedback but just had no idea of how to do a one-on-one talk. Some of them then gave up. The others did have the  talk but via a "free" chat style. The "free" chat is good somehow – without any pressure to "give meaningful input" to each other, the both sides can be very relaxed, open heart to each other and then become closer. However, it doesn’t work all the time. There are a few times I found after 60-min chat we hadn’t touched any ‘crucial’ points. I now use a M33 format shown as below, which I think is more effective and fun.
Simply saying,

Start with the Mood Curve over the period, then
Share each other the top 3 things well-did; top 3 things learned/gained;  and the top 3 things want to improve, finally
Share ideas and suggestions to help each other come up with actions.

Start with the mood curve
Usually I start the conversation by asking feelings:

Over the past one month, how do you feel generally? Can we try to draw out our mood curves?

The mood curve is just like the picture shown above: on a scale of 1 to 10, follow your feeling, draw out the "happiness" on each time point. More often people will start to talk about the impressive events happened; explain the peaks and valleys when they draw the curves.
A thing worth mentioning – make sure your both sides are drawing out the curves. The one-on-one is a two-way process so don’t just ask the other side drawing his own, while you just watch. By exchanging your mood curves, the both sides will start to open up.
Share the 3-3-3 things
Once we’re warmed up by the mood curves, we can start to share the things well/learned/want-to-improve as we do in a retrospective but with a different focus – reflections on the person behaviors instead of the team’s. Two important tips for this stage:
1. Using self-reflections instead of "giving feedback".
You may notice that in the picture above, I use "my mood", "3 things I did well, rather than "your mood", "3 things you did well". "Feed back instead of giving feedback". Nobody knows better about us than ourselves. So let each side share what he has found about himself first. Play back to help him get better insights and make him feel being heard. If there is anything you see differently or anything important but he missed out, tell him in a tentative way only after he finishes all the 3 things and you’ve clearly heard and understood his thoughts. For example，

"Thanks for sharing your reflections. There is one thing my thought is different. You said one thing you did not do well was being involved in too many activities so you couldn’t focus. Well I see it actually as a positive thing because I think you first may need to get to know about the various areas then identify in which area you’re really interested. Also, it shows you passion on learning and improve as a new joiner. How do you think of this?"

2.  Follow the order: first share about "Well", then "Learned/Gained", finally "Want to improve".
First talking about the "well" part can help both sides strengthen confidence. Sometimes people tend to ignore the "Learned/Gained" part, but I actually like it the most. Every time when I exchange what I learned/gained with my one-on-one pair, I found something new. Next, naturally, comes the "things want to improve". After that, both then can take this opportunity to provide ideas or suggestions for actions to improve.
This format works for all the one-on-one pairs, no matter it is a senior-junior, junior-junior, or senior-senior pair.  You can try it out and see where we can improve.

总结：

如何快速插入当前时间？

嗷呜，手速够快就行！一秒打完当前时间！Alfred 的 snippet 啦哈哈哈


Issues with positioning and sizing for .ss-scroll element

Currently, there's a moveBar function which calculates height, top and right;

because of height calculations, I can't have a fixed-height .ss-scroll element (usable for really long scrolling areas),
because of right calculations, i can't move .ss-scroll further from the edge (right: 5px;), a design issue

I see there's a .ss-content.rtl class which uses calc() to adjust width, but there's no .ss-scroll.rtl which would adjust right for this element in rtl direction, where at least the calculation for right could be fully dropped for css.

The calculation here also doesn't take the height of .ss-scroll into account:
top:' + (_this.el.scrollTop / totalHeight ) * 100
totalHeight should be decreased by the height of the .ss-scroll element, so that it doesn't overflow the container. Most likely there's also an issue with the css, as the more reasonable definition for the element seems to be position: absolute; right: 5px !important; and not position: relative; right: -107px which is the default. These two changes patch the second part of this issue.

I know this is probably too late, but you can move .ss-scroll with negative margin-left.


Add GetTeleportCost and describe formula.

Let me know if this should be done differently.

I believe this is a static function, so will need to be declared as one in client structs otherwise it'll use Telepo instance as a1

Oh yeah, I actively looked into where or how to put free functions and then forgot to actually write the static, derp.


items cant be crafted 1.21

void backpack
crafting table on stick
and bundles dont work

The tag for wool changed in the void pack, I will fix.
crafting tables and bundles are no longer part of the mod.

fixed in next update.


Remove .go-version

Summary
Removes the .go-version file from the repo.
For some context, see this PR comment
The last usage of this file was for CircleCI cache keys. Since it was a place of duplication that added an extra step to upgrading our go version, it'd be nice if we didn't need it at all :)
See:
#11118
#11053
#11103
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)

Verification Steps for Reviewers
These are to be checked by a reviewer.
How to test

Verify CircleCI builds & deploys

Frontend

No changes

Backend

[ ] Still runs locally

Database

No changes

Ah, right, this needs to be after checkout

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against c2b409794f13b1d8685383828d49aa577553f7b3


Add registering / log-in view

Based on user story: https://github.com/sopra-fs22-group-36/screw-your-neighbor-server/issues/14

Depends on closing userstory https://github.com/sopra-fs22-group-36/screw-your-neighbor-server/issues/14 this one is obsolet.
Information to https://github.com/sopra-fs22-group-36/screw-your-neighbor-server/issues/14
We are closing this issue without an implementation. The reasons are varied:
No added value for the fun of the game
Heroku does not have a persistent DB
All common games in this form are played without registration.
We have already implemented user account login and registration in M1.
Low priority
It took place in consultation with our TA.


Update tests to handle local omnibus packages from Buildkite artifacts api

Description
Updated tests to support the use of skip-artifactory-platforms within the shim pipeline definition where packages will be downloaded directly from the buildkite artifacts api.
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

https://buildkite.com/chef/chef-chef-server-master-omnibus-adhoc/builds/2453


Update eslint and eslint-plugin-github to latest versions

I wanted to update webpack to version 4 after I can't find the documentation for webpack 3 but one of the thing that happened is that eslint updated as well and started giving me a lot of failures. I figured I'd isolate it in it's own PR and fix all the new violations that are coming from the new rules we have.

Both #27 and this is failing on some relay error but not on my local machine. I'm looking at it now 👀. I think it might be that I have relay-compiler: 1.6.0 on my local machine but the package.json asks for 1.5.0.

So something got updated that causes relay not to generate the graphql schema files on the first go. I'm wondering if it's cool to just check it in or if I should add to the test suite to generate them once if they don't exist.

So something got updated that causes relay not to generate the graphql schema files on the first go.

🤔 strange. This is supposed to be handled by the relay-compiler-webpack-plugin.

It seems that relay-compiler-webpack-plugin did something weird and it's fixed in the latest version but that also pulls in version 0.13 of graphql which fails the test with a dedupe error since eslint-plugin-github pulls in<EMAIL_ADDRESS>via eslint-plugin-relay.
Not sure what I should do other then wait for<EMAIL_ADDRESS>to be released, update it in relay-compiler-webpack-plugin and then update it here.

It seems that relay-compiler-webpack-plugin did something weird and it's fixed in the latest version but that also pulls in version 0.13 of graphql which fails the test with a dedupe error since eslint-plugin-github pulls in<EMAIL_ADDRESS>via eslint-plugin-relay.

Ugh, yeah, more dependency hell we'll need to resolve. GraphQL doesn't really work when there's conflicting packages in the dep tree.

Whoo hoo! #32 fixed CI and now the PR is ✅


check-quota function error

JSONDecodeError                  Traceback (most recent call last)
 in ()
3 requested_cores = ffmpeg_node_count * vm_dict[vm_family]["cores"]
4
----> 5 diff = check_quota(vm_family)
6 if diff <= requested_cores:
7     print("Not enough cores of DSv3 in region, asking for {} but have {}".format(requested_cores, diff))
 in check_quota(vm_family)
19         "--query", "[?contains(localName, '%s')].{max:limit, current:currentValue}" % (vm_family)
20     ], stdout=subprocess.PIPE)
---> 21     quota = json.loads(''.join(results.stdout.decode('utf-8')))
22     return int(quota[0]['max']) - int(quota[0]['current'])
~/anaconda3/envs/batchscoringdl_aml/lib/python3.6/json/init.py in loads(s, encoding, cls, object_hook, parse_float, parse_int, parse_constant, object_pairs_hook, **kw)
352             parse_int is None and parse_float is None and
353             parse_constant is None and object_pairs_hook is None and not kw):
--> 354         return _default_decoder.decode(s)
355     if cls is None:
356         cls = JSONDecoder
~/anaconda3/envs/batchscoringdl_aml/lib/python3.6/json/decoder.py in decode(self, s, _w)
337
338         """
--> 339         obj, end = self.raw_decode(s, idx=_w(s, 0).end())
340         end = _w(s, end).end()
341         if end != len(s):
~/anaconda3/envs/batchscoringdl_aml/lib/python3.6/json/decoder.py in raw_decode(self, s, idx)
355             obj, end = self.scan_once(s, idx)
356         except StopIteration as err:
--> 357             raise JSONDecodeError("Expecting value", s, err.value) from None
358         return obj, end
JSONDecodeError: Expecting value: line 1 column 1 (char 0)

@daden-ms following up


PDF not generating in Android

I tried following the example but it is not working on my end
I saw an exception in the console that might help
E/chromium(29944): [ERROR:aw_browser_terminator.cc(154)] Renderer process (28383) crash detected (code -1).

What does you mean by environment?

Here is my environment
Flutter 3.19.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 300451adae (9 weeks ago) • 2024-03-27 21:54:07 -0500
Engine • revision e76c956498
Tools • Dart 3.3.3 • DevTools 2.31.1

@penkzhou please can you look into this ASAP ?

I am getting the same issue. my platform info:
Flutter: Channel stable, 3.22.2
Dart SDK version: 3.4.3 (stable)
Android Studio (version 2024.1)


Similarities to other CPU feature projects (deduplicate?)

ART on Android works around quirks/bugs in /proc/cpuinfo and aux vector, has support for cross compilation, detection of features via C preprocessor, CPUID instructions, undefined instruction exceptions, supports MIPS, ARM, Intel 32 and 64, etc.:
https://android.googlesource.com/platform/art/+/master/runtime/arch/instruction_set_features.h#36
Android NDK has something simple:
https://developer.android.com/ndk/guides/cpu-features.html
V8 CPU feature detection:
https://github.com/v8/v8/blob/master/src/base/cpu.h#L32
As there are OS/CPU bugs that cause issues standardizing on 1 library makes sense. As an author of the ART code I'm have a bias :-)

Thx for reaching out @captain5050 !
cpu_features is literally born from the fact that many libraries are coding their own routine (libwebp, libyuv, android NDK, OpenCV to name a few). It seems simple enough at first sight but you learn along the way how hard it is to get right.
This library has been designed with the author of the NDK's one you're referring to, the end goal was to replace it entirely with this library. It supports all of what you described for ART except the "undefined instruction exceptions" (additions welcome!). We also have a list of fixes for corrupted kernels and fallback for auxval via getauxval, /proc/self/auxval, /proc/cpuinfo.
Last but not least, this library is usable outside the Android ecosystem.

cpuid2cpuflags, although built for a specific use case, shares some similarities as well.

for x86, I can throw my hat into the ring.
From my benchmarks, I saw that I am a tick faster than cpu_features when detecting sse4_1 (7x):
 Run on (4 X 3600 MHz CPU s)
2018-05-14 17:37:29
***WARNING*** CPU scaling is enabled, the benchmark real time ...
--------------------------------------------------------------
Benchmark                       Time           CPU Iterations
--------------------------------------------------------------
BM_compass_sse4_1              31 ns         31 ns   22705074
BM_cpu_features_sse4_1        242 ns        241 ns    2870098

I saw that I am a tick faster than cpu_features when detecting sse4_1

Why are you sure it is correct?

@psteinb I suppose you were talking about https://github.com/psteinb/compass right, not about https://github.com/psteinb/deeprace?
Can you compare the implementations to understand where "tick faster" comes from?

@psteinb I suppose you were talking about https://github.com/psteinb/compass right, not about https://github.com/psteinb/deeprace?
Can you compare the implementations to understand where "tick faster" comes from?


Refactoring + Parser Plugin

Massive Refactoring

Adds #15
Established a more pydantic approach to configs #11
Establishes Plugin Interface #16


Upgrade to v17

version: v16.0.0
The Angular v17 will be released soon, so the lib is going to be upgraded to v17.0.0 which based on Angular 16

It is published now!


Feature Request: Replace IR Aircon "Current Temperature" with reading from SwitchBot Thermometer & Hygrometer

Problem
The temperature in Homekit always reads as "24°c" on all my IR Air Conditioners.  I do have SwitchBot Thermometer & Hygrometers in each room which shows the correct temperature.  It would be nice if there was a configuration option to use the "Current Temperature" reading from another device to replace the static number that the IR device shows.
Solution
Add a configuration option to replace "Current Temperature" for an IR Aircon with another Switchbot device.  The user would specify the temperature sensor's device name or device id and the plugin would report that device's "Current Temperature" as the IR Aircon's "Current Temperature".
Alternatives
If the specified device doesn't have a "Current Temperature" or the device becomes unavailable, then it reverts to 24°c.
Additional context
It would be a nice quality of life thing to have the numbers on the IR Aircon units be accurate instead of a static number that can't change.  I'm not sure what restrictions exist that may prevent it from being possible, but if you can, that would be great!  And if I missed a setting that would let me, then sorry!  I tried to look, but I'm fairly new to homebridge.  Thanks for the plugin!

This is already possible

Sorry for the bother then. Is there documentation about how to do it?  The only thing I saw in IR Aircon config instructions was to disable auto mode.  When I searched the web I only found old comments.

I didn't realize that the IR Device settings section actually wrote the json code for you.  Since my devices were already listed in the devices tab, I thought that I didn't need to "Add a new IR Device".  I didn't realize that I could use that area to configure my existing devices.  Maybe for dum-dums like me that don't make the connection, adding a button to the devices tab with "Configure this device" would help.  Then it could populate the device ID, name and type into the IR Device settings so you can make the rest of the changes.

I didn't realize that the IR Device settings section actually wrote the json code for you.  Since my devices were already listed in the devices tab, I thought that I didn't need to "Add a new IR Device".  I didn't realize that I could use that area to configure my existing devices.  Maybe for dum-dums like me that don't make the connection, adding a button to the devices tab with "Configure this device" would help.  Then it could populate the device ID, name and type into the IR Device settings so you can make the rest of the changes.

Yeah, I would like to do that, but don't know how too... :)


Nits: JTAGG -> JTAG

There is a minor spelling in the file: "JTAGG" should be "JTAG".
https://github.com/Wren6991/Hazard3/blob/e8b4578b4003d838f006dca00b5fea5852e4ab98/example_soc/fpga/fpga_ulx3s.f#L9C48-L9C53

Nope, that is the correct name of that ECP5 primitive
https://github.com/Wren6991/Hazard3/blob/e8b4578b4003d838f006dca00b5fea5852e4ab98/hdl/debug/dtm/hazard3_ecp5_jtag_dtm.v#L6
See notes here: https://github.com/tomverbeure/ecp5_jtag

It's a bit of an unfortunate name, but if you've already had JTAGA through JTAGF then what follows is inevitable.

Ah, my mistake, sorry.


Proposal: Make StackLayout respect Horizontal/Vertical Alignment

Proposal: Make StackLayout respect Horizontal/Vertical Alignment
Summary
StackLayout does not current respect a child control's HorizontalAlignment and VerticalAlignment settings. Would it be difficult to implement this?
Rationale

Existing panels do respect these properties
As the docs for HorizontalAlignment say: "Gets or sets the horizontal alignment characteristics that are applied to a FrameworkElement when it is composed in a layout parent, such as a panel or items control." which suggests it should be respected.

Wait... this does already work. Sorry for the noise.

For anyone coming across this on google etc, the reason I thought it wasn't supported was due to #1410.


[KO] Translation of Pipe Operator Lesson

#201
Please refer to Dalgona/elixir-school#2 for side-by-side comparison and comments. :rocket:

@devleoper, @rijgndqw012 :tada:

Thank you @Dalgona, I appreciate your help 👍

Very well. But, i found something awkward point and i commented on that. Can you check my comment? :smiley_cat:


Use an existing C++ project to increase amount of code/files to build

Currently there are too few things to compile; in some regards it can be hard to say whether the CMake setup scales well as the project grows. For example, from work experience, using the .clang-tidy file in this project on a medium-scale project seems to cause it to take over an hour to build; we might have to try and find which passes take so long and remove them.

Since we want an executable, and two libraries, we could try to find a CLI (which would be the executable) which links against a library (which would be lib2) which, in turn, would link against another library (which would be lib1).


TransformAndCatalogToLuma uses incorrect dbt<>Luma integration workflow

The flow executes docs generate after running the test command, which results in run-results.json being overwritten. This results in any test failures that appeared during the test command to disappear from the run-results.json, which in consequence means that the tests incorrectly show up as successful in Luma UI.
The fix is to remove the docs generate as it's not required for model run metadata ingestion.

I think this is done in #860. @Rafalz13 remember you can link PRs back to issues by using the "Closes #issue-number" keyword. This way, related issues are automatically closed once PR is merged.


Handle differences in N-padding

Ensembl adds N-padding to some of the PATCH contigs to make their coordinates during alignment identical to the position on the primary contig the patch is derived from:
All alternative assembly and patch regions have their sequence padded
with N's to ensure alignment programs can report the correct index
regions

e.g. A patch region with a start position of 1,000,001 will have 1e6 N's added
its start so an alignment program will report coordinates with respect to the
whole chromosome.

Example contig from the HG-2365 PATCH:

Reference Build
Name
Length

Ensembl
CHR_HG2365_PATCH
102714182

gencode
ML143371.1
5500449

NCBI
NW_021160017.1
5500449

So for this contig Ensembl has added 97,213,733 extra Ns to the sequence. Which means we cannot match it with gencode's ML143371.1 or NCBI's NW_021160017.1.

In order to address this we would have to amend the file format of the recontig mapping files to have an optional column that contains the offset to be applied to record starting positions.


plugin: update bozja intl opcode for 6.1

Tested and working!
Had to manually spawn a CE reeee

Currently your title is: plugin: update bozja intl opcode for 6.1, but it should:

Did you mean: plugin: update bozja intl opcode for 6.1

bot please.

Had to manually spawn a CE reeee

oh noooo
Thank you for your service.


Results of tests?

I don't understand how I get the test results. I successfully ran the tests but I can't make anything of the result:
---
Model         API Level   Locale     Orientation
NexusLowRes   24          portrait   en
---
- AppPackageID:
- TestType: instrumentation
- TestApkPath: /bitrise/deploy/app-production-debug-androidTest.apk
- InstTestPackageID:
- InstTestRunnerClass:
- InstTestTargets:
- UseOrchestrator: false

Upload APKs
=> APKs uploaded

Start test
=> Test started

Waiting for test results
- Validating
- (1/1) running
- (0/1) running
=> Test finished

Test results:
Model         API Level   Locale   Orientation   Outcome
NexusLowRes   24          en       portrait      failure
|                                                                              |
+---+---------------------------------------------------------------+----------+
| x | App Instrumentation Test (exit code: 1)                       | 513 sec  |
+---+---------------------------------------------------------------+----------+
| Issue tracker: ...se-steplib/steps-virtual-device-testing-for-android/issues |
| Source: ...thub.com/bitrise-steplib/steps-virtual-device-testing-for-android |
+---+---------------------------------------------------------------+----------+

Hi @PaulWoitaschek ,
Right now the results are only shown on the UI:

Related intro blog post: https://blog.bitrise.io/introducing-solid-and-snappy-virtual-device-testing-for-android

Thank you! Maybe just add a message where to find test results?

Definitely, on our list! ;)


Update README.md

New to Open Source and this is where it's all getting real.
Related issue

What changes does this PR do?
Added my Name and gitHub Link into the readme file
Screenshots (If any)
Questions

hey Kartik, Hope you having a good day !!
I have made it alphabetic,
but I'm kinda confused with how can I delete my previous commit?

hey Kartik, Hope you having a good day !! I have made it alphabetic, but I'm kinda confused with how can I delete my previous commit?

you removed older one last line right so looks good

Alright!!..Thanks for pointing out the mistakes!!

hey Kartik, Hope you having a good day !! I have made it alphabetic, but I'm kinda confused with how can I delete my previous commit?

Hey,
you don't need to delete any commit, you can as many modification in your PR.
__
  Summary: :hammer_and_wrench: This code needs a fix


image selection supporting multiple configurable options

Related issues
No issue
Short description and why it's useful
If you have a shop with configurable children products having multiple configurable options that look totally different (each having different thumbnail), this code change enables showing the right product for each
Screenshots of visual changes before/after (if there are any)
Screenshot of passed e2e tests (if you are using our standard setup as a backend)
feature not present in demo shop
Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility)
[ ] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

Contribution and currently important rules acceptance

[x] I read and followed contribution rules
[x] I read the TypeScript Action Plan and adjusted my PR according to it
[x] I read about Vue Storefront Modules and I am aware that every new feature should be a module

@mdesmet yeah, thats look a way better:) please just update changelog.md with description of your feature


bug: Invalid ESM build

The ESM build of the library should use the .js file extension for relative imports, else it is not a valid ES module.
Reference: https://nodejs.org/api/esm.html#mandatory-file-extensions
This also causes issues (MODULE_NOT_FOUND) with tools that support ESM (like Vitest).
Adding file extensions to relative imports should also cause no issues with the CJS build of the package.

Maybe you can achieve the src/foo/index.ts import with this tsconfig paths.
Also, you need to write the imports using the .js file extension, even though you are using typescript.
It is a little bit confusing at first, but I think this gives a solid explanation (a little bit long :slightly_smiling_face:)

Maybe you can achieve the src/foo/index.ts import with this: tsconfig paths. Also, you need to write the imports using the .js file extension, even though you are using typescript.
It is a little bit confusing at first, but I think this gives a solid explanation (a little bit long 🙂)

Much appreciated! It seems like hybrid CJS/ES builds are more complicated than I thought. Will look more into this next week.

Here is some extra information, maybe it will be of some help:

If you need some reference implementation, I tested the fixes in this repo here: fork.
Adding .js to relative imports produces a correct output, but the solution is not that simple. The same issue is also present in the @metamask/utils  dependency.
If you set module resolution to nodenext example, it will show you some more problems.
If you want eslint to show errors for missing .js file extensions you need to set module resolution to nodenext and set the type in package.json to module. I am not sure how this impacts the CJS build though.

There is a fix for this in the @metamask/utils repository (MetaMask/utils#144). When that is merged I will apply the same fix to key-tree.

Thank you for looking into it so fast. I like the solution you chose, it didn't require any actual code change, only replacing the build tool and bundling everything as 1 file. Smart :+1:.

Does bundling everything as 1 file impact tree-shaking? I'm not worried about the bundle size here, just curious if you maybe know or thought about that.

I like the solution you chose, it didn't require any actual code change, only replacing the build tool and bundling everything as 1 file. Smart 👍.

Thanks! I definitely wanted to avoid having to change every import, update linting rules, and so on. I couldn't find a good way with TSC or SWC to add extensions to the imports at build time, and tsup seems to do just what we need out-of-the-box.

Does bundling everything as 1 file impact tree-shaking? I'm not worried about the bundle size here, just curious if you maybe know or thought about that.

I don't think it makes any difference (but don't quote me on that). Currently, the entire bundle is imported regardless (through the root index.js). And most bundlers, like Webpack and Rollup, are pretty good at tree-shaking ESM code (not so much at tree-shaking CJS, but that's why we provide the ESM file), especially considering that most of our libraries are free from side effects.

Ran into this today. Is a fix on the horizon?

@Mrtenz It seems like this is addressed via https://github.com/MetaMask/key-tree/pull/157, is this true?

Will there be a release soon?

I thought we released this already, but apparently not. We weren't planning to do a release before the next audit (mid April), so I have to figure out if we can release without including the latest PR.


Add injection-regex to package.json

This allows automatically using this language in injections.
See https://tree-sitter.github.io/tree-sitter/syntax-highlighting#language-detection

This looks straight forward, but injections seem to work already without
it. How exactly is this regex used?
On Tue, Jan 9, 2024, 08:43 Trevor Gross @.***> wrote:

This allows automatically using this language in injections.
See
https://tree-sitter.github.io/tree-sitter/syntax-highlighting#language-detection
You can view, comment on, or merge this pull request online at:
https://github.com/tree-sitter-perl/tree-sitter-perl/pull/157
Commit Summary

0096cbf
https://github.com/tree-sitter-perl/tree-sitter-perl/pull/157/commits/0096cbf34d2e44e2a1b4140b386e9492684cb115
Add injection-regex to package.json

File Changes
(1 file
https://github.com/tree-sitter-perl/tree-sitter-perl/pull/157/files)

M package.json
https://github.com/tree-sitter-perl/tree-sitter-perl/pull/157/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519
(3)

Patch Links:

https://github.com/tree-sitter-perl/tree-sitter-perl/pull/157.patch
https://github.com/tree-sitter-perl/tree-sitter-perl/pull/157.diff

—
Reply to this email directly, view it on GitHub
https://github.com/tree-sitter-perl/tree-sitter-perl/pull/157, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFURPKUB2QWC6GCGRGGKQBDYNTRKHAVCNFSM6AAAAABBSSYLDWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA3TCNZSHA2TANI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I think that editors are able to figure out injections because they provide their own regex patterns for each language, but tree-sitter CLI doesn't seem to pick up @injection.language unless it is specified in package.json like this.
But I am not entirely clear on how exactly it works myself, the one paragraph in the docs linked above and some anecdotal evidence are really all I'm going off of. In any case, specifying these queries in the upstream language repo is probably a good thing because it means editors will hopefully one day be able to pull this information automatically.

ah, great. Thanks for the PR!

Thanks!


Initial sqlcommenter checkin with copyright changes

This is an initial checkin which contains all sqlcommenter files from google repository (https://github.com/google/sqlcommenter).
The only change that is added here is the change in copyright from Copyright 2019 Google LLC  to Copyright The OpenTelemetry Authors.

@SergeyKanzhelev, @lonewolf3739 any more changes  that need to be done in this PR ?
If not can i get an approval ?

^@jsuereth look at this thread about changing copyright https://github.com/open-telemetry/opentelemetry-sqlcommenter/pull/1#discussion_r720139303.

Are we concerned with losing the Git history here from the original repo? There may be some complications with previous commiters not having signed the CNCF CLA if we try to keep the history.

Are we concerned with losing the Git history here from the original repo? There may be some complications with previous commiters not having signed the CNCF CLA if we try to keep the history.
@aabmass that was one point I discussed briefly with @SergeyKanzhelev and this PR doesn't contain the history for that reason. If history is too important, we can discuss how to do it.


v-pre do not work in template tags

Version
2.5.7
Reproduction link
https://jsfiddle.net/3s9sem0m/
Steps to reproduce
I am using laravel-admin.org and I have incorporated vuejs into that system. The problem is that there are some functionalities that use the tag template, to then duplicate it using jquery, and vuejs are rendering those tags (even if you add the v-pre directive) and breaking all that functionality.
What is expected?
The directive v-pre should prevent render the tag template
What is actually happening?
The tag template is rendered and disappears, preventing the later use of the js script and breaking all the functionality

It works fine on the fiddle you posted. I'll revisit if you add correct steps to reproduce and fix the repro. Thanks

How is working fine if the template tag is missing in the source

I need the template tag remain in the source code to use it from Jquery later ... that is how the laravel-admin works.

Even if I put the template tag inside a div with v-pre, the tag is rendered and missing

Ah, didn't get what you said in the first place. Not sure if this a good idea because template has a special meaning in html. For the moment you can use v-html to completely escape the HTML but v-pre should indeed leave the template during compilation phase

Hi, the use that I am giving to the template tag is the one that should be, but outside of vuejs, and that is exactly the problem.
Using the v-html works. Is too complicate, but works.
https://jsfiddle.net/ft53kwrw/

It would by nice and easier if v-pre works in the template tag.
Thanks

I'm not familiar with laravel-admin. Does script type="text/x-template" solve your problem?
https://vuejs.org/v2/guide/components-edge-cases.html#X-Templates

Maybe should work. But the template tag, is inside a blade file, and if a put a script inside the root vue element, I get another warning. So, I think for the moment v-html is the best way. Thanks !


Cache the iRODS password

When running the iCommand iinit on the Linux command line, a .irodsA file is created storing an encrypted version (actually a hash) of the password.  It would be ideal to support this functionality, both creating the file and parsing it so that it is more convenient for the user who will not need to enter their password endlessly.  A configuration option could be added to switch off this functionality if the user does not desire it.
A more advanced version of this would be to natively support password managers, but this functionality is far too advanced at this stage.

Also, add environmental variable awareness for IRODS_ENVIRONMENT_FILE and IRODS_AUTHENTICATION_FILE.

Resolved in PR #17


bug: Out-of-memory (OOM) panic in Sierra deserialization leads to denial-of-service (DoS)

Out-of-memory (OOM) panic in Sierra deserialization leads to denial-of-service (DoS)
Author(s): @Fuzzinglabs (Darius & Patrick)
Executive Summary
An attacker can cause a DoS on a node that processes Starknet contracts by sending a serialized Sierra program causing the crash. This is due to missing sanitization/verification of vector size before calling Vec::with_capacity(size) in Program::deserialize.
Vulnerability Details

Severity: Critical
Affected Component: cairo/crates/cairo-lang-starknet
Commit: fccf41a8ff259c437f16a6d351df8593de48b8a3

Steps to reproduce
Compile to following
  let serialized_program: Vec<BigUintAsHex> = vec![
      BigUintAsHex {
          value: BigUint::from_str("1000000000000000000").unwrap(),
      },
      BigUintAsHex {
          value: BigUint::from_str("1000000000000000000").unwrap(),
      },
  ];

  Program::deserialize(&serialized_program);

Root Cause Analysis
When deserializing vectors, the deserialize function first reads the length and then tries to allocate a vector with the supplied length. Passing a length that's too big causes the allocation to fail which will result in a crash.
https://github.com/starkware-libs/cairo/blob/fccf41a8ff259c437f16a6d351df8593de48b8a3/crates/cairo-lang-starknet/src/felt252_serde.rs#L370
Detailed Behavior
thread 'main' panicked at library/alloc/src/raw_vec.rs:534:5:
capacity overflow
stack backtrace:
   0: rust_begin_unwind
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/panicking.rs:595:5
   1: core::panicking::panic_fmt
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/panicking.rs:67:14
   2: alloc::raw_vec::capacity_overflow
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/alloc/src/raw_vec.rs:534:5
   3: alloc::raw_vec::RawVec<T,A>::allocate_in
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/alloc/src/raw_vec.rs:177:27
   4: alloc::raw_vec::RawVec<T,A>::with_capacity_in
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/alloc/src/raw_vec.rs:130:9
   5: alloc::vec::Vec<T,A>::with_capacity_in
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/alloc/src/vec/mod.rs:670:20
   6: alloc::vec::Vec<T>::with_capacity
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/alloc/src/vec/mod.rs:479:9
   7: <cairo_lang_sierra::program::Program as cairo_lang_starknet::felt252_serde::Felt252Serde>::deserialize
             at /home/fuzzinglabs/Documents/cairo/crates/cairo-lang-starknet/src/felt252_serde.rs:370:37
   8: sierra_serialization::main
             at ./src/main.rs:28:5
   9: core::ops::function::FnOnce::call_once
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Recommendations
The deserialize function could check if the supplied vector length makes sense when compared to the program size and output an error result if it doesn't.
Additionally, this pattern of Vec::with_capacity(size) without checking the size value seems used multiple times in the code:

https://github.com/starkware-libs/cairo/blob/fccf41a8ff259c437f16a6d351df8593de48b8a3/crates/cairo-lang-starknet/src/felt252_serde.rs#L382
https://github.com/starkware-libs/cairo/blob/fccf41a8ff259c437f16a6d351df8593de48b8a3/crates/cairo-lang-starknet/src/felt252_serde.rs#L393

Fix PR merged - will be released at the next 2.4.0-rc4 or 2.4.0 release (whatever comes first)


Use ICU for text search

The ultimate goal of this PR was to use ICU for text search to

Improve Unicode support
Previously we used towlower and only supported BMP glphs.
Improve search performance (10-100x)
This allows us to search for all results in the entire text buffer
at once without having to do so asynchronously.

Unfortunately, this required some significant changes too:

ICU's search facilities operate on text positions which we need to be
mapped back to buffer coordinates. This required the introduction of
CharToColumnMapper to implement sort of a reverse-_charOffsets
mapping. It turns text (character) positions back into coordinates.
Previously search restarted every time you clicked the search button.
It used the current selection as the starting position for the new
search. But since ICU's uregex cannot search backwards we're
required to accumulate all results in a vector first and so we
need to cache that vector in between searches.
We need to know when the cached vector became invalid and so we have
to track any changes made to TextBuffer. The way this commit solves
it is by splitting GetRowByOffset into GetRowByOffset for
const ROW access and GetMutableRowByOffset which increments a
mutation counter on each call. The Search instance can then compare
its cached mutation count against the previous mutation count.

Finally, this commit makes 2 semi-unrelated changes:

URL search now also uses ICU, since it's closely related to regular
text search anyways. This significantly improves performance at
large window sizes.
A few minor issues in UiaTracing were fixed. In particular
2 functions which passed strings as wstring by copy are now
using wstring_view and TraceLoggingCountedWideString.

Related to #6319 and #8000
Validation Steps Performed

Search upward/downward in conhost ✅
Search upward/downward in WT ✅
Searching for any of ß, ẞ, ss or SS matches any of the other ✅
Searching for any of Σ, σ, or ς matches any of the other ✅

URL search now also uses ICU

Does it just so happen to work across wrapped lines now? @asklar literally reported that to me on Teams this weekend 😄

Oh wait, I think you meant a different problem, right? I think that's something like Terminal::GetHyperlinkAtBufferPosition, because that one searches only a single line?

Oh wait, I think you meant a different problem, right? I think that's something like Terminal::GetHyperlinkAtBufferPosition, because that one searches only a single line?
@lhecker yes this sounds about right - will this PR address hyperlinks being broken up across lines (and launching to the wrong url)? if not, is there a bug tracking that already?

I need @carlos-zamora to sign off on the UIA changes

Basically same here. I think I've got like 3 files left but I didn't want to ✅ without him reading that (cause obviously no one knows how that works)

@asklar If you meant the issue where hyperlink navigation isn't correctly selecting entire URLs then this PR won't fix it. The other bug where after a window resize, hyperlinks aren't immediately recognized won't be fixed either.
But if you meant a different problem, one where clicking a hyperlink inside the viewport opens an incorrect URL, then I'd probably require repro steps, because I couldn't figure out any other issues apart from the above two.


HydraNFC decode sniffed NFC exchanges [Wireshark-RFID-dissector]

nickludlam commented on Feb 16
I've just got hold of the HydraBus + HydraNFC hardware, and I've sniffed my first NDEF record read + write.
With the text file, is there any way I can use a graphical tool to visualise the exchanges? I've looked at Wireshark-NFC, but that expects pcap format files. Would there be an easy way to convert the ascii sniffer output to pcap at all?

bvernoux commented on Feb 16
Yes a pcap ng format will be a very nice feature.
I have planned that and need to add it in TODO also any help/suggestions are welcome.
The idea will be to save at end both format .txt and .pcapng with same name
The best will be to add that idea in hydrafw issue here https://github.com/bvernoux/hydrafw/issues

stupidlinh commented 18 hours ago
I will try to write a piece of code to translate it to pcap.

bvernoux commented 18 hours ago
Nico has done such work see https://github.com/NicoHub/Wireshark-RFID-dissector
I'm just waiting he release the firmware source code to merge it on hydrafw

Hi Benjamin,
I imported/merged Nico's changes into my fork (no commits so far) - just to have a look.
While this looked promising initially, I am not too sure if it needs to be developed further: Wireshark LUA script for dissection is not context aware, i.e. you can't 'remember' the mode, the card is in, to interpret a message correctly.
This greatly decreases the value of decoding as ideally, say, for 14443-4 you'd want to work with PCB frames to see complete APDUs; determine 'past ATS' stage for it etc.
This, in turn, makes me think that a simple-ish Python script, which would be context-aware, would do a much better job, not only decoding all protocol levels, but providing an APDU export that can then be decoded using other tools (or in fact in this very script). But then such a script can work on existing text output as it is today. PCAP is not really needed.
Do you think it's still worth working on?
PS In addition to UART, do you think SPI2 can be used for 'eternal sniffing'? That would help recording on RPi or clones where SPI is reliable whereas UART has clock issues etc.

It is an amazing project to do that with Python (Python 3.x as 2.x is dead now)
You are welcome to work on that and I'm impatient to see the result (with a bonus like an HydraNFC v2)
About using SPI2 for 'eternal sniffing' as of today SPI2 is used to communicate with the NFC chipset to configure it for sniffer mode and then it is free and so it is possible to re-use it with RPi in order to stream the sniffed data for example (with RPi as SPI slave) as I suspect it is what you would like to do ?
An other simpler / better alternative will be to use USART1 (which is free) as SPI (using USART1_CK (PA8) / USART1_TX(PA9) / USART1_RX(PA10)) with minimal SW modifications in order to connect it to a RPi SPI Slave and stream data on this link (HydraBus USART1 SPI Mode => RPi SPI Slave)
We can also discuss about that in live on IRC Freenode channel #hydrabus (my nickname is bvernoux)

First draft of a Python Script is available here - https://github.com/AAsyunkin-se/nfcdecoder

Very nice work congratulations !!


Less Expensive For Loops and Tphere Fix

https://github.com/RocketMod/Rocket/issues/781

Excuse me but I can't accept the change of foreach to for. While it's true that enumerating a collection is in theory faster this change is against the c# language paradigm. I am using alot of Linq calls which all add up a little of time, but they make the code readable and easier to maintain. To come back to your specified mentioned spots, I bet the time you save is insignificant to the time the code would spend in the body anyway, feel free to proof me wrong by setting up an example with a stopwatch. Generally speaking I would rather have clean easy to follow and read code then the 100% optimization, I would have chosen c++ over c# if that was my intent.

If you can make a commit with just the tphere fix, you maybe able to get it accepted.

I've manually merged the change for tphere, thanks for your contribution!


Add review comments from PR #15604

Add the following support versions to content/en/docs/concepts/storage/storage-classes.md
here is support/version matrix of various volume types look like:
EBS (1.11)
Cinder (1.11)
Glusterfs (1.11)
GCE PD (1.11)
Portworx (1.11)
Azure File (1.11)
RBD (1.11)
Azure Disk(1.11)
Flex Volume (1.13)
CSI (1.14, 1.15 - Alpha, 1.16 beta)
It might be worth updating this document and explicitly specify which version of k8s the volume plugin type supported expansion.

/assign


digital shades: html: fixed roll formula

*fixed all the rolls formulas with appropriate skill names (had forgotten the _  )

Just FYI, these changes will probably go live tomorrow.

i suppose so. Thanks! Do you know why my css does not/did not work???
2016-07-12 22:39 GMT+02:00 Phil<EMAIL_ADDRESS>
Just FYI, these changes will probably go live tomorrow.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Roll20/roll20-character-sheets/pull/1764#issuecomment-232173230,
or mute the thread
https://github.com/notifications/unsubscribe/ASdEVHeS6B_I8H3hxWpn3uAiencHct1Zks5qU_tzgaJpZM4JKGyM
.


Updating image for social sharing meta

Description & motivation
Currently the image appears when you share on social media (Twitter in particular) uses our old logo.
This fix is to bring the sharing image up to date with our new logo and brand guidelines.
Pre-release docs
Is this change related to an unreleased version of dbt?
No

Perfect, thank you so much!

Hey @annafil I don't think Madelyn has the level of permissions needed to approve, so I think you will need to approve this. Would you have a chance to review by the end of (your) day, so that I can merge tomorrow morning?

@lfracalossiFA  you should be able to merge this in now


Rename Repository and Composer Package

As the term "GAX" is a bit arcane, we should rename the repository and package. One option is as follows:

Repo: googleapis/gax-php => googleapis/php-core
Package: google/gax => google/api-core.

Because other repositories are using "common"  (e.g. api-common-java and nodejs-common), and also because we have other namespaces besides ApiCore in google/gax, we should consider the following as well:

Repo: googleapis/gax-php => googleapis/php-common
Package: google/gax => google/api-common.

Thoughts?

@michaelbausor Any updates on this? Is this properly prioritized?

I think that this can perhaps be closed. There is not a push to move away from "gax" across languages, so sticking with gax as repo and package name is acceptable. @dwsupplee @bshaffer please reopen the issue and the discussion if you disagree.


local ref to bunnify

coverage: 82.254%. remained the same
when pulling 30c6891cdfe7c3e0f06857284552ac8ea4c4ff3d on local-ref-outbox
into 3409d03df2a13215a2499507e287c7422049482b on main.


compile: what's the recommendation now that compile option is gone?

The compile option was removed in v3.
What's the best way to compile js/less files?
I used to have something like this in my docma-template.json file:
{
   compile: {
      "js/app.js": "js/app.min.js"
  }

Should I add some build scripts into mytemplate/package.json and call them in the prebuild() callback?

Recommendation is to use previously compiled files so the end-user builds their docs faster. So in your case, this means including the final minified JS file rather than integrating an uglify operation into the build process.
Workaround is to use preBuild() option. You don't call preBuild() yourself. It will be part of the docs-build process run by Docma core. In other words, preBuild() is a way to tell docma to run some operation every time the user builds their documentation.
For helpers you can use utils.js.minify() or utils.js.minifyFile() methods. These utils are not officially documented but they will not go anywhere since docma uses them for its own files. Or you can use your own functions/helpers...
module.exports = (template, modules) => {

    // Docma passes the following modules to this template module for your convenience.
    // _, Promise, fs, dust, HtmlParser, utils

    const { Promise, utils, fs } = modules;

    // Set template main HTML file (default is "index.html")
    template.mainHTML = 'main.html';

    // Template default options.
    template.defaultOptions = {
        title: 'Docs',
        header: true
        // other options specific to your templated web app
    };

    // ignore files relative to /template directory. other files in the root of the module directory are already ignored.
    template.ignore = [];

    // optionally run some code before the build has started.
    template.preBuild(async () => {
        // minify your JS files and write them out to destination...
        const minifed = await utils.js.minifyFile('path/to/original.js');
        return fs.outputFile('path/to/minified.js', minified.code); // returning a promise
    });
};


Comparing array with the same array with added object produces a non beautified object

I believe this behavior was supposed to be fixed in #13, so I would appreciate if you could tell me if I need to change some options or if this is actually a bug.

It's fixed in 1.0.19, please have a try :)

Fantastic, thank you very much!


Completed Longest String Exercise

How do I resolve the issue of two strings having the same length? What if I wanted to return both strings?

Great work, @sags1083 . Thanks for bundling all this into one pull request. You're off to a good start. :D
Also, thank you for pointing out the typos and errors in the source file. It's much appreciated.

Hey @sags1083, I'm going to close this PR since it looks like we covered the feedback in it.


Remove seal key secret management type from piped configuration

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/changelog


shaded dependencies to avoid conflicts if crate-jdbc

is used with another library which has a common
dependency

@kovrus @mikethebeer please review


Add query-validator command, and make set-validator check version.

Motivation
Before adding a new validator to a testnet we want to check whether they are online and have a compatible version and genesis config.
Proposal
Make set-validator check the validator's version and genesis config before creating the new committee. (Can be omitted with --skip-online-check.)
Add a query-validator command that works just like query-validators, but instead of querying the current committee it queries a given address, and it prints a warning if the version or genesis config is incompatible.
Test Plan
set-validator is already used, and query-new-validator was added to the reconfiguration end-to-end test.
Links

Closes https://github.com/linera-io/linera-protocol/issues/2288
reviewer checklist

I addressed the comments and rebased.


docs: properly format code in NFS configuration note

Currently, the code is rendered without newlines and proper formatting.
Signed-off-by: Tobias Klauser<EMAIL_ADDRESS>

This change is

test-docs-please

Coverage increased (+0.02%) to 44.704% when pulling 904bb785dbb3af12d291cff5c789d3e5958168f0 on tklauser:contrib-guide-fix-code-note-nfs into 161fcd49340561363521f71e76d4408dcae9fe5c on cilium:master.