Increasing memory usage on writes

It looks like we might have a memory leak somewhere involving writes. A simple loop like this running on Android shows a small continuous increase in memory usage
viewModelScope.launch {
   while(true) {
       realm.write {
           copyToRealm(Sample().apply { stringField = Utils.createRandomString(1024*1024) }
       }
   }
}

It can take several minutes (if not more for it to show, so I am still not 100% sure there is a leak, but we have a customer is running into crashes after several hours with a loop that writes 25 times pr. second)

After more testing, it seems to be related to the number of active versions:

Modifying the code to
  GlobalScope.launch(Dispatchers.Default) {
    while(true) {
      val p: Float = Random.nextFloat()
      val now = Instant.now()
      viewModelScope.launch(Dispatchers.IO) {
          realm.write<Unit> {
            copyToRealm(Pressure().apply {
              timestamp = now.toEpochMilli()
              hPa = p
            })
          }
      }
      _text.postValue("Version: ${realm.getNumberOfActiveVersions()}")
    }
  }

Show that the number of active versions keep increasing even though nothing should hold on to them. My best guess is that that it is our internal GC that isn't fast enough to keep up with the massive number of references being created.

Surprisingly, the above run actually eventually seemed to catch up:

But then crashed because it went OOM:
 java.lang.OutOfMemoryError: Failed to allocate a 24 byte allocation with 1744152 free bytes and 1703KB until OOM, target footprint 201326592, growth limit 201326592; failed due to fragmentation (largest possible contiguous allocation 0 bytes). Number of 256KB sized free regions are: 0
                                                                                at com.oliverclimbs.realmtest.ui.home.HomeViewModel$1$1.invokeSuspend(HomeViewModel.kt:67)
                                                                                at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
                                                                                at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
                                                                                at kotlinx.coroutines.internal.LimitedDispatcher$Worker.run(LimitedDispatcher.kt:115)
                                                                                at kotlinx.coroutines.scheduling.TaskImpl.run(Tasks.kt:100)
                                                                                at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:584)
                                                                                at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:793)
                                                                                at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:697)
                                                                                at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:684)
                                                                                Suppressed: kotlinx.coroutines.internal.DiagnosticCoroutineContextException: [StandaloneCoroutine{Cancelling}@85dc4cc, Dispatchers.IO]

After looking more into this, it looks like this behavior can be explained by our RealmFinalizer thread either not keeping up or pointers simply not being GC'ed in time.
By adding an atomic counter to the finalizer, I was able to show that the reference queue keeps growing with the following code:
    GlobalScope.launch(Dispatchers.Default) {
      while(true) {
        val p: Float = Random.nextFloat()
        val now = Instant.now()
        viewModelScope.launch(Dispatchers.IO) {
            realm.write<Unit> {
              copyToRealm(Pressure().apply {
                timestamp = now.toEpochMilli()
                hPa = p
              })
            }
        }
        withContext(Dispatchers.Main) {
          _text.postValue("Version: ${realm.getNumberOfActiveVersions()}")
        }
      }
    }

I could see incremental bursts of things being GC'ed, but the overall trend was that the queue kept growing and growing.... just pausing the writes didn't help either. My guess there is that the memory allocator didn't consider the many thousand NativePointers important enough to GC.
The result of this would either be that 1) We ran out of Disk space because the Realm file kept growing (because of unclaimed versions) or 2) We went OOM because we exhausted the native memory space.
Only by stopping the writes and then manually calling the GC was I able to empty the queue.
This is not ideal in "fast write"-scenarios like listening to sensor updates.
I tried to modify our GC thread to have Max priority. This seemed to help a little bit, but the queue of pointers was still growing.
So right now I guess that for these scenarios, we need some sort of "allocation-free" insert, or at least an insert that automatically cleans up as soon as the write is completed.
In Realm Java we have a bulk insert method called insert() that does this, and it is tracked for Kotlin here: https://github.com/realm/realm-kotlin/issues/959. I would guess this kind of method would also fix the problem described in this issue.

Hi, we moved to Realm Kotlin from Realm Java in our project and we see 10-30% increase in memory usage.
Also I checked memory usage in a test app and I observed similar memory increase 10-30% depending on the use case. For example, this use case seems to indicate a memory leak:

Open Realm
Observe ~5000 entities with 100 queries
Write 1000 entities
Close Realm
Close the app
Trigger GC few times from profiler

