Log contains "Row limit hit", but none of these tables are that big

FYI, I am seeing this in my log:
Row limit hit - Attempting to cancel query...
Query cancelled successfully

I read somewhere that the default row count limit is 50,000.  Even if it were only 1000, the fact is that none of the tables I have queried today contain more than a few dozen rows.
I think the row limit is being hit by some internal query that SqlPad is doing automatically.  This might indicate a bug somewhere...
I'm using 1.12.0 installed via NPM.  My DB is MS SQL Server.  If you need other information or would like to recommend some settings I can change to get more detailed logs, just let me know.

The only thing I can think of is the schema info query. How many schemas / tables do you have? If you run this query outside sqlpad, how many rows does it return?

The number of resultant rows varies depending on which context I am in.  I mean, which database I am connected to.
When I invoke use MASTER before invoking that SQL, I only back 44 rows.
When I use some_work_related_database, I get over 22,000 rows.  (That DB is a beast!)
I hope this helps.

@dcelasun I was thinking it was the schema info query too, but that query overrides the max row setting.
@daveloyall are you executing multiple queries at a time returning multiple result sets? I think the row count would count all rows returned in the batch, which could maybe explain what we are seeing...

@rickbergfalk nope, I'm certain none of the queries I ran manually would return more than a few hundred rows, even if I ran them all at the same time.
What about enabling debugging logs?  Sorry, this is the first node app I've ever used, so I don't know how to do that.  I'm a Java developer, so I'm familiar with Log4j for example.

@daveloyall no worries - right now SqlPad isn't logging much. If you wanted to see what's happening out of curiosity, you can clone the repo and run server.js directly, and add some console.log() statements various places.
Otherwise I think it might be good to add a --debug flag to the application that logs a variety of activity.
I'll open a separate issue for that work.

@daveloyall SqlPad 1.13.0 includes a --debug flag that will log all queries run within SqlPad. It isn't anything fancy at this point, but should help figure out which query is causing this message.

Thanks, @rickbergfalk!

@daveloyall I'm going through and cleaning up some old issues. going to close this for now - let me know if you're still running into this issue


Df isin

Hello @Rubtsowa! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file sdc/datatypes/hpat_pandas_dataframe_functions.py:

Line 669:1: W391 blank line at end of file

.idea/workspace.xml and other unnecessary files should be removed.

Do you exactly need to keep .pyc files in the repo?

As far as I remember, the implementation wasn't handling case with index correctly. Assuming it is fixed, why there are still no tests for it?


Library is not working on an Android 12 device

I'm using my phone to develop React Native apps, but this library seems to not support it.
Phone: POCO X3 Pro
Android verison: 12 SKQ1.211006.001 with xiaomi.eu MIUI 13.0.4
Console traceback:
Material You is not supported on this device.
at node_modules/@assembless/react-native-material-you/lib/index.js:117:18 in getPalette

@comav Did you find a solution?

This module appears to be working: https://github.com/alabsi91/react-native-material-you-colors


backend receiving wrong http scheme

Request is receiving http instead of https. I think it's due to some settings in the traefik dockerfile but I can't figure which setting is causing this. Here's part of the docker-compose file.
      labels:
        # Enable Traefik for this service, to make it available in the public network
        - traefik.enable=true
        # Use the traefik-public network (declared below)
        - traefik.docker.network=${TRAEFIK_PUBLIC_NETWORK?Variable not set}
        # Use the custom label "traefik.constraint-label=traefik-public"
        # This public Traefik will only use services with this label
        - traefik.constraint-label=${TRAEFIK_PUBLIC_TAG?Variable not set}
        # traefik-http set up only to use the middleware to redirect to https
        - traefik.http.middlewares.${STACK_NAME?Variable not set}-https-redirect.redirectscheme.scheme=https
        - traefik.http.middlewares.${STACK_NAME?Variable not set}-https-redirect.redirectscheme.permanent=true
        # Handle host with and without "www" to redirect to only one of them
        # Uses environment variable DOMAIN
        # To disable www redirection remove the Host() you want to discard, here and
        # below for HTTPS
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-http.rule=Host(`${DOMAIN?Variable not set}`) || Host(`www.${DOMAIN?Variable not set}`)
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-http.entrypoints=http
        # traefik-https the actual router using HTTPS
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-https.rule=Host(`${DOMAIN?Variable not set}`) || Host(`www.${DOMAIN?Variable not set}`)
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-https.entrypoints=https
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-https.tls=true
        # Use the "le" (Let's Encrypt) resolver created below
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-https.tls.certresolver=le

Did you ever figure this out? I think im having a similar issue. I'm receiving the following response when I make an API call to the backend, even though I direct it to https:
Mixed Content: The page at 'https://XXX' was loaded over HTTPS, but requested an insecure XMLHttpRequest endpoint 'http://XXXX'. This request has been blocked; the content must be served over HTTPS.

For some reason it switches the call to http
I also noticed that navigating to http doesn't auto redirect to https

unfortunately, no. I exactly have the same problem with you. I initially fixed this by replacing http:// with https:// in the client (vuejs). but since I only needed that for my images that I have since moved to firebase storage, I don't have a use for it.
So the issue remains. The solutions I made are workarounds.

I experienced the same problem and after some trying finally found a "solution".
The problem didn't occur on every route, only specific API endpoints seemed to be broken. So after some investigation, I found out that only requests that used query parameters seemed to fail. However, the api/v1/items/?skip=0&limit=10 endpoint still worked fine. Looking at the code in the ItemService class that issues the request, the URL /api/v1/items/ ends with a /, which sadly seems to be the only reason why my custom requests failed. After adding that missing trailing / char, it all worked fine. I know this isn't really a solution, but at least it's an easy fix.
public static readItems(data: TDataReadItems = {}): CancelablePromise<ItemsPublic> {
    const {
        limit = 100,
        skip = 0,
    } = data;
    return __request(OpenAPI, {
        method: 'GET',
        url: '/api/v1/items/',
        query: {
            skip, limit
        },
        errors: {
            422: `Validation Error`,
        },
    });
}


[Refactor]: Remove request wrappers in xline-client

We implement some request wrappers in the xline-client crate, like
/// Xline/crates/xline-client/src/types/lock.rs
/// Request for `Lock`
#[derive(Debug, PartialEq)]
pub struct LockRequest {
    /// The inner request
    pub(crate) inner: xlineapi::LockRequest,
    /// The ttl of the lease that attached to the lock
    pub(crate) ttl: i64,
}

impl LockRequest {
    /// Creates a new `LockRequest`
    #[inline]
    #[must_use]
    pub fn new(name: impl Into<Vec<u8>>) -> Self {
        ...
    }

    /// Set lease.
    #[inline]
    #[must_use]
    pub const fn with_lease(mut self, lease: i64) -> Self {
        self.inner.lease = lease;
        self
    }

    /// Set session TTL.
    /// Will be ignored when lease id is set
    #[inline]
    #[must_use]
    pub const fn with_ttl(mut self, ttl: i64) -> Self {
        self.ttl = ttl;
        self
    }
}

Therefore, when we want to perform a lock operation by xline-client, we should write some code like this:
#[tokio::main]
async fn main() -> Result<()> {
     ...
     let lock_request = LockRequest::new("lock-test").with_ttl(50);
     // acquire a lock session
     let session = lock_client
         .lock(lock_request)
         .await?;
     ....
     Ok(())
}

This way is not very user-friendly. I prefer to use lock_client.lock("lock-test", 50) rather than lock_client.lock(LockRequest::new("lock-test").with_ttl(50)).

/assignme

/assignme


[build] Fix targets fail to build with import_prefix on C++

Changelist:

Moving the protoc output directory

Related to https://github.com/grpc/grpc/issues/33377

Looks like the tests are failing. Are you able to access the logs?


Updated the links to the Quick Start Templates

Updated links to quick start templates to github

@dhanrajks : Thanks for your contribution to the Azure documentation! The author, @tfitzmac, has been notified to review your proposed change.

@dhanrajks - thanks for this change.

#sign-off

@tfitzmac Hi Tom Please resolve the merge conflicts to enable the merge.

@dhanrajks  and @jomolnar - these same changes were made in another PR #347 . Let's close this PR since the other has already been merged.


Post-fork sync

After the recent Ropsten fork I can not synchronize to the normal Ropsten network.
I am using parity 1.10.6
I tried warp sync and --no-warp sync. Neither helped. I end up on the chain with best block  ~3,069,000, while the actual latest block now is ~3410246.
Are there any stable Ropsten enodes to connect to in order to sync to normal Ropsten? How would one avoid such forked networks generally?

I second this. I've asked in Gitter as well as here, the Ropsten Revived enodes listed in the readme are not currently working, I sat at 0 peers for a full day.

After a couple of delete-resync iterations I finally synced to correct node


Update housing example tutorial

Description
Update housing_example to use get_function to transform test data.
Type of change

[ X ] This change is a documentation update

How Has This Been Tested?
Run the notebook.

The update makes sense as the data preprocessing no longer involves components that have to be reused (e.g., vectorizer trained on a specific version of data). Once we resolve the issue with wonky code chopping in pipeline modularization, we can get the vectorizer back to the example to restore its realism. But, it is good for now.

Can you add url parametrization to to_pipeline as well? Context for the request here.

Can you add url parametrization to to_pipeline as well? Context for the request here.

Added


Show lowest dual gap in last iteration in PFW.

The same behaviour as in BPCG.

@matbesancon @pokutta In the callback, we hand in  dual_gap. But the actual dual gap should be stored in phi_value if I am not mistaken. Is this behaviour intended?


Have to enable debugging for scripts every use

Right Click and select inspect

Navigate to the Scripts Tab in web inspector

Note that you need to enable debugging

Select Always enable and click "enable debugging"

Restart Ripple and repeat the above steps and note that you still need to enable debugging (UNEXPECTED)

I would expect this value to persist
[2103465]

Do you know offhand where Web Inspector stores it's settings? Is it a cookie
or a local database thing?
On Wed, Oct 12, 2011 at 12:20 PM, Gord Tanner <
<EMAIL_ADDRESS>

Right Click and select inspect

Navigate to the Scripts Tab in web inspector

Note that you need to enable debugging

Select Always enable and click "enable debugging"

Restart Ripple and repeat the above steps and note that you still need to
enable debugging (UNEXPECTED)

