2 issues with the dialog component on ipad safari

On ipad (with safari):

The modal won’t open when you tap on it the first time. You have to tap it twice.
After closing the modal it redirects to a route that isn’t found.

This is on https://salad-storybook.fly.dev/core_components/dialog

Thanks for reporting, I don't have an iPad. Let me try to reproduce on desktop

Here’s a recorded screenshot :-) if I can do anything let me know. https://vimeo.com/966712369

@joepstender are you referring to the redirect when a user dismisses the modal/ dialog?

Yes, by tapping on the X in the top right side corner, or by tapping anywhere else on the screen.

Oh, it's because I open a new link on_cancel event. Updated the example.
The other issue happened on Safari for Mac too. I'm investigating

Now animation is handled by tailwind-animate and this issue would be fixed in PR #22

Indeed the redirection issue has been fixed on ipad! I still have to tap a second time on the button to open the modal though, which is not the case on https://ui.shadcn.com/docs/components/dialog. There only only one tap is enough. That issue might be hard to debug without an actual ipad…

I suspected that is storybook issue.
This issue occurred on my iPhone, but when I switch to playground, it doesn't happen any more

Yeah, I suppose we can close this issue.


Spelling !!

Christmas is early this year, so presents arrived !
(Had some free time so you get a bundle of joy ! )
I went over almost all markdown files across the entire repo and came up with quite a few spelling corrections.
It is hard to see them every time I use the docs to train others on ADO, so its time i fixed it :)
P.S. you really need to think about the rules in markdownlint
to start, but all the flags are false (you are missing some), and you have TONS, of places
where MD047 (easy to fix) and MD051 are just yelling at my face, every time
I open one of the markdown file (and you have almost 3000 of them in the repo. :) )

Thank you for improving these documents @ArieHein! Your changes look great. I added a couple more and will merge the PR shortly.

Thank you for improving these documents @ArieHein! Your changes look great. I added a couple more and will merge the PR shortly.
@steved0x and @mijacobs for awareness

Awesome. I didn't want to include grammar together with the spelling but i can do that :)


Add i32 literals

Part of #1

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (2c5841a) to head (6c17ca1).
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #11   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        13
  Lines          420       450   +30
=========================================
+ Hits           420       450   +30

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


feat(@effection/node): switch to cross-spawn, implement windows support

This PR switches to cross-spawn. Currently we are stuck with nodejs emulation of signals on windows not killing nested processes gracefully.
This closes #182 (eventually).

I'm totally cool with the decision to switch to cross-spawn, which is what execa uses under the covers... did it make it easier to just stick with the existing semantics? One thing I know is that kill() on the process-group strategy did not work on windows. Is that something that cross-spawn can take care of for us?

for clarity, in discord I said:

well, I think @effection/node really requires the ability to stream stdio as some uses in BigTest open a process, send messages to it and wait for the event rather than strictly waiting on close.
The tests in effection imply that but digging into BigTest, it seems to be the case from my understanding. execa can't stream and it's built more around running a process and returning the output.
It seems like it's worth using cross-spawn instead so we can still stream, and do the processing killing ourselves.

It made it so much easier to stick with the semantics. You are right about the kill() and I remember watching you struggle with that some back when you were working on this part of the code. cross-spawn is mostly a drop in replace of child_process spawn, so it doesn't seem to take care of it. I'm currently looking at the interesting bits of the execa kill, but there seems to still be bugs and orphaned processes that don't get cleaned up in windows under execa.

So I've tried a handful of considerations (and pushed a bunch to check on a clean build and make certain it wasn't bad processes hanging around on my end). From what I'm seeing and reading, the only way the child_process seems to close on forced exit SIG* is with stdio set to 'inherit'. The side effect is that with the stream on the parent, we can't observe it and waitFor fails on undefined. If we use pipe, it runs but won't seem to respond to any SIG* event. I think it's something to do with the stream on the parent vs the child.
The only other thing I can think to try is to create a separate stream on the parent, pass that too the child to pipe into and let the parent manage the data control. I haven't had a chance to try this though, and I think it might require a bit of refactoring if it does work.
Anything else that you can think of? /cc @cowboyd @taras

Well, it's hacky but this might work for windows:
spawn("taskkill", ["/pid", child.pid, '/f', '/t']);
Spawn that in response to a kill signal.

So I made a super minimal test case, and that proceeds and finishes as expected. So there must be something in BigTest / Effection that is keeping the process hung / open.
const spawn = require("cross-spawn").spawn;

async function main() {
  const child = spawn("ping", ["www.google.com"], {
    shell: process.env.shell || true,
    detached: true,
    stdio: "pipe",
  });

  child.stdin.end();
  child.unref();
  console.log(child.pid, process.pid);

  child.stdout.on("data", function (data) {
    console.log(data.toString());
  });

  child.stderr.on("data", function (data) {
    console.error(data.toString());
  });

  setTimeout(function () {
    child.kill("SIGTERM");
  }, 500);

  child.on("exit", function (code, signal) {
    console.log("process exit " + code + " " + signal);
    child.stdout.end();
    child.stderr.end();
  });
}

main();

This hang only seems to happen on SIGTERM as far as I can tell, so a cheap win might be to just do this. I think everything in BigTest comes through wrapped with that. I am testing in BigTest so see if that holds valid.

@jbolda awaiting the fruits of your research with baited breath....

I think the last bit of hangs are related to the socket streams not being cleaned up and Windows not shutting down because of that. Putting this after the World.halt() seems to let the processes close, and get green builds on Windows. It's feels bad and uses a private API, but I don't see any other way to close the connection through effection. Do you have any other ideas?
    // @ts-ignore
    process._getActiveHandles().forEach((handle) => {
      try {
      handle.stdout.end()
      handle.stderr.end()
      } catch (e) {
        // no-op
      }
    });

This is the current diff (which has some formatting changes that I should probably switch back [ie it prettiered]):
https://github.com/thefrontside/bigtest/compare/v0...jbolda:fix/child_process_in_windows
and the PR I've been running CI on (but now I'm getting chrome binary errors after merging v0 in):
https://github.com/jbolda/bigtest/pull/3
I think we get BigTest working (if even with patch-package), and then get the end result from BigTest working in the tests here, release and push that over to BigTest. Unless you think we should do it all directly in BigTest for now?

Still having troubles with the graceful shutdown for windows. I think we get this merged which prevents the extra windows and hanging at the very least (and I think we might need it in regardless even if something like Rust is being considered).

@jbolda I'd like to sit down and have a complete re-think of what we need from a process library. We actually have some other work that's going on which will require some refactoring of the child process supervision resource. I'm going to sit down and document today all of our use-cases, error-handling, and what we need it to do. Then we can approach it from a "what we need" rather than "work with what we have" aspect.

Alright. I think it might make sense to get this merged in as a good first step at least. It takes care of a windows bugs that kill BigTest. We likely will have issues killing nested processes, but it would a) be nice to do a full repro to test and confirm, and b) I don't think it outweighs the benefits to getting this in BigTest as a decent step forward.
For the past week+, I haven't actually been having trying to solve any issues with child_process. It has been dealing with ts-node and it not picking up the config on windows. That is seemingly what is causing all of these tests to time out (since it compiles the generators and never ends up being able to yield properly). My hope was the resolve the last of that in BigTest that I am confident enough with what we have here isn't breaking anything else.
Does that sound like a reasonable path forward? I would like to get windows functioning at least partially before we undertake a (possibly big R&D) project especially if it involves rust.

@jbolda I've migrated the main.test.ts file to use exec. The only requirement will be that you'll have to implement shutting down the process "the windows way" (this might involve invoking taskillfor now) https://github.com/thefrontside/effection/blob/ec6635f1a115bf358748f807b9c572128720b5b3/packages/node/test/helpers.ts#L41-L62
This is a bit of duplication of the actual termination code between posix.ts and helpers.ts, and so there will be some duplication between win32.ts and helpers.ts but I think that's ok. What we're simulating in the TestProcess is what it's like for a user or the OS to shutdown a process from the outside in. Wheer as in our process resource implementation, we're requesting a process shutdown from the inside out.
So I think the process should be:

close this pr
get #199 and #200 merged
implement exec/win32.ts, TestProcess#terminate(), and TestProcess#interrupt() in a separate PR

If you want to get started, I'd base your changes off of the branch in #200

Sounds good. I started a branch a handful of days ago when I posted my comment, https://github.com/thefrontside/effection/pull/195#issuecomment-703206584, but I can merge in from your recent additions. I was sitting tight seeing how you were planning on dealing with the old APIs, and it looks like we are wholesale replacing and deprecating the old things.
RE: splitting things up, I will close this PR and open fresh based on your new APIs. Pretty minimal changes that are easy to port. For BigTest, I will have to check, but I think I've split out everything that isn't directly (or indirectly, eg ts-node not finding paths) related to child_process. I can start a new branch with the changes you have made to BigTest as well.

@jbolda thefrontside/bigtest#591 has been merged so the path should be completely cleared! Now we get to see how terribly our hypotheses perform IRL 🤣

Woo! Nice, I should be able to make some time this weekend to see if I can make some green windows builds.

Closing in favor of #202.


Software rendering doesn't support hardware bitmaps

Software rendering doesn't support hardware bitmaps--when I use BlurTransformation in Android O.

I think you can use a workaround for this, by forcing Glide to always use ARGB_8888 instead of HARDWARE.
public class GlideConfiguration extends AppGlideModule {

    @Override
    public void applyOptions(Context context, GlideBuilder builder) {
        // Apply options to the builder here.
        builder.setDefaultRequestOptions(
                new RequestOptions().format(DecodeFormat.PREFER_ARGB_8888));
    }

}

However, I don't know how to make the SDK use this, as I'm quite new to this.
Can you please let me know how to use it?
This is how I use the image loading with the transformation:
  String url = "https://upload.wikimedia.org/wikipedia/commons/e/e0/Large_Scaled_Forest_Lizard.jpg";
  Glide.with(MainActivity.this).load(url).transition(new DrawableTransitionOptions().crossFade())
                    .apply(RequestOptions.bitmapTransform(new BlurTransformation(MainActivity.this,2)))
                    .into(mInternetImageView);

