Cant re-arrange services on UI

(Previous issue https://github.com/getferdi/ferdi/issues/1895#issuecomment-920308071 was closed because i couldnt respond in time, so here is new one, with answers about OS, Ferdi Version and developer console )
Hello,
Yes, of course this is still issue, i just dont have time to chime in very often.
 Also, please post the version of Ferdi, your OS, etc
Windows 10
Version	10.0.19042 Build 19042
Ferdi version 5.6.0 (doesnt matter, happens on 5.5. as well)
Electron 13.1.7
Chrome 91.0.4472
Whats weird, it doesnt happen if im not connected to ferdi services,  i've run ferdi without internet connection, and icons were arranged correctly, but as soon as i've plugged network cable and ferdi cot internet connection, they re-arranged incorrectly.
As for developer console, its spammed with:
https://i.imgur.com/2TPQvhG.jpg
Best Regards,
A

If this is working without an internet connection, but fails with the connection, then I suspect that this is more of a bug in the ferdi-server rather than ferdi itself. Could you please close this bug, and create one here?
Also, the ferdi server is only used to store:

the services you have signed up for, their order & preferences
the workspaces & their order & the services related to the workspaces

The only use of this, is to be able to switch machines and get these prefs transferred to the new machine. If you use Ferdi only in a single machine most of the time, I would suggest that you move to using Ferdi without a server. You can export your current prefs before this switch, and import that configuration back in so that your transition is seamless.

Hello, thanks for answer.
I never saw a need for ferdi account, and it felt strange it required me to make one - I only did it, because every restart, Ferdi keep losing all my services (I rarely restart my work pc, but setting up 10 services every month or so was annoying) - and stopped when I've set up an account.
Could you please point me to exact option for exporting preferences? I have my account exported as backup, but since I'm going to not use it, I assume preference export is different thing from account export.
All the best,
A.

and follow the instructions there. (It might be the same as what you meant by "account export"). Once done, you can use the "Logout" link in the lower left of the Settings/Preferences window/screen. Then choose "Use Ferdi without a login" and import the exported ferdi.data file.

sadly, i have no such option

Since you are using it with a server, you can go to https://api.getferdi.com/user/login and export your data.
Also - please upgrade to version 5.6.2 of Ferdi. It's very hard to find and fix issues in older versions of Ferdi - we are all voluntary contributors.

Since you are using it with a server, you can go to https://api.getferdi.com/user/login and export your data.
thats what i did, it gave account data - even if its actual preferences, how do i import it to ferdi (...i dont want to login to import data that already is when im logged to log out and dont have the data, because it was imported for login not for non-login ferdi usage)
So i cant export it, and im at risk of losing settings if "something" happens.
I can confirm that on newest version i cant re-arrange services as well, so its correctly assumed an server issue.

If you simply logout of Ferdi-server (without deleting the online account in getferdi.com), then you can always go back to it.
I dont know why the Help menu is not showing up for you, but, after you start ferdi, you can visit this local site - and follow the links to import your previously exported data.

@propiro - I'm closing this ticket since there's a new ticket in the appropriate server repository. Please continue the conversation here in case you need help with the import/export of settings. Or else, you can also reach to us on our gitter channel or the stackoverflow room.


Publish 01/14/2022 3:30 PM PT

Docs Build status updates of commit e7b6b69:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsTenantFederationConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


wanted: DaanDeMeyer/reproc

Module location
https://github.com/DaanDeMeyer/reproc
Link to bzlmod issue in the module's repository
No response
Any other context to provide?
No response
Fund our work

[ ] Sponsor our community's open source work by donating a feature bounty

Done.


Add more code coverage

-Test coverage was no longer triggered due to the check for BUILD_GOOS
environment variable that was removed. Removed the check.
-Re-run test package with server code coverage.
-Remove unused functions in test.go.
-Add test for a function in test.go.
-Add missing parse +OK test.

Changes Unknown when pulling ad1198db859682a7fcb734b1d5c98c50c21c3294 on more_test_coverage into ** on master**.

LGTM.

Changes Unknown when pulling 2b8891364aa3a2278405f8529b788d4cca528ed6 on more_test_coverage into ** on master**.

Changes Unknown when pulling 88af5f193af779f13f0e835f6b364ebc91e7e7d7 on more_test_coverage into ** on master**.

Changes Unknown when pulling 16bd806c4646b03f5cb01597382e19395cc05330 on more_test_coverage into ** on master**.

Changes Unknown when pulling a5c393f79a6eb0bf493b2ec67faa25c9ab5045c0 on more_test_coverage into ** on master**.

Changes Unknown when pulling dc906fdd897f497451fda9ffd5e5f7b868aa6a7b on more_test_coverage into ** on master**.

Changes Unknown when pulling acddee9c4a4aed6893087c32bba07fc6cf354d20 on more_test_coverage into ** on master**.


Changed fields array does not contain correct changed fields

I installed this as a package in a brand new project and I seem to be having issues.
The changeFields array does not contain the actual fields that were changed. I changed the website on an account record as an example and I receive AnnualRevenue and TickerSymbol fields in the changedFields but when I look at the payload itself it shows the website and lastmodifieddate fields as the ones with actual changed data.
Here is a preview of what I am looking at, I promise I did not edit this in any way. I replace the sfids just in case.
{   "replayId": 29718,   "payload": {     "ChangeEventHeader": {       "entityName": "Account",       "recordIds": [         "sfid"       ],       "changeType": "UPDATE",       "changeOrigin": "com/salesforce/api/soap/57.0;client=SfdcInternalAPI/",       "transactionKey": "0002d5d2-d8a7-1ba3-9743-9f168046112b",       "sequenceNumber": 1,       "commitTimestamp":<PHONE_NUMBER>000,       "commitNumber":<PHONE_NUMBER>1929,       "commitUser": "sfid",       "nulledFields": [],       "diffFields": [],       "changedFields": [         "AnnualRevenue",         "TickerSymbol"       ]     },     "Name": null,     "Type": null,     "RecordTypeId": null,     "ParentId": null,     "BillingAddress": null,     "ShippingAddress": null,     "Phone": null,     "Fax": null,     "AccountNumber": null,     "Website": {       "string": "goog.com"     },     "Sic": null,     "Industry": null,     "AnnualRevenue": null,     "NumberOfEmployees": null,     "Ownership": null,     "TickerSymbol": null,     "Description": null,     "Rating": null,     "Site": null,     "OwnerId": null,     "CreatedDate": null,     "CreatedById": null,     "LastModifiedDate": {       "long":<PHONE_NUMBER>000     },     "LastModifiedById": null,     "PersonContactId": null,     "PersonMailingAddress": null,     "PersonOtherAddress": null,     "PersonMobilePhone": null }
As you can see from my payload, it does not match up with what is in the changedFields array. Is there some sort of setting or configuration issue on my end that is making this happen?

Hi @ValeriyRCG, this is likely a bug as there's no configuration that could lead to such an issue.
I fixed a similar problem earlier. Can you tell me which version of the client you're using?
If it's not the latest, can you upgrade and let me know if the issue persists.

Hi @pozil I am using the latest client package version. These are the dependencies in my package json.
"dependencies": {     "jsforce": "^1.11.0",     "salesforce-pubsub-api-client": "^2.3.0"   }
Could it be because I have person accounts enabled?

I haven't tested with person accounts and I don't know if this can introduce some extra complexity. The truth is that I'm struggling with the way change fields are encoded.
I'll keep this issue open for now but I'm a bit short on time to diagnose this at the moment, sorry.

Thank you @pozil. I will try to fix this on my end and maybe I can submit a solution if I find it myself.

Hi @ValeriyRCG sorry for the delay but I managed to dedicate some time to this and finally found the issue. Please use the latest version of the library for a fix.


Clicking once sometimes triggers two clicks

I do not know if this is a vanilla problem, but it sometimes happens.
Noticed this while clicking through the resolutions. In some cases it quickly jumps two.
Since I tinker with the clicks, the issue might be part of my code.
Hopefully not the case:
Something with the screen actualization. This would be problematic.

My mouse was defect. 😄 However, I did not test it again.
I will close this regardless. Feel free to open it again should this issue appear.


new features

hi Dan, is it possible to get this data from Nissan Connect?
Identify open doors
tyre pressure
brake system
abs
airbag
engine malfunction warning light
oil pressure
thanks

So last time I looked at this, there seems to be a huge variation in which 'error states' are available for each vehicle.
Which vehicle do you have and do these show in the app for you?


Impove localization features

1. Diacritic displaying
Some diacritics in Vietnamese have been displayed not very clearly with the Open Sans font included in Serene. I tried to use Open Sans by Google webfont then it was much better.
Original font:

@*@Html.Stylesheet("~/Content/font-open-sans.css")*@
<link href="https://fonts.googleapis.com/css?family=Open+Sans:300,400,600,700&subset=vietnamese" rel="stylesheet">

2. Searching without diacritics
This feature works well in Lookup Editor or in Column Picker dialog as diacritics has been removed before searching.
But the quick search feature in entity grid doesn't work the same way. I know it's a searching query to SQL so we can't do the samething like in Lookup Editor (client searching) or in Column Picker dialog (searching in C# code). But can we use collation _CI_AI so that SQL Server ignore diacritics in 'like' command as well?
This feature is very important in Vietnamese software as user don't want (or sometimes can't) type diacritics for searching.
3. Local text key fallback
In many entities I have some similar fields like Code, Name, DisplayOrder, IsActive... and I don't want to translate all those texts. And with one entity, Employee for example , i have to translate these resource strings: DB.HR.Employee, DB.HR.Employee.EntitySingular, DB.HR.Employee.EntityPlural, xxx.EmployeeId... while they are the same in Vietnamese.
So if we have some fallback for local text key, it'll save time and may be improve the performance as well (as we have less resource strings with shorter keys). For example:

if "DB.HR.Employee.EntitySingular" can't be found, then it looks up for "DB.HR.Employee"
if "DB.CRM.Task.EmployeeId" can't be found, then it looks up for "Employee"
if "DB.HR.Employee.Name" can't be found, then it looks up for "Name"
...
Then developer can put some common text in site.texts.xxx for using in all modules.
I've implemented (3), if you agree with that, it'll create a pull request

I don't want to use internet versions of fonts by default, this would be not ok for intranet apps. If font is somehow updated, we might update it from there and take a new local copy, but not a culture specific one. You are free to use any you like in your own apps though.

Entity grid uses the collation in your column. If you use X_CI_AI in your SQL column, it will use that. So you better change collation in database. It's also much better performance wise. We have similar requirement in Turkish_CI_AI etc and I can't imagine why would you want to use different collation for search, but if really need, that would require another attribute e.g. SearchCollationAttribute etc.

I would create a base class for 3, especially for shared fields. Anyway i understand there can be cases where it wouldn't solve all. I would implement that as a fake service wrapping ILocalTextRegistry, e.g.

Dependency.Register<ILocalTextRegistry>(new MySuperFallbackRegistry(Dependency.Resolve<ILocalTextRegistry>()));

I'd be happy with a pull request containing such a class, similar to LogicOperatorPermissionService.

Btw, as a side note, i think you are putting english texts on your texts. I would use Vietnamese texts on my rows, as for you Vietnamese is primary language. That way you would do translations for English texts.
Invariant language is language neutral to your application, e.g. default, its not English

I also don't want to use Internet version. I was just wondering why some Vietnamese diacritics looked like they are missing and not comfortable for reading. I thought that the local font hadn't Vietnamese subset and hoped that it would be included as most Vietnamese are Latin Extended, few characters are culture specific. OK I'll download and use my own fonts.
I didn't notice that we can change collation for columns in SQL Server. I changed the database collation to _CI_AI but all nvarchar columns remained old collation _CI_AS, so the quick search didn't work, then I thought about adding collation to the query... It works OK now
Base class is a good idea for shared fields. I added code to the TryGet method of LocalTextRegistry class directly. I'll look at LogicOperatorPermissionService to see how to implement a fake service

I understand you point on primary language. Our products must support both Vietnamese and English, and all tables and fields are in English, so the generated rows have English with quite good quality. So I don't want to modify display name in all rows to Vietnamese and then translate them back to English

Thanks for your pull request, a wiki entry later would be useful i think.

Dear @volkanceylan , I tried to use FallbackLocalTextRegistry for a sample in Serene and write a wiki.
However, it didn't work because:

The TryGet method of LocalTextRegistry never return a null string as it search in language fallbacks or default language.
The ILocalTextRegistry was casted to LocalTextRegistry serveral times in the code so it caused exceptions

I also tried to fix above issues by:

Adding parameter bool searchInFallbacks = true to TryGet method
Adding all methods to be called of LocalTextRegistry to ILocalTextRegistry, implementing them on FallbackLocalTextRegistry. Then I  removed all casting so I could register FallbackLocalTextRegistry instead of LocalTextRegistry.
then I could built and run the application without exceptions. However, for some reasons, it couldn't register texts (approvedTexts had no element) and Dynamic Script for Local Text didn't work.

I think that implement this feature as a fake service made it too complicated while a little modification to TryGet method of LocalTextRegistry would work fine. Key fallback should be implemented the same way as the language fallback be.

I understand that it didnt work like you hoped but i'm not sure this is a valid fallback mechanism. For example en-us might be a fallback for en-uk and if you prioritize key fallback, it might never hit en-us even if there is a translation there. You might subclass LocalTextRegistry and change interface implementation of TryGet.

You can also make private members of LocalTextRegistry protected and TryGet method virtual if needed.

Thanks for you suggestion. I also thought that subclass would be OK. I think language fallback should be prioritized except the default (invariant) one.
I'm going to reimplement this feature. If it fit my need then i think it may be useful for someones else. It should be optional and easy to switch to in Serene without modify Serenity Dlls.


Apply working pattern to linked period of sickness in calculate-statutory-sick-pay (without change in order of questions)

Supersedes #2031.
Summary
Earlier this year the calculator was extended to take into account a previous linked period of sickness. However, this new functionality had a bug in it which was raised by HMRC in this ZenDesk ticket.
The problem was that the calculator was considering all calendar days in the linked period of sickness to be Qualifying Days when in fact it should have only been considering the days the employee usually worked i.e. the same as for the current period of sickness.
The effect of this was to (in some circumstances) incorrectly credit the employee with more Waiting Days than they should've had. In these circumstances the calculator output might've said that the employee should've started receiving sick pay sooner than they should've done.
This pull request fixes that bug. It also fixes a couple of related minor issues and one unrelated minor issue that were requested during the fact check process. See below for details.
Note that the actual changes are not that big, but unfortunately there are a lot of changes to the regression test artefacts. I've attempted to break these changes down as much as possible into smaller commits so it's easier to see what's going on.
Note also that both the current and linked periods of sickness are referred to as Periods of Incapacity for Work (PIW) in the policy documentation. I've used that terminology below.
Expected changes
Linked PIW must be separate from the current PIW
New validation error message: You need to enter a date at least 1 day before the start of the current period of sickness or it isn't a separate linked period of sickness

Before: https://www.gov.uk/calculate-statutory-sick-pay/y/ordinary_statutory_paternity_pay/yes/no/2015-01-07/2015-05-03/yes/2015-01-01/2015-01-07
After: https://www.gov.uk/calculate-statutory-sick-pay/y/statutory_paternity_pay/yes/no/2015-01-07/2015-05-03/yes/2015-01-01/2015-01-07

Linked PIW must be at least 4 calendar days long
New validation error message: The linked period of sickness must be at least 4 calendar days long

Before: https://www.gov.uk/calculate-statutory-sick-pay/y/ordinary_statutory_paternity_pay/yes/no/2015-10-01/2015-10-04/yes/2015-09-01/2015-09-03
After: https://www.gov.uk/calculate-statutory-sick-pay/y/statutory_paternity_pay/yes/no/2015-10-01/2015-10-04/yes/2015-09-01/2015-09-03

Only usual working days are Qualifying Days in linked PIW
In the following scenario the linked period of sickness is 8 calendar days (Thursday-Thursday) and the employee works Sundays & Fridays. Previously it was assumed that all 8 days were qualifying days; whereas now only the Sunday & Friday are counted as qualifying days.
Thus previously the linked period of sickness accounted for all of the 3 waiting days (leaving 0 waiting days); whereas now it only accounts for 2 of the 3 waiting days (leaving 1 waiting day). Therefore where previously the 2 days was not reduced, now the 2 days is reduced by 1 day to 1 day; and the SSP amount for the week ending 14th March 2015 is reduced to £0 accordingly.

Before: https://www.gov.uk/calculate-statutory-sick-pay/y/none/yes/no/2015-03-12/2015-03-19/yes/2015-03-02/2015-03-09/eight_weeks_more/weekly/2015-02-27/2015-01-02/1000.0/0,5

"Your employee is entitled to SSP for 2 days out of 2 working days"
Week ending 14 March 2015, SSP amount: £43.78

After: https://www.gov.uk/calculate-statutory-sick-pay/y/none/yes/no/2015-03-12/2015-03-19/yes/2015-03-02/2015-03-09/eight_weeks_more/weekly/2015-02-27/2015-01-02/1000.0/0,5

"Your employee is entitled to SSP for 1 days out of 2 working days"
Week ending 14 March 2015, SSP amount: £0.00

Remove 'Ordinary' from 'Ordinary Statutory Paternity Pay'
The Ordinary prefix should no longer be used anywhere in the SSP calculator.

In the 'Is your employee getting any of the following?' question

Before/After: https://www.gov.uk/calculate-statutory-sick-pay/y

In the URL when the above option is selected

Before: https://www.gov.uk/calculate-statutory-sick-pay/y/ordinary_statutory_paternity_pay
After: https://www.gov.uk/calculate-statutory-sick-pay/y/statutory_paternity_pay

On the outcome page when the above option is selected and the employee is entitled to sick pay

In the 'What you need to know' section:

Before: https://www.gov.uk/calculate-statutory-sick-pay/y/additional_statutory_paternity_pay,ordinary_statutory_paternity_pay/no/no/2013-04-02/2013-04-10/yes/2013-01-01/2013-02-15/before_payday/fortnightly/2000.0/30/0,6
After: https://www.gov.uk/calculate-statutory-sick-pay/y/additional_statutory_paternity_pay,statutory_paternity_pay/no/no/2013-04-02/2013-04-10/yes/2013-01-01/2013-02-15/before_payday/fortnightly/2000.0/30/0,6/

Useful references

Statutory Sick Pay (SSP)
Statutory Sick Pay: manually calculate your employee’s payments
SPM10135 - Statutory Sick Pay (SSP) - General Information
Statutory Sick Pay calculator - LOGIC

Note that although I'm pretty confident that this is now ready for deployment, I'm still waiting for the final go ahead from HMRC.

We've had the go ahead from HMRC to deploy this. I'm removing the "Waiting on factcheck" label.
Could someone please review it? Thanks.

This looks good to me. Great work manually updating the regression test artefacts. It made it much easier to follow the changes we were expecting to see.

I've updated the commit notes that @chrisroos highlighted, rebase against master and force-pushed in preparation for merging.


fix: mark storage command as experimental

What kind of change does this PR introduce?
Bug fix, feature, docs update, ...
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 58.63%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.31%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5673

Relevant Lines:
9676

💛  - Coveralls


WIP: model : Add Support for Archive Storage of Models

Fixes: #662
Have currently made a basic implementation to receive feedback on various aspects.
Various tests are failing currently , would update all the relevant models soon.
Have to update docs and tests for the following list of models (if needed to support archive storage):

[ ] SLR model
[ ] Auto Sklearn model
[ ] Daal4py model
[ ] Pytorch model
[ ] Scikit model
[ ] Scratch model
[ ] Spacy model
[ ] Tensorflow model
[ ] Vowpal Wabbit model
[ ] Xgboost model

@pdxjohnny @sakshamarora1  please give your inputs.

Codecov Report

Merging #1174 (af4eb3b) into master (7a91f3b) will decrease coverage by 0.64%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master    #1174      +/-   ##
==========================================
- Coverage   86.21%   85.56%   -0.65%
==========================================
  Files         168      169       +1
  Lines       10624    10735     +111
  Branches     1746     1768      +22
==========================================
+ Hits         9159     9185      +26
- Misses       1117     1195      +78
- Partials      348      355       +7

Impacted Files
Coverage Δ

dffml/df/archive.py
29.85% <29.85%> (ø)

dffml/model/model.py
67.85% <38.00%> (-12.98%)
:arrow_down:

dffml/util/data.py
72.92% <0.00%> (-2.77%)
:arrow_down:

tests/util/test_net.py
98.36% <0.00%> (-1.64%)
:arrow_down:

dffml/noasync.py
78.94% <0.00%> (-1.06%)
:arrow_down:

dffml/base.py
83.23% <0.00%> (-0.29%)
:arrow_down:

dffml/util/testing/consoletest/commands.py
75.56% <0.00%> (-0.06%)
:arrow_down:

dffml/source/df.py
75.60% <0.00%> (ø)

Continue to review full report at Codecov.

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

Where is this at right now? We're closing on GSoC deadline and want to make sure this gets completed

@pdxjohnny
This should be merged BEFORE #1199
Because this branch has the older version of #1199 merged in it,  doing otherwise would remove the latest changes from #1199 , not merging latest changes from #1199 into this branch because the tests take around 6 hours to complete since swportal tests were included.

Merged as https://github.com/intel/dffml/commit/38390a45ea5a2aacd1dc2398ac2acbcf9b8c7a53


read from /etc/container/rhel/secrets also

Fix https://github.com/projectatomic/docker/issues/186
@rhatdan @cgwalters @parthaa PTAL

I'll take care of moving this patch once merged in 1.10.3 fedora and rhel
I've also to modify docker.spec to include /etc/container/rhel/secrets in %files as we do for /usr/share/rhel/secrets

Do we want to create this directory on install, or just document that the user can create the directory to override the default?  Or should we just read both directories?

Do we want to create this directory on install, or just document that the user can create the directory to override the default? Or should we just read both directories?

I'd go with shipping the directory as we do with the other so people are aware of this functionality on install but this is just my 2c. @cgwalters wdyt?

If we ship the directory now and the directory is empty then it will break the default secret passing.  Since it only grabs /etc/containers/rhel/secrets if it exists.

:bike: Can we drop the rhel/ from the path in /etc now, since this functionality can apply to other environments?

Can we drop the rhel/ from the path in /etc now

no strong opinion on this, @rhatdan ?

Yes drop it.


Update Dockerfile

Issue #, if available:
Description of changes:
Update docker file to use public ecr image
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.71%. Comparing base (98e0d82) to head (895ce9f).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                Coverage Diff                 @@
##             release/v1.25.x     #916   +/-   ##
==================================================
  Coverage              85.71%   85.71%
  Complexity                19       19
==================================================
  Files                      3        3
  Lines                     49       49
  Branches                   5        5
==================================================
  Hits                      42       42
  Misses                     3        3
  Partials                   4        4

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


hello-world job is stuck when running in microk8s 1.22/stable using docker executor

Summary
Try to run hello-world job
▶ argo submit --watch -n argo https://raw.githubusercontent.com/argoproj/argo-workflows/master/examples/hello-world.yaml

Then it stucks forever, even after hello world job is already completed.
Name:                hello-world-2fnzw
Namespace:           argo
ServiceAccount:      default
Status:              Running
Conditions:
 PodRunning          True
Created:             Fri Sep 17 06:38:46 +0000 (5 minutes ago)
Started:             Fri Sep 17 06:38:46 +0000 (5 minutes ago)
Duration:            5 minutes 24 seconds
Progress:            0/1

However, after switching to pns, it works fine.
Diagnostics
👀 Yes! We need all of your diagnostics, please make sure you add it all, otherwise we'll go around in circles asking you for it:
What Kubernetes provider are you using?
sudo snap install microk8s --classic --channel=1.22/stable

What version of Argo Workflows are you running?
v3.1.12. Here is the installation spec: https://gist.github.com/junjie-landing/0281b939eb33301d78f12e08fbd575c1
What executor are you running? Docker/K8SAPI/Kubelet/PNS/Emissary
Docker
Did this work in a previous version? I.e. is it a regression?
It was not working in 3.0.10 neither.
It is working well in minikube v1.22.0 tho.

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Could you run a kubectl describe on your workflow pod and controller pod? Any logs would be useful as well.


Update climate.py

If climate have mode AUTO and MANUAL all of them was HVACMode.HEAT_COOL so in HA you cant see real mode of climate, only heat/cool 2 times

After changing this part of the code, the operating mode is displayed correctly, but it is still impossible to change the mode from the home assistant, it is always Auto, I will think about how to fix this. I would also like to add a button to turn on the anti-freeze mode and lock the control panel (child lock), but I have just started my research

I have a floor heating thermostat DunWore F60. It has modes 'auto', 'comfort', 'eco', and 'holiday'. However, in the preset settings, only 'eco' mode is visible. How could this be fixed?

I have had the same issue . but after leaving ony one line  "auto": HVACMode.AUTO, everything works ok . I am using this intergation to control some TRVs. I can tell you that I have at least 3 models from different manufacturers and all of them have the Auto mode as "main/default" mode. So it is important to have it exposed correctrly .
Since the integration si Beta there must be feedback from real users.
I hope this will be pushed to main branch as it will help a lot with TRV users.


Those attributes are not displaying in select option while creating cart rule whose attribute name are not written for en locale.

Framework Version: 0.1.6

Fixed, wait till next merge.

Framework Version: 0.1.6

Fixed


Customization of the filter input

Hi,
I would like to know if there are issues to custom the input which we use for the filter in the table.
I see filterBy which allow to put a default filter when the component is mounted but I didn't find something to change the input filter itself.
For the moment I use the innerHTML in js to change this but it's, I think, not the best solution.

I've updated the readme to include the filterPlaceholder option

Readme has been rewritten, filterPlaceholder is not documented, but still an available prop.
Keywords: placeholder, input, filter.


Automatic Line Breaks in Display Math

Description
Similar to #826 but for display math. Possibly related to #1186.
Display math equations should provide the ability to facilitate automatic line breaks within them. A possible inspiration could be the breqn for LaTeX2e
Excerpt from breqn documentation showing one example of line breaking in math:

Use Case
This would improve UX when writing math equations with limited space especially  for equations in multi-column. This use case can be mitigated today by using a grid with auto columns but it is not an appropriate solution for all use cases. writing long equations that span multiple lines.
Example of the grid w/ auto columns trick:

Example of where the trick does not work:

Expected line breaking (manual line breaking w/ align):

It's close to impossible to do this fully automatically in a satisfactory way, as there is just so much variation in mathematical notation. I've never seen breqn uses for anything serious "in the wild". It's all amsmath and mathtools, which give you full control.

While this is true, I think it's a good starting point people targeting multiple page sizes such as book authors and such and especially if #188 goes through.
I'm currently reading "Homotopy Type Theory" which is written in LaTeX2e and the author targets multiple "book" sizes. The method they're accomplishing this is via manual "breakpoints" using a macro they call \narrowbreak where they explicitly define it be a line break depending on the chosen page size. While this rudimentary method suffices it as they're only targeting a limited # of sizes, it doesn't scale.
If full on automated breaking is not possible, perhaps manual breaking opportunities given by the user (a la HTML's <wbr> or Unicode U+200B ZERO-WIDTH SPACE) is more feasible.

Specifying manual breaking opportunities is definitely the way to go in my opinion


Newline-below behavior for multiple platforms

Updating change based on feedback from previous pull request.  Should work on multiple platforms.

@jcoulter I just released 0.2.3 with your additions and you are mentioned in the contributions list. Thanks again!


mariadb-client depends on gcc at runtime

Issue description
mariadb-client depends on gcc at runtime
Steps to reproduce
[clever@amd-nixos:~]$ nix-build '<nixpkgs>' -A mariadb.client
/nix/store/phvyvha8amcw7dm8557hkk4c66jyl6ac-mariadb-client-10.1.17-bin
[clever@amd-nixos:~]$ grep gcc result-bin/bin/mysqlbug
COMP_CALL_INFO="CC='/nix/store/45qrn064f92kfxnjg99z39zxda671h6f-gcc-wrapper-5.4.0/bin/gcc'  CFLAGS=''  CXX='/nix/store/45qrn064f92kfxnjg99z39zxda671h6f-gcc-wrapper-5.4.0/bin/g++'  CXXFLAGS=''  LDFLAGS=''  ASFLAGS=''"

See https://github.com/NixOS/nixpkgs/issues/12898 and https://github.com/NixOS/nixpkgs/issues/5895

Is "mysqlbug" the only thing referencing gcc? It was removed in mariadb 10.2, I think.

yeah, only mysqlbug references gcc, and master is on 10.1.17, bug still present

I think until MariaDB 10.2 becomes stable, we should leave this alone. As stated, mysqlbug has been removed from the 10.2.x series. Once they declare 10.2.x stable, we can update our mariadb and should no longer have the gcc dependency at runtime.

(triage) fixed?

Fixed now.
$ nix path-info -rf. mariadb.client | xargs du -sc | sort -n
128     /nix/store/bv6znzsv2qkbcwwa251dx7n5dshz3nr3-zlib-1.2.11
732     /nix/store/lgi2vjw1r2q67h8jibwx07snwscf1gcw-mariadb-client-10.2.17
1192    /nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23
3616    /nix/store/9kr8r78bwk12050ppywfbhg1vrsd6dp8-openssl-1.0.2p
5636    /nix/store/zk5zj2307zxaq7dx585yia3dn5k4qlsl-gcc-7.3.0-lib
10780   /nix/store/s2n99784krxl91mfw3cnn9ylbb5fjvkx-ncurses-6.1
26428   /nix/store/fg4yq8i8wd08xg3fy58l6q73cjy8hjr2-glibc-2.27
39564   /nix/store/jv5ri2hp7m7714085qai3ysjilmcjngd-mariadb-client-10.2.17-bin
88076   total

The client has suspiciously large binaries in bin/ – possibly due to static linkage or something, but no gcc anymore.


routeData Is Not Called from the root.tsx File

Exporting a routeData function from root.tsx doesn't seem to do anything. The function is not called under any circumstances.
It seems like this has been mentioned on Discord already, but I'm opening this as a bug just to make sure the issue doesn't get lost. :sweat_smile:

Thank you. We should wire this up. For those stuck here. You can do a default export from root.data.ts in the meanwhile. But we should definitely support this.

Hi @ryansolid! It doesn't seem that the root.data.ts default export is allowed to return async data, is it something planned?
(my use case is to set the <Html lang={lng}> lang attribute based on the saved user session in the db, but I am not sure there is a good way to use async data for the root <Html> attributes)

This would be extremely useful. I want to have a component in root so I can show "You are logged in as {user}" on every page. At the moment this is only possible with client side rendering.

@benoitv-code  the async data call should be wrapped inside createResource(async () => {}) like the normal routeData function does.
@ryansolid I suggest adding some documents about the root.data.ts. It took really a while for me to correctly put my server side graphql request in side root.data.ts along with the cookie.

@hardywu Can you show me more about how you achieved that? I tried using createResource within root.data.ts but the root component always starts rendering before the promise has resolved. I just can't get server side root rendering to wait for a promise with everything that I have tried.

So I figured out how to get the resource promise result to render server side. Just wrap the display of the result in a < Suspense> tag.
Now I can finally have async wherever I want on the server side!

In setting up for SolidStarts next Beta Phase built on Nitro and Vinxi we are closing all PRs/Issues that will not be merged due to the system changing. If you feel your issue was closed by mistake. Feel free to re-open it after updating/testing against 0.4.x release. Thank you for your patience.

See https://github.com/solidjs/solid-start/pull/1139 for more details.


message_edit: Avoid registering duplicate events

hotkey.js is the file that handles the 'e' keyboard shortcut. It maps to the 'edit_message' event and will simply call message_edit.start().
message_edit.start() doesn't check whether it's already been opened previously, so it will go through and try to register handler for the clipboard button again. When the clipboard button gets clicked, the handler will be called twice. Once with a properly target element, and once with null.
Fixing this issue by checking if message_edit.start() has already operated on the given $row.

Fixes: #25462

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Thanks @tridentpancake ! Looks like there's a text to fix before this is ready for review.

Nice find! I fixed the linter error with lint --fix while I had it downloaded to test it and have marked it to merge once CI passes.

Nice find! I fixed the linter error with lint --fix while I had it downloaded to test it and have marked it to merge once CI passes.

cool, thanks. Was away on vacation and didn't get to it.


CLI[E2E]: Create cluster and install chart before running the E2E tests

Currently E2E tests for node setup cannot run in CI pipeline since node cluster is not up before running npm run test-e2e. So we need to deploy cluster and install chart before running the E2E.
There is chicken-egg problem here. We could use fsnetman to deploy and setup the network to test itself. Another approach would be leverage existing bash code to setup the cluster and then test fsnetman, but we anticipate bash code will become obsolete, so need to think which way to go here....
For the time being, I've disabled e2e in the pipeline until we solve this ticket (https://github.com/hashgraph/full-stack-testing/pull/491/commits/8b81b66da333a0012d67e8cbf4e415ed07491841).

:tada: This issue has been resolved in version 0.13.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update to use chai 2.x

So far chai-version is limited to < 2 would it be possible to update it to use 2.x (2.0.0 has been release last month with a couple of fixes)

Sorry, didn't notice there were already 2 PR - it would be great if https://github.com/Bartvds/chai-json-schema/pull/11 could be pulled. If someone else finds it in the meanwhile, i'm going with the suggestion and using chai2-json-schema for now.
Hope this gets fixed eventually though.

Anyone want to take ownership and get this sorted? See https://github.com/Bartvds/chai-json-schema/issues/23

The dependencies are updated, this will be included in the next release.


Speeding the training on mixed data set - categorical data, numerical and text.

Trying to train the model on custom data which has various categorical feature with very high diversity like City, text features and numerical feature. Data size is small - 380K.
But the training was never starting! It is stuck at this for few hours!

How to improve the training?

Hi @vinay-k12 , I think it's because of the cardinality of your data that makes the bootstrapping part not progress. What you could try is not to use the automated termination based on the bootstrap statistic; instead, you can use a validation sample.
Try:
# Use 20% of the data as a validation set early-stopping.
rtf_model = REaLTabFormer(
    model_type="tabular",
    gradient_accumulation_steps=4,
    logging_steps=100,
    train_size=0.8,
)

# Fit the model without sensitivity bootstrapping.
rtf_model.fit(df, n_critic=0)

This will fit the data directly. Hope this helps!


Need of JWT?

Hey, thanks for this project. While i was experimenting with this, i realized CI, automatically creates session and puts all data into it then sent this in cookie. Then on following requests, checking cookies then matches related session with it. So i am trying to implement react Login and auth system without jwt token, just bare sessions.
In your code, you are both sending JWT, and creating session for it. I couldn't understand the principle behind this. If we want to use JWT's we dont need sessions, if sessions are included in cookie, its OK to go with cookie and session based system if security concerns are not too much.

Yes, for your case you can go with that, approach.
For this project I added the JWT for API authentication. Not for login,
For login/signup we can use session.


PGX v5

There's a new pgx v5 beta announced and available for testing already.
https://github.com/jackc/pgx/issues/1273
Will pggen be ported to support pgx v5? Some major incompatible changes at it seems.

I've been following the beta threads closely. I'd like to upgrade but looks like a pretty decent lift. I'll wait until it's released before planning the migration.

Thank for for the switch response! Yes, lots of changes there.
We're currently in the situation to start a project with pgx v4 (with pggen of course) but then needing to migrate at some (soon?) point.

FYI The official release happened 10 days ago.

I've started figuring out the work involved here. The overall goal is that pggen will migrate to pgx v5.
Some helpful migration threads:

https://github.com/jackc/pgx/blob/master/CHANGELOG.md#v500-september-17-2022
https://brandur.org/fragments/pgx-v5-sqlc-upgrade

The upgrades in PGX v5 that affect pggen:

A bunch of pgtype types moved or were deleted.
Use of codecs for encoding and decoding--separate from values.
Support for tracing and query rewriting.
The batch interface now takes a callback to decode.

As a high-level plan:

Start a pgxv5 branch
Rewire the types so that simple queries work.
Support query tracing and rewriting in pggen-generated code.
Allow custom codec overrides per query column and arg.
Add support for batch queries.

I think most code should be straightforward to upgrade. The tricky upgrades will involve batch queries since the API dramatically changed, and maybe some--go-type flag usage.

Some changes are easier than others. Looking at the errors after pointing the generated file to v5, here are some of the simpler changes:

Change []pgtype.DataType to []pgtype.Type
Change *pgtype.ConnInfo to *pgtype.Map
Change pgtype.NewConnInfo to pgtype.NewMap()
Change ci.RegisterDataType(&typ) to ci.RegisterType(&typ)
Change tr.connInfo.DataTypeForName(name) to tr.connInfo.TypeForName(name)

Some other changes are needed for newTypeResolver, NewTypeValue, findValue, and setValue but unsure what those are. @darkxanter I'm curious what manual changes you did between v4 and v5 to get everything working properly.
@jschaf What's the current status of the migration to pgx v5?

I would like to migrate, but it's been a low priority for me. The main reason is that we use the batch interfaces almost exclusively for our API endpoints, and I'm not looking forward to migrating to the callback approach (which is better).

Some other changes are needed for newTypeResolver, NewTypeValue, findValue, and setValue

Some of those probably get easier. In particular, pgxv5 has a nicer interface for building composite types.

I am interested in a migration to pgx5, since pgx5 exposes a pipelining API I'd like to use on the same connection that I create the querier object from.

Okay, how about this for a plan to build some momentum?

Upgrade to pgxv5 with a new release for the "easy" changes--no batch queries, and maybe skip composite types
Add new releases to catch up on features.

How valuable is pggen if it uses pgxv5 with a more limited feature set?

For me 100% valuable because it allows to use pggen with pgx v5 in general.
Maybe you should put batching behind an opt-in flag anyway because a lot of code is generated but if you don't use batching then it is unnecessary.
I personally create pg functions.

I agree with aight. I use either stored procedures / functions or pipelining if a future query depends on the result of a previous query. Having pgx5 for the pipelining support, as well as a flag to disable batch creation, would be great.

It would be great to have pgxv5 support. Batch queries are not essential to me but composite types are. I considered switching to sqlc for pgxv5 support but its lack of composite type support makes it not a viable alternative to pggen.

Any update on this? If a portion of this can be delegated, I can help.


Add hlint support.

Hlint is cool. Closes #42 and #37.

Merged to master starting with https://github.com/lwm/tasty-discover/commit/52ce07633919e68861bf5c619f3320393f556087.


[Snyk] Fix for 2 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

718/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 6.5
Server-side Request Forgery (SSRF) SNYK-JS-PARSEURL-3023021
No
Proof of Concept

643/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 5
Improper Input Validation SNYK-JS-PARSEURL-3024398
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: @lerna-lite/cli
    The new version differs by 72 commits.

367e2d9 chore(release): publish new version 1.11.3
fcaf80b docs: tweak using project logo size
e6ec47b docs: add largest project using lerna-lite
aa0b630 docs: add logos to all largest projects using lerna-lite
b23e64f docs: add Jest logo
9473ae6 docs: simplify largest project links
a97cbcb docs: tweak largest project links
84388a1 docs: add largest project links to readme
b84384a Merge pull request #341 from renovate-bot/renovate/all-minor-patch
bcce5ae fix(deps): update dependency git-url-parse to ^13.1.0
e0ba992 Merge pull request #340 from renovate-bot/renovate/all-minor-patch
5be84d6 chore(deps): update dependency @ types/jest to ^29.0.3
1fca7b6 Merge pull request #339 from renovate-bot/renovate/all-minor-patch
8a990c4 chore(deps): update all non-major dependencies
62e5ddf Merge pull request #338 from ghiscoding/chore/use-sequential-npm-scripts
55c22c5 chore: use sequential task in npm scripts instead of pre/post
c1f2431 Merge pull request #337 from ghiscoding/bugfix/lerna-cli-build-missing-type
836d7f0 fix(cli): add missing Type to fix TSC build error
9815d64 docs(version): add `GH_TOKEN` scope requirements for `--create-release` option
0900f96 Merge pull request #336 from ghiscoding/bugfix/lerna-run-exclude-deps
3c222ed fix(run): exclude dependencies with `--scope` when `nx.json` is not present
f269d5e Merge pull request #335 from ghiscoding/bugfix/underscore-to-overrides-nx
4af981f chore(run): Update deprecated nx arg `_` to `__overrides__`
934adfa Merge pull request #334 from ghiscoding/chore/deps-2022-09-12

See the full diff

Package name: @lerna-lite/run
    The new version differs by 72 commits.

367e2d9 chore(release): publish new version 1.11.3
fcaf80b docs: tweak using project logo size
e6ec47b docs: add largest project using lerna-lite
aa0b630 docs: add logos to all largest projects using lerna-lite
b23e64f docs: add Jest logo
9473ae6 docs: simplify largest project links
a97cbcb docs: tweak largest project links
84388a1 docs: add largest project links to readme
b84384a Merge pull request #341 from renovate-bot/renovate/all-minor-patch
bcce5ae fix(deps): update dependency git-url-parse to ^13.1.0
e0ba992 Merge pull request #340 from renovate-bot/renovate/all-minor-patch
5be84d6 chore(deps): update dependency @ types/jest to ^29.0.3
1fca7b6 Merge pull request #339 from renovate-bot/renovate/all-minor-patch
8a990c4 chore(deps): update all non-major dependencies
62e5ddf Merge pull request #338 from ghiscoding/chore/use-sequential-npm-scripts
55c22c5 chore: use sequential task in npm scripts instead of pre/post
c1f2431 Merge pull request #337 from ghiscoding/bugfix/lerna-cli-build-missing-type
836d7f0 fix(cli): add missing Type to fix TSC build error
9815d64 docs(version): add `GH_TOKEN` scope requirements for `--create-release` option
0900f96 Merge pull request #336 from ghiscoding/bugfix/lerna-run-exclude-deps
3c222ed fix(run): exclude dependencies with `--scope` when `nx.json` is not present
f269d5e Merge pull request #335 from ghiscoding/bugfix/underscore-to-overrides-nx
4af981f chore(run): Update deprecated nx arg `_` to `__overrides__`
934adfa Merge pull request #334 from ghiscoding/chore/deps-2022-09-12

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Server-side Request Forgery (SSRF)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Done in #16.


Don't retain ovn_match_northd_version from adopted host

There is no good reason to have this setting set to anything but false. If set to true, this will render OVN controllers broken until northd on control plane is fully upgraded.
Since [1], we guarantee that ovn controllers are always updated before northd.
See more details in [2].
[1] https://github.com/openstack-k8s-operators/openstack-operator/pull/792
[2] https://github.com/openstack-k8s-operators/edpm-ansible/pull/651
Depends-On: https://github.com/openstack-k8s-operators/edpm-ansible/pull/651

/lgtm


Adding support for older configs; only trying to load config from file if file exists

Previously, ZPUI could load config from /boot/pylci_config.json. It was thoughtlessly replaced with /boot/zpui_config.json by me some time ago, this PR adds a fallback - and one more fallback to config.json.example, in case all other configs are broken for some reason. Also, this commit unclutters the boot logs, avoiding exceptions by not trying to load config file paths if those files do not exist.

Even though the change is miniscule and I've tested it, this part of ZPUI is critical, therefore, it's better if somebody else tests it before merging it =)

Decided to merge this into master - it's a critical fix (since somebody might already have a pylci_config.json file that'd stop working after the update), and it's a relatively small one.


