Incorporate colors into current scripts

Merge #432 before this.
This PR is an extension of #432
After merging #432 there are going to be maybee 1 - 3 conflicts with this PR.
Gaming-setup was skipped due to a complete-rewrite in #380
Colors were only added in the most necessary places.
Type of Change

[x] New feature
[x] Refactoring
[x] UI/UX improvement

Description
Scripts actually use colors now. Before, a lot of scripts were lacking where colors were needed; now they're not.
Checklist

[x] My code adheres to the coding and style guidelines of the project.
[x] I have performed a self-review of my own code.
[x] My changes generate no errors/warnings/merge conflicts.

@ChrisTitusTech You messed up a function name in kitty-setup, I'll fix it after you resolve conflicts, don't merge this yet!!


Wrong year reported

The number of years between a date and the same date the year before plus one day should be 0 but it's reported as 1.
require "rubygems"
require "time_difference"

d1 = Date.new(2014, 1, 20)
d2 = Date.new(2013, 1, 20)
td2 = TimeDifference.between(d1.to_time, d2.to_time) # expected 1 year

d3 = Date.new(2013, 1, 19)
td3 = TimeDifference.between(d1.to_time, d3.to_time) # expected 1 year

d4 = Date.new(2013, 1, 21)
td4 = TimeDifference.between(d1.to_time, d4.to_time) # expected 0 years

p td2.in_years == 1.0
p td3.in_years == 1.0
p td4.in_years == 0.0

p td4.in_each_component

Output
true
true
false
{:years=>1.0, :months=>11.97, :weeks=>52.0, :days=>364.0, :hours=>8736.0, :minutes=>524160.0, :seconds=>31449600.0}

years == 1.0 and days == 364.0 seem contradictory.

simply add :
elsif time_component == :years
diff = @time_diff / 31536000.0 #(365 * 24 * 60 * 60)
instead of * 1.year (= 365.25 day)

It can be quite difficult to depend on 365 days being 1 year for this case
https://www.wolframalpha.com/input/?i=2013%2F1%2F21+to+2014%2F1%2F20

More than one year passed since I opened this issue. I had to compute the age of a person and this is how I solved it.
def age(birth_date, now)
  return 0 if birth_date >= now
  years = now.year - birth_date.year
  if now.month < birth_date.month
    years -= 1
  elsif now.month == birth_date.month
    if now.day < birth_date.day
      years -= 1
    end
  end
  years
end

Run irb,  copy that code into it and
2.2.1 :013 > require "date"
 => true
2.2.1 :014 > age(Date.new(2013, 1, 21), Date.new(2014, 1, 20))
 => 0
2.2.1 :015 > age(Date.new(2013, 1, 20), Date.new(2014, 1, 20))
 => 1

This is a much less general case than what time_difference has to do but maybe it can hint to a way to fix it.

If there is a need to implement custom calculations like what you, @crazyjin, and @Argorate suggested, would prefer to have it as a separate configuration. Would still like to have the time calculation result to match what Wolfram Alpha is returning.


Change default thickness from 4px to 4dp

Pixels are of no use, because devices have different density. There's a huge difference between xxxhdpi and mdpi.

Fixed in 73cee9ca1e48c428a888130b110b5e09979b38f4


MOB-24: Like Functionality

What's up?
We need to make like button interactive.
To Do

[x] Make like button interactive
[x] Change like icon when pressed

Deadline
26.12.2022
Additional Information
No response
Reviewers
@ooodogodogodogo @MustafaEmreErengul

@ooodogodogodogo changes made and pushed. Waiting for review

Reviewed. It works as expected. Great!


[Feature] wss protocol in CLI

Motivation
The CLI does not support wss protocol while the desktop does.
Detailed design
The CLI conn/pub/sub should support this syntax:
mqttx conn -h 'my.server.com' --protocol 'wss'  --path '/mqtt' -p 443

Options to deal with SSL/TLS are also required : --ssl-tls and --ssl-secure
Alternatives
The desktop version can be used as an alternative.
But it is not confortable in some cases : Like debugging a MQTT service in a kubernetes cluster where the service is only reachable with a SSH console.
More detail (optional)
Please note the current version 1.8.3 clearly states the wss protocol is not supported:
 Only mqtt and mqtts are supported.

Thank you for your feedback, the MQTT X CLI does not support the ws and wss protocols, this is a limitation due to the different environment and we cannot support it.
You can use --insecure to not validate the server certificate.

Thanks for your feedback, if you want to use ws or wss protocol to connect, you can choose the MQTT X Desktop or MQTT X web. I will close this issue.


Is there any way to ignore routes?

I have installed flutter web app azure server and also i have installed wordpress in sub folder. When i use this plugin package detects it as url because it is installed in subfolder.
https://www.exampleflutterdomain.com
https://www.exampleflutterdomain.com/wordpress/
with this installations it is trying to open url as a flutter path.
Is there anyway ignoring urls ?

Hi,
Is that what you are looking for: https://flutter.dev/docs/development/ui/navigation/url-strategies#hosting-a-flutter-app-at-a-non-root-location?

I'm talking about the last part where you change the href in the index.html.
In your case you would have <base href="/wordpress">

@lulupointu yes i understood that. But as i said flutter app installed base folder but wordpress is subfolder. I want to show flutter app as a first.

I'm sorry but I'm not sure I understood what you are trying to do. I don't understand what is the link between your wordpress app and your flutter app. Could you try explaining differently ?

@lulupointu
Let me explain in a little more detail. Flutter web project and wordpress are kept on the same hosting. Flutter project is in the root folder while worpdress is in the same directory in a separate folder name.
The problem is that when I try to access http://www.domain.com/wordpress/wp-json/ it redirects to the home page of the flutter web page because the vrouter can't find it as a route.
because of this, i think i need to edit below lines.
VRouteRedirector(
redirectTo: AppRoutes.home,
path: r':_(.)', // . is a regexp which matching every paths
)

Do you want http://www.domain.com/wordpress/wp-json/ to be considered as part of your flutter app or your wordpress ?

Do you want http://www.domain.com/wordpress/wp-json/ to be considered as part of your flutter app or your wordpress ?

Not exactly, I actually use wordpress as a backend. But since Flutter sees it as url path, it can't find this path and redirects immediately. Therefore, I cannot access the wordpress json api.

@lulupointu what I actually want is that it does not take all the paths that start with wordpress and continue and do not redirect to flutter home page.

If you use toExternal instead of to, and your server is configured properly, I think you should be able to make it work

I found another way, thanks.

Could you share you solution?


iFrame Snapshot

We have an intelligence portal which is very expensive to run, so we turn on the server only 2 or 3 times a week.  This information should be displayed in the website as an iFrame.  However, since we turn off the server, the information is only available when the server is turned on.  So we need to integrate a solution into the webserver to create a snapshot (screenshot) of an iFrame.  Can you please do some research and see how to integrate a function like this?

Maybe check this out...
https://stackoverflow.com/questions/55389292/how-to-capture-an-iframe-as-an-image/55389590
@usman-asrg

@heldyboy  When this server will be activated and can you share the URL or details so I can check where it is and find the exact solution for this ?

We can setup anytime. We just need to plan it.
The website is:
https://www.asrg.io/knowledge/intelligence/
The iframe is:

The decision is we'll update image every week on Sunday at 22:00 CEST? We have uploaded the latest iFrame image that is live on the intelligence page

Please add code to update every week at Sunday 22:00 CEST.

The Servers in Azure will be turned on at 9pm on Sunday, and turned off at 10:30pm on Sundays

The Updated IFRAME Code is

Issue with the ELK server start up.  The elastic service does not start correctly.  Need to investigate.


v2.1.0 (Fermion) Roadmap

[ ] #40: Some parts of the toolkit need to be reworked in order to utilize ARIA roles instead of the custom solutions implemented until now.
[ ] #41: A pure CSS tootip can be created and could fit in the contextual module. However, if the size is rather large and bloats the module, a separate repository should be created to ship it with certain flavors.
[ ] #42: An extension class for the <footer> element should be implemented to address the issue.
[ ] Look into #50, #51 and #21 in order to update the input_control module. Some or all of these issues might not be addressed in this update.
[ ] Look into #26 and maybe implement an extension class for the <header> element to address the issue.
[ ] #53: Add a flag to enable the functionality described in the issue and create certain classes that will only be available if the flag is set to true in order to add this functionality to the grid system.
[ ] Update documentation for said changes, especially the customization.html page, along with the module pages of the changed modules.
[ ] Create new flavor, mini-classic, address #45 by using Bootstrap-like naming.
[ ] Document mini-classic flavor in flavors.html, add codepen demo.
[ ] Update documentation pages to use the v2.1.0 version of the toolkit's mini-default flavor.
[ ] Update all links and embedding instructions to mirror the latest v2.1.0 version of the toolkit.
[ ] Recalculate size of toolkit, update pages accordingly.
[ ] Update README, CHANGELOG etc.

Estimated release date: Late February, 2017.

Update
Lots of new minor changes, additions and features have just been added to the list. Hopefully, I will be able to add all of these before the end of February, but the release might get pushed back a couple of days.

Update
Most of the work on v2.1.0 has been completed. Flavor updates are the only thing remaining to be implemented until release.
Estimated release date: 21st February 2017

Update
The promised mini-classic flavor will not ship with v2.1.0 or in fact any other release. This decision was made due to three key reasons:

The mini-classic flavor would just be a worse version of the mini-default flavor, the only difference being some minor design decisions and naming conventions.
The mini-classic flavor would look far too much like a Bootstrap template or the way Bulma looks, which is pointless, due to the fact mini.css is not trying to imitate those frameworks, but rather provide a decent alternative to them. Therefore, copying their style would not be appropriate.
Finally, the lack of manpower on the mini.css team (I work on it alone) limits the possibility that many flavors will be supported officially and maintained properly, thus I will try to stick to the mini-default and mini-sucroa flavors, which are both properly maintained. If I somehow find a way to maintain more flavors, I will reconsider the possibility of implementing mini-classic, however I will most likely pursue the addition of new, more stylish and varied flavors instead.

Apart from that, most of the other things in the roadmap have been implemented, so v2.1.0 should be released very soon!


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 33cb16b:
:clock10: Pending: waiting for processors (50 builds ahead of you)
:warning: Docs Build is busy, currently there are 50 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 33cb16b:
:clock10: Full build: the average full build time is 6 min(s) 5 sec(s), this is based on the last 495 build(s) for this repository.

