vagrant up fails with bosh-lite VagrantFile

I am trying to setup a local cloud foundry instance using bosh-lite on my mac laptop.
I unzipped the contents of bosh-lite-master ran the following command
see below
sh-3.2#vagrant up --provider=virtualbox
Error Message I got
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Box 'cloudfoundry/bosh-lite' could not be found. Attempting to find and install...
default: Box Provider: virtualbox
default: Box Version: 2776
The box 'cloudfoundry/bosh-lite' could not be found or
could not be accessed in the remote catalog. If this is a private
box on HashiCorp's Atlas, please verify you're logged in via
vagrant login. Also, please double-check the name. The expanded
URL and error message are shown below:
URL: ["https://atlas.hashicorp.com/cloudfoundry/bosh-lite"]
Error:
sh-3.2#

Hi,
I just tried and could not reproduce the error.
DX-191:tmp dx191$ cd bosh-lite/
DX-191:bosh-lite dx191$ vagrant up --provider=virtualbox
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Box 'cloudfoundry/bosh-lite' could not be found. Attempting to find and install...
    default: Box Provider: virtualbox
    default: Box Version: 2776
==> default: Loading metadata for box 'cloudfoundry/bosh-lite'
    default: URL: https://vagrantcloud.com/cloudfoundry/bosh-lite
==> default: Adding box 'cloudfoundry/bosh-lite' (v2776) for provider: virtualbox
    default: Downloading: https://vagrantcloud.com/cloudfoundry/boxes/bosh-lite/versions/2776/providers/virtualbox.box

Did you want to try again and let us know if you are able to reproduce?
Thanks!

This sounds similar to issue in this thread  https://groups.google.com/a/cloudfoundry.org/forum/m/#!topic/bosh-users/n2qYrpPUJaE

We've pushed fix to master (c82c939833fd05ad8e7ad93595ece6d2a054f806). Please try again.

Added a point in the troubleshooting section for this.

Yes this is the same issue discussed in above link


Move Instruction, T&C Modal to components folder

This issue is already solved. No need tester check @nguyenduythuc


Config support

Checklist for v0 (hopefully all this will make the 11 release cut)

[x] design: tinyurl.com/clangd-config
plumbing

[x] provider/compilation flow - llvm/llvm-project@f12cd99c440a83d53a8717a9c8cdc4df41f39f3d
[x] propagate through TUScheduler, Background index - llvm/llvm-project@ 9963d93b0731e21dd1c9c1cebf8baaecf2010330 llvm/llvm-project@15a60fe09f4618a7fb451f37aebfd1a671f83713

config sources

[x] parsing YAML - llvm/llvm-project@e9fb1506b83d001082dc535af12acf45050a527c
[x] (re)loading from concrete user + project configs - llvm/llvm-project@8bd000a65fe4452c09855115d5204a2a46838004 llvm/llvm-project@69c22edb7d30983f1bb9d21154e427ebcc5f699c (still needs default flagflip)
[x] move index files out of the way - llvm/llvm-project@9b55bc4d1197aedd76b0f4e04270265cb74cd0b8
[ ] parsing JSON
[ ] setting via LSP

Configurable features

[x] filtering by path
[x] adding compile commands
[x] removing compile commands
[x] enable/disable background indexing

[x] Documentation

JSON and LSP didn't make the initial cut. See discussion in https://reviews.llvm.org/D83822. Following up in #525
Closing this as initial support for this has landed and window for clangd 11 is closed.


Refactor and add reloadData function

Should close #3

Sorry for late review.
Thank you for your contribution!


Mlflow Platform Example

Platform Example using Mlflow
This PR has parity with all of the existing examples in the repo, and also introduces a more complex application which uses an operator for both Postgres and Minio

The chart is well laid out. Its a good example
Would it make sense to add add a secret/configmap template in the chart to install a support bundle spec? I'm thinking helm CLI installs. Perhaps the KOTS support bundle can be moved in the chart instead, and enable/disable via config

yup for sure. going to create a follow up story for that


(1)fixed test (points changed in osm (2) updated osmnx package

(1)fixed test (points changed in osm (2) updated osmnx package.
update the package to the latest version and update the environment yml file.

Minor conda-environment change


read_next_message string index out of range

I've seen this error happen from time to time, I cannot reproduce it:
Exception in server.py-read_next_message.
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/remi/server.py", line 177, in read_next_message
    length = self.bytetonum(length[1]) & 127
IndexError: string index out of range

Hi awesomebytes, this exception happens when the browser disconnects or is closed. This is managed. ;-)

Ok, great to know, next one who googles this will be fine haha
Thank you :)

Hi
How do you mean managed? I have this issue on version 1.0 installed from pip.
After disconnecting the browser and receiving the message I am then unable to launch again as the socket is already bound.
Excellent project by the way, looking forward to slapping it onto some raspberry pi's!

Hello @buxtonpaul  .
This exception is not an issue. Is a known 'normal' behaviour caused by browser (websocket) disconnection. This isn't the cause of your problem. Can you provide more info about the problem? Such as host system, web browser and so on...

Actually I feel a little silly.  This issue made it look like the server had shutdown, however it is still running, which is why when I attempt to run it again it fails. Doh!

@buxtonpaul no problem, you are welcome ;-)


EntityFramework Core sometimes rounds average to 2 decimal places

I have a table with a column of type DECIMAL(18, 8) among others. Corresponding model property Price has type decimal`
When I call _db.PriceTickers.Average(p => p.LastPrice) and the result is less that 0.01 it would return 0.01 or even 0.00 sometimes. If however average is greater than 0.01 it shows actual value.
SQL profiler revealed that when EF sends a query it casts the column to DECIMAL(18,2)
Here is my code EntityFrameworkSqlServerAverageBug.zip
Steps to reproduce
Compile and run the code
Data:
1	0.00112000
2	0.00232111
3	0.00345223
Tried explicitly setting column type
            modelBuilder.Entity<PriceTicker>()
                .Property(b => b.Price)
                .HasColumnType("DECIMAL(18,8)");

Further technical details
EF Core version: 2.0.2
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10
IDE: Visual Studio 2017 15.6
Database Server: SQL Server Express 2017
Workaround
Materialize the list of decimal values and then call Average method

Generated SQL
SELECT AVG(CAST([pt].[Price] AS decimal(18, 2)))
FROM [PriceTickers] AS [pt]

Since we are casting to decimal(18,2) here, we truncate useful data. It is one of the case of type inference. For explicit cast expression we are just looking at CLR Type ignoring type set on property altogether.

_db.PriceTickers.Average(p => p.LastPrice)
QM
      Optimized query model:
      '(from PriceTicker pt in DbSet<PriceTicker>
      select [pt].Price).Average()'


Setup tests

Set up tests (using either pytest or unittest)

This needs to happen, as I want to upgrade to MDAnalysis 1.0.0 and therefore some unit test and the odd regression test should be included... The current tests folder can essentially be deleted as it's a huge freakin mess.


Unable to set podCidr

Unable to set podCidr, when I set its value through parameters, I only get its default value of <IP_ADDRESS>/8

cilium-cli  install --set ipam.operator.clusterPoolIPv4PodCIDRList=<IP_ADDRESS>/16
cilium-cli  install --set  cluster-pool-ipv4-cidr=<IP_ADDRESS>/16
cilium-cli  install --set  cluster.pool.ipv4.cidr=<IP_ADDRESS>/16
The default values will not be changed through its parameters and other parameter settings, and changing the values in kubernetes configMap will not be successful either

@rookie-z
Struggled with that almost all day - there changed syntax
now it is
cilium install --set=ipam.operator.clusterPoolIPv4PodCIDRList=<IP_ADDRESS>/16,otherConfigs...

@stepanovdg
Thank you.

@rookie-z Struggled with that almost all day - there changed syntax now it is cilium install --set=ipam.operator.clusterPoolIPv4PodCIDRList=<IP_ADDRESS>/16,otherConfigs...

fyi: this helped me resolve my own issue with customizing the CIDR list, but it has changed since then
as of cilium-cli (v1.16.3), this is the formatting that is documented in cilium help install:
 cilium install --set ipam.operator.clusterPoolIPv4PodCIDRList=<IP_ADDRESS>/16 --set key=value

This particular configuration accepts a list of CIDR values, so it appears a comma separated list of values would be acceptable as such:
 cilium install --set ipam.operator.clusterPoolIPv4PodCIDRList=<IP_ADDRESS>/16,<IP_ADDRESS>/16 --set key=value

(i didn't test the second command or inspect code to verify)
list of values that can be set is provided here -- https://docs.cilium.io/en/stable/helm-reference/


Enhanced support for multi-service challenges

The current system for services in challenges severely limits the flexibility of services that can be deployed. This is mainly due to the inability to pass configuration settings for services when the docker compose is generated. CTF Architect should not impede the flexibility of services that can be made by challenge authors.
This will probably be done during the chall-architect rewrite and use the docker compose include element

Fixed in #20


Add a member function into_split for cedar_policy::Entity

Category
User level API features/changes
Describe the feature you'd like to request
Add a new member function, into_split, under Entity with the following function signature:
/// Disassemble an `Entity` into its unique ID, attributes and parents.
pub fn into_split(self) -> (EntityUid, HashMap<String, RestrictedExpression>, HashSet<EntityUid>);

This function will allow users to mutate an entity's attributes and parents after splitting and then re-assemble back into a cedar_policy::Entity.
Describe alternatives you've considered
None
Additional context
No response
Is this something that you'd be interested in working on?

[X] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

Seems useful, only note is that I think the more idiomatic name would be into_inner()

Closed by #685


Slower performance than the notebook?

Just wondering, do you have any idea why the runtimes for the detections in this node are about double the detection times in the jupyter notebook it's based off of? I get about 50ms in the notebook and 100ms in this repo. The code is copied almost verbatim, so I can't see why there is an issue. Can you reproduce this issue?

Hmm. nice point! I am also getting around 100ms. I did not test the notebook code's time, though.
Currently, I cannot reach my computer but I will try it and let you know. Btw, did you disable all other notes?
Could you try just launching this cob_people_object_detection_tensorflow.launch?
Otherwise, I will try it nearly in 1 week.

Yeah I’m only using the detection node. I’m using Jetson Xavier which might perform differently, I just though it was strange that this code took double the time when it’s basically the same as the notebook code. Wondering if it’s reproducible on your system.

Ok let's try for 10 imgs. I'll let you know.

Here is the result:

2.6096081733703613
2.943655252456665
2.531284809112549
2.513815402984619
2.484112024307251
2.609175205230713
2.50939679145813
2.451589584350586
2.4029273986816406
2.3995888233184814


Feature: Change Wallpaper

It would be nice, if the app could change the wallpaper to a preselected one or a specific color.

Check the new version https://github.com/zehfernandes/pliim/releases/tag/v1.3.0

Hi @zehfernandes, the latest version does change the wallpaper but it doesn't change to a specified one or anything. For example, I have a folder of car pictures that I cycle through automatically. What I would like is to change it to a default macOS wallpaper when I turn on Pliim.
What do you think?

Hi, @joshuapinter you are right. I will think in a way to do that :)

@zehfernandes Excellent! Thanks.  Now you can choose a specific wallpaper in Preferences right-click in the icon:
https://github.com/zehfernandes/pliim/releases/tag/v1.5.0

Boom!
Does Pliim have an auto-update?

unfortunately not. (it's not a signed app)
I recommend download and replace for the new version

Hmm, trying out 1.5.0 but it doesn't seem to change to the default black screen or the one I've selected.

@joshuapinter weird. Do you download the .zip? I test here, and everything seems fine. Do you close the app and replace? Can you check the version in the about?

Yes and it's 1.5.0.
I have my wallpaper to cycle through a folder and whenever I turn on Pliim (even in the latest version), it just cycles to a different wallpaper in that same directory. It doesn't set it to the black wallpaper or to the one that I specified. It's tough to show this happening in a screen cast but that's what's happening.

@joshuapinter I will try to simulate this scenario and come back with an answer.

My background stays Black color, and I can't change it even after disabling Pliim. Is there any way to solve the issue.


Make Slate Compatible with Ruby 3.0.0

I followed the Slate installation instructions to do a fresh install of Ruby, it installed Ruby 3.0.0.
When running bundle install, I naturally got
Your Ruby version is 3.0.0, but your Gemfile specified ~> 2.5

I then changed Gemfile to support Ruby 3.0.0 and got the following error when running bundle exec middleman server:
bundler: failed to load command: middleman (/usr/local/lib/ruby/gems/3.0.0/bin/middleman)
/usr/local/lib/ruby/gems/3.0.0/gems/middleman-core-4.3.11/lib/middleman-core/util/paths.rb:7:in `require': cannot load such file -- webrick (LoadError)

