FromSqlRow is not implemented for uuid::Uuid

the trait diesel::types::FromSqlRow<diesel::types::Uuid, diesel::pg::Pg> is not implemented for uuid::Uuid

I get :point_up: on insert from get_result binding. Looks like binding of UUID fields does not work or is broken unless I am doing something wrong.
        let connection = try!(db::postgresql().get());
        insert(&user)
            .into(users::table)
            .get_result::<User>(&*connection)
            .map_err(convert::Into::into)

[dependencies]
iron = "0.5.1"
router = "0.5.1"
mount = "0.3.0"
lazy_static = "0.2.8"
num_cpus = "1.4.0"
log = "0.3.7"
env_logger = "0.4.2"
logger = "0.3.0"
diesel = { version = "0.12.0", default-features = false, features = ["postgres", "chrono", "uuid", "serde_json"] }
diesel_codegen = { version = "0.12.0", default-features = false, features = ["postgres"] }
dotenv = "0.10.0"
r2d2 = "0.7.2"
r2d2-diesel = "0.12.0"
quick-error = "1.2.0"
chrono = { version = "0.3.1", features = ["serde"] }
serde = "1.0.2"
serde_derive = "1.0.2"
serde_json = "1.0.2"
uuid = "0.5.0"
params = "0.6.0"
bcrypt = "0.1.3"

use chrono;
use uuid::Uuid;
use schema::users;
use schema::campaigns;
use diesel::prelude::*;

#[derive(Debug, Identifiable, Queryable, Associations, Serialize, Deserialize)]
#[has_many(campaigns)]
#[belongs_to(tokens)]
pub struct User {
    id: i64,
    email: String,
    #[serde(skip_serializing, skip_deserializing)]
    uuid: Uuid,
    #[serde(skip_serializing, skip_deserializing)]
    password_hash: String,
    created_at: chrono::DateTime<chrono::UTC>,
    updated_at: chrono::DateTime<chrono::UTC>,
}

Diesel 0.12 doesn't support serde 1.0 nor uuid 5.0
If you want to use them, use diesel master branch.


Build and install i18n files

Copied over the gettext handlers from suspended-sentence
The setup.py hacks are a bit ick, but I couldn't figure out anything neater.

At some point, we should investigate some of the existing i18n python tooling, such as Babel, rather than continuing to roll our own.


Character Rendering Weirdness

Describe the issue
Just sitting in a session at 120 and noticed this. Not a big deal, but figured I'd drop it in here just in case it's an easy fix. Characters were not rendering properly...and then later, they were. So this issue was transient; possibly due a font not loading or...? I didn't really do much troubleshooting or thinking about this. I'm running Brave version 1.64.109 Chromium: 123.0.6312.58 (Official Build) (arm64).

It wasn't page specific and showed up incorrectly in several views:

...until it didn't and everything was normal again.

Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

This is a duplicate of #7736 (and that is a repeat of something we saw at IETF-118). Closing in favor of that issue, but thank you for the report!


fix bugs while building on macOS 12 wiht apple silicon

What do these changes do?

add new vineyard version which can build on MacOS with LLVM<=14, find homebrew_prefix in install_cppkafka(), disable build tests and examples in install_cppkafka(), which can not compile on macOS.
Related issue number

#1734

 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.

@sighingnow Can you take a brief look.

Basically LGTM. Just some minor comments.

fixed


about the

Hello,
could you tell me the method to measure the body_control_points.txt
or the means of the each clumn information?
Thank you.

Hi,
Thank you for your interest.

There are 19 measurements defined in the body_control_points.txt. Each measurement is defined by a set of points, which are defined by drawing in the template manually by us.
We sum the total distance of the set of points to represent the corresponding measurement.
For example,
#neck 101 1 10963 1 2 10941 10963 0.25 0.75 2 10941 10963 0.5 0.5 xxxxx
means we use 101 points around the neck to calculate the length of the neck. In the first line, it denotes the first point that locate on #id 10963. The second line denotes the second point that set between #id 10941 and #10963 with a distance weight of 0.25 and 0.75.

Hope it helps!

Hi，
Thank you for your reply. It’ s so helpful.
Sincerely,
DU CHUNQI
<EMAIL_ADDRESS>Windows 10 版のメールから送信
差出人: zyh
送信日時: 2020年2月23日 11:51
宛先: 1900zyh/3D-Human-Body-Shape
CC: DCQ24; Author
件名: Re: [1900zyh/3D-Human-Body-Shape] about the (#23)
Hi,
Thank you for your interest.

There are 19 measurements defined in the body_control_points.txt. Each measurement is defined by a set of points, which are defined by drawing in the template manually by us.
We sum the total distance of the set of points to represent the corresponding measurement.
For example,
#neck 101 1 10963 1 2 10941 10963 0.25 0.75 2 10941 10963 0.5 0.5 xxxxx
means we use 101 points around the neck to calculate the length of the neck. In the first line, it denotes the first point that locate on #id 10963. The second line denotes the second point that set between #id 10941 and #10963 with a distance weight of 0.25 and 0.75.
Hope it helps!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

The specific implementation of calculating measurements can be found here: https://github.com/1900zyh/3D-Human-Body-Shape/blob/6c20fb96f841cb9cf9f8f4bcfcea75db9f468295/src/utils.py#L72


[Candidate 3v3 Nocky Mountains ZH v2] Oil Refinery rotations are not aligned to train tracks

Observed behaviour
Oil Refinery rotations are not aligned to train tracks.
Expected behavior
Oil Refinery rotations are aligned to train tracks.
Screenshots

as designed


Output on the action the Deployment ID

Output deployment ID on action.
Allow additional changes to the deployment information on an App Service for tracking deployment changes from GitHub actions.
GitHub doesn't have a deployment board.  Pulling deployments from the Azure App Service's is a great way to generate a visual deployment board that ties the GitHub deployment workflow to the App Service Deployment logs.

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

Looks like my VS Code altered some of the lines.  Let me know if you want me to revert that.


Add list of last 10 or event all squares for each miners in miners page

we could see if they are signaling intermittently or constantly. Not sure at all but it feels it goes on and off for some miners.

I would prefer to have the miners incorporated in the overview page such that hovering or clicking a pool highlights the associated blocks.


Plugin expansion- Use input plugins param instead of ctx

Description
Plugin expansion- Use input plugins param instead of ctx
Motivation
Reduce usage of ctx in inner methods.
This also allows custom logic in the rules to which attributes they use

@andyscott @johnynek this is where we actually messed up (apart from the proto coverage which I'll check). We should strive to use the inputs wherever possible and not use ctx if we can.

Yep, agreed!


[draft] feat: Allow props to be exported from other files

Fixes #121 and allows for props to exported in the following manner
export {getStaticProps} from '../props';

export default function Page(props) {
	// ...
}

Okay so, the test is passing at this point so just wanted to ask for any feedback, maybe we could implement some more test cases?
cc @Skn0tt @flybayer

@Skn0tt The import changes were due to a type mismatch between @babel/core and @babel/types — they're the same functionally just without having conflicting types (for whatever reason that was). I can revert the imports but it will leave a few anys and things lying around.

Interesting! Yeah, reverting would be nice.

I'll look at that type mismatch separately, that sounds interesting

@Skn0tt  do you need some help with this one? I would love to see it merged!

@apieceofbart this PR contains a lot of unrelated changes, which is why I'm not moving forward with it. if you want to work on it, I think it'd be best to open a new PR :)

@apieceofbart this PR contains a lot of unrelated changes, which is why I'm not moving forward with it. if you want to work on it, I think it'd be best to open a new PR :)

cool, yeah, I'll give it a try especially that a lot of code is already written :)

Ohh I'm really sorry haha, I'm not sure why I never completed this. I think I did have the changes locally but didn't push for some reason. Happy to help as well @apieceofbart if you need a hand. :D

@alii  I only checked briefly yesterday - tried to be smart and copy your code but without the type changes part. Unfortunately when I did this there were a couple of TS errors.
What is weird is that when I open index.ts without any changes I get error in vscode here Argument of type 'Identifier' is not assignable to parameter of type 'Expression | V8IntrinsicIdentifier'.
However running tsc from code line is fine.
This is not a big deal but a bit annoying since when working on the code you don't know which errors are "real".


refactor: (Preview) Disallow creating DataChunk with empty columns and no cardinality

What's changed and what's your intention?
This PR is not intended to be merged directly. It is just a preview / attempt of the following DataChunk API change:

Remove DataChunkBuilder.
Remove TryFrom<Vec<Columns>> for DataChunk.
DataChunk::new requires cardinality as a parameter.

Limitations:

Nested loop join is still not handling zero column correctly (https://github.com/singularity-data/risingwave/issues/2663#issuecomment-1140827168) - likely this PR is faulty
DataChunk::new should verify consistency among cardinality, visibility and columns
The cardinality passed into new may be incorrect - need to split PR and review each caller carefully.

Also thinking the following:
enum Visibility {
  Bitmap(Bitmap),
  Capacity(usize),
}
struct DataChunk {
  columns: Vec<Column>,
  visibility: Visibility,
  // visibility: Option<Bitmap>,
  // cardinality: usize,
}

Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

Superseded by these parts:

#2928
#2934
#2936
One last PR to migrate to new in remaining cases


iOS ERROR:  [!] CocoaPods could not find compatible versions for pod "Google-Mobile-Ads-SDK"

google_mobile_ads: ^2.3.0
Flutter (Channel stable, 3.3.2, on macOS 12.6 21G115 darwin-arm,
Xcode Version 14.0   (Minimum Deployments: iOS 11.0)
MacOS 12.6  M1
Error
[!] CocoaPods could not find compatible versions for pod "Google-Mobile-Ads-SDK":
      In Podfile:
        google_mobile_ads (from `.symlinks/plugins/google_mobile_ads/ios`) was resolved to 0.0.1, which depends on
          Google-Mobile-Ads-SDK (~> 9.13)

    None of your spec sources contain a spec satisfying the dependency: `Google-Mobile-Ads-SDK (~> 9.13)`.

    You have either:
     * out-of-date source repos which you can update with `pod repo update` or with `pod install --repo-update`.
     * mistyped the name or version.
     * not added the source repo that hosts the Podspec to your Podfile.

Hi @wadychou
Please try following the instructions from the log by executing pod repo update and try again to see if the issue still persists.

Hi @wadychou Please try following the instructions from the log by executing pod repo update and try again to see if the issue still persists.

I've tried all of the following and it doesn't work.
1 -------
pod repo update
2-------
sudo arch -x86_64 gem install ffi
arch -x86_64 pod install
flutter clean
flutter pub get
3------
sudo arch -arm64 gem pristine ffi
pod install
flutter clean
flutter pub get

I have ready fix it, look below.

use Rosetta to open Terminal

cd into iOS folder

sudo arch -x86_64 gem install ffi

arch -x86_64 pod repo update

arch -x86_64 pod install


Add OnError

See #52 and #32.

Can I help on moving forward with this PR? I'm very interested on start using it. Thanks

See the discussion in #52. I’m still not convinced the API is right. I can issue an alpha version with an unstable API though if you don’t mind being broken if it changes again.

I released an alpha version. go get -v -u<EMAIL_ADDRESS>I would appreciate feedback if you end up using it.

Thanks, we will give it a try and provide feedback.
On Mon, Oct 31, 2022, 15:18 Carl Johnson @.***> wrote:

@.**** commented on this pull request.
In builder_core.go
https://github.com/carlmjohnson/requests/pull/54#discussion_r1009473322:

@@ -165,6 +166,12 @@ func (rb *Builder) Config(cfgs ...Config) *Builder {
return rb
}

+// OnError adds an ErrorHandler to run if any part of building, validating, or handling a request fails.
Add more documentation.
—
Reply to this email directly, view it on GitHub
https://github.com/carlmjohnson/requests/pull/54#pullrequestreview-1161959886,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAACVLXBJFBKP6JUI2GIENTWF7ILVANCNFSM6AAAAAAQVGMAFE
.
You are receiving this because you commented.Message ID:
@.***>

See #63, which has been merged.


allow dynamic draggable data with use of updated hook

Hi,
i had the problem that the dragdata wasnt updated, if the dragdata was dynamically created, e.g. with a computed property which creates an object.
you can see the problem in demo 8
also i changed the event names, since onDrag-drop never fired with vue 3.0.4

Great work, thanks for your contribution! I've decided to release this as a major update (already available on npm) because the behavior is slightly changed (well, actually it's fixed now) and the event names worked in Vue 3.0.0 but stopped working in Vue 3.0.1.


Make homepage carousel responsive

fixes #521

Copy is cut off on landscape on mobile.

Looks awesome here. :)