I would expect this value to persist
[2103465]
--
Reply to this email directly or view it on GitHub:
https://github.com/blackberry/Ripple-Framework/issues/64


[v4] reorder-group does not work properly when used in custom stencil components

Bug Report
I am implementing a set of custom stencil components which are partly made up of Ionic V4 components. However, there are issues when using the ion-reorder-group component.
Ionic Info
@ionic/core:  4.0.0-beta.15
@stencil/core: 0.15.1
Describe the Bug
When the ion-reorder-group element is not within the initial viewport, the reorder animations are not displayed properly and the currently dragged item is not inserted at the right position within the reorder-group.
Steps to Reproduce

Init Stencil project ( component - starter )
Install ionic/core module
Replace initial my-component implementation with the example provided below ( Make sure the height of the placeholder div is sufficient to push the ion-reorder-group element out of the viewport )
run project and try to reorder list items

import { Component} from '@stencil/core';
import "@ionic/core"

@Component({
  tag: 'my-component',
  styleUrl: 'my-component.css'
})
export class MyComponent {

  reorder(event){
    event.detail.complete();
  }

  render() {
    return [
      <div style={{width: '100%', height: '1000px'}}>
        Just for demonstration purposes
      </div>,
      <ion-list>
        <ion-reorder-group disabled={false} onIonItemReorder={ev => this.reorder(ev)} >
          <ion-item>
            <ion-label>Test 1</ion-label>
            <ion-reorder></ion-reorder>
          </ion-item>
          <ion-item>
            <ion-label>Test 2</ion-label>
            <ion-reorder></ion-reorder>
          </ion-item>
          <ion-item>
            <ion-label>Test 3</ion-label>
            <ion-reorder></ion-reorder>
          </ion-item>
          <ion-item>
            <ion-label>Test 4</ion-label>
            <ion-reorder></ion-reorder>
          </ion-item>
        </ion-reorder-group>
      </ion-list>
    ];
  }
}

Expected Behavior
When dragging an item within an ion-reorder-group, the animations should be displayed properly. When dropping an item, it should be inserted at the correct position within the list.

Thanks for the issue! Sorry I know this is a late reply but I am working on updating the documentation for the reorder-group and found this issue. I think the issue here is true has to be passed to complete():
event.detail.complete(true);

Otherwise it assumes that you don't want to finish the reorder and bounces the item back. I was able to get this working in the stencil conference app by passing true.  I am running version "@ionic/core": "4.3.0". If you're still having issues with this version please let me know!
I personally think the complete() should work the same with or without true being passed, but I'd have to look into if this would be a breaking change or not.

I'm going to reopen this because our documentation states it should work with just complete() and no variable passed so I believe this is a bug.


Bug fix window funnel function coredump when upgrade

Proposed changes
as the branch-2.1 is used the new impl of window funnel, and the be_exec_version is 5
but in branch-3.0 this be_exec_version have update to 7, so when upgrade from branch-2.1 to branch-3.0
maybe have error send the branch-3.0 version of version 7 to branch-2.1([0---version--5])

run buildall


How to select average users for image reconstruction

Hi, author：
I see that your article not only performs user (1, 2, 5, 7), but also performs 'average_user' image reconstruction. But there is no 'average_user' processing found in your code and it is also not found 'average_user' in the NSD dataset . How should this 'average_user' processing be set?
Looking forward to your answer. Thank you very much.
Your fans

Hi @MRLQ-Q , 'average_user' means latent values of (1,2,5,7) are averaged and then fed to the networks


Suggest adopting a common style guide for documentation

Motivation
The Druid docs have evolved with only a basic attention to documentation style. Some effort has been made to provide stylistic guidance in the documentation contribution guide:
https://druid.apache.org/docs/latest/development/contribute-to-docs/
As the project matures, it makes sense to adopt a common style so that contributors have a common reference when composing and reviewing documentation.
Proposed changes
Rather than create a new style, I recommend we follow the Google Developer Documentation Style Guide: https://developers.google.com/style. Open to other style guides if someone has a strong opinion.
Where the Google Developer Documentation Style Guide is not consistent with the goals of the Apache Druid documentation, we document the exception for clarity. One such example would be to avoid code font for SQL syntax which can make the documentation more difficult to read.
Rationale
It is easier to adopt an existing style that aligns with the principal goals of the documentation (developer docs) rather than to attempt to create an exhaustive unique style
Operational impact
Improved reference when creating and reviewing documentation.
Future work (optional)
There will need to be ongoing maintenance of the style guide to keep it relevant.

The Google doc style guide seems pretty reasonable to me. And, it's helpful to have something to point to. So I am in favor of this.
Would you suggest removing the https://druid.apache.org/docs/latest/development/contribute-to-docs/#style-guide section and replacing it with a link to https://developers.google.com/style + any exceptions we have decided to adopt?

@techdocsmith point of clarity – the Google SG refs to the MS SG, and thereby cites https://learn.microsoft.com/en-gb/style-guide/capitalization which differs from https://developers.google.com/style/capitalization. In adopting the Google SG, "How to eat data with Apache Druid" would be correct in a title or heading, and "How to Eat Data with Apache Druid" would be incorrect. Correct?

@gianm , Yes, we would link to google style and call out notable exceptions. It might be nice to leave in a few common helpers styles there that contributors will run into frequently. but removing the suggestions altogether could also work if that seems like a better plan.
@petermarshallio : the docs already follow sentence case in most cases as specified in https://developers.google.com/style/capitalization . Note that Google says "If the preceding references don't provide explicit guidance", which in this case they do, "then see these third-party references, depending on the nature of your question."

This is done: https://druid.apache.org/docs/latest/development/contribute-to-docs#style-guide


persist /etc/postfix

persist /etc/postfix

related to https://github.com/uyuni-project/uyuni/pull/7411


Latest release v.2.0.14 has invalid format

Hey,
thanks for keeping that fork alive! I just noticed that the latest release name is invalid v.2.0.14. I think this should be v2.0.14.
Thanks!

Yeah it should be. I will make an another release after you PR is merged.


Rewrite using sublime-syntax format

The current highlight is adapted from the ST2 HTML syntax, originally written in the tmlanguage format and compiled from YAML.
ST3 now supports its own sublime-syntax format and ideally this package should be rewritten based on the new HTML syntax in ST3.
This is an open call if anyone wants to take a stab at it - I have included a samples directory to verify basic highlighting behavior. Documentation for ST3 syntax definitions are here.

I would very much like to contribute to this. Will work on it right now and report back if I could make progress.

Can we also add support for  blocks highlighted as JavaScript by default and  blocks highlighted as markdown by default?

The current core HTML syntax definition uses the new embed/escape idiom implemented in dev build 3153. It would probably be best to use the same method here. It's not in the syntax definition docs yet, but partial documentation is available here.
Also, I don't know much about Vue in general or this syntax in particular, but if this syntax definition is basically HTML extended with additional features, then it might be possible to use my YAML Macros package to directly extend the core HTML syntax in a forward-compatible manner. That way, the Vue syntax could avoid reimplementing HTML itself. An example (implementing JSX by extending the core JavaScript package) can be seen here.

@Thom1729 I've made a little bit of progress. I discovered embed/escape while looking at the HTML language but found that it doesn't support with_prototype or rather threw an error while using it directly.
In fact, I tried various methods which worked with less success and am taking this direction right now which seems to be working
%YAML 1.2
---
name: Vue Component
file_extensions: [vue]
scope: source.vue

contexts:
  main:
    - match: '<template>'
      scope: text.html.basic
      push:
        - meta_scope: text.html.basic
        - match: '</template>'
          pop: true
        - include: scope:text.html.basic
        - match: '{{'
          push:
          - meta_scope: source.js
          - match: '}}'
            pop: true
          - include: scope:source.js
    - match: '<style>'
      push:
        - meta_scope: source.css
        - match: '</style>'
          pop: true
        - include: scope:source.css

    - match: '<script>'
      push:
        - meta_scope: source.js
        - match: '</script>'
          pop: true
        - include: scope:source.js

I noticed that we can include: scope:source.js and I found this in the RestructuredText language file. If you're familiar with this, would appreciate some feedback on whether this direction is ok.

@Thom1729 I probably would clarify that vue template syntax isn't 100% html or JSX.
For example:
  <div>
    {{ foo * 10 + 'hi' }}
    <span
      v-text="foo * 10 + 'hi'"
      :id="foo + 'baz'"
      @click="onClick('hello')">
      Hello
    </span>
  </div>

We need to have some parts as JavaScript like {{ foo * 10 + 'hi' }} and other expressions.

The issue with include is that you're relying on the included syntax to pop itself off. In the case of JavaScript, this won't work; the Vue syntax has to force it to pop. This is what caused #107.
The old way of dealing with this is with_prototype. It's the most powerful, but also the most complicated, and Sublime basically compiles an entire copy of the included syntax into yours. (It will always do this if you include.) For Vue, this is a significant downside, because it might want to embed a lot of other syntaxes.
The new way is embed/escape. I think that it should work here. In the case of JavaScript, if you set escape: (?=</script), then you shouldn't need with_prototype. The escape will force the JavaScript contexts to pop when </script is encountered, no matter how deeply nested those contexts are.

@Thom1729 I actually had some other issues with with_prototype and am not using it right now. Using a technique I discovered in this syntax file.
I don't think we need toextend and add to the JS/CSS/etc, and will definitely switch them to embed. But is there a way to add new matchers/extensions with embed approach?

I'm not familiar with reStructuredText, but here's an example of how that could go wrong:
.. raw:: html <style>

span { color: red; }

The remainder of the file will be highlighted as CSS because the <style> tag was never closed. This may be correct for reStructuredText, but it isn't for HTML. Consider the following:
<script>0</script>/

The HTML syntax wants the closing </script> tag to end the JavaScript.  And when the parser hits </script>, there will be several JavaScript contexts on the stack above the HTML context that is looking for </script. As a result, bug #107.

But is there a way to add new matchers/extensions with embed approach?

I'm not sure what you mean. Just like the HTML syntax supports embedding several other syntaxes, you can write rules to embed as many syntaxes as you like under basically whatever conditions you like. As a bonus, it doesn't bloat the syntax, and you don't have to mess around with with_prototype. The downside is that the escape rule is "dumb" in the sense that it can't handle escaping or recursion.

@Thom1729 I've opened a PR #109 please provide comments on whether the approach is ok.

