ng serve not detecting file changes

Please provide us with the following information:

OS?  Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)
ubuntu 16.04
Versions. Please run ng --version. If there's nothing outputted, please run
in a Terminal: node --version and paste the result here:
angular-cli: 1.0.0-beta.16
node: 6.6.0
os: linux x64
Repro steps. Was this an app that wasn't created using the CLI? What change did you
do on your code? etc.
created new project nothing changed only changed the title
The log given by the failure. Normally this include a stack trace and some
more information.
Mention any other details that might be useful.

While updating the source files the build is not auto updating the app

Thanks! We'll be in touch soon.

I face the very same issue except that my angular-cli version is 1.0.0-beta.15.
It used to work in a previous version on the same computer

Same here on
OS: Windows 10
angular-cli: 1.0.0-beta.16
node: 6.6.0

Maybe a duplicate of #2356 - does the path to your project folders include a space?

@bonnici  no the path doesn't have any space or special character. The path is /home/harilal/works/ng2/routing/
To my wonder after removing global angular-cli and removing the nod-modules and reinstalling after cleaning the cache it worked. I need to check whether it will work next time.

At my side, I found the issue: the system that detects changes (inotify) can't handle so much watches by default.
To change the amount of watches it can handle (=the maximum amount of files that will be in the project) you must run this command:
echo 65536 | sudo tee -a /proc/sys/fs/inotify/max_user_watches
The problem with inotify is reseting this counter every time you restart your computer.
In the previous version, there was an alert to tell that the whole project couldn't be observed. So we knew we had to run this command. It's not the case anymore.
Something, somehow, somewhere should be done so that this file detection always works out of the box on linux.

@paillave  I think that's solved my issue. Now the project files gets compiled and it's updating. But i need to check after restarting the system. Last time when i reinstalled the angular-cli it's worked. BUt next day it's stopped working. I will update the issue once i have tested that also.

@harilalinapp out of angular 2 app, many have the same issue with inotify, especially on ubuntu: http://askubuntu.com/questions/716431/inotify-max-user-watches-value-resets-on-reboot-how-to-change-it-permanently,
Here is the solution... (dunno it this actually works):
https://bbs.archlinux.org/viewtopic.php?id=193020
or
https://github.com/guard/listen/wiki/Increasing-the-amount-of-inotify-watchers

@paillave Thanks for the support. I have to run the command each time at restart. Anyway it's works thanks.

Thank you @paillave seems to be an issue on Ubuntu and the fix is
echo fs.inotify.max_user_watches=524288 | sudo tee -a /etc/sysctl.conf && sudo sysctl -p
as highlighted on
https://github.com/guard/listen/wiki/Increasing-the-amount-of-inotify-watchers

Closed in favor of https://github.com/angular/angular-cli/issues/1610, linking @paillave's solution there.

just remove spaces and special characters from all root directories of project directory(inclusive).

@paillave  thank you, works fine!!

Same issue,
Angular CLI : 1.0.0-beta.28.3
Node: 7.5.0
OS : Win32 x64
No spaces to app .. here's the path : c:\dev\myapp
I reinstalled node and angular a couple of time but nothing worked.

I am having a similar issue. Changing a .html file works fine. Changing a .ts file, the compiler fires but on reload there is no change.

Same here as @raducostea when .ts file is changed, the compiler fires, but the Hash (in terminal) is the same as before file change. I use Webstorm 2016.3.3 on Win 7 x64. Also tried to reinstall, set new project.

Same as  https://github.com/angular/angular-cli/issues/4338

The same happens to me, the html takes the changes but the .ts does not

I think problem is in WebStorm. Webpack watch doesn't work if the file is not saved directly. Please try turning 'Safe write' option ( Settings | Appearance & Behavior | System Settings | Use "safe write" (save changes to temporary file first)) off. After this change all works fine for me.

It is a bug in the latest CLI. Execute the following command in your project folder:
npm install --save<EMAIL_ADDRESS>and
ng serve
And it should work fine

@toni8810 , thank you!

@toni8810 thank you!

@toni8810 Thanks!  Fixed the problem.  I assume we'll want to remove that specific version once the bug is fixed in the next release? Thanks again!

This is fixed for me in the @angular/cli latest release.

@raducostea thanks!

I've noticed a strange case sensitivity bug.
My casing was incorrect e.g. (export * from './MyHeader'), instead of (export * from './myHeader').  Would compile and render without any errors, but ng serve was not detecting file changes.

I checked the latest (beta 30) release and it doesn't pick up changes in a lazy loaded shared module without the npm install --save<EMAIL_ADDRESS>fix

I faced with the same problem on Ubuntu 16.04 and angular CLI 1.0.0-beta.30
The problem was related with Inotify Watches Limit on Linux.
To solve to issue I increased  the watches limit to 512K
sudo sysctl fs.inotify.max_user_watches=524288
sudo sysctl -p --system

After that I restarted my IDE (WebStorm in my case) and after that the change detection started to work.

sudo sysctl fs.inotify.max_user_watches=524288
sudo sysctl -p --system
That fix the problem for me. Thanks @ulymor!

My node_modules folder contains 27487 files in total. I suspected Webpack was also listening for each file in this library, which is nonsense since they never change.
I was able to fix the issue without having to change my system configuration by changing tsconfig.json to exclude node_modules:
{
    "compilerOptions": {
        ...
    },
    "angularCompilerOptions": {
        ...
    },
    "exclude": [
        "node_modules"
    ]
}

You likely don't want to increase fs.inotify.max_user_watches to half a million. Each file watcher takes up 1kB of kernel memory (unswappable). Half a million of those watchers would take up 512MB of RAM, which no other program can use. The above method might also fix your issue without giving up so many system resources.

I had this issue and also found that I was getting a warning from WebStorm about inotify being set to low, It linked to this: https://confluence.jetbrains.com/display/IDEADEV/Inotify+Watches+Limit which has fixed it for me on Ubuntu 16.04

I stumbled onto this issue trying to get the detection working in a Vagrant VM (Ubuntu 16.04). This plugin worked flawlessly for me right out of the box with no additional commands: https://github.com/mhallin/vagrant-notify-forwarder
Hope this helps someone else!

I have noticed changes detected only on Existing Method or variables, is not detecting new methods and variables. I hope the others also having the same issues.

I have same issue,
macOS Sierra
Version 10.12.5
npm 5.3.0
@angular/cli: 1.2.1
node: 8.0.0
Thanks

Got the similar issue,
New methods in ts file not reflected,
@angular/cli: 1.3.2
node: 8.4.0
os: win32 x64
@angular/animations: 4.3.6
@angular/common: 4.3.6
@angular/compiler: 4.3.6
@angular/core: 4.3.6
@angular/forms: 4.3.6
@angular/http: 4.3.6
@angular/platform-browser: 4.3.6
@angular/platform-browser-dynamic: 4.3.6
@angular/router: 4.3.6
@angular/cli: 1.3.2
@angular/compiler-cli: 4.3.6
@angular/language-service: 4.3.6
Do any one know the solution?

Can anyone other than @filipesilva continue this discussion in another thread? This isn't working. https://github.com/angular/angular-cli/issues/8033#issuecomment-336305650

thank you ...

I am also facing the same issue. ng serve is not getting reloaded when changes made in a particular directory Model. Apart from those Model classes it is detecting the changes..
I ran the command ng --version and got this result..
@angular/cli: 1.3.2
node: 6.11.2
os: win32 x64
@angular/animations: 4.4.3
@angular/cdk: 2.0.0-beta.11
@angular/common: 4.4.3
@angular/compiler: 4.4.3
@angular/core: 4.4.3
@angular/forms: 4.4.3
@angular/http: 4.4.3
@angular/material: 2.0.0-beta.11
@angular/platform-browser: 4.4.3
@angular/platform-browser-dynamic: 4.4.3
@angular/router: 4.4.3
@angular/cli: 1.3.2
@angular/compiler-cli: 4.4.3
@angular/language-service: 4.4.3
Please help me to fix this issue..

I am using angular-cli 1.6.3, and having the same issue, but only my directive's code has this problem, I tried to modify the code, remove the code completely, restart my desktop but the ng serve always load the old directive code

@jeanpaulattard thanks, it helps me with PhpStorm too

I'm having this issue not on Linux but Windows 10 machine and VS Code. Anyone solutions for that platform?

@paillave thanks man you had rescued me :+1:  :thumbsup:

I am also on Windows 10 and all of the above mentioned solutions are not working for me.
I tried it with Ubuntu 18.04 in WSL and Arch Linux in WSL

I'm having same issue on Windows 10 and tried exluding the huge node_modules in tsconfig.json, but still shows up old display. I see that the typescript file does not transpilate into an updated .js

it's works for me @paillave, I was working with Linux CentOs 7; node -v: v8.11.3; npm -v: 5.6.0

I have the same issue on Kubuntu 18.04, npm 5.6.0, node 8.11.4, Angular CLI: 6.1.5.
I confirm the @vignesh007vicky solution works fine: https://github.com/angular/angular-cli/issues/2389#issuecomment-377191808 , thanks.

Same here (HTML changes automatically is rebuilded, TS changes nothing happens). I'm using Ubuntu 18.04. @chapultepec's answer works fine for me, but we SHOULD NOT have to do this... Thanks @chapultepec.

sudo ng serve works!

Angular CLI: 6.2.8
Node: 8.10.0
OS: linux x64
Angular: 6.1.10
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.8.8
@angular-devkit/build-angular     0.8.8
@angular-devkit/build-optimizer   0.8.8
@angular-devkit/build-webpack     0.8.8
@angular-devkit/core              0.8.8
@angular-devkit/schematics        0.8.8
@angular/cdk                      7.1.1
@angular/cli                      6.2.8
@angular/material                 7.1.1
@ngtools/webpack                  6.2.8
@schematics/angular               0.8.8
@schematics/update                0.8.8
rxjs                              6.2.2
typescript                        2.9.2
webpack                           4.16.4

I had the same issue working on Vagrant vm on Ubuntu 16.04 LTS.
Angular CLI: 6.0.8
Node: 11.10.0
OS: linux x64
Angular: 6.1.7

I tried all solutions, increasing notify watches, using --watch reinstalling npm and node. Nothing worked.
This is what solved it for me. adding --poll=2000 like so:
ng serve --poll=2000 --host <IP_ADDRESS>

