YARN-2098. App priority support in Fair Scheduler

https://issues.apache.org/jira/browse/YARN-2098

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 7 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
29m 28s
trunk passed

+1 :green_heart:
compile
0m 58s
trunk passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
compile
0m 51s
trunk passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+1 :green_heart:
checkstyle
0m 47s
trunk passed

+1 :green_heart:
mvnsite
0m 55s
trunk passed

+1 :green_heart:
shadedclient
15m 34s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 41s
trunk passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javadoc
0m 37s
trunk passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+0 :ok:
spotbugs
1m 46s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m 44s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 47s
the patch passed

+1 :green_heart:
compile
0m 49s
the patch passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javac
0m 49s
the patch passed

+1 :green_heart:
compile
0m 41s
the patch passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+1 :green_heart:
javac
0m 41s
the patch passed

+1 :green_heart:
checkstyle
0m 37s
the patch passed

+1 :green_heart:
mvnsite
0m 45s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedclient
13m 52s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 37s
the patch passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javadoc
0m 33s
the patch passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+1 :green_heart:
findbugs
1m 45s
the patch passed

_ Other Tests _

-1 :x:
unit
89m 55s
hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 33s
The patch does not generate ASF License warnings.

164m 36s

Reason
Tests

Failed junit tests
hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairScheduler

hadoop.yarn.server.resourcemanager.applicationsmanager.TestAMRestart

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2293/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2293

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 3edb8c044b9d 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 773ac799c63

Default Java
Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

unit
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2293/1/artifact/out/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2293/1/testReport/

Max. process+thread count
892 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2293/1/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 40s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 7 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
33m 10s

trunk passed

+1 :green_heart:
compile
0m 57s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
0m 50s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
0m 54s

trunk passed

+1 :green_heart:
mvnsite
0m 57s

trunk passed

+1 :green_heart:
javadoc
0m 58s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 46s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m 59s

trunk passed

+1 :green_heart:
shadedclient
21m 15s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 45s

the patch passed

+1 :green_heart:
compile
0m 49s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
0m 49s

the patch passed

+1 :green_heart:
compile
0m 42s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
0m 42s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 39s

the patch passed

+1 :green_heart:
mvnsite
0m 45s

the patch passed

-1 :x:
javadoc
0m 39s
/results-javadoc-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 generated 1 new + 1 unchanged - 0 fixed = 2 total (was 1)

-1 :x:
javadoc
0m 39s
/results-javadoc-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09 with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09 generated 1 new + 1 unchanged - 0 fixed = 2 total (was 1)

+1 :green_heart:
spotbugs
1m 52s

the patch passed

+1 :green_heart:
shadedclient
20m 56s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
99m 53s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 39s

The patch does not generate ASF License warnings.

191m 26s

Reason
Tests

Failed junit tests
hadoop.yarn.server.resourcemanager.applicationsmanager.TestAMRestart

hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairScheduler

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2293/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2293

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 4c0a0853d998 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 09ad0c08d8b511fdf7439f7f462fd8cf5076a48c

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2293/2/testReport/

Max. process+thread count
963 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2293/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


Improved output typing for lookup with inc parameter

Okay, so after experimenting a little bit we would need to do something like this
export interface ReleaseIncludes {
  labels: ILabel;
  recordings: IRecording;
}

and use it in the lookup functions like this:
public lookupRelease<T extends (keyof mb.RInc)[]>(releaseId: string, inc: T) {
  return this.lookupEntity<mb.IRelease & { [K in T[number]]: mb.RInc[K] }, keyof mb.RInc>('release', releaseId, inc);
}

const r = await this.lookupRelease("", ["labels"]);
const title = r.title;
const labels = r.labels.name;
// This throws an error
const recordings = r.recordings;

Originally posted by @Haschikeks in https://github.com/Borewit/musicbrainz-api/issues/504#issuecomment-1812830743

I'll try and get a pull request, with updated lookup return types based on the inc paramter, ready till the end of the week.

Do you want me to wait with a new release until you implemented this one?

no, go ahead. This might take me a little bit longer.

Hey @Borewit
I've pushed to the connected branch. I noticed that some interfaces like IArtist already had some of the possible includes as part of there interface. I've removed some of them but I need to do some further investigation to check all the fields.
I also noticed to problem of different inc parameters not using the key in the json result. One example is
const release = await mbApi.lookup(
      "release",
      "478aaba4-9425-4a67-8951-a77739462df4",
      ["recordings"]
    );

This call results into the following json. But notice that there is no key for recordings, but instead its data is in the media key.
I'm currently not sure on how to handle those cases.
Details
{
  "title": "Anomalie",
  "media": [
    {
      "tracks": [
        {
          "id": "3ff33b1b-aed6-44cc-a1be-4b50aab03dc1",
          "title": "Anomalie",
          "length": 189000,
          "position": 1,
          "recording": {
            "video": false,
            "disambiguation": "",
            "length": 189520,
            "title": "Anomalie",
            "id": "a985da67-a33c-499b-80ef-23f9046b83d5",
            "first-release-date": "2015-10-21"
          },
          "number": "1"
        },
      ],
      "position": 1,
      "track-offset": 0,
      "track-count": 10,
      "title": "",
      "format-id": "907a28d9-b3b2-3ef6-89a8-7b18d91d4794",
      "format": "Digital Media"
    }
  ],
  "status": "Official",
  "quality": "normal",
  "country": "FR",
  "packaging-id": null,
  "asin": null,
  "date": "2016-02-12",
  "packaging": null,
  "status-id": "4e304316-386d-3409-af2e-78857eec5cfe",
  "cover-art-archive": {
    "back": false,
    "count": 1,
    "darkened": false,
    "front": true,
    "artwork": true
  },
  "id": "478aaba4-9425-4a67-8951-a77739462df4",
  "barcode": null,
  "text-representation": {
    "language": "fra",
    "script": "Latn"
  },
  "disambiguation": "",
  "release-events": [
    {
      "area": {
        "iso-3166-1-codes": [
          "FR"
        ],
        "type-id": null,
        "type": null,
        "name": "France",
        "disambiguation": "",
        "sort-name": "France",
        "id": "08310658-51eb-3801-80de-5a0739207115"
      },
      "date": "2016-02-12"
    }
  ]
}

I've pushed to the connected branch.

Maybe good to turn that into a draft PR, that makes it easier to discuss.

I noticed that some interfaces like IArtist already had some of the possible includes as part of there interface.

Indeed.

But notice that there is no key for recordings, but instead its data is in the media key.

Which is declared here:
https://github.com/Borewit/musicbrainz-api/blob/7bd63d8b1b26ad50a0ba6931a2577aa566dd1801/lib/musicbrainz.types.ts#L140
which should probably be made optional:
recording?: IRecording;

I honestly think that is more then enough. I strongly doubt if it is worth to derive the presence of those nested fields from the provided inclusion parameters.

I'm currently not sure on how to handle those cases.

Just keep those as optional as described above.
Maybe better to spend our effort in unit tests, which check if the MusicBrainz result is complaint with our typings. I bet we can shake some errors and incomplete definitions out that way.


Remove explicit go get for rice and goblin pkgs

Both of these packages will be pulled in by the go get invocation on the following line as they are references by imports, so there's no need to list them separately.

Yes, we can definitely remove the reference to goblin. We will still need to go get or go install the github.com/GeertJohan/go.rice/rice package since it is a binary and needs to be installed to $GOPATH/bin

we might be able to do something like this:
go get -t -v ./...
go install go get github.com/GeertJohan/go.rice/rice

the rice repository would get cloned during the first go get and then the binary would be installed with the subsequent command.

Ah, my bad - I saw the import here but that wouldn't include the rice binary.
I think the following would do the trick:
go get github.com/GeertJohan/go.rice/rice
go get -t -v ./...

I've rebased to fix.


Infinite Loop when using Remix-Development-Tools showRouteBoundaries prop

with <RemixDevTools showRouteBoundaries={true} />

Also see https://github.com/Code-Forge-Net/Remix-Dev-Tools/issues/21
This is because root.tsx ThemeProvider would loop infinitely. The way to fix it per @AlemTuzlak is to ensure that RDT is to ensure remix dev tools are supposed to be rendered at the root as far up as possible.

Closed by #211


Throw a runtime error when calling Rewind on unsupported BodyStreams and update the exception message to be actionable.

Fixes https://github.com/Azure/azure-sdk-for-cpp/issues/1704
Throwing invalid_argument, as the issue described, didn't fit well, especially since the API doesn't accept any arguments directly.
Other alternatives are just throw an std::exception (base type of runtime_error).

@JeffreyRichter would it be bad to upgrade our curl and winhttp body streams to support rewind? It already has the connection handle, it only need to keep a copy of the request and re send.. supposing we don't consider any headers like date or things like that important to be maintained

Add changelog

I thought it and his isn't too relevant from an end customer perspective, particularly since it is just a change in exception/message.


Consider removing spark-timeseries

spark-timeseries is officially no longer developed: https://github.com/sryza/spark-timeseries/commit/17b78f7d62f4262e0d8b5d9ee7aec2bf221f1f07.
Should we keep it in the awesome?

Rather deprecating ?

I'd rather it stays. It seems like still "kinda" active even if @sryza isn't officially developing.
@souellette-faimdata are you still maintaining the repo ? or was it a one time thing ?


Actor Memory

Add the ability to remember where we left off by persisting the next binlog position by way of a "saver" actor as the final step in the process.
Save the last known position periodically in case of a crash/hardware failure, and also cleanly persist the last emitted position on shutdown.

[x] Finish wiring up integration tests
[x] Green/Green cleanup/refactor
[ ] Test in staging
[ ] Bonus points for gtid support

https://github.com/mavenlink/changestream/issues/4

https://github.com/mavenlink/changestream/issues/38


Sponsors 2018

@erfanio The old sponsors are still being used in the sponsor us page. I've updated to use the new sponsors images instead of the old ones.

Forwarded you JS logos this morning

@sph-huynh Jane Street has just been added. Merging in now.


Unified way of storing binary data in archives