This usecase shows that native memory size doesn't decrease after closing Realm and triggering GC.
Additionally, I profiled native memory allocations using Perfetto (heapprofd) (https://perfetto.dev/docs/case-studies/memory#heapprofd). It showed 4 MB memory leaks in realm_results_count, realm_results_resolve_in. I’m not sure what might be causing the leak in realm_results_count.
It also seems calling toList() on RealmResult<> increase memory leakage. Calling toList() invokes RealmResultsImpl::size matching size of RealmResult (see jvm implemention of toList() https://github.com/JetBrains/kotlin/blob/master/libraries/stdlib/jvm/runtime/kotlin/jvm/internal/CollectionToArray.kt#L72).
Also no leaks are visible in java/kotlin heap.
Here is a test project I used for profiling: https://github.com/santaevpavel/realm-benchmark
Realm version: 2.3.0 and 3.0.0
Kotlin version: 2.0.20
AGP version: 8.3.2

I also tried to figure out what makes memory usage high when toList() is called on RealmResults.

Calling size() on RealmResultsImpl invokes native function realm_results_count and passes pointer to Results (from realm core) with query mode (m_mode == Mode::Query).
realm_results_count invokes Result::size() which invokes Results::do_size().
do_size() calls m_query.count(m_descriptor_ordering).
m_query.count() creates TableView and calls apply_descriptor_ordering and size().
TableView::apply_descriptor_ordering() calls TableView::do_sync which iterates through all entities and fills m_key_values.

So iterating RealmResults with N elements  (or calling public fun <T> Iterable<T>.toList(): List<T> on RealmResults) calls size() N times (iterator of AbstractList calls size() on every hasNext() call) that iterates over elements N*N times (or calls do_sync() N times that is not looks ok). It may explain why we see memory usage peaks if toList() is called.
I wasn't able to debug it as native calls can't be debugged without sources. Also I tried to run android tests in this repository, but it didn't work. So basically it's just my thoughts. @nhachicha could you check it or confirm that the problem that I described is correct?

I've found what's is leaking. It's happening in JNI layer that is generated by SWIG. Array of longs are not released. I've already found a fix. I will open a PR.


VSCode Extension - When a prefix is used, the IntelliSense is not work

UnoCSS version
@unocss/nuxt: ^0.58.3
Describe the bug
Before set prefix

uno.config.ts
import {
  // ...
  presetAttributify
} from 'unocss'

export default defineConfig({
  // ...
  presets: [
    presetAttributify({
      prefix: 'u-',
      prefixedOnly: false
    })
  ],
  // ...
})

After

Reproduction
uno.config.ts
import {
  // ...
  presetAttributify
} from 'unocss'

export default defineConfig({
  // ...
  presets: [
    presetAttributify({
      prefix: 'u-',
      prefixedOnly: false
    })
  ],
  // ...
})

app.vue
<template>
  <div
    u-flex
    u-flex-wrap
  >
  </div>
</template>

System Info
VS code
Windows11
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

VSCode Extension:
UnoCSS
v0.58.3

https://github.com/unocss/unocss/blob/main/packages/vscode/src/utils.ts#L23C29-L23C37
It seems that there is no way to correctly parse the element with prefix. I wonder whether I should use regular expressions to erase the content of prefix and then parse it again.
Do you have any good advice? @antfu


Module 2, Exercise: create and export a configuration package

Module: 02
Exercise: Create and export a configuration package
Task: 04 prepare data in excel
Step: b
Description of issue
Enter the information as provided by contoso... what information? there is nothing in the lab to show what information needs to be provided. Please make it clear - this step refers to the vendor import file.
Now i have located the file, i found out that my lab environment does not allow me to edit the excel file. Is this an error in the setup by the lab provider?

The excel is data is now updated in the Lab guide accordingly.

that's great. thank you.


Blue decile lines are in front of the red ones

It (suddenly) seems difficult to hover over the red lines where they are close to blue ones... e.g. https://openprescribing.net/practice/C84023/measures/ - like the blue ones are in front of the red ones. I don't think this has been like this for long!

Note similar issue also applies to the analyse page: the trendline for selected org(s) not in front of others so can be impossible to see values on hover

Here's an example: the measure on the front page for this practice has it's line under a decile line for much of the last few months meaning that you can't hover to see how it's been changing: https://openprescribing.net/practice/K84021/


first pass at S3-backed BinaryObjectCache

This needs the following appSetting (note the bucket name rather than directory path)
...
  "BinaryObjectCache": {
    "StorageMaps": {
      "AvoidCaching": false,
      "AvoidSaving": false,
      "WriteFailThrowsException": true,
      "Folder": "wellcomecollection-stage-iiif-storagemaps",
      "Prefix": "stgmap-",
      "MemoryCacheSeconds": 60
    }
  },

(Suggest changing Folder to Container - I have made a similar change elsewhere in this PR to use language that sort of works for files and folders as well as buckets and keys).

The appsettings in initial comment can go into appsettings.staging.json


Fix plugins/title for titles containing \n characters

http://imgur.com/h2zXk was used for testing.


📎 derive(biome_deserialize::Deserializable): add support for ignoring unknown fields

Description
This could help to address #1898.
Currently, biome_deserialize::Deserializable doesn't allow unknowns fields.
By default serde allows unknown fields. A user have to add the serde(deny_unknown_fields) to forbid unknown fields.
There are two possible approaches:

Keeping our current default and add the struct (container) attribute deserializable(allow_unknown_fields) to allow unknown fields.

Adopting serde's defaults and support both serde(deny_unknown_fields) and deserializable(deny_unknown_fields).

A third option is to introduce unknown_fields attribute that takes a value among deny, warn and allow. serde(deny_unknown_fields) could be equivalent to unknown_fields = deny. We could default to unknown_fields = warn.
@arendjr @ematipico any opinion?

I like your last proposal.
As for the default behaviour, I don't have strong preferences. I would favour the one that makes less changes to our code base.


Fix version links

Fixes https://github.com/verdaccio/website/issues/178

@sergiohgz @VentyCZ unfortunately this PR breaks the master, we need to undo this changes and try to find a different solution
rm 'zh-CN/versions.html'
rm 'zh-CN/versions/index.html'
Error: Copying build assets failed with error 'Error: ENOENT: no such file or directory, stat 'build/website''
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Exited with code 1

@juanpicado Yeah, I saw that. Will try to fix it other way, probably use the repoUrl directly from the config - did not see that before.

@juanpicado you are right, we have to make better CI in this repo. @VentyCZ let me a moment to revert master branch :wink:

@sergiohgz @juanpicado Ok, created another PR for this: https://github.com/verdaccio/website/pull/182


Remove "Full Bleed" option

Feature Description

Per comment in #1840, after UX revisions and review the full bleed option is considered to have more cons and pros. The user can always achieve a similar result by expanding the element manually, the "full bleed" option/shortcut value add is limited.

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Verified in QA:


Export Excel just first line ?

When i add export Excel server side i have just the first line in in the downloaded file.
So i followed a the problem i found you to limit the result. is there a reason or is a bug.
File : vendor/omines/datatables-bundle/src/Exporter/DataTableExporterManager.php
Code :
/**
* Browse the entire DataTable (all pages).
*
* A Generator is created in order to remove the 'DT_RowId' key
* which is created by some adapters (e.g. ORMAdapter).
*/
private function getAllData(): \Iterator
{
$data = $this->dataTable
->getAdapter()
->getData($this->dataTable->getState()->setStart(0)->setLength(-1))
->getData();
    foreach ($data as $row) {
        dd($data);
        if (array_key_exists('DT_RowId', $row)) {
            unset($row['DT_RowId']);
        }

        yield $row;
    }
}

Solution : when i remove "setLength(-1)" i have what i want even if i sort.

Hi @SlimArfaaoui,
$this->dataTable->getState()->setStart(0)->setLength(-1) : this is supposed to retrieve the entire table (otherwise it would only export the current page).
Are you able to reproduce your issue with a test case?

Hello @MaximePinot,
Yes i reproduce the error in a test case.
I think i see now whats is the problem :
its about limit in MongoDB to get all data, the value to set no limit is not -1 like SQL is 0.
Can you please confirm this.
Thank you

This issue was actually fixed in the 'breaking changes' of the 0.7 release following #257 - we changed semantics for 'no limit' to the correct null instead of -1 which was indeed adapter specific.
Adapters now all follow this rule and this issue should be fixed after that.


SyntaxError: Use of const in strict mode.

attempting to install on Windows 2012
node 0.12.2
npm 3.10.10
`PS C:\Windows\system32> npm install --global windows-build-tools

<EMAIL_ADDRESS>postinstall C:\Program Files\nodejs\node_modules\windows-build-tools
node ./lib/index.js

C:\Users\dev-services\AppData\Roaming\nvm\v0.12.2\node_modules\windows-build-tools\node_modules\fs-extra\lib\index.js:3
const assign = require('./util/assign')
^^^^^
SyntaxError: Use of const in strict mode.
at exports.runInThisContext (vm.js:73:16)
at Module._compile (module.js:443:25)
at Object.Module._extensions..js (module.js:478:10)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at Object. (C:\Users\dev-services\AppData\Roaming\nvm\v0.12.2\node_modules\windows-build-tools\lib\utils.
js:3:10)
at Module._compile (module.js:460:26)
at Object.Module._extensions..js (module.js:478:10)
npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\Program Files\nodejs\\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "i
nstall" "--global" "windows-build-tools"
npm ERR! node v0.12.2
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: node ./lib/index.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node ./lib/index.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the windows-build-tools package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./lib/index.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs windows-build-tools
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls windows-build-tools
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Windows\system32\npm-debug.log`

Please update to a recent Node version. 0.12 is no longer supported by Node and neither by this module.
https://github.com/nodejs/LTS

Does Not Work

same issue on  both Node LTS and latest:
node --version
v8.2.1
`
Starting installation...
events.js:182
throw er; // Unhandled 'error' event
^
Error: spawn powershell.exe ENOENT
at exports._errnoException (util.js:1024:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:192:19)
at onErrorNT (internal/child_process.js:374:16)
at _combinedTickCallback (internal/process/next_tick.js:138:11)
at process._tickCallback (internal/process/next_tick.js:180:9)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: node ./lib/index.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
`

same issue here, run cmd as Administartor fixed the problem.

@pestig not work to run as Admin for both cmd, powershell, and any other terminals

@erikusaj's problem appears to be caused by powershell.exe not being in the user's path (see https://github.com/felixrieseberg/windows-build-tools/issues/20).
@cnscorpions Would you mind opening a new issue with your specific details? Ensure you are using Node >= 8.x, start a PowerShell session as Administrator, and run npm install --global windows-build-tools. If it fails please share the output as well as any other relevant information.


Basic E2E Test using ibctest library

Description

closes: #1583

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #1553 (37862b3) into main (71d7480) will increase coverage by 0.03%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main    #1553      +/-   ##
==========================================
+ Coverage   80.29%   80.33%   +0.03%
==========================================
  Files         166      166
  Lines       12304    12346      +42
==========================================
+ Hits         9880     9918      +38
- Misses       1959     1963       +4
  Partials      465      465

Impacted Files
Coverage Δ

...ules/apps/27-interchain-accounts/types/metadata.go
80.19% <0.00%> (-3.31%)
:arrow_down:

...interchain-accounts/controller/keeper/handshake.go
87.87% <94.44%> (+1.43%)
:arrow_up:

...7-interchain-accounts/controller/ibc_middleware.go
83.63% <100.00%> (+0.30%)
:arrow_up:

modules/apps/29-fee/keeper/events.go
100.00% <100.00%> (ø)

modules/apps/29-fee/keeper/msg_server.go
96.20% <100.00%> (+0.20%)
:arrow_up:

Codecov Report

Merging #1553 (1efd8d1) into main (e7d5bda) will increase coverage by 0.00%.
The diff coverage is 85.00%.

@@           Coverage Diff           @@
##             main    #1553   +/-   ##
=======================================
  Coverage   80.33%   80.33%
=======================================
  Files         166      166
  Lines       12346    12363   +17
=======================================
+ Hits         9918     9932   +14
- Misses       1963     1965    +2
- Partials      465      466    +1

Impacted Files
Coverage Δ

modules/core/02-client/keeper/proposal.go
86.36% <50.00%> (-3.81%)
:arrow_down:

modules/apps/29-fee/keeper/events.go
100.00% <100.00%> (ø)

modules/core/02-client/keeper/events.go
100.00% <100.00%> (ø)

is there anything we need to do to get this merged?

@jackzampolin this PR was actually a reference PR which is why I kept it as a draft, the test funcitonality has already been merged. I'll close this as it's not needed anymore!


Legend out of a map

Hello,
please we would need Legend component from Widgets put out of a WebScene. Is that possible?
Actually we have reach our goal to create it out of the WebScene container through the 'view' attribute but after its loaded the legend is shown still in the Map.
We really appreciate your work!
Thank you,
Jakub

There is whole code of this component. This solution works but it is very bad to move those element in React. Is there any chance to set an ID of container where will be the Legend created?
import React, { Component } from 'react';
import { WebScene, Widgets } from 'react-arcgis';

const Search = Widgets.Locate;
const Legend = Widgets.Legend;

export default class Map extends Component {
    constructor(props){
        super(props);
        this.state = {
            map: undefined,
            view: undefined,
            position: undefined,
        };
    }

    componentDidMount(){
        console.log('nactena componenta');
    }

    onMapLoad = (map, view) => {
        console.log("nactena mapa");
        this.setState({
            map: map,
            view
        });
    }

    onLegendLoad = () => {
        document.getElementById('legenda').appendChild(document.getElementById('legendaFlexible'));
    }

    render () {
        return (

                <WebScene
                    id={this.state}
                    style={{ display: 'block', width: '100%', height: '90vh'}}
                    onLoad={this.onMapLoad}
                >
                    <Search  />
                </WebScene>

                    {this.state.view &&
                        <Legend onLoad={this.onLegendLoad} widgetProperties={{ container: 'legendaFlexible' }} view={this.state.view} />
                    }

        )
    }
}

@baierjak , the ArcGIS API for JavaScipt allows you to set a specific container element for the legend either during construction or after the widget has been instantiated.
You will need to use esriPromise to load the legend module from the API, then add it to your map with the desired container. Here's a link to the ArcGIS JS API docs for this


Add 5 new icons

Minified with optipng -o7.
:eyeglasses: @keybase/react-hackers

👍 just an fyi imageoptim (on top of your already compressed assets here)

Updated w/ advpng compression.

👍


The Raw Serialisers do not preserve object ordering

The Raw Serialisers (RawLongSerialiser, RawDateSerialiser, etc.) do not always preserve object ordering.
For example if you serialise the following longs you, they should be serialised into bytes that maintains the same ordering.

1496709602656   ->  [96, 29, -40, 122, 92, 1, 0, 0]
1496710980959   ->  [95, 37, -19, 122, 92, 1, 0, 0]
1496710980960   ->  [96, 37, -19, 122, 92, 1, 0, 0]
1496710980961   ->  [97, 37, -19, 122, 92, 1, 0, 0]

So the values of the serialised bytes ends up reordering the values into:

1496710980959   ->  [95, 37, -19, 122, 92, 1, 0, 0]
1496709602656   ->  [96, 29, -40, 122, 92, 1, 0, 0]
1496710980960   ->  [96, 37, -19, 122, 92, 1, 0, 0]
1496710980961   ->  [97, 37, -19, 122, 92, 1, 0, 0]

The ordering of the serialisers is important when using an ordered database/store, like Accumulo or HBase. If using either of these Gaffer Stores, you should use an serialiser that preserves object ordering for your vertices. If your vertices are serialised on an ordered fashion then you are able to do range scans.
It is advised that Properties included in the groupBy should also be serialised with an ordered serialiser to help maintain true ordering of the Accumulo/HBase row keys. However, this is not essential, but you will not be able to carry out range scans over other columns in the row key (column qualifier/visibility).
To fix this bug, we should deprecate all of the Raw Serialisers and write new serialisers based on:
https://github.com/apache/accumulo/blob/f81a8ec7410e789d11941351d5899b8894c6a322/core/src/main/java/org/apache/accumulo/core/client/lexicoder/ULongLexicoder.java.
We will need to update the Serialisation Factories, removing the deprecated serialisers and adding the new ones. This will break the serialisation of all the serialised data that has been serialised with the Raw Serialisers and have not explicitly defined them in their schema. This would occur due to new serialisation factory selecting the new serialiser, which will not be compatible with the existing serialised bytes. We must inform users that if they have an existing table in accumulo or hbase that they need to manually define the Raw Serialisers they are using in their schema.

Merged into develop

Migration Steps
If you have an existing table in accumulo or hbase and your schema does not have serialisers defined for all of your properties and vertices then you will need to add them in manually rather than relying on Gaffer to select them for you. This is because there have been changes to which serialisers it would automatically select for you.
The Serialisation Factories were updated to replace the deprecated raw serialisers with the new ordered serialisers. This will break the serialisation of all the serialised data that has been serialised with the Raw Serialisers and have not explicitly defined them in their schema. This would occur due to new serialisation factory selecting the new serialiser, which will not be compatible with the existing serialised bytes.


Use tabs to indent "react component" skeleton

Use tabs instead of spaces to indent the snippet, so if you use spaces for indentation then Sublime will adjust the indentation-level-spaces-count to your setting.

:+1:


requirements.txt has a number of unknown version dependencies

The requirements.txt file has a number of packages that are pulled from a local drive, e.g.:
albumentations @ file:///home/conda/feedstock_root/build_artifacts/albumentations_1594363620148/work
Since I can't access these specific packages, I don't know the version or if any other modifications have been made to the package locally. As a result, attempting the installation with the most current version of each package, but this results in installation errors. Recommendations are appreciated.

You can delete this line and then try 'pip install albumentations'.

Thanks again for your help!
On Tue, Jul 20, 2021, 4:54 AM Hao Luo @.***> wrote:

You can delete this line and then try 'pip install albumentations'.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/michuanhaohao/AICITY2021_Track2_DMT/issues/7#issuecomment-883221494,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHKKSGIB4W66EZWR25S4B33TYU2TJANCNFSM5ALSZOTQ
.

Hi Hao, this is what I did for the entire requirements file. My question was because I didn't know if there were local modifications made to the files that would impact simply installing the most current version.


rake-compiler 1.2.4 broken on Ruby 2.5 and below

Hello there :wave:!
I work at @datadog on the ddtrace gem. We use the rake-compiler gem to build our native extension, and it's extremely useful, thanks for your work on it! :pray:
Yesterday our CI started failing after it picked up rake-compiler 1.2.4 on Ruby 2.5 and below:
mkdir -p tmp/x86_64-linux/stage/lib
rake aborted!
NoMethodError: undefined method `cleanpath' for #<String:0x000055c199dee558>
/usr/local/bundle/ruby/2.3.0/gems/rake-compiler-1.2.4/lib/rake/extensiontask.rb:166:in `block in define_compile_tasks'
/usr/local/bundle/ruby/2.3.0/gems/rake-13.0.6/exe/rake:27:in `<top (required)>'
/usr/local/bundle/gems/bundler-2.3.26/lib/bundler/cli/exec.rb:58:in `load'
/usr/local/bundle/gems/bundler-2.3.26/lib/bundler/cli/exec.rb:58:in `kernel_load'

We've had to pin an older version of rake-compiler to work around this.
The issue seems to be that #relative_path_from in those Rubies doesn't work with strings:
https://github.com/rake-compiler/rake-compiler/blob/77d9294a203a11b39ee2e3d295e91706049b91ea/lib/rake/extensiontask.rb#L166
...I believe the fix would be to change it to:
relative_lib_path = Pathname(lib_path).relative_path_from(Pathname(tmp_path))

I can submit a PR if that helps!

Oh, could you open a PR for this?

Sounds good, I should have a PR for this within the next days! (Travelling a bit right now!)

I submitted PR #226

Could you try 1.2.5?

Yeap, can confirm it works now, thanks for the speedy fix :)


Remove explicit disabling of partner integrations

The explicit disabling of EKS partner integrations that no longer exist is causing the EKS stack to fail to deploy. Removing as this is unnecessary as they will never be enabled by default. Additionally, we know that in the future more of these may be removed.
Issue #, if available:
#122
Description of changes:
remove explicit disabling of EKS partner integraions
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

@aaronmeza I had to bump the EKS submodule to latest as part of this due to a sporadic bug popping up in CI that has since been fixed (local builds were fine). Bonus: I noticed that https://github.com/aws-quickstart/quickstart-amazon-eks/issues/418 appears to be resolved as part of this.
@leclaist Since I took care of upgrading the EKS submodule, you'll probably need to rebase your feature branch for your PR for #107, but now you only need to update the templates.


Support for training XCiT models

Could you add support or provide an example on how to train XCiT models with DINO?

Hi @yadamonk
Thanks for your interest!
Yes I am planning to add support for XCiT models training in a couple of weeks. I don't have much bandwidth at the moment :/

Done: 144f70567a6ff853b9ea44503ba37fdcf8ca41fb

Thank you so much @mathildecaron31 🤗

@yadamonk I realized that the backbone only checkpoint led to the path of the student weights... Sorry about that. I've just corrected that to that they point to the teacher weights. If you have been using the backbone only checkpoints please reload them.
Sorry for the inconvenience.

Thanks for the quick fix @mathildecaron31 🤗

Hey @mathildecaron31 sorry to reopen this again, but will the visualization and video generation code be modified to be compatible with XCiT?


New version is available.

@alangrafu
New obsidian image is available please update it when you have time.

Thanks for the contribution! Change is committed and new build will be available soon.


Block-out Dates

-- tried to set 3/28-4/7, but program only accepted 3/28-31; I had to go back and add 4/1-7.

This has been fixed!


Request: --watch mode should try to gracefully kill running apps, sbt-revolver style

I had a look at this some time back but felt it was too involved for me to come up with a good solution.
Basically the watcher is blocking a thread which prevents—or so it seemed to me at least—any detection and termination of the ongoing process, from the context where the process handle is available.

https://github.com/lihaoyi/mill/commit/57f4968e8c37c857e4c8d6cac80dca739186754d implements JavaModule#runBackground, which runs something in the background and only kills it the next time you call runBackground on the same module

Fixed in 0.2.5


Support for current estimators and V1 soon?

Hello @baggepinnen,
I'm working in implementing the current form for all the estimators in the package. Two points about that:
1- The predict and correct is a good idea except for the MovingHorizonEstimator: this separation is impossible afaik. It needs to be a single function that solve the full optimization problem over the window $N_k$. I prefer to have a uniform API for all the state estimators. About the possible confusion between the current and delayed form: I'm hesitating between keeping the updatestate! method for both forms (and it should be called before or after moveintput, depending on the chosen form, a bit more confusing IMO but simpler API). The other option would be to create a new method for the current form e.g. evalstate!. Do you have a preference or other ideas ?
2- After these new features are merged, I feel like it would be a good timing to move to v1. The user base seems to be growing according to stars and I'm not a fan of keeping packages in v0 eternally.

The way I have it setup in LowLevelParticleFilters.jl is to offer predict! and correct!, but also an update! that performs them both. This package could do something similar, in which case all estimators would support predict/correct except for MHE which only supports the combined update. Having them separate when possible is nice since it offers a lot of flexibility in how the estimator is used, enabling things like

Performing multiple steps of prediction when no measurement is available
Precomputing the prediction when idling to reduce latency when obtaining a measurement

In principle, a moving-horizon estimator can also perform both of those steps, the correction involves solving the optimization problem, while the prediction only advances the state using the internal prediction model.
It's not quite clear to me what the function evalstate! does, would postupdate! and preupdate! be descriptive names? Alternatively, predict_and_correct! / correct_and_predict! to make it abundantly clear what's happening, at the expense of some verbosity?

Alright thanks for your tips!
Clearly, I will implement predict! and correct! function, but it will be in a new minor version.
For now, I'll just code the current-form for all observers and offer the alternative in the API

@baggepinnen Another point, I'm thinking on switching to the current form by default (to remove the one sample delay). Do you agree with this breaking change ?
And about that, why did you chose the the delayed form by default in kalman and place function of ControlSystemsBase.jl ?

Do you agree with this breaking change ?

I do

why did you chose the the delayed form by default in kalman and place function of ControlSystemsBase.jl ?

I didn't, this precedes my involvement in the package. The choice was likely made to conform to other similar toolboxes.

Okay after some deep reflection (sadly also in the night, the static gain on my brain is sometimes zero haha) here is what would be the new API for state estimation:
I purposefully avoided the "correct" and "predict" keyword in the API, to avoid possible confusion with your toolbox and similar toolboxes. Moreover, I will presumably add these two functions latter in the API.
For now, my goal is to create a simple API that works for all the state estimators, no matter if it's in the current or delayed form. This way, a user can write its own for loop with e.g. setpoint bumps and disturbances, and switch the estimator inside the constructor, before the for loop. The code inside the for loop will still work without any change. Here's the new API:
for i = 1:100
    ry = ... # current output setpoints
    ym, d = ... # current measured outputs and disturbances
    preparestate!(mpc, ym, d) # this method will do nothing with delayed estimators
    u = moveinput!(mpc, ry, d)
    # sending the current manipulated inputs to the plant...
    updatestate!(mpc, u, ym, d) # this method will always do something (with more computations in the delayed form)
end

My initial idea was to move everything in the moveinput! function (that's what MATLAB do for linear MPC). But I don't like this idea since it mix the concept of estimation and control, meaning that the estimator cannot be used without a mpc controller. I also like that updatestate! is always called at the end of the for loop, no matter if its a StateEstimator or a SimModel object.
P.S.:

In principle, a moving-horizon estimator can also perform both of those steps, the correction involves solving the optimization problem, while the prediction only advances the state using the internal prediction model.
Doing that would neglect the constraints on the state estimate during the prediction step.

I finished the implementation for all the estimators except the MHE. Thanks for the tips, it does clearly improve the closed-loop performance for e.g. disturbance rejection. The detailed equations are also documented in the internal documentation.

Cool, Nice work!
Disturbance rejection when there is no measurement of the disturbance relies solely on the measurement feedback, so it makes sense that reacting to this one sample earlier improves the rejection 😊


Ignoring hidapi: markers 'platform_system != "Linux"' don't match your environment

Hi,
I'm running the command
mbed compile -m EP_AGORA -t GCC_ARM --flash
and I receive:
[mbed] Working path "/home/symeon/Mbed Programs/EmbeddedPlannet/mbed-os-example-blinky" (program)
[mbed] WARNING: Missing Python modules were not auto-installed.
       The Mbed OS tools in this program require the following Python modules: six, idna, cryptography
       You can install all missing modules by running "pip install -r requirements.txt" in "/home/symeon/Mbed Programs/EmbeddedPlannet/mbed-os-example-blinky/mbed-os"
       On Posix systems (Linux, etc) you might have to switch to superuser account or use "sudo"
---
Could not find executable for GCC_ARM.
Currently set search path: No path set
[mbed] ERROR: "/usr/bin/python3" returned error.
       Code: 2
       Path: "/home/symeon/Mbed Programs/EmbeddedPlannet/mbed-os-example-blinky"
       Command: "/usr/bin/python3 -u /home/symeon/Mbed Programs/EmbeddedPlannet/mbed-os-example-blinky/mbed-os/tools/make.py -t GCC_ARM -m EP_AGORA --source . --build ./BUILD/EP_AGORA/GCC_ARM"
       Tip: You could retry the last command with "-v" flag for verbose output

Then I run
sudo pip install -r mbed-os/requirements.txt
and I receive:
Ignoring hidapi: markers 'platform_system != "Linux"' don't match your environment
Ignoring pywin32: markers 'platform_system == "Windows"' don't match your environment
Ignoring wmi: markers 'platform_system == "Windows"' don't match your environment

I'm running python3, pip3 on
sudo lsb_release -a
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 10 (buster)
Release:	10
Codename:	buster

Thank you for raising this detailed GitHub issue. I am now notifying our internal issue triagers.
Internal Jira reference: https://jira.arm.com/browse/IOTCORE-1724

I have managed to do the following:
sudo pip install -U -r requirements.txt
This installed the missing modules idna, cryptography, but not six. Then from requirments.txt I saw that "six==1.12.0" So I run
sudo pip install 'six==1.12.0' --ignore-installed
The ignore-installed was necessary otherwise I couldn't install it. Now what I receive with
mbed compile -m EP_AGORA -t GCC_ARM --flash
[mbed] Working path "/home/symeon/Mbed Programs/EmbeddedPlannet/mbed-os-example-blinky/mbed-os" (library)
[mbed] Program path "/home/symeon/Mbed Programs/EmbeddedPlannet/mbed-os-example-blinky"
Could not find executable for GCC_ARM.
Currently set search path: No path set
[mbed] ERROR: "/usr/bin/python3" returned error.
       Code: 2
       Path: "/home/symeon/Mbed Programs/EmbeddedPlannet/mbed-os-example-blinky/mbed-os"
       Command: "/usr/bin/python3 -u /home/symeon/Mbed Programs/EmbeddedPlannet/mbed-os-example-blinky/mbed-os/tools/make.py -t GCC_ARM -m EP_AGORA --source .. --build ../BUILD/EP_AGORA/GCC_ARM"
       Tip: You could retry the last command with "-v" flag for verbose output

Ok...finally I've managed to make it work:
I downloaded the gcc compiler and I configured it by running:
mbed config --global GCC_ARM_PATH /gcc-arm-none-eabi-9-2019-q4-major/bin
Please fix also your documentation to include these commands as well because it took a while to understand what's going on

Thank you for raising this detailed GitHub issue. I am now notifying our internal issue triagers.
Internal Jira reference: https://jira.arm.com/browse/IOTOSM-2046

Hi @symeonmattes,
Glad that you managed to install mbed cli and got the program to work. We do have documentation https://os.mbed.com/docs/mbed-os/v6.3/build-tools/install-and-set-up.html but I agree that the link is not easy to find.

@iriark01 Could you look into replacing https://os.mbed.com/docs/mbed-os/v6.3/quick-start/build-with-mbed-cli.html with https://os.mbed.com/docs/mbed-os/v6.3/build-tools/install-and-set-up.html as the quick start guide does not provide information to configure compiler location?

Do you still need to configure it when you use an installer?

Closing as it has not been updated since the last year.

Hi,
Sorry for my late reply, as I didn't notice the comment. Just in case somebody else has the same issue, actually it's a conflict of the pip library. I tried different things such as:
pip install -r mbed-os/requirements.txt
The idea is to uninstall the packages i.e.
pip uninstall six, idna, cryptography
and then use this
pip install -r mbed-os/requirements.txt


update licenses to Apache

Summary
crypto: update licenses to Apache
fs: update licenses to Apache
include: update licenses to Apache
libs: libxx: update licenses to Apache
libs: libc: update licenses to Apache
mm: update licenses to Apache
net: update licenses to Apache
tools: update licenses to Apache
Impact
License
Testing
None

LGTM


Elasticsearch does not listen on port 9200 upon reboot. (Deb package installiation for Ubuntu 16.04)

After installing Moloch with the .deb file and installing elasticsearch with the demo install, if I start the services via systemctl start elasticsearch.service they work just fine. But doing a systemctl enable elasticsearch.service upon reboot it fails to listen on port 9200 even if the service is running.

it works for me.
What do your mean "even if the service is running?"
Make sure you dont have both /etc/init  and /etc/systemd versions of the startup script installed.

I apologize as you can tell I'm not the sharpest when it comes to linux. Unfortunately our Linux god is on vacation this week and is unable to help. The /etc/init version is not running. After installing moloch via the .deb I can start the services manually via sudo and they run correctly, but if I set the services to start automatically via systemctl enable elasticsearch.service upon reboot of the system and doing a systemctl status elasticsearch.service it shows it as "active (running)" but using lsof -i -n -P the service is not listening on port 9200. It's looking more like a permissions thing to me but then again I'm not super fluent in linux.

And this is with 0.17.1?

This was an attempt to update an existing moloch. When doing a fresh install it works perfectly fine.

great


[Feature Request] Allow custom hotkey to be set per browser

Right now, the number keys 1..9 are the only ways to select browsers from the keyboard, assuming the chooser appears. The numbers are chosen based on the order of the browsers in the list, and are therefore opaque to the user (though specified on the manual)
I think the ability to set a specific hotkey per browser would be nice and clear. For example, browser 1 could be selected with D, and browser 2 could be selected with F, for example.

I've implemented the basics in the attached branch, but missing user interface to set it up


multi camera

If I have 3 cameras how do I get 3 cameras connected into a single room so when a real user connects they see the 3 cameras

I'm afraid that feature is not supported by WebRTC itself. Your browser will ask you to pick a single camera. I think there is support for multiple streams, but I'm afraid it's only for remote ones (although I would have to check).


⚠ Bofrost.fr has degraded performance

In 75d52a3, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 821 ms

Resolved: Bofrost.fr performance has improved in d931b1a.


Moved exercises from toplevel to exercises subdirectory

See #82

Please do not merge before #81. When #81 is merged I will rebase that one into this and update the PR a last time.
PS: If you want me to squash all this commits I'll do it.

I'd just make them all one big commit, "Move exercises into subdirectories." :rocket: once you rebase

Cheers


Language picker is cut-off when there are too many languages

To Reproduce
Add this to payload config:
locales: [
      'de-DE',
      'de-AT',
      'de-CH',
      'fr-CH',
      'it-CH',
      'nl-NL',
      'nl-BE',
      'fr-BE',
      'fr-FR',
      'fr-CA',
      'it-IT',
      'no-NO',
      'dk-DK',
      'se-SE',
      'fi-FI',
      'en-UK',
      'en-IE',
      'en-CA',
      'en-AU',
      'en-NZ',
      'en-US',
      'es-US',
      'es-ES',
      'en-MY',
      'pt-PT',
      'br-BR',
      'pl-PL',
      'cz-CZ',
      'hu-HU',
      'ro-RO',
      'sk-SK',
      'hr-HR',
      'sl-SL',
      'sr-RS',
      'es-CL',
      'es-MX',
      'es-AR',
      'gr-GR',
    ],

Describe the Bug

de-DE is not visible there, for example.
Payload Version
1.8.2

Would love to see a small dialog be introduced. Wondering if others think that would be a good addition to core 🤔

Tbh I think it'd be enough to add a small search field, as well as a scrollbar. I do like that it's just a dropdown - feels quicker and more learn than a dialog.

We could use react-select to make it filter and scrollable. That will also be good to use when we allow multiple locales to be editable.

As discussed, this is an interim solution until we overhaul this localization popup: PR here


[Bug] I cannot buy-name via REST

I try to buy-name by using CLI and it works. However, I try to buy-name by using REST API and  It doesn't work. The command I used is as follow as below.
curl -XPOST -s -k http://localhost:1317/nameservice/names --data-binary '{"base_req":{"from":"jack","password":"12345678","chain_id":"namechain","sequence":"0","account_number":"0"},"name":"jack1.id","amount":"5nametoken","buyer":"$(nscli keys show jack -a)"}' | jq .

Output:
{
  "error": "invalid from address: jack"
}

Which I run these commands below to init chain.
# Initialize configuration files and genesis file
nsd init --chain-id namechain

# Copy the `Address` output here and save it for later use
# [optional] add "--ledger" at the end to use a Ledger Nano S
echo 12345678 | nscli keys add jack

# Copy the `Address` output here and save it for later use
echo 12345678 | nscli keys add alice

# Add both accounts, with coins to the genesis file
nsd add-genesis-account $(nscli keys show jack -a) 1000nametoken,1000jackcoin
nsd add-genesis-account $(nscli keys show alice -a) 1000nametoken,1000alicecoin

Please help advice.

curl ... '{"base_req":...,"buyer":"$(nscli keys show jack -a)"}'  would got curl ...  "buyer":"$(nscli keys show jack -a)"}' ,  not expected curl....  "buyer":"cosmos1r...3279f2jt"}' . Because command in single-quotes would not be execute, variable too.
req.from should be address, not key.
req.password was not necessary.

You should use:
curl -X POST \
  http://<IP_ADDRESS>:1317/nameservice/names \
  -H 'Content-Type: application/json' \
  -d '{
	"base_req":{
		"from":"cosmos1r9awxg0jwk6qv9hs8kkajaj7u33d503279f2jt",
		"chain_id":"namechain",
		"sequence":"0",
		"account_number":"0"
	},
	"name":"jack1.id",
	"amount":"5nametoken",
	"buyer":"cosmos1r9awxg0jwk6qv9hs8kkajaj7u33d503279f2jt"
}
'

and will get
{
    "type": "auth/StdTx",
    "value": {
        "msg": [
            {
                "type": "nameservice/BuyName",
                "value": {
                    "Name": "jack1.id",
                    "Bid": [
                        {
                            "denom": "nametoken",
                            "amount": "5"
                        }
                    ],
                    "Buyer": "cosmos1r9awxg0jwk6qv9hs8kkajaj7u33d503279f2jt"
                }
            }
        ],
        "fee": {
            "amount": null,
            "gas": "200000"
        },
        "signatures": null,
        "memo": ""
    }
}

Thank you very much for suggestion.

I don't understand how this solution is a real solution. The shown JSON does not have a signature and is not broadcasted, so it needs to be signed and broadcasted right?


incorrect link in wadm docs - non-destructive undeploys

In the docs there's a link in a :fire: Danger popout to the top-level wasm repo issues page. Should it link to this: https://github.com/wasmCloud/wadm/issues/113 ?
I've got another branch for random docs updates on my fork so I could easily update this.

@ceejimus I'm just now seeing this, but that works for me! @thomastaylor312 are you okay with this change?


Feature request: Add more application support for notification service

Hello,
On my phone with Waterfly III installed there are also applications from which I receive notification from on transactions. Those are:

Google Messages: I get a SMS message on each transaction I do over my bank card and it pops up as a message
Revolut: All trasnactions are displayed in notifications
DH Mobilni: Main application from my bank. In Slovenian language.
DH Denarnik: Support application from my bank. For confirming web transfers, for sending/receiving instant SEPA transfers, ... Also in Slovenian language
Google Wallet: Connected to Revolut card. So basicaly every Google Pay transaction goes over to my Revolut account

For this I would like to capture the notitication messages and hope that Waterfly III would automaticaly create transactions.
Where to start?
Thank you.

Sure. Here are the links:

Google Messages: https://play.google.com/store/apps/details?id=com.google.android.apps.messaging
DH Mobilni: https://play.google.com/store/apps/details?id=com.hrc.eb.mobile.android.hibismobiledh
DH Denarnik: https://play.google.com/store/apps/details?id=si.delavskahranilnica.dhdenarnik

Do you use generic regexp string to extract or you can set it per application? I can pass you exact regexp strings per application to extract account, value, description and maybe even transaction type if needed.
How does this work? I add the application into the list in Waterfly III and it will automatically read the notification messages? Or when I get notification I need to open Waterfly and start the service (it seems to stop automatically) and it would parse the notifications that are visible?

@dreautall how would this work would i have a extra button on my notification like "open in watefall" or would all notification automatically transformed into a new transaction?

@gamebeaker after you receive the banking notification, Waterfly III shows a notification asking if you would like to create a new transaction. Tapping this notification brings you to the create transaction view in-app with some prepopulated data from the original banking notification.

Google Messages could be hard to implement, because the syntax isn't the same for every bank/ payment processor out there and they all send you an sms.
Wouldn't it be better if Waterfly III only shows the notification if it was able to parse the sms correctly?

Please add also Tinkoff mobile application. It's not in google.play, but it can be downloaded from the official website https://mobile-apps-prod.cdn-tinkoff.ru/tinkoff-bank-6-26-0-7924.apk


sync performance drop of distributed fast histogram

looks like latest branch has per-node sync again https://github.com/dmlc/xgboost/blob/4ef6d216b9989f0ea7101fcbecefe25f6125c195/src/tree/updater_quantile_hist.cc#L615-L617

and the grads is also synced per node
https://github.com/dmlc/xgboost/blob/4ef6d216b9989f0ea7101fcbecefe25f6125c195/src/tree/updater_quantile_hist.cc#L643 (only needed for root actually , check https://github.com/dmlc/xgboost/pull/4140)

Seems the optimization PR didn't make a clean rebase.

more discussion at https://github.com/dmlc/xgboost/pull/4716

Re-opening this, since #4716 fixed correctness but did not address performance. @thvasilo points out that syncing for every nodes leads to significant slow-down due to extra communication.

Copying from the other issue:  While the second per-node sync seems straightforward to fix (see this WIP) getting rid of the first loop could entail changes to how we build and use the hist_ container (of HistCollection type) which would require a lot of effort to untangle.

@thvasilo Note that this issue is not present in release 0.90. I believe #4529 introduced it. A proper fix would need to first analyze #4529 in detail and selectively revert the parts that cause per-node sync. In 0.90, hist performs per-level sync (unless grow_policy is specifically set to lossguide).


double free or corruption (out)

This still applies for me even when using the flags suggested. Things are fine if there is just one thread though Executor executor(1);
The errors seem to be changing though, sometimes I get corrupted double-linked list or Signal: SIGSEGV (Segmentation fault)
I think it has to do with taskflow and ForwardPass on JacRev.

Originally posted by @ealione in https://github.com/heal-research/operon/issues/25#issuecomment-2387223596

Hi, could you post more information to help debug this issue:

operon revision
output of e.g. operon_gp --version or operon_nsgp --version
CPU model

Any additional details or a small script/code snippet to help reproduce this issue is appreciated.

Originally posted by @foolnotion in https://github.com/heal-research/operon/issues/25#issuecomment-2387711808

VERSION:
operon rev. 28ac41b  Linux-6.8.0-45-generic x86_64, timestamp 1979-12-31T16:00:00Z
single-precision build using eigen 3.4.0, ceres n/a, taskflow 3.7.0
compiler: Clang 18.1.6, flags: -g -O3 -DNDEBUG
CPU: AMD Ryzen 7 7800X3D
I was testing a script similar to https://github.com/heal-research/operon/blob/main/test/source/implementation/poisson_regression.cpp

Apologies but this was my mistake, I had placed std::cout << trace in ForwardPass and forgot about it. I re-installed a clean copy and it is OK.

I'm not sure why me trying to print that value caused the issue but Operon works great!

Operon uses a number of dependencies which have to be compiled with the exact same flags in order to avoid errors like the ones you encountered. Could you try to build with optimizations turned on:
cmake -S . -B build-linux --preset build-linux -DCMAKE_BUILD_TYPE=Release
cmake --build build-linux -j

./build-linux/cli/operon_gp --version
operon rev. 9a15fba Release Linux-6.11.1 x86_64, timestamp 1980-01-01T01:00:00Z
single-precision build using eigen 3.4.0, ceres n/a, taskflow 3.8.0
compiler: Clang 18.1.6, flags: -g -O3 -DNDEBUG -Wall -Wextra -Werror -pedantic -fsized-deallocation -fno-math-errno -march=x86-64-v3

I'm not sure why me trying to print that value caused the issue but Operon works great!

Operon uses multiple threads. ForwardPass runs concurrently on multiple threads.
As you noted, the problem does not occur in a single threaded configuration.
Without checking it, I'd say writing zu cout is not thread-safe and produced a race-condition.

The interpreter is not thread-safe and should never be reused between threads. It is a very light object which should be initialized anew for every tree.


airflow: Revert changes in Dockerfile from #2316 that broke the CI

Problem
PR #2316 broke the CI, this reverts some changes in Dockerfile that cause errors
Checklist

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

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

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

Comparison is base (d145314) 84.29% compared to head (4f4326f) 83.43%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2318      +/-   ##
==========================================
- Coverage   84.29%   83.43%   -0.86%
==========================================
  Files          59       54       -5
  Lines        3342     3169     -173
==========================================
- Hits         2817     2644     -173
  Misses        525      525

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


Send email doesn't trigger new verification email

The Issue
If you don't confirm your email and then press the send verification email button (leave the rewards screen and come back if you don't see the button), nothing is sent.
Anything Else

Screenshots

couldn't reproduce, closing

I ran into this again...seems to happen if I use an email that's already existing in our system - maybe we aren't calling resend email, but just email new which 500s?  If I change the email to something like<EMAIL_ADDRESS>then it sends.


[Bug] [Admin] Task is deleted. Task instances and history are not deleted consecutively

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Task is deleted. Task instances and history are not deleted consecutively
What you expected to happen
When a Task is deleted, the task instance and task history can be deleted in cascade
How to reproduce
A flinksql task is added on the data development interface, and the task is submitted. A record is generated on the operation and maintenance center interface. After a test, the flinksql task is deleted
Anything else
No response
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

The same situation occurred in the 0.7.2version

We need to commit it into 0.7 branch~


fixed readme documentation so it works with private repos

This PR fixes https://github.com/ArtiomTr/jest-coverage-report-action/issues/280

Thank you for your contribution!

@all-contributors please add @nikodevv for docs


added getStringPermutations

Added getStringPermutations which prints out all the available permutations of a string

@Chalarangelo I have followed the guidelines, what is the specific issue

@rubiin same issue with me

@rubiin same issue with me

yes if the specific issue is not specified, then how will it be fixed.


Update for Laravel 8 compatibility.

Fixes #960

Please try supporting both versions and not only 8.0 😉

I don't believe that is possible due to the method constructor signature changes.

@ArlonAntonius I was able to get it to work. Tested with Laravel 7 and got all tests to pass. Had to hack around the model factory changes, but that's only in the test, so should be ok.

Will check on this tomorrow.

The pipeline for this PR fails. There are some things that might need to be picked up on this PR:

Multi Laravel Testing (can peak at t/t to see how we do it)
Multi PHP Testing

@bkintanar Any news on this PR?

@bkintanar merge please

Sorry I haven't checked this yet. Will try to make time asap.

Unfortuantely I don't have access to those test results, so those are corrections you would have to make. For what it's worth, I have been using my PR as a patch in Laravel 8 without issue.

Would be great to see that PR merged. It has been a while since L8 is available and seems that the PR is working. Any idea on how to get moving forward?

Hi @mikebronner,
I really appreciate the effort you have done for this. We would love to support Laravel 8. However, it would take us quite some time to adjust the test suite to get testing running properly on GHA. Instead, we decided that it would be fine to not support Laravel 7 anymore and completely move to Laravel 8. Would you be willing to revert the changes on this PR to your old commit that seemed to be working c01a9cdbab2bb19151a3296219c04362af679ff0
Once that test suite runs, we can figure out if there is anything wrong with it or if it's okay to release that as an L8 compatible version. BTW: You can check the CI yourself by simply logging with Github, should only take a few seconds and then you should be able to view the errors.
I hope you are willing to do this 🙏

Hi @ArlonAntonius, sure. I can probably get to this today or this weekend.

@ArlonAntonius @bkintanar Reverted back to Laravel 8-only and brought all tests to green.

@ArlonAntonius I removed the composer dependency for legacy factories. I didn't find any actual legacy factories remaining in the code. The testing env variable is required for the tests to run. Without it I was unable to run them.

@sniper7kills Can you update the docs with a new version and simply change the requirements to reflect that L8 is supported and PHP 7.3 or higher?

Hi @ArlonAntonius, updated readme. Should hopefully be good to go? Let me know if you need anything else. Thanks! (Assuming the sniper7kills reference was in error and you meant me?)

Hi @mikebronner, nope, really meant @sniper7kills , meant to update the actual docs at tenancy/docs


Fixed some typos.

I fixed some nasty typos in the code.

@microsoft-github-policy-service agree

Thanks for all the updates here! Glad someone finally decided to go through and fix all our dodgy spelling :)


[ci] Add GitHub Action to run unittests on macOS

Codecov Report

Merging #2207 (c7a4ed6) into master (021a055) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2207   +/-   ##
=======================================
  Coverage   85.54%   85.54%
=======================================
  Files          55       55
  Lines        9746     9746
=======================================
  Hits         8337     8337
  Misses       1409     1409

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 021a055...c7a4ed6. Read the comment docs.


Handling of authentication failures

dimoapi should handle all API specific exceptions and the HA integration should catch the propagated ones and map them to HA error states.

Solved with #36


coredump when empty redis client exec specific commands

changed from examples/example.c, my code is as follows:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#include <hiredis.h>

int main(int argc, char **argv) {
    unsigned int j;
    redisContext *c;
    redisReply *reply;
    const char *hostname = (argc > 1) ? argv[1] : "<IP_ADDRESS>";
    int port = (argc > 2) ? atoi(argv[2]) : 6379;

    struct timeval timeout = { 1, 500000 }; // 1.5 seconds
    c = redisConnectWithTimeout(hostname, port, timeout);
    if (c == NULL || c->err) {
        if (c) {
            printf("Connection error: %s\n", c->errstr);
            redisFree(c);
        } else {
            printf("Connection error: can't allocate redis context\n");
        }
        //exit(1);
    }

    /* PING server */
    //reply = redisCommand(c,"PING");
    reply = redisCommand(c, argc > 3 ? argv[3] : "PING");
    if (reply == NULL || c == NULL || c->err) {
        return 1;
    }
    //printf("PING: %s\n", reply->str);
    freeReplyObject(reply);

    /* Disconnects and frees the context */
    redisFree(c);

    return 0;
}

I want to reconnect redis when connection is wrong or reply is empty, which code is " if (reply == NULL || c == NULL || c->err)". Simple commands, such as "PING", "SET foo bar", "GET foo", etc., will reach code "return 1;". Some commands will crash.  For example
./a.out 0 6379 "DEL a b c d"
./a.out 0 6379 "LPUSH mylist 0 1"

ps. ./a.out is my compiled binary file. 0:6379 should be an redis server, but I didn't start intentionally.
In my opinion, commands which support a variable number of arguments, such as DEL, LPUSH, ZADD etc. will crash.

Weird. Looking into that.

You're checking for the error on connect and even free it, but you're still using the connection context after that. That's clearly wrong and a use-after-free bug. There's not much hiredis can do here (yey for poor C semantics).


Submission for data_release causes export to fail

The addition to data_release causes export to fail.

Although this is addressed in #8 , we should leave open until a more general exception handling approach is taken.


xmldom dependency in ecschema-metadata should be a real dependency, not devDep

it is being used at runtime in Schema.ts and therefore must be a real production dependency, not a devDependency
as it is now, it breaks devDependency-pruned builds of electron desktop apps that transitively depend on this, the workaround for now is to explicitly depend on xmldom in those apps until this is fixed
to prevent this in the future, can we use something like this eslint rule? If we have an approval process for non-standard eslint rules we can go through, I can add such a rule to this PR after it is approved

@MichaelBelousov this was a devDependency because when you're on the frontend you technically do not need an this implementation. It is only needed when running, and using, ecschema-metadata on the backend.  Maybe at that point it should really be an optionalDependency but I don't agree that it should always be included as a real dep.

@mergifyio update

@calebmshafer can you make a final call on this? Should we make xmldom an optional dependency and throw an error when using Schema.toXmlString on "optional dependency not satisfied" when it isn't installed?

I'd be in favor of doing that instead of requiring the xmldom dependency. I must have missed the initial PR that added the Schema.toXmlString method, could have saved you the headache...

I'd be in favor of doing that instead of requiring the xmldom dependency. I must have missed the initial PR that added the Schema.toXmlString method, could have saved you the headache...

not a problem, I'll add that

@calebmshafer, not sure if you wanted to avoid this, but because the imodel transformer in imodeljs-backend uses this feature, I had to add a dependency on xmldom to the backend package. We could add it as an optional dependency to the backend package, and put a note in the transformer documentation that consumers will need to install xmldom to use the imodel transformer. It would be nicer if imodel-transformer were its own package of course, but I don't believe we can split that out right now.

I need to at least change this to use browser APIs where available, but considering maybe removing toXmlString right now entirely and have the transformer which requires it on the backend serialize the xml document it gets from toXml itself

@MichaelBelousov that sounds like a better alternative to me rather than complicating what ecschema-metadata is doing with it.  Then it'd be more isolated.
Also, I forget what PR comment it was in but for one of them you mentioned it'd be easier if iModelTransformer was out of the backend package.  I completely agree and meant to chat with you about that at some point. With us working on 3.0, I feel like that's the time to make the split if we're going to.  Any objections or hesitation with doing it?

Also, I forget what PR comment it was in but for one of them you mentioned it'd be easier if iModelTransformer was out of the backend package. I completely agree and meant to chat with you about that at some point. With us working on 3.0, I feel like that's the time to make the split if we're going to. Any objections or hesitation with doing it?

No, let me know if you need help or want to give that task to me.

@MichaelBelousov if you could pick that up that'd be great. Thanks!

I'm going to change this PR to just remove toSchemaXml, but it will be dependent on #2023 which removes the dependency in imodel-transformer

the removal of toSchemaXml is here https://github.com/imodeljs/imodeljs/pull/2092


chore: Update registry.access.redhat.com/ubi9/s2i-base Docker digest to ca5194f

This PR contains the following updates:

Package
Type
Update
Change

registry.access.redhat.com/ubi9/s2i-base
final
digest
abebf84 -> ca5194f

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 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 Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update. You will not get PRs for the registry.access.redhat.com/ubi9/s2i-base ca5194f update again.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Made some Selenium tests less flaky by adding explicit wait.

Flakey test runs for test_updating_related_objects_updates_fk_selects_except_autocompletes:

https://github.com/django/django/actions/runs/5450377222/jobs/9915579013
https://github.com/django/django/actions/runs/5450377222/jobs/9915578779

Flakey test run for test_list_editable_popups:

https://github.com/django/django/actions/runs/5452823563/jobs/9920858724

Flakey test run for test_select:

https://github.com/django/django/actions/runs/5452823563/jobs/9920858724

Test runs with them passing to show flakiness is hopefully gone:

https://github.com/django/django/actions/runs/5453005282?pr=17043
https://github.com/django/django/actions/runs/5453127139?pr=17043
https://github.com/django/django/actions/runs/5453179602?pr=17043
https://github.com/django/django/actions/runs/5453225225?pr=17043
https://github.com/django/django/actions/runs/5453261530?pr=17043
https://github.com/django/django/actions/runs/5453334823?pr=17043
https://github.com/django/django/actions/runs/5453382117?pr=17043
https://github.com/django/django/actions/runs/5454055716?pr=17043
https://github.com/django/django/actions/runs/5454099125?pr=17043

test_select_multiple is another one that fails sometimes:
======================================================================
FAIL: test_select_multiple (admin_views.test_autocomplete_view.SeleniumTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/unittest/case.py", line 59, in testPartExecutor
    yield
  File "/usr/local/lib/python3.10/unittest/case.py", line 591, in run
    self._callTestMethod(testMethod)
  File "/usr/local/lib/python3.10/unittest/case.py", line 549, in _callTestMethod
    method()
  File "/django/tests/admin_views/test_autocomplete_view.py", line 546, in test_select_multiple
    self.assertEqual(len(select.all_selected_options), 2)
  File "/usr/local/lib/python3.10/unittest/case.py", line 845, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/usr/local/lib/python3.10/unittest/case.py", line 838, in _baseAssertEqual
    raise self.failureException(msg)
AssertionError: 1 != 2

@sarahboyce Thanks :+1:


Choose package manager

There is a inconsistency in the readme about which package manager is used: npm or yarn.
The currently checked-in package lock file is in npm format, so that's what I've been using.
yarn currently fails with an obscure error:
➤ YN0066: │ typescript@patch:typescript@npm%3A4.7.2#builtin<compat/typescript>::version=4.7.2&hash=a45b0e: Cannot apply hunk #2 (set enableInlineHunks for details)

@rossmoody I wasn't sure if you had a strong preference for yarn.


Add Configurations Tab

LinksScreen component made resuable as it now receives label
and icons as props, this can be passes as child to <BottomTab.Screen>,
configurations case adde to getHeaderTitle for main title on tabs header.
Resolves: #19

LinksScreen component made resuable as it now receives label
and icons as props, this can be passes as child to <BottomTab.Screen>,
configurations case adde to getHeaderTitle for main title on tabs header.
Resolves: #19


Question: Gradle support?

That's it, the question is if this will support Gradle projects.

https://github.com/ia3andy/bundler-gradle-jquery


Testing

:star: PR built and available in a preview environment nattymachado-natalia-teste-1-pr-1 here


When will the code be made public

Hi, Minkai. Great work, when will the code for the work be made public?

Hi Yueming. Thanks for your interest, and sorry that busy with NeurIPS these days. Would provide it after NeurIPS deadline. Thanks!

Hello, I apologize for bothering you. I would like to ask, will the code be updated in the near future?

Sorry for the delay. I rushed the release today and you can take a look! @Yueming-Yin @pearl-rabbit

Thank you for sharing.


Functional Timer Test Hard Codes Configuration Value

The functional timer test (/src/tests/timer-test/timer-test.c) hard codes the number of timers to 4, which may be higher than the OS_MAX_TIMERS configuration set in osconfig.h.
It is recommended to update the test to use the OS_MAX_TIMERS configuration macro or add protection with an #if guard.

Imported from trac issue 173.  Created by sstrege on 2016-08-03T17:27:41, last modified: 2019-08-14T14:11:46

Trac comment by jhageman on 2019-07-26 15:13:16:
Milestone renamed

Trac comment by jhageman on 2019-07-03 13:34:40:
Moved open 4.2.2 tickets to 4.3.1


FR: onDatesChangeFinal

Needing a callback for when the user has finished selecting both dates (both start and end).
onDatesChange fires separately on start and end selections, not extremely helpful.
onClose was a good workaround, except it doesn't fire with ClearDates.

Duplicate of #1070.

I'll close this and continue the conversation on #1070


Fixed Add Template Button

Apparently in the last release, I mapped the AddTemplate button in AddExpander on TripDetail to the item modal on iPhone. Quick fix here

forgot to merge, but the update went out like last saturday


gensio: 2.8.5 -> 2.8.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gensio is: General Stream I/O
meta.homepage for gensio is: https://sourceforge.net/projects/ser2net/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.8.8 with grep in /nix/store/9y5rqblpikwqvip0l6llxa78kfdm3227-gensio-2.8.8
found 2.8.8 in filename of file in /nix/store/9y5rqblpikwqvip0l6llxa78kfdm3227-gensio-2.8.8

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

gensio
ser2net

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9y5rqblpikwqvip0l6llxa78kfdm3227-gensio-2.8.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gensio https://github.com/r-ryantm/nixpkgs/archive/4257760920477c399be04c9fa4da40b37ba76659.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/4257760920477c399be04c9fa4da40b37ba76659#gensio

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9y5rqblpikwqvip0l6llxa78kfdm3227-gensio-2.8.8
ls -la /nix/store/9y5rqblpikwqvip0l6llxa78kfdm3227-gensio-2.8.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

gensio
ser2net

Maintainer pings
cc @Emantor for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gensio is: General Stream I/O
meta.homepage for gensio is: https://sourceforge.net/projects/ser2net/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.8.9 with grep in /nix/store/x15cfq4mx8lf4spsji0afq29ykhz0y87-gensio-2.8.9
found 2.8.9 in filename of file in /nix/store/x15cfq4mx8lf4spsji0afq29ykhz0y87-gensio-2.8.9

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

gensio
ser2net

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/x15cfq4mx8lf4spsji0afq29ykhz0y87-gensio-2.8.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gensio https://github.com/r-ryantm/nixpkgs/archive/c52e8e94297adf2e179062486c65f40fdd7a2cbd.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c52e8e94297adf2e179062486c65f40fdd7a2cbd#gensio

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/x15cfq4mx8lf4spsji0afq29ykhz0y87-gensio-2.8.9
ls -la /nix/store/x15cfq4mx8lf4spsji0afq29ykhz0y87-gensio-2.8.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

gensio
ser2net

Maintainer pings
cc @Emantor for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 340871 run on aarch64-darwin 1

1 package built:

gensio


[Enterprise] Global enabled state for all utilities GPO

Summary of the Pull Request
This PR adds a "global enabled state for all utilities" policy. It can be overridden using the individual utility policies.

PR Checklist

[x] Closes: #25953
[x] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[x] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

@jaimecbernardo
That's correct behavior. You set the individual policy which has a higher priority as the global policy. Whether it is set in user or computer is not important in this case because individual overrides global. But you shouldn't be able to override the same policy (global set in computer and user).
The point is that a user without admin rights don't have the permission to manipulate HKCU\...\Policies and HKLM\...\Policies. So it's impossible that he can manipulate something.
Admins will expect that each policy can be set in user or computer regardless of other policies.


Add support for eslint v9

We switched to biome since the eslint plugins are no longer compatible with eslint9


RFC: Remove experimental MapEffect composable

If you would like to recommend against this proposal, please comment below or react with 👎 .
MapEffect was added in #140 to expose the underlying GoogleMap Maps SDK object to enable extension and usages such as clustering with the utility library.
With Compose-native clustering being added by #258, we're hoping that MapEffect is no longer needed. Providing direct access to the map is potentially buggy as noted in this comment.
It seems like MapEffect would still be necessary for other Android Maps Utils such as GeoJSON layer (#29) and KML layer (#129).
Please comment below if you have other use cases that would require exposing the map object.

#282

As others already pointed out removing MapEffect will not make possible to add any Kml or GeoJson layer to a compose map.


Updated validations.py python script.

Updated validations.py python script.
Closes #1
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

Signed the license

@googlebot I signed it!

@googlebot I signed it!

@googlebot I signed it!

@googlebot I signed it!


Providing all pre-compiled Python versions 3.9 to 3.12

It might be good to provide all the precompiled versions of Python. Or maybe add a message (on Linus) saying that only 3.11 is precompiled and readily available, while the others require compilation (2-3 minutes).

Adding a warning/alert is better I believe.
Because if we decide to add all precompiled versions of python, it will increase the package size by alot. Almost doubling it or more..

We might want to go with a different approach though. Maybe uploading the precompiled versions as assets to the release, and allowing the app to download them when requested can save us some time.
For now let's add a warning - I will share my thoughts with you guys for the second approach later on.


Media invalid url

i have trying to link the storage, but the url become invalid
am i missing something?

i try many step to recovery this issue using command then im stuk on this problem.
I am very open and grateful for any solution


Difference between Async Fusion and Probabilistic Fusion

Can someone provide a very short high level description that describes the fusion options offered in Apollo and the differences between them?
Thanks

Async fusion fuses multiple sensor data whenever it receives any sensor output. On the contrary the current synchronous probabilistic fusion wait until all sensor outputs are ready for the fusion.


Moldova - convert to new pipelines

https://github.com/open-contracting/kingfisher-scrape/issues/122
https://github.com/open-contracting/kingfisher-scrape/issues/123

 Kingfisher Moldova Scrapers


stm32/can: docs, cleanup interrupt handling.y

stm32/can: cleanup interrupt traits.
stm32/can: docs.

bender run


Gradient/images assets

Should provide isolated SVGs of all graphics used on home and landing pages

gradients.zip

excite.gif

Closed by #185


command typst-lsp.showPdf not found

Component:

VSCode Extension

Extension version: v0.12.0
LSP version:  v0.12.0
OS version and name: Windows 11, version 23H2

I am on the latest stable version of the extension/LSP.  （0.10.0）
I have searched the issues of this repo and believe that this is not a duplicate.

Issue

After setting the extended instructions, I clicked the "show PDF" button, but the compiled PDF did not display,

and the following error appeared:

Logs

No logs output

i think there is confliction between typst lsp plugin and tinymist typst plugin, turning off or uninstalling the latter one works for me

I solved it by updating  the plug tinymist (0.11.15)  typst lsp (0.13.0)
and updating  typstVersion 0.11.1 (May 17, 2024)
then it works


Is this JWT exploit in pgjwt?

https://www.darkreading.com/vulnerabilities-threats/jsonwebtoken-security-bug-opens-servers-rce
It talks about how 9.0 is the safe JWT version.

This vulnerability targets a specific javascript library, no relation to what pgjwt uses internally. JS client code may need review, but that is another story.

@maparent Thank you.


Add option "suppressNotices" to prevent exec* methods from printing "starting" and "exited" notices

As discussed in #47, it's sometimes convenient to use DevCmd's exec* methods without having them output any of their notices ("Starting process..." and "Process xy exited...").
This PR adds this optional behavior to all exec* methods, including unit tests.
~This PR is based on #48 and must be merged after it.~ Rebased onto new main since previous PR is now merged.

Note that with this functionality, we can probably use ProcessExecutor internally in devcmd in some places where we previously dropped to raw spawn etc, e.g.:
https://github.com/XITASO/devcmd/blob/d34cd8bb986b58991d1947fce6783cc5b64730ea/packages/devcmd/src/from-cli/index.ts#L115-L119
and
https://github.com/XITASO/devcmd/blob/d34cd8bb986b58991d1947fce6783cc5b64730ea/packages/devcmd/src/utils/npm_utils.ts#L3-L11

@lieberlois @schierla, it would be great if you could give this a quick look and review.
Due to #54 and another fix (e8ef0599ec76eb0f88049c2633829c0ff39214e4) I'd like to push out new releases of devcmd-cli and devcmd soon.


ChatGPT connection failed

Describe the issue in detail here. Attach any relevant screenshots or files. The section below was automatically generated. Remove any information that you do not wish to include in the report.

Debug Data

Property
Value

Name
"Wolfram/Chatbook"

Version
"0.0.27"

EvaluationEnvironment
"Session"

FrontEndVersion
"Desktop: 13.2 for Mac OS X x86 (64-bit) (January 30, 2023)"

KernelVersion
"13.2.1 for Mac OS X x86 (64-bit) (January 27, 2023)"

SystemID
"MacOSX-x86-64"

Notebooks
True

DynamicEvaluation
False

SynchronousEvaluation
False

TaskEvaluation
False

Settings

Property
Value

TemperatureSetting
0.7

Role
"System"

Model
"gpt-4"

Stack Data
catchTopAs
throwFailure
trimURL

Failure Data
<|
  "Evaluation" :> Wolfram`Chatbook`Actions`Private`prepareChatOutputPage[CellObject["ff5a97ba-41fb-4032-886e-6d1b508b4e61", "d6258f8f-3114-4ded-a502-aa0bf2e72693"], Cell[BoxData[DynamicBox[ToBoxes[Refresh[Wolfram`Chatbook`Actions`Private`dynamicTextDisplay[Wolfram`Chatbook`Actions`Private`container$11885, True], TrackedSymbols :> {}, UpdateInterval -> 0.4], StandardForm], Initialization :> If[$SessionID =!=<PHONE_NUMBER>1856861813, NotebookDelete[EvaluationCell[]]], Deinitialization :> Quiet[TaskRemove[Wolfram`Chatb...
  "Failure" -> Failure["ConfirmationFailed", <|"Information" :> "NotebookRead", "MessageTemplate" -> "`` does not match ``.", "MessageParameters" :> {$Failed, _Cell}, "ConfirmationType" -> "ConfirmMatch", "Expression" :> $Failed, "Pattern" :> _Cell|>]
  "Arguments" -> {}
|>

This NotebookRead is failing:
https://github.com/WolframResearch/Chatbook/blob/dd413a89d815e694e20b0e8e6cc21ba9f7f4ab0e/Source/Chatbook/Actions.wl#L984
It's almost certainly a direct result of #187.


change to old method to fix missing method createParentDirectories

Description
Change  createParentDirectories to forceMkdir to fix missing method issue.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

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

Codecov Report

Merging #758 (872d638) into 2.5 (a247112) will increase coverage by 0.10%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##                2.5     #758      +/-   ##
============================================
+ Coverage     84.25%   84.35%   +0.10%
- Complexity     1051     1053       +2
============================================
  Files            99       99
  Lines          3836     3836
  Branches        351      351
============================================
+ Hits           3232     3236       +4
+ Misses          453      450       -3
+ Partials        151      150       -1

Flag
Coverage Δ

ml-commons
84.35% <ø> (+0.10%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../cluster/MLCommonsClusterManagerEventListener.java
79.41% <0.00%> (+11.76%)
:arrow_up:

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


Add unit tests

I know -- I should have been writing them along the way, but I was kind of designing in real time... Anyway, better late than never!

Pull requests can be submitted for any number of unit tests. No need to expect full coverage in a single PR.

Hey, I would like to work on this issue. Can you assign it to me?


Gradle Koans: sample challenges for getting familiar with Gradle

Expected Behavior
There should be a guide with challenges so people can get familiar with Gradle by solving small tasks.
It would be great if the tool (standalone? IDE plugin?) could automatically verify the solutions as well.
Current Behavior
It looks like Koans are missing: https://gradle.org/guides/
Context
https://play.kotlinlang.org/koans/overview

Sorry for the late reply
This seems like a resource that would be useful for the community.
This idea overlaps with the recently launched DPE University that contains free course about Gradle build tool basics, as well as deeper topics.

@oleg-nenashev, what do you think about having a dedicated resource for Koans?

I think it would be a good addition for the Gradle cookbook where we try to consolidate tool and solution specific guidelines https://community.gradle.org/cookbook/

Thank you for the suggestion. At this point in time we are focusing on more tutorials and samples in the documentation but it is unlikely that we will add something like Koans.


Videos Issue: Simplify Building Containers w/ Cloud Native Buildpacks

Describe the bug
The video appears to have been removed by the Youtube owner
To Reproduce
Steps to reproduce the behavior:

Go to the page: https://tanzu.vmware.com/developer/videos/cloudnativebuildpacks-lightboard/
Click on 'play'
See error 'video unavailalable - the video has been removed by the uploader'

Expected behavior
The video plays
Screenshots

Desktop (please complete the following information):

OS: MacOS
Browser Chrome
Version 91

Appreciate the report! I just posed up a fix for this so should be updated on the live site in just a few minutes. Thanks!


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat: Feeds | Add Filters Of Users

closes #601
Changes proposed
added filter button group on top of search box

Code of Conduct

[x] By submitting this pull request, I confirm I've read and complied with the CoC 🖖

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

You can also join our Discord community.
Feel free to check out other cool repositories of the Thecyberworld.
Join the Thecyberworld GitHub Organisation by raising an issue (you will be sent an invitation).

@Ben-aoun-1 Hi, thanks for your contribution, before I could review it, please fix the bug that occurs when the user is logged out. In addition, when the user is logged out he/she should not see those buttons or when he/she clicks on Following filter will pop out a screen for registration.
https://github.com/thecyberworld/TheCyberHUB/assets/76536506/0baad5bc-2a4f-45ac-8abc-82abf235c446
for example:
https://github.com/thecyberworld/TheCyberHUB/assets/76536506/a8f58138-4ac4-47ae-b2b7-f8c5f0847ac4

@Ben-aoun-1 @92206 are you the same person? I don't mind but to know that if not, you both working in same pr so I hope that you both ok with it.


Improve ui of create post + side account options+hover effects

Description

I want to improve its ui please assign it to me under gssoc'23
Steps to reproduce

Screenshots
No response
Desktop (please complete the following information)
No response
Smartphone (please complete the following information)
No response
Additional information
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Hi @akanksha984 we've made several changes in the application, can you please update your issue?
Ps: Perhaps, UI improvement of create post component is already under progress, you may fix the camera component to sync with this component :)
Also, you can improve those post buttons

Hi @akanksha984 we've made several changes in the application, can you please update your issue?
Ps: Perhaps, UI improvement of create post component is already under progress, you may fix the camera component to sync with this component :) Also, you can improve those post buttons

which camera component? The account section one ?
Ok i will improve the like,share,comment,save buttons

Hi @akanksha984 we've made several changes in the application, can you please update your issue?
Ps: Perhaps, UI improvement of create post component is already under progress, you may fix the camera component to sync with this component :) Also, you can improve those post buttons

which camera component? The account section one ? Ok i will improve the like,share,comment,save buttons

Camera component, that comes after you click on take picture in create post dialog box

Hi @akanksha984 we've made several changes in the application, can you please update your issue?
Ps: Perhaps, UI improvement of create post component is already under progress, you may fix the camera component to sync with this component :) Also, you can improve those post buttons

