Lodash error using request promise

While making a get request, I get the following error
"Uncaught TypeError: _.assign is not a function", source: /Users/ZECTbynmo/Projects/ConnectorTester/node_modules/request-promise/lib/rp.js (19)
Any ideas?

I just ran a unit tests with the latest module versions and they are green.
Which node.js/io.js version and which lodash version do you use?

I'm running inside of an electron app. It's running io.js v2.2.1

The Travis CI build uses io.js 2.3.0. Maybe the issue is related to the specific environment electron is creating. (However, I never used electron before.)
Are you able to debug the code and look into the _ object? Somehow the lodash object is not loaded properly.
FYI, all we are talking about is var _ = require('lodash'); and then the use of assign in var temp = _.assign({}, require.cache);.

@ZECTBynmo Any news regarding this issue? Can I help?

Please check if you installed lodash correctly. That is all I can tell right now. Please reopen this issue if you need further assistance.


Update README.md

Hi
This PR closes #145 to add Open Graph Image to README.md file
Thanks

Hi @TawfikYasser,
Nice work, questions about it.
Does it update automatically?

Hi @TawfikYasser,
Nice work, questions about it.
Does it update automatically?

Yes, It's a GitHub API so it works automatically

Hi @TawfikYasser,
Could you help me to check this?


Simplify CSP when all variables have the same domain

Hi,
You can simplify code for your samples, by adding an additional constructor which assigns the same domain to all variables.
public CSP(List<V> variables, List<D> domain) {
     assert variables != null;
     assert domain != null;

     this.variables = variables;
     this.domains = new HashMap<>();

     for (V variable : variables) {
         domains.put(variable, domain);
         constraints.put(variable, new ArrayList<>());
     }
 }

The creation of the CSP for MapColouringConstraint will reduce to
final CSP<String, String> csp = new CSP<>(variables, List.of("red", "green", "blue"));

HI @z669016,
It's a good idea. Because the book is nearing publication and I don't want to make changes for non-errors or stylistic mistakes/lack of usage of Modern Java idioms at this point, I won't do it here. But I will mark this as something to remember for the next edition or another language port.
Thank you,
David


Do not modify an application bundles autostart setting when stopping a bundle when the spring application context fails

When using NEBA with Spring, NEBA registers a ContextShutdownHandler that will stop a bundle who's Spring Application Handler failed to start. This is rather useful as otherwise the bundle remains Active and it becomes hard to spot that a bundle exhibited a failure of part of its infrastructure.
The current implementation stops the bundle by invoking Bundle#stop(). This however persists the bundle's autostart setting. That is, the framework will not automatically try to re-start the bundle when the framework restarts.
This is an issue in case there is nothing to resolve, e.g. when the Application Context Failure was temporary. For instance, it may be that a framework restart is issued while an application context is in the process of starting, thus leading to a context start failure. Subsequently, the app bundle has to be re-started manually.
Here, the bundle state must not be persisted, i.e. Bundle#stop must be invoked with a suitable argument.

Done


Add option or remove log props for errors with server rendering

Since props can be huge, we don't want to log the props when server rendering.
Either:

Never log props with server rendering errors
Make it another option, possibly piggy-backing on some other option.

CC: @robwise

If we make it another option, I don't think we should piggy-back it, but make it something really specific and obvious like: log_props_with_server_rendering_errors

We have a a ton of data sent via props during server side rendering. Having them logged as part of the error message quickly becomes noisy and not useful.
I do believe it can be beneficial as people are starting up, especially when they have fairly small props.
log_props_with_server_rendering_errors sounds ideal.

@benigeri
We currently have:
Default is true. Logs server rendering messages to Rails.logger.info
config.logging_on_server = true
I think we could:
Change the default of true to not log props
Alternately, somebody, if somebody wanted it, we could add something like "verbose" instead of true.
Are the props ever interesting on the server?

Hey Justin, what about this issue ? Should we replace
 config.logging_on_server = true by
 config.logging_on_server = false
?

@EEtancelin I recommend keeping this as true. If you do change to false, you'll only get logs from React on Rails in the browser.

I'll take a look at this

If we make it another option, I don't think we should piggy-back it, but make it something really specific and obvious like: log_props_with_server_rendering_errors

i'll try to implement this

Done in #1059.


custom annotation on queryParam is interpreted as "body"

@QueryParam before the method argument works properly. However, we defined custom annotation for parsing string query parameter into a Set, where this argument is considered as Parameter Type "body" by swagger.
The parameter name can be overridden by the name @ApiParam. Is there a way to override the parameter type?

@ApiParam won't allow you to override the type but @ApiImplicitParam will (you'll be redefining the parameter from scratch). If you go with that path, you'd have to hide the original parameter manually (@ApiParam(hidden=true)).

Thanks for the quick response! will try out @ApiImplicitParam.

worked elegantly. thanks!


Duplicate account names

Bug description
CLI wallet allows multiple account creation under a single alias.
Rust version
Which version of Rust are you running?

Rust version: 1.48.0

Version

Version number, commit, or branch: 4ef9161ec7c8b0ef37e14a3bc4fb4a443dd8e8e0

Steps To reproduce the bug

Run ./wallet new --node http://localhost:14265 --alias Umair
Exit the wallet
Run ./wallet new --node http://localhost:14265 --alias Umair (It will create a new account under alias Umair)

Expected behaviour
The wallet shouldn't allow duplicate account creation under a single alias. It is confusing.

Actual behaviour
Wallet allows duplicate account creation under a single alias.
Errors
N/A

Will be available on next release.

Will be available on next release.


README link to LESbrary data is broken

Clicking on https://engaging-web.mit.edu/~alir/lesbrary/ gives

Ah yeah it was an engaging issue. It's been fixed now.


Event date not consistent

The dates are not consistent.

It's a timezone issue, we already fixed a couple of them lately.
We'll have a look shortly as it's quite critical :)

Hey Nicco!
Just to confirm, what is the correct time?
Also, is the event on the 22nd Oct supposed to be at 14:00, or should it be the same as all the others, at 15:00?

The correct time for all events would be 14:00 - 17:00.

Here is something that did not arrive via Mail:
My mental model:
A Dojo has a place. The dojo time is always local to that place.
I conclude from that:
It is unnecessary to transfer the time to a time zone model.
Converting back and forth only creates error.
It also introduces unforseen errors: When I copy an event over the summer time barrier, the time changes.
If the time is always converted from the local time to the time zone time back to the local time, it is more complex than just storing the local time.
Here we see the error of a conversion being forgotten.
What I suggest:

Do not convert dojo times, thus reduce complexity.
Create a method that computes local time to time zone time if you really need this functionality.
Saving the local time reflects the users mental model in your database.

This is just a suggestion, so you may have a different view on the topic.

We agree on "what you input is what you get".
The main problem is that we can't ignore Tzone because of DST in this case : when you create a recurring event, the TZ may change because of DST changes.
So that we prefer to handle the TZ overall than just an subset of it, even more when changing the actual behavior of event (what you see is what you get, whatever the TZ) will break all the actual events  :p
Daniel is pushing a fix today normally, we had a codebase which was splitted (different handling of dates) which was creating this issue. Nearly there :)

@DanielBrierton @niccokunzmann does the latest changes seems allrighty ?

@Wardormeur All dates are consistent for me, now. I vote for closing the issue.

Thanks @niccokunzmann !


Expose extension methods to wrap and unwrap adapters

As it currently stands, we only support using casting to convert between the two representations. This change exposes the extension methods we are using internally to expose the ASP.NET Core or System.Web objects.

@davidfowl Here are the extension methods. I'm not happy with the name, though, as it feels fairly generic (but maybe sufficient?).


Is the fieldset hint guidance correct?

Related documentation
https://design-system.service.gov.uk/components/fieldset/
Suggestion
It reads

Include any general help text which is important for filling in the form and cannot be written as hint text in the legend, but try to keep it as concise as possible.

This would make help text look like a heading, which I'm not sure is correct? If it is, it would be useful to see an example.

My quick thought on this: Instead of adding the hint-text type of text into the legend, should we be working to add it as a  after the legend, and before the first field? That might mean it gets a bit lost though for screen reader users who are tabbing between fields.

Great catch, thanks @joelanman. My hunch is there should have been a comma after 'hint text'. The sentence is overlong anyway so I'll try rewriting it.

on support, the context was asking for an address, with the first field in the fieldset being the address, then postcode and country. In that context a hint attached to the first field I think could work. I'm not sure this is generalisable to all fieldsets. It has the advantage of getting read out by screen readers


Add OpenID login functionality with backend

We removed backend functionality like this, as yasgui is completely client-side now


is it possible to dump the html that the terminal renders?

First of all, fantastic job! 🥳
Since this browser deals with javascript would be great to wait until the DOMContentLoaded event and get a snapshot how the HTML looks like. The use case would be that instead of using old curl you could query a page that uses some JS framework, let's say react, and then you get how the page actually looks like after all that javascript:
Something like carbonyl https://google.fr --dump index.html
Thanks

Output would be a text stream with ANSI codes, isn't It?

Output would be a text stream with ANSI codes, isn't It?

I guess as long as it can be human readable (or easily converted) should be fine :)

I guess as long as it can be human readable (or easily converted) should be fine :)

Dump as an image is dumb, there are already other tools for that.

Output would be HTML.  So for example if the input is <script>document.write("<"+"hr>")</script> the output would be <hr>

I think they are different things, you are asking about a dump of the DOM, that's already feasable. I'm talking about doing screenshots, but instead of imagen, text output.

That was what i was looking for, get the html page after the javascript run, is it possible now? How?

There's lote of other tools that can do It: Selenium, PhantomJS...

Yes you can do it using Selenium with Headless Chrome or Headless Firefox (or the no longer maintained PhantomJS), but that takes loads of RAM and sometimes hangs, so it would be nice to have a lighter weight alternative if possible.  Edbrowse can do it but has more limited DOM support so it doesn't work on all sites.

I think RAM usage of Carbonyl would be similar to Chrome.

chromium already has --dump-dom if that's useful


Subscriptions/Billing Management

"Subscriptions" should have their own management page. Similar to "Shopping Cart" page.
Submenu shows 3 pages:
Manage Plans / Payment History / Billing Information
Manage Plans
Grid list with columns:
Plan |  Price | Period | Next Billing |  [edit column]
Business | $800 | Monthly | 12.09.2916 | [@]
The edit button should show menu with options:

Switch to another plan
Change payment method
Change billing period
View payment history
Request cancellation

Payment History
Grid list with columns:
Dropdown with "All plans / Plan A , Plan B, etc"
Plan | Amount | Date
Business | $800 | 12.09.2916
Billing Information
Grid list with columns:
Status | Payment Method |   [edit-column]
(0  ) |  Visa (via Stripe) |  [@]
(  0) |  PayPal |  [@]