Marking this issue closed 😄  Closed in #109


[release/1.23] repo: Release v1.23.12

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/docs

pushing through to expedite release


chore(cloudtrail): update Go and dependencies

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area plugins

/area registry

/area build

/area documentation

What this PR does / why we need it:
The main goal was to update the github.com/aws/aws-sdk-go-v2/config package so that the AWS_ENDPOINT_URL env var works.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
The initial discussion started in Kubernetes Falco Slack channel: https://kubernetes.slack.com/archives/CMWH3EH32/p1714641429503799

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

b78b80e chore(cloudtrail): update Go and dependencies
9082496 chore(cloudtrail): replace moved package

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Welcome @nunoadrego! It looks like this is your first PR to falcosecurity/plugins 🎉

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: nunoadrego
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Why didn't you bump up to Go 1.22? Can you also update the plugin version, and the changelog please. Moreover, it also requires to update the versions of Go in the CI, but it could create issues with other plugins.
cc @leogr @jasondellaluce

I think 1.21 is fair enough. The plugin is also a shared Go package, and increasing the required Go version inherently mandates a version bump to adopters importing the plugin's code (other plugins included). This is the rationale by which we always updated the Go version when needed, but to the bare minimum version required. As you see there's also the problem of forcing other plugins to update as well. For now, I'd suggest sticking to 1.21 as proposed in the PR.

Suggestions implemented, @Issif.
I generated the changelog by running ./build/changelog/bin/changelog --plugin cloudtrail --from 16306f2 and then checking the commit history and release to see to which they belong. Is there a better way?

LGTM label has been added.  Git tree hash: 1145641bce116210825353e4d0c77b6d21cac6f0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, nunoadrego
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


fix(azure-iot-device): several race condition bugs discovered via stress tests

All of these bugs were discovered with stress tests.

fake_payload = "__fake_payload__"

does these need to be per class basis ?

Refers to: azure-iot-device/tests/common/pipeline/test_pipeline_stages_base.py:263 in 08c03e2. [](commit_id = 08c03e209836f7a44e8638a352237a13bb75330b, deletion_comment = False)


[BUG] Sametime subject cleanup does not work

HCL Product Version
12.0
Describe the bug
If I have a meeting subject of "@Home" (or just "@") and I try to book a Sametime Meeting, the request will fail with an HTTP 403 error.
The problem is due to the subject that is sent on the request.  It still contains the @ character which is not allowed by the ST server / APIs.
Looking at the enhanced logging output I see that the @ was not removed from the subject that was sent:
Body JSON
{
"meeting":{
"name":"@Home_BKAN-C4VRZQ",
...
To Reproduce
Steps to reproduce the behavior:

Create a new meeting and give it a subject of just @ (or @Home )
Click on Schedule Online Meeting / Sametime
See error

Expected behavior
I would expect unsupported characters to be filtered out by the cleanupSTMeetingName() routine.  Unsupported characters like @ are supposed to be replaced by that routine but they are not.
Screenshots
N/A
Desktop (please complete the following information):

OS: Win 10 Enterprise

Additional context
The failure happens because the logic in cleanupSTMeetingName() is incorrect.  It only makes the last string change in badList() due to this incorrect For loop:
ForAll b In badList
	cleansubj = Replace(subj, ListTag(b), b)
End ForAll

All replacements except for the last one are lost because the code reuses the starting input string as the string to modify instead of the modified copy.  The fix for this is to change this block of code to be:
cleansub = subj

ForAll b In badList
	cleansubj = Replace( cleansubj, ListTag(b), b)
End ForAll

That way all string replacements will be performed instead of just blanks to underscores.

Resolved by pull request #88


hiding db secrets

Old build with secrets printed
New build with secrets hidden

Was the change in https://github.com/myhelix/golang-orb/pull/17 / 0.6.3 not sufficient?

Was the change in #17 / 0.6.3 not sufficient?


Move update-node-template to Polkadot's pipeline

According to https://github.com/paritytech/release-engineering/issues/142

The desired behaviour is that this job should be triggered only once when the final release tag is created.

The simplest solution I found for that requirement entails moving the job to Polkadot's pipeline and triggering it on the tags of that repository (https://github.com/paritytech/polkadot/pull/6522).
Alternatively we could keep the job here and trigger a remote pipeline from Polkadot to this repository (example), but the extra complexity of that solution seems unwarranted for this script.
close https://github.com/paritytech/release-engineering/issues/142

bot merge

0.9.37 did not trigger a PR downstream, this is why correct?
Unless the overall release strategy is changing, we want to keep this happening for sure @joao-paulo-parity .
cc @sacha-l @lisa-parity

0.9.37 did not trigger a PR downstream

The job to produce the PR was triggered (https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2282467), but it appears there's a problem with the script. I'll submit a fix shortly.


[SE-0304] Align Task API implementation with the fourth revision of the proposal

Specific changes are:

Renaming the task group async operations to addTask
Rename TaskPriority.default to TaskPriority.medium
Rename Task.yield() to Task.suspend()
Add task group waitForAll()
Implement cancellable Task.sleep(nanoseconds:)

All changes with the exception of Task.sleep(nanoseconds:) are implemented in an ABI-neutral manner.
Task.sleep(nanoseconds:) is ABI-additive; it's too much code to want to inline.
Implements rdar://80339563

@swift-ci please test

@swift-ci please nominate


[Feature Request] Allow nodeSelector reflection for spawned jobs/pods

When a transcode job/pod is created, it currently doesn't appear to reflect any custom nodeSelectors into the created resource. This would be handy if, for instance, you have some nodes in your k8s cluster which are better suited to handle the transcoding. In theory you should be able to pull the pods nodeSelectors in a similar way to how you pull the kube-plex annotations during resource creation.

Marking this as fixed too.


Suppress windows build warnings to reduce win build log size.

Can't add as reviewers, but also happy to hear comments from @guschmue @vit-stepanovs
The suppressed warnings are:

4003 - not enough actual parameters for macro 'identifier'
4244 - 'conversion_type': conversion from 'type1' to 'type2', possible loss of data
4267 - 'variable': conversion from 'size_t' to 'type', possible loss of data
4503 - 'identifier': decorated name length exceeded, name was truncated
4506 - no definition for inline function 'function'
4800 - 'type': forcing value to bool 'true' or 'false' (performance warning)
4996 - 'function' or 'variable' marked as deprecated

The numbers of times we encounter these:
   1374 C4003:
   1106 C4244:
    850 C4267:
   1104 C4503:
    156 C4506:
    102 C4800:
    116 C4996:

I also have the following linker warnings:
    444 LNK4049:
      2 LNK4070:
    246 LNK4197:
   1332 LNK4217:
    126 LNK4221:

do I need to change somewhere else for linker suppressions, or should I just add /ignore:<id> same way I add compiler warning suppressions.

Change looks good to me; I don't know the answer about linker warnings.

The linker errors: should do no harm to /ignore: them.
Most of them seem to be coming from the external png lib. I can take a look why we see them.
If you are looking at disk usage for the builds in general: I noticed that some unit tests are collecting a good amount of disk space in \windows\temp\


Multiple menu Imagery item selection UI bug

Description
Switching between background imagery still shows the purple selection on the previous background on the list
Screenshots
This is clicking on another background in the list

This is using the keybindings to switch between the previous and next background in the list

Version
2.2
What browser are you seeing the problem on? What version are you running?
Chrome v119
The OS you're using
MacOS 14.1.1
Steps to reproduce

, switches to previous background in list
. switches to next background in list
or click on another imagery option

The browser URL at the time you encountered the bug
https://rapideditor.org/rapid-test#background=Bing&datasets=fbRoads,msBuildings&disable_features=service_roads,paths,buildings,building_parts,indoor,landuse,boundaries,water,rail,pistes,aerialways,power,past_future,others&map=20.00/39.06909/-108.56054
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
No response

Yes, I think it's been like this before - to show which 2 imageries get toggled between with the ⌘B shortcut ?

Ahh I see what you are saying. I will close this issue because that is actually helpful


WebStorm TF source control does not detected changes

I successful installed and configured VSTS plugin with TF source control and pulled the repo, when I change a file, it turn blue like the documentation says it should, and if I right click on the file and go to:
TFS > Compare With The Same Repository Version
It correctly shows all the differences. However, when I try to check in the file, I will always get a alert box saying "No changes detected", despite the fact that it JUST correctly showed me all my changes when I compared it to the repo version.

@incode4it (and anybody else who has the same issue), could you please try again with the recent version and upload the IDE logs?


getMore command doesn't use batchSize and maxTimeMS options

Hi!
If we try to set the options batch_size and max_time when we execute the find or aggregation functions, the first query to Mongo use these options and we get the first batch correctly, but when the driver execute the get_more command, it cannot use these options again,  because these were deleted from options keyword in previous find/aggregate function, so get_more use default config, usually the rest of batch with 16MB size limit and no time out.
I'm using last version (0.7.4)
Regards!

Yes, you are right: The options are getting dropped:
    drop = ~w(limit hint single_batch read_concern max min collation return_key show_record_id tailable no_cursor_timeout await_data batch_size projection comment max_time skip sort)a
I will fix it.


Feature Request: com.atproto.server.createAccount

Request for com.atproto.server.createAccount
1. Problem to solve
I need to create an account on my own pds server.
But I could not find the function.
2. Proposal
So, could you make a function to create acount ?
com.atproto.server.createAccount.
3. More information

Hi @tacsotai !
Yes it's needed, I will implement soon!

Thank you. I am looking forward to be implemented.

Your service seems to be only for
https://bsky.social or https://foo.bar
I want to test it as
http://localhost:2583 or http://192.168.xxx.yyy
Could you support not only HTTPS but also HTTP ?
https://github.com/myConsciousness/atproto.dart/blob/main/packages/xrpc/lib/src/xrpc.dart#L150

Hi @tacsotai ,
It's possible, I imaged an Enum like Protocol that would allow the user to select HTTP or HTTPS. Also, the default if this parameter is omitted would be HTTPS.
But since the Internet these days is based on HTTPS and you need to issue an SSL certificate and publish as HTTPS anyway, I don't know if we should provide an HTTP option. Is this option really necessary?

@tacsotai ,
You can set HTTPS for localhost, like:
https://www.section.io/engineering-education/how-to-get-ssl-https-for-localhost/

Thank you for a information.
In my experience, it is good not using https while testing.
After test, go to staging env, https is good I was taught by senior.
Now I am investigating pds server.
https://zenn.dev/tac519/articles/727fca3783010c
And in the future, I gonna operate a small world with https://foo.bar/hoge
So, I want use the lib not only root domain and sub folder.

@tacsotai
I see, I'm going to add this option to atproto and bluesky, anyway I want to make these packages developer friendly:)
And do you need "createAccount" right now? I already implemented it and I can release, but I'm going to implement other endpoints to if it's not so urgent :)

Thank you for your consideration.
No, I want use it later. I am happy it will be relased in one month.
Actually, I am creating bluesky GUI client. I have also to create a little to access method.
https://github.com/tacsotai/flutter_bluesky
I want to use your library in my client because your libraly is shown at the ecosystem.

@tacsotai
It takes less than 2 weeks! I will release it as early as this Saturday or Sunday ;)