Gonna look at the code. If it looks clean, I'll merge.

Hm, I don't like the setting styles directly from the js (better to add/remove classes and set the styles in the CSS) but that's not unique to this branch. I'll open a separate issue for that.

I have no issue for setting calculated styles like min-height: calculatedValue because that can't be done in CSS.


Fix NODE_PATH for Node process

I'm going to close this as it should be resolved in #30. I'd recommend opening an issue if this is still an issue that needs to be resolved.


Adding maxwell probe layout

Playing with the software. Maybe I'm doing something wrong, since when plotting the probe, the scale for xaxis and yaxis are clearly not optimal. However, the layout seems correct...

C'est top!!!

C'est top!!!

I don't known the this format but if it exists 2 variant maybe the function could make an auto guess of which one by testing the 2 possible path.
Alessio do you have a file that Pierre could test ?

I don't known the this format but if it exists 2 variant maybe the function could make an auto guess of which one by testing the 2 possible path.
Alessio do you have a file that Pierre could test ?

Alessio. Idon't anderstand. In the existing extractor the mapping is at the same place for both case. fihandle['mapping'] am I wrong ?

Alessio. Idon't anderstand. In the existing extractor the mapping is at the same place for both case. fihandle['mapping'] am I wrong ?

Salut Pierre.
Alessio could you test it ?

Salut Pierre.
Alessio could you test it ?

@yger I made a couple of suggestions. After that it's good to go. Tested with maxone and maxtwo.
@samuelgarcia the plotting always shows a square axis. I think that the limits shoule depend on the electrodes and the plot should be a rectangle if the geometry is not suared..otherwise it looks weird:

@yger I made a couple of suggestions. After that it's good to go. Tested with maxone and maxtwo.
@samuelgarcia the plotting always shows a square axis. I think that the limits shoule depend on the electrodes and the plot should be a rectangle if the geometry is not suared..otherwise it looks weird:

@alejoe91 : yes I known I need to change the au auto lims approach. I did it quickly to ensure aspect = 1

@alejoe91 : yes I known I need to change the au auto lims approach. I did it quickly to ensure aspect = 1

@yger : you should also do this
probe.annotate(name='probe name', manufacturer='maxwell')

and maybe if you have some relevant information about the MEA you could also put it in annotations. (like serial number, ...)

@yger : you should also do this
probe.annotate(name='probe name', manufacturer='maxwell')

and maybe if you have some relevant information about the MEA you could also put it in annotations. (like serial number, ...)

Pierre can you fix the doc and latest comments so I could merge this ?

Pierre can you fix the doc and latest comments so I could merge this ?

Cool. Can I merge ?

Cool. Can I merge ?

yep

yep


Process Validation Error Code Mismatch: Incorrect Status Code for Any Thrown Error

What version of Elysia is running?
"elysia": "^1.1.22"
What platform is your computer?
Microsoft Windows NT 10.0.19045.0 x64
What steps can reproduce the bug?
import { Elysia, NotFoundError, t } from "elysia";

export const errorPlugin = new Elysia().onError(
  { as: "global" },
  ({ code, error }) => {
    console.log(code); // VALIDATION

    return { error };
  }
);
function decryptId(value: string) {
  if (value.length > 2) throw new NotFoundError(); // should give 404 status code, not 422
  return 1;
}
const transform = t
  .Transform(t.String())
  .Decode((value) => decryptId(value))
  .Encode((value) => `1`);

const aid = t.Object({
  aid: transform,
});
const activity = new Elysia().post(
  "activity/aid/:aid",
  ({ params, body, query }) => {
    console.log({ body: body.aid, query: query.aid, params: params.aid });
    const array = Array(10).fill({ aid: body.aid });
    return {
      data: array,
    };
  },
  {
    params: aid,
    body: aid,
    response: {
      200: t.Object({
        data: t.Array(t.Object({ aid: t.Integer() })),
      }),
    },
  }
);

const app = new Elysia()
  .use(errorPlugin)
  .use(activity)
  .listen(3000);

What is the expected behavior?
The expected behavior is that when a validation process throws an error, the correct HTTP status code corresponding to that error should be returned in .onError.
In this case, when a NotFoundError is thrown, the expected behavior is that a 404 HTTP status code should be returned, not a 422 status code.
What do you see instead?
Thrown errors during t.transform gives always 422 status code in .onError
Additional information
No response
Have you try removing the node_modules and bun.lockb and try again yet?
no

throw new NotFoundError() in transform gives

HTTP 422 with aot: true
HTTP 500 with aot: false

The issue seems to be that Typobox wraps the NotFoundError in a TransformDecodeError, so error.status is not picked up by app.handleError.
I took a stab at unwrapping the error in PR #885

maybe this solution will solve your problem click


Update location type in schema-specs and implementation

Relating to the P2P network and how we display the location of a concept.
Current:
{
	"schema": {
		"inCity": {
			"$cref": "cfd0b3ea-9128-4fe5-a024-b7e68fe22206",
			"locationUrl": "http://localhost",
			"type": "Thing"
		}
	}
}

New:
{
	"schema": {
		"inCity": {
			"$cref": "weaviate+http://weaviate-a/cfd0b3ea-9128-4fe5-a024-b7e68fe22206"
		}
	}
}

Where weaviate-a also can be localhost.
Important:
This should also be reflected in the open-api schema and while creating concepts (id est, things or actions).

@laura-ham: FYI, can you take this into account in the prototype?
@dljcollette: FYI, does the above have an impact on your work?


Integrating Groq LPU Inference Engine API with Verba

Hi all!
Here an instruction how to integrate Groq API with Verba.
Obtain API from

pip install groq
Create "GroqGenerator.py" at goldenverba/components/generation folder. Paste this code:

import os
from dotenv import load_dotenv
from groq import AsyncGroq

from collections.abc import Iterator
from goldenverba.components.interfaces import Generator

load_dotenv()

GROQ_API_KEY = os.getenv("GROQ_API_KEY")

class GroqGenerator(Generator):
    """
    Groq Generator.
    """

    def __init__(self):
        super().__init__()
        self.name = "GroqGenerator"
        self.description = "Generator using Groq's LPU INFERENCE ENGINE"
        self.requires_library = ["groq"]
        self.requires_env = ["GROQ_API_KEY"]
        self.streamable = True
        self.model_name = os.getenv("GROQ_MODEL", "llama3-8b-8192")
        self.context_window = 8192

        # Initialize Groq client
        self.client = AsyncGroq(api_key=GROQ_API_KEY)

        # Define model details
        self.models = {
            "gemma-7b-it": { "tokens": 8192, "developer": "Google"},
            "llama3-70b-8192": { "tokens": 8192, "developer": "Meta"},
            "llama3-8b-8192": { "tokens": 8192, "developer": "Meta"},
            "mixtral-8x7b-32768": { "tokens": 32768, "developer": "Mistral"},
        }

        # Select a model
        self.max_tokens = self.models[self.model_name]["tokens"]

    async def generate_stream(self, queries: list[str], context: list[str], conversation: dict = None):
        """
        Generate a stream of response dictionaries based on a list of queries, a list of contexts, and includes conversational context. This function uses an asynchronous API to generate responses incrementally, suitable for streaming applications where responses are expected to be delivered in real-time as they are generated.

        @parameter queries: list[str] - A list of user queries to which the generator should respond.
        @parameter context: list[str] - Contextual information relevant to each query, which helps the model to generate more accurate and relevant responses.
        @parameter conversation: dict - A dictionary representing the conversational history and context. This helps in maintaining the flow and relevance of the conversation.

        @returns: Iterator[dict] - An iterator that yields dictionaries representing parts of the generated response. Each dictionary includes a 'message' key with the generated text, and a 'finish_reason' key that indicates whether the generation has completed ('stop') or if it is still ongoing ('').
        """
        if conversation is None:
            conversation = []
        messages = self.prepare_messages(queries, context, conversation)

        try:
            chat_completion = await self.client.chat.completions.create(
                model=self.model_name,
                messages=messages,
                max_tokens=self.max_tokens,
                temperature=0.000000001,
                stop=None,
                stream=True
            )

            async for chunk in chat_completion:
                finish_reason = chunk.choices[0].finish_reason
                if chunk.choices[0].delta.content:
                    yield {
                        "message": chunk.choices[0].delta.content,
                        "finish_reason": ""
                    }
                else:
                    yield {
                        "message": "",
                        "finish_reason": finish_reason,
                    }

        except Exception as e:
            print(f"An error occurred: {str(e)}")
            raise e

    def prepare_messages(self, queries: list[str], context: list[str], conversation: list[dict]) -> dict[str, str]:
        """
        Prepares a list of messages formatted for a Retrieval Augmented Generation chatbot system, including system instructions, previous conversation, and a new user query with context.

        @parameter queries: A list of strings representing the user queries to be answered.
        @parameter context: A list of strings representing the context information provided for the queries.
        @parameter conversation: A list of previous conversation messages that include the role and content.

        @returns A list of message dictionaries formatted for the chatbot. This includes an initial system message, the previous conversation messages, and the new user query encapsulated with the provided context.
        """
        messages = [
            {
                "role": "system",
                "content": "You are a Retrieval Augmented Generation chatbot. Please answer user queries only their provided context. If the provided documentation does not provide enough information, say so. If the answer requires code examples encapsulate them with ```programming-language-name ```. Don't do pseudo-code.",
            }
        ]

        for message in conversation:
            messages.append(message)

        query = " ".join(queries)
        user_context = " ".join(context)

        messages.append(
            {
                "role": "user",
                "content": f"Please answer this query: '{query}' with this provided context: {user_context}",
            }
        )

        return messages