I then added gem 'webrick' to the Gemfile and ran bundle install one last time based on https://github.com/middleman/middleman/issues/2422
After that, I got bundle exec middleman server to work on Ruby 3.0.0 without any issue.
It seems good to support Ruby 3 for Slate in general, and I don't really see a lot of downsides here?

Could you add Ruby 3.0 to the build matrix in https://github.com/slatedocs/slate/blob/main/.github/workflows/build.yml so that it's minimally tested against?

@MasterOdin yep, done!

Собрать пожертвование
2000dollar

Visa


Create news post about Maplibre Navigation V2

This PR adds a news post about the latest release of Maplibre Navigation. Please let me know if you need any changes or if you are fine with this post.

Ah and run npm run format...

Thanks for the review, I updated the spelling and ran format.

https://www.linkedin.com/posts/maplibre_navigation-android-activity-7046153642944405505-7C2U?utm_source=share&utm_medium=member_android


Move iteration logic to tasks

Is your feature request related to a problem? Please describe the problem.
Today iter8 has built in logic to read and modify traffic weights. However, if another mechanism such as helm or gitops is used, the user must use loop tasks and modify the experiment to use loops.
Describe the feature/solution you'd like
Replace the iter8 logic with tasks that mirror those used for gitops and/or helm. The approach to read/set weights is then the same for all approaches.  This is related to elimination of Spec.VersionInfo (issue https://github.com/iter8-tools/iter8/issues/953).
Tasks to look something  like:
# this task will update the current weight distribution in experiment status
# by getting the actual weights from cluster
- task: clusterops/get-weights
  with:
    versionInfo: <list of weight Obj refs -- just like we would normally provide>

# this task requires clusterops/get-weighs to be specified earlier in the action pipeline,
# it will reuse versionInfo from there.
- task: clusterops/set-weights # re-uses params from clusterops/get-weights

Will this feature/solution bring new benefits for Iter8 users? What are they?
Common approach to writing experiment for different assumptions.
Does this issue require a design doc/discussion? If there is a link to the design document/discussions, please provide it below.
Use this issue.
How will this feature be tested?

[ ] Modify existing test cases; create new ones in handler

How will this feature be documented?

[ ] Reference documentation updated; handler documentation updated.

Perhaps it is better to have just one method? get-and-set-weights?


Add a property for tarLongFileMode

Add a property for tarLongFileMode so we can build cxf with posix tarLongFileMode

https://issues.apache.org/jira/browse/CXF-8152


Add operators

Pull Request Test Coverage Report for Build 795

0 of 58 (0.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-1.4%) to 81.406%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

inoutBound.go
0
58
0.0%

Files with Coverage Reduction
New Missed Lines
%

iterable_channel.go
2
96.0%

Totals

Change from base Build 792:
-1.4%

Covered Lines:
2767

Relevant Lines:
3399

💛  - Coveralls


Visual Studio 2017 Crash when executing code in FSI

This is hard to reproduce but from time to time I'm getting:

Exceptions displayed like this:

(the file is attached)

Visual Studio crashes, and then it restarts. This happens when sending code to the fsi window.

Repro steps
Unfortunately I can't find steps to reproduce it at the moment.
Expected behavior
No exceptions, no crashes.
Actual behavior
Exceptions, crashes.
Known workarounds
None
Related information

Operating system: Windows 10 Pro N
Branch: Visual Studio 2017 26228.9
.NET  Version: 4.6.01055
Editing Tools (e.g. Visual Studio Version): 2017
Links to F# RFCs or entries on http://fslang.uservoice.com: N/A
Links to performance testing scripts: ActivityLog.zip
Indications of severity: Medium to High

This is fixed in master.


Reviving Ratchet: Let's give Ratchet the love it deserves!

Ratchet is a great piece of software and it is successfully used in countless projects! But let's be realistic: It could use some love!
At the moment, we're making very good progress with ReactPHP v3 (see e.g. reactphp/event-loop#271) and Framework X. As a next step, we want to achieve the same level of velocity for Ratchet. Supporting the latest PHP and PHPUnit versions feels like a good starting point to bring Ratchet back to speed again, followed by tackling the huge pile of open support requests. So in a gist, we know what to do with Ratchet, the problem is that we have to find the time to actually do it. I've also reached out to the other maintainers, but before spending too much time on the details, I would like to see how much demand there is to work out a sustainable plan for the longer-term development.
As a first step, help us figure out demand by leaving a comment or an emoji reaction:

👍 Like the idea, generally interested
❤ Love the idea and would consider sponsoring further development of Ratchet
🚀 Ratchet is used in our company (let's have a chat!)
Anything else, please join the discussion!

So whats the plan because this package is now blocking to upgrade to symfony 7 ?
Can we upgrade to the latest version and then do a release and then work on other features ?

@dvdknaap That is ultimately the plan. Like @clue stated above, we're currently looking into project funding and once we know how to proceed with Ratchet, we'll probably start by adding support for the latest PHP and PHPUnit versions. We also have a separate discussion for Symfony 7 in #1045, so I think it makes sense to discuss further details in there.
@clue and the rest of our team are currently busy working on the release for a first version of Framework X, so we'll come back here once we freed some time :+1:

I have enough time to participate in the development of Ratchet, and we are currently using it. How should I participate in development?

@clue maybe add write access to people who want to help ? like @userofjack

I think that many people would support this post, but no one knows about it, because everyone is used to seeing the message “last modified 3 years ago” and simply do not know that you are going to continue supporting
overall this is great news and we are still using ratchet
just start making changes or apply pull requests :)

I think that a lot of people would support this post, but no one knows about it, because everyone is used to seeing the message “last modified 3 years ago” and simply do not know that you are going to continue supporting
overall this is great news and we are still using ratchet
just start making changes or apply pull requests :)

Since I saw the last commit, I thought it was abandoned.
I'm deciding to integrate it into my company. They did a very good job!

Since I saw the last commit, I thought it was abandoned.

Ditto. Perhaps update the README with a reference to this thread. That will then also reset the last commit time.

Thank you so much for all the amazing feedback we've received on Ratchet! We are truly grateful for the excitement and the offers to help with the development. Excited to use this momentum to move forward! 🚀

First we need to get the project up to date with the latest versions, so this is something I personally would like to work on first. Once this is completed, I would love to get more people involved, so we greatly appreciate any help and contributions.

Right now, we are finalizing the project handover with @cboden and hope to get the project up to speed again as soon as possible. In the meantime, we also work on a number of other bigger open-source projects like @ReactPHP (which Ratchet builds on top of) and need to balance our priorities and find adequate project funding to allow us to keep shipping.

You can help us! For now, I would like to set up a simple sponsorship goal of reaching 10 new sponsors (any amount) to see how much the community is willing to support this project. Once reached, we will start pushing the next PRs to get Ratchet updated to support the latest PHP and PHPUnit versions and use this as a starting point for bigger updates to come. Even just 10 x $1 would help achieve our initial goal, but the more sponsors the better obviously, so especially curious about any company sponsors reaching out!

Please help us getting Ratchet up to speed by becoming a sponsor through ReactPHP, we're thankful for every contribution! ❤

Also agree it's a good idea to link more prominently to this ticket, I've updated the readme for the next couple of weeks and will continue posting more updates as soon as possible!

I just cancelled an old GitHub pro account that was never used. Using that money to sponsor this one.

Sorry to cross over but https://github.com/ratchetphp/Pawl is in exactly the same boat. My company uses Pawl specifically and it's to the point where we're ready to maintain our own fork instead if that project is truly unmaintained. To that effect I can participate in development on either repo but would like to get an idea of what's going on with Pawl.

