Add |  Max button on inputs

Description
This PR adds a "Max" button to the BigNumberInput component used in the WrapToken, UnwrapToken, and Send Assets modals. The button appears only when the maxValue prop is passed to the component. When clicked, the input value is set to the user's balance or the provided maximum value.
Notes

Added a maxValue prop to the BigNumberInput component to control the display of the "Max" button and set the maximum value for the input.
Updated the WrapToken, UnwrapToken, and Send Assets modals to pass the user's balance or the maximum value to the BigNumberInput component.

Issue / Notion doc (if applicable)
Fixes #1122
Testing

Navigate to the WrapToken, UnwrapToken, and Send Assets modals.
Ensure the "Max" button appears when the maxValue prop is passed to the BigNumberInput component.
Click the "Max" button and verify that the input value is set to the user's balance or the provided maximum value.

Screenshots (if applicable)

Nice, code looks good and works great.
One thing, could you reset the value to 0 when you change assets in the Send Assets modal?

We probably should have been doing that before the max button actually, but it's more apparent and confusing now


Minor errors running latest S5

Since merging the latest edits and re-running S5, I have noticed several things that I want to flag for others if that can help:

my S4 outputs obtained with the previous version of the code were not compatible with the new S5 read-in (there was stuff missing in the S4 meta file). I had to re-run S4, then S5 worked fine -> just be aware that S5 is not reverse-compatible with previously-generated S4 outputs
the template S5 ecf file for parameters would require the time_offset  parameter to be specified (even if it is zero), otherwise I get an error.
For me, it was only a matter of adding one line to the params ecf file. It is a very minor edit which I will include in my next pull request if needed, but it might help others if you run into this error! Another option would be to set it to zero by default if it is not specified in the params file.

another minor issue (now fixed in my code, will be fixed in my next pull request): the plots for the results from the light-curve fitting did not have the specified time offset correctly printing in the xlabel.

Yeah, unfortunately I had to make some edits to the inputs and outputs of some of the stages that broke backwards compatibility. I tried to get those merged ASAP so that it causes as little trouble for people with the data challenge, and hopefully the I/O from each stage will be more stable (although it will be more variable near the edges, aka S1-S2 and S5-S6).
Indeed, I forgot to include that in the S5 template and it is fixed in my currently under review PR #122
Hah, indeed - I hadn't even noticed that one! Good catch!


FlaUI.UIA3 doesn't work with DataGridView in windows 10

Describe the bug
Hi all,
I cannot work with DataGridView in windows 10. Everything works fine in windows 11.
Here is what happened: I got the Window by AutomationId, then I got the DataGridView inside that Window with FindFirstDescendant with AutomationId of the datagridview I wanted to interact to. But it seems like something's wrong with the DataGridView. If we interact with the DataGridView, it cannot get the rows inside. If you keep working with the Window (without touching the datagridview), then you retrieve the datagridview, it loaded the rows of previous render of that Window (not current state).
Code snippets
// Method to get the DataGridView of the table
private DataGridView GetTable(Window screen)
{
    return screen.FindFirstDescendant("m_dgSuchresultate").AsDataGridView();
}

registerForm = (RegisterabgleichScreen)service.GetForm(StarScreen.RegisterabgleichForm); // get the window (form)
var table = GetTable(registerForm.Screen); // (Screen is the Window)
var valueOfCell10 = table.Rows[0].Cells[10].Value; // Interact with the DataGridView

Screenshots
Only code is enough for this problem
I want to remind you window 11 works perfectly with the code above. But it's not right for window 10. I don't know what's going on here. I'm stuck with this for 2 days.
Nuget package: FlaUI.UIA3, version 4.0.0

Hi @anhphuong22tuoi can you provide the error that you see? Could you also attach any screenshot showing FlaUI Inspect, capturing the problematic object?
I'm using several DataGridViews in both Win10 and Win11 without any issue. I use them like this:
DataGridView dataGridView = screen.FindFirstDescendant("blahblah").AsDataGridView();
var oneTextFromDGV = dataGridView.Rows[4].Cells[4].AsTextBox().Text;


[Bug] Markdown links display incorrectly when used with backticks

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: v1.60.0-insider
OS Version: Windows 10

Steps to Reproduce:

Paste the following code into a markdown file:

[`code`](https://example.com)

Use the markdown.showPreview command.

Displays as code, can't tell it's a link:

@medmor Thanks, I've opened an issue there https://github.com/markdown-it/markdown-it/issues/814.

@medmor Quote from a member of markdown-it:

That's not this package's problem. This package generates html only. You can style it as you wish with css.

@medmor Quote from a member of markdown-it from here:

That's not this package's problem. This package generates html only. You can style it as you wish with css.

Hey, I am new to open source, can I contribute to this?

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: v1.60.0-insider
OS Version: Windows 10

Steps to Reproduce:

Paste the following code into a markdown file:

[`code`](https://example.com)

Use the markdown.showPreview command.
Displays as code, can't tell it's a link:

Even here it is clickable, but can't figure out by looking at it if its a link

In GitHub markdown, this is how it shows up
code

Which CSS file has to be changed to fix it ?

Color is set to blue here, why is it appearing as golden
https://github.com/microsoft/vscode/blob/8dbad73de575e47e127c8b1a4324f2b61257fbdc/extensions/markdown-language-features/media/highlight.css#L16-L19

Section is given golden color , when section is put inside link, it is overriding the blue color , is this the issue ?
https://github.com/microsoft/vscode/blob/8dbad73de575e47e127c8b1a4324f2b61257fbdc/extensions/markdown-language-features/media/highlight.css#L76-L78

How to run the app after cloning?

I think the Section is given golden color, when the section is put inside a link, it is overriding the blue color, is this the issue?
https://github.com/microsoft/vscode/blob/8dbad73de575e47e127c8b1a4324f2b61257fbdc/extensions/markdown-language-features/media/highlight.css#L76-L78

@san-coding Thanks for contributing, only restoring the blue color won't be enough, since we won't be able to tell it's code, a better solution is to simulate the GitHub markdown display: Linked Code Plain Code.

@wenfangdu What should I click on to run the app after cloning this repo?

Please see Build and Run.

Thanks for looking into this. While seeing how to provide advice on fixing this, I realized it was a simple change so I pushed a fix
Please test in 1.61 insiders and let me know if it doesn't work as expected

@mjbvz Thanks for the fix, however, only restoring the blue color won't be enough, since we won't be able to tell it's code:

A better solution is to simulate the GitHub markdown display: Linked Code Plain Code.

Code is rendered in a monospace font

@mjbvz The difference is quite subtle, would be nice to add a background to the code:


Verify static pages

What is good enough? Nick to take ownership of this.

To verify approval of pages it relates to what's good enough representation of the AEM components. Going through the pages:

Carousel component in top banners is stacking. This component is rarely used so we should replace this component with just an image in the AEM solution.
Carousel component on pages. Ideally this can be fixed by AEM dev team, as it impacts a few pages. ref https://energyvision.app.radix.equinor.com/sustainability/climate
Accordion component displays all content on the page without the drop down and display an odd piece of text. Should we replace this content with BASIC Accordion in AEM? REF: https://energyvision.app.radix.equinor.com/sustainability/environment
Anchor link/horizontal doesn't work. Ref: https://energyvision.app.radix.equinor.com/what-we-do/renewables
Quick link component doesn't work for external links?. Ref: https://energyvision.app.radix.equinor.com/what-we-do/equinor-ventures
Animation on column control and components not working: Ref: https://energyvision.app.radix.equinor.com/what-we-do/renewables. I have removed the functionality and published on this page in AEM.
Iframe component. Hard to verify as cookie policy shows and cannot be accepted.
Download button has option to display a symbol for type of file (ie adobe reader logo) This is missing from the button and instead displays a red box. Think we can live with this. Or remove the logo option from the button component option (not really required and barely used)

Issues 4,6 and 7 are to be addressed.
5 is not reproduceable.
1, 2 , 3 , 8 are fixed.

Sven is looking into as part of #501


uptake sbt-pekko-build

WIP towards centralising some build logic in https://github.com/pjfanning/sbt-pekko-build
early days

@mdedetrich is it possible to merge this as at least a short term fix for the mima-filters issue (found in #344). A better solution can be added later but right now the mima-filters logic doesn't work in pekko-http build - and copying the Mima.scala from incubator-pekko (where mima-filters work) via the sbt-pekko-build helps.


No binaries for arm (32-bit v7)

In order to support Raspberry Pi's, can you add a Linux build target of arm for go binary releases?

Are you actually using ship from a Raspberry Pi?

Yes. I teach Kubernetes in online courses and a small, but increasing group of them are using 32-bit and 64-bit Linux on arm/v7 and arm/v8.  I've also got a surprising number of Darwin arm64 users (Apple M1 arm/v8) starting to show up (so that's a 2nd build target request I guess).  I'll also be putting this request in kots :)

I can definitely support this for kots!


Identical Value Sets On Artifacts Page

Section Number Identify the most specific section number the issue occurs (e.g. 4.1.2)
Artifacts Summary
Issue Describe your issue. Don't write a book, but do include enough to indicate what you see as a problem.
https://profiles.ihe.net/ITI/mCSD/ValueSet/MCSDEndpointTypesCoreDocShareVS and https://profiles.ihe.net/ITI/mCSD/ValueSet/MCSDEndpointTypesDocShareVS appear identical on the artifacts page
Proposed Change Propose a resolution to your issue (e.g., suggested new wording or description of a way to address the issue). The committee might simply accept your suggested text. Even if they don't, it gives a good sense of what you are looking for. Leaving this blank means you can't imagine how to resolve the issue, which makes it easier for the committee to admit they can't imagine how to resolve it either and leave it unresolved.
Update titles and descriptions of these two value sets to differentiate them.
Priority:

Low: Typo or other minor classification that an editor can manage. Requires no group discussion.

Fix titles in FSH files for these ValueSets.


dpkg: error: cannot access archive 'packages-microsoft-prod.deb': No such file or directory

Problem encountered on https://dotnet.microsoft.com/download/linux-package-manager/ubuntu18-04/sdk-current
Operating System: Linux Mint 19.2 (Linux Ubuntu 18.04) - x64
wget -q https://packages.microsoft.com/config/ubuntu/18.04/packages-microsoft-prod.deb -O packages-microsoft-prod.deb
results in
packages-microsoft-prod.deb: No such file or directory
The suggested alternatives result in similar errors.
However, the package is downloadable from https://packages.microsoft.com/config/ubuntu/18.04 using eg Chromium.

Lunix Mint 19.2

Just to make sure, this is meant to be Linux Mint 19.2, right? (Just in case it's actually a new distro. 🙂)
I found https://stackoverflow.com/a/31706461, which suggests that error from wget may mean your directory isn't writable.
Something like: your browser is able to download it into your user downloads folder fine, but your terminal's current directory is somewhere you can't write to and you need to cd to a writable location.

You're correct - got confused by the err message. I'm closing this.

Yeah, it had me scratching my head for a while until I stumbled upon the right search terms to get that particular q&a. Thanks for confirming.


Scrollbar can not update UI when changing controller.offset from outside?

hi:
I tried using a scrollbar and update offset from programming, but the scrollbar can not update without setState.

Scrollbar(
 controller: mycontroller,
 child: ...
)

 //Timer: each 1s
   mycontroller.jumpTo(x  + 10);
   x += 10;

Hi @imiskolee, thanks for filing the issue, please provide a complete minimal reproducible code sample as well as the output of flutter doctor -v.
Thank you


Private repos shows up duplicated

Public repos shows up fine, only one per repo. However, the private repos are shown duplicates. See the below git-video. I have run gh-update, but it didn't help.
When I search on "ball" it shows duplicates (private repos), and searching for "python" it shows not duplicates (public records).

Done in v1.1.0

Great, confirmed to be working!


Provide API to know if we missed backingTrackStart

So that we can start the video for people who come in late.

I believe this is no longer an issue


Rework CoreData with custom fetchRequests

[x] Sortdescriptions in Fetchjrequests to sort after Date
[x] add lastEdited & createdAt + isWrittenNote
[x] Override awakeFromInsert for setting date
[x] For migration set lastedited & created at to today
[ ] Option to sort gradesy after their type,. halfyear, name, creationdate, lastediteddate, is written ectc
[ ] change icon fro written notes if active
[ ] option to soert subjects

update lastEdited every edit

[x] edit subejct
[x] editGrade
[x] display subject
[x] display grade

Maybe Stuff

[ ] Predicates for halfyears/isWrittenNote
[ ] maybe add createDefault for Items
[ ] switch to direct type in Relationships instead of NSSet

mapping model test created at wird nicht zum testdatujm gesetzt qwq


Properly reference external python objects

Fixes #20
Code Review

[ ] Is the code of an acceptable quality?
[ ] Does the code conform to the [coding standards]?
[ ] Are the unit tests small and test the class in isolation?
[ ] If there are changes in the release notes then do they describe the changes appropriately?

Functional Tests

[ ] Do changes function as described? Add comments below that describe the tests performed?
[ ] Do the changes handle unexpected situations, e.g. bad input?
[ ] Has the relevant (user and developer) documentation been added/updated?

Does everything look good? Mark the review as Approve. A member of @qef/gatekeepers will take care of it.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   95.83%   95.83%
=======================================
  Files           4        4
  Lines          96       96
=======================================
  Hits           92       92
  Misses          4        4

Impacted Files
Coverage Δ

qef/models/deltadirac.py
100% <ø> (ø)
:arrow_up:

qef/models/strexpft.py
93.33% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

:fire:


Show the snack bar to show a session is bookmarked on TimetableScreen

Issue

none

Overview (Required)

Show the snack bar to show a session is bookmarked on TimetableScreen

Links

Screenshot (Optional if screenshot test is present or unrelated to UI)

Before
After

Screen_recording_20230823_172827.webm

She said we are going to have an animation for this and when we bookmark continually, it might be a little noisy.
So let me close this! Thanks for your suggestion! 🙏


WIP: improve Unit_cell_symmetry class

Start over and try to simplify the symmetry class.

Hi @simonpintarelli ! can you have a quick look?


Update documentation and dependency build process

Update grpc cmake command to install absl, protobuf and ssl package as part of grpc build instead of relying on available packages since we strictly require minimum protobuf version 3.18.1. Fix documentation with missing steps
Signed-off-by: Nupur Uttarwar<EMAIL_ADDRESS>

PR is signed


Bumped to 1.16.4, No code changes added some stuff to .gitignore

I added some stuff to the .gitignore mostly because however my dev environment is set up it was trying to upload a bunch of Gradle cache stuff.
Signed-off-by: Gannon Burks<EMAIL_ADDRESS>

No problem! I mostly just did this because I was playing with a modpack and really wanted to use this mod in 1.16.4 lol.

No problem! I mostly just did this because I was playing with a modpack and really wanted to use this mod in 1.16.4 lol.

Does it not work under 1.16.4? I'll release an update then.

Does it not work under 1.16.4? I'll release an update then.

idk I think I tried to use it on 1.16.4 awhile ago and it didn't work so I kind of assumed the current version that was out wouldn't work for it either. So there is a decent possibility that it does work and I was just being dumb.

idk I think I tried to use it on 1.16.4 awhile ago and it didn't work so I kind of assumed the current version that was out wouldn't work for it either. So there is a decent possibility that it does work and I was just being dumb.

Closing since a 1.16.4 version was now released.

Closing since a 1.16.4 version was now released.


Add map_values to Gradient

Add map_values method to Gradient
Proposed solution to https://github.com/Arcadia-Science/arcadia-pycolor/issues/41
Something that gave me pause: on one hand I want to "buy in" to the existing data structures and make the return type list[HexCode], but on the other hand I feel it's more practical to return the hex strings since I suspect any callers of this function are typically using it as a final step in their process with the library. I'm curious what you think @mezarque.

Before merging, I do wonder if we should get a sense of how often we want to cut new releases on GitHub and distributions to PyPI. @keithchev any thoughts? I imagine it would make Evan's life a little easier if this is included in the PyPI-distributed version, but we should be disciplined about what amount of change necessitates a minor or patch version bump.

Yeah, if we end up with like version 0.5.200 we're probably doing something wrong. And of course we want to be disciplined about releasing breaking changes. But I think we can err on the side of leniency for minor versions for now, at least for new features like this one that may unblock or facilitate a user's work.

Sounds good! Feel free to merge @ekiefl. @keithchev would you mind distributing to PyPI?

Merging now. Made a small change described here: https://github.com/Arcadia-Science/arcadia-pycolor/pull/43#discussion_r1681397776


youtube-dl: 2020.12.12 -> 2020.12.14

https://github.com/ytdl-org/youtube-dl/releases/tag/2020.12.14

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Can you also backport this time, please? :)

20.09: 61956d97af11501687b0b278a43711eb570b24ed


Use config files over API for regeneration

Allows for generation of jobs that don't exist yet so long as they exist in config files
Fixes indentation
Various syntax and output improvements to regenerators
Includes documentation updates for changed files

Closes: #1814
Signed-off-by: Morgan Davies<EMAIL_ADDRESS>

Tested in a custom job. Seems to work

Using HEAD
Using JDK8

This may require updates elsewhere that use this job (pr-tester?)

@andrew-m-leonard Tested on JDK11 and JDK14 seems ok too


Support OTel collector in flagd

In order to get data into OTel, we need to be able to configure flagd to communicate with a collector. This entails:

a new CLI arg for flagd, specifying the collector (taking the collector URI)

if CLI arg is set, configure OTLP in the OTel SDK (see example here)
deploy collector + configure it with Jaeger and confirm works as expected

documentation and tests for the above

I researched on this topic and now have a working solution inspired by OTEL demo [1]. With the knowledge, I have few questions to clarify so that I can finalize the first version

Jaeger is specialized in tracing and currently, we do not record any tracing
Our metrics are right now backed by Prometheus metrics [2], so if we need metrics at Collector (and then export to Prometheus), then we should also migrate the metric recorder to be backed by OTEL

A solution I see is to switch behavior based on the startup flag

If otel collector is setup, then expose metrics and traces to otel. Else, fallback to Prometheus metrics

And regarding traces, I will start with a simple trace which we can expand as we go.
Let me know your opinion @beeme1mr @thisthat @toddbaert  :)
[1] - https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/examples/demo
[2] - https://github.com/open-feature/flagd/blob/main/core/pkg/runtime/from_config.go#L85

I agree that at the moment, Jaeger is not necessary since we don't provide any trace.
On the metric point, we're using OpenTelemetry metrics with a Prometheus exporter. We could switch the exporter to use OTLP, but I suggest keeping the Prom exporter as the default one to be more flexible. The OpenTelemetry collector supports scraping Prometheus endpoints (see Prometheus-receiver).
Another approach would be to use the env var OTEL_METRIC_EXPORTER to decide which exporter to use (doc). Quickly looking at the go-sdk, this looks like it is not yet implemented. Hence, we would need to add that logic by ourselves.

Hey @Kavindu-Dodan, there's another issue open for adding distributed traces to flagd. You can see that here.
I agree with @thisthat that we should leave the Prometheus exporter as the default. We just need to figure out a way to allow users to change metric exporter. Perhaps we could do something similar to the source configuration. That would make it easy to use cli flags when the use case is simple and a configuration when it becomes more complex. The OTEL_METRIC_EXPORTER env var may also be a good idea but I think it should behave like other flagd env vars.

Thank you @thisthat & @beeme1mr for the feedback.
Can we have the following agreement then:

Prometheus exporter will be the default metrics exporter
Configurability to change the behavior -  TELEMETRY_EXPORTER or similar flag with an accepted value of otlp(handled manually) to switch to otel exporter
Traces - Otel will be the default trace provider - will set the ground work for #575 through this issue

PR is ready for review - https://github.com/open-feature/flagd/pull/586
I can start work on #575 if this is the next priority task :)


DoneJS 0.8.0 Release Checklist

[ ] - Do releases of all subprojects in pre-release
[ ] - Release of donejs-cli 0.8.0
[ ] - Merge minor into master
[ ] - Release donejs 0.8.0
[ ] - Verify the site is updated
[ ] - Forum post with release notes
[ ] - Update chat.donejs.com to use 0.8.0
[ ] - Update place-my-order.com to use 0.8.0

Features:

Sessions
can-zone
The Generator generator

We probably don't want to advertise the generator generator until the general donejs add functionality in 0.9

k


Question: How is edit & continue supposed to work?

Hi, I'm currently looking into Blazor, and in the NDC prototype video I saw that it should be possible to edit a component, save it, and at saving have Roslyn compile and update the assembly directly in the background and then a simple browser-reload should execute the changed component.
However, I don't seem to be able to replicate this workflow with my local installation.
I even tried to manually recompile the project, but the dev server does not seem to pick up the new assembly when I reload in the browser. So to see my changes I deed to stop and restart the dev server.
So my question is, what am I missing here?
Additional info: I installed 15.7.0 Preview 1 of VS and installed the Blazor tooling vsix compiled from the develop branch into that. So all I did is based on the VS preview build.

I think that live reloading is not working at the moment - Issue #5, but it looks like it should be resolved soon in PR #193

That's correct - live reloading just isn't implemented yet. It won't be included in the upcoming 0.1.0 release, but we are regarding it as a priority to add as soon as we can after that.

I even tried to manually recompile the project, but the dev server does not seem to pick up the new assembly when I reload in the browser.

It should certainly update after you have recompiled. Maybe you have a browser caching issue. Could you try opening your browser's dev tools and checking the "disable cache" button, and then reloading while you have cache disabled?
Firefox:

Chrome:

Edge:

Ah, that helped indeed. So while the dev server is running, I now can edit the components, save & recompile, and reload the browser to see the changes. Great.
But yes, a real live-reload like demoed on the prototype would be super powerful.


Compatibility with MongoDB ODM

Closes #3.
This PR removes all hardcoding of EntityManager in favor of the ObjectManager interface from Doctrine Common, ensuring compatibility with multiple mappers. This allows for keeping most of the logic within one event subscriber instead of having to duplicate it. The event subscriber can be attached to either ORM or ODM and can handle being attached to both at the same time.
There are a few of things I still want to work on:

[x] The indexer at the moment contains a state, meaning it needs to store the object manager in order to later index the object in questions, I'd like to remove that (skipped)
[x] There is still some type checking in the subscriber to ensure compatibility to both ORM and ODM. I'd prefer this logic being in an event adapter, or even better, the events being compatible across ORM and ODM
[x] Currently, tests only work with ORM. I'd like to extend them to ODM so the travis job can test indexing against both an SQL database as well as MongoDB.

Note: since this is a BC break which requires a major version change, this would go great in tandem with #53.

@alcaeus, Sorry for the late answer. That's looks really good.
You can for sure move forward with the two last points. Ping us when the first point (The indexer at the moment contains a state, meaning it needs to store the object manager in order to later index the object in questions, I'd like to remove that) is done and we'll start to review it.
Agree with you that merging #53 would be a good timing.

@maxiloc Unfortunately, extracting the ObjectManager from the Indexer is not as easy as I'd hoped. I'm not sure how well this will work when two object managers are in play (since both ORM and ODM can be used at the same time); I'll have to run some more tests with this.

@maxiloc Done. Tests fail on travis due to the missing secrets but run locally:
$ vendor/bin/phpunit
PHPUnit 4.8.27 by Sebastian Bergmann and contributors.

..........S.....S...............30 entities indexed
.30 entities indexed
.30 entities indexed
............................... 65 / 78 ( 83%)
......30 entities indexed
.30 entities indexed
.30 entities indexed
....

Time: 3.3 minutes, Memory: 40.50MB

OK, but incomplete, skipped, or risky tests!
Tests: 77, Assertions: 173, Skipped: 2.

@alcaeus Thanks, we'll have a look asap

@maxiloc Is there any chance to have this merged anytime soon?

@alcaeus we have plans to support ODM in a future release. We might want to merge this in the meantime given it has been around for quite some time.
@julienbourdeau let us know when you have a moment to take a look.
Thank you all for your patience.

@rayrutjes Thanks for the updated. Rebased commits to fix conflicts.

Would you mind rebasing one last time and I can merge this one otherwise I'll merge the odm branch.
Thanks again!

@alcaeus When you rebase, can you remove the merge commit I just pushed. The resolve conflicts feature didn't work as I expected -_- Sorry about that!

@julienbourdeau feel free to rebase on your side, you don't need to bother @alcaeus :)
Let me know if I can help.

@rayrutjes @julienbourdeau No worries - rebased, resolved conflicts and removed the merge commit. 👍

🎉 Thanks Andreas!


Broken (404) link

https://docs.aws.amazon.com/solutions/latest/video-on-demand-on-aws-foundation/welcome.html

Hi ArmandoRiveroPi,
Thanks for posting. We will update this in our next release.
https://docs.aws.amazon.com/solutions/latest/video-on-demand-on-aws-foundations/welcome.html


rpi v2.0.2 custom image boots in emergency shell, immucore can't find cos_state

Kairos version:

kairos-opensuse-leap-arm-rpi:v2.0.2-k3sv1.26.3-k3s1
CPU architecture, OS, and Version:

Describe the bug

immucore can't find cos_state, see attached log
To Reproduce

Dockerfile:
FROM quay.io/kairos/kairos-opensuse-leap-arm-rpi:v2.0.2-k3sv1.26.3-k3s1
WORKDIR /usr/sbin
RUN curl -L https://github.com/Ognian/sdmon/releases/download/latest/sdmon-arm64.tar.gz | tar zxf -

custom build .sh file
mkdir -p build
docker build -t kairos-rpi-ogi .
docker pull quay.io/kairos/osbuilder-tools:latest
docker run -v $PWD:/HERE -v /var/run/docker.sock:/var/run/docker.sock --privileged -i --rm --entrypoint=/build-arm-image.sh quay.io/kairos/osbuilder-tools:latest \
 --model rpi64 \
 --state-partition-size 6200 \
 --recovery-partition-size 4200 \
 --size 15200 \
 --images-size 2000 \
 --local \
 --config /HERE/cloud-config.yaml \
 --efi-dir /HERE/boot \
 --docker-image kairos-rpi-ogi /HERE/build/out.img

Expected behavior

Logs

rdsosreport.txt
Additional context

Looks like immucore is not able to infer the FS of the cos-state partition.... immucore v0.0.18 has a patch to avoid this

Fix released as part of v2.0.3!
Images will be ready shortly: https://github.com/kairos-io/provider-kairos/releases/tag/v2.0.3
@Ognian let us know if this still fails (it should not!)

@Itxaka unfortunately it still fails :-(
rdsosreport.txt

umm, weird, empty label for cos_state and empty fs type??
[75.102240] localhost immucore[621]:<nil>ERR error="no such device" options=["ro"] type=  what=/dev/disk/by-label where=/sysroot/run/initramfs/cos-state

In my setup the rpi is booting from an USB HD; with previous versions this worked flawless ...

interesting. How are you setting up the usb boot? Pivoting from sd to usb or just having an usb disk with kairos installed?
I might be able to reproduce it if I know how its setup @Ognian

Built an image with the steps given, it boots on sdcard but fails on usb-hdd
Need to dust my serial to usb to track this :D

looks like a race condition becuase the usb takes a bit to come up:
[   24.530190] immucore[282]: <nil> INF work/pkg/mount/dag_steps.go:373 > Setting sentinel file to=active_mode
[   24.570207] immucore[282]: <nil> ERR work/internal/utils/mounts.go:69 > blkid error="exit status 2"
[   24.610185] immucore[282]: <nil> DBG work/internal/utils/mounts.go:72 > Partition FS type type= what=/dev/disk/by-label
[   24.650184] immucore[282]: <nil> ERR work/internal/utils/mounts.go:69 > blkid error="exit status 2"
[   24.690182] immucore[282]: <nil> DBG work/internal/utils/mounts.go:72 > Partition FS type type= what=tmpfs
[   24.730199] immucore[282]: <nil> DBG work/internal/utils/mounts.go:272 > vgchange out=
[   24.780430] immucore[282]: <nil> DBG work/internal/utils/mounts.go:170 > fsck command cmd="fsck /dev/disk/by-label -a"
[   24.820235] immucore[282]: <nil> INF work/pkg/mount/state.go:177 > mount done options=["rw"] type=tmpfs what=tmpfs where=/tmp
[   24.860201] immucore[282]: <nil> DBG work/internal/utils/mounts.go:173 > fsck error="exit status 8" out="fsck from util-linux 2.37.2\nfsck.ext2: No such file or directory while trying tol
[   24.910195] immucore[282]: <nil> ERR work/pkg/mount/state.go:174 > error="no such device" options=["ro"] type= what=/dev/disk/by-label where=/sysroot/run/initramfs/cos-state
[   24.960213] immucore[282]: <nil> ERR work/internal/utils/mounts.go:69 > blkid error="exit status 2"
[   25.000185] immucore[282]: <nil> DBG work/internal/utils/mounts.go:72 > Partition FS type type= what=/dev/disk/by-label
[   25.047746][   T75] scsi 0:0:0:0: Direct-Access     SanDisk  Ultra USB 3.0    1.00 PQ: 0 ANSI: 6
[   25.057702][   T75] scsi 0:0:0:0: Attached scsi generic sg0 type 0
[   25.079257][   T75] sd 0:0:0:0: [sda] 240254976 512-byte logical blocks: (123 GB/115 GiB)
[   25.088880][   T75] sd 0:0:0:0: [sda] Write Protect is off
[   25.095105][   T75] sd 0:0:0:0: [sda] Write cache: disabled, read cache: enabled, doesn't support DPO or FUA
[   25.191180][   T75]  sda: sda1 sda2 sda3 sda4
[   25.199294][   T75] sd 0:0:0:0: [sda] Attached SCSI removable disk

Gonna try to introduce some safe checks arounds the cos_state partition and a retry function so it tries to get it for a bit.

This will be fixed byt the next version of immucore with this patch: https://github.com/kairos-io/immucore/pull/115
Tested myself and indeed its due to the usb appearing later so immucore could not find the state label. That patch introduces a retry in order to get label or panic if it fails to.

@Itxaka thanks for keeping us up to date.

I'll close this since it's already fixed and will be included in the next release. @Itxaka @Ognian feel free to re-open if this is not working or I missed something. Thanks.

@jimmykarily is there a way to install the newest immucore in my custom Dockerfile (see my first entry) so that I can test this before v2.1.0 ( I don't know how long v2.1.0 would take but it looks like there are still a lot of open tasks...) ...

@Ognian we bumped immucore on latest kairos@master. You just need to build a kairos core image from latest master. latest from here should do: https://quay.io/repository/kairos/core-opensuse-leap-arm-rpi?tab=tags

@jimmykarily cool, I started my build now, lets see... Is there also a latest-k3sv1.26.3-k3s1 build also? If not could you initiated it?

Same error, cos-state label nor fs could be read:
   27.403231] localhost immucore[620]: <nil> ERR error="no such device" options=["ro"] type= what=/dev/disk/by-label where=/sysroot/run/initramfs/cos-state

Although Im a bit worried due to not getting the proper immucore version:
[   25.958752] localhost immucore[620]: <nil> INF Immucore commit=none compiled with=go1.20.2 version=v0.0.1

So I annot pinpoint the error exactly but Im not sure that is the proper immucore version with the fix. The new one has a panic if we cant get the label: https://github.com/kairos-io/immucore/pull/115/files#diff-310bfebac0279274ed530947884107087ff0aea1845732db964697735a848212R185 and in this case it looks like it cant get the label. So I would have expected it to panic and stop booting.
@Ognian can you provide the same logs with rd.immucore.debug in the grub command line to enable debug mode? I would like to see whats going on under there, thanks!

@Itxaka here it is
new.txt
In the meanwhile I'll try to manually pull the latest image before building...

@Itxaka pulled the newest manually, but the result is the same.... version is also v0.0.1
hope the previous with debug info helps...
1.txt

Definitively, that version does not contain the fix for the usb-hdd stuff as it only tries to get it once and then continues:
[   25.920990] localhost immucore[621]:  <nil>  DBG  luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:168 Get state label  what=/dev/disk/by-label
[   25.920990] localhost immucore[621]:  <nil>  DBG  luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:168 Get state label  what=/dev/disk/by-label

instead you should see something like this:
:/ # cat /proc/cmdline
root=LABEL=COS_ACTIVE cos-img/filename=test.img rd.immcure.debug
:/ # immucore
<nil> INF Immucore commit=none compiled with=go1.20.2 version=v0.0.1
<nil> INF Booting on active/passive/recovery.
^C
:/ # immucore
<nil> INF luetbuild/go/src/github.com/kairos-io/immucore/main.go:29 > Immucore commit=none compiled with=go1.20.2 version=v0.0.1
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/main.go:32 > cmdline content="root=LABEL=COS_ACTIVE cos-img/filename=test.img rd.immucore.debug\n\n"
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:130 > Target device what=test.img
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:131 > Target label what=/dev/disk/by-label/COS_ACTIVE
<nil> INF luetbuild/go/src/github.com/kairos-io/immucore/main.go:56 > Booting on active/passive/recovery.
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:181 > Cannot get state label, retrying try=0
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:181 > Cannot get state label, retrying try=1
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:181 > Cannot get state label, retrying try=2
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:181 > Cannot get state label, retrying try=3
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:181 > Cannot get state label, retrying try=4
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:181 > Cannot get state label, retrying try=5
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:181 > Cannot get state label, retrying try=6
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:181 > Cannot get state label, retrying try=7
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:181 > Cannot get state label, retrying try=8
<nil> DBG luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:181 > Cannot get state label, retrying try=9
<nil> PNC luetbuild/go/src/github.com/kairos-io/immucore/internal/utils/common.go:185 > Could not get state label error="All attempts fail:\n#1: could not get label\n#2: could not get label\n#3: could not get label\n#4: could not get label\n#5: could not get label\n#6: could not get label\n#7: could not get label\n#8: could not get label\n#9: could not get label\n#10: could not get label"
panic: Could not get state label

Ummm, maybe 2.0.3 did not get the bumped repositories for the newest immucore, let me have a look, because with the latest image from quay.io/kairos/kairos-opensuse-leap-arm-rpi:latest I get the same behaviour.

omg, Im an idiot. fix was released on v0.0.19. No wonder its not fixed, kairos v2.0.3 contains immucore v0.0.18 which I thougth was the one to fix this.
Sorry about that @Ognian I totally screwed from the start thinking it was already in for v2.0.3 :sob:
Anyway, then the only thing you can do for now is wait for 2.1.0 to be released or build the image yourself from the provider-kairos repo with something like:
earthly +all_arm --CORE_VERSION=latest --MODEL=rpi64 --K3S_VERSION=v1.26.3-k3s

That should build a opensuse-leap based image with k3s v1.26.3-k3s for the rpi based on the latest images published (master).

@Itxaka tested with quay.io/kairos/core-opensuse-leap-arm-rpi:v2.1.0 and at least boots now... I'm having now the problem that the config is not applied (I don't know why yet..) and of course Im waiting on the "standard" v2.1.0 version including k3s...
BUT at least it boots now.


Move the dist-arm-linux job to arm runner

In rust CI, the dist-arm-linux job uses an x64 runner to cross-compile to arm.
Switch that job to use the large arm runner, similar to https://github.com/rust-lang/rust/pull/133809 to improve performance.

The job is still being "cross-compiled" (using crosstool-ng), but it runs on an ARM runner now, thus saving CI cost. Implemented in https://github.com/rust-lang/rust/pull/133902.

Should we remove crosstool-ng?

Well, if it works, it seems more elegant than doing the whole CentOS 7 dance where we compile the world from scratch. So in fact I would like to try the opposite (https://github.com/rust-lang/rust/issues/134118) and replace CentOS with crosstool, if it is possible.

what about performance?
If we are already on arm, it seems a waste to cross compile to arm. Or am I missing something?
For the other runners, do you think it's worth to use arm to cross compile to x86 or there would be a performance penalty?

I haven't measured if the cross-compilation is slower or not, so hard to say 🤷♂ Same for the other question. I wouldn't overdo it though, just because the ARM runner are cheaper, it doesn't mean that we have to use them everywhere, crosscompilation in general is probably contains a can of worms, especially when it's actually being done across platforms.


appnexus bid adapter: flocid support

Small change to note the appnexus bidder supports the flocId userId.
Related to:
https://github.com/prebid/Prebid.js/pull/6801

can you document https://github.com/prebid/Prebid.js/pull/6802 on this pr as well?

updated.


knex object is not available on knexjs.org

Environment
This happens on knexjs.org
Bug
Users should be able to pop open the console and build queries using the knex object. However, as at today (28 Dec 2018), if you open the console, you will meet an error message:
Uncaught TypeError: expecting a function but got [object Undefined]

Support for WebSQL was dropped, actually. I guess we need to update documentation to remove all references to it.

Actually that kind of should work, since all the exam ples in documentation page are built with knex. So query building n browser should work. However there is some js errors on documentation page right now, which might cause this. There is an open issue about it in the documentation repo. Linkin this to the same issue and both can be fixed at the same time https://github.com/knex/documentation/issues/163


 This was done to prevent errors in third party libraries. Moveit for example expects a collision geometry in each joint that can be controlled. You can see this behaviour by running the panda_moveit_config demo.launch launch file. If the old description is used moveIt throws the following errors:
[ WARN] [1629295377.872753565]: Link panda_leftfinger has visual geometry but no collision geometry. Collision geometry will be left empty. Fix your URDF file by explicitly specifying collision geometry.
[ WARN] [1629295377.872794348]: Link panda_rightfinger has visual geometry but no collision geometry. Collision geometry will be left empty. Fix your URDF file by explicitly specifying collision geometry.

Old collision meshes
Safety distance 0.03
Open state:

Closed state:

Safety distance 0.0

New collision meshes
Safety distance 0.03

Safety distance 0.0

Closed as this is implemented in #188.


--load-image and page.settings.loadImages "blank creative loaded"

This is my command line script looks like:
phantomjs --ssl-protocol=tlsv1 --load-images=no --web-security=yes --ignore-ssl-errors=true --proxy myproxy.net:9050 --proxy-type=socks5 $@

Also, since I load config files in the javascript I do
page.settings.userAgent = self.getUserAgent();
page.settings.resourceTimeout = Config.resourceTimeout; // timeout
page.settings.loadImages = Config.loadImages; // load images flags

where the defaults are to false.
Now I see in the console callback:
page.onConsoleMessage = function(msg) {
 console.log(msg);
};

logs like
blank creative loaded:<PHONE_NUMBER>9 (1600 x 1000)

This is the page.settings dump
{
    "XSSAuditingEnabled": false,
    "javascriptCanCloseWindows": true,
    "javascriptCanOpenWindows": true,
    "javascriptEnabled": true,
    "loadImages": false,
    "localToRemoteUrlAccessEnabled": false,
    "userAgent": "Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/37.0.2049.0 Safari/537.36",
    "webSecurityEnabled": true,
    "resourceTimeout": 60000
}

Considering those messages logs, does this mean that phantomjs is loading images anyways or it is a blank 1600x1000 image placeholder?

I don't have enough information to tell you. I am not sure. Remember I wrote just a wrapper around phantomjs. I don't even know where blank creative loaded would be.


svg file work's not good

http://ol60v9vzz.bkt.clouddn.com/%E5%B1%8F%E5%B9%95%E5%BF%AB%E7%85%A7%202017-04-17%20%E4%B8%8B%E5%8D%883.43.34.png
svg file http://www.iconfont.cn/plus/collections/detail?cid=2844
svg file in chrome is ok,but in react-native work's not good
"react-native": "0.42.3",
"react-native-svg": "^4.6.1",

update ver fix


encrypt: dont upload and enumerate concurrently

nvm, still crashes with:
Sep 02 15:35:00 ubuntu perkeepd[24224]: 2022/09/02 15:35:00 Caught panic installer handlers: error instantiating storage for prefix "/b2-enc/", type "encrypt": error scanning metadata on start-up: error with meta blob sha224-3ff458dbc01f8f37d0d486a34536c2ae249b68a8c78b1866959e6741: fetch failed: unexpected EOF


[Props] Parent chain crash

Unfortunate thanks to #2248 for the idea.
  Leaving the chain frozen and allowing it to grow unbounded will just cause the server to close after it gets too long (around 128 children).
I propose we limit the length of such chains to something before the "problematic" range.  I cannot think of any legitimate use cases for a parent chain like this besides trying to crash servers.

parentTo is from propcore so it is disabled by default. I guess I can do something similar to what I'm trying to do with EGP and add OPS depending on how deeply parented a prop is.
Still dunno if it's our place to do this though


Exception blocks like wool and cobwebs not breakable

the wool and the cobwebs are not breakable in the wool room regions.

Please provide what map this is on.

it was on deepwind jungle,
filter name="allow-some" parents="deny-all">
allow>
block>glass
block>fence
block damage="9">wool
block damage="13">wool
block damage="10">wool
block damage="4">wool
/allow>
/filter>

Yes, this also happened to me recently. I was going to post it here when I had time, but now kroest did it.

Es el XML el que esta mal, no el plugins. (The xml is bad, no the plugin)

i use the exact same xml from the maps.oc.tc, http://pastebin.com/xtvLQbXU

I have noticed this on almost the whole of Twisted. It's to do with the void.

The void is defined by the xml, no by the plugin :v

The reason the filter is not working is because referencing a filter inside a filter is not working

Right now a quick fix for server owners would be to convert from the legacy filters to the standard ones, to do this do the following:

Rename the <allow> tag to <any>
Remove the parents="deny-all" attribute

That should fix the error on your side while we fix it for the plugin

yes, i use an older version of the xml, and its running fine now. the xml from maps.oc.tc was a bit broken i guess. thanks for your help.

This is still an issue we need to fix


Smarty and template support in HTML formatter

What does this implement/fix? Explain your changes.
Add smarty and tpl file extension support to the default HTML format engine.
Does this close any currently open issues?
#940
Any other comments?
Sorry I have not read into your build process to actually test this, but it looks pretty clean and simple to me.
Checklist
Check all those that are applicable and complete.

[ ] Merged with latest master branch
[ ] Added examples for testing to examples/ directory
[ ] Travis CI passes (Mac support)
[ ] AppVeyor passes (Windows support)

Add smarty and tpl file extensions to default HTML format engine.

I am not entirely sure how your grammar system and matching work, but i have updated the PR to include a new coffee file that seems to mirror the functionality of the twig template handler, and reverted the default html grammar.  Thanks, and let me know if that works for you.

Good work so far @nickhagen.
I'd recommend that you take a look at https://github.com/Glavin001/atom-beautify/blob/master/docs/add-languages-and-beautifiers.md#how-to-add-a-language

Also be sure to add examples that can be tested to https://github.com/Glavin001/atom-beautify/tree/master/examples/nested-jsbeautifyrc

And be sure to update the documentation so that other users can see Smartly is supported!
See https://github.com/Glavin001/atom-beautify/blob/master/docs/add-languages-and-beautifiers.md#options-documentation for details.
# Update src/options.json file to have new/changed options
npm run postinstall
# Update documentation to include information about those options
npm run docs

I do not see any changes since the last review so I am temporarily removing myself as assignee and going to review other Pull Requests. Let me know when you're ready for re-review! Thanks!

Closing since there were no changes made after PR review.


Add -Wconversion -Wpointer-arith options to linux builds

The following essential two flags are not turned on by -Wall:
-Wconversion Warn for implicit conversions that may alter a value.
-Wpointer-arith Warn about anything that depends on the “size of” a function type or of void.
See https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html

These were merged into master, but they enabled the flags for both Linux and Windows, and CL.exe doesn't know what those flags are, so this was causing the Windows build to fail!  Theses changes got past our CI gating because we've had a Windows CI outage (without knowing about it) until today.
So the changes necessary in the source code to enable these flags are still in master, but the flags themselves were removed from master until we can re-enable them solely for Linux builds.  Refer to #1143 to see the changes that disabled these flags.

This was resolved in #1135.


The answer to question 76 is wrong.

The result of implementation is

On the other hand

@qiufeihong2018 ,
Please, visit chrome://flags/#enable-javascript-harmony and enable the flag before check latest features in your browser.

@oshliaer But there is no precondition for deleting name.So I still believe in undefined.

But there is no precondition for deleting name.So I still believe in undefined.

This is because your environment may be unclean. A variable named name can easily leak into the global context, because many variables with the same name exist. Replace the name of this variable with something more unique.
How about this?
const { qiufeihong2018: myName } = { qiufeihong2018 : "Lydia" };

console.log(qiufeihong2018);

You must also be sure that the variable name does not exist. If you started the deletion delete name and it returned true, it means that your environment already had a variable name. You cannot get ReferenceError error in this situation.

@oshliaer I'm understand!Thank you!

@qiufeihong2018 you can close the issue.

window.name is a useful but not recommended way to implement cross-domain messaging. It is always there. :)

But would not be more more correct to use variable which not crossing with browser web API (e.g. const { someName: myName } = { name: "Lydia" };) or add answer like (depends from execution enviroment)?


add GH action pipeline to run automated tests and automate publishing

This PR introduces the file ci.yaml which includes the GH action pipeline to run automated tests and automate publishing to PYPI. The publish a new release, we need to push a new tag with the version format as vX.X.X or vX.X.X.dev. Moreover, we can create a test release by introducing the word test to the tag name.
This PR adapts the pipeline from https://github.com/FlexMeasures/flexmeasures-client/tree/main.
Follow up tickets:

Add linting and apply it to the whole project
Check issues with Python v3.9

This looks really really good. Thanks for adding your workflow to this! Some general questions. Once addressed, I think we can definitely merge it.


Set proxy advertisedAddress with pulsar advertisedAddress

Motivation
Currently, the AoP proxy use advertisedAddress in config file broker.conf directly, use the Pulsar advertisedAddress instead of it.
We could don't care about the config advertisedAddress anymore when use AoP proxy. In the future, we could provide the zk address to AoP proxy to find available brokers. #128
Modifications
Make AoP proxy use Pulsar advertisedAddress to connect with Broker.
Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (yes)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (yes)

Documentation

Does this pull request introduce a new feature? (no)

@codelipenghui Ok, I'll improve the ProxyConfiguration.


child-src support was added in FF 45

taken from here: https://developer.mozilla.org/en-US/docs/Web/Security/CSP/CSP_policy_directives#Browser_compatibility

Thanks!


Implement Function Executor in a separate process

A new indexify-cli command called function_executor is added. It
reads function inputs from host file system, runs the function
and saves function outputs in host file system so they can be
read by main Executor process once the function_executor process exits.
The command imitates an RPC server with request-response pattern
but no RPC framework with a network connection is used yet.
We're going to use it later. The request-response pattern helps
in long term while we're adding more features into host
Executor <-> Function Executor protocol.
Function Executor only passes binary data in the protocol messages
this allows to decouple host Executor from Python SDK and gives
a possibility to rewrite it in another non-Python language in the
future.
The protocol messages are currently encoded using jsonpickle.
This is the right option in short term as it allows to serialize
the protocol objects without any additional workarounds. E.g.
standart Python json module doesn't allow that - it can't serialize
bytes or custom objects. Inheriting protocol messaged from BaseModel
generates familiar constructors and allows to use .field notation
with protocol messages. .field notation is not available when using
TypedDict. In medium term we're going to migrate to an RPC framework
so all these data wiring considerations won't be relevant anymore.
This PR can be reviewed per commit
Testing:
make test
make fmt
make check
Contribution Checklist

[X] If the python-sdk was changed, please run make fmt in python-sdk/.
[N/A] If the server was changed, please run make fmt in server/.
[x] Make sure all PR Checks are passing.

We discussed internally and it looks like we need to implement execution of multiple concurrent tasks in Function Executor. Otherwise it'd break some existing customers that use a shared in-process state to optimize loading of their models and etc.
I'll create a new PR with this functionality implemented + the functionality of this PR.


Problem using Bot.startconversation

convo.ask('What is your favorite team?', function(response, convo1) {
if(response.type=="slash_command"){
convo1.context.bot.replyAcknowledge()
convo1.context.bot.reply(response,"Slash command")

convo1.next();
}else{
convo1.next();
convo1.say('Cool, I like team' + response.text)
}
            })

convo.ask('What is your favorite team?', function(response1, convo1) {
if(response1.type=="slash_command"){
convo1.context.bot.replyAcknowledge()
convo1.context.bot.reply(response1,"Slash command")

convo1.next();
})
}else{
convo1.next();
convo1.say('Cool, I like team' + response.text)
}
            })

