Use name instead of UUID for .mobileprovision copy

Signed-off-by: Goffredo Marocchi<EMAIL_ADDRESS>
Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

By default fastlane match generates the profiles with a descriptive name with Profile type (App Store, Enterprise, Development), and bundle id.
Whenever profiles are recreated, which can happen for a multitude of reasons, such as adding new devices to the portal, a new entry appears inside the user's ~/"Library/MobileDevices/Provisioning Profiles" folder as each profile will have a different filename currently based on the profile UUID. With enough applications and enough variations of the profiles (One team for InHouse Distribution can have its enterprise, adhoc, and development profiles) which may easily lead to a folder growing a fair bit each time.
For the same reasons we use full manual signing with match or with our own match like setups, being able to have a single copy of a certain provisioning profile and not having to rely on Xcode or other tools to use the latest profiles created with all the devices you have just added, for example, it would be nice to have clear names for the profiles we install on our computer and replace the older one with the contents of the new one we have recreated.
I am not sure if this Xcode behaviour was always there or if it occurred when the support for Provisioning Profile names instead of their UUID in your Xcode project signing settings landed (Xcode 8.x), but I have noticed that simply copying the files I downloaded from the Developer Portal in my ~/"Library/MobileDevices/Provisioning Profiles" folder ended up working just fine from Xcode's point of view. I also do realise that fastlane is a tool used by people that not only may have an appreciation for automation and DevOps-y stuff in general, but that have scaling needs (tons of apps, several Apple Developer Teams, mixing company and personal work, etc...).
This pull requests aims to have match copy the filename it has created as is, but adding a factor of uniqueness that should satisfy the needs we have. This would make the apparent structure of the final destination filename to be the following:

<Apple_Developer_Team_ID>_<InHouse/Development/AppStore>_.mobileprovision

Part in bold is what this PR adds, the rest is the name match was already using.
The filename is calculated with the Team ID and the original name Fastlane created for the profile, helping the profile be unique (containing the risk of collisions), but not in such a way that we are providing protection from collision from itself, i.e. we allow by design the new profile name to be equal to the old profile name for the same Team ID, the same profile type, and the same bundle ID as before and we delete the older file if present before copying the new one.
Testing

Ran the unit tests, followed the instructions in https://github.com/fastlane/fastlane/blob/master/YourFirstPR.md#test-the-changes-for-your-application and used the local fastlane modified version  to sync all the provisioning profiles from the App Store, Enterprise, and Development provisioning profiles types
#!/bin/bash

BUNDLE_IDS=("com.company.corp.app1" "com.company.corp.app2" "com.company.corp.app3" "com.company.corp.app4" "com.company.corp.app5" "com.company.corp.app6" "com.company.corp.app7" "com.company.corp.app8" "com.company.corp.app9" "com.company.corp.app10" "com.company.corp.app11" "com.company.corp.app12" "com.company.corp.app13" "com.company.corp.app14" "com.company.corp.app15")

for bundle_id in "${BUNDLE_IDS[@]}"; do
	echo "Building venture with bundle id $bundle_id"
	fastlane match development --readonly -a $bundle_id --team_name "Company" --skip_docs
done

[...]

bundle exec rspec Match
[Coveralls] Set up the SimpleCov formatter.
[Coveralls] Using SimpleCov's default settings.
Changing stdout to /tmp/fastlane_tests, set `DEBUG` environment variable to print to stdout (e.g. when using `pry`)

Match::CommandsGenerator
  :appstore option handling
    can use the git_url short flag from tool options
    can use the git_branch flag from tool options
  :adhoc option handling
    can use the git_url short flag from tool options
    can use the git_branch flag from tool options
  :development option handling
    can use the git_url short flag from tool options
    can use the git_branch flag from tool options
  :enterprise option handling
    can use the git_url short flag from tool options
    can use the git_branch flag from tool options
  :change_password option handling
    can use the git_url short flag from tool options
    can use the shallow_clone flag from tool options
  :decrypt option handling
    can use the git_url short flag from tool options
    can use the shallow_clone flag from tool options
  nuke development option handling
    can use the git_url short flag from tool options
    can use the git_branch flag from tool options
  nuke distribution option handling
    can use the git_url short flag from tool options
    can use the git_branch flag from tool options

Match
  Match::Encrypt
    encrypt
    raises an exception if invalid password is passed
    raises an exception if no password is supplied

Match::Generator
  calling through to other tools
    configures cert correctly for nested execution
    configures sigh correctly for nested execution

Match
  Match::GitHelper
    generate_commit_message
      works
    #clone
      skips README file generation if so requested
      clones the repo
      clones the repo (not shallow)
      checks out a branch

Match
  Match::Runner
    creates a new profile and certificate if it doesn't exist yet
    uses existing certificates and profiles if they exist

Match
  Match::Setup
    works

Match
  Match::Utils
    import
      finds a normal keychain name relative to ~/Library/Keychains
      treats a keychain name it cannot find in ~/Library/Keychains as the full keychain path
      shows a user error if the keychain path cannot be resolved
      tries to find the macOS Sierra keychain too
    fill_environment
      #environment_variable_name uses the correct env variable
      #environment_variable_name_team_id uses the correct env variable
      #environment_variable_name_profile_name uses the correct env variable
      #environment_variable_name_profile_path uses the correct env variable
      pre-fills the environment

Finished in 1.34 seconds (files took 1.52 seconds to load)
38 examples, 0 failures

Robocop detected no issues:
bundle exec rubocop -a
Inspecting 989 files
.............................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................

989 files inspected, no offenses detected

When running all tests on my machine I have 4 failures in portions of the code I have not touched and also occur if I do a clean clone of the official fastlane repo's master branch, install dependencies with bundler, and execute the tests. These are the results:
[...]

Failures:

  1) FastlaneCore::TagVersion correct? returns true for versions supported by Gem::Version
     Failure/Error: expect(FastlaneCore::TagVersion.correct?(tag)).to be true

       expected true
            got #<Fixnum:1> => 0
     # ./fastlane_core/spec/tag_version_spec.rb:20:in `block (3 levels) in <top (required)>'

  2) FastlaneCore::TagVersion correct? returns true for tags that can be converted to a Gem::Version using version_number_from_tag
     Failure/Error: expect(FastlaneCore::TagVersion.correct?(tag)).to be true

       expected true
            got #<Fixnum:1> => 0
     # ./fastlane_core/spec/tag_version_spec.rb:25:in `block (3 levels) in <top (required)>'

  3) FastlaneCore::TagVersion correct? returns false for tags that are not versions
     Failure/Error: expect(FastlaneCore::TagVersion.correct?(tag)).to be false

       expected false
            got nil
     # ./fastlane_core/spec/tag_version_spec.rb:30:in `block (3 levels) in <top (required)>'

Finished in 4 minutes 41.1 seconds (files took 2.83 seconds to load)
4735 examples, 3 failures

Failed examples:

rspec ./fastlane_core/spec/tag_version_spec.rb:18 # FastlaneCore::TagVersion correct? returns true for versions supported by Gem::Version
rspec ./fastlane_core/spec/tag_version_spec.rb:23 # FastlaneCore::TagVersion correct? returns true for tags that canbe converted to a Gem::Version using version_number_from_tag
rspec ./fastlane_core/spec/tag_version_spec.rb:28 # FastlaneCore::TagVersion correct? returns false for tags that are not versions

This is my gem env
RubyGems Environment:
  - RUBYGEMS VERSION: 2.6.14
  - RUBY VERSION: 2.3.3 (2016-11-21 patchlevel 222) [x86_64-darwin16]
  - INSTALLATION DIRECTORY: /Users/goff.marocchi/.rvm/gems/ruby-2.3.3
  - USER INSTALLATION DIRECTORY: /Users/goff.marocchi/.gem/ruby/2.3.0
  - RUBY EXECUTABLE: /Users/goff.marocchi/.rvm/rubies/ruby-2.3.3/bin/ruby
  - EXECUTABLE DIRECTORY: /Users/goff.marocchi/.rvm/gems/ruby-2.3.3/bin
  - SPEC CACHE DIRECTORY: /Users/goff.marocchi/.gem/specs
  - SYSTEM CONFIGURATION DIRECTORY: /Users/goff.marocchi/.rvm/rubies/ruby-2.3.3/etc
  - RUBYGEMS PLATFORMS:
    - ruby
    - x86_64-darwin-16
  - GEM PATHS:
     - /Users/goff.marocchi/.rvm/gems/ruby-2.3.3
     - /Users/goff.marocchi/.rvm/gems/ruby-2.3.3@global
  - GEM CONFIGURATION:
     - :update_sources => true
     - :verbose => true
     - :backtrace => false
     - :bulk_threshold => 1000
  - REMOTE SOURCES:
     - https://rubygems.org/
  - SHELL PATH:
     - /Users/goff.marocchi/.rvm/gems/ruby-2.3.3/bin
     - /Users/goff.marocchi/.rvm/gems/ruby-2.3.3@global/bin

Changes Description

The major change in the code is really in the file that manages the provisioning profiles (provisioning_profiles.rb in fastlane_core):
      # @return [String] The Apple TeamIdentifier of the given provisioning profile
      def team_identifier(path)
        parse(path).fetch("Entitlements").fetch("com.apple.developer.team-identifier")
      end

      def profiles_path
        path = File.expand_path("~") + "/Library/MobileDevice/Provisioning Profiles/"
        # If the directory doesn't exist, create it first
        unless File.directory?(path)
          FileUtils.mkdir_p(path)
        end

        return path
      end

      # Installs a provisioning profile for Xcode to use
      def install(path)
        UI.message("Installing provisioning profile...")
        profile_filename = team_identifier(path) + "_" + File.basename(path)
        destination = File.join(profiles_path, profile_filename)

        if path != destination
          # copy to Xcode provisioning profile directory and delete previous file if present
          FileUtils.remove_entry(destination, force: true)
          FileUtils.copy(path, destination)
        end

        destination
      end

The filename is calculated with the Team ID and the original name Fastlane created for the profile, helping the profile be unique (containing the risk of collisions), but not in such a way that we are providing protection from collision from itself, i.e. we allow by design the new profile name to be equal to the old profile name for the same Team ID, the same profile type, and the same bundle ID as before and we delete the older file if present before copying the new one.