Need to realize Subscription Change for Stripe integration.


Generalize the log4j.appender.CARBON_LOGFILE.layout.ConversionPattern cofigurations

Description:
The default log4j.appender.CARBON_LOGFILE.layout.ConversionPattern in log4j configurations are not exactly same for all wso2 products. It would be good if we could have a single pattern in all products.
Suggested Labels:
Carbon logs, Improvement
Affected Product Version:
All wso2 product
Current pattern
apim
log4j.appender.CARBON_LOGFILE.layout.ConversionPattern=TID: [%T] [%S] [%d] %P%5p [%t] {{ '{%c}' }} - %x %m {{ '{%c}' }}%n
esb
log4j.appender.CARBON_LOGFILE.layout.ConversionPattern=TID: [%T] [%S] [%d] %P%5p [%t] {{ '{%c}' }} - %x %m {{ '{%c}' }}%n
is
log4j.appender.CARBON_LOGFILE.layout.ConversionPattern=TID: [%T] [%S] [%d] %P%5p {{ '{%c}' }} -%x %m %n
apim anaytics
log4j.appender.CARBON_LOGFILE.layout.ConversionPattern=TID: [%T] [%S] [%d] %P%5p {{ '{%c}' }} -%x %m {{ '{%c}' }}%n

All the products have their own log4j2.properties files. Hence, this has to be fixed in the log4j2.properties of the product.


Performance check utiliy

add to readme pls

add to readme pls

add to readme pls

Readme updated.

add to readme pls

Readme updated.


Try first a credential provider using laravels env function. Fixes #4

The CredentialProvider has most of the useful functions private, so extending it won't work. It does however provide the chain function to chain providers. Using this, we can first try to use the laravel env function instead of getenv.

Coverage decreased (-5.9%) to 94.118% when pulling e6edfcf81206b4e7bb1aea91ffc0860dd9b03bee on SaxJ:issue-4 into 53842bb4177d41b630487e4d75f6d40a749cb78a on HealthEngineAU:master.

Coverage decreased (-5.9%) to 94.118% when pulling e6edfcf81206b4e7bb1aea91ffc0860dd9b03bee on SaxJ:issue-4 into 53842bb4177d41b630487e4d75f6d40a749cb78a on HealthEngineAU:master.


Fix RessourceManager constructor

I'm currently working on a CI job with downstream builds for ros2_control, and this package here fails.
The API of the RessourceManager has changed. To be able to compile this repository on jazzy+rolling, one has to pass some more arguments to it. I think it is time to branch-off humble and fix that for the newer distros.

Well, that is also a possibility. but there is a simpler way: as we use ament_generate_version_header you simply can use
#include<hardware_interface/version.h>
#if HARDWARE_INTERFACE_VERSION_GTE(major, minor, patch)
#endif

@JafarAbdi shall I change it? And the CI job has now a wrong (humble) in the title.

Well, that is also a possibility. but there is a simpler way: as we use ament_generate_version_header you simply can use
#include<hardware_interface/version.h>
#if HARDWARE_INTERFACE_VERSION_GTE(major, minor, patch)
#endif

I thought about using it, but it was introduced in 4.8.0 :crying_cat_face:

@JafarAbdi shall I change it? And the CI job has now a wrong (humble) in the title.

It should be fixed now

I thought about using it, but it was introduced in 4.8.0 😿

Oh, we should backport that :see_no_evil:

We have it now in the humble branch as well, maybe you'd consider updating this after the next release of ros2_control.

We have it now in the humble branch as well, maybe you'd consider updating this after the next release of ros2_control.

Thanks! I'll open a PR to use it


Updated llama-cpp-python (and its dependencies with env variables)

Updated llama-cpp-python to 0.3.1 using Pypi.
Conda had 0.2.24 as the latest version. I was not able to run any ChatModel in langchain in this version (was getting an error that Llama.create_chat_completion() got an unexpected keyword argument 'logprobs')
I had to add a few environment variables and dependencies for updating llama-cpp-python

@SnehalSwadhin This represents a major change to the existing dependencies. Are you certain that llama-cpp-python 0.2.24 cannot be used for your solution? Is it possible to debug or work around the error in 0.2.24?
If not, it will take us some time to determine if all of these changes are compatible with solutions that worked in the previous runtime. This is a really impressive deep dive into advanced package installation with pixi 👏

I was able to get results with structured_structured output in langchain pretty easily with ollama (about a week). Just using llama.cpp has been a pain for more than a month now. I actually gave up at some point😅, but came back today to give it a last try.
Because it works on my device, its very difficult to debug. And I was not able to find any other fix for: Llama.create_chat_completion() got an unexpected keyword argument 'logprobs'
Anyway, it was fun figuring out something new🙌

At this point I just want to see one of my submission run successfully. Though the sad part is that it probably does not score as high as some submissions on the leaderboard.

If the change would be important to your solution we can certainly consider, just let me know!
I also enjoyed reading the PR -- this is the first competition for which we've used Pixi to manage dependencies, and your code uses a lot Pixi functionality that I hadn't seen before.
In case you hadn't seen it, there is a second DrivenData competition that uses the same data, but is an unsupervised challenge that does not rely on code execution for submissions. If you're interested, you could experiment with some of these methods for that challenge.

I've got a teammate working on the second track.
Let me try one more day with llama-cpp-python 0.2.24
If it still doesn't work, I'll tag you here.

@klwetstone Although I managed to bypass the logprobs error by overriding a method in ChatLlamaCpp, but my code ultimately does not seem to work in 0.2.24. Though it works without any issue in the newer version.
Invoking any llm with structured_output only returns None. I tried many combinations of different prompts, old & new models (mistral, meta-llama, granite, dolphin), and even tried the basic LamaCpp (non chat_models).
I'd request you to try and update the library.
If it seem to go beyond the rules of the competition, please do let me know. I appreciate your cooperation.

HI @SnehalSwadhin. Unfortunately, we are not able to support changes to the runtime that would break existing submissions. Since we already had llama-cpp-python 0.2.24 as part of the runtime, changing it to 0.3.1 would risk breaking existing submissions. We should have flagged when you first submitted your PR, and deeply apologize for the inconvenience to your team. It is a well constructed PR, and I applaud your team for finding clever workarounds in the pixi.toml file.
I’m closing this PR, since we’re unable to support llama-cpp-python 0.3.1. Good luck, and our apologies once again for the inconvenience.


Ensure the correct Content-Length header when streaming uploads

Fixes #447
When the user's PHP installation includes the curl extension (most do), the curl handler would fix the content length. This only surfaced for PHP instances without curl, but should still be fixed.

Everything LGTM. I'd be happy to get this in if it's ready for merge, just let me know.

LGTM

Yeah, it should be ready.


Add support for SSH connections

This add support for connecting to a Teamspeak server over SSH with the new client option SSH(...)
golang.org/x/crypto/ssh is used for SSH communication.
Example usage:
...

sshConfig := &ssh.ClientConfig{
	User: "serveradmin",
	Auth: []ssh.AuthMethod{
		ssh.Password("my-password"),
	},
	HostKeyCallback: ssh.InsecureIgnoreHostKey(), // has to be handled by the user correctly
}

c, err := ts3.NewClient("localhost", ts3.SSH(sshConfig))

...

Everything should be fully backwards compatible.
Closes #31

I will get this merged when I get my permissions sorted out.

Don't worry its been sat in my todo for some time, so thank you for your patience.
I just noticed that the team removed travis ci a while back, so I've just put a PR in to add github actions to bring back running tests and linting.
Once this team approves this, I'll ask you to rebase so we can validate everything is good according to the linters.

Hey @HalloTschuess can you rebase on top of the latest master to fix the conflicts and enable CI validation please.

Hey @stevenh I rebased my changes.

Thanks @HalloTschuess please see the lint failure.

Thanks for the notice @stevenh. Linting errors should be fixed now.

Thanks for the PR @HalloTschuess all merged, will cut a release once I've added auto release functionality.

Looks like this introduced a race condition in the tests, could you have a look please @HalloTschuess

PR https://github.com/Unity-Technologies/go-ts3/pull/38 should fix, if you could review @HalloTschuess


Whenever MS App Password contains | operator, Composer gives an error message

Whenever MS App Password contains | (pipe) operator, Composer gives an error message.
For eg: if my password is "H2Uc_|^Ax.j(+3m>NVVx]DOx9", it gives following error message.

Closing this as a dupe of https://github.com/microsoft/BotFramework-Composer/issues/7936. A PR is already out to fix this issue.


WebSite https://suaurl.com

Domain
https://suaurl.com
Link
https://suaurl.com/LastTokyoRevengersRP
Version
Version 0.2256
What browsers are you seeing the problem on?
Chrome
What OS are you seeing the problem on?
Windows
Anything else?
Not working in https://suaurl.com

We do not currently bypass this site based on this list, please read https://github.com/FastForwardTeam/FastForward/issues/656 for more details.
Once we have finished with the project, you are free to reopen the issue.


Supporting Azure DevOps Git repos in Flux similar to other public git hosts

Looking for Azure DevOps git repo support in flux by default similar to GitHub, GitLabs, and other public git hosts without going through private git host setup.

Microsoft/bedrock#81


Remove dead branches in scenecut code

This function never gets called if distance < min_keyint, so these two branches are unnecessary. (If distance == min_keyint, then the math works out so that the result equals THRESH_MIN.)

Verified no change in BD rate on vimeo set: AWCY

Coverage decreased (-0.07%) to 83.315% when pulling 350a56db59ae1a06533f00805f1b38509dcedf11 on shssoichiro:remove-dead-sc-code into 803b7071c953e4a7e2a3ed5e21ff63a6ff486d14 on xiph:master.


Sharing itinerary

User Story
As a frequent traveler,
I want to be able to create and manage my trip itineraries easily using the TravIs application,
so that I can efficiently plan and organize my travels.

Acceptance Criteria
Travel Community and Social Features

Given that I have created a trip,
When I choose to share my itinerary with others,
Then I should be able to share it via a unique link or invite specific users to view it.

in handlebars;

[ ] Add little gear icon or something like that in the top right of the view page (alongside 'edit' button)
[ ] Create a 'public-view-itinerary' page, which is a duplicate of the 'view-itinerary' page, but without the 'gear' icon & the 'edit' button

in (public) javascript

[ ] Clicking the 'gear' icon causes a modal to pop up, displaying a text element with a link (/trips/public-view/[current trip ID]), and a 'switch' to make the itinerary page public

[ ] Off by default, when switched on, sends a POST request (/api/trips/change-public) containing the current trip ID to update the 'public' field of the current trip to 'true'
[ ] Similar POST request to set the 'public' field to 'false' when switched off

