feat: resolve domains in enode strings

Introduces a type that should properly deserialize bootnode enode strings with domains, since the current NodeRecord type does not do this.
fixes https://github.com/paradigmxyz/reth/issues/8187

@Rjected I looked into this unit test failure
thread 'dns_node_record::tests::test_url_parse' panicked at crates/net/types/src/dns_node_record.rs:130:9:
assertion `left == right` failed
  left: DNSNodeRecord { host: Domain("<IP_ADDRESS>"), tcp_port: 30303, udp_port: 30301, id: 0x6f8a80d14311c39f35f516fa664deaaaa13e85b2f7493f37f6144d86991ec012937307647bd3b9a82abe2974e1407241d54947bbb39763a4cac9f77166ad92a0 }
 right: DNSNodeRecord { host: Ipv4(<IP_ADDRESS>), tcp_port: 30303, udp_port: 30301, id: 0x6f8a80d14311c39f35f516fa664deaaaa13e85b2f7493f37f6144d86991ec012937307647bd3b9a82abe2974e1407241d54947bbb39763a4cac9f77166ad92a0 }

The enode prefix in the URL leads the parser to use parse_opaque() on the host string which always returns a Host::Domain whereas we expect a Host::Ipv4 here.
From what I can see our only option is to post-process the url.host after that parsing has occurred. Would be better if we could control the url parsing logic but haven't seen a way to do that in our favour here.

not super obvious to me why we need this.
this basically replaces the Noderecord usage, why?
what's the progress here?

I have only just started contributing to reth so please take what I say with a grain of salt.
My understanding is that the current impl of NodeRecord does not support domain names, only IPv4 and IPv6 addrs:
pub struct NodeRecord {
    /// The Address of a node.
    pub address: IpAddr

This PR is adding support for domain names. @mattsse are you suggesting we should be adding this support with the existing NodeRecord struct instead of DNSNodeRecord? Given that domain name support will be a requirement for many L2 networks (like the ones I work with).
Apologies if I have misinterpreted anything 🙏

gotcha,
please document all of this because having like 4 different node variants is super confusing

@Rjected what's the path forward here?

@Rjected what's the path forward here?

sorry, have been occupied with other things, need to review https://github.com/paradigmxyz/reth/pull/8358, and then will polish this up and add a ton of docs


Entity outlines

Added the Emscripten Bindings to the IndoorrMapEntitiyInfromation
Adding the Outline data to the indoor_map_entity

Branch already merged -- closing.


Undefined index: name when TranslatableBootForm::open()->action('/url') is called

When I open my form adding attributes I get a FatalErrorException: Method Propaganistas\LaravelTranslatableBootForms\TranslatableBootForm::__toString() must not throw an exception.
Calling TranslatableBootForm::open()->action('/url')->render() I get Undefine index: name in propaganistas/laravel-translatable-bootforms/src/TranslatableBootForm.php line 403. I've added dd($this->arguments()) before the return and I get an empty array.
Opening the forms with no attributes or using BootForm::open()->action('/url') directly works perfectly.

Thanks for reporting! That should do the trick. A new release is rolled out as well.


Require the view field in the bulk (CSV) import to avoid the importer failing the import

Versions
NetBox Version: 4.1.4
NetBox DNS Version: 1.1.4
Python Version: 3.11 (Debian Bookworm Stable)
Describe the bug
The bulk (CSV) importer will fail the import if the view column is omitted.
To Reproduce
Steps to reproduce the behavior:

Go to the record importer
Navigate to the data importer
import data with only the columns of "zone,name,type,value,status"
See error

Expected result
The importer fails the task when using the headers of "zone,name,type,value,status". But works when providing the columns of "view,zone,name,type,value,status"
Actual result
The listed required columns are "name,name,type,value". The importer will fail, if the view column is omitted with the message "object not found".
Code Examples
(Requires the view to exist prior to import, view_name is case-sensitive)
zone,name,type,value
example.com,test1,CNAME,test.othersubdomain.example.comg

view,zone,name,type,value
Test,example.com,test1,CNAME,test.othersubdomain.example.comg

Looked around in the code-base. Was unable to locate the function/code-block needed to be edited to require the view field in the bulk importer.

It's not really necessary ...
We can apply the following logic:

Zone names are unique -> no view needed.
Zone names are not unique -> view needed to disambiguate.

I did that recently for a different (as of yet non-public) plugin.

Zone names are unique -> no view needed.
Zone names are not unique -> view needed to disambiguate.

I did try that. Even if the zone-name is unique (within my installation). The bulk (CSV) importer will fail the import task with error "no object found <zone_column_name>"

zone,name,type,value
zt,test1,CNAME,test.example.com

Sidenote: .zt is used by the ZeroTier VPN solution. Not relevant for the overall discussion. Just a note if anybody asks.

Requiring view generally ist clumsy as it enforces using it for importing even if the environment does not have any non-default views.

It is indeed clumsy. I do not disagree. On the other hand. It makes the use of the bulk importer painful. As the error msg presented (by Netbox) is misleading, IMHO.

I believe you that you tried ... that's why it's a bug :-)

I can import the (sliightly changed) example you provided without a view and without any problem:
zone,name,type,value
example.com,test1,CNAME,test.othersubdomain.example.comg

What error are you seeing?

zone,name,type,value
zt,test1,CNAME,test.example.com

works as well after adding the zone zt.

I think now I could reproduce it:

Define a zone zt in view internal
Define the same zone in view external(the view and zone names don't matter as long as the views are not default and the zone names are identical)
Now try to import

zone,name,type,value
zt,test1,CNAME,test.example.com

Resukting error message: "Record 1 zone: Object not found: zt"
Is this correct?

Bottom line:

If you don't specify a view, the default view is assumed
If you're not working with the default view you need to specify the view name
The error message is ... well, it could be better. But if you can confirm the above, there is no bug.

I have now tested your examples and a couple of others manually and cannot reproduce any errors in handling imports. Your problem was probably caused by trying to refer to a zone outside the default view without specifying the view, which does not work (nor is it supposed to work). To avoid that kind of error I improved the description of the view field in the import form
Apart from that I improved the error messages a bit - there's unfortunately not much space for improvement, e.g. I can't refer to the name of the view within the invalid_choice error for a zone.
I also added a section about importing records to the documentation.

Let me know if you have issues beyond what I outlined here, and if so please specify exactly how to reproduce them - as you can see it was a bit of as guess, seek and hunt game (and I hope I covered everything).

Since there seems to be no further input I will close this issue no. If you have any additional information you're welcome to re-open it.


about wgan's loss function

according to the wgan paper , when training the critic, we need to maximize Er(f(x)) - Eg(f(x)), and when training the generator, we need to minimize Er(f(x)) - Eg(f(x)), that is to say minimize - Eg(f(x)), but in the code, wgan's loss use  self.discriminator_loss = tf.reduce_mean(logits_real - logits_fake) , and self.gen_loss = tf.reduce_mean(logits_fake) , so I'm confused.

what I'm confused is whether use negative, because according to tensorflow's API , we use optimizer.compute_gradients(loss), the loss is what we what to minimize, but the loss in WGAN code is to maximize, so I'm confused

I have the same question here.
I think the one should negate the discriminator loss

I ask the question on Reddit, the website is: https://www.reddit.com/r/MachineLearning/comments/5qxoaz/r_170107875_wasserstein_gan/
and ajmooch answer me, this is what he said:  'I suspect this is just a case of sign flippage--if you just switch conventions so that your goal is to make the critic output highly negative values for real samples and highly positive values for fake samples (and change the generator's objective accordingly) then it still works the same; since there's no nonlinearity we can flip those signs with impunity'.
I think maybe he is right, if we change the sign of the discriminator loss, we are just minmax W(pr,pg) or minmax W(pg, pr), so if we keep the same sign with the generator loss, we are doing the same thing.

ah you are right. I didn't notice the g_loss is reduce_mean(fake) instead of -reduce_mean(fake)

Duplicate of #6

Well simply understanding is according to WGAN loss D should try to maximize the distance between Real and Fake Data. This will be completed by the logits given by the discriminator neural net. So somehow we need to make this tractable. So what the Generator does is try to reduce the minus value of logits generated for gen output from the discriminator. So basically Gen will try to generate more and more plus values. But discriminator is trying to make logits for gen output more minus and maximize the plus value of discriminator logit output for real dat

I have a problem: the loss of the generator is descending from positive to negative but I have negated the generator loss. Is the negative value of the generator loss bad for the optimization?

@nicholasly I also met with the same issue. The WGAN was trained for timeseries data. The D and G loss all become negative after several epoch. What should I do? Have you figure out why? Thx

I currently think a negative value is acceptable when considering the weight changes in the network during the backpropagation. But I have not figured out how to monitor the training if the loss can converge to the value samller than zero.

@nicholasly But why this could happen? Maybe I use the timeseries data with different length and feature numbers? Or the 2D CNN structure is not working under my data circumstance? lol...I am tring to figure out

@nicholasly Have you figure out why the loss of the generator is descending from positive to negative? Is that normal?

@LexieYang @nicholasly
Have you guys figure out why the Wasserstein loss can be negative? Even thought the definition of the Wasserstein loss makes it can be negative, but what's the negative value stand for?


[Chore] Fixture data

This PR is the kick-off towards streamlining the development process, easing external collaboration.
The main script, a Django command refresh_fixtures, generates random data so to populate the DB for development and dumps it to a JSON fixture file.
Together with the script, the fixture file and development DB were versioned, so anyone who clones the project can start using it right away. As new models are included, the fixtures script needs to be updated, along with the DB file.
Some other small changes: the DB file for development is db-dev.sqlite3, so to make sure there is no production DB file overwritten. DEBUG is now False by default, and the ALLOWED_HOSTS is * (meaning any hosts) when on debug mode i.e. during development.

Hi! Yes, the names are completely random, first names and surname shuffled from name generators lists, and e-mails are nonexistent (given the way they are generated). Having close-to-real data, such as the names and e-mails, helps with debugging and design concerns.
During development (i.e. DEBUG=true), in any case, the e-mail service must be routed to the developer's e-mail, parametrized by an environment variable.


cant connect

in default setting when i run it and use proxy cant connect to my vps proxy why?please fix it

in english pleaseeeeee


Remote service does not respond to _accessibilityMachPort

