Application Insights SDK Not Running In Local Mode (ASP Core 2.0)

Moved from https://github.com/Microsoft/ApplicationInsights-Home/issues/177
Reported by @Faraz2018
@cijothomas can you please take a look?

I am using VS 2017 (Community). While following the beginners' course on MVA https://mva.microsoft.com/en-US/training-courses/aspnet-core-beginner-18153?l=AWB8596dE_6211787171 I came to know about Application Insights being available for running in local mode. Upon adding the same to my sample project I was surprised to find the activity to be showing 0's (zeros) for All?
I had:

Right clicked the project, selected Add > Application Insights Telemetry
Upon Application Insights Configuration Page selected Start Free
On the next page I selected the option Or just add SDK to try local only mode at the bottom of page
Ran the application by pressing IIS Express
The Application Insights button I had added on the toolbar didn't show any number of activity
Upon clicking that button the Application Insights Search page appeared, but with every event/result as 0 (Zero) as per the image below

The Immediate Window shows Application Insights Telemetry (unconfigured) at some places.
The Startup file is as:
using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using Microsoft.AspNetCore.Builder;
using Microsoft.AspNetCore.Hosting;
using Microsoft.AspNetCore.Http;
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Logging;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Configuration;

namespace FAQ
{
    public class Startup
    {
        public Startup(IConfiguration config)
        {
            configuration = config;
        }

        public IConfiguration configuration { get; }

        public void ConfigureServices(IServiceCollection services)
        {
            services.AddDbContext<FAQ_TempDB>(options => options.UseInMemoryDatabase("Some_Name"));
            services.AddLogging();
            services.AddMvc();
        }

        public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
        {
            loggerFactory.AddConsole();
            loggerFactory.AddDebug();
            app.UseDeveloperExceptionPage();
            app.UseBrowserLink();
            app.UseStaticFiles();
            app.UseMvc(routes =>
            {
                routes.MapRoute(
                    name: "default",
                    template: "{controller=Home}/{action=Index}/{id?}");
            });
        }
    }
}

while I have used it on the Create Model in a single CRUD page model as:
using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using Microsoft.AspNetCore.Mvc;
using Microsoft.AspNetCore.Mvc.RazorPages;
using FAQ.Models;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Logging;

namespace FAQ.Pages
{
    public class CreateModel : PageModel
    {
        private readonly FAQ_TempDB _TempDB;

        private ILogger<CreateModel> Log;

        public CreateModel(FAQ_TempDB tempDB, ILogger<CreateModel> log)
        {
            _TempDB = tempDB;
            Log = log;
        }

        [TempData]
        public string xMessage { get; set; }

        [BindProperty]
        public Worker Worker { get; set; }

        public async Task<IActionResult> OnPostAsync()
        {
            if (!ModelState.IsValid) {return Page();}

            _TempDB.Workers.Add(Worker);
            await _TempDB.SaveChangesAsync();
            var xMsg = $"Record of {Worker.W_FName} added!";
            xMessage = xMsg;
            Log.LogCritical(xMsg);                                        //LOG ENTRY NOT WORKING
            return RedirectToPage("/BackToDB");
        }
    }

    public class BackToDBModel : PageModel
    {
        private readonly FAQ_TempDB _TempDB;

        public BackToDBModel(FAQ_TempDB tempDB)
        {
            _TempDB = tempDB;
        }

        public IList<Worker> Workers { get; private set; }

        [TempData]
        public string xMessage { get; set; }

        public async Task OnGetAsync()
        {Workers = await _TempDB.Workers.AsNoTracking().ToListAsync();}

        public async Task<IActionResult> OnPostDeleteAsync(int id)
        {
            var worker = await _TempDB.Workers.FindAsync(id);
            if (worker != null)
            {
                _TempDB.Workers.Remove(worker);
                await _TempDB.SaveChangesAsync();
            }
            return RedirectToPage();
        }
    }

    public class EditModel : PageModel
    {
        private readonly FAQ_TempDB _TempDB;

        public EditModel(FAQ_TempDB tempDB)
        {_TempDB = tempDB;}

        [BindProperty]
        public Worker Worker { get; set; }

        public async Task<IActionResult> OnGetAsync(int id)
        {
            Worker = await _TempDB.Workers.FindAsync(id);
            if (Worker == null)
            {return RedirectToPage("/BackToDB");}
            return Page();
        }

        public async Task<IActionResult> OnPostAsync()
        {
            if (!ModelState.IsValid)
            {return Page();}

            _TempDB.Attach(Worker).State = EntityState.Modified;

            try
            {await _TempDB.SaveChangesAsync();}
            catch (DbUpdateConcurrencyException ex)
            {throw new Exception($"Worker {Worker.W_FName} Not Found!", ex);}

            return RedirectToPage("/BackToDB");
        }
    }
}

The app works well on console reflecting the requisite critical portion in red but I don't understand why not on the local mode of Application Insights? Is it necessary to use Azure?
Please see if any of you experts can help.
Thanks

@Faraz2018 I just downloaded VS Community 2017, and created a new Asp.Net Core Project - Web API (all defaults). And simply hit F5 to start debugging - I am able to see Application Insights events.
I didn't do anything special.
Before I ask folks who own ApplicationInsights VS experience , can you just retry only the same steps I mentioned and see if it helps.  Please let me know what your find after your try this. We can definitely involve right experts once your give it another try.

@faraz2018 Closing this issue as no-repro. Please reopen if you need help.


[NT-203] (2/2) Pledge summary section

📲 What

This is the second part of a 2-part work to build the Pledge summary section of the  Manage Pledge screen.

🤔 Why
Because the work has to be done 😎 .
🛠 How

Added a new property locationName to the Backing model.
Updated strings
Added missing tests

👀 See
It shouldn't have difference between the work done here and and this PR.

Hey I have the one question and also wondering if we could rename ManageViewPledgeViewController -> ManagePledgeViewController in this PR?

Yeah, since the PR is small and simple, could be a good opportunity to rename it


Add secret tasks

https://github.com/neuralaudio/hear-eval-kit/pull/209/files

Where is the README update? Docs must be up to date. We need docs both for us (with secret tasks) and end users


Can't render sidebar inside my split pane windows properly

I am using react and have a split pane window. Now I'd like to render the sidebar inside this pane window. I have sidebarStyle as following:

unfortunately it can not render properly as the height is always 0.
if I change the above position from "absolute" to "relative". It renders, but the sidebar fills the entire pane. The size is wrong.

Here is my wrapper for the pane window

If I render sidebar outside the pane window, it works.
Any suggestions are greatly appreciated.

Let me make it clear. If I render Sidebar without my SBContainer, it works great.
As long as I render it inside my SBContainer, it can not calculate the height of Sidebar (height=0). Here is my render function and the SBContainer.

Any suggestions are greatly appreciated.


[PIRK-9] Run Apache Rat checks as part of project POM verify

+1
Thanks for adding this into the project!


User reported error using ConsMarkovModel

A user reports an error with the following code:
https://gist.github.com/sbenthall/16ef17989a1ef336b83205a39a9121ce

“List in IncShkDstn is not the right length, it should be length equal to number of states”

And asks:

What exactly needs the format for the ‘IncShkDstn’ to be?
How does the variable ‘IncShkDstn’ relate to the variable ‘IncShkDstn_list’? Looking at the code, former is an element of latter. So I would imagine ‘IncShkDstn_list’ specifies the list of Income Distributions for each state for all points in time, whereas ‘IncShkDstn’ specifies the income distributions for all possible states at a given point in time.

The answers to these questions and the proper way to configure the model should be knowable through the documentation and examples. Will need to revisit these.
Here is the current documentation for the MarkovConsumerType model:
https://hark.readthedocs.io/en/latest/reference/ConsumptionSaving/ConsMarkovModel.html#HARK.ConsumptionSaving.ConsMarkovModel.MarkovConsumerType
A hint is in the docs for ConsMarkovSolution but this is not the place to document model configuration parameters!
https://hark.readthedocs.io/en/latest/reference/ConsumptionSaving/ConsMarkovModel.html#HARK.ConsumptionSaving.ConsMarkovModel.ConsMarkovSolver

Type signature on IncShkDstn_list in the solution class is wrong.
https://github.com/econ-ark/HARK/blob/f880be6f03a6eab351581438c6ce6a876022fc39/HARK/ConsumptionSaving/ConsMarkovModel.py#L89-L94
It says [[np.array]] but that is the old way of representing a distribution; should be [distribution.DiscreteDistribution]

There are a few idiosyncracies here.
First and most importantly, for some reason the MarkovConsumerType expects the user to set the IncShkDstn manually after the class has been initialized; you can't pass in the (list of lists of) distribtuion as a parameter.
So the code works with:
agent_markov = cmm.MarkovConsumerType(**markov_dict)
agent_markov.IncShkDstn =  [[Income_Dstn, Income_Dstn]]

The second issue is that the warning message is inaccurate. It reports a mismatched length when triggered by a mismatched type.

having the standard workflow be to change the IncShkDstn manually, especially when the code is producing a non-Markov version of the same variable automatically, is pretty broken. A symptom of the 'whiteboard' problem maybe. see #889.
This user expected to be able to input the distribution directly as a parameter, which is in line with #620

The "list of lists" to automatically create IncomeDstn for the Markov model
was definitely written, but it might have been lost somewhere along the
way.  Part of the reason it didn't originally exist is that the Markov
model is partly meant to represent / be used for models where our
permanent-transitory shock structure (with lognormal shocks) is not
appropriate.
IncomeDstn_list is internal to the solver, not user-facing. I believe
it's a re-naming of the IncomeDstn object passed to the solver, for
internal reference.  As the solver loops through the different states, it
loads one future-state-dependent IncomeDstn at a time into the attribute
IncomeDstn. This allows it to use some inherited code from its parent
solver.
The Markov solver is the biggest headache in HARK, in my opinion. There are
so many bizarre workarounds and patches so that it can kinda sorta use a
few lines of code from ConsIndShockSolver. The entire thing would be better
and easier to read if it didn't inherit from ConsIndShockSolver (i.e. we
abandoned solver classes).
On Wed, Feb 17, 2021 at 1:21 PM Sebastian Benthall<EMAIL_ADDRESS>wrote:

having the standard workflow be to change the IncShkDstn manually,
especially when the code is producing a non-Markov version of the same
variable automatically, is pretty broken. A symptom of the 'whiteboard'
problem maybe. see #889 https://github.com/econ-ark/HARK/issues/889.
This user expected to be able to input the distribution directly as a
parameter, which is in line with #620
https://github.com/econ-ark/HARK/issues/620
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/952#issuecomment-780753084, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFIMFSHIRFDZACD7M4DS7QCJTANCNFSM4XVNIEFA
.

