Storing Values with the Assignment Operator

Challenge Name
Storing Values with the Assignment Operator
Issue Description
Do not change code above the line
a should have a value of 7
b should have a value of 7
a should be assigned to b with =
Browser Information
In JavaScript, you can store a value in a variable with the assignment operator.
myVariable = 5;
Assigns the Number value 5 to myVariable.
Assignment always goes from right to left. Everything to the right of the = operator is resolved before the value is assigned to the variable to the left of the operator.
myVar = 5;
myNum = myVar;
Assigns 5 to myVar and then resolves myVar to 5 again and assigns it to myNum.
Instructions
Assign the value 7 to variable a.
Assign the contents of a to variable b.

The issue tracker is for reporting bugs only.
Please use the chat rooms:

HTML/CSS help chat room
JavaScript help chat room
Other Official Rooms

or try looking through our forum for help with a specific challenge.
Happy Coding!

What's the issue here? As I see in the screenshot, you need to follow the instructions and then run tests.

@sanjayk722 Please read the question carefully and code accordingly.The question is correct.

This challenge works, although i can understand the confusion...
Also returning the answer does not pass the challenge, because it is not specific to the instructions
for example
return c;
/*does not pass the challenge*/

If you're teaching someone to code, telling them how to assign a variable in one exercise, and then later in the exercise presenting unclear terms telling them to assign a set of variables that makes no logical sense to a newbie your teaching style is at fault, not the newbie's understanding of the instructions. Please rethink the instructions.

I had a problem with this question from https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/basic-javascript/storing-values-with-the-assignment-operator
They want you to
Assign the value 7 to variable a.
Assign the contents of a to variable b.
I have tried
var a =7;
var b=7;
var a = var b;      and also a = b;
It has across on:
Do not change code above the line
a should be assigned to b with =
I wonder why it doesn't like what I have put, how can I work out this trick question?


Warning message for routes added with addRoutes

Version
4.0.0-beta.13
Steps to reproduce
When I use addRoutes to dynamically add routes. [Vue Router warn]: No match found for location with path ”xxx” warning will appear. This warning appears before addRoutes. But does not affect the use
What is expected?
Remove the warning because I have added it using dynamic routing
What is actually happening?
Warning

Please open a new issue with a boiled down reproduction using the jsfiddle link provided on the issue helper.

This problem only occurs locally. I wrote a GitHub repo
Reproduction link
https://github.com/anncwb/vue-router-next-warn-repo
Steps to reproduce

