All EmergencyContactsComponents

I wrote the code for all of the EmergencyContactsComponents. There is a flow that allows users to input emergency contact information in EmergencyContactInputComponent.js, their input then makes its way through eventhub.js until it is accessible by EmergencyContactListComponent.js and is displayed by EmergencyContactComponent.js

Changes look good. Nice job.


String expandedAbbr.detail is not translatable

See line: https://github.com/ramya-rao-a/vscode-emmet-helper/blob/master/src/emmetHelper.ts#L47
This is impossible to translate.

For (own) reference the line is at https://github.com/microsoft/vscode-emmet-helper/blob/master/src/emmetHelper.ts#L155 now.

Closing, fixed in v2.0.9.

Verifier: code review
https://github.com/microsoft/vscode-emmet-helper/commit/7b44b9a33f473b168369e4cb8b731108baba36a1

Verifier: code review
https://github.com/microsoft/vscode-emmet-helper/commit/7b44b9a33f473b168369e4cb8b731108baba36a1


Reference Section Issue (Report)

In my report.md I currently have 2 comments, not following template and a comment to the reference section.  I have compared my reference section to other students and can not see any difference.  Can you please point out in detail where the error appears.  Additionally, each reference appears to render on its own line through the report website.

add the tags, i think i fixed refs a while back

ERROR: Please indicate final submission with Status: final after the Check Report Icon. Please see https://raw.githubusercontent.com/cybertraining-dsc/fa20-523-312/main/project/project.md for an example where to place the Status: flag
Is this what you are referencing by tag?

Please take a look at the template

we have posted a video if you do not know where to find it.

Resolved, thank you for your help!


Trying to ride self in freecam causes server crash!

Describe the bug
I'm using freecam (modrinth edition) and when I'm in freecam mode (not spectator since my original body is still where it is) I accidentally tried to ride myself. This caused the server to crash. I have tried this with other players on the server as well.
To Reproduce
Steps to reproduce the behavior:

Install freecam and fsit
Press F4 (default) to detach camera from player
Move and right click (or which ever key is for sit) on your self
Make sure player sitting is enabled in the config file