I started writing serialisation for an Eigen matrix, so save and load functions because the only way to save an Eigen matrix is through a data pointer, since it has an interface like this:
int rows();
int cols();
T* data();

I quickly realised though that there is no single interface to write the binary data across archives (even though json and XML have the saveBinaryValue and loadBinaryValue functions, which are of course not part of the common archive interface, as is the saveBinary and loadBinary functions of the Binary archive).
I already added functions similar to the ones in the binary archive for getting a BinaryData object and saving it properly in the json.hpp and xml.hpp archives. I am going to open a PR soon as well, just wanted to document this here in case there is an objection/different solution.

Have you seen this solution on stack overflow for serializing an Eigen matrix with cereal?

I have, and that's what I tried to implement. That solution only uses the Binary archive. But the BinaryData argument is not accepted by Json and XML, correct me if I am somehow wrong there. I applied the fix that I mentioned, locally and then it works for both those archives.


lbesson.bitbucket.org/md redirects to https://bitbucket.org

When I visit https://naereen.github.io/StrapDown.js/, it's written on top:

StrapDown.js on lbesson.bitbucket.org/md

Yet, visiting http://lbesson.bitbucket.org/md will redirect me to https://bitbucket.org.

Hi,
Yes, the link should be http://lbesson.bitbucket.io/md not http://lbesson.bitbucket.org/md.
Bitbucket, like GitHub, changed the domain used for user websites from bitbucket.org to bitbucket.io years ago.
I fixed this.


tests segfault on OS X

I ran this on f784fc26 on macOS Sierra (10.12.6).
~/rust-rocksdb:master$ cargo test -vvv
[...snip...]
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs
     Running `/Users/dongaro/rust-rocksdb/target/debug/deps/rocksdb-c4d44a0ec3b28825`

running 10 tests
test db::errors_do_stuff ... ok
test db::external ... ok
test compaction_filter::compaction_filter_test ... ok
test db::iterator_test ... ok
test db::test_db_vector ... ok
test db_options::tests::test_enable_statistics ... ok
test db::snapshot_test ... ok
test db::writebatch_works ... ok
test merge_operator::test::mergetest ... ok
test merge_operator::test::counting_mergetest ... ok

test result: ok. 10 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running `/Users/dongaro/rust-rocksdb/target/debug/deps/test_column_family-171e54ab4ded8d69`

running 4 tests
test test_merge_operator ... ignored
test test_create_missing_column_family ... ok
test test_column_family_with_options ... ok
test test_column_family ... ok

test result: ok. 3 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out

error: process didn't exit successfully: `/Users/dongaro/rust-rocksdb/target/debug/deps/test_column_family-171e54ab4ded8d69` (signal: 11, SIGSEGV: invalid memory reference)
~/rust-rocksdb:master$ lldb  /Users/dongaro/rust-rocksdb/target/debug/deps/test_column_family-171e54ab4ded8d69
(lldb) target create "/Users/dongaro/rust-rocksdb/target/debug/deps/test_column_family-171e54ab4ded8d69"
Current executable set to '/Users/dongaro/rust-rocksdb/target/debug/deps/test_column_family-171e54ab4ded8d69' (x86_64).
(lldb) r
Process 71092 launched: '/Users/dongaro/rust-rocksdb/target/debug/deps/test_column_family-171e54ab4ded8d69' (x86_64)

running 4 tests
test test_merge_operator ... ignored
test test_create_missing_column_family ... ok
test test_column_family_with_options ... ok
test test_column_family ... ok

test result: ok. 3 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out

Process 71092 stopped
* thread #2, name = 'test_column_family', stop reason = EXC_BAD_ACCESS (code=2, address=0x7fffb59714c4)
    frame #0: 0x00007fffccbefca9 libsystem_platform.dylib`OSSpinLockLock + 7
libsystem_platform.dylib`OSSpinLockLock:
->  0x7fffccbefca9 <+7>:  lock
    0x7fffccbefcaa <+8>:  cmpxchgl %ecx, (%rdi)
    0x7fffccbefcad <+11>: jne    0x7fffccbefcb0            ; <+14>
    0x7fffccbefcaf <+13>: retq
Target 0: (test_column_family-171e54ab4ded8d69) stopped.
(lldb) bt
warning: could not execute support code to read Objective-C class data in the process. This may reduce the quality of type information available.
* thread #2, name = 'test_column_family', stop reason = EXC_BAD_ACCESS (code=2, address=0x7fffb59714c4)
  * frame #0: 0x00007fffccbefca9 libsystem_platform.dylib`OSSpinLockLock + 7
    frame #1: 0x0000000100d183f0 test_column_family-171e54ab4ded8d69`je_tcache_bin_flush_small [inlined] je_malloc_mutex_lock at mutex.h:99 [opt]
    frame #2: 0x0000000100d183db test_column_family-171e54ab4ded8d69`je_tcache_bin_flush_small(tsd=0x0000000102a6b188, tcache=<unavailable>, tbin=0x0000000102ad1028, binind=0, rem=0) at tcache.c:119 [opt]
    frame #3: 0x0000000100d19594 test_column_family-171e54ab4ded8d69`tcache_destroy(tsd=0x0000000102a6b188, tcache=0x0000000102ad1000) at tcache.c:371 [opt]
    frame #4: 0x0000000100d193ca test_column_family-171e54ab4ded8d69`je_tcache_cleanup(tsd=0x0000000102a6b188) at tcache.c:410 [opt]
    frame #5: 0x0000000100d19d00 test_column_family-171e54ab4ded8d69`je_tsd_cleanup(arg=0x0000000102a6b188) at tsd.c:82 [opt]
    frame #6: 0x0000000100d1a8ca test_column_family-171e54ab4ded8d69`je_tsd_cleanup_wrapper(arg=0x0000000102a6b180) at tsd.h:688 [opt]
    frame #7: 0x00007fffccbfc355 libsystem_pthread.dylib`_pthread_tsd_cleanup + 470
    frame #8: 0x00007fffccbfc0d9 libsystem_pthread.dylib`_pthread_exit + 152
    frame #9: 0x00007fffccbfa946 libsystem_pthread.dylib`_pthread_body + 191
    frame #10: 0x00007fffccbfa887 libsystem_pthread.dylib`_pthread_start + 286
    frame #11: 0x00007fffccbfa08d libsystem_pthread.dylib`thread_start + 13

I have same results

I think this was fixed with #199.  Can you confirm?

#200 shows that tests are running for OS X now. Closing.


added testing for validation module

Hello @DenisSidoren! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file test/test_validation.py:

Line 10:1: E265 block comment should start with '# '
Line 31:1: E302 expected 2 blank lines, found 1
Line 34:1: E302 expected 2 blank lines, found 1
Line 37:1: E302 expected 2 blank lines, found 1
Line 40:1: E302 expected 2 blank lines, found 1


Minified file missing semicolon at last line, causing other scripts concatenated with this file to throw error saying that factory is not a function

.factory("dndDropEffectWorkaround",function(){return{}}) this line needs a semicolon at the end.

Fixed


ENG-338] [OATHPIT] Throw S3 into the Oath Pit

Ticket
https://openscience.atlassian.net/browse/ENG-338
Purpose
Enable and update the S3 provider for aiohttp 3.
Changes
It is unknown why aiohttp-0.18.x stores HTTP response header names in all-uppercase (e.g. 'CONTENT-LENGTH', 'ETAG', etc.). aiohttp-3.5.x now uses the more popular title case (e.g. 'Content-Length', 'ETAG', etc.). This breaks the old implementation for class S3FileMetadataHeaders whose calling self.raw['ETAG'] throws Key Error. The class and related tests has been updated.
Side effects
No
QA Notes
TBD
Deployment Notes
TBD

Coverage increased (+4.6%) to 58.892% when pulling 89c7a96049a24c3bf8f2ebc31ef87c41da3de3e9 on cslzchen:feature/oathpit-s3 into 12f540acf2a5bb9983b5bc7e213b1513797beb07 on CenterForOpenScience:feature/oathpit.


Update README.md

Fixed typos and improved.
Summary of the pull request
Fixed typos and improved.
References and relevant issues
Detailed description of the pull request / Additional comments
Validation steps performed
PR checklist

[ ] Closes #xxx
[ x] Tests added/passed
[ x] Documentation updated

Hi @Techuuu! Thanks for taking the time to suggest changes to the wording, but I think we're going to stick to the wording and grammar we currently have.


file in npm package is in year 2147

the following file: src/__tests__/fixtures/cli.watch-import-import.css has an incorrect date in the npm package of cssnext:
-rw-r--r--  1 me  wheel     7B 14 sep  2147 cli.watch-import-import.css
the file has a correct date when cloning the repo from github

Haha that's weird. Is this a real issue or just a small detail ?
I am not sure why npm changes the dates...

small detail, but in case someone wants to make an archive containing this it will give an error saying that some files are in the future

If you have a minute could you check with latest version (1.8.1)? Thanks in advance.

same problem

Not relevant anymore (I guess). cssnext package is deprecated in favor of postcss-cssnext.


💥 EDA on Netflix Data

💥 Proposal
Performing Exploratory Data Analysis on Netflix Dataset.
Netflix Dataset Kaggle
@khusheekapoor
@theyashwanthsai
Can you please assign me this issue?

@NisargPipaliya  - you can go ahead! We are assigning you 21 days for this project, after which it will be assigned to someone else if not completed. All the best!
Name the file as: algorithm_dataset.ipynb and link it in the readme of the labeled directory as algorithm - dataset.

As this issue has not been mentioned in any PR, Can I work on this issue?


client: throw on error

Hi, I'd like the platformatic/client methods to throw if they receive an error level response, which should be reasonbly trivial to add as an option but since the default behaviour is to return only the body, you would not be able to decern the call errored based on the status code, only on the response data. So should throwOnError also before default behaviour? Or just separate option.
I will happily open a PR for either, ta.

So should throwOnError also before default behaviour? Or just separate option.

I don't understand this