The Botkit conversation system doesn't work with Slash commands.  Slash commands should be "top level" commands rather than answers to prompts within a conversation.

The isuue is if there is  a ongoing conversation between bot and a user and user tries slash command instead of replying to question.Slash command message comes as response in conversation instead of going to slash command listener.That is why i am trying to handle it inside conversation.

hey @Vishvadeep93 you could try to apply middleware to exclude slash commands from conversations at all.
controller.middleware.receive.use(function (bot, message, next) {
	if (message.type === 'slash_command') {
		return bot.botkit.trigger(message.type, [bot, message]);
	}
	next();
});

Good suggestion, @ihorrusinko!  This is exactly what I'd recommend.


Documentation: Path to follow for Second Pull Request

For a Second Pull Request, I used a  Fork & Branch Workflow mentioned below:
Step 1: Fork the Repository
As you're making your 2nd PR, you already would have so you can skip this step.
For the sake of mentioning here is how you can do that:
Click on the Fork in the top right corner (below your profile icon) on https://github.com/twindle-co/twindle.
Step 2: Make a Local Clone

Open Git or any terminal you're using.
Type git clone <url of forked repo>
It will by default create Git remote as origin

Step 3: Adding a Remote
Syntax: git remote add <remote> <url of original repo>

git remote add upstream https://github.com/twindle-co/twindle.git

