[Bug]: Mod Manager <IP_ADDRESS> Update

Operating System
Windows 10
BG3 Mod Manager Version
<IP_ADDRESS>
BG3 Game Version
Full Release patch 3 hot fix 9
Bug Summary
Whenever I load in my mods and export them to the game, as soon as I get into the game it unselects all the mods from mod manager and tells me that the ones, I had from the previous in game save have now been removed. When I go back into mod manager, they have all been moved to the inactive side even though I had moved them to active and saved the order, as well as exported it to the game before launching. I did uninstall all the mods and re installed them just to be sure, they are all up to date as well. I even restarted a new game, and I thought the issue had been resolved but when I got to a stopping point and closed out of the game and went back in, it did the same thing again. I wasn't having this issue earlier until I downloaded the new BG3 Mod manager <IP_ADDRESS>. I'm not sure if there is something I am doing wrong or what is happening. Thank you!
Links
No response

im having the same issue
any help would be appreciated

I discovered it was a mod i was using, i don't know if i installed it improperly but once i removed it completely the problem vanished.

@HeathenReine  Which mod was it, if I may ask?
having the same issue with a friend.

I discovered it was the gales wizardly updo, I very well could have installed it improperly and that's what caused the issue, but once i removed it the problem went away.


All multivariate distributions should be generic over dimension

#209 changes our API for the multivariate distributions. We'll need to update the other uses of multivariate for consistency.

[x] #254
review relevant multivariate PR's

off the cuff I know about #178 and #176 -> #266

[x] #255
[ ] Wishart and InverseWishart are proposed in #170

Will make sure to mention #170 needs these updates as well. But I'll drop the item from the above list (edited initial post to express that) since all of what's released should consistently be generic over dimension.


display of tabulator in editor

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[x] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
The Spyder editor does not display tabulators right.
What steps reproduce the problem?

open Spyder with default settings

load the attached file into Spyder editor
temp.txt

see it?

What is the expected output? What do you see instead?
A Tab should be EXACTLY 4 spaces wide, as configured in the settings. But in appears more like ~4.5 spaces making lines not aligned
Versions

Spyder version: 3.2.6
Python version: 3.6.5
Qt version: 5.9.5
PyQt version: 5.10.1
Operating System name/version: Ubuntu 18.04 LTS

Dependencies
IPython >=4.0    :  5.5.0 (OK)
cython >=0.21    :  None (NOK)
jedi >=0.9.0     :  0.11.1 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
numpy >=1.7      :  1.13.3 (OK)
pandas >=0.13.1  :  None (NOK)
psutil >=0.3     :  5.4.2 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pyflakes >=0.6.0 :  1.6.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pylint >=0.25    :  1.8.3 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
rope >=0.9.4     :  0.10.5 (OK)
sphinx >=0.6.6   :  1.6.7 (OK)
sympy >=0.7.3    :  None (NOK)

But in appears more like ~4.5 spaces making lines not aligned

Please post a screenshot because I don't understand what you mean by 4.5 spaces.

Sceenshot:

The same file with a different editor (Geany):

I can't reproduce this

so it looks like a problem with the font you choose.

Mixing spaces and tabs is a very bad idea, so I'd be in favor of closing this one. Realistically we also don't have time to invest in fixing this one.
What do you say @bcolsen?

Mixing spaces and tabs is a very bad idea, so I'd be in favor of closing this one.

This is not really a case of mixing spaces and tabs. The bug is there when you just use tabs (also a bad idea) The current way tabs are drawn would make it quite difficult to use the editor. The spacing can be off by 1.5 spaces depending on the font and size(Monospace 12) :

The two sections should be identical. Notice how the guides don't work either. This is a rendering bug that could impact code readability if you use tabs.

Realistically we also don't have time to invest in fixing this one.

This is true, but this bug should be kept as reference to the issue.  I don't use tabs so it's low on my priorities

The bug is there when you just use tabs (also a bad idea)
Well, in my opinion it is a bad idea to use SPACES to indent, but that's a different discussion. Everyone should have the choice of using tabs OR spaces. It is true that mixing spaces and tabs does not make sense, I just used them in the example to make the problem clear.

The point is, that a program editor has to use a monospace font and render it correctly in order to make things line up. But in Spyder editor this does not work when using tabs. So in my opinion this is a SEVERE bug.
I mean, this is a text editor. If this was a MINOR bug, what are your SEVERE bugs that you have to fix first??? Not being able to load a text file?
@bcolsen: Did you find a combination of font and font size so that the editor is usable again?

So in my opinion this is a SEVERE bug.

You're welcome to fix it. I consider this low priority because you're the first person to report it.

You're welcome to fix it.
OK, I will give it a try. Can you give me a hint where to start? Which file is responsible for rendering the editor window content?

For the record, I can reproduce it on Spyder 3.3.0 on Win8.1/Python 3.6.6/Qt 5.6 as well.
I'm not as familiar with the details of the underpinnings as some, but the Editor widget would be one place to start. Since this is a low-level bug, if it ends up being straightforward you could probably justify fixing this on 3.x.

@feichtinger, the fix for this should be available in our 3.3.1 version.

Thanks for investigating this bug. Unfortunately the issue is still not fixed on my system.

I pulled the latest 3.x branch from github (commit: a2c60d714) and used the bootstrap.py script.

This should fix it:
https://github.com/spyder-ide/spyder/pull/7502


Luci device tracker luci-mod-rpc: AttributeError: 'list' object has no attribute 'upper'