Sorry to cross over but https://github.com/ratchetphp/Pawl is in exactly the same boat.
May be it's a good idea to join source an make a common version that's client and server in the same project?

The existing repository structure makes sense; RFC6455 defines agnostic components that anyone could use in their Server/Client implementations. These components are separate from the implementations themselves. Pawl is purely an issue of not having any maintainers at the moment.

Please help us getting Ratchet up to speed by becoming a sponsor through ReactPHP, we're thankful for every contribution! ❤
@clue whats the progress on that :) ?

I look forward to it becoming part of the reactphp middleware, so that it can work together with reactphp/http or framework-x. It is very useful to provide both http service and websocket service on the same port, especially when http and websocket are related, as they are in the same process.

looks like this project is still dead. does anyone has a replacement ?

still no update after all those months. they are only hidding replies and collecting money.

Hello, is there any progress on the revivification of Ratchet? I would like to participate and contribute.

I'm very much appreciating the decission to bring ratchet up to new life, as i'm planning to use it in the future and was a bit worried about the staleness.
Would also like to support whereever possible. How can i?


fix(quickstart): remove pinned 0.33

Feature or Problem
As of the latest component-sdk-go, this isn't required anymore
Related Issues

Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

❤


Add Edit comment text to follower email

We should add the content of the version commit comment to the emails that get sent to followers when a case is updated.  That will hopefully help give them some guidance as to what has changed in the article.

on production


Make a list of future features

It would be great to make a list of upcoming features in Readme/different file so people can know what to expect in the future/work on them.

Thanks for the issue! I added a minimal list of features to the Readme. If you would like to see any other features in future updates feel free to add them to the list 😃


chore(deps): update module github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper to v1.0.16

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.0.15 -> v1.0.16

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper
v1.0.16
Compare Source
Bug Fixes