Hi,
Is this something we can address from your plugin?
I would be happy to send a pull request if you have any idea on how to fix it.
Many thanks.
Bug Information
Version Number of Plugin: 5.1.0
Device Tested On: iPad Pro 12
Version of VS: VS Mac 7.3.3 (build 17)
Version of Xamarin: Xamarin.Forms -> <IP_ADDRESS>934
Versions of other things you are using:
BTProgressHUD -> <IP_ADDRESS>
LiveXAML -> 2.1.16
Microsoft.AppCenter.Analytics -> 1.3.0
Microsoft.AppCenter.Crashes -> 1.3.0
Plugins.Forms.ButtonCircle -> 1.3.9
Refit -> 4.1.0
SQLiteNetExtensions -> 2.0.0
SQLiteNetExtensions.Async -> 2.0.0
Xam.Plugins.Messaging -> 5.1.0
Xamarin.Forms -> <IP_ADDRESS>934
Xamarin.TestCloud.Agent -> 0.21.3
SegmentedControl.FormsPlugin -> 1.3.4

Steps to reproduce the Behavior

Install the latest version of plugin
Copy-paste code snippet
Run application and try to send email

Expected Behavior
Mail application open successfully without noises.
Actual Behavior
Mail application open with the error AX Exchange error: Error Domain=Accessibility Code=0 "Remote service does not respond to _accessibilityMachPort" UserInfo={NSLocalizedDescription=Remote service does not respond to _accessibilityMachPort} in console every 0.2 seconds.
Code snippet
AppDelegate:
#if ENABLE_TEST_CLOUD
            Calabash.Start();
#endif

Actual sender:
var builder = new EmailMessageBuilder()
    .To(to)
    .Cc(cc)
    .Bcc(bcc)
    .Subject(subject)
    .Body(body);

CrossMessaging.Current.EmailMessenger.SendEmail(builder.Build());

Screenshots

@gabrielrobert Just to clarify, this is happening when running your app via Xamarin Test Cloud service?

@gabrielrobert seems to be related to this

@cjlotz Exactly. Have you ever heard of this problem? Is there something we can do in your (very useful!) library?
Just disabling Calabash is not a solution, unfortunately.

Unfortunately this is outside the scope/control of the messaging library.  The enabling/disabling of Calabash lies in the hands of the app developer, not in hands of a 3rd party library.  From the Xamarin docs they mention that you should only set the ENABLE_TEST_CLOUD compiler directive for Debug builds and not Release builds as the app will be rejected.  Are you doing this?

This is what I expected, thank you for your time @cjlotz.


Drag slider precision preference

I saw this on the to-do list and attempted to implement it myself into the AppPreferences class, but it became a mess since I had to change the parameters for a lot of functions just so I can access that value. Not sure of a good solution.

I usually round numbers to the hundredths or thousandths place when doing things like converting RGB values from 0 - 255 to 0 -1 but that's obviously a personal preference. I honestly don't think there's gonna be a definitive Goldilocks precision value.

This is purely a visual thing. The DragSlider is still storing the full precision if you edit the value by clicking on it, which was fixed recently. The reason to adjust the number of displayed decimal digits is to account for different widths for the control rather than a user preference.


acs kubernetes get-credentials - [Errno 101] Network is unreachable

Environment summary
CLI Version: azure-cli (2.0.27)
OS Version: Ubuntu 16.04
Shell Type: bash
CLI COMMAND THAT CREATED ERROR
az acs kubernetes get-credentials --name=$NODE --resource-group=$GROUPNAME
PROBLEM DESCRIPTION
I've created a kubernetes cluster using azure-cli.  Running az acs list  produces JSON output  that includes the line "provisioningState": "Succeeded", which indicates the cluster was successfully created. I am now trying to deploy a docker application into the cluster but cannot seem to get kubectl configured locally on my Ubuntu machine.
To install kubectl, I ran sudo az acs kubernetes install-cli which produced the message: Downloading client to /usr/local/bin/kubectl from https://storage.googleapis.com/kubernetes-release/release/v1.9.3/bin/linux/amd64/kubectl  which looked good and I have verified the kubectl executable is present in /usr/local/bin.
The problem occurs when I run the command to download credentials and configure Kubernetes CLI to use them.  Here is the command:
az acs kubernetes get-credentials --name=$NODE --resource-group=$GROUPNAME
Here is part of the error message:
[Errno 101] Network is unreachable
Traceback (most recent call last):
File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 194, in invoke
.
.
.
File "/opt/az/lib/python3.6/site-packages/paramiko/client.py", line 338, in
retry_on_signal(lambda: sock.connect(addr))
OSError: [Errno 101] Network is unreachable
Any thoughts on this are greatly appreciated

@tonybyorick the az acs get-credentials command expects to be able to ssh to the master node. This looks like for some reason the masterProfile -> fqdn address isn't reachable from where you are running the CLI. Can you ping or SSH to that address outside of the az CLI?
az acs list -o jsonc | grep fqdn # find the fully-qualified domain name of the master node
ping maboersmac-maboersm-e220e2mgmt.eastus.cloudapp.azure.com
ssh<EMAIL_ADDRESS>

@mboersma , Thanks for your reply..... get nodes is now working, but not sure why....
~$ kubectl get nodes
NAME                    STATUS    ROLES     AGE       VERSION
k8s-agent-596de907-0    Ready     agent     1d        v1.7.7
k8s-master-596de907-0   Ready     master    1d        v1.7.7

get nodes began working after I copied the file used as the argument for --file ( --file=$XDGDIRECTORY/kubeconfig.yaml ) to ~/kube/config.
So, seems like kubectl expects output from get-credentials to reside in ~/kube/config regardless of the --file parameter to get-credentials.  Or maybe there is a command that sets kubectl to look in a specific folder for kubeconfig.yaml?


Implement interchangeable persistence layer

