Add a better status message

We are adding a better status message that will include the Namespace and the PipelineRun.
We link the Namespace to the Namespace view in the consoles.
Introducing a new setting for that custom-console-url-namespace that would need to be setup when using a custom console.

Changes

Submitter Checklist

[ ] 📝 A good commit message is important for other reviewers to understand the context of your change. Please refer to How to Write a Git Commit Message for more details how to write beautiful commit messages. We rather have the commit message in the PR body and the commit message instead of an external website.
[ ] ♽ Run make test before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI. (or even better install pre-commit and do pre-commit install in the root of this repo).
[ ] ✨ We heavily rely on linters to get our code clean and consistent, please ensure that you have run make lint before submitting a PR. The markdownlint error can get usually fixed by running make fix-markdownlint (make sure it's installed first)
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

lgtm
cc @savitaashture

/retest

/test linters

I installed this PR on dogfooding cluster you can see the example here:
https://github.com/openshift-pipelines/pipelines-as-code/pull/1492/checks?check_run_id=18482061202
redid most of the test reconciler to test this properly (it didn't before) with golden


Concurrent mouse presses results in crash during game.

Godot version:
Unofficial Nightly Build 11/28/2018 - https://hugo.pro/projects/godot-builds/
OS/device including version:
Windows 10
GeForce GTX 980Ti (2x SLI) - Up to date Nvidia drivers
Issue description:
When more than one mouse button is concurrently clicked during a running game, the game crashes with:
ERROR: _get_socket_error: Socket error: 10054
  At: drivers/unix/net_socket_posix.cpp:190

Steps to reproduce:
Create a new project.
Create a 3D Spatial Node.
Play (F5) the scene.
Hold Any mouse button, and press another.
Game crashes.
Notes
I am not currently able to validate that this behavior exists in the current master with a clean build.

Happens with a scene using a Node2D root as well.
Also using Windows 10 and build 5b8dde4

Still same error occurs in master (f730d18)

CC @guilhermefelipecgs

@qarmin same backtrace?

ok, I found other situations that the error occurs, this PR #24125 should cover all.


#1574 #1579

#1579 Crash viewing in Excel in specific scenario

check for misc files solution
add catching com exceptions just in case

#1574 It is possible to get duplicate signature sessions

minor issue when caret is at the end of an unterminated string
fix tokenizer so it actually creates token when opening quote is at the very end of the file

:+1:


Validate scriban from cli

Summary of the Pull Request
What is this about?
Adds a cli command onefuzz debug notification template. You pass a template string and it will attempt to render it for you via scriban.
Help text:
$ onefuzz debug notification template -h
usage: onefuzz debug notification template [-h] [-v] [--format {json,raw}] [--query QUERY] [--context TemplateRenderContext] template

positional arguments:
  template

options:
  -h, --help            show this help message and exit
  -v, --verbose         increase output verbosity
  --format {json,raw}   output format
  --query QUERY         JMESPath query string. See http://jmespath.org/ for more information and examples.
  --context TemplateRenderContext
                        JSON for TemplateRenderContext. use @file to read from a file

Closes #2501
Info on Pull Request

Adds the new template subcommand
Adds some new C# types that weren't migrated to pytypes
Update TaskConfig type to match C#

What does this include?
Validation Steps Performed
How does someone test & validate?
I ran the command with some valid and invalid scriban template strings to see if they're rendered correctly.

Codecov Report

Merging #2800 (c9e9fa1) into main (8f418f6) will increase coverage by 1.60%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2800      +/-   ##
==========================================
+ Coverage   29.93%   31.54%   +1.60%
==========================================
  Files         324      199     -125
  Lines       38990    25754   -13236
==========================================
- Hits        11671     8123    -3548
+ Misses      27319    17631    -9688

Impacted Files
Coverage Δ

src/ApiService/ApiService/Functions/Pool.cs

...ice/ApiService/onefuzzlib/NodeMessageOperations.cs

...piService/ApiService/Functions/QueueProxyUpdate.cs

src/ApiService/ApiService/onefuzzlib/Storage.cs

src/ApiService/ApiService/HttpClient.cs

...ce/ApiService/TestHooks/PoolOperationsTestHooks.cs

...rc/ApiService/ApiService/onefuzzlib/orm/Queries.cs

...c/ApiService/ApiService/onefuzzlib/VmOperations.cs

src/ApiService/ApiService/OneFuzzTypes/Enums.cs

...rvice/ApiService/onefuzzlib/orm/EntityConverter.cs

... and 115 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Update flow misc. issues

The current flow for updating filters is pretty confusing. This issue will bundle a few of those together, and explain some alternatives. For each issue, I will provide commands you can run in a fresh project to see the issue.
For this example, the newest version of texture_list was 1.1.2, and I use 1.1.0 to represent an older state.
Feedback is bad for 'pinned' versions.
By default when you install a filter, it is installed 'pinned' to the current version, such as 1.1.0. Pinned versions don't really play nice with regolith update, since by definition they are pinned.
regolith init
regolith install texture_list==1.1.0 (get a older version -simulates an older installation)
regolith update texture_list

This gives you a message like this:
PS C:\SirLich\git\regolith-filters\test> regolith update texture_list
[INFO]  Updating filters...
[INFO]  Filter "texture_list" is up to date. Instaleld version: "1.1.0". Skipped update.
[INFO]  Successfully updated the filters.
[INFO]  Finished

Proposal 1) An idea to fix this is that regolith update will actually bump pinned repos properly. Equivalent to a fresh install. So the new feedback would be:
[INFO]  Filter "texture_list" updated from "1.1.0" -> "1.1.2"
Proposal 2) A second idea, would be to simply write better feedback. Something like this:
[INFO]  Filter "texture_list" is pinned at version 1.1.0
    >> Version 1.1.2 is available
    >> Run with '--force' to update

Please note that this proposal requires you to actually add the concept of --force into regolith update and regolith update-all
Update doesn't allow specifying a version
For example, this command has very confusing feedback.
PS C:\SirLich\git\regolith-filters\test> regolith update texture_list==latest
[INFO]  Updating filters...
[WARN]  Filter "texture_list==latest" is not installed and therefore cannot be updated.
[INFO]  Successfully updated the filters.
[INFO]  Finished

Proposal 1) The regolith update command should allow specifying an exact version as shown.
Proposal 2) The regolith update command should complain if == is used, and explain what's going wrong properly.
Regolith install --force simply doesn't work
See this command:
PS C:\SirLich\git\regolith-filters\test> regolith install texture_list==1.0.0 --force
[INFO]  Installing filters...
[ERROR] Failed to install filter "texture_list==1.0.0".
[+]: The filter "texture_list" is already on the filter definitions list.
   >> Please remove it first before installing it again or use the --force option.

Proposal 1) The --force keyword should actually work...

Another issue is that regolith init cannot be run in a non-empty folder. For example, a folder where you just ran git init.
I think that either regolith init should allow creation in this case, or maybe just allows a --force option that removes safeguard.


Backport #8009 into 3.0?

It would be super nice to have the fix for #8009 backported into 3.0.x.
@wojtek-t @mml @jpbetz

Unlikely to happen. Since the fix doesn't sync down the free list, booting into the patched etcd then booting into an older version of 3.0 will leak free pages at best / corrupt the db at worst.

What does this imply for our ability to roll back from a fixed 3.1.x to a broken 3.0.x?

@mml 3.3.0 will have the patch and proper rollback support. Issuing a rollback would restore the backend to the slow mode with free lists before reverting the cluster version to 3.2.

