Non-deterministic routing danger warning

Whenever Gastby generates my site, I get this error:

warn Non-deterministic routing danger: Attempting to create page: "/auth/callback/", but page "/auth/callback" already exists
This could lead to non-deterministic routing behavior

Could be caused by the createPages handler in gatsby-node.js:
exports.createPages = ({ actions }, options) => {
  const { createPage } = actions;
  const { callbackPath } = options;

  createPage({
    path: callbackPath || "/auth/callback",
    component: path.resolve(`${__dirname}/src/pages/auth/callback.tsx`),
  });
};

I'm thinking this code may be redundant. Gatsby seems to be already creating this page because it's in the pages directory. I'm not sure the original intention of this code, though, so I could be wrong.

Ahh, re-read the docs, and I think I see the problem.
The default callbackPath is /auth/callback, but every other page on my site has a trailing slash, so if I change it to /auth/callback/, the warning disappears.
I think it might help to change the default to have a trailing slash, though that could be a breaking change for people, so I'm not sure.

I didn't want to monkey patch the package so I've made sure that all pages generated from gatsby-node.js don't end with a trailing slash, and I still see the warning. @epilande, any ideas?

@dandv, I didn't have to monkeypatch. It's actually something you can change with a legit setting in gatsby-config. Here's what I have:
{
    resolve: "gatsby-theme-auth0",
    options: {
        domain: process.env.AUTH0_DOMAIN,
        clientID: process.env.AUTH0_CLIENT_ID,
        redirectUri: process.env.AUTH0_CALLBACK_URL,
        callbackPath: "/auth/callback/"
    }
},

I set callbackPath like you described and the warning went away, but what I was saying earlier was that I didn't have any routes generated that ended in a trailing slash, so it looks like that's not the cause of the warning.
BTW, did you figure out how to redirect to the pre-login URL from the callback? #176

Interesting that your pages don't have trailing slashes. Are you generating them yourself or letting Gatsby do it for you? I'm doing the latter, and they seem to have the trailing slash by default, but maybe I set something without realizing it.

Looks like you figured out the answer to #176. I do indeed have AUTH0_CALLBACK_URL defined in my environemnt variables.

I'm generating them from gatsby-node by implementing createPages,
actions.createPage({
  path: `/posts/${slug}`,
  ...
});

Does Gatsby generate other pages and let you control the trailing /?

Not out of the box that I know of but looks like there's a plugin for it:
https://www.gatsbyjs.org/packages/gatsby-plugin-remove-trailing-slashes/

@dandv, I didn't have to monkeypatch. It's actually something you can change with a legit setting in gatsby-config. Here's what I have:
{
    resolve: "gatsby-theme-auth0",
    options: {
        domain: process.env.AUTH0_DOMAIN,
        clientID: process.env.AUTH0_CLIENT_ID,
        redirectUri: process.env.AUTH0_CALLBACK_URL,
        callbackPath: "/auth/callback/"
    }
},

This removed the warning for me as well. It should probably be the default behavior.

