Support new kubernetes resource EndpointsSlice

https://github.com/kubernetes/enhancements/blob/master/keps/sig-network/20190603-EndpointSlice-API.md adds a new API to Kubernetes. It has a lot of overlap with our ServiceEntry. It seems plausible that we could completely remove ServiceEntry in favor of the new API.
At the very least, we will need to update Pilot/Galley to understand EndpointSlices
Actions:

[x] Initial support: https://github.com/istio/istio/pull/18394
[x] Improve unit test coverage - run all tests with endpoints and slices: https://github.com/istio/istio/pull/19645
[ ] Improve e2e coverage
[ ] Add a mode where both endpoints and slices are read
[ ] Do not read nodes if just using slice
[ ] Do not require pod reading if just using slice? Needs more stuff put into the slices like SA
[ ] Documentation
[ ] Replace/add Endpoint Slice in place of STATIC service entry in tests/docs/users usage
[ ] Investigate backporting endpoint slice as a CRD to old clusters

Test using kind:
kind: Cluster
apiVersion: kind.sigs.k8s.io/v1alpha3
kubeadmConfigPatches:
- |
  apiVersion: kubeadm.k8s.io/v1beta2
  kind: ClusterConfiguration
  metadata:
    name: config
  apiServer:
    extraArgs:
      "feature-gates": "EndpointSlice=true"
      "runtime-config": "discovery.k8s.io/v1alpha1=true"
  controllerManager:
    extraArgs:
      "controllers": "endpointslice"

kind create cluster --image aojea/kindnode:latest --config config.yaml (there is not official 1.16 node image yet, config.yaml refers to above)

@howardjohn EndpointsSlice is to replace Endpoints, it is  1:n  relation for service and endpointslices , so it is a little complicated to support it.  In k8s 1.16 it is an alpha feature, so maybe in 1.17 or 1.18, it can be beta.
BTW, kube-proxy is a good example to understand EndpointSlices.

It seems we also cannot use EndpointSlice until we update client-go. We cannot update client-go to 1.16, because then we lose support for things that were removed in 1.16. And additionally, we depend on other libraries that refuse to upgrade to 1.16 for the same reason... not sure what we can do here.

cc: @freehan @robscott

status update: blocked by https://github.com/istio/istio/issues/17831, which shouldn't be too much work to get working. Once we have that done, I have a WIP that is probably 80% done for base level support. To actually do similar splicing in Envoy is much more work and dependent on some Envoy side changes

I am about 80% done with this in https://github.com/istio/istio/pull/18394/files, just need to fix a few tests and clean things up I think

Hello. From which version does Istio support the new EndpointSlice resource?

Hello. From which version does Istio support the new EndpointSlice resource?


Feature exif orientation from android stream

** Briefly describe changes and the motivation behind them here **
This PR contains:

[ ] Changelog
[ ] Documentation
[x] Tests

@kristianhasselknippe: Please fill in that template-placehoder text ;)

Great stuff!


Fix the rest of the deprecated gradient warnings

Hey there,
#307 was a great start to fix the gradient warnings, but only fixed one instance. This change updates the gradient helper that gets used everywhere else, so this will fix gradient CSS warning throughout all of Hopscotch.
Fixes #232, #307
Thanks!

Hi, can this be merged asap?

Thanks @zimmi88!


Fix unlikely but possible XSS in modern-skylight.js

An XSS attack would probably need another extension to work, like with https://www.mediawiki.org/wiki/Extension:AnchorHandler
This should be safer code anyway.

Thank you for your contribution and concern, even lacks of i18n support. On behalf of the WikiChan Development team, I would like to thank you :D


[WIP] Introduce journal fragments

This PR changes the way a journal is written. Right now for each transaction a file is created. This is not appropriate when having lots of transactions. Now the transactions are written to fragment files. Each can be 16MB of size which reduces the number of files significantly.

Merging into feature branch in order not too get a too big PR


Decide on a launch date

When launching, reach out to the following people/places:

ClimateAction.tech
Work on Climate
Green Web Foundation
???

Ideally I'd like this to be the start of September. Let's see how we go; I think the slowest part will be incorporating the charity; see #2 .

When launching, reach out to the following people/places:
* ClimateAction.tech

* Work on Climate

* Green Web Foundation

Would it make sense to reach out even before launching? Not necessarily with the communities, but with the administrators, to get feedback on the idea from people who are already working in the field

yeah that's a great point


Laravel Spark integration

Hi, is there any support for Laravel Spark? Im using laravel spark for teams so if somebody is going to register a new team i want a tenancy for that team.
So for example if i create a new team called Test, i want a test.mydomain.com tenanct.
I hope this will be added soon or if someone could help me that would be nice.

Hi. Documenting Spark integration is planned, but it's not of high priority at the moment.
The exact thing you're describing sounds like you want to listen to a "team was created" event and do Tenant::new() with the tenant's data and domains.

Hi, would it be possible with for example financial support to give this feature a higher priority? ;)

Hi. Documenting Spark integration is planned, but it's not of high priority at the moment.
The exact thing you're describing sounds like you want to listen to a "team was created" event and do Tenant::new() with the tenant's data and domains.

That is what im trying to do, but also de routing isnt working because of the tenancy routes

To mark the routes as tenant routes, apply the 'tenancy' middleware group to Spark routes.
I think this should be somewhat similar to Nova integration: https://tenancy.samuelstancl.me/docs/v2/nova/

To mark the routes as tenant routes, apply the 'tenancy' middleware group to Spark routes.
I think this should be somewhat similar to Nova integration: https://tenancy.samuelstancl.me/docs/v2/nova/

Oke thankyou i will give it a try

Hi. Documenting Spark integration is planned, but it's not of high priority at the moment.
The exact thing you're describing sounds like you want to listen to a "team was created" event and do Tenant::new() with the tenant's data and domains.

I guess I have to do this before a team is registered? Because all of the team stuff isn't in the apps DB anymore this will be in the tenant right?

I hope to look into Spark soon. Right now I'm sorry, but I don't know enough about Spark to help with that.
You need to decide what should be in the central database and what should be in the tenant databases. If teams = tenants, then teams should be in the central database.
I recommend reading through this package's docs. They might help with these things.

@BeniSt

Hi, would it be possible with for example financial support to give this feature a higher priority? ;)

That would be very much appreciated. Though I don't have a Spark license and I can't promise if I can realistically get to this soon, so I don't want to get donations for a feature that I'm not sure I can make happen. Maybe I could help you integrate this package into a Spark app and then document that?

@BeniSt

Hi, would it be possible with for example financial support to give this feature a higher priority? ;)

That would be very much appreciated. Though I don't have a Spark license and I can't promise if I can realistically get to this soon, so I don't want to get donations for a feature that I'm not sure I can make happen. Maybe I could help you integrate this package into a Spark app and then document the process?

To mark the routes as tenant routes, apply the 'tenancy' middleware group to Spark routes.
I think this should be somewhat similar to Nova integration: https://tenancy.samuelstancl.me/docs/v2/nova/

Sadly for Spark there is no such thing as a config file like Nova has, so can't place the tenancy middleware. Could be cool if we can figure this out together

Isn't there a SparkServiceProvider or something?

@stancl I will of course give you access to a new and empty spar installation. It would be fantastic, if you will have a look how to integrate this.

@BeniSt Could you contact me on Telegram or Discord?

Decided to not do this. Spark is too old.

What's up with spark next?


Add muxcover -mux2=cost option

Why? All mux costs are relative to each other. Making the mux2 cost configurable doesn't add anything.

The reason behind this was that, instead of exporting the default costs from muxcover:
https://github.com/YosysHQ/yosys/blob/6951e3207063f66814e765d7d72e553b8b8eace1/passes/techmap/muxcover.cc#L27
I decided to let the user determine this base cost so that it can compute new relative costs:
https://github.com/YosysHQ/yosys/blob/6951e3207063f66814e765d7d72e553b8b8eace1/techlibs/xilinx/synth_xilinx.cc#L287-L304
I can revert and go the former route, if you think that's better.

I can revert and go the former route, if you think that's better.

No, it's fine. I was just curious about the reason for this change.


linuxPackages.rtl8821ce: unstable-2021-03-21 -> 2021-06-06

Motivation for this change
old version didn't compiled with linux 5.12
Things done

[ ] 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)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

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

[x] Fits CONTRIBUTING.md.

Run with no problem on my computer that use it after upgrading to linux 5.12 (linuxPackages_latest) (tested with overlay and overrideAttrs, with the same src attribute)

Need to change commit name


The following section does not work

can-send-msg: true
messages:
  sender-error: "§fNo se ha encontrado ese jugador"
  receiver-message: "§c%sender intentó enviarte un mensaje: §7%message"

I have also added the vanish.see permission to the Guest range (which is the current one I have) and it still doesn't work, I don't know if it's a PurePerms problem.

Again everything seems to be working here so without enough information I can't help you with that either.

[01:05:06.611] [Server thread/INFO]: This server is running PocketMine-MP
[01:05:06.611] [Server thread/INFO]: Server version: 4.2.3 (git hash: 6c1c0c867e178f8da8b266e962ca13f4b4280ab4)
[01:05:06.611] [Server thread/INFO]: Compatible Minecraft version: 1.18.10 (protocol version: 486)
[01:05:06.611] [Server thread/INFO]: PHP version: 8.0.16
[01:05:06.611] [Server thread/INFO]: PHP JIT: not supported
[01:05:06.611] [Server thread/INFO]: Operating system: linux

VanishV2 version: Dev#126

When i use "/tell MrBlasyMSK Hola" you do not get the message "§fNo se ha encontrado ese jugador" and the other do not get the message that you have been whispered.
(I have tested it with and without the vanish.see permission)

You have to set can-send-msg to true

I meant false

Oh, now it works, mb sorry


feat: add transformation bindings

What kind of change does this PR introduce?
see title

Codecov Report
Base: 87.01% // Head: 86.90% // Decreases project coverage by -0.11% :warning:

Coverage data is based on head (42a9ed3) compared to base (323e65e).
Patch coverage: 89.47% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
- Coverage   87.01%   86.90%   -0.12%
==========================================
  Files          12       12
  Lines         285      313      +28
==========================================
+ Hits          248      272      +24
- Misses         37       41       +4

Impacted Files
Coverage Δ

storage3/_async/file_api.py
86.48% <86.66%> (-1.02%)
:arrow_down:

storage3/_sync/file_api.py
86.48% <86.66%> (-1.02%)
:arrow_down:

storage3/types.py
93.54% <100.00%> (+2.24%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


NOP

@scotthasbrouck
I will delete this branch in 7 days because it's rather outdated. Let me know if you think otherwise.

this one can be removed


Add JsonField plugin

This adds a JsonField plugin for handling json columns. This
handles reading json encoded strings and from the db and decoding
them. It also encodes new values into json encoded strings when
writing fields. This also includes plugins for records and fake
records which ensure json columns are synced before a record is
saved.
The JsonField is a mixin that can be used on models and read
models. It provides a DSL for defining json field accessors,
readers and writers. A json field reader adds a reader method that
reads from a source attribute (expecting a json encoded string) and
returns the decoded value. The json field writer adds a writer
method that takes a value and encodes it, writing it to its source
attribute. The JsonField also tracks its json field names and
json source field names which allows inspecting what has been
configured and also is needed for the record/fake record mixin.
The Record and FakeRecord mixin handle syncing up json fields
with their source attributes. This is needed because json fields
can be modified without using the json field writer. For example,
hashes and arrays can be modified by adding new items or
key-values. Since the writer isn't used these changes won't be
persisted to the source attribute. The record adds a before save
callback that uses the configured json field accessors from its
model, reads their current values and writes them using their
writer. This makes them update their source attributes as expected.
The fake record does the same logic only it overwrites the save!
method, syncs the json fields and then supers.
Finally, this includes test helpers which make it easy to test that
a class has json fields configured. This avoids having to know
internals to how json field works.
@kellyredding - Ready for review.

@jcredding "... handles reading json encoded strings to and from the db and..."

@jcredding could you maybe discuss the encoder/decoder API, its purpose, and how it uses JSON by default in the commit msg?

@jcredding I'm assuming this is close to what it is extracted from, implementation-wise?  Any major departures (besides configuring the encoder/decoder)?  I noticed some differences in the main implementation (haven't scoured the tests yet), but everything seems like changes to be more thorough or straight-forward).

@jcredding how did we get away with not having a fake record mixin in the thing we extracted this from?

@jcredding a couple of questions - otherwise 👎

@kellyredding - Yeah, the big differences between this and what it was extracted from are that it has a configurable encoder/decoder and I added the FakeRecord mixin. There were a few other smaller changes:

The reader and writer rescue standard errors that come from encoding/decoding and raise an invalid json error. This was copied for the reader but I added it for the writer. This should help with debugging
which field is being written to with invalid json and where its happening.
I added methods for the json_field_readers and json_field_writers and used them when adding field names to their collections. These were just ivars in what I extracted the json logic from but having the methods allowed removing code that optionally set the ivars to an array everytime a DSL method was called.
I reworked the tests to provide more coverage and break them into smaller chunks. Specifically, I broke up testing the individual DSL methods into different tests and added tests for fields with a custom source. I also split testing the DSL methods from testing the instance methods that are defined by calling the DSL method into different contexts. I felt this provided better organization.

As far as the FakeRecord mixin goes, I'm mostly providing it for completeness. In our usage of JSON fields it doesn't provide much for us because we never check the source in our tests. This is mostly due to the JSON string being hard to compare values against because it may reorder hash keys. So for us, it hasn't mattered that the source field may not match the json field. So, while this doesn't fix an issue or fill a current need for us, I felt it was more complete. There are probably edge cases where having this would be needed that we just haven't run into. One thing I like that it does is if we modify a json field not using the writer, like adding a new key-value to a hash and that somehow creates invalid json, this will error now where it previously wouldn't.

@jcredding that all makes sense and was helpful to know - thanks man.  Update the commit msg and I'm good here 👎


How to enable Completion other than main.rs file?

How can I enable Rust-Analyzer in files other than main.rs file?

Working in a single rs file. But not work when not mod {file} in main.rs.


Protecting against timing attacks

The previous string comparison method didn't operate in constant time.
If the string under test wasn't the same length as the secret the method
instantly returned.
This implementation is constant time with respect to the untrusted
string and executes very quickly (that's why the style is not quite as compact
as one might expect).

hmmm it doesn't look like the SDK even uses this function

The SDK uses it for CSRF checking.
I'm not sure if it's possible to write something that's constant-time for all lengths of inputs.  For example, in your code, trusted_chars[idx] will trigger different code paths depending on whether the index is in bounds or not.  It will be "more" constant time than the current early exit code, but I wonder if it's better to draw a hard line and say "we don't leak contents; we do leak the length; use appropriately" instead of "we don't leak contents; we sorta leak the length".

BTW, it looks like neither the old nor new code even hides the content correctly.  The old one calls .all? at the end, which I assume has an early exit.  Probably my fault.  The new one has a secret-dependent branch (the unless).

Ok, I made a fix, sorta.

Switch from all? to reduce to fix the leak.
The docs now explicitly say that the length will be leaked.

Thanks for bringing this issue up!  Though I think we'll leave the length leak as-is, it did make me think more rigorously about the issue and also incidentally identified a blatant/embarrassing bug.


TestInitialScalePostUpgrade failed as postupgrade test, after upgrading to 0.17 serving

https://prow.knative.dev/view/gs/knative-prow/pr-logs/pull/knative_operator/251/pull-knative-operator-upgrade-tests/1296540928600707074
Using operator to upgrade serving 0.16.0 to serving 0.17.0, TestInitialScalePostUpgrade tests failed after switching to 0.17.0 knative serving.
=== RUN   TestInitialScalePostUpgrade
=== PAUSE TestInitialScalePostUpgrade
=== CONT  TestInitialScalePostUpgrade
=== CONT  TestInitialScalePostUpgrade
    service_postupgrade_test.go:148: Getting service "init-scale-service"
=== CONT  TestInitialScalePostUpgrade
    service_postupgrade_test.go:154: Post upgrade Service is not ready with reason ""
--- FAIL: TestInitialScalePostUpgrade (0.20s)

huh, interesting.

