Using Special type activation function

Hi,
I am pretty new to neurodiffeq, thank you very much for an excellent library. I want to use special type activation function in particular
f(x)= cos(1.75 x)* exp (-x**2/2)
so how I will define the class for using this activation function? and how to call this function in this line
fcnn=FCNN(hidden_units=(50,50, ), actv=nn.Tanh)
Thank you in advance

Hi @SK-Math , you can easily achieve this by implementing your own torch module.
import torch
from neurodiffeq.networks import FCNN

class MyActivation(torch.nn.Module):
	def forward(self, x):
		return torch.cos(1.75 * x) * torch.exp(-x**2/2)

fcnn = FCNN(hidden_units=(50, 50), actv=MyActivation)

Let me know if there are further questions.

Thnka you @shuheng-liu for your quick responce

def forward(self, x):
        return  x*torch.exp(0.5)

when calling in solver getting error like
TypeError: exp(): argument 'input' (position 1) must be Tensor, not float
how to fix it

exp(0.5) is just a constant. You can use np.exp or math.exp instead of torch.exp, which only works for pytorch tensors (similar to bumpy arrays)

Hi @shuheng-liu
How can I use Butterfly Optimization Algorithm instead of  default ADAM optimization?
As I see torch.optim. has not Butterfly Optimization
Thank you

I haven't heard of the butterfly optimization algorithm. If you really want to try it out, you probably have to implement your own Optimizer. Here is an article.

Can you be a little bit more specific as to "Loss and validation graph"? In general, you can inject behavior, including custom plotting and logging using the callback feature.

Hi,
Can I write ode= 0.5*u''(x) + u'(0.5x) + u(x) like  ode= lambda u, t: [ 0.5* diff(u, t, order=2) + diff(u, t/2) + u]
Just want to confirm writing of this term u'(0.5x)
please check.

Is neurodiffeq support fractional order diff. eq?

Thank you

Hi,

Can I write ode= 0.5u''(x) + u'(0.5x) + u(x) like ode= lambda u, t: [ 0.5 diff(u, t, order=2) + diff(u, t/2) + u]
Just want to confirm writing of this term u'(0.5x)
please check.

Is neurodiffeq support fractional order diff. eq?

Thank you

Hi, the first equation is, by definition, not a differential equation. It is a generic functional equation with differential operators. Hence, NeuroDiffEq cannot be used to solve this equation out of the box. But you can still solve it with neural networks, you just have to write some codes by yourself.
At the moment, NeuroDiffEq doesn't support fractional derivatives.


NTR: "chorionic villi sample - blood" and "fibroblasts - blood"

I'm putting these potential terms up for discussion here, although I haven't prepared logical definitions yet.
There are a decent number of lab results in our EHR for which the specimen is "chorionic villi sample - blood": ~ 3000 for a sample of 40 patients, compared to ~ 100k for blood and ~ 15k for NA.
I think there are also "chorionic villi sample - fibroblast" specimens in the EHR, too (just not for this patient subset).
In addition to talking about the semantics of these terms, I'd be delighted if any of the clinically-oriented people in our group could say what these specimens really are and what they're good for. I see them matched up with just about the same analytes that are associated with plain blood specimens. Casual Googling hasn't led me anywhere useful yet. I may also talk to some clinicians here at Penn.

Thanks for the great discussion. I think @cstoeckert and I are going to put this on hold until we have more evidence that iur collaborators would actually search over "chorionic villi sample - blood" specimens when building cohorts.

This issue arose from the TURBO project which has not ended up needing these terms. Can reopen if that changes.


Switch from request to phin!

request is way too large and has way too many dependencies! Use phin instead!

This is great and good to be merged imo

Looks like a good implementation to me also. @oliver-moran?


fix: delete before apply use namespaced API

What this PR is for?
This PR fixes the behavior  of the delete-before-apply annotation by using the namespaced API endpoint instead of the cluster one, thus requiring lower privilege to execute the action.

Can you also update the CHANGELOG please?

Can you also update the CHANGELOG please?

Done


Allow OTP sending via email

one user is requesting X digit pin numbers to be sent via email instead of SMS
use-case is desktop app (electron), so that users don't get sent to chrome before redirecting back to the app
would also be useful for mobile users

Additional use case to support this feature:
Some of our users browse our site and browse emails with different devices. The current magic link implementation doesn't authenticate the user on the device which the sign in email was requested from.

Hey!
@liaujianjie, @awalias has this issue been resolved?

Hey
@liaujianjie, @awalias was just browsing through the open issues when I saw this. Has this issue been resolved or is it being actively worked on?

I'm not working on it. Maybe @awalias or @kangmingtay would know the status of this issue.

I haven't started working on this yet, but i can add this next since there seems to be quite abit of demand for it!

Off the top of my head, there are a few possible ways we can implement this:

Add a sendOtp param to the signup / magiclink endpoints. If it's set to true, we send an otp instead of a confirmation link.

# POST /signup
{
  "email"<EMAIL_ADDRESS>  "password": "test",
  "sendOtp": true
}

Make the change within the internal gotrue config such that it can be set via the supabase dashboard (or the gotrue env vars if self-hosted).