Hey I wanted to elaborate some @devuxer @dandv ...
For me, as I just said, adding the / removes the warning, but doesn't fix the behavior in production at Netlify (see #217 ). As discussed, I updated my gatsby-config.js file to add the optional line as follows, with the trailing slash.
    {
      resolve: `gatsby-theme-auth0`,
      options: {
        domain: process.env.AUTH0_DOMAIN,
        clientID: process.env.AUTH0_CLIENT_ID,
        redirectUri: process.env.AUTH0_CALLBACK_URL,
        // Optional fields:
        // audience: process.env.AUTH0_AUDIENCE,
        // responseType: process.env.AUTH0_RESPONSE_TYPE,
        // scope: process.env.AUTH0_SCOPE,
        callbackPath: "/auth/callback/",
      },

The other changes I made were updating the environment variables (the AUTH0_CALLBACK_URL) to have the trailing slash for both local development (.env.development) and the Netlify environment variables for production. I updated both to read ${url}/auth/callback/ with the trailing slash after the correct URL.
In the Netlify environment variables, I had to trigger a deploy for the site after updating it. I did Trigger deploy --> Clear cache and deploy site but that may have been optional. Specifically, I probably should have updated the environment variables first, then committed the updated gatsby-config.js file in order to have a working deploy from the start. But even so, Netlify still doesn't work. We're discussing a Netlify fix over in #217 at the moment.
I also added trailing slashes (/auth/callback/) to my Auth0 Callback URLs (both in environment variables and in the Auth0 configuration) to be safe & consistent, but that seems optional. For deployment at Netlify, they have a "Pretty URLs" feature hidden inside of Asset Optimization that forces trailing slashes. Reference: https://docs.netlify.com/routing/redirects/redirect-options/#trailing-slash Turning on Pretty URLs didn't make a difference in fixing my production app, but I did it for consistency purposes.
Overall, I think it's probably a better default to have the trailing slash as the callback path (/auth/callback/ instead of /auth/callback as it is now). Of course, we still need to get Netlify working out of the box (see #217 ). If I can figure it out, I'll submit a pull request to @epilande ... because I really love the idea of this plugin.

@devuxer @dandv @epilande Also I tried the gatsby-plugin-remove-trailing-slashes along with Pretty URLs turned on. That meant the redirect went to /auth/callback in production instead of /auth/callback/ in production, though I did get the non-deterministic routing error message on build again about that path. It still didn't resolve #217 ...
Here's the site https://friendly-owl-450cb.netlify.app/auth/callback or https://friendly-owl-450cb.netlify.app/


Automatically detect MSVC tools on Windows via vswhere

Simmilar to #11495, except it uses vswhere to detect the installation path. vswhere is looked up in the following sequence:

The same directory as the compiler currently running
The default location (%ProgramFiles(x86)%\Microsoft Visual Studio\Installer\vswhere.exe), valid since VS2017 15.2
PATH

This does not fall back to the COM interfaces. Everything other than retrieving the list of Visual Studio installations is exactly the same. Note that as long as a sufficiently new version of Visual Studio is installed, everything will work even if we don't redistribute vswhere.exe ourselves.

Hm so this cuts off less than half of the size of the previous pull request, so I'm still not sure about it, when this is the alternative (and I know it could be directly ported to Crystal): https://github.com/neatorobito/scoop-crystal/blob/3b33560/scripts/crystal.ps1

I think it's best not to simply invoke the MSVC developer command prompt on behalf of the user even if it is only for a single linker command. Explicitly capturing the paths ourselves means the linker command as outputted by the --verbose compiler flag also works outside the MSVC prompt.

I see. That's a good point.
Then we'll need to keep considering this option. Or even the other pull request as the main option.
I said

this cuts off less than half of the size of the previous pull request

If we're already going to all these lengths, may as well also reimplement vswhere :/

Yeah, considering a big part of the diff to #11495 is lib bindings, I think it would be acceptable to go the little extra way so we don't depend on vswhere.exe.
On the other hand, is there an actual downside of using vswhere.exe?

On the other hand, is there an actual downside of using vswhere.exe?

Certainly! You need to most likely ship it with Crystal

Or, well, maybe not, according to the main post of this PR

I understand it is available in %ProgramFiles(x86)%\Microsoft Visual Studio\Installer\vswhere.exe if you have VS 2017 or newer installed. So we shouldn't have to ship it. I'm not even sure if anything below VS 2019 works with Crystal.


fix(imp module): replace deprecated module

imp module has been replaced by importlib in python 3.12 and is not available anymore
Maintainers will complete the following section

[ ] Commit messages are descriptive enough
[ ] Code coverage from testing does not decrease and new code is covered
[ ] Python type annotations added to new code
[ ] JSON/YAML configuration changes are updated in the relevant schema
[ ] Changes to metadata also update the documentation for the metadata
[ ] Pull request has a link to an osbs-docs PR for user documentation updates
[ ] New feature can be disabled from a configuration file

This should be 1-1 replacement, but for some reason tests are 7 times slower, something is off here


Angular supports DDC (DartDevCompiler)

We will initially only support DDC. I've changed the title to reflect that.

We have reports 3.0.0 already supports this, although the SDK only will formally support DDC externally in 1.24.0. Closing the issue since there is not more work from our end.


'Collapse Menu Entries' Plugin

Collapses duplicate context menu entries on ground items, reducing clutter. Ten occurrences of "Take Spade" becomes one occurrence of "Take Spade (x10)".
It sort of blew up on /r/2007scape!

if this is exclusively for ground items, it should be in the ground items plugin

#3570 does this in a better way


Add links to the github repo for azure sdk for go

for better discoverability adding links to the GitHub repo.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-go-sdk-conceptual/includes/azure-sdk-go-get.md
:white_check_mark:Succeeded
View

azure-go-sdk-conceptual/index.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@sptramer, @joshgav  can you review please

LGTM I like it :+1:

awesome, thanks!


the iPod has the below problem

in the  "AFURLRequestSerialization.m" file, we met a problem with return the nil object in the iPod.
NSString * urlStr =  [[NSURL URLWithString:URLString relativeToURL:self.baseURL] absoluteString];

-- (AFHTTPRequestOperation *)HTTPRequestOperationWithHTTPMethod:(NSString *)method
                                                     URLString:(NSString *)URLString
                                                    parameters:(id)parameters
                                                       success:(void (^)(AFHTTPRequestOperation *operation, id responseObject))success
                                                       failure:(void (^)(AFHTTPRequestOperation *operation, NSError *error))failure
{
    NSError *serializationError = nil;
    //NSString * url =  [[NSURL URLWithString:URLString relativeToURL:self.baseURL] absoluteString];
    NSMutableURLRequest *request = [self.requestSerializer requestWithMethod:method URLString:[[NSURL URLWithString:URLString relativeToURL:self.baseURL] absoluteString] parameters:parameters error:&serializationError];
    if (serializationError) {
        if (failure) {
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wgnu"
            dispatch_async(self.completionQueue ?: dispatch_get_main_queue(), ^{
                failure(nil, serializationError);
            });
#pragma clang diagnostic pop
        }

        return nil;
    }

    return [self HTTPRequestOperationWithRequest:request success:success failure:failure];
}

the url is nil, this lead to the request may not be right.

This is probably best suited for stackoverflow. We use Github to track bugs and feature requests.
:beers:

From our Contribution Guidelines
Asking Questions
We don’t use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you’ll be more likely to quickly solve your problem, and you’ll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


[WIP] OG-USA handling of UBI

This PR updates OG-USA to properly handle the UBI estimates from Tax-Calculator.  Background discussion for this PR can be found in Issue #572.

Codecov Report

Merging #573 into master will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #573      +/-   ##
==========================================
+ Coverage   84.18%   84.30%   +0.12%
==========================================
  Files          47       47
  Lines        6657     6708      +51
==========================================
+ Hits         5604     5655      +51
  Misses       1053     1053

Impacted Files
Coverage Δ

ogusa/get_micro_data.py
98.21% <ø> (ø)

ogusa/output_tables.py
88.88% <100.00%> (+3.55%)
:arrow_up:

ogusa/tax.py
97.98% <100.00%> (ø)

ogusa/tests/test_output_tables.py
93.93% <100.00%> (+0.60%)
:arrow_up:

Continue to review full report at Codecov.

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

@jdebacker and @MaxGhenis. The correct way to incorporate UBI from Tax-Calculator into OG-USA is through the effective tax rate function and not through the marginal tax rate functions. The household budget constraint is the following:

UBI will enter the OG-USA model exclusively through the tax liability term T_{s,t}, which is the last term in the budget constraint. The net tax liability is defined as an effective tax rate because it is easier to estimate an effective tax rate function tau than it is to estimate a net tax liability function T. Let T_0 be the net tax liability without UBI. Then T_0 is defined as the following,

where tau_0 is the effective tax rate without UBI, x is total labor income, y is total capital income, and x + y is total income. The difficulty is that x + y does not include other types of income like UBI. So we need a new effective tax rate function tau_1 that incorporates UBI so that the net tax liability T_1.
The simplest conceptual specification is to have the new net tax liability with UBI T_1 equal the original tax liability minus the UBI payment.

But it is difficult to get the units right of UBI dollars from the data to model units. Note that the first term on the right-hand-side of the equivalence is just T_0 the net tax liability without UBI. So what we want to do is specify a new effective tax rate tau_1 such that multiplying it by model income x + y without UBI income gives the same net tax liability T_1 with UBI specified above.

The effective tax rate is defined in line 188 of get_micro_data.py as the following:
'etr': calc1.array('combined') / calc1.array('expanded_income')

The only change that needs to be made is to subtract UBI from the numerator, and not the denominator.
'etr': (calc1.array('combined') - UBI) / calc1.array('expanded_income')

I agree with most of what @rickecon  said, however, I do think expanded_income needs to be modified in the definition of the ETR to correctly map to the income definitions in OG-USA.
In particular expanded_income != x+y, but rather expanded_income = x+y+UBI+consumption value of other benefits in TC.
Therefore, for the ETR to correctly reflect the tau_1 above, we need to defined it as:
'etr': (calc1.array('combined') - UBI) / (calc1.array('expanded_income')  - UBI - other benefits from TC)

@jdebacker . I think you're right. The denominator in the ETR calculation in get_micro_data.py needs to be as similar as possible to our denominator in the OG-USA model, which is only labor income and capital income.

@rickecon This PR is now ready for review.


After Build Finished:  Node version 20.9.0 is not supported. Please use one of the following versions 12, 14, 16, 18.

Failed after build of Azure static web service.  Not a lot of detail in the error.
---End of Oryx build logs---
Try to validate location at: '/bin/staticsites/0eb197a4-d222-477e-a851-c0c1c72645fb-swa-oryx/app/build'.
Finished building app with Oryx
Node version 20.9.0 is not supported. Please use one of the following versions 12, 14, 16, 18.
Additionally I searched the entire code base for >=20   It does not exist.

To specify that your project should use Node.js 18, you can set the "engines" field in your package.json file like this:
"engines": {
  "node": "18.17.1"
},

Thank You for the help. Greatly appreciated
I did have the engines in the package.json but it was ignoring it because it was in the wrong section.   Just so anyone can benefit.  Here is a example of where it goes.
"devDependencies": {
"typescript": "^5",
"@types/node": "^20",
"@types/react": "^18",
"@types/react-dom": "^18",
"autoprefixer": "^10.0.1",
"postcss": "^8",
"tailwindcss": "^3.3.0",
"eslint": "^8",
"eslint-config-next": "14.0.3"
},
"engines": {
"node": "18.17.1"
}


Progressively get rid of {eval-rst} blocks

We have several {eval-rst} blocks here and there, which are a way to keep reST features that are unsupported in MyST. We should progressively get rid of them, and if there are any remaining features that are unsupported, link to the corresponding https://github.com/executablebooks/MyST-Parser/ issue.

This would be nice to migrate as it will remove all dependence on .rst

Hello there,
can I contribute solving this issue?

Hello @brivadeneira , sure! ☺ Start with a small pull request, maybe modifying only one file, and we'll see from there.

Hello @brivadeneira , sure! relaxed Start with a small pull request, maybe modifying only one file, and we'll see from there.

https://github.com/poliastro/poliastro/pull/1330

#1330 was merged! @brivadeneira you're welcome to keep working on this issue :) If other readers come by and want to help, make sure you read #1330 to see what is expected for this work.


hotkey-event-Bug of GitHub custom hotkeys

For example:

t t
	{root}/notifications

Please note that:  I defined two t .
keydown t and keydown t again, This is normal, no problem.
But if this is the case, the problem/bug is exposed:
keypress(e.g. holding⩾100ms) t will also trigger, this feels bad.
keypress ≠ keydown or keyup
So, I think that since there are 2 t is defined, it should be triggered by pressing 2 times t.

That means you cannot use keydown and  keypress at the same time, For example:👇
Linkclump
https://chrome.google.com/webstore/detail/lfpjkncokllnfokkgpkobnkbkmelfefj

Hi @dnknn!
This userscript manages adding hotkey links to the page within the defined scope. GitHub's hotkey code handles the keyup/keydown events. Please open an issue in their repo to let them know about the problem. Thanks!

👌

Since github there replied, I also closed this issue, thank you!


How to find html attribute whose value begins with a number?

Say the html document I'm parsing has the attribute class="1post". Is there a way to use the Find() function for this class? If I run doc.Find(".1post"), I get this error:
panic: expected identifier, found 1 instead
This might be a cascadia issue. Do you know of a way around it?

In theory a class name cannot start with a number, except if the number is escaped. So I guess cascadia is doing the right thing here. You may want to try with the escaped syntax, I haven't tested it, but you can read the details here:

Sorry I guess I forgot to reply. I didn't test this, but I found a workaround (for my case, at least). The page I was parsing had something like span class="apost 1post", so I just did this:
doc.Find("span").Each(func(i int, s *goquery.Selection) {
    class, _ := s.Attr("class")
    switch class {
    case "1post":
        fmt.Println("got it")
    default:
        fmt.Println("wrong")
    }
})

Thanks for the reply, that may help others with the same problem.


Add SPA support for playground image

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

NONE

Nice.
/approve


didCloseWithTrailingMetadata:error: not called on turn off wi-fi (objc grpc)

(gRPC 1.31.0)
I make GRPCCall2
        let requestOptions = GRPCRequestOptions(host: configuration.address, path: method, safety: .default)
        let callOptions = GRPCMutableCallOptions()
        callOptions.retryEnabled = false
        callOptions.timeout = 0
        return GRPCCall2(requestOptions: requestOptions, responseHandler: handler, callOptions: callOptions)

after I invoke call
let data = try request.serializedData()
call.start()
call.writeData(data)

of course it's work good with success or failure response but  if I turn off wi-fi I don't get error about 30 second. But if I turn on wi-fi I get error immediately
Right now I use SCNetworkReachability for detect switch network but I think it's not right?

I think it's because I'm trying on simulator


Fix OracleFMWInfrastructure image

Improve Dockerfile to make image smaller
Fix install of libaio library
Remove VOLUME directive
Correct README to indicate to use -v to map domain home to host volume

Fixed #1292

Approved, but I suggest updating line 100 of the README to include an actual example usage of the -v option (for people who don't use our scripts or use an automation/scheduling engine).


ImportError: No module named netifaces

Kali 17.3 X64  $$  python 2.7.14+
root@kali:~/Empire# ./empire
Traceback (most recent call last):
File "./empire", line 13, in
from lib.common import empire, helpers
File "/root/Empire/lib/common/empire.py", line 29, in
import helpers
File "/root/Empire/lib/common/helpers.py", line 53, in
import netifaces
ImportError: No module named netifaces

@Skyw3lker did any errors occur during the install?

@Skyw3lker also try a standalone install with pip.

Experiencing the same error. I did not notice any errors during install, and a stand alone pip says:
Requirement already satisfied: netifaces in /usr/local/lib/python2.7/dist-packages

@Dockmaster87 what version of python is being used with Empire??

@xorrior Python 2.7.14+ on Kali

Full Error:
Traceback (most recent call last):
File "./empire", line 13, in
from lib.common import empire, helpers
File "/root/Tools/Empire/lib/common/empire.py", line 29, in
import helpers
File "/root/Tools/Empire/lib/common/helpers.py", line 53, in
import netifaces
ImportError: No module named netifaces

@xorrior After I did the pip uninstall and install I ran reset.sh and now it has started successfully.

@Skyw3lker can you try @kylesmithit's solution?

I'm going to close this out for lack of activity. @Skyw3lker feel free to comment if the above solution doesn't work for you.


Support T-HEAD processor

Hi ,
I am desired to port T-HEAD processor in FreeRTOS. the porting project is done. I wonder to know how to request to merge it into FreeRTOS.
Vincent

Grateful if you can provide a link to the T-HEAD processor - thanks.

https://www.t-head.cn/product/s802?spm=a2ouz.12987<IP_ADDRESS>9248abc8ds69&lang=en
Richard, please have a look.
We still have RISC-V core which is scheduled to support FreeRTOS in future
发自我的iPhone

在 2019年10月28日，11:34，RichardBarry<EMAIL_ADDRESS>写道：

Grateful if you can provide a link to the T-HEAD processor - thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi Rechard
Have you check the processor link I paste?
Vincent
发自我的iPhone

在 2019年10月28日，11:34，RichardBarry<EMAIL_ADDRESS>写道：

Grateful if you can provide a link to the T-HEAD processor - thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

It is not clear what the core is from the link.  Could you please contact me using the 'business contact' link on https://www.freertos.org/contact so we can discuss in more detail - thanks.

Hello, I am closing this issue as the conversation has moved to email. Please reopen if you would like to discuss this further via this forum.


CHEBI:81485, please check

Hi,
I need CHEBI:81485, osthenol, for a new IUBMB reaction, so please check it.
Cheers,
Kristian

This compound is already checked


New theme

could you also consider adding [Everblush[https://github.com/Everblush) theme.

Thanks! I will add it as soon as possible.

thanks :D

Is it good? @Mangeshrex

Try the accent colors
#9bdead - teal
#67b0e8 - blue
Else it's good to go

I am not good with colors. Can you tell me which colors I need to change?

I feel this is good enough. This is better.


Slack injector honours excluded configs

Related to https://github.com/openshift/release/pull/55810#issuecomment-2316794196

/lgtm


Exam mode: Copy grading instruction when preparing second correction round

Checklist
General

[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.

Motivation and Context
Closes #4754
Description
When preparing the assessment of the second correction round, the existing feedback gets copied. In this process, the copyFeedback()-method gets called. Here the gradingIstruction-field is now copied as well so that the connection doesn't get lost.
Steps for Testing

Prerequisites:

1 Exam with different exercises (preferably every type), each exercise with grading instructions.
Second correction enabled for the exam
2 Tutors
1 Student

Tutor 1: Use Grading instructions during the first correction round
Tutor 2: Make sure the grading instructions are visible during the second correction round. Change the assessment.
Student: Make sure the grading instructions used in the second correction round are visible in the feedback.

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Good finding, thanks for the fast response 👍 👍 👍
Would be awesome if some people can test that this actually works correctly now


use lite custom op api for math

add a header for the lite custom op api, this is built on top of ortw::cusotmop api in extension folder and can works with legacy version of onnxruntime.
update ops under math folder to use new api

update the PR to include cv2 / image kernels.
Now we have two limitation remaining: 1) string tensor support. 2) support initialization.

Update the PR to support register a custom op kernerl struct (with construct and compute). now the only thing remain is the string tensor.

migrate all the operators


a11y: skip to content link

Insert a "Skip to Content" link at the top of each page on the website. This will allow screen reader users to skip the top menu directly to the beginning of the chapter content. You can see an example on this page: https://opentextbc.ca/studentsuccess/chapter/learning-action-plan/ If you start tabbing, a "Skip to content" link pops up that takes you directly to the beginning of the chapter. Here is more information: WCAG - Bypass Blocks

Reported to BookML: https://github.com/vlmantova/bookml/issues/8


Is eth_newPendingTransactionFilter method disabled in Base node?

Is eth_newPendingTransactionFilter method disabled in Base node?
Or can I enable it on my own node?

Hi @dreygur, eth_newPendingTransactionFilter is disabled by default in op-geth. This is to prevent any transaction leakage/MEV opportunities. There is more context on this here and here.
You can enable it on your own node, but you'll only see transactions in that nodes mempool.


Reduce state size

What does this PR do?

This PR optimizes the size of the state. From benchmarking with the following example, the state seems to be 5 times smaller and the time spent with delta queue transfer is also reduced as the state grows.
from lightning import LightningApp, LightningFlow, LightningWork
from lightning.app.runners import MultiProcessRuntime
from pympler import asizeof

class SizeWork(LightningWork):
    def __init__(self, **kwargs):
        super().__init__(**kwargs)
        self.counter = 0

    def run(self, signal: int):
        self.counter += 1

class SizeFlow(LightningFlow):
    def __init__(self):
        super().__init__()
        self.work0 = SizeWork(parallel=True, cache_calls=True)
        self._state_sizes = {}

    def run(self):
        for idx in range(self.work0.counter + 2):
            self.work0.run(idx)

        self._state_sizes[self.work0.counter] = asizeof.asizeof(self.state)

        if self.work0.counter >= 20:
            self._exit()

if __name__ == "__main__":
    app = LightningApp(SizeFlow())
    MultiProcessRuntime(app, start_server=False).dispatch()
    print(app.root._state_sizes)

python app.py

On master
{0: 6232, 1: 8208, 2: 11208, 3: 13184, 4: 15032, 5: 16880, 6: 18728, 7: 20576, 8: 22704, 9: 24552, 10: 26400, 11: 28248, 12: 30096, 13: 31944, 14: 33792, 15: 35096, 16: 36944, 17: 39336, 18: 40032, 19: 43568, 20: 44264}

and reached 0.1MB at counter 46.
With this PR
{0: 5896, 1: 5440, 2: 5496, 3: 5552, 4: 6120, 5: 6176, 6: 6232, 7: 6288, 8: 6344, 9: 6400, 10: 6456, 11: 6512, 12: 6568, 13: 6624, 14: 6680, 15: 6736, 16: 6792, 17: 6848, 18: 8440, 19: 8496, 20: 9128}

and reached 0.1MB at counter 1095.
Fixes #13944
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
cc @tchaton @rohitgr7

If I'm not missing something, calls are essentially only useful for Flow, and not for Work, therefore we could avoid transmitting the whole calls down to Work (and up to FE).

Hey @lantiga,

If I'm not missing something, calls are essentially only useful for Flow, and not for Work, therefore we could avoid transmitting the whole calls down to Work (and up to FE).
This is already done this way right there: https://github.com/Lightning-AI/lightning/pull/13970/files#diff-5347e507417db0b69cdee291b23e64f0105cf382f743f8f2d7e60f308cf05b5dR65. I am popping the call_hashes before sending the state through the caller_queue.
This made the responsiveness of the work much better compared to before.

Hey @lantiga,
Quote: If I'm not missing something, calls are essentially only useful for Flow, and not for Work, therefore we could avoid transmitting the whole calls down to Work (and up to FE).
This is already done this way right there: https://github.com/Lightning-AI/lightning/pull/13970/files#diff-5347e507417db0b69cdee291b23e64f0105cf382f743f8f2d7e60f308cf05b5dR65. I am popping the call_hashes before sending the state through the caller_queue. This drastically increased the responsiveness of the work when I was doing my tests compared to master.

Ah, awesome. I didn’t catch that. Perfect, it would be good to add a comment and making it more prominent.


Enable 2d variables offline diags

Recently we've starting training models to predict 2D variables (surface radiative flux, surface precip) but the offline diagnostics were not compatible with these models as they were previously set up to only work with vertical profiles. This PR adjusts parts of the offline diags to allow reports to be saved for models that have 2D and/or 3D outputs.
model only predicts 2D output: https://storage.googleapis.com/vcm-ml-public/annak/2021-03-31-offline-diags/test-single-2d-output/index.html
model predicts both 2D and 3D output: https://storage.googleapis.com/vcm-ml-public/annak/2021-03-31-offline-diags/test-2d-and-3d-output/index.html

if a model is not predicting dQ1 or dQ2 should we plot profiles of Q1, Q2, dQ1, and dQ2? Would it be difficult to remove those in that case?

It would be some work since the original code had quite a few hard coded references to combinations of dQ/Q/pQ variables, but I am happy to do that in this PR since I think it would be much better to have the report be smarter about figuring out which variables should be included in the plots, and some of the changes here are dealing with that type of logic already.

One other question -- is there an easy way to add a test that using this works for only 2D predicted variables (clearly it does based on your example, but I don't know if we want to add some sort of integration test)? That could wait until another PR though, if there is still some infrastructure that needs to be built/merged to train a model to predict 2D fields.

If we set up the integration test to  have prescribed fluxes or some other 2D variable that can be set in the wrapper, then it would be pretty easy. The train-diags-prog template assumes all predicted variables are used to update the model state in the prognostic run, so any 2D outputs would need to be able to be set in the wrapper. I can do a future PR to add this to the integration test.


Server rejected event

There is error message in log:
E0205 11:22:06.986200       1 event.go:263] Server rejected event '&v1.Event{TypeMeta:v1.TypeMeta{Kind:"", APIVersion:""}, ObjectMeta:v1.ObjectMeta{Name:"xxxx-web-db6785fc9-v6f7q.15f07dbea482b25a", GenerateName:"", Namespace:"xxxx-production", SelfLink:"", UID:"", ResourceVersion:"", Generation:0, CreationTimestamp:v1.Time{Time:time.Time{wall:0x0, ext:0, loc:(*time.Location)(nil)}}, DeletionTimestamp:(*v1.Time)(nil), DeletionGracePeriodSeconds:(*int64)(nil), Labels:map[string]string(nil), Annotations:map[string]string(nil), OwnerReferences:[]v1.OwnerReference(nil), Finalizers:[]string(nil), ClusterName:"", ManagedFields:[]v1.ManagedFieldsEntry(nil)}, InvolvedObject:v1.ObjectReference{Kind:"Pod", Namespace:"xxxx-production", Name:"xxxx-web-db6785fc9-v6f7q", UID:"22ae8c9e-4809-11ea-b920-4201ac100108", APIVersion:"v1", ResourceVersion:"27132912", FieldPath:""}, Reason:"Descheduled", Message:"pod evicted by sigs.k8s.io/descheduler", Source:v1.EventSource{Component:"sigs.k8s.io.descheduler", Host:""}, FirstTimestamp:v1.Time{Time:time.Time{wall:0xbf86c737b97a865a, ext:158538930, loc:(*time.Location)(0x211e400)}}, LastTimestamp:v1.Time{Time:time.Time{wall:0xbf86c737b97a865a, ext:158538930, loc:(*time.Location)(0x211e400)}}, Count:1, Type:"Normal", EventTime:v1.MicroTime{Time:time.Time{wall:0x0, ext:0, loc:(*time.Location)(nil)}}, Series:(*v1.EventSeries)(nil), Action:"", Related:(*v1.ObjectReference)(nil), ReportingController:"", ReportingInstance:""}': 'events is forbidden: User "system:serviceaccount:kube-system:descheduler-sa" cannot create resource "events" in API group "" in the namespace "xxxx-production"' (will not retry!)

Is that ok ?

Hey @slayer, did you use https://github.com/kubernetes-sigs/descheduler/#create-a-cluster-role ?

/assign

The important part of the error message is all the way at the end User "system:serviceaccount:kube-system:descheduler-sa" cannot create resource "events" in API group "" in the namespace "xxxx-production"' (will not retry!). I added k8s Event support in #183, but I did not update the ClusterRole in k8s YAML files to handle this.
FYI ... on the master branch the ClusterRole is actually duplicated in multiple places right now.

https://github.com/kubernetes-sigs/descheduler#create-a-cluster-role
https://github.com/kubernetes-sigs/descheduler/blob/master/kubernetes/rbac.yaml#L2
https://github.com/kubernetes-sigs/descheduler/blob/master/examples/descheduler.yaml#L2

Pull request #208 will remove all of the above mentioned duplication.

@ingvagabund I can fix the issue with the ClusterRole in my PR #208 if you want. Let me know if you want me to do this.

/kind documentation

@seanmalloy sounds good as long as it's in a separate PR. Thanks!!!

/assign @seanmalloy
I will fix this after #208 is merged.

/unassign @ingvagabund

@slayer this is fixed in #237


Work out why AppVeyor is failing

The tests for Google.Cloud.Diagnostics.Common look like they're passing, but we get an odd test runner failure. It used to happen very occasionally - now looks like it's permanent. Investigating...

I'm happy to take this issue if you want just let me know

Interestingly, after a spate of errors it looks like I've got one build which is actually working.

The failures all have something like this:
[00:07:57] [xUnit.net 00:00:46.1409336] When sending 'PUT http://localhost:1563/api/tests/batch' with body '[{...}] , exception was thrown: A task was canceled.

This looks very related: http://help.appveyor.com/discussions/problems/3940-appveyor-test-service-erroring

(I'm not sure how AppVeyor hooks into it though. We're using xUnit 2.2 which looks like it has changes from there already.)

Looks like it autodetects AppVeyor via the APPVEYOR_API_URL environment variable. This is how AppVeyor has the tests tab as well as the log.
As this is causing us problems, I think for now it might be best to try to disable that. Will send a PR.


Making the Application a PWA

Currently, the application is running on a browser, but now I want to convert it to an app a PWA  so that it can installable on the machine.

@Abhishek-kumar09 I am facing an issue while creating a Pull Request, I rebase my currently forked main branch with upstream/main breach, but it is still showing this.

and when I create a pull request it shows my previous commits like:

Please Help me regarding this issue.

And also can you please review my issue also @Abhishek-kumar09, regarding PWA.

@kunal-kushwaha @KeenWarrior please help 🙏.

Hey @dhwaj1902  actually this react app is gonna be converted to an apk file using the Capacitor js library. So you don't have to make it as a PWA.

Ok, I understand that but what about this issue? I am not able to make any or any commit because this is showing.

@dhwaj1902 Please look into advance git functionality. You might have to hard reset to a previous commit.

I guess each time you pull from the upstream you commit with a message and push so it takes it as an update and shows that your branch is ahead of the forked project. So after pulling simple push without commiting to bring your GitHub repo equal to the project repo. Hope this will help you in future.

@ADRE9 I did the same but it is still showing like this.

Hi, @dhwaj1902
To delete those extra commits and make your branch even with main branch, Do rebasing.
git rebase -i HEAD~5 // 5 being the number of commits to look into an interactive terminal
now delete the commits that you dont want by replacing "pick" with "drop"
Once done, press "ctrl+s and then ctrl+x"
Your branch is rebased and is now even with the main branch of edu-clent


Multiple Toastr message

Laravel Version: 5.4.33
Voyager Version: 0.11.14
PHP Version: 7.1.7
Database Driver & Version: 5.7.19

Description:
How to send multiple Toastr message by voyager controller. so to display 2-3 or n number of Toastr message using a controller.
so like sending an array in controller :
$msg = [];
$info = [];
$uarray = array(value for updation);
$darray = array(value for deletion);

foreach ($uarray as $key => $value){
    {businesslogic to update record}
    $msg[] = "{$value} has been successfully updated";
    $info[] = "success";
}

foreach ($darray as $key => $value){
    {businesslogic to delete record}
    $msg[] = "{$value} has been successfully deleted";
    $info[] = "error";
}

return redirect()
    ->route("voyager::bread.index")
    ->with([
        'message'    => $msg,
        'alert-type' => $info,
    ]);

Steps To Reproduce:

Looks like wi dont have a loop for that in master.blade
if (alerter) {
    alerter(alertMessage);
} else {
    toastr.error("toastr alert-type " + alertType + " is unknown");
}

What if i copy
master.blade.php to resources/views/vendor/voyager/master.blade.php
and replace the code :
@if(Session::has('message'))
    // TODO: change Controllers to use AlertsMessages trait... then remove this
    var alertType = {!! json_encode(Session::get('alert-type', 'info')) !!};
    var alertMessage = {!! json_encode(Session::get('message')) !!};
    var alerter = toastr[alertType];
    if (alerter) {
        alerter(alertMessage);
    } else {
        toastr.error("toastr alert-type " + alertType + " is unknown");
    }
@endif

with :
@if(Session::has('message'))
    // TODO: change Controllers to use AlertsMessages trait... then remove this
    var alertType = {!! json_encode(Session::get('alert-type', 'info')) !!};
    var alertMessage = {!! json_encode(Session::get('message')) !!};
    if (alertType.constructor === Array && alertMessage.constructor === Array) {
        for (var i = 0; i < alertType.length; i++) {
            var type = alertType[i];
            var alerter = toastr[type];
            alerter(alertMessage[i]);
        }
    } else {
        var alerter = toastr[alertType];
        if (alerter) {
             alerter(alertMessage);
        } else {
             toastr.error("toastr alert-type " + alertType + " is unknown");
        }
    }
@endif

As of now its working for my case, is my solution suggestible and good to use ?

Yes if you are using custom view you can do whatever you want.
Here is more comapct solution
@if(Session::has('message'))
      @foreach (Session::get('message') as $msg)
          ///Your toastr code here
      @endforeach
@endif

Check Voyager Academy for tutorials
https://laravelvoyager.com/academy/views/

Thank you so much for your help, i will give it a try and will let you know the results. Thanks

I'd be interested to see a PR for this as well.  Assuming you get it working properly first!


[Suggestion] Token Monochrome Vision Color Override

Canonically, Fire Genasi have red darkvision.  It would be nice to have an override on individual tokens.

There is. You can find it in the token's configuration under the Vision tab.
But it wasn't there in v1.2.0 because of a bug, which has been fixed in v1.2.1.


Improved Yaml ParseException exception message

Presently, if a Yaml mapping file has a parse error, it can be tough to track down as the resulting error message only tells you the line of the error and not the filename.
This PR adds a better exception error message that appends the filepath to the end of the parse exception.
The current PR catches only "\Symfony\Component\Yaml\Exception\ParseException" and perhaps it would be better to catch all exceptions, but then I don't know which exception class to "re-throw". Thoughts?

@speedytwenty would you mind squashing your commits?

Not having much luck with squashing. Seems to always lead to a merge. Trying to figure it out.

git rebase -i master while you're on your branch :)

I tried git rebase -i master but afterward when I try git push origin feature_branch it tells me my local branch is behind the remote branch and forces me to pull, which adds an empty merge commit. Guessing the commits are getting rebased in the wrong order or something... sorry haven't had to rebase much. But anxious to figure this out!

@Steveb-p - Thank you! The --force-with-lease was exactly what was I missing!
@malarzm Commits squashed, nit picks too :)

@speedytwenty thanks for all the effort! Merging since the tests are green now.
Offhand, @Steveb-p thanks for chiming in :)


fix: Use unless-stopped for docker containers to prevent unexpected starts after docker daemon reboot

Description and Motivation
Has all bots restarted after a server reboot, also the one I've stopped manually.
unless-stopped is similar to always, except that when the container is stopped (manually or otherwise), it is not restarted even after Docker daemon restarts.

https://dev.to/rohithv07/restart-policies-docker-compose-2c14
https://docs.docker.com/config/containers/start-containers-automatically/

Related Issues
n/a
Checklist:

[x] I have read Migaloo's contribution guidelines.
[x] My pull request has a sound title and description (not something vague like Update index.md)
[x] All existing and new tests are passing (if any).
[x] I updated/added relevant documentation.
[x] The code is formatted properly using eslint

@silvermind can you please change the PR to release branch? We only release updates to the main branch on a scheduled basis.


Update slf4j dependency

Update to 1.7.31
This version has an interesting performance fix (https://jira.qos.ch/browse/SLF4J-497) which may have some (though probably minor) positive performance impact on rdf4j

Not a real bug fix, though I'd like to suggest to include it in 3.7.2 anyway


Potential rewards sometimes are not displayed according to slider value.

See screenshot. The amount on the slider was manually chosen. This happens sporadically. Sometimes it will show the correct rewards estimation, other times it'll show the amount corresponding to the default 1000 ADA.

Dear @Klobbinger ,
Thank you for letting us know. We will investigate and let you know in due course.
Kind regards,
Mior

Hi @Klobbinger ,
I hope that you are well. Do you mind giving it another go please and see if you could reproduce this issue on the latest Daedalus (please note that the pool estimated rewards will be treated in its own ticket). .
Many thanks,
Mior

I was able to replicate the bug once after initial start up. After that, I couldn't replicate it. I will try again tomorrow and report back.

So it seems like this only happens after starting Daedalus and going to the stake pool tab. first time the slider is adjusted it does not update the expected rewards. After moving the slider a second time the correct value is shown.

Hi @PhilippeLeLong ,
Thank you for your replies. We have been trying to reproduce the issue without any success. We tested on multiple platforms including Windows, Mac and Linux on at least two networks which are Mainnet and Testnet. I have been wondering if you could still reproduce it? If possible, could you kindly let us know the following details please:
1)Your operating system and version
2)RAM size
4)The hard disk space you currently have left in the machine
4)The Daedalus that produces the issue, ie (Mainnet/Flight or Testnet) and the build number
5)Is the Daedalus a fresh install or an upgrade?
6)Any other details that you feel might be beneficial for us
In the meantime, we will continue with our investigation and look forward to receiving the above information from you.
Kind regards,
Mior

