jsonwatch: 0.6.0 -> 0.7.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/dbohdan/jsonwatch/releases.
meta.description for jsonwatch is: Like watch -d but for JSON
meta.homepage for jsonwatch is: https://github.com/dbohdan/jsonwatch
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7.0 with grep in /nix/store/f81m4hnb32b8a1g2aaq75mx810h6kcsb-jsonwatch-0.7.0
found 0.7.0 in filename of file in /nix/store/f81m4hnb32b8a1g2aaq75mx810h6kcsb-jsonwatch-0.7.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

jsonwatch

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/f81m4hnb32b8a1g2aaq75mx810h6kcsb-jsonwatch-0.7.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jsonwatch https://github.com/r-ryantm/nixpkgs/archive/bc5a0edad94449d442ffe73856b61f3814840908.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/bc5a0edad94449d442ffe73856b61f3814840908#jsonwatch

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/f81m4hnb32b8a1g2aaq75mx810h6kcsb-jsonwatch-0.7.0
ls -la /nix/store/f81m4hnb32b8a1g2aaq75mx810h6kcsb-jsonwatch-0.7.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

jsonwatch

Maintainer pings
cc @fabaff for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 358872

x86_64-linux

:white_check_mark: 1 package built:

jsonwatch


Remove deleted keys during memtable flushing for lock cf

Description
The work pattern of lock CF in TiKV is insert one when prewrites and then remove it when commits and they are usually in the same memtable. However, RocksDB flushes all entries in the memtable to the disk which is wasteful for lock CF. You should implement it in https://github.com/tikv/rocksdb. Please be careful about the correctness of compaction in RocksDB.
SIG Slack Channel
You can join #sig-transaction on slack in your spare time to discuss and get help with mentors or others.
Score

5000

Mentor(s)

@youjiali1995

Contact the mentors: #tidb-challenge-program channel in TiDB Community  Slack Workspace
Recommended Skills

C++
RocksDB

Learning Materials

https://github.com/facebook/rocksdb/wiki/Compaction

Why not use SingleDelete in RocksDB?

Why not use SingleDelete in RocksDB?

I didn't know that! I will take a look at it. Thanks!

/pick-up

SingleDelete has such a constraint:

SingleDelete() removes the database entry for a key, but has the prerequisites that the key exists and was not overwritten.

In TiKV, a key in lock CF may be overwritten multiple times, for example, increasing TTL, updating overlap field. If we want to use it, we need to SingleDelete it and then put it to overwrite a key in lock cf.
There is a compatible issue during the rolling upgrade. If a new version TiKV SingleDeletes a key and replicates the log to old version replicas, replicas can't apply it because they don't know how to handle it which results in data inconsistency.
Actually, I'm not sure whether it's worthy to do it and it's complicated. I think we need to figure out how much disk flow is consumed by the compaction of lock CF.

SingleDelete has such a constraint:

SingleDelete() removes the database entry for a key, but has the prerequisites that the key exists and was not overwritten.

In TiKV, a key in lock CF may be overwritten multiple times, for example, increasing TTL, updating overlap field. If we want to use it, we need to SingleDelete it and then put it to overwrite a key in lock cf.
There is a compatible issue during the rolling upgrade. If a new version TiKV SingleDeletes a key and replicates the log to old version replicas, replicas can't apply it because they don't know how to handle it which results in data inconsistency.
Actually, I'm not sure whether it's worthy to do it and it's complicated. I think we need to figure out how much disk flow is consumed by the compaction of lock CF.

Close it for now until we figure out how much disk flow is consumed by the compaction of lock CF.

According to the comment https://github.com/tikv/tikv/issues/8140#issuecomment-862190703, the lock-cf compaction flow is high.

/cc @SpadeA-Tang

@youjiali1995 Seems what you want is SingleDelete? it's supported by RocksDB, and all CF can use it. But considering the reentability, we didn't use it.

@Connor1996 Yes, we will make a naive POC to get its improvments before deciding whether to do it or not.

But considering the reentability or idempotence, we didn't use it.

It indeed needs some work to solve these.

@youjiali1995 Okay, please let me know if you get the result


Using Local Docker Registry

For our efforts we need to use a local Docker Registry instead of Docker Hub.  Not sure if this is an issue or not but this is what I found.  Docker registry installs and works fine.  To get and existing image pushed into my new local registry I need to pull them down from the hub and then push them.  Once they are pulled Shipyard recognizes them as usual.  After I push that same image into my local repository, shipyard recognizes the second identical image on the same line as the first image instead of two separate images from separate repositories.  So, simply, two images have the same image ID but in different repositories accessible by Docker.  Using CLI, I am able to specify by name which image I would like to manipulate.

Untagging the old image, fixed the issue.


Update app.js

If config.popUp is set to true, the call to authenticationContext.acquireToken will fail with an error message of "Token renewal operation failed due to timeout".
I moved the handleWindowCallback so that it is always called on redirect from AAD, rather than just when popup == false

