Added theme logo

I created a quick logo in Inkscape, because every certified cool project needs a logo.

I didn't miss this. I am going to apply it to my dev site and use it for a bit, and brainstorm if there are any small tweaks I want to make before committing. Logos are something I usually put a lot of thought into. I like a lot of the design aspects you have used for it!

All good. Be aware of Inkscape's metadata. I purposefully included Metadata in the SVG which gives licensing, authoring, etc, but Inkscape also saves your local path of the file by default, I found. You'll want to scrub the SVG for things that might dox you. For example, there was a variable which lead to '/home/simbleau/...' - where imbleau is my last name - and that might be privacy you value.

I got an idea for similar but different logo to this one. to use in the top left corner of the demo site.
What I am thinking is that I will make a small tweak to this one to match the one that I have in mind, and this one can be used at the top of the readme, and the one that I have in mind can be used in the top left of the demo site.
I need to make some small tweaks to this one so that the two will look similar

Sounds good! You've got total creative freedom, this is your project. :)

Ok, here is the one I made for the top left corner of the demo site:

Ok, here is the one I made for the top left corner of the demo site:

Or one the scales to full width of its bounding box:

The logo that you made does not display correctly in my file browser (dolphin), I tried to figure it out, im sure inkscape is to blame but I am not sure what is causing it:

Now Im thinking I should split these 2 paths into 3, and make the A frame a bit thicker than the bridge part, after looking at your logo and some font As

Very cute! I like it more!

Here is the small tweak I applied to it, I thickened the outer A frame and thinned the bridge slightly:

full:

not really sure, still starring at it and thinking about it, let me know if you have any thoughts.

Your Logo gave me the inspiration and motivation to do this, thanks!
I incorporated some of the idea into the one I created.


catkin should handle different etc folder layouts

Currently, the etc folder is assumed to be inside the installation prefix, e.g. /opt/ros/groovy/etc for /opt/ros/groovy. This is incorrect for the /usr path, which expects /etc to be used.
See:
https://groups.google.com/forum/?fromgroups=#!searchin/ros-sig-buildsystem/etc/ros-sig-buildsystem/eVaiQ4Annu4/ekNaasxxYo0J
This is a problem for packaging catkin packages as /usr prefixed binaries. There are other locations where this is possible a problem, like rospkg.
@dirk-thomas if you still have the searches up can you reference the affect packages that we know about?

Closing due to inactivity.


DevTools failed to load SourceMap

How fix this error?
https://ibb.co/fCj8MSm

Is swiper-bundle.min.js.map file exists in /wp-content/themes/magazine/js/?
Can you open this file directly?


[3.0] Reader - Explode and style improvement

There's some validations to add in the explode an style properties of the parameter object.

[x] The style parameter depends on the "in" attribute
[x] The explode depends on the parameter type ("in") and the schema type.

@fehguy @webron

fixed in https://github.com/swagger-api/swagger-core/pull/2294


Update app-governance-visibility-insights-view-apps.md

@batamig

Pulled into https://github.com/MicrosoftDocs/CloudAppSecurityDocs-pr/pull/3591


Add debug terminal context menu

Description
Adds the debug terminal command to the terminal context menu. I didn't know this shortcut existed, but had checked the context menu, and only saw copilot. Should be helpful for others if they aren't aware the shortcut exists.

development notes

I used navigation@top for the group so its up on top, but happy to change to another one.
I changed the title of the command with the prefix Continue: because it wasn't clear to me by the command name that it used the extension. Open to other ideas but this is what Copilot does in it's context menus

Checklist

[x] The base branch of this PR is preview, rather than main

This is great! I think you made all the right choices


How to cover rotEdge(variable in getManhattanAndAlign.m ) to t7 ?

I have successfully ran your code getManhattanAndAlign.m and test testNet_pano_full.lua on your demo, but i want to try it on my own pic.
I see your code
lne_ts = torch.load('./data/panoContext_line_test.t7')
and
img_ts = torch.load('./data/panoContext_img_test.t7')
it needs t7 file, but how can i cover  variable in matlab to t7 file?  it should be saved to image then saved to t7 file ? really thank you for your help.

@Ein027 You can now refer to the sample code "preProcess_pano.lua" that converts gt data to .t7 file.

@Ein027 Yes you need to save the images to png/jpeg format first then use the "preProcess_pano.lua" scripts to convert them to .t7.

thank you so much, i will try it!

Thanks ！


Extract LINUX_VERSION_CODE from running kernel via vDSO.

This method avoids runtime header files or regexes, and the vDSO
structure should remain stable over time. Fall back to the compile time
constant if search fails.
Closes #206
Closes #274
See comments at end of #274 leading to this approach

I've pushed another version that restores the old strategies, but augments the first with the new method.

I tested your patch locally and it didn't seem to fix the error:
$ sudo ./build/src/bpftrace -e 'tracepoint:syscalls:sys_enter_read /comm == "a.out"/ { printf("hit\n") }'
chdir(/lib/modules/4.18.9-200.fc28.x86_64/build): No such file or directory
definitions.h:9:3: error: unknown type name 'u64'
definitions.h:11:3: error: unknown type name 'size_t'
Attaching 1 probe...
Error loading program: tracepoint:syscalls:sys_enter_read (try -v)

$ sudo ./build/src/bpftrace -e 'tracepoint:syscalls:sys_enter_read /comm == "a.out"/ { printf("hit %d\n", args->fd) }'
chdir(/lib/modules/4.18.9-200.fc28.x86_64/build): No such file or directory
definitions.h:9:3: error: unknown type name 'u64'
definitions.h:11:3: error: unknown type name 'size_t'
Attaching 1 probe...
^C

I'm not fully convinced I have the same issue. I'll keep debugging too.

Looks like I'm hitting #198 . Please ignore me :)

I haven't done C in quite a few years, as best I can tell the integer types / arithmetic is fine, but I always sucked at this stuff even when I did C regularly :)  Stylistically also wasn't sure about 'auto' use either

thanks @dw and @danobi. Looks good, and works for me.


automatic upgrade

I am not sure if part of this content: how can i make sure that when i push changed code for a module (e.g. in registry) one or many Edge devices automatically detect changes and download new module's image? And how can i do it programatically?
thanks.

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

ID: 877a68cb-b6ad-52a2-64ff-2965e936b3a0
Version Independent ID: 07f38c9a-bafb-bfeb-01e9-cfc420d844b5
Content: Deploy, monitor modules for Azure IoT Edge
Content Source: articles/iot-edge/how-to-deploy-monitor.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@smoms  Thanks for the feedback! We are currently investigating and will update you shortly.

@smoms When you make changes to a module you can use tags to keep track of the new versions. For example, in the solution templates in VS Code we start all modules off at version 0.0.1.

You can update the version number in the module.json file, say to 0.0.2, then build and push the changes to your registry. Then, update your deployment manifest(s) to point to the new version number.

This causes the Edge device to pull the new version of the module. You can see which version of a module is deployed to (desired) and running on (reported) a device by clicking into the module information in the Azure portal.

@kgremban many thanks! last question is, how can i push deployment manifest programmatically?

@smoms By programmatically, do you mean that you want to set up a CI/CD pipeline? We have one article about that currently: https://docs.microsoft.com/en-us/azure/iot-edge/how-to-ci-cd. If that's not what you're looking for, can you describe your scenario more specifically?

@kgremban i mean how to programmatically check in/push the deployment manifest. thanks

@smoms You didn't really clarify the question, you just changed the word order around :) Are you saying that you want to check a deployment manifest JSON file into your source control system, and then have it automatically update certain devices/targeted device groups that you've set up previously?

@kgremban you have explained the process to make changes:

You can update the version number in the module.json file, say to 0.0.2, then build and push the changes to your registry. Then, update your deployment manifest(s) to point to the new version number.

I know how to push code in the registry by command line. My question is how to push the through command line or API also the deployment.json? I understand that code is pushed in a registry like Docker or ACS whereas the manifest into IoT Hub registry, therefore i assume there are also different API? I know how it works in VSC but is manual process (right-click).

Thanks for the details, @smoms. We do have an article about deploying modules using the IoT extension for the Azure CLI: https://docs.microsoft.com/en-us/azure/iot-edge/how-to-deploy-modules-cli

@kgremban thanks lot. So the CLI command would be the following right?
az iot hub apply-configuration --device-id [device id] --hub-name [hub name] --content [file path]

Yes, that one works but is being deprecated. The newer version is az iot edge set-modules with the same parameters (device ID, hub name, file path). The article should be updated to reflect the new version of the command this evening.
Is there anything else we can help you with, or have we resolved this issue?

@kgremban thanks lot. please close the issue.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


cannot use namespace and scope resolution operator in .erpc file

Hi,
I am trying to include namespaces and to use scope resolution operator in .erpc file. But i m getting error while compiling.Can anyone give a solution to include these two in my IDL file.
Thank you...!

As for including namespaces, try to do it via including a dedicated header file, use the @include("include_file.h") annotation. As for scope resolution operator in .erpc, I am afraid this is not supported.

As for including namespaces, try to do it via including a dedicated header file, use the @include("include_file.h") annotation. As for scope resolution operator in .erpc, I am afraid this is not supported.

Hi MichalPrincNXP,
Thanks for your reply. In my case i have the callback function needed to be implemented in the client side but the callback function and its parameters should be there under my defined namespace.
Can you suggest me how to implement this because i am facing issues while i m using scope resolution operator in .erpc file as callback function to be defined in .erpc file.
Thank you...!

I am afraid I am not able to help you much, @Hadatko could you please advice?

Hi,
the issue here is that until now the priority was supporting C use cases. I think you want C++ specific feature. C++ specific features are not yet supported.

Hi, i am preparing version where C++ related files will be generated erpc namespace. Is that enough for you or do you need your custom namespace.

i think we understand this issue wrongly. Solution should be implement callback generated by erpc where you can call your callback inside. We will see if we want support more in future, but i think this will be no priority soon.


self.ds.tag()['AREA_OR_POINT'] gets updated into 'Area' when cropping (updating)

see #120

Could you elaborate on this, @rhugonnet ? In #265, I was very confused as to why the "Area" representation is hardcoded after certain operations.

Some more details: when using the crop() function (and maybe others! might come from the from_array), the self.ds.tag()['AREA_OR_POINT'] property of the Raster is updated into Area even if it was originally Point.
This might be problematic. So we need to check if:

This is respecting the pixel position (i.e. shifting the grid by half a pixel), which would be fine.
Overwriting the AREA_OR_POINT attribute without making any grid change, which would be a wrong behaviour.