If you're using Vagrant and the fs-notify plugin doesn't work, I've also had luck explicitly specifying chokidar for polling:
export CHOKIDAR_USEPOLLING=true && ng serve ...
Obviously you can set that environment var anywhere, but for a disposable VM it's quick and dirty to just add it to the npm script.

Ubuntu 18.04
Delete node_modules and run npm install again.

@toni8810 Thank you!! :+1:


[Dev] Fix small bug where you can't turn off analytics popovers

Currently you need to be showing red dots for martsy views in order to turn off analytics popovers 😄

👍


Dell 5450/5550 Issues

Crossposting, as its probably more relevant here.
The Automation Tool is bringing down the Old E5450/E5550 driver packs (A08/A09) rather than the new Latitude 5450/5550 A00 CAB's from Dell.

Despite clicking the Latitude 5550, It actually brings down E5550/5550 which is not the correct cab.

Having near to the same issue. 5450 isn't even downloading the cab:

Other driver packs work. So it does indeed looks like the script is confused between E5450 and 5450 and Windows 11.

We have the same problems on 5350/5450/5550. Does anyone have a solution to this problem or a workaround? Have any of you reported this to Dell?

The workaround is to manually download the correct package and create a custom package for each model as per the Driver Automation Tool Documentation and then unselect those models in the make and model selection page.
Dell would probably be uninterested. And any change to the naming of their driver package would likely take an age. But it might be worth raising is someone has the ear of someone in their Enterprise team.

Same Issue Here.. Took quite a while to figure this was the issue.

Does anyone have the baseboards handy for these models? It won't let me create the custom package without them and we don't yet have the physical devices available.

I presume you are referring to the SKU.
We have some already, here they are:
5450 (Models included;0CB2;0CB4)
5550 (Models included:0CB9;0CBC)

I presume you are referring to the SKU. We have some already, here they are:
5450 (Models included;0CB2;0CB4) 5550 (Models included:0CB9;0CBC)

Thanks! There's also a baseboard number the tool asks for that can be pulled using the following powershell command on a device: (Get-CIMInstance -ClassName MS_SystemInformation -NameSpace root\WMI).BaseBoardProduct

This is an issue with the Dell XML. The driver package data for the Latitude E5450 is missing the E within SupportedSystems. If you review the DriverautomationTool.log you can see the incorrect additional Dell System Model ID

Dell System Model ID is : 0645;0CB2;0CB4
The Model ID of 0645 is for the Latitude E5450.
With this incorrect data, multiple other parts of the tool break since it uses the first value in the list.

I was able to manually modify the DriverPackCatalog.xml to correct this mistake and the tool was able to download and build the Latitude 5450.  I believe there are other models in a similarly broken state.

This is an issue with the Dell XML. The driver package data for the Latitude E5450 is missing the E within SupportedSystems. If you review the DriverautomationTool.log you can see the incorrect additional Dell System Model ID

Dell System Model ID is : 0645;0CB2;0CB4
The Model ID of 0645 is for the Latitude E5450.
With this incorrect data, multiple other parts of the tool break since it uses the first value in the list.

I was able to manually modify the DriverPackCatalog.xml to correct this mistake and the tool was able to download and build the Latitude 5450. I believe there are other models in a similarly broken state.

Awesome find! I had done some looking into the XML but wasn't sure 100% where to begin, but this makes perfect sense. It looks like you can also tell which is the old E- series by comparing the rstDate= parameter as well.

This is an issue with the Dell XML. The driver package data for the Latitude E5450 is missing the E within SupportedSystems. If you review the DriverautomationTool.log you can see the incorrect additional Dell System Model ID

Dell System Model ID is : 0645;0CB2;0CB4
The Model ID of 0645 is for the Latitude E5450.
With this incorrect data, multiple other parts of the tool break since it uses the first value in the list.

I was able to manually modify the DriverPackCatalog.xml to correct this mistake and the tool was able to download and build the Latitude 5450. I believe there are other models in a similarly broken state.

So you just need the "E" for each of the names that show up when I search for systemID="0645" ?
Before:
After:

Yes, adding that should resolve the Latitude 5450. I believe it is there 2 times, once each for Windows 10 and Windows 11.

Good spotting. I started looking through there, but it was a bit overwhelming.
I have raised this via our Account Exec, hopefully he can put me on to the Enterprise CAB team.

I was able to successfully build the Latitude 5550 with these steps

Delete any cab and xml files in the Driver Automation Tool\Temp  directory
Launch the tool to download the latest versions
Modify the xml as mentioned
Close and relaunch the tool
Verify the xml file is still the modified version
Start the download

FWIW, I brought this up during my school district's weekly Dell meeting. The tech on the call said the real fix is to use the System Model ID, which will always be different. But he will bring it up with a contact on the team that deals with the XML. Who knows when it will get updated, though?

I was able to successfully build the Latitude 5550 with these steps

Delete any cab and xml files in the Driver Automation Tool\Temp  directory
Launch the tool to download the latest versions
Modify the xml as mentioned
Close and relaunch the tool
Verify the xml file is still the modified version
Start the download

I just tested and can confirm that the steps above worked. Thanks!

Due to the DAT issue we pulled drivers manually and added to task sequence. But after imaging to Win 11 and run Dell command update, it completely disable NIC drivers. Does anybody facing this issue? I'm thinking Dell command update pulls old drivers for latitude 5550/5450 machines.


Fast autocomplete on Win10

Hello!
I wanted to try out fast autocomplete in one of my projects, running on Python 3.6 and Windows 10.
I installed all fast autocomplete packages correctly, but on importing AutoComplete I get an error:

from fast_autocomplete import AutoComplete triggers ModuleNotFundError: No module named 'termios'
I looked up the 'termios' module on the web and it appears to be available only for Unix.
Is there a way to make fast autocomplete run on Windows?
Thank you!

Hi @vmc4u
We use termios only for the "demo" mode and not actual production use of fast-autocomplete. We can make it optional.

@vmc4u I just published a new version of fast-autocomplete that makes the demo code optional. Please upgrade the fast-autocomplete to the latest version.

Thank you @seperman !
I'll try the new version.
All the best!


License Information

Hi,
Really great work guys! I really appreciate the project and the efforts you guys have put into it. We would like to use the project for a commercial training online/classroom. So, I just wanted to clarify, if we acknowledge the source, would that be sufficient and covered under the terms of use?

Thanks a lot @hatlord for accepting the license changes and @einfallstoll for helping me out on a really short notice. Really appreciate it! We had planned to use it for teaching on SNMP. But had to skip it due to the license.
Thanks!


GitHub Enterprise Cloud to GitHub Enterprise Server migration

Repositories
https://github.com/casa-vega/internal-1
https://github.com/casa-vega/private-1

Target repository visibility
Private

/run-dry-run-migration


Can you

Include the visio file for the diagram above?

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

ID: f3d6aeb8-ec6a-4d27-2d10-1b404de450b7
Version Independent ID: 65b165e4-a2e4-7f57-19eb-884fce7bf880
Content: Build an enterprise-grade conversational bot
Content Source: docs/reference-architectures/ai/conversational-bot.md
Service: architecture-center
GitHub Login: @roalexan
Microsoft Alias: pnp

Hi @MCKRUZ Thank you for your feedback! We have assigned this issue to the author to review further and take the right course of action.

Yes, I'll track it down.

@roalexan did this get found?

@doodlemania2 @roalexan - I have the file.


Add Template for Kava RPC

We'd like to support being able to spin up a Kava RPC node via console. Three things need to happen for this:

[ ] Test out https://github.com/akash-network/cosmos-omnibus/blob/master/kava/deploy.yml and confirm it works (@troian )
[x] Add JSON template for kava rpc to https://github.com/akash-network/deploy-templates (@anilmurty )
[ ] Add tile for Kava to https://console.akash.network/new-deployment (@jtary ). Will also need an icon for kava (@aktdenis)

PR for the second item in the list https://github.com/akash-network/deploy-templates/pull/2

Re. 1 (@troian ) - I deployed the omnibus image using console (import SDL) and it seems fine: http://vvbb5s95a9a5j4c2qked8i3bjg.ingress.bdl.computer/net_info?

icons (from @aktdenis): https://drive.google.com/drive/folders/1v3El7Eut7ZmKpcB8hD8vQiySkN0pGvVo

PR for the template https://github.com/akash-network/console/pull/43

Tested on v0.1.245 on staging and prod and confirmed this works.
Side note that rpc node does not sync (catching_up remains true and earliest/ latest blocks show 0) but that is not a console issue


proxy doesn't work on new ionic-app-scripts 2.0.0

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
proxy doesn't work after upgrade, endpoints resolves with status code "304 Not Modified", response data is the content of the index.html
What behavior are you expecting?
proxied api should return my valid data
Steps to reproduce:

updated to<EMAIL_ADDRESS>add proxy to project
fire request to the proxied api.

insert any relevant code between the above and below backticks

Which @ionic/app-scripts version are you using?
2.0.0
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
$ ionic info
global packages:
@ionic/cli-utils : 1.4.0
Cordova CLI      : 7.0.1
Ionic CLI        : 3.4.0

local packages:
@ionic/app-scripts              : 2.0.0
@ionic/cli-plugin-cordova       : 1.4.0
@ionic/cli-plugin-ionic-angular : 1.3.1
Cordova Platforms               : android 6.1.2
Ionic Framework                 : ionic-angular 3.5.0

System:
Node       : v6.10.3
OS         : Windows 10
Xcode      : not installed
ios-deploy : not installed
ios-sim    : not installed
npm        : 3.10.10

I have the same problem, I review the difference with the previous version and I found that if you removed this line  it works
Regards

Yes I confirm, we have the same problem with app-scripts 2.0.
Proxy wan't work anymore
global packages:

    @ionic/cli-utils : 1.4.0
    Cordova CLI      : 7.0.1
    Ionic CLI        : 3.4.0

local packages:

    @ionic/app-scripts              : 2.0.0
    @ionic/cli-plugin-cordova       : 1.4.0
    @ionic/cli-plugin-ionic-angular : 1.3.1
    Cordova Platforms               : android 6.2.3
    Ionic Framework                 : ionic-angular 3.5.0-201707061803

System:

    Node       : v6.11.0
    OS         : Linux 4.10
    Xcode      : not installed
    ios-deploy : not installed
    ios-sim    : not installed
    npm        : 5.1.0

yep..broken... proxy not working