Hi @tacsotai ,
I released new version of atproto and bluesky packages! I added createAccount and other endpoints, and added protocol parameter in ATProto and Bluesky object, you can set Protocol.http now :)

Thank you very much.
I will try it.

Now, I am trying below sample.
https://github.com/myConsciousness/atproto.dart/blob/main/packages/atproto/example/example.dart
    final session = await atp.createSession(
      service: 'SERVICE_NAME', //! The default is `bsky.social`
      identifier: 'YOUR_HANDLE_OR_EMAIL', //! Like `shinyakato.bsky.social`
      password: 'YOUR_PASSWORD',
    );

Can I set 'SERVICE_NAME' like '192.168.xxx.yyy:2583' ?
I got below error log on my M1 mac.
Unhandled exception:
HandshakeException: Handshake error in client (OS Error:
	WRONG_VERSION_NUMBER(tls_record.cc:242))
#0      _SecureFilterImpl._handshake (dart:io-patch/secure_socket_patch.dart:99:46)
#1      _SecureFilterImpl.handshake (dart:io-patch/secure_socket_patch.dart:142:25)
#2      _RawSecureSocket._secureHandshake (dart:io/secure_socket.dart:915:54)
#3      _RawSecureSocket._closeHandler (dart:io/secure_socket.dart:908:15)

Hi @tacsotai !
I use http package internally, so I guess it's possible to use local service name like that! Is this locally connectable server information?

Thank you for the reply.
I atatched protocol.
    final session = await atp.createSession(
      service: 'http://192.168.xxx.yyy:2583',
;

Then I got another error.
Unhandled exception:
FormatException: Invalid radix-10 number (at character 1)
//192.168.xxx.yyy:2583
^

#0      int._handleFormatError (dart:core-patch/integers_patch.dart:126:5)
#1      int._parseRadix (dart:core-patch/integers_patch.dart:152:16)
#2      int._parse (dart:core-patch/integers_patch.dart:98:12)
#3      int.parse (dart:core-patch/integers_patch.dart:60:12)
#4      _Uri._makeHttpUri (dart:core/uri.dart:1791:49)
#5      new _Uri.https (dart:core/uri.dart:1693:12)
#6      new Uri.https (dart:core/uri.dart:272:20)
#7      procedure
xrpc.dart:291
#8      AnonymousClient.post
;

@tacsotai ,
Please try with following:)
service: '192.168.xxx.yyy:2583'

Thanks. but in my env, same error occured.
The log tell me the problem on handshake. I will check it.
HandshakeException: Handshake error in client (OS Error:
	WRONG_VERSION_NUMBER(tls_record.cc:242))


solve_ivp: solution object to return additional info

It would be desirable to have some additional information about the solution process returned in the solution object:

Arguments args passed to fun
Name of the solver actually employed
Step size: average, min, max
Number of rejected trial step sizes

More extended data in additional arrays (not normally provided, but as an option to be requested through an additional flag):

Step size h for each step
Function value fun at each step

Hi there! I would like to try to work on this issue, but I'm willing to listen to maintainer's opinion first.

solve_ivp returns a results class that can be extended in a backwards-compatible way, so adding more returns is fine from that perspective.

Arguments args passed to fun

Hmm, that's kind of unusual. It's also easy to get separately from within fun itself.

Name of the solver actually employed

makes sense to me

Step size: average, min, max

Number of rejected trial step sizes

these could make sense, not sure since it requires changes in all solver claesses.

(not normally provided, but as an option to be requested through an additional flag):

Please let's not do optional returns, we have regretted doing such things often in the past.
Shouldn't a lot of this be accessible via events? In that case, adding examples on how to do that may be better.
@nmayorov, @drhagen any thoughts?

The rational behind my proposal is to have the ability ex post (i.e. after the solver has finished) to analyse, retrace and understand the solution process. "ex post" would not necessarily happen immediately after the solver has finished, but would also cover the case of an off-line analysis later, e.g. after a bunch of different problems had been solved and/or different solvers had been employed.
The approach would be to safe the solution object, and perform the analysis later by comparison of the different solution objects. Hence, the solution object should serve as the container for all information relevant to draw meaningful conclusions.
In view of this I'd like to add the following remarks:

Arguments args passed to fun

To pull the args from fun separately would be cumbersome due to the need to manipulate and inflate the code of fun. And fun might be provided externally and not amenable to code changes. Of course, it would be possible to save the args beforehand - provided one already anticipated the need to later recall them. Furthermore, to be a bit fussy, the args are part of the definition of the problem. Hence they should be replicated in the solution object, in order for the solution object to document the problem as well as its solution.

Shouldn't a lot of this be accessible via events?

This would be fine, and serve my purpose, provided that events could be as easily saved as an object besides the solution object.
Best regards and thanks for tending to my proposal,
zechenhund

Arguments args passed to fun
Name of the solver actually employed

Does SciPy typically echo the input arguments to the output? This isn't particularly harmful except that it may keep memory-heavy inputs alive longer. If this is common in SciPy, then I support it. Otherwise, I am weakly opposed. It should be noted that the OdeResult can be mutated, so the user can add anything he wishes after-the-fact:
args = ([5],)
method = 'Radau'
sol = solve_ivp(my_fun, [0, 10], [2, 4, 8], args=args, method=method)
sol.args=args
sol.method=method

Step size: average, min, max
Number of rejected trial step sizes
Step size h for each step
Function value fun at each step

I am weakly opposed to adding these, mainly because the first three only make sense for adaptive solvers and there are fixed-step-size solvers we may want to add to solve_ivp in the future. Remember, solve_ivp is a rather thin wrapper around the step solvers. If you want to pull out info from the solver that solve_ivp does not return, you can wrap a for loop around solver.step() and grab anything you want from the solver after each step.

Does SciPy typically echo the input arguments to the output?

No

This isn't particularly harmful except that it may keep memory-heavy inputs alive longer.

Agreed. Name of the solver is fine (and potentially useful), but keeping input args around can indeed increase memory use.

Sorry that I haven't comment for so long.
This is not an easy decision to make in my opinion. Because all suggestions are reasonable, but none of them are clearly necessary for a typical use case.
I think attaching args and method was nicely handled by @drhagen suggestion.
As for step statistics. By adding it as guaranteed output from solve_ivp we increase amount of work to include a new solver and add challenges to interpret its behavior - for example, how do we count rejection steps. Also in some cases it might be hard to extract such information without modifying complicated C/C++ or Fortran code.
Probably what you've suggested could be combined into another substructure result.stats along with nfev, njev, nlu. I think it might be a logical way to reorganize output. For stats we might potentially remove guarantees, i. e. different solvers might fill different fields within stats.
What do you guys think about combing characteristics of integration process into a separate field (stats) and allowing to extend it as @zechenhund have suggested?


API for controlling playback, authentication...

Is it possible to have an API for controlling playback, kicking users even?

How would I go about filtering who can and can't connect to librespot?

Yes it is possible, but we only currently implement a Spotify Connect receiver i.e. for playback. We don't implement a (remote) controller side. Related projects like spotcontrol do implement this.
Alternatively, users can control playback via Spotify's Player Web API. Librespot can provide an access token, or you can obtain one yourself using Spotify's API endpoints. I guess transferring your playback session to the device will kick off anyone already using it?
Librespot does provide a Disconnect method but you'd need to write additional code to expose it. You'd also need to write additional code to implement access control/user filtering, nothing like that exists. I don't think we have any extensions like that. If it did exist, it'd probably belong in ConnectConfig.

Thank you. I think a DBus interface would prove to be useful! I'm not really sure how I'd expose those functions

spotifyd, a related project, exposes an MPRIS interface.


Hiding unsupported products

Kinecta Federal Credit Union is not part of the default set of banks that are displayed on the bank selection screen, so we have to search for it:

After going through the regular flow we get back PlaidError : PRODUCTS_NOT_SUPPORTED when making a POST:/identity/get request after retrieving the access token from a POST /item/public_token/exchange  request . This is happening on the sandbox and production instances.
We would prefer to hide banks that do not support products that we make use, is there a configuration to achieve this behaviour?
Alternatively, is there a way to  provide a list of banks for the PlaidLink component to display? We would have to find a way to filter the list of banks based on the products they support.
We are are v1.5.0.
EDIT: More details.

From the docs it looks like the products attribute would work for this scenario. Closing.


SparkPost-API 1.1.0 Sending Transmission Fails

I have updated the SparkPost-API to version 1.1.0 and receive the following exception when sending a transmission:
An unhandled exception of type 'System.IO.FileNotFoundException' occurred in mscorlib.dll
Additional information: Could not load file or assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>,
Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed' or one of its dependencies. The located
assembly's manifest definition does not match the assembly reference. (Exception from HRESULT:
0x80131040)
I'm using Newtonsoft.Json version 8.0.3. Any help appreciated.

This is not related to SparkPost. I have gotten this error before. Make sure you have the Newtonsoft.Json entry in your Web.config under the website root. What I have is the following:
  <dependentAssembly>
    <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" culture="neutral" />
    <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
  </dependentAssembly>

This library also requires .Net 4.5, so I would check on that as well.

I'm going to close this for now... but please feel free to respond here and reopen if it is discovered to be an issue with this library.


fret: "undefined value" on reading *.otf

Maybe I missed something in the docs, but fret throws an error on *.otf files:
$ fret LOV.WieynckKursiv.otf
Can't call method "read" on an undefined value at /Users/helmut/perl5/perlbrew/perls/perl-5.20.1/lib/site_perl/5.20.1/Font/Fret.pm line 187.