Thanks for the additional info!
Perhaps a test-driven approach could verify this: we write the tests with the expected outcome and see if the functionality fits?

Yes! This thread is a bit old and I was not so familiar with Python tests at the time :laughing:

Oh yeah, it's almost a year old! I went through old issues to see how many were solved by removing the memoryfile, and this caught my attention.


Screen cheese with terminal info icon

have a terminal that shows the I
open search
🧀 both overlap unpleasantly

@Tyriar How do I now see this notification/warning?

"terminal.integrated.environmentChangesIndicator": "on"

thanks


Remove old workflows from actions history

The list of workflows under "All Workflows" is getting longer  and longer as it shows every workflow we've ever used in the past, including dupes. There is no way to clean up through GitHub yet, but there are some scripted workarounds maybe worth looking at. See https://github.com/orgs/community/discussions/26256

Removed a few of these. (Release only has one now for example)
Archiving for now.

I just did a few by hand. Many had just a few runs from tests


Hosting with Microsoft.AspNet.Server.WebListener does not work in RC1

Functional impact
If you use Microsoft.AspNet.Server.WebListener as your server when hosting an Asp.Net 5 application it fails during startup, terminating the process.
Minimal repro steps

Run Visual Studio 2015 Update 1 NOT as an administrator*
Create a new Asp.Net 5 project using the project template in Visual Studio 2015 Update 1
Add a dependency to Microsoft.AspNet.Server.WebListener in project.json.

"dependencies": {
....,
"Microsoft.AspNet.Server.WebListener": "1.0.0-rc1-final",
....
},

Add a new command for WebListener
"commands": {
...,
"webListener": "Microsoft.AspNet.Server.WebListener",
...
},

Start the webListener Command.

Expected result
A console app should start, hosting the web application using WebListener and listening on port 5000 (this worked OK in beta-8)
Actual result
The console app crashes with the following exception:
Microsoft.Net.Http.Server.WebListener: Information: Start
Microsoft.Net.Http.Server.WebListener: Information: Listening on prefix: http://localhost:5000/
Microsoft.Net.Http.Server.WebListener: Error: Start
Microsoft.Net.Http.Server.WebListenerException (0x80004005): Access is denied
at Microsoft.Net.Http.Server.UrlPrefixCollection.RegisterPrefix(String uriPrefix, Int32 contextId)
at Microsoft.Net.Http.Server.UrlPrefixCollection.RegisterAllPrefixes()
at Microsoft.Net.Http.Server.WebListener.Start()
Further Information
Firewall is configured correctly to allow listening on port 5000.  This can be tested by using Microsoft.AspNet.Server.Kestrel instead, which works fine.
The host machine is running Windows 7 Professional Service Pack 1
*WebListener will start if run as an administrator but you get "HTTP Error 503.  The service is unavailable" if you try and access the application.

Are you trying to run this behind the platform handler? That's not supported. Use kestrel.

@davidfowl - do you mean IISPlatformHandler?  As in app.UseIISPlatformHandler() in Startup.Configure()?  The behaviour is the same with or without it.  I want to self host the application on Windows (i.e. no IIS) and I need Windows Authentication so I think WebListener is my only option??

WebListener self-host should be fine, just don't use it with IIIS/Express (the default F5 in VS).

That port may have been reserved for another app. See netsh.exe. You can try another port by adding "webListener": "Microsoft.AspNet.Server.WebListener --server.urls=http://localhost:5001/"

@Tratcher Thanks for the response.
At some point I had used netsh to allow listnening on port 5000 for the user I am using to run the asp.net application:
netsh http add urlacl url=http://+:5000/ user=<user>
Until now I thought that was required to allow listening for http on a port.  In fact it prevents you listening on a port with WebListener!  Kestrel will work though.

As far as I am concerned the issue can be closed, but I am not sure I am able to do it??

The issue is that the url you pass into netsh must match the one you give to weblistener. You do not need to preregister with netsh for localhost or running as admin.
Kestrel is unaffected by netsh because it uses raw sockets, not http.sys.

I thought the url did match, but maybe I was missing a / or "+" doesn't work in this instance.  Anyway, I am working with localhost, which is all I need right now.  Now I know what is required I'm sure I can work out what I need to do when I want to access my application from remote machines.
Thanks for your help.

does not match localhost from a registration perspective, registrations need to be exact string matches. If you used + in both places then it would match localhost sent from the client.


Use prepend_before_filter in InvitiationsController

Are you open to using prepend_before_filter instead of before_filter in InvitiationsController? This is the standard way that devise ensures that filters are run before any before_filters in your subclasses.
See https://github.com/plataformatec/devise/blob/master/app/controllers/devise/registrations_controller.rb for an example.
I can make a pull request if you'll merge this.
Thanks

@scambra I had an issue with the use of prepend_before_filter in devise_invitable.
The thing is that I am developing a multitenant application, using postgres schemas to scope certain records for a given tenant, each tenant has its own accounts table (a great tutorial for using postgres schemas to scope records can be found in railscasts).
The thing is that I need an around filter wrapping each and every action in all of my controllers, the hack hasn't much of a big deal:
class Accounts::InvitationsController < Devise::InvitationsController
  include MultitenancyConcern

  skip_before_filter :authenticate_inviter!
  skip_before_filter :has_invitations_left?
  skip_before_filter :require_no_authentication
  skip_before_filter :resource_from_invitation_token

  before_filter :authenticate_inviter!, :only => [:new, :create]
  before_filter :has_invitations_left?, :only => [:create]
  before_filter :require_no_authentication, :only => [:edit, :update, :destroy]
  before_filter :resource_from_invitation_token, :only => [:edit, :destroy]
end

And here's my multitenancy_concern.rb to further illustrate my point:
module MultitenancyConcern
  extend ActiveSupport::Concern

  included do
    helper_method :current_tenant, :subdo
    around_filter :scope_current_tenant, if: :subdomain_present?
  end

  private
  def subdomain_present?
    request.subdomain.present?
  end

  def current_tenant
    if request.subdomain.present?
      @current_tenant ||= Tenant.find_by_subdomain(request.subdomain)
    end
  end

  def current_tenant!
    current_tenant or
      raise ActiveRecord::RecordNotFound.new("No tenant `#{request.subdomain}`")
  end

  def scope_current_tenant(&block)
    current_tenant!.scope_schema('public') { yield }
  rescue ActiveRecord::RecordNotFound => e
    render template: 'errors/not_found', layout: 'home', status: :not_found
  end
end

Could you consider using before_filter instead of prepend_before_filter?
Thanks for the gem! I appreciate the effort and knowledge putted in it.

Doesn't prepend_around_filter work for you?


Trace load and json parsing optimizations

What does this PR do?
WIP
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)

[ ] N/A

[ ] Did you write any new necessary tests?

[ ] N/A

[ ] Did you make sure to update the docs?

[ ] N/A

[ ] Did you update the changelog?

[ ] N/A

Added a comment to consolidate the two new parser options into the ParserConfig argument to keep the method signature simple and easy to extend in the future.

Codecov Report
Attention: Patch coverage is 68.62745% with 96 lines in your changes are missing coverage. Please review.

Project coverage is 89.42%. Comparing base (c980c48) to head (a90d5a9).

Files
Patch %
Lines

hta/common/trace_parser.py
55.86%
79 Missing :warning:

hta/common/trace_symbol_table.py
88.05%
8 Missing :warning:

tests/test_trace_parse.py
71.42%
4 Missing :warning:

hta/configs/parser_config.py
57.14%
3 Missing :warning:

hta/common/trace.py
90.47%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #110      +/-   ##
==========================================
- Coverage   91.30%   89.42%   -1.89%
==========================================
  Files          50       52       +2
  Lines        4727     4889     +162
==========================================
+ Hits         4316     4372      +56
- Misses        411      517     +106

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


Rename plugin

See jenkinsci/jenkins@434135e

Nice, shall we also rename the repo?

Included in #16 .  Thanks!


Switch sensu_ctl resouce to use archive_file

This removes the need for a dependency on the seven_zip cookbook since
archive_file is a builtin chef resources since Chef 15
fixes #110

Pull Request Checklist
Is this in reference to an existing issue?
Yes.
General

[ ] Update Changelog following the conventions laid out here

[ ] Update README with any necessary configuration snippets

[ ] Cookstyle (rubocop) passes

[ ] Foodcritic passes

[ ] Rspec (unit tests) passes

[ ] Inspec (integration tests) passes

New Features

[ ] Added a Testing Artifact as either an automated test or a manual artifact on the PR.

[ ] Added documentation for it to the README.md

Purpose
Modernize archive handling and remove dependency on seven_zip cookbook
Known Compatibility Issues
Unknown. Windows testing is not complete in CI, will need manual testing.

@derekgroh Can you take a look at this and confirm that behavior is the same as before? I don't easily have a Windows setup to do enough testing on.

@majormoses same for me, marked as Draft for now until we can confirm behavior better on Windows

enterprise needs to be removed from line 70 in the -Outfile
Had some networking issues that I had to work around related to the configuration action, but the install action is working successfully with the change.
We may want to introduce some guards to help speed up the run after it is setup as it runs through each resource every time.. Unfortunately, the exe doesn't publish with any version info to use so we'd have to track with another method, maybe a version.info file.

Good feedback @derekgroh thanks! I'll be working on the cookbook again over this weekend as I have time so any suggestions are appreciated

Rebased on latest master

