refactor scan command

Now "scan" commands are very redundant and complicate, see #137. It is time to refactor them.
For compatibility, I may not remove old XSCAN, XLSCAN, etc..., so maybe another API is better, like this:
XKEYSCAN type cursor [MATCH match] [COUNT count]

"type" is kv, list, hash, set, and zset. This API likes redis SCAN, to iterates keys, for different data type.
But if we don't care compatibility, we may use XSCAN instead, @pkieltyka.
XHSCAN key cursor [MATCH match] [COUNT count]

XHSCAN likes redis HSCAN to iterates hash field and values.
XSSCAN key cursor [MATCH match] [COUNT count]

XSSCAN likes redis SSCAN to iterates set memebers.
XZSCAN key cursor [MATCH match] [COUNT count]

XZSCAN likes redis ZSCAN to iterates zset element and score.

Good stuff! I don't think backwards compatibility for these kinds of commands matter.
Also, what do you think about the naming of XHSCAN instead of HXSCAN ? I like the idea of prefixing any "extensions" to redis commands with an X.. makes for more consistency.

Aha, @kouhate suggests me to prefix data type for commands like Redis. I think using X prefix may be good to distinguish Redis commands.
Maybe we can think another name, like HSEEK?

Hi there, OK. I agree to use X prefix.
I think H S Z prefix makes the target data type more certain..., but this may disturb the consistency.

Hi @kouhate,  Support in develop branch now. Commands:
XSCAN type cursor [MATCH match] [COUNT count]
XHSCAN key cursor [MATCH match] [COUNT count]
XSSCAN key cursor [MATCH match] [COUNT count]
XZSCAN key cursor [MATCH match] [COUNT count]

Thanks @siddontang, I'll test them later.

Hi @siddontang, would you also support reverse scan?
As I know reverse scan is more slower than forward one, but this may useful for some use case.
Maybe XREV.. bring more redundancy like before, so if you could, supporting by option would be better.
XSCAN type cursor [MATCH match] [COUNT count] [ASC | DESC]

Hi @kouhate , redis does not support reverse scan, and I don't find a useful use case too.
Btw, supporting reverse scan is easy, is it necessary to support reverse scan for hash, set and zset?

@siddontang, especially for hash, reverse scan is necessary I think.
In my use case, Hash filed-name will TimeUUID and I want to retrieve recent data.
For time series data,  maybe reverse scan cannot be avoided, or app must plan the key name to be reversed.
In the case of Cassandra CQL3, they support WITH CLUSTERING ORDER BY (clumn-name [ASC | DESC]) option for setting default order.
Once the table are created with the option, saved data is always ordered by the specified order.
So, the performance of reverse heavy operation will be boosted after setting the DESC option.
Furthermore, ZSET also have a similar problem.
For ranking or access counting, what we are interested in is usually the highest score data.
So, we easily use reverse scan such as ZREVRANGE or ZREVRANGEBYSCORE, but unfortunately, these commands are 10 times slower than forward scan, especially using leveldb or rocksdb as storage.
This is not happend when using Redis.
In my app, sometimes a score of ZSET is prefixed by "-" on purpose of reverse sorting, but this force me to decode and encode when inserting and retrieving.
Anyway, reverse scan is necessary for my use case.

@kouhate , I will add reverse scan later, but you must know below:

reverse scan is slower than forward, unless leveldb or rocksdb improves it.
migrating a huge hash data (with many fields and values) is very terrible in xcodis when resharding. We must dump this huge data first, then transfer into another server and load it, it may take huge memory, cost many time and block other write operations for a long time.

Btw, I think ZREVRANGE* may be more convenient than XZSCAN DESC.

@siddontang, ok, I know the two cons, but does XMIGRATEDB keep blocking any write operation until  finished?
If so, it may be a fatal problem for me.

Now, yes, because I must guarantee the atomic migration. When I migrate the key, other write operation may update the key at same time, it may cause data inconsistency.
But this can be improved later, only using a lock for the key and not for the whole.

@kouhate , for redis, scan can not guarantee the return key order, the same for ledisdb. e.g. for hash, the key saved in ledisdb like this db(1 byte):hash tag(1 byte):key length(2 byte, big endian):key, so the key with short size will be scanned first. If the two key have same size, we will use lexicographical ordering.
So I think you should have the same size for your hash field if you want to scan them with ASC or DESC.
Btw, for redis, ZSCAN first scans elements with small score, then uses lexicographical ordering for same score, but for ledisdb, it only use lexicographical ordering, is this a problem for you?

Hi @siddontang, those difference is not a problem.
My purpose of using *XSCAN is to scan by value, so this is rather welcomed.

Hi @kouhate, I have been trying to use ledisdb + xcodis + redis-failover to build a high availability distributed key-value store service in my company.  I guess you may do the similar thing.
Could you tell me your email? Maybe we can have a in-depth discussion or even work together to build a NoSQL product.

Hi @siddontang, I've just updated my prof, pls check it.


Add mandatory field selector

This selector fails the stream if a selected field is not present in the
object the selector is applied to. It can be useful when you know your
data must contain the field and want to raise an error if not (e.g. to
detect typos).
fixes #30

Apart from the test compilation failures which I think are easy to fix, it looks good!

:man_facepalming:


Expose http_client_config from  odata_client

Then it could be used to customize http request, like send client cert.

What you suggested has been done in the fork I made (because I had the same need). You can clone it from here:
https://github.com/PBRCW/odatacpp-client

What you suggested has been done in the fork I made (because I had the same need). You can clone it from here:
https://github.com/PBRCW/odatacpp-client


how to change backgroud color?

