Add bro and suricata to detectionlab

Added bro and suricata to the logger vagrant
configured splunk to ingest both bro and suricata logs
created indexes for bro and suricata
reconfigured logger vagrant box to have 2 CPUs and 4GB of RAM

Awesome PR! Just thinking about this though, the logger host isn't configured as the gateway for DC/WEF/Win10, so it's not going to see any of that traffic. I'm just a bit curious what the use-case for having Bro and Suricata installed on this host would be.

Neat, I didn't realize it would actually be able to capture traffic between the windows machines. Unfortunately the CI system doesn't work with forked builds for security purposes, but give me some time this week to test out this PR and I see no issue approving this if all goes well!

@jbeley - I'm building this PR now, but one thing I'm noticing is that building Bro from source adds a fairly significant amount of time to the build process for the logger VM. Are there pre-compiled packages available that could be used instead?
If we can't cut down the Bro installation time, I think it might make sense to have the Bro and Suricata steps commented out by default in bootstrap.sh. It would allow people to easily enable them without forcing everyone to deal with the added build time.
Let me know what you think.

@clong, Agreed. I will issue another pull request that pulls bro packages
from https://www.bro.org/download/packages.html . That should greatly
reduce the build time.
On Mon, Sep 3, 2018 at 2:07 AM Chris Long<EMAIL_ADDRESS>wrote:

@jbeley https://github.com/jbeley - I'm building this PR now, but one
thing I'm noticing is that building Bro from source adds a fairly
significant amount of time to the build process for the logger VM. Are
there pre-compiled packages available that could be used instead?
If we can't cut down the Bro installation time, I think it might make
sense to have the Bro and Suricata steps commented out by default in
bootstrap.sh. It would allow people to easily enable them without forcing
everyone to deal with the added build time.
Let me know what you think.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clong/DetectionLab/pull/137#issuecomment-418020915,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE7zWxaWAQAO13ylJ_OtsB-2KnI5a6y4ks5uXNU_gaJpZM4WR9HI
.


chore(packages): latest

Coverage remained the same at 100.0% when pulling c5e72dc9494c8f8ae420a9f34c532827c55e7aa6 on update-packages into e86c6f8ba2c34ff259071bc1734817facc206c36 on master.


Post Grid Break

The detailed explanation of the blog settings is very helpful for me to study. Thank you!
but I have a problem of post gridsetting.
As shown in this picture, grids with 2 columns sometimes have problems of misinterpretation of posts..

I don't know what happened, but I think it's probably a matter of setting margins.
Below is the code of  _sass/minimal-mistakes/_archive.scss I modified.
Has anyone experienced the same problem as me? I'll wait for the advice of a competent person.
 @include breakpoint($small) {
    float: left;
    width: span(1 of 2); //5 10

    &:nth-child(2n) {
      clear: clear;
      margin-top: 0.5em;
      margin-left: 1em;
    }
  }

  @include breakpoint($medium) {
    margin-left: 1em; /* override margin*/
    margin-right: 1em; /* override margin*/
    width: span(1 of 2); //3 12

    &:nth-child(2n) {
      clear: clear;
      margin-top: 0.5em;
      margin-left: 1em;
    }
  }

It's like a deliberate design(!), or it's like a crazy hole...

I think your CSS is invalid on this line clear: clear
Try clear: both instead to properly clear the floats every 2nd item.

Thank you! My problem has been completely solved 👍


Model setup tutorial

Perhaps it would be good to have one tutorial/example of how to generate a generic model setup as in the LaMEM example, but without all the LaMEM specific functions.

Yes we have planned one for the JOSS paper

added in PR#92


Increase bfcMaxConcurrencyDeadline

Inrease bfcMaxConcurrencyDeadline from one to four.
This lowers the risk of getting stuck fetching a block
from a slow peer in case of slot collision.

It would be even better if the (default) parameters for all network-related things were all defined in the network code, not in consensus code.

It would be even better if the (default) parameters for all network-related things were all defined in the network code, not in consensus code.

I'm trying really hard to fix this issue with out changing anything related to how configuration is handled.

It would be even better if the (default) parameters for all network-related things were all defined in the network code, not in consensus code.

I'm trying really hard to fix this issue with out changing anything related to how configuration is handled.

I understand that, but I can still dream, can't I? :slightly_smiling_face:.
If all network-related things were defined in the network code, you wouldn't even have to ask consensus for a review :slightly_smiling_face:.

Superseded by #2363


[npm] Add back src

Revert https://github.com/callemall/material-ui/pull/3080.
This is breaking my npm install.
I'm using Material UI like this:
    "material-ui": "git://github.com/callemall/material-ui.git#master",

@oliviertassinari Out of curiosity how is it breaking your npm install/what happens when you try?

The src folder is no longer present.

Well... that's not good. -_- I guess we should revert it for now.

@oliviertassinari @alitaheri Yeah, now that I think about it makes sense. Let's revert this.
I think most people that follow this strategy keep lib in source control. Maybe the best strategy is to ignore src and include lib on releases. And for everything else have both.

@newoga I suggest you to have a lookt at https://github.com/pouchdb/pouchdb/pull/4798.

@oliviertassinari Nice, thanks for sharing. I'll investigate this more when it's time to revisit.


Cannot compile vault-env

Describe the bug
Cannot compile vault-env with go-get
To Reproduce
go get github.com/banzaicloud/bank-vaults/cmd/vault-env
Expected behavior
Having a binary at the end :)
Screenshots
package github.com/hashicorp/vault/helper/consts: cannot find package "github.com/hashicorp/vault/helper/consts" in any of:
        /usr/local/go/src/github.com/hashicorp/vault/helper/consts (from $GOROOT)
        /home/raf/go/src/github.com/hashicorp/vault/helper/consts (from $GOPATH)

The file have changed to "github.com/hashicorp/vault/sdk/helper/consts" I think.
But now I got
go/src/github.com/banzaicloud/bank-vaults/pkg/vault/operator_client.go:902:4: cannot use pluginType (type "github.com/hashicorp/vault/sdk/helper/consts".PluginType) as type "github.com/hashicorp/vault/vendor/github.com/hashicorp/vault/sdk/helper/consts".PluginType in field value

What version of Go do you use?
We switched to Go Modules recently and require at least Go 1.11.5 to build this project.
I've just tried and with go modules enabled I can build that binary.

go version go1.12.3 linux/amd64.
Did I have to enable something for go modules working ?

Can you try to build it with the following command?
GOPATH=/tmp/gopath-for-bank-vaults GO111MODULE=on go get github.com/banzaicloud/bank-vaults/cmd/vault-env

It works with gomodule 👍

Yeah, that's probably a good idea. We are going to improve the documentation soon!

Hi @sagikazarmark
Could you help me to fix the same problem with my environment?
OS: win10
GO Version: 1.11.2 windows/amd64

Hello @kazaff , as @sagikazarmark mentioned to build this project require at least Go 1.11.5.

The docs have now clear steps on how to compile the project: https://github.com/banzaicloud/bank-vaults#installing closing.

@pbalogh-sa  Thank you~

Vault recently moved vault/helper/consts to vault/sdk/helper/consts. You might try removing modules for github.com/hashicorp/vault that are not tied to the specific release needed. I used a local go.mod to specify the included version as I was just including bank-vaults for the client library.

I think it is not yet released, but will do once it gets a stable tag, thanks @mbrancato !

@mbrancato FYI https://github.com/banzaicloud/bank-vaults/pull/573


Skeletons, Skins and Keyframes

Description

Testing

TODO list:

[ ] Test keyframe interpolation with an example
[ ] Create Animation Manager/Packer
[ ] Extend IMesh or IMeshBuffer with Animation data references
[ ] Create Transformation Manager + Animator
[ ] Redo animator API
[ ] Splines example
[ ] Hardware Instancing
[ ] Hardware Skinning
[ ] Remove CFinalBoneHierarchy.h
[ ] Reimplement camera animators
[ ] Remove SceneManager

Review done


Compatibility problem with Mars 5e for batch compedium conversion

Converting whole compendium (or all of them at once), does not seem to work.
There is a progress and metrified compendiums are created, but they seem to be an exact copy of SRD compendiums (just editable and with metrified in name).
Metrifying spells/items manually does work though.
Foundry VTT v0.7.8
DnD 5e v1.2.2