I recently started having problems with the Luci Device Tracker (luci-mod-rpc) in Home Assistant.
The error occurs regardless of whether openWRT uses SSL or not.
It runs perfectly before updating luci & homeassistant. Does anyone have any idea what the problem could be?
[LuCI openwrt-23.05 branch (git-24.006.68745-9128656)](https://github.com/openwrt/luci) / [OpenWrt 23.05.2 (r23630-842932a63d)](https://openwrt.org/)

luci-mod-rpc | git-21.020.56896-af422b1
rpcd-mod-luci | 20230123-1

ERROR:
Logger: homeassistant
Source: components/luci/device_tracker.py:94
First occurred: 18:50:21 (5 occurrences)
Last logged: 18:51:01

Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/device_tracker/legacy.py", line 432, in async_device_tracker_scan
    found_devices = await scanner.async_scan_devices()
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/device_tracker/legacy.py", line 952, in async_scan_devices
    return await self.hass.async_add_executor_job(self.scan_devices)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/luci/device_tracker.py", line 67, in scan_devices
    self._update_info()
  File "/usr/src/homeassistant/homeassistant/components/luci/device_tracker.py", line 94, in _update_info
    result = self.router.get_all_connected_devices(only_reachable=True)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/openwrt_luci_rpc/__init__.py", line 42, in get_all_connected_devices
    return self.router.get_all_connected_devices(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/openwrt_luci_rpc/openwrt_luci_rpc.py", line 178, in get_all_connected_devices
    device_entry['hostname'] = utilities.get_hostname_from_dhcp(
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/openwrt_luci_rpc/utilities.py", line 28, in get_hostname_from_dhcp
    host = [x for x in dhcp_result.values()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/openwrt_luci_rpc/utilities.py", line 32, in <listcomp>
    and x['mac'].upper() == mac]
        ^^^^^^^^^^^^^^
AttributeError: 'list' object has no attribute 'upper'

My Config in Homeassistant:
device_tracker:
  - platform: luci
    host: <IP_ADDRESS>
    username: !secret router_user
    password: !secret router_password
    ssl: true
    verify_ssl: false
    interval_seconds: 10
    consider_home: 180

HomeAssistant
Core                          2024.1.3
Supervisor                 2023.12.0
Operating System     11.3

Tough call. I run a similar combination. Still works for me. The python error is specific, however: it thinks that upper is an attribute, not the uppercase function. Maybe dhcp_result.values() is not returning what it thinks it should. i.e. a string.
I don't run 2024 builds yet. I'm unfamiliar with that codebase, so flag an error with them.


nodemailer

| Fixes   #521
Description
The Nodemailer is used to send email to any user. you can use message body to send messages to any user with this.
Type of change

[ x] New project (non-breaking change which adds functionality)

How Has This Been Tested?
Just install dependencies and follow steps given in Readme.md
Tech Stack used:
Checklist:

[ x] My code follows the style guidelines of this project
[x ] I have performed a self-review of my own code
[x ] I have commented my code, particularly in hard-to-understand areas
[x ] I have included a README.md file
[ x] My project follows this structure: Domain Folder/ Project Name Folder/ Project files and Readme.md files
[x ] I have added demonstration in the form of GIF/video file
[x ] I am a GSSoC'22 Participant

@manisha-27  maam ,It is a new and fresh PR containing everything you told to add  in previous PR of Nodemailer, you can merge this.

@manisha-27  maam, I updated readme, now you can check

@manisha-27  maam today is last date, you should merege it


Explore the Management APIs button links to a 404

The link from this button is broken.

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

ID: 1fc1c70b-6ce5-8115-921d-8db80c1d43db
Version Independent ID: 5b033609-afc0-a8e0-4059-89e77bbcd28d
Content: Azure Virtual Machine libraries for Python
Content Source: docs-ref-services/latest/compute.md
Service: compute
Product: azure
Technology: azure
GitHub Login: @lisawong19
Microsoft Alias: ramyar

Hi @tevoinea thanks for reporting this! @msyyc could your route this to the appropriate team?

@tevoinea Thanks for reaching out to us and sharing this feedback. This PR https://github.com/MicrosoftDocs/azure-docs-sdk-python/pull/1543 has been created to fix this issue. It is currently under Review phase.

Awesome, thank you both! Closing the issue since the fix is in PR.


Use Open edX singal hooks instead of manual hooks

Instead of manually connecting signals, streamline the signals to use Open edX plugin system:
Example from CAG: https://github.com/appsembler/course-access-groups/blob/6774dc4cf41959f66e9f11ca6c9fc3c6826a68cf/course_access_groups/apps.py#L25-L39
# course_access_groups/apps.py
PluginSignals.CONFIG: {
    ProjectType.LMS: {
        PluginSignals.RECEIVERS: [
            {
                PluginSignals.RECEIVER_FUNC_NAME: 'on_learner_account_activated',
                PluginSignals.SIGNAL_PATH: 'openedx.core.djangoapps.signals.signals.USER_ACCOUNT_ACTIVATED',
            },
            {
                PluginSignals.RECEIVER_FUNC_NAME: 'on_learner_register',
                PluginSignals.SIGNAL_PATH: 'openedx.core.djangoapps.user_authn.views.register.REGISTER_USER',
            },
        ],
    }
}

Closed by #8. No signals is needed anymore.


Adding snippets

Adding commonly used snippets to project.

Thanks! I'll experiment with this in a few days once I have some time. Generally looks good but I may opt to simplify some of the snippets for more general usage.

Merged and deployed to v1.2.0 of the extension.
Thanks!


HTML editing support for Android - ICS browsers

This is regarding the html5test.com site's content editable test case.
This test case currently puts all Android browsers in the blocked list and thus even though Android browsers (the ICS flavor) support contenteditable property and do launch the IME to enable the user to edit the content, they still remain in the blocked list and hence are unable to pass this test case.
Because of this, we are losing out on 17 points.
The blocked list adds all mobile and tablet user-agents and as per your test site any Android user-agent shall either be classified as a mobile or a tablet user-agent.
Request you to please verify the contenteditable support (on ICS browser) and change the test case accordingly so that ICS based Android browsers are able to pass the test cases.

Thank-you for the verification.


Feature Request: SMS integration option

The K9 app should allow as a opt-in, to integrate with SMS/messaging.
So it "replaces the default SMS app on phone".
Also another feature, related to this, is two options, one to select to:
"Send everything as Email"
"Default"
And following (Both greyed out when "Default" is set on above option):
"Suffix to use when sending SMS as Email" [inputbox]
Account to use for sending SMS: [select popup that lists all IMAP/POP3 accounts configured]
In the inputbox for suffix, you can write anything, that will be added as suffix when sending a message to a receiver that does only contain digits and +.
So if I write<EMAIL_ADDRESS>as "Suffix to use when sending SMS as Email", and have "Send everything as Email" selected.
If I then compose a message to lets say<PHONE_NUMBER>, then it would transmit the message to<EMAIL_ADDRESS>via the selected email account.
(But suffix can also be left blank if your mail provider has builtin SMS functionality, like "RCPT TO: <0707555555> would send a SMS to<PHONE_NUMBER>")
(If receiver contains anything other than digits and +, it should be considered a email message and be sent as-is)
If "Default" is selected, receiver that does only contain digits and + should be considered SMS, and be sent via SIM card. Subject line is then ignored. cc and bcc is considered a multi-receiver SMS.
Regardless of option, if its the default message app, it should receive all SMS messages in a special inbox called "SMS". Sent messages should only appear in inbox "SMS" if you have "Default" selected, else any sent messages should land in the "Sent" box of the chosen IMAP/POP3 account.

We're focusing on writing a decent email client. Support for SMS is not something we want to add to the app.


Changes To Fix Rerouting on TLS Encrypted Connections

Needed to make the following changes to support (in connection.coffee unless otherwise stated)

New event needed for rerouting called as such "rerouting"
Replaced previous "end" event on rerouting with aforementioned new event, "rerouting"
Initialize tlsNegotiationComplete as false on start of SENT_TLSSSLNEGOTIATION
Add server to serverName property in Login7Packet in the event the servername differs from the redirected hostname address (eg connect to serverA.net -> redirected to serverB.net -> server at serverB.net expects serverA's name for the server)
Fixed up the integration/connection-test.coffee test to account for redirection on an encrypted server requiring 2 separate "encrypt" events, resulting in 3 additional successes in addition to the original expectation of 5 success

Changes not included in this commit

Need to update the Connection api webpage to document the new rerouting event

FYI - Changes related to issues #222 and #223

Published as 1.8.1. Thank you.


In Chrome if user selects the "Remember my choice for all links of this type" check box the fail callback always gets called.

Hi there, first off this js file is really useful, great job on it. I have run across a problem though now that I have implemented it my self. Details below:
Chrome ver. 42.0.2311.90 m / Windows
If a user selects the "Remember my choice for all links of this type" check box in the browser default protocol handler alert from there on when ever the protocolCheck method is called it will always run the fail callback as well as opening the app associated with the protocol handler. I believe this is due to the time out checking on if that dialog opens or not after a certain amount of time, but have not had the opportunity to dig further into the issue.

Hi spstratis, thanks for the feedback!
You are absolutely right, this Chrome implementation is based on the assumption that there will be app loaded by the custom protocol handler and thus will steal the focus from the window. Could you tell me what the behavior of your protocol handler in this case?

For me the protocol handler is opening a windows app. It opens it successfully yet the fail call back still fires off as if focus is not being put on to the app the is opening.

That sounds strange. Could you do a bit more debugging to see whether the onBlur method is being triggered at all? One possibility is that the app is slow to load (need more than a second) or the loaded app doesn't take the focus at all.

Yea I was thinking of adding additional time to the timeout and checking to see if that makes a difference.  I'll let you know the result of some additional debugging.

So far it looks like setting the time out to 2000 takes care of the issue.

You were right the loaded app was not taking focus. I notice from time to time it will still hit the failure state but I believe that may be due to propagation on the button click to launch the app. You should feel free to close this defect. Thanks for responding back!

Hi Ismail Habib
we have exactly the same problem, but we can't find a way around it. even if we increase the timeout, the behaviour is the same. The app is launched and the call back fail function. Hope i didn't misunderstood somthing crucial.


Successfully text shown even though it fails to delete an environment

What happened:
As shown below, I could see a doubled popup that confused me.

What you expected to happen:
How to reproduce it:
Environment:

piped version:
control-plane version:
Others:

@cakecatz Could you take a look?

@nakabonne Sure, I'll fix it!

Thank you!


Access denied

Hi,
I'm unable to deploy using https://production-ready-prestashop.s3.ca-central-1.amazonaws.com/versions/latest/production_ready_prestashop.yaml
Access denied.
Regards

Should be fixed now, thanks for reporting.


The parser is overly permissive of values types

For example, (ctype:specifier-ctype '(function ((values integer)))) returns a ctype with a values type in the lambda list, which is nonsensical. It should signal a syntax error.
It would probably be good to have separate specifier-ctype and values-specifier-ctype functions, where the former errs on values types and the latter coerces everything to a values type, and then call them appropriately. But I'm not sure how that would work with the customization, i.e. cons-specifier-ctype being generic.

I could imagine there being another parameter values to cons-specifier-ctype that is either of type null or t.
But I don't feel having a separate specifier-ctype and values-specifier-ctype is a good way forward, or I don't get the motivation for doing this. I am unable to see what goes wrong/difficult with having just specifier-ctype and do the appropriate checks sticking as close to CLHS/implementations as possible, disallowing types like (function ((values integer))). (May be stick to sb-kernel:specifier-type unless it becomes unreasonable?)

specifier-ctype and values-specifier-ctype would be how those checks are done. For example the function type parser would call specifier-ctype for the parameter types, which would signal an error if a values type is written there, and then would call values-specifier-ctype for the return value type, which would accept values or non-values but coerce the latter.
I'm not sure I understand what you mean. This would be fixing a CLHS incompatibility in ctype, and it is roughly what SBCL does in its own system. You can see here - arg types are parsed with sb-kernel:specifier-type (https://github.com/sbcl/sbcl/blob/ab6658fec390036865dabb4c77722f45d48e7700/src/code/type.lisp#L622) whereas the result type is parsed with basic-parse-typespec and coerce-to-avlues (https://github.com/sbcl/sbcl/blob/ab6658fec390036865dabb4c77722f45d48e7700/src/code/type.lisp#L680), in a way similar to the sb-kernel:values-specifier-type function here https://github.com/sbcl/sbcl/blob/ab6658fec390036865dabb4c77722f45d48e7700/src/code/type.lisp#L1582-L1589

Okay, TIL about sb-kernel:values-specifier-type, and yes given the following behavior, things make sense:
CL-USER> (sb-kernel:values-specifier-type '(values &rest t))
#<SB-KERNEL:NAMED-TYPE *>
CL-USER> (sb-kernel:specifier-type '(values &rest t))
; Evaluation aborted on #<SIMPLE-ERROR #<SB-FORMAT::FMT-CONTROL "VALUES type illegal in this context:~% ~/SB-IMPL:PRINT-TYPE-SPECIFIER/"> {102D3A3963}>.

If the actual work is being done by sb-kernel::basic-parse-typespec, then sb-kernel:specifier-type and ctype:specifier-ctype are only needed so long as we want to provide the guarantee that the return value is a non-values type. My previous understanding of specifier-ctype (and current semantics in the code?) was that it was meant as the entry point into the parser. Instead that role could be subsumed by something equivalent to basic-parse-typespec - so basic-parse-typespec could serve as the entry point into the parser, while specifier-ctype and values-specifier-ctype could serve as wrappers around it; in essence, the current specifier-ctype could be renamed to basic-parse-typespec or equivalent, and a new specifier-ctype that ensures that its return value is not cvalues could be defined. But I don't see the need to touch cons-specifier-ctype in this way of doing things.
One other alternative to implementing cons-specifier-ctype itself as a generic function could be to keep cons-specifier-ctype as a normal non-generic function, and instead of using etypecase inside it, use typecase, and make the otherwise case dispatch on a generic-function say nonstandard-cons-specifier-ctype. (Don't recall ATM if there were any downsides to this way, or I just overlooked this solution back then.)

yeah, shouldn't need to change cons-specifier-ctype if it works like sbcl


As of React 18.x being used, importing react on files is not needed

In previous versions, importing React on tsx files was mandatory for it to transform the file, however in the latest versions that ended up changing and now it automatically transforms the file without even being imported. Here's a quote from the React blog announcing the changes for React 17.x:
"With the new transform, you can use JSX without importing React."
However since we will not import react globally to the file, hooks must be imported in some way, and the best one is being destructured from the react package, therefore some changes need to be made into some files that use hooks.
import { useState } from "react";

References:
React - Introducing the New JSX Transform

Closing as there's only one occurrence, in the main.tsx file.
This is perfectly fine.


[SUPPORT] Flink-Hudi - Upsert into the same Hudi table via two different Flink pipelines (stream and batch)

Describe the problem you faced
Hi,
My team wants to build Flink pipelines to generate financial report and save the report results into a Hudi COW table.
The data sources for the report consist of two types of data - snapshot and incremental data. To have a complete report we need to ingest both snapshot and incremental data, and for that we are thinking about running two Flink jobs against the same Hudi table - batch and incremental - sequentially where the batch job processes all snapshot data up to the current time, and the stream job continuously processes new incremental data.
According to Hudi's documentation, it uses Flink state to store index information for the records it has processed and relies on that information to perform upsert correctly.  My question is that the stream job doesn't have access to the state information of the batch job, would Hudi in the stream job be able to perform upsert operations to update records that were previously ingested via the batch job correctly? If not, do you have any recommendation on how we can set up the Flink-Hudi workflow to meet our use case?
To Reproduce
Environment Description

EMR emr-6.15.0, Hudi 0.14.1, Flink 1.17.1, Hadoop 3.3.6, Hive 3.1.3, Zeppelin 0.10.1

You can use the bulk_insert for history data and regular upsert for incrmental streaming ingestion.
Note that when you choose flink_state index instead of bucket index, the incremental streaming pipeline needs to enable the index bootstrap through index.bootstrap.enabled option.

@danny0405 thanks for replying.
I tried enabling index.bootstrap.enabled and noticed that there was a index_bootstrap task created in my Flink pipeline accordingly. Initially I thought the boot_strap task would complete and reach the finished state after indexing the existing table, but it continued to run without releasing its resource. Is this an expected behavior that the bootstrap task will run continuously?

Is this an expected behavior that the bootstrap task will run continuously?

yes, and after a checkpoint succeed, you can disable the bootstrap by setting up index.bootstrap.enabled as false.

Would the steps be like: checkpoint succeeds -> cancel the Flink job -> restart the Flink job from the checkpoint with bootstrap disabled?
Currently I am running Flink job via Flink SQL on Zeppelin notebook, do you know how I can perform the steps above?

Currently I am running Flink job via Flink SQL on Zeppelin notebook

You can disable with Flink SQL hint: /*+ OPTIONS('index.bootstrap.enabled'='false')*/.

@danny0405 Thanks for the answer. The SQL hint worked for me for disabling index bootstrap.

Close it now, feel free to reopen it if you still think it is an issue.

Would the checkpoint of job 3 contains all index information retrieved from the index bootstrap process in job 2? Asking this as I noticed a significant size differences between the checkpoint of job 2 and job 3. (500GB in job 2 vs < 50GB in job 3)

yes, one successful checkpoint indicates the bootstrap has finished.

If job 3 fails and I need to start a job 4 using job 3's latest checkpoint, do I need to have index bootstrap enabled?

No need to do that.

@danny0405
Is it expected that the checkpoint size of bucket_assigner operator changes significantly from 500GB in the job 2 to less than 50GB in the job 3 mentioned above?
The Hudi sink table has 9.6 billion records with around 570.3 GB, and is currently partition by a time attribute so the size of partition is expected to dynamically grow to store data for a even data distribution. Would my use case benefited by using Bucket index compared to Flink index?

In addition, I found some duplicates written by my bulk_insert batch job 1 and upsert stream job 2 (the one that had index bootstrap enabled).
For bulk_insert batch job, it had write.precombine set to true so there shouldn't be any duplicates in the result table?
For upsert stream job, it had write.precombine set to true and index bootstrap task had parallelism set to 480. I found this previous issue https://github.com/apache/hudi/issues/4881 which suggests duplicates can happen when index bootstrap task parallelism > 1. Is that still the case in Hudi 0.14.1? The table that needs to be index bootstrapped is large so I am not sure if setting parallelism to 1 would work.


GFM 0.2.0 blocked by Carbon Black security software

Hi there and thanks for the GFM!
It looks like v0.2.0 is being flagged as a security risk by the Carbon Black corporate security software:

Version 0.1.14 worked fine, so perhaps something has changed in the way how the JXBrowser is being installed?
Best,
Jan

I did some more digging and it looks like making the installation location of jxbrowser configurable would solve the problem.
Context: companies using Carbon Black often specify a location on the file system where developers can install their tools (like "C:\Projects", "C:\Tools", etc.). If GFM allowed us to install the jxbrowser at such custom location it would address the problem.

I'll add an option for dynamic jxbrowser download in a next release. It will 'fix' #32 and will hopefully fix this issue as well. You can expect new version the end of this weekend.

Oh cool, that sounds great! Thanks for putting so much effort into this 👍


Raise error when guard fails

I'm not sure this falls inline with your design but I was expecting a guard method to raise an exception when it failed, much like invalid transitions raise errors.
Personally, I don't like a mixture of functionality, either fail hard or use the return value. With guards I need to check the return value (I think) and with invalid state transitions I need to rescue an exception.
Since guards can be multiple response, an exception might be the best solution since it can carry with it which guards failed. This is especially useful for sharing failure state with the end user.

I suppose I can work around this by raising an exception in the guard. That way I know what failed but it seems hacky. Even the event_failed callback doesn't seem to get passed enough context to know why it failed.

I agree with you. Guards should raise just like invalid transitions.
I'll put this on my xmas list ;)

I did give this some thoughts and I'm not so sure anymore that this would be the right thing to do.
First of all, this:

but I was expecting a guard method to raise an exception when it failed, much like invalid transitions raise errors.

is not really true - transitions merely re-raises an exception that ActiveRecord would have thrown anyway as you can see here, so using exceptions is not something that is common in transitions.
Secondly I'm concerned what this would do to readability and maintenance. Just imagine you have 6 transitions and 3 of them have guards. In this case you'd have to:
1.) Treat 3 transitions conceptually as potentially exceptional, the other 3 not which feels weird.
2.) Treat those 3 different in source code. Meaning 3 out of 6 might have rescue blocks while the others don't. I can imagine this being confusing. And in case you remove a guard you have to potentially adapt all places where this transition is used as well.
3.) You still can make guards exceptional in your user code which keeps it out of transitions - so basically the way you do it right now. I woulnd't actually call this a workaround what you're doing, but rather a legit way.
What do you think?

@troessner
Thanks for your careful thought. I think that it makes sense. But I think a small amount more documentation in the README might help clarify how to make use of a guard. Maybe even example code showing the boolean response from the event.
I'm currently raising a specific exception from within the guard which I think rolls everything back so that the state transition never happens. This allows me to handle this state transition failure in a specific way. I'm not positive that this would decrease legibility if it were to be a transitions exception though. Here's what I'm thinking it might look like:
class Model < ActiveRecord::Base

  include ActiveModel::Transitions
  state_machine do
    state :first
    state :second

    event :promote, from: :first, to: :second, guard: :will_fail
  end

  def will_fail
    false
  end

  def tester
    self.promote!
  rescue Transitions::GuardedTransition => e
    puts e.guards # => [:will_fail]
  end
end

Another possibility would be to allow me to define a hash of guards with exception classes, maybe something like:
event :promote, from: :first, to: :second, guard: { will_fail: MyOwnErrorClass }

I guess that would be a bigger change but it would allow for flexibility :smile: either way, if raising an error from within the guard is a good way to do it, i'll just do that.

But I think a small amount more documentation in the README might help clarify how to make use of a guard.

Excellent point. Let me think about a good example, then I'll create a pull request for our README and you can review if the example would have helped before you started to wonder why guards work the way they work :)

Merged #156 - closing this one.


Kubeflow could not be enabled:  (NotFound): mutatingwebhookconfigurations.admissionregistration.k8s.io "katib-mutating-webhook-config"

I have installed microk8s on Windows using the installer. I have enabled some services with microk8s enable dashboard dns registry. But when I try to enable  kubeflow I get the following error:
C:\Users\biogeek>microk8s enable kubeflow
Enabling dns...
Enabling storage...
Enabling dashboard...
Enabling ingress...
Enabling metallb:<IP_ADDRESS>-<IP_ADDRESS>...
Waiting for DNS and storage plugins to finish setting up
Deploying Kubeflow...
Kubeflow deployed.
Waiting for operator pods to become ready.
Waited 0s for operator pods to come up, 34 remaining.
Waited 15s for operator pods to come up, 34 remaining.
[...]
Waited 390s for operator pods to come up, 2 remaining.
Operator pods ready.
Waiting for service pods to become ready.
Kubeflow could not be enabled:
Error from server (NotFound): mutatingwebhookconfigurations.admissionregistration.k8s.io "katib-mutating-webhook-config" not found
Error from server (NotFound): validatingwebhookconfigurations.admissionregistration.k8s.io "katib-validating-webhook-config" not found

Command '('microk8s-kubectl.wrapper', 'delete', 'mutatingwebhookconfigurations/katib-mutating-webhook-config', 'validatingwebhookconfigurations/katib-validating-webhook-config')' returned non-zero exit status 1
Failed to enable kubeflow
An error occurred when trying to execute 'sudo microk8s.enable kubeflow' with 'multipass': returned exit code 1.

(Error is similar this Github issue from 2 weeks ago, but which hasn't had any response yet.)
Output of microk8s status --wait-ready:
microk8s is running
addons:
dashboard: enabled
dns: enabled
ingress: enabled
kubeflow: enabled
metallb: enabled
storage: enabled
cilium: disabled
fluentd: disabled
gpu: disabled
helm: disabled
helm3: disabled
istio: disabled
jaeger: disabled
knative: disabled
linkerd: disabled
metrics-server: disabled
prometheus: disabled
rbac: disabled
registry: disabled

When I run microk8s inspect to troubleshoot the issue, it generates a logfile at /var/snap/microk8s/1702/... although I am on Windows!?
C:\Users\biogeek>microk8s inspect
Inspecting Certificates
Inspecting services
  Service snap.microk8s.daemon-cluster-agent is running
  Service snap.microk8s.daemon-flanneld is running
  Service snap.microk8s.daemon-containerd is running
  Service snap.microk8s.daemon-apiserver is running
  Service snap.microk8s.daemon-apiserver-kicker is running
  Service snap.microk8s.daemon-proxy is running
  Service snap.microk8s.daemon-kubelet is running
  Service snap.microk8s.daemon-scheduler is running
  Service snap.microk8s.daemon-controller-manager is running
  Service snap.microk8s.daemon-etcd is running
  Copy service arguments to the final report tarball
Inspecting AppArmor configuration
Gathering system information
  Copy processes list to the final report tarball
  Copy snap list to the final report tarball
  Copy VM name (or none) to the final report tarball
  Copy disk usage information to the final report tarball
  Copy memory usage information to the final report tarball
  Copy server uptime to the final report tarball
  Copy current linux distribution to the final report tarball
  Copy openSSL information to the final report tarball
  Copy network configuration to the final report tarball
Inspecting kubernetes cluster
  Inspect kubernetes cluster

Building the report tarball
  Report tarball is at /var/snap/microk8s/1702/inspection-report-20201123_042132.tar.gz

How can I resolve this and successfully install Kubeflow on Windows using Microk8s?

@BioGeek I am not sure about the window's microk8s.
In ubuntu, however, I switched from 1.19/stable to 1.20/edge version of microk8s and it worked.
sudo snap install microk8s --classic --channel=1.20/edge

I get the same issue. Tried out to reinstall Ubuntu 20, reinstall MicroK8s with a newer version 1.20 but nothing works.


nixos: use supportedSystems instead of hardcoded list for netboot

The default value for the supportedSystems argument is identical to the hardcoded list used for defining netboot,
but using the argument allows to build netboot images for other platforms without modifying release.nix.

Motivation for this change
See above.
Things done
nix build netboot.i686-linux -f ./nixos/release.nix --arg supportedSystems '[ "i686-linux" ]'
This should not break any dependencies on that file, but please test.

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Good idea! I bet that list dates back to when x86_32 was in the supported systems list.

I don't know anything about this part of NixOS.


Friendly-fire powerup

SWEPs are too complicated for me. 🤣
Anyways, here's an attempt at implementing the friendly fire powerup. Hunters don't get any notification when it turns on/off, but maybe that's actually desirable...

+1 for no hunter notification being a good thing 😈

Seconded on the no hunter notification XD


Is it allowed for me to deploy resources on the workload node before deploying Helm?

User Story
I would like to define some configuration resources before deploying Helm.
Detailed Description
When I deploy the config/samples/calico-cni.yaml. Tigera Operator allows you to define a ConfigMap named "kubernetes-services-endpoint" to modify the environment variables of Tigera Operator. However, when deploying Helm directly, this ConfigMap is not generated automatically. Therefore, you have to define it in the workload cluster first. Is there a better way for CAAPH to achieve this?
Anything else you would like to add:
For example, adding an optionsTemplate field to define the desired configuration resources, and deploying the resources defined in optionsTemplate before deploying the Helm resources.
apiVersion: addons.cluster.x-k8s.io/v1alpha1
kind: HelmChartProxy
metadata:
  name: calico-cni
spec:
  clusterSelector:
  # Target workload clusters with specific labels.
  #  matchLabels:
  #    calicoCNI: enabled
  # Target all workload clusters.
    matchLabels: {}
  releaseName: calico
  repoURL: https://docs.tigera.io/calico/charts
  chartName: tigera-operator
  namespace: kube-system
  valuesTemplate: |
    installation:
      cni:
        type: Calico
        ipam:
          type: HostLocal
      calicoNetwork:
        bgp: Disabled
        mtu: 1350
        ipPools:{{range $i, $cidr := .Cluster.spec.clusterNetwork.pods.cidrBlocks }}
        - cidr: {{ $cidr }}
          encapsulation: None
          natOutgoing: Enabled
          nodeSelector: all(){{end}}
  optionsTemplate: |
    apiVersion: v1
    data:
      KUBERNETES_SERVICE_HOST: example.com
      KUBERNETES_SERVICE_PORT: "6443"
    kind: ConfigMap
    metadata:
      name: kubernetes-services-endpoint
      namespace: kube-system

/kind feature

Thanks for the feature request! Just so I understand correctly: when you're installing Calico (without CAAPH), you would create your ConfigMap and then run helm install. And when using CAAPH, you'd like to specify that the ConfigMap should be created first. Is that correct?

Yes, that's correct. Additionally, I encounter a similar issue when installing the autoscaler. I have to first install a secret containing the master-kubeconfig in the workload cluster before I can use the plugin to install the autoscaler on the master cluster.

Gotcha, there's currently not a way to specify that. I think it would make sense to introduce some kind of "hook" system for creating resources before/after and waiting until they're ready.
@dtzar WDYT?

@littleBlackHouse - can you use a helm dependency with the dependency being a file of your configmap?

Sure, I can achieve that by creating a custom chart. Thank you.

/close


fix: handle errors in closures

Same as https://github.com/archtechx/jobpipeline/pull/16 but for 2.x

@stancl here you go 😊


MAINT: Migrate style checks to Azure

Trying to:

Load balance -- Azure backs up less than Trivis, and
Make style checks easier to parse.

This moves the style checks to Azure, in separate steps so they should hopefully show up nicely in separate statuses/prompts.

Style checks ran in 3m on Azure


.return and .return-rw methods not documented

There looks to be a .return (and .return-rw) methods provided by Mu, but I'm not seeing it documented:
$ perl6 -e 'sub f($a) { .return with $a; return "something undef-y" }; say f "foo"; say f Nil'
foo
something undef-y

$ perl6 -e 'Mu.^can("return")[0].signature.say'
(Mu $: | is raw)

fixed with https://github.com/perl6/doc/commit/2196187eef


Update demo env

Resolves https://github.com/eclipse-pass/main/issues/346
Local validation before updating eclipse-pass.demo env
Updates

pass-core
pass-auth
pass-ui
sample data loader
proxy

Manual Testing

Run ./demo.sh up -d && ./demo.sh logs -f to bring up the local demo env
Wait until pass-core shows a log message saying that it has started up:
[main] [Pass, ] INFO  org.eclipse.pass.main.Main.logStarted - Started Main in 8.677 seconds (JVM running for 9.406) At this point, pass-core has initialized the DB.
Run ./demo.sh up loader to (re)run the sample data loader, which should succeed and exit with code 0
In your browser, navigate to https://pass.local
Login and poke around the PASS UI

On the surface this looks good.  I'll test it on our EF servers and then look to update those env variables for *.eclipse-pass.org deployments.

That would be helpful, thanks. If you're looking into that, I'll move on to fixing the bad service mocks that will be pushed in a separate PR (it'll be an image update only, no config changes)

The app launched as expected.

I updated the URLs (locally) to be demo.eclipse-pass.org and it seemed to progress.  I am seeing issues, but they are probably expected.

If we get this merged then I'll update the eclipse-pass configs to get us that one step closer


JWK private key

I have been using your code as a reference and I tried to use it but every time I get this error:
The request lacked necessary authorization to be completed. Please see the certificate authority logs for more info.

And on the server side it says:
authority.Authorize: authority.authorizeSign: authority.authorizeToken: provisioner not found or invalid audience (https://***:8277/1.0/sign)

I did not change the audience part of the jwt_body
One another thing that made me wonder was in your nodejs code I saw this:
https://github.com/smallstep/clients/blob/a175a9cd3072859d4d3053b34a7596b746e4f8a8/node/signer.js#L111
I don't quite understand why the sender (client) needs the CA's private key JWK.

Hi!
The client needs the JWK in order to authenticate its request with the CA.
The client uses the JWK to sign a base64-encoded one-time-use CA authentication token (which is a JWT).
This is just one way of authenticating a certificate request with the CA—using the JWK provisioner.
The step client also does this for step ca token or step ca certificate commands, when using a JWK provisioner.
The difference is that, for usability, step reads an encrypted JWK from the /provisioners endpoint of the CA, and then asks you for the JWK password in order to decrypt it in memory and then sign the CA authentication token.

I've updated the main README for this repo with a "you will need section" that describes how to get and decrypt the JWK.

Amazing, thanks for the help.
Just one thing about "You will need" part of README, I don't see the decrypt subcommand in "step crypto jwk".
https://smallstep.com/docs/step-cli/reference/crypto/jwk#commands

I think you meant jwe in the cli than jwk

good catch, thank you!


Dynamic loading of tiles; @concord-consortium/diagram-view npm module

I tried this out locally and I have 2 new questions:
Question 1
I see that it creates separate js files for:

react-components
slate-editor
jsxgraph
diagram-view
vendor

And slate-editor, vendor, and react-components are referenced in the index.html generated by the HtmlWebpackPlugin.
How does Webpack decide what goes in the index.html and what is only loaded when needed?
Is it because of the imports?
Basically it splits everything up and then figures out which things are referenced by static imports, and those things are added to the index.html. And then everything else left out and will get loaded when the dynamic import is called which references it.
Question 2
I see the names used in the dynamic import annotation like webpackChunkName: "Diagram". Because the generated files are not based directly on those imports, do those chunk names have value?
Perhaps they help you analyze the size of the chunks?

So, webpack makes it fairly easy to do two things: automatically generate seemingly random bundles with random names or to carefully control the bundles that are generated and give them reasonable names. After playing around with the former I ended up opting for the latter. I presume it's possible to tweak this even further but I figured our primary goal was met once the diagram-view and jsxgraph libraries were being dynamically loaded so I decided to stop there.
If I were going to spend more time on it I'd like to understand why slate-editor is being pulled into the main bundle but the documentation on this stuff is severely lacking and and it basically just requires a lot of trial-and-error and guesswork.

I see the names used in the dynamic import annotation like webpackChunkName: "Diagram". Because the generated files are not based directly on those imports, do those chunk names have value?
Perhaps they help you analyze the size of the chunks?

They're essentially a suggestion to Webpack, which it ignores if it doesn't find them helpful. In this case, I think my naming of the chunks overrides the naming based on those comments. The unfortunate thing is that if I don't override the names and let Webpack do its default thing, then the chunks that get those names are small/insignificant/irrelevant ones and all the chunks that matter are just named index.xhashx.js. So in this particular instance, those comments probably aren't doing any good but in an ideal world chunks that are only used with the Diagram tile would be named/labeled in a way that made that clear, and so on for the other tiles. Manually overriding the naming to make sure that the largest libraries were named appropriately so that we can ascertain whether or not they're loaded when they're not needed seemed like the best option for now.


DataOps, not DevOps

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2549
Originally created by @DouweM on 2021-01-30 01:24:59

It's been pointed out to me that “DataOps era” would be better than “DevOps era”: https://www.reddit.com/r/dataengineering/comments/l863az/meltano_elt_for_the_devops_era_open_source/

View 1 previous comment from the original issue on GitLab


Our github page

Hi,
I noticed that we don't have a main github webpage yet for our organization. I think we should create one with a brief summary of our mission and a list of all upcoming workshops at UM. I am happy to get this started unless someone else wants to?
-Michelle

Thanks - go for it!

Michelle: did you or anyone else create a main github page for our organization? Thinking about where to post a code of conduct for an R User listserv, that could be a good option.

Nope I never got around to it and I'm pretty sure nobody else has either
On Friday, June 17, 2016, duffymeg<EMAIL_ADDRESS>wrote:

Michelle: did you or anyone else create a main github page for our
organization? Thinking about where to post a code of conduct for an R User
listserv, that could be a good option.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/UMSWC/Administration/issues/10#issuecomment-226842706,
or mute the thread
https://github.com/notifications/unsubscribe/AGG9HPS9RIaL22ntu2uAYT_PTEg8fr-jks5qMuSAgaJpZM4F9A4-
.

It would be nice to have a main github.io site up soon--I'm happy to help, Michelle, although I'm still a beginner with GitHub pages!
I've been thinking about this lately because I've had several grad students email me asking for help with Python, OpenRefine, and text mining, and I'd love to be able to direct them to our main UMSWC presence (especially as a place to watch for upcoming workshops). For most people I talk to (beginners) it doesn't feel appropriate to tell them to join the MCommunity email group, as a lot of it is administrative stuff that isn't relevant.

Any progress on this?  @alixk @pschloss ?
Anyone want to be assigned to this issue and spearhead?


fixed: Not responsive in Mobile Layout|| Issue: #808

I made changes in index.html which is twitter logo (previously not there) and
Github badge page responsive and also fixed achievements list table overlay issues
Landing page

Githubbadge Page

The reason for too many changes was because of the early problem while i
make the pull request was merge conflicts...
So i make a new branch and copy the whole code( which includes gh-pages
code + mine new updates in it) into new branch "responsive"
On Fri, 18 Oct, 2024, 3:59 am Sanjay Viswanathan, @.***>
wrote:

@.**** requested changes on this pull request.
there is many file changed here.
—
Reply to this email directly, view it on GitHub
https://github.com/recodehive/awesome-github-profiles/pull/901#pullrequestreview-2376431407,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5SKIPOJ5FDBD4SVIZU6Q7DZ4A223AVCNFSM6AAAAABQEM2LEGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNZWGQZTCNBQG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


RuntimeConfig explicit typing not working after adding nuxt security module

Environment
npx nuxi info
Working directory: /home/divagueame/web/nuxt/runtime                                                                                10:49:53 PM
Nuxt project info:                                                                                10:49:53 PM

------------------------------
- Operating System: Linux
- Node Version:     v20.17.0
- Nuxt Version:     3.14.1592
- CLI Version:      3.17.1
- Nitro Version:    2.10.4
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      default
- Runtime Modules<EMAIL_ADDRESS>- Build Modules:    -
------------------------------

Nuxt Security Version
2.1.5
Default setup used?
Yes, the bug happens even if the security option is not customized
Security options
// https://nuxt.com/docs/api/configuration/nuxt-config
export default defineNuxtConfig({
  compatibilityDate: "2024-11-01",
  devtools: { enabled: true },

  runtimeConfig: {
    apiSecret: "", // can be overridden by NUXT_API_SECRET environment variable
    public: {
      apiBase: "", // can be overridden by NUXT_PUBLIC_API_BASE environment variable
    },
  },

  modules: ["nuxt-security"],
});

Reproduction
https://github.com/divagueame/runtime-config-demo
Similar issue was discussed here.  https://github.com/nuxt-modules/security/issues/544
Description
I want to type the runtime config like explained here:
https://nuxt.com/docs/guide/going-further/runtime-config#typing-runtime-config
On a new project, the steps explained on the docs work fine and the types in 'index.d.ts' related to PublicRuntimeConfig.
When I add nuxt security module, this types are not working anymore.
declare module "nuxt/schema" {
  interface RuntimeConfig {
    apiSecret: string;
  }
  interface PublicRuntimeConfig {
    apiBase: string | number;
  }
}
// It is always important to ensure you import/export something when augmenting a type
export {};

Commit 2 - Manually typing works fine and types from 'index.d.ts' are infered as expected

Commit 3 - Afer adding nuxt-security module, manual types are not working anymore

Additional context
nuxt.config.ts
// https://nuxt.com/docs/api/configuration/nuxt-config
export default defineNuxtConfig({
  compatibilityDate: "2024-11-01",
  devtools: { enabled: true },

  runtimeConfig: {
    apiSecret: "", // can be overridden by NUXT_API_SECRET environment variable
    public: {
      apiBase: "", // can be overridden by NUXT_PUBLIC_API_BASE environment variable
    },
  },

  modules: ["nuxt-security"],
});

index.d.ts
declare module "nuxt/schema" {
  interface RuntimeConfig {
    apiSecret: string;
  }
  interface PublicRuntimeConfig {
    apiBase: string | number;
  }
}
// It is always important to ensure you import/export something when augmenting a type
export {};

Logs

Hi @divagueame
Is the issue that the type of apiBase is string where it should be string | number ?

Hi @divagueame Is the issue that the type of apiBase is string where it should be string | number ?

Correct.
Another example, here it should be the union:
interface PublicRuntimeConfig {
apiBase: "meow" | "woof";
}

When i add nuxt security module:

Hi @divagueame
I'm using your Github repro and I don't understand where Nuxt Security is involved.
I can see the types being overwritten, that's right. But on my end the behavior is exactly the same whether or not Nuxt Security is installed.
Can you elaborate a bit more and explain which setup I need to have in order to explicitly demonstrate that the issue is linked to Nuxt Security ?
Thanks

@vejja Thanks for taking a look into this.
I think you can see the issue if you check my repo on the second commit:
git clone https://github.com/divagueame/runtime-config-demo.git
cd runtime-config-demo
git checkout bcd56dc0c3fc4391cc1b57a3facaf44529115151
npm install

Go to plugins/config.ts and check if the type is a union (it should be). Here on my side:

Then run
npx nuxi@latest module add security
and check again plugins/config.ts, now, on my side,  now it's just typed as string, but I expected it still to be a union (string | number).

Maybe it's the issue does not stem on nuxt security itself, that I don't know but it seems to occur when I add it. I had a similar issue on a bigger project and removing nuxt security was the toggle point for me.
Let me know if I can help in any other way.

The type is a string on my end

I'm in a standard Github Codespace, editing with VSCode

I tested on vscode on my side and I get the same behaviour. On this gif you can see the steps I commented above.
npm install
Then I check the type of config.public.apiBase and it's string | number as expected.
npx nuxi@latest module add security
Then I check the type of config.public.apiBase and it's unkown.
Then I close and open vscode and it's string. Which is not what is expected, it should still be string | number

is it a string on your side at all moments?

@divagueame
Edit.
My bad. After deleting node_modules and all build artefacts, I'm in line with you.
But now I have another deviation 😭
Let's try to find out if you have it also.
1- Without Nuxt Security
git checkout bcd56d
rm -rf node_modules
rm -rf .nuxt
npm install

Then apiBase has the right type:

2- With Nuxt Security
npx nuxi@latest module add security

Now apiBase is undefined

For sake of clarity: deleting node_modules and .nuxt, then npm reinstalling does not change the issue.
3- C-Surf Module interaction:
runtimeConfig().public has no properties except csurf:

The type definition of PublicRuntimeConfig is being forwarded to the nuxt-csurf module:

Do you have the same analysis on your side ?

@divagueame
I don't think this is related to Nuxt Security.
If you do what you described above (close and reopen VSCode) without installing Nuxt Security, the types are also destroyed

@divagueame
I don't think this is related to Nuxt Security.
If you do what you described above (close and reopen VSCode) without installing Nuxt Security, the types are also destroyed

I agree with your assessment. It seems this happens independently from Nuxt Security. I'll try to open to an issue on Nuxt repository. Thanks a lot for your help.

Thanks @divagueame
I think it is probably related to the way nuxi prepare generates the dynamic types, but I could be wrong.
Will close here


fix(stats.py): Refactor row-fetching for stats.

This commit address performance bottlenecks in the stats fetching
module. Specifically it:

No longer uses release_id index, instead opting out to an IN query
that includes relevant tests for the release
The 15 test limit is now achieved via PER PARTITION LIMIT clause in
the stat query, instead of discarding indexed data
To avoid exceeding maximum cartesian product of an IN query a
batching mechanism is introduced for reads, splitting stats query
into N amount of 100 test queries
The queries themselves are using execute_async to execute in parallel
Most linear searches are now replaced with map lookups (trading
multiple searches with an upfront map setup cost)

@k0machi this look o.k., but I would still suggest talking with @nyh seeing if there a better way


missing license file

Description
Is this repository missing a License file ? Most of the ignition packages have a License file but I can't find one here

PR created: https://github.com/ggulgulia/ign-plugin/pull/1

I think https://github.com/ignitionrobotics/ign-plugin/blob/ign-plugin1/COPYING can be used as license file for packaging, not sure why this repo is different anyhow.

will be good it have a license file since I'm adding conan-recipes for ignition packages on official conan center index, also known as CCI (see my PR's both open and merged) and most of ignition repos have a LICENSE file which needs to be copied to a license directory as  per the standards of CCI.
ign-plugin conan repo fails pre-build checks due to missing license file. I can use the file you mentioned but it will be easy to maintiain the conan recipes if they have uniform structure


The 'Undelete' action is always in progress for one SAS attached blob container with permissions 'List, Read, Delete'

Storage Explorer Version: 1.21.0-dev
Build Number: 20210809.3
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 18.04/MacOS Big Sur 11.4
Architecture: ia32\x64
How Found: From running test case
Regression From: Previous release (1.20.1)
Steps to Reproduce

Expand one storage account -> Blob Container.
Select one blob container -> Upload one blob.
Right click the blob container -> Click 'Get Shared Access Signature...'.
Select permissions 'Read, List, Delete' -> Create and copy the SAS URL.
Open the connect dialog -> Attach the blob container via the copied SAS URL.
Switch to the attached blob container->Delete the blob.
Switch to 'Active and soft deleted blobs' item.
Select this blob ->click 'Undelete'-> check whether the activity log shows correctly.

Expected Experience
The activity log shows correctly.

Actual Experience
The 'Undelete' action is always in progress.

Verified this issue on the build 20210810.8. Fixed.


3.0.0

its time this library gets another rewrite.
this should include the following:

all existing features
patches for all ffi problems
fixed ci
better unit testing

i forgot :skull:

im busy with other stuff ok


miniupnpd: use iptables-legacy

Description of changes
As noted in https://github.com/NixOS/nixpkgs/pull/80984#issuecomment-1189280812 , UPnP-related tests have been failing since we switched to iptables-nft in 21.11. This seems to happen because in practice miniupnpd ends up using iptables-nft for initial setup scripts, but it uses iptables (-legacy) directly from the C code. As such, it is unable to find the chains it needs at runtime.
This PR makes it always use iptables-legacy. I am aware this is a less-than-ideal solution (users will end up having rules in both iptables-nft and iptables-legacy, which can be confusing). However, this is a straightforward fix that at least makes the module functional for now.
As misuzu has noted in https://github.com/NixOS/nixpkgs/pull/80984#issuecomment-1189342730 , we could in theory use miniupnpd nftables backend with the NixOS iptables-nft firewall. In practice, I have not been able to successfully implement this as of now. For one, iptables-based NixOS firewall sets up separate ip and ip6 chains, while miniupnpd's nftables backend seems to assume an inet chain (which would be true in a pure-nftables firewall) and so fails to update the rules.
I am interested in eventually putting in some work to support the proper nftables backend in this module (when networking.nftables.enabled = true;). I would say that at least https://github.com/NixOS/nixpkgs/pull/207758 is a prerequisite for that.
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1964


feat(ci): Merge agent 6 and 7 rules - part 1

What does this PR do?
Remove distinction between agent6 and 7 rules. Building an agent alone is not used

Motivation
Simplify the gitlab configuration

Additional Notes
Test pipelines generated:
[standard deploy pipeline]
Merge queue pipeline generation test
Branch comparing to itself for modifications on yaml files
integrations-core like pipeline

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable DEPLOY_AGENT=true --variable RUN_UNIT_TESTS=on

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable DEPLOY_AGENT=true --variable RUN_UNIT_TESTS=on

LGTM, when the issue with jobs created in the MQ is tackled. And if all the deleted rules are not used elsewhere

Added a link with the workflow generation test on MQ
Thanks again

/trigger-ci --variable RUN_KITCHEN_TESTS=false --variable RUN_E2E_TESTS=off --variable DEPLOY_AGENT=false --variable RELEASE_VERSION_6=nightly --variable RELEASE_VERSION_7=nightly-a7 --variable BUCKET_BRANCH=dev --variable INTEGRATIONS_CORE_VERSION=master

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable DEPLOY_AGENT=true --variable RUN_UNIT_TESTS=on

/merge


(FACT-2799) Fix fact loading fort nested fact calls

Problem:
Custom fact i18ndemo_fact was not executed on ubuntu 16.04.
Solution
The fallowing flow happened:
Puppet -> Facter (-> FactLoader -> custom fact(that called puppet again)) -> Puppet -> Facter -> (-> FactLoader -> cached custom facts (this is the reason why it was not an infinit loop))
Because Facter was called from within another Facter run, @external_facts object was replaced and the concat was done on an object thaw was discarded.
With the fix
Ubuntu 16.04
root@gowned-decking:/opt/puppetlabs/puppet/lib/ruby/vendor_ruby/facter# facter os
{
  architecture => "amd64",
  distro => {
    codename => "xenial",
    description => "Ubuntu 16.04.5 LTS",
    id => "Ubuntu",
    release => {
      full => "16.04",
      major => "16.04"
    }
  },
  family => "Debian",
  hardware => "x86_64",
  name => "Ubuntu",
  release => {
    full => "16.04",
    major => "16.04"
  },
  selinux => {
    enabled => false
  }
}
root@gowned-decking:/opt/puppetlabs/puppet/lib/ruby/vendor_ruby/facter# rm -rf /opt/puppetlabs/puppet/cache/lib/facter/
root@gowned-decking:/opt/puppetlabs/puppet/lib/ruby/vendor_ruby/facter# puppet agent -t
Info: Using configured environment 'production'
Info: Retrieving pluginfacts
Info: Retrieving plugin
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter]/ensure: created
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/facter_dot_d.rb]/ensure: defined content as '{sha256}8a17c7f9b470dbaaff51e7a4f2103c4e5c4d92667c4b7396cd55d76ebcedab1b'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/i18ndemo_fact.rb]/ensure: defined content as '{sha256}970d26ac91fd7b801062e0c17dcbf6b4ef46be2dd1d4e18adcc1ba0912158006'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/package_provider.rb]/ensure: defined content as '{sha256}4f396a5e0b1016e4ceeb7e9273423171d5a51e083e489c9f8477de88f2c254b6'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/pe_version.rb]/ensure: defined content as '{sha256}9d3a1b46fd1e2d2b604a68994d4b8197b9ca1d8344fe5fa2c2797d8de5742f6f'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/puppet_settings.rb]/ensure: defined content as '{sha256}727f7e6d154cbb5773cea227f6a17019b0a3a73624bbfaa6590ed1de8314ae7b'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/root_home.rb]/ensure: defined content as '{sha256}e857cd21ee4fe14739c8e0f330de645f1f54c41229d608731d4af29a55b8d532'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/service_provider.rb]/ensure: defined content as '{sha256}1beaee8dd3c87c9d887184b9e69c3053762f2261a6ebceaa75f682dab54ba823'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/util]/ensure: created
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/util/puppet_settings.rb]/ensure: defined content as '{sha256}af86574985faba7b25893444917da5197f00bbb39182f7a56123cca6ee71e0f4'
Info: Retrieving locales
Info: Loading facts
Error: Facter: Error while resolving custom fact fact='i18ndemo_fact', resolution='<anonymous>': i18ndemo_fact: this is a raise from a custom fact from eputnam-i18ndemo
Info: Caching catalog for gowned-decking.delivery.puppetlabs.net
Info: Applying configuration version '1601280423'
Notice: Applied catalog in 0.01 seconds