Note: Git Remote to pull changes from the original repo is upstream here. You can name it anything.
We will need it at a later point.
Step 4: Working In a Branch
1) Create and Checkout
Method 1

git branch feature
git checkout feature

OR
Method 2
(shorthand for Method 1)
Syntax: git checkout -b <branch-name>

git checkout -b feature

Executing either method creates & switches to the new branch i.e feature.
2) Make changes to the files via any editor of your choice
Any changes made will be in the feature branch not main.
3) Commit those changes

git add .
git commit -m "<message>"

Step 5: Pushing changes to GitHub
Syntax: git push <remote> <branch>

git push origin feature

Step 6: Opening a Pull Request

GitHub makes it incredibly easy.
Click on Create Pull Request button.
Wait for it to be merged.

Step 7: Cleaning Up After a Merged PR

git pull upstream main
Pull changes from original repo
git branch -d feature
Delete the feature branch locally
git push origin main
Push changes to forked repo > main branch
git push origin --delete feature
Delete the feature branch remotely

Step 8: Keeping your Forked Repo in Sync with the Original Repo
(You need to do this time-to-time to update your Forked Repo > main branch)

git pull upstream main
git push origin main

OR
you can also do this step as mentioned in #41

There's a type in step 7 last line
it should be git push origin --delete feature
Good Work 🎉