Java: fixing issue #89 and adding tests

This fixes issue #89 for Java and adds the tests as outlined. Also fixes an unrelated typo (line 152).

Cool, thanks @bocops !


[3.x] #375 file delete folder

Description
Deletes the folders if the directory is empty after media deletion.
Related Issues
Fixes #375

Tested with aws a3 and also functions properly.


Fixed ThermalStorageResults having multiple entries

resolves #924

ThermalGrid.results() will carry out both, thermalHouse and thermalStorage results whenever one of both got a change of its state. Since the state of the other hasn't changed, it was written out with the tick of the last change of it. This can and should be changed within another PR (Issue needs to be created). As workaround the tick of the result that triggered the output of the results will be used for both.

Good idea! Thanks.

@danielfeismann Adapted the code a bit for simplicity. Let me know what you think

Looks good to me!


DEP PHP8 support

Issue https://github.com/silverstripe/silverstripe-totp-authenticator/issues/70
Have tested locally on php 8, things worked as expected
Changelog for dependency upgrade indicates that it's straightforward https://github.com/Spomky-Labs/otphp/blob/v10.0/doc/UPGRADE_v9-v10.md

Tested on all our platforms


3.2.0 breaks WASM compilation

Version
3.2.0
Reproduction link
https://github.com/diachedelic/vue-cli-wasm-repro
Node and OS info
Node v8.12.0 / yarn 1.10.1 / MacOS 10.14.1
Steps to reproduce
git clone https://github.com/diachedelic/vue-cli-wasm-repro.git &&
  cd vue-cli-wasm-repro &&
  yarn install &&
  yarn build