Ubuntu 20.04
root@gowned-boundary:/opt/puppetlabs/puppet/lib/ruby/vendor_ruby/facter# facter os
{
  architecture => "amd64",
  distro => {
    codename => "focal",
    description => "Ubuntu 20.04 LTS",
    id => "Ubuntu",
    release => {
      full => "20.04",
      major => "20.04"
    }
  },
  family => "Debian",
  hardware => "x86_64",
  name => "Ubuntu",
  release => {
    full => "20.04",
    major => "20.04"
  },
  selinux => {
    enabled => false
  }
}
root@gowned-boundary:/opt/puppetlabs/puppet/lib/ruby/vendor_ruby/facter# rm -rf /opt/puppetlabs/puppet/cache/lib/facter/
root@gowned-boundary:/opt/puppetlabs/puppet/lib/ruby/vendor_ruby/facter# puppet agent -t
Info: Using configured environment 'production'
Info: Retrieving pluginfacts
Info: Retrieving plugin
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter]/ensure: created
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/facter_dot_d.rb]/ensure: defined content as '{sha256}8a17c7f9b470dbaaff51e7a4f2103c4e5c4d92667c4b7396cd55d76ebcedab1b'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/i18ndemo_fact.rb]/ensure: defined content as '{sha256}970d26ac91fd7b801062e0c17dcbf6b4ef46be2dd1d4e18adcc1ba0912158006'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/package_provider.rb]/ensure: defined content as '{sha256}4f396a5e0b1016e4ceeb7e9273423171d5a51e083e489c9f8477de88f2c254b6'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/pe_version.rb]/ensure: defined content as '{sha256}9d3a1b46fd1e2d2b604a68994d4b8197b9ca1d8344fe5fa2c2797d8de5742f6f'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/puppet_settings.rb]/ensure: defined content as '{sha256}727f7e6d154cbb5773cea227f6a17019b0a3a73624bbfaa6590ed1de8314ae7b'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/root_home.rb]/ensure: defined content as '{sha256}e857cd21ee4fe14739c8e0f330de645f1f54c41229d608731d4af29a55b8d532'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/service_provider.rb]/ensure: defined content as '{sha256}1beaee8dd3c87c9d887184b9e69c3053762f2261a6ebceaa75f682dab54ba823'
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/util]/ensure: created
Notice: /File[/opt/puppetlabs/puppet/cache/lib/facter/util/puppet_settings.rb]/ensure: defined content as '{sha256}af86574985faba7b25893444917da5197f00bbb39182f7a56123cca6ee71e0f4'
Info: Retrieving locales
Info: Loading facts
Error: Facter: Error while resolving custom fact fact='i18ndemo_fact', resolution='<anonymous>': i18ndemo_fact: this is a raise from a custom fact from eputnam-i18ndemo
Info: Caching catalog for gowned-boundary.delivery.puppetlabs.net
Info: Applying configuration version '1601280481'
Notice: Applied catalog in 0.01 seconds