Thanks for weighing in @mnwhite
I'm not sure where the Markov income distribution builder is; in most of the examples and Demarks that use the Markov class wind up building that distribution object externally.
I agree it sounds like it would be much better if the Markov classes didn't depend on ConsInShock.
It sounds like that is a more general architecture.
I don't know if you've seen the new MarkovProcess class in HARK.distribution, but I think it gets at the generality we could potentially have:
https://github.com/econ-ark/HARK/blob/master/HARK/distribution.py#L1012
I was just thinking of adding a ticket for a ConditionalDistribution, which would represent a distribution (such as an income distribution) that was conditional on another variable (such as a Markov state).
I think that would be a more explicit way of doing what is currently done with a list of distributions, which is especially confusing when it's a list-of-list-of-distributions, with very different semantics to the two 'list-of' operations.

With #955 merged, the next improvement from this is to allow income distributions to be input as a parameters.
That will have to wait until HARK 1.1

This issue can now be fixed with the constructors functionality. A new "Markov lifecycle income process" constructor can be written, which takes in TxN arrays of income parameters and loops over them to create the lifecycle income process.

@mnwhite I am adapting the ConsMarkovModel to solve a 2 Sector HANK model with Econ-ARK 13.0.
The parameters and IncShkDstn is a list of length T with N elements. Do you think it is better to have everything as arrays or lists?
Happy to make a PR once I am solving some other issues I am facing.

IncShkDstn should be a list of lists, but the raw parameters (PermShkDstn,
TranShkDstn, IncUnemp, UnempPrb) can be arrays. Note that the constructor
functionality that I mentioned is not in 0.13.x, so whatever you're
currently doing there won't be compatible with the most recent version of
HARK.
On Wed, Jul 10, 2024 at 10:14 AM Adrian Monninger @.***>
wrote:

@mnwhite https://github.com/mnwhite I am adapting the ConsMarkovModel
to solve a 2 Sector HANK model with Econ-ARK 13.0.
The parameters and IncShkDstn is a list of length T with N elements. Do
you think it is better to have everything as arrays or lists?
Happy to make a PR once I am solving some other issues I am facing.
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/952#issuecomment-2220631039, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFJFPD7L3VDBO4PVA4TZLU6VZAVCNFSM4XVNIEFKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMRSGA3DGMJQGM4Q
.
You are receiving this because you were mentioned.Message ID:
@.***>


Unspecified error

I tryed your exemple and I received this:

I got the same problem with an app whilch was working. And now i got this error.

Anyone else got this issue?
Its very strange and it's look like it is inside ga lib line 91

:+1: Same issue. Maybe the API changed?

Yeah its a big bug. I got this since a month and none answer.
I am considering changing lib..

Hi every one, 2 months later i would like to know if someone found a solution for this.
It still does'nt work for me.

I thin that author abandoned this project.
2015-09-04 8:54 GMT-03:00 Oldwo1f<EMAIL_ADDRESS>
Hi every one, 2 months later i would like to know if someone found a
solution for this.
It still does'nt work for me.
—
Reply to this email directly or view it on GitHub
https://github.com/ncb000gt/node-googleanalytics/issues/39#issuecomment-137716360
.

did you find a way to use goodanalityc in node projects?
2015-09-04 14:10 GMT+02:00 Igor C. de Paula<EMAIL_ADDRESS>
I thin that author abandoned this project.
2015-09-04 8:54 GMT-03:00 Oldwo1f<EMAIL_ADDRESS>
Hi every one, 2 months later i would like to know if someone found a
solution for this.
It still does'nt work for me.
—
Reply to this email directly or view it on GitHub
<
https://github.com/ncb000gt/node-googleanalytics/issues/39#issuecomment-137716360
.

—
Reply to this email directly or view it on GitHub
https://github.com/ncb000gt/node-googleanalytics/issues/39#issuecomment-137718583
.

No, No yet.
2015-09-04 9:11 GMT-03:00 Oldwo1f<EMAIL_ADDRESS>
did you find a way to use goodanalityc in node projects?
2015-09-04 14:10 GMT+02:00 Igor C. de Paula<EMAIL_ADDRESS>
I thin that author abandoned this project.
2015-09-04 8:54 GMT-03:00 Oldwo1f<EMAIL_ADDRESS>
Hi every one, 2 months later i would like to know if someone found a
solution for this.
It still does'nt work for me.
—
Reply to this email directly or view it on GitHub
<

https://github.com/ncb000gt/node-googleanalytics/issues/39#issuecomment-137716360

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/ncb000gt/node-googleanalytics/issues/39#issuecomment-137718583

.

—
Reply to this email directly or view it on GitHub
https://github.com/ncb000gt/node-googleanalytics/issues/39#issuecomment-137718734
.

Sorry. I haven't exactly abandoned any projects, I'm just extremely busy with other stuff so I don't have a lot of time to spend on my side projects. And then, the projects that tend to get attention are the ones I use. I haven't used this project in years. =\
I'd be happy to accept pull requests.

Ok no problem man.
But the thing is :  Your project was working and all of a sudden it dont.
Maybe you can provide a little help to help to identify that bug. Then i ll
try to fix it and make a pull request
2015-09-04 18:55 GMT+02:00 Nick Campbell<EMAIL_ADDRESS>
Sorry. I haven't exactly abandoned any projects, I'm just extremely busy
with other stuff so I don't have a lot of time to spend on my side
projects. And then, the projects that tend to get attention are the ones I
use. I haven't used this project in years. =\
I'd be happy to accept pull requests.
—
Reply to this email directly or view it on GitHub
https://github.com/ncb000gt/node-googleanalytics/issues/39#issuecomment-137789933
.

So, I looked around briefly and it looks like the ClientLogin api no longer works.
It requires OAuth 2.0 to use the apis...
https://developers.google.com/identity/protocols/AuthForInstalledApps

Sorry, man, I understand. I think that we can close this issue, yes?

yes you can
2015-09-06 14:13 GMT+02:00 Igor C. de Paula<EMAIL_ADDRESS>
Sorry, man, I understand. I think that we can close this issue, yes?
—
Reply to this email directly or view it on GitHub
https://github.com/ncb000gt/node-googleanalytics/issues/39#issuecomment-138078021
.


Add Python 3.12-dev to tests actions

We should add Python 3.12-dev here and see if all tests pass: https://github.com/microsoft/vscode-autopep8/blob/main/.github/workflows/pr-check.yml

Can you assign this to me?


Better Documentation

Hello,
Is this plugin capable with Okta?
I need help setting this up
I am setting up an app in my Okta platform, it is asking for the Single Sign On URL, Audience URI.

Any update on this?

Hello, the plugin is based on onelogin saml2. Please check your issue at the address below.
https://github.com/onelogin/php-saml/issues


Can't Access filesystem. - Electron

Expected Behavior

use fs.writeFileSync() to write a file to the user's local folder.
use Jim.read('file://..path/to/file') to read the file, manipulate it and save it.

Current Behavior

fs.writeFileSync() works fine.
Jimp.read('file://..path/to/file') throws an error "Can't Access filesystem"
The issue is on line 33773 where it is checking if fs is available (which it should be)

Failure Information (for bugs)
I'm not sure why this is failing..

Steps to Reproduce
Follow the Current behavior to reproduce the issue.

Screenshots

Context

Jimp Version: 0.5.0
Operating System: MacOS
Node version: 6.4.1

Failure Logs

Hi, I put it into a new project and it appears to work. I place the same code in my existing project and it doesn't. So there is something in my project causing the problem. There is a strange thing happening. When I save the file using fs, and then try to access that file using Jimp in the one that works, it's finding the file fine without needing to add 'file://' to it. In the version that doesn't work, it's looking for the file at http://localhost:9080 where it's obviously not located because it is in the user folder. Do you happen to know why this could be happening?

I've never tried to use it in an electron app before.
The offending code is probably somewhere in:
https://github.com/oliver-moran/jimp/blob/master/packages/core/src/index.js#L83
loadFromURL might be getting called when it shouldn't.
Can you link the sample repo with steps to reproduce?

Hi there!
I am facing the same issue: "Cannot access filesystem".
Here is the repository on GitLab: https://gitlab.com/Kristof89/s3-image-uploader
I have also raised an issue on Stackoverflow: https://stackoverflow.com/questions/52715291/error-cant-access-the-filesystem-you-can-use-the-getbase64-method-electronjs

I've never tried to use it in an electron app before.
The offending code is probably somewhere in:
https://github.com/oliver-moran/jimp/blob/master/packages/core/src/index.js#L83
loadFromURL might be getting called when it shouldn't.
Can you link the sample repo with steps to reproduce?

I'd be happy to link you to my project's repository but it is private. Is there a way that you can contact me so that I can provide it to you?
As far as I can see, it might be an issue with the combination of node modules that I am using conflicting with JIMP. When I have a clean project and only use Jimp and FS, it works fine. When I use the same code in my project, it doesn't work and I get the issue where it looks for the files using https instead of the local file.
I've tried completely uninstalling and reinstalling Jimp, using the latest version, clearing the NPM cache, deleting and reinstalling all Node modules and I've even tried disabling as many modules as possible to see if there were potential, although unlikely conflicts.
In my case, it really appears that the node modules like fs and path are not being loaded within the JIMP script itself.

You can shoot me an email at<EMAIL_ADDRESS>and i'll try to take a look

@Ksistof Did the answer on stack overflow solve your problem? It seems like electron apps need to access the filesystem in a different thread.
@SwissArmyMike You should take a look too. Does your project happen to use webpack?

The main.js should create windows and handle all the system events your application might encounter. A more complete version of the above example might open developer tools, handle the window being closed, or re-create windows on macOS if the user clicks on the app's icon in the dock.

@Ksistof Did the answer on stack overflow solve your problem? It seems like electron apps need to access the filesystem in a different thread.
@SwissArmyMike You should take a look too. Does your project happen to use webpack?

I'm not sure how the window creation applies to this issue. Yes, I'm using webpack. Do you have a link to the stack overflow you are referencing?

@SwissArmyMike https://stackoverflow.com/questions/52715291/error-cant-access-the-filesystem-you-can-use-the-getbase64-method-electronjs
if you are using webpack it might be resolving the browser version of jimp and that could be messing you up. Look at my comment here