Thanks, fixed it! 👍

Great walkthrough @tusharkandpal :partying_face:

Thanks, @johnjacobkenny 😃

good

Awesome, good work!

Great work !

Great !

Awesome @tusharkandpal 👍

This was referenced in the FAQ document and updated in the PR #201  ✅


Support flatpak

Flatpak seems to be the best and most popular Linux package mechanism, with availability on flathub and such, package managers like those on KDE and thus the steam deck, support flatpak too.

Since i cannot find any solution with AppImage, i added flatpak as a build target, so maybe you can install it with flatpak.
This does not include publishing to Flathub, i may never publish it on a store.
I simply installed it with flatpak install MIDI-Jar-1.5.1-x86_64.flatpak
Hope this is working well for you too.

Yep this is awesome thank you! In kde\archlinux it's just a matter of clicking it and it opens up the Discover software center to install or remove it
It integrates better with the launchers too so thank you for doing this
Not having flathub is fine, I think it would probably help your visibility and people's ability to see it in these ecosystems...I do think that would be very valuable for you, as they should then be searchable inside it
Quite honestly, I've been praising this software in music communities. Your software is awesome, constantly getting great features, free and open source, it's much better than ChordieApp which is the "competitor" that's paid and Windows only (though does work in wine)
I use this app every day in my practice, the web server is really great too because I can either pull it on my desktop or my tablet depending on what I switch to during my practice routine
This has been huge for me to learn chords and theory and understand it all better
You're doing a great thing :) If there's a donation button somewhere I'd love to send some your way
I'm looking forward to the quiz component to getting improvements here too. There's a few things I'd like to see it do there and if you add those I could add those to my daily practice and see my progress accelerate even more again