in (controller) javascript;

[ ] GET route (/trips/public-view/:id) uses authentication middleware described below, which, if passed, renders the 'public-view-itinerary' page of the appropriate trip using the route parameter ':id'
[ ] API POST route (/api/trips/change-public) retrieves trip_id (number) & public (boolean) from req.body, uses it to update the 'public' field of the trip with the matching 'id'

in utils;

[ ] Create authentication middleware to check if the trip the user is currently trying to access has a 'public' field = true

[ ] If it is true, continue with the routing process. If it is false, redirect to the 'bad request' page

in models;

[ ] Add a 'public' field to the 'Trip' model (boolean, not null, default: false)


WFCORE-725 Initial work on removing ClassIndex

This moves similar functionality found in ClassIndex into DeploymentClassIndex. Once
a core release with these changes has been released then full can remove all references
to ClassIndex, and it can be removed from the code base.

Core - Full Integration Build 1042 is now running using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b

Linux Build 1902 is now running using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b

Windows Build 1453 is now running using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b

Windows Build 1453 outcome was SUCCESS using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b
Summary: Tests passed: 3122, ignored: 71 Build time: 0:21:08

Linux Build 1902 outcome was SUCCESS using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b
Summary: Tests passed: 3122, ignored: 71 Build time: 0:22:56

Core - Full Integration Build 1042 outcome was FAILURE using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b
Summary: Tests failed: 1 (1 new), passed: 2873, ignored: 374, muted: 1 Build time: 0:50:33
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.remote.distinctname.defaultname.DefaultDistinctNameTestCase.testRemoteSLSBInvocation: <no details avaliable>

retest this please

Windows Build 1480 is now running using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b

Core - Full Integration Build 1067 is now running using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b

Linux Build 1927 is now running using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b

Windows Build 1480 outcome was SUCCESS using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b
Summary: Tests passed: 3122, ignored: 71 Build time: 0:22:14

Linux Build 1927 outcome was SUCCESS using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b
Summary: Tests passed: 3122, ignored: 71 Build time: 0:23:55

Core - Full Integration Build 1067 outcome was SUCCESS using a merge of 4f0b80aa1f4c59d9f9dda1a26c1c29ac2414cf2b
Summary: Tests passed: 2872, ignored: 375, muted: 1 Build time: 0:52:00


getAssayfor() Optional Column possible correction

I think m.matches() here should be replaced with m.find() because a possible value for identifier is assay[1]_peak_mz or as given in this example, it could be assay[1]_my_value for which m.matches() will give false

@SurajViitk Snapshot build has been deployed. Please test!

test done, working fine


Office Addin Vue js quickstart TYPO

[Enter feedback here]
in reference to: https://docs.microsoft.com/en-us/office/dev/add-ins/quickstarts/excel-quickstart-vue
There is a TYPO in part 3 in the HTML code is presented.
SHOULD not be id="content-main", it should be class="content-main"
since it is defined as a class in the styles.
.content-main {
background: #fff;
position: fixed;
top: 80px;
left: 0;
right: 0;
bottom: 0;
overflow: auto;
}

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

ID: ea230e95-f483-6d19-8167-2341a03cf396
Version Independent ID: 3846765f-7587-fec4-711e-84edfce9c30c
Content: Build an Excel task pane add-in using Vue - Office Add-ins
Content Source: docs/quickstarts/excel-quickstart-vue.md
Product: excel
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Fixed, submitted PR to merge fix to live. Check back tomorrow!

@ramalva92 P.S. THANK YOU very much!


Self-building

Can I build it by myself
Can you provide tutorials or Dockerfile?

@GreenHatHG You can absolutely build locally. I've added the basic steps to the README here. I also added a link  to the create-react-app README, which has more comprehensive instructions.
If you specifically need a dockerized version there are a lot of resources for dockerizing a create-react-app, for instance https://mherman.org/blog/dockerizing-a-react-app/
I hope that helps. If you have trouble feel free to open another issue.
Cheers!


chore: added placeholder to repo info form

Description
Added dynamic placeholder for the parameter input element
Related Issue

issue #385
Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

Hey! Your solution is great, but someone created a PR 2 days ago to solve the same issue: https://github.com/privacy-scaling-explorations/bandada/pull/387.
Thank you so much.
Here is a list of good first issues, feel free to ask questions, take a new one and ask us to assign you: https://github.com/privacy-scaling-explorations/bandada/issues?q=is%3Aissue+is%3Aopen+label%3A"good+first+issue"

Duplicated #387


add unlikeconstraint

Marathon supports UnlikeConstraints to run tasks on slaves not matching a given attribute.
This adds the equivalent to Chronos. Most of this has been c+p from the LikeConstraint class, with the case conditions negated in the matches() function of the Constraint.
Warning: I don't have any prior experience writing Scala, so I'm open to ideas for cleaning this up. The range constraint has been left unimplemented as in the LikeConstraint.

Assuming this is a good approach, can you also update the docs? https://github.com/mesos/chronos/blob/c91d2fb35688c361be12f8cab02ce1d2af1e34c0/docs/docs/api.md#constraints

hi @mwasn. I see you've been assigned this PR - could I bump this for you to have a look? Thanks.

Although I hate to +1, after ~month I feel obligated to bump this. @mwasn can you or someone from mesossphere review this?

Over two months have passed since I opened this PR now - @gkleiman @ssk2 sorry to bump you but I see you've been active on other PRs. Can either of you or anyone else from mesosphere help me get this merged?

@Rob-Johnson, let me see what I can do.

@ssk2 hello - another month has passed, anything you can do to help me get this merged?


GeometryInstance3D transparency does not work in mobile or compatibility renderers

Godot version
4.0.2 stable
System information
win10 960m + win11 3050
Issue description
The transparency parameter on a geometry instance 3d object only works  in the forward+ renderer. It does not work with the vulkan mobile or opengl compatibility renderers.
@clayjohn has confirmed that it has not been added to these renderers.
Steps to reproduce

add a cube mesh
move the transparency slider

Minimal reproduction project
Not really necessary.