In the new release 1.0.1 you can use app:panelBackgroundColor in xml or setPanelBackgroundColor(int _colorRes) in Java to set the color of the panel at the back. It's 6% black (#06000000) by default.
Note that the default color, as in the guideline, is the black (#000000) color with the transparency of 6%, so when you're customizing and want it to still be transparent you have to set a color with transparency.


Unknown solution item type

When trying to open the XamarinStudio project I receive this error.

Xamarin Version 5.9.4

I was able to correct this by adding the Add-In Maker But its still unclear to me how I use this inside Xamarin

@rebelzach Unfortunately, it's not working for XS yet. It's a blank extension project. The plugin works only for Visual Studio so far.


Hyperf PostgreSQL - which packages to use?

https://github.com/hyperf/database-pgsql ?
https://github.com/hyperf/db-pgsql-incubator ?
https://github.com/hyperf/database-pgsql-incubator ?
What is the correct one, and what is the difference?
Do I need hyperf/db ? hyperf/database ? hyperf/db-connection ?
Is there PDO connection pool for postgres?
Any clarification would be very appreciated 🙏 Thank you!

database
If you want to use it in Hyperf
You also need db-connecrion

From here https://github.com/hyperf/hyperf/blob/master/docs/en/db/quick-start.md#hyperf-framework
It sounds like if I use hyperf/database then that is NOT for hyperf?
Is it maybe more correct the docs should say:
To use in Hyperf or in other frameworks
composer require hyperf/database

For Hyperf framework you also need
composer require hyperf/db-connection


Fix typo

Fixes #4626 :)
I've signed the CLA.
Name: Mihir Karandikar

I've confirmed @KarandikarMihir has signed the CLA.


Add filter LabToRgb e fix return image in RgbToLab

Description
In order to implement #322, we need to convert back from the LAB space to the RGB space.
Moreover, it fixes a bug in the filter RgbToLab, where by mistake the resulting Lab image was converted to a PIL image, messing up the value range and type (it should be float in the range −128 to 127).
Types of Changes

[ ] Core
[x] Bugfix
[x] New feature
[x] Enhancement/optimization
[ ] Documentation

Issues Fixed or Closed by This PR

Fixes:

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

PR #334 needs to be merged first


Readme design changes

I've reworked and organized the README.md and also created a new logo for the project.
The new logo is a combination of the FIAT emblem together with their parent company's logo STELLANTIS.

New
Stellantis

Let me know what you think.

I also noticed that under ./FiatChampAddon/README.md there is another readme. Is it needed or is it possible to delete it?

@seriousm4x  its because home assistant uses the readme from the addon folder not the one from the root folder. readme gets just copied over every release.


Motor connection to board

Hi, I noticed that the motor that you are using, it has a ribbon cable instead of normal wires. Could you please tell me how it connects to the PCB board? Thanks.

OK thank!


WireMock Tests

Add more test cases.
one : analyse the code for possible test cases
two : add endpoints to wiremock class
three : add the test cases to the test classes

Also important:
the method that creates the mocks should be refractored into smaller methods!


Update tests for TableMath

The command tests use time series commands since copied from TSTool.  Need to udpate to use an alternative.

I fixed theses tests.  Close the issue.


Get started | rework day 1 customer experience

The purpose of this PR is to improve the customer "Day 1 experience" through landing page simplification, and day-1 docs freshness updates.

Merging as this PR does not require engineering review.


Add ssh to the php images

To fix the error error: cannot run ssh: No such file or directory occurring when trying to require a private repository via composer

Could we use ssh-client instead of ssh?

That seems to work yes, will do the update shortly.


Lazy select

This PR fixes https://github.com/CartoDB/support/issues/952.
A new lazy select component to manage the 40 rows per page request's limitation has been created. Basically, when the user fills the search input, a new request is made to grab all items that match with the value filled.

In order to make the acceptance, create a filter analysis with a string column:

[ ] Check that the select works properly. It mainly covers the same functionality that a [select.
[ ] Check that the searching works properly.
[ ] No request returns more than 40 rows.
[ ] it's important to check if the analysis itself works properly with this change.

cc @noguerol @ramiroaznar

could you please take a CR @rubenmoya?

Probably yes, I'll take a look at a possible refactor.

Anywhere in staging we can test it?

In ded11.

@rubenmoya even the code seems pretty similar, I found subtle differences, especially when working with the form model, that they will do that making a refactor turns into a really hard task. Also, the DO piece of code is kind of special, that's why the functionality we have here in one component, it's there in 3. Because of this, I think we should leave the refactor out of scope for now.

Then is better to keep it 3 similar separated components than in a complicated and hard to work with one.

Found a glitch in ded11:


Pressing back twice to exit from dashboard

Summary:
When on the Dashboard when back is pressed we exit from the application.We could add a feature that when the user presses the back button once a toast is displayed asking whether he wants to exit and then he needs to again press the back button.
Steps to reproduce:
Go to dashboard and press back button.
I would like to work on this issue.

Hi, I would like to work on this issue.


conditional requiredEnv

Hi
Is it possible to conditionally use requiredEnv?
I can provide the usecase.
C:.
├───.idea
└───releases
├───grafana
│   ├───transformers
│   └───values
├───influxdb
│   ├───1.1.9
│   │   ├───transformers
│   │   └───values
│   └───2.2.0
│       ├───transformers
│       └───values
└───telegraf
└───values
helmfile.yaml:
helmfiles:
  - path: releases/grafana/helmfile.yaml
    values:
      - grafana:
          enabled: {{ env "GRAFANA_ENABLED" }}
  - path: releases/influxdb/1.1.9/helmfile.yaml
    values:
      - influxdb:
          enabled: {{ env "INFLUXDB_ENABLED" }}
  - path: releases/influxdb/2.2.0/helmfile.yaml
    values:
      - influxdb2:
          enabled: {{ env "INFLUXDB2_ENABLED" }}
  - path: releases/telegraf/helmfile.yaml
    values:
      - telegraf:
          enabled: {{ env "TELEGRAF_ENABLED" }}

now helmfile.yaml of grafana:
{{$condition := "grafana.enabled"}}

releases:
  - name: grafana
    chart: helm-virtual/grafana
    condition: {{$condition}}
    namespace: {{ requiredEnv "GRAFANA_NAMESPACE"}}
    values:
      - values/values.yaml.gotmpl
      - env:
          LDAP_PASSWORD: {{ requiredEnv "GRAFANA_LDAP_PASSWORD" }}
    transformers:
      - transformers/route-op.yaml
      - transformers/route-content.yaml

and one helmfile of influxdb:
{{$condition := "influxdb.enabled"}}

releases:
  - name: influxdb
    chart: helm-virtual/influxdb
    version: 2.2.0
    condition: {{$condition}}
    namespace: {{ requiredEnv "INFLUXDB_NAMESPACE" }}
    values:
      - values/values.yaml.gotmpl
    transformers:
      - transformers/route-op.yaml
      - transformers/route-content.yaml

the point is that if i run the installation over main helmfile i have to provide all requiredEnvs in subhelmfiles even that the condition for those subhelmfiles are set to false hence not required. Is there a way how to correlate on conditions for enabled releases to provide required envs? Or the only way is to use env and make it just optional?
thx

@tomassatka Hey! Seems impossible.
The biggest blocker for me seems to be that you don't have access to each release's values in helmfile.yaml template. You only have access to helmfile-wide state values.
That's why something like the below doesn't work.
namespace: {{`{{ if .Values.influxdb.enabled }}{{ requiredEnv "INFLUXDB_NAMESPACE" }}{{end }}`}}

Perhaps we need some way to expose release values to the release template? If we name it .Release.Values, something like the below would become possible:
namespace: {{`{{ if .Release.Values.influxdb.enabled }}{{ requiredEnv "INFLUXDB_NAMESPACE" }}{{end }}`}}

Please feel free to try implementing this .Release.Values thing for review. I think it's a valid addition to helmfile.
Also, below is the relevant portion of code for the addition:
https://github.com/roboll/helmfile/blob/f502a0a779f7506dbf02598c392c1f10b716ffc4/pkg/state/state_exec_tmpl.go#L19

Hi @mumoshu
Thanks for your support.
Will try to build it. It will take me some time but once i have something for review I will create PR


useIsAuthenticated hook returns false on every refresh

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.14.1
Wrapper Library
MSAL React (@azure/msal-react)
Wrapper Library Version
1.0.0
Description
I try to use the useIsAuthenticated hook to see if I am logged in or not. If not, I render a login screen and clicking the button will call instance.loginRedirect(). I see the login screen on every fresh login (which is correct). I go through the login process and after navigating through the app, when I refresh the browser, I keep coming back to the login screen. Instead I expect to see the same page since I am already logged in.
MSAL Configuration
import {
  BrowserCacheLocation,
  Configuration,
  RedirectRequest,
} from '@azure/msal-browser';

const CLIENT_ID = process.env.REACT_APP_MSAL_CLIENT_ID || '';
const TENANT_ID = process.env.REACT_APP_MSAL_TENANT_ID || '';
const AUTHORITY = `https://login.microsoftonline.com/${TENANT_ID}`;

const REDIRECT_URI = `${window.location.protocol}//${window.location.host}/`;

/**
 * Config object to be passed to Msal on creation
 */
export const msalConfig: Configuration = {
  auth: {
    clientId: CLIENT_ID,
    authority: AUTHORITY,
    redirectUri: REDIRECT_URI,
    postLogoutRedirectUri: REDIRECT_URI,
  },
  cache: {
    cacheLocation: BrowserCacheLocation.LocalStorage,
    storeAuthStateInCookie: false,
  },
};

/**
 *  Add here scopes for access token to be used at MS Identity Platform endpoints.
 */
export const apiRequest: RedirectRequest = {
  scopes: [process.env.REACT_APP_API_SCOPE || ''],
};

/**
 *  Add here the endpoints for MS Graph API services you would like to use.
 */
export const graphConfig = {
  graphMeEndpoint: 'https://graph.microsoft.com/v1.0/me',
};

Relevant Code Snippets
const App: FunctionComponent = () => {
  const { instance, inProgress } = useMsal();
  const isAuthenticated = useIsAuthenticated();

  // useEffect(() => {
  //   instance
  //     .handleRedirectPromise()
  //     .then((authResult) => {
  //       // Check if user signed in
  //       const account = instance.getActiveAccount();
  //       if (!account) {
  //         // redirect anonymous user to login page
  //         instance.loginRedirect(apiRequest);
  //       }
  //     })
  //     .catch((err) => {
  //       // TODO: Handle errors
  //       console.log(err);
  //     });
  // }, [instance]);

  return (
    <ThemeProvider theme={theme}>
      {isAuthenticated ? (
        <ProfileComponent>
                <Routes />
        </ProfileComponent>
      ) : <Login /> }
    </ThemeProvider>
  );
};

Identity Provider
Azure AD / MSA
Source
External (Customer)

@a2441918 Can you please confirm that the MsalProvider component is rendered above the App component?
If so it's possible the login is not succeeding. Can you try calling handleRedirectPromise and see if it returns a result or throws an error?

@a2441918 Can you please confirm that the MsalProvider component is rendered above the App component?
If so it's possible the login is not succeeding. Can you try calling handleRedirectPromise and see if it returns a result or throws an error?

Yeah.
import './support/polyfills';
import './index.scss';

import React from 'react';
import ReactDOM from 'react-dom';

import {
  AuthenticationResult,
  EventMessage,
  EventType,
  PublicClientApplication,
} from '@azure/msal-browser';

import { MsalProvider } from '@azure/msal-react';

import App from './components/shared/App';
import { msalConfig } from './services/AuthConfig';

/**
 * Initialize a PublicClientApplication instance which is provided to the MsalProvider component
 * We recommend initializing this outside of your root component to ensure it is not re-initialized on re-renders
 */
export const msalInstance = new PublicClientApplication(msalConfig);

// Account selection logic is app dependent. Adjust as needed for different use cases.
const accounts = msalInstance.getAllAccounts();
if (accounts.length > 0) {
  msalInstance.setActiveAccount(accounts[0]);
}

msalInstance.addEventCallback((event: EventMessage) => {
  if (event.eventType === EventType.LOGIN_SUCCESS && event.payload) {
    const payload = event.payload as AuthenticationResult;
    const account = payload.account;
    msalInstance.setActiveAccount(account);
  }
});

ReactDOM.render(
  <MsalProvider instance={msalInstance}>
    <App />
  </MsalProvider>,
  document.getElementById('root')
);

Is it because I clear localStorage in index.ts?

This could definitely be it. What happens if you don't clear localStorage?

Is it because I clear localStorage in index.ts?

This could definitely be it. What happens if you don't clear localStorage?

If I dont clear it, it works great. The reason I clear localStorage is that I am accessing this app from a portal. The portal sends me a token and I store that token in localStorage and use that for API calls. There are cases where I can access this app directly instead through the portal. The existing token in localStorage is expired but still the expired token is being used as a Bearer token.

@a2441918 Can you clear just the token in question (I would also suggest adding an expiration check if this is an issue)? MSAL needs to maintain state between page loads so clearing all storage will definitely break MSAL.

Sure, yeah I'll do that. My other plan is to validate the token from the portal and if its a good one, I'll make a call to MS and get the token through msal and use this new token for Bearer auth. Is this a good approach?

If you have a valid token why do you need to get a new one? If you want to make sure all tokens are obtained by MSAL I'd suggest not passing the token in from the portal at all. But it's ultimately up to you and what makes sense for your app and the scenarios you need to support.

Got it. Thanks man. Appreciate the help.


Resizing paletted PNG to PNG returns non-sensical result

I have a PNG image with a palette (aka "indexed"?) that imgproxy returns gibberish for. The exact result depends on what size and fit I resize to, but none of them looks anything like the original image, usually I just get fully colored squares.
It needs to be both to PNG and the original PNG should be in the "indexed"/palette format to reproduce.
The PNG with palette

The result after imgproxy

My settings:

I tested locally with docker run -p 8080:8080 -it darthsim/imgproxy and docker run -p 8080:8080 -it darthsim/imgproxy:v3.8.0 and could reproduce on both. It was working for version docker run -p 8080:8080 -it darthsim/imgproxy:v3.7.2, so this is a recent regression!

I just want to clarify that this is not limited to just this png, but happens with other indexed pngs as well. For example:  https://upload.wikimedia.org/wikipedia/commons/e/e1/Screen_color_test_Amiga_4colors.png has the same issue.

Hey @DrPhil!
Thanks for the report! I patched libvips right in the v3.9.0 Docker image.

Well done! I've been able to test both locally and in production. This issue is resolved. :smiley: Thanks for great debugging and a speedy resolution. Well done!


Update getting-started-local-single-node.md

Wget link for macos

Thanks for the PR @yeshvantbhavnasi .   We can always use the extra help. Especially with documentation.


Small toolbar style fix

The toolbar sometimes shows it's scrollbar at max width if the first dropdown contains a long enough string #2848
Fixes the below:

We might need to consider nested dropdowns at this point 😁
Not sure if it's still the case, but some collab failed in the past as it wasn't able to click on right-most dropdown (as it's using split view to show two editors). In that case will need to increase playwright browser's start width to be maxWidth x2

We might need to consider nested dropdowns at this point 😁

@fantactuka Yeah, I'm starting to think we should have several smaller playgrounds with different examples than our one kitchen sink example. This is somtehing other editors do better.


notary: When deposit parameter till defaults?

docs say that till prm

Default till value is the current chain height + 5760.

i tried to make deposit with zero till and got till shouldn't be less then the chain's height panic.
i've found this place of code where panic occurs, it prevents prm defaulting, isn't it?
https://github.com/nspcc-dev/neo-go/blob/f0a5430b5b42440ffc09e06a4505de714e366650/pkg/core/native/notary.go#L235-L237
i've also caught place where value is reset. According to this, till defaults on first self-deposit
https://github.com/nspcc-dev/neo-go/blob/f0a5430b5b42440ffc09e06a4505de714e366650/pkg/core/native/notary.go#L249-L251
Please review code/docs and clarify default till behavior

I'd keep the behavior as is and fix the documentation. Rationale: if you're the owner, you're responsible for your till value. When there is some data, it inevitably has both the to address and till. Address can be nil, till can't, so set some value you want.
The other option (or extention maybe) though is using this default if args[2] == nil, so if one doesn't set any data at all (and we just error out in this case now) then we can assume to == nil and this default till value, we have nothing else anyway. But this is an incompatible change and I'm not sure we don't have any transactions transferring GAS with data == nil, it might require some attention/hardfork.
@AnnaShaleva?

The code works properly, as Roman said, it's prohibited to set Till if you're not the deposit owner. And if you're the owner, then you must set some valid value. I'll update the documentation, it's probably not as clear as expected.

The other option...

I'd prefer to keep the code as is because the owner should be aware of the exact value when deposit is unlocked. And if it's not set, then it's better to let the user know that something is wrong with the deposit. Although if there's a strong demand to set the default till value, then I'd prefer to still treat args[2] == nil case as an error and allow to set args[2][1] (i.e. till) to stackitem.Null value like it is done with args[2][0] (to) value.


Unhandled exception. System.FormatException: Input string was not in a correct format.

Unhandled exception. System.FormatException: Input string was not in a correct format.
at System.Text.ValueStringBuilder.ThrowFormatError()
at System.Text.ValueStringBuilder.AppendFormatHelper(IFormatProvider provider, String format, ParamsArray args)
at System.String.FormatHelper(IFormatProvider provider, String format, ParamsArray args)
at System.String.Format(String format, Object[] args)
at Microsoft.WingetCreateCLI.Logging.Logger.Log(LogLevel level, String format, Object[] args)
at Microsoft.WingetCreateCLI.Logging.Logger.Error(String message, Object[] args)
at Microsoft.WingetCreateCLI.Program.Main(String[] args)
at Microsoft.WingetCreateCLI.Program.(String[] args)

Hey @IceBreaker2451,
Thanks for catching this bug. This is a known issue and we are currently working on resolving this bug. It is hard to tell what the issue is from just the error message. Could you list out what steps you took to get to this state? It would be helpful to know what commands you executed as well as maybe what manifest you were trying to submit, that way I can take a look into this further.

I am experiencing the same issue when trying to run winget-create from a github action workflow. The workflow file can be found here if it helps in debugging.

I suspect what's happening here is that your URL is not getting correctly resolved in your pipeline, leading to the url getting passed to the tool containing a format string, for example: https://foo.com/{version}.exe
Then in the Logger.Error call, we call string.Format() and that throws the above exception


Terragrunt not respecting the provider.tf variables

I have an odd issue.  I have this root terragrunt.hcl:
# ---------------------------------------------------------------------------------------------------------------------
# TERRAGRUNT CONFIGURATION
# Terragrunt is a thin wrapper for Terraform/OpenTofu that provides extra tools for working with multiple modules,
# remote state, and locking: https://github.com/gruntwork-io/terragrunt
# ---------------------------------------------------------------------------------------------------------------------

locals {
  # Automatically load domain-level variables
  domain_vars = read_terragrunt_config(find_in_parent_folders("domain.hcl"))

  # Automatically load region-level variables
  region_vars = read_terragrunt_config(find_in_parent_folders("region.hcl"))

  # Automatically load tenant-level variables
  project_vars = read_terragrunt_config(find_in_parent_folders("project.hcl"))

  # Automatically load common variables
  common_vars = read_terragrunt_config(find_in_parent_folders("common.hcl"))

  # Extract the variables we need for each access - corrected to use .locals
  auth_url          = local.common_vars.locals.os_auth_url
  tenant_id         = local.project_vars.locals.os_project_id
  tenant_name       = local.project_vars.locals.os_project_name
  user_domain_id    = local.domain_vars.locals.os_user_domain_id
  user_domain_name  = local.domain_vars.locals.os_user_domain_name
  project_domain_id = local.domain_vars.locals.os_project_domain_id
  region            = local.region_vars.locals.os_region_name
}

# Generate an Openstack provider block
generate "provider" {
  path      = "provider.tf"
  if_exists = "overwrite"
  contents  = <<EOF
terraform {
  required_version = ">= 1.6.0"
  required_providers {
    openstack = {
      source  = "terraform-provider-openstack/openstack"
      version = "~> 1.48.0"
    }
  }
}

provider "openstack" {
  auth_url            = "${local.auth_url}"
  tenant_id           = "${local.tenant_id}"
  tenant_name         = "${local.tenant_name}"
  user_domain_name    = "${local.user_domain_name}"
  project_domain_id   = "${local.project_domain_id}"
  region              = "${local.region}"
  enable_logging      = true
  user_name           = sensitive(coalesce(local.os_username, "${get_env("OS_USERNAME", "")}"))
  password            = sensitive(coalesce(local.os_password, "${get_env("OS_PASSWORD", "")}"))
}
EOF
}

I have a module down a couple of levels:
# terragrunt.hcl
include "root" {
  path = find_in_parent_folders()
}

terraform {
  source = "git::ssh://GITHUB/infrastructure-modules.git//composite/project-network?ref=v0.0.1"
}

inputs = {
  module_version = "v0.0.1"
  network_name = "template-project-network"
  subnet_cidr = "<IP_ADDRESS>/24"
  external_network_id = "12345"

  security_group_rules = [
    {
      direction        = "ingress"
      ethertype       = "IPv4"
      protocol        = "tcp"
      port_range_min  = 22
      port_range_max  = 22
      remote_ip_prefix = "<IP_ADDRESS>/0"
      remote_group_id = null
    }
  ]
}

When I run the terragrunt apply, it will create these resources, but in the wrong tenant, the tenant that is default to the keystone user I'm using.
The provider.tf being generated in .terragrunt-cache is correct:
# Generated by Terragrunt. Sig: nIlQXj57tbuaRZEa
terraform {
  required_version = ">= 1.6.0"
  required_providers {
    openstack = {
      source  = "terraform-provider-openstack/openstack"
      version = "~> 1.48.0"
    }
  }
}

provider "openstack" {
  auth_url            = "https://keystone.TEST.com:5000/v3"
  tenant_id           = "5aaaaaaaaaa"
  tenant_name         = "template-project"
  user_domain_name    = "admin_domain"
  project_domain_id   = "caaaaaaaaa"
  region              = "us1"
  enable_logging      = true
  user_name           = sensitive(coalesce(local.os_username, "admin"))
  password            = sensitive(coalesce(local.os_password, "aaaaaaa11112222"))

The only way I can get this to work with terragrunt is if I set the OS_PROJECT_ID="5aaaaaaaaaa" via export.  Then it works as expected but that is not an option since this is something generated by a project.hcl file in the project level.
Interesting enough, I can go into the .terragrunt-cache directory where all the files are generated, backend.tf, main.tf, outputs.tf, provider.tf and variables.tf and using this, I can run a terraform apply or a tofu apply and the resources will be created in the correct tenant.
I've tried literally everything I know to troubleshoot this and tried very different version of terragrunt to see if this is the issue but no luck.
Looking for any insight.  Thanks!

Running terragrunt like this works:
OS_PROJECT_ID=5aaaaaaaaaa terragrunt apply

I've also removed  the following from the provider and it still works because it uses the users default tenant.  So it's as if these aren't recognized unless set via ENV.
tenant_id           = "${local.tenant_id}"
tenant_name         = "${local.tenant_name}"

I found a workaround but this seems like it should work without this by default:
terraform {

  extra_arguments "project_vars" {
    commands = ["init", "plan", "apply", "destroy"]
    env_vars = {
      OS_PROJECT_ID = local.project_vars.locals.os_project_id
    }
  }
...

Hi,
it looks strange,
can you confirm that there is no OS_PROJECT_ID environment variable set when you run terraform apply in the .terragrunt-cache directory?

Hey @cheops82 ,
To avoid having this go stale, can we either close out this issue or continue the discussion?


[BUG] Model output keeps repeating itself, after certain input/max token combination was executed.

Feel free to ask any kind of questions in the issues page, but please use English since other users may find your questions valuable.
Describe the bug
When certain audio and max tokens settings were used to generate the first audio, all subsequent generations will keep using the inputs of the first generation no matter what is given to the model.
To Reproduce
Steps to reproduce the behavior:
See video below:
https://www.awesomescreenshot.com/video/23829885?key=2c741f2c20efe2939f38085c73b3d740
Expected behavior
A clear and concise description of what you expected to happen.
Subsequent generations should generate what is given as the input, not what is given in the first generation.
Screenshots / log
If applicable, add screenshots / logs to help explain your problem.
https://www.awesomescreenshot.com/video/23829885?key=2c741f2c20efe2939f38085c73b3d740
Additional context
Created a PR to fix it. Possibly related setup_caches, reason unknown, maybe the repo owner can help investigate.

This seems to be a webui issue, can you test calling the http api directly to see if that happens?

Same thing with HTTP API.

Do you mind to share your ref files? I will test it later.

Sure, GitHub doesn't seem to allow audio files to be uploaded, I've converted it into an mp4.
https://github.com/fishaudio/fish-speech/assets/97192387/80ebfad4-7014-4233-8749-4fc808dd7aa4

Prompt text: "今天去哪里玩好呢，咱们一起去海边吧，感受一下人工智能的清新，和阳光的温暖怎么样。"
Max tokens: 2000

Are you using npy path for http api?

i only tried .wav path

this seemed to be an issue with reallocating the kv cache.

Have encountered the same problem, is there a solution?

After changing max_new_tokens from 1024 to 4096 this issue did not occur again ~


chore: enable debug symbols and clean up chatty output

This PR tweaks the compilation profile for our "default" debug builds so that they actually include debug symbols. This helps when you want to use rust-gdb on a buck2 build binary or running process (they work perfectly now, including source lines!)
I also turned down the level for two chatty lines in the cyclone server and the raw telemetry settings, which make the test output usable again.

bors merge


[S3]Add Eventbridge notification for object storage class change

send eventbridge notification when object's storage class changes
https://github.com/getmoto/moto/issues/7363

Hey @bblommers!
I'm trying to implement eventbridge notification for object storage class change based on previous PR
any tips on how I should implement it? I'm thinking adding getter and setter is not enough, as we change object's storage class by creating a copy...

Hi @MikiPWata!
If I'm reading the documentation right, there are three scenario's where the StorageClass is set in AWS:

When creating an object
When copying an object (and the StorageClass-argument is set)
When a LifeCycle rule kicks in

We can ignore 1, because that's not a change. Number 3 is irrelevant, because we don't act on the lifecycle rules.
So we only have to worry about number 2, the copy operation - and because that's not yet implemented, we have full control on how we do that. Am I correct so far, or have a missed a scenario?
If it's just the copy that we have to worry about, than it makes sense to use a getter/setter pattern IMO. We can use the setter during object creation as well, to check that the StorageClass is valid - as that doesn't happen atm.

Hey @bblommers!
Thanks for your comment!
I think you are right, we just need to worry about the copy operation.
I tried implementing getter and setter, but the following error comes up from all test cases for S3's Eventbridge notification.
FAILED tests/test_s3/test_s3_eventbridge_integration.py::test_delete_object_notification - botocore.errorfactory.BucketAlreadyOwnedByYou: An error occurred (BucketAlreadyOwnedByYou) when calling the CreateBucket operation: Your previous request to create the named bucket succeeded and you already own it.
As I'm not that proficient in getters and setters, would love to get some advice on that.
thanks in advance

@bblommers
This PR currently contains the changes I made. Can you take a look at it?
There's no issue when I rollback the storage_class getter and setter, so I'm pretty sure that my implementation is wrong.

Ah, I understand @MikiPWata. The indentation of both the getter and setter is wrong, I guess that broke things in a weird way. If you indent both methods to make it part of the class, the other tests do pass.


first cut data layer

Topic: data-layer

Reviews in this chain:
└https://github.com/andrewspinks/revup/pull/4 first cut data layer

#
head
base
diff
date
summary

0
55d76d30
6719a476
diff
Feb 15 21:37 PM
1 file changed, 1 insertion(+)


Subsequent columns sometimes "appear" not to be marked

With this addon comes the feature that columns will be automatically "marked" so the user needs but to type in order to change any given value (instead of deleting it manually).  I noticed that occasionally the next value "appears" not to be highlighted, although it actually is. I tried to capture this hiere:

I initiated the edit mode by "directly typing in a new value" (the 'wasd' in the picture). The second (and sometimes even the third) value will usually not be marked, although they are.
This might be confusing for some.

I can't promise this will ever work, due to the hacky nature of this extension - we're racing the Grid for events as it is. I'll take a look and try to figure out why this happens, though.

I have not seen this behavior in latest master

New version released - please reopen the issue if you find it persists.


Coursier plugin update

Fixes #.
We are running into few errors while running sbt:verify through our CI/CD. This PR contains a change to upgrade Coursier sbt build plugin  which will fix the OverlappingFileLockException. Copying the stacktrace below for your reference
Changes in this pull request:

Update sbt-coursier to latest  2.0.0-RC4-1
(Coursier Docs)[https://get-coursier.io/docs/sbt-coursier]

Downloaded https://repo1.maven.org/maven2/com/ironcorelabs/cats-scalatest_2.12/2.3.1/
[error] coursier.ResolutionException: Exception during resolution
[error] Caused by: java.nio.channels.OverlappingFileLockException
[error] 	at sun.nio.ch.SharedFileLockTable.checkList(FileLockTable.java:255)
``

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


CA1416: is only supported on: 'windows' 5.1.2600 and later

Using
_ = PInvoke.CreateHardLink(pcwstr_pHardlink_filename, pcwstr_pPhoto_filename, null);

and getting the following warning:
Warning	CA1416	This call site is reachable on: 'windows' all versions. 'PInvoke.CreateHardLink(PCWSTR, PCWSTR, SECURITY_ATTRIBUTES*)' is only supported on: 'windows' 5.1.2600 and later.

Context

CsWin32 version: [e.g. 0.4.422-beta]: 0.3.49-beta
Win32Metadata version (if explicitly set by project):
Target Framework: [e.g. netstandard2.0]: .NET 8
LangVersion (if explicitly set by project): [e.g. 9]: 12

That looks like it's working as expected. Have you looked at the docs for CA1416 to see how to resolve it?

@AArnott, thanks. It seems I left out the part about what I've tried. I've added this to the method, but it doesn't have the desired effect of "muting" that warning.
[System.Runtime.Versioning.SupportedOSPlatform("windows")]

I had other CA1416 warnings and the above worked on those except for ones for PInvoke.CreateHardLink()

Your attribute doesn't mention what version of Windows the method requires. The warning you're seeing mentions specifically that the API is only available on 5.1.2600 and later, so your attribute needs to carry the same version requirement in order to suppress that warning.
Of course, that may have the effect of moving the warning from the CsWin32 call to the call of your own method. That might be good, if your callers will include runtime version checks (which should suppress the warning automatically) before calling your method. Or I believe if you just change your TargetFramework property from windows to windows5.1.2600 (or whatever the syntax is, then that effectively raises the water-level of your whole project to assume that your minimum version requirement has been met, thereby removing all warnings.


Set container in directive

If I have an image in a second page your code doesn't work because locates for 'window' or 'container' in the main page.
... in config code...
scrollable = document.querySelector('#scrollable')
container: angular.element(scrollable)

Pull requests are welcome : )

+1 Anyone have solution ?

Tomorrow I start my vacation, then I'll jump into this issue : )

I added a directive "lazyImgContainer" which when added to some element as attribute will trigger on scroll "checkImages" the same way as window (or container if set in provider)

Nice, i will test it tomorrow and provide some info :D

How is it working exactly ? I have div with style like and added directive lazy-img-container:
overflow-y:scroll
height:400px
and in there ng-repeat with div where inside are images, but its not working

Doesn't work for me either. Tried passing container inside the options object and also attaching lazy-img-container to the scrollable element.
I think this should be reopened :)

Bower registry has an old version of this module: https://github.com/Pentiado/angular-lazy-img/issues/8
bower install --save angular-lazy-img#fa48333273f9e

or
bower install --save angular-lazy-img#master

will work for now.

Yes, sorry guys I forgot about publishing tag. Now it should be working.


Replace deprecated isAccessible with canAccess

In file https://github.com/VirtusLab/ide-probe/blob/master/core/probePlugin/src/main/scala/org/virtuslab/ideprobe/handlers/IntelliJApi.scala
Replace usage of java.lang.reflect.AccessibleObject.isAccessible (deprecated) with java.lang.reflect.AccessibleObject.canAccess(Object obj). When change is done - remove the @nowarn annotation as well.

Ok, but the actual problem is that:

canAccess is available since JDK 9
isAccessible is deprecated since JDK 9
So, for now - it would be better to leave it as it is, using the isAccessible method. If we would switch to canAccess - there would be errors for jdk 1.8.


Update 22.02 changelog to latest [skip ci]

As the title
Signed-off-by: Tim Liu<EMAIL_ADDRESS>

build

can you regenerate to include https://github.com/NVIDIA/spark-rapids/pull/4771, this PR forget to add project. Just added

build


Update nist-sp-800-53-r5.md

Enterprises are getting caught up in trying to meet the MS version of these controls and we need to help them understand that it is OK and encouraged to tailor this approach to meet their needs.

@timwarner-msft - can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


avoid manual parsing in GraphHopperWeb

Make it possible to just write:
GHResponse res = objectMapper.readValue(str, GHResponse.class);
and this will also simplify or remove the code in SimpleRouteSerializer.
(We need further custom serialize code for the polyline encoding stuff)

What is the general plan for the client? Elsewhere I was suggesting to remove it, but only because it's duplicated in another project.
In general, it can of course be a good idea to have the (same-language) client in the same repository, precisely because serialization code and representations can be shared. Dropwizard recommends it. It should go into a different JAR, of course, so rather than being in "web", it should be in a "client" module. Client code can then include the client JAR and that's it..
So maybe we move the "hand-coded" client over here? In general?

Yes, this might be a good solution to move the hand-coded into a new client maven module. The only problem is that we would either have to split the hand-coded matrix part and keep it externally or put it here, which would mix the open source with the API which I would like to avoid to reduce confusion.

which would mix the open source with the API which I would like to avoid to reduce confusion.

Hmmh, this is kind of 'violated' already as the GraphHopperWeb points to the commercial offering per default. So moving the hc client into this repo sounds good to me and we can finally avoid duplicate code.

Related: What is the plan for the Swagger spec? Is it just a side-project, or do we want to make it more authoritative by reviewing it, adding documentation and perhaps eventually generate the official API docs from it?

Is it just a side-project

Definitely not. Why do you think so?

Related: What is the plan for the Swagger spec?

The swagger spec and clients generated are already working. Also there is a bit documentation for this already inside the spec. But a few bigger issues remain (limitation of either the codegen or swagger spec itself) like https://github.com/graphhopper/directions-api-clients/issues/1

Definitely not. Why do you think so?

Just asking. For example, it does not generate the official API documentation, but duplicates it.

So moving the hc client into this repo sounds good to me and we can finally avoid duplicate code.

I could do that on the Dropwizard-PR. Would fit nicely there because of close connections to "doing more with Jackson" and "sharing representations".

I could do that on the Dropwizard-PR. Would fit nicely there because of close connections to "doing more with Jackson" and "sharing representations".

I'm more for smaller PRs :) maybe a second PR based on the Dropwizard branch?

Okay, sure.

I thought the approach would be the other way around? Move the GraphHopperWeb file into the hc part and use this as a test dependency? The GraphHopperWeb is only used for testing purposes.

The problem then will be that we have a circular dependency of the repositories: test web module (GH repo) -> hc client (other repo) -> core (GH repo) which might be too complex (?)

Depndencies aside, the argument for having client and server in the same repository is that they can (easily) share the same representation objects and serialization/deserialization code.
What are the arguments for having it in a separate repository?
"A user may only want the client" -- okay, but they will usually use a Maven dependency, not check out the source code.

Note: I also really like the Swagger stuff and auto-generated clients, but I think those are a different story, and would perhaps be better off if they are in their repository alone, without the "hand-coded" stuff. The whole repository could basically be auto-generated by Swagger.

What are the arguments for having it in a separate repository?

Well it depends, if we put the client into it's own package, I don't see a good reason against this, besides some users might be a bit confused that suddenly the location of the client changed. If we leave the client (GraphHopperWeb) in the web client the downside is that anybody using the client would end up with all the dependencies of the web module.

I also really like the Swagger stuff and auto-generated clients, but I think those are a different story, and would perhaps be better off if they were in their repository alone, without the "hand-coded" stuff. That whole repository could basically be auto-generated by Swagger.

Yes, I am fine with that. We should also note that right now the hc client cannot solve all functions of the API (e.g. geocoding).

the downside is that anybody using the client would end up with all the dependencies of the web module.

Agreed that wouldn't be nice.

Agreed that wouldn't be nice.

Yeah, it would go in its own module, of course.


update ci to push with tags

I suppose we cannot test your PR until after we merge it and create a new tag? Or would creating a tag on your branch trigger the workflow?

on dockerhub
https://hub.docker.com/r/mlexchange/mlx-ui/tags

Great! Thanks @Tomcli


chore(4.5): release 1.8.8

:robot: I have created a release beep boop
1.8.8 (2023-03-06)
Bug Fixes

add aws envs and unquote aws config (7e40c38)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/GluuFederation/cloud-native-edition/releases/tag/v1.8.8 :sunflower:


Issue clickable area

Would potentially be nice for the area to open the PR was the entire of the card instead of just the title.

As in a screen that lets you see the entire PR? I did think about that but I don't want to get carried away and recreate the entire GitHub PR UI 😆 Or do you mean something else?

No, I mean to extend the clickable area so instead of just clicking the title to open the PR I could click anywhere in the card?

Oooh I see haha. That should be easy enough to do

Although there are other clickable items in the card so not sure if that will work correctly?

Made a bit of an adjustment in 0e45f60
made the clickable area a bit bigger but still around the title, you don't have to be quite as precise


book

Hi @ryan-ally. Thanks for working on our repo.
We are going to upgrade some packages in all of the book codebases soon, so we will have updated package.json and yarn.lock files, which I see you have submitted here.

Your welcome =]


Unable to change kernel.core_pattern with --privileged

Hi,  trying to setup core dump files. --privileged with "docker exec" or "docker-compose exec" both run into read-only issue for me. Searched a lot on this, everyone gives example with docker run. I assume --privileged with RUN and EXEC should work the same.
$ docker exec --privileged -it CONTAINER sysctl -w kernel.core_pattern=/tmp
sysctl: error setting key 'kernel.core_pattern': Read-only file system

I am running docker desktop on mac
version: <IP_ADDRESS>
Any idea please

For support, please try the Docker Community Slack or forums:

https://dockercommunity.slack.com/
https://forums.docker.com/

If there's an issue in the documentation, please let us know.


Nvidia support - Update configuration to accommodate switch to FSDP

Blocked on:

[ ] https://github.com/instructlab/training/issues/292

I think @jaideepr97 has already done this with https://github.com/instructlab/instructlab/pull/2323? The issue is a little vague tbh so not sure. cc @cdoern


a lot of dependency is outdated  can this project be updated

this project works really well    but a lot of dependency is outdated Can you update it

Every once in a while I update dependencies, and so I will soon when I get some free time. But I would also appreciate a PR of course :D

Hey all dependencies are up to date with latest versions in the new release 1.8.0 :)


Fix the issue of incorrectly calculating the number of buffer pretty hex dump rows

Motivation:
Fix the issue of incorrectly calculating the number of dump rows when using prettyHexDumpmethod in ByteBufUtil. The way to find the remainder is either length % 16 or length & 15
Modification:
Fixed the way to calculate the remainder
Result:
Fixed #9301

@netty-bot test this please

@qeesung thanks lot!


CLI spl-token update-metadata doesn't follow expected JSON format

If this isn't the right place to report CLI bugs, then please copy the right person. The problem is that this command and subcommand are supposed to insert new fields and values within additionalMetadata of a Token-2022 structure on chain. For example:
"additionalMetadata": []
can be expanded to:
"additionalMetadata": [
[
"animal": "horse"
]
]
using the above command. However, what actually happens is this:
"additionalMetadata": [
[
"animal",
"horse"
]
]
Thus both the format (comma rather than colon) and the display (skipped line rather than not) of the JSON are inconsistent with the usual "field: value" format. It's acting as though "animal" and "horse" are equivalent items in a list, instead of a field and its value. As a result, downstream consumers can't parse it correctly.

On second thought, it could be just a display problem with "spl-token display", in which case the downstream issues are due to other causes. It's hard to tell because while that command shows commas, and solscan.io shows the same, solana.fm shows colons.

@SolBugz The Token Metadata additional metadata is one field with a Vec<(String, String)>. To put this in JavaScript terms (closer to JSON) it looks like: [string, string][].
So the JSON you've pasted here appears correct. It's a list of tuples [string, string]. ie:
{
  "additionalMetadata": [
    ["animal", "horse"],
    ["plant", "daisy"],
    ["rock", "granite"]
  ]
}

@buffalojoec Thanks but I'm not sure what "dynamic for" means in this context. You can have the exact same dynamically-sized, updateable structure in your example with colons instead of commas. They wouldn't be Vecs in the Rust sense but the format would be consistent with all the metadata already out there.
This might explain why it seems that no explorer or wallet out there currently picks up "image" when it resides in "additionalMetadata". This is problematic because it forces the existence of off-chain metadata, at least for tokens with images, when in fact Token-2022 would allow one to implement an SPL token entirely on-chain.
I suppose it's too late to change this (unless comma-vs-colon is left undefined in the Token-2022 spec). If that's the case then it's an awareness problem on the part of all the explorers and wallets out there, both in terms of the punctuation and also that "additionalMetadata" should be imported just like off-chain metadata.


Parameter 'x' cannot be referenced in its initializer.

nightly build

function fn<a>(y: Y, set: (y: Y, x: number) => void): a {
    return undefined;
}
interface Y { x: number }

class C {
    constructor(
        y: Y,
        public x = fn(y, (y, x) => y.x = x) // expected to work, but actually doesn't
    ) {
    }
}

(tested on 1.9.0-dev.20160509)
An attempt to simplify and isolate the problem:
function Example(x = function(x: any) { return x; }) { // Error: parameter 'x' cannot be
												       // referenced in its initializer
}

Seems like the problem may be that 'x', in the body of the anonymous function, is seen as referencing the outside function ("Example") parameter instead of its own parameter.


How to flip the health bar?

Describe your problem here.
Is there any way I can make the health bar flipped? In other words, where the player's icon is on the left and opponent's icon on the right? If so, tell me. Thanks!
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No

duplicate, use the search feature before opening an issue #10100


Error: 'POST param “code” doesn’t exist?

Getting an error when saving a feed entry with the latest craft build (Craft CMS 2.6.2930). Here is the error from craft.log (changed paths to hide server details). I'll try and track it down, but any ideas what might be causing this? Thanks — Douglas
******************************************************************************************************
2016/09/08 05:50:19 [error] [exception.Craft\HttpException.400] exception 'Craft\HttpException' with message 'POST param “code” doesn’t exist.' in /path_to_domain//craft/app/services/HttpRequestService.php:509
Stack trace:
#0 /path_to_domain//craft/app/controllers/UsersController.php(336): Craft\HttpRequestService->getRequiredPost('code')
#1 /path_to_domain//craft/app/framework/web/actions/CInlineAction.php(49): Craft\UsersController->actionSetPassword()
#2 /path_to_domain//craft/app/framework/web/CController.php(308): CInlineAction->runWithParams(Array)
#3 /path_to_domain//craft/app/framework/web/CController.php(286): CController->runAction(Object(CInlineAction))
#4 /path_to_domain//craft/app/framework/web/CController.php(265): CController->runActionWithFilters(Object(CInlineAction), Array)
#5 /path_to_domain//craft/app/framework/web/CWebApplication.php(282): CController->run('setpassword')
#6 /path_to_domain//craft/app/etc/web/WebApp.php(818): CWebApplication->runController('users/setpasswo...')
#7 /path_to_domain//craft/app/etc/web/WebApp.php(288): Craft\WebApp->_processActionRequest()
#8 /path_to_domain//craft/app/framework/base/CApplication.php(185): Craft\WebApp->processRequest()
#9 /path_to_domain//craft/app/index.php(62): CApplication->run()
#10 /path_to_domain//html/index.php(30): require_once('/nfs/c07/h05/mn...')
#11 {main}
REQUEST_URI=/admin/feedme/feeds/5
HTTP_REFERER=http://domain.org/admin/feedme/feeds/5
---

******************************************************************************************************

This seems to be triggered by the actionSetPassword controller? There are no references to getRequiredPost('code') in Feed Me. Does this only happen on the Feed Me pages?

So far. Yes. Entries save fine. And haven’t experienced any other issues. Although haven’t tested much with the latest update. — Douglas

On Sep 7, 2016, at 11:36 PM, Josh Crawford<EMAIL_ADDRESS>wrote:
This seems to be triggered by the actionSetPassword controller? There are no references to getRequiredPost('code') in Feed Me. Does this only happen on the Feed Me pages?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/engram-design/FeedMe/issues/91#issuecomment-245506986, or mute the thread https://github.com/notifications/unsubscribe-auth/AIXOltKIw37LU153Jut6dibo4aJQK7jHks5qn6ztgaJpZM4J3pEV.

hmmm… just tested local and not getting the error. Only on the server. Maybe a server issue? Either php version incompatibility? Or not all the app files uploaded properly? Let me do some more testing… I’ll let you no results regardless. Thanks for your help. — Douglas

On Sep 7, 2016, at 11:38 PM, Douglas McDonald<EMAIL_ADDRESS>wrote:
So far. Yes. Entries save fine. And haven’t experienced any other issues. Although haven’t tested much with the latest update. — Douglas

On Sep 7, 2016, at 11:36 PM, Josh Crawford<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
This seems to be triggered by the actionSetPassword controller? There are no references to getRequiredPost('code') in Feed Me. Does this only happen on the Feed Me pages?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/engram-design/FeedMe/issues/91#issuecomment-245506986, or mute the thread https://github.com/notifications/unsubscribe-auth/AIXOltKIw37LU153Jut6dibo4aJQK7jHks5qn6ztgaJpZM4J3pEV.

Correction: I’ll let you know results regardless.

On Sep 7, 2016, at 11:42 PM, Douglas McDonald<EMAIL_ADDRESS>wrote:
I’ll let you no results regardless

I think I may have discovered the issue — seems the javascript and css files did not update properly when committed to the server due to Issue 32 (https://github.com/engram-design/FeedMe/issues/32). I deleted, committed, reinstalled, and committed again. All better. I should have suspected as I did notice some layout issues in the footer, and interaction issues with the select dropdowns. Thanks again for your help. — Douglas

Ah, yes that issue creeps up again. Wish I'd known about that back then!


Interface packet counter deviation

This change add deviation functionality for ipv4/ipv6 discard packet counters. By default it checks for discard counters and the test will fail if paths are not supported.
To pass the test, add deviation (deviation_subinterface_counter_supported) - that will skip the discard counter lookup

Duplicate : https://github.com/openconfig/featureprofiles/pull/372


Issue 1072 rework show query

This breaks down the big "show route" query into individual queries with more targeted selections. The performance improvement is significant for "normal" packages like Alamofire and dramatic for packages with many libraries:
┌──────────────────────────────┐   ┌──────────────────────────────┐
│                              │   │                              │
│2.53.3                        │   │this branch:                  │
│                              │   │                              │
│Alamofire-package             │   │Alamofire-package             │
│-----------------             │   │-----------------             │
│Count:     100                │   │Count:     100                │
│Average:   0.121s             │   │Average:   0.066s             │
│Median:    0.119s             │   │Median:    0.065s             │
│Min:       0.112s             │   │Min:       0.062s             │
│Max:       0.169s             │   │Max:       0.082s             │
│Std dev:   0.008s             │   │Std dev:   0.003s             │
│90% Pctl:  0.127s             │   │90% Pctl:  0.070s             │
│                              │   │                              │
│soto-package                  │   │soto-package                  │
│------------                  │   │------------                  │
│Count:     100                │   │Count:     100                │
│Average:   2.652s             │   │Average:   0.096s             │
│Median:    2.647s             │   │Median:    0.096s             │
│Min:       2.606s             │   │Min:       0.091s             │
│Max:       2.751s             │   │Max:       0.110s             │
│Std dev:   0.028s             │   │Std dev:   0.003s             │
│90% Pctl:  2.689s             │   │90% Pctl:  0.098s             │
│                              │   │                              │
│                              │   │                              │
└──────────────────────────────┘   └──────────────────────────────┘

This is probably very hard to review at the code level. I've had to move lots of things around and there was no way to break this down into steps and keep it building/passing tests.
There is some chronology of the changes in the sequence of commits. I.e. it should be possible to follow the journey by looking at the steps as they each touch a part of the overall changes pretty much in isolation.
The change was functionally complete with commit f5f393bbafb40340e3721e2f50b115363eb2b8a3 when the tests all passed again. After that I removed code that was now unused as well as all the warning/deprecation reminders I'd left for myself.
The only functional change I've made was to require a default branch version for packages (see the comment in test_show). I believe this could be lifted but I think it'd be non-trivial undertaking and something I'd like to avoid.
It's probably easiest to simply run this branch locally and compare some packages to prod. Thanks to the test suite there shouldn't be any nasty surprises :)
Fixes #1072

Thanks for the tick! I’ll merge this now so we can let this run a bit on staging before actually deploying it.


Use libvirt-go-xml for marshalling/unmarshalling libvirt's XML

We could use https://github.com/libvirt/libvirt-go-xml for doing all the marshalling/unmarshalling of libvirt's XML.

I can have a look at that.

Sure, I started to look into that for fun. There are some things that are not exposed by the xml bindings, I fear because they are not really needed to create an object.
Spoiler alert: this is not going to be fun :)

Assigning to @monstermunchkin who is already working on that.

Ops, I spoke too early... @dmacvicar can you add @monstermunchkin as a contributor for this repository?


#1: add support for horses in database initialization

added 'horse'

/paulo - can you please review this pull request?

/paulo - can you check now?


fix: conProf Profile URL

The issue is:
Two different profiling results of one component have same Flame Graph on Speedscope, this caused by the same issue, which had been fixed here.

@shhdgit Please organize a release. Thanks!


Update component with comments like StateObject

Update with jsdoc comments

[x] Component
[x] AbstractApp
[x] App

Completed in https://github.com/RoundingWellOS/marionette.toolkit/pull/29


(DEVELOPMENT_GUIDE): fix typo for input and output file parameter

(DEVELOPMENT_GUIDE): change --input-file to --template-file
(DEVELOPMENT_GUIDE): change --output-file to --output-template

Codecov Report

Merging #902 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #902      +/-   ##
==========================================
- Coverage   94.71%   94.62%   -0.09%
==========================================
  Files          69       69
  Lines        3047     2997      -50
  Branches      567      554      -13
==========================================
- Hits         2886     2836      -50
  Misses         85       85
  Partials       76       76

Impacted Files
Coverage Δ

...el/preferences/deployment_preference_collection.py
88.88% <0%> (-4.92%)
:arrow_down:

Continue to review full report at Codecov.

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

@lestephane Thanks so much for fixing this!


The Road to Charterdom

I thought it would be helpful to create an issue to track what's still needed to get CommComm chartered by the board. It's probably too late to get any updates before the board meeting next Monday, but maybe we could target having an updated revision to present at the meeting after that?
My understanding is that there was some feedback from the board in a previous meeting, and that we still need to update the charter with that feedback. Is that correct @hackygolucky?

we don't need this issue as the charter was approved by the board.

lol @ashleygwilliams that seemed abrupt 😖
Indeed @nebrius, it was completed via email.

Oh interesting, I didn't know we were approved, but yay! 🎉

Is the charter somewhere public? Should it be in the repo here, like the TSC has it?

It is here:
https://github.com/nodejs/community-committee/blob/master/GOVERNANCE.md
I hadn't realized it was called "GOVERNANCE" till this moment 🤔


Added hds-le.de

@WichtigesYT First stage of verification passed, official domain is heinrich-drake-hauptschule.de as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://heinrich-drake-hauptschule.de/personen/kollegium/

@WichtigesYT Pull request merged. Thank you.


Remove statement about auto-generating workload CA cert in iotedge 1.2 doc

This doc has the following statement, which is no longer true for 1.2:
"The IoT Edge security manager also generates a workload CA certificate signed by the device CA certificate"
I suggest adding a 'Changes in version 1.2' section like in the following doc, where it mentions that the workload CA certificate is deprecated in iotedge 1.2: https://docs.microsoft.com/en-us/azure/iot-edge/iot-edge-certs?view=iotedge-2020-11#changes-in-version-12

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

ID: 94792ff1-5976-90f4-5092-0d8d8ff01635
Version Independent ID: d65482d2-64cc-56b1-c52a-b3ddde07b205
Content: Manage device certificates - Azure IoT Edge
Content Source: articles/iot-edge/how-to-manage-device-certificates.md
Service: iot-edge
GitHub Login: @PatAltimore
Microsoft Alias: patricka

@nlcamp Thanks for the feedback! We are currently investigating and will update you shortly.

Assigning this issue to the author @PatAltimore  since the changes needs to be updated to the entire doc and linked doc.
@PatAltimore  Please review this and update.

Thanks for the feedback. The 'workload' line was removed from the 1.2 sections of the documentation and a new 1.2 section was added that explain changes with 1.2. Closing this issue, but feel free to reopen if you need to continue the conversation. New changes should be published in the next 24 hours.
#please-close


[SPARK-37837][INFRA] Enable black formatter in dev Python scripts

What changes were proposed in this pull request?
This PR proposes to enable black formatter (automatic Python code formatter) for dev directory as well.
Why are the changes needed?
To have the consistent style, and make for a better development cycle
Does this PR introduce any user-facing change?
No, dev-only.
How was this patch tested?
I manually verified it as below:
dev/reformat-python
dev/linter-python

cc @zero323 @dongjoon-hyun FYI.

Although there is no logical change, I want to see the running, @HyukjinKwon .
BTW, @HyukjinKwon , your public GitHub Action seems to be crowded. This PR is in the queue still.

Merged to master.


Update parameter names for better readability

Description

closes: #5463

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (f9b918f) 81.19% compared to head (c4932aa) 81.19%.
Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5512   +/-   ##
=======================================
  Coverage   81.19%   81.19%
=======================================
  Files         197      197
  Lines       15253    15253
=======================================
  Hits        12384    12384
  Misses       2404     2404
  Partials      465      465

Files
Coverage Δ

modules/apps/29-fee/ibc_middleware.go
88.27% <100.00%> (ø)

...7-interchain-accounts/controller/ibc_middleware.go
69.56% <85.71%> (ø)

modules/apps/transfer/ibc_module.go
69.10% <87.50%> (ø)

...les/apps/27-interchain-accounts/host/ibc_module.go
84.12% <50.00%> (ø)


ISPN-11545 session embedded fix

https://issues.redhat.com/browse/ISPN-11545

@pruivo I will move the tutorials in another PR

session

Covers actuator metrics for caches

@pruivo I fixed and pushed

session

Covers actuator metrics for caches

I'm not following, sorry!

@pruivo the actuator line specifies that it covers caches, not the session. in any case these examples are moving out once merged
ISPN-11549

integrated! thanks @karesti !


cleos get currency balance broken??

root@eos-test-01:/opt/JungleTestnet# ./cleos.sh get currency balance eosio votertester1
Error 3060003: Contract Table Query Exception
Most likely, the given table doesnt' exist in the blockchain.
Error Details:
Table accounts is not specified in the ABI
This could happen if balance is 0 and therefore no table record?

Hi,thank you very much,but this problem may by solve,the command is ./cleos.sh get currency balance eosio.token votertester1.thank you very much

@bytemaster I have also encountered such a problem, and there is still a problem with following your tutorial.


Fg5 updates

There is some stuff on my Fork, feel free to pull it, it just extra stuff really @Srdjan-V

if there are no more changes required, then this can be merged


Upgrade to fyne 2.5.2

Thank you for the update!

Hm ... rye got reverted back to 0.0.25. No problem, just not sure how it happened ... wouldn't it be a conflict in git?

Ohh, that must have been a mistake. I probably relied on the replace directive, which points to the most recent version of Rye, so upgrading to that should work just as well.

No problem ... just wasn't sure if something got lost or I didn't commit ... but I see it in diff. :+1:


Migrate _gitignore to .gitignore for CLI builds

Description
This happens during NPM publish, but we need to make sure it happens during template compilation.

I think this updates was not merged to @shopify/create-hydrogen npm 🤔
https://www.npmjs.com/package/@shopify/create-hydrogen

All the cli commands now live in this repo https://github.com/Shopify/cli now, so maybe check there / open an issue there?

Sure, I'll check that. Thanks!


[stable/kapacitor] Fixes broken Persistent Volume support

There are a couple of errors in the Kapacitor Persistent Volume config, which only manifest when you enable persistence in the config.

The Deployment is incorrectly indented, resulting in invalid YAML
The PVC itself has metadata keys in the wrong place, resulting in a failure to install

Considering I unknowingly did the same thing over in https://github.com/kubernetes/charts/pull/740 I would give this a +1. Thanks for the patch @jagregory!

@jagregory @jackzampolin The chart was a little messed up. Thank you both.

@k8s-bot ok to test

CI doesn't seem happy with this one -- anyone know why?

@prydonius it says that the container didn't come up. Maybe a timeout? I have tested this on a couple of clusters and had no issues with it. Can we restart the e2e?

@linki thanks. There's supposed to be a catch to not fail if no pods are found (https://github.com/kubernetes/charts/blob/master/test/verify-release.sh#L42), I guess this isn't working correctly? Could it be picking up some other pods?

@prydonius Thanks for the link. The script never enters the PODS_FOUND=1 branch as it always gets an empty string back from listing the pod in the namespace. The No resources found. that is return since v1.5 is going to stderr and not considered I think. So PODS_FOUND stays 0 and results in the error.
I wonder if that final if should rather be if [ "$PODS_FOUND" -eq 0 ];?

@linki looks like you're right, I'm going to try changing it in this PR.


Add flux support to InfluxDB OSS

High level tasks

[ ] Add flux package dependency to InfluxDB
[ ] implement storage.Reader adaptor over services/Store
[ ] add http endpoint to execute flux queries

[ ] implement flux query service to handle HTTP queries and read via storage.Reader

Questions

HTTP handler path for flux queries?

Should be this https://github.com/influxdata/platform/blob/master/http/swagger.yml#L950 which is a /v2/query endpoint


Export to library offset due to double episode

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request

General infomration
Prerequisites

[x] Are you running the latest version?

Description
When exporting a show to the library double episodes are not handled correctly. If the first two episodes are a double episode, episode 3 is exported as episode 2, causing an offset between the metadata and the stream url.
Steps to Reproduce

Export a show (ie Star Trek Voyager)
Play episode 2 (ep1 part 2)
Episode 3 starts playing instead

Expected behavior: ep1+2 should be exported as S01E01-02.strm with matching metadata, ep3 should be exported as S01E03.strm with matching metadata.
Actual behavior: ep1+2 is exported as S01E01.strm, metadata matches ep1 only. Ep3 is exported as ep2 with metadata for ep2, ep4 is exported as ep3 and so on.
Context (Environment)
Installation

[x] I installed the plugin via zip from the Releases page
[ ] I´m using the Netflix Repo
[ ] I´m using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[x] Linux (ARM)
[ ] Android

Additional informatin on the environment
Running osmc latest nightly on rpi 3B.
Thanks!

Good catch, thanks for reporting. Will look into it.


feat: handle multiple possible accounts when logging in

I hereby confirm that I followed the code guidelines found at engineering guidelines
⚙ Affected Components

[x] Commands
[ ] Modules
[ ] Services
[ ] SDK
[ ] Tests

📝 Notes for the Reviewer

NOTE: This PR is paused while we do a full user apiKey feature on its own first. Will adjust the endpoint / logic if need be here after we have that squared away.
This change allows the CLI to correctly interface with the updated findUserApiKeys endpoint handler in the API in order to fetch a user based apiKey as well as handle the case where a user may be a part of multiple accounts correctly.

Sister backend PR: https://github.com/checkly/checkly-backend/pull/2216
🏗 New Dependency Submission

🎟 Affected Issue

Closing this because was already implemented in other PR


Use solar-sunrise-sunset to avoid dealing with strings.

solar-sunrise-sunset returns the same information as sunrise-sunset
in data structure instead of string. Using it greatly reduces complexity
of the program.

Nice improvement! Thanks so much for the PR 👍
I didn't know about solar.el when first building circadian - thanks for introducing it to me 🙂 really appreciate it.


DaVinci Resolve crashes

Describe the bug
Whenever I try to launch DaVinci Resolve, it crashes. Console logs are attached below.
Steps To Reproduce
Steps to reproduce the behavior:

Install davinci-resolve
Run davinci-resolve

Expected behavior
DaVinci Resolve runs
Screenshots

Additional context
Console logs in text form:
❯ davinci-resolve
ActCCMessage Already in Table: Code= c005, Mode= 13, Level=  1, CmdKey= -1, Option= 0
ActCCMessage Already in Table: Code= c006, Mode= 13, Level=  1, CmdKey= -1, Option= 0
ActCCMessage Already in Table: Code= c007, Mode= 13, Level=  1, CmdKey= -1, Option= 0
ActCCMessage Already in Table: Code= 2282, Mode=  0, Level=  0, CmdKey= 8, Option= 0
PnlMsgActionStringAdapter Already in Table: Code= 615e, Mode=  0, Level=  0, CmdKey= -1, Option= 0
<IP_ADDRESS>0 Linux/Clang x86_64
Main thread starts: 16033680
log4cxx: setFile(./logs/rollinglog.txt,true) call failed.
log4cxx: IO Exception : status code = 30
0x7fd616033680 | Undefined            | INFO  | 2022-08-11 18:40:14,436 | --------------------------------------------------------------------------------
log4cxx: No output stream or file set for the appender named [RollLogAppender].
0x7fd616033680 | Undefined            | INFO  | 2022-08-11 18:40:14,436 | Loaded log config from /home/flexagoon/.local/share/DaVinciResolve/configs/log-conf.xml
0x7fd616033680 | Undefined            | INFO  | 2022-08-11 18:40:14,436 | --------------------------------------------------------------------------------

Notify maintainers
@jshcmpbll
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
❯ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.18.15, NixOS, 22.11 (Raccoon), 22.11.20220731.5857574`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.10.3`
 - channels(root): `""`
 - channels(flexagoon): `""`
 - nixpkgs: `/etc/nix/inputs/nixpkgs`

Could you include your ~/.local/share/DaVinciResolve/logs/ResolveDebug.txt file? I had the similar problem and it was fixed by correcting the QT_PLUGIN_PATH in davinci-resolve package. Perhaps your problem is related.

@brcha I can't, because I am no longer using NixOS. Should I close this issue?

@flexagoon I have no idea, probably better not to. I've made a pull request that fixed the issue for me, and the issue looked exactly like yours. When somebody reviews my PR, I'll ask them about closing this issue.

this issue persist


Wording fixes

another round of wording fixes, mostly lower case "du", "ihr", "deine" etc
We use lowercase writing for these from now on, e.g. "Teile dein Wissen"

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Make coverage great again

coverage: 99.046% (+0.5%) from 98.578%
when pulling fdde078c645c05b6226eb0be283ba5c43ea34349 on pump-up-coverage
into 98a1ed9104f5d4ce8117b2fad0e41d8c882543c0 on main.

coverage: 99.046% (+0.5%) from 98.578%
when pulling fdde078c645c05b6226eb0be283ba5c43ea34349 on pump-up-coverage
into 98a1ed9104f5d4ce8117b2fad0e41d8c882543c0 on main.

coverage: 99.046% (+0.5%) from 98.578%
when pulling fdde078c645c05b6226eb0be283ba5c43ea34349 on pump-up-coverage
into 98a1ed9104f5d4ce8117b2fad0e41d8c882543c0 on main.

coverage: 99.046% (+0.5%) from 98.578%
when pulling fdde078c645c05b6226eb0be283ba5c43ea34349 on pump-up-coverage
into 98a1ed9104f5d4ce8117b2fad0e41d8c882543c0 on main.

coverage: 99.046% (+0.5%) from 98.578%
when pulling fdde078c645c05b6226eb0be283ba5c43ea34349 on pump-up-coverage
into 98a1ed9104f5d4ce8117b2fad0e41d8c882543c0 on main.


[Bug]: Crashes after saving a file in watch mode

System Info
System:
OS: macOS 15.0
CPU: (10) arm64 Apple M1 Max
Memory: 1.36 GB / 64.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 20.10.0 - ~/.nvm/versions/node/v20.10.0/bin/node
Yarn: 1.22.21 - ~/.nvm/versions/node/v20.10.0/bin/yarn
npm: 10.2.3 - ~/.nvm/versions/node/v20.10.0/bin/npm
pnpm: 8.12.0 - ~/.nvm/versions/node/v20.10.0/bin/pnpm
bun: 1.1.0 - ~/.bun/bin/bun
Browsers:
Chrome: 129.0.6668.100
Edge: 112.0.1722.68
Safari: 18.0
npmPackages:
@rspack/cli: ^1.0.10 => 1.0.10
@rspack/core: ^1.0.10 => 1.0.10
Details

<EMAIL_ADDRESS>watch:rs
rspack --watch

Rspack compiled successfully in 715 ms
[rspack-cli] [Error:   × TypeError: Cannot set property errors of #<Compilation> which has only a getter
│     at removeCompilationTSLoaderErrors (/Users/mbear/bearspace/test-sourcemap/packages/main/node_modules/ts-loader/dist/after-compile.js:274:24)
│     at /Users/mbear/bearspace/test-sourcemap/packages/main/node_modules/ts-loader/dist/after-compile.js:29:9
│     at Object.fn (/Users/mbear/bearspace/test-sourcemap/packages/main/node_modules/ts-loader/dist/instances.js:206:13)
│     at next (/Users/mbear/bearspace/test-sourcemap/packages/main/node_modules/@rspack/lite-tapable/dist/index.js:523:25)
│     at AsyncSeriesHook.callAsyncStageRange (/Users/mbear/bearspace/test-sourcemap/packages/main/node_modules/@rspack/lite-tapable/dist/index.js:543:9)
│     at /Users/mbear/bearspace/test-sourcemap/packages/main/node_modules/@rspack/lite-tapable/dist/index.js:86:18
│     at new Promise ()
│     at AsyncSeriesHook.promiseStageRange (/Users/mbear/bearspace/test-sourcemap/packages/main/node_modules/@rspack/lite-tapable/dist/index.js:85:16)
│     at QueriedHook.promise (/Users/mbear/bearspace/test-sourcemap/packages/main/node_modules/@rspack/lite-tapable/dist/index.js:224:26)
│     at /Users/mbear/bearspace/test-sourcemap/packages/main/node_modules/@rspack/core/dist/Compiler.js:676:320
│
] {
code: 'GenericFailure'
}
In watch mode, an error occurs when modifying any file
Reproduce link
No response
Reproduce Steps
git clone https://github.com/MBearo/issue-rspack-sourcemap-error/tree/ts-loader-watch-error
npm i
npm run watch:rs
modify any file

packages/rspack/src/Compilation.ts

Based on the current implementation, it seems that setters can be supported. I will give it a try.

It is not recommended to set compilation.errors directly. This will cause the errors originally generated during the compilation to be lost. You can add new errors through the errors.push method.

I understand, but that's how it's written in ts-loader, and it's not convenient to modify.


Fresh Checkout Doesn't Build

Fresh Checkout Doesn't Build
A fresh checkout doesn't seem to build correctly from the command line, even when following the directions from the Building From Source page.
Functional impact
I am unable to build and contribute to the project, as well as build and run the sample projects.
Minimal repro steps

Clone the repository.
Navigate to the repository folder in a terminal.
Run build.cmd.
Wait for the build process to exit.

Expected result
I expected a successful build.
Actual result
The build failed with several errors.
Further technical details
I'm just getting started with this project, so I'm unfamiliar with what is expected from the build and I'm not yet attempting to build from Visual Studio.
I've logged the build output and created this gist (link to the raw content--because of its size, the pretty view truncates it).
Any input or advice you might have on how to get up-and-running would be greatly appreciated.

Git this myself, right now.

Made some progress after I fixed the dotnet env path also the msbuild path. Now I've reached this point:

Your second post has references to Microsoft.AspNet.SignalR which is github.com/SignalR/SignalR (a different repo). Which repo are you trying to build? This issue tracker is for issues with the aspnet/SignalR repo, not SignalR/SignalR

indeed, sorry, the second screen is from other machine that seems to have the old SignalR. switched to the new one.


[CS2103T-W15-3] TrackO

TrackO helps small home business owners to keep track of their orders and customers in one central application. It is optimized through the use CLI for business owners who prefer typing to speed up their workflow.

Codecov Report

Merging #53 (0035e2d) into master (77a32bf) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #53   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fix: type for cred request ldp

The type was incorrectly set, as according to draft 11 of OID4VCI the LDP credential request MUST have a credential_definition object.
The same is true for draft 12.
The code still works with top-level types as well, (as currently both credential_definition.types and types already worked), but the type is corrected now.

Not sure that is correct. V11 had types and credentialSubject as a toplevel property. That moved in v12 to credential_definition. For types it works indeed, but I whether that also is the case for credentialSubject ?
We actually have a v12 request type but that is not used it seems. IMO the solution should be to add the v12 request type to union, instead of removing v11

Can't wait to remove all the draft version support

The type I modified in this PR is only for the Credential Request. The issuer metadata indeed has them top-level, but that is a separate type is declared here: https://github.com/Sphereon-Opensource/OID4VCI/blob/5218c5c59a1d02f2d2b3d323c6b791aa1788b85b/packages/common/lib/types/Generic.types.ts#L86
See the relevant section in the draft 11 spec here: https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0-11.html#section-e.1.3.5

Oh yeah, you are correct. Wow, managed to get it wrong twice :P


Start Graph fail with BSC RPC

I was setup my graph-node to query from BSC Mainnet with RPC: bsc:https://bsc-dataseed1.binance.org/
But when I start graph-node by docker-compose what emitted an error:
WARN Trying again after eth_getBlockByNumber(0, false) RPC call failed (attempt #12) with result Err(Decoder error: Error("invalid type: number, expected a (both 0x-prefixed or not) hex string with length between (0; 64]", line: 0, column: 0)), provider: bsc-rpc-0
I was trying to change other RPC (public and private) but also. This problem only happens with BSC because I still use aurora what work well
Please help me

Hi @vincentho312 are you still having this issue? That suggests there is an issue on the BSC node side. Please ping if this is still ongoing


Not fetching Videos anymore

not working to fetch videos , getting error
no video is downloaded.
(venv1) PS E:\pinterest-downloader-master> python .\pinterest-downloader.py -f -d comp https://in.pinterest.com/Atwidee/electronics/

[i] Job is download single board by username/boardname: Atwidee/electronics
[...] Getting all images in this board: electronics ... [ 0 / ? ] [➕] Found 1 image/video
Download into directory:  comp\Atwidee\Electronics
[...] Downloading: |#########################-------------------------| 50.0% Complete
[Retry] Getting error pin id: '624170829630227793'...
[Retry] Getting error pin id: '624170829630227793'...
[Retry] Getting error pin id: '624170829630227793'...
[Retry] Getting error pin id: '624170829630227793'...
[Retry] Getting error pin id: '624170829630227793'...
[✖] Get this video pin id failed :<PHONE_NUMBER>30227793
[✔] Downloaded: |##################################################| 100.0% Complete
[i] Time Spent: 0:00:08
Please help
@limkokhole

have also tried in multiple python version 3.8 , 3.10

just comment the line 900, it will work

900            v_pin_id = image['id']
this is used multiple times in further code @bonayonpatrick

Then it is only downloading Images not the videos from the board @bonayonpatrick

i tested it, but it's only working on a board link (batch) not on pin link (single)

we have different code in a line number, find this code around line 800 to
1000, and comment it
image = get_pin_info(v_pin_id, None, None, None, False, False, None, None,
None, None, IMG_SESSION, V_SESSION, PIN_SESSION, proxies, cookie_file, True)
On Mon, Dec 25, 2023, 4:57 PM Bhautik Bhalala @.***>
wrote:

Then it is only downloading Images not the videos from the board
@bonayonpatrick https://github.com/bonayonpatrick
—
Reply to this email directly, view it on GitHub
https://github.com/limkokhole/pinterest-downloader/issues/30#issuecomment-1868867669,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVOLSA4Y63HFR2HA4HPNLZDYLE5WDAVCNFSM6AAAAAA73OJEGWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRYHA3DONRWHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Great It Worked Thanks @bonayonpatrick


xDhcpClient doesn't set DNS servers back to DHCP

It would be great if xDhcpClient checked if DNS settings for an adapter were set to "Obtain DNS server address automatically".  Right now it only checks for IP settings and I'd like to switch DNS settings back to DCHP as well.  Let me know if this is a reasonable request.
Greg

Hi @gmatusko
A great point (and needed feature). It would actually need to be implemented in the xDNSServerAddress resource because to reset the DNS Client back to using DHCP settings you actually issue:
Set-DnsClientServerAddress -ResetServerAddresses -InterfaceAlias Ethernet

If we added this in DHCP client then there would be a risk of a config that was conterminously reapplied.
At first I thought this would be relatively easy, but it isn't quite so straight forward.
One way of doing this would be to allow the Address property to accept a $null value. If $null is set then the Resource should ensure that the DNS client is set to DHCP mode. However, I can't find a way of identifying if the DNS client is currently set to DHCP mode or not. I'll need to do some more digging about on this.

I'll revisit this one this weekend and see if I can't figure out a solution.

In this case of switching from static to DHCP it should only reset back to automatic if no primary/secondary DNS is supplied, as it's possible to use both DHCP and a manual DNS server address.

Thanks @andy1547 !
As I mentioned a while back, the solution wouldn't be to change xDHCPClient, but to change xDNSServerAddress so that it allowed a blank list of addresses - which would cause
Set-DnsClientServerAddress -ResetServerAddresses -InterfaceAlias Ethernet

to be called when the DNS client server addresses need to be changed back to DHCP.
However there is one very major problem:
In PowerShell there is no way to detect if the DNS Server Addresses assigned to an adapter were assigned as static or are DHCP assigned - so the DNS Client that was to be configured as being DHCP assigned will constantly be reported as not "in state" and will be continuously be reset to DHCP.
For example, if you run:
Get-DnsClientServerAddress -AddressFamily IPv4 | fl *

you will see the information about the DNS Client server addresses - but you are not able to tell if the addresses in the ServerAddresses property are assigned via DHCP or not. So we can't tell if the resource is in state or not - which will result in continuously resetting the DNS Server Addresses - which will likely cause connectivity issues every time the config is applied.
So I think the only solution is to be able to figure out if the DNS Server Addresses are set with DHCP or not - which will require some changes to PowerShell Net Client.
Does that make sense?

@PlagueHO
I agree with you that it should be part of xDNSServerAddress not xDhcpClient.
The dynamically returned DNS IPs I've have I battled the same problem in my custom script resource and have came up with a great solution which I've described below.
The scenario you described happens both when the DNS is automatic or manual yet blank (sometimes mine would be <IP_ADDRESS>, other times blank). This registry being null or empty holds the answer
HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Services\Tcpip\Parameters\Interfaces\\{Network_Adaptor_GUID}\NameServer
When null or empty this flag would indicate that the DNS addresses returned may potentially be dynamically generated, therefore instead of returning them to be compared (which is pointless) you can return an empty list of IPs.
If the desired configuration of xDNSServerAddress IPs are empty, we have a match and we should not call set. If the configuration of xDNSServerAddress Ips are not empty, then the static IPs will be set.
When this flag is false (the IPs returned from the NIC are statically set) we simply return them and compare with what the desired configuration is.
Moreover, the decision in the SetResource to call either reset or set static should be based on whether the desired configuration DNS IP array is empty or not.
Hope that clears things up, you could combine DHCP enabled check with this flag to determine the difference between automatic and manual yet empty but this is not required as if DHCP is enabled and the DNS is blank windows will always force it back to automatic.

Hi @andy1547 - that is some great info.
I would definitely prefer to use native cmdlets rather than the registry for this, but in the absence of the cmdlets I should be able to make your suggestion work. The plan would definitely to try and only make changes to xDNSServerAddress resource as that would be the safest solution.
Let me see what I can do.

I'm working on this now and aim to have it completed by the end of today.


Optimize code lines

https://github.com/davipon/esbuild-plugin-pino/blob/118b7c3f46c442ae9f22e1e991eac87109b6b9ef/src/index.ts#L131-L136

this could be written as
    globalThis.__bundlerPathsOverrides = {
      ...(globalThis.__bundlerPathsOverrides || {}),
      "thread-stream-worker": pinoBundlerAbsolutePath("./thread-stream-worker.js"),
      "pino-worker": pinoBundlerAbsolutePath("./pino-worker.js"),
      "pino-pipeline-worker": pinoBundlerAbsolutePath("./pino-pipeline-worker.js"),
      "pino/file": pinoBundlerAbsolutePath("./pino-file.js"),
      "pino-pretty": pinoBundlerAbsolutePath("./pino-pretty.js"),
      "pino-sentry": pinoBundlerAbsolutePath("./pino-sentry.js")
    }

so the actual entries do not need to be duplicated.

Thanks for pointing that out! I'd optimized it in the latest PR #43.


Fixes issue when a class isn't in the classpath and NPE is thrown when seeing snapshot.

@loosebazooka @patflynn @elharo fixes #420

I'm going to merge this so I can cut a release this morning.


Update direct-routing-plan-media-bypass.md

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.


More info section is illegible after switching to RTL language and back

Set your device to an RTL language such as arabic
Set your device back to LTR language such as english
🐛  The more info section become illegible.
A reboot seems to fix this.

Actually, fully expanded these don't even make much sense. Where is the day of the month, what is this timestamp?

Seems like an electron/chromium issue, because I am using the underneath JS Date library to get the string format
https://github.com/microsoft/vscode/blob/930fd0e469a43fd923cd732c178e3cc9e878b58d/src/vs/workbench/contrib/extensions/browser/extensionEditor.ts#L946

Which OS is this on, and did you restart the device in between?
I actually saw a similar issue recently with the Windows 11 start menu.

Which OS is this on, and did you restart the device in between? I actually saw a similar issue recently with the Windows 11 start menu.

Mac, didn't restart the device. Restarting the device appears to fix the issue

Merging to https://github.com/microsoft/vscode/issues/169114

@rzhao271 is this merged into https://github.com/microsoft/vscode/issues/169114 because the more info section also does not require the system locale and reverting it to using application locale is the preferred fix ?

@TylerLeonhardt and I were discussing whether that section should even be using the system locale, and I am using #169114 as a catch-all issue for that.