drivers: all nrf: add hardware description for nrf kconfig files

Simplify Kconfig.nrfx for serial to use HAS_HW_NRF_UART0.
Signed-off-by: Jakub Rzeszutko<EMAIL_ADDRESS>

Purpose of this PR is to allow to display and configure in Kconfig only these drivers that are available for particular SoC.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8363   +/-   ##
=======================================
  Coverage   64.61%   64.61%
=======================================
  Files         422      422
  Lines       40284    40284
  Branches     6800     6800
=======================================
  Hits        26030    26030
  Misses      11121    11121
  Partials     3133     3133

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 8f3fea3...6a885b7. Read the comment docs.

@jarz-nordic I think such explanation (why all these config entries are introduced) should be contained in the commit message. And the change in Kconfig.nrfx for serial driver should be mentioned as an example of usage of these entries, or moved to a separate commit.

This is duplicate from information available in device tree.
So one will have to populate this, maintain SoC dtsi file and make sure that they are aligned (else I guess there will be some issues in build system)..
Not sure this will make things more simple.

Hi,
Yes, in some way this is duplicate information with DTS but it was done for the reason.
This is our interim solution to let Kconfig know what peripherals are available to display and configure only available hardware.
There is an ogoing discussion how to solve it in future:
https://github.com/zephyrproject-rtos/zephyr/issues/7302
but until that time we want to use proposed solution.
Our approach is simplar to proposals that are made, so we will need no much effort to adopt.

Fixed changes requested by @ioannisg

@jarz-nordic

This is our interim solution to let Kconfig ...
Thanks for explanation (it would have been useful to detail in initial PR description, but ok).

Looks good.
One last thing: "arch: nrf:" is wrong; we need "arch: arm: nrf: ..." instead.

I've changed prefix to: arch: arm: nrf:

My personal point of view is that Kconfig symbols to describe HW are duplicate information vs what should be extracted from dts. #7302 might eventually output to generation of some symbols that could then be used in Kconfig space.
Problem is that we're not even aligned on what output of #7302 should be.
I'm not sure everybody aligns on the need to be able to amend dts configuration by menuconfig.

This PR has the purpose of

allow to display and configure in Kconfig only these drivers that are available for particular SoC.

It is reasonable to expect this from the configuration system. Duplicating information between DTS and Kconfig seems reasonable to me until #7302 is resolved.

@MaureenHelm Does the NXP hardware support use a similar approach to this?

@jarz-nordic could we make some of those common to all drivers instead of them being Nordic-specific?

Does the NXP hardware support use a similar approach to this?

I've defined some HAS_* configs in ext/hal/nxp/mcux/Kconfig to set if a given peripheral is present in an SoC, and make the corresponding driver depend on that config. It's inconsistent though, because not every type of NXP peripheral has one of these configs, and thus there are cases that a user could incorrectly enable a driver for peripheral that doesn't exist in the SoC. I tried to reconcile this in #4317 but it was rejected on the grounds that I was introducing too many new configs.

@MaureenHelm thanks for the input. I spoke to @nashif and I think we need to reconsider this. It might take a while until we get a proper way to obtain DT info from Kconfig, so in the meantime we could have HAS_HW_ADC which would be shared by all hardware implementations, including NXP and Nordic. I suggest we put all of those in a centralized place instead of having them spread around the drivers/ folder.
So something like dts/common/Kconfig.peripherals where we define all of those. This file would be removed once we reach an agreement on how to properly fix this.

There are a few cases in the Kinetis family where we have multiple variants of a given peripheral type (e.g., SPI, DSPI, LPSPI), so it isn't sufficient to define a generic peripheral Kconfig option like HAS_HW_SPI

so it isn't sufficient to define a generic peripheral Kconfig option like HAS_HW_SPI

It should be ok to define HAS_HW_DSPI and HAS_HW_LPSPI which can be used by other vendors if they support them. I know some of those would be vendor specific and only one vendor might have them, but at least in this case it is consistent and safe for future support.
What I dislike is having those (many) HAS_HW_XXX defined for every vendor individually.

So something like dts/common/Kconfig.peripherals

I would not put this in dts/ and make it appear as a DTS thing, DTS does nothing with those configs, at some point when move this decision to DTS we can make the change and remove them all hopefully.
Waiting to hear from the participants of the meeting last week where DTS in general was discussed @erwango , @galak , @tbursztyka

What is wrong about having such file by each vendor individually?
In my opinion one file with peripheral list containing all vendors and all possible peripheral variants will make developers work more difficult.
@carlescufi : generic list of peripherals like HAS_HW_ADC is not enough for us. Some SoCs have ADC other SAADC only, some have SPI other SPIM or both.  The best example is with nrf52840.
It has possibility to configure 2 serial ports at time: UART0 and UARTE1 or UARTE0 and UARTE1. It does not have UART1.

I'm not sure if this all should be in common kconfig with the list of all possible options when in many cases names may be unique to vendor. It will generate long list and when vendor will be adding new peripheral it will require change to common file. Let say that there is LPUART which is low power UART, but other vendor has LEUART which is low energy uart. Should it use LPUART or create LEUART in common list. If LPUART is used it may confuse somebody because there will be kconfigs like:
config LEUART0_ENABLE
depends on LPUART0
Additionally, we might want to not only put peripherals there but also specific features in those peripherals. Example: SPI in device X has max speed 4M and in device Y has 8M. I want to limit menu for those instances of SPI which do not support 8M. That i can to by creating flag like HAS_HW_SPI3_8M. Such flag to do belong to generic, common file for sure.
Summarizing, I think that each vendor should has own list of peripherals/features since those be used only in Kconfig specific to that vendor and nowhere else.

Merging but keeping track of this in #8481


Register package

Could you please register the package? It takes several days from submission to registration. Moving to an org after a registration is not a problem.

Done.
(See https://github.com/JuliaRegistries/General/pull/52762 )

Awesome! Sorry if I stressed you, I really didn't mean to.


How can i ref to value when calling function from DllImport

Hi devs!
I have problem with using pointers from library...
Anyway... I did bindings for library but pointers doesn't work well i think so...
// Define the camera to look into our 3d world
camera as Camera3D
camera.position = Vector3(10.0f, 10.0f, 10.0f)  // Camera position
camera.target = Vector3(0.0f, 0.0f, 0.0f)       // Camera looking at point
camera.up = Vector3(0.0f, 0.0f, 0.0f)           // Camera up vector (rotation towards target)
camera.fovy = 45.0f                             // Camera field-of-view Y
camera.type = CameraType.CAMERA_PERSPECTIVE     // Camera mode type
cubePosition = Vector3(0.0f, 2.0f, 0.0f)
cubeSize = Vector3(2.0f, 2.0f, 2.0f)

c = do(ref c as Camera3D):
	UpdateCamera(c)
c(camera)

So i think it doesn't work properly, Here is definition from DllImport:
// Update camera position for selected mode
[DllImport("raylib", CallingConvention: CallingConvention.Cdecl)]
def UpdateCamera(ref camera as Camera3D):
	pass

idk why it doesn't work as intended, Sorry but i did not found right docs about pointers...
How can i use ref with DllImport for function uses pointers? Sorry i know this shouldn't be here...
Any code example would be welcome...

You say "it doesn't work."  How so?  What do you expect it to do and what it it doing instead?
Can you share the definitions of Camera3D, from both the Boo side and the C side, to verify that they match?  Do you have access to the code?  It looks like you're using raylib, so can you build it from source and trace into it with mixed-mode debugging to see how the camera's properties match up with what you're seeing on the managed side?

@masonwheeler Sure!
Here is bindings repo, It's working but problem persists in using pointers somehow...
Here is the Camera3D from Boo side...
https://github.com/Rabios/raylib-boo/blob/master/raylib_boo.boo#L367
Here is the Camera3D from C side...
https://github.com/raysan5/raylib/blob/master/src/raylib.h#L288
But see this example, I'm porting it to Boo...
https://github.com/raysan5/raylib/blob/master/examples/core/core_3d_picking.c
Problem in line 48 of the example, idk how to call it correctly...

About what it does, It should show a cube in 3D world space can be pickable with mouse...
See here: https://www.raylib.com/examples/web/core/loader.html?name=core_3d_picking
But instead, It does not shows 3D world, Only text and when i click everywhere it sets collision to true, idk why this happen...

One thing I notice right away is that fovy is defined as float in the original code but int in your translation.  Floats can represent int values, but their binary representation is very different, so it's likely that you're ending up with a camera with a bizarrely wrong field of view that's not focusing properly on your 3D scene.
Again, have you tried using the debugger in mixed mode to trace into the call and verify that the camera's data values come out the same on the native side as what you set on the managed side?  If not, this would indicate a problem with either the data typing or the alignment.

Thanks!
And no...I'm using shared lib of raylib, So you need me to debug via print values right?

I tried to edit and change fovy to single as you said, But it does not...

Yeah, use print values to debug if you can't get in with the debugger.  Try and verify if what's coming through on the native side is the same as what you're feeding into it on the managed side.

Info:
Camera position: 10, 10, 10 (x, y, z)
Camera target: 0, 0, 0 (x, y, z)
Camera up: 0, 0, 0 (x, y, z)
Camera fovy: 45
idk if this can help

Look, I changed camera position to 0, 1, 0
But i get weird behaviour...
Camera position: 0, 1, 4.371139E-08
Camera target: 0, 0, 0
Camera up: 0, 0, 0
Camera fovy: 45

That's very odd.  The data looks right, at first, but it's definitely getting corrupted in some way when you change it.
How are you changing it?  Just changing the code that sets the initial values and re-running it, or adding code to modify it later?

Changing from code, After loop it should be as i set it...

Is it possible that something on the native side is adjusting the camera's position slightly in response to something?  (8.742278E-08, 0.9999999, 0.9999999) is extremely close to (0, 1, 1).

Maybe?
But what's with the x?

OK, really i don't know what should i do with this, And dll working right with my Lua bindings...
https://github.com/Rabios/raylua
Do you think it's bug by Boo compiler?

It's hard to tell from this, whether it's a bug in the compiler, or if it's something weird going on on the native side.

OK, Is using pointer when i made issue correct?
How to call with ref?

I think what you're doing should work.  There are a handful of common mistakes when doing stuff like this, but aside from the fovy error, I'm not seeing any of them in your camera code.  The "wrong" numbers look like they're only off by a tiny amount, so it could easily be a precision/rounding issue, the type of thing that pops up in floating point code all the time.
In the spirit of always asking the obvious questions and verifying assumptions, have you made sure that the C equivalent of this same code works as expected?

Yes, It works!
But Boo one doesn't, I think it's problem with Boo...

Well, you're setting up the camera correctly, and it's displaying text correctly, but the 3D part isn't rendering.  Seems to me the most likely culprit is a bad call in the 3D code somewhere.

OH MY GOD!

Then it's problem in dimensions...

Solved, Thanks and will close issue!

Glad to see you got it working.  What was the problem?

Problem was that i used values from Raylib-cs port, Which is not compatible with C one (And i don't know how this can happen...)
Sorry, I was dump...
Here is screenshot of victory!

Source Code

Congratulations on getting it working!


bump go version, use Clarilab eventhorizon fork

Will be taken care of in tracygo. https://github.com/Clarilab/tracygo/pull/52


Fix for dragging an item over a nested ListView ref. #266

Bug reported in #244 seems not to be completely fixed in #266
@punker76 You can reproduce it as described in #266. First take your current gong dll (I tried with 1.2.0 alpha0057 and current dev) and replace it in the showcase project of @Mertsch, downloadable at https://github.com/Mertsch/gong-wpf-dragdrop
Now run the showcase:

Select "Fixed #Issues"

Select #146

Drag and Drop more than one item from left to right so that you get multiple rows

Select #244

Try to drop an item from the left side onto the right side:
a) when dropping onto the inner item box of the first row everything works as expected
b) but when dropping onto the inner item boxes of following rows it only works when dropping onto the "Root" item - as before the fix

