Do tickets need to be refreshed if changes are made to delegation for a service account?

I am curious to what is all in the ticket for Kerberos and if I have a app running that has a cached token do I need to purge then when delegation changes are done or does the ticket only contain info for the user?

For the user getting delegated or the service account? In the user case, sorta. The forward bit is set to indicate the client can delegate it's TGT to the target if unconstrained delegation is enabled. For constrained delegation, I don't recall if the bit is set.
Service account might require the forward bit, though I don't recall if that's for all cases or just the second half of the S4U call for proxy.
In either case the delegation controls shouldn't be modified regularly so it shouldn't be taxing to clear the ticket cache afterword.

From: RussKahler1970 @.>
Sent: Tuesday, February 6, 2024 4:29:41 PM
To: dotnet/Kerberos.NET @.>
Cc: Subscribed @.***>
Subject: [dotnet/Kerberos.NET] Do tickets need to be refreshed if changes are made to delegation for a service account? (Issue #365)
I am curious to what is all in the ticket for Kerberos and if I have a app running that has a cached token do I need to purge then when delegation changes are done or does the ticket only contain info for the user?
—
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/Kerberos.NET/issues/365 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAJHTYJZYEPXENOVBY25W53YSKVFNBFKMF2HI4TJMJ2XIZLTSSBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJLJONZXKZNENZQW2ZNLORUHEZLBMRPXI6LQMWBKK5TBNR2WLKJVGY2DENZSGM2TLJDOMFWWLKLIMFZV63DBMJSWZAVFOZQWY5LFUR2HE5LFURXGC3LFVZ3WC5DDNBPWCY3UNF3GS5DZVRZXKYTKMVRXIX3UPFYGLJKJONZXKZNGORXXA2LDOOJYFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVA4DKNBYHEYTGOECUR2HS4DFUVUXG43VMWSXMYLMOVS2UMRRGIYTONZUHE3TPAVEOR4XAZNFNRQWEZLMUV3GC3DVMWUTKNRUGI3TEMZVGWTXI4TJM5TWK4VGMNZGKYLUMU.
You are receiving this email because you are subscribed to this thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

The forward bit is part of the ticket?  do I need to request that set or is the based on the users delegation?

So the ticket I am getting to make the web request is using spn= http/[host dns]  but for SQL I need  MSSQLSvc/[SQL instance]  do I have to do something to provide that ticket with the first ticket to the web request?

When you're making the ticket request are you also including the Negotiate delegate flag? From memory I don't recall if that's required for constrained delegation, but I'd guess yes.

From: RussKahler1970 @.>
Sent: Wednesday, February 7, 2024 11:47:51 AM
To: dotnet/Kerberos.NET @.>
Cc: Comment @.>; Subscribed @.>
Subject: Re: [dotnet/Kerberos.NET] Do tickets need to be refreshed if changes are made to delegation for a service account? (Issue #365)
So the ticket I am getting to make the web request is using spn= http/[host dns] but for SQL I need MSSQLSvc/[SQL instance] do I have to do something to provide that ticket with the first ticket to the web request?
—
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/Kerberos.NET/issues/365#issuecomment-1932569475 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAJHTYOPP35ARMRITRG4BBLYSO44PBFKMF2HI4TJMJ2XIZLTSSBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVE2TMNBSG4ZDGNJVURXGC3LFVFUGC427NRQWEZLMVRZXKYTKMVRXIX3UPFYGLLCJONZXKZKDN5WW2ZLOOSTHI33QNFRXHE4CUR2HS4DFVJZGK4DPONUXI33SPGSXMYLMOVS2QOBVGQ4DSMJTHCBKI5DZOBS2K2LTON2WLJLWMFWHKZNKGIYTEMJXG42DSNZXQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKJVGY2DENZSGM2TLJ3UOJUWOZ3FOKTGG4TFMF2GK.
You are receiving this email because you commented on the thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

how do I include this flag ( Negotiate delegate flag) to get GetServiceTiicket method?  I don't see a setting for that?  or in the Configuration on the client.
Thanks for your help?

I am a bit lost on how to request a delegable ticket.  should I be requesting a different ticket type?  When I login with an app that does connect properly I see these flags on my ticket.   that is what I need to make my ticket work.  How do I request or get those flags set on my ticket?
Ticket Flags 0x40a50000 -> forwardable renewable pre_authent ok_as_delegate name_canonicalize

I think I need to set the flag OkAsDelegate  but I cannot figure out how to set that on the client.

I am stuck at this point.  I cannot generate a ticket that is delegable and could use some help if someone has some example or ideas as to what to do.

There's a command in the Bruce tool that shows how this works: https://github.com/dotnet/Kerberos.NET/blob/3fd93b8f9628b611ed30fca422e533f3ee10c60f/Bruce/CommandLine/KerberosConstrainedDelegationCommand.cs
You can also set the Forwardable bit on the request itself:
client.KdcOptions |= KdcOptions.Forwardable;

Not sure what is going wrong.  I am still not getting the kerberos to delegate through to my sql login.
Here is how I am creating my client.
var clientArgs = new Krb5Config();
clientArgs.Defaults.Forwardable = true;
clientArgs.Defaults.EnforceOkAsDelegate = true;
clientArgs.Defaults.DnsLookupKdc = true;
clientArgs.Defaults.AllowWeakCrypto = true;
clientArgs.Defaults.KdcDefaultOptions |= KdcOptions.RenewableOk | KdcOptions.Forwardable;
_kClient = new KerberosClient(clientArgs, LoggerFactory);
Here is how I am requesting my ticket.
var args = new RequestServiceTicket
{
ServicePrincipalName = spn,
CacheTicket = true
       };
       args.KdcOptions |= KdcOptions.Forwardable;
       var session = await KClient.GetServiceTicket(args).ConfigureAwait(false);

       return session.ApReq;

Then I add to my httpclient header with.
client.DefaultRequestHeaders.Authorization = new AuthenticationHeaderValue("Negotiate", Convert.ToBase64String(ticket.EncodeGssApi().ToArray()));
using my logging on the server its says my identity in IIS is [domiain][service account] but when it tries to connect to SQL it tells me that Login failed for user 'NT AUTHORITY\ANONYMOUS LOGON'
The same site if I connect with another app we use,  connects and passes identity correctly with the Kerberos identity showing up as the user in SQL session.
I looked at the bruce command line file but that seems to be loading the krb5.conf so its not clear what I should all set in both create client and when I am getting the ticket.
Any help would be greatly apricated .

Also note my service account has delegation setup to allow delegation to any service.

You might try setting
args.ApOptions |= ApOptions.MutualRequired;
// OR
args.GssContextFlags |= GssContextEstablishmentFlag.GSS_C_DELEG_FLAG;

I vaguely recall we enforce some things with mutual auth so that might be a requirement. The deleg flag isn't technically right, but I don't have the Windows behaviors in front of me at the moment to verify.

I tried args.ApOptions |= ApOptions.MutualRequired;  and no different still not delegating the identity.
I tried args.GssContextFlags |= GssContextEstablishmentFlag.GSS_C_DELEG_FLAG; and the web site requiring windows authentication fails with a 401 error.

Any other thoughts on how to get the ticket to authenticate through delegation?  I am a bit stuck at this point.

I don't have any specific ideas why it's not working. I'd need to see more data unfortunately. Clearly there is a difference between working and non-working requests but I don't know what. It's almost certainly something small like a flag.
You could diff the two through a tool like the Bruce Decode utility. That would dump out the structures and could be compared. Downside is that you need the secret of the target server.

From: RussKahler1970 @.>
Sent: Tuesday, February 13, 2024 5:53:55 AM
To: dotnet/Kerberos.NET @.>
Cc: Comment @.>; Subscribed @.>
Subject: Re: [dotnet/Kerberos.NET] Do tickets need to be refreshed if changes are made to delegation for a service account? (Issue #365)
Any other thoughts on how to get the ticket to authenticate through delegation? I am a bit stuck at this point.
—
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/Kerberos.NET/issues/365#issuecomment-1941569927 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAJHTYJEVTL42AIUS2OC7FDYTNV7HBFKMF2HI4TJMJ2XIZLTSSBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVE2TMNBSG4ZDGNJVURXGC3LFVFUGC427NRQWEZLMVRZXKYTKMVRXIX3UPFYGLLCJONZXKZKDN5WW2ZLOOSTHI33QNFRXHE4CUR2HS4DFVJZGK4DPONUXI33SPGSXMYLMOVS2QOBVGQ4DSMJTHCBKI5DZOBS2K2LTON2WLJLWMFWHKZNKGIYTEMJXG42DSNZXQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKJVGY2DENZSGM2TLJ3UOJUWOZ3FOKTGG4TFMF2GK.
You are receiving this email because you commented on the thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

How do I get the secret from the server?   I have access to all of them.

this is the event viewer error I am getting.
A Kerberos error message was received:
on logon session
Client Time:
Server Time: 22:59:16.0000 2/13/2024 Z
Error Code: 0xd KDC_ERR_BADOPTION
Extended Error: 0xc000006e KLIN(0)
Client Realm:
Client Name:
Server Realm: SALELYTICS.LOCAL
Server Name: MSSQLSvc/sqlg1t.salelytics.local:1433
Target Name<EMAIL_ADDRESS>Error Text:
File: onecore\ds\security\protocols\kerberos\client2\kerbtick.cxx
Line: 128c
Error Data is in record data.

this is what the ticket I am getting looks like in the decoder.

Unfortunately the thing to check is in the EncryptedPart which requires the service account password. Unfortunately I can't tell you how to get that password, though there are ways. Worst case you can run the bruce tool on that server as SYSTEM.
The error you posted above is certainly interesting. That is indicating the USER_NOT_DELEGATED flag is set on the user account, which presents as the forwardable flag not set on the ticket.
The other question is whether the user is from another forest. The library doesn't support it when acting as the middlebox, but it should support it as the client.

Thanks for the info.  Which server would have have to run the bruce tool
on? the server I am running as the client (web server )  that is making the
call to get the ticket or do I need to run it on the server trying to parse
the ticket.   I have access to all of them.  I am currently using my own
account and that account is delegatable.   I have another app that
uses this site and my account connected to that app works fine and
delegates to SQL.
I will see if I can get more info.  If I provided you a serialized token
can you decode it?
THanks  for all your help on this.
Russ
On Tue, Feb 13, 2024 at 7:32 PM Steve Syfuhs @.***>
wrote:

Unfortunately the thing to check is in the EncryptedPart which requires
the service account password. Unfortunately I can't tell you how to get
that password, though there are ways. Worst case you can run the bruce tool
on that server as SYSTEM.
The error you posted above is certainly interesting. That is indicating
the USER_NOT_DELEGATED flag is set on the user account, which presents as
the forwardable flag not set on the ticket.
The other question is whether the user is from another forest. The library
doesn't support it when acting as the middlebox, but it should support it
as the client.
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/Kerberos.NET/issues/365#issuecomment-1942951182,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR2QYLHHLDLUEWIFE5LDX63YTQH3TAVCNFSM6AAAAABC4ZGCUSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBSHE2TCMJYGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Is there anyway using the client that got the ticket to decode it?

I stepped into getting the service ticket.  I think its all right there.   So I assume the ticket is right.   could it be in the GSS encoding where something is going wrong?  is there a way to decode the encrypted part of that ticket?

running the decode app from the solution provides more details.   not sure if this helps.  I don't see Ticket Flags here yet, is that still encrypted in the other parts?

Not sue how to move forward.  I am using an account that connects with Kerberos to the same site and works to delegate through to SQL so it seems to me it has to be an issue with the ticket/token.   From above ticket seems to have the flags set so it seems to be an issue with the encoding.   Any help or ideas would be great.   I am stuck at this point.  I guess some help decoding the ticket might help. if I could see that the ticket was ok to delegate.  then I might be able to consider other factors.

I bit more information.  if I use my personal account it fails unless our other client app using the same site has recently used sql access that is using the same impersonation.  I assume either the server is maintaining a ticket it its cache and that is why it worked.    Anyway please let me know if there is more I can provide.   the ticket I am getting/sending in the httpclient request is not working with delegation but the user is valid to delegate and the server and sql are all working with impersonation and delegation.   Not sure if there is anything else we can do.

Turns out I had and spn issue in our domain so it was not able to delegate the ticket through.  I have since resolved that and all is working fine.  I am going to close this issue.

Issue was an spn issue not a ticket generation issue.


[editor] Inactive heading block is the same color as active heading block

Describe the bug
When a block is active, other blocks are greyed out except headings, which remain the same color as the active block.
Expected behavior
Heading blocks are greyed out when inactive.
Screenshots

Fixed 🥳


React tooltip example error on mergeRefs → This expression is not callable

This is the official example in the docs: https://codesandbox.io/s/winter-tree-wmmffl?file=/src/Tooltip.tsx:0-1458
If I freshly install both @floating-ui/react-dom-interactions and react-merge-refs I get an error on
const ref = useMemo(() => mergeRefs([reference, (children as any).ref]), [reference, children]);

import mergeRefs
This expression is not callable.
  Type 'typeof import("/PROJECT_PATH/node_modules/react-merge-refs/dist/index")' has no call signatures.ts(2349)

Here are the version used
        "@floating-ui/react-dom": "^1.0.0",
        "@floating-ui/react-dom-interactions": "^0.10.1",
        "react-merge-refs": "^2.0.1",

Docs use v1 of that package but now it's v2, it's probably a default export problem. Though the releases do say both should work as of 2.0.1, try changing this:
Try import { mergeRefs } from ... instead of import mergeRefs from ...


Wallet Sync Optimization

After bdk sync, our native Wallet-Transactions-Addresses has a high time complexity to sync.
Every transaction is iterated over and each outAddr is attempted to be matched against addresses and updates made accordingly.
This should happen in the background while the wallet continues to be usable.
We should also make the bdkWallet available before sync so that send and recieve can work on a pre-synced wallet. If the utxos used in a tx is valid, send should work completely even if sync is still in progress. Recieve will keep generating addresses from its lastUnused.

Update: dart sync is not causing the delay, its bdk sync. So optimizing List operations is not a major priority.

If wallet is not synced before sending coins, coins do not appear.

@i5hi  utxo list was not being saved in storage


Fix Material-UI example

This example was using a really old version of Material-UI and broke the build. I updated it to use the latest Material-UI version.

Thx @irontitan76
You're absolutely right with the sheets manager. Will change that.

@vuhrmeister Thank you for contributing this. I found it useful in building razzle-examples-with-material-ui-typescript, which is what you did + typescript. (Who knows what the naming convention should be for examples with more than one thing going on?). <3


Only get one day of history for ENT.L symbol

Code:
import yfinance as yf

ticker = yf.Ticker('ENT.L')
hist = ticker.history('1y')
print(hist);

Output:
              Open    High     Low   Close   Volume  Dividends  Stock Splits
Date
2021-01-22  1269.0  1288.5  1249.5  1280.0  2895976          0             0

There are much more candles for ENT.L on yahoo finance website.

Still happening?


Handling Tillage Prescriptions

ADAPT Prescriptions currently require at least 1 product that maps to rate attributes in the data.   In the case of tillage prescriptions, there is only a depth attribute.  Can we simply remove the requirement that Prescriptions always have products and clarify intended usage in the definitions?

Agreement in 16 August 2023 meeting to simply remove the requirement and add instructions in definition.

We removed the concept of Prescriptions when we standardized Operation across document types.  See #122.   Product is omitted when not relevant.


Add entry for rika2mqtt subdomain - Update domains.json

According to the doc, I would like to add a custom domain for my documentation.
domain: rika2mqtt.cookiecode.dev
should point to project sebastienvermeille/rika2mqtt

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


readme on installing dotnet-example seems wrong

@ErikEJ Not sure what is wrong. Could you elaborate on this? Thanks!

The guide to install dotnet-example I'd
dotnet tool restore

@ErikEJ Are you seeing an error when running that command? If so, what is the error?

@ErikEJ It doesn't install it as a global tool. It's part of the project's tool manifest (dotnet-tools.json).

Learned something new today!


better get_var_codes + implications

get_var_codes includes more variables (type, var_src), uses a tsv->sysdata rather than making a new data.frame on each call, does more querying for you, and is integrated into the rest of mda.streams with these new behaviors.

@jread-usgs rocking the Travis & AppVeyor here - want to take a look and merge, please?


[IOS] text color is changed

case.

input mention data first.
send message and reset MentionInput value.
input 'non-english' character. Then, text color is changed.

https://github.com/dabakovich/react-native-controlled-mentions/assets/54658590/7866a65c-cca7-491b-8448-2d75bfde92eb

Hey @Choyeongdeok! Thanks for your feedback. Is the question still actual? If so, please provide me your code to help you resolve this issue.

@dabakovich I think it's a problem of cjk(chinese, japanese, korean) language in IOS text input. It works well in Android.
https://github.com/facebook/react-native/issues/30834

Understood. In any case, I suggest trying the pre-release version 3 of the library, which uses a different text parsing approach. It might help solve this problem for you.


Add indicatorWeight, indicatorPadding to TabBar

Enable customizing the selected tab indicator line's weight and how far it's padded on the left and right vis the selected tab's bounds.
Fixes https://github.com/flutter/flutter/issues/10475

@Hixie

LGTM except for the part where you mutate the state of the painter. All the mutable state should be private, and all the configuration should come in through the constructor.


New favicons (warning: deleted file)

Please note that due to being on a Windows machine, I could not clone the /app/packages/meteor-tooltips/example/packages/lookback:tooltips file. Therefore there is a delete of this file in the commit- it's not intentional and the file should not be deleted. Sorry. :(
@paulproteus

Committed as 018ad0f (with the deletion undone).


CUDNN_STATUS_BAD_PARAM

I'm trying to build a CNN on Chainer.
It seems that Cudnn is not installed correctly, but I can run Tensorflow, Keras and Pytorch successfully on my PC, while failing to run Chainer with Cupy.
I tried all kinds of way from  Cupy Installation Guide, but failed with same mistake.
Can anybody solve this issue? Thanks!

Conditions
Chainer version 4.5.0
CuPy version 4.5.0
OS/Platform Ubantu 16.04
CUDA/cuDNN version 8.0.61/5.1.10

Code
class MODEL(chainer.Chain):

    def __init__(self):
        super(MODEL, self).__init__()
        with self.init_scope():
            self.first=chainer.links.Convolution2D(5, 64, ksize=5, stride=1, pad=2)

    def __call__(self, x: chainer.Variable, test=False):
        x0 = self.first(x)
        return x0

Error Log
Traceback (most recent call last):
  File "main.py", line 207, in <module>
    train()
  File "main.py", line 140, in train
    sr = generator(lr)
  File "main.py", line 86, in __call__
    x0 = self.first(x)
  File "/root/anaconda3/envs/tf/lib/python3.6/site-packages/chainer/links/connection/convolution_2d.py", line 175, in __call__
    groups=self.groups)
  File "/root/anaconda3/envs/tf/lib/python3.6/site-packages/chainer/functions/connection/convolution_2d.py", line 582, in convolution_2d
    y, = fnode.apply(args)
  File "/root/anaconda3/envs/tf/lib/python3.6/site-packages/chainer/function_node.py", line 258, in apply
    outputs = self.forward(in_data)
  File "/root/anaconda3/envs/tf/lib/python3.6/site-packages/chainer/function_node.py", line 367, in forward
    return self.forward_gpu(inputs)
  File "/root/anaconda3/envs/tf/lib/python3.6/site-packages/chainer/functions/connection/convolution_2d.py", line 161, in forward_gpu
    return self._forward_cudnn(x, W, b, y)
  File "/root/anaconda3/envs/tf/lib/python3.6/site-packages/chainer/functions/connection/convolution_2d.py", line 234, in _forward_cudnn
    auto_tune=auto_tune, tensor_core=tensor_core)
  File "cupy/cudnn.pyx", line 820, in cupy.cudnn.convolution_forward
  File "cupy/cudnn.pyx", line 590, in cupy.cudnn._get_algorithm_fwd
  File "cupy/cuda/cudnn.pyx", line 807, in cupy.cuda.cudnn.getConvolutionForwardAlgorithm_v6
  File "cupy/cuda/cudnn.pyx", line 514, in cupy.cuda.cudnn.check_status
cupy.cuda.cudnn.CuDNNError: CUDNN_STATUS_BAD_PARAM

@psychopa4 Could you provide the code giving an actual input to your MODEL object? We need more information. And could you put the result of this following command?
$ !python -c "import chainer; chainer.print_runtime_info()"

Also discussing on https://github.com/cupy/cupy/issues/1691

@psychopa4 Could you provide the code giving an actual input to your MODEL object? We need more information. And could you put the result of this following command?
$ !python -c "import chainer; chainer.print_runtime_info()"

I have changed my cuda and cudnn version to 9.0 and 7.0.4, but still got same mistake.
Should I reinstall cudnn to 7.3?
Platform: Linux-4.15.0-30-generic-x86_64-with-debian-stretch-sid
Chainer: 4.5.0
NumPy: 1.13.1
CuPy:
  CuPy Version          : 4.5.0
  CUDA Root             : /usr/local/cuda-9.0
  CUDA Build Version    : 9000
  CUDA Driver Version   : 9010
  CUDA Runtime Version  : 9000
  cuDNN Build Version   : 7300
  cuDNN Version         : 7004
  NCCL Build Version    : 2213

The code to generate data is as follows:
class ImageDataset(dataset_mixin.DatasetMixin):
    def __init__(self, paths, in_size=32, scale=4):
        self.paths=sorted(glob.glob(join(paths,'*.png')))
        self.in_size=in_size
        self.scale=scale
        self.out_size=in_size*scale

    def __len__(self):
        return len(self.paths)

    def get_example(self, i): #-> Image:
        path = self.paths[i]
        #HR = Image.open(path)
        HR = scipy.misc.imread(path)
        HR = randomcrop(HR, size=self.out_size)

        h,w,c=HR.shape
        hr=Image.fromarray(HR)
        lr=hr.resize((h//self.scale,w//self.scale),Image.BICUBIC)

        LR=np.asarray(lr)
        LR=np.transpose(LR,(2,0,1))
        HR=np.transpose(HR,(2,0,1))
        return LR.astype(np.float32), HR.astype(np.float32)

In general, it returns low-resolution (LR) and corresponding high-resolution (HR) images.
During training, the input x is a tensor with shape like [batch_size, channels, height, width ] ( [16,3,32,32]  for  instance).
Thank you for your enthusiasm!

Thanks for the details.

During training, the input x is a tensor with shape like [batch_size, channels, height, width ] ( [16,3,32,32] for instance).

In that case, in_channels should be set to 3 instead of 5.

        self.first=chainer.links.Convolution2D(5, 64, ksize=5, stride=1, pad=2)

I sent a PR to improve error message in this situation. #5460

Thanks for the details.

During training, the input x is a tensor with shape like [batch_size, channels, height, width ] ( [16,3,32,32] for instance).

In that case, in_channels should be set to 3 instead of 5.

        self.first=chainer.links.Convolution2D(5, 64, ksize=5, stride=1, pad=2)

See https://docs.chainer.org/en/stable/reference/generated/chainer.links.Convolution2D.html for details.
I sent a PR to improve error message in this situation (channel mismatch). #5460

Thank you for your help!
I have solved this issue, however, my network converges to NAN after one step backward update.
May you run this code and see what the problem is, please?
Thanks again for your enthusiasm, really appreciate it!

It seems that the NAN problem is specific to your code.
As we're using Github issues/PRs to manage development tasks, could you use post it on Stack Overflow or Forum?
Thanks for understanding!

It seems that the NAN problem is specific to your code.
As we're using Github issues/PRs to manage development tasks, could you use post it on Stack Overflow or Forum?
Thanks for understanding!

I understand that, thank you!


[Async Storage, Gameplay, Android & iOS]Return to Home doesn't save progress

If you press the "return to home" button in the pause screen, you have to restart the whole session of exercises. It doesn't pick up where you left off. - according to Raymond.

Also tried this, got through 1 x math, 1 x writing, 1 x reading and then returned to home using pause feature during 2nd section of math. Tapped "start exercises" again, got the "You will be doing a mixture of reading, writing, and math " screen so I assume it restarts.

So this isn't a bug, it's a feature that was never implemented; the pause screen is for resuming progress. If we are going to implement this here are some things to consider @jlatour3 :
-If the HomeScreen is in the paused state, does it look any different?
-Does the paused version of the HomeScreen become the normal HomeScreen (i.e ResumeHome becomes StartExerciseHome after some period of inactivity)
-The easiest way to get around all of this is to have the Pause screen only have a resume button.
I don't think I'll have time to do this before Sunday @DeveloperKhan. The easiest way to do this is probably to use canGoBack() + goBack()/pop(2) for the navigator since it would use history. Otherwise we'd have to preserve the progress in local AsyncStorage

Take away the option to "return to home". @shopaniuk, Can we ensure that the screen will stay on resume until the user finishes their session, so that progress is not lost?

Here's my solution

I like it


[Rust]  Add better and faster support for dictionary types

Note: migrated from original JIRA: https://issues.apache.org/jira/browse/ARROW-8464
Usecases: Efficiently process large columns of low cardinality Strings

BatchIterator should accept both DictionaryBatch and RecordBatch
Type Coercion optimizer rule should inject expression for converting dictionary value types to index types (for equality expressions, and IN(values, ...)
Physical expression would lookup index for dictionary values referenced in the query so that at runtime, only indices are being compared per batch

Comment from Andrew Lamb(alamb) @ 2020-10-06T12:25:16.639+0000:
FYI [~andygrove] -- I am doing some part of this in ARROW-10159 -- however, the initial implementation effectively converts DictionaryArray --> PrimitiveArray / StringArray and then uses the existing processing.

To support the actual efficient processing usecase, I think significant work will be needed to add appropriate dictionary support to the arrow compute kernels


Update semgrep.yml

Added tags for all non-experimental languages per https://semgrep.dev/docs/
Added service tag (https://semgrep.dev/manage)
Updated homepage
Updated description based on https://semgrep.dev/docs/

Thanks so much @underyx; great improvements.
I extended the description a little bit to mention that semgrep rules look like code, which is their unique selling proposition in my book. (Took that from the page you linked as well.)


Customized error code not working

Hi,
I created an exception class and the handler class for this exception, assign an status code 461 to it, the code is not a standard http code, also I copied your errorMessages.properties file and added the following:
org.test.exception.MyURLParamException.title=Wrong Query URL Parameter
org.test.exception.MyURLParamException.detail=The parameter you have sent is probably malformed.

The exception class looks like:
public class MyURLParamException extends Exception {
	  private int code;

	  public MyURLParamException(String message) {
	    super(message != null ? message : "MyURLParamException was thrown");
	    this.code = 461;
	  }

	  public int getCode() {
	    return this.code;
	  }
}

I configured the bean as this:
<bean id="httpErrorMessageSource"
	      class="org.springframework.context.support.ReloadableResourceBundleMessageSource">
	        <property name="basenames" value="classpath:/WEB-INF/errorMessages.properties"/>
    		<property name="defaultEncoding" value="UTF-8"/>
    </bean>

then I run the web app to test the error handling, I got
java.lang.IllegalArgumentException: No matching constant for [461]
at org.springframework.http.HttpStatus.valueOf(HttpStatus.java:488)
at org.test.exception.MyURLParamExceptionHandler.handleException(MyURLParamExceptionHandler.java:15)
at org.test.exception.MyURLParamExceptionHandler.handleException(MyURLParamExceptionHandler.java:11)
at cz.jirutka.spring.exhandler.RestHandlerExceptionResolver.handleException(RestHandlerExceptionResolver.java:140)
at cz.jirutka.spring.exhandler.RestHandlerExceptionResolver.doResolveException(RestHandlerExceptionResolver.java:118)
But if I changed status code from 461 to 400, it will work.
Anything I did is not correct?
Thanks
Dave

How does class  org.test.exception.MyURLParamExceptionHandler look like?

Thanks for your response, this is the handler class:
public class MyURLParamExceptionHandler  implements RestExceptionHandler<MyURLParamException, ErrorMessage> {
  @Override
  public ResponseEntity<ErrorMessage> handleException(MyURLParamException exception, HttpServletRequest request) {
    HttpStatus status = HttpStatus.valueOf(exception.getCode());
    ErrorMessage errorMessage = new ErrorMessage();
    errorMessage.setTitle("MyURLParamException");
    errorMessage.setStatus(status);
    errorMessage.setDetail(exception.getMessage());

    return new ResponseEntity<>(errorMessage, status);
  }
}

I hope that you already see where’s the problem…

Oh, HttpStatus.valueOf(461) will throw exception since 461 is not a http status code.

Exactly. It’s a bug in your own code. ErrorMessage.setStatus() accepts both HttpStatus and Integer.

OK, thanks, I changed to this, it works, another problem is it is not using my configuration in my errorMessages.properties file, which I have the values as following:
org.noblis.exception.CWxMapURLParamException.type=461
org.noblis.exception.CWxMapURLParamException.title=Wrong Query URL Parameter
org.noblis.exception.CWxMapURLParamException.detail=The parameter you have sent is probably malformed.
So how do I get these values used with your framework?
  @Override
  public ResponseEntity<ErrorMessage> handleException(MyURLParamException exception, HttpServletRequest request) {
    //HttpStatus status = HttpStatus.valueOf(exception.getCode());
    ErrorMessage errorMessage = new ErrorMessage();
    errorMessage.setTitle("MyURLParamException");
    errorMessage.setStatus(exception.getCode());
    errorMessage.setDetail(exception.getMessage());

    return new ResponseEntity<>(errorMessage, HttpStatus.BAD_REQUEST);
  }

I guess your framework is not automatically doing that, so I need to load the errorMessages.properties resource file and explicitly read the pairs and set them in the handler class, is that right?
Thanks

No, you’re not. Please read carefully the readme, it’s all here. Note missing .properties in the basename.

I followed the readme, you advised to use this:

but I got p is not defined error, so I changed to:

But I still got the same result, the configured values are not used in the error messages:
org.noblis.exception.CWxMapURLParamException.type=461
org.noblis.exception.CWxMapURLParamException.title=Wrong Query URL Parameter
org.noblis.exception.CWxMapURLParamException.detail=The parameter you have sent is probably malformed.
The error messages look like this:

URL parameters are not specified correctly: is

461
MyURLParamException

There’s basename, not basenames in the readme.
The p: is called p-namespace.

I changed to this:

And added this:
xmlns:p="http://www.springframework.org/schema/p"
Retested, it is still the same, any clues?
Thanks

I mean the whole bean looks like this:

</bean>

I mean the whole bean looks like this:

</bean>

Hmm, I’ve missed this before, classpath:/WEB-INF/errorMessages is almost certainly wrong, /WEB-INF is not on the classpath. It’s odd that Spring didn’t throw an exception that it can’t find the file…
Also please learn the basic of Markdown formatting on GitHub, I don’t enjoy fixing every comment of yours…

Sorry, you mean class path problem and file is not loaded in at runtime. got it.
Thanks

I don’t use WEB-INF at all these days, but if I recall it correctly, you can use just /WEB-INF/errorMessages (without classpath: prefix). However my advice is to put this file on the classpath, e.g. src/main/resources if you use Maven structure.

I put under classes as this:
p:basename="classpath:/WEB-INF/classes/errorMessages"
I can see it is in the undeployed folder in tomcat.
but the result is still the same, must be something else.

 If you use standard project structure, then move errorMessages.properties into the directory src/main/resources/org/test/exception and set basename="classpath:org/test/exception/errorMessages". Alternatively, if you have somehow messed project structure, then put it directly to the same directory as MyURLParamExceptionHandler.java.

Hi, sorry to bother, I downloaded the source code and added a lot debug messages, and rebuild a jar and deployed to my local repository, then I changed my pom file to point to that jar, retested with my app, I found out the ErrorMessageRestExceptionHandler has never got called, there is a getMessage() method, which is supposed to get message from the messageSource, anything I did wrong? the following is some of the debug messages I added:
./catalina.out:DDDD RestHandlerExceptionResolver constructor returnTypeMethodParam: org.springframework.core.MethodParameter@850e6f61
./catalina.out:DDDD RestHandlerExceptionResolver build contentNegotiationManager: null
./catalina.out:DDDD RestHandlerExceptionResolver build defaultContentType: application/json
./catalina.out:DDDD RestHandlerExceptionResolver build resolver: cz.jirutka.spring.exhandler.RestHandlerExceptionResolver@59233c53
./catalina.out:DDDD RestHandlerExceptionResolver build withDefaultMessageSource: true
./catalina.out:DDDD RestHandlerExceptionResolver build messageSource: org.springframework.context.support.ReloadableResourceBundleMessageSource: basenames=[classpath:/org/noblis/exception/errorMessages]
./catalina.out:DDDD RestHandlerExceptionResolver build messageSource: org.springframework.context.support.ReloadableResourceBundleMessageSource: basenames=[classpath:/org/noblis/exception/errorMessages]
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@5d6649b5
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@453a9c2b
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@78aa8f9a
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.HttpMediaTypeNotSupportedExceptionHandler@38abe578
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.HttpRequestMethodNotSupportedExceptionHandler@576f3839
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@46a52d3b
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@6a5fac2a
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@d38043c
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@23b84b52
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.MethodArgumentNotValidExceptionHandler@4c1b66eb
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@5bb2bdbe
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.NoSuchRequestHandlingMethodExceptionHandler@16e64b10
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@77657045
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@605e6c7c
./catalina.out:DDDD RestHandlerExceptionResolver build handler: org.noblis.exception.RSQLExceptionHandler@117876fd
./catalina.out:DDDD RestHandlerExceptionResolver build handler: cz.jirutka.spring.exhandler.handlers.ErrorMessageRestExceptionHandler@2ecf7284
./catalina.out:DDDD RestHandlerExceptionResolver build handler: org.noblis.exception.MyURLParamExceptionHandler@2caaf2
./catalina.out:DDDD RestHandlerExceptionResolver constructor returnTypeMethodParam: org.springframework.core.MethodParameter@850e6f61
./catalina.out:DDDD RestHandlerExceptionResolver build contentNegotiationManager: null
./catalina.out:DDDD RestHandlerExceptionResolver build defaultContentType: application/json
./catalina.out:DDDD RestHandlerExceptionResolver build resolver: cz.jirutka.spring.exhandler.RestHandlerExceptionResolver@a551286
./catalina.out:DDDD RestHandlerExceptionResolver resolveExceptionHandler clazz: org.noblis.exception.MyURLParamException
./catalina.out:DDDD RestHandlerExceptionResolver handleException: org.noblis.exception.MyURLParamException
./catalina.out:DDDD RestHandlerExceptionResolver processResponse entity: <400 Bad Request,ErrorMessage(type=null, title=MyURLParamException, status=461, instance=null),{}>
Thanks

Finally I got it. I just need to write  my handler class to extend from ErrorMessageRestExceptionHandler, since this class loads error messages from the resource file.
Thanks for your help.

Aha, I’ve forgotten your original objective meanwhile. If you wanna fill the ErrorMessage from the properties file, then why you even defines your own handler? Moreover, you’re just setting the ErrorMessage properties based on the given exception, so it’s totally unnecessary. The main benefit of this library is support for simple mapping of exceptions without writing exception handlers.
You can do this instead:
<bean id="restExceptionResolver"
      class="cz.jirutka.spring.exhandler.RestHandlerExceptionResolverFactoryBean">
    <property name="messageSource" ref="httpErrorMessageSource" />
    <property name="defaultContentType" value="application/xml" />
    <property name="exceptionHandlers">
        <map>
            <entry key="org.noblis.exception.CWxMapURLParamException" value="461" />
        </map>
    </property>
</bean>

org.noblis.exception.CWxMapURLParamException.type=461
org.noblis.exception.CWxMapURLParamException.title=MyURLParamException
org.noblis.exception.CWxMapURLParamException.detail=URL parameters are not specified correctly: #{ex.message}


"flutter create --with-driver-test" fails to create project because of version solving

Steps to Reproduce

Run flutter create --with-driver-test version_solving_failure.

Expected results:
The project gets created and fully initialized.
Actual results:
During the creation, flutter pub getfails:
version_solving_failure/test_driver/main_test.dart (created)
Because test >=1.6.2 <1.6.4 depends on test_api 0.2.5 and every version of flutter_driver from sdk depends on test_api 0.2.11, test >=1.6.2 <1.6.4 is incompatible with flutter_driver from sdk.
So, because version_solving_failure depends on both flutter_driver any from sdk and test 1.6.2, version solving failed.
Running "flutter pub get" in version_solving_failure...
pub get failed (1; So, because version_solving_failure depends on both flutter_driver any from sdk and test 1.6.2, version solving failed.)

Manually running flutter pub get in the app directory produces the same error.

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.12.13+hotfix.7, on Mac OS X 10.15.3 19D76, locale en-GB)
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
    ✗ Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
[✓] Android Studio (version 3.5)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.3.2)
[✓] VS Code (version 1.41.1)
[✓] Connected device (1 available)

! Doctor found issues in 1 category.

Hi @ksilz
Can you please provide flutter run --verbose?
Thank you

I did. Here's the relevant section from my pubspec.yaml now:
version: 1.0.0+1

environment:
  sdk: ">=2.1.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^0.1.2
  test_api: 0.2.14

dev_dependencies:
  flutter_test:
    sdk: flutter
  flutter_driver:
    sdk: flutter
  test: 1.6.2

And I attached the output of running flutter run --verbose with that new configuration.
flutter-run.txt

Hi @ksilz
You can you this combination:
dependencies:
  flutter:
    sdk: flutter
  cupertino_icons: ^0.1.2
  test_api: 0.2.11

dev_dependencies:
  flutter_test:
    sdk: flutter
  flutter_driver:
    sdk: flutter
  test: 1.9.4

I don't have the conflict with these versions, but I'm not sure about the rest of your dependencies (if you have more the in your sample)
Is it solved?

Yep, with your config it works just fine.
So Flutter "just" creates the pubspec.yaml out of the box when you use --with-driver-test?

Does not reproduce on 9cdc3f0f31, closing


Remove ShowerMaxEstimator

Unused
Has a test that does not actually anything (just that it returns something in meters)
Contains magic numbers

yes, remove it.


Fix sha256 capitalization

I had an 'unknown digest algorithm' error in Cloudflare Workers, and this fixed it (maybe related with a different implementation of the crypto library for his part?)


Fix references to eslint-plugin-i in README.md, npm alias

fixes #253 (corrects the aliased npm install command)
fixes #281

eslint-plugin-import-x is not an alias anymore, so the alias installation is only for eslint-plugin-i.


[test] Implement rom_e2e_asm_watchdog

Fixes #14491

It looks like //sw/device/tests:aon_timer_irq_test_fpga_cw310_rom is failing now that we've enabled NMIs from the watchdog timer. I may need to define ottf_external_nmi_handler() and clear the NMI state.

tests:aon_timer_irq_test_fpga_cw310_rom

Feel free to disable it for rom. I enabled as many functests as possible but they shouldn't block our progress. We can come back and fix them later.

@alphan, I managed to implement both cases of the test as we discussed. PTAL (on Monday) and have a great weekend!

I was holding off on these tests because they started failing on my local branch. It turns out that's related to a hardware change — see #16067.  I just rebased on the old, good commit (f3da3f66ef03e2485f51e7626d5dc46a93273ac5) and verified that the new tests pass with the following command.
./bazelisk.sh test --define bitstream=vivado --test_output=streamed --cache_test_results=no //sw/device/silicon_creator/rom/e2e:rom_e2e_asm_watchdog

Let's just rebase this PR on master and merge it. The new tests will fail until #16067 is resolved. If we merge this PR first, it will obsolete #16077.

Looks like two pre-existing FPGA tests are failing, which I'm guessing is related to the NMI change in rom_start.S [log].

//sw/device/tests:aon_timer_smoketest_fpga_cw310_rom
//sw/device/tests:flash_ctrl_idle_low_power_test_fpga_cw310_rom

@alphan do you think I should disable these tests and come back to it later, like we already did with //sw/device/tests:aon_timer_irq_test?

@dmcardle SGTM, CI runs those tests with test_rom anyway. Thanks!


Add TAB completion for console

Describe the bug
When i start hsf and use command use  + double tabs, it shows all files in current folder. It suppose to suggest all modules instead
To Reproduce
Steps to reproduce the behavior:

Start hsf (version 2.0) as root sudo hsf. Accept Term and build db
use  double tab
See output
Expected behavior
It should show modules instead. Like "exploit", "scanner", ...

Screenshots

Desktop (please complete the following information):

OS: Parrot OS
Version 4.11.2 amd64. Python3.9

Uhm after checking i think current framework doesn't support auto completion?

@dmknght Yeah, HatSploit does not support this feature, I mean it has autocompletion for commands history, but not for modules.

@enty8080 IMO autocompletion is a very huge feature for UX. May i know why HatSploit doesn't support it?

@dmknght I do not know how to implement it, but maybe at HatSploit 3 it will be added

@dmknght I do not know how to implement it, but maybe at HatSploit 3 it will be added

Oh i did it before. It is not very hard but you'll need a very long module and oop. I reused the code of routersploit and it worked very well. Well it would be great if i can create pull request for HatSploit but i'm afraid i don't have enough time for an other project :(

@dmknght #80 - This patch should implement what you've requested.


Populate denormalized fields for change events

lol. thought this said "demoralized" instead of "denormalized"


error on build project

When I try to build the project (0.3) for IntelliJ , Gradle give me this error:
startup failed:
build file '/home/andrea/tmp/zwave4j-master/build.gradle': 69: unable to resolve class TargetPlatformConfiguration
@ line 69, column 72.
TargetPlatformConfiguration() {
I 'use gradle 2.4 and the commeand I've used is: gradle idea
The same problem there is also when I try to build project

Use Gradle 1.10-1.12


Remove unsupported solution configs, make get_testwinrt curdir-independent, assert TestWinRT requirement

fixes #692

I see we are removing Arm and Arm64 -- when I was doing nuget work I know I made sure we included these host dlls. But only if they exist, which made me think eventually we will make these to support arm architectures. I see we are removing those platforms, is the plan to add support for arm{64} in the future or are we sticking with x86/x64?

I see we are removing Arm and Arm64 -- when I was doing nuget work I know I made sure we included these host dlls. But only if they exist, which made me think eventually we will make these to support arm architectures. I see we are removing those platforms, is the plan to add support for arm{64} in the future or are we sticking with x86/x64?

I see we are removing Arm and Arm64 -- when I was doing nuget work I know I made sure we included these host dlls. But only if they exist, which made me think eventually we will make these to support arm architectures. I see we are removing those platforms, is the plan to add support for arm{64} in the future or are we sticking with x86/x64?

No plans to ever build a cswinrt.exe (or other native components) for ARM32/64

I see we are removing Arm and Arm64 -- when I was doing nuget work I know I made sure we included these host dlls. But only if they exist, which made me think eventually we will make these to support arm architectures. I see we are removing those platforms, is the plan to add support for arm{64} in the future or are we sticking with x86/x64?

No plans to ever build a cswinrt.exe (or other native components) for ARM32/64

I see we are removing Arm and Arm64 -- when I was doing nuget work I know I made sure we included these host dlls. But only if they exist, which made me think eventually we will make these to support arm architectures. I see we are removing those platforms, is the plan to add support for arm{64} in the future or are we sticking with x86/x64?

No plans to ever build a cswinrt.exe (or other native components) for ARM32/64. Removing those configs, and Any CPU, prevents confusion for devs when they open the sln

Does this mean we should update our nuget pipeline / build cmd to not consider winrt.host dlls for arm architectures?

I see we are removing Arm and Arm64 -- when I was doing nuget work I know I made sure we included these host dlls. But only if they exist, which made me think eventually we will make these to support arm architectures. I see we are removing those platforms, is the plan to add support for arm{64} in the future or are we sticking with x86/x64?

No plans to ever build a cswinrt.exe (or other native components) for ARM32/64. Removing those configs, and Any CPU, prevents confusion for devs when they open the sln

Does this mean we should update our nuget pipeline / build cmd to not consider winrt.host dlls for arm architectures?

Opened #708 to track ARM64 support for WinRT.Host.

Opened #708 to track ARM64 support for WinRT.Host.


Update anywidget

Some really interesting new developments: https://github.com/manzt/anywidget/pull/395

Closed by #344


refresh login should check if the current session id is good

Lets check if the current session id is good, and if not, refresh the session id with a new login id
https://github.com/HewlettPackard/oneview-golang/blob/master/icsp/auth.go#L74
https://github.com/HewlettPackard/oneview-golang/blob/master/ov/auth.go#L76

This causes another side-effect in that docker-machine ls stops working after the original tokens are expired.

@miqui been hacking today at google cafe in london, this is first half of the fix for icsp.  Will work on ov half if you don't beat me to it first :dancers:

@miqui ok, i was able to quickly update the ov part too... so just need to try this out with some new exe's


Feature Request: add a way for minions summoned creatures to be traceable to the source

What id like to see if any creature is summoned from a spell and becomes a minion. a way of tracing that back to the person who casted the spell. so that if there is alot of creatures doing the same thing you can link back to each caster. and have an indication of when these summoned creatures/minions are supposed to go on the combat tracker.

Traceable implies there being something to trace


Support render .ipynb file as a new feature

This is done through 2 main steps:

Adding new extension "ipynb" as external handler. This also requires
jupyter and its converter, but can gracefully fall back to raw content.
Add an alternative parser, only for .ipynb type, because .ipynb file
is one whole JSON object. That will have different proccessing than parsing
JSON frontmatter.

The frontmatter for .ipynb is expected in the "metadata" of the notebook:
{
  "metadat":{
    "frontmatter": {
      "title": ""
    }
  }
}

Hi @bep , I use an old thread to discuss about the change I propose in this PR:
https://discourse.gohugo.io/t/ipython-notebook-support/3211/3
I am eager to implement the change that we are discussing. Would you join us and give us some feedback?

@bep

but as I have no idea what ipynb is

It's a jupyter notebook file. You can mix computations and markdown, and the outputs are saved in the file. See, for example, this article in Nature, which notes that GitHub has more than 2.5 million ipynb files. GitHub, GitLab, Nickola, Indico, and others all support rendering this file format. Not sure if this PR is the right way to do it, but it would be a great way to write blog posts for anyone in the sciences!

@bep
Thanks for creating this awesome one-stop-shop to easily write, generate and publish static websites - I enjoy using it for my website.
I am writing to ask you to add ipynb as a first class citizen to hugo. I would be happy to resume work on this branch and settle merge conflicts, where necessary.
I would like to use it and I think having native support will solidify hugo's popularity for people, who primarily work in notebooks.
Jupyter is the literate programming paradigm for the scientific community, which is increasingly growing in popularity in academia, machine learning, data science and data exploration.
There are several plugins written by keen users of jupyter and Hugo, so we are not the only people keen on this feature:
https://www.vincent-lunot.com/post/toward-publishing-jupyter-notebooks-with-hugo/
https://github.com/knowsuchagency/hugo_jupyter
https://pypi.org/project/nbhugoexporter/
I am a user of org-mode and I think that since Hugo supports org-mode, it only makes sense to support jupyter as well.

What's the update on this? It would be really cool if this was added.

any update on this? I like this feature as I have many notes written in .jpynb


Eleborate on Migration Scenarios from V1 to V2

Data Factory V1 had extensiv support for trigger dependencies from the beginning.
Could you elaborate in the documentation about migration screnarios?
Will there be an updated ADF Migration tool which will migrate V1 dependencies to V2 ADF?
How is it regarding self dependecy and the Initial Start of the Trigger (the first self dependey, that das not have a previous run because its outside of the windows - handled automatically?)

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

ID: e90a49ad-b872-757e-4f63-7fac92327d2b
Version Independent ID: 60e35bd6-449c-20f7-245f-f55cec44546a
Content: Create tumbling window trigger dependencies in Azure Data Factory
Content Source: articles/data-factory/tumbling-window-trigger-dependency.md
Service: data-factory
GitHub Login: @douglaslMS
Microsoft Alias: douglasl

@hmayer1980 Thanks for the feedback. We are currently investigating into the issue and will update you shortly.

@hmayer1980  I have assigned the issue to the content author to review further and update the document as appropriate.

@hmayer1980 Did you try the migration tool for migrating ADF V1 Pipeline to V2?
ADF v2 offers a tumbling window trigger that can be triggered back in time. Could you clarify the exact requirement?

@hmayer1980 Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.

I am asking about a Migration Scenario from ADF V1 which did Support Slice Dependency via..
            "inputs": [
                {
                    "name": "ADFXXXX"
                },
                {
                    "name": "ADFyyyyy",
                    "startTime": "Time.AddHours(SliceStart, -3)",
                    "endTime": "Time.AddHours(SliceEnd,  -3)"
                }

and the new documentation about ADF V2 Trigger Dependencies (https://docs.microsoft.com/en-us/azure/data-factory/tumbling-window-trigger-dependency#dependency-offset)
You could add an link to the migration tool (which currently does not support migrating a dataset with self / other dependencies at different times)...
And you could elaborate on (https://docs.microsoft.com/en-us/azure/data-factory/tumbling-window-trigger-dependency#dependency-offset)
If there is a self dependency - lets say to the prev run - how does it behave at the initial deployment (Trigger Start 2019-01-01) with a self dependency 24h bevor (the run before) - if it is before the start time. In ADF V1 you hat so set this slice to "run" via powershell, otherwiese the ADF V1 would not start. Does ADF V2 behave the same? Or does it realize, hat if the self denepdeny references a point in time before the trigger start, it should be ignored?

@nabhishek Could you please look into this issue?


No feedback when logged out and trying to start the timer from the context menu

Expected behavior
Something should tell the user to log in.
Actual behavior
Nothing happens
Steps to reproduce the behavior

Log out of button
Right click -> start timer
...

Olden UI, closing 🔨


Implement Hard Level AI

Would be nice if AI choose not the longest possible sequence but a subsequence (I think it is partially implemented in ai_imp file)
Not an option to block opponent pieces, no need to avoid blocking, it is rare case in this game.
Good option would be to follow some ladder skeleton: checkpoints implementation but it doesn't seem to work perfect (check checkpoints file). Implemented two different approaches, but looks like they are not finished, only work in magenta implementation but not in green implementation (some other bug screwing it up? don't know really)

https://github.com/Patlatus/Pygame-Corners/tree/ai_experiments


Travis CI failed

Trace issue related to Regression: Chrome fails to start due to incorrect default chrome-sandbox permissions

Related to https://blog.travis-ci.com/2017-12-12-new-trusty-images-q4-launch


[nginx-ingress-controller] Enable configuration to disable http2

fixes #960
Latest safari have issues with http2 https://trac.nginx.org/nginx/ticket/979

ping @bprashanth

lgtm, but you have merge conflicts

done

LGTM


FederatedStore FederatedViewValidator Bug

Incorrect view validation occurs when a FederatedStore has getElements with a view that requests elements where the view is BOTH ENTITIES or BOTH ELEMENTS and they exist on separate sub graphs.
The bug does not occur if the different edges and entities exist on different graphs.

Merged into develop


Upgrade with master

I believe the branch that is now used is allow-different-serializer-for-metadata


fixPayChanV1

This commit introduces a new fix amendment (fixPayChanV1) which prevents the creation of new PaymentChannelCreate transaction with a CancelAfter time less than the current ledger time. It piggy backs off of fix1571.
Once the amendment is activated, creating a new PaymentChannel will require that if you specify the CancelAfter time/value, that value must be greater than or equal to the current ledger time.
Currently users can create a payment channel where the CancelAfter time is before the current ledger time. This results in the payment channel being immediately closed on the next PaymentChannel transaction.
I did swap tecNO_PERMISSION  for temBAD_EXPIRATION as I believe that is a more descriptive error message.

Although the issue can be resolved with an amendment, it's not necessarily a flaw at the protocol level. Instead, it's a potential user error. While many such errors can be detected, catching every single one might not be feasible. A strong case can be made for identifying these mistakes through checks in client libraries or at the API level. With this approach, it wouldn't be necessary to make changes that impact transaction processing. For instance, xrplcluster prevents transactions with extremely high fees, or payments to known scammers. Instead of trying to address every possible user error within the protocol, it's more practical to handle these in a layer in front of rippled. One of the benefits is that the fix can be rolled out immediately, permissionlessly and without total consensus.

You did the same fix for Escrow. Actually the fix you did was >= to >.

Important notes:

I don't see anything wrong with this amendment. It fixes a problem (even if it's a small problem with satisfactory alternative workarounds). I'm very happy to see developers open small fix PRs like this one.
My comment offered a holistic view, considering both the benefits of the amendment and the practicality and immediacy of other solutions.

If @dangell7 (or anyone else in the community) would like this amendment, please proceed. Reopen this PR or open a new one. This protocol change can happen in parallel, and in addition to, any fixes at the client or API service layers.

@dangell7 it looks like tests are failing on this PR.

@dangell7 it looks like tests are failing on this PR.

develop was actually failing so this should pass now. If not I will fix whatever I need to assuming we want to move forward with this.

@dangell7 - confirming - is this ready for review? (or are there any additional changes you're planning to make?)

@intelliot Ready for review

@dangell7 - do you have an opinion on whether this should be considered for the next release (2.0.0) or if it's ok for it to wait for 2024?
(The first release in 2024 is expected for Jan or Feb.)

@intelliot I have no opinion. 2024 is fine. I don't think many use paychan but its nice that this is fixed at some point.

bug fix. Should not require perf signoff. cc @sophiax851

@dangell7

This PR has conflicts that must be resolved.
After resolving, please confirm that you still think this PR is ready to merge.


Credential Composer plugin

A definition for a new CredentialComposer plugin that can be used to customize CA/X509-SVID/JWT-SVID credentials.
See SPIRE Issue 3253 for the justification.

Sorry, fixed up DCO and force pushed. Hopefully that doesn't mess things up :)


SyncSect Invalid value authorization.openshift.io/v1

We are using hive in order to provision dev clusters on demmand, specifically:

branch: master
commit: f95446d21ae47da6689f8de91c9b1463955223b7

We are trying to deploy application-monitoring-operator using a SyncSet object, including for example alertmanager-clusterrole_binding.yaml (whose apiVersion is authorization.openshift.io/v1), and we got some errors telling that there are some an invalid values:
$ kubectl create -f application-monitoring-operator-syncset.yaml
The SyncSet "dev-eng-application-monitoring-operator" is invalid:
* spec.resources[2].APIVersion: Invalid value: "authorization.openshift.io/v1": must use kubernetes group for this resource kind
* spec.resources[3].APIVersion: Invalid value: "authorization.openshift.io/v1": must use kubernetes group for this resource kind
* spec.resources[4].APIVersion: Invalid value: "authorization.openshift.io/v1": must use kubernetes group for this resource kind
* spec.resources[10].APIVersion: Invalid value: "authorization.openshift.io/v1": must use kubernetes group for this resource kind

We can see on hiveadmission pod logs:
time="2020-03-02T16:51:26Z" level=info msg="Validating request" group=hive.openshift.io method=Validate operation=CREATE resource=syncsets version=v1
time="2020-03-02T16:51:26Z" level=info msg="SyncSet.hive.openshift.io \"dev-eng-application-monitoring-operator\" is invalid: [spec.resources[2].APIVersion: Invalid value: \"authorization.openshift.io/v1\": must use kubernetes group for this resource kind, spec.resources[3].APIVersion: Invalid value: \"authorization.openshift.io/v1\": must use kubernetes group for this resource kind, spec.resources[4].APIVersion: Invalid value: \"authorization.openshift.io/v1\": must use kubernetes group for this resource kind, spec.resources[10].APIVersion: Invalid value: \"authorization.openshift.io/v1\": must use kubernetes group for this resource kind]" group=hive.openshift.io method=validateCreate object.Name=dev-eng-application-monitoring-operator operation=CREATE resource=syncsets version=v1

Can you confirm if this is the expected behaviour?
If we deploy the object directly on the Openshift cluster provisioned by hive (without using hive SyncSet), it works as expected:
$ oc apply -f deploy/cluster-roles/alertmanager-clusterrole_binding.yaml
clusterrolebinding.authorization.openshift.io/alertmanager-application-monitoring created

Yes I believe this is intentional and expected, you can see the relevant commit here: https://github.com/openshift/hive/commit/a1dbceb6e706369db1cf500d55f4cf17186f22d5
Trying to recall specifics but I believe these types are quite strange in which they're almost aliases between the k8s type and the openshift type. Interacting with the openshift types was posing some pretty big problems but I can't remember the specifics. @abutcher do you?
Can you switch your yaml to use the kube apigroups for these types instead of openshift?

https://issues.redhat.com/browse/CO-532 is the bug that prompted Hive to reject resources in authorization.openshift.io/v1. The short story is that you should not be using authorization.openshift.io/v1 and should use rbac.authorization.k8s.io instead.

Thank you guys, after switching to k8s api, it is working now, closing the issue!


Color Sheet disableSwitchColorView icon remains

Describe the bug
On the color sheet, disableSwitchColorView() disables the ability to switch to the custom color view. However, the icon still remains - it just doesn't do anything.
Library Version:
1.1.0
To Reproduce
Steps to reproduce the behavior:

Add disableSwitchColorView() to the code
The icon to switch the color view remains

Expected behavior
The icon should go away if disableSwitchColorView() is enabled.
Screenshots
I've attached a video showing this bug in the sample app
Sample app

Thank you a lot. I'm sorry for the 2 issues you have reported - I've rushed a bit with the release.

Thank you a lot. I'm sorry for the 2 issues you have reported - I've rushed a bit with the release.


Support mappings

Just as nilportugues/sql-repository does.
Exact same thing.

https://github.com/PHPRepository/php-mongodb-repository/commit/576eb369a5a5382512a5712127c56c27e7953fcd


Issue with contentRect.bounds

Hi, we're using react-measure to determine the size of an image we want to render. It seems that after updating to the latest release (v2.2.6), contentRect.bounds.width and contentRect.bounds.height are consistently returning 0, which prevents our component from loading.
Here is a comparison showing the component using v2.2.6 versus v2.2.5. And some additional context on how we're using contentRect.
Is this intended behavior or possibly a regression introduced by changes in the last release? Any advice would be appreciated. Thanks!

Ah, sorry about that! It looks like a regression from here 😞. I'll work on getting a fix together. Just want to make sure you aren't relying on onResize and only what comes back in the child function?

Thanks for quick response! And yes I believe that is correct.

I was about to post the same issue. The data is passed properly to bounds when the windows size changes. After downgrade works.
We do not use onResize and simply just render <Measure> and use ref={measureRef} with contentReact for different components

Should be fixed under 2.3.0 🤞. Thanks for letting me know about the issue and please feel free to reopen if you see any similar problems.


Add first two modules (diamond and kaiju), missing docs and tests

This is more of a PoC to draft rough structure. Subject to change during development.
~Note input validation not working correctly, as only need to require one of either fasta_dna or fasta_aa, or both. But if neither supplied then it just runs with empty lists :(~ now working thanks to @mirpedrol :D
Adds:

Input schema sheet draft
DIAMOND
Kaiju

database building and nf-tests :)
PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/createtaxdb branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

I know this is very early work. I'm fine with merging as is but I had two concerns that I propose to change in the long run:

Do you mind if you give me an approval then I can merge this in and follow up depending on feedback of my status below? Then can start doing more parallel PRs to add the below

There are two separate input options for nodesdmp and namesdmp. I would expect a path/tar of a directory with those files (possibly containing more of the dump files).

Is expectation this based purely on ncbi taxdump files? I currently had kept it separate because if people want to make custom databases that may include customised dump files whereby I don't see why one would necessarily re-tar...
That said it is reasonable to expect someone may want to do that... I may consider adding it as an option (if one gives the taxdump as tar it'll auto extract - but I vaguely remember plucking specific files from a directory is not directly trivial with Nxf).
But I would make this a separate issue as a separate functionality (including maybe auto downloading taxdump files, but I'm not sure yet how to do this properly e.g. with gtdbtk taxdump stuff etc)

I would create one subworkflow per tool that the main createtaxdb workflow calls to.

Yes that's my plan for multi-module database construction commands (e.g. kraken2), or do you have a motivation to do this for single build modules too?

Do you mind if you give me an approval then I can merge this in and follow up depending on feedback of my status below?

I didn't approve due to the open comments. Do you plan to address them or see them as irrelevant?

Is expectation this based purely on ncbi taxdump files? I currently had kept it separate because if people want to make custom databases that may include customised dump files whereby I don't see why one would necessarily re-tar...

That said it is reasonable to expect someone may want to do that... I may consider adding it as an option (if one gives the taxdump as tar it'll auto extract - but I vaguely remember plucking specific files from a directory is not directly trivial with Nxf).

My expectation is based on taxonkit usage, yes. With a custom taxonomy I would just pass a path to a directory and then expect {dir}/nodes.dmp and {dir}/names.dmp to exist.

Yes that's my plan for multi-module database construction commands (e.g. kraken2), or do you have a motivation to do this for single build modules too?

No, I don't see a reason for single modules. I didn't read the file properly and somehow thought the input channel transformation was Kaiju-specific.

Do you mind if you give me an approval then I can merge this in and follow up depending on feedback of my status below?

I didn't approve due to the open comments. Do you plan to address them or see them as irrelevant?

Hm, I thought I had addressed them 🤔, but I see now there is no commit. Maybe I didn't push...

Is expectation this based purely on ncbi taxdump files? I currently had kept it separate because if people want to make custom databases that may include customised dump files whereby I don't see why one would necessarily re-tar...

That said it is reasonable to expect someone may want to do that... I may consider adding it as an option (if one gives the taxdump as tar it'll auto extract - but I vaguely remember plucking specific files from a directory is not directly trivial with Nxf).

My expectation is based on taxonkit usage, yes. With a custom taxonomy I would just pass a path to a directory and then expect {dir}/nodes.dmp and {dir}/names.dmp to exist.

Hrm, ok. I'll maybe look more in detail to taxonkit and try and use that as a structure to follow. But I think I would still do that as a follow up PR (because it should be quite straightforward to just change how those files are picked up and passed to the module).

Yes that's my plan for multi-module database construction commands (e.g. kraken2), or do you have a motivation to do this for single build modules too?

No, I don't see a reason for single modules. I didn't read the file properly and somehow thought the input channel transformation was Kaiju-specific.

Ah ok! Then I leave that bit as is for now at least.

OK thank you for the reviews @maxulysse @Joon-Klaps @Midnighter !
I've addressed all the suggestions now (except for the larger one from @Midnighter regarding taxdump, but I'll do that as a follow up), so I will merge so others can start to get involved, and other unaddressed changes can be dealt with in a follow up :)


Implements read support for appending symbols to the current local symbol table.

This is achieved by cloning the previous local symbol table (if necessary) and appending new symbols to the clone.
Also makes a few improvements around symbol table handling:

Consolidates the text and binary readers' potential symbol table processing logic into _ion_reader_process_possible_symbol_table. Previously, the two readers used different logic, which led to the binary reader not supporting adding shared symbol tables to the catalog and using the incorrect local symbol table owning pool.
In accordance with the spec, any value in the symbols list that is null or is not a string is treated as a valid SID mapping with unknown text.
A symbol table struct must not declare more than one symbols or imports list. Allowing this could lead to inconsistent mappings since struct fields are unordered.

Testing: passes all existing tests as well as the new LST-append and null slot vectors added in https://github.com/amzn/ion-tests/pull/35 . Once that PR is merged, ion-c's ion-tests submodule will be pointed to the latest revision.

Merged in #92 ; leaving open for a while in case there is additional feedback.


readonly css not apply fix

Proposed changes
#6081
Screenshots (if appropriate) or codepen:
(https://github.com/Dogfalo/materialize/blob/master/CONTRIBUTING.md)**. -->

Types of changes

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

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

You have to set the readonly attribute to the readonly value in your project.


FIFOQueue as a part of custom layer

Hello,
I need a queue as a part of my custom input layer in Transformer for Short-term memory purposes. I need to store the coming features "in first in first out" manner and remove the oldies features when it's full and clean all content if it's needed. I see here something like that for working with Tensor, but is it a good choice for me and contains all that I need? Is the FIFOQueue faster than using Python's deque or Numpy based queue directly in a custom layer?
Compare Python's deque collection with TF's FIFOQueue:

Python
TensorFlow

deque(maxlen=max_size)
tf.queue.FIFOQueue(capacity, dtypes, shapes=None, names=None, shared_name=None)

append(x)# if it's full discard from the left end
enqueue(vals)

clear()
???

len(queue)
size()

pop()
dequeue()

queue[i]
???

Or alternatively do it with Numpy like:
queue = np.zeros((maxlen, features)) # my queue
x = np.random.normal(size=features) # new features

# append()
queue = np.concatenate([queue[:-1], x[np.newaxis, :]])

# predict
y = model(queue[np.newaxis, :, :])

With Python's timeit I get a benchmark on 100000 cycles with maxlen=1000 and features=6:

Method
Time

Numpy
0.1765849579999994  s

deque
12.360834958000002  s

Thanks, have a nice day.

@fchollet

Thanks for opening this feature request. Development of keras moved to separate repository https://github.com/keras-team/keras/issues
Please post this issue on keras-team/keras repo.
To know more see;
https://discuss.tensorflow.org/t/keras-project-moved-to-new-repository-in-https-github-com-keras-team-keras/1999
Thank you!

Okej, I create a feature request there: https://github.com/keras-team/keras/issues/16015.
Have a nice day.


NUnit/Automated test(s)

Please add and document JUnit tests for use in our daily builds.
Documentation can look similar to this repo(s):
https://github.com/Esri/solutions-widgets-wpf#getting-started-with-the-solution-widgets-wpf
https://github.com/Esri/joint-military-symbology-xml#getting-started-with-the-solution

Looks like (for this repo at least) these are here: https://github.com/Esri/coordinate-conversion-addin-dotnet/tree/dev/source/CoordinateConversion/CoordinateConversionLibrary.Tests
And just need some doc so Avi knows how to run with the build

After a successful build of Coordinate Conversion solution, you should be able to run the following command for unit testing:
MSTest /testcontainer:CoordinateConversionLibrary.Tests.dll
This is usually done from a Developer command prompt, shortcut is found in C:\Program Files (x86)\Microsoft Visual Studio 12.0\Common7\Tools\Shortcuts

@ACueva we need unit tests added to Jenkins.  I've updated the readme to have unit test instructions also.

@ACueva @csmoore anything else needed for unit testing to work with Jenkins?

Just where to find the tests and how to run & I see in the readme: https://github.com/Esri/coordinate-conversion-addin-dotnet#development-environment
It looks like this isn't calling/triggering any SDK calls, ex. AOInitialize (ArcObjects) or Host.Initialize (Pro SDK) so we shouldn't have any of the x86/x64 configuration setting/issues so should be straightforward - @ACueva - just ping me if you want any help but it should be the same as when you call msbuild (vcvars.bat + MSTest.exe )

Unit tests were added for visibility-addin-dotnet
After a successful build of Visibility solution, you should be able to run the following command for unit testing:
MSTest /testcontainer:ArcMapAddinVisibility.Tests.dll *

Visibility and Coordinate tool tests implemented and passed:
http://:8080/job/military-tools-for-arcgis-build/374/console
@csmoore please verify.

One final additon, distance and direction

Sorry, didn't notice this was assigned to me - verified


api: port_forward.host docs

Hello @nicks,
Please review the following commits I made in branch maiamcc/port-forward-host:
480dfbe45808aa3607911ae5fb3d2d2ec3025c6e (2020-12-07 15:57:37 -0500)
api: port_forward.host docs
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

Docs for https://github.com/tilt-dev/tilt/pull/3971


Missing configuration option for URI length (414 Request-URI Too Large)

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
2.8
Current Behavior
When I call the Kong gateway with a long URI, I get an I get an 414 Request-URI Too Large error. Basically, this has also been the expected behavior. However, I would now like to adjust the maximum length of the URI for a specific use case or API, but cannot find a configuration option for this. Is there none for this so far? Issue #1266  does not really explain a solution to extend the length.
Expected Behavior
I would expect this to be configurable.
Steps To Reproduce
Call the gateway with an URI with e.g. 10000 characters,
Anything else?
No response

Please try to configure nginx_http_client_header_buffer_size and nginx_http_large_client_header_buffers in kong.conf.
They are the same as the Nginx configuration directives client_header_buffer_size and large_client_header_buffers.

Thank you very much for the quick reply. I didn't know that Kong was running an nginx in the backend.


Building a Cipher: step 3 - use a less ambiguous word

Describe the Issue
The instructions.

For now, print your text variable to the screen by including the variable name between the opening and closing parentheses of the print() function.

The word "including" might be misinterpreted as the text variable + something more. That something more might be understood as quotes.
I would suggest using "adding" or "putting" instead.

For now, print your text variable to the screen by adding the variable name between the opening and closing parentheses of the print() function.

We should change it in the hint as well

You should pass text to your print() function by adding the name of this variable within parentheses.

We might also test for quotes or just make it part of the hint

You should pass text to your print() function by adding the name of this variable within parentheses. Do not put quotes around the variable name.

As suggested in the thread, a code example with the print() function passed a variable as an argument might also help.
Affected Page
https://www.freecodecamp.org/learn/scientific-computing-with-python/learn-string-manipulation-by-building-a-cipher/step-3
Your code

Expected behavior
Use less ambiguous words.
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
Forum: https://forum.freecodecamp.org/t/step-3-not-making-sense/662472

I think this could also use a concrete example:
a = "Apple"
print(a)

This will print the string "Apple" because it is stored in the variable a.

I agree, an example would help.
Considering this is step 3 in the very beginning of the camper learning Python (and for all we know coding) the first time they are asked to use a variable it should be the main focus of the challenge. I would suggest switching the order of importance here.
Start by explaining how to use a variable, then explain print and show an example of using a variable with the print function. Do not start by explaining the print function.

I will open a PR to change many steps in this project soon. Considering my current draft, this step does not exist anymore as is. The camper is asked to add an empty print() call and pass a variable only in the next step.


How to add/delete a tree data node ? Can a slot be provided

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

See #87

See #87

I tried to use "ControlledTreeEnvironment" and suppose I have full control of the tree data.
In my situation, I have a param called "defaultTreeData" to save original tree data.
First, I'm trying to add one node key "new-0", and reset the tree data by setState(defaultTreeData) (The tree is reset to default in UI).
After that, I add the node key "new-0" again, it will occur "Encountered two children with the same key," and show 2 "new-0" in UI.
const onResetNode = () => {
    setTreeData(defaultTreeData);
}

// Call from "renderItem"
const onAddNode = (item) => {

    const temp = { ...treeData };

    const nodeId = item.index;
    const parentId = Object.values(temp).find(o => o.children.includes(nodeId)).index;

    const prefix = "new-";
    const count = Object.keys(temp).filter(o => o.match(prefix)).length;
    const newNodeId = `${prefix}${count}`;

    temp[`new-${count}`] = {
        index: newNodeId,
        canMove: true,
        isFolder: true,
        children: [],
        data: {
            id: newNodeId,
            name: `New Location ${count}`
        },
        canRename: true
    }

    // Add node to current node children
    temp[nodeId].children = [
        newNodeId,
        ...temp[nodeId].children
    ];

    setTreeData(temp);
}


PDF.js not rendering when it should

Summary
example:
https://adl.b2.adventistdigitallibrary.org/concern/published_works/20218025_ellen_white_and_her_work

Acceptance Criteria

[ ]

Screenshots or Video
Testing Instructions
To be filled out by dev
Notes

I'm seeing some unsplit works (uploaded before the cutover) are failing to show the PDF in the PDF.js viewer, but the thumbnail is displaying in the UV. So weird!
Example 1: https://adl.b2.adventistdigitallibrary.org/concern/published_works/20122816_the_laws_of_life_november_0_1863

Dropping a comment on behavior I'm seeing. A couple weeks ago when the team ran our big OAI import post cutover, the new works showed in PDF.js. Today I'm running a big import (but small compared to that OAI feed) and I'm seeing thumbnails only. The viewer isn't rendering. The import is still going, so maybe the viewer is just slow? But that is not how it behaved before the cutover. Before recent changes, PDF.js would render within a few seconds of a file attaching to a work. Generally I only had to refresh once.
Here's the first work that came from the recent import. It was ingested over 20 hours ago, and it shows only a thumbnail.
https://adl.b2.adventistdigitallibrary.org/concern/published_works/38016901_signs_of_the_times_april_25_1916
And the importer still in progress:
https://adl.b2.adventistdigitallibrary.org/importers/523?locale=en
I definitely want to check this again to confirm the viewer is still missing, but for now, here's a comment to mark what i see.

This seems to be a case of re-ingesting things that were previously ingested... the link you provided was ingested a year ago, by the dates on the child works. The child works will still be in fedora. I haven't checked the parent, so I don't know what happens with the reingest... I assume it's in Valkyrie now.
Because there are child works, it tries to use the universal viewer rather than the PDFjs viewer. And the universal viewer can't handle the mixed situations well.
In short, I think we can expect to have viewer issues until we complete the migration to Valkyrie.


First Page doesn't show

I just installed this component and when I run it, I see this:

If I start to swipe (even a slight bit) then the screens show correctly.

It happens to me too on some Android devices.

happening to me on ios simulator too


Posibility to use daemonset along with the service.

Hello. I was wondering is it possible to have mode set to daemonset and have service available at the same time?
I'm asking about opentelemetry-collector helm chart.

The charts do not currently support a service with mode=daemonset. What use case did you have in mind?

@TylerHelmuth replacing node-exporter but since our cluster using spot instances - it would be reasonable to use daemonset to assume that every new node gets opentelemetry-collector deployed.
Unless I'm missing something and metrics still can be scraped even without the service?

Unless I'm missing something and metrics still can be scraped even without the service?

I might have interpreted this question incorrectly. Are you referring to scraping the collectors metrics on itself
(<IP_ADDRESS>:8888/metrics) ?

Are you referring to scraping the collectors metrics on itself
(<IP_ADDRESS>:8888/metrics) ?

Probably. I've initially though that it is possible to have only agent exposing metrics by itself (as having mode: deployment and service enabled giving desired results) but now realizing that collector might be required to have external scraping done by prometheus at first and vmagent later.
Is my understanding correct, that having opentelemetry-agent with mode: daemonset and opentelemetry-collector with enabled prometheus receiver as well as pipeline will get desired results?

I think it would be possible for a deployment instance of the collector to scrape the telemetry of the agent instances, but the current charts set up the collector to scrape itself: https://github.com/open-telemetry/opentelemetry-helm-charts/blob/main/charts/opentelemetry-collector/examples/daemonset-only/rendered/configmap-agent.yaml. When deployed as a daemonset (or a deployment) the default config for the chart is to have the collector scrape its own metrics, which could then be exported anywhere and anyway you want.

I have another use case which could benefit from service for daemonset.
Imagine that apps could be both in a k8s cluster and not (e.g. in the process of migrating or for some other reason some apps are outside of the cluster). As a developer, I want to send tracing data from inside and outside of the cluster to the collectors.

As a developer, I want to send tracing data from inside and outside of the cluster to the collectors.

In this scenario running a deployment instance of the collector in addition to a daemonset seems like the correct approach. Applications inside k8s would send traces to the daemonset (and the daemonset would forward to the deployment if necessary). Applications outside the collector would send directly to the deployment.

Handled via https://github.com/open-telemetry/opentelemetry-helm-charts/pull/427


Is pip show output reliable? Can we make it standard?

This came up during https://github.com/fractal-analytics-platform/fractal-server/pull/198

is this still an issue? I removed the -f flag in pip show as it is not necessary in be55347

I was rather thinking about a machine-readable output for pip show command, but it looks like previous PRs in that direction were closed and the current output is considered to be machine-readable:

https://github.com/pypa/pip/issues/5261
https://github.com/pypa/pip/pull/5514

However pip list does have a --json flag. This is not high-priority, but I propose we use pip list --json > /some/tmp/file.json, and then
# load /some/tmp/file.json
location = next(package for package in package_list if package["name"] == "devtools")["location"]

didn't know about this. moving this issue to the backlog milestone

to address this issue, see
https://github.com/fractal-analytics-platform/fractal-server/blob/main/fractal_server/tasks/collection.py#L389

seeing a bit more in detail the output of pip, I think it is better to keep the parsing of pip show [pkg] rather than moving to pip list --json and then parsing the output.
However I do think that it is better to make package installation path discovery more robust by re-activating the -f option and using its output to extract the base path. The final package_root should be the value of location as it is now path-concatenated with the base path of the output of -f, e.g.,
stdout_inspect_lines = stdout_inspect.split("\n")
location = Path(
        next(
            line.split()[-1]
            for line in stdout_inspect_lines
            if line.startswith("Location:")
        )
    )
package_name = Path(stdout_inspect_lines[-1]).parts[0]
package_root = location / package_name

-f option seems to be good enough for us atm


From private repo

@PRMerger9 : Thanks for your contribution! The author, @, has been notified to review your proposed change.


Size of Charts Requires Refresh of Page

Currently, charts do not resize dynamically, requiring a refresh of the page. Not intended

Fixed in #75 with great stress! :D


[Bug] Lonsonho Tuya 2 gang dimmer not working as expected

Describe the bug
Recently got a Lonsonho QS-Zigbee-D02-TRIAC-2C-LN, which is a 2 gang dimmer.
This exposes 2 "endpoints" meaning that there are two additional topics, one for each  connection (l1 and l2).
Current behavior
After some fiddling around I ended up with 3 services in HomeKit: one that controls the brightness of one of the lights and two that control the state of one of the lights (but these do not really seem to work).
Expected behavior
One on/off + brightness control for each light/endpoint.
MQTT messages
On each topic (l1 and l2), a stateandbrightness` property is available.
Versions used

This plugin 0.0.9

Not sure if I should fix this or if I should wait for the exposes attribute from zigbee2mqtt to be released. 🤔

Decided to use homebridge-mqttthing as a workaround for now.

The exposes information in the device definition from zigbee2mqtt shows that the properties suffixed with _l1 and _l2 should be used (i.e. brightness_l2).
However, during the refactoring of this plugin to use this new exposes information, I noticed this does not actually work.
See: koenkk/zigbee2mqtt#5186

Closing this issue as #18 will fix this. While testing my implementation, the lights behaved as I would expect.


FirmwareControl: Status field updated with new case

https://jenkins.code.rdkcentral.com/job/github-triggered/1460/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/1480/ started


Better access

This change lets us overload [] operator and []= operator, aswell as introducing the a{} operator


About gitea free-hosting - is there any service like that?

I think the gitea hosting is easy to found.But...most of them are expensive.So is there any service to host gitea without pay? 🥇

codeberg.org, gitea.com to name a few.

https://gitea.com/gitea/awesome-gitea/#user-content-organizations
You could even run gitea on your own computer for free too!


fix(notification/tag): update docs and add storybook styles

Related Ticket(s)
carbon-design-system/carbon-for-ibm-dotcom#9757
Description
Update storybook styles to fix token values coming through from components with token mixins. Add docs for adopter to include on there end.
Changelog
New

add styles to storybook container
add documentation

opened in new repo


Parallax Section

@66eli77  Pull request :)

I don't see you use the file kickstarter_parallax_template.html anywhere on your branch, but the commit contains lots of changes on this file..


v7-login: Fixed issues with login

Fixed issues with login

opened new PR: https://github.com/ciscospark/spark-js-sdk/pull/548
hence closing this one.


Zoom on marker when clicked

Hi,
Would it be possible, when in map view, to add the ability to zoom on a marker when clicked and then display a summary of the note in a hovering panel without opening the note right away ?
Many thanks once more for your amazing work with this plugin !

That's a very nice idea. Not sure I can prioritize it at the moment, but (1) maybe someone else can pick it up and do a pull request, (2) if there's indication that many users will want this, I can prioritize it at some point.
Thanks!


Fix journal pages, settings constants, video duration

Fix use of 'Play as Transition' button on journals.
Move setting ids to constants for easy replacement.
Set fade out delay to video duration.

TY very much!


Stop scrolling while drag and drop

Godot version:
3.1 Alpha 4
OS/device including version:
Windows 10
Issue description:
It's quite annoying that the FileSystem always scrolls up when you want to (for example) put an audio file into a variable.
It should scroll only when the cursor hovers over the "FileSystem" area
Steps to reproduce:

Duplicate of  #23375