@heyitsanthony If we only backported "Garbage collect pages allocated after minimum txid" (https://github.com/coreos/bbolt/pull/3) to etcd 3.0.x and 3.1.x. We might be able to resolve the most pressing issues with #8009.
We would either need to get https://github.com/boltdb/bolt/pull/694 merged or build a version of bbolt that contains https://github.com/coreos/bbolt/pull/3 but not https://github.com/coreos/bbolt/pull/1. After that, Etcd would just pick up the new versions for the minor release of 3.0.x and 3.1.x.
Does this sound reasonable? I'll be available to contribute.

We might be able to resolve the most pressing issues

We tried it, and it will not solve what #8009 hit. A sudden free pages release due to compaction (and a previous spike on page usage) will still trigger the problem unless we stop syncing the free pages.
coreos/bbolt#3 helps more on reducing page usage on concurrent read/write txn case.

Thanks @xiang90 How did you try it? Do you have a way to replicate #8009?

@xiang90 we are also considering to back port the PRs to 3.1.9 to fix the  "database space exceed" issue, Is there any way to do this ?

The backport policy is documented here: https://github.com/coreos/etcd/blob/master/Documentation/branch_management.md
We could backport patches to more than one minor releases in theory, but given the people we have today, it is not feasible. I am closing this.


Remove extra NUFFT computation if user passes in specific baselines

As the simulation function is currently written, fftvis performs separate NUFFTs for the positive and negative u-modes if a complex beam is passed in. This is currently done even if the user specifies certain baselines to be simulated instead of the entire array, potentially doubling the amount of work the simulation function does. This PR fixes that issue

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

Project coverage is 92.91%. Comparing base (6d895ab) to head (0e2945e).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #14   +/-   ##
=======================================
  Coverage   92.91%   92.91%
=======================================
  Files           4        4
  Lines         127      127
  Branches       35       35
=======================================
  Hits          118      118
  Misses          5        5
  Partials        4        4

Flag
Coverage Δ

unittests
92.91% <100.00%> (ø)

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

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


show seconds

Hi.
Would it be possible to show in which second each of the three frames was found?
Don't know anything about programming but think it would be useful for the application.
Congratulations for the work.

This should be quit easy to do! I'll implement it tomorrow and let you know

thanks man. this will help me with video editing.

well done,i have done the same job recently.
i want to know  whether you store the extracted image on the google drive,if no,how can i search when i do this next time.
and the colab can only work for sometime,what if i want to deploy on my computer,does it need high-efficiency computer?

further,any progress on training my own data model,cause sometimes i need more accurate result on specific man or things?

much better than i imagined. thanks again

thanks for your response


Docai forms

Includes the following changes:

DocAI based online classifier utility
DocAI based batch classifier utility
DAG changes incorporating the online classifier
IAM changes for Airflow to be able to call DocAI APIs

I tested it in my GCP environment. It works!


[filter] - allow operands and sub queries

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request => Please do not submit support request here, instead see https://github.com/ceolter/ag-grid/blob/master/CONTRIBUTING.md#question

Current behavior
You can currently set filters for multiple columns using:
let hardcodedFilter = {
country: ['Ireland', 'United States'],
age: {type: 'lessThan', filter: '30'},
athlete: {type: 'startsWith', filter: 'Mich'},
date:{type: 'lessThan', dateFrom: '2010-01-01'}
};
gridOptions.api.setFilterModel(hardcodedFilter);
gridOptions.api.onFilterChanged();
Desired behavior
The current code essentially ANDs all those conditions together - it would be great if you could do more complex filters - at least via the API - I can see how doing anything but AND via the current GUI would be problematic, but API or a new filter GUI in the Toolpanel could allow more advanced options - API only is sufficient for my needs at the moment:
// ActB gt 0 or ActP gt 0
let basicor = {
ActB: {type: 'greaterThan', filter: '0'},
ActP: {type: 'greaterThan', filter: '0'},
Operand: 'or'
}
// ActB eq 0 and ActP eq 0 and (NumB gt 0 or NumP gt 0)
let complex = {
ActB: {type: 'equal', filter: '0'},
ActP: {type: 'equal', filter: '0'},
Sub1: [{ NumB: {type: 'greaterThan', filter: '0' }, { NumP: {type: 'greaterThan', filter: '0' }, Operand: 'or' ]
Operand: 'and'
}
I just put the subquery in an array to easily differentiate it - there might be a better way.  I'm migrating a project from infragistics silverlight grid to ag-grid and need this feature because we have quite a few complex filters saved away we'd like to keep.
It looks like this would be a small self contained change to rewrite doesFilterPass in filterManager.ts - which I might attempt assuming I can figure out how to build everything but wanted to check if there was an easier way and/or get feedback on my approach?  (One minor problem I see with above is if they had a column with Operand as the colId, that would be problematic but how likely is that).

thank you for raising. i have created a JIRA for this AG-424, so it will get considered next times we decide what gets into a release.
in the mean time, it is possible to have your own complex filters using an API. we call this External Filter
https://www.ag-grid.com/javascript-grid-filter-external/

Hi,
We are changing the process and closing all issues with a ticket number since they can be tracked from our pipeline https://www.ag-grid.com/ag-grid-pipeline/
Thanks
===============
If you are an ag-Grid Enterprise Customer, please raise this in our Zendesk support system - contact<EMAIL_ADDRESS>for access. This is our primary channel for Support.
Bugs Raised Here:
We will do a periodic review of GitHub issues for any bugs raised and these can be tracked on our pipeline page: https://www.ag-grid.com/ag-grid-pipeline
Feature Requests/Support Raised Here:
We are reviewing how we process Feature Requests that come from GitHub. We currently have a very full pipeline of work which will be our priority for the coming months.
As a result, we don’t have capacity to take feature requests from GitHub.
We will review this as we move through our current backlog.
Thanks for your understanding.
ag-Grid Team.


Insights Configuration Proposal

Hey,
I've tried to understand, why we even need initializer here, and I was not able to find a good answer for that.
First proposal, store configuration values in the application configuration. Configuration should be split by environment:
var ENV= {
analyticsInsights: {
    development: {
      debug: true,

      insights: {
        ALL_TRANSITIONS: true,
        ALL_ACTIONS: false
      },

      trackers: {
        googleAnalytics: {
          setup: {
            webPropertyId: 'UA-2318996-47',
            linkid: true,
            displayFeatures: true
          },

          config: {
            trackTransitionsAs: 'pageview',
            fields: {
              version: ENV.APP.version
            }
          }
        }
      }
    }
  }
}

If you take a closer look, you can see some benefits from such configuration:

[x] You specify environment, so no need to manually initiate tracking.
[x] You specify debug mode for each environment
[x] You specify as many trackers as you want by its name

Based on this config, addon is able to start Insights tracker on its own, register it in the container and if anyone wants to use it in their components/routes/controllers etc, will create his own initializer to inject this dependency.
As we talked before, I'd like to use several trackers, first of them are going to be GA, Yandex.Metrics and Customer Metrics (they are gonna be saved into the local storage or just variable, and flushed to the server when XX metrics is gathered, later on stored in the ELK for insights)
As I don't want to create my own bicycle I want to focus my effort on something that is already in development and contirbute :)
I'd like to know what you guys think about or maybe some different ideas?

Sure. I love to help you. Let's make a time for facilitated hangout. As a result, we'll produce a bunch of clear and useful issues.


registry: Remove reference.go

This removes some very old vestigial code that really should have been removed during the content addressability transition. It implements something called "reference" but it behaves differently from the actual reference package. This was only used by client-side content trust code, and is relatively easy to extricate.
cc @dmcgowan @cyli @endophage

LGTM

LGTM


Yet another apporach of copying logical timeline size during branch creation

Alternative to #2117

Some tests fail:
2022-07-21T14:06:54.8984360Z failures:
2022-07-21T14:06:54.8984527Z     layered_repository::tests::test_traverse_ancestors
2022-07-21T14:06:54.8984739Z     layered_repository::tests::test_traverse_branches
2022-07-21T14:06:54.8984919Z     repository::tests::test_branch
2022-07-21T14:06:54.8985122Z     repository::tests::test_parent_keeps_data_forever_after_branching
2022-07-21T14:06:54.8985362Z     repository::tests::test_retain_data_in_parent_which_is_needed_for_child
2022-07-21T14:06:54.8985623Z     repository::tests::timeline_load_with_ancestor

Some tests fail:
2022-07-21T14:06:54.8984360Z failures:
2022-07-21T14:06:54.8984527Z     layered_repository::tests::test_traverse_ancestors
2022-07-21T14:06:54.8984739Z     layered_repository::tests::test_traverse_branches
2022-07-21T14:06:54.8984919Z     repository::tests::test_branch
2022-07-21T14:06:54.8985122Z     repository::tests::test_parent_keeps_data_forever_after_branching
2022-07-21T14:06:54.8985362Z     repository::tests::test_retain_data_in_parent_which_is_needed_for_child
2022-07-21T14:06:54.8985623Z     repository::tests::timeline_load_with_ancestor

Just fixed it.


Create Proxy backend collection and UI for mandatory fields

In order to decouple API backends from Api Umbrella, we will need a collection and schema for proxy Api settings (currently only Umbrella ) settings that is separate from "pure" API backend settings/details. As a first step, this collection will contain the mandatory API Umbrella fields like backend and frontend prefix, backend host URL.
For the UI, we can either add a Proxy tab or a partial view in Settings tab to connect the API to a proxy to and to configure the backend.
The page should have a possibility to select an existing API proxy and define the backend configuration. Use the opportunity to explain frontend and backend prefix in a more user friendly manner, possibly selecting new terminology.
The tab/view should be only available if platform administrator has created a proxy and it is only shown to administrators and the API manager
Definition of done

[ ] Collection and schema

[x] There is a collection for Proxy API backend
[x] There is a schema for Proxy API Backend
[x] The collection is connected to Api Umbrella
[x] The schema contains all mandatory API Umbrella fields
[x] The schema is aligned with Api Umbrella
[ ] The data in collection gets stored properly in API Umbrella

[ ] UI

[ ] There is a form for backend settings either in settings or in its own tab
[ ] Proxy tab or partial view for selecting proxy exists
[ ] User can select an existing proxy
[ ] After selecting the proxy, user is required to fill in the mandatory backend configuration details
[ ] Ensure inline help text clarifies the meaning of each field
[ ] Optionally, come up with easier terms for frontend and backend prefix

[ ] When the settings are saved, a relational link is made between API backend document and Proxy API backend document

@jykae Would you be interested in this? Related to #1091, need to work in close cooperation.

User interface work > create task
Big testing impact

After discussion, we decided to name the Proxy APIs collection ProxyApis. We are not clear on how the schema will be structured, but have some initial thoughts:
# Proxies
- _id
- name
- description ?
- type (allowed values: 'api-umbrella', 'kong', 'tyk')
- settings
  - url
  - apiKey
  - authToken
    - custom validation: required if `type` === 'api-umbrella'
  - elastic
    - custom validation: required if `type` === 'api-umbrella'

# ProxyApis
- apiId
- proxyId
- settings
  - apiUmbrella [apiUmbrella_Api schema]
   - rate_limiting

For future reference Kong API object schema: https://getkong.org/docs/0.9.x/admin-api/ Look section "API object" fields we need for Kong API schema.

@frenchbread right now we need to remove references to Kong/Tyk from the code and user interface. The form can keep the dynamic functionality, but should only allow the user to select API Umbrella. Default the selection to API Umbrella, if possible.

@brylie How can we implement this? I was able to set the default selection (apiUmbrella) for the selectbox, but it did not force autoform automatically render APIUmbrella schema.

Hm, it seems harder than I thought. How did you set the 'type' field value? Did you do it via the schema?

Lets ask @bajiat for input here.

Comments from chat:

remove the API Umbrella settings from the initial setup wizard. I am in the process of removing the initial setup wizard in another branch, if you want to mirror the changes
For now, perhaps remove the dynamic toggle. Rather, just include the apiUmbrella.settings field and sub-fields directly in the AutoForm.
Hide the 'type' field from view, and default the value to apiUmbrella in the schema.
add inline help texts and icons for manditory fields.

Check if the setup wizard contains help text strings, and use those in your Proxy form.

Note: Discussion in this issue is related to Proxy settings (adding a proxy to the deployment/platform) and not backend settings, but the issue itself is about connecting a single API to a proxy.

@Nazarah here are the manditory fields:
"name": {
    type: String,
    optional: true
  },
  "frontend_host": {
    type: String,
    optional: true
  },
  "backend_host": {
    type: String,
    optional: true
  },
  "backend_protocol": {
    type: String,
    optional: true
  },
  "balance_algorithm": {
    type: String,
    optional: true
  },
  "url_matches": {
    type: [Object],
    optional: true
  },
  "url_matches.$.frontend_prefix": {
    type: String,
    optional: true
  },
  "url_matches.$.backend_prefix": {
    type: String,
    optional: true
  },
  "servers": {
    type: [Object],
    optional: true
  },
  "servers.$.host": {
    type: String,
    optional: true
  },
  "servers.$.port": {
    type: Number,
    optional: true
  }

Here is a current screenshot of the AutoForm based on the above schema:

@Nazarah here are brief descriptions of the fields:

name: a human friendly name for the Proxy API
frontend_host: server address for the Proxy (might not be on the form)
backend_host: URL of the API server
backend_protocol: protocol to access API (e.g. HTTP or HTTPS)
balance_algorithm: how the load balancer should balance traffic
url_matches: path prefixes from the Proxy and API

frontend_prefix: prefix added to the Proxy path in all requests
backend_prefix: prefix added to the API path in all requests

servers: one or more servers responsible for handling API traffic (probably for load balancing)

host: URL where the API server can be accessed
port: port number where the API service is running

thanks @brylie !
I am adding the descriptions for INFO icons as per the fields given in your earlier screenshot.
As suggested, I changed the "frontend to "Proxy" and "backend" to "API"
Name: "Give a human friendly name to identify your proxy API." [feedback: we can skip info for this field. Not urgently needed.]
Proxy host: "Add the server URL  for your proxy API."
API host: "Add the server URL for your API."
Load balancer algorithm: "Mention the procedure to balance the load of API calls made." [feedback: any alternatives?]
Proxy Prefix: "The added prefix will be available in the proxy path of all requests. Example: "
API Prefix: "The added prefix will be available in the API path of all requests. Example: "
Host: "Add the URL from which the API server can be accessed."
Port: "Add the port number of the server in which the API service is running."

Based on @Nazarah's text suggestions and discussions with @jykae and @bajiat, here is the ProxyBackend form:


LogAddDiagnosticItems takes in array of layer names

Codecov Report
Base: 44.35% // Head: 44.36% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (a339183) compared to base (cb6c840).
Patch coverage: 33.33% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #165      +/-   ##
==========================================
+ Coverage   44.35%   44.36%   +0.01%
==========================================
  Files          48       48
  Lines        8033     8033
==========================================
+ Hits         3563     3564       +1
+ Misses       4273     4272       -1
  Partials      197      197

Impacted Files
Coverage Δ

axon/logging.go
0.00% <0.00%> (ø)

examples/boa/boa.go
83.75% <100.00%> (+0.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Need to integrate with GitHubActions

The tool is supposed to support Github Actions
Describe the solution you'd like
To be added
Additional context
Add any other context or screenshots about the feature request here.

Can people outside the core team work on the issues?

Can people outside the core team work on the issues?

Hi Karuppiah. Yes, absolutely! Regarding how to map GA workflow executions to our domain models, there are some docs available in our drive. I'm looking to move them to the Wiki very soon

Now working on integrating with Github Actions CI/CD tool.

Create, save and verify pipeline configuration (about 3 point)
Sync and save data (about 3 point)
UI change for config uration page (about 1 point)

Mostly done with Github Actions integration


Update NSHD questionnaires

Update/add NSHD questionnaires: SMWT, Chair Rises, and GHQ12:
Update Chair Rises questionnaire instructions as requested and fix field names
Adds new SMWT and GHQ12 questionnaires

LGTM


native_layer: Add support for delaying SliceBegin

Supports delaying the SliceBegin event, allowing us to put more debug_annotations into it before it is sent. This lets us use data collected from on_record to lazily add new annotations.

Merged with IRL review


Optimize the code of nacos discovery example and add comments.

I found that there is still system output in the example code of Nacos discovery example, and I hope it can be optimized to log output. At the same time, I hope I can optimize the code specification and add comments to the code for reading.

We are welcome you to create a PR to optimize it.

I think this document also needs to be updated.


react-native-android-wifi install and Link doesnt work

Hi there,
Followed the steps to install the library and tried linking both automatic and manually. When am trying to call the one of the method am getting an error "Undefined is not an object (evaluating "wifi.setEnabled"). Infact, same issue with other methods as well.
Please let me know if I missed any steps to use this library. Thank you.
Cheers,
Vinay.

Change the import to
import wifi from 'react-native-android-wifi';

Take a look to the Readme file and the Example app

@devstepbcn I tried that and it still doesn't work. I'm using expo, and react-native link shows this:
rnpm-install info Linking assets to ios project
rnpm-install ERR! Something went wrong while linking. Error: Cannot read property 'pbxprojPath' of null
Please file an issue here: https://github.com/facebook/react-native/issues

Cannot read property 'pbxprojPath' of null```

You mean I need to push the app into the device via Android SDK? I am using "react-native run-android" command to push the app

I am using a real device.

I'm using a real device too. I installed all dependencies, ran react-native link, react-native run-android (it should show at least 3 projects) and then react-native start

Think I did the same:

npm install react-native-android-wifi --save
react-native link (Done Manual as well)
import wifi from 'react-native-android-wifi'; then
ComponentDidMount() { wifi.getSSID((ssid) => { console.log(ssid); }); }

This is exactly I have done. Is there anything else I need to do.

Very wired but finally made it working. I have to restart the server, uninstall the app and install again. May be it is issue with WebStrom.


Add support building on Swift version 2.1.1

Xcode 7.2 beta  introduced Swift 2.1.1. Build fails due to returning Swift version 2.1.1. This PR will just create a symbolic link to RealmSwift-swift2.0 because Swift 2.1.1 does not break anything our code.
@jpsim @bdash

Thanks @kishikawakatsumi


pin gdal 2.1.x

Unfortunately fiona is not compatible with gdal 2.2.* yet, so we need to pin everything to 2.1.* is we want to install them along side with fiona. Note that installing the previous build number with latest gdal is still possible when not installing fiona.
I also update the boost pin to 1.64.*.
@gillins does it make sense to change the dependency here to libgdal instead of gdal? Or do you need the python bindings for this package?

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

does it make sense to change the dependency here to libgdal instead of gdal? Or do you need the python bindings for this package?

The python bindings are needed so better to keep as gdal

The python bindings are needed so better to keep as gdal

Thanks for the clarification @danclewley!

I guess that conda-inspect is taking to long :unamused: I will try commenting it out.
Removing test files
84 TESTS COMPLETED - 0 FAILURES LISTED BELOW:
+ conda inspect linkages -p /feedstock_root/build_artefacts/rsgislib_1500577686313/_t_env rsgislib

command ./ci_support/run_docker_build.sh took more than 10 minutes since last output

Sorry @ocefpaf I forgot all about this PR. Do you want to re-sync it, and then I'll merge it?

No problem. I used the GitHub web interface. Hopefully everything is alright.

Does the build number need to be updated also?

Does the build number need to be updated also?

Done!


Added IPv6 information to various Component documents

What this PR does / why we need it:
Added IPv6 related information to following documents:
docs/concepts/configuration/overview.md
docs/reference/generated/kube-apiserver.md
docs/reference/generated/kube-controller-manager.md
docs/reference/generated/kube-scheduler.md
docs/reference/generated/kubeadm.md
docs/reference/generated/kubelet.md
Which issue(s) this PR addresses:
addresses #53824

This change is

cc: @leblancd @rpothier @pmichali

Most of the files in the PR are auto generated, so they should not be updated manually. I will open a PR to update the auto generated docs. For this PR, please remove these files:

docs/reference/generated/kube-apiserver.md
docs/reference/generated/kube-controller-manager.md
docs/reference/generated/kube-scheduler.md
docs/reference/generated/kubeadm.md
docs/reference/generated/kubelet.md

We also need to remove kubelet.md from this PR, because it is auto generated. See #6658.

Changing base to master post-1.9 release
Please resolve the merge conflicts

@zacharysarah


Add docstring examples to collators

Codecov Report
Patch coverage is 100.00% of modified lines.

Files Changed
Coverage

src/gravitorch/data/datacreators/vanilla.py
ø

src/gravitorch/data/dataloaders/collators/base.py
ø

src/gravitorch/datapipes/map/factory.py
ø

src/gravitorch/utils/seed.py
ø

src/gravitorch/data/dataloaders/collators/pack.py
100.00%

src/gravitorch/data/dataloaders/collators/pad.py
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


feat: fast code edits

Introducing Fast Code Edits
https://github.com/carlrobertoh/CodeGPT/assets/15891408/ff049e54-daa0-4ca8-84ea-6bb4c836a6d6
We're moving into an era where natural language programming is gaining more attention. Therefore, I'm proud to introduce a new feature called Fast Code Edits. This feature is intended for users who want to edit their code directly via prompts.
The feature will be available for CodeGPT users first and can be accessed in two different ways:

Via CodeGPT's context menu

Via the editor's floating toolbar

Nice feature :) thank you for your time on this plugin, it definitely rocks!

Nice feature :) thank you for your time on this plugin, it definitely rocks!

Thank you for your feedback! This feature is still in beta, and I have more improvements planned :)

Sharing a blog post link for others to reference: https://codegpt.ee/blog/introducing-fast-edits

Ah, is this really only for the payed version? Thats such a shame!

If that is the case, at least let us disable the feature so that we don't have an option in our context menu that doesn't work!

@tvdiepen-reconi
I'm sorry you feel that way. This feature is available through CodeGPT's own provider, and it doesn't require you to pay in order to use it.
As stated on the website, we do give early access to new features to our own customers first, before launching them to others.
Hang tight, we plan to roll this out to other providers soon.


Compress corrupted on zip mode

I am trying to download the file in zip format & zip  file download on corrupted mode .
It's issue face only when i download file in zip file however not tar.gz file.tar.gz file successfully downloaded or extracted.

Do you have fancy name for the .zip file? Like lost+found hellow" there.zip?

no simple it's a folder name which come automatically COFFEEV2 DATA TEMPLATE

Hmm. Let's see what the dev says. I shouldn't be happening.

You know, try renaming it to COFFEEV2_DATA_TEMPLATE if it works, then the issue is obvious.

Try that but issue still exist

What version of Filemin do you use?

Version 1.1.1

What if you download this file with FTP? Will it be corrupted too?

with FTP we can directly download the folder & it's successfully downloaded.
Even I download the zip file from FTP it's successfllut downloaded but with filemin it's not download on winzip format I assume it's convert the fail folder to zip file.

I do not actually understand what do you mean under corrupt zip file. How do you obtain it? Can you provide the file itself?

I am trying to dowload one folder.Two typer are appear zip or  tar.gz.When I select zip file then it's download  very soon but when trying to open the file it's says corrupt.

Show me this corrupt file.

I am not able to attach corrpted file even in mail also.Please find attach screen shot.

What's the size of the file? Can you view it's contents for example with Total Commander. I have no clue what's wrong with this file as I do not see it's contents.


Performance benchmark

Is there a performance benchmark how does it compare to other great key value stores like redis?
Question

Due to the different hardware environments, especially the difference in disk performance, I suggest that you deploy a cluster on your own machine for testing.
Redis stores data in memory, it's different from TiKV. So I think Redis may have better performance(but also more expensive)


added confirmation dialog boxes

Changes
Added confirmation boxes whenever user cancels an item, clears the order, or submitting the order

Notes
create a dbconfig.js  in the root folder and set this up
module.exports = {
dbURL: "postgres://postgres:dbpass@localhost:5432/dagobah" ==>set your own dbURL here
};


Timestamps refactoring, docs and tests.

Current coverage is 64.10%

Merging #98 into master will increase coverage by +0.75% as of 6460059

@@            master    #98   diff @@
=====================================
  Files           80     80
  Stmts         2235   2237     +2
  Branches       265    265
  Methods
=====================================
+ Hit           1416   1434    +18
- Partial         85     87     +2
+ Missed         734    716    -18

Review entire Coverage Diff as of 6460059

Powered by Codecov. Updated on successful CI builds.

@alexander-yevsyukov, PTAL.

OK, I've removed this method with default context. Anything else?

Please change the min method name length to 2, and it'd be LGTM.


when updating the example everything will break please update

Operating System:
Node Version:
NPM Version:
webpack Version:
${package} Version:

Expected Behavior
when updating the example project evrything will break please can anyone update this so it will work with all latest webpack dependecies like webpack 5 with web dev server 4
Actual Behavior

Code
  // webpack.config.js
  // If your bitchin' code blocks are over 20 lines, please paste a link to a gist
  // (https://gist.github.com).

  // additional code, HEY YO remove this block if you don't need it

How Do We Reproduce?

i hope someone can update it its very outdated and it will be very helpfull to learn from

Is this repo death ?

Problem not related to webpack-hot-middleware, webpack.optimize.OccurrenceOrderPlugin was removed for webpack v5


Imcompatibility with newer TBomb obfuscation

Hey there, been stalking around and found out that TBomb's using new obfuscation lately.
Here's a quick patch to make bombcrack work just as perfect.
The obfuscation is all the same, but zlib and lzma compressions are swapped.
How to install (note that you need Git to be installed):

Reset bombcrack with git reset --hard (backup its folder before doing this if you have modified it in any way)
Download newcompat.patch file to the bombcrack location (where Makefile is located)
Run git apply newcompat.patch
Use bombcrack as usual

How about adding this to README, huh?

Confirmed - bombcrack indeed does not work on newer versions of TBomb, and this patch of yours resolves this issue.
However, this workaround won't be added to README, but rather pinned in Issues.
Thanks!

Thanks for pinning this, @darkestentropy!
Will keep TBomb on track and once they mess up their code some other way - I'll be sure to find a workaround!

Patch Update
Patch updated for the changes made in crack scripts.
Downloads/installation instructions

@scpketer, patching no longer works.

Yep, @darkestentropy, problem spotted right after tabbing update. Working on a fix.

Patch Update
Patch applying problem resolved.
Downloads/installation instructions

You're now in charge here, @scpketer.
assigns scpketer to this issue/thread

Due to publishing source code of TBomb, BombCrack project is being frozen for an indeterminate time. Thanks for your contribution! I'll be sure to notify you once BombCrack is back on.

Hey, @scpketer! A historical update was released, and your compatibility patch should be updated.
Historical update commit: c2f6075ff8ed9754e54afbcb369d5c5ad3d58e1e

Sure, @darkestentropy! Rolled it out!
Patch Update
Historical update compatibility fixes.
Downloads/installation instructions

This patch has been merged in in #8.

Yeah, thanks a lot for applying it. I'd make it a PR back then, though it's better not to, though.


project asset files deleted when exporting scene

the title said it all, when you export a scene the whole assets folder of your project is replaced by a new one containing only visEditor elements, so if you don't have a backup you lose all your project assets..

That's true. It's explicitly mentioned in project creator that assets are cleared on each export. Also when creating libGDX type project, editor copies original resources from android/assets to vis/assets. And when creating generic project asset folder must be empty otherwise you won't be able to create project.
Do you suggest any additional things to add/change?

ok but if you have files like this android/asset/myFile.xml for example the file is no restored when exporting scene, at least should pop a warning about it, although i don't exactly see why assets are moved when it could just add exported elements to android/assets but i guess that it's in relation with the internal structure of the editor

Actually it's possible that files not inside any dir (eg. vis/asset/myFile.xml) are not copied, it should work when it in it's some dir, eg. vis/asset/directory/myFile.xml. I'll need to check that.

although i don't exactly see why assets are moved when it could just add exported elements to android/assets

It's generally safer this way, it prevents situations when user file would be replaced by export, also file formats in android/asset/scene and vis/asset/scene are completely different so it wouldn't be possible to keep them in one directory.
I think I will add warning dialog before doing export and there will be option to "never show it again".

ok you're the boss! and i have already tried when the file is in a directory under vis/assets it works.

Alright, we now have fancy dialog before export:

And files inside vis/assets but not in directory are now copied during export as well ;)


How scoket cluster really works ??

I just want to understand few things that i am confused.

How many sockets does it create, when we publish to a channel, assuming 100 users.
Can we have a way to create dynamic channels?
what if i have 100k users open, how it can manage the load or sockets, i just want to know how it works and manipulates sockets.

Thanks

One socket per connected client. Two opened tabs in a browser == 2 clients. The sockets quantity doesn't correlate with publishing at all.
All channels are being created dynamically, when a first client subscribes to a channel. When the last, subscribed to the channel, client unsubscribes - the channel is being destroyed by SocketCluster automagically.
All clients(sockets) are distributed ~equally between workers. The same goes for channels, they're distributed between brokers. In theory, you can work with 100kk clients simultaneously, if you have enough SocketCluster's clusters in your setup.

Anyway, @Himansh-Tiwari, this is a bug tracker. So, please, close the issue.
If you want to understand SC workflow better, feel free to ask any questions in the SocketCluster's chat room. There are many people who could help you there. https://gitter.im/SocketCluster/socketcluster


Decorators for authentication and permsissions

What are the thoughts on decorators? I am gradually working authentication/permissions into my API and was not sure which approach was planned for apistar.

It's most likely that views will be 'annotated', rather than having the function wrapped up in extra behavior.
@annotate(auth=..., permissions=...)
def my_view(...) {
}

my_view.auth == ...  # True
my_view.permisions == ...  # True

We might also have a class based method of being able to compose and reuse annotation groups. That'll differ from, say, REST framework, in that it'd only be able being able to compose attributes, rather than class based views with overridable methods.
For now anything related to authentication or permissions needs to be coded into reusable components or into the view itself.


AttributeError: 'NoneType' object has no attribute 'find_all'

Python: 3.10
OS: ubuntu 22.04 (LTS) running in bitbucket pipelines image: atlassian/default-image:4
Tried running on windows and WSL2 with no issues, but as I wanted to automate this script via bitbucket pipelines,
from finsec import Filing
cik = "0001536411"
filing = Filing(cik)
filing_df = filing.latest_13f_filing

I encountered the following traceback issue (prob related to beautifulsoup?):
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/opt/atlassian/pipelines/agent/build/src/modules/sec.py", line 108, in <module>
    df = get_13f_filing(duquesne_family_office_llc, session)
  File "/opt/atlassian/pipelines/agent/build/src/modules/sec.py", line 43, in get_13f_filing
    filing_df = filing.latest_13f_filing
  File "/usr/local/lib/python3.10/dist-packages/finsec/filing.py", line 16, in latest_13f_filing
    return self.get_latest_13f_filing()
  File "/usr/local/lib/python3.10/dist-packages/finsec/base.py", line 181, in get_latest_13f_filing
    self._get_last_100_13f_filings_url()
  File "/usr/local/lib/python3.10/dist-packages/finsec/base.py", line 41, in _get_last_100_13f_filings_url
    for headers in results_table.find_all('th'):
AttributeError: 'NoneType' object has no attribute 'find_all'

Could you replicate this bug on a linux machine? What could be the issue here?

Hi @philsv I haven't had a chance to replicate and test this yet. I was wondering if you can provide me some more details? How are you going about installing the finsec package when using the cicd pipeline? I suspect this could be in the .yml file you have? Can you share a snippet of the script that is run for finsec install in the pipeline?

Sure @git-shogg I have tested it with the following pipeline configuration and got the above error message.
bitbucket-pipelines.yml
image: atlassian/default-image:4

pipelines:
  custom:
    sec_test:
      - step:
          name: "[Ubuntu 22.04] Update sec_test 13f filings"
          caches:
            - pip
          script:
              - apt-get update
              - apt-get install -y --no-install-recommends software-properties-common python3.10-distutils python3-pip
              - export DEBIAN_FRONTEND=noninteractive
              - pip3 install --upgrade pip
              - pip3 install --no-cache-dir --use-feature=fast-deps finsec
              - python3.10 -m src.modules.sec_test

sec_test.py
from finsec import Filing

cik = "0001536411"
filing = Filing(cik)
filing_df = filing.latest_13f_filing
print(filing_df.head(5))
print("Done!")

Hi @philsv , thanks for sending this through. I think I may have identified the issue... Firstly, the script below is working just fine on my end (with Python 3.10).

from finsec import Filing

cik = "0001536411"
filing = Filing(cik)
filing_df = filing.latest_13f_filing
print(filing_df.head(5))
print("Done!")

I suspect it is something to do with a change that has been made to the yml file. I can see in the initial error message that the yml file is pointing to src.modules.sec (that is sec.py not the sec_test.py file you send through) and this sec.py file is executing a line I am unfamiliar with:
df = get_13f_filing(duquesne_family_office_llc, session)

This line appears to be executing a function that sits directly in the base.py file of the finsec package (generally I'd recommend using the off-the-shelf functions included in the package - as you have with sec_test.py above). The function get_13f_filing only accepts self and qtr_year as arguments so suspect that passing duquesne_family_office_llc and session as arguments is most likely the cause of the issue.
def get_13f_filing(self, qtr_year: str):
        """Returns the requested 13F-HR filing."""

Can I ask that you re-test ensuring the yml file is pointing to test_sec.py and that test_sec.py aligns with exactly what you've sent me above? Let me know how you go with it?

@git-shogg
get_13f_filing(duquesne_family_office_llc, session) is a custom function to upload my results to my db.
I have tried to change the name but I can say that, this is not the issue.
As still with sec_test.py I get the same error:
+ python3.10 -m src.modules.sec_test
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/opt/atlassian/pipelines/agent/build/src/modules/sec_test.py", line 5, in <module>
    filing_df = filing.latest_13f_filing
  File "/usr/local/lib/python3.10/dist-packages/finsec/filing.py", line 16, in latest_13f_filing
    return self.get_latest_13f_filing()
  File "/usr/local/lib/python3.10/dist-packages/finsec/base.py", line 181, in get_latest_13f_filing
    self._get_last_100_13f_filings_url()
  File "/usr/local/lib/python3.10/dist-packages/finsec/base.py", line 41, in _get_last_100_13f_filings_url
    for headers in results_table.find_all('th'):
AttributeError: 'NoneType' object has no attribute 'find_all'

also have no problems using windows, WSL2 or linux on my own system. I also have tested it within docker no problem.
Only when using bitbucket pipelines I encounter this issue on different images, same error unfortunatly.
I can't really point to anything why this happens.

Hi @philsv , I spooled up an Ubuntu 22 server and tested this line by line to get to the bottom of this issue so thank you for your patience. The issue relates to the user agent that is passed to the SEC website. In general, the SEC requires that users identify themselves with their own user agent being passed to the website (generally this is not required when running local instances), this can simply be your name and email.
I have now updated and released a new version of the finsec package (v0.0.10) allowing you to easily pass in your own user agent when running on any server. Code would be updated as follows.

from finsec import Filing

cik = "0001536411"
filing = Filing(cik,declared_user="Joe Blog<EMAIL_ADDRESS>filing_df = filing.latest_13f_filing
print(filing_df.head(5))
print("Done!")

@git-shogg Amazing it works now, thank you very much for the update! :)


Update iSponsorBlockTV to latest

Hey there. Is it possible to get updated iSponsorBlockTV from couple weeks ago?

Sure, I didn't see anything relevant in the release notes so didn't bother updating. Is there a feature you are after in particular?

Thanks. I’m interested in the “Fix for the ad skips” - it’s been buggy for me. I was hoping it would be better.

I suspect you are encountering https://github.com/dmunozv04/iSponsorBlockTV/issues/119 as both the code author and myself have been experiencing. Sadly there is no fix for this yet.
I've updated to the latest upstream release, it looks like there may have been some config file changes so let me know if you experience any issues with that and I'll look into it. Cheers.

YouTube ad skips (not segments from sponsorblock) were indeed improved/fixed in v2.0.4 and I’ll make another change soon(ish) to make them work better

Fantastic cheers @dmunozv04


Migrate kotlin-logging from MicroUtils.org to oshai

As per https://github.com/oshai/kotlin-logging/issues/266, the Kotlin-logging library moved packages from microutils to oshai. Is there any plan to apply this migration in reactive-logger library too?
I tried to do so but there are plenty of breaking changes between the version we currently use here (3.x) and the latest kotlin-logging version (6.x). Unfortunately, I have no time to fix it in the end, otherwise I'd raise a PR.
Cheers!
Michal

Thanks,
I will check it out soon.

This will take a while. The API has completely changed.

Thank you very much for bringing my attention to this Kotlin logger version. I managed to rework it. There is a new limitation that was previously supported. The Kotlin logger no longer supports transitioning from Logger to KLogger, but the KLogger (specifically a given instance) carries its Slf4j Logger, so it can be extracted. It's just that there is no API for the Kotlin Logger in the reverse direction.
There are still refinements and tests, but this part is done.

Fantastic work, @Numichi! Appreciate you handling this on such short notice!


Publish app to F-Droid

Hi,
I really like your app, so I wanted to ask if you maybe want to publish it to the F-Droid repository, since this is where many, if not most people search for open source apps.

In the meantime see the izzysoft repo https://apt.izzysoft.de/fdroid/index/apk/com.gbros.tabslite

the versions of this app on GH releases as well as Izzy are very outdated. the only location being maintained seems to be Play Store

Hey all, I've updated the releases on this page, and the izzysoft repo should be able to be updated soon.  Although I'd love to publish to FDroid, the app currently contains google play services which makes it not allowable for fdroid.  I may update it at some point but it's not a high priority for me at the moment.


Modify graph elements sizes and color from summary side bar

Is your feature request related to a problem? Please describe.
Describe the solution you'd like
Include a modify button in sidebar elements
This should open a modify section to select sizes and colors or auto configuration
Describe alternatives you've considered
Use element values to assign calculated attributes
Additional context

:tada: This issue has been resolved in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


docs: improve the docs

Coverage remained the same at 97.37% when pulling 36fbdc4e2546454c294ac88b513d06435ccac100 on better_docs into 71ce23597fac52c22cb50efb7bea940d38a7ab06 on main.


How fix texture in unity3d

Hi, sorry fo my eng.
How fix texture after export in unity?

Thank for this best programm :)

@novichok63 Apologies for the late reply.
Did you figure this out? You need to set the texture's filter mode to "Point". Does that work?

Oh, you already did that. Maybe we can get @JohnPablok to comment please?

@simlu I think thise problem in size texture. In programm Qubicle i save object with texture size 1024x512 and i can set "Filter Mode" = Bilinear in unity, object looks fine, without thise problem.

@novichok63 You could just upscale the textures. However there is a better solution. I know other people have solved this. Wish I could get @JohnPablok to comment...

I don't know why the "non power of 2" is greyed out for you. I always have to change this to "None" instead of "ToNearest".   When exporting from VoxelShop I also uncheck Overlapping and Skewed UV's-though I don't know if this could be the cause of your issue.  I only uncheck them in case I make extra texture maps for the same model-Metalness, etc.


bump 1.1.0

https://github.com/react-component/queue-anim/issues/35

Coverage decreased (-4.02%) to 90.221% when pulling 033948ad5eded13155935267804edc0198b3728f on v1.1.0 into 71bee640865d740b16046baabe98922d92b1b717 on master.

Coverage decreased (-3.08%) to 91.167% when pulling 5547bdf70ee0a403bcfb18ece397fa88b392c6e4 on v1.1.0 into 71bee640865d740b16046baabe98922d92b1b717 on master.

commit message 应该写成 close #35


Update LSTM Attention Model to use tf.contrib.seq2seq.AttentionWrapper

The LSTM Attention model was updated:

Replaced old attention model with AttentionWrapper based code
HParams added for Bahdanau Type Attention and Luong Type Attention
HParams added for multi-head attention
Set weight decay to 0 to fix NaN training loss issues

I did a quick comparison with the variants on the translate_ende_wmt_bpe32k task.
After 2000 train steps:

Model
Parameters
Validation Loss
BLEU
Accuracy

LSTM Bahdanau Attention
14867456
6.25
0.2351
11.73%

LSTM Luong Attention
14850944
6.27
0.2319
11.72%

LSTM Bahdanau Attention Multi (4)
15228416
6.18
0.2370
11.99%

LSTM Luong Attention Multi (4)
15211520
6.16
0.2358
12.27%

Transformer (Base Single GPU)
63103488
5.95
0.1223
14.22%

Please review, thanks!

Is there any explanation for the disagreement between BLEU and Accuracy (or Validation loss)? Transformer has the best accuracy, but the worst BLEU according to your table.

@martinpopel there appears to be some problem with the way BLEU is computed by t2t, almost as if lower is better! Here are the results after 5000 train steps.

I think you need to wait 50k or 100k to get meaningful values.
But a few questions:
What learning rate / decay did you take for LSTM ?
Do I read well saying you took 2x128 as default for layers / hidden size ?

Yes that's true, 5K steps are not enough, but it was more of a sanity check.
I kept the default hyperparameters in T2T:
learning_rate = 0.1
num_layers = 2
hidden_size = 128

Great!
Should be able to close #113 #205 #375 #378

@kolloldas Do the approx_bleu values after 100k steps look reasonable?
I've noticed the approx_bleu curve always starts from quite high values at 0 steps (sometimes even 0.8), then drops (e.g. to 0.05 at 10k steps) and then starts growing and only from that point it correlates with the real BLEU.
Does anyone know why?
Is it possible that approx_bleu is computed in a way that when predicting the n-th (subword) token, tokens 1..n-1 from the reference translation are known (and backpropagated), so the network first tries to repeat the last token (and shifting words by one has just a minor effect on BLEU).

I'm yet to run this task for 100K steps but I think BLEU is initially high because the model makes very short predictions at the beginning. The BLEU drops as predictions get longer (but are mostly gibberish) and then finally begins to pick up as predictions become more meaningful..

No. BLEU contains a brevity penalty term, so too short translations get very low BLEU. I think the reason is what I have described above, ie. approx_bleu by default is not computed autoregressively, but using previous gold words as input for the decoder. You can use --eval_run_autoregressive, but this seems to be currently buggy (#407).


Set an order on @inject to run behind @namespace

The problem is that @inject uses the type name, and @namespace can change
it. Setting an explicit order makes sure that these things happen in a
sensible sequence.

c0e3519bc3ddbd58a7226c6f9c14cb58558fb93f


lib: Provides headers, operation name and variables to sidecar handler

based on https://github.com/inigolabs/inigo-js/pull/49

Stack:

#72
#71
#70
#69
#68 ⬅
#67
#66
#65
#64
#63

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #72


make-adopt-build-farm API query fail

Platform:
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk/job/jdk-windows-x64-openj9/24/console
I'm hoping this is just a temporary glitch. If it isn't, the purpose of this issue is to fix and print a more helpful error. Otherwise, just the latter is fine
13:26:13  + ./build-farm/make-adopt-build-farm.sh
13:26:13  This appears to be JDK Head. Querying the Adopt API to get the JDK HEAD Number (https://api.adoptopenjdk.net/v3/info/available_releases)...
13:26:13  Failed to query or parse the adopt api

manually curling that URL works for me.

The strange part is that curl didn't even run. The expected output should be
13:26:13  + ./build-farm/make-adopt-build-farm.sh
13:26:13  This appears to be JDK Head. Querying the Adopt API to get the JDK HEAD Number (https://api.adoptopenjdk.net/v3/info/available_releases)...
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100   308  100   308    0     0    376      0 --:--:-- --:--:-- --:--:--   376
13:26:13  Failed to query or parse the adopt api

The numbers are missing

Seen a few pipelines running without this problem over the weekend. I'm happy to close this under the assumption it was a glitch and we now have a more useful error message if it reoccurs

Reoccured on https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk/job/jdk-linux-s390x-hotspot/153/console
14:30:59  This appears to be JDK Head. Querying the Adopt API to get the JDK HEAD Number (https://api.adoptopenjdk.net/v3/info/available_releases)...
14:30:59    % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
14:30:59                                   Dload  Upload   Total   Spent    Left  Speed
14:31:00
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  2909    0  2909    0     0   7571      0 --:--:-- --:--:-- --:--:--  7575
14:31:00  Failed to query or parse the adopt api. Dumping headers via curl -v https://api.adoptopenjdk.net/v3/info/available_releases...
14:31:00    % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
14:31:00                                   Dload  Upload   Total   Spent    Left  Speed
14:31:00
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0* About to connect() to api.adoptopenjdk.net port 443 (#0)
14:31:00  *   Trying <IP_ADDRESS>...
14:31:00  * Connected to api.adoptopenjdk.net (<IP_ADDRESS>) port 443 (#0)
14:31:00  * Initializing NSS with certpath: sql:/etc/pki/nssdb
14:31:00  *   CAfile: /etc/pki/tls/certs/ca-bundle.crt
14:31:00    CApath: none
14:31:00  * SSL connection using TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256
14:31:00  * Server certificate:
14:31:00  * 	subject: CN=adoptopenjdk.net,O="Cloudflare, Inc.",L=San Francisco,ST=CA,C=US
14:31:00  * 	start date: Mar 19 00:00:00 2020 GMT
14:31:00  * 	expire date: Oct 09 12:00:00 2020 GMT
14:31:00  * 	common name: adoptopenjdk.net
14:31:00  * 	issuer: CN=CloudFlare Inc ECC CA-2,O="CloudFlare, Inc.",L=San Francisco,ST=CA,C=US
14:31:00  > GET /v3/info/available_releases HTTP/1.1
14:31:00  > User-Agent: curl/7.29.0
14:31:00  > Host: api.adoptopenjdk.net
14:31:00  > Accept: */*
14:31:00  >
14:31:00  < HTTP/1.1 200 OK
14:31:00  < Date: Mon, 15 Jun 2020 13:31:00 GMT
14:31:00  < Content-Type: application/json
14:31:00  < Content-Length: 308
14:31:00  < Connection: keep-alive
14:31:00  < Set-Cookie: __cfduid=d5e9223cea9bcb827960882d376ba23881592227860; expires=Wed, 15-Jul-20 13:31:00 GMT; path=/; domain=.adoptopenjdk.net; HttpOnly; SameSite=Lax
14:31:00  < Access-Control-Allow-Origin: *
14:31:00  < Set-Cookie: b7b892882bae631693e1ea44963ef628=064339b8dcebf139e60c125241755208; path=/; HttpOnly; Secure
14:31:00  < Cache-control: private
14:31:00  < CF-Cache-Status: DYNAMIC
14:31:00  < cf-request-id: 0359c626550000c5fc84352200000001
14:31:00  < Expect-CT: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
14:31:00  < Server: cloudflare
14:31:00  < CF-RAY: 5a3ca61d593ac5fc-EWR
14:31:00  < alt-svc: h3-27=":443"; ma=86400
14:31:00  <
14:31:00  { [data not shown]
14:31:00
100   308  100   308    0     0    596      0 --:--:-- --:--:-- --:--:--   596
100   308  100   308    0     0    596      0 --:--:-- --:--:-- --:--:--   596
14:31:00  * Connection #0 to host api.adoptopenjdk.net left intact
14:31:00
14:31:00  {
14:31:00      "available_lts_releases": [
14:31:00          8,
14:31:00          11
14:31:00      ],
14:31:00      "available_releases": [
14:31:00          8,
14:31:00          9,
14:31:00          10,
14:31:00          11,
14:31:00          12,
14:31:00          13,
14:31:00          14
14:31:00      ],
14:31:00      "most_recent_feature_release": 14,
14:31:00      "most_recent_feature_version": 15,
14:31:00      "most_recent_lts": 11,
14:31:00      "tip_version": 16
14:31:00  }
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] }
[Pipeline] // stage
[Pipeline] }
[Pipeline] // node
[Pipeline] }
[Pipeline] // stage
[Pipeline] echo
14:31:00  Execution error: script returned exit code 1

I wonder if we're being rate-limited by the API as this is becoming an ever more intermittent problem. @johnoliver Is there any rate limit protections enabled on the API?

The API was down a few hours ago, so I suspect this was just bad luck again.

The API was down a few hours ago, so I suspect this was just bad luck again.

I don't think so. I can view the API in browser and locally using curl. This build was only executed 10mins ago too

Hang on though, that call is returning a 200 OK - so is it the step after that's failing?

The step after is here https://github.com/AdoptOpenJDK/openjdk-build/blob/master/build-farm/make-adopt-build-farm.sh#L31-L42
The logic looks ok to me but you may spot something I haven't

@karianna that log is from a new pipeline that was kicked off by me today. All platforms were kicked off together - not all of them failed. My PR should help us diagnose exactly what went wrong with the call as it looks to me as though the amount of data receieved by the API operation (2909) exceeded what I would expect the API to return (308 based on running it just now).
My PR above should hopefully show us what error code the API call we really made to retrieve the data came back with ...

Summary
Our bash scripts are now using retryable API queries but groovy is proving to be an issue as the jenkins helper is difficult to work with in this context. I have made a second attempt via PRs on both the helper side and build script side that can be merged tomorrow morning for testing pending review.

This looks to be fine now. I haven't seen api failures in at least a week. This can be reopened if that changes


chore: update PoS mainnet seed list

As per https://github.com/nimiq/core-rs-albatross/pull/3055, keep the PoS mainnet seed list aligned.

deployed https://www.nimiq.com/developers/migration/migration-integrators


updated DeveloperGuide

bro, there is a Checkstyle error. Please check and Commit. thanks

Pull Request Test Coverage Report for Build 214

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 50.872%

Totals

Change from base Build 212:
0.0%

Covered Lines:
1488

Relevant Lines:
2925

💛  - Coveralls


"来園者" needs to be replaced with "ゲスト"

Strongly recommend to replace "来園者" with "ゲスト" because, in the paragraph of [Azure Active Directory] in this page, I can see "来園者" is used as a localized Japanese word of "guest" in English, however, it should be "ゲスト" which stands for guest user of Teams.

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: 6f1d577c-1360-c2c1-a850-9ac6a941ec2f
Version Independent ID: 346e43d4-22e0-2800-b80d-9fbdac324f75
Content: Microsoft Teams でのゲスト アクセスを管理する
Content Source: Teams/manage-guests.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: rramesan

Strongly recommend to replace "来園者" with "ゲスト" , I can see "来園者" is used as a localized Japanese word of "guest" in English, however, it should be "ゲスト" which stands for guest user of Teams.


[mgmt] containerservicefleet release

https://github.com/Azure/sdk-release-request/issues/5146

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/arm-containerservicefleet


[FIX 🛠] Post Updates Doesn't clear form data

Issue #73 📐

abdulrehmanjaved-1 have worked for #73

Guidelines 🔐
I accept the fact that I have followed the guidelines and have not copied the codes from around the internet
Issue to be closed 🛅

My pull request closes #73

Glad to hear it

Everything correct now?

Everything correct now?

Yes I have merged it and added hacktoberfest tags


Update stable version in repo from 2.2.3 to 2.3.0

Think about update stable version to 2.3.0
Version: stable=2.2.3 (290 days old) => latest=2.3.0 (15 days old)
Installs: stable=47 (83.93%), latest=3 (5.36%), total=56
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 2.3.0 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

PR is placed