Support for multiple resource in a single app

I was trying to have multiple resourceIds in a single app. Is it possible?
I tried having multiple classes with @EnableResourceServer but only one would be picked up. I also tried to have only one with multiple @Configuration that extend ResourceServerConfigurerAdapter but that wasn't working either?
Is it supported and if not, could it be? :)
Thanks.

This does not work for me since only the resource configuration with the higher order is utilized during authentication of the access token. What am I doing wrong?
`
@Configuration
@EnableWebSecurity
public class OAuth2ResourceServerConfig {
private static final String CLIENT1_REST_RESOURCE_ID = "client1_api";
private static final String CLIENT2_REST_RESOURCE_ID = "client2_api";

@Resource(name = "dataSourceOauth2")
private DataSource dataSource;

@Bean
public JdbcTokenStore tokenStore() {
    return new JdbcTokenStore(dataSource);
}

@Bean
protected ResourceServerConfiguration client2Resources() {
    ResourceServerConfiguration resource = new ResourceServerConfiguration() {
        public void setConfigurers(List<ResourceServerConfigurer> configurers) {
            super.setConfigurers(configurers);
        }
    };
    resource.setConfigurers(Arrays.<ResourceServerConfigurer> asList(new ResourceServerConfigurerAdapter() {
        @Override
        public void configure(ResourceServerSecurityConfigurer resources) {
            resources.tokenStore(tokenStore()).resourceId(CLIENT2_REST_RESOURCE_ID).stateless(false);
            OAuth2AuthenticationEntryPoint authenticationEntryPoint = new OAuth2AuthenticationEntryPoint();

            ExceptionEncodingTranslator translator = new ExceptionEncodingTranslator();
            authenticationEntryPoint.setExceptionTranslator(translator);
            resources.authenticationEntryPoint(authenticationEntryPoint);
        }

        @Override
        public void configure(HttpSecurity http) throws Exception {
            MultiTenantContext.setStrategy(new OAuth2MultiTenantContextStrategy());
            // define URL patterns to enable OAuth2 security

            // @formatter:off
            http.authorizeRequests() //
                    .antMatchers( //
                            "/mainpath/v2/api-docs", //
                            "/mainpath/webjars/**", //
                            "/**/favicon.ico", //
                            "/mainpath/swagger-ui.html", //
                            "/mainpath/swagger-resources/**", //
                            "/mainpath/health/**") //
                    .permitAll() //
                    .antMatchers("/mainpath/subpath5/subsubpath/**", //
                            "/mainpath/subpath4/**", //
                            "/mainpath/subpath3/**") //
                    .access("#oauth2.hasScope('read') or (!#oauth2.isOAuth() and hasRole('CLIENT2'))")
                    .antMatchers("/mainpath/**").denyAll();

            // @formatter:on
        }
    }));
    resource.setOrder(4);
    return resource;
}

@Bean
protected ResourceServerConfiguration client1Resources() {
    ResourceServerConfiguration resource = new ResourceServerConfiguration() {
        public void setConfigurers(List<ResourceServerConfigurer> configurers) {
            super.setConfigurers(configurers);
        }
    };
    resource.setConfigurers(Arrays.<ResourceServerConfigurer> asList(new ResourceServerConfigurerAdapter() {
        @Override
        public void configure(ResourceServerSecurityConfigurer resources) {
            resources.tokenStore(tokenStore()).resourceId(CLIENT1_REST_RESOURCE_ID).stateless(false);
            OAuth2AuthenticationEntryPoint authenticationEntryPoint = new OAuth2AuthenticationEntryPoint();

            ExceptionEncodingTranslator translator = new ExceptionEncodingTranslator();
            authenticationEntryPoint.setExceptionTranslator(translator);
            resources.authenticationEntryPoint(authenticationEntryPoint);
        }

        @Override
        public void configure(HttpSecurity http) throws Exception {
            MultiTenantContext.setStrategy(new OAuth2MultiTenantContextStrategy());
            // define URL patterns to enable OAuth2 security

            // @formatter:off
            http.authorizeRequests() //
                    .antMatchers( //
                            "/mainpath/v2/api-docs", //
                            "/mainpath/webjars/**", //
                            "/**/favicon.ico", //
                            "/mainpath/swagger-ui.html", //
                            "/mainpath/swagger-resources/**", //
                            "/mainpath/health/**") //
                    .permitAll() //
                    .antMatchers("/mainpath/subpath1/**", //
                            "/mainpath/subpath2/**", //
                            "/mainpath/subpath/**") //
                    .access("#oauth2.hasScope('read') or (!#oauth2.isOAuth() and hasRole('CLIENT1'))")
                    .antMatchers("/mainpath/**").denyAll();

            // @formatter:on
        }
    }));
    resource.setOrder(3);
    return resource;
}

}`

Was @jlmehta 's question ever answered? I'm going to be implementing this soon and following the pattern above. Is this known to still be working @dsyer ?

This is an issue tracker, not a support forum, so I don’t think it’s appropriate to get into much detail here. Stackoverflow would be better. Also the resource server features in this project have been superseded by Spring Security core. Check out the main user guide.

Was @jlmehta 's question ever answered? I'm going to be implementing this soon and following the pattern above. Is this known to still be working @dsyer ?

hi what approach did you end up using? need help on this soon


Selectable content in modals

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
In introductory modal and few others, including "Create New Diagram", text and icons can be selected outside of input boxes. But as far as I can call there is no purpose for such behavior in application.
To Reproduce
Steps to reproduce the behavior:

Open application and see introductory modal as on screenshot 1.
Place cursor at any point of application near by center.
Drag mouse down after left-click.
Content appears to be selected as on usual webpage.

Expected behavior
No content should have been selectable in modals, except for input boxes.
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

12.6.5

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome 79.0.3945.130

Additional context
Extra: hard to copy version number in "About draw.io..." modal. Modal immediately closes after releasing left mouse button, thus version cannot be copied so easily. Perhaps, some input box with convenient "Copy" button won't hurt.

IMO, this bot makes no sense. Issue remains the issue 🤷♂

The bot clears up issues that have been untouched for 100 days. You can remove the wontfix label by commenting, as you have.


Input geometry area limit

The aligner should have an option to limit the size of the input geometry, to prevent resource heavy processing.

@roefem Should this area be limited in the python-library, or is this only a concern for the webservice?
The python library can handle big geometries. It mostly depends on the combination of which referencelayer is used.
When using parcels, a lot of parcels are covered with a geometry of 100000m², but if you take for example municipality-borders as referencelayer it's maybe not a problem to have inputgeometries of 100000m².

@roefem ; When a dictionary of multiple geometries is processed, will only the big geometry be blocked, and all the other geometries will be processed?; or will the full dictionary be blocked?

I think it's the responsibility of the library to allow for a way to limit the area size, but the webservice should determine what this limit is.
Only the geometries that are too big will be skipped, the others will still be processed as normal

This issue seems resolved. Can it be closed?

for me , it can be closed,
@roefem, ok?


Question: Is this library a kotlin multiplatform compatible?

Hello, I'm not an experienced kotlin developer, but I tried to use your Kotlin-BLE-Library in Kotlin Multiplatform (K/MPP), because I want to share UI components for IOS and Android applications.
I have no problem with your library when I import it into "just kotlin for android environment", but I had no luck to make it work in multiplatform environment even if I add lib as android dependency only.
I suspect it has to do something with how libraries are compiled for JVM and K/MPP but I don't know how to verify it by myself, thus I'm asking my question - can I use your library in K/MPP environment?

Sorry, I found this https://github.com/NordicSemiconductor/KMM-BLE-Library


Use golangci-lint instead of gofmt and goimport

golangci-lint is a lot more flexible and easier to enable more checkers in the future.
see: https://golangci-lint.run/usage/linters/

Codecov Report

Merging #196 (bec1e48) into develop (801811b) will decrease coverage by 0.11%.
The diff coverage is 16.66%.

:exclamation: Current head bec1e48 differs from pull request most recent head 36acdab. Consider uploading reports for the commit 36acdab to get more accurate results

@@             Coverage Diff             @@
##           develop     #196      +/-   ##
===========================================
- Coverage    50.87%   50.75%   -0.12%
===========================================
  Files           38       38
  Lines         4246     4242       -4
===========================================
- Hits          2160     2153       -7
- Misses        1953     1956       +3
  Partials       133      133

Flag
Coverage Δ