Several ideas for data providers:
From https://twitter.com/LukasDonSchmidt => use invisible items on the board
From https://github.com/hojak => Write the data in the comment fields
From Falk and Michael Use a KeyValue Store on the internet

Implement persistence option "hidden objects on the board"

Implement persistence option "Write the data in the Comment fields (Kanban Plugin)"

Implement persistence option "Use a key-value store from the internet"

Implement persistence option "Use a Textfield directly on the board"


How Can I Place The JWT Token In A Cookie Instead Of The Body?

Haven't found any relevant docs on how to do this. It would be more preferable to doing this as suppose storing it in memory or local storage.

Check token extractors: https://github.com/lexik/LexikJWTAuthenticationBundle/blob/master/Resources/doc/1-configuration-reference.md#full-default-configuration

Thanks @stipic.
Closing here.

@stipic or @chalasr I don't think that is what I want at all as far as I can tell? If I am understanding correctly the token extractor is just looking for the token after authentication. Where I am trying get the initial response after authenication to be a cookie instead of a json response.  I have in my lexik_jwt_authentication.yaml
    token_extractors:
            # check token in a cookie
            cookie:
                enabled: true
                name:    BEARER

But this still sends the token in the body of the response.

Or maybe I got this figured out? I need to extend the JWTTokenAuthenticator and follow the same idea as found in !76? Could either one of you confirm this is the right track?


Add casting from arbitrary arrow types

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
While porting some tests to sqllogictest I want t ensure the same coverage for specific types.
For example
https://github.com/apache/arrow-datafusion/blob/350cb47289a76e579b221fe374e4cf09db332569/datafusion/core/tests/sql/mod.rs#L1233
It is easy to cast to some SQL datatype:
-- casts `x` to `bigint` which maps to DataType::Int64
select x::bigint from foo;
-- casts `x` to timestamp (DatType::Timestamp)
select x::timestamp from foo;