@SwissArmyMike https://stackoverflow.com/questions/52715291/error-cant-access-the-filesystem-you-can-use-the-getbase64-method-electronjs
if you are using webpack it might be resolving the browser version of jimp and that could be messing you up. Look at my comment here

It looks like this worked for me!
Adding this to the webpack config files solved the problem.
resolve: {
    mainFields: ['module', 'main']
 }

Awesome! glad I could help


[WIP] ESLint Config Migration: Fix indentation linter warnings

Summary
This is a PR that should be merged into #1024 and incrementally addresses #842.
This PR contains no configuration changes, just indentation fixes.
Before
✖ 927 problems (737 errors, 190 warnings)
  187 errors and 0 warnings potentially fixable with the `--fix` option.

After
✖ 875 problems (685 errors, 190 warnings)
  135 errors and 0 warnings potentially fixable with the `--fix` option.

Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

Merged this into #1024 - closing.


Add user information to log

Codecov Report

Merging #2392 into dev will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev    #2392      +/-   ##
============================================
- Coverage     32.80%   32.78%   -0.03%
+ Complexity     2267     2265       -2
============================================
  Files           422      422
  Lines         19830    19830
  Branches       2363     2363
============================================
- Hits           6506     6502       -4
+ Misses        12746    12744       -2
- Partials        578      584       +6

Impacted Files
Coverage Δ
Complexity Δ

...heduler/api/controller/TaskInstanceController.java
22.22% <0.00%> (ø)
2.00 <0.00> (ø)

...scheduler/api/controller/TaskRecordController.java
14.28% <0.00%> (ø)
2.00 <0.00> (ø)

...he/dolphinscheduler/api/service/LoggerService.java
82.60% <0.00%> (-8.70%)
6.00% <0.00%> (-1.00%)

...e/dolphinscheduler/remote/NettyRemotingClient.java
51.07% <0.00%> (-2.88%)
9.00% <0.00%> (-2.00%)

...inscheduler/common/task/sqoop/SqoopParameters.java
80.00% <0.00%> (ø)
15.00% <0.00%> (ø%)

...scheduler/alert/template/AlertTemplateFactory.java
71.42% <0.00%> (+28.57%)
3.00% <0.00%> (+1.00%)

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 320469d...e4abbe0. Read the comment docs.


How to add Child Controller, When it  is an uncertain number

    override func viewControllersForPagerTabStrip(pagerTabStripController: PagerTabStripViewController) -> [UIViewController] {

        var controllers = [UIViewController]()

        for _ in 0...kinds.count - 1 {
            let chid = SecondViewController()
            controllers.append(chid)
        }
        return controllers
    }

kinds.count is Child Controllers number but it request from internet ,it is an uncertain number

@volkanx @dowhilenet  Like reusing the same viewconrtoller with different data? check attachment . Is this achievable with this framework?

@khanxc sure, it is possible.
when you create the child view controller you need to assign the data it's going to display. Something like:
let child1 = MenuViewController()
child1.menuItems = [BurgerItem] // Populate the data beforehand
children.append(child1)

let child2 = MenuViewController()
child2.menuItems = [ChickenItem]
children.append(child2)

Same view controller, different data to display

Well explained by @volkanx. I would just add that children is an array of UIViewControllers, therefore in the for loop you need to instantiate let child = UIStoryboard(name: "Main", bundle: nil).instantiateViewController(withIdentifier: "ChildViewController") as! ChildViewController
That works fine for me.
Now the question i have is how can we deallocate some of the ViewControllers we don't need. For example, i would like to have 3 adjacent ViewControllers at the same time (so to keep smooth paging) and the rest should be deallocated so that the memory would be saved. My ViewControllers are quite heavy and thats the reason i would like them to be deallocated.
Any help would be appreciated


CAN'T LOGIN ADMIN

Dear Mr. Zulfi
That is a great app I think. I am learning and trying to use it. Unfortunately, I cannot use login admin. Can you tell me how to fix it?
Thanks a lot.

coba bro...
look at cbt.sql or your database, table admin. row Administrator.
change default password '2y$12$b66V3qOxwG44wSaoFLK.g.g4lN2T.g.CnfeHCBuNHWA8x/Sb1xBGS' with this one '$2y$12$EgzAfVa3E6HzfmownYWs0eh2.G0ZylFrnoIj6gt87.F6HnmcdS7aa' dibaca oleh manusia normal "tank-sys".
god luck.
we can not decrypt password_hash php. we only can encrypt and password_verify.

Oke


Change model

Motivation and Context
The API for getting sample units has changed significantly. It no longer
inserts something into the database which is later polled and actioned
on. The changes should be backwards compatible as the API should allow
fields that are not defined.
What has changed
Changed CollectionExerciseJobCreationRequestDTO to SampleUnitRequest removing additional fields
How to test?

Checkout https://github.com/ONSdigital/rm-sample-service/pull/62
Run integration tests

Links
PR
Trello

If we push this change, we might have to modify the sample and collex services to accommodate. Have we tested this change with those services?

Superseded by changes from @NickGrantONS


Add error reporting in topology builder and provider

In the (upcoming) files topology/builder.go and provider/provider.go, we should replace the logger by some kind of error reporting mechanism that not only logs the errors but also aggregates them, in order to have a list of the issues present in the dynamic configuration.
This could be used later on to add an entrypoint to the API to list the configuration errors with precise context.

We could also have the status of objects propagated with errors, so that multiple tools could handle parsing or configuration errors at once.

@dtomcej I'm not sure what you mean exactly? What tools do you have in mind? What would be the benefit of multiple tools handling those errors at once?

@Ullaakut I see there being 2 sources of errors: User errors, and provider errors.
User errors are when a user adds an annotation that is broken, or something, and doesn't realize it.
It would be nice to update the status on that kubernetes object to indicate the error:
For example, if they add a broken annotation to a service, update that service status with the error. Then it can be visible to the user that the error is with that specific service, and what is happening.
The provider errors would be passed along as per your original issue.

Also, is this related to https://github.com/containous/traefikee/issues/2450?

It is, yes.
As for the source of errors, we were planning on getting all issues with the generation of dynamic configuration. We thought the topology and provider parts were enough, can you point me in the direction of the other packages where we'd be able to report useful errors?

Right now errors can also come from deployment, but that may be deprecated in the future.

Closing in favor of #522.


Refresh commands

This adds a command to allow for the refreshing of keys to get around the 1 hour limits put in place by AWS.

See: https://github.com/wheniwork/keyme/issues/10#issuecomment-326468253


Building successful but python modules not being found

I am building the LungAIRApp-real file but, when I run it, various things that it is supposed to be finding aren't found.  That is, the app launches and shows a traditional 3D Slicer layout but all (or at least much) of the lung-air functionality is missing.  Especially if you know off the top of your head, I could use help with which of the following are supposed to be built with the LungAIRApp and which are supposed to be installed on my system already?  Or if you have any other leads, like environment variables that I need to set or not set?!

I have Qt 5.15.2 and am building with -DQt5_DIR:PATH="$HOME"/Support/Qt/5.15.2/gcc_64/lib/cmake/Qt5 to the point of getting an app that runs (without lung air functionality), so I am especially perplexed that qt cannot be found.

I am using Ubuntu 20.04 and Python 3.8.10.

"ModuleNotFoundError: No module named" for ctk, qt, SegmentEditorEffects, slicer, SubjectHierarchyPlugins, torch, and vtk.

"Fail to instantiate module" for AddManyMarkupsFiducialTest, ColorLegendSelfTest, CropVolumeSequence, DICOM, DICOMEnhancedUSVolumePlugin, DICOMGeAbusPlugin, DICOMImageSequencePlugin, DICOMPatcher, DICOMScalarVolumePlugin, DICOMSlicerDataBundlePlugin, DICOMVolumeSequencePlugin, Home, MarkupsInCompareViewersSelfTest, MarkupsInViewsSelfTest, NeurosurgicalPlanningTutorialMarkupsSelfTest, PluggableMarkupsSelfTest, ScreenCapture, SegmentEditor, SegmentStatistics, and SelfTests.

Let me try doing a clean build and see if I can reproduce the error.
There's only one environment variable that I set while building: adding my custom path to qt-everywhere-build-5.15.1/lib into my LD_LIBRARY_PATH. That doesn't seem to be the issue here though, since you are building just fine and what ends up not being found is on the python end.
When you run the application is it possible for you to get into a python console?
When I go into my python console and
import ctk
print(ctk.__file__)

I get '/home/ebrahim/LungAIR-SuperBuild/CTK-build/CTK-build/bin/Python/ctk/__init__.py'.
Does your superbuild directory have, for example, a successfully built CTK-build/CTK-build/bin/Python ?

Thank you @ebrahimebrahim for creating this issue and the help.  If I go into the Python Interactor, via ctrl-3 from LungAIR, and type import ctk then I get No module named 'ctk'.  I do have the file lungair-SuperBuild-Debug-Qt5.15.2/CTK-build/CTK-build/bin/Python/ctk/__init__.py for what it is worth.
It feels like a missing environment variable (or part) to me, such as a missing entry in my LD_LIBRARY_PATH or the corresponding thing for Python.
If not that ... I have a fear that the problem is related to the fact that I recently upgraded my system to use Python 3.8 as the default, via sudo update-alternatives because I also have additional Python versions installed.  Now that 3.8 is the default, perhaps I need to apt reinstall several packages so that 3.8 can find them.

Slicer should pull in and build its own python. In the superbuild directory there should be a directory with a name like Python-3.9.10 which contains the python source code that will be compiled during the build process. This makes me think that your system Python can't cause problems... but I'm not confident in saying that.
Hmmm we can try using ldd to see where the executable is trying to look for the libraries it linked to.
When cd into the superbuild directory and then run
ldd Slicer-build/bin/LungAIRApp-real  | grep python

I get
libpython3.9.so => /home/ebrahim/LungAIR-SuperBuild/python-install/lib/libpython3.9.so (0x00007effe3084000)

Here is another one we might look at:
ldd Slicer-build/bin/LungAIRApp-real  | grep CTK