Windows 10 Home, 21H1, Build 19043.1415
16GB
40 GB and 500 GB. Daedalus folder is on D: drive. Same issue though before when everything was on C:
Mainnet Build 20052
Upgrade from previous versions, fresh install in October 2020, don't know which version.
Steps to reproduce:

Open Daedalus wait for connection and full sync
Go to Stake Pools tab
click list view
move slider once and wait for calculation to be finished. Result see attached screnshot.
Any other action afterwards that updates the stake pool list again will result in the correct rewards shown.

Hope that helps.

Installed Daedalus on a new PC. Latest version 4.9.1 and the problem still persists.

Daedalus version 4.11.0
Same faulty behavior. The first time after opening Daedalus and going to the stake pools tab and sliding to a custom amount, the potential rewards are calculated incorrectly. Only after adjusting the slider again the UI will show correct values.


how to force re-rendering when content changes?

Hi! Great project, enjoy using it so far :)
I have a span-element with the v-katex:auto attribute, to which I pass user input. How can I call the renderMathInElement function from my component?

Nevermind, found a solution – I pass the content as a key to the span :)

Nevermind, found a solution – I pass the content as a key to the span :)

Just wanna put an example for those who are going to view the topic later:
<template>

    <input v-model="question" />

{{question}}

</template>