This also applies to visibility range distance fade using the Self or Dependencies fade mode. Both act the same as Disabled when using the Forward Mobile or Compatibility rendering methods (https://github.com/godotengine/godot/issues/69877).

I spent a few hours before discovering this issue. I might recommend documenting this limitation . The GeometryInstance3D.transparency (etc) docs don't mention that these are only available in Forward+.

Just ran into this. Should at the very least be documented, but ideally support could be added.

BTW, should this ticket be replicated into #55871? Currently it's only on the OpenGL/compatibility tracker.

Done.


Audio distortion generated using the bark model

After selecting speaker, the generated audio is incorrect.
For example, the audio generated with 'v2/en_speaker_3' sometimes comes out as a female voice, when it should have been a male voice. And sometimes the audio is distorted.
Environment:

vox-box version: 0.0.2
model: Hugging Face/suno/bark

Related issue: https://github.com/huggingface/transformers/issues/34634


CB-15442: Made all usages of ManagedChannelWrapper avoid creating new instances unnecessarily

Original PR (#13480) had to be reverted (#13512)


Replace old TOF sensor with Evo TOF sensor in main file and add cache to Evo sensor

Main Changes
Replace old TOF sensor with Evo TOF sensor in main file and add cache to Evo sensor.
Issues
Closes #188
Tests
Not tested yet.

I've added a few comments. By looking at this it seems you want to make the EvoSensor class as similar to TimeOfFlightSensor to work in main.py which makes sense to start with. Later we might want to change the data is saved at the end.
I think its important to test changes on the quadcopter before merging as well.

I think PR #202 Covers issue #188 and since we are doing a new altitude hold with more sensors I don't think there is much need for this PR?
What's everyone else's thoughts?

I think PR #202 Covers issue #188 and since we are doing a new altitude hold with more sensors I don't think there is much need for this PR? What's everyone else's thoughts?

Do you mean that we shall close this PR and make changes about the new TOF sensor in PR #202 ?

Yea because we are using the new ToF sensor in #202 to read and save the distance is from the ground.

I think PR #202 Covers issue #188 and since we are doing a new altitude hold with more sensors I don't think there is much need for this PR? What's everyone else's thoughts?

Do you mean that we shall close this PR and make changes about the new TOF sensor in PR #202 ?

Yea because we are using the new ToF sensor in #202 to read and save the distance is from the ground.

Good to me

Should we close this now since I uploaded all the changes to PR #202


Number of Interation

I'm using the optimus-py in my system, and I have a cube with 6 dimension, when I use the optimus-py it tests 19 interations, but with 6 dimension we have a possible number of 6 factorial (720) interations, So Why it only tests 19, and how we know that the best is one of this 19 and not one of the other 701 the we do not have tested.

You gotta read this

Thanks


Loading array data : one option auto selected

    $("#unit_chan_id").select2({         "placeholder"    : 'unit',         "minimumResultsForSearch" : Infinity,         "allowClear"              : true     });
when i reset options:
                $("#unit").select2({                     "data" : data.info                 });
then, placeholder lost, one option auto selected.
When I reset is how to display placeholder, don't selected  option

This issue does not appear to conform to our contributing guidelines.
Please re-read the contributing guidelines carefully and then edit your question.  Once your issue complies with the guidelines, we can reopen it.


558: Terminal entry not found in terminfo "cygwin"

When using vim on the VM I get following message. Vim also behaves in a very weird way. Is there a possibility to add cygwin in the VM.
558: Terminal entry not found in terminfo
cygwin' not known. Available builtin terminals are:
   builtin_ansi
   builtin_xterm
   builtin_iris-ansi
   builtin_dumb

Moved to https://github.com/coreos/bugs/issues/1322


Added ability to add classes to the containing form of the editable input

Added ability to add classes to the containing form of the editable input.
Manually remove the watcher for "has-error" to prevent a memory leak on it.
Fixes #476

@ckosloski, thanks for your commitment.


Genome annotation fixes

This is a continuation of https://github.com/kbase/data_api/pull/128
The only significant change is the copying of the feature lists when they are filtered and returned. Previous code modified these in place, which was a side-effect on the object. I added the 'blist' library to reduce the time spent manipulating the lists (e.g., deletion goes from O(N) to O(logN)).
While I was in there, I cleaned up the output from the genome annotation client driver (which is the code that showed the error to begin with). A possible future enhancement would be more tests of the idempotent nature of the filtering operations, but I'm pretty satisfied with it for now.

Current coverage is 68.71%

Merging #131 into develop will decrease coverage by -0.57% as of b63f70d

@@            develop    #131   diff @@
=======================================
  Files            11      11
  Stmts          2331    2362    +31
  Branches          0       0
  Methods           0       0
=======================================
+ Hit            1615    1623     +8
  Partial           0       0
- Missed          716     739    +23

Review entire Coverage Diff as of b63f70d

Powered by Codecov. Updated on successful CI builds.

Sorry, noticed that also some docs changes got picked up by this commit, but this is (a) harmless, and (b) quite harmless, so I'm not going to bother cherry-picking them out.


SiteName is still required as of <IP_ADDRESS>

Sitename is being passed as an empty string when not included instead of $Null.
Line 135 of the file MSFT_xADDomainController.psm1
Is currently:
if ($SiteName -ne $null)
and needs to be modified to:
if ($SiteName -ne $null -and $Sitename -ne “”)

fixed in
MSFT_xADDomainController.txt

Pull request submitted with fix

Fixed by #133


When to release VoiceAssistant-400K dataset

请问下作者，论文中提到的VoiceAssistant-400K什么时候能发布出来呢，谢谢

Waiting for your great training code!

For now, we are sorting things out. The dataset will come out shortly. Regarding the training code, there is currently no definitive release timeline.

Is the dataset currently available?

the dataset is uploading, and the path will be: https://huggingface.co/datasets/gpt-omni/VoiceAssistant-400K (if the uploading process goes well)


Issue after applying proguard

I m facing the below error in my app after applying proguard. Please help me in applying proguard for my app
Fatal Exception: java.lang.NullPointerException: value for name X-AUTH-TOKEN == null        at okhttp3.Headers$Builder.checkNameAndValue(SourceFile:334)        at okhttp3.Headers$Builder.set(SourceFile:318)        at okhttp3.Request$Builder.header(SourceFile:165)        at skillsapp.org.vidyalaya.android.api.BaseApi$1.intercept(SourceFile:102)        at okhttp3.internal.http.RealInterceptorChain.proceed(SourceFile:147)        at okhttp3.internal.http.RealInterceptorChain.proceed(SourceFile:121)        at okhttp3.RealCall.getResponseWithInterceptorChain(SourceFile:200)        at okhttp3.RealCall$AsyncCall.execute(SourceFile:147)        at okhttp3.internal.NamedRunnable.run(SourceFile:32)        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)        at java.lang.Thread.run(Thread.java:761)

Thanks for your question. This issue tracker is only for bug reports with test cases and feature requests. Please ask usage questions on Stack Overflow.
https://stackoverflow.com/questions/tagged/okhttp


Routing any prefix and diferent menu

Version information

Laravel: v5.7
Voyager: v1.1.9
PHP: 7.2
Database: MySQL 5.7

Description

How can I route many prefixes using Voyager::routes() in ./routes/web.php
example for admin, main, dev
Route::group(['prefix' => 'admin'], function () {
Voyager::routes();
});
Route::group(['prefix' => 'main'], function () {
Voyager::routes();
});
Route::group(['prefix' => 'dev'], function () {
Voyager::routes();
});
and get a different menu for each prefix in ./dashboard/sidebar.blade.php
{!! menu('admin', 'admin_menu') !!}
{!! menu('main', 'admin_menu') !!}
{!! menu('dev', 'admin_menu') !!}
Steps To Reproduce
Expected behavior
for 'prefix' => 'admin'
{!! menu('admin', 'admin_menu') !!}
for 'prefix' => 'main'
{!! menu('main', 'admin_menu') !!}
for 'prefix' => 'dev'
{!! menu('dev', 'admin_menu') !!}
Screenshots
Additional context

Please ask questions in our Slack group (as mentioned in the issue-template).
If you really need that, you'd have to override the view but we suggest doing this with roles/permissions.

Sorry, but I do not know how to go to the Slack group to ask my question

ready, I'm already registered in Slack


Prerequisites

[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.

Criteria

[X] Reasonably well-known and widely used (e.g. if it's a GitHub project, it should have at least 100 stars and/or 50 forks)
[X] English interface (or at least English documentation)
[X] Latest stable version
[X] Full version (i.e. not a trial version)
[ ] Fairly standard install (e.g. uses a version-specific download URL, no elaborate pre/post install scripts)

Name
MultiMC
Description
A custom launcher for Minecraft that allows you to easily manage multiple installations of Minecraft at once
Homepage
https://multimc.org
Download Link(s)
https://files.multimc.org/downloads/mmc-develop-win32.zip
Some Indication of Popularity/Repute
GitHub reviews and other reputable sites like https://download.fo

Downloads as a zip file, requries unzipping to use

Calinou/scoop-games

oh bless
I thought there was an official one under the org's account but I didnt see it


tests fail in appveyor for Python 3.5, 3.6 and 3.7

The master branch is also failing the testsuite on appveyor (see #1 where this has also been encountered). The problem is simply a non supported option of py.test
[command]/bin/bash --noprofile --norc /home/vsts/work/_temp/417b02aa-2ee9-4d00-becc-a515a96ad954.sh
py35 create: /home/vsts/work/1/s/.tox/py35
py35 installdeps: hypothesis, pytest, pytest-cov
py35 develop-inst: /home/vsts/work/1/s
py35 installed: atomicwrites==1.3.0,attrs==19.1.0,coverage==4.5.3,cypari==2.3.0,future==0.17.1,hypothesis==4.26.3,importlib-metadata==0.18,more-itertools==7.1.0,mpmath==1.1.0,packaging==19.0,pathlib2==2.3.4,pluggy==0.12.0,py==1.8.0,pyparsing==2.4.0,pytest==5.0.1,pytest-cov==2.7.1,-e git+https://github.com/MarkCBell/realalg@e8d0a0e722dec47bac02e502165f229f71faffe2#egg=realalg,six==1.12.0,sympy==1.4,wcwidth==0.1.7,zipp==0.5.1
py35 run-test-pre: PYTHONHASHSEED='294238053'
py35 run-test: commands[0] | py.test --hypothesis-profile=ci --duration=20 --junitxml=results.xml
ERROR: usage: py.test [options] [file_or_dir] [file_or_dir] [...]
py.test: error: unrecognized arguments: --duration=20
 inifile: /home/vsts/work/1/s/tox.ini
 rootdir: /home/vsts/work/1/s

ERROR: InvocationError for command /home/vsts/work/1/s/.tox/py35/bin/py.test --hypothesis-profile=ci --duration=20 --junitxml=results.xml (exited with code 4)

Thank you very much for letting me know. The issue comes from py.test having recently switched from using the --duration flag to the --durations flag (although they don't appear to have documented this anywhere). Originally I caught this for travis but missed appveyor and azure. The same issue affected flipper and curver but all three have now had this fixed in their dev branches and the master will be updated at the weekend.


⚠ ClamAV has degraded performance

In 53cda8b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 499 ms

Resolved: ClamAV performance has improved in ca6ab01 after 6 minutes.


Indirectly calling setState from constructor doesn't trigger render

Here's some example code (in TypeScript)
class TestComponent extends React.Component<{}, {foo: number}> {
  constructor(props: {}) {
    super(props);

    this.state = { foo: 1 };

    setTimeout(() => this.setState({ foo: 5 }), 1000);
  }

  render(): JSX.Element {
    return <div>{ this.state.foo }</div>;
  }
}

const TestComponentWrapped = Radium(TestComponent);

Without wrapping the component in Radium, foo gets set to 5 after a second.
Once I wrap with Radium, foo stays 1.

Hmmm.... that should work. Can you open up a minimal repository with install + error reproduction steps for us to clone and verify? Thanks!

ping @johnfn

Closing for inactivity. @johnfn if this is still an issue let us know and we can reopen!

I'm certain this is still an issue and shouldn't be closed, however I do not have time to create a repo.


Feature plan

Implementation features to be included before 1.0 release:
[ ] - #2 auto review comments per component definition
[ ] - #3 failure comments on failed WFLY CI jobs

not needed anymore


Save button not working

Installed with docker compose. Console throws this error when I click the save button.

can't replicate, have you set a volume?
version: '3'
services:
  ha-fusion:
    container_name: ha-fusion
    image: ghcr.io/matt8707/ha-fusion
    volumes:
      - /path/to/ha-fusion:/app/data
    network_mode: bridge
    ports:
      - 5050:5050
    environment:
      TZ: Europe/Stockholm
      HASS_URL: http://<IP_ADDRESS>:8123
    restart: always

ahhh i had a typo in the volume path. that was my bad. this can be closed.


feat: gitpodify project

Fixes Issue
Closes #30
Changes proposed
Added following changes to the repo :point_down:

open in gitpod badge to the readme.
gitpod config file with initial gitpod environment setup.

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[X] I have updated the documentation accordingly.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

README.md

.gitpod.yml

Note to reviewers

Thank you +1

Most welcome :pray:


⚠ Bofrost.be has degraded performance

In 5f106b3, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1390 ms

Resolved: Bofrost.be performance has improved in b48162f after 6 minutes.


Upgrade cheerio to version 0.20.0

$ npm test
><EMAIL_ADDRESS>test /Users/amercier/Workspace/esdoc
> node ./script/test.js

[...]

  93 passing (3s)

@amercier Sorry for late reply 🙇
Thanks this PR!


As TA, "Set Score for Empty Cells" doesn't appear when you have grade for "All sections/groups" selected

This probably relates to the changes made in #1376, since it's now looking to specific sections wherein grading access is provided.

This is up and running. However, interesting issue:
When the TA has a single rule providing grade access for "All sections/groups," the individual sections/groups do not appear as selectable options in the "Set Score for Empty Cells" dropdown, even though they can filter by section/group in the primary filter on the Grades screen:

Is there a way to provide cascading section permissions for "Set Score for Empty Cells" when grading access for "All sections/groups" is applied?

Interestingly, when categories are enabled everything is peachy.. this only happens when the Gradebook has categories disabled.  Will dive in...

Confirmed.


Consider documenting required webpack configuration

libphonenumbers-js needs:

the JSON loader: {test: /\.json$/, loader: 'json'} in config.module.rules

'.json' in config.resolve.extensions (perhaps it would be easier to just reference the data file with the full name .min.json)

Note -- this is a webpack2 configuration.

Did you mean module.config.loaders instead of config.module.rules?
config.resolve.extensions doesn't need to be set, it has json by default:
https://github.com/webpack/webpack.js.org/issues/68

I think it's config.module.loaders in webpack1, config.module.rules in webpack2.
I had to change my extensions setting because I overrode the default with ['.js', '.jsx', '.css', '.sass'] -- I didn't include '.json' because I didn't need it until I faced an error when trying to use libphonenumbers-js
BTW thanks for a great alternative to Google's library :-)

Oh, so webpack 2 conflicts with webpack 1.
I guess then the project will stick with webpack 1 until webpack 2 is released.

Oh, I get it now, you're bundling it with Webpack and json loader is required.
Yes, I guess I will specify the default .json extension then, and after that it should be working.
And as for json loader, I think webpack should enable it by default (that would be logical).

Anyway, I don't know how your Webpack even made it into /node_modules/ folder.
Didn't you exclude /node_modules/ from your js-loader config?

Ok, I think, this is gonna be the answer:

By overriding the default extensions you take the responsibility to include .json there, so it's not a bug, it's intended to work that way
Webpack team should have enabled json-loader by default for .json files. If they chose to impose that responsibility on Webpack users then there's nothing to be done here.

To be honest I'm not an expert with webpack and I'm not sure what happens exactly.
I followed your (very standard) instructions:
npm install libphonenumber-js --save

import { asYouType } from 'libphonenumber-js'

and I was surprised to see my build crash.
The error most likely originates at this line in node_modules/libphonenumber-js/build/as you type.js:
var _metadata = require('../metadata.min');

I was easy enough to debug. I thought I'd report it in case you wanted to document the requirements in order to save other users some trouble. No big deal either way :-)

The users most likely won't have any trouble because most of the users don't override extensions, and those who do must understand exactly what they're doing, so your case is very rare.
Also, the fact that Webpack even parses node_modules folder is also very rare and I don't know why didn't you exclude node_modules from babel-loader.

I'm sorry if I'm being thick — we can stop this discussion if it isn't interesting for you; I have solved my problem; I'm just trying to understand.
In case you're still with me... as far as I can tell:

node_modules/libphonenumber-js/package.json says: "main": "index.common.js"
node_modules/libphonenumber-js/index.common.js says: `exports.asYouType   = require('./build/as you type').default
node_modules/libphonenumber-js/build/as you type.js says: var _metadata = require('../metadata.min')

How I could that work without having something (webpack in my case) take these files inside node_modules, convert require calls into something that browsers can handle, including require('../metadata.min')?
The latter seems to require webpack with some options, which are enabled by default.
My config excludes node_modules from babel-loader. However I don't think that's relevant here.
I can't comment on how many users override extensions or whether they know what they are doing. FWIW some popular boilerplates do that and omit .json. But, yeah, anecdote, not data :-)

I'm open to any discussion which can clarify bugs or inconveniences, especially if those are inside my code.
Ok, I get it, seems that Webpack actually needs that json-loader to process json files correctly.
https://github.com/webpack/webpack/issues/965
But, I think it's Webpack's bug rather than something that needs to be changed in this repo.
The Node.js convention is that .js and .json requires are always valid so Webpack team should do something about that.
The popular boilerplates may be "popular" which doesn't imply "correct", so if they exclude .json then it means they're wrong.

At the same time Webpack lacks a lot in terms of user-friendliness, usability and sensible defaults.
But since Webpack Foundation has been formed recently, I think they'll get it right some day.

@aaugustin Ok, I added the notes for Webpack since this is likely a common case. Have a look if the notes are correct and exhaustive.

Great! Thanks!

Hey Guys, I am trying to use libphonenumber-js in my angular application with webpack 2.2.1, I installed libphonenumber-js through npm and then imported "import { parser, format, asYouType } from 'libphonenumber-js';"
I get an error message saying cannot find module 'libphonenumber-js'
Any ideas? I am I missing any steps?


Missing section.

Looking for this section: "Running scripts after setup and on startup" in the documentation but couldn't find it.

Hi, you'll need to refer to the original documentation from Oracle for this.


Can't import styles

Not sure this is really fixed yet - it also happens on regular npm package.  I assume it's a problem here because 10.x works but 11.x does not - I don't have this issue with any other npm package.  I migrated to 11.0.0 when it came out and had to roll back to latest 10.x because of this problem.  I upgraded to 11.1.0 this morning and still have this issue.
dependencies:
"dependencies": {
  ...
  "highlight.js": "11.1.0",
  "markdown-it": "12.1.0",
  "markdown-it-attrs": "4.0.0",
  ...
}

imports:
import MarkdownIt from 'markdown-it';
import MarkdownItAttrs from 'markdown-it-attrs';
import hljs from 'highlight.js/lib/core';
import 'highlight.js/styles/github.css';

hljs.registerLanguage('json', require('highlight.js/lib/languages/json'));
hljs.registerLanguage('markdown', require('highlight.js/lib/languages/markdown'));
hljs.registerLanguage('python', require('highlight.js/lib/languages/python'));
hljs.registerLanguage('xml', require('highlight.js/lib/languages/xml'));
hljs.registerLanguage('yaml', require('highlight.js/lib/languages/yaml'));

attempt to lint (build same problem, but not from eslint obviously):
Oops! Something went wrong! :(

ESLint: 7.29.0

Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './styles/github.css' is not defined by "exports" in /Users/johnm/src/sdvi/swarm/node_modules/highlight.js/package.json
    at throwExportsNotFound (internal/modules/esm/resolve.js:285:9)
    at packageExportsResolve (internal/modules/esm/resolve.js:491:3)
    at resolveExports (internal/modules/cjs/loader.js:444:36)
    at Function.Module._findPath (internal/modules/cjs/loader.js:484:31)
    at findModulePath (/Users/johnm/src/sdvi/swarm/node_modules/eslint-import-resolver-alias/index.js:99:27)
    at Object.exports.resolve (/Users/johnm/src/sdvi/swarm/node_modules/eslint-import-resolver-alias/index.js:75:10)
    at v2 (/Users/johnm/src/sdvi/swarm/node_modules/eslint-module-utils/resolve.js:117:23)
    at withResolver (/Users/johnm/src/sdvi/swarm/node_modules/eslint-module-utils/resolve.js:122:14)
    at fullResolve (/Users/johnm/src/sdvi/swarm/node_modules/eslint-module-utils/resolve.js:139:22)
    at Function.relative (/Users/johnm/src/sdvi/swarm/node_modules/eslint-module-utils/resolve.js:84:10)
error Command failed with exit code 2.

However, this works, but is not right:
import MarkdownIt from 'markdown-it';
import MarkdownItAttrs from 'markdown-it-attrs';
import hljs from 'highlight.js/lib/core';
import '../../node_modules/highlight.js/styles/github.css';

hljs.registerLanguage('json', require('../../node_modules/highlight.js/lib/languages/json'));
hljs.registerLanguage('markdown', require('../../node_modules/highlight.js/lib/languages/markdown'));
hljs.registerLanguage('python', require('../../node_modules/highlight.js/lib/languages/python'));
hljs.registerLanguage('xml', require('../../node_modules/highlight.js/lib/languages/xml'));
hljs.registerLanguage('yaml', require('../../node_modules/highlight.js/lib/languages/yaml'));

Edit
Actually, the ../../node_modules/ bit makes eslint work, but both my current webpack and rollup builds fail with the path specified.
Originally posted by @bitencode in https://github.com/highlightjs/highlight.js/issues/3223#issuecomment-880005163

Splitting this out as this is not the same issue.  The prior thread is related to being unable to import .min.js files from the CDN build (which has been resolved) and you're talking about importing styles from the npm build (afaict).

is not defined by "exports" in /Users/johnm/src/sdvi/swarm/node_modules/highlight.js/package.json

I would guess that the highlight.js version used by swarm is broken/older, not 11.1... If you look at the latest NPM builds for the main library (not CDN) you'll see we very much export styles.  That issue has also been resolved AFAIK.

Oh I was reading that like swarm was a dependency of your project but perhaps it's the main project...? if so then I'm not sure what is happening... but if you open package.json in highlight.js you should be able to confirm that the exports are all listed there...
Maybe you aren't using the version you think you are? I'd check out the file it's saying doesn't have the exports and see if it does or doesn't.

Thanks @joshgoebel, sorry for not realizing it was a separate issue - thanks for splitting it out.
When I look at the npm package locally I can see the exports, exactly like you show above, so that has me a bit confused 😕.
One thing I did forget to mention (though it shouldn't make a difference), is this is a mono-repo where swarm is the git repository and we have services, ui libraries, and several application all using the libraries in a packages folder.  That results in the npm packages being located in swarm/node_modules/... and our libraries/apps being located in swarm/packages/<package name>/  everything else seems to be working. Yarn is linking everything up properly though.
I did try to disable eslint for those lines (the failure happens for the github.css file and all language files that are being required - require('highlight.js/lib/languages/markdown'). However, there is nothing I can find to disable - the error is happening in what appears to be an internal node require function, so evening using /* eslint disable */ at the top of the file doesn't prevent the error from occurring.
Just for confirmation - I have the following versions:
node: v12.19.1
yarn:  v1.23.0-20200928.1349
highlight.js: 11.1.0
eslint: v7.29.0
Sounds like no one else is having problem with this, so I'm going to guess it's some configuration problem we've missed in our lint, build, tool chain.  I'll look at it some more and when I figure out what's gone amiss I'll come back here, outline the problem and close (assuming I find and fix a configuration problem)
Thanks!

Good luck.

v12.19.1

This isn't the latest release, perhaps there are subpath bugs that a more recent v12 release fixes. A lot of the ESM stuff was back-ported to v12 in minor releases and therefore it improved with time... just one idea.

Although, node can't import styles  - it imports JS... so I'd suggest perhaps it's your build pipeline that's choking up and may need one of the pieces upgraded?

@joshgoebel Thanks for you help and input on this.  I dug in a bit more and found that one of our packages ended up with a slight misconfiguration of build tools when we move them all into the mono-repo.  For some reason everything seems to be working "fine" until this upgrade that explicit defines the exported packages.  Anyway, it was a problem with the configuration for eslint-import-resovler-alias and a similar config problem for babel-plugin-module-resolver.  Once I fixed those, it all started working.
Thanks! and sorry for the issue logged here - you can close it.

No problem. Glad you got it sorted.


synchronize ref definitions

This PR syncs the definitions of netcore netstandard and netfx ref projects where possible. This is mainly by reordering the types into but some attributes have been added to netcore where they were present in netfx since this isn't an interface change and enhances mainly designer experiences. It's now possible to diff the ref definitions and get a pretty clear picture what the differences in public surface are between the different versions.

I started looking at the changes and there are quite a few additions to the netcore ref file beyond the designer attributes.

SqlCommand.NotificationAutoEnlist
SqlConnectionStringBuilder.AsynchronousProcessing

(Quite a few under here actually, that have not been brought to netcore.)
I've run out of time to go through the rest. It's hard to tell what's been added through all the re-ordering. I'll try to continue on Monday.

removed from SqlConnectionStringBuilder.ConnectionReset, ContextConnection, NetworkLibrary, TransparentNetworkIPResolution. removed SqlCommand.NotificationAutoEnlist.
I might see if I can do something with reflection if I can find the non-ref binaries.

I write a program which dumps the definitions from a binary and then did some manual comparisons. interesting results overall.
net462-ref.txt
netcore-impl.txt
netcore-ref.txt
net462-impl.txt

The comparisons are definitely interesting. I'm having to do a lot of research for this PR. Learning new things is not bad. Some questions in my mind are:

When would we want the ref definition to not match the implementation and why?
What exactly do the various designer attributes do? (I'm new to these so reading a lot but the MS docs tend to be sparse.) Is it safe to add them or is there anything that needs to be verified about methods for which we are adding those attributes?

We don't have any tests around the ref definition, so verifying this PR is all manual. It might be easier if this were split into different PRs where we can easily accept a round of reformatting/white space changes, a round of only sorting changes (no lines added or changed), and then a round of actual changes where we can ask questions about specific changes. Like why has Microsoft.Data.SqlClient. been prepended to a reference? Or why has throw null; been added or removed? Or how is it that SqlDataReader.Close() did not already exist in ref and no one noticed until now? (I guess everyone is just wrapping it in a using or calling Dispose().)

When would we want the ref definition to not match the implementation and why?

From my understanding they should never. They can differ with varying consequences, least damaging is the metadata. The only purpose of the ref assemblies is to be bound against by compilers when the exact TFM version has not or can not be determined. If something exists in ref but not impl then an attempt to use it will cause a runtime failure. If something exists in impl but not ref then it may or may not be visible to the compiler which could cause editor/intellisense visibility strangeness or compiler errors complaining about missing types.
The attributes I took over are all the ones that didn't need any additional references. The Security attributes are irrelevant on netcore, security is stubbed. The rest were componentmodel designer metadata which influences the propertygrid and other visual studio designer experiences, so not needed but having them present will make things feel more natural to people who use visual designers. HostProtectionAttribute was the only one that I didn't attempt to sync, that doesn't exist and won't on netcore because the reliability infrastructure it's part of is not in that runtime.

Like why has Microsoft.Data.SqlClient. been prepended to a reference?

Any type which has a language keyword can use that language keyword. Anything which does not should use a fully qualified name. My assumption for this is that it prevents there being any possibly namespace mistakes by virtue of there being no using directives at all, everything must be explicit about the exact type. The ref assemblies can be tool generated in corefx which is why everything is strictly in alphanumeric order.

Or why has throw null; been added or removed?

if an attempt is made to use the ref assembly as an impl is made it should fail as soon as possible. throwing null is the pattern used throughout the other ref assemblies for this so if it was missing I added it.

Or how is it that SqlDataReader.Close() did not already exist in ref and no one noticed until now? (I guess everyone is just wrapping it in a using or calling Dispose().)

That's what I assume. Since Close is inherited from DbDataReader it will have been present in the intellisense from the base type so even if you looked for it you'd find it and the callvirt would mean you called the right one. Not a damaging oversight. It does make it clear that the existing ref in corefx wasn't tool generated.
I can rework from the beginning but I'm not sure it'll be less confusing to do so. The process logically involves diffing the files and making repeated rounds of modifications to limit the diff output. interim points in the process have no relative value,

SqlFacetAttribute is in the ref but not impl. Do you want me to copy the definition from netfx into netcore? It's a surface area compatibility class since it can't possibly be useful in netcore.

SqlFacetAttribute is in the ref but not impl. Do you want me to copy the definition from netfx into netcore? It's a surface area compatibility class since it can't possibly be useful in netcore.

I don't see an impl in netfx. I think SqlFacetAttribute probably was meant to be removed from ref entirely when we resolved the Microsoft.SqlServer.Server namespace issue. We removed that namespace entirely and moved SqlDataRecord + dependent classes into Microsoft.Data.SqlClient.Server since SqlDataRecord seemed to be the main class used by users (we can easily bring others in if we are wrong about usage but it's hard to remove stuff).

@Wraith2 I think this is good to merge as-is. If you want to add the SqlFacetAttribute change to this PR, let me know. Otherwise, I can approve it.

If you're happy with the current state I'd say merge it. I can come back to null throwing and the SqlFacetAttribute later in a dedicated PR if they need changes.


harpoon invalid window id

...vim/site/pack/packer/start/dimmer.nvim/lua/dimmer/ui.lua:74: Invalid window id: 1054
...vim/site/pack/packer/start/dimmer.nvim/lua/dimmer/ui.lua:74: Invalid window id: 1054

doesn't happen always
not necessarily only happens with harpoon either I think

it happens when I open splits, then close splits. state.overlays retains the split data but obv they don't exist anymore so dim_others trip up.
Need a win_close event


add interceptors pattern

With the validator we can't do async pre prossesing.

Closing as #207 will completely replace the implementation in this project.


orderBy array

I came across a problem with select query. When i have tried to join two different tables to sort some results, i couldn't get correct sequence of 'order by' clause.
After analyzed, it seems that you split usual 'order by' from the one which is used in join. It is any chance to get correct sequence? I mean, i would like to get for example:
order by 'table1.col1', 'table2.col1', 'table1.col2',
but at the moment i have got something like:
order by 'table1.col1', 'table1.col2', 'table2.col1'.

Mind sharing the code used @qlarin ?

The code which is responsible for orders it's really simple. Something like that:
protected function querySort(Query $query, $sorts = null)
{
}


WIP: Create python-package.yml

Current certificate issues resolved in https://github.com/beerfactory/hbmqtt/pull/232


Python code-gen; doc about setting

Checklist

[*] I have filed an issue about this change and discussed potential changes with the maintainers.
[ ] I have received the approval from the maintainers to make this change.
[*] This is not a stylistic, refactoring, or cleanup change.

Please note that the maintainers will not be reviewing this change until all checkboxes are ticked. See
the Contributions section in the README for more
details.
Discussion thread for this change
Issue number: 6725
Description of this change
The recently added Python code-generation handling system now has a setting to control the additional query. This PR contains a documentation tweak to point this out.

@andponlin-canva this is irrelevant now, isn't it?


Pulling out to reconsider what should be done.


Laravel webosckets not working with Laravel 9 & 10

I have installed laravel web socket on laravel 9 and 10 but it is not working and it shows a message
WebSocket connection to wss://... failed

but when i try with insecure protocol ws not wss, its work but i can't broadcast events, e.G.:
event (new \App\Events\MyEvent('Data..'))

I've configuration fully right configuriered !!
Any help?
Thanks for this amazing package.

Moved to Swoole!
Thanks!


Disambiguating version control system and repository

In OSPS-01:

The project's version control system MUST require multi-factor authentication for collaborators modifying the project repository settings or accessing sensitive data.

The definition for VCS points to examples like Git that cannot enforce multi-factor authentication. The definition for "repository" is closer to a source control platform (SCP) such as GitHub or GitLab that can enforce multi-factor authentication. I'm unsure if just replacing VCS with repository is sufficient, especially since the word "repository" doesn't always mean the centrally hosted copy to everyone.
Separately, does this requirement exclude other options for hosting source code? For example, I'm unsure if https://git.kernel.org/ can meet the MFA requirement.

This has been a point of contention for us on other entries as well. The current stance is that if a project or tool doesn't meet the baseline, it's their prerogative to either explain the reasoning to their users, or add features to conform as needed. (We shouldn't lower the bar simply because someone isn't currently passing it.)

Overall, I agree with not lowering the bar because someone isn't currently passing it. But I think there's a question of applicability here which goes back to the definitions and desired properties for the control. Perhaps we first have to establish whether git.kernel.org even fits the SCP / VCS platform definition? It's possible it doesn't.
"Require multi-factor authentication for the project's version control system, requiring collaborators to provide a second form of authentication when accessing sensitive data or modifying repository settings." I parse this to mean git.kernel.org doesn't meet the definition, but at first read it would seem that the control does apply and the project doesn't meet it. Maybe the controls could emphasize the objective more and indicate that MFA is one way to meet the objective for SCPs like GitHub?


RN v0.61 upgrade followup.

Awaiting #4151. Done!
Fixes: #4091

Cool, and #4151 has been merged! Thanks, @gnprice! I'll fix these conflicts and push this branch.

Mentioning this here, as #4091 will be closed automatically once this PR lands:
There'll be another important step after upgrading Unimodules, which would be easy to forget about, I think—https://github.com/zulip/zulip-mobile/commit/c8d0c8df5efe808214db78c2a5daad186c9dc91f should be reverted once we're on the new version, IIUC.
I could make that part of this PR if you want, @gnprice.

c8d0c8d should be reverted once we're on the new version, IIUC.

SGTM!

(huh, looking into the tools/verify-webview-js failure now, that's odd)

(huh, looking into the tools/verify-webview-js failure now, that's odd, and reproduces locally)

OK, just pushed a fix! Yikes, the culprit was a yarn yarn-deduplicate && yarn upon installing jest-expo, where tons of Babel-related things were updated. One more reason to stop considering jest-expo, I suppose—the idea to use it was mostly a whim, though I slightly like not having to make boring mocks for expo- modules.

Merged! Thanks again.
I looked into the Babel-related changes in chat, and as discussed there I merged some changes before this that isolated the Babel upgrades and simplified that commit back again.


Single player

If you don't plan on using this branch anymore, you can delete it.

@radu-matei We will continue to add features to this branch until we publish the app on the store


[Automation] Generate SDK based on TypeSpec 0.22.0

[Automation] Generate SDK based on TypeSpec 0.22.0

API change check
API changes are not detected in this pull request.


Going to System32 folder instead of parental directory

To Reproduce
Steps to reproduce the behavior:

Go to any directory
Click on h
Expected behavior
Go to the parental directory

Actual behavior
https://github.com/user-attachments/assets/c9ec62a9-5c95-4757-96bf-6239358d8b10
System information (please complete the following information):

OS: Windows 11 Home
Version 23H2
Superfile Version 1.1.4

I met the same bug. It always go to start dir in the config file when i type h


Use the TypedConstantDeclaration type

The TypedDeclaration::ConstantDeclaration(..) variant currently is dead code---it is not possible to create it in the AST.
https://github.com/FuelLabs/sway/blob/685f96e92cce8cd2755ffffb64a78b8fd4514660/sway-core/src/semantic_analysis/ast_node/declaration.rs#L29
Instead const declarations are lumped together with TypedVariableDeclarations during type checking.
https://github.com/FuelLabs/sway/blob/685f96e92cce8cd2755ffffb64a78b8fd4514660/sway-core/src/semantic_analysis/ast_node/mod.rs#L339
We will want to change this so that const declarations use exclusively the TypedConstDeclaration type.

@vaivaswatha I noticed that TypedDeclaration::ConstantDeclaration(..) is never used in the code on master: https://github.com/FuelLabs/sway/blob/6f3f7421c9c171e0b169e3f4f6d0b268ad57b89b/sway-core/src/semantic_analysis/ast_node/declaration.rs#L27
In your PR for this issue, will you ensure that TypedDeclaration::ConstantDeclaration(..) is either used, or removed as dead code?
Originally posted by @emilyaherbert in https://github.com/FuelLabs/sway/issues/996#issuecomment-1152700380

Fixed by #2158


[FLINK-19723][Connector/JDBC] Solve the problem of repeated data submission in the failure retry

What is the purpose of the change
for example,  if statement.executeBatch() failur for some reason, but the "statement" was not closed in the retry, there is a risk of data duplication:
for (int i = 1; i <= executionOptions.getMaxRetries(); i++) {
	try {
		attemptFlush();
		batchCount = 0;
		break;
	} catch (SQLException e) {
		LOG.error("JDBC executeBatch error, retry times = {}", i, e);
		if (i >= executionOptions.getMaxRetries()) {
			throw new IOException(e);
		}
		try {
			if (!connection.isValid(CONNECTION_CHECK_TIMEOUT_SECONDS)) {
				connection = connectionProvider.reestablishConnection();
                                jdbcStatementExecutor.closeStatements();
			        jdbcStatementExecutor.prepareStatements(connection);
			}
		} catch (Exception excpetion) {
			LOG.error("JDBC connection is not valid, and reestablish connection failed.", excpetion);
			throw new IOException("Reestablish JDBC connection failed", excpetion);
		}
		try {
			Thread.sleep(1000 * i);
		} catch (InterruptedException ex) {
			Thread.currentThread().interrupt();
			throw new IOException("unable to flush; interrupted while doing another attempt", e);
		}
	}
}

Should be always re-prepareStatement to solve the risk of data duplication
...
try {
	if (!connection.isValid(CONNECTION_CHECK_TIMEOUT_SECONDS)) {
		connection = connectionProvider.reestablishConnection();
	}
         jdbcStatementExecutor.closeStatements();
	jdbcStatementExecutor.prepareStatements(connection);
} catch (Exception excpetion) {
			LOG.error("JDBC connection is not valid, and reestablish connection failed.", excpetion);
			throw new IOException("Reestablish JDBC connection failed", excpetion);
}

Brief change log

When the retry occurs in flush method, we should be always re-prepareStatement

Verifying this change
This change is already covered by existing tests, such as JdbcTableOutputFormatTest#testJdbcOutputFormat.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented)

Hi @wuchong , could you help me review this ? :)

Could you add an unit test in JdbcDynamicOutputFormatTest to verify this bug fix?

Could you add an unit test in JdbcDynamicOutputFormatTest to verify this bug fix?

Thanks for review, i will add unit tests soon

Hi [~jark], after testing, this is not a bug, could you close the issue?:D
Generally, after executing executeBatch in jdbc, regardless of success or failure, the list of preparedStatements  will be cleared

Thanks @shizhengchao .


The old MYST_PATH that allowed multiple paths similar to $PATH has been removed in favor of MYST_HOME, which should always point to the install location of the currently-active version of Myst. This is both simpler in terms of the implementation, and more intuitive to reason about
This should correspond with changes coming in mtenv that will set MYST_HOME in the shim before running myst.
For now, users who do not use mtenv will have to set this manually, though we can probably check and ensure it is set as part of the VM setup. I think this should probably emit a warning and notify the user what the VM will use in its place, since it might not always be accurate (e.g., if a user wrongly installed myst directly into /usr/local/bin without moving the standard library as well).
Perhaps a better solution would be to disallow execution if MYST_HOME is not set?

One other thing to consider with this is that running scripts without a login shell would probably end up not setting MYST_HOME for a lot of users, at least ones who aren't using mtenv.
With mtenv, /usr/local/bin/myst is a shim that automatically sets MYST_HOME, rather than relying on the shell profile or something else to set it, so this isn't an issue.
On manual installs, even if a user sets MYST_HOME in something like ~/.bash_profile, it won't be loaded for non-login shells. Realistically, this probably won't ever be an issue, and it can be solved by editing /etc/profile or ~/.bashrc or whatever with the variable as well. The only case I can really think of is background processes/daemons, and we can always come back to address this later.

A myst daemon sounds like a terrible idea IMO. It'd be more like a myst demon :stuck_out_tongue:
As the primary goal of myst is to support users as best as possible, I think a strong competitor for how to set MYST_HOME in manual installations is the Makefile – I mean, the purpose of Makefiles is very much to aid installations.
A manual installation should be done with the Makefile, any kind of other manual installation would be little, but meaningless. This is how I imagine a manual installation:

make install MYST_HOME=/opt/myst/%v (or wherever you want it – and of course with a default value)

Where  %v gets substituted with version just because its nice. This install a myst executable to /opt/myst/<version>/bin/myst, the stdlib to /opt/myst/<version>/stdlib, and installs a shell script /usr/local/bin/myst (unless otherwise specified) that sets env MYST_HOME to what specified in the Makefile installation, and executes myst.
If not: every OS should have some way for environment variables to be set (even when not running login shells) – so worst case scenario is to force manual setting.

That sounds like a pretty solid plan to me. I certainly think it's fair to say "if you don't use mtenv or make install, it's your responsibility to set MYST_HOME before running Myst".


Lubricant

https://github.com/AxelFougues/Thermal-implant-reverse-engeneering/blob/36317fffd6df549d8704c1b0b88a50f165a2974d/README.md?plain=1#L20
This is explained here : https://forum.dangerousthings.com/t/detached-nubbin/19138/4

Note that in this case the implant is held by a dent in the needle


Memoryfilesystem

This PR does some of the work to switch from Jimfs to memoryfilesystem.
This has failing tests:

JimfsFileSystemProviderImplTest.theCreatedFileSystemSupportsDirectoryStreams:166 » IllegalState #iterator() already called. DirectoryStream#iterator() is specified to throw IllegalStateException if the iterator has already been returned
JarFactoryImplTest.exceptionRaisedIfOutputJarDirectoryDoesNotExist:76

Expecting message to be:
  "/some-non-existent-directory/HelloWorld.jar"
but was:
  "/some-non-existent-directory"

assertions on exception messages are quite brittle. It would probably be safer to use java.nio.file.FileSystemException#getFile()

FileUtilsTest.retrieveRequiredUrlCanDereferencePathsToUrls:89

Expecting actual:
  "memory:45377dad-eb8b-42ac-96df-5d3d9f18e5c9:///foo/bar/baz.txt"
to start with:
  "memory://"

The JimfsFileSystemProviderImpl* classes should probably be renamed.
I do not plan to put any more work into this PR.

Thanks for taking a look on your side and putting the work into putting this up, it is greatly appreciated! Will merge this into a feature branch on my side to have a play with getting some of those tests running.
Looks all good to me though!


Should ribbons be (x-axis) ordered automatically?

Ribbons only make sense in the context of ordered data. So something like the following should work automatically.
library(plot2)

mod = lm(mpg ~ wt, mtcars)
pred = predict(mod, interval = "confidence")

mtcars2 = cbind(mtcars, pred)

with(
  mtcars2,
  plot2(
    x = wt, y = mpg,
    ymin = lwr, ymax = upr,
    type = "ribbon"
  )
)

Created on 2023-08-07 with reprex v2.0.2

Reopening since grouped ribbons still aren't working unless manually ordered beforehand.
pred = predict(lm(mpg~wt+factor(cyl),mtcars), interval = "confidence")
m = cbind(mtcars, pred)

library(plot2)

with(
  m,
  plot2(wt, fit, ymin = lwr, ymax = upr, by = cyl, type = "ribbon")
  )

with(
  m[order(m$wt),],
  plot2(wt, fit, ymin = lwr, ymax = upr, by = cyl, type = "ribbon")
  )

Created on 2023-08-09 with reprex v2.0.2

Fixed with https://github.com/grantmcdermott/plot2/commit/b0bcb94fae07dc3ad50d4a3e24a5f96e46d65ad9
pred = predict(lm(mpg~wt+factor(cyl),mtcars), interval = "confidence")
m = cbind(mtcars, pred)

library(plot2)

with(
  m,
  plot2(wt, fit, ymin = lwr, ymax = upr, by = cyl, type = "ribbon")
  )

Created on 2023-08-09 with reprex v2.0.2


Unable to scroll in Firefox

STR:
In Firefox 57,

open http://web-confluence.appspot.com/#!/confluence
attempt to scroll down to see Edge graphs, etc.

I didn't dig in deeply to understand why... but there's an overflow: hidden rule on body:
/* Sticky Footer */
      body {
        display: flex;
        flex-direction: column;
        height: 100vh;
        width: 100vw;
	overflow: hidden;
      }

This issue also affects latest Edge. The layout pattern in effect (which maybe wrong according to specs) is as follows:

Nest everything from <body> down to scroll container with display: flex; flex-direction: column;
All ancestors of scrollable container are flex-grow: 1; flex-shrink: 1;
Others (header and footer) are flex-grow: 0; flex-shrink: 0;
Scrollable container (which differs in terms of how deep it is in DOM on front page, API Catalog, and Metrics page) has, among other things, overflow: auto;

Does Firefox (and Edge) honour prefixed flex properties only? Or honour non-prefixed differently?

In Firefox, that last scrollable container is tall enough for all its content (and overflowing its parents, but they hide overflow), so doesn't get a scrollbar.
Both Firefox and Edge support unprefixed flex.
See also #165.

Is it possible that this markup is depending on https://bugs.chromium.org/p/chromium/issues/detail?id=487302 to work?

See also https://github.com/webcompat/web-bugs/issues/10168

@bzbarsky that's entirely possible. Is there a corresponding bug for WebKit? Safari renders scrolling "correctly".

Closing this as conversation has moved to #165

Is there a corresponding bug for WebKit?

I don't know what the status of this stuff in WebKit is, sorry.  Maybe @dholbert does.

No, I don't think there is a WebKit version of the bug. Back when I filed https://bugs.chromium.org/p/chromium/issues/detail?id=487302, I suspect WebKit didn't have the magic min-width:auto behavior implemented yet (but I'm not sure).


Implement Default for Callback

This allows using #[derive(Default)] in more cases for Properties.

@davidkna You should prefer to use Option<Callback> instead of an empty callback because it's an explicit approach to get an empty default value (equals None).

In my case I was using this for required callbacks, but I can see why you prefer not to include this.


On "Receive" screen the generated address should be copiable at a long tap

After long tap on address show notification https://zpl.io/VYpkXxJ , but do not close Receive screen

@Miolin  Please, add haptic feedback

checked


Add copy button to code blocks

It would be nice to have a button to copy the entire content of code blocks, like the one supported in GitHub markdown:
pip install dvclive

Can we maybe reprioritize this and simplify? Having guides without this seems lacking
No need for fancy logic IMO - we can start with:

Asking for this explicitly (i.e. not a feature of cli). We can add a component similar to cli, but "asking for this feature"- called cli-guide or cli-with-copy to make this explicit and not break all existing snippets.
Have it copy all the contents as is, with the exception of shaving terminal-curser token (^$ to support multiple commands) which should be relatively easy I hope.

It will be possible to expand on that in the future with robust logic to differentiate between the command(s) and outputs. But we can do without it today.
Thoughts?


Add WebJars support

This adds support for WebJars. Please see the commit message for more details.
I know we're planning on releasing M8 soon, but it would be nice to get this in Lift 3.0 if possible. I'm not sure if that means it must be in M8 or not. If it's too late for 3.0, then it's ready whenever.
I created a couple of example apps that I tested with. They are here: lift-examples
This would also fix #1755
There is one small problem with the build configuration. If you run publish or publishLocal from lift-framework the package task in lift-webkit will not get triggered properly and the assets will not get included in the JAR. I tried adding a dependency to them, but that only helps when you're in the lift-webkit project. In order to publishLocal, you need to run:
./liftsh "project lift-webkit" package "project lift-framework" publishLocal

I added a similar command to travis.sh, so that should run okay there, but it'd be nice to get this figured out.

In the ML thread, I had asked about thoughts regarding combining js files into one asset.  While I agree that many may choose to combine assets statically as part of the build process, it seems we should provide a means of combining for folks opting to utilize our webjar support. Have you thought more about this?

I get a jshint problem when building locally:
jlift.js:0: Bad option: 'esversion'
This commit on my branch resolves it for me.

BTW, I notice that the code for my comments are not showing up in my view of this PR.  Is it my view, or am I doing something dorky here?

I think you clicked directly into the commit instead of clicking the "Files Changed" tab on the PR page.

You're right.  I'll do that next time :+1:

@joescii regarding combining files; I see 3 problems that need to be solved:

How to tell Lift the dependencies to combine. This would take the place of having to put the script tags in your HTML, so wouldn't be a big deal.
How to minify and combine. I don't think just combining them is enough, so I would want to minify as well. The only Java based minifier that I know of is google-closure. We could use that but it would add a dependency that some may not use. We could roll our own. The combining part should be pretty easy, but the minifying, not so much.
Testing is another concern. That's even more of an obstacle than 2 as I don't know of any Java libraries that do this.

So, I'm still not sure if that's something I'd want to tackle. But, I do understand the desire to not have to rely on build tools.
The main reasons I wanted to add WebJar support to Lift are:

To have lift.js available as a WebJar asset, so it's available to build tools.
To have an easy way to create starter and demo apps. I doubt I would use it for more than a few dependencies in a real project.

Personally, I'll stick to using gulp and grunt for now. I understand the hesitancy of having to install node, which is why I'm working on an sbt plugin that helps with that.

My initial thoughts on combining files was pretty simple, but perhaps naively so.  I figured we could just provide multiple src params in the script tag for each asset.  Combining could just be a newline between each asset.  We're already depending on the webjars to provide the minified assets, and a newline between them should be no big deal.
Note however, I'm not talking about minifying the project's js sources.  Perhaps that's what you thought I meant.

I'm closing this for now. I have some build tool improvements that I'm working on that should help simplify the JavaScript build that I'll update this with once ready.
I created #1764 to track it.


Progress bar overlay does not disappear when game window or ACT are not in focus.

Progress bar overlay does not disappear when game window or ACT are not in focus. For example, when I Alt+Tab to Firefox or Skype window, I want the overlay to disappear, and appear again when I go back to game window. Picture of the problem below:

Can confirm this is an issue, otherwise the plugin is amazing, Would love to see this fixed.


feat: Add writeClientInLib option to disable lib/prisma.ts

This addresses [#59] by exposing writeClientInLib option in the module configuration.

Thanks for opening the PR @lewebsimple 🙏 ! I left a suggestion for the option name to make it more clear to our users.


best sms (sega 8-bit) virtual button position

where's your RESET button?
It's missing

Can you hide the navigation bar while the user is playing a game?


feat: add manifests and robots.txt

Description

added the manifest.json and robots.txt in /public folder
This PR fixes #33

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@sudhay23 can you please resolve the merge conflict and please rebase the branches as the commits are piling on top on each other,

Resolved everything


Should uvx use a project dependency if available?

Right now, uvx will run in an isolated environment even if the target tool is installed in the current project. For example, uvx ruff will not use the version of Ruff declared in the project (e.g., with uv add ruff==0.3.0). If we're in a project, it might make sense to respect the version there instead of ignoring it?
This may be a reasonable way to support defining tools in projects until we design and implement "project-level tool definitions" per #3560.
Related:

https://github.com/astral-sh/uv/issues/6377
https://github.com/astral-sh/uv/issues/5903#issuecomment-2336546258
#3560

My preference would be to use uv tool run, but place symlinks for the tools in a directory that can be added to the $PATH (like pipx does).
That way, I could install poetry globally with uv tool install poetry, run it with just poetry ..., and use uvx for stuff in .venv/bin (like npx with node_modules/bin).

My preference would be to use uv tool run, but place symlinks for the tools in a directory that can be added to the $PATH

Isn't this just uv tool install?

That way, I could install ruff globally with uv tool install ruff, run it with just ruff check, and use uvx for stuff in .venv/bin (like npx with node_modules/bin).

Except npx will also run arbitrary versions without installing them, right? I'm not sure I follow.

Sorry, I'm on my phone so I can't try things. Reworded my previous comment right after posting.
Didn't know npx could run commands without installing. I don't personally use it now but did a few years ago, when npx came out it was used for easlily running installed binaries in the project.
For global installations I rather use the $PATH. I do use uv run a lot and would prefer uvx be used to shorten that. As for running without installing, could uvx try these in turn?

in .venv
in tools dir
isolated

Yep that's the suggestion this issue tracks.

Currently uvx is really just a shorthand for uv tool run but it could be a more intelligent command using context.
Meaning it will either call uv run or uv tool run as necessary. And if you actually mean one or the other, you could force it by using the explicit form. If you bake "run" inside "tool run", you lose this ability.
I don't know if that is feasible as these commands have different options. My original suggestion was to have a 3rd command but I am seeing it as more confusing to the end user than a smarter uvx

The idea here is that uv tool run would check the project for the dependency too. I don't think uvx will ever have different semantics than uv tool run (it'd be too confusing).

This actually seems to be two different questions, no?

Should uv tool run ruff use the version of ruff installed in the project, if there is one, rather than the global/tools version?
If uv tool run ruff uses a project version of ruff, should it be run in the project environment or in an isolated one?

Because so far as I can see (with my limited knowledge of the problem) it would be theoretically possible to do 1 but not 2, no? I.e. reuse the installed wheel but still run it in an isolated environment?
While I'm here, my feedback would be that this seems to me on the face of it not such a great idea. The whole "tool" interface is already a little challenging for newcomers conceptually (was for me, anyway). Assuming that both 1) and 2) are being proposed, this proposal would muddy the waters and make it even more confusing, as the behaviour of uv tool run would vary, and it might not always be clear what is being invoked. Learning and teaching becomes harder as it is no longer possible to break it down as "uv tool uses global/isolated packages and environments and uv run uses local/project packages and environments".
Sometimes it makes a big difference whether a tool-like package runs in an environment or not (e.g. pyinstaller), and then the clear delineation of the two behaviours is useful and crucial.
If anything I would say that uvx having different behaviour to uv tool run is less confusing than having uv tool run show context-dependent behaviour. But both are a little confusing.
If using the project dependency with uv tool run was implemented, I think it'd be much more user-friendly if it was off by default and turned on with e.g. --use-project rather than the other way round.

I've been using uv run lately to have tools available in the current env of a project without having it installed as a dependency. Tools such as ones that are useful once in a blue moon such as checking for licenses of libs:
uv run --with pip-licenses -- pip-licenses

This is what I would hope for uvx to be when running inside a project.
Reading the docs: Relationship to uv run
For uv run to behave like uvx, it would require --no-project and replace (optional) --from with --with.
So why can't uvx just have a --project flag and a UVX_PROJECT=true env var which would behave like uv run?
This would be very easy to implement and save us a lot of keystrokes and mental gymnastics to remember the usage.


nexthop master upmerge

This corrects problematic merges to the nexthop branch, and can supersede #3896
Since @a-canary says no actual changes have gone into the nexthop branch independent of master, we may just want to reset the nexthop branch to master and force-push it. Doing so is destructive and would be a break from how we normally handle feature branches.

[approve ci autest]

We won't be able to fix this under the restriction that we must "Rebase and merge", because the earlier commits we are trying to revert were cherry-picks, and so the hashes do not match. Therefore attempting to replay them atop master will likely result in conflicts.
We can either allow a normal merge here, rebase the branch and force push it, or try to resolve the conflicts manually from #3896.


dbeaver/dbeaver#23349 [CrateDB] ER diagrams can't be opened

Please give some explanations of your PR. Also I wonder how does it look like (where do we see this error message)?

I think the problem here is the inability to read table references. You need to hide foreign keys or find the correct way to read them.

Not, exactly true, not needed to fix data base if it has implementation issue. The idea behind, don't to throw any exceptions if you already have enough details. Instead we have to log non critical exception and proceed visialisation.

So, does ER diagrams work now? Please add description to this PR.

Yes it is working for crateDB

Verified


Double Click Issue

When I click twice quickly, the timePicker dialog comes out twice...

is there any way to use listener when the picker is closing?

@nuggy875  did you find out something, because i have the same problem


Add link to gutenbergpy project

Codecov Report

Merging #136 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #136   +/-   ##
=======================================
  Coverage   95.97%   95.97%
=======================================
  Files          29       29
  Lines        1094     1094
=======================================
  Hits         1050     1050
  Misses         44       44

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 8177652...2debbca. Read the comment docs.


Change retirement date

@yoavfrancis : Thanks for your contribution! The author, @rkarlin, has been notified to review your proposed change.

#sign-off


Fix invalid iconUrl on Safe App creation

Closes #1100
Summary
This PR fixes a bug in the SafeApp creation flow. As SafeApp is not persisted to the database yet when safe_app_icon_path is executed, it has no access to the app_id field (as it depends on the database PK). So this field cannot be used directly as part of the file URL. As a result, before this PR, None was being appended to the URL instead of the app ID.
Changes

Adds the case where app_id is not available (because safe_app_icon_path is called during a SafeApp creation).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 99.354%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
1076

Relevant Lines:
1083

💛  - Coveralls