Added initial HTML elements for project UI

Hey Team,
I've gathered (I believe) all the basic HTML elements we're planning to use. Before we move on to the next phase, I'd love to get your eyes on it. Take a quick peek and let me know:

[ ]  Does it cover all the bases?
[ ]  Are there any elements that need to be added, moved, deleted, or edited?

Cheers!
Kate :)

Great work Kate - happy to merge.
Looks good - think we have everything we need from a structure point of view now.

Might be good to look over the visuals today (on Zoom) and do a checklist of elements in the doc to make sure the flow of the site is how we all want it to look (top to bottom)?

Checked - no errors. Merge accepted.


When "ctrl+enter" or "shift+enter" it run the wrong cell

Environment data
VS Code version: 1.60
Jupyter Extension version (available under the Extensions sidebar): v2021.8.1054968649
Python Extension version (available under the Extensions sidebar): v2021.7.1060902895
OS (Windows | Mac | Linux distro) and version: Windows 10: 21H1
Python and/or Anaconda version: 3.8.10/3.9.6
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Jupyter server running: Local | Remote | N/A: Local and remote
Expected behaviour
Run the selected cell when "ctrl+enter" or "shift+enter".
Actual behaviour
Sometimes, when "ctrl+enter" or "shift+enter" to run a cell, it run the wrong cell.
It seems to me that it is running the last cell ran.
Steps to reproduce:
Hard to reproduce as it is happening randomly. I would suspect it to happen after a high memory consumption (reaching to max) that will make it bug until we restart the apk/computer.
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

Thanks for filing this issue.
Are you using a notebook or the Interactive Window?

Likely an interactive window problem. @Tirbo06, let us know what you know.

It happens in Jupyter Notebooks

I will try to edit the post with a gif when it will happen again

I've also been having this happen, and I can't figure out the pattern either. Sometimes it gets stuck so that no matter which cell I tell it to run, it just keeps running the same one. This stops when I switch to another tab and back.

I'm also having this issue.


Additional config for toolboxes

If it's easily possible, config options for the amount of scrap you get from disabling hazards, as well as the time it takes, would be nice

Not easily done as far as I know but I don't know much so maybe when I have time to spare. If you have some ideas to better balance those values so it's more enjoyable I'm all ears. The current values weren't tested much but I wanted to get the update out so I settled with them.

Makes sense, I think the current values could be a bit lower, I'd personally let the toolbox just turn traps into either 1 metal sheet or 1 big bolt, since it'd make it more of a long-term investment. Although I've only played with the toolbox a bit.

I've reduced some values in my latest unreleased version after having some free time to playtest so you may see that release sometime soon. As for the configs request, I will add more configs every once in a while and will take note that you've asked for these config options but I can't say when I'll get to adding them.

Of course, I appreciate your consideration regardless

being able to choose the deactivation time of the turrets, mine, see being able to deactivate the fact of being able to destroy the turrets because I find that strong personally


problem with metattack

Thanks for this wonderful repo. However, when I run the metattack example ,the result is not promising
Here is my result when attack Cora with metattack
Training...
100/100 [====================] - Total: 520.68ms - 5ms/step- loss: 0.0713 - acc: 0.996 - val_loss: 0.574 - val_acc: 0.847
Evaluating...
1/1 [====================] - Total: 2.01ms - 2ms/step- loss: 0.522 - acc: 0.847
Before attack
╒═════════╤═══════════╕
│ Names   │   Objects │
╞═════════╪═══════════╡
│ loss    │  0.521524 │
├─────────┼───────────┤
│ acc     │  0.846579 │
╘═════════╧═══════════╛
Peturbing graph...: 100%|██████████| 253/253 [01:00<00:00,  4.17it/s]Evaluating...
1/1 [====================] - Total: 2.08ms - 2ms/step- loss: 0.528 - acc: 0.844
After evasion attack
╒═════════╤═══════════╕
│ Names   │   Objects │
╞═════════╪═══════════╡
│ loss    │  0.528431 │
├─────────┼───────────┤
│ acc     │  0.844064 │
╘═════════╧═══════════╛
Training...
32/100 [=====>..............] - ETA: 0s- loss: 0.212 - acc: 0.956 - val_loss: 0.634 - val_acc: 0.807
100/100 [====================] - Total: 407.58ms - 4ms/step- loss: 0.0601 - acc: 0.996 - val_loss: 0.704 - val_acc: 0.787
Evaluating...
1/1 [====================] - Total: 1.66ms - 1ms/step- loss: 0.711 - acc: 0.819
After poisoning attack
╒═════════╤═══════════╕
│ Names   │   Objects │
╞═════════╪═══════════╡
│ loss    │  0.710625 │
├─────────┼───────────┤
│ acc     │  0.818913 │
╘═════════╧═══════════╛

Thank you for pointing that out. There are some reasons for the poor results:

Randomness. Current implementation is unstable and you need to run it multiple times with different random seeds to get a decent performance.
Validation with model picking. As mentioned here, you can disable the validation step to improve the poisoning attack performance:

# Disable the validation step and the model picking
trainer.fit(attacker.data(), mask=splits.train_nodes)


Shape names are empty when using relative path SVGs, which causes svgo to fail rendering optimised icons

Current behaviour
When using relative paths with .add, file paths are modified in such a way that Shapes fail to extract a name for each input SVG. Because shapes don't have names, they don't have an ID, and SVGO generates optimised SVGs into the wrong file path.
Expected behaviour
Shapes should have a valid name no matter if the input SVG's path is relative or absolute
PR #325 fixes this issue. Note that I have 32 tests passing and 2 failing on my PR and machine, but I also have 32 passing and 2 failing on HEAD on master.

@Sidnioulz Hello! Which tests are failed?

@Sidnioulz thank you for your participation :)


OLS-1001: Source image optimization for the service

Description
OLS-1001: Source image optimization for the service
Type of change

[ ] Refactor
[x] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

Related Tickets & Documents

Related Issue #
Closes # https://issues.redhat.com/browse/OLS-1001

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.86%. Comparing base (32b78e7) to head (b3890cb).
Report is 37 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1635       +/-   ##
===========================================
- Coverage   94.89%   73.86%   -21.03%
===========================================
  Files          60       58        -2
  Lines        2685     2610       -75
===========================================
- Hits         2548     1928      -620
- Misses        137      682      +545

Flag
Coverage Δ

73.86% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 32 files with indirect coverage changes

/retest

/retest

/hold


error bug bash: PolicySetError and LinkingError

Description of changes
Improve error messages and docstrings, and carry more information with some error variants
The change in this PR is (choose one, and delete the other options):

[x] A change "invisible" to users (e.g., documentation, changes to "internal" crates like cedar-policy-core, cedar-validator, etc.)

I confirm that this PR (choose one, and delete the other options):

[x] Does not update the CHANGELOG because my change does not significantly impact released code.

I confirm that cedar-spec (choose one, and delete the other options):

[x] Does not require updates because my change does not impact the Cedar Dafny model or DRT infrastructure.

Disclaimer
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

note: some changes in this PR are breaking and need to wait for Cedar 3.0.  Perhaps I should separate out some of the non-breaking changes into a different PR

I will update this PR to remove the breaking change, and put the breaking change in a different PR

Removed breaking changes from this PR; asking for re-review


Document initial Visual Representation of CI/CD Pipeline

[x] Meeting to discuss the high level workflow of CI/CD Pipeline
[ ] Identify stages
[ ] Identify triggers
[ ] Identify Testing Types
[ ] Document Exists

Assignee:

Srinivas

Met with Jason and Yefim and discussed the DRAFT version of the proposed CI/CD pipeline. Proposed version of the pipeline uses GitHub Tags to trigger various stages of the pipeline. However based on the feedback received using the TAGs may complicate the pipeline especially promoting the application across multiple environments.
I plan to update the pipeline to keep it simple and have Dev, Test, Stage and Prod as different stages of the pipeline and perform tests at different stages.
Will work with Yefim on integrating his Testing framework documentation with the pipeline diagram.

https://app.zenhub.com/files/287570343/0fe8222a-00ee-432e-8b7e-a2caa5f7e7db/download


feat(trouble): add open with trouble mapping telescope

Adds this by default , really handy when using trouble

That's already included. see line 243 of the same file


Add callbacks

I'd like to add callback functionality to iterative_unfold. These Callback objects would be called after each iteration of unfolding. A couple I can think of off the top of my head are a callback for regularization and a callback for saving iteration info to disk.
Something like
from pyunfold import iterative_unfold, HDFLogger
# Set up callback
hdflogger = HDFLogger('/path/to/outputfile.hdf')
# Pass callback to iterative_unfold
unfolded_result = iterative_unfold(*, callbacks=[hdflogger])

I'll assemble a list of callbacks to implement:

[ ]  regularization callback
[ ] HDFLogger for saving iteration info to disk
[ ] PlotTS callback for rendering a test statistic vs. iteration plot that is updated


[Typo] Dendrobate stat line typo

x% increased Damage with Poison if you have at least 300 Dexterity
y% increased Poison Duration at least 150 Intelligence

Second line should read: y% increased Poison Duration if you have at least 150 Intelligence
Nothing much else to say.. low priority fix

Fun fact: That's actually how that stat was originally written in-game... But it was fixed, I just didn't get around to changing it. I'll do that now, since it's easy.


Added specific API methods and typings

I added some typing for the component and the FullStory API. The API typings were based off of the fullstory documentation. I also added some additional methods for calling specific methods directly. So instead of calling FullStoryAPI('identify', 'blah', { foo: bar}) you can can call identify( 'blah', { foo: bar}).

Updated the documentation and moved the typing file! Had the types in the typings folder out of habit.

Ahh yes, of course, @shiya - my apologies.  I merged, but forgot to release a new version.  You can bump to 1.3.1 which will include the Typescript bindings.  😄


PR template

This PR adds a suggestion for a company-wide PR template that can be used in all those projects that don't have one.

LGTM, @inigomarquinez ! I like the format, seems very complete and useful. Let's see how the contributors use the template and let's keep the door open for iterations in the future 👍 .

