SupportsNoItemContext code examples should show diffs

The code examples at https://learn.microsoft.com/en-us/office/dev/add-ins/outlook/contextless expect the user to have just run through some outlook quick start example, making it hard to know how to use the feature in existing projects. It would be much more helpful if the code examples showed the diff required to enable the feature.

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

ID: 6c1ca1e3-adbb-5cb7-93fb-09b258a101b8
Version Independent ID: 6c1ca1e3-adbb-5cb7-93fb-09b258a101b8
Content: Activate your Outlook add-in without the Reading Pane enabled or a message selected - Office Add-ins
Content Source: docs/outlook/contextless.md
Service: outlook
Sub-service: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

@unhammer Thanks for letting us know about this.
@samantharamon Can you take a look?
Thanks.

Hi @unhammer,
Thanks for your feedback on this article.
Although the walkthrough uses the Outlook quick start as a base project, the key features needed are highlighted in the 'Configure the manifest' section. To your point, however, let me add a note somewhere in the article to provide further guidance for folks with existing projects.

Well, I've added <SupportsNoItemContext>true</SupportsNoItemContext> under
     <Hosts>
       <Host xsi:type="MailHost">
         <DesktopFormFactor>
          <ExtensionPoint xsi:type="MessageReadCommandSurface">
            <OfficeTab id="TabDefault">
              <Group id="msgReadGroup">
                <Control xsi:type="Button" id="msgReadOpenPaneButton">
                  <Action xsi:type="ShowTaskpane">

and the button is still greyed out when no message is selected. What else is required for it not to be greyed out?

I finally got it working by removing the contents of the 1.0 versionoverride. I originally had
  <VersionOverrides xmlns="http://schemas.microsoft.com/office/mailappversionoverrides" xsi:type="VersionOverridesV1_0">
      <Requirements>…</Requirements>
      <Hosts>…</Hosts>
      <Resources>…</Resources>
      <VersionOverrides xmlns="http://schemas.microsoft.com/office/mailappversionoverrides/1.1" xsi:type="VersionOverridesV1_1">
          <Requirements>…</Requirements>
          <Hosts>…</Hosts>
          <Resources>…</Resources>
      </VersionOverrides>
  </VersionOverrides>

Then I changed it to
  <VersionOverrides xmlns="http://schemas.microsoft.com/office/mailappversionoverrides" xsi:type="VersionOverridesV1_0">
      <VersionOverrides xmlns="http://schemas.microsoft.com/office/mailappversionoverrides/1.1" xsi:type="VersionOverridesV1_1">
          <Requirements>…</Requirements>
          <Hosts>…</Hosts>
          <Resources>…</Resources>
      </VersionOverrides>
  </VersionOverrides>

and the addon can now show without a message selected.
However, it no longer shows in outlook for web, only on desktop. I'm not sure I understand https://learn.microsoft.com/en-us/javascript/api/manifest/versionoverrides?view=common-js-preview#namespace-values – is there a full example of a manifest.xml somewhere that enables SupportsNoItemContext for desktop while still letting one use the addin without that feature on web?

Thanks for the update, @unhammer. I'm glad to hear you were able to get it to work in the Outlook desktop client. I've made a note to make sure that we clearly specify that VersionOverrides 1.1 is required in the walkthrough.
Regarding the add-in not showing up in Outlook on the web, due to caching, it may take up to four hours for a sideloaded add-in to appear if any changes were made to it. Did the add-in eventually reappear for you or are you still experiencing issues?

I did eventually get it to reappear, yes. I found out the hard way that I just need to wait a few hours :)
Similarly when removing the add-on when installed through appsource, it took some time for it to disappear from desktop Outlook. I don't suppose there's a "clear cache" button somewhere?

I'm glad to hear that the add-in eventually reappeared in Outlook on the web. There isn't a Clear Cache button available, however, you can manually clear the cache per Clear the Office cache.

No worries. I created #4207 to integrate the doc feedback you provided. Once the changes are merged, this issue will automatically close, but please don't hesitate to create another issue if you need additional assistance.
Thank you again for your feedback to help improve our documentation!


Add "antctl get bgppeers" agent command

Add antctl get bgppeers agent command to print current status of all BGP peers
of effective BGP policy applied on the local Node.
For https://github.com/antrea-io/antrea/issues/6209

/test-all

/test-conformance

/test-kind-conformance


Request: Handle Okta user password expiration

In addition to the user password expiration the other user status values shown in the Okta documentation should be handled.
Expected Behavior
When a user's Okta password is expired gimme-aws-creds should print out a friendly error message without a large stack trace.
A message similar to the LOCKED_OUT message https://github.com/Nike-Inc/gimme-aws-creds/blob/master/gimme_aws_creds/okta.py#L302 would be preferred.
Current Behavior
Currently a large stack trace is printed with the RuntimeError of Unknown login status.
Example:
Traceback (most recent call last):
  File "/usr/local/bin/gimme-aws-creds", line 17, in <module>
    GimmeAWSCreds().run()
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/main.py", line 468, in run
    self._run()
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/main.py", line 795, in _run
    for data in self.iter_selected_aws_credentials():
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/main.py", line 771, in iter_selected_aws_credentials
    for role in self.aws_selected_roles:
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/main.py", line 671, in aws_selected_roles
    selected_roles = self._get_selected_roles(self.requested_roles, self.aws_roles)
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/main.py", line 662, in aws_roles
    self.saml_data['SAMLResponse'],
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/main.py", line 653, in saml_data
    self._cache['saml_data'] = saml_data = self.okta.get_saml_response(self.aws_app['links']['appLink'])
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/main.py", line 646, in aws_app
    self._cache['aws_app'] = aws_app = self._get_selected_app(self.conf_dict.get('aws_appname'), self.aws_results)
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/main.py", line 599, in aws_results
    self.auth_session
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/main.py", line 581, in auth_session
    auth_result = self.okta.auth_session()
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/okta.py", line 158, in auth_session
    login_response = self.auth()
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/okta.py", line 151, in auth
    flow_state = self._next_login_step(
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds/okta.py", line 317, in _next_login_step
    raise RuntimeError('Unknown login status: ' + status)
RuntimeError: Unknown login status: PASSWORD_EXPIRED

Possible Solution
Add the following lines of code after https://github.com/Nike-Inc/gimme-aws-creds/blob/master/gimme_aws_creds/okta.py#L302
        elif status == 'PASSWORD_EXPIRED':
            raise errors.GimmeAWSCredsError("Your Okta password has expired.", 2)

Steps to Reproduce (for bugs)

Wait for Okta password to expire.
Attempt to get AWS creds
View Unknown login status RuntimeError

Context
Okta passwords expire periodically and the large stack trace is confusing to some users.
Your Environment

App Version used: 2.3.4
Operating System and version:  macOS 10.15.5

Please consider adding this. I run into the same exact problem and have to go here in order to know what was going on


Set mirror for tests

This PR is a follow-up of https://github.com/gradle/gradle/pull/6129 . It reads the system properties passed by the main build and use them.

This broke the Travis build: https://travis-ci.org/gradle/kotlin-relocation-test/builds/410138893#L1164-L1177

Thanks @lptr . I think we should use info log level in travis configuration file, otherwise it's hard to find out the real problem.

I wonder if we should tell Travis CI to enable --info by default for Gradle builds... This is probably not the forum to discuss that, though.


Can't even Download the Mod

The Title says it,
Because Mekanism does not have Mirror Downloads or is not on Curse in case the Normal Downloads don't work it makes it quite hard to get it in case that one Download Option falls Flat.
For some reason i can't Download them, the adf.ly link just sits there (Site is Empty as Always) until i can Skip to the Download, but when i click it a new Tab opens up and Closes again Instantly.
Nothing Happens except that.

You can:
A) Download a modpack like Hermitpack, which I think has the current Mekanism version.
B) Copy the links from the download page and remove the adf.ly redirect part.