So I wanna add a throwOnError option to the platformatic client which will set the same option on the undici request call:
app.register(pltClient, {
  type: 'openapi',
  name: 'movies',
  path: join(__dirname, 'movies.openapi.json'),
  url: opts.url,
  throwOnError: true,  // adding this
  fullResponse: false
})

So should throwOnError also before default behaviour? Or just separate option.

By this I just meant should throwOnError default to true or false? Presumably false which would reflect the current behaviour!


Error in Build FastDeploy Library on Nvidia Jetson Platform

Environment
FastDeploy version: the latest code in develop branch
OS Platform: Jetson AGX Xavier
Program Language: e.g. Python 3.6
Problem description
Python build success with only onnxruntime on CPU. When running with the trt backend, libonnxruntime.so is not found.

This shouldn't happen. How did you build the FastDeploy?

I only found libonnxruntime.so. Also, if I am not wrong, Jetson Zoo has prebuilt onnxruntime-gpu for jetson. https://elinux.org/Jetson_Zoo#ONNX_Runtime

There should be a libonnxruntime.so.1.12.0, I'm not sure why this happen, could you rebuild the wheel, and check if it works?
And for the Jetson Zoo, I haven't found a prebuilt onnxruntime c++ library for jetson yet, if you know, please let me know.

此ISSUE由于一年未更新，将会关闭处理，如有需要，可再次更新打开。


Add metadata endpoints to the schema

WIP in #5
Initial endpoints proposal targeting accounts, should be duplicated per mosaic and namespaces.

GET /account/{accountId}/metadata/
Returns the complete accountId current state metadata. Ideally with pagination + order.

GET /account/{accountId}/metadata/{key}
Returns all the account current state metadata for that key. The response should be an array, as multiple signers can add metadata with the same key.

POST /account/{accountId}/metadata
Returns metadadata for an array of accountIds.

Let's use this issue to discuss changes in those endpoints and discuss if we need more of them.

Close #5


Chronograf 1.8

Updates for Chronograf 1.8
Hi @glinton, @hoorayimhelping, @TCL735, @russorat please review questions in update Chronograf 1.8 release notes...and @russorat, will you please review upgrade doc changes too?
Thank you!

Looks good to me once the date is updated. Thanks so much Kelly!

There's also https://github.com/influxdata/chronograf/pull/5362 that i'm open to feedback from y'all on the choices of terminology/flags/defaults.

@kelseiv Be sure to update /data/default_versions.toml and /data/versions.toml with the appropriate values.

There's also influxdata/chronograf#5362 that i'm open to feedback from y'all on the choices of terminology/flags/defaults.

thank you, @glinton!

Also, does this address #2610 ?
yes

could we ensure this option is doc'd as well: https://github.com/influxdata/chronograf/pull/5308


@eessex => Limit to 10 on /articles fetch

In order to get an accurate read of performance, lets do a fetch for 10 instead of 30, which is what we previously had on this page.

Lookin good!

Ok, merging!


An Error Occurred, when adding a team

After i add a team (with the admin as a member, or a single user as a member). I cant view the page anymore. Receiving the message:
An error occurred when processing your request. Please try again or contact the system administrator.
I cant find a detailed error in the Windows event log, is it possible to get this from somewhere else?
fyi: im running this on a Windows 10, iis (32bit).

This is a dublicate of #331 and I have suggest a correction in #332 - you can try out - unless you will wait for a new release :) @mvklingeren

Closed as duplicate


NeighborSampler for k-hop sampling

❓ Questions & Help
I have a hard time understanding NeighborSampler even with small examples. It doesn't make sense in most cases.
l = np.array([[0, 1], [1, 4], [1, 5], [1, 6], [0, 2], [2, 7], [0, 3], [3, 8], [3, 9], [3, 10], [3, 11]]).transpose()
r = np.array([l[1, :], l[0, :]])
edge_index = torch.from_numpy(r)

train_loader = NeighborSampler(edge_index,
                               sizes=[2, 2], batch_size=1)

for batch_size, n_id, adjs in train_loader:
    print("###")
    for edge_ind, e_id, size in adjs:
        print(e_id)

gives me:
tensor([ 4,  0, 10,  8,  3,  1])
tensor([6, 0])

It looks like it jumps over edges?
How can I use it for k-hop sampling?

I also tried TwoHop() with edge_attr, but it gives me error for not symmetric graphs, even the example you have in test is a symmetric one. Is there any way around this?

The NeighborSampler example looks correct to me. It samples two edges in the last layer for node 0: to nodes 1 and 3. in the first layer, it samples 6 edges for nodes 0, 1 and 3: to nodes 1, 4, 6, 2, 9, 11.
Regarding your second issue: You pass in two edges, but 3 edge features. So that‘s an expected crash and isn‘t related to symmetric adjacencies.

The NeighborSampler example looks correct to me. It samples two edges in the last layer for node 0: to nodes 1 and 3. in the first layer, it samples 6 edges for nodes 0, 1 and 3: to nodes 1, 4, 6, 2, 9, 11.
Regarding your second issue: You pass in two edges, but 3 edge features. So that‘s an expected crash and isn‘t related to symmetric adjacencies.

Thanks for your response. Is there any simple way to get the edge_id for edges between hops? for example if I have:
edge_index = torch.tensor([
    [0, 1],
    [1, 2]
])

and TwoHop gives me 0-2. Is there any way to get the edges that create this without changing the internal code?

I am not really sure what you mean. TwoHop just performs a sparse-sparse matrix multiplication.

The NeighborSampler example looks correct to me. It samples two edges in the last layer for node 0: to nodes 1 and 3. in the first layer, it samples 6 edges for nodes 0, 1 and 3: to nodes 1, 4, 6, 2, 9, 11.
Regarding your second issue: You pass in two edges, but 3 edge features. So that‘s an expected crash and isn‘t related to symmetric adjacencies.

why are edge 0 (node 1 to 0) and edge 4 (node 2 to 0) sampled in the first layer? all the rest makes sense to me, but not these two edges.
source nodes 4 and 6 send to target 1, source nodes 11 and 9 send to target 3 in the first layer, and in the second layer source nodes 3 and 1 send to target 0. why these other two edges?

Edge 0 and Edge 4 are sampled for node 0 in layer 1. The NeighborSampler will recursively sample neighbors/edges for all nodes encountered in all previous layers, e.g., for applying skip-connections.


Possible memory leak?

I found that loaded audio resources will not be released after it stops, here is a quick example:
use anyhow::Result;
use kira::{
    manager::{backend::DefaultBackend, AudioManager, AudioManagerSettings},
    sound::static_sound::{StaticSoundData, StaticSoundSettings},
    tween::Tween,
};

fn main() {
    play().unwrap();
}

fn play() -> Result<()> {
    // Create an audio manager. This plays sounds and manages resources.
    let mut manager = AudioManager::<DefaultBackend>::new(AudioManagerSettings::default())?;

    let path = std::env::current_dir()
        .unwrap()
        .join("assets/audio/test.ogg");

    let file = std::fs::File::open(path)?;

    let sound_data = StaticSoundData::from_media_source(file, StaticSoundSettings::new())?;

    let mut handle = manager.play(sound_data)?;

    let mut i = 0;

    loop {
        std::thread::sleep(std::time::Duration::from_secs(1));
        i += 1;

        if i == 10 {
            handle.stop(Tween::default())?;
        }
    }
}

The music will play for 10 seconds and then stop, seeing that no memory usage drop in task manager.
I think it is just because a missing implement of removing it here:
https://github.com/tesselode/kira/blob/4c047c3cc60d4f778416e09bc910960d1b482cba/crates/kira/src/manager/backend/resources/sounds.rs#L35-L48
Stopped sound is moved to unused_sound_producer but no next steps, while simply comment out line 40~45 will fix the problem.
Not sure if the problem exist in other backends or wasm.

Hello!
All finished sounds are unloaded whenever you play a new sound - you can see the code that does that here.
while self
	.unused_resource_consumers
	.sound
	.get_mut()
	.expect("unused resource consumer mutex poisoned")
	.pop()
	.is_some()
{}

Other resources work this way as well.
While removing the lines you mentioned would cause sounds to be unloaded as soon as they finish, it would also mean memory is deallocated on the audio thread, which in rare causes could cause undesirable audio stutters. So this mechanism of sending finished sounds through an SPSC to the gameplay thread to be deallocated later is intentional.


Fill KafkaAutoConfigurationTest

Currently there is an empty test class called KafkaAutoConfigurationTests.
This should obviously be filled up with actual tests of the Kafka auto configuration suite.

Thanks for creating the issue, was waiting for SpringBoot2PR to be merged, since its merged now PR should be soon.


Comment name for /// doesn't match ST3 convention

I came across this issue while troubleshooting an issue in the Wrap Plus bundle for ST3. The "dartdoc" style comment /// is named TM_COMMENT_BLOCK_START, which doesn't fit the ST3 convention. The convention says it should be called TM_COMMENT_START_2.
Indeed, other plugins (such as Wrap Plus) cannot detect the existence of this comment style because of the misnomer.
diff --git a/Support/Comments.tmPreferences b/Support/Comments.tmPreferences
index 72ab5c9..fd20fde 100644
--- a/Support/Comments.tmPreferences
+++ b/Support/Comments.tmPreferences
@@ -24,7 +24,7 @@
                        </dict>
                        <dict>
                                <key>name</key>
-                               <string>TM_COMMENT_BLOCK_START</string>
+                               <string>TM_COMMENT_START_2</string>
                                <key>value</key>
                                <string>/// </string>
                        </dict>

A quick grep through the project suggests that TM_COMMENT_BLOCK_START isn't referenced anywhere else, so it seems like it would be a safe change. Thoughts?

Thanks for reporting!
I think the problem is that Dart currently has multiple types of comments. In fact, /// signal doc comments, not comments, so I don't really think they belong with other comments.
For example, Visual Studio has a shortcut for // comments in C#, but not for /// (as far as I know).
In any case, if it's still named ..._BLOCK, that's an issue.
BTW, I wrote those ST docs. :)

It is a problem for packages that want to process comment text, such as @ehuss's Wrap Plus package. For example, see https://github.com/ehuss/Sublime-Wrap-Plus/issues/22.