In the logs I see
tive-tests.iam.gserviceaccount.com,},OwnerReferences:[]OwnerReference{},Finalizers:[],ClusterName:,ManagedFields:[]ManagedFieldsEntry{},} {{&ObjectMeta{Name:,GenerateName:,Namespace:,SelfLink:,UID:,ResourceVersion:,Generation:0,CreationTimestamp:0001-01-01 00:00:00 +0000 UTC,DeletionTimestamp:<nil>,DeletionGracePeriodSeconds:nil,Labels:map[string]string{},Annotations:map[string]string{autoscaling.knative.dev/window: 6s,},OwnerReferences:[]OwnerReference{},Finalizers:[],ClusterName:,ManagedFields:[]ManagedFieldsEntry{},}} {[{   <*>true <*>100 <nil>}]}} {{1 [{ConfigurationsReady False  {2020-08-20 20:35:02 +0000 UTC} RevisionFailed Revision "scale-to-zero-upgrade-service-lpdm8" failed with message: Initial scale was never achieved.} {Ready False  {2020-08-20 20:35:02 +0000 UTC} RevisionFailed Revision "scale-to-zero-upgrade-service-lpdm8" failed with message: Initial scale was never achieved.} {RoutesReady True  {2020-08-20 20:35:03 +0000 UTC}  }] <nil>} {scale-to-zero-upgrade-service-lpdm8 scale-to-zero-upgrade-service-lpdm8} {<*>http://scale-to-zero-upgrade-service.serving-tests.example.com <*>{<*>http://scale-to-zero-upgrade-service.serving-tests.svc.cluster.local} [{ scale-to-zero-upgrade-service-lpdm8  <*>true <*>100 <nil>}]}}} <nil>}

Maybe the cluster is getting close to full?

@vagababov is this same with the autoscaling bug?

Is the bug that the autoscaler scales things down to 0 with an initScale?

yeah it looks like the one we fixed already.
Will be released in 0.17.1.


Enhancement : Move tool : use modifier to apply to all layers

Could be useful to have modifiers on Move tool to apply to all layers (and all frames ?).

Fixed via #256


Rollback image-webpack-loader

Use v2 of image-webpack-loader as v3 requires environmental dependencies on mac, see #1335
I'm not sure if we keep note of which dependencies are pinned anywhere? Should we create a doc for this?

Coverage remained the same at 98.592% when pulling 9231d4626977d3a03377ac2000c8a770c71c9bc2 on samit4me:rollback-image-webpack-loader into 5a00e267bf8020cd8d7d2573db3bbc2242d0a4b1 on mxstbr:dev.

I'm not sure if we keep note of which dependencies are pinned anywhere? Should we create a doc for this?

We could add a note to CONTRIBUTING.md, but I'm going to ship this for now to make sure we can get v3.4.0 out!


generated manifest needs to have sample images and image refs

Right now the manifest that is generated for you if you don't have one doesn't include any image references in it, but that is often the step that is the most work in preparing a manifest.  We already have all the default images, we should put them in generated manifest by default.

Fixed in v0.5.0 (see https://github.com/manifoldjs/ManifoldJS/pull/206).


JDK 17 with Eclipse 2021-09: WindowBuilder Design page blank

JDK <IP_ADDRESS> (Windows x64 version)
Summary
JDK 17 with Eclipse 2021-09: WindowBuilder Design page blank
Steps to reproduce
Any Java code window using WindowBuilder fails to show anything on the Design page.
This is the same bug as Eclipse had with JDK 16, but there was a workaround for 16 in that you could add the line
--illegal-access=permit
to the eclipse.ini file to fix the problem. This no longer works with JDK17.
Also fails with this entry removed.
Expected results
As above
Actual results
As above
Triaging info
Paste the entire output of java -version.
openjdk version "17" 2021-09-14
OpenJDK Runtime Environment Temurin-17+35 (build 17+35)
OpenJDK 64-Bit Server VM Temurin-17+35 (build 17+35, mixed mode, sharing)
What is your operating system and platform?
Windows 10 21H1 19043.1237 on x86-64
How did you install Java?
msi installer
Did it work before?
See above re JDK 16
Did you test with other Java versions?
Worked before (plus eclipse.ini fix) with:
openjdk version "16.0.1" 2021-04-20
OpenJDK Runtime Environment AdoptOpenJDK-16.0.1+9 (build 16.0.1+9)
OpenJDK 64-Bit Server VM AdoptOpenJDK-16.0.1+9 (build 16.0.1+9, mixed mode, sharing)

Unlikely a JDK bug, please refer JEP 403: Strongly Encapsulate JDK Internals
Key parts of JEP 403 being:

--illegal-access=permit - Default mode from JDK 9 through JDK 15
-illegal-access=deny - Default mode in JDK 16, which you have overcome by usage of --illegal-access=permit
JDK-17 - No longer be possible for end users to use the --illegal-access option to enable access to internal elements of the JDK. It will still be possible to use the --add-opens command-line option, or the Add-Opens JAR-file manifest attribute, to open specific packages.

What warning message[s] pop up when you are running your application.

There are no warning messages I’ve seen – just a blank Eclipse screen.
Normally, when switching to WB Design, there is a popup noting the Design page is opening – this does not appear.
From what you say it appears this is a WB error? And they need to conform to the JEP403 restrictions?
So I need to log a WB bug?
Or can I still do a workaround by use of –add-opens? In which case what should that look like in full?
From: BrijeshNekkare @.>
Sent: Saturday, 25 September 2021 1:57 PM
To: adoptium/adoptium-support @.>
Cc: HREferg @.>; Author @.>
Subject: Re: [adoptium/adoptium-support] JDK 17 with Eclipse 2021-09: WindowBuilder Design page blank (#370)
Unlikely a JDK bug, please refer JEP 403: Strongly Encapsulate JDK Internals https://openjdk.java.net/jeps/403
Key parts of JEP 403 being:

--illegal-access=permit - Default mode from JDK 9 through JDK 15
-illegal-access=deny - Default mode in JDK 16, which you have overcome by usage of --illegal-access=permit
JDK-17 - No longer be possible for end users to use the --illegal-access option to enable access to internal elements of the JDK. It will still be possible to use the --add-opens command-line option, or the Add-Opens JAR-file manifest attribute, to open specific packages.

What warning message[s] pop up when you are running your application.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/adoptium/adoptium-support/issues/370#issuecomment-927005624 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGNL7QGEX56LR76DEUWNKLDUDVCAFANCNFSM5EXBPKXQ .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://github.com/notifications/beacon/AGNL7QHDPXZGNBMAIGL32D3UDVCAFA5CNFSM5EXBPKX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOG5APXOA.gif

From a long term perspective its recommended to stop access to internal elements of the JDK, so you should certainly seek support of your application team to migrate to JDK17.
List of the packages affected is available here. The sun.misc and sun.reflect packages will still be exported by the jdk.unsupported module, and will still be open so that code can access their non-public elements via reflection. No other JDK packages will be open in this way.
The java option --add-opens $module/$package=$reflectingmodule can be used to open $package of $module for deep reflection to $reflectingmodule. Code in $reflectingmodule can hence reflectively access all types and members in $package but other modules can not. When setting $reflectingmodule to ALL-UNNAMED, all code from the class path can reflectively access that package.
Example:

Caused by: java.lang.reflect.InaccessibleObjectException: Unable to make ClassLoader.defineClass accessible: module java.base does not "opens java.lang" to unnamed module

Workaround using add-opens:

--add-opens java.base/java.lang=ALL-UNNAMED

Reference: add-opens

Have done some digging around and it looks like the WindowBuilder team are aware of this and intend to fix in v1.9.8 (due Oct 9).
On that basis this should be closed as not a JDK 17 bug.


Final ug and dg update

Updated visuals in UG

Codecov Report

Merging #156 (878a7ed) into master (51b8ad9) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #156   +/-   ##
=========================================
  Coverage     75.01%   75.01%
  Complexity      540      540
=========================================
  Files            87       87
  Lines          1737     1737
  Branches        186      186
=========================================
  Hits           1303     1303
  Misses          377      377
  Partials         57       57

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[Bug]: Iam no talbe to use addMenuItemFields select with relationship

What happened?
when you add select with relationship, like eg. linkable_id, there is error Call to a member function isRelation() on null  <x-filament-actions::modals />###
How to reproduce the bug
just add in to your AdminPanelProvider plugin
->addMenuItemFields([
Select::make('linkable_id')
->label('Článek')
->relationship('linkable', fn($record) => $record->title)
->preload(),
])
Package Version
0.5.4
PHP Version
.3
Laravel Version
11.9
Which operating systems does with happen with?
Windows
Notes
No response

thank you for you great work anyway, iam for sure able to fix it on my own

@NifeCz
Because you're trying to use a standard Select component with a MorphTo relationship. When dealing with polymorphic relationships in Filament, you need to use the specialized MorphToSelect component instead.
https://filamentphp.com/docs/3.x/forms/fields/select#handling-morphto-relationships

ok, i see, thank you!! but still is not working:
 FilamentMenuBuilderPlugin::make()
                    ->usingResource(MenuResource::class)
                    ->addMenuItemFields([
                        MorphToSelect::make('linkable')
                            ->label('Článek')
                            ->types([
                                MorphToSelect\Type::make(Article::class)
                                    ->titleAttribute('title'),
                            ]),
                    ])

class Article extends Model implements MenuPanelable


Keycloak runs into OOM when starting

Describe the bug
Enabling the "standard auth service", Keycloak comes up and directly runs into an out-of-memory situation.
To Reproduce
Steps to reproduce the behavior:

Deploy EnMasse with -a standard
Look at the logs of the keycloak pod

Expected behavior
Not run into OOM situation.
Logs
keycloak-86f7594ccf-xt6sh.log

Setting the memory to -Xmx2048m fixed the issue for me.

Ok, it freaked out a bit later. Now I have -Xmx3072

I can possibly accept that 1GB is maybe tight for a Java application so that may explain the startup failure.  It is odd that I haven't been seeing this at all.
I am slightly surprised by the need to bump from 2 to 3GB.
Can you tell me:

EnMasse version
Number of address spaces (each address space gets its own keycloak realm)
Approximate number of authentications that had taken place (order of magnitude estimate fine)

EnMasse version: 0.22.0
Number of address spaces: 1
Approximate number of authentications that had taken place: 2

@ctron Which version of OpenShift? I have seen this behavior of Keycloak ~1 year ago, but it was with an older version, and there was something that only happened on that particular cluster.

OKD 3.10

Unless another occurrence has been seen, I think this issue should be closed.  If it is seen again, turn on  -XX:+HeapDumpOnOutOfMemoryError so the reason of for OOME can be investigated.

I think a have a new instance of this problem. 107 restarts in 9 days.
@k-wall I can give you access to this cluster if this is of any help. I will enable the Java option as well.

I am re-opening this, as I can now provide a heapdump: https://web.tresorit.com/l#Asmvkjv85mkoJPUNpufc4g
Note: this link will expire in 30 days

A quick look reveals a lot of byte[] objects from qpid proton sasl:
Some resources that needs to be cleaned up @rgodfrey ?

Just to add it seems like all the non closed connections are coming from the same IP address (<IP_ADDRESS> for what it's worth)


[stable/sentry] Creating the admin user in optionally

What this PR does / why we need it:
The sentry chart installation failed caused by the user-create job that complaining the admin user already exists in the scenario of re-installing the chart with an existing database.
For compatible with the above similar scenarios(with the existing DB volume), we should provide a flag that enables to create the admin user in optionally.

cc @rothgar

/ok-to-test

/lgtm

Thanks for pushing this through @mattfarina

Bump for GitHub search re-index.


Wrong width when image has max-width 100% property

Excellent lightbox, great job! But I ran into this issue: When img has a "Max-width: 100%" property (so, on your demo site, using developer tools, make img { border: 0; } become img { border: 0; max-width: 100%; }. ), the lightbox opens a distorted image, where the width is locked to the size of the literal thumbnail size.
I went through the code, and it happens here:
while (data.contentHeight > data.viewportHeight && count < 2) {
data.imageHeight = (count === 0) ? data.naturalHeight : data.$image.outerHeight();
data.imageWidth  = (count === 0) ? data.naturalWidth  : data.$image.outerWidth();
Count runs twice, why is that? On the first loop, the width is set to naturalWidth, which is correct, but on the second loop, because of data.$image.outerWidth(), the width var takes the thumbnail's outerWidth(), and that is what is fed to the lightbox CSS (var values of the last loop). I can change it to simply run the while loop once to get the correct values, but it looks like there is a reason you wanted it to loop twice, can you please let me know if that is the case and why?
Thanks in advance.

The loop runs twice to ensure the sizing takes into account any meta data (the width of the image may effect how caption text wraps, and thus it's height). The sizing doesn't do anything with the thumbnail, it's all based on the dimensions of the image and browser window.

Having the exact same issue, I'm using this code as a quick fix.
img:not(.boxer-image) {
	max-width: 100%;
	height: auto;
}

That is a really nice quick fix @ghost! Thanks for sharing!


Frontend Unit Testing is Broken

tests:
ng test --source-map=false
currently do not even compile

I remember some tests are out of date, and I have commented out some of the tests that are related to the user dashboard.

Github tests working pretty well! thanks Yicong!


Swift format

Perform a couple of small cleanups to the swifft-format tool.  The bulk of the changes here are related to the introduction of a small python based wrapper script to aid invoking the swift-format tool from vim.  This mirrors the clang-format integration script to integrate swift-format into vim.

@swift-ci please test

CC: @regnerjr  - could you please take a look at the IDE change?

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 161b1295b09d61b57a808e6aa56b9ef70baeca25
Test requested by - @compnerd

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 161b1295b09d61b57a808e6aa56b9ef70baeca25
Test requested by - @compnerd

@swift-ci please smoke test

CC @jrose-apple

@swift-ci please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 6e4f88e7f61dce076354fce0083b8371be7be3a9
Test requested by - @compnerd

@jrose-apple ignoring the indentation issue (Ill fix that later) ... is this okay?

No, please ask Ted on swift-dev if the policy has changed regarding editor support beyond syntax highlighting.

CC: @tkremenek

@swift-ci test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 6e4f88e7f61dce076354fce0083b8371be7be3a9
Test requested by - @tkremenek

@swift-ci please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 6e4f88e7f61dce076354fce0083b8371be7be3a9
Test requested by - @compnerd

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 6e4f88e7f61dce076354fce0083b8371be7be3a9
Test requested by - @compnerd

@swift-ci please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 9a0cfc4f54ca3e107e99d5e2e25534a9e70dcf2e
Test requested by - @compnerd

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 9a0cfc4f54ca3e107e99d5e2e25534a9e70dcf2e
Test requested by - @compnerd

@swift-ci please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 60e4652e15d60abcd49faf23b9a55f0b132585a0
Test requested by - @compnerd

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 60e4652e15d60abcd49faf23b9a55f0b132585a0
Test requested by - @compnerd

@swift-ci please test macOS platform

This was discussed on the mailing list and was okayed.  @jrose-apple's comments were addressed.

@compnerd This broke the build slightly. Please don't forget to run pylint next time ;-)


feat(ui): ui improvement of follower cards and posts

Didn't put any solid border around posts. Was not looking good.


Simplify HTTPS listening, leverage net/http methods

Make use of net/http Server.ListenAndServeTLS — this gives upcoming
http2 support in Go 1.6 for free.

@artyom can you clarify for me why Server.Serve() wouldn't support http2 automatically?

At least current http.Server implementation calls private method setupHTTP2 which can only be applied in a meaningful way when tls.Listener is set up by Server.ListenAndServeTLS, since setupHTTP2 calls http2ConfigureServer performing a lot of low-level setup on Server.TLSConfig which is ignored if Server is started on pre-established TLS connection.

Thanks for the pointers. I've not familiarized myself with the hooks/implications w/r/t http2 as that has started to land in golang tip.
I'll try to familiarize myself w/ the implications of http2 support in net/http and check back in here soon.


there is no type : Mode.RAW

version:
vite-plugin-markdown: "^2.1.0",
I would like to load markdown files in a string/raw format.
i had check the README , but i cant find the solution
What should I do?

You could try installing<EMAIL_ADDRESS>though there appears to be an issue with the table of contents parser


GH1280: Added Go.CD build provider

Added support for Go.CD. See documentation here: https://docs.go.cd/current/faq/environment_variables.html

Hi @RehanSaeed, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@RehanSaeed in line with our contributing process (https://github.com/cake-build/cake/blob/develop/CONTRIBUTING.md) can you please create an issue, and link to this PR from it?
Also, would it be possible for you to rebase your changes on top of the latest develop branch? Thanks!

Done.

Relates to #1280

@RehanSaeed LGTM!
Could you squash your commits to a single one and rebase against the develop branch?
Thanks!

Done

Thanks @RehanSaeed, your changes have not been merged and will be available in the next release. Thanks for your contribution 👍


Page permissions form (table) not rendering.

Summary
Not being able to see or edit the page permissions at menu location:  Page > Permissions > Page Permissions (table). I can see the form/table in the DOM with class='form-row row1 empty-form' class="pagepermission_set-2-empty''. This is being done using a superuser account.

Expected behaviour
I expected to be able to add specific page permissions to a user in this section but nothing appears below the table headers.
Actual behaviour
No rendering of table/form to edit page permissions.
Environment

Python version: 3.6.7
Django version: 1.11.20
django CMS version: 3.6.0

Update - I moved down to django-cms==3.5.3 and the table renders correctly...

Update - I moved down to django-cms==3.5.3 and the table renders correctly...

I did a diff between 3.6.0 and 3.5.3 and was not able to see much changes in the code related to permissions.
Did you had a chance look for a solution ?
I'm too long in my project to change of CMS so I have a big requirement and it is to have permission working and easy to set/change.
If you want we can collaborate to find a fix.

Ok, I'm making progress slowly but I'm in the right direction I think.
I created the following management command to debug my installation:
`from django.core.management.base import BaseCommand, CommandError
from django.core.management import call_command
from cms.api import create_page, add_plugin
from filer.models.filemodels import File
import locale
from django.contrib.auth.models import User, Group
from cms.api import Page, PageUser
from cms.models.permissionmodels import PagePermission
from cms.models import ACCESS_PAGE, ACCESS_DESCENDANTS, ACCESS_CHILDREN, ACCESS_PAGE_AND_CHILDREN, ACCESS_PAGE_AND_DESCENDANTS
class Command(BaseCommand):
args = ''
help = 'Create test permissions'
def handle(self, *args, **options):
    # These are the full set of available permissions. We exclusively want to test can_view.
    view_perms = {
        'can_change': False,
        'can_add': False,
        'can_delete': False,
        'can_change_advanced_settings': False,
        'can_publish': False,
        'can_change_permissions': False,
        'can_move_page': False,
        'can_view': True,
    }

    p = Page.objects.filter(title_set__title='Page 2')
    u = User.objects.get(pk=4)

    groups = u.groups.all()

    for pageToPermit in p:
        perm_a, _ = PagePermission.objects.get_or_create(grant_on=ACCESS_PAGE_AND_DESCENDANTS, group=groups[0], page=pageToPermit, **view_perms)
        perm_a.save()`

and I got this while before everything related to the page was empty

I think that there must be an Add button missing

@rolandf You're awesome man, I'm dealing with this issue with my site too
setup:
-  django 1.11.12
- django-cms==3.6.0
- python 3.7.4
And cannot add restrictions/permissions directly to pages. Love this framework and want to see this working. I'll give your code above a shot tomorrow at some point.  Thank you for providing it!

According to this thread in Django Cms google group, the issue doesn't seem to be linked to a particular version of Django-Cms but to a version of Django:
https://groups.google.com/d/msg/django-cms-developers/ApZ4seVaFc8/zTttVmBMBAAJ

Interesting... I am going to read through the Django CMS and Django release notes to see if anything stands out. Will update if I find anything to help us.

Unfortunately I don't do javascript but maybe related to jquery versions if you have multiple versions of the library ? (One for django, an other one for DjangoCms if you use Bootstrap ?)
django/contrib/admin/static/admin/js/change_form.js that is not triggered ?

Unfortunately I don't do javascript but maybe related to jquery versions if you have multiple versions of the library ? (One for django, an other one for DjangoCms if you use Bootstrap ?)
django/contrib/admin/static/admin/js/change_form.js that is not triggered ?

That could make sense because I could still manually display the table by removing display:false; in the html divs when I originally discovered this issue. Let me look into that as well. Good thought.

Unfortunately I don't do javascript but maybe related to jquery versions if you have multiple versions of the library ? (One for django, an other one for DjangoCms if you use Bootstrap ?)
django/contrib/admin/static/admin/js/change_form.js that is not triggered ?

That could make sense because I was able to still manually display the table by removing display:false; in the html divs when I originally discovered this issue. Let me look into that as well. Good thought.

I found this:
.empty-form {
display: none;
}
once I find where it's defined and where it's generated I will better understand this too.

It's not change form, I was able to enable it and there was a popup that appeared, it's not this.
I believe it's more in
django/contrib/admin/static/admin/js/inlines.js

.empty-form {
display: none;
}
is defined in django/contrib/admin/static/css/forms.css

Today I used djangocms-installer to install Django-cms with a lot of distinct config with variation of django version and django-cms version. Django 1.11 has a problem. Django-cms 3.5 with Django 1.9 is OK, Django-cms 3.6 with django 2.1 is ok too. I will not spend time trying to support Django 1.11, I prefer to use 2.1 and continue from there.

Today I used djangocms-installer to install Django-cms with a lot of distinct config with variation of django version and django-cms version. Django 1.11 has a problem. Django-cms 3.5 with Django 1.9 is OK, Django-cms 3.6 with django 2.1 is ok too. I will not spend time trying to support Django 1.11, I prefer to use 2.1 and continue from there.

Yep, you are correct. Just upgraded my dependencies to latest versions of Django / Django CMS and the permissions system seems to work as intended. I am going to close this issue and conclude our issue was a versioning problem. Thanks for all the help and happy coding!

Will django 2.1 be added to the list of officially supported versions for django-cms, right now the docs only say Django 1.8, 1.9 or 1.10 on the installation requirements here: http://docs.django-cms.org/en/release-3.5.x/introduction/install.html (the latest & 3.6 pages don't work currently)


[Errno 2] No such file or directory: 'install.conf.yaml'

I just installed dotbot as per this tutorial and receive the following error when running install:
~/.dotfiles$ ./install
Could not read config file:
  [Errno 2] No such file or directory: 'install.conf.yaml'

folder contents are:
.dotfiles/
┣ dotbot/
┣ .bashrc
┣ .condarc
┣ .gitmodules
┣ README.md
┣ install
┗ install.config.yaml

install.config.yaml contents:
- defaults:
    link:
      relink: true

- clean: ['~']

- link:
    ~/.bashrc: bashrc
    ~/.condarc: condarc

- shell:
  - [git submodule update --init --recursive, Installing submodules]

Any ideas?

Rename your install.config.yaml to install.conf.yaml (or change the ./install script to use the alternative name)


specify the db host

add option to specify the database host for the director to connect to (this is important for postgres where the default is to only allow "peer" connections and to use a password, you have to use "localhost" or an ip address).
Connecting bacula to postgres 9.5 without the host is giving this error:
 >FATAL:  Peer authentication failed for user "bacula"
 >DETAIL:  Connection matched pg_hba.conf line 11: "local	all	all		ident	"

because of these default pg_hba rules:
# This file is managed by Puppet. DO NOT EDIT.

# Rule Name: local access as postgres user
# Description: none
# Order: 001
local	all	postgres		ident

# Rule Name: local access to database with same name
# Description: none
# Order: 002
local	all	all		ident

Cool, thank you for the pr.  This looks to be incomplete.  The variable needs to get used somewhere for this to work.

Yeh, I forgot to commit the template change, oops.
Updated now :)

Looking better.  This will need a rebase now that I've merged your other PR, and the semi-colon and this looks good to go.

I'll close this off, tidy it up and re-submit.


The partial derivative of loss with respect to x in equation (5)(6) in paper

Thanks for your great work! The method is impressive and the result is amazing. I have a question about the derivation about the partial derivative of loss with respect to x in equation (5)(6) mentioned in the paper.

As described in the paper "dataset distillation" by Tongzhou Wang, the $\theta$ is a function of $x$, so the partial derivative of loss with respect to x can be writen as
$$\frac{\partial l(\theta(x), x)}{\partial x}=\frac{\partial l}{\partial \theta}\frac{\partial \theta}{\partial x}+\frac{\partial l(\theta(x), x)}{\partial x}\frac{\partial x}{\partial x}=\frac{\partial l}{\partial \theta}\frac{\partial \theta}{\partial x}+\frac{\partial l(\theta(x), x)}{\partial x},$$
in which the partial derivative about the second term $\frac{\partial l(\theta(x), x)}{\partial x}$ is only computed for $x$ not for $\theta(x)$.
However, I think the gradient with respect to $x$ as shown in equation (6) doesn't take the $\theta(x)$ into account and treats it as a constant. Maybe I miss something or I get a wrong derivation, so can you help to solve my question? Thanks a lot.

Hi, Chenyang,
Thank you very much for your interest in our work. Which equation are you referring to in Tongzhou's paper? I am not sure if I understand your question correctly, but I think we are not treating $\theta(x)$ as a constant as MTT's loss depends on the gradients computed using $X$ which is similar to DSA, maybe the writing is confusing because in DSA (algorithm 1), it uses y in the loss whereas in MTT (algorithm), it uses $\hat{\theta}$.

Thanks for your reply. Chapter 3.1 in Tongzhou's paper mentions that: "We derive the new weights $\theta_1$ as a function of distilled data $\tilde{x}$", in which $\tilde{x}$ means the distilled image.

Tongzhou's paper then introduce more steps on the synthetic images, obtaining $\theta_1$ from $\tilde x_0$, then obtaining $\theta_2$ from $\tilde x_1$ based on $\theta_2$, etc, assuming each batch size is 1. In this case, I think the $\theta_2, \theta_3, \cdots$ are also the function of $\tilde{x}_0$.

However, in the derivation of formula (6), the paper seems to simplify
\frac{\partial}{\partial \hat{X}_i}\sum_{j=0}^{T-1} \nabla_\theta \ell\left(\hat{\theta}_{t+j} ; \tilde{X}_j\right)

to
\frac{\partial}{\partial \hat{X}_i}\nabla_\theta \ell\left(\hat{\theta}_{t+i} ; \tilde{X}_i\right),

which I think the exact form should be
\frac{\partial}{\partial \hat{X}_i}\sum_{j=i}^{T-1} \nabla_\theta \ell\left(\hat{\theta}_{t+j} ; \tilde{X}_j\right),

since all the $\hat{\theta}_{t+j}(j\geq i)$ should be the function of $\tilde{X}_i$. To put it another way, $X_i$ should be the leaf node in the computation graphs of all the $\frac{\partial}{\partial \hat{X}_i}\nabla_\theta \ell\left(\hat{\theta}_{t+j}  ; \tilde{X}_j\right)(j\geq i)$. So I still don't understand the drop of other $\nabla_\theta \ell\left(\hat{\theta}_{t+j}  ; \tilde{X}_j\right)$ in equation (6).

Sorry for the late reply, I'm new to dataset distillation and managed to read the paper and code of DD, DC, MTT etc. Then I check the formula (6) again and still can't understand it.

@j-cyoung I agree with your point.


ask for the .zsh-theme file in the readme.md

what is the .zsh-theme file in the readme.md ? it is so beautiful!!

As I remember it's https://github.com/robbyrussell/oh-my-zsh/blob/master/themes/robbyrussell.zsh-theme
And probably it's a default theme in oh-my-zsh.

thanks，very nice of you，i get it！


Bugfix 2356

Bug Fix for Issue 2356, changed probability.py, updated FreqDist.tabulate() to consider samples keyword, which was ignored earlier

Thanks @anantvir. Sorry for the delay!


Activate NFDI AAI in the Repo4Cat, OpenID Connect

Configure AAI mechanisms in Repo4Cat using OpenID Connect.
See here:

https://guides.dataverse.org/en/latest/installation/oidc.html
https://doc.nfdi-aai.de/
https://doc.nfdi-aai.de/incubators/

TODO:

[ ] authentication via institutional account
[ ] authentication via ORCID (see https://guides.dataverse.org/en/latest/installation/oauth2.html and https://orcid.org/content/register-client-application-0)
[ ] activate and test OpenID Connect in the demo repository
[ ] activate and test OpenID Connect in the central repository

Is "authentication via institutional account" the same as "authentication via eduroam account" or is the latter another item for your TODO list?
In my opinion this issue and updating dataverse are most urgent.


Add Sentry logging to crawler

We must capture comprehensive details of errors raised on the crawler to support debugging and resolution.  To achieve this we should integrate Sentry in to the crawler applications such that errors and associated context / stack trace is logged to sentry.

implemented in #194


Missing step to configure NuGet before adding packages

The add package step fails with an error if NuGet source hasn't been set up in the project. I had to perform this step before adding packages to get it to work
dotnet nuget add source --name nuget.org https://api.nuget.org/v3/index.json

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

ID: 5389e8ca-c178-1b35-5745-105638c647c6
Version Independent ID: c0e2213c-aba0-24b9-4285-644ca1d39837
Content: Tutorial: Code a client app - Azure Digital Twins
Content Source: articles/digital-twins/tutorial-code.md
Service: digital-twins
GitHub Login: @baanders
Microsoft Alias: baanders

Hi @rikgarner Thank you for reporting this issue here. The command dotnet nuget add source --name nuget.org https://api.nuget.org/v3/index.json is used to add the official NuGet package source (nuget.org) to the NuGet configuration in the .NET CLI. However, Visual Studio Code (VS Code) does provide support for managing NuGet packages without executing the dotnet nuget add source command manually.
Can you please share more details on the Visual Studio Code version you are using to develop the client application? I would appreciate it if you can test steps again by creating a new .Net console application and see if you still encounter the same error.
Best regards,
Leela.

Hi @rikgarner We will be proceeding with closing the issue as the step you have mentioned is generic to Visual Studio code instance and cannot be tied to a single article. Please let me know if you have any issues or need further clarification by tagging me in the comment below. We would be happy to answer any additional questions you may have.


Large amount of loglines "Ignoring Class '\x01', type 'A'"

As it says on the tin, the plugin is producing a large amount of log lines as such:
log:
coredns  | [INFO] plugin/traefik: Ignoring class '\x01', type 'A'

config:
traefik https://traefik.local/api {
        a <IP_ADDRESS>
        refreshinterval 30
        ttl 5
        resolveapihost
        insecureskipverify
    }

Sorry about that. I was just curious what else was coming across on my
setup at home. I removed the log line.
On Fri, Aug 9, 2024 at 6:49 PM Joshua Hogendorn @.***>
wrote:

As it says on the tin, the plugin is producing a large amount of log lines
as such:
log:
coredns  | [INFO] plugin/traefik: Ignoring class '\x01', type 'A'
config:
traefik https://traefik.local/api {
a <IP_ADDRESS>
refreshinterval 30
ttl 5
resolveapihost
insecureskipverify
}
—
Reply to this email directly, view it on GitHub
https://github.com/scottt732/coredns-traefik/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABQDGZRT3GRO6QB7L6R76TZQVBQ7AVCNFSM6AAAAABMJHV2MSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2TQNZVHE3DQMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

no need to be sorry, was more just giving a heads up in case there was an issue :) Thanks for sorting it out!
I've rebuilt my container and it appears resolved.


Does Baresip support video conference?

We are using FreeSwitch/Kamailio and Baresip on our project. The audio/video calls are ok on mobile devices now. However, we need to support video conference. Can we implement it using Baresip?

calling into a server-side Video conference is supported by Baresip.
Hosting a video conference inside Baresip is not supported.

Thank you.


Cloned InheritableSetting seems to clone unnecessary stuff

#739 creates a point_in_time_copy of route configuration. It deep_dups everything in the hash, including classes. Problem is:
desc "...", {
  entity: ArticlePresenter
}

The ArticlePresenter class is also getting deep_duped, which produces a #<Class:0x...> which leads to problems with equality and also not copying all class methods. It doesn't play nice with grape-swagger in some cases.
Currently I solved it by adding
class Grape::Entity
  def self.deep_dup
    self
  end
end

I think I understand the need to make a current copy, but do you think it's OK to take out classes from the deep_dup?

Some of the dup'ing introduced in #739 was eventually removed, because we couldn't come up with a single failing test case. You should try with this one, too.
In theory, the not duping classes makes sense to me.

I'm referring specifically to https://github.com/intridea/grape/blob/master/lib/grape/util/inheritable_setting.rb#L50
It uses ActiveSupport's clone and deep_dup. So to prepare a more efficient copy it needs to copy only instances, hashes, arrays etc.
grape-swagger for example, can show hundreds of models from all APIs, even when there are only a few unique.
What was the specific case to make a point in time copy?

I am not sure, @dspaeth-faber can speak to it. I think a lot of this was theoretical to avoid modifying a parent's namespace's settings down the nesting path.

Well, it's not easy to explain, I think. You need point in time copies because values can change.
This happens when configurations are made for a namespace.  Some thing like this:

namespace :statuses do
  params do
    requires :user_id, type: Integer, desc: "A user ID."
  end
  namespace ":user_id" do
    desc "Retrieve a user's status."
    params do
      requires :status_id, type: Integer, desc: "A status ID."
    end
    get ":status_id" do
      User.find(params[:user_id]).statuses.find(params[:status_id])
    end
  end

 # now the inherited param should be user_name and not user_id
# to solf this you need point in time copies
  params do
     requires :user_name, type: String, desc: "A user name."
  end
  namespace ":user_name" do
     ...
  end
end

I think there have been other examples too.
The approach to generate the config at "compile" time / when a class is loaded and evaluted
make things hard. A better way would be to create Commands and
replay them at need. But unfortunately at the moment I can't afford the time to do this change.
I apology.

Thanks @dspaeth-faber, no need to apologize :) Refactoring is hard, that change generally was really good for Grape's evolution and maturity, even though we have to fix issues as a fallout.

The next evolution would be to create command-Objects instead of filling Hashes. This command chain can then be applied to Endpoints.

@toddmazierski Do you think https://github.com/intridea/grape/pull/1034 closes this?

@dblock:

Do you think #1034 closes this?

No, these settings appear to be handled with InheritableValues, whereas #1034 only touched StackableValues. I'm happy to create a new branch to drop deep_dup usage there, too, if recommended!

I think you should go for it @toddmazierski.


[fortress] Forward Port #909

➡ Forward port #909

https://github.com/ignitionrobotics/sdformat/pull/909
https://github.com/ignitionrobotics/sdformat/pull/919

Forward ports <anti_aliasing> element to ign-fortress.
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

Codecov Report

Merging #922 (7675518) into sdf12 (4d9f8a4) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 7675518 differs from pull request most recent head b0b2b64. Consider uploading reports for the commit b0b2b64 to get more accurate results

@@           Coverage Diff           @@
##            sdf12     #922   +/-   ##
=======================================
  Coverage   87.95%   87.96%
=======================================
  Files         101      101
  Lines       14756    14765    +9
=======================================
+ Hits        12979    12988    +9
  Misses       1777     1777

Impacted Files
Coverage Δ

src/Camera.cc
87.78% <100.00%> (+0.23%)
: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 4d9f8a4...b0b2b64. Read the comment docs.


Greenkeeper/express 4.17.1

Pull Request Test Coverage Report for Build 295

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.039%

Totals

Change from base Build 288:
0.0%

Covered Lines:
38

Relevant Lines:
39

💛  - Coveralls

Pull Request Test Coverage Report for Build 295

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.039%

Totals

Change from base Build 288:
0.0%

Covered Lines:
38

Relevant Lines:
39

💛  - Coveralls


docs: Fixed a Typo in bigint.dart

I fixed a Typo in the documentation of the bit-wise exclusive-or operator.

@googlebot I signed it!

Thanks! CQ running here: https://dart-review.googlesource.com/c/sdk/+/185040

Nit: ^ would be bit-wise exclusive-or, | is just bit-wise or.


factor out ecc signing operations into separate package/repo and replace crypto libraries with saner/smaller ones

if there aren't that many callers, just update steem-js itself.
in that case (of fixing callers), all the encrypted memo support in src/auth/memo.js can be removed, as encrypted memos are not on our support list.


[BUG] Windows not detecting GPU  in 0.6 causing models to crash on training

[x] I have checked that this bug exists on the latest stable version of AutoGluon
[ ] and/or I have checked that this bug exists on the latest mainline of AutoGluon via source installation

Describe the bug
Clean windows install as per instructions or upgrade from working 0.52 conda environment to 0.6  detects that GPU is available to Torch but running a simple test for a tabular training causes an error where it appears that the GPU is not detected causing training to fail.
Expected behavior
Training completes.
To Reproduce
print('Torch GPU available: ',torch.cuda.is_available())  # Should be True
print(torch.cuda.device_count())

Returns:
Torch GPU available:  True
1

But:
import numpy as np
import pandas as pd
from autogluon.tabular import TabularDataset, TabularPredictor
import torch
train_data = TabularDataset('https://autogluon.s3.amazonaws.com/datasets/Inc/train.csv')
test_data = TabularDataset('https://autogluon.s3.amazonaws.com/datasets/Inc/test.csv')
#predictor = TabularPredictor(label='class',verbosity=4).fit(train_data=train_data)
predictor = TabularPredictor(label='class').fit(train_data=train_data,ag_args_fit={'num_gpus': 1})

Returns:
 Specified num_gpus per <Any Model> is more than the total: 0

It works if set num_gpus=0 or run using previous 0.52 conda environment
Screenshots
If applicable, add screenshots to help explain your problem.
Installed Versions
Which version of AutoGluon are you are using?
 Turning interactive mode on.

Additional context
Add any other context about the problem here.

I also had the same error when running tabular on Kaggle.

Hi, this is intended behavior starting ag 0.6 as we updated the logic to control resources.
You are expected to specify num_gpus when calling fit() to enable AG to use gpus.
You can refer to this section of our doc to learn more about it

Thanks for checking @pingusix , that does appear to be some kind of bug!
@yinweisu FYI. Also I thought if num_gpus is not specified, it would automatically use the GPUs available if relevant for the model. Why is it not detecting?

@pingusix and @vnghia I believe this has been fixed in #2536 and is available in AutoGluon v0.6.1 release. Please give this a try and let us know if it works.

Looks like the change has fixed it for me in 0.6.1 on Windows. Thanks.


fix(snownet): use unused channels before reused expired ones

Within each allocation, a client has 4095 channels that it can bind to a different peers. Each channel bindings is valid for 10 minutes unless rebound. Additionally, there is a 5min cool-down period after a channel binding expires before it can be rebound to a different peer.
This patch fixes a bug in snownet where we would have first attempted to rebind the last bound channel instead of just picking the next unused one. In the case of a clock drift between client and relay, this caused unnecessary errors when attempting to rebind channels.
Fixes: #5603.

@conectado Thanks for adding the

Great work! Thanks for fixing that test

Thanks for adding the test!


[TX] Add remaining platform tx to caminoExecutor (spend / lock)

Camino TX / spend adjustment
PR implements all platformTx from utxoset which use getMinimumSpendable.
Instead the JS based UTXO logic we let camino-node / go evaluate the resulting UTXOs.

I was not happy because of all the code changes in UTXO and API, so I reworked platform API:
-> move createTx functions from UTXO into new Builder (which can now either get old or new getMinimumSpendable from interface.
-> Create a Builder class which capsulates the spend call into camino-node.
Pls. @Chriserus review again, I still have to fix some of the examples because of slight param changes


Publish bot by CLI

Describe the bug
I'm following the document at https://github.com/microsoft/BotFramework-Composer/blob/v1.1.1/docs/archived docs/how-to-publish-bot.md
But on both Web and Desktop version 1.1.1 there is no way I can get the command as shown in the document.
Version
1.1.1
Browser

[ ] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[x] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

Hello @tannguyen3, those docs have been archived and are not kept up-to-date. Here is the best source of documentation for publishing via the CLI: https://docs.microsoft.com/en-us/composer/how-to-publish-bot

@tannguyen3 Were you able to get your bot published?

in the the article you suggested, there is no CLI to publish the bot.

hi @tannguyen3 That is correct. Publishing is done from within Composer itself. Are you able to follow the steps in that newer document to get it published?

Yes, i am. Do we have any plan to support CICD for bot composer?

I don't believe so, how would you desire/expect this to work? Maybe if you can give more explanation on what you are desiring, we can give better guidance.
If you are wanting the the declarative resources (.dialog, .lg, .lu, etc) that composer creates, to then be ushered into CICD/bot's pipeline; you should be able to do that. Everything there can be published to the Composer Dialogs directory (or whereever appropriate for your actual bot) of your bot. You can have a separate pipeline for your runtime, if that's even needed.
For example, here is my bot folder:

And here is what I have in the azure web app:

You can do this publishing in any of the typical approaches; az cli, PowerShell, REST or more. It really depends on your needs.

Thanks @dmvtech. Basically I want to setup a CICD pipeline on Azure DevOps to automatically build, deploy and run test for any new changes my team made on bot composer. I'll take a look on the Az Web App to see what we have in there.

Hi @tannguyen3, did you manage to have a CI/CD pipeline in Azure DevOps for this?

Hi @tannguyen3, did you manage to have a CI/CD pipeline in Azure DevOps for this?

@ruifelixpereira how about this? https://github.com/microsoft/BotFramework-Composer/tree/main/runtime/dotnet/azurewebapp/Scripts


I lost one node in 'all_nodes'. How can I add it again?

Description
I lost one node in 'all_nodes'. How can I add it again?
What issue happened
I have 4 ubuntu virtual machines, and I use docker to run a CouchDB container with customized local.ini and vm.args.
Everything was perfect. I got a 3 nodes cluster worked well like this:
{"all_nodes":["couchdb@172.xx.xx.133","couchdb<EMAIL_ADDRESS>"cluster_nodes":["couchdb@172.xx.xx.133","couchdb<EMAIL_ADDRESS>
Last night, one of the machines had something wrong with connection, when I try to ssh connect the machine, it shows 'shell request failed on channel 0'.
So I soft reboot the ubuntu machine, which is one node of the cluster.
Then I found that I lose this node in "all_node"...in other worked well nodes shows like this:
{"all_nodes":["couchdb<EMAIL_ADDRESS>"cluster_nodes":["couchdb@172.xx.xx.133","couchdb<EMAIL_ADDRESS>
In the losing node view like this:
<EMAIL_ADDRESS>"cluster_nodes":["couchdb@172.xx.xx.133","couchdb<EMAIL_ADDRESS>
What measures did I take

First, I tried this command to restart my docker container:
'docker restart my_container_id'
Then I found it doesn't work. I still cannot get the losing node.

I've tried to enable the cluster mode again but the result tells me the cluster is already finished so I cannot add nodes with curl -X POST -H just like the beginning.

Then I tried to use 'curl -X PUT "http://user:passwd<EMAIL_ADDRESS>-d {}' to add the node back. However, the system showed '{"error":"conflict","reason":"Document update conflict."}'.

Could you please help me how to add the node back again?
Thank you so much for reading this.
My Environment

CouchDB version used: 2.3.1 using docker
Operating system and version: ubuntu 18.04 bionic

Hi there,
This is not a CouchDB bug. GitHub is for actual CouchDB bugs only.
If you are looking for general support with using CouchDB, please try one of these other options:

The user mailing list. Signup instructions are here
The Slack/IRC chat room. Joining instructions are here


Geolocation Provider

Hi,
Is it possible do develop a custom "Geolocation Provider" (or Location Sensor)?  By that I mean I'm looking for a mechanism to feed data from an external source to the UWP Geolocator.
I'm seeing a "Microsoft Visual Studio Location Simulator Sensor" in my Device Manager, so I'm tempted to think this is possible.
Please forgive if the question hasn't been addressed in the appropriate forum.  Just can't seem to find the right search term for this one. :)
Thanks,
Rene

Driver samples are not kept in this repo. You can find a sample location sensor provider on MSDN. There is also an entire repo for driver samples.


Why do the results disappear?

Why do the results disappear?

I can open this link. I suggest changing the network or computer and try to open this link again.

Thanks bro, your work is great


Fix pipeline build (S2I)

Add yarn install to assemble
Move build step out of post-install in package.json
Update Dockerfile

@sutherlanda please review. I think this might fix the pipeline build

@asanchezr, this seems to have fix the pipeline deployment issue.  Nice work!

👍


Minor issue with editor screen

On MacOS Catalina on a high DPI screen the left right buttons that activate when there are a number of buffers open get "clipped". See attached [picture.]

Thanks for reporting.
@goanpeca, any chance you can take a look at this one?

@goanpeca, do you think a fix for this could be ready for 4.1.4?

do you think a fix for this could be ready for 4.1.4?

Not sure, I tried a quick fix and did not work, will look into this after I finish the scrollbar issue.


Event delay when using material components inside web components

What are you trying to do?
The events in the dialog are delayed when material components are used inside web components.
eg. When using mat datepicker inside a kendo-ui grid, it works fine when we try to open it from a normal flow but when the same datepicker is accessed from a web component it misbehaves and
does not open sometimes and sometimes it just appears as an empty white rectangle.
In some cases, eg while clicking on a close icon inside a matDialog it takes some 5-10 seconds to close the dialog when opened inside a web component otherwise it works properly.
What troubleshooting steps have you tried?
Initial thought was that this might be an issue with the event detection in angular as we are running some code outside zone.js so tried to run this outside zone as well but did not work. Not sure about the exact issue.
What are you seeing that does not match your expectations?
Would just need your insight to figure out if this is a known issue or something that we might be doing wrong.
Environment

Angular: 8
CDK/Material: 8.0.0
Browser(s): google chrome
Operating System (e.g. Windows, macOS, Ubuntu):  windows

This issue doesn't have enough information for the team to investigate. Please include:

A reproduction of the issue (preferably via StackBlitz)
Any relevant errors and stack traces
Any relevant details about your build environment

That said, it sounds like the problem is that something is running outside of the zone that should be in the zone.


docs: Updated basic setup docs

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
N/A
Fixes: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

LGTM except the commit message doesn't follow our guidelines. Please update to type(libraries): message.


[BUG]: inconsistent pointers to helm chart ocis

Attempting to download the helm chart oci we receive the following
> helm pull oci://public.ecr.aws/zenml/zenml-server --version 0.44.3 \
        --untar --untardir ./charts/
Error: unexpected status from HEAD request to https://public.ecr.aws/v2/zenml/zenml-server/manifests/0.44.3: 400 Bad Request

We also tried oci://public.ecr.aws/zenml/zenml as it appears differently on artifacthub versus the docs website.
Based on the workflow you’re using to publish the chart oci,
https://github.com/zenml-io/zenml/blob/18824ef77df44f3dee4ba1e7f61d767db8ee58c2/.github/workflows/publish_helm_chart.yml#L48
I guess it should be zenml and not zenml-server, but both produce the same error.
These are executed following login with
> aws ecr-public get-login-password \
     --region us-east-1 | helm registry login \
     --username AWS \
     --password-stdin public.ecr.aws
Login Succeeded

> helm version
version.BuildInfo{Version:"v3.13.0", GitCommit:"825e86f6a7a38cef1112bfa606e4127a706749b1", GitTreeState:"clean", GoVersion:"go1.21.1"}

It’s not hard to workaround by downloading from github, just outside the current docs.

Hi @cameronraysmith, thanks for pointing out the bug in the docs. You're right, it should be zenml and not zenml-server. As for the error, I'm not able to reproduce it unfortunately, using a different account and multiple roles. I'll do some more testing and ask someone from the community to try it out too.

This was an issue in v3.13.0 of helm that was fixed in v3.13.1.
https://github.com/helm/helm/issues/12483
https://github.com/helm/helm/commit/0ac78941abfed981a47f263fa59931e9123f73a1

https://github.com/zenml-io/zenml/pull/1942 marks the resolution, but feel free to close it without merging if it's more convenient.

Thank you so much for the analysis and the PR @cameronraysmith!


Perception module involves tracking ?

env : ubuntu 18.04, apollo v6.0
Hello I run the lidar perception module by using this page.(https://github.com/ApolloAuto/apollo/blob/master/docs/howto/how_to_run_perception_module_on_your_local_computer.md)
Below is the result video. (I got raw data with hesai64)
https://user-images.githubusercontent.com/79086973/123418744-60574280-d5f4-11eb-87c9-8d15408fc305.mov
In step 8, I run
cyber_launch start /apollo/modules/perception/production/launch/perception_lidar.launch
In this case, perception_lidar.launch file has a tracking module?
If not, how can I run the perception module with tracking ? (I want to run lidar perception with tracking)

The lidar module is default with tracking.
You can confirm it by checking the ID on the obstacle. If the ID does not change, it means it is work. If the ID of the same obstacle changes frequently, it means it is not work.
Your video resolution is too small to see clearly!

Thanks for the reply.
How to check the obstacle id?

Thanks for the reply.
This video is part of the full video. There are quite a few cases where the bounding box is recognized and then disappears and then reappears, but the ID value is changing every time it happens. It seems that tracking is not being applied in this case, how to solve it?

This is the cyber_monitor for the perception module video above.

Why the /apollo/sensor/hesai64/PointCloud2 frequency is doubled.
I think the tracking is already in effect, because you can see that the id of the obstacle that has not disappeared is constant.
So the remaining question is why the obstacle disappears?

https://user-images.githubusercontent.com/79086973/123597602-80724600-d82e-11eb-997e-5c98d24e49d0.mp4
This video is raw data of using hesai lidar.
When I saw raw data, I can't find any problem in this case.
In generally, how obstacle disappearing problems are happened?

Why the /apollo/sensor/hesai64/PointCloud2 frequency is doubled.
I can't understand this comment. Is that any prob?

/apollo/sensor/hesai64/PointCloud2 frequency is normally 10HZ, but the frequency in your screenshots is 20HZ. The strange thing is that if the frequency of /apollo/sensor/hesai64/PointCloud2 is 20HZ, then the frequency of /apollo/perception/obstacles should also be 20HZ
You should check the frequency and find out if there is an error message data\log

How can I check and change the frequency ?
(You should check the frequency and find out if there is an error message data\log)
Also How can I check an error message data\log ... ?
As far as I can see, there was no error message when running the perception module.
I'm not familiar with Apollo yet, so please understand that the question is inexperienced...

From the above information, I can’t tell what the problem is, you need to try to analyze it

Thanks for your reply !

@daohu527
Through lidar settings, /apollo/sensor/hesai64/PointCloud2 frquency was changed to 10HZ.

But the frequency of /apollo/perception/obstacles is about 8Hz, is this difference causing the problem of objects disappearing?
The problem that the bounding box of an object is drawn well, disappears right next to the vehicle and then reappears. Do you know the common cause?

https://drive.google.com/file/d/1TObm97yCTQ60ODfKXIRuTcfvVa-bpg5P/view?usp=sharing
https://drive.google.com/file/d/1TObm97yCTQ60ODfKXIRuTcfvVa-bpg5P/view?usp=sharing
This is a video of running the perception module on the raw data received by hesai pandar64.
You can see the problem that the bounding box of an object is drawn well first, but disappears right next to the vehicle and then reappears.
If you have a solution, please let me know, I would appreciate it.

I don't know what the problem is, why is the system localization frequency 200? Are you sure you only activate Apollo, or additional changes.
I guess there are duplicate news releases, and the current frame is overwritten
/apollo/canbus/chassis
/apollo/localization/pose

Ok, Thanks for your reply.
I try to analyze the prob!

hello!@rlaehddls9
I run cyber_launch start /apollo/modules/perception/production/launch/perception_lidar.launch, but the process is stuck for several hours. As shown in the figure, i have to close this process. Do you have this problem? How long did your process (Perception_lidar.launch) take？
Thanks！

From the picture, there is a problem with your map.
The lane 2563a_1_-1 is to narrow to pass a car. At the same time, I suggest you open a new issue. If there are different problems, it is convenient for others to solve the problem.

Hello @zhangchuang-zc
I had same prob such as your image. But I changed perception_common.flag's obs_enable_hdmap_input=false then error message was disappeared. Also I can run the perception_lidar.launch without hdmap. In Dreamview, I can see the running module at Navigation mode.

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


Elaborate on snapshot collision TODO

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#445  👈 (View in Graphite)
#436 : 1 other dependent PR (#440 )
#435
#434
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Me using Graphite L


Unexpected token in combobox.js

I've npm install react-autocomplete. but when I gulp, I get this:
events.js:72
throw er; // Unhandled 'error' event
^
Error: Parsing file /Users/oriharel/oriApp/sixdegrees/node_modules/react-autocomplete/lib/combobox.js: Unexpected token (358:6)
at Deps.parseDeps (/Users/oriharel/oriApp/sixdegrees/node_modules/browserify/node_modules/module-deps/index.js:418:28)
at fromSource (/Users/oriharel/oriApp/sixdegrees/node_modules/browserify/node_modules/module-deps/index.js:361:48)
at /Users/oriharel/oriApp/sixdegrees/node_modules/browserify/node_modules/module-deps/index.js:356:17
at ConcatStream. (/Users/oriharel/oriApp/sixdegrees/node_modules/browserify/node_modules/concat-stream/index.js:32:43)
at ConcatStream.EventEmitter.emit (events.js:117:20)
at finishMaybe (/Users/oriharel/oriApp/sixdegrees/node_modules/browserify/node_modules/readable-stream/lib/_stream_writable.js:460:14)
at endWritable (/Users/oriharel/oriApp/sixdegrees/node_modules/browserify/node_modules/readable-stream/lib/_stream_writable.js:469:3)
at ConcatStream.Writable.end (/Users/oriharel/oriApp/sixdegrees/node_modules/browserify/node_modules/readable-stream/lib/_stream_writable.js:436:5)
at DuplexWrapper.onend (/Users/oriharel/oriApp/sixdegrees/node_modules/browserify/node_modules/readable-stream/lib/_stream_readable.js:537:10)
at DuplexWrapper.g (events.js:180:16)
at DuplexWrapper.EventEmitter.emit (events.js:117:20)
at /Users/oriharel/oriApp/sixdegrees/node_modules/browserify/node_modules/readable-stream/lib/_stream_readable.js:934:16
at process._tickCallback (node.js:415:13)
It looks like I'm expected to react transform the combobox jsx to js. but it's in the node_modules.
So, is this the case and if so, how to add the task to do it?

:+1: having the same issue here

Same issue here

Don't waste your time guys. This is a dead project. get react-select instead.

Thanks! That react-select looks good.


Clean up/simplify delegated logout flow

This PR includes two changes we've had to make locally:
Prevent delegated logout from firing before logout has been confirmed
This is a bugfix: because delegatedAuthenticationClientLogoutAction is attached to terminateSession as an entry action, it will fire before any checks have even happened for logout confirmation. This PR patches this behavior by implementing the same check as is used in TerminateSessionAction, but this may not be the ideal approach - for example, it might be more appropriate to attach the delegated logout action to finishLogout, or one of the end states of the logout flow.
One interesting avenue to consider would be trying to get a delegated IdP to send the user back to CAS, allowing jumping off in the middle of the logout flow, and then picking it back up. This would not automatically solve the problem of skipping the logout confirmation page, but it might make it easier to choose a new webflow state to attach the delegated logout action to. In OIDC, this could be done with post_logout_redirect_uri  and related settings.
Tests
This should be easy to test, but I do not know how to modify the CAS properties within a test.
Bypass Pac4j SessionStore when locating a UserProfile during logout
This may be somewhat more contentious. We have encountered issues where the Pac4j SessionStore eventually becomes inaccessible, likely due to a cookie misconfiguration, leading to a failure to log users out of delegated clients. While debugging this behavior, it struck me that the method of locating the relevant UserProfile in DelegatedAuthenticationClientLogoutAction need not be dependent on the Pac4j SessionStore at all - the UserProfile is tracked in the Authentication bundle. Retrieving the profile from the authentication, rather than from the JEEContext, is the change made by my second commit.
Possible interactions with Pac4j APIs
Due to Pac4j's approach of bundling all possible information into a CallContext object, it's a little hard to evaluate what is or is not needed to generate the logout action. I have inspected the following LogoutActionBuilders:

CasLogoutActionBuilder uses only the WebContext part of the CallContext
GoogleLogoutActionBuilder also uses only the WebContext
OidcLogoutActionBuilder only interacts with the SessionStore to unset a requested URL, and only in an error state
SAML2LogoutActionBuilder is substantially harder to evaluate but I am fairly certain that it does not interact with the SessionStore

Tests
This also seems like it should be easy to test, but so far I am not sure what to provide RegisteredServiceTestUtils::getAuthentication for an AuthenticationHandler, is required in order to be able to set an appropriate Credential.
Let me know what you think of these changes. Happy to pursue some tests but I would appreciate some guidance, e.g. existing tests that make use of some of the same APIs.

🙃

@mmoayyed There is no call for a snarky bot reply with an even snarkier image macro to immediately shut down a PR - not even prompt for test cases, good lord - on a project that appears to receive 1 or 2 contributions per month from people not directly affiliated with it.


Put sources in src/

To be consistent with other php-http repositories I would suggest to put the source files in src/ and tests in tests/. It is already 11 files in the repository root and only one file belongs to the bundle.
This is bundle which you might want to treat differently, that's why I ask a questions before sending a PR.
Do you want this change?

that would be against symfony best practices. i would prefer to follow
the symfony conventions here. afaik there are no technical reasons that
would prevent us from moving the code into a src/ folder - but i think
its easier to understand following the symfony bundle conventions.

Okey, I must have missed that in the best practices document. You are right then, we should not put sources in src/


Seemingly imports g874.1-model

Class list export indicates import of CommonDataTypes/g874.1-model.uml#3B8BC86A0312
Presumably a wrong datatype is applied somewhere.
Wrong datatype to be identified and replaced by one of either UML or ImplementationCommonDataTypes.

Misunderstanding due to wrong class list.

Misunderstanding due to wrong class list.


Add Copy URL and Open Logs to desktop app menu

Perhaps we could also include Settings that opens up the Settings page in Livebook.

Yup!


Speed up importing antropy

Create a file called import.py with the single line import antropy. On my machine (Linux VM), this takes at least 10 seconds to run.
Using pyinstrument tells me that most of the time is spent importing numba. Is there any possibility of speeding this up? Seems like this is a known issue with numba, though: see e.g. https://github.com/numba/numba/issues/4927.
$ pyinstrument import.py

  _     ._

To view this report with different options, run:
    pyinstrument --load-prev 2021-06-17T16-36-28 [options]

Hi @jftsang,
Unfortunately, I don't think there's anything that we can do to fix this. Numba allows to greatly improve the computation time of some functions in antropy, and the small cost of that is a longer import time. I think even 10 seconds is definitely reasonable. Let's hope that future versions of Numba will improve this.
Thanks,
Raphael

Hi @raphaelvallat, just another thought - would it be possible to make the compilation use of the @jit-ted functions optional? I wonder what the tradeoff would be depending on the scale of the data: for a short array is there much to gain beyond plain old numpy?
Jonny

Hi @jftsang,
I believe you can set up the export NUMBA_DISABLE_JIT=1 environment variable to disable JIT (see https://numba.pydata.org/numba-doc/dev/reference/envvars.html).

That seems to work well. I shall do some profiling to estimate the scales at which @jit-ting becomes worthwhile.


fix: node module resolution for react-native/normalize-color

Summary
I attempted to use this package in a node module but node module resolution will fail to resolve it properly.  A package.json without a main field will default to ./index.js, but the default file here is ./base.js.
For future readers; even after this is merged it'll still not work with node tools because flow types are present in the file.

Changelog

[General] [Fixed] - Added main field to package.json of @react-native/normalize-color
Test Plan

eyeball'd it

would be nice to remove flow as well, or move them to an optional type definition

would be nice to remove flow as well, or move them to an optional type definition

Yeah, we could move them to a separate library definition file.

I just published<EMAIL_ADDRESS>which works using Node.js by splitting out Flow type definitions into an accompanying library definition file. I also included the rename from base.js to index.js to make usage more idiomatic.
The internal commit will be landing soon and sync'd to the repository for you to see the changes. But the published package is ready for consumption immediately. I'll be closing this PR because it has been superseded.

See https://github.com/facebook/react-native/commit/65e58f26e1fbd06b1ae32e2ab3a2616c8eef08e0 for the changes which went into<EMAIL_ADDRESS>


bootstrap file changed to reflect new kernel

@RackHD/corecommitters @tldavies @VulpesArtificem @rolandpoulter  this is one of a 2 part PR (also in https://github.com/RackHD/on-imagebuilder/pull/31) to support the new micro kernel.

@DavidjohnBlodgett also need some doc/build updates at https://github.com/RackHD/RackHD/blob/master/packer/ansible/roles/images/tasks/main.yml#L15-L19 and https://github.com/RackHD/docs/blob/25c037d64ecd54b7e4960efe7e976923c559dd2f/docs/rackhd/ubuntu_package_installation.rst (search for 3.16).

Please don't merge until ^^ are updated.

:+1: after update with @benbp 's comments

Still :+1: with a refactor suggestion.

*** BUILD #517 ***

*** BUILD #518 ***

*** BUILD #519 ***


Pumpkins cannot naturally generate on spooky grass

Probably need to add the block to some tag

This has been fixed by recent PRs.


The cookies for the stats page aren't persisted for a long time on mobile Safari

After a couple of hours I had to reenter a token. Chrome on my Mac not the case, though it hasn't quit since then, so maybe they're by default as long as the browsing session.

🙏


Sync bug

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

Comparison is base (4c5747e) 73.70% compared to head (706efc9) 73.66%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #398      +/-   ##
==========================================
- Coverage   73.70%   73.66%   -0.05%
==========================================
  Files          49       49
  Lines        5755     5760       +5
==========================================
+ Hits         4242     4243       +1
- Misses       1220     1223       +3
- Partials      293      294       +1

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


Build guide?

A guide on how to make this build successfully would be great!

Hey devs, have a look at my fork (https://github.com/ramonmelo/HumbleNet), I've added some instructions to building the library. Still a WIP, but it's a start.

@ramonmelo thanks.. I've included your build instructions


Fix misleading doc

Fixes https://github.com/istio/istio/issues/30840
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
1c1ede99f715fb57ebb9da3c155d8e0a5537f558
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Trades clearing everytime you edit them

I placed down a shopkeeper NPC and sneek+right clicked to edit trade. Then when just right clicking they show up as normal. Then one you shift+right click again all the trades get lost and when you exit it saves with no trades if none where inputted.
This is on 1.20.2

Wrong project?


Missing the ability to create a label from another label (context)

Your null label https://github.com/cloudposse/terraform-null-label module allows "context" to build powerful tagging strategies but this module is missing that feature.
I tried using null-label module but it was missing a feature this label module had, I think it was to force lowercase.
Kinda sucks to not have them be somewhat even in features since they are really similar. Will make this issue in case I want to PR a fix in the future.

Yes please

@shadycuz we're moving away from this module as terraform-null-label provides the same functionality and is actively maintained. See #29 for more details.


Modernize for 2024

because: This repo has 5 outstanding security vulns and is public.
this commit: Brings the codebase into 2024 with modern styling, libraries and testing.
note: while prettier is not installed here, it has been used for consistent formatting to align with our other repos.

Hi everyone! Nice to see I'm still making a difference at ActBlue, 5 years on 😉


[Bloomberg] ERROR: Unable to extract id

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.11.04. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.11.04

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', 'http://www.bloomberg.com/news/videos/2016-11-07/steve-ballmer-bloomberg-studio-1-0-11-06']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2016.11.04
[debug] Python version 3.4.4 - Windows-10-10.0.10240
[debug] exe versions: rtmpdump 2.4
[debug] Proxy map: {}
[Bloomberg] steve-ballmer-bloomberg-studio-1-0-11-06: Downloading webpage
ERROR: Unable to extract id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpqdvyr8vw\build\youtube_dl\YoutubeDL.py", line 694, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpqdvyr8vw\build\youtube_dl\extractor\common.py", line 357, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpqdvyr8vw\build\youtube_dl\extractor\bloomberg.py", line 52, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpqdvyr8vw\build\youtube_dl\extractor\common.py", line 660, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
<end of log>

Description of your issue, suggested solution and other information
Downloading Bloomberg Studio videos results in aforementioned error. Tried rtmpdump 2.3, 2.4 and none at all, same result. I think this worked before, so I hope it can be fixed?

Could you run this command and upload all *.dump files?
youtube-dl -v --write-pages http://www.bloomberg.com/news/videos/2016-11-07/steve-ballmer-bloomberg-studio-1-0-11-06

Hi Yan, thanks for your reply.
To my surprise, running the same command again worked flawlessly today, although I made no other changes.
Maybe it was a server-side issue?
Thanks for your help though!


New Liqo Release

Description
This PR includes a major refactoring of Liqo, including:

fully declarative approach on all functionalities (install, peering, offloading, etc..)
a new division in 3 main modules: networking, authentication, offloading
modularity and independence between the 3 modules (e.g., the possibility to turn off individual modules)
new networking fabric
possibility to implement new networking technologies
new authentication workflow
possibility to have multiple virtual nodes targeting the same remote cluster, allowing more flexibility and giving the possibility to target specific node pools on the remote cluster
docs: new structure, improvements, new pages (e.g., advanced usage examples), fix deprecated features, and a lot more

/test

/test

/test