Jenkins please test this on all

CLA signed by all contributors.

Jenkins please test this on all


Sequelize port with postgres

Hello,
I've worked on a drywall port to sequelize. It's a work in progress (and I think it will remain like this) but it works for most of the parts.
Some changes:

most of the pages are working, login, logout, account creation and update
removed mongoose for sequezlie and pg + pg-hstore
switched to redis session support (can be changed to memeory session for testing, or install redis)
it's missing role support, all roles have admin support

This is mostly done as a proof of concept and I hope it helpes. I've chose pstgres for json support.
https://github.com/ieugen/drywall/tree/sequelize

Thanks for creating an issue! That's a really cool fork. Maybe we should create a wiki pages to feature different forks of Drywall.

Closing this issue for now. Anything new with your project?

Ok. Nothing new at this time.


Crashes when FSL is not available

When FSL is not available, instead of an intuitive error message, it says

/bin/sh: 1: smoothest: not found
plus a cascade of other errors.

Should test for FSL and provide a simple error message.

This is now implemented as part of #69.


Watch Qovery video & give feedback

https://youtu.be/E0RejclnL4w
see this thread https://meilicommunity.slack.com/archives/CP9DVS1RQ/p1610114184105800

It's too late for that now...


Investigate the idea of premultiplied alpha in PixelType