This is now failing on windows with #116 merged in.
Resource needs to be updated to:
default_ca_cert = value_for_platform_family(
  %w(rhel fedora amazon) => '/etc/ssl/certs/ca-bundle.crt',
  'debian' => '/etc/ssl/certs/ca-certificates.crt',
  'windows' => ''
)```

There seems to be an odd bug with the :configure resource as well as sensuctl isn't available to the shell until the second converge with execute, but powershell_script is successful. Not sure if this is an environment specific issue (found on 2012R2, 2016 and 2019) as I recall it working in the past without issue.

You might be referring to the behavior I saw a while back that led to my attempt at a fix in #74
I'm not sure of the state of change now since it's a bit old. It should be reviewed though to ensure correct behavior.

@webframp It is a Windows Path issue being immediately available to the execute resource.
My testing showed this could be solved by using powershell_script resource instead.
The challenge is you have to ensure that the test runs successfully the first time, because any secondary run will succeed, but only because the path value is already set.


Crash preventing world from running on server.

Describe the bug
Server is crashing and freezing up when occultism is present.
To Reproduce
Steps to reproduce the behavior:
Unsure what is causing it directly as it happens when nobody is online or maybe at server restarting? I cannot tell.
Expected behavior
No crashes if possible.
Screenshots
If applicable, add screenshots to help explain your problem.
System (please complete the following information):

Occultism Version: 1.114.0
OS: [e.g. Windows] Server Environment, not sure what os it would be.
Minecraft Version: 1.20.1
Modpack Link and Version, or list of mods: Neoforge
SwissSocietyV5.2.txt

Additional context
Crash Report:
crash-2024-02-20_23.01.11-fml.txt

I'm asking in the neoforge discord about this, because it (it being registering compostables) worked before without issues, maybe they changed something.
You can try using an older neo version, or forge 47.1.3


Update how-to-authenticate-with-aad.md

rename token cache file

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

#sign-off

@cosh: I'm sorry - only the author of this article, @orspod, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off

@shsagir: I'm sorry - only the author of this article, @orspod, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Clean up and customise debug test

What type of PR is this?
/kind cleanup
What does does this PR do / why we need it:
$SUBJECT
Which issue(s) this PR fixes:
Fixes - Part of #3385
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:
Run make test-cmd-devfile-debug should pass.

/approve
/lgtm


Title after Abstract

Hello,
i do not know, if this is the right place to ask my question (if not, I can delete this post):
I would like to disable the printing of the title after the abstract page. Before the first section starts, the title is printed. It is not necessary for my needs.
Is there any way to disable the title printing after the abstract?
The "\maketitle" function does not change anything related to the title after the abstract.
Best wishes,
Max Berentelg

You can add the donotrepeattitle option in your documentclass and that should do the trick. Looks like that option was left out of the manual, I will fix that.


migrate more samples

Rewrote amp-bind sample to be shorter and more sample focused.
Moved a few samples to the respective component samples
Fixed a indentation bug resulting in doc not being formatted

@kul3r4 PTAL
I've started improving the parser (to avoid the case where docs are printed as code). This might still need more work. Will address in another PR.


Issues with Parameters and Avatar Names with non-alphanumeric characters

Describe the bug
Expression parameters that contains whitespaces in its name cannot be modified via OSC input.
According to the debug consoles, it seems that address and value are received normally but expression parameters are not modified.
To Reproduce

Upload an avatar that has an expression parameter with whitespaces in its name.
Send OSC message with address for problematic parameter. (I copied its address from autogenerated JSON.)

Expected behavior
Parameter will be modified similar to when its name does not contains whitespaces.
Screenshots

Info (please complete the following information):

Type: Desktop
VRChat version: Build 1166

Thanks for reporting - my first thought is that we can replace spaces with underscores when generating the config and matching names. FWIW - you should be able to do this with your existing config - just change the input.address in the config to have an underscore while leaving the name of the parameter the same.

Thanks for reporting - my first thought is that we can replace spaces with underscores when generating the config and matching names. FWIW - you should be able to do this with your existing config - just change the input.address in the config to have an underscore while leaving the name of the parameter the same.

There are other characters that don't work as well such as }. It's unlikely they're in parameter names, but it would make sense to either replace all of them, even if they're not likely to be used.
I can see the utility in the ability to intentionally ensure OSC can't be used against an avatar (e.g. people who use pincode systems on their avatars so they can share them public with friends but random people can't use it if they close it) can't be bruteforced, but it is somewhat edge case.

@NarryG - agreed on the sanitation approach. I think we'll replace every non-alphanumeric character with either nothing or an underscore.

Fixed in next release! Any non-alphanumeric characters will be replaced with underscore. You can patch this yourself in the meantime by replacing them yourself in the config.
So a Boolean parameter named "Toggle With Space" can be modified to:
{
    "name" : "Toggle With Space",
    "input" : {
        "address" : "/avatar/parameters/Toggle_With_Space",
        "type" : "Float"
    },
    "output" : {
        "address" : "/avatar/parameters/Toggle_With_Space",
        "type" : "Bool"
    }
}


Document example a few example API requests

Document example API authentication, and authenticated request after authentication for those not familiar with the way Guardian works.

Is there a way to POST login information and get a token back via the API?  It seems like an older version of Sentinel had that functionality, although I may have hand rolled it.  In any case, it seems like a reasonable thing to provide out of the box.

Looks like in v2 with the Ueberauth addition I must have removed that. I'll work on adding that back in. In the meantime you should be able to achieve that by posting to /auth/session.
Sorry about that oversight.

I look forward to a resolution to this regression.
Just in case anyone else runs into this problem, something like the following can be used to get a guardian token from the command line by posting to /auth/session.
<EMAIL_ADDRESS>PASSWORD=my_password
HOST=http://localhost:8080
COOKIE_FILE=$(mktemp /tmp/api_auth_session_html.cookie.XXXXXX)
curl -s -c "${COOKIE_FILE}" -X POST -F "session[email]=${EMAIL}" -F "session[password]=${PASSWORD}" "${HOST}/auth/session" 2>/dev/null > /dev/null
curl -s -b "${COOKIE_FILE}" -X GET "${HOST}" | grep guardian_token | sed -e 's/^.*content="//g' | sed -e 's/".*$//g'
rm -f "${COOKIE_FILE}"

Forgot about this one during my review process. I'll add it to the features/lockable_and_plug branch changes, which include a number of other changes.


Basic frontend

yarn start gives a dev server.
The UI is not yet wired up to anything.
The first step is for a user to choose their role.

I'm hitting a problem here with incompatibility between vite (which requires esnext module type) and hardhat (which requires commonjs, unless we don't use typescript).
I think possibly the only/best way around is to turn this repo into a yarn workspaces monorep (we know that can work from looking at go-nitro).

...or maybe find a replacement for vite. Probably easier.

In the end, it seems possible to just ignore the error and everything works 🤷 . Sticking with commonJS.


Generate defaults for 5eShapedSpellbook Script

Hi,
I mentioned a while back that I'd (a) like to have a more robust way of ensuring that people are using the right version of your character sheet with my evil spellbook script and (b) would like the unpleasant copy-paste-convert-to-json thing for the default values to be automated so that it is less tedious and error-prone.
What I've done here is to edit you gulpfile.js to make it generate a file called spellDefaults.js. It builds this by extracting info from components/spells/spell-page.html and writing it out as JSON. It then takes a hash of the generated json and adds it as a property on a wrapper object. Finally, it injects this hash value as a hidden attribute on the character sheet.
This means that people can install spellDefaults.js whenever they update their character sheet. My script can check that the hash from the character sheet corresponds with the hash in the defaults javascript to ensure that compatible versions are being run.
I also took the liberty of checking in a package.json (and associated .gitignore) so that executing the gulpfile is easier for others checking out the repository.
I hope you don't feel that this is all too evil to include in your repository.  In the meantime it seemed like the least-bad solution to providing a way for people who have API access to do more with spells.
Cheers,
Lucian

I'm on vacation for the weekend, but can take a closer look when I'm back.
Looks good overall but the json structure should match the one I use for my spell importer - see that github for it. If you match that it'll make it easier for both of us.
Thanks for adding package.json

Cool. WIth regards to the JSON format, I'm not sure I understand how that would work. The JSON format that I'm exporting from the sheet isn't really designed as a transmission format for spell information: it's designed to provide the script with defaults for the character sheet attributes themselves. This is especially important because many of the attributes that do end up being set explicitly (and are thus available to the API) end of containing references to other attributes (e.g. the hidden variables) which are not. The script needs to be able to lookup these references as they are named in the sheet, or this mechanism won't work. Many of these (like all the toggle vars) are not things that would belong in the JSON format that you use for importing spells anyway - they are specific to the mechanics of the character sheet itself.
To the extent that a mapping exists between your JSON format and the attributes that exist in the character sheet, it is contained in the code of your importSpell function. I guess we could document this mapping elsewhere in another JSON file, but this seems like a lot of extra work for not much benefit. It was really never my intention that this JSON format should ever be used for anything other than documenting what attributes existing in the character sheet, and exposing those defaults for them that are coded into the sheet, but which Roll20 hides from the API. I certainly wouldn't want it to end up being an alternative data store/import format for spells - that's something you've covered already with the sheet + your importer, and I have no desire to reinvent any of that!
Perhaps I'm misunderstanding what you mean, though, or I've just missed something very obvious about how this could be done in a nicer way.
Anyway, we can continue all this next week perhaps. Enjoy your vacation!

The problem is the data on the sheet is not organized well due to legacy names. Though I won't be a stickler here - as long as it's just used for it's current purpose and not for storing the values of individual spells.
I'm going to move some of the files around though.

Actually I need you to update and commit (this pull request will be updated).

Hey Mark,
I've rebased my commit onto your latest, and removed the compiled versions from the commit in the process. Should now merge OK. Obviously I haven't rebuilt now since I've removed the compiled versions from the commit. I don't think I've messed anything up in the process, but rebasing gets confusing sometimes!
As concerns the legacy names, one thing that may help put your mind at rest is that the idea of this system is precisely to avoid any dependency between my script and your naming. I only reference a small number of the values (e.g. 'spellname') directly by name to handle some of the core logic - the rest are simply there to ensure that all the internal references between the various attributes resolve properly in the absence of the defaults via the API. To the extent that I am referencing the attributes directly, I fully accept that I'm working with an undocumented private API and will need to keep track of changes myself to make sure things don't break. Hopefully repeating sections will get fixed soon enough that I never even have to worry about that!
Cheers,
Lucian

Sorry that I've been so delayed with this.

It seems to generate "undefined" though. Maybe it was me moving files around. If you could fix it that'd be great as I leave for vacation for a week soon.

Hey Mark,
I've just pulled your latest changes and I'm not quite sure what your intention was so I don't know what the best way of fixing it would be. The main problem seems to be that you've changed the src of the addSpellDataHash task to be precompiled/spellDefaults.json, but the gulpfile is already in "precompiled" so that resolves to precompiled/precompiled/spellDefaults.json, which doesn't exist.
I'm guessing that the intention is that both spellDefaults.json and spellDefaults.js should end up in the "precompiled" directory? If so, then I think changing the src for addSpellDataHash back to just "spellDefaults.json" should fix the problem. But I'm not 100% clear what you want do do, and I'm also a bit puzzled because although I don't get spellDefaults.js generated thanks to this issue, I don't get "undefined". Where is it giving you "undefined"? In the json? Do you have some changes that aren't pushed to github maybe?
Cheers,

My intention is to have it all in precompiled, ya
I can edit it tomorrow if you don't.
Sorry, my brain is a little frazzled, but I didn't want to forget it again.


baichuan-7b return value of apiserver is garbled

I try to use vllm in combination with baichuan-7b, it can run normally, but the return value of apiserver is garbled, I don't know what is possible, can you give me some advice and help

Hi @abc20220327, thanks for reporting the issue. How did you try the Baichuan model? Currently it is not supported.

Hi @abc20220327, thanks for reporting the issue. How did you try the Baichuan model? Currently it is not supported.

You can use Vicuna to test Chinese output, then you will see garbled characters!

Hi @abc20220327, thanks for reporting the issue. How did you try the Baichuan model? Currently it is not supported.

Here is the situation I encountered.

Can you try the latest main branch? Thanks!

I believe this can be fixed by #589

Closing as this should now be fixed.


[META][AutoDiff] Tech debt from tests

Description
I've assigned preliminary priorities for the issues, but this should be revisited.
High priority:

[ ] #54214
Note: FIXME(TF-977): Fix incorrect derivative for array literal with tuple_element_addr elements.
Test: test/AutoDiff/validation-test/array.swift

[ ] Note: FIXME(TF-584): Investigate incorrect (too big) gradient values for repeat-while loops.
Test: test/AutoDiff/validation-test/control_flow.swift

Medium priority:

[ ] Note: TODO(TF-1030): This will eventually not be an error.
Test: test/AutoDiff/SILOptimizer/differentiation_diagnostics.swift (see hasImplicitlyDifferentiatedClosureDefaultArgument).
The same error was fixed for curry thunks in #77615

[ ] Note: FIXME: missing reverse-mode differentiability witness for foo(_:)
Test: test/AutoDiff/SILGen/has_symbol.swift

[ ] Note: FIXME(TF-1176): Some values are incorrectly not marked as active: %16, etc.
Test: test/AutoDiff/SILOptimizer/activity_analysis.swift
This is probably class-related, but need to ensure that for non-class cases the issue is not present.

[ ] #54231
Note: TODO(TF-982): Lift this restriction and add proper support.
Test: test/AutoDiff/Sema/derivative_attr_type_checking.swift

[ ] Note: FIXME(TF-284): Fix unexpected diagnostic.
Test: AutoDiff/Sema/differentiable_attr_type_checking.swift

[ ] #54445
Note: FIXME(TF-1103): Derivative registration does not yet support @_alwaysEmitIntoClient original functions.
Test: AutoDiff/stdlib/simd.swift

[ ] #55690
Note: FIXME: Disabled due to test failure with -O
Test: test/AutoDiff/validation-test/differentiable_protocol_requirements.swift

[ ] #54628
Note: TODO(TF-433): operator ?? lowers to an active try_apply.
Test: test/AutoDiff/validation-test/optional.swift

[ ] #55084
Note: TODO: Support modify accessors
Test: test/AutoDiff/validation-test/property_wrappers.swift

Low priority:

[ ] #77762
Notes:

TODO: cannot use literal true because it crashes
TODO: cannot use literal false because it crashes

Tests:

test/AutoDiff/compiler_crashers_fixed/58660-conflicting-debug-info-inlining.swift
test/AutoDiff/compiler_crashers_fixed/issue-58123-mutating-functions-with-control-flow.swift
test/AutoDiff/validation-test/address_only_tangentvector.swift
test/AutoDiff/validation-test/control_flow.swift

[ ] #77775
Note: TODO(TF-851): Uncomment the tests below after @differentiable function SILGen thunking is fixed.
Test: test/AutoDiff/SILOptimizer/generics.swift

[ ] Note: TODO: Re-enable the boolean-literal-folding pass and fix the test accordingly
Test: test/AutoDiff/SILOptimizer/activity_analysis.swift

[ ] Note: TODO(TF-956): Improve location of active enum non-differentiability errors
Test: test/AutoDiff/SILOptimizer/differentiation_control_flow_diagnostics.swift

[ ] Note: TODO(TF-957): Improve non-differentiability errors for for-in loops
Test: test/AutoDiff/SILOptimizer/differentiation_control_flow_diagnostics.swift

[ ] Note: TODO(TF-788): Re-enable non-varied result warning.
Test: test/AutoDiff/SILOptimizer/differentiation_diagnostics.swift

[ ] Note: TODO(TF-482): Remove diagnostics when @differentiable attributes are also uniqued based on generic requirements.
Test: test/AutoDiff/Sema/differentiable_attr_type_checking.swift

[ ] Note: TODO(TF-650): It would be nice to diagnose protocol default implementation with missing @differentiable attribute.
Test: test/AutoDiff/Sema/differentiable_attr_type_checking.swift

[ ] Note: TODO(TF-632): Fix "'TangentVector' is not a member type of 'Self'" diagnostic. The underlying error should appear instead: "covariant 'Self' can only appear at the top level of method result type".
Test: test/AutoDiff/Sema/differentiable_attr_type_checking.swift

[ ] Note: FIXME: Enable test for all platforms after debugging (https://github.com/apple/swift/issues/55186).
Test: test/AutoDiff/validation-test/control_flow.swift

[ ] Note: TODO(TF-1065): Consider disallowing qualified operators / operator names
Tests:

test/AutoDiff/Parse/transpose_attr_parse.swift
test/AutoDiff/Sema/transpose_attr_type_checking.swift

Existing issues which require discussion about priorities, semantics, etc:
Class-related:

#65012 and its sub-issues.
#52130 and its sub-issues.

Forward-mode-related:

#55906
#55650

Entire tests which require discussion whether we want to address tech debt for them:
Class-related:

test/AutoDiff/SILGen/vtable.swift
test/AutoDiff/Sema/DerivedConformances/class_differentiable.swift
test/AutoDiff/validation-test/class_differentiation.swift

Forward-mode-related:

test/AutoDiff/SILOptimizer/forward_mode_diagnostics.swift
test/AutoDiff/validation-test/forward_mode_simd.swift
test/AutoDiff/validation-test/forward_mode_simple.swift

Marked as TODO/FIXME/etc but require discussion whether we want to address them:
Class-related:

Note: FIXME(rdar://74380324)
Test: test/AutoDiff/TBD/derivative_symbols.swift
FIXMEs are related to initializer and other special functions differentiation support
Note: FIXME(TF-1080): Fix incorrect class property modify accessor derivative values.
Test: test/AutoDiff/validation-test/inout_parameters.swift
Note: TODO(TF-649): Enable @derivative to override derivatives for original declaration defined in superclass.
Test: test/AutoDiff/Sema/derivative_attr_type_checking.swift
Note: FIXME: Enable derivative registration for class property/subscript setters (https://github.com/apple/swift/issues/55542).
Test: test/AutoDiff/Sema/derivative_attr_type_checking.swift
This looks working for structs.
Note: FIXME(rdar://74380324)
Test: test/AutoDiff/TBD/derivative_symbols.swift
The FIXME is related to the following crash:1.      Swift version 6.1-dev (LLVM 0f86f354a7bc883, Swift e6b4e0f9f171a51)
2.      Compiling with effective version 4.1.50
3.      While evaluating request ASTLoweringRequest(Lowering AST to SIL for module test)
6.      While verifying SIL function<EMAIL_ADDRESS>

Note: FIXME(TF-1175): Class operands should always be marked active.
Test: test/AutoDiff/validation-test/property_wrappers.swift

Forward-mode-related:

Note: TODO(TF-1254): Support forward-mode differentiation and test generated differentials.
Test: test/AutoDiff/SILOptimizer/semantic_member_accessors_sil.swift
Note: TODO(TF-1173): Move floating-point mutating operation tests to test/AutoDiff/stdlib/floating_point.swift.gyb when forward-mode differentiation supports inout parameter differentiation.
Test: test/AutoDiff/validation-test/inout_parameters.swift
Note: TODO(TF-1254): Support and test forward-mode differentiation.
Test: test/AutoDiff/validation-test/property_wrappers.swift
Note: FIXME(TF-1033): Fix forward-mode ownership error for tuple with non-active
Test: test/AutoDiff/validation-test/simple_math.swift

Linear functions related:

Note: TODO(TF-900, TF-902): Uncomment the following line to test loading a linear function from memory and direct calls to a linear function.
Test: test/AutoDiff/SILGen/differentiable_function.swift
Note: TODO: Add tests for @differentiable(_linear) functions.
Test: test/AutoDiff/IRGen/loadable_by_address.swift

Additional information
No response

Tagging @asl

Oh also, the TF related links don't seem to work and redirect to the following page: https://github.com/apple/swift-issues/issues
Is this a recent change that these issues are no longer hosted there? Not sure I haven't looked at the page for a while.

Oh also, the TF related links don't seem to work and redirect to the following page: https://github.com/apple/swift-issues/
Is this a recent change that these issues are no longer hosted there? Not sure I haven't looked at the page for a while.

It looks like they are not hosted there for at least several months. As far as I understood, they are just gone and were not transferred properly (some of them were, but not all of them), unfortunately.
It looks like that swift repository still has some rules so TF-XXX text in issues is automatically converted to corresponding dangling link - it might be a bit confusing.
@asl Please correct me if I'm mistaken

TF issues from TensorFlow Jira were never converted to github issues. So, they are gone forever.

@JaapWijnen Thanks for assigning priorities! One minor comment regarding the following: it looks like that the observable error is the same but root cause is different (not related to curry thunks). So it looks like it would need a separate patch

Note: TODO(TF-1030): This will eventually not be an error. (which you've already provided a fix for)


Clean up some misuses of INVALID_HANDLE_VALUE (fixes #427)

Almost all functions in the Windows API that open or create objects and return HANDLEs to them return null on failure; only a few (mostly to do with the file system) return INVALID_HANDLE_VALUE on failure. This PR scrubs the repo of a few, but not necessarily all, cases where INVALID_HANDLE_VALUE was mistakenly used or tested against instead of null. In particular, it fixes 2 cases (one reported in issue #427) where the return value of CreateThread() was compared against INVALID_HANDLE_VALUE against null, which caused the error handling code to be executed at the wrong times.
There are a lot of other uses of INVALID_HANDLE_VALUE I found that looked questionable, but which I left alone. Most of these were used to initialize HANDLE-typed variables and as a sentinel to see if those variables remained unset to a "real" value.
PR Checklist

[x] Fixes #427
[x] CLA signed - N/A; I work for Microsoft
[x] Tests added/passed - no tests added, existing unit tests pass
[x] Requires documentation to be updated - N/A; no doc changes needed
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan.

References
This PR was inspired by my previous PR #1015 which also resolved some misuses of INVALID_HANDLE_VALUE.

I like this! I might file a follow-on workitem to move to WIL's unique_handle in the near term for a bunch of APIs like this. We can also specialize our own WIL unique_any derivatives for other resource-managed things.

I'm not actually sure if Microsoft's compiler will warn for unused
variables declared at non-static file scope - I couldn't find such a
warning after a quick search, and I imagine it might not even be a good
idea to have this be a warning, since another file may have the same
variable declared extern.
One of the MSBuild files intended to be included by .vcxproj files (
https://github.com/microsoft/terminal/blob/v0.1.1502.0/src/common.build.pre.props#L59-L66
) changes all enabled warnings to errors, but at the same time I have seen
at least one compiler warning when running a clean build, so this script is
probably not being included everywhere or is being overridden.
If you do create an issue to clean all existing compiler warnings out of
the repo and enable warnings as errors everywhere, I'd definitely +1 it.
On Mon, Jun 3, 2019 at 7:26 PM dlong11<EMAIL_ADDRESS>wrote:

@dlong11 commented on this pull request.
In src/host/ft_host/CJK_DbcsTests.cpp
https://github.com/microsoft/terminal/pull/1105#discussion_r290105264:

@@ -278,8 +278,6 @@ class DbcsTests
END_TEST_METHOD()
};

-HANDLE hScreen = INVALID_HANDLE_VALUE;
Shouldn't we get a warning for this? Also - what about warnings as errors?
Should this be turned on? I am not familiar with the test projects or the
settings. (or what the current practice is for these types of things) I was
assuming all of the projects would have warnings as errors on but looking
at the CI it doesn't appear so. Is this a goal? Should an issue be logged
to make sure this is on for the projects?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/pull/1105?email_source=notifications&email_token=AAH535WNMFWYR3Z3LLHJJ6LPYXHFVA5CNFSM4HSBUKN2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB2OY7DI#discussion_r290105264,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAH535TVT5EQB55GUUYTO33PYXHFVANCNFSM4HSBUKNQ
.


Application doesn't start

Despite multiple attempts, accessing http://localhost:8080/ results only in message "Hello World from Flask in a uWSGI Nginx Docker container with Python 3.6 (default)" on the web page.

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

ID: 65f634bf-2280-a093-5897-724310dafa2d
Version Independent ID: f11c0af7-bfaf-6d64-2c75-9015b0eb1f30
Content: Kubernetes on Azure tutorial  - Prepare an application
Content Source: articles/aks/tutorial-kubernetes-prepare-app.md
Service: container-service
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@mcurko I ran through the doc but it seems to be working as expected

Likely there is something with your local docker setup causing the problem.
I would suggest you make sure you are running Linux containers and that docker is fully up to date.
You also will want to run docker-compose down to stop all containers.
From there make sure you are in the correct file path before starting the cluster again so it starts the correct app.

Switching to Linux containers indeed resolves the issue. I think it should be mentioned in the article, because it's not obvious for a Docker beginner that this has to be done.

@mcurko by default Docker should pick Linux Containers. But if you switched to Windows Containers it will not automatically switch back.
@iainfoulds can we mention that users should be using Linux Containers for this example?

Sorry you ran into issues, @mcurko.
I've added a call out in the 'Before you begin' section that notes the local Docker environment must be running Linux containers. This update should merge and publish today.
#in-progress


Add EventType for Mandates

Adding EventTypes for mandates events (see https://docs.mangopay.com/endpoints/v2.01/events#e251_the-event-object )

Thanks very much @nboire - I'm not sure how these got missed out, so I'm sorry about that :-/


Connection from input to output

Hi @jerosoler ,
was trying to customize the plugin for getting the connections from input to outputs.
Most things working as expected,

draw connections from input,
continue drawConnection when dragged,
3. but in the end "mouseUp" or "dragEnd(e)" is not identifying the output class and node-id, when drag is ended.
Tried setting it from the "mouseMove" and even from the mouselastPosition.
Failing to get output-id when drag-ended.

any light on this issue?

Hello
In the click function, you should have an "input" option on the switch.
I would create a new variable connectionStart = "input or output".
I would duplicate the drawConnection to drawConnectionInput function.
I dragEnd will do the reverse function of creating connection.
You can get the output id and the output_class from the event itself. It would have to be in the variable "ele_last".

Perfect!.
I implemented the same approach for achieving it,

created switch for "input" click and starting "drawConnection"
drawConnectionInput, which draws the svg(main-path) till it encounters "dragEnd".
But only in DragEnd, I am always getting "ele_last" as "svg(main-path)" .
issue->I am not gettin "ele-last" as "output" element for completing the connection.

Try to modify the z-index of the outputs. Perhaps the connection that is being created is above. I is the last element it detects.
Even with the connections, they should have an event pointer none AND a z-index 0.

Hi @jerosoler,
Thanks a lot for all your suggestions, "pointer-events":none. Worked! :)
So finally, we can connect from Inputs to outputs points.


Refactored TimeCacheMap to extend RotatingMap.

This should allow current users of the TimeCacheMap to progressively migrate off the deprecated TimeCacheMap to the simpler RotatingMap.
This also improves code reuse since many of the methods were shared.

For large changes like this and #683, Please file a jira at https://issues.apache.org/jira/browse/STORM/

Issue raised: https://issues.apache.org/jira/browse/STORM-990

@ddebree Thanks. I should have mentioned it, but we also include the issue name in the title of our pull requests by convention.
[STORM-990] Refactored TimeCacheMap to extend RotatingMap.

I'm in favor of using RotatingMap to implement TimeCacheMap, but I'd rather have a RotatingMap as a member of TimeCacheMap than as a superclass.

I have updated the class to rather use the RotatingMap as a member variable. It comes out a lot more similar to the original.

+1 Looks good to me.


Add section to components article on using a code behind file

We should document how to setup a code behind file using @inherits. Later, we can update this section when we implement support for partials (https://github.com/aspnet/Blazor/issues/278)

@danroth27 We had it for a bit but cut it temporarily. Here's what we cut ... would this suffice?
Note: I also cut BlazorRocks.cshtml and BlazorRocksBase.cs out of the sample app, but I have the files here, so I can shoot them back into the sample with the section.
Base class inheritance for a "code-behind" experience
Blazor's Razor files (*.cshtml) mix HTML markup and C# processing code in the same file. The @inherits directive can be used to provide Blazor with a "code-behind" experience that separates view markup from processing code.
The Components Sample app shows how a component can inherit a base class, BlazorRocksBase, to provide its properties and methods.
BlazorRocks.cshtml:
[!code-cshtml]
... it shows this ...
using Microsoft.AspNetCore.Blazor.Components;

public class BlazorRocksBase : BlazorComponent
{
    public string BlazorRocksText { get; set; } =
        "Blazor rocks the browser!";
}

BlazorRocksBase.cs:
[!code-cshtml]
... it shows this ...
@page "/BlazorRocks"
@inherits BlazorRocksBase

<h1>@BlazorRocksText</h1>

The base class must derive from BlazorComponent in the Microsoft.AspNetCore.Blazor.Components namespace.


Update stomp and sensu-plugin gems to remove Object::timeout deprecation

Fixes the Object::timeout deprecation issue.

I don't believe the community is ready to bump up the version of sensu-plugins to 1.4 as other repos have not done so.  Also we prefer ~> 1.4 vs 1.4.4
@cwjohnston

I reverted the sensu-plugin update as I don't believe that it is required to fix the issue.


fixed broken link

Because the path of this file was changed, the link led nowhere.

Coverage: 28.057%. Remained the same when pulling 12bc932e7c5f4965442a47fdc4a9f7a8c1e5f0e3 on arneBersch:master into 15980ea0acd38a49c1c5f9d18351e0e71fa4ee25 on mcallegari:master.

Thanks :+1:


Add Cargo.lock to .gitignore, and rm it

Library crates, as opposed to binary crates, should not track their Cargo.lock.  See https://doc.rust-lang.org/cargo/guide/cargo-toml-vs-cargo-lock.html
this is a git infrastructure change, so let's say 🔦
Checklist
(no item applies)
Context
relates to no GitHub issue
Semver Changes
no semver changes

Thanks!


Create issue template

I just saw that GitHub added this. Feel free to make any modifications or suggestions.

LGTM and I Love the Notion of people being Waynes about this being strictly for metadata :)


Fix a DataDog agent issue with ECS_FARGATE hack

Running the DataDog agent as a separate service (currently recommended by Render) comes with a bunch of caveats, including that it sends a whole raft of system metrics that are pointless, since nothing except the agent is actually run on the system it's gathering metrics for. Making the agent think it's on ECS Fargate using the ECS_FARGATE environment variable causes it not to send those. This may also remove a host from our DataDog account (saving some money), but I haven't seen for sure whether that's true yet.

Switching to the API instead of the agent entirely (see #943) would obviate this, but in the mean time, it’s probably useful. We’re actually already running with this setting right now, so this doesn’t change anything except putting it in the source and explaining it.
See also this discussion on Slack: https://usdigitalresponse.slack.com/archives/C01S4JWSMT7/p1661815283123649


[jdk8 solaris] sun/jvmstat/monitor/MonitoredVm/MonitorVmStartTerminate.java  There is insufficient memory for the Java Runtime Environment to continue

[jdk8 solaris] sun/jvmstat/monitor/MonitoredVm/MonitorVmStartTerminate.java consistently failed with  There is insufficient memory for the Java Runtime Environment to continue
16:10:55  Java Process f9d4d85c-a746-450d-b573-4cb768f9306f_4 stdout:#
16:10:55  # There is insufficient memory for the Java Runtime Environment to continue.
16:10:55  # Native memory allocation (mmap) failed to map 180355072 bytes for committing reserved memory.
16:10:55  # An error report file with more information is saved as:
16:10:55  # /export/home/jenkins/workspace/Test_openjdk8_hs_extended.openjdk_x86-64_solaris_testList_0/aqa-tests/TKG/output_1652548416534/jdk_tools_0/work/scratch/hs_err_pid13361.log

Same runtime/NMT/HugeArenaTracking.java

Do we know what the requirement is for this test? We have limited control over the size of these systems and there is a direct cost associated with them.


Also work in jsx like environments?

Hi! I was wondering if I could also make this work in JSX environments? :)

Yes, I updated it in v0.1.5
You can update from vscode store
Thx for feedback~

Superb! :) 👍


dev 1.3.0 How to use the conditional branch node? Can you explain?

How to use the conditional branch node? Can you explain?How to use conditional judgment node? Could you please introduce it?At present, I also found a bug, that is, after the conditional judgment node is successfully created in the front page, it is deleted, and then click save workflow to report an error.

It's better to put the part of "custom parameters" into the document or something else


TBI - @sap-ux/project-access - add missing constants

Description (include screenshots)
Will add missing constants under DirName such as
    Schemas = '.schemas',
    Pages = 'pages',
    Webapp = 'webapp',

Internal issue 28284.
Value
Allow the use of constants in development, instead of hardcoding the strings.
Architecture Elaboration
Does it requires architecture elaboration?

Yes

Notes

Discovered as a part of consuming @sap-ux/project-access into Fiori Tools

Tasks

Create constants

A suggestion is to look into each module consuming project-access and ensure if there are duplicate constants that these are refactered out to project-access instead;
export enum DirName {
    Sapux = 'src',
    Schemas = '.schemas',
    Pages = 'pages',
    Webapp = 'webapp',
    Temp = '.tmp',
    Changes = 'changes',
    LocalService = 'localService',
    Controller = 'controller',
    View = 'view',
    Fragment = 'fragment',
    Ext = 'ext',
    VSCode = '.vscode',
    AppConfig = 'appconfig',
    Db = 'db',
    Csv = 'csv',
    Data = 'data',
    Mockdata = 'mockdata',
    Dist = 'dist'
}

Consumed into tool-suite, issue can now be closed.


configure.sh: line 190: ts-node-esm: command not found | DEBIAN 11.7

Hello,
When I run the "bash install.sh" command, the script freezes and gives me the following error:
root@S-ONEUPTIME:~/oneuptime# bash install.sh
hint: Pulling without specifying how to reconcile divergent branches is
hint: discouraged. You can squelch this message by running one of the following
hint: commands sometime before your next pull:
hint:
hint:   git config pull.rebase false  # merge (the default strategy)
hint:   git config pull.rebase true   # rebase
hint:   git config pull.ff only       # fast-forward only
hint:
hint: You can replace "git config" with "git config --global" to set a default
hint: preference for all repositories. You can also pass --rebase, --no-rebase,
hint: or --ff-only on the command line to override the configured default per
hint: invocation.
Already up to date.
Welcome to the OneUptime 🟢 Runner
Please enter your sudo password now:
Thanks! 🙏
Ok! We'll take it from here 🚀
Making sure any stack that might exist is stopped
Grabbing latest apt caches
Hit:1 http://security.debian.org/debian-security bullseye-security InRelease
Hit:2 http://deb.debian.org/debian bullseye InRelease
Hit:3 http://deb.debian.org/debian bullseye-updates InRelease
Hit:4 http://deb.debian.org/debian bullseye-backports InRelease
Hit:5 https://download.docker.com/linux/debian bullseye InRelease
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
All packages are up to date.
Installing OneUptime 🟢
hint: Pulling without specifying how to reconcile divergent branches is
hint: discouraged. You can squelch this message by running one of the following
hint: commands sometime before your next pull:
hint:
hint:   git config pull.rebase false  # merge (the default strategy)
hint:   git config pull.rebase true   # rebase
hint:   git config pull.ff only       # fast-forward only
hint:
hint: You can replace "git config" with "git config --global" to set a default
hint: preference for all repositories. You can also pass --rebase, --no-rebase,
hint: or --ff-only on the command line to override the configured default per
hint: invocation.
Already up to date.
changed 19 packages in 6s
SSL Certificate exists. Skipping generating a new one.
configure.sh: line 190: ts-node-esm: command not found
root@S-ONEUPTIME:~/oneuptime#

Can you please pull latest release and rerun it should now be fixed.

Hello @nzdhandala i have pulled the latest and i am still running into this error . OS: ubuntu 20.4

Hello @nzdhandala i have pulled the latest and i am still running into this error . OS: ubuntu 20.4

Same here.
# lsb_release -a
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.2 LTS
Release:        22.04
Codename:       jammy

Let me see if there's anything I can do to fix it.

Let me see if there's anything I can do to fix it.

configure.sh: line 193: ts-node-esm: command not found
for  Ubuntu 22.04

I had the same error but it seems like after a reboot it worked for me without a problem

I had the same error but it seems like after a reboot it worked for me without a problem

This isn't really helpful, what OS?

I had the same error but it seems like after a reboot it worked for me without a problem

This isn't really helpful, what OS?

Sorry, i have updated the comment with more info's

Hello, Did you push?

A quick reboot fixed it for me as well. Unsure of the actual issue but at least there is a quick work around for now.


Directory .vscode in Repo

Hi @ns-rse, we have the .vscode folder at the root of the repo.
Shouldn't we delete this folder and add it to .gitignore?

Good spot @groadabike
I'm in two minds, technically its not strictly necessary. On the flip side any contributors may find the configuration it provides for VSCode useful. In this instance its saying that the Python plugin should use the black formatter which is kind of a good thing since we apply black using pre-commit anyway. This just means that the formatting will be applied on file save rather than by failing pre-commit, having the formatting applied and then the contributor having to then repeat their git commit.
The other argument is that whilst its convenient its down to the contributor to configure their IDE as they desire. Personally I use Emacs as my IDE so wouldn't use this.
Input/opinions welcome.

Good spot @groadabike
I'm in two minds, technically its not strictly necessary. On the flip side any contributors may find the configuration it provides for VSCode useful. In this instance its saying that the Python plugin should use the black formatter which is kind of a good thing since we apply black using pre-commit anyway. This just means that the formatting will be applied on file save rather than by failing pre-commit, having the formatting applied and then the contributor having to then repeat their git commit.
The counter argument is that whilst its convenient its down to the contributor to configure their IDE as they desire.
Input/opinions welcome, personally I use Emacs with lsp-mode as my IDE so wouldn't use this.

That's a good point @ns-rse, I didn't know you can export the config with that file, I use IntelliJ myself.
But, I don't think we need to export .vscode configs, is not that we are encouraging people to use vscode.
We could just add a badge indicating we use black like in Asteroid project https://github.com/asteroid-team/asteroid

Badges are good, others would be..

codecov
codestyle : flake8
pypi package
No doubt more.

By the way @groadabike IntelliJ is a Java orientated IDE, you may want to give JetBrains Python IDE a whirl which is PyCharm (or Emacs :wink: )

This was a deliberate addition at some point: I think vscode layers 'workspace' settings over 'user' settings. The .vscode in the directory is a workspace setting which has been kept to a minimum of turning on the black formatter. It's a bit redundant as the pre-commit hooks enforce black anyway but I thought it might be helpful for people who did not know how to set this up. I'd suggest to keep it unless it is causing problems for people (The other useful thing it could contain is additions to the spellcheck dictionary)

I'm happy to leave this in if it simplifies the workflow for those using VSCode.
Shall we close this issue?

Yes, please - I don't see it causing any real issues. Let's close it for
now. Can always revisit.
On Wed, 23 Nov 2022 at 08:43, Neil Shephard @.***>
wrote:

I'm happy to leave this in if it simplifies the workflow for those using
VSCode.
Shall we close this issue?
—
Reply to this email directly, view it on GitHub
https://github.com/claritychallenge/clarity/issues/112#issuecomment-1324713376,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACO7SJDMEHXY7RJOWXC7UYTWJXKJTANCNFSM6AAAAAASA4RREA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Professor Jon Barker,
Department of Computer Science,
University of Sheffield
+44 (0)<PHONE_NUMBER>

Closing this issue for now.


show architecture for Alpine Linux

Checklist

[x] npm test passes
[ ] documentation is changed or added (if applicable)
[ ] permission has been obtained to add new logo (if applicable)
[x] contribution guidelines followed here

linter issue can be ignored


Javascript example for browser

Now there is only javascript example, that works inside nodejs environment(https://github.com/imgproxy/imgproxy/blob/master/examples/signature.js) due to module 'crypto'.
It would be good to have reference example for usage in browser

Hi,
I'd not recommend signing URLs on the client-side since you'll leak your keys to the world. But https://github.com/imgproxy/imgproxy/pull/200 makes the example work with browsers anyway.


Add Typescript typings

As the title says, it'd be great to have typings for this library. Seeing Typescript on the rise and in use in many projects (including one I'm working on), it'd make developer experience better.

Thanks for the idea! Can you share some pain points where adding TypeScript would have helped you with this library?

It's not that painful since the functionality offered by the library isn't that complex and there's only a handful of functions but it's still nice to have. With typings:

when working with TS, a developer wouldn't need to provide module declaration, i.e. declare module '@miragejs/graphql'
automatic imports by IDE would be supported
developers would get better IntelliSense experience
TS would check for correct function call arguments

Like I've said, these aren't very outstanding issues, but it's pretty standard nowadays to offer TS support

Checking in now that #23 was merged to see if this issue can be closed or if there's more work to do. Thanks!!

#23 is certainly a good start but I'd say there's still more work to be done to close this issue. #23 provides basic types that tsc can infer on its own, but for many cases that may not be good enough. For example, let's look at the generated typings for createGraphQLHandler:
export function createGraphQLHandler(graphQLSchema: any | string, mirageSchema: any, { context, resolvers, root }?: {
    context: any;
    resolvers: any;
    root: any;
}): graphQLHandler;

While its certainly much better than nothing, the typings could be improved. In general, any any (:wink:) instance is a place for improvement because any pretty much disables any type checking. graphQLSchema excepts either string or an AST schema - types for this exists inside graphql package. Same with resolvers - those could be typed as well.
There's also one issue I see introduced with this PR, when I try to provide custom resolvers to the handler, I now also need to pass context and root because those aren't marked as optional, so this example from the README won't work in TS-enabled environment:
const graphQLHandler = createGraphQLHandler(graphQLSchema, this.schema, {
  resolvers: {
    Mutation: {
      // Coerce the record returned from the default resolver to a Boolean
      deletePerson: (obj, args, context, info) =>
        !!mirageGraphQLFieldResolver(...arguments)
    }
  }
})

Thanks for the feedback!

It has taken years to find the time to work on a TypeScript version of this library but I finally got around to it. I have released version 0.2.0-alpha.1 to npm. This version is based on the typescript branch. If anyone wants to give it a try, it would be great to see if there are any issues.
There should be not breaking changes when it comes to creating a request handler (the main functionality) but there are some breaking changes to a few functions that I would be surprised to see anyone use.
Thanks in advance for any feedback!


master-details pattern

Does rivets have support for the master-details pattern?
If not, how would you implement this using a new binder?

Master details can be done without new binder. Just use the args formatter described in #672
Here is a full example https://jsfiddle.net/jccazeaux/sn584fn5/

@jccazeaux That is dope. Never thought of an args formatter before.
@erdalkaraca
Here is my 5 minute version of the master details pattern done in rivets: https://jsfiddle.net/Vornagreg/Lmky5hg1/1/
Off the top of my head I can think of 3 diff ways to do this with rivets.  You can fit the library to match the pattern of your domain.
Let us know if this makes sense or you need additional help! Otherwise, please close this sucker!

That is exactly what I was looking for!
Thanks to you both for your precious time!


What counts as a reproducer(your not clear you just saying it does not count or you are lying)

Issue type
Wrong or misleading information
Problem description
Title says all
Context (optional)
No response
Page with the problem
N/a

I give you a reproduction capability template you did not eccept it

Hi @Danjhop4,
We've repeatedly asked you to abide by the code of conduct, but you still make derogatory comments, imply we're lying or intentionally trying to not help you. You seem to understand this behavior isn't acceptable because you expect to be banned for it.
I'm sorry that you are having trouble using Gradle. I don't know what's the cause of your problem, but we're not going to be able to help you.
Since you repeatedly open new issues that we have to triage and close, I'm blocking you.


Fix Networking.ipAddress webhook's bug

Closes: https://github.com/metal3-io/ironic-standalone-operator/issues/60

@dtantsur  Please

/ok-to-test

looks good, just wondering if would be good to have a test for this.

looks good, just wondering if would be good to have a test for this.

I will try it.

looks good, just wondering if would be good to have a test for this.

@iurygregory Hi, I would like to raise a new PR for your request.

/approve
/lgtm
We need to have unit tests for the validations but we currently don't. Help welcome.


Refactored GUI mockup

For this mockup I wanted to use space more efficiently and use more standard OS elements. Is something like this worth investing more of my time into? The gist of what I'm aiming for is here, but it needs more work.
All feedback is welcome.

Personally I don't like the layout.
What Is "Use previous address"?

Can you be more specific about what you don't like?

What Is "Use previous address"?

It's this button on the Send tab.

I prefer each view separated. I get the point of this layout, but I don’t think it’s the correct approach here. Having this as an option would add extra reviewing effort which I don’t think is also desirable.

I prefer each view separated.

Agree, this page looks too busy. I would keep send/recieve on their own page and re-structure the overview page to be cleaner

@dillonjohnbrown have you joined the bitcoin design community? https://bitcoin.design/guide/
You can present any designs to the #bitcoin-core-gui slack channel, a couple of designers on there who will review with you.

I'd suggest to move this and any similar discussion to https://github.com/bitcoin-core/gui-qml.


TemporaryJobs: move old job cleanup to apply()

Instead of attempting to clean up new jobs in the TemporaryJobs
constructor, instead perform the cleanup (just one time) when the
Statement is evaluated.
In other words, we attempt to cleanup old jobs when the test method is
being executed (if the user is using @Rule).
This avoids conflicts when used with HeliosSoloDeployment in a RuleChain
(as the docs suggest) like:
@ClassRule
public static final RuleChain chain = RuleChain
    .outerRule(soloResource)
    .around(temporaryJobs);

If the old job cleanup is performed in the constructor, then
TemporaryJobs will start attempting to query the Helios cluster before
it has been fully initialized by the HeliosDeploymentResource named
soloResource in the above example.

@gimaker @davidxia @rohansingh

Current coverage is 51.50% (diff: 100%)

Merging #1063 into master will increase coverage by 0.09%

@@             master      #1063   diff @@
==========================================
  Files           275        275
  Lines         13084      13089     +5
  Methods           0          0
  Messages          0          0
  Branches       1694       1695     +1
==========================================
+ Hits           6726       6741    +15
+ Misses         5855       5845    -10
  Partials        503        503

Powered by Codecov. Last update b933f7e...a1be940

Nice catch!
removeOldJobs() is called both in the ctor and in apply() - is that intentional? The PR description suggests that it should no longer be called in the ctor

@gimaker yep that was a mistake - fixed in 6606572


Installing trigger cli fails with repository not found

flogo install github.com/project-flogo/trigger/cli
Error in installing 'github.com/project-flogo/trigger/cli'
Error installing contribution/dependency: go get github.com/project-flogo/trigger/cli@latest: module github.com/project-flogo/trigger/cli: git ls-remote -q origin in /Users/mmussett/src/go/pkg/mod/cache/vcs/d2184ee5c3f75ba69f657d9e271de453da32be2d5f29a09c6632ea84faa465e4: exit status 128:
remote: Repository not found.
fatal: repository 'https://github.com/project-flogo/trigger/' not found

Documentation incorrect. Should be:
flogo install github.com/project-flogo/contrib/trigger/cli
Not:
flogo install github.com/project-flogo/trigger/cli


SAS snapshot support & test added

Well. The last try didn't come out super great, did it?
Let's go in for take 2.

The change seems to have broken the CI

@rickle-msft could you please take a look?

@rickle-msft Could you take another look? It's 90% just the test that matters. No snapshot options are provided on the delete, and an attempted use as a normal SAS token fails, as intended.


RHEL SDK

This change is required, because we don't have dependency provider in place.
Normally we should be able to install packages and resolve them if needed, sadly it never will be prioritized.


Fix bugs of ZoomBlure about resolution & size

Bug:
The raduis rendered is not as same as user set.
The wrong resolution.
Update:
Don't allow set custom size anymore. it's useless & would take some bugs.

BAD PR. I'll create a new one.


Need to reset expected value

Beware: when atomic_compare_exchange_weak() fails it will set expected equal to the current value, i.e. LOCKED. The next time around when it is called expected will equal LOCKED when it should be UNLOCKED. One need to reset expected (i.e. expected = UNLOCKED;) on line 19. https://github.com/tchajed/futex-tutorial/blob/5001c874c253dcc9a4945eacb4285b43ea85dddd/mutex.c#L19
I know this is a trivial github project but it's like the first thing that shows up on google for implementing a mutex on linux.

Thanks, I really appreciate that you pointed this out! While this repo is indeed not a useful piece of software I wouldn't feel right having a tutorial steer people wrong.
The example program called futex_demo in the futex(2) man page seems to have an incorrect explanation of atomic_compare_exchange_strong which may be where I got the wrong semantics from. It would be great to submit a patch to that.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 494 ms

Resolved: ClamAV performance has improved in f918add.


Support for inline tests

When working on fall, one feature that turned out to be really useful were inline test. Basically, the idea is that you place an example Rust code alongside the grammar rules, like this:
pub rule where_clause {
  'where' <commit>
    {type_reference type_bounds {',' | <eof> | <not <not '{'>>}}*
}

test r"
fn f()
  where T: Clone + Copy, Foo: Bar
{}
"

this allows one to easily map the structure of the rules to the code which is supposed to be parse by them.
I think we should add the same feature to libsytanx. What's more, we will be using libsyntax itself to implement this feature, and that's why this issue has the fun label! So, here's the plan.
The tests will be placed in comments in the source code:
// test:
// fn foo() where T: Clone {}
// test:
// fn foo() where T: Clone, {}
fn where_clause(p: &mut Parser) {
 ...
}

Each test:

begins with test: prefix and runs until the end of the comment or until the next test:
is associated with a function: it either immediately precedes the function, or is inside it.
has a name which is formed by concatenating the name of the function with a source order of the test, like where_clause_01, where_clause_02.

To actually run these tests, we reuse the usual tests/data/parser/ok machinery. Specifically, we write a tool, which parses libsyntax source code, extracts all the tests and writes each one of them as a separate file to tests/data/parser/ok directory. The tool must deal smartly with adding, removing and changing the tests. Specifically, for added tests it should add a file to the ok directory with the next test number, for removed test it should probably give an error (simply removing the test file will break tests numbering), and for changed test it should edit the existing file.
Now, the fun fact is that to write such tool, you'll need the ability to parse Rust source code, and that is exactly what libsyntax is for! Currently, it can parse only a small subset of the Rust source code, but it should be enough for the task at hand if we fill a small number of missing things and make a robust error recovery.

Hm, actually, I don't think we need to really parse the source code, we need only to extract line comments, and that's easier to implement by hand. Will implement this simple solution for now, but it would be cool to one-day group the tests by function name!


Find a replacement for Bootstrap's well

To be able to completely implement the samples section, there should be some equivalent of Bootstrap's well.

Or, maybe, put these links into the side-bar.

I will try pushpin

Any opinions on this?

Unfortunately it looks funny on phones.

So materialize doesn't have a control like Bootstrap's well?
If that's the case, can we create an Aurelia custom element which renders a menu? So with repeat.for on a list item and styles via css (and js for waves)

Materialize doesn't but their website has something like it on its right edge. The above is a copy of that (with borders) and it works like you said:

So with repeat.for on a list item and styles via css (and js for waves)

You're right, we should make a custom element of it.

Does that work? If so, very nice :)

Why should it not work? :smile:
It is just html/css with a bound router.

Then look no further :)

Alright then.. :smile:


Provide missing @Override annotation

Thank you for your contribution, I don't think that this necessitates a JIRA issue as it is such a small change. I've merged this to master do you mind closing this PR? Thanks 👍


iOS Speech Recognition permission issue

Hi there!
I am using your package and it is not working properly on iOS. I have the latest Chrome version v126.0.6478.153 and iOS version 17.5.1
I have allowed permission for speech recognition in the Chrome iOS setting and microphone. Yet, I am still getting errors, that
"Speech Recognition error detected: aborted".
Please provide a solution for this.

any solutions?

Hi,
It seems that speechRecognition is not supported by chrome iOS, and it needs to develop pollyfills


WIP:  (--debug=json) JSON debug time output

Allow writing some of the --debug option outputs to a JSON file
--debug=json,time for example would write the build info and --debug=time info into a json file.
TODO:

[ ] Not sure if we need to provide an API to change the file name/location?

Contributor Checklist:

[x] I have created a new test or updated the unit tests to cover the new/changed functionality.
[x] I have updated CHANGES.txt (and read the README.rst)
[x] I have updated the appropriate documentation

Attaching example output file for --debug=count,memory,time,action-timestamps,json (basically enabling all parts of the file)
scons_stats.json.txt


Same-named mobs cause errors.

Same-named mobs crash the server after being killed.
Probably because their UIDs aren't being generated uniquely.

https://github.com/AbooMinister25/Curse-of-the-Mites/commit/0cbf71efa02c0d6f4fbc79d6e0411f8017c561f9 this fixes that.


Limit connection buffering

I've been experimenting with fowl for an idea I have in mind, and I noticed an issue: It seems like a listening port will happily accept tons of data, to the point that it can lock up the local machine, while only being able to slowly send that data out.
To reproduce:
On one machine:

iperf3 -s localhost -p 54321
fowl
On another machine:
fowl -L 12345:54321
iperf3 -c localhost -p 12345 -n 1G

Then notice that the Gigabyte of data is written almost instantly, while sending will take quite a while. On my first try I left out the -n 1G part and managed to completely freeze my laptop with it.
I assume there is some kind of buffer on the connection that is a bit too large.

Yes, this would be a great feature.
Internally, I simply haven't yet hooked up any of the Twisted backpressure stuff.
That's "Producer / Consumer" if anyone reading this is thinking about a PR (that is, the relevant Twisted documentation to look at is IProducer / IConsumer interfaces and friends). This can be tuned, so ideally a command-line option could control how much buffering (note that this is "Python process-space buffering" and not anything the OS etc does already).

So, I have no clue about twisted, but what I am getting from the docs is that this would just be a case of adding a ProducerConsumerProxy, or maybe a ShapedConsumer, in the right place(s) and exposing the bufferSize via CLI arguments, right?

To give a better answer I'd need to remind myself of all the pieces there, but "something like that". It might actually be as easy as a .registerProducer() (or consumer) in the right spot since everything should already be "producer aware".


WIP - DO NOT MERGE Change docker login parameter to --username

Summary
Explain the goal of this change and what problem it is solving. Format this cleanly so that it may be used for a commit message, as your changes will be squash-merged.
Test Plan
How did you test these changes? Please provide the exact scenarios you tested in as much detail as possible including commands, output and rationale.

This was just POC, closing


Support non-ASCII characters on Windows

This PR builds on #130 and implements the TODO: replacing the Ruby Env hash with a powershell script that builds up a ruby hash.

Coverage increased (+4.3%) to 95.044% when pulling 69e62a59eec4f94ded6274b1adbeb73db0ca9126 on da-ar:js_test into 16c30fcb4b6026ad6b9b32fc943ceb7b74840ff9 on enkessler:master.

Coverage increased (+4.3%) to 95.044% when pulling 69e62a59eec4f94ded6274b1adbeb73db0ca9126 on da-ar:js_test into 16c30fcb4b6026ad6b9b32fc943ceb7b74840ff9 on enkessler:master.


Fix crash when converting size string to UInt64

For some tar files, sizeString returns a string with a whitespace at the end.
UInt64(sizeString, radix: 8) would then return nil, and unwrapping the optional with ! would lead to a crash.
Unlike the UInt64 constructor, strtoull properly handles the string with whitespace at the end.
Another option would be to trim the whitespaces at the end of sizeString but it seemed safer
to avoid manipulating that string.

I presume you've tested this PR, and that it does indeed compile and run, etc., so I'm giving it the green light.


leftwm: 0.2.8 -> 0.2.9

Motivation for this change
Fixes https://github.com/leftwm/leftwm/issues/554
Things done
Bump version 0.2.8 -> 0.2.9

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 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

[x] Fits CONTRIBUTING.md.

Output of nixpkgs-review pr 143971:
building '/nix/store/c6d6s8hg2z9lragirkgqw6gdxcpzxi10-review-shell.drv'...

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/143971

1 package built:
leftwm```