Sure, @UlisesGascon ! Always open to feedback :)


FSDP EKS Example failing with: module 'torch.library' has no attribute 'register_fake'

Following instructions in HyperPod EKS workshop, trying to run FSDP EKS example on 2 p5 nodes is failing with the following error, pointing towards error in train.py:
The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/fsdp/train.py", line 281, in <module>
        main(args)if hasattr(module, attr):

  File "/fsdp/train.py", line 168, in main
  File "/usr/local/lib/python3.10/dist-packages/transformers/utils/import_utils.py", line 1766, in __getattr__
    model = AutoModelForCausalLM.from_config(model_config)
  File "/usr/local/lib/python3.10/dist-packages/transformers/models/auto/auto_factory.py", line 439, in from_config
    model_class = _get_model_class(config, cls._model_mapping)
  File "/usr/local/lib/python3.10/dist-packages/transformers/models/auto/auto_factory.py", line 388, in _get_model_class
    module = self._get_module(self._class_to_module[name])
  File "/usr/local/lib/python3.10/dist-packages/transformers/utils/import_utils.py", line 1780, in _get_module
    supported_models = model_mapping[type(config)]
  File "/usr/local/lib/python3.10/dist-packages/transformers/models/auto/auto_factory.py", line 763, in __getitem__
    return self._load_attr_from_module(model_type, model_name)
  File "/usr/local/lib/python3.10/dist-packages/transformers/models/auto/auto_factory.py", line 777, in _load_attr_from_module
    raise RuntimeError(
RuntimeError: Failed to import transformers.models.llama.modeling_llama because of the following error (look up to see its traceback):
module 'torch.library' has no attribute 'register_fake'
    return getattribute_from_module(self._modules[module_name], attr)
  File "/usr/local/lib/python3.10/dist-packages/transformers/models/auto/auto_factory.py", line 693, in getattribute_from_module
    if hasattr(module, attr):
  File "/usr/local/lib/python3.10/dist-packages/transformers/utils/import_utils.py", line 1766, in __getattr__
    module = self._get_module(self._class_to_module[name])
  File "/usr/local/lib/python3.10/dist-packages/transformers/utils/import_utils.py", line 1780, in _get_module
    raise RuntimeError(
RuntimeError: Failed to import transformers.models.llama.modeling_llama because of the following error (look up to see its traceback):
module 'torch.library' has no attribute 'register_fake'
[2024-11-12 02:26:41,444] torch.distributed.elastic.multiprocessing.api: [ERROR] failed (exitcode: 1) local_rank: 0 (pid: 1544) of binary: /usr/bin/python3

Suspect this is due to an issue in the underlying docker container used in FSDP example. Needs further investigation.
cc @sean-smith


Bump solnet version and fix references to sysvars

Status
Type
⚠ Core Change
Issue

Ready
Hotfix
No
Closes #25

Problem
What problem are you trying to solve?
Invalid references to sysvars
Solution
How did you solve the problem?
Bumped Solnet version and used SysVars.RentKey instead of the old reference to system program

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

Solnet.Mango/MangoProgram.cs
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
1850

💛  - Coveralls


Only prints the last column in a row

Tested with the example app, for e.g line:
    ticket.row([
      PosColumn(text: '2', width: 1),
      PosColumn(text: 'ONION RINGS', width: 7),
      PosColumn(
          text: '0.99', width: 2, styles: PosStyles(align: PosAlign.right)),
      PosColumn(
          text: '1.98', width: 2, styles: PosStyles(align: PosAlign.right)),
    ]);

It only prints 1.98 aligned to the right.

Nvm it was just printing to the wrong paper size. Looking for a way to print something smaller than size1 if possible though.


Crash?

What version of ViaVersion are you using? Type /ver viaversion
0.6.4
What plugins are you using? Type /plugins
Plugins (46): AsyncWorldEditInjector, WorldEdit, EggCatcher, Skript, LogBlock, PluginManagerReloaded, SkQuery, RandomTP, Lockette, CustomRecipes, UnCrafter, WorldGuard, SkinsRestorer, ImageMaps, Shopkeepers, WildSkript, BlocksHub, PermissionsEx, HorseTpWithMe, Simple-AutoSave, BlocksHub-LogBlock, AntiJoinBot, ViaVersion, StaffChat, Tablist, Vault, SilkChests, Titlebar, Essentials, multispleef, MobArena, Trading, PreciousStones, ProtocolLib, LibsDisguises, NoCheatPlus, AsyncWorldEdit, ColoredTags, BlocksHub-WorldGuard, EssentialsChat, Parkour, EssentialsSpawn, SexyMotd, AuthMe, ScoreboardStats, HolographicDisplays
Is there an error in the console? Use pastebin.com. Is there a kick message?
http://hastebin.com/xuyuqiqanu.coffee

You are giving yourself invalid items, this is vanilla functionality?


[Github profile readme generator] -> [Others]

Github profile readme generator
Link:  https://rahuldkjain.github.io/gh-profile-readme-generator/

Not a design resource.


[Question] External service

Hello, I am new using this type of architecture and I would like to ask some questions

What is the best way to obtain information from an external service?
If the connection string is obtained from an external service (Azure Function). How should I affect the command and query handler?

Many thanks

Hi!

Keep external services as infrastructure - interfaces within app and implementation outside. What exactly you wanna know? Best way - you mean http instead of something else like socket or what?
Correct me if I'm wrong but you probably can use under development secret file or use Azure Key Vault. On production you will probably keep your secrets in Vault / Env Variables. Then you can access them via IConfiguration.

I don't have much experience with Azure Serverless so keep that in mind.

What is the best way to obtain information from an external service?

Add an interface and supporting types to Application.
Add an implementation and mappings to Infrastructure.
Remember that Infrastructure depends on Application, so you won't have access to the external service types.

If the connection string is obtained from an external service (Azure Function). How should I affect the command and query handler?

The devil is in the details, but perhaps you will need to access the connection string as part of dependency injection or follow up on @rafalschmidt97 recommendations.

Happy coding


Imp tests updates

Jenkins: test this please

This PR has the same commits as another reviewed and closed PR Imp tests #2020.
I only duplicate it here due to a Jenkins copyright check bug.


Add SnapLogic to ecosystem consumers

Problem
SnapLogic is missing from the Ecosystem page!
Solution
This makes SnapLogic an official member of the OpenLineage ecosystem. Welcome, SnapLogic team!

[ ] Your change modifies the core OpenLineage model
[ ] Your change modifies one or more OpenLineage facets

If you're contributing a new integration, please specify the scope of the integration and how/where it has been tested (e.g., Apache Spark integration supports S3 and GCS filesystem operations, tested with AWS EMR).
One-line summary: add SnapLogic to Ecosystem
Checklist

[ ] You've signed-off your work
[ ] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[ ] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] Your comment includes a one-liner for the changelog about the specific purpose of the change (not required for changes to tests, docs, or CI config)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2024 contributors to the OpenLineage project


Tutorial to use this package

I was wondering if there is any tutorial to use this package? Thanks!

This just attempts to implement the DEC method in PyTorch; I wouldn't suggest using this code for anything other than learning or research. An example using MNIST data can be found in examples/mnist/mnist.py, which is probably the nearest thing. In the paper the authors suggest keeping the autoencoder structure reasonably constant so that should be a good starting point.


UI change

In the app  bottom nav bar button is very light .Its hard to understand what is write there.so I wanna work with this issue .Please assign it to  me .

In this we can add process bar. Please assign this issue to me under GSSOC'22.


container with hostPort can not access the local host's portIP:hostPort from the container

container with hostPort can not access the local host's portIP:hostPort from the containe
Expected Behavior

Current Behavior

I have 2 host with calico as the container network, assume the host name like hostA and hostB.
hostA create a container with hostPort named tomcatA with IP <IP_ADDRESS>, then I login container and can access my tomcat service with containerIP:port, but can not access with hostIP:hostPort.
Then I create same container service with hostPort on hostB, and I can access the hostB's service from container on hostA with both hostB's hostIP:hostPort and hostB's containerIP:containerPort.
so the main problem was, why I can not access the container service with the hostIP on same host plus hostPort?
Here is the sample yaml file which I used to create my test app:
kind: Deployment
apiVersion: extensions/v1beta1
metadata:
  name: tomcat
spec:
  template:
    metadata:
      labels:
        app: tomcat
    spec:
      hostNetwork: false
      containers:
      - name: tomcat
        image: hchenxa1986/tomcat:cni
        imagePullPolicy: IfNotPresent
        ports:
        - protocol: TCP
          hostPort: 8088
          containerPort: 8080
        resources:
          limits:
            cpu: 500m
            memory: 512Mi

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

calicoctl version (e.g. calicoctl version):

root@hchenxa-2:~# calicoctl version
Client Version:    v2.0.0
Build date:        2017-12-21T21:02:56+0000
Git commit:        8006d580
Cluster Version:   v3.0.4
Cluster Type:      k8s,bgp

calico/node version (e.g. docker run --rm calico/node versions)
Orchestrator type and version (e.g. kubernetes, mesos, rkt):
Operating System and version:
Link to your project (optional):

append some more info related to the test:

create the resource and login docker container:

root@hchenxa-2:~# kubectl exec -it tomcat-76bffcdf79-npbx8 bash
root@tomcat-76bffcdf79-npbx8:/usr/local/tomcat# ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
2: tunl0@NONE: <NOARP> mtu 1480 qdisc noop state DOWN group default qlen 1
    link/ipip <IP_ADDRESS> brd <IP_ADDRESS>
4: eth0@if17: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP group default
    link/ether a6:95:aa:00:b5:0d brd ff:ff:ff:ff:ff:ff link-netnsid 0
    inet <IP_ADDRESS>/32 scope global eth0
       valid_lft forever preferred_lft forever

then access the containerIP:containerPort

root@tomcat-76bffcdf79-npbx8:/usr/local/tomcat# telnet <IP_ADDRESS> 8080
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.

then check the HostIP:hostPort, found the connection timeout.

root@tomcat-76bffcdf79-npbx8:~# curl <IP_ADDRESS>: 8088 --connect-timeout 2
curl: (28) Connection timed out after 2001 milliseconds

The issue only exist when using the container to access the same host with hostIP:hostPort.

/cc @caseydavenport @tmjd any comments for this? Thanks!

And I tried with the calico/cni:v3.2.1 but still not work as expect.
But After update the portmap plugin with v0.7 from https://github.com/containernetworking/plugins/compare/v0.7.0...master, it works as when create the hostport container, it will also update the iptables to add CNI-HOSTPORT-DNAT.
below the result:

use the portmap bin which build in calico-cni
result: the container can not access the service with hostIP:hostPort

and the iptables rule like below:
root@hchenxa-3:/opt/cni/bin# iptables -t nat -S | grep 8089
-A CNI-DN-07366525d6cfad4a202cd -p tcp -m tcp --dport 8089 -j DNAT --to-destination <IP_ADDRESS>:8080

update the portmap plugin, and it can access the service now.
the iptables rule like below with the new portmap plugin

-A CNI-DN-88757da0fc8a2331cdc53 -s <IP_ADDRESS>/32 -p tcp -m tcp --dport 8089 -j CNI-HOSTPORT-SETMARK
-A CNI-DN-88757da0fc8a2331cdc53 -s <IP_ADDRESS>/32 -p tcp -m tcp --dport 8089 -j CNI-HOSTPORT-SETMARK
-A CNI-DN-88757da0fc8a2331cdc53 -p tcp -m tcp --dport 8089 -j DNAT --to-destination <IP_ADDRESS>:8080
-A CNI-HOSTPORT-DNAT -p tcp -m comment --comment "dnat name: \"k8s-pod-network\" id: \"c0de752c193ead0aeb419f1d2c0667f5796c13d672b28598ade3750d1c3948eb\"" -m multiport --dports 8089 -j CNI-DN-88757da0fc8a2331cdc53

this commit fix this issue https://github.com/containernetworking/plugins/commit/5576f312

the issue was related to calico cni, so post the issue to https://github.com/projectcalico/cni-plugin/issues/585 and close this one.


[WIP] feat: add managed model registry prometheus job, metrics, and alering rules, fixes RHOAIENG-4273

Description

Adds managed model registry prometheus config for the following:

job
metrics
alering rules

Fixes RHOAIENG-4273

https://issues.redhat.com/browse/RHOAIENG-4273
How Has This Been Tested?

WIP
Screenshot or short clip

Merge criteria

[X] You have read the contributors guide.
[X] Commit messages are meaningful - have a clear and concise summary and detailed explanation of what was changed and why.
[X] Pull Request contains a description of the solution, a link to the JIRA issue, and to any dependent or related Pull Request.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

This is a draft PR with prometheus config changes to the yaml file. I'm creating another PR to enable loading this config in odh incubation. This PR should be merged after the model registry component itself is merged into rhds.
Please review and let me know if there are any changes or anything else is needed.

@zdtsw I updated the file name etc. to match the merge request for the SOPs document. I'm waiting on the managed tenants team to review it.

@zdtsw sops have been added.

@ykaliuta if  https://github.com/red-hat-data-services/rhods-operator/pull/329/ has  the latest changes from here, we can have https://github.com/red-hat-data-services/rhods-operator/pull/318 closed.

@ykaliuta if #329 has the latest changes from here, we can have #318 closed.

sure, but let's finish #329 first.

Closing this PR since these changes were included in https://github.com/red-hat-data-services/rhods-operator/pull/329


Filter controls for fields to send

With the feeder sending everything it gets from the AIS weather message (#2), there will probably be ERDDAP installations that don't want some of the fields, like ice. A filter control in the accepted_packets section should do, listing all the fields to send (or not send), and then filter the resulting query args to drop appropriately.

Resolved in c8f2ec961441c00c2158b59cad2a2bb58ee53502


Fix/do not write errors to file when used as library

Summary
We are using this package to migrate our content models. Whenever a runtime error occurs in one of our backends based on Lambda functions, the whole handler errors out because of an unhandled promise rejection caused by the Lambda's filesystem being mounted as readonly and thus not allowing the error files to be written.
Description
In order to fix this issue, the PR adjusts two things:

Error files will not be written in case the function is used as a library/imported from another location (shouldThrow === true).
The writing of error files will be awaited so that any possible exception raised will not result in an unhandled promise rejection.

Motivation and Context
When used as a library, the error files are probably irrelevant, except for debugging purposes, or might - as in our case - even be harmful, as depending on the runtime environment, no filesystem is available.
Todos
This is rather a proposal than an actual finished implementation, as I do not have the time and knowledge, to build and verify the changes.

[x] Provide proposal
[ ] Verify implementation
[ ] Add/adjust tests

Hey, hey, sorry you are running into this, and thanks for the proposal! I think this makes a lot of sense.
I will discuss it internally and let you know once there is more to share regarding whether or not and when this will be implemented! Will close this PR for the time being.


Issue with docs.zowe.org/stable/user-guide/initialize-zos-system

Description
You don't specify in the documentation on where to execute the ZWE command in the section "Initializing the z/OS system". I found on a GITHUB page and not in your documentation.  You assume to much to think everyone would know as to where this command is located and how it is executed.

Pages to Update

Screenshots

Expected behavior
This section should indicate where the ZWE is located and how to execute it by opening OMVS shell:

Execute "TSO OMVS" on the ISPF command line
cd  to your zowe/bin directory:  cd /usr/lpp/zowe/bin  or  cd directory.where.installed/bin
Execute /zwe init --config /path/to/zowe.yaml

Additional context

Several issues coming up with this new level.  Attempting to run with -mvs or just -config results in the same error that yaml file is missing dataset prefix:

cd /u/cai/zowe/runtimev2/bin
./zwe init mvs -v -c /u/cai/zowe/instancev2/zowe.yaml

Initialize Zowe custom data sets

Output:
Error: error reading input file: All collection items must start at the same column
Error ZWEL0157E: Zowe dataset prefix (zowe.setup.dataset.prefix) is not defined in Zowe YAML configuration file.   I tried stripping the comments and quoting fields with no success.  The yaml file clearly has the dataset prefix defined.

zowe.yaml.txt
Also,  the documentation does not indicate how to start Zowe without high availability instance.  I wouldn't be running that unless I was running in a multi system sysplex.

Another suggestion is to provide sample JCL to run the "zwe" command in batch instead of through command line.  When you are trying to figure out why this stuff doesn't work it makes alot easier to repeat then typing the command over and over again...
Also,  z/OS mainframe sysprogs prefer batch so they have record of what they did and for review.
//ZOWEINST EXEC PGM=BPXBATCH,REGION=0M
//STDOUT DD SYSOUT=*
//STDPARM      DD *
sh set -x;
cd /u/cai/zowe/runtimev2/bin;
./zwe init mvs -v -c /u/cai/zowe/instancev2/zowe.yaml;
/*

Attempting to start ZWESLSTC results in undocumented messages.  At least a search brings back nothing for the following:
ZWEL0047E, ZWEL0051W, and ZWEL0033E
ZWESVUSR ERROR ZWEL0047E failed to parse zowe.yaml - /u/cai/zowe/instancev2/zowe.yaml
ZWESVUSR WARN ZWEL0051W failed to read zowe.yaml, launcher will use default settings
ZWESVUSR ERROR ZWEL0033E failed to get ROOT_DIR dir

It appears the yaml file issues has to do with the security section being uncommented.  I am testing that now and will update the issue when I determine if this is the cause.


Implement check for tokens unvesting.

Call Rob's API to check if tokens need to be unvested.
Do this call each time the wallet is opened, but do it in the background so it doesn't interrupt anything else. Or if that's not possible, make sure it's only done a couple of times per month max.

related to https://github.com/threefoldtech/wallet-next/issues/114
closing


Unexpected token: new

I'm relatively new to bip32-utils and it's the reason I'm finally learning Node.js. So far all good but I've encountered this problem and I don't know how to solve that, and I've took it from your examples. Thanks
var account = new bip32utils.Account([
     new bip32utils.Chain(external.neutered())
    new bip32utils.Chain(internal.neutered())
])

Error message:
^^^
new bip32utils.Chain(internal.neutered())
SyntaxError: Unexpected token new
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.runMain (module.js:604:10)
at run (bootstrap_node.js:393:7)
at startup (bootstrap_node.js:150:9)

Process finished with exit code 1
^^^

You're missing a comma after external.neutered()) ... thanks for pointing out the mistake in our README :+1:


chore: fixing postman API publishing

Reverting postman/collections/Interledger.json file to alpha5 version and re-adds all of the newest changes since then to properly making publishing Postman APIs work (tested publishing manually and via the CLI)

Changes proposed in this pull request

Ever since commit, the Postman API publish to main has not been working, since in the PR, the collection file, Interledger.json was exported and updated manually instead of via the Postman Git integration. It looks like each request needs an id to properly publish the API. This PR takes the collection from the working API collection of the alpha 5 tag, ands adds the changes since the alpha 5 release to make Postman API publishing work again.
Context

Fixes #1949
Also opened up an issue on the Postman side: https://github.com/postmanlabs/postman-app-support/issues/12577 to see if I can properly debug if this happens in the future.
Checklist

[x] Related issues linked using fixes #number
[ ] Tests added/updated
[ ] Documentation added
[ ] Make sure that all checks pass
[x] Postman collection updated

Should we just merge and hope for the best or is there a way to actually test this?

@sabineschaller
I ran the Postman CLI publish api ... command locally (identical to the Github Actions one) and it worked


[Application] Update yolo example of torch for tracking gradients

For tracking the gradients of LOSS class,
I removed in-place operation in the LOSS class.
Signed-off-by: Seungbaek Hong<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2159. 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://ci.nnstreamer.ai/.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2159-202303291846000.84455800056458-5063ada174cc22790240b55054ca1eb441248890/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2159-202303291939100.85935592651367-dc36ab00c41aaa613dc258c8bd3f1a5279b54e2b/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2159-202303291940340.14704895019531-a17fdd74c061802fc1297cd617f9abf4db5565f7/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2159-202303291957550.58578205108643-84a3a44f1824d56ad55b7ba6534289fa92a0662f/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2159-202303301459400.38258004188538-e9a30963ab0a48e7b3bdddc7e0b61cfec8b07408/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2159-202303301509530.19843196868896-9b566118cd4e5d4667c3e5395198e8005aa4d7a4/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2159-202303301516170.92813491821289-e9a30963ab0a48e7b3bdddc7e0b61cfec8b07408/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2159-202303301540440.15232992172241-462485860df0d59541ae686abd33624410bdf3ad/report/pylint_result.txt.


⚠ T:Riza has degraded performance

In 34c3a82, T:Riza (https://www.triza.dev) experienced degraded performance:

HTTP code: 500
Response time: 1142 ms

Resolved: T:Riza performance has improved in 8098c10.


Merge readthedocs and internal wiki for CHPModel

the internal wiki gives additional information that should be transfered to RTD

All necessary inputs from internal wiki are included in RTD.