gives me
	libCTKVisualizationVTKWidgets.so.0.1 => /home/ebrahim/LungAIR-SuperBuild/CTK-build/CTK-build/bin/libCTKVisualizationVTKWidgets.so.0.1 (0x00007f4d630ae000)
	libCTKScriptingPythonWidgets.so.0.1 => /home/ebrahim/LungAIR-SuperBuild/CTK-build/CTK-build/bin/libCTKScriptingPythonWidgets.so.0.1 (0x00007f4d63094000)
	libCTKDICOMWidgets.so.0.1 => /home/ebrahim/LungAIR-SuperBuild/CTK-build/CTK-build/bin/libCTKDICOMWidgets.so.0.1 (0x00007f4d62fcf000)
	libCTKWidgets.so.0.1 => /home/ebrahim/LungAIR-SuperBuild/CTK-build/CTK-build/bin/libCTKWidgets.so.0.1 (0x00007f4d62d50000)
	libCTKImageProcessingITKCore.so.0.1 => /home/ebrahim/LungAIR-SuperBuild/CTK-build/CTK-build/bin/libCTKImageProcessingITKCore.so.0.1 (0x00007f4d5a9b8000)
	libCTKVisualizationVTKCore.so.0.1 => /home/ebrahim/LungAIR-SuperBuild/CTK-build/CTK-build/bin/libCTKVisualizationVTKCore.so.0.1 (0x00007f4d5a961000)
	libCTKDICOMCore.so.0.1 => /home/ebrahim/LungAIR-SuperBuild/CTK-build/CTK-build/bin/libCTKDICOMCore.so.0.1 (0x00007f4d5a8a0000)
	libCTKScriptingPythonCore.so.0.1 => /home/ebrahim/LungAIR-SuperBuild/CTK-build/CTK-build/bin/libCTKScriptingPythonCore.so.0.1 (0x00007f4d5a826000)
	libCTKCore.so.0.1 => /home/ebrahim/LungAIR-SuperBuild/CTK-build/CTK-build/bin/libCTKCore.so.0.1 (0x00007f4d5a75d000)
	libPythonQt.so => /home/ebrahim/LungAIR-SuperBuild/CTK-build/CMakeExternals/Install/lib/libPythonQt.so (0x00007f4d58aef000)

@Leengit Try now (without the symbolic link).
On first run you should get an error saying that monai is not found, then you can use the button to install monai and restart the application. Then it should work. Note that it will freeze up for a while during the large torch download. Let me know how it goes!

Yes, very good, it now launches, warned me about missing MONAI, and gave me a nice button in the Advanced section to check for and install MONAI.  I also installed ITK-python, pandas, and matplotlib.
Perhaps unrelated and perhaps merely an ignorable warning but, just in case ... upon relaunch the console is saying
Warning: Found an unexpected qMRMLPlotWidget named "qMRMLPlotWidget"; there may be UI setup issues.

Great! Yes, that warning is ignorable.... I need to fix an assumption I was
making about Slicer's creation and naming of qMRMLPlotWidgets on launch.
Best,
Ebrahim
On Wed, Jul 20, 2022 at 1:59 PM Lee Newberg @.***>
wrote:

Yes, very good, it now launches, warned me about missing MONAI, and gave
me a nice button in the Advanced section to check for and install MONAI. I
also installed ITK-python, pandas, and matplotlib.
Perhaps unrelated and perhaps merely an ignorable warning but, just in
case ... upon relaunch the console is saying
Warning: Found an unexpected qMRMLPlotWidget named "qMRMLPlotWidget"; there may be UI setup issues.
—
Reply to this email directly, view it on GitHub
https://github.com/KitwareMedical/lungair-desktop-application/issues/39#issuecomment-1190586953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOK5T2MWYGV5NRVQUIXYBDVVA5A5ANCNFSM54A4BY3A
.
You are receiving this because you modified the open/close state.Message
ID: <KitwareMedical/lungair-desktop-application/issues/39/1190586953@
github.com>


Add optional FJP args for indexing and quantization

By default, SIMD operations are using PhysicalCoreExecutor and non-SIMD operations are using FJP.commonPool().
With this addition, one can define custom fork-join pools for these tasks in GraphIndexBuilder, ProductQuantization and BinaryQuantization.

Thanks, looks reasonable in principle.  I'll ask @jkni to review.
In the meantime, would you mind signing the CLA at https://cla.datastax.com/ ?


Implement langlib.xml functions to support new xml proposal.

Description:
We need to implement the xml langlib functions to support the xml proposal.
Proposed langlib functions https://github.com/ballerina-platform/ballerina-spec/blob/master/lang/proposals/xml/xml.bal
This is a part of https://github.com/ballerina-platform/ballerina-lang/issues/19570
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

This is fixed with https://github.com/ballerina-platform/ballerina-lang/pull/21603


AKS Spot Nodepool error

Bicep version
0.28.1 (ba1e9f8c1e)
Describe the bug
When we try to create an AKS Nodepool with scaleSetPriority value to Spot we receive the following error:
{"code": "InvalidParameter", "message": "Preflight validation check for resource(s) for container service aks-neuralsearchx-prod in resource group neuralsearchx_group failed. Message: The value of parameter agentPoolProfile.upgrade.maxSurge is invalid. Error details: Spot pools can't set max surge. Please see https://aka.ms/aks-naming-rules for more details.. Details: "}
We are not setting maxSurge, neither any upgradeSettings. We have tried to set this parameter as null, but no success.
To Reproduce
Try to create an AKS nodepool with spot nodes using bicep script.

We'll get a fix in so that in the future you don't need this workaround (setting Type).
It won't roll out for a few weeks though


Updated to README

"I have read the CLA Document and I hereby sign the CLA"

"I have read the Code of Conduct and I hereby accept the Terms"


Session Error: Dialog[/] not found. when trying to talk to Skype Bot [NodeJS]

I'm facing the following error when I'm trying to talk to Skype Bot using the skype-sdk for NodeJS:
Session Error: Dialog[/] not found.

You can see that in my code I've added / as one of the dialogs:
bot.add('/', dialog);
botSkype.add('/', function (session) {
   session.send('Hello World');
});
...
// Setup Restify Server
var server = restify.createServer();
server.post('/api/messages', bot.verifyBotFramework(), bot.listen());
server.post('/v1/chat', skype.messagingHandler(botService));
server.listen(process.env.PORT || 8080, function () {
    console.log('%s listening to %s', server.name, server.url);
});

Any help?

Interesting... Have you tried running the more stripped down hello-SkypeBot example? Just wondering if you get the same issue. I see you trying to run two different styles of bot. I can't think of any reason why that wouldn't work but it is an added complexity.

Hi @Stevenic, as you said I'm using two kinds of bot sharing the same logic but I also think that should'nt be related. I'm going to execute the hello-world example I can find in the web and get back to you with the results.

If I try, exactly, the hello-world I don't get the error message but it doesn't work neither. I'm using ngrok as the tutorial says, and I cand see the request coming to the bot application, but I can't see any reply on the Skype screen:

As you can see, no answer:

My by band, when I replace my code with the one from the tutorial I replaced the appsecret too. Now, the tutorial is working fine, without any error. Probably, the original error is related to Luis Dialog inside Skype Bot?

Ok!! I've found the error and fixed, but now I'm not sure if it is a bug or not.. :)

Glad you found it. What did you have to do? I can tell you if I think it's a bug :)

I'm going to try to explain the situation:
The problem was I have two SkypeBots with the same botService object. One without any dialog assigned and the other as you could see with dialogs asigned. When I create the REST server I use the one with the dialogs assigned, but even doing that it seems that it uses the first one without the dialogs.
I'm going to create a small test to reproduce the error and I get back to you and you can decide if it is a bug or not.

Here is the code without my keys :)
var restify = require('restify');
var builder = require('botbuilder');
var assert = require('assert');
const skype = require('skype-sdk');

// Initialize the BotService
const botService = new skype.BotService({
    messaging: {
        botId: "XXX",
        serverUrl : "https://apis.skype.com",
        requestTimeout : 15000,
        appId: "XXX",
        appSecret: "XXXX"
    }
});

// Create bot and add dialogs
var bot = new builder.SkypeBot(botService);
var botSkype = new builder.SkypeBot(botService);
dialog.setThreshold(0.5);

botSkype.add('/', function (session) {
   session.send('Hello World');
});

// Setup Restify Server
var server = restify.createServer();
server.post('/v1/chat', skype.messagingHandler(botService));
server.listen(process.env.PORT || 8080, function () {
    console.log('%s listening to %s', server.name, server.url);
});

I think the problem here is that when you link the REST server with the bot you don't use the bot object but the botService object which is related to two bot instances, and it have to choose one of them and I think it chooses the first one.

The issue is both bots are sharing the same botService instance and sharing the same rest endpoint. It will be somewhat random as to which one replies

you need 2 separate botService instances and 2 different endpoints

I have no problem  closing the issue if you tell me that's expected behavior :)

Knowing how the Skype service works internally I'd say it's pretty much expected.

Thank you for all your support :)

i am using FormDialog to create form and is getting rendered properly in emulator.
here is the image in emulator

however when i integrate same thing in skype ,it shows blue screen dialog anf does not dispaly button/choice  to select category.
here is image in skype

could you please let me know problem ?

I cannot publish my skype bot. Plz help for me

As per my understanding,You need to host it somewhere on the cloud/on
premise server. localhost can not be used.
On Tue, Jul 19, 2016 at 6:52 PM, santhosh117<EMAIL_ADDRESS>wrote:

I cannot publish my skype bot. Plz help for me
[image: capture]
https://cloud.githubusercontent.com/assets/20519603/16951249/d26a2b1c-4de1-11e6-8c82-4d8471f0f558.PNG
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/365#issuecomment-233630320,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGMJefQqhn42xo8CUKdlXie4vKDRyFVoks5qXM-PgaJpZM4IpTwz
.

thanks a lot @dharmendracs. I install ngrok i got https protocol using that as a message endpoint


Slightly less bad

There is an enormous amount of work to do in order to clean up the mux layer's outgoing connection support. I have 12 issues in my personal list and that is without having done a proper audit of all the connections. But we have already checked in a bunch of this code and this version should (famous last words) be better than what we have now.
See the comments on the commits for a summary of the changes

This change is

PR is added to the queue for testing as 1. task. (92b8613)

Test (Fail) 63008475 build is completed (92b8613)
See https://github.com/ThaliTester/TestResults/tree/6300847592b8613_Slightly_less_bad_yaronyg/ for the logs

PR is added to the queue for testing as 1. task. (dba4423)

Test (Fail) 63008475 build is completed (dba4423)
See https://github.com/ThaliTester/TestResults/tree/63008475dba4423_Slightly_less_bad_yaronyg/ for the logs

PR is added to the queue for testing as 1. task. (d624ed8)

Test (Success) 63008475 build is completed (d624ed8)
See https://github.com/ThaliTester/TestResults/tree/63008475d624ed8_Slightly_less_bad_yaronyg/ for the logs