A. If we add 6 new pixel types (lap, rgbap for 8-bit, 16-bit and 32-bit FP), then we can support pre-multiplied with quite a bit of changes. The interesting thing is that PNG with the unspecified ipHone PNG does support premultiplied

or

B. Decode unpremultiplied always, do not keep track of status, just provide ways to do it and undo it at the discretion of the user. This is simpler, but people will not get the advantages of premultiplied or less often.

[ ] Does premultiplied alpha encodes better, and how much better in terms of space? Check with PNG and QOIX.

[ ] Implement function to premultiply, or unpremultiply. This is the easy part, already exist in Dplug ImageKnob.

Big question is (above) whether to keep that information in the type system (PixelType) or not. This is NOT a LayoutOptions, and I don't think it should be a separate status also. Premultiplied alpha is better for compositing, but not for editing. It also looses a bit of data, but it repeatable.

In future world, PixelType will probably explode in complexity with opEquals being non-trivial.

The reading of the color library made by Manu is interesting and points towards the likely shape of a future "Colorspace" concept, which would either the same as PixelType or PixelType would be purely derived from it.

Very much related to #20

Premultiplied alpha doesn't seem to strictly need ICC profile or full colorspace struct.

Numbers for
Size in kb

Input
PNG
PNG-premul
Reduction
QOIX
QOIX-premul
Reduction