Docs Build status updates of commit 33cb16b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docfx.json
:white_check_mark:Succeeded

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


Probably wrong compilation infos output order

Describe the bug
When compiling, the information lines seems to be written in the wrong order (we go i.e. from 7% to 4% and then to 6%, etc.).

Expected behavior
We should only progress from 0% to 100% without going back.
Error output
N/A
Related Environment
Please provide compiling and running environment information:

xmake version: v2.3.9+202011250023
os: Windows 10 1909
target platform: Windows

Additional context
I encountered this behavior once before, and I was able to reproduce it just by recompiling it from scratch (after having deleted the .xmake folder).
Maybe you are using a cache system for the output and the order is not conserved. I don't know, I did not read the code.

This is caused by a large number of parallel tasks, and I knew this problem before. But since this is not a big problem, it is just some display disorder, so I did not modify it before. When I have time, I will try to improve it.

I have improved it on dev branch. you can try it again. xmake update -s dev


Update minikube.md

Extended the instruction a bit  by adding "If you are not using any registry, you have to add a version to your docker image, something other than 'latest'."

This change is

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

docs/getting-started-guides/minikube.md, line 132 at r1 (raw file):

Minikube contains a built-in Docker daemon for running containers.
If you use another Docker daemon for building your containers, you will have to publish them to a registry before minikube can pull them.
You can use minikube's built in Docker daemon to avoid this extra step of pushing your images. If you are not using any registry, you have to add a version to your docker image, something other than 'latest'.

Could you expand a bit more? I'm not sure what you mean here. Is this related to the default PullPolicy? What was in your yaml?

Comments from Reviewable

@dlorenc Thanks for the review. I updated to make it a bit more clear.

I signed it!

@dlorenc I hope, its a bit clear now, please let me know if you need any more details

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

LGTM

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

I signed it!

@dlorenc This is my first pull request. To get this change added to documentation, is there any action item from my side pending? Thanks.

@devin-donnelly changed as you asked. Please take a look when you get a chance. Thanks.

created another PR after making changes as per review comments. Closing this one


Enhancement: Acceptance of Terms and Privacy Policy on First Login

What features would you like to see added?
It would be beneficial to introduce an intermediate step upon the first login where users must accept the Terms of Service and Privacy Policy before they can use the service. This acceptance should be stored in the database. Also, the feature should be configurable via variables in the configuration file to allow optional activation.
More details

Without accepting the Terms of Service and Privacy Policy, the user should not be able to access the main functionalities of the service.
When a user account is deleted, the related acceptance entries must also be removed from the database.
This feature would ensure that all users actively accept the Terms of Service and Privacy Policy, enhancing both compliance and user awareness.

Which components are impacted by your request?
General, UI
Pictures
No response
Code of Conduct

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

I've actually done that as well, I made a new react component based on DialogTemplate that shows the modal - but I didn't save it in the database because I didn't have time to create a new route to store it there, so I went with the localStorage.
This is what it looks like. I am not gonna send a pull request until I can add the db storage though, doesn't feel complete.

I actually implemented this myself, but I don't feel ready to send a pull request because it's only using LocalStorage rather than the database as I didn't have time to add the routes to store it in Mongo. It works well though, if you don't accept it logs you out, otherwise you get to use it (until you change browser or device).

Maybe we can make this a more general feature to have popups on login/first time use/etc.?

Aweseome @maxesse
@danny-avila a more general approach would be nice. Maybe it helps to have a first impression from @maxesse solution?

Next week I'm gonna tidy it up, put the text in some .env variables, and see if I can add the route to store the acceptance in the db, and I'll do a pull request if you want. If @danny-avila prefers to take it over I can also just paste here the component.