unittests
50.75% <16.66%> (-0.12%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/adapters/grpc/document_grpc.go
0.00% <ø> (ø)

pkg/adapters/grpc/errors.go
96.42% <ø> (-0.09%)
:arrow_down:

pkg/adapters/grpc/event_grpc.go
31.74% <ø> (ø)

pkg/adapters/grpc/faas_grpc.go
0.00% <ø> (ø)

pkg/adapters/grpc/queue_grpc.go
0.00% <ø> (ø)

pkg/adapters/grpc/secret_grpc.go
0.00% <ø> (ø)

pkg/adapters/grpc/storage_grpc.go
0.00% <ø> (ø)

pkg/membrane/membrane.go
75.12% <ø> (-0.13%)
:arrow_down:

pkg/plugins/document/document.go
77.85% <ø> (-0.08%)
:arrow_down:

pkg/plugins/events/dev/eventing.go
0.00% <ø> (ø)

... and 22 more

Continue to review full report at Codecov.

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


Splitting doesn't split?

Forge: <IP_ADDRESS>36
Apotheosis: 1.12.2-1.9.2
When dropping an anvil on any enchanted book the anvil is damaged but no splits ever happen. I tried with high and low level books, multiple heights, etc. The enchanted book remains the same and the only one.

Splitting splits books with multiple enchants. Does not duplicate singular ones.


[Bug]: Clicking on the PRISM header in the top left breaks date functionality

What happened?
If I load a climate layer (10-day rainfall for example), and then click on the 'PRISM' text on the top left, the layer stays visible but the date is removed from the URL. If I then try to change the date through the calendar date picker, clicking on a tick in the timeline, or clicking on the left / ride side of the timeline, the layer doesn't update with the new date and the timeline also doesn't update.
We should remove the hyperlink in the header. It isn't needed
Which country / deployment are you running?
RBD on main
Add a screenshot (if relevant)

Fixed in #958


Fix disable editing buttons

This PR removes the buttons to edit the markdown of patterns if editing is switched off in the configuration.

Related to https://github.com/PatternAtlas/pattern-atlas-ui/issues/96 but it implements only a subset of the features

@lharzenetter The default configuration values which are used if the config server is not reachable are all set to true:
https://github.com/PatternAtlas/pattern-atlas-ui/blob/57b37d6a5608905d529453aaa8f75b371d44551d/src/app/core/directives/pattern-atlas-ui-repository-configuration.service.ts#L41

Hmm, I guess should make this false, what do you think?


Failed to verify and configure downloaded files...Again

Im having this issue again :(
I :: platform: wine
I :: image: "/home/thane/Games/Gameimage/build/wine.flatimage"
I :: Only checking SHA
I :: platform: wine
I :: image: "/home/thane/Games/Gameimage/build/wine.flatimage"
I :: Open file '"/home/thane/Games/Gameimage/build/gameimage.fetch.json"' as READ
I :: Optional (caught): Failed to open '/home/thane/Games/Gameimage/build/gameimage.fetch.json'
I :: Fetch file 'https://raw.githubusercontent.com/gameimage/runners/master/fetch.json' to '"/home/thane/Games/Gameimage/build/fetch.base.json"'
I :: key identifier: /home/thane/Games/Gameimage/build/fetch.base.json
I :: Generated message_queue key:<PHONE_NUMBER>
I :: Message queue id: 32804
I :: Download progress: 100%
I :: Open file '"/home/thane/Games/Gameimage/build/fetch.base.json"' as READ
I :: url base  : https://github.com/gameimage/runners/releases/download/wine-gnu-x86_64/base.flatimage.tar.xz
I :: Download url: 'https://github.com/gameimage/runners/releases/download/wine-gnu-x86_64/base.flatimage.tar.xz
I :: SHA256: Trying to find in url
I :: Fetch file 'https://github.com/gameimage/runners/releases/download/wine-gnu-x86_64/base.flatimage.tar.xz.sha256sum' to '"/home/thane/Games/Gameimage/build/wine.tar.xz.sha256sum"'
I :: key identifier: /home/thane/Games/Gameimage/build/wine.tar.xz.sha256sum
I :: Generated message_queue key:<PHONE_NUMBER>
I :: Message queue id: 32805
I :: Download progress: 100%
I :: SHA256 found in url
I :: Calculating SHA for: "/home/thane/Games/Gameimage/build/wine.tar.xz"
I :: SHA Calculated: FED15AF3DD832A77D54FC5E07BD834BCE1B079876881CD76FC485497E2AF9578
I :: SHA Reference : E5EE9877374869975E0ADA75BCD7DC30A06EC802E9382E6D3923149CE1134A85
I :: Could not verify with SHA: SHA failed for /home/thane/Games/Gameimage/build/wine.tar.xz
I :: SIZE: Reference is 398458880
I :: SIZE: Calculated is 530356592
Failed to verify and configure downloaded files

It's only when i try to verify wine options. linux option works fine

That seems to have done the trick, i archived and deleted the old build folder and regenerate a new one. Thanks 🙌


Add support for highlighting values

We should use either the string.quoted.double or constant.character scope name.

Closed by #21


outputEvents only works when selecting a date from the calendar popup.

It would be great if the outputEvents function were to also be fired when the user updates the date by using the input field.

In order to get the result I did the following:-

I'm not sure if that is the best way to do it however it would be good if we had something like that so that the control detects user input from the keyboard

Oh man that's a really good idea.  I used the ngDoCheck() event hook with some logic to achieve this, but it's more complex than I would like for it to be.

why not use (ngModelChange)="cb($event)"?

Doesn't work for me.


feat(search): handle search indexing coordination among nodes

Signed-off-by: Phil Kuang<EMAIL_ADDRESS>Hey, I just made a Pull Request!
Fixes #7784

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hey! Sorry to muddy the PR with unrelated discussion, but this thing made me think - especially in relation to the lunr question by @iamEAP - maybe the scheduling shouldn't be driven from inside the index builder at all? I see it copying some parameters that actually are just a passthrough into the scheduling.
Taking some inspiration out of the EntityProvider + Connection duality, would it make sense if the index builder offered to the ability to collators to push changes over some "connection like" object?
The installation would now be more like either
const c = MyCollator.fromConfig(...);
indexBuilder.addCollator(c);
await taskScheduler.scheduleTask(..., () => c.run());

Or maybe better,
indexBuilder.addCollator(
  MyCollator
    .fromConfig()
    .schedule(taskScheduler, { /* some of the options directly out of TaskScheduler */ })
);

Then the index builder would know nothing about scheduling, and instead it just calls async start() on each collator at startup and then every now and then receives "events" for large updates to push through.
And the lunr one would have a different schedule method with other arguments than the distributed ones. That method would not be part of the interface, but rather the concrete class as a convention, and just store the intent of what to do once the start happens, and then return this at the end. The lunr one would always schedule locally.
Alright, done waiting for builds, just food for thought :)

Hey! Sorry to muddy the PR with unrelated discussion, but this thing made me think - especially in relation to the lunr question by @iamEAP - maybe the scheduling shouldn't be driven from inside the index builder at all?

Interesting thoughts!  And brings up maybe more of a foundational question from me behind the intention of the task scheduler @freben: is there any particular intention behind where task scheduler code is invoked?  Was the intention to keep task scheduler invocation within a plugins/foo.ts execution context?  Or is it fine for a task scheduler to be injected into plugin code and then executed within the plugin?

It's fine to inject! This was more about clarifying responsibilities in this particular instance.

Hmm...  Maybe a little bit of mix-up between search engines and collators in your thinking, @freben?  Or maybe I'm just having a hard time parsing your pseudo-code.  I'm not sure it's appropriate for collators to know anything about scheduling.  Nor search engines either, really.  My mental model:

Collators: "All I know is how to retrieve documents."
Decorators: "All I know is how to modify and filter documents."
Search Engine: "All I know is how to store and query sets of documents."
Index Builder: "All I know is how to wire up collators, decorators, and search engines into a pipeline."

Now that the first three are actually all just streams, and the fourth more or less just pipes them together (sorry @kuangp, gonna need a rebase w/minor changes 😬 ) I'm not sure the "connection object" comparison with entity providers is quite the right mental model.

Perhaps the place to address the "lunr in prod" warning/error messaging is in the index builder's constructor?  We could (additionally) explicitly pass in the database manager (which would make it clearer that such a dependency exists--it's a little more hidden, when we only rely on the taskScheduler dependency), and check "is lunr search engine? is pg (or just not sqlite) knex client? then warn about possible misconfiguration"