PE-2583: Login Flow

--- Releases ---
Android release: https://appdistribution.firebase.google.com/testerapps/1:305132849030:android:6cf0cd5ec064fad3ffce07/releases/4si2aomlsk9v0

Note to myself: this it the Hackathon PR 🤓

Closing this one. When we prioritize this feature again we can reopen it.


SUA over https download

SUA using https download RAUC bundle

Prepared to be tested over https download only
Working for now as http download
Tested for https and http download from browser

Looks good


Fix revision age

Changes Proposed

Check List

[ ] The title of my pull request is a short description of the changes
[ ] This PR relates to some issue:
[ ] I have documented the changes made (if applicable)
[ ] I have covered the changes with unit tests

Codecov Report
Patch and project coverage have no change.

Comparison is base (1e432ea) 30.04% compared to head (f8b685b) 30.04%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@                Coverage Diff                 @@
##           helm-dashboard-v2     #420   +/-   ##
==================================================
  Coverage              30.04%   30.04%
==================================================
  Files                     10       10
  Lines                   1348     1348
==================================================
  Hits                     405      405
  Misses                   902      902
  Partials                  41       41

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Is a certain icon library needed?

I'm using ignite, and with npm issues I've been using yarn. So I installed via
yarn add react-native-datepicker

and got the following error on import:

Thoughts on how to resolve?

I removed the date_icon.png file, and then re-added it and everything worked fine. Strange, but it worked.

Restart the RN packager should work.


clarify data-in-modules usage

I merged the suggestion from @ekohl . Please have a look again. If there isn't a majority against this in the next days, I'm going to merge this PR.


fix(curriculum): typos in BETA Balance Sheet section

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Step 34 Removed the word the from the phrase  within the :not(), though keeping the and adding pseudo-class after not() would also work. I think the latter would be better since pseudo selectors if the focus of this section.
Step 46 Changed :active to :focus since that is the required CSS selector for the lesson.
Step 50 Removed the last comma in the 2nd paragraph
Step 51 Removed - and added a period then capitalized  create

If anyone sees this, should I wait for this to be resolved? Is there something I should do on my end?

Thank you for your contribution to the page! 👍
We are happy to accept these changes, and look forward to future contributions. 🎉


Refactor requests to use verification tokens

This has fewer security holes than the request files wrt replay attacks.

In the end, the status quo was just easier. We added some expiry on the request files to reduce the potential for replay attacks.


apex suggestion lot 2

apex suggestion:

[ ] load internal class (List, Map, Set,...)
[ ] suffix become (); on class afte =
[ ] primitive integer, boolean, Decimal,
[ ] if , if else for, while loop snippet
[ ] class method, properties (Map value to type)
[ ] Sobject
[ ] SOQL