Awesome! Thank you all.
@maxesse @danny-avila  Maybe i am wrong but the buttons in the modal are only colored while hover. Maybe samething like this?
diff --git a/client/src/components/ui/TermsAndConditionsModal.tsx b/client/src/components/ui/TermsAndConditionsModal.tsx
index c9bc7912..a15c41e2 100644
--- a/client/src/components/ui/TermsAndConditionsModal.tsx
+++ b/client/src/components/ui/TermsAndConditionsModal.tsx
@@ -72,13 +72,13 @@ const TermsAndConditionsModal = ({
           <>
             <button
               onClick={handleDecline}
-              className="border-border-none bg-surface-500 dark:hover:bg-surface-600 inline-flex h-10 items-center justify-center rounded-lg px-4 py-2 text-sm text-white hover:bg-gray-600"
+              className="border-border-none inline-flex h-10 items-center justify-center rounded-lg px-4 py-2 text-sm text-white hover:bg-gray-700 bg-gray-500 dark:border-gray-600"
             >
               {localize('com_ui_decline')}
             </button>
             <button
               onClick={handleAccept}
-              className="border-border-none bg-surface-500 inline-flex h-10 items-center justify-center rounded-lg px-4 py-2 text-sm text-white hover:bg-green-600 dark:hover:bg-green-600"
+              className="border-border-none inline-flex h-10 items-center justify-center rounded-lg px-4 py-2 text-sm text-white hover:bg-green-700 dark:hover:bg-green-700 bg-green-500
"
             >
               {localize('com_ui_accept')}
             </button>

You're right - I should have tested it in light mode, as I always use it in dark mode and I didn't spot the issue. @danny-avila are you happy to apply @derhelge fix or do you want a PR? It works fine with his change.


Missing package on npm

Hi @jonathantneal just to let you know that postcss-register-property is missing on npm and it returns 404.

I was waiting for the spec to be real.

Ok this make sense, but don't you think is better add a disclaimer to tell this? :)

Well, nobody noticed this repository for months until I tweeted it a half hour ago, and I’ve published it, so I’m okay with the 30 minutes of ambiguity. 😄

I knew this repo and i tweeted about it time ago :) but never tried it until now.

@jonathantneal Do you know where to fine all the syntax types?


OFFSET should be allowed to be 0

We're programmatically generating queries similar to SHOW SERIES LIMIT 10 OFFSET 10; our code wants the first page to be SHOW SERIES LIMIT 10 OFFSET 0, but currently that query returns an error like "error parsing query: OFFSET must be > 0 at line 1, char 90".

Great! I've confirmed the fix.


Sync published with master

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit 75a3f35a3761ec003bf048e104d951e253ea0e0d
https://deploy-preview-8442--docsdocker.netlify.com


fix zap tests failing on all PRs

follow up to #145
ZAP tests are failing on all PRs because they haven't been changed yet to run in the monorepo
these tests shouldn't be running yet and, when they do, should only be triggered by the centralized test.yml workflow

this fix is moot since we merged #126
no urgency to change workflow_dispatch to workflow_call on main


Get the bounding box of all nodes?

I really want to find a way to scale and center my graph so that all nodes fit the svg area. I can't seem to find a way to get the bounding box of all nodes. I've tried things like svg.select("g.node").getBBox() but that function is not available on the group.
I want to use this data to do a "zoom to bounding box" like here
Any thoughts on this?

Any update on this issue. I came across the same. Can we limit zoom scale to a particular range?


Taking flash-attn out of deps

I've noticed that the flash-attn package has a lot of installation issues. It seems like unsloth doesn't actually need the package, so I recommend removing it from the README. We spent a a bit trying to get flash-attn running in a Docker container and ended up realizing that we don't even need it for unsloth.

Fair point! It's mainly torch 2.3 causing issues


Added Olajide Ayeola's name

Added Olajide Ayeola's name to the name.txt file

Hello Olajide!
Your PR is about to be merged and you will be able to see your name on this url.
To enable me to do this, READ the README again carefully, update your code with the latest and push the update. I am here for you for the next 6 hours before I close the PR so you can initiate a new one which will be easier to merge.

Sorry I have to close this PR, kindly update your code and initiate a fresh PR
Cheers 🥂


Autofocus on 2nd field not applying, and field.touched is true, with initialValues

Hi, thanks for creating Redux-Form. Overall I'm sure it is saving lots of dev time.
I'm seeing unexpected interactions between initialValues, field.touched, and autofocus.
I have the first field ("subject") populated from initalValues, and the second field ("emailBody") set to autofocus.
There are two problems:

When the form loads, the Subject field has focus even though it is NOT set to autofocus.
emailBody.touched is set to true even though the user has not interacted with the form at all. This means the validation error is displayed right when the form opens up.

My guess is that it happens like this:

Form loads
Autofocus happens, sets focus to emailBody field
Redux-form populates subject field, inadvertently setting focus to it and causing a blur event on the emailBody field

Can you recommend a workaround?
The form:
const { circleName,
    fields:{ subject, emailBody },
    error,
    handleSubmit,
    mySubmitHandler,
    submitting,
    onCancel,
    ...otherprops } = this.props;
<form onSubmit={ handleSubmit((values)=>{mySubmitHandler(values, otherprops);}) }>
        <input className="input-field" type="text" name="subject" placeholder="Message subject" {...subject}/>

        <textarea autoFocus name="emailBody" placeholder="Type your message here"
          rows="15" className="input-field"
          {...emailBody}/>

        { (emailBody.touched && error) &&
{ error }
 }

        <button name="send-message" className="submit-button">Send Message</button>

        <Link to="/circles" className="modal-cancel-link">Cancel</Link>
      </form>

The container:
function mapStateToProps(state, ownProps)
{
    return {
        initialValues: {'subject': 'A message from someone'},
    };
}

function mapDispatchToProps(dispatch, ownProps)
{
  return {
    mySubmitHandler: (values, props) =>{
      const { user } = props;
      dispatch(sendEmail(values, user.id));
    }
  };
}

function clientValidate(values)
{
  const { subject, emailBody } = values;
  console.log("Client validate message");
  if (!subject || !emailBody)
  {
    console.log("Client validation failed.");
    return {
      subject: "Enter a subject",
      description: "Enter a message",
      _error: "Enter a subject and a message."
    };
  }
}

const fields = ['subject', 'emailBody'];
const MessageContainer = reduxForm ({
  form: 'message_create',
  fields,
  submitting: false,
  validate: clientValidate,
}, mapStateToProps, mapDispatchToProps)(Circle_Message);

same issue
autoFocus attribute don't work with redux-form

+1 same issue

+1, autoFocus doesn't work with (at least an older) version either, currently using 5.3.3.
I'm also using a custom <InputField> component which renders, among other stuff, an ordinary <input> tag. Passing autoFocus via props (<input autoFocus={this.props.autoFocus}>) or adding autoFocus, i.e. <input autoFocus> doesn't yield any results.
The rendered <input> only contains other props such as required, title etc. Other bool-based props such as readOnly work.
Interestingly enough, passing some other obscure HTML form props, such as formMethod: "foo", results in the attributes getting attached to the <input>.

+1 please pass all of the correct attributes to native html form elements
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input

Closing with bulk close of v5 issues. If this issue still requires action and/or also affects v6/7, please request a reopen with details or raise a new issue. v5 PR's are still welcomed.


fix es6 class method

@facebook-github-bot shipit

@facebook-github-bot shipit

Thanks. I think I don't have the access
Best,
Matt

@knowbody Nah, the bot is acting weird for past few days.

merged


[Bash] Prefer ${variable//search/replace} over sed

As proposed by static analysis tool:
https://github.com/koalaman/shellcheck/wiki/SC2001

Can one of the admins verify this patch?

Closing these all out. Mind making all these changes in one pull request? It's easier to run through tests that way.


Comments should not be included in static/index.html

🐞 Bug report
Description
The static, generated index.html file should not include comments for styles, but it currently concats all of those comments together at the top of the inlined styles.
🔬 Minimal Reproduction

Add comments in multiple Sass partials and in your styles.scss file and then build and run Scully.
Check your dist/static/index.html output for comments

💻Your Environment
Angular Version:

Angular CLI: 9.0.0-rc.6
Node: 12.13.0
OS: darwin x64

Angular: 9.0.0-rc.6
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router, service-worker
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.900.0-rc.6
@angular-devkit/build-angular     0.900.0-rc.6
@angular-devkit/build-optimizer   0.900.0-rc.6
@angular-devkit/build-webpack     0.900.0-rc.6
@angular-devkit/core              9.0.0-rc.6
@angular-devkit/schematics        9.0.0-rc.6
@angular/cdk                      9.0.0-rc.5
@angular/fire                     5.2.3
@angular/material                 9.0.0-rc.5
@ngtools/webpack                  9.0.0-rc.6
@schematics/angular               9.0.0-rc.6
@schematics/update                0.900.0-rc.6
rxjs                              6.5.3
typescript                        3.6.4
webpack                           4.41.2

Scully Version:

    "@scullyio/init": "0.0.8",
    "@scullyio/ng-lib": "latest",
    "@scullyio/scully": "latest",

🔥 Exception or Error
I see this kind of output
<style>/*!
 * Copyright 2013-2019 DevIntent - All Rights Reserved
 *
 * Unauthorized copying of this file, via any medium is strictly prohibited
 * without the express permission of DevIntent.
 *
 * No warranty, explicit or implicit, provided.
 * In no event shall DevIntent be liable for any claim or damages related to this code.
 */
/*!
 * Copyright 2013-2019 DevIntent - All Rights Reserved
 *
 * Unauthorized copying of this file, via any medium is strictly prohibited
 * without the express permission of DevIntent.
 *
 * No warranty, explicit or implicit, provided.
 * In no event shall DevIntent be liable for any claim or damages related to this code.
 */
/* Theme for the ripple elements.*/
/* stylelint-disable material/no-prefixes */
/* stylelint-enable */
/*!
 * Copyright 2013-2019 DevIntent - All Rights Reserved
 *
 * Unauthorized copying of this file, via any medium is strictly prohibited
 * without the express permission of DevIntent.
 *
 * No warranty, explicit or implicit, provided.
 * In no event shall DevIntent be liable for any claim or damages related to this code.
 */
/*!
 * Copyright 2013-2019 DevIntent - All Rights Reserved
 *
 * Unauthorized copying of this file, via any medium is strictly prohibited
 * without the express permission of DevIntent.
 *
 * No warranty, explicit or implicit, provided.
 * In no event shall DevIntent be liable for any claim or damages related to this code.
 */
/* Theme for the ripple elements.*/
/* stylelint-disable material/no-prefixes */
/* stylelint-enable */
/* app drawer breakpoint */
.mat-badge-content {
  font-weight: 600;
  font-size: 12px;
  font-family: Roboto, "Helvetica Neue", sans-serif;
}
...

You can see that this includes Copyrights, stylelint overrides, and style comments.
I've figured this out, but I'm still going to open this so that others who run into this can find the solution.

Run ng build --prod and the Angular CLI will strip out your Sass comments. Then you can run npm run scully and they won't be included in your build.


Update docker

This PR:

Adds license headers to all files
Removes wpantund and NCP firmware from the docker image (now we use ot-br-posix in all examples, let's remove confusion)
Updates nRF Command Line Tools version and broken DTS link
Adds simple CI check for building the docker using GitHub workflow

CI Build seems to pass: https://github.com/LuDuda/nrfconnect-chip-docker/runs/1763012089
We should build and release it just after west 0.9.0 is available.

CI Build seems to pass: https://github.com/LuDuda/nrfconnect-chip-docker/runs/1763012089
We should build and release it just after west 0.9.0 is available.


Release 0.12

Closes #1104
This is a PR drafting the release of pyo3 0.12! 🎉
I've bumped the version numbers and edited the contents of the changelog to be as consistent and simple as I could make it.
I propose the following release text (links don't work yet as they point to the 0.12 guide). Please post any suggested changes.
If I don't hear reasons otherwise, I plan to put this release live on Saturday evening. (12th September)

This release has seen a few careful revisions to the PyO3 API with the goal of making it easier to learn and use. The PyErr type has been reworked to implement std::error::Error.  The FromPy trait is removed. The PyObject struct is now just a simple type alias to Py<PyAny>.
Also added is a new #[derive(FromPyObject)] macro, which enables a convenient way to accept arguments of the Python "type" Union.  (See the guide entry on this new feature.)
There have been many other improvements and bugfixes too numerous to go into detail here. For the full list, see the CHANGELOG.

Many thanks to you all who have helped design and implement this release!

got one more fix coming...

@davidhewitt
Thank you!


Notification when a News is created

Describe the solution you'd like
When a news is created on an API, add an option to notify by mail all teams members which have an API key subscription for this API.

FIX by 2a566e4e83b40d96ca93846b871d6975e6d1cdd2


[d3d9] Port Royale 3 hangs when transitioning to city view

The game hangs (background music still plays and the "loading" ship wheel still turns, yet the city view never renders on screen) while transitioning from map view to city view. Occasionally I have been able to get a city view after multiple tries, but hangs are the standard behavior. WineD3D works just fine. I've captured a trace with 3 transitions, but annoyingly it worked just fine (without any hangs) on replay with DXVK. I was able to reproduce the hangs both on an Nvidia card and an Intel iGPU. They only show up when using DXVK.
Here's a screen capture.
Initially I suspected a memory allocation issue and tried enabling LAA and setting various d3d9 memory limits without any results.
Hopefully you can figure out what's going on here, since I'm out of ideas.
Also, as far as I can tell, this is not a regression since I noticed the same problem a while ago with DXVK 1.8.1, but did not have the time to look into it until now.
Software information
Port Royale 3
System information

GPU: GTX 1080
Driver: 465.31
Wine version: 6.12 Staging
DXVK version: 1.9-f903400

Apitrace file(s)
PortRoyale3.trace.tar.xz
Log files

d3d9.log: PortRoyale3_d3d9.log
d3d11.log: N/A
dxgi.log: N/A

Can confirm on R9 380 mesa git.
Made a PR

Can confirm on R9 380 mesa git.

I was actually hoping someone would confirm this on AMD last year when I reported the issue (and the fix), but better late than never. Thanks for testing this and for the PR!

@WinterSnowfall Can you quickly try something? I'm running into something weird with this issue.
Can you try to use this version with the game without the conf option and see if it still happens?
It's the 1.10.1 version that comes along with steam proton 7.0-2rc, which for some reason i cannot reproduce it with. Only the versions i download from the github i can reproduce the the issue with.
dxvk-1.10.1-proton.tar.gz

I'm running into something weird with this issue.

I'm not sure what's going on here either, but I am seeing the same thing. Can't get the game to lock up on transition with the proton version you provided, however with the release version of dxvk (taken from this github) it locks up immediately...

For more info see discussion in PR https://github.com/doitsujin/dxvk/pull/2610


Slack link on web page is broken

Currently, the join Slack link on the homepage is leading to a page that can't be loaded. I suspect the server behind https://slack.kitura.dev/ is currently down?

Thanks for letting me know!  I'll update when it's fixed.

@cameronbroe please check the link now.

The link no longer goes to an error page, but it does seem I am unable to create a new account for that Slack workspace though.

The link seems to be working for me.


The title of the page is misleading. It should say "Copy ... using Data Factory" not "Copy .... using Powershell".

PowerShell is purely used to setup the pipeline.

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

ID: d8c61c14-edcc-f52e-34c4-bde2ea84e81b
Version Independent ID: 43b15730-c822-87f7-c824-f89392f96926
Content: Incrementally copy a table using PowerShell - Azure Data Factory
Content Source: articles/data-factory/tutorial-incremental-copy-powershell.md
Service: data-factory
Sub-service: tutorials
GitHub Login: @dearandyxu
Microsoft Alias: yexu

@forsajt
Thanks for your feedback! We will investigate and update as appropriate.

@forsajt
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


Add Scope: Machine to PowerShell <IP_ADDRESS> installer manifest

Resolves #95172

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

@mdanish-kh I added Machine: scope. It wasn't intentional to remove, we just used the default option. Someone installing with User scope would still be able to do so with the scope listed as machine now, is that correct?

@anamnavi If the installer supports dual context installation, then there should be separate InstallerSwitches present in the manifest for each scope informing WinGet how to carry out the installation. I'll use Oh-my-posh as an example manifest:
https://github.com/microsoft/winget-pkgs/blob/a0fd41b1ad6c8c14f87f5d337b468562a30ecb9d/manifests/j/JanDeDobbeleer/OhMyPosh/14.0.1/JanDeDobbeleer.OhMyPosh.installer.yaml#L15-L26
A quick google search on the MSI Installer Switches showed that ALLUSERS=2 MSIINSTALLPERUSER=1 need to be passed to the installer at install-time for a per-user installation (Ref: MSIINSTALLPERUSER property). I checked in a fresh VM that silent installation with these switches fails with no installation. If I don't provide the silent switches and go for an interactive installation I get this error:

I'm not an expert, but this leads me to believe that PowerShell installer supports only per-machine installation.

@wingetbot run

A better example would be Microsoft.RemoteDesktopClient which is also an MSI based installer and works with both winget install Microsoft.RemoteDesktopClient --scope machine and winget install Microsoft.RemoteDesktopClient --scope user as separate switches for it are present in the manifest and the installer seems to support them as well as I've manually verified the behavior in a VM
https://github.com/microsoft/winget-pkgs/blob/d5b6185dca3e271f7242773e5bbb696c255faf6d/manifests/m/Microsoft/RemoteDesktopClient/1.2.3497.0/Microsoft.RemoteDesktopClient.installer.yaml#L16-L32

@mdanish-kh thanks this explains it well! I checked with my team and we don't support user-scope so just the Scope: machine being specified should suffice

@mdanish-kh any idea why the internal error may be occurring? thanks :)

@stephengillie would know more, but it's probably because the pipelines were updated recently to add validation for .zip packages so we may see these hiccups. Would also be the reason why my moderator powers are not working today to approve this PR. 😅


Enyo2 lists do not scroll in the Pre3 web browser

The list example at http://enyojs.com/samples/list/ does not scroll in the Pre3 web browser.

Closing out - issue quite old and not a priority to retest and assess current behavior.


Simplify ldap user directory implementation

The previous user directory implementation was reimplementing parts of the OpencastLdapAuthoritiesPopulator.
This PR removes the same logic from the user directory and uses the OpencastLdapAuthoritiesPopulator instead.
We only use LDAP for authorization, so I could only test it with Shibboleth for authentication. The behavior did not change for us.
This PR supersede #4377
Follow up:
Reusing the implementation throws an IllegalStateException caused by a recursive load under certain circumstances. This is caused by enriching the LDAP roles with opencast-managed roles.
AuthoritiesPopulator.getGrantedAuthorities->SecurityService.getUser->UserAndRoleDirectory.loadUser->AuthortitiesPopulator.getGrantedAuthorities
Caused by: java.lang.IllegalStateException: Recursive load of: (mh_default_org,test)
        at com.google.common.base.Preconditions.checkState(Preconditions.java:591) ~[!/:?]
        at com.google.common.cache.LocalCache$Segment.waitForLoadingValue(LocalCache.java:2172) ~[!/:?]
        at com.google.common.cache.LocalCache$Segment.lockedGetOrLoad(LocalCache.java:2162) ~[!/:?]
        at com.google.common.cache.LocalCache$Segment.get(LocalCache.java:2045) ~[!/:?]
        at com.google.common.cache.LocalCache.get(LocalCache.java:3962) ~[!/:?]
        at com.google.common.cache.LocalCache.getOrLoad(LocalCache.java:3985) ~[!/:?]
        at com.google.common.cache.LocalCache$LocalLoadingCache.get(LocalCache.java:4946) ~[!/:?]
        at com.google.common.cache.LocalCache$LocalLoadingCache.getUnchecked(LocalCache.java:4952) ~[!/:?]
        at org.opencastproject.userdirectory.UserAndRoleDirectoryServiceImpl.loadUser(UserAndRoleDirectoryServiceImpl.java:265) ~[?:?]
        at org.opencastproject.kernel.security.SecurityServiceSpringImpl.getUser(SecurityServiceSpringImpl.java:127) ~[?:?]
        at org.opencastproject.userdirectory.ldap.OpencastLdapAuthoritiesPopulator.getGrantedAuthorities(OpencastLdapAuthoritiesPopulator.java:274) ~[?:?]
        at org.springframework.security.ldap.userdetails.LdapUserDetailsService.loadUserByUsername(LdapUserDetailsService.java:40) ~[?:?]
        at org.opencastproject.userdirectory.ldap.LdapUserProviderInstance.loadUserFromLdap(LdapUserProviderInstance.java:252) ~[?:?]
        at org.opencastproject.userdirectory.ldap.LdapUserProviderInstance$1.load(LdapUserProviderInstance.java:164) ~[?:?]
        at org.opencastproject.userdirectory.ldap.LdapUserProviderInstance$1.load(LdapUserProviderInstance.java:161) ~[?:?]
        at com.google.common.cache.LocalCache$LoadingValueReference.loadFuture(LocalCache.java:3529) ~[!/:?]
        at com.google.common.cache.LocalCache$Segment.loadSync(LocalCache.java:2278) ~[!/:?]
        at com.google.common.cache.LocalCache$Segment.lockedGetOrLoad(LocalCache.java:2155) ~[!/:?]
        at com.google.common.cache.LocalCache$Segment.get(LocalCache.java:2045) ~[!/:?]
        ... 121 more

A solution is to implement an UserDetailsContextMapper and return the UserDetails object retrieved via the UserDetailsService.

[ ] have a concise title
[ ] close an accompanying issue if one exists
[ ] be against the correct branch
[ ] include migration scripts and documentation, if appropriate
[ ] pass automated tests
[ ] have a clean commit history
[ ] have proper commit messages (title and body) for all commits
Closes  #4377

I would be glad to get feedback as you developed/tested the last LDAP PR.

We are testing right now in our dev cluster, right now seems to work very well


fix warning from new react 18

Fixes #34

Warning: ReactDOM.render is no longer supported in React 18. Use createRoot instead. Until you switch to the new API, your app will behave as if it's running React 17. Learn more: https://reactjs.org/link/switch-to-createroot.

@jeffposnick Could you review and merge this one?

@jeffposnick I just pushed a new update as you requested.

This is now part of the v2.0.0 release on npm.


Monitoring error. Any stderr from virsh domblkstat command should be filtered.

Bug Report
Version of OpenNebula

[ ] 5.2.2
[ ] 5.4.0
[ ] 5.4.1
[ ] 5.4.2
[X] 5.4.3
[ ] Development build

Component

[ ] Authorization (LDAP, x509 certs...)
[ ] Command Line Interface (CLI)
[ ] Contextualization
[ ] Documentation
[X] KVM
[ ] Networking
[ ] Orchestration (OpenNebula Flow)
[ ] Sunstone
[ ] Storage
[ ] Scheduler
[ ] vCenter

Description
Expected Behavior
Monitoring script should return clean information for each host.
And even virsh have some stderr stream it should be filtered.
Actual Behavior
All stderr returned with monitoring information so opennebula can not parse this outuput and we have errors like that:
Sat Nov 25 20:45:57 2017 [Z0][ONE][E]: Error parsing host information: syntax error, unexpected VARIABLE, expecting EQUAL or EQUAL_EMPTY at line 1, columns 5:11. Monitoring information:
error: internal error: cannot find statistics for device 'ide0-0-1'
ARCH=x86_64
MODELNAME="Intel(R) Xeon(R) CPU E3-1284L v4 @ 2.90GHz"
HYPERVISOR=kvm
TOTALCPU=800
...

How to reproduce

Just add empty cdrom into kvm's RAW= into vm
Or eject cd from existing cdrom using virsh change-media command

PR here:
https://github.com/OpenNebula/one/pull/1526

to be backported to one-5.4


Adds construction graphs for shuttle structures

Changelog:

Adds proper construct/deconstruct graphs for shuttle windows
Adds construct/deconstruct graphs for shuttle walls
Adds deconstruct graphs for diagonal shuttle walls (construct doesn't rotate as it should and may need to be done another way{by not using base girder graph} if we want them)
Adds interior shuttle walls and associated construct/deconstruct graphs

:cl: Velcroboy

add: added constructable/deconstructable shuttle walls/windows!

OK, I just did a quick test to make sure no errors and construct/deconstruct worked. Everything should be good here.

I'll need a bit longer to think about this because we were moving towards removing shuttle construction...

While the conversation about these started when shuttle building was still a thing, I think it's more about consistency and having walls that can be properly constructed/deconstructed and have appropriate HP. Currently you can deconstruct the windows but not the walls which is just odd.
Shuttle building itself, I would say, is much less about the walls and more about the parts(console, thrusters, gyros.) Without those, these are more for style and allowing sanctioned shuttles some extra/balanced playibility. At best?worst? these just allow pre-made shuttles to be altered by players.
Interior walls would be nice for the consistency throughout ships and ship maps and create more of a exterior hull(higher HP) vs interior wall(lower HP) dynamic.
One thing though that does need to be merged if the rest isn't, is the shuttle window changes. Because currently they can be deconstructed in like 2 seconds with a screwdriver. Also a fix was added for missing destruction sounds in rWalls.
I guess that's my take on it. I don't particularly think it encourages shittle building any more/less but rather just creates consistency throughout the game.

I feel like there's two possible ways to go with this regarding consistency. Either give the walls/windows proper graphs, or make windows not deconstructible. I can see good points for going either way.
Making them have graphs means the cargo shuttle can be repaired with shuttle walls if it gets hit by a meteor.
Making all shuttle parts not constructible/deconstructible makes things cleaner from the standpoint of removing shuttle construction completely and will cut down on questions about "how do I build a thruster" or PRs aiming to make them buildable.

shuttle maintenance / repair sounds good to me. if shuttle construction continues to be an issue it can be clamped down on via other methods


Update image-webpack-loader's query option

This PR fixes the following warning which come as a result of using images in the components:
./app/assets/images/image.png
(Emitted value instead of an instance of Error) DEPRECATED. Configure gifsicle's interlaced option in its own options. (gifsicle.interlaced)
 @ ./app/containers/HomePage/index.js 24:0-75
 @ ./app/containers/HomePage/Loadable.js
 @ ./app/containers/App/index.js
 @ ./app/app.js
 @ multi eventsource-polyfill webpack-hot-middleware/client?reload=true ./app/app.js

The solution was found in this SO thread: https://stackoverflow.com/questions/42206190/webpack-2-warning-in-png-svg-deprecated-configure-optipngs-optimizatio.

Coverage remained the same at 100.0% when pulling 298b15149310bfe2d4046bc5209c8b9e10d93699 on ghoshnirmalya:dev into 0bf495edb216d6e3177538e47ee2d3d2ed4bb859 on react-boilerplate:dev.

Coverage remained the same at 100.0% when pulling 4bfad1a6d7bc6fe1f258af22466e5928eba3f38e on ghoshnirmalya:dev into e3b4bbbf5213b1bcdffe56e635ad097298cd6f4e on react-boilerplate:dev.


Quick Switch: elements are dark even in light mode

Screenshots

Related: The SelectChannelDialog shows a similar behavior.


Expose setSelectedDate

#2677

Hey @bogdan-litescu, I think this PR might fix my ticket #2685.
Did this get merged elsewhere or would it just not work?


Add new ioBroker.cleveron adapter

Adapter-checker checked, Appveyor-tested

RE-CHECK!

Automated adapter checker
ioBroker.cleveron

[ ] :heavy_exclamation_mark: [E201] Bluefox was not found in the collaborators on NPM!. Please execute in adapter directory: npm owner add bluefox iobroker.cleveron
[ ] :eyes: [W400] Cannot find "cleveron" in latest repository

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.cleveron

:thumbsup: No errors found

[ ] :eyes: [W400] Cannot find "cleveron" in latest repository

Add comment "RE-CHECK!" to start check anew

"RE-CHECK!"

RE-CHECK!

Forum testing thread
But since cleveron is a startup, I don't expect really feedback yet.

Hi,
sorry that it took that long, here are my review comments:

Your testing is red, it seems to be like a broken package-lock.json file
The way you encrypt the password works ... but you could have it more easy by using "encryptedNative" - see readme, optional
please remove https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L71 if not needed
same for https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L77
without logic for statechanges please remove https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L153
There are unknown roles used e.g. in https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L451 ... please verify the used roles and do not invent new ones yourself --> https://github.com/ioBroker/ioBroker/blob/master/doc/STATE_ROLES.md
maybe startusing async and await all the object creations? Then such hacks like in https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L521 are not needed ... they will break!
The timeout set anew in yur timeout function can not be stopped by unload. https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L149

Please check and asjust, thank you

Thanks for the Feedback here what I could do:

Your testing is red, it seems to be like a broken package-lock.json file

package-lock.json fixt, testing ok now.

The way you encrypt the password works ... but you could have it more easy by using "encryptedNative" - see readme, optional.

tried to settle it with 'encryptedNative' but did not work - will change that later.

please remove https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L71 if not needed

done

same for https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L77

done

without logic for statechanges please remove https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L153

done

There are unknown roles used e.g. in https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L451 ... please verify the used roles and do not invent new ones yourself --> https://github.com/ioBroker/ioBroker/blob/master/doc/STATE_ROLES.md

done

maybe startusing async and await all the object creations? Then such hacks like in https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L521 are not needed ... they will break!

fixed with 'asyc function' and 'await ... setObjectNotExistsAsync'

The timeout set anew in yur timeout function can not be stopped by unload. https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L149

fixed, changed to setInterval / clearInterval
thank you for your support

Thanks for the Feedback here what I could do:

Your testing is red, it seems to be like a broken package-lock.json file

package-lock.json fixt, testing ok now.

The way you encrypt the password works ... but you could have it more easy by using "encryptedNative" - see readme, optional.

tried to settle it with 'encryptedNative' but did not work - will change that later.

please remove https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L71 if not needed

done

same for https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L77

done

without logic for statechanges please remove https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L153

done

There are unknown roles used e.g. in https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L451 ... please verify the used roles and do not invent new ones yourself --> https://github.com/ioBroker/ioBroker/blob/master/doc/STATE_ROLES.md

done

maybe startusing async and await all the object creations? Then such hacks like in https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L521 are not needed ... they will break!

fixed with 'asyc function' and 'await ... setObjectNotExistsAsync'

The timeout set anew in yur timeout function can not be stopped by unload. https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L149

fixed, changed to setInterval / clearInterval
thank you for your support

Hi,
I had anther look:

Please do not log the password :-) Then all encryption is useless :-)
Which problems you had with encryptedNative? if you "Play around" did you alwys did "iob upload adaptername"?
Note: if you do it that way then you also need to "await" the call in https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L341 ... else this runs async in parallel ... this is ok in your case because last command
How many objects are created there usually? Because also these many states will be set later and they are not set "Asnc" ... If there are too many this can produce issues in execution ... maybe also use setStateAsync with await?

Additionally please also relese new version to npm after adjustments.
Thank you!

Hi
Thanks a lot for review & feedback.

Please do not log the password :-) Then all encryption is useless :-)