It should work, right?

@AndroidDeveloperLB  use DecodeFormat.DEFAULT on Android O

@ksc91u Can you please put a full snippet to this? You can use what I wrote if you wish.


AMP <IP_ADDRESS> No such user (Controller/Target)

Bug Report
System Information

Operating System
Debian 11
AMP version and build date
v<IP_ADDRESS>, built 10/09/2022 16:30
Which AMP release stream you're using (Mainline, Nightly or FastTrack)
Mainline

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms

What are you trying to do?
A new user tries the first-time login and needs to set up 2fa and change password on login.
What are you expecting to happen?
The user is able to set up 2fa and change the password
What is actually happening? ('Nothing' is not an acceptable answer!)
The error message "No such user" is presented most of the time. Sometimes it works.

Reproduction

Tested on Controller/Target setup.

Create a new user on the controller.
Set a password for the new user and save.
Give the user "normal" user role.
Only active setting for the new user should be "Require Password Change" and save the changes.
On the controller the setting for 2fa is required.
Try to login with the newly created user.
The user is prompted to set up 2fa, but on next step following is presented most of the time (sometimes the expected screen shows)

I told my user to try it several times, but it hasn't worked for him. So I changed 2fa to optional. After that, the user got the following message.

Luckily, it worked the second time for him.
On previous versions, there was never such a problem. It's properly related to the async changes.

As you found, the workaround for this right now is to disable 2FA until the user signs in the first time.

Also disable required password change for the user as workaround if they have trouble with first login. I don't know if the problem occurs in other situations. Also I don't know if the problem exists also for standalone instances, but should be the same 🤔
Or just try it multiple times until it works 😸

Accidentally tested on AMP v<IP_ADDRESS> :D
Wanted to change my 2FA Device. I disabled 2FA, logged out and on login I was presented with "No such user" (: Even if I got the prompt for 2FA setup with the QR code, after entering the 2FA code from my device, I got again "No such user". Well... I will keep 2FA as optional for now..
If someone else needs to change 2FA from required to optional without WebGUI access, follow these steps:

Stop your ADS instance ampinstmgr stop ADS01
Change inside file AMPConfig.conf the line Security.TwoFactorMode=Required to Security.TwoFactorMode=Optional (WARNING: You have to edit this file as your amp user!)
Start your ADS instance ampinstmgr start ADS01


Code of Conduct based on chef's

We should add our modified version of chef's code of conduct at least to this repo so other repos can link to it

https://github.com/sous-chefs/meta/pull/51

Merged into meta repo


Add hashInHttpHeaders option for Coursier resolver

Add a new resolver option hashInHttpHeaders for Coursier, which tells bazel-deps to use checksums in HTTP headers if possible, instead of downloading the jar artifact and computing hash digests locally. The option is read from the resolverOptions object in the input YAML file.
If this option is true, when computing checksums, bazel-deps will:

First try to make a HEAD request to the artifact.
If the HEAD request was successful:

Save the headers as a JSON file in the Coursier cache directory
If the headers include the necessary checksum, return that.
If the headers don't contain the checksum, fall back to downloading the artifact itself.

If the HEAD request was unsuccessful:

If the status code is 404, but the .pom file exists for this artifact, assume that the artifact will never be published in the future. Cache the error status in the Coursier cache directory to avoid downloading this file in the future. This is the same heuristic that Coursier itself uses.
Return error.

This change reduces the fully-cached runtime of bazel-deps on our internal repo (which uses Artifactory, which supplies both SHA-1 and SHA-256 hashes via headers) from 80s to 7s, and reduces the size of the Coursier cache directory by 99%.
Co-authored-by: Keith Lea<EMAIL_ADDRESS>cc @keithl-stripe

Also note that this is the first bit of serious Scala that I've written, so please feel free to suggest anything stylistic also :)

also it looks like the tests fail to build:
 test/scala/com/github/johnynek/bazel_deps/ModelGenerators.scala:126: error: type mismatch;
 found   : Option[Serializable]
 required: Option[com.github.johnynek.bazel_deps.ResolverType]
    resolverType,
    ^
one error found
Build failed

ideally we could generate Coursier with and without the option set to test the parsing and formatting code.


Can i get the Train and Test Data used to  emotions Facial Recognition Model

📚 Documentation
(A clear and concise description of what the issue is.)
Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Train and Test Data Used to Train Enotions Recognition model

Hi,
We have used a slightly modified version of this dataset. https://www.kaggle.com/msambare/fer2013
Good Luck!


Revise qiskit_bot file post-reorg

Update the list of people to auto-tag in PRs to ensure we aren't creating noise for anyone working exclusively on the enablement side.

The team changes were adjusted and they no longer require updates to qiskit_bot.yaml. https://github.com/Qiskit/documentation/pull/1646 made some other good changes.


Allow to install the library as a CMake project

(Thanks for sending a pull request! Please make sure you click the link above to view the contribution guidelines, then fill out the blanks below.)
What does this implement/fix? Explain your changes.
These changes add the cmake mechanism to install the project so it can be used in a find_package by other projects.
Any relevant logs, error output, etc?
There is one change in this MR which is just commenting some lines causing a linker error on recent clang compilers.  The error is provided in comment
Where has this been tested?
Operating System: Gentoo Linux
I also used fespp to ensure this library is correctly installed.
Other comments:
This PR is related to another MR updating fespp to uses this library as expected (will be provided here when available).

Thanks a lot @CharlesGueunet
I try to look at it deeper next week and wil let you know.

@CharlesGueunet Would it be possible to target the dev master instead of the master one?

@CharlesGueunet Would it be possible to target the dev branch instead of the master one?

Done

I try to look at it deeper next week and wil let you know.

Thank you

Here is a patch trying to resolve my comments above.
0001-Proposal-after-review.patch
I also rebased on origin/dev to check and it is fine.


Fix: Fixed source code in README.

This should fix issue #17.

Coverage remained the same at 100.0% when pulling 0810b61d08b53aadf77ee10533f215c54459d460 on blacksun1:test_markdown into 9a4f2c50fdf1964f1b75aa62a994bc4d233116cb on midknight41:master.

Coverage remained the same at 100.0% when pulling b58439fb21250602819d07c42d25da0f552909e0 on blacksun1:test_markdown into 9a4f2c50fdf1964f1b75aa62a994bc4d233116cb on midknight41:master.

Coverage remained the same at 100.0% when pulling 58b1ca42e24b8ee60498b4236ae10e345370cb34 on blacksun1:test_markdown into 9a4f2c50fdf1964f1b75aa62a994bc4d233116cb on midknight41:master.


Fixed UI bugs and added improvements

added chat scrolling on voice commands
updated modal screen for "Save conversation Log"
minor changes on the general layout like chat height management, scroll distance on the anchor link in "my skills" page and such

You need to rebase on top of current master, it looks like


raw html and no styling

I pulled this repo down to a centos 7 machine and ran the npm install and start commands. When hitting the server via Chrome on the exposed 9966 port all that is returned is the raw index.html file.
Am I missing or doing something wrong? I was hoping to see the leaflet plugin in action with the ORSM map data.
Thanks.

The demo server is currently down, maybe that's your issue. Otherwise, npm start should spawn up a local HTTP server and serve Map + text input + instructions. You can point the frontend to your local routing engine instance (check src/leaflet_options.js and you are good to go.
For a local backend you can either compile it yourself (see osrm-backend repository Wiki for instructions) or use npm install osrm to get NodeJs bindings which also bundle pre-build binaries at ./node_modules/osrm/lib/binding/osrm-*.

Figured it out. I had it fronted by nginx. Looking at our ngnix config i had to uncomment a header security setting "add_header X-Content-Type-Options nosniff;". Now its working.

Perfect!


test fork

Coverage remained the same at 98.214% when pulling 184e5f42c67537c05592d92cd19f4e3c599d802f on GuillaumeBrousse:master into 1bc47215c4496a5e85f5da2a3161f47f45bdc430 on broussegui:master.


Cow: Watch hrchambers.com

Cow requests the watch of the watch_keyword hrchambers\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

hrchambers\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


[WIP] Add support for tracklets

This PR exposes tracklet data to the users. A tracklet is a set of spatially connected alerts (typically belonging to the same exposure) from a fast moving object. This is somehow similar to solar system object, expect that these objects are mainly man-made and they typically orbit around the Earth. The tracklet data can be accessed in different ways:

search by class
search for tracklets

more information at http://<IP_ADDRESS>:24000/api.
Search by class
Tracklets are an object class like the other classes. Hence you can look for these using the class search:

In the API you would just use:
import requests
import pandas as pd

# Get latests 100 alerts belonging to tracklet
r = requests.post(
  'http://<IP_ADDRESS>:24000/api/v1/latests',
  json={
    'class': 'Tracklet',
    'n': '100'
  }
)

# Format output in a DataFrame
pdf = pd.read_json(r.content)

Look for the column pdf["tracklet"] to find out individual tracklets.
Search by Tracklet
You can  also perform a search for tracklets directly. Here you have the choice to specify: (1) a tracklet ID if you know if, or (2) a ZTF night ID, or (3) a date at the format YYYYMMDD.

In the API, you would use the tracklet endpoint:
import requests
import pandas as pd

# Get all alerts corresponding to tracklets in the night 20210810
r = requests.post(
  'http://<IP_ADDRESS>:24000/api/v1/tracklet',
  json={
    'date': '20210810',
  }
)

# Format output in a DataFrame
pdf = pd.read_json(r.content)

Tracklet data display
The tracklet data (if it exists) is displayed on the right most tab of an alert belonging to that tracklet:

Alternatively, you can also display the tracklet when querying the results, using the Sky Map tab:

Todo: see #203

Replaced by #257


Windows long paths support

Refactor directory iteration to avoid replicating the conversion to the
NT style paths everywhere.  The iteration logic can be done at a single
site with a callback to handle the iteration.
Improve handle long file paths on Windows when performing a removeItem
call.  Furthermore, correct a few areas where we were mishandling
junctions.  This would result in an early termination of the loop in
removeItem causing us to fail to clean up directories which we should
have been able to.  This improves the DocC test coverage on Windows.
Rework the file attribute reading on Windows operation to reformulate
the path to the absolute path representation and then into the NT form
before performing the operation.  This technically is a partial repair
as drive relative paths where the current directory is deep enough, the
path evaluation would fail.

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test Windows platform


integrations: Add Zabbix webhook.

@adambirds: I decided to re-submit #9667 as a new PR with a few changes on top. I had to change the documentation's style and structure quite a bit so that it conformed to the integration docs writing guide we talked about earlier.
It would have been very hard for me to communicate the requisite language changes in a review! The rest is the same! Thanks for working on this! Really looking forward to getting this merged! :)
@timabbott: Could you please merge this? :) Thanks!