Make passing of resource group conditional (#91) (f80b205)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 2.2.1 :tada:
The release is available on:

GitHub release
v2.2.1

Your semantic-release bot :package::rocket:


Optimizer: Variables are not always initialized

Good morning
In the last two days, I suspected that my mind was slipping because I could no longer do the math. What a relief that the optimizer is probably a little too active in the variable initialization:
With this code:
x = 1
print("\n x: ", x)
print("\n x+x: ", x+x)

the compiler does not initialize the variable x, but it calculates afterwards with the uninitialized variables to get x+x:
set __tmp2 "\n x+x: "
print "\n x: "
print 1
op add __tmp3 x x
print __tmp2
print __tmp3
end

For the other readers: This small workaround (let's the compiler calculate the variable x by adding 0) solves the problem:
x = 1 + 0
print("\n x: ", x)
print("\n x+x: ", x+x)

Thanks a lot, kind regards,
Thomas

I have a 2nd example, it looks like the optimizer mixes up two things.
Interesting are the lines 4 and 5:
1 varX = 99
2 print("\nvarX: ", varX)	     // OK: The game displays 99
3
4 copyVarX = varX                    // This assignment is removed in the compiled code
5 print("\ncopyVarX: ", copyVarX)    // Error: The game displays null
6
7 printflush(message1)

This is the compiled code:
1 print "\nvarX: "
2 print 99
3 print "\ncopyVarX: "
4 print varX                         // Interesting: The Compiler uses varX (but it does not initialize this variable)
                                     // and although the source code wants to output the variable copyVarX
5 printflush message1
6 end

btw: in general the compiler and optimizer works great, I have scripts with about 900 compiled lines and I guess that the Script-Bugs are more related to my logic than to the compiler :-)

Hey! You’re very good at finding reproducible examples and that is a GREAT help!
The issue in this case is the DeadCodeEliminator. It isn’t intelligent enough to detect usages after the first use. I’ll take a look over the weekend.
Thanks again!


chore: custom breakpoints in tailwind config with other repo standards

tailwind.config.js

It is better if we add custom breakpoints.

      screens: {
        xs: '200px',
        sm: '640px',
        md: '768px',
        lg: '1024px',
        xl: '1280px',
        // 2xl: '1536px',
      },

Since this is going to be public at some point, so it will be better if we can add basic standards like

Security.md
CODE_OF_CONDUCT.md

I will make separate issues for completing different sections of the website as per figma design.

Looks good. Also, do you have access in our Projects Board? it is currently private.

Looks good. Also, do you have access in our Projects Board? it is currently private.

Yes, I have the access.


[IRGen] Assign layout strings for static layout generic types in meta…

…data initialization function
Explanation: The layout string needs to be assigned before completion, to make it available in recursive metadata initialization. Setting it in the completion function causes a race between other metadata initializers using it and the completion function running.
Scope: Compact Value Witnesses only
Issue: rdar://127535274
Original PR: https://github.com/apple/swift/pull/74058
Risk: Low. Only affects Compact Value Witnesses.
Testing: Added a validation test
Reviewer: @mikeash @aschwaighofer

@swift-ci test


Typo in doc

At doc/install.html#225, https://golang.org/wiki/SettingGOPATH should be https://golang.org/wiki/Setting-GOPATH (missing dash between Setting and GOPATH

The problem is that someone destroyed the SettingGOPATH wiki page, which was empty (it just said to go to Setting-GOPATH) which I believe was left there to avoid breaking links (general policy is that we don't delete wiki page and/or page subsections that can be linked to to avoid breaking links).

I have sent CL 77470 to fix the doc links to SettingGOPATH. Even if we decide to restore the wiki page, it's better to link directly to the real Setting-GOPATH in the docs, instead of linking to a page that will tell you to go to that other page.

The real page should never have been Setting-GOPATH.  I've renamed that page back to SettingGOPATH.
@inancgumus Why did you delete the SettingGOPATH page?
@headless32 Why did you replace the page with a meaningless one?
I'm starting to feel that we need code review on the wiki pages.
Anyhow, this issue is now fixed, so closing.

I think it was moved to "Setting-GOPATH" because it's rendered as "Setting GOPATH", which is the human-readable way to write it (space in the middle instead of using camelcase). See for example how this page's title is rendered: https://github.com/golang/go/wiki/Well-known-struct-tags

I'm starting to feel that we need code review on the wiki pages.

Yeah looking at the history it seems like that page was deleted and then restored multiple times, this is really suboptimal..


Carve out Core function to a core module

From #285 #270
We already have TextlintCore, but it includes fs module.
We want to work textlint on browser.
To need carve out core core function that is not depended on fs to a module.
import TextLintCoreCore from "@textlint/core"
const textlint = new TextLintCoreCore()
textlint.lint(text).then(() => {})

Tasks

[ ] Carve out Core into a module
[ ] Need good name!

@textlint/kernel ?

https://english.stackexchange.com/questions/32206/kernel-vs-core


External client electron build produces broken result

Describe the bug
In an external, client using the latest @dev releases for builder, core, plugin-core-support, and plugin-openwhisk, the electron artifact produced behaves as in the screenshot below.
Steps to reproduce the behavior

Unzip the supplied attachment (it is a minimal external client constructed as per kui instructions).  testClient.zip
In the testClient directory

npm install
NO_INSTALLER=true npx kui-build-electron /tmp mac
open dist/electron/Kui-darwin-x64/Kui.app

In the resulting open shell type wsk.

Expected behavior
The usage for wsk should display as in a monorepo build.
Screenshots

would you be able to try again? i think you caught a brief but bad window. i just tried this, with success:
npm init -y
npm install --save-dev @kui-shell/builder@dev
npm install --save @kui-shell/core@dev @kui-shell/plugin-k8s@dev @kui-shell/plugin-core-support@dev @kui-shell/plugin-editor@dev @kui-shell/plugin-bash-like@dev
npx kui-init
npx kui-build-electron
open dist/electron/Kui-darwin-x64/Kui.app/

I tried again and I agree that (1) your example works (2) the original example I submitted using the OW plugin also works, and (3) most importantly, the rebase is no longer failing with the problem either.   So, thanks for looking into it and I am back on the air.   Closing this now.

Unfortunately, in continuing to work on the rebase, I find this is only fixed for mac.

When I build for Linux on Linux, I continue to see the Cannot find module 'yargs-parser' error on many commands of the resulting executable.
When I build for Linux on mac (something I thought I was now going to be able to do), I do not see the yargs-parser error and all non-pty commands work.   However, contrary to my expectations, the pty commands are still broken in the same way as before the rebase.

Reopening on a preliminary basis as a heads-up.   I will try the smaller examples on Linux and see if I can pin the problem down further.

I have not been able to make a smaller example that would be easy for you to reproduce.   But, the fact is, that we will have no way to build for Linux after the rebase so I'm leaving the rebase on indefinite hold.   I want to keep this issue open since it's still an issue for us.  I'm hoping for some advice on how to pin down the problem.
Specifically, is the symptom "Can't find module 'yargs-parser'" just as mysterious to you as it is to us or do you have a clue about what might be going on?

possibly this? https://github.com/IBM/kui/pull/2790

Plausibly.   Is that PR about to merge?   If so, I'll wait for it and then just advance the rebase and retry.    Thanks.   If the PR will be pending for a while, I may try patching it in this PM (will be out much of the morning).

merged

Yes, that seemed to fix the problem.   I now have a solid build for LInux (when built on Linux).   There is the separate question of whether the cross-platform building scenarios are really working but that was never part of this issue so closing again.


Test: Support a server mode for debug adapters

Test for #22080:
Complexity: 2

[ ] AnyOS - @isidorn
[ ] AnyOS - @roblourens

VS Code supports to run debug adapters in server mode for a long time.
However, this server mode was only officially supported when developing debug adapters, not when running them in production. With this release we are starting to support the "server mode" as an official run mode for debug adapters.
Please note that this feature did not involve any code changes. We just tried to find a way to use the server mode for production by only using existing functionality.
Here is how the feature is used:
In your extension you can register a hook for the 'startSession' event, which is triggered when a new debug session is launched. In this hook you determine the port on which the debug adapter listens (or you can even start the debug adapter there). Then you assign the port number to the debugServer attribute (if this is still undefined).
Here is the complete 'xxx'-extension code that goes into the activate hook:
context.subscriptions.push(vscode.commands.registerCommand('extension.xxx.startSession', config => {
	if (!config.debugServer) {
		config.debugServer = 12345;	// determine debug port
	}
	vscode.commands.executeCommand('vscode.startDebug', config);
});

In the package.json you need these contributions:
"activationEvents": [
	// ...
	"onCommand:extension.xxx.startSession"
	// ...
],
"contributes": {
	"debuggers": [
		{
			"type": "xxx",
			// ...
			"startSessionCommand": "extension.xxx.startSession"
			// ...
		}
	]
}

Verify that...

a debug adapter (e.g. node-debug) works correct in server mode
focus on testing the life-cycle functionality of the debug adapter, e.g. 'restart'
there is no end user observable difference when running a debug adapter in server mode

Everything works fine. When the debug adapter gets restarted - the user session is stopped but the user can start new and it works fine which is expected.


chore:(use cases): Update use cases to CDK V2

Issue #, if available:
Description of changes:
-Updated README with clearer deployment instructions
-Changed old imports to new CDK V2 style imports
-Updated tests to CDK V2 style standards
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 21d6914cef5f60e681bfa35f99cd6efee0385e24
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 21d6914cef5f60e681bfa35f99cd6efee0385e24
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 2415e3ae8e7d6f005eddda201c02a77706bbad72
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 2415e3ae8e7d6f005eddda201c02a77706bbad72
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: b3cb2167d968e8bcd0f6ee88ac19096900d8b8d4
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 238d0bff7ff78623bb84bd6523be5781a1ed353f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 238d0bff7ff78623bb84bd6523be5781a1ed353f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 9f014709d6f3eb3e75df56f59e109dd2fa309b09
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 7317dc688e91e60f40d668d5ffabaf957d906fe4
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 7317dc688e91e60f40d668d5ffabaf957d906fe4
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: d1058622484a21e6e72b4b658d0345224929afec
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: d1058622484a21e6e72b4b658d0345224929afec
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 3d7f36208167f5193c99eeef80b24a1868cb0e88
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 3d7f36208167f5193c99eeef80b24a1868cb0e88
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: c7da2f37f841bf0bd11a0088f253a1985466df06
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: c7da2f37f841bf0bd11a0088f253a1985466df06
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 2c7295c917d5f4896304d01edc379a7a41a17ccf
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 2c7295c917d5f4896304d01edc379a7a41a17ccf
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 10fc01c44e748d8057ea7e33d9db510e7df67a99
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 10fc01c44e748d8057ea7e33d9db510e7df67a99
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Dockerized Icarus-terminal suggestion

Hey team,
First off, I just want to say I absolutely love what you've built here—this project is seriously awesome! I'm not super deep into coding or the technical side of things, so I don't have the chops to contribute directly to the tracker. But I did want to share something I put together that I think might help others in the community.
I dockerized the Icarus-terminal to run on a server at home. The only thing not inside the Docker container is a systemd unit to mount a folder via SSHFS to read the journal, and then it checks it in case your gaming rig sleeps. I host this on one of my homelab Docker VM servers. I use fedora silverblue as my daily driver/gaming rig. My hypervisor runs  a vm server that uses uses ubuntu server to run dockers. I apologize if I cant provide more detail to make it work but for the project leader and or those who know here you go. Yes chatgpt helped me acheive this and yes it seems to work great and all i do is then open a web page on one of my monitors on the gaming rig or any screen or computer or tv. its sweet!
Docker Run Command
docker run -d --name icarus-terminal --cap-add SYS_ADMIN \
  -v /mnt/elite-dangerous:/usr/src/app/logs \
  -e LOG_DIR=/usr/src/app/logs \
  -p 3300:3300 \
  --restart always \
  icarus-terminal

SSHFS Mount Script
#!/bin/bash

# Configuration
LOGFILE="/var/log/sshfs-mount.log"
MOUNT_POINT="/mnt/elite-dangerous"
REMOTE_DIR="/var/home/goffy59/.var/app/com.valvesoftware.Steam/.local/share/Steam/steamapps/compatdata/359320/pfx/drive_c/users/steamuser/Saved Games/Frontier Developments/Elite Dangerous/"
REMOTE_USER="goffy59"
REMOTE_HOST="pg1"
RETRY_DELAY=60
RESTART_DELAY=10

# Log function
log() {
    echo "$(date): $1" >> "$LOGFILE"
}

# Check if remote server is reachable
check_remote_server() {
    log "Checking if remote server $REMOTE_HOST is reachable..."
    if ssh -o ConnectTimeout=10 "$REMOTE_USER@$REMOTE_HOST" 'exit' 2>/dev/null; then
        return 0
    else
        return 1
    fi
}

# Check if mount point is mounted
check_mount() {
    if mountpoint -q "$MOUNT_POINT"; then
        return 0
    else
        return 1
    fi
}

# Mount remote directory
mount_remote_directory() {
    log "Mounting remote directory $REMOTE_DIR to $MOUNT_POINT..."
    if sshfs -o allow_other,uid=1000,gid=1000 -o cache=yes -o max_read=16384 "$REMOTE_USER@$REMOTE_HOST:$REMOTE_DIR" "$MOUNT_POINT"; then
        log "Mount successful."
        return 0
    else
        log "Mount failed."
        return 1
    fi
}

# Restart Docker container
restart_docker_container() {
    log "Restarting Docker container icarus-terminal..."
    docker restart icarus-terminal
    sleep "$RESTART_DELAY"
    # Removed rdesktop container restart
}

# Main function
main() {
    while true; do
        if check_remote_server; then
            if ! check_mount; then
                # Mount the remote directory if not already mounted
                if mount_remote_directory; then
                    # Restart Docker container if mount is successful
                    restart_docker_container
                fi
            fi
        else
            log "Remote server $REMOTE_HOST is not reachable. Retrying in $RETRY_DELAY seconds..."
        fi
        sleep "$RETRY_DELAY"
    done
}

# Run the main function
main

Systemd Unit to Run Script
[Unit]
Description=SSHFS Mount Service
After=network-online.target
Wants=network-online.target

[Service]
ExecStart=/usr/local/bin/sshfs-mount.sh
Restart=always
User=root
Group=root
StandardOutput=append:/var/log/sshfs-mount.log
StandardError=append:/var/log/sshfs-mount.log

[Install]
WantedBy=multi-user.target

Dockerfile
# Use an official Node.js image as a base
FROM node:18

# Set the working directory inside the container
WORKDIR /usr/src/app

# Install SSHFS, FUSE3, and Supervisor
RUN apt-get update && \
    apt-get install -y fuse3 sshfs supervisor && \
    apt-get clean

# Enable allow_other in FUSE configuration
RUN echo "user_allow_other" >> /etc/fuse.conf

# Clone the ICARUS Terminal repository into the current directory
RUN git clone https://github.com/iaincollins/icarus.git ./

# Install Node.js dependencies
RUN npm install

# Expose the port ICARUS Terminal will run on
EXPOSE 3300

# Set environment variables (adjust LOG_DIR later)
ENV LOG_DIR=/logs

# Ensure logs directory exists
RUN mkdir -p /usr/src/app/logs && \
    chmod -R 777 /usr/src/app/logs

# Copy the Supervisor configuration
COPY supervisord.conf /etc/supervisor/conf.d/supervisord.conf

# Copy the startup script
COPY start-app.sh /usr/src/app/start-app.sh
RUN chmod +x /usr/src/app/start-app.sh

# Command to start Supervisor, which will manage the app
CMD ["supervisord", "-c", "/etc/supervisor/conf.d/supervisord.conf"]

.env File
LOG_DIR=/mnt/elite-dangerous

start-app.sh Script
#!/bin/bash

# Function to check the logs for specific error messages
check_logs() {
    log_file=/usr/src/app/logs/icarus.log

    if grep -q "Failed to lookup engineer" $log_file || \
       grep -q "Scanned 0 files" $log_file || \
       grep -q "Imported 0 events" $log_file; then
        echo "Detected error in logs, restarting the app..." >> /usr/src/app/logs/startup.log
        return 1
    else
        return 0
    fi
}

# Start the Node.js app
npm start >> /usr/src/app/logs/app_output.log 2>&1 &

# Monitor the logs and restart if necessary
while true; do
    sleep 120  # Check every 2 minutes
    if ! check_logs; then
        pkill -f "node src/service/main.js"
        npm start >> /usr/src/app/logs/app_output.log 2>&1 &
    fi
done

supervisord.conf
[supervisord]
nodaemon=true

[program:icarus]
command=/usr/src/app/start-app.sh
stdout_logfile=/usr/src/app/logs/supervisor_stdout.log
stderr_logfile=/usr/src/app/logs/supervisor_stderr.log
autostart=true
autorestart=true
startretries=3
startsecs=5

Project Structure
icarus-docker/
├── .env
├── Dockerfile
├── start-app.sh
└── supervisord.conf

I hope this can be useful to others. Keep up the amazing work!
Cheers,
goffy59

Oh this is amazing, thank you for putting so much work into this.
I had thought about this but I wasn't previously bothered to do it given what I thought would be a very small number of people, but folks have logged the issue here #55 and even messaged me directly about it.
Even if I can resolve the issue this is really nice to have, I will see if I can get it merged it.
I am also happy to open up Pull Request access. I had previously disabled that as I wasn't in a position to be able to accept / review requests at the time (just enough free time to work on the project, not enough time for that!) but things are bit different now and would be happy to accept changes!

Hey Iain,
Thanks for the kind words! I'm glad the Docker setup and SSHFS script were helpful.
I use SSHFS mainly for the security, but I understand it can be a bit slow and tricky to set up. Samba might be a better choice for most users, especially those on both Linux and Windows, since it’s generally easier and faster. I just wanted to share what I’ve got in case it helps anyone looking for a more secure setup.
Feel free to use or tweak it as you like. If you need more info, just let me know—I’m happy to help if I can. While I’m not an expert in Dockerizing entire projects from GitHub, the Dockerfile I shared should be pretty close to what other projects use for Docker deployment. You might want to double-check for best practices, but it’s a solid starting point.
Thanks again for all your hard work on this!
Cheers,
goffy59


Added Nueva Vizcaya State University

lib/domains/ph/edu/nvsu.txt

Here are some proofs that Nueva Vizcaya State University is a recognized institution here in the Philippines:

The University's Official Website: https://www.nvsu.edu.ph/
The University's Offered Programs (Including Information Technology and Computer Science): https://www.nvsu.edu.ph/programs (Note: It is under College of Arts and Sciences)
The University uses Microsoft as its email domain provider through Outlook and also I am enrolled as a student in the said institution. (Here is the log in link where students here access our own institution account.)

Here are some proofs that Nueva Vizcaya State University is a recognized institution here in the Philippines:

The University's Official Website: https://www.nvsu.edu.ph/
The University's Offered Programs (Including Information Technology and Computer Science): https://www.nvsu.edu.ph/programs (Note: It is under College of Arts and Sciences)
The University uses Microsoft as its email domain provider through Outlook and also I am enrolled as a student in the said institution. (Shown in the image below)
Also Here is the log in link where students here access our own institution account: https://cloud.nvsu.edu.ph/

@daeyeonnn First stage of verification passed, official domain is nvsu.edu.ph as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@daeyeonnn Pull request merged. Thank you.


Fix flaky translation test

This replaces the call to a stubbed method with a call to the original
method. This should be more reliable and less likely to break during
CI runs.

I think I fixed it but I'm feeling good about it 😅


chore: Drop the UnexpectedSpecVersion error variant

This error variant is only used on startup, it doesn't bring any value to have it exposed through the RPC interface.

IMHO spec version mismatch is a valid error variant and there is nothing wrong with it per se.
What is wrong is the unnecessary coupling between different error kinds: run and rpc ones.
I see the first step in organizing errors properly more like this: https://github.com/eigerco/beerus/pull/614


[Wait4PR#672] [ Layer ] Add input_layers keyword

This PR includes enabling 'input_layers' keyword.
With this keyword, we could specify layer's input tensor.
. Added skip in parse_util to remove space and '[',']' in string and
split with ',' delimiter.
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #684. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @jijoongmoon, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.

:octocat: cibot: @jijoongmoon, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/684-202010291620460.9317889213562-ce12b0d09677e08ed097d378bf04f3b58fbc818a/.

:octocat: cibot: @jijoongmoon, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.

Related to #469

:octocat: cibot: @jijoongmoon, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.

:octocat: cibot: @jijoongmoon, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.


[chore] migration script for when assistant config strings are changed

stuff saved on the /data volume need to be updated to use new strings

Did manual migration
fly ssh console
cd ../data/student
sed -i 's/rune-/rune_/g' *

Added a note to the top of the config strings warning about changing the strings


[Proposal]Optimize the memory consumption of TransactionMgr

In production enviorment, the data loading is 7*24 with lots of txns, but we found that the TransactionMgr costs too much memeory, which will leads OOM.If we want to improve the concurrent of stream load, we must optimize the memory usage in order to support frequently data load.There are some methods to optimize such as clean up useless data after txn finished #3130, reduce the interval of clean txn, use ZGC to recycle memory in a short time....But It is not enough, I found FE will OOM accidental cause by TransactionMgr usage.
There are two way to reduce the memory usage, one is change the data struct in DatabaseTransactionMgr to save memeory, the other is scattered the Txn to other follower FE to share the memory usage, I will investigate the feasibility of two solutions, any advice or solution is welcome.

By the way, too much DatabaseTransactionMgr will cause some more serious accident.For example, if DatabaseTransactionMgr costs too much memory, checkpoint image will not work because it will check java heap memory in order to create image in memory, so when FE down cause by OOM, it will replay too much EditLog, in the worst case, it will replay more than one hour, It means that data load is not available in the meantime.

I found that most of memory usage in DatabaseTransactionMgr is a HashMap struct idToFinalStatusTransactionState, following shows that 270W TransactionState object almost cost 50G memory, but some data in TransactionState is useless, so we have no need to save the data.
Label | Number of Objects | Used Heap Size | Retained Heap Size
-- | -- | -- | --
java.util.HashMap$NodeFirst 10 of 2,718,787 objects | 2,718,787 | 130,501,776 | 53,717,135,376

I think the Check Point Thread can run in follower.
It's not necessary for Master FE to run it.


Pull Request Dashboard

Summary
Customizable version of https://github.com/pulls designed to help users quickly see which pull requests need their attention. Users today must click through multiple tabs and continually refresh the page to get the same list, often resulting in delays and inconvenience on other users that must continually "ping" other users to review or perform other actions on their pull requests.
Intended Outcome

Help users quickly tell which pull requests are waiting on them
Reduce the avoidable delays that inevitably happen in every pull request that slow or block progress
Higher productivity for individual contributors and teams

How will it work?
When a user navigates to https://github.com/pulls, they will see a consolidated view of the pull requests needing attention. This view can be easily filtered or adjusted and saved as a custom view that can be returned to later. This view is also updated on the fly to ensure the latest status is presented so the user can stay on top of the pull requests they are involved in.

This issue is being closed as outdated. For more information, please check out this Discussion post. Stay tuned for new additions to our refreshed public roadmap!

This issue is being closed as outdated. For more information, please check out this Discussion post. Stay tuned for new additions to our refreshed public roadmap!

@ankneis, you really should provide specific rationale for why you've closed each issue, since some of them are incredibly popular, and continue to appear to make significant sense (to me, at least).

We wanted to provide more details on why this issue was removed from the public roadmap. An improved experience for finding pull requests that most need attention is currently in an exploratory phase, so we have removed it from this roadmap while we develop more definition and certainty on our scope and timelines.


Docker swarm DNS and load balancing behaviour

Will swarm internal DNS keep entries for unhealthy containers ? If i have scaled a service to 4 containers & if one of them goes down how the DNS & LB will react to it ?

I meet the same problem , Please anybody help me?

This is coming in docker 1.13, see https://github.com/docker/docker/pull/27279, and https://github.com/docker/docker/issues/24092

I'll close this issue, because it's fixed in master, and will be coming in 1.13, but feel free to comment. If you have a system to test on, release candidates for 1.13 are now available for testing https://github.com/docker/docker/releases (but don't use them for production yet)


Add release dates to changelog

It's often helpful to see the release dates in the changelog.

I guess I don’t have a problem with doing this retroactively. FYI, the reason it isn’t typically done is that the release notes are built into the app and we don’t necessarily know when the actual date will be. We might put a new version as a developer preview for a month. If it looks good, the same build will just get flagged as a “pre-release” in the update system. If it still seems OK, we’ll flag it as a final release sometime later. So the “release” date is often several weeks after the build date.
In the long run, those weeks probably don’t matter, so maybe we should start putting the date of the initial build. Yeah, you’ve talked me into it.
FYI, there are dates listed on https://qsapp.com/archives/ if you want to add any more. Otherwise, I’ll merge this.

Cool, I've added the released dates in for all of the changelog.

Thanks!


can't open file

\terminal\src\inc\LibraryIncludes.h(51,10): fatal error C1083: can't open file : “wil/Common.h”: No such file or directory

You gotta restore submodules man, it’s in the READMEZ

thanks！！

You gotta restore submodules, friend, it’s in the README.
thanks！！


error listing containers: dial unix /tmp/docker.sock: connection refused

When running nginx-proxy on my docker host I get the following error in the container log:
error listing containers: dial unix /tmp/docker.sock: connection refused

I'm connecting remotely to docker host using certificate authentication - is it possible to configure nginx-proxy to use cert auth?

I'm guessing that I could set the environment variables as per https://docs.docker.com/reference/commandline/cli/#environment-variables. However, the cert for my docker host doesn't match the host name.
This means that I have to use the --'tis cmd line switch rather than --tlsverify. Looking at the docs I can't see a way to match this in environment variables. Is there a way to trigger this for nginx-proxy/docker-gen?

You can add a hosts entry (using docker run's --add-host param) to the nginx-proxy container that matches the name in the certificate to the IP of the host - would that solve the host name verification issue?
You can then set DOCKER_TLS_VERIFY, DOCKER_HOST and DOCKER_CERT_PATH environment variables on the container in order to configure TLS on the docker client inside nginx-proxy.

I don't think so - the name in the cert is wildcarded.

I think the same approach would work, though, wouldn't it? That is, pick a domain that matches the certificate's wildcard and add a host entry in the container which resolves it to the docker host's IP.

I just ran into this issue after updating to Docker v1.7.1

Is this related to #166?

No longer having this issue after rebooting.

I got an error when I tried add host with docker run. For now I've just gone with static config. I definitely want to get this working, so will come back to it. I may recreate the docker host without a wildcard cert and then try it :-)

Had the same issue when upgrading to 1.7.1, and like with @adambiggs rebooting my docker machine solves the issue.

I have now got my docker host configured so that --tlsverify works, and nginx-proxy worked first time! Thanks :-)

@stuartleeks do you mind sharing your final config that got all this working? Rebooting the VM isn't the end of the world, but one less thing for my team to worry about would be great.


UBERONparcellation: V4

Hi! I want to add V1 and V4 to a macaque dataset (this one specifically: https://www.nature.com/articles/s41597-022-01180-1#citeas). I can use UBERONparcellations, but only primary visual cortex exists, so I would like to ask you to add V4. I could however not immediately find it in the ontology search (https://www.ebi.ac.uk/ols/ontologies/uberon). Hope you can help me find a viable solution.

@ingrre V4 is not registered in UBERON (yet). Did you ask the data provider if they typically use a specific macaque atlas/parcellation?
For EBRAINS curation: Feel free to add areas you cannot find anywhere as Custom Anatomical Entity. Note: For macaques we should move to using the areas of the MEBRAINS atlas if the data provider does not state he/she used an other specific macaque atlas. Note that the MEBRAINS areas are not yet registered in SANDS but for EBRAINS you can find them some of them already registered as Custom Anatomical Entity (Area V4 would be actually two V4v + V4d in MEBRAINS).

This was solved by the use of custom anatomical entities. We considered using UBERON parcellations but it turns out that V4 does not exist or at least I couldn't find it. The closest match was "visual association cortex" (which includes more than just V4), "extrastriate cortex" (which includes V2, V3, V4 and V5) or "Brodmann area 19" (which still includes V2, V4 and V5, and also has a "human" ring to it).
This is just to show how the issue was solved and to note that V4 does not exist stand-alone in UBERON. I will close this issue now since there is nothing more to do in relation to it as far as I can tell.


Letting each condition control its frequency

Using throttling, each condition was being checked once per minute. But a condition may need to perform its check at a different pace like for example the TaskMonitor condition. This change applies throttling at individual condition instead of applying it generically at the subprocess level. The TaskMonitor condition will not be executed once per 10 seconds instead of one minute. This will allow for faster evaluation of worker idleness which would help in scaling down the worker fleet a lot more effectively.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

These changes also need to be ported to 2.10.1.

I believe I have covered these in https://github.com/aws/amazon-mwaa-docker-images/pull/151. Is there anything you saw that is missing in that PR?

These changes also need to be ported to 2.10.1.

I believe I have covered these in #151. Is there anything you saw that is missing in that PR?
I didn't check the other PRs to see if the changes were ported there. Nothing seems to be missing. LGTM.


Create a simple shader plugin with CanvasKit

Supports:

Text area to modify/input shader code
Preview shader in the plugin UI
Insert shader code into the shared plugin data of the selected node that supports background image fill
Default shaders(from skia and shader toy)
Generate an image and insert into the figma selected node

What don't work yet:

Not allow other uniforms other than iTime and iResolution
Don't support strokes

Towards: #1714

We have some pretty serious licensing concerns to deal with before I can unblock this PR.

Added a LICENSE file for MIT license(used by star nest) and Unlicense license(examples from skia/twitter). Please take a look.

We have some pretty serious licensing concerns to deal with before I can unblock this PR.

Added a LICENSE file for MIT license(used by star nest) and Unlicense license(examples from skia/twitter). Please take a look.

Ok, I think that's all fine. I've just found go/releasing/preparing#third-party-components, however, which states that those files need to be moved to a /third_party directory in the root of the repo. So we'll need to make that change as well.

We have some pretty serious licensing concerns to deal with before I can unblock this PR.

Added a LICENSE file for MIT license(used by star nest) and Unlicense license(examples from skia/twitter). Please take a look.

Ok, I think that's all fine. I've just found go/releasing/preparing#third-party-components, however, which states that those files need to be moved to a /third_party directory in the root of the repo. So we'll need to make that change as well.

Done.

We have some pretty serious licensing concerns to deal with before I can unblock this PR.

Added a LICENSE file for MIT license(used by star nest) and Unlicense license(examples from skia/twitter). Please take a look.

Ok, I think that's all fine. I've just found go/releasing/preparing#third-party-components, however, which states that those files need to be moved to a /third_party directory in the root of the repo. So we'll need to make that change as well.

Done.

Looks good.


"Edit on GitHub" for "stable" points to commit-id

Details

Project URL: https://readthedocs.org/projects/pyinstaller/
Build URL:http://pyinstaller.readthedocs.io/en/stable/
Read the Docs username: htgoebel

Expected Result
The link behind "Edit on Github" should point to the blob under the most current tag (here: v3.3.1):
https://github.com/pyinstaller/pyinstaller/blob/v3.3.1/doc/index.rst
Actual Result

The link points to a blob under a commit-id:
https://github.com/pyinstaller/pyinstaller/blob/6ed6dd2f52f3d4409292a484e8f19a8ce1daef43/doc/index.rst
Github is not able to show this blob (Error 404)

Additional information

This commit-id is the one of the tag:

$ git rev-parse v3.3.1
6ed6dd2f52f3d4409292a484e8f19a8ce1daef43

latest stable version tag is v.3.3.1 with commit-id 5b84a8da71048fc888bc4240d653034394a76d32
tried wiping and rebuilding without success https://readthedocs.org/projects/pyinstaller/builds/7009974/

Thanks for reporting, this issue is a duplicate of #1820
Also, RTD is moving to use gitpython instead of parsing the tags/branches manually, so I suppose that would fix the issue.


Support C++17 features in Catch2

Does the Catch2 port support compiling with C++17 features enabled?
For example:
inline std::string_view test()
{
    throw std::runtime_error{"bad"};
}

TEST_CASE("Log exception")
{
    REQUIRE(test() == "good");
}

I get a linker error when using catch2 v3.0.1 but not when using v2.*.
Note that Catch2 compiles with C++14 by default.
For more context, see:
https://github.com/catchorg/Catch2/issues/2046
https://github.com/catchorg/Catch2/issues/2210
Is some kind of feature switch needed, or should the port just always use C++17?

We do not accept adding c/cxx standard as a feature.
See https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/maintainer-guide.md#do-not-use-features-to-control-alternatives-in-published-interfaces

What is the recommended solution to provide a port that "just works" for consumers on C++17 or higher? The abseil[cxx17] technique from the linked page?

I'm looking for a way that doesn't require using a port overlay (or patch on top of the vcpkg repo). What's the current best solution in vcpkg? (Perhaps there's a larger vcpkg feature ask here.)

The best way is to let upstream itself set the default language standard to c++17.

Upstream intentionally supports back to C++14, as documented in their About section here:
https://github.com/catchorg/Catch2
How do other upstream libraries handle supporting older C++ versions?
Suppose a project wants to support compiling with both the latest C++ standard or with older versions, and "just work" either way. Is that possible to do with vcpkg?

Suppose a project wants to support compiling with both the latest C++ standard or with older versions, and "just work" either way. Is that possible to do with vcpkg?

It is their freedom to use that language standard upstream, vcpkg does not make any restrictions on this.
If some users need to use some specific language standards, they can either report their requirements to the upstream, or use the overlay ports feature of vcpkg to customize the ports they need locally.

Does vcpkg support projects that offer compilation with multiple language versions, with a lower minimum but more features on more recent versions?

If the port itself supports the use of different language standards under different conditions, then vcpkg supports such a port using different conditions as its own feature.

In that case, can we add a c++17 feature to the catch2 port? (Or am I misunderstanding your reply?)

Are there explicit conditions in the cmakelists.txt provided by upstrem to control the use of c++17 or c++14?

Not that I see, though I'm not a CMake expert:
https://github.com/catchorg/Catch2/blob/devel/src/CMakeLists.txt
What would upstream need to do for vcpkg to support a c++17 feature for it?

Could you please submit a PR for this issue?
We can judge whether it conforms to the specification according to the actual modification of the PR.

There is #25019.
And there is discussion in #24762.


Blog Post

Hi,
I think it's super existing to work on a blog post on this package. I am planning to do it on next week.
If you can write it that'd be awesome. If not, I can work on that too.
Either way, I'm pretty excited about this.
@tomitrescak what do you think?

Hi, I think it is best if you write the article. You have your audience and a bigger impact. I would love to have someone to help as the semester starts and I have way too much teaching to do so I'll need someone to help with bug calls ;) Thanks! Looking forward to your article!

I liked the blog post;) thanks!


Anything you wish to add as a feature

Add any ideas you have, and I will create an issue, so you can work on this feature or improvement.

@mnosov622 I identified two bugs

The click listener to mark tasks as complete is only applied when a new task is added. So for tasks loaded from the localStorage, the listener is not applied by default. It is only applied when a new task entry is made, upon which createItem() will add the listener to all existing tasks.

lines 26 to 34 are irrelevant. listOfTasks is undefined.

I would love to work on these, so it would be great if you could assign them to me.

Hi @abinjohn123 !
Thank you for your attentiveness!
I transferred the logic into function. It all works now. You can make a PR if you see something not working properly :)

Got it 👍🏽

Hello! @mnosov622 I want to help in this issue. Please assign me.
I want to add a footer


WIP: Use itk_generate_factory_registration

ITK 5.3 adds FFT factories which need special configuration to
ensure the generated FFT factory code is generated and registered for
the appropriate SimpleITK targets.
Use new itk_generate_factory_registration CMake function to generate
private registration methods, which find grain control of factories.

Dependent on InsightSoftwareConsortium/ITK#3059


Add Greg on bech32 design

Is this just a copy of the referenced bitcointalk post? Do we consider this a transcript?
We also have the example of https://github.com/bitcointranscripts/bitcointranscripts/pull/126 for which I also suggested that we could introduce an article category. Maybe a category distinction here would also make sense?

@kouloumos: This is the approach I personally take.

I feel that an interesting explanatory Reddit post or Bitcointalk post on a technical topic can squeeze into the transcripts definition (others may disagree) as a standalone transcript even though it was never delivered verbally. Especially if you would seek to link to it from another transcript or it would likely get lost and forgotten about otherwise. Just be clear where the "transcript" has come from.

I don't think we want to collect every article ever written on Bitcoin technical topics in this repository but Greg especially has a number of educational comments scattered around Reddit and Bitcointalk that aren't indexed and risk getting lost otherwise.

I agree with that approach.
I also saw that this is not the first bitcointalk/reddit post to be included. Do you think that having bitcointalk,reddit categories for this kind of content would make sense? Mostly for better indexing.

Sounds good!


Updates Cohere connector info in connectors.md

Description
Update the Cohere connector documentation:
https://opensearch.org/docs/latest/ml-commons-plugin/remote-models/connectors/#cohere-connector
Update the blueprint to use the latest version for cohere embed v3. Please work with @ylwu-amzn to obtain the latest blueprint: https://github.com/opensearch-project/ml-commons/blob/2.x/docs/remote_inference_blueprints/cohere_connector_embedding_blueprint.md#1b-crea[…]nnector
Issues Resolved
Addresses the first item in https://github.com/opensearch-project/documentation-website/issues/6648
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @kolchfa-aws , can you please do a doc review on this? Thanks!


[WIP] Support for Point component

Description:
Adds support for the minut.com Point sensor/alarm.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):
point:
  client_id: <CLIENT_ID>
  client_secret: <CLIENT_SECRET>

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

Todo:

[ ] Smoother authentication (does HA have a preferred way to handle components with OAuth2?).
[ ] Integrate the new webhooks (where is the documentation/example code for this?)

You should look at doing this integration through config entries.

We are required to support for:

Handles internet unavailable. Log a warning once when unavailable, log once when reconnected.
Handles device/service unavailable. Log a warning once when unavailable, log once when reconnected.

for the Silver Quality Scale, what is the recommended way of doing so? Is this enough?

For logging? That should be enough. Maybe some kind of identifier if you can connect to multiple clients
Don't forget that you have to move your MinutPointClient to an external lib

For logging? That should be enough. Maybe some kind of identifier if you can connect to multiple clients

Great!
What about SCAN_INTERVAL is there some example code I can look at on how it should be implemented or is this ok?
Or is it better to use track_time_interval(hass, self._sync, scan_interval)?

Don't forget that you have to move your MinutPointClient to an external lib

No I'm fully aware of that but have had so many changes that it's easier if everything is in the same place.

scan interval is used like this: https://github.com/home-assistant/home-assistant/blob/6b3e4ca7bd167295e7984d2c00b2c4c5e748781a/homeassistant/components/switch/unifi.py#L24

scan interval is used like this:
home-assistant/homeassistant/components/switch/unifi.py
Line 24 in 6b3e4ca
SCAN_INTERVAL = timedelta(seconds=15)

And how is that used (I can't find a documentation for this)? Only the platform needs to update as that updates all sensors..

Sometimes you need to ask the code ;)
Somewhere in entity platform or something will use that parameter to set the update interval for that platform.

Was looking for something else and found it.
https://github.com/home-assistant/home-assistant/blob/6b3e4ca7bd167295e7984d2c00b2c4c5e748781a/homeassistant/helpers/entity_component.py#L138

@Kane610, I have a working configuration flow with authentication against minut.com now! Just a few questions:

What do I need to do to get my devices (sensors) to show up under integrations?
Do I have to make all translations or will it fall back to components/point/strings.json?
I'm not super happy with the current authentication flow, I would like to redirect the user after entering API secrets and automatically show the success dialog when directed back. I.e., the second dialog below should be automatic.
I'm using http.register_view() as below, is there a way to de-register the view after the user is authenticated (I read the code but couldn't find anything). https://github.com/home-assistant/home-assistant/blob/8c61b86dce33ea53f4fb0ad048c1b7c5687d22f5/homeassistant/components/point/config_flow.py#L111