Clone the repo (https://github.com/anncwb/vue-router-next-warn-repo)
yarn install
yarn run dev
go to http://localhost:3000/layout/pageA
Open the Google console
refresh page
appear  [Vue Router warn]: No match found for location with path "/layout/pageA"

The page is accessible, but this warning appears. Can it be deleted
@posva

I see what you mean now. If you are trying to create some kind of declarative routing, there are a few things that differ in terms of options (like using end: false) and that will be shown in the docs once they are created.
The warning is still helping a lot of people with typos and mistakes, so it's better to keep it.

What if I need to cancel this reminder? An urgent need


401 Unauthorized error when calling OData V4 Service on S4 HANA On Premise

Hi community,
we implemented an OData v4 service on a S4/HANA On Premise System and ran in a similar error like described here. Since there isn't a solution to fix the issue described and we had some findings, we decided to open an issue here.
Our On Premise system is connected via a cloud connector to our SAP BTP subaccount. When calling the OData v4 service via the executeHttpRequest function of the @sap-cloud-sdk/http-client module we get an 401 Unauthorized response.
When calling an OData v2 service on the same system we get an 200 OK response when running the same code only with another service path.
Do we have to configure something on our S4/HANA system to "allow" calling an OData v4 service from outside? Or is something wrong with our destination in the BTP subaccount or is our implementation only valid for v2 services? Or is it a bug in the cloud-sdk?
We are calling the http-client like this:
import * as httpClient from "@sap-cloud-sdk/http-client";

(...)

const destinationOptions = {
    "destinationName": "<MY_DESTINATION>",
    "isolationStrategy": "tenant",
    "jwt": "<MY_TOKEN>"
};

const requestOptions = {
    "url": "/sap/opu/odata4/mns/my_srv_v4/srvd/mns/my/0001/<MyEntity>", // v4 -- not working
    // "url": "/sap/opu/odata/MNS/MY_SRV_V2/<MyEntity>" // v2 -- working
    "method": "GET",
    "headers": {},
    "data": {},
    "params": {}
};

try {
	const response = await httpClient.executeHttpRequest(destinationOptions , requestOptions);
	(...)
} catch (error) {
	(...)
	// here we get the 401 Unauthorized when calling the v4 service
}

We debugged a bit in the node module and compared the request objects which are actually sent at this line of code:

The req object at this point looks exactly the same regardless which service we call.
OData v4 (which results in a 401 Unauthorized error):
{
    "baseURL": "http://<VIRTUAL_HOST_FROM_CLOUDCONNECTOR>:<PORT>",
    "headers": {
        "authorization": "Basic <BASE64_USER_PASSWORD_FROM_DESTINATION>",
        "Proxy-Authorization": "Bearer <JWT_FROM_CONNECTIVITY_SERVICE>",
        "sap-client": "<SAP_CLIENT_OF_ADDITIONAL_PROPERTIES_FROM_DESTINATION>"
    },
    "params": {},
    "httpAgent": {
        "_events": {},
        "_eventsCount": 0,
        "timeout": null,
        "maxFreeSockets": 1,
        "maxSockets": 1,
        "maxTotalSockets": null,
        "sockets": {},
        "freeSockets": {},
        "requests": {},
        "options": {},
        "secureProxy": false,
        "proxy": {
            "host": "connectivityproxy.internal.cf.eu10.hana.ondemand.com",
            "protocol": "http",
            "port": 20003
        }
    },
    "url": "/sap/opu/odata4/mns/my_srv_v4/srvd/mns/my/0001/<MyEntity>",
    "method": "GET",
    "data": {}
}

OData v2 (successful):
{
    "baseURL": "http://<VIRTUAL_HOST_FROM_CLOUDCONNECTOR>:<PORT>",
    "headers": {
        "authorization": "Basic <BASE64_USER_PASSWORD_FROM_DESTINATION>",
        "Proxy-Authorization": "Bearer <JWT_FROM_CONNECTIVITY_SERVICE>",
        "sap-client": "<SAP_CLIENT_OF_ADDITIONAL_PROPERTIES_FROM_DESTINATION>"
    },
    "params": {},
    "httpAgent": {
        "_events": {},
        "_eventsCount": 0,
        "timeout": null,
        "maxFreeSockets": 1,
        "maxSockets": 1,
        "maxTotalSockets": null,
        "sockets": {},
        "freeSockets": {},
        "requests": {},
        "options": {},
        "secureProxy": false,
        "proxy": {
            "host": "connectivityproxy.internal.cf.eu10.hana.ondemand.com",
            "protocol": "http",
            "port": 20003
        }
    },
    "url": "/sap/opu/odata/MNS/MY_SRV_V2/<MyEntity>",
    "method": "GET",
    "data": {}
}

The Proxy-Authorization Bearer token and the authorization string are identical in both requests.
In both requests the destination is fetched correctly from the BTP subaccount, so we could see the URI, username and password of the desired S4 HANA system:

In the logs of our Cloud Connector we also see that the request to the S4/HANA system is "allowed":

... but at some point the 401 Unauthorized is thrown.
Used Versions:
@sap-cloud-sdk/http-client: 3.2.0
@sap/cds: 6.8.4
@sap/cds-compiler: 3.9.4
@sap/cds-dk: 6.8.3
@sap/cds-dk (global): 6.8.3
@sap/cds-foss: 4.0.2
@sap/cds-mtx: 2.6.4
@sap/hdi-deploy: 4.6.1
@sap/instance-manager: 3.5.3
Node.js: v18.16.0
Kind regards,
Johannes

Hey @thisisevanfox,
from what I can tell the SAP Cloud SDK is doing the expected authentication flow, the fact that it's still throwing a 401 indicates you are doing something wrong on your end.
To narrow it down, you could try to generate a client for your service with one of our generators.
If it also fails with one of these clients, it is definitely a setup issue, meaning you would have to change something, most likely in the hana.
Unfortunately, we're no HANA experts, therefore I would have to advise you to ask on answers.sap.com and follow up on the issue you've linked there.

Hi @tomfrenken,
we have reimported the certificates on our OnPremise system, now it works. Sorry for bothering you and thanks for the quick reply.
Kind regards,
Johannes


Future of the project

Hey everyone! 👋
Bit bittersweet to write this, but I think the best path forward will be to wind down this project and move any consuming applications to Launch Library 2 API by The Space Devs.
Biggest reasons why this is probably for the best:

Nearly all of the data points in this API have been adopted by LL2, especially when it comes to SpaceX specific reusability data
LL2 has a strong community surrounding it with timely, crowdsourced data updates
Data updates for this project have been tough for 1 person to keep up with, especially as SpaceX has ramped launch cadence. LL2 has a community of enthusiasts who keep the data current, so it seems counterproductive to fragment data update efforts

Bittersweet because this project has been a big part of my life over the past 6 years, but I can't give the project the time it deserves, especially on data updates. This project helped spark my interest in software development, taught me the value and importance of the open source community, and even helped me get my first professional software job. It's been a privilege to lead an open source project of this magnitude, and I deeply appreciate all the community issues, PR's, and feature suggestions over the years.
So this is the current plan, pending community feedback: In its current state the API is cheap to run on a VPS, so I'm happy to keep it running to avoid broken links around the internet, but there won't be any more data updates going forward from myself.

Thanks for the update

Your project helps me a lot in building my website.
Really appreciate all of your work!

Thanks for everything, @jakewmeyer! Your work has meant the world to me. Hope we get to meet sometime and grab a drink.

Thank you for all your hard work @jakewmeyer! This API has been a blast to use, and I couldn't agree more to your closing remarks –  this API was a special one for me as well. Thank you again & best of luck on any future endeavors!

please continue this, OSS is awesome

Thank you for all your efforts over the years! equally rewarding for us users of your service!! 🚀🚀

I've used your site a lot over the years for teaching space science to kids in Denmark. I'm sorry to see it end, but I want to thank you for providing this - I know it's been a source of inspiration for many of my students.

Bit late to this, but thank you so much for the hard work you put in over all of those years @jakewmeyer! It's been my go-to API for a very long time and I really appreciate you keeping it online for the foreseeable.
Thanks again!

Thanks for the data over the years! I made a pull request to update the README so people aren't confused about why the data in the API is stale. https://github.com/r-spacex/SpaceX-API/pull/1329

Nice

Cosmorama

😊

Cosmorama


Use ID To Make Sure Test Results are Correct

Right now we're getting the test results as is, however we should probably use an id to make sure we aren't just getting an older version.

Using a temp store that clears itself after use see ad59e5d31cb9bd3933614ed68bd440d1f7e5ab65

This does not work. We will likely need to use an ID (thinking like hash(date + time + runcount)) to make sure its unique?

This should be solved by af28db03a75beda9b210a8775347a23039bb3593 (but we'll need to update the cli for this + other stuff)


escalate some errors

removing some code with deprecation errors, upgraded two warnings into errors
stub.py now 583 lines 🎉  (i think it was almost 1k lines earlier this year)

@prbot approve


Headphones Integration

Hey, I love the software a lot. To make it more spotify-like, are there any plans to integrate something like Headphones (https://github.com/rembo10/headphones) for new music discovery and adding on the fly? Cheers.

@Tomtoise what do you mean integration. just use headphones to download into the music directory and scan that.

I don't see this happening. There's very little chance that Koel will support downloading (illegal) music. That part you'll have to handle yourself.


MQTT 异常断开errno=Software caused connection abort (IDFGH-2953)

您好，IDF3.2.2SDK  MQTT直接使用IDF3.3.1内版本，经挂机测试12小时左右设备MQTT异常断开多次，请帮忙看下原因，谢谢！断开连接部分log如下
[19:58:18.539] NTP send: ip = <IP_ADDRESS>, port = 123
[19:58:18.539] Date & Time (CST): 2020/3/24-2-19:58:13
[19:58:18.570] Date & Time (UTC): 2020/3/24-2-11:58:13
[19:58:18.570] E (17191377) esp-tls: read error :-76:
[20:43:51.137] E (17191377) TRANS_SSL: esp_tls_conn_read error, errno=Software caused connection abort
[20:43:51.137] E (17191377) MQTT_CLIENT: mqtt_message_receive: transport_read() error: errno=113
[20:43:51.137] E (17191387) MQTT_CLIENT: mqtt_process_receive: mqtt_message_receive() returned -1
[20:43:51.137] [APP MQTT] MQTT_EVENT_DISCONNECTED
[20:43:51.137] I (17203587) MQTT_CLIENT: Sending MQTT CONNECT message, type: 1, id: 0000
[20:44:03.328] [APP MQTT] MQTT_EVENT_CONNECTED

Hi @shenyong-iot01
This means the connection was closed from the server side. Perhaps you can get some information what has happened from the server side log?

@david-cermak 谢谢您的回复，我将和服务器端沟通。另外这两个log打印相差将近1小时，系统时间戳却是一样的17191377，是否函数内阻塞...
[19:58:18.570] E (17191377) esp-tls: read error :-76:
[20:43:51.137] E (17191377) TRANS_SSL: esp_tls_conn_read error, errno=Software caused connection abort

@shenyong-iot01
Can you please share the exact version of the SDK you use? e.g. running
git rev-parse HEAD

There was such an issue, that the socket read/write blocked for a long time if error comes. This has been fixed in https://github.com/espressif/esp-idf/commit/11995d808c7963e1692047e44af3f09c83d2c121
So if you're on 3.2.2 (https://github.com/espressif/esp-idf/commit/055943e29346e77c50589c61a8a26101a8b35d7b), seems like the fix is not included. Can you please check if the issue goes away if you switch to the 3.2 release branch or cherry-picking that commit?

之前使用ESP32 SDK version: v3.2.2-5-gf90a429-dirty，MQTT 使用IDF3.3.1内版本。我后续将sdk更新到ESP32 SDK version: v3.3.1-dirty后再复测这个问题，谢谢！

ESP32 SDK version: v3.3.1-dirty 内修复相应问题，我这边按照11995d8修复方法添加进去，目前未复测到长时间阻塞，这个问题暂时可以关闭，谢谢！


[hotfix][doc] Fix the incorrect description for TO_TIMESTAMP function

What is the purpose of the change
The function TO_TIMESTAMP is independent of timezone. Therefore, we need to remove the misleading description that refers to it being 'under the UTC+0'.
Brief change log
change the function description.
Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
Yes

cc@leonardBang

@MartijnVisser Thank you for the suggestion, I have updated the pull request.


node: make apiEndpoint an alias for servicePath

Fixes #2787. Also, adds some test coverage for the new generated static method.

The Ruby test failure is obviously unrelated and is @hzyi-google and @michaelbausor are looking into it.


'shard' and 'raw' have same training speed.

When preprocessing data, I don't generate tar, but directly save WAV and TXT in the folder, and then used 'raw' for training,  training speed is almost the same with 'shard'.

shard is faster than raw on very large data.

shard is faster than raw on very large data.

How many hours above is very large data?

shard is faster than raw on very large data.

@robin1001 do you have an example? like the training speed of some specific hours' data when using shard and raw,  respectively.

shard is faster than raw on very large data.

@robin1001 do you have an example? like the training speed of some specific hours' data when using shard and raw, respectively.

I use more than 30k hours data, and 8A100 GPU with 16 accum_grad, 50 batch_size, and 8 num_workers. When use raw data, i get ahout 100sec/100batches.
the log is as following:

I wonder how fast when I use "shard" mode, I will do some experiment later.

shard is faster than raw on very large data.

@robin1001 do you have an example? like the training speed of some specific hours' data when using shard and raw, respectively.

I use more than 30k hours data, and 8A100 GPU with 16 accum_grad, 50 batch_size, and 8 num_workers. When use raw data, i get ahout 100sec/100batches. the log is as following:
I wonder how fast when I use "shard" mode, I will do some experiment later.

I use shard mode, with 8A100，16 accum_grad, 32 batch_size(because max_length of audio increase from 12s to 20s), and 8 num_workers, and I get the following log:

considering the decrease of batch_size(the speed of per 100batch will be faster), the total net speed of shard is truely faster than raw.
Additionally, the TAR operation of the total 30k hours' data cost about 26 hours, which is worthy.
Another advantage of using SHARD mode I think is its training stability. When I use raw mode, training will be slower when runing last some time, and the training even could be stuck, because of IO. For now, SHARD mode is satisfactory.


Update stop-cluster-upgrade-api-breaking-changes.md

According to the AKS release notes, this feature is no longer in preview. https://github.com/Azure/AKS/releases/tag/2023-10-08
Removed all references to preview, including for the CLI, since it appears that the az aks update command now has the --upgrade-override-until parameter https://learn.microsoft.com/en-us/cli/azure/aks?view=azure-cli-latest#az-aks-update

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

#sign-off

Closing this PR because the changes are already reflected in the live article: https://learn.microsoft.com/azure/aks/stop-cluster-upgrade-api-breaking-changes


Better representation of enums

I need to parse log of protobuf messages, generated by {:?}, back into protobuf objects, to replay messages. I forked RON deserializer, and made few changes to be able to deserialize {:?} representation. However, I cannot deserialize prost objects, because they use i32 instead of enum to represent enum, so information about actual type of field is lost. Unfortunately, this is limitation of Rust language itself. I hope, it will be fixed soon when RFC 2008 #[non_exhaustive] will be implemented and will work properly for #[repr(C)] enum's.
As workaround, I propose to use union(enum, i32) instead of plain i32, to save information about type. My proof of concept can be found here:
https://play.rust-lang.org/?gist=6131d0fcadd0bf2bba48a3efe26278f0
prost already generates is_valid() method for enum's, so it will not be hard to add generic union, use it instead of i32, and implement IsValidEnum trait for every protobuf enum.

Closed in favor of #43, because #[non_exhaustive] tag is stabilized/


[Feature Request] Callback functions in styled.attrs

So I've been looking far an wide to find a solution to a problem I'm trying to solve - and unable to come up with a decent answer I figured maybe this feature just simply doesn't exist yet.
My current problem is that I'm trying to create a responsive Menu/Navigation that doesn't rely on media queries.  While its been a good step forward I can't help but feel theres a large unecessary overhead with always doing 3-4 different media queries for various screen widths as this also means that in practice we're still very tightly coupled to other components in the application.
My solution so far has landed me with having to use a ref on my wrapper element to identify when a child element has an offsetTop that is larger then its siblings to identify when my menu starts wrapping elements with flex-wrap.  Its not ideal though because the React folks removed the ability to set a ref directly on a functional component, and adding a forwardRef to the child component just because one of the parents components where it might be used needs it isn't really following the philosophy of creating reusable components IMO.
It should also be mentioned I'm very fond of using a SFC pattern with styled components where I write:
import styled from 'styled-components';

const MyComponent = ({ className }) => {
  componentCode();
  return (
    <div className={className}>
      {children}
    </div>
  )
};

export default styled(MyComponent)`
  ${props => css`
      ${someStyles};
    `;
  }
`;

This is one of the reasons I'm so fond of styled-components, because it allows me to write code in what I find to be a very logical and legible way.
However, I wish there was more flexibility when working with styled-components this way - and what I find myself lacking now, is the ability to use variables defined in my component.  I imagine the easiest and most logical way to implement this would be by setting a prop as a callback in styled(MyComponent).attrs({ callback: value => value }) for example, to then get access to the value through the callback function in the styles similar to how you would do in a regular React component when passing props back up to the parent.
In fact I was even expecting this to be possible, but when testing it out I find that my callback function is converted to a string when I try to use it in my styled-component.
Would this be a terribly difficult feature to implement?  Or is there some other, obvious feature I'm missing here that solves the same problem?  I imagine this would be useful in many other scenarios too.

The .attrs() function now expects a factory function itself, since before otherwise you'd have to pass .attrs({ prop: () => () => 'value' }) so that's possibly related. I'm not quite sure what the proposal or question here is from the entire write-up, since there's no clear problem statement. Generally, if you use attrs(() => ({ ... })) you should be able to pass callback props to your underlying component.
Although, you do have to be careful in that attrs will always be forwarded to the underlying element or target, meaning that they'll be attributes on your HTML element, if they're forwarded, See: https://github.com/styled-components/styled-components/blob/d96101a102b59a8f9ac4b6f278594674897282bb/packages/styled-components/src/models/StyledComponent.ts#L132


Question: How to use withAWS() in parallel

Hi,
I've noticed that when I execute statements similar to
parallel "us-east-1": {
  withAWS(region:"us-east-1") {
    //Do something in us-east-1
  }
}, "us-east-2: {
    withAWS(region:"us-east-2") {
    //Do something in us-east-2
  }
}

this doesn't seem to work. I'm guessing the command is setting an ENV var or something similar so the statements are interfering with each other when executed in parallel. Is that the case, and if so is there any way for me to structure this so that they can execute in parallel without impacting each other?

@KenHundley try something like this then:
parallel(
    'us-east-1': {
        env.AWS_DEFAULT_REGION = 'us-east-1'
        sh ' do some staff ' },
    'us-east-2': {
        env.AWS_DEFAULT_REGION = 'us-east-2'
        sh ' do some staff ' }
)

@Constantin07, I will try this on Monday and see how it goes. Ty for the suggestion

Sorry, work got busy this week and didn’t test it yet, but I have some time now. Will let you know

@Constantin07 @hoegertn
This suggestion worked perfectly
withAWS(credentials:'prod') {
    parallel "us-east-1": {
        withEnv(['AWS_DEFAULT_REGION=us-east-1']) {
            sh "aws command"
        }
    }, "us-east-2": {
        withEnv(['AWS_DEFAULT_REGION=us-east-2']) {
            sh "aws command"
        }
    }, "ca-central-1": {
        withEnv(['AWS_DEFAULT_REGION=ca-central-1']) {
            sh "aws command"
        }
    }
}

Ty very much for the suggestion @Constantin07!


DownArrow, LeftArrow, etc. should be symbols on Windows

Is there a reason we don't show the symbol here? Perhaps because older versions of Windows didn't have the symbols in the default fonts that we no longer support?

Relevant code:
https://github.com/microsoft/vscode/blob/b03fd7c49878d0431ab57d803c3f1f0f9a3488ed/src/vs/platform/keybinding/common/usLayoutResolvedKeybinding.ts#L21-L35


created a variable to hold the completed default preferences

created a new variable:
defaultPreferences

set it underneath the task create default_preferences to be the completed run of this function.
setup under task lint & lib the new variable
ran npm test and confirmed that it completed without issues.
Please let me know if there are any other changes that need to be made!
Related to #11414.

This honestly doesn't seem like an appropriate solution, since running any particular gulp task by default every time that the gulpfile is loaded quite frankly seems incorrect.
The correct approach would probably be something along these lines instead: https://github.com/gulpjs/gulp/issues/1392

So,
I've reviewed the link you shared.
The way they  got rid of the second call was to setup a new task that called all the other functions at once together.
I tried to setup something similar however, the newly created task never runs?
gulp.task('createPreferences', gulp.series('default_preferences', gulp.series('default_preferences-pre'), gulp.parallel('lib', 'lint') ));

I have set this up underneath the tasks lint and lib but it's giving me an error related to the settings.

I think the problem is outlined well in https://github.com/gulpjs/gulp/issues/1392#issuecomment-157161298. Our code is converted from Gulp 3, where we had the concept of dependencies in the API, to Gulp 4 which doesn't have this concept in the API anymore, without taking this into account, so everything we now treat as a dependency is actually executed multiple times. Normally that's not really a problem aside from increased build times, but in the case of the default preferences it is.
Hence we should look at this in a broader sense: how do we map Gulp 3's dependency model onto Gulp 4? The linked issue should give some ideas about how to do this, but there is also https://fettblog.eu/gulp-4-parallel-and-series/ which seems to explain it well.

created a task to generate the default_preferences
then a second task under lib and lint that calls the series on first task and parallel on lib and lint
[23:21:54] Starting 'createPreferences'...

[23:22:00] Finished 'createPreferences' after 7.36 s

[23:22:00] Finished 'createPreferences' after 5.63 s

createPreference is started once and finishes twice once for lib and one for lint

so the tests are completely locally on my side when I use nvm with node v8.16.0
Is there a reason that travis ci file is setup to use node v13.5?

It's very difficult to understand/follow these changes, and looking at the long list of commits I'm sorry to say that it does seem a bit like you've just tried a bunch of things without perhaps a clear plan as to the end result. (There's at least one newly added gulp task that doesn't even appear to be used.)
Furthermore, I'm not sure that these changes are that close to addressing the underlying issue, since the whole point here is that the 'default_preferences' task should only run once for each npm test invocation (which will most likely require more invasive changes, as alluded to in https://github.com/mozilla/pdf.js/pull/11411#issuecomment-566785830). All in all, it might be that some prior familiarity with gulp is necessary to more easily make progress on the issue.

I was pushing a number of commits because things were passing locally on my computer but when they hit you're side it was failing. I'm following the guide you provided in which they separate out the individual tasks into functions and then create a task to run the function as needed.

to confirm that default_preferences function is only running once, included a console.log at the top

I'm also having a hard time understanding why this actually fixes things, because as far as I can tell we still run the task multiple times, but with more indirection. Why does this actually make it run once?

As I've said elsewhere, this probably isn't really a good beginner bug unfortunately.

Isn't this issue already closed by you're commit?
Fixed by #11413
so I can just close this right?

The intermittent failure is fixed by that, but the underlying issue is not. We have made a separate ticket to fix that, but as indicated this is a bit harder than we expected, so let's indeed close this and come up with a good plan for that. Thanks.

I'm also having a hard time understanding why this actually fixes things, because as far as I can tell we still run the task multiple times, but with more indirection. Why does this actually make it run once?

As I've said elsewhere, this probably isn't really a good beginner bug unfortunately.

I mean I wish it could have been properly labelled then since I spent almost two days trying?

Yes, unfortunately sometimes it's not entirely clear how hard it will be until it's actually being implemented, which was the case here. Sorry for that.

Also,
@Snuffleupagus
I just wanted to leave a comment that if someone on you're side is going to take over an issue, it seems like common courtesy to notify the person who currently trying to fix the problem?
When I asked to work on this issue, it was assigned as a beginner issue with no priority noted for how quickly this needed to be completed. I feel that at 2 days it was still acceptable for me to be trying to resolve this and it shouldn't have been ripped away from me.
Thanks


better translation

"app_web": "Browser",
"taskmanager_launch": "Lanceer", >> "taskmanager_launch": "Open",

Nice, btw we have GitLocalize to make/improve translations now
https://gitlocalize.com/repo/5170


improve README documentation.

add comments for -log-dir option
fix section hierarchy
remove trailing spaces
some fix for markdownlint
some misspelling fixed

Many Thanks :)


Google Cloud Drive files are no longer valid.

I noticed that the link you uploaded on Google Cloud Drive is no longer working, could you please re-upload it so I can reproduce it better? Thanks.

tensorflow for deep learning models, and matlab for data processing

But very sorry, the problem is still not solved, I may have misrepresented your link 'https://drive.google.com/file/d/0B5Hl9mO74DHvUEowa1hyWmVsMmc/view?usp=sharing' and it's not working.

Sorry it might be a problem with my previous formulation, your link https://drive.google.com/file/d/0B5Hl9mO74DHvUEowa1hyWmVsMmc/view?usp=sharing
It's still not working, I can't open it.Thanks.


feat: add userprofile and fetch memes based on dropdown option

Description
Added user profile in my-meme section and also added the dropdown option of the meme owner whose meme you have seen previously and from down by selecting the owner you can see all the memes related to that owner.
Screenshots

Please review it and let me know if you need any changes.

@miralsuthar I feel like it's a bit confusing to have the memelord search on my memes.
It would be nice if you create a profile page for each meme lord.
So in the pages folder you would have /profile/[memelordAddress].tsx
And there you can see more infos about the memelord and all his memes.
Also when you're on a meme and you click on the meme lord it would bring you to his profile page.

yeah, that would be nice to have a profile of every single memelord, got your point working on it. 👍

Hey, @Cali93 I have made the changes as discussed above, take a look


Tab names for SSH session to Windows 10 OpenSSH host

Environment
Windows build number: 19041.208
Windows Terminal version (if applicable): 0.11.1251.0

Any other software?

Steps to reproduce
Open a new tab by SSH to a Windows 10 OpenSSH host.
Expected behavior
New tab title contains machine name or IP of host.
This would help identify tabs when multiple SSH sessions are active.
Actual behavior
New tab title contains "C:\Windows\System32\conhost.exe".
There is no SSH related host identifying information in the tab.

So, two things you can do today:

Configure your remote shell to put the hostname and path in the terminal title

This works today, and many stock distribution shell configurations do this by default.

Wait for /dup #1079, in PR at #5775.


Chromium crashes if using a combination of headless, incognito, and single-process

Chromium crashes while using a combination of headless, incognito, and single-process.
Steps to reproduce
Tell us about your environment:

Puppeteer version: 2.0.0
Platform / OS version: MacOs 10.14
URLs (if applicable): N/A
Node.js version: 12.13.0

What steps will reproduce the problem?
Launch puppeteer with options:
{
  headless: false,
  args: [
    '--incognito',
    '--single-process'
  ],
}

Please include code that reproduces the issue.
const puppeteer = require('puppeteer');

async function main() {
    const browser = await puppeteer.launch({
        headless: false,
        args: [
            '--incognito',
            '--single-process',
        ],
    });

    const page = await browser.newPage();

    await page.goto('https://google.com');
    const pageTitle = await page.title();
    console.log('Title:', title);
};

main().catch(err => console.error(err))

What is the expected result?
The title for google.com
What happens instead?
node example.js
Error: Protocol error (Target.createTarget): Target closed.
    at ~./demo/node_modules/puppeteer/lib/Connection.js:74:56
    at new Promise (<anonymous>)
    at Connection.send (~./demo/node_modules/puppeteer/lib/Connection.js:73:12)
    at Browser._createPageInContext (~./demo/node_modules/puppeteer/lib/Browser.js:174:47)
    at BrowserContext.newPage (~./demo/node_modules/puppeteer/lib/Browser.js:367:26)
    at Browser.newPage (~./demo/node_modules/puppeteer/lib/Browser.js:166:33)
    at Browser.<anonymous> (~./demo/node_modules/puppeteer/lib/helper.js:112:23)
    at main (~./demo/example.js:12:32)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
{
  message: 'Protocol error (Target.createTarget): Target closed.'
}

The browser crash report begins with:
Process:               Chromium [61359]
Path:                  /Users/USER/*/Chromium.app/Contents/MacOS/Chromium
Identifier:            org.chromium.Chromium
Version:               79.0.3945.0 (3945.0)
Code Type:             X86-64 (Native)
Parent Process:        node [61358]
Responsible:           iTerm2 [274]
User ID:               502

Date/Time:             2019-12-18 14:10:36.107 -0800
OS Version:            Mac OS X 10.14.6 (18G1012)
Report Version:        12
Bridge OS Version:     3.0 (14Y906)
Anonymous UUID:        1088BFF9-4A27-D608-A8A1-5B98417C9077

Sleep/Wake UUID:       885BBD08-6441-4848-8BE3-464FF37CF99A

Time Awake Since Boot: 260000 seconds
Time Since Wake:       3600 seconds

System Integrity Protection: disabled

Crashed Thread:        0  CrBrowserMain  Dispatch queue: com.apple.main-thread

Exception Type:        EXC_BREAKPOINT (SIGTRAP)
Exception Codes:       0x0000000000000002, 0x0000000000000000

From what I can tell, these three pieces are related:
OK:
headless=true
headless=false, incognito XOR single-process
Crash:
headless=false, incognito AND single-process
It may be worth noting that I'm successfully using this combination with the browser included in the chrome-aws-lambda project, but only in a deployed aws lambda, not locally.  The browser included with puppeteer does not seem able to handle this combination.
Any tips are welcome, I'm open to user error as a possibility.

Scratch that, I realized it's working in the other browser because it's running headless=true.
I can't use headless=false, incognito, and single-process together.
Can anyone else?  Is this just me?  Maybe it's just an undocumented and unsupported combination of flags?

I think this is related to this, could you check it?
https://github.com/puppeteer/puppeteer/issues/2608#issuecomment-393955311


Can't add Item to rebar

I tried to change my old dcxdll to a newer one found somewhere a compiled v3 (<IP_ADDRESS>) .
After changing dll I get this error (id 1100 is a rebar element)
D_ERROR /xdid ("dcConf 1100 -a 0 + 0 200 0 0 16711680 Skripte && Module 	 1101 treeview 0 30 200 440 showsel hasbuttons haslines linesatroot" error: Control with ID "1101" already exists)
Then I tried to check if it really exists with
//echo -s $xdid(dcConf, 1101).type
Dialog waqas still open, and I got
D_ERROR $xdid ("dcConf 1101 type" error: Invalid ID : 1101 (dialog dcConf))
changing back to my old dll and it works fine

try the latest C++14 branch builds & let me know how it goes.


Fix issue tgriesser/knex#1180

Fix the problem on installing project from the github.

Actually it still not working. Because it allow install the project without that error of cli.js not found. But since the build is only applied during the pre-publish. The lib files are not available on installing from github.
I don't like the idea of building the lib during the postinstall. Because it  requires installing "useless" dependencies for production environment such as babel. And it makes the installing slower.
There is a proposal for NPM include a prepublish for Git. But I guess it will not be implemented.
Seems most projects doesn't provide the build files on installing from git. So I guess the best option for #1180 is just not provide the lib on git. And anyone installing from git, changes their own fork-branch to include the lib files.

I think that this change would still be good. This way one can install from github and then add post install scripts to own project which builds the knex if he really wants to use something from github. To me this fix would be enoguh to close #1880.

This would also need manual testing since nothing in test suite is actually running the cli.js client installed by npm scripts.

Nowadays installing from github does not include the scripts folder. Which has the build.sh. I guess because is not included into the files property. So to allow users installing from github and later building the project I think would be good include the scripts folder as well.
We could use .npmignore to ignore it and even the src folder on installing from npm. I guess the .npmignore is only applied on publishing to npm. So would not affect those installing from github.
@elhigu  I can test these ideas and apply them if you think they are a good approach for the problem.

@maxcnunes Sounds good. I don't know much about npm packaging / so I cannot anticipate if that really is viable, but would be great if you can test it :+1:

https://docs.npmjs.com/files/package.json#files

files
The "files" field is an array of files to include in your project. If you name a folder in the array, then it will also include the files inside that folder. (Unless they would be ignored by another rule.)
You can also provide a ".npmignore" file in the root of your package or in subdirectories, which will keep files from being included, even if they would be picked up by the files array. The .npmignore file works just like a .gitignore.

So package.json specification confirms @maxcnunes words.
I was thinking if there could be a way to build (+ babel installing) with postinstall if lib directory is missing. This way it is not needed to explicitly invoke building from depending project.
#1425 would be a great improvement along with this one.

Yeah. The solution I was thinking was exactly this one using postinstall. Although I don't have time to implement this right now. Please, feel free anyone to take over.

I will look into it in few days.

So the latest 0.12 should allow you to install from github, but for now you'll also need to configure a script yourself that goes into the knex directory and runs npm install & npm run build

@tgriesser
There is no need for separate script that goes into the knex directory and runs ``npm install & npm run build`. It can be handled the same way as solved in https://github.com/tgriesser/bookshelf/pull/1403

Yeah, I guess I was just trying to make it possible but discouraged to install from github. But yeah, buildDependencies seems like a good compromise. Feel free to open a PR.


Update Spring Initializr to include current version of the app-starters

As a developer, I'd like to update the fork of Spring Initializr to pull the latest app-starters Currently, it is pointing to 1.0.0.M1, which is very old.

Resolved via https://github.com/ghillert/initializr-scs/pull/4


0.8.0 fails after reboot

I know this is already pointed out in the documentation, but since there is a thread on .7:
https://github.com/luxas/kubernetes-on-arm/issues/112
I thought I'd start one here, I mean we ought to at least document what to do after a restart, right now I'm doing this on the master:
kube-config disable
kube-config enable-master

and then on each worker I do nearly the same thing:
kube-config disable
kube-config enable-worker <IP_ADDRESS>

now back on the master everything looks happy:
# kubectl cluster-info
Kubernetes master is running at http://localhost:8080
KubeDNS is running at http://localhost:8080/api/v1/proxy/namespaces/kube-system/services/kube-dns
kubernetes-dashboard is running at http://localhost:8080/api/v1/proxy/namespaces/kube-system/services/kubernetes-dashboard

To further debug and diagnose cluster problems, use 'kubectl cluster-info dump'.

I have been letting the disable command wipe out the kubelet directory, anyone have experience persisting everything?

I'm going to close this issue to clear up the open ones a bit.  Feel free to add notes for anyone else here, I might not be doing the most 'optimal' route by just clearing out and restarting with yes Y|kube-config disable, and I'm open to discussion.
That being said, I'll close this up and leave what a dead k8s looks like after a reboot and right before I run the above script, just in case it helps some person get here by searching
[root@apollo ~]# docker ps
CONTAINER ID        IMAGE                                                COMMAND                  CREATED             STATUS
PORTS               NAMES
a1a823f1d169        gcr.io/google_containers/hyperkube-arm:v1.3.6        "/setup-files.sh IP:1"   15 hours ago        Up 10 hours
k8s_setup.e7e837b8_k8s-master-<IP_ADDRESS>_kube-system_ae686c7fc175b500b899801f6c8067a6_cad55286
7e85cd7d1574        gcr.io/google_containers/hyperkube-arm:v1.3.6        "/hyperkube scheduler"   15 hours ago        Up 15 hours
k8s_scheduler.e737fc67_k8s-master-<IP_ADDRESS>_kube-system_ae686c7fc175b500b899801f6c8067a6_d3fed3e6
deb3851baeb0        gcr.io/google_containers/hyperkube-arm:v1.3.6        "/copy-addons.sh"        15 hours ago        Up 15 hours
k8s_kube-addon-manager-data.1fa1945_kube-addon-manager-<IP_ADDRESS>_kube-system_12985f6c2276246edebe61849d91e5be_082d9e4d
2f0c542d0db3        gcr.io/google-containers/kube-addon-manager-arm:v4   "/opt/kube-addons.sh"    15 hours ago        Up 15 hours
k8s_kube-addon-manager.f42a8c48_kube-addon-manager-<IP_ADDRESS>_kube-system_12985f6c2276246edebe61849d91e5be_a491b356
e0d40ca05b11        gcr.io/google_containers/hyperkube-arm:v1.3.6        "/hyperkube proxy --m"   15 hours ago        Up 15 hours
k8s_kube-proxy.3d0a47fc_k8s-proxy-<IP_ADDRESS>_kube-system_a1f94bf6df71ad6ef8dd95542737efd3_cdf57f46
2a0834d87e8c        gcr.io/google_containers/pause-arm:3.0               "/pause"                 15 hours ago        Up 15 hours
k8s_POD.da6fe110_k8s-master-<IP_ADDRESS>_kube-system_ae686c7fc175b500b899801f6c8067a6_83921def
0cd2415df5be        gcr.io/google_containers/pause-arm:3.0               "/pause"                 15 hours ago        Up 15 hours
k8s_POD.da6fe110_kube-addon-manager-<IP_ADDRESS>_kube-system_12985f6c2276246edebe61849d91e5be_ebb792b7
e2f3b1e37b6c        gcr.io/google_containers/pause-arm:3.0               "/pause"                 15 hours ago        Up 15 hours
k8s_POD.da6fe110_k8s-proxy-<IP_ADDRESS>_kube-system_a1f94bf6df71ad6ef8dd95542737efd3_49e6f04f
90f2dd9ca1ea        gcr.io/google_containers/hyperkube-arm:v1.3.6        "/hyperkube kubelet -"   3 days ago          Up 15 hours
kube_kubelet_2cc6a
[root@apollo ~]# kubectl get nodes
The connection to the server localhost:8080 was refused - did you specify the right host or port?


Removing the Generics from the Dataflow *Configuration classes.

The Generics in the Builder hierarchy causes all sorts of issues.  Removing them required some additional verbosity in overridden methods.  I think that the change will increase framework code, but reduce complexity from the user's perspective.

LGTM


Ejecting non-existed CD-ROM causes VMI creating

Problem:
According to the current ejectCdRom implement,
https://github.com/rancher/harvester/blob/788d1a08a9e349d1d57238b73a8d7e687624ebee/pkg/api/vm/handler.go#L104-L114
if we pass a non-existed CD-ROM name to VM ejectCdRom action, it will delete the existing VMI. The template resource(VM) has not changed, so the generated resource(VMI) should not be recreated.
Solution:
We can enhance the ejectCdRomFromVM to validate the non-existed CD-ROM name.

Validated that ejecting a non-existing cdrom does not trigger VMI recreation


Error in ST_Contains

Expected behavior
celllocgeomvoronoi
root
|-- polygeom: geometry (nullable = false)
celllocgeom
root
|-- Center_Longitude: double (nullable = true)
|-- Center_Latitude: double (nullable = true)
|-- geom: geometry (nullable = false)
val celllocgeomvornoipolygon=spark.sql("select * from celllocgeom,celllocgeomvoronoi where ST_Contains(celllocgeomvoronoi.polygeom, celllocgeom.geom)")
Getting below error
A fatal error has been detected by the Java Runtime Environment:
EXCEPTION_ACCESS_VIOLATION (0xc0000005) at pc=0x0000000004477ee5, pid=11780, tid=0x00000000000052b8
JRE version: Java(TM) SE Runtime Environment (8.0_101-b13) (build 1.8.0_101-b13)
Java VM: Java HotSpot(TM) 64-Bit Server VM (25.101-b13 mixed mode windows-amd64 compressed oops)
Actual behavior
Steps to reproduce the problem
Settings
GeoSpark version = 1.2.0
Apache Spark version =2.1.0
JRE version = 1.8?
API type = Scala or Java? Scala

I don't thin this error is caused by GeoSpark


fix: Fix delete all models and CreateModels.AllEntityTypes

Fixed a bug I noticed in delete all models that was due to a prior code change I had made.
Fixed a bug in CreateModels.AllEntityTypes that was due to a change in product code.
Fixed the test runner changes I just made in my last PR due to wanting to keep DeleteAllModels out of the main list of tests and instead keeping it as a separate tool.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix exp. time kernel colors

IMPORTANT: Please search among the Pull requests before creating one.
Description

Fix color creating in :class:cellrank.tl.kernels.ExperimentalTimeKernel.
How has this been tested?

1 new test.
Closes

closes #782

Thanks @michalk8


trafilatura behing a proxy

Hi,
Trafilatura doesn't seem to work behind our proxy.
Config : a docker container, with openjdk11-slim-bullseye.
A dig soa works like a charm (so internet is up).
print env says :
no_proxy=*.c**************,<IP_ADDRESS>/8,<IP_ADDRESS>/8,*************12,<IP_ADDRESS>/16
HOSTNAME=***********************
JAVA_HOME=/usr/local/openjdk-11
PWD=/app
HOME=/
LANG=C.UTF-8
https_proxy=http://p********************************:xxxx
TERM=xterm
no_proxy=*.c**************,<IP_ADDRESS>/8,<IP_ADDRESS>/8,*************12,<IP_ADDRESS>/16
SHLVL=1
HTTPS_PROXY=http://p********************************:xxxx
HTTP_PROXY=http://p********************************:xxxx
http_proxy=http://p********************************:xxxx
PATH=/usr/local/openjdk-11/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/app
RELEASE=***********
DEBIAN_FRONTEND=noninteractive
JAVA_VERSION=11.0.15

Any idea?

Hi @hpiedcoq, I'm afraid the question is too specific for me to reproduce or to answer.
The underlying library used for downloads is urllib3, I suggest you try to run a simple script with a pool manager (i.e. like trafilatura) and see if it works. If it doesn't then it would be an issue related to urllib3.
Another option would be to use another download utility and then process the HTML files locally (on your docker instance) with trafilatura.


Adjustable label size

For me, the labels of my notes are too large and numerous, obscuring the whole visualization.

This is duplicate with #15. The node label should fade like the 2D graph view.


Cannot read properties of undefined (reading 'map')

caught TypeError: Cannot read properties of undefined (reading 'map')
    at eval (index.js:44:25)
    at Array.map (<anonymous>)
    at getSuperTagTraits (index.js:42:29)
    at getFullSchemaFromCurrentTiddler (index.js:89:21)
    at SupertagFormWidget.render (index.js:122:45)
    at Widget.renderChildren (widget.js:632:15)
    at ElementWidget.render (element.js:84:7)
    at Widget.renderChildren (widget.js:632:15)
    at ListItemWidget.render (list.js:342:7)
    at Widget.renderChildren (widget.js:632:15)

const traits = tags.map((traitTitle) => {


[Question]: sent-len of sentence-spliter

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
there is one arg of SentenceSplitter: chunk_size, i assume this could control the length of each split-sentence, but i found that sentence-length can be larger than the chunk_size, i want to know why this happens

@guangyuli-uoe the sentence splitter splits into chunks, while trying to respect sentence boundaries. It does not split and return single sentences.

@logan-markewich thanks !


Add Blockstrings support

This PR addresses #64 "GraphQL doesn't support 'Blockstrings' which are often used in Descriptions".
There are multiple ways to implement this support and I tried to pick one that matches the existing style of lexing/parsing.  Lots of tests are included, though I'm sure more could be thought of.
There's one piece that I just noticed is missing: support for escaped Blockstrings  \"""
I'm out of time for this project and don't actually need those (yet), so I'm going to punt on that for now.  Shouldn't be terribly difficult, but… ¯_(ツ)_/¯.
I hope you find these contributions acceptable in form and substance.  Feel free to let me know if there's things you'd prefer to be done differently.
All my work has been done on macOS 10.15.6 using Xcode 12b5, so I will be interested to see if this builds on linux.
Fixes #64

oi.  Forgot I was going to go back and try to clean up the test function names…

Hello, @haikusw. Thank you very much for your contribution! Just so you know, the structure of this repo mainly mirrors the structure of the javascript canonical GraphQL implementation. For your next contributions would be valuable to check how the js counterpart implements what you're after. For example:
https://github.com/graphql/graphql-js/blob/fe3040fffe7f34c8f8b8e1cb849f09138debbfab/src/language/lexer.js#L526
Most of the times a direct "translation" from Typescript do Swift is possible. 🙂

Hello, @haikusw. Thank you very much for your contribution! Just so you know, the structure of this repo mainly mirrors the structure of the javascript canonical GraphQL implementation. For your next contributions would be valuable to check how the js counterpart implements what you're after. For example:
https://github.com/graphql/graphql-js/blob/fe3040fffe7f34c8f8b8e1cb849f09138debbfab/src/language/lexer.js#L526
Most of the times a direct "translation" from Typescript do Swift is possible. 🙂

Ah, I did not realize this.  That does sound like it might have saved some time.  Thank you.

Hello, @haikusw. Thank you very much for your contribution! Just so you know, the structure of this repo mainly mirrors the structure of the javascript canonical GraphQL implementation. For your next contributions would be valuable to check how the js counterpart implements what you're after. For example:
https://github.com/graphql/graphql-js/blob/fe3040fffe7f34c8f8b8e1cb849f09138debbfab/src/language/lexer.js#L526
Most of the times a direct "translation" from Typescript do Swift is possible. 🙂

Ah, I did not realize this. That does sound like it might have saved some time. Thank you.

I see that they did what I did initially which was to make a new .blockstring token type. I ended up backing that out because it would have required a lot of checking for either .string or .blockstring token types for things like description where it can be either.  Seems like ideally I would go back and make this set of changes match the canonical GraphQL implementation since that is the essence of what this is.  I will try and do that.

Seems like ideally I would go back and make this set of changes match the canonical GraphQL implementation since that is the essence of what this is. I will try and do that.

That sounds great!

done:  #68


jimp.read() hangs on await/async

The following code block appears to hang for me.
await jimp.read(req.file.buffer, () => {});

Jimp in its current, published state has a weird mixture of Promises and callbacks. In your example, you are using both. await is to be used with Promises, but you also use the callback capability for the function. Specifically, your callback is a no-op which doesn't resolve or reject so when paired with await, it never finishes.
The easiest fix is to remove the callback like this:
(async function() {
  await jimp.read(req.file.buffer);
  // Do something after reading
})();

It should be noted that the current HEAD version of Jimp does not use callbacks for the read function so this confusion will eventually not be a factor.

Also remember there are two ways to write the files:
https://github.com/oliver-moran/jimp/tree/master/packages/jimp#writing-to-files


Extension method

Originally we would have written something like this:
Expression<Action> func = () => Console.WriteLine();
string result = func.ToString();

I would like for ExpressionToString to offer an extension method. The method name ToString is taken. Maybe call the extension method something like:

ToReadableString
ToAwesomeString
ToSafeString
ToStringSafe
ToText
ToString
Dump
GetString
AsString
AsString

(Naming things is hard)
I can do this if you want it.

This is why I released this as < v1. I thought about this too =)
.ToCodeString() maybe.

I am not particularly attached to any name (although I think the name should start with the word To for discover ability purposes).
Which name do you like the most? ToCodeString is fine in my opinion.

We could also be compatible with https://github.com/jbevain/mono.linq.expressions/#api

Ah. If I understand correctly, you want to call the extension method ToCSharpCode so users can swap out mono.linq.expressions for ExpressionToString with ease?

Unsure, will have a think. Just put it here as an option..


refactor(async)!: refactoring async client and minor documentation improvements

Description
It builds on top of #95. Applies minor improvements on some docstrings, and Cargo.toml standard.
The main change is adding the common HTTP methods for the AsyncClient, it removes duplicated code from each Esplora API request, and follows the approach done for BlockingClient.
It makes it easier to extract these methods into an AsyncEsploraClient trait (to be done in another PR, initially done here 9cbc3873c2a56258e13cb36e0e8c32039f0947a7), which the user can implement with any HTTP client of its choice. Also, makes it simpler to rebase and update the AsyncAnonymizedClient from #67.

Notes to the reviewers
It has some commits from #95, as it builds on top of it and should be merged afterward. Please let me know what you think about the proposed changes and approach.

Changelog notice

Applies minor improvements on documentation.
Add common get_response and post_request methods to AsyncClient, previously duplicated through the esplora API calls. It follows the approach done for BlockinClient.

Checklists
All Submissions:

[x] I've signed all my commits
[x] I followed the contribution guidelines
[x] I ran cargo fmt and cargo clippy before committing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

99 of 117 (84.62%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.2%) to 88.697%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/async.rs
99
117
84.62%

Totals

Change from base Build<PHONE_NUMBER>:
1.2%

Covered Lines:
1028

Relevant Lines:
1159

💛  - Coveralls

A recent PR #87 exposed a get_request method for the blocking client that returns a new Request, would you want to do a similar thing for the async client?

A recent PR #87 exposed a get_request method for the blocking client that returns a new Request, would you want to do a similar thing for the async client?

IIUC exposes a get_request because it does not have a minreq client as a field on BlockingClient, as minreq does not have this "strategy" as reqwest has :think
I think it could be helpful to get exposed in async/reqwest too, I'll check it.

A recent PR #87 exposed a get_request method for the blocking client that returns a new Request, would you want to do a similar thing for the async client?

IIUC exposes a get_request because it does not have a minreq client as a field on BlockingClient, as minreq does not have this "strategy" as reqwest has :think
I think it could be helpful to get exposed in async/reqwest too, I'll check it.

Ah ok so maybe not necessary since you can already get a handle to the reqwest::Client?

A recent PR #87 exposed a get_request method for the blocking client that returns a new Request, would you want to do a similar thing for the async client?

IIUC exposes a get_request because it does not have a minreq client as a field on BlockingClient, as minreq does not have this "strategy" as reqwest has :think
I think it could be helpful to get exposed in async/reqwest too, I'll check it.

Ah ok so maybe not necessary since you can already get a handle to the reqwest::Client?

Yes, that's what I thought.

ACK 9cb662a
Overall this looks great.
In 9cb662a

refactor(async)!: add common GET and POST methods

Maybe I missed it but I didn't see how this change is API breaking in any way?

Sweet! I updated the commit message, and add a new one applying the same style on other methods.

@oleonardolima since it seems we are planning to go down the #97 route, is this PR still relevant?

@oleonardolima since it seems we are planning to go down the #97 route, is this PR still relevant?

I guess this one can still land if we'd like to cut a release with these improvements, and mainly with #98. Then, cut a new release specifically with the refactor of #97. 🤔


Detect max named map templates import error

Description
In its configuration, Windshaft states a maximum number of maxUserTemplates. This is typically set to 4096.
When a dataset is imported, CartoDB creates a named map for it. If the named maps limit has been reached, the named maps wrapper was raising an HTTP exception that was understood as an unknown import error by the platform.
Proposed solution
In order to detect this limit, two new exceptions have been defined.
The first one, TooManyTemplatesError is created in the wrapper side and it's raised when it detects the limit error in the response of the named map creation. In the near future, this part should be re-done and we should  take care of comparing the status code 409: Conflict which will be added in the Named Maps API. (More info about the open issue in CartoDB-Windshaft here).
In the import side, I have defined a new exception TooManyNamedMapTemplatesError here, that will be responding to the import error code 6670 (the next number, following the already existent list of import codes related to limits).
A rescue will recover the CartoDB::NamedMapsWrapper::TooManyTemplatesError from Windshaft, and the import process will then handle the error by using the  CartoDB::Importer2::TooManyNamedMapTemplatesError exception, as can be seen here.
I tested the solution by forcing the exception in the wrapper at first. Then in staging I reached the maximum number of templates by creating them programatically with the named maps API, and the result was as follows:

irb(main):001:0> DataImport['85fe4f0a-51c8-4064-9349-f6279f836547'].log
=> #<CartoDB::Log @values={:id=>"5ce2002e-9fb7-438f-a29b-2b8cc7ef0f9f", :type=>"import", :user_id=>"367c0edc-b2ad-4bab-ad43-3d58a6179a93", :created_at=>2015-12-23 15:25:03 +0000, :updated_at=>2015-12-23 15:25:06 +0000, :entries=>"2015-12-23 15:25:04 UTC: Running on server cdb with PID: 2570\n2015-12-23 15:25:04 UTC: new_importer()
2015-12-23 15:25:04 UTC: Fetching datasource public_url metadata for item id /home/ubuntu/www/production.cartodb.com/current/public/uploads/37f53b29756028fdb325/cositis.csv
2015-12-23 15:25:04 UTC: File will be downloaded from /home/ubuntu/www/production.cartodb.com/current/public/uploads/37f53b29756028fdb325/cositis.csv
2015-12-23 15:25:04 UTC: Before importer run\n2015-12-23 15:25:04 UTC: Starting import for /home/ubuntu/www/production.cartodb.com/current/public/uploads/37f53b29756028fdb325/cositis.csv
2015-12-23 15:25:04 UTC: Unpacking /home/ubuntu/www/production.cartodb.com/current/public/uploads/37f53b29756028fdb325/cositis.csv
2015-12-23 15:25:04 UTC: Filename: /tmp/imports/20151223-2570-19ta0p4/cositis.csv Size (bytes): 25
true: Importing data from /tmp/imports/20151223-2570-19ta0p4/cositis.csv
true: File-based import load\ntrue: Detected encoding ISO-8859-1
true: Using database connection with {:adapter=>\"postgres\", :encoding=>\"unicode\", :host=>\"localhost\", :port=>5432, :database=>\"cartodb_dev_user_367c0edc-b2ad-4bab-ad43-3d58a6179a93_db\", :username=>\"postgres\", :conn_validator_timeout=>900, :user=>\"development_cartodb_user_367c0edc-b2ad-4bab-ad43-3d58a6179a93\"}\nfalse: ogr2ogr call:            OSM_USE_CUSTOM_INDEXING=NO PG_USE_COPY=YES PGCLIENTENCODING=ISO-8859-1  /usr/bin/ogr2ogr -f PostgreSQL   PG:\"host=localhost port=5432 user=development_cartodb_user_367c0edc-b2ad-4bab-ad43-3d58a6179a93 dbname=cartodb_dev_user_367c0edc-b2ad-4bab-ad43-3d58a6179a93_db password=-b2ad-4bab-ad43-3d58a6179a93\"  -lco DIM=2 -lco PRECISION=NO /tmp/imports/20151223-2570-19ta0p4/cositis.csv  -nln cdb_importer.importer_57588876a98911e5b085080027880ca6 -nlt PROMOTE_TO_MULTI
true: ogr2ogr output:
true: ogr2ogr exit code:       0
true: Georeferencing...
true: Disabling autovacuum for \"cdb_importer\".\"importer_57588876a98911e5b085080027880ca6\"
true: Trying ip guessing...
true: Trying country guessing...
true: country_proportion(carla) = 0.0
true: country_proportion(pepe) = 0.0
true: Creating the_geom column
true: Enabling autovacuum for \"cdb_importer\".\"importer_57588876a98911e5b085080027880ca6\"
true: Georeferenced\ntrue: Finished importing data from /tmp/imports/20151223-2570-19ta0p4/cositis.csv
2015-12-23 15:25:06 UTC: Proceeding to register
2015-12-23 15:25:06 UTC: Before renaming from importer_57588876a98911e5b085080027880ca6 to cositis
2015-12-23 15:25:06 UTC: Before moving schema 'cositis_1' from cdb_importer to public
2015-12-23 15:25:06 UTC: Before persisting metadata 'cositis_1' data_import_id: 85fe4f0a-51c8-4064-9349-f6279f836547
2015-12-23 15:25:06 UTC: Exception: User reached the limit on number of templates
2015-12-23 15:25:06 UTC: ERROR!

"}>

@rafatower CR please. Not very happy with the code in data_import.rb, so I'm open to all feedback. :-)

Looks good to me :+1:
Bear in mind that improving the code in data_import is not really possible without a complete overhaul of exception handling. I'd leave it as it is for now.

I will deploy this tomorrow morning first time.

I did a quick change here in order to check for the error code 409, which is the one that the API is expected to give from now on on these cases. After tests pass this should be fine to deploy I guess.

Docs update is being taken care of in: https://github.com/CartoDB/docs/pull/674


Add SwiftPM Support

Added Package.swift to allow use from Xcode 11's SwiftPM integration.
.travis.yml is also fixed to check if tests pass with both .xcodeproj and Package.swift.

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

Hi, @akkyie !
Thank you for supporting SwiftPM👍
I'll merge this.


no data saved by db_logger

Hi
To play with logging into SQL database I've set up a preset with a single Mock 0D detector. Database is configured, connect works. A database with the name of the preset is created and contains a couple of tables. After pressing the two green "play" buttons, the dashboard displays the incoming data and "N saved" on the log panel counts up (but faster than the data rate, ??). However, the table datas0D, where I would expect the logged data to show up, remains empty.
At the same time, continued messages
ERROR:pymodaq.db_logger:The index should be a positive integer lower than the data length
fill the console window from where the dashboard was started. I guess that sending data to the database is skipped because of that error. There are two locations where this message is issued: utils/data.py:1727 and utils/data.py:1733 which are to be found in the getitem and setitem methods of DataToExport
Any ideas what is going wrong?

yes I do,
The data logging to database has not been refreshed since I moved to the new database layout (as I presented in the pymodaq days). I thought mostly no one was using this feature so it was not my priority. I'll mark your issue

Thanks. I thought this feature could be used for a logging system of the "environmental conditions" in the laser lab, temperature, humidity and beam pointing / stabilizers. Typically one wants to browse though some time intervals during which something "interesting" happened. A data base looks to me being more adequate for such task than an ever-growing large file. However, its not on my priority list neither. And it could also be implemented with saving to file, at least for the time being.

Hi Bernard, branch dblogger_pymo4 and PR #219  should be allowing to use again the DAQ_Logger extension using postgresql database. I've checked it on my computer using a local database. Could you also test it and let me know if you still have issues!

Looks like my reply didn't make it until here. I've tested it and get the expected result. Thanks!


Make the discord.py datetimes offset-naive

You know the drill, due to discord.py 2.0a0 datetimes are now offset-aware, breaking some code.
Closes python-discord/sir-lancebot#875

I think we'd have the same problem here, correct me if i'm wrong
https://github.com/python-discord/sir-lancebot/blob/8a410f3abd39a1b48c514d32651a50d4bdced492/bot/exts/events/hacktoberfest/hacktober-issue-finder.py#L55-L58

Indeed, it is a bit of a kaizen change but who cares. Changed in 24e099e.


Channel priority should be set to strict and conda info should be printed with conda-store

Right now some solves are pulling in multiple channels that were not specified.

I think that channel priority strict is already set. It would be nice to have the output of:

conda info
conda config --show

I'd also like to confirm that we are setting channel priority to strict and ensure that its not pulling in things from defaults or anywhere else.

Note: the block of code that Chris refers to above is actually this (no hash was specified in the link and it's been refactored since then):
git show 5f7adfb5737edef307f0a70a42f7d04d66adce30:conda-store-server/conda_store_server/build.py:
145 def build_environment(conda_command, environment_filename, conda_prefix):
146
147     return subprocess.check_output(
148         [
149             conda_command,
150             "env",
151             "create",
152             "-p",
153             conda_prefix,
154             "-f",
155             str(environment_filename),
156         ],
157         stderr=subprocess.STDOUT,
158         encoding="utf-8",
159     )

Status update: this should be fixed by #693.


Utils: package changes insight

duplicate of https://github.com/relative-ci/bundle-stats/issues/2832


Automated cherry pick of #2413: host migrate

Cherry pick of #2413 on release/2.11.
#2413: host migrate

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@1508805). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #2557   +/-   ##
==============================================
  Coverage                ?   7.04%
==============================================
  Files                   ?     455
  Lines                   ?   72042
  Branches                ?       0
==============================================
  Hits                    ?    5076
  Misses                  ?   66548
  Partials                ?     418

Flag
Coverage Δ

#aFlag
7.04% <0%> (?)

Impacted Files
Coverage Δ

pkg/cloudcommon/db/opslog.go
2.66% <ø> (ø)

pkg/compute/models/guest_actions.go
0% <0%> (ø)

pkg/compute/models/hosts.go
0.38% <0%> (ø)

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 1508805...1ae5f24. Read the comment docs.

/lgtm

/approve


Getting pyspark working

Hi there.
I couldn't find any documentation for initializing pyspark.
I've initialized the spark sessions with
%%classpath add mvn
org.apache.spark spark-sql_2.11 2.3.1

followed by
%%spark

Everything is all good.
Then when I go to link the spark session in python I don't have an entry point to the spark because spark has been initialized through the jar files and not through python.
Is there a tutorial or documentation which outlines how to use spark in python or through pyspark?
Cheers.

Hi Thomas thank you for your question.
We do not yet support pyspark, just scala spark.
Expanding into this area is on our agenda: https://github.com/twosigma/beakerx/issues/7544

Thanks for the reply.

Please leave this one open.

Is there any movement on this? We'd really love to see BeakerX's capabilities expanded to include PySpark

I would also love to try BeakerX with PySpark :)

Is this improvement still being looked at? Native PySpark support would be really great!

I also would love to see BeakerX working with PySpark, a much needed feature.

This issue lacks for :+1: responses on the first post. Mine was added second.


php parser version 5 added into composer

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.08%. Comparing base (9b675df) to head (e89cbad).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##             master     #274   +/-   ##
=========================================
  Coverage     96.08%   96.08%
  Complexity      194      194
=========================================
  Files            36       36
  Lines           639      639
=========================================
  Hits            614      614
  Misses           25       25

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

The PHP versions 8.0.16 and 8.1 have failed. We should evaluate the following options:
Drop support for these versions.
Adjust the versions of our dependencies to maintain compatibility.

i think need support 8.1+ and drop supporting for 8.0.x


Align default Kubernetes version with AKS service

Is your feature request related to a problem? Please describe.
Default Kubernetes version in AKS is now 1.27.7, need to align AKS Construction to this
Describe the solution you'd like
Update the default version to 1.27.7
Describe alternatives you've considered
n/a
Additional context
n/a

Bumped main branch to AKS 1.28.5, but still waiting for Bicep and new Helper application to be build + deployed.

Version bumped to 1.29.7 in #705


ci: Run podman e2e test with Rocky 9.1

What problem does this PR solve?:
Previously, we used CentOS 7.9, but we cannot use it, because we dropped support for it recently (#1072), as it went EOL.
Which issue(s) does this PR fix?:

https://jira.nutanix.com/browse/NCN-101343

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Thanks for your reviews. I'm going to restructure the changes slightly, to make it clear that we're making a change to the product, as well as CI. I won't change the code in any way. I'll force push.

For reference, the podman e2e test passes: https://github.com/mesosphere/konvoy-image-builder/actions/runs/9877813263/job/27280651164?pr=1116


Delete Cotx product type X3 in Readme

Please try creating a new branch/PR? This picked up an old set of commits somehow


PR for code review - Capstone project

This is Analytics Reporting, an app that takes live data from an Airtable database which collects user traffic for sites that I manage.
The app uses the Airtable API to fetch data and Redux to keep track of the state of the app.
Features:

Vite + Tailwind
Jest test for components
API calls

Video Walkthrough
Live view
Most interesting part of the project

Hi @wickathou,
Great work on making the changes requested by a previous reviewer clap You've done well implementing some of the requested changes, but there are still some which haven't been addressed yet.
Highlights
white_check_mark Linters are passing
Required Changes
Check the comments under the review. I had to add comment a here because it could not be done inline
[REQUIRED] As requested by the previous reviewer kindly point out a piece of code you find interesting. This is a requirement for this project.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you consider them as they can improve your code. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!clapclapclap
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think the code review was unfair, you can request a second opinion using this form.

Done https://www.loom.com/share/e78dac7adb5446919af71b0741e67c6f
Thank you


The Application that is running AXElements is not trused to control your computer

After playing around with this a bit (due to AXElements/accessibility_core#19) I came up with the following solution. Before installing AXElements I do the following`
def install_accessibility_core_gem
  directory '/usr/local/src/accessibility_core' do
    recursive true
  end.run_action(:create)

  git '/usr/local/src/accessibility_core' do
    repository 'https://github.com/AXElements/accessibility_core.git'
  end.run_action(:sync)

  execute 'gem build accessibility_core.gemspec' do
    cwd '/usr/local/src/accessibility_core'
  end.run_action(:run)

  gem_file = '/usr/local/src/accessibility_core/accessibility_core-0.6.1.gem'

  gem_package 'accessibility_core' do
    options '--clear-sources'
    source gem_file
  end.run_action(:install)

  chef_gem 'accessibility_core' do
    options '--clear-sources'
    compile_time(false) if respond_to?(:compile_time)
    source gem_file
    action :nothing
  end.run_action(:install)
end

That gets me to at least install the AXElements gem but then trying to start the mac app store I run into permission problems (below error).
================================================================================
Error executing action `open` on resource 'mac_app_store[default]'
================================================================================

RuntimeError
------------

-------------------------------------------------------------------
The Application that is running AXElements is not trused to control
your computer. A window prompting you to grant permission to the
application should appear right now. Please grant the application
permission to control your computer and try again.
-------------------------------------------------------------------

Cookbook Trace:
---------------
/Users/mhenrixon/installs/sprout-wrap/cookbooks/mac-app-store/libraries/provider_mac_app_store.rb:68:in `block in <class:MacAppStore>'

Resource Declaration:
---------------------
# In /Users/mhenrixon/installs/sprout-wrap/cookbooks/mac-app-store/recipes/default.rb

 35: mac_app_store 'default' do
 36:   username node['mac_app_store']['username']
 37:   password node['mac_app_store']['password']
 38:   action :open
 39:   notifies :quit, 'mac_app_store[default]'
 40: end
 41:

Compiled Resource:
------------------
# Declared in /Users/mhenrixon/installs/sprout-wrap/cookbooks/mac-app-store/recipes/default.rb:35:in `from_file'

mac_app_store("default") do
  action [:open]
  updated true
  updated_by_last_action true
  retries 0
  retry_delay 2
  cookbook_name :"mac-app-store"
  recipe_name "default"
  username<EMAIL_ADDRESS>  password "****************"
end

Prior to OS 10.11, the OS would pop up a dialog asking about granting Accessibility permissions to the application trying to use the accessibility core.  Probably "Ruby" in your case.
The dialog box hasn't been showing up in 10.11 Beta.  Still a problem in 10.11 beta 6.  I had to add the application to directly the access DB.   By issuing:
sudo sqlite3 /Library/Application\ Support/com.apple.TCC/TCC.db "insert into access(service,client,client_type,allowed,prompt_count) values('kTCCServiceAccessibility','/usr/bin/ruby',1,1,1)"

For me, navigating to system preferences, security and privacy, going to accessibility section, and adding iTerm as an 'approved' application for 'controlling your computer' solved the issue.

Looks like this is, sadly, still the case in the final release of El Capitan. Apple replaced the explanatory pop-up message with just assuming users would know to go into their Accessibility settings.
The cookbook attempts to "just work" and grant itself accessibility rights. This has been tested against Chef running inside iTerm, Terminal, and over SSH, but results may vary from one app to another. I remember it being tricky to figure out that much, and the worst case scenario remains, as mentioned, watching the first Chef run fail and having to go into the Accessibility settings manually.
@jacobjennings How were you running Chef where you were seeing errors even with using iTerm? I'm curious to see if I can reproduce that. I tried deleting iTerm from my Accessibility settings just now, and doing a Chef run correctly re-added it (this is on 10.11.2 beta).

Was provisioning a brand new laptop on El Capitan and suddenly able to reproduce this in both Terminal and iTerm. I think it was a behavior change introduced in newer versions of the privacy_services_manager script the cookbook uses.
The auto-detection is fixed by #27 so both Terminal and iTerm will again get their access privileges granted as part of the Chef run. I'll cut a new release soon.
OS X still won't pop up a helper dialog anymore when the app running Chef isn't trusted, but there's not much that can be done about that.


bump http client's MaxIdleConns and MaxIdleConnsPerHost

Overview
The idea here is for Pilosa to behave better under high query load where a node
might be making many connections to the other nodes in the cluster in order to
support lots of concurrent batches of SetBit queries (for example). By allowing
for more idle connections and more idle connections per host, we reduce
connection churn, and allow more connections to be reused rather than creating
new ones and potentially having many stale sockets in the TIME_WAIT state. See
https://github.com/golang/go/issues/16012
Fixes (hopefully) #906
Pull request checklist

[ ] I have read the contributing guide.
[ ] I have agreed to the Contributor License Agreement.
[ ] I have updated the documentation.
[ ] I have resolved any merge conflicts.
[ ] I have included tests that cover my changes.
[ ] All new and existing tests pass.

Code review checklist
This is the checklist that the reviewer will follow while reviewing your pull request. You do not need to do anything with this checklist, but be aware of what the reviewer will be looking for.

[ ] Ensure that any changes to external docs have been included in this pull request.
[ ] If the changes require that minor/major versions need to be updated, tag the PR appropriately.
[ ] Ensure the new code is properly commented and follows Idiomatic Go.
[ ] Check that tests have been written and that they cover the new functionality.
[ ] Run tests and ensure they pass.
[ ] Build and run the code, performing any applicable integration testing.

Per discussion here https://github.com/pilosa/pilosa/issues/906#issuecomment-340908337 @codysoyland can we cut a new 0.7 build for @seanbrant if this PR is approved?

anybody else want to take a peek at this before merge?

not sure why this is failing in Go 1.8 - it is also failing locally in docker.
It is also failing for v0.7.1 with no changes, so I think we can ignore for the purposes of this  PR - we'll just have to make sure we don't build the release with Go1.8

@codysoyland - do you just do make release to build the release binaries? I'd like to get them to @seanbrant

@jaffee yes, that will produce a release build in the build directory. Also, I can cut a proper tagged release with changelog today if you need.


boot dev reports command not found

Hi I'm very new to the edge, I got the error: "command not found" when I tried to clone the project and run the 'boot dev' command. Is there sth missed there?

ok, I found the problem, which I should install the boot-clj first:
brew install boot-clj


Reduce memory usage by 3x

With no measurable impact in time benchmaks.
Fixes #158

Pull Request Test Coverage Report for Build 406

145 of 151 (96.03%) changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+7.05%) to 89.571%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/index.js
40
42
95.24%

src/helpers.js
105
109
96.33%

Files with Coverage Reduction
New Missed Lines
%

src/index.js
1
89.29%

Totals

Change from base Build 399:
7.05%

Covered Lines:
204

Relevant Lines:
217

💛  - Coveralls


Issues with fvtt container

Good evening, I was following your tutorial on your website. and got everything working and installed. I have access to traefik, and portainer up and running. I was also get the container configured based on your tutorial. Based on the containers logs the foundry server is up and running without issues, but when i go to my play.domain website, all i get is a 404 page not found error.
Container Log:
inflating: resources/app/templates/journal/sheet.html,
inflating: resources/app/templates/playlist/create-playlist.html,
inflating: resources/app/templates/playlist/edit-playlist.html,
inflating: resources/app/templates/playlist/edit-track.html,
inflating: resources/app/templates/scene/config.html,
inflating: resources/app/templates/scene/drawing-config.html,
inflating: resources/app/templates/scene/grid-config.html,
inflating: resources/app/templates/scene/light-config.html,
inflating: resources/app/templates/scene/note-config.html,
inflating: resources/app/templates/scene/sound-config.html,
inflating: resources/app/templates/scene/template-config.html,
inflating: resources/app/templates/scene/tile-config.html,
inflating: resources/app/templates/scene/token-config.html,
inflating: resources/app/templates/scene/wall-config.html,
inflating: resources/app/templates/setup/eula.html,
inflating: resources/app/templates/setup/install-dependencies.html,
inflating: resources/app/templates/setup/install-package.html,
inflating: resources/app/templates/setup/player-create.html,
inflating: resources/app/templates/setup/setup-config.html,
inflating: resources/app/templates/setup/update-notes.html,
inflating: resources/app/templates/setup/updater.html,
inflating: resources/app/templates/setup/user-management.html,
inflating: resources/app/templates/setup/world-config.html,
inflating: resources/app/templates/sheets/active-effect-config.html,
inflating: resources/app/templates/sheets/actor-sheet.html,
inflating: resources/app/templates/sheets/combat-config.html,
inflating: resources/app/templates/sheets/combatant-config.html,
inflating: resources/app/templates/sheets/item-sheet.html,
inflating: resources/app/templates/sheets/macro-config.html,
inflating: resources/app/templates/sheets/roll-table-config.html,
inflating: resources/app/templates/sheets/sheet-config.html,
inflating: resources/app/templates/sidebar/actor-directory.html,
inflating: resources/app/templates/sidebar/apps/av-config.html,
inflating: resources/app/templates/sidebar/apps/controls-reference.html,
inflating: resources/app/templates/sidebar/apps/folder-export.html,
inflating: resources/app/templates/sidebar/apps/invitation-links.html,
inflating: resources/app/templates/sidebar/apps/module-management.html,
inflating: resources/app/templates/sidebar/apps/permission-config.html,
inflating: resources/app/templates/sidebar/apps/settings-config.html,
inflating: resources/app/templates/sidebar/chat-log.html,
inflating: resources/app/templates/sidebar/chat-message.html,
inflating: resources/app/templates/sidebar/combat-tracker.html,
inflating: resources/app/templates/sidebar/compendium-create.html,
inflating: resources/app/templates/sidebar/compendium.html,
inflating: resources/app/templates/sidebar/entity-create.html,
inflating: resources/app/templates/sidebar/folder-edit.html,
inflating: resources/app/templates/sidebar/item-directory.html,
inflating: resources/app/templates/sidebar/journal-directory.html,
inflating: resources/app/templates/sidebar/macro-directory.html,
inflating: resources/app/templates/sidebar/playlist-directory.html,
inflating: resources/app/templates/sidebar/roll-table-directory.html,
inflating: resources/app/templates/sidebar/scene-directory.html,
inflating: resources/app/templates/sidebar/settings.html,
inflating: resources/app/templates/sidebar/sidebar.html,
inflating: resources/app/templates/user/player-config.html,
inflating: resources/app/templates/user/players.html,
inflating: resources/app/templates/views/error.hbs,
inflating: resources/app/templates/views/game.hbs,
inflating: resources/app/templates/views/join.hbs,
inflating: resources/app/templates/views/layouts/game.hbs,
inflating: resources/app/templates/views/layouts/main.hbs,
inflating: resources/app/templates/views/license.hbs,
inflating: resources/app/templates/views/players.hbs,
inflating: resources/app/templates/views/setup.hbs,
inflating: resources/app/templates/views/stream.hbs,
inflating: snapshot_blob.bin,
inflating: swiftshader/libEGL.so,
inflating: swiftshader/libGLESv2.so,
inflating: v8_context_snapshot.bin,
inflating: vk_swiftshader_icd.json,
FoundryVTT | 2021-01-09 00:46:04 | [info] Foundry Virtual Tabletop - Version 0.7.9,
FoundryVTT | 2021-01-09 00:46:04 | [info] Running on Node.js - Version 12.18.2,
FoundryVTT | 2021-01-09 00:46:04 | [info] Loading data from user directory - /data/foundryvtt,
FoundryVTT | 2021-01-09 00:46:04 | [info] Application Options:,
{,
"port": 30000,,
"upnp": true,,
"fullscreen": false,,
"hostname": null,,
"routePrefix": null,,
"sslCert": null,,
"sslKey": null,,
"awsConfig": null,,
"dataPath": "/data/foundryvtt",,
"proxySSL": true,,
"proxyPort": null,,
"minifyStaticFiles": false,,
"updateChannel": "release",,
"language": "en.core",,
"world": null,,
"serviceConfig": null,,
"isElectron": false,,
"isNode": true,,
"isSSL": false,,
"demo": false,,
"noupdate": false,
},
FoundryVTT | 2021-01-09 00:46:04 | [warn] Software license requires signature.,
FoundryVTT | 2021-01-09 00:46:04 | [info] Requesting UPnP port forwarding to destination 30000,
FoundryVTT | 2021-01-09 00:46:04 | [info] Server started and listening on port 30000,
Any assitance would be greatly appreciated.

disregard.. issue with formatting it is resolved. Thank you for the fantastic container and fantastic tutorial.

disregard.. issue with formatting it is resolved. Thank you for the fantastic container and fantastic tutorial.


HVertOffsets

Offsets for HVerts. These are now going to be restricted to the implementation of symmetrical side terrain. I'm going to make this highest priority because it will make such a difference to the visual appearance, especially for lower scale grids.

I think this complete for the time being."


Create a visual designer from a yaml file

Using the visual designer is way easier than configuring a yaml file. I'd love to (at least) be able to import a yaml pipeline and get the visual designer loaded. That way, I can edit my pipeline in the visual designer and then export it back out to a new yaml file.
Even better, we can have the visual designer work with our yaml pipelines under source control. Switching branches reflects the changes in the visual designer.
I tried advocating switching our pipelines to source controlled yaml, but people really love the convenience of the visual editor. Even though we can easily break branches or lose something we need to revert to!
Feature request. Please let us import yaml pipelines into the visual editor.
Another request. Allow the visual editor to work with source controlled pipelines.
Thank you!

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

ID: 02b747eb-53c2-dd5f-cd2b-f337c15e0864
Version Independent ID: 04538505-00cd-33ac-8008-4b85cca089e3
Content: Use the visual designer to create a CI/CD pipeline - Azure Pipelines & TFS
Content Source: docs/pipelines/get-started-designer.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

Please request features here: https://developercommunity.visualstudio.com/spaces/21/index.html


Add logback.xml for JSON logging on stdout

Add logback.xml for JSON logging on stdout. We should also update the configuration.md on how to enable this file.

This will be implemented in #122

Closed by #122


AWS AL2023 support

This PR aims to introduce transparent support for installing Cilium on clusters using the AL2023 AWS node image family.
Please see per commit for full details :)

/ci-eks

/ci-eks

Moving from draft to ready for review to get initial feedback -- but this is not yet quite ready to merge, still running some CI tests as per the temporary commit.

/ci-eks

/ci-eks

/ci-eks

/ci-eks

/ci-eks

I've pushed two commits:

Use K8S ListNodes API to extract OS image instead of eksctl tool: https://github.com/cilium/cilium/pull/35529/commits/47c8f339aba4476d52a28397180396dc83bc276a
Egress interfaces selector improvement: https://github.com/cilium/cilium/pull/35529/commits/0df6ea09dd5073fdb600ffe9aac0c0d73cab7ba3 (suggested by @gandro)

I can't run EKS conformance tests for the PR due to the changes in the main branch.
@nbusseneau any ideas how to make CI green before merge?
I've done testing on the separate branch:

EKS 1.29: https://github.com/cilium/cilium/actions/runs/11914812464/job/33203638097
EKS 1.30: https://github.com/cilium/cilium/actions/runs/11914812464/job/33203638539

Also, I've checked locally that it's possible to override egressMasqueradeInterfaces value during Cilium installation using input param: --helm-set egressMasqueradeInterfaces=eth0.
@gandro @auriaave @brlbil please review.

I'm closing this PR in favor of https://github.com/cilium/cilium/pull/36076 because I don't know how to fix EKS workflows here.


Sprite Improvements and Adjustments

I've run into various issues with trying to make custom sprites, particularly with custom groups.
Here are my proposed changes:

Merge AdvancedSprite to Sprite and deprecate it. Allow program in the initial Sprite class.
Add get_sprite_group to Sprite to allow users to change initialization of a custom SpriteGroup without having to overwrite multiple functions.
Add blend_mode property to allow sprites to change the default blend modes.
Use _user_group to specify the intended group rather than relying on group.parent which may cause confusion with deeper nested groups.
Update docs and typing.

Open to any feedback, particularly around the get_sprite_group function naming.

I was also thinking about exposing program setting on the default Sprite, so no complaints there.
Changing the blend modes looks useful. It's better than having to re-create a Sprite.
get_sprite_group looks fine as it is, but I guess I don't see a need for it to be a public method. What did you have in mind?
One thing we might do is add an assert to the @program.setter that checks if the user ShaderProgram has all the same ShaderProgram.attributes.keys() as the default one.  If they don't have the vertex attributes, we could warn that some default Sprite functionality won't work/crash. We can leave this idea for later though.

I was also thinking about exposing program setting on the default Sprite, so no complaints there. Changing the blend modes looks useful. It's better than having to re-create a Sprite.
get_sprite_group looks fine as it is, but I guess I don't see a need for it to be a public method. What did you have in mind?

I was mostly just making it public, so that the docs can print it out for users, but maybe it's fine to make it private?

One thing we might do is add an assert to the @program.setter that checks if the user ShaderProgram has all the same ShaderProgram.attributes.keys() as the default one. If they don't have the vertex attributes, we could warn that some default Sprite functionality won't work/crash. We can leave this idea for later though.

That sounds like a good idea.


[AutoPR reservations/resource-manager] Add reservations/resource-manager/readme.typescript.md

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4226

(message created by the CI based on PR content)


Updated DRF and Python version combinations

This should be supported out of the box, but let's find out...

[x] Get Django 2.1 passing
[x] Backfill Python 3.5 support across DRF versions

By the end of this we are going to be at over a hundred different supported combinations. We may want to look into dropping really old combinations from the testing matrix, or cutting some of the middle versions since we don't have much (if any) special casing going on.

@kevin-brown I'd definitely drop anything pre Django 1.11 at this point and perhaps just test against latest stable DRF release. That would simplify the matrix considerably.

Once again, no code changes were required to support the newer versions (latest DRF, latest Django, Python up to 3.6). I dropped support for Django 1.6/1.7 and Python 3.3 because it made it easier to set up the Tox matrix. I also switched to tox-travis to make specifying that matrix in Travis much less painful.

@kevin-brown thank you! 🎉


Update to use new Meteor compiler plugin system.

The Meteor compiler system had a significant overhaul in Meteor 1.2.
While the current implementation still worked in Meteor 1.2, it was
likely written pre-Meteor 0.8.0 and is now no longer supported after
Meteor 1.3.
This should work with Meteor 1.4.4.x.
Closes #6.

@stubailo Any aversion to publishing it? 😃


chore: bdd-nh-1587051918 to 0.0.1

chore: Promote bdd-nh-1587051918 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


Bugfix. Make sure extension temp file is in the APK bucket

We were looking for it in the APK bucket, but we put it in the Source bucket. This change fixes that!
Change-Id: I89d68b5a5f6b369c9a57e433806d7fc563d6e9d0

Tested.

@ewpatton Thanks. The temp file literally lasts only seconds...


[Event Request] Codeunit 10755 SII Initial Doc. Upload - OnHandleExistingCustomerLedgerEntriesOnBeforeCreateNewRequest

We need a new event in Codeunit 10755 "SII Initial Doc. Upload" named OnHandleExistingCustomerLedgerEntriesOnBeforeCreateNewRequest:
[IntegrationEvent(false, false)]
local procedure OnHandleExistingCustomerLedgerEntriesOnBeforeCreateNewRequest(VAR CustLedgerEntry: Record "Cust. Ledger Entry");
begin
end;
called from:
local procedure HandleExistingCustomerLedgerEntries()
var
SIIDocUploadState: Record "SII Doc. Upload State";
CustLedgerEntry: Record "Cust. Ledger Entry";
begin
CustLedgerEntry.SetRange("Posting Date", StartDate, EndDate);
if CustLedgerEntry.FindSet then begin
repeat
//INIT INSERT
OnHandleExistingCustomerLedgerEntriesOnBeforeCreateNewRequest(CustLedgerEntry);
//END INSERT
if CustLedgerEntry."Document Type" in [CustLedgerEntry."Document Type"::"Credit Memo",
CustLedgerEntry."Document Type"::Invoice]
then
SIIDocUploadState.CreateNewRequest(
CustLedgerEntry."Entry No.", SIIDocUploadState."Document Source"::"Customer Ledger",
CustLedgerEntry."Document Type", CustLedgerEntry."Document No.", CustLedgerEntry."External Document No.",
CustLedgerEntry."Posting Date");
until CustLedgerEntry.Next = 0;
end;
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.