Do you have an idea how to fix this completely?

@punker76 just debugged code and found the problem:
DropPosition is relative to parent items control, therefore it needs to be recalculated to be relative to the child item, i.e. the ItemContainer. E.g.:
Outer ListView with 5 rows, each row has a Height of 100, inner ListViews respectively. Lets say DropPosition is on third row of outer ListView, and has coordinates X=50, Y=250. The inner ListView ist under the DropPosition. Now the ItemContainer's Y-Bound is [0, 100], DropPosition's Y-coordinate is 250 => DropPosition is not in the descendant bounds.
Fix: get the ItemContainer's position relative to the Parent's (0,0)-Coordinate an subtract the ItemContainer's relative coordinates from the DropPosition coordinates:
var relativeItemPosition = item.TranslatePoint(new Point(0, 0), itemsControl);  var relativeDropPosition = new Point(DropPosition.X - relativeItemPosition.X, DropPosition.Y - relativeItemPosition.Y); directlyOverItem = VisualTreeHelper.GetDescendantBounds(item).Contains(relativeDropPosition);

@punker76 just opened pull request #284 with a possible solution


Removing the full log href message

It's already available in GitHub checks directly just underneath it
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>

Codecov Report

Merging #128 (71fca5b) into main (a76a370) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #128      +/-   ##
==========================================
- Coverage   73.34%   73.30%   -0.04%
==========================================
  Files          13       13
  Lines         799      798       -1
==========================================
- Hits          586      585       -1
  Misses        149      149
  Partials       64       64

Impacted Files
Coverage Δ

pkg/pipelineascode/status.go
74.24% <ø> (-0.39%)
:arrow_down:

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 a76a370...71fca5b. Read the comment docs.


Fix Spelling mistake in APIM application registration task related files

Purpose
This PR contains fix for https://github.com/wso2/product-apim/issues/2724
Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