yes, confirmed proxy is not working any more.
Gives this error in my case SyntaxError: Unexpected token < in JSON at position 0 as the web api is not returning expected JSON and it seems to be parsing the html instead and producing this error.

Same here, and as @Xepe says, removing that line works.

This is fixed in nightly. I reverted the change to the http-server. That came in as a PR.
npm install @ionic/app-scripts@nightly

Thanks,
Dan

After installing the nightly I unfortunately get
> ionic serve -b --verbose
[DEBUG] Loading global plugin @ionic/cli-plugin-proxy
[DEBUG] Loading local plugin @ionic/cli-plugin-cordova
[DEBUG] Loading local plugin @ionic/cli-plugin-ionic-angular
[DEBUG] !!! ERROR ENCOUNTERED !!!
Error: Cannot find module 'fs-extra'
[DEBUG] Error: Cannot find module 'fs-extra'
        at Function.Module._resolveFilename (module.js:485:15)
        at Function.Module._load (module.js:437:25)
        at Module.require (module.js:513:17)
        at require (internal/module.js:11:18)
        at Object.<anonymous> (C:\Users\Jan\Documents\yatsa\node_modules\@ionic\app-scripts\dist\util\config.js:4:18)
        at Module._compile (module.js:569:30)
        at Object.Module._extensions..js (module.js:580:10)
        at Module.load (module.js:503:32)
        at tryModuleLoad (module.js:466:12)
        at Function.Module._load (module.js:458:3)


chore: remove unused imports

remove unused imports and the unused package express

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#12  👈
#11
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kcoderhtml and the rest of your teammates on  Graphite

Merge activity

Jun 25, 4:25 PM EDT: @kcoderhtml started a stack merge that includes this pull request via Graphite.


enhancement #4321 done with all required changes

enhancement #4321 done with all required changes

Hi @vaishaliwebkul @ghermans
refer enhancement of this PR - https://github.com/bagisto/bagisto/pull/4326

wishlist or compare icon added b/w buy now & add to cart button in RTL, kindly fix this https://prnt.sc/vvlxqq

Hi @vaishaliwebkul
Fix according to comment.
Kindly test it.


Hacked buffer memory alignment for the iOS simulator

When running in the iOS simulator, constant buffer offsets must be
aligned to 256 bytes. vkGetBufferMemoryRequirements is reporting the
wrong alignement in this case. see: https://github.com/KhronosGroup/MoltenVK/issues/1250

+1
would also apply to the tvOS simulator so could update the commit/PR title

commit message updated to specify tvOS too


vm.max_map_count [65530] is too low in Docker-Windows.

Hello all,
Im trying to run elasticsearch 5.5.0 in windows with docker, but i'm having a issue:
elasticsearch_1  | ERROR: bootstrap checks failed
elasticsearch_1  | max virtual memory areas vm.max_map_count [65530] is too low, increase to at least [262144]

Any option to avoid this error and let elasticsearch run normally?
Thanks!!

Hi guys, i will answer myself:
docker-machine ssh
sudo sysctl -w vm.max_map_count=262144
exit

Thanks!

Thanks @jlruizmlg ; this is needed when using docker-machine for Windows as opposed to Docker for Windows. Some clarifications between those two in https://forums.docker.com/t/how-can-i-ssh-into-the-betas-mobylinuxvm/10991/8.
I took a note to update the docs to mention OSX+Windows with Docker Toolbox instead of just OSX with Docker Toolbox where this procedure is explained.

I'm getting this error for Linux Mint 18 and image tag 5.5.1

@khusseini If you are not running elasticsearch in the dev mode you'll need to set the vm_max_map_count as documented in the important note in the installation docs.

@dliappis, ah alrighty, I skimmed the page a bit too fast

No worries @khusseini :+1:


[Event Request] CU96 "Purch.-Post Prepmt. (Yes/No)" - procedure PostPrepmtInvoiceYN()

    procedure PostPrepmtInvoiceYN(var PurchHeader2: Record "Purchase Header"; Print: Boolean)
    var
        PurchHeader: Record "Purchase Header";
        ConfirmManagement: Codeunit "Confirm Management";
    begin
        PurchHeader.Copy(PurchHeader2);
        with PurchHeader do begin

----------------------- ADD EVENT ----------------------
             OnBeforePostPrepmtInvoiceYN(PurchHeader);
----------------------------------------------------------

            if not ConfirmManagement.GetResponseOrDefault(StrSubstNo(Text000, "Document Type", "No."), true) then
                exit;

            PostPrepmtDocument(PurchHeader, "Document Type"::Invoice);

            if Print then begin
                Commit();
                GetReport(PurchHeader, 0);
            end;

......

[IntegrationEvent(false, false)]
local procedure OnBeforePostPrepmtInvoiceYN(var PurchHeader: Record "Purchase Header")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Finish embed migration

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Follow up to https://github.com/kubernetes/perf-tests/pull/2200.
We finished migration of all the usages. Let's remove the deprecated API.
Which issue(s) this PR fixes:
Special notes for your reviewer:
/assign @tosi3k

/lgtm
Thanks!


UI elements not responding to user inputs on iOS

I'm loading a page with some input fields and buttons but on iOS:

Tapping in any input field doesn’t cause the cursor to appear or the keyboard to show.
Tapping any buttons in the UI, except for text editor buttons, has no effect, although the buttons register the interaction, i.e. their colour changes.
Tapping the text editor buttons works, and doing so causes the cursor to appear in the associated textarea and the keyboard to show. This lets me enter / edit text in that textarea, but all the other UI elements remain unresponsive as before.

The same works fine on Android.
I don't know what causes the issue, much less how to reproduce it outside of my application, so I don't quite expect a solution but rather ideas as to what could be the cause because I'm out of those.

That sounds weird and I have no thoughts on what the problem is, but in terms of generally debugging things like this, my thoughts are (maybe you've already tried):

is this a page you can load in Safari, rather than in your WebView?
Have you tried attaching the Safari DevTools?

I did try it in Safari and it works fine there. I also fiddled with Safari DevTools for some time. What I discovered so far is that reloading the page there fixes the problem. Not sure why.

What I discovered so far is that reloading the page there fixes the problem

Does reloading via react-native-webview help? Does deferring the load (i.e. initially configure the WebView with just "about:blank" or something and then switch to your desired content) help?

This ended up being a non-issue after all. In my endless briliance, I tried the form I have in Safari on a Mac instead of in iOS. Sure enough, in iOS not only Safari but also Chrome had the same problem, so the matter had nothing to do with the webview.
For posterity: If you've encountered a similar problem, check if you are altering the height of your inputs somehow. Especially if you are doing so in a hook that runs repeatedly. That was the root of it in my case.


Region availability

I would like this page to contain availability of the listed VM sizes in each region, or links to that information. I just requested access to the FSv2 series VMs, waited 4 days, then found out they are not available in my region, so have to start all over again and still can't find what's available in my region (from this page anyway). So all a bit frustrating and easily solved by linking to region availability info from this page!  :)

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

ID: 8ef96d58-ce74-a034-993a-c02a6cfa4351
Version Independent ID: b31f4717-d0cf-2938-778d-79a69a0cc83a
Content: Azure Windows VM sizes - Compute optimized
Content Source: articles/virtual-machines/windows/sizes-compute.md
Service: virtual-machines-windows
GitHub Login: @jonbeck7
Microsoft Alias: jonbeck

@tim-at-software-forge-co-uk Thanks for reporting! I have assigned the issue to the author to investigate and update as appropriate.

@tim-at-software-forge-co-uk I work with Jon on the VM sizes documentation. Much of what is available is dependent on your region and your subscription, which makes it hard to add into a reference table like we have in the VM sizes docs. We are working on finding a better solution. In the meantime, take a look at this site: https://computeinfo.azurewebsites.net/. Is that more helpful?

#please-close


implementing ngOnChange to become sensitive to changeing data

basicly one line to implement OnChange the same way as you did with OnInit.
Helps alot though.

Could anyone merge this PR please?

Hi sorry for the late reply .. I ll merge the code soon