Graillon overlay
582 kb
332 kb
0.57
643 kb
425 kb
0.66

Lens overlay
328 kb
161 kb
0.49
360 kb
183 kb
0.5

Inner overlay
157 kb
64 kb
0.4
171 kb
65 kb
0.38

This is breaking change, since decoders are now free to give premultiplied output.

Encoding rgba8 vs rgbap8 images.
QOIX is modified to just signal premultiplication, with a QOIX_SRGB_PREMUL = 2 constant.


[SPOILERS] Command for checking solution doesn't work

Hi, on my system the command to check a solution does not work.
This is the command I'm executing (copied from the solution file):
[felix@sun clmystery]$ echo "Jeremy Bowers" | $(command -v md5 || command -v md5sum) | grep -qf /dev/stdin solution && echo CORRECT\! GREAT WORK, GUMSHOE. || echo SORRY, TRY AGAIN.

and the output is:
SORRY, TRY AGAIN.

while the command for revealing the solution outputs exactly the name I entered. This happens in both zsh and bash.

What OS are you using?

What happens if you just run:
echo "[the answer]" | $(command -v md5 || command -v md5sum)

What happens if you just run:
grep -q "32cacbec4b918e2fd130fd68cf818ae5" solution && echo $?

Archlinux

felix@sun ~/gits/clmystery (git)-[master] % echo "Jeremy Bowers" | $(command -v md5 || command -v md5sum)
32CACBEC4B918E2FD130FD68CF818AE5  -