Hi,
This package is also failing for me, because the zip file is packaged with the folder name in it.
This is the error in the wso2carbon.log
TID: [-1234] [] [2018-06-19 07:38:43,855] ERROR {org.wso2.carbon.humantask.deployer.HumanTaskDeployer} -  Error deploying HumanTask package : ApplicationRegistrationTask-1.0.0.zip {org.wso2.carbon.humantask.deployer.HumanTaskDeployer}
org.wso2.carbon.humantask.core.deployment.HumanTaskDeploymentException: htconfig.xml file not found for the ApplicationRegistrationTask-1.0.0
at org.wso2.carbon.humantask.core.deployment.ArchiveBasedHumanTaskDeploymentUnitBuilder.buildDeploymentConfiguration(ArchiveBasedHumanTaskDeploymentUnitBuilder.java:159)
at org.wso2.carbon.humantask.core.deployment.ArchiveBasedHumanTaskDeploymentUnitBuilder.(ArchiveBasedHumanTaskDeploymentUnitBuilder.java:104)
at org.wso2.carbon.humantask.core.store.HumanTaskStore.createNewDeploymentUnit(HumanTaskS


Background job is not working while Application is in kill mode in IOS.

"react": "16.13.1",
"react-native": "0.63.3",
"react-native-background-actions": "^3.0.0",

@VaishaliG2020, when the app is killed by the user, iOS will close all running background tasks.

@VaishaliG2020, when the app is killed by the user, iOS will close all running background tasks.

Hey @Rapsssito! Sorry to bother, but do you know a place in the official documentation where it's clearly stated that killing the app by the user (swiping it up from task manager) closes all running background tasks? I am aware that's the case, although I cannot find it anywhere in the docs 😅


Improve efficiency of occupied

Previously occupied checke if version was odd by self.version % 2 > 0 However this requires a costly modulos when we instead can use a bitwise and and check whether the LSB is 0 or 1.

The compiler does this automatically.


answers to assignment 1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Include the Codes for assignment 1
What did you learn from the changes you have made?
training models
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
No
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Some of the codes are difficult to write and figure out were errors are coming from
How were these changes tested?
It was tested on VSCode
A reference to a related issue in your repository (if applicable)
Chatgpt
Checklist

[- ] I can confirm that my changes are working as intended

I have submitted assignment 1

You have unfortunately made a pull request to the wrong repo. You shouldn't be making a pull request to UofT-DSI/applying_statistical_concepts, but rather your own forked applying_statistical_concepts repo. Please read the instructions on your onboarding Assignment Submission Guide more carefully.


Find player aliases

Is there a means to search for player aliases as in the old python client? Would be a 'nice to have'. Oh and thanks for all the hard work on the new client - its looking really good now

You can right click a player and click player information which will show u his name history. Is that what u want or is that that you want to know how was named "cristal13" before?

Player information is currently broken due to API issues. The loading of achievements takes so long, that you run into a Gateway Timeout.

Yeah but that should not effect other stuff loading or does it?

Yeah that's what the Python client allows. You can search for a specific name and get all their aliases.
Get Outlook for Android
On Sun, Nov 25, 2018 at 11:59 AM +0000, "Alexander"<EMAIL_ADDRESS>wrote:
You can right click a player and click player information which will show u his name history. Is that what u want or is that that you want to know how was named "cristal13" before?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yeah same here just search for a player in chat and see this name history. But does not work if player is offline etc.

It's working again. As such I'm closing the ticket:


Typo "@mention"→"@mention"

Describe the bug
A clear and concise description of what the bug is.:
Typo "**@mention**"→"**\@mention**"(Bold with escape characters)
en-es:"https://docs.microsoft.com/en-us/business-applications-release-notes/october18/microsoft-flow/post-richer-messages-in-microsoft-teams"

#PingMSFTDocs
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@hyoshioka0128 This issue is about a release note, which is blogged at https://flow.microsoft.com/en-us/blog/adaptive-cards-for-microsoft-teams-microsoft-flow-us-government-now-available/

@rootsmusic - Thanks!
Closed #3623 .


Cannot destructure property 'testCaseStartedId' of 'properties' as it is undefined.

Current behavior

An error is thrown when @env(something=1) is used in a feature file.

Desired behavior

It should work. It actually does work, but only on a subsequent run. Aka I can run the test, then update feature file by adding @env decorator. When change is detected and test runs again, error is not thrown.
Test code to reproduce

@env(foo=2000)
Feature: duckduckgo.com
  Scenario: visting the frontpage
    When I visit duckduckgo.com
    Then I should see a search bar

Versions

Cypress version: 8.5.0
Preprocessor version: 12.0.0
Node version: 16.2.0

Checklist

[x] I've read the FAQ.
[x] I've read Instructions for logging issues.
[x] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

Hey, @FDIM. Thanks for reporting the issue. This has been fixed in v12.0.1.

I'm getting this error on the following versions
Cypress version: 10.7.0
Preprocessor version: 14.0.0
Node version: 14.18

If you're asking for help, then you need to provide a full example that reproduces the issue.

I'll try to create a reproduction for this.


support for CloseOnAuthenticationExpiration

Is your feature request related to a problem? Please describe.
It seems that feature added in SignalR for closing expired connections is not implemented in Azure SignalR.
https://learn.microsoft.com/en-us/aspnet/core/signalr/configuration?view=aspnetcore-6.0&tabs=dotnet#advanced-http-configuration-options
Option: CloseOnAuthenticationExpiration
Implemented in: https://github.com/dotnet/aspnetcore/pull/32431
There is related change for Azure https://github.com/dotnet/aspnetcore/issues/35544
but seems its about sending config options, but not actually supporting them in Azure.
Describe the solution you'd like
Ideally support for CloseOnAuthenticationExpiration in Azure SignalR.
Additional context
I would appreciate any suggestions for workaround to support authentication expiration from server side, i.e., I would like to avoid sending message to clients to initiate disconnect.
I found https://github.com/dotnet/aspnetcore/issues/5283#issuecomment-797446519
but context.GetHttpContext().Abort(); does not work when using Azure SignalR
context.Abort(); closes the connection, but it disallows for client to reconnect.

The support of the feature is not released in the runtime side, I'll notify you once it's ready to use it.

The runtime is ready for the feature.


Slack form does not work at all in iPhone 7

He is not able to see any of the slack forms since the redesign was launched.
iOS 10.2, in Chrome and Safary.
Reported by @rporres irl

I can not reproduce this bug

I've asked a friend and it looks fine in a regular iPhone 6

@rporres is this still the case on your device?

Yes it is. I've only found people around me with iPhone 6 or lower, where it is not reproduced.


Testing: Import and reimport Veracode test

Related to https://github.com/DefectDojo/django-DefectDojo/pull/6452
This test is to show that current reimporter fails to import the same veracode report twice without changes. It should fail for now, and work correctly once #6452 is merged

Weirdly, the new test test_import_veracode_reimport_veracode_active_verified passed without any issues, where it should have failed...

Found out why: the many_findings in veracode tests doesn't have a mitigated static flaw, the only one there is "proposed" 🤦

Test failing as expected


Refactor Cookie, add CookieCollection

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔

Tests pass?
✔

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

Some preliminary refactoring and additional class (CookieCoolection) needed for #86 implementation.
Also I think changes are useful even without #86.
The Cookie class property defaults were changed to standard cookie defaults so that minimal valid cookie can be created (containing only name and empty value) and parsed by this class and recommended/useful defaults are now set in constructor with flag switch (enabled by default).

Also what about naming of methods in Cookie class?
Do we keep them as they are expires(), domain(), path()... or we rename it to withExpires(), withDomain(), withPath... to clearly indicate that they are copying the original object?

Previously cookie was used as write-only so there was no confusion about what these methods do. Now, as you're adding collections, it's a bit different and with may come in handy.

Pushed all the changes with tests. Added all getters to Cookie for consistency. Removed expire() method from Cookie as it was used only in one place in CookieCollection which is just replaced with setting date in the past. Also I'm in doubt whether expireWhenBrowserClosed method needs to remain (though if we remove it, the only way to convert already existing cookie to session cookie would be to create a new one with all the same parameters except expires).

I've re-added expire() to Cookie and removed it from collection.

Would be great to have a self-sufficient Collection, not CookieCollection only.
We have yiisoft/yii2-collection. Would you like to try to port it?

I think @romkatsu said he'll do it.

@samdark @xepozz The general idea with CookieCollection is to implement some Collection interface. which I was planning to extract later on to yiisoft/arrays. I didn't even know about existence of yiisoft/yii2-collection all these years working with yii2. Is there any reason to have a separate yiisoft/collection package for just a couple of classes and interface (it can easily fit to yiisoft/arrays). The Collection class from yiisoft/yii2-collection is just my planned ArrayCollection that implements Collection interface. CookieCollection is specialized and type-hinted case of that collection just for working with cookies. My collection interface was inspired from CookieCollection from yii2 and Collection from Doctrine collections. I will be glad to extract Collection interface after this PR and ArrayCollection implementation.

I've re-added expire() to Cookie and removed it from collection. See #235

Do you mean to re-add Cookie::expire method for the mentioned PR. What about keeping also CookieCollection::expire in that case as it will save a few lines of code. Without it to expire just one cookie from the collection you would do:
$cookie = $collection->get('someName');
$cookie->expire();

with it you would just do:
$collection->expire('someName`);

As it is just a helper also in Cookie class, it can stay also CookieCollection, I just think it can be renamed expireCookie for clarity.

What about keeping also CookieCollection::expire in that case as it will save a few lines of code.

It doesn't save much:
// collection
$collection->expire('someName');

// cookie
$collection->get('someName')->expire();

What about keeping also CookieCollection::expire in that case as it will save a few lines of code.

It doesn't save much:
// collection
$collection->expireCookie('someName');

// cookie
$collection->get('someName')->expire();

This will not expire the cookie in the collection as Cookie is immutable, it will create a new cookie. My example is also incorrect. So to expire cookie in the collection we would need:
$collection->set('name', $collection->get('name')->expire());

Or we will need to re-add expire().

I have pushed some more changes:

Fixed StyleCI issue.
Fixed Scrunitizer issues.
Added more tests to verify the correct behavior of CookieCollection::fromArray() method.
Renamed some methods and added CookieCollection::walk() method which helps to apply user defined function (closure) to each cookie. The usage is for batch editing cookies (later will be used when signing and/or encrypting cookies). The name of the method was chosen to be consistent with native array_walk and is planned in Collection interface.

It fails tests now: https://travis-ci.com/github/yiisoft/yii-web/builds/161267353#L340

About the expire... let's re-add the method to collection but keep the method in the Cookie as well.

Renamed some methods and added CookieCollection::walk() method which helps to apply user defined function (closure) to each cookie.

I'd keep collection interface to a minimum for now. @romkatsu is on general collection implementation that is reusable checking Yii 1 collections, yii2-collection and yiisoft/data to get the max out of it. Then it could be used for RBAC, this collection etc. But before it's done we're fine with current implementation.

Renamed some methods and added CookieCollection::walk() method which helps to apply user defined function (closure) to each cookie.

I'd keep collection interface to a minimum for now. @romkatsu is on general collection implementation that is reusable checking Yii 1 collections, yii2-collection and yiisoft/data to get the max out of it. Then it could be used for RBAC, this collection etc. But before it's done we're fine with current implementation.

Ok, I will keep an eye on that and will propose any ideas/changes later, when @romkatsu will publish the work.
I will add an expire() method back to CookieCollection to and fix the tests today.

Good job on this one 👍


Towards 0.1

In fact it has been stable for a while, and nothing is preventing me from releasing 0.1 or even 1.0. but before that, I need to add some documentation.

0.1 has been released.


fetch cloudflare ipv4/ipv6 fail #1405

Only append valid IPs to ip_ranges

This is an automated message from CI:
Docker Image for build 1 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-1703
Note: ensure you backup your NPM instance before testing this PR image! Especially if this PR contains database changes.

jc21/nginx-proxy-manager:latest
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 01_perms.sh: executing...
Changing ownership of /data/logs to 0:0
[cont-init.d] 01_perms.sh: exited 0.
[cont-init.d] 01_s6-secret-init.sh: executing...
[cont-init.d] 01_s6-secret-init.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
❯ Enabling IPV6 in hosts: /etc/nginx/conf.d
  ❯ /etc/nginx/conf.d/production.conf
  ❯ /etc/nginx/conf.d/default.conf
  ❯ /etc/nginx/conf.d/include/ip_ranges.conf
  ❯ /etc/nginx/conf.d/include/assets.conf
  ❯ /etc/nginx/conf.d/include/ssl-ciphers.conf
  ❯ /etc/nginx/conf.d/include/letsencrypt-acme-challenge.conf
  ❯ /etc/nginx/conf.d/include/force-ssl.conf
  ❯ /etc/nginx/conf.d/include/block-exploits.conf
  ❯ /etc/nginx/conf.d/include/proxy.conf
  ❯ /etc/nginx/conf.d/include/resolvers.conf
❯ Enabling IPV6 in hosts: /data/nginx
[12/30/2021] [4:47:09 AM] [Global   ] › ℹ  info      No valid environment variables for database provided, using default SQLite file '/data/database.sqlite'
[12/30/2021] [4:47:09 AM] [Global   ] › ℹ  info      Generating SQLite knex configuration
[12/30/2021] [4:47:09 AM] [Global   ] › ⬤  debug     Wrote db configuration to config file: ./config/production.json
[12/30/2021] [4:47:10 AM] [Migrate  ] › ℹ  info      Current database version: none
[12/30/2021] [4:47:10 AM] [Setup    ] › ℹ  info      Creating a new JWT key pair...
[12/30/2021] [4:47:12 AM] [Setup    ] › ℹ  info      Wrote JWT key pair to config file: /app/config/production.json
[12/30/2021] [4:47:12 AM] [Setup    ] › ℹ  info      Logrotate Timer initialized
[12/30/2021] [4:47:12 AM] [Setup    ] › ℹ  info      Logrotate completed.
[12/30/2021] [4:47:12 AM] [IP Ranges] › ℹ  info      Fetching IP Ranges from online services...
[12/30/2021] [4:47:12 AM] [IP Ranges] › ℹ  info      Fetching https://ip-ranges.amazonaws.com/ip-ranges.json
[12/30/2021] [4:47:14 AM] [IP Ranges] › ℹ  info      Fetching https://www.cloudflare.com/ips-v4
[12/30/2021] [4:47:14 AM] [IP Ranges] › ℹ  info      Fetching https://www.cloudflare.com/ips-v6
[12/30/2021] [4:47:14 AM] [SSL      ] › ℹ  info      Let's Encrypt Renewal Timer initialized
[12/30/2021] [4:47:14 AM] [SSL      ] › ℹ  info      Renewing SSL certs close to expiry...
[12/30/2021] [4:47:14 AM] [IP Ranges] › ℹ  info      IP Ranges Renewal Timer initialized
[12/30/2021] [4:47:14 AM] [Global   ] › ℹ  info      Backend PID 242 listening on port 3000 ...
[12/30/2021] [4:47:15 AM] [SSL      ] › ✖  error     Error: Command failed: /usr/sbin/nginx -t -g "error_log off;"
nginx: [emerg] invalid number of arguments in "set_real_ip_from" directive in /etc/nginx/conf.d/include/ip_ranges.conf:300
nginx: configuration file /etc/nginx/nginx.conf test failed

    at ChildProcess.exithandler (node:child_process:397:12)
    at ChildProcess.emit (node:events:390:28)
    at maybeClose (node:internal/child_process:1064:16)
    at Socket.<anonymous> (node:internal/child_process:450:11)
    at Socket.emit (node:events:390:28)
    at Pipe.<anonymous> (node:net:687:12)

jc21/nginx-proxy-manager:github-pr-1703
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 01_perms.sh: executing...
Changing ownership of /data/logs to 0:0
[cont-init.d] 01_perms.sh: exited 0.
[cont-init.d] 01_s6-secret-init.sh: executing...
[cont-init.d] 01_s6-secret-init.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.


nixos/udev: silence harmless warnings

Description of changes
This is something I had on my list for a while.
udevRulesFor generates a lot of warnings like:
substituteStream(): WARNING: pattern '"/sbin/modprobe' doesn't match anything
in file '/nix/store/.../95-dm-notify.rules'

due to the (preemptive) substitution of common paths in the default udev rules.
In this case a file having no matches is not unepected and poses no issue at
all.
Things done

[x] Tested with nixosTests.simple, not sure what else to do
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Thanks for cleaning up this mild annoyance.


TOCPanel: scroll into view of active item

I'm fixing the auto-highlighting of the active tocitem right now.. what i have no solution for yet is when the active item is out of the view... maybe we want the toc to automatically follow the active item.
I would use the same scrollToNode API as we have in other panels, however i'm afraid this will get a bit jumpy.. especially when you scroll back up after the view has switched.
 Or have better idea...?

I share the concerns.
Don't know.. maybe an icon that show up when active item out-of-view and click scrolls to?

If it requires extra user interaction I would rather do nothing and let people scroll themselves. But i will play a bit.. i mean.. also maybe now the wrong time to make it perfect. :)
There will be another TOC iteration for Lens because i want to bring thumbs of figures.


Fix Waterbodies dataset notebook to correctly identify bit flags

https://github.com/GeoscienceAustralia/dea-notebooks/blob/develop/DEA_products/DEA_Water_Observations.ipynb
Update table to correctly represent how bitflags are calculated, and:
Check that the bit flags in the table are correct for collection 3

the DEA Waterbodies dataset notebook explanation of bitflags is inverted in the way the numbers are explained, and we need to check the values for the flags are correctly identified for collection 3.

Done!


Private configuration

Might be needed for both JunOS, EOS etc.
The controller should use configure private (or configure exclusive?) to make sure that no one else commits the candidate configuration.

For junos:
https://www.juniper.net/documentation/us/en/software/junos/junos-xml-protocol/topics/ref/tag/junos-xml-protocol-open-configuration.html

Actually, this changed to "exclusive" access.
The reason is:
(1) private candidate is not a standardized concept
(2) Even in the juniper proprietary solution, private cannot be combined with confirm-commit which is necessary for failures at commit
(3) exclusive is standard using lock/unlock and can be combined with confirm-commit

In branch https://github.com/clicon/clixon-controller/tree/exclusive-commit

Verified.


Adding Azure Functions project to do stream processing logic

Purpose

Adds the actual Azure Function that will do stream processing to the project
Does this introduce a breaking change?

[ ] Yes
[x] No

Author pre-publish checklist:

[x] Added test to prove my fix is effective or new feature works
[x] No PII in logs
[ ] Made corresponding changes to the documentation

Closes #102

Closes #102


fix: locally install rocks on windows and add packages

On unix, rocks are installed by default on --prefix, this is not the same on windows. This PR aims to fix this.
Should mostly solve #15

Fantastic, thank you very much!


Show Spaces at Start of Lines in Quiz Exercises

Describe the bug
When you create a Quiz Exercise of type Short Answer Question with spaces at the start of lines inside of the markdown editor, these spaces are removed in the exercise preview, see example. This is a problem for fill-in-the-blank coding exercises, as the indentation is removed.
To Reproduce

Log in as instructor.
Click 'Course Management' at the top.
Click 'Exercises' of the respective course.
Under 'Quiz Exercises' click 'Create Quiz'.
Click 'Add Short-Answer Question'.
Add a code to the editor, such as the following:

`public class Foo {`
`    public `[-spot 1]`() {}`
`}`
[-option 1] Foo

Click 'Save'.
Go back and click 'Preview Quiz'.

Expected behavior
The indentations and the respective spaces at the start of lines are shown in front of public Foo() {}.
Screenshots
Markdown:

Preview:

How is the behavior if you use three ticks to start and end a code segment?

Markdown:

Preview:

So it seems like the three back ticks don't seem to work here. In other exercise types, they can be used to fomat the section enclosed in them into a monospace font where indentation is retained.
The behavior that single back ticks do not retain all white spaces occurs in other exercises as well. I assume this is intentional?
So I'd say that the handling of segments within three back ticks in the "short answer quizzes" should be fixed.

A solution using code blocks with three ticks respectively would be much preferred of course, as it would be much easier to copy and paste your code then. I just thought that the proposed solution from the original post is easier to implement.


MuiSnackbar

Snackbar

@Antony-M1 or @manikandanmohan10  Please review the code and merge in the main branch


ODBC and JDBC Driver Version

It is very valuable for DBADASH to track ODBC and JDBC driver versions.

There isn't a lot of information in this request as to why this would be useful.  None of our applications use ODBC or JDBC so I'm unlikely to look at this.  I'd consider pull requests.

The jdbc driver is it is outdated causes lots of implicit transactions
which is horrible. Odbc and ole drivers if they are outdated have security
and connectivity implications. Most apps are using one or the other to talk
to SQL.
On Mon, Oct 14, 2024, 12:59 PM David Wiseman @.***>
wrote:

There isn't a lot of information in this request as to why this would be
useful. None of our applications use ODBC or JDBC so I'm unlikely to look
at this. I'd consider pull requests.
—
Reply to this email directly, view it on GitHub
https://github.com/trimble-oss/dba-dash/issues/1076#issuecomment-2412098671,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGN5RX2GNSOUQ6OIO4YB743Z3QPCBAVCNFSM6AAAAABP5LSWK2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJSGA4TQNRXGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm not sure if it's possible to capture what driver versions are being used by clients.  The client interface name is available in sys.dm_exec_sessions but that doesn't tell you much other than it's ODBC, etc.

We have to powershell through it
Get-OdbcDriver -CimSession MyServer -Name sql | Format-Table name,
platform -AutoSize
02.
name platform
On Mon, Oct 14, 2024, 1:38 PM David Wiseman @.***>
wrote:

I'm not sure if it's possible to capture what driver versions are being
used by clients. The client interface name is available in
sys.dm_exec_sessions but that doesn't tell you much other than it's ODBC,
etc.
—
Reply to this email directly, view it on GitHub
https://github.com/trimble-oss/dba-dash/issues/1076#issuecomment-2412247576,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGN5RX5SKGZY4H5IZOEJHLLZ3QTV7AVCNFSM6AAAAABP5LSWK2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJSGI2DONJXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


fix: Copy link should be on the item page

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #5551
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

pay 10 usd

😍 Perfect, I’ve sent the payout
💵 $10 @ 32.8 USD/KSM ~ 0.305 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0xb952414163902bb0c54e970b9000100c46dffd687a96f50796dc0e3bc029b9c7
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Fix NVorbis warning

Summary
Currently NVorbis is generating a warning that it cannot be references from netstandard because it build with netframework 4.5. Because this warning will end up in user projects we should strive to fix it.
Warning	NU1701	Package 'AdamsLair.NVorbis 0.7.6' was restored using '.NETFramework,Version=v4.6.1, .NETFramework,Version=v4.6.2, .NETFramework,Version=v4.7, .NETFramework,Version=v4.7.1, .NETFramework,Version=v4.7.2, .NETFramework,Version=v4.8' instead of the project target framework '.NETStandard,Version=v2.0'. This package may not be fully compatible with your project

We are using a fork of nvorbis currently but it seems there is also a nuget package nowadays from the makers of nvorbis themselves which does support netstandard:
https://www.nuget.org/packages/NVorbis
Alternatively we could try see if simply changing the target framework in https://github.com/AdamsLair/nvorbis and publishing a new package will work.

Made a draft PR in #798 to test this. So far the code compiles and unit tests are green but need to do more testing


Extend LeaseEndpoints to allow path-based lease revocation

Hey, there!
I'm sorry for bringing this up again, but I feel like #480 was never fully explored or explained, and it's still an annoyance due to false positives in error logs as well as a slight security risk.
I don't believe most admins will grant all their applications access to sys/leases/revoke. Doing so would give those applications the ability to revoke any lease as long as they know the lease ID.
With that assumption in mind, I can only see two possible solutions:

Disable revoking of leases when an application shuts down since it's likely to fail anyway. This only really removes the error messages from the log, but doesn't actually solve the main problem with dangling leases
Allow applications to revoke a specific lease by their ID via the sys/leases/revoke-prefix/ endpoint, which is documented here

I know the revoke-prefix endpoint also has its caveats, most notably that it'll revoke anything that's "after" whatever prefix you define. I do believe it should be up to each project/organization to decide if they want to leverage it, however. It is possible to give an application access to a specific lease ID while rejecting all other revocation attempts, for example.
One relatively simple (and possibly naïve, I haven't looked through the whole spring-vault code base!) solution to the problem could perhaps be to add another constant to the LeaseEndpoints enum. That makes it easy for people to opt in by setting the spring.cloud.vault.config.lifecycle.lease-endpoints to SysLeasesPrefix (or whatever the name ends up being).
// TODO: Come up with a better name, maybe
SysLeasesPrefix {

	@Override
	public void revoke(Lease lease, RestOperations operations) {

		// TODO: Consider if this is secure enough or if it needs to be sanitized
		String endpoint = "sys/leases/revoke-prefix/" + lease.getLeaseId();
		operations.put(endpoint, null);
	}

	@Override
	public Lease renew(Lease lease, RestOperations operations) {

		return SysLeases.renew(lease, operations);
	}
};

In any case, keep up the good work! 😃

I'm sorry for bringing this up again, but I feel like #480 was never fully explored or explained

No worries. The referenced ticket had a strong focus on request body parameters vs. request path parameters.

I don't believe most admins will grant all their applications access to sys/leases/revoke.

That's a fair assumption and probably a good measure to disallow /revoke/{leaseId} as a sufficient number of attempts may yield a hit to revoke a lease that wasn't controlled by the application.
It makes a lot of sense to introduce a new enum constant that allows revocation through the prefix. Since we use the full lease Id, we don't accidentally revoke other leases.
While being on it, it probably makes also sense to revisit enum constant naming. Introducing an alias (well, delegate) Leases als alternative to SysLeases makes it easier to come up with a name. Prefix or RevokeByPrefix are names that could help to find a good constant name.
We also should switch the default in SecretLeaseContainer from Legacy to SysLeases.
In any case, feel free to submit a pull request.

Thanks for your thoughts and ideas, I'll give it a go!
Another question that came up is: should this be implemented only for the /sys/leases version of the API, or for the legacy version as well? I'm struggling to find the documentation for that API so I'm not even sure if such an endpoint exists, but I need to know if I'm to add a 4th enum constant.

Introducing an alias (well, delegate) Leases als alternative to SysLeases makes it easier to come up with a name.

Do you mean I should add a new constant, Leases, which behaves exactly like SysLeases, or do you just mean to use the word Leases as a tool for coming up with a descriptive name for the new constant?
The most unambiguous constant name I can come up with is SysLeasesUsingRevokePrefix, which incorporates the /sys/leases version of the API and also which endpoint it uses to revoke leases. I honestly feel like these two operations should've been separated because that would've made it slightly easier to add permutations like this later on. But that's difficult to predict ahead of time.
Regarding changing the default implementation from Legacy to SysLeases, maybe that warrants its own PR? That way it won't interfere with this issue in case you want to add this to the 2.2.x train.

should this be implemented only for the /sys/leases

Yes, please. We don't want to evolve around the legacy endpoints as we're going to deprecate that constant anyways.

Do you mean I should add a new constant, Leases, which behaves exactly like SysLeases

Exactly.

Should I deprecate SysLeases then, in favour of Leases, or will that happen at some later time?
Right now the enum contains the following constants:

Legacy
SysLeases
Leases
LeasesRevokedByPrefix

Do you think that's okay, or should I brainstorm the name even more?

I think that's fine for now. Feel free to deprecate SysLeases in favour of Leases.

Alright, I created a PR and I believe it's ready for a review 😃


Create the Categories screen

Point to include:

there is only a list of links to a category

Notes:

we should try to use the list component also on the settings screen

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


rm console.logs

Fixes # .
Changes proposed in this PR:

just remove some console.log on provider (think it was from a previous merge)

let's see if this breaks again the system tests on main branch. hopefully not!


Very very high ram usage

This is a chat mod which should not be using close to as much ram as it just did, it used almost 2gb itself when running this and made my computer crash. I am not the only one who is having this issue. A streamer I watch is having this same issue. I only installed it to see if I had the same issue and it did. This is with the latest build of chattweaks for 1.12.2 I have no other chat mods installed either. So it shouldn't be clashing with anything. I cannot give the log because it crashed before it could be created. I will add it was with a very light mod pack also, (Just another skyblock JASB) If you want to check if anything in that modpack has another mod that clashes. Wish I could provide more details but at the moment I cannot.

This is happening too me as well. literally got the same numbers

As a quick first solution you can disable Twitch Subscriber Emotes in the config. If you're using Twitch Chat Integration, emotes from Twitch chat will still show up fine (without the memory issues).
This is happening because Twitch rolled out the affiliate program and there's a lot of new channel emotes now (totalling up to 1.1 million emotes). Chat Tweaks doesn't load the image until it's needed, but it does store the emote code and some metadata, which adds up.
To resolve this,

subscriber emotes will no longer be enabled by default and will have a warning in their config description
the amount of stored metadata (especially parts that were duplicates) will be reduced to a minimum and instead computed when needed
emotes of channels with less than five emotes will be ignored


Reloadsolution

Adding a "/reloadsolution" endpoint to solve this issue : https://github.com/OmniSharp/omnisharp-roslyn/issues/19

there should probably be a restart or reload method on the project system interface. I think it's not a safe assumption to think that Initialize will be OK to call over and over

I agree with @filipw is this open to memory leaks?

Doesn't seem to work.
I asked @cubitouch to submit it so I could test. It was an experiment :)

So, I will start over following that list :

Reload "Workspaces" objects
Reload "Context" objects
Reload "Projects" collection
Reload "ProjectMappings" collection

Please let me know if I'm forgetting something, that I can manage that before you review the PR.

@cubitouch sounds good. At the very minimum, your code should compile ;)
After that, I would expect adding new files, projects, deleting files, should just work after firing the reloadsolution command.