Is that what you did, I guess?


bug when using consul with http1.1

-after first lookup, do the block query, it will return the diff status.
-when first lookup, the consulindex is 0, it will do the quick lookup

@BalloonWen It should work for the consul client for this particular case. However, I think we should put the fix into the Http2Client so that we can handle HTTP/1.1 gracefully for all services who are leveraging the Http2Client.

Codecov Report

Merging #524 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #524      +/-   ##
============================================
- Coverage     49.62%   49.58%   -0.05%
  Complexity     1826     1826
============================================
  Files           260      260
  Lines         11205    11215      +10
  Branches       1587     1588       +1
============================================
  Hits           5561     5561
- Misses         4991     5001      +10
  Partials        653      653

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/com/networknt/consul/ConsulConfig.java
62.96% <0%> (-7.88%)
10 <0> (ø)

.../com/networknt/consul/client/ConsulClientImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


"You are X hours ahead of server time" warning does not have left padding

This should speak for itself. An ml-2 is in order. Vertical centering would also be nice.

This is unrelated to the issue, but we've been using Unfold in production for both our support team and client portal. The speed of development it grants us is awesome. This project inspires me to build something in this space...


fix: added async line to auth generator in order to avoid issues with…

When the auth controller template is generated it shows an issue with the type returned with the promise.Types. It was solved by adding the async keyword before the function handleResetEmail.