I signed it!

I signed it

I signed it!


NAudio with .NET c# webform

How do I use NAudio in webforms?  I tried to build a player but Pause doesn't work and keep playing the audio.

NAudio could be used server-side in a web project, but to play audio in the browser, that would need to be done client-side, so you would use standard web technologies such as the HTML  tag.


pull request for e2e test at 12/30/2016 14:15:10

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


Add ValueAtRisk

Added the Jupyter Notebook in Tutorial Folder.

@PyDataBlog Have a look at it and let me know if you want me to make some changes.

@ApurvShah007 nice work. Try and see if you can add Conditional VAR as well.

@PyDataBlog I feel like you should merge this for now. Meanwhile, I am working on conditional VaR even for my own project at portfolio-optimizer. I will submit another pull request addressing that within a day or two.

@ApurvShah007 merged!


elixir_1_10: 1.10.0 -> 1.10.1

Motivation for this change
https://github.com/elixir-lang/elixir/releases/tag/v1.10.1
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This change is

Thanks a lot.


srf: srf.c:143: ca_kmer_read: Assertion `(i&1) == 1' failed. Aborted (core dumped)

Hi, Heng. Can you please advise on what might be the issue?
I have set up a separate conda environment, and in this environment, I have installed the following: k8, minimap2, and blast.
The installation of the srf and kmc programs was done according to the instructions on github pages.
I used kmc to compute 50-mers, and the program completed without any errors. However, I encountered an error when trying to run srf with the following command:
srf -p prefix count_kmc.txt > srf_contigs.fa
The error message looks like this:
srf: srf.c:143: ca_kmer_read: Assertion `(i&1) == 1' failed.
Aborted (core dumped)

I would appreciate your assistance!

Please try a k-mer of odd number (like 51bp, not 50bp).


add toggle debug mode to toggle to pseudo locale

Coverage decreased (-0.03%) to 39.285% when pulling 9261f86e7abd7bca182eb6a67a1d88d108a6e9af on caffffe:locale-setlocale into a18208900fd7ac574034cd0669e536bbaca0aa73 on ringcentral:master.


remember the color of the note when refreshed

I have updated it, please check once

It still doesn't work as we have these functions from js files,

And in JS, modules don't work like this, we should have these function bindings inside of a JS module.

Then please fix this issue by opening a new issue

Hi ArunBohra12, for some reason when we used 'modules' the changeTheme function does not work. So if it required that we need to add the modules, i know how to fixed :)
I am going to make the changes, give me a couple minutes to do that :)
working on it...

@yamitrvg12 It doesn't work because you must import functions from modules you can't use like js scripts. Read differences here
And we need modules as we need to build the app. Please make sure that you run npm run build and then npm run preview. And once that success then only you can push without errors.

@ArunBohra12 Could you please take a look again https://github.com/aditya-singh9/notem/pull/20

Seems ok to me. I left review #20
If you want to do that just update the pull req instead of adding another one


Switch to new focus brushes

Description

Switching to new focus brushes.
Motivation and Context

Closes #4560
How Has This Been Tested?

Tested manually.
Screenshots (if appropriate):

@marksfoster @YuliKl Is the design intent to change these targeted backgrounds or to update the system focus rect colors ?

I apologize, I was too vague when originally creating this work item internally. The intent is to update two brushes from generic.xaml, SystemControlFocusVisualPrimaryBrush and SystemControlFocusVisualSecondaryBrush to use new colors: FocusStrokeColorOuter and FocusStrokeColorInner.
https://github.com/microsoft/microsoft-ui-xaml/blob/d2f08547d2a4c4a1b8be251811aa10d0640759c4/dev/CommonStyles/Common_themeresources_any.xaml#L77

@YuliKl So we would introduce those two brushes in the WinUI code base now and point them to those two resources?

Updated that now.

/azp run

@chingucoding do these visual verification failure makes sense?

Failures looked good to me, we changed quite a core brush that is referenced by multiple controls. most importantly ListView which is used in other controls. Updated the verification files now.

/azp run

@chingucoding looks like a few failures in rs4 where the baselines dont match.

Updated them now.

/azp run


[CdapIO] Add integration tests for CdapIO (Batch)

Resolves #22315

Added test CDAP Plugins for reading/writing from/to PostgreSQL database and Input/Output Formats and FormatProviders for these plugins.
Added CdapIOIT class  on HadoopFormatIOIT with integration test that performs the following:

Writes TestRowDBWritables to the PostgreSQL database using DBBatchSink CDAP plugin.

Reads test rows from the PostgreSQL database using DBBatchSource CDAP plugin.

Asserts expected and actual hashcodes for given rowCount.

Added Jenkins job called job_PerformanceTests_CdapIO for running CdapIOIT tests.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Spotless PreCommit

Run Java HadoopFormatIO Performance Test

Run Java CdapIO Performance Test

Run seed job

Run Seed Job

Run Seed Job

Run Java CdapIO Performance Test

Run Seed Job

@chamikaramj Cham, PR is ready for review. Thanks!

Run Java CdapIO Performance Test


Update CheatManager.hpp

Utilized a member initialization list in the constructor, which can be more efficient than assignment inside the constructor body because it avoids default-constructing the member variables and then assigning to them.
Moved the member variables to the private section of the class to encapsulate the internal state and prevent unwanted external access or modification. This ensures that the class is used as intended and protects its internal state.
Retained the use of noexcept specifier in the constructor, signaling that it doesn’t throw exceptions.

Could you please release a compiled version of it? I wanna give it a try, but I'm just bad at compiling :(

Could you please release a compiled version of it? I wanna give it a try, but I'm just bad at compiling :(

#2


Do not work

Hi again,
with the libs I can compile, but wen I exacute the code stops in
if(glewInit() != GLEW_OK)
What should I do?
Thank you

Just to add more info...
In the command line is written "GL-Version: 4.5 clGetDeviceIDs: -1"

Are you sure the crash happens at the glew initialization? Your output seems like the initialization of OpenCL is not working as expected. clGetDeviceIDs is unable to find a device. You might have to debug what is going wrong. It might have to do with the way the initialization is done - it is a bit weird when you try to get a shared context between OpenCL <=> OpenGL.
Desipite of that the application was tested on AMD and Nvidia hardware so in theory it should work on most platforms.


Re-creating official secretary role

At one point in the past, we had an official secretary role. With our previous election for CommComm chair, the role got folded due to lack of individuals to hold it. At the time, it wasn't a massive issue to not to have an official secretary, but as time has proceeded our workload has increased with the addition of new projects/initiatives and more observers and members engaging in the Committee.
I'd like to propose that we reinstate an official Secretary role. The individual(s) who take on this role would be tasked with:

Taking meeting minutes for each CommComm meeting
Submitting a PR with those meeting minutes into the CommComm repo once the meeting is over

Something that we probably need clarity on before electing a secretary is whether or not they need to be an existing CommComm member or not. I don't particularly mind one way or the other, but I do see it as a way to significantly contribute to the Committee - but it is certainly also a non-trivial commitment in terms of time.

This does look like a good idea. I hear you typing every time, it must be pretty hard chairing and writing at the same time. I don't know how exactly it will be done, and I don't know if I'm sufficiently fluent in English to understand everyone properly, but if no native speak wants to do it, I'll apply. But I'd prefer if someone else with better abilities would do it.

Awesome!! what are the criteria for the position?

I'm always happy to help take the notes during the meetings, it's one small way I've been able to contribute a bit. (But I won't stop anyone else who might be eager too.) Just one or two quick thoughts...
I've noticed it can be tough to both take notes and also handle the roll-call / attendance, so anything we can do to cue up the names of attendees/observers/etc ahead of time is helpful (especially since we're doing introductions a little quicker now). I'll try to get in there at least a couple minutes early next time and start jotting down some of the names on the call right away.
Also: the fact that we ultimately output the notes on github as markdown makes using google docs a little awkward. Would love to hear if there's a collaborative markdown solution that's easy to use, but I acknowledge that docs is at least free and accessible. No matter who is on notes, there's always a little awkwardness while trying to format bullet items or links.
@bnb I hung out near your Twitch stream one night as you were getting the note template all put together. Seems mostly straightforward, though you were still doing some work manually adding issue links. Would you still be the one putting the agenda together?

I agree that the formatting of google docs does make it harder to keep it in markdown but don't have a better suggestion.

@JemBijoux so, @mhdawson has mostly automated the issue, agenda, and doc creation - it's super convenient now. Basically little/no effort on our end.
I will also suggest trying out this Google Apps script - I use it almost daily for my own work to convert a properly formatted Google Doc into Markdown. @mhdawson we may want to investigate its usefulness to us.
https://gist.github.com/bnb/a02712c3f65d18d4ff2c55fc8bab7ace

I'm definitely +1 on this, even if only for the timely PRing of meeting notes.

My approach has been to try to capture the meeting minutes in markdown in the google doc, but it does sometimes fight you so finding a better easier way could make sense.

I am not a member of CommComm, but would love to help.

This was discussed in #261. @JemBijoux has kindly accepted taking on this role! 🎉 The door is open to anyone, members and observers alike, who wants to help out.

@chowdhurian will love to help too.


xdg-desktop-portal-xapp: init at 1.0.0

Description of changes

The Cinnamon module now enables XDG desktop integration by default. If you are experiencing collisions related to xdg-desktop-portal-gtk you can safely remove xdg.portal.extraPortals = [ pkgs.xdg-desktop-portal-gtk ]; from your NixOS configuration.

Things done

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, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Tested setting wallpaper via flatpak ashpd-demo

Tested the same thing on MATE and Xfce, the implementation on Xfce relies on some scripts provided by xapp, e.g. xfce4-set-wallpaper.

I imagine we still want to keep cinnamon-desktop even with https://github.com/linuxmint/xdg-desktop-portal-xapp/commit/5130d36ff670a53ae0d23cab238496b5c04b95b0 for wrapGAppsHook to pick up stuff for org.cinnamon.desktop.background :thinking:


Adds sRGB and Gamma conversions to ColorUtility.sdsl