I've been testing this. It doesn't work reliably yet. Sometimes it works just like you'd expect, but other times it causes intellisense to stop working for the entire solution.

Is this totally random or some cases just do not work ?
Do you have an idea what can cause the intellisense stop working properly ?

I have no idea what is causing the issue because I didn't debug. I spent quite a while testing this though.
Please test this yourself.

@cubitouch this has been stalled for more than a year. Is this PR still valid?

@troydai no I don't think so, I close it.


Demo not working

Describe the bug
https://nft-app-demo.productsway.com/
Reproduction
https://nft-app-demo.productsway.com/
System Info
https://nft-app-demo.productsway.com/

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Thanks for your report. Fixed by https://github.com/jellydn/nft-app/commit/a0aad9e3937ec4482d7106e2ac037e293a40765b
✨ Demo


[1] Formally deprecate ck8s-cluster and promote kubespray

We have been leaning more and more toward kubespray for cluster life-cycle management. Despite this, we still mention ck8s-cluster in the readme and do not mention kubespray anywhere.
To make it easier to understand how to get started and to avoid guiding users to ck8s-cluster, we should make these things clear in the readme. Preferrably, this could be done after we have investigated a possible migration path from ck8s-cluster to kubespray and moved the pipeline to kubespray. Then we could also describe how to move from ck8s-cluster to kubespray.
Definition of done: There is no mention of ck8s-cluster in this repo and there is a short description (or link) to explain how to set up a cluster to run compliantkubernetes-apps in.