Fret only works with TrueType outlines and a glyf table in the font.
GB,
Martin
On Wed, 18 Apr 2018, 13:37 Helmut Wollmersdorfer<EMAIL_ADDRESS>wrote:

Maybe I missed something in the docs, but fret throws an error on *.otf
files:
$ fret LOV.WieynckKursiv.otf
Can't call method "read" on an undefined value at /Users/helmut/perl5/perlbrew/perls/perl-5.20.1/lib/site_perl/5.20.1/Font/Fret.pm line 187.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/silnrsi/font-ttf-scripts/issues/19, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AFbHbZGUoRP1ACTIt4oS5uT7YG6Go_roks5tpt85gaJpZM4TZf3r
.

@mhosken Thanks for the reply.
After reading the specs of OpenType, TTF/AAT and Adobe, and also investigating a dozen OSS packages, I have a better understanding of the problem.
Comparing the most promising alternatives (Adobe Type Tools Font-API, Python fonttools, Font::TTF) to build on, I decided to start with Font::TTF and modify it into Font::OTF.
Roughly estimated 30-70% of Font::TTF is reusable to some degree to support .otf. It's well structured and readable.
Additional question: Is there a preferred or most popular XML-format for font data, which should be supported?

Dear Helmut,
I recommend fonttools with the ttx tool if you want to go the XML route.
GB,
Martin
On Thu, 19 Apr 2018, 15:47 Helmut Wollmersdorfer<EMAIL_ADDRESS>wrote:

@mhosken https://github.com/mhosken Thanks for the reply.
After reading the specs of OpenType, TTF/AAT and Adobe, and also
investigating a dozen OSS packages, I have a better understanding of the
problem.
Comparing the most promising alternatives (Adobe Type Tools Font-API,
Python fonttools, Font::TTF) to build on, I decided to start with Font::TTF
and modify it into Font::OTF.
Roughly estimated 30-70% of Font::TTF is reusable to some degree to
support .otf. It's well structured and readable.
Additional question: Is there a preferred or most popular XML-format for
font data, which should be supported?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/silnrsi/font-ttf-scripts/issues/19#issuecomment-382659121,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFbHbXgUtoZPlDoZ0sz9-BptbSrjAYN2ks5tqE81gaJpZM4TZf3r
.

Dear Martin,
personally I prefer Perl structures in my application of OCR and OFR (Optical Font Recognition). Serializing a Perl structure to JSON is just one line of code. Nearly all popular script languages like Python, PHP, JS can handle JSON nicely, and developers like it.
In my experience dealing with different XML formats e. g. for the results (the page segmentation, bounding boxes) of OCR systems, XML can be boring.
Supporting XML is more a matter of integration with other tools. Popular ones seem to be tx and ttx.
For OFR alone I can also use bitmaps of the glyphs, i. e. render them, or use scans of old type specimen catalogs. That's the way I do it now. But dealing with the guts of fonts directly opens new opportunities. One is creating fonts (of poor quality) from scans automatically.


New package: GNRProdEst v1.0.0

Registering package: GNRProdEst
Repository: https://github.com/MarkusTrunschke/GNRProdEst.jl
Created by: @MarkusTrunschke
Version: v1.0.0
Commit: e1b596aa96bd1b32fdabfb07f9c1f13edeac0f36
Reviewed by: @MarkusTrunschke
Reference: https://github.com/MarkusTrunschke/GNRProdEst.jl/issues/3#issuecomment-2161852419
Description: Implementation of the Gandhi, Navarro, Rievers (2020) production function estimator.

@JuliaRegistrator register


[Feature] Translate Netlify's Image Transformation parameters to Cloudinary params

Feature Request
Netlify offers a set of image transformations using three query parameters nf_resize, w and h. These parameters can be mapped to Cloudinary parameters c_, w_ and h_. Consider re-writing the Netlify URLs to Cloudinary URL format.
Additionally, if the setup has a default transformation, this may also need to be applied. For example, f_auto,q_auto.
Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

Large Media is officially deprecated: https://answers.netlify.com/t/large-media-feature-deprecated-but-not-removed/100804
probably makes even more sense to support it for people who want to transition away from this
i think this could also be expanded to support other cloudinary features but i wouldnt expect it to be all-encompassing as it could start to create issues between development and production which would lead to unexpected changes


New version of app no longer has testnet

So one of the benefits of this app is to help students learn about crypto. This app no longer seems to support test app. It would be nice if all releases of the app were also saved in github so at the very least i could get students to use the apk file from github.
I noticed that there does not appear to be a build system within the repo. Are the apk builds made manually? Could we not just leverage travis and have apks built and pushed to github+ other places.

Can you message me on telegram @vikmeup? I will send you enough real tokens to educate new students.


Custom Pager

Hi,
How to make a custom pager to replace the existing one ?
I want to make a pager contains pages each contains a text
Thanks

I used goToSlide() and it works :)


Update to latest upstream

Hi there,
You'll want to add in something like:
4278: login: Switch to SettingsProvider for "Alt Sign in" | https://review.calyxos.org/c/CalyxOS/platform_external_microg_GmsCore/+/4278
to have the spoofing code work well with the SettingsProvider refactor.

Hi there,
You'll want to add in something like:
4278: login: Switch to SettingsProvider for "Alt Sign in" | https://review.calyxos.org/c/CalyxOS/platform_external_microg_GmsCore/+/4278
to have the spoofing code work well with the SettingsProvider refactor.

Should be already done in the current code, with the same previous (re-adapted) code.

Hi there,
You'll want to add in something like:
4278: login: Switch to SettingsProvider for "Alt Sign in" | https://review.calyxos.org/c/CalyxOS/platform_external_microg_GmsCore/+/4278
to have the spoofing code work well with the SettingsProvider refactor.

Should be already done in the current code, with the same previous (re-adapted) code.

I don't see it at https://github.com/YTVanced/VancedMicroG/blob/4c9230d27994711465425d6942e3bf9419d9f182/play-services-core/src/main/java/org/microg/gms/auth/login/LoginActivity.java#L166
It's still calling getDefaultSharedPreferences directly, which should not be done.

Hi there,
You'll want to add in something like:
4278: login: Switch to SettingsProvider for "Alt Sign in" | https://review.calyxos.org/c/CalyxOS/platform_external_microg_GmsCore/+/4278
to have the spoofing code work well with the SettingsProvider refactor.

Should be already done in the current code, with the same previous (re-adapted) code.

I don't see it at
https://github.com/YTVanced/VancedMicroG/blob/4c9230d27994711465425d6942e3bf9419d9f182/play-services-core/src/main/java/org/microg/gms/auth/login/LoginActivity.java#L166
It's still calling getDefaultSharedPreferences directly, which should not be done.

This is the code to hide microg icon from launcher. I don't know if it's the case of change it.

Hi there,
You'll want to add in something like:
4278: login: Switch to SettingsProvider for "Alt Sign in" | https://review.calyxos.org/c/CalyxOS/platform_external_microg_GmsCore/+/4278
to have the spoofing code work well with the SettingsProvider refactor.

Should be already done in the current code, with the same previous (re-adapted) code.

I don't see it at
https://github.com/YTVanced/VancedMicroG/blob/4c9230d27994711465425d6942e3bf9419d9f182/play-services-core/src/main/java/org/microg/gms/auth/login/LoginActivity.java#L166
It's still calling getDefaultSharedPreferences directly, which should not be done.

This is the code to hide microg icon from launcher. I don't know if it's the case of change it.

What happened was calling getDefaultSharedPreferences from everywhere would override settings.
So we changed it in microG to use SettingsProvider. So this should be changed here too - both the hide launcher icon part, and also the lines below it, which I covered in my linked patch.
https://github.com/YTVanced/VancedMicroG/blob/4c9230d27994711465425d6942e3bf9419d9f182/play-services-core/src/main/java/org/microg/gms/auth/login/LoginActivity.java#L169-L173
This should all be changed to use SettingsProvider to make sure it doesn't get overriden when something else writes to it.
My commit handles this for the Huawei button, but we don't have hide launcher icon. See https://github.com/CalyxOS/platform_external_microg_GmsCore/commit/a68d262abd36a33c2aa88c69aad5b143fc251ceb

Hi there,
You'll want to add in something like:
4278: login: Switch to SettingsProvider for "Alt Sign in" | https://review.calyxos.org/c/CalyxOS/platform_external_microg_GmsCore/+/4278
to have the spoofing code work well with the SettingsProvider refactor.

Should be already done in the current code, with the same previous (re-adapted) code.

I don't see it at
https://github.com/YTVanced/VancedMicroG/blob/4c9230d27994711465425d6942e3bf9419d9f182/play-services-core/src/main/java/org/microg/gms/auth/login/LoginActivity.java#L166
It's still calling getDefaultSharedPreferences directly, which should not be done.

This is the code to hide microg icon from launcher. I don't know if it's the case of change it.

What happened was calling getDefaultSharedPreferences from everywhere would override settings.
So we changed it in microG to use SettingsProvider. So this should be changed here too - both the hide launcher icon part, and also the lines below it, which I covered in my linked patch.
https://github.com/YTVanced/VancedMicroG/blob/4c9230d27994711465425d6942e3bf9419d9f182/play-services-core/src/main/java/org/microg/gms/auth/login/LoginActivity.java#L169-L173
This should all be changed to use SettingsProvider to make sure it doesn't get overriden when something else writes to it.
My commit handles this for the Huawei button, but we don't have hide launcher icon. See CalyxOS/platform_external_microg_GmsCore@a68d262

We use another approach to Huawei spoofing (without a preference on settings page, from what you can see), so at the moment we just need this improvement for microg hide icon.
Can you provide it?

Hi there,
You'll want to add in something like:
4278: login: Switch to SettingsProvider for "Alt Sign in" | https://review.calyxos.org/c/CalyxOS/platform_external_microg_GmsCore/+/4278
to have the spoofing code work well with the SettingsProvider refactor.

Should be already done in the current code, with the same previous (re-adapted) code.

I don't see it at
https://github.com/YTVanced/VancedMicroG/blob/4c9230d27994711465425d6942e3bf9419d9f182/play-services-core/src/main/java/org/microg/gms/auth/login/LoginActivity.java#L166
It's still calling getDefaultSharedPreferences directly, which should not be done.

This is the code to hide microg icon from launcher. I don't know if it's the case of change it.