Modify "manager.py":

from goldenverba.components.generation.ClaudeGenerator import ClaudeGenerator
from goldenverba.components.generation.GroqGenerator import GroqGenerator

class GeneratorManager:
    def __init__(self):
        self.generators: dict[str, Generator] = {
            "ClaudeGenerator": ClaudeGenerator(),
            "GPT4Generator": GPT4Generator(),
            "GPT3Generator": GPT3Generator(),
            "CohereGenerator": CohereGenerator(),
            "Llama2Generator": Llama2Generator(),
            "GroqGenerator": GroqGenerator(),
        }

we added from goldenverba.components.generation.GroqGenerator import GroqGenerator and "GroqGenerator": GroqGenerator(),

Modify "verba_manager.py" add:

        # Check Groq ENV KEY
        try:
            import groq

            groq_key = os.environ.get("GROQ_API_KEY", "")

            if groq_key != "":
                self.environment_variables["GROQ_API_KEY"] = True
                self.client = groq.Groq(api_key=groq_key)
            else:
                self.environment_variables["GROQ_API_KEY"] = False
                raise ValueError("GROQ_API_KEY environment variable is not set.")

        except Exception as e:
            self.environment_variables["GROQ_API_KEY"] = False
            print(f"Error initializing Groq API: {e}")

somewhere at 200+th line
and
        try:
            import groq

            self.installed_libraries["groq"] = True
        except Exception:
            self.installed_libraries["groq"] = False

somewhere at 300+th line

Add this variables to your environment or .env file:

# GROQ
GROQ_API_KEY=<you_groq_api_key>

GROQ_MODEL=llama3-70b-8192

GROQ_MODEL can be:
gemma-7b-it
llama2-70b-4096
llama3-70b-8192
llama3-8b-8192
mixtral-8x7b-32768

Thanks! Feel free to create a PR

Hi ! I want to contribute to Verba. Can I submit a PR for this ? Starting from instructions above ?

I would recommend waiting for the v2 update, which will be released in a couple of weeks!

Hey @symdec , we just released v2 of Verba! Feel free to create a PR

Thanks a lot for the PR! 🚀


not being able to scan all interfaces

Scenario: wifi and eth both enabled, with device on eth. rdt scan does not find device, but taking down wifi results in the following error:
$rdt scan
dgram.js:438
    throw errnoException(err, 'addMembership');
    ^

Error: addMembership ENODEV
    at exports._errnoException (util.js:907:11)
    at Socket.addMembership (dgram.js:438:11)
    at Socket.<anonymous> (/home/greg/prog/nvm/versions/node/v4.6.1/lib/node_modules/resin-device-toolbox/node_modules/resin-sync/node_modules/resin-discoverable-services/node_modules/bonjour/node_modules/multicast-dns/index.js:52:14)
    at emitNone (events.js:72:20)
    at Socket.emit (events.js:166:7)
    at startListening (dgram.js:121:10)
    at dgram.js:221:7
    at nextTickCallbackWith3Args (node.js:453:9)
    at process._tickCallback (node.js:359:17)

Please see resin-io-modules/resin-discoverable-services#10.


Dependabot auto merge does not respect permissions.

My team is trying to set up dependabot auto merge. We have auto approve working using the github token and granted PR write permissions but the same does not work for auto merge. Is there a way to get auto merge working without a PAT? Setting up a PAT for each repo in our Org isn't feasible.

This question isn't really about fetch-metadata, but rather the overall Dependabot tokens, and is already tracked/explained in detail here:

https://github.com/dependabot/dependabot-core/issues/1973

Is there a way to get auto merge working without a PAT?

For now, you need a PAT.

Setting up a PAT for each repo in our Org isn't feasible.

Yeah, that's painful. I don't think there's currently a way to set an org-level PAT, but I'm not fully sure TBH. I suggest open a discussion in https://github.com/orgs/community/discussions/categories/code-security


Wrong cache directory

according to: https://github.com/lukka/run-vcpkg/blob/d950c946406d5336fc77433f256c813b94f330d9/src/vcpkg-action.ts#L102
it's save a wrong cache (from vcpkg docs):
Currently, our file-based backend is enabled by passing the undocumented --binarycaching flag to any
Vcpkg command or setting the undocumented environment variable VCPKG_FEATURE_FLAGS to binarycaching.

We will replace this feature flag with an on-by-default user-wide behavior, plus command line and environment-based configurability.

The on-by-default configuration will specify the file-based archive protocol on either
%LOCALAPPDATA%/vcpkg/archives (Windows) or $XDG_CACHE_HOME/vcpkg/archives (Unix).
If XDG_CACHE_HOME is not defined on Unix, we will fall back to $HOME/.cache/vcpkg/archives based
on the XDG Base Directory Specification.
This can be redirected with a symlink, or completely overridden with the command line or environment.
In the future we can also consider having a user-wide configuration file,
however we do not believe this is important for any of our key scenarios.

https://vcpkg.readthedocs.io/en/latest/specifications/binarycaching/#solution-aspects
for example with your actions every run it's what happens:
  Could not locate cached archive: /home/runner/.cache/vcpkg/archives/b0/b0c723932ac57ff82b8bb08dad40b2b9c5ed146d.zip

  Stored binary cache: /home/runner/.cache/vcpkg/archives/b0/b0c723932ac57ff82b8bb08dad40b2b9c5ed146d.zip

Caching paths: '/home/runner/work/atomicDEX-Desktop/atomicDEX-Desktop/ci_tools_atomic_dex/vcpkg-repo,!/home/runner/work/atomicDEX-Desktop/atomicDEX-Desktop/ci_tools_atomic_dex/vcpkg-repo/packages,!/home/runner/work/atomicDEX-Desktop/atomicDEX-Desktop/ci_tools_atomic_dex/vcpkg-repo/buildtrees,!/home/runner/work/atomicDEX-Desktop/atomicDEX-Desktop/ci_tools_atomic_dex/vcpkg-repo/downloads'
  Running save-cache
  /usr/local/bin/tar --use-compress-program zstd -T0 -cf cache.tzst -P -C /home/runner/work/atomicDEX-Desktop/atomicDEX-Desktop --files-from manifest.txt
  Cache saved successfully

^ we can see that $HOME/.cache/vcpkg/archives or C:\Users\runneradmin\AppData\Local\vcpkg\archives\ is missing in the caching path
would be nice to have a v6 that cache the good directory
I never made typescript in my life but an equivalent in c++ would be :
std::vector<std::filesystem::path> get_cache_paths()
{
     std::vector<std::filesystem::path> paths;
#if defined(_WIN32) || defined(WIN32)
        const char* localappdata = nullptr;

        if (localappdata = std::getenv("LOCALAPPDATA"); localappdata)
        {
              paths.push_back(std::filesystem::path(localappdata) / "vcpkg" / "archives");
        }
#else
       const char* home = nullptr;

        if (home = std::getenv("XDG_CACHE_HOME"); home)
        {
              paths.push_back(std::filesystem::path(home) / "vcpkg" / "archives");
        } else if (home = std::getenv("HOME"); home) {
              paths.push_back(std::filesystem::path(home) / ".cache" / "vcpkg" / "archives");
        }
#endif
   return paths;
}

@Milerius the run-vcpkg action has been designed for vcpkg "classic", where all the installed ports are installed in $VCPKG_ROOT/installed directory.
The binary caching is a preview feature this action is not currently supporting. Before supporting "modern/future" vcpkg, I'd need to understand in detail how this newly created features (e.g., binary caching and manifest) are working and how to support them.
I'd be glad if the exact and complete specs would be provided in here before I can do it.
If I do it first, I'll share them in here before rushing implementing those, as feedback is always useful.

@lukka binary caching is no longer unstable, and it's the recommended way to do caching nowadays; see the docs here. The readthedocs are usually out of date, since someone has to manually run the script to update them.

@strega-nil: good to know! I need time to write a design around it, i.e. how would the flowchart change.
TL;DR: I think it would be helpful to have in vcpkg a command that prints out the result of the vcpkg's cache directory computation logic (i.e. it prints the directory).
Details:
Is it needed to implement an high fidelity replica of the logic described here?
run-vcpkg needs to know whether binary caching is being used or not;  and it needs to reliably know what is the directory where the cache needs to be restored into (before running vcpkg), and saved later from. I.e. whatever vcpkg thinks it is the right directory for caching, that is the one run-vcpkg must accurately know as well.

@ras0219 should be able to tell you more :)