done, sorry

Which problems you had with encryptedNative? if you "Play around" did you alwys did "iob upload adaptername"?

Thanks for insisting on this one. I tried again and now it works. Might be, that I did not change index_m.html with my last tries.

Note: if you do it that way then you also need to "await" the call in https://github.com/iobroker-community-adapters/ioBroker.cleveron/blob/main/main.js#L341 ... else this runs async in parallel ... this is ok in your case because last command

'till now, that was my way to handle asynchronity (cascade). Left it as is, might change it in a futur version, if the API the adapter works with will get more complex.

How many objects are created there usually? Because also these many states will be set later and they are not set "Asnc" ... If there are too many this can produce issues in execution ... maybe also use setStateAsync with await?

Difficult to say. It's 13 objects per (physical) device. I think that the average number of devices per ioBroker-instance will be below 10. Nevertheless, changed to await setStateAsync.

Additionally please also relese new version to npm after adjustments.

done, it's<EMAIL_ADDRESS>Thanks a lot for doing this really great job with ioBroker

Thanks for insisting on this one.

Sure because it makes the developer live thaaaat much easier :-)
Thank you very much for the changes


Not found hardware back button and 4 tap to zoom

In android does not work the hardware back button and to zoom you have to press the image 4 times.

Duplicate of #24
Working on it shortly.