To fix, downgrade @vue/cli-service:
yarn add '<EMAIL_ADDRESS>&& yarn build

What is expected?
Project builds without errors, like it did with<EMAIL_ADDRESS>What is actually happening?
Log from repro
 ERROR  Failed to compile with 1 errors                                                                    15:13:41

This dependency was not found:

* env in ./node_modules/multimath/lib/unsharp_mask/unsharp_mask.wasm

Log from my actual project
$ vue-cli-service build

 ERROR  ModuleNotFoundError: Module not found: Error: Can't resolve 'env' in '/Users/me/Code/proj/node_modules/multimath/lib/unsharp_mask'
ModuleNotFoundError: Module not found: Error: Can't resolve 'env' in '/Users/me/Code/proj/node_modules/multimath/lib/unsharp_mask'
    at factory.create (/Users/me/Code/proj/node_modules/webpack/lib/Compilation.js:821:10)
    at factory (/Users/me/Code/proj/node_modules/webpack/lib/NormalModuleFactory.js:397:22)
    at resolver (/Users/me/Code/proj/node_modules/webpack/lib/NormalModuleFactory.js:130:21)
    at asyncLib.parallel (/Users/me/Code/proj/node_modules/webpack/lib/NormalModuleFactory.js:224:22)
    at /Users/me/Code/proj/node_modules/neo-async/async.js:2825:7
    at /Users/me/Code/proj/node_modules/neo-async/async.js:6886:13
    at normalResolver.resolve (/Users/me/Code/proj/node_modules/webpack/lib/NormalModuleFactory.js:214:25)
    at doResolve (/Users/me/Code/proj/node_modules/enhanced-resolve/lib/Resolver.js:184:12)
    at hook.callAsync (/Users/me/Code/proj/node_modules/enhanced-resolve/lib/Resolver.js:238:5)
    at _fn0 (eval at create (/Users/me/Code/proj/node_modules/tapable/lib/HookCodeFactory.js:32:10), <anonymous>:15:1)
    at resolver.doResolve (/Users/me/Code/proj/node_modules/enhanced-resolve/lib/UnsafeCachePlugin.js:37:5)
    at hook.callAsync (/Users/me/Code/proj/node_modules/enhanced-resolve/lib/Resolver.js:238:5)
    at _fn0 (eval at create (/Users/me/Code/proj/node_modules/tapable/lib/HookCodeFactory.js:32:10), <anonymous>:15:1)
    at hook.callAsync (/Users/me/Code/proj/node_modules/enhanced-resolve/lib/Resolver.js:238:5)
    at _fn0 (eval at create (/Users/me/Code/proj/node_modules/tapable/lib/HookCodeFactory.js:32:10), <anonymous>:12:1)
    at resolver.doResolve (/Users/me/Code/proj/node_modules/enhanced-resolve/lib/DescriptionFilePlugin.js:42:38)