PR Details
Adds the basic conversion methods for sRGB, and screen gamma.
Stride already has ToLinear methods, which are there same as SRgbToLinear, I could not remove them because it could potentially break existing shaders. Stride also has additional methods for single float and float3, which I have not provided, but I am open to adding this if requested.
Related Issue
We had this in a temporary file in the vvvv code, but it would be better to put this into the official ColorUtility of Stride:
https://github.com/vvvv/VL.Stride/pull/395#issuecomment-760253956
Motivation and Context
Getting color issues right when working with designers.
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Should we consider adding this note below to our PR template and perform for both new and existing code?
If this PR is approved, could we include the following sections for enhanced readability in our documentation: summary, param, returns, and remarks?
For reference, you can see an example at the top of this page: https://dotnet.github.io/docfx/docs/basic-concepts.html
Our new documentation will be released soon, and incorporating these changes could greatly improve its quality. 🙂

summary, param, returns, and remarks?

I am not sure if these work in SDSL shader code. In C# code I would say definitely yes!

Of course!🤣🤦♂ It's not C# like C# especially if it is not C# 🤣. I will reuse my comment in the C# PRs then..


About Mano parameter

Why the trans parameter of mano can be replaced by the T of the camera extrinsic?

Check this out: https://docs.google.com/document/d/1XIINXfWrFwrJos9TDeDBE6FcmlGd4Uab/edit
The code is combining the transformation of SMPL->World and World->Camera


proposal: x/crypto/argon2: Check if the parameters and the key length are valid

Proposal Details
According to RFC 9106, the valid values for the parameters and the key length are:

$1<=t<=2^{32}-1$ (number of passes, time)
$8p<=m<=2^{32}-1$ (memory size, memory)
$1<=p<=2^{24}-1$ (degree of parallelism, threads)
$4<=T<=2^{32}-1$ (tag length, keyLen)

No need to check maximum values of time, memory and keyLen, since these are math.MaxUint32. Also, since threads is uint8, there is no need to check the maximum value.
The minimum values of time and threads are already checked. However, the minimum values of memory and keyLen seem not to be checked. So, I would suggest checking for these minimum values and either panic or return an error if they are invalid.

CC @golang/security


Optimize the the endpoint update in SyntheticServiceEntry controller

We should not call configUpdate once the EDSUpdate has been triggered, also if a service changes is detected we should only call configUpdate and not the edsUpdate for individual endpoints.
https://github.com/istio/istio/blob/90b41f6426fa65c616d81cb1053a107a42f996f7/pilot/pkg/config/coredatamodel/syntheticserviceentrycontroller.go#L220
Depends on the completion of: #18416
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version)
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

Closing this in favor of https://github.com/istio/istio/issues/18625


Alter ande Delete Topics

Hello,
I have a question regarding the alter option,
I've created a topic with the following configuration:
version: 1
topics:
- configs:
    cleanup.policy: compact
    compression.type: producer
    min.insync.replicas: '1'
    retention.ms: '1000'
    max.message.bytes: '64000'
    flush.messages: '1'
  name: prueba-v2
  partitions: 10
  replication_factor: 1

when i excuted describe the configuration of that topic is shown:
../kafka/kafka/bin/./kafka-topics.sh --describe --zookeeper localhost:2181 --topic prueba-v2
Topic: prueba-v2	PartitionCount: 10	ReplicationFactor: 1	Configs: compression.type=producer,cleanup.policy=compact,max.message.bytes=64000,min.insync.replicas=1,retention.ms=1000,flush.messages=1
	Topic: prueba-v2	Partition: 0	Leader: 0	Replicas: 0	Isr: 0
	Topic: prueba-v2	Partition: 1	Leader: 0	Replicas: 0	Isr: 0
	Topic: prueba-v2	Partition: 2	Leader: 0	Replicas: 0	Isr: 0
	Topic: prueba-v2	Partition: 3	Leader: 0	Replicas: 0	Isr: 0
	Topic: prueba-v2	Partition: 4	Leader: 0	Replicas: 0	Isr: 0
	Topic: prueba-v2	Partition: 5	Leader: 0	Replicas: 0	Isr: 0
	Topic: prueba-v2	Partition: 6	Leader: 0	Replicas: 0	Isr: 0
	Topic: prueba-v2	Partition: 7	Leader: 0	Replicas: 0	Isr: 0
	Topic: prueba-v2	Partition: 8	Leader: 0	Replicas: 0	Isr: 0
	Topic: prueba-v2	Partition: 9	Leader: 0	Replicas: 0	Isr: 0

Now I'm going to modify the max.message.bytes property and assign it the value from 64000 to 128000
version: 1
topics:
- configs:
    cleanup.policy: compact
    compression.type: producer
    min.insync.replicas: '1'
    retention.ms: '1000'
    max.message.bytes: '128000'
    flush.messages: '1'
  name: prueba-v2
  partitions: 10
  replication_factor: 1

When I execute command:
docker run -it --net host -v $(pwd)/kafka-specs.yaml:/kafka-specs.yaml streamthoughts/kafka-specs --file /kafka-specs.yaml --bootstrap-server localhost:9092  --execute --alter --entity-type topics --verbose --yes

It indicates to me that you have not performed any operations
ok : 0, changed : 0, failed : 0
You know what I might be missing?
It's the same with the delete

Thanks in advance!!
Regards!

Hi


Add httpClient.close

PPG 391

will merge after TravisCI approves.    Although we are in PARANOID MODE, this change is stability-increasing, and thus will be pulled.

On a connection timeout, a client can still leak. Should close in the outer catch also.

never mind, the diff view was misleading.


New analyse task: check for empty blocks

Is your feature request related to a problem? Please describe.
Empty blocks should be avoided.

Done.


"gulp build" does not work properly

The problem is in the 'scripts' task.:
runSequence(['clean:dev', 'clean:dist'],
['scripts', 'compass', 'imagemin'],
'bundle', 'copy');
The tasks do not run in the correct sequence --the 'bundle' task starts before the 'scripts' task finishes. Because of this lag, there is no scripts folder available in the 'dist' folder for the bundle to use.

I guess the issue comme from gulpfile.js, can you try to replace at the end of the function of gulp.task('scripts', function() {}), the part of code:
} else {
bundle();
}
By this one below, and try again?
} else {
return bundle();
}
If it works, i will create a pull request for the fix, thx.

In order to implement your suggestion I also needed to add a return to the 'bundle' function. Now it works fine.
Thank you!


Mono 'The 'ExeConfigFilename' argument cannot be null.'

Hi all,
We've recently updated https://github.com/QuantConnect/pythonnet/pull/47 to be aligned with https://github.com/pythonnet/pythonnet and started getting the following error
The 'ExeConfigFilename' argument cannot be null.
  at System.Configuration.ExeConfigurationHost.CheckFileMap (System.Configuration.ConfigurationUserLevel level, System.Configuration.ExeConfigurationFileMap map) [0x0001b] in <65bca3ec97b145069173491f1a974cbf>:0
  at System.Configuration.ExeConfigurationHost.InitForConfiguration (System.String& locationSubPath, System.String& configPath, System.String& locationConfigPath, System.Configuration.Internal.IInternalConfigRoot root, System.Object[] hostInitConfigurationParams) [0x00030] in <65bca3ec97b145069173491f1a974cbf>:0
  at System.Configuration.InternalConfigurationSystem.InitForConfiguration (System.String& locationConfigPath, System.String& parentConfigPath, System.String& parentLocationConfigPath) [0x00000] in <65bca3ec97b145069173491f1a974cbf>:0
  at System.Configuration.Configuration..ctor (System.Configuration.InternalConfigurationSystem system, System.String locationSubPath) [0x0001f] in <65bca3ec97b145069173491f1a974cbf>:0
  at System.Configuration.InternalConfigurationFactory.Create (System.Type typeConfigHost, System.Object[] hostInitConfigurationParams) [0x0000d] in <65bca3ec97b145069173491f1a974cbf>:0
  at System.Configuration.ConfigurationManager.OpenExeConfigurationInternal (System.Configuration.ConfigurationUserLevel userLevel, System.Reflection.Assembly calling_assembly, System.String exePath) [0x000ef] in <65bca3ec97b145069173491f1a974cbf>:0
  at System.Configuration.ClientConfigurationSystem.get_Configuration () [0x0000e] in <65bca3ec97b145069173491f1a974cbf>:0

Steps to reproduce:
from pythonnet import load
from clr import AddReference
AddReference("RestSharp")
from RestSharp import *
client = RestClient("https://www.google.com/")
restsharpResponse = client.Execute(RestRequest())
print(restsharpResponse.ErrorException.InnerException)

I suspect it's related to this new clr-loader, seen some old related issues https://github.com/pythonnet/pythonnet/commit/37a4ea597631266c44593db2a365f9df58c8c06e & https://stackoverflow.com/questions/35872504/pythonnet-exception-the-execonfigfilename-argument-cannot-be-null .

Seems mono_domain_set_config isn't being called anymore? should it be called automatically internally (as before I believe) or we are missing some call for it in? Or is it maybe we are screwing it up in the way we consume it/missing some step

Thanks!

Request to close this because of #9
Also if possible can we do a version bump and publish the latest?

Fixed in 0.1.6, thank you very much :)


Add option for common CMake Tools commands in Side Bar