However, it is not possible to use SQL to name certain arrow types (e.g TimestampMicrosecondasTIMESTAMP` maps to TimestampNanos
I would like a way to convert an expression to an arbitrary arrow type.
Describe the solution you'd like
I would love to do something like
arrow_cast(source, target_type)

Where target_type is a string that describes an arrow tyoe
For example:
--  casts x to a Int8 (which I don't think is possible in sql)
select arrow_cast(x, 'Int8') from foo;
--  casts x to a LargeBinary
select arrow_cast(x, 'LargeBinary') from foo;

I would like the values accepted as 'target_datatype' to t same as returned by the arrow_typeof() function that goes the other direction (expression to string that represents Arrow type)
❯ select arrow_typeof('5');
+------------------------+
| arrowtypeof(Utf8("5")) |
+------------------------+
| Utf8                   |
+------------------------+
1 row in set. Query took 0.028 seconds.
❯ select arrow_typeof(5);
+-----------------------+
| arrowtypeof(Int64(5)) |
+-----------------------+
| Int64                 |
+-----------------------+

Describe alternatives you've considered
It would be nice to have some way to specify Arrow types in SQL datatype syntax. Perhaps like CAST X as CUSTOM TYPE 'Int64'. This would allow using functions such astry_cast, as well as creating tables with specific column types.
However, I am not really sure how to do this in the parser
Additional context
See https://github.com/apache/arrow-datafusion/issues/4460 for more details

I think this is apretty good first issue as it is well specified and you can follow the model of arrow_typeof to add a new function and there is an arrow cast kernel that does all the data type conversions

@alamb I'd like taking this

Thanks @comphead !

It would be also interesting to support "custom" types in CREATE TABLE statements. This would help to migrate tests like this one https://github.com/apache/arrow-datafusion/blob/master/datafusion/core/tests/sql/aggregates.rs#L913-L976 to sqllogictest.

It would be also interesting to support "custom" types in CREATE TABLE statements. This would help to migrate tests like this one https://github.com/apache/arrow-datafusion/blob/master/datafusion/core/tests/sql/aggregates.rs#L913-L976 to sqllogictest.

I was thinking we could use CREATE TABLE ... AS SELECT ... which creates columns based on the types of the SELECT query
So I think we could make a int8 column in a table like
CREATE TABLE foo as select arrow_cast(c', 'Int8') as c_int8 from foo

But for the potential arrow_cast function, we will return Expression as well with Arrow datatype inside, how it differs from usual cast?

It differs because normal casting CAST(x as target_type) or x::target_type can only cast to SQL types (not Arrow types). DataFusion has a mapping from SQL Type --> Arrow type but there are Arrow types without any corresponding SQL type.

Perhaps you have the analogue in PG? I went through https://www.postgresqltutorial.com/postgresql-tutorial/postgresql-cast/ but didn't find anything similar

I don't think there is any analog in postgres; The difference is that you can refer to all types in postgres directly using SQL. Since posgres types are sql types there is no analog to this ticket's proposal in SQL

@alamb please correct me if I'm wrong so
select arrow_cast(x, 'Int8') from foo;

will have the same output as
select x::tinyint from foo;

@alamb please correct me if I'm wrong so

@comphead you are correct. Here is an example I don't think you can write with SQL:
select arrow_cast(x, 'TimestampMillisecond')

because
select x::timestamp

defaults to TimestampNanosecond

@comphead  I wonder if you plan to work on this issue soon? It would be very helpful for writing sqllogictests for dictionary arrays (e.g. https://github.com/apache/arrow-datafusion/pull/5151). I may take a shot at writing it up this weekend if you haven't had a chance

@alamb sorry, haven't started it yet. made couple of drafts, but thats it. If this is a priority thing, please feel free to take this on

@alamb sorry, haven't started it yet. made couple of drafts, but thats it. If this is a priority thing, please feel free to take this on

Thanks @comphead  -- I will post on this ticket if/when I start work

I am starting to hack on this

https://github.com/apache/arrow-datafusion/pull/5166 is now ready for review (I really want this function so I can more easily write / port tests and reproduce bugs)


Speed up RTD env generation

Description

Motivation and context

Resolves #???
How has this been tested?

Change log

This should speed up our RTD builds!

I was surprised this didn't work since it is the same as it is here? https://github.com/choderalab/openmmtools/blob/main/readthedocs.yml maybe I am missing something @ijpulidos

@ijpulidos should be good to go now!


How to deal correctly with eslint

In our cheminfo eslint preferences we import 2 plugins as peer dependencies and dev dependencies:
https://github.com/cheminfo/eslint-config/blob/c4cdac80b9e621605bd8e4f03fb8a27c4bdb5e8d/package.json#L23-L27
Nevertheless when I use our eslint preferences in this project without explicitly import the 2 plugins:

eslint-plugin-import
eslint-plugin-jest

I have an eslint error for those 2 missing plugins.
So the only way was to add the plugins
https://github.com/mljs/airPLS/blob/485b1aa2e4cc3f79d930c176cc1d8ecfeee39670/package.json#L41-L45
Is this normal ?

And what is the reason why it is a peer-dependency and not simply a dependency ?

Because it doesn't work as a dependency. That's how all eslint configs are setup


Fix plug update

Hey,
I just tried to run plug update and saw some errors. Looks like there was a typo with a variable name, so here's a PR to fix it.
Hope it helps!

Yup, seems like it. Thanks for the patch!
FYI, this project is being rewritten. The master branch should be stable enough for daily usage but I can't ensure there will be no bug. Also no feature will be planned.


Make Message Writable After It Is Initially Set

In Hapi, if an ES6Error is thrown outside of the request/reply lifecycle, Boom will attempt to catch the uncaught error and return a badImplementation. During this process they rewrite the message property to have "Uncaught Error:" in front of it (https://github.com/hapijs/boom/blob/master/lib/index.js#L124)
ES6Error's message isn't writable after initially being said so this causes Boom to throw a TypeError, which will crash the application.
Making the message writable solves this fatal crash (tested on my local branch)

:+1:

Published v2.1.0.


142 add rainfall and river discharge plots

@GundulaW, test_constant_rainfal in test_events.py fails for me. The reason seems to be that you are asserting the value of intensity in inches while in the backend you transform it to mm.


[Obsidian] Option to not open new tab for already open files

Extension
https://www.raycast.com/marcjulian/obsidian
Description
If possible there should be an option to always open a file in an existing tab if it's already open rather than opening a new tab. This helps avoid an explosion of multiple open tabs for the same file and makes it easier to find open files in the tabs list.
Who will benefit from this feature?
No response
Anything else?
No response

Ideally this would work like the Quick Switcher++ plugin which has a "Default to open in new tab" option.

Closing because I suspect this is an issue w/ how Obsidian handles external requests to open files and isn't under the control of an external tool like this extension.


Fix BatchNorm

Description
Fix https://github.com/apache/incubator-mxnet/issues/18373.
The PR https://github.com/apache/incubator-mxnet/pull/17679 introduced the bug @xinyu-intel
@nttstar Should fix your problem.

@xinyu-intel I think the fix should also be in 1.7.

@sxjscience 1.7 doesn't contain #17679 , you can cherry-pick this patch to v1.x when #18160 merged.


Unclear that moderation is only available for public channels

It would be great to mention that channel moderation is only available for public channels. I had created a private channel to try to play around with this, and the feature was not present-- and this documentation does not mention that it is only available for public channels.

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

ID: 097ecd40-1157-98ae-f438-96f9208afc03
Version Independent ID: ea36ff93-46dc-920d-8d52-9b3b81420958
Content: Set up and manage channel moderation in Microsoft Teams - Microsoft Teams
Content Source: Teams/manage-channel-moderation-in-teams.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@kevinpartington-tr Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@kevinpartington-tr Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Change view data for tooltip

Can I somehow add values or replace with my tooltip ?

Hi @Tonypz,
Not sure if I understood right, do you want to render the large tooltip on a bar chart?

Yes @Golodhros , I want to have more information in the tooltip than on default

I am afraid is not possible for now.
We have plans to rework a bit the tooltip and merge both to give you more options, but for now each chart has only one Tooltip option.

ok ! thanks !


Implement writing the log to a file

The ability to write the log to file (without the ansi codes) would be useful for future reference. Perhaps an option within logger with a file path, writing to that file whenever the write method is called (output if buffer is set)?

Functionality included in 4571fc6
Documentation requested in #3


docs: Pulsar How-To-Guide

Add documentation for ingestion from mongodb. Use the kafka or http ingestion as a template for reference.
Depends on issue #40

Note that cookbook project still needs to be done.


feat: Logpush for worker script

@jacobbednarz do we have a clean way of reading the logpush setting for a worker? I can't find anything in the API or in the SDK.
I see the logpush status in /accounts/<account_id>/workers/services/<script_name> but I don't know if it is the right way to retrieve it.

@jacobbednarz do we have a clean way of reading the logpush setting for a worker? I can't find anything in the API or in the SDK.
I see the logpush status in /accounts/<account_id>/workers/services/<script_name> but I don't know if it is the right way to retrieve it.

not to my knowledge. if it doesn't exist in the SDK/API docs, we won't add support for it here as the stability isn't yet ready for public use.

Any chance to get this in?


DNS record update

Hi,
I didn't understand where should I need to add the new value to be updated.
For example, Let's say that this is my configuration file content:
{
"apiKey": "some_api_key_values",
"secret": ""some_secret_values,
"domain": "mydomain.com",
"records": [{"type": "CNAME", "name": "mysubdomain", "ttl": 600}]
}
I want to update the CNAME value (which means destination value) and I didn't find where should I add it.
If you could share with me a full configuration file it would be better.
Please help.
Thanks.

Hello @asafs133,
first of all thanks for adopting this project and for raising an issue here (please, if you like the software remember to give it a star 😇).
Regarding your question,
this tool doesn't make much sense to be used with CNAME records.
The only goal of the tool (for what concerns the current implementation) is to update the IP of an A type record with the current public IP of the machines that updates the record.
The classic use case is when you have a web server (or another public service) on a machine that can change its public IP and you want to update your DNS entry every time it happens.
CNAMEs are generally used as aliases, so you shouldn't generally need to trigger a change if your ip changes...
Let me know if this answer makes sense to you!

Closing this for now. Feel free to re-open it if you still have questions related to this issue


preview.catalogue.life is down

It looks like https://preview.catalogue.life/ is down today:

thanks Yuri. Please post these kind of serious problems in our skype chat in the future so we can respond quickly

Search in the Tree does not work.
Searching for Tanacetum aureum

Thomas also had problems. They seem to be gone now. Please reopen if it persists

It works now on my side


No English version

[x] I have checked that a similar feature request does not already exist.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
Mac download link is on Chinese only. Can't find a way to switch to ENGL. Please, support. Regards!
Additional context
Add any other context or screenshots about the feature request here.

Mac download link is on Chinese only. Can't find a way to switch to ENGL. Please, support. Regards!


Check btrfs-progs is installing, and cockpit can use it

User on Twitter reports cockpit isn't offering BtrFS as a formatting option:
https://twitter.com/timotchee/status/1556016866023333891
Reminder to myself to test from a clean install.

btrfs-progs installs as required but cockpit does not allow formatting a part as btrfs

btrfs support is only PoC https://github.com/cockpit-project/cockpit/pull/16408
i don't think we should switch to this, users will need to setup btrfs manually through terminal or we can write a script to setup recommended

Given that it's still PoC on cockpit's end, and seems to be a bit stale from digging through their status, I might close this off.
I've got some BtrFS doco in the wiki, but I need to add a bunch more around adding disks, changing RAID levels on the fly, etc.

reopening this as the problem with cockpit does not seem to be missing poc but a missing udisk dependency. How far this gets us functionally is yet to be reviewed and i expec we'll still need the poc for complete functionality.

the missing package was udisks2-btrfs
Reviewing deps on cockpit-storaged show udisks2-lvm as the only dep
https://packages.debian.org/bullseye-backports/cockpit-storaged

had a play, no dice. still going to be functionality dependent on the poc. have added the udisks2-(lvm2|btrfs) modules to install_filesystems which also adds lvm support to the cockpit ui. consider it prep work
will keep an eye on the poc


Error: connect ECONNREFUSED <IP_ADDRESS>:3000

Describe the bug

on the frontend, I am getting this error
ERROR: Failed connection to server. Please ensure the server is reachable at ws://:3001/ws.
at the backend, I am getting the below error
Running the app...

<EMAIL_ADDRESS>start
vite --port 3001 --host <IP_ADDRESS>

VITE v5.2.7  ready in 578 ms
➜  Local:   http://localhost:3001/
➜  Network: http://:3001/
➜  press h + enter to show help
4:38:45 PM [vite] http proxy error: /litellm-agents
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1481:16)
4:38:45 PM [vite] http proxy error: /litellm-models
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1481:16)
4:38:45 PM [vite] http proxy error: /litellm-models
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1481:16) (x2)
4:38:45 PM [vite] http proxy error: /litellm-agents
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1481:16)
4:38:45 PM [vite] ws proxy error:
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1481:16)
Setup and configuration
Current version:

commit c37124d74022fc73f32a576d88dd59db2ebfe2ac (HEAD -> main, origin/main, origin/HEAD)
Author: Xingyao Wang<EMAIL_ADDRESS>Date:   Tue Apr 2 14:58:28 2024 +0800

My config.toml and environment vars (be sure to redact API keys):
cat config.toml
LLM_API_KEY="sk-xxxxxxxxxxxxxxxxxxxxxxxxxx"
LLM_MODEL="open-devin-preview-gpt4"
WORKSPACE_DIR="./workspace"

My model and agent (you can see these settings in the UI):
Initializing agent (may take up to 10 seconds)... nothing after that
Commands I ran to install and run OpenDevin:

Steps to Reproduce:

the only change I made apart from the instructions provided is in the makefile
@cd frontend && npm run start -- --port $(FRONTEND_PORT) --host "<IP_ADDRESS>" <- added <IP_ADDRESS> to be accessible over the network.

Logs, error messages, and screenshots:

Additional Context

Error: connect ECONNREFUSED <IP_ADDRESS>:3000

Looks like you forget to start the backend or mock server at 3000

#576

Open file opendevin/frontend/vite.config      and there corect these lines of code "const BACKEND_HOST = process.env.BACKEND_HOST || "<IP_ADDRESS>:3000";
// check BACKEND_HOST is something like "example.com"
if (!BACKEND_HOST.match(/^([\w\d-]+(.[\w\d-]+)+(:\d+)?)/)) {
throw new Error(
Invalid BACKEND_HOST ${BACKEND_HOST}, example BACKEND_HOST <IP_ADDRESS>:3000,
);
}"   with those corected ones "const BACKEND_HOST = process.env.BACKEND_HOST || "<IP_ADDRESS>:3001";
// check BACKEND_HOST is something like "example.com"
if (!BACKEND_HOST.match(/^([\w\d-]+(.[\w\d-]+)+(:\d+)?)/)) {
throw new Error(
Invalid BACKEND_HOST ${BACKEND_HOST}, example BACKEND_HOST <IP_ADDRESS>:3001,
);
}"    - the error is that frontend try to comunicate on <IP_ADDRESS>:3000   and the backend try to communicate on <IP_ADDRESS>:3001 . Afer that save and rerun . It will work ok now .

the only change I made apart from the instructions provided is in the makefile
@cd frontend && npm run start -- --port $(FRONTEND_PORT) --host "<IP_ADDRESS>" <- added <IP_ADDRESS> to be accessible over the network.

Why do you need the backend to be accessible over the network? Are you running backend and front end on different boxes? I suspect your setup will run fine if you dropped --host "<IP_ADDRESS>"
If you need to keep it, you need to set the environment variable BACKEND_HOST to your backend server's IP address or DNS.

You should check the log. I guess it is also related to #573

#576

@JustinLin610 - Yes it is the issue
When I tried wget without sudo, I got
Cannot write to ‘/tmp/llama_index/models--BAAI--bge-small-en-v1.5/snapshots/5c38ec7c405ec4b44b94cc5a9bb96e735b38267a/1_Pooling/config.json’ (Success)
the fact that it prints (Success) at the end could be misleading
with sudo
2024-04-02 21:46:29 (2.78 MB/s) - ‘/tmp/llama_index/models--BAAI--bge-small-en-v1.5/snapshots/5c38ec7c405ec4b44b94cc5a9bb96e735b38267a/1_Pooling/config.json’ saved [190/190]
Success
Starting backend...
INFO:     Started server process [12999]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:3000 (Press CTRL+C to quit)
have a question though, do I have to start the back end separately? If so, where can I have a nohup added?

The newest makefile has a run target that runs both backend and frontend for you.

have a question though, do I have to start the back end separately? If so, where can I have a nohup added?

@lijom80 You don't need to, now make run will let you start both the front and back end at the same time and see their outputs simultaneously.

i've got the same problem with the new make run command, giving me this error:
5:39:57 PM [vite] http proxy error: /litellm-models
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1602:16)
5:39:57 PM [vite] http proxy error: /litellm-agents
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1602:16)
5:39:57 PM [vite] http proxy error: /litellm-models
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1602:16)
5:39:57 PM [vite] http proxy error: /litellm-agents
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1602:16)
5:39:58 PM [vite] ws proxy error:
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1602:16)
5:40:03 PM [vite] http proxy error: /litellm-models
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1602:16)
5:40:03 PM [vite] http proxy error: /litellm-agents
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1602:16)
5:40:03 PM [vite] http proxy error: /litellm-models
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1602:16)
5:40:03 PM [vite] http proxy error: /litellm-agents
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1602:16)
5:40:03 PM [vite] ws proxy error:
Error: connect ECONNREFUSED <IP_ADDRESS>:3000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1602:16)

Backend always take some time to start, it there any output logs from backend?

My services are running as well. I realised after reboot the node, and
running backend again, it installed few more components.
Also, the new make run does not run the backend automatically for some
reason.
I have a question, what is the ideal infra requirements for make it run
smoothly?
On Wed, 3 Apr 2024, 22:02 Francesco Dotta, @.***> wrote:

no thanks, for me the issue was some cuda library not properly installed
on the wsl.
I solved it.
—
Reply to this email directly, view it on GitHub
https://github.com/OpenDevin/OpenDevin/issues/568#issuecomment-2035070098,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BF3SFTXL2WJLBENCJ2EGGO3Y3QVLLAVCNFSM6AAAAABFTF6E7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZVGA3TAMBZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

My services are running as well. I realised after reboot the node, and running backend again, it installed few more components. Also, the new make run does not run the backend automatically for some reason. I have a question, what is the ideal infra requirements for make it run smoothly?

make build will install dependences to run smoothly.

Seems user can run opendevin now. But maye still keep an eye on backend launch slowly problem.


Correct SAML Scope Documentation

In the docs/oauth.md file the scope said in the documentation lists mail as the category alongside User.Read openid profile
when setting the scope this must be email other wise you will get a error when it is forwarded to EntraID. This caused me and a few members of my organization a bit of head scratching but can be solved with a simple letter added to the documentation.
With the scope set as the documentation provides
It gives the following error in the url User.Read mail openid profile
The+application+%27Cyber-PowerDNS-TEST%27+asked+for+scope+%27_mail_%27+that+doesn%27t+exist+on+the+resource+
Changing the scope to url User.Read email openid profile Resolves the issue

@C0mmanderCrud perhaps there's a misunderstanding with the contribution guidelines here. You can't just create an issue and then a follow-up PR with zero interaction from the maintainers. Furthermore, your PR contains boilerplate without the actual issue reference attached. Additionally, the PR is against the master branch and not the dev branch. Finally, this project has been put on a freeze.
So if you would really like to see this simple change added, please make the necessary changes and I will push it through.

@AzorianMatt  Sorry for the incorrect submission i have just been over to the dev branch and it seems you have already corrected this in Documentation Fix: Azure Authentication Scope Setting #1745. This issue can be closed as you have already made the same commit i was trying to make.
Thanks a lot for taking the time to message back what i had done wrong not just a blanket close.
Thanks for all the work you do with this project.
Looking forward to the new release.
C0mmanderCrud

@C0mmanderCrud apology accepted and apologies all the same if I come across a bit harsh. If you can imagine, this is a repeating pattern of sorts that never ends.
With that being said, I'm glad you caught that PR as well. I had forgotten about it at this point. I don't believe I deserve many kudos as I could be doing a lot better, but I'm facing my own struggles right now.
Nevertheless, the sentiment is greatly appreciated!


fix: error for select max bid query

See more about the problem here: https://artsy.slack.com/archives/CCS8TMQ0K/p1676554860006719
Demo
Before
https://user-images.githubusercontent.com/3513494/219456002-8881af7b-5943-41a1-af42-2d179e8af4cc.mp4
After
https://user-images.githubusercontent.com/3513494/219456063-2be59239-60ba-4734-a34f-37aa5986bc62.mp4

PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

error for select max bid query - gkartalis, dimatretyak

Need help with something? Have a look at our docs, or get in touch with us.

Should a test be added to cover this scenario, or (if impractical) an additional case for our Applause or manual QA scripts?


[regression] export not working

Exporting got broken recently, needs to be fixed.

Addressed by PR #314


Can create AdmissionPolicy and ClusterAdmissionPolicy with empty rules

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Apparently it's possible to create either a ClusterAdmissionPolicy or an AdmissionPolicy that have empty rules.
This leads the policy in pending state, because it's not possible to create a webhook for it.
Expected Behavior
Our controller is also exposing an endpoint that is used to validate Kubewarden's CRDs. This validating admission controller should prevent the creation of these kind of broken resources.
Steps To Reproduce
Create a ClusterAdmissionPolicy with the following spec:
apiVersion: policies.kubewarden.io/v1
kind: ClusterAdmissionPolicy
metadata:
  name: test-bug
spec:
  mode: protect
  module: ghcr.io/kubewarden/policies/disallow-service-nodeport:v0.1.2
  mutating: false
  policyServer: default
  rules:
  - {}
  settings: null

The ClusterAdmissionPolicy is created, but it stays in pending state forever.
Environment
No response
Anything else?
No response

To keep documented. @stormqueen1990 is working on this.

Hello, I'm interested in working on this!

Hello there!
I was making the changes for these and a question appeared on my mind: what are the required fields for the rules of a policy to be correctly reconciled?
I took a look at the schema and it seems the following fields are all required:

spec.rules[].operations
spec.rules[].rule.apiGroups
spec.rules[].rule.apiVersions
spec.rules[].rule.resources

I am therefore assuming all four fields must have a non-empty value for a rules object to be valid. Is this a correct assumption?

Hi!

I am therefore assuming all four fields must have a non-empty value for a rules object to be valid. Is this a correct assumption?

I think that's incorrect; I think we need to ensure that at least some are not empty.
A policy would set their Rules[] so they target something. Having an empty Rules[] means that they definitely do not target anything, so no webhooks will be created for them in the admission & mutation controller (this current issue):
https://github.com/kubewarden/kubewarden-controller/blob/7dec7401746e33fa96dc11e1e2ed5c0be268fa4a/apis/policies/v1/policy_types.go#L61-L63
The Rules struct is a https://pkg.go.dev/k8s.io/api/admissionregistration/v1#RuleWithOperations one, which inside contains an Operations (can be empty, but note that a policy that doesn't check for any operations is a bit moot), and then the admissionRegistration Rules, which has the rest of the slices, and have omitempty. But we should add checks so not all are empty.
Example:
The following has rule.apiGroups empty because it's targeting Pods,
https://github.com/kubewarden/apparmor-psp-policy/blob/f5cadaba0179dee66942c4861b641832b2fffc13/metadata.yml#L1-L5
This other one targeting Ingresses has the 4 of them defined,
https://github.com/kubewarden/ingress-policy/blob/main/metadata.yml

Hi @viccuad!

I think that's incorrect; I think we need to ensure that at least some are not empty, on policy creation or edit.
A policy would set their Rules[] so they target something. Having an empty Rules[] means that they definitely do not target anything

I see! I have some tweaking to do on my patch but will be creating a PR soon (between today & tomorrow).
Thanks for the context!

Hi again, @viccuad!
I just created #375 for this. I still left it in draft because on second thought I feel like the API versions and resources should always exist when one of them is specified. I'll update this later, but could you perhaps take a first-pass look?

@stormqueen1990 I think you are right, for our usecase at least one of the APIVersion or Resources slices should be non-empty.
Did a quick review, looking forward to the ready PR :).


⚠ Secret Website 3 has degraded performance

In b01449c, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2751 ms

Resolved: Secret Website 3 performance has improved in 5dc6e87.