Looks great, merged, thanks @eeshangarg and @adambirds!

Excellent. Thanks @eeshangarg for doing that, having looked at your changes I now understand the way that it should be better. Couldn't seem to find an example plugin, with this much manual work involved to sort of copy off. Thanks to Zabbix not having built in webhook support, the docs have to be a lot bigger than be could be. The future integrations I plan on working on should have a lot less docs than this one thankfully. And thanks @timabbott for merging. Look forward to seeing this in the next release.


[Bug] Trying to get in touch with you LIHU

Version
1
Steps to Reproduce
Hey, hi Lihu, hope you are fine this is GoCharting.
Are you available for a quick call with us?
Been trying to get in touch with you after we last spoke.
hope all is well.
Current Behavior
1
Expected Behavior
1
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

@angadgocharting how about you go maintain/fully document your react stockcharts library instead of trying to quell the potential of having private and/or os competition?

@angadgocharting how about you go maintain/fully document your react stockcharts library instead of trying to quell the potential of having private and/or os competition?

Firstly who are you and why this irrelevant comment when you dont know the context?  When two open-sourcing project contributors are trying to collaborate on another open-source project, what value are you adding with such comments?  It will add immense value to the dev community as we see it.  So unless you have something constructive to add to the open-source community, suggest you stay out of it.

@angadgocharting

why this irrelevant comment when you don't know the context?

Not irrelevant. These inquiries of yours (a private entity -> gocharting.com / gocharting app), requesting private communications in this repo's issues, in combination with the years long abandonment of react-stockcharts, gives off the sense of an anti-competitive agenda.

(which from your profile is clear that your own contribution has been minimal)

As is clear from your profile as well.

When two open-sourcing project contributors are trying to collaborate on another open-source project, what value are you adding with such comments? It will add immense value to the dev community as we see it. So unless you have something constructive to add to the open-source community

Then why not openly post your inquiries for all to see in the dev community, rather than in private? Can we know how this potential collaboration would benefit the dev community? What's this "immense" value you speak of that cannot be posted publicly?
Is this about the "dev" community or the open source community? Your use of the term "dev community" is vague and could very well be interpreted as "a private api and/or library that gocharting and/or <insert private entity and/or partnership> offers to developers at a cost".
It's suspicious considering that you're reaching out to the dev of a charting library which trumps react-stockcharts in base quality, interactivity, performance, etc, and enables others to compete against the business interests, revenue, and market share of gocharting.com and the gocharting app. Not to mention the lack of documenting and maintaining of the react-stockcharts repo ever since pivoting to focus on private endeavors. Why jump back into (supposedly) contributing to open source charting/TA projects now, when that could have been done in the years since abandoning react-stockcharts?
In addition to your lack of transparency, your motives are questionable seeing that you're using a handle that is in reference to your private business rather than @rrag or an account related to @rrag / react-stockcharts. Years of radio silence from @rrag, but now there's activity from an account whose name references his private business and appears to only be used to reach out to projects which compete with their business's interests/bottom line?
Sorry, but that's sus and infuriating to think that it could be an attempt to disrupt the development of this awesome project many seem to be flocking to.


disturbing alert when adding another folder to a workspace

[I'm filing this issue as a long time VS Code user, not as a dev who understands what's going on...]
When trying to add a new folder to an existing workspace, I get the following alert:

As a long time VS Code user, I am a bit surprised by the wording because in my current understanding I have already a "workspace" and I cannot really understand why the alert says that I do not have a "workspace".
Could we change the wording so that it sounds less scary, e.g. something like:
"Before we can add another folder we need to upgrade your workspace to support multiple folders..."

Agreed that above dialog is not helping in understanding the transition from Opening a folder to Opening a multi folder workspace. I have changed the wordings and behaviour today so that the existing or new user understands what is happening.
@weinand Please verify with the insiders tomorrow and let us know if it makes sense to you.

IMO the new wording is even worse:

Adding "window" to the wording doesn't help.
I still believe that the wording should be more along the lines of this:
"Before we can add another folder we need to upgrade your workspace to support multiple folders..."

I missed why there is a confirmation dialog at all. Why not just "upgrade" silently?

We cannot upgrade without window reload due to the fact that we have to migrate some state in the background for which we are not ready yet. Hence there is a confirmation dialog.
Upgrade without reload is on the cards for future.
Currently shifting from 1-2 is heavy weight operation of creating a new workspace and opening it.

We do not upgrade yet so that I can have some nice vacation without being worried.

Agree with @weinand. We should not use phrasing that makes you too cautious about loosing your working state, such as "not supported", or restrictive sentence as "you can do only ...". We just need to inform a user on what will happen.
@weinand proposal sounds good to me

"Before we can add another folder we need to upgrade your workspace to support multiple folders..."

"Adding a second folder requires a one-time reload of the workbench so @bpasero can go on vacation without worries?"

I am happy to have suggestions on the wordings in the dialog here. What we are trying to convey to user is that any multi root operation on a current workspace (opened with a folder) needs:
Creating a multi root workspace with current folder in the background, do the operation and open it, which needs window reload.
I accept suggestion from @weinand and the new dialogs shown as follows

I would have the focus on reloading the window, since that's more interesting to the user. Maybe if the button said "Reload and Add". As a user I'm not sure what upgrading is going to do or if it's going to be reversible.

Btw here is my comment on the famous multi root issue why we currently require an explicit step when entering workspace mode: https://github.com/Microsoft/vscode/issues/396#issuecomment-315402932

@weinand Today's insiders have above dialogs. Please reopen if you think they are not good enough. Thanks.

I think the wording is fine now but there is still the typo "updgrade"  (see your screenshot above).
How important is it to mention the "reload the window"?
There will be no data loss, right?
So on a regular "Open..." we do not show a "reload window" warning either.

Will fix the typo.
As mentioned earlier, we do not retain any state (UI, workspace settings, extensions storage and hot exit). It means after reload user will see a different workspace, opened editors dirty files are not retained. If there are workbench settings like different themes and colours in the workspace, these will not be retained too.
In Open... case, user is explicitly asking to open a new workspace. But in this case, I am not sure if we are clear enough in the dialog that upgrading is opening a brand new workspace.

Ok, I see. So leave the "reload window" in.

saw this while testing #30722

I react negatively to the "upgrade your workspace" portion of the message. This sounds like the tool is going to do something to my files and it scares me. Can I revert whatever is going to happen? Is this a one way street? It is very daunting :)
Have we considered providing an info message after we restart the environment? We could provide a "more info" link where we can describe what is different now that you are in MRW mode.

Are you also scared when VS Code offers you an "upgrade"?
Would "prepare" sound less scary than "upgrade"?

something like this...

Or, we can really simplify the message to:
"To open multiple folders, VS Code requires a reload."
As a user, what else do I need to know?

Yes, this is in line with https://github.com/Microsoft/vscode/issues/31014#issuecomment-316396043

Liked the suggestion from @chrisdias. It is simple and short. Will change to it. Thanks.

+1, the new flow is very cool.
@weinand @sandy081 @chrisdias my only concern now is that it is not just a window reload that we are doing compared to a normal window reload: UI state, hot-exit (dirty files) and extension state is lost after the window reload. I can look into what it would take to keep and restore these things after the reload. E.g. as a user I would not expect to be asked to save files only for a window reload (we do not ask this for a manual window reload either).


meta: exit process if no setting found

Codecov Report

Merging #1359 (4050e46) into main (5c6d557) will increase coverage by 0.21%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##             main    #1359      +/-   ##
==========================================
+ Coverage   59.41%   59.63%   +0.21%
==========================================
  Files         128      128
  Lines       21663    21681      +18
==========================================
+ Hits        12872    12930      +58
+ Misses       7254     7207      -47
- Partials     1537     1544       +7

Impacted Files
Coverage Δ

pkg/meta/redis.go
71.67% <0.00%> (+0.16%)
:arrow_up:

pkg/meta/tkv.go
76.10% <0.00%> (-0.09%)
:arrow_down:

pkg/meta/sql.go
67.67% <25.00%> (-0.09%)
:arrow_down:

cmd/bench.go
78.78% <0.00%> (-1.35%)
:arrow_down:

pkg/chunk/disk_cache.go
78.32% <0.00%> (-0.50%)
:arrow_down:

pkg/chunk/cached_store.go
73.18% <0.00%> (+2.42%)
:arrow_up:

cmd/mount_unix.go
54.23% <0.00%> (+6.77%)
:arrow_up:

cmd/gc.go
64.76% <0.00%> (+15.84%)
:arrow_up:

Continue to review full report at Codecov.

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

Force quit the process is way too dangerous


fix: update find commands to support GNU and FreeBSD flavors

The arguments to GNU find and FreeBSD (MacOS) find are slightly different. This PR orders the options so that GNU find is happy. It also only uses options that are available in both flavors.
Closes #800.

@joprice Could you check if this fixes #800?

Just tried it and it works great!


Added right-click on nickname to reply on whisper.

I improved work of the right click on nickname a little.
Now, if you right click on user that whispered to you, /w is automatically added to input field to answer him.

(And finally dynamic_cast is collapsed to condition!)

Looks good to me, thank you sir!


Artifact Weapon iLvl

The item level on the artifact weapons are wrong.
Wondering if i missed something but thought id post it here.

Just want to point out that #10 reports that it has some issues with (my guess) legion items :P (just helping out Cogwerkz since it must be bothersome to have multiple reports of the same thing)