infra: change autoray to >= 0.6.11

Issue #, if available:
Description of changes:
Testing done:
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[ ] I have read the CONTRIBUTING doc
[ ] I used the commit message format described in CONTRIBUTING
[ ] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Wouldn't call this infra, since it changes the installed package


Add TextBufferProjectService Mac implementation.

Was unable to test the text buffer project service class due to signing issues with the MonoDevelop.Sdk dependency.
At runtime this implementation doesn't entirely work due to MonoDevelop not updating their Roslyn workspace.

#1789

🆙 📅


Performance update: & is faster than modulus.

Duplicate of #11.

#12 too.
but I just wanted to contribute to the most useful javascript library :/

but I just wanted to contribute to the most useful javascript library :/

To be clear, is this a real PR, or are you trolling and violating GitHub's code of conduct?

this NaN check actually won't work
https://stackoverflow.com/a/19955919/8222484


Option to set VISCA ID

I have multiple PTZ camera's using their own unique VISCA ID's, this because of the RS232 controller I use.
The current datavideo visca module is set to  VISCA ID 1 per default. It would be very help helpfull when I can set the Visca ID in the same screen where I need to provide the IP and port nr..

Will look into this later in the month. Should not be too hard to implement.
Closing as it's a duplicate, please follow the other issue.


1.19.4, plugin kick players from server without any errors in console

Hi, can you provide me more context about the issue. Where's the plugin (spigot or forks or bungeecord or forks), version of the plugin and, if possible, the list of the plugins

paper 1.19.4 (last build +-). Ver. pl. 1.1.0 and latest from SpigotMC. May help, I updated the server from 1.19.3 to 1.19.4.  The plugin stopped working after the update, and it kicks out players when trying to join the server. It was difficult to locate the problem because it does not display any logs in the console and the only thing that can be done is to remove it step by step.

I can't provide a list of plugins, and jars. I have a few plugins of my own.

The latest version is 1.1.3 and you can find it either on spigot or modrinth. I suggest using modrinth that is updated more frequently.
https://modrinth.com/plugin/customf3brand/version/1.1.3

I updated it, still the same (spigotmc). the plugin is not stable, I can't risk blocking the server again.


I-D Submission: confirmation of E-Mail sent has wrong TZ info

Describe the issue
I just submitted an I-D at 2023-04-27 05:58:36 UTC and got a confirmation
Email was sent at 2023-04-27 05:58:36 PST8PDT

(That is in the future.)
Code of Conduct

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

could you please forward the entire email message you received to me?

Problem is here: https://github.com/jennifer-richards/datatracker/blob/b691cb4a9f85866a175890260b80be215461b372/ietf/utils/mail.py#L327

Indeed, the problem wasn't in the mail, it was in the Weg page that confirmed the submission and showed that the mails had been sent.


java.lang.NoSuchMethodError on suite run

Greetings!
I'm trying to integrate munit-zio into my project, but the very first simple test suite failed with uknown runtime error.
Code to reproduce:
import munit.ZSuite
import zio._

class ConfigSuite extends ZSuite {

  testZ("Just add two numbers") {
    assertEqualsZ(ZIO.succeed(1 + 3), 4)
  }
}

It compiles, but fails with java.lang.NoSuchMethodError: 'zio.Runtime zio.Runtime$.global()'.
I'm using zio v2.0.0 and munit-zio v0.0.3.
Could you please help me to understand what I did wrong?

Hello! Version 0.0.3 does not support ZIO 2.0.
I created PR with update, new version coming soon.

Oh, my bad...
I should have looked at dependecies definition before opening this issue.
Thank for your help!


Implement more advanced address checks

This change becomes necessary for addresses that are not 45 characters long, i.e. when address prefix is not 6 chars long.

Looks good, really appreciate you catching these subtleties


fix filename & title getting improperly defaulted in filesUploadV2

fix filename & title getting improperly defaulted when uploading multiple files using MethodsClientImpl#filesUploadV2
fixes #1345
Category (place an x in each of the [ ])

[ ] bolt (Bolt for Java)
[ ] bolt-{sub modules} (Bolt for Java - optional modules)
[x] slack-api-client (Slack API Clients)
[ ] slack-api-model (Slack API Data Models)
[ ] slack-api-*-kotlin-extension (Kotlin Extensions for Slack API Clients)
[ ] slack-app-backend (The primitive layer of Bolt for Java)

Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.

Hi @Cheos137! 👋  Thank you for taking the time to send in this PR and sign the CLA!
This logic is looking solid to me but we might want to include a test to ensure this behavior is correct and doesn't regress in future changes 🙏

cuz i suspect there might not have gone out a notification from it: yesterday i've added tests to this pr


add file format list to readme

Fixes #21 - thoughts @andrewmelis?

We'll definitely work on documenting what customers will digest of GK formats in customer-facing ways.


Adding Computer Science Questions