Other considerations:
(1) would be more flexible than (2) since it provides more flexibility for the developer to choose when to send an otp over a confirmation link (although i'm not too sure what are the potential use-cases here). However, if there are additional custom verification steps after entering the otp, then this interface might expose a potential vulnerability since the sendOtp flag can easily be changed on the client-side.
An attacker can just hit the api directly and set sendOtp to false to trigger a link being sent instead.
Feel free to let me know what you guys think! Or if there's a better way to do it.
PS: I'm open to renaming the param too, right now, sendOtp is the best i can think of :)

Hey Kang Ming,
Thanks for the detailed response! Was actually thinking of working on it as I'd like to familiarize myself with the GoTrue codebase but seems like you're on top of things. Will look for other suitable issues.
If you'd be okay with it, do you mind tagging me on the PR though? Would love to take a look.

Is it possible to have both the confirmation link and the OTP in the same email? I imagine the sudden switch from confirmation links to only OTP would be quite jarring for our end-users. It'll be nice if as a user, I am a able to choose if I want to sign in via the confirmation link or to enter the OTP.
Suggested new default template for such a scenario:

Magic Link

Click this link to login:

Log In

Or copy enter this OTP to login: {{ .Code }}

Is it possible to have both the confirmation link and the OTP in the same email? I imagine the sudden switch from confirmation links to only OTP would be quite jarring for our end-users. It'll be nice if as a user, I am a able to choose if I want to sign in via the confirmation link or to enter the OTP.
Suggested new default template for such a scenario:

Magic Link

Click this link to login:

Log In

Or copy enter this OTP to login: {{ .Code }}

Somewhat tangential question:
Is it possible to also use n-digit pin numbers as confirmation tokens? That way we OTPs are confirmation tokens. Then, OTPs and magic links can share an internal implementation. However, I'm not sure how much of security risk that reducing the entropy of the refresh token is considering SMS OTPs are already using it.

Somewhat tangential question:
Is it possible to also use n-digit pin numbers as confirmation tokens? That way we OTPs are confirmation tokens. Then, OTPs and magic links can share an internal implementation. However, I'm not sure how much of security risk that reducing the entropy of the refresh token is considering SMS OTPs are already using it.

Is it possible to also use n-digit pin numbers as confirmation tokens?

Hmm i think there might be a security concern here. With magic links, gotrue can only identify a user by the confirmation token from the query param. So it has to have a high enough entropy such that a collision is near impossible.
With OTPs, gotrue requires either an email or phone number to be sent as well in the POST request body so it can identify the user the otp was intended for. The email or phone number acts as a namespace in some sense.
We can't do the same for magiclinks because they have to be GET requests so we can't send the email / phone number in the query param.

Is it possible to also use n-digit pin numbers as confirmation tokens?

Hmm i think there might be a security concern here. With magic links, gotrue can only identify a user by the confirmation token from the query param. So it has to have a high enough entropy such that a collision is near impossible.
With OTPs, gotrue requires either an email or phone number to be sent as well in the POST request body so it can identify the user the otp was intended for. The email or phone number acts as a namespace in some sense.
We can't do the same for magiclinks because they have to be GET requests so we can't send the email / phone number in the query param.

If the above is possible, we could even expose APIs to allow devs to generate OTPs/confirmation tokens as an admin API.

Good idea! I think this is still possible for otps, we currently have a /admin/generateLink endpoint to generate magic / invite / recovery links.

If the above is possible, we could even expose APIs to allow devs to generate OTPs/confirmation tokens as an admin API.

Good idea! I think this is still possible for otps, we currently have a /admin/generateLink endpoint to generate magic / invite / recovery links.

With OTPs, gotrue requires either an email or phone number to be sent as well in the POST request body so it can identify the user the otp was intended for. The email or phone number acts as a namespace in some sense.

Right, thanks for the explanation! Does this also mean that it would still be possible if the confirmation URL also includes the email or phone number to identify the user? "Possible" here means (1) the likelihood of clash is near-zero, and (2) the reduced entropy would not be a security concern.

With OTPs, gotrue requires either an email or phone number to be sent as well in the POST request body so it can identify the user the otp was intended for. The email or phone number acts as a namespace in some sense.

Right, thanks for the explanation! Does this also mean that it would still be possible if the confirmation URL also includes the email or phone number to identify the user? "Possible" here means (1) the likelihood of clash is near-zero, and (2) the reduced entropy would not be a security concern.

Yup it's still possible if we pass the email / phone in the confirmation url because those fields are unique in the db.
However, based on OWASP, it's recommended not to pass sensitive information in the query param.

Yup it's still possible if we pass the email / phone in the confirmation url because those fields are unique in the db.
However, based on OWASP, it's recommended not to pass sensitive information in the query param.

closing as duplicate of https://github.com/supabase/gotrue/issues/264


Error enabling sub-modules

I have begun to customize my drupal.yml make file to add in the standard modules I like to have on every default site. I've even added a drupal_disable_modules task to get rid of the few modules I don't like (we're looking at you overlay! :) ).
I'm wondering if I have the syntax wrong perhaps? I've tried the module list with CSV, and space separated, both work fine.  The problem I'm having is that this set of modules works fine, and enables properly on install:
drupal_enable_modules: [ 'devel admin_menu module_filter' ]
but this set of modules, specifically with admin_menu_toolbar, fails
drupal_enable_modules: [ 'devel admin_menu module_filter admin_menu_toolbar' ]
with the following errors. The only thing I can see that is different is that admin_menu_toolbar is a sub-module of admin_menu.
==> drupaldev: TASK: [Generate Drupal site with drush makefile.] *****************************
==> drupaldev: changed: [localhost]
==> drupaldev:
==> drupaldev: TASK: [Install Drupal (standard profile) with drush.] *************************
==> drupaldev: changed: [localhost]
==> drupaldev:
==> drupaldev: TASK: [Install configured modules with drush.] ********************************
==> drupaldev: failed: [localhost] => {"changed": true, "cmd": ["drush", "en", "-y", "devel", "admin_menu", "module_filter", "admin_menu_toolbar"], "delta": "0:00:05.507379", "end": "2015-04-08 05:56:29.088433", "rc": 1, "start": "2015-04-08 05:56:23.581054", "warnings": []}
==> drupaldev: stderr: admin_menu was not found.                                              [warning]
==> drupaldev: module_filter was not found.                                           [warning]
==> drupaldev: admin_menu_toolbar was not found.                                      [warning]
==> drupaldev: No release history was found for the requested project                   [error]
==> drupaldev: (admin_menu_toolbar).
==> drupaldev: The following projects provide some or all of the extensions not            [ok]
==> drupaldev: found:
==> drupaldev: admin_menu
==> drupaldev: module_filter
==> drupaldev: stdout: Would you like to download them? (y/n): y
==> drupaldev: Project admin_menu (7.x-3.0-rc5) downloaded to                         [success]
==> drupaldev: /var/www/drupal/sites/all/modules/contrib/admin_menu.
==> drupaldev: Project admin_menu contains 3 modules: admin_menu_toolbar, admin_devel, admin_menu.
==> drupaldev: Project module_filter (7.x-2.0) downloaded to                          [success]
==> drupaldev: /var/www/drupal/sites/all/modules/contrib/module_filter.
==> drupaldev:
==> drupaldev: FATAL: all hosts have already failed -- aborting
==> drupaldev:
==> drupaldev: PLAY RECAP ********************************************************************
==> drupaldev:            to retry, use: --limit @/root/playbook.retry
==> drupaldev:
==> drupaldev: localhost                  : ok=134  changed=7    unreachable=0    failed=1
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

@tgroff - The best way to get around that (since drush will spit out an error whenever you try enabling a submodule that's not yet present in the codebase—drush en isn't very intelligent about downloading extra dependencies) is to make sure you have all the modules you will need in your make file.
For example, for your site, you will probably want to set your drupal.make.yml file like so:
---
api: 2
core: 7.x

projects:

  drupal:
    type: core
    version: 7.x

  devel: 1.x-dev
  admin_menu: 3.x-dev
  module_filter: 2.x-dev

Then, in config.yml, for the drupal_enable_modules variable:
drupal_enable_modules: [ 'devel', 'admin_menu', 'module_filter', 'admin_menu_toolbar' ]

Thanks! That did the trick! This is a D7 site I'm messing with. I am just now learning about drupal.make.yml as a make file, very cool that we can use yml with D7.  My drupal.make.yml was as such:
---
api: 2

# Basic Drush Make file for Drupal. Be sure to update the drupal_major_version
# variable inside config.yml if you change the major version in this file.

# Drupal core (major version, e.g. 6.x, 7.x, 8.x).
core: "7.x"

defaults:
  projects:
    subdir: "contrib"

projects:

  drupal:
    type: "core"
    version: ~

  devel: "1.x-dev"
  module_filter: ~
  admin_menu: ~

... so it seems the ~ is what caused the problem. I thought ~ could be used to get the latest stable release... but as I see it now admin_menu has never had a stable D7 release!? It's still at 7.x-3.0-rc5!. When I used version numbers as you suggested everything worked fine. Thanks!

Yeah, admin_menu should add a stable release sometime :P


Error decoding strings with non-ascii chars using Data.Text.Encoding.decodeUtf8With

Two cases:

print $ decodeUtf8With strictDecode "ñ"

***Exception***: Cannot decode byte '\xf1': Data.Text.Internal.Encoding.decodeUtf8: Invalid UTF-8 stream
trace.txt

print $ decodeUtf8With lenientDecode "ñ"

prints "" instead "ñ"

print $ decodeUtf8With lenientDecode "añ"

Exception in thread "main" java.nio.BufferOverflowException
trace2.txt

i've rerun the tests, comparing etlas/eta with stack/ghc (resolver lts-6.27)

Code:

  putStrLn "\xf1"
  putStrLn "ñ"
  putStrLn "\xc3\xb1"
  print $ decodeUtf8With lenientDecode "ñ"
  print $ decodeUtf8With lenientDecode "añ"
  print $ decodeUtf8With strictDecode "ñ"

With stack/ghc:

ñ
ñ
Ã±
"\65533"
"a\65533"
eta-test.EXE: Cannot decode byte '\xf1': Data.Text.Internal.Encoding.decodeUtf8:
 Invalid UTF-8 stream

With etlas/eta

±
±
├▒
"\65533"
"a\65533"
Exception in thread "main" eta.runtime.exception.EtaException: Cannot decode byt
e '\xf1': Data.Text.Internal.Encoding.decodeUtf8: Invalid UTF-8 stream
        at base.ghc.Exception$$Lr8O7Ylvl.enter(Exception.hs)

So the behaviour of decodeUtf8With (the cause of the issue) is the same, but the handling of unicode chars is not.
Maybe we should close this issue and open another about unicode handling in eta.

Examining more carefully the output it turns out eta is handling correctly utf-8 strings but in windows console you need to do some steps:

change the console font to  Lucida
change the code page to 65001: chcp 65001
execute java with -Dfile.encoding=UTF-8

With this steps the non-ascii chars are correctly displayed.
To avoid this we would have to read the current default encoding (with System.getProperty("file.encoding") and replace eta.runtime.io.MemoryManager.loadStringUTF8(String s) calls with eta.runtime.io.MemoryManager.loadString(String encoding, String s)

Thanks for looking into this. This looks like a recurring problem - what should be the final solutions for this? I'm not sure setting the encoding for loadString is the right solution since that has to do with the encoding at compile-time (the source files), while this problem appears to be the runtime. Is it safe to always send -Dfile.encoding-UTF-8 with setting the codepage?

Just saw you filed #515. Closing this one.


Drawers Cut Off their Content

Describe the bug
The size of a drawer always follows its anchor. This is usually fine, but sometimes the content of the drawer is actually the larger of the two. In that case the content is unceremoniously truncated.
Steps to Reproduce

Create a rio.Drawer

Use rio.Text (or anything small) as anchor
Fill the content with any components until they exceed the screen size.

Screenshots/Videos
No response
Operating System
No response
What browsers are you seeing the problem on?
No response
Browser version
No response
What device are you using?
No response
Additional context
No response

Drawers are now always large enough to contain both their anchor and their content.


[cPP CHANGE] PP module -> PP-Module

What is the change request for the cPP? Please describe.
The current "standard" for a module is to write it as "PP-Module" according to the CC R5
Describe the solution you'd like
All instances of "PP module" or similar should be changed to "PP-Module"
Describe alternatives you've considered
N/A
Additional context
This is how the modules are defines/written in CC R5 CEM

Change implemented.


Fix wrong example for message generate

Codecov Report
Base: 77.21% // Head: 77.21% // No change to project coverage :thumbsup:

Coverage data is based on head (d7d49c9) compared to base (74609dc).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #154   +/-   ##
=========================================
  Coverage     77.21%   77.21%
  Complexity      786      786
=========================================
  Files           108      108
  Lines          2199     2199
  Branches        249      249
=========================================
  Hits           1698     1698
  Misses          410      410
  Partials         91       91

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

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


pull request for e2e test at 01/08/2017 22:26:10

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


ADD to .gitignore (.DS_Store)

Add to .gitignore file (MacOS file):
.DS_Store

https://github.com/masa-finance/masa-node-v1.0/pulls#:~:text=ago by myuriy-,Remove Mac OSX DS_Store directory index from repo,-%234 opened 1

#4

good job!

Fixed


Deleting Cesium Appvar and Exiting the Settings Menu Causes Reset

Deleting Cesium's Appvar and exiting the settings menu causes a reset. (See title)

(Apparently CEmu didn't want to capture the reset…)
Possible fixes:

Preventing the user from deleting the Cesium Appvar
Recreating the Appvar every time it exits the Settings menu, not just edit the Appvar.

Thanks.

It now checks if the appvar exists and creates it if it doesn't.


fix: docker/no-driver config and aliasing

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area build

What this PR does / why we need it:
This PR fixes again the docker/no-driver image and its tagging that's currently broken on master - sorry, my fault! :angel:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
/milestone 0.23.0
/cc @fntlnz
/cc @leodido
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign kris-nova
You can assign the PR to them by writing /assign @kris-nova in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: dd467ae5052469be610d9a3f7d191509f713d1c7

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz,leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Implicit Project discovery discovers cabal-project although there is a stack.yaml

Reproduce, have cabal and stack installed:
git clone https://github.com/haskell/haskell-ide-engine
rm cabal.project # currently needed to force stack usage
code . # lsp client, and open Cradle.hs

We can see in the logs that hie-plugin-api is selected as the root of the project, but it should be haskell-ide-engine.

@marinelli This problem could explain the behaviour you observed yesterday!


Feature Request: Be able to stop CONCISE from adding a new dimension to coordinate variables

Getting a new error in the regression test notebook for the SAMBAH service chain.... An error when trying to open an output dataset using xarray — I think this is likely because of how CONCISE is producing the result (i.e., stacking layers):
MissingDimensionsError: 'xtrack' has more than 1-dimension and the same name as one of its dimensions ('subset_index', 'xtrack'). xarray disallows such variables because they conflict with the coordinates used to label dimensions.
Perhaps we can add a way to tell CONCISE that a variable or variables—in this case, xtrack— is a coordinate variable that should not have a new subset_index dimension added to it.  In other words, keep the 1D coordinate variable as a 1D variable in CONCISE's output.

to fix your trouble try download this fix, i see it in another issue,
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

The way CONCISE works is that it performs merging amongst multiple granules by introducing the subset_index dimension to all variables to allow access to the individual merged granules' values. That way the original granule data is fully preserved. Making an exception for 1D variables would break that design
It is known that xarray doesn't always play nice with all CONCISE-compatible, but I always viewed that more as a limitation of xarray than CONCISE itself. For example, we had full support for NetCDF groups before xarray had their support for them
Is there a way to work around xarray on SAMBAH? I'd prefer to see a downchain solution over needing to change how CONCISE works since its been around for a fairly long time for users

@danielfromearth Oh wait, is this an issue with the SAMBAH chain itself or just an issue with a regression test because xarray can't open the dataset?

We're currently determining whether to consider these outputs an unacceptable output from the chain, or whether to consider it an output to work around.
The original data structure depends on two coordinate variables (as defined here) called xtrack and mirror_step. For instance, a derived geophysical variable like "vertical_column" has dimensions of ('mirror_step', 'xtrack'), "latitude" has dimensions of ('mirror_step', 'xtrack'), and "xtrack" has dimensions of ('xtrack').
Therefore, when the data is processed by CONCISE, it ends up creating a netCDF with vertical_column('subset_index', 'mirror_step', 'xtrack'), latitude('subset_index', 'mirror_step', 'xtrack'), and xtrack('subset_index', 'xtrack').  It seems okay from one perspective, but it also means that xtrack no longer satisfies the CF/COARDS definition of a coordinate variable.

The only way I can see CONCISE passing through the 1D variable untouched is if CONCISE can verify that every granule its merging contains the same data in that variable otherwise there would be a loss of data during merging. But then I would question whether or not the existing user base is okay with changing CONCISE's behavior at this point while it's in maintenance mode
My opinion is leaning more towards that CONCISE should remain fairly consistent in its behavior so far and its more of a downstream tooling issue to support its output

@joshgarde, thanks for the comments on this!
After digging into this further with @ank1m, it looks like there might already be a resolution to this difficulty in xarray (see this issue https://github.com/pydata/xarray/issues/2233 and pull request https://github.com/pydata/xarray/pull/7989), when using a version newer than currently in the harmony regression test notebook.  Seems promising, and will update in this thread more as we determine if the issue will be solved with dependency updates.


add return context, fix tests for additional out parameter

for #56
Signed-off-by: Artem Nistratov<EMAIL_ADDRESS>

Thanks for the contribution! What's the rational behind returning implicit values? I can see that it gives flexibility but it's nothing short of returning interface{} values. What did you try to solve when applying this?

Maybe I misunderstood you, but return type is clear. It's context.Context and it was created to resolve such issues like share some variables across all execution pipleline. I return a user model associated with added or extracted token, token usage information and some other options. With context I link refresh and access token in PersistAuthorizeCodeGrantSession, because token saving logic already implemented in CreateAccessTokenSession and CreateRefreshTokenSession. I don't want copy&paste it in PersistAuthorizeCodeGrantSession. Instead I put RefreshToken & AccessToken models into new context in its creation functions. I think there are many other suitable variants for return context usage.

Ah sorry, my bad, I misread :)

Sharing data is intended to be happening through Requester.GetSession() interface{} / Requester.SetSession(session interface{}). That's where you would store user ids and that stuff. Or did I miss something?

Could you maybe show me some usage examples, that would make it easier to understand for me :)

Do you maybe know an example project where contexts are returned as well? I have looked at some go projects but never seen contexts get returned. Docker for example only returns context.Context within NewSomethingContext().
Only because nobody uses it, it doesn't imply that it's not a good idea. However, I need to nullify any doubt of unwanted sideffects or other issues :)

I use the JWT Session as session interface. Hm, I didn't look at this from this point. May be more preferable to extend JWT session.
In examples I seen only nil in session. It is misleading (

Sorry for that. :( Documentation is on my list, but I have a lot to do right now.
Would you maybe be interested in helping here a little bit? Seems like you've already a good picture of how things work!

I will try)
Sorry for wasting your time, I close PR.

No worries, I'm sorry that the docs didn't mention that :)


[HTTPS] What about HPKP?

I saw you you support DANE which is crazily fantastic, but nevertheless it would also be nice if you could enable HTTP Public Key Pinning. A good guide to do this you can for example find here: https://noncombatant.org/2015/05/01/about-http-public-key-pinning/
(it's just that not many people/browser are using DANE...)

Will do that this afternoon! Thanks for bringing it to my attention!

Just a small reminder.. :smile: (HPKP is still not supported)
And BTW here German instructions for HPKP: https://blog.pregos.info/2015/02/23/http-public-key-pinning-hpkp-erklaerung-und-einrichtung/
Please also notice issue #2...

Added :+1:

or not,  I think I need to create a backup pin.

Created a backup pin and now firefox does not complains any more about about an illegal pinning entry.


problem parsing us zips with embedded json column

hey @samthor!
i wanted to take this thing for a spin, but ran into the same issue as i did with https://github.com/samthor/but-csv.
it looks like this free zipcodes file cannot be properly parsed by either lib.
sample:
"zip","lat","lng","city","state_id","state_name","zcta","parent_zcta","population","density","county_fips","county_name","county_weights","county_names_all","county_fips_all","imprecise","military","timezone"
"00601","18.18027","-66.75266","Adjuntas","PR","Puerto Rico","TRUE","","16834","100.9","72001","Adjuntas","{""72001"": 98.74, ""72141"": 1.26}","Adjuntas|Utuado","72001|72141","FALSE","FALSE","America/Puerto_Rico"
"00602","18.36075","-67.17541","Aguada","PR","Puerto Rico","TRUE","","37642","479.2","72003","Aguada","{""72003"": 100}","Aguada","72003","FALSE","FALSE","America/Puerto_Rico"
"00603","18.45744","-67.12225","Aguadilla","PR","Puerto Rico","TRUE","","49075","551.7","72005","Aguadilla","{""72005"": 99.76, ""72099"": 0.24}","Aguadilla|Moca","72005|72099","FALSE","FALSE","America/Puerto_Rico"
"00606","18.16585","-66.93716","Maricao","PR","Puerto Rico","TRUE","","5590","48.7","72093","Maricao","{""72093"": 82.27, ""72153"": 11.66, ""72121"": 6.06}","Maricao|Yauco|Sabana Grande","72093|72153|72121","FALSE","FALSE","America/Puerto_Rico"
"00610","18.2911","-67.12243","Anasco","PR","Puerto Rico","TRUE","","25542","265.7","72011","Añasco","{""72011"": 96.71, ""72099"": 2.82, ""72083"": 0.37, ""72003"": 0.1}","Añasco|Moca|Las Marías|Aguada","72011|72099|72083|72003","FALSE","FALSE","America/Puerto_Rico"
"00611","18.27698","-66.80688","Angeles","PR","Puerto Rico","TRUE","","1315","47.7","72141","Utuado","{""72141"": 100}","Utuado","72141","FALSE","FALSE","America/Puerto_Rico"
"00612","18.41283","-66.7051","Arecibo","PR","Puerto Rico","TRUE","","63312","321.1","72013","Arecibo","{""72013"": 98.94, ""72065"": 0.94, ""72017"": 0.11}","Arecibo|Hatillo|Barceloneta","72013|72065|72017","FALSE","FALSE","America/Puerto_Rico"
"00616","18.41878","-66.6679","Bajadero","PR","Puerto Rico","TRUE","","9625","341.4","72013","Arecibo","{""72013"": 100}","Arecibo","72013","FALSE","FALSE","America/Puerto_Rico"
"00617","18.44598","-66.56006","Barceloneta","PR","Puerto Rico","TRUE","","22573","474.8","72017","Barceloneta","{""72017"": 99.63, ""72054"": 0.37}","Barceloneta|Florida","72017|72054","FALSE","FALSE","America/Puerto_Rico"
"00622","17.98892","-67.1566","Boqueron","PR","Puerto Rico","TRUE","","7577","93.9","72023","Cabo Rojo","{""72023"": 100}","Cabo Rojo","72023","FALSE","FALSE","America/Puerto_Rico"
"00623","18.08429","-67.15336","Cabo Rojo","PR","Puerto Rico","TRUE","","39406","388.1","72023","Cabo Rojo","{""72023"": 100}","Cabo Rojo","72023","FALSE","FALSE","America/Puerto_Rico"
"00624","18.05905","-66.71932","Penuelas","PR","Puerto Rico","TRUE","","21648","178.0","72111","Peñuelas","{""72111"": 94.94, ""72113"": 5.06}","Peñuelas|Ponce","72111|72113","FALSE","FALSE","America/Puerto_Rico"
"00627","18.41905","-66.86037","Camuy","PR","Puerto Rico","TRUE","","32733","272.6","72027","Camuy","{""72027"": 100}","Camuy","72027","FALSE","FALSE","America/Puerto_Rico"
"00631","18.1852","-66.83169","Castaner","PR","Puerto Rico","TRUE","","1431","176.8","72081","Lares","{""72081"": 50.58, ""72001"": 49.42}","Lares|Adjuntas","72081|72001","FALSE","FALSE","America/Puerto_Rico"
"00637","18.081","-66.94659","Sabana Grande","PR","Puerto Rico","TRUE","","22882","251.2","72121","Sabana Grande","{""72121"": 93.98, ""72125"": 4.22, ""72079"": 1.8}","Sabana Grande|San Germán|Lajas","72121|72125|72079","FALSE","FALSE","America/Puerto_Rico"
"00638","18.28462","-66.5137","Ciales","PR","Puerto Rico","TRUE","","17605","95.2","72039","Ciales","{""72039"": 91.88, ""72107"": 7.48, ""72091"": 0.64}","Ciales|Orocovis|Manatí","72039|72107|72091","FALSE","FALSE","America/Puerto_Rico"
"00641","18.26742","-66.70212","Utuado","PR","Puerto Rico","TRUE","","24299","111.5","72141","Utuado","{""72141"": 100}","Utuado","72141","FALSE","FALSE","America/Puerto_Rico"

Thanks for your help! It looks like this is failing to close a quoted string because I look for "\n" or "," after the quotes conclude, and this file instead has "\r\n".
Will fix in both, thanks.

I've made it support CRLF now. I should just use uDSV ;-)

I should just use uDSV ;-)

haha, wait for the benchmarks first, in Bun.js this time! :P

pretty good :)
poor Papa parse chokes on quotes :(
Bun.js:

what's really weird is that tiddlycsv doesn't parse correctly using identical code in Node:

the code i use is really simple:
module.exports = {
    name: 'tiddlycsv',
    repo: 'https://github.com/samthor/tiddlycsv',
    load: async () => {
      const { parseCSV } = await import('tiddlycsv');

      return (csvStr, path) => new Promise(res => {
        let rows = parseCSV(csvStr);
        res(rows);
      });
    },
  };


babashka: make rlwrap optional in wrapper

Fixes #246839
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

IMO, when the withRLwrap option enabled, the binary should call something else to differentiate it from original bb. And we should always provide un-rlwraped bb as default same as upstream.

In my opinion no, it would be surprising to rename the binary and also this is an explicit option from the user, so if the user is enabling it they should know the consequences.
I do think we may need a comment though explaining what this is used for and the known issues.

I don't think a warning or comment is enough this case. This is essentially introduced compatibilities issues and potential bugs not recommend by upstream. This is the same effect as packaging clj/clojure as the same clojure binary with an optional flag, which we normally wouldn't do. Also to quote some discussion regarding this on the clojurian slack #nix channel the other day.

Again, this is already not set to default and unless someone look at the code and know what it is looking for they will not be using the rlwrap. And renaming the binary is surprising, it is not like upstream has a recommendation of how to name this new binary either.

I don't mean as renaming the rlwrap binary when the option enabled, I mean add an additional renamed binary for the rlwarped one when the option enabled.
Without enabling option -> bin/bb(no rlwrap) With this option enabled -> bin/bb(no rlwrap), bin/bb-rlwrap (or whatever name it may be)

I understand this, but this is still surprising. I don't want to have 2 separate binaries depending on what I am doing, and I am using alias bb=rlwrap bb for a few years already before this wrapper and never had issues in my use cases.

Without enabling option -> bin/bb(no rlwrap)
With this option enabled -> bin/bb(no rlwrap), bin/bb-rlwrap (or whatever name it may be)

If we are going this route though, there is no reason to not enabling this to default.

rlwrap bb and bb are two different things, the example I gave in original #246839 already demonstrate the potential issue it may cause. Just because your alias bb=rlwrap bb use case hasn't cause issue is just pure luck since your aliased bb broke the contract defined by upstream and what everybody else uses as bb. All the bb scripts ships by others assume bb=bb, not some alias.

Or maybe I know the difference between a script and interactive usage, and use the proper babashka when writing scripts. You can still do so in how the repository is organized right now, because we still have babashka-unwrapped and using this with nix is really easy:
#!/usr/bin/env nix-shell
#!nix-shell -p babashka-unwrapped -i bb

Thats fine, I don't mind enabling that to default.

I will leave the decision to @jlesquembre on how he prefer to handle the situation, I am fine either way:

Leaving on as default and having 2 binaries
Leaving off as default and having 1 binary

You know the difference but how would you use both in the same shell? You still need one defined as alias under different name.

I could just use nix-shell -p babashka-unwrapped to have bb aliased to non wrapped version for that session. This is how nix works in general, not just for this particular change.

Also as a script author, when you distribute scripts like bb tasks | bb -io '(->> input (drop 2) (map #(-> % (str/split #" ") first)))' to people, you want to explain people to use bb without rlwrap every-time?

I don't think most people would classify this as a script, more like interactive usage instead. Script are generally put in a file and I can use the same trick I showed before to have the version of babashka I want inside that context.


Approve/Swap transactions loading

Describe the bug
Description of the bug here.The approve transaction tap on taiko l3test site takes too long to be approved and after approval the Swap tap takes too long to pop out
Steps to reproduce
Steps to reproduce here.
Additional context
Additional context here.

Lets wait to see how the Uniswap v3 UI perform on Taiko's next testnet.


Add support for fetching k8s Ingress TLS data from secrets

What does this PR do?
Enable Traefik to read the k8s secret defined in a k8s ingress resource via the tls property.
Motivation
This allows a k8s user to easily setup a tls terminated endpoint
More

[x] Added/updated tests
[x] Added/updated documentation

Additional Notes
This is my first try on this. I would be pleased to get your opinion on this. This is the implementation to #2438

@gopenguin Could you fix?
---> Making bundle: validate-autogen (in .)
checking autogen is up-to-date with templates...
The result of 'go generate' differs

 M autogen/gentemplates/gen.go

Please do "go generate" to update the `autogen` package.

When I regenerate the file all file modes change from os.FileMode(436) to os.FileMode(420). Is this supposed to happen?

It's not the expected behavior.
Don't change nothing for now, I fix this problem in #2440.

@ldez I've just pulled your changes and I think there is sill an issue with the generated file.  It isn't go fmt'ed but the version in the repo is.

If you just run go generate, the file changes but after running go fmt the changes are gone.

I had some time this morning and wrote some docs but I realy appreciate any help.

Thanks for the review. I'll probably get it fixed by tomorrow evening.

I think, that I've missinterpreted the traefik api and there is currently no way for a provider to pass specific information about certificates like to which domain a wildcard cert is assigned to the core of traefik.

@nmengin: Thanks for your review. I think your suggestion sounds reasonable.
I haven't found any way to access all the defined EntryPoints from inside a provider. At least I belive, that the GlobalConfiguration wourd be the right struct to use but a can't get any references.
Any suggestions?

@gopenguin
I just created a PR which allows managing the by default behavior (#2534).
Thanks to it you have not to look for the default TLS entryPoints.
If the service does not define the traefik.frontend.entryPoints, you can set nil.
I hope that can help you.

Thanks. Thats now much simpler

I found out, that I removed the section in the template passing the certs. I'll add it back again and add a test to cover this.

@nmengin have you been able to complete your tests? I'd like to make sure the entry point part is approved before I take another look at the PR.

@timoreimann No, I did not.
I've got few problems with my minikube instance :cry:
I'll try to test it next week.


Link.target() sets errno to ENOENT even if there's a file or directory at the specified path

e.g. https://github.com/dart-lang/sdk/blob/master/runtime/bin/file_macos.cc#L389
To be more consistent with the behavior of Link.rename() & Link.delete(), it seems like Link.target() should set errno to EISDIR & EINVAL if there's an existing directory or file at the path identified in the Link object.

@zanderso


Standardize dartdoc macro names

Description
This standardizes the dartdoc template macro names to roughly follow the convention:
flutter.library.Class.member[.optionalDescription]
Where there was no need for a description, I didn't add one.
Some (about 1/4) of the macros were already in this form, but standardizing the rest (which were in various forms) resulted in the following transforms:

flutter.animation.curves.catmull_rom_description => flutter.animation.CatmullRomSpline
flutter.cupertino.navBar.actionsForegroundColor => flutter.cupertino.CupertinoNavigationBar.actionsForegroundColor
flutter.cupertino.navBar.automaticallyImplyLeading => flutter.cupertino.CupertinoNavigationBar.automaticallyImplyLeading
flutter.cupertino.navBar.backgroundColor => flutter.cupertino.CupertinoNavigationBar.backgroundColor
flutter.cupertino.navBar.border => flutter.cupertino.CupertinoNavigationBar.border
flutter.cupertino.navBar.brightness => flutter.cupertino.CupertinoNavigationBar.brightness
flutter.cupertino.navBar.heroTag => flutter.cupertino.CupertinoNavigationBar.heroTag
flutter.cupertino.navBar.leading => flutter.cupertino.CupertinoNavigationBar.leading
flutter.cupertino.navBar.padding => flutter.cupertino.CupertinoNavigationBar.padding
flutter.cupertino.navBar.previousPageTitle => flutter.cupertino.CupertinoNavigationBar.previousPageTitle
flutter.cupertino.navBar.trailing => flutter.cupertino.CupertinoNavigationBar.trailing
flutter.cupertino.navBar.transitionBetweenRoutes => flutter.cupertino.CupertinoNavigationBar.transitionBetweenRoutes
flutter.cupertino.cupertinoRouteTransitionMixin.title => flutter.cupertino.CupertinoRouteTransitionMixin.title
flutter.cupertino.switch.dragStartBehavior => flutter.cupertino.CupertinoSwitch.dragStartBehavior
flutter.frame_rasterized_vs_presented => flutter.flutter_driver.WaitUntilFirstFrameRasterized
flutter.flutter_test.frame_policy => flutter.flutter_test.LiveTestWidgetsFlutterBindingFramePolicy
flutter.flutter_test.moreOrLessEquals.epsilon => flutter.flutter_test.moreOrLessEquals
flutter.flutter_test.drag => flutter.flutter_test.WidgetController.drag
flutter.flutter_test.fling => flutter.flutter_test.WidgetController.fling
flutter.flutter_test.timeddrag => flutter.flutter_test.WidgetController.timedDrag
flutter.foundation.bindingBase.registerServiceExtension => flutter.foundation.BindingBase.registerServiceExtension
flutter.gestures.gestureRecognizer.kind => flutter.gestures.GestureRecognizer.kind
flutter.material.dataTable.dataRowColorCode => flutter.material.DataTable.dataRowColor
flutter.material.dataTable.headingRowColorCode => flutter.material.DataTable.headingRowColor
flutter.material.drawer.dragStartBehavior => flutter.material.DrawerController.dragStartBehavior
flutter.widgets.inputDecorator.textAlignVertical => flutter.material.InputDecorator.textAlignVertical
flutter.material.ListTile.shape => flutter.material.ListTileTheme.shape
flutter.widgets.Clip => flutter.material.Material.clipBehavior
flutter.material.progressIndicator.parameters => flutter.material.ProgressIndicator.ProgressIndicator
flutter.material.rangeSlider.shape.argument.isDiscrete => flutter.material.RangeSliderThumbShape.getPreferredSize.isDiscrete
flutter.material.rangeSlider.shape.argument.isEnabled => flutter.material.RangeSliderThumbShape.getPreferredSize.isEnabled
flutter.material.rangeSlider.shape.argument.activationAnimation => flutter.material.RangeSliderThumbShape.paint.activationAnimation
flutter.material.rangeSlider.shape.argument.context => flutter.material.RangeSliderThumbShape.paint.context
flutter.material.rangeSlider.shape.argument.enableAnimation => flutter.material.RangeSliderThumbShape.paint.enableAnimation
flutter.material.rangeSlider.shape.argument.sliderTheme => flutter.material.RangeSliderThumbShape.paint.sliderTheme
flutter.material.rangeSlider.shape.argument.thumb => flutter.material.RangeSliderThumbShape.paint.thumb
flutter.material.rangeSlider.trackSegment => flutter.material.RangeSliderTickMarkShape.paint.trackSegment
flutter.material.rangeSlider.shape.argument.parentBox => flutter.material.RangeSliderValueIndicatorShape.paint.parentBox
flutter.material.button.mouseCursor => flutter.material.RawMaterialButton.mouseCursor
flutter.material.slider.shape.argument.center => flutter.material.SliderComponentShape.paint.center
flutter.material.slider.shape.argument.context => flutter.material.SliderComponentShape.paint.context
flutter.material.slider.shape.argument.enableAnimation => flutter.material.SliderComponentShape.paint.enableAnimation
flutter.material.slider.shape.argument.isDiscrete => flutter.material.SliderComponentShape.paint.isDiscrete
flutter.material.slider.shape.argument.parentBox => flutter.material.SliderComponentShape.paint.parentBox
flutter.material.slider.shape.argument.sizeWithOverflow => flutter.material.SliderComponentShape.paint.sizeWithOverflow
flutter.material.slider.shape.argument.sliderTheme => flutter.material.SliderComponentShape.paint.sliderTheme
flutter.material.slider.shape.argument.textScaleFactor => flutter.material.SliderComponentShape.paint.textScaleFactor
flutter.material.slider.seeAlso.rangeSliderThumbShape => flutter.material.SliderTheme.rangeSliderThumbShape
flutter.material.slider.seeAlso.rangeSliderTickMarkShape => flutter.material.SliderTheme.rangeSliderTickMarkShape
flutter.material.slider.seeAlso.rangeSliderTrackShape => flutter.material.SliderTheme.rangeSliderTrackShape
flutter.material.slider.seeAlso.rangeSliderValueIndicatorShape => flutter.material.SliderTheme.rangeSliderValueIndicatorShape
flutter.material.slider.seeAlso.sliderComponentShape => flutter.material.SliderTheme.sliderComponentShape
flutter.material.slider.seeAlso.sliderTickMarkShape => flutter.material.SliderTheme.sliderTickMarkShape
flutter.material.slider.seeAlso.sliderTrackShape => flutter.material.SliderTheme.sliderTrackShape
flutter.material.slider.shape.argument.isEnabled => flutter.material.SliderTickMarkShape.getPreferredSize.isEnabled
flutter.material.slider.trackSegment => flutter.material.SliderTrackShape.paint.trackSegment
flutter.painting.boxDecoration.clip => flutter.painting.BoxDecoration.clip
clip_with_box_fit => flutter.painting.BoxFit.cover
flutter.material.progressIndicator.semanticsLabel => flutter.progress_indicator.ProgressIndicator.semanticsLabel
flutter.material.progressIndicator.semanticsValue => flutter.progress_indicator.ProgressIndicator.semanticsValue
flutter.rendering.annotatedRegionLayer.restrictions => flutter.rendering.AnnotatedRegionLayer.restrictions
flutter.clipper.clipBehavior => flutter.rendering.ClipRectLayer.clipBehavior
flutter.leaderFollower.alwaysNeedsAddToScene => flutter.rendering.FollowerLayer.alwaysNeedsAddToScene
flutter.rendering.layer.findAnnotations.aboutAnnotations => flutter.rendering.Layer.findAnnotations.aboutAnnotations
flutter.rendering.object.needsCompositing => flutter.rendering.PaintingContext.pushClipRect.needsCompositing
flutter.rendering.object.pushClipLayer.offset => flutter.rendering.PaintingContext.pushClipRect.offset
flutter.rendering.object.oldLayer => flutter.rendering.PaintingContext.pushClipRect.oldLayer
flutter.rendering.object.pushLayer.offset => flutter.rendering.PaintingContext.pushLayer.offset
flutter.rendering.persistentHeader.showOnScreenConfiguration => flutter.rendering.PersistentHeaderShowOnScreenConfiguration
flutter.rendering.platformView.gestures => flutter.rendering.RenderAndroidView.gestures
flutter.rendering.platformView.layout => flutter.rendering.RenderAndroidView.layout
flutter.rendering.platformView.updateGestureRecognizers => flutter.rendering.RenderAndroidView.updateGestureRecognizers
flutter.rendering.editable.cursorOffset => flutter.rendering.RenderEditable.cursorOffset
flutter.rendering.editable.paintCursorOnTop => flutter.rendering.RenderEditable.paintCursorAboveText
flutter.rendering.editable.select => flutter.rendering.RenderEditable.selectPosition
flutter.rendering.followerLayer.anchor => flutter.rendering.RenderFollowerLayer.leaderAnchor
flutter.rendering.wheelList.diameterRatio => flutter.rendering.RenderListWheelViewport.diameterRatio
flutter.rendering.wheelList.itemExtent => flutter.rendering.RenderListWheelViewport.itemExtent
flutter.rendering.wheelList.magnification => flutter.rendering.RenderListWheelViewport.magnification
flutter.rendering.wheelList.offAxisFraction => flutter.rendering.RenderListWheelViewport.offAxisFraction
flutter.rendering.wheelList.overAndUnderCenterOpacity => flutter.rendering.RenderListWheelViewport.overAndUnderCenterOpacity
flutter.rendering.wheelList.perspective => flutter.rendering.RenderListWheelViewport.perspective
flutter.rendering.wheelList.renderChildrenOutsideViewport => flutter.rendering.RenderListWheelViewport.renderChildrenOutsideViewport
flutter.rendering.wheelList.squeeze => flutter.rendering.RenderListWheelViewport.squeeze
flutter.rendering.wheelList.useMagnifier => flutter.rendering.RenderListWheelViewport.useMagnifier
flutter.rendering.sliverPadding.limitation => flutter.rendering.RenderSliverEdgeInsetsPadding
flutter.rendering.platformView.preview => flutter.rendering.RenderUiKitView
flutter.rendering.viewport.cacheExtent => flutter.rendering.RenderViewportBase.cacheExtent
flutter.rendering.viewport.cacheExtentStyle => flutter.rendering.RenderViewportBase.cacheExtentStyle
flutter.services.autofill.autofillHints => flutter.services.AutofillConfiguration.autofillHints
flutter.services.autofill.AutofillScope => flutter.services.AutofillScope
flutter.services.glfwKeyHelper.modifiers => flutter.services.GLFWKeyHelper.modifierCapsLock
flutter.services.gtkKeyHelper.modifiers => flutter.services.GtkKeyHelper.modifierShift
flutter.services.rawKeyEventDataIOS.modifiers => flutter.services.RawKeyEventDataIos.modifierCapsLock
flutter.services.rawKeyEventDataMacOs.modifiers => flutter.services.RawKeyEventDataMacOs.modifierCapsLock
flutter.services.rawKeyEventDataWindows.modifiers => flutter.services.RawKeyEventDataWindows.modifierShift
flutter.services.restoration.bucketcreation => flutter.services.RestorationBucket.empty.bucketCreation
flutter.services.autofill.autofillContext => flutter.services.TextInput.finishAutofillContext
flutter.services.textInput.enableSuggestions => flutter.services.TextInputConfiguration.enableSuggestions
flutter.services.textInput.smartDashesType => flutter.services.TextInputConfiguration.smartDashesType
flutter.services.textInput.smartQuotesType => flutter.services.TextInputConfiguration.smartQuotesType
flutter.widgets.actions.multipleAdds => flutter.widgets.Action.addActionListener
flutter.widgets.platformViews.constructorParams => flutter.widgets.AndroidView.constructorArgs
flutter.widgets.platformViews.gestureRecognizersDescFoot => flutter.widgets.AndroidView.gestureRecognizers.descFoot
flutter.widgets.platformViews.gestureRecognizersDescHead => flutter.widgets.AndroidView.gestureRecognizers.descHead
flutter.widgets.platformViews.gestures => flutter.widgets.AndroidView.gestures
flutter.widgets.platformViews.hittestParam => flutter.widgets.AndroidView.hitTestBehavior
flutter.widgets.platformViews.layout => flutter.widgets.AndroidView.layout
flutter.widgets.platformViews.directionParam => flutter.widgets.AndroidView.layoutDirection
flutter.widgets.platformViews.lifetime => flutter.widgets.AndroidView.lifetime
flutter.widgets.platformViews.createdParam => flutter.widgets.AndroidView.onPlatformViewCreated
flutter.widgets.autofill.AutofillGroupState => flutter.widgets.AutofillGroupState
flutter.widgets.followerLayer.anchor => flutter.widgets.CompositedTransformFollower.targetAnchor
flutter.widgets.layoutBuilder.builderFunctionInvocation => flutter.widgets.ConstrainedLayoutBuilder
flutter.widgets.editableText.complexCharacters => flutter.widgets.EditableText.onChanged
flutter.widgets.reassemble => flutter.widgets.Element.reassemble
flutter.widgets.focus_manager.focus.keyEvents => flutter.widgets.FocusNode.keyEvents
flutter.widgets.focus_manager.focus.lifecycle => flutter.widgets.FocusNode.lifecycle
flutter.widgets.focusorder.comparable => flutter.widgets.FocusOrder.comparable
flutter.widgets.form.autovalidateMode => flutter.widgets.FormField.autovalidateMode
flutter.widgets.image.chainedBuildersExample => flutter.widgets.Image.frameBuilder.chainedBuildersExample
flutter.widgets.interactiveViewer.onInteraction => flutter.widgets.InteractiveViewer.onInteractionEnd
flutter.widgets.wheelList.childCount => flutter.widgets.ListWheelChildBuilderDelegate.childCount
flutter.widgets.widgetsApp.localeResolutionCallback => flutter.widgets.LocaleResolutionCallback
flutter.widgets.modalRoute.barrierColor => flutter.widgets.ModalRoute.barrierColor
flutter.widgets.modalRoute.barrierDismissible => flutter.widgets.ModalRoute.barrierDismissible
flutter.widgets.modalRoute.barrierLabel => flutter.widgets.ModalRoute.barrierLabel
flutter.widgets.modalRoute.maintainState => flutter.widgets.ModalRoute.maintainState
flutter.mouseRegion.triggerTime => flutter.widgets.MouseRegion.onEnter.triggerTime
flutter.widgets.navigator.pushNamed.arguments => flutter.widgets.Navigator.pushNamed
flutter.widgets.navigator.restorablePush.arguments => flutter.widgets.Navigator.restorablePush
flutter.widgets.navigator.restorablePushNamed.arguments => flutter.widgets.Navigator.restorablePushNamed.arguments
flutter.widgets.navigator.restorablePushNamed.returnValue => flutter.widgets.Navigator.restorablePushNamed.returnValue
flutter.widgets.pageRoute.fullscreenDialog => flutter.widgets.PageRoute.fullscreenDialog
flutter.widgets.pageView.allowImplicitScrolling => flutter.widgets.PageView.allowImplicitScrolling
flutter.widgets.child => flutter.widgets.ProxyWidget.child
flutter.widgets.slots => flutter.widgets.RenderObjectElement.insertRenderObjectChild
flutter.widgets.restoration.primitivevalue => flutter.widgets.RestorableNum
flutter.widgets.restoration.primitivevalue.constructor => flutter.widgets.RestorableNum.constructorArgs
flutter.widgets.restoration.scope => flutter.widgets.RestorationScope
flutter.widgets.scrollable.scroll_increment_type.intent => flutter.widgets.ScrollIncrementType.intent
flutter.widgets.sliverChildDelegate.lifecycle => flutter.widgets.SliverChildDelegate.lifecycle
flutter.widgets.sliverMultiBoxAdaptor.delegate => flutter.widgets.SliverMultiBoxAdaptorWidget.delegate
flutter.widgets.subscriptions => flutter.widgets.State.initState
flutter.widgets.textSelection.onSelectionHandleTapped => flutter.widgets.TextSelectionOverlay.onSelectionHandleTapped
FilterQuality => flutter.widgets.Texture.filterQuality
flutter.widgets.transitionRoute.opaque => flutter.widgets.TransitionRoute.opaque
flutter.widgets.transitionRoute.reverseTransitionDuration => flutter.widgets.TransitionRoute.reverseTransitionDuration
flutter.widgets.transitionRoute.transitionDuration => flutter.widgets.TransitionRoute.transitionDuration
flutter.widgets.implicit_animations.tweenVisitorArguments => flutter.widgets.TweenVisitor.arguments
widgets.inspector.trackCreation => flutter.widgets.WidgetInspectorService.getChildrenSummaryTree

I think flutter.widgets.child => flutter.widgets.ProxyWidget.child is kind of dubious, since it's a pretty core thing to Widget, and kind of a detail that it's defined in ProxyWidget, but at least you do know where the template resides if you wanted to change it, and makes it easy to find.
The slider and range slider changes also might be a step back, but they contain the same information, just in the more standard form.
[And, no, I didn't audit all of the templates by hand: I used a hacked version of dartdoc to print out the canonical names of the symbols each of the templates was attached to, and then (by hand) fixed up the ones that all mapped to the same one by adding their descriptions back in, and made a quick script to do the substitutions in place, verifying first that none of the new names already existed.]
Related Issues

https://github.com/flutter/flutter/issues/69252

Tests

This is a "docs only" only change, so no tests.

Please make sure to update the style guide accordingly.
Since this can be mechanically tested, it might be worth doing so.
I share your dubiousness about things like "flutter.widgets.Clip => flutter.material.Material.clipBehavior". I agree that an argument can be made that at least now we know where to go to find the canonical definition.
cc @goderbauer

Yes, I'll update the style guide to match.
Auditing it might be hard: I'd have to modify dartdoc to spit out the failures, which is a bigger job than just hacking it like I did, since I wouldn't want to impose our rules on others. Or I could build my own script that used the analyzer. Maybe the second option's not too bad, actually.  I'll check it out if I have time.

Definitely time box any effort to write automated testing, yeah. What we need is a way to schedule ourselves to do this kind of manual audit regularly. Maybe we should have a Wiki page that says "In October, someone should audit the templates. In May and December, someone should check the deprecation notices for any that need clearing out." along with some helpful hints for how we did each of these in the past.

That's a good idea.  I do that about once a year or so with the spell checking, since it's really kind of a manual process.

Can you document the naming style for macros in the style guide (e.g. that they should be named based on the lib they are defined in and based on the property they document?).
We should also document in the wiki how to check for this manually every so often.

Yes, I added a section to the style guide last week.  It reads:

Dartdoc templates and macros
Dartdoc supports creating templates that can be reused in other parts of the code. They are defined
like so:
/// {@template <id>}
/// ...
/// {@endtemplate}

and used via:
/// {@macro <id>}

The <id> should be a unique identifier that is of the form flutter.library.Class.member[.optionalDescription].
For example:
// GOOD:
/// {@template flutter.rendering.Layer.findAnnotations.aboutAnnotations}
/// Annotations are great!
/// {@endtemplate

// BAD:
/// {@template the_stuff!}
/// This is some great stuff!
/// {@endtemplate}

The optionalDescription component of the identifier is only necessary if there is more than one
template defined in one Dartdoc block. If a symbol is not part of a library, or not part of a class, then
just omit those parts from the ID.


feature/517 added to learning plan notification replacement

@jenbutongit Gonna leave the message as per current story - so [Title] successfully added. [Title] has been successfully added to your learning plan
Once Rich has settled down his requirements we'll add new task and update.


unable to run localy on MacOS, @nullable can't be used as an annotation

The error is
lib/data/app_user.freezed.dart:49:4: Error: This can't be used as an annotation; an annotation should be a reference to a compile-time constant variable, or a call to a constant constructor.

  @nullable

   ^

My MacOS version is
macOS Catalina 10.15.7
Flutter version
Flutter 2.1.0-10.0.pre • channel dev • https://github.com/flutter/flutter.git
Framework • revision cc9b78fc5c (8 天前) • 2021-02-25 13:26:03 -0800
Engine • revision a252ec09b7
Tools • Dart 2.13.0 (build 2.13.0-77.0.dev)

pls have a look at the attachment for all errors
error.log

I met same error
lib/data/book_data.freezed.dart:1739:8: Error: This can't be used as an annotation; an annotation should be a reference to a compile-time constant variable, or a call to a constant constructor.
@nullable TextAlign align}) = _$_BoxStyle;


feat: Docker Image

Hello, this little PR add a simple docker image to permit the usage of the bot in egg/docker environment !
I have also added a workflow to generate the docker image when a tag is released.

@HirziDevs Your thoughts?

@HirziDevs Your thoughts?

I think it's fine? But do it works with the new pterostats?

@HirziDevs Your thoughts?

I think it's fine? But do it works with the new pterostats?

I will try to rebase and test it

@HirziDevs Ok so it worked but with the new way of setup that you have introduce. It require to have the config.yaml already setup before starting the bot. (It's not a big problem because the bot will show an error if the config.yaml is not present so it's okay !)
I have also patch the conflict (check the README.md if i haven't made an issue ^^)

@PandeoF1 can you please update the code? i did fix the config error issue

@PandeoF1 can you please update the code? i did fix the config error issue

Can you please revert the merge ? I have added a little bit of docs and two lines in the docker compose

@PandeoF1 can you please update the code? i did fix the config error issue

Can you please revert the merge ? I have added a little bit of docs and two lines in the docker compose

Or i can make another PR to add the new change if u want

@PandeoF1 can you please update the code? i did fix the config error issue

Can you please revert the merge ? I have added a little bit of docs and two lines in the docker compose

Or i can make another PR to add the new change if u want

You can create another PR


[BUILD ERROR] lining failed in call to always_inline '__mm512_xxx_xxx'   xxxxxxxxx

Is there an existing issue for this?

[X] I have searched the existing issues

Description of the Build Error
build from source ERROR  pipe branch
may error while build  bind.cpp/adam_cpu.hpp
my cpu: AMD EPYC 7742 64-Core Processor
Expected Behavior

To Reproduce

Environment Information
- Operating System version:20.04
- GCC version:9
- Pybind version:11
- CUDA version:11.8
- NVIDIA NCCL CU11 version:
- CMake version:3.16.3
- Pip version:22.3.1

Installation Method
Source Code Installation
Full Error Traceback
not available
Additional Information
No response
Confirmation

[X] I have reviewed and verified all the information provided in this report.

Check if your cpu can support AVX512. If not , try export BUILD_DOCKER_ENV = 1 and it will let compiler add avx512 flag instead of using native option.


ExponentialMovingAverage not found in checkpoint

when eval
bazel-bin/inception/imagenet_eval
--data_dir ~/dataset/imagenet-data/
--net alexnet
--image_size 224
--batch_size 50
--checkpoint_dir /tmp/imagenet_train
--restore_avg_var True
--eval_dir /tmp/imagenet_eval
error : ExponentialMovingAverage not found in checkpoint

Try to add --tower 0
Node that, in TernGrad, parameters are allocated in each GPU to reduce communication because we can communicate quantized gradients instead of floating parameters. By default, the program saves parameters in all GPUs. To evaluate/test, use --tower <gpu_id> to specify which GPU's parameters you want to test on. (We will try to remove this feature to save storage, because the parameter sets are identical across all GPUs). Alteratively, you can use --save_tower 0 in training executables to avoid saving duplicated parameters, in which case, --tower <gpu_id> is unnecessary during evaluation/testing.

Thank you for your help. I solved my problem.
And I have an other question.I don't know how many epochs I have trained. which script should I see to
know ?  I try to vi inception/inception_train.py  and there seems no answer

Somehow, the flag is moved to https://github.com/wenwei202/terngrad/blob/master/terngrad/inception/bingrad_common.py#L23
Use bazel-bin/inception/inception_train --help for all usages.


feat(admin-ui): add page to generate token for service user

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9466

Relevant Lines:
98889

💛  - Coveralls


install failed

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

git clone the repo
checkout to the tag of release-1.0.0
run the shell script by "bash ymir.sh start"
give me an error message like this screen

Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

Server OS: ubuntu:20.04
Ymir Version [ release-1.0.0]

Additional context
Add any other context about the problem here.

update docker-compose to version 1.29.2 or higher


Would you be open to PR to provide a custom RegionLabel in the props?

Hi,
I really like your library, thank you for open sourcing it.
Our use case requires us to tag more than just a string and a list of string. Would you accept a PR when we give a custom form and use it instead of the provided RegionLabel ?
Cheers

Hi @ThibautGeriz,
Do you mean a custom html form? What if we allowed JSON like this:
[
{ "type": "radiogroup", name: "Is valid?", options: ["Valid", "Invalid"] },
{ "type": "text", name: "What type of object?" }
]

We may be able to list such a form on the sidebar if it gets too long. Let me know what you think.

Our use case has logic between the inputs so I was thinking giving a react component in the props with a custom logic within. It will implement all the callback and props that you are giving.
By default the library would use RegionLabel unless the prop was provided.
I have already started to work a prototype and test it with our platform, your code is easy to read and the changes are small. I will open a PR soon as I would rather have that merged instead of us maintaining our custom fork.

Thanks!

merged in #32


Being able to see Torchlights and Other effects through blocks

Minecraft Version
1.19.2
ModLoader
Forge
Shimmer version
0.1.14
Modpack info or mod list(please reduce mod range)
https://pastebin.com/Qg4pYbzH
The latest.log file and hs_err_pidXXX if exists
https://pastebin.com/WtSM3W9E
optifine , Rubidium , flywheel or any rendering related mods. mod version is required.
No response
Issue description
I seem to be able to see Torch particles for Ex. through walls with this mod installed after pickin up an item

as u might be able to see on the screen there is the flame in the Item shadow.
Thats the flame of the Torch underground.
its only a short amount of time as it basicly bursts in an out so it was hard to catch.
Steps to reproduce
place a torch (for example, works with quark crystals too if the Bloomin Blocks TP is installed) underground.
Cover it.
Look at its direction and pick an item up
Profit?
Other information
I originaly expected it to be an issue with the Bloomin Blocks TP
Or my vanilla shaders.
But i removed any other mods and Texturepacks and at least as my testing goes its a general issue.

After further testing i found this to be an issue with any Shimmer versions i could test (all except the 1.20). Only have tested with forge so far will try fabric soon.
Since i forgot in the Original Post:
Cpu:Intel i9-9900K
GPU: 2080ti
If anything else might be required tell me
My GPU drivers are always up to date.
Also i asked some of my friends who i play on a server with if they have the same issue and they all said yes so far.
Well as soon as the 1.20 loadin issue is fixed imma try that version too. But currently its seems unfixable for me.
The only way to get rid of it is to Deactivate all Particles. Cause if there is no Particles none can be shown tru walls.

Okay its also a thing for me in Fabric :/

same issue here
removing the injectCreateParticle inject avoids the issue
guess the post particles don't cull correctly

Good that at least means its fixable. Ig.

As this seems to be fixed in the 1.20.1 Versions ima close this issue now


Add Plethora metadata for Item Magnets

I'm suggesting chat messages be sent for updates with the item magnet, such as toggle notifications, and low charge notifications. I can make a PR if this suggestion is approved.

Closed by 64fbf40bedcdc24a065b9acd003424069484f3d3


Game menu

Able to start game
Settings ?
Exit

duplicate -> https://github.com/AnonymusRaccoon/Bomberman/issues/21


AQCU-1125: adding in blue/green basic/processor derived ts descriptio…

…n to legend. also attempt to add in the noprocessor since seems like that got missed. will have marty check that out.

Looks like you covered everything @mhines-usgs


Remove locking from nerve_httpd.go

Since we have introduced service whitelisting, there is no need
to store status of the service for polling.
Storing of status was purely implemented to avoid spamming
unrelated services. But since now we have whitelisting, this is
not necessary.

lg2m


Added a colon

Added a colon

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

#sign-off


test(log-messages): adapt expected output logs to Snakemake v8

It runs without an error and passes the tests nicely.


Readme changes

with new release changes have already been pushed to master branch hence no need for this PR


Changes so that the test fails when first run.

I assumed that the tests should fail as a starting point and changed the capitalisation of Hello.
The local gitignore ignores .pyc-files.

Actually, in this case I'm happy with the test passing at the first attempt. The other language versions are the same, I'd rather keep the existing behaviour, thank you.


Leetcode Question Number 1915

Leetcode Question Number 1915 solution needs to be added in the repo .
The link for the question is this.
Accepted Languages C++ / JAVA .
You Only need to add the  class Solution in the file.
Make Sure you have read the contributions guidelines before creating an PR.

assign


Glowing inline suggestions

https://codepen.io/yuin-chien/pen/aaYqVR is Yuin's animation and styling for the inline chips.
Can we apply the same CSS to the inline suggestions?
(Once we seem them in context we can adjust frequency and speed of the animation)

keeping open to adjust animation and color.


Respect NDEBUG flag

Respect NDEBUG flag to build non-debug version of Zopfli.

That's nice! I'm seeing 6% speed improvement with -DNDEBUG

NDEBUG disables all aserts so there are no checks to abort execution when something doesn't go right. I think without this commit and just by issuing -D NDEBUG You should have similar improvement.


Add option to always include changesets

Azure DevOps Extensions
Generate Release Notes (Node Cross Platform)
Is your feature request related to a problem? Please describe.
I would like to include the changesets associated to a build, even there is nothing new, also, since i'm running it from the release pipeline, i would like to have the changesets every time i run the release of the same build!
The old version did this just fine without any setting needed! :(
Describe the solution you'd like
A checkbox would be fine, to always include changesets, no matter what!
Describe alternatives you've considered
No response
Additional context
No response

This task has only ever returned the list of changes between two builds, so I am unclear what you mean by
'The old version did this just fine without any setting needed! :('.
Which version of this task were you using?

@rfennell thank you for your attention to this! sorry my english is not very good, so i will try to explain it via an example!
Let's say i make a build, with 2 changesets, it will be build 1, and in the release pipeline i want to make a release notes to send out in email, with that 2 changesets. But after that, i want to make a new build without any new changesets, it will be build 2! So there is no difference between the two builds, so no changesets would be gathered by your tool in the release pipeline!
Also, if i want to re-release the build 1, the same thing happens!
I used the V2 beforehand, but i ran into some encoding issue which i could not resolve, and it just doesnt work anymore!

@rfennell Thank you, but i need this to be automated! :(

As to automating this look back, I have no plans to alter the current functionality.
As to your error can you provide the full log so I can see where the error is occurring

@rfennell of course, here is the full output:
2023-04-05T07:09:11.2630554Z ##[section]Starting: Generate release notes (PowerShell)
2023-04-05T07:09:11.2735599Z ==============================================================================
2023-04-05T07:09:11.2735906Z Task         : (deprecated) Generate Release Notes for Pipeline Builds or Releases (PowerShell)
2023-04-05T07:09:11.2736298Z Description  : (deprecated) Task is now deprecated. Please consider my Cross Platform Version https://marketplace.visualstudio.com/items?itemName=richardfennellBM.BM-VSTS-XplatGenerateReleaseNotes
2023-04-05T07:09:11.2736627Z Version      : 3.9.10
2023-04-05T07:09:11.2736755Z Author       : Black Marble
2023-04-05T07:09:11.2737730Z Help         : Version: 3.9.10. More Information
2023-04-05T07:09:11.2738014Z ==============================================================================
2023-04-05T07:09:15.2017119Z ##[error]A parameter cannot be found that matches parameter name 'Encoding'.
2023-04-05T07:09:15.2270138Z ##[section]Finishing: Generate release notes (PowerShell)

Ah, I see you are using the 'Generate release notes (PowerShell)' based release notes, and not the 'Generate Release Notes (Node Cross Platform)' task as this issue was logged under. So, you can ignore all my previous answers as I was discussing a completely different task.
I ceased support for the 'Generate release notes (PowerShell)' in Jan 2022, and focused all my efforts on the far for capable 'Generate Release Notes (Node Cross Platform)' task. This deprecation is noted in the task log.

Task : (deprecated) Generate Release Notes for Pipeline Builds or Releases (PowerShell)
Description : (deprecated) Task is now deprecated. Please consider my Cross Platform Version https://marketplace.visualstudio.com/items?itemName=richardfennellBM.BM-VSTS-XplatGenerateReleaseNotes

You can of course pull down the source and fork your own version of this old task, but will not be publishing any updates.
If you do that, I suspect you will find the the issue is that your Azure DevOps agent runner itself has been updated to from 2.x to 3.x. as the V3 agents are now generally available.  This could have changed the underlying installed version of PowerShell and my guess is that a function's parameters has changed.

@rfennell Yes, i'm using the old, depreceted one! I really would like to use your latest, but it does not work for me! :(
I'm not able to create and run the plugin for myself, however i can run powershell scripts in our pipeline, so if you would be so kind and give me the old, deprecated script which i could use inline, i would be most thankful!

The code for all my tasks is freely available in this repo. They are all in the extensions folder.
The one you want is generatereleasenotes one. You should be able to extract that task as a script.

@rfennell Thank you again, for your help!
I tried the scripts, but i have the following error:
2023-04-12T09:31:56.5530397Z D:\Agent0_work\r243\a\Demo\drop\GenerateReleaseNotes.ps1 : The term 'Get-VstsInput' is not recognized as the nam
2023-04-12T09:31:56.5531034Z e of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, v
2023-04-12T09:31:56.5531370Z erify that the path is correct and try again.
2023-04-12T09:31:56.5531730Z At D:\Agent0_work_temp\abcbf565-2a18-4732-b966-e53b0ac02780.ps1:3 char:1
2023-04-12T09:31:56.5532086Z + . 'D:\Agent0_work\r243\a\Demo\drop\GenerateReleaseNotes.ps1'
2023-04-12T09:31:56.5532390Z + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2023-04-12T09:31:56.5532983Z     + CategoryInfo          : ObjectNotFound: (Get-VstsInput:String) [GenerateReleaseNotes.ps1], CommandNotFoundExcept
2023-04-12T09:31:56.5533557Z    ion
2023-04-12T09:31:56.5533796Z     + FullyQualifiedErrorId : CommandNotFoundException,GenerateReleaseNotes.ps1
2023-04-12T09:31:56.5534038Z
2023-04-12T09:31:56.6082902Z ##[error]PowerShell exited with code '1'.
The Get-VstsInput is not a built in command?

All Azure DevOps PowerShell based tasks are reliant on the Azure DevOps PowerShell SDK which is shipped in this folder with the task. I would suspect you will have to load this module, or replace it's functions.
For example Get-VstsInput gets an Azure DevOps task parameter value, but if you are just running the code as a script this could be replaced by a simple PowerShell Param block

@rfennell Thank you, i'm really starting to get the jist of it!
But as always, new problems arose...
2023-04-12T13:00:02.3109382Z VERBOSE: Using SystemVssConnection personal access token
2023-04-12T13:00:02.3998910Z Invoke-GetCommand : The term 'Get-VstsEndpoint' is not recognized as the name of a cmdlet, function, script file
I guess i could replace this with a prameter as well, but how can i get the needed information for this?
Thank you!

Can you provide the full log so we can see what build/releases numbers it has selected.
I doubt you are doing anything wrong, it is just that if using Classic Releases you only see associates on the first deployment of a given build

@rfennell  Here you go, it was a first deployment:
2023-04-27T07:17:27.6556169Z ##[section]Starting: Generate Release Notes based on Release Comparison API
2023-04-27T07:17:27.6649839Z ==============================================================================
2023-04-27T07:17:27.6650138Z Task         : Generate Release Notes (Crossplatform)
2023-04-27T07:17:27.6650398Z Description  : Generates a release notes file in a format of your choice from the build or release history
2023-04-27T07:17:27.6650609Z Version      : 4.6.2
2023-04-27T07:17:27.6650759Z Author       : Black Marble
2023-04-27T07:17:27.6651822Z Help         : Version: 4.6.2. More Information
2023-04-27T07:17:27.6652166Z ==============================================================================
2023-04-27T07:17:41.6087963Z Creating Azure DevOps API connections for https://demo.hu/DEMOCollection/ with 'allowRetries' set to 'true' and 'maxRetries' count to '20'
2023-04-27T07:17:41.6092618Z Creating the credential handler from the OAUTH token
2023-04-27T07:17:42.2375088Z Getting the current release details
2023-04-27T07:17:42.2762074Z Show associated items for primary artifact only is set to false
2023-04-27T07:17:42.2762454Z Getting the Environment Id
2023-04-27T07:17:42.2762684Z Identified [dev] as having id [690]
2023-04-27T07:17:42.2765202Z Finding successful deployments
2023-04-27T07:17:42.2955784Z Found 6 releases to consider
2023-04-27T07:17:42.2958479Z Finding the last successful release Release-28
2023-04-27T07:17:42.2958727Z Getting all artifacts in the current release...
2023-04-27T07:17:42.2960432Z Found 1 artifacts for current release
2023-04-27T07:17:42.3342142Z Getting all artifacts in the most recent successful release [Release-28]...
2023-04-27T07:17:42.3342839Z Found 1 artifacts for most recent successful release
2023-04-27T07:17:42.3343096Z Looking at artifact [Demo]
2023-04-27T07:17:42.3343282Z Artifact type [Build]
2023-04-27T07:17:42.3343472Z Build Definition ID [905]
2023-04-27T07:17:42.3343669Z Build Number: [1.0.0-12]
2023-04-27T07:17:42.3343851Z Is Primary: [true]
2023-04-27T07:17:42.3344091Z Looking for the [Demo] in the most recent successful release [Release-28]
2023-04-27T07:17:42.3344405Z Found artifact [Demo] with build number [1.0.0-11] in release [Release-28]
2023-04-27T07:17:42.3344756Z Checking what commits and workitems have changed from [1.0.0-11][ID 47374] => [1.0.0-12] [ID 47442]
2023-04-27T07:17:42.3345137Z Defaulting on the workaround for build API limitation (see issue #349 set 'ReleaseNotes.Fix349=false' to disable)
2023-04-27T07:17:42.3345479Z Using workaround for build API limitation (see issue #349)
2023-04-27T07:17:42.4365825Z Getting test associated with the latest build [47442]
2023-04-27T07:17:42.4722684Z Adding the build [47442] and its associations to the unified results object
2023-04-27T07:17:42.4723158Z Detected  0 workitems between the current build and the last successful one
2023-04-27T07:17:42.4723576Z Found 0 tests associated with the build [47442] adding any not already in the global test list to the list
2023-04-27T07:17:42.4866177Z Found 0 manual tests associated with the build [47442] adding any not already in the global test list to the list
2023-04-27T07:17:42.4867652Z
2023-04-27T07:17:42.5392507Z Removing duplicate Commits from master list
2023-04-27T07:17:42.5395658Z Expanding the truncated commit messages...
2023-04-27T07:17:42.5395906Z Expanded truncated commit messages 0
2023-04-27T07:17:42.5396132Z Leaving CS in default order as returned by API
2023-04-27T07:17:42.5396389Z Find any WorkItems linked from GitHub using the AB#123 format
2023-04-27T07:17:42.5398160Z Adding 0 found using AB#123 links in GitHub comments
2023-04-27T07:17:42.5398475Z Removing duplicate WorkItems from master list
2023-04-27T07:17:42.5398752Z Getting tests linked to WorkItems
2023-04-27T07:17:42.5399769Z Checking for user defined WIQL query
2023-04-27T07:17:42.5401334Z No WIQL query parameters defined
2023-04-27T07:17:42.5402481Z Leaving WI in default order as returned by API
2023-04-27T07:17:42.5525669Z The default artifact for the release was not linked to an Azure DevOps Git Repo Pull Request
2023-04-27T07:17:42.5526169Z Getting all completed Azure DevOps Git Repo PRs in the Team Project IFP
2023-04-27T07:17:42.5526406Z Getting associated PRs
2023-04-27T07:17:44.7616471Z Found 3164 Azure DevOps PRs in the repo
2023-04-27T07:17:44.7617140Z Enriching known Pull Requests
2023-04-27T07:17:44.7619105Z Total Builds: [1]
2023-04-27T07:17:44.7619684Z Total Commits: [0]
2023-04-27T07:17:44.7620056Z Total Workitems: [0]
2023-04-27T07:17:44.7620378Z Total Related Workitems (Parent/Children): [0]
2023-04-27T07:17:44.7620745Z Total Release Tests: [0]
2023-04-27T07:17:44.7620996Z Total Tests: [0]
2023-04-27T07:17:44.7621566Z Total Manual Test Runs: [0]
2023-04-27T07:17:44.7621846Z Total Manual Test Configurations: [0]
2023-04-27T07:17:44.7622157Z Total Pull Requests: [0]
2023-04-27T07:17:44.7622447Z Total Indirect Pull Requests: [0]
2023-04-27T07:17:44.7622718Z Total Associated Test WI: [0]
2023-04-27T07:17:44.7622970Z Total Published Artifacts: [0]
2023-04-27T07:17:44.7623202Z Total Consumed Artifacts: [0]
2023-04-27T07:17:44.7623423Z Total WIQL Workitems: [0]
2023-04-27T07:17:44.7848779Z Generating the release notes, the are 1 template(s) to process
2023-04-27T07:17:44.7849712Z Using in-line template
2023-04-27T07:17:45.8862399Z Loading handlebars-helpers extension
2023-04-27T07:17:55.8041927Z Completed processing template
2023-04-27T07:17:55.8042573Z Writing output file D:\Agent0_work\r243\a\Demo\drop\releasenotes.md
2023-04-27T07:17:55.8045899Z Finished writing output file D:\Agent0_work\r243\a\Demo\drop\releasenotes.md
2023-04-27T07:17:55.8046299Z Output variable 'undefined' set to value of first generated release notes
2023-04-27T07:17:55.8858046Z ##[section]Finishing: Generate Release Notes based on Release Comparison API

I can see you are running in a Classic Release (not a Classic Build or multi-stage YAML), this reduces what the task can do, due to the APIs available, and makes testing harder
So your flow is

The task found 6 past releases to consider, the last successful one was Release-28, so this is the base for comparison
It looks for artifacts on the current release, and found 1 [Demo]
It then compares the version of [Demo] released in Release-28 and the the current release to see any new WI or CS added, Checking what commits and workitems have changed from [1.0.0-11][ID 47374] => [1.0.0-12] [ID 47442]
It find none, hence nothing in the release notes.

As I said testing Classic releases it awkward as you really have to trigger new builds (with new Commits & WI) for every test.

@rfennell I have changesets
159529 and 159731 in Release-28,
159821 and 159823 in Release-29
So it must be some mistake on my end if the generator finds nothing?

In the end you will find that basically all the task does is make calls to the APIs

Builds - Get Changes Between Builds
Builds - Get Work Items Between Builds

You can test these calls in a browser or Postman to see what is returned.
After that, all I can suggest is you experiment with the command line tester and step through the code being run.

@rfennell Well, the Build 11 has 47374 and the Build 12 has 47442 id.
This one: _apis/build/workitems?fromBuildId=47374&toBuildId=47442 returns abolutely nothing:
{"count":0,"value":[]}
However this one: _apis/build/changes?fromBuildId=47374&toBuildId=47442 has what i need:
{
"count": 2,
"value": [
{
"id": "C159821",
"message": "!Android belső frissítés",
"type": "TfsVersionControl",
"author": {
"displayName": "Aros Zoltán",
"id": "63d966a0-82c8-49d5-a068-85af1366bc9d",
"descriptor": "win.Uy0xLTUtMjEtMTQ4MjQ3NjUwMS0xMjc1MjEwMDcxLTcyNTM0NTU0My0xMjQ5NDU"
},
"timestamp": "2023-04-26T13:54:52.433Z",
"location": "/_apis/tfvc/changesets/159821"
},
{
"id": "C159733",
"message": "!Automatic application versioning check-back from pipeline",
"type": "TfsVersionControl",
"author": {
"displayName": "Project Collection Build Service (MAVICollection)",
"id": "5be81851-307c-4a4d-b06f-f0dbfea8d4d5",
"descriptor": "svc.ZTE2ZjIzOGQtMDJmOS00MGNhLTliOWUtYzJhNjI4NDRjOTdhOkJ1aWxkOjgyYWM0YjU4LWFmNDMtNDVkZi1iZWQwLWNmYjRlYzFmM2FlZA"
},
"timestamp": "2023-04-23T12:06:45.537Z",
"location": "/_apis/tfvc/changesets/159733"
}
]
}

The first query is getting associated WI between the builds, the second is the changeset difference between the builds

@rfennell Thanks! so if the API endpoints give back the data, why does your generator give me back nothing?

What does the log show if you run a pipeline with the variable system.debug=true or via the test console
With debug on it shown where all the data is gather from

A new release has been i 2.7.15 that now provides the option that is a retry is needed (after the commit is made locally when the push fails) that you can (default as before) try a get pull or a pull pull --rebase=true
To enable this option you need to set the retrymode parameter and set your branch name
  - task: richardfennellBM.BM-VSTS-WIKIUpdater-Tasks.WikiUpdaterTask.WikiUpdaterTask@2
    displayName: 'Git based WIKI Updater - GitHub folder'
    inputs:
      repo: github.com/rfennell/demorepo.wiki
      filename: TestFolder/${{parameters.platform}}-PageFromWikiUpdater.md
      message: 'Update from Release - $(Release.ReleaseName)'
      contents: |
        Header
        ---
        $(OutputText)
        ---
        Footer
        message: 'Update from Release - $(Release.ReleaseName)'
      gitname: builduser
      gitemail: 'build@demo'
      RetryMode: Rebase
      branch: main
      user: rfennell
      password: '$(githubPAT)'
      localpath: '$(System.DefaultWorkingDirectory)\githubrepo'

I am far from sure this change will actually help, and I have struggled to test it, so let me know how it goes


Integration test for ProxyStatus and Experimental AuthZ Check

Part one of the fix for https://github.com/istio/istio/issues/23042
Once the istioctl subcommands that query Envoy are tested during integration we will remove the static unit tests for those subcommands.
cc @howardjohn

@esnible: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

Can anyone from @istio/wg-user-experience-maintainers review this for me?

LGTM. Refinements can be done as needed. @esnible would you file a bug to track the ads: default problem @howardjohn found?

Issue https://github.com/istio/istio/issues/23526 tracks that problem.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test lint_istio

integ-local-tests_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test lint_istio

integ-local-tests_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test integ-local-tests_istio

integ-pilot-k8s-tests_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test lint_istio

integ-local-tests_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test integ-local-tests_istio

integ-pilot-k8s-tests_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test integ-pilot-k8s-tests_istio

integ-multicluster-k8s-tests_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
b8a1c42d37ae110c85bf782060a0730f19f0c3da
link
/test integ-multicluster-k8s-tests_istio

integ-pilot-k8s-tests_istio
84920a88de89c540928417d9e38a90bb33f78d89
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@googlebot I fixed it.

I fixed it.  Question.  I did git rebase upstream/master, made some changes, added, committed, pushed.
The PR report hundreds of files chosen.  You commented there was a bad merge.
To get it back to 2 files, I clicked "Edit" near the title, changed to another base, did it again, to change it back to be based on master.  That made the PR look normal.  What git command should I have used instead?

@esnible: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
84920a88de89c540928417d9e38a90bb33f78d89
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@esnible: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
84920a88de89c540928417d9e38a90bb33f78d89
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

This needs to go on hold until https://github.com/istio/istio/pull/23724 merges.  A performance improvement in istioctl proxy-status <pod> is keeping this test from passing.  When that PR merges it it will be safe to retest this one.

/retest

@esnible: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
84920a88de89c540928417d9e38a90bb33f78d89
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Added a fix that will also resolve https://github.com/istio/istio/issues/23735

@howardjohn Can you re-review?

In response to a cherrypick label: new pull request created: #23766


[BI-1494] - Fixing pedigree navigation due to merge

Description
Story: BI-1494 - Improve Pedigree View Display
(Fixes QA failure found for BI-1494)
Merging/rebasing changed order of parameters in BrAPIService.ts:get, which in turn resulted in an invalid url for the get call and being redirected to a blank page with a console error. This change fixes the parameters in the GermplasmDetails.vue call to BrAPIService.ts:get.
Dependencies
bi-api/develop
Testing
In a Breedbase program, open germplasm details page for a germplasm with parents and children
Click on different nodes in the pedigree viewer, they should redirect to the correct germplasm details pages
Checklist:

[X] I have performed a self-review of my own code
[X] I have tested my code and ensured it meets the acceptance criteria of the story
[ ] I have create/modified unit tests to cover this change
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to documentation
[ ] I have run TAF: <link to TAF run>

happy-path test passes,

if a the program's data is stored on a breedBase instance and
if a Chrome (not a Firefox) browser is used.


How to use

It's seem a good sdk, but i can't understand how to import to my project,
Installation and Usage is empty
This is no .jar file, no compile like
compile 'com.adobe.creativesdk.foundation:auth:0.9.1251'
when i use
compile project(':photoeditorsdk')
i get error
Error:(52, 0) Project with path ':photoeditorsdk' could not be found in project ':app'.
This sample has two floder has AndroidManifest.xml
photo-editor-android-master/photoeditorsdk/src/main/AndroidManifest.xml
photo-editor-android-master/app/src/main/AndroidManifest.xml
I can't know what is the main
Please help:)

I did it now, thx a lot


[Track 2 migration] Migrate DataboxEdge

Migrate DataboxEdge to track 2 SDK.

Track 2 migration

Supported in https://github.com/Azure/azure-cli/pull/18678


No module named 'kthread'

in win platform

can support cross platform?

pip install kthread

and
pip install keyboard
success!