No errors in the console.
I did also try it on tiny new compendium with just 3 items, but it failed also there (I've checked that all 3 convert just fine individually)
Are there any steps I can do to debug/provide more data?

Hi, I am not sure what is causing that problem, can you try to provide me with an export of the world that you are using?
Also if you disable all mods except foundry-mgl is the problem reproducible? You can give me a message on discord assassinduke#0983.

Hi, I am not sure what is causing that problem, can you try to provide me with an export of the world that you are using?
Also if you disable all mods except foundry-mgl is the problem reproducible? You can give me a message on discord assassinduke#0983.

Sorry you are right, it was caused by incompatibility with Mars 5e https://foundryvtt.com/packages/mars-5e/

Sorry you are right, it was caused by incompatibility with Mars 5e https://foundryvtt.com/packages/mars-5e/

If you want, you can disable mars, convert the stuff you need then disable my module and enable mars again, but I will look into it for a better fix

If you want, you can disable mars, convert the stuff you need then disable my module and enable mars again, but I will look into it for a better fix

Should be fixed in https://github.com/HadaIonut/Foundry-mgl/releases/tag/v1.1.1

Should be fixed in https://github.com/HadaIonut/Foundry-mgl/releases/tag/v1.1.1


Upgrade leapc

Change api usage to match new api, only small changes like removing IsEmbedded
Added a better unit test to make sure that TransformCopy produces a binary equivalent frame when using the identity transform.  Should help copy errors in the future.
Upgraded IE to make sure to fill all fields of the new data layout
Upped IE version
Updated all dlls to newest version

fixes #427
fixes #428
@jselstad
@ajohnston33
@jo3w4rd

This needs to merge in parallel with http://sf-github.leap.corp/leapmotion/platform/pull/6859
I will be building new dlls shortly, a version check will fail without them.

@ajohnston33 could you push the updated dlls into this branch when you are finished?

Those dlls are added.  This should be ready to merge.

Blocked on http://sf-github.leap.corp/leapmotion/platform/pull/6870
LeapTemporalWarping (and a bunch of other things) are horribly broken when using Rigel if this is not applied.

Rigel-friendly .dlls are in and it works!


Node#unwrap()

In this example I need to dispose of unnecessary spans around chunks of text which follow one another (with spaces between spans) .  Before unwrapping spans:

Start of string text_chunk_1 space space space text_chunk_2 text text.

After unwrapping spans .a and .b:

Start of string text_chunk_1text_chunk_2text text.

The internal text chunks of spans now glued -- spaces around spans are disappeared.
I tried to insert TextNode with space before and after spans and then unwrap spans. The result is the same.

I wasn't able to reproduce this. You can see the test case I added. If you can still repro in the current release, please add a full test case.


Logo appears low in the logo box in IE

BB # 136
From @cpwillett
The institutional logo in IE appears low in the logo box and covers part of the nav bar. See screen shot. This was reported by inst admin at Cornell.

testing to see if this comment appears in pivotal


Terminate smart config smoothly when user enter wrong password for hotspot in ESPTOUCH (IDFGH-2593)

There is a little problem in smart config example of the ESP-IDF master branch.
If user entered wrong password of the WIFI, the esp32 will retry to connect again and agian, and ESPTOUCH wait for long period to report failed status.
We can terminate the ESP32 wait easily, but how can send a message to ESPTOUCH that the password is wrong? How can end smart config in ESPTOUCH smoothly and rapidly?

hi @esonhon ，
If user entered wrong password, the disconnect reason will be 15, user can set the number of the reconnection when receiving disconnected event.
ESPTOUCH APP will wait 1min to report failed status, currently we can't send message to ESPTOUCH  to show that password is wrong.

@esonhon Thanks for reporting, feel free to reopen if any updates or file a new ticket for new issue/new feature request.


Playlists should be updated and not created from scratch

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behaviour
When a Playlist gets created it checks if there is an existing one with the same name and deletes and recreate it instead of updating it
Steps To Reproduce
No response
Expected behaviour
Playlist should be updated
Spotisub Version
0.2.2
Python Version
3.10
Example Code
No response
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

https://github.com/crustymonkey/py-sonic/issues/33

Implemented in v0.2.3


Input objects not clickable

It is not possible to click on input objects and see their content. In the this screenshot

I cannot click on RankInput. RankInput has the following definition :
input RankInput {
# Virksomhed uuid/ Company uuid
uuid: String
# a bunch of other fields
}

After some investigation it turns out you were right, we currently don't expand input type objects so we only support hardcoded query parameters.
I am considering a major rewrite to fix this issue, as well as implement support for mutations, subscriptions, unions, interfaces and whatever else lies between the current state and full spec compliance (well thats the feverdream anyways).. There are also quite a few graphical improvements i would like to make, to make the most of sparse screen real-estate..


Default value for IsCustomLogFile appears to prevent creating log file

It appears that the if statement of line 43, and the fact that IsCustomLogFile initializes to false by default means no logs are saved to the LogFilePath on a fresh custom server
Unless I am mistaken this is not the intended functionality based off that variable name and it makes it impossible to debug server builds uploaded to the game manager, which is particularly unclear when starting out.

-logFile <log_file_path>   is the proper logfile path which is in build params in the Game Manager.  Unity Servers log differently than normal game servers written in pure C#


fix(tasks): Reference the correct secret

Resolves #144

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign tumido after the PR has been reviewed.
You can assign the PR to them by writing /assign @tumido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold

This is not right, merging changes to manifest base would update production manifests. This changeset was introduced via https://github.com/open-services-group/peribolos-as-a-service/pull/138 and is already applied to dev overlay, see:
https://github.com/open-services-group/peribolos-as-a-service/blob/main/manifests/overlays/dev/kustomization.yaml

Superseded by #156 (cherry-picked into).


关于metrics模块指标的含义需要文档

The problem
你们提供了基于spring-boot的标准的监控指标，我也能拿来导出到prometheus中去，但是想要拿来应用到grafana时却不知道面板该怎么配置，能提供监控值的详细说明吗？包括单位等

能提供集成好的grafana面板是最好的了

https://github-com.translate.goog/yidongnan/grpc-spring-boot-starter/issues/894?_x_tr_sl=zh-TW&_x_tr_tl=en

We only added support for the classes provided by micrometer:
https://github.com/micrometer-metrics/micrometer/tree/main/micrometer-core/src/main/java/io/micrometer/core/instrument/binder/grpc
So IMO the example/explanation should there as well. And I will link to it from here.
Some basic documentation is available here:
https://yidongnan.github.io/grpc-spring-boot-starter/en/actuator.html
If you have some suggestions for improvements, please let us know.

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近有事，无法亲自回复您的邮件。我将在事情做完后，尽快给您回复。

Thank you for your reply.
I noticed that you mentioned the support you provide for the classes provided by Micrometer. There are numerous metric values, and i am attempting to export them to Prometheus and visualize them in Grafana using graphs. It would be beneficial if you could provide me with a Grafana dashboard to demonstrate how to use it effectively.

gRPC-Spring -> Actuator/Micrometer -> Prometheus -> Grafana is a typical observability setup we are working on, stay informed.


libgit: get ready for object-repacking during GC

It's turned off for now by always passing in MaxObjectPacks=-1, but once more re-packing testing has been done, we'll flip it on.
Issue: KBFS-2627


Fix function comments based on best practices from Effective Go

Every exported function in a program should have a doc comment. The first sentence should be a summary that starts with the name being declared.
From effective go.
PR generated by CodeLingo. Install here to drive Continuous Higher Standards.

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:

Hi yahoo and gryffin contributers,
Shane from CodeLingo here. This PR is testing out a new service. I'd love to hear your feedback or answer your questions. We're keen to learn how we can best help dev teams without getting in your way.
Shane & the CodeLingo Team


[Bug]: IcebergIO opens a writer using the table's schema, which can cause data loss

What happened?
I'm experimenting with HiveCatalog and noticed that when writing data with nested records, there is data loss. Specifically, the nested records are never committed to the table. I read back null values instead. I can confirm this happens even without our Beam library:
    TableIdentifier tableIdentifier =
        TableIdentifier.parse(String.format("%s.%s", TEST_DB, TEST_TABLE));
    org.apache.iceberg.Schema icebergSchema = IcebergUtils.beamSchemaToIcebergSchema(ROW_SCHEMA);
    Table table = catalog.createTable(tableIdentifier, icebergSchema);

    // write
    List<Record> records = LongStream.range(1, 10).boxed().map(l -> IcebergUtils.beamRowToIcebergRecord(icebergSchema, ROW_FUNC.apply(l))).collect(Collectors.toList());
    String filepath = table.location() + "/" + UUID.randomUUID();
    OutputFile file = table.io().newOutputFile(filepath);
    DataWriter<Record> writer =
            Parquet.writeData(file)
//                    .schema(table.schema())    <---- xxxx this is the problematic line xxxx
                    .schema(icebergSchema)
                    .createWriterFunc(GenericParquetWriter::buildWriter)
                    .overwrite()
                    .withSpec(table.spec())
                    .build();
    for (Record rec: records) {
      System.out.println("xxx writing: " + rec);
      writer.write(rec);
    }
    writer.close();
    AppendFiles appendFiles = table.newAppend();
    String manifestFilename = FileFormat.AVRO.addExtension(filepath + ".manifest");
    OutputFile outputFile = table.io().newOutputFile(manifestFilename);
    ManifestWriter<DataFile> manifestWriter;
    try (ManifestWriter<DataFile> openWriter = ManifestFiles.write(table.spec(), outputFile)) {
      openWriter.add(writer.toDataFile());
      manifestWriter = openWriter;
    }
    appendFiles.appendManifest(manifestWriter.toManifestFile());
    appendFiles.commit();

    // read
    table = catalog.loadTable(tableIdentifier);
    TableScan tableScan = table.newScan().project(icebergSchema);
    for (CombinedScanTask task : tableScan.planTasks()) {
      InputFilesDecryptor decryptor = new InputFilesDecryptor(task, table.io(), table.encryption());
      for (FileScanTask fileTask : task.files()) {
        InputFile inputFile = decryptor.getInputFile(fileTask);
        CloseableIterable<Record> iterable =
                Parquet.read(inputFile)
                        .split(fileTask.start(), fileTask.length())
                        .project(icebergSchema)
                        .createReaderFunc(
                                fileSchema -> GenericParquetReaders.buildReader(icebergSchema, fileSchema))
                        .filter(fileTask.residual())
                        .build();

        for (Record rec : iterable) {
          System.out.println("xxx reading: " + rec);
        }
      }
    }

I've tried the same with HadoopCatalog and everything seems to be working fine. I'm not sure why I'm seeing this error with HiveCatalog. There may be a gimmick in how the catalog fetches and returns the table schema.
I believe in this case we shouldn't rely on the catalog, and instead create our writers using the schema of the records in our PCollection. ie. line 70 here:
https://github.com/apache/beam/blob/21009e68eabc9d8e05c5cfadaf5016fa23646fb7/sdks/java/io/iceberg/src/main/java/org/apache/beam/sdk/io/iceberg/RecordWriter.java#L66-L70
Issue Priority
Priority: 1 (data loss / total loss of function)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Infrastructure
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

I see similar behavior when using the BigQuery Metastore catalog

Update: I realized this is actually due to a bug in our Iceberg utils. Fixing in #32095


Fix issues with DRK multi-hit and resource modules

I intentionally have not updated the changelog yet.  I'm working on one more change that will change "no resource gain on miss/invuln" handling to use the hit type enum rather than checking for damage = 0, but wanted to split that into a separate pull request because it will impact core.  I will update the changelog to include all three changes with that pull.


fixing 2 typos that bugged me in the readme

There's a pair of typos in the readme file I wished to have updated.

Thanks for the contribution!


Launcher - Classpath env var must contain vert.x core

Following by instuction

IntelliJ - create a Run configuration (Application), set the Main class to io.vertx.core.Launcher. In the Program arguments write: run your-verticle-fully-qualified-name --redeploy=**/*.class --launcher-class=io.vertx.core.Launcher. To trigger the redeployment, you need to make the project or the module explicitly (Build menu → Make project).

Always get error
25.02.2016 12:21:25.833 [main] INFO  io.vertx.core.impl.launcher.commands.Watcher - Starting the vert.x application in redeploy mode
Starting vert.x application...
ef5985cd-6a55-4506-948c-3ee655ea6997-redeploy
Error: Could not find or load main class io.vertx.core.Launcher

It's because there is no vertx-core-3.2.1.jar in the classpath. The error arises because of in my system has been defined CLASSPATH var. After reset CLASSPATH the problem has gone.
I think it's good idea to mention about this in the documentation.

Definitely !
Yes the "CLASSPATH" needs to contain vert.x. I will add a "warning" in the documentation.

Fixed in https://github.com/eclipse/vert.x/pull/1352.

PR merged


Key Guardians (Share Activation link)

Is your feature request related to a problem? Please describe.
Currently user has to select Copy and then past in relevant messages and other communication apps.
Describe the solution you'd like
INSTEAD the user should be able to select SEND (represented by the icon) and then it will open the system dialogue to sent via other messaging apps.

@JGDoh What are the steps to get to this screen? Is there a test account I could use for testing this? testingseeds?

🛑 @JGDoh What are the steps to get to this screen? Is there a test account I could use for testing this? testingseeds?

Hey Sky, try my testing account "faruandluca1" you should be able to test this with it.

Thanks, @gguij004 🙏 That worked and I was able to test the case below:

Run the app.
If logged in, log out.
Tap Login screen > Lost your key? Recover your account here.
Enter Username > faruandluca1.
Tap on Faru Luca or Next.
Observe a generated link under Link to Activate Key Guardians.
Tap the share button.
Observe the sharing options.

However, if I:

Cancel the recovery process
Open the activation link in the browser
Observe the exception below

@gguij004 Any ideas what's happening? (I can dig in deeper, if need be)

Any ideas what's happening? (I can dig in deeper, if need be)

Hmm, might be networking issues here at the airport. Although, I tried on a VPN connection and I'm getting the same error 🤷♂ I'll try again later.

Probably out of scope for this ticket and to be addressed separately, but the request screen could use Spanish localization:

@buildc0de was this already fixed? maybe a new ticked to be able to close this

@RaulUrtecho As we discussed in the last meeting, I will be looking into implementing https://weblate.org, so we can close this 🤓


[Stats] Add a test for long input-path bug (0e5b982d)

Add a testcase for the bug in 0e5b982d where we were forming over-long stats-dir output filenames.

@swift-ci please smoke test and merge

@swift-ci please smoke test Linux platform

@swift-ci please clean smoke test Linux platform


[4177] Sanitize funding tables

Context
Now that we will be able to import funding data for providers and lead schools, we need to make sure we are sanitising this data in db/scripts/sanitize.sql so we only pull down an anonymised version.
https://trello.com/c/wqSZRyL9/4177-sanitise-funding-data
Changes proposed in this pull request

Set payable IDs to NULL
Obscure Lead school names and URNs

Guidance to review

Run against your local database and check that there is no PII

Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Sanitise funding data


Fix benchmarks

A dependency called benchmarked was updated in fc56f6b but that update contained a breaking change: jonschlinkert/benchmarked@fbf093e - see the removal of the add property as well as the addition of the code property that replaced the add property.
This commit makes the benchmarks run again, because it replaces add with code 😄

Coverage remained the same at 100.0% when pulling 173351afc8849059f0710d57fbe65ef67368294e on jamescostian:fix_benchmarks into fc56f6be3a80917e21f6207668f99ea03af4c985 on blakeembrey:master.


Suggestions to work around current connection issues (Cloudflare).

I have recently been experiencing issues with the library, but I really like its simplicity and wouldn't want to switch to another one. While looking for solutions, I just came across an interesting topic that might be our solution to recent problems. I haven't implemented it yet, but I'm leaving it here for your analysis.
You can find more details in this link: https://github.com/danny-avila/LibreChat/issues/370#issuecomment-1560382302

Hi, thanks for opening this issue! I had a look at the linked issue, but this is not something that I have encountered before. Also, you can check the latest CI run in which all tests passed.
If you're experiencing any issues related to the Captcha challenge, there is a way to fix it. I wrote about it here. The solution requires some manual actions, but it worked on my side.
Let me know if you have more questions, or this did not cover the problem you're facing, please provide more details on how to reproduce. Thanks!

Should be fixed by 0.18.0


TypeScript definitions

There doesn't seem to be any @types/koa-connect package. Can we add TypeScript definitions to this project?

Thanks for the suggestion. Sure, I'd support that. I would also support going ahead and writing the source in typescript. Either way, PRs welcome!

One year later... still no typescript?

Thanks for the ping @jonaskello, got me motivated to actually act on this. Just pushed a couple more commits to @aaronbeall's PR. If y'all can help review it I'll make sure a new version with TypeScript support gets published this week.

@lourd Thanks for working on this, and sorry for the somewhat harsh ping :-)
I did a quick review and left a comment in the PR. Other than that I think it looks good.

No worries at all, thank you @jonaskello.
I really think there should be some sort of integration tests with a couple of the popular Express middleware libraries to make sure the TypeScript version will actually work with ecosystem. I'm not confident in the functionality without that. Can you or @aaronbeall or one of the other folks subscribed here volunteer to do that?

Closed via #13


feat(mlp): add application configuration into MLP's runtime configuration

Motivation
To support https://github.com/gojek/mlp/pull/72
Modification

.Values.deployment.applications section is added to Helm values
Lower camel-case is used to define keys in the config map

Checklist

[x] Chart version bumped
[x] README.md updated

Thanks, @maniteja6799 🙌

hi @romanwozniak, I realized an issue with this PR just now. The values for feast/merlin/turing api/prefixes were earlier being set using the helper functions which take the values from global values if present. Now that you've moved them to Values.deployment.applications key as a yaml, we are not looking at the global values if present to override if mlp is being installed via the caraml chart.
I'll raise a fix for this.
cc @mbruner

@maniteja6799 yeah, while working on this, I was thinking that it would be great if the .Values.deployment.applications is generated dynamically, based on the components that are enabled for the current deployment of CaraML (i.e. based on Helm tags enabled from the umbrella chart). But I'm not sure if this can be done now.


CONTRIBUTING.md: fix syntax highlighting

Add syntax highlighting to the YAML front-matter
Fix the syntax of the "JSON" code snippet: it’s not valid JSON because of the …s and so it shows all red. Using JS syntax fixes the issue.

Thanks for you contribution.
However, the fixes to the issues you've mentioned are unnecessary.

There's no need to add yaml for the example markdown front-matter section.
There's no need to change json to js, since it is actually an excerpt of a json file, and ... indicates that you can add more.

Hope to hear from you for other contributions soon. Thanks again. 👍

The front-matter section is YAML and the bogus JSON syntax highlighting renders like below, but it’s your choice.


I will take a look!


Various URL fixes

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Other information:

fixed another way


Model generator update to fix #1387

Aims to fix #1387 by prefixing the migration automatically created when generating a model with "Constrain" and adding force: true to the migration. i.e.:
rails g model Person will create
app/models/person.rb
db/neo4j/migrate/xxxxxx_constrain_person.rb

with db/neo4j/migrate/xxxxxx_constrain_person.rb containing
class ConstrainPerson < Neo4j::Migrations::Base
  def up
    add_constraint :Person, :uuid, force: true
  end

  def down
    drop_constraint :Person, :uuid
  end
end

This is incomplete. So far all I've done is add force: true to the model migration template. I imagine I can update the name of the migration file created by rails g model by updating the model generator code, but I haven't found where in the code the model generator calls the migration generator.
Pings:
@cheerfulstoic
@subvertallchris

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1390   +/-   ##
=======================================
  Coverage   97.33%   97.33%
=======================================
  Files         205      205
  Lines       12360    12360
=======================================
  Hits        12031    12031
  Misses        329      329

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 42e09c6...f686705. Read the comment docs.

Unfortunately force: true shouldn't be the default behavior (otherwise we might make it the default value in the code accepting the value).  The more fundamental problem is that migrations should not be created with the same name as models (again, I'd want to know how ActiveRecord handles this)

@cheerfulstoic,
When creating a constraint migration manually (i.e. rake neo4j:generate_schema_migration[constraint,Person,uuid]), neo4j already adds force: true by default. Why would it be ok there but not when running rails g model Person? Or am I missing something


add niu

RE-CHECK!

Automated adapter checker
ioBroker.niu

[ ] :heavy_exclamation_mark: [E201] Bluefox was not found in the collaborators on NPM!. Please execute in adapter directory: npm owner add bluefox iobroker.niu
[ ] :eyes: [W156] Adapter should support admin 5 UI (jsonConfig)
[ ] :eyes: [W400] Cannot find "niu" in latest repository

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.niu

[ ] :heavy_exclamation_mark: [E201] Bluefox was not found in the collaborators on NPM!. Please execute in adapter directory: npm owner add bluefox iobroker.niu
[ ] :eyes: [W156] Adapter should support admin 5 UI (jsonConfig)
[ ] :eyes: [W400] Cannot find "niu" in latest repository

Add comment "RE-CHECK!" to start check anew

Welcome


Rename ProgressBar.percent_visible to show_percentage

Judging by this comment's confusion, https://github.com/godotengine/godot/pull/64665#issuecomment-1228345052, a rename may be needed, even if it may not be this one https://github.com/godotengine/godot/pull/64665#issuecomment-1229426587.
For ProgressBar:
percent_visible -> show_percentage
set_percent_visible -> set_show_percentage
is_percent_visible -> is_percentage_shown
Note that this is no longer truly "required", because Label and RichTextLabel's percent_visible, an entirely different and unrelated property, had its name changed already in the aforementioned PR.

Thanks!
Merged manually as 08c2aab169dd3f36bf43bd1e1f48b68bf3fed5a9 (should have been auto-closed and shown as merged but I wrongly rebased locally after doing a local merge...).


navbar-refactor

Description
Describe change or new feature here.
Change status

[x] Complete, tested, ready to review and merge
[ x] Work-in-Progress, PR is for discussion/feedback

Checklist
Remove any items which are not applicable.

[x ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works AND the tests pass

Lookin good!


update values when switching property type

Is your feature request related to a problem? Please describe.
When changing property types, the values are not updated until the db is reloaded.
Describe the solution you'd like
update the values when changing the properties.
Describe alternatives you've considered
change the property, then reload database manually to see changes
Additional context
https://user-images.githubusercontent.com/38974541/174877019-9e4aed64-dbb4-4e45-952c-1971f2392684.mov

Yeah, happens to me a lot. Could be added too =)

Even though it is marked as done in the roadmap, I'm not getting the new behavior, still the same as it was before (see video)
https://user-images.githubusercontent.com/38974541/176727414-f6a7e1af-e7b5-4610-9083-59e1a98b2af9.mov

was fixed but looks like something breaks it again. With the v15 of obsidian API core stuff needs a review. Marked as reopen


Fix is wrong

Map Fixer is using a partial workspace path that is specific to a given data source to incorrectly repair a broken link.  For example if a workspace /a/b/c.gdb is document as moving to /1/2/3.gdb,
but we also have that datasource g in /a/b is moved to /5/6, MapFixer will incorrectly replace /a/b with /5/6 returning /a/b/c.gdb => /5/6/c.gdb

Fixed with 858f21082a2c1e2810b95d8e28d6da9dabf0c2d5


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2024-01-10T02:11:05+00:00

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio_release-1.20
7ef9201775745f2a0cd55cea3ba75f2ecd7f2a6a
link
true
/test unit-tests

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.

/test unit-tests


add suport for mips64 platform

Since tensorflow is not supported on MIPS64 platform， use this patch，it can be build pass on MIPS64 platform。
and then， tensorflow can be supported on MIPS64 platform！

dear sir，
Can this patch be merged？

@zhangqiang-hf please edit your change according to the comments, then we can start merging.


sched: Introduce Bound Multi-Processing (BMP) into NuttX

Summary
sched: Introduce Bound Multi-Processing (BMP) into NuttX
Bound multiprocessing provides the scheduling control of an asymmetric
multiprocessing model, while preserving the hardware abstraction and
management of symmetric multiprocessing.
BMP is similar to SMP, but you can specify which processors a thread
can run on. You can use both SMP and BMP on the same system, allowing
some threads to migrate from one processor to another, while other
threads are restricted to one or more processors.
As with SMP, a single copy of the OS maintains an overall view of all
system resources, allowing them to be dynamically allocated and shared
among applications. But, during application initialization, a setting
determined by the system designer forces all of an application's threads
to execute only on a specified CPU.
Compared to full, floating SMP operation, this approach offers several
advantages:

It eliminates the cache thrashing that can reduce performance in an SMP
system by allowing applications that share the same data set to run
exclusively on the same CPU.
It offers simpler application debugging than SMP since all execution
threads within an application run on a single CPU.
It helps legacy applications that use poor techniques for synchronizing
shared data to run correctly, again by letting them run on a single CPU.

Bound Multi-Processing (BMP):
---------------------------------------------
|   APP 0  |  APP 1   |  APP 2   |  APP 3   |  <- Application bound to CPU
---------------------------------------------
|  Data[0] |  Data[1] |  Data[2] |  Data[3] |  <- NuttX Kernel Data supports multiple CPU instances
---------------------------------------------
|                Share Code                 |  <- NuttX kernel code shared for all CPUs
---------------------------------------------
|   UART 0 |   SPI 0  |   SPI 1  |   I2C 0  |  <- Driver is only registered to CPUs with application needs
---------------------------------------------
|  TIME 0  |  TIME 1  |  TIME 2  |  TIME 3  |  <- Core/CPU timers
---------------------------------------------
|   CPU0   |   CPU1   |   CPU2   |   CPU3   |  <- CPUs run independently
---------------------------------------------

Some subsystem data does not need to be duplicated, especially the components bound to the application.
For shared hardware devices, Use spinlock to avoid race-condition for multi-core.
---------------------------------------------
|   APP 0  |  APP 1   |  APP 2   |  APP 3   |
---------------------------------------------
| NetStack |  BTStack |  AUDIO   |   ...    |  <- Components bound to the application, data no need to duplicate.
---------------------------------------------
|                Share Code                 |
---------------------------------------------
|      Share UART (Protected by Spinlock)   |  <- Driver shared for all CPUS will protected by spinlock(e.g print logs)
---------------------------------------------
|   CPU0   |   CPU1   |   CPU2   |   CPU3   |
---------------------------------------------

Reference:
https://www.ghs.com/products/safety_critical/integrity_178_multicore.html
https://www.qnx.com/developers/docs/7.1/#com.qnx.doc.neutrino.sys_arch/topic/smp_BMP.html
https://www.nxp.com.cn/docs/en/brochure/PWRARBYNDBITSRAS.pdf
Signed-off-by: chao an<EMAIL_ADDRESS>Impact
Depends on: https://github.com/apache/nuttx-apps/pull/2342
N/A
Testing
qemu-armv7a/bmp  ostest on single core
nuttx$ qemu-system-arm -cpu cortex-a7 -nographic      -machine virt,virtualization=off,gic-version=2 -net none -chardev stdio,id=con,mux=on -serial chardev:con -mon chardev=con,mode=readline -kernel ./nuttx -smp 4

NuttShell (N
NuttS
Nutt
NH) NutShellShetSutlltX- (NSH) NuttX-10.4.0
4t(NheSH) NuttX-ns0.h> 1ll (NSH) Nu0.4.0
nstX-1h>
nsh> ps
  PID GROUP PRI POLICY   TYPE    NPX STATE    EVENT     SIGMASK           STACK   USED  FILLED COMMAND
    0     0   0 FIFO     Kthread   - Ready<PHONE_NUMBER>000000<PHONE_NUMBER>36  13.1%  CPU0 IDLE
    1     1 192 RR       Kthread   - Waiting  Semaphore<PHONE_NUMBER>000000<PHONE_NUMBER>96   7.3%  hpwork 0x4013f51c 0x4013f530
    2     2 100 RR       Task      - Running<PHONE_NUMBER>000000<PHONE_NUMBER>68  28.7%  nsh_main
nsh> irqaff 33 1
nsh> ps
  PID GROUP PRI POLICY   TYPE    NPX STATE    EVENT     SIGMASK           STACK   USED  FILLED COMMAND
    0     0   0 FIFO     Kthread   - Ready<PHONE_NUMBER>000000<PHONE_NUMBER>36  18.0%  CPU1 IDLE
    1     1 192 RR       Kthread   - Waiting  Semaphore<PHONE_NUMBER>000000<PHONE_NUMBER>96   7.3%  hpwork 0x4013f544 0x4013f558
    2     2 100 RR       Task      - Running<PHONE_NUMBER>000000<PHONE_NUMBER>88  31.7%  nsh_main
nsh> irqaff 33 2
nsh> ps
  PID GROUP PRI POLICY   TYPE    NPX STATE    EVENT     SIGMASK           STACK   USED  FILLED COMMAND
    0     0   0 FIFO     Kthread   - Ready<PHONE_NUMBER>000000<PHONE_NUMBER>36  18.0%  CPU2 IDLE
    1     1 192 RR       Kthread   - Waiting  Semaphore<PHONE_NUMBER>000000<PHONE_NUMBER>96   7.3%  hpwork 0x4013f56c 0x4013f580
    2     2 100 RR       Task      - Running<PHONE_NUMBER>000000<PHONE_NUMBER>68  28.7%  nsh_main
nsh> irqaff 33 3
nsh> ps
  PID GROUP PRI POLICY   TYPE    NPX STATE    EVENT     SIGMASK           STACK   USED  FILLED COMMAND
    0     0   0 FIFO     Kthread   - Ready<PHONE_NUMBER>000000<PHONE_NUMBER>36  18.0%  CPU3 IDLE
    1     1 192 RR       Kthread   - Waiting  Semaphore<PHONE_NUMBER>000000<PHONE_NUMBER>96   7.3%  hpwork 0x4013f594 0x4013f5a8
    2     2 100 RR       Task      - Running<PHONE_NUMBER>000000<PHONE_NUMBER>68  28.7%  nsh_main

@anchao could you split irqaff change to a new pr? So the change crossing apps/nuttx could be merged first. Since the remaining change touch many files, it's better to ensure it can pass ci standalone.

Can BMP ensure that if an application bound to a separate processor crashes, it will not affect other processors?

Can BMP ensure that if an application bound to a separate processor crashes, it will not affect other processors?

Of course, this is just the initial pull request of BMP. MPU protection and assertion chain related optimization will be added in the future.

Can BMP ensure that if an application bound to a separate processor crashes, it will not affect other processors?

Of course, this is just the initial pull request of BMP. MPU protection and assertion chain related optimization will be added in the future.

Great!

Cool work, I'm curious would this work on a asymmetrical system witch it's own caches i.e. Cortex-M7 and Cortex-M4 but without hardware cache coherency?

Cool work, I'm curious would this work on a asymmetrical system witch it's own caches i.e. Cortex-M7 and Cortex-M4 but without hardware cache coherency?

Yes, but the implementation requires more customized modifications. If platforms without hardware cache consistency, all data must be correctly placed on cache line aligned sections, which will depend on some labeling for specific data/bss in the link script.

What is the difference between this approach and the pthread_setaffinity_np functions implemented by NuttX?Does threads spawned by a task bound to a specific processor can also be automatically bound to that
processor with this approach？

What is the difference between this approach and the pthread_setaffinity_np functions implemented by NuttX?Does threads spawned by a task bound to a specific processor can also be automatically bound to that
processor with this approach？

What is the difference between this approach and the pthread_setaffinity_np functions implemented by NuttX?Does threads spawned by a task bound to a specific processor can also be automatically bound to that processor with this approach？

Please refer PR summary. Compared with SMP, BMP can provide more performance, stability and isolation advantages.

What is the difference between this approach and the pthread_setaffinity_np functions implemented by NuttX?Does threads spawned by a task bound to a specific processor can also be automatically bound to that processor with this approach？

Please refer PR summary. Compared with SMP, BMP can provide more performance, stability and isolation advantages.

i see，it seem's that BMP can resolve two problems of SMP processor affinity: constraining threads in third-party code, and constraining dynamically created threads

@anchao please fix these conflicts and I will merge it

@anchao please fix these conflicts and I will merge it

@acassis  current implementation may impact the distribution of NuttX data resources. I need evaluate whether there is more flexible way to implement this feature further, @xiaoxiang781216 please let me know If any suggestions

@anchao please fix these conflicts and I will merge it

@acassis current implementation may impact the distribution of NuttX data resources. I need evaluate whether there is more flexible way to implement this feature further, @xiaoxiang781216 please let me know If any suggestions

do you still need patch all global variables?

@anchao please include Documentation/ about BMP

@acassis  I will provide documentation later, this PR still needs further enhancement, thanks for review.


MSBuild package references should be marked PrivateAssets="all"

The MSBuild packages are effectively development-only dependencies, because the assemblies included in them are part of the MSBuild distribution, for both the .NET Framework and .NET Core builds.
Having this listed as an actual package dependency needlessly brings Microsoft.Build.Utilities.Core into the set of dependencies of any project that consumes the SourceLink packages. You can see this by looking at the project.assets.json file.

This is important because it's causing package downgrades.
Severity	Code	Description	Project	File	Line	Suppression State
Error	NU1605	Detected package downgrade: System.IO from 4.3.0 to 4.1.0. Reference the package directly from the project to select a different version.
 System.Reactive (>= 1.0.0) -> SourceLink.Create.CommandLine (>= 2.1.2) -> Microsoft.Build.Utilities.Core (>= 15.1.548) -> System.Console (>= 4.0.0) -> runtime.win.System.Console (>= 4.3.0) -> System.IO (>= 4.3.0)
 System.Reactive (>= 1.0.0) -> SourceLink.Create.CommandLine (>= 2.1.2) -> Microsoft.Build.Utilities.Core (>= 15.1.548) -> System.IO (>= 4.1.0)	System.Reactive	C:\dev\RxNET\Rx.NET\Source\src\System.Reactive\System.Reactive.csproj	1

@ctaggart It appears that this change wasn't made for the GitHub package: https://github.com/ctaggart/SourceLink/blob/master/SourceLink.Create.GitHub/SourceLink.Create.GitHub.csproj#L11


How to get current build number?

Actually, there is no way to get the CURRENT build number. Instead we could get the last build number.

Actually, there is no way to get the CURRENT build number. Instead we could get the last build number.

Do you mean that after I create a job using BuildJobWithParams, I then use GetJob to get the last id of Builds[]Build as the build number of the current jobid?

It's true if there is only one build request in the short time.
By the way, Jenkins will create a queue job, then there is a worker will consume that queue item. So, we cannot get the build ID after the build request. Because it might does not have the ID yet in sometime.

Thank you, is there any way I can customize the job number?
this is my code:
func BuildJobWithParams(params []job.ParameterDefinition) (result bool, err error) {
	jenkins, err := GetJenkinsCore()
	if err != nil {
		return false, err
	}
	jobClient := job.Client{JenkinsCore: jenkins}

	err = jobClient.BuildWithParams(PHPMigrateScript, params)
	if err != nil {
		return false, err
	}
	result = true
	return
}

params := []job.ParameterDefinition{
		{
			Name:        "IMAGE_TAG",
			Value:       "v1.4.251",
			Description: "Docker image and tag to use",
		},
		{
			Name:        "ENV",
			Value:       "pre",
			Description: "prod or pre",
		},
		{
			Name:        "PROJECT",
			Value:       "xthk-financial-management-system",
			Description: "project name",
		},
		{
			Name:        "COMMAND",
			Value:       "php artisan  cancel_class_export test",
			Description: "command",
		},
	}

	result, err := utils.BuildJobWithParams(params)

I found that ParameterDefinition.DefaultParameterValue have a Number field

As far as I know, the job build number can only be generated by Jenkins itself.

As far as I know, the job build number can only be generated by Jenkins itself.

If it cannot be customized, then I can only add a unique identifier to distinguish it.

add a unique identifier to distinguish it

This is a good idea I believe.

job/JOB_NAME/BUILD_ID/api/json
how to get build.actions. parameters

func BuildJobWithParams(params []job.ParameterDefinition) (build *job.Build, err error) {
	jenkins, err := GetJenkinsCore()
	if err != nil {
		return nil, err
	}
	jobClient := job.Client{JenkinsCore: jenkins}
	err = jobClient.BuildWithParams(PHPMigrateScript, params)
	if err != nil {
		return nil, err
	}
	build, err = jobClient.GetBuild(PHPMigrateScript, -1)

	return
}

2023/10/10 11:16:06 &{SimpleJobBuild:{Number:2931 URL:https://example.com/job/php-migrate-script/2931/} Building:true Description: DisplayName:#2931 Duration:0 EstimatedDuration:42531 FullDisplayName:php-migrate-script #2931 ID:2931 KeepLog:false QueueID:5057 Result: Timestamp:1696907748017 PreviousBuild:{Number:2930 URL:https://exmaple.com:443/job/php-migrate-script/2930/} NextBuild:{Number:0 URL:}}
problem：
The build number I am currently building is 2932, but the build number returned by jobClient.GetBuild(PHPMigrateScript, -1) is 2931, but when I use the browser to visit example.com/job/php-migrate-script/lastBuild/api/ json, the returned jobnumber is 2932, the browser result is correct, and the task is still under construction at this time.

It might be a bug if you only triggered the job build once.

It might be a bug if you only triggered the job build once.

Yes, I checked to make sure it was only executed once

Please feel free to fix  this if you're interested in it.


Internationalization

Resolves #180
Hardcoded strings are not actually translated into other languages, but are represented as placeholders denoted by <normal string> (<language>), as we do not have the language expertise to properly translate everything.
This PR will be merged after every other feature PR so that any new hardcoded strings are accounted for, hence the delayed label.

Make sure this is also properly documented in the appropriate files as well so that Kiva understands the code as well as how to insert new languages or where to put the correct translations.


add comet lake support #248

fix issue #248

Thx for the PR @ajax16384 (and for the review @toor1245 🙂)


Add additional regexp functions

Is your feature request related to a problem or challenge?
I would like to see the following regexp functions implemented. These exist in some, but not all, versions of PostgreSQL.

[ ] regexp_count()
[ ] regexp_instr()
[ ] regexp_matches()
[ ] regexp_split_to_array()
[ ] regexp_split_to_table()
[ ] regexp_substr()

Describe the solution you'd like
Implement these functions.
Describe alternatives you've considered
These operations can be performed using the existing functions, so I am currently unblocked for my immediate use case but having these functions built in would be convenient.
Additional context
We currently have the following regexp functions implemented. The source is in datafusion/functions/src/regex/mod.rs
regexp_like()
regexp_match()
regexp_replace()

I could work on this. The only concern is whether we implement the regexp function in this project or in arrow-rs.
Hey @alamb, would you prefer implement function in arrow-rs directly or we put them in datafusion and port later?

I could work on this. The only concern is whether we implement the regexp function in this project or in arrow-rs.
Hey @alamb, would you prefer implement function in arrow-rs directly or we put them in datafusion and port later?

Thanks @xinlifoobar
I would personally recommend we start implementing them in datafusion as that will avoid the need to wait for coordinated releases of arrow-rs, and then port backupstream to arrow-rs as a follow on step.

@xinlifoobar  I suspect there will be several other contribtuors interested in helping out and learning during the process. If we have a good example to follow the work would be straightforward to scale I think
One way to do this might be:

You implement one of these functions in a PR, along with good docs, tests, etc
Then we can file additional tickets for the other functions, linking to your first implementation

Related to this, substring in Postgres supports regex matching (see https://www.postgresql.org/docs/current/functions-matching.html), would it be reasonable for DataFusion to also support it?
Postgres's regex substring takes a string, a pattern, and an escape character, so I don't think there would be a conflict.

Spark's version of this is https://spark.apache.org/docs/latest/api/sql/#regexp_substr

Based on prior conversations it sounds like the group is most interested in making sure we are supporting Postgresql so I think adding this is a very good idea. We can also have regexp_substr as an alias.

It is a good idea to support these functions, especially for regexp_matches

It is a good idea to support these functions, especially for regexp_matches, Thank you.

Adding regexp_matches would be trivial - it's essentially the current regexp_match but the 'g' (global) flag is always present (whereas with the regexp_match it's disallowed)

Related to this, substring in Postgres supports regex matching (see https://www.postgresql.org/docs/current/functions-matching.html), would it be reasonable for DataFusion to also support it?
The currently accepted argument types are:
                    Exact(vec![Utf8, Int64]),
                    Exact(vec![LargeUtf8, Int64]),
                    Exact(vec![Utf8, Int64, Int64]),
                    Exact(vec![LargeUtf8, Int64, Int64]),
                    Exact(vec![Utf8View, Int64]),
                    Exact(vec![Utf8View, Int64, Int64]),

Postgres's regex substring takes a string, a pattern, and an escape character, so I don't think there would be a conflict.

I took a look at the syntax for that pg function, and frankly it's awful. Personally I think that is a function best ignored.

I see your point for the postgres - since it essentially does either substring from this character index to that OR regex matching, I think it would probably add more confusion than value to support in exactly the way they do. However I'm far from a SQL expert.


Calculate 15/30 minute rolling average on headways

We'll also need to do something similar for scheduled headways. This is something the MBTA used to to for the v2 Performance API but not for LAMP, so it will smooth out the current metrics and also bring us to parity with the old version of the dashboard data.

Initial code for this exists in https://github.com/transitmatters/mbta-performance/pull/21. I am missing the last step of merging the calculated averages into the results, but it should offer a good base


Make py2k/py3k independent, add setup.py

I have to work on RHEL-7 where we still have only python 2.7. Given that this doesn't break py3k compatibility, I think it should be painless to add this modification to py2k.
Also, setup.py is a good thing to have, so that the script can be finally installed.

I don’t what’s the versioning policy on the script, so I left it with 0.0.0.


Prompt when equipping artifacts from artifact card when destination character already have artifact equipped

Similar to the ones we use for build results:

Stale


When will there be a release or a separate branch for Swift 4.2?

Hello @ninjaprox! When will there be a release or a separate branch for Swift 4.2?

@alexanderkhitev: There'll be a new version as soon as Xcode 10 releases officially. Stay stunned 😉

@alexanderkhitev
Workaround:
pod 'NVActivityIndicatorView', :git => 'https://github.com/ninjaprox/NVActivityIndicatorView.git', :branch => 'swift4.2'

@MSavisko
Thanks, I made it. I asked this question when there was no Swift 4.2 branch

just for information Xcode 10 final version has been released.

@alexanderkhitev: Version 4.4.0 with Swift 4.2 is out. 🚀

@ninjaprox thank you!


Auto-restart like alis.io does not work in version 1.6.1

I got the auto-restart as alis.io, it works in version 1.4.2 but in version 1.6.1 it does not work help me!:
var tEnd = process.hrtime(tStart);
this.updateTime = tEnd[0] * 1000 + tEnd[1] / 1000000;
for (var i = 0; i < this.clients.length; i++) {
if (!this.isRestarting) {
var client = this.clients[i].playerTracker;
if (client && client.cells.length > 0) {
if (client._score / 100 > this.config.serverRestartSize) this.resetServer();
}
}
}
};
GameServer.prototype.resetServer = function() {
this.isRestarting = true;
var newLB = ["Restarting", "In", "1", "Minute"];
this.gameMode.packetLB = 48;
this.gameMode.specByLeaderboard = false;
this.gameMode.updateLB = function(gameServer) {
gameServer.leaderboard = newLB;
gameServer.leaderboardType = 48;
};
setTimeout(function() {
process.exit(1);
var gm = Gamemode.get(this.gameMode.ID);
    // Replace functions
    this.gameMode.packetLB = gm.packetLB;
    this.gameMode.updateLB = gm.updateLB;
    this.isRestarting = false
    }.bind(this), 20000)

};

https://github.com/Megabyte918/MultiOgar-Edited/issues/1281
Fantasy's comment.

Why not

Because it's custom. This is not StackOverflow. If you want something to work put effort into it instead of immediately asking everywhere.

Ok thx


does Azure really work ?

I followed the setup described in the docs like this:

import nest_asyncio
nest_asyncio.apply()

from dotenv import dotenv_values, load_dotenv

load_dotenv()

config=dotenv_values(".env")
open_ai_base = config['AZURE_OPENAI_API_BASE']
openai_api_key = config['AZURE_OPENAI_API_KEY']
openai_api_type = config['AZURE_OPENAI_API_TYPE']
openai_api_version = config['AZURE_OPENAI_API_VERSION']
model_name=config['MODEL_NAME']
embedding_model=config["EMBEDDING_MODEL"]

lmql_model = "openai/" + str(model_name)

print ("lmql_model ===> ", str(lmql_model))

import lmql

llm: lmql.LLM = lmql.model(
    # the name of your deployed model/engine, e.g. 'my-model'
    lmql_model,
    # "openai/gpt-3.5-turbo",
    # set to 'azure-chat' for chat endpoints and 'azure' for completion endpoints
    api_type=openai_api_type,
    # your Azure API base URL, e.g. 'https://<YOUR_BASE>.openai.azure.com/'
    api_base=open_ai_base,
    # your API key, can also be provided via env variable OPENAI_API_KEY
    # or OPENAI_API_KEY_{<your-deployment-name>.upper()}
    api_key=openai_api_key,
    # API version, defaults to '2023-05-15'
    api_version=openai_api_version,
    # tokenizer="tiktoken",
    # prints the full endpoint URL to stdout on each query (alternatively OPENAI_VERBOSE=1)
    verbose=False,
)

# simple generation
llm.generate_sync("Hello", max_tokens=10)

I get this error
TokenizerNotAvailableError: Failed to locate a suitable tokenizer implementation for 'MY_DEPLOYMENT' (Make sure your current environment provides a tokenizer backend like 'transformers', 'tiktoken' or 'llama.cpp' for this model)

Versions:
Python: 3.10.13
LMQL: 0.7.3

What did I do wrong ? Has Azure been tested with LMQL ? BTW, my deployment works when I do a Postman request... So I don't understand...

@lbeurerkellner : Could you please take a look at this as it's a blocking issue for me ? Also, I'd like to know if I misconfigured sth or if it's rly a bug. Thank you

I noticed something:
When I try this:
import lmql

llm: lmql.LLM = lmql.model(
    # the name of your deployed model/engine, e.g. 'my-model'
    "openai/gpt-3.5-turbo",
    api_type='azure-chat',
    api_base=open_ai_base,
    api_key=openai_api_key,
    api_version=openai_api_version,
    verbose=True,
)

I get this in the logs:
Using Azure API endpoint: https://XXXXXXXXXX-XXX-XXX-XXXXXX.openai.azure.com/openai/deployments/gpt-3.5-turbo/chat/completions?api-version=2023-09-01-preview True
openai complete: {'model': 'gpt-3.5-turbo', 'max_tokens': 10, 'temperature': 0, 'user': 'lmql', 'stream': True, 'messages': [{'role': 'user', 'content': 'Hello'}]}

However, when I try with ``    "openai/my-deployment-gpt-35", I get the error:
OSError: cma-cgm-gpt-35-turbo-sbx-ibm is not a local folder and is not a valid model identifier listed on 'https://huggingface.co/models'
If this is a private repository, make sure to pass a token having permission to this repo either by logging in with `huggingface-cli login` or by passing `token=<your_token>`

TokenizerNotAvailableError: Failed to locate a suitable tokenizer implementation for 'my-deployment-gpt-35' (Make sure your current environment provides a tokenizer backend like 'transformers', 'tiktoken' or 'llama.cpp' for this model)

It seems as if for LMQL, considers deployment and model to be the same... which is not the case in Azure

I've encountered the same issue, hoping someone could provide a resolution.

Hi there. Can you try passing a tokenizer via tokenizer="openai/gpt-3.5-turbo" to the lmql.model constructor. From the error, the problem appears to be that no compatible tokenizer can be derived from the provided deployment name.

Great. I added a note to the docs here: https://lmql.ai/docs/latest/models/azure.html

Awesome! Thank you @lbeurerkellner !

Hi @lbeurerkellner , I'm hitting a similar issue. Given a lmql.model() setup of this form, trying to use Azure:
lmql.model(
            'openai/MY_MODEL',
            api_type='azure',
            api_base=MY_API_BASE,
            api_key=MY_API_KEY,
            api_version=MY_API_VERSION,
            verbose=True,
            tokenizer='openai/MY_MODEL',  # same as first argument
        )

When I run with this, I still get the error regarding failure to identify a good tokenizer:
lmql.runtime.tokenizer.TokenizerNotAvailableError: Failed to locate a suitable tokenizer implementation for 'MY_MODEL' (Make sure your current environment provides a tokenizer backend like 'transformers', 'tiktoken' or 'llama.cpp' for this model)

Could you advise on what is supposed to go in tokenizer in the lmql.model() call? Is it supposed to match the model name (as above)? Or is it suggested to hard-code it to openai/gpt-3.5-turbo? When I do the latter, I get an error about a missing api.env, possibly because that hard-coding suggests that I am trying to access OpenAI directly.
Thanks!


Fix #1318

Coverage decreased (-0.3%) to 92.015% when pulling 68a1e412861c68744e1fbb93f1bd3af964db2dd6 on bpandola:fix-1318 into d187942eeb2efa3376ed4273a21bb1eb57a92c24 on spulec:master.

Coverage decreased (-0.3%) to 92.015% when pulling 68a1e412861c68744e1fbb93f1bd3af964db2dd6 on bpandola:fix-1318 into d187942eeb2efa3376ed4273a21bb1eb57a92c24 on spulec:master.

Travis timed out on lambda tests... will close and re-open PR.

Coverage remained the same at 92.313% when pulling 68a1e412861c68744e1fbb93f1bd3af964db2dd6 on bpandola:fix-1318 into d187942eeb2efa3376ed4273a21bb1eb57a92c24 on spulec:master.

Nice fix, @bpandola !


Pointcharge support

Hi, very nice project.
I'm developing ASH, a general QM/MM program with interfaces to many programs including pySCF: https://github.com/RagnarB83/ash
In order for gpu4pyscf to be applicable to QM/MM, basic pointcharge support would be required (i.e. for standard electrostatic embedded QM/MM).
As far as I can tell, a compatible pyscf.qmmm.mm_charge function would be needed:
currently we get an assertion error:
assert (isinstance(method, (scf.hf.SCF, mcscf.casci.CASBase)))
Is pointcharge support planned?

Thanks. It would be very interesting to see the application of GPU4PySCF in QM/MM. We should be able to complete the module near future.
The assertion error is raised because GPU4PySCF object is different from PySCF object. The PySCF module does not recognize GPU4PySCF object.


fix update jvm and webserver

The update of jvm and webserver was not allowing to update the json body. Now i fixed it by comparing the original jvm name with new jvm name.

good


[Core] Rename byte_mlperf to general task

In general, in order to support llm_perf / micro_perf in the feature, we need to reformat the current byte_mlperf and rename it to general_task

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update @balena/jellyfish-core from 16.0.0 to 16.0.1

Change-type: patch
Signed-off-by: Josh Bowling<EMAIL_ADDRESS>

@balena-ci I self certify!


Add GOVERNANCE.md and OWNERS files

Closes #263

Please let us know your thoughts @caniszczyk. The GOVERNANCE.md file is based on the one we use in Artifact Hub.


Improve the consistency in KEDA chart

Working with KEDA chart, I found that it hasn't got consistency in the names when it refers to each application, for example, sometimes the operator is named keda  and other times is named operator. With metrics server happens the same, sometimes is called as metricsApiServer, metricServer, metricsServer, metricsAdapter.
This is a breaking change, but maybe it could be in the next major release, and it reduces the complexity of the chart. IDK, maybe the chart values could be something like:
operator:
  resources: {}
  tag: {}
  logging: {}
  # .....
metricsServer:
  resources: {}
  tag: {}
  logging: {}

IMHO, at least the naming should be unified to reduce the general complexity
Wdyt?

Good feedback, adding it to milestone for when we plan a new major version!

Until then, we can change it so that it's done in a non-breaking manner but it can be quite messy so not sure if it's worth the time

Probably it's better if we wait until 2-3 weeks before the major release because in other case 2 charts should be maintained :/

During the context of this task, who will do it have to take in consideration that the resources are pending to split. They'd be separated here, but we decided to wait until this task to avoid one breaking change

This will indeed be a nice update!

I think that it's the moment to address this issue. I created it almost 2 years ago and we still don't plan a major KEDA release.
As we have recently introduced the "big changes" in the architecture and we don't plan to introduce new big changes soon, I'd prepare the v3 of the chart, refactoring the whole chart, solving all the gaps and duplications that we currently have.
Currently, we have a chaotic chart with a lot of parameters without any sense (at least for me)
WDYT @kedacore/keda-helm-contributors ?


Crash while making request to WIS

I got this error and the device rebooted (successfully) afterwards. It appeared to happen while it was processing the input audio.
I (12:02:06.269) WILLOW: AUDIO_REC_WAKEUP_START
I (12:02:07.792) WILLOW: AUDIO_REC_VAD_START
E (12:02:07.793) AUDIO_ELEMENT: [http_stream_writer] Element already stopped
E (12:02:07.795) AUDIO_ELEMENT: [raw_stream_writer_to_api] Element already stopped
I (12:02:07.804) WILLOW: Using WIS URL 'https://infer.tovera.io/api/willow'
I (12:02:07.813) WILLOW: WIS HTTP client starting stream, waiting for end of speech
Guru Meditation Error: Core  0 panic'ed (LoadProhibited). Exception was unhandled.

ESP-ROM:esp32s3-20210327

We had some downtime for an upgrade tonight on the best effort server. Have you repeated this?

No problems since then, it was just a one-off.
I haven't looked at the HTTP client code - is it fairly dependent on the WIS server returning a reasonable response at this point?

We have an open issue that we are in the process of addressing before our first stable release:
#25


Request: set CH max via MQTT

Hi,
I've been using your Opentherm board now for over a year in combination with Home Assistant and it works great.
To further tweak the heating I would like to be able to set the "CH max" value via MQTT. This would allow me to (slidely) increase the value with dropping outside temperatures.
Could it me made possible to set the CH max value via MQTT?
Thanks for your hard work.
Best regards,
Mozzy

Hi @Mozzzy ,
We plan to add this option in the next version.

We've added CHmax to the mqtt. You can use diyless_thermostat/cmnd/things/thermostat/properties/chmax topic to set the maximum heating water temperature. Full list of supported topics are in the blog.


Update dependency @types/mock-fs to v4.13.4

This PR contains the following updates:

Package
Type
Update
Change

@types/mock-fs (source)
devDependencies
patch
4.13.3 -> 4.13.4

Configuration
📅 Schedule: Branch creation - "before 4am on the first day of the month" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Self-Hosted Renovate Bot. Find all logs here.

 You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Update: Mirantis.Lens version 5.1.2-latest.20210719.1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

/azp run


Question: Recommended viewer for H2 database and fcrepo schema

Hi All - do you have a recommended viewer for the H2 database embedded in the webapp? Additionally, default connection settings and schema?

@ross-spencer You can connect to the H2 db by:

Downloading a copy of h2 (needs to match the version used by the version of fedora that you're using, which is 2.1.210 for the latest release): https://mvnrepository.com/artifact/com.h2database/h2/2.1.210
Launch the h2 web console (this should open a webpage in your browser): java -jar h2-2.1.210.jar
Open the db by specifying the jdbc url, for example jdbc:h2:/home/pwinckles/Downloads/fcrepo/fcrepo-home/data/fcrepo-h2. When you start fedora, there should be a log line like (DatabaseConfig) JDBC URL:.... Copy the value from there.
Remove the default user name
Click "connect"

Note, you can't have fedora running when you connect to the db.

Thanks @pwinckles I will give this a go!

Thank you @pwinckles, I'll make a page for this in the wiki too.


why "--cut_by_quality3"  reduce reads counts

hi sfchen:
 I want to known why "-3" option will reduce  reads passed filters.

fastp -i SRR1770413_1.fastq -I SRR1770413_2.fastq -q 20 -u 20  -o out.SRR1770413_1.fastq -O out.SRR1770413_2.fastq

fastp -i SRR1770413_1.fastq -I SRR1770413_2.fastq -q 20 -u 20 -3 -o out.SRR1770413_1.fastq -O out.SRR1770413_2.fastq

Your data is strange, PE 300?
Can you post the complete HTML files here?

@sfchen
yes, the data is 300 PE, download from  SRA

the two html reports are in attached file
fastp.zip
thank you!

Thanks, I will try to look into the data when I have time.

This seems like a small bug about quality filtering, I just fixed it, could you please try to build fastp with latest code on master. Or download http://opengene.org/fastp/fastp to test.

Thank you for your prompt reply.
the reads counts is the same for the two conditions "-q 20 -u 20" ,"-q 20 -u 20 -s 3", but the total bases After filtering  is identity too, there is no per read cutting by quality in tail (3').
when combined the two filters, Is there a priority for filtering?

You should use "-q 20 -u 20 -3", not "-q 20 -u 20 -s 3"

Can you close this issue right now?


⚠ SceneTime has degraded performance

In 1ce9b10, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1732 ms

Resolved: SceneTime performance has improved in 2f3baf6 after 12 minutes.


Security 3.7c: Manage Samples - Enforcement of 200-level security points

The relevant security points are as follows.

Security point
Description

200
Can access Manage Samples activity

210
Create samples

220
Edit only samples created by self

230
Edit all samples

240
Delete only samples created by self

250
Delete all samples

200 Tasks:

app_server.py: Under route for Manage Samples, copy 403 abort code as in route for Manage Plate Templates
Ensure Manage Samples is under correct menu (Edit as a non-admin activity)
Modify base.html to add link based on security point
Modify logic to determine if Full menu should appear

200 Test Plan:

If you open /manageSample route without security, 403 error is returned
Can see Manage Samples link iff 200 is present
Can see Edit menu iff one of (100, 110, 200) is present
Can open activity if 200 is present

210 Tasks:

app_server.py: Under /editSample route, check for id of -1 and if so, verify security point 210 - TEST before proceeding
In html, disable create button as on bottom of manage_plate.html if 210 is absent

210 Test Plan:

BEFORE form disabling of create button, can create new record iff 210 is granted; message is appropriate; db is changed only as appropriate
AFTER form disabling of create button, can press button iff 210 is granted; workflow functions correctly when creating a new record and db is updated properly

220 and 230 Tasks:

Write dao function to retrieve creating user for a given record
app_server.py: Under edit sample route, check for id != -1 and if so, verify security point 220 and 230 - TEST before proceeding
- If have 230, don't need to figure out who created record nor verify 220
- If not have 230, check who created record and verity 220
In manage_samples.html, disable Edit button and make all fields except the selector read-only, as on bottom of manage_plate.html, if do not have 210, 220, or 230

220 and 230 Test Plan:

BEFORE form disabling, test edit record behavior (message is appropriate; db is changed only as appropriate)

220 and 230: Can edit any sample
230: identical to previous case
220: can edit any sample created by self, cannot edit samples created by others
neither of 220 or 230: cannot edit any record
Behavior of creation wrt 210 is not affected by above cases
Works for multiple users
Works for multiple team login contexts

Repeat tests AFTER form changes

can press Edit button and edit fields iff have one of 210, 220, or 230

240 and 250 Tasks:

app_server.py: Under delete sample route verify security point 240 and 250 - TEST before proceeding
- If have 250, don't need to figure out who created record nor verify 240
- If not have 250, check who created record and verity 240
In manage_plate.html, disable delete button, as on bottom of manage_user.html, if do not have 240 or 250

240 and 250 Test Plan:

BEFORE form disabling, test delete record behavior (message is appropriate; db is changed only as appropriate)

240 and 250: Can delete any sample
250: identical to previous case
240: can delete any sample created by self, cannot edit samples created by others
neither of 240 or 250: cannot delete any record
Behavior of creation and editing is not affected by above cases
Works for multiple users
Works for multiple team login contexts

Repeat tests AFTER form changes

can press delete button iff have one of 240 or 250

Final Tests:

Repeat all test plans

200 Test Plan - SUCCESS!
210 Test Plan - SUCCESS!
220 & 230 Test Plan - SUCCESS!
240 & 250 Test Plan - SUCCESS!
Need to execute final tests.

Final tests - SUCCESS!


Update the data extraction pipeline

Now we use nltk and PyMUPDF to extract text from pdf and do tokenization, which seem out of date. We should use more modern tools in our pipeline, which will provide convenience for the latter application of pretrained models and achieve better performance. Now we can consider similarities and sentencepiece. This work can be a long term run.

sentencepiece needs formatted corpus which is hard to construct. Consider using existing models instead of training our own.

We should try to remove References in text, which have little relevance with the documents level semantics and could affect the retrieval process.

To train a sentencepiece tokenizer, we need a corpus. Text data from pdfs is hard to clean, so we can use abstract, which is easy to   split into sentences. But  we need more abstracts than only cs.IR.

Use langchain to optimize the existing pipeline. We don't need to write abstract crawling function ourselves.

Now we only store metedata and raw text. langchain has textspliter module to split text. It can be used in data processing stage.


Make getprinc raise an exception on insufficient privileges

Previously, getprinc would silently failed if permissions were missing. With this change, such errors can nicely be detected by the user using code such as:
try:
return kadm.getprinc(princ)
except kadmin.AuthGetError:
...

This looks fine, it is likely that a universal get_princ wrapper should implement this exception for all locations where the principal is retrieved from the server.


CI: IPSec tests are skipped when running with our kube-proxy replacement

https://github.com/cilium/cilium/pull/13241#issuecomment-696523262

By the way, I noticed the IPSec tests are skipped when running with our kube-proxy replacement. Why is that? I think that's why we missed this regression, because we only run with newer kernels in CI builds where we also run without kube-proxy.

@jrfastab are you aware of this?

It's because NodePort BPF cannot be used together with IPSec, see https://github.com/cilium/cilium/pull/12985.

Could we switch one of our 4.19 or net-next CI pipelines to run with kube-proxy? That would allow us to (1) run tests that are incompatible with our kube-proxy replacement on newer kernels and (2) have tests for Cilium's kube-proxy replacement when kube-proxy is also present on the node.

@pchaigno SGTM. Also, this would help to test what new deployments with default settings on newer kernels are running  (kpr=probe + kube-proxy). I'd nominate 4.19 for it.


IPv6: Validation of proxy addresses

During PoC work to try Istio with IPv6, the validation of IPv6 proxy addresses (statsd in particular) was failing, due to the code's inability to handle IPv6 addresses. The logic just scans for a colon to determine the split point between IP and port, and this doesn't work with IPv6 addresses.
I'll push up a PR in the next few days that will address this issue in pilot/pkg/model/validation.go

We plan to shortly assign new Istio contributors to start work on this,  but if anyone else is interested please go ahead


adds conda recipe

Adds a simple conda recipe. Building the recipe also runs a test procedure:

creates a test environment
installs fisx and its requirements
attempts to import fisx
runs the test suite

The test suite reports some errors that I don't believe are specific to the conda package:
AssertionError: unassigned fisx.DataDir.DATA_DIR
AssertionError: unassigned fisx.DataDir.DOC_DIR
As far as I can tell, there is nothing in this package to set DATA_DIR and DOC_DIR. DataDir does contain FISX_DATA_DIR and FISX_DOC_DIR variables.

Thanks, I have corrected the test suite too. Unless you have in mind other changes, I would like to generate a 1.0.1 version.


vm: Remove runInWindowsVM implementation

Originally this was meant to support other Windows versions than just Windows XP, but before I actually got a chance to implement this I left the project that I implemented this for.
The code has been broken for years now and I highly doubt anyone is interested in resurrecting this (including me), so in order to make this less of a maintenance burden for everybody, let's remove it.

For future reference, https://git.m-labs.hk/M-Labs/wfvm seems to do something similar, but supports modern windows versions.


Silverlight does not installed

Severity	Code	Description	Project	File	Line	Source	Suppression State
Error		Could not install package 'Splat 1.0.0'. You are trying to install this package into a project that targets 'Silverlight,Version=v5.0', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.			0	IntelliSense

Splat does not support Silverlight. Either you are trying to use RxUI in a Silverlight project, or you have a PCL that includes Silverlight support. The fix is to stop targeting Silverlight.


Matrix format of pfms in motif pfms directory

Hi!
The format for my pfm files in my motifs/pfm/ directory are of MEME format (example below):
MEME version 4

ALPHABET= ACGT

strands: + -

Background letter frequencies
A 0.25 C 0.25 G 0.25 T 0.25

MOTIF MA0002.2 RUNX1
letter-probability matrix: alength= 4 w= 11 nsites= 2000 E= 0
 0.143500  0.248000  0.348000  0.260500
 0.117000  0.242500  0.233500  0.407000
 0.061500  0.536000  0.074500  0.328000
 0.028500  0.000000  0.003500  0.968000
 0.000000  0.037500  0.936000  0.026500
 0.043500  0.063500  0.035000  0.858000
 0.000000  0.000000  0.993500  0.006500
 0.008500  0.021000  0.924000  0.046500
 0.005000  0.200000  0.125500  0.669500
 0.065500  0.231500  0.040500  0.662500
 0.250000  0.079000  0.144500  0.526500
URL http://jaspar.genereg.net/matrix/MA0002.2

However, I'm getting errors from line 146 in report.py when it tries to read these files ValueError: could not convert string 'MEME version 4' to float64 at row 0, column 1. For these pfm files do they just need the matrix of numbers alone? I'm using the JASPAR Core (2022) for vertebrates database and the only download format options they have for the pfms are JASPAR, MEME and TRANSFAC. Could you possible provide an example format for the pfm so I know how to format them accordingly?
Cheers!

Yes, the /pfm/ directory has to have the extracted matrices in numpy array format. I'm currently travelling but when I get back I was going to add a way to just take in the meme file. It was originally programmed to take in both so that it would be fast to load each motif, but ultimately I don't think it should take that much time relative to the FIMO time.

Hi, @jmschrei thanks a lot for this rework, it is much more manageable than the original version. Providing only the batch meme file would be nice, as this is a bit redundant.
In the meantime, @due23 and anyone looking, through some trial and error I got it to work with files having only the numbers:
0.333401	0.598017	0.933644	0.881044	0.027918	0.017398	0.014364	0.021849	0.975926	0.572324	0.346753
0.218895	0.091240	0.025895	0.008092	0.952053	0.961966	0.964394	0.008901	0.006878	0.075258	0.217075
0.219907	0.137973	0.019421	0.089622	0.007081	0.007283	0.011127	0.011531	0.008901	0.263605	0.095893
0.227797	0.172770	0.021040	0.021242	0.012948	0.013352	0.010115	0.957718	0.008295	0.088812	0.340279

where the delimiter is \t (note the shape 4 x m not m x 4).

This should be fixed in v2.0.4. Now, you only need to pass in the MEME file. Your command will look something like:
 modisco report -o test/ -s test/ -i modisco_results.h5 -m motifs.meme.txt

You can get the latest with pip install --upgrade modisco-lite


More info

Would be nice to have an explanation as to why the azuread cmdlet returns far fewer roles than the msolrole cmdlet. Now I am thinking it might just be there is only so much you can interact with via the Graph (may increase over time) but I would like to know, rather than assume :) As here this is not just two different methods, it provides two completely different sets of results, meaning this is a scenario where you would need to use one over the other, rather than just a preference thing

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

ID: cbee503e-39f0-b277-20a5-659f1255d775
Version Independent ID: 884eeb43-1b0b-e6fc-a405-2d50d646819e
Content: Assign roles to user accounts with Office 365 PowerShell
Content Source: Enterprise/powershell/assign-roles-to-user-accounts-with-office-365-powershell.md
Service: o365-administration
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

@officedocsbot assign @mypil

@henryphillipsid - Thank you for submitting feedback.
I will get this issue over to the Enterprise writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@henryphillipsid The output of Get-AzureADDirectoryRole is the list of currently installed Azure AD role templates. The updated article links to the definitive list of role names and the new block adds the role template if it doesn't already exist. Thanks for pointing this out.


Command line fixes

Fixes a bug when passing the resource directory path per command line, closes #18068
Also fixes help text and removes an unnecessary restriction on runnable presets.

fixed

Note for @hpvb: 942f6df should not be cherry-picked (was reverted in #18810), but the other two should be fine.

Cherry-picked for 3.0.7 (apart from 942f6df).


Implement cairo1 deploy syscall

Implements cairo1 deploy syscall. It is interesting because it uses state_diff to prepare os_hints data.

FYI, I have some work on Cairo0 and Cairo1 declare tests in #177


Build fails with "fatal error: textflag.h: No such file or directory"

Following the README, I tried this:
go get -u github.com/nmrshll/gphotos-uploader-cli/cmd/gphotos-uploader-cli

That fails with:
# github.com/steakknife/hamming
../../go/src/github.com/steakknife/hamming/popcnt_amd64.s:11:10: fatal error: textflag.h: No such file or directory
 #include "textflag.h"
          ^~~~~~~~~~~~
compilation terminated.

It's solved!


Bridged JS Module does not work when placed inside a function

I believe this is more related to my lack of knowledge in Python rather than an issue with JSPyBridge and I would be more than happy to delete my comment if this is out of scope for submitting an issue.
I am trying to use a JS module inside python and to my relief that it works perfectly fine. So first of all thank you for that. My question is that when I place the module inside a function it does not behave accordingly. For example this:
a = web3JS.eth.abi.encodeFunctionCall({
    "name": "name",
    "type": "function",
    "inputs": []
}, [])

works perfectly fine while this:
def encodeBasicFunction(funcName):
    web3JS.eth.abi.encodeFunctionCall({
        "name": funcName,
        "type": "function",
        "inputs": []
    }, [])
a = encodeBasicFunction("name")

does not work.
Like I've said I'm happy to remove my issue if this is too basic/unrelated to JsPyBridge and more of a StackOverflow question.
Cheers!


Refresh open-node grant

This PR aims to give a refresh of the open-node-framework grant.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @h4x3rotab. A few comments:

Are all of the features you want to implement not present in polkadot-deployer, and otherwise could you specify which are and which aren't (especially the list under "Open Node Framework wants to feature"?
Is the grant still related in any way to your Open Node MVP? You say you want to work on a fork of polkadot-deployer but the document still mentions the former in multiple places (e.g. "we are going to expand Open Node Framework in aspects...").
The link on line 49 is broken. Where is that repository?
Looping in @ironoa who is maintaining polkadot-deployer in case he wants to add anything, in particular on contributing to upstream.

Oops, made a mistake. Now reopening

Thanks. Could you avoid force-pushing, so we can more easily see changes, from now on?

Replied in-line:

Thanks @h4x3rotab. A few comments:

Are all of the features you want to implement not present in polkadot-deployer, and otherwise could you specify which are and which aren't (especially the list under "Open Node Framework wants to feature"?

Slightly adjusted the deliverables table, and marked the features with "addition" or "integration" to address the relationship with the upstream.

Is the grant still related in any way to your Open Node MVP? You say you want to work on a fork of polkadot-deployer but the document still mentions the former in multiple places (e.g. "we are going to expand Open Node Framework in aspects...").

We have decided to not continue to work on the MVP repo. Instead, we have forked polkadot-depoyer here (https://github.com/Phala-Network/open-node-deployer) and will cherry-pick some of the existing code when necessary.

The link on line 49 is broken. Where is that repository?

It's a private repo, shared with the reviewers in the past. I can add you to the viewer if required.

Thanks, looks good! As far as I understand you have one person mainly working on this - could you adjust the FTEs (should be 1)?
And yes please, I would be happy to have a look at the MVP.

Thanks, looks good! As far as I understand you have one person mainly working on this - could you adjust the FTEs (should be 1)?
And yes please, I would be happy to have a look at the MVP.

The staffing is rather flexible. We have one FET who leads a team of 4. Do you intend to change the amount of the grant? I think the work loads match the proposed amount, and actually the proposed budget is already very tight from our end. We just want to be more focus. So we don't want to re-negotiate the amount.


Wolicheng.WoMicDriver version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I don't think there is a silent installer for this app/driver.

@quhxl
When I tried to install it gets succesfully installed but at the end of the setup it the program asks if the user wants to check for updates
And after that it quits with error 2

I think that's enough. It's not completely silent.

This is actually why it fails, but this is a dependency that we can't install i believe, so this might have to go under not 100% unattended since as you said this also pops up


Why callback webScoketclient's onClose() so immediately when close network by myself on Android phone

I finished my development tasks that dependency your lib in Android platform . but I have a question
: why callback onClose  immediately when close network by myself on Android phone? do you have some special implements about network listener on Android platform？

sorry ，my English is not very well .I mean is the library can auto to callback  webScoketclient's onClose() and its so immediately when close network by myself on Android phone

Hello @VayneKaso,
this is most likely due to the underlying network layer directly indicating a disconnect.
There is nothing special this lib does on android, especially since it is mainly targeting Java.
Greetings
marci4

thanks for your reply
I want to know in this lib . there are some frameworks or operates that likely loop pull to check the network state?

Hello @VayneKaso,
no, we are relying on the InputStream.read() method.
Greetings
marci4

@marci4 请问你解决了这个断网socket立马断开的问题了么

@VayneKaso 请问你解决了断网socket立马断开的问题了吗


Support interrupts

We currently run without interrupts, which is not going to cut it moving forward.
A VIRQ_TIMER interrupt handler would be great interrupt feature to support. Before that, though, we'll need to have general support for Xen's callback system.

This has been implemented!