Please review the following and let me know if you are on the same page.
TL;DR
There is no reason to cache the binary cache directory content on the GitHub cache service, as it will just increase the restore from cache scenario. Continuing caching the $VCPKG_ROOT directory is the most performant solution.
Details
Please remember that run-vcpkg puts into the GitHub cache service basically all of $VCPKG_ROOT (excluding some directories to dramatically reduce the required space). So when the action installs sqlite3 package and the package has been restored from the GitHub cache, it takes <0.1 second.
OTOH, if run-vcpkg is changed to do as requested here, installing sqlite3 (i.e. vcpkg install sqlite3) with the $VCPKG_ROOT/installed directory empty, it would take >3 seconds. The times looks like spent mostly in detecting the "compiler hash for triplet..." (longer on Windows than on Linux, but in both cases it's >1 second), and expanding the archive and writing the new files onto $VCPKG_ROOT/installed directory.

In the cache hit scenario, that's the correct TL;DR -- as long as every package is in the installed\ folder, we won't consult the archives. The only way a package would be missing from the installed folder is if the cache didn't hit, in which case the archives would also be missing. There is potentially some improvement available via fallback keys on the archives directory, however I think this is relatively low value.
Instead, we recommend using GitHub Packages[1] to store the binaries via our NuGet backend. This will reuse packages as much as possible and complement the Cache Task to maximize performance and minimize rebuilds.
[1] https://github.com/microsoft/vcpkg/blob/master/docs/users/binarycaching.md#github-packages

based on the conclusions, no changes are needed to this action regarding the cached directories.

There is no reason to cache the binary cache directory content on the GitHub cache service, as it will just increase the restore from cache scenario. Continuing caching the $VCPKG_ROOT directory is the most performant solution.

Caching VCPKG_ROOT is hardly helpful, at least for manifest mode. All it helps with is avoiding needing to bootstrap the vcpkg executable. Caching VCPKG_ROOT/packages, VCPKG_ROOT/buildtrees, VCPKG_ROOT/downloads does not help; vcpkg still does a full rebuild every time (in manifest mode at least).

So when the action installs sqlite3 package and the package has been restored from the GitHub cache, it takes <0.1 second.
OTOH, if run-vcpkg is changed to do as requested here, installing sqlite3 (i.e. vcpkg install sqlite3) with the $VCPKG_ROOT/installed directory empty, it would take >3 seconds. The times looks like spent mostly in detecting the "compiler hash for triplet..." (longer on Windows than on Linux, but in both cases it's >1 second), and expanding the archive and writing the new files onto $VCPKG_ROOT/installed directory.

For these use cases caching is hardly relevant anyway. For complex applications with large dependencies like FFmpeg (30 minute build), wxWidgets (10 minute build), or Qt5 (>2 hour build!), the cost of computing the hash for vcpkg's package cache key is trivial compared to the build time. Running vcpkg install for this dependency list for Tenacity:
  "dependencies": [
    "libflac",
    "lilv",
    "libmad",
    "libogg",
    "libsndfile",
    "soxr",
    "libvorbis",
    "soundtouch",
    "libsbsms",
    "portaudio",
    "portmidi",
    "portsmf",
    "mp3lame",
    "libogg",
    "libvorbis",
    "zlib",
    "expat",
    "sqlite3",
    "ffmpeg",

    "libpng",
    "tiff",
    "zlib"
  ]

takes 9 seconds, which is very fast from my perspective.

There is potentially some improvement available via fallback keys on the archives directory, however I think this is relatively low value.

The benefits of fallback restore keys are huge if an application has even one dependency that takes a long time to build.
I have fixed these issues in #94.

@Be-ing Thanks for adding this comment! I'd like to put your comments on the right context first, as I think we are almost on the same page, only some clarifications are still needed.

Caching VCPKG_ROOT is hardly helpful, at least for manifest mode. All it helps with is avoiding needing to bootstrap the vcpkg executable. Caching VCPKG_ROOT/packages, VCPKG_ROOT/buildtrees, VCPKG_ROOT/downloads does not help; vcpkg still does a full rebuild every time (in manifest mode at least).

Caching $VCPKG_ROOT means to save its content except for what is not useful. That is, all but packages, buildtrees and downloads. And please note that when the vcpkg.json manifest mode is used (which is the mode the action and the samples are tailored with) a full rebuild does not take place.
I'd be curious to see what is the run-vcpkg run time in cache-hit scenario on the tenacity project (which is the "restore VCPKG_ROOT scenario except what is not needed"), compared to the time spent to leverage vcpkg's binary caching created archives (and restoring those from cache).
Also, I really appreciated your contribution on #94, which is something I was preparing already but I did not publish yet. So you beat me to it!

I'd be curious to see what is the run-vcpkg run time in cache-hit scenario on the tenacity project (which is the "restore VCPKG_ROOT scenario except what is not needed"), compared to the time spent to leverage vcpkg's binary caching created archives (and restoring those from cache).

Without #94 every package is fully rebuilt for any cache miss. The action is hardly useful for my purposes like that.

I just realized I got confused by the incorrect information in the README:
          # Ensure the vcpkg artifacts are cached, they are generated in the 'CMAKE_BINARY_DIR/vcpkg_installed' directory.
          additionalCachedPaths: ${{ env.buildDir }}/vcpkg_installed

Manifest mode installs to vcpkg_installed in the root of the repository being built. This directory is outside the CMake build directory.

Caching the vcpkg_installed directory in manifest mode is indeed slightly faster that caching the user local binary cache.


Implementation of explicit storage equations in MESSAGE

Motivation:
There are different ways to represent storage (such as electricity storage, water storage, heat storage, etc.) using the current mathematical model in message_ix. However, it may be more efficient to add explicit equations represnting storage technologies and their functionality. This issue is opened to discuss the best way of doing this so that the added equations will serve as many as possible storage technologies and applications.
Requirements:
In this context, the desired equations should meet the following requirements:

providing the possibility to invest in some storage technologies for shifting a specified commoditiy (like electricity) between subannual time steps
showing the state of storage at any sub-annual time interval
ensuring the state of storage is the same at the beginning and end of the year (any desire for inter-year storage?)
represnting the storage losses ideally relative to the duration of storage (number of sub-annual time steps)
being consistent with other equations, like COMMODITY_BALANCE

Please add your views and comments in this thread, and we try to synthesize the best possible way forewards.

Adriano's suggestion:
This method is based on the changes in commodity balance equation and adding a new block of equations for storage:

Adding the following lines to COMMODITY_BALANCE to represnt quantity taken out from ( >0 ) or put into ( <0 ) inter-period stock (storage):
_+ STOCK_CHG(node,commodity,level,year,time)$( map_stocks(node,commodity,level,year) )

STORAGE_CHG(node,commodity,level,year,time)$( map_storage(node,commodity,level,year,time) AND bound_storage_up(node,commodity,level,year,time) )_

Adding storage equations (including CAP variable and a possible mapping in the BOUND equations):
STORAGE_BALANCE(node,commodity,level,year,time)$( map_storage(node,commodity,level,year,time) AND bound_storage_up(node,commodity,level,year,time) )..
STORAGE(node,commodity,level,year,time)

fixed level of storage (i.e. in first period)
- commodity_storage(node,commodity,level,year,time) =E=
( storage_to_activity(time)*STORAGE_CHG(node,commodity,level,year,time) )
the next line is just saying STORAGE(t-1)
+ SUM((time2,year2)$seq_year_time(year2,year,time2,time), STORAGE(node,commodity,level,year2,time2) *
(1 - storage_loss(node,commodity,level,year2,time2)) ) ;

STORAGE_BOUND_UP(node,commodity,level,year,time)$( map_storage(node,commodity,level,year,time) AND bound_storage_up(node,commodity,level,year,time) )..
STORAGE(node,commodity,level,year,time) =L= map_storage_tec(node,tec,commodity,level,year,time) * CAP(node,tec,vintage,year) ;
STORAGE_BOUND_LO(node,commodity,level,year,time)$( map_storage(node,commodity,level,year,time) AND bound_storage_up(node,commodity,level,year,time) )..
STORAGE(node,commodity,level,year,time) =G= map_storage_tec(node,tec,commodity,level,year,time) * CAP(node,tec,vintage,year) *
storage_lo_factor(node,commodity,level,year,time) ;

pictures are attached for a clearer notation in GAMS.

Technology-based storage implementation
This alternative is based on introducing two technologies as charger and discharger for storing a commodity and discharging that from storage, respectively. This resembles the pump and turbine sections in a pumped hydropower storage plant. The two technologies can be incorporated in one but working in two different "mode"s. The storage media itself will be another technology (like a dam).
The advantage here is that the techno-environmental-economic parameters of the charging and discharging technologies can be conveniently specified in the model, using the current parameters. For example, technologies can have their own efficiency, investment and O&M costs, emissions (if any), etc., independent from the storage media.
To implement this storage model:

Charging and discharging technologies should be specified in the model input parameters. This can be done by, first, specifying storage levels, such as introducing a set called "level_storage" similar to "level_renewable", for example. Then, those technologies with input and output to this storage level can be mapped as “charge_tec” and “discharge_tec”, and both technologies together as storage technologies, "storage_tec".

The order of subannual timesteps will be defined by the user in a set or parameter called “seq_time”. This will be a dataframe showing the order as a number for each subannual time step, e.g., 1 for “Jan”, etc.

In the “period_parameter_assignment” file of the model, the sequence of time and year together will be defined as:

Each subsequent time step can be either in the same year or in the next ones.

Activity (ACT) of storage technologies will define the change in the storage content. This will replace the part called "STORAGE_CHG" in the first implementation alternative above. Hence, the “STORAGE” equation, which drives the content of storage, will look like:

The upper bound and lower bound on the content of storage can be defined as follows. If storage media will be introduced as a technology itself, for example a dam, then the installed capacity (CAP) of that technology multiplied by “capacity_factor” will be the upper bound of storage content. the lower bound should be explicitly defined by the user for each subannual timestep as “bound-storage_lo”. These two bounds can be as the following:

Simple Approach to Add Seasonal Storage to MESSAGE
Behnam has proposed a possibility for adding seasonal storage technologies to MESSAGE. The approach is simple to understand and implement. We have implemented and it seams to work well.
It consists of creating a new technology, for example, 'stor_SPS' in which the 'electricity' input and output months are explicitly entered in the technology, as presented below using the Austria Tutorial example:
The months in which the SPS plant stores energy are explicitly added to the model.
base_input = {'node_loc': country,'year_vtg': vintage_years,'year_act': act_years,'mode': 'standard','node_origin': country,'commodity': 'electricity','time': '1','time_origin': '1',}
water_in_SPS = make_df(base_input, technology = 'stor_SPS',commodity='electricity', level = 'secondary',value = 1.2, unit = '%')
base_input = { // 'time': '4','time_origin': '4' //}
base_input = { // 'time': '5','time_origin': '5' //}
base_input = { // 'time': '6','time_origin': '6' //}
base_input = { // 'time': '7','time_origin': '7' //}
base_input = { // 'time': '8','time_origin': '8' //}
base_input = { // 'time': '9','time_origin': '9' //}
The months in which the SPS plant generates electricity are explicitly added to the model.
base_output = {'node_loc': country,'year_vtg': vintage_years,'year_act': act_years,'mode': 'standard','node_dest': country,'time': '1','time_dest': '1','unit': '%',}
water_out_SPS = make_df(base_output, technology='stor_SPS', commodity='electricity',level='secondary', value=1.0, unit = '%')
base_output = { // 'time': '1','time_origin': '1' //}
base_output = { // 'time': '2','time_origin': '2' //}
base_output = { // 'time': '3','time_origin': '3' //}
base_output = { // 'time': '10','time_origin': '10' //}
base_output = { // 'time': '11','time_origin': '11' //}
base_output = { // 'time': '12','time_origin': '12' //}
The capacity factor for the input and output should change with months, according to the intended operation of the plant.
base_capacity_factor = {'node_loc': country,'year_vtg': vintage_years,'year_act': act_years,''time'': '1','unit': '%',}
df = make_df(base_capacity_factor, technology='stor_SPS', value=0.8)
base_capacity_factor = // 'time'': '2' //   df = // value=0.4)
base_capacity_factor = // 'time'': '3' //   df = // value=0.1)
base_capacity_factor = // 'time'': '4' //   df = // value=0.1)
base_capacity_factor = // 'time'': '5' //   df = // value=0.4)
base_capacity_factor = // 'time'': '6' //   df = // value=0.8)
base_capacity_factor = // 'time'': '7' //   df = // value=0.8)
base_capacity_factor = // 'time'': '8' //   df = // value=0.4)
base_capacity_factor = // 'time'': '9' //   df = // value=0.1)
base_capacity_factor = // 'time'': '10' //   df = // value=0.1)
base_capacity_factor = // 'time'': '11' //   df = // value=0.4)
base_capacity_factor = // 'time'': '12' //   df = // value=0.8)
This however, only represents one possible operational scenario in which the technology could be applied. In other to add all other possible operational scenarios, different technology 'modes' should be created. For example, 12 models of the SPS technology could be created to fit the described operation above with different starting months. Alternatively, more models could be created to describe all possible operational variations of the SPS plant.
The advantage of this approach is that it is simple to understand and to implement to MESSAGE. For example, it does not require to change the GAMS equations.
The disadvantage of this approach is that there are infinite scenarios in which the SPS plant could operate and the operation will be restricted to the operational scenarios set in the different 'modes'.
I will create an example for the Austrian case study and report the finding to share with the group.

Thank you Julian for the points. The input and output parameters should link the time steps by having a common "time" index. I mean for example:
base_input = { // 'time': '1','time_origin': '1' //}
base_output = { // 'time': '1','time_dest': '12' //}
OR
base_input = { // 'time': '12','time_origin': '1' //}
base_output = { // 'time': '12','time_dest': '12' //}
In the first alternative the technology is active in "time" 1, while in the second it is active in "time" 12. But both alternative does the same task; shifting a commodity from time step 1 to 12.

This issue is closed in favor of PR 190:
https://github.com/iiasa/message_ix/pull/190


Threading in Python

Description 📜
Explanation about Threading in Python.
Fixes #4543

Type of change 📝

[x] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)

Domain of Contribution 📊

[x] Python_Domain

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@adityabisoi sir it would be great if you add the hactoberfest label in this PR.
Thank you


Add new template / theme in template list

Add a link to clean-jsdoc-theme. A clean, responsive template / theme for JSDoc.

Q
A

Bug fix?
no

New feature?
no

Breaking change?
no

Deprecations?
no

Tests added?
no

Fixed issues
N/A

License
Apache-2.0

Hi, do you have any plan to merge this PR! It would be nice if you could merge it. It will help clean-jsdoc-theme to get some recognition. clean-jsdoc-theme is one of the JSDoc theme libraries which is actively maintained. To view this library visit: https://ankdev.me/clean-jsdoc-theme/v4/


Convert register data to string based on data type.

Signed-off-by: zongke<EMAIL_ADDRESS>

@fisherxu please help to run the cicd.


Problem with exporting music

at the beginning I had a problem only with changing the cover of the songs but now I have a problem with downloading music

The download stops in this position and does not move.
Before I had a similar problem, but after downloading music and changing to an mp3 file
changing covers and tags stopped on one of the songs (usually in half).
I only had this problem when I chose to download covers. Only some songs had covers and used playable names such as -.
There is my beatmaps
otiskujawa's beatmaps.txt

This is going to take some time to debug as there's a lot to download.

Yes

Sorry for the silence. I finally got time to look at this.
Importing the beatmaps are a total pain due to the beatmap importing system of the game itself is on a half-working state. The game marked them with totally wrong beatmap IDs which not only causes leaderboards to not show up but broke the app as well.
I may need a few more days before I can tackle the issue.

Ok, I can wait up to several weeks

It seemed the app failed to read one of the mp3s and the whole thing just crashed.
The audio file reader is not made by me so I can't really fix this without looking deep in others work. I'll make it so that it will skip any unreadable mp3 files for now.

Please check out the new release.
You may notice a lot of errors popping up in the log.
If it is about errors relating to the audio file, that's likely due to my dependencies are too outdated. I may update those when I have the time.
If it is about audio files not found, blame osu!lazer. Cause that's not supposed to happen.

Can you add skip option for errors ?
Beatmap 119 AKITO - Sakura Kagetsu has no audio file
NullPointerException : null
Failed processing song: 332 Sigh - Midnight Sun
NullPointerException : null
Failed processing song: 573 Steel Angels - Kiss Kara Hajimaru Miracle
NullPointerException : null
Failed processing song: 648 Dethklok - Duncan Hills Coffee Jingle
NullPointerException : null
Failed processing song: 698 RIZE - Lady Love
NullPointerException : null
Beatmap 73 LM.C - BOYS & GIRLS has no audio file
NullPointerException : null
Failed processing song: 96  - Speed Over Beethoven
InputFormatException : Error while opening file
Beatmap 351167 Kana Hanazawa - Renai Circulation has no audio file
NullPointerException : null
Failed converting song. Keeping ogg format: Let's Dance - Delaney Wolff
it.sauronsoftware.jave.EncoderException : Error while opening codec for output stream #0.0 - maybe incorrect parameters such as bit_rate, rate, width or height
Cannot read album art. Skipping album art for Fire Emblem Theme (Brawl Version) - Shogo Sakai
java.lang.IllegalArgumentException : image == null!

Now the files have been exported but not all have new tags.
Thank you for your help

The app will continue to function despite the errors, meaning it already skips them.
It will just display them just to let you know something went wrong instead of nothing. You can safely ignore them.
About the album arts, this is also normal. Many music players, including Windows itself. Don't like album arts at anything other than 1:1 aspect ratio. They will sometimes outright refuse to read them.

I can't reproduce the missing title and author tags however.

Ok
Finaly i can make .. 32 hours of music in one playlist


Circular import in blueprints tests

https://github.com/terrapower/armi/blob/2571216afadbd0b773555f3dfc3d77b66840fa8f/armi/reactor/blueprints/tests/test_gridBlueprints.py#L26
While reviewing a PR, I (well, vscode's debugger) found this circular import.
=================================== ERRORS ====================================
_ ERROR collecting framework/armi/reactor/blueprints/tests/test_gridBlueprints.py _
ImportError while importing test module 'c:\Users\aalberti\codes\nala\framework\armi\reactor\blueprints\tests\test_gridBlueprints.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
C:\tools\Python\Python39\lib\importlib\__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
framework\armi\reactor\blueprints\tests\test_gridBlueprints.py:26: in <module>
    from armi.reactor.blueprints.tests.test_blockBlueprints import FULL_BP, FULL_BP_GRID
framework\armi\reactor\blueprints\tests\test_blockBlueprints.py:21: in <module>
    from armi.reactor.tests import test_blocks
framework\armi\reactor\tests\test_blocks.py:26: in <module>
    from armi.reactor.blueprints.tests.test_blockBlueprints import FULL_BP
E   ImportError: cannot import name 'FULL_BP' from partially initialized module 'armi.reactor.blueprints.tests.test_blockBlueprints' (most likely due to a circular import) (c:\Users\aalberti\codes\nala\framework\armi\reactor\blueprints\tests\test_blockBlueprints.py)
=========================== short test summary info ===========================
ERROR framework/armi/reactor/blueprints/tests/test_gridBlueprints.py
!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!
==================== no tests collected, 1 error in 0.87s =====================

I fixed a similar bug recently in test_blockBlueprints.py.
Easy to fix.


Does inline mode work?

I tried to implement the inline mode for like FittedSheet original library but it does not work

I’ll check this soon. and i reply on this issue.
thx u. :)

I updated inline mode in FittedSheets-SwiftUI v1.1.1 and finished testing. Can you leave an issue again if you have any problems after trying it?


Attributes missing by included object

My Model:
const store = fortune({
    swot: {
        name: String,
        createdAt: Date,
        createdBy: [ 'user', 'boards'],
        postits: [ Array('postit'), 'swot']
    },
    postit: {
        headline: String,
        text: String,
        createdAt: Date,
        createdBy: [ 'user', 'postits' ],
        swot: [ 'swot', 'postits'],
...

I want to read a swot object with postits included.
URL: ...:8080/swots/JiKOS26X3cgXaCMFrcrY?include=postits
...
"included": [
    {
      "type": "postits",
      "id": "j8SP8VJzdCldUoD0yMCt",
      "relationships": {
        "created-by": {
          "links": {
            "self": "/postits/j8SP8VJzdCldUoD0yMCt/relationships/created-by",
            "related": "/postits/j8SP8VJzdCldUoD0yMCt/created-by"
          }
        },
        "swot": {
          "links": {
            "self": "/postits/j8SP8VJzdCldUoD0yMCt/relationships/swot",
            "related": "/postits/j8SP8VJzdCldUoD0yMCt/swot"
          }
        },
        "guiinfo": {
          "links": {
            "self": "/postits/j8SP8VJzdCldUoD0yMCt/relationships/guiinfo",
            "related": "/postits/j8SP8VJzdCldUoD0yMCt/guiinfo"
          }
        }
      },
      "links": {
        "self": "/postits/j8SP8VJzdCldUoD0yMCt"
      }
    }
...

Why are the attributes missing?

This was most likely a bug that was fixed in this commit https://github.com/fortunejs/fortune/commit/b18c07f8681f1217f991d2a56b6e9ac76961ec93
Please update to the latest version, and let me know if the bug still exists.

Yes, it works. Many thanks.


Code indentation doesn't work for {}

 Issue by shafreenAnfar
Tuesday Sep 03, 2019 at 09:28 GMT
Originally opened as https://github.com/ballerina-platform/plugin-vscode/issues/116

Description:
See below.

Similar report https://github.com/wso2/ballerina-plugin-vscode/issues/47

Similar report https://github.com/wso2/ballerina-plugin-vscode/issues/68


Fixing the TLA_PATH

Extracted from #244
@konnov I'd be interested to know what this fixes, actually, since it looks like the code should be strictly equivalent to what it's replacing :D

Once this is merged in, we'll rebase #255 on to unstable to pick up the change. This is a first, very trivial, inroad to staging #255 into a sequence of logically distinct changesets.


Make job pods run for some time

What type of PR is this?
/kind feature
What this PR does / why we need it:
This is a more realistic scenario. The job controller might do extra syncs before the job actually finishes.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
30s seems appropriate as it is the time it takes for all the jobs to be created in 100 nodes, using the default flags.
Does this PR introduce a user-facing change?:

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/assign @marseel

/lgtm
/approve


upgrade to node.js v7.5

:white_check_mark: coverage increased from 0% to 87.98%

Making an exception while waiting results from TravisCI


Checking the matching of white-space characters after a line break

I tried another search pattern out.
[Markus_Elfring@fedora lokal]$ printf 'else:\n  X\n' | ./comby -stdin -diff -matcher .py 'else::[_\n]:[ ]X' Z

:thought_balloon: May I expect that such a command should generate a simple transformation result?

Will the following test result trigger also any further software development considerations? :thinking:
[Markus_Elfring@fedora lokal]$ printf '    if active:\n  X\n' | ./comby -stdin -stdout -matcher .py ':[ indentation]:[check~if active]::[_\n]:[ ]X' Surprise
    if active:
  X


Handle 500 internal error properly

In certain cases when an error occurred the user will get redirected to a default tomcat error page, which prints a huge error trace. Although this can be disabled, it will show the error in a blank page.  There is no current option to customize this page from a central location.
Therefore some method should be implemented to send the user to a default error page for any kind of 500 internal error occurred.

Do we know in which cases this is being observed and on which web apps

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


namespaces in macros give an error (tuple! macro)

When using namespaced functions in the tuple! macro I got an obscure error.
named!(pub link <&[u8],(&str,&str)>,
    tuple!( parse::common::brackets, parse::common::parentheses )
);

gives the following error:
<nom macros>:2:40: 2:41 error: no rules expected the token `,`
<nom macros>:2 { tuple_parser ! ( $ i , 0usize , (  ) , $ ( $ rest ) * ) } ) ;
                                                      ^

But when I change that code snippet to
use parse::common::{brackets, parentheses};

named!(pub link <&[u8],(&str,&str)>,
    tuple!( brackets, parentheses )
);

It works!
It looks like the macros (at least the tuple! macro) do not play nice with namespaces.

Yes, it's an issue related to the type of argument Rust macros can accept. Either it's an expression (which allows closures, variable names, function names, etc), an ident (limited to named things without prefix) or a path (with the namespaces). I have no way to fix it right now.


Uncaught ReferenceError: resetTriggers is not defined

Using in create-react-app as a fresh install on the latest versions of everything, I'm getting this error. Not sure why it's happening, since the function is actually getting registered. Maybe it's not getting hoisted correctly?
The call is being made from here:
https://github.com/sdecima/javascript-detect-element-resize/blob/master/detect-element-resize.js#L124

We came across the same issue. It would appear this is due to the fact that, as of ES2015 - a function within a conditional if block, is scoped to that block - and thus not available outside. Given that I'm using this in a require, the solution I've taken here is to use my own copy of the detect-element-resize file - with the if (!attachEvent) { conditional commented out.

mrjamesriley's fix is working. Thanks


Spotify clone website

Describe the solution you'd like
I want to add a project on spotify clone website. It gives option to play different songs, we can pause or play the songs or skip to next song as we like. Please assign it to me @Astrodevil
Additional context
It is build using html, css, javascript.
Steps Followed✅
Check the box with [x] If you followed all the steps.

[x] Describe the solution you'd like
[x] Additional context

Code of Conduct✅

[x] I agree to follow this project's Code of Conduct.
[x] I checked the current issues for duplicate problems.
[x] I have gone through project README.

Screenshots

It exists


Task: implement publishing packages in cargo-component.

Blocked by #53

This is now completed (based on the current publish API).


[Metrics] Implement metrics controller

Context

Add initial implementation of GDTCORMetricsController.m (will be refined in following PR)
Stub out unit tests for GDTCORMetricsController.m (will be completed in following PR)
Added new types like GDTCORMetricsMetadata and GDTCORMetricsCounter that are used by GDTCORMetricsController.m

There were no conflicts when rebasing the metrics-main branch onto main so I rebased and pushed that updated branch to the remote. Then I rebased this PR's branch onto metrics-main and pushed that to the remote.

Whoops, I have some more warnings to resolve that did not surface earlier...

Merging this PR now... feel free to leave more comments and I can address in following PRs!


add Gradio Web Demo for cvpr 2022 call for demos

Hi, would you be interested in adding WALT to Hugging Face as a Gradio Web Demo for CVPR 2022 call for Demos? The Hub offers free hosting, and it would make your work more accessible and visible to the rest of the ML community. Models/datasets/spaces(web demos) can be added to a user account or organization similar to github.
more info on CVPR call for demos: https://huggingface.co/CVPR
and here are guides for adding web demo to the org
How to add a Space: https://huggingface.co/blog/gradio-spaces
Please let us know if you would be interested and if you have any questions, we can also help with the technical implementation.

Hi, thanks for the comment. I would be happy to make a demo. I have just finished the final version of the code. Let me know how to integrate it with HuggingFace

@dineshreddy91 great, you can join the organization here: https://huggingface.co/organizations/CVPR/share/RjqpcpbHuMUTqHAbHJcaKIvgrHurkBipkQ and add the gradio demo by following the steps in the organization card or the tutorial here: https://huggingface.co/blog/gradio-spaces, thanks!

Demo added to readme


Added MultiLang | Issue #166

Added Multi Lang Support
https://github.com/user-attachments/assets/9744fe70-a8d0-44d6-8f8b-27930380ad68
Completed #166

Make that drop down fix in footer I think you understand what I am talking

Make that drop down fix in footer I think you understand what I am talking

Yup, got it

@TejasNasre sir, VC ?

At 11

@TejasNasre sir, VC ?

At 11

okay

Sir update ?

Sir update ?

@TejasNasre making it sticky to the footer is not possible due to restricted access with this script.
so we have to compromise a bit with this.
Well, it is looking good at all, a modern floating multi-lang button.

sir solution nikaloo chatgpt karo the ux experience is not good

sir solution nikaloo chatgpt karo the ux experience is not good

sir solution nikaloo chatgpt karo the ux experience is not good

okay sir

sir kuch solution nikla

sir kuch solution nikla

@TejasNasre i will definitely fix this, please allow me some time to just get ride of backlogs, I make sure to complete this.

bhai ny hua kya fix i will close this issue now and assign it to other


Support exception state stack in Py3.7

CPython 3.7 adds a new exception state stack for generators:
https://bugs.python.org/issue25612
https://github.com/python/cpython/commit/ae3087c6382011c47db82fea4d05f8bbf514265d#diff-6b7f39a70f95725eaff0aa0640cc17f6R126
typedef struct _err_stackitem {
    /* This struct represents an entry on the exception stack, which is a
     * per-coroutine state. (Coroutine in the computer science sense,
     * including the thread and generators).
     * This ensures that the exception state is not impacted by "yields"
     * from an except handler.
     */
    PyObject *exc_type, *exc_value, *exc_traceback;

    struct _err_stackitem *previous_item;

} _PyErr_StackItem;

It is used like this:
     gen->gi_running = 1;
+    gen->gi_exc_state.previous_item = tstate->exc_info;
+    tstate->exc_info = &gen->gi_exc_state;
     result = PyEval_EvalFrameEx(f, exc);
+    tstate->exc_info = gen->gi_exc_state.previous_item;
+    gen->gi_exc_state.previous_item = NULL;
     gen->gi_running = 0;

https://github.com/python/cpython/commit/ae3087c6382011c47db82fea4d05f8bbf514265d#diff-23c87bfada1d01335a3019b9321502a0R215
Cython should do the same thing in its coroutines in Py3.7+.
The previous Cython changes in #1956 and the problem description in #1955 are incomplete.

Implemented in #2607.


oneOf does not seem to do the right thing

> Foldable.oneOf (Nothing : [Just "something"])
(Just "something")

> NonEmpty.oneOf (Nothing :| [Just "something"])
[Nothing, (Just "something")]

It's a different oneOf - you're not comparing equivalent operations here. In Foldable terms this is
Foldable.oneOf ([Nothing] :| [[Just "something"]])

since the NonEmpty version is using the Alternative instance of the f, not the g. NonEmpty.oneOf is guaranteed to always give you a result, that's not true/possible in the Foldable case :)
There's some discussion on #1

Well ok, but the function name is very misleading then.  It should probably be changed, right?
E.g. how about allOf? ^^

I don't think it's the function name which is misleading, I think this is just a potentially surprising behaviour due to the fact that you're using two different Alternative instances here. Changing the name to allOf wouldn't make sense for NonEmpty Maybe, for instance:
> NonEmpty.allOf ("hi" :| Just "bye")
Just "hi"

In fact, for the majority of Alternative instances, I think allOf would be more misleading than oneOf.
Whenever you're working with the Alternative Array instance, everything will make more sense if you think of Array a as a "nondeterministic computation" which produces a single result of the type a nondeterministically. Of course you have to squint a bit because it's not actually nondeterministic, but from this viewpoint many of the Array instances make more sense, e.g. f <$> x <*> y can be thought of as the nondeterministic computation which calls f with the results of the nondeterministic computations x and y.  In the case of NonEmpty Array, the NonEmpty.oneOf function is the function which takes a nondeterministic computation which is guaranteed to be able to produce a result and returns one of its potential results.


fix(types): fix generics for refetchPage function

two issues:

The types inferred for the refetch function returned from useInfiniteQuery were wrong, as the input to the function is not of type InfiniteData; It's quite hard to infer the correct type, so making the function generic and defaulting to unknown is the safer choice. Users can now still explicitly provide the generic.

for all other methods on the queryClient, generics were mising and types for pages / allPages were always unknown. Adding a generic to refetchQueries / invalidateQueries / resetQueries that also defaults to unknown fixes that and makes those function easier to work with

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


WIP: Listing Details stories

DSS-421 View Listing Details (Front end)
DSS-438 Listing Actions History (FE)
DSS-439 Legend for Details page (FE)
DSS-437 Listing History (Front end)

TODO:

Paging for more than 12 Listing history tab;le rows.
empty table should have a row with - per cell


distutils DeprecationWarning in Python 3.10

lib/python3.10/site-packages/decouple.py:8: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
from distutils.util import strtobool

I have the same issue

relates to:

https://github.com/henriquebastos/python-decouple/pull/128


🐛 Reduce permission for cluster manager

Summary
Related issue(s)
Fixes #

/lgtm
add a followup to update the hub-hosted mode deploy to set enable-hosted-mode flag.


SublimeLinter's message doesn't fit status bar

.. and it seems there is no  option to read the whole message without increasing editor window size (far beyond the screen width, sometimes).
There might be something useful in the end of message, sometimes. Here's an example:
https://www.evernote.com/l/ACgO99QJbJlJN6iCo8xOinMEwiAr8jyVLxc
So, to conclude:

The message doesn't fit;

Whole message, split on several lines could probably be displayed:

in a tooltip when the mouse is over the status bar
in a popup, when the status bar is clicked
or, maybe, there could be a command available for Cmd+Shift+P, like: "show current error message"

Neither of options is available (or I couldn't find). There should be a solution, I believe. Maybe an option to see full message in the panel.

This is a case for the main llinter package. Open a issue there. https://github.com/SublimeLinter/SublimeLinter3


One more weight increase

Coverage remained the same at 98.17% when pulling d2df355cc060d81d4e19e6519f700ee6f3008ce8 on feature/solr_weights into 0abd73a6fb97d548966426da55c6d91b4dbec7cd on development.

Coverage remained the same at 98.17% when pulling d2df355cc060d81d4e19e6519f700ee6f3008ce8 on feature/solr_weights into 0abd73a6fb97d548966426da55c6d91b4dbec7cd on development.


small typo fix

extra > and broken url

Jenkins job istio/presubmit passed

Jenkins job istio/presubmit passed

Jenkins job istio/presubmit passed


"Auto Replace" option support for RKE2 machine pools

Detailed Description
For RKE1-provisioned clusters, there is currently an option to specify the threshold on how many minutes a node can be unreachable in a node pool before it's automatically replaced:

For RKE2-provisioned clusters, we want the same functionality to be available, probably by adding a checkbox somewhere on the base pool details section highlighted below:

Context
This is needed for RKE2 provisioning parity with RKE1
Additional Details
Backend support is needed: https://github.com/rancher/rancher/issues/35275

ref https://github.com/rancher/rancher/issues/35275

See https://github.com/rancher/rancher/pull/35916#issue-1083643527 for implementation
Support for self healing node pools was added with the pr above. The following fields were added to support this.
related: https://cluster-api.sigs.k8s.io/tasks/healthcheck.html

NodeStartupTimeout *metav1.Duration (Duration of time the node is given to initially become ready before it is replaced.)
UnhealthyNodeTimeout *metav1.Duration (How long a node can be unhealthy before it is replaced)
MaxUnhealthy *intstr.IntOrString (The maximum number of nodes that can be un healthy in a pool, if this is exceeded no nodes are replaced)
UnhealthyRange *string (A range of nodes that can be unhealthy and replaced see https://cluster-api.sigs.k8s.io/tasks/healthcheck.html#unhealthy-range for formatting)

Thanks @paynejacob -- does this mean that UI can being work on this as QA tests the backend?

@gaktive yes qa can test backend and the ui work can start.  Let me know if you have any questions about the api.

@paynejacob It looks like the UI just needs to include the UnhealthyNodeTimeout property when creating the node pool. Do we need to send it with an uppercase U? does it matter? I noticed that we don't capitalize the other properties that we send when creating the node pool.
https://github.com/rancher/rancher/pull/35916/files#diff-e05181e81036cad4014bb4843ea91837fa7dd238dc3434bc4fcc598c56db6428R26

@catherineluse it looks like I accidently capitalized it, I will have a  pr up soon to make it lowercase.

@catherineluse fixed

Setup For Feature Testing
Rancher Version: v2.6-head(e55a04c)
**Steps for Reproduction: **

Start creating a node driver rke2 cluster
go to the advanced settings and you will see the option in the screenshot below:

Result
Was able to successfully view the noted amount post provisioning.
Sanity Checks

successfully provisioned cluster
successfully edited the yaml to change the values before provisioning.
successfully edited the values post provisioning via ui
successfully edited the values post provisioning via yaml

Confirmed with @catherineluse and @gaktive to add release note label.


Remove centos ci badges from repo

Signed-off-by: Josh Pinkney<EMAIL_ADDRESS>What does this PR do?
This PR removes the centos ci badges now that we've gotten rid of those jobs
What issues does this PR fix or reference?
Is it tested? How?

/test ?

/test all


Use LiveData-ktx 'map' & 'switechMap' extensions function replace Transformations function

Replace the map and switchMap static functions in the Transformations class with the map and switchMap extension functions in the lifecycle-livedata-ktx dependency package, which is more in line with the Kotlin syntax.

I signed it!

Thank you for the PR! Added a minor nit.

Hmm, I'm not seeing the line comment.  The nit was sorting the dependencies in alphabetical order.

@tiembo I fix it in this commit(d53ff5e).

Thank you!


Using with neovim's built in LSP

Hi.
I am trying to configure neovim's built in LSP for this server. I have been able to get neovim to detect the language server, but it is crashing on launch.
I am getting the same error as in #36, but this is happening even when I am running the language server inside of neovim.
The neovim log shows same thing as commnad line:

Expand
[ ERROR ] 2021-05-28T14:12:39+0530 ] /usr/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"wolfram"	"stderr"	"[INFO  2021-05-28T14:12:39] Initializing Wolfram Language Server\n"
[ ERROR ] 2021-05-28T14:12:40+0530 ] /usr/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"wolfram"	"stderr"	"[INFO  2021-05-28T14:12:40] Language server is connecting the client through\nsocket.\n"
[ ERROR ] 2021-05-28T14:12:40+0530 ] /usr/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"wolfram"	"stderr"	"\n"
[ ERROR ] 2021-05-28T14:12:40+0530 ] /usr/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"wolfram"	"stderr"	"SocketConnect::zmqexception: \n   A ZeroMQLink exception was thrown - Connection refused (code 111)\n"
[ ERROR ] 2021-05-28T14:12:40+0530 ] /usr/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"wolfram"	"stderr"	"[ERROR 2021-05-28T14:12:40] Cannot connect to client via socket.\n"

I am able to run the server from Mathematica itself using
initfile = "/path/to/lsp-wl/init.wls";
args = {};
Block[{$ScriptCommandLine = Prepend[args, initfile], Quit = Function[{}, Throw[Null]]},
    Catch[<< (initfile)]
];

Although I don't know if it will help, here's my neovim config for lsp-wl.

Expand
local lspconfig = require "lspconfig"
local configs = require "lspconfig/configs"
local util = require "lspconfig/util"

if not lspconfig.lsp_wl then
  configs.lsp_wl = {
    default_config = {
      cmd = {"wolfram", "-script", "/home/manish/lsp-wl/init.wls"},
      filetypes = {"mma", "wl"},
      root_dir = util.path.dirname,
      settings = {}
    }
  }
end

lspconfig.lsp_wl.setup {}

In #36 it was recommended to look at the vs-code client with claim that it would not run into this issue.
Should I try that out to  figure out what I can do?

Sorry, can you please explain? I am a novice.
I see from your vs code client that you require three things: wolfram path, lsp-wl/init.wls path, and the port. Looking at config file of coc-lsp-wl (from where I got the keys) and what you said I did this, which I am not sure is the thing you asked for:
local lspconfig = require "lspconfig"
local configs = require "lspconfig/configs"
local util = require "lspconfig/util"

if not lspconfig.lsp_wl then
  configs.lsp_wl = {
    default_config = {
      cmd = {"wolfram", "-script", "/home/manish/lsp-wl/init.wls"},
      filetypes = {"mma", "wl"},
      root_dir = util.path.dirname,
    -- additions
      settings = {
         lsp_wl = {
            port = 6536,
            wolframExecutablePath = "/home/manish/Software/Mathematica/Executables/wolfram",
            wolframLanguageServerPath = "/home/manish/lsp-wl/init.wls"
         }
       }
   --
    }
  }
end

lspconfig.lsp_wl.setup {}

This is giving the same error as before. I tried to add the port value in lsp_wl.setup {} as well, if that's what you meant by 'settings' but that also gives the same result.
The code above is mainly relying on cmd to do everything it appears, and because that command doesn't work from terminal, it is not working from here either.

I wonder if the socket is already supported by neovim lsp, but from this issue https://github.com/neovim/neovim/issues/11911, it seems not. Godot has made a workaround using nc to redirect the socket reads/writes as the example linked in that issue. In that case, you have to manually start a wolfram language server externally by wolfram -script /path/to/lsp-wl/init.wls --tcp-server=<port>, and let neovim connect to that server.

Thanks, it worked once but somehow now its not working. I will check and report later !

Thank you.  I checked and have got the hover and signature help working fine. The autocomplete however is not working for now.
Here's my config file:
local configs = require "lspconfig/configs"
local util = require "lspconfig/util"

if not lspconfig.lsp_wl then

  configs.lsp_wl = {
    default_config = {
      cmd = {"nc", "localhost", "6536"},
      filetypes = {"mma", "wl"},
      root_dir = util.path.dirname,
      settings = { }
    }
  }
end

which I have placed in nvim-lspconfig/lua/lspconfig/lsp_wl.lua and I do
lua require'lspconfig'.lsp_wl.setup{on_attach=require'completion'.on_attach}

in my init.vim.
In mathematica I do as you say:

!wolfram -script ~/path/to/lsp-wl.wls --tcp-server=6536
LspStart

to get going.
For autocomplete, I use completion-nvim.. and I tried to see why its not working.
I looked at the default bashls config for comparison (below), which looks surprisingly the same, yet completion works for it. That makes me think this is a problem with completion-nvim - I just wanted your thoughts on that. I will raise issue with the plugin in that case.

Expand
local configs = require 'lspconfig/configs'
local util = require 'lspconfig/util'

local server_name = "bashls"

configs[server_name] = {
  default_config = {
    cmd = {"bash-language-server", "start"};
    cmd_env = {
      -- Prevent recursive scanning which will cause issues when opening a file
      -- directly in the home directory (e.g. ~/foo.sh).
      --
      -- Default upstream pattern is "**/*@(.sh|.inc|.bash|.command)".
      GLOB_PATTERN = vim.env.GLOB_PATTERN or "*@(.sh|.inc|.bash|.command)"
    },
    filetypes = {"sh"};
    root_dir = util.path.dirname;
  };
  docs = {
    description = [[
https://github.com/mads-hartmann/bash-language-server

Language server for bash, written using tree sitter in typescript.
]];
    default_config = {
      root_dir = "vim's starting directory";
    };
  };
};

I try your config with nvim-cmp, the autocomplete work fine.
For your information, here's my config for reference:
local enhance_attach = function(client,bufnr)
  api.nvim_buf_set_option(bufnr, "omnifunc", "v:lua.vim.lsp.omnifunc")
end

vim.notify("Config lsp_wl")
configs.lsp_wl = {
  default_config = {
    cmd = {"nc", "localhost", "6536"},
    filetypes = {"mma", "wl"},
    root_dir = util.path.dirname,
  }
}

lspconfig.lsp_wl.setup {
  on_attach = enhance_attach
}


Pass container labels in endpoint create & join

We need this patch in order to support the long standing request :
https://github.com/docker/libnetwork/issues/161
Signed-off-by: Madhu Venugopal<EMAIL_ADDRESS>

Just for reference, this relates to https://github.com/docker/docker/issues/6743

LGTM.

LGTM

I'm temporary NOT LGTM until I give this more thinking. Especially, using labels as a vehicle for driver specific network options makes me feel like a LABEL instruction in a Dockerfile could impact the running environment of a container down the road.

@icecrime the idea is to send the container labels to libnetwork core and have the core filter out the labels based on the namespace the label belong to. For example, we should not pass all the labels to every driver. But if there is a driver named foo with a registered label namespace com.foo.xyz that is supposed to perform a specified action and if the user-label's prefix matches that particular driver's label-namespace, then that driver will get that specific label.
For any well-known labels supported by libnetwork core (namespaced by com.docker.libnetwork, we can filter them out entirely from the call to the driver & use well-known (non-label-based) configuration mechanism to pass the information to driver if required.
Does that address your concern ?
I see this as a genuine use-case for labels and provides flexibility in Docker core to not overload with many use-case specific configurations.

@mavenugo Yes, but users can pass in labels in a Dockerfile. This would in effect allow a Dockerfile writer to affect the networking of containers created from it.
Images should not be encoded with this information.

@cpuguy83 @icecrime okay. will it be okay if we filter out the image labels from this list and make it purely a container labels driven operation ?

I have to say I'm uncomfortable, I just don't feel labels are the proper vehicle for that. It's one thing to say the container is blue, and a totally different thing that it should be in this or that subnet.
Furthermore, I could imagine actually being useful at some point in Docker to have a "rule mechanism" to bind label "blue" with "subnet X" (thus properly decoupling the build time and run time considerations). With this patch in, we make it impossible.

@icecrime in the new CNM model, container/sandbox itself is not specified with the ip-address. Rather it is the service endpoint that carries this info. Containers can freely attach and detach to the service endpoint (as explained by Solomon in #13441.
I guess the reason you are uncomfortable is that we create a default service endpoint for a container during the docker run for convenience & ease of use. Lets discuss this further in the context of #13441 and see if we can agree on a solution.

ping @icecrime just trying to find a way to allow the users to get the functionality that I explained earlier. Also, since the container labels are made available via the APIs, drivers (or) any other external system is free to get the exact same info without a more formal approach as defined in this PR.
Should we take the same approach as --mac-address and go with a more static and well-defined option such as --ip to set static IP instead of labels ?

I think I agree with @icecrime, I would prefer keeping labels portable and user-specific (eg. color:blue, prod:true), and have out-of-band matching of container label to driver config. That matching could be done in the driver itself or, even better, as a configurable feature of Docker. So a sysadmin could configure the daemon with css-like selectors eg. "if color == blue && prod == true { drivercfg.netmask = <IP_ADDRESS>/24"
We could make that selector a lua scripttoo, would be cool :)
Also: image labels should NOT translate to container labels. They are not the same thing.

I'm not sure where this PR is headed but if it does head down the "merge" path can we please get some testcases?  With the recent regressions that have been found (e.g. by @ibuildthecloud ) I think we need to push harder for more testing - especially w.r.t. what the users sees.  I love how @vdemeester is working hard on adding tests for the existing code!

At the end of the day, we need to be able to pass additional information to network plugin  drivers via docker api. Each container/driver combination can have different networking needs, and it is not really possible to make the configuration static. Without the ability to pass additional context to network drivers via docker api, the networking plugin drivers become useless.
If the "labels" support for network drivers don't get merged, then we will really need alternatives.

@shykes @icecrime one of the main reasons of opening this PR was to support a request that has come up multiple times on static IP Address assignment to a container by the user via the docker run command. It is certainly not declarative.
We decided to use Labels for this because we were not inclined towards introducing yet another static flag with a particular semantics assigned to it such as --ip or --ipv6 and make it docker engine managed flag. But we would like to avoid such very prescriptive configuration on docker run and we are forced to provide support for many cases and it can quickly get confusing on various situations. Having a Label makes it a contract between the Driver and the user and the Engine itself is not directly involved.
Coming back to the declarative & out-of-band matching  example : "if color == blue && prod == true { drivercfg.netmask = <IP_ADDRESS>/24", how exactly will the driver get hold of these declarative labels ?
The example of assigning an IP to a container is real-time and an out-of-band matching may not work.
At this point, I think it would be better to keep these discussions separate and give a simpler and direct answer to the user requests for assigning IP/IPv6 addresses. Once we unblock that, we can have more indepth discussion on making a declarative approach and eventually a scriptable  interface as you suggested.

What do we do regarding this PR? Was there out-of-band discussion that lead to some conclusion or is it still floating? Thanks!

Closing it as it seems the conversation is really about specifying IP and MAC addresses now, not about labels anymore.


[Bug]  skywalking-skywalking-helm-oap  not started(keep restarting)

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
OAP server (apache/skywalking)
What happened
helm install "${SKYWALKING_RELEASE_NAME}"   oci://registry-1.docker.io/apache/skywalking-helm   --version "${SKYWALKING_RELEASE_VERSION}"   -n "${SKYWALKING_RELEASE_NAMESPACE}"   --set oap.image.tag=9.6.0   --set oap.storageType=elasticsearch   --set ui.image.tag=9.6.0
What you expected to happen

The logs show that this will restart
How to reproduce
null
Anything else
No response
Are you willing to submit a pull request to fix on your own?

[ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

oap.log
oap (2).log


Fix RichTextLabel context menu not customizable

Follow-up #72167.
This PR makes the RichTextLabel context menu API consistent with LineEdit and TextEdit.

Thanks!


GitHub Enterprise Server Monitor Dashboards appear out of order in Nav Menu

Describe the bug
Within GitHub Enterprise, the Monitoring dashboards appear in a top to bottom order all on one page. However in Splunk, the different segments are split into separate dashboards. With that, they appear in alphabetical order in the navigation menu unlike the order they appear in GitHub Enterprise.
To Reproduce
Steps to reproduce the behavior:

Go to 'GitHub App for Splunk'
Click on 'Enterprise Server Monitor' navigation menu
See error

Expected behavior
It would be expected that they appear in the same order in the navigation as they do on the Monitor page in GitHub Enterprise Server.

This has been fixed and will be included in the next release.


msys2 windows workers: "fatal error LNK1101: incorrect MSPDB120.DLL version"

Building some targets (e.g. grpc_csharp_ext.sln) only works on cygwin based windows workers (the ones without the "newwindows" label).
the build only fails when ssh-ed using a private key to the worker (which is what jenkins is doing).
Error message is:
LINK : fatal error LNK1101: incorrect MSPDB120.DLL version; recheck installation of this product
The reason is that windows authentication model has problems setting up user profile when  authenticating user without a password. For cygwin based Jenkins workers, this is solved by cyglsa-config (but killing Jenkins jobs under cygwin doesn't work properly).

see https://cygwin.com/cygwin-ug-net/ntsec.html, section "Switching the user context without password, Method 2: LSA authentication package"

So I'm not convinced this is the rootcause, by the way. I think the investigation needs to go in a slightly different direction than user authentication. My gut feeling is that under cygwin using their SLA script is causing a side effect in the environment that eventually results in that DLL working properly.

Yeah, might be. but the cyglsa-config script seem to be our biggest hint so far anyway.

I think this really has to do with switching user profile:
when ssh'd through msys2 sshd the current username is USERNAME=sshd_server'  but it should be USERNAME=jenkins` (reported correctly by cygwin sshd).

Quoting from https://cygwin.com/cygwin-ug-net/ntsec.html:
A more important problem is that using NtCreateToken is not sufficient to create a new logon session for the new user. What does that mean? Every logon usually creates a new logon session. A logon session has a couple of attributes which are unique to the session. One of these attributes is the fact, that Windows functions identify the user domain and user name not by the SID of the access token owner, but only by the logon session the process is running under.
*This has the following unfortunate consequence. Consider a service started under the SYSTEM account (up to Windows XP) switches the user context to DOMAIN\my_user using a token created directly by calling the NtCreateToken function. A process running under this new access token might want to know under which user account it's running. The corresponding SID is returned correctly, for instance S-1-5-21-1234-5678-9012-77777. However, if the same process asks the OS for the user name of this SID something wierd happens. For instance, the LookupAccountSid function will not return "DOMAIN\my_user", but "NT AUTHORITY\SYSTEM" as the user name.
You might ask "So what?" After all, this only looks bad, but functionality and permission-wise everything should be ok. And Cygwin knows about this shortcoming so it will return the correct Cygwin username when asked. Unfortunately this is more complicated. Some native, non-Cygwin Windows applications will misbehave badly in this situation. A well-known example are certain versions of Visual-C++.*

http://stackoverflow.com/questions/17978984/running-msvc-via-cmake-using-ssh-to-cygwin-works-if-using-a-password-to-login-bu

Eventually I used an ugly workaround and made user "jenkins" privileged user that is also running sshd.
That way, we are not switching user sessions without password and Visual Studio is not failing with the original error anymore.
ssh configure script is here: https://gist.github.com/jtattermusch/9c8d340120d820f482fa, all the necessary steps are documented in go/grpc-jenkins-setup
I create a new windows worker snaphot and recreated all workers using that snapshot.
Currently all jenkins windows workers should be identical and seem to be working fine (except missing APPDATA variable is breaking grpc_credentials_test, but I'm going to fix that).

For now, I manually added APPDATA variable pointing to C:\Users\jenkins\AppData\Roaming in configuration for all workers (on Jenkins).

Seems there are still issues with the new windows workers.

The tweaks I tried to do didn't really help and windows workers were having weird issues (VS2015 not working, missing env variables, etc), so I switched all win workers to username and password authentication.
to allow properly switching the user session, I ran passwd -R on all workers, that has finally resolved the "incorrect MSPDB120.DLL" problem for C# builds.

https://cygwin.com/cygwin-ug-net/ntsec.html: "Switching the user context without password, Method 3: With password"


Rework sitemaps to align better with google and algolia needs

Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 78a42c812b241f883515fb11a761549e689e0a71
https://deploy-preview-2864--calico.netlify.com

Deploy preview for calico ready!
Built with commit 78a42c812b241f883515fb11a761549e689e0a71
https://deploy-preview-2864--calico.netlify.com


Error: "This method requires an authenticated users access token"

Please help. I'm getting the message below when testing when testing "example/index.php"...
Fatal error: Uncaught exception 'Exception' with message 'Error: _makeCall() | users/self/media/recent - This method requires an authenticated users access token.' in /home/whychoos/public_html/feeds/master/src/Instagram.php:441 Stack trace: #0 /home/whychoos/public_html/feeds/master/src/Instagram.php(158): MetzWeb\Instagram\Instagram->_makeCall('users/self/medi...', true, Array) #1 /home/whychoos/public_html/feeds/master/example/success.php(35): MetzWeb\Instagram\Instagram->getUserMedia() #2 {main} thrown in /home/whychoos/public_html/feeds/master/src/Instagram.php on line 441

You're making a call that doesn't include an access token. If you have an access token (as a result from the OAuth call), you can set it via $Instagram->setAccessToken($token)

Actually, I just ran into this issue... after digging in, the getUserMedia() method appears to only use the auth if you're getting your OWN media... which doesn't make sense, because some users profiles are private and require the auth key (hence the thrown error). I'm submitting a pull request that will resolve this issue.

Merged pull request cf6bb9fdff7ae2b79eee760901b670662faf1332. Thanks for your contribution!


Concurrent worker update from worker process and scheduler

The scheduler uses AssignTask to assign a task to a worker, which updates the worker record in the database. Simultaneously, the worker is sending updates to the server, updating the same worker record. This may cause write conflicts, where both systems are updating the same record. The system prevents this conflict using record versioning.
The problem is, in the test suite, when the updates are happening very frequently with a small amount of workers, these conflicts are much more likely and are causing the tests to take a long time and even fail. Also, it's possible this could arise in real-world scenarios.

This has been observed happening in real-world scenarios where lots of tasks are concurrently writing updates


Add wait_time flag for Travis config

When the test takes more than 10 minutes without producing any output,
Travis job will fail. This pr makes Travis to emit a short log for
every minute for 30 minutes (Our current travis job runtime is 30~35min)
https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received

Codecov Report

Merging #4057 into master will increase coverage by 0.13%.
The diff coverage is n/a.

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

Integration test with travis_wait 45 was finished in 34min. Since it does not wait for 45 minutes, I will bump this up to 50minutes because it does not give us any harm.


"Stăncuța" is listed in the drop-down menu

Description:
"Stăncuța" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Stăncuța" in the search bar.

Expected results
The location is listed in the drop-down menu as "Stăncuța, Brăila (Stăncuța)".


Glimmer Lights Connect to Fences and Walls

I understand the glimmer crystal/lights are from Embers and not Soot, but since Soot seems to be the main place to find updates and bugfixes for Embers since it was discontinued, I'm going to post it here. Anyway, as it says in the title, lights from the glimmer crystal connect to fences and walls, which seems odd.

this is a problem on basicly all embers blocks. i would like to crusade for proper BlockFaceShape integration in all mods but as Embers itself isn't actively developed and the fix in Soot would require overrides for every single block in Embers, I will not put this fix in Soot. (But another solution may present itself sometime in the future)

https://github.com/DaedalusGame/EmbersRekindled/issues/1