error Command failed with exit code 1.

Changes to yarn.lock:
diff -r 7dccb1b3ad20 -r dcd6c7f55d03 yarn.lock
--- a/yarn.lock	Wed Nov 28 15:31:37 2018 +1100
+++ b/yarn.lock	Wed Nov 28 15:32:35 2018 +1100
@@ -969,7 +969,7 @@
     babel-plugin-transform-vue-jsx "^4.0.1"
     core-js "^2.5.7"

-"<EMAIL_ADDRESS>+"<EMAIL_ADDRESS>   version "3.2.0"
   resolved<EMAIL_ADDRESS>   integrity sha512-RKMSfgTtRs4VOXQhmbrNZJaCCheshebji9NcPNGyXzukCMBtoAbu3cG9HxizCSUA//oFFAdPP5BGeHvv0cpu/A==
@@ -1003,18 +1003,18 @@
     "@vue/cli-shared-utils" "^3.2.0"
     workbox-webpack-plugin "^3.6.3"

-"<EMAIL_ADDRESS>-  version "3.2.0"
-  resolved<EMAIL_ADDRESS>-  integrity sha512-HTiaz1IBXV/JCfrmyhHJvDOYpPTBK0uQAekSVRTt5AddCeOV68ktdUqHbDe3VPDfrWFA5x3d3kIHlJd6WK31mA==
+"<EMAIL_ADDRESS>+  version "3.1.4"
+  resolved<EMAIL_ADDRESS>+  integrity sha512-dArLh5yd2B2an/jTD+cmkRmyRxKSE+5il9408vhciyqvZGVjkajTN95YjY5VLkBMQvrv/Z+goGSn+7wtnQSOFg==
   dependencies:
     "@intervolga/optimize-cssnano-plugin" "^1.0.5"