which camera component? The account section one ? Ok i will improve the like,share,comment,save buttons

so would you be improving that postnav component, which includes these like,share,comment,save button? Try designing them like insta or other socials. That'd be level 1, you can go.
If you're adding the camera component as well, that includes the take picture dialog box, it'd go level 2


Present Prez Profile system config

In Prez docco
<system>
    ex:hasProfiledClass [
        ex:class geo:Feature ;
        ex:profile  # inverse of <Profile> constrains <Class>
            <http://www.opengis.net/def/geosparql> ,
            <http://www.opengis.net/spec/ogcapi-features-1/1.0/req/oas30> ;
        ex:defaultProfile <http://www.opengis.net/def/geosparql> ;
    ] ;
    ex:formats [
        ex:class geo:Feature ;
        ex:profile <http://www.opengis.net/def/geosparql> ;
        dcterms:format
            "application/ld+json" ,
            "application/rdf+xml" ,
            "text/html" ,
            "text/turtle" ,
            "application/geo+json" ,
        ex:defaultFormat "text/html" ;
    ] ,
    [
        ex:class geo:Feature ;
        ex:profile <http://www.opengis.net/spec/ogcapi-features-1/1.0/req/oas30> ;
        dcterms:format "application/geo+json" ;
        ex:defaultFormat "application/geo+json" ;
    ] ,
.

ex:hasConstrainingProfile can be derived from the constrains <Class> details presented by all loaded profiles.
ex:hasDefaultConstrainingProfile is a per-system configuration with Prez shipping with defaults, e.g. as per above, GeoSPARQL is the default for geo:Feature.
Decide where to put this config: is it just another RDF file in the Profiles dir?

This is a profiled view of classes (i.e. a "class" profile), where we are after inbound links to classes e.g. profile X constrainsClass Y. So it can be created with the current profiling mechanisms

would make sense to add /classes and /classes/{class} endpoints, even if they're not exposed in the frontend