What happened was calling getDefaultSharedPreferences from everywhere would override settings.
So we changed it in microG to use SettingsProvider. So this should be changed here too - both the hide launcher icon part, and also the lines below it, which I covered in my linked patch.
https://github.com/YTVanced/VancedMicroG/blob/4c9230d27994711465425d6942e3bf9419d9f182/play-services-core/src/main/java/org/microg/gms/auth/login/LoginActivity.java#L169-L173
This should all be changed to use SettingsProvider to make sure it doesn't get overriden when something else writes to it.
My commit handles this for the Huawei button, but we don't have hide launcher icon. See CalyxOS/platform_external_microg_GmsCore@a68d262

Sorry for the previous (nonsense) message.
I'll merge your code soon.

Sorry for the previous (nonsense) message.
I'll merge your code soon.

No worries, and to be clear we're using the exact same mechanism for brandSpoof, it's an extra button on the login screen.

P.S: Same thing should eventually be done with others preferences?

Yeah. it is done with all upstream preferences, but if you have added anything you should switch them all over.
See https://github.com/YTVanced/VancedMicroG/pull/152/commits/aa3a2d65cda328717ac75626d3466b37482e8d4c (and ofc my commit) for some examples.

Sorry for the previous (nonsense) message.
I'll merge your code soon.

No worries, and to be clear we're using the exact same mechanism for brandSpoof, it's an extra button on the login screen.

Yeah...I thought you followed another way.
In every case I'm currently working on hide microg icon switch and the one about duplication cast.


Pathfinding Improvements: Bound Algorithm to Canvas

The current algorithm is only bounded by walls and the top and left of the canvas. If you try to move a token to a space there is no path to (e.g. from a room with the door closed to outside that room) and that space touches the right or bottom of the canvas, the algorithm will hang forever as it tries to find a way around off the grid.
I've added bounds to the right and bottom of the canvas as well. Fixes #165.
This PR also moves the existing caching method, so it isn't in between the methods used by the algorithm.

Looks great! Thanks.


feat: Add tvOS support

Description
Adding tvOS support to allow using AEP with tvOS

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@cdhoffmann Can you please add these changes too to be included in the next release along with the https://github.com/adobe/aepsdk-core-ios/pull/798

failing tests related to CircleCi image that needs to be updated to newer one (13.4.0 is the latest)

@alexzchut Thanks for your contribution. We have release the full tvOS support with v3.1.0


Refactoring into multiple components

As the main component is getting bigger, ytcFilter will need to be refactored to make it more manageable and ensure a sane code base for future features.

Done with v2


Update README-editors.md to point to actual documentation

We have a README-editors.md file in this repo, as per convention with other OBO ontologies.
The first line states:
A step-by-step guide for GO Ontology Editors can be found at http://go-ontology.readthedocs.io/.
But we decided to move from RTD to the wiki. This should be updated.
It then goes on to say
This Jenkins job will create releases https://build.berkeleybop.org/job/release-go-ontology-snapshot
But this is out of date

@cmungall
Do you mind if I delete this folder ?
https://github.com/geneontology/go-ontology/tree/master/docs

I would like to merge the 2 README:
General
https://github.com/geneontology/go-ontology/blob/master/README.md
Editors:
https://github.com/geneontology/go-ontology/blob/master/src/ontology/README-editors.md
Anything not relevant to editors should go in the General doc.

@cmungall
Do we still need the SynGO build ?

TEMPORARY

https://build.berkeleybop.org/job/sync-go-editors-from-github/ - copy from this repo to old svn repo, from which current pipeline currently runs

syngo or sync go?
yes but we don't need it in the README

syngo or sync go?

It's sync go - sorry.

[x]  Edited  go-ontology/src/ontology/README-editors.md
[x]  Edited   go-ontology/README.md
[x] Deleted folder go-ontology/docs/ (now replaced by the wiki; links to the wiki are in the READMEs)

https://github.com/geneontology/go-ontology/pull/19041
https://github.com/geneontology/go-ontology/pull/19042
https://github.com/geneontology/go-ontology/pull/19043


Added missing generic on.sortChanged handler

sortChanged event handler which is in the documentation (under Methods, not Event handlers): http://ui-grid.info/docs/#/api/ui.grid.core.api:PublicApi was missing in .d.ts file.
This is a re-submission (and a fix) of previous PR #5901
And also possible fix of #6076

ui-grid/ui-grid.d.ts
to authors(@btesser @joeskeen). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

@vvakame, LGTM :+1:

thanks mate!


Question: Self hosted editing MySQL Directly

Ahoy
I just found out you can self-host the LuckPerms Web UI, which would mean we could just plug in our MySQL details and use the editor directly with the MySQL table without need for the commands in game to generate URLS for temporary pastebins of the permissions. Is this how it works at present? or does it just duplicate the current method of Log into Minecraft > Run command > Open Website in browser & make changes, Copy URL back in-game?

Self-hosting the web interface just lets you point the editor/other tools to your own domain, it does not enable any new functionality between the UI and the plugin.

Shame, it could work like phpMyAdmin with permissions.

Not really. The editor works without any backend logic. Meaning that you can't connect to a database server. Or if you could and wanted to preconfigure the database details you'd send them over the internet.
As mentioned before I've been toying with the idea of creating an extension for LP that allows the edtior to work live on the data. The main issue here being security.

But it would improve it drastically fixing the bad workflow at present, if you have a web application running locally, why on earth would you choose to send data to a remote server to then pull the data into the web application when you have direct access via the database! Flawed design, it requires extra steps.
The fact you are using nodejs means you are fully capable of making database queries.

The vast majority of our users have no means or desire to self-host the editor, hence the current centralised design. I do not see it as a flaw.

If you have a web application running locally, why on earth would you choose to send data to a remote server to then pull the data into the web application when you have direct access via the database! Flawed design, it requires extra steps.

Considering it would require significant effort to implement it, require a locally running node instance (the current editor works with static files only and only needs node to generate those files) and it would now only support a single storage type I can very much see why this hasn't been tackled.
My proposed solution with an extension to LP would be better. As that would have all the upsides you have with the current editor design (static web files only, support for all storage options) and would need only one additional port the MC server needs to be able to listen to.

Yeah definitely out of scope of your user base, but requested as an alternative to the symbolic links if I were required to use this workflow.
Looks like the Symbolic link issue was fixed, so I will be able to use that.

I'll say it again: If you're working so much on permissions that opening an editor session is a significant slow down in your workflow, you should consider using a developement server instead of doing everything live.


Evict realpath cache

As we recently learned in our room11 discussion, the php-src realpath cache eviction works only suboptimal.
Would it make sense to use some kind of timer/request count based check which verifies that the realpath cache is not full or triggers cache clear.
This just to make sure that no unnecessary file io happens within php-src because of a filled realpath-cache

Could you link the discussion? I think that's something that should be solved in php-src rather than with hacks here. How would we clear that cache?

It will finally be solved in php-src but it feels like we should have a solution in userland as not everybody will use the latest versions

http://chat.stackoverflow.com/transcript/message/34952738#34952738 Starting 5:32 PM

@staabm We're not the kind of developers working around every possible issue if it's not important. We did something for PHP 5.6 TLS in amphp/socket, but generally we just require the newer PHP versions then.
If it only hits performance a little bit in earlier versions, that's fine with me.

it will lead to lots of IO when the cache is full... we could also emit a warning or document how people should configure the php-ini setting properly

@staabm What's the current state @ php-src?

As far as I remember we increased a lot of default settings so the peoblem does not happen that often, see https://github.com/php/php-src/pull/2271
The eviction strategy wasn changed, see https://php-lxr.adamharvey.name/source/xref/master/Zend/zend_virtual_cwd.c#632
The bug still is a thing see https://bugs.php.net/bug.php?id=73887

Ot sure whether @krakjoe @nikic or @bwoebi will find some time to work on the eviction strategy

The current implementation just doesn't evict at all, but just doesn't store any new entries in case the cache is full. https://github.com/php/php-src/blob/2cfd64cad856324eead686724d12cb11a1859ab6/Zend/zend_virtual_cwd.c#L625

If you want to see movement on this, I would suggest implementing some self-contained representative stress-tests that can be used to gauge the quality of any solution we may implement. I think there are some very simple improvements we can make (e.g. discard fixed percentage of LRU on full cache), but we need something to test against. Maybe it's possible to collect a trace of all realpath cache accesses for a couple of cases where you are currently observing degraded performance attributable to realpath cache misses?

I don't think we need a particularly high quality solution, a trivial LRU should really be enough here I assume.
A simple LRU will be already a vast improvement over no eviction at all upon full cache.

I implemented a simple round robin eviction, see https://github.com/php/php-src/pull/2407.

I'm going to close this. I haven't seen real world problems yet. If they occur, please add them to the bug reports and issues that exist for php-src.


Merge PRs associated with v0.8.0 to main

Merge all PRs associated with v0.8.0 release.

[ ] Create a branch stable/0.8-ckt after everything is merged to main and before closing this issue

I created a stable/0.8 branch, which forked from stable/0.7 instead of main so it would be as much like 0.7 as possible.  Calling this one done.


How do I correctly set the Api endpoint for the Android simulator

On the android simulator, the data is not loaded because the endpoint it is trying to hit is https://localhost:3100, which as you indicated, does not work on the simulator.

How do I fix this?

You would have to figure out the IP address of your computer and use that

Ok I just tried setting API_ENDPOINT to https://<IP_ADDRESS>:3100/ which is the ip of my computer but then the api stopped working. This is probably not the way you want me to use this - am I right?

Yeah starting the app with API_ENDPOINT=https://{MY_IP_ADDRESS}:3100/  yarn start worked


[B03709] [MERGE] Copy iiif manifest uri to clipboard option when available

Coverage remained the same at 33.407% when pulling 561848a9ca964d06e9897be8d8585740259683bd on b03709-iiif-urls-merger into 2b2ac34101ad89d66c3aea6dbb8333c092182e9d on master.


about the minimum GPU memory required

Hi, @hszhao ,
Thanks for releasing the package. I'd rather to know the minimum requirement on GPU memory. Is a GPU with 12GB enough to run the training?
THX!

Hi, our experiments on ImageNet (crop size as 224x224, batch size as 256) are conducted on a server with 8 x Quadro RTX 6000 (24G). Detailed memory usage for each GPU is listed below:
Pairwise-SAN10/15/19: 13091/17259/20781MiB.
Patchwise-SAN10/15/19: 9007/11479/13493MiB.