The issue is because sequelize returns a bluebird promise (a library that implements promises, used when promises where not fully standardized in javascript, no longer needed, but sequelize v5 uses it still).
The correct fix is to make the function definition say it returns a PromiseLike.


Extending lists with xyml.for

Hi, some early users tried to extend a list with a for loop. they tried:
parallelBuild:
- name: test1
  buildTarget: BUILD
- name: test2
  buildTarget: BUILD

pipelines:
- nodes:
  - label: Should stay
    executionOrder: 0
    stages:
      - name: "Should stay stage"
        executionOrder: 0

  xyml.for: build_config:parallelBuild
  xyml.content:
  - label: "{{build_config.name}}"
    executionOrder: 1
    stages:
      - name: "{{build_config.buildTarget}}"
        executionOrder: 0

This swallows the node and results in:
parallelBuild:
- name: test1
  buildTarget: BUILD
- name: test2
  buildTarget: BUILD
pipelines:
- label: test1
  executionOrder: 1
  stages:
  - name: BUILD
    executionOrder: 0
- label: test2
  executionOrder: 1
  stages:
  - name: BUILD
    executionOrder: 0

If we indent the xyml.for etc. into a list it works as expected.
parallelBuild:
- name: test1
  buildTarget: BUILD
- name: test2
  buildTarget: BUILD

pipelines:
- nodes:
  - label: Should stay
    executionOrder: 0
    stages:
      - name: "Should stay stage"
        executionOrder: 0

  - xyml.for: build_config:parallelBuild
    xyml.content:
    - label: "{{build_config.name}}"
      executionOrder: 1
      stages:
        - name: "{{build_config.buildTarget}}"
          executionOrder: 0

Creates the expected
parallelBuild:
- name: test1
  buildTarget: BUILD
- name: test2
  buildTarget: BUILD
pipelines:
- nodes:
  - label: Should stay
    executionOrder: 0
    stages:
    - name: Should stay stage
      executionOrder: 0
  - label: test1
    executionOrder: 1
    stages:
    - name: BUILD
      executionOrder: 0
  - label: test2
    executionOrder: 1
    stages:
    - name: BUILD
      executionOrder: 0

Is this the expected behavior?
Should we maybe extend the documentation on how to extend lists?

I think so, it is somehow an expected behavior for how this package works!
The first time it tries to load the yaml content and the xyml.for is direct child for pipelines[0] (first item of list) so this will join into an object with nodes, xyml.for, xyml.content and this will couse the ignoring what you have as a key near those keywords.
At src\yaml_extender\resolver\inline_loop_resolver.py:28 k = pipelines and self._Resolver__resolve returns <<what you have in results>>

Hi,
The xyml.for statement needs to be a member of the node. Not of the Pipeline.
Just add an additional indentation, No need to make IT a list though.

Hi, The xyml.for statement needs to be a member of the node. Not of the Pipeline. Just add an additional indentation, No need to make IT a list though.

If I understood you correctly, is this what you mean? That swallows everything else in that node. So we do need to move it to its own list entry.
parallelBuild:
- name: test1
  buildTarget: BUILD
- name: test2
  buildTarget: BUILD

pipelines:
- nodes:
  - label: Should stay
    executionOrder: 0
    stages:
      - name: "Should stay stage"
        executionOrder: 0

    xyml.for: build_config:parallelBuild
    xyml.content:
    - label: "{{build_config.name}}"
      executionOrder: 1
      stages:
        - name: "{{build_config.buildTarget}}"
          executionOrder: 0

parallelBuild:
- name: test1
  buildTarget: BUILD
- name: test2
  buildTarget: BUILD
pipelines:
- nodes:
  - label: test1
    executionOrder: 1
    stages:
    - name: BUILD
      executionOrder: 0
  - label: test2
    executionOrder: 1
    stages:
    - name: BUILD
      executionOrder: 0

Other example
parallelBuild:
- name: test1
  buildTarget: BUILD
- name: test2
  buildTarget: BUILD

pipelines:
- nodes:
  - label: XXXXX
    executionOrder: 0
    stages:
      - name: "XXXXX"
        executionOrder: 0

  - label: Should stay
    executionOrder: 0
    stages:
      - name: "Should stay stage"
        executionOrder: 0

    xyml.for: build_config:parallelBuild
    xyml.content:
    - label: "{{build_config.name}}"
      executionOrder: 1
      stages:
        - name: "{{build_config.buildTarget}}"
          executionOrder: 0

parallelBuild:
- name: test1
  buildTarget: BUILD
- name: test2
  buildTarget: BUILD
pipelines:
- nodes:
  - label: XXXXX
    executionOrder: 0
    stages:
    - name: XXXXX
      executionOrder: 0
  - label: test1
    executionOrder: 1
    stages:
    - name: BUILD
      executionOrder: 0
  - label: test2
    executionOrder: 1
    stages:
    - name: BUILD
      executionOrder: 0

Hi, i will Check when i'm Back from vacation.
You can Go with the workaround until then .

No worries. Take your time and have a nice vacation.

The issue is that all other items that are siblings with xyaml.for will be shadowed due the processing. You simply need to put it in a separate object.
Here is your example:
parallelBuild:
- name: test1
  buildTarget: BUILD
- name: test2
  buildTarget: BUILD

pipelines:
- nodes:
  - label: XXXXX
    executionOrder: 0
    stages:
      - name: "XXXXX"
        executionOrder: 0

  - label: Should stay
    executionOrder: 0
    stages:
      - name: "Should stay stage"
        executionOrder: 0

  - xyml.for: build_config:parallelBuild
    xyml.content:
    - label: "{{build_config.name}}"
      executionOrder: 1
      stages:
        - name: "{{build_config.buildTarget}}"
          executionOrder: 0

That produce:
parallelBuild:
- name: test1
  buildTarget: BUILD
- name: test2
  buildTarget: BUILD
pipelines:
- nodes:
  - label: XXXXX
    executionOrder: 0
    stages:
    - name: XXXXX
      executionOrder: 0
  - label: Should stay
    executionOrder: 0
    stages:
    - name: Should stay stage
      executionOrder: 0
  - label: test2
    executionOrder: 1
    stages:
    - name: BUILD
      executionOrder: 0
  - label: test1
    executionOrder: 1
    stages:
    - name: BUILD
      executionOrder: 0

ok the problem is tougher than i though...
It is hard to keep the order in statements like this:
`
array:

name: second
value: 2
name: third
value: 3

commands:

name: first
value: 1
xyml.for: element:array
xyml.content:

name: "{{element.name}}"
value: "{{element.value}}"

name: fourth
value: 4
`

I'm wokring on it and will provide a solution later this week. I will also release a new version then.

The problem should be fixed now.
Please verify using verison 0.3.1


Strange sorting

How sort in output (JSON array)? I have 6 files atlas and need sort by name (because use animation), but elements placed in random atlas.

Please let me know the approximate completion date for the current issues
сб, 22 авг. 2020 г. в 15:39, nonunknown<EMAIL_ADDRESS>
Yeah I agree, Optimal packer's method should have a sortByName method,
this way you can have animations organized, and play in sequence.
[image: Screenshot from 2020-08-22 09-38-55]
https://user-images.githubusercontent.com/58845030/90956268-51938280-e45b-11ea-9cd9-715abce78975.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/odrick/free-tex-packer/issues/52#issuecomment-678635812,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGFJCGK2JRA2WIB3NDJFYB3SB637PANCNFSM4QBVIVOA
.

This is not the responsibility of the texture packer. The purpose of the texture packer is to create the most optimal atlases. Generating animation is the task of an application that uses atlases. By design.

How is it not the responsibility of this tool to output in a reasonable way compared to input?

Why would you just completely write off all of the great work that has been done on this tool by dropping the ball on the last .5%? Absolutely insane.


Can move confirm button to left side?

how can i move confirm button to left side.
if i change text in confirmBtnText to cancel and cancelBtnText to confirm i also can't change cancel button background color to green. or i should create custom button on all widget.
Sorry for my bad language.

Hello @NoNSteP, i just introduced a new property called reverseBtnOrder, set it to true is you want to switch the button positions.


Connection refused on the worker node

Environmental Info:
K3s Version:

k3s-v
-bash: k3s-v: command not found
[root@lp-rh12 ~]# k3s -v
k3s version v1.26.3+k3s1 (01ea3ff2)
go version go1.19.7
Node(s) CPU architecture, OS, and Version:

5.14.0-70.30.1.el9_0.x86_64 #1 SMP PREEMPT Fri Oct 14 10:30:46 EDT 2022 x86_64 x86_64 x86_64 GNU/Linux
Cluster Configuration:

a cluster with 3 nodes: node 1 installed k3s service successfully. node 2 installed k3s-agent but got connection issue as mentioned in the above
Describe the bug:

Node 1: run "curl -sfL https://get.k3s.io | sh -", successful
Node 2: run "curl -sfL https://get.k3s.io | K3S_URL=https://{server}:6443 K3S_TOKEN={token} sh -", got error
Verified from node2, I can ping the server, and vice versa
Verified the token from server is correctly fed to the command.
Error:
from systemctl status I see
xxx.com k3s[1170]: time="2023-04-04T16:46:50-07:00" level=error msg="Failed to connect to proxy. Empty dialer response" error="dial tcp <IP_ADDRESS>:6443: connect: connection timed out"
xxx.com k3s[1170]: time="2023-04-04T16:46:50-07:00" level=error msg="Remotedialer proxy error" error="dial tcp <IP_ADDRESS>:6443: connect: connection timed out"
xxx.com k3s[1170]: time="2023-04-04T16:46:55-07:00" level=info msg="Connecting to proxy" url="wss://<IP_ADDRESS>:6443/v1-k3s/connect"
xxx.com k3s[1170]: time="2023-04-04T16:47:03-07:00" level=info msg="Waiting to retrieve kube-proxy configuration; server is not ready: CA cert validation failed: Get "https://<IP_ADDRESS>:6444/cacerts\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)"
~
Steps To Reproduce:

Installed K3s:

Expected behavior:

Actual behavior:

Additional context / logs:

=== Installing on the other node
[INFO]  Finding release for channel stable
[INFO]  Using v1.26.3+k3s1 as release
[INFO]  Downloading hash https://github.com/k3s-io/k3s/releases/download/v1.26.3+k3s1/sha256sum-amd64.txt
[INFO]  Downloading binary https://github.com/k3s-io/k3s/releases/download/v1.26.3+k3s1/k3s
[INFO]  Verifying binary download
[INFO]  Installing k3s to /usr/local/bin/k3s
Updating Subscription Management repositories.
Rancher K3s Common (stable)                                                                                7.1 kB/s | 2.9 kB     00:00
Dependencies resolved.
===========================================================================================================================================================
 Package                            Architecture                  Version                           Repository                                        Size
===========================================================================================================================================================
Installing:
 k3s-selinux                        noarch                        1.2-2.el8                         rancher-k3s-common-stable                         20 k

Transaction Summary
===========================================================================================================================================================
Install  1 Package

Total download size: 20 k
Installed size: 94 k
Downloading Packages:
k3s-selinux-1.2-2.el8.noarch.rpm                                                                                40 kB/s |  20 kB     00:00
-----------------------------------------------------------------------------------------------------------------------------------------------------------
Total                                                                                39 kB/s |  20 kB     00:00
Running transaction check
Transaction check succeeded.
Running transaction test
Transaction test succeeded.
Running transaction
  Preparing        :                                                                                1/1
  Running scriptlet: k3s-selinux-1.2-2.el8.noarch                                                                                1/1
  Installing       : k3s-selinux-1.2-2.el8.noarch                                                                                1/1
  Running scriptlet: k3s-selinux-1.2-2.el8.noarch                                                                                1/1
Failed to resolve typeattributeset statement at /var/lib/selinux/targeted/tmp/modules/200/k3s/cil:21
Failed to resolve AST
/usr/sbin/semodule:  Failed!

  Verifying        : k3s-selinux-1.2-2.el8.noarch                                                                                1/1
Installed products updated.
Last metadata expiration check: 0:00:10 ago on Tue 04 Apr 2023 04:33:04 PM PDT.

Installed:
  k3s-selinux-1.2-2.el8.noarch

Complete!
[INFO]  Creating /usr/local/bin/kubectl symlink to k3s
[INFO]  Creating /usr/local/bin/crictl symlink to k3s
[INFO]  Creating /usr/local/bin/ctr symlink to k3s
[INFO]  Creating killall script /usr/local/bin/k3s-killall.sh
[INFO]  Creating uninstall script /usr/local/bin/k3s-agent-uninstall.sh
[INFO]  env: Creating environment file /etc/systemd/system/k3s-agent.service.env
[INFO]  systemd: Creating service file /etc/systemd/system/k3s-agent.service
[INFO]  systemd: Enabling k3s-agent unit
Created symlink /etc/systemd/system/multi-user.target.wants/k3s-agent.service → /etc/systemd/system/k3s-agent.service.
[INFO]  systemd: Starting k3s-agent

=== Error on the node:

# kubectl get node
E0404 16:34:14.911536    1211 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp <IP_ADDRESS>:8080: connect: connection refused
...
The connection to the server localhost:8080 was refused - did you specify the right host or port?
# kubectl get cluster
E0404 16:34:19.856355    1224 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp <IP_ADDRESS>:8080: connect: connection refused
...
The connection to the server localhost:8080 was refused - did you specify the right host or port?

=== systemctl of k3s-agent shows the following error:
00" level=info msg="Waiting to retrieve kube-proxy configuration; server is not ready: CA cert validation failed: Get \"https://<IP_ADDRESS>:6444/cacerts\": >
00" level=error msg="Failed to connect to proxy. Empty dialer response" error="dial tcp <IP_ADDRESS>:6443: connect: connection timed out"
00" level=error msg="Remotedialer proxy error" error="dial tcp <IP_ADDRESS>:6443: connect: connection timed out"
00" level=info msg="Connecting to proxy" url="wss://<IP_ADDRESS>:6443/v1-k3s/connect"
00" level=info msg="Waiting to retrieve kube-proxy configuration; server is not ready: CA cert validation failed: Get \"https://<IP_ADDRESS>:6444/cacerts\": >

xxx.com k3s[1170]: time="2023-04-04T16:46:50-07:00" level=error msg="Failed to connect to proxy. Empty dialer response" error="dial tcp <IP_ADDRESS>:6443: connect: connection timed out"
xxx.com k3s[1170]: time="2023-04-04T16:46:50-07:00" level=error msg="Remotedialer proxy error" error="dial tcp <IP_ADDRESS>:6443: connect: connection timed out"
xxx.com k3s[1170]: time="2023-04-04T16:46:55-07:00" level=info msg="Connecting to proxy" url="wss://<IP_ADDRESS>:6443/v1-k3s/connect"

It looks like the agent can't reach the server. Are you sure they're allowed to talk to each other on the required ports? Can you successfully curl -ks http://<IP_ADDRESS>:6443 ?


Cannot change resolution when webcam is open, please close it first