felix@sun ~/gits/clmystery (git)-[master] % grep -q "32cacbec4b918e2fd130fd68cf818ae5" solution && echo $?
0

The md5sum from 2) is uppercase, and thus isn't matched by grep. Using grep -qif instead of grep -qf makes it work. Thanks for the quick answer! :)

Ohhh interesting.  I'll add the -i, thanks!

Also strange that md5sum is all uppercase for you, I tested this with m5sum on Ubuntu and it was lowercase!

Also strange that md5sum is all uppercase for you, I tested this with m5sum on Ubuntu and it was lowercase!

Mhhm, my md5sum was also lowercase, it was the md5 that produced uppercase output (see my previous edit). But anyways, same thing :)


the model of Pre-tokenized dataset openchat_v3.2_super.train.parquet   is Llama2 or Mistral?

the model of Pre-tokenized dataset openchat_v3.2_super.train.parquet is Llama2 or Mistral?

It is Llama2, not Mistral. However, the data is a merge of sharegpt_clean.json and sharegpt_gpt4.json. Who knows?


[SMALL] Tweaks to performance tests

A few miscellaneous tweaks including:

Adding a for loop to some of the init tests as the warm numbers are so small they report as zero
Adding missing config for some server generated properties in the EF6 model

:shipit:


Automated cherry pick of #3145: scheduler: fix not setSchedPendingUsage condition

Cherry pick of #3145 on release/2.10.0.
#3145: scheduler: fix not setSchedPendingUsage condition

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@17c24a5). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release/2.10.0   #3146   +/-   ##
================================================
  Coverage                  ?   7.45%
================================================
  Files                     ?     414
  Lines                     ?   66173
  Branches                  ?       0
================================================
  Hits                      ?    4932
  Misses                    ?   60817
  Partials                  ?     424

Flag
Coverage Δ

#aFlag
7.45% <ø> (?)

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 17c24a5...ba66d3c. Read the comment docs.


Feature/add mobx router

[x] implement mobX router to navigate between flows
[x] adds screens for splash and login pages
[x] setup base app flow (actual logins and user flows not wired up)

Awesome! Thank you!

@florida Unfortunately none of the code from your pull requests is working on my machine. :( I'm getting multiple errors. I spent some time trying to fix the errors I was seeing, but I wasn't able to figure it out. Each time I fixed something, a new error popped up. It looks like some of this stuff might still be a work in progress? For instance, in the splashscreen component, it is importing a NavigationBar from AppWrapper that does not exist. Should NavigationBar be changed to an AppWrapper componentin the appscreen jsx? I had lots of question marks like that, where I wasn't sure what direction you were headed in, so I didn't want to make changes to the code before understanding what you were trying to do. When you've fixed your changes and all of the views are working on your local machine, please make another pull request (it's fine if both of your previous pull requests are compacted down to one) and I can merge it as soon as I can verify that it is working! Thank you! Here are some of the errors I am seeing when I try to visit the base route:

In splashscreen: Cannot find module "../components/appWrapper"
In App.jsx: Cannot resolve 'file' or 'directory' ./components/dashboard
In appWrapper/index.js: SyntaxError: JSX attributes must only be assigned a non-empty expression (it's pointing to onClick={} in the first button)

I hope that makes sense! Thank you!

I just removed the onClick and it worked for me @mattmallon
On Sun, Jan 15, 2017 at 8:52 AM Matt Mallon<EMAIL_ADDRESS>wrote:

@florida https://github.com/florida Unfortunately none of the code from
your pull requests is working on my machine. :( I'm getting multiple
errors. I spent some time trying to fix the errors I was seeing, but I
wasn't able to figure it out. Each time I fixed something, a new error
popped up. It looks like some of this stuff might still be a work in
progress? For instance, in the splashscreen component, it is importing a
NavigationBar from AppWrapper that does not exist. Should NavigationBar be
changed to an AppWrapper componentin the appscreen jsx? I had lots of
question marks like that, where I wasn't sure what direction you were
headed in, so I didn't want to make changes to the code before
understanding what you were trying to do. When you've fixed your changes
and all of the views are working on your local machine, please make another
pull request (it's fine if both of your previous pull requests are
compacted down to one) and I can merge it as soon as I can verify that it
is working! Thank you! Here are some of the errors I am seeing when I try
to visit the base route:

In splashscreen: Cannot find module "../components/appWrapper"
In App.jsx: Cannot resolve 'file' or 'directory'
./components/dashboard
In appWrapper/index.js: SyntaxError: JSX attributes must only be
assigned a non-empty expression (it's pointing to onClick={} in the first
button)

I hope that makes sense! Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SwimmyApp/project-swimmy-mockup/pull/2#issuecomment-272696517,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXu_EVYjY1VbCeg3Tz4jg21vo6QNPSSmks5rSiScgaJpZM4Ljy7Q
.

Also we were working on the prototype that Tino created! It's posted in the
#project-swimmy room. You need to have flinto in your phone to view it on
your phone, or I can also show it to you via conf! ❤❤
On Sun, Jan 15, 2017 at 8:57 AM Progressive Coders Network<EMAIL_ADDRESS>wrote:

I just removed the onClick and it worked for me @mattmallon
On Sun, Jan 15, 2017 at 8:52 AM Matt Mallon<EMAIL_ADDRESS>wrote:
@florida https://github.com/florida Unfortunately none of the code from
your pull requests is working on my machine. :( I'm getting multiple
errors. I spent some time trying to fix the errors I was seeing, but I
wasn't able to figure it out. Each time I fixed something, a new error
popped up. It looks like some of this stuff might still be a work in
progress? For instance, in the splashscreen component, it is importing a
NavigationBar from AppWrapper that does not exist. Should NavigationBar be
changed to an AppWrapper componentin the appscreen jsx? I had lots of
question marks like that, where I wasn't sure what direction you were
headed in, so I didn't want to make changes to the code before
understanding what you were trying to do. When you've fixed your changes
and all of the views are working on your local machine, please make another
pull request (it's fine if both of your previous pull requests are
compacted down to one) and I can merge it as soon as I can verify that it
is working! Thank you! Here are some of the errors I am seeing when I try
to visit the base route:

In splashscreen: Cannot find module "../components/appWrapper"
In App.jsx: Cannot resolve 'file' or 'directory'
./components/dashboard
In appWrapper/index.js: SyntaxError: JSX attributes must only be
assigned a non-empty expression (it's pointing to onClick={} in the first
button)

I hope that makes sense! Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SwimmyApp/project-swimmy-mockup/pull/2#issuecomment-272696517,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXu_EVYjY1VbCeg3Tz4jg21vo6QNPSSmks5rSiScgaJpZM4Ljy7Q
.

@florida @rapi-progcode Thanks for the help, Rapi! That was one of the errors, there was also another one I found where there was an outdated reference to the dashboard component (which had been moved) in an import statement. I got it working now. I reverted my reverts (ha!) and then made a new commit that fixed the errors. If you pull from master, you should be good to go.

@mattmallon @rapi-progcode totally missed this but I hope that y'all have gotten it working! I also have #8 open, there are however a few merge conflicts that needs to be addressed in that PR.
There's also feature/login-and-auth that has a basic auth screen.
I'll go and fix that PR, cheers!