Show holidays

Please specify the holidays with their corresponding colors. Additionally, if possible, allow users to manually register a specific day with a specific color in the component.

@Mostafa-Moradi The feature has been successfully implemented.

@farshaddavoudi
Dear Farshad,
Thank you very much

YW @Mostafa-Moradi. If you want to add/modify the holidays, you can update the Holidays.cs class and create a PR. Particularly I'm not sure wether to include Fridays in the holidays or not.


Adding light/dark toggle

In website adding light/dark toggle
assign this issue to me under gssoc ext and hacktoberfest

Since this issue has'nt been addressed by @Aditijainnn . Thanks for showing interest to contribute .
Now anyone else who wants to take this up can do the same.

@tmsagarofficial Can you assign it to me? I would love to work on this

@tmsagarofficial Can I work on this issue ?

Can you please assign this to me?

@tmsagarofficial Can you assign it to me? I would love to work on this

Assigned . Pls go ahead . 🚀


Support for Vulkan context

Description of Change
Partial support for Vulkan backend, allowing to create GRContext, GrBackendRenderTarget and GrBackendTexture which can be used to render on/sampled from.
skia submodule changes refer to https://github.com/mono/skia/pull/60
PR Checklist

[ ] Has tests (if omitted, state reason in description)
[x] Rebased on top of master at time of PR
[x] Changes adhere to coding standard
[ ] Updated documentation

@tuccio I added a hook for the environment variable VULKAN_SDK_HOME so we can add a build that does Vulkan.
Are you able to add any tests so we can test things out? My Vulkan-foo is very low, so I might take a while to get up to the basics.

I added some test for context and surface creation, used SharpVk to setup vulkan objects, but it's not signed and ended up making a new project for my tests, couldn't think of a convenient solution.


scrub password for "key" config keys

Azure Respository Plugin [1] uses "key" config property to store
sensitive data.
[1] https://www.elastic.co/guide/en/elasticsearch/plugins/current/repository-azure-usage.html

I'm adding this key, but I've decided to do it in a fashion other than just adding another value to the code. Since it's probably other values will wind up requiring this, I've moved the values to look for to the diags.ylm file and use that instead of hard coded values.


Ingress gateway logs TLS handshake timeout

Hi, I have installed istio successfully and able to see logs for all the istio pods except istio-ingress gateway. Following is the command and error message
C:\Users\my> kubectl logs -l istio=ingressgateway -c istio-proxy -n istio-system
Error from server: Get https://<IP_ADDRESS>:10250/containerLogs/istio-system/istio-ingressgateway-55d64d6575-qrxbg/istio-proxy?tailLines=10: net/http: TLS handshake timeout

am i missing anything or need to make any changes ?

This does not go through any Istio code at all, and is entirely Kubernetes specific. Please open with kubernetes or your kubernetes platform. Thanks!

I am seeing this same issue for a custom ingress gateway.


Doc mistakes on download page

http://learnsemantic.com/guide/download.html
"These may be useful for projects where only a few additional theming isn't necessary."
Is this sentence missing a few words?

Permission denied (publickey).
I tried to clone the Docs and got the same error today.


Question: renaming a pyenv install

Is there any way to name or rename a pyenv install version?
I've tried renaming the corresponding directory (and rehashing afterwards) but when I create a new virtualenv based on that python version it fails with the message 'pip: command not found'.
In concrete:
env PYTHON_CONFIGURE_OPTS="--enable-framework ...etc..." pyenv install 3.5.1
cd ~/.pyenv/versions; mv 3.5.1 framework351
pyenv rehash
pyenv virtualenv framework351 plotlib
pyenv: pip: command not found

Also tried pyenv virtualenv -p framework351 plotlib but it fails.
Apologize if I'm missing something obvious here.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/30407631-question-renaming-a-pyenv-install?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github).

I'd not recommend to rename directory of built Python installation because path may be hardcoded in built binary or some library files. It'd be better specify custom prefix. pyenv-alias may be what you are looking for.
https://github.com/s1341/pyenv-alias

pyenv-alias did it.
Thanks a lot!


Webcmdlets set default charset encoding to UTF8

PR Summary
Changes the default charset encoding of webcmdlets from Ascii to UTF8
It's just a small fix, it does not solve BOM related problems #11547

PR Context
Fixes #10971

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[ ] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

@SteveL-MSFT I think the PR makes sense but we need WG/Committee conclusion since we are moving away from RFC standard.

@SteveL-MSFT do you have any updates?

@CarloToso please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

Contributor License Agreement

@microsoft-github-policy-service agree [company="{}"]

@CarloToso You should sign CLA before we can merge.

@CarloToso the command you issued was incorrect. Please try again.
Examples are:
@microsoft-github-policy-service agree

and
@microsoft-github-policy-service agree company="your company"

@microsoft-github-policy-service agree

@iSazonov I fixed the tests, an error remains, but I don't think it's related to this pull request

@CarloToso For CLA please one command without comments.

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

