[Bug report] Uncaught TypeError: Cannot read property '2' of undefined

Date Seen:
01-05-2015
Versions:

KC3 Kai v004 (git build)
Chromium 64bit (currently on: 42.0.2311.135 (Developer Build 575981d43c6dc74b1443ca77e015c218daf050dd Linux))
Archlinux 64bit (currently on: Linux: 4.0.1-1-ARCH)

Bug Description:
Error found approximately during the loading of the main page after user clicks "Game start!". Issue is not replicable as the current build of KC3 Kai (v005 now) and newer version of Chromium build. Also because there was no way to clear the amount of accumulated errors it is uncertain as to whether or not has the bug resurfaced or has it been fixed.
Screenshots are provided below where possible as this issue has not been seen again. Also details regarding to stack trace could not be sourced.
Severity:
Unknown - possibly minor.
Steps to Reproduce:

KC3 Kai button left click and click on Play via DMM Frame.
Click on start the game anyway.
Whilst loading Ctrl + Alt + J key was used (as F12 was reserved for another program).
Under developer tools click on KC3 Kai tab.
Game displays "Start game!" wording.
Enter into user's personal page and click on fleet (inside game - not KC3 Kai).
Go into Settings under Chromium.
Click on Extensions.
Error appears as titled in red words.

Actual Behavior:
Unknown. I do not know what is actually happened other than seeing the error.
Expected Behavior:
Unknown - possibly error to not show up.
Troubleshooting/Testing Steps Attempted:

Tried to play the game normally which appears to have impacted nothing. Other than that no new error messages have since been re-appearing.
Updated to the latest version of KC3 Kai via 'git pull' and the errors are no longer visible.

Workaround:
None.
Attachments:

ah this could be because the F12 was loaded late. the correct step should be Ctrl + Shift + J and not click on "Start Game anyway". There is a specific API call during game start that loads all your equipment, if the F12 panel was late to be opened and did not get the equipment list, it will generate error when computing for eLoS

Thanks for the response! I am starting the KC3 Kai before I click on Game start and it appears to not show up any more errors.


Mouse.position change not reflected

Hi,
Finally found some time to play with your webgl library and it is great.
I found an issue, not sure if it is a webgl-elm or elm issue but with the code below I got the following behavior
http://ibin.co/1qp7HBTcTGBJ
where the red arrow indicate the mouse pointer position.
This happen when I move the mouse pointer suddenly.
If I move the pointer again it still reset to the correct behaviour:
the screen is filled from the top left corner (As it should)
It seems like there is an issue with webgl not being updated properly after the signal from the Mouse position
import Mouse
import Window
import Signal (map, map2)
import Text (asText)
import WebGL as GL
import Math.Vector3 (..)
import Math.Vector2 (..)
import Math.Matrix4 as Mat4
import Graphics.Element (..)

main = map view Mouse.position

type alias Vertex = { position : Vec2, color : Vec3 }

mesh : List (GL.Triangle Vertex)
mesh =
    [ ( Vertex (vec2 0  0 ) (vec3 1 0 0)
      , Vertex (vec2 1  1 ) (vec3 0 1 0)
      , Vertex (vec2 1 0 ) (vec3 0 0 1)
      ),
      ( Vertex (vec2 0  0 ) (vec3 1 0 0)
      , Vertex (vec2 0  1 ) (vec3 0 0 1)
      , Vertex (vec2 1 1 ) (vec3 0 1 0)
      )
    ]

ortho2D w h = Mat4.makeOrtho2D 0 w h 0

view : (Int,Int)  -> Element
view (w,h) =
  let matrix = ortho2D 1 1
  in GL.webgl (w,h)
    [ GL.entity vertexShader fragmentShader mesh { mat = matrix } ]

-- Shaders

vertexShader : GL.Shader { attr | position:Vec2, color:Vec3 } { unif | mat:Mat4.Mat4 } { vcolor:Vec3 }
vertexShader = [glsl|

attribute vec2 position;
attribute vec3 color;
uniform mat4 mat;
varying vec3 vcolor;

void main () {
    gl_Position = mat * vec4(position, 0.0, 1.0);
    vcolor = color;
}

|]

fragmentShader : GL.Shader {} u { vcolor:Vec3 }
fragmentShader = [glsl|

precision mediump float;
varying vec3 vcolor;

void main () {
    gl_FragColor = vec4(vcolor, 1.0);
}

|]