basic suff
Map<String, String> referenceToType
Map<String, List> typeToProperties // and Methodes with argument
keywords


feat: implement basic v-memo

Related: #18.
Demo
I don't know if this is the best way to implement it. So, Please give me some feedback, thanks very much!

The issue has been marked as on hold. We are currently not implementing v-memo.


Preselect first is not working after update using axios

Hello,
I am using vue-multiselect and I'd like to preselect a value like this:
<multiselect id="new-item-unit"
             name="new-item-unit"
             placeholder="Select unit"
             :class="{'is-invalid': errors[0] }"
             :options="units.map(unit => unit.value)"
             :custom-label="opt => units.find(x => x.value == opt).text"
             :allow-empty="false"
             :preselect-first="true"
             :multiple="false"
             v-model="newitem.unit"/>

That works fine, but once newitem is being changed (in this case by replacing it using axios), the field is empty.
fetchItem (id) {
	axios.get (`/api/item/${id}`).then (response => {
		let itm = response.data.data;
		this.newitem = JSON.parse (JSON.stringify (itm));
	});
},

How can I "reload" the preselect-first value?

@internethelden Did you find any solution to this?


The daily working time of Beijing time cannot open https://realm.io

Sorry, I'm not sure what we can do about that.

。。。。。。。


Remove 0 values in aggregated report