developerquiz.org currently has over 600 quiz questions. We are looking to expand on the Computer Science questions and we encourage other developers to add their quiz questions to the site.
You can find the complete list of questions below.
https://github.com/freeCodeCamp/Developer_Quiz_Site/blob/main/src/data/general-cs-quiz.ts
You can add your own questions to the top of that file.
Please first check to make sure your question doesn't already exist in the file.
Here is an example of one of the current CS questions.
 {
    Question: "Which one of these is NOT a common time complexity equation for BigO notation?",
    Answer: "O(n14)",
    Distractor1: "O(1)",
    Distractor2: "O(log n)",
    Distractor3: "O(n)",
    Explanation: "O(1), O(log n), O(n) are just some of the common time complexity equations for BigO notation.",
    Link: "https://www.youtube.com/watch?v=zg9ih6SVACc&t=867s",
  },

For the link field, please make sure to use a freeCodeCamp article, or freeCodeCamp YouTube video.
If you choose to reference a video, please make sure to include the timestamp for the topic covered.
Here is an example of a timestamp video for BigO notation
https://www.youtube.com/watch?v=zg9ih6SVACc&t=867s
You can read more about how to create timestamps in this helpful article.
Please note: Do not close this issue. We want to keep this issue open for multiple contributors.
Happy contributing!

Is this repo still active?

Yes it is

Can a question like what the HTML abbreviation stands for be added to the pool of questions?

@a2937
I ran a search in the full-quiz.ts and there is already a question for the abbreviation of HTML on line 6566.
But the question should be modified because the current answer says
Answer: "Hyper Text Markup Language",
But it should say
Answer: "HyperText Markup Language",
But in general, it is always best to check the list of questions first to see if it exists so we don't create duplicate questions.
Hope that helps!

I wanna update an distraction answer to be "Friend Function" instead of "Friend functiom" ; as soon as I can think of an proper question involving template strings.

@jdwilkin4 Hello Sir I have added two CS quiz questions , Can you please help me out I am not able to do pull request.
This is my first open source contribution.Thank You

@jdwilkin4  Hello Ma'am I have added two CS quiz questions .I had pushed the changes but can you please help me out with pull request.
This is my first open source contribution.Thank You.

Hi @Aman0786-git !
I think it would be best if you post your question, with the results you are getting in the terminal in the freeCodeCamp forum so we can get a better idea of what the issue is. We will be able to better assist you there.
https://forum.freecodecamp.org/

@Aman0786-git
You can also read through this helpful article on how to create pull requests
https://opensource.com/article/19/7/create-pull-request-github

For the attribute "link", can we add link to Wikipedia articles?

If people have questions about this issue, or questions in general on how to contribute, please use the GitHub discussions area.
It will be easier to track conversations over there 👍

For anyone interested in contributing, please read through the contributing documentation first.


Delete Measurements

Version usage: Latest @ ohif / extension-cornerstone @ 2.10.10
When you create a measurement (circle, annotation, angle) in a view with several images (that has scroll), when deleting from measurements (the side panel) or from the trash icon, it deletes the measurements but they continue to appear. Only delete well if you right click, delete measurement.
This only happens when there are several images, if it is only one image, it erases it well.
Play bug:

Select a series with several images.
Put a measurement.
Give delete from the trash button or from the side panel.
See how the measurement is cleared but not cleared from view.

AFAIK, this might be caused by OHIF-2479.

@PoyangLiu the solution is definitely -> #2479 Thanks!


Minor edit suggestions for stale models blog

You may disregard these recommendations if you used the Edit on GitHub button from dvc.org to improve a doc in place.

❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.
🐛 Please make sure to mention Fix #issue (if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.
Please choose to allow us to edit your branch when creating the PR.
Thank you for the contribution - we'll try to review it as soon as possible. 🙏

noticed a typo :)
Some other suggested edit (there are some other places with "all of the" if you want to purge them completely, I only changed the ones referring to "data")

Thanks @omesser !


Filter packages by maintainer name.

Have you checked for existing feature requests?

[X] Completed

Summary
Currently Atom does not allow us to filter search results by maintainer name.
Enter your response below:
This would allow package maintainers to find their own packages quickly. My guess is that they would usually make
tools that they are using themselves to solve a problem that they've found. This benefits the rest of the community as they do not have to go through the same mistakes as that maintainer.
Any alternatives?
None that I can think of now.
Other examples:
No response

I've transferred this issue over to the backend repo, as that functionality would have to made available here first. I think this is a totally fine idea, and see no issue with implementing it.
Although as a reminder any enhancements to the Backend API are awaiting a 2.0 release as per the plan of the backend, which you can see more details in the readme of this repo.
Otherwise I'll go ahead and add that to the Backend Roadmap.
Thanks for the contribution!

I am having trouble understanding exactly what you mean. For installed community packages, Atom does filter out search results if you input the package author's name.

This would be in the Atom editor (Install package search). Sorry I forgot to clarify, however, I do not use the website anymore now since the images were not working a long time ago (months ago?), and now there is an increased presence of spam packages on the front page of the website.

I am having trouble understanding exactly what you mean. For installed community packages, Atom does filter out search results if you input the package author's name.

This would be in the Atom editor (Install package search). Sorry I forgot to clarify, however, I do not use the website anymore now since the images were not working a long time ago (months ago?), and now there is an increased presence of spam packages on the front page of the website.
Another thing to clarify is that I consider the website and Atom Install search the similar/same things. They pull from the same source and you want to find the same package whatever that pkg may be. This may not be the direction that Pulsar is going, the website and the desktop app may differ which is fine. They might not even have an external website and solely rely on desktop to find packages- I do not know what direction they may go in the far future.
What I mean is that we should not only have one liner search engine. Maybe click on a button that says search by maintainer name or something.

Well to answer some of those questions.
I've moved your issue to this repo because this repo is the 'external website'.
This repo is the API backend, that is already hosted in the cloud, that would be the service that a frontend website talks to, to find packages, as well as the editor itself, the cli and so on. This should be the single source for all of those.
Additionally while I realize that Atom's service does have broken images and many spam packages, this is our new backend. Which has been built from the ground up, and no longer contains any spam packages (as far as we are aware) and is the backend that any recent binary of the Pulsar editor will use for searching within the editor itself.

This could be made linking packages table to users one. One to many or many to many, we have to decide.

Resolved by #215
The backend now supports the query parameter of owner which will match with the owner of the repository that's been published. For example https://web.pulsar-edit.dev/packages?owner=savetheclocktower


The WebSocket connection to mydomain failed

The problem
I cannot connect to HA, since I have the following:  "The WebSocket connection to mydomain failed"
I tried to use this module in local mode, or in remote mode, and did not solved the problem.
What version of Cloudflared has the issue?
5.1.8
What was the last working version of Cloudflared?
No response
What type of installation are you running?
Home Assistant OS
Add-on YAML Configuration
http:
  use_x_forwarded_for: true
  trusted_proxies:
    - <IP_ADDRESS>/24

Anything in the logs that might be useful for us?
Anything in the logs that are related to websocket failing or something.

Steps to reproduce the issue
Install the extension, set the token to an existing tunnel. Configure a domain ha.mydomain.com, access it and it won't work.
Additional information
Really strange, because I use this extension to also expose uptime, and the websocket for this service is working properly, only the HA websocket wont work.

Which Ports are you using? When you say "local mode" and "remote mode", do you mean locally within your network and externally of your network?
Can you please share your add-on config?

Hello Thanks a lot for your answer.
In relation to local and remote mode, I was referring to the addon configuration. Here: Local and remote
Here is my addon config:
external_hostname: ""
additional_hosts: []
tunnel_token: >-
  my_cloudflare_token

Pretty basic config. Let me know if you need more details.
I also wanted to add a detail: The socket problem is not systematic, and seems to be a chrome/ium related problem, since in Firefox, I never had a problem ... I don't know if it help

Thanks. Which domain registrar are you using? The reason being, that we do have some issues with freenom...

so you are simply using HA in your browser and that is where the websocket error happens? In any case, I suggest that you first do a full un- and reinstall from the Add-On and if this does not help, backup your config and do a clean install of HA, if possible

Thank you, I have to to try your solution.


Skip remote Correios test, service unavailable for the informed route.

A Correios remote test is failing because the service is unavailable for the informed route. Skip it for now.
@kmcphillips @RichardBlair

:+1:

:+1:


Use our logo for the avatar in PyPI

It will help our users to be sure that it's us.

Same thing in Docker Hub

Thanks for bringing in to our attention @amiorin. I will take care of this!

👍


XtraCord Bot

XtraCord bot is coded in discord.py library, its still under development moderators can test it out if needed to assure its a safe bot!

Unless you have a GitHub page, please delete the GitHub lines


Tajriba.emit error

Participants see a white screen, and in my sentry logs I see:
Error: Unhandled error. (undefined)
  at Tajriba.emit(/assets/index.8f633f52.js:33507:15)
  at error(/assets/index.8f633f52.js:33944:16)
  at Object.emit(/assets/index.8f633f52.js:24654:21)
  at socket.onerror(/assets/index.8f633f52.js:24712:47)

Probably related to:
https://github.com/empiricaly/empirica/issues/483
In that issue, the server reconnected on its own. Unfortunately ours doesn't, and users stay on the white page indefinitely.
I've seen the issue associated with a fully blank screen, and also when the "report bugs" is still available:

Seems to occur in about 5% of participants.

notes from nicolas:

A disconnection shouldn't trigger an "Unhandled error". It should reconnect cleanly.
Avoiding disconnections entirely is impossible, though I'd hope for them to be relatively rare. 5% seems like a rather high number to me. I don't know how long your experiments go for, maybe with long enough connections, disconnections will eventually happen at that rate. It could also be your environment, whatever reverse proxy or load balancer you're using (maybe some default timeout, or it's not very stable with long-running connections). That being said, even if it were 0.1%, we'd still want the client to be able to reconnect automatically. Just saying this to see if there might be a special reason you're seeing this because of your environment.
2 things in the short terms you can try:

you might want to increase the idle_timeout on your load balancer aws_lb.  While this could help, it is a bit odd, since the default seems to be 60s, and there should be a ping on the websocket every 5 or 10s. So it doesn't completely make sense. Though maybe these are not noticed by the load balancer for some reason (payload too small :shrug:) https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/lb#idle_timeout
Maybe you can add a React Error Boundary at the top of the React tree and catch this error, and reload the page when you see it. https://docs.sentry.io/platforms/javascript/guides/react/features/error-boundary/ Maybe in the fallback, add a useEffect with an empty dependency array, and in there you wait 5 seconds (to let sentry report the error, maybe, I don't know if it needs that or will report the error – I think so, since they offer a custom error boundary) and then reload the page. And you can show a message saying an error was detected, reloading. I don't know if this will work, I have not had a chance to test it. I'm actually a bit suspicious whether the error will be triggered in the React tree or not. If not, I guess the best place to catch the error is in an unhandledrejection. But I'm not sure. https://developer.mozilla.org/en-US/docs/Web/API/Window/unhandledrejection_event

Added react error boundary to catch this in https://github.com/Watts-Lab/deliberation-empirica/pull/828


Enter key and esc key dose not click "=" and "C"

Up until the last few days, when hitting the Enter key on the keyboard it would '=' the sum.  And the Esc key would 'C' clear all This no longer functions. My best guess is that this is due to a Windows Update   and i try to delete data for calculator setting  but I thought I would report it as an issue in case.
If possible, I would like to fix this.
Note : somtimes enter key and esc key work correctly
Thank you, Abdo_Ka

me neither. It probably happened with the last update.

it is at windows issue. this works for me:
press WINDOWS & R key , paste C:\Windows\System32\ctfmon.exe ENTER
if this works , press WINDOWS & R key paste shell:startup ENTER
new window will be opened. Paste LINK to ctfmon.exe in startup

Latest version of my system is 2022-10 Cumulative Update for Windows 11 Version 22H2 for x64-based Systems (KB5018427) and I'm able to use enter key and esc key for computation(=) and clear.

Latest version of my system is 2022-10 Cumulative Update for Windows 11 Version 22H2 for x64-based Systems (KB5018427) and I'm able to use enter key and esc key for computation(=) and clear.

thank you very much ❤❤

it is a windows issue. this works for me: press WINDOWS & R key , paste C:\Windows\System32\ctfmon.exe ENTER try now calculator. this fixes search doesnt work issue too. if this works , press WINDOWS & R key paste shell:startup ENTER new window will be opened. Paste LINK to ctfmon.exe in startup

It is worked thank you ❤❤

Shouldn't [Del] be used in addition to [Esc]? [Del] key is more obvious key for resetting your current expression and is used in most of linux calculators. I was going to open an issue on it and just found out that there's [Esc] key for that


Can be merged fix - 113

[Auto generated]
Number: [#113]

github-gitlab-webhook-server: Closing issue for PR: Can be merged fix.
PR was closed.


Add my first file.

I created a new branch, and a PROFILE.md file, and made a commit.

I added a PROFILE.md file.


Show label list on label set

Showing the list of labels underneath the label set name, making it clear to the user which labels will be created. Related to issue #7812

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e80fe20). Click here to learn what that means.
The diff coverage is 75%.

@@            Coverage Diff            @@
##             master    #9251   +/-   ##
=========================================
  Coverage          ?   41.06%
=========================================
  Files             ?      556
  Lines             ?    72555
  Branches          ?        0
=========================================
  Hits              ?    29796
  Misses            ?    39011
  Partials          ?     3748

Impacted Files
Coverage Δ

models/repo.go
48% <72.72%> (ø)

models/issue_label.go
65.06% <76.92%> (ø)

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 e80fe20...45ec0d5. Read the comment docs.

Please add screenshots


Refactor(SQLAlchmey)

Need
The way we are using database session in corpora is confusing and potentially error prone. Database sessions should be managed outside of the common code, because the each component has different database access requirements.
SQLAlchemy has well supported methods for performing most of the operation we wrote custom code for. We can reduce the amount of code we must maintain by taking advantage what SQLAlchemy offers.
Approach

[x] remove db_util, entity, and anything that inherits entity.
[x] replace db_util with a session that is global to each API request or the processing container.
[x] Pass in the session as a parameter to common code
[ ] Use SQLAlchemy ORM for all CRUD operations.
[ ]

The only thing left to do is Use SQLAlchemy ORM for all CRUD operations. This is something we should do so simplify our code base and testing.


Deployment manager

Refactoring to help with cloud version.
Please test this throughly before merging!

so regarding the https://github.com/Agenta-AI/agenta/pull/707/files
Most important thing am seeing with a big difference is that the endpoints are not concatenated with the domain
Right now in oss/demo it’s localhost/uri
In cloud it will be another endpoint something like https://app-1234.llm-apps.agenta.ai/
uri in general should contain the protocol + domain + path
this should also be adjusted so we can fetch the url from the deployment
https://github.com/Agenta-AI/agenta/blob/main/agenta-web/src/lib/services/api.ts#L200
    const url = `${process.env.NEXT_PUBLIC_AGENTA_API_URL}/api/containers/container_url/`


Sometimes  behaves like i

Any ideas why shift-a may behave as if I pressed i?
It seems to happen only on certain lines. F.e. when I'm on a vimrc comment, pressing shift+a behaves like i, and the cursor doesn't go to the end of the line when it enters insert mode.
But on most code lines shift+a does go to the end of the line as expected, while on some code lines it does not.
For example:
                    " Shift-a doesn't work on this comment, but it works on the next line:
                        Plug 'junegunn/vim-easy-align', {'commit': '12dd6316974f71ce333e360c0260b4e1f81169c3'}
                            " shift-a doesn't work on this comment, but it works on the next line:
                            xmap <leader>a <Plug>(EasyAlign)
                            " Also doesn't work on this comment, but works on the next line:
                            let g:easy_align_delimiters = {
                            \     'a': {                               <-- it works on this line
                            \         'pattern':       '\<as\>',        <-- doesn't work on this line
                            \         'left_margin':   1,              <-- doesn't work on this line
                            \         'right_margin':  1,              <-- doesn't work on this line
                            \         'stick_to_left': 0              <-- doesn't work on this line
                            \     },
                            \ }

Here's a recording of pressing shift+a followed by ctrl+c on each line:

dup of #467
upgrading neovim to latest nightly must solve this


Feature request: Clickable logo that points to specific url

Hi!
I'd love to specify url somewhere somehow. And I'd like my logo to be clickable and point to that url.
Is it possible, now?
Thanks!

Great idea!
Not possible now. We'll need to implement it.
Would you try to create a PR for this?