added support for Azure Translation Service

There may be a better way to add support for Azure Translation service. can you please review and let me know how I can improve this?

@anand4u13,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

I signed the cla...

Hi, I am closing and re-opening this PR to bump the CLA bot. Sorry for the inconvenience!

@anand4u13,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks for your pull request. This is better to be implemented as an (IPostToBot)[https://github.com/Microsoft/BotBuilder/blob/master/CSharp/Library/Microsoft.Bot.Builder/ConnectorEx/PostToBot.cs#L50] and be dynamically injected to the IPostToBot middleware pipeline: https://github.com/Microsoft/BotBuilder/blob/master/CSharp/Library/Microsoft.Bot.Builder.Autofac/Dialogs/DialogModule.cs#L342
Also (botbuilder-azure)[https://github.com/microsoft/botbuilder-azure] github repo might be a better home for this IPostToBot.


change CloseMenu to ClosePopup

Substitute ClosePopup

Will be available on beta.11 release. Thanks for the PR!


Dynamic configuration (@type opensearch_dynamic) is not supported to use configurations to depend on information in messages

There is no support available for configurations to depend on information in messages as it is available in fluent-plugin-elasticsearch (https://github.com/uken/fluent-plugin-elasticsearch/tree/v5.1.4#dynamic-configuration)
Steps to replicate
<match test.logging.**>
@type copy

@type opensearch_dynamic
host opensearch
port 9200
resurrect_after 5s
type_name fluentd
time_key time
utc_index true
time_key_exclude_timestamp true
logstash_format true
logstash_prefix fluentd-${tag_parts[2]}-${tag_parts[3]}
<buffer tag, time, namespace, type>
@type file
path /var/log/td-agent/opensearch-buffer/test.logging.all.all
flush_mode interval
flush_interval 30s
timekey 3600
retry_forever true
retry_max_interval 5s
overflow_action block
total_limit_size 1024m

Expected Behavior or What you need to ask
Indices should get created in correct format with  fluentd-${tag_parts[2]}-${tag_parts[3]} after evaluating place holder values. However getting below error in the logs: "2022-01-31 08:45:42 +0000 [error]: config error file="/etc/td-agent/td-agent.conf" error_class=Fluent::ConfigError error="Unknown output plugin 'opensearch_dynamic'. Run 'gem search -rd fluent-plugin' to find plugins"
...
Using Fluentd and OpenSearch plugin versions
Fluentd version 1.14.3
Opensearch version: 1.2.3
'fluent-plugin-opensearch' version '1.0.1'
Please suggest how to use dynamic configuration with this plugin.

opensearch_dynamic plugin is already obsoleted and removed.

dynamic plugin won't support anymore on OpenSearch flavor of Fluentd plugin.

<match test.logging.**>
      @type copy
      <store>
          @type opensearch_dynamic
          host opensearch
          port 9200
          resurrect_after 5s
          type_name fluentd
          time_key time
          utc_index true
          time_key_exclude_timestamp true
          logstash_format true
          logstash_prefix fluentd-${tag[2]}-${tag[3]}
          <buffer tag, time, namespace, type>
              @type file
              path /var/log/td-agent/opensearch-buffer/test.logging.all.all
              flush_mode interval
              flush_interval 30s
              timekey 3600
              retry_forever true
              retry_max_interval 5s
              overflow_action block
              total_limit_size 1024m
         </buffer>
      </store>
    </match>

would work

Thanks @cosmo0920 , This configuration has worked.


Add options for additional success, failure, advantage, threat, etc. to dice rolls

Add dialogue or something similar to the dice roller for adding automatic symbol results to the roll. Similar to the modifier box in Roll20.

I think this may be a 2-part deal.

Ability to manually add success/failure/threat/etc to dice rolls through dice roller dialog
Ability to add modifier to items that will automatically add non-situational success/failure/threat/etc to dice rolls, like setback/boost.

Now in dev branch.  You can add positive or negative additional symbols (success/failure/advantage/threat), The added amounts will show up in the roll message in chat.


Allow debug to be set via envvar/toml

Just running this on heroku, and it would be nice to be able to set this dynamically from an envvar without pushing a new release :)

Added in master, you can use DEBUG=1 now

You da best!


Error while extracting existing model // 提取现有模型时出现错误

I have access to a model of an artist that someone had already trained & sent me. This is what the directory is looking like (I manually put it in logs):
(中文翻译：我可以访问到一个已经被别人训练并发送给我的艺术家模型。下面是目录的样子（我手动放在了日志中）：)

I'm trying to extract the model so that the model will show up on the "timbre" dropdown, but I'm met with this error:
(中文翻译：我试图提取模型，以便该模型会在“音色”下拉菜单中显示，但是我遇到了这个错误：)
Traceback (most recent call last):
File "C:\Users\samir\OneDrive\Desktop\RVC-beta\train\process_ckpt.py", line 107, in extract_small_model
opt["weight"][key] = ckpt[key].half()
AttributeError: 'dict' object has no attribute 'half'

I've reinstalled dependences, ensured that the correct pytorch is installed, tried placing the model files outside of the 'logs' folder. Can anyone tell me what I'm doing wrong?
(中文翻译：我已经重新安装了依赖项，确保正确安装了PyTorch，并尝试将模型文件放在“logs”文件夹之外。有人可以告诉我我做错了什么吗？请随意用中文或英文回答，非常感谢你所做的工作。)

You don't need extract it. The model is the extracted one. Just put it into "weights" folder. And click "refresh timbre list".

Put .pth file into weights folder and refer .index file in the Model inference tab.


[Feature] Support for zip file processing

Most transit agencies support gtfs which is the protocol google created to handle the processing of transit information. There are two parts of the system. Real-time data which is transferred via protocol buffers and gtfs static which is exposed as a zip file. The minimal needed to use this data for a Tidbyt app is the static zip but to do that we need the ability to parse zip files.
Context: I built a community app for the Long Island rail road. It relied on a secondary file that the MTA provided that took the zip and converted it into json. They got rid support for that file on the ground that developers can get the same data from the raw zip.

This and a lot of other useful functionality could be added relatively easily by making starlib available to applets.

Actually, it looks like several other starlib modules are already included, including gzip (but not zipfile)


Question: Files often get accidently deleted a few seconds after opening

Thanks for your great plugin.
Relatively often when I open a note after starting Obsidian, it appears briefly before being deleted. It doesn't matter whether I open them via File Explorer or via recent files.
(I can help myself by using core plugins -> data recovery.)
When I deactivate the plugin, it seems that the problem has gone. Can you please check if it could be your plugin?
I'm on Windows 10.

I have tested it extensively and I'm sure, your plugin is the cause.

Hello, sorry for the very late response :)
If you're still interested in this plugin, can you check if it's still happening in 1.2? I refactored a lot of the code.
I can't think of what could be causing this though. The plugin only opens files, it doesn't modify or delete them in any way. Maybe your sync provider or another plugin doesn't like the same file being opened twice in quick succession?

Thank you very much for refacturing, I'll test it later this day. It would be great if I could use your plugin productively again.

The good thing: I installed your plugin in a TestVault an there it works. ;-)
But in my productive vault, it doesn't work. these are my settings:

The console says:

SYSTEM INFO:
Obsidian version: v1.6.7
Installer version: v1.6.7
Operating system: Windows 10 Home 10.0.19045
Login status: not logged in
Insider build toggle: off
Live preview: off
Base theme: dark
Community theme: MagicUser v20.8.0
Snippets enabled: 55
Restricted mode: off
Plugins installed: 181
Plugins enabled: 62
1: Set View Mode per Note v1.0.3
2: Advanced Tables v0.21.0
3: Android nomedia v0.1.0
4: Auto Link Title v1.5.4
5: Beautitab v1.6.1
6: Better Export PDF v1.9.1
7: Commander v0.5.1
8: Copy document as HTML v0.8.1
9: Creases v0.7.0
10: Custom Classes v2.6.1
11: Dictionary v2.22.0
12: Editing Toolbar v2.4.16
13: Editor Syntax Highlight v0.1.3
14: Enhancing Export v1.10.7
15: Excel to Markdown Table v0.4.0
16: Export Image plugin v2.3.10
17: File Explorer Note Count v1.2.2
18: Google Calendar v1.10.14
19: Linter v1.25.0
20: Local Backup v0.1.7
21: markdown export v1.0.24
22: Natural Language Dates v0.6.2
23: Navigate Cursor History v1.1.0
24: Number Headings v1.16.0
25: Omnisearch v1.24.0
26: Paste image rename v1.6.1
27: Quiet Outline v0.3.36
28: Recent Files v1.4.1
29: Sheet Plus v1.3.2
30: Show Current File Path v0.5.2
31: Sortable v0.3.1
32: Status Bar Organizer v2.1.1
33: Style Settings v1.0.8
34: Tag Wrangler v0.6.1
35: Tasks v7.8.0
36: Templater v2.4.1
37: Vault Changelog v0.1.0
38: Vault Statistics v0.1.3
39: Webpage HTML Export v1.8.01
40: Dataview v0.5.67
41: Plugins Annotations v1.4.0
42: Docxer v2.0.1
43: Supercharged Links v0.12.1
44: Query Control v0.6.0
45: Note Toolbar v1.9.12
46: Refresh Preview v1.3.0
47: Hover Editor v0.11.20
48: Advanced URI v1.42.0
49: Find orphaned files and broken links v1.10.1
50: Multi Properties v1.1.2
51: Table Generator v1.4.1
52: Image in Editor v2.2.6
53: Quick Explorer v0.2.10
54: Dashboard navigator v1.1.1
55: Quick Switcher++ v4.4.0
56: Backlink Cache v1.8.1
57: Tab Switcher v1.4.0
58: Open File by Magic Date v0.1.2
59: Reset Font Size v1.0.2
60: Enhanced Symbols Prettifier v1.1.2
61: Typing Transformer v0.4.0
62: Auto Split v1.2


Fix for issue #411 URL for WIN11_x64_Enterprise_22H2_EN_Eval not working

Download link and checksum fixed.
The download link is from here: https://www.microsoft.com/en-us/evalcenter/download-windows-11-enterprise

Done.


After effects of Committing to component library, new details to figma design

If new UI decided, figma must be affected to utilize the full benefit of UI, navbar should meet template first etc etc