You need to add https://github.com/home-assistant/home-assistant/blob/fc8af22191d054524c9e124a5fed7ebade32f16c/homeassistant/components/binary_sensor/deconz.py#L124
you need to do one translation, preferably English, but Blekingska is good as well ;)
You can redirect to different flows whenever you want, if I understood you correctly
I've never used register view so I can't help you there, but can't you just set the object to None?

@MartinHjelmare, do you know how I can redirect the user to a flow from a hass.http.register_view?

I don't know specifically about that. Maybe if you describe what you want to do on a higher level.
We have the docs on data flows here:
https://developers.home-assistant.io/docs/en/data_entry_flow_index.html

@MartinHjelmare I'm trying to do what I described before:

I'm not super happy with the current authentication flow, I would like to redirect the user after entering API secrets and automatically show the success dialog when directed back. I.e., the second dialog below should be automatic and go to self.async_create_entry. When the user is redirected back to HomeAssistant.

If I could generate a short html page (and not a content-type: json) in the HomeAsssitantView and I know the url to the FlowDialog I could make the last "Submit" automatically..

Why can't you use ordinary config flow entries to get parameters and just do a web request?

Why can't you use ordinary config flow entries to get parameters and just do a web request?

Because that is not how oauth2 works.

I think everything is done now :1st_place_medal:
I do not know how to remove the WIP-status though.