Deprecate ck8s-base-vm as well?

Sure, i created https://github.com/elastisys/ck8s-base-vm/issues/29


Feat added layout

@dleard
Created this branch on the top of sign-off page, which is not yet merged,
For test you can check layout.tsx , FormContext and OperationReview.tsx

@ayeshmcg can this be closed?

@dleard  I added this initially as exploration, to see if this might be helpful for task list and multistep form with task list, but I think @pbastia has already been working on a solution.


Call to Undefined Method

Hey Nuno
When working with menus in Zero I'm getting this error:

Call to undefined method NunoMaduro\LaravelConsoleMenu\Menu::setUnselectedMarker()

I don't need it, I was just testing out different settings via the Zero docs for menus. But I thought you should know.

Jumped the gun,

setSelectedMarker

is also missing.

@secondmanveran, these haven't been available for quite a while (Feb 2020) as they were removed in v3.x of this package (see https://github.com/nunomaduro/laravel-console-menu/pull/24#issuecomment-591117020) which upgraded to v4.x of php-school/cli-menu. 👍🏻
Looks like I removed them from the docs for the package, but they are still on the LZ docs, I'll update this. Thank you. 👍🏻


gobuster error

gobuster changed the usage syntax in a recent update, should add 'dir' before some flags

I've submitted a pull request with the appropriate syntax corrections.

The issue has now been resolved.


Collect and render track logs

Neither aircraft nor ships show track logs. This is a must have feature.

Basic functionality added in b959eb041efe864a48ac9f95cd70eac97456355e & 7f7870902bfe4cc90ff16fc4333b2bfface8290c

Basic functionality added in b959eb041efe864a48ac9f95cd70eac97456355e & 7f7870902bfe4cc90ff16fc4333b2bfface8290c


DNF sort

I didn't find the code of dnf sort here. So I would like to code it and add it here. I code it in cpp as a gssoc'21 participant.

I want to work on this issue. Please assign to me.

closing it

can u please update the gssoc and level so that I can get my score.

Please assign this to me  in C++ or python

Please assign this to me in Java

Please Assign this issue to me in C++


Interpreting string-based values?

I have a question about some concerns I have about pre-interpreting data of certain VR types.
For example, the VR::IS is being interpreted as an [i32] internally.
Because of this, if we were to load a dicom file, and save it without making any modifications, the saved file would be normalized, in that the VR::IS would be converted into an array of integers, and then back to a string.
This would also pose a problem with implementing validation checkers. I'd like to develop a command line tool that loads a DICOM file, and validates it's contents. I can't validate any field at all if any one field prevents me from loading the entire DICOM file.
IMO, it is important to keep values that are represented with the Default Character Repertoire as characters. We could provide utilities to re-interpret the values into their typed representation (date times, date ranges, integer strings, etc) after the DICOM file is loaded.

If I understand correctly, it is a planned feature to support loading a file without parsing textual values right away. There's already some code for this string-preserving parsing mechanism, although it might not be accessible at the object API, and this might also require a bit of extra code at the element struct to only parse these values on demand. At some point this should be possible indeed.


added MsgSeqNum fields to CME & ICE dialects

it's important to know MsgSeqNum. may worth adding it to all the dialects?

@divaykin Yes, definitely worth adding. Every dialect has it so perhaps we could always generate it in the code generator script.


DebugCallback::new - urctbase.dll!strlen STATUS_ACCESS_VIOLATION crash when injecting RenderDoc

The latest version of Vulkano crashes when I attempt to use RenderDoc by injecting into a running process.

Version of vulkano: 0.17
OS: Windows 10
GPU (the selected PhysicalDevice): Radeon RX 580
GPU Driver: Driver Version <IP_ADDRESS>-200117a1-351300C-RadeonSoftwareAdrenalin2020

This short program creates the following output:
    Finished dev [unoptimized + debuginfo] target(s) in 0.09s
     Running `target\debug\crashes_on_my_machine.exe`