-    "@vue/cli-overlay" "^3.2.0"
-    "@vue/cli-shared-utils" "^3.2.0"
+    "@vue/cli-overlay" "^3.1.0"
+    "@vue/cli-shared-utils" "^3.1.1"
     "@vue/preload-webpack-plugin" "^1.1.0"
     "@vue/web-component-wrapper" "^1.2.0"
-    acorn "^6.0.4"
-    acorn-walk "^6.1.1"
+    acorn "^6.0.2"
+    acorn-walk "^6.1.0"
     address "^1.0.3"
     autoprefixer "^8.6.5"
     cache-loader "^1.2.5"
@@ -1029,7 +1029,7 @@
     escape-string-regexp "^1.0.5"
     file-loader "^2.0.0"
     friendly-errors-webpack-plugin "^1.7.0"
-    fs-extra "^7.0.1"
+    fs-extra "^7.0.0"
     globby "^8.0.1"
     hash-sum "^1.0.2"
     html-webpack-plugin "^3.2.0"
@@ -1037,7 +1037,7 @@
     lodash.defaultsdeep "^4.6.0"
     lodash.mapvalues "^4.6.0"
     lodash.transform "^4.6.0"
-    mini-css-extract-plugin "^0.4.5"
+    mini-css-extract-plugin "^0.4.4"
     minimist "^1.2.0"
     ora "^3.0.0"
     portfinder "^1.0.19"
@@ -1052,14 +1052,14 @@
     thread-loader "^1.2.0"
     url-loader "^1.1.2"
     vue-loader "^15.4.2"
-    webpack "^4.26.1"
+    webpack "^4.18.1"
     webpack-bundle-analyzer "^3.0.3"
     webpack-chain "^4.11.0"
     webpack-dev-server "^3.1.10"
     webpack-merge "^4.1.4"
     yorkie "^2.0.0"

-"<EMAIL_ADDRESS>"<EMAIL_ADDRESS>+"<EMAIL_ADDRESS>"<EMAIL_ADDRESS>"<EMAIL_ADDRESS>   version "3.2.0"
   resolved<EMAIL_ADDRESS>   integrity sha512-FCX5ABFg5pWhomyXLpCaogJktMvjsS5d4Mn5BfvqcJxCvzOX6ze8ihFK3u//XMeM78dOFpHSjxnRSvHtkEwgsg==
@@ -1307,7 +1307,7 @@
   resolved "https://registry.yarnpkg.com/acorn-jsx/-/acorn-jsx-5.0.1.tgz#32a064fd925429216a09b141102bfdd185fae40e"
   integrity sha512-HJ7CfNHrfJLlNTzIEUTj43LNWGkqpRLxm3YjAlcD0ACydk9XynzYsCBHxut+iqt+1aBXkx9UP/w/ZqMr13XIzg==

<EMAIL_ADDRESS><EMAIL_ADDRESS>   version "6.1.1"
   resolved "https://registry.yarnpkg.com/acorn-walk/-/acorn-walk-6.1.1.tgz#d363b66f5fac5f018ff9c3a1e7b6f8e310cc3913"
   integrity sha512-OtUw6JUTgxA2QoqqmrmQ7F2NYqiBPi/L2jqHyFtllhOUvXYQXf0Z1CYUinIfyT4bTCGmrA7gX9FvHA81uzCoVw==
@@ -1322,7 +1322,7 @@
   resolved "https://registry.yarnpkg.com/acorn/-/acorn-5.7.3.tgz#67aa231bf8812974b85235a96771eb6bd07ea279"
   integrity sha512-T/zvzYRfbVojPWahDsE5evJdHb3oJoQfFbsrKM7w5Zcs++Tr257tia3BmMP8XYVjp1S9RZXQMh7gao96BlqZOw==

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>   version "6.0.4"
   resolved "https://registry.yarnpkg.com/acorn/-/acorn-6.0.4.tgz#77377e7353b72ec5104550aa2d2097a2fd40b754"
   integrity sha512-VY4i5EKSKkofY2I+6QLTbTTN/UvEQPCo6eiwzzSaSWfpaDhOmStMCMod6wmuPciNq+XS0faCglFu2lHZpdHUtg==
@@ -4506,7 +4506,7 @@
     jsonfile "^4.0.0"
     universalify "^0.1.0"

<EMAIL_ADDRESS><EMAIL_ADDRESS>   version "7.0.1"
   resolved "https://registry.yarnpkg.com/fs-extra/-/fs-extra-7.0.1.tgz#4f189c44aa123b895f722804f55ea23eadc348e9"
   integrity sha512-YJDaCJZEnBmcbw13fvdAM9AwNOJwOzrE4pqMqBq5nFiEqXUqHwlK4B+3pUw6JNvfSPtX05xFHtYy/1ni01eGCw==
@@ -6307,7 +6307,7 @@
   resolved "https://registry.yarnpkg.com/mimic-fn/-/mimic-fn-1.2.0.tgz#820c86a39334640e99516928bd03fca88057d022"
   integrity sha512-jf84uxzwiuiIVKiOLpfYk7N46TSy8ubTonmneY9vrpHNAnp0QBt2BxWV9dO3/j+BoVAb+a5G6YDPW3M5HOdMWQ==