@RomanGotsiy I'd love to but I'm not good in Angular :(

Also, would be great to have a link at the top something like Go back to <url>.

Wow, link on the logo would be a great addition!

@RomanGotsiy could you describe what parameter and in what section you expect, so we would know what PR you will accept. Thanks!

@RomanGotsiy I'm trying to prepare PR, will discuss my prev comment in the PR, if you don't mind.


Part 1 - Chapter 3

Please refer to Readme  for contents of this chapter.

This chapter has had a final review by Joel, Firas, and Adam. All edits were made.


Sorting is not correct when looking for Profiles:  commands

Whats the issue here?

Yes, this is somewhat intentional for commands: we have always sorted matching commands by alphabet to try to group related entries together that share the same prefix and keep the order predictable. We have intentionally not added fuzzy search ranking to commands.

People have been hitting this issue more, recently. I wonder why... could be more commands in the command palette making finding the right command challenging

@bpasero this does not look related to fuzzy matching. But shorter titles should get some boost. For example: if my command title has 50 characters and 5 are matching. It should be ranked below a title with 20 characters where 5 are matching.

I cannot find the issue anymore where I explained our reasoning in greater detail, let me post here again: the sorting in the commands picker is historic and we had reasons to do it back then, which I am happy to revisit today if we all agree.
For the longest time, commands picker did not have a list of "most recently used commands" and it was very frustrating to find the right command, given the alphabetic sorting. I added commands history as a way to mitigate that: the history will put results to the top in their order of usage if matching the search query.

With the idea being that you can build muscle memory very quickly that a query such as "copy" will very likely put the one command to the top that you had always used.
Now, if you leave the history beside and just look at the commands sorting, as you know commands appear in alphabetic order if they match the search input. There is no boost, no fuzzy match, just simple sorting by alphabet. The reason here was to try to keep command results predictable: the order will not change based on your input and similar commands are always appearing together in the same order.

This is getting more important now. Consider this flow:
https://github.com/user-attachments/assets/d3b18806-3cd6-40f4-86e0-fa0c6a206b0a
Thus, Copilot is sending more "exact matches" to the command palette. And imagine a new user coming to this. We need to make sure that the 100% match is the first result.

If chat knows exactly the command to use, an alternative would be to open command palette with the ID of the command. In that case we only show the matching command.

Well it is AI driven from @vscode, so I am not sure if it can always provide a command ID to use @bhavyaus
I would still prefer to "fix" the sorting.

@bpasero @vscode does have the original command ID, but setting up the quick pick to open with the exact command id results in this:

So, we use the exact command label instead. Revisiting the sorting would be a better fix here.

I leave it up to Tyler, who owns this. A pragmatic solution would be to force a command to the top when it literally matches the words you typed entirely.

@bhavyaus whats the problem with your screenshot? The command you want is at the top, no?

In this case, the sorting is so bad, I'd see it as a bug:

If multiple words match, the rank should be the sum of the indexes of all matching words, multiplied by some factor (first index by 1, second index by 2, third by 3 and so on).
Then the results should be sorted ascending by that rank.

@bhavyaus whats the problem with your screenshot? The command you want is at the top, no?

My screenshot was to show why searching via command id when opening the command palette from @vscode responses is not the best approach to teaching users about a command. Ideally the sort order would be fixed so that searching via keywords would surface the right commands.

I'm running very frequently into this:

I'd suggest to sort them ascending by 2 * item.length + item.indexOf(search), text when a search is active (or something similar to boost short results and an early occurence of the search). It works very well in the suggest widget, maybe @jrieken has some ideas here?


resume from sleep

Hey!
I have wifibox successfully running on a Dell Vostro 5470 w/ FreeBSD 13. WiFI won't reconnect automatically after resume from sleep. Manually restarting wifibox helps, but it's kinda inconvenient.
Can I get some pointers on how to debug this?
Thanks!

Do you have devd(8) configured to call wifibox resume command on resume?  Note that this is not done automatically on install.

Yes, I also tried manually running wifibox resume to no effect:
$ cat /usr/local/etc/devd/wifibox.conf
# This is a devd(8) configuration file to run the resume action of
# wifibox on the ACPI resume event.

notify 11 {
        match "system"          "ACPI";
        match "subsystem"       "Resume";
        action "/etc/rc.resume acpi $notify && /usr/local/sbin/wifibox resume";
};

$ service devd status
devd is running as pid 30082.

So you are saying that wifibox restart works for you after resume, but wifibox resume does not?  In that case, you can try using the restart command in the devd(8) configuration file.  My other idea is to insert a call to logger(1) at the action, something like this:
notify 11 {
        match "system"          "ACPI";
        match "subsystem"       "Resume";
        action "/etc/rc.resume acpi $notify && logger "Restarting wifibox" && /usr/local/sbin/wifibox restart";
};

Then you can check it in /var/log/messages if the action has been called at all.
It may also happen that you are have something attached to the ACPI resume event on the side of devd(8) but with a higher priority (that is 11 for wifibox, which can override the default 10 coming from the default configuration).

Was I able to help you in resolving the issue?

Sorry for replying so late @janondrusek I did not notice your response.  I have checked what is going on, and it seems restart is only needed because the tap(4) interface is not brought up after resume.  This can be fixed by setting the net.link.tap.up_on_open sysctl(8) variable to 1.  But I am planning to embed this functionality into wifibox so one would not need to bother with configuring this.

Version 0.6.0 fixes the issue for me, but please test it.

I've been using sleep / resume for about two weeks now without any issues. Thanks!


register_buffer

I have an issue regarding the training, I launch the program I get :
KeyError: "attribute 'mem_bias' already exists"
It seems that you define 2 times mem_bias in memory.py :
self.mem_bias = Variable(torch.Tensor(N, M))
self.register_buffer('mem_bias', self.mem_bias.data)

It is perhaps a problem with the pytorch version (I have pytorch 0.3).

Hi @Forbu , thanks for reporting 👍


Kudu rest api for changing app setting doesn't work

Hi,
I was using Kudu rest api to change my app setting.
The request I was using is:
url:  https://docs-live-sitesearch-ppe.scm.azurewebsites.net/api/settings/
Method: POST
body: { "DoNotRunCrawler": "true"}
response is 204.
Then I use get API:
url:  https://docs-live-sitesearch-ppe.scm.azurewebsites.net/api/settings/DoNotRunCrawler
or  get API
url:  https://docs-live-sitesearch-ppe.scm.azurewebsites.net/api/settings/
Both results indicates the key of "DoNotRunCrawler" is "true".
But if I go to Azure portal to check this key in app service "docs-live-sitesearch-ppe",  it still shows "false", which is the original value before I changed.
And this value is consumed by my web job.  I tested my web job and it seems the old value(DoNotRunCrawler: false) is consumed.
Could I know why the change doesn't take effect?

They are two different things.  Kudu can run on-premise or on Azure.  To change Azure App Service appSetting, do use portal instead.

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Use rustfmt standard Rust syntax

@sushant94 now that rustfmt works with stable rust 1.3, seems like a reasonable change to stick to the Rust defaults. Even if I prefer tabs, having a standard tool for indenting code give us much more benefits than having a handy editor.

There's poor tab indentation support in rustfmt, looks like some constructions are not supported yet, but that's no issue for our current codebase. It would be good to define a working .vimrc for it.

Ok, this is ready to be merged now as I've made formatting changes in the other rust repositories too.
Thanks!

Kewl!

On 31 Oct 2015, at 12:08, Sushant Dinesh<EMAIL_ADDRESS>wrote:
Ok, this is ready to be merged now as I've made formatting changes in the other rust repositories too.
Thanks!
—
Reply to this email directly or view it on GitHub.


You cannot change your ringtone

Description

Try to type anything in the ringtone menu and it doesn't work at all.
Reproduction

Screenshots

Additional context

it cant be your uplink code id thats what anything is

without an uplink this does not happen
(you clicked set right?)
video soon

also saves when its not your uplink code
@DogZeroX close


Inconsistent SET: setting non existent key throws error (error) sequence item 0: expected str instance, list found

Steps to reproduce

Start DiceDB instance on x86_64 Ubuntu 20.04.1
Login to DiceDB using DiceDB CLI
Set a key using the command

<IP_ADDRESS>:7379> SET key 100

Navigate to the previous command using arrow key and update key name to key1 and rerun

<IP_ADDRESS>:7379> SET key1 100

Expected output
The expected output when the above set of commands (maybe when run on Redis)
OK

Observed output
The observed output when the above set of commands when run on DiceDB
(error) sequence item 0: expected str instance, list found

Expectations for resolution
This issue will be considered resolved when the following things are done

changes in the dice code to meet the expected behavior
addition of relevant test case to ensure we catch the regression

You can find the tests under the integration_tests directory of the dice repository and the steps to run are in the README file. Refer to the following links to set up DiceDB and Redis 7.2.5 locally

setup DiceDB locally
setup Redis 7.2.5 locally
setup DiceDB CLI

Note

Running DiceDB branch master and DiceDB CLI on Warp Terminal

Can I get this assigned to me please? Thanks in advance.

Can I take this up?

@shashank-priyadarshi is this behaviour only with Dice CLI or even with redis-cli you face the same issue?

No haven't been able to reproduce this using redis-cli. Retried the same with similar setup as well and Dice CLI but the issue was gone. Cannot say for sure if it was an issue from the CLI or the terminal.

@hariharanragothaman please try to reproduce the issue, if not able to, we ca close this for now.

Hello @hariharanragothaman,
There has been no activity on this issue for the past 5 days.
It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.
We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.
Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.
Thanks again.


I don’t know how to use it

It’s self explanatory. I have no idea how this works

ok i will tell you

but dont use email mostly dont check it

50$

wait its probably night for you...

Plz tell me I want free membership

Sorry about the closing and reopening. I am so confused about how to use github. Cause I joined a few days ago

sorry was at school and you sent that 13 hours ago but i will still tell you

well the just ignore it was at 9: something

step 1  go to prodigy and join any world
step 2  once you loaded in either do ctrl shift i or go to your bookmark if you do bookmark put javascript: then the code or if you do ctrl shift i just go to console and paste in the code
finish then you got what you needed but the free membership is temporary meaning when you log out you dont have membership

well you can do it with any hacks but you have to copy the code that you want then do the steps thing i should have done step 2 as go on github go to prodigy hacks click on the best hack which is rxzyx just its the only one that works then go to player click on what hack you need the click the 2 squares then do the rest that i commented

and im telling you this because by the time your reading this im at school go from 8:15 to 2:15 boring 6 hours

pain

I know

But I have iPad

No pc

oh

yeah just do step 1 and 2 but the url part on step 2 try that

which is javascript: then the code or hack you want

I tried that

Bro I have 8 hours of school

Rip me

i know painful

like whats the point for long hours we are not gonna become Einstein the more ours we have of school

what time is it for you for me its 3:02 AM

I have to go

ok bye

I’m busy

ok

i have a feeling the ghosts here

So do I copy the code?

Cause I did that

you copy then paste it

Ah

ok see if it works

:(

oh

ima show you a screen shot of what iy looks like

Nonsense

Ok

forgot how to take a screen shot bruhhhhh

Huh

Bruh

Ok

It’s like 11 26 am here

its 3:27 AM

I have school ☹☹☹☹☹

sameeee

got the screenshot

Like I am in school

In the building itself

Yes

focus

thats what you put in url bar

then paste the code you want

Aha

dont mind the blue dont know what happened

Same thing…

Lunch time

ok bye

Get some rest bro

not tired

probably gonna sleep at 7 but ima try at 5 for me

right now 3:41

ima sleep its 4:09

best sleep ever

POV me:

4:42 pm for me

Bro

You told me that at 12 30 for me

Am

oh

what time is it for you?

for me its 1:19

am

hello?

oh wait your in school...

It works

There was just a small space

Yay

nice

you closed this as completed

well, Ima help other people if they even reply.

bye

ima sleep

ima sleep

ima sleep

wow 100 comments most of it is me talking to myself nice

Bruh I’m in class

i konw

think its like 1 something pm think you were taking building?

i think not yet

yeah ima sleep so tired i got tears coming out of my eyes

im so lonely 💀

make sure to not get caught doing the hacks


Align with canonical resource attributes for browsers

For the tags/attributes, we're using to denote browser properties, we should use the OpenTelemetry semantic conventions.
Proposal:
https://github.com/martinkuba/opentelemetry-specification/pull/1/files

this should be part of the events API issue once we have one


Some cache improvements

4 kind of chache that cache both registry and taget
moved all cache instead the codecoverage one on buildjet service that provide 20Gb free but a alimit on a single cache entry of 5Gb (covergare is more than 8)
Disabled CARGO_INCREMENTAL because is useless on CI
Disable native cache because again are useless on CI

Little note about 3. Cargo incremental is useful on develop machine because help on skip some partial artifacts compilation on local sources, but cargo use files timestamp to decide if should compile it or not and on CI all owned sorce files will have a timestamp older than the artifacts on the cache ones. Disabling the incremental build we can save 20/30 % in time
To test this PR I've run a cache manager step and then I'll run an orchestrator.
ATTENTION
Some notes about this caching strategy. We decided to use a cache singleton because we want to leverage on job parallelism and our cache blobs are really huge and then all new cache element kick out the old ones really fast. BUT is not possible to share cache across different branches (reasonable) but we can only use cache generated from parent: that's means the cache created on main can only be used from the branch that are based after the main/HEAD tip at the time the cache is computed. To make it explicit imagine the you generate a new branch on Friday and  then a new PR was merged on main, the cache manager run on Sunday and if you try to execute the orchestrator on your new branch Monday you will not use any cache till you rebase the branch on main.
In order to mitigate the last described issue we can introduce a check that just check if the cache are present (in the orchestrator) and fail fast is some was missed with an explicit message that ask to rebase your branch on main (we can introduce a flag to override it if needed). In this case we should save money and time.... I hope.
CACHE IS A MESS

Thank you very much ! So to recap:

In order to use the cache we need to make sure that our branch is always aligned with main
How frequent the cache manager job should be ran on main ? Every time a new branch is merged ?

Cache manager run 1 time per week (on Sunday), the important thing is that we should have PR branches based on cache tip before run orchestrator.


Create README for sampling test case

Pull Request Functional Test Report for #2327 / 5eccbe95803a3f39520763d65212e97526ed1ce3
No tests identified for validation.
Help

Closed in favor of #2337