Done!


Automatically add "X-Correlation-ID" to outgoing HTTP Client request via HttpClientFactory & DelegatingHandlers

I was reading your articles of HttpClientFactory  usage in AspNet Core 2.1 ,  I was thinking we can add "X-Correlation-ID" header automatically to outgoing HTTP Client request via HttpClientFactory & DelegatingHandlers.  Is this a possibility ?

Same here. I found your example:
        var builder = new WebHostBuilder()
           .Configure(app => app.UseCorrelationId())
           .ConfigureServices(sc => sc.AddCorrelationId());
        var server = new TestServer(builder);
        var response = server.CreateClient().GetAsync("").GetAwaiter().GetResult();
        var expectedHeaderName = new CorrelationIdOptions().Header;
        var header = response.Headers.GetValues(expectedHeaderName);

However, I don't get how I can set up my  HttpClientFactory to include Correlation ID in all my outgoing HTTP Client requests.

Consider using header propagation instead; https://gist.github.com/davidfowl/c34633f1ddc519f030a1c0c5abe8e867
Just a thought :-)

@ajtowf Agree that is a good option. I've added a feature to support basic header forwarding directly from this library if people prefer to use that.


init() got multiple values for argument 'device'

I run ./onpolicy/scripts/train_mpe_scripts/train_mpe_spread.sh after change 'algo' to mappo and user_name to my wandb user name in train_mpe_spread.sh. My train_mpe_spread.sh is as follows:
#!/bin/sh
env="MPE"
scenario="simple_spread"
num_landmarks=3
num_agents=3
algo="mappo" #"rmappo" "ippo"
exp="check"
seed_max=1

echo "env is ${env}, scenario is ${scenario}, algo is ${algo}, exp is ${exp}, max seed is ${seed_max}"
for seed in `seq ${seed_max}`;
do
    echo "seed is ${seed}:"
    CUDA_VISIBLE_DEVICES=0 python ../train/train_mpe.py --env_name ${env} --algorithm_name ${algo} --experiment_name ${exp} \
    --scenario_name ${scenario} --num_agents ${num_agents} --num_landmarks ${num_landmarks} --seed ${seed} \
    --n_training_threads 1 --n_rollout_threads 128 --num_mini_batch 1 --episode_length 25 --num_env_steps 20000000 \
    --ppo_epoch 10 --use_ReLU --gain 0.01 --lr 7e-4 --critic_lr 7e-4 --wandb_name "xxx" --user_name "my_wandb_name"
done

Then I got an error
Traceback (most recent call last):
  File "/home/drl/Projects/on-policy/onpolicy/scripts/train_mpe_scripts/../train/train_mpe.py", line 174, in <module>
    main(sys.argv[1:])
  File "/home/drl/Projects/on-policy/onpolicy/scripts/train_mpe_scripts/../train/train_mpe.py", line 158, in main
    runner = Runner(config)
  File "/home/drl/Projects/on-policy/onpolicy/runner/shared/mpe_runner.py", line 14, in __init__
    super(MPERunner, self).__init__(config)
  File "/home/drl/Projects/on-policy/onpolicy/runner/shared/base_runner.py", line 80, in __init__
    self.policy = Policy(self.all_args,
TypeError: __init__() got multiple values for argument 'device'

I found on-policy/onpolicy/runner/shared/base_runner.py, line 66~71 choose R_MAPPOPolicy as Policy:
        if self.algorithm_name == "mat" or self.algorithm_name == "mat_dec":
            from onpolicy.algorithms.mat.mat_trainer import MATTrainer as TrainAlgo
            from onpolicy.algorithms.mat.algorithm.transformer_policy import TransformerPolicy as Policy
        else:
            from onpolicy.algorithms.r_mappo.r_mappo import R_MAPPO as TrainAlgo
            from onpolicy.algorithms.r_mappo.algorithm.rMAPPOPolicy import R_MAPPOPolicy as Policy

The constructor of R_MAPPOPolicy is
    def __init__(self, args, obs_space, cent_obs_space, act_space, device=torch.device("cpu")):

but on-policy/onpolicy/runner/shared/base_runner.py, line 80~85 provide incompatible arguments to the constructor of R_MAPPOPolicy. The second last one is redundant.
        self.policy = Policy(self.all_args,
                            self.envs.observation_space[0],
                            share_observation_space,
                            self.envs.action_space[0],
                            self.num_agents, # default 2
                            device = self.device)

What should I do? Thanks!

Same issue It appears to be related to a mismatch in the passed parameters. One potential, albeit unsafe, solution would be to simply remove self.num_agents.

OK, thanks!


Feature request: support for domain connect

A thing referred to as 'domain connect' exists:
https://datatracker.ietf.org/doc/draft-kowalik-regext-domainconnect/
What this does it automating the update of DNS records at DNS operators that support this.
Whether or not this is the case, can be discovered by checking for a TXT record _domainconnect , for example:
dig +short TXT _domainconnect.example.nl
"api.cloudflare.com/client/v4/dns/domainconnect"

I don't know how hard it is to implement support for this in Mox, but I could imagine it would be a great feature to have.

Thanks, this indeed looks interesting. It was mentioned here too: https://github.com/mjl-/mox/issues/126#issuecomment-2228134348
I read through the domainconnect I-D again. Some new notes:
It says the synchronous flow (where you make a one-time DNS change) is more commonly supported than the asynchronous flow (where we could make DNS changes at a later time again). The flow is web-based. Based on that, domainconnect may especially help with the initial setup of mox (at least when mox gets a web-based guided setup), less easily with key rotations.
I'm still a bit suspicious of the central database of service templates, and that we would have to get each dns operator to use ours. Would be good to hear about experiences from developers about that, especially those developing self-hosted software, i.e. not a centralized online service. Signing requests seem recommended (for security), with a public key in the template. But on first look, that seems to assume a service provider that can keep the corresponding private key secret. If that's the case, it wouldn't work for software everyone runs themselves.
I find it surprising that the domainconnect I-D doesn't mention DNS updates (rfc2136), and doesn't mention DNS propagation time considerations, or DNS record rotation considerations (e.g. for DKIM keys, or DANE; it does have a section on merging SPF records, so DNS records for email were at least considered).
Next steps could be to do some prototyping:

Write a domainconnect template for the records the mox quickstart suggests.
Find software that can fake being a domainconnect server, for testing the template. Or get it included by a dns operator and get an account with them (seems a bit heavy just for testing).
Change quickstart to do domainconnect discovery, and print a URL for the user to get the records created.

In other news: I prototyped some code for automated DKIM key rotations, changing DNS records with DNS dynamic updates (rfc2136) and checking/verifying them with AXFR/IXFR, taking propagation times into account (zone negative cache TTL and record TTL). No problems encountered. Still seems like a reasonable approach, at least for my own use case. It will allow for making more kinds of changes from the admin UI/cli (i.e. one click/command to enable acme account binding). The problem of course is that DNS operators don't offer access with those standard protocols. But writing a simple DNS updates/[ia]xfr proxy (that uses an existing Go library (I found multiple candidates) to get/set records through cloud API's) should solve that problem.

Thank you for the reply. Indeed I was thinking about using domainconnect primarily for the initial setup of Mox.
With regard to the 'in other news';
I recently came across this project. It is, among other things, a pretty impressive, but simple  authoritative name server that can even do DNSSEC signing on the fly.
Ant it made me wonder; why not extend Mox with such a server for the DNS records that need to change regularly ? The Go-code of the above project can be used as a great starting point for that.


RPC erroneously passes EncodedObjectAsId

Godot version:
3.1.1

OS/device including version:

Windows 10 - client
Amazon linux on EC2 - server
Issue description:

An RPC function that previously worked in 3.0 was called with a reference to a node to translate the node to a preset position. RPC call failed as parameter received was of type EncodedObjectAsId.
Steps to reproduce:
Create a simple server/client architecture. Immediately after client joins and spawns a player object, attempt to use an rpc call to translate it to a new location
Minimal reproduction project:

@kiidmale this was changed due to: #27395 (breaking compat).
See the release notes of 3.1.1 and #27485 .
Basically, you now need to set the NetworkedPacketPeer allow_object_decoding property to true (as was originally intended, but never implemented correctly until now).
This is done for security reason, as decoding objects allows remote code execution.

What is the recommended method of passing parameters to ensure that remote code execution attacks do not occur?

What is the recommended method of passing parameters to ensure that remote code execution attacks do not occur?

Do not allow object encoding, and only use built-in types for RPCs (e.g. String, Vector, PoolVector, Transform, Array, Dictionary, etc, except Object of course).

Closing - appreciate the informative responses, @Faless


move authentication into credible git submodule

Linked Issue(s)

closes #63

Description
Extracts authentication logic into https://github.com/thombruce/credible
Checklist

[x] I have read the Contributing Guide and Code of Conduct

Still todo...

[ ] Extract logic from models into a model helper in Credible gem


Enable HTTP

Could an option to allow HTTP instead of redirecting to another browser be added?
Global or per webapp based it would be welcomed since is almost impossible to login on certain sites which use http pages for redirection between other https whih would go through

I get you, often you put in a site address and it kicks you out into a browser. If you copy & paste the resulting Url from the browser into WebApps, it should then normally then work (if it is HTTPS). Enabling HTTP is just way too dangerous - if it redirects to an HTTP login page, you wouldn't know. That said, this is common enough that I could look at detecting an HTTPS->HTTP->HTTPS redirect sequence and open the final HTTPS page.

I was having this login issue with gmail and this webpage https://curiouscat.me/ which is just a dumb question and answers page but I didn't wanted my Twitter logged on my main browser. Copy and pasting the https URL didn't worked, I tried before asking.
I know I can use Gapps browser for gmail and it works nicely but is kinda annoying to have two apps with such similar behaviour to just sandbox three or four webpages.
That being said thanks a lot for your work and the quick answer :)


Add Triggers Release

Add Triggers to operator alongside pipeline.

@khrm
just few linting errors
./deploy/resources/v0.8.0/03-triggers/tektoncd-triggers-v0.1.0.yaml
271:16    error    too many spaces after colon  (colons)
558:16    error    too many spaces after colon  (colons)
571:1     error    too many blank lines (1 > 0)  (empty-lines)

/retest

/retest

/test e2e

/retest

retest

/retest

/retest

/retest

/retest

/retest

/retest