It seems the cla-bot is broken (https://cla.microsoft.com/ gives 403 error)

@microsoft-github-policy-service agree

@SteveL-MSFT Please update you review.

Marking Cmdlet WG to review.

Added to WG queue

@PowerShell/wg-powershell-cmdlets reviewed this and compared the behavior to curl which correctly handles the UTF-8 BOM so we agree that the default should be UTF-8 instead of Ascii"

@CarloToso Thanks for your contribution!

@iSazonov Thank you for your guidance


Add Scalding job to count events by event type

page_view 23139
unstruct_event 13472

kind of thing.

Not going to do this


Add platforms to transit layer

As referenced in https://github.com/mapzen/vector-datasource/issues/194:
BART platforms should really be marked railway:platform and be included in the transit layer.
See how to do it right in Moscow:
http://www.openstreetmap.org/#map=17/55.75196/37.60592

After #429, and with a wholly appropriately festive style:
    platforms:
      data: { source: osm, layer: transit }
      filter: { kind: platform }
      draw:
        lines:
          color: '#ff0000'
          width: 5m
          order: 10
        polygons:
          color: '#ff0000'
          order: 10

Glen Park station
Before:

After:

(Drawing under the buildings seemed like the right thing to do here, and is the default anyway. But worth noting that the OSM map draws platforms, even underground ones, above buildings. Of course, this is under the control of the style, but I thought it was worth mentioning in case we look at a place where we're expecting to see platforms but they're rendered underneath a building.)
Atlantic Center station
Before:

After:

@zerebubuth What zoom would these start showing up with your PR, zoom 16?
Since these are in the transit layer (and not more general landuse or roads layers) I'd like to see them start at zoom 14. This will allow for constructing transit maps that highlight platforms on the zoomed out map, can be very effective visual technique.

Verified, including starting at zoom 14.


feat(#692): add useLocalStorage.ts for save state to localstorage

Description
discord username: naim#2823
closes #692
The selected option in filtering the map according to the "depremzede", "erzak yardımı", "teyitli" options under the notifications menu has been made to return the default values selected by the user even if the page is refreshed.
## Things to check before creating a PR

[x] I have inspected my topic, checked.

[ ] If it is a core feature, executed detailed tests.

Creating PR rules

[ ] PR must be created for an issue with approved tag. Otherwise PR will be rejected.
[ ] Relevant issue number: The issue number related to PR must be attached to head of PR header, after prefix after prefix # must be attached in parenthesis. A header like this could be used "prefix(#issue_number): PR header" .
[ ] A descriptive and understandable title: The PR title should clearly describe the nature and purpose of the changes. The PR title should be the first thing displayed when the PR is opened. And it should follow the semantic commit rules. For example, a title like "docs(#issueId): Add README.md" can be used.
[ ] Related file selection: Only relevant files should be touched and no other files should be affected.
[ ] Format and Lint Suitability : The code should be made in accordance with a certain format standard and examined according to the lint rules.
[ ] Clean commit history: The commits where the changes are made should be clear and organized.
[ ] Opening PR when job is completed: PR should be opened when job is completed and sent for review by other team members.

Changes

[ ] A new feature (a change that adds a new feature, not a breaking change)
[ ] A new refactor (a change that is not a breaking change, that improves the readability or performance of the code)
[ ] A breaking change
[ ] Documentation change

How were these changes tested?
Please describe the tests you did to test the changes you made. Please also specify your test configuration.
Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Could you edit this PR with the correct document? When you edit the document with the correct information, we will reopen this PR
https://github.com/acikkaynak/deprem-yardim-frontend/blob/development/.github/PULL_REQUEST_TEMPLATE.md

Is it enough to just add a template or will you do the test? @aysenur-aydin

It's enough to edit this PR as a correct document @naimyurek

@aysenur-aydin ok check please

duplicate #704

@absolutezero13 i think this solution better than https://github.com/acikkaynak/deprem-yardim-frontend/pull/704

#740 new pr new solution


Fix #5854 (get_contact_collider_velocity_at_pos() strange behavior)

Fix for incorrect velocity report due to a typo.
Closes  #5854

To me it definitely looks like a typo.

But shouldn't it be in reverse? I just debugged it, rigid body is B, and if B->can_report_contacts() is true then crA becomes zero vector, which is not what you want.

not sure I understand, can you show me an example?

If I apply your patch, the demo you provided only prints 0,0,0 for both rigid bodies.

ah I see. So it is not the correct fix.

No it is not, on contact velocity can never be zero (because of the gravity), try this instead:
`
if (A->can_report_contacts()) {
Vector3 crB = A->get_angular_velocity().cross( c.rA ) + A->get_linear_velocity();
A->add_contact(global_A,-c.normal,depth,shape_A,global_B,shape_B,B->get_instance_id(),B->get_self(),crB);
}
	if (B->can_report_contacts()) {
		Vector3 crA = B->get_angular_velocity().cross( c.rB ) + B->get_linear_velocity();
		B->add_contact(global_B,c.normal,depth,shape_B,global_A,shape_A,A->get_instance_id(),A->get_self(),crA);
	}

`

your solution looks more consistent, looks like gravity gets somewhat reported too, both now look like the right side one.
The value changes by setting Gravity Scale in the editor but seems to not be affected by the mass

Yeah that's the weirdest thing about physics, objects with different masses will hit floor at the same time. By the time both objects hit the ground, their acceleration value will be the same, because acceleration is not dependent on the mass.

feel free to update your PR with the updated code, I'm just here to help :)

ah I think I figured out at what the gravity corresponds on the script side:
it's just state.get_total_gravity() * state.get_step()

@Alex-doc Your IDE changed all indentation from tabs to spaces, could you fix that? And once you edit your commit, you can take the chance to edit the log too as we discussed ;)

For the reference, see how the same feature is implemented in 2D: https://github.com/godotengine/godot/blame/master/servers/physics_2d/body_pair_2d_sw.cpp#L386


[cli] ParsedAddress support Bitcoin address

Closes #1782

Please help add a test case for this feature in the cmd.feature and update the CLI comment docs for the address argument.


What does this branch graph mean?

Hi,
What is the best way to interpret this git branch history graph? I have tried it a few ways but given what I know about the branches I have, it does not make much sense to me.

Also, based on this knowledge I want to know what branch has all the changes and which branches I can delete safely.
PS: If this is not the right forum for this issue, where can I ask?
Thanks,

Hi, sorry for the slow response - would you mind opening this issue in https://developercommunity.visualstudio.com/spaces/21/index.html - that's the best place to ask non-task specific questions about Azure Devops since this repo is dedicated mostly to tasks


Indentation Fixes

There are some small indentation bugs that don't conform to the elm style guide and in some cases can actually lead to code that doesn't compile.
According to the style guide, multi-line union types should have the following indentation:
type Boolean
    = Literal Bool
    | Not Boolean
    | And Boolean Boolean
    | Or Boolean Boolean

Due to an omitted = operator in a regex, elm-vim was indenting this as:
type Boolean
    = Literal Bool
        | Not Boolean
        | And Boolean Boolean
        | Or Boolean Boolean

This has been fixed by amending the regex.
There was also a missing word boundary near one of the indentation checks for the in keyword. This was causing function names with in as a prefix to indent if there is a type annotation on the line above it (this causes a compilation error). Since the elm architecture uses a function called init, this can unintentionally break a lot of stuff.

@ajhager can this be merged?


Add "first" method

Issue: https://github.com/supercharge/set/issues/10

@Perafan18 Hey Pedro, you’re quick! Thank you, merged!


log should support non-char[] values

Expected Behavior
ut::log supporting at least std::string.
Actual Behavior
ut::log supports only compile-time known char[] string literals.
utility::string_view duplicates some std::string_view functionality, but is incomplete and not convertible.
Specifications

Version: latest master (b937560b4698dc0b2fdf67a30ac904a711407b34)
Platform: Windows Clang/LLVM 10 HEAD & MSVC 19.24.28307.1

Thank you for your efforts, I honestly didn't expect it to be fixed in what, like, 2 hours? :clap:
I've tested and confirmed this to work. This library is truly amazing and really shows that C++20 rocks!


fix: docker backend should not close 'engine.Tail' result

Closes https://github.com/woodpecker-ci/woodpecker/issues/1615
The error described in https://github.com/woodpecker-ci/woodpecker/issues/1615 is happening because Tail method of the docker backend closes the instance of io.ReadCloser it returns in defer function. As a result anything that try to read data returned by Tail method eventually will attempt to read from closes reader and get an error:
https://github.com/woodpecker-ci/woodpecker/blob/2171212c5a31bfffc8f882716bdd503c65413eee/pipeline/backend/docker/docker.go#L229
The fix is just don't close returned reader and let the consumer of Tail method do it. Good thing is that Tail is used only in one place and reader is correctly closed:
https://github.com/woodpecker-ci/woodpecker/blob/2171212c5a31bfffc8f882716bdd503c65413eee/pipeline/pipeline.go#L231-L237
Example of woodpecker exec output using pipeline from https://github.com/woodpecker-ci/woodpecker/issues/1615 with the fix:
woodpecker exec .woodpecker.yaml
[step1:L0:0s] + echo step1
[step1:L1:0s] step1
[step2:L0:0s] + echo step2
[step2:L1:0s] step2

Codecov Report
Patch coverage has no change and project coverage change: -0.05 :warning:

Comparison is base (a2f226f) 40.64% compared to head (c20c442) 40.59%.

: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    #1616      +/-   ##
==========================================
- Coverage   40.64%   40.59%   -0.05%
==========================================
  Files         169      169
  Lines       11083    11084       +1
==========================================
- Hits         4505     4500       -5
- Misses       6287     6293       +6
  Partials      291      291

Impacted Files
Coverage Δ

pipeline/backend/docker/docker.go
0.00% <ø> (ø)

server/model/task.go
0.00% <ø> (ø)

server/queue/persistent.go
0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes
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 in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

backport -> #1620


Serialize enums as a string instead of number

Discussed in https://github.com/serverpod/serverpod/discussions/879

Originally posted by andriyndev May  6, 2023
(Copy-pasted from https://github.com/serverpod/serverpod/discussions/715 because it's a separate issue)
I think enum's values should never be serialized by numeric values because if the app's developer edits the enum so that the values of the current elements are shifted, it'll cause nasty consequences. For example, user selects one value, but it's written to the database as another one, and the worse is that they'll see it as they specified it while other users with more recent versions of the app will see it as it was written to the database. It might be especially dangerous if the data represents delicate information like user's religion, political view, sexual orientation, gender etc. And even if the app's developers are aware of this nuance, they still might be susceptible to human factor. And if the mistake will be made, it might be impossible to fix the consequences without interacting with each user who triggered writes of the data to the database because there is no way for the app's developers to know whether the value is what the user actually selected.

Since toString() can be overridden now in Dart enums, probably the toString value can be written to the database.
A serverpod generate test can run through the enum's values, checking that all the toString values are unique, and then an invertible map can be created mapping from values to strings, and strings to values.
To make this backwards-compatible, the database column type can be checked -- if it's int-typed, then the old (positional) scheme can be used; if it's string-typed, then the new toString mapping can be used.

Please see PR #1080.

Re @tgrushka's comment about supporting custom serialization: I originally modified the yaml parser to support a custom serialization value, e.g.
  - active serialize='A'
but then I ran into an issue with this approach. Specifically, packages/serverpod/lib/src/database/expressions.dart contains the query syntax code, and it defines the class class ColumnEnum<E extends Enum> extends Column. Since this is a generic Enum, any additional value (such as the serialized value) is not available to the methods of this class, without ugly casting, or the use of an external dictionary.
Consequently, I opted for simply serializing to the enum values' name. Anyway this is the most predictable way to serialize an enum constant.
The enum value names may be too long for some usecases. I would prefer enums to be serialized as a single letter, to save on database storage. I could probably just use enum constants like A rather than active, then explain in the documentation comment what A actually means (so that the explanation pops up in an IDE).

Thanks @lukehutch! Just dealing with this now. Let's say I have the following enum, so serializing by only first letter would not work (but to me it is totally fine to use the .name value in the database, which might even be possible to address/catch in the migration process if changed as well):
enum: PlatformType
values:
  - androidCTV
  - androidMobile
  - appleIOS
  - appleTVOS
  - roku
  - webBrowser
  - webCTV

^^ Ha! I guess the strict lowerCamelCase is a good thing for this as well, which seems a bit funky with some of the names but will help prevent simple user error like changing the case of a value and having it break the database:
Error on line 3, column 5 of lib/src/protocol/platform_type.yaml: Enum values must be lowerCamelCase.
  ╷
3 │   - androidCTV
  │     ^^^^^^^^^^
  ╵

Right, using just one letter was just my requirement for keeping data size small. In gernwral using name is the right thing to do. (My patch does that.)
I ran into the camel case requirement for enum constants Gender.M and Gender.F... I think that should be a warning at worst, not an error!

Updated PR in #1518.


Failed to resolve import "@/composables" from "src/components/Footer.vue". Does the file exist?

error screens

resolve config

electron config

tsconfig.json

router.ts

You can try use vite-plugin-dynamic-import


Tag Page

Sorry, I think I meant to open this pull request for my fork, and I accidentally opened it here.


Are attribute values being attached to the right keys?

When I run mapbox-geostats on a tileset of Los Angeles TIGER roads, many of the sample attribute values seem to be attached to the wrong keys. For example:
            "attributes" : [
                {
                    "values" : [
                        "100th St E",
                        "102nd St E",
                        "105th St W",
                        "106th St E",
                        "107th St E",
                        "10th East Aly",
                        "10th St E",
                        "1101101328336",
                        "1101433129869",
                        "1101433872341",
                        "1101576647997",
                        "1101576648014",
                        "1101576648019",
…
                        "1101576648968",
                        "1101576649007",
                        "1101576649054",
                        "1101576649127"
                    ],
                    "count" : 1000.000000,
                    "attribute" : "FULLNAME",
                    "type" : "string"
                },

Some of these are FULLNAME street names, but many more of them are actually LINEARID way ids.

Sorry, false alarm! It is my tileset that is wrong (from experimenting with vector tile encoding changes), not mapbox-geostats!


Navigation Mesh weird behavior; Finds path to only certain points on the map.

Godot version:
3.1.stable.mono.official
OS/device including version:
Windows 10.
Expected Behavior: The player to be able to walk to any point in the navmesh.
Issue: The player is only able to climb the ramp or get close to the boxes; Also, the player can only move to immediate surrounding regions.
GIF: https://imgur.com/a/kZJTlKS
Steps to reproduce:
Left click anywhere to move. See that you can't move anywhere, other than the ramp and the cubes.
Left click on player; The player will do a small step depending on where it is clicked. This is not intended.
Minimal reproduction project:
https://drive.google.com/open?id=1bnH84uPaNOCkRrRgWuXnmxsh86FJYOic

Is this a single NavMeshInstance, or are you trying to use several?

It is a single one, yeah.

It was just a static body missing. I'm feeling pretty dumb.


Many links on org chart page are broken

A lot of the links on https://handbook.sourcegraph.com/company/team/org_chart are broken - possibly something to do with the auto-generated redirects and nested lists? For instance, the "Code graph" link goes to https://handbook.sourcegraph.com/company/team/code-graph but should be https://handbook.sourcegraph.com/engineering/code-graph.

Actually, I think you are right, this is about redirects. The links are correct.


Handle URI query string when instantiating Requests

GET requests can be created like this:
$request = new Request('/hello', 'GET', array(
    'GET' => array('this' => 'isdog')
);

But also like this:
$request = new Request('/hello?this=isnotdog', 'GET', array(
    'GET' => array('this' => 'isdog')
);

In this case, $request->uri() would evaluate to '/hello?this=isnotdog' and $request->get('this') would evaluate to 'isdog'.
Essentially, the query string of the request URI is not being handled in any way and just set as $this->data['server']['REQUEST_URI'] where $this is the Request instance.
The query string needs to be stripped and/or merged with the given GET array.
Edit: In Symfony's Request object, the URI always takes precedence.

Implemented by cbf8db4.


Bot offline?

Hey there, Love the bot! Excellent work.
Just in the last 24 hours, we have noticed the bot is unresponsive and offline? We installed it from discord direct link. Any sort of help or update would be perfect. https://imgur.com/a/RwhvrTy
Thank you.

Should be fixed now. Recently had some server issues with the hosting platform and had to migrate off of it. Been a few hiccups.


Filtering S3 keys

Sometimes we just want to select a subset of keys from a S3 location and process only those.
I've added a feature to filter only matching keys.

Thanks for your patience and the contribution!


 Can we add that.
Also one player had the issue that the building meter wasn't accurate.

This is dependant on contract upgrade - That will be done first


6.4 update is unstable

It's super inconsistent.
One floor, everything could be alright, while in the next it doesn't mark anything or show any aggro cones/circles. Or it straight up shows things wrong (i.e. showing the Passage as a gold chest). Or some things show up while others don't on the same floor.
Just thought I'd let you know.

I'm also having this same issue. (Note for Jukkales: Love your work!!!)

closing this one to continue this issue in #6


feat(daffio,design): highlight active tree item in sidebar

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/develop/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Fixes: #2854
Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information
The way the options must be set is unfortunate. There is no way to set the options from the     DaffTreeItemDirective because of https://github.com/angular/angular/issues/50700#issuecomment-1590896233

So the tree technically doesn't handle the "active" state unless ROUTER_LINK_ACTIVE_CONFIG is set in the implementation?
Is there a different way to do this?

no it does but the "overview" link will be active when a child guide is selected so you'll see 2 active items at once

Also, getting this error in console in design-land:
template.html:1 ERROR Error: NG0311: Directive _RouterLinkActive does not have an input with a public name of daffTreeItem.
    at validateMappings (

Also, getting this error in console in design-land:
template.html:1 ERROR Error: NG0311: Directive _RouterLinkActive does not have an input with a public name of daffTreeItem.
    at validateMappings (

I cannot repro that

So the tree technically doesn't handle the "active" state unless ROUTER_LINK_ACTIVE_CONFIG is set in the implementation?
Is there a different way to do this?

no it does but the "overview" link will be active when a child guide is selected so you'll see 2 active items at once

Why are the links below overview treated as child guides of overview? Shouldn't they be sibling to overview?

@xelaint I'll use cart as an example. the cart driver guide is at /docs/packages/cart/drivers. the cart "overview" guide has the link /docs/packages/cart. The angular router considers /docs/packages/cart/drivers to be a "child" of /docs/packages/cart and therefore considers /docs/packages/cart to be active on the route /docs/packages/cart/drivers unless exact: true is set.

I'd expect a parent tree item to be open if there is an active/selected tree item within it. Should this be done in this PR or create a separate issue for it?

I'd expect a parent tree item to be open if there is an active/selected tree item within it. Should this be done in this PR or create a separate issue for it?

separate PR


[feature] Email Contributors Added to Forks and Templates [OSF-7699]

Purpose

Send emails to contributors when they are added to a project that was created by forking or templating.

Changes

Send the contributor_added signal in fork_node() and use_as_template() functions (in order to subscribe the user that created the project to notifications) but with email_template=false so that the contributor added email is not sent when a user forks/templates a project.
Remove the check in notify_contributor that prevented contributor emails from being sent for nodes where is_fork or template_node was truthy.

Side Effects

None expected.

QA

Fun fact: from what I can tell, this functionality has never worked and we've never sent emails to contributors that were added to forked/templated nodes. Both should work now.
Additionally, if a user templated a project, that user was not subscribed to notifications for the new templated project. This should work now (it wasn't broken for forks).

Ticket

OSF-7699

Added 🦄


Error shown but cannot find where it's being thrown

When I use express - app starts on port 8080 as I designate and runs.
I trade out express for nanoexpress and I see:
git:(nanoexpress) ✗ node --trace-uncaught --trace-warnings server.js
Text and data can only be passed by String, ArrayBuffer or TypedArray.

No other stack trace is emitted.  I have no idea how to track down this particular error.
I'm going to guess that what this means is you cannot pass an argument to a function if that argument is not one of String, ArrayBuffer or TypedArray.  And if that's true, then I'm going to suppose that passing a JSON object as an argument to a function is what's causing this error:
    let { token, expiresAtTimestamp } = setupTokenAndTimestamp(contextPayload);
In the above - contextPayload here would be a JSON object.
If what I'm assuming is all correct, then I suppose this is not an issue with nanoexpress, but rather a feature request - because I can't use this server if it's not going to support JSON as arguments to functions.

Hi.

Are you really using Pro version?
Without route code really hard to understand how and why this error occurs

See this please and make sure check your code again.
Check examples, documentation and some tests before creating issue.
I sure the error was occured because on your code something wrong.
I am made library on top of uWS and provide premium support only for Sponsors.
I hope you understand me.


Add Conversion to and from Chrono Times.

This is in there? At least a one way conversion is. Check and close.


[MM-13175] Only put channels on top of the sorted list for unread channels group

Summary
Only put channels on top of the sorted list for unread channels group.
I added this comment into the function itself:
// mapAndSortChannelIds sorts channels, primarily by:
//   1) hasMentionedChannelIds - Only for unread channels group where channels with mention are always sorted first.
//   2) otherChannelIds - All channels not included in hasMentionedChannelIds and mutedChannelIds are sorted next to hasMentionedChannelIds
//   3) mutedChannelIds - Muted channels are always sorted last regardless of channel grouping
// and then secondary by:
//   1) alphabetical ("alpha") or chronological ("recency") order
export const mapAndSortChannelIds = (channels, currentUser, myMembers, lastPosts, sorting, isUnreadChannel = false) => {...}

Ticket Link
Jira ticket: MM-13175
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit tests passed
[x] Ran make flow to ensure type checking passed
[x] Updated unit tests (reverted the test)

Test Information
This PR was tested on: [Chrome/FF, iOS simulator / Android emulator]

@saturninoabril Just a heads up, you cherry picked the change onto the release-5.6 branch on your fork (saturninoabril/mattermost-redux), but you didn't push that upstream to the main repo (mattermost/mattermost-redux)

Elias cherry picked it onto the upstream branch though, so we should be good now

Thanks, yeah I thought I've push it to upstream but also realized when I saw the commits in release-5.6.


Uv Index

Sun with the word UV written inside for use for UV index in weather map

@cjh9 Does any of these matches what you are looking for?

Nice I like the right one. How does it look if the text: uv is centered?

@ericfennis

hmm, not sure yet.
Can you try to make the spacing in the letter U to be 2 px?
Also, I think we need to refactor the sun icon, it doesn't have the 1px safezone.
Can you try to make one with this icon?

<svg width="24" height="24" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
<path d="M12 16C14.2091 16 16 14.2091 16 12C16 9.79086 14.2091 8 12 8C9.79086 8 8 9.79086 8 12C8 14.2091 9.79086 16 12 16Z" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M12 2V4" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M12 20V22" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M5.21997 5.21997L6.63997 6.63997" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M17.36 17.36L18.78 18.78" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M2 12H4" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M20 12H22" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M5.21997 18.78L6.63997 17.36" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M17.36 6.63997L18.78 5.21997" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
</svg>

I'm not sure that we have to change the sun to have the right margin. I think that it was created with visual balance in mind. Of course the smaller sun would be better if you think it looks better but it shouldn't be changed just for the sake of it.
 _
Some revisions:
Small sun, othervise the same:
Small sun, more spacing between U and V:
Big sun, more spacing in U:
Smal sun, more spacing in U:

I don't think it would be a great idea to add this icon in Lucide, it is way too specific. Wouldn't simply using the text "UV" fit your use case?

@Andreto Great! Exactly what I had in mind.

Shouldn't there be some guidelines for working with alphabets in icons?

Yes!

I agree with @locness3 this icons is too specific.
I close this for now.


adds social icon button

Done!

You wan tot merge it?

ohk @rafibomb merging this .... but if for caring purpose will be great if you can add that comment later!

Thanks @Deckluhm ! Are you able to make a Pull Request to fix it or open a new issue?

Maybe @blynnc (who reported the issue) wants to open the PR to fix this? If not, I will handle it.


Define Hudi error types across hudi-core

Use thiserror to define error enums to fit different error handling flows, and replace anyhow::Error usage.

@codope any plan to resume the work? Updated the milestone/release date.

does this still wip? while i try to add some create table methods, just found this is a little chaos.
maybe i will try to refact that.

@gohalo Please go ahead with your refactoring. I can help with review. I had opened https://github.com/apache/hudi-rs/pull/59 to refactor error handling but it is incomplete and preferably we shall use thiserror.

Submit a pr, just replaced the anyhow package, but still have some problems to fix.
First, how to classified the errors.

User configurations, invalid file/object path, unsupported features.
Communication failed, mostly from the obejct_store crate.
Invalid hoodie table, including file layout, file format, crashed data file and so on. Including from parquet arrow crates.
and more.

Second, how to simplify the error generation or mapping process. thiserror provide many macros for this, such as error from etc. But still have many situation to simplify, through some trait like Into.


Persistent storage options for Prometheus and Grafana display horizontally but would be better vertical

What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Steps to reproduce (least amount of steps as possible):
In the cluster or project monitoring pages (Tools > Monitoring) the options to Enable Persistent Storage for Prometheus and Enable Persistent Storage for Grafana appear side by side but if you enable either, the Persistent Volume Size and Default StorageClass options for Prometheus and Grafana appear in rows underneath.
It would be much more intuitive to have them appear in columns under each option or place Enable Persistent Storage for Prometheus and Enable Persistent Storage for Grafana in separate sections, one above the other.
Result:
How it appears:

It would be better if they were separated in some way like:

or

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.2.8
User Interface: v2.2.94

We should really have the Enable Persistent Storage for Grafana below the Prometheus fields.  It is confusing that the granfana fields are now in the middle of the prometheus fields.

I went ahead and moved grafana options to be below the tolerations and above the advanced options which just appear to be answers.

Available in UI tag latest-2.3

Issue in 2.3.0:

Validated the fix in 2.3 head with ui tag: latest-2.3


Move over rest of testutil to use cmp.Diff

We have a few testutil functions that still use reflect.DeepEqual that should instead use cmp.Diff for better output.

Like #381


Transaction routes

All routes are guarded by JWT authentication

[x] GET /transaction => Get paginated list of Transactions from all Accounts for authenticated user, filterable by account via query parameter
[x] POST /transaction => Create new Transaction with given data
~ GET /transaction/{id} => Get details for Transaction with given ID~
[x] PUT /transaction/{id} => Update Transaction for given ID with given information
[x] DELETE /transaction/{id} => Delete Transaction with given ID & return details of deleted Transaction
[ ] PUT /transaction/{id}/spent_from/{envelope_id} => Mark Transaction with given ID as spent from Envelope with given ID; return updated Transaction, updated Envelope balance, & updated Available Funds Balance

No longer implementing details routes on any object. Instead, consumer will make multiple calls as needed to compile whatever details they might need.

Pends #7 implementing moving money in & out of Envelopes.

Fully implemented.


Adding vShpere volume section to Volumes doc

This documentation is being added in reference to vSphere cloud provider
support work (issue #23932).

These Doc updates are associated with the following PR:
vSphere Volume Plugin Implementation

@johndmulhausen the respective PR has now merged (PR [24947])(https://github.com/kubernetes/kubernetes/pull/24947).  Can we get this merged as well to keep docs updated?

@lavalamp @thockin - you helped @dagnello with the code PR. Do the docs here look OK?

lgtm

@johndmulhausen good to merge

@johndmulhausen is there anything else remaining for this PR?

@johndmulhausen is there anything else remaining for this PR?

@johndmulhausen can we get this merged?


[Feature] Add caption option for code blocks

Problem description
When adding code snippets to markdown, reference text to the snippet needs to appear either above or below the snippet as regular text. This is very much a nice to have,  but it would be more elegant if the text appeared nested with the snippet itself.
Why would it be useful?
ease of use, referencing code snippets
Describe the solution (optional)
enable fern developers working on markdown to assign caption text to a code snippet.
Additional context

We don't plan to add a prop to a code block for a caption. If this becomes more than a nice to have, let us know!


Missing optimizer checkpoints

I tried training based on the pre-trained models that were published, but this requires the checkpoints for the AdamW optimizer.
Are you planning on publishing them soon?
Many thanks

I don't think we have plans to publish the optimizer checkpoints, but in the past I've found it possible to resume training when cold-starting the optimizer.


Promote release-service from staging to production

Included PRs:

https://github.com/redhat-appstudio/release-service/pull/284
https://github.com/redhat-appstudio/release-service/pull/281

/lgtm

/retest-required

depends on https://github.com/redhat-appstudio/infra-deployments/pull/2602

/retest

/unhold

/lgtm

/lgtm

/lgtm


How to support Emoji？

I use asciidoctor/asciidoctor-maven-plugin to generate PDF document.
I write the AsciiDoc documents in Chinese, so I configure the theme and the fonts. But when I use Emoji in the document and generate the PDF document, it throws some warnings:
...
[INFO] asciidoctor: WARN: Could not locate the character `😜' in the following fonts: Source Han Serif SC, Source Han Serif SC
...
[INFO] asciidoctor: WARN: Could not locate the character `😂' in the following fonts: Source Han Serif SC, Source Han Serif SC
...

The document is:  https://github.com/diguage/jdk-source-analysis/blob/master/docs/preface.adoc

The Example Repo: https://github.com/diguage/jdk-source-analysis
# 1. download the fonts
$ bash cfg/fonts/download-fonts.sh

# 2. Run the command to generate PDF document
$ mvn package

Please direct questions about usage to the asciidoctor-pdf stream in the project chat at https://asciidoctor.zulipchat.com. I'd be happy to follow-up on your question there. The issue tracker is now reserved for verified bug reports and new feature requests.

Zulip is in beta for the ability to make streams public, without login, which will solve the external search issue. Until then, you can search inside the Zulip client.

@diguage As the maintainer of this project and the lead of the Asciidoctor community, I am asking you to honor my request to ask questions in the Zulip-based chat. That's the direction we have decided to go in this community to make it more sustainable and engaging for users. You can learn lots of tips from what other people share there, even more so than from the issue tracker. Given that I'm the one who handles the bulk of the issues, it also makes it a lot easier for me to do my job so I have more time to work on the software.
To address the concern you raised about searching, you'll find that the Zulip interface provides both a topic and full text search across all project streams. The search returns very relevant results, far more accurate than what a generic search engine can provide. As James mentioned, we will enable public archives as soon as Zulip enables them on zulipchat.com. We expect that to happen soon. Even then, I think you'll get better results using the Zulip interface.

@mojavelinux Thank you very much. Asciidoctor is a great and complex project. Thank you for creating such a wonderful project and toolset.
I respect your request and have sent the question to Zulip.

Thanks! I look forward to chatting with you there.


Fix tar URL

Coverage remained the same at 77.941% when pulling ecf01d3f9a5d4011031367f261da87bedfc39dd6 on mathiasbynens:patch-1 into 485aa89cbb7c73bd65e4c70f4346361c62f20c37 on alanhoff:master.


Settings: Loads of settings display with their template string

How are you using the lua-language-server?
Visual Studio Code Extension (sumneko.lua)
Which OS are you using?
Linux
What is the issue affecting?
Other
Expected Behaviour
Extension settings should contain descriptions on what they're doing.
Actual Behaviour
Many extension settings (53 to be exact) are displayed using their template string.

Reproduction steps

Install vscode extension
open settings
search for @ext:sumneko.lua %

Additional Notes
No response
Log File
No response

https://github.com/microsoft/vscode/issues/158649

ah ok, didn't know the extension had proper template translations and frankly, I was too lazy to check x)

Fixed since VSCode 1.74.0


Collaboration agreement link pointing to the wrong file

Anthony Mallama pointed out that the link to the collaboration agreement is pointing to the membership application form. I think the membership agreement is here.

the better link to use would be https://drive.google.com/file/d/1tPChesKIz9VSQshb3oPcM1J71xpayS9v/view - since that's what the new membership application form links to, at https://docs.google.com/forms/d/e/1FAIpQLSdFL6JviMl4-W3xIlWrZACFOWV-6B8SVptPpxTYtLZOXgUHmg/viewform


Change "Fixer" name for something else

Package is not fixing, but detecting... so Fixer is not the best name

https://github.com/nilportugues/php_forbidden_functions/commit/788a4f8d5c14fb1ac0bc4a0bcf6f92b4ffea8ead


TPU utilization could be improved further?

Training details are in https://github.com/google-research/big_vision/issues/2
I think the TPU utilization is a bit lower than expected:

Is this expected?
I understand there might be other network access factors that can contribute to this but wanted to know.

TPU utilization should be close to 100%. I think your dashboard is showing something else.
My guess it shows percentage of time when TPUs do FLOP-heavy operation, like matrix multiplications. The rest is various data reshapes, weight synchronization and so on. IIUC it is hard to do substantially better than what we have now.

Here's what I am doing.

Go to https://console.cloud.google.com/compute/tpus/detail.
Then on the monitoring panel, I am getting:

I'm probably wrong in selecting that. But you've suggested clues which are enough for now.


Changes made to support node-krb5 on z/OS

This PR contains changes made to support node-krb5 on z/OS, it included modification in src and change installation process.

Thank you for the contribution. A few notes:

Why do you introduce the "MANUAL" env variable
We could have only one *install.shscript, the plaform could be obtained withnode -e 'process.stdout.write(os.platform())'` inside of it

Why do you introduce the "MANUAL" env variable

Currently the default behavior of running npm install is executing node install.js, if the platform is z/OS, it will always run install_krb5_zos.sh, so I want to provide a way for the user to flexibly install krb5 themselves and npm install without running install_krb5_zos.sh

We could have only one *install.shscript, the plaform could be obtained withnode -e 'process.stdout.write(os.platform())'` inside of it

I am not sure what do you mean, I added install.sh before as the command to execute for npm install, but later realise it doesn't support windows so I changed it to install.js and execute node install.js instead, currently the branch have install.js, install_krb5_zos.sh and install_krb5.sh and doesn't have any install.sh.
I check the failure in travis it seems to occur because docker-compose.yml doesn't include install.js nor install_krb5_zos.sh in the container.

OK, your points are valid. However, we choose to not download the source or binary of Kerberos and instead document the procedure for each plateform and let the user do it. I am not saying it is a good thing nor a bad thing but I believe the behavior shall be the same for every plateform, wether we do one strategory or both based on the presence of an environmental variable like MANUAL. @alexcfyung @RReivax what are your opinions ?

I see what you mean. I can share why I choose to download the binary for krb5 just for z/OS. It is because z/OS is a special case and different than other platform, in other platform krb5 are widely supported and available, but in z/OS I need to build the libraries and ship them, this is the location of the repo: https://github.com/ibmruntimes/libkrb5-zos. It is only meant to used with this npm, because of this special situation, I choose to download the binaries as well to save user some trouble.

I kind of faced the same concerned a few years back with the first version of this package on OSX (probably still the case). Here's what I propose:

we document the process of installing a package or downloading/compiling the source code or binary for each plateform, whatever is the easiest.
we provide a script which does the kerberos installation for each platforme but which is only activated with options like KRB5_SOURCE and KRB5_BINARY. Note, we could add a KRB5_CHECKSUM to validate the download.

For number 2, I think it make sense to unique platform like z/OS, because it is rather difficult for user to install krb5 themselves and they most likely going to use the krb5 I provided, but for other well supported platform such as linux, user may have their preferred way to install krb5.

Do we have a conclusion on the way to go for?

I reverted back the install changes and add some instruction for zos in README, please review the changes.

This works for me. Thank you

Quick note, version 0.5.3 was released last week with the changes


"BUG":  EUMETSAT WV wind from Met-9 not thinned (uv kx=254, subtype 56)

Austin discovered today a large increase in EUMETSAT water vapor AMVs.  The data is from Meteosat-9 (satid=56) which is temporarily replacing Meteosat-11 while the latter satellite undergoes decontamination.
The reason for the increase is because the Meteosat-9 data is not being thinned due to a bug in the 'gmao_global_convinfo' resource file which has the thinning turned off for that satellite and data type.  Oops.  Hopefully this will be fixed in OPS but it also needs to be fixed in the tag.

I verified that this problem is fixed in the code that was brought over from CVS - so the issue can be closed.


Mouse wheel zoom is very slow in Firefox

I think Firefox uses a different scroll step size in the legacy mouse wheel event.  There's probably some more modern way to get the amount of scrolling.

/cc @UX3D-nopper

We will have a look at it.

should be fixed with #145


Upload videos on browser test failures when running on CI

Description
This way it can help us understand the root cause of intermittent azure failures.
Checklist

[x] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Codecov Report

Merging #3200 (31174c9) into main (5ddadb1) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #3200   +/-   ##
=========================================
  Coverage     79.46%   79.46%
  Complexity     2270     2270
=========================================
  Files           282      282
  Lines          9022     9022
  Branches        602      602
=========================================
  Hits           7169     7169
  Misses         1623     1623
  Partials        230      230

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


gnn

A PR from Xiaoqin Zhou

Please deploy your branch via Jeykll so we can view it at https://zhouxdd.github.io/cs224w-notes/machine-learning-with-networks/

done: https://zhouxdd.github.io/cs224w-notes/machine-learning-with-networks/graph-neural-networks

not sure how to deploy with Jeykll, but I did get it to display the page

using deep encoder should be using deep encoders
"for a new node arrives" => "when a new mouse arrives"

I only went through the first half. Please read through the whole notes again with a fine tooth comb, and strive for correctness, clarity, conciseness, and consistency.

will update the PR in a little bit


add support to finally()

It would be nice to add support to finally() function for performing cleaning task.
db.collection(...).find(...).then(...).catch(...).finally(..)
I tried to use it but got a error of no finally function.

It's not implemented in native Promises: https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Promise


build: Update to header 1.3.244

Update known-good
Generate source

CI Vulkan-Tools build # 934 running.

CI Vulkan-Tools build # 934 passed.

CI Vulkan-Tools build queued with queue ID 42827.

The memory funcs look to be missing useful bodies.

Good catch! The mock has to actually pass a valid pointer back to the application or else things will get a bit crashy.

CI Vulkan-Tools build # 935 passed.


Added logic to pop to root if selected tab is tapped again.

This is standard UITabController behavior that works well if the developer is using navigation controllers.

Will this be merged or closed anytime soon?

thanks


Lite: Add Op Refatcored

1:> Optimized Macro usage to reduce code duplication.
2:> Unnecessary Template usage removed.
3:> Wrong error message corrected.

@ANSHUMAN87 can you please check failed build errors

@ANSHUMAN87 can you please check failed build errors

@gbaned : I have checked, the errors not related to my changes.

@rthadur : The errors are not related to my changes,, would you please help proceed with this PR, Thanks!

@lu-wang-g can you please approve this PR internally

I took a second look at this PR, and found that the changes and benefit were not very obvious at this moment. The original version works well, and let's keep it for now.

I took a second look at this PR, and found that the changes and benefit were not very obvious at this moment. The original version works well, and let's keep it for now.

@lu-wang-g : Thanks for your effort in reviewing, I am sorry i could only partially agree on your conclusion. I totally agree that this PR does not address any functionality issues. But as per benefit part
i beg to differ from your conclusion by stating that, this kind of PR even though not necessary to have, but it is important to have, as it falls under one of the objectives of TFLite("Reduced binary size").
Please correct me if i am wrong, Thanks a lot!

Considering reviewers comments, I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen!

@gbaned : I think @lu-wang-g  has not yet went through my reply to his conclusion earlier, also i strongly feel this PR is important to consider, so lets have conclusion from @lu-wang-g.

@gbaned : I think @lu-wang-g  has not yet went through my reply to his conclusion earlier, also i strongly feel this PR is important to consider, so lets have conclusion from @lu-wang-g.


Copy ERRORLOG.* to log folder for all SQL Server instances.

Enumerate all SQL Server instances and locate the ERRORLOG folder.
Create a log folder called ERRORLOG_ and copy all the ERRORLOG files into it.
Q. Do we need 32-bit SQL support or just assume all modern SQL Servers are 64-bit?

I've just added a pull request to address this task.
For the moment we are not considering instances with 32-bit architecture.

Added and fixed.


Can we add limit to allure history?

Hello, Taisia. I hope you're doing well
Currently, we can see 20 runs in the trend and history, but is it possible to set this parameter to 10 or some other number? Thank you in advance for your attention to this

Hello, Taisia. I hope you're doing well
Currently, we can see 20 runs in the trend and history, but is it possible to set this parameter to 10 or some other number? Thank you in advance for your attention to this

Hello, @AndriiZhydkykh
I assume you are talking about Allure Report.
Actually, allure report history is something that is being controlled by previous runs. Take a look how it is done https://allurereport.org/docs/history-and-retries/#how-to-enable-history
This plugin helps to create json files for test run, but not report itself.

Thanks for this link. I saw that the default state is 20 and we cannot change this state. And I thought maybe there is some function in your version of the plugin that can help change this setting.

@AndriiZhydkykh
Unfortunatelly, there allure report generation has nothing common with json files that this plugin generates.
If you need just to see latest history trend that contains N results before generating report you could delete extra results from allure-results/history/history-trend.json which looks like:
[
  {"data":{"failed":3,"broken":0,"skipped":0,"passed":3,"unknown":0,"total":6}},
  {"data":{"failed":3,"broken":0,"skipped":0,"passed":3,"unknown":0,"total":6}}
]

Okey, thank you. I will try it.


[SPARK-25504][Docs] Update doc about retained tasks, jobs and executions

What changes were proposed in this pull request?
Updated the documentation about spark.ui.retainedTasks, spark.ui.retainedJobs, spark.ui.retainedStages
How was this patch tested?
NA

Test build #99240 has started for PR 23134 at commit ca62b7e.

Hi @srowen , Some of the users seems getting confused with the threshold values, is it worth to update the docs? Thanks

Can one of the admins verify this patch?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5329/
Test PASSed.

Test build #99241 has started for PR 23134 at commit 44d2c58.

Test build #99240 has finished for PR 23134 at commit ca62b7e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99240/
Test PASSed.

In the JIRA, when the user want retainedTasks as 60000, actually retained around 58000.
As per the code, it retained atleast 0.9*retainedTasks to retainedTasks. if we can clearly specify about the threshold, the confusion doesn't arises.
If it is okay with the documentation, I can close the PR.

I think the existing docs are fine.


Typescript: Component is missing the following properties from type

Bug report 🐞
Version & Environment

Version of browser: Firefox 125
Version of geist-ui/core: ^2.3.8
Next.js project with React and Typescript

Expected Behavior
No typescript errors
Actual results (or Errors)
I got an error:
TS2739: Type { children: string; onClick: (_: MouseEvent<HTMLButtonElement, MouseEvent>) => void; } is missing the following properties from type Pick<Props & NativeAttrs & RefAttributes<HTMLButtonElement> & ScaleProps, "title" | "color" | "translate" | ... 260 more ... | "key"> : placeholder, onPointerEnterCapture, onPointerLeaveCapture
My code:
import { Button } from "@geist-ui/core";

export default function Page() {
  ...
  return (
    <div>
      <Button onClick={_ => console.log("Hello")}>Action</Button> // error shows here
    </div>
  )
}

Apparently props like placeholder, onPointerEnterCapture, and onPointerLeaveCapture are made required, is this intentional?

Same error.

// @ts-ignore can be used to solve compilation errors temporarily.

Same error. And i don't want to use //@ts-ignore

Same error. And i don't want to use //@ts-ignore

I had to add all the properties as undefined. Please a fix is needed. Code is looking very unclean.
Properties causing issues: placeholder, onPointerEnterCapture, and onPointerLeaveCapture

Same issue here. 🙁

The geist-ui is no longer maintained apparently. Probably just switch to another component library instead.

For a small toy project I was able to fix the error using an additional declaration file.
I am not sure if this is depending on other parts of my setup, notably the project uses "type": "module" in package.json
Adding a declaration file anywhere in my repository with this content fixed the issue for me, without removing geist-ui:
import "react";

/*
workaround for https://github.com/geist-org/geist-ui/issues/858
*/
declare module "react" {
  export interface ButtonHTMLAttributes {
    placeholder?: ButtonHTMLAttributes["placeholder"];
    onPointerEnterCapture?: ButtonHTMLAttributes["onPointerEnterCapture"];
    onPointerLeaveCapture?: ButtonHTMLAttributes["onPointerLeaveCapture"];
  }
  export interface AnchorHTMLAttributes {
    placeholder?: AnchorHTMLAttributes["placeholder"];
    onPointerEnterCapture?: AnchorHTMLAttributes["onPointerEnterCapture"];
    onPointerLeaveCapture?: AnchorHTMLAttributes["onPointerLeaveCapture"];
  }
}

React DefinitelyTyped declarations are referenced in @geist-ui/core/esm/button/button.d.ts and @geist-ui/core/esm/link/link.d.ts.
But I couldnt find out what exactly goes wrong there, of course updated to the latest React types, but I couldn't find anything there declaring these props/attributes as required.
Removing
& NativeAttrs from this line seems to make all props/attributes required:
declare const Button: React.ForwardRefExoticComponent<Pick<Props & NativeAttrs & React.RefAttributes<HTMLButtonElement> & import("../use-scale").ScaleProps, "children" | "form" | "slot" | "style" | "title" | keyof import("../use-scale").ScaleProps | "value" | "placeholder" | "onChange" | "onFocus" | "onBlur" | "autoFocus" | "formAction" | "formEncType" | "formMethod" | "formNoValidate" | "formTarget" | "name" | "defaultChecked" | "defaultValue" | "suppressContentEditableWarning" | "suppressHydrationWarning" | "accessKey" | "contentEditable" | "contextMenu" | "dir" | "draggable" | "hidden" | "id" | "lang" | "spellCheck" | "tabIndex" | "translate" | "radioGroup" | "role" | "about" | "datatype" | "inlist" | "prefix" | "property" | "resource" | "typeof" | "vocab" | "autoCapitalize" | "autoCorrect" | "autoSave" | "color" | "itemProp" | "itemScope" | "itemType" | "itemID" | "itemRef" | "results" | "security" | "unselectable" | "inputMode" | "is" | "aria-activedescendant" | "aria-atomic" | "aria-autocomplete" | "aria-busy" | "aria-checked" | "aria-colcount" | "aria-colindex" | "aria-colspan" | "aria-controls" | "aria-current" | "aria-describedby" | "aria-details" | "aria-disabled" | "aria-dropeffect" | "aria-errormessage" | "aria-expanded" | "aria-flowto" | "aria-grabbed" | "aria-haspopup" | "aria-hidden" | "aria-invalid" | "aria-keyshortcuts" | "aria-label" | "aria-labelledby" | "aria-level" | "aria-live" | "aria-modal" | "aria-multiline" | "aria-multiselectable" | "aria-orientation" | "aria-owns" | "aria-placeholder" | "aria-posinset" | "aria-pressed" | "aria-readonly" | "aria-relevant" | "aria-required" | "aria-roledescription" | "aria-rowcount" | "aria-rowindex" | "aria-rowspan" | "aria-selected" | "aria-setsize" | "aria-sort" | "aria-valuemax" | "aria-valuemin" | "aria-valuenow" | "aria-valuetext" | "dangerouslySetInnerHTML" | "onCopy" | "onCopyCapture" | "onCut" | "onCutCapture" | "onPaste" | "onPasteCapture" | "onCompositionEnd" | "onCompositionEndCapture" | "onCompositionStart" | "onCompositionStartCapture" | "onCompositionUpdate" | "onCompositionUpdateCapture" | "onFocusCapture" | "onBlurCapture" | "onChangeCapture" | "onBeforeInput" | "onBeforeInputCapture" | "onInput" | "onInputCapture" | "onReset" | "onResetCapture" | "onSubmit" | "onSubmitCapture" | "onInvalid" | "onInvalidCapture" | "onLoad" | "onLoadCapture" | "onError" | "onErrorCapture" | "onKeyDown" | "onKeyDownCapture" | "onKeyPress" | "onKeyPressCapture" | "onKeyUp" | "onKeyUpCapture" | "onAbort" | "onAbortCapture" | "onCanPlay" | "onCanPlayCapture" | "onCanPlayThrough" | "onCanPlayThroughCapture" | "onDurationChange" | "onDurationChangeCapture" | "onEmptied" | "onEmptiedCapture" | "onEncrypted" | "onEncryptedCapture" | "onEnded" | "onEndedCapture" | "onLoadedData" | "onLoadedDataCapture" | "onLoadedMetadata" | "onLoadedMetadataCapture" | "onLoadStart" | "onLoadStartCapture" | "onPause" | "onPauseCapture" | "onPlay" | "onPlayCapture" | "onPlaying" | "onPlayingCapture" | "onProgress" | "onProgressCapture" | "onRateChange" | "onRateChangeCapture" | "onSeeked" | "onSeekedCapture" | "onSeeking" | "onSeekingCapture" | "onStalled" | "onStalledCapture" | "onSuspend" | "onSuspendCapture" | "onTimeUpdate" | "onTimeUpdateCapture" | "onVolumeChange" | "onVolumeChangeCapture" | "onWaiting" | "onWaitingCapture" | "onAuxClick" | "onAuxClickCapture" | "onClickCapture" | "onContextMenu" | "onContextMenuCapture" | "onDoubleClick" | "onDoubleClickCapture" | "onDrag" | "onDragCapture" | "onDragEnd" | "onDragEndCapture" | "onDragEnter" | "onDragEnterCapture" | "onDragExit" | "onDragExitCapture" | "onDragLeave" | "onDragLeaveCapture" | "onDragOver" | "onDragOverCapture" | "onDragStart" | "onDragStartCapture" | "onDrop" | "onDropCapture" | "onMouseDown" | "onMouseDownCapture" | "onMouseEnter" | "onMouseLeave" | "onMouseMove" | "onMouseMoveCapture" | "onMouseOut" | "onMouseOutCapture" | "onMouseOver" | "onMouseOverCapture" | "onMouseUp" | "onMouseUpCapture" | "onSelect" | "onSelectCapture" | "onTouchCancel" | "onTouchCancelCapture" | "onTouchEnd" | "onTouchEndCapture" | "onTouchMove" | "onTouchMoveCapture" | "onTouchStart" | "onTouchStartCapture" | "onPointerDown" | "onPointerDownCapture" | "onPointerMove" | "onPointerMoveCapture" | "onPointerUp" | "onPointerUpCapture" | "onPointerCancel" | "onPointerCancelCapture" | "onPointerEnter" | "onPointerEnterCapture" | "onPointerLeave" | "onPointerLeaveCapture" | "onPointerOver" | "onPointerOverCapture" | "onPointerOut" | "onPointerOutCapture" | "onGotPointerCapture" | "onGotPointerCaptureCapture" | "onLostPointerCapture" | "onLostPointerCaptureCapture" | "onScroll" | "onScrollCapture" | "onWheel" | "onWheelCapture" | "onAnimationStart" | "onAnimationStartCapture" | "onAnimationEnd" | "onAnimationEndCapture" | "onAnimationIteration" | "onAnimationIterationCapture" | "onTransitionEnd" | "onTransitionEndCapture" | "key" | keyof Props> & React.RefAttributes<HTMLButtonElement>>;

(NativeAttrs is a local alias for Omit<React.ButtonHTMLAttributes<any>, keyof Props>; in geist-ui's esm declarations)
So it seems that geist-ui does some override thing that should make these attributes optional, but it broke with an update of the React DT types.
ofc I would also recommend migrating away from this non-maintained component library.
but for making a project "just work" that is itself not really maintained, this is enough, as it is purely an issue with external declarations.


[[RFE] : creating users when authenticated through defined authentication source (like oauth2)

Gitea version (or commit ref): 1.3.2
Operating system: CentOS 7
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
Not a bug but request for enhancement : Actually, gitea permits users logged it through openid to have the corresponding user automatically created in gitea  :
ENABLE_OPENID_SIGNIN = true
ENABLE_OPENID_SIGNUP = true
But it doens't work if one decides to create a authentication source like oauth2 (so tested with OpenID Connect on my side) : once authenticated, and redirected to gitea, it asks to either :

login with existing (local) user (doesn't exist yet)
create a new user account

Of course I'd like to only allow people authenticated through oauth2 to sign in and so be registered automatically (so DISABLE_REGISTRATION  = true)
As discussed in #gitea on freenode, creating a RFE ticket here so that people can discuss it and/or track a potential status change
...

is there actually some progress on this?

@schmitch Looks like #5123 is the current PR

closed by #5123


feature request: ssh jumphosts

Hi,
Please add ability to open ssh proxy connection through one or more ssh jumphosts (like ssh -J). Proof of concept patch can be used like:
pproxy.Connection('ssh://<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>#tunnel1:password\ntunnel2:password2\ntunnel3:password3\ntunnel4:password4')

This feature is very cool, although it doesn't conform with the JUMP syntax pproxy are using.
I have submitted v2.5.1 supporting this new feature.
The JUMP syntax is using double underscore "__":
$ pproxy -r ssh://server1#user1:pass1__ssh://server2#user2:pass2__ssh://server3#user3:pass3
It is flexible. We can use protocol other than "ssh://" on the second or third jump:
$ pproxy -r ssh://server1#user1:pass1__socks5://server2:port2__http://server3:port3
However, using "ssh://" without preceding "ssh://" jump is illegal.


[EdgeTPU] Add EdgeTPU Advanced Option 'Min Runtime Version' to cfgEditor

Implemented Min Runtime Version dropdown on View
There are only 5 versions in Documents
On 'help', add description about compiler and runtime version
Not Styled Version, Only function

ONE-vscode-DCO-1.0-Signed-off-by: hohee-hee<EMAIL_ADDRESS>

LGTM :)


You are absolutely prolific!

I see your name everywhere on github these days :) So glad to see you're doing great stuff!!


src_dirs should be able to accept a list of source_directories without participant_ids

There are cases where you have multiple data directories (like rest1,rest2) where you want to search for files. All of these directories contain files for MULTIPLE subjects. Right now, the function expects a mapping from each directory to ONE participant id and therefore throws an error.

Specifcally this error is raised
       if isinstance(src_dirs,(list,pd.Series)):

            if not len(src_dirs) == len(participant_ids):
                raise ValueError('Participant IDs and source directories must be of the same length')

because the function assumes that each src_directory maps on one subject_id which is not true for the described case above.

Just realized that one can also use the exclude_dirs option for this case. For example, I had a parent folder called data with rest1,rest2,nback,social_task etc. in it. I only want the files in the two rest folders. I can use the exclude dirs option to ignore everything that is not rest1 or rest2. But still this should be fixed, since it would be more intuite to pass [path/to/rest1,path/to/rest2] as src_dirs to the function.

The idea is, that it should be allowed to pass multiple src_dirs that do NOT represent different participant folders. The function is too specific right now.


Ability to mount data to Tool Launcher containers

Description
Currently, the Tool Launcher copies data into the Docker container via the Clowder API (via HTTP).  As per ticket #215, for some development workflows this model won't work (e.g., hyperspectral), as the files are too large.  This ticket encompasses changes to the Tool Launcher to support mounting data on ROGER directly (RO).
https://github.com/terraref/computing-pipeline/issues/215
please edit this template before submitting a new pull request
Details
This will eventually (hopefully) allow us to replace the current shared VM with a Docker-based workflow.
Completion Criteria

[ ] Move tool launcher VM on ROGER/OpenStack with data mounted via NFS
[ ] Update tool launcher to mount data RO to containers
[ ] Update tool launcher design documentation
[ ] Update tool launcher test case
[ ] Tests pass
[ ] Merge PR

This has been replaced by Labs Workbench in #278


Status Definitions

I cannot find any documentation on the status of sales order and purchase orders.
For Example in the Sales Order the status can be either Open, Cancelled or Released. What other status are available also where else can Status be used? Purchase Orders?
I want to suggest that more status are created but I cannot see a list of statuses. I will create a BCideas for more statuses.
[Enter feedback here]

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

ID: 9f842e9d-aa0b-1b1c-c6be-dfac6730e393
Version Independent ID: 5ba6fbd0-487c-c742-9acc-8b120c9337ff
Content: Using General Functionality Across Business Areas - Business Central
Content Source: business-central/ui-across-business-areas.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: edupont

Hello Ben,
Thank you for your feedback! The product has many fields with the name Status, and they fall into a couple of different "types" of status, depending on the type of page that the field is on.
Specifically for documents such as sales orders/purchase invoices/other sales and purchasing documents, the Status field has 4 values in the base application: Open, Released, Pending Approval, and Pending Prepayment. These are somewhat explained in the tooltip for the field. Those 4 values support two fairly common workflows in many businesses, namely document approval (you can only post the invoice after XYZ has signed off on it) and prepayments (don't ship the products until the agreed-upon prepayments have been registered). There is a lot of business logic hinging on this field for documents to test that the document is in the right "state" before being processed.
Different ISVs may have extended the Status field to support other workflows, and you can extend it yourself with a customer-specific or app-specific extension, for example.
The many other Status fields in the base product have different logic around them but typically don't fall into a collective bucket like the Status fields on documents.
As for suggesting we add more states, I'll leave that to the engineering team to process.


Browser in php-fpm Docker times out and never fetches data (default LoopInterface implementation)

Prerequisites:

The script is run in a php-fpm Docker container
The command is wrapped in an instance of Symfony\Component\Console\Command\Command
Host machine MacOS latest

Test:
$loop = React\EventLoop\Factory::create();

$loop->addTimer(11, function()use($loop) {
            echo "Timed out\n";
            $loop->stop();
        });

$browser = new Browser($loop);

            /** @var Promise $promise */
            $promise = $browser->get('http://ukr.net/');
            $promise->done(
                function(Response $data){
                    echo 'Fulfilment code =: ' . $data->getCode();
                },
                function($reason) {
                    echo 'Rejection reason: ' . (string)$reason;
                }
            );

$loop->run();

Please note that requesting the same URL through php curl OR curl in console works just fine.
The result is always a Timed out output.
Am I doing something wrong? If not, where do I start to dig to find out what the issue is?

Are you using the latest version? It looks like your resolver function will not be called due to the unknown Response type hint. The then() method will return a rejected promise in this case, you can chain this with an empty ->done() call to report any issues in your chain.
It also looks like you may want to cancel the timer once the response is done. Currently, it will print the "timed out" message irrespective of whether the response resolved or rejected. See also #1.
I hope this helps :+1:


mihome-vacuum adapter version 2.0.1 does not start

hi,
yesterday i installed the new version (2.0.1).
After starting the instance i get the following error log:

mihome-vacuum.0
2020-02-14 06:38:46.781
info
(1181) Terminated (NO_ERROR): Without reason

mihome-vacuum.0
2020-02-14 06:38:46.779
info
(1181) terminating

mihome-vacuum.0
2020-02-14 06:38:46.718
error
at promise.then (/opt/iobroker/node_modules/standard-as-callback/built/index.js:19:49)

mihome-vacuum.0
2020-02-14 06:38:46.718
error
at tryCatcher (/opt/iobroker/node_modules/standard-as-callback/built/utils.js:11:23)

mihome-vacuum.0
2020-02-14 06:38:46.718
error
at client.keys (/opt/iobroker/node_modules/iobroker.js-controller/lib/states/statesInRedis.js:684:17)

mihome-vacuum.0
2020-02-14 06:38:46.718
error
at states.getKeys (/opt/iobroker/node_modules/iobroker.js-controller/lib/adapter.js:6814:25)

mihome-vacuum.0
2020-02-14 06:38:46.718
error
at initLogging (/opt/iobroker/node_modules/iobroker.js-controller/lib/adapter.js:7117:22)

mihome-vacuum.0
2020-02-14 06:38:46.718
error
at Adapter.emit (events.js:208:7)

mihome-vacuum.0
2020-02-14 06:38:46.718
error
at emitNone (events.js:106:13)

mihome-vacuum.0
2020-02-14 06:38:46.718
error
at Adapter.main (/opt/iobroker/node_modules/iobroker.mihome-vacuum/main.js:1173:15)

mihome-vacuum.0
2020-02-14 06:38:46.718
error
at new Maphelper (/opt/iobroker/node_modules/iobroker.mihome-vacuum/lib/maphelper.js:35:24)

mihome-vacuum.0
2020-02-14 06:38:46.718
error
at JSON.parse ()

mihome-vacuum.0
2020-02-14 06:38:46.718
error
(1181) SyntaxError: Unexpected end of JSON input

mihome-vacuum.0
2020-02-14 06:38:46.717
error
(1181) uncaught exception: Unexpected end of JSON input

mihome-vacuum.0
2020-02-14 06:38:46.696
info
(1181) Create state clean_home for controlling by cloud adapter

mihome-vacuum.0
2020-02-14 06:38:46.694
info
(1181) Expert mode enabled, states created

mihome-vacuum.0
2020-02-14 06:38:46.641
info
(1181) starting. Version 2.0.1 in /opt/iobroker/node_modules/iobroker.mihome-vacuum, node: v8.16.0

mihome-vacuum.0
2020-02-14 06:38:43.305
debug
(1181) statesDB connected

mihome-vacuum.0
2020-02-14 06:38:43.304
debug
(1181) States connected to redis: <IP_ADDRESS>:9000

mihome-vacuum.0
2020-02-14 06:38:43.283
debug
(1181) States create System PubSub Client

mihome-vacuum.0
2020-02-14 06:38:43.278
debug
(1181) States create User PubSub Client

mihome-vacuum.0
2020-02-14 06:38:43.264
debug
(1181) Redis States: Use Redis connection: <IP_ADDRESS>:9000

mihome-vacuum.0
2020-02-14 06:38:43.258
debug
(1181) objectDB connected

mihome-vacuum.0
2020-02-14 06:38:43.244
debug
(1181) Objects connected to redis: <IP_ADDRESS>:9001

mihome-vacuum.0
2020-02-14 06:38:43.214
debug
(1181) Objects client initialize lua scripts

mihome-vacuum.0
2020-02-14 06:38:43.212
debug
(1181) Objects create User PubSub Client

mihome-vacuum.0
2020-02-14 06:38:43.210
debug
(1181) Objects create System PubSub Client

mihome-vacuum.0
2020-02-14 06:38:43.201
debug
(1181) Objects client ready ... initialize now

mihome-vacuum.0
2020-02-14 06:38:42.313
debug
(1181) Redis Objects: Use Redis connection: <IP_ADDRESS>:9001

host.raspberrypi
2020-02-14 06:38:40.648
info
instance system.adapter.mihome-vacuum.0 started with pid 1181

host.raspberrypi
2020-02-14 06:38:40.468
info
"system.adapter.mihome-vacuum.0" enabled

system information:
Node.js v8.16.0
NPM 6.13.6

Already fixed


How is to invoke TraceParserGen to generate .cs file for providers in .NET Core?

I cloned the repo, and compiled to generate TraceParserGen.exe. However, I am not getting an example how to get auto-generated .cs file for a provider in .NET Core.
For example, the doc explains a bunch of providers. Out of which, I am looking to generate .cs for provider Microsoft-System-Net-Http. Could you please share the exact syntax with steps and parameters to generate associated .cs file?

The provider Microsoft-System-Net-Http is implemented as an EventSource, so in order to produce a C# parser, you will want to point traceparsergen.exe at the dll that contains the EventSource.  Something like:
traceparsergen.exe <path to dll> -EventSource:Microsoft-System-Net-Http`

I'm not sure which dll contains the EventSource implementation, but that's the one you'll want to target with this.  Also, FWIW, you should not need to do this, though it can help with performance and code readability.  You can use DynamicTraceEventParser directly to consume these events without needing to generate a C# parser.

@brianrob
Thank you for the response! Do we have any example how DynamicTraceEventParser can be leveraged? I could find the source of it, but not any example. Could you please help?

Here's a bit of documentation on how it works: https://github.com/microsoft/perfview/blob/master/documentation/TraceEvent/TraceEventProgrammersGuide.md#static-vs-dynamic-traceeventparser-parsers
Basically, you get a TraceEvent instance in your callback, and you can use the available properties to figure out if it's the event you want (e.g. provider, event name, etc.) and then get the fields by asking for them by name.


Update GIT/BAT API

Context
We need to make a few more changes to the V1.1 API. We're going to move application status and phase to sit under each application form rather than at the top level on candidate.
This will be done in 2 phases to avoid breaking changes. Firstly, we will add it to application form, once GIT have updated on their end. Once that is done we will remove it from the top level (candidate) on our end.
We also need to make sure we're updating the candidate_api_updated_at  when either of the above values are updated on any application form.
Changes proposed in this pull request

Update the candidate_api_updated_at when an application status or phase are updated
Add application status and phase to the application forms

Guidance to review
As application status is an abstracted state we need to see if an application forms status is updated when an application choices status is updated, as that is what triggers potential state changes at the application form level.
Link to Trello card
https://trello.com/c/waFnqttv/3884-git-bat-api-v11-changes
Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] Required environment variables have been updated added to the Azure KeyVault

This looks good. I think we should start a changelog for this API. I'll raise a separate PR.


Change Autocomplete onChange event to be a real Event fired from the input element

Feature description
I would like the onChange event be a real Event fired from the input component to easily integrate the Autocomplete component with libraries managing forms and inputs.
Trying to integrate react-hook-form using uncontrolled state is giving the current error when changing the autocomplete value:

Describe the solution you'd like
The solution I would like is to have the onChange event be a real Event fired from the input component the ref prop is forwarded to.
Describe alternatives you've considered
No response
Additional context and information
We need this feature to easily integrate react-hook-form hooks into our toolbelt. Without this we can integrate the autocomplete but in a more manual and tedious way because we need to control the input ourselves, making the process possibly error prone.
Code of Conduct

[X] I've searched across open and closed issues for similiar requests

Currently, onChange prop refers to option picking. Maybe the name is misleading. In any case, the textfield already uses the onChange event to filter options inside the autocomplete, so it cannot be used to detect the picked value.

onChange event and the forwarded ref are now bound to the input element


python3 support

Anyone happen to be working on python3 support?

I posted this not realizing that I was asking in the wrong repo; apologies.
The elastalert repo already has an issue tracking python3 support, which is a prereq.
https://github.com/Yelp/elastalert/pull/1210


Perplexity ask

Resolves #866
Quality Assurance:

https://github.com/ubq-testing/bot-ai/issues/18#issue-1961709713 Perp vs GPT
https://github.com/ubq-testing/bot-ai/issues/17#issuecomment-1779542172 Shit QA

So after a lot of fucking around I managed to get our tokens estimates pretty dead on, but adding in the additional context from linked sources is throwing it off again
I tried to create formats where if we had the space we'd select the next format up and try to consume tokens that way but it wasn't very fruitful, so as is right now we are having GPT dictate the context that gets fed into Perp.
I think it's lacking a lot compared directly against gpt, plus it's required us to bring in another package into the build which I know you want to avoid at all costs.
If funded I'd spend the time and hack together a TikToken based tokenizer for use

Perhaps using Perplexity as the researcher and ChatGPT as the thinker?

 I'm unsure how best to apply it efficiently with the context size, if the token limits were flipped that would be workable. Until such times they release the bigger context window I think using it adds too much complexity to /ask where gpt3.5 performs just fine on it's own.
But I could see feeding perp all of our additional context and having it research and summarize, then gpt acting on the information that would probs be v effective.


fix: Cache node labels being overwritten

The previous code was using the same instance of LabelSet for creating labels for all cache nodes, this would override all cache nodes to use the same labelset as the last one.
This solution simply clones one instance of the CacheCluster label set to each of the nodes before overriding it.

@maxbrunet this error makes it impossible to monitor memcached clusters with multiple nodes, which is very critical for us, and I guess for the tool, can you take a look at it?


[TF2.0] How to get the intermediate layers output of pretrained model?

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Y
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): 5.3.5-arch1-1-ARCH
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.0.0
Python version: 3.7.4
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce: N/A

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
This is exactly the same issue as https://github.com/tensorflow/models/issues/5236
which has just become more of a real issue with TF 2.0.
In v1, I do this:
outputs = list(map(lambda tname: tf.get_default_graph().get_tensor_by_name(tname), [
    'DCNN/block3_pool/MaxPool:0',
    'DCNN/block4_pool/MaxPool:0',
    'DCNN/block5_pool/MaxPool:0'
]))

with tf.Session() as sess:
    val_outputs = sess.run(outputs)

What is the recommended way to do this with TF 2.0 python API now that graphs and sessions have been eliminated? Especially without access to the code that builds the model, which has been the greatest use case of get_tensor_by_name()? Is there any solution other than using tf.compat.v1?
Example:
dcnn = tf.keras.applications.VGG16(include_top=False, weights='imagenet')
# How to access vgg16/block3_pool/MaxPool:0 in TF 2.0 without the knowledge of the model source code?

Is there any upgrade instruction? Or plans to accommodate such use cases?
Thanks!

I meet the same problem. Is there any instruction ?

Try this
from tensorflow_core.python.keras.applications.mobilenet_v2 import MobileNetV2
from tensorflow_core.python.keras.engine.input_layer import Input
from tensorflow_core.python.keras.models import Model

IMG_SIZE = image_size = 224
IMG_SHAPE = (IMG_SIZE, IMG_SIZE, 3)

inputs = Input(shape=IMG_SHAPE)
base_model = MobileNetV2(input_shape=IMG_SHAPE,
                             include_top=False,
                             weights='imagenet')
base_model.summary()
feature = base_model(inputs)
Model(inputs=inputs, outputs=feature)

intermediate_from_a = base_model.get_layer("Conv1").output
print(feature.shape)
print(intermediate_from_a.shape)

it is no longer necessary to compile the model to use predict.

@aleozlx, Just to verify, did you go through this link. Thanks!

Try this
import tensorflow as tf
from tensorflow_core.python.keras.applications.mobilenet_v2 import MobileNetV2
from tensorflow_core.python.keras.layers.convolutional import Conv2D
from tensorflow_core.python.keras.models import Model
import numpy as np
from tensorflow_core.python.ops.init_ops_v2 import he_normal

IMG_SIZE = image_size = 224
IMG_SHAPE = (IMG_SIZE, IMG_SIZE, 3)

base_model = MobileNetV2(input_shape=IMG_SHAPE,
                         include_top=False,
                         weights='imagenet')
base_model.trainable = False
base_model.summary()
layer_name = "block_13_expand_relu"
layer_output = base_model.get_layer(layer_name).output
x = Conv2D(16,
           kernel_size=[3, 3],
           strides=[1, 1],
           padding='same',
           kernel_initializer=he_normal(),
           activation=tf.nn.relu,
           name="conv3",
           kernel_regularizer=tf.keras.regularizers.l2(0.01))(layer_output)

model = Model(base_model.input, outputs=[layer_output, x])

input_data = np.random.rand(1, *IMG_SHAPE)
result = model.predict(input_data)
print(result[0].shape)
print(result[1].shape)

Not sure if I misunderstand the question. I think this Stack Overflow thread covered your question.
The only difference is that in Keras,
earlyPredictor = Model(dcnn.inputs,dcnn.get_layer(theNameYouWant).outputs)
will become the following in TF 2.0 (note it's output instead of outputs)
earlyPredictor = tf.keras.Model(dcnn.inputs,dcnn.get_layer(theNameYouWant).output).

@gadagashwini It is unclear how the link provided answers to the question. That does not seem to help.
The rest of the answers are acceptable and do provide a feasible direction with .get_layer(). Thanks!
However, at least 80% of our codes are custom and therefore I want to wait a little more to see if there is a non-Keras answer cuz we don't much care for wrapping everything up into Keras layers and models and all that OOP goodness.
And the ultimate doubt that I still have is whether the DAG paradigm is so far completed eliminated from the v2 API? An official/definitive answer would be of great help because there was value to that. If such is the case, I'll just adapt. Thanks again!

@aleozlx This is not Build/Installation or Bug/Performance issue. Please post this kind of support questions at Stackoverflow. There is a big community to support and learn from your questions. GitHub is mainly for addressing bugs in installation and performance. Thanks!

The question seems sufficiently addressed at this point. Thank you all!

Hi @aleozlx . Please see if my recent answer is maybe close to the non-keras way you were looking for.
https://stackoverflow.com/questions/59493222/access-output-of-intermediate-layers-in-tensor-flow-2-0-in-eager-mode/60945216#60945216

Seems like this is a little old, but just in case, if you want to do this similar to what you had before, you could do this.
dcnn = tf.keras.applications.VGG16(include_top=False, weights='imagenet')

network_function = tf.function(lambda inputs: dcnn(inputs),
                               [tf.TensorSpec(tf.TensorShape([None, 224, 224, 3]))])

outputs = list(map(lambda tname: network_function.get_concrete_function().graph.get_tensor_by_name(tname), [
    'vgg16/block3_pool/MaxPool:0',
    'vgg16/block4_pool/MaxPool:0',
    'vgg16/block5_pool/MaxPool:0'
]))

Seems like this is a little old, but just in case, if you want to do this similar to what you had before, you could do this.
dcnn = tf.keras.applications.VGG16(include_top=False, weights='imagenet')

network_function = tf.function(lambda inputs: dcnn(inputs),
                               [tf.TensorSpec(tf.TensorShape([None, 224, 224, 3]))])

outputs = list(map(lambda tname: network_function.get_concrete_function().graph.get_tensor_by_name(tname), [
    'vgg16/block3_pool/MaxPool:0',
    'vgg16/block4_pool/MaxPool:0',
    'vgg16/block5_pool/MaxPool:0'
]))

how do i predict  when get the outputs ?
    model = train_model()
    func = tf.function(model)
    print(model.input)
    tensor_specs1 = tf.TensorSpec.from_tensor(model.input)

    call = func.get_concrete_function(tensor_specs1)
    graph = call.graph

    tensor_names = [n for n in graph.as_graph_def().node]
    for name in tensor_names:
        print(name)

    outputs = graph.get_tensor_by_name('functional_1/output/Softmax:0')

    pred_model = tf.keras.models.Model(model.input,outputs)
    results = pred_model(tensor_specs1)

but raise an exception:
  File "/Users/anaconda/envs/tf2/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 3786, in _as_graph_element_locked
    raise ValueError("Tensor %s is not an element of this graph." % obj)
ValueError: Tensor Tensor("functional_1/output/BiasAdd:0", shape=(?, 10), dtype=float32) is not an element of this graph

Try this
import tensorflow as tf
from tensorflow_core.python.keras.applications.mobilenet_v2 import MobileNetV2
from tensorflow_core.python.keras.layers.convolutional import Conv2D
from tensorflow_core.python.keras.models import Model
import numpy as np
from tensorflow_core.python.ops.init_ops_v2 import he_normal

IMG_SIZE = image_size = 224
IMG_SHAPE = (IMG_SIZE, IMG_SIZE, 3)

base_model = MobileNetV2(input_shape=IMG_SHAPE,
                         include_top=False,
                         weights='imagenet')
base_model.trainable = False
base_model.summary()
layer_name = "block_13_expand_relu"
layer_output = base_model.get_layer(layer_name).output
x = Conv2D(16,
           kernel_size=[3, 3],
           strides=[1, 1],
           padding='same',
           kernel_initializer=he_normal(),
           activation=tf.nn.relu,
           name="conv3",
           kernel_regularizer=tf.keras.regularizers.l2(0.01))(layer_output)

model = Model(base_model.input, outputs=[layer_output, x])

input_data = np.random.rand(1, *IMG_SHAPE)
result = model.predict(input_data)
print(result[0].shape)
print(result[1].shape)

Can anyone tell me why we are using x here?


Fix DefaultSubmarinerOperatorVersion calculation

eq can't be used with $(if), this uses $(findstring) instead which
also allows the overall test to be simplified: all we care about is
whether "devel" appears somewhere in $(CALCULATED_VERSION).
See
https://www.gnu.org/software/make/manual/html_node/Text-Functions.html
for details.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

something is still off @skitt :
2021-04-06T12:18:18.1790947Z �[36m[submariner-operator]$ CGO_ENABLED=0 go build -ldflags -s -w -X github.com/submariner-io/submariner-operator/pkg/version.Version= 		   -X=github.com/submariner-io/submariner-operator/pkg/versions.DefaultSubmarinerOperatorVersion= -o bin/subctl--linux-amd64 ./pkg/subctl/main.go

Shall we move that calculation to a script which is easier to debug may be?

yeah CALCULATED_VERSION is empty in CI E2E - this is what I noted in https://github.com/submariner-io/submariner-operator/pull/1217. I'm not sure why. Perhaps we could just have the GHA yaml set/pass CALCULATED_VERSION to make. So for E2E job it passes "devel". Shipyard does override CALCULATED_VERSION := ...  which I believe will initialize if not set.

Superseded by https://github.com/submariner-io/submariner-operator/pull/1227


Package swiftmailer/swiftmailer is abandoned, you should avoid using it. Use symfony/mailer instead

The swiftmailer/swiftmailer package is in the end of its life-cycle.
The composer suggestion is:
Package swiftmailer/swiftmailer is abandoned, you should avoid using it. Use symfony/mailer instead.
Since Laravel 8 also uses this package "monolog/monolog", the "monolog/monolog" must also switch in order for Laravel 8 to switch to the symfony/mailer package.

Sorry I Don't understand how this is affecting Laravel.
I see that swiftmailer is under the "require-dev" Section of monolog's composer.json file... Not in "require".
Can you please explain better?
Also... There is always a chance to write a custom Handler... In casé you wanted to refer to the SwiftmailerHandler.
This is the second issue regarding this topic and I would like to know if We Are under a problem that we Don't see..... And that needs to be treated ASAP.
Pd: even though that... Yes... This event should bé considered but also we Have to keep in mind that the SwiftmailerHandler SHOULD NOT bé deleted as there might be many systems still using it.
Thanks in Advance!

please read the answers in https://github.com/Seldaek/monolog/issues/1611
To me, there is nothing to change in monolog. Monolog does not force you to use Swiftmailer at all. It allows you to do so if you want.

@trianity any comment on this? If you agree with us, this issue can be closed, otherwise let us know.

@juan-morales @stof You are right. Thank you for the attention, and I am sorry that I missed the #1611 conversion. Ok, for close.