<EMAIL_ADDRESS><EMAIL_ADDRESS>   version "0.4.5"
   resolved "https://registry.yarnpkg.com/mini-css-extract-plugin/-/mini-css-extract-plugin-0.4.5.tgz#c99e9e78d54f3fa775633aee5933aeaa4e80719a"
   integrity sha512-dqBanNfktnp2hwL2YguV9Jh91PFX7gu7nRLs4TGsbAfAG6WOtlynFRYzwDwmmeSb5uIwHo9nx1ta0f7vAZVp2w==
@@ -10125,7 +10125,7 @@
     source-list-map "^2.0.0"
     source-map "~0.6.1"

<EMAIL_ADDRESS><EMAIL_ADDRESS>   version "4.26.1"
   resolved "https://registry.yarnpkg.com/webpack/-/webpack-4.26.1.tgz#ff3a9283d363c07b3494dfa702d08f4f2ef6cb39"
   integrity sha512-i2oOvEvuvLLSuSCkdVrknaxAhtUZ9g+nLSoHCWV0gDzqGX2DXaCrMmMUpbRsTSSLrUqAI56PoEiyMUZIZ1msug==

So the issue appears to be in @vue/cli-service as no other package's version resolved differently after downgrading from v3.2.0 to v3.1.4

A temporary fix:
// vue.config.js
module.exports = {
  chainWebpack: config => config.resolve.extensions.delete('.wasm')
}

It is because in 3.2 we added .wasm extension to the resolve list (which is also default in webpack).
And in the library https://github.com/nodeca/multimath/blob/master/lib/unsharp_mask/index.js they put the .wasm file of the same name with the js file in the same directory and require the js file without extension…

It worked!


Issue with generate_output_file (shutil.move: OSError: [Errno 18] Invalid cross-device link)

Description
A user is reporting this issue on spinalcordmri.org. At first glance, it looks like they are using SCT across multiple different filesystems, which might be causing issues when we call shutil.move.
I am still gathering information, but preliminary searching leads me to believe a fix might be needed on our end, because the issue seems very similar to https://github.com/ewels/MultiQC/issues/806, which suggests:

shutil.move command doesn't work correctly if source and dest are in different file systems

There is some very useful analysis in this comment here: https://github.com/ewels/MultiQC/issues/806#issuecomment-601113985, and they were able to implement a fix here: https://github.com/ewels/MultiQC/pull/1130
I'm creating this issue in advance so that we can discuss, and to be able to quickly let the user know we are addressing their issue if need be.
Steps to Reproduce

TODO: Update based on response from user to these questions.

Expected behavior: Temporary files are copied successfully.
Actual behavior: Throws the following error:

 Long console output hidden, click to show
Generate output files…
WARNING: File PAM50_t1_reg.nii.gz already exists. Deleting it…
File created: PAM50_t1_reg.nii.gz
WARNING: File subj01_eddy_fa_crop_reg.nii already exists. Deleting it…
Traceback (most recent call last):
File “/home/jnavas/sct_4.3/python/envs/venv_sct/lib/python3.6/shutil.py”, line 550, in move
os.rename(src, real_dst)
OSError: [Errno 18] Invalid cross-device link: ‘/tmp/sct-20201006165629.329375-register-cws9p3au/dest_reg.nii’ -> ‘subj1_eddy_fa_crop_reg.nii’

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File “/home/jnavas/sct_4.3/scripts/sct_register_multimodal.py”, line 372, in
main()
File “/home/jnavas/sct_4.3/scripts/sct_register_multimodal.py”, line 348, in main
path_out=path_out)
File “/home/jnavas/sct_4.3/scripts/msct_register.py”, line 338, in register_wrapper
os.path.join(path_tmp, “dest_reg.nii”), os.path.join(path_out, file_out_inv + ext_dest), param.verbose)
File “/home/jnavas/sct_4.3/scripts/sct_utils.py”, line 749, in generate_output_file
shutil.move(fname_in, fname_out)
File “/home/jnavas/sct_4.3/python/envs/venv_sct/lib/python3.6/shutil.py”, line 564, in move
copy_function(src, real_dst)
File “/home/jnavas/sct_4.3/python/envs/venv_sct/lib/python3.6/shutil.py”, line 264, in copy2
copystat(src, dst, follow_symlinks=follow_symlinks)
File “/home/jnavas/sct_4.3/python/envs/venv_sct/lib/python3.6/shutil.py”, line 205, in copystat
follow_symlinks=follow)
PermissionError: [Errno 1] Operation not permitted

I am still gathering information, but preliminary searching leads me to believe a fix might be needed on our end, because the issue seems very similar to ewels/MultiQC#806, which suggests:

I agree, it should be our responsibility to not assume that /tmp has write permission (IIUC the problem)

The user has reported two things:

"the /media/Data/ is a shared Server of my Insitute."
"The script runs without errors on my local folder. So the problem was solved."

I no longer think that this is urgent, because this use-case (running script on one filesystem but accessing data on another) seems uncommon for our users, and the error itself is supposed to have been fixed with Python3 (see: this stackoverflow comment chain). Closing for now.


Jump in distribution

Hello!
I am trying to compare a MR relationship to existing kepler data however the forecast function jumps drastically and I was wondering if I am using forecast properly?

import mr_forecast as mr
import pandas
import numpy as np
import matplotlib.pyplot as plt

def get_data(keys=['pl_pnum','pl_radj','pl_radjerr1','pl_radjerr2','pl_bmassj']):
    # query the api for the newest data
    api_url = 'https://exoplanetarchive.ipac.caltech.edu/cgi-bin/nstedAPI/nph-nstedAPI?table=exoplanets&format=json&select='
    query_url = api_url+','.join(keys)
    return pandas.read_json( query_url )

data = get_data()
nulls = ~data.isnull().any(axis=1)
data = data[nulls]

std = 0.5*(data['pl_radjerr1'] + data['pl_radjerr2'])

pred, up, low = [],[],[]
for i in range(data.shape[0]):
    if i%50 ==0:print(i)
    Mmedian, Mplus, Mminus = mr.Rstat2M(data['pl_radj'].values[i], abs(std.values[i]), unit='Jupiter', sample_size=100)
    pred.append(Mmedian)
    up.append(Mplus)
    low.append(Mminus)

up = np.array(up)
low=np.array(low)

f,ax = plt.subplots(2)
ax[0].plot( np.log10(data['pl_bmassj']),data['pl_radj'],'k.',label='Measured')
#ax.plot(np.log10(pred),data['pl_radj'],'ro')

ax[0].errorbar(np.log10(pred),data['pl_radj'] ,xerr=[np.log10(low),np.log10(up)],color='magenta',ecolor='red',alpha=0.5,fmt='.',errorevery=4,label='forcaster')
ax[0].set_xlim([-2,1.7])
ax[0].set_ylim([0.01,2.15])
ax[0].set_xlabel('Log10( Mass [Jupiter] )')
ax[0].set_ylabel('Radius [Jupiter]')

ax[1].hist( np.log10(pred) - np.log10(data['pl_bmassj']),bins=20)
ax[1].set_xlabel('Estimate - Measured')

plt.show()

bump.
Any insight into what could be causing this jump in the distribution?

I am also trying to use this to get Jupiter-mass posteriors from Radius measurements-- any fixes on this yet?