Remove 0 value snapshots in aggregated reports as well.

Codecov Report

Merging #1755 (fd9cc0c) into master (c12e552) will decrease coverage by 58.51%.
The diff coverage is 0.00%.

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

Codecov Report

Merging #1755 (fd9cc0c) into master (c12e552) will decrease coverage by 58.51%.
The diff coverage is 0.00%.

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


Added missing index item for ACA

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

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

Thanks, @roberto-mardeni!
#sign-off


Saving the point cloud to a .ply file using the Python wrapper

Required Info

Camera Model
D400

Firmware Version
SDK 2.0

Operating System & Version
Windows 10

Kernel Version (Linux Only)

I would like to save the point clouds to .ply files using the Python wrapper of the SDK, in the same way that I can save .ply files from the SDK's GUI. But I am unable to find documentation explaining how to achieve this. All I found is the following line:
       points = pc.calculate(depth_frame)
But what sort of object is 'points'? What methods can one call for it? Where can I read about it?
Thanks in advance!

Hi @felix-tracxpoint
At the moment PLY conversion is not part of the core SDK but only the UI.
We will consider adding it to the API (making it available to all the wrappers) in one of the future releases.
For now, the PLY format is rather simple, and you can use the following code as reference:
model-views.cpp:199-289

Is there some other format I can export to, without implementing the whole thing by myself?
Thanks!