Test 63008475d624ed8(d624ed8) has failed
See https://github.com/ThaliTester/TestResults/tree/63008475d624ed8_Slightly_less_bad_yaronyg/ for the fail logs

@yaronyg: The CI build failure yesterday was not due to the changes in the PR. I today made some improvements to the build machine and hopefully we wouldn't see the same error again - will keep monitoring the situation.

I did not do a full code review, because fully understanding all the muxing code would require spending some amount of time with the review. If you absolutely want a full review before merging this, please assign back. Otherwise, from my perspective, this can be merged since there is anyways some amount of todo-tests and such that needs to be written.


There was an Internal Problem with your device Dialog then blank screen

Description
After installing any version 0.2.0_40 causes There was an Internal Problem with you device dialog resulting in factory reset but go back to version 0.2.0_40 works perfectly fine
Steps to reproduce
Root Using Patched Boot Image(Magisk)
Install Xposed
Install AndroidP-ify
ROM info
Official Motorola Stock 8.0-Moto X4(X19100-5)
Module version (build version)
AndroidP-ify version 0.2.0_40( Last Working Version)
AndroidP-ify version 1.0.0_116(Broken)
Xposed log / logcat
https://drive.google.com/file/d/1FooWHBlqT_iHSYJiJoCIX0GpaBPIVk4O/view?usp=sharing

Issue still persistent on version 1.0.0_146


Is there a way to config like *.domian.com?

got an error like this, any idea how to use the plugin?

I believe this example uses Let's Encrypt HTTP-01 challenge type. In order for Let's Encrypt to issue wildcard certs, the DNS-01 challenge type is required.
The DNS-01 challenge type is more complex and requires a DNS provider that provide a supported API. There is a list of supported DNS providers and some code to integrate with them at https://community.letsencrypt.org/t/dns-providers-who-easily-integrate-with-lets-encrypt-dns-validation/86438 .
Additionally, wildcard domains must be validated using the DNS-01 challenge type.
This means that you’ll need to modify DNS TXT records in order to demonstrate
control over a domain for the purpose of obtaining a wildcard certificate.

https://community.letsencrypt.org/t/acme-v2-and-wildcard-certificate-support-is-live/55579

I believe this example uses Let's Encrypt HTTP-01 challenge type. In order for Let's Encrypt to issue wildcard certs, the DNS-01 challenge type is required.
The DNS-01 challenge type is more complex and requires a DNS provider that provide a supported API. There is a list of supported DNS providers and some code to integrate with them at https://community.letsencrypt.org/t/dns-providers-who-easily-integrate-with-lets-encrypt-dns-validation/86438 .
Additionally, wildcard domains must be validated using the DNS-01 challenge type.
This means that you’ll need to modify DNS TXT records in order to demonstrate
control over a domain for the purpose of obtaining a wildcard certificate.

https://community.letsencrypt.org/t/acme-v2-and-wildcard-certificate-support-is-live/55579

Thank you for the tips, following it got some help.
https://medium.com/@saurabh6790/generate-wildcard-ssl-certificate-using-lets-encrypt-certbot-273e432794d7


Uncaught (in promise) undefined

Hi there!
I'm authenticating requests via cookies. If a user is not logged in, the server responds with this error message:

Now, of course I want to catch this error in my Vue-Component:
methods: {
    tryNow() {
      this.$apollo.addSmartQuery('user', {
        query: gql `
          {
              me {
                  username
              }
          }`,
        error(e) {
          console.log(e.message)
          return true;
        },
      })
    }
  }
}

But for some reason this error always shows up in my console, saying that there is an uncatched Promise:

I already had a look around here at related issues, especially #519 and #265. Sadly, I can't really pinpoint where this error even comes from, because Chrome and Firefox show me completely different stacktraces, not even pointing to the error. At least Chrome said the error is from somewhere inside vue-apollo.esm.js, so here I am now.
I created the project with the Vue-Cli and didn't change much of the original config, so I'm thinking this might be a bug?

I'm also seeing this issue, don't quite understand what is causing it

note: only seems to happen when using the apollo object in the component options:
This throw the 'uncaught (in promise) undefined' error:
  apollo: {
    userCurrent: {
      query: USER_CURRENT,
    },
  },

where as this does not:
  created() {
    this.$apollo
      .query({
        query: USER_CURRENT,
      })
      .then(result => {
        console.log('result=', result);
      });
  },

I'm having similar problem. Uncaught promise rejection or exception can be handled in vue global errorHandler, if error was thrown from lifecycle hook, but there's no way to create global handler for vue-apollo smart query. This is a little bit frustrating and adding some headache while handling errors.
One of solutions is simply not to use smart queries, but would be great to have some global hook for vue-apollo.

@georgyfarniev https://vue-apollo.netlify.com/api/apollo-provider.html#constructor

@Akryum I believe we're using the appropriate constructor, it seems any error thrown in the global error handler for errorHandler of the VueApollo will result in this uncaught promise, regardless of whether the error is captured or not.

@Akryum for example, here is my constructor, all the unauthorized error gets caught and handled appropriately, I always get this extra uncaught promise error:

  const apolloProvider = new VueApollo({
    defaultClient: apolloClient,
    defaultOptions: {
      $query: {
        fetchPolicy: 'cache-and-network',
      },
    },
    errorHandler(error) {
      if (isUnauthorizedError(error)) {
        if (router.currentRoute.name !== 'login') {
          router.replace({
            name: 'login',
            params: {
              wantedRoute: router.currentRoute.fullPath,
            },
          });
        }
      } else {
        console.log(
          '%cError',
          'background: red; color: white; padding: 2px 4px; border-radius: 3px; font-weight: bold;',
          error.message,
        );
      }
    },
  });

So I had a look at the code, and this promise seems to cause the error:
https://github.com/Akryum/vue-apollo/blob/131a8821d64f29ee100621e0f457f32c052ce6c3/src/smart-query.js#L27-L30
When my auth-error occurs, firstRunReject() first gets called in Line 175, where _firstRunReject is set to null, then firstRunReject() is called a second time in Line 140, which causes the eventual promise rejection.
https://github.com/Akryum/vue-apollo/blob/131a8821d64f29ee100621e0f457f32c052ce6c3/src/smart-query.js#L292-L297
For testing purposes I just replaced resolve and reject with () => {}, which fixed the error while the rest still worked fine.
I'm still not quite sure where this rejection error is coming from, but I also don't know what this promise is supposed to do in the first place. Git-blame suggests it's SSR related (d88e6ace17dba94f84b3111d41894da5e19a240e by @Akryum).

Any updates on this?

Possibly a similar problem here.
In the error() hook, not only is the e error object just a string, also the this context is undefined.

still seeing it in 3.0.0-beta.29, but still unclear with no insights, I'm working around my problems by skipping the offending queries until the loading conditions are valid but under normal circumstances I wouldn't expect that extra undefined error.

I think I got exactly the same trouble that the others have described.
Any of my error handlers would work fine, but additionally that weird Uncaught in (promise) undefined keeps throwing and I tried stepping through the code for a while: I didn't get a deeper understanding either though, but I can also observe what @JulianWels has described.
However, I managed to get half-way out of this trouble, but instead discovered another disappointment. When I'm putting the errorPolicy: 'all' right in my specific query's options (not one of the defaults, which would also impact other queries). The error doesn't occur anymore, but against my expectations I also don't see my error within my query's optional result(...args) callback either...

ya still no joy, curious if there is any official position on this issue

+1 This is a big issue in my Nuxt application. I'm trying to appropriately handle errors, but I can't chain a .catch() onto the $apollo.addSmartQuery and any "caught" error in the error (error) {} is "uncaught." Please advise.
this.$apollo.addSmartQuery(...).then is not a function

@firstaml-dima
Your first issue may be able to be solved by deconstructing the error: error ({ error }) {}.
Your second issue may be related to the scoping of the method. Try: error: ({ error }) => { console.log(this) }. That may preserve your context.

Thanks a lot @Akryum ✌

thanks much @Akryum

Thanks a lot for fixing it, @Akryum !


Restore reverted pull request #147

My suggestion to resolve the discussion that was started with #147 about if or how to use deprecation warnings would be to first add the changes I made with #147, and then find an agreement about deprecation procedure (discussion in #149).
Thanks for any opinions and reviews.

I agree with this sequence, it makes sense to me to handle it this way. Once there is agreement on #149 there can be a follow-up PR.

Closing in favour of #151.


verify in Session object ignored when REQUESTS_CA_BUNDLE is set

Summary.
When REQUESTS_CA_BUNDLE has been set as an environment variable, the verify property in a requests.Session() will always be ignored.
Expected Result
requests.Session always uses REQUESTS_CA_BUNDLE unless a certificate bundle has been assigned
Actual Result
requests.Session always uses REQUESTS_CA_BUNDLE and overides any certificate bundle set on the Session object.
Reproduction Steps
REQUESTS_CA_BUNDLE="/lib/python3.6/site-packages/pip/_vendor/certifi/cacert.pem"
import requests
s = requests.Session()
s.verify = "/non_existing_bundle"
s.get("https://google.com")

Expected result:
OSError: Could not find a suitable TLS CA certificate bundle, invalid path: /non_existing_bundle
Actual result:
<Response [200]>
System Information
$ python -m requests.help

{
  "chardet": {
    "version": "3.0.4"
  },
  "cryptography": {
    "version": ""
  },
  "idna": {
    "version": ""
  },
  "implementation": {
    "name": "CPython",
    "version": "3.6.4"
  },
  "platform": {
    "release": "3.10.0-693.11.6.el7.x86_64",
    "system": "Linux"
  },
  "pyOpenSSL": {
    "openssl_version": "",
    "version": null
  },
  "requests": {
    "version": "2.18.4"
  },
  "system_ssl": {
    "version": "100020bf"
  },
  "urllib3": {
    "version": "1.22"
  },
  "using_pyopenssl": false
}

Not sure if this is a bug, but I now need to unset the environment variable first before I can use a different certificate bundle in a session object or use the following workaround.
sessions.py:
            #if verify is True or verify is None:
            if verify is True or verify is None and not isinstance(self.verify, str):
                verify = (os.environ.get('REQUESTS_CA_BUNDLE') or
                          os.environ.get('CURL_CA_BUNDLE'))

In the future, please search closed and open issues before creating new ones that are duplicates.
Duplicate of #3829


RKE Cluster creation UI stuck with error Cannot read property 'constructor' of undefined

What kind of request is this:
Steps to reproduce:

Install  Rancher HA k8s master-head (09/14/2020) e919796c808d0aa643cb3491581bffcd64cfbe56
Try Deploy an RKE cluster (I tried DigitalOcean)

Result:

UI is stuck after clicking the Create button.
Error in Web Browser console

Uncaught TypeError: Cannot read property 'constructor' of undefined
    at n.isObject (cluster.js:881)
    at n.isEmptyObject (cluster.js:884)
    at n.save (cluster.js:678)
    at n [as save] (utils.js:365)
    at n.doSave [as _super] (new-or-edit.js:86)
    at n.doSave (component.js:330)
    at n [as doSave] (utils.js:365)
    at new-or-edit.js:61
    at w (rsvp.js:491)
    at rsvp.js:557

Other details that may be helpful:
Environment information

Rancher version: master-head (09/14/2020) e919796c808d0aa643cb3491581bffcd64cfbe56
Installation option: HA k8s

Rancher version master-head (09/14/2020) e919796c808d0aa643cb3491581bffcd64cfbe56
Seems fixed after refreshing the browser now. I was able to deploy clusters besides EKS.


Fix deprecated warnings in test

This PR fixes the deprecated warnings about the using of trim_left and trim_right in the test code.

If this crate needs to support gnu-1.6.0, feel free to close this PR then.


Probable conflict with another package

OS: OpenSuse Tumbleweed
DE: KDE Plasma
Install method: rpm package
Description:
After installing, software update is initiated with error. Probable conflict of similar named package https://software.opensuse.org/package/terminus

Have the Same issue under Ubunut 20.10.

Have the Same issue under Ubunut 20.10.

same with Ubuntu 21.04.

The package has been renamed in the nightly builds.


Replace updated with timeago

What about putting a timeago filter on this?  There's a nice stimulus component that's multi-lingual.

https://www.stimulus-components.com/docs/stimulus-timeago
You could put a title attribute with the actual date/time (as it is) so that hovering over it would show the exact time.  For me, a relative time is almost always preferable.

Looks nice!!


Add requests-async dependency

Description:
Fix missing dependency for enphase_envoy component.
Related issue (if applicable): fixes #25224, #25331
Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

Hi @zenman94,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


Added C code for issue #115

Please use pull request template to open new issue and do provide issue number and language you used and all in that template.

I didn't get it like how?


Remove old Envoy runtime flag

Please provide a description of this PR:
This flag was removed in Envoy 1.22 as described in the changelog:

http: removed envoy.reloadable_features.require_strict_1xx_and_204_response_headers and envoy.reloadable_features.send_strict_1xx_and_204_response_headers and legacy code paths.

https://www.envoyproxy.io/docs/envoy/latest/version_history/v1.22/v1.22.0.html#removed-config-or-runtime

/ok-to-test

/retest

/cherrypick release-1.15

@GregHanson: new pull request created: #40465

In response to this:

/cherrypick release-1.15

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


RFC: Add routeLinkAttr

Motivation: routeLink as it is makes it impossible to say add a class attribute (eg: for semantic-ui's menu tab) to the created a element .
The solution this PR adapts is to extend the function to take in a custom attr parameter to set whatever element attributes we want; just as elAttr exists alongside el.

Open question: what if I want to pass in a dynamic of attributes (a la elDynAttr)?

Closing, because #547 looks better and up to date.


Can't go into the luci web manage page when device runs a long time , For me the time is 42days

Below shows the error message.
/usr/lib/lua/luci/dispatcher.lua:562: Failed to execute function dispatcher target for entry '/'.
The called action terminated with an exception:
/usr/lib/lua/luci/dispatcher.lua:562: Failed to execute firstchild dispatcher target for entry '/admin'.
The called action terminated with an exception:
/usr/lib/lua/luci/dispatcher.lua:562: Failed to execute function dispatcher target for entry '/admin/status'.
The called action terminated with an exception:
/usr/lib/lua/luci/dispatcher.lua:562: Failed to execute template dispatcher target for entry '/admin/status/overview'.
The called action terminated with an exception:
/usr/lib/lua/luci/template.lua:97: Failed to load template 'admin_status/index'.
Error while parsing template '/usr/lib/lua/luci/view/admin_status/index.htm':
No such file or directory
stack traceback:
[C]: in function 'assert'
/usr/lib/lua/luci/dispatcher.lua:562: in function 'dispatch'
/usr/lib/lua/luci/dispatcher.lua:306: in function </usr/lib/lua/luci/dispatcher.lua:305>
I use Telnet to go into the file system and see index.htm is only in /overlay/usr/lib/lua/luci/view/admin_status/index.htm
/rom/usr/lib/lua/luci/view/admin_status/index.htm
and they are abnormal
treebear@Witown:/overlay/usr/lib/lua/luci/view/admin_status# ls -l
lrwxrwxrwx    1 treebear root            18 Feb 25 10:33 index.htm -> (overlay-whiteout)
treebear@Witown:/overlay/usr/lib/lua/luci/view/admin_status# ls -l
lrwxrwxrwx    1 treebear root            18 Feb 25 10:33 index.htm -> (overlay-whiteout)
Then I look at Top information, it goes well because cpu and memory has lots of idle resource:
Mem: 134908K used, 121092K free, 0K shrd, 6116K buff, 45960K cached
CPU:   0% usr   2% sys   0% nic  97% idle   0% io   0% irq   0% sirq
Load average: 0.04 0.12 0.13 2/80 17638
PID  PPID USER     STAT   VSZ %VSZ %CPU COMMAND
17638 14983 treebear R     1512   1%   2% top
24433  1882 treebear S    57108  22%   0% /usr/bin/ppcloud
1465     1 treebear S    10060   4%   0% /usr/sbin/dnsmasq -C /var/etc/dnsmasq
1827     1 treebear S     4224   2%   0% /usr/sbin/lighttpd -f /etc/lighttpd/l
1201     1 treebear S     1544   1%   0% /sbin/netifd
1924     1 treebear S     1528   1%   0% /sbin/syslogd -l 7 -O /etc/log/system
3636  1831 treebear S     1520   1%   0% {update_cur_stat} /bin/sh /usr/sbin/u
14983  1759 treebear S     1520   1%   0% -ash
1759     1 treebear S     1516   1%   0% /usr/sbin/telnetd
1804     1 treebear S     1512   1%   0% /usr/sbin/ntpd -n -u 86400 -r 600 -p
471     1 treebear S     1512   1%   0% /bin/ash --login
1882     1 treebear S     1512   1%   0% {witownapp.sh} /bin/sh /usr/sbin/wito
1927     1 treebear S     1508   1%   0% /usr/sbin/crond start
1348  1201 treebear S     1508   1%   0% udhcpc -R -p /var/run/udhcpc-eth0.5.p
1925     1 treebear S     1504   1%   0% /sbin/klogd
1831     1 treebear S     1504   1%   0% {runing_status_m} /bin/sh /usr/sbin/r
17637  3636 treebear S     1500   1%   0% sleep 10
1     0 treebear S     1360   1%   0% /sbin/procd
1782     1 treebear S     1084   0%   0% /sbin/mountd -f
923     1 treebear S      980   0%   0% arp_broadcast lan
I try to use rm -rf /tmp/luci-* but it seems useless
Does this issue comes from openwrt filesystem or it is luci's bug ?
System info:
Linux  3.10.34  mips
lua-5.1.5
openwrt CC

Maybe it is a bug, but this version of OpenWrt is 5 years old and not supported. You should try the latest version and see if this bug still exist.

But our customized system is based on openwrt CC, this will cost too much to change to another version. @ysc3839 . Thx for your reply.

I would also suggest going to a new stable openwrt stable.
I remember having the same problem with 18.06.
a /etc/init.d/uhttpd restart fixed the problem on my system.

You said you have the same problem with 18.06. So it means that the newest openwrt version can't work well then probably no help to me if I go to a new stable openwrt.
I tried /etc/init.d/lighttpd restart and failed.   Thanks for your reply ! @feckert

Not a LuCI issue. Something deleted essential files on your system.


Feature request: Add caching headers for mainfest.json

I noticed that https://github.com/silviolleite/laravel-pwa/blob/master/Http/Controllers/LaravelPWAController.php does not add Cache-Control: and Expires: headers to the response for manifest.json. It seems like this causes Chrome to re-download manifest.json with every navigation.
Would it be possible for the route registrar to add the corresponding caching middleware with a configurable delay, e.g. one hour?

Is it an option to control this using the HTTP Server? I like to control everything there.


 Can you check? I still lazyload is the issue, as the grid-layout is already implemented by left-floating divs. See #35

Duplicate of #35

Psst: have you already starred this repo? All you need to do is to press the Star ★ button on the top right corner of the website. ;)


Categoryless Training/Testing

Hi,

Would like to ask if this could work with a data set that has no defined category (example vases could be mixed in with mugs in the same dataset)
Are the number of segmented parts fixed? Or can there be a unlimited number of segments?
Can it be trained to segment finer details? Example i have a wooden plank, it will be segmented into 6 different surfaces. And if the wooden plank had holes drilled, the segments would increase by 1 for each hole. (i think this term is called surface segmentation? if it exists)

Regards,
Kessho

Would like to ask if this could work with a data set that has no defined category (example vases could be mixed in with mugs in the same dataset)

This code provides a means for doing supervised semantic segmentation. So basically, you can use any dataset which has meshes and per edge segmentation labels for training the network. I am not sure if this is what you are getting at: but say you take the set of "aliens" (4 possible segments) and "chairs" (3 possible segments) from this repo (coseg segmentation). I provided code to train each one as a separate network. But you could train one network which could recieve chairs or aliens and predict a label from 1-7. The first 4 being  aliens , and the last 3 being chairs.

Are the number of segmented parts fixed? Or can there be a unlimited number of segments?

You can train any number of segments you like (assume you have enough data , network capacity and GPU power). The number of segments is defined by the number of classes in the 2d-cross entropy loss.

Can it be trained to segment finer details? Example i have a wooden plank, it will be segmented into 6 different surfaces. And if the wooden plank had holes drilled, the segments would increase by 1 for each hole. (i think this term is called surface segmentation? if it exists)

Not sure I completely understood, but same thing applies as above. With enough data labels you should be able to learn to do a fine grained segmentation.
Also, make sure to check out the segmentation wiki.

Hi,
allow me to rephrase and clarify some of my questions:
Yes i can combine your set of "aliens" and "chairs" but would it still work if i do not explicitly categorize them as "aliens" and "chairs" when training/testing?. So for example when predicting an unknown (but is actually an alien) object with legs, would the legs be then be segmented as a chair's legs? what about not having a fixed set of parts? https://www.youtube.com/watch?v=6JQdNzsw7jA perhaps the first 20seconds of the video helps to bring across what i'm trying to say