To remove WIP status you have to rebase and reword the commit messages that mention wip.

To remove WIP status you have to rebase and reword the commit messages that mention wip.

I rebased quite heavily, (went from 31 commits to two).
🤞that it is good enough now 😃

@MartinHjelmare, @balloob I'm waiting for your replies on my comments on the last review. Is there anything for me to change or fix, or is the current implementation on.

@MartinHjelmare is my changes ok or are there more things that I need to fix?
What is the timeframe for 0.82.1?

I think this looks good, but I'd like to read the tests before I approve.
This will not go out before 0.83 since it's a new feature.

I trust Martin :)

Beta hasn't been cut so it's part of 82.

I think that was a typo: 0.82 -> 0.83

@MartinHjelmare & @balloob, Thank you for your comments and improvements for enabling this. I really evaluate the code quality that you are striving for.


ARROW-11081: [Java] Make IPC option immutable

By making it immutable, the following benefits can be obtained:

It makes the code easier to reason about.
It allows JIT to make more optimizations.
Immutable objects can be shared, so many object allocations can be avoided.

This PR also fixes a bug in ArrowMessage#ArrowMessage(ArrowDictionaryBatch, IpcOption)

Looks good to me

Looks good to me

@liyafan82 does this actually make a difference in benchmarks?  I agree it is easier to reason about, but is there any way to avoid backward incompability?