Hi,
I just noticed the same inconvenience when trying to get giant planet masses from their radii. No reliable at all the derived masses :(
Has anyone solved this?
Thanks!


Select all default for language search

Fix for #155 .
Also fixed problems with language classifications badge value.

Current coverage is 64.67%

Merging #191 into master will not affect coverage as of d73ba73

@@            master    #191   diff @@
======================================
  Files          101     101
  Stmts         2955    2955
  Branches         0       0
  Methods          0       0
======================================
  Hit           1911    1911
  Partial          0       0
  Missed        1044    1044

Review entire Coverage Diff as of d73ba73

Powered by Codecov. Updated on successful CI builds.


clarify that the url is a web socket

Problem
The names of the env vales are confusing.
Link to GitHub Issue(s): #567
Solution
Clarify that one is a websocket.
Steps to Verify:

Run it and make sure it still works
No more instances of "FREQUENCY_URL"

Sorry; just getting around to looking at this--last 2 weeks with hurricane & off-site have me in catch-up mode.
A couple of points:

We've changed the name of the env var to include _WS_, but the config validation still allows http protocol
Technically, there's no reason you couldn't specify an HTTP RPC endpoint; WebSocket is only required if you're doing subscriptions, which we avoid.

We should either disallow http protocol in the config, or eliminate specifying WebSocket/WS in the docs & env name.
I'd be okay with disallowing http, since it's less efficient (needs to establish a connection with each request); however, we might want to consider that some client might have network constraints in their environment that preclude using WebSocket.


React.createRef type doesn't match actual API

See https://github.com/facebook/flow/pull/5920#discussion_r178429127 and https://github.com/facebook/flow/pull/5920#issuecomment-379057682.
The typing that was merged uses an old version of the createRef API -- different from the one that eventually landed and was released in React 16.3.
value needs to be changed to current.

A fix is landing.


Github token permissions

The documentation currently suggests creating a github token with permissions to everything, including delete_repo and admin:write:org. This isn't great, could you please find out what is actually necessary and update the docs?

repo:public_repo seems to be enough for public repositories - but that's getting updated data with a setup that was originally set up with much greater access.

We probably also need read:user and user:email for contributor information.

read:org is probably needed for teams, collaborators. There's also an events but don't know yet what that requires.
Seems X-Accepted-OAuth-Scopes response header described in https://developer.github.com/apps/building-oauth-apps/understanding-scopes-for-oauth-apps/ could be used to narrow down the requirement.


OCN's new attack speed is 5x

Well, so the last update was Pablete setting attack speed to 3x, but it has been changed to 5x now. You can check it out here.
https://oc.tc/forums/topics/5701ebc75f35b9140c000013

@MrSharpie multiplying it x3, makes it 4 times faster, but still, look at #994, that should fix your issue

fixed in fd4498f1a6724851055f486edc5a00c45156fd2e


Max height for panes

I'm trying to solve #3 .
I haven't committed all codes and I still need to clean up some logic here and there, but this is a preview of what I have currently:

hi. I've finished this PR and also included #1 in it. let me know if you need anything else to  merge it in

I'll test this out locally today, thanks!

This looks good, but it seems like the panes are correctly resizing if you change the terminal window size.
I think it's alright to merge for now, but if you have interest you could look into fixing that next.

I've released this in 0.2.0, and updated my homebrew tap (in case you were using that, you should be able to brew update && brew upgrade)


Questions on Adding Shape of each points to visualize 2nd Categorical Value

Wondering if possible, customize the feature in build scripts?

Unfortunately this library is not in active development, but we will re-open this ticket if that changes!


MOROCO

Add MOROCO to huggingface datasets.

@lhoestq Thank you for all the feedback. I've added the suggested changes in my last commit.


attributes.windowLevel = .custom(level: lastWindow.windowLevel + 1)

Describe the bug
attributes.windowLevel = .custom(level: lastWindow.windowLevel + 1)
This will cause error, I have to rollback to 0.6.0 version

Hi @linshizai - It is probably because the Swift version in the build settings of the target is less than 4.2 (is it?). In case you don't use Swift 4.2 please pod install 'SwiftEntryKit' '0.7.2'.

.
Let me know your thoughts.

@huri000 Thank you very much, after switch to 0.7.2, the problem has been solved

After switch to 0.7.2, it will cause the problem below. In 0.6.0 version,it's used normally

@huri000 Xcode 10, Swift4.0

@huri000 iOS12

Hi, I've managed to reproduce the crash after lots of playing around with the Swift version and the installed pod version.
But, pod deintegrate, Podfile.lock deletion and pod install fix the issue and all goes back to normal. I think it is happens because something is wrong with the build process.
Could you please try what I suggessted above and let me know? Thanks!

Yes!! After reload pods, It seems everything  successfully


Minor issue when launch app in v2.2.5-rc2

Version: v2.2.5-rc2
This is a minor issue
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

At the global level click on Apps and just before you create any Multi-Cluster app watch just above the Launch button and click launch

Result:
When you click on Launch you will see a red ribbon flash very briefly on the screen.
When you inspect the http traffic you will see that we made a GET request to get a generic svg icon but it was either canceled (in some cases) or was an HTTP 404

Other details that may be helpful:
This doesn't appear to hurt anything. And generic SVG icon for apps without icon still work fine. This doesn't appear to have any functional issues based on my testing in this area and general smoke testing around apps overall.
I don't think this should necessarily gate v2.2.5 release if it is not a very easy fix.

I still observe the issue. Reopening.

Additional: My app is missing the svg icon so this is breaking that icon sometimes.

@loganhz or @vincent99 is the svg icon not showing for the app related (or no and I should create a separate issue)? Thanks
Pictured is docker-registry app

No browser dev console errors, no 404's or 500s

@davidnuzik please file a new issue for it. Thanks!

Created issue https://github.com/rancher/rancher/issues/22152 to track the svg icon issue. Looks like we might be blocking loading SVG icons (CORS).

Version: master-head (v2.3) (8/13/19)
I followed my steps to reproduce and I no longer observe the issue. The error no longer briefly flashes on the screen when I launch my apps.


Adds a bootstrap dismissable warning about caps lock

ref: https://github.com/siddhantgoel/stuckinvim.com/pull/5#issuecomment-311089979

Looks good! Two comments -

Could you adjust the width of the warning? You'll need to move the warning content to a different row (so that the question text appearance does not change) and basically center the warning. Something like

... warning ...

 should work.
Bringing in bootstrap/jquery for only one close button feels heavy weight, especially when Vue is already there on the page. Could you use VueJS transitions for this - https://vuejs.org/v2/guide/transitions.html#Transitioning-Single-Elements-Components ?

@siddhantgoel That should do iit.

Should be ready to merge now. Let me know if there's anything I missed.

Merged, thanks!


What to do if conda build fails because submodule not cloned

Since #756 the data dir is now a submodule.
If this repo is cloned with git clone git@github.com:vais-ral/CCPi-Framework.git the submodule will not be cloned as well and the data dir will be empty.
 To have the submodule installed issue the following
$ git submodule update --init
Submodule 'Wrappers/Python/data' (https://github.com/vais-ral/CIL-data.git) registered for path 'Wrappers/Python/data'
Cloning into 'Wrappers/Python/data'...
remote: Enumerating objects: 16, done.
remote: Counting objects: 100% (16/16), done.
remote: Compressing objects: 100% (14/14), done.
remote: Total 16 (delta 1), reused 12 (delta 1), pack-reused 0
Unpacking objects: 100% (16/16), done.
Submodule path 'Wrappers/Python/data': checked out '77bfd47b7116a1c91e77af8dd0420fc2bc4e566c'

Now you'll have the data directory filled with whatever is at the CIL-data repo.

In readme documentation https://github.com/TomographicImaging/CIL#getting-the-code


[BUGish] Move "mint tokens" to the tokens "manage" modal

Description
Currently the button allowing the project owner to mint tokens is front and center above the pay button. This button doesn't need to be so prominent, and may seem to be visible to everyone instead of just the project owner.
Proposal
A more intuitive place for this might be to move the button to the tokens "manage" menu.
Screenshots

fixed


feat(core): add dangerous option to disable compile time CSP injection

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[ ] No

Checklist

[ ] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information

I think the naming could be improved here, disable_asset_csp makes it sound as if it would disable the whole csp mechanism. I have no idea for better names either though

Is dangerous_disable_asset_csp_injection better? @JonasKruckenberg this is so hard :joy:


Latest release dates back to April 27th, release tag 0.0.5

Would it be possible to push a new release with all fixes/enhancements issued in the meantime?
That would help those that use bower to manage dependencies.
Thanks a lot for the great work!

Yes of course, I'm going to release a new version just now.


Unable to open temporary file in Windows

Description
While running test_3to2_conversion(self), the program crashed for me. Going down the stack trace, the issue appears within the doc_serialize_rdf2xml(self, outfile): function  in document.py. When this function is called on a Windows computer, the program crashes due to how Windows handles the NamedTemporaryFile() function.
Stack Trace
Error
Traceback (most recent call last):
  File "C:\Users\Yehuda\SBOL-Utilities\test\test_sbol2_sbol3_direct.py", line 27, in test_3to2_conversion
    doc2.write(tmp2.name)
  File "C:\Users\Yehuda\SBOL-utilities\venv\lib\site-packages\sbol2\document.py", line 404, in write
    self.doc_serialize_rdf2xml(filename)
  File "C:\Users\Yehuda\SBOL-utilities\venv\lib\site-packages\sbol2\document.py", line 810, in doc_serialize_rdf2xml
    with open(outfile, 'w') as out:
PermissionError: [Errno 13] Permission denied: 'C:\\Users\\Yehuda\\AppData\\Local\\Temp\\tmp7jshttyj.xml'

Explanation
I found this Stack Overflow Thread to be helpful.
The cause of the bug seems to be documented by this python bug report.
According to Richard Oudkerk:

(...) the only reason that trying to reopen a NamedTemporaryFile fails on Windows is because when we reopen we need to use O_TEMPORARY.
and he gives an example of how to do this in Python 3.3+

import os, tempfile

DATA = b"hello bob"

def temp_opener(name, flag, mode=0o777):
    return os.open(name, flag | os.O_TEMPORARY, mode)

with tempfile.NamedTemporaryFile() as f:
    f.write(DATA)
    f.flush()
    with open(f.name, "rb", opener=temp_opener) as f:
        assert f.read() == DATA

assert not os.path.exists(f.name)

How to Reproduce
Run the test_3to2_conversion(self) function in test/test_sbol2_sbol3_direct.py on a Windows computer
Environment
OS: Windows 10 Home
Device name: LAPTOP-BJRJGSN3
Processor: 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz   2.42 GHz
System type: 64-bit operating system, x64-based processor
Fixing the Bug
I am currently working on getting the bug fixed, although I am having some trouble, as I am not so familiar with the code. If anyone can give me some advice, then I would much appreciate it.

Looks like there are actually two bugs here:

The Windows CI isn't working right, or else we would have seen this before it broke for you.
The temporary file bug.

I can see a good work-around for the temporary file bug: rather than making a temporary file, make a temporary directory. Files can then be made in the temporary directory that will not suffer the same problem.

I believe that I've resolved both this and the underlying testing failure that allowed it to happen. Please re-open if it didn't fix it for you.


Define a Tupleable instance with a subrelation

Right now, haskell datatypes with list attributes are derived as  t :: List (a :: SomeTypeT) is there a way of forcing the deriving mechanism to use t :: relation {a::SomeTypeT} instead?

I couldn't find the code but I once wrote something like an instance Atomable a => Atomable Set a where toAtom =  -- relation{element a})
And the Tupleable deriving mechanism works pretty well.
For what its worth, relation{element a} is a set and cannot keep duplicated elements. relation{index Integer, element a} can.

Thanks, I will try that!
Do you guys think it's something useful to have? Should I make a pull request?

A shortcut here could be to wrap the specific list item with newtype and add an Atomable instance for it that does what you want. In general, Tupleable and Atomable don't support subrelations by default because doing so would make the type variables more annoying.
Certainly, if you make some progress on this front, a PR would be much appreciated.

@Valdsonjr , were you able to resolve this issue?

@agentm  I made it work with this ugly thing:
{-# LANGUAGE ScopedTypeVariables #-}

-- 'throws' something that is not a member of the Exception type class
throwIfLeft :: (Show a) => Either a b -> b
throwIfLeft = either (error . show) id

instance (Tupleable a, NFData a, Ord a, Serialise a, Show a) => Atomable (Set a) where
    toAtom :: Set a -> Atom
    toAtom set = RelationAtom (Relation attrs tuples)
        where
        tuples = RelationTupleSet $ map toTuple $ toList set
        attrs = toAttributes (Proxy :: Proxy a)

    fromAtom :: Atom -> Set a
    fromAtom (RelationAtom (Relation _ (RelationTupleSet tuples))) =
        fromList $ map (throwIfLeft . fromTuple) tuples
    fromAtom _ = error "wrong Atom constructor for (Set a)"

    toAtomType :: proxy (Set a) -> AtomType
    toAtomType _ = RelationAtomType (toAttributes (Proxy :: Proxy a))

I really wish I could remove some of those constraints, but at least it works for my toy programs

That code is pretty straightforward- I'll wrap it into Project:M36, if you don't mind.
What bothers you about the constraints?

@agentm Sure! I can open the PR tomorrow night if you want. (and add the toAddTypeExpr)

What bothers you about the constraints?
I felt like there were too many constraints but on second thought  I don't know of any other way


Add macos targets

Build and publish macos (arm64 and x64) libraries.

Note: secp256k1KmpVersion must be updated once https://github.com/ACINQ/secp256k1-kmp/pull/104 is merged and secp256k1-kmp is released


Add a Contribution guide

We need to add a guide for contributors, to outline some of the development choices we are using. This GitHub article seems to be a good guide for creating the CONTRIBUTING file.
Some links that might be helpful to add could be:

15 rules for communicating at GitHub
GitHub Flow branching strategy

This also needs to describe testing, as mentioned in #2 .

Description of documentation development should include the following:
livereload
The very useful tool livereload for monitoring
changes in Sphinx documentation in real-time is included
in the turbopy conda environment (environment.yml file).
Here’s a simple script that detects changes in any *.rst files in doc or *.py files in turbopy,
rebuilds the Sphinx documentation, and starts a livereload server:
#!/usr/bin/env python
from livereload import Server, shell
import formic

PATTERNS = ["doc/**.rst", "turbopy/**.py"]

SERVER = Server()
for filepath in formic.FileSet(include=PATTERNS):
    SERVER.watch(filepath, shell('make html', cwd='doc'))
SERVER.serve(root='doc/_build/html')

Run it inside the root directory of your clone of the turbopy repo, then open
http://localhost:5500/, and you can see any
documentation changes you are making in real time.
Sphinx cross-references
If you are having trouble determining the syntax for a Sphinx cross-reference (internal
or external), try the command-line tool https://github.com/bskinn/sphobjinv,
which is included in the turbopy conda environment (environment.yml file).

I'd also like to start using templates for issues and pull requests. They should be described in the contribution guide.

Also need to add info about the branch naming convention that we discussed.

@littlewatkins I've assigned this to you. Feel free to tag some interns to help you with this 😄

I made a first pass at a contributing markdown and noted in the pull request what should get added. Will continue adding tomorrow

@NRL-Plasma-Physics-Division/interns The contributing guide still needs a section that describes documentation. It should note that we use numpydoc style docstrings, and that the documentation gets generated automatically at ReadTheDocs.
Anyone interested in writing that section?

@arichar6 I'll do that this afternoon

I think that with #91, we have completed a minimal contributing doc. Might be nice to add the livereload stuff from Paul's comment above, but I don't that's necessary at this time.

I take back my suggestion on livereload. It's a useful tool for seeing real time edits to documentation, but there are many ways to do that...doesn't really belong in contributing.md IMHO.


adding nb_conda to conda script and updating setup docs

Description
Includes nb_conda in conda packages (which also installs nb_conda_kernels). Supports automatic installation of conda envs for jupyter and ability to inspect them through conda tab.

note there will likely need to be some work done to the test setup to ensure this is supported there

Related Issues
#665
Checklist:

[X] My code follows the code style of this project, as detailed in our contribution guidelines.
[ ] I have added tests.
[X] I have updated the documentation accordingly.

also I win the 😈 prize!


Rename Visual Studio Online to VSTS

Hi @TomShinder, Would you review the proposed changes? Thanks.

@shanecastle-osharp Thank you for taking the time to make that change!!
@arob98
@erickson-doug
#sign-off


something weird with the combined results report of 2 accounts

Initially I thought this was a bug due to the combination of my 2 aws accounts, however digging deeper I see that this is misidentification even when only 1 account is present.
In my EC2 VPC reservation section I am getting multiple 'NOT RESERVED'  listing instance id's that are ec2 classic instances.

I have no idea what was going on, but I cannot repro this anymore.  closing.

Digging a little deeper it seems like the issue is that when it displays the instances in the 'NOT RESERVED' it doesn't really discriminate between nodes.   I get the entire list of m3.large across us-east-1c vpc+non vpc in every 'NOT RESERVED' line.


How to turn off css compression?

new HtmlWebpackPlugin({
      inject: true,
      template: paths.appHtml,
      minify: {
        removeComments: true,
        collapseWhitespace: true,
        removeRedundantAttributes: true,
        useShortDoctype: true,
        removeEmptyAttributes: true,
        removeStyleLinkTypeAttributes: true,
        keepClosingSlash: true,
        minifyJS: true,
        minifyCSS: true,
        minifyURLs: true
      }
    })

Is the setting here?

No. This is confusing but in Webpack 1, you need to explicitly opt out of compression in the loader definition. It gets turned out implicitly when Uglify plugin is used. This behavior is changed in Webpack 2 but we are not using it yet. See https://github.com/webpack-contrib/sass-loader/issues/299.


Email not working on the staging server

Describe the bug
When joining a private or hidden group an email should be sent to the user to notify them that they've been invited. Those emails are not being sent. The reply-by-email functionality is also not working.
To Reproduce
Steps to reproduce the behavior:

Log in as hctester50
Go to the group "Dave's Group 2023"
Note the email address for the group
Error returned: "Your message couldn't be delivered to<EMAIL_ADDRESS>because the remote server is misconfigured. See technical details below for more information."

Expected behavior
User should be able to create topics and reply-by-email and also receive emails when invited to groups.
Context
Does this bug on production, development, or both?

[x] Tested on development instance
[x] Occurs on development instance
[x] Tested on production
[ ] Occurs on productions

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

I believe this should be working now.

Email is working!!


mixed mode debugging problem with Symbols

I'm struggling with using mixed mode debugging of a Python Tools for Visual Studio project that contain a c extension for my Python script. I downloaded the Symbols and specified the path to the Symbols of the Python interpreter being used (2.7.10: 32-bit).
When attempting to debug, Microsoft Visual keeps asking for the Symbols. I'm able to debug the Python code when "native code debugging" is disabled.
Any leads?
I'm using the following:
Microsoft Visual Studio Community 2015
Version 14.0.24720.00 Update 1
Python Tools for Visual Studio   2.2.31124.00
Python 2.7.10: 32bit

It may be that the symbols do not match - and that may not be your fault. I do recall there being an upload incompatibility between a recent version of Python 2.7 and the symbols that came with it. I'm not currently able to check that easily though, but someone else on the team might (easiest way to check is to attach to python.exe without Python debugging - just native - and try and load the symbols for python27.dll).

Thanks for the reply.
Looks like the symbols don't match.
Hope somebody can take a look and address it soon. I'm happy to share the project if it helps to reproduce the problem.
On Jan 8, 2016, 6:28 PM, at 6:28 PM, Steve Dower<EMAIL_ADDRESS>wrote:

It may be that the symbols do not match - and that may not be your
fault. I do recall there being an upload incompatibility between a
recent version of Python 2.7 and the symbols that came with it. I'm not
currently able to check that easily though, but someone else on the
team might (easiest way to check is to attach to python.exe
without Python debugging - just native - and try and load the
symbols for python27.dll).

Reply to this email directly or view it on GitHub:
https://github.com/Microsoft/PTVS/issues/1047#issuecomment-170155692

Try re-downloading Python 2.7 (even if the version on the website matches what you have installed). The current download for most recent version should match.

Thanks Pavel for the suggestion but I'm still struggling with this.
What I have done:

Downloaded the 32 version of Pyhton  2.7.10 from following the link
https://github.com/Microsoft/PTVS/wiki/Symbols-for-Python-mixed-mode-debuggingfrom
the git website.
Downloaded and installed Pyhton from Windows x86 MSI installer
https://www.python.org/downloads/release/python-2710/.
Followed instructions provided on the the git page - "Symbols for Python
mixed-mode debugging"

The mixed-mode debugging is partially working. It keep asking for the
symbols and also fails when the scope returns to python.
Any ideas?
German
On Fri, Jan 8, 2016 at 9:29 PM, Pavel Minaev<EMAIL_ADDRESS>wrote:

Try re-downloading Python 2.7 (even if the version on the website matches
what you have installed). The current download for most recent version
should match.
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/PTVS/issues/1047#issuecomment-170182491.

I've checked the symbols and installer from those links and they are fine, so something else must be going wrong here.
Can you try the following:

Start running python.exe outside of Visual Studio
In Visual Studio, select "Debug, Attach to Process"
Click "Select..." and manually choose only "Native Code" (ensure that "Python" is not selected)
Find and double-click "python.exe"
Open "Debug, Windows, Modules" and find "python.exe"
If it says symbols are loaded, then it should show the path to where you put the pdb file. If not, right click and select "Load symbols", then browse to where you put the pdb file and select it.

(Next time you go to attach to a process, you'll need to manually change the debug type back to automatic.)
Let us know what happens in the last step of those instructions. Screenshots of the Modules window may also be helpful.

thanks Steve for replying.
When attaching debugger to a process (per your
instructions) things seem to work. The path to the Symbols is as expected.
I'm attaching a screenshot.
When I run the debugger from inside the VS it asks for the symbols.
Any ideas?
On Thu, Jan 21, 2016 at 12:55 PM, Steve Dower<EMAIL_ADDRESS>wrote:

I've checked the symbols and installer from those links and they are fine,
so something else must be going wrong here.
Can you try the following:

Start running python.exe outside of Visual Studio
In Visual Studio, select "Debug, Attach to Process"
Click "Select..." and manually choose only "Native Code" (ensure
that "Python" is not selected)
Find and double-click "python.exe"
Open "Debug, Windows, Modules" and find "python.exe"
If it says symbols are loaded, then it should show the path to where
you put the pdb file. If not, right click and select "Load symbols", then
browse to where you put the pdb file and select it.

(Next time you go to attach to a process, you'll need to manually change
the debug type back to automatic.)
Let us know what happens in the last step of those instructions.
Screenshots of the Modules window may also be helpful.
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/PTVS/issues/1047#issuecomment-173653247.

@ls233 Attachments don't show up on GitHub when you reply directly to an email - you need to send it through the web site and drag/drop onto the text box.
But since it worked, we must have an issue on our side of things. Hopefully @int19h or I will have a chance to look deeper into it soon (it seems to work fine for us right now, and until we can reproduce the problem these issues are much harder to discover).
Perhaps you'd be able to try some other Python versions as well? Maybe roll back to 2.7.9 or try Python 3.5 or 3.4.

If attach finds symbols, but starting via F5 doesn't, I suspect that the problem is that F5 is actually using a different interpreter somehow.
When you're attaching, are you starting your project with Ctrl+F5 (start without debugging)? Or are you manually running python.exe from command line?

I precisely followed the instructions by zooba including starting python from command prompt outside of VS.
Here is the screenshot:

Interesting. Can you show your interpreter settings for this interpreter (in Tools -> Options -> Python Tools - where it shows all the paths)?

looks right to me.

Symbols doesn't for me with Python 3.5.2 too

My issue was because virtual environment create by Visual Studio somehow not copy PDB files from the folder with python to virtual environment folder

Ah, that's an interesting problem I hadn't thought of before. I wonder whether we can detect and insert the base environment as a symbol path automatically (and whether we should also get venv updated to copy the symbols if they are there).


Loading forever [BUG]

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Install Zipline and do the initial setup (through Docker in our case)
Go to /user/setup

Expected behavior
You shouldn't be able to go through the initial setup again. Going to that subdirectory is possible for anyone and poses a big security risk as it means anyone can just create an administrative account.
Screenshots

Desktop (please complete the following information if desktop):

OS: Windows 10 Pro 20H2
Browser: Firefox Developer Edition
Zipline Version: Newest upstream (self-built image with newest code from repo as of February 2)
^ This shouldn't be important for the bug as it seems to be backend-related

Browser Logs (for desktop browsers)
Not applicable for this bug

user/setup is accessible only when it's first setup, after you setup you can't access the page again

I tested it earlier and was able to access the setup page AFTER it's setup, so that's clearly not true.
If you want me to I can record a video of the bug later.

Sure, I did reproduce this and it didn't happen for me.

I don't know what the issue is then. The person whose server I administer created the original administrative account and I can navigate to https://i.example.com/user/setup and get access to that initial setup page.
EDIT as I was trying to record: It appears that I now get a "could not create" when attempting to do it again, which is really baffling since the bug did clearly exist (for our version at least) and just poofed into the void.
Any pointers on what could have caused this?

I'd guess some browser cache or something, not sure what happened. What database are you using?

SQLite, this happened to a person who didn't load it before and to me in two different browsers (Firefox and Edge)

as I'm not able to reproduce this the only way of fixing this on your end would be to delete the sql db file and I think it should work fine after that

Gotcha, I think that's not necessary as the bug resolved itself but I'll do some further testing later. If I find something interesting I'll try to fix it and send a PR. You can close this now as it's resolved. Thank you for your time!

Np!