B worked. thanks.
(still wonder why it isn't on Curse Directly tho)

Mekanism has adfly links different from most I see. When you click on the mod to download it brings you to a URL like http://adf.ly/1140958/dl.aidancbrady.com/download?u=http%3A%2F%2Faidancbrady.com%2Fwp-content%2Fuploads%2Fmekanism%2F295-recommended%2FMekanism-1.10.2-<IP_ADDRESS>5.jar&wiki=http%3A%2F%2Fwiki.aidancbrady.com
All you have to do is delete the http://adf.ly/1140958/ from the URL and the remaining link will bring you to the actual download.

uhh, yes.
maybe you missed the Part where i said that Option B worked.
i have the mod already.

The mod is not on Curse yet because Aiden wanted to get a few more bug releases out before he uploaded there.
He then got busy with University/College and hasn't been able to update the mod very much.


[feature] #3383: Implement a macro that parses a socket address at compile time

Linked issue
Closes #3383
Limitations
This can't replace all uses of the existing socket_addr! macro, as it parses only literal addresses. Some of the socket_addr! usages use an expression as port number: https://github.com/hyperledger/iroha/blob/19984d91c2add850a197050e6cbee52af1bc6fa2/core/test_network/src/lib.rs#L598. Supporting this is possible, but would require a more complicated parser.
As 5/11 usages of the original macro use it, it probably doesn't make sense to merge it without support for variable ports
It also doesn't support hostname addresses, but they are not used anywhere except in the doc-test for the socket_addr! macro. Probably fine going without it, but should be relatively easy to implement if needed.
Checklist

[x] I've read CONTRIBUTING.md
[x] I've used the standard signed-off commit format (or will squash just before merging)
[ ] All applicable CI checks pass (or I promised to make them pass later)
[x] (optional) I've written unit tests for the code changes
[ ] I replied to all comments after code review, marking all implemented changes with thumbs up

In PR description you say that custom port is not supported. But by looking at the code it looks like it is supported, no?

I forgot the link to an example..
https://github.com/hyperledger/iroha/blob/19984d91c2add850a197050e6cbee52af1bc6fa2/core/test_network/src/lib.rs#L598
Basically, it uses an arbirary expression instead of an integer literal for port. This won't work with the "stringify and parse it" approach

So what blocks you to undraft this PR?

So what blocks you to undraft this PR?

Using expressions as port numbers, which is done a lot throughout the codebase. Will probably tackle this tomorrow

Also, please make meaningful checklist in the PR's description (the current one is template for external contributors, AFAIC). It might make sense to move some TODOs from your description/code there.

I wonder, which syntax would you use for expressions interpolation? I have a couple of ideas:

I think it's possible to parse the natural syntax: socket_addr!(<IP_ADDRESS>:port). Everything before the colon token is converted to string and parsed as an IP address, while everything after the colon would be parsed as an expression.

Will you replace all usages of the old macro?

Yes, this is what I am planning to do

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

183 of 195 (93.85%)  changed or added relevant lines in 3 files are covered.
312 unchanged lines in 9 files lost coverage.
Overall coverage increased (+0.1%) to 59.532%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/test_network/src/lib.rs
0
4
0.0%

primitives/derive/src/socket_addr.rs
166
174
95.4%

Files with Coverage Reduction
New Missed Lines
%

data_model/src/transaction.rs
1
47.66%

wasm_codec/derive/src/lib.rs
1
90.71%

tools/kagami/src/main.rs
3
2.26%

cli/derive/src/lib.rs
5
93.2%

cli/src/samples.rs
5
0%

config/src/client.rs
13
70.42%

data_model/src/block.rs
30
42.21%

wasm_builder/src/lib.rs
95
41.67%

core/src/smartcontracts/wasm.rs
159
64.44%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
19889

Relevant Lines:
33409

💛  - Coveralls

rustfmt wants socket_addr!(<IP_ADDRESS>: port); to be formatted as socket_addr!(127.0.0 .1: port);...
There is an option to exclude a certain macro invocations from being formatted, but

the config option parsing seems broken https://github.com/rust-lang/rustfmt/issues/5816
the toolchain used in CI doesn't support it

I guess we would have to live with this ugly formatting for some time?


DatePicker v1.0.0

Hello,
I would like to ask why datepicker has been removed in v.1-beta branch? It is available in 0.19.4

@leebake It wasn't removed, it was simply deprioritized, please follow #4787.

So, since v.1 all components have been rewriting?


Uploading multiple apks to Crashlytics doesn't work

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
I have multiple flavors in my app and a setup similar to this. When I run fastlane beta it runs assembleRelease task and builds 10 apks. However, it doesn't upload them to Crashlytics. In the output, it says that upload to Crashlytics is successful, but I don't see it in the portal. Is there any other configuration I need to do or is this a bug? I would expect it to upload all generated apks.
lane :beta do
  # ...
  gradle(
    task: 'assemble',
    build_type: 'Release'
  )

  crashlytics(
    api_token: '[insert_key_here]',
    build_secret: '[insert_secret_here]'
  )
  # ...
end

@GokhanArik i think the crashlytics action only works for single APKS - can you do an iteration over your results, and calling crashlytics multiple times?

@hjanuschka  How would I do that? I'm not familiar with Fastlane. One option I thought was to get the list of flavors I have and assemble them separately, but I don't know how to get the list of build flavors in Fastlane.
I looked at the source code and Crashlytics developer tools take -uploadDist for APK path and as you said it is for single APK only. When I provide directory, it doesn't work. Knowing that supply takes multiple APKS, I thought Crashlytics could do the same thing.

tbh. i don't have a android project here to test (could you provide a sample?)
but i think it should be something like:
lane :beta do
  # ...
  gradle(
    task: 'assemble',
    build_type: 'Release'
  )

  lane_context[SharedValues::GRADLE_ALL_APK_OUTPUT_PATHS].each do | apk |
    crashlytics(
      api_token: '[insert_key_here]',
      build_secret: '[insert_secret_here]',
      apk_path: apk
    )
  end
  # ...
end

untested, unofficial

@GokhanArik Crashlytics doesn't support uploading multiple apks at once. @hjanuschka's idea might be a good solution 👍

I wasn't aware of lane_context, thanks, @hjanuschka and @asfalcone.
It is working pretty well now. I will post structure of what I have, in case someone needs it.
lane :beta do |options|
    # Clean and build the Release version of the app.
    # Usage `fastlane beta app:FLAVOR` or `fastlane beta` to build and deploy all

    gradle(task: "clean")
    gradle(task: "assemble",
           build_type: "Release",
           flavor: options[:app])

    # If user calls `fastlane beta` command, it will build all projects and push them to Crashlytics
    if options[:app].nil?
        lane_context[SharedValues::GRADLE_ALL_APK_OUTPUT_PATHS].each do | apk |

            puts "Uploading APK to Crashlytics: " + apk

            begin
                crashlytics(
                  api_token: [apiToken],
                  build_secret: [buildSecret],
                  apk_path: apk
                )

                slack(
                  message: "Successfully deployed new build for #{/([^\/]*)$/.match(apk)}",
                  success: true,
                  default_payloads: [:git_branch, :lane, :test_result]
                )
            rescue => ex
                # If project is inactive in Crashlytics, deployment will fail. Handle it here and report to slack
                slack(
                    message: "Error uploading => #{/([^\/]*)$/.match(apk)} : #{ex}",
                    success: false,
                    default_payloads: [:git_branch, :lane, :test_result]
                )
            end
        end

        after_all do |lane|
            # This block is called, only if the executed lane was successful
            slack(
                message: "Deployment is completed for #{lane_context[SharedValues::GRADLE_ALL_APK_OUTPUT_PATHS].size} app(s)",
                default_payloads: [:git_branch, :lane, :test_result],
                success: true
            )
        end
    else
        # Single apk upload to Beta by Crashlytics
        crashlytics(
            api_token: [apiToken],
            build_secret: [buildSecret],
            notifications: "true"
        )

        after_all do |lane|
            # This block is called, only if the executed lane was successful
            slack(
                message: "Successfully deployed new build for #{options[:app]}",
                default_payloads: [:git_branch, :lane, :test_result],
                success: true
            )
        end
    end

    error do |lane, exception|
        slack(
            message: exception.message,
            success: false,
            default_payloads: [:git_branch, :lane, :test_result]
        )
    end
end


Feature/mobile view

Fix mobile issues
past-event section is still pending
past-event section over to @Safnaj

Also need the clean the code

@ebonynon make sure u always get the latest changes before u commit. Pls share a screenshot or video

@ebonynon make sure u always get the latest changes before u commit. Pls share a screenshot or video

I'm done. Now you can merge
cc: @Shehanka

@Shehanka can u quickly review this and merge. I have one PR pending

@ebonynon why the images missing in the mobile view ?

@ebonynon why the images missing in the mobile view ?

https://github.com/sliit-foss/sliitfoss/blob/508ab9507d92868a37549b12fd9a2fd9f8e724e0/src/app/components/member/member.component.html#L16
https://github.com/sliit-foss/sliitfoss/blob/508ab9507d92868a37549b12fd9a2fd9f8e724e0/src/app/components/member/member.component.html#L27
I set images auto hide for xs n sm screen sizes as there in no way to fix images overlapping issue in mobile view .

@ebonynon images should be there for mobile views too. Use col-md-7 instead of col-7


refactor: simplify data structures

I reworked the code, mostly for my own understanding. My goal was to eliminate as much state as possible (without impacting performance).
Up to you whether to merge this -- I don't mind either way!

Will revisit later. (ref: https://github.com/sorenlassen/einsum-experiment/pull/5#discussion_r856449133)


v0.38.x: spammy "discard late recheck response" for initial check

We were evaluating the latest v0.38.x branch since it contains a number of promising performance improvements over v0.38.7, and in testing we are observing new error logs, which seem to be a result of the backport of https://github.com/cometbft/cometbft/pull/2268 to v0.38.x (done here).  This seems to differ from the intent of the original pull request in that it applies to initial check as well as recheck.  When #2268 was merged to main, there was a nested switch checkType { to target only recheck for that error log, but on v0.38.x that didn't exists.
The effect we are observing in evaluating v0.38.x is that this "discard late recheck response" error log happens the instant a transaction is broadcast to the node, on the initial check and without any delay as the message suggests.
Is this the expected behavior, or a regression in v0.38.x after v0.38.7?
https://github.com/cometbft/cometbft/blame/ee556e375891db6bfbd64ea54e5671c1a0944858/mempool/clist_mempool.go#L290
More recently in https://github.com/cometbft/cometbft/pull/3178, some refactoring created an explicit recheck function: https://github.com/cometbft/cometbft/commit/9710d69f65804f346a88b15829f0c5fc05711e15#diff-f71975385bf5bc21c04eb08f496140cc8cca405b8dbd7b628342e6d8afb61cc5R429
We didn't see this log before, but we could be doing something wrong.  I can't tell.  Thanks!

Thanks for flagging @jchappelow! This is indeed a bug and here's a potential fix: #3264


Multi threaded HTTP Web Server in Rust

Proposed title of article
Multi-threaded HTTP Web Server in Rust
Proposed article introduction
When you're browsing the website, you're basically communicating with a web server. This web server will send a response from the requesting website. A multi-threaded web server enables to handle multiple simultaneous requests from the same server.
When a program is running, it is first loaded and given an address space in the main memory from your hard disk. Once loaded into the memory it is then scheduled onto an available CPU. When a process is scheduled onto a CPU, the program instructions are executed sequentially.
The CPU will execute each line of the program sequentially. It uses a program counter to determine which instruction it is about to execute. The program counter represents part of an execution context. An execution context is a current state of a running process.
At the same time, you can have multiple execution contexts within a single process. A single execution context is called a thread. A single process can execute one or more threads. These threads can execute simultaneously to run different sections of the program.
In this case, a process is allocated in address space. The threads will share this address space that the parent process has created. This way, threads are considered a lightweight option to a process. This is because a new thread consume less memory than a new process would.
This concept of running more than one thread is called multi-threading. It enables us to do parallel computing efficiently. This guide will implement this concept using Rust.
Key takeaways

What is a Multi-threading
How a Multithreaded HTTP Web Server
Thread-based Server Architectures
Why do you need Multi-threading
How to set up the Multi-threaded HTTP Web Server in Rust

Article quality
There are some guidelines that run the title Multi-threaded HTTP Web Server in Rust, such as

rust-lang.org
oreilly
sergey blog

This guide aims to export this concept of Multithreaded HTTP Web Server. The reader will then be able to build a full-fledged Multi-threaded HTTP Web Server using Rust.
References
Please list links to any published content/research that you intend to use to support/guide this article.
Conclusion
Finally, remove the Pre-Submission advice section and all our blockquoted notes as you fill in the form before you submit.
We look forwarding to reviewing your topic suggestion.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Topic approved @JuliusGikonyoNyambura - Please makes sure your article is unique and adds value to what is available online.


When we do not specify value attribute, an empty string is sent to a server

When we use a plain checkbox element, we do not need to explicitly provide a value attribute. When checkbox is checked, a "on" value is sent to the server on form submit.
For this component, the value attribute must be specified, otherwise internally the component provides its <input type="checkbox"> with an empty value attribute. And therefore, when checkbox is checked, an empty value is sent to a server on form submit.
I spent a lot of time figuring out what's wrong.
I think that if no value specified for the component from outside, the component shouldn't add the value attribute to the internal <input type="checkbox"> element at all. In this case, browser will sent "on" to the server natively.

The default value is "on" for checkbox as well as radio:
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/checkbox#Value
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/radio#Data_representation_of_a_radio_group

Thank you, I'll check out but I need some time.

No problem. For your convenience:
https://github.com/mariomka/vue-checkbox-radio/blob/master/src/components/Checkbox.vue#L94
and
https://github.com/mariomka/vue-checkbox-radio/blob/master/src/components/Radio.vue#L91
default: 'on' solves this issue.


switch to v8 5.3 and clang build

this will fail first time on native c++ plugins test

run ci

@NativeScript/android-runtime : review needed

30 Mbs

👍

:100:


Server Access Request

Please answer the following questions
What is your experience playing modded Minecraft?
I mostly make custom modpack to mess around with mods and like playing modded Minecraft with other people.
Have you successfully been able to play this modpack in single-player?
I have not yet downloaded the pack.
What is your Minecraft username?
BlocksterA
Please re-state the server rules in your own words
No pvp with neutral players or factions
Don't destroy other players creations
Don't tear up the terrain for no reason
Next Steps
Please be patient while waiting for your request to be considered!  Access requests are considered once per day, so you should generally receive a response in less than 24h.  In the meantime, you can set up your Minecraft instance to access the server at:
istari.tungstonminer.com:24565
Finally, it is highly recommended that you read the wiki for the modpack.  There are a lot of changes to world generation, mob spawning, ore placement, recipes and general game mechanics which will be very confusing otherwise.

Sounds good!  Welcome to the server!  If you run into any problems, feel free to ask for help on the Discord channel.


Unable to select text

Check that this is really a bug

[X] I confirm

Reproduction link
https://swiperjs.com/demos/010-default/core
Bug description
How can ? make the text select?
Expected Behavior
No response
Actual Behavior
No response
Swiper version
9.2.0
Platform/Target and Browser Versions
Chrome
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Do you want to ask a question? Are you looking for support? Stack Overflow or Swiper Discussions is the best places for getting support

Please, don't use GitHub issues for questions


polyfill: add support for globalThis

Think globalThis should be the only one...
globalThis should be a polyfill in itself for anyone who needs it.


Fix AdditionalFile inputs to vs-threading analyzers

Some vs-threading analyzers are now data-driven and vssdk-analyzers provide some files that inform VSTHRD010 on which types require the UI thread.
But these files from vssdk-analyzers were not being packaged. And one of the files had an old filename pattern. This PR fixes both.

Codecov Report

Merging #55 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #55   +/-   ##
=======================================
  Coverage   94.44%   94.44%
=======================================
  Files           9        9
  Lines         432      432
=======================================
  Hits          408      408
  Misses         24       24

Continue to review full report at Codecov.

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


Inverse square root (rsq) swizzle

In the D3D8 docs for rsq vertex shader instruction there is a contradiction: the code snippet shown uses the w-component of the source register. However it says "If source has no subscripts, the x-component is used."
It also says x-component is used in D3D9 docs, but I have no reason to believe this is correct in either case. D3D9 docs also state that the source register "requires explicit use of replicate swizzle,that is, exactly one of the .x, .y, .z, .w swizzle components (or the .r, .g, .b, .a equivalents) must be specified." Despite this, for D3D9, DXSO uses an arbitrary, not replicate, swizzle. So it's assumed whatever the shader compiler outputs is correct.
In the spheremap example, a vs 1.0 shader, the shader gives us D3DVS_NOSWIZZLE, which is actually the identity swizzle (.xyzw - 0xE4) and treated as such by d9vk. But this produces incorrect results. Correct results are produced if this is remapped to use the w-component before it reaches DXSO.
In Indiana Jones (#35), which uses vs 1.1: we get both .xyzw (0xE4) and other swizzles including .zwxy (0x4E), .xxxx (0x00), .yyyy (0x55), and .zzzz (0xAA). Remapping the arbitrary swizzles to .wwww produces no noticeable difference, so I assume only the output  w is used anyway.
.zwxy (0x4E) is weirdest one as it appears to be 0xE4 (xyzw) but with backwards byte order. Either that or it is an intentional arbitrary swizzle (after all, the D3D8 docs don't explicitly forbid it and neither does DXSO).
Questions:

Should xyzw (or other arbitrary swizzles) be remapped to wwww only in vs1.0, or in vs1.1 too?
Should xyzw (or other arbitrary swizzles) ever be remapped to wwww in any D3D9 shaders?
Is there ever a case where xxxx is actually used or are the docs lying?

Following code may be helpful to test this:
if (((token & D3DVS_NOSWIZZLE) == D3DVS_NOSWIZZLE)) {
  token &= ~D3DVS_SWIZZLE_MASK;
  token |= (D3DVS_X_W | D3DVS_Y_W | D3DVS_Z_W | D3DVS_W_W);
  Logger::info("RSQ swizzle xyzw");
} else {
  Logger::info(str::format("RSQ swizzle ", std::hex, VSD_SHIFT_MASK(token, D3DVS_SWIZZLE_)));
}

For now I will only remap xyzw to wwww, but there may be issues in the future if certain cases are missed.
Additionally, this issue will serve as a place to put other instructions with hardcoded swizzles if there are any others.

I believe the software vertex VM always uses .wwww, but I'm not sure, and again doing that would break Indiana Jones.


"Beleți-Negrești" is listed in the drop-down menu

Description:
"Beleți-Negrești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Beleți-Negrești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Beleți-Negrești, Argeș".

ENVIROMENT: ui.orase
URL: https://peviitor-ro.github.io/ui.orase/
Browser: Chrome
Device: Laptop
OS: Windows 11 Pro
STEPS TO REPRODUCE:
Open URL in browser
Type "Beleți-Negrești" in the search bar
EXPECTED RESULTS:
The location is listed in the drop-down menu as "Beleți-Negrești, Argeș".
ACTUAL RESULTS:
"Beleți-Negrești, Argeș" doesn't exist in the drop-down menu, the "Arges" is doubled.

Fixed! Poti sa retestezi


Multiple assignments are not working like in Go

The following program fibs.go doesn't run the same way like in Go:
package main

import "fmt"

func main() {
	a, b := 1, 1
	for i := 0; i < 10; i++ {
		fmt.Println(a)
		a, b = b, a+b
	}
}

Expected result:
$ go run fibs.go
1
1
2
3
5
8
13
21
34
55

Got:
$ yaegi fibs.go
1
2
4
8
16
32
64
128
256
512

Thanks for fixing the issue!


feat(time): Turn on Chrono Furnaces

KS can now automatically turn on chrono furnaces, in case you forget.
Fixes #80

@petethered That is the intended behavior for this automation though. If you'd want to have this automation enabled/disabled, based on other factors, then that would be another feature on top of it or an alternative one (like "Build best unicorn building"). I don't like these kinds of features as they produce way too many branches in behavior.
In general, I'm not in favor of features that try to "play the game for you". Automations should be simple and predictable, especially when combined with each other. They may not be always ideal for fast progression, but they should be reasonable building blocks to automate specific stages of the game.
I know counter-examples of this exist in KS today, but those are mostly due to late contributions to v1.5 that were never in line with the original design of KS (as I perceive it).
Turning buildings on automatically is a feature already established by the steamworks. Similar features should behave consistent with this feature. So the related feature in any other tab should also be homed on the same tab as the building that is being turned on. Conditionally activating (and especially deactivating) isn't something that is really being used in KS so far, unless I'm forgetting something.
That being said, I could see adding a trigger to these "turn on" features, but "turning off" is still a new concept. There might also be room for more complex automations in the future. If you want to open a dedicated issue for your improvement idea, we can come back to it in due time :)

In general, I'm not in favor of features that try to "play the game for you". Automations should be simple and predictable, especially when combined with each other. They may not be always ideal for fast progression, but they should be reasonable building blocks to automate specific stages of the game.

This is a reasonable line of thinking.  Thanks for clarifying the goals (make it easier, not make it play for you).
  Admittedly, trying to figure out why things were/weren't happening got me digging through the source, but I'm the exception not the rule.
In this case:
Kittens Game - Run 22 - Year 5350 - Spring, day 7.txt
When you load that save, I'm in "Charon" , Time control is DISABLED for this cycle.
If you enable "Turn on Chrono Furnaces", I'm ripped OUT of the cycle via the furnaces.

If I'm a "general user", I'm confused as to why I'm skipping out of a cycle that I have disabled (to indicate I want to stay in it).
Especially given that the thing that did it is in a different part of the UI.

A simple , half step solution, that keeps to the spirit (don't auto play) but reduces confusion is to move "Turn on chrono furnaces" to time control below Tempus Fugit.  At least then, all the features that involve moving the clock automatically are in one place.

@petethered I agree that this is exactly the kind of situation that should be avoided and I also follow your reasoning for why "Time Control" is the better home for this feature due to it.
So it shall be moved, but I might rename it to set it apart from other "Turn on" options. Maybe "Burn Chrono Heat"?
We'll track this under:

#113

In general, users don't have the desired amount of control over time, especially when it comes to selecting a target time period to be in. When constructing a solution for this, the behavior of enabling Chrono Furnaces must be taken into account. It'll probably still take quite some time before a full iteration on this feature though.

+1 to doing something here. Can the "Time Skip" logic take into account how much stored heat you have and enable the automation on the Chrono Furnace, then turn off the automation to use fewer crystals? Semantically I would like to skip all the non-redmoon years and I can't see to encode that, since in its current state the system will skip using x45 crystals, build up 500% heat and then automatically skip over the exact season I wanted to be in.


Update docs for main website

.

Summary by Sourcery
This pull request significantly updates the documentation for the main website, including a comprehensive Quick Start guide and improved formatting for various guides and integration documents. It also introduces a new README file to better organize the documentation structure.

Documentation:

Completely overhauled the main documentation index to provide a comprehensive Quick Start guide, including installation steps, model definitions, type definitions, schema building, and API serving instructions.
Updated the 'Filtering' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Ordering' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Defining Fields' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Defining Types' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Query Optimizer' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Mutations' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Authentication' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Permissions' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Relay' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Resolvers' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Django Choices Field' integration guide to include a new title and improved formatting for tips and warnings.
Updated the 'Export Schema' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Queries' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Pagination' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Views' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Django Debug Toolbar' integration guide to include a new title and improved formatting for tips and warnings.
Updated the 'Settings' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Community Projects' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Frequently Asked Questions' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Subscriptions' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Unit Testing' guide to include a new title and improved formatting for tips and warnings.
Updated the 'Django Channels' integration guide to include a new title and improved formatting for tips and warnings.
Updated the 'django-guardian' integration guide to include a new title and improved formatting for tips and warnings.
Added a new README file for the documentation, providing an organized index of all guides and integrations.

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

Project coverage is 88.73%. Comparing base (9025ee5) to head (10cb5c5).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #605      +/-   ##
==========================================
- Coverage   88.84%   88.73%   -0.12%
==========================================
  Files          41       41
  Lines        3605     3605
==========================================
- Hits         3203     3199       -4
- Misses        402      406       +4

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


Add script to start Hive in Docker

That's good to know about the other command, though it appears to require the server to be built:
./presto-product-tests/conf/docker/singlenode/compose.sh up hadoop-master
Invalid path: [././presto-product-tests/conf/docker/singlenode/../common/../../../bin/../../presto-server/target/presto-server-313-SNAPSHOT/]. Working directory is [/Users/dphillips/src/presto].

This one is the same as the one used by the Hive integration tests. I often hack up those scripts to just start the environment, so I decided to pull it out into a separate script.


Unable to create assymetric key encryption on Config Server

Hi!
I am working on creating a configserver that can use the key management solution with an RSA keypair detailed here, but I cannot seem to get it to work.
My expected workflow: I add the public key to the java keystore for the application to the config server, and use the /encrypt endpoint to add encrypt the property. Then I mount the private key in the client application which will decrypt it locally.
Here is what I did:
Create a Config Server with the following properties:
spring.cloud.config.server.git.uri=${HOME}/repos/config-repo
encrypt.keyStore.location=classpath:/server.jks
encrypt.keyStore.alias=encrypt
encrypt.keyStore.password=letmein
encrypt.keyStore.secret=changepass
spring.cloud.config.server.encrypt.enabled=false

Created the keypair, then exported the public key as a certificate into server.jks, which is created with the import command.
[da3mon@localhost resources]$ keytool -genkeypair -alias encrypt -keyalg RSA -validity 360 -keystore encrypt3.jks -storepass letmein -keypass changepass
What is your first and last name?
  [Unknown]:  Config
What is the name of your organizational unit?
  [Unknown]:  Server
What is the name of your organization?
  [Unknown]:  App
What is the name of your City or Locality?
  [Unknown]:  Budapest
What is the name of your State or Province?
  [Unknown]:  Budapest
What is the two-letter country code for this unit?
  [Unknown]:  HU
Is CN=Config, OU=Server, O=App, L=Budapest, ST=Budapest, C=HU correct?
  [no]:  yes

[da3mon@localhost resources]$ keytool -export -alias encrypt -rfc -keystore encrypt3.jks -storepass letmein -keypass changepass -file encrypt3.cer
Certificate stored in file <encrypt3.cer>

[da3mon@localhost resources]$ keytool -import -file encrypt3.cer -alias encrypt -keystore server.jks -storepass letmein -keypass changepass
Owner: CN=Config, OU=Server, O=App, L=Budapest, ST=Budapest, C=HU
Issuer: CN=Config, OU=Server, O=App, L=Budapest, ST=Budapest, C=HU
Serial number: 4f1d48ec
Valid from: Tue Oct 25 11:18:28 CEST 2016 until: Fri Oct 20 11:18:28 CEST 2017
Certificate fingerprints:
     MD5:  DA:CD:F7:59:41:67:92:72:BB:FC:09:D7:CB:79:52:78
     SHA1: 40:93:B3:51:19:2E:DF:CA:86:DF:5B:31:E4:E8:1F:7D:4B:D7:B2:68
     SHA256: 86:01:46:ED:4F:96:76:0C:4F:2B:22:AA:6E:9F:97:27:5D:D3:81:E4:12:CC:60:66:0D:05:A9:96:C7:C7:2C:12
     Signature algorithm name: SHA256withRSA
     Version: 3

Extensions:

#1: ObjectId: <IP_ADDRESS> Criticality=false
SubjectKeyIdentifier [
KeyIdentifier [
0000: 74 43 71 2D 02 A0 4C 9D   96 19 61 9C EF 45 DC 66  tCq-..L...a..E.f
0010: B5 80 53 76                                        ..Sv
]
]

Trust this certificate? [no]:  yes
Certificate was added to keystore
[da3mon@localhost resources]$ keytool -list -keystore server.jks
Enter keystore password:

Keystore type: JKS
Keystore provider: SUN

Your keystore contains 1 entry

encrypt, 2016.10.25., trustedCertEntry,
Certificate fingerprint (SHA1): 40:93:B3:51:19:2E:DF:CA:86:DF:5B:31:E4:E8:1F:7D:4B:D7:B2:68

However when I check whether I can encrypt variables, I am unable to do so:
[da3mon@localhost resources]$ curl localhost:8888/encrypt/status
{"timestamp":1477387291480,"status":500,"error":"Internal Server Error","exception":"java.lang.IllegalStateException","message":"Cannot load keys from store: class path resource [server.jks]","path":"/encrypt/status"}[da3mon@localhost resources]$`

Any Ideas what I am doing wrong?
EDIT:
I did some research:
According to the manual:
The encryption is done with the public key, and a private key is needed for decryption. Thus in principle you can configure only the public key in the server if you only want to do encryption (and are prepared to decrypt the values yourself locally with the private key). In practice you might not want to do that because it spreads the key management process around all the clients, instead of concentrating it in the server. On the other hand it's a useful option if your config server really is relatively insecure and only a handful of clients need the encrypted properties.
However KeyStoreKeyFactory seems to require the private key anyway:
https://github.com/dsyer/spring-security-rsa/blob/master/src/main/java/org/springframework/security/rsa/crypto/KeyStoreKeyFactory.java
public KeyPair getKeyPair(String alias, char[] password) {
        try {
            synchronized (lock) {
                if (store == null) {
                    synchronized (lock) {
                        store = KeyStore.getInstance("jks");
                        store.load(resource.getInputStream(), this.password);
                    }
                }
            }
            RSAPrivateCrtKey key = (RSAPrivateCrtKey) store.getKey(alias, password);
            RSAPublicKeySpec spec = new RSAPublicKeySpec(key.getModulus(),
                    key.getPublicExponent());
            PublicKey publicKey = KeyFactory.getInstance("RSA").generatePublic(spec);
            return new KeyPair(publicKey, key);
        }
        catch (Exception e) {
            throw new IllegalStateException("Cannot load keys from store: " + resource, e);
        }
    }

Is this a doc issue then?

You can keep all encryption/decryption on the server, in fact that is the default behavior.

Hi Ryan!
Sorry for the late response, but I did manage to figure out a solution, and just forgot about this ticket :/ I needed to implement 2 config servers: One has full encryption functionality, and is used to encrypt passwords, before storing them in git. The second serves the actual configuration, and does not have any encrypt/decrypt functionality. The actual decryption is handled by the spring-config-client in the application itself.
This was a requirement from ITSec so that different banking apps of ours use different keys, thanks for your answer anyway!

Hi @da3mon-01,

One has full encryption functionality, and is used to encrypt passwords

Does this config server with full encryption functionality still need the private key to avoid the IllegalStateException?


Slow

Works well
But the slow switching between options!
ubuntu-gnome

Will try to reproduce in a VM with Ubuntu GNOME later

Can't reproduce on Ubuntu GNOME 17.04 in Virtualbox.
Anyway, as noted in "The Definitive Guide To Theming GRUB 2" on page 73, try to add scrollbar = false to the boot_menu section of theme.txt


Native client not found when backing up or restoring Postgres 11 database

System information:

Windows 10 Home 1909 64bit
DBeaver version <IP_ADDRESS>002011957

Connection specification:

PostgreSQL 11 (on Heroku)
PostgreSQL JDBC Driver

Describe the problem you're observing:
When trying to make a backup of a Postgres 11 database, or restoring one, I get the following error:
Task execution failed

Reason:
Native client home 'postgresql-x64-11' not found

Steps to reproduce, if exist:
Right click on a database -> tools -> backup
Select all views
Click start
OR
Right click on a database -> tools -> restore
Select backup file
Click start
Include any warning/errors/backtraces from the logs
2020-02-05 18:33:54.614 - org.jkiss.dbeaver.model.exec.DBCException: Native client home 'postgresql-x64-11' not found
	at org.jkiss.dbeaver.tasks.nativetool.AbstractNativeToolHandler.validateClientHome(AbstractNativeToolHandler.java:104)
	at org.jkiss.dbeaver.tasks.nativetool.AbstractNativeToolHandler.doExecute(AbstractNativeToolHandler.java:229)
	at org.jkiss.dbeaver.tasks.nativetool.AbstractNativeToolHandler.lambda$0(AbstractNativeToolHandler.java:52)
	at org.jkiss.dbeaver.runtime.RunnableContextDelegate.lambda$0(RunnableContextDelegate.java:39)
	at org.eclipse.jface.operation.ModalContext$ModalContextThread.run(ModalContext.java:122)

Partial workaround
Changing the client in the backup process to PostgreSQL Binaries as suggested here https://github.com/dbeaver/dbeaver/issues/4106#issuecomment-416759677 seems to resolve the backing up part of the problem. However, when restoring from a backup created in this way and using PostgreSQL Binaries for the client, I get an error saying [archiver] input file does not appear to be a valid archive

can you solve it ?

If you click "Client..."

and then select the empty value in the dropdown:

you can then restore/backup.

I solved this by adding a new home directory. At first followed the steps @Tri125 recommended, selecting browse instead of the blank option, then clicked the Add Home button and selected the bin directory (C:\Program Files\PostgreSQL\12\bin in my case) then selected that created home as the client to do the backup and it worked fine.

Note: the PostgreSQL 12 home directory was pointing to *C:\Program Files\PostgreSQL\12*

I was having the same problem just now and what @jyupanquil suggested worked perfectly in my Windows 10 env.

It worked well. Click the client button.

On macOS with PostgreSQL installed via homebrew, you can click on "Local Client...", click on the dropdown, click "Browse...", click "Add Home", press CMD+shift+G to open the folder jumper, then navigate to /opt/homebrew/bin, and add that folder.

@wub thank you very much. You saved my day 👍

Adding to wub's answer in MacOS (Catalina) i had to find the bin folder at<EMAIL_ADDRESS>not sure if the path would be the same for further versions like Monterey

if you type in a terminal pg_config --bindir. this command give you the path to psql home. then you can add the home path in local client config.

step 1 instal postgress
install posgress-client https://www.postgresql.org/download/linux/ubuntu/
step 2 install psql client is not exist command
sudo apt-get install -y postgresql-client

step 3
set local client  /usr/bin for add home button for dbeaver

For DBeaver 23.2+ (and possibly also lower versions), there is no option to choose a client in the 'Restore' GUI. I've tried adding all the paths I can think of to the 'Native Client' pane as suggested in above replies (Right click connection > Edit connection > Connection Settings/Main > Driver Settings > Native Client) to no avail. Still getting "Native client home<EMAIL_ADDRESS>not found" despite the fact that I have only Postgres 13 installed.
Restore pane

DB connection settings pane

Hello. I am trying to use dbeaver to backup a postgres 12 but I cannot use it.
I configured local client to /usr/bin and it automatic loaded it ( /run/user/1000/doc/6f337d65/bin ) but It failed to use it.
HOW CAN I SOLVE THIS MESS PLEASE?


[Feature Request] Support search by id for register command

Currently only searching by manga name is supported like this:
mangathr register -s mangadex "Berserk"
It would be nice to be able to search by id in case of MangaDex being the source, as it would eliminate false positives:
mangathr register -s mangadex "801513ba-a712-498c-8f57-cae55b38cc92"

Hey @nuxencs, thanks for opening this issue!
This will be added in the next minor release (later today / this week).


MockModule does not work for MatSelectModule in 9.6.4

ng-mocks version: 9.6.4.
angular material version: 9.2.4
When mocking the mat-select component in unit test, I did MockModule(MatSelectModule), and it causes the unit tests to fail with error "No value accessor for from control with name 'xxx'".
The component template was like:
<mat-form-field>     <mat-select formControlName="xxx">...</mat-select> </mat-form-field>
The behavior was OK in ng-mocks 9.5.0. Do not know what has changed since then.

Hi, thanks for the report.
could you try the latest version?

For my project, I am still using Angular 9, so I tried with your latest 9.x.x version. Even if the latest 11.x.x version had fixed it, I am not going to upgrade in a while before the upgrade of Angular. So I do hope you can have a fix release for the version 9 if you confirm this issue. Thanks.

There is no link between angular and ng-mocks version. You can use any on any.

ok good to know it, thanks. I will have a try with the latest version 11.

The latest version 11 seems to be Good! Thanks. Although still don't know the cause of this issue in the early version.

I'm quite sure, I broke something that time. There were many changes regarding control value accessor lately.
Then I'll close the issue for now. Feel free to reopen it if something is still wrong.
Happy coding!


Hidden property causes ambiguous exception

Describe the bug
If a class hides a similarly named property, the AmbiguousMatchException is thrown.
Reproducible Example
public abstract class BaseClass {
    public string Value { get; set; } = "base";
}
public class ImplClass : BaseClass {
    new public int Value { get; set; } = 1;
}
public interface IShim {
    string Value { get; }
}

var obj = new Issue12Class();
var shim = obj.Shim<IShimIssue12>();
var res = shim.Value; // Expected "base"

Expected behavior
The property with the matching signature is shimmed.
Environment (please complete the following information):

Version 1.6.2

Additional context
Issue was hidden until related issue was resolved: #11

Fixed by matching to property by type first, and then falling back to previous method


Error while using while using  Kitura v. 20 for onResponseEnd member

Error on  "response.onResponseEnd "  -   value of type  'RouterResponse' has no member 'onResponseEnd;
Function:
public func handle(request: RouterRequest, response: RouterResponse, next: () -> Void) throws {
startTime = NSDate()
    let previousOnResponseEnd = **response.onResponseEnd** ?? nil
    **response.onResponseEnd** = { [unowned self, unowned response] in

}

I'm sorry we missed this issue. Please reopen if you still need help.


end of service report is now only required if at least one delivery session is attended, rather than attempted

What does this pull request do?
Updates the response so that an end of service report (endOfServiceReportCreationRequired in dto) is only required if at least one delivery session has been attended.
What is the intent behind these changes?
The current logic is incorrect. Currently and end of service report is required if at least one session is attempted, regardless if it was attended or not.

I think there's logic about this in the reports as well

 Only ask for EOSR where IAA / ICA has occurred

I think yes, as now

requiresEndOfServiceReportCreation operates on the bases on attendance (yes, late)
all other places assume "end of service report is missing" when determining cancelled (ie here, here and here)
therefore, a referral with all sessions in "did not attend" would constitute a "cancellation" with this change

 Only ask for EOSR where ICA has occurred


Add custom resource for egress nat policies

Adds CiliumEgressNATPolicy crds for egress nat gateway configuration
apis. Specifically each policy consists of a list of endpoint selectors,
a list of destination cidrs and SNAT ip. This information
is used to update egress ebpf map in the datapath.
Signed-off-by: Bolun Zhao<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin

Related Issue: #13575

@MasterZ40 I've marked this PR as draft given that we discussed yesterday about the design doc. Feel free to mark it ready for review as you wish.
@aanm thanks for letting me know. I think the discussion on Wednesday is mainly about the future CRD for mcc functionalities. I synced with Yuan who's the stakeholder of that feature and confirmed that this CRD change is not going to affect future design of other features. So as far as egress nat policy is concerned, we don't have any additional changes planned.
Let me know if you have other issues regarding this CRD in particular. I will make sure to update.

test-me-please

test-me-please

test-me-please

test-me-please

I checked all team reviews are covered. Tests are passing. Marking as ready to merge.


Add swan ui infrastructure

Current coverage is 45.45% (diff: 100%)

Merging #128 into master will increase coverage by 1.31%

@@             master       #128   diff @@
==========================================
  Files            16         16
  Lines          8907       8907
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3932       4049   +117
+ Misses         3940       3837   -103
+ Partials       1035       1021    -14

Powered by Codecov. Last update 18ca5ef...46f17e0

LGTM


tests: added unit test in fedimint-core/src/runtime.rs

Fix
This PR adds unit test fin fedimint/fedimint-core/src/runtime.rs and closes #5140

Please don't use sleep, or at least not with 1 second values. CI time is a scarce resource. Where you really need to guarantee sleep, maybe use 1 millisecond. where you need coordination between threads use https://docs.rs/tokio/latest/tokio/sync/oneshot/index.html to unblock things.

Hi @dpc, I have changed from 1 second to 1 millisecond. Can you please review this pr again?

Closing. Feel free to reopen with improvements or just create a different PR.


[WIP] Move version field into DeltaTableState struct

Description
The description of the main changes of your pull request
Related Issue(s)
#390
Documentation
//TODO

@explicite let us know if you need any help in finishing up this PR :)

@explicite - are you planning on finishing this? If not, I may just include it in #632, since it is quite helpful over there.

@roeap let's include it in #632 then if you need it there.

closed in favour of #649


Peer review: Add one more choice to the list

peer review feature:
No FoP in copyright-related reasons for deletion (the second question). This is one of the most common reason for deletion, it should be in the list. Proposed message: "There is no freedom of panorama in the country where this picture was taken, so some installations (even permanently displayed in public spaces) can not be freely photographed and shared."

Here is the screen where I believe "No FoP" should appear. Considering the size, I believe something like "Breach of Freedom of Panorama rules" would be OK as a text. If you can add links, you cen link Freedom of Panorama to C:FOP

And here is the incomplete deletion request. It reads "Because it is ~~~~" in Romanian. Maybe you should use the non-localized version for deletion proposals?

Thanks! I created more issues based on your feedback, feel free to subscribe to them too and add more information:
https://github.com/commons-app/apps-android-commons/issues/4800
https://github.com/commons-app/apps-android-commons/issues/4801

I would like to work on this but I am not able to understand this issue properly. Can you please elaborate and tell me the meaning of FoP

@Rishavgupta12345 you have a more detailed explanation below the picture. You need to add another entry in the list, with the text "Breach of Freedom of Panorama rules" and do not create the request If no option is chosen.
Given that I expect other people to ask for other options, maybe using a dropdown instead of checkboxes would be a good idea? @nicolas-raoul any thoughts on replacing the checkboxes with a dropdown? I'm not even sure what happens if more options are selected.

Maybe just add another checkbox for now? If we run into space problem, then switching to a dropdown can definitely be considered (as another GitHub issue).
In parallel, it would indeed be good to test what happens when selecting several options, and if the result is bad then propose a change as another issue. Thanks again! :-)

I would like to work on this issue please assign me @nicolas-raoul

@nicolas-raoul I have a doubt . whenever I am running the app from android studio its running the beta version of the app. but I want to run the main version of the app in my emulator , how should i do it

Since this issue is more complicated than expected, I split the second part to its own issue: #4844

Hi I was trying to work on #4801 and I saw that its linked to this one. It seemed that the other one cant be solved with this one finished first, so I tried to work on this one instead. I`ve made a pull request for this issue, can you help me check if it would work alright? I will try to fix remaining problems. Thank you! @nicolas-raoul

As I see this is still open ,I would like to work on this .Please assigned it to me @nicolas-raoul

As I see ,this is still open and I want to work on this .Please assigned me @nicolas-raoul .

@awanishyadav967 Please use the prodDebug build flavor.

@nicolas-raoul ,Done ,created new pull request please review it.Thanks


Strip binaries

What this PR does / why we need it:
Strip binaries
Special notes for your reviewer:
see e.g. https://blog.filippo.io/shrink-your-go-binaries-with-this-one-weird-trick/
Release note:
NONE

seems this is not wanted without more research on consequences etc....
/close

this is stripping out the debug symbols, which in reality really isn't a big deal. I did the same thing in ocs-operator recently.
I think the only reason we want debug symbols is during development to diagnose a crash or something... but even then i doubt we need debug symbols. We aren't inspecting coredumps or anything here with gbd.


<EMAIL_ADDRESS>

Add<EMAIL_ADDRESS>

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (cel-spec) have been updated in this PR. Please review the changes.


feat: add option to delete android emulators

This utility is not reliable, I'm closing this PR for now as it's not ready to be relased.


Ubuntu WSL hangs at launch

Environment
Windows build number: 10.0.19041.450
Your Distribution version: Ubuntu 20.04
Whether the issue is on WSL 2 and/or WSL 1: Linux version 4.4.0-19041-Microsoft<EMAIL_ADDRESS>(gcc version 5.4.0 (GCC) ) #1-Microsoft Fri Dec 06 14:06:00 PST 2019

Steps to reproduce

Launch a WSL terminal (if you're lucky)

Expected behavior

Opening a responsive WSL terminal
Actual behavior

Sometimes when I open a new WSL terminal, the terminal hangs and becomes completely unresponsive. It will not show up in the task manager, and is impervious to external kill commands. I've unsuccessfully tried to terminate the process using the solutions proposed in this Stack Overflow thread, including:

Restarting the LxssManager from the task manager. When I try this, restarting the service hangs at "Stopping"
Terminating Ubuntu from the Apps & Features panel (Advanced options -> terminate). This doesn't do anything
Opening a Powershell and executing wslconfig /t Ubuntu. This command also just hangs

None of these attempts had any effect. The only remedy seems to reboot my laptop. Before attempting to kill the terminal, if I simply open another terminal it usually works fine, so most of the time I simply leave the zombie process alone and continue working in another terminal. It doesn't seem to consume any resources.
I've tried to get detailed WSL launch logs, but I've not been able to reproduce the problem while logman.exe is running, as the issue manifests itself only occasionally and seemingly randomly. No errors or exceptions are raised at any point.

having the same problem

duplicate of #5747

duplicate of #5747

Wouldn't you rather say that #5747 is a duplicate of #5744 (this one)?


Ignored files apply to download

Cleaned up the file watcher code a bit to use the theme clients filter and applied it to the asset lists from the server so that it filters downloads as well.
fixes https://github.com/Shopify/themekit/issues/126
cc @chrisbutcher @ilikeorangutans @jessicaxiejw

🚢


Rename method retrieve_job to job in IBMQBackendService

What is the expected enhancement?
Rename method retrieve_job to job in IBMQBackendService to align with terminology used in runtime jobs.

1


Move LVL temporary & worker MSVC cmake targets into solution explorer subfolder

This sets up cmake to locate the temporary or worker MSVC projects into a subfolder called lvl_cmake_targets. This simply unclutters tte Visual Studio Solution Explorer projects pane, and does not affect functionality.
For example, all of the solution explorer projects such as *-def-file, *-json, and the file generator targets will show up in the lvl_cmake_targets folder.

@mark-lunarg looks like build errors on screenshot and api-dump layers. Ping for re-review when CI is green.


Adding microbatching support and setting for RAW

This modifies the RAW aggregation to support micro-batches (previously it assumed a micro-batch of size 1). The size of the micro-batch is configurable but recommended to be left at 1 unless the amount of raw tuples being shuffled from Filter to Join Bolts becomes a problem.
Since it changes the interface of what the RAW aggregation emits, bumping the minor version.

CLA is valid!

Coverage increased (+0.2%) to 98.871% when pulling 435f98c2f27b8d9d86f53d6ccdce96b3fbeada91 on micro-batching-for-raw into c3b03a46c75190ed9eaaf7ddb15eeb0b180c2195 on master.

Coverage increased (+0.2%) to 98.871% when pulling 3772137b3d39d43d7205238e4352694e5b7af2f9 on micro-batching-for-raw into c3b03a46c75190ed9eaaf7ddb15eeb0b180c2195 on master.


Enable Metadata\MetadataFactory::setIncludeInterfaces()

Any particular reason why Metadata\MetadataFactory::setIncludeInterfaces() is not enabled per default and/or no setting exists. Would be cool to use interfaces for shared metadata.

I realized this might sound demanding, which is not what I meant. Much rather: has this been considered before?

jms serializer uses the same strategy, and since this library is just a plugin for the serializer, it has to follow the same.
On the other side, in jms that decision has been taken to avoid some unexpected behaviors when defining metadata (makes harder to override metadata). It has been used a simple rule  "metadata are per-class" anything else (interfaces, traits) is just ignored.


Enhancements for frozen PE tests.

This patch adds testcase for max frozenPE and also
fixed basic frozenPE issues.
Suggested-by: Pridhiviraj Paidipeddi<EMAIL_ADDRESS>Signed-off-by: Venkat R B<EMAIL_ADDRESS>

This change is

@stewart-ibm This version is working fine, You can merge if it looks good.

Thanks, merged to master as of 5b80cb968e2a


Website gives type error when I add a certain image

Whenever I add a png of one of my portfolio pieces into the projects section it gives a type error. I asked shauna about it and she thought it was the file name so I changed it, but I tried it a couple minutes ago and it still happened. It happens with some of my other portfolio images too. Could the generator just not take certain pngs? hers some screenshots of what im getting.

Hi! I'll take a look at your repo and see if I can figure this out and fix it.

@angelinasav20 can you make sure that all your code is committed and pushed? It looks like it's not synced back to GitHub right now. In the Source Control panel, the 'Commit' button will change to a 'Sync' button - once you click that I'll be able to take a look and see what went wrong.

I figured out what the bug was! your image had an uppercase PNG as its extension, and my code was only looking for images with lowercase extensions. I fixed the issue in a pull request on your repo.
If you click into the pull request and click the 'Merge' button it will pull those edits into your copy of the site. You will then have to re-sync your codespace when you open it again. The 'Commit' button should turn into a 'Sync' or 'Pull' button; I can't remember which it is.
Let me know if that works! If you're having trouble with the pull request, Shauna or I can help you with that in class tomorrow night.


How to read the multi-layer perceptrons model in Golang written using python

I am using the wrapper of scikit-learn Multilayer Perceptron in Python https://github.com/aigamedev/scikit-neuralnetwork to train the neural network and save it to a file. Now, I want to expose it on production to predict in real time. So, I was thinking to use Golang for better concurrency than Python. Hence, my question is whether can we read the model using this library written using Python or above wrapper? The code below I am using for training the model and last three lines I want to port to GOLang to expose it on production
import pickle
import numpy as np
import pandas as pd
from sknn.mlp import Classifier, Layer
from sklearn.model_selection import train_test_split
from sklearn.metrics import accuracy_score

f = open("TrainLSDataset.csv")
data = np.loadtxt(f,delimiter = ',')

x = data[:, 1:]
y = data[:, 0]
X_train, X_test, y_train, y_test = train_test_split(x, y, test_size=0.3)

nn = Classifier(
    layers=[
        Layer("Rectifier", units=5),
        Layer("Softmax")],
    learning_rate=0.001,
    n_iter=100)

nn.fit(X_train, y_train)
filename = 'finalized_model.txt'
pickle.dump(nn, open(filename, 'wb'))

**#Below code i want to write in GoLang for exposing it on Production** :
loaded_model = pickle.load(open(filename, 'rb'))
result = loaded_model.score(X_test, y_test)
y_pred = loaded_model.predict(X_test)

I would say that, at present, Golearn's neural API is not currently sufficient for your use-case because

It's not vectorized, meaning that it will likely be slower than doing it in Python.
It doesn't implement the concept of feed-forward layers efficiently.
It doesn't offer the requisite per-node control over the activation function.

Basically, I wrote this bit back in 2014, when I knew significantly less about neural networks that I do now. I'd also say that provided you're deploying your Python server-side application behind something like gunicorn, you'll have most of the concurrency benefits anyway.

@Sentimentron is it foreseeable that we will soon have a vectorized ML lib in Golang?

@Sentimentron That means this is (and will be) just a simple implementation for experimenting with ML and there is no way to "port" existing models from other libs like Inception v3? Would be really nice to have a native Go lib for inference.


unique constraint violated! but on which property of the object?

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[ ] 3.0

On this operating system:

[ ] Linux

[ ] Fedora .rpm

I have a user model created with Foxx app, there is email and username fields that has been indexed and should be unique.
the problem is on conflict on either property I get the same error regardless of the field. I need to notify the user about where is the conflict, how should i go abut that?
I'm very new to Arango (two days)

I'm sorry this is currently not easily possible.
As a workaround you need to catch the error and run a subsequent query which looks for these conflicts.
It is already being discussed in #342 - thus closing this one as duplicate.

I'm sorry to hear that :-( as my app heavily relies on letting the user know about these errors preferably a customized one too.

well, just another query filtering for these unique index columns and you got the offending documents?

@mygnu This is possible now. with Arangodb 3.2 which is available for download.


Perform QA on site

Here is the production launch QA checklist:
Testing

[ ] Browser test: IE 9-11, Chrome, Firefox, Safari (current versions)
[ ] Device testing: i0S (current minus 1) , Android (4.1 and up)
[ ] Test responsive form factor on a variety of devices
[ ] Test with CSS turned off
[ ] Stress test using Blitz.io
[ ] PC / old display check

Accessibility

[ ] WAI-ARIA roles
[ ] Keyboard navigation
[ ] Proper form markup
[ ] Color contrast check
[ ] Image alt tags
[ ] Go through 508 checklist

Content

[ ] Spell-check and grammar check
[ ] Source quality assets
[ ] Check all links
[ ] Error pages
[ ] Favicon
[ ] Footer - directs to Bocoup.com

Functionality

[ ] Confirm Google Analytics working
[ ] Optimization and Validation

Performance

[ ] Delete unused files
[ ] Gzip CSS
[ ] CriticalCSS if appropriate
[ ] Consolidate JS requests
[ ] Validate HTML
[ ] Optimize images
[ ] Implement responsive images
[ ] Run through Web Page Test

We will most likely be running the tests end of this week/ early next week. Please sign up to volunteer by replying to this comment.

We'll need a way to put the new site up on a server or something for people to be able to do this. Do we have ideas on how we'll do this?

I'm in for testing! Book me!

Happy to help as well!

thanks @labond and @jorydotcom - we will be in touch next week.

Thanks @reconbot, I'll figure out urls, after I get the site cleaned up and ready for actual QA, we still have some final issues.

Making urls work for different situations in Jekyll:

http://ricostacruz.com/til/relative-paths-in-jekyll maybe this is the best way to do this?
https://byparker.com/blog/2014/clearing-up-confusion-around-baseurl/ I'm not sure about this one, because i think it will set the urls to be wrong for when we launch production.
There are ideas in this issue as well, so this seems like an issue people are still struggling with (comments from years ago to very recently) https://github.com/jekyll/jekyll/issues/332

@susanjrobertson has the QA site up and we are doing the QA in the QA repo

One note: the CNAME in the branch site-with-content has been changed to match the QA url, that will need to be changed back when we want to launch.


Camera is missing a lot of features

I am not sure how to structure a Camera class that will work for 2D and 3D. Maybe look into how other 3D frameworks handle that.
The biggest problem right now is that a lot of the camera's values are assumed. Added flexibility isn't really an issue. However, I have no clue how to abstract most of the confusing matrices and math away and provide a neat interface.

Honestly, I just made it so the camera is completely flexible. I am not sure if my abstraction layer is ideal, but for now it will have to do!


feat: add all resource type support for egctl x status

What type of PR is this?

follow-up of #2550, add support for egctl x status all to get the summary of all resource types status
for example, the resource types with empty status will be ignored:

can we keep this similar to kubectl get all which looks like
the resource type is appended into the name with /
kubectl get all
NAME                           READY   STATUS    RESTARTS       AGE
pod/backend-5cbb9cd947-rgdb7   1/1     Running   2 (116m ago)   2d3h

NAME                 TYPE        CLUSTER-IP     EXTERNAL-IP   PORT(S)    AGE
service/backend      ClusterIP   <IP_ADDRESS>   <none>        3000/TCP   2d3h
service/kubernetes   ClusterIP   <IP_ADDRESS>      <none>        443/TCP    2d3h

NAME                      READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/backend   1/1     1            1           2d3h

NAME                                 DESIRED   CURRENT   READY   AGE
replicaset.apps/backend-5cbb9cd947   1         1         1       2d3h
``

can we keep this similar to kubectl get all which looks like
the resource type is appended into the name with /
kubectl get all
NAME                           READY   STATUS    RESTARTS       AGE
pod/backend-5cbb9cd947-rgdb7   1/1     Running   2 (116m ago)   2d3h

NAME                 TYPE        CLUSTER-IP     EXTERNAL-IP   PORT(S)    AGE
service/backend      ClusterIP   <IP_ADDRESS>   <none>        3000/TCP   2d3h
service/kubernetes   ClusterIP   <IP_ADDRESS>      <none>        443/TCP    2d3h

NAME                      READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/backend   1/1     1            1           2d3h

NAME                                 DESIRED   CURRENT   READY   AGE
replicaset.apps/backend-5cbb9cd947   1         1         1       2d3h
``

done, it's much simpler this way.

great work @shawnh2, one suggestion - can we use small case for kind ?

great work @shawnh2, one suggestion - can we use small case for kind ?

of course

/retest

/retest


'element feedback'

ideas/discussion - "sparsity aware element feedback"
use case would be learning rules for AI using sparse matrices, e.g. steps of backprop modifying a matrix of weights, or whatever else AI researchers may imagine. (eg backprop: 'vec_a' would be previous layer activations, 'vec_b' would be error values). Also further bridges the gap between a "sparse matrix lib" and 'graph processing'

is there an existing interface in any existing matrix libraries that supports this functionality?
are there other ways of expressing this (like "a hadamard product of a sparse matrix and (the tensor product of vec_a,vec_b)")?
is this already possible?
my prefered option would be an interface that takes a lambda function, then people could use that to apply whatever operations they liked (eg expressing a product operation). (Personally I also  think it would also be interesting to implement matrix multiply via a traversal taking generalised element combination & reduction functions aswel..)

This is trivial enough for dense matrices, fairly trivial for COO vs dense vectors, trickier for any compressed sparse formats X sparse vectors, and where it would get extremely useful (and difficult) is threaded implementations of these.
One may also want to consider different permuations of what to do with empty elements (eg would we want to apply this with all 'a[j]','b[j]',and m[i][j] occupied, or any occupied m[i][j] for either a[i] or b[j] occupied

The hadamard product is rather simple, but requires left and rigth to have the same sparsity structure. In which case you can iterate over vec.data() of lhs and rhs.
For the tensor product we have some code, as this is simply a Nx1 by 1xM´ matrix product. You can have a look at the smmp` part of the code to get an idea on how to parallelize this. The serial part of this requires figuring out the sparsity pattern of the output, but this section could be optimized in the case of vectors.


[stable/jaeger-operator] No TillerVersion in minikube 1.4 and helm 3

can't evaluate field TillerVersion in type interface running installation of jaeger-operator via  helm 3 into minikube 1.4 on windows 10.

Same here


Migrate AndroidKeyManager to web5-kt

Similar to AWSKeyManager there is an AndroidKeyManager implementation currently in the android example: https://github.com/TBD54566975/tbdex-android-example/blob/main/app/src/main/kotlin/AndroidKeyManager.kt and a test for it here: https://github.com/TBD54566975/tbdex-android-example/blob/main/app/src/androidTest/kotlin/AndoidKeyManagerTest.kt
It may make sense to move this into web5-kt similar how AWSKeyManager is in there.
Some considerations:

Don't want to bring in the android libraries for non android projects
Want to build with android libraries
Need to test with Android environment (a github action in the example project shows how: https://github.com/TBD54566975/tbdex-android-example/blob/main/.github/workflows/integration.yml)

cc @ALRubinger for thoughts on packaging and testing.

Packaging, testing, bringing it together: #287


Module Flattening may break relative imports

Issue Description:
My project have such structure.
foo/
  |- bar/
  |  |- __init__.py
  |  |- core.py
  |- __init__.py
  |- constant.py

Inside foo.bar.core, I have a relative import to the foo.constant as below.
from ..constant import some_thing

It seems like the module flatten will not modify the relative imports to the correct level which cause
ValueError: attempted relative import beyond top-level package

when trying to import foo.bar.core using complied library.
Using absolute imports solves the problem with no issue.
from foo.constant import some_thing

I know it is not a good design pattern, and it will be good to support it.
Possible remedy:

Always transform the relative imports to absolute imports before flatten

Well, this tool was developed with absolute imports in mind. I'll see if it's possible to do the transformation

Added support for relative imports. Please upgrade to v0.3.0 to use it

Thank you for the quick update!


Fixed: printinfo's 'histogram' output

fixes issue #2217.
Let me know if there should be any changes.

@Shivakumar-SG It looks like the build failed due to not being formatted correctly. You can run something like mvn package -DskipTests at the root of the project and it will apply the proper formatting for you. Include those changes into a new commit for this PR and the build should pass from there.

I have been trying from past few hours to run mvn package -DskipTests, but its throwing me many errors and I couldn't fix them. What would you want me to do?

@Shivakumar-SG what errors are being thrown?

@Shivakumar-SG what errors are being thrown?

A whole lot many actually :(

if the build failed because of just the formatting, it would be great if someone could help me with the formatted code of my version

if the build failed because of just the formatting, it would be great if someone could help me with the formatted code of my version

Sure, I can make a PR to your branch with the formatting changes.

@DomGarguilo That would be great! Thank you so much

Can you provide more information?

What platform are you running on?
What command are you running?
What is the first or  first few errors that you are getting?

You can do the same build as the verification checks, but skipping all of the tests by running:
mvn package javadoc:jar -Psec-bugs -DskipITs=true -DskipTests=true
The javadoc and sec-bugs will add to the build time, but running those before submitting the PR can highlight issues that will fail the automatic build.  This should also format the files to the community standards.
More information on building is at (Building Accumulo](https://accumulo.apache.org/contributor/building)

It looks like you are missing the import for 'bigNumberForSize()' which is NumUtil.

It also looks like an extra file (.DS_Store) was pushed along with you commit - that file should not be included.

It looks like with these changes the output is something like:

Is this what we are looking for?

It looks like you are missing the import for bigNumberForSize() which is import org.apache.accumulo.core.util.NumUtil

Initially, I did import initially and when the command mvn package -DskipTests is run, it takes it out.

Initially, I did import and when the command mvn package -DskipTests is run, it takes it out.

That is most likely due to the import being unused since you weren't calling the function correctly (as suggested above). One of the format rules removes all unused imports.

Is this what we are looking for?

That output looks good to me.

For discussion on output format - it looks like there are some options - using powers pr 10 or 2.  To produce a sample:
    for (int x = 1; x < 15; x++) {
      System.out.println(
          String.format("%s%11s", " ", NumUtil.bigNumberForQuantity((long) Math.pow(10, x))));
    }
    System.out.println("---");
    for (int x = 1; x < 16; x++) {
      System.out.println(
          String.format("%s%11s", " ", NumUtil.bigNumberForSize((long) Math.pow(16, x))));
    }

This produces:
          10
         100
       1.00K
      10.00K
     100.00K
       1.00M
      10.00M
     100.00M
       1.00B
      10.00B
     100.00B
       1.00T
      10.00T
     100.00T
---
          16
         256
       4.00K
      64.00K
       1.00M
      16.00M
     256.00M
       4.00G
      64.00G
       1.00T
      16.00T

Having even numbers seems more pleasing (to me) base 2 or 10 work, but being the buckets are calculated using 100, bigNumberForQuantity might be easier to implement with this PR.  Being this is size, an argument for the second option is valid, but might require additional work to calculate the buckets correctly.

Having even numbers seems more pleasing (to me) base 2 or 10 work, but being the buckets are calculated using 100, bigNumberForQuantity might be easier to implement with this PR. Being this is size, an argument for the second option is valid, but might require additional work to calculate the buckets correctly.

Good point, I concur that Quantity might be preferable since it lines up closer to the original output.

@Shivakumar-SG What are your thoughts on the conversation above?

Having even numbers seems more pleasing (to me) base 2 or 10 work, but being the buckets are calculated using 100, bigNumberForQuantity might be easier to implement with this PR. Being this is size, an argument for the second option is valid, but might require additional work to calculate the buckets correctly.

@Shivakumar-SG What are your thoughts on the conversation above?

Having even numbers seems more pleasing (to me) base 2 or 10 work, but being the buckets are calculated using 100, bigNumberForQuantity might be easier to implement with this PR. Being this is size, an argument for the second option is valid, but might require additional work to calculate the buckets correctly.

@Manno15 Thank you for showing your interest in my views but I'm just a student who is still learning and knows pretty much nothing about this repo. So I don't think so that I'm fit to share my views on such a big project.
I will just do what you tell me to do sir!

Thanks for the contribution @Shivakumar-SG and welcome to the Apache Accumulo community! If you would like to be listed as a contributor on Accumulo's people page please make a website PR for people.md. https://github.com/apache/accumulo-website/edit/main/pages/people.md
If you intend to make more contributions, please consider subscribing to the dev list and introducing yourself. Also, you can use the dev list or slack to reach out if you have questions about anything. https://accumulo.apache.org/contact-us/

I will consider your options. Thank you so much for letting me do this.

Thanks for the contribution Shivakumar-SG and welcome to the Apache Accumulo community. If you would like to be listed as a contributor on Accumulo's people page please make a website PR for people.md. https://github.com/apache/accumulo-website/edit/main/pages/people.md by:

adding your name (in alphabetical order) to the contributor's list.
reference the PR number (#2220)
If you intend to make more contributions, please consider subscribing to the dev list and introducing yourself. Also, you can use the dev list or slack to reach out if you have questions about anything. https://accumulo.apache.org/contact-us/

Thank you for helping me out in this @EdColeman and @DomGarguilo :)


[Improvement][User] Query all users filter un enabled.

Purpose of the pull request
Query all users filter un enabled
Brief change log
#11503
Verify this pull request
Manually verified the change by testing locally.

Codecov Report

Merging #11504 (9bab063) into dev (2a7e0be) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##                dev   #11504   +/-   ##
=========================================
  Coverage     39.36%   39.36%
  Complexity     4630     4630
=========================================
  Files           987      987
  Lines         37566    37567    +1
  Branches       4184     4182    -2
=========================================
+ Hits          14789    14790    +1
+ Misses        21226    21225    -1
- Partials       1551     1552    +1

Impacted Files
Coverage Δ

...inscheduler/api/service/impl/UsersServiceImpl.java
72.12% <100.00%> (+<0.01%)
:arrow_up:

...erver/master/processor/queue/TaskEventService.java
75.00% <0.00%> (-5.36%)
:arrow_down:

...uler/plugin/task/api/k8s/impl/K8sTaskExecutor.java
35.00% <0.00%> (-1.54%)
:arrow_down:

...r/plugin/task/sqoop/parameter/SqoopParameters.java
55.12% <0.00%> (-1.29%)
:arrow_down:

...nscheduler/service/process/ProcessServiceImpl.java
31.87% <0.00%> (ø)

...api/service/impl/ProcessDefinitionServiceImpl.java
32.27% <0.00%> (ø)

.../server/master/runner/StateWheelExecuteThread.java
0.40% <0.00%> (ø)

...org/apache/dolphinscheduler/alert/AlertConfig.java
0.00% <0.00%> (ø)

...lphinscheduler/plugin/task/api/utils/K8sUtils.java
3.84% <0.00%> (ø)

...inscheduler/plugin/task/datax/DataxParameters.java
36.48% <0.00%> (+2.20%)
:arrow_up:

... and 1 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Error with accimage

I am running Python version 3.8.10 with torch version '1.9.0+cu111' and torchvision version 0.2.1. I installed additional packages as instructed in the INSTALL.md file.
Despite this, upon testing, I encountered the following error message.
2023-04-12 10:52:45,419 fcos_core.inference INFO: Start evaluation on cityscapes_foggy_val_cocostyle dataset(500 images).
  0%|                                                                                | 0/125 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "tools/test_net.py", line 112, in <module>
    main()
  File "tools/test_net.py", line 96, in main
    inference(
  File "/root/autodl-tmp/UDA/SIGMA/fcos_core/engine/inference.py", line 87, in inference
    predictions = compute_on_dataset(cfg, model, data_loader, device, inference_timer)
  File "/root/autodl-tmp/UDA/SIGMA/fcos_core/engine/inference.py", line 23, in compute_on_dataset
    for _, batch in enumerate(tqdm(data_loader)):
  File "/root/miniconda3/lib/python3.8/site-packages/tqdm/std.py", line 1195, in __iter__
    for obj in iterable:
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 521, in __next__
    data = self._next_data()
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1203, in _next_data
    return self._process_data(data)
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1229, in _process_data
    data.reraise()
  File "/root/miniconda3/lib/python3.8/site-packages/torch/_utils.py", line 425, in reraise
    raise self.exc_type(msg)
AttributeError: Caught AttributeError in DataLoader worker process 0.
Original Traceback (most recent call last):
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/_utils/worker.py", line 287, in _worker_loop
    data = fetcher.fetch(index)
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 44, in <listcomp>
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/root/autodl-tmp/UDA/SIGMA/fcos_core/data/datasets/coco.py", line 94, in __getitem__
    img, target = self.transforms(img, target)
  File "/root/autodl-tmp/UDA/SIGMA/fcos_core/data/transforms/transforms.py", line 15, in __call__
    image, target = t(image, target)
  File "/root/autodl-tmp/UDA/SIGMA/fcos_core/data/transforms/transforms.py", line 59, in __call__
    image = F.resize(image, size)
  File "/root/miniconda3/lib/python3.8/site-packages/torchvision/transforms/functional.py", line 188, in resize
    if not _is_pil_image(img):
  File "/root/miniconda3/lib/python3.8/site-packages/torchvision/transforms/functional.py", line 19, in _is_pil_image
    return isinstance(img, (Image.Image, accimage.Image))
AttributeError: module 'accimage' has no attribute 'Image'

The issue appears to stem from accimage, which is a pre-installed package in torchvision. This is unexpected because the version of torchvision being utilized is accurate.

Hi, thanks for your interest!
The package version in README.md is for 10.X CUDA. For CUDA 11.X, you have to install the latter pytorch and the corresponding matched torchvision, since the GPU doesn't support the old version. You can check this issue https://github.com/CityU-AIM-Group/SIGMA/issues/23

I have managed to solve the problem by assigning accimage to None. I will closed this issue for now.


merge remote branch teracy-dev/develop

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 4 committers have signed the CLA.:x: hieptranquoc:x: hoatle:x: datphan:x: hoavtYou have signed the CLA already but the status is still pending? Let us recheck it.


introduce report generator and corresponding api

The report generator class consists of two parts, a data collector and a document creator.
The data get fetched by meta data from the subcontainer and stored/copied into a tmp file, sorted by the items in folders.
After analyze description on image references (if no image reference, get added automatically at the end of description) and adding '#' character to each headline, a process iterate over all data and create a final pdf document (named by data and time).
Some additional notes:

no filter is established yet, cause functionality of the api is just tested by url (so no filter is possible)...so default state is loading the data by latest achievement
achievement as itself is so far not integrated in the report doc
some lines may be are not coded universally enough (not clear how much different formats of e.g. images should be provided or even other types of attachment/media)
good to test functionality with 'check-for-displaying-correct-description.py' (because two different descriptions in the items....different by two characters at the end of each description)

Signed-off-by: Alexander Sarmanow<EMAIL_ADDRESS>

From a function perspective, fine. From a maintainable, understandable, extendable perspective ... not that ideal.
What I assume:
class ReportGenerator:

      @staticmethod
      def generate(output=ReportGenerator.PDF, filter=ReportGenerator.LAST_ACHIEVEMENTS):
               list_of_list = ReportGeneratorCollector(filter)
               tmp_path = mkdir.mkdir()
               pdf_out_path = os.path.join(tmp_path, "10-10-10-report.pdf)
               rpd = ReportGeneratorDocument(list_of_list)
               # generate markdown document based on all files, copy everything
               # into own mkdir.mkdir()
               rpd.generate(pdf_out_path)

      class ReportGeneratorDocument:

      class ReportGeneratorCollector:
           def search(filter=ReportGenerator.LAST_ACHIEVEMENTS):
                    if filter == ReportGenerator.LAST_ACHIEVEMENTS
                            return search_last_achievements()

           def search_last_achievements():
                    # iterating over database and return all achievements
                    return [ [<PHONE_NUMBER>, 332923932, 0 ] ]

Futhermore, hippod/api_report.py should be nearly empty, all major functionality should be in hippod/report_generator.py.
hippod/api_report.py should contain more or less the following functionality:
def generate_doc(app, filter):
    # convert filter into a filter DocumentGenerator
    # can deal with it. e.g. DocumentGenerator.FILTER_LAST_ACHIEVMENTS
    # later DocumentGenerator.FILTER_BY_LABEL with a list of label strings or
    # DocumentGenerator.FILTER_BY_SUBMITTER with list of submitter names
    # hand over filter to DocumentGenerator( ...)
    # if something buggy happen DocumentGenerator will raise an ApiError as
    # usual

def handle(request):
    if request.method != "GET":
        msg = "Internal Error...request method: {} is not allowed".format(request.method)
        raise hippod.error_object.ApiError(msg)
    app = request.app
    # try/catch case when no JSON data from client
    filter = await request.json()
    try:
        start = time.clock()
        generate_report(app, xobj):
        end = time.clock()
    except ApiError as e:
        return e.transform()

    o = hippod.ex3000.Ex3000()
    o['data'] = data
    o['processing-time'] = "{0:.4f}".format(end - start)
    o.http_code(204)
    return o.transform()


Fragment on interface is not working

flutter: "3.3.3"
graphql_codegen: 0.11.1
graphql: 5.1.1
schema:
interface InterfaceA {
   id: ID!
   name: String!
}

type TypeA implements InterfaceA {
  id: ID!
  name: String!
}

type SomeData {
  data: TypeA!
}

query:
fragment FragmentA on InterfaceA {
    id
    name
}

query someQuery {
  data {
    ...FragmentA
  }
}

Server works just fine - data is there, but its not appearing in QueryResult - not even in .data
When I pull parsedData ->
type 'Null' is not a subtype of type 'String' in type cast
In generated code - there is
const fragmentDefinitionFragmentA = FragmentDefinitionNode(
  name: NameNode(value: 'FragmentA'),
  typeCondition: TypeConditionNode(
      on: NamedTypeNode(
    name: NameNode(value: 'InterfaceA'),
    isNonNull: false,
  )),

if I change name: NameNode(value: 'InterfaceA'), InterfaceA on  TypeA suddenly it's working.

Faced this issue, had to add __typename in queries.

Hmm, placed __typename like so
query someQuery {
  data {
    __typename
    ...FragmentA
  }
}

it did not worked

If the data isn't even present, then it's probably not an issue with the code generator.

Data is present, I see it on http logger.
I solved it just now, it was cache normalization issue. Passed down generted possibleTypesMap and it worked. Didn;t know it was a thing.
Closing the issue

Data is present, I see it on http logger.
I solved it just now, it was cache normalization issue. Passed down generted possibleTypesMap and it worked. Didn;t know it was a thing. Closing the issue

Seems to be facing the same issue. Can you detail how you solved yours ?


Feature/c idev

Description
A new workflow was added for running the SWELL Tier-1 HOFX experiment using the latest nightly JEDI Build. This workflow and the nightly workflow can be manually launched. However, it is not clear if labels can be used to constrain initiation of manual workflows. This PR will enable all authorized users to execute a workflow.
Some other important points:
(1) We may want to add additional triggers to the un-scheduled workflows that we develop. Do we want the Tier-1 HOFX experiment to automatically execute on a PR?
(2) Manual execution of the nightly CI workflow will block other manual runs and the scheduled run if overlap occurs.
(3) The workflows are now configured to acquire the SWELL code from the branch that initiates the workflow. Scheduled workflows will always use the main branch unless we explicitly encode the branch name to acquire in the CI workflows.
(4) Please note that the CI workflow for the nightly run (under the CI-workflows repo) was modified to only update the "latest" build pointer if the Tier-1 HOFX experiment succeeds.

You also have some of your own YAML coding norm issues here:
./.github/workflows/discover-tier1-hofx.yml
  12:1      error    trailing spaces  (trailing-spaces)

./.github/workflows/discover_nightly.yml
  7:1       error    trailing spaces  (trailing-spaces)