@guillermooo

In any case, if it's still named ..._BLOCK, that's an issue.

On master:
		<dict>
			<key>name</key>
			<string>TM_COMMENT_BLOCK_START</string>
			<key>value</key>
			<string>/// </string>
		</dict>

The ST3 docs say:

TM_COMMENT_START
Defines a default comment marker.
To define additional comment markers, name them TM_COMMENT_START_2, TM_COMMENT_START_3, etc.

The TM_COMMENT_BLOCK_START key doesn't appear anywhere in the Sublime Text documentation. I can't find anything on Google that indicates that It's called TM_COMMENT_BLOCK_START` was ever a valid configuration key for Sublime Text.
May I submit a PR to correct this?

@guillermooo Any update on this? I'm happy to submit a PR. Or discuss further if you're not yet convinced. Thanks!


♻🚸Dialog: refactor to use native dialog

resolves #2949
removes Scrim and floating-ui as dependencies and instead uses a native <dialog> element in modal mode (uses DialogElement.showModal())
I kept the functionality the same as the old Dialog to not introduce breaking changes (which means i did block pressing escape to close unless isDismissable is true, maybe something to discuss

Note to self: check dialog inside dialog case and hidebodyscroll


Dynamic weights of backends

Create a sidecar container that runs in the same pod as your backend and reports it's qps to a service annotation. The servicelb will lookup these service annotations and update the weights of the backends by writing to the haproxy socket (or  https://github.com/kubernetes/contrib/pull/120). The goal here is to send less traffic to backends that have historically been slow to repond (so not exactly leastconns). I expect this to help with noisy neighbors.
@aledbf @jayunit100

Ohh you can probably just wrap this into a sidecar https://github.com/ssasso/lbwwd

cc @timothysc @rrati probably interesting to us for our scale work.

The goal here is to send less traffic to backends that have historically been slow to repond

@bprashanth this sounds interesting but I see one potential problem with something like this.
For instance if some application has just 3 urls, 2 are GET and 1 POST. The GET requests return in less than 1 second and the POST requires at least 5 seconds
Then:

adjusting the weight without "context" (GET vs POST) is not "correct"
is not possible to get stats per request using the haproxy socket (this requires parsing the haproxy log)
what's the definition of noisy neighbors?

Maybe using the stats about the las 1024 can help to change the weight (and it's available in the socket stats)

what's the definition of noisy neighbors?

If you have 2 machines with 3 containers, 2 of which are nginx backends and 1 is something heavy, say a database, and the database machine start hammering cpu, how do I send all requests to the other nginx backend?

Maybe using the stats about the las 1024 can help to change the weight (and it's available in the socket stats)

This is actually 2 problems:

how do we figure out the load/response time/stats of a given backend?
how do we tell haproxy about this, so it can adjust the weights?

If we solve (2) we can iterate on 1. I haven't completely understood how haproxy modifies the rr weights yet. From the docs:
This is the smoothest and fairest algorithm when the server's
processing time remains equally distributed. This algorithm
is dynamic, which means that server weights may be adjusted
on the fly for slow starts for instance.

have you had a chance to play with that?

how do we figure out the load/response time/stats of a given backend?

Parsing the stats from haproxy (it returns a csv). For example: curl http://<service-loadbalancer-ip>:1936/;csv
http://cbonte.github.io/haproxy-dconv/configuration-1.5.html#9.1
 58. qtime [..BS]: the average queue time in ms over the 1024 last requests
 59. ctime [..BS]: the average connect time in ms over the 1024 last requests
 60. rtime [..BS]: the average response time in ms over the 1024 last requests
     (0 for TCP)
 61. ttime [..BS]: the average total session time in ms over the 1024 last
     requests

This is grouped by type (frontend or backend), name (service name) and server (ip:port) so checking this is not complex

how do we tell haproxy about this, so it can adjust the weights?

Using the haproxy socket is possible to dynamically adjust the weight http://cbonte.github.io/haproxy-dconv/configuration-1.5.html#set weight


Invariant Violation: Native module cannot be null.

Hey, not sure if this is the best place to ask but i just started using react-native-mock and I keep getting this error which I am not sure how to fix and might be related to the library.

    Invariant Violation: Native module cannot be null.

      at invariant (node_modules/fbjs/lib/invariant.js:44:7)
      at RCTNetworking.NativeEventEmitter (node_modules/react-native/Libraries/EventEmitter/NativeEventEmitter.js:32:1)
      at new RCTNetworking (node_modules/react-native/Libraries/Network/RCTNetworking.ios.js:23:165)
      at Object.<anonymous> (node_modules/react-native/Libraries/Network/RCTNetworking.ios.js:59:16)
      at Object.<anonymous> (node_modules/react-native/Libraries/Network/XMLHttpRequest.js:15:19)```

Looking at that stack trace, it probably is a react-native-mock bug. However it's very difficult to tell what the issue is with just that stack trace. Can you try and reproduce it more clearly with an example, or provide a stack trace

I'll try to reproduce in the meantime here is some more info. The above is what I keep getting when I try to run jest, so currently can run any tests.
What led me to use react-native-mock was the fact that I couldn't mock NativeEventEmitter which we are using.
dependencies
  "dependencies": {
    "babel-plugin-transform-decorators-legacy": "1.3.4",
    "bugsnag-react-native": "2.2.0",
    "class-autobind": "^0.1.4",
    "color": "0.11.3",
    "lodash": "4.15.0",
    "moment": "2.17.1",
    "react": "15.4.2",
    "react-native": "0.42.3",
    "react-native-calendar": "0.10.0",
    "react-native-extended-stylesheet": "0.3.0",
    "react-native-fs": "2.1.0-rc.1",
    "react-native-keyboard-aware-scroll-view": "0.2.3",
    "react-native-material-kit": "0.4.1",
    "react-native-sensitive-info": "3.0.2",
    "react-native-snap-carousel": "1.2.0",
    "react-native-svg": "5.1.5",
    "react-native-vector-icons": "2.1.0",
    "react-redux": "4.4.5",
    "redux": "3.6.0",
    "redux-async-action": "0.0.4",
    "redux-thunk": "2.1.0",
    "victory-native": "0.7.0"
  },
  "devDependencies": {
    "babel-eslint": "6.1.2",
    "babel-jest": "20.0.3",
    "enzyme": "2.8.2",
    "enzyme-to-json": "1.5.1",
    "eslint": "3.7.1",
    "eslint-config-airbnb": "10.0.1",
    "eslint-plugin-import": "1.16.0",
    "eslint-plugin-jsx-a11y": "2.2.3",
    "eslint-plugin-react": "6.4.1",
    "gradle-to-js": "0.2.5",
    "husky": "0.11.9",
    "jest": "20.0.4",
    "localenv": "0.2.2",
    "react-addons-test-utils": "15.4.2",
    "react-dom": "15.4.4",
    "react-test-renderer": "15.4.2",
    "redux-mock-store": "1.2.3",
    "request": "2.81.0",
    "react-native-mock": "0.3.1"
  }

Jest config
"jest": {
    "preset": "react-native",
    "verbose": true,
    "setupFiles": ["./jestsetup.js"],
    "snapshotSerializers": [
      "enzyme-to-json/serializer"
    ],
    "transformIgnorePatterns": [
      "node_modules/(?!react-native|bugsnag-react-native)/"
    ]
  },

jestSetup.js
require('react-native-mock/mock');

jest.mock('NativeModules', () => ({
  Environment: {
    BUGSNAG_API_KEY: 'ApiKey',
  },
}));

Hmm, interesting. I'd never considered people wanting to use react-native-mock with jest, but i guess it should work. But yes please try and narrow down exactly why it happens. It's possible it's some conflicting between react-native-mock and jest's builtin mock

@miguelc1221 I am also experiencing the same issue. Have you found a solution to this?

+1

I get this because KeyboardManager is not included in the mocked native modules


When I had a jsbundle

Description
When I have a jsbundle, I perform release compilation on Xcode, but it will still be overwritten by the jsbundle generated by the bundle react native code and images. However, if I empty the contents of the bundle react native code and images, it is correct
Version
any
Output of npx react-native info
System:
OS: macOS 12.5
CPU: (8) arm64 Apple M1 Pro
Memory: 95.64 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 18.7.0 - /opt/homebrew/bin/node
Yarn: 1.22.19 - /opt/homebrew/bin/yarn
npm: 8.15.0 - /opt/homebrew/bin/npm
Watchman: Not Found
Managers:
CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 15.5, macOS 12.3, tvOS 15.4, watchOS 8.5
Android SDK: Not Found
IDEs:
Android Studio: Not Found
Xcode: 13.4.1/13F100 - /usr/bin/xcodebuild
Languages:
Java: Not Found
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.68.2 => 0.68.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
When I have a jsbundle, I perform release compilation on Xcode, but it will still be overwritten by the jsbundle generated by the bundle react native code and images. However, if I empty the contents of the bundle react native code and images, it is correct
Snack, code example, screenshot, or link to a repository
react native inti any

Snack, code example, screenshot, or link to a repository
react native inti any

Please fill in the issue template + provide a reproducer as the current informations are not sufficient to triage this issue.
I'm closing for the time being, feel free to re-open once you can provide the details.


Must be running with administrator priviledges to listen on non-localhost addresses

Question
A clear and concise description of your question.
Hi,
I have a pode script installed as a windows service, but I want to be able to run it under a service account that has non-admin privileges. Currently if I run it as local system, it work fine, but when I run it as a non-admin account I get the error
"Must be running with administrator priviledges to listen on non-localhost addresses"
Is there any way to run the script as non-admin windows service account?

Thanks Badgerati,
If I set the address to be localhost and hostname the FQDN of the server, it starts under the service account, but I can't access it remotely. I get
"Invoke-RestMethod : Unable to connect to the remote server"
Is there anyway to have the address as localhost and be able to access it remotely?

Hey @berki7867,
Unfortunately not, the address has to be all (*) or the IP of the machine if you want to access the script remotely.
You can run it using localhost and access remotely, but you'll need to use a tool like IIS or NGINX to proxy traffic for you. For NGINX I'm not too sure, but with IIS your script will have to be run via IIS and not as a Windows Service.

Do you have temporary admin access?
You could possibly NAT a port on the local windows firewall to a high port that your user account can run.
@Badgerati, wouldn't using netsh http add urlacl as admin be an option too?

@RobinBeismann I did consider netsh http, but thought that only works for tools using HTTP.SYS?

Ah correct yeah - that won't work for PodeListener.
This would work through:
https://docs.microsoft.com/en-us/windows-server/networking/technologies/netsh/netsh-interface-portproxy#add-v4tov4
We had to use a local portproxy to tunnel an application which did not support IPv6 over an IPv6 based VPN - works pretty reliable. However it requires the user to atleast have admin rights once in order to create the portproxy.

@RobinBeismann Oh wow, yeah that works!
netsh interface portproxy add v4tov4 listenport=<external-port> connectaddress=<IP_ADDRESS> connectport=<pode-port>

#ie:
netsh interface portproxy add v4tov4 listenport=8085 connectaddress=<IP_ADDRESS> connectport=8086

As you said it requires admin initially, but after that you can run pode on localhost as non-admin. I'll have to get that into the External docs 😮 thanks!
@berki7867 the above should work for you, to access externally but running Pode as non-admin on localhost 😄

Thanks both this worked.
Thanks again 😊


Error changing dnx port

The actual code is dnx --port 5050 . web and yields (in spanish.. sorry)

System.Net.Sockets.SocketException (0x80004005): No se puede establecer una conexión ya que el equipo de destino denegó expresamente dicha conexión <IP_ADDRESS>:5050
en System.Net.Sockets.Socket.DoConnect(EndPoint endPointSnapshot, SocketAddress socketAddress)
en System.Net.Sockets.Socket.Connect(EndPoint remoteEP)
en Microsoft.Framework.Runtime.DesignTimeHostProjectCompiler..ctor(IApplicationShutdown shutdown, IFileWatcher watcher, IRuntimeOptions runtimeOptions)

It runs ok on port 5000 without the --port option, strange enough it fails on the same port using --port.

Unfortunately, thats the wrong flag. That's actually the compilation server port, not the webserver port. We'll need to rename that in the future to avoid confusion.

@davidfowl Thanks!
How can I set the port?

Found it on the project.json, I had already checked there but didn't see it since it looks like a link. Consider making "web" and object so you can express it more naturally as json
"commands": {
      "web": {
          "host": "localhost",
          "port": 5050,
          ....
      }
  }

I'm not sure we have a doc on how commands works ( @danroth27 ), but web is not a special command. Look at the value of it. The name can be anything, the value is what's important.


How to make x-axis start from 0 ?

I have searched a lot on this and was not able to find any solution on this in latest chart library.
I have created a fiddle on this http://stackoverflow.com/q/42992645/7402844
What I basically want is to start the x-axis at 0 of the graph?

@ultraxion is correct.
scaleOptions: {
  ticks: {
    beginAtZero: true;
  }
}


**continuous-integration/travis-ci/pr ** — The Travis CI build failed

I cannot upload my token image I have an error with travis

Fixed


Refactor to_chars_result to reflect 22.13.2

Uses std::errc instead of int. Also adds definition of operator== since defaulting the operator is not available until C++20.

I used int to avoid including the entire <system_error> for just std::errc. It's a relatively heavy include because it pulls <string> and most people don't want that; for embedded uses, <string> might even be forbidden.

That makes sense to me. Are we planning for embedded uses, and need to avoid throwing as well?

In general, each new functionality should come with a test. In this case, we need a test for chars_format and its operator==.
Also, I think that == default in C++20 also causes operator!= to be defined, so we need this and a test for it as well.

Are we planning for embedded uses, and need to avoid throwing as well?

This functionality (directly or indirectly via Boost.JSON) is definitely useful for embedded, yes.
It would definitely be good if these functions can be noexcept. I know that some corner floating-point cases require multiprecision arithmetic, so we might need to allocate and hence throw, but it would be good to avoid that in as many cases as possible.

Also, I think that == default in C++20 also causes operator!= to be defined

Indeed it does, https://godbolt.org/z/Yj3YWTGzc.

A general comment on the inline friend in the standard: the function is declared as a friend inside the struct definition, even though all the members are public and therefore there's no need for it being a friend at all, to denote an inline friend. That is, a function whose definition is inline in the friend declaration.
This is a special kind of function called "inline friend" or "hidden friend" because it's not visible to normal lookup, and therefore does not contribute to the enormous overload sets that are usually formed when code does x == y where one of x or y is in namespace std. A hidden friend only participates in overload resolution when the class is mentioned.
TL;DR:
struct to_chars_result
{
    char* ptr;
    int ec;

    friend bool operator==(const to_chars_result& r1, const to_chars_result& r2)
    {
        return r1.ptr == r2.ptr && r1.ec == r2.ec;
    }
};


Enable/disable xdebug or multiple php instances

Currently you need to rebuild the php image to turn xdebug on and off. This thread is for working on a better way. Perhaps running multiple versions of php, or having an easy way to turn it on/off on the fly.
https://github.com/sinnbeck/laravel-served/issues/25#issuecomment-720344288

As a possible solution, it is now possible to enable/disable xdebug using php artisan served:xdebug enable and php artisan served:xdebug disable.
If anyone wants to try it out, it can be installed using
composer require sinnbeck/laravel-served:dev-xdebug

@sinnbeck in my mind that is far better than having to rebuild the container. Thanks for listening.

@zendraxl Glad to hear that you like it. It is now part of the main branch :)


rhea: scale less

It's been commonly hapenning that we spin up many machines but can't keep them occupied due to bottlenecks in the central machine (probably it's mostly the compression of copying-results step)
So let's scale less aggressively and thus waste less.

I forgot to say that this has been deployed for about 30h now, but let me open it here for discussion (and further tweaks perhaps).

Ideally we'd have some mechanism that prevents further scaling if rhea can't keep up feeding the machines with work.  Currently the scaling is based just on the count of jobs ready to be built, but there clearly are some other bottlenecks sometimes.
My understanding is that we'll be using the scaling much less by May anyway, so the impact of this issue should become lower than now.

The changes are intentionally harsher for aarch64 than for x86_64, as that seems desirable.

It's true that we shouldn't keep this open/unresolved for too long, as deploying something else to rhea would undo the changes.


vmauth: don't mount config in the vmauth container when using the cus…

…tom reloader
When the custom reloader is in use, there is no config volume defined and the vmauth container cannot start with a config volume mount.
This patch also adds basic tests for the vmauth pod spec.

address https://github.com/VictoriaMetrics/operator/issues/962


In the readme it is not cleat that "Create and compile component" is an alternate to the one above

Section "Create a new Vue project" doesn't end with "try this in your browser now".
Therefore section "Create and compile component" is almost like a step after, when it is really an alternate to he first section for the end-user educating themselves.  It itself should end wthj a "try this in your browser now ..  .. etc".

"Create and Compile" actually is a step after "Create a new Vue project" step.  The four lines in the "Create a new Vue project" step will bootstrap everything for you so you can just create and compile in the next step.  But I do realize that I probably need to add a bit more detail and maybe step numbers.


Stability improvements

This PR improves general robustness of the pipeline by fixing some bugs encountered while reviewing the logs of failed web jobs. See also https://github.com/nextflow-io/nextflow/issues/99

Cool, thanks. I'll update the CHANGELOG later.


Spanned documentation is missing from rustdoc again

Basically #309 regressed. Seems to be caused by https://github.com/alexcrichton/toml-rs/commit/d99f5665b420f570c4aa26602360f7d5eaee1dc9. I think the doc(no_inline) should be removed again, or the spanned module made public?

Yea, I'm not sure why the no_inline was added.  @bgeron  can you clarify why no_inline was added in #377?

Shoot. I didn’t realize that that introduced a bug. I think we should revert both of the no_inline lines I added in that commit. Sorry!
I guess my rationale was stylistic reasons, not important.
Can you revert the two lines for me please?
Thanks!
Am Mo, 2. Nov 2020, um 00:58, schrieb Eric Huss:

Yea, I'm not sure why the no_inline was added. @bgeron https://github.com/bgeron can you clarify why no_inline was added in #377 https://github.com/alexcrichton/toml-rs/pull/377?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/alexcrichton/toml-rs/issues/408#issuecomment-720173255, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHIAOEX5C5XN3SEGV2BXD3SNXY25ANCNFSM4TGZXFXA.

Great find!
Am Mo, 2. Nov 2020, um 07:13, schrieb Bram Geron:

Shoot. I didn’t realize that that introduced a bug. I think we should revert both of the no_inline lines I added in that commit. Sorry!
I guess my rationale was stylistic reasons, not important.
Can you revert the two lines for me please?
Thanks!
Am Mo, 2. Nov 2020, um 00:58, schrieb Eric Huss:

Yea, I'm not sure why the no_inline was added. @bgeron https://github.com/bgeron can you clarify why no_inline was added in #377 https://github.com/alexcrichton/toml-rs/pull/377?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/alexcrichton/toml-rs/issues/408#issuecomment-720173255, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHIAOEX5C5XN3SEGV2BXD3SNXY25ANCNFSM4TGZXFXA.


[BUG] Refreshing everything is breaking the UX

When someone recently worked on the refresh event something broke. Now if i share a URL this gets cleaned up once I visit it.
Try it by going to :
https://eos-icons.com/?iconName=bootstrapping&type=static
then see that the URL would have cleaned now, so you can't copy-paste it again.
Additionally, something major is what happens when:
1- Search for an icon name, for instance, "pin"
2- Now click on the Pin icon
Results:
the search form is cleaning up (this is because the page is refreshing)
the URL isn't populating with the parameters to be able to share the Pin icon.
Can someone take this issue ASAP?

I would like to work on this issue.

I am planning to do the following -

Everything including URL and search params remain intact on refresh
Fix behaviour of clicking on icons after search
Retain the search and url params in a shared link
Implement auto scroll to selected Icon in a shared link( I need help for setting up the build for this issue)


Lualatex

Lualatex uses a cache folder which is not accessible if you change the user to something else than root.
The error:
quiting: fix your writable cache path

My fix:
RUN chmod 1777 /var/lib/texmf

This changes the permissions so any user can write to the cache folder.  The location of the cache folder is specified by the TEXMFCACHE environment variable. More info:
https://wiki.archlinux.org/index.php/TeX_Live#Usage

On second thought, this is probably better:
ENV TEXMFCACHE /tmp/texmf/

I made a docker file:

docker-lualatex
docker-lualatex-build

The first version includes only the packages needed to run lualatex. The size is 316MB, which is the lowest I could get without mucking with vanilla texlive (still way better than 3GB+ full image). The second additionally includes build tools (latexmk and make).


A very small plot_convergence improvement

The x axis starts from iteration 1 instead of iteration 0, for less ambiguity and more readability.
This is the code that was used in the final version of my thesis.
Ready for a cursory review @whoburg @bqpd

Looks good to me.


Generate and manually add ATTRIBUTION file in native-web

Description
Add script to generate attribution file

Closing as done


Merge search & form

I believe I haven't broken any functionality. I tried to test the form and it looks like it works perfectly.

@viktor-yakubiv It will be probably very hard for you to review code. I don't expect that you will go through it. Please focus more on functionality/design if possible. Thanks :)

It works and looks like a charm ✨
A few small remarks:

Change the navigation link to /data-providers accordingly. The current /data/providers will be removed.

We will need to change links on the static website too.
After some time we will need to remove the page from the static website.

If the button change URL, it should probably be a link. Let's think about it, at least, I am not completely sure about the URL structure but I like how it's done now.
The 'Become data provider' button does not scroll down if the form is opened already, it's not expected behaviour.

You can reproduce this if enlarge the page clicking 'Load more' once or twice, then click 'Add data provider' and then scroll up and click 'Become data provider'
I understand the technical issue with this so I don't mind if you omit this fix.

Out of the PR scope:

Footer's 'Browse' should point to /browse.
The number of data providers (10230) could have a nice formatted look (10,230)
The text actually contains 2 paragraphs, can this be restored?
Are the statistical numbers in sync with the API?
Can you enlarge the map twice or 3 times? It's not useful so small.

If you maintain the ratio, it can be 1:1 or better 1:2. Other more standard option — 2:3. Whaever better to your eye.

Also, completely unrelated, it would be nice if this could be solved. If you have any idea why this is happening.


change temp regex to use decimal values (Issue 1)

Changed regex so that it can support new flashforge firmwares that return decimal temps.

I don't think the regex is completely right.
I have added one test with the network response you gave me and changed the regex in the main branch. I'll update the HASS integration in a moment...
Thanks again for your effort and time and pointing me to right line in the code :)

Ok, I thought I tested the regex, but since I'm not a python person who knows...if it was c# I would have known exactly what to do ;)


Make build faster

This should make the build a bit faster!

Make sure we fix #370 as part of this.

Problem: Grunt
Solution: Gulp
Problem: Gulp

Fair point. The issue should really just be 'make the build faster' - I don't really care which took it uses!

Done :)

Concrete things to consider:

The CSS is built / minified before JS unit tests are run, which means that the build is slow to report breaking changes
Do the visual tests need to use the minified / bundled output?
BrowserSync?

I expect that time-grunt would be useful.
This is the output when running grunt ci:

I've had a go at running the jasmine tests using jsdom rather than phantom (jsdom plays nice with Windows since node 4) and that seems to make a big difference to the jasmine:test task. I've also spotted we're jshint/jscs-ing twice and there's no need to run uglify for dev builds.
That adds up to about 50% of the time taken, I'd suggest we start there.

I also tried to use grunt-newer to only lint changed files but I couldn't get it to work. It looks like it's been abandoned as a project though so I only mention it so that no one else has the same problem.

The build is certainly a lot faster at reporting unit test failures, which is good :-)

I'm happy with the build speed now :-)


defaultFramebufferObject

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin please rerender

@conda-forge-admin please rerender

for my education, could you explain what you are looking for?

hello, I think I found the cause of #45

🎉
I can confirm the issue is solved!
hasattr(QOpenGLWidget, "defaultFramebufferObject")
reports True on macOS 13.1 arm64 for pyside2 build live on conda-forge.
Huge thanks! 🙏


Config changes to reflect the default branch from master to main. Please merge to complete the default branch renaming job.

Auto branch renaming content.

Docs Build status updates of commit 6521657:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add schema as a post setting

Hello 👋
According to this documentation, Ghost automatically handles the Article schema for each post, which is great. However, it is not possible to customize it to use something different from Article.
The use case here is to have the NewsArticle schema. Here is what I suggest in that PR:

Allow a new setting in the post settings, so that the user can choose the NewsArticle schema instead of the default article.
Use that flag in schema.js to create a dedicated NewsArticle schema for that post.

I tried to make things work correctly, but I am not able to save the update. When I change the dropdown value and click on Update, I receive a success message (PUT 200) about the post being updated, but it still uses the previous value. What did I miss here?
Thanks in advance for your support.
Note: The dropdown was a good choice for me since it can easily be extended to other schemas, like Recipe for cooking blogs.

[x] There's a clear use-case for this code change, explained below
[x] Commit message has a short title & references relevant issues
[ ] The build will pass (run yarn test:all and yarn lint)

🤖 Generated by Copilot at c9e50a7
This pull request adds a new feature to Ghost that allows users to select a schema type for their posts, either 'article' or 'newsArticle'. The schema type is a metadata attribute that affects how the posts are displayed and indexed by search engines and social media platforms. The pull request modifies the post model, validator, service, exporter, and schema helper to support the new attribute, and adds a new component and a migration to handle the user interface and the database changes. The pull request also updates the tests and the mock server to reflect the new feature.

Any updates for this PR?


smoke-test schema updates & rollbacks

Seems, we should have way to ensure that sql files are typo-free:
11:30 <hellais> Ok I pushed to the branch
11:33 <hellais> sorry actually I made a mistake in the schema
11:34 <darkk> @hellais do you think it makes sense to write a smoke-test for schema installs & rollbacks?
11:35 <darkk> to ensure that they are typo-free
11:35 <hellais> @darkk I think that would be useful
11:35 <hellais> ideally it would be run as part of CI

That's done by 95dc49e9aaaec3673b3a27ca3cc86df552a52101 since #144, so I'm closing the ticket.


ValueError: could not convert string to float

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce

Try to upload .mpv video file which has decimal comma in the metadata
import fails with the error:

Could not create the task
Error: Could not create the task on the server. ValueError: could not convert string to float: '44,877000000'.
Expected Behavior
Import should succeed
Possible Solution
sed -i 's|float(_sec)|float(_sec.replace(",","."))|g' cvat/apps/engine/media_extractors.py
for example: https://github.com/opencv/cvat/pull/6923
Context
Happens with mkv video exported from Networkoptix NX Witness VMS software. Works correctly for avi format.
Environment
v2.4.4 tag in the container run via docker-compose

I think this is a pretty straightforward fix. Should I go ahead and write a solution ?
I plan to change cvat/apps/engine/media_extractors.py so that the commas are replaced by dots

@ChanBong , please try to reproduce the issue first. It looks like the number of seconds can be encoded differently (depends on locale?). Need to understand what DURATION should have and how to decode it properly. I don't think it is right to replace ',' by a '.'. I'm not sure it will work always. Need to reproduce and understand the reason behind the problem.

You are right. This depends on the local encoding, mostly on your local computer language. I wasn't able to get/make such a video with commas in the duration. Maybe if @gronki can provide such video


Remove EOL distro versions

The following distro versions are now unmaintained:

Fedora 26
Fedora 27
Ubuntu 17.10

We need to:

Remove them from obs-packaging/distros.
Communicate with users on the ML that they need to upgrade to newer distro versions.

Related:

https://github.com/kata-containers/packaging/pull/267/files#r239923916
https://github.com/kata-containers/ci/issues/41

@jodh-intel can you reopen this, we still need to:

[ ] Communicate on the ML that these distros are no more supported.
[ ] Update the install documentation.
[ ] Remove EOL distros builds on OBS (they are still there for master).

@marcov - all yours ;)


Add Web Bluetooth Availability Sample

This PR is adding two samples for Web Bluetooth getAvailability()
Does that look good @odejesush?
Live preview:
https://beaufortfrancois.github.io/samples/web-bluetooth/availability.html
https://beaufortfrancois.github.io/samples/web-bluetooth/availability-async-await.html
https://beaufortfrancois.github.io/samples/web-bluetooth/index.html
FYI @jyasskin @g-ortuno @scheib

Yup LGTM, thanks for adding these.
I still need to implement the availabilitychanged event though.

I'll wait for availabilitychanged to be implemented before merging those.
Then, I'll add these samples to https://www.chromestatus.com/feature/6229592081694720

@odejesush I've updated those samples to be in M78 and added checks for "onavailabilitychanged" so that it works in Chrome 79.

I've added those samples to https://www.chromestatus.com/feature/6229592081694720.
They are live:

https://googlechrome.github.io/samples/web-bluetooth/availability-async-await.html
https://googlechrome.github.io/samples/web-bluetooth/availability.html


Fix mouse region double render

Description
This PR fixes an issue where MouseRegion will render its children twice.
Related Issues

Fixes https://github.com/flutter/flutter/issues/32529

Tests
I added the following tests:

MouseRegion paints child once and only once

When MouseRegion is inactive
When MouseRegion is active

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

Current state: I'm waiting for https://github.com/flutter/flutter/commit/5bb8d8fcde0f3b7db0ce198e175bbc7b24fe805c to be rolled into google3 before merging dependent PRs such as this in case of a rollback.

@dkwingsmt Looks like the commit 5bb8d8f has been rolled into google3. Should we submit this PR?

Yahoo!


Add Compression

Not sure if we need this but I noticed that in eth-analysis-rs we add a compression layer.
Should we do the same here ? (For the current data size I'd guess it is not needed but would love to hear @alextes thoughts on this).

When usage/cost picks up or better performance is demanded we'll want compression. Until then it doesn't matter.


net/frr: Migration needed for 1.21?

Important notices
Our forum is located at https://forum.opnsense.org , please consider joining discussions there in stead of using GitHub for these matters.
Before you ask a new question, we ask you kindly to acknowledge the following:
[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues and I'm convinced that mine is new.
I'm wondering whether we will need a migration after merging https://github.com/opnsense/plugins/pull/2143 to remove the old ruby script from existing installations?
Similarly, will a migration also need to take care of uninstalling ruby, or are packages without dependencies automatically removed during updates?
I currently can't test it, but i'm sure one of the more knowledgeable Devs will be able to tell :-) If a migration is needed, feel free to assign the ticket to me. I'll probably find time over the weekend to write a small script.

pkg should take care of removal, since there are no model changes we should be safe there as well.

pkg should take care of removal, since there are no model changes we should be safe there as well.

tested an upgrade today. pkg cleans up everything. looks good, closing.


RGB Colours are not retained while toggling using Home Assistant

If I toggle the light off and on again using Home Assistant and using the simple light config provided, the state defaults to a RGB value of [255,147,41]. Is this normal behaviour for it to not save the last used RGB values?

It’s is indeed. Using the example sends the command of “1” when the light is turned on, the code uses the default colours for the All On mode which is the RGB value you see (a warm light).
I had a quick look at potentially changing the payload_on value but I think this will break the state change detection.
If you’re after different behaviour a couple of options come to mind:

Change the hard coded default light colour in the code (lines 130 to 132).
Add automations to HA to set the colour you want when the light turns on.

I’ve also seen that you can create temporary scenes on the fly now which may be of use here if your looking to restore the last state. I haven’t played with this feature myself but worth a look.
Hope this helps.

Thanks for the quick response! I might have a look at the code when I find the time, in the meantime I will use your suggestion of using an automation. Thanks for the tips!


Make channel name comparisons case-insensitive

According to https://twitter.com/discord/status/753961990016659456, "text-channels have to have lower case characters only."

I can't tell from just looking since it's been a while. Will this let the bot create two channels of different casing? If it does, I recommend we add a check to make sure it doesn't, that way it doesn't get into a state where joining one intended room actually dumps them into another.

@jamesreiati Yeah, either the SDK or the Discord backend forces everything to be lowercase, even if someone tries to create it with uppercase letters. Even if that weren't the case, we're checking whether the channel name already exists using a case-insensitive comparison before creating a new channel: https://github.com/jamesreiati/chill-bot/blob/b865f0c69e1fe767ded2d5b3e3e4db9a2f9f2479/Behavior/OptinChannel.cs#L64


Case when an R function returns an instance of an R6 class

Hi,
I realize this library is a prototype, and not that actively maintained (per #5). However I'm really glad it's here as more and more R libraries use R6 classes and there is occasionally (frequently in my case) a need to port these to Python.
I have an example of a Model class (implemented as an R6 class) which has a method that returns a ModelFit class (separate R6 class).  Each class works really well on its own. The problem comes when I try to use the ModelFit returned by a method implemented in the Model class (generated in R), rather than one I created in Python.
To be more concrete, let's say I run the following:
# pseudocode
data = pd.DataFrame()
mod = TSModel.new()
fit = mod.fit(data)

At this point, the Model object has a class rpy2_r6.r6b.TSModel whereas the ModelFit object has a class rpy2.robjects.environments.Environment:
In [73]: mod
Out[73]: <class 'rpy2_r6.r6b.TSModel'> at 0x142a3ed00

In [74]: fit
Out[74]:
<rpy2.robjects.environments.Environment object at 0x143dc8100> [RTYPES.ENVSXP]
R classes: ('TSModelFit', 'R6')
n items: 20

Is there a way to turn this fit object into something of the rpy2_r6.r6b.TSModelFit class? Or, to do it analogously using the r6a formulation?

It is hard to be certain without more details about the implementation of your Python class TSModel. However, r6a and r6b are 2 different approaches to map R6 to Python. r6b is trying to dynamically create Python classes to match the R6 classes. What you worked out should be fine. The meta-programming in r6b might be trying be too clever for its own good.
If you followed this to get TSModel, the method fit was created dynamically by looking at the R6 class definition. However, there is currently no dynamic inference of the return type mapped to an rpy2_r6.r6b class (and I am not sure that it is possible to achieve this).
mod.fit() returns an Environment because R6 instances are R environment objects and r6b does not connect those dots yet.
If you want to stay with r6b all the way, the following might work:
# Assumes that the R6 class definition for `TSModelFit` is in `your_r_package`.
TSModelFit = r6b.R6DynamicClassGenerator(your_r_package.TSModelFit)

TSModelFit.__R6CLASS__(mod.fit)


Fix logging

TL;DR
Minor bug with logging.
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[x] Unit tests added
[x] Code documentation added
[x] Any pending items have an associated Issue

Complete description
NA
Tracking Issue
NA
Follow-up issue
NA

👍

Codecov Report

Merging #119 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #119   +/-   ##
=======================================
  Coverage   81.83%   81.83%
=======================================
  Files         211      211
  Lines       13661    13661
  Branches     1126     1126
=======================================
  Hits        11179    11179
  Misses       2215     2215
  Partials      267      267

Impacted Files
Coverage Δ

flytekit/__init__.py
100.00% <100.00%> (ø)

flytekit/clients/raw.py
51.96% <100.00%> (ø)

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 e65b1bb...e0b5f39. Read the comment docs.


Blank page at first launch

Describe the bug
Blank page at first launch of ocis 3.0.0
With failed request https://localhost:9200/.well-known/openid-configuration:

Steps to reproduce
Steps to reproduce the behavior:

fresh new rpi 4 with rpsbian 64
Setup etcd
install ocis 3.0.0 via official doc

Expected behavior
Access launching page
Actual behavior
Blank page loading for ever + failed https://localhost:9200/.well-known/openid-configuration request
Setup
Please describe how you started the server and provide a list of relevant environment variables or configuration files.

export PROXY_TLS=false
export MICRO_REGISTRY=etcd

Command to launch ocis:
OCIS_INSECURE=true IDM_CREATE_DEMO_USERS=true PROXY_HTTP_ADDR=<IP_ADDRESS>:9200 OCIS_URL=https://localhost:9200 ocis server

Additional context
Exact same behavior on ocis 2.0.0. I think it may be due to HTTPS/TLS but I feel like I have deactivated them for now...

You've set: PROXY_TLS=false and OCIS_URL=https://localhost:9200
This can only work if you're using a TLS terminating reverse proxy in front of your ocis instance. If you want to use ocis without a reverse proxy you need PROXY_TLS=true (the default). For details see also: https://doc.owncloud.com/ocis/next/deployment/services/s-list/proxy.html#special-settings

I am not sure to understand your answer.
The ocis server is running on a machine on my local network. I want to access it from a second machine.
Two questions:

How can I do it by disabling TLS?
How can I do it by enabling TLS?

So far, if I strictly followed your documentation, I lead to the exact same blank page I already sent (which is the error on distant machine accessing to ocis machine).

Two questions:
1. How can I do it by disabling TLS?

When using the built-in IDP you can't run completely without TLS. For the reasons described in the documentation linked above ("because the embedded libreConnect shipped with the IDP service has a hard check if the connection is on TLS and uses the HTTPS protocol. If this mismatches, an error will be logged and no connection from the client can be established").
Your only option, if you want to run ocis without TLS is to put a reverse proxy (like e.g. traefik) in front of ocis and let that proxy handle the TLS termination. But your clients will still need to use https to access that reverse proxy.

2. How can I do it by enabling TLS?

So far, if I strictly followed your documentation, I lead to the exact same blank page I already sent (which is the error on distant machine accessing to ocis machine).

I suspect you still had the OCIS_URL setting pointing to localhost? That will only work if you access ocis only from localhost. For accessing it remotely. You need to set OCIS_URL to the hostname and port via which ocis will be reachable from the external machine. (Additionally ocis must be able to reach itself via that hostname and port).
BTW, I'd suggest to take this to central.owncloud.org, which IMO is more suitable for discussing configuration issues. If this turns out to be a bug in the code or that documentation needs improvement we can still open new issues in github.

Thanks @rhafer , working great by modifying the OCIS_URL. I didn't understand from reading the docs, that this uri needs to be the reachable from outside.


Update to Braintree 5.5

Update composer to run on braintree 5.5
Update class references to new namespace and class names

you've got mergies

@christaggart this is now ready for review


Help us Saleem, you're our only hope

Help us @saleem-hadad , we need you!
There are a number of PRs which were merged after the most recent date shown in packagist. Would it be possible for you to please update this?
Also, while a different repo but very related, the docs for larecipe are stale.

hahah 😂that made my day
Sorry for being inactive lately, I just got married and a job in Dubai, so yeah a lot of changes.

Lol glad you liked it and no problem. Congrats 🎊🍾🎉 @saleem-hadad !!!

Thank you @WillGoldstein 🌹


Cannot change a specific string after localization

I ran into the following issue:
Assume, in a page we have many different Select2 instances, that need to be translated to the page's language. But we have a specific instance, that needs a specific string (eg. instead of "Searching..." we need "Fetching data...").
When we add eg. language: 'hu' in the options object, after that, we can't override it. When we override a specific string, we got the further strings on the default language, even if we have a "lang" attribute on the element or parent.
It would be great, if we get individual properties for the language "eg. locale" and editing strings one by one.
But it would be good either, when we add "lang" attribute and if we want to override a string, the further strings comes from the language file.

I have this same problem. I only need to override the 'noResults' string to add some HTML to it. It would be good if other strings followed the selected language.

This sounds like a real issue that shouldn't be closed just because it wasn't attended to quickly (though of course attending to it quickly would be better)


In sample progress=true prints more than a progress-bar

Thanks for this package. I tried the example in the README with  progress=true (on Julia 1.10 and AdvancedHMC v0.6.1):
using AdvancedHMC, ForwardDiff
using LogDensityProblems
using LinearAlgebra

# Define the target distribution using the `LogDensityProblem` interface
struct LogTargetDensity
    dim::Int
end
LogDensityProblems.logdensity(p::LogTargetDensity, θ) = -sum(abs2, θ) / 2  # standard multivariate normal
LogDensityProblems.dimension(p::LogTargetDensity) = p.dim
LogDensityProblems.capabilities(::Type{LogTargetDensity}) = LogDensityProblems.LogDensityOrder{0}()

# Choose parameter dimensionality and initial parameter value
D = 10; initial_θ = rand(D)
ℓπ = LogTargetDensity(D)

# Set the number of samples to draw and warmup iterations
n_samples, n_adapts = 2_000, 1_000

# Define a Hamiltonian system
metric = DiagEuclideanMetric(D)
hamiltonian = Hamiltonian(metric, ℓπ, ForwardDiff)

# Define a leapfrog solver, with the initial step size chosen heuristically
initial_ϵ = find_good_stepsize(hamiltonian, initial_θ)
integrator = Leapfrog(initial_ϵ)

# Define an HMC sampler with the following components
#   - multinomial sampling scheme,
#   - generalised No-U-Turn criteria, and
#   - windowed adaption for step-size and diagonal mass matrix
kernel = HMCKernel(Trajectory{MultinomialTS}(integrator, GeneralisedNoUTurn()))
adaptor = StanHMCAdaptor(MassMatrixAdaptor(metric), StepSizeAdaptor(0.8, integrator))

# Run the sampler to draw samples from the specified Gaussian, where
#   - `samples` will store the samples
#   - `stats` will store diagnostic statistics for each sample
samples, stats = sample(hamiltonian, kernel, initial_θ, n_samples, adaptor, n_adapts; progress=true, verbose=false)

This has a progress-bar, and in the addition the following information is printed:
  iterations:                                   1272
  ratio_divergent_transitions:                  0.0
  ratio_divergent_transitions_during_adaption:  0.0
  n_steps:                                      7
  is_accept:                                    true
  acceptance_rate:                              0.7875153541240331
  log_density:                                  -7.884029732795562
  hamiltonian_energy:                           12.447784679126514
  hamiltonian_energy_error:                     0.3198459860125489
  max_hamiltonian_energy_error:                 0.4934198534404306
  tree_depth:                                   3
  numerical_error:                              false
  step_size:                                    0.6715960136781923

Is this a feature, and is there anyway to only print the progressbar?

This is possible using the AbstractMCMC interface with a different callback keyword argument.
To do it propertly, one should just implement a callback similar to
https://github.com/TuringLang/AdvancedHMC.jl/blob/37f0995ea466ec5a9c9b09ecacc04c317ba77886/src/abstractmcmc.jl#L175-L194
https://github.com/TuringLang/AdvancedHMC.jl/blob/37f0995ea466ec5a9c9b09ecacc04c317ba77886/src/abstractmcmc.jl#L203
which only shows the progressbar without all the extra information.
Buuut because in the AbstractMCMC-interface impl we only use the HMCProgressCallback if no callback is manually specified, e.g.
https://github.com/TuringLang/AdvancedHMC.jl/blob/37f0995ea466ec5a9c9b09ecacc04c317ba77886/src/abstractmcmc.jl#L82-L85
we can just provide a callback that does nothing and set progress=true, which then hits the default progress meter in AbstractMCMC.sample, which is exactly what you want.
In short, try the following:
# HACK: a callback that does nothing to avoid hitting `HMCProgressCallback`
callback(rng, model, sampler, transition, state, i) = nothing

samples = AbstractMCMC.sample(
    model,
    sampler,
    n_adapts + n_samples;
    nadapts = n_adapts,
    initial_params = initial_θ,
    callback=callback,
    progress=true,  # will use default progress meter i `AbstractMCMC`
)


goldendict-ng: Update checkver regex & autoupdate url

[X] I have read the Contributing Guide.

/verify

/verify

@brian6932 Thank you so much for your pr! I really appreciate your contribution, as it made me aware that the upstream project has updated the download links and naming conventions. I have now adopted the new naming style in the bucket.


Where is the Context Menu in v2.0.0?

In the Meteor version there was a context menu, accessible via right click on studies:

The menu had a distinct export feature I need to use:

I can't find it in v2.0.0 - was it moved? Or disabled?

Please review the latest code in the master branch. This issue might have been resolved. If it persists, kindly reopen the issue with updated details.
Try viewer-dev.ohif.org instead of viewer.ohif.org
Our viewer.ohif.org is deployed from release branch while viewer-dev.ohif.org is our master  branch
Read more about branch explanations here https://docs.ohif.org/development/getting-started#developing


Implement a testing strategy

which gems should we use?

rspec -> minitest
cucumber?
capybara?

give priority to API
check if views are valid html5
use jenkins or travis http://pastebin.com/urGaVb9M

some links

http://everydayrails.com/2012/03/12/testing-series-rspec-setup.html
http://robots.thoughtbot.com/how-we-test-rails-applications
https://github.com/ZeusWPI/Gandalf

Original issue by @bmesuere on Thu Jun 14 2012 at 16:43.
Closed by @bmesuere on Sun Sep 28 2014 at 11:35.

Check if something from #119 can be reused
Original comment by @bmesuere on Tue Sep 09 2014 at 14:17.


Tracer

This change is

Reviewed 18 of 18 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Feature suggestion: Connecting instruments

I've been tasked with implementing a way to connect multiple simulated instruments with inputs and outputs. Would this be appreciated as part of PyVISA-sim? Otherwise I will create it as a separate layer.
If yes, do you have any suggestions on how to do this?

I think the first step is to define an API to get access to the internal state of the simulated instrument. This definitively belong within pyvisa-sim I would say. After how one connects multiple instruments together could be external but may require some sort of observer pattern on the internal state of the simulated instruments.


Add Data model Info to the Mlayer All Datasets API Response

Can one of the admins verify this patch?

ok to test


Deprecation of get_ntp_peers

Some time ago I noticed that Aristas are pretty different when it comes to NTP. It is possible to set NTP servers but cannot establish peerings with other devices.
I am going soon enough to propose the method that retrieves the NTP servers configured on the device: get_ntp_servers (base ref: https://github.com/napalm-automation/napalm-base/pull/36).
What would be the best thing to do with the current get_ntp_peers? Should return empty dict or raise NotImplementedError?

If it's not supported I'd rather return and empty dict. I did the same for lldp_neighbors and fortios, for example.

Fixed by #34


Crafting Error

Crafting independent of either materials or skills

Details
Crafting: In the menus anything that can't be crafted, either because you don't have the materials or the skill is lacking) is shown in a darker color (or with an X if you use STAR AI) .
Yesterday I hit, purely by chance, one of these and to my amazement it worked. I tried with all the other menus (weapon, suits, drugs, food etc.) and it worked everywhere. This makes the whole skill and material system nonsensical.
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
To test I reinstalled the game, left FSFE and mods out, e.g. put it to a basic condition, started a new game but the bug persists.

Sounds like it might be a bug with your UI mod. Make sure you can replicate it WITHOUT any other mods loaded.

I did, I made a clean install with only one mod (SKK Start a new game), no SFSE and no UI mods and the problem is still there.

Here is the culprit:
StarfieldCustom.ini
[Crafting]
bFreeItemCrafting=1
But I have no idea which mod installed it.


Request more info to be provided by MirrorS

With f35db90bdef8030979d1944c558dc859556d1138, popular ISOs and APPs can be downloaded in https://mirrorz.org without browsing detailed directories of MirrorSes. However, only a few MirroSes have provided their info. Hence more maintainance is requested.
Below are a few possible directions

info. Some MirrorS does not have the tool to provide the info needed.
cname. For example, USTC provided ArchLinux while TUNA MirrorSes provided Arch Linux
frontend. A modern and/or best-practive frontend is wanted.

site.logo is now used by frontend, hence one MirrorS may provide their logo in their meta.

site.logo is now used by frontend, hence one MirrorS may provide their logo in their meta.


fix: update compile withserver script

Update the compile:withserver script to create a soft
link in the out/sever folder for node_modules directory
of ansible-language-server to handle new package
depedency added in the language-server code while setting
up development environment

LGTM !

@ganeshrn Are we going to support development of this extension under Windows? Does ln work under WSL? Asking because we already have a Windows jobs for the language server, known to be broken.
If we do not support development we will need to setup jobs that only install the already built extension artifact and test it under Windows, a bit more complex pipeline but doable.
If we consider that as a supported use case, I would prefer not to deepen the breakage. I personally do not use Windows at all, so I am ok with each path.

If you are staying in wsl, everything works the way you would expect it to:

It works best if you aren't crossing the filesystem boundary into Windows. The file will exist it probably just won't be usable:

This is a familiar pattern to WSL users and the WSL Remote capability in VSCode also follows this pattern. Users of that remote will also understand it well.


Failed To Find pattern "ytplayer.config\s*=\s*(\{.*?\});"

I am using version 6.0.1

I have the same problem too.I used the latest version

I have the same problem too.


Can this model test with gray image?

there is a gray image in the set14, i want to test it. but i failed. so i want to know whether this model can test gray image.

You can just repeat the gray channel to RGB channels.
But the color of the results changes a little.
Some examples (Set14) : https://drive.google.com/drive/folders/1iaM-c6EgT1FNoJAOKmDrK7YhEhtlKcLx


[Feature] README Updates

Is there an existing feature request for this?

[X] I have searched the existing issues

Describe the Feature
This packages README is not currently inline with our existing standards. It would be ideal for this repo to see README updates to be in line with our current documentation guidelines.
Describe alternatives you've considered
No response
Are you interested in contributing this feature?

[X] Yes.
[ ] Yes, but I will need assistance and will schedule time during your office hours for guidance.
[ ] No.

Anything else?
During these updates, we should also add the identifier variables and ensure the integration_tests/dbt_project.yml references the identifiers as well.

PR #23 addresses this feature

The "What does this package do" still needs to be added in.

Updated in v0.6.0.