Press enter...
Using physical device (DiscreteGpu): Radeon RX 580 Series
error: process didn't exit successfully: `target\debug\crashes_on_my_machine.exe` (exit code: 0xc0000005, STATUS_ACCESS_VIOLATION)

The example program was essentially copy-pasted from the examples in this repository, with which I've also been able to reproduce this behavior.
To reproduce:

cargo run the program
In RenderDoc, File -> Inject into Process
Select crashes_on_my_machine.exe and press inject
Press enter in the console window to begin Vulkan calls

Through some println debugging, I've narrowed it down to possibly this macro instantiation in vk-sys to generate bindings for vkCreateDevice, where anything after this line is unreachable.
I've spoken with Baldur Karlsson via email about a bug report with the same conditions as the example program above, wherein he stated it's possible this is a driver bug. The report indicates possible interference with RivaTuner but I've verified that the behavior occurs with or without RivaTuner. Another bug report I filed pointed to a crash due to strlen in ucrtbase.dll which I believe is related. I can't say for sure since all of this is way over my head anyway 😬
I also have the rdoc-generated .dmp file for this sample program zipped in the repo. I tried out windbg and I didn't really understand what I was doing, but the output from messing around with the minidump is there as well if it's useful.

It was pointed out to me that RTSS was still active when I ran into some of this behavior. I've updated the windbg output and it pointed me to one of CStr::from_ptr here in DebugCallback::new.
I found that by commenting out the sample program's DebugCallback::new call, the sample program would operate normally.


Position doesn't seem to work

Even with bottom right in the notification object it still shows up in the top right.
      this.$notify({
        group: 'messages',
        position: 'bottom right',
        title: 'Important message',
        type: 'error',
        text: 'Hello user! This is a notification!'
      });

Is there something that I have incorrect?
Using version 1.3.4

Hi. You should set position on Notifications component. Please look into readme.

Ah, Yes that worked, i took props as properties of the $notify object.
For others that search for this here is what fixed it
<notifications group="messages" position="top center"/>

it could be helpful if we could control from the $notify object.

it could be helpful if we could control from the $notify object.

For the suggestion above - one would need to mount containers for each position every time (ie system will not know upfront where you will want to show your notification - hence should be "ready")... which is 6 components.
This means that it will just eat memory and pollute the component tree for most of the users who will never use this feature.

For the suggestion above - one would need to mount containers for each position every time (ie system will not know upfront where you will want to show your notification - hence should be "ready")... which is 6 components.
This means that it will just eat memory and pollute the component tree for most of the users who will never use this feature.