Yes i can combine your set of "aliens" and "chairs" but would it still work if i do not explicitly categorize them as "aliens" and "chairs" when training/testing?.So for example when predicting an unknown (but is actually an alien) object with legs, would the legs be then be segmented as a chair's legs? what about not having a fixed set of parts? https://www.youtube.com/watch?v=6JQdNzsw7jA perhaps the first 20seconds of the video helps to bring across what i'm trying to say

So as for explicitly knowing whether it's a leg that came from an alien vs. chair, it depends how the ground truth-label is built. For example, if we said that 1-3 are the segments for the chair and then 4-7 are the alien segments, so the category of the object is implied by the segmentation labels. Since predicting segment 1-3 must mean that the object is from the class chair. However, this constraint is of course not required in test time, since the network will just predict the segment.
About grouping them together. You would have to train the network to possibly predict a max number of segments (say 10). but that doesn't mean that all the segments must be present in each mesh model. The network will predict a probability the edge belongs to a particular segmentation ID, so that can be any value (for example, for one particular mesh it could predict most of the edges segment 9 and the rest 8). There is no constraint that all segments should be present in each mesh, only that each edge should predict a probability for all segments. For segments that are not present, this probability should be close to 0.

in addition, for your 'aliens' data set you have labeled them 1-4 corresponding to their different parts, e.g. head, body, tail etc. am i able to train the model in such a way that it doesn't have this constraint? i can just say that this section of the alien is a part and that section is another part, and the model is able to segment the object into its different parts

It is technically possible, but I am not sure the meaning of such a thing. I mean, what exactly determines which part corresponds across different shapes? In the case of very similar shapes which have similar semantic parts, I think this could make sense (e.g., different legged animals, humans, etc.). Otherwise I am not sure.
All in all it seems you are looking for something unsupervised, or at least, weakly supervised. Which this code as is doesn't support. But you are welcome to take the layers we provided and build an unsupervised segmentation approach :)

I see, i kind of understand what you are trying to say.
So it would be close to impossible to label the alien model parts with 1-3 and then the chair parts with 1-4 and then throw the model a object and say hey, segment it into different parts. The part labeled '1' would have to at least have similarities across the training sets instead of it being just a generic part then.
perhaps the solution i am looking for lies in the unsupervised training. I'll check that out as well then.
Thanks for your kind reply!

I see, i kind of understand what you are trying to say.
So it would be close to impossible to label the alien model parts with 1-3 and then the chair parts with 1-4 and then throw the model a object and say hey, segment it into different parts. The part labeled '1' would have to at least have similarities across the training sets instead of it being just a generic part then.

Exactly. Another option is to use MeshCNN as a binary classifier, to predict the boundaries between segments. This should be transferable across different shapes.

Oh. 1 more question. After creating your models in blender, how did you annotate them?

Did you check out the scripts in the Segmentation Wiki?

Exactly. Another option is to use MeshCNN as a binary classifier, to predict the boundaries between segments. This should be transferable across different shapes.

That sounds like a plausible way to segment the parts that i'm looking for. Any advice on how to proceed?

I think you can try to work with the data I have first, to see if it will work first on a single shape class (i.e., just aliens). And then you can extend it to see if it will work on aliens and humans, for example. You should search for all boundary edges (edges which have different segmentation IDs in their 1-ring, called the gemm data-structure in the code).


Model to JSON

I understand that this might be out of scope for Argo, but have you guys considered adding the ability of going from a Model object to JSON? Even if it's not part of this lib, maybe as a separated library?

@RuiAAPeres Have you seen Argo's dual Orga? :)

Ah I missed it! Thanks @wouterw


Added env var to disable api for services

What are you changing?

[ ] Releasing a new version of Notify
[X] Changing kubernetes configuration

Provide some background on the changes
Following this PR which disables access to API related pages in the admin component, we need to add an environment variable to control which services will be restricted. Changes in this PR prepare the way to test for the staging environment.
If you are releasing a new version of notify, what components are you updating

[ ] API
[ ] Admin
[ ] Documentation
[ ] Document API
[ ] Document UI

Checklist if releasing new version:

[ ] I made sure that both API and Admin changes are present in Notify
[ ] I have checked if the docker images I am referencing exist - ex: https://gallery.ecr.aws/v6b8u5o6/notify-admin

Checklist if making changes to Kubernetes:

[ ] I know how to get kubectl credentials in case it catches on fire

 Modify a single RCMP service to not allow them to create API keys for any reason


small tweaks/clean up

constant time insertion, update, and access

Hey thanks for this! I'll merge this right now.


fix: remove e2e tests for disabled SAST tasks

Remove the sast-go and sast-java-sec-check tests

Signed-off-by: dirgim<EMAIL_ADDRESS>Description
This change removes the sast-go and sast-java-sec-check e2e tests because they've been disabled in the HACBS build pipeline.
Issue ticket number and link
HACBS-1072
Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[X] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("[test_id:01] DEVHAS-62 devfile source")
[ ] I have updated labels (if needed)

Hi @dirgim,
the CI configuration in main branch of e2e-tests repository was recently updated and it no longer uses pre-kcp branch of infra-deployments for bootstrapping appstudio on OCP cluster. Now it uses main branch, which leverages kcp.
Since appstudio/hacbs services are not fully integrated with kcp yet, lots of tests implemented in this repo can't be run successfully against it.
To address this, a branch "pre-kcp" was created in e2e-tests repo and can be used for adding/updating tests that can be run against "pre-kcp" version of appstudio.
Once we are able to run e2e tests for all appstudio/hacbs services successfully, the plan is to merge back all changes done in pre-kcp branch back to main.
If your PR is intended to update test used for "pre-kcp" version of appstudio, please update the base branch in your PR like this:

Thanks!

@psturc thanks for the tip, I updated the base branch to pre-kcp


Fix for create-diff not including transactions

Create_diff function, as part of block production, tries to add as many transactions given the scan state size. In cases where transactions in a block span two scan state trees (when previous blocks have less than max transactions), create-diff  tries to add transaction to the first tree and the remaining transactions + coinbase to the second tree. It handle cases when it is unable to include transactions, prioritizing inclusion of coinbase over user payments.
In one of the cases where first tree has only 1 or 2 slots, user transactions were not added included at all. This PR fixes that by matching on those cases and attempting to add user transactions.
This was causing a flake in zkapp integration test.
Existing tests-Payments and zkapp integration tests, staged ledger unit tests should detect any regression
Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me


DBZ-763 | Correcting the JDBC driver class

https://issues.jboss.org/browse/DBZ-763

@jpechane Can you verify that this gets rid of the warnings?

All good, the warning doesn't show up any longer with this change. Rebased and applied. Thanks, @vamossagar12!


Update with 2022 data

Data now point to the R:/ server and the newest version of the data from naturbase.
Some column names and field values were changed since the last version, and the code is updated accordingly.

This PR is not so critical to review, so I will just force it though. I also need the new data set to continous working on other chapters,

ok, i'll delete the branch now


docs: add neptune docs

Summary of Changes
Add docs, add sample terraform file.
All credit to @dsimms for the documentation and terraform file. Thanks Dan!!
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.

@friendtocephalopods , @cpu  thanks for working on this.  We are currently investigating to migrate our backend from Neo4j to AWS Neptune for Amundsen.  Is the AWS Neptune ready to use in production since it seems didn't announce in publicly release notes?  Thanks!

:wave: Hi @Samlinxia I think there is still work required to be done to get the databuilder to support Neptune. See this RFC. In the #gremlin channel of the Amundsen slack the RFC author mentioned they hoped to have a PR open in early Dec but I can't speak personally to where that's at. Hope that helps!


Feat:Want to Fix Nav Bar

Describe the feature
Nav Bar is Not Good there is repeated icons
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Ok fix that

I Want To Work on this issue . Could you please Assign me this task? @VanshKing30


Populate statusCode and statusMessage on listener response documents

Currently this is done for Adapter responses, but not for listeners.  Incoming requests should retain the status codes of responses we sent back.  The columns are DOCUMENT.STATUS_CODE and DOCUMENT.STATUS_MESSAGE. The values are protocol specific, and initially we'll only support HTTP codes and messages.

This was completed and included in 6.0.02.   Currently, the status code is set in metaInfo Map (key=HttpStatusCode / Listener.METAINFO_HTTP_STATUS_CODE) by the service, or present in Exception object (metaInfo entry overrides exception code).  Otherwise, defaults to 200 for all methods, which matches what Tomcat will respond with unless specified in metaInfo Map.    If a different success code (i.e. 201, 202, etc.) is desired, the service itself will have to set that in metaInfo Map based on actual type of operation performed and outcome from that operation (i.e. new record was created, existing record modified, etc.).


[core] Make java.groupId settable using env vars or sys props

My use case is a bit different: I have a quarkiverse-release repository with a jreleaser.yml file used to push to Maven Central. Given that any project in the same org can invoke this workflow, is it possible to set the project groupId dynamically?

Not at the moment but could be added as a feature. All secret values may be set using env vars or sys props. Only a few non secret values may be set in the same way. java.groupId is not one of them as of the latest release.
Originally posted by @aalmiray in https://github.com/jreleaser/jreleaser/discussions/1708#discussioncomment-10479961

🎉 This issue has been resolved in v1.14.0 (Release Notes)


Binder fails to launch

The link in the README fails to launch with the error...
ERROR: Could not find a version that satisfies the requirement PIL (from versions: none)
ERROR: No matching distribution found for PIL

I will dig in to this in the coming days.

Probably just need to kill this line:
https://github.com/mne-tools/mne-binder/blob/master/Dockerfile#L43
I don't see a reason to try installing both PIL and pillow, they are designed to provide the same functionality, but pillow is the newer/preferred one


Invalid Gopkg.toml file

I wanted to give you a heads up that your Gopkg.toml file is invalid and causes problems for people trying to depend on your library.
Entries like this
https://github.com/docker/compose-on-kubernetes/blob/3fcc42fdbcf6e37cc1da2363369846a74a7ef0dd/Gopkg.toml#L25-L27
should be
[[constraint]]
  name = "k8s.io/apiextensions-apiserver"
  version = "kubernetes-1.11.5"

revision in the Gopkg.toml means a specific git commit hash. version indicates a tag.
Otherwise consumers of your library run into this error when running dep ensure when another package requires one of these projects and uses version correctly
Solving failure: Could not introduce<EMAIL_ADDRESS>as it has a dependency on k8s.io/apimachinery with constraint kubernetes-1.11.5, which has no overlap with existing constraint kubernetes-1.11.5 from (root)