There is no export functionality at the moment. If all you need is to serialize the data to save it for later use, you might want to check SDK recording capabilities. However, ros-bag format is fairly proprietary and will not help you export the 3D model into 3rd party software.

You can now save MeshLab-compatible PLY files directly from pyrealsense

This is great! But how...?

We added an example - export_ply_example.py.
At the moment, it is still only on the development branch, but it will be merged to master with next release (and also added to next binary installer)

Is there any way to render (visualize) the pointcloud obtained from Intel RealSense API in Python? There are examples for rendering in C++ but not in Python. I hope you may give me a hint here. Thanks


[ Feature Fix] Import token page jump

Purpose
In IE's default setting, window.location.reload() will load a new page and start at the top of the page. After users import token, they have to scroll down the page and then they could come back to the config setting part. In this fix, it will firstly set the scroll location with hash tag before reloading the page.
Resolved #2690
Change

Set hash tag before reloading -- window.location.hash = "#configureAddonsAnchor";
Add IE browser recognition code. Only when user uses IE will hash tag be set.

Side effects
None. Set hash and only limit to IE browser
Others
Among those commits, I also tried to use cookie to record the scroll position before reloading. However, it is much more complicated and has potential security risk.  Hashtag is easier way and hopefully could be applied to solve the similar page jump issues in IE.

Looks good. :octopus: In the future, please use a gitflow tool to create your feature branches.


Add DOS2 manifest for editing your service

https://trello.com/c/iptxy2Oy/397-allow-dos-suppliers-to-view-their-services
Note, the manifest has no editable questions and will only be used for viewing the service

LGTM 👍