@navyasric : do you want to have a look (long term we'd split JS)

@benjamin-lewis-ms- Does always calling handleWindowCallback resolve the token renewal timeout? The token renewal timeout may happen due to other reasons. The handleWindowCallback is intended to be used only for the login and acquireToken methods which performs full page redirect. When popUp =true is enabled, the auth flows should occur in a Popup window and your main page is not redirected right?

Closing as we are archiving this Azure AD v1.0 sample (and redirecting to the v2.0 sample)


Image stamp prevents lightcurve being viewed across full screen

The image stamp that now appears top right of every request prevents the lightcurve from being viewed across the full width of a page.
E.g.:
https://fallingstar-data.com/forcedphot/queue/86781/
Even requests which I could previously view the lightcurve across the page are no longer viewable that way.  I'm guessing there's a new bootstrap column for the image contents.

Could you add a screenshot of what you are seeing? The lightcurve is displaying as expected for me on Chrome and Safari.

Interesting... When I first click the link I get this... (I'm using Safari.)

But when I refresh the page, I get this...

Perhaps some old javascript code was cached? Try a shift + refresh in Safari. Does the problem come back or did the refresh fix it for new pages as well?

Looks like I had some old cached javascript.  Shift-refresh seemed to fix it.  I've closed the issue!  Thanks.


Home button

I feel the ui lacks an home button in the upper left corner

Duplicate of #19, closing.


Problems with v-model and a components $data

http://codepen.io/anon/pen/qOgWjz

v-on:submit has no effect (does not trigger specified method)
v-model doesn't update the components data. (it's updating something though)
:class doesn't reflect components data changes.

Looks like I've fallen victim to : http://vuejs.org/guide/components.html#Compilation_Scope

in short, I needed to go from this:
	<form is="contact-form" :class="state" v-on:submit.prevent="onSubmit">
	...
	</form>

to this:

	<form :class="state" v-on:submit.prevent="onSubmit">
	...
	</form>

@airtonix Any reason you aren't placing the markup in the component's own template?

<form :class="state" @submit="send">

        ...

</form>

<contact-form></contact-form>

<form is="contact-form"></form>

@rpkilby because, reasons.
https://github.com/vuejs/vue/issues/963#issuecomment-156256104


Have problem in label  when import customize opendrive map

Platform: win10
Carla Version: 0.9.9
UE4 Engine Version: 4.24.3
Python Version: 3.7.3
Hi, From the official documentation, I can import the official and carla-provided opendrive files into UE4_Carla through the following script, but importing a custom opendrive file and parsing the  tag caused UE4 to crash and make an error.
The correct content is as follows(running is the TownBig.xodr file):

The error content is as follows(running is the customize xodr file):

UE4 crashed directly！
Then I found that there was an error in the trafficlightmanager.cpp file, so I deleted all the signals tags in the custom xodr file.
Surprisingly, I can import a customized map into UE4, but unfortunately, all the signal lights and road signs have disappeared
The content is as follows(running is the customize xodr file without signals tags):

So I compared the official and my customized  and found that there is a difference. Can you help me explain the difference and how to change the customized  into an official similar and successful Import into UE4_carla：
my.xodr:
        <signals>             <signal s="3.0000000000000000e+00" t="4.5999999999999996e+00" id="175" name="_Sg175" dynamic="yes" orientation="+" zOffset="2.9999999999999999e-01" type="1000002" country="OpenDRIVE" subtype="-1" hOffset="0.0000000000000000e+00" pitch="0.0000000000000000e+00" roll="0.0000000000000000e+00" height="2.62" width="0.35">                 <validity fromLane="0" toLane="0"/>                 <userData code="3" value="0.0000000000000000e+00"/>             </signal>             <signal s="3.0000000000000000e+00" t="-4.5999999999999996e+00" id="176" name="_Sg176" dynamic="yes" orientation="+" zOffset="2.9999999999999999e-01" type="1000002" country="OpenDRIVE" subtype="-1" hOffset="0.0000000000000000e+00" pitch="0.0000000000000000e+00" roll="0.0000000000000000e+00" height="2.62" width="0.35">                 <validity fromLane="0" toLane="0"/>                 <userData code="3" value="0.0000000000000000e+00"/>             </signal>         </signals>
TownBig.xodr:
        <signals>             <signal name="Sign_Yield" id="6080" s="6.3103172793201823e-2" t="5.6349899716323364e+0" zOffset="2.5534391922906252e+0" hOffset="3.1415927410125732e+0" roll="7.2123139708765716e-13" pitch="3.8721170858480036e-4" orientation="+" dynamic="no" country="OpenDRIVE" type="205" subtype="-1" value="-1.0000000000000000e+0" text="" height="1.0000000000000000e+0" width="1.1252747774124146e+0">                 <validity fromLane="0" toLane="0"/>                 <userData>                     <vectorSignal signalId="{424cac1f-bc26-4dcc-971b-faa881bb081c}"/>                 </userData>             </signal>         </signals>

@Axel1092 this needs your attention.

Hi @H-Whisky, thank you for your report, this is a problem that happens when a signal is having an effect in a lane that has 0 width which may happen when the lane is.

If you work from source you may try to update the code, otherwise a work arround would be to make sure that signals are placed or references in lanes with width greater than 0.

Thanks @Axel1092! I will update the code in the current master branch. And try again.

Hi @H-Whisky, thank you for your report, this is a problem that happens when a signal is having an effect in a lane that has 0 width which may happen at the start of a lane. This issue will be fixed in the next release and is fixed in the current master branch. If you work from source you may try to update the code, otherwise a work arround would be to make sure that signals are placed or references in lanes with width greater than 0.

Hi@Axel1092,  I meet the issues when I update the code to master. I can run "make launch" and run a simulator by the Unreal Edior. But when I run "make PythonAPI" to get the corresponding *.egg file, I meet the following error:
d:\workspace\carla\pythonapi\carla\source\libcarla\OSM2ODR.cpp(7): fatal error C1083: Cannot open include file: “OSM2ODR.h”: No such file or directory
error: command 'D:\\software\\VisualStudio2017\\VC\\Tools\\MSVC\\14.16.27023\\bin\\HostX86\\x64\\cl.exe' failed with exit status 2

-[BuildPythonAPI]: Carla lib for python has been successfully installed in "D:\workspace\carla\PythonAPI\carla\dist"!

I see the error notification, so I go to the carla/build, and I see that the xerces-c-3.2.3-install folder is null, and the xerces-c-3.2.3-source has been download, I don't know how to make the xerces-c-3.2.3-install really installded. Need you help. Thank you!


Android "verifyFingerprint" can be bypassed when setting "useCustomAndroidUI" to "true"

"verifyFingerprint" can be bypassed when setting "useCustomAndroidUI" to "true"
Steps to reproduce:

Have the Custom Android UI on screen
Click "USE PASSWORD"
Input incorrect password
resolve as success

P.S. No issue when not using "useCustomAndroidUI"
FYR

Android 8.1.0 on Nexus 6P
nativescript-fingerprint-auth 6.0.3
tns-android 3.3.1

Thanks! See #35.


Add HomeKit Accessory timeout, can not recover.

if the "enter the code" to add accessory times-out.   Then an error shows, and the accessory disappears, and never comes back :(  can this please be looked at?
Client Disconnected
Server Disconnected
New Session 51080
Server Connection Established 58339
Client Disconnected
New Session 51081
Server Connection Established 58340
Start Verify M1
Verify M1 Finished
Server Disconnected
Start Verify M3
M3: Verify Success

I hat a similar case too, check the "intitalValue: "  in your service. I think if it's undefind then it doesn't work.

not sure.  I am just going to reset the homekit on the iphone and rm  the homebridge and try again.

I reset everything, took me 27 minutes to reinstall homebridge and reset homekit on iphone.
this time if I entered the code quicker it worked.  but if you leave it for a minute.. it all falls apart, and you have little choice but to throw it all away and start again :(
TCP server accepting connection on port: 51882
HAP Server is listening
New Session 51126
Server Connection Established 58585
Start Pair M1
Start Pair M3
Start Pair M5
Client Disconnected
New Session 51127
Server Connection Established 58586
Start Verify M1
Verify M1 Finished
Server Disconnected
Start Verify M3
M3: Verify Success

Can you try using the latest homebridge from npm or from master branch of this repo? The pairing process has been majorly reworked since last npm version.

@nfarina yes I am using your new version ALWAYS. i clearly stated I have done "npm install homebridge".

@nzhome the latest version shouldn't log anything like this ...
Start Verify M1 Verify M1 Finished Server Disconnected Start Verify M3 M3: Verify Success

@nfarina, thanks for updating the npm install homebridge..  thats much better now.
"nfarina published 15 hours ago"  new https://www.npmjs.com/package/homebridge
the new version comes via npm install, thanks.   I will close the issue.

@KhaosT
What was the problem?
I am working with ESP32, i am struggling same issue


Split opaque and transparent phases

Objective
Fixes #8089.
Solution
Splits the MainPass3dNode into 2 nodes, one for the opaque + alpha passes and one for the transparent pass.

Changelog

Split MainPass3dNode into MainOpaquePass3dNode and MainTransparentPass3dNode
Combine opaque and alpha phases in MainOpaquePass3dNode into one pass
Create START_MAIN_PASS and END_MAIN_PASS empty nodes as labels
Main pass becomes START_MAIN_PASS -> MAIN_OPAQUE_PASS -> MAIN_TRANSPARENT_PASS -> END_MAIN_PASS

Migration Guide
Nodes that previously added edges involving MAIN_PASS should now add edges to or from START_MAIN_PASS or END_MAIN_PASS respectively.

I'd like to keep the ability of scheduling things relative to a MAIN_PASS that isn't concerned about transparency. Unfortunately, that might mean adding a START_MAIN_PASS/END_MAIN_PASS empty node.

I'd like to keep the ability of scheduling things relative to a MAIN_PASS that isn't concerned about transparency. Unfortunately, that might mean adding a START_MAIN_PASS/END_MAIN_PASS empty node.

I had the same thought, it did seem odd going into the ui and pbr node and having to just know which pass to order relative to, happy to also add those if we think it would help with usability.

I'd like to keep the ability of scheduling things relative to a MAIN_PASS that isn't concerned about transparency. Unfortunately, that might mean adding a START_MAIN_PASS/END_MAIN_PASS empty node.

Sounds like a good solution for now. More indication that we should split up "node graph markers/labels" and actual running nodes though.

Ooo this is really nice, since it allows a more “pluggable” approach with the (potential) transmissive phase and might allow in the future to have it also optionally go after the transparent phase (for scenes where you also want to refract transparent meshes)

Please address the merge conflicts and then we can get this merged.


PR #17 - All filenames in lowercase - WMS #8461

PR #17 - All filenames in lowercase

Issue resolved


MCDU Values?

Hi,
i wanted to ask you if its possible to read out the MCDU Display values?
Why? I have to explain it more:
Normaly on building an MCDU you would use a small lcd display with HDMI, pop out the panel and move it to the display. For me this is not a good solution and drops performance (if you have open more panels like PFD, MFD, ECAM, .....).
I've made a small project for Mobiflight (https://github.com/gagagu/Mobiflight-A320-Efis-Display-with-Arduino) which uses an additional Arduino and an OLED Display which emulates a normal LCD Display. This means for Mobiflight its an normal LCD but in reality it is a OLED Display. Instead of sending the display content, Mobiflight sends commands to control the Arduino which displays the correspondent values to the OLED Display. For example Mobiflight sends "setBaro(1031)" and the Arduino set the OLED Display to "1031", Mobiflight sends "setStd()" and the Arduino set the OLED Display to "Std".
The URL posted above shown an old version with an Arduino, in the meantime i am using an ESC32 which has second I2C Ports, one for Mobiflight communication and the second for the Display, which is much much more faster.
With this concep i want to create an Efis, FCU and it's possible to create an MCDU. But this provided that i am able to get the display content values.
This is the reason for my question...
Thx

I'm afraid that the Answer is rather simple: no :/
Alone adding the V-Speeds for Takeoff was a huge untertaking, increased Scan-Times considerably, was the only the got broken with V2 and need very specific Situations to find it at all.
So adding even more such develop and maintenance intensive Values for maybe a handful Users is just too much, sorry.

Hi, thx for reply.
I'm a c# developer for Database Apps so may be i will find the time to analyze it.
Greetings


search.fuzzyFilePicker might not be a good name

In our vscode.d.ts we call it quick pick, not file picker.
So I would call the setting search.fuzzyQuickPick

I am also not 100% sure about the name, but the setting has no impact on the picker that extenson writers can use.
@egamma @chrisdias @stevencl maybe you have an idea how to call this setting and describe it better. It enables alternate scoring and matching for the file picker.

Interestingly Atom calls it like this:

Why is it under the search category? The other search option(s) apply to the search viewlet (search.exclude).  Perhaps we need the quickPick category? If so I suggest:
"quickPick.useFuzzySearch": boolean

or
"quickPick.useAlternateScoring": boolean

Yes, we can introduce a new category. Note though that the search.exclude setting has an impact on quick open too.

"quickPick.useFuzzySearch": boolean works for me.

@egamma @chrisdias would people not expect that fuzzy search is then applied to all of quick pick, where today it is only applied to the file search and nothing else? I also think "fuzzy" might be a bad name because we are not matching as crazy fuzzy as sublime does. I have a feeling that in sublime the typed characters are matched anywhere in the file and path, whereas for us (and Atom I think) at least the characters must be in the same order as in the string.
search.alternateFilePickerMatching
Changes the file picker to use an alternate matching and scoring approach that will produce more results based on the query

The description should mention experimental then we set the expectation.
Suggest the following description, I would still use fuzzy this was also the term used in #968.

Experimental support for fuzzy matching of file names in the file picker

how about this option name to make it clear it applies to the file picker only:

filePicker.alternateFileNameMatching

:+1:


Proctored MCQ 1 : Quiz1,Quiz2 Scores

Study Plan

Order
Progress
Description

1
✔
I will work hard to go back and review the notes I took on the Collegeboard videos from trimesters 1 to 2. From the proctored MCQ 1's results, I understand that I'm still lacking a firm and complete understanding of some CS concepts such as lossless vs lossy, binary (...6, 7, 8 digits), parallel execution time. I will review these notes to refresh my memory.

2
✔
I will then go back and review my test corrections from tri 1, tri 2, and tri 3. Since there's a pattern in the questions of CSP tests, I feel like it would be very helpful if I review questions I missed and gain an understanding so I don't make the same mistakes.

3
✔
I will continue to refine my responses to Collegeboard requirements for my Create Task. Although I finished the code portion and feel pretty well about it, I feel like it's also essential to ensure that I answered all parts of the questions and met all the requirements.

4
✔
Lastly, I will try to watch the Collegeboard AP exam review videos to enhance my understanding. Since the videos are from Collegeboard, they sure will be helpful.

Create task
Quiz Corrections on Github pages
Finals-Quiz1 42/50
Finals-Quiz2 46/50

Study plan seems thorough and helpful
Create task is working and runs correctly(there is the way to change the display so it does not say 0 number
Corrections seem thought out and scores are good.
5/5


Support newer node versions

Thanks a lot for developing this project! The result is pretty impressive.
Currently, the package.json only specifies that Node 18 is supported. It would be nice if newer Nodes could be used. I just tried it with Node 21 (the "Echo" Chat mode), and it seemed to work.

Thanks for letting me know! Fixed here 4bb90386d574aa23b5ddea31de96e7ca6da05d85

Thanks!


Got failure when change bridge multicast-snooping from false to true

Describe the bug
Got failure when change bridge multicast-snooping from false to true
Nmstate version

0.3.3

NetworkManager version

1.26.0

Platform

All

How reproducible

~100%

Additional context
Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1855160

This blocked by https://bugzilla.redhat.com/show_bug.cgi?id=1859244


TypeScript issue on AgoraVideoPlayer

When publishing a export const useCreateScreenVideoTrack = createScreenVideoTrack(createScreenVideoTrackConfig);
and using
<AgoraVideoPlayer
  style={{ height: "100%", width: "100%" }}
  videoTrack={screenVideoTrack}
/>

I get this error
(property) videoTrack: ICameraVideoTrack | ILocalVideoTrack | IRemoteVideoTrack
Type 'ILocalVideoTrack | [ILocalVideoTrack, ILocalAudioTrack]' is not assignable to type 'ICameraVideoTrack | ILocalVideoTrack | IRemoteVideoTrack'.
  Type '[ILocalVideoTrack, ILocalAudioTrack]' is not assignable to type 'ICameraVideoTrack | ILocalVideoTrack | IRemoteVideoTrack'.ts(2322)
index.d.ts(183, 5): The expected type comes from property 'videoTrack' which is declared here on type 'IntrinsicAttributes & ClassAttributes<HTMLDivElement> & HTMLAttributes<HTMLDivElement> & { ...; } & { ...; }'

createScreenVideoTrack returns either a videoTrack or an array of [videoTrack, audioTrack]. You can check if the type of track isn't an array to get rid of the Typescript error.


Missing more information link in summary

In the summary section of DataLakePathClient.SetAccessControlRecursive the last paragraph seems to be lacking a link to further documentation.
Current state: "For more information, see ."

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

ID: d7bf759a-798d-1a9f-907f-5fe18249f08e
Version Independent ID: 25150b6a-0c84-273a-2987-2f99afe7756b
Content: DataLakePathClient.SetAccessControlRecursive(IList<PathAccessControlItem>, String, AccessControlChangeOptions, CancellationToken) Method (Azure.Storage.Files.DataLake) - Azure for .NET Developers
Content Source: xml/Azure.Storage.Files.DataLake/DataLakePathClient.xml
Service: azure
GitHub Login: @CamSoper
Microsoft Alias: casoper

Thank you for your feedback.  Tagging and routing to the team best able to assist.  Please be aware that due to the holidays, responses may be delayed.

Thank you for your feedback.  Tagging and routing to the team best able to assist.  Please be aware that due to the holidays, responses may be delayed.

Closing this issue as the PR was merged.

Closing this issue as the PR was merged.


No forwarded headers from the load balancer

If you believe you have an issue that affects the security of the platform please do NOT create an issue and instead email your issue details to<EMAIL_ADDRESS>Your report may be eligible for our bug bounty but ONLY if it is reported through email.
Describe the bug
A clear and concise description of what the bug is.
Is there any requirement for the header forwarding to work? I used to receive header values but I believe after I change the backend of the load balancer to use HTTPS, all headers are gone. No forwarded headers from the load balancer to the app at all.
I have the following code in startup.cs:
services.Configure<ForwardedHeadersOptions>(options => options.ForwardedHeaders = ForwardedHeaders.All);

            app.Use(async (context, next) =>
            {
                // Request method, scheme, and path
                _logger.LogInformation(JsonConvert.SerializeObject(new RequestLog(context?.Request?.Method,
                                    context?.Request?.Scheme,
                                    context?.Request?.PathBase,
                                    context?.Request?.Path,
                                    context?.Request?.Host.ToString(),
                                    context?.Request?.ContentLength,
                                    context?.Connection?.RemoteIpAddress?.ToString(),
                                    context?.Request?.QueryString.ToString(),
                                    context?.Request?.ContentType,
                                    context?.Request?.Protocol,
                                    context?.Request?.Headers
                                    )));
                await next();
            });

I extract the headers in RequestLog:
            if (headers != null)
                foreach (var header in headers)
                    if (header.Key.Equals("X-Forwarded-For") || header.Key.Equals("X-Original-For"))
                        IP = header.Value;
                    else if (header.Key.Equals("X-Forwarded-Proto") || header.Key.Equals("X-Original-Proto"))
                        X_Forwarded_Proto = header.Value;
                    else if (header.Key.Equals("X-Forwarded-Host") || header.Key.Equals("X-Original-Host"))
                        X_Forwarded_Host = header.Value;

But I never get any value in all 3 headers. Any advice and insight is appreciated.
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core '...': 2.2
Run this code '....'
With these arguments '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Include the output of dotnet --info
# dotnet --info

Host (useful for support):
  Version: 2.2.6
  Commit:  7dac9b1b51

.NET Core SDKs installed:
  No SDKs were found.

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.2.6 [/usr/share/dotnet/shared/Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.2.6 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.2.6 [/usr/share/dotnet/shared/Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

Please share the output from the troubleshooting instructions here:
https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/proxy-load-balancer?view=aspnetcore-2.2#troubleshoot
What proxy are you using, and how did you configure it?

Log shown in Kibana that there is no X-Forwarded- received:

Bash console showing no CORS header from the ASP.Net Core 2.2 app:

I have added KnownNetworks but to no avail:
options.KnownNetworks.Add(new IPNetwork(IPAddress.Parse("<IP_ADDRESS>"), 16));

Why is "X_Forwarded_Host" showing up in that screen shot. That's not the same as "X-Fowarded-Host".
Log all of the headers, not selected fields.

That's the property names defined in my POCO class serialized and logged by Serilog.
                _logger.LogInformation(JsonConvert.SerializeObject(new RequestLog(context?.Request?.Method,
                                    context?.Request?.Scheme,
                                    context?.Request?.PathBase,
                                    context?.Request?.Path,
                                    context?.Request?.Host.ToString(),
                                    context?.Request?.ContentLength,
                                    context?.Connection?.RemoteIpAddress?.ToString(),
                                    context?.Request?.QueryString.ToString(),
                                    context?.Request?.ContentType,
                                    context?.Request?.Protocol,
                                    context?.Request?.Headers
                                    )));

OK. I am rebuilding and redeploying with logging ALL headers now...

I notice that the X-Forwarded-For is recording the LB's IP and HttpRequest.Connection.RemoteIpAddress records the original client's IP. Shouldn't it be the other way round? The LocalIP in the attached picture is HttpRequest.Connection.RemoteIpAddress while IP is X-Forwarded-For:

IP is X-Forwarded-For

Or is it X-Original-For?
                    if (header.Key.Equals("X-Forwarded-For") || header.Key.Equals("X-Original-For"))
                        IP = header.Value;

The forwarded headers middleware removes X-Forwarded-For, sets RemoteIpAddress , and saves the old values of RemoteIpAddress in  X-Original-For.

So X-Original-For / X-Forwarded-For is the origin of the request, the client's IP address (https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-For). However, what happen now with the middleware is the opposite of this expected behaviour?

After going through the logs collected, I have also observed that the X-Forwarded- / X-Original- only appears when Host equals the destination domain in the format of www.domain.com. It doesn't work if the Host is IP address of the domain. Not sure if this is due to the LB ingress matching rules or some other reasons..?

But I never get any value in all 3 headers. Any advice and insight is appreciated.

The whole point of forwarded headers is to remove the X-Forwarded-For header and update the properties (Scheme, Host, etc.) with the values, so that's expected.

So X-Original-For / X-Forwarded-For is the origin of the request, the client's IP address

X-Original-For is the origin of the request, which in a proxy situation is not the browser client, it is the proxy.
Can you walk back to what you are actually seeing and what you expect to see, but don't see? We're having trouble assessing if there's an issue or just a misunderstanding about how the middleware works (possibly docs need to be improved).

Thanks for your explanation. I will just use RemoteIpAddress as the browser client's IP address and X-Original-For as the immediate hop's IP address.
I have updated my application to capture all X-<foo> headers in my logger's properties and this is what I get:

As seen, all X-Forwarded-<foo> are null. X-Original-<foo> captures the LB's information but why is the X-Original-Host null?

I will just use RemoteIpAddress as the browser client's IP address and X-Original-For as the immediate hop's IP address.

Yep, when the forwarded headers middleware is in place, that's the correct behavior. That middleware configures the HttpContext to look like it came from the original client, so RemoteIpAddress should be the browser client (as long as you are after the forwarded headers middleware in the pipeline).

X-Original-<foo> captures the LB's information but why is the X-Original-Host null?

My assumption (though @Tratcher would know better) is that since X-Original is supposed to be capturing the original data from the request the load-balancer made, the load balancer may not have provided a Host header value (or perhaps provided an empty value). You could check the headers provided by the original request using a middleware before the Forwarded Headers middleware to see what the original request provided. Is it possible the load balancer is making HTTP/1.0 requests?

Yes AWS LB is terminating HTTP/2 and only using HTTP/1.1 with the backend.

only using HTTP/1.1 with the backend.

HTTP/1.1 is not HTTP/1.0 ;). If using HTTP/1.1 the Host header is generally required. Can you capture the original headers sent by the load balancer (before the forwarded headers middleware runs)? That might help us identify why X-Original-Host is showing as empty.


MIDIVisualizer: Fix build on x86_64-darwin

Description of changes
See https://github.com/NixOS/nixpkgs/pull/206218/checks?check_run_id=10814950194
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 217992 run on aarch64-darwin 1


Pre-viewer

📝 I think that a pre-viewer for files like images, and docs, wiill turn the use experience more practice.
When pressed the space bar the document will apper in a window. It's like on the finder of MacOs and some Linux distributions

If you'd like to see this feature implemented, add a 👍 reaction to this post.

Where would you want to see this pre-view?

In macOs and some Linux distributions, the previewer is activated by clicking once on the file and then pressing the space bar. It quickly displays, in a small window, the image (jpg, png, gif), the audio or video file or document (PDF, doc, docx, etc.), as well as showing your metadata as details of the file.
This is a useful option for those who want to quickly view the file.
De<EMAIL_ADDRESS>Enviado:quarta-feira, 21 de outubro de 2020 16:46
Para<EMAIL_ADDRESS>Cc:Leônidas<EMAIL_ADDRESS><EMAIL_ADDRESS>Assunto: Re: [microsoft/PowerToys] Pre-viewer (#7452)
Where would you want to see this pre-view?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/7452#issuecomment-713833115, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANZ4L5P5JJX4YRQHBD26C4DSL43ATANCNFSM4S2F7XHQ.

Believe you're looking for https://github.com/microsoft/PowerToys/issues/80


Route file and middleware with problems

The route or middleware files are not being downloaded with the project when running composer, when trying to publish them it does not find the files, and looking in the downloaded folder in the project they do not exist.

In the next version, you will be able to publish the route!
Do you say publish middleware, routes, and ... automatically?

No, I have already run composer install and update, to download the package requirements, when trying to publish the route and middleware files manually, they do not exist in the project that is being downloaded by composer, including the new files that should be in the src of the file

For example, I try to run php artisan vendor:publish --tag="laravel-user-monitoring-routes" and it even gives an error because it says that this file does not exist to be published, and when consulting the folder where this file was supposed to be The same file does not exist.

Ohh I said before, you can publish route and middleware in the new version of Laravel-User-Monitoring.
The new version will be published next week

Also, you can download the dev version!


test PR-Robot

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
make sure your commit is signed off

/woof

/woof

/woof

/kind    bug


Data Upload Template - when edit template node can be added multiple times

Describe the bug
https://www.berrycast.com/conversations/661358be-2ad5-57e5-a30c-c5747e3906b0

that is ok,
needed as functionality


handle issue from azure-mgmt-resource 17.0.0 upgrade

In azure-mgmt-resource 17.0.0, the resources.get_by_id().properties is now a string, rather than a dict.
In practice, it's not a json string (uses single quotes rather than double quotes).  This converts the quotes & uses json to decode it.
For example, in practice, the value is: "{'tenantId': 'GUID', 'principalId': 'GUID', 'clientId': 'GUID'}

For reference: https://github.com/Azure/azure-sdk-for-python/issues/18763


Document offline mode

Hey @Bowbaq, not sure if you are aware of this feature which Git Town supports since 6 months. I wrote it on airplanes without Wifi  :)


Please invite me to the GitHub Community Organization

Please invite me to the GitHub Community Organisation. I am exited About contributing in Open-Source.

Name: Rohan Kulkarni

Discord Username(if exists):  Rohank2502#9618

Where did you meet Eddie?
I came to know about him as a Git-Hub Star from Code for Cause Youtube Channel.
What do you like about this community/ why do you want to join?
I am a first-year Btech student. I am curious regarding the open-source community. I will like to contribute to open source. As I am learning Cpp and Python and Also Front end development it will be helpful for a beginner like me to get guidance from the experts. And surely my journey will be amazing

Invitation sent for the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/EddieJaoudeCommunity/people
Tips for practising:
Customise your GitHub profile, here is a live stream on it https://www.youtube.com/watch?v=cT6GXCuS0Zo
Practice repo, instructions on how to add your name to the README in the README https://github.com/EddieJaoudeCommunity/hacktoberfest-practice
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps

Invitation sent for the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/EddieJaoudeCommunity/people
Tips for practising:
Customise your GitHub profile, here is a live stream on it https://www.youtube.com/watch?v=cT6GXCuS0Zo
Practice repo, instructions on how to add your name to the README in the README https://github.com/EddieJaoudeCommunity/hacktoberfest-practice
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps


feat: retain replaceable events with the lowest id when timestamps ar…

…e the same

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 96.294%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
892

Relevant Lines:
939

💛  - Coveralls


Transaction page crashes

Describe the bug

To Reproduce

Expected behavior

Screenshots

Desktop (please complete the following information):

OS:
Browser:
Version:

Smartphone (please complete the following information):

Device:
OS:
Browser
Version

Additional context

fixed on develop


[BUG] Feed ID 1233 has stale data (should be inactive?)

Describe the bug
Feed ID 1233 seems to have data in calendar / calendar_dates from 2022-05-01 -> 2023-01-28. We should mark this as inactive right?

Transitland has a working feed URL for that region, can resolve this by updating Mobility Database to match.


Rename EIDSCA docs folder to fix build script on unix

Update-EidscaTests.ps1 defaults to uppercase folder-name for EIDSCA docs.
https://github.com/maester365/maester/blob/ffe36539a86170c287ebf1414bd97010f14381d9/build/EIDSCA/Update-EidscaTests.ps1#L17-L18
Renamed folder to make it work on case sensitive filesystems like UNIX.

Thanks for the PR! I'm currently working on the build script to use lower-case paths only for all EIDSCA content which seems to fits also to the overall folder structure in maester.


Python 3.7 and 3.8 compatibility

I'm mainly posting this issue for discussion and to highlight possible todos.
I was trying to install PyKED and PyTeCK in an environment that requires python 3.7+. I ended up spending a fair amount of time banging my head against the wall with Anaconda until I realized that PyKED (and PyTeCK subsequently) uses python 3.6. Given that Python 3.8 has been released, should there be an effort to convert/modify aspects of PyKED (and PyTeCK) to support 3.7 and 3.8? If so what changes need to be made?

I think the only thing that needs to change is that new packages need to be built. Basically, if you bump the build number in the conda.recipe/meta.yaml that might work. It's also possible that something like limiting, e.g., Pandas is causing the reliance on 3.6, because there aren't built packages for that old of a dependency on 3.7 and 3.8.
You could probably install with Pip, provided that you installed all the dependencies of PyKED first and then just used pip to install PyKED. Or you could install from source.

Yeah, I was about to say the same. I imagine/hope most dependencies support 3.7, but 3.8 is probably less likely at this point.

Quick look at dependencies:

Pandas supports 3.7
Pyyaml 3.13 supports 3.7
numpy v1.17.2 supports 3.7
I think Pint v0.8.1 has a noarch package
habanero v0.6.0 has noarch
uncertanties v3.0.3 supports 3.7

The problem might be Cerberus. Right now we only support up versions below 1.2, and v1.1 only supports up to Python 3.6. (It looks like for v1.2 and v1.3.1 now have noarch packages...).
@bryanwweber do you recall why we limit Pint to <0.9?

The upper limit on all the dependencies is basically best practice, to avoid breaking changes assuming they follow semver. Since Pint is still zero-ver, it seemed best to limit to the next point release.

@nateharms if you have the bandwidth, it'd be great if you could check out PyKED's compatibility with newer versions of those dependencies—that will make it easier for us to support Python 3.7+.
But, if cerberus is really limiting us here, then I guess we'll need to revisit the issues limiting us to v1.1

I don't mind looking into it, I'll let y'all know how things progress.

Okay, from working with the build-environment.yaml file, I had to make the following changes to the dependency list get the environment to solve:
Added:

python == 3.7

Modified:

cerberus>=1.0.0,<1.2 -> cerberus>=1.2,<1.3
pandas >=0.22.0,<0.23 -> pandas >=0.23.0,<0.24

And when running conda list I get the following
# packages in environment at /Users/nathan/anaconda/envs/build-env:
#
# Name                    Version                   Build  Channel
alabaster                 0.7.12                   py37_0
anaconda-client           1.7.2                    py37_0
appnope                   0.1.0                    py37_0
asn1crypto                1.2.0                    py37_0
attrs                     19.3.0                     py_0
babel                     2.7.0                      py_0
backcall                  0.1.0                    py37_0
beautifulsoup4            4.8.1                    py37_0
blas                      1.0                         mkl
bleach                    3.1.0                    py37_0
bzip2                     1.0.8                h1de35cc_0
ca-certificates           2019.9.11            hecc5488_0    conda-forge
cerberus                  1.2                        py_1    conda-forge
certifi                   2019.9.11                py37_0    conda-forge
cffi                      1.13.0           py37hb5b8e2f_0
chardet                   3.0.4                 py37_1003
clyent                    1.2.2                    py37_1
conda                     4.7.12                   py37_0
conda-build               3.18.10                  py37_3
conda-package-handling    1.6.0            py37h1de35cc_0
cryptography              2.3.1            py37hdbc3d79_0
decorator                 4.4.0                    py37_1
defusedxml                0.6.0                      py_0
doctr                     1.8.0                      py_0    conda-forge
docutils                  0.15.2                   py37_0
entrypoints               0.3                      py37_0
filelock                  3.0.12                     py_0
glob2                     0.7                        py_0
habanero                  0.6.0                      py_0    conda-forge
idna                      2.8                      py37_0
imagesize                 1.1.0                    py37_0
importlib_metadata        0.23                     py37_0
intel-openmp              2019.4                      233
ipython                   7.8.0            py37h39e3cac_0
ipython_genutils          0.2.0                    py37_0
jedi                      0.15.1                   py37_0
jinja2                    2.10.3                     py_0
jsonschema                3.1.1                    py37_0
jupyter_core              4.6.0                    py37_0
libarchive                3.3.3                h786848e_5
libcxx                    4.0.1                hcfea43d_1
libcxxabi                 4.0.1                hcfea43d_1
libedit                   3.1.20181209         hb402a30_0
libffi                    3.2.1                h475c297_4
libgfortran               3.0.1                h93005f0_2
libiconv                  1.15                 hdd342a3_7
liblief                   0.9.0                h2a1bed3_2
libxml2                   2.9.9                hf6e021a_1
lz4-c                     <IP_ADDRESS>              h1de35cc_0
lzo                       2.10                 h362108e_2
markupsafe                1.1.1            py37h1de35cc_0
mistune                   0.8.4            py37h1de35cc_0
mkl                       2019.4                      233
mkl-service               2.3.0            py37hfbe908c_0
mkl_fft                   1.0.14           py37h5e564d8_0
mkl_random                1.1.0            py37ha771720_0
more-itertools            7.2.0                    py37_0
nbconvert                 5.6.0                    py37_1
nbformat                  4.4.0                    py37_0
nbsphinx                  0.4.3                      py_0    conda-forge
ncurses                   6.1                  h0a44026_1
numpy                     1.17.2           py37h99e6662_0
numpy-base                1.17.2           py37h6575580_0
openssl                   1.0.2r               h1de35cc_0    conda-forge
packaging                 19.2                       py_0
pandas                    0.23.4           py37h6440ff4_0
pandoc                    <IP_ADDRESS>                       0
pandocfilters             1.4.2                    py37_1
parso                     0.5.1                      py_0
pexpect                   4.7.0                    py37_0
pickleshare               0.7.5                    py37_0
pint                      0.8.1                      py_1    conda-forge
pip                       19.3.1                   py37_0
pkginfo                   <IP_ADDRESS>                  py37_0
prompt_toolkit            2.0.10                     py_0
psutil                    5.6.3            py37h1de35cc_0
ptyprocess                0.6.0                    py37_0
py-lief                   0.9.0            py37h1413db1_2
pycosat                   0.6.3            py37h1de35cc_0
pycparser                 2.19                     py37_0
pygments                  2.4.2                      py_0
pyopenssl                 19.0.0                   py37_0
pyparsing                 2.4.2                      py_0
pyrsistent                0.15.4           py37h1de35cc_0
pysocks                   1.7.1                    py37_0
python                    3.7.0                hc167b69_0
python-dateutil           2.8.0                    py37_0
python-libarchive-c       2.8                     py37_13
pytz                      2019.3                     py_0
pyyaml                    3.13             py37h1de35cc_0
readline                  7.0                  h1de35cc_5
requests                  2.22.0                   py37_0
ripgrep                   0.10.0               hc07d326_0
ruamel_yaml               0.15.46          py37h1de35cc_0
setuptools                41.4.0                   py37_0
six                       1.12.0                   py37_0
snowballstemmer           2.0.0                      py_0
soupsieve                 1.9.3                    py37_0
sphinx                    2.2.0                      py_0
sphinxcontrib-applehelp   1.0.1                      py_0
sphinxcontrib-devhelp     1.0.1                      py_0
sphinxcontrib-htmlhelp    1.0.2                      py_0
sphinxcontrib-jsmath      1.0.1                      py_0
sphinxcontrib-qthelp      1.0.2                      py_0
sphinxcontrib-serializinghtml 1.1.3                      py_0
sqlite                    3.30.0               ha441bb4_0
testpath                  0.4.2                    py37_0
tk                        8.6.8                ha441bb4_0
tqdm                      4.36.1                     py_0
traitlets                 4.3.3                    py37_0
uncertainties             3.0.3                 py37_1000    conda-forge
urllib3                   1.24.2                   py37_0
wcwidth                   0.1.7                    py37_0
webencodings              0.5.1                    py37_1
wheel                     0.33.6                   py37_0
xz                        5.2.4                h1de35cc_4
yaml                      0.1.7                hc338f04_2
zipp                      0.6.0                      py_0
zlib                      1.2.11               h1de35cc_3
zstd                      1.3.7                h5bba6e5_0

And then the tests (python setup.py test) won't run when you have cerberus >= 1.2 because of import errors:
__________________________________ ERROR collecting pyked/tests/test_validation.py ___________________________________
pyked/tests/test_validation.py:35: in <module>
    v = OurValidator(schema)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:169: in __init__
    self.schema = kwargs.get('schema', None)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:509: in schema
    self._schema = DefinitionSchema(self, schema)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/schema.py:69: in __init__
    self.validate(schema)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/schema.py:197: in validate
    self._validate(schema)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/schema.py:218: in _validate
    if not self.schema_validator(schema, normalize=False):
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:879: in validate
    self.__validate_unknown_fields(field)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:912: in __validate_unknown_fields
    if not validator({field: value}, normalize=False):
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:877: in validate
    self.__validate_definitions(definitions, field)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:940: in __validate_definitions
    result = validate_rule(rule)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:922: in validate_rule
    return validator(definitions.get(rule, None), field, value)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:1236: in _validate_schema
    self.__validate_schema_mapping(field, schema, value)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:1247: in __validate_schema_mapping
    if not validator(value, update=self.update, normalize=False):
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:877: in validate
    self.__validate_definitions(definitions, field)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:940: in __validate_definitions
    result = validate_rule(rule)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:921: in validate_rule
    validator = self.__get_rule_handler('validate', rule)
../../anaconda/envs/build-env/lib/python3.7/site-packages/cerberus/validator.py:338: in __get_rule_handler
    "domain.".format(rule, domain))
E   RuntimeError: There's no handler for 'isvalid_history' in the 'validate' domain.

NOTE: I modified the setup.py file by commenting out line 55 install_requires=install_requires to get it to run with my build environment.

So, at least hypothetically, @kyleniemeyer (as the maintainer of the Cerberus conda-forge feedstock) could push a commit there that bumps the build number for 1.1 and sets it to a noarch package, then another commit to come back to the current version. I'm not sure if conda-forge would approve of that, but what they don't know?

But honestly, you're probably better off with pip here. Just create a new environment, conda install everything except PyKED and Cerberus, then pip install pyked should pull in the right version of Cerberus.

Update: I set up my environment according to @bryanwweber's suggestion and everything is working in order. Thanks! It's a good workaround for now, but maybe a change that could be addressed in the future?
Thanks again for y'all's help!

Right, the key here is that we need to fix or remove the dependence on Cerberus. Feel free to jump in to that 😄


bug fix: restricting scipy version

Running the notebook results in the error message:
cannot import name 'triu' from 'scipy.linalg'
This PR restricts the version of scipy to avoid the import error message
See: https://stackoverflow.com/questions/78279136/importerror-cannot-import-name-triu-from-scipy-linalg-gensim

Thanks! It seems that it still gives the same error: ImportError: cannot import name 'triu' from 'scipy.linalg'. The cause seem to be from gensim. Maybe we should limit the version of gensim instead?

Thanks for the contribution!
It seems that it still gives the same error: ImportError: cannot import name 'triu' from 'scipy.linalg'. The cause seem to be from gensim. Maybe we should limit the version of gensim instead?

I hit the same issue. It seems that gensim is not constraining their use of scipy. The functionality itself is said to have been moved to numpy:

https://numpy.org/doc/stable/reference/generated/numpy.triu.html
https://docs.scipy.org/doc/scipy-1.12.0/reference/generated/scipy.linalg.triu.html#scipy.linalg.triu

The scipy.linalg.triu is marked as deprecated in scipy.


emoji: Start compose typeahead from the first letter.

Currently we only start the typeahead on the second letter, after the :. We should instead start it on the first letter.
You can see the current behavior by typing :oc in the compose box.

@zulipbot claim

@zulipbot claim

@rishig From the comment in composebox_typeahead.js:

// We don't want to match non-emoji emoticons such
// as :P or :-p
// Also, if the user has only typed a colon and nothing after,
// no need to match yet.

I think we have left typeahead from just first characters because many users find it easy to use just :D,:P or maybe some user-defined symbol like :C.

hmm, interesting point. Just checked what some other products do: github and dropbox paper autocomplete at the :, and slack does our current behavior (second letter).
One compromise might be to autocomplete on :[a-z], but not :[A-Z]. (And we won't be autocompleting on :), :-) etc anyway).
Could you start a discussion about this in #design?

I like that compromise.  Lower-case letters aren't really used in smileys, and all emoji have lower-case names, so it seems like it has all the benefits of both options.

I came up with this solution:
        if (/^:-.?$/.test(current_token)  || /^:[A-Z]?$/.test(current_token)) {
            return false;
        }

It covers the cases like :D',:X', :-p, :-l, :-) and as far as :) is concerned there is no emoji name at zulip which have ).
Another solution might be
if (/^:-.?$/.test(current_token)  || /^:[^a-z]?$/.test(current_token)) {     //second condition is different
            return false;
        }

it prevents any character other than a-z but it also prevents some first character typeaheads of emojis like :+1:
@rishig @timabbott I would like to know which one is better or is there any regex possible?

I guess we left this a bit underspecified. For 1 character: A-Z is no autocomplete, a-z is yes autocomplete.

For + and -, there is :-1: and :+1:. I'm tempted to say yes autocomplete for +, and no autocopmlete for -.
For 0-9, there will soon be things like 7.00 for 7 o' clock. I think the only emoticons here are :3 and :0? I could go either way, but first thought is no autocomplete.

In which case, I think the [^a-z+] strategy is better.

Yeah, sounds good.  And of course, @shubhamdhama make sure you include test cases for these corner cases we're talking about in the frontend_tests/node_tests/composebox_typeahead.js tests.


Content sync

Fulfills outcome 4. This code can sync two-way to a Google Drive.  Any files that are only on the Drive are copied to local; files that are only on local disk are copied to drive. If files exist on disk and drive and have different modification dates,  most recent one overwrites older one, regardless of location. Works recursively for folders, including empty ones.
Includes Mockito integration test. Please run app at least once to authorize drive before trying tests.


recorder controls not accessible by code

methods like StartRecording are not accessible by code.
missing a delegate setup or change visibility of control methods

Hi there,
Nice plugin. I'm using it to record touch gestures with Lean Touch without issues. Now I'm trying to use the recorder by code to record user input. What would be a quick way to be able to access the recorder controls by code? Would it be enough to call StartRecording() and SaveRecording() ?
thanks

Hi @ReallyRad sorry for the delay,
yes that would be sufficient. a quick work around would be to make Start and StopRecording public. I'll try to push a proper solution in the next days.

Hey there,
I managed to hack a solution by setting the corresponding booleans in the base recorder which I made public, but that also broke the recorder editor. Now I'm trying to load the recording back into the player by code, so I can immediatly see how the recording went. Any pointers on how to achieve that?

original issues solved with latest release.
@ReallyRad - I haven't looked into setting up the Unity Timeline via code. not sure if loading clips by code is supported. were you able to find a solution for your usecase?

@fx-lange
Hey there,
No, I wasn't. Custom workflows with scripting in the Timeline are tricky it seems. Should be doable though.
Cheers for the update.


4.8.1 -> 4.8.2

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


Keyboard Spacer and isNative={false}

Hi,
Any thoughts on how to get this to work?
I really need to use isNative={false} as I don't want the modal overlay to take up the whole screen on iPads.
Thanks
Sam

Well, first of all, what do you want to achieve? The native modal component fills the entire space by default and there is no way to change this behaviour. If you want to remove the overlay, there is a property that do that, but if you want to limit the visibility of the modal to a portion of the screen, you can play with containerStyle and isNative={false}. Try for example to pass this object to the containerStyle, something like { top: 100, left: 100 }, you should be able to limit the modal changing its absolute positioning

Hi,
Sorry, I really didn't explain that very well.
I was poking around the code trying to get the keyboardspacer to work with the js solution and saw that it was explicitly removed with the {isJs && keyboardspacer} line. Was wondering if you had any thoughts on how to fix it.
As for what I'm trying to achieve. On iPads I'm using something similar to this https://blog.typework.com/react-native-universal/ to approximate a split screen app.  Which works well, except for popups. I'd like to limit them to either side of the split screen, rather than covering both sides.
This nice little component is one of the few that will do this, but unfortunately the keyboard spacer doesn't work.
Thanks for response

This is actually an issue of react native with the iPad, as it is mentioned
in the readme file: https://github.com/facebook/react-native/issues/7340.
To overcome this, you can pass a property called screenHeight and compute
the screen height by yourself.
On 2 May 2017 at 18:21, samcolby<EMAIL_ADDRESS>wrote:

Hi,
Sorry, I really didn't explain that very well.
I was poking around the code trying to get the keyboardspacer to work with
the js solution and saw that it was explicitly removed with the {isJs &&
keyboardspacer} line. Was wondering if you had any thoughts on how to fix
it.
As for what I'm trying to achieve. On iPads I'm using something similar to
this https://blog.typework.com/react-native-universal/ to approximate a
split screen app. Which works well, except for popups. I'd like to limit
them to either side of the split screen, rather than covering both sides.
This nice little component is one of the few that will do this, but
unfortunately the keyboard spacer doesn't work.
Thanks for response
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/raynor85/react-native-modal-wrapper/issues/7#issuecomment-298702149,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFjD9SQrWV_-EB_Xb5V0q9UQXMHSks2vks5r12YdgaJpZM4NOSJz
.

Yes, I saw that, but that only works when isNative={true} . Which is full screen.
The keyboardspacer doesn't work if isNative={false} on an iPad or phone.

Keyboardspacer doesn't work on iPads for the react native bug mentioned
above, and only when you rotate your device, but on iPhones it works fine.
The problem you are facing is different, and I assure you the positioning
is working fine. Please tell me which version of react native you are
using, post your code and provide a screenshot.
On 2 May 2017 at 18:31, samcolby<EMAIL_ADDRESS>wrote:

Yes, I saw that, but that only works when isNative={true} . Which is full
screen.
The keyboardspacer doesn't work if isNative={false} on an iPad or phone.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/raynor85/react-native-modal-wrapper/issues/7#issuecomment-298704862,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFjD9V1voOqzqWLPMirppOkEn-fB7f05ks5r12hkgaJpZM4NOSJz
.

Hi, I'm on my phone at the moment so can't. I'll try later today or maybe tomorrow now.
I'm pretty sure that it doesn't get applied though.
You have this line in renderContainer which stops it from being added if native={false}
{isJs && keyboardSpacer}
Code to follow soonish. Thanks for help on this !

Hi,
Your completely right.  It turns out I had some flexbox issues with the way I was trying to do that yesterday.
I'm really sorry for the confusion.  That will teach me to reply on my phone!
Great little library btw.
Thanks
Sam

No problem, any feedbacks are always welcomed. Enjoy!


Long execution time for username completion

With commit bfd33e406d6b9f26cbe365bc4796df07a5fe3044 I added the functionality to complete usernames, but the method I used is slow, but the fastest I could think of. With only 5 tracked users it takes most of the time over 0.7 seconds. You can test it yourself on the current HEAD
$ time grep '"username":' $HOME/.keybase/keybase.idb/*/*/* | sed -r 's/.*"username":"([^"]+).*/\1/' >/dev/null

real    0m0.749s
user    0m0.049s
sys     0m0.079s

On a side note: keybase list-tracking is completely useless:
$ time keybase list-tracking >/dev/null

real    0m6.501s
user    0m0.925s
sys     0m0.139s

THIS IS TOO SLOW!
Has anybody an idea to get the cached names a bit faster?

canceled: Completion for node.js version ov keybase is discontinued in favor of the Go client


OK in DPAD center not working on TCL Flip 2

I'm using the TCL Flip 2 4058W and the OK button doesn't work in the messages app, for example it selects the word but then won't send the text, though in some other apps it does.

I works properly in the apps that accept a default action for this key, but unfortunatelly, many apps do not.
I guess, this is mostly an issue in messaging apps. Me too, I would want to fix it only for a predefined list of the most popular apps (Telegram, Messenger, Whatsup and whatnot) and make the key perform the "SEND" action. Detecting the application and checking if it is in the list is easy (the name is in EditorInfo.packageName), but I haven't found a way of forcing an action.
Any help with this will be appreciated.

So how does the default IME support the SEND function if the app itself
doesn't support default action, there must be a way for an IME to send the
SEND command. Alternatively, would it be possible to unpack the messenger
app and edit it to allow default action?
Thanks for any help, I'm a bit out of my depth here.
On Mon, Feb 13, 2023, 9:04 AM Dimo Karaivanov @.***>
wrote:

I works properly in the apps that accept a default action
https://developer.android.com/reference/android/inputmethodservice/InputMethodService#sendDefaultEditorAction(boolean)
for this key, but unfortunatelly, many apps do not.
I guess, this is mostly an issue in messaging apps. Me too, I would want
to fix it only for a predefined list of the most popular apps (Telegram,
Messenger, Whatsup and whatnot) and make the key perform the "SEND" action.
Detecting the application and checking if it is in the list is easy (the
name is in EditorInfo.packageName), but I haven't found a way of forcing
an action.
Any help with this will be appreciated.
—
Reply to this email directly, view it on GitHub
https://github.com/sspanak/tt9/issues/182#issuecomment-1427992772, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDPZXR2YP5WYR5TBK7A4YTWXI5ONANCNFSM6AAAAAAUZCLBDY
.
You are receiving this because you authored the thread.Message ID:
@.***>

WhatsApp and Signal both have settings on the Android versions that allow you to send a message with the ENTER key.
Signal has it disabled by default, and it seems to vary by manufacturer and client version with WhatsApp whether it is enabled or disabled by default.
Might  narrow down your research space.

I did a quick research and according to this link, there is literally nothing to do if the application does not want to process the key press event.

So how does the default IME support the SEND function if the app itself
doesn't support default action, there must be a way for an IME to send the
SEND command

I suppose the default IME emulates ENTER key press (the one on computer keyboards), instead of OK key press (the one on most phones) or it is a custom action. Currently, TT9 does not work with custom actions and does not send ENTER if there is no default action. I could try one of these or both, but beyond that, I am affraid, nothing else is possible.

I connected the phone to scrpcy and was able to type using my keyboard and the ENTER key doesn't do anything (newline is SHIFT+ENTER) so it must be a custom action, but I don't know much about that.

This bug is also in Qin1S+. But, a long press on the center, functions as send

I've just learned that on the S22 Flip, there is a translation layer always running (except when it crashes, which is how I found out it existed at all) for the hardware keys.  It might be possible that the same thing is happening on other phones, which might explain all the varied types of behaviors being observed.

Thanks everyone for the comments. You gave me some ideas and I've found a way to make OK work at least as good as in Gboard or OpenBoard. Still, some applications such as Messenger refuse to send when the button is pressed, but I don't think anyone can do anything about it.

I have released beta-17.4 which fixes the problem. Note that in messaging applications "send with ENTER" must be enabled, otherwise they just ignore OK key. Try it out!

In the TCL Flip 2 SMS app, even the newer version doesn't work. this can be bypassed by pressing options, then send. just a bit annoying.
I will try mapping another key instead of the OK key and see if that helps

@sspanak if you can point me to the the code responsible for that it would be amazing. thanks.

@biden2020prez, this is not a bug. The default SMS application on Android (and some other apps as well) actually request the keyboards not to send anything with OK or ENTER, so I decided to honor the setting and disabled sending in such cases.
In other words, it is what the applications author wanted. It may be inconvenient in some cases, but it may be a good idea to save the user form a mishap in others. It depends.
Of course, it is possible to ignore the app request, but I haven't considered such option in TT9 so far.

I have solved this issue for this phone specifically by changing line 619 in TraditionalT9.java from sendDownUpKeyEvents(KeyEvent.KEYCODE_ENTER);  to sendDownUpKeyEvents(KeyEvent.KEYCODE_DPAD_CENTER);

@DarthFlip Nice job! I think that was a bug fix for the Cat S22 Flip, which couldn't select words with the D-Pad.
Good catch. Now we need to make it a setting probably. I wonder if I can figure out how.

OK, I am reopening this as it seems the issues still persists in some cases. Android is really confusing with all these alternative keys having almost similar, but not exactly the same functions. I may have gotten it wrong again.

Now we need to make it a setting probably. I wonder if I can figure out how.

This must not be a setting. The OK key must just work by sending the correct key code. Or maybe it should send both DPAD_CENTER and ENTER in order to cover all cases... I really don't know... :smile:

I think that was a bug fix for the Cat S22 Flip, which couldn't select words with the D-Pad.

The code there has nothing to do with selecting the suggestions.

@biden2020prez, @DarthFlip, could you please try out this APK and let me know if OK is working for you?

Still doesnt work to send text message by clicking Dpad Center. It works fine (and always has worked fine) for selecting things.
What did you change? When I made

changing line 619 in TraditionalT9.java from sendDownUpKeyEvents(KeyEvent.KEYCODE_ENTER);  to sendDownUpKeyEvents(KeyEvent.KEYCODE_DPAD_CENTER);

It worked fine to send messages.
I didn't go through all the code but can it just be switched to DPAD_CENTER ?

Or maybe it should send both DPAD_CENTER and ENTER in order to cover all cases...

You could try that, would it potentially cause a double click?

What did you change?

I ignored the application request to disable the key. Now OK always performs the target action and on my phone it is able to send an SMS.
But if this doesn't work for you, it means your SMS app works differently. It only relies on receiving the DPAD_CENTER key code.
I wish it was as simple as replacing the key code, but the problem is, using ENTER instead of DPAD_CENTER breaks sending messages in common apps like Viber, Telegram and likely WhatsApp. And I don't want to risk sending both, because it may indeed cause double click.
This is tricky. I see these possible solutions:

Leave it as it is. Virtually no one that I know sends SMS anymore, including myself, so a slight inconvenince for a very rarely used function is not a big deal.
Pressing OK works like now (ENTER), but holding would force any defined action, or if the app reports no action, send DPAD_CENTER. There will be a setting for controlling this behavior called "Force Send Messages" and located on the Hotkey screen. The possible options would be: no key (default), hold OK, F1, F2, ... or other unassigned special keys (only available on some phones).

I don't want to make force sending the default action for holding OK, because it may be used for something else in the future.
I am also open for alternative ideas, if you have any.

@sspanak Maybe you can do a different function for long press sms dpad center and short press? I know many people who send SMS.
Keep in mind you are developing for keypad phones with touchscreen, but many people use this on old phones with no touchscreen.
But I guess it would be nice to confirm whether or not it works on other phone's SMS app.

@biden2020prez , @DarthFlip , I've come up with a hacky way sending both DPAD_CENTER and ENTER without causing a double click (hopefully!). Could you please double check if this APK allows sending SMS on your phones?


Stuck in conda installation

Conda installation unsuccessful when using the command provided in the instructions: conda install -c bioconda haystac. Returns the following then gets stuck indefinitely on "Collecting package metadata
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): |
System: Ubuntu 14.04.5 LTS

Can you check if the command conda install -c conda-forge -c bioconda haystac works? I experienced the same issue and this fix helped.

Thanks, I have tried both suggestions, but the first still gets stuck in the same step and the second prompt an installation error.

Can you check if the command conda install -c conda-forge -c bioconda haystac works? I experienced the same issue and this fix helped.
If that doesn't work, you could also try python -m pip install git+https://github.com/antonisdim/haystac.git. This was from the file example/tutorial_example.sh

Hello @alvesmjoel,
I hope you are doing great and apologies for the super delayed response !
We have tried to reduce the list of dependencies needed for the installation of haystac. Could you possibly follow the updated installation instructions provided on readthedocs, and try to install the latest version of haystac from github please ?
Please let us know if you face any other issues !
Best,
Antony


Loading StateSpaceEcon

I am trying to run the SW07 example as provided. I did the necessary installation by running the code below in the working directory:
mypath = dirname(@FILE)
using Pkg
Pkg.activate(mypath)
pkg"instantiate"
using StateSpaceEcon
using ModelBaseEcon
using TimeSeriesEcon
When I start running the Main.jl file it runs fine the point of using parameters(m), variables(m), equations(m).
Also the @link macro is not recognized?
What should I do differently?

Compat sections have been updated


Bulk MarcXML upload not working (staging & prod)

We have been unable to upload marcXMLs in zip files in both staging and production Goobi though individual uploads still work.
Expected behavior:

Go to process templates, select JP2_Manual_Edit_METS with Archives Digitisation as the project
Choose File Import Upload tab
Choose the plugin_import_fileupload_wellcome from the drop down menu
Click Choose File and navigate to processed_files.zip on hard drive
Click Save
GoobiScript activates and creates 34 processes

Current behavior:

Go to process templates, select JP2_Manual_Edit_METS with Archives Digitisation as the project
Choose File Import Upload tab
Choose the plugin_import_fileupload_wellcome from the drop down menu
Click Choose File and navigate to processed_files.zip on hard drive
Click Save
A 100% status bar is generated across the screen and completes but
GoobiScript is not activated and
The 34 are not created

There is no error message either on production or staging Goobi to indicate what error has occurred.
However, a 504 gateway timeout screen did appear in production Goobi after I clicked save and left it for sometime
I have attached the exact zip file to test with for you.
processed_files.zip

I found and fixed the issue. It is already installed on the staging system.
The reason for this is that you files start with '.', which means in the linux world that it is a hidden file. For another customer we had to ignore problematic files like thumbs_db or hidden files from our default folder listing method. Therefore all xml files where skipped when the zip was read. I implemented a separate method for the import where hidden files are allowed.

Oh that makes complete sense! I'm not sure why Branwen's files are exported from sierra with the . in front of them. Just to clarify though, is the exception you made just for the marcXML upload?
I know we've had to have the S3 ingest ignore the macosx hidden files and stuff so I wouldn't want this to conflict.

yes, the exception is only used for the marc import

Nice! Ok. I've just tried to import the process_files.zip attached to this page. The 34 items were successful and then, I think they tried to run again so I have 34 unsuccessful. I think it was user error because Branwen has tested a few small zips without any errors so I think this is working now. I think we can deploy to prod.

We need to deploy a new goobi container to install it. This results in ~15 min downtime.
When is a good time to do this?

I'm the only one in Goobi at the moment and heading for lunch soon. We could do it now or later this afternoon when you're ready?

I have installed the update, Goobi is up and running again

Great, thanks! I have let Branwen know and she'll add her marcXMLs sometime soon and let me know.

Branwen has been able to upload a zip of marcXMLs on prod Goobi as expected.


fix(deps): update dependency react-transition-group to v4.4.1

This PR contains the following updates:

Package
Type
Update
Change

react-transition-group
dependencies
minor
4.3.0 -> 4.4.1

Release Notes

reactjs/react-transition-group
v4.4.1
Compare Source
Bug Fixes

transition SSR (#619) (2722bb6)

v4.4.0
Compare Source
Features

add nodeRef alternative instead of internal findDOMNode (#559) (85016bf)

react-transition-group internally uses findDOMNode, which is deprecated and produces warnings in Strict Mode, so now you can optionally pass nodeRef to Transition and CSSTransition, it's a ref object that should point to the transitioning child:
import React from "react"
import { CSSTransition } from "react-transition-group"

const MyComponent = () => {
  const nodeRef = React.useRef(null)
  return (
    <CSSTransition nodeRef={nodeRef} in timeout={200} classNames="fade">

Fade

    </CSSTransition>
  )
}

Bug Fixes

set the values of constants attached to Transition to match the exported ones (#554)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

PR has been edited
:construction_worker: This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.

:tada: This PR is included in version 5.0.43 :tada:
The release is available on:

v5.0.43
GitHub release

Your semantic-release bot :package::rocket:


[Core] Validate and enrich raw configs using module

Add support for config validation and transformations by resources module

For now we have decided to move the scope of transformations made to the apply stage itself.
We plan to keep these enrichments and transformations light and have no dependency on any external service.


error while using view push commands

i am getting error "cannot perform on non tty device " while using "aws ecr get-login-password --region us-west-2 | docker login --username AWS --password-stdin .dkr.ecr.us-west-2.amazonaws.com/studentcoursesevice"
p.s - i removed my account id above for security purpose

goto your aws ecr then select your repository
click on view push commands
then copy the login commad of your ecr and paste it in your terminal.
this command is helpful for login into your ecr


fix(emitter): wrong package name in trace logs

Description
fix a regression when back porting changes from microsoft/typespec in pr: https://github.com/Azure/autorest.csharp/pull/4705
fix #4751
Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

updated package name


Facebook insists on formal application review or will ban the bot channel on Dec 16.

Hi
We have bot which is working in Workplace by Facebook environment for more than 1 year. The channel created as Custom Integration in Workplace and corresponding token passed to the Azure Bot Channel Registration.
Now this integration has the following warning "This integration has not yet completed this updated review process. On December 16, 2019, we will remove access to any integration that has not passed our updated review process."
I used Workplace Direct Support to convince them that it's just custom integration and not a SaaS for other Workplace customers but without success.
Direct Support answer is "According to our system this particular bot is hosted by 'Microsoft Bot Framework', you will have to work with Microsoft on creating a third party app."
Could you please suggest what to do to avoid ban in December?
Best Regards
Valerii

Hello @Valerii-Poliakov
Microsoft is currently working to make sure your bot will continue to work. The only thing I know of that's required from your end is to enable your integration:

I will ask for more information and let you know if there's anything else you need to do.

Thank you for response. Waiting for updates.

Hi @Valerii-Poliakov -
You're correct, starting December 16, 2019, Workplace by Facebook is changing security model for custom integrations. Current integrations built using Microsoft Bot Framework v4 on JavaScript/Node.js may be updated to use the Bot Framework Facebook for Workplace Adapter and deployed using Azure Web App to continue working past that date. New Microsoft Bot Framework bots that are targeting Workplace should also be developed using that adapter.

Hi @dandriscoll
So, I have to migrate my bot code from ver. 3.15 to ver. 4.x and use "Bot Framework Facebook for Workplace Adapter" in order to avoid Dec 16th ban. Am I understand right?
Best Regards
Valerii

That's correct @Valerii-Poliakov. Closing as resolved. If you have questions arise during the 3.5 > 4 migration, please open an issue in the BotBuilder v3 repo and add a reference to this issue. Someone on the Bot Framework Support Engineers team will assist you.

Hello @dandriscoll !
I want to provide a third party bot on Workplace (so no custom integration). I am currently using the REST API for our bot and we configure it on the Azure Bot Channel Registration portal for other channels (Slack ..).
I understood from the Workplace integration doc that the deployment process changed, in particular :

Page ID is not needed anymore to integrate the app
There is a complex process to get the token needed for the API calls on the Facebook Graph API

However, the Bot Channel Registration portal has not changed : it's mandatory to provide a Page ID, it does not seem to do the OAuth redicrection correctly ...

My questions are :

Are there any backward compatibility for the Bot Service portal integration with new Workplace process ? If so, how can we configure that (doc does not details how to do it for 3rd party apps)
If not, is there any plan in the BotFramework team to adapt the portal for that new process ? Or are we obliged to move to a self hosted bot using the adapter you mentionned earlier ?

Best Regards !


How to run spark cep with kafka topic

Hello,
I am trying to run spark-cep job. I have created the jar and trying to submit it using spark-submit and below query.
SELECT COUNT(DISTINCT t.duid) FROM stream_test OVER (WINDOW '300' SECONDS, SLIDE '5' SECONDS) AS t
It says that query could be made corresponding to a kafka topic (stream_test in your case). Kafka does not stores information about a column so how does it queries for distinct duid?
Please suggest how to run this.
Thanks
Janesh Mishra

Hello, Janesh
Thank you for your interest of Spark CEP.
Spark CEP architecture is based on Spark SQL and stores the aggregated info to RDDs which is the in-memory data structure of Spark.
to use a kafka topic as a hive table, you should execute a DDL query with kafka configurations.
Regards,
Robert


docs(v2): change docs for markdown features

Motivation
This PR fixes the templates used by @docusaurus/init.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan

Created a new project using @docusaurus/init using the fixed templates.
Previewed updated docs.

@yangshun thanks for you additions ;)  I think that we need more  <!-- prettier-ignore --> comments. Have a look at this change from you commit: https://github.com/facebook/docusaurus/pull/2786/commits/436035bb5dcab5e631fd9e3106ba6b5707f39be0#diff-e59546939eacd6ba19a807a2d1896dbbR54

Thanks for catching! Fixed in 20fbceb9f. I'm removing the paragraph example as it's obscure syntax.


Build HTML pages locally - inform the change directory

Current text:
Inside the cloned repository, run sphinx-build . _build
May be explicitly say
Inside the cloned repository, cd into docs directory and then ....
Or
Inside the cloned repository, run sphinx-build doc _build

Please check whether this does not conflict with my open PR: https://github.com/coderefinery/documentation/pull/88

What I mean: please merge my change first since the "current text" might have changed - but I do agree with explicit better than implicit.

that's a good change, there was a bit of confusion by some participants who tried sphinx-build in the root directory of the repo. I think the former is better: "cd into docs directory and then ...."


fix: Documentation release

Closes https://github.com/lenra-io/components-api/issues/117
Please note that I am not 100% sure that the semantic-release action will wait for the documentation action to be done before releasing. This could make the CI fail.
Checklist

[x] I didn't over-scope my PR
[x] My PR title matches the commit convention
[x] I did not include breaking changes
[ ] I included unit tests that cover my changes
[ ] I added/updated the documentation about my changes
[x] I made my own code-review before requesting one

Description of the changes

Technical highlight/advice

@jonas-martinez To ensure that the doc artifact is run before the sementic-release, you can use two jobs in the same wokflow. Then you can use the need keyword to ensure that the pre-release that create the artifacts is done before.
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobs
https://stackoverflow.com/a/63148947

:tada: This PR is included in version 1.0.0-beta.58 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-beta.58 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-beta.58 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-test-ci.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-beta.58 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Compatibility with Yosys 0.39

struct Yosys::AST::AstNode changed recent.y, so Synlig unfortunately will currently not compile with Yosys 0.39

Yosys::AST::AstNode does not have a multirange_dimensions anymore (or it is renamed)
Similarly multirange_swapped
Also make_struct_member_range

Are there any updates on this? Synlig is the only thing keeping us on 0.38…

Thank you for taking interest and working on it in: https://github.com/chipsalliance/synlig/pull/2496. While it fixes compilation with newer yosys, we also need to make all tests pass there.
Please note, we are also working on updating yosys in: https://github.com/chipsalliance/synlig/pull/2487 and in upcoming PRs. We took different approach there: we are updating yosys to next version that is breaking our tests and fixing issues one by one, so it might take a while while we will support upstream yosys.
Having said that, if you will manage to get all tests passed in your PR, I'm happy to merge it too.

I expect your approach to probably work better lol. I'll see what I can do on my end.

We managed to update compatibility to yosys 0.44 (current upstream): https://github.com/chipsalliance/synlig/pull/2510
Closing this issue at this point. If you will encourage any new problems with updated yosys version, feel free to open new issue.


First attempt at InterledgerPacket

This PR is a first-attempt at the InterledgerPacket and ILQP Packet interfaces.  The ILQP interfaces are not complete, but are instead stubs to help us determine the best design for these classes.
Design Questions

There are similarities between the overall InterlederPacket, and its children (InterledgerPaymentPacket.java and InterledgerQuotingProtocolPacket.java).  This PR unifies them, but perhaps there's an argument in favor of making them completely separate classes in the Java heirarchy.
We should create some concrete implementations of the interfaces in an attempt to see how accurate/desirable the interface design is.  I plan to do that later this week, but I encourage others to do the same.

Design Note
This PR chooses to model packets as interfaces instead of concrete classes.  This is to allow developers to choose implementations of these packets using different styles and/or technologies.  For example, @sappenin might create implementations based upon the Immutables Project, while others might create implementations using other libraries like Lombok or even Java POJOs.
To this end, java-ilp-core should be flexible enough to accommodate various technologies chosen by implementors, while at the same time providing default implementations to aid developers who may simply want something "out of the box" -- see InterledgerAddressBuilder.java for one example of how this project might accomplish that.

I think it doesn't match the specs at https://github.com/interledger/rfcs/tree/master/asn1
There is an InterledgerPacket containing the type and the specific packet for each protocol
(Payment, PaymentRequest, QuoteResquest, QuoteResponse).
I think using stream readers/writers (extending the OER reader/writer) will simplificate the code.
As I see it protocol paquets must not be aware of the InterledgerPacket (must not extends it).
I send a summary of the specs at https://github.com/interledger/rfcs/tree/master/asn1
for clarity:
PacketSet PACKET ::= { {1 InterledgerProtocolPaymentMessage} | {2 QuoteLiquidityRequest} |  ... }
-------------------------------------------+----------------------------------------
InterledgerPacket ::= SEQUENCE {           | InterledgerProtocolError ::= SEQUENCE {
    -- One byte type ID                    |     code IA5String (SIZE (3)),
    type PACKET.&typeId ({PacketSet}),     |     name IA5String,
    -- Length-prefixed header              |     triggeredBy Address,
    data PACKET.&Type ({PacketSet}{@type}) |     forwardedBy SEQUENCE OF Address,
}                                          |     triggeredAt Timestamp,
                                           |     data OCTET STRING (SIZE (0..8192))
                                           | }
-------------------------------------------+----------------------------------------

-------------------------------------------+----------------------------------------
InterledgerProtocolPayment ::= SEQUENCE {  | InterledgerPaymentRequest ::= SEQUENCE {
    amount UInt64,                         |     version UInt8, -- IPR Version
    account Address,                       |     condition UInt256, -- Cryptographic condition
    data OCTET STRING (SIZE (0..32767)),   |     packet OCTET STRING -- ILP Payment Packet
    extensions SEQUENCE {  ... }           | }
}                                          |
-------------------------------------------+----------------------------------------

QuoteLiquidityRequest ::= SEQUENCE {           QuoteLiquidityResponse ::= SEQUENCE {
    requestId UInt128,                             requestId UInt128,
    destinationAccount Address,                    liquidity LiquidityCurve,
    destinationHoldDuration UInt32,                appliesToPrefix Address,
    extensions SEQUENCE { ...  }                   sourceHoldDuration UInt32,
}                                                  expiresAt Timestamp,
                                                   extensions SEQUENCE { ...  }
                                               }

QuoteBySourceAmountRequest ::= SEQUENCE {      QuoteBySourceAmountResponse ::= SEQUENCE {
    requestId UInt128,                             requestId UInt128,
    destinationAccount Address,                    destinationAmount UInt64,
    sourceAmount UInt64,                           sourceHoldDuration UInt32,
    destinationHoldDuration UInt32,                extensions SEQUENCE { ...  }
    extensions SEQUENCE { ...  }               }
}

QuoteByDestinationAmountRequest ::= SEQUENCE {  QuoteByDestinationAmountResponse ::= SEQUENCE {
    requestId UInt128,                              requestId UInt128,
    destinationAccount Address,                     sourceAmount UInt64,
    destinationAmount UInt64,                       sourceHoldDuration UInt32,
    destinationHoldDuration UInt32,                 extensions SEQUENCE { ...  }
    extensions SEQUENCE { ...  }                }
}

Re-reading @earizon 's comments I think I agree with him. We should maybe have some simple business objects that represent the packets and leave complexity like mapping between type ids etc to readers/writers.
I'd go so far as to define the data member of an InterledgerProtocolPayment class as simply a byte[].

I'm going to close this PR - the feedback in here was great, so I'll just open a new PR when that's ready.