We will add the option to pin shortcuts of CMake Tools commands (such as Delete Cache and Reconfigure in the Side Bar.

Instead of just common commands, I think this should be the ability to pin any CMake command


Python not recognized

Typing python dance.py or python3 dance.py doesn't work
Adding python to environment variables doesn't work
Typing py dance.py works for whatever reason
It would be cool if instead of having to do everything in command prompt this came with an installer and launcher, if that's at all doable for dev. thanks.

Hi!
The issue with Python is quite odd, I've never had something like that happen to me. Does it work on other Python files? If not, it's probably an issue with Python. Either way, my understanding is that it's not a major problem, right?
I agree, an installer would be great! One was suggested in # 18 and it was shown to be easily possible. However, this project is no longer actively developed, so it's unlikely that this would be added. In the meantime, you could try creating a .ps1 file that opens JoyDance, then create a shortcut that opens that file in PowerShell. You can even place it in the Start Menu! You can find info on that online. (If you want, I have one that works that I could provide you, but my computer is off right now and I don't feel like waiting for it to boot).
I hope these help :D

I can try getting my ps1 script to you later today, it's not very complicated.

I can try getting my ps1 script to you later today, it's not very complicated.

thank you so much, really appreciate it

I've uploaded the .ps1 and the shortcut in this file. I'd recommend placing the .ps1 in the root of the directory in which you have JoyDance installed, but you should be able to put it wherever. Edit it to point to the correct directory. Put the shortcut on the desktop, then go to Properties and edit the path to point to the .ps1 script. If you want, you could then move or copy it to the Start Menu. If you have any questions, I'm happy to try to help :D
JoyDance Shortcuts.zip


add check for non nans in satellite data

Pull Request
Description
Have option for PVnet to raise error if there are nans in the satellite data
Helps with https://github.com/openclimatefix/PVNet/issues/31
How Has This Been Tested?
Added a test

[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Looks good.
We might want to com back to this later to allow some NaNs. If the 30/60 minute delay wasn't the cause of the zig-zag then we might want to go back to a model which can use either, and therefore best use the available inputs. But I think this is good now

Yea good point, I think then having the bool to turn it on and off is good


Add limitation for number of Blockly blocks

Some challenges require this. For now just on Blockly side.
Later we can think of integration with arena/challenge selection

For anyone who's interested in picking this up:

It seems Blockly natively supports this, via its maxBlocks "Injection Options": https://developers.google.com/blockly/guides/configure/web/configuration_struct
Corresponding demo program https://github.com/google/blockly/blob/master/demos/maxBlocks/index.html


Grammar and formatting fixes

Prevent ### from showing in the README.
Change "Install" to "Installation" to match the form of other headings, like "Development" and "Deployment".

👍


add flavorjones/webhook-notification-resource

Signed-off-by: Mike Dalessio<EMAIL_ADDRESS>

Related: https://github.com/flavorjones/webhook-notification-resource/issues/2


Experimental language translation

This PR resolves #2624.
This PR adds a translate function to the String prototype, which when called searches a given YAML file using the string as key, returning the YAML file's value if found, and the original text if not.

The current state of this PR is that only the Account menu has been modified to use the new translate function.

I could put this syntax into all texts, should i or do you think we should wait in case we want another syntax?

I could put this syntax into all texts, should i or do you think we should wait in case we want another syntax?

I've kept it as a Draft PR for now, specifically so we could let the syntax discussion play out.
That said, it's been a couple of days, and I think the following ideas are worth following up on:

The template file key should match the literal string being translated.
A translateDefaults function, so that calls can be .translate() instead of .translate(translateOpts).
React's useContext hook. It seems to be ideal for this feature, but may require some refactoring to make it work.
Use AccountPage to hold the UI elements for language selection.

On 1. As it currently stands, if the string to be translated does not exist in the localization file, then the string itself is returned, allowing as to use the exact text as the key. That said, it doesn't HAVE to be the exact text, and where the text is particularly long, it's probably a good idea to use an Identifier instead.
However, I have not put any thought into a process for identifying strings that should be referenced by ID instead of literal text. Maybe string.length > 20 or some other arbitrary number?
This will effect the localization template, so I'd like to lock it down before we push too much further into the process; there may be work already completed that will be affected by this decision.
On 2. This seems like a good idea - as each JSX file deals with one area of the UI, we should be able to set the defaults once per file and have it "just work" from then on. Implementation, however, is usually where things fall apart.
For 3. I think we need useContext in order to switch the state of the entire UI, and that appears to be the exact purpose of the feature. That said, I have only a passing familiarity with useContext, so I need to go back and look at how to actually implement such a thing.
Finally, 4. Ultimately, no matter what page this is implemented on, a value needs to be manipulated and stored by the page, then retrieved and applied to the data source that the translate() function uses.
a. AccountPage: build the UI to manipulate the value
b. AccountPage: store the value
c. Translate(): retrieve the value
d. Translate(): apply the appropriate localization data

I intend to take a further stab at some of these as time allows in the next couple of days; however, given the time of year, time is a fleeting thing right now...

The template file key should match the literal string being translated.
A translateDefaults function, so that calls can be .translate() instead of .translate(translateOpts).

On 1. As it currently stands, if the string to be translated does not exist in the localization file, then the string itself is returned, allowing as to use the exact text as the key. That said, it doesn't HAVE to be the exact text, and where the text is particularly long, it's probably a good idea to use an Identifier instead.

It is entirely reasonable to have the source string in the code be "Error #234" and the translation be "An error occured (#234). This sometimes happens because [long reasons]. You can try [stuff]". The "identifier", per se, doesn't have to be a neologism, it can be a simple plain language string (one we'd be happy getting stuck as being permanent as a key only visible in the code .. so no rude words pls).
If we do go with using strings-in-the-code as the keys-in-the-translations, we definitely should have a code-to-English translation file (and not rely on the fallback-to-key method for English UI). Otherwise we can't easily change the English UI (because it would break the keys to other langs).
Shouldn't be hard to find a volunteer to write up the English translation file ;-)

Quick check .. does this PR include setting the <html lang="%UI-LANG%"> attribute?
As of v3.10, THB does not assert a lang on <html>. It does have Open Graph tags, of which the spec defaults to en_US for og:locale .. though that should be set from the brew language, not the UI language, so that's a separate PR.

does this PR include setting the  attribute?

We could, not sure it is necessary, but it is best practice

Last time we talked about this, we talked about what languages we could add as a start, we agreed on:

EN (is there value in doing EN-US and EN-GB?)
DE (German)
ES (Spanish)
PT-BR (Brazilian Portuguese
CA (Catalan)
VA (High Valyrian) @G-Ambatte we are doing it, right?

* HV (High Valyrian) @G-Ambatte we are doing it, right?

Engos is tongue, but if I remember correctly can also be used to mean "language"; while arilla is "beer" but might be close enough to "brews".
This is why we need to finish a template file, to determine exactly what words we actually need.

This is why we need to finish a template file, to determine exactly what words we actually nee

base-localization-file.zip
You mean this one still lacks strings?

Right now the user page takes the following syntax:
Username's published brews
Username's unpublished brews

This is not only redundant, but difficult to translate, since the 's is conditional to the word, and added using another function.
We propose the following change:
Brews by Username

Published brews
Unpublished brews

ToDo:

[ ] Resolve comments
[ ] Snippet names
[ ] Add Languages

Numbered, because they should follow that order.

@calculuschild there is no point on deploying if we can't log in to change the language.

Also no harm to deploying.
In any case you can log in by copying the cookie from the live site over to the deployment.

Quick check .. does this PR include setting the <html lang="%UI-LANG%"> attribute?
As of v3.10, THB does not assert a lang on <html>. It does have Open Graph tags, of which the spec defaults to en_US for og:locale .. though that should be set from the brew language, not the UI language, so that's a separate PR.

Fixed that, the lang attribute is now set in the html tag, following the official docs syntax.

Getting an error trying this on the home page:

I think in the spots where we use MomentJS to set relative times ("1 year ago"), such as in the metadata viewer in the navbar and in the Recent Items dropdown, we can use translation as well.  Check out this page from MomentJS docs: http://momentjs.com/docs/#/i18n/instance-locale/

I think i have fixed most comments and reviews, except for those which require discussion or i need an answer about something, if i'm wrong, please notify me about it.

A few comments in this review, and then a couple of separate comments because I didn't think i'd do as many as I did.
Also-- Do you have any ideas on how tooltips can be translated? It seems right now that tooltips are done entirely with LESS/CSS, going all the way to tooltip.less. Can translate() work in a LESS file?

If i understand the code correctly, we use an html attribute which is then referenced in the less, so we don't need to change anything in the less file.

Has this been shelved for another path?


Updated app.py

Disabled strict slashes for the Flask app.

I'm not sure it's a good idea to override the default behavior here. I'd rather stick with the default and allow users of this template to change it if they need to.
Thank anyway for the PR!


how  to trace the batch task

Please answer these questions before submitting your issue.

Why do you submit this issue?
Question

Question
i have a batch task. the tasks in the batch will be executed in each new thread one by one. but now the tasks have the same tid printed in log. so how can i do to trace the each task.

There are many ways, how to measure a trace completed, and propagate context.
DolphinScheduler, Spring Async, Elastic Job are all batch task frameworks, and we have plugins for them.

thx


Improve rendering performance by isolating canvas SVG elements in a Shadow DOM

This aims to reduce the time spent in the "Recalculate Style" part of the "Rendering" slice of the page framerate performance in the Chome DevTools profiler. However it does seem to come at the cost of additional time spent on "Parse HTML". All in all, it seems to be roughly faster by 10-20 FPS when the frames are taking around 250-300ms to render with a stress test showing tons of overlays.

We should also probably isolate the thumbnails in a shadow DOM, and perhaps rate limit their updating when performance is tanking.

I may or may not revisit this someday and attempt to extend this to also handle the thumbnails that are potentially also adding overhead of the similar sort, and then benchmark it again to see if the results aren't a wash like they are in the current state.


fix: tweak build.sh handling of package.json

The includes and excludes are handled a bit oddly in the old build.sh pattern.
This change always does the requested installs, but then always does the override remove for excludes, which enables excluding dependancies of packages which were just requested to be installed.
This should cause #519 to work as expected.

I believe there are scenarios in which the old build.sh handling of packages is needed. In my fork, I install vim-default-editor which requires the package nano-default-editorto be removed beforehand. In this scenario the new build.sh fails with an error like
error: Could not depsolve transaction; 1 problem detected:
 Problem: conflicting requests
  - package nano-default-editor-7.2-2.fc38.noarch from @System conflicts with system-default-editor provided by vim-default-editor-2:9.0.1429-1.fc38.noarch from fedora
  - package vim-default-editor-2:9.0.1429-1.fc38.noarch from fedora conflicts with system-default-editor provided by nano-default-editor-7.2-2.fc38.noarch from @System
  - package nano-default-editor-7.2-2.fc38.noarch from @System conflicts with system-default-editor provided by vim-default-editor-2:9.0.1927-1.fc38.noarch from updates
  - package vim-default-editor-2:9.0.1927-1.fc38.noarch from updates conflicts with system-default-editor provided by nano-default-editor-7.2-2.fc38.noarch from @System
  - package nano-default-editor-7.2-2.fc38.noarch from @System conflicts with system-default-editor provided by vim-default-editor-2:9.0.1440-1.fc38.noarch from updates-archive

when previously it passed with output like
rpm-ostree override remove nano-default-editor --install=vim-default-editor

I believe there are scenarios in which the old build.sh handling of packages is needed. In my fork, I install vim-default-editor which requires the package nano-default-editorto be removed beforehand. In this scenario the new build.sh fails with an error like
rpm-ostree override remove nano-default-editor --install=vim-default-editor

Thanks for this comment @bpbeatty! I was thinking about this yesterday and realized we probably had a new problem, and you have described it perfectly.
I'll PR a revert, and think about how to better address my concern.


(LogManager): Same file processing information keeps being written to config.tlog infinitely.

Describe the bug
Same file processing information keeps being written to config.tlog infinitely. At the same time, kworker process occupied lots of CPU resources.
To Reproduce
Use LogManager to handle logs of greengrass and custom components.
Expected behavior
Same file processing information of LogManager should not be written to config.tlog repeatedly which seems to cause high CPU usage.
Actual behavior
The same currentComponentFileProcessingInformation keeps written to /greengrass/v2/config/config.tlog repeatedly in a very high rate.
{"TS":1666597838548,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","System","currentProcessingFileName"],"W":"timestampUpdated","V":null}
{"TS":1666597838548,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","System","currentProcessingFileStartPosition"],"W":"timestampUpdated","V":null}
{"TS":1666597838548,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","System","currentProcessingFileLastModified"],"W":"timestampUpdated","V":null}
{"TS":1666597838598,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","box_configuration","currentProcessingFileName"],"W":"timestampUpdated","V":null}
{"TS":1666597838598,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","box_configuration","currentProcessingFileStartPosition"],"W":"timestampUpdated","V":null}
{"TS":1666597838598,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","box_configuration","currentProcessingFileLastModified"],"W":"timestampUpdated","V":null}
{"TS":1666597838599,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","io.cognitohealth.DataGenerationV2","currentProcessingFileName"],"W":"timestampUpdated","V":null}
{"TS":1666597838599,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","io.cognitohealth.DataGenerationV2","currentProcessingFileStartPosition"],"W":"timestampUpdated","V":null}
{"TS":1666597838599,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","io.cognitohealth.DataGenerationV2","currentProcessingFileLastModified"],"W":"timestampUpdated","V":null}
{"TS":1666597838599,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","System","currentProcessingFileName"],"W":"timestampUpdated","V":null}
{"TS":1666597838599,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","System","currentProcessingFileStartPosition"],"W":"timestampUpdated","V":null}
{"TS":1666597838599,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","System","currentProcessingFileLastModified"],"W":"timestampUpdated","V":null}
{"TS":1666597838698,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","box_configuration","currentProcessingFileName"],"W":"timestampUpdated","V":null}
{"TS":1666597838698,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","box_configuration","currentProcessingFileStartPosition"],"W":"timestampUpdated","V":null}
{"TS":1666597838698,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","box_configuration","currentProcessingFileLastModified"],"W":"timestampUpdated","V":null}
{"TS":1666597838698,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","io.cognitohealth.DataGenerationV2","currentProcessingFileName"],"W":"timestampUpdated","V":null}
{"TS":1666597838698,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","io.cognitohealth.DataGenerationV2","currentProcessingFileStartPosition"],"W":"timestampUpdated","V":null}
{"TS":1666597838698,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","io.cognitohealth.DataGenerationV2","currentProcessingFileLastModified"],"W":"timestampUpdated","V":null}
{"TS":1666597838698,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","System","currentProcessingFileName"],"W":"timestampUpdated","V":null}
{"TS":1666597838698,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","System","currentProcessingFileStartPosition"],"W":"timestampUpdated","V":null}
{"TS":1666597838698,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","System","currentProcessingFileLastModified"],"W":"timestampUpdated","V":null}
{"TS":1666597838756,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","box_configuration","currentProcessingFileName"],"W":"timestampUpdated","V":null}
{"TS":1666597838756,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","box_configuration","currentProcessingFileStartPosition"],"W":"timestampUpdated","V":null}
{"TS":1666597838756,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","box_configuration","currentProcessingFileLastModified"],"W":"timestampUpdated","V":null}
{"TS":1666597838756,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","io.cognitohealth.DataGenerationV2","currentProcessingFileName"],"W":"timestampUpdated","V":null}
{"TS":1666597838756,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","io.cognitohealth.DataGenerationV2","currentProcessingFileStartPosition"],"W":"timestampUpdated","V":null}
{"TS":1666597838756,"TP":["services","aws.greengrass.LogManager","runtime","currentComponentFileProcessingInformation","io.cognitohealth.DataGenerationV2","currentProcessingFileLastModified"],"W":"timestampUpdated","V":null}

Output of top command:
top - 10:32:59 up 12 min,  1 user,  load average: 3.33, 3.35, 2.19
Tasks: 119 total,   1 running,  84 sleeping,   0 stopped,   0 zombie
%Cpu(s): 32.7 us, 62.1 sy,  0.0 ni,  0.0 id,  2.3 wa,  0.0 hi,  2.9 si,  0.0 st
KiB Mem :  1016304 total,     7424 free,   648096 used,   360784 buff/cache
KiB Swap:        0 total,        0 free,        0 used.   348336 avail Mem

  PID USER      PR  NI    VIRT    RES    SHR S %CPU %MEM     TIME+ COMMAND
  490 root       0 -20       0      0      0 I 43.5  0.0   2:46.83 kworker/0:4H
  886 root      20   0  322744  92888   8528 S 35.0  9.1   3:49.57 java
 1987 root      20   0   40720  25424   6800 S  6.0  2.5   0:24.73 python
  172 root      20   0       0      0      0 S  4.1  0.0   0:24.39 jbd2/mmcblk3p2-
 1813 ggc_user  20   0  286396  73936  12880 S  2.2  7.3   0:30.15 java
 1313 root      20   0   42304   5364   3516 S  1.9  0.5   0:13.76 chdemo
 2142 root      20   0    8520   2448   2048 R  0.9  0.2   0:00.10 top
    8 root      20   0       0      0      0 I  0.3  0.0   0:01.37 rcu_preempt
  331 root      20   0   27184     76      0 S  0.3  0.0   0:00.50 rngd
  361 root      20   0  949608  32912  14176 S  0.3  3.2   0:01.90 containerd
 1508 ggc_user  20   0  365004  38528  10508 S  0.3  3.8   0:07.88 java

Environment

OS: Debian 9
JDK version: OpenJDK8 (build 25.332-b09)
Nucleus version: 2.4.0
Log Manager version: 2.2.6

Additional context
It can't be solved by rebooting the device. The way to get rid of this situation is to remove LogManager and reboot device/restart greengrass. After that, the currentComponentFileProcessingInformation messages are stopped being written to config.tlog and kworker process will no longer occupy lots of CPU resources.

HI @adamchen-cognitohealth - can you please share your LogManager configuration? Does the issue reproduce with the latest versions? (Nucleus 2.8 + LogManager 2.2.7)

Hi @jbutler - thanks for your reply. Below is part of the configuration of LogManager, format of the remaining are the same as the part. I didn't use the latest versions. (Nucleus 2.8 + LogManager 2.2.7)
---
componentVersion: 2.2.6

configurationUpdate:
  merge:
    periodicUploadIntervalSec: "600"
    logsUploaderConfiguration:
      systemLogsConfiguration:
        diskSpaceLimit: "10"
        diskSpaceLimitUnit: MB
        uploadToCloudWatch: "true"
        logFileRegex: "greengrass\\w+.log"
        minimumLogLevel: WARN
        deleteLogFileAfterCloudUpload: "true"

      componentLogsConfigurationMap:
        box_configuration:
          minimumLogLevel: WARN
          deleteLogFileAfterCloudUpload: "false"
          logFileRegex: "box_configuration.log.*"
          logFileDirectoryPath: /greengrass/v2/logs

        data_generation_v2:
          minimumLogLevel: WARN
          deleteLogFileAfterCloudUpload: "false"
          logFileRegex: "data_generation_v2.log.*"
          logFileDirectoryPath: /greengrass/v2/logs

        io.cognitohealth.BoxConfiguration:
          diskSpaceLimit: "10"
          diskSpaceLimitUnit: MB
          minimumLogLevel: WARN
          deleteLogFileAfterCloudUpload: "true"

        io.cognitohealth.DataGenerationV2:
          diskSpaceLimit: "10"
          diskSpaceLimitUnit: MB
          minimumLogLevel: WARN
          deleteLogFileAfterCloudUpload: "true"

        aws.greengrass.StreamManager:
          diskSpaceLimit: "10"
          diskSpaceLimitUnit: MB
          minimumLogLevel: WARN
          deleteLogFileAfterCloudUpload: "true"

hi @adamchen-cognitohealth,
I have looked at the LogManager for several days, and could not reproduce this issue. But it seems like a real bug. Could you help us to reproduce it?

Hi @dgggIoT - thanks for looking at this issue.

Here is the greengrass log file, this issue happened during the time in the period of the log file. Sorry that I don't have the effectiveConfig.yml which includes LogManager because we removed LogManager temporarily. Do you need it without LogManager?

We started to observe this issue in September since we noticed that CPU usage of some devices is very high. I am unable to tell what changes we made before it, because it is not an obvious problem and we may make changes each day. I think this issue should happen before we start to observe it. I don't have specific steps for reproducing the issue.

Like the description of the second section, I am unable to provide last non-issue setups. There are many changes we made and it is not an obvious problem. It's hard to get the start point of this issue.

More description:
The severity(CPU usage of kworker) of issue to devices are different, and not occurred to all the devices with the same configuration. Seems the CPU usage is related to the speed of LogManager information written to /greengrass/v2/config/config.tlog.

Hi @adamchen-cognitohealth , please check our latest release of LogManager v2.3.0 and Nucleus 2.9.1. We have some suspects of the issue, and the latest release should solve it.

This issue is closed because the fix is out. You can re-open this issue if it still exist.


Feature | Add some cool page transitions

Needed page transitions, when a user visits from one page to another.
What to do:

Some kinds of animations from one page to another, such as sliding, fading, etc.
Can use any library or manually but make sure it looks great.
Do other necessary changes for a good and better UI.

I would like to try and contribute to this issue under JWOC.

Great @mridul4101, Go for it.

@imniladri ,Can I work on this issue under JWoC? I am not sure whether I can do it, but I can try my best.

Go for it @Srijita-Mandal, Try it out.


Fixes for autotools, compile warnings and MacOS compatibility

GOT_PREFIX is a direct inconvenience that Fedora and other distros disable by patching. Symlinks for sources are usually unnecessary, and I don't want to rely on bootstrap to adjust the source tree.

I don't have tested to see if it breaks something, but appears good, merging.


fix warning and background block problem

Signed-off-by: zhichenj<EMAIL_ADDRESS>What problem does this PR solve?

Fix warning about no return type.
Workaround to solve block problem for background cleaner.

Tests

Unit test
Integration test
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM
Consumes more PMEM

Signed-off-by: zhichenj<EMAIL_ADDRESS>What problem does this PR solve?

Fix warning about no return type.
Workaround to solve block problem for background cleaner.

Tests

Unit test
Integration test
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM
Consumes more PMEM

Signed-off-by: zhichenj<EMAIL_ADDRESS>What problem does this PR solve?

Fix warning about no return type.
Workaround to solve block problem for background cleaner.

Tests

Unit test
Integration test
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM
Consumes more PMEM

That's a horrible fix.
A bug fix caused KVDK to consumer more CPU, VMem and PMem?
Take some time to review your PR before actually submit.

Signed-off-by: zhichenj<EMAIL_ADDRESS>What problem does this PR solve?

Fix warning about no return type.
Workaround to solve block problem for background cleaner.

Tests

Unit test
Integration test
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM
Consumes more PMEM

Signed-off-by: zhichenj<EMAIL_ADDRESS>What problem does this PR solve?

Fix warning about no return type.
Workaround to solve block problem for background cleaner.

Tests

Unit test
Integration test
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM
Consumes more PMEM

That's a horrible fix.
A bug fix caused KVDK to consumer more CPU, VMem and PMem?
Take some time to review your PR before actually submit.


fix devDependencies

npm 上没有@ali/ali-lowcode-materials这个包

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Shuffle Image Generation Bug

without shuffle collection

with shuffle collection

so I picked up on a bug, and I think i know why it happens but not how to fix it

say i set collection to 24^^^^
it knows to start at 0 for both
it knows make 24 image files

it thinks it has to get to 23 (correct, no collection shuffle)
it thinks it has to get to 24 (error , yes collection shuffle)

repeated attempts yield same if not similar result

I fixed it......


Keyboard Selection Spec

Summary of the Pull Request
This introduces a spec for keyboard selection. This enables the user to create and update a selection without the use of a mouse or stylus.
References
Contributes to #715
Validation Steps Performed
N/A

When can we expect this PR to be merged? This is almost the last thing stopping me from using Terminal over ConEmu.

@carlos-zamora With #3391 about to merge, I'd want to see those changes reflected in this PR if possible :)

This is a bit annoying but I feel like Mark Mode is just big/confusing enough that it deserves its own space. So please check out #5804 if you want to have a closer discussion there.
That also means that the spec in its current state better fits #3758. So if you want to use the current implementation of this feature, you can build that branch.
Lingering question(s) that I have for anybody that comes along to review this:

Different/Cleaner name for the keybindings (and args)?

@DHowett changes from spec meeting have been applied

Does the spec include a way to trigger selection mode without selecting something first? I couldn't tell from the spec if I'm able to go to the "readonly" part of the console (output from former commands) and start the selection there, as shown in https://github.com/microsoft/terminal/issues/715#issuecomment-520145424

@MarkusAmshove you might also be interested in #5804, which discusses that exact scenario in greater detail ☺

Are there any other blockers for this?

Cool it, bot

Bump 🙂

Heck let's just merge this. #10824 is like close enough to what's laid out here and I'm not sure we need to really discuss more.

Discussed offline with @DHowett:

we should remove the customizable keybindings (unless requested later)
instead, handle this in ControlInteractivity. Benefit is that all consumers get it for free and they don't appear in the command palette. This is just how the terminal works.
If we get feedback during Preview that this isn't desired, we can reevaluate. I'd rather we don't have the option now, and add it in later, than have customizability now and remove it.

we should remove the customizable keybindings

Which ones? Would the defaults eventually still satisfy this use case? https://github.com/microsoft/terminal/issues/6528
I'm just trying to figure when I can finally switch!

we should remove the customizable keybindings

Which ones? Would the defaults eventually still satisfy this use case? #6528
I'm just trying to figure when I can finally switch!

@ofek The idea is to remove the updateSelection action and just have these work "naturally".
https://github.com/microsoft/terminal/blob/9a5c2d2ace5bc33e73be0d6deb3037d7677e9d5f/doc/specs/Keyboard-Selection.md#L91-L109
Hoping to fix up the spec and that other PR this week! 🙂

@DHowett We've discussed this like 80 times in team syncs since the first block - I'm 🤏 close to dismissing you since we've got a verbal approve

Alright, I'm dismissing @DHowett's review. We've talked about it in person a few times this past week, and the implementation includes those changes. We're having an internal bug bash soon and we still have some time before release, so we'll use that time to polish this feature if anything comes up.

I feel like we are missing some discussion about swapping the active selection endpoint, though I may have missed it(?); I would hope that it would cover what happens when the two endpoints are more than one screen apart (so, swapping would require jumping up and down).

I'll cover that in #5804. It'll fit better over there as a wholistic story of "how do we create a selection", whereas this one is more just "how do we update one we created with the mouse". It could honestly go in either but this spec is intended to cover the basics, and that's more of an advanced concept imo.

Call out a future concern around how keyboard selection may be slower than mouse selection

I don't actually think it'll be slower than mouse selection (or even slower enough to matter tbh). Mouse selection already relies on the same function calls on the text buffer:

move by char (wide glyphs) --> both need to check if we're on a wide glyph and update appropriately
move by word --> same text buffer calls (especially if you consider double-click + drag)
move by line/viewport --> just add/subtract the width of the viewport or clamp to the sides; should be a O(1)
move by buffer --> O(1); buffer's origin and mutable bottom are really quick-n-easy to get

The only difference I can think for mouse selection is that we convert the viewport position to a buffer position. Am I missing something here?

we will need to consider how to handle a text buffer that is changing while the user is selecting it. What if the top endpoint circles out of the buffer before the user locks the selection?

Isn't this something we already have to worry about (and handle) with mouse selection? I guess the only addition would be "if both endpoints moved off the buffer when the buffer circled, we need to ensure IsSelectionActive() is false"?

How does it interact with VT mouse mode (right now it is an impediment to starting a selection(!)) which also uses shift as a modifier?

I'm confused by your comments here. When in VT Mouse Mode, shift+click creates a selection. The bindings covered in this spec are used to alter the selection that was created. We're handling these key bindings exactly like we handle ESC when a selection is present (in any context): we alter the selection if one is present, otherwise we send the event to the app.

It may not be sufficient to say “keyboard selection is more accessible”; there are probably accessibility APIs we can plug into so folks don’t have to discover that they can use shift+arrows to move around.

The UIA APIs are already implemented, so yes, it's up to the screen reader to expose that. That's out of our control. However, having a more direct in-house solution is always preferable. We're bridging the gap between different experiences based on the user, and creating a consistent experience for all of our users.
As for discoverability, shift+arrows are already standard in Conhost. Windows users are pretty familiar with that experience. I guess this might be an issue for users who come from a different environment, but even text editors are consistent in having shift+arrows be how you select text.

I would suggest having key events (vkey, scancode, modifiers) go directly into ControlInteractivity and not have TermControl use _core.TryXxx (Sorry, you may have already done this). A clearer delineation in Interactivity about what we are capturing to handle “as part of a terminal control” versus what we are giving the core to process as a terminal-encoded output event would be helpful.

_core.TrySendKeyEvent() already exists. I think we talked a bit about moving that over to ControlInteractivity altogether, because key event handling makes sense to be near mouse event handling. That's an implementation detail imo though. Like, it's not really an issue until we complete the interactivity layer work right?

We never expect crashes… but you may want to mention the test collateral you’re going to write for this!

Isn't that out of scope for the spec? Like, can't you say that about just about any feature that gets specced?

@DHowett FYI, updated #5804 with some of the things I missed from your review. I think you'll find that to be a more interesting read. 😉

🎉🎉🎉


Add Dockerfile

Feel free to reject in case you do not want this added in the repository.

I got the following errors from my Pi (had to change the image to arm32v7/golang:1.13)
Any ideas?
Sending build context to Docker daemon  165.9kB
Step 1/6 : FROM arm32v7/golang:1.13
 ---> 19643ce633cc
Step 2/6 : RUN mkdir /app
 ---> Using cache
 ---> c8cecad304d4
Step 3/6 : ADD . /app/
 ---> Using cache
 ---> 46e05afa3757
Step 4/6 : WORKDIR /app
 ---> Using cache
 ---> 58db8088c4a0
Step 5/6 : RUN go build -o gramarr .
 ---> Running in ccb7a657596a
config.go:9:2: cannot find package "github.com/alcmoraes/gramarr/radarr" in any of:
	/usr/local/go/src/github.com/alcmoraes/gramarr/radarr (from $GOROOT)
	/go/src/github.com/alcmoraes/gramarr/radarr (from $GOPATH)
config.go:10:2: cannot find package "github.com/alcmoraes/gramarr/sonarr" in any of:
	/usr/local/go/src/github.com/alcmoraes/gramarr/sonarr (from $GOROOT)
	/go/src/github.com/alcmoraes/gramarr/sonarr (from $GOPATH)
conversation.go:7:2: cannot find package "github.com/patrickmn/go-cache" in any of:
	/usr/local/go/src/github.com/patrickmn/go-cache (from $GOROOT)
	/go/src/github.com/patrickmn/go-cache (from $GOPATH)
conversation.go:6:2: cannot find package "gopkg.in/tucnak/telebot.v2" in any of:
	/usr/local/go/src/gopkg.in/tucnak/telebot.v2 (from $GOROOT)
	/go/src/gopkg.in/tucnak/telebot.v2 (from $GOPATH)

The Dockerfile needs to run go get before go build will work.
$ go build -o gramarr .
config.go:9:2: cannot find package "github.com/alcmoraes/gramarr/radarr" in any of:
	/usr/local/Cellar/go/1.13.4/libexec/src/github.com/alcmoraes/gramarr/radarr (from $GOROOT)
	/Users/apetrovic/code/go/src/github.com/alcmoraes/gramarr/radarr (from $GOPATH)
config.go:10:2: cannot find package "github.com/alcmoraes/gramarr/sonarr" in any of:
	/usr/local/Cellar/go/1.13.4/libexec/src/github.com/alcmoraes/gramarr/sonarr (from $GOROOT)
	/Users/apetrovic/code/go/src/github.com/alcmoraes/gramarr/sonarr (from $GOPATH)
conversation.go:7:2: cannot find package "github.com/patrickmn/go-cache" in any of:
	/usr/local/Cellar/go/1.13.4/libexec/src/github.com/patrickmn/go-cache (from $GOROOT)
	/Users/apetrovic/code/go/src/github.com/patrickmn/go-cache (from $GOPATH)
conversation.go:6:2: cannot find package "gopkg.in/tucnak/telebot.v2" in any of:
	/usr/local/Cellar/go/1.13.4/libexec/src/gopkg.in/tucnak/telebot.v2 (from $GOROOT)
	/Users/apetrovic/code/go/src/gopkg.in/tucnak/telebot.v2 (from $GOPATH)
$ go get
$ go build -o gramarr .

Alternatively, if the project switches to using modules, then the build command will work.
I have sent up a PR to start using modules: https://github.com/alcmoraes/gramarr/pull/6

@joshghent Can you retry?

Thanks guys! Made a few changes in README as well, tested and its ready to go!


MobileTabItem can't include other view

I have a demo in https://github.com/j4587698/SukiUiTest
MainView only have a TabControl
  <TabControl Classes="MobileTabView">
      <TabItem Header="Test1" Classes="MobileTabItem">
          <views:TestView></views:TestView>
      </TabItem>
      <TabItem Header="Test2">
          <views:TestView></views:TestView>
      </TabItem>
  </TabControl>

TestView only have a string.
Run it
Test1 page is a empty page

Test2 page can show the string

The text disappears because of the ScaleTransform effect for the header of the TabItem that is applied (should not) on the Content too when the Tab is selected. Your TestView is in fact scaled *1.4 :(
So I will make a new nuget package monday with the ScaleTransform removed, and maybe with another design when a mobile tab is selected.

Solved in 4.5.5


Support for ActiveRecord 6

Since Rails is now in beta2, it seems like a good time to add support. In the best case scenario, this involves just loosening the dependency. 😬

Opened #13 for this

released v0.1.5

Thank you for the response, @betesh. Looking forward to continuing using this with Rails 6. ❤


How to add product_appearance attribute to the basic information cluster (CON-1248)

Hi,
I wanted to add to my product the informations about color and finish, I found out this is possible in the attribute "product_appearance" of the cluster "basic information".
In my case the root node is created with the cluster basic information but the only attributes are node_label and cluster_revision, I know that using the function create_product_appearance it's possible to create the attribute.

I couldn't fins an example where this function is used, and in the documentation is not clear what parameters I have to provide in "value", "length" and "count".
What parameters do I have to provide if I want to display color: white and finish: satin when in the chip tool i use the command "./chip_tool basicinformation read product-appearance 1 0"?
Thanks

@FMistrorigo , thanks for reporting the issue. The product appearance attribute can be set by creating a factory_partiton using esp-matter-mfg-tool. It is read using the DeviceInstanceInfoProvider implementation. We will add the support for the attribute soon.

@FMistrorigo ,the change is now present in connectedhomeip here.
The product-appearanace attribute is read using the platform specific DeviceInstanceInfoProvider  implementation
Till we update the submodule, you can apply do these changes accordingly in your connectedhomeip submodule and try out the product-appearance attribute in esp-matter.
For trying this out, you need to create factory partition containing the product-finish and product-color attributes.
For e.g:
  ./generate_esp32_chip_factory_bin.py -d 3434 -p 99663300 \
                                    --product-name ESP-lighting-app --product-id 0x8001 \
                                    --vendor-name Test-vendor --vendor-id 0xFFF2 \
                                    --hw-ver 1 --hw-ver-str DevKit  --product-finish polished --product-color navy

You can try out the factory script present here with the changes in the above PR for generating the factory partition.
You will have to enable the following options in the menuconfig for testing out the prodcut-appearance attribute.
CONFIG_ENABLE_ESP32_FACTORY_DATA_PROVIDER=y
CONFIG_ENABLE_ESP32_DEVICE_INSTANCE_INFO_PROVIDER=y

For more information on factory data, you can refer this link.

@FMistrorigo , any updates on the issue.

Hi @shripad621git,
I didn't want to create a Factory partition only for these attributes, for the Device Instance Info Provider I use a custom one and I tried to implement the two functions in this way, so when the get functions were called the output was what I wanted, but it didn't work.

I decided to go back to the function create_product_appearance and did this:

(0x02 is Satin, and 0x0E is White)
Everything was good in the chip tool was working so I left it like this.
Tell me if ther's any problem in what I did.
Thanks.

@FMistrorigo , the above changes LGTM if you don't want to use factory partition.


deleteAppUserForCurrentApplication mutation

Essentially identical to https://github.com/parameter1/identity-x/pull/38 except allows for either a userId or email to be submitted along with the applicationId in order to delete the specified user record should it exist.

 IdentityX API Enhancements


Launching main.py with --dmtet flag fails with an error

Description
I tried to fine-tune the dreamfusion mesh with the dmtet but the script fails miserably with an error
[F glutil.cpp:338] eglInitialize() failed Aborted (core dumped)
Steps to Reproduce

Build docker from dockerfile
Train dreamfusion python3 main.py --text "fantasy character dwarf, full height, realistic, high quality" --workspace trial -O --vram_O
Run dmtet fine-tuning python3 main.py -O --text "fantasy character dwarf, full height, realistic, high quality" --workspace trial_dmtet --dmtet --iters 5000 --init_ckpt trial/checkpoints/df.pth
Get an error

Expected Behavior
Finetuned with dmtet mesh
Environment
Ubuntu 20.04.5 LTS, PyTorch 1.12, CUDA 11.6

@SavvaI Hi, please check https://github.com/ashawkey/stable-dreamfusion#trouble-shooting.

@SavvaI Hi, please check https://github.com/ashawkey/stable-dreamfusion#trouble-shooting.

I am using Nvidia-docker as specified above, so I tried to as suggested in https://github.com/ashawkey/stable-dreamfusion/issues/131#issuecomment-1414727699 but it did not work

Solution
Go to ./nerf/renderer.py
Replace  dr.RasterizeGLContext  to  dr.RasterizeCudaContext
It work well!!
So does the same problem in GET3D and nvdiffrec.

Solution
Go to ./nerf/renderer.py Replace dr.RasterizeGLContext to dr.RasterizeCudaContext It work well!! So does the same problem in GET3D and nvdiffrec.

Thank you it helps!

But after solving the described above issue now it fails while trying to load the mean_density from the checkpoint.
https://github.com/ashawkey/stable-dreamfusion/blob/3bd3c085afcf97c0a044e77088eee735076228b5/main.py#L148
The problem is, vanilla dreamfusion (without --dmtet) did not save any "mean_density" field in the checkpoint.

But after solving the described above issue now it fails while trying to load the mean_density from the checkpoint.
https://github.com/ashawkey/stable-dreamfusion/blob/3bd3c085afcf97c0a044e77088eee735076228b5/main.py#L148
The problem is, vanilla dreamfusion (without --dmtet) did not save any "mean_density" field in the checkpoint.

Ok I get it. The reason is that I had to remove --cuda_ray flag from the python3 main.py --text "fantasy character dwarf, full height, realistic, high quality" --workspace trial -O --vram_O because after the recent commit it stopped working and now it gives an error. Before the last commit it worked fine.

What you saying is remove --cuda_ray from this code parser.add_argument('-O', action='store_true', help="equals --fp16 --cuda_ray --dir_text") ?

What you saying is remove --cuda_ray from this code parser.add_argument('-O', action='store_true', help="equals --fp16 --cuda_ray --dir_text") ?


Swiss QR-Bill Encode procedure AddPartyInfo Country empty

App: QR-Bill Management for Switzerland
In the Swiss QR-Bill Encode codeunit, the procedure AddPartyInfo(var TargetText: Text; Customer: Record Customer; AddressType: Enum "Swiss QR-Bill Address Type") adds a empty Country/Region Code if the customer is from the local country.
That field must be filled out. So we either need a event to fill it out ourself or you use the value from the CompanyInformation table.
What would be the better option?

Hello! We have not received any support requests that this module is not working properly. We also use blank country code for local country in many places in application code, so probably this is not an issue. If you have any business scenario or repro steps to confirm this is an issue, you can create support request. Please also be aware that this is not an extensibility request and our team cannot process it.

That is definitely a issue, as the SIX Group (which defines the specifications for the QR-Bill) specifies that the country code must be filled. But then I will open a support request on mbs, correct?

This procedure contains last line to add Customer "Country/Region Code".
AddLine(TargetText, "Country/Region Code");

Correct, but as you said, the Country/Region Code is always empty for the local country. And in my opinion the Swiss QR-Bill App should build the correct qr code, so it should check for empty Country/Region Code.
But I will open a support request for that, if this is the wrong place for that.


Added commitish as input

The original implementation can be found here.
The fork of Zendesk somehow removed it. We want to have it again

See comments. Additional to this we should extend the README with the new capability 🥳

Thanks for the review. Please see my comment in the PR in the original repo here. The testing currently is broken and would need a revamp but the code would need to be changed as well to make it testable. I only made the tests green tbh

I extended the README.md 👍

After exploring the inner workings of Jest, I discovered that we are providing getInput as an argument in our function call and anticipating the correct sequence, which can be confusing and difficult to interpret.
However, I learned that Jest offers a helpful solution through its spyOn function. This approach can help us write more comprehensible tests in the future.
You can find more information on this here
If the tests are unreliable, it's important to question their value. We should consider removing them if they aren't providing the accuracy and consistency that we need.

Created a ticket for refactoring the test to unblock your current task.


Preparing for release. Stable 3rd party dependencies + stable/latest uploads

Needed a bit of a trick to get the correct upload CHANNEL configured.
All 3rd party dependencies will now use the stable channel. The remote has been changed in the conan_config repo.
New behavior is:

PRs are not uploaded anywhere (this is same as before)
Merges to master/dev will get uploaded to includeos/latest.
Tags/releases will upload to includeos/stable.

S2N is not version 1.1.1 its a dummy to make things work but its not based on an official TAG to say its stable is imho a bit wrong. but i am open to discuss how to handle it

@KristianJerpetjon I've changed to the 0.8 package that was uploaded last friday. I've also added the python_requires functionality to this pr.


Improved logging in Worker class. Worker class shouldn't produce more…

… than 200 log messages when performing inversions, and the interval between should not be less than 100 inversions. Logging should cycle through the different workers (so worker 0 logs first, then worker 1 when the next log message is needed, etc).

First off, thanks for submitting a PR!
Now that ray updates are complete and we've passed checks, I've taken a look at this in detail.  While it looks like a reasonable strategy for the case where you want a limited log, it's also quite reasonable to want significantly more output (IE, if we're inverting a million spectra and want to be sure things are continuously progressing) - and I might even go so far as to advocate for this as the default.
Proposal: rather than implementing a hard logging rate cap at 200 lines (and limiting subsequent logging to this level), we add a configuration element to configs/sections/implementation called logging_interval, which we default to 100.  If set at -1, we utilize the approach as described, otherwise we use the prescribed rate.  Provided we propagate the option to apply_oe, this provides all desired features.
Thoughts @quarkpt or others?  I'm happy to implement directly to the initiated branch if approved.

I would recommend that the default be the -1 setting (i.e., no more than
200 log messages in the inverter, inverter log messages no closer together
than every 100 inversions), with the user able to increase the messaging
rate. Rationale:
IMO, the logging scheme I implemented is the better one for most users most
of the time, in that the log messages from the inverting step don't wind up
dominating the log file. I think this is desirable because I suspect from
my own experience that the inverter is either "all or nothing," i.e.,
either it works every single time (in which case the volume of log messages
provide no value), or it fails immediately (in which case a log message
explaining the failure is needed but the volume of log messages is
irrelevant because there's only going to be approximately 1). My proposal
also avoids the situation in which every single worker logs at every single
logging interval which, again, is for most use-cases mere duplication and
adds no insight.
I didn't make my approach to logging configurable because in my experience
adding options that control non-algorithmic behavior (things like logging
density) can really clutter an implementation without providing
commensurate benefits to the user. In these cases I prefer to find an
implementation that is more or less correct (as I believe mine to be), and
then just make the code use that implementation all the time.
-PT
On Mon, Nov 22, 2021 at 5:54 PM Phil Brodrick @.***>
wrote:

First off, thanks for submitting a PR!
Now that ray updates are complete and we've passed checks, I've taken a
look at this in detail. While it looks like a reasonable strategy for the
case where you want a limited log, it's also quite reasonable to want
significantly more output (IE, if we're inverting a million spectra and
want to be sure things are continuously progressing) - and I might even go
so far as to advocate for this as the default.
Proposal: rather than implementing a hard logging rate cap at 200 lines
(and limiting subsequent logging to this level), we add a configuration
element to configs/sections/implementation called logging_interval, which
we default to 100. If set at -1, we utilize the approach as described,
otherwise we use the prescribed rate. Provided we propagate the option to
apply_oe, this provides all desired features.
Thoughts @quarkpt https://github.com/quarkpt or others? I'm happy to
implement directly to the initiated branch if approved.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/isofit/isofit/pull/253#issuecomment-976097538, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBA63UFMLDOUX47M6VTWT3UNLX5VANCNFSM5HUEDSNQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

If everything's working smoothly, I think this implementation is totally reasonable.  I'm mostly nervous using it as a default given that I end up using the current logs in consultation with lots of users when debugging (particularly for their particular hardware setups).
Quick question - is the issue just the verbosity of the log when examining after a run completes?  If so, a mutually agreeable solution would be to just add a prefix to the logging file during the inversion reports, and then completely clear them out of the log after the fact...since they're just there to report progress (/handle debugging), and we provide summary information anyway.  I'd definitely agree this would be nice to post-hoc examination of files.

The verbosity is the issue, but at a higher level is the question of how
best to make the log files useful, something I'm still struggling with on
my day jobs with TESS and SBG. It seems to me that for most users, most of
the time, an occasional log message is best (so you know that the
processing hasn't gone totally out to lunch), and increasing the verbosity
is something you do deliberately when there's a problem. You shouldn't let
the "occasional problem" use-case be the one that drives the logging
configuration. Over in our Java code we do this by having a bunch of
messages that are only written if a debug configuration parameter is set
for the logger. If there's a problem we re-run the problematic code with
the debug flag set (or ask the user to do so).
Anyway, you do know far better than I do what you want to do with the log
messages, etc. It just seems like a default behavior of producing an
enormous number of messages that say "everything's fine here" is not the
right answer.
-PT
On Mon, Nov 22, 2021 at 6:26 PM Phil Brodrick @.***>
wrote:

If everything's working smoothly, I think this implementation is totally
reasonable. I'm mostly nervous using it as a default given that I end up
using the current logs in consultation with lots of users when debugging
(particularly for their particular hardware setups).
Quick question - is the issue just the verbosity of the log when examining
after a run completes? If so, a mutually agreeable solution would be to
just add a prefix to the logging file during the inversion reports, and
then completely clear them out of the log after the fact...since they're
just there to report progress (/handle debugging), and we provide summary
information anyway. I'd definitely agree this would be nice to post-hoc
examination of files.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/isofit/isofit/pull/253#issuecomment-976107513, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBA63SJJYLCEN57TTMQOPTUNL3UNANCNFSM5HUEDSNQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Obviated by developing codebase and new issue #349.


add image_data

Part of https://issues.redhat.com/browse/APPSRE-3925
Design doc: https://gitlab.cee.redhat.com/service/app-interface/-/merge_requests/34670
Signed-off-by: Feng Huang<EMAIL_ADDRESS>

Can you add documentation using description field to all the new schema elements?


Answering inbound call by foregrounding app from background does not connect

Repro steps:

launch app on iOS
background it
on Web, call user on iOS
foreground iOS app
click answer on inbound call alert dialog
iOS sits as if it were on a call (no media)
Desktop continues to ring, with no evidence of the call having connected

Moving to iOS10 CallKit would help with a lot of the background answering issues in Riot.

@manuroe I can't reproduce this


chore: remove sync data access from a few places

Gotta make sure to test this "sideloading" case in app-show on real data:


3.1 使用webpack构建时提示依赖rollup-plugin-babel

我只引入了cropper\dist\cropper.css，并不需要使用rollup-plugin-babel，但由于cropper文件夹下面有postcss.config.js，postcss-loader会自动去读取，导致构建时提示依赖rollup-plugin-babel。

Also having this error after update to latest version from 2.3.4: Error: Cannot find module 'rollup-plugin-babel. It's when I execute webpack --progress --watch command.

Please install with npm install cropperjs which will not include the postcss.config.js.

@fengyuanchen 就是用npm安装的

@fengyuanchen please provide more info.
Both packages (cropper and cropperjs) include postcss.config.js. However, I removed cropper from disk and installed it again with npm install cropper. It's the same.
May it be something with npm&node versions? I'm using node v6.10.2, npm 3.10.10.

Try to upgrade NPM first:
npm i -g npm

Upgraded to 5.5.1. Didn't help. Maybe webpack upgrade can help. I'll try it in a few days. Can't do it right now because webpack update will broke something for sure.
I didn't mention this, but even with this error, cropper works just fine. It's just irritably when I see that error message.

我就说了有这个问题嘛，作者还说我网络问题，我单独起一个目录去安装 还是这样 依赖没有自动安装

@all Please update to v3.1.2 which has fixed this.


Label community PRs

When new PRs are opened, if they are not from bots or from users with admin or write permissions, then they get labeled as community contributions.
Microsoft Reviewers: Open in CodeFlow

@DamianEdwards we have this automation in repos like dotnet/runtime and it is helpful. Let me know what you think.


[Bindable] tag doesn't work

"Unkown error" appears if trying to compile class with the [Bindable] field
[Bindable(event="evantName")] works fine

It's strange but [Bindable] works fine in the class extended EventDispatcher
FeathersControl extended EventDispatcher, but in class extended FeathersControl, [Bindable] doesn't work

I'm not able to reproduce this issue. The following code is working correctly for me:
package
{
	import feathers.core.FeathersControl;

	public class CustomComponent extends FeathersControl
	{
		public function CustomComponent()
		{
			super();
		}

		[Bindable]
		public var bindableVar:String;
	}
}

I also tried this, and it worked correctly too:
package
{
	import feathers.core.FeathersControl;

	[Bindable]
	public class CustomComponent extends FeathersControl
	{
		public function CustomComponent()
		{
			super();
		}

		public var bindableVar:String;
	}
}

Hey Josh!
Sorry to contact you like this, since this has nothing to do with the issue, but I didn't know how else to get in touch with you. So my question: Which framework do you recommend to use with Feathers/Starling? Is the Robotlegs port for Starling (https://github.com/joshtynjala/robotlegs-framework-starling) still active, or is there something better?
Thanks!
Peter

@OregonPete I just used Robotlegs to build the Feathers SDK Manager. Still works great!

Agreed! I am using that port in some large applications of mine for Android and it works perfect.

Thanks, guys! I appreciate it!

Ok, i see , the problem is in the subclass of extended class, for example if you create CustomComponent2 extended CustomComponent , [Bindable] will not work in  CustomComponent2
Error:(6, 18) [FeathersTest]: Class TestComponent2 must extend the required base class starling.events.EventDispatcher.

Thanks. I can reproduce now.