From: Bohdan Kalynovskyi (Dan<EMAIL_ADDRESS>Sent: Wednesday, March 8, 2017 9:34:07 PM
To: vimalavinisha/angular2-google-chart
Cc: Subscribed
Subject: Re: [vimalavinisha/angular2-google-chart] implementing ngOnChange to become sensitive to changeing data (#16)
Could anyone merge this PR please?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/vimalavinisha/angular2-google-chart/pull/16#issuecomment-285083125, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKh98Mdf2U_gPo0XLXainGDzZIHqQ8lnks5rjtF3gaJpZM4KR3Wb.


Ability to manually configure base details

In situations where an app is (for example) behind a load balancer with SSL termination, Ziggy is unable to properly parse the environment details required for a correctly working front-end, which causes other front-end libraries (like Vue or Axios) to make invalid requests.
Here's an example:

This PR solves this by changing the way these settings are gathered, and is backwards-compatible with existing behaviour. Configurable are the following options:

scheme (http, https, ssh, ftp ...)
host (example.com)
port(443, 80, 1234 ...)

Additionally or alternatively, a user can also provide an url option, which overwrites most if not all of those configurable options, depending on what parts of the URL were provided. For instance, if you provide //example.com, only the host will be overwritten (as only it can be parsed)

Could you explain your use case in a bit more detail?

in situations where an app is (for example) behind a load balancer with SSL termination, Ziggy is unable to properly parse the environment details required for a correctly working front-end

I'm not sure what you mean here, or what that screenshot is showing exactly.
I'm pretty sure most of this is already possible by passing a URL to Ziggy directly, as you've done in one of your tests. With a couple small tweaks your tests all pass for me locally with Ziggy as-is, before this PR, by passing in a URL:
(new Ziggy)->toArray();
// baseUrl      = http://ziggy.dev/
// baseProtocol = http
// baseDomain   = ziggy.dev
// basePort     = null

(new Ziggy(false, 'ssh://tighten.co:1337'))->toArray();
// baseUrl      = ssh://tighten.co:1337/
// baseProtocol = ssh
// baseDomain   = tighten.co
// basePort     = 1337

Here's a screenshot that demonstrates what my situation is a bit better:

As you can see, the request itself is loaded over HTTPS, but because there's a load balancer in our environment that strips the HTTPS and forwards the request as HTTP (TLS termination), Laravel receives the request as-if it were a HTTP request.
Because Ziggy's @routes directive doesn't allow for passing in an URL (that's only possible when instantiating Ziggy manually), it uses Laravel's url() helper instead to guess the current request details:
https://github.com/tightenco/ziggy/blob/90dcb58f062b078a632e5cc6302fa9472fdecc5a/src/ZiggyServiceProvider.php#L28
https://github.com/tightenco/ziggy/blob/90dcb58f062b078a632e5cc6302fa9472fdecc5a/src/BladeRouteGenerator.php#L5-L11
https://github.com/tightenco/ziggy/blob/90dcb58f062b078a632e5cc6302fa9472fdecc5a/src/Ziggy.php#L18-L22
At the end of the line, this cause a HTTP response to be given by Ziggy, which then causes front-end libraries to make improper calls when using Ziggy, resulting in browser blocking the request and showing the error (screenshot) I attached in the first post.
Laravel itself also has this problem, but it has the option to 'enforce' https through the secure_url() helper.
I hope this clarifies things a bit better.

Interesting, thanks for clarifying. Is your APP_URL http or https?
For now, can you try adding something like this directly after @routes:
<script>
    Ziggy.baseUrl = 'https://tradus-switchboard.test/';
    Ziggy.baseProtocol = 'https';
</script>

I feel like the easiest solution for this would just be to allow passing a URL to @route, but I'll think about it some more.

Hi @bakerkretzmar, sorry for the ultra-slow response, this one slipped through!
The APP_URL is https, but the url helper (used in Ziggy.php:22) actually ignores this:

The url helper itself (with input $to = '/')
URLGenerator::to('/' ...)
$this->formatScheme($secure = null)
Symfony's request: getScheme()

The workaround would indeed help, but it's a bit hacky if we're talking a long-term solution (what if Ziggy changes it's JS component?).
Thanks for looking into this again!

@claudiodekker okay, I see what you mean. The url() helper reads from the request, which I assume in your case ends up being http by the time it reaches Laravel?
You're right, that javascript workaround isn't great. I think the best way to tackle this is to either add a URL parameter to the @routes directive or a config option like ziggy.url to just override it globally.
I'll keep looking into this.

Yep, spot on. Looking forward to the (more permanent) solution :)

@claudiodekker do you think this would solve your problem?
In Ziggy.php:
- $this->baseUrl = Str::finish($url ?? url('/'), '/');
+ $this->baseUrl = Str::finish($url ?? config('app.url'), '/');

That way it'll always use APP_URL, which I believe in your case should ensure it's https. I realize this isn't nearly as configurable as the other options, but I'm wondering if they're overkill? Let me know what you think.

Hi @bakerkretzmar,
Sorry for the slow reply. Yes, I believe that'll do the trick!

Closed by #334. @claudiodekker thanks again for thinking this through with me!

I would expect the possibility to enter a base address (domain) in route() in js
I use tenancy for laravel and want to make requests from the URL of the administration into the URL of the tenants

@sbc640964 you can—if the Ziggy config object contains a location key, Ziggy will use its value instead of the window URL.


Service Dialog; Dynamic Field; Entry Point - OpenStack Provider - Help needed.

Hello,
I'm testing the latest version of ManageIQ (Najdorf), the launch itself was smooth. I was also able to add an infrastructure provider and a cloud provider based on OpenStack. The connection with OS was successful and I can see virtual machines running in the environment, image template's, networks etc. I can even manualy provision vm (Compute -> Cloud -> Instances -> Instances -> Provision Instances - Select an Image...) giving the necessary data such as "Cloud-Network" or "Instance Type" form drop-down menues that display available options from my OS environment.

The whole process is running smoothly and new vm starts up after creation.
The problem arises when I try to create a new service catalog item. I am trying to create a form in which the user, when ordering new VM, can select data such as "Network", "Instance Type" or "Guest Access Key Pair" from drop-down menu's (as you can see in the screenshot above), but I am unable to properly prepare the Service Dialog for this.
I created a simple Service Dialog for the testing purposes:

I marked this element to be dynamic (I assume it should be dynamic, if we want to display on this list the options available on the environment connected to MIQ)

And here we come to the problem that I cannot deal with - Entry Point

I don't know what should I choose here

Unfortunately, I did not find any hints in the documentation or on the Internet. Can anyone tell me what to enter there so that the drop-down window returns the options available in the environment? For example, available Flavors or Nets.
I will be grateful for any hints.

Hi @maly-glod.  The Entry Point is the Automate task that you want to run that will return the contents for the drop down you want to populate.  You will need to create an Automate class that will run the code that will return this information. Chapter 29 of the Automate book has the details on how dynamic fields are populated: https://pemcg.gitbooks.io/mastering-automation-in-cloudforms-4-2-and-manage/content/service_dialogs/chapter.html

Thank you very much for the hint. I managed to create the right automat class.


Added 'Hola mundo' in README

Testing another comments functionality until I finally found how to create pull requests.

@jverara what are you trying to accomplish here?

@jverara mistery solved Jose, no worries.


How to extract metadata including geolocation from media files

NOTE: All 1st-time contributors should know Topic approval and PR submission does NOT guarantee your Topic/Article will be published. Our team of Peer Reviewers and Content Moderators will review all PRs that come in to make sure they adhere to the standard of quality we expect from the EngEd community.
We expect all community memebers to go through provided examples, resources, and previously published material before submitting content. As a rule of thumb, please only submit articles (pull requests) that are complete, formatted correctly, and include a fully polished article (ready to be published and error free).
All revisions and edits should be completed on your own forked repo (as to not take up room in the queue). Any PR that is submitted incomplete (meaning not ready to be published as is) will be closed. A PR being closed does NOT mean the article can not be published (or fixed) it simply means that the edits, revisions, and fixes will happen outside of the queue.
NOTE: Any topic suggestion copied/taken/modified from YouTube may Lead to a permanent ban from the community.
Topic Suggestion
Enter topic suggestion here...
Pre-submission advice
By following all our pre-submission advice and reviewing our Resources folder, you will maximise your chances of your topic being approved.
We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner.
Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published (subject to vary depending on volume and/or backlog of articles).
Be sure to visit our Resources Page for tools, resources, and example articles that will help you propose and write a successful article.
Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
Writing sample(s):
Include any links or writing samples - to help our team better gauge your writing quality.
Proposal Submission
Proposed title of article
How to extract matadata from videos and JPEG files
Proposed article introduction
Photo and video are snapped and recorded by use of different formats. With the photos come along matadata which is in layman's microdata of a whole set of data which makes up the overall data. The article will show users how to extract matadata about time, type of phone and with this we will be able to integrate geo-location data to tracing the specific location where the photo or video was taken from. This will show users how to obtain it by integrating exit explorer,excel and google earth. Upon the user having the image, when he or she follows my laid steps, he or she is able to fully posses all matadata on a photo or video.
Key takeaways

How matadata solves ransacking problem on kidnapped individuals.
How to interpret matadata on images and videos.
The user will learn on aspects of photography and clearing matadata before sharing.
The challenges of matadata on laying on bad hands ie people who are unauthorized.

Article quality
This article explains generally about matadata on media files and how to interpret matadata to comprehend a scenario or a place or victim. It brings about readability case of understanding, and knowledge understanding about media files.
References
Please list links to any published content/research that you intend to use to support/guide this article.
Conclusion
Finally, removeNOTE: All 1st-time contributors should know Topic approval and PR submission does NOT guarantee your Topic/Article will be published. Our team of Peer Reviewers and Content Moderators will review all PRs that come in to make sure they adhere to the standard of quality we expect from the EngEd community.
We expect all community memebers to go through provided examples, resources, and previously published material before submitting content. As a rule of thumb, please only submit articles (pull requests) that are complete, formatted correctly, and include a fully polished article (ready to be published and error free).
All revisions and edits should be completed on your own forked repo (as to not take up room in the queue). Any PR that is submitted incomplete (meaning not ready to be published as is) will be closed. A PR being closed does NOT mean the article can not be published (or fixed) it simply means that the edits, revisions, and fixes will happen outside of the queue.
NOTE: Any topic suggestion copied/taken/modified from YouTube may Lead to a permanent ban from the community.
Topic Suggestion
Enter topic suggestion here...
Pre-submission advice
By following all our pre-submission advice and reviewing our Resources folder, you will maximise your chances of your topic being approved.
We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner.
Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published (subject to vary depending on volume and/or backlog of articles).
Be sure to visit our Resources Page for tools, resources, and example articles that will help you propose and write a successful article.
Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
Writing sample(s):
Include any links or writing samples - to help our team better gauge your writing quality.
Proposal Submission
Proposed title of article
How to extract matadata from videos and JPEG files
Proposed article introduction
Photo and video are snapped and recorded by use of different formats. With the photos come along matadata which is in layman's microdata of a whole set of data which makes up the overall data. The article will show users how to extract matadata about time, type of phone and with this we will be able to integrate geo-location data to tracing the specific location where the photo or video was taken from. This will show users how to obtain it by integrating exit explorer,excel and google earth. Upon the user having the image, when he or she follows my laid steps, he or she is able to fully posses all matadata on a photo or video.
Key takeaways

How matadata solves ransacking problem on kidnapped individuals.
How to interpret matadata on images and videos.
The user will learn on aspects of photography and clearing matadata before sharing.
The challenges of matadata on laying on bad hands ie people who are unauthorized.

Article quality
This article explains generally about matadata on media files and how to interpret matadata to comprehend a scenario or a place or victim. It brings about readability case of understanding, and knowledge understanding about media files.
References
Please list links to any published content/research that you intend to use to support/guide this article.
Conclusion
Finally, remove the Pre-Submission advice section and all our blockquoted notes as you fill in the form before you submit.
We look forwarding to reviewing your topic suggestion.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template
the Pre-Submission advice section and all our blockquoted notes as you fill in the form before you submit.
We look forwarding to reviewing your topic suggestion.

Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Hi @evalinenjoki
You haven't cleared the pre-submission advice and templates. Please format your article suggestion so we can easily make of it. When done, ping the @section-engineering-education/content-moderator-team so we can re-open it and give it a review.
As of now, we will close it to clear the backlog in the issues section.

Please open issue #6623
On Thu, 17 Feb 2022, 21:43 linusmuema, @.***> wrote:

Closed #6623
https://github.com/section-engineering-education/engineering-education/issues/6623
.
—
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/issues/6623#event-6092565698,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXQSFIHDWRHVMX4DR4DR4B3U3U6T3ANCNFSM5OI2VLJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
<section-engineering-education/engineering-education/issue/6623/issue_event/6092565698
@github.com>


SNOW-697912: Fix SQL simplifier for a join case.

Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes #SNOW-697912

Fill out the following pre-review checklist:

[x] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
When sql simplifier is enabled, a DataFrame's and its previous level DataFrame (the subquery) share the same Attribute instances. This generates a wrong SQL in a specific case. The fix is to copy those Attribute instances.

Codecov Report

Merging #649 (85faff6) into main (214b359) will decrease coverage by 3.30%.
The diff coverage is 78.57%.

@@            Coverage Diff             @@
##             main     #649      +/-   ##
==========================================
- Coverage   98.79%   95.48%   -3.31%
==========================================
  Files          49       49
  Lines        7941     7972      +31
  Branches     1330     1344      +14
==========================================
- Hits         7845     7612     -233
- Misses         38      249     +211
- Partials       58      111      +53

Impacted Files
Coverage Δ

...ke/snowpark/_internal/analyzer/select_statement.py
57.52% <25.00%> (-41.27%)
:arrow_down:

...nowflake/snowpark/_internal/analyzer/expression.py
96.69% <100.00%> (-3.31%)
:arrow_down:

src/snowflake/snowpark/dataframe.py
94.85% <0.00%> (-4.87%)
:arrow_down:

src/snowflake/snowpark/dataframe_reader.py
96.29% <0.00%> (-3.71%)
:arrow_down:

src/snowflake/snowpark/table.py
97.10% <0.00%> (-2.90%)
:arrow_down:

...snowflake/snowpark/relational_grouped_dataframe.py
98.54% <0.00%> (-1.46%)
:arrow_down:

src/snowflake/snowpark/session.py
97.56% <0.00%> (-1.40%)
:arrow_down:

src/snowflake/snowpark/_internal/type_utils.py
98.48% <0.00%> (+0.09%)
:arrow_up:

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


refactor: cleanup of abi.nr in aztec-nr

Cleanup of abi.nr + removing PublicGlobalVariables struct since it's a copy of GlobalVariables.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

master

PR #4421

PR #4473   👈

This stack of pull requests is managed by Graphite.

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit bbfa3374 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,700
180,580
720,100

l1_rollup_calldata_gas
222,856
871,384
3,464,236

l1_rollup_execution_gas
316,047
989,832
3,684,614

l2_block_processing_time_in_ms
1,157 (+1%)
4,358
17,322

note_successful_decrypting_time_in_ms
309 (+1%)
1,006
3,760 (+2%)

note_trial_decrypting_time_in_ms
19.1 (+21%)
32.8 (-2%)
144 (+2%)

l2_block_building_time_in_ms
17,844
70,446
283,070

l2_block_rollup_simulation_time_in_ms
13,117
51,738
208,404

l2_block_public_tx_process_time_in_ms
4,699 (+1%)
18,628
74,402

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,803 (+1%)
26,406

note_history_successful_decrypting_time_in_ms
2,350
4,639

note_history_trial_decrypting_time_in_ms
92.3 (-3%)
216 (-3%)

node_database_size_in_bytes
17,313,872
33,067,088

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
307
44,528
25,825

private-kernel-ordering
194
44,145
16,225

base-rollup
1,407
128,654
933

root-rollup
82.7
4,192
729

private-kernel-inner
436
71,331
25,825

public-kernel-private-input
241
32,287
25,825

public-kernel-non-first-iteration
237
32,329
25,825

merge-rollup
7.91 (+1%)
2,712
933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
16,859
40,167

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
553 (+1%)
1,312 (+1%)

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.521 (+2%)
576


crystal 0.23 bug compiling in release on ubuntu

Ubuntu 14.04.1 LTS
Crystal 0.23.1 [e2a1389e8] (2017-07-13) LLVM 3.8.1 (downloaded from releases page)
crystal eval 'p 1'
1

crystal eval 'p 1' --release
crystal: /var/cache/omnibus/src/llvm/llvm-3.8.1.src/lib/CodeGen/LexicalScopes.cpp:160: llvm::LexicalScope* llvm::LexicalScopes::getOrCreateRegularScope(const llvm::DILocalScope*): Assertion `cast<DISubprogram>(Scope)->describes(MF->getFunction())' failed.
/opt/crystal-0.23.1-3/bin/crystal: line 102:  5263 Aborted                 (core dumped) "$INSTALL_DIR/embedded/bin/crystal" "$@"

Duplicate of #4719
Please look for existing issues before posting a new one


add a codeowners

Adding a codeowners will allow us to track correctly update

Not sure why? How many codeowners do we have? I dont see many people working on this repo

Do we have amaro team ? Maybe put typescript team

we have typescript team, but imho does make sense to ping everyone on every PR

I know what you mean. And I agree with you. But you'd still have to have a manager/maintainer, which is explicitly marked.


clang error: the clang compiler does not support '-march=native' [TF Lite, Android ARM]

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS 10.14.6
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: no
TensorFlow installed from (source or binary): trying to build
TensorFlow version: 2.2.0
Python version: 2.7
Installed using virtualenv? pip? conda?: not installed
Bazel version (if compiling from source): 2.0.0
GCC/Compiler version (if compiling from source): not sure, I'm cross-compiling
CUDA/cuDNN version: none
GPU model and memory: none

Describe the problem
I'm trying to cross-compile TF Lite to Android. I want to create a cc_library that depends on //tensorflow/lite:framework, but it fails for ARM with the message:

clang: error: the clang compiler does not support '-march=native'

I can reproduce this in a clean TF repo with lite:framework itself,
bazel build -c opt --config=opt --config=android_arm64 --cxxopt=--std=c++11 --jobs=1 -s //tensorflow/lite:framework

It works fine for x86/x64 targets. It also works fine without --config=opt, which is adding the march=native flag. This --config=opt is advertised in many places of the source code, it's even configured in ./configure, so it's quite confusing to see that it does not work with common architectures.
Provide the exact sequence of commands / steps that you executed before running into the problem
See above.


Make root='/' work cross-platform

If root already ends in os.sep, don't add another one
This used to fail on windows, but somehow worked on unix

Ping! Can this be looked at again?

Hard to unit-test on non-windows systems, but it looks okay. Thanks


Add slack message on publish to gh-pages?

Not going to do this because we would have to commit .slack api key the way the app is configured now.


Packet len is out of range  when  starting

when consumer starting , it will get all dubbo node from zookeeper, then "Packet len is out of range"
why not only get  node that with @DubboReference
just for less request ?
    void readLength() throws IOException {
        int len = this.incomingBuffer.getInt();
        if (len >= 0 && len < this.packetLen) {
            this.incomingBuffer = ByteBuffer.allocate(len);
        } else {
            throw new IOException("Packet len" + len + " is out of range!");
        }
    }

repeat #8622


suggest: use vercel

we are using Netlify, it's safe but in future we will use NextJs so if we use vercel then it will help a lot
I will suggest use vercel

I had no idea vercel might be better but I'll move our website there if that's what you're suggesting :smile:

currently, it's working fine on Netlify. we don't need to switch to vercel so I'm closing this issue.


Feature Request: regex conversion custom copy?

I'm finding use cases where I need to manipulate a certain set of attributes rather frequently and would like to convert them to another form of output that simplifies it over standard HTML or other options. Could you give us a custom option that will utilize a (or a series of) regular expression to reflow the copied content to a new format output for the pasteboard/clipboard?

This requirement is more suitable to be implemented with AutoHotKey.
Example:
#Persistent
OnClipboardChange("ClipChanged")
Return

ClipChanged() {
  result := RegExReplace(Clipboard, "(.*)", "$1$1")
  If result {
    Clipboard := result
    Sleep 500
  }
}


when testing the skill the only value returned is an error

using a fresh app installation, when testing the skill all i get back is:
"The remote endpoint could not be called, or the response it returned was invalid."
I have tested with various SSL certificates and received SSL handshake errors instead so im sure the connectivity is there and the cert is valid. Has the amazon API changed since the app was released? i had to switch back to the legacy portal version to be able to use the self-signed cert in the first place

Looks like its easily fixable by downloading the latest alexa skills kit and accompanying jars from maven into SPLUNKHOME/apps/alexa/bin/lib

Not exactly sure what you think you are fixing , but the latest release of the App (on Splunkbase https://splunkbase.splunk.com/app/3298/) , that is compiled and tested to the jars that it ships with , is working just fine for me and has been running for well over 1 year now in my test environment perfectly well. So I'm sure you have some other setup issue. Check the logs (detailed in the docs) for any more specific error messages.

This was a simple app install from splunkbase using the GUI that turned up with version 1.1.1 (published in 2015, latest in maven is 1.8.1 published 3 months ago) of the Alexa skills kit library, nothing fancy on my side. Followed the same steps as i did last time on a different environment but ran into this brick wall while testing where even thought the certificate is correct the response from splunk is invalid. As soon as i dropped in the updated jar file and restarted splunk the connection worked
If you dont think its an issue fair enough but i think its a good idea to document this for when it happens to someone else and i would reccomend setting up a fresh alexa project and splunk server so you can see what i mean
side note this is an awesome app, gj :P


[JavaFX-Part 4] Add explanation on fx:root

Current:

Problem:
The tutorial states to check Use fx:root construct but does not explain what it does.
Proposed:
Insert a link to the JavaFX-FXML documentation on what is fx:root and add the following:
Checking this enables the use of the fx:root element, allowing you to reference a predefined root element. More about fx:root on the documentation Introduction to FXML | JavaFX 2.2 (oracle.com).

@JiaXinEu You can send a PR for this.


Proxy mode?

What if instead of issuing GitHub tokens, OctoSTS could also act as a proxy?
Clients would never receive their GitHub token, they'd call https://octo-sts.dev/github-api/repos/octo-sts/app/contents/README.md using an OIDC token as credentials, and the octo-sts service would swap the tokens before making the request to https://api.github.com/repos/octo-sts/app/contents/README.md and returning the response.
To clients it would be a GitHub Enterprise Server endpoint. The octo-sts/setup-proxy-action might export the GH_HOST and GH_ENTERPRISE_TOKEN variables, so tools like the gh CLI prefer the custom endpoint.
It could generate an installation token for each API call, or use a cache and revoke upon eviction.
The goal is to allow trust policies to get even more restrictive: instead of granting an identity contents:write to the repository, the client could be artificially restricted to updating the ref that triggered an actions workflow, or restricted to accessing certain paths. In the future an actor could be prevented from using [ci-skip] in commit messages, or only allowed to create (not edit/close) issues that include  generated by an LLM.
Between the v3/REST, the v4/GraphQL and various Git transports - I would not want to fund its bug bounty 😆 . Researchers will work around the additional restrictions, but the "fail open" behaviour is identical to today's model of giving the client the token.

It's an interesting thought.  I'd be a bit wary of proxying that volume of (potentially sensitive) data though, which I suspect could become a meaningful amount of egress as well.

It's an interesting thought.  I'd be a bit wary of proxying that volume of (potentially sensitive) data though, which I suspect could become a meaningful amount of egress as well.

Id have to agree with mattmoor here. It is bad practice to proxy without pure necessity. This will only introduce unnecessary load into infrastructure.
The only, non-critical, benefit I see here is a developer QOL improvement.
Better practice for developer QOL improvement would be to build a library to properly handle token exchange and refresh.


Try eval via multi-lingual sentence transformer

https://huggingface.co/sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2

Use a multi-lingual sentence transformer, using which we can compute embeddings for an original sentence and for a translated sentence

no need for a previous 'known good' translation

This would be alternative to requiring 'golden dataset' of translations, like
Ubuntu Translation dataset:
https://opus.nlpl.eu/Ubuntu/en&nl/v14.10/Ubuntu
GNOME Translation dataset:
https://opus.nlpl.eu/GNOME/en&nl/v1/GNOME (edited)


Differentiate between auth and deployment target

Closes #203

One thing which I recognized, but is not related to this PR specifically (also discovered it on develop): The form is only showing errors when the field is empty, but no hint if wrong content

Is this intended?

I will have a look into this. Thanks for notice.

Did a bit of root cause analysis and found out that this check is broken, I think after we refactored the configuration structure, cf. #197
fieldName is filled with endpoint.xxx, but it checks for xxx only, so resolved false and not the correct error.

We can open an issue for that little one or simply fix it with this PR, what do you think @barmac ?

I will have a fix for that. Thanks for hint!

OK I applied the fix. I still get funny result when changing target type (unknown error for cluster ID), but I guess we're not getting rid of this with any quick fix.

I can not see what you mean, it works perfectly for me. Maybe you can open an issue for that if it's reproducible. Do you think it is blocking the release?

I can not see what you mean, it works perfectly for me. Maybe you can open an issue for that if it's reproducible. Do you think it is blocking the release?

I think it is not blocking. I will create an issue for that.

I played with this a bit and there's this weird glitch when switching between the Auth types. I personally don't think this is a blocking issue. Should we maybe open an issue about this? Might be that we need to cancel the ongoing requests or something.
Besides that looks good, well done @barmac 👍

A suggestion about the glitch would be that: Do not try to validate the connection if auth fields are empty.

Yeah, I guess this will be implemented when we refactor the connection check.

I guess that's the thing @barmac was mention before. Let's open an issue for that 👍 Maybe @oguzeroglu as you're able to reproduce it properly.

Ok, I'll open an issue.

https://github.com/zeebe-io/zeebe-modeler/issues/208


Prevent race condition on errorsTotal increment

Original discussion: https://github.com/tus/tusd/commit/f50f03fe6f357b98ebef6a97833503e121931149#commitcomment-20637394
This PR prevents a race condition on the errorsTotal counting.
The HTTPError interface is also used to have detailed metrics (by status as
well):
# HELP tusd_errors_total Total number of errors per cause.
# TYPE tusd_errors_total gauge
tusd_errors_total{cause="upload not found",status="404"} 2

This can be helpful when displaying some statistics (errors 5xx, 4xx...)

@Acconut: done!

After testing your branch locally I discovered a flaw in how the errors are stored in the map. Here are steps to reproduce:

Start the tusd server in background: go run cmd/tusd/main.go --dir data-dir &
Remove the data directory: rm -r ./data-dir
Try to create multiple uploads by running following curl command a few times:  curl -X POST -H "Tus-Resumable: 1.0.0" http://localhost:1080/files/ -H "Upload-Length: 100" -v. These requests will fail because the upload directory does not exist any more but that is expected and wanted.
Visit http://localhost:1080/metrics to view the metrics but you see a following error:

An error has occurred during metrics collection:

3 error(s) occurred:
* collected metric tusd_errors_total label:<name:"cause" value:"upload directory does not exist: ./data" > label:<name:"status" value:"500" > gauge:<value:1 >  was collected before with the same name and label values
* collected metric tusd_errors_total label:<name:"cause" value:"upload directory does not exist: ./data" > label:<name:"status" value:"500" > gauge:<value:1 >  was collected before with the same name and label values
* collected metric tusd_errors_total label:<name:"cause" value:"upload directory does not exist: ./data" > label:<name:"status" value:"500" > gauge:<value:1 >  was collected before with the same name and label values

The problem is following:
In the datastores an error object may be created every time they occur (see https://github.com/tus/tusd/blob/master/filestore/filestore.go#L67). Even though they share the same message, they are different instances and have different addresses in memory. If you then add all these errors to your ErrorsTotalMap they will be considered as different keys (because HTTPError is an interface and the error instances have different addresses) and insert a new key-value pair. In the example above, the ErrorsTotalMap would contain three entries, all pointing to an "upload directory does not exist" error. These duplicates cause Prometheus to blow up, when you try to collect them (https://github.com/tus/tusd/pull/105/files#diff-362f42fe2d092bfd04057043da99f142R79) because they have the same error message and status code.
My suggested solution to this problem would be to revert the ErrorsTotalMap to a map[string]*uint64 again. However, we would lose the status code label in the /metrics page (which I personally do not see as a huge sacrifice). Or you could make it a map[struct{Message string, Status int}]*uint64. The difference is now that the map keys are not interfaces but structures now.

Godd catch!
Another solution might be to include the address of the error (&err) in the metrics to ensure its uniqueness for prometheus (but it will be ugly if a new object is created everytime).
I would prefer to implement your second suggestion (map[struct{Message string, Status int}]*uint64) if you are fine with it.

I would prefer to implement your second suggestion (map[struct{Message string, Status int}]*uint64) if you are fine with it.

Of course, I would be delighted if you could take on this task :)

@Acconut: implemented!
Regarding the conflict, I think this part of the code should be moved inside the simplifiedHTTPError function (it may be interesting to have the original message in some other logs: only the metrics should be simplified):
func simplifiedHTTPError(err HTTPError) simpleHTTPError {
	msg := err.Error()

	// Errors for read timeouts contain too much information which is not
	// necessary for us and makes grouping for the metrics harder. The error
	// message looks like: read tcp <IP_ADDRESS>:1080-><IP_ADDRESS>:53673: i/o timeout
	// Therefore, we use a common error message for all of them.
	if netErr, ok := err.(net.Error); ok && netErr.Timeout() {
		msg = "read tcp: i/o timeout"
	}
	return simpleHTTPError{
		Msg:  msg,
		Code: err.StatusCode(),
	}
}

Good idea!

Completed.

Great! I will give it a final review soon and then merge it!

It still appear here, but it might be a local problem if neither you, nor travis, nor appveyor have the issue !

That's interesting. Which Go version are you running? They changed it strictness a bit in 1.8 (https://golang.org/doc/go1.8#tool_vet) but this actually be visible on Travis...

Solved with go get -u github.com/tus/tus (the tus import from the prometheuscollector subpackage wasn't up to date)

Thank you for your help and in particular your patience with me :heart: :+1:  I have done some minor cleanups and then squash merged your branch in https://github.com/tus/tusd/commit/74fca8c6c3033b7a55735095359c6e42e7ae532f.


Fix color parsing for numbers starting with non-digits

A recent change to our scene loading code that converts strings to doubles has exposed this bug in a color parsing function. doubles need not start with a character that passes isdigit().

Yeah, it's a pretty involved pipeline for all of the style parameters right now. I think it could help to pass the un-modified "node" type further down the pipe before making conversions.


🐞 Textfield is not behave properly

What happened
I haven't had to fully text the TextField yet but this is what I'd pick up on Android Emulator. The input seems janky like in the video. The text won't show up until I paste the text from the clipboard and the TextField started to behave normally
Evidence
Implemetation
                <mdc:TextField
                    IconKind="Pets"
                    IsError="{x:Binding PetForm.PetNameValid}"
                    LabelText="Cung cấp tên"
                    SupportingText="{x:Binding PetForm.PetNameInvalidMessage}"
                    Text="{x:Binding PetForm.PetName,
                                     Mode=TwoWay}" />

Platform tested

Windows - fine 👍🏽
Android - issue happened 💀

Have you managed to solve this? Having same problem.
Screen recording

Have you managed to solve this? Having same problem.
Screen recording

Not yet! hope the author that could look at this

The current focus of work is #16 , where all controls are being refactored either simply or thoroughly. TextField is a big trouble, but I will try to make it work more perfectly when I turn my attention to it.

Hi, is there any workaround meanwhile?

The current focus of work is #16 , where all controls are being refactored either simply or thoroughly. TextField is a big trouble, but I will try to make it work more perfectly when I turn my attention to it.

Hi, is there any workaround meanwhile?

Not now.

Fixed in 158da7b

Fixed in 158da7b

Cool how long will the nuget got updated ?

One week? If I'm not busy.

¿Ha conseguido solucionar esto?

Fixed in 158da7b

Hi, when will the nuget be updated?

Hi, any update on this issue

Hi Any update on this issue? I do face the same problem with the TextField on 0.1.2-beta version. May I know when this issue will be addressed?

Hi, any update here please?

@yiszza When will the next version of Nuget will be released with these fixes 158da7b?


StateAccessors are confusing and provide no perceived benefit

When working with docs, samples, and tutorials that require state management, the StateAccessors paradigm doesn't appear to serve any discernible purpose and provides only a layer of indirection; another pattern developers must grok in order to build a useful bot.
We should strive to make state management as straightforward as possible as this is fundamental to useful bot creation.
Request: Make state accessibility & manipulation native to the SDK, not something that must be hand-rolled every time it wants to be used (via something like Accessors); showcase this way of doing state management in our docs & tutorials.

@johnataylor is working on this. I don't know how far towards what you're asking we'll be able to get, in part due to the need to preserve compat.

Anyone have a good example of conversation state that works?  I had a user state object worked, but really needed to be conversation state. The refactor has not gone well.  TIA

Yes absolutely, the title of the issue says it all.
So analyzing this a little: Accessors look like classes however they are really just awkward methods on a BotState class. They have odd semantics, at first they look like a classic get/set patter however set is never called - anywhere. Anyhow... they are public classes and back peddling is difficult.
The good news is we have a growing set of samples that should demonstrate how not to worry about this too much.
Couple of specifics to look at:

Sample 42 "Scaleout" shows how to build your own state solution from the ground up. In order to integrate this with the existing Dialog stack you'll find a completely generic shim class called RefAccessor buried deeply in the implementation. (You might ask yourself why Accessors exist if its so easy to write a completely generic implementation.)

In the revised samples (temporarily on a branch called samples-work-in-progress) there is a couple of classes: DialogBot and the Run extension method that handle Dialogs for you. Just use those and you don't have to worry about your own accessors - at least for ConversationState.

If you want to use UserState you will have to grab an Accessor from it. But we have provided samples of how to do this correctly. Please follow exactly the sample because the semantics of the BotState/Accessor model have some surprises.
So the path we are on is to firstly help you deal with this so you hardly have to worry, and secondly to revise the model more completely and replace it with something more conventional.


jsdoc failure in .ts file

TypeScript Version:  2.6.2
VSCode Version: 1.19.0
Code
/**
 * @param {string} nonPrimitive
 */
function test(nonPrimitive) {}

test(1)

tsconfig.json
{
  "compilerOptions": {
    "checkJs": true,
    "allowJs": true,
    "target": "es5",
    "moduleResolution": "node",
    "module": "es2015",
    "strict": true,
    "noImplicitThis": true
  }
}

Expected behavior:

Actual behavior:

Since you can add type annotations in typescript, it does not use JSDoc for types. This is by design.
You need to use nonPrimitive: string instead.
IIRC there's a quickfix that converts JSDoc types to type annotations.

In some cases my project team can't use the compile tools , And now I have a problem in vue typescript component , in .ts file the vue component this type is right, but if I rename the ext .ts to .js, this vue component this type is any, I don't know why.
Because typescript is a superset of javascript, so only write javascript is good, but I found I can't define type by jsdoc, it bothered me and make me write javascript without typescript help .
The git url is can report my problem

yeah, I find out jsdoc since begin is design for js not ts , so ts file can't use jsdoc to define type .


Update cloud-governance.md

I believe COBIT is a better example than ITIL when it comes to IT governance
http://www.isaca.org/COBIT/Pages/default.aspx

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

I agree with that, probably removing all 3 (ITIL, ITSM & COBIT) is a better choice IMHO since I would like to avoid a discussion with the customer around frameworks all together and keep the narrative focused on Azure governance.

Hi @brianblanchard - Could you indicate how you'd like to proceed with the pull request? Thanks.

@alexbuckgit, please ensure the short term approach taken here is consistent with other articles.
Adding separate tasks to the backlog to test and take a holistic stance betting this article & PR.

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

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


Apply fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

Codecov Report

Merging #12 (ef94ce5) into main (508fdfb) will decrease coverage by 0.06%.
The diff coverage is 68.08%.

@@             Coverage Diff              @@
##               main      #12      +/-   ##
============================================
- Coverage     72.14%   72.08%   -0.07%
  Complexity      246      246
============================================
  Files            91       91
  Lines           894      899       +5
============================================
+ Hits            645      648       +3
- Misses          249      251       +2

Impacted Files
Coverage Δ

domains/Composer/ComposerDependency.php
50.00% <ø> (ø)

domains/Composer/FlushException.php
0.00% <0.00%> (ø)

domains/CreateProjectForm/Components/Logo.php
0.00% <0.00%> (ø)

...CreateProjectForm/Components/StarterKit/Breeze.php
0.00% <0.00%> (ø)

...ateProjectForm/Components/StarterKit/Jetstream.php
0.00% <ø> (ø)

...reateProjectForm/Components/StarterKit/Laravel.php
0.00% <0.00%> (ø)

...CreateProjectForm/Components/StarterKit/Option.php
0.00% <0.00%> (ø)

...tForm/Http/Controllers/CreateProjectController.php
0.00% <0.00%> (ø)

...t/CreateProjectRequest/BuildsCreateProjectForm.php
100.00% <ø> (ø)

...ojectForm/Sections/Cashier/CashierMollieDriver.php
0.00% <0.00%> (ø)

... and 48 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 508fdfb...ef94ce5. Read the comment docs.


Exporting to CSV is failing without any error

Hi Srikanth,
First, I would like to Thank you for providing this wonderful utility.
I am currently using 2.3.1 version. We are exporting around 10 tables to a zip file using this library. After exporting to zip file, I see zip file has only first 3 tables. However, in the logs I don't see any error and all 10 tables export process is getting completed. If i exclude 4th  table, I see zip file has  remaining 9 tables. So, the 4th table is causing the issue. But I don't see any error message related to it.
Is there any way, I can identify the error, that is causing? By any chance, is it fixed in later versions?

What do you mean by "tables"? Is it database tables you are talking about? How are you exporting it? Any sample code? I would highly recommend trying with the latest version and to see if that fixes the issue.

Closing this as there is no further feedback. Feel free to reopen if you have a feedback to my questions in the post above.


Add feature data

In the original Ruby variant you can set Feature data:
$rollout.set_feature_data(:chat, description: 'foo', release_date: 'bar', whatever: 'baz').
I added the method and of course also the clearFeatureData method.

This is a very much needed addition to a package, as it makes more open to additional information about the feature. Right now it is really hard to extend the Feature itself without having an ability to set extra details.
@richardfullmer Can we expect this to be merged any time soon?

Tagged as 2.3.0

Thanks a lot @richardfullmer!

Could you check Packagist regarding the new 2.3.0 version? Somehow this new release is not picket by Pakcagist (https://packagist.org/packages/opensoft/rollout). There still 2.2.1 is the newest version.


[QnA Maker] ActiveLearningDisable bug fix

Fixes #2140

Pull Request Test Coverage Report for Build 126887

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.941%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botbuilder-ai/src/qnaMakerDialog.ts
0
2
0.0%

Totals

Change from base Build 126677:
0.0%

Covered Lines:
12404

Relevant Lines:
14809

💛  - Coveralls

#2159 should fix the build issue blocking this PR


 What might be the reason?

Model is working, loss was indeed Nan


rpm: require Ansible >= <IP_ADDRESS>

This is the only version that our CI uses for testing, so it's the only version we can confidently say works.
Update the RPM packaging to specifically require this version of Ansible.

A side effect of this change is that yum will no longer try to pull in the "ansible19" package from EPEL in order to satisfy the dependency.


Grpc client creation spends seconds in objectHash

Every call to loadProtoJSON runs the given json object through object_hash. For ginormous protocol objects this can take seconds. Could we cache the object itself to avoid the hashing overhead on repeat call? See poc
As an example using the grpc client in google-ads-node each client creation spends few seconds hashing which is annoying as we are creating clients multiple times for different customers.

We have a plan to get rid of the hashing. Because it estimate as a breaking change, we will do with next major release as long as the drop node. ETA end of Q1


BUG : Added missing quote to the slider image

Without quotes in the background-image property, images with parentheses in the url are not displayed.
The style is not valid .

@DanielRuf @acburst Looks good to me. What do you think?

Looks good, but I would prefer this style: $(this).css('background-image', 'url("' + $(this).attr('src') + '")' );

Fixed in https://github.com/Dogfalo/materialize/commit/b1491c37885fc947dd88890dd4d7185619823412


What should I do if I don’t know which app the callback information belongs to?

I have more than a dozen apps setting the same callback address in the background. My new AppStoreServerAPIClient(encodedKey, keyId, issuerId, bundleId, environment); I don’t know which app the callback information belongs to, so I don’t know who to change. Configuration information, is there a solution to this? For example, I don't verify the first layer of base64 and directly decrypt it. After getting the bundleId, can I determine which app it is?

In this case, we recommend in App Store Connect adding a query param or some other identifier to the URL to allow your server to know which app the notification URL is being called for, but still allowing them to all share the same domain and path.

Thanks for the reply. The method used is to add a unique identifier after each callback address. Another point is to distinguish between the sandbox and the production environment. It is also necessary to add an identifier after the link.


[HUDI-5010] Fix flink hive catalog external config not work

Change Logs
Flink hive catalog external config not work
Impact
Describe any public API or user-facing feature change or any performance impact.
Risk level: low
Documentation Update
N/A
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[x] CI passed

5010.patch.zip
Thanks for the contribution, i have reviewed and apply a patch ~

Thanks a lot for review @danny0405 ,i have apply it


Invalid warnings produced by rasa test

Rasa versions: 2.6.0, 2.5.0, 2.4.0, and 2.3.0  (this issue does not occur with 2.2.0)
Rasa SDK version: 2.6.0
Python version: 3.8.8
Operating system: macOS-10.16-x86_64-i386-64bit
Issue:
I upgraded to the latest version of rasa, and rasa test is giving me some invalid warnings. It complains that all of my intents are missing from the domain file when they aren't.
You can reproduce the warnings by creating a new project using:
rasa init

Then running:
Rasa test

You should get the following warnings for each intent:
UserWarning: Issue found in './tests/test_stories.yml': Found intent 'greet' in stories which is not part of the domain.

and
UserWarning: Issue found in 'data/stories.yml': Found intent 'greet' in stories which is not part of the domain.

And also warnings for intents in the rules.yml file:
UserWarning: Issue found in 'data/rules.yml': Found intent 'goodbye' in stories which is not part of the domain.

This is a duplicate of #8135


Question: show a custom view

Hi, can I show a custom view inside progressBar, not just a string but an ImageView with some text

Sure could try to customize it and provide others more control.Unfortunately I work only on weekend due to no free time.Will Surely try to sort this out :+1:


Feature Request: Instructions on Focus Chart page

When loaning your goggles to someone to try them out, you would generally put the goggles onto the Focus Chart page, as that is the first thing they will need to do.
Why not put some instructions on this page? For example, some text pointing to where the focus/IPD knobs are (And how to use them), plus some text pointing to the top left saying how to exit out of the menu (eg "Long press button HERE twice to exit to FPV view")

Adding instructions to change the IPD is hard on the existing focus chart, since there is not much space left.
Also, if you lend your goggles to somebody, you can just explain that to them.
I've added some filler text all around the focus chart because while having the circles is handy,I find it important to get text that you ought to be able to read everywhere on the screen as well.

Adding instructions to change the IPD is hard on the existing focus chart, since there is not much space left. Also, if you lend your goggles to somebody, you can just explain that to them.

Agreed, not that big of a deal as they are in a pretty standard place.
Looks good!

TODO


SPOI-10711 Add documentation for security/authentication and system alerts

@amberarrow @wvary pls review and merge as appropriate

@amberarrow @wvary  another reminder for PR

Made a couple of comments, rest looks good.

@amberarrow comments incorporated for review and merge


Langchain python rag doc example Mac only

The example requires a Mac only package but I think that's not actually the case.

This is on Linux Mint (latest)
====
python3 main.py
/home/memo/.local/lib/python3.10/site-packages/langchain/init.py:34: UserWarning: Importing PromptTemplate from langchain root module is no longer supported. Please use langchain.prompts.PromptTemplate instead.
warnings.warn(
Traceback (most recent call last):
File "/home/memo/Projects/ollama/examples/langchain-python-rag-document/main.py", line 26, in
data = loader.load()
File "/home/memo/.local/lib/python3.10/site-packages/langchain/document_loaders/pdf.py", line 134, in load
return loader.load()
File "/home/memo/.local/lib/python3.10/site-packages/langchain/document_loaders/unstructured.py", line 86, in load
elements = self._get_elements()
File "/home/memo/.local/lib/python3.10/site-packages/langchain/document_loaders/pdf.py", line 57, in _get_elements
from unstructured.partition.pdf import partition_pdf
File "/home/memo/.local/lib/python3.10/site-packages/unstructured/partition/pdf.py", line 24, in
from pdfminer.utils import open_filename
ImportError: cannot import name 'open_filename' from 'pdfminer.utils' (/home/memo/.local/lib/python3.10/site-packages/pdfminer/utils.py)

I think it is fixed now:

Uninstalled pdfminer and installed pdfminer.six


Add example to read out table_entry direct_counters

I had a hard time finding out how to read out the byte and packet counters for specific table entries.
Adding examples like these can help the user get acquainted to the shell more easily!
For the future, there could also be executable examples using Mininet + bmv2?
Rendered view
Related: https://github.com/p4lang/p4runtime-shell/issues/40

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   71.77%   71.77%
=======================================
  Files           6        6
  Lines        2002     2002
=======================================
  Hits         1437     1437
  Misses        565      565

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d4b1182...d814c0b. Read the comment docs.

I think such an example belongs under https://github.com/p4lang/p4runtime-shell/tree/master/usage to avoid cluttering the README file.
BTW, if you have your table entry object te (read from the server), you should be able to read the direct counters like this as well:
te.counter_data.packet_count = 0
te.read

The first line ensures that the counter_data field is set in the Protobuf message, which according the P4Runtime spec will guarantee that the counter values will be returned (https://p4.org/p4runtime/spec/master/P4Runtime-Spec.html#sec-direct-resources)

Thanks for the answer! I adapted the commit as you suggested, namely:

moved the example in the usage/ folder
added a notice in the main README.md that the usage folder exists


Support .po Compose

Implements #8

Coverage remained the same at 100.0% when pulling 776f267b34dd8724b9344ca0c4b446b254ede114 on enhance_tests into 4d625a96b468ae0cc625333479f096a6c13f9abb on main.


Missing yo-doc for some zefops

The following zefops have no yo documentation:

yo(L)
yo(allow_tombstone)
yo(expect)
yo(collect)
yo(func)

L is a relic from old traversal styles and should be removed altogether if there are no dependencies on it anymore.

allow_tombstone and expect were both throwing an exception when yo was called on them, that has been fixed now, but they will remain for the time being without a docstring as they don't have any implementations but are used as flags, I think these should be revisited and changed to ValueTypes as they don't have any implementations. Would you agree @pengwyn?

Both collect and func had an ugly repr and that has been fixed now.  As for showing a docstring, it would be tricky as they aren't exactly a ZefOp. But it is something I can play around with at some point.

I agree with making those allow_tombstone and expect turned into ValueTypes. Although expect I don't think has been used yet, so we might have to rejig that for a different purpose later on.


checksum-retry

These two patches make upload a bit more resilient against networking issues:

We see uploads failing with a 404, but eventually they succeed with the patch
we also upload files that are symlinks, and without the 2nd patch this plugin will try to do a full upload of all the files, without retrying the checksum deploy in case of network issues

This is probably not the best way to solve this, open for better suggestions :-)

@andred,
If I understand correctly, your code overrides the http client's retry mechanism, and implements it manually, so that a retry is performed even if the PUT requests returns 404.
Do you know what is the root cause of the 404? I'm not sure that a network hiccup can cause a 404 response. I'm concerned that this code change will hide the issue on your side, as well as add redundant retries for other users.
Maybe we should dig down further and understand how one PUT return a 404 and an identical one right after does not.

Hi,
In a way, yes. At the moment, the http client retries itself (up to three times if I remember right), unless the server (Artifactory) returns a 404 to the POST. In that case, the upload fails, there is no retry (and the Jenkins build fails subsequently).
So this means
a) that the http client's retry is not in line with the number of retries set within the rest of the Jenkins Artifactory plugin.
b) if the same file has been successfully uploaded in the mean-time by some other means, the http-client instance doesn't know about that, and will not try to do a checksum-deploy before retrying the upload for real on subsequent attempts
c) a 404 causes the build to fail, even though Artifactory might not return a 404 the next time, e.g. because the upload succeeds on the subsequent attempt by the same http-client, or because the file appeared via a different means on the Artifactory server and checksum deploy could have been executed
b) above happens regularly enough for me, because we upload large files, where some files are just symlinks, and Artifactory doesn't really support symlinks transparently, but they work OK because of Artifactory's checksum deploy feature / checksum comparison.
As for the last question:
I am not sure why our Artifactory instance returns 404 sometimes. We've started to look into that, there is nothing in the Artifactory logs itself. It does looks like a network issue (TCP timeouts), our IT department is still looking into that. What basically happens is that the upload stalls, and after a while Artifactory returns 404, probably because of hitting an internal timeout.
So yes, this patch here is maybe a work-around. But keep in mind that network issues are nothing unusual, so making the plugin more resilient against that doesn't seem like a bad idea.

@andred,
I'd be really surprised if the 404 is returned by Artifactory to a PUT request.
If it does, you should find this in the requests.log, which is available also from the Artifactory UI. If you do find a 404 there, can you please share it with us?
It is more likely that the 404 is returned by a proxy somewhere between the build agent and Artifactory. In that case, there's a chance that only some of the requests go through that proxy (for example, a load balancer can cause this) and that's why your workaround works.
If indeed this is what happens, this is not a common / normal network issue / hiccup, and this workaround may hide a real issue on your environment. Apache's http client does nor support retries for 404 responses because it is wrong to do so. I therefore don't think we should support this workaround officially. The build-info client is a core components used by many systems, and it is expected to use the common http standards.
You can use your fork till the issue is resolved, but I'd recommend fixing the real cause of the issue and revert back to the master branch code once the issue is resolved.

Thank you for sharing your thoughts.
I think I remembered wrong, though. I looked at some older logs that our Artifactory admins shared with me last year:
user@host $2022 ~/tmp > grep<PHONE_NUMBER>3942.rootfs.ext4 ArtifactoryLogs.txt   Line 19868:<PHONE_NUMBER>0036|2|REQUEST|<IP_ADDRESS>|username|PUT|/my-project-images/nightlies/2018-10-09T17%3A17%2B00%3A00/the-platform/my-project-image-the-platform-20181009173942.rootfs.ext4;build.timestamp=1539105430142;build.name=my-project+%3A%3A+yocto-my-project-pipelines+%3A%3A+space-integration-yocto-master;build.number=31|HTTP/1.1|404|0  Line 20378:<PHONE_NUMBER>0140|62995|REQUEST|<IP_ADDRESS>|username|PUT|/my-project-images/nightlies/2018-10-09T17%3A17%2B00%3A00/the-platform/my-project-image-the-platform-20181009173942.rootfs.ext4;build.timestamp=1539105430142;build.name=my-project+%3A%3A+yocto-my-project-pipelines+%3A%3A+space-integration-yocto-master;build.number=31|HTTP/1.1|404|486539264  Line 21898:<PHONE_NUMBER>1744|21|REQUEST|<IP_ADDRESS>|username|PUT|/my-project-images/nightlies/2018-10-09T17%3A17%2B00%3A00/the-platform/my-project-image-the-platform-20181009173942.rootfs.ext4;build.timestamp=1539105430142;build.name=my-project+%3A%3A+yocto-my-project-pipelines+%3A%3A+space-integration-yocto-master;build.number=31|HTTP/1.1|201|0
As can be seen, there is 404 for the PUT of initial checksum deploy, which is fine and expected.
Afterwards, there is a 404 on the PUT when actually trying to upload the 460MiB file.
Please correct me if I'm interpreting the log incorrectly.

Thanks @andred - you are correct!
This seems like a bug in Artifactory, which RTFACT-18518 tracks.
When the 404 is returned, the Artifactory log should also include an exception, which should hint to the real cause fo the issue. The exception should be logged at the time that the 404 is returned. If you're unable to track down the exception in the log, try looking for a message that contains failed due to

Thanks.
Unfortunately, there is no 'failed due to' in the request.log that I have. Would it have been in a different log file? Those three lines pasted above are the only mentions of the file involved.

@andred - I discussed this with the developers.
Once RTFACT-18518 is fixed, Artifactory will return the correct HTTP response code instead of a 404.
There is however a real problem, which can be traced in Artofactory logs at around the time of the failure.

Since RTFACT-18518 tracks this issue, I think that this once cam be closed.


firebase testing for sercurity rules

once firebase rules are setup we need some testing probably with register or login that we can use these rules correctly

Due to knowledge and time I was unable to complete this task


Remove cleartext credentials from config.json and add pop up login

Hi Karim,
Thanks for creating such a useful tool.
I noticed that in order to connect to vCenter you should insert credentials in the config.json file. I would recommend to use the built in pop-up login mechanism in PowerShell.

Thanks for the notice and for the PR @alaanasser00,
I have added a tiny review, I also have one tiny request to keep the docs inline, please update the usage section of the README file.

LGTM, thank you so much @alaanasser00!