My bad, overlooked that post.
I ussually disenchant or sell the green items without paying attention to them.

fixed in 1.0.9


Check Access, Gid, Uid of /opt/app-root/lib and /opt/app-root/shares  [ODS-486]

Signed-off-by: Pranav Suryawanshi<EMAIL_ADDRESS>

https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/580/console
Result : Passed


Merge pull request #2 from dannymiracle/readme-edits

Merge pull request #1 from dannymiracle/main

Attempting pull request for the hello-world repository for Code KY


Pip-compile fails on sub-directory as find-links

Pip-compile not able to take subdirectory as find-links
Environment Versions

OS Type - macOS
Python version:  python 3.6.8
pip version:  pip 20.0.2
pip-tools version: pip-compile, version 5.2.1

Steps to replicate

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/piptools/scripts/compile.py:297: FutureWarning: --index and --no-index are deprecated and will be removed in future versions. Use --emit-index-url/--no-emit-index-url instead.
  category=FutureWarning,
WARNING: Requirement 'funcsigs==1.0.2-py2.py3-none-any.whl' looks like a filename, but the file does not exist
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/bin/pip-compile", line 11, in <module>
    sys.exit(cli())
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/piptools/scripts/compile.py", line 405, in cli
    options=repository.options,
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/piptools/_compat/pip_compat.py", line 29, in parse_requirements
    yield install_req_from_parsed_requirement(parsed_req, isolated=isolated)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/pip/_internal/req/constructors.py", line 462, in install_req_from_parsed_requirement
    line_source=parsed_req.line_source,
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/pip/_internal/req/constructors.py", line 396, in install_req_from_line
    parts = parse_req_from_line(name, line_source)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/pip/_internal/req/constructors.py", line 337, in parse_req_from_line
    wheel = Wheel(link.filename)  # can raise InvalidWheelFilename
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/pip/_internal/models/wheel.py", line 33, in __init__
    "{} is not a valid wheel filename.".format(filename)
pip._internal.exceptions.InvalidWheelFilename: funcsigs==1.0.2-py2.py3-none-any.whl is not a valid wheel filename.

#Expected result
It will write requirements.txt.

@pombredanne: This is our major issue but this can solved by moving all wheel thirdparty/dev to thirdparty . So no worries .

@atugushev : Please solve this bug ASAP

@Abhishek-Dev09 PRs are welcomed 👍

@Abhishek-Dev09
I believe the issue is in command, please try this instead:
$ pip-compile --annotate --generate-hashes --find-links thirdparty/dev --no-emit-find-links --pip-args "--no-index"

@Abhishek-Dev09

Please solve this bug ASAP

In the meantime, please, try to avoid phrases like "fix this bug ASAP" in open source. pip-tools is free/open-source software maintained by a non-profit organization and a group of volunteer users. It's rude to ask people to fix urgently a bug for free.
Here is a good article about open-source etiquette you might find useful. Thanks!

@atugushev : Sorry for using that phrases. I have updated the comments that i found bug.

pip._internal.exceptions.InvalidWheelFilename: funcsigs==1.0.2-py2.py3-none-any.whl is not a valid wheel filename.

Looks like you have funcsigs==1.0.2-py2.py3-none-any.whl in your requirements.in. It should be funcsigs==1.0.2.

@atugushev i have already tried ur solution before .

Could you attach your requirements.in?

I m sure , my requirements.in is 100% correct.


feat: Add ImageDataset and Layer for ConvolutionalNeuralNetworks

Closes #579, #580, #581
Summary of Changes
feat: added Convolutional2DLayer, ConvolutionalTranspose2DLayer, FlattenLayer, MaxPooling2DLayer and AvgPooling2DLayer
feat: added InputConversionImage, OutputConversionImageToColumn, OutputConversionImageToTable and OutputConversionImageToImage
feat: added generic ImageDataset
feat: added class ImageSize and methods ImageList.sizes and Image.size to get the sizes of the respective images
feat: added ability to iterate over SingleSizeImageList
feat: added param to return filenames in ImageList.from_files
feat: added option None for no activation function in ForwardLayer
feat: added Image.__array__ to convert a Image to a numpy.ndarray
feat: added equals check to OneHotEncoder
fix: fixed bug https://github.com/Safe-DS/Library/issues/581 in removing the Softmax function from the last layer in NeuralNetworkClassifier
refactor: move image.utils to image._utils
refactor: extracted test devices from test_image to helpers.devices

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

v0.24.0
GitHub release

Your semantic-release bot :package::rocket:


npm link runs install scripts

The npm link [...] command runs the underlying npm install [...] command. However, in doing so, it also fires all the pre-install, install, and postinstall hooks that are build in to the package.json file...
I'm writing a script that is meant to be installed globally (because it runs a daemon), but then linked to each project. However, when linking, it runs the install hook again and starts a second daemon. Yeah, I could modify my code to check if the daemon is already running, but having link behave the same as install doesn't seem reasonable to me.
Additionally, if this is agreed, it makes sense to add hooks for prelink, link, and postlink, as it's very possible they are different operations...

In the vast majority of cases, you do definitely want to run install scripts the first time, when the package is linked into the global space (or installed implicitly).
However, you're right, it is unnecessary to run them again when it links it into the local space.  This is a bug.

Awesome :). I look forward to seeing an innovative solution,

I'm going to close this as I'm no longer using NPM or working on this project...

fixed in #8501


[New Port Request] libcap

Library name
libcap
Library description
No response
Source repository URL
https://git.kernel.org/pub/scm/libs/libcap/libcap.git
Project homepage (if different from the source repository)
No response
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
No response

Bump.

libcap was added together with libsystemd on September 20.

Fixed by PR https://github.com/microsoft/vcpkg/pull/31150


Compilation fails in branch kutsaev_2021b