Hello,
I made a simple program to take a photo. The photo will be taken when I press this button.
After the photo is taken, I open my Scherm.java and it shows the image that is taken. So in that frame I made an JToggleButton, that opens the first frame (Camera.java). When I press this button, I go back to my frame and press the button to take the photo again. I get this error:
"Exception in thread "AWT-EventQueue-0" java.lang.IllegalStateException: Cannot change resolution when webcam is open, please close it first at com.github.sarxos.webcam.Webcam.setViewSize(Webcam.java:544)"
I have been searching and I found out that if I remove the webcam.setViewSize everything works. But then my photo is too small so I can't use it. Does someone know how I can fix this problem?
This code is my Camera.java
           class foto implements ActionListener {
    public void actionPerformed(ActionEvent a) {
        System.out.println("Foto maken... ");
    	 Webcam webcam = Webcam.getDefault();
        if (webcam != null) {
            System.out.println("Webcam: " + webcam.getName());`
            webcam.setViewSize(new Dimension(640,480)); //If I remove this line, it works fine.
            webcam.open();
            Scherm scherm = new Scherm();
            frame.dispose();

            try {
                ImageIO.write(webcam.getImage(), "PNG", new File("Test.png"));
            } catch (IOException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            }
        } else {
            System.out.println("No webcam detected");

`

Hi @Zeeebas,
You have several options:

Close webcam after you took photo.

try {
  webcam.open();
  // do your stuff here, take photo etc
} finally {
  webcam.close();
}

Open webcam only once, on application start and set view size only once.
Check if webcam is not open before you setViewSize(..) (this one is the best IMO).

if (!webcam.isOpen()) {
  webcam.setViewSize(WebcamResolution.VGA.getSize());
}

Hi @Zeeebas, can I close this ticket? I hope I was able to answer your question in my last response.

Hey @sarxos
I am so sorry I didnt respond.  My death line was the next day and your solution fixed the problem, I was so happy and glad that I worked I forgot to thank you :)
Thank you so much, yes you can.

Cool. Take care!


Add proxy to playbook - error in call intersight.com

Hello, in my job i have proxy to connect internet, but dont have ssl intercept...
in example_claim.yml i add environment: variable
ansible_proxy = http://lalala.com:80/'
ansible_noproxy = <IP_ADDRESS>/8
- name: Claim targets
  script: /usr/bin/env python claim_target.py -t ' [{{ stash_json }}]'
  register: result
  environment:
    http_proxy: "{{ ansible_proxy }}"
    https_proxy: "{{ ansible_proxy }}"
    no_proxy: "{{ ansible_noproxy }}"
  changed_when: '"true" in result.stdout'

it working fine... but i recive this error:
Exception: HTTPSConnectionPool(host='intersight.com', port=443): Max retries exceeded with url: /api/v1/resource/Groups?%24filter=Name+in+%28%27__omit_place_holder__b1208bd441ac629031fd9c714763730d807dd83e%27%29&%24select=Moid (Caused by SSLError(SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')],)",),))
and
/var/lib/awx/venv/ansible/lib64/python3.6/site-packages/urllib3/connectionpool.py:1004: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
InsecureRequestWarning,
/var/lib/awx/venv/ansible/lib64/python3.6/site-packages/urllib3/connectionpool.py:1004: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
InsecureRequestWarning,
/var/lib/awx/venv/ansible/lib64/python3.6/site-packages/urllib3/connectionpool.py:1004: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
InsecureRequestWarning,
/var/lib/awx/venv/ansible/lib64/python3.6/site-packages/urllib3/connectionpool.py:1004: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
InsecureRequestWarning,
/var/lib/awx/venv/ansible/lib64/python3.6/site-packages/urllib3/connectionpool.py:1004: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
InsecureRequestWarning,
/var/lib/awx/venv/ansible/lib64/python3.6/site-packages/urllib3/connectionpool.py:1004: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
InsecureRequestWarning,
/var/lib/awx/venv/ansible/lib64/python3.6/site-packages/urllib3/connectionpool.py:1004: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
InsecureRequestWarning,

The error points to intersight.com's certificate not being valid:
Exception: HTTPSConnectionPool(host='intersight.com', port=443): ... (Caused by SSLError(SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')],)",),))
Is a transparent proxy in use (transparent proxy presenting a cert for intersight.com)?


BeautifulSoup invocation erroneously alters the resulting html

this bit of code seems to be harmful.
I have observed it producing from this:
line with a line break (so ending with a double space)
line without a line break

the following erroneous html:
line with a line break (so ending with a double space)

line without a line break

(note the
 tag). This results in large extra empty space added to a text with a lot of line breaks.

Agree. Moreover: it makes almost impossible to include HTML code inside code blocks as soup.decode(formatter=None) replaces all entities (including < &rt;) to its corresponding symbols. So if I have something like print("") in the source I get actual  tag in the output. And this is unavoidable in general, as BeautulSoup converts entities to corresponding Unicode symbols on parsing and therefore losses some information.
This part of code is used to remove all cells with #ignore text. Personally, I'm willing just to comment it out as this feature is not crucial for me. Nevertheless, I'm not sure how to solve this problem better.

Most probably it is better not to tweak with HTML tree but to remove the corresponding cells from ipynb JSON file before invoking HTMLExporter.

Seems correct, an nbconvert Preprocessor is the way to go. I didn't need the feature, so I just removed it entirely.


How to ingore dependency

Hi, I try to bundle my angular2 application via builder.buildStatic. I also have to include reflect.js which has the following dependency.
   var nodeCrypto = isNode && require("crypto");

Obviously the builder does not detect the "isNode" variable and tries to integrate the dependency into my bundle. How can I tell them to ignore it?
I tried the following options:
     defaultJSExtensions: true,
      meta: {
          'crypto': {
              build: false
          }
      }

But I always get the message: Global SFX bundle dependency "crypto.js" must be configured to an environment global via the globalDeps option.

You can use map config to set crypto to undefined for that specific parent package:
builder.config({
  packages: {
    'path/to/package-folder-using-crypto': {
      map: { crypto: '@empty' }
    }
  }
});

Alternatively you can provide that map config on install of the package in jspm.

Thank you for your fast response. Is it also possible to make the builder stop and predefined dependencies. I want to use the prebuild angular2 packages and I want to have one file with my custom code.
I think my scenario is the same like the following issue and I thought it is already released: https://github.com/systemjs/builder/issues/407
I tried:
         meta: {
          'angular2/*': {
              build: false
          }
      }

Did you manage to sort out your issue here?

hi @guybedford I have a similar issue. I'm also trying to build angular2 but in fact instead of building everything with the systemjs builder I include the systemjs-register-only js file in index.html, then include the angular.js file which is already transpiled to System.register format, and then want to include my own code. My own code is something like import { Component } from 'angular2/core', do something with Component. I currently have the following in my systemjs config:
System.config({
  defaultJSExtensions: true,
  transpiler: 'babel',
  babelOptions: {
    optional: ['es7.decorators']
  },
  meta: {
    'angular2/*': {
      build: false
    }
  }
});

And I'm getting the same error: "Global SFX bundle dependency "angular2/core.js" must be configured to an environment global via the globalDeps option."
It would be ideal if I can just completely ignore angular2/* imports with just adding build:false.
Is there something I can add to make this work right now? Having a bit larger example for globalDefs would already help.

@guybedford Yes, it is working fine now, but I use the same approach like joerideg. I only build my custom code and use the existing angular2 bundles.
@joerideg Had the same issues. The problem is that all references are resolved with the ".js" extension. The following approach works for me:
var config = {
	baseURL: scripts,
	packages: {
		'app': {
			defaultExtension: 'js'
		}
	},
	meta: {
		'angular2/*': {
			build: false
		},
		'rxjs/*': {
			build: false
		}
	}
};

All my typescript files are in the "app" folder.

hmm I hoped that would do it. I do use babel, not typescript but still.
Currently the builder code im using is this:
        var systemjs = new Builder();
        systemjs.config({
          transpiler: 'babel',
          babelOptions: {
            optional: ['es7.decorators']
          },
          meta: {
            'angular2/*': {
              build: false
            },
            'rxjs/*': {
              build: false
            }
          }
        });
        return systemjs.buildStatic(cfg.src.indexScript, cfg.dist.indexScript, {
          sourceMaps: true
        });

unfortunately that still ends in the aformentioned error.

You need to provide globalDeps when doing a static global build so it knows which modules correspond to which global names:
systemjs.buildStatic(cfg.src.indexScript, cfg.dist.indexScript, {
          globalDeps: { 'angular2/core.js': 'angular', 'rxjs/main.js': 'rx-global-name-on-window' }
          sourceMaps: true
        });

hmm ok. I kinda understand globalDeps now, thing is that angular2/core is not really a global dependency. It does not set anything on the window, its a System.register('angular2/core') module. I tried your example and actually it now works for angualr2/core. Unfortunately it then simply errors on the next angular2/somemodule import. So I tried star patterns:
globalDeps: {
    'angular2/*': 'angular2/*'
}

Star patterns don't seem to work, so it would require me to list every single module im importing from angular2 anywhere in my code in the globalDeps :(
Btw, in another branch of my build system Im now simply transpiling my own code with gulp-babel first and then doing a System.import of my own code's main module. Which works! It would be nice to ignore external module libraries with just build:false without listing every single one of the modules of the module library in the globalDeps.

A static build means building a module. Any module.must have its
dependencies explicitly declared.
Perhaps you just want a normal bundle which will support easy arbitrary
exclusions as each individual module is registered into the main registry.
On Wed, 30 Dec 2015 22:39 Joeri de Gooijer<EMAIL_ADDRESS>wrote:

hmm ok. I kinda understand globalDeps now, thing is that angular2/core is
not really a global dependency. It does not set anything on the window, its
a System.register('angular2/core') module. I tried your example and
actually it now works for angualr2/core. Unfortunately it then simply
errors on the next angular2/somemodule import. So I tried star patterns:
globalDeps: {
'angular2/': 'angular2/'
}
Star patterns don't seem to work, so it would require me to list every
single module im importing from angular2 anywhere in my code in the
globalDeps :(
Btw, in another branch of my build system Im now simply transpiling my own
code with gulp-babel first and then doing a System.import of my own code's
main module. Which works! It would be nice to ignore external module
libraries with just build:false without listing every single one of the
modules of the module library in the globalDeps.
—
Reply to this email directly or view it on GitHub
https://github.com/systemjs/builder/issues/434#issuecomment-168070038.

Ah right. Yes if I do .bundle and not .buildStatic it does transpile without having to declare all the globalDeps. Awesome. I guess I then just have to manually System.import the main module to execute the app. Thanks for your help and keep up the great work, cant wait untill you have your babel6 plugin ready.

Thanks @robwormald for the update here! Just et me know if there is anything further I can do to help with any of these configurations.


Insertion Sort Using C++

Contribution For Hacktoberfest

Not accepting CPP/C DSA codes
Alternative ->
1)Create a page using HTML & CSS Add DSA problem statement and solution in that
2)You can contribute in python and java OR
Ask for an issue assign
3)Contribute to Web Project


canplaythrough event not removed when AudioTag sound used

Loader does not remove canplaythrough event from audio tag when loading audio if using AudioTag audio. It tries to remove fileComplete function, but there is no such as it's wrapped when attached. This causes "Phaser.Loader fileComplete invalid index 0" errors when audio is played back.
This with phaser 2.1.3 and latest Chrome.

I believe this was addressed in the Loader update in 2.3.


Switch to feedparser for fetching planet latest posts

After Google's Feed API going away we need an alternative way of fetching latest post from reps planet.
Taking a "minimum effort" approach the best solution probably is to do this at the backend. I removed deprecated js code and added feedparser, which takes care of everything and even fails silently in case the planet doesn't respond.

Thanks. I amended the commit.


Supports multiple profile cards

Had to undo all my changes in between to avoid conflicts. ;-p

Is it necessary to use createElement instead of innerHTML to get this? It will make code complicated to work on. Also as per my knowledge for more number of elements createElement is slower than innerHTML in most of the browsers.

Yeah, I think you are right. We don't need createElement in this case.


Reddit button looks strange in development

Can anyone please have a look?
Don't want to look into it myself.
The orange square on this screenshot is Reddit icon (Firefox v.51, development environment).

In Chrome it shows up as a white square on orange background.

I made a quick fix by copy-pasting the SVG file from the gem into my assets and overriding Reddit button background with it.
.social-share-button .ssb-reddit { background-image: image-url('reddit.svg'); }


Please install the required when the 'install' target is called

It currently doesn't install anything.
install.sh wouldn't do because it doesn't do the right things:

It uses sudo that isn't needed to install the project
It expects /usr/local/bin to be writable which isn't what should be done to install

The install target should install into $DESTDIR$PREFIX and should never need to use sudo.

install.sh doesn’t require sudo, it offers if it cannot write into the installation directories.
The directories in which to install are fully customizable. The installer states this every time it is run:
If you prefer different locations you can rerun me and provide me with other locations like so:
	 ./install.sh [binary location] [packages location] [include location]

The installation process should be fully automatic. There's nobody there to read what the installaion script states.

Feel free to open a pull request that makes the necessary changes for this to run automatically (if possible in terms of calling install.sh, this is what I wanted to hint at).

Sorry, I don't have resources. I am only a package maintainer.
Project authors are expected to make projects installable, otherwise users would simply have no chance to even see your software because most people install software using package managers these days.

There is absolutely nothing the install.sh script that CMake's install() function can't do, and better


More flexible answer generation

Really like the interface and very impressed with the quick progress on the package! I've been exploring different approaches to using the raw value returned from documents in question answering. I did some experiments and found that instead of the refining approach, it works better for me to do a "hierarchical" approach and summarize and then combine. See overview here.
I did find a way to get the raw sources out via this approach:
query = gpt_index.indices.query.vector_store.simple.GPTSimpleVectorIndexQuery(
                        index.index_struct,
                        similarity_top_k=k,
                        embed_model=index.embed_model,
                        prompt_helper=lambda: None,  # dummy
                    )
queries[index] = query
query = queries[index]
nodes = query.get_nodes_and_similarities_for_response(question)

This was fine, but I couldn't figure out how to store the metadata I wanted in the nodes too. Maybe this is me missing something in the docs, but I wanted to store page numbers in the metadata to use later on. I couldn't figure out how to do that though.
TL;DR
So here's what I would propose:

[ ] Have two paths for answering: one based on refinement, one based on summarizing from multiple sources see prompts
[ ] Enable metadata to be stored with documents that can be access in prompt templates
[ ] Make it possible to easily get nodes - without instantiating query obj
[ ] [minor] Expose max_marginal for FAISS (maybe already done?) - which I found improves diversity of responses

These are just my observations in my use-case, it seems that gpt-index has been very successful for others.
I'm happy to help write code on this - in fact I tried to - but this seems to involve some deep rethinking of library. So - happy to discuss and contribute where I can - but will need help from gpt-index team.

I think this is a great idea! Will let @jerryjliu address the best way to PR, but specifically regarding the metadata, you can include it using extra_info when creating a Document. So like:
doc = Document(text, extra_info={"page number": 1})

For example, SimpleDirectoryReader uses this to enable you to pass in arbitrary metadata for files. This metadata is then prepended to each node if the document is split up. The ArxivReader uses it to pass in things like the title of the paper, the author, date published, etc.

Great thanks! I actually want to format the metadata myself in the prompt - maybe this is part of the larger changes; but I couldn't figure out how to format the metadata as I wanted. It just seems to appear in the prompt.

Ah yeah more fine-grained customization there would be really useful for sure. Looking forward to your PR :)

@whitead not sure if this helps, but following up on jesse's point, you can set include_extra_info=False when initializing the index. This way, you can still propagate metadata from documents to nodes to source nodes in the response, but it won't be injected into each node. Then you could still format the metadata yourself in the prompt (i think).

That solves part of my problem @jerryjliu! Do you think it's possible to have the node text and metadata be returned for a query, without all the boilerplate code above? Would that be something I can PR if not?

@whitead if i'm understanding correctly you could just fetch source_nodes from the response
response = index.query(...)
source_nodes = response.source_nodes
for source_node in source_nodes:
   # print node text and document metadata
   print((source_node.source_text, source_node.extra_info))

Got it - thanks @jerryjliu for helping me understand this. Does that still generate an LLM call from the query, aside from the query embedding?

Got it - thanks @jerryjliu for helping me understand this. Does that still generate an LLM call from the query, aside from the query embedding?

oops oh you can also disable the LLM call by setting response_mode="no_text" during the index.query call - that way we don't generate an answer at all, we only return the source nodes

Perfect - this addresses my use case. I can implement the other points in my own code.


Implement Curated Packages

What would you like to be added:
As a user, I would like the following functionalities to be added:

Installation of curated packages as part of cluster creation
Installation of curated packages in an existing cluster
Generation of curated packages configurations
Discovery of curated packages to be installed to a cluster
CLI commands to manage the lifecycle of curated packages
Downloading and importing curated packages artifacts to a private registry

Why is this needed:
As a user, I would like the ability to:

Extend the functionalities of my kubernetes cluster

As in Day-2 packages?
Close on-prem Registry e.g. Harbor package, Monitoring, Logging, backup, security baseline, ..etc
Also, DX, as in enabling some sort of PaaS e.g. DevFile?

This issue has been completed and all the functionalities have been merged.


Updated fh-wfm-workorder module

Changes
Related to changing the structure of the fh-wfm-workorder module in https://github.com/feedhenry-raincatcher/raincatcher-workorder/pull/15

LGTM


Meross Lan not loading

Hello everyone
Very new to home assistant so please bear with me.
After upgrading HA to
Home Assistant 2023.3.6
Supervisor 2023.03.3
Operating System 9.5
Frontend 20230309.1 - latest
and Meross Lan in hacs, meross lan wont load
this is the log
Logger: homeassistant.setup
Source: setup.py:189
First occurred: 13:14:33 (1 occurrences)
Last logged: 13:14:33
Setup failed for meross_lan: Integration not found.
I do have meross_lan installed under hacs, and it did do an update which took some time
any ideas would be greatly appreciated thank you

Hi Managed to resolve it reinstalled meross_lan and working now


[Request for External] page 508 "Blanket Sales Order Subform" - DeltaUpdateTotals

We would like to set procedure DeltaUpdateTotals() to global like it has been done in the other sales subpages so we can use it in page extensions:
procedure DeltaUpdateTotals()
begin
    //A108F0000.Event18.03 local=false
    DocumentTotals.SalesDeltaUpdateTotals(Rec, xRec, TotalSalesLine, VATAmount, InvoiceDiscountAmount, InvoiceDiscountPct);
    if Rec."Line Amount" <> xRec."Line Amount" then
        Rec.SendLineInvoiceDiscountResetNotification();
end;

We have this instead
procedure CallDeltaUpdateTotals()
begin
    DeltaUpdateTotals();
end;


Enable fields to be marked for omission from a struct

This way you can have structs that marshal / unmarshal as TLS, but also have extra information that is produced locally.  The proposed syntax is:
type Foo struct {
  A uint16
  B uint16 `tls:"omit"`
  C uint16
}

Why not just use lower-case for this? Those fields are inaccessible to reflection anyway.
For instance: https://github.com/ekr/minq/blob/master/codec.go#L155

Renewing this PR.  @ekr - you could use private fields for this, but I don't see a need to conflate the two ideas. encoding/json, for example, makes exactly this distinction, allowing a "-" annotation for a public field you want to omit in the JSON encoding.


[Fix #12832] Fix an error for Style/ArgumentsForwarding

Fixes #12832.
This PR fixes an error for Style/ArgumentsForwarding when using block arg in nested method definitions.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Looks good to me. Thanks!


various language & punctuation fixes in README.md

@theRealImy @bourgeoa -- Perhaps this could be reviewed and merged before any further edits are made downstream, requiring another round of merge conflict resolution?


Training on Multiclass Image Dataset

Is there already a way to use the code for multiclass image datasets? Documentation shows only for binary image datasets. Tried changing the "label_mode" variable in image_data.py to "categorical" and change the return value of the "number_of_classes" function to the number of classes. Still an error.

The first error was found at the metrics_fns.py file because of the incompatibility of shapes between prediction (None, ) and labels (None, {number of classes}). what I did was to add labels=tf.argmax(labels, 1) under the def _metric_fn(labels, predictions, weights=None) function of def make_accuracy_metric_fn(label_vocabulary=None):
Now the error says that there is incompatibility in sizes for the logits and labels under the loss_fns
tensorflow.python.framework.errors_impl.InvalidArgumentError: 2 root error(s) found.
(0) Invalid argument: logits and labels must be broadcastable: logits_size=[16,4] labels_size=[64,4]
[[node Phoenix/Trainer/softmax_cross_entropy_loss/xentropy (defined at D:\Programming\TuKoy\model_search-master\model_search\loss_fns.py:95) ]]
[[Phoenix/Trainer/Mean/_161]]
(1) Invalid argument: logits and labels must be broadcastable: logits_size=[16,4] labels_size=[64,4]
[[node Phoenix/Trainer/softmax_cross_entropy_loss/xentropy (defined at D:\Programming\TuKoy\model_search-master\model_search\loss_fns.py:95) ]]


Publish statically linked go binary

I'm trying to install Go 1.7.5 on Alpine from the official binary tarball. However, file shows that this binary is dynamically linked to /lib64/ld-linux-x86-64.so.2 (glibc), which makes it difficult to run this binary on embedded Linux systems.
Perhaps Go could publish these binaries as statically linked?
Perhaps Go could publish musl-linked binaries in addition to glibc-linked binaries?

Thanks for the report.
This is a dup of #18773 [Official Go binary does not work in Alpine Linux (not statically linked)], where they're asking pretty much the same thing:

we thought it would be good to have a statically-linked binary available

You are also suggesting another alternative (publish musl-linked binaries), but I think it would be better to keep all the discussion related to this issue on #18773; so I'm closing this. Feel free to write about publishing musl-linked bins on that issue.


Icons won't work with directive

Library Version: 0.1.2
Laravel Version: 7.23.0
PHP Version: 7.4.4

Description:
Icons in the package work as expected when using the Laravel 7 component syntax but don't when using the @svg directive or helper.  The directive gives a "File does not exist at path" error with a path that ends at the Laravel base level.
Steps To Reproduce:

Install the package in a Laravel project.
Add <x-heroicon-o-adjustments/> to a view to see the icon.
Add @svg('heroicon-o-adjustments') to a view and view the error.

Hey @jfsullivan. That works for me though. Have you tried php artisan view:clear?
Going to close this as I can't reproduce it but feel free to reply.

Sorry for wasting your time.  I still had Adam Wathan's old SVG package in my composer file and it was causing the issue.  I removed it and ran php artisan view:clear and all worked fine.
Thanks for helping and for all of your work on Blade UI Kit packages.

Glad you got it sorted!


jpype._jexception.RuntimeExceptionPyRaisable

Hi I am trying to run python wrapper but I'm getting   raise _RUNTIMEEXCEPTION.PYEXC("Class %s not found" % name) jpype._jexception.RuntimeExceptionPyRaisable: java.lang.RuntimeException: Class clojure.java.api.Clojure not found
I'm running simple example
 from duckling import Duckling d = Duckling() d.load() print(d.parse('tomorrow'))
I'm not sure what I'm doing wrong, may be in installations. I have installed
Duckling 1.8
JPype1 0.6.3
And I'm running it ancaonda env.

I have got the solution Thanks.

@SumitrajPatil  I am also getting same error. What was the resolution?

@SumitrajPatil I am also getting same error. What was the resolution?

Hey, I found the solution but I don't remember exactly what was the solution. Sorry for not responding with the solution earlier.

I ran into this issue as well and found the solution. I had been importing another python library that already initialized a JVM. So when I tried to initialize Duckling(), it skipped over the JVM creation and also skipped over the additions to the classpath. I was able to fix it by removing the other library which creates a JVM. But I would guess there is also a way to use the jvm_started=True flag and load the required jars into your existing JVM manually. Hope this helps!


Error: Unable to resolve module path from node_modules/libsodium/dist/modules/libsodium.js

Hi Team,
I have come across below error when, I had upgraded to DAF5
bundling failed: Error: Unable to resolve module path from node_modules/libsodium/dist/modules/libsodium.js: path could not be found within the project.
I had used the guideline https://github.com/uport-project/daf/blob/master/docs/Docs.md which was primarily for DAF 4 upgrade, where we are using DAF-libsodium But when I had compared the same in daf-mobile implementation, then it was daf-react-native-libsodium instead of daf-libsodium
https://github.com/uport-project/daf-mobile/blob/master/src/lib/setup.ts#L5
So should we use daf-react-native-libsodium instead of daf-libsodium for keyManagement?

Yes, For react native use daf-react-native-libsodium for Node use daf-libsodium

Hi @jasonhealy Thanks for your input , Issue had resolved after updating to daf-react-native-libsodium from daf-libsodium (for react Native)


Fuku02/fix

Thank you for your code review.
Fix pull requests.
1.ERROR: The named parameter 'maxLengthEnforced' isn't defined.
https://pub.dev/packages/input_history_text_field/score
MaxLengthEnforced => MaxLengthEnforcement
2.ERROR 'package:flutter/src/rendering/object.dart': Failed assertion: line 2462 pos 12: 'attached': is not true.
https://github.com/trashfeed/input_history_text_field/issues/5
Add OverlayEntry.remove()
3.Fix filterHistory
It seems to include the current input text in filterHistory.

@fuku02
Thanks for ↓ PR.
https://github.com/trashfeed/input_history_text_field/pull/9
We made some modifications to release it. Sorry. This caused a conflict.🙏

please merge this. works fine


Dedupe olegend and pover in Place.hs

Deep understanding of the code not needed.

Fixed in https://github.com/LambdaHack/LambdaHack/commit/e6c9021ab4b21da6f7297947da7b4065d36062e8. Closing.


Cost for Cloud Monitoring increased from $0 on Feb 14

I have a question to clarify. For my understand, it does means Investigate why increase from $0 after Feb 14. Am I understand exactly?
If not, could you explain about it please?

I have a question to clarify. For my understand, it does means Investigate why increase from $0 after Feb 14. Am I understand exactly?
If not, could you explain about it please?

Yes!

On February 29, Cloud monitoring costs disappeared 🤔


Pipeline

Hello and thanks again for the interesting work,
however, I wonder about your pipeline.
Why do you crop the images before feeding them to the network?
Additionally, is your early stopping really based on test_loss?
Greetings,
Thomas

Early stop is on test_loss/valida_loss.
We are cropping as we are following ProtoPNet pipeline


Fixed typo on Makefile templates (AVRDUDE)

Hello,
I just noticed that typo (but that doesn't change the functionality)
Regards,

thanks a lot! I so that the other day in another project and forgot to change it here :) Merging it now!