Expected behavior
Ideally, nothing should happen since a player should not be able to ride themselves
Screenshots
If applicable, add screenshots to help explain your problem.
Logs
I need to crash my server again to do it. If it's really important, I'll do it :(
Info (please complete the following information):

OS: Linux ARM64 Ubuntu 22.04
Minecraft Version: 1.20.1
Mod Version: 1.3.2

Additional context
If I use the carpet player use command, I think the issue doesn't happen AFAIK (not fully tested though). Happens for sure if I right click on myself in freecam mode.
Remember, this only happens in freecam, not in normal vanilla gameplay.

Thank you for contributing!
I can't test it right now, so a patch release will be delayed. If you need it ASAP, you can use the developer version with hotfix. fsit-5c4a283.jar is what you need from this archive

Fixed in v1.5.0.
1.20-1.20.1 and 1.20.2


Name conflict with fontParts fontShell

I know this was raised on twitter a while ago, but that was likely lost. This is your package of tools, so you are of course free to do whatever you want, but there is a naming confusion between this fontShell and fontShell from fontParts.

Ah, good point. OK, so there is a clash, but I wonder if it actually matters - one is an executable and one is a library. If people are referring to them in conversation I can see that it might get confusing, but I can't see any confusion in use.

I think the thing in fontParts that is a library also comes with a sample CIL implementation, so yes there seems to be a clash.

Two things: they are both based in python, and work in a python environment, so it isn't quite a clash, but confusing. Second, the fontShell name has been around and is known to Simon, so I'm confused as to why the name clash in the first place (yes, I am being grumpy, I know!).
Both those things said, yes, I don't know if there will be a lot of overlap/confusion in practice, and I'm not going to get in the way of a nice package of tools, so I'll leave this up to Simon and close this issue.


Rationalize fault domains implementation for managed disk

@mboersma can we summarize this work as:

ensure that if a region supports 3 FDs that we implement managed disk w/ 3 FDs?

@seanmck FYI

@jackfrancis that's most of it, but another driver for this change was that users can't create acs-engine clusters in canary regions at all, since they support only 1 FD and we're currently hard-coded at 2.

@jackfrancis exactly what are you looking for? spreading the VMS over 3+ FDs?


Redirect has flashing behavior - possible to "synchronously" redirect?

This is a question – I searched around the issues & Spectrum but couldn't find an answer.
I'm trying to make a redirect that's "synchronous", in that it doesn't cause a double render. The reason is because I want my top-level nav to <Link to='/docs' />, but the first real page of the docs is actually /docs/getting-started/introduction. I tried adding a <Redirect /> and also using navigateTo, but in both cases I get a flash as /docs renders, then redirect, then /docs/getting-started-introduction renders.
Also, once I'm on /docs/getting-started/introduction, clicking the link to /docs flashes again.
You can see the behavior if you click Documentation here: https://5d52b7a44d225b0009a37a20--miragejs.netlify.com/
My question is, is there a way to sort of statically/synchronously redirect, without any flashing? I want that URL to always go to /docs/getting-started/introduction.
(The reason I don't want to use <Link to='/docs/getting-started/introduction' /> is because I want the Documentation to be partiallyActive when any route under /docs is active.)
Thanks for this great libary 🙂

Ended up writing this component in my app
function NavLink({ activeFor, ...props }) {
  activeFor = activeFor || props.to;

  return (
    <Match path={activeFor}>
      {({ match }) => {
        return (
          <Link
            {...props}
            className={`${match ? "text-gray-900" : "text-grey-500"}`}
          />
        );
      }}
    </Match>
  );
}

and using it like this
<NavLink
  to="/docs/getting-started/introduction"
  activeFor="/docs/*"
>
  Documentation
</NavLink>

but, still curious if this idea has a place in this library!

redirectTo and <Redirect/> both throw a redirect which is caught with componentDidCatch and should prevent a double render, navigate however doesn't throw so it will (probably?) cause double renders.
Can you make a test case on codesandbox?

Found your sourcemaps:
in app.js:
<Redirect from="/docs" to="/docs/getting-started/introduction" noThrow />

You opted into the flashing with noThrow.
In development without noThrow its annoying (because react doesn't actually catch the errors, then tools like create-react-app throw up a big error screen).
But in production you won't get that big error screen, but instead get a single render (and no flashing).
// change this
<Redirect from="/docs" to="/docs/getting-started/introduction" noThrow />
// to this
<Redirect from="/docs" to="/docs/getting-started/introduction" />

Oops, this needs better docs so people know hte point of noThrow and why it throws in the first place, and to not worry about the error page that a lot of tools throw up in development.

Removing the noThrow prop does not fix the issue, even in prod mode, getting errors in console too:

You'll still get the error, I don't believe you'll get flashing, but I haven't tested it lately. Can you make a test case in code sandbox?

I tried to reproduce this in this fiddle. The console shows 2 errors whenever a redirect happens, one of which talks about the component tree having been recreated from scratch (a bit unsettling).
Other than that it seems to work. I didn't observe flashing.

@ryanflorence Thanks for the response :)
These are the errors I see for one redirect without noThrow:

These are all expected? If so I'll probably just stick with my wrapper component. I'm new to React and find all these errors pretty confusing, also I'm used to writing test suites that fail whenever there's an unhandled exception.
Feel free to close if that's the case!

In terms of how a component should work, in my opinion a component shouldn't change how it acts and behaves from dev to prod, we should be making components stable and not need flags.

I get this same behavior (404 page flashes, then redirects) trying to redirect home if authentication fails.
I've added authorization from Auth0 to my simple website as per this tutorial:
https://auth0.com/blog/securing-gatsby-with-auth0/
...and in my protected page I have a check to see if we're authorized, if not, redirect back home:
  if( !isAuthenticated() ) {     return (       <Redirect to="/" />     )   }
I can supply code, but there's not much to see.
Here's my trace:

Fyi this is still a problem, I spent quite a bit of time trying to discover the source of the flashing in my team's app and it actually occurs as a side effect of loading the Redirect component from the @reach/router module... So, not even using the component, but loading its reference in via the module will produce the issue.
Hoping we can get this thing fixed, I'd try to make a PR if I had time, but unfortunately I don't :/
Tags for others, hopefully this will bubble up this post in Gooogle's result w/ their crawler:
Reach router screen flashes redirect. Reach router screen flickers redirect. Reach router screen flashes when changing paths. Reach router screen flickers when changing paths. Reach router screen flashes when changing routes. Reach router screen flashes when changing routes.


Make Svelte Blogger responsive

Make Svelte Blogger responsive. Test for iPhone and iPad in
https://responsivetesttool.com/

Resolved.


Obj-c fixed

Hello. There's some wrong for frames. So, I fixed/changed some codes.
Thanks and have a nice day.

Fixed rotate bug if use at UITableviewCell (both view and uitableviewcell working)

change event function to UIDeviceOrientationDidChangeNotification when rotate
check current view width

Fixed If use both options "Item width by text" and "center"

fixed set frames
change checking tap point (check in rect) for "_menuItemWidthBasedOnTitleTextWidth"

This version is compatible with objective-c version:  https://github.com/alejouribesanchez/ScrollHideNavigationController


Lag/Delay problem

I'm experiencing 1-2 sec lag/delay using this. The chrome extension has a small lag also, but tiny compared to this. Anyone else?

Yes this is happening

My output is lagging behind by about 10 seconds. I can play something and have the receiving device make an echo. I'm using the Nexus Player, in case that's relevant.

I also am having this issue, the app works great and am loving it but i am having about a 5 second delay. Any ideas about why this is happing?
I am casting to a LG H7

Currently looking into it, but its not looking good. I'll investigate another method of capturing system audio, but it takes a little more system configuration.

Hi. Thank you for the program, it is great for listening to audio stored on the PC.
I'm having the same problem syncing to video, a three or four second delay. Acidhax will you be able to let users know if and when you fix it?


Go doc

Write code documentation to be generated and exposed at pkg.go.dev. Reference: https://go.dev/blog/godoc

Implemented by d9285fe2828863eeeff4bfa329e3f15a3b4685b1.


Ensuring that embargo date is a string

Prior to this commit, the embargo-date for ROF was an array. This
commit enforces the embargo date to be a string.

Approve.


Suggestion

Not really an issue, but a suggestion.
If you could change the way the switcher stops it would be alot better for me atleast. I normally use 'Escape' key to go to my inventory and this ends the program, often resulting in me closing it by mistake while playing.
Otherwise the switcher works great!

Do you suggest allowing the user to specify which key they want to use to shutdown the autoswitcher?

I didn’t know if this was possible, but yes that would be optimal.

From: Steffen Carlsen<EMAIL_ADDRESS>Sent: Wednesday, March 13, 2019 5:05:02 PM
To: SteffenCarlsen/OSRS-AutoSwitcher---PvPHelper
Cc: Jegersej7; Author
Subject: Re: [SteffenCarlsen/OSRS-AutoSwitcher---PvPHelper] Suggestion (#3)
Do you suggest allowing the user to specify which key they want to use to shutdown the autoswitcher?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/SteffenCarlsen/OSRS-AutoSwitcher---PvPHelper/issues/3#issuecomment-472488889, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AuQRSbR1QR3VzzyqIYear1uBJlq6YgVPks5vWSGugaJpZM4bsOaT.

No problem, I will look at implementing easier modification of all available hotkeys without doing a full renew of settings, including allowing the user to modify which hotkey shuts down the switcher.

Cool, sounds great.
A quick temp fix could be just to change the stop hotkey to something which is not used as F key in runescape - like 'Delete'

Fra: Steffen Carlsen<EMAIL_ADDRESS>Sendt: 14. marts 2019 12:11
Til: SteffenCarlsen/OSRS-AutoSwitcher---PvPHelper
Cc: Jegersej7; Author
Emne: Re: [SteffenCarlsen/OSRS-AutoSwitcher---PvPHelper] Suggestion (#3)
No problem, I will look at implementing easier modification of all available hotkeys without doing a full renew of settings, including allowing the user to modify which hotkey shuts down the switcher.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/SteffenCarlsen/OSRS-AutoSwitcher---PvPHelper/issues/3#issuecomment-472806890, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AuQRScTTM64bgYGCX2-vKqptJCGNwUd4ks5vWi5wgaJpZM4bsOaT.

Soo, 361 days later I finally got around to fixing this. It's on the master branch currently. I'm working on allowing mouse-keys to work as input keys, after that I will release a new version.


TypeError: Cannot set property 'distinctId' of undefined when call identify method

After signup I call mixpanel.identify(email). It logs I see LOG [Mixpanel] Identify '...'. But then immediately get this error:
TypeError: Cannot set property 'distinctId' of undefined.
I've checked email has value. So it's not undefined. Even in mixpanel logs email is displayed for identify.
mixpanel.track() works as expected though.

hi @klalex , do you mind sharing the full log and the code snippet that can reproduce the issue?

@zihejia I used example from docs:
const mixpanelToken = '...';
const mixpanel = new Mixpanel(mixpanelToken, false);
mixpanel.setLoggingEnabled(true);
mixpanel.init();
mixpanel.identify(email);
Also when I downgraded to version 2.4.1 it works now. So something wrong with version 3.0

hi @klalex , would you like to try 3.0.2?

I got the same error.
It should be "await mixpanel.init();" because the init function is an async function.

I'm still facing this issue


Switch from Bootstrap to Materialize

Material Design seems to have better tools.
Materialize is a CSS framework that implements Google's Material Design.

Materialize is still in Alpha and less stable than Bootstrap in my opinion

Yes. I'll probably abandon this idea. A friend told me that Materialize JS can be weird sometimes...

If you want to have a material theme you can use Bootswatch Paper : https://bootswatch.com/paper/

I'll keep an eye on that! Thanks!


Removing or editing acme-dns account

I executed acme-dns-client register and created proper CNAME record for domain.example.org - now I'd like to remove every trace of it from acme-dns-client as well as acme-dns.
How this can be done? The acme-dns-client only accepts register, check and list as a command.
Similar issue is with the case when I'd like to amend the value of -allow parameter for already registered user. Can this be done?

I finally setup my host certificate, but I did some tests before, and I want to get rid of those.
How can I remove them? or where are they stored?


[(Brainde)Code Sprint] Improve documentation

Currently, the documentation of Braindecode is very research-oriented. Some parts are not clear for new users and some parts are simply not present at all.

The goal of this meta-issue is to achieve a more user-oriented documentation. This means:

Add docstrings in the NumPy/SciPy format where they are missing;
Make sure that every function/class parameter is described;
Achieve a homogeneous level of documentation across the different functions and classes;
Revisit the experience of a new user arriving on the doc for the first time.

@apmellot, @Sara04, @dengemann, @jpaillard


pythonPackages.libusb1: 2.0.1 -> 3.0.0

Motivation for this change
Version bump
Things done

Built on platform(s)

[x] x86_64-linux (NixOS)
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change
[x] Tested basic functionality of all binary files (none)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@grahamcofborg build python3Packages.hidapi python3Packages.trezor python3Packages.pylibftdi python3Packages.hwi python3Packages.fx2 python3Packages.adb-shell python3Packages.keepkey python3Packages.glasgow python3Packages.adb-homeassistant python3Packages.nkdfu python3Packages.pyusb


Add support for defaults accounts/apps/environments

Many commands require flags like --account-id, --app-id, and --environment to run.
This long form can be useful in scripts that run in CD pipelines, because they make it very explicit what's going on:
sectionctl deploy --account-id 2094 --app-id 7598 --environment production --directory commands/testdata/deploy/valid-nodejs-app/ --skip-delete

The flip side of this is that the local dev experience can be a clunky, because you need to specify the same arguments every time you run sectionctl:
sectionctl apps info --account-id 2094 --application-id 7598

This is particularly unergonomic for new users with a single account and a single app. The ideal experience should be running:
sectionctl apps info

and it runs successfully.

This has been implemented for deploy with https://github.com/section/sectionctl/pull/120 and was released in https://github.com/section/sectionctl/releases/tag/v1.11.1
I plan on making this a possibility for all commands later, using a kong resolver.


unicode message not send and revived

I compile tdlib in windows, normally without any issue compile ,
but in your example c# application cannot send message Unicode character like "سلام"
convert to Unicode see like this "????"
what am i mistake?

It is a bug in TDLib 1.2.0 binding for C#. It will be fixed in TDLib 1.3.0, but currently you need to patch https://github.com/tdlib/td/blob/master/tdutils/td/utils/port/CxCli.h and replace at the end of the file code
inline std::string string_to_unmanaged(String^ str) {
  if (!str) {
    return std::string();
  }
  return msclr::interop::marshal_as<std::string>(str);
}

inline String^ string_from_unmanaged(const std::string &from) {
  return msclr::interop::marshal_as<String^>(from);
}

with
inline std::string string_to_unmanaged(String^ str) {
  if (!str || str->Length == 0) {
    return std::string();
  }

  Array<System::Byte>^ bytes = System::Text::Encoding::UTF8->GetBytes(str);
  cli::pin_ptr<System::Byte> pinned_ptr = &bytes[0];
  std::string result(reinterpret_cast<const char *>(&pinned_ptr[0]), bytes->Length);
  return result;
}

inline String^ string_from_unmanaged(const std::string &from) {
  if (from.empty()) {
    return String::Empty;
  }

  Array<System::Byte>^ bytes = REF_NEW Vector<System::Byte>(static_cast<ArrayIndexType>(from.size()));
  cli::pin_ptr<System::Byte> pinned_ptr = &bytes[0];
  for (size_t i = 0; i < from.size(); ++i) {
    pinned_ptr[i] = from[i];
  }
  return System::Text::Encoding::UTF8->GetString(bytes);
}

@levlam try to call Ms technical support to ensure that base marshaling is working for utf-8. Our common solution is like a bicycle.

Thanks , yesterday i found this , it worked for me

Today I looked again at this decision and it again reminded me of a mountain bike race on snow-capped mountains in a extraterrestrial costume

Fixed in 1.3.0.


Add OCM pull model integration support by importing clusters without …

…the managed cluster secret config data. Updated project to be more compatible with upstream ArgoCD and not just Openshift GitOps operator.
Signed-off-by: Mike Ng<EMAIL_ADDRESS>/hold

@mikeshng   one case we may need to consider to support - some clusters could be imported to support push model and pull model as well e.g.

cluster1 is imported to argocd NS first to support push model. Then there is a real cluster secret there
Later cluster1 is selected to support pull model. Then another dummy cluster secret is created.

As a result, we will have two cluster secrets for cluster1. I am not sure which cluster secret will be picked up by the ArgoCD applicationSet controller when propagating the ArgoCD applications.
So ideally ArgoCD applicationSet controller should apply real cluster secrets to propagate ArgoCD applications in the push model, apply dummy cluster secrets in the pull model.
There may be some issues here. We may run some tests to verify this.


Use logical and to avoid TypeError when props is undefined

Adjusts MDXContent function output to allow undefined props as a parameter without throwing a TypeError.
Closes #82

Sweet, thanks for working on this!
I think it would be better to solve it in the argument list: MDXContent(props = {}).
Reasoning is that it’s documented that props is an object in one example of the readme: https://github.com/wooorm/xdm#mdx-content.
Also, could you update the example in the readme following “The actual output of running node example.js is:”?
There are also two errors in the tests on CI. the babel one looks a bit flakey (I’ll check if that’s still the case), but the error message might need to be changed?

ps. run npm test locally to test your changes! And you can use ASTExplorer to figure out what the AST is for certain JS code

Rebasing your PR against main should solve the weird babel error

The other error was related due to a change in Node 16.9. Updates the tests to fix that as well.

@wooorm Sounds like a plan. I'll adjust the fix to provide a default value for the props parameter and work on the test errors.

Thanks, released!


How do I access aggregations after serializing the result?

I am using the plainelastic version <IP_ADDRESS>. I am unable to use the geo_distance and date_range aggregates of elasticsearch. I am working around by using custom options instead. Though I would prefer to use their power instead of custom option.
Also I am unable to access the aggregation field after deserializing the result. I only get the option to use facets here and not aggregations.
var serializer = new JsonNetSerializer();
var facets=serializer.ToSearchResult<>(result).facets;

I have the same issue.

I have the same issue with facets vs aggregations

Same issue.  Anybody have a solution for accessing Aggregations

We started to implement aggregations response in our branch here: https://github.com/VirtoCommerce/PlainElastic.Net, the package in nuget is VirtoCommerce.PlainElastic.Net (beta). Will PR when ready.


chore: update handlebars to v6, fix build error

DirectorySourceOptions is marked as #[non_exhaustive], and only exposes the Default trait, so we need to instantiate a mutable instance using default(), and then change tpl_extension to the value we want.
Fixes #14764

Thank you!
@bors r+

:pushpin: Commit 9b4f03a876cf9931f89d3a8e2189bfd6c003dede has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 9b4f03a876cf9931f89d3a8e2189bfd6c003dede with merge 3728f064138319cd5168ed6bd65ce3eff51faf6e...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 3728f064138319cd5168ed6bd65ce3eff51faf6e to master...


Remove unused workspaces config

Closes https://github.com/davidmyersdev/vite-plugin-node-polyfills/issues/60

@davidmyersdev any chance to get this merged and released? I was not able to find a workaround to get rid of this warning otherwise


Enable id and crossorigin attributes

So you can hook it up to audio visualizers

Thanks for the PR!  Both worthy attributes that I'm surprised haven't come up before.
I'm going to merge this, add some quick docs, and then release a new version.  I'll post back here when it's live.

New version released!  Thanks again.  Let me know if you still can't get the audio visualizer working for some reason 👍
https://github.com/justinmc/react-audio-player/releases/tag/v0.10.0

Awesome! Thanks! Great project btw.
I got it to hook up with https://github.com/hu-ke/react-audio-spectrum just fine.


ci: Implement conversion stryker report to sonar

Description
copilot:summary
Related Issue

Explanation of Changes
copilot:walkthrough
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] Implemented tests for new features.
[ ] I tested the features already implemented.
[ ] Added usage examples in the readme.

:tada: This PR is included in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Bump @noble/* to the latest versions.

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

Project coverage is 88.70%. Comparing base (3eae003) to head (7f713b2).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #446      +/-   ##
==========================================
+ Coverage   88.47%   88.70%   +0.23%
==========================================
  Files          60       60
  Lines        4712     4712
  Branches      448      448
==========================================
+ Hits         4169     4180      +11
+ Misses        542      531      -11
  Partials        1        1

Flag
Coverage Δ

unittests
88.70% <ø> (+0.23%)
:arrow_up:

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

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


Issues Using PackageBrowser and ProjectList when logging in as DataCurator and user without code modification privileges

[x] STONReader>>lookupClass: (see bd4dba5 for patch and description)
[x] Issue #325 fix (see 162da4d for patch and test)
[x] Issue #340 fix

*STONReader>>lookupClass: stack:
a STONReaderError occurred (error 2710), Cannot resolve class named #'RowanProjectService'
---
STONReaderError (AbstractException) >> _signalWith: @5 line 25
STONReaderError (AbstractException) >> signal: @3 line 7
STONReaderError class >> signal:streamPosition: @4 line 4
STONReader >> error: @4 line 6
[] in  ExecBlock0 (STONReader) >> lookupClass: @4 line 10
Array (SequenceableCollection) >> detect:ifNone: @11 line 10
[] in  ExecBlock0 (STONReader) >> lookupClass: @4 line 9
IdentityDictionary (AbstractDictionary) >> at:ifAbsentPut: @7 line 14
STONReader >> lookupClass: @14 line 8
STONReader >> parseClass @6 line 7
STONReader >> parseObject @2 line 3
STONReader >> parseValue @8 line 5
STONReader >> parseListDo: @9 line 8
[] in  ExecBlock1 (STONReader) >> parseList @2 line 5
Array class (SequenceableCollection class) >> new:streamContents: @5 line 4
Array class (SequenceableCollection class) >> streamContents: @2 line 2
STONReader >> parseList @5 line 4
STONReader >> parseValue @16 line 9
STONReader >> next @3 line 4
STON class >> fromStream: @4 line 2
STON class >> fromString: @3 line 2
JadeServer64bit32 (JadeServer) >> updateFromSton: @3 line 4
GsNMethod class >> _gsReturnToC @1 line 1

Final Rowan-specific issue for pending v1.1.0 release is in the can ...


How to scope grep to specific directory?

I often want to grep within a certain directory.  I know I can specify the directory as the last argument to the Grep function – but I'm not sure how to do that dynamically.
If I were to do it statically, I could set up different maps where I know the specific directory up front.  For example (legacy syntax):
nnoremap <leader>ga <scriptcmd> scope#fuzzy#Grep('rg --vimgrep --no-heading --smart-case', v:true, '<cword>', 'app/')
nnoremap <leader>gc <scriptcmd> scope#fuzzy#Grep('rg --vimgrep --no-heading --smart-case', v:true, '<cword>', 'config/')
nnoremap <leader>gd <scriptcmd> scope#fuzzy#Grep('rg --vimgrep --no-heading --smart-case', v:true, '<cword>', 'db/')

But that clogs up my maps a bit and having to specify the directories ahead of time is not ideal.
Is there a nice way to take the directory from the user input?  For example if the first or last input word ends in /, use that as the directory.
I imagine you wouldn't want to change the Grep function itself because then there would need to be a way to escape app/ if you wanted to search for that literal string, so perhaps there's a way to write a wrapper function that has the directory-extraction logic and then forwards the rest of the input to Grep.  Does anything come to mind?
Thank you!  I really like using this plugin.

You can use the Vim command. See README:
Grep takes additional arguments. :Scope {Grep|grep} [dir] [str] starts a live search with 'str' as the initial search string if non-empty. If 'dir' is specified, search in that directory instead of the current directory.

Oh, wait, you want to use rg instead. Just define your own Vim command. See command.vim and plugin/scope.vim
def DoGrep(...)
    fuzzy.Grep(...)
enddef
command -nargs=+  ScopeGrep DoGrep(<f-args>)

Thanks for the suggestions!  Sadly I haven't managed to make it work yet.  This is what I have so far:
function! DoGrep(...)
  echom a:000
  if a:0 == 1
    let dir = '.'
  else
    let dir = a:000[-1]
  endif
  let needle = join(a:000[:-2], ' ')
  call scope#fuzzy#Grep('rg --vimgrep --no-heading --smart-case', v:true, '', dir)
endfunction

command! -nargs=+ ScopeGrep call DoGrep(<f-args>)

Where you invoke it like this: :ScopeGrep foo app/.  However it doesn't work at all :/  I can't see how to pass needle into the Grep function, the way you would with :Scope {Grep|grep} [dir] [str].

I'd like to be able to invoke the grep feature with a keystroke, and when it pops up the window, type my search pattern followed by the directory into the input area – the same way you would if invoking rg on the command line.
When I try it with the out-of-the-box API via the following map...
nnoremap <leader>g <scriptcmd>vim9cmd scope#fuzzy#Grep('rg --vimgrep --no-heading --smart-case', v:true, '<cword>')<CR>

...and I type into the popup...

...the command echoed is:
rg --vimgrep --no-heading --smart-case user\ app/ ./

It's almost what I'd like.  If the space between user and app weren't escaped, and the ./ weren't automatically appended, it would be perfect.

I see. Wait for my next commit.

Please see the diff in README for usage. I gave several examples. For your use case try :
command -nargs=1 -complete=dir ScopeGrep fuzzy.Grep('rg --vimgrep', true, null_string, <f-args>)
nnoremap <yourkey> :ScopeGrep<space>

You have to enter the dirname once before it starts live grep. The design you proposed above will not work well with this plugin, since every space is escaped (so you can search multiple words). If you pass dirname as the last word, it will just be treated as part of the pattern to search. In the API, the last argument is the dirname, which is used in the above command.

That works perfectly.  Thank you very much!


Provide examples for all types of controllers

[x] standard controller: coverd by controller-rs (uses in Controller from #148), configmapgen_controller example also exists
[x] finalizer controller - was blocked by #291 , and easier path in #475 - we now have an example for it
initializer controller these are deprecated
[x] admission controllers - doable now with #484, see admission_controller example

Started writing a finalizer controller the other day as an experiment. Will add it as an example later on, but the key logic here is just an Informer on the object you put the finalizer annotation on, and checking if its object.metadata.deletion_timestamp is a Some.
Might be worth adding a helper so mutate the finalizer list (an impl on ObjectMeta perhaps), but will sort that out once the example is being converted

Going to close this. The last; finalizer is now a lot easier with the last finalizer helper prs, and there's an example for it.


Unable to Start Demo

Describe the bug
When I attempt to watch a demo, I keep getting the notification that an error has occurred.
Demo links
No response
Steps to reproduce

Start Application
Analyze The Demo
Try to Watch Demo

Expected behavior
I expected the demo to be launched is CS
How often does this bug happen?
Every time
Log file
Wed, 06 Dec 2023 04:25:21 GMT | ERROR | server | An error occurred while starting the game
Wed, 06 Dec 2023 04:28:20 GMT | LOG | server | WS:: message with name fetch-player-stats and uuid e6034aec-50ab-411d-8550-0053a0d89507 received from renderer process
Wed, 06 Dec 2023 04:28:53 GMT | LOG | server | WS:: message with name fetch-player-stats and uuid ff8bfca8-827e-4442-85a4-c9fd4650a725 received from renderer process
Wed, 06 Dec 2023 04:28:59 GMT | LOG | server | WS:: message with name is-cs-running and uuid 01f7b4f0-1561-47eb-b013-b2eeb0fcc0e3 received from renderer process
Wed, 06 Dec 2023 04:28:59 GMT | LOG | server | WS:: message with name watch-demo and uuid e5ec4d85-8fd2-41f9-80eb-15e8bae89d70 received from renderer process
Wed, 06 Dec 2023 04:29:00 GMT | LOG | server | Starting game with command "D:\program files (x86)\steam\steamapps\common\Counter-Strike Global Offensive\game\bin\win64\cs2.exe" -insecure -novid -netconport 2121 +playdemo "D:\Users\ford\CS Demos\Pro Demos\1-030ee9c3-58c2-48d8-a32b-29cc800aa000-1-1.dem"  -width 1024 -height 768 -sw
Wed, 06 Dec 2023 04:29:00 GMT | LOG | renderer | WS:: message with name "starting-game" received from server
Wed, 06 Dec 2023 04:29:00 GMT | LOG | server | Game process exited with code 1
Wed, 06 Dec 2023 04:29:00 GMT | LOG | server | Game output:
Access is denied.
Wed, 06 Dec 2023 04:29:00 GMT | ERROR | server | An error occurred while starting the game
Wed, 06 Dec 2023 04:29:13 GMT | LOG | server | WS:: message with name is-cs-running and uuid 8f86567a-2ebe-4942-9a33-0cffb9b68f36 received from renderer process
Wed, 06 Dec 2023 04:29:14 GMT | LOG | server | WS:: message with name watch-demo and uuid 9c5dbb76-580c-4701-babd-bb2bc3600b9b received from renderer process
Wed, 06 Dec 2023 04:29:14 GMT | LOG | server | Starting game with command "D:\program files (x86)\steam\steamapps\common\Counter-Strike Global Offensive\game\bin\win64\cs2.exe" -insecure -novid -netconport 2121 +playdemo "D:\Users\ford\CS Demos\Pro Demos\1-030ee9c3-58c2-48d8-a32b-29cc800aa000-1-1.dem"  -width 1024 -height 768 -sw
Wed, 06 Dec 2023 04:29:14 GMT | LOG | renderer | WS:: message with name "starting-game" received from server
Wed, 06 Dec 2023 04:29:14 GMT | LOG | server | Game process exited with code 1
Wed, 06 Dec 2023 04:29:14 GMT | ERROR | server | An error occurred while starting the game
Wed, 06 Dec 2023 04:29:14 GMT | LOG | server | Game output:
Access is denied.
Wed, 06 Dec 2023 04:32:06 GMT | LOG | server | WS:: message with name fetch-last-migrations and uuid 9ee097a0-2325-43e0-a3e4-eb4c2febff97 received from renderer process
Environment information

Verions 3.0.1
Windows 11

Additional context
No response

I will add, this does persist between reboots

I had this problem because I had faceit ac open. closing it made it work

As @ar1a said the "Access denied" exception is due to the faceit AC.
I added a mention about this here.
In the next version, the application will show a notification with a link to the doc when the game crashes


Avoid piping curl'ed scripts into bash

In 3 installers (java, node, ruby), the install.sh script includes a step where a script is curl'ed and piped into bash:
https://github.com/devcontainers/features/blob/7a3a9c5fcaa59cf4d7dbbcece47094a6d642a9b0/src/java/install.sh#L155
https://github.com/devcontainers/features/blob/7a3a9c5fcaa59cf4d7dbbcece47094a6d642a9b0/src/node/install.sh#L121
https://github.com/devcontainers/features/blob/7a3a9c5fcaa59cf4d7dbbcece47094a6d642a9b0/src/ruby/install.sh#L211
Those could be entrypoints for supply chain attacks. I think it would be preferable if these tools could be installed using a step that does checksum verification on the bash scripts before running them.

Hi 👋
Thanks for pointing it out. We have added checksum verification for the Features which installs from artifacts, but looks like they were missed for script verifications.

https://github.com/devcontainers/features/blob/7a3a9c5fcaa59cf4d7dbbcece47094a6d642a9b0/src/java/install.sh#L155
Note: SDKMAN started providing checksum verification for the script.
Ref: https://github.com/sdkman/sdkman-cli/issues/1040#issuecomment-1023455180

I have found alternatives to installing Node safely.
The first method is to use NodeSource's manual installation.
The second method is to download a source code from nodejs.org and then verify it.
The third method is to install pnpm from the pnpm standalone script (it can be verified) and then use pnpm env.
Using the above two methods, it is impossible to manage multiple Node versions.
Using the third method makes corepack unavailable. Currently, corepack is the recommended way to install yarn berry.
Currently, the feature uses nvm, but there is no reason to manage multiple Nodes.
(I think that managing multiple sources in a single container is against the purpose of Dev Container.)
For this reason, I would suggest using the above two means.
Choosing between them is a matter of preference.


[REQUEST] PC Steam: Fate/stay night REMASTERED

https://store.steampowered.com/app/2396980/Fatestay_night_REMASTERED/

The game works just fine with Textractor, though. Is there any need for this?

The game works just fine with Textractor, though. Is there any need for this?

Are you sure it works with textractor? Unfortunately I tried but it doesn't work, is there something I did wrong? Thanks anyway


Transaction abort hangs indefinitely

Hi! First of all, thanks for this library, the API makes working with IndexedDB so much better! I think I've found a bug when calling abort() on a transaction.
When running the following code, the returned JavaScript promise never resolves, and the abort() method hangs indefinitely. The 'Transaction aborted' message never gets logged. In addition, the value is actually committed and stored in IndexedDB.
#[wasm_bindgen]
pub async fn temp() -> Result<JsValue, JsValue> {
    let db = Rexie::builder("temp").version(1).add_object_store(ObjectStore::new("kv")).build().await?;
    let tx = db.transaction(&["kv"], TransactionMode::ReadWrite)?;
    let st = tx.store("kv")?;
    st.put(&JsValue::from("test"), Some(&JsValue::from("test"))).await?; // Removing this makes no difference
    trace!("Transaction aborting");
    let xx = tx.abort().await?;
    trace!("Transaction aborted");
    Ok(JsValue::TRUE)
}

For comparison, the following code works correctly, and the value is committed and stored in IndexedDB. The 'Transaction committed' message is logged successfully.
#[wasm_bindgen]
pub async fn temp() -> Result<JsValue, JsValue> {
    let db = Rexie::builder("temp").version(1).add_object_store(ObjectStore::new("kv")).build().await?;
    let tx = db.transaction(&["kv"], TransactionMode::ReadWrite)?;
    let st = tx.store("kv")?;
    st.put(&JsValue::from("test"), Some(&JsValue::from("test"))).await?;
    trace!("Transaction committing");
    let rx = tx.commit().await?;
    trace!("Transaction committed");
    Ok(JsValue::TRUE)
}

I'm testing this with rexie-0.2.1

Hi @tobiemh,
Thanks for creating this issue!
This was a silly bug where it was waiting for a transaction to be aborted without calling abort. 🤦🏻♂
Anyway, this issue forced me to delve deeper into IndexedDB's transaction processing and I found that the commit feature is still in working draft. Current commit function in this crate is not forcing the DB to commit the values but waiting for DB to perform auto-commit. I've changed commit -> done (and updated documentation) so that we can add a commit function to transaction when that feature becomes available in IndexedDB.
I'll release a new version soon (Unfortunately it'll be a breaking change).


Amazon International bug fixes

Proposed changes:

Added support for Amazon India in const.py
Previous code was unintentionally parsing all Amazon emails, instead of shipping confirmation mails. Implemented proposal in this comment: https://github.com/moralmunky/Home-Assistant-Mail-And-Packages/issues/125#issuecomment-643792745
International Amazon emails (UK/IN) have subject length == 10, not 6

Type of change

[ ] Dependency upgrade
[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #125
This PR is related to issue: #125

We'll need to look at the subject line situation before merging this.

The subject line is "Your Amazon.x order #y of z item has been dispatched" for UK and IN at least.
Check this comment for screenshot of UK: https://github.com/moralmunky/Home-Assistant-Mail-And-Packages/issues/125#issue-619018025

Ah yes I see, I didn't want to merge it so late without giving a proper view. I have an idea for the subject lines that'll make things easier as well.


A few ideas for new diagnostics for wave-mean interaction problems

Here are some ideas for additional diagnostics inspired by discussions with @jonathanlilly and @JeffreyEarly.
Eigensolver for vertical modes
I think it would be nice to have an eigensolver to compute vertical modes from a horizontally-averaged / reduced buoyancy profile (or taking in the vertical derivative of buoyancy directly). This means solving the eigenproblem
$$ \partial_z \left ( \frac{f^2}{N^2} \partial_z \phi_n  \right ) + \frac{1}{R_n^2} \phi_n = 0 \qquad \text{with} \qquad \partial_z \phi = 0 \qquad \text{at} \qquad z = -H, 0 $$
where $1/R_n^2$ are the eigenvalues, $R_n$ is the Rossby radii of deformation of mode $n$, $f$ is the Coriolis parameter (a constant) and $N^2(z)$ is the vertical derviative of buoyancy (a function of $z$). The eigenfunctions $\phi_n(z)$ are the "vertical modes".
Finite volume details: $\phi$ is located at Center in the vertical, while N^2 is located at Face. This works out naturally so no shenanigans seem to be required here.
I think this can be implemented by meshing Oceananigans.AbstractOperations with Krylov.jl. In other words I am imagining a user interface wherein we write something like
B = Field(Average(b, dims=(1, 2)))
N² = ∂z(B)
ϕ = Field{Nothing, Nothing, Center}(grid)
L = ∂z(f^2 / N² * ∂z(ϕ))
sol = eigensolve(L)

where b would be the buoyancy field and f the Coriolis parameter. Many permutations on the above syntax might make sense too.
Diagnostics for wave-mean decomposition
Related to vertical modes would be diagnostics that allow one to decompose solutions into an internal wave and mean (or quasi-geostrophic) component. I think @JeffreyEarly knows how to do this well.
We can also support a direct solver for the Omega equation; eg solving

using a solver similar to the FourierTridiagonalPoissonSolver (combining horizontal-FFT with vertical tridiagonal). This would require modest (not heroic) efforts. The screenshot is from Danioux et al 2016.

@tomchor we need an issue label for "new diagnostics idea" or something. Presumably ideas for new diagnostics are a prime thing to discsuss here.

A related initiative is using Krylov to solve the Poisson equation in Oceananigans: https://github.com/CliMA/Oceananigans.jl/pull/3812
@amontoison knows how to build matrix-free eigensolvers, I think.

I think the omega-equation approach that you are suggesting is higher-order than what we are doing, but I don't understand the implications.
What we are doing is simply inverting QGPV using the projection operator in equation 78 here. To do the PV inversion with our lower order expression but using your approach you sketched out, I think you could compute,
$$\nabla^2 \psi +\frac{\partial }{\partial z} \left( \frac{f^2}{N^2} \frac{\partial \psi}{\partial z} \right) = Q$$
where Q is just QG PV computed from,
$$Q=\partial_x v - \partial_y u - f \partial_z \eta$$
which is equation 42 in the above paper.

I think the objective is different, the omega equation obtains the balanced vertical velocity, while the QG inversion obtains the balanced streamfunction. So they are complementary / consistent approaches to obtain different variables perhaps?
It seems like the lower-order inversion could be a warm up to doing the higher-order inversion? Or should we ignore the low-order inversion.
It seems like it could make sense to solve a cleverly-forced tracer equation for the isopycnal displacement, rather than attempting to diagnose it. Does that make sense? I think that could be done fairly easily. The only fudging (associated with the staggered grid we use) is that the interface heights $\eta_e$ are at $w$-locations whereas a tracer would be shifted by half a cell. But another option could be to write a totally new dynamics that evolves the interfaces properly, at $w$ locations.

My take on this is that you probably want to do QGPV inversion as a first pass, because it is really the only order at which everything is diagnostically crystal clear. The geostrophic streamfunction gives you $(u_g,v_g,\eta_g)$ from which you can deduce the wave portion of the flow without ambiguity---a separation of which is both complete and energetically orthogonal.
At the next order things get very murky: when you invert the $\Omega$ equation, you now have a geostrophic vertical velocity $w_g$ as well as corrections to $(u_g,v_g,\eta_g)$. But now there are many choices of equations of motion that result in these solutions which conserve (or don't) a variety of different expressions for PV and energy... so as a diagnostic it is more challenging to interpret. Remmel and Smith 2009 have a nice introduction summarizing various results, but maybe there's a newer take on this where things are more clear.
Seems like once you have the QGPV inversion the $\Omega$ equation would be fairly straightforward---so yeah! You should totally do both if it is not too hard.
For the QGPV inversion you can compute $\eta_e$ directly from density perturbation, you'll just need to scale it by $\bar{\rho_z}$. This is how we do it in the wave-vortex model. To compute the true APV, which requires the true isopycnal deviation, we start with a $\bar{\rho}$, and then use the bisection method using some code I borrowed from chebfun. It seems pretty fast, but I haven't fully validated its accuracy.


WIP LGA-2601 and LGA-2602 Add test to edit financial eligibility

What does this pull request do?
Bug showed that a specialist provider could not edit the eligibility check. These new tests check to see if the provider and the operator can edit and save changes to an eligibility check.
Have stopped work on this as unit tests cover the issue and this is an added extra - could be finished.
Note that there is an error
# selenium.common.exceptions.ElementClickInterceptedException: Message: element click intercepted: Element ... is not clickable at point (134, 20). Other element would receive the click: ...         (Session info: chrome=112.0.5615.165)
associated with the step
When I return to the specialist provider cases dashboard page
It wants to click on the top right link that refreshes the page (with the case ref) rather than the back button. No idea why.
Any other changes that would benefit highlighting?
This may need another fixture. Thought we could use the accepted case from fixtures
CLA_SPECIALIST_CASE_TO_ACCEPT = "JT-4272-9443"
but using this case and editing the values results in an error in the test checking the legal help form...
Checklist

[ ] Provided JIRA ticket number in the title, e.g. "LGA-152: Sample title"

@jogilder have you tried using https://pypi.org/project/chromedriver-py/
It has a method for auto locating the chromedriver so you don't have to look it up

@means_test_operator_edit_case is the test to be pursue
Some @todos are guiding the next person for implementation of what's left
This doesn’t have to be finish if the next person want to go directly to point 4.
The goal would be to create a fixture for the test_operator and the specialist_provider
=> To do so, create the fixtures in cla_backend repo (take eg. to legalaid>fixtures)
=> Add them in create_db.sh>load_end_to_end_test_data()
=> Refer the Reference Number attributed in backend in a variable in the E2E repo
=> Bring it inside of your test (take eg. to constant.py>CLA_SPECIALIST_CASE_TO_ACCEPT)

@jogilder have you tried using https://pypi.org/project/chromedriver-py/
It has a method for auto locating the chromedriver so you don't have to look it up

@said-moj this method didn't seem to work.


At times step-files containing DEGENRATE_TOROIDAL_SURFACES fail to mesh properly

In some cases a geometry step-file containing one or more degenerate toroidal surfaces can fail to mesh. It seems to be an underlying problem with the step-importer.

ends up being something like this:


Smarter Dev Setup Scripts

Have Distro setup scripts check for various ways to install rust:
rustup, multirust, etc.
Use that tool to install the correct tool chain or by default install rustup since that will be the preferred way by rust-lang in the near future.
If sqlite libraries are already installed skip.

Rustup is the preferred way now, should aim for that.

Preferably I'd like to have one script that figures it all out.

The script should ensure alchemist is installed, then install the packages needed for developing Alchemist.

It depends on what they are doing. If it's the client then they just need to clone the repo but if it's the server side they'll need setup locally to make changes and test it. This means they'll need the DB setup somehow.

We could have a test db for working with the API code, could use that db for the cargo tests of the API.

Since we're using sqlite it's a small file. I think we should have it available in the repo as a testdb for local tests.

I don't think we'd be using sqlite for the server though, it would be pretty inefficient to do so.

Postgres then? it could be sqlite for a dev dependency and local testing?

That sounds like a good idea.


After first view it wont continue until the vm restarts.

After its done with the first view it wont clear the browser, nor will it continue to Linkvertise. It just loops through and tries to locate Ngrok to Linkvertise. Then because it couldnt find anything, it restarts.
Imgur
Chrome_blank and Chrome_restore is both false, suggesting that on loop it skips this step.
Also I requested a payout on Linkvertise today. I had some problem with my ip being banned, so I had to use a vpn to request it. Not sure why though.

@BhaskarPanja93

@KevinGeLe
Is this loop issue before today's commit or after?
Good luck with your payout, hopefully everything works.

@KevinGeLe Is this loop issue before today's commit or after? Good luck with your payout, hopefully everything works.

Its after it gets one view 3 minutes in on the dashboard, it is stuck that way, but then restarts at around 8-9 minute mark. (I have also tried on another computer), also a few times the clear history doesnt have time enough to clear.
@BhaskarPanja93

@KevinGeLe Is this loop issue before today's commit or after? Good luck with your payout, hopefully everything works.

Its after it gets one view 3 minutes in on the dashboard, it is stuck that way, but then restarts at around 8-9 minute mark. (I have also tried on another computer), also a few times the clear history doesnt have time enough to clear.
EDIT: I had to delete the files on the vm and then let it get them again and now it doesnt restart after 5 minutes of doing a view/nothing. But even on first start 50/50 Chrome_blank and Chrome_restore is both false and it just waits till restart.
If adding so that Chrome_blank and Chrome_restore needs to be true before continuing will make sure, that chrome was opened and ready perform the view, instead of just continue the viewing if it havent even went onto ngrok to get the Linkvertise link.
New edit: And all my screenshot cant be uploaded for some reason
@BhaskarPanja93

Stupidity hit me hard.
I have to have the vm window open, because the mouse wont be captured if its in headless mode. So I have to start it in headless, open the vm window, then wait till the chrome window have opened, click on the search bar and then close vm window.
Its not a fault with the code but rather the mechanics of VirtualBox, I will look into it, maybe all we need is a script that acts as a real mouse and then shut it down after. Because what I have heard pyautogui works be getting the mouse location, but what if it cant because the mouse isnt captured yet.
@BhaskarPanja93

@KevinGeLe Is this loop issue before today's commit or after? Good luck with your payout, hopefully everything works.

Its after it gets one view 3 minutes in on the dashboard, it is stuck that way, but then restarts at around 8-9 minute mark. (I have also tried on another computer), also a few times the clear history doesnt have time enough to clear.
EDIT: I had to delete the files on the vm and then let it get them again and now it doesnt restart after 5 minutes of doing a view/nothing. But even on first start 50/50 Chrome_blank and Chrome_restore is both false and it just waits till restart.
If adding so that Chrome_blank and Chrome_restore needs to be true before continuing will make sure, that chrome was opened and ready perform the view, instead of just continue the viewing if it havent even went onto ngrok to get the Linkvertise link.
New edit: And all my screenshot cant be uploaded for some reason
@BhaskarPanja93

Stupidity hit me hard. I have to have the vm window open, because the mouse wont be captured if its in headless mode. So I have to start it in headless, open the vm window, then wait till the chrome window have opened, click on the search bar and then close vm window.
Its not a fault with the code but rather the mechanics of VirtualBox, I will look into it, maybe all we need is a script that acts as a real mouse and then shut it down after. Because what I have heard pyautogui works be getting the mouse location, but what if it cant because the mouse isnt captured yet.
@BhaskarPanja93

Ignore everything I said now.
The reason why it sometimes doesnt work is because it sometimes takes longer time for chrome to start and there it doesnt capture the search field with "ctrl + l" and therefor it cant continue. Also replacing longer delay to line 155:
def __start_chrome_forced():         global clear_chrome, link         system_caller('start chrome')         sleep(randrange(10,15))         pyautogui.hotkey('ctrl', 'l')         sleep(0.1)         if clear_chrome:             link = 'chrome://settings/resetProfileSettings'         elif not link:             link = get_link()         pyautogui.typewrite(link, typing_speed)         sleep(0.1)         pyautogui.press('enter')         if clear_chrome:             sleep(6)             coordinates = __find_image_on_screen('reset settings', confidence=0.8)             if coordinates:                 __click(coordinates)             sleep(3)             link = 'chrome://settings/clearBrowserData'             pyautogui.hotkey('ctrl', 'l')             pyautogui.typewrite(link, typing_speed)             sleep(0.1)             pyautogui.press('enter')             sleep(6)             coordinates = __find_image_on_screen('clear data', confidence=0.8)             if coordinates:                 __click(coordinates)                 sleep(3)
This have enough time to wait. So that it wont skip past to quickly. Also I had to create a new Instance.py with the parameters already filled out. Or else the code I edited directly into it would be replaced when running vm_main.py.
also I cant seem to find minSearchTime=10 If using this, we could skip the sleep() and just have it instantly click on it if it shows up within 10 seconds like here. More neat code

the reason i didnt put a minSearchTime because i didnt use pytesseract for this code, pytesseract needs Tesseract to be in PATH, which in our case isnt, so i took pyteserract functions in my own code and modified the location of Tesseract binary in it. and i didnt copy the minSeachTime function in it, so i used my own sleep statements, which is almost same. Next update i will integrate this function inside the __find_image_on_screen() function, but the problem with it is, if the image is not on screen, it will wait out the whole duration and waste more time, maybe i will integrate a self-learning model in the code which will know how long to wait for each image in the specific VM, it will make wait times more efficient. Am working on it.
@KevinGeLe

@KevinGeLe
I made some changes, Wait times have been changes, for link typing and button clicking, and i also added the delete Downloads folder and Recycle bin folder to keep local storage clean.
If the bot still has problems clicking links, just show me a video of what the bot is doing from the start, and i will fix it.

Any updates on this one? Problem persists or no?
@KevinGeLe

@KevinGeLe How do I make sure that Chrome_blank and Chrome_restore are rue because mine are always false and the script just loops till restart?

@KevinGeLe How do I make sure that Chrome_blank and Chrome_restore are rue because mine are always false and the script just loops till restart?

It sounds like your vm is just very VERY slow. Have you enabled virtualization in the BIOS? Also make sure that you have allocated 2 vCPUs to the vm's. Instead of the default 1. I run 2 vCPUs and 892 MB of ram with MemReduct installed on the vm's.

I'll try using your config. What about the cookies popup, can the bot get past it too?

@KevinGeLe Everytime memreduct runs, the bot seems to stop functioning, as if it was stopped too. How can I avoid this?


Add project to awesome-runners/reference blog post

:wave: @tfenster ,
I've spent a bit of time to help people find self-hosted GitHub Action solutions, I think its going to be huge and wanted to help people to discover them.
I think your blog post is one of the very few ✨  examples on how to use self-hosted Windows runners, so I added it to the list https://github.com/jonico/awesome-runners 🏃
I have also added a badge (if you are interested):

Keep up the great work 👍

@jonico thanks for adding me!


Docker compose boilerplate

Hi,
Can anyone share working docker-compose.yml file for basic FastAPI code.
Preferred if connected to PostgreSQL
Thanks,
Rakesh

You might find the docker-compose.yml file provided in this project useful. It's much simpler than the full-stack setup provided and a bit easier to understand(although it lacks much of the fancier items).
Using that repo, if you wanted the most basic setup, I'd just use the backend and postgres services and you should be good to go!

Hi @IanSaucy
Thanks for your reply. I am quite new to docker if you don't mind can you please paste minimal Dockerfile and docker-compose.yml file. Hope I am not asking much

It's hard to post docker files without knowing the structure of your entire project. Docker files do not include your source code(ie all the routes for fastAPI etc), instead they say where and how it should run the code you already have -- thus it needs to factor in your file structure. If you have a repo with your project or something I might be able to help.
I would look up some basic docker and docker-compose info/tutorials. Personally, I found it useful to understand the technology a little before implementing it into a project otherwise you end up copy-pasting without an idea of what is happening.
Maybe one of these tutorial would be helpful? Also, I really think the react or full stack example linked in the previous example would be a good starting point.
<EMAIL_ADDRESS>https://testdriven.io/blog/fastapi-crud/
Good luck!

Thank you @IanSaucy Links look amazing I will definitely check them out. I was trying docker for this minimal fast-api temlpate.
The docker files that I have already tried are in readme file. But couln't get them working

Hey @rakesh4real,
Here is a version of these files
Dockerfile

uses Python 3.8
exposes the app on host port 8000
starts the app

FROM python:3.8.1

ENV PYTHONUNBUFFERED 1

EXPOSE 8000
WORKDIR /app

COPY requirements ./
RUN pip install -r requirements.txt

COPY . ./

CMD uvicorn --host=<IP_ADDRESS> app.main:app

docker-compose.yml

Runs the app service
Runs a Postgres database (but feel free to change that to something else or remove it)

version: '3.7'

services:

  app:
    build: .
    ports:
      - "8000:8000"
    depends_on:
      - db
    environment:
      SQLALCHEMY_DATABASE_URL: "postgres://postgres:test@db/postgres"
  db:
    image: postgres:9.6
    ports:
      - "5432:5432"
    environment:
      POSTGRES_PASSWORD: test

Hope this helps

Thank you @mickeypash @IanSaucy. Issue resolved closing it. I have updated the repo with files.

Giving the docker file, could you explain how PYTHONUNBUFFERED afects fastapi?
Thanks

By setting the PYTHONUNBUFFERED=1 or TRUE you are telling Python to not buffer the stdout/stderr streams, see docs. In other words, log messages are immediately dumped to the stream, thus receiving them in a timely manner. This means that you can do docker logs <container-id> and see the logs for your FastAPI app as they come in rather than having to wait for them to be flushed from the buffer.

"This also ensures that no partial output is held in a buffer somewhere and never written to terminal logs in case the python application crashes."

source

That will explain why prints are printed after restarting the server
Thanks!

Thanks for the help here everyone! :clap: :bow:
Thanks for reporting back and closing the issue :+1:


.spec.js files is displaying as normal .js files

.spec.js files is displaying as normal .js files.

Steps to reproduce:

Just create a file ended with .spec.js.

I see you're using Insiders, isn't it? Could you try in stable version? Maybe the vsc team has changed something in the new version.

Can you tell us which specific version of vsc are you using?

It's working for me in Windows:

I think its something with this old Insiders version.
I've updated and it started to work again.
Thank you to pointing me that. :+1:


[FEATURE] Add API Key configuration option to SDK

Is your feature request related to a problem? Please describe.
Since for changing, creating and deletion of an SAP Object, we'll be relying on an API mgmt policy for the xcsrf handling, I would be good if we have a way of proving the API Mgmt key to the c# Client
Describe the solution you'd like
Retrieve the API mgmt key from a configuration file or Azure Key vault
Describe alternatives you've considered
No response
Additional context
No response

This could simply be added to the public options class BasicHttpAuthHandlerOptions.
This would enable us to load it using the existing config method and then add it to the request headers collection in the BuildHttpRequestMessageAsync method of the ODataHttpClientContext.
Line 105 of this class describes how we use the auth string / token, it would be VERY similar to this code if implemented.

OK thankyou MartinPankraz


Fixed issues with the license

We are not the "Astropy Team"!
Extended the copyright years

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

Please upload report for BASE (main@0a47673). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #145   +/-   ##
=======================================
  Coverage        ?   55.83%
=======================================
  Files           ?       20
  Lines           ?     3170
  Branches        ?        0
=======================================
  Hits            ?     1770
  Misses          ?     1400
  Partials        ?        0

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


Enhance dashboard based on user feedback

Relates #164

Show open / closed / in-progress status
Show which admin assigned to request
Ability to change status
Ability for admins to assign admins (including themselves)

This looks great. Did you mean to have an 'Are you sure?' message when you close it? That was mentioned in the issue but isn't part of the behaviour.

Thanks! I thought about that, and I'm not sure it's necessary. They need to do three clicks to close the request (toggle the request window, click 'change status', click 'close') so it's unlikely they'd do it accidentally? Might just be annoying if it's too complicated to close it?

Sounds good to me. In that case, merging.


Oculus Device Manager prefab constantly reverts to non-working state on UPM

Describe the bug
The Oculus Device Manager prefab always reverts changes made after running "Add Oculus Integration Tool". This is due to the prefab's guids not matching the guid's provided by the Oculus Integration Package as it updates.
To reproduce
Steps to reproduce the behavior:

Add MRTK foundations via UPM
Add Oculus Integration package
Run MixedRealityToolkit -> Utilities -> Oculus -> Integration Oculus Integration Unity Modules
See that the OculusXRSDKDeviceManagerProfile asset has the prefabs set correctly
Reboot unity and see that the asset has been reverted.

Expected behavior
DeviceManger settings should saved between sessions

To be fixed in a release coming soon.

To be fixed in a release coming soon.

This is fixed in the MRTK 2.5.4 release that just went out today.


fix non-coalesced frame dimension in case of upsampling

Fixes https://github.com/libjxl/libjxl/issues/1041

Can we add a test?

There is OrientedCroppedFrameTest. Perhaps we have to extend it with "upsampling" dimension.

#1059 also does this, so closing this one


Add SVG parser/renderer for CartoCSS icons

Icon set are usually distributed as SVG files. Currently SDK requires all SVG images to be converted into raster images. For better compatibility, try to integrate NanoSVG parser/renderer into the SDK.

Btw Mapbox converts SVG in style projects to sprite png (texture atlas)+json in servers-side, using https://github.com/mapbox/spritezero

I haven't looked at 4.1 yet, but would this allow us to use SVG image sources with a Mark geometry in a LocalVectorDataSource for example ?

@Shtong Not at the momement, but we may add this in the near future.

Implemented, closing.

this was fast 👍


Suggestion: Add some basic CI

GitHub Actions is fairly easy to setup and doesn't have much of a maintenance burden. I'd be happy to open a PR with a workflow config if it's welcome.

It's very welcome!


[test] Modernize QuantumEspresso test

Codecov Report

Merging #2384 (55e9d86) into master (b466030) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2384   +/-   ##
=======================================
  Coverage   86.14%   86.14%
=======================================
  Files          56       56
  Lines       10225    10225
=======================================
  Hits         8808     8808
  Misses       1417     1417

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 b466030...55e9d86. Read the comment docs.

Would it make sense to parametrize it on the cpu vs gpu variants as well instead of having to additional classes?

It would need more if blocks, that's why I left it as is, since it already contains some.


Next release

It's time to do a new release — I can do this
Are we OK with 0.5.3 or do we do 0.6.0? Ref https://github.com/PRQL/prql/issues/1997
Should I re-revert the switch syntax change now or wait for https://github.com/PRQL/prql/issues/1795? I don't really see much reason to wait, it's experimental and so we should feel free to change it lots

I've commented on #1795 regarding the control value (tl;dr in favour of renaming to case, against control value). What would re-reverting the syntax change mean? Where did we end up on -> vs =>?

How much documentation do we have on loop? I wanted to try this out and maybe write up a blog post about it. Perhaps in combination with ducker?
Given that loop is quite a large feature addition, I think a 0.6,0 release would be appropriate.

Update from dev call: let's release 0.6.0 since we do have breaking changes.
TODO:

[ ] re-revert switch arrows syntax change,
[ ] rename switch to case,
[x] documentation for loop

(FYI I may not get to this today, but will do early this week if I don't)

I think everything is done now


Recursively search for presentationController

In my project I have a more complex view controller hierarchy with embedded view controllers containing a table view. In order to dismiss, when the scroll view of the table view is scrolled to the top and further I use:
func scrollViewDidScroll(_ scrollView: UIScrollView) {
    SPStorkController.scrollViewDidScroll(scrollView)
}

But presentationController(for controller: UIViewController) -> SPStorkPresentationController? will return nil in my case as the modally presented view controller is some levels higher than the view controller containing the scroll view.
The code looks recursively for the presentationController until it finds it or the parent property of the controller is nil.

Thanks for PR!
You can add example project when presentations controller not detected?


fix(interrupts): add compiler fences to enable and disable

This actually bit me on AArch64, but this should apply to x86-64 as well:
Without this PR, the compiler may reorder memory accesses over these asm! calls. While interrupts::enable and interrupts::disable don't make claims about synchronizations, the docs strongly imply synchronization of the calling thread with itself.
Essentially, without compiler fences, code after interrupts::disable may still execute while interrupts are enabled. Conversely, code before interrupts::enable may only execute after interrupts have been enabled. This also translates to without_interrupts: code inside the closure may actually execute with interrupts.
This bug may require very specific conditions to show up. In my case (the Hermit kernel), some aggressive inlining resulted in improper access to a RefCell from an interrupt handler.
What do you think? :)

I'm not yet sure that this is a bug: AFAIK the general consensus is that interrupt handlers are modeled as second threads (even though they execute on the same CPU). Under that model sharing a RefCell with an interrupt handler is unsound because it's not Sync. Also under that model there is no such thing as a thread synchronizing with itself. To make things worse the semantics of compiler_fence are unclear: https://github.com/rust-lang/unsafe-code-guidelines/issues/347.
Can you provide an example of code where reordering with the enable and disable functions could cause problems even under the assumption that interrupt handlers are second threads?

My understanding of compiler fences was based on its documentation. It's unfortunate that they are not properly specified yet.

AFAIK the general consensus is that interrupt handlers are modeled as second threads (even though they execute on the same CPU thread).

Coming from the embedded space, I was thinking that synchronization through compiler fences together with interrupt masking would be sound on the same hardware thread, that normal execution and interrupt handlers share.
AFAIK, the whole embedded Rust ecosystem is built on this assumption: Concurrency—The Embedded Rust Book. See the critical-section crate for more information:

"For bare-metal single core, disabling interrupts in the current (only) core."

The cortex-m crate also uses compiler fences to make this behave as expected: rust-embedded/cortex-m/src/interrupt.rs#L29-L53.

Also under that model there is no such thing as a thread synchronizing with itself.

I took that wording straight from the compiler_fence docs, which provide a motivation using a program being interrupted by a signal.

Can you provide an example of code where reordering with the enable and disable functions could cause problems even under the assumption that interrupt handlers are second threads?

Assuming that this may not be used to synchronize !Sync types across software threads on the same hardware thread, I don't think you can produce unsoundness, since Sync types are synchronized internally. It would still be quite counterintuitive for without_interrupts to run code with interrupts enabled. Furthermore, you might be able to produce deadlocks if you don't synchronize with atomic accesses, similar to what is shown in the compiler fence's docs.
Related issues: I just found https://github.com/rust-lang/rust/issues/30003, which describes the same issue, specifically to RefCells, I think.
There is also https://github.com/rust-lang/unsafe-code-guidelines/issues/444#issuecomment-1676502374, which might go in a similar direction.
Finally, it is probably an opsem question if using compiler fences for synchronization like this is fine. On one hand, it seems difficult to ensure the RefCell's state is reread after the compiler fence. On the other hand, it would seem very unfortunate if you had to use atomics and atomic fences for this kind of synchronization on the same hardware thread, since the hardware would not require any special atomic instructions for consistent memory accesses.
I am a bit lost where to track this. Is this a new opsem issue? Does it fit to one that already exists? Maybe @RalfJung can help?
Leaving opsem aside for the moment, would you consider merging this even if the opsem question is unresolved? It does make without_interrupts consistent and aligns the behavior with common practice from embedded Rust. While not necessarily being fine for synchronization from opsem perspective, this would improve things somewhat, I think.
Alternatively, it might even be necessary to introduce atomic fences instead. If we handle interrupt handlers as different threads, the main thread might unlock a spinlock, enable interrupts and the interrupt handler may lock the spinlock. Without an atomic fence, the unlocking may be reordered to after interrupt enabling.
Sorry for such a big comment, this is quite a big topic. 👀

My understanding of compiler fences was based on its documentation. It's unfortunate that they are not properly specified yet.

AFAIK the general consensus is that interrupt handlers are modeled as second threads (even though they execute on the same CPU thread).

Coming from the embedded space, I was thinking that synchronization through compiler fences together with interrupt masking would be sound on the same hardware thread, that normal execution and interrupt handlers share.
AFAIK, the whole embedded Rust ecosystem is built on this assumption: Concurrency—The Embedded Rust Book. See the critical-section crate for more information:

"For bare-metal single core, disabling interrupts in the current (only) core."

The cortex-m crate also uses compiler fences to make this behave as expected: rust-embedded/cortex-m/src/interrupt.rs#L29-L53.

Also under that model there is no such thing as a thread synchronizing with itself.

I took that wording straight from the compiler_fence docs, which provide a motivation using a program being interrupted by a signal.

Oops, I wasn't aware the compiler_fence uses that wording, thanks for reminding me.

Can you provide an example of code where reordering with the enable and disable functions could cause problems even under the assumption that interrupt handlers are second threads?

Assuming that this may not be used to synchronize !Sync types across software threads on the same hardware thread, I don't think you can produce unsoundness, since Sync types are synchronized internally. It would still be quite counterintuitive for without_interrupts to run code with interrupts enabled. Furthermore, you might be able to produce deadlocks if you don't synchronize with atomic accesses, similar to what is shown in the compiler fence's docs.
Related issues: I just found rust-lang/rust#30003, which describes the same issue, specifically to RefCells, I think.

Yup, this one seems closed to the problem presented here.

There is also rust-lang/unsafe-code-guidelines#444 (comment), which might go in a similar direction.
Finally, it is probably an opsem question if using compiler fences for synchronization like this is fine. On one hand, it seems difficult to ensure the RefCell's state is reread after the compiler fence. On the other hand, it would seem very unfortunate if you had to use atomics and atomic fences for this kind of synchronization on the same hardware thread, since the hardware would not require any special atomic instructions for consistent memory accesses.
I am a bit lost where to track this. Is this a new opsem issue? Does it fit to one that already exists? Maybe @RalfJung can help?

https://github.com/rust-lang/rust/issues/30003 seems like the closest match to me, but it's old and hasn't receive much attention. https://github.com/rust-lang/unsafe-code-guidelines/issues/444 and https://github.com/rust-lang/unsafe-code-guidelines/issues/347 are also relevant though neither of them talk about behavior around !Sync types specifically.

Leaving opsem aside for the moment, would you consider merging this even if the opsem question is unresolved? It does make without_interrupts consistent and aligns the behavior with common practice from embedded Rust. While not necessarily being fine for synchronization from opsem perspective, this would improve things somewhat, I think.

I agree that in practice this might improve things with how people are writing code and even though it's not guaranteed to be a perfect solution I don't think there are significant downsides to this either.

Alternatively, it might even be necessary to introduce atomic fences instead. If we handle interrupt handlers as different threads, the main thread might unlock a spinlock, enable interrupts and the interrupt handler may lock the spinlock. Without an atomic fence, the unlocking may be reordered to after interrupt enabling.

Another alternative would be to remove the no_mem option from the asm! block.

Sorry for such a big comment, this is quite a big topic. 👀

No worries, it's also an important topic.

Yeah https://github.com/rust-lang/unsafe-code-guidelines/issues/347 still mostly summarizes what I know about compiler fences. Pretty much the only situation where I can make sense of them is for signal handlers, which in C++ are other threads kind of -- they are other threads except that compiler fences are sufficient for synchronization. Arguably on a single-core CPU in the kernel space with preemption disabled, interrupts are like signal handlers. So it could actually make sense to use them here. I do lack all the context to evaluate the details though.

Thanks so much for clarifying!

compiler fences are sufficient for synchronization

That's precisely what I was hoping for! 🥳

Arguably on a single-core CPU in the kernel space with preemption disabled, interrupts are like signal handlers.

That's mostly our use case. In our case, we might also have multiple cores, but each core has independent interrupts and is only expected to synchronize with those. So that should be fine then.

Yes, that sounds about right. But this is a bit outside my comfort zone so I suggest asking on Zulip. :)


Uncaught TypeError: Cannot read property 'destroyItem' of undefined

This error appeared after removing a project folder from the Atom tree view, I noticed a file was still showing in the editor pane, when attempting to close the file tab this error showed.
Atom Version: 1.4.0
System: Mac OS X 10.10.5
Thrown From: tab-close-double-click package, v2.0.1
Stack Trace
Uncaught TypeError: Cannot read property 'destroyItem' of undefined
At /Users/pedrosilva/.atom/packages/tab-close-double-click/lib/tab-close-double-click.coffee:54

TypeError: Cannot read property 'destroyItem' of undefined
    at tabs-tab.<anonymous> (/Users/pedrosilva/.atom/packages/tab-close-double-click/lib/tab-close-double-click.coffee:54:18)
    at atom-pane-container.jQuery.event.dispatch (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-space-pen-views/node_modules/space-pen/node_modules/jquery/dist/jquery.js:4435:9)
    at atom-pane-container.elemData.handle (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-space-pen-views/node_modules/space-pen/node_modules/jquery/dist/jquery.js:4121:28)

Commands
     -0:23.1.0 tabs:close-all-tabs (div.title.temp)
     -0:17 tree-view:remove-project-folder (div.header.list-item)
     -0:15.2.0 application:add-project-folder (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "sublime-default-syntax"
    ],
    "disabledPackages": [
      "double-click-tree-view",
      "metrics",
      "project-manager"
    ],
    "openEmptyEditorOnStart": false,
    "packagesWithKeymapsDisabled": [
      "tree-view"
    ]
  }
}

Installed Packages
# User
atom-pair, v2.0.10
auto-indent, v0.5.0
autohide-tree-view, v0.25.1
build, v0.52.0
emmet, v2.3.17
language-latex, v0.6.1
latex, v0.28.2
linter, v1.11.3
pdf-view, v0.39.0
sublime-default-syntax, v0.2.7
sync-settings, v0.6.0
tab-close-double-click, v2.0.1
terminal-plus, v0.14.5

# Dev
No dev packages

Strange. Can't localize the problem. I've made a new version that will not give you this exception. I'll try to catch the problem. Let me know if you'll have problems again.

I have updated to the newest package version 2.0.2, I will reopen the issue if the problem persists


loss = sum(layer_losses)

Hello, I am new to pytorch and this scene in general, but when I run the code I get an error;torch.sum()  Use tensor.detach().numpy() instead.
I am not familiar with how to detach or couldnt find an example of detaching each tensor on the list.
Is the following equivalent code?
:
for elem in layer_losses[1:]:
loss += elem

Same problem here

loss = torch.sum(torch.stack(layer_losses))


[S2GRAPH-133] Deprecating Vagrant environment.

Deprecating Vagrant dev environment as per discussed on the dev mail list.

+1
@HyunsungJo Thx for you contribution.


Conversation Analysis for .NET (February 2022)

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'IoT:0.07437525,Cognitive - Text Analytics:0.057791986,Storage:0.03947495'


Notifications module is getting skipped

@d3db0t, the notifications module is getting skipped because of the absence of the WORDS constant on runtime. It's location needs to be changed to melissa/ from melissa/actions. Thanks!

This issue would be solved by https://github.com/Melissa-AI/Melissa-Core/issues/177.


Make this repository private?

I'm a Professor at Stanford, and I'm teaching an Operating Systems course that uses Pintos for its class projects. I'm writing because an automated search of GitHub identified this repository as possibly containing solutions for the Pintos programming projects. This is a problem for us, because it makes it very easy for students to cheat. Even if they just look at your solution, and don't copy it, that will still reduce the educational experience for them. If this repo does indeed contain Pintos solutions, would you be willing to take it down or at least make it private? If so, that would be a great help to us.

I can't make it private since github doesn't support private repos (for free).
I'll take it down later today.
btw, thank you for such project. It was by far the best project I've done in my programming career.


Phoebemirman/ch14232/keep user logged in after browser refresh

This pull request introduces 1 alert when merging f4617cc8ea565814cdd48926afa244a36f463136 into cb535f433371a71c2a752a50d98dec0f0d5fc202 - view on LGTM.com
new alerts:

1 for Clear text storage of sensitive information

Comment posted by LGTM.com

Coverage decreased (-0.04%) to 63.365% when pulling f4617cc8ea565814cdd48926afa244a36f463136 on phoebemirman/ch14232/keep-user-logged-in-after-browser-refresh into cb535f433371a71c2a752a50d98dec0f0d5fc202 on master.

Coverage decreased (-0.04%) to 63.365% when pulling f4617cc8ea565814cdd48926afa244a36f463136 on phoebemirman/ch14232/keep-user-logged-in-after-browser-refresh into cb535f433371a71c2a752a50d98dec0f0d5fc202 on master.

This pull request introduces 1 alert when merging d1a3335140254f5d628d62c6bc639ba01e3c301f into cb535f433371a71c2a752a50d98dec0f0d5fc202 - view on LGTM.com
new alerts:

1 for Clear text storage of sensitive information

Comment posted by LGTM.com


New Development Strategy

At first, we thought that at some point, the OpenTelemetry project would slow its development pace and stop breaking things, like what happens in many Go libraries.
We intentionally waited for this moment to continue otelchi development. However, we were wrong; instead of slowing down, its evolution is becoming much faster. This makes otelchi quickly obsolete in the OpenTelemetry ecosystem. 😅
On the other hand, we cannot just make otelchi comply with the latest version of the OpenTelemetry Go library because we need to set the minimum Go requirement to at least 1.20, which will make this library inaccessible for many Go versions (1.16 - 1.19).
Yet, people usually start to implement tracing when their system is already proven to be stable. So changing the minimum Go version of the system to use the latest version of the otelchi could lead to many unseen (and possibly catastrophic) consequences.
With these constraints in mind, I would like to share the new development strategy for this library.

base otelchi version
min go version
semconv version
otel version

0.6.0
1.16
1.10.0
1.7.0

0.7.0
1.17
1.12.0
1.10.0

0.8.0
1.18
1.17.0
1.14.0

0.9.0
1.19
1.17.0
1.19.0

0.10.0
1.20
1.20.0
1.24.0

So, first, we will develop version 0.6.0 by adjusting the otelchi implementation with the versions of semconv and otel shown in the table. After version 0.6.0 seems stable, we will start developing 0.7.0. This cycle repeats until 0.10.0.
So what happens if there is a bug in version 0.6.0? We will patch it and release version 0.6.1. Then, we will merge this patch into all versions after 0.6.1 whenever applicable.
After these versions are stable, we will start following the compatibility policy chosen by OpenTelemetry, which is mentioned here.

Some may ask how I get the version mapping for the semconv, otel, and Go versions. I get it from summarizing the repo of the otelchi sister library, which is otelmux. Here is the summary for it:

go version
otelmux version (max)
semconv version (max)
otel version (max)

1.16
1.7.0
1.10.0 (source)
1.7.0 (source)

1.17
1.10.0
1.12.0 (source)
1.10.0 (source)

1.18
1.15.0
1.17.0 (source)
1.14.0 (source)

1.19
1.20.0
1.17.0 (source)
1.19.0 (source)

1.20
1.24.0
1.20.0 (source)
1.24.0 (source)


Separation of aquatic and terrestrial species code tables

[ ] separate list into aquatic and terrestrial plant species (@mikeshasko did you do this already?)
[ ] ensure terrestrial plant forms using to terrestrial list
[ ] ensure aquatic plant forms using aquatic list

@mikeshasko did you do this already?


YouTube example not functioning?

It looks like the YouTube Recycler demo is not functioning, as it's not getting a data response. I changed the playlist and am receiving the same blank display. I've began looking into this, but if there's already a known-fix, please advise!

@rtnlsltn maybe it is that your API key needs to be activated. Please double check the Google API Console.


Wagtail Documentation favicon icon is missing (Not Found)

Pertinent section of the Wagtail docs

https://docs.wagtail.org/en/stable/getting_started/index.html
https://docs.wagtail.org/en/stable/getting_started/tutorial.html
https://docs.wagtail.org/en/stable/reference/index.html
https://docs.wagtail.org/en/stable/reference/pages/index.html
The issue persists in all the pages of documentation.
Details

Wagtail has a nice favicon as per the logo which is displayed fine on this page of the documentation https://docs.wagtail.org/en/stable/
But on all the other pages the favicon is missing and not showing on the tab of chrome or any other browser tabs. When I checked the page source I found the favicon.ico is linked via <link rel="shortcut icon" href="../../_static/favicon.ico" /> and this is going to https://docs.wagtail.org/en/_static/favicon.ico which is Not Found!
When I checked other sources for example CSS or logo image I found that is sourced like src="../_static/img/wagtail-logo-new.svg and takes to https://docs.wagtail.org/en/stable/_static/img/wagtail-logo-new.svg which is correct.
The difference between the favicon going 404 and the logo being available is that the favicon icon source is '../../_static' with an extra ../ which needs to be removed.

Working on this

It's a very minor fix and I already described the issue above. I could fix it but I am not sure exactly where in the documentation this favicon is coming from.
Anyone can contribute to this. View our contributing guidelines, add a comment to the issue once you’re ready to start.

Thanks @dostogircse171 - we'll keep this issue open but it also has been raised on the Sphinx theme.
https://github.com/wagtail/sphinx-wagtail-theme/issues/281
We thought we had fixed it in time for 6.0 but it wasn't a full fix.
Any suggestions for code changes or a PR would be appreciated on that repo if you have time.
There is this PR up also that may fix it. https://github.com/wagtail/sphinx-wagtail-theme/pull/282

Please Assign me this task

Please Assign me this task. I can solve this issue, please assign it to me.

Actually I am working on this issue. Have find the solution and will fix it as soon as possible.

Please Assign me this task. I can solve this issue, please assign it to me.

Actually I am working on this issue. Have find the solution and will fix it as soon as possible.

Ohk Bro no issues you can give it a go if you figured this out. Thank you.

This has been fixed in upstream https://github.com/wagtail/sphinx-wagtail-theme/pull/282
Once a new release of the sphinx-wagtail-theme package is out, I'll update our version in setup.py in #11660.


[Bug]: LinkageError thrown when the source has more than one module var declaration that uses the wildcard binding pattern

Description
$title.
Steps to Reproduce
import ballerina/io;

() _ = io:println("hello");
() _ = io:println("hello");

Running executable

Error: LinkageError occurred while loading main class $_init
        java.lang.ClassFormatError: Duplicate field name "_" with signature "Ljava.lang.Object;" in class file $_init

Affected Version(s)
2201.3.0
OS, DB, other environment details and versions
No response
Related area
-> Runtime
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Consider the following module level variable declarations,
int _ = 5;
int _ = 7;
int x = 10;

Currently the BIR emits,
 $annotation_data map<any>;
_ int;
_ int;
x int;

public .<init> function() -> error{map<ballerina/lang.value:0.0.0:Cloneable>}|() {
    %0(RETURN) error|();
    %1(TEMP) typeDesc<any|error>;
    %3(TEMP) int;
    %4(TEMP) int;

    bb0 {
        %1 = newType map<any>;
        $annotation_data = NewMap %1;
        %3 = ConstLoad 5;
        %4 = ConstLoad 7;
        x = ConstLoad 10;
        %0 = ConstLoad 0;
        GOTO bb1;
    }
    bb1 {
        return;
    }

}

Here two module level variables are created with the var name _.
Since BIR uses the symbol names to create references for the BIRGlobalVariableDcl, there are two fields with the name _ created in the $_init class.
It seems the more correct way to fix this is by generating proper distinct names for the module variables declared with wildcard binding patterns in parser level.

Removing jBallerina label since it's more appropriate to fix it from compilerFE

Also see https://github.com/ballerina-platform/ballerina-lang/issues/34610#issuecomment-1010923887


Proper Way of Selecting an Option from a Dropdown List

Hi. What's the current best way of doing this in protractor tests:
select('orderProp').option('Alphabetical')

for
<select ng-model="orderProp">
   <option value="name">Alphabetical</option>
   <option value="age">Newest</option>
 </select>

This was taken from the tutorial on http://docs.angularjs.org/tutorial/step_04 (last test at the end of the page). I am trying to convert e2e tests to protractor as an exercise and can't figure out a clean way to select an option from a drop down list. Some examples found on the web seemed too verbose and, perhaps, outdated. Thank you.

Hi there, the solution you have suggested above is not working for me and my dropdown element is not having model attribute. it has id locator and under that i have some options.

Hi,
I tried the way @demisx proposed, but wasn't working fine completely. I modified what he recommends  dividing it in two steps as:
element(by.model('orderProp')).click();
$('[value="age"]').click();
Im not shure if the first step is necessary...
Try this @SeleniumShooters, this is working for me 10/10.

I read this solution from somewhere and it perfect that selecting by option number, and therefore wrote a method that takes an element and the optionNumber, and selects that optionNumber. If the optionNumber is null it selects nothing (leaving the dropdown unselected).
var selectDropdownbyNum = function ( element, optionNum ) {
  if (optionNum){
    var options = element.findElements(by.tagName('option'))
      .then(function(options){
        options[optionNum].click();
      });
  }
};

Hi,
My scenario: I wanted to search address and need to click the search suggestions.
I have a html like below:

Address Results

Hyde
, Cheshire England United Kingdom

Hyde
St, Spotswood VIC 3015

Hyde
's Quality Drycleaners, 28 Hamilton St 3437

----------------
Below is my protractor spec:
element.all(by.css('.tt-suggestion.tt-selectable')).first().click();
I can able to select and click the search suggestions on chrome, InternetExplorer, MicrosoftEdge.
But in firefox, click() is not happening.
Could you please anyone help me.

guys use this command
select obj=new select(driver.findElement(by.xpath("//select[@class='gwt-ListBox']")));
obj.selectbyvisibleText(" ");
guys as per your  convenience  use locators and depends on situation use selectbyvalue or selectbyindex

It is really working thanku  @demisx
// JavaScript
element(by.model('orderProp')).$('[value="age"]').click();

Can anyone tell me that for jasemine language in protractor, this will be the code of selecting one option out of dropdown list.
element.findElements(by.tagName('option')) ,will this be used inplace of "....element.all()..."????
var selectDropdownbyNum = function(element, optionNum) {
if (optionNum) {
var options = element.all(by.tagName('option'))
.then(function(options) {
options[optionNum].click();
});

@123-anju it didn't worked for me. i used element.all(...)...

YES! this is absolutely what I meant! Sorry for the confusion.
var select = element(by.model('orderProp'));
select.$('[value="age"]').click();

This is perfect!!
works for me /:

Hi I want to display all the value coming in Dropdown using Typescript protractor can anyone tell me how to do it


 start fabric-samples as -n mode

start fabric-samples as -n mode (do not deploy the chaincode mycc), and detach remove_mycc

Codecov Report

Merging #455 (241ec73) into master (1f64632) will decrease coverage by 6.29%.
The diff coverage is 11.96%.


inject monaco instance when init instead of cdn

The community keeps asking how to load monaco from node_modules, here's the solution. Now you can inject the monaco instance to loader by giving an option to init.
import * as  monaco from 'monaco-editor'
loader.init({ monaco }).then(result => {
  console.log(monaco === result)
})

The first init will call out a promise, every init call behind will return that promise.
you can reload also
import * as  monaco from 'monaco-editor'
loader.init({ monaco }).then(result => {
  console.log(monaco === result) // true
})

// somewhere in code
loader.dispose()
loader.init().then((result) => {
  console.log(result) // monaco from default cdn
})

#14
suren-atoyan/monaco-react#327
suren-atoyan/monaco-react#322

Very cool - just for visibility for others on the issue, https://github.com/suren-atoyan/monaco-react/issues/217 has a good related discussion

@suren-atoyan Is there any problem I could help with?

Why this has not been merged yet @syumai

This would be really nice to have. Any reason to not merge? This project is great BTW

@suren-atoyan are you still around?

@suren-atoyan I'm sorry to bother you, but would it be possible to merge this?
Thank you for this project!

@suren-atoyan my project is stuck as well, would be cool if this gets merged please

Hi everyone 👋
Sorry, for the late answer. This will be ready this weekend 🙂
@pakholeung37 thank you for your support 💚 There are some inconsistency issues in the PR, plus I have other changes in other places, so, most probably I'll close your PR, but your suggested API will be shipped 🚢 🙌

hey everyone 👋 @Sparkenstein @BrianMcBrayer @ShivamJoker @pakholeung37 @ColmanHumphrey
The new version is released 🎉 v1.3.0 check this
Thanks everyone for support!


[SCB-947] delete old jackson protobuf logic

Coverage decreased (-0.03%) to 86.17% when pulling b1b678a540769a9eb3e0d3b97c060662a264e96a on wujimin:delete-old-jackson-protobuf-logic into 6aadae0de82b3b3940f0c163b815d6675aedc2bc on apache:master.