The build fails with these error messages along the way:
src/physics/Functions.h: In function ‘AnsiString GetFileName(AnsiString&)’:
src/physics/Functions.h:415:11: error: ‘class AnsiString’ has no member named ‘SubString’
415 |  return F.SubString(F.LastDelimiter("/\")+1,F.Length());
|           ^~~~~~~~~
src/physics/Functions.h:415:23: error: ‘class AnsiString’ has no member named ‘LastDelimiter’
415 |  return F.SubString(F.LastDelimiter("/\")+1,F.Length());
|                       ^~~~~~~~~~~~~
src/physics/Functions.h: In function ‘AnsiString GetFileCaption(AnsiString&)’:
src/physics/Functions.h:420:11: error: ‘class AnsiString’ has no member named ‘SubString’
420 |  return F.SubString(0,F.LastDelimiter("/.")-1);
|           ^~~~~~~~~
src/physics/Functions.h:420:25: error: ‘class AnsiString’ has no member named ‘LastDelimiter’
420 |  return F.SubString(0,F.LastDelimiter("/.")-1);
|                         ^~~~~~~~~~~~~
[...]
src/physics/BeamSolver.cpp: In member function ‘TResult TBeamSolver::Output(AnsiString&)’:
src/physics/BeamSolver.cpp:5496:2: error: ‘Memo’ was not declared in this scope
5496 |  Memo->Lines->SaveToFile(FileName);
[...]
make: *** [makefile:35: lib/BeamSolver.o] Error 1

This problem is being addressed via Issue #110.

Closing as redundant.


Assigning Application User did not work

During "Verify requirements and grant access" point 4 the assignment of Application Users in Dataverse step did not work following the steps. The problem was that the new UI to add the Application users will use the graph applications API which returns AAD Application Objects but the applications are not present as such. They can be retrieved as SPs. The "legacy" page still works or the systemusers API of CRM. There are also 3rd party blog posts which explain legacy creation approach but would be good to follow the docs site from start to end.

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

ID: df8e9efc-9258-3876-e05f-20d35c907274
Version Independent ID: 4b5041ec-4a48-0565-e47a-b972e65a6d60
Content: System requirements and prerequisites - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/data-entities/dual-write/requirements-and-prerequisites.md
Service: dynamics-365-cross-app
GitHub Login: @NHelgren
Microsoft Alias: nhelgren

@d3fr0g, thank you for providing this feedback.
@NHelgren, please update the topic as appropriate.

I will review

Bug 647691 has been logged to track this issue.

This has been logged and a work item has been assigned to the PAuth User Sync team.  See ICM Incident-277906449 Details - IcM (microsofticm.com) to


Movement in map

The player should be able to actually move around in the map

This requires issue #14 to be finished

solved


extract_adapters: plugin docs generation fails when hook is present

Fixes #1643
Summarize your change.
This change cleans up the code a bit and makes sure it ignores hooks plugin types, as they are not getting the same output by plugin_info_map() needed for type formatting.
Reference associated tests.
test_plugin_documentation (test_serialized_schema.PluginDocumentationTester)

Codecov Report

Merging #1644 (f402941) into extract_adapters (6ac5e90) will decrease coverage by 0.01%.
The diff coverage is 88.88%.

Additional details and impacted files

@@                 Coverage Diff                  @@
##           extract_adapters    #1644      +/-   ##
====================================================
- Coverage             74.59%   74.59%   -0.01%
====================================================
  Files                   175      175
  Lines                 11977    11980       +3
  Branches               2732     2734       +2
====================================================
+ Hits                   8934     8936       +2
- Misses                 1293     1294       +1
  Partials               1750     1750

Flag
Coverage Δ

py-unittests
74.59% <88.88%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

...timelineio/console/autogen_plugin_documentation.py
64.40% <88.88%> (-0.82%)
:arrow_down:

... and 1 file with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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

So the build error is legitimate.  In the extract_adapters branch, rationalTime.h is missing a #include <cstdint>, from this commit in this PR by @JeanChristopheMorinPerso -- #1605 .  I think a merge or a rebase would solve this.  @apetrynet @reinecke how do you want to handle this?  Could one of you merge or cherry pick into extract_adapters?  After accepting that fix, then this PR should build correctly.

If @reinecke doesn't beat me to it, I can attempt a cherry pick

@jminor rebased!


Opinion: Can chalk be removed?

Issue Summary
I've noticed that the chalk package is being included in my production builds. This is adding an unnecessary 20kb extra to  the deployed codebase.
Its being included here:
https://github.com/sendgrid/sendgrid-nodejs/blob/main/packages/helpers/classes/response-error.js
Although I've not had to make use of it in the past, i think the logic inside the toString method is valid but i don't think making the error colourful is,  so i'm proposing that it should be removed.
Technical details:

sendgrid-nodejs version: 7.2.6
node version: 14

I agree.
This issue has been added to our internal backlog to be prioritized. Pull requests and +1s on the issue summary will help it move up the backlog.

nice one, thanks @nck @childish-sambino


Add default image to Gravatar image lookup

When we lookup gravatar images we can use d=url as a parameter to default to our own image instead of the gravatart logo
http://gravatar.com/avatar/6dc4a43fdaf5dfe0c69e3ed54ade665f?s=140&d=http://arquillian.org/images/404_ike.png

Shall we do it? I've introduced a fallback to fetch GitHub images in case of default gravatar. With their generated "qr-code like" avatars it might be a bit harder to figure out what we are dealing with.
The change was introduced @2dcaf7cafe97f9acf2a9f32520812a93103b3a12 and is only available for production build, as it takes a bit more time :)


Validate extractor and loader configuration on save

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1238
Originally created by @gtsiolis on 2019-11-01 16:18:10

Problem
Currently, you can configure an extractor or a loader without validating input data on save. This can create confusion and cause a pipeline error after configuring all these steps (load, extract, transform) which can be also frustrating to the user.
Proposal
Let's validate plugin configuration on save. :elephant:
Design
TBD

View 6 previous comments from the original issue on GitLab


Run from Docker

Hi. Thanks for this excellent piece of code. I've added a docker container to make it easy to get it up and running in a containerised environment and submit for your consideration. Readme has also been updated to reflect how to build and run it.

Hi @blackrat! Thanks for the contribution.
I just pushed out some changes to pre-compile babel/browserify, add a CLI utility, and make the package easier to use via npm. This means that npm run dist is now required as part of the build process, and that we can run npm prune --production right after it to reduce the size of the docker image by removing the devDependencies. The application should then be started with ./dist/cli.js instead of npm start (which is used for starting the development server).
I'd also really like to see a docker-composer.yml file for setting this up together with Lightning Charge's Docker image. Getting everything up & running with a single docker-compose up command would be sweet!
Do you think you could look into making these changes, or should I go ahead and make them?

I may get time to look at this in the coming week or two. Shouldn't take too long to spin it up. Do you have a wrapper project you want to do it from with these as children?

Closing in favor of #9.


Build arm64 desktop app documentation

Being new in flutter I am trying to figure out how and if it is possible to build a flutter arm64 desktop app.
Found a few issues and solution though none had clear details on how to build an arm64 desktop app.
Please advise, Is it possible and if so which command does compile arm64 as also a universal app?

Hi @cliqer
Current targeting arm desktop app is not supported
https://github.com/flutter/flutter/issues/60678
https://github.com/flutter/flutter/issues/60113
When this is supported, documentation will be available
Thank you


Improved jsdocs and documentation

travis failed

Coverage increased (+0.04%) when pulling 0f7b1a41fe1c9482a9e411805194f0c84a0d6e04 on eordano:docs/improve into b5626be2b49912b7d4342057437024467f896528 on bitpay:master.

Coverage increased (+0.04%) when pulling 0f7b1a41fe1c9482a9e411805194f0c84a0d6e04 on eordano:docs/improve into b5626be2b49912b7d4342057437024467f896528 on bitpay:master.

ACK


Enemy AI

Designing Enemies using Abstract Classes

[x] Waypoints
[x] Moving/Ideling
[ ] Atacking
[x] Enemy Types

[x] Moss
[x] Spider
[x] Skeleton

https://ktmarine1999.medium.com/designing-enemies-4ba43eba3ef9
All enemies created. Animations Created. Animator Controller Created.

Enemy Detects if the Player is in front of them , Will Go into Combat Mode.

Enemy Detects Player in Front and Behind them if within Combat Range., Turns to face the Player, and attacks. Once the Player is out of attack range the Enemy will continue its movement.

Combat Check Offset added, Moss Giant need it to be moved closer to the feet.

Damageable Code added to Moss Giant. Combat Distance set.

Attack Added To Moss Giant.

Spider Attacks.

Death Logic Added.


[feature] Integrate to Flink 1.17

Fix column comments.
Integrate alter table column
Integrate DELETE
Integrate UPDATE
Implements SourceReader.pauseOrResumeSplit

I'm on the way of this feature, can we share this? @SteNicholas


Is RNGCryptoServiceProvider is FIPS 140-2 complaint?

Is RNGCryptoServiceProvider is FIPS 140-2 complaint? Please Kindly provide feedback for this.

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

ID: e93f07bf-c0dd-dd83-d7bf-8562e7f197c3
Version Independent ID: a6c44187-8ff1-6519-5291-146b4c1bf46c
Content: FIPS 140 Validation
Content Source: windows/security/threat-protection/fips-140-validation.md
Product: w10
GitHub Login: @DaniHalfin
Microsoft Alias: daniha

Hello @GauravLad12 , according to this article, only classes that end in "CryptoServiceProvider" or "Cng" are allowed by FIPS and FIPS-2, so yes, it is complaint.

@officedocsbot assign @e0i

Hello, @GauravLad12
It seems that your question has already been answered by the community and thus we are marking this issue as resolved.
Feel free to re-open the issue if you feel it hasn't been resolved.
Thank you for providing feedback.

@officedocsbot close


net.tomp2p.storage.TestStorage.testResponsibility: NullPointerException

in TestStorage.java, line 206:
	Assert.assertEquals(2, storage.findContentForResponsiblePeerID(locationKey).size());

as called from
@Test
public void testResponsibility() throws Exception
All other tests in TestStorage passed.
Occurred during a Jenkins build, Maven 3.2.3, JDK 8, on Ubuntu 14.04 LTS; storage was 2 SSDs in RAID-0 config.

Possibly a Jenkins problem. Test passes when run from the command line in the Jenkins job's workspace, i.e. with exactly the same configuration, bytecode and storage environment.

In the latest Beta-3, I changed the storage of resposibility. I think this works (no bug anymore) and I'm closing this issue. If the bug is not gone, please reopen.


 It was working fine until yesterday when it just stopped. Please refer to my command line below to see the steps I tried to take to fix the problem.
This is high priority for us as we have quite some management to do on our farms in the coming days and weeks, and this ThreeBot is the only one that can do it.
The following commands result in errors:

threebot browser
threebot restart
threebot stop
threebot start

@rkhamis That's great news.
Should I just wait for the next version? When do you think it will be out?
Or should we try to fix this on my machine with the current version? I definitely don't want to be without access to our farm management for too long.
I'll be more careful to stop all containers before a reboot, though rarely the machine reboots of its own accord (eg. when doing a mac os update). Should I turn off automatic updates? If so, it might be an idea to put this warning in the documentation.

@joefoxton I'll create a hotfix now and ask you to update.
you needn't turn off the automatic updates on your Mac then

Awesome. Thank you.

On 21 May 2020, at 11:07 AM, Reem Khamis<EMAIL_ADDRESS>wrote:

@joefoxton I'll create a hotfix now and ask you to update.
you needn't turn off the automatic updates on your Mac then
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@joefoxton what version of the 3sdk are you using?
it's already part of the released 10.5.3
https://github.com/threefoldtech/jumpscaleX_threebot/issues/674

I'm using 10.5.3

I tried again, showing the full flow...

Another symptom... when I start 3sdk, it looks like it launches two identical processes. Is this right?

hotfixed on master.


test(transformer/class-properties): override fixtures

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7689  👈 (View in Graphite)
#7688
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


GUACAMOLE-107: Correct references to old pre-Incubator site and resources.

Looks good to me! :shipit:


Export-csv not working

Export-csv does not appear to be working. Tried in several different browsers. Also, in jsfiddle: http://jsfiddle.net/highcharts/cqjvD/ demo the option is not appearing in the drop-down menu.
I had this working fine in my code before, not sure what changed.

It's fixed now.
The problem was only with our jsFiddle demos. We changed our user name from highslide-software to highcharts, so the script tag needed to be changed from
<script src="http://highslide-software.github.io/export-csv/export-csv.js"></script>

to
<script src="http://highcharts.github.io/export-csv/export-csv.js"></script>`


Fix and/or

fix for and or methods lacking initial 'and' and 'or'

@yisraelU I think this one has some overlap with an earlier PR of @zagyi. Can you merge conflicts and see if there's anything left to do in this pull request? Thank you! 🙏

@yisraelU I think this one has some overlap with an earlier PR of @zagyi. Can you merge conflicts and see if there's anything left to do in this pull request? Thank you! 🙏

I'll check that out. Thank you

Please feel free to re-open if you get a chance to look at the conflicts / overlap! 🙏


Getting errors when trying to run a lammacpp model

this model works normally when running server.py
The model loads and the bot connects but when I try the reply command it fails with the following error.
Traceback (most recent call last):
File "/home/user/miniconda3/envs/textgen/lib/python3.10/site-packages/discord/app_commands/commands.py", line 842, in _do_call
return await self._callback(interaction, **params)  # type: ignore
File "/home/user/text-generation-webui/bot.py", line 179, in reply
await ll_gen(ctx, queues)
File "/home/user/text-generation-webui/bot.py", line 113, in ll_gen
for resp in chatbot_wrapper(**user_input):
TypeError: chatbot_wrapper() got an unexpected keyword argument 'check'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/user/miniconda3/envs/textgen/lib/python3.10/site-packages/discord/ext/commands/hybrid.py", line 438, in _invoke_with_namespace
value = await self._do_call(ctx, ctx.kwargs)  # type: ignore
File "/home/user/miniconda3/envs/textgen/lib/python3.10/site-packages/discord/app_commands/commands.py", line 856, in _do_call
raise CommandInvokeError(self, e) from e
discord.app_commands.errors.CommandInvokeError: Command 'reply' raised an exception: TypeError: chatbot_wrapper() got an unexpected keyword argument 'check'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/user/miniconda3/envs/textgen/lib/python3.10/site-packages/discord/ext/commands/hybrid.py", line 438, in _invoke_with_namespace
value = await self._do_call(ctx, ctx.kwargs)  # type: ignore
File "/home/user/miniconda3/envs/textgen/lib/python3.10/site-packages/discord/app_commands/commands.py", line 856, in _do_call
raise CommandInvokeError(self, e) from e
discord.ext.commands.errors.HybridCommandError: Hybrid command raised an error: Command 'reply' raised an exception: TypeError: chatbot_wrapper() got an unexpected keyword argument 'check'

Looks like its unrelated to the new model and just prior changes to code. I change line 100 from:
user_input["check"] = False
to
user_input["stop_at_newline"] = False
and it runs now.

@keninishna hey thanks for the bug report. I've fixed this bug and some others that have been created from the updates webui has made. It should run correctly rn.


wrong order of inputs

after doing what is mentioned in the docs to Login with Username or Email, the username and email moved after password and password_again fields
how can i fix this?

See #141

I've updated the Guide acordingly.
Tnx!


feat: Feature Flagging Default Traits

Description

This PR sets the default traits for the users in the same SDK call to get features.
This ensures the traits are present for all users and also the SDK calls are contained.

Fixes #25159
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

New feature (non-breaking change which adds functionality)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[x] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test


Fixing the testing units related to UTC timezone

It's related to issue #51
Replacing cases like this:
final var now = new Date(1545047719295L);

to
final var now = Date.from(LocalDateTime.of(2018,12, 17, 12, 55, 19, 295 * 1000000)
                .toInstant(ZoneOffset.UTC));

The change would not render the test unusable and would not cause any problem, but the predictability of the test would be lower. In this version, as supposed by this pull request the time value used in the test would depend on the time zone the test is running in. Since the actual time value does not make any difference it would and should not make harm, but

the test stability would be less
this test, as documentation, would increase the confusion that caused this pull request in the first place.


remove async keyword argument

because it is a reserved word in python 3.7

Codecov Report

Merging #182 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #182   +/-   ##
=======================================
  Coverage   89.06%   89.06%
=======================================
  Files          30       30
  Lines        2186     2186
=======================================
  Hits         1947     1947
  Misses        239      239

Impacted Files
Coverage Δ

scvi/inference/posterior.py
83.86% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Adding link in README.md to Solaris configuration.

This PR just adds a link to Solaris configuration in the README.md missed by initial PR,
"Introducing Solaris in OCI" , #411
Signed-off-by: Abhijeeth Nuthan<EMAIL_ADDRESS>

LGTM


Review config for update-state

Need to review implicit queue config for the update-state lambda

I looked into this a bit and found the following:

How to specify a DLQ is unclear with the old version of serverless we are using
EventBridge by default will retry events 185 times (for retriable errors) and will keep events for 24 hours
It is unclear to me how frequently it will retry through those 185 attempts
If we do miss an event for some reason, the result will be a hanging cirrus job (stuck in PROCESSING)

It seems that a prerequisite for specifying a DLQ is upgrading our version of serverless (or replacing serverless). That or we could convert the events on the update-state function to be pure cloudformation, and not use the serverless event templating. If we were to do the latter, it points to a need to allow arbitrary cloudformation resources on lambda definitions, similar to how we support cloudformation on batch tasks.
While I believe we need to upgrade and/or replace serverless, and I think lambda cloudformation support makes sense, given the effort involved with the above relative to the need with regard to update-state, I have to question if we really need to be concerned about an issue here. That is, the worst case here is a job stuck in PROCESSING, which we could rather trivially detect by comparing jobs in PROCESSING for a long duration with the step function execution state. Adding a DLQ only makes us aware of this condition by the presence of a message in that DLQ assuming the DLQ actually works, which if the lambda doesn't I'm not sure if it is a given that the DLQ would.
I would therefore suggest we not worry about the configuration here. The defaults seem sufficient for >=99.99% of executions. I would recommend, if we were to do anything, to add some way to verify jobs in PROCESSING against their executions as an out-of-band way of detecting problems.
@matthewhanson how do you feel about closing this issue without any further action?

Closing this out for now as we are okay with the current behavior. If we have issues in the future we will create a new issue to address them.


refactor(Inbox): indexed message tree

Unrelated to the inbox update itself but more to supporting public reads with the AVM, we will likely need to alter the tree to be a indexed merkle tree.
Also, we should likely be show the message fetching mechanism a bit more love (#697)

Include the index in the tree as part of the leaf hash. For example leaf = H(index, message.hash()) If we are doing this, we could alter the event at the Inbox` to be
event LeafInserted(uint256 indexed blockNumber, uint256 index, uint256 totalIndex, bytes32 value);

As the local storage can then have both the values and the totalIndex which is where it is in the message tree. So it is still the same API for the oracles to prove membership inclusion.

This might not be needed. Depending on what the AVM requires, could be done without going indexed.


how to use 3070ti to run nice-slam

I run NICE-SLAM on a short ScanNet sequence with 500 frames, following the readme.md, which notes that It takes a few minutes with ~5G GPU memory. Although I have 8G GPU memory, but it occured the error

RuntimeError: CUDA error: out of memory

so how can i adopt the hyperparameters to fit my configuration?
I have seen other issues, which you notes that your classmates have run successfully in 1080ti. I think my configuration is enough, right?

Have you tried to run with the demo sequence that we provided? Is that fitting in your memory? If yes, what config do you run your ScanNet sequence? Maybe you can see the difference between the two config files?
BTW, 1080ti has ~11G GPU memory and I don't think we have tested on any GPUs with 8G memory.
Best
Songyou

alright, maybe I need to put this project on hold until I update my hardware
I run the results of Nice-SLAM which you provide successfully, but I can not run Nice-SLAM demo

Maybe you can try out some GPU lease websites, e.g. AutoDL, Amazon AWS. Then you can choose the hardware setting.
Good luck!

thanks, I am a beginner, and I thought that only deep learning model training could run on the server.
This is good news for me.


Nuget Package

Is there any chance to get a nuget package published either on nuget gallery or github packages?
Looking to use this in a mod but noticed its the only one i have to download and reference manually. Would be a huge help to just have  a package for version control.

Wish granted!
https://www.nuget.org/packages/LethalSettings/1.4.0


Determine header components better

Fixes #182

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!

@borisyankov I added a fix in #216 as well

I'm going to close this because #216 also fixes it


Feature/lac deserializer for JSON events

I found it impossible to process a poisoned JSON formatted event as by the time the client/poll! function received the exception, the events had been lost. So I added a Log And Continue JSON deserializer that as the name says, in the case of one of the events being un-parseable, logs an error message using the clojure.tools.logging package, and returns a {:json-deserializer-error: "Bad Message Detected" :message { (String. payload)} map back. This way the consumer can detect a poison message and continue processing, or if it choses, abort processing.
I could not think of a reason why the payload could not be represented as a string in the :message tag, but I could be wrong.

Hi @fhitchen, I feel like this is a better candidate for an ad-hoc deserializer implementation

I agree, what I did was not very useful and a protocol based serializer/deserializer that can be modified externally to kinsky would be a better approach.


past events issues

Day1 schedule not exported for the 2023-06 event (need to pick an older commit by a day or so before i commented it our)
a refresh on the 2023-06 pages for speakers that work don't load, because the pages are not speaker_name.html, but just speaker_name - they do load the first time though (like https://conf.techmids.io/past/2023-06/speakers/lin-sun but it wont load from here)

This requires the static build to generate all the speaker pages at build time. I think this involves adding getStaticPaths to the Speaker component, which should read speakers.js, but I haven't fully worked this out yet. (Anyone who knows nextjs more intimately would probably understand this easily enough if they want to jump in!)

this was implemented (getStaticPaths/Props) in main, https://github.com/techmids/conf.techmids.io/blob/main/src/pages/speakers/[speaker].jsx#L40
not sure how we could back-port


ServerStyleSheets does not propagate props to StylesProvider component.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

I'm trying to add Material-UI to my application in Next.js. I started from checking out example that you have in your official docs: https://github.com/mui-org/material-ui/tree/master/examples/nextjs
I wanted to change injection order of CSS - in my application, components will be styled with SCSS files so I wanted those stylesheets to override jss stylesheets. According to documentation passing an object to ServerStyleSheets spreads that object to StylesProvider component: https://github.com/mui-org/material-ui/tree/master/examples/nextjs so I passed injectFirst prop, but it doesn't get passed to StylesProvider. Other props, like disableGeneration do get passed and they work, but injectFirst doesn't.
Expected Behavior 🤔
Passing injectFirst prop to ServerStyleSheets should pass it to StylesProvider which should result in changed injection order of CSS files.

Steps to Reproduce 🕹

Link to codesandbox: https://codesandbox.io/s/github/mui-org/material-ui/tree/master/examples/nextjs
Steps:

Add object { injectFirst: true } as parameter to ServerStylesSheets() in _document.js
Check order of stylesheets in <head> tag

Context 🔦
It's impossible to have both SSR and changed injection order -  injectFirst prop doesn't work, so Material-UI stylesheets always get injected last, thus overriding all my custom SCSS styles.

Your Environment 🌎

Tech
Version

Material-UI
v4.9.14

React
16.13.1

Browser
Chrome

TypeScript
x

etc.
Next.js @ 9.4.0

You need to use <StylesProvider injectFirst> in the _app.js.

I think I haven't explained the problem properly. Adding <StylesProvider injectFirst> in _app.js does resolve the issue with injection order, but it doesn't work properly with SSR - it takes a little bit of time until JS loads and that results with something like FUOC - first there is the default injection order and default Material-UI styles are more important, then the JS loads and <StylesProvider> starts working which results in our custom styles to become more important and change the looks of certain elements. I created this issue because I thought that passing injectFirst into ServerStyleSheets will change the injection order on build time (or on request if we are talking about SSR) but apparently it doesn't work this way.
Is there any way to achieve changed injection order on build time?

For the initial render, you have to make sure the CSS is send in the right order, it has to be handled at the Next.js level. injectFirst won't help. You can ask on StackOverflow for more help.

@pbelzek Were you ever able to find a work around for this for SSR? I am running into the same issue.

@BlakeCrowther sorry, no work around. I didn't touch this issue since then. As @oliviertassinari said, you might have more luck on StackOverflow with this.

@pbelzek Ok, thanks for the response!


Remove documentation publish from AppVeyor build

The current documentation build process can expose the value of
BOND_TOKEN to authors of pull requests. We will need to find a different
way to build and publish the Bond documentation.
Until then, BOND_TOKEN used has been invalidated, and the publishing
step has been removed from the AppVeyor build.

AppVeyor's secure environment variables are not exposed to pull requests. This change is not needed, but we still need to roll the token, as it was exposed--but never used--via other testing.


Redis operator deployment works fine, but I can't get a working redis cluster

I tried with both modes: namespace and cluster but I get the same result:
kubectl get distributedrediscluster
NAME        MASTERSIZE   STATUS   AGE
redis-one   3                     15s

So here is how I proceed:
Git
git clone https://github.com/ucloud/redis-cluster-operator.git
cd redis-cluster-operator
git checkout v0.2.0

Kubectl
kubectl create -f deploy/crds/redis.kun_distributedredisclusters_crd.yaml
kubectl create -f deploy/crds/redis.kun_redisclusterbackups_crd.yaml
kubectl create -f deploy/service_account.yaml
kubectl create -f deploy/namespace/role.yaml
kubectl create -f deploy/namespace/role_binding.yaml
kubectl create -f deploy/namespace/operator.yaml

Checking the operator:
$ kubectl get deployment
NAME                     READY   UP-TO-DATE   AVAILABLE   AGE
redis-cluster-operator   1/1     1            1           78s

$ kubectl get pods
NAME                                     READY   STATUS    RESTARTS   AGE
redis-cluster-operator-d866cbd6c-rqxfp   1/1     Running   0          12m

So far, so good.
I need the password mode, so I'm creating a Secret file:
apiVersion: v1
kind: Secret
metadata:
  name: redis-auth
  namespace: p6y
type: Opaque
data:
  password: QmhvbyEK

kubectl apply -f secret.yaml

And finally the DistributedRedisCluster file:
apiVersion: redis.kun/v1alpha1
kind: DistributedRedisCluster
metadata:
  name: redis-one
  namespace: p6y
spec:
  image: uhub.service.ucloud.cn/operator/redis:5.0.4-alpine
  masterSize: 3
  clusterReplicas: 1
  passwordSecret:
    name: redis-auth
  resources:
    limits:
      cpu: 200m
      memory: 200Mi
    requests:
      cpu: 200m
      memory: 100Mi

kubectl apply -f drc.yaml

Now checking the health of the cluster:
$ kubectl get distributedrediscluster -n p6y
NAME        MASTERSIZE   STATUS   AGE
redis-one   3                     100s

Here are the operator logs:
$ kubectl logs -f redis-cluster-operator-d866cbd6c-rqxfp
{"level":"info","ts":1579703505.3296764,"logger":"cmd","msg":"Go Version: go1.13.3"}
{"level":"info","ts":1579703505.3298333,"logger":"cmd","msg":"Go OS/Arch: linux/amd64"}
{"level":"info","ts":1579703505.329855,"logger":"cmd","msg":"Version of operator-sdk: v0.13.0"}
{"level":"info","ts":1579703505.3298721,"logger":"cmd","msg":"Version of operator: 0.1.1+0000000"}
{"level":"info","ts":1579703505.330385,"logger":"leader","msg":"Trying to become the leader."}
{"level":"info","ts":1579703506.30127,"logger":"leader","msg":"No pre-existing lock was found."}
{"level":"info","ts":1579703506.3117602,"logger":"leader","msg":"Became the leader."}
{"level":"info","ts":1579703507.2682493,"logger":"controller-runtime.metrics","msg":"metrics server is starting to listen","addr":"<IP_ADDRESS>:8383"}
{"level":"info","ts":1579703507.2686155,"logger":"cmd","msg":"Registering Components."}
{"level":"info","ts":1579703511.1277542,"logger":"metrics","msg":"Metrics Service object created","Service.Name":"redis-cluster-operator-metrics","Service.Namespace":"default"}
{"level":"info","ts":1579703512.079728,"logger":"cmd","msg":"Could not create ServiceMonitor object","error":"no ServiceMonitor registered with the API"}
{"level":"info","ts":1579703512.079791,"logger":"cmd","msg":"Install prometheus-operator in your cluster to create ServiceMonitor objects","error":"no ServiceMonitor registered with the API"}
{"level":"info","ts":1579703512.0798006,"logger":"cmd","msg":"Starting the Cmd."}
{"level":"info","ts":1579703512.080263,"logger":"controller-runtime.manager","msg":"starting metrics server","path":"/metrics"}
{"level":"info","ts":1579703512.0804253,"logger":"controller-runtime.controller","msg":"Starting EventSource","controller":"distributedrediscluster-controller","source":"kind source: /, Kind="}
{"level":"info","ts":1579703512.080463,"logger":"controller-runtime.controller","msg":"Starting EventSource","controller":"redisclusterbackup-controller","source":"kind source: /, Kind="}
{"level":"info","ts":1579703512.1810398,"logger":"controller-runtime.controller","msg":"Starting Controller","controller":"distributedrediscluster-controller"}
{"level":"info","ts":1579703512.1811314,"logger":"controller-runtime.controller","msg":"Starting workers","controller":"distributedrediscluster-controller","worker count":4}
{"level":"info","ts":1579703512.1819806,"logger":"controller-runtime.controller","msg":"Starting EventSource","controller":"redisclusterbackup-controller","source":"kind source: /, Kind="}
{"level":"info","ts":1579703512.2825923,"logger":"controller-runtime.controller","msg":"Starting Controller","controller":"redisclusterbackup-controller"}
{"level":"info","ts":1579703512.2826486,"logger":"controller-runtime.controller","msg":"Starting workers","controller":"redisclusterbackup-controller","worker count":2}

Other information:
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.5", GitCommit:"20c265fef0741dd71a66480e35bd69f18351daea", GitTreeState:"clean", BuildDate:"2019-10-15T19:16:51Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.4", GitCommit:"67d2fcf276fcd9cf743ad4be9a9ef5828adc082f", GitTreeState:"clean", BuildDate:"2019-09-18T14:41:55Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

@polefishu I have the same problem.

@zedtux @yufan022 If you run operator as namespace-scoped, Make sure that the namespace of the operator watched  is same as that DistributedRedisCluster runs on.
If you run operator as cluster-scoped, add redis.kun/scope: cluster-scoped annotations to the DistributedRedisCluster.

I'm no more using this operator.


fields

we want to add support for the generic application of fields (read: 3D data displayed in two dimensions, for example the display of a sinusoidal wave centered at some point).
fields should generally be expressed as a function of the form z = F(x, y) where are inputs/outputs are real numbers. to support the display of real-number outputs it will be necessary to create more useful utilities around color sequences (see #25).

resolved by #36


WINK devices become unavailable after upgrade to 102.3

Home Assistant release with the issue:

arch
armv7l

dev
false

docker
true

hassio
true

os_name
Linux

python_version
3.7.4

timezone
America/New_York

version
0.102.3

virtualenv
false

Last working Home Assistant release (if known):
102 I think
Operating environment (Hass.io/Docker/Windows/etc.):

PI4 with docker and Home assistant
Integration:

Description of problem:
Devices connected vial WINK hub randomly become "unavailable" until I restart HA, even though WINK says they are online.
Aslo the stat of WINK connected devices isn't always detected correctly
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

wink:

    `

Traceback (if applicable):

Additional information:
I tried to attach the log file but it is too big

@franmiele please minimise the configuration to the integration with which you have an issue.
If the log is too big you can paste it on https://pastebin.com/ and link it in this issue.

The log is too big for pastebin

Can you isolate the problem in the logs?
Maybe split it into multiple entries for pastebin?

the log will be useless. It's only for today and the issues haven't appeared yet today.

@franmiele okay please try to add the log if you can, it really makes debugging easier.

@franmiele okay please try to add the log if you can, it really makes debugging easier.

I will if the issue arises again

Hello Beautiful Home-Assistant community,
In my side the Wink integration "work" on 103.0b0 but the states of the plug is not updated.
If the switch is OFF, I'm able to switch ON but after 2 sec in HA the switch is status ti OFF but the switch is still in really ON.
I'm not able to switch OFF the plug with HA, I need to switch on Homekit or Wink to do it.
Best regards,
Quentin

I have the same problem as the above after upgrading.  Here's the error I get from the Check Home Assistant Configuration Add-on.
Failed config
General Errors:
- Component error: wink - cannot import name 'ENTITY_SERVICE_SCHEMA' from 'homeassistant.helpers.config_validation' (/usr/local/lib/python3.7/site-packages/homeassistant/helpers/config_validation.py)


websocket client in esp-idf (IDFGH-511)

Can you provide a websocket client for esp-idf and example codes. Please help me to find the supported lib and examples

Maybe checkout the cpp_utils from nkolban:
https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/WebSocket.cpp

Thanks for your response.
I am new to this, can you provide an example code for websocket client using this library.

I think you have first to setup a Socket connecting to your WebSocket Server.
( https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/Socket.h - use Socket::connect(..); )
At the second step you can make an instance from WebSocket with the Socket as parameter in the constructor.
I don't know how to exactly use the WebSocket because I only used the HttpServer before.
Maybe this file  helps or @nkolban can provide some examples.

https://github.com/nkolban/esp32-snippets/tree/master/sockets

Hi @tharunjoy ,
Sorry for the lack of an update on this issue. An official WebSockets client is being worked on for ESP-IDF V4.0, and this issue will be updated when it's available.
Angus

FWIW, I just wrote a WebSocket client, if this is of use to anyone else: https://github.com/tricot-io/xsp
(it's currently on top of a slightly patched esp-idf v3.2).

@projectgus
I am using older esp idf version. Can I manually add this new websocket component to my older version of esp idf?

@tharunjoy You haven't said what ESP-IDF version you are currently using, but anything is possible if you are prepared to some development work yourself and support the result yourself.
I would strongly recommend waiting for ESP-IDF v4.0 release, or trying the v4.0-beta2 version, though.

@projectgus
Currently I am working on 'esptool.py v2.8-dev, 2.8-dev'. Since I am using 'Arduino over esp idf' component to support Arduino library and as this is not implemented over the latest IDF(https://github.com/espressif/arduino-esp32/issues/3201#issuecomment-530899455) version, I am forced to stick on this version.
If I download the source code for official websocket library and add them to the my IDE version component file as an external library and add it in the component.mk file, will I be able to compile and use them?

@tharunjoy That is the esptool.py version not the ESP-IDF version, but I understand what you mean.
There is no easy way to integrate this into the IDF version used with Arduino, I'd recommend waiting for Arduino to update the IDF version. Sorry about that.

Hi I see there is IDF component esp_websocket_client in 4.4.3 , but it is not there in IDF v5.0 , why is that ? is it  got modified with new name ,  or else is it deprecated ?


[WIP] keep secrets in encrypted form in memory

Description
This proposal attempts to reduce the likelihood of sensitive data being exposed.
Issue reference
dapr/dapr#750
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation

@flier this PR is very old and I am closing it for now to keep PRs manageable across all our repos. Feel free to reopen when you plan to work on it again.
Looking forward to your contribution, thanks!


tutorial_translate.py raise Exception("no %s exist" % ckpt.model_checkpoint_path)

Hi Guys, I have trained the translation model using python tuaorial_translate.py, then i want using my own words to test the model,so i switch the main_train() to main_decode(), but the python shell raise one exception; more information as follow:
first, the exception occurs at 391 lines of tutorial_translate.py if ckpt and tf.gfile.Exists(ckpt.model_checkpoint_path)；and the ckpt object is not null, ckpt.model_checkpoint_path value is 'wmt/model_translate_enfr.ckpt-10'; in fact, the directory does not have this file , i wonder why the pre-sentence ckpt = tf.train.get_checkpoint_state(train_dir)  return the error data, hope your help, please! thanks a lot

It seem that you have problem with ckpt.
Did you set a correcttrain_dir?
Alternatively, can you set is_npz = True then it will save the model into npz file instead of ckpt, it may help.

yeah, my train_dir is  a relative file path wmt, isn't it true? i'll try to set is_npz=True, should i remove the generated data and train the translate model again?


New Space Ninja intro song

About the PR
Replaced the current Space Ninja intro song with a custom one. Also updated attributions.
Why / Balance
I don't like the current Space Ninja intro song. It's not original and it doesn't really match the feel of a futuristic cyber space ninja. This new one also serves to excite the player, similarly to the other antagonist intros.
Media
https://github.com/user-attachments/assets/2cd0332e-06a1-4d1a-897e-5ab6b7b48115
Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Space Ninjas now have a new intro song.

Whats the Morse Code saying?

Whats the Morse Code saying?

It's supposed to be saying "Might of Spiders" but I put an extra "D" at the end to make the Morse sound better. Don't tell anybody I did that.

Sounds good to me and i dislike the current sound anyway.

Man I'm going to miss the goodf flute.  I like both.

This sounds really cool. Thank you for your contribution!


Fix errors when updating existing assessment

Temporary hotfix. Do not merge. Need to rewrite parts of the assessment update code.

Pull Request Test Coverage Report for Build 3146

12 of 15 (80.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 97.185%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/cadet/assessments/assessments.ex
9
12
75.0%

Totals

Change from base Build 3145:
0.3%

Covered Lines:
1450

Relevant Lines:
1492

💛  - Coveralls


feat: sign data with prefix

What Changed

wallet_fullOwnership_signData() should be sign with [magic prefix bytes]

Change Type
Indicate the type of change your pull request is:

[ ] documentation
[x] patch
[ ] minor
[ ] major

@zhangyouxin can you help to resolve conflicts

hi, @zhangyouxin, can you merge with main to avoid workflow failure


Bug: Black Line at the corner of the video on Safari

Is there an existing issue for this?

[x] I have searched the existing issues

Which Mux Elements/Packages does this apply to? Select all that apply
mux-player-react
Which browsers are you using?
Safari
Which operating systems are you using?
macOS
Description
There are black lines at the corner of the video on the Safari browser
Desktop

Mobile Web (See the left and right corner of the video)

Reduced test case
No response
Steps to reproduce

Use the Safari browser
Make sure the website page has a white background
Use any video with a white background. You can use the video from your website (Reference: https://docs.mux.com/core/stream-video-files)
Stop the video at the part where it has a white background
Observe the corners on both desktop and mobile view

Current Behavior
Has black lines at the corner of the video
Expected Behavior
Should not have black lines at the corner of the video
Errors
No response
What version of the package are you using?
No response

related https://github.com/muxinc/media-chrome/discussions/831

duplicate of https://github.com/muxinc/elements/issues/1020


Fixing Signer list on explorer: Multiple signer for one Partner

Hi @andresgalante
Xverse is running two signers
signer 1 pubkey: 0x02877ce29ba35458b827a6ea18510b9058ae4c30e2c33d288f2982c13497caec6e
signer 2 pubkey: 0x0302328212d5e430a8a880f8e2365a8f976ee50490ff030c106866c0b789eae91a
Signer 1 address: SP3WEKWTCJ2FXTMAQK674GHEWTJKPP6KFH0D38T7S
Signer 2 address: SP3WEKWTCJ2FXTMAQK674GHEWTJKPP6KFH0D38T7S
Signer 1 Stacked amount:<PHONE_NUMBER>73412 ustx
Signer 2 Stacked amount:<PHONE_NUMBER>270 ustx
Pool operator address for both signers: SPXVRSEH2BKSXAEJ00F1BY562P45D5ERPSKR4Q33
Hiro API is always showing the following which is wrong:
"pooled_stacker_count": 0  for Signer 1
"pooled_stacker_count": 5998 for signer 2
Can we have this fixed in explorer? there's only one signer listed under xverse with no associated addresses

@BLuEScioN is this done?


Separator does not occur

Hey, I am really obliged to your contribution of this Zurb's twenty-twenty plugin but I do not see the separator between the two images. How to show that and also how to customize that? Please help me ASAO.

There’s a section in the docs

You can include a third child to be a slider. The slider is placed inside a zero-width, full height element, positioned to the correct left offset: basically just set height: 100% on your slider

It’s sort of BYOB slider element, so you can customise it however you want. Let me know if you have any further issues!

Thanks alot.
On Mon, Oct 16, 2017 at 2:39 PM, Jacob Parker<EMAIL_ADDRESS>wrote:

There’s a section in the docs
You can include a third child to be a slider. The slider is placed inside
a zero-width, full height element, positioned to the correct left offset:
basically just set height: 100% on your slider
It’s sort of BYOB slider element, so you can customise it however you
want. Let me know if you have any further issues!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jacobp100/react-twentytwenty/issues/3#issuecomment-336791643,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYtAM_w5bjICC2CQqJQqfkEUOe4g8Os5ks5ssvbDgaJpZM4P6HNT
.


Azure Open AI Functions calling integration into Kernel

With the announcement of Azure Function Calling (https://techcommunity.microsoft.com/t5/azure-ai-services-blog/function-calling-is-now-available-in-azure-openai-service/ba-p/3879241) evaluate how SK planner and kernel can incorporate support for our customers.

Related to #1450


Placeholder for supporting collector_tstamp override in Tracker Protocol

The use case is re-processing failed events and preserving the original collector_tstamp.

This is a won't fix, the collector_tstamp should not be used for analytics anyway (use the derived_tstamp).


Update run

directory has to be changed

I’d prefer to update the documentation instead to have the run file in line with the repository name.


Add scaffolding for docs plus man pages for dnst and dnst nsec3-hash.

I've resolved my comments as they pertain to specifics to one command but we can come back to specific command related texts in PRs for those commands, this is more about getting the scaffolding place.


ERROR at the saving stage in the starcoder finetune scripts

Hi friends,
I was trying the finetuing/starcoder/finetune.py using torch.distributed.launch, and met the following error :
FileNotFoundError: [Errno 2] No such file or directory: './checkpoints/checkpoint-10/pytorch_model.bin' caused at /finetuning/starcoder/finetune.py", line 35.
Could anyone let me know how to solve it?

not quite sure what is pytorch_model.bin, when it is generated and why we should remove it..

Hi. This part of the code is due to the fact that we wrote it when the peft library was still nascent. With recent version of transformers and peft, you don’t have to create your own callbacks. Feel free to remove this callback from your trainer.

Thanks a lot for the reply. I'll try right now.


Community Meeting February 1, 2022

At 11am Eastern, 8am Pacific, 3pm UTC; find your time
Video call link: https://meet.google.com/ohf-kwvd-mrp
Or dial: (US) +1<PHONE_NUMBER> PIN: 936<PHONE_NUMBER>#
More phone numbers: https://tel.meet/ohf-kwvd-mrp?pin=9361820877398
Or join via SIP<EMAIL_ADDRESS>Meeting recordings available on YouTube at https://www.youtube.com/channel/UCfP_yS5uYix0ppSbm2ltS5Q
This issue will collect prospective agenda topics. Add topics you'd like to discuss below!

Scoping proposal WIP

Next https://github.com/kcp-dev/kcp/issues/467

Recording: https://youtu.be/4Kl52nE92CY


是否支持windows环境下安装？

google了一下没有相关问题。

@JusufUs
Please do some research on the following aspects:

Install a ruby development environment to build the site files;
Serve those generated site files on your Windows machine.


Correct application package version env var

Separator between package and package version is # character, not _

@davidkydd : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Actually this PR is incorrect, it is _ on Linux and # on Windows. Described precisely here  https://docs.microsoft.com/en-us/azure/batch/batch-application-packages#execute-the-installed-applications

@davidkydd, I'll update the environment variable example to show both Windows and Linux nodes. I'll close this PR, but I appreciate you pointing this out. Thanks!
#please-close