@liyafan82 does this actually make a difference in benchmarks?  I agree it is easier to reason about, but is there any way to avoid backward incompability?

@liyafan82 does this actually make a difference in benchmarks? I agree it is easier to reason about, but is there any way to avoid backward incompability?

@emkornfield Sorry I do not have a benchmark result. I think it should be a positive change, as it avoids object allocations.
To avoid backward incompability, I have restored the default constructor.

@liyafan82 does this actually make a difference in benchmarks? I agree it is easier to reason about, but is there any way to avoid backward incompability?

@emkornfield Sorry I do not have a benchmark result. I think it should be a positive change, as it avoids object allocations.
To avoid backward incompability, I have restored the default constructor.

In general I think this is good in the longer but since this is a breaking change, I'd like to get others thoughts before merging.  @BryanCutler @lidavidm any objections?

In general I think this is good in the longer but since this is a breaking change, I'd like to get others thoughts before merging.  @BryanCutler @lidavidm any objections?

I'd be in favor of this, if we're going to break compatibility, I'd rather we do it sooner. And if this is a major regression, it should be simple to revert as well (we can just make the fields not final again and encourage people to migrate to the constructors).

+1 going to merge.

Apologies for the delay, I agree sooner than later for this so +1 for me.


Theme css files not being pulled in

I have just started using ember-cli-toggle and I'm finding the css theme files are not getting compiled. If I copy and paste those files into base.css it works.

Did you specify the theme in the config?

No I didn't touch environment variables. Is that required for install?

The 'default' theme is imported by default, other themes have to be specified in the config.

I could be wrong but I don't believe I was even getting the default themes.

Should be fixed in latest version.

Awesome thanks.


Bug fixes

made bug fixes to psykana tab, repeatable fields, and fixed macro that was doubling Willpower/Perception/Psyniscience during channel rolls.

You've got merge conflicts and I cannot complete the PR. You need to make sure that your local branch is up to date before making changes. You can do this by either pulling from the master branch or deleting and re-forking before you do your work. Please fix these conflicts and reopen the PR and then I will look into merging it.


Add tests for the cohost linked editing range endpoint

Part of https://github.com/dotnet/razor/issues/9519
Finally some cohosting tests! Some people would say that I'm being lazy in adding tests for the simplest endpoint we have. Those people have a point.
The test infra here almost entirely avoids ServiceHub, and certainly avoids the Roslyn solution sync mechanism, but it does use our real services and service factories, including the OOP services' separate MEF composition, so the services themselves are partying on a real Roslyn (test) solution and are using real implementations of all of their dependencies.
Things not covered by this test infra yet:

OOP initialization
Adding generated C# files to the Roslyn solution (IDynamicFile does this in real life)
Dealing with Html documents in any way

Future PRs to add tests for more endpoints should add these as needed.

Have responded to the feedback here, thanks very much. Going to merge this as is for now, just to keep the code flowing. Lots more to come though, outlined in https://github.com/dotnet/razor/issues/10603, so I suspect some of this infra will change soon enough.


Logger with hierarchy

With this changes you can now display Logs with Hierarchy. Works with popup() and fullscreen() if ypu provide suitable profile.
( If you call popup() with a Profile it displays it in fullscreen anyways )
New Parameter is filled with DEFAULT VALUE 1.
Should work with all Message types, I Tested:

Exceptions
Strings
System Messages
BAPIRET
Table of BAPIRET

To test it you simply have to fill new Parameter 'detlevel' and get the correct profile:
DATA(log) = zcl_logger_factory=>create_log(  ).

"First Group
log->s( obj_to_log = 'Airline 1' detlevel = '1' ).
log->e( obj_to_log = 'Passenger 123736GHB has not arrived at the Gate' detlevel = '2' ).

DATA: l_s_display_profile TYPE bal_s_prof.
CALL FUNCTION 'BAL_DSP_PROFILE_DETLEVEL_GET'
  IMPORTING
    e_s_display_profile = l_s_display_profile
  EXCEPTIONS
    OTHERS              = 1.
log->fullscreen( l_s_display_profile ).

Add new Logic. Resolves https://github.com/ABAP-Logger/ABAP-Logger/issues/140

Resolved conflicts. Will get back to it after #135 is merged

Merge branch main into logger_with_hierarchy first, then merge here