The scroll bar locking behavior (What if it'd be customizable with this global CSS)

          added this

body { overflow: auto !important; }
To overcome any CSS or JavaScript preventing the body from scrolling or locking the scroll bar.
But I ended up with the MUI offset of the right margin of the screen, like it fluctuates to the opposite.
I made that to stop the fluctuation from the scrollbar hiding, but MUI still got another offset.
Originally posted by @samueltesfayegari in https://github.com/mui/material-ui/issues/17353#issuecomment-2305711595
Search keywords:

I guess you are suggesting to add it to the docs?

Yes I wish the MUI offset could be overridden by something else or this.


esm.sh init throws Cannot convert undefined or null to object

Hello all,
When I try to use the init task on a brand new project, it throws Cannot convert undefined or null to object, has this error been encountered before?
Here is the full output of the command when run. (Ubuntu 22.04 on WSL version <IP_ADDRESS>)
adam@computer:~$ mkcd esmtest
adam@computer:~/esmtest$ deno init
:white_check_mark: Project initialized

Run these commands to get started

  # Run the program
  deno run main.ts

  # Run the program and watch for file changes
  deno task dev

  # Run the tests
  deno test
adam@computer:~/esmtest$ deno run -A -r https://esm.sh init
error: Uncaught (in promise) TypeError: Cannot convert undefined or null to object
    Object.entries(imports).sort(sortByValue),
           ^
    at Function.entries (<anonymous>)
    at sortImports (https://esm.sh/:427:12)
    at saveImportMap (https://esm.sh/:335:25)
    at Object.init (https://esm.sh/:183:11)
    at eventLoopTick (ext:core/01_core.js:181:11)
    at async https://esm.sh/:514:5

Thank you for your time,
Adam

there is a bug of the cli script, i just added a patch for it. you can add an external importMap in your deno.json as workdaround, i will deploy the patch soon. thanks!


Pub can't use transformers from dev dependencies.

According to FAQ:
$ uname -a
Linux glinyanov-PC 3.14.14-031414-generic #201407281153 SMP Mon Jul 28 15:54:19 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
$ pub version
Pub 1.9.3
$ pub --verbose update > pub.log 2>&1
pub.log is here: https://goo.gl/PL1cUU
My team was working on refactoring and extracting libraries to separate repositories in particular, and we found a bug, that we could reproduce: if in package:A we use package:test as dev_dependency and use test/pub_serve transfomer, then if in package:B we use package:A as 'git' dependency (and NOT 'path' dependency), then pub get/update for package:B fails with exception:
Loading source assets...
The null object does not have a getter 'pubspec'.

NoSuchMethodError: method not found: 'pubspec'
Receiver: null
Arguments: []
dart:core
... # more in pub.log

I created the example packages to show it, it's here: https://goo.gl/2k8Kea.
Nevertheless, my real project builds successfully even after bad pub update, meaning pub build does not fail, and package:A itself can successfully use that transformer.
For those, who experience same issue: move your transformer's dependency from dev_dependencies to dependencies (e.g. test for test/pub_serve), it solves the problem.

This issue was moved to dart-lang/pub#1324


Explain where to get the magic values

Sticks docs in the file that people will be staring at when confused about where the encrypted values came from.

 Warning

Pull requests are usually filed against the gh-pages branch for this repo, but this one is against master. Please double check that you specified the right target!

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.


Terraform fails to create/update team on Buildkite Pro plan

Describe the bug
We just switched from the Legacy to Pro plan, and our buildkite_team resources started to fail.
To Reproduce
Create a team resource:
resource "buildkite_team" "viewers" {
  name                = "Viewers"
  privacy             = "VISIBLE"
  default_team        = false
  default_member_role = "MEMBER"
}

Error:
│ Error: Unable to update Team
│
│   with buildkite_team.viewers,
│   on buildkite.tf line XX, in resource "buildkite_team" "viewers":
│   XX: resource "buildkite_team" "viewers" {
│
│ Unable to update Team: input: You cannot modify
│ members_can_create_pipelines on your current plan
│
╵
Operation failed: failed running terraform apply (exit 1)

Expected behavior
The team should be updated correctly.
Additional context
Buildkite Terraform module version 1.5.2
Terraform version 1.8.5

@tovbinm thanks for the report!
This looks to have been caused by a backend change that we're looking at reverting, behaviour should revert once that's done.

Hey @tovbinm we have reverted this behaviour so you should no longer be seeing this error. Please reopen if you are


Med-RT validation support

Steps to test:

Unit tests pass
Run previous dictionary test rules with the --med-rt argument pointing to a directory containing a valid medrt DTS xml file. (see attached) and verify the rule runs correctly.

Core_MEDRT_XML.zip
Core_MEDRT_DTS.zip


Support editor.tabType

This does the simplest thing possible. When editor.tabType is set to auto, it does what it did before. When it is set to hard or soft, it uses those values all the time unless setSoftTabs() is explicitly called. The API is unchanged. Specs have been consolidated a bit as there were a couple places where the tab stuff was being tested.
Eventually (another PR), I'd like to change the name of editor.softTabs setting to editor.defaultTabType or something more clear.
Closes https://github.com/atom/atom/pull/8186 @Abdillah, I've used a couple of your commits.
Closes https://github.com/atom/atom/issues/3719

Do you think we should deprecate editor.softTabs?

One caveat with this is: if the user has explicitly set the soft tabs value to something and an event happens (config or grammar change), the user's setting is ignored and computed based on the rules of the setting. I thought it added extra complexity to support tracking the explicitly set choice, and I wasnt sure if it would be the correct behavior anyway.

Do you think we should deprecate editor.softTabs?

We could change the name to editor.defaultTabType or something? Or yeah, we could deprecate it.

Anyone opposed to merging this?

:ship:


[EVENT] February 13th - February Member's Talk Evening @ Spiced Academy

Head organizer: @tiaversa
Support organizer: Anastasia Mikheeva
Slides/Moderation: Anastasia Mikheeva
Speaker support:
Location: Spiced Academy - Ritterstraße 12-14, 10969 Berlin
Speaker: Potentially (Nura Kawa - waiting for response)
Talk:
MONTH BEFORE

[ ] Define Speakers
[ ] Send Speakers guidelines email
[x] Announce event with speakers, location, gender policy, photo policy on meetup page
[x] Define hosts & collect host information
[ ] Send Host guidelines email
[x] Organise Food/ drinks
[ ] Tweet event when announced
[ ] Check with speaker if coaches are needed and search for coaches

TWO WEEKS BEFORE

[ ] Retweet event
[ ] Check with speaker about installation instructions/reading material and update announcement
[ ] Update meetup announcement with agenda, networking breaks, if snacks or drinks will be available
[ ] Check-in with hosts and ask if support is needed for reception

WEEK BEFORE

[ ] Remind attendees to install requirements with comment and/or message on meetup
[ ] Send coaches the workshop/relevant material
[ ] Dry run with speaker

WEEK OF THE EVENT

[ ] Tweet after event
[ ] Ask host if attendees list is needed and/or special location directions
[ ] Double check with host: microphone (if needed), soft drinks, power plugs, internet
[ ] Send to host to print PyLadies arrow signs
[ ] Print [feedback forms](feedback forms)
[ ] Prepare introduction slides past example

DAY OF THE EVENT
(arrive 30-60 minute before event starts)

[ ] Check projector, microphone, table setup, power extensions, recording if available
[ ] Support speaker(s) to test equipment
[ ] Stick PyLadies signs to guide attendees the location
[ ] Hang up or put out print outs with the WIFI (or find a similar solution)
[ ] Manage reception, give name tags if available
[ ] During the COVID 19 pandemic (and perhaps later) there might be additional health checks such as vaccination status or proof of a recent test to allow entry

DURING THE EVENT

[ ] Do introduction and Community announcements
[ ] Take photos of all the speakers and audience
[ ] Tweet about it (or after the event with photos)

AFTER THE EVENT

[ ] Tweet after event (if not done during the event)
[ ] Upload photos to google drive (you need to be part of the google group to access this)
[ ] Open PR with resources to this repository
[ ] Email sponsors & host to thank them
[ ] Copy feedback forms to feedback spreadsheet
[ ] Adjust guides if valuable feedback came up

Event done!


config: ini No such file or directory

When trying to create the docker image, i get no such file or directory for config.ini, what am i doing wrong?

So the latest code commit is actually broken. I would suggest pulling the latest image from dockerhub
(docker pull bgulla/boozer) and using that rather than building the codebase. I accidentally pushed to the wrong branch last week and havent had a chance to fix it. If you are still seeing the error, reply with the docker run command you are using.

i did that, how would i go about finishing the docker command?

docker run --rm  -d --name="boozer" \
    --privileged \
    -v ./config.ini:/boozer/config.ini \
    -v ./db.sqlite:/boozer/db.sqlite \
    -t bgulla/boozer

Note: This assumes that you copied your config.ini and your db.sqlite to your current working directory.

Thank you, please add the note to your readme file
On Tue, Apr 2, 2019 at 12:49 AM Brandon<EMAIL_ADDRESS>wrote:

docker run --rm  -d --name="boozer"
--privileged
-v ./config.ini:/boozer/config.ini
-v ./db.sqlite:/boozer/db.sqlite
-t bgulla/boozer
Note: This assumes that you copied your config.ini and your db.sqlite to
your current working directory.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-478884417, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZJctByu77-Zni85yJkEgnbenYPnN6nLks5vcwtzgaJpZM4cWs1r
.

just got a chance to try that, now i am getting includes invalid characters
for a local volume name, only "a-zA-Z0-9][a-ZA-Z0-9]" are allowed.  if you
intended to pass a host directory, use absolute path.  See docker run --help
On Tue, Apr 2, 2019 at 10:38 AM Brandon<EMAIL_ADDRESS>wrote:

Closed #7 https://github.com/bgulla/boozer/issues/7.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#event-2247347360, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AZJctMcQ-We5c3ZK4G41H2_KldyEBSg7ks5vc5WTgaJpZM4cWs1r
.

i was really hoping to get this to work this week, love this project...
docker run --rm  -d --name="boozer"
--privileged
-v ./config.ini:/boozer/config.ini
-v ./db.sqlite:/boozer/db.sqlite
-t bgulla/boozer
changed it to
docker run --rm  -d --name="boozer"
--privileged
-v ./config.ini:/opt/boozer/config.ini
-v ./db.sqlite:/opt/boozer/db.sqlite
-t bgulla/boozer
and it ran however, when i do sudo docker ps, there's no docker image
"boozer" :( :( :(
On Tue, Apr 2, 2019 at 7:23 PM David Beiler<EMAIL_ADDRESS>wrote:

just got a chance to try that, now i am getting includes invalid
characters for a local volume name, only "a-zA-Z0-9][a-ZA-Z0-9]" are
allowed.  if you intended to pass a host directory, use absolute path.  See
docker run --help
On Tue, Apr 2, 2019 at 10:38 AM Brandon<EMAIL_ADDRESS>wrote:

Closed #7 https://github.com/bgulla/boozer/issues/7.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#event-2247347360, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZJctMcQ-We5c3ZK4G41H2_KldyEBSg7ks5vc5WTgaJpZM4cWs1r
.

You need to add ${PWD} to your volumes as indicated from above, this is an environment variable that references the current-working-directory that you current are residing in your bash prompt. The way that docker works is that it has to use the absolute-path to mount the files into a container, ./db.sqlite won't work because docker is unaware of what directory you are currently sitting in.
Assuming that you placed your files in /opt/boozer like you mentioned, your run command would be the following.
docker run --rm  -d --name="boozer" \     --privileged \     -v /opt/boozer/config.ini:/boozer/config.ini \     -v /opt/boozer/db.sqlite:/boozer/db.sqlite \     -t bgulla/boozer
I know that it's never fun to RTFM, but I'd recommend going through the following walkthrough to explain some of the Docker basics.
https://blog.codeship.com/the-basics-of-the-docker-run-command/

I have four other services running in docker..., i will give this a try
tonight  appreciate your help
On Wed, Apr 3, 2019 at 1:41 AM Brandon<EMAIL_ADDRESS>wrote:

You need to add ${PWD} to your volumes as indicated from above, this is an
environment variable that references the current-working-directory that you
current are residing in your bash prompt. The way that docker works is that
it has to use the absolute-path to mount the files into a container,
./db.sqlite won't work because docker is unaware of what directory you are
currently sitting in.
Assuming that you placed your files in /opt/boozer like you mentioned,
your run command would be the following.
docker run --rm -d --name="boozer" \ --privileged \ -v
/opt/boozer/config.ini:/boozer/config.ini \ -v
/opt/boozer/db.sqlite:/boozer/db.sqlite \ -t bgulla/boozer
I know that it's never fun to RTFM, but I'd recommend going through the
following walkthrough to explain some of the Docker basics.
https://blog.codeship.com/the-basics-of-the-docker-run-command/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479396304, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZJctEK8mLP_pmOzRMQXhz3WglBKLWs9ks5vdGk4gaJpZM4cWs1r
.

that doesn't work, why isn't anything being populated into the docker image?
On Wed, Apr 3, 2019 at 10:40 AM David Beiler<EMAIL_ADDRESS>wrote:

I have four other services running in docker..., i will give this a try
tonight  appreciate your help
On Wed, Apr 3, 2019 at 1:41 AM Brandon<EMAIL_ADDRESS>wrote:

You need to add ${PWD} to your volumes as indicated from above, this is
an environment variable that references the current-working-directory that
you current are residing in your bash prompt. The way that docker works is
that it has to use the absolute-path to mount the files into a container,
./db.sqlite won't work because docker is unaware of what directory you are
currently sitting in.
Assuming that you placed your files in /opt/boozer like you mentioned,
your run command would be the following.
docker run --rm -d --name="boozer" \ --privileged \ -v
/opt/boozer/config.ini:/boozer/config.ini \ -v
/opt/boozer/db.sqlite:/boozer/db.sqlite \ -t bgulla/boozer
I know that it's never fun to RTFM, but I'd recommend going through the
following walkthrough to explain some of the Docker basics.
https://blog.codeship.com/the-basics-of-the-docker-run-command/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479396304, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZJctEK8mLP_pmOzRMQXhz3WglBKLWs9ks5vdGk4gaJpZM4cWs1r
.

docker run --rm -d --name="boozer" \ --privileged \ -v
/opt/boozer/config.ini:/boozer/config.ini \ -v
/opt/boozer/db.sqlite:/boozer/db.sqlite \ -t bgulla/boozer
what is boozer/config.ini, what exactly is "boozer" in this case? a folder
to a username or a folder after a username?
On Wed, Apr 3, 2019 at 6:50 PM David Beiler<EMAIL_ADDRESS>wrote:

that doesn't work, why isn't anything being populated into the docker
image?
On Wed, Apr 3, 2019 at 10:40 AM David Beiler<EMAIL_ADDRESS>wrote:

I have four other services running in docker..., i will give this a try
tonight  appreciate your help
On Wed, Apr 3, 2019 at 1:41 AM Brandon<EMAIL_ADDRESS>wrote:

You need to add ${PWD} to your volumes as indicated from above, this is
an environment variable that references the current-working-directory that
you current are residing in your bash prompt. The way that docker works is
that it has to use the absolute-path to mount the files into a container,
./db.sqlite won't work because docker is unaware of what directory you are
currently sitting in.
Assuming that you placed your files in /opt/boozer like you mentioned,
your run command would be the following.
docker run --rm -d --name="boozer" \ --privileged \ -v
/opt/boozer/config.ini:/boozer/config.ini \ -v
/opt/boozer/db.sqlite:/boozer/db.sqlite \ -t bgulla/boozer
I know that it's never fun to RTFM, but I'd recommend going through the
following walkthrough to explain some of the Docker basics.
https://blog.codeship.com/the-basics-of-the-docker-run-command/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479396304, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZJctEK8mLP_pmOzRMQXhz3WglBKLWs9ks5vdGk4gaJpZM4cWs1r
.

so i dont think its an issue with docker, it pulls information and the
image, but does not start
On Wed, Apr 3, 2019 at 6:52 PM David Beiler<EMAIL_ADDRESS>wrote:

docker run --rm -d --name="boozer" \ --privileged \ -v
/opt/boozer/config.ini:/boozer/config.ini \ -v
/opt/boozer/db.sqlite:/boozer/db.sqlite \ -t bgulla/boozer
what is boozer/config.ini, what exactly is "boozer" in this case? a folder
to a username or a folder after a username?
On Wed, Apr 3, 2019 at 6:50 PM David Beiler<EMAIL_ADDRESS>wrote:

that doesn't work, why isn't anything being populated into the docker
image?
On Wed, Apr 3, 2019 at 10:40 AM David Beiler<EMAIL_ADDRESS>wrote:

I have four other services running in docker..., i will give this a try
tonight  appreciate your help
On Wed, Apr 3, 2019 at 1:41 AM Brandon<EMAIL_ADDRESS>wrote:

You need to add ${PWD} to your volumes as indicated from above, this is
an environment variable that references the current-working-directory that
you current are residing in your bash prompt. The way that docker works is
that it has to use the absolute-path to mount the files into a container,
./db.sqlite won't work because docker is unaware of what directory you are
currently sitting in.
Assuming that you placed your files in /opt/boozer like you mentioned,
your run command would be the following.
docker run --rm -d --name="boozer" \ --privileged \ -v
/opt/boozer/config.ini:/boozer/config.ini \ -v
/opt/boozer/db.sqlite:/boozer/db.sqlite \ -t bgulla/boozer
I know that it's never fun to RTFM, but I'd recommend going through the
following walkthrough to explain some of the Docker basics.
https://blog.codeship.com/the-basics-of-the-docker-run-command/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479396304, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZJctEK8mLP_pmOzRMQXhz3WglBKLWs9ks5vdGk4gaJpZM4cWs1r
.

ok... so after some more troubleshooting, it's as if the app starts and then stops, i downloaded a python app called runline which extracted the docker run command, does this look right?
root@raspberrypi:/opt/boozer# runlike 45366bd4574e
docker run --name=boozer --hostname=45366bd4574e --env=PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin --env=TINI_VERSION=0.14.0 --env=LC_ALL=C.UTF-8 --env=DEBIAN_FRONTEND=noninteractive --env=UDEV=on --env=QEMU_CPU=arm1176 --env=container=docker --env=HOME=/boozer --env=APP_HOME=/boozer --volume=/opt/boozer/config.ini:/boozer/config.ini --volume=/opt/boozer/db.sqlite:/boozer/db.sqlite --volume=/sys/fs/cgroup --privileged --restart=no --label io.resin.device-type="raspberrypi" --label io.resin.architecture="rpi" --label io.resin.qemu.version="" -t bgulla/boozer python boozer.py

What you mean populated?
On Wed, Apr 3, 2019 at 9:50 PM gbdavidx<EMAIL_ADDRESS>wrote:

that doesn't work, why isn't anything being populated into the docker
image?
On Wed, Apr 3, 2019 at 10:40 AM David Beiler<EMAIL_ADDRESS>wrote:

I have four other services running in docker..., i will give this a try
tonight appreciate your help
On Wed, Apr 3, 2019 at 1:41 AM Brandon<EMAIL_ADDRESS>wrote:

You need to add ${PWD} to your volumes as indicated from above, this is
an environment variable that references the current-working-directory
that
you current are residing in your bash prompt. The way that docker works
is
that it has to use the absolute-path to mount the files into a
container,
./db.sqlite won't work because docker is unaware of what directory you
are
currently sitting in.
Assuming that you placed your files in /opt/boozer like you mentioned,
your run command would be the following.
docker run --rm -d --name="boozer" \ --privileged \ -v
/opt/boozer/config.ini:/boozer/config.ini \ -v
/opt/boozer/db.sqlite:/boozer/db.sqlite \ -t bgulla/boozer
I know that it's never fun to RTFM, but I'd recommend going through the
following walkthrough to explain some of the Docker basics.
https://blog.codeship.com/the-basics-of-the-docker-run-command/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479396304, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AZJctEK8mLP_pmOzRMQXhz3WglBKLWs9ks5vdGk4gaJpZM4cWs1r

.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479719202, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACkoMzi_SToWoHXTLJnYp_MrGOfh4KMuks5vdVpjgaJpZM4cWs1r
.

--
Brandon

I'm won't on a new build with updated debugging. You may want to wait until
this weekend.
On Thu, Apr 4, 2019 at 6:33 AM Brandon Gulla<EMAIL_ADDRESS>wrote:

What you mean populated?
On Wed, Apr 3, 2019 at 9:50 PM gbdavidx<EMAIL_ADDRESS>wrote:

that doesn't work, why isn't anything being populated into the docker
image?
On Wed, Apr 3, 2019 at 10:40 AM David Beiler<EMAIL_ADDRESS>wrote:

I have four other services running in docker..., i will give this a try
tonight appreciate your help
On Wed, Apr 3, 2019 at 1:41 AM Brandon<EMAIL_ADDRESS>wrote:

You need to add ${PWD} to your volumes as indicated from above, this is
an environment variable that references the current-working-directory
that
you current are residing in your bash prompt. The way that docker
works is
that it has to use the absolute-path to mount the files into a
container,
./db.sqlite won't work because docker is unaware of what directory you
are
currently sitting in.
Assuming that you placed your files in /opt/boozer like you mentioned,
your run command would be the following.
docker run --rm -d --name="boozer" \ --privileged \ -v
/opt/boozer/config.ini:/boozer/config.ini \ -v
/opt/boozer/db.sqlite:/boozer/db.sqlite \ -t bgulla/boozer
I know that it's never fun to RTFM, but I'd recommend going through the
following walkthrough to explain some of the Docker basics.
https://blog.codeship.com/the-basics-of-the-docker-run-command/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479396304,
or mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AZJctEK8mLP_pmOzRMQXhz3WglBKLWs9ks5vdGk4gaJpZM4cWs1r

.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479719202, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACkoMzi_SToWoHXTLJnYp_MrGOfh4KMuks5vdVpjgaJpZM4cWs1r
.

--
Brandon

--
Brandon

The container is there but it’s not running when I do sudo docker ps
On Thu, Apr 4, 2019 at 3:36 AM Brandon<EMAIL_ADDRESS>wrote:

I'm won't on a new build with updated debugging. You may want to wait until
this weekend.
On Thu, Apr 4, 2019 at 6:33 AM Brandon Gulla<EMAIL_ADDRESS>wrote:

What you mean populated?
On Wed, Apr 3, 2019 at 9:50 PM gbdavidx<EMAIL_ADDRESS>wrote:

that doesn't work, why isn't anything being populated into the docker
image?
On Wed, Apr 3, 2019 at 10:40 AM David Beiler<EMAIL_ADDRESS>wrote:

I have four other services running in docker..., i will give this a
try
tonight appreciate your help
On Wed, Apr 3, 2019 at 1:41 AM Brandon<EMAIL_ADDRESS>wrote:

You need to add ${PWD} to your volumes as indicated from above, this
is
an environment variable that references the current-working-directory
that
you current are residing in your bash prompt. The way that docker
works is
that it has to use the absolute-path to mount the files into a
container,
./db.sqlite won't work because docker is unaware of what directory
you
are
currently sitting in.
Assuming that you placed your files in /opt/boozer like you
mentioned,
your run command would be the following.
docker run --rm -d --name="boozer" \ --privileged \ -v
/opt/boozer/config.ini:/boozer/config.ini \ -v
/opt/boozer/db.sqlite:/boozer/db.sqlite \ -t bgulla/boozer
I know that it's never fun to RTFM, but I'd recommend going through
the
following walkthrough to explain some of the Docker basics.
https://blog.codeship.com/the-basics-of-the-docker-run-command/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479396304,
or mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AZJctEK8mLP_pmOzRMQXhz3WglBKLWs9ks5vdGk4gaJpZM4cWs1r

.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479719202, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/ACkoMzi_SToWoHXTLJnYp_MrGOfh4KMuks5vdVpjgaJpZM4cWs1r

.

--
Brandon

--
Brandon
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgulla/boozer/issues/7#issuecomment-479844285, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZJctJJ2jyMAmyvr1z_mWUWf64Gmlcslks5vddWggaJpZM4cWs1r
.


There is a typo in the folder name 'script' in README.md

Hello, Kwonyul.
I appreciate your effort in providing this nice CDK asset.
README.md has a little typo.
The folder's name is "script," without the final s.
The following section of instruction needs be modified:
For more details, open script/setup_initial.sh file.
sh script/setup_initial.sh config/app-config-demo.json
sh script/deploy_stacks.sh config/app-config-demo.json
I hope it would be helpful for your nice asset.
Thank you
Wookjae

sociengineer,
Thanks for your comments. I fixed that.
Thanks again.


Implement document.registerElement

Closes #429 - though without IE8 support (yet).
While researching this, I found that actually document.registerElement() was removed from the spec in favour of window.customElements.define, which could have bigger implications elsewhere (ref. https://github.com/w3c/webcomponents/commit/d95392f734895b2c72e1822f464dd70a22b322a4 and later https://github.com/w3c/webcomponents/commit/815910d197077523fb449dfb5a1e6fceab728019).
Does implementing this still make sense, since it is already in-use in some Origami components, or should we skip and go straight for window.customElements.define?  (As far as I can tell nothing has implemented the changes yet).
Lemme know if this is a question to raise elsewhere.

Wow, we're using registerElement in origami?  Where?!
If this is no longer on standards track I would be against putting it in the polyfill service unless to not do so would cause issues for FT

@antoligy did you get anywhere looking into this?  I'm tempted to close if registerElement is dead.  From the diffs it looks like defineElement still lives, but I haven't read all the changes in detail so I don't know where customElements.define sits in relation to defineElement.  Maybe this is something to defer for now - I've no objection to hosting polyfills for features that are not yet stable, but it's a lot of work to keep up.

Yep, there's an o-share web-component which prompted me to start looking for this polyfill!
For now it seems as though the usage of registerElement in Origami comes with feature detection, and I doubt that even with that it's seeing widespread use (though this is a massive assumption, to confirm I'll ask around on Slack tomorrow).
As discussed in https://github.com/WebReflection/document-register-element/issues/58, customElements.define is the final (stable?) version of this API and this has already been upstreamed into HTML, so we should be safe to use that going forward.
I'm happy to close this PR for now, and will raise another at a later date with something more concrete.

Ah yes, o-share is a testing ground for web components, but I don't think we intended the WC version to actually be used in production.  I think we're a fair way off that.  If you want to contribute a customElements.define method it would be gratefully received!


Do not look for self-contained JAR files when dependencies are specified separately

This PR is not to look for self-contained JAR files (labelled $embulk-deps$) when dependencies are specified separately from StaticInitializer.addDependency or StaticInitializer.addDependencies.

Thanks!