Oh for sure, in a streaming scenario any comments regarding scheduling may not be relevant at all and can be ignored :) Sorry also if I mixed some terms.
If scheduling is still of interest, note that we just merged this one https://github.com/backstage/backstage/pull/10018/files#diff-9e7abe371d1c9ea77c6222428a56dba520d8a0cbf0d27c8d4c22e9a3fb62b6c8R81 which may be a pattern of interest; it just passes a TaskRunner into the constructor, e.g.
LdapOrgEntityProvider.fromConfig(config, {
  schedule: scheduler.createScheduledTaskRunner({
    frequency: Duration.fromObject({ minutes: 30 }),
    // ..

this detaches the class entirely from what scheduling "means", it's just a runner callback basically after that

thanks @iamEAP @freben for your feedback and apologies for the delay here (been OOO and busy recently).
I updated my change based on your suggestions to shift all scheduling logic to the consumer by having them be responsible for supplying a scheduled task runner (following the LDAP pattern). This also gives consumers more control over configuring task scheduling based on their use-case eg. using the TaskScheduler for distributed setups or implementing a "local" task runner when using lunr. I also added notes in the docs and changeset regarding the "local" setup to explicitly call out this use-case.
Let me know what you think!

@iamEAP made some small tweaks based on your comments, let me know what you think!


Docs general layout: variable width?

Currently, the width of the different pages on docs.codewars may be pretty narrow. In full screen mode, I get this:

Would be nice(r) to either:

set the default width larger?
make it autoadaptable to the width of the window itself?

This problem makes the glossary rather massive too: the middle column is as narrow there as it is in the picture above, making some longer definitions looking pretty... "bad" (see the preview of the PR #31 )

Yeah, the glossary page can be little wider. I'll adjust after merging your changes and seeing how it looks.

make it autoadaptable to the width of the window itself?

Normal pages (generated from Markdown) are responsive and currently like this:
480

640

768

1024

1280

1440

Your screen looks unusually wide. (I use ultrawide, but never have single window filling the screen.)
We'll adjust things as we go, but the main content shouldn't become too wide even for the super wide screen like yours. Maybe max around 100-120 characters per line.

yeah, right... x) My screen is 1440*2560.... XD
:+1:

closing.


Feature: Add Binary Search related problems in C++

Total count of problems (5 - 10)
Add question on top of program
Kindly explain approach with comments

@akgmage kindly assign this issue to me
and in which folder should I add the programs ?

@akgmage kindly assign this issue to me and in which folder should I add the programs ?

Binary search

ok and all the program in single file or separate file ? also is it mandatory to add only leetcode problems ?

ok and all the program in single file or separate file ? also is it mandatory to add only leetcode problems ?

Separate files, not mandatory lc problem

can i take this? i can add some more questions


3.0.0 does not match the docs

While reviewing the 3.0.0 release: https://my.diffend.io/gems/spdx/2.0.12/3.0.0#d2h-088930-307 I noticed that the find method has been removed but is still present in the master docs. Is that intentional? Where can I find the upgrade notes for this release?
Also, would you mind providing some insight on why you've removed the fuzzy matching methods? It worked remarkably well.

Sorry about that! The docs have been updated. We decided to remove the fuzzy matching methods from the 3.* series because we had some issues that led us to conclude that doing license matching with a levenshtein distance wasn't providing us with a sufficient accuracy level. Some examples:
Spdx.find("MirOS License (MirOS)") => #<Spdx::License:0x000055f6b9894cd8 @id="MIT", @name="MIT License", @osi_approved=true>
Spdx.find("BSD") => #<Spdx::License:0x000055f6b98bdb60  @id="BSD-3-Clause",  @name="BSD 3-Clause \"New\" or \"Revised\" License",  @osi_approved=true>
Spdx.find("MIT or GPLv3") => #<SpdxLicenses::License:0x0000564f1a7cb7c0 @id="MIT", @name="MIT License", @osi_approved=true>
Spdx.find("GWT Terms") => #<SpdxLicenses::License:0x0000564f1a6f7e48  @id="Unicode-TOU",  @name="Unicode Terms of Use",  @osi_approved=false>

@mellisio thank you for the update. Wouldn't it be solved if there would be a probability included in the result? Then matches below a certain threshold could be rejected.


Update swiftmailer to >= 5.4.4

Current version is 6.0.1 so it would be great to be able to use the newest version.
As Swiftmailer 5.4.4 has a security vulnerability at least a minor update would be reasonable.
http://www.cvedetails.com/cve/CVE-2016-10074/

This is the code I used to make sfMailer work with Swiftmailer 6.0.1:
Instantiation with $subject via "new":
public function compose($from = null, $to = null, $subject = null, $body = null)
{
    return (new Swift_Message($subject))
        ->setFrom($from)
        ->setTo($to)
        ->setBody($body)
    ;
}

and type hint in send():
public function send(Swift_Mime_SimpleMessage $message, &$failedRecipients = null)
{
    //...
}

Feel free to provide a PR to update the composer.json, the submodule and sfMailer (be careful about compatibility)

Migrate to swiftmailer 6 will break the php mail transport. That's ok?

I'm not ok.
Maybe we can provide a different sfMailer or a way to configure it to be used with switf 5 or 6.
I've checked swiftmailer 6.

Something to take into account is that Swiftmailer 6 has set PHP 7.0 as minimum.

Which means we won't upgrade the version within symfony1 to avoid losing PHP 5 compatibility

@j0k3r How about cloning sfMailer into a sfMailer6 of sorts with the needed changes to give the option of using swiftmailer 6 for those that are using PHP 7.0+? There would be a bit of code duplication, but on its own class it shouldn't interfere with the rest. Happy to try to put a PR together for this if you think this could be a viable option.

@j0k3r How about cloning sfMailer into a sfMailer6 of sorts with the needed changes to give the option of using swiftmailer 6 for those that are using PHP 7.0+? There would be a bit of code duplication, but on its own class it shouldn't interfere with the rest. Happy to try to put a PR together for this if you think this could be a viable option.

@mentalstring see #240

@mentalstring see #240

@KatharinaSt #240 has been merged. You can now use swiftmailer 6 :)


test: write benchmark for UFDP + QUIC

We have 2 benchmark types; one with criterion, and another binary based solution for distributed benches (the latter wip).
Both of these benchmarking strategies should support QUIC, along with the existing TCP implemented atm.
For quic, I noticed most of the libraries implement asyncwrite and asyncread on 2 separate structs. So we may need to switch our implementation to accept both halves instead of a single thing


Azure SDK Review - [Introduction to IC3 ACS CCaaP]

New SDK Review meeting has been requested.
Service Name: IC3 ACS CCaaP
Review Created By: Rajarshi Sarkar
Review Date: 02/07/2024 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links:
Description: Blocking time to review feature addition for maxConcurrentOffers on worker
Detailed meeting information and documents provided can be accessed here

Meeting updated by Rajarshi Sarkar
Service Name: IC3 ACS CCaaP
Review Created By: Rajarshi Sarkar
Review Date: 02/07/2024 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: .NET,
Description: Blocking time to review feature addition for maxConcurrentOffers on worker
Detailed meeting information and documents provided can be accessed here

Meeting updated by Rajarshi Sarkar
Service Name: IC3 ACS CCaaP
Review Created By: Rajarshi Sarkar
Review Date: 02/07/2024 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: .NET, Python,
Description: Blocking time to review feature addition for maxConcurrentOffers on worker
Detailed meeting information and documents provided can be accessed here

Meeting updated by Rajarshi Sarkar
Service Name: IC3 ACS CCaaP
Review Created By: Rajarshi Sarkar
Review Date: 02/07/2024 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Java, .NET, Python,
Description: Blocking time to review feature addition for maxConcurrentOffers on worker
Detailed meeting information and documents provided can be accessed here

Meeting updated by Rajarshi Sarkar
Service Name: IC3 ACS CCaaP
Review Created By: Rajarshi Sarkar
Review Date: 02/07/2024 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Javascript, Java, .NET, Python,
Description: Blocking time to review feature addition for maxConcurrentOffers on worker
Detailed meeting information and documents provided can be accessed here

Meeting updated by Rajarshi Sarkar
Service Name: IC3 ACS CCaaP
Review Created By: Rajarshi Sarkar
Review Date: 02/07/2024 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Javascript, Java, .NET, Python,
Description: Blocking time to review feature addition for maxConcurrentOffers on worker
Detailed meeting information and documents provided can be accessed here


Grant statuses have different capitalizations

Sorry I missed this ticket - these values will be obtained either from the backend or otherwise external from the UI. According to documentation, if present, grant.awardStatus can take one of three values (https://github.com/OA-PASS/pass-data-model/blob/master/documentation/Grant.md#status-options), so the previously capitalized grant status we saw before were incorrect.
The merged changes certainly look fine for the demos, but the display logic will need to be modified at some point to account for the actual values.


[Project Addition] Medical MNIST Classification using DL

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title : Medical MNIST Classification using DL
:red_circle: Aim : Classifying images of MNIST-like X-Ray, MRI and CT Scans into 6 classes.
:red_circle: Dataset : https://www.kaggle.com/datasets/andrewmvd/medical-mnist
:red_circle: Approach : Creating custom CNN model as well as 5 pre-trained models for multi-class classification task.

📍 Follow the Guidelines to Contribute in the Project :

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

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

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

:red_circle::yellow_circle: Points to Note :

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

:white_check_mark: To be Mentioned while taking the issue :

Full name : Arihant Bhandari
GitHub Profile Link : https://github.com/Arihant-Bhandari
Email ID<EMAIL_ADDRESS>Participant ID (if applicable): Arihant Bhandari (eternal_insight)
Approach for this Project : using pretrained and custom cnn models for classification (pre-trained: resent50, vgg16, inception, xception)
What is your participant role? (Mention the Open Source program)
GSSoC 2024 Contributor

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

Hi @Arihant-Bhandari you can not add more contributions in this repo. As per the guidelines suggested by the GSSoC team, no contributor should exceed 150 points in a single repo.
Closing this issue. Thanks for your overall contribution in this repository.


Move the deprecated admin API to a separate module

Moving the deprecated org.apache.geode.admin package out of geode-core and into a separate geode-old-admin module.

Could you please considered a different name like 'deprecated' that implies this shouldn't be used and may disappear at anytime.

Closing this out, without tests I don't think we can be confident in this move the old admin API.


[RFC] Add react-native-config (forked) dependency

New Dependency
Name: react-native-config (forked, for now)
URL: https://github.com/artsy/react-native-config
Motivation

Currently, Eigen uses cocoapods-keys to store environment configuration for the app (the list of used keys is here). CocoaPods only works for iOS apps, and we are planning in MX-489 to migrate to a cross-platform solution, to support upcoming Android work.
react-native-config is a good solution. However, it lacks one important feature from cocoapods-keys: key obfuscation. Please note that it is impossible to ship a binary to users that guarantees security of the keys, but key obfuscation makes it much more difficult to extract them. Unobfuscated keys are present as plaintext in the app binary, which is not acceptable. We have forked react-native-config and ported the obfuscation code from cocoapods-keys in this PR. Eventually, we'll need to recreate that logic for the Android app too.
A big upside to using react-native-config over cocoapods-keys is that we will migrate to storing local keys in a .env file, just like most other apps at Artsy. This will reduce the overhead for new contributors to the app. We will need to change our beta deploys to write the environment variables to a .env file prior to deploying.
Check List

[x] Have you read over the source code?
[x] Has had a release in the last year, or looks done and stable?
[x] Could you fit this codebase in your head after reading the source?
[x] Is this the stand-out obvious answer to a particular domain problem?
[x] Do you expect your team to be the only people who know about this dependency?

nope

[x] Is this obviously being used in production by the maintainers? Is it battle-tested?
[x] Does our bundle already include a (transitive) dependency that solves the problem and could we use that
instead?
[x] Do you feel well versed in the domain of this dependency and/or could you maintain it if that needs to become
an option?

Alternatives

We chatted about alternatives in MX Knowledge Share yesterday, and nothing came up.

react-native-config is nice, so def 👍 on that.
Are you planning to create a PR with the obfuscation code on the main repo? It would be good not to have to maintain a fork. Ah, just read the PR in the forked repo. Nice!

nice touch with the obfuscation.

Resolution
We decided to adopt the dependency.
Level of Support
1: Overwhelming positive feedback.
Additional Context:
None
Next Steps
Let's merge https://github.com/artsy/eigen/pull/3725 and then announce.
Exceptions
None.


vaadinRun fails with exception

When I run "gradle vaadinRun" I get the error "FAILURE: Build failed with exception". See below the error log from build/logs/payara.log. I'm using "com.devsoap.plugin.vaadin" version "1.2.0.beta1" and Windows 10.
As a workaround, I downgraded to "fi.jasoft.plugin.vaadin" version "1.1.12" and got it working.
Here's the error log. Note: "Käyttö estetty" is Finnish and means "Permission denied". I tried running in Administrator mode but it didn't help.
[INFO] Listening for transport dt_socket at address: 8000
[ERROR] kesäkuuta 27, 2017 9:51:11 IP. com.devsoap.plugin.PayaraServerRunner main
[ERROR] INFO: Starting Payara web server...
[ERROR] kesäkuuta 27, 2017 9:51:11 IP. com.devsoap.plugin.PayaraServerRunner main
[ERROR] INFO: Running on port 8080
[ERROR] kesäkuuta 27, 2017 9:51:12 IP. com.devsoap.plugin.PayaraServerRunner main
[ERROR] SEVERE: Failed to start Payara server
[ERROR] org.glassfish.embeddable.GlassFishException: java.io.IOException: Käyttö estetty
[ERROR] at com.sun.enterprise.glassfish.bootstrap.StaticGlassFishRuntime.newGlassFish(StaticGlassFishRuntime.java:139)
[ERROR] at com.devsoap.plugin.PayaraServerRunner.main(PayaraServerRunner.java:68)
[ERROR] Caused by: java.io.IOException: Käyttö estetty
[ERROR] at java.io.WinNTFileSystem.createFileExclusively(Native Method)
[ERROR] at java.io.File.createTempFile(File.java:2024)
[ERROR] at com.sun.enterprise.glassfish.bootstrap.StaticGlassFishRuntime.createTempInstanceRoot(StaticGlassFishRuntime.java:214)
[ERROR] at com.sun.enterprise.glassfish.bootstrap.StaticGlassFishRuntime.setEnv(StaticGlassFishRuntime.java:174)
[ERROR] at com.sun.enterprise.glassfish.bootstrap.StaticGlassFishRuntime.newGlassFish(StaticGlassFishRuntime.java:106)

Hi,
The initial impression I get from looking at the stacktrace is that your Windows TEMP directory is not writable as it should be. It seems that it is a common issue for Windows installations. Could you check if that is the case?
You can easily fix it by following the instructions at https://customer.real.com/hc/en-us/articles/204039863-Permission-to-write-to-TEMP-folder-and-Windows-system-folders

You are right, my TEMP directory is read-only. For some reason I was not able to make it writeable. There is no error message, it seems to succeed, but in the end it doesn't do anything.

You could also try to change the TEMP system property to point to a directory that is writable.
Unfortunately, this is more a system related issue the plugin can't do anything about really, you most likely will get the same when doing File.createTempFile(..) in any Java app, so closing this.

@johndevs I encountered the same issue. Did some investigation. I think the issue should be reopened.
What is happening:
Glassfish/Payar tries to create temporary file/directory under "c:\Windows".
It's because environment variables are not set (process is being created with empty environment), nor java.io.tmpdir is explicitly set.
Default value for java.io.tmpdir is resolved as:

1.The path specified by the TMP environment variable.
2.The path specified by the TEMP environment variable.
3.The path specified by the USERPROFILE environment variable.
4.The Windows directory.

Workaround:
vaadinRun {
    server 'payara'
    jvmArgs "-Djava.io.tmpdir=$System.env.TEMP"
}

@NiematojakTomasz What version of the plugin are you using? This should have been recently fixed in #437 by setting java.io.tmpdir to point to the build directory for the payara process.

@johndevs Oh, sorry. I thought I am using newest version.
I was using:
plugins {
    id 'com.devsoap.plugin.vaadin' version '1.2.0'
}

I am no longer encountering the issue when using:
plugins {
    id 'com.devsoap.plugin.vaadin' version '1.2.5'
}

Thanks!


Allow dictionary-like getters on result objects

Issue description
Result objects should have dictionary-like getters to avoid try/except blocks as logical flow when the shape of a response is not known.
Example: a json response like
{
    "some_key": {
        "nested_key": "nested_value"
    }
}

If we don't know the shape of the response, we'll have code like:

nested_key = None
try:
    nested_key = response.result.some_key.nested_key
except AttributeError:
    pass

Better:
nested_key = response.result.some_key.get('nested_key', None)

I have similar issue. There should be a way to access raw/original payload. Why is construct_object even a thing?

Hey @shulcsm,
We construct objects from HTTP results to allow for convenient dot notation access to results. I've submitted a PR for supporting the index operator and the in operator on result objects. Take a look and let me know what you think.


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 1.13.123 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add message routes content

@drnaylor-oc - Krishna has approved this PR. However, I would like you to review the changes to make sure that they are correct. Thanks.

@drnaylor-oc - Agree with your feedback. The table suffered a little from consolidation of information in one location and needed to be thinned out. I have moved some of the table content to the next sub-section. See 2nd commit.


rust-mode ignores my tab indentation settings

I have (setq-default indent-tabs-mode t) in my ~/.emacs file. This is being respected by all emacs modes appart from rust-mode. Now I do realize that it is recommended to use spaces in Rust, however I'd still like to use tabs. How can I make rust-mode respect my settings?

Rust mode is explicitly overriding it:
https://github.com/rust-lang/rust-mode/blob/c62185ae1c6edf0335261f169241eb8ee9713ad5/rust-mode.el#L1557
But it's easy enough to override with
(add-hook 'rust-mode-hook
          (lambda () (setq indent-tabs-mode t)))


ellipsis shows at end of tab name

As a result to the fix of #2 the rtl ordering does this on OS X:

Also, I'd recommend removing the gradient on the right hand side.

Can you provide me the exact width in px of your tab and the exact text content of the node please ? I can't reproduce (os x too here).

width: 264px

…/service_referral_form/index_spec.jsx

The ... always goes to the right hand side on my machine: http://codepen.io/aaronjensen/pen/WvLBmG


#359 Fixed typo

Can we add a test to make sure this doesn't get typo'd again?

Ping @marcelovani or @m1guelpf to implement this test.

I would rather just go and merge than wait another two years and find that the original reporter is still not interested anymore.

Thank you. :)

Backported this to 9.17.x series. Released in 9.17.1.


Integrating conversions from Arcane

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I was wondering if there was a way you can integrate importing of xp and/or levels from the Arcane bot
Describe alternatives you've considered
I have been manually transferring over levels with the [p]lvlset setlevel
Additional context

If you can provide me with  as much of the following as you can i might be able to do it.

documentation for their api
server ID i can use to test
API key (if it requires one)
the URL for the endpoint

Nevermind, They don't offer exports. I wish they did, as it would make life much easier.


Add Yarn resolutions link

I guess many people don't know what it is.

@allcontributors please add @diegohaz for docs


Fixed Anaxi temperature. Updated temperatures for a few herbs.

Changed statically indexed essences array to dynamically indexed. Additional herbs mapped. Removed temperature for unmapped essences.


Shutdown notification is quite annoying...

I find the rcon shutdown notification quite annoying because it does the notification in minute intervals. It would be nice if the time between notifications would be a bit longer, let's say 5min.
So actually I did exactly that. I modified the start_shutdown_timer function in the server controller file, where you can actually change the interval. In addition it will in all cases tell when 1 min is left for the update.
function start_shutdown_timer() {
    _t_val="$1"
    _interv=5
    _i=0
    _j=0

    while true; do
        if [ $_i == $_t_val ]; then
            break
        fi
        if [ $_j == $_i ] || [ $_i == $((_t_val-1)) ]; then
                notifier_debug "Shutdown Server in $((_t_val - _i)) minutes"

                if [[ ${CONANEXILES_Game_RconPlugin_RconEnabled} == 1 ]]; then
                        /usr/bin/rconcli broadcast --type shutdown --value $((_t_val - _i))
                fi
                if [ $_j == $_i ]; then
                        _j=$((_j+_interv))
                fi
        fi
        sleep 60
       _i=$((_i+1))
    done
}

Maybe it would be possible to push this edit also to master?

Can you provide the path to the script?

I am looking into it. Maybe we should add a Flag to configure this?

@TitanSpirit the timer functions should be located here: https://github.com/alinmear/docker-conanexiles/blob/master/src/usr/bin/conanexiles_controller
@alinmear I think it is a good idea. By adding '_interval'  to Docker environment variable this should work fine.

I updated the function; notification will be send at 15,5,1 Minute


feat: customizable model categories

https://github.com/gpustack/gpustack/issues/810
https://github.com/gpustack/gpustack/issues/598
Deprecate previous category fields and add a unified categories field.
It defaults to Auto in the UI and users can custom in advanced settings.

Updated


Instructions

Can you please create better detailed instructions in the README on how to perform inference and training? I am running into many errors. If you do this, I would contribute a Docker container for better reproducibility for this.

sorry about that. please open an issue to show us the exact errors.
On Fri, Aug 12, 2022 at 1:37 PM salinaaaaaa @.***>
wrote:

Can you please create better detailed instructions in the README on how to
perform inference and training? I am running into many errors. If you do
this, I would contribute a Docker container for better reproducibility for
this.
—
Reply to this email directly, view it on GitHub
https://github.com/xuebinqin/DIS/issues/27, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSGORIYTFA6OCQ2CNCJCMTVY2YXRANCNFSM56MZYXKQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Xuebin Qin
PhD
Department of Computing Science
University of Alberta, Edmonton, AB, Canada
Homepage: https://xuebinqin.github.io/

Hey I'm a total noob, also getting errors when I follow step 4:
/home/hobo/.local/lib/python3.8/site-packages/torch/nn/_reduction.py:42: UserWarning: size_average and reduce args will be deprecated, please use reduction='mean' instead.
warnings.warn(warning.format(ret))
building model...
batch size:  8
--- create training dataloader ---
------------------------------ train --------------------------------
--->>> train  dataset  0 / 1   DIS5K-TR <<<---
-im- DIS5K-TR ../DIS5K/DIS-TR/im :  3000
-gt- DIS5K-TR ../DIS5K/DIS-TR/gt :  3000
1  train dataloaders created
--- create valid dataloader ---
------------------------------ valid --------------------------------
--->>> valid  dataset  0 / 1   DIS5K-VD <<<---
-im- DIS5K-VD ../DIS5K/DIS-VD/im :  470
-gt- DIS5K-VD ../DIS5K/DIS-VD/gt :  470
1  valid dataloaders created
--- build model ---
--- define optimizer ---
/home/hobo/.local/lib/python3.8/site-packages/torch/nn/functional.py:3509: UserWarning: nn.functional.upsample is deprecated. Use nn.functional.interpolate instead.
warnings.warn("nn.functional.upsample is deprecated. Use nn.functional.interpolate instead.")
/home/hobo/.local/lib/python3.8/site-packages/torch/nn/functional.py:3631: UserWarning: Default upsampling behavior when mode=bilinear is changed to align_corners=False since 0.4.0. Please specify align_corners=True if the old behavior is desired. See the documentation of nn.Upsample for details.
warnings.warn(
[1]    7685 killed     /bin/python3 /home/hobo/Documents/dev/DIS/IS-Net/train_valid_inference_main.p
Any idea what's wrong please?
Thanks so much

The warning should be fine, it won't hurt the performance.
That's because newer version pytorch change the F.upsample to
F.interpolate, you can search for the upsample function and replace it with
the "interpolate" function. Or you can downgrade your pytorch version.
On Fri, Sep 2, 2022 at 12:00 PM Will @.***> wrote:

Hey I'm a total noob, also getting errors when I follow step 4:
/home/hobo/.local/lib/python3.8/site-packages/torch/nn/_reduction.py:42:
UserWarning: size_average and reduce args will be deprecated, please use
reduction='mean' instead.
warnings.warn(warning.format(ret))
building model...
batch size: 8
--- create training dataloader ---
------------------------------ train --------------------------------
--->>> train dataset 0 / 1 DIS5K-TR <<<---
-im- DIS5K-TR ../DIS5K/DIS-TR/im : 3000
-gt- DIS5K-TR ../DIS5K/DIS-TR/gt : 3000
1 train dataloaders created
--- create valid dataloader ---
------------------------------ valid --------------------------------
--->>> valid dataset 0 / 1 DIS5K-VD <<<---
-im- DIS5K-VD ../DIS5K/DIS-VD/im : 470
-gt- DIS5K-VD ../DIS5K/DIS-VD/gt : 470
1 valid dataloaders created
--- build model ---
--- define optimizer ---
/home/hobo/.local/lib/python3.8/site-packages/torch/nn/functional.py:3509:
UserWarning: nn.functional.upsample is deprecated. Use
nn.functional.interpolate instead.
warnings.warn("nn.functional.upsample is deprecated. Use
nn.functional.interpolate instead.")
/home/hobo/.local/lib/python3.8/site-packages/torch/nn/functional.py:3631:
UserWarning: Default upsampling behavior when mode=bilinear is changed to
align_corners=False since 0.4.0. Please specify align_corners=True if the
old behavior is desired. See the documentation of nn.Upsample for details.
warnings.warn(
[1] 7685 killed /bin/python3
/home/hobo/Documents/dev/DIS/IS-Net/train_valid_inference_main.p
Any idea what's wrong please?
Thanks so much
—
Reply to this email directly, view it on GitHub
https://github.com/xuebinqin/DIS/issues/27#issuecomment-1235814892, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSGORMNSJVKBS4BZINNIBDV4JFDHANCNFSM56MZYXKQ
.
You are receiving this because you commented.Message ID:
@.***>

--
Xuebin Qin
PhD
Department of Computing Science
University of Alberta, Edmonton, AB, Canada
Homepage: https://xuebinqin.github.io/


DartPad shows white screen with WebAssembly errors in Chrome and Edge

What happened?
4 days ago, I accessed DartPad using Google Chrome, the screen was completely white.
Steps to reproduce problem
When I tried to reload again, I decided to see what was goign on using the dev tools. And I found this:

I checked the official site and ensured I’m using the latest version of Chrome:

Despite updating and restarting Chrome, the problem persists.
Here are the steps I’ve taken:

Tried using other browsers:

Firefox(version: 130.0): Works fine.
Microsoft Edge(version: 128.0.2739.67): Same issue, even after updating with same errors and warnings.

Checked Google Chrome Incognito Mode and Guest Mode: Still the same issue.

What could be the main causes? Any answers would be appreciated.

I also posted aquestion on StackOverflow regarding the issue: https://stackoverflow.com/questions/78948694/dartpad-shows-white-screen-with-webassembly-errors-in-chrome-and-edge

Similar issue in Opera version: 114.0.5282.185
Failed to load resource: net::ERR_ADDRESS_INVALID dartpad.dev/:1 <meta name="apple-mobile-web-app-capable" content="yes"> is deprecated. Please include <meta name="mobile-web-app-capable" content="yes"> 2main.dart.wasm:0x253c16 Uncaught RuntimeError: illegal cast     at main.dart.wasm:0x253c16     at main.dart.wasm:0x251195     at main.dart.wasm:0x2510f2     at main.dart.wasm:0x253a48     at main.dart.wasm:0x25202d     at main.dart.wasm:0x253e95     at main.dart.wasm:0x25202d     at main.dart.wasm:0x253e95     at main.dart.wasm:0x25202d     at main.dart.wasm:0x25732d skwasm.js:14 Aborted() Oa @ skwasm.js:14 main.dart.mjs:72 JavaScriptError main.dart.mjs:72 at https://dartpad.dev/main.dart.wasm:wasm-function[2047]:0x13afb1 main.dart.mjs:72     at https://dartpad.dev/main.dart.wasm:wasm-function[2046]:0x13ae62 main.dart.mjs:72     at https://dartpad.dev/main.dart.wasm:wasm-function[2042]:0x13ad78 main.dart.mjs:72     at https://dartpad.dev/main.dart.wasm:wasm-function[2537]:0x143782 main.dart.mjs:72     at https://dartpad.dev/main.dart.wasm:wasm-function[15523]:0x27bf5c main.dart.mjs:72     at https://dartpad.dev/main.dart.wasm:wasm-function[15530]:0x27c2db main.dart.mjs:72     at https://dartpad.dev/main.dart.wasm:wasm-function[883]:0x11ce0b main.dart.mjs:72     at https://dartpad.dev/main.dart.mjs:244:81 skwasm.js:14 Aborted() Oa @ skwasm.js:14 main.dart.mjs:72 Another exception was thrown: Instance of 'minified:Class1179<void>' skwasm.js:14 Aborted() Oa @ skwasm.js:14 main.dart.mjs:72 Another exception was thrown: Instance of 'minified:Class1179<void>' skwasm.js:14 Aborted() Oa @ skwasm.js:14 main.dart.mjs:72 Another exception was thrown: Instance of 'minified:Class1179<void>' skwasm.js:14 Aborted() Oa @ skwasm.js:14 main.dart.mjs:72 Another exception was thrown: Instance of 'minified:Class1179<void>' skwasm.js:14 Aborted() Oa @ skwasm.js:14 main.dart.mjs:72 Another exception was thrown: Instance of 'minified:Class1179<void>' skwasm.js:14 Aborted() Oa @ skwasm.js:14 main.dart.mjs:72 Another exception was thrown: Instance of 'minified:Class1179<void>' skwasm.js:14 Aborted() Oa @ skwasm.js:14 25main.dart.mjs:72 Another exception was thrown: Instance of 'minified:Class1179<void>' skwasm.js:14 Aborted() Oa @ skwasm.js:14 32main.dart.mjs:72 Another exception was thrown: Instance of 'minified:Class1179<void>' main.dart.wasm:0x253b3d Uncaught RuntimeError: illegal cast     at main.dart.wasm:0x253b3d     at main.dart.wasm:0x251195     at main.dart.wasm:0x2510f2     at main.dart.wasm:0x253e9a     at main.dart.wasm:0x25202d     at main.dart.wasm:0x253e95     at main.dart.wasm:0x25202d     at main.dart.wasm:0x253e95     at main.dart.wasm:0x25202d     at main.dart.wasm:0x25732d
Opened BUG-1747 with Opera

Hi @Sithila-Sihan-Somaratne
I'm also facing the same issue on both Edge v131.0.2903.112 (Official build) (64-bit) and Google Chrome v131.0.6778.205 (Official Build) (64-bit) browsers with the latest version.
Also failing to browse embedded DartPad blocks on Flutter Dart API Docs
Troubleshooting DartPad also won't help.
Screenshots:

Any updates from Dart team on how to resolve or fix this?


update release version and add year to dates

@3nids sorry for not taking care of this one... I see current version has even more info then yours. (and this one has merge problems...) so closing this one.
Thanks


add null check for better testability

Short description of what this resolves:
Better testability.
Changes proposed in this pull request:

add null check to prevent null dereference while testing with karma

Ionic Version: 2.x / 3.x
Fixes: #11185

where should i put the null check code? in test case or the component? Also if in the  test case then where should i write it in the test case as i am not even able to create the page instance?.


document how to use HeliosSoloDeployment in tests

This is just a start.

@negz @davidxia @gimaker @rohansingh

Fixed some awkwardness in the wording and added a note about the localhost/<IP_ADDRESS> problem. Please review!

Current coverage is 44.92%

Merging #814 into master will increase coverage by +4.04% as of b52d1ca

@@            master    #814   diff @@
======================================
  Files          114     259    +145
  Stmts         4589   10852   +6263
  Branches       845    1511    +666
  Methods          0       0
======================================
+ Hit           1876    4875   +2999
- Partial        202     433    +231
- Missed        2511    5544   +3033

Review entire Coverage Diff as of b52d1ca

Powered by Codecov. Updated on successful CI builds.

@davidxia I have a feeling that won't be possible without a lot of junit hacking.
Assuming you are looking to avoid the time-wait of starting up helios-solo for each test class, I wonder if revamping how HeliosSoloDeployment works could also work:

HeliosSoloDeployment uses a fixed name for the container it starts (e.g. helios-solo-auto)
at startup, if the helios-solo-auto container is running, just use it
otherwise create and start that container
after 10 minutes (or some other value) of non-use, the container kills itself

I think the solo container should be a unique name per test run, otherwise other projects' ITs running on the same host will end up using the same solo container.

:+1:

@mattnworb Should the docs say to be aware of HELIOS_HOST_FILTER? https://github.com/spotify/helios/blob/master/helios-testing/src/test/java/com/spotify/helios/testing/HeliosSoloDeploymentTest.java#L66

@davidxia good point, they should. And/or we could hack TemporaryJobs to ignore filters when using HeliosSoloDeployment as the two concepts don't make sense together.

The latter makes more sense.


NotImplementedError: Writing as Excel with a MultiIndex is not yet implemented.

Description
I get the following error: when I run the example in the
raise NotImplementedError("Writing as Excel with a MultiIndex is "
NotImplementedError: Writing as Excel with a MultiIndex is

What steps will reproduce the problem?
1.temp_df = table.xs(manager, level=0)

2.temp_df.to_excel(writer,manager)
3.

What is the expected output? What do you see instead?
I expect to see it create an excel sheet called output with the data I asked t to write.

Please provide any additional information below

Version and main components

Spyder Version:  2.3.4
Python Version:  2.7.9
Qt Version    :  4.8.6, PyQt4 (API v2) 4.10.4 on Windows

Optional dependencies
pyflakes >=0.5.0:  0.8.1 (OK)
pep8 >=0.6      :  1.6.2 (OK)
IPython >=1.0   :  3.2.0 (OK)
zmq >=2.1.11    :  14.5.0 (OK)
pygments >=1.6  :  2.0.2 (OK)
pandas >=0.13.1 :  0.16.2 (OK)
sphinx >=0.6.6  :  1.2.3 (OK)
rope >=0.9.2    :  0.9.4 (OK)
jedi >=0.8.1    :  0.8.1 (OK)
matplotlib >=1.0:  1.4.3 (OK)
sympy >=0.7.3   :  0.7.6 (OK)
pylint >=0.25   :  None (NOK)

This is not a Spyder error, it seems to be a Pandas one. Please open a new issue here
https://github.com/pydata/pandas/issues


Security - Replace 'std::rand'

Security issue detected by the CI build:
https://sonarcloud.io/project/issues?id=imageworks_OpenColorIO&resolved=false&sinceLeakPeriod=true&types=BUG

Is this only a "security" issue because rand() isn't a strong enough to use for crypto -- which is not at all how we're using it?

Excellent point Larry, the Sonar reason is "The use of srand together with rand to seed the random number generator and then generate numbers usually produces low-quality randomness."  So it is not that the call itself is a security issue, just that the results aren't totally random (which we don't really care about for our usage).

This falls into a topic worth raising at the ASWF CI working group meeting (OpenEXR had the same question recently). As to whether we answer false positives with a comment in code and/or explicitly dismissing the warning in SonarCloud for the specific cases we know are not actually an issue.

All the tools (from gcc warnings, unit tests to SonarCloud) are here to help us finding hidden bugs. Running these tools is the mandate of the CI build system. However, the cost is to have frequent false-positive warnings because these tools are not perfect and not tailored to the project. But disabling warnings would only inhibit more and more validations over the time and finally hide problems. At Autodesk we experience that on various projects where some warnings are disabled on part of the code (i.e. muting potential problems) and/or never addressed leading to unusable compilation output (i.e. too many warnings).
Looking at this specific case, the fix is only a copy & paste of 2 lines, so a short pull request could easily change the code to keep the check and return SonarCloud to ‘green’.


Failed to "ack" for 3000 message(s). Reason: 3 INVALID_ARGUMENT: Request payload size exceeds the limit: 524288 bytes

NOTE:
I have already created Stackoverslow question for this issue and was asked by a Google Cloud Employee (Kamal Aboul-Hosn) to raise the issue over here.
Environment details

OS:
Node.js version:  v18.16.1
npm version: 9.5.1
@google-cloud/pubsub version: ^3.7.1

Steps to reproduce
I am using Node JS client for Google PubSub (RPC StreamingPullRequest API) with flow control. My subscription options look like that:
 {
      streamingOptions: {
        // decrease this to reduce re-deliveries
        maxStreams: 2, // default: 5
      },
      flowControl: {
        allowExcessMessages: false,
        maxMessages: 4000, // No issues at 2000
      },
    },

I reduced maxStreams from 5 (default) to 2 to reduce re-deliveries, as mentioned here in the docs. When I set the maxMessages to 4000, I get the following debug warning, which causes re-deliveries of messages:
Pubsub subscription received debug message: Failed to "ack" for 3000 message(s). Reason: 3 INVALID_ARGUMENT: Request payload size exceeds the limit: 524288 bytes.

Why am I getting the warning, and how can I increase the payload size?

@ablbol Thanks for the issue. I could've sworn we already had an issue about this, but either way:
The problem is that, as the message suggests, there is a maximum payload size for ack messages to the server. For a very high level of throughput or a very large batch size, it can overflow that size while trying to send back all of the ack IDs (especially with exactly once delivery enabled). Our intent here is basically to queue up multiple ack messages to the server if there are too many, but that hasn't been implemented yet.

We ran into a similar issue when our downstream system had backpressure from our downstream system.
The failed ack requests due to request size was leading to memory leak, eventually causing our workers to be OOM killed and restarted.
We were able to work around this by setting the BatchOptions.maxMessages parameter to be less than the default of 3000. We used 1000 and the requests then succeeded and we no longer leaked memory.

im testing a fix by adding a maxBytes param, to make sure it doesnt pass the threshold

@dermasmid Thanks for the PR. I think the max is always the same, and I'm not sure there's a case where you'd want to make it less than the server max? Let's run this by @kamalaboulhosn , we might be able to simplify further.

Quick update here - what we want to do with this is adapt the PR to not take a max from the user, but just to use the server's max. I need to verify what the exact number is, before doing that.

The 4.7.2 release should fix this, just released a moment ago.


Install gplsrc prereqs on Centos

When installing against Centos, the task " build | Install prereqs for building GPL from source"  is only checking RedHat os and not Centos.
An easy fix is just adding  RedHat OR Centos.
Or use Ansible_ value that cover both Centos And RedHat
And why are we using block for this task?
    - block:  ## when: rhel8 node
        - name: build | Install prereqs for building GPL from source
          dnf:
            name: "{{ scale_build_gplsrc_el8_prereqs }}"
            state: present
      when: ansible_distribution == 'RedHat' or ansible_distribution == 'CentOS' and ansible_distribution_major_version == '8'

Looks like this is fixed in Pullrequest:  https://github.com/IBM/ibm-spectrum-scale-install-infra/pull/98

@olemyk Yes i added this fix with mixed os support code .thanks

We may link #98 to this issue so that the issue is closed once the PR gets merged...

New method that use scale_rhel_distribution in branch staging_development:

Pull request:  [https://github.com/IBM/ibm-spectrum-scale-install-infra/pull/225] should fix this.

#225 got merged so this can be closed, I guess?


op-node in testnet cannot start due to missing required parameter

During make testnet-up I get this error:
$ docker logs ops-bedrock_op-node_1
t=2023-10-30T07:26:51+0000 lvl=info msg="Initializing Rollup Node"
t=2023-10-30T07:26:51+0000 lvl=eror msg="Unable to create the rollup node config" error="flag auth-token is required"
t=2023-10-30T07:26:51+0000 lvl=crit msg="Application failed"       message="flag auth-token is required"

I remedy this by adding --auth-token="".
Note:
For devnet mode, auth-token is set to $CELESTIA_NODE_AUTH_TOKEN.
I assume for testnet mode the celestia light client node is accessible without an auth token.

Fixed in #254 as flag is no longer required.


can't refresh page in server after build

cant refresh page in server after build. teh problem is internal server error. after build the project what needs to be adjusted(htaccess or code in project?)

Hello there, @evaniac ,
This is not a bug with our products, but rather with React and React-Scripts, to solve this you can either change the BrowserRouter with HashRouter, or change the webpack config: https://stackoverflow.com/questions/49189880/blank-page-when-refreshing-in-react-router
Best,
Manu


feat-multi-device Hands will duplicate across Service Providers when insufficient Leaps are connected

Creating a scene with multiple Leap Service Providers will cause hand data to be duplicated across any assigned a value greater than the number of individual Leap Motions connected. This occurs both when setting a Leap Service Provider to either a serial number or when using a device array index.
Feels as though it is the opposite of what should be occurring?
Any subsequent Leaps connected during run time will start overriding their correct index or serial number.

Two devices connected, Service Providers 1, 3 and 4 using hand data from device 1. Device 2 is covered and thus not reporting anything. When device 3 is connected (either during run time or before playing) the 3rd Service Provider will start operating uniquely.
Running in Unity 2018.3.0f2, tested in both Editor and Built projects on multiple machines.

I'm having the same problem here to handle multiple devices in the same project. Did you end up finding a solution?


Enable Content-Security-Policy reporting

This will report content security policy violations to Sentry, but will not block content.  This might be a useful step toward discovering if the rules configured here are safe.
Enketo Express requires a much more permissive policy than central-frontend, so its policy is configured separately.
It might be useful to pull the sentry config (organization subdomain, project ID, API key) into e.g. docker-compose.yml, as after https://github.com/getodk/central/issues/311 is fixed, they will all have to be configured twice (once for nginx, once for central-backend).

This will report content security policy violations to Sentry, but will not block content.

That sounds like a reasonable next step to me. So we would ship Content-Security-Policy-Report-Only with the next release (v2022.3), then maybe ship Content-Security-Policy in the release after that?
Once we feel reasonably confident about our Content-Security-Policy-Report-Only setting, maybe we could put that on some ODK Cloud servers even before v2022.3.

I just noticed that a report of a CSP violation will include query parameters, including the sensitive st query parameter for public links. In other cases, we remove the st query parameter before sending the error to Sentry: https://github.com/getodk/central-backend/blob/87aedb0b58973d7454ddd3a543fad7547646bcdd/lib/external/sentry.js#L19-L21. For a CSP violation, is there a way that we could filter out the st query parameter (or all query parameters)? Just a thought: could we set the report-uri to a Backend endpoint that does that filtering before sending on the violation to Sentry? (If we did that, we also wouldn't need to plumb the Sentry configuration to nginx because Backend would be the one communicating with Sentry.)

I'm also noticing that the settings page in Sentry for Content Security Policy says:

We recommend setting this up to only run on a percentage of requests, as otherwise you may find that you've quickly exhausted your quota.

Is this something worth doing for us? Maybe we wouldn't have enough CSP violations to risk exhausting our quota?

Maybe we wouldn't have enough CSP violations to risk exhausting our quota?

Currently the number is very low, and I'm working to decrease.

For a CSP violation, is there a way that we could filter out the st query parameter (or all query parameters)?

It looks like this could be done on the Sentry side with https://docs.sentry.io/product/data-management-settings/scrubbing/server-side-scrubbing/.  Would that be sufficient?

Just wanted to note that it looks like there's now a conflict.

It looks like this could be done on the Sentry side with https://docs.sentry.io/product/data-management-settings/scrubbing/server-side-scrubbing/. Would that be sufficient?

I feel like it'd be preferable to do that scrubbing before the data even reaches Sentry. That'd also make it clear to anyone interested that that scrubbing is happening: folks can't look at our server-side Sentry setup, but they can look at our source. But that's just my two cents / it might be worth discussing further as a team. Maybe it'd depend on how hard it'd be to add a Backend endpoint to modify the violation report before forwarding it on to Sentry?

Do we not want to see IP addresses in Sentry? It looks like those are coming through the CSP reports as well.

To avoid collecting end user IP addresses, proxying the CSP report requests through nginx might be a simple option, and the query param scrubbing could be done there too - I'll look into this.

@matthew-white I've updated this PR to scrub IPs by proxying the reports through nginx.
It's less clear how easy it is changing values inside the forwarded request body.  Would it be acceptable to do this in Sentry?

I've updated this PR to scrub IPs by proxying the reports through nginx.

Just to make sure I understand what's happening here, will the IP address of the server still be shown in Sentry? I think we probably want to avoid that as well. If that's what's happening, maybe we could keep the new /csp-report endpoint, but proxy all CSP reports through data.getodk.cloud/csp-report specifically?

It's less clear how easy it is changing values inside the forwarded request body. Would it be acceptable to do this in Sentry?

I could see how that might be the easiest thing to do. CC @lognaturel for a second opinion on whether it's important to scrub data before it reaches Sentry.

Just to make sure I understand what's happening here, will the IP address of the server still be shown in Sentry?

Yes.  But at this point I'd be interested to know what proxying through data.getodk.cloud is trying to protect against - if a user wants to protect their server's IP address being known to Sentry, then they have little reason to trust data.getodk.cloud vs sentry.io, and should probably just disable Sentry completely.
Also, end users are already sending their IP address to https://getodk.github.io/central/news.html every time they load the frontend homepage.  This should probably be considered alongside CSP-related IP address sharing.

I think that we shouldn't expect many violations to be reported by the time we release this and that as @alxndrsn states there are ways for users who really don't trust us to opt out of all of this.
I'd rather we focus on making sure we feel really good about the rules and not worry too much about the reporting, especially since it's short-term.

I did some research to try to understand what we should do about IPs.
Any service you use will store IP addresses as part of maintaining the security and integrity of that service. There's nothing wrong or illegal with that. Under GDPR, that's considered a "legitimate interest" and only requires opt-out.
Personally, what I'd like us to avoid is storing IPs of non-Cloud users so we can't identify them. Ideally, we'd anonymize client side, but it's not terrible if Sentry can do it server side for us.
Reducing client IP and query param exposure via the local nginx server is straightforward and helpful and so let's keep doing that. Proxing everything through data.getodk.cloud doesn't seem helpful because we are just shifting the storage of IPs from Sentry's servers to our servers.
Because our goal is increase the security of Central and reduce the number of violations over time, I think the above approach is reasonable, defensible, and legal.

OK cool, that's helpful! It sounds like @alxndrsn's approach should work well for IP addresses.
It's looking like it might be hard to scrub the session token query param within nginx. If I understand correctly, the query param isn't on the URL itself, but instead is in the request body: the query param is part of a value within a JSON report. @yanokwa, is it OK to scrub the st query param within Sentry? (Typically, the value of that query parameter is a session token that provides access to a form.) The alternative would probably be to do that scrubbing within a new Backend endpoint.
@lognaturel, that makes sense about focusing more on the rules than the reporting. I do think that we're planning to keep the reporting indefinitely: even once we change the header from Content-Security-Policy-Report-Only to Content-Security-Policy, we'll still specify report-uri so that we continue to hear about violations.

I'm eager to get it back on the QA server so we can see how it does! Looks like https://github.com/getodk/central/pull/313#discussion_r981573982 and https://github.com/getodk/central/pull/313#discussion_r981554605 need resolution.


Native support for google drive,mega like third party online storage

🚨🚨 Feature Request

[x] Related to an existing #400

A method or support for  google drive,mega,Microsoft Dropbox to store the dateset and we can use them for personal usage so the users get a option whether they want to store to their own personal accounts or hub servers it may save storage for hub.

duplicate


⚠ Bofrost.be has degraded performance

In 4a3a219, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1626 ms

Resolved: Bofrost.be performance has improved in dc8a44b after 20 minutes.


[PAX-11407] [merge after 8/25] removing client_secret_env_key from manifest

Summary
Jira ticket: https://jira.tinyspeck.com/browse/PAX-11407

The environment variable name client_secret_env_key became redundant and the internal 3p auth users are supposed to be using a new subcommand  add-secret instead of the .env variable
Therefore we are removing the field client_secret_env_key from the SDK object
The webapp should be okay to accept the oauth2 object now if this field is optional there (TODO check)
There will be a CLI change as well to get rid of this field completely. PR
This PR can be merged after the new add-secret subcommand is available to the end user.

How to test:

This needs to be tested after the client_secret_env_key field becomes optional in the PR
We need to update the SDK to point to this version by creating an app and updating the import_map.json file
Try updating the manifest with a provider object that doesnt have the client_secret_env_key field
There should be no error when the app is deployed
Both client_secret and client_secret_env_key field should be empty in the providers table in db
import.map file example:

{
  "imports": {
    "deno-slack-sdk/": "https://raw.githubusercontent.com/slackapi/deno-slack-sdk/6527d84/src/",
    "deno-slack-api/"<EMAIL_ADDRESS>  }
}

Manifest.ts file in app file example:

const GoogleProvider = DefineOAuth2Provider({
  provider_key: "google",
  provider_type: Schema.providers.oauth2.CUSTOM,
  options: {
    "provider_name": "Google",
    "authorization_url": "https://accounts.google.com/o/oauth2/auth",
    "token_url": "https://oauth2.googleapis.com/token",
    "client_id":
      "1086877417652-b1o23iogo2dqithh2rrspt967i70kig6.apps.googleusercontent.com",
     //  "client_secret_env_key": "google_client_secret", --> this is deleted
    "scope": [
      "https://www.googleapis.com/auth/spreadsheets",
      "https://www.googleapis.com/auth/userinfo.email",
      "https://www.googleapis.com/auth/userinfo.profile",
    ],
    "authorization_url_extras": {
      "prompt": "consent",
      "access_type": "offline",
    },
    "identity_config": {
      "url": "https://www.googleapis.com/oauth2/v1/userinfo",
      "account_identifier": "$.email",
    },
  },
});

Requirements (place an x in each [ ])

[X] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[X] I've read and agree to the Code of Conduct.

@nupurgoyall , can we merge this right now ? What is the SDK release cutoff ? Is it the same as the cli?


[Enh]: Preserve null values in old Polars versions for ewm_mean

Per  https://github.com/narwhals-dev/narwhals/pull/1298#discussion_r1834009043

Addressed with #1574


Introduce mls-rs-uniffi

Issues:
Addresses #81.
Description of changes:
This new create is a thin wrapper around mls-rs which exposes a subset of it’s interface using a UniFFI-compatible API.
We will expand this over the next few weeks to be a full API usable by messaging apps on different platforms.
Call-outs:
The code currently only works with mls_build_async, but this should be fixable in the future. From talking to @mulmarta, there should be a way to use traits to handle both cases.
Testing:
I've mostly looked at the auto-generated Kotlin code to see that it has a useful shape. I expect us to add/remove things from the API in the next few weeks.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT license.

To add more context to the sync build problems, the following code shows the issue and one not too satisfying solution. @stefunctional maybe you have ideas how to improve?
uniffi::setup_scaffolding!();

#[maybe_async::must_be_sync]
async fn bar() {}

#[derive(uniffi::Object)]
struct Foo;

/*
// This does not work because maybe_async must be called _before_ uniffi
// so that uniffi sees the sync function. On the other hand, uniffi must be called
// on the impl block which is higher up (and maybe_async can't be called on
// the impl block).

#[uniffi::export]
impl Foo {
    #[maybe_async::must_be_sync]
    async fn foo(&self) {
        bar().await
    }
}
*/

// This works
#[maybe_async::must_be_sync]
trait FooType {
    async fn foo(&self);
}

#[maybe_async::must_be_sync]
#[uniffi::export]
impl FooType for Foo {
    async fn foo(&self) {
        bar().await
    }
}

fMeow/maybe-async-rs#24 should help with the interoperability between #[uniffi::export] and maybe_async macros.

fMeow/maybe-async-rs#24 should help with the interoperability between #[uniffi::export] and maybe_async macros.

Thanks @stefunctional, that sounds great! I'll take a look!

fMeow/maybe-async-rs#24 should help with the interoperability between #[uniffi::export] and maybe_async macros.

Thanks @stefunctional, that sounds great! I'll take a look!

This worked beautifully, I can now compile both with and without mls_build_async!

Another thing @mgeisler there seems to be no way to apply_pending_commit? Should process_incoming_message maybe take message or ack?

Codecov Report
Attention: Patch coverage is 0% with 244 lines in your changes are missing coverage. Please review.

Project coverage is 90.73%. Comparing base (cdc826f) to head (8497736).

Files
Patch %
Lines

mls-rs-uniffi/src/lib.rs
0.00%
244 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #82      +/-   ##
==========================================
- Coverage   91.46%   90.73%   -0.74%
==========================================
  Files         168      169       +1
  Lines       30306    30548     +242
==========================================
- Hits        27719    27717       -2
- Misses       2587     2831     +244

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


Issue with importing

When I try to import a char this massage appears

What version of:

Foundry
5e
DDB Importer

are you using, and please provide a character url

https://www.dndbeyond.com/characters/74619425
thanks

yes it works now (-: Thanks


Directory structure recommendation for multiple slide decks?

Hi @cpojer, first of all, thanks for ReMDX, really cool project!
Looking at the demo repos, it seems like the slides.re.mdx is referenced in index.html in the root:
https://github.com/nkzw-tech/dev-velocity-talk/blob/c1f3cb336e29d3b16d652f815af5e83e0ec57521/index.html#L20

Is it ok to have multiple slide decks per repo / folder (while still maintaining one package.json with the vite, @nkzw/vite-plugin-remdx dependencies and the associated scripts)? eg. by passing in another CLI arg to npm run dev which would be passed along to the index.html file...
If there's nothing technically holding this back, it would be great to get a recommendation on directory structure for multi-slide-deck folders or repos (either just in this issue as a reply or also in the readme as permanent documentation)

Hey Karl,
thanks so much for trying out ReMDX! ReMDX should work and scale just fine with any number of slide decks in any type of repo configuration that you have. For example, my primary Athena Crisis presentation works by integrating ReMDX directly into the game's monorepo, which allows me to add any parts of the game into a slide deck. ReMDX at its core is just a Vite plugin that transforms MDX (with "slide page breaks") into a data structure with React components. Once you import a remdx file in JS, you can do anything you like: merge multiple slide deck files together, further transform the deck, or render multiple decks on the same page.
If you are working with a monorepo setup, add ReMDX to the frontend project next to where you are already using Vite. If you just want to have multiple slide decks in one repo, what you described should work out of the box: just create multiple index.html files – one for each slide deck – or use one index file and toggle based on the URL or build a UI to select the slide deck that should be shown:
const currentDeck = new URLSearchParams(location.search).get('deck');

const deck = currentDeck === 'apple' ? import('./apple.re.mdx') : import('./banana.re.mdx');

render(document.getElementById('app'), deck);

With this you can have one index.html file and toggle the deck via ?deck=apple and ?deck=banana. This should give you enough of an idea of how flexible ReMDX is. If you like, you can also build a React UI to select slide decks and manage loading and showing them via React state.
In terms of recommendations for a repo structure, my goal is generally to allow for flexibility. Do whatever works for you, and if it doesn't work or breaks down, let's figure out a path forward. The only thing I'd like to do over time is add a more opinionated set of basic components in a separate package. Currently ReMDX is barebones because I don't want to be too prescriptive, and it's also why I open sourced some of my slide decks so you can get an idea of the core components you'll likely need to put together a presentation.
I'm going to close this issue since there are no changes necessary for ReMDX, but feel free to keep the conversation going if you have questions.

Great thanks! I'll look at these options and see what works.
I'm guessing for our slide decks (lecture slide decks for upleveled.io), we'll want to both run dev server and also build specific slide decks from the command like, so I'll look at how I can deal with the argument of the slide deck name from there.
I'll circle back around and report my findings / approach when I'm further :)

In that case instead of using the URL search for state you could use an env variable that you set through your Vite config based on the params that your dev server or build process is started with.

So, some progress - not quite there yet though.
Here are the things I've tried, and the dead ends I've run into with each option:

Vite define option + dynamic import - run with pnpm dev <slide deck name>, where "dev": "vite dev --" in package.json
Upsides: Works in development, also with hot reloading
Downsides: Build fails, ...Cp(()=>import("./slides.re.mdx"),[])) appears in dist/assets/index-7dc2a7e8.js built file, with no slides.re.mdx in the dist/ folder

// vite.config.ts
import remdx from '@nkzw/vite-plugin-remdx';
import react from '@vitejs/plugin-react';
import { defineConfig } from 'vite';
export default defineConfig(() => {
  return {
    define: { __DECK__: JSON.stringify(process.argv[4]) },
    plugins: [remdx(), react()],
  };
});

Custom Vite Plugin for dynamic slide content using Vite Virtual Modules
Upsides: Works with dev mode and build mode
Downsides: hot reloading is broken (I assume because of my fs.readFile in the Vite config)
import remdx from '@nkzw/vite-plugin-remdx';
import react from '@vitejs/plugin-react';
import { readFile } from 'fs/promises';
import { defineConfig } from 'vite';

function virtualSlideDeckPlugin() {
  const virtualModuleId = 'virtual:slides.re.mdx';
  const resolvedVirtualModuleId = '\0' + virtualModuleId;

  return {
    name: 'dynamic-html-plugin',
    enforce: 'pre',
    resolveId(id: string) {
      if (id === virtualModuleId) {
        return resolvedVirtualModuleId;
      }
    },
    load(id: string) {
      if (id === resolvedVirtualModuleId) {
        return readFile('slides.re.mdx', 'utf-8');
      }
    },
  };
}

export default defineConfig(({ mode }) => {
  return {
    plugins: [dynamicHtmlPlugin(), virtualSlideDeckPlugin(), remdx(), react()],
  };
});

@rollup/plugin-virtual to provide dynamic html for entry point - delete index.html and past HTML content into Vite config
Upsides: None, doesn't work yet
Downsides: Could not get this working - Vite server returned 404
// vite.config.ts
import remdx from '@nkzw/vite-plugin-remdx';
import react from '@vitejs/plugin-react';
import { defineConfig } from 'vite';
import virtual from '@rollup/plugin-virtual';

export default defineConfig(({ mode }) => {
  return {
    build: {
      rollupOptions: {
        input: 'html',
        plugins: [
          virtual({ html: `<html content here>`}),
        ],
      },
    },
    plugins: [remdx(), react()],
  };
});

Also tried creating a Vite plugin with the Vite transformIndexHtml option to dynamically transform the import module id in the HTML, but this ran too late, leading to module resolution problems

Ok, got something working  🚀
Digging a bit deeper on option 4 with the Vite transformIndexHtml option, it seems like there's an option order: 'pre' to run this before the module resolution:
The CLI interface can be used to run a deck at ./slideDecks/deck1/slides.re.mdx with this command:
pnpm dev deck1

Building is pretty much the same:
pnpm build deck1

Files:
package.json
{
  "name": "slides",
  "version": "0.0.1",
  "type": "module",
  "scripts": {
-    "build": "vite build",
-    "dev": "vite dev"
+    "build": "vite build --",
+    "dev": "vite dev --"
  },
  "dependencies": {
    "@nkzw/remdx": "*",
    "react": "^18.2.0",
    "react-dom": "^18.2.0"
  },
  "devDependencies": {
    "@nkzw/vite-plugin-remdx": "*",
    "@rollup/plugin-virtual": "^3.0.2",
    "@types/react": "^18.0.26",
    "@types/react-dom": "^18.0.9",
    "@vitejs/plugin-react": "^4.0.0",
    "prettier": "3.2.5",
    "vite": "^5.0.0",
    "vite-plugin-dynamic-import": "^1.5.0"
  },
  "pnpm": {
    "overrides": {
      "shiki": "^0.11.0"
    }
  }
}

vite.config.ts
import remdx from '@nkzw/vite-plugin-remdx';
import react from '@vitejs/plugin-react';
import { defineConfig } from 'vite';

function dynamicSlideDeckImportPlugin() {
  return {
    name: 'html-transform',
    transformIndexHtml: {
      order: 'pre',
      handler(html: string) {
        return html.replace(
          './slideDecks/__SLIDE_DECK_NAME__/slides.re.mdx',
          `./slideDecks/${process.argv[4]}.re.mdx`,
        );
      },
    },
  };
}

export default defineConfig(({ mode }) => {
  return {
    plugins: [dynamicSlideDeckImportPlugin(), remdx(), react()],
  };
});

index.html
<!doctype html>

    <meta charset="utf-8" />
    <title>ReMDX</title>
    <meta
      name="viewport"
      content="width=device-width, user-scalable=no, initial-scale=1, viewport-fit=cover"
    />
    <meta name="apple-mobile-web-app-capable" content="yes" />
    <meta name="apple-mobile-web-app-status-bar-style" content="default" />

That's awesome, love seeing that progress 🎉


Publish crates in alphabetical order.

Please help keep the CHANGELOG up to date by providing a one sentence summary of your change:
Publish crates in alphabetical order. 😄
Part of https://github.com/rusoto/rusoto/issues/1610 .

A small change and tested locally so I think this is good to go. 👍


Add example config files to the repo

As part of my work on #239, I want to be able to produce an example script (or snakemake file) to show how to run a SA. The problem is that the data is not part of this repo (and contains hardcoded paths; see #264) and ideally I don't want to produce an example that requires getting data files from elsewhere.
Given that the data doesn't seem sensitive, it might make sense to just add these config files to this repo. Yes, there are some binary files in there, but they are tiny, so the filesize isn't worth worrying about IMO.
It's nice for users to be able to run an example directly from a repo directly, without having to try to obtain the data from elsewhere or construct their own based on the docs.

Good idea and the file sizes are indeed tiny. We should avoid changing them around too much obviously!
Over in pyrealm I followed a suggestion from @dalonsoa of having a second package within the project to contain pyrealm_build_data. We could do that with vr_demo_data package, which could then be imported and used in docs (and maybe tests), but which users could also import and access. Makes for nice path discovery via importlib for one thing.

@davidorme That's a good idea. It means if anyone downloads it e.g. over PyPI they still get the example data.


v1alpha1.ScrapeConfigSpec metricsPath is not converted

The metricsPath field of v1alpha1.ScrapeConfigSpec is not converted by the operator. v1beta1.VMScrapeConfigSpec has a path field, but the operator does not handle the translation.

Will be fixed as part of upcoming v0.47.1 release

Issue must be fixed at https://github.com/VictoriaMetrics/operator/releases/tag/v0.47.1 release


Feature Request: Support for cursor-based pagination in collections

Currently, only offset-based pagination is what we can use when we want to retrieve data in separated small chunk to not overwhelm end users with a lot of data displayed on the uis. It would be great to also have cursor-based pagination and use created field as the default cursor.
I've never touched go code... But I might be able to help with this if I have some guidance from you the original maintainers

I think this is already supported but it depends on what field you want to use as cursor (we don't have an autoincrement field at the moment).
For example if you want to use the built-in created/updated date fields (dates have ms granularity), this could look something like:
let lastCreated = "";

async function load() {
    const result = await pb.collection("example").getList(1, 30, {
        sort:   "created",
        filter: `created > "${lastCreated}"`,
    });

    if (result.items.length) {
        lastCreated = result.items[result.items.length - 1].created;
    }

    return result;
}

load();

Pretty straightforward. Thanks @ganigeorgiev ❤


feat(name-translator): allow custom naming fn for promisifyAll

Allows users fine-grained control over the name of a method that
is being promisified with promisifyAll.
closes #1261

totally open to bikeshedding on naming here

@benjamingr @petkaantonov any comments on this?

What about cases where the translator causes name conflict?


⚠ Update contribution guidance so issues/PRs are filed against lit/lit

Context
This repository gets overwritten by changes from the Lit mono-repo. Communication needs to be updated to prevent the confusion of issues or pull requests being made against this repo.
Why?
Because we merge the lit-element-starter-js from the Lit monorepo. Thus any changes made only here will get overwritten.
Impact
Reduce churn and communication burden, and also make the process for improving starter templates clear. This may result in more changes to the starter kits and improve their quality.
Completion Criteria

Clear README guidance to file issues & PRs against https://github.com/lit/lit/tree/main/packages/lit-starter-js
Address Issue creation user flow. Creating an issue should communicate steps to make an issue on Lit/lit.
Address Pull Request user flow. Creating a pull request should either not be possible, or should communicate making the change against Lit.

With https://github.com/lit/lit-element-starter-js/commit/2dadf6cae7ff2967bcc47fd78c2a014b3314aa29 merged, Github issue page now links to the Lit monorepo.


Module name should be consistently used in examples

Module name in the readFile example code should be identical to the one used with the writeFile example and a prior example where XLSX is defined as the object the require('xlsx') returns

Good catch! will merge then amend the commit message


python310Packages.eve: 2.0.1 -> 2.0.2

Description of changes

https://github.com/pyeve/eve/releases/tag/v2.0.2
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python310Packages.eve
python39Packages.eve


⚠ Processors has degraded performance

In c7b31be, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 48 ms

Resolved: Processors performance has improved in a8b8b11.


[Bug][DISTRIBUTION][2.0.0]: Plugin build fails for JDK lower than 17 (11 and 14)

Describe the bug
Build for kNN plugin PRs fail when JDK 11 or JDK 14 has been used.
Example of failed build - https://github.com/opensearch-project/k-NN/runs/5668411237?check_suite_focus=true.
Error in the log:
Reaper process died unexpectedly! Check the log at /home/runner/work/k-NN/k-NN/.gradle/reaper/build-1813/reaper.log
reaper.log:
Error: LinkageError occurred while loading main class org.opensearch.gradle.reaper.Reaper
java.lang.UnsupportedClassVersionError: org/opensearch/gradle/reaper/Reaper has been compiled by a more recent version of the Java Runtime (class file version 61.0), this version of the Java Runtime only recognizes class file versions up to 58.0
More details exception is here - https://gist.github.com/martin-gaievski/ea1d3918e940c348f5e13dc5f9cc27a6.
The error comes at the very end of the build, when all tasks are finished successfully.
Locally I'm able to build is successfully for JDK 17, same as a github action has been cancelled as soon as build for JDK 14 fails.
This PR that uses OpenSearch 2.0.0-alpha1-SNAPSHOT, with added support for the build.version_qualifier.
To reproduce
Feel free to restart https://github.com/opensearch-project/k-NN/runs/5668411237?check_suite_focus=true failed task for JDK 11 or JDK 14, or download https://github.com/martin-gaievski/k-NN/tree/integrate/opensearch20-lucene91 and try to build is locally with JDK 14.
Expected behavior
Build should finish successfully, same as for JDK 17.

moved to opensearch repo, so closing this issue here for now.

Looks like this PR was causing the failures on plugin https://github.com/opensearch-project/OpenSearch/pull/2407. When I revert this PR, build was successful. I will further dig and see which change in PR caused this and update more info here

This https://github.com/opensearch-project/OpenSearch/blob/cc0e66b1dcc2cfe17b76bcea1168ebb996fbb090/buildSrc/src/main/java/org/opensearch/gradle/testclusters/OpenSearchNode.java#L913 line in PR where method call registerPid() happens, is having problem. This call tries to register and then later kill Pid, this is failing somehow and plugins are getting failed here. I will try to sync up and get suggestions from @dblock in how to move forward to a fix for this.

The reaper code was taken out when we re-introduced compatibility with JDK8, then put back in #2407. If the reaper is broken, it should just be fixed.

@dblock looking, the source/runtime level should be 11

Thank you @reta I tried your fix in local. It fixed this issue.

@reta You are a rockstar :)


IPC error: StringError(Utf8("invalid utf-8: corrupt contents"))

It was been pinpointed that this issue is occurring during the app_registered operation and that it's not occurring in safe_client_libs and may likely be a garbage collection issue in this library.
To replicate, build and execute WebId-MasterMerge branch of Peruse. Login and Accept authorization request for browser application. Logout and login until you see the error occur that carries the same message as the title of this issue.
Exposing manual garbage collection with --js-flags="--expose-gc" in Peruse and just --expose-gc in this library. Attempting to be able to replicate error without having to use Peruse.
https://maidsafe.atlassian.net/browse/MAID-2959

This bug may be consistently reproduced with this branch. Run it with yarn test-gc or yarn test --expose-gc.

Seeking further help: https://stackoverflow.com/questions/51793236/corrupt-contents-in-nodejs-ref-struct-upon-garbage-collection

Although a PR has been opened to resolve workaround this bug, this issue will remain open as the root problem lies in the ref/ref-struct/ref-array dependencies.