@HuprichT : do you have the option to rebase your branch (cf. https://github.blog/changelog/2022-02-03-more-ways-to-keep-your-pull-request-branch-up-to-date/)? I would be easier to resolve the conflicts and merge your PR.
Thanks!

I did some testing...

Default for detlevel

I think it's a good enhancement. However, using detlevel = '1' as the default breaks existing implementations. This becomes apparent when executing the unit tests:

Change it to detlevel TYPE ballevel OPTIONAL and it works fine.

Structures and exceptions

detlevel should be passed to add_structure and drill_down_into_exception as well.

HI @mbtools , thanks for the suggestions.
However I would like to discuss the Impacts based on that. The Unit Tests Fail because expected_data is not met. During the runtime we add detlevel = '1' to all Logs ( which does not change the output in reality though ) and this value is then missing.
Now let's say detlevel is changed from DEFAULT '1' to OPTIONAL, now we have the problem:
If a Log is build with hierarchy, and over the time the Developers forget to add detlevel to new entries, it's then not shown on the Output.
Example Code
DATA(log) = zcl_logger_factory=>create_log( object = 'ABAPUNIT' ).

log->s( obj_to_log = 'Airline 1' detlevel = '1' ).
log->s( obj_to_log = 'Passenger too late' detlevel = '2' ).

log->s( obj_to_log = 'Airline 2' detlevel = '1' ).
log->s( obj_to_log = 'Passenger too late' ). "Will not be display if provided with detlevel display profile

DATA: display_profile TYPE bal_s_prof .
CALL FUNCTION 'BAL_DSP_PROFILE_DETLEVEL_GET'
  IMPORTING
    e_s_display_profile = display_profile                 " Display Profile
  .

log->display_fullscreen( profile = display_profile ).

Output: Should vs. Actual:

So in the approach with OPTIONAL we would have to implement some kind of check inside if previous logs do have detlevel. If yes the default value '1' should be applied.
What do you think?

Currently, the expectation is that detlevel is 0 for all entries. We don't know what solutions exist that depend on it. So changing this would be breaking.
If developers want a hierarchical log, they have to ensure themselves that detlevel is passed everywhere. That's no different than setting other optional parameters. Adding checks makes this more complex than necessary.

Thanks to both of you!


Updates Dependencies

Blindy bumps all the base dependency versions.
Updates the default babel configuration to use the "latest" preset rather than ES2015 to ensure it stays up-to-date.

Warnings

:warning:
Please add a changelog entry for your changes.

  Generated by :no_entry_sign: dangerJS


Don't set securityContext.fsGroup by default. Fix #105

do not set fsGroup by default. see #105

Thanks @rigazilla


Doesn't work for cloudflare

Hi,
I tried to run your example in my project, but the only thing that I get when running yarn build / post build is:
file:///build/server/index.js:12
import { renderToReadableStream } from "react-dom/server";
         ^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: Named export 'renderToReadableStream' not found. The requested module 'react-dom/server' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from 'react-dom/server';
const { renderToReadableStream } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:146:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:229:5)
    at async ModuleLoader.import (node:internal/modules/esm/loader:473:24)```

    Can you help me out?

TL;DR
Try this.
https://github.com/caprolactam/remix-cloudflare-sitemap/blob/73a02981c7fc244420b9564a2ad6533ffce11887/types/deps.d.ts#L1-L3
https://github.com/caprolactam/remix-cloudflare-sitemap/blob/e49db7a533224f5368391d4fdf86298b3c9e3ff3/app/entry.server.tsx#L10
Why?
This script runs on Node.js. However, there is an issue where renderToReadableStream cannot be imported in a Node.js environment. I think that this is because react uses different exports for Node.js and edge environments. [Article](This is in japanese,  but you can follow codes.)
You can work around this issue by explicitly importing renderToReadableStream from react-dom/server.browser. Additionally, since React does not provide types for it, you will need to either use @ts-ignore or create your own type definition file.
// @ts-ignore
import { renderToReadableStream } from "react-dom/server.browser";

or
// type/deps.d.ts
declare module 'react-dom/server.browser' {
  export * from 'react-dom/server'
}

By the way, React Router v7 supports pre-rendering. While I haven't tried it, it might be possible to use @nasa-gcn/remix-seo normally and add prerender option.
// react-router.config.ts
import type { Config } from '@react-router/dev/config'

export default {
  prerender: ['/sitemap.xml'],
} satisfies Config


Sorting by non-string columns fails with ApplyGridDataProviderRequest

Issue
ApplyGridDataProviderRequest is a useful feature that allows applying grid parameters to a queryable source. However, it does not work when sorting by columns that are not of type string.
Example
When I try to sort by an int column, I get the following exception (shortened):
System.InvalidOperationException: 'The LINQ expression
...
.OrderBy(i => (IComparable)i.Age)' could not be translated.

The reason is that the sorting expression is stored as Func<T, IComparable>. When the property is string, the expression is simple, like p=>p.Name. But when the property is a value type like int, the expression is complex, like p=>Convert(p.Age, IComparable). EF cannot translate such an expression to a query.
Possible solution
I have a modified method that parses the expression and extracts the inner property expression. However, it uses reflection. The key part is
if (expression.Body is UnaryExpression unaryExpression)
{
  var propertyExpression = (MemberExpression)unaryExpression.Operand;
  var parameters = expression.Parameters;
  //Here we RELY that there is only ONE method with two parameters.

  MethodInfo? method = typeof(Queryable)
    .GetMethods()
    .Where(m => m.Name == methodName && m.GetParameters().Length == 2)
    .SingleOrDefault();
  MethodInfo? generic = (method ?? throw new AmbiguousMatchException($"More then one method {methodName} with two parameters exists in {nameof(Queryable)} class"))
              .MakeGenericMethod(typeof(T), propertyExpression.Type);
  Type delegateType = typeof(Func<,>).MakeGenericType(typeof(T), propertyExpression.Type);
  var lambda = Expression.Lambda(delegateType, propertyExpression, false, parameters);
  return (IOrderedQueryable<T>?)generic?.Invoke(null, new object[] { entities, lambda });
}

This is not very elegant but it works. I can create a pull request if you are interested.

@jirikanda Can you please take a look? Thanx

Maybe I missed something but If you want to pass information about sorting between layers, there is a SortString property for this use.
If you want to sort data in memory & automatically, you should use SortKeySelector.
See docs

SortKeySelector
Returns column sorting expression for automatic grid sorting. To be used for "strongly typed" setting of sorting, required for client-side sorting. Must be IComparable. Sorting of the column does not support multiple expressions. Create an artificial property and implement IComparable.

SortString
Returns column sorting as string. Use to set sorting as a string, ie. to get value to pass to backend. Ignored for client-side sorting.

Using filters & SortString in our backend code can look this way (SortString is Sorting in the code below):
return someQueryableDataSource
	.WhereIf(filter.CompanyId != null, o => o.CompanyId == filter.Compan.Id)
	...
	.OrderByMultiple(Sorting, sortingExpression => sortingExpression switch
	{
		nameof(OrderDto.CustomerReferenceNumber) => new() { o => o.CustomerReferenceNumber },
		nameof(OrderDto.DeliveryPlace) => new() { o => o.CompanyBranch.Street, o => o.CompanyBranch.Zip, o => o.CompanyBranch.City },
		nameof(OrderDto.Id) => new() { o => o.Id },
		nameof(OrderDto.OrderedBy) => new() { o => o.Contact.Surname, o => o.Contact.FirstName },
		nameof(OrderDto.OrderedDate) => new() { o => o.OrderDate },
		...
		_ => throw new InvalidOperationException($"Unknown SortingItem.Expression {sortingExpression}.")
	})
	.Select(o => new OrderDto
	{
		Id = o.Id,
		...
	});

I wanted to keep the issue report short, so I might have left out some key information. Here is a more detailed explanation:
I have a grid with several columns, including these two:
<HxGridColumn ItemTextSelector="@(item => item.ProfileName)" SortKeySelector="@(item => item.ProfileName)" />
<HxGridColumn ItemTextSelector="@(item => item.Modified.ToString())" SortKeySelector="@(item => item.Modified)" />

The type of ProfileName is string and the type of Modified is DateTime.
The data for the grid are created using the queryable approach described in  docs. The difference is that the query is not a list converted to IQueryable, but an Entity Framework expression. This expression has to be translated to SQL, and that's where the problem occurs.
In my ApplyToAsync method (created by #66), I get the sort key selector from request.Sorting[0].SortKeySelector. For ProfileName, it is item=>item.ProfileName, as shown in this image:

For Modified, it is item=>Convert(item.Modified, IComparable), as shown in this image:
.
The second expression is not valid for EF and causes a LINQ exception.
I hope this makes the issue clearer. Please let me know if you need more clarification.

@TPIvan, I will look at your report later.
However, first impression: If there is a problem with Convert expression and no problem without Covnvert expression, why not try to remove the Convert expression? how?

@jirikanda You might be right and I would be glad if I could avoid the ugly reflection based code. The challenge is that it still has to be done inside ApplyGridDataProviderRequest.

Hi, I am experiencing exactly same issue.
Can you please provide example how to use RemoveConvert to get sorting by DateTime work?
Thank you!

Any update please?

I hope it is the same as #883 which was fixed.
(Feel free to disaggree!)

After a quick test with the prerelease version, it appears to work. However, a final, fully tested confirmation will take some more time.

@TPIvan Great, thanks for this preliminary confirmation.

I have tested the solution more thoroughly and confirm that it resolves the reported problem. I have removed my workaround and replaced it by calling the ApplyGridDataProviderRequest method.


Bug when saving a shape after showing a shape layer

Bug found by @timtreis, reproduce using Visium data.
If you open the shapes in the hires coordinate system and then add a shapes layer and save the annotation, the shape will be saved in the wrong coordinate system.

Fixed in https://github.com/scverse/napari-spatialdata/commit/c3cd50adecd6d71911c0439c0bac02463996797c and https://github.com/scverse/napari-spatialdata/commit/4385a20a1439358d016d76a9a38c210066f0826e.


Enabled handler class extension

This pr enables extension of existing handler classes. This is useful since sometimes you want to add a feature to an existing handler without having to reimplement the entire class. In my case i wanted to add userid based command access control, since all handlers are executed concurrently, there is no way to stop the execution flow before the method gets executed (unless there is some trick with groups). So instead of implementing my logic in every method or rewriting the entire command handler, i've extended the default one.
Example code:
annotation AdminCommand; end

class AdminCommandHandler < Tourmaline::CommandHandler
  ANNOTATION = AdminCommand

  def initialize(@admins : Set(Int64), *splat, **dsplat, &block : Context ->)
    super *splat, **dsplat, &block
  end

  def call(client : Tourmaline::Client, update : Tourmaline::Update)
    if message = update.message
      if issuer = message.from
        if @admins.includes? issuer.id
          return_value = super
          Log.info {
            "Executed admin commands: #{commands}"
          }
          return return_value
        else
          Log.debug {
            "Unathorized use by #{issuer.id}"
          }
        end
      end
      return true
    end
  end
end

Thanks for this!