I wasn't able to reproduce after updating the code to 0.15.1 - what's the behavior specifically?
There's definitely some flickering when you resize the context, because that's going to create lots of new elements (I think, maybe not, at the very least, it's doing lots of DOM manipulation)


UPDATE 4

Hello! Is there any chance to have this MOD up to date in UPDATE 4?

Hello! Is there any chance to have this MOD up to date in UPDATE 4?

Hello! Is there any chance to have this MOD up to date in UPDATE 4?

20 days with no response. So the mod is abandoned?

i think so.


Remove dynamic memory allocation from ota lib

This is just a not high priority feature enhancement request;
It would be very nice in environments where dynamic memory allocation is forbidden (coming from https://github.com/aws/ota-for-aws-iot-embedded-sdk/issues/412#issue-1100132314 )
There seem to be not much places that would need to change, one is in ota_os_freertos.c for the timer creation, and then there are only thre os.mem.malloc's in ota.c hat also look to have only limtied used and seem like they could be done static as well without wasting too much RAM - this could also be just a transparent define/compile option for the ota_os_freertos port.

Hi @lightblu
This seems like a great suggestion. I'll inspect the code to verify feasibility and see what changes are needed.

Hello @lightblu ,
The OTA library can be used without any dynamic allocation and os.mem.alloc is provided optional. You can set the buffers while initializing the OTA agent.
The ota_os_freertos.c is reference implementation for freeRTOS, not part of OTA library and you can modify it as per your requirements for example timer creation.

Closing this topic now, please reopen for same topic or create new one for others.


changing confirm, prompt of simulator to dialogs

Fixes #
Describe the changes you have made in this PR -
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 84.379% when pulling ad504f49780ff12567b33b2949cabf9711588433 on abstrekt:mod into 4fca4d3b5bf5d906850c1ebe07145afe99c37bd5 on CircuitVerse:new_UI.

Can you please also attach screenshots for the same? Thanks.

@nitin10s

1. After clicking Cancel the first time and then again opening the box - This appears:
   ![image](https://user-images.githubusercontent.com/16988558/89557255-293b3f80-d830-11ea-9fef-d14e542f7e1a.png)

fixed

2. "New Project" doesn't work. Clicking on the option reloads the window.

I looked into it, that's one trouble with using modals instead of confirms, the new project function runs clear project first, previously we had confirm prompts in clear project func which used to pause the new project function until users gave a confirm, now the clear project is a modal and modals don't have that property.. what should be done?

@tachyons, I would like to work on this ...Can I start working??

@Prerna-0202 yes

@tachyons, There is a problem while changing the save online prompt to the dialog, previously we had a prompt that used to pause the back function until users gave an input, now it is a modal dialog and modals don't have that property to wait until user gave the input... Can you please help me out?
https://user-images.githubusercontent.com/89515816/148682186-6ad2b324-28c7-4399-91e9-7d767a2df8b7.mov

@tachyons, There is a problem while changing the save online prompt to the dialog, previously we had a prompt that used to pause the back function until users gave an input, now it is a modal dialog and modals don't have that property to wait until user gave the input... Can you please help me out?
screen-capture.1.mov

I think @Prerna-0202 the post method is being called when save online button clicked, we have to change the scope to confirm button in popup.

@devartstar FYI, alerts and prompts should be replaced like this

@devartstar FYI, alerts and prompts should be replaced like this

yes, will keep it in mind.

Closing as we are doing this as part of vue based simulator


[resource-limits] Use collect-status

Issue
Status updates are communicated via custom events.
Solution
Use collect-status.
In tandem with:

https://github.com/canonical/alertmanager-k8s-operator/pull/202

Is this how you'd expect collect-status to be used in components such as libs, @benhoyt @tonyandrewmeyer?

@sed-i Sort of ... though it looks like in this case the charm lib is firing a custom patch_failed event and has is_ready() and last_error methods/attributes. So you probably wouldn't actually do this in the charm lib at all. Keep that nice and stand-alone, and leave the status handling to the charm itself. So the charm would do something like this:
class SomeCharm(ops.CharmBase):
    def __init__(self, *args):
        # ...
        self.resources_patch = KubernetesComputeResourcesPatch(...)
        self.framework.observe(charm.on.collect_unit_status, self._on_collect_unit_status)

    def _on_collect_unit_status(self, event):
        if self.resources_patch.is_ready():
            event.add_status(ops.ActiveStatus())
        elif self.resources_patch.last_error:
            event.add_status(ops.BlockedStatus(self.resources_patch.last_error))
        else:
            event.add_status(ops.MaintenanceStatus('waiting for patch'))

Note that because KubernetesComputeResourcesPatch has a last_error, you could just use this directly, instead of listening to patch_failed.
That would be my suggestion, at any rate.

Thanks @benhoyt!
The custom event is how we currently propagate status updates. Certainly that part won't be needed with collect-status.
I tend to agree that it's probably best to have only the charm use add_status.

One slightly unfortunate thing to note about collect-status is that your charm has to be "all in" with it. You can't really switch half of your charm to use collect-status / add_status and leave the rest using unit.status = x, because the unit.status assignment will be overwritten with the (partially-implemented) collect-status status.

I don't mind this approach, but it feels a bit awkward in that is_ready would return False iff a last error has been set, which means that if you get a False, then you have to go search for a last error.

Not sure what you mean. Could you point to a line in code?

I don't mind this approach, but it feels a bit awkward in that is_ready would return False iff a last error has been set, which means that if you get a False, then you have to go search for a last error.

Not sure what you mean. Could you point to a line in code?

Can't point to a line in the code because it's about the API it exposes.
IIUC the current API implies that, as a user of this lib, the charm would:
lib = Lib()

status: Success | Failure = lib.get_status()
if isinstance(status, Success):
    # go about your happy path

else:  # Failure
    # retrieve the error msg
    error = status.message

Which I think is a bit awkward.
An alternative I was proposing is:
lib = Lib()

if lib.is_ready():
    # go about your happy path

else:
    # retrieve the error msg
    error = lib.last_error

I realize this deviates from the nice is_ready pattern we've been gliding so far, but fact is now we're introducing a new piece of data: it's not just 'are you ready? --> YES | NO', now it's: 'are you ready, and if not, why? --> YES | NO (because)'
And if we plan to build this out to all other is_ready methods we have in our charm libs, it might be worth it to figure out a generalization-worthy pattern.

That is a good point.
What immediately comes to mind is the result package.
But I am not sure we want to couple readiness with errors:

Could there be a circumstance where we want is_ready to return True, while at the same time having an error (Blocked)?
last_error is set during operations, not necessarily during "get status", so we would need that variable anyway (perhaps prepend with an underscore, _last_error). Which means later on we could hide this behind a "get status" if we choose to.
Could "last error" and "is ready" go out of sync? last_error gets reset per charm reinit so if an issue is resolved during custom hooks, is_ready could be True, but last_error still holds the past.

That is a good point. What immediately comes to mind is the result package.

Ha, never heard of that. Interesting!

But I am not sure we want to couple readiness with errors:

Could there be a circumstance where we want is_ready to return True, while at the same time having an error (Blocked)?

At charm level, sure, something like the 'degraded' status @jnsgruk was brainstorming about a few Pragues ago, but in this case we're talking about a pattern charm libraries, which are (for the most part, at least our libs) simple enough to either be 'ready' or 'not ready', without intermediate 'ready-but-kind-of-borky' states.

last_error is set during operations, not necessarily during "get status", so we would need that variable anyway (perhaps prepend with an underscore, _last_error). Which means later on we could hide this behind a "get status" if we choose to.

Fair enough, I'm game.

Could "last error" and "is ready" go out of sync? last_error gets reset per charm reinit so if an issue is resolved during custom hooks, is_ready could be True, but last_error still holds the past.

Which makes me think, last_error may also quickly become stale as it currently stands (independently of is_ready).
if not lib.is_ready():
    err = lib.last_error  # set because not ready
    lib.do_something_to_fix_error()
    err2 = err.lib.last_error  # still set because it's only cleared on __init__

Which makes me think, last_error may also quickly become stale as it currently stands (independently of is_ready).

I'm not sure we can do anything about it when the lib has multiple entry points (is_ready, on_config_changed -> patch).
Even if I give each error individual attention,
@dataclass
class LibErrors:
    obtain_limit: Optional[str]  # Error obtaining resource limit from user function
    validity: Optional[str]  # Resource spec is invalid
    config: Optional[str]  # ConfigError
    api: Optional[str]  # ApiError
    patch: Optional[str]  # Failed applying patch

        try:
            do_something()
            self.errors.obtain_limit = None
        except ValueError as e:
            msg = f"Failed obtaining resource limit spec: {e}"
            self.errors.obtain_limit = msg

They could still be stale for the same reason.
If we stop using custom events, then...

I closed the tandem alertmanager PR for now.
Let's try to come up with a solid collect status pattern first.
Closing this too, for now.


Support HSQL and Postgrees using the same Liquibase config

This is an effort to minimize the clutter in the config and the support hsqldb in junit developer testing with the same changesets that are using in Postgre in production/integration testing

Hi @alero ,
Would you resolve branch conflict and  consider adding tests for this PR?  Here's some guidance on that below:
Thanks for your pull request!
Here’s what happens next:
A member of the Liquibase team will take a look at your contribution and may suggest:

Changes based on guidelines for PR submission
Additional tests (unit and/or integration)
Provide other feedback, like requesting changes.

The PR will be prioritized according to our internal development and testing capacity.
We’ll let you know when it’s ready to move to the next step or if any changes are needed.

@alero thanks for getting rid of the merge conflict!  I'll size it as a medium, but happy to size it low if we get tests added.

Hello @alero ! A lot changed in the last years on Liquibase. Is this change still required?

HI @filipelautert we still use this adaption and have actually forked it last month again to be up to date with latest releases

@alero ! Thanks for the quick answer. Do you think you can fix the conflicts (as we renamed liquibase-core to liquibase-standard and so on) so I can keep on moving this one along?

Hello @alero ! I fixed the conflicts and tried to push the fix to your master branch but I don't have permission to do so. Would you be able to grant permission or fix the conflicts ?

Hello @filipelautert I have added you to the repo, so should be able to push now and thank you for the support.

Hello @filipelautert I have added you to the repo, so should be able to push now and thank you for the support.

Thanks , almost there! But seems the branch is protected and I can't push to your master anyway, so I created PR https://github.com/alero/liquibase/pull/1 in your repo to your master 🤣  could you approve and merge it?

approved and merged it to master


[HUDI-7929] create k8s example for flink hudi integration

Change Logs
Add K8s example for Flink Hudi ingegration
Impact
No public API changes.
Risk level (write none, low medium or high below)
No Risk
Documentation Update
Added README for the new module
Contributor's checklist

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

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure


Use Value for ResolveObjectResponse value type.

If the discussion here is implemented, I think we want to change ResolveObjectResponse from:
message ResolveObjectResponse {
    google.protobuf.Struct value = 1;
    string reason = 2;
    string variant = 3;
}

to:
message ResolveObjectResponse {
    google.protobuf.Value value = 1;
    string reason = 2;
    string variant = 3;
}

This allows us to return arrays (as well as other protobuf values) as the top level object in the object resolver.
Is there any objections or challenges to doing this? I've POC'd it in the flagd-java provider. Would it be reasonable in flagd?
@james-milligan @skyerus @beeme1mr

I have had play around with this library:
google.golang.org/protobuf/types/known/structpb
It seems pretty doable, we can cast the Value type into an interface{} and return this directly for the provider.
Within flagd we can just do the inverse
No objections from me


Snakebites game added

PR Description 📜
Added Snake Bites Game with intresting features ,adding sound effects, and adding level of the game so that snake can move slower or faster!!
Fixes #3549

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

https://github.com/kunjgit/GameZone/assets/140592148/cb67f081-ba8e-4a47-86ea-35f84231e449

Thank you soo much for contributing to our repository 💗

@kunjgit , added README.md file.

@kunjgit Changes done!! Please review it and merge ..

I have updated to the latest README.md file @kunjgit , please review and merge it.

@kunjgit , I have updated to the latest readme.md file sir , its showing 0 Conflicts also

@SrijaVuppala295 its not having numbering please have a look at it

@SrijaVuppala295 its not having numbering please have a look at it
@kunjgit , do you mean here.

Please provide the file that need to be changed

@kunjgit ,please provide the file to be changed

@ishita-43 , please review this , i have resolved merge conflicts.

Please follow the contribution guidelines while naming the game and respectively update the name in Readme.md file of the project!!

@ishita-43 ,please review it

@ishita-43 , if again the conflicts are not resolved, i will try to create a new PR , or say what should i change in that file

Thank you  @SrijaVuppala295 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


[v2] Base cross-arch support

As discussed in #1, I have:

got rid of all arch-specific bits of the public API, and all arch specific tests
added copyright headers
squashed the CRC duplication in igzip.

In addition, I wanted to make sure I hadn't broken 32-bit, which turned out to be a bit more complex than I thought as it didn't build out of the box with autotools. There are patches to:

enable 32 bit builds with autotools (patch 2)
use the base versions for crc32 and igzip, which don't seem to have working 32 bit versions (correct me if I'm wrong)
added a new dispatcher to dispatch the SSE3 functions on for RAID on 32-bit systems.

I have enabled 32-bit Linux and 64-bit OS X tests in the final patch as well.
I have verified that 'make test check perf' works on 64 and 32 bit x86, and ppc64le. I will add any fixes to compile on Windows tomorrow.

ergh, turns out this is actually broken on x86_64! Fix to come.

and it turns out that gf_vect_mad_avx2 returns different results to gf_vect_mad_{avx,sse}, which is what's causing the problem.

Reported upstream: https://github.com/01org/isa-l/issues/19

Ok, that should be good to go now :)

The fixes look ok.  I have all the base aliases already and some of the x86 fixes but the others should apply cleanly.  We have a lot of changes to compression coming and I'll ask you then to rebase on that.

Excellent - let me know once they land and I'll send you a v3.


Support for Outline

Added outline.scm to allow Zed display file structure. This includes macros followed by a block of code (like for tests or DSLs) and object variants.
(Things could probably be enhanced)

Thank you for your contributions, Will test it and add it to the next release.


data_race: link to docs for 'unusual' race conditions

@bors r+

:pushpin: Commit e1462ad6b820b0329e8826eb7096502f0ee048c4 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit e1462ad6b820b0329e8826eb7096502f0ee048c4 with merge eb4e895ff9276a97729e0c700686fb5eafe37860...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing eb4e895ff9276a97729e0c700686fb5eafe37860 to master...


Play around with the Opentelemetry Operator and see if it makes sense to use it

We can try being fancy and use the operator in kubernetes. If not we will use the helm chart with agents and a gateway.
Acceptance Criteria

There is understanding about best practices and ways of running the agents using the operator
The options are documented on this ticket
Decision is made on whether to use it
Tickets are raise for implementation if decided

Nah, it might be obscuring inner workings while not providing extra clarity


remove_ascii_art option doesn't work

https://github.com/DataDog/mkdocs-click/blob/a8742d9afc6bfd0ce70e048383a623fc218625f8/README.md?plain=1#L137
docs/index.md:
    :remove_ascii_art: True

Docstring:
    """
    Replace titles names from anidb.
    The result will be copied to the clipboard.

    \b
    >>> 1 	The Prince`s New Clothes
    <<< e1: EP1 «The Prince`s New Clothes»
    """

Result:

With that docstring doesn't work either.
    """
    \b
    >>> 1 	The Prince`s New Clothes
    <<< e1: EP1 «The Prince`s New Clothes»

    Replace titles names from anidb.
    The result will be copied to the clipboard.
    """


[#DIMA-853] Move Export to BitcrowdEcto

[#DIMA-853]

[ ] changelog

nevermind this for now, thinking about a different way


Adds audience field to sign token messages.

Our JWT tokens are missing the audience field, which is required by many JWT validators, such as this one, which will be needed for our new GQL API: https://github.com/auth0/go-jwt-middleware.
More info on Audience Claim https://datatracker.ietf.org/doc/html/rfc7519#section-4.1.3

LGTM


nacos registry doesn't work if simplified registration is enabled.

If enable simplified registration for nacos:
<dubbo:registry address="nacos://localhost:8848" simplified="true"/>

Then providers:::: is registered on nacos for the service name, which is apparently unexpected.

see https://github.com/apache/dubbo/blob/master/dubbo-registry/dubbo-registry-nacos/src/main/java/org/apache/dubbo/registry/nacos/NacosRegistry.java#L457

https://github.com/apache/dubbo/issues/4621

cool, then I will close this issue. dup of #4621


Issue #2: Add indexes to collections

Based on issue #2 this PR is creating all possible indexes based on current collection usage

Thanks! I will review it soon.

Great job


add active property to taxonomy classes

needed for https://github.com/ethyca/fides/issues/3709
Description Of Changes
Adds an active flag to the models for taxonomy elements. Flag defaults to true.
Code Changes

[x] add active boolean property to the taxonomy element models: DataUse, DataSubject, DataQualifier, DataCategory

Steps to Confirm
not much here - corresponding fides PR has some actionable confirmation steps
Pre-Merge Checklist

[x] All CI Pipelines Succeeded
[ ] Documentation Updated
[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[x] Update CHANGELOG.md

@ThomasLaPiana just a small fideslang update that's needed for some fides functionality. once we get this PR merged and the packaging issues i describe in https://github.com/ethyca/fides/pull/3784#issuecomment-1636424081 fixed, i'd like to release a point-release of fideslang here so we can get these updates properly into fides!

@adamsachs I'm not sure I'm onboard with this change, can we discuss here a bit?
I totally understand the need for the core change itself (hiding certain taxonomy items in the UI) but this doesn't feel like the right place to do it.
Why do we need to update Fideslang to make it so that certain items don't show up in a UI, in a downstream dependency? It doesn't seem like the active flag adds any value to Fideslang in isolation. This is a tight coupling of functionality.
As an alternative, could we add this as a database field in Fides directly to keep track of? Or somehow store it elsewhere without Fideslang needing an update?
I know this is definitely the "fastest" solution, but I'm pushing back here since I don't think it is the right one

@adamsachs is there a reason not to extend the existing Pydantic classes for those models? That seems like the simplest solution to me but maybe I'm missing something there

@adamsachs is there a reason not to extend the existing Pydantic classes for those models? That seems like the simplest solution to me but maybe I'm missing something there that required the mixin

oh yeah, i just thought a mixin is a slightly cleaner way of doing that - i.e. extend the models in fides, but extend them by sharing a common mixin base class (which defines the active field) rather than all of the extensions "happening" to have the same active field.
maybe i'm not thinking of mixins correctly though? i think we're roughly on the same page, so i can get a draft PR up for review and that'll probably be easier than trying to talk through it at this point 😄

closing as we're going in a different direction per above conversation - see https://github.com/ethyca/fides/pull/3784, the changes are self-contained there in fides 👍


IU Issue

Hello,
I have the RAM cache set to a value (200M) and the disk cache set to 0. However, the UI in the status bar displays an identical >0 value for both caches. Please assist.
Firefox 35.0.1
Thank you,
Robert

I'm sorry I wasn't able to resolve this issue at the time. Given its age (and probably irrelevance at this point), I'm going to close this.


cloudscraper, update csrf_regexp, auth dict

handling for cloudflare bot protection
update csrf_regexp for new page format
update auth payloads for new dict structure

@haochi any chance you can review/approve this PR? Many thanks for providing this library!

+1

+1

I think cloudscraper needs to be listed in requirements

Also there's a typo in line 60 of main. It should be
        pc.authenticate_password(email, password)

I want to use this API and open the account every morning using Selenium automatically. Using API, I wan to download all transactions, included just showed up in the account (but not fully posted) and balances of all the accounts. I would like to know few things.

Can I know if it will work. Can API bypass the 2FA and login automatically?
If I login in the morning after the banking hours, do all the transactions updated but not fully posted  get downloaded?
Can Posted balance and Available balance both be downloaded?

Thanks in advance.
Jay

I am unable to login using the latest pull #21 .
It gave me this error first. ERROR:root:[Errno 2] No such file or directory: 'session.json'
If I create a json file with that name with {} value, it gives me this error personalcapital.personalcapital.LoginFailedException.
I tried to debug :
print(initial_csrf)
print(csrf)
print(auth_level)
Output:
None
None
None
Please help.

@UnusualPi , I also encountered this issue and realized just updating the User-Agent will satisfy CloudFlare's checks (see #24 ).  I'm sure there are many other ways to address, but the User-Agent tweak avoids adding a dependency to the library.

@UnusualPi , I also encountered this issue and realized just updating the User-Agent will satisfy CloudFlare's checks (see #24 ). I'm sure there are many other ways to address, but the User-Agent tweak avoids adding a dependency to the library.

I like this better than my solution, much cleaner without a new dependency.  I assume the user agent validation is the primary reason cloud flare would catch it so should work for most.

I am unable to login using the latest pull #21 . It gave me this error first. ERROR:root:[Errno 2] No such file or directory: 'session.json' If I create a json file with that name with {} value, it gives me this error personalcapital.personalcapital.LoginFailedException. I tried to debug : print(initial_csrf) print(csrf) print(auth_level) Output: None None None Please help.

I'm experiencing the same errors in the same situations.

I'm experiencing the same errors in the same situations.

@brodia10 , try using #24 , it may resolve the issues

I'm experiencing the same errors in the same situations.

@brodia10 , try using #24 , it may resolve the issues

@hammem thank you this worked wonderfully 👍

Looks like @UnusualPi deleted their fork of this repo. I made the same changes they made in my own repo here: https://github.com/banool/personalcapital.

@banool , I think they deleted it to use the changes put up in #24 .  You may find this works well (at least it did for me as recently as this week).


Fix ViewModels IllegalStateException

Task/Issue URL: https://app.asana.com/0/1200204095367872/1207733727777585/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#4720  👈
develop

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

Join @joshliebe and the rest of your teammates on  Graphite


Removal of duplicate max character mention.

Prompt related to self reply includes multiple mentions of keeping under 280 characters.

Thanks @0xGF , good catch


[Easy first task] Bug: Email button overlaying text

This is an easy-first-task for someone looking to get into OSS or contribute to this project :)
Example: https://jobs.nashdev.com/jobs/4

Current behavior
The
<a href="mailto: element is overlaying the text above it.
Expected behavior
The button should not overlay the text.

Thanks for reporting @matthamil!


playground triggers SIGSEGV

The playground seems to trigger SIGSEGV runtime exceptions when accessing undefined elements in a large slice. I could not reproduce this locally (I get compile-time too-much-required-memory errors for large slices and normal runtime behavior for smaller slices).
What version of Go are you using (go version)?
Go 1.12.9
Does this issue reproduce with the latest release?
Not on a local machine, only on the playground.
What operating system and processor architecture are you using (go env)?
Native Client, NaCl.
What did you do?
I ran the following 3 programs.

https://goplay.space/#Rbv05s_S5ax
https://goplay.space/#Maqi_gYFakb
https://goplay.space/#NIfrZa_IIxb

What did you expect to see?

Not a SIGSEGV
I got what I expected.
I got what I expected.

What did you see instead?

A SIGSEGV runtime exception.
What I expected.
What I expected.

I do get a seg fault when running locally, although not with the full stack trace.
GOOS=nacl GOARCH=amd64p32 go run foo.go
signal: segmentation fault

I do get a seg fault when running locally, although not with the full stack trace.
GOOS=nacl GOARCH=amd64p32 go run foo.go
signal: segmentation fault

You have to setup this https://github.com/golang/go/tree/master/misc/nacl for running nacl

Woops, thanks @cuonglm. I never ran a basic program to check whether it works or not.

Oh, these are old issues. Sorry, @bcmills: I didn't do my due diligence. I'll go ahead and close this. No point in issuing a fix.


Search & Filter for Resources List

Currently, we allow users to sort by Name and Date/Time. Users should be able to sort (also by Type), search, and filter from the Deployment/Resources list.

Removing as a good first issue, only because the contributor would need to be familiar with setting up a cluster to really test this, until we get Storybook set up again. to be able to work with dummy data.


apacheHttpd package options cannot be properly overridden

Problem: apacheHttpd is called with standard arguments like sslSupport = true and should be overridable via config.packageOverrides infrastructure.
{ config, pkgs, lib, ... }:
{
  nixpkgs.config.packageOverrides = pkgs:
  rec {
    apacheHttpd_2_2 = pkgs.apacheHttpd_2_2.override { ldapSupport = false; sslSupport = false; };
    apacheHttpd_2_4 = pkgs.apacheHttpd_2_4.override { ldapSupport = false; sslSupport = false; };
    aprutil = pkgs.aprutil.override { ldapSupport = false; sslSupport = false; };
  };
}

Result:
# nixos-rebuild --show-trace build
building the system configuration...
error: while evaluating the attribute ‘buildCommand’ of the derivation ‘nixos-system-*-16.03.git.6d31e9bM’ at /.../nixpkgs/nixos/modules/system/activation/top-level.nix:102:7:
...
assertion failed at /.../nixpkgs/pkgs/servers/http/apache-httpd/2.4.nix:13:1

Where 2.4.nix line 13 is the assertion:
assert sslSupport -> aprutil.sslSupport && openssl != null;

Expected result: It should work and build apacheHttpd and aprutil without sslSupport.
Reproducible: Always.
Further remarks:

apacheHttpd reacts on changing overrides and produces different derivations
The problem can be mitigated by setting the default values for sslSupport ? false and ldapSupport ? false in the function argument list of apacheHttpd in 2.4.nix. Then, the overrides get correctly detected and the assertions evaluate properly (including asserting aprutil.sslSupport).

Clearing up things:

The services.httpd.package defaults to pkgs.apacheHttpd, but it does not pick the version from the packageOverrides.
Applying the override directly in services.httpd.package works.

So it's like the pkgs passed as argument to the module is not taking in account the overrides from nixpkgs.config.packageOverrides. I confirmed this.
cc @nbp

I think I'm coming to the conclusion. The problem is apacheHttpd.override, together with the aprutil. It's not possible to call .override on apacheHttpd because it will fail due to aprutil not supporting ssl.
However I wonder why this happens, since pkgs.apacheHttpd.override should be using the older pkgs...

Ah.
This works:
apacheHttpd = super.callPackage .../pkgs/servers/http/apache-httpd/2.4.nix { sslSupport = false; ldapSupport = false; }

This doesn't:
apacheHttpd = super.apacheHttpd.override { sslSupport = false; ldapSupport = false; };

So it's .override that is using the new pkgs... is that the correct behavior?
cc @edolstra

This is another strictness issue similar to #11094. .override needs to evaluate the original package to check whether it is a function or a set; that hits the assertions because aprutil is compiled without ssl/ldap support.
Moving the asserts down like
    optional ldapSupport (assert aprutil.ldapSupport && openldap != null; openldap) ++    # there is no --with-ldap flag

should work.

Or let callPackage/override use the old pkgs when checking if it's a function/set ?

So, thanks to research done in https://github.com/NixOS/nixpkgs/issues/31884 I've got list of problematic asserts:
/home/danbst/dev/nixpkgs/pkgs/applications/version-management/subversion/default.nix:assert bdbSupport -> aprutil.bdbSupport;
/home/danbst/dev/nixpkgs/pkgs/servers/http/apache-httpd/2.4.nix:    assert ldapSupport -> aprutil.ldapSupport && openldap != null;
/home/danbst/dev/nixpkgs/pkgs/servers/http/apache-httpd/2.4.nix:    assert sslSupport -> aprutil.sslSupport && openssl != null;
/home/danbst/dev/nixpkgs/pkgs/development/libraries/libxslt/default.nix:    assert pythonSupport -> libxml2.pythonSupport;
/home/danbst/dev/nixpkgs/pkgs/development/tools/misc/dialog/default.nix:      assert unicodeSupport -> ncurses.unicode && ncurses != null;

and here is module that show problems with these overrides:
{ config, pkgs, lib, ... }:
let
    brokenApache = {
        environment.systemPackages = [ pkgs.apacheHttpd ];
        nixpkgs.overlays = [ (self: super: {
            apacheHttpd = super.apacheHttpd.override { ldapSupport = false; sslSupport = false; };
            aprutil = super.aprutil.override { ldapSupport = false; sslSupport = false; };
        })];
    };
    brokenSubversion = {
        environment.systemPackages = [ pkgs.subversion ];
        nixpkgs.overlays = [ (self: super: {
            aprutil = super.aprutil.override { bdbSupport = false; };
            subversion = super.subversion.override { bdbSupport = false; };
        })];
    };
    brokenXslt = {
        environment.systemPackages = [ pkgs.libxslt ];
        nixpkgs.overlays = [ (self: super: {
            libxml2 = super.libxml2.override { pythonSupport = false; };
            libxslt = super.libxslt.override { pythonSupport = false; };
        })];
    };
in {
    config = lib.mkMerge [
        ({
            fileSystems."/".device = "nodev";
            boot.loader.grub.devices = [ "nodev" ];
        })
        brokenApache
        brokenSubversion
        brokenXslt
    ];
}

This crap happened because asserts are done before .override can be monkeypatched into derivation. If we move them inside derivation, then override machinery starts to work. It isn't hard to fix these cases, but how to prevent them in future?

I believe this one has outlived itself.


Allow performance-analyzer webservice-listener and rpc ports to be changed in docker build

Problem
When running multiple instances in docker, while using the host networking interface for performance reasons, performance analyzers trying to bind to the same port causes containers to die.
Solution?
Allow these aforementioned ports to be changed via environment variables.
plugins.pa_webservice_listener_port: 9651
plugins.pa_rpc_port: 9601
Or better yet, make configuration with environment variables more expressive and easier to manage:
version: "3.9"

services:
  opensearch-node:
    image: opensearchproject/opensearch:1..3.0
   environment:
     # OPENSEARCH_<opensearch.yml settings>
     OPENSEARCH_CLUSTER_NAME: opensearch-cluster
     OPENSEARCH_DISCOVERY_TYPE: single-node
     OPENSEARCH_HTTP_PORT: 9201
     OPENSEARCH_TRANSPORT_PORT: 9301
     # OPENSEARCH_PLUGINS_<plugin-name>_<plugin-setting>
     OPENSEARCH_PLUGINS_PA_WEBSERVICE_LISTENER_PORT: 9651
     OPENSEARCH_PLUGINS_PA_RPC_PORT: 9601
     OPENSEARCH_PLUGINS_SECURITY_KEYSTORE_PATH: /usr/share/opensearch/keystore.jks
     OPENSEARCH_PLUGINS_SECURITY_KEYSTORE_PASSWORD: p@ssword!
     ...

I think this naming scheme would make it so much easier to configure containers using environment variables. Because as it is, if I'm not mistaken you're just splitting environment variables by a "=" and only capturing key=value pairs for opensearch options. As seen here:
# Parse Docker env vars to customize OpenSearch
#
# e.g. Setting the env var cluster.name=testcluster
# will cause OpenSearch to be invoked with -Ecluster.name=testcluster
opensearch_opts=()
while IFS='=' read -r envvar_key envvar_value
do
    # OpenSearch settings need to have at least two dot separated lowercase
    # words, e.g. `cluster.name`, except for `processors` which we handle
    # specially
    if [[ "$envvar_key" =~ ^[a-z0-9_]+\.[a-z0-9_]+ || "$envvar_key" == "processors" ]]; then
        if [[ ! -z $envvar_value ]]; then
        opensearch_opt="-E${envvar_key}=${envvar_value}"
        opensearch_opts+=("${opensearch_opt}")
        fi
    fi
done < <(env)

I've tried
Mounting the performance-analyzer.properties files manually to force the behaviour. For some reason it didn't work, and documentation being lacking I cannot find a work around.

@DogukanL would you be able to open a PR for this if passing an environment variable worked for you?

Hi,
someone mixed it up,  please double check!
PORT
it actually is:

960x -> for the Performance analyzer webservice-listener
965x -> for the RCA

# WebService exposed by App's port
webservice-listener-port = 9600
#...
# Port for RPC Communication
rpc-port = 9650

BIND to Interface
in addition please make the BIND host also available as config + environment variable:
# WebService bind host; default only to local interface
webservice-bind-host = <IP_ADDRESS>

the RCA bind option is missing in the performance-analyzer.properties please implement this.
Please note that both, BIND and PORT is required for a docker based (host mode) and/or native multnode installation on a single host !


Bump metronome on DC/OS 1.13

High-level description
Added missing HTTP metrics for Metronome.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-5020 Add missing HTTP metrics in Metronome.

Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): https://github.com/dcos/metronome/compare/b4016b0...9055768
[x] Test Results: https://jenkins.mesosphere.com/service/jenkins/view/Metronome/job/Metronome/job/metronome-pipelines/job/master/12/console
[x] Code Coverage (if available): N/A

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot label Ready For Review


UI updates and cleanup for v0.9

Requirements:

Show installed ODH components

Shown as tiles

Show available ODH components

Shown when "Show All" is selected with a slightly gray font and a badge "Not Installed"

Give link to external documentation

Documentation link in the kebab menu for each card

Launch ODH component UI (if available) in a new tab.

Launch link in the kebab menu for each card
Launch link in the footer.

@vpavlin I think this also resolves https://github.com/opendatahub-io/odh-dashboard/issues/11

resolves https://github.com/opendatahub-io/odh-dashboard/issues/11


Feature: Free Rotation and Translation

Mousedrag and Shift + Mousedrag on perspective view only now implement pointer lock, pointer disappears and movement is unimpeded. Cursor restored upon mouseup

Woops this isn't ready for the google branch


Can't build on linux from Haxelib

Hi
I'm trying to build anything on linux but I can't get anything to work I get this error that I think is the reason
Haxe error: Error: spawn /usr/lib/haxe/haxe ENOENT
I have no idea what that is about, after that I get a shader compiler error
Shader compiler stderr: /usr/lib/haxelib/kha/15,7,3/Kore/Tools/krafix/krafix-linux64: /usr/lib/x86_64-linux-gnu/libstdc++.so.6: version `CXXABI_1.3.8' not found (required by /usr/lib/haxelib/kha/15,7,3/Kore/Tools/krafix/krafix-linux64)
/usr/lib/haxelib/kha/15,7,3/Kore/Tools/krafix/krafix-linux64: /usr/lib/x86_64-linux-gnu/libstdc++.so.6: version `GLIBCXX_3.4.20' not found (required by /usr/lib/haxelib/kha/15,7,3/Kore/Tools/krafix/krafix-linux64)

you can see the rest in the pastebin here http://pastebin.com/CQ9APwxr
Using Linux mint 17.1 64 bit

OK already solved the compiler GCC issue by installing gcc 4.9
now i get just this
Generating Kha project.
Assets done.
Haxe error: Error: spawn /usr/lib/haxe/haxe ENOENT
Haxe process exited with code -2
Done.

thanks

So I figure out that the ENOENT error means it can't find the haxe executable there
which is correct, in linux the haxe executable is in /usr/bin/ not /usr/lib/haxe
anyway I created a symlink there and not the error is this
Generating Kha project.
Assets done.
Haxe stderr: ../Sources/Unknown.hx:10: lines 10-12 : Invalid override on field 'init': class has no super class
../Sources/Unknown.hx:7: characters 2-18 : Current class does not have a super
../Sources/Main.hx:8: characters 16-29 : Unknown should be kha.Game
../Sources/Main.hx:8: characters 16-29 : For function argument 'game'

Haxe process exited with code 1
Done.

thank you

Hm, that haxe location seemed to work for others, have to investigate.
Looks like whatever is hidden in Unknown.hx (which is unknown to me I think?) has to extend kha.Game.

Hi Rob
I just figure it out, yeah the generated class is missing extends Game
now it compiles to html5
now how to I test it?
if I just open the html file I get an error
loadBlob failed
seems to me like I need to run a nodejs server to view it, I'm new to nodejs to have to investigate that
still I think those starting issues should be addressed
thanks

Fully agree about the starting issues.
Running haxelib run kha --server will run the webserver on 8080 (opening the html file directly doesn't work in most browser because they don't allow JavaScript to load files when it is executed locally).

Thank you Robert that was what I needed
is working now
Hope my issues help, if I can contribute more I will thank you

Of course, feedback like that is very valuable.
Will keep this issue open until I understand/fix the haxe path.

also note that gcc 4.9 is required, I think most distros don't ship with it yet
I'm not getting any result from building to linux, it succeeds but there's no app to run

You can add the --compile and/or --run parameters. By default khamake only creates IDE projects for manual compilation/debugging/profiling.

ok using --compile works but I had to move the executable from the linux-build folder to the linux folder, I saw somebody already reported that issue.
using --run doesn't work
--run not yet implemented for this platform

Hmm, --run not working is a new bug. OK, bunch of stuff to fix.

you said report bugs :)
and I really want to use Kha for a big WebGL application
thank you for your work Rob, really appreciate it

And I mean it, didn't want to sound negative :)

You didn't man, just joking, the workflow is way better now, I'll see if I can make a build system for Sublime text to make it even more streamlined also auto refreshing the page I think is possible with nodejs.

Lubos already did some Sublime stuff: https://github.com/luboslenco/sublime_kha

awesome, thank you

All fixed I think.


feat(examples): hello-java and web-service-java

Adds a couple examples here to kick things off.
This is example dist/hellojava.k8s.yaml file:
apiVersion: v1
kind: Service
metadata:
  name: hello-java-service-3a4f9468
spec:
  ports:
    - port: 80
      targetPort: 8080
  selector:
    app: hello-k8s
  type: LoadBalancer
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: hello-java-deployment-d312868f
spec:
  replicas: 1
  selector:
    matchLabels:
      app: hello-k8s
  template:
    metadata:
      labels:
        app: hello-k8s
    spec:
      containers:
        - image: paulbouwer/hello-kubernetes:1.7
          name: hello-kubernetes
          ports:
            - containerPort: 8080

Signed-off-by: campionfellin<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Closing in the meantime. Let's reopen when ready


implement --watch for status subcommand

An example on how to implement --watch for the status subcommand (See #26).
Comment are welcome :)

Closed due to immature GitHub name


add backup links for wavlm and unispeech-sat models

Besides the download links of Azure Storage, this PR adds the backup download links of Google drive.

Hi @Sanyuan-Chen !
Thanks for taking care of the link backup plan!
I think for the gdown agent to work, we might need to slightly modify the _local entries in the hubconf.py to catch the agent.
Take wavlm for example:
def wavlm_url(ckpt, refresh=False, agent="wget", *args, **kwargs):
    """
    The model from google drive id
        ckpt (str): URL
        refresh (bool): whether to download ckpt/config again if existed
    """
    return wavlm_local(
        _urls_to_filepaths(ckpt, refresh=refresh, agent=agent), *args, **kwargs
    )

Thank you @leo19941227 for the comments. I have fixed it in the latest commit.

Cool! I am merging this. Thanks!


Migrate to PureScript 0.14

This PR makes necessary changes to build with the shiny new PureScript 0.14.0 compiler. There were just a few kinds of changes:

generics-rep is not a thing anymore, it's been rolled into prelude.
Data.Generic.Show and Data.Generic.Eq are now Data.Show.Generic and Data.Eq.Generic respectively.
Data.Either.fromRight changed from Either a b -> b to b -> Either a b -> b. Replaced its use sites with fromJust <<< hush.

While I was at it, and partially to ease my own workflow, I added Spago and package.json with specific versions of purescript and spago. I left bower.json in place, because I wasn't sure it wasn't necessary for some other workflow.

I was just taking a look to see if there was a 0.14 PR ready for this library. Glad to see it!
I suspect that this will be merged much faster if it only includes changes to the source code and updated major dependencies in the bower.json file (all that’s strictly necessary to update). The bower file is still needed to publish libraries to Pursuit at the moment — hopefully not for much longer.

Bot doesn't the bower file get generated by Spago now?

It can be, for libraries that are primarily spago-based. But for maintainers who don’t want to switch until the new registry is released, the bower file is still maintained using the bower cli (or by hand).

So like I said in https://github.com/cdepillabout/purescript-email-validate/pull/17, I tried with Bower, and it spits out a ton of conflicts. My hunch is that this is because<EMAIL_ADDRESS>(which is in the current psc-0.14.0 package set) still depends on<EMAIL_ADDRESS>and that ripples out to everything else.
I'm trying to figure this out, but I don't really understand the mysterious ways of Bower...

Ok, @thomashoneyman, so I've made it work via Bower, but it's resolutions galore!

I upgraded all packages that were under "dependencies" to whatever version they have in the current psc-0.14.0 package set.
But one of them - purescript-ansi - still has old version 5.0.0, which has three other old dependencies -<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>Of those three, only lists wasn't in the Bower file before, so I added it as<EMAIL_ADDRESS>Still, somehow Bower managed to find a whole lot of outdated packages. Perhaps a problem similar to ansi exists somewhere in second- or third-tier dependencies?
So to make it work I added all those resolutions.
Still, even after that, the free package managed to end up outdated. No idea how. The way I noticed it was that it failed to compile.
So I fixed that by pinning free's version under "dependencies" too.

So now it compiles, but it feels supremely flimsy to me. I mean, if free managed to slip through, other packages could too, right? And I only noticed free because it failed to compile, so there could be silent failures.

Oof — you’ll need to update ansi, too, as you’re right its bower file is out of date and it will need a new release. It works in the package sets but unfortunately we do still have to support the bower workflows. I’m with you — bower is awful for this kind of thing — but it’s what we have to do for now. It often comes down to finding the transitive dependency that wasn’t updated.

Opened https://github.com/hdgarrood/purescript-ansi/pull/12

But it's not enough. Even pointing purescript-ansi to my branch in bower.json doesn't get rid of dependencies.
Investigating further...
You wouldn't, by any chance, know a way to ask Bower which path of transitive dependencies leads to a particular one, a la npm why? A brief search didn't show any results :-/

All the libraries in the bower file are core or contrib libraries except for pipes and ansi. Core, contrib, web, and node libraries are all guaranteed updated for 0.14, so I suspect it’s the pipes library.

I don’t know of any command like that, unfortunately, though I’m no bower expert. It would help!

It's the pipes library. This is where I got to before I stopped with my fork when I was working on my learning repo.

@JordanMartinez I just found pipes too and working on it, but it goes deeper still: pipes depends on mmorph, which is itself outdated. But that seems to be the last one.

Success! Updating ansi, pipes, and mmorph did the trick!

https://github.com/hdgarrood/purescript-ansi/pull/12
https://github.com/felixSchl/purescript-pipes/pull/18
https://github.com/hdgarrood/purescript-ansi/pull/12

All looks good to me +1 I’m not a maintainer, though, so I’m not able to merge or make a release.

You are now! :+1: Feel free to make a new release if that works.

Oh good heavens! I'm a maintainer too!

https://pursuit.purescript.org/packages/purescript-spec/5.0.0

@fsoikin enjoy :smile:


add Xcode project file

I needed to move the 'extern' folder into source, otherwise Xcode picks up the wrong 'getopt' file.

The other build files are not updated, yet.

Have you ever tried to use Premake?
I am using Premake to manage the building process, for both Linux and Windows.
I do not have a Mac so that there is no configuration for XCode.
ps. the getopt.c is for Windows MSVC only. On Linux or using MinGW it is simply ignored.

I did compile it as described in the Linux section. That worked fine. But debugging is so much easier in Xcode ...

@schriftgestalt I think you can add some configurations in premake.lua to add a xcode action.

@schriftgestalt I've add a xcode4 action to premake5.lua in this branch.
You can checkout whether it works (using the premake5 xcode4 and check out the workspace files inside build/xcode).

I checked it. The configuration need this change:
ALWAYS_SEARCH_USER_PATHS = YES
USER_HEADER_SEARCH_PATHS = ../../dep/** ../../src/**
And for projects like this, it is a but annoying to have so many sub projects.

@schriftgestalt
Searching subdirectories in dep and src are needless, given that dep/polyfill-msvc/getopt.h is needless for Xcode.

Nur otherwise it does not finds the files in 'extern'. That was the reason to put the extern folder into src.

But for some reason your Xcode projects didn't complain about the wrong getopt. I have to check that tomorrow.

@schriftgestalt
XCode does not support specifying multiple header search directory? That's bad.
I will open another PR to talk about project structure and more building backends.
Sorry for closing this PR, I will not include any IDE-specific files that can be generated from premake5.

The discussions and latest code are moved to #19.


possible race condition (only seen in dev)

I've seen this consistently when doing a scan with several changes, but I have to ^C the dev instance and restart:
21:31:53 system      | sending SIGTERM to all processes
21:31:53 css.1       | terminated by SIGINT
21:31:53 typecheck.1 | terminated by SIGINT
21:31:58 worker.1    | [Worker(host:win pid:12291)] Job ModelFileScanJob [2d3b3716-0d42-4ee8-8c41-26f2edb86b91] from DelayedJob(default) with arguments: [{"_aj_globalid"=>"gid://van-dam/ModelFile/6565"}] (id=8127) (queue=default) COMPLETED after 0.2440
21:31:58 system      | SIGINT received, starting shutdown
21:31:58 rails.1     |   User Count (0.7ms)  SELECT COUNT() FROM "users"
21:31:58 worker.1    | [Worker(host:win pid:12291)] Job ModelFileScanJob [969c190e-7c44-4d4d-8b57-9e5e2818a1af] from DelayedJob(default) with arguments: [{"_aj_globalid"=>"gid://van-dam/ModelFile/6566"}] (id=8128) (queue=default) RUNNING
21:31:58 rails.1     |   ↳ app/controllers/application_controller.rb:10:in auto_login_single_user' 21:31:58 worker.1    | [Worker(host:win pid:12291)] Job ModelFileScanJob [969c190e-7c44-4d4d-8b57-9e5e2818a1af] from DelayedJob(default) with arguments: [{"_aj_globalid"=>"gid://van-dam/ModelFile/6566"}] (id=8128) (queue=default) COMPLETED after 0.2360 21:31:58 rails.1     |   User Load (0.6ms)  SELECT "users".* FROM "users" ORDER BY "users"."id" ASC LIMIT ?  [["LIMIT", 1]] 21:31:58 worker.1    | [Worker(host:win pid:12291)] Job ModelFileScanJob [e1f0637a-52ed-4c19-8b86-c0997d7aff24] from DelayedJob(default) with arguments: [{"_aj_globalid"=>"gid://van-dam/ModelFile/6567"}] (id=8129) (queue=default) RUNNING 21:31:58 rails.1     |   ↳ app/controllers/application_controller.rb:10:in auto_login_single_user'
21:31:58 worker.1    | [Worker(host:win pid:12291)] Job ModelFileScanJob [e1f0637a-52ed-4c19-8b86-c0997d7aff24] from DelayedJob(default) with arguments: [{"_aj_globalid"=>"gid://van-dam/ModelFile/6567"}] (id=8129) (queue=default) COMPLETED after 0.2010
21:31:58 rails.1     |   CACHE User Load (0.0ms)  SELECT "users". FROM "users" ORDER BY "users"."id" ASC LIMIT ?  [["LIMIT", 1]]
21:31:58 worker.1    | [Worker(host:win pid:12291)] 21 jobs processed at 3.2545 j/s, 0 failed
21:31:58 rails.1     |   ↳ app/controllers/application_controller.rb:11:in auto_login_single_user' 21:31:58 worker.1    | [Worker(host:win pid:12291)] Exiting... 21:31:58 rails.1     |   User Load (0.6ms)  SELECT "users".* FROM "users" WHERE "users"."id" = ? ORDER BY "users"."id" ASC LIMIT ?  [["id", 1], ["LIMIT", 1]] 21:31:58 worker.1    | [Worker(host:win pid:12291)] Exiting... 21:31:58 rails.1     |   ↳ app/controllers/application_controller.rb:11:in auto_login_single_user'
21:31:58 rails.1     |   Delayed::Backend::ActiveRecord::Job Count (0.7ms)  SELECT COUNT() FROM "delayed_jobs"
21:31:58 rails.1     |   ↳ app/controllers/application_controller.rb:21:in `check_scan_status'
21:31:58 rails.1     |   Creator Load (0.7ms)  SELECT "creators". FROM "creators" ORDER BY "creators"."name" ASC LIMIT ? OFFSET ?  [["LIMIT", 12], ["OFFSET", 0]]
2

I've seen this too and forgot to note it down, normally because I'm debugging something else at the time! Thanks for making the issue. I think it's to do with the SQLite lock, but I'm not sure.

I thought the same thing about the lock...   I encountered it several times before I noticed the pattern enough to recognize it as a real issue.
I suspect the 4 threads that are running are involved as well, since the scan is running in the background while the browser keeps doing it's thing.  (And it most definitely seems like it's running a scan that triggers this).
https://www.sqlite.org/threadsafe.html

The right solution is probably to abandon sqlite in dev as well, and use postgres. 🤷🏻

I think some of these SQLite tuning tips that increase concurrency, avoid locks, etc, might help: https://kerkour.com/sqlite-for-servers

Rails specific: https://fractaledmind.github.io/2023/09/07/enhancing-rails-sqlite-fine-tuning/

Changing to https://github.com/Betterment/delayed, an optimised version of delayed_job might help too.


Refactor bindgen snapshot unit tests to use insta

As @frol suggested in https://github.com/near/near-sdk-rs/pull/1036#discussion_r1246573665 we should refactor bindgen snapshot unit tests to use the insta crate.
Unit tests that could benefit from this:
https://github.com/near/near-sdk-rs/blob/c39fba338c2fbbb2e4dd1487a4c6c978a780f89c/near-sdk-macros/src/core_impl/code_generator/item_impl_info.rs#L30
https://github.com/near/near-sdk-rs/blob/c39fba338c2fbbb2e4dd1487a4c6c978a780f89c/near-sdk-macros/src/core_impl/code_generator/item_trait_info.rs#L30
https://github.com/near/near-sdk-rs/blob/c39fba338c2fbbb2e4dd1487a4c6c978a780f89c/near-sdk-macros/src/core_impl/code_generator/ext.rs#L146
https://github.com/near/near-sdk-rs/blob/6887b411d2852fbcb41232264cc4178d09895d2b/near-sdk-macros/src/core_impl/abi/abi_generator.rs#L295

Hi @agostbiro, are you working on this issue? If not, I would like to try it out.

@agostbiro @jaswinder6991 @frol  Do we have any progress? Can I take this task?

@iho Please, go ahead! Take inspiration from borsh-rs setup.


Set up a throw away email

references #106
Create gmail account to use between everyone for testing purposes, could also be used on Cybersecurity Entrepreneurship Challenge

[x] Create gmail account
[x] Ensure everyone has the credentials

Created email and sent credentials
[t:5]


Zombie processes

I have noticed that Ferdi produces between 10 and 20 zombie processes right after launching it and it only gets more. When closing Ferdi, they are all gone.

What version of Ferdi are you running? On which operating system?

I am running the newest flatpak version available on Pop OS 20.10 which - if I'm not mistaken - is 5.5.0.

@PhiCecHey - could you please try with the latest nightlies that can be found here - v5.5.0 is quite old - and we have made huge changes over the past couple of months. Though it says "nightlies", they are quite stable - and the contributors have been dog-fooding these builds for the past few weeks.

Ok sure. Does that mean that one should always use the latest nightlies?

For now, yes. Once we are able to fix the broken signing process, and we successfully release 5.6.0, then we should be able to go back to the "norm" of GA releases, betas and nightlies.

It seems that there are no more zombie processes. :D Thank you. Since I found Ferdi via the Pop Shop, maybe there is a way to put a notice there to use the latest nightlies?

Since I don't use PopOS - I'm not sure how I would go about doing that - could you please do it for us? Also, I'm closing this ticket - since the issue is now resolved.


Ensure ordering of tables and relationship

resolves https://github.com/datnguye/dbterd/issues/80

Description
Ensure ordering of tables and relationships. This will allow CI/CD to suppress unnecessary differential output when automatically generating documents.

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR

Nice & very simple change! 🎉


Fix a possible thread stall

Fix for #1491

Tests pass, except for the load tester which has some kind of internal Travis CI error that intermittently happens


Change plugin engine to operate on graphql compose schema

Currently for creating schema we pass immutable schema object. We should
operate on graphql compose schema and generate the immutable schema.after all plugins will finish. See core package for more details.
Interesting places:
GraphbackMetadata - schema field (this in immutable)
GraphbackPluginEngine - we passing this schema over
Each plugin has some hacks.
The most interesting place to look on is OffixPlugin (to be Renamed to DataSync plugin)

This is the implementation that we need to use:
https://graphql-compose.github.io/

@craicoverflow Can you add more information what is currently missing.
On top of my head issues with the composer are:

composer adding null to params
composer removing descriptions

Maybe we can investigate why this happens.

schemaComposer.buildSchema() will throw an error if the data model does not have any root query defined.
This and the two issues you mentioned can be worked around (hacked to work) but this is far from ideal.

We should operate on graphql compose schema

Is this a good idea? We would be moving away from the reference implementation. As we saw in https://github.com/graphql-compose/graphql-compose/pull/237 there can be divergences which causes issues.
Obviously it seems like the best approach right now other than writing something ourself.

Considering how good and responsive GraphQL-Compose team is we might actually log issue for them to not remove comments etc. but we should make sure first if the problems are not on our side

I think that we good really good deal here already thanks to graphql compose contributor. Do you think we can do more? Maybe docs for plugin creation?

I think that we good really good deal here already thanks to graphql compose contributor. Do you think we can do more? Maybe docs for plugin creation?

We could have a dummy plugin described in the docs that use graphql-compose or something. We don't want to push it too hard as the only way to do things.

I think offix plugin can become that example. Descriptions are back so things should be quite easy from now on. I might create some helpers in core but we will see how it goes once we start refining offix plugin

So we will create new plugin for this, or modify Offix?

@ssd71 For what specifically. There is PR that @cecobask done that was partially resolving the problem. https://github.com/aerogear/graphback/pull/829

I misunderstood, sorry about that! I will try some stuff and get back to you

there was a fix done here https://github.com/aerogear/graphback/pull/912 - custom  root fields were not getting added to the final schema.
This is a temporary solution, we believe that this current issue #810 will enable us to merge schemas more easily. So once this PR is complete @ssd71 we can work together to improve #912 :)

Some quick offline discussion with @craicoverflow and we decided to use graphql-compose across the board. This means that we can start work by rewriting SchemaPlugin to use graphql compose (currently does some hacks on graphql-js itself)

Understood, I will make the necessary changes

From what @wtrocki said in #913 , the approach:

Have every plugin extending schema creating GraphQL compose instance making edits and then converting things back to normal schema.

Seems extremely sensible, I am working on this, will create PR in sometime

Looks like we have resolved all issues. Thx


AWS collector is picking up us-east-1 as its default region.

After several tests I found out more things on Hygieia. It runs on EC2, CloudWatch and Autoscaling. It takes data from the billing dashboard on CloudWatch. The policy that is attached to the IAM role on Hygieia instance shows this dependency. As of now, our account default region for AWS billing is Virginia which is why it has to go there to get EC2 data.
How we can make our AWS collector to collect data from us-west-2 region

I have tried everything including aws.profiles  & IAM roles but no success. Please help

+1 Facing same issue.
Related issue https://github.com/capitalone/Hygieia/issues/1852#issuecomment-347026887

I made local changes to Hygieia code to get it going (until it is being fixed) for the region I want to connect.
Changes that we need to make it  are present in https://github.com/capitalone/Hygieia/issues/1852#issuecomment-347030067

@yks0000, @rohit0503 - please continue this discussion here: https://github.com/capitalone/Hygieia/issues/1852


kustomize build failed: filter cannot create or delete objects

Describe the bug
Using a strategic merge yaml patch in kustomize build with delete option $patch: delete now causes a kustomize build failed: filter cannot create or delete objects exception in flux. Locally this still works with kustomize build, and it did work in older versions of flux onmodified.
To Reproduce
Create a base folder with some kubernetes resources, reference it in a kustomization.yaml, alongside a strategic merge patch containing one resource to be omitted:
kustomization.yaml
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
bases:
- ../../../../products/base/prometheus/custom-resources
patchesStrategicMerge:
- patch.yaml

patch.yaml
---
$patch: delete
apiVersion: monitoring.coreos.com/v1
kind: PrometheusRule
metadata:
  name: kube2iam.rules
  namespace: monitoring

And point to the folder containing the kustomization.yaml in a Kustomization Resource looking something like this:
apiVersion: kustomize.toolkit.fluxcd.io/v1beta1
kind: Kustomization
metadata:
  name: prometheus-custom-resources
  namespace: flux-system
spec:
  interval: 10m0s
  path: ./patches/preprod/prometheus/custom-resources
  dependsOn:
  - name: prometheus-release
  - name: traefik-release
  prune: true
  sourceRef:
    kind: GitRepository
    name: flux-system
  validation: client

Expected behavior
kustomize build doesn't contain the element, even though it is present in the bases.
Additional context

Kubernetes version: 1.18
Git provider: GitHub
kustomization controller image version (broken): 0.6 - 0.10
kustomization controller image version (definitively still working): 0.5.3

I think this is caused by https://github.com/fluxcd/kustomize-controller/blob/main/controllers/kustomization_generator.go#L335-L356 as we have disabled AllowResourceIdChanges.

Tested this with AllowResourceIdChanges = true and it still breaks, seem that this is a bug in sigs.k8s.io/kustomize/api v0.7.4 and we can't upgrade to latest due to kyaml panics and other breaking change (https://github.com/fluxcd/flux2/issues/918).

Allow Resoure ID Changes? We definitively also use Kustomization features like namespace override and nameSuffix and the likes. I didn't see an issue there yet, though we also still have 0.5.3 kustomization controller in that cluster as well.


Quick adaptation to new pm

Codecov Report
Attention: Patch coverage is 37.64045% with 222 lines in your changes missing coverage. Please review.

Project coverage is 71.36%. Comparing base (a889bb6) to head (c550c42).
Report is 1 commits behind head on main.

Files
Patch %
Lines

src/compiler/optimize.jl
40.81%
116 Missing :warning:

src/compiler/utils.jl
22.22%
77 Missing :warning:

src/compiler.jl
52.45%
29 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1538      +/-   ##
==========================================
- Coverage   74.26%   71.36%   -2.91%
==========================================
  Files          30       30
  Lines       12417    12584     +167
==========================================
- Hits         9221     8980     -241
- Misses       3196     3604     +408

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


panic: send on closed channel on broker disconnect

Hey there. Thanks for this library! I'm having troubles implementing reconnection when the broker can no longer be reached - it kinda works 50% of the times, the other 50% I have this panic:
panic: send on closed channel

goroutine 116 [running]:
github.com/streadway/amqp.(*Connection).shutdown.func1()
        /home/antonio/go/pkg/mod/github.com/streadway/amqp@v0.0.0-20200108173154-1c71cc93ed71/connection.go:395 +0x5bd
sync.(*Once).doSlow(0xc0000ec280, 0xc000108da8)
        /usr/lib/go/src/sync/once.go:66 +0x101
sync.(*Once).Do(0xc0000ec280, 0xc000108da8)
        /usr/lib/go/src/sync/once.go:57 +0x69
github.com/streadway/amqp.(*Connection).shutdown(0xc0000ec280, 0xc00000ea80)
        /home/antonio/go/pkg/mod/github.com/streadway/amqp@v0.0.0-20200108173154-1c71cc93ed71/connection.go:389 +0x92
github.com/streadway/amqp.(*Connection).dispatch0(0xc0000ec280, 0x7ec940, 0xc00000ea20)
        /home/antonio/go/pkg/mod/github.com/streadway/amqp@v0.0.0-20200108173154-1c71cc93ed71/connection.go:451 +0x531
github.com/streadway/amqp.(*Connection).demux(0xc0000ec280, 0x7ec940, 0xc00000ea20)
        /home/antonio/go/pkg/mod/github.com/streadway/amqp@v0.0.0-20200108173154-1c71cc93ed71/connection.go:434 +0x6d
github.com/streadway/amqp.(*Connection).reader(0xc0000ec280, 0x7f40502dcfe8, 0xc000010090)
        /home/antonio/go/pkg/mod/github.com/streadway/amqp@v0.0.0-20200108173154-1c71cc93ed71/connection.go:528 +0x10a
created by github.com/streadway/amqp.Open
        /home/antonio/go/pkg/mod/github.com/streadway/amqp@v0.0.0-20200108173154-1c71cc93ed71/connection.go:233 +0x3eb
exit status 2

What I'm doing is basically have a goroutine ranging over a chan *amqp.Error and every time it receives something it re-initializes the connection, call NotifyClose() on the same chan, then re-init channel, exchange, queues, etc.
From the doc:

NotifyClose registers a listener for close events either initiated by an error accompanying a connection.close method or by a normal shutdown.
On normal shutdowns, the chan will be closed.

apparently the chan sometimes is closed even on non-normal shutdowns.

I was probably doing something weird with channels, I basically rewrote my logic being inspired by this code: https://github.com/isayme/go-amqp-reconnect/blob/master/rabbitmq/rabbitmq.go and the error is gone.
Leaving this as a reference for the future :smile:


Added warning for additional costs related to enabled WVD Insights in azure-monitor.md

Added warning for additional costs related to enabled WVD Insights in azure-monitor.md
By enabling the default settings in the configuration workbook for WVD insights, a small environment of 5-7 nodes can quickly generate x-amount of GBs of data.
A warning should be presented prior to enabling these configuration settings.

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

The issue that's addressed in the PR appears to be addressed in master. We'll close the PR. If there are outstanding issues or questions, please add a comment or create a new pull request. Thanks.


Enum.NONE object

Why do we need the enum.NONE objects like country.NONE? It makes working with the enumeration more cumbersome in code, e.g. checks like  const countries = Object.values(Country).filter(e => e != Country.NONE); to display a dropdown. I realise it's meant to be the default value for this enum type but maybe there is a more elegant way to solve this. Feel free to close this issue if you feel it is nitpicking.

The problem is: when we define
interface user {
    role: Role;
}

and want to instantiate this, we need to pick a value of the Enum.
const user: user = {
    role = "";
}

is invalid.
const user: user = {
    role = Role.STUDENT;
}

is valid.
We could argue to pick "Role.STUDENT" as default, but thats not that nice either.
Another idea is that you use the API enumeration for UI stuff (which  would not contain NONE) and we define a new Enum (similar to the entities) which "extends" the enum with NONE.
When I looked into this last week, I did not find a short and nice way to extend an enumeration, but I can look into this again. I don't want to repeat all the enumeration fields again, but import/extend them instead.

Could we make the user class abstract and force at instantiation to pick a more specified type (e.g. student/admin/lecturer)?
Although, I reckon this could class with the userManagement.getAllUsers()  method as we don't now their subtype yet, right?
IMO querying the types via API each time is less favourable than the original solution.

IMO querying the types via API each time is less favourable than the original solution.

Thats not what I meant. But we could define two enums, one with and one without, so that const countries = Object.values(Country).filter(e => e != Country.NONE); would not be necessary, we could just use the enum instead.
The userEntity object could then still have Country.NONE, but we would not show it in the UI, it would just be a blank value, as this is not a value of the enum.

Although, I reckon this could class with the userManagement.getAllUsers() method as we don't now their subtype yet, right?

Not sure I understand what you mean. The method returns a UserList, which is an object with 3 attributes, a student list, lecturer list and admin list.
However, having a Role.NONE is helpful at a few places, e.g. I instantiate a new APIResponse object for getRole, which always returns a correct role value. If the API call fails (user not found, auth error, whatever), I can return NONE. You could argue that a http statuscode between 400 and 600 would invalidate whatever is in the response, but still, I would feel weird to return "student".

I had a misunderstanding, sorry.
Maybe we could solve the APIResponse problem by marking the object optional (like here https://stackoverflow.com/a/17220359/6516194).


Edit session sequence number not increasing

Testing #161860
Store several Edit Sessions on the same machine. The Edit Sessions view shows the items but all #1.

The number denotes the machine and not the edit session number.


chore: home screen design fixes

Description
Fixes #13110
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Manually
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
// Code coverage diff between base branch:release and head branch: fix/home-screen-design-fixes

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
app/client/src/components/ads/Icon.tsx
99.44 (0)
85 (0)
100 (0)
99.44 (0.01)

/ok-to-test sha=1b26cb6

/ok-to-test sha=0b83095

/ok-to-test sha=ce084e8

/ok-to-test sha=deb7865

/ok-to-test sha=082e9b1

/ok-to-test sha=2891689

/ok-to-test sha=428e7e9