Thanks for the report @carolynvs, I'll take a look!


Update HTTP-Proxy to 1.18.1 to resolve https://github.com/advisories/GHSA-6x33-pw7p-hmpq

http-proxy has been updated to 1.18.1 to resolve DoS issue:   https://www.npmjs.com/package/http-proxy
This dependency will need to be updated for cors-anywhere and pushed out to npm.
npm audit report
http-proxy  <1.18.1
Severity: high
Denial of Service in http-proxy - https://github.com/advisories/GHSA-6x33-pw7p-hmpq
No fix available
node_modules/http-proxy
cors-anywhere  *
Depends on vulnerable versions of http-proxy
node_modules/cors-anywhere

This is a duplicate


[WIP][SPARK-31168][BUILD] Upgrade Scala to 2.12.12

What changes were proposed in this pull request?
This PR(SPARK-31168) upgrade Scala from 2.12.10 to 2.12.12.
Scala 2.12.11 release notes:
Highlights

Performance improvements in the collections library
Performance improvements in the compiler
Improvements to -Yrepl-class-based, an alternative internal REPL encoding that avoids deadlocks
A new -Yrepl-use-magic-imports flag that avoids deep class nesting in the REPL, which can lead to deteriorating performance in long sessions
Fix some toX methods that could expose the underlying mutability of a ListBuffer-generated collection

JDK 9+ support

ASM was upgraded to 7.3.1, allowing the optimizer to run on JDK 13+
:javap in the REPL now works on JDK 9+

Scala 2.12.12 release notes:
Highlights
This release adds compiler support for scala-async and backports substantial performance improvements to collections.

The compiler now includes a phase that performs the transformation for the async/await DSL for working with Scala Futures.
This transform is based on the macro implementation in scala-async but has been substantially rewritten to fix some
long standing bugs, reduce the overhead of compilation and simplify the extension points for third party effect systems.
scala-async 1.0.0 will be released shortly building atop this phase.
immutable.{TreeSet, TreeMap} are now based on a backport of the 2.13.x implementation.
The builders for immutable.{HashMap, HashSet} are more efficient -- they now use mutation during building to reduce memory churn.

Further highlights

Minor improvements to compiler performance. The allocation rate of the compiler is reduced by 10%.

Why are the changes needed?
Improvement performance.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Exist test.

Thanks, @wangyum .
cc @srowen

I feel like there was some reason we couldn't update to 2.12.11; not sure if it's still an issue or not. Tests will help tell. Yes, generally fine otherwise to update.

Test build #125879 has started for PR 29124 at commit 3adc82a.

Merged build finished. Test PASSed.

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

The failure looks consistent. Could you take a look at that, @wangyum ?
[info] org.apache.spark.serializer.KryoSerializerSuite *** ABORTED *** (324 milliseconds)
[info]   java.lang.NoSuchFieldError: numNonEmptyBlocks
[info]   at org.apache.spark.scheduler.HighlyCompressedMapStatus.<init>(MapStatus.scala:174)

Test build #125879 has finished for PR 29124 at commit 3adc82a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Just a note: both -Yrepl-class-based and -Yrepl-use-magic-imports are very nice improvements to the Scala REPL, and I'd love to see them in Spark. But they can also potentially significantly impact how Spark's Scala REPL interact, so I wonder whether or not we need some more tests cases to show that the REPL behavior is consistent / has not regressed with regards to them. The test coverage in Spark's RPEL is quite weak...

Hi, @wangyum . Could you resolve the conflicts? Or, do you think we need to skip 2.12.12 like 2.12.11? If then, please update JIRA.

Given the current status, do we need to skip Scala 2.12.12 like 2.12.11, too?

Yes. We should skip Scala 2.12.12. At least this bug should be fixed first: https://github.com/scala/bug/issues/12096

Thank you so much for the investigation, @wangyum !


systemd fails with dune exec

When running dune exec src/fugit.exe, it fails with
Failed to find executable src/fugit.exe: No such file or directory
This is due to Bos.OS.exec, which is Sys.argv.(0), which is src/fugit.exe.
Running with dune is perhaps not the normal use case, but could this fail in other scenarios? Would Sys.executable_name be a better alternative?

Yeah, this is all quite annoying. I'm not sure how to retrieve the actual path to the binary currently executed. Note that calling through dune is particularly ill-advised here, since the executable would disappear after a clean.
Maybe it's a good argument for just calling notify-send instead of fugit ding in the systemd action, but that has several other disadvantages.

My understanding is that Sys.executable_name is precisely the path to the executing binary. Some quick tests shows it gives the right result also with dune.


Shell improvement

@DivineDominion
This is just a pre pull request for #212 to keep #212 reduced to the necessary changes

Sweet! 👍


Support for nocking DNS queries

Support for nocking DNS queries would be swell, though I understand if that's out of scope.

Just out of curiosity: Does anybody actually know an actual module for this?

actually from a user perspective, i dont care if it's dns or http. Just want to nockback and run my tests without requiring other parts of my infrastructure/third party services etc.


[WIP]: feat(platform): e2e https for v1.x + k8s scheduler

Fixes #4611 and #4629, but only with k8s as the scheduler.
This is currently lacking documentation, but I wanted to start getting this going with CI-- hence the WIP.
If anyone wants to try this, enabling requires two things:

You must have provided a wildcard cert for the platform.  See here.  This cert is re-used for the link between the routers and the application pods if...

You must opt into this on an app-by-app basis.  Because extensive modifications to the router are not in scope now (but may be for v2), the only reasonable way to accomplish this is by keeping all routing at layer 7 using SNI.  This requires terminating SSL at the router and re-initiating it for the upstream connection to the application.  This imposes a minor performance overhead that not all operators will want-- thus this remains opt in.  Opt in like so:

[/my/app/directory]$ deis config:set E2E_HTTPS=true

Ping @mboersma ... note that this introduces a new "component."  It's a Docker image used for SSL-terminating "sidecar" containers in each pod (if e2e HTTPS is enabled, that is).  It's not anything that needs to be scheduled or anything-- controller just adds the sidecar where applicable.
I'm mentioning this in case this might have any bearing on the release process, although I think I've got that covered in this PR... I'm not positive.

Closing this.  This won't land in V1.x.  The exercise will inform the e2e implementation in V2.


Feedback about an address that needs to be activated multiple times to use the transfer and mint nft functions

I often use the zksync network for transfers and mint nft, but I found that in the past few months, the system will require me to activate my account multiple times to use the transfer function. This is different from the advertised account that only needs to be activated once.
I hope to check out what caused this error
My address is 0x6eCe332E1324FB0C080a6364e84171EEc1571f5f
thank you very much

Hi @age-wang!We are sorry for the inconvenience :( Is the issue on https://wallet.zksync.io/?

嗨@age-wang！对于给您带来的不便，我们深表意意:( https://wallet.zksync.io/上有问题吗？

Yes, I used the metamask wallet to log in at https://wallet.zksync.io/.

@age-wang There was a similar issue connected to zknft.xyz. Maybe you used this tool?
Please note that the wallet most likely asks you only for authorization of the transfer and not for an unlock

@age-wang There was a similar issue connected to zknft.xyz. Maybe you used this tool?
Please note that the wallet most likely asks you only for authorization of the transfer and not for an unlock

Yes, I have used zknft.xyz,       but you can check the record of my address, which shows that it is not authorization but Account activation
My address is 0x6eCe332E1324FB0C080a6364e84171EEc1571f5f

@age-wang It is a known old issue. We are sorry for the problems. The Account activation was removed from zknft to avoid such errors in the future

@age-wang It is a known old issue. We are sorry for the problems. The Account activation was removed from zknft to avoid such errors in the future

It’s good to solve it, because the feedback has not been confirmed before, so I propose it again on GitHub. Thank you very much for your patience.


Can I add Movie Time Skill

I would like to add Movie Time Skill, Can I proceed?

@garimasingh128 : Please have a look at latest PR for this skill.

@garimasingh128 : Please have a look at latest PR for this skill.


Warning on number of max-len options - but I can't find such a restriction documented in eslint

I have a eslintrc.yml file like so:-
env:
  browser: true
  node: false
parserOptions:
  ecmaVersion: 6
  sourceType: "module"
rules:
  max-len:
    - error
    - 120
    - ignoreComments: true
    - ignoreTrailingComments: true
    - ignoreTemplateLiterals: true

globals:
  Polymer: false
  PAS: false
  Pikaday: false
  PasMonthBase: false
  import: false

I would really all these max-len options - but this plugin tells me I am only allowed 3.  I can't find any restriction in the documentation of eslint for this rule and I can't see if there is a way for this plugin to increase the number.

@akc42 what happens if you run eslint directly in a terminal. Will it complain as well. This extension uses the ESLint npm module for reading eslintrc.yml files.


Add supports_nulls_first_in_sort to Dialect

The MySQL dialect doesn't support parsing the NULLS FIRST/LAST clause in ORDER BY expressions. This PR adds a supports_nulls_first_in_sort function to the Dialect trait (which is also a requirement in DataFusion for unparsing using the MySQL dialect, see https://github.com/apache/datafusion/pull/10625). Using this, we now validate that SQL statements that were previously using NULLS FIRST/LAST will fail properly when parsed with the MySQL syntax.
This is the error I get from MySQL when trying to use NULLS FIRST in a query:
Query 1 ERROR at Line 3: : You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'nulls first limit 10' at line 3

Thanks @phillipleblanc  -- looks good to me
One thing I thought of while looking at this PR  was that it is on the borderline between syntax and semantics as explained in
https://github.com/sqlparser-rs/sqlparser-rs?tab=readme-ov-file#syntax-vs-semantics
However, since the syntax is rejected by MySQL perhaps it is ok to not parse the syntax in this crate too (though this crate is definitely not consistent in this regard)
🤔

One thing I thought of while looking at this PR was that it is on the borderline between syntax and semantics as explained in
https://github.com/sqlparser-rs/sqlparser-rs?tab=readme-ov-file#syntax-vs-semantics

That is a good callout - I did not see that previously.

However, since the syntax is rejected by MySQL perhaps it is ok to not parse the syntax in this crate too (though this crate is definitely not consistent in this regard)

I do think we should allow parsing of it then, as this is more of a semantic issue (since NULLS FIRST is really part of the SQL standard - MySQL why are you different! shakes fist)
I will close this PR then.