two-way binding on variable with setter causes initial call to setter

I have a pretty involved grid component that uses two-way binding to (among other things) control pagination and sort order data. For sort order, I have a number of grid-column-header elements which two-way bind to an order variable, and toggle it based on click of that header: works fantastically. What I am not noticing (and hadn't in previous versions) is that on the initial display, each of the individual column headers is causing a set on the order variable which is of course leading my code to update the entire table N times (N === number of columns). Is there any way to avoid this behavior?
relevant example: http://plnkr.co/edit/3tsFCI0xG30QHYU6rsPt
note that when you start the example you have 20 console.log statements for sharedValue set

Hmm, this might be more of an issue with the design on my part. It seems that even if I short circuit the initial set call back to the origin, every time I change it in each grid-column-header that also triggers N-col sets. So it seems the behavior is inherent to the binding type?

Changing sharedValue directly from the App class, will call sharedValue's set method twice.
I wouldn't expect it to be set twice either. The bindeable is clearly setting the value again, after it registered the change.
I don't believe it should behave that way, but I have been wrong before haha

@CasiOo yeah I'm hoping this is indeed a bug, it would save me some refactoring time :smile:

I've updated plunker with a button on the top to change the sharedValue directly from the view model, so you can see that setting it there also causes 20 additional sets to the variable

@jdanyow Can you look into this or comment?

The controller calls the BehaviorPropertyObserver.  Maybe this isn't necessary?  Then what happens is the behavior property's oldValue is always initially null here, causing the behavior property observer to call it's subscribers (eg the binding instance) resulting in sourceExpression.assign.
I wonder if we should remove the call in the controller and also set the BehaviorPropertyObserver's initial value according to what it's bound to.

@jdanyow the last sentence sounds like a decent proposal, would that potentially break other expected behaviors?

A bindable has a particular behavior as part of the binding cycle of a component:

If the bind callback has been implemented on the component, then the bindable's callback should not be called because the developer will recieve the notification via the general bind callback.
If the bind callback has not been implemented on the component, then the bindable's callback must be called when the initial value is set, otherwise the developer has no way to respond to the initial value of the property.

@jdanyow Are you suggesting to have a second implemention, similar to call but that doesn't notify the subscribers?

I'm not sure that would work...when would the subscribers be notified? Consider one component that binds to another component's property. That subscriber needs to be notified of the initial value.

@EisenbergEffect so you're saying the way to avoid the "initial bind" behavior is to implement bind on the VM of the bindable? That doesn't seem to be the case here, in fact in my personal project I have:
  bind(bindingContext, overrideContext) {
    this.$parent = bindingContext;
  }

so that I can use the VM as a parent for a sub-dialog

@EisenbergEffect makes sense, so I guess we have no choice but to evaluate the source expression and compare it to the new value.
This line:
this.updateSource(newValue);

Would become:
if (this.sourceExpression.evaluate(...) !== newValue) {
  this.updateSource(newValue);
}

sound good?

What do you think would be the impact of that? It's only going to affect two-way bindings correct? So, that shouldn't be a perf issue. Am I following correctly?

right- two-way bindings only, shouldn't be an issue.

ok, sounds good


Bugfix: use array_shift in error iterator instead of Laravel collection method

This brings in the fix tagged as v3.0.1.
See https://github.com/laravel-json-api/laravel/issues/287

Tagged as v4.1.1


Meaningful unit test coverage - ContentSection

Problem
As part of issue #59, all snapshots tests where removed. We would now like to replace these snapshots with a more meaningful unit test.
Acceptance Criteria
Using @testing-library/react create a test for ContentSection.js found in src/components/ContentSection/

@rellison192 Can you speak a little more about the acceptance criteria for this component? What kinds of unit tests are you thinking of?

feel free to add any details you think are missing @SheaBelsky  :)

I'm happy to work on this.

@rellison192 @SheaBelsky since nobody claimed this issue I went ahead and worked on it and I opened a PR.


[Refactor][Investigate]: MetaDB: Use protobuf for data encoding instead of json

Currently we use jsons, and marshal/unmarshal is very time consuming and if the data set is very large, zot performance decreases.

https://github.com/project-zot/zot/pull/1829
Some working and non-working pseudocode ...

First step is to remove unnecessary unmarshals.
Second step is PR #1829


Could you add another pokemon?

Hi~
I like your plugin because I'm big fan of pokemon.
I wonder if you add another pokemon.
I want Gyarados, Slowpoke, Koffing.!
I think 1st generation pokemon is more cute than other generation! :)
Thanks for making this awesome plugin.

Hello! +_+//
your plugin  So CUTE!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
this plugin make me happy ♥
Thanks~!

I should be able to do that :)
Thanks for the kind words!


Migrate packages.config -> PackageReference causes 'VTable setup of type SQLitePCL.SQLite3Provider_sqlite3 failed'

After migrating 'packages.config' -> 'PackageReference' using VS2019, Xamarin forms project compiles fine, but I receive an error during iOS/Simulator startup at App.cs/Initialize()
SQLitePCL.Batteries_V2.Init():
->
System.TypeLoadException: 'VTable setup of type SQLitePCL.SQLite3Provider_sqlite3 failed'
It happens only on iOS (I tried only in Simulator), the Android runs fine, no such error.
Before migrate:
packages.config:
<package id="SQLiteNetExtensions" version="2.0.0" targetFramework="xamarinios10" />
<package id="sqlite-net-pcl" version="1.4.118" targetFramework="xamarinios10" />
<package id="SQLitePCLRaw.bundle_green" version="1.1.5" targetFramework="xamarinios10" />
<package id="SQLitePCLRaw.bundle_sqlcipher" version="1.1.5" targetFramework="xamarinios10" />
<package id="SQLitePCLRaw.core" version="1.1.10" targetFramework="xamarinios10" />
<package id="SQLitePCLRaw.lib.sqlcipher.ios_unified.static" version="1.1.10" targetFramework="xamarinios10" />
<package id="SQLitePCLRaw.provider.internal.ios_unified" version="1.1.10" targetFramework="xamarinios10" />
<package id="SQLitePCLRaw.provider.sqlite3.ios_unified" version="1.1.5" targetFramework="xamarinios10" />

ios.csproj
<Reference Include="SQLite-net, Version=<IP_ADDRESS>, Culture=neutral, processorArchitecture=MSIL">      <HintPath>..\..\packages\sqlite-net-pcl.1.4.118\lib\netstandard1.1\SQLite-net.dll</HintPath>
<Reference Include="SQLiteNetExtensions, Version=<IP_ADDRESS>, Culture=neutral, processorArchitecture=MSIL">      <HintPath>..\..\packages\SQLiteNetExtensions.2.0.0\lib\portable-net45+netcore45+wpa81+wp8+MonoAndroid1+MonoTouch1\SQLiteNetExtensions.dll</HintPath>
<Reference Include="SQLitePCLRaw.batteries_v2, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=8226ea5df37bcae9, processorArchitecture=MSIL">      <HintPath>..\..\packages\SQLitePCLRaw.bundle_sqlcipher.1.1.5\lib\Xamarin.iOS10\SQLitePCLRaw.batteries_v2.dll</HintPath>
<Reference Include="SQLitePCLRaw.core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=1488e028ca7ab535, processorArchitecture=MSIL">      <HintPath>..\..\packages\SQLitePCLRaw.core.1.1.10\lib\Xamarin.iOS10\SQLitePCLRaw.core.dll</HintPath>
<Reference Include="SQLitePCLRaw.lib.sqlcipher">      <HintPath>..\..\packages\SQLitePCLRaw.lib.sqlcipher.ios_unified.static.1.1.10\lib\Xamarin.iOS10\SQLitePCLRaw.lib.sqlcipher.dll</HintPath>
<Reference Include="SQLitePCLRaw.provider.internal">      <HintPath>..\..\packages\SQLitePCLRaw.provider.internal.ios_unified.1.1.10\lib\Xamarin.iOS10\SQLitePCLRaw.provider.internal.dll</HintPath>

After migrate:
ios.csproj
<PackageReference Include="SQLiteNetExtensions">      <Version>2.0.0</Version>    </PackageReference>
<PackageReference Include="SQLitePCLRaw.bundle_sqlcipher">      <Version>1.1.5</Version>    </PackageReference>
<PackageReference Include="SQLitePCLRaw.lib.sqlcipher.ios_unified.static">      <Version>1.1.10</Version>    </PackageReference>
<PackageReference Include="SQLitePCLRaw.provider.internal.ios_unified">      <Version>1.1.10</Version>    </PackageReference>

The migrate dialog offered this by default, I accepted it:

Any idea what could cause this error?

There are multiple problems here, and I'm not sure which of them might be causing the symptom you are seeing.
(1)  If you explicitly include SQLitePCLRaw.bundle_sqlcipher, you do need to explicitly include SQLitePCLRaw.lib.sqlcipher.ios_unified.static or SQLitePCLRaw.provider.internal.ios_unified.  They will come in as transitive dependencies, and doing it that way will keep them matched to the version SQLitePCLRaw.bundle_sqlcipher.
(2)  If you want to use SQLitePCLRaw 1.x (which is no longer actively maintained) instead of 2.x (which contains breaking changes),then I suggest you use version 1.1.14.
(3)  You have bundle_sqlcipher as well as bundle_green, but you should never have more than one SQLitePCLRaw bundle package.  It is a surprising that this ever worked.  It looks like what you want is bundle_sqlcipher, and that bundle_green is coming in transitively through sqlite-net-pcl which is coming in transitively through SQLiteNetExtensions.  The usual advice is to use sqlite-net-base (1.6.x, in your case) instead of sqlite-net-pcl.  But since bundle_green is coming in transitively, you probably need to do something like this to get rid of it:
<PackageReference Include="SQLitePCLRaw.bundle_green" Version="1.1.14" ExcludeAssets="All" />

Thank you.
Unfortunately we suspended this project for now, so I cannot test your suggestions.
One thing is sure, it works, regardless how crazy way packages are mixed.
So it does not seem to be an issue, but a user error - so I will close this ticket.
Later I can report back if we do the upgrade.


RHIROS-597 | Executive Report API changes

Changes:

Route and handler class for executive report response
Util functions

Additional:

These API changes should be independent of db schema updates, that said these changes should be merged
only after #203

Sample Records:
-[ RECORD 1 ]----+--------------------------------------------------
id               | x
account_id       | x
inventory_id     | xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
cloud_provider   | aws
display_name     | xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
fqdn             | xxxxxxxxxxxxxxxxxxxxxxxxxxxxx
instance_type    | t2.micro
state            | Undersized
stale_timestamp  | 2022-05-06 12:07:15.952262+00
region           | us-east-1
operating_system | {"name": "RHEL", "major": 7, "minor": 7}
cpu_state        | {CPU_UNDERSIZED,CPU_UNDERSIZED_BY_PRESSURE}
io_state         | {IO_UNDERSIZED_BY_PRESSURE}
memory_state     | {MEMORY_UNDERSIZED,MEMORY_UNDERSIZED_BY_PRESSURE}

-[ RECORD 2 ]----+--------------------------------------------------
id               | x
account_id       | x
inventory_id     | xxxxxxxxxxxxxxxxxxxxxxxxx
cloud_provider   | aws
display_name     | xxxxxxxxxxxxxxxxxxxxx
fqdn             | xxxxxxxxxxxxxxxxxxxxxxxxxxxxx
instance_type    | xxxxxxxxxxxxxx
state            | Undersized
stale_timestamp  | 2022-05-06 12:08:03.98634+00
region           | us-east-1
operating_system | {"name": "RHEL", "major": 8, "minor": 4}
cpu_state        | {CPU_UNDERSIZED}
io_state         |
memory_state     | {MEMORY_UNDERSIZED}

-[ RECORD 3 ]----+--------------------------------------------------
id               | x
account_id       | x
inventory_id     | xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
cloud_provider   | aws
display_name     | xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

...skipping 1 line
instance_type    | xxxxxxxxxx
state            | Under pressure
stale_timestamp  | 2022-05-06 12:08:40.739415+00
region           | xxxxxxxxxxxxxxxxxxxxx
operating_system | {"name": "RHEL", "major": 7, "minor": 7}
cpu_state        | {CPU_OVERSIZED,CPU_UNDERSIZED_BY_PRESSURE}
io_state         | {IO_UNDERSIZED_BY_PRESSURE}
memory_state     | {MEMORY_OVERSIZED,MEMORY_UNDERSIZED_BY_PRESSURE}

-[ RECORD 4 ]----+--------------------------------------------------
id               | x
account_id       | x
inventory_id     | xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
cloud_provider   | aws
display_name     | xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
fqdn             | xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
instance_type    | xxxxxxxxxxxxxxxxxxx
state            | Optimized
stale_timestamp  | 2022-05-06 12:10:17.319451+00
region           | xxxxxxxxxxxxxxxxx
operating_system | {"name": "RHEL", "major": 7, "minor": 7}
cpu_state        |
io_state         |
memory_state     |

Sample Response:

{
    "systems_per_state": {
        "optimized": {
            "count": 1,
            "percentage": 25.0
        },
        "under_pressure": {
            "count": 1,
            "percentage": 25.0
        },
        "undersized": {
            "count": 2,
            "percentage": 50.0
        },
        "oversized": {
            "count": 0,
            "percentage": 0.0
        },
        "waiting_for_data": {
            "count": 0,
            "percentage": 0.0
        },
        "idling": {
            "count": 0,
            "percentage": 0.0
        }
    },
    "conditions": {
        "cpu": {
            "count": 5,
            "percentage": 41.67,
            "undersized": 2,
            "oversized": 1,
            "under_pressure": 2
        },
        "io": {
            "count": 2,
            "percentage": 16.67,
            "undersized": 0,
            "oversized": 0,
            "under_pressure": 2
        },
        "memory": {
            "count": 5,
            "percentage": 41.67,
            "undersized": 2,
            "oversized": 1,
            "under_pressure": 2
        }
    },
    "meta": {
        "total_count": 4,
        "non_optimized_count": 3,
        "conditions_count": 12
    }
}

Note: Tests via GH Actions will be failing as of now.

@kgaikwad @r14chandra @PreetiW  Please find the same response based on an example set of db records, in the description comment, thanks!

Closing this PR on behalf of https://github.com/RedHatInsights/ros-backend/pull/224

All unresolved comments will be track in a separate PR.


Improved circle packing algorithm

🐛 Bug Fix
For circle packing algorithm, when trying to find a optimum position, sometime it will get a position out of the x range. In this pr, an constrain is added to get void of this situation.
Before:

After:

Coverage increased (+0.02%) to 84.444% when pulling 03055e26a49ac8fc3fd186535ea3d7b0af523b51 on conglei:circle-packing-fix into 86929e9464f125288243318a6221fbb1f9c45023 on williaster:master.

discussed points offline, lgtm 👍


Explore removing SmallVec from Event enum

This PR explores removing the dependency on smallvec and replacing it with Vec from std.
smallvec is a third party dependency that leaks into the public API. If applications that depend on rustyline wish to create custom key bindings, they must also depend on smallvec to construct Event::KeySeq.
This PR replaces smallvec, which is a complicated data structure with lots of unsafe code and several known security advisories, with a vocabulary type from std.
This has the effect of reducing the size of the Event enum.
Because Event is a public enum, changing the type of the field in Event::KeySeq is a breaking change. I've shown this change in the sequence of commits in this branch and the addition of a test.
Motivation of this PR is reducing dependencies.
@gwenn if you'd prefer, I'd be willing to try an alternate PR that conditionally enables smallvec with a Cargo feature. I think it looks easy enough to support.

$ cargo tree
...
├── radix_trie v0.2.1
│   ├── endian-type v0.1.2
│   └── nibble_vec v0.1.0
│       └── smallvec v1.8.0
├── smallvec v1.8.0

😅 I am hoping to follow up with additional PRs that make the radix_trie dependency optional as well.

😅 I am hoping to follow up with additional PRs that make the radix_trie dependency optional as well.

I've followed up on this here: https://github.com/kkawakam/rustyline/pull/615


GH1669: fixed parsing of versions with prerelease

Issue
Closes #1669
Reason for PR
Fixed the parsing of version number which includes prereleases.
Features, i.e. what has been implemented?
Change the ´Version´-class used in the parsing of release notes the another class SemanticVersion-class written by devlead. This allows the ReleaseNotesParser to handle the version number which contains prereleases or similiar. Tests have been added to check that this works.

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


Fix credential type input item of subflow template

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes

If a typedInput-style environment variable input item of subflow template is defined with credential as default type, it changes to credential-style input item.
This PR try to fix the issue.
Steps to reproduce:

Create a subflow
Open subflow template properties edit panel.
Add an environment variable.
a. Set  "input type" of the variable to "input" with "credential" selected,
b. Change default value type to "credential".
Re-open subflow template properties edit panel.
"input type" of the variable changes to "credential".

Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.006%) to 68.228% when pulling 4bd71da056a7fbd3acbc233f0375ffe6a38643be on node-red-hitachi:fix-subflow-credential-input into 1f5588b803c7e2f216f2bfa31df7ac2cfa874480 on node-red:master.

Verified bug on v3.beta and 2.x branch (added backport2.x tag)
Tested fix - unfortunately while it does prevent the typedInput changing to a "credential only" type, it does not remember the default choice (reverts to "str" choice)...

@Steve-Mcl Thank you for checking.

Tested fix - unfortunately while it does prevent the typedInput changing to a "credential only" type, it does not remember the default choice (reverts to "str" choice)...

This is because typedInput types definition for default value field does not consider the UI definition.  I made an additional commit to fix it.


[#2584] Refactor AsyncTraceId

#2584

Codecov Report

Merging #3106 into master will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3106      +/-   ##
==========================================
+ Coverage   38.93%   38.95%   +0.02%
==========================================
  Files        2004     2004
  Lines       66442    66403      -39
  Branches     9296     9288       -8
==========================================
  Hits        25866    25866
+ Misses      37957    37921      -36
+ Partials     2619     2616       -3

Impacted Files
Coverage Δ

...oint/profiler/context/id/StatefulAsyncTraceId.java
0% <0%> (ø)
:arrow_up:

...oint/profiler/context/DefaultBaseTraceFactory.java
50.74% <0%> (ø)
:arrow_up:

...avercorp/pinpoint/profiler/context/AsyncTrace.java
0% <0%> (ø)
:arrow_up:

...t/profiler/context/DefaultAsyncContextFactory.java
36.36% <0%> (-10.7%)
:arrow_down:

...npoint/rpc/server/SimpleServerMessageListener.java
80% <0%> (-20%)
:arrow_down:

.../com/navercorp/pinpoint/rpc/packet/PongPacket.java
50% <0%> (-14.29%)
:arrow_down:

...pinpoint/profiler/instrument/AccessorAnalyzer.java
57.14% <0%> (-8.58%)
:arrow_down:

...om/navercorp/pinpoint/rpc/codec/PacketDecoder.java
68.67% <0%> (-7.23%)
:arrow_down:

...inpoint/rpc/server/DefaultPinpointServerState.java
88.13% <0%> (-1.7%)
:arrow_down:

...java/com/navercorp/pinpoint/rpc/DefaultFuture.java
70.83% <0%> (-1.67%)
:arrow_down:

... and 6 more

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 8230383...3d2d242. Read the comment docs.


Do not allow ng g module ng

Would avoid confusion later with imports

I signed it!

Although somewhat ill-advised to do so, I don't think we should prevent it. For that reason I don't think we should add this.


NetworkType.CONNECTED doesn't work when using proxy.

hello. I really like the work manager.
thank you for developing like this wonderful library🙇!
ISSUE

device : Galaxy S10 (Android 9.0)
workmanager : 2.2.0

When we develop apps, we often use CHALES to see packets.
but when we use constraintsNetworkType.CONNECTED, my device is not recognized as a network connected state.

when I turn off the Charles, it works well.

Thanks for reporting this.
I'm closing this issue because this forum is for issues with the codelab.
If you think that this is a problem of the WorkManager library, please open a bug on the WorkManager public issue tracker: https://issuetracker.google.com/issues/new?component=409906&template=1094197
Please reopen if you still think this is an issue with the codelab.


Enhance CacheHeadersFilter to allow for caching of .cache.* files

Cherry picked from master for (newly created) https://bugzilla.redhat.com/show_bug.cgi?id=1283304
This makes the compiled permutations (*.cache.js) cacheable :) (because they are). When refreshing [1] or revisiting the host page, we will now get a 304 (Not Modified) and these files will be served from cache. Note that every new compile generates the *.cache.js permutation files with different hashes. This is on purpose so they are downloaded once after every compile/redeployment and we'll never end up with a cached version when a new one should be loaded.
This should significantly improve the loading time of our workbench over slow network connections. Of course, the host page and the *.nocache.js files are still not cached and downloaded every time.
More details here [2], expiry date was set to one year in the future as recommended by [3].
[1] regular refresh NOT ctrl->shift->r or any other cache invalidating reload
[2] https://www.mnot.net/cache_docs/
[3] https://www.ietf.org/rfc/rfc2616.txt

@mswiderski @manstis as discussed, we were asked to cherry-pick the cache improvements to 0.7.x/6.3.x.

+1 to merge.

@krisv can you quickly confirm that we want this in the 6.2 release so that @porcelli can merge. ;)

@krisv ?

https://github.com/uberfire/uberfire-extensions/commit/21e205350b4af661bd9a278f2628e98e2d120f94


If SMILES output is invalid: Give the user a warning about this

Check this Structure: COC1=C(C=C(C=C1)/C=C\2/C(=O)N([C@]34C5=CC=CC=C5S[C@@]3(S2)C6=CC=CC=C6N4)OC7=C(C=C(C=C7)OC)O)OC
RdKit and CDK could depict it


H_FXT element's type is not always number,causing parsing error.

H_FXT will be -99 that indicates no data and will be a timestamp (format yyyy-MM-ddThh:mm:ss+08:00) if the data present.

example for the format is
2017-10-19T07:29:00+08:00


npm run dev fails

Windows 10, build 14342
λ npm -v
3.3.12

λ node -v
v5.4.0

λ npm run dev

><EMAIL_ADDRESS>dev c:\git\oss\spectacle-editor
> concurrently --kill-others "npm run hot-server" "npm run start-hot"

[1]
[1] ><EMAIL_ADDRESS>start-hot c:\git\oss\spectacle-editor
[1] > cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development
[1]
[0]
[0] ><EMAIL_ADDRESS>hot-server c:\git\oss\spectacle-editor
[0] > node -r babel-register server.js
[0]
[1] fs.js:584
[1]   return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
[1]                  ^
[1]
[1] Error: ENOENT: no such file or directory, open 'c:\git\oss\spectacle-editor\node_modules\electron-prebuilt\path.txt'
[1]     at Error (native)
[1]     at Object.fs.openSync (fs.js:584:18)
[1]     at Object.fs.readFileSync (fs.js:431:33)
[1]     at Object.<anonymous> (c:\git\oss\spectacle-editor\node_modules\electron-prebuilt\index.js:4:42)
[1]     at Module._compile (module.js:398:26)
[1]     at Object.Module._extensions..js (module.js:405:10)
[1]     at Module.load (module.js:344:32)
[1]     at Function.Module._load (module.js:301:12)
[1]     at Module.require (module.js:354:17)
[1]     at require (internal/module.js:12:17)
[1]
[1] npm
[1]  ERR! Windows_NT 10.0.14342
[1] npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "c:\\git\\oss\\spectacle-editor\\node_modules\\npm\\bin\\npm-cli.js" "run" "start-hot"
[1] npm ERR! node v5.4.0
[1] npm ERR! npm  v3.9.2
[1] npm ERR! code ELIFECYCLE
[1] npm ERR<EMAIL_ADDRESS>start-hot: `cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development`
[1] npm ERR! Exit status 1
[1] npm ERR!
[1] npm ERR! Failed at the<EMAIL_ADDRESS>start-hot script 'cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development'.
[1] npm ERR! Make sure you have the latest version of node.js and npm installed.
[1] npm ERR! If you do, this is most likely a problem with the spectacle-editor package,
[1] npm ERR! not with npm itself.
[1] npm ERR! Tell the author that this fails on your system:
[1] npm ERR!     cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development
[1] npm ERR! You can get information on how to open an issue for this project with:
[1] npm ERR!     npm bugs spectacle-editor
[1] npm ERR! Or if that isn't available, you can get their info via:
[1] npm ERR!     npm owner ls spectacle-editor
[1] npm ERR!
[1]  There is likely additional logging output above.
[1]
[1] npm ERR! Please include the following file with any support request:
[1] npm ERR!     c:\git\oss\spectacle-editor\npm-debug.log
[1] npm run start-hot exited with code 1
--> Sending SIGTERM to other processes..
[0] events.js:141
[0]       throw er; // Unhandled 'error' event
[0]       ^
[0]
[0] Error: listen EADDRINUSE <IP_ADDRESS>:3000
[0]     at Object.exports._errnoException (util.js:856:11)
[0]     at exports._exceptionWithHostPort (util.js:879:20)
[0]     at Server._listen2 (net.js:1237:14)
[0]     at listen (net.js:1273:10)
[0]     at net.js:1382:9
[0]     at GetAddrInfoReqWrap.asyncCallback [as callback] (dns.js:63:16)
[0]     at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:82:10)
[0]
[0] npm
[0]  ERR! Windows_NT 10.0.14342
[0] npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "c:\\git\\oss\\spectacle-editor\\node_modules\\npm\\bin\\npm-cli.js" "run" "hot-server"
[0] npm
[0]  ERR! node v5.4.0
[0] npm ERR! npm  v3.9.2
[0] npm ERR! code ELIFECYCLE
[0] npm ERR<EMAIL_ADDRESS>hot-server: `node -r babel-register server.js`
[0] npm ERR! Exit status 1
[0] npm ERR!
[0] npm ERR! Failed at the<EMAIL_ADDRESS>hot-server script 'node -r babel-register server.js'.
[0] npm ERR! Make sure you have the latest version of node.js and npm installed.
[0] npm ERR! If you do, this is most likely a problem with the spectacle-editor package,
[0] npm ERR! not with npm itself.
[0] npm ERR! Tell the author that this fails on your system:
[0] npm ERR!     node -r babel-register server.js
[0] npm ERR! You can get information on how to open an issue for this project with:
[0] npm ERR!     npm bugs spectacle-editor
[0] npm ERR! Or if that isn't available, you can get their info via:
[0] npm ERR!     npm owner ls spectacle-editor
[0] npm ERR! There is likely additional logging output above.
[0]
[0] npm ERR! Please include the following file with any support request:
[0] npm ERR!     c:\git\oss\spectacle-editor\npm-debug.log
[0] npm run hot-server exited with code null

npm ERR! Windows_NT 10.0.14342
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "dev"
npm ERR! node v5.4.0
npm ERR! npm  v3.3.12
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>dev: `concurrently --kill-others "npm run hot-server" "npm run start-hot"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script 'concurrently --kill-others "npm run hot-server" "npm run start-hot"'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the spectacle-editor package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     concurrently --kill-others "npm run hot-server" "npm run start-hot"
npm ERR! You can get their info via:
npm ERR!     npm owner ls spectacle-editor
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     c:\git\oss\spectacle-editor\npm-debug.log

it seems you have another process running at Port 3000 already
Op wo 25 mei 2016 05:31 schreef Luís Rudge<EMAIL_ADDRESS>
Windows 10, build 14342
λ npm -v
3.3.12
λ node -v
v5.4.0
λ npm run dev

<EMAIL_ADDRESS>dev c:\git\oss\spectacle-editor
concurrently --kill-others "npm run hot-server" "npm run start-hot"

[1]
[1] ><EMAIL_ADDRESS>start-hot c:\git\oss\spectacle-editor
[1] > cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development
[1]
[0]
[0] ><EMAIL_ADDRESS>hot-server c:\git\oss\spectacle-editor
[0] > node -r babel-register server.js
[0]
[1] fs.js:584
[1]   return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
[1]                  ^
[1]
[1] Error: ENOENT: no such file or directory, open 'c:\git\oss\spectacle-editor\node_modules\electron-prebuilt\path.txt'
[1]     at Error (native)
[1]     at Object.fs.openSync (fs.js:584:18)
[1]     at Object.fs.readFileSync (fs.js:431:33)
[1]     at Object. (c:\git\oss\spectacle-editor\node_modules\electron-prebuilt\index.js:4:42)
[1]     at Module._compile (module.js:398:26)
[1]     at Object.Module._extensions..js (module.js:405:10)
[1]     at Module.load (module.js:344:32)
[1]     at Function.Module._load (module.js:301:12)
[1]     at Module.require (module.js:354:17)
[1]     at require (internal/module.js:12:17)
[1]
[1] npm
[1]  ERR! Windows_NT 10.0.14342
[1] npm ERR! argv "C:\Program Files\nodejs\node.exe" "c:\git\oss\spectacle-editor\node_modules\npm\bin\npm-cli.js" "run" "start-hot"
[1] npm ERR! node v5.4.0
[1] npm ERR! npm  v3.9.2
[1] npm ERR! code ELIFECYCLE
[1] npm ERR<EMAIL_ADDRESS>start-hot: cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development
[1] npm ERR! Exit status 1
[1] npm ERR!
[1] npm ERR! Failed at the<EMAIL_ADDRESS>start-hot script 'cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development'.
[1] npm ERR! Make sure you have the latest version of node.js and npm installed.
[1] npm ERR! If you do, this is most likely a problem with the spectacle-editor package,
[1] npm ERR! not with npm itself.
[1] npm ERR! Tell the author that this fails on your system:
[1] npm ERR!     cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development
[1] npm ERR! You can get information on how to open an issue for this project with:
[1] npm ERR!     npm bugs spectacle-editor
[1] npm ERR! Or if that isn't available, you can get their info via:
[1] npm ERR!     npm owner ls spectacle-editor
[1] npm ERR!
[1]  There is likely additional logging output above.
[1]
[1] npm ERR! Please include the following file with any support request:
[1] npm ERR!     c:\git\oss\spectacle-editor\npm-debug.log
[1] npm run start-hot exited with code 1
--> Sending SIGTERM to other processes..
[0] events.js:141
[0]       throw er; // Unhandled 'error' event
[0]       ^
[0]
[0] Error: listen EADDRINUSE <IP_ADDRESS>:3000
[0]     at Object.exports._errnoException (util.js:856:11)
[0]     at exports._exceptionWithHostPort (util.js:879:20)
[0]     at Server._listen2 (net.js:1237:14)
[0]     at listen (net.js:1273:10)
[0]     at net.js:1382:9
[0]     at GetAddrInfoReqWrap.asyncCallback [as callback] (dns.js:63:16)
[0]     at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:82:10)
[0]
[0] npm
[0]  ERR! Windows_NT 10.0.14342
[0] npm ERR! argv "C:\Program Files\nodejs\node.exe" "c:\git\oss\spectacle-editor\node_modules\npm\bin\npm-cli.js" "run" "hot-server"
[0] npm
[0]  ERR! node v5.4.0
[0] npm ERR! npm  v3.9.2
[0] npm ERR! code ELIFECYCLE
[0] npm ERR<EMAIL_ADDRESS>hot-server: node -r babel-register server.js
[0] npm ERR! Exit status 1
[0] npm ERR!
[0] npm ERR! Failed at the<EMAIL_ADDRESS>hot-server script 'node -r babel-register server.js'.
[0] npm ERR! Make sure you have the latest version of node.js and npm installed.
[0] npm ERR! If you do, this is most likely a problem with the spectacle-editor package,
[0] npm ERR! not with npm itself.
[0] npm ERR! Tell the author that this fails on your system:
[0] npm ERR!     node -r babel-register server.js
[0] npm ERR! You can get information on how to open an issue for this project with:
[0] npm ERR!     npm bugs spectacle-editor
[0] npm ERR! Or if that isn't available, you can get their info via:
[0] npm ERR!     npm owner ls spectacle-editor
[0] npm ERR! There is likely additional logging output above.
[0]
[0] npm ERR! Please include the following file with any support request:
[0] npm ERR!     c:\git\oss\spectacle-editor\npm-debug.log
[0] npm run hot-server exited with code null
npm ERR! Windows_NT 10.0.14342
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "dev"
npm ERR! node v5.4.0
npm ERR! npm  v3.3.12
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>dev: concurrently --kill-others "npm run hot-server" "npm run start-hot"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script 'concurrently --kill-others "npm run hot-server" "npm run start-hot"'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the spectacle-editor package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     concurrently --kill-others "npm run hot-server" "npm run start-hot"
npm ERR! You can get their info via:
npm ERR!     npm owner ls spectacle-editor
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     c:\git\oss\spectacle-editor\npm-debug.log
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/FormidableLabs/spectacle-editor/issues/59

oh, that was dumb :D

I still can't run:
λ npm run dev

><EMAIL_ADDRESS>dev c:\git\oss\spectacle-editor
> concurrently --kill-others "npm run hot-server" "npm run start-hot"

[1]
[1] ><EMAIL_ADDRESS>start-hot c:\git\oss\spectacle-editor
[1] > cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development
[1]
[0]
[0] ><EMAIL_ADDRESS>hot-server c:\git\oss\spectacle-editor
[0] > node -r babel-register server.js
[0]
[1] fs.js:584
[1]   return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
[1]                  ^
[1]
[1] Error: ENOENT: no such file or directory, open 'c:\git\oss\spectacle-editor\node_modules\electron-prebuilt\path.txt'
[1]     at Error (native)
[1]     at Object.fs.openSync (fs.js:584:18)
[1]     at Object.fs.readFileSync (fs.js:431:33)
[1]     at Object.<anonymous> (c:\git\oss\spectacle-editor\node_modules\electron-prebuilt\index.js:4:42)
[1]     at Module._compile (module.js:398:26)
[1]     at Object.Module._extensions..js (module.js:405:10)
[1]     at Module.load (module.js:344:32)
[1]     at Function.Module._load (module.js:301:12)
[1]     at Module.require (module.js:354:17)
[1]     at require (internal/module.js:12:17)
[1]
[1] npm
[1]  ERR! Windows_NT 10.0.14342
[1] npm ERR!
[1] argv "C:\\Program Files\\nodejs\\node.exe" "c:\\git\\oss\\spectacle-editor\\node_modules\\npm\\bin\\npm-cli.js" "run" "start-hot"
[1] npm ERR! node v5.4.0
[1] npm ERR! npm  v3.9.2
[1] npm ERR! code ELIFECYCLE
[1] npm ERR<EMAIL_ADDRESS>start-hot: `cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development`
[1] npm ERR! Exit status 1
[1] npm ERR!
[1] npm ERR! Failed at the<EMAIL_ADDRESS>start-hot script 'cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development'.
[1] npm ERR! Make sure you have the latest version of node.js and npm installed.
[1] npm ERR! If you do, this is most likely a problem with the spectacle-editor package,
[1] npm ERR! not with npm itself.
[1] npm ERR! Tell the author that this fails on your system:
[1] npm ERR!     cross-env HOT=1 NODE_ENV=development electron -r babel-register ./main.development
[1] npm ERR! You can get information on how to open an issue for this project with:
[1] npm ERR!     npm bugs spectacle-editor
[1] npm ERR! Or if that isn't available, you can get their info via:
[1] npm ERR!     npm owner ls spectacle-editor
[1] npm ERR! There is likely additional logging output above.
[1]
[1] npm ERR! Please include the following file with any support request:
[1] npm ERR!     c:\git\oss\spectacle-editor\npm-debug.log
[1] npm run start-hot exited with code 1
--> Sending SIGTERM to other processes..
[0] Listening at http://localhost:3000
[0] webpack built e6db7fb0ad00f3086907 in 9518ms
[0] Hash: e6db7fb0ad00f3086907
[0] Version: webpack 1.13.1
[0] Time: 9518ms
[0]     Asset    Size  Chunks             Chunk Names
[0] bundle.js  8.4 MB       0  [emitted]  main

... a bunch of webpack stuff ...

[0] WARNING in ./~/unified/index.js
[0] Module not found: Error: Cannot resolve module 'node-extend' in c:\git\oss\spectacle-editor\node_modules\unified
[0]  @ ./~/unified/index.js 25:13-35
[0] webpack: bundle is now VALID.

@luisrudge That looks like this issue: https://github.com/electron-userland/electron-prebuilt/issues/76 Take a look at the solution there and report back. Hope that works!

Yeah, I had to cd node_modules/electron-prebuilt && node install.js. Should this be handled by spectacle-editor?

I'm unable to repro this on OSX, Trying to find a windows machine to check. From that issue, it sounded like a local environment issue and not a problem with spectacle-editor itself.

You can find windows vms here: https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/mac/
It's definitely an environment issue, not sure where to fix though

My guess is, since I had node.js for a while, that I may have been running a 32-bit version. I couldn't find the original .msi that I ran to install it. WIth the 64-bit msi, everything is honky-dory. I will close this issue. I can finally play with electron a bit.

Maybe that's the cause? That issue is a little light on possible causes :/

I'm running the 64-bit version
λ nvm arch
System Default: 64-bit.
Currently Configured: 64-bit.

λ node -v
v5.4.0

λ npm -v
3.3.12

So far I haven't been able to reproduce this. I'm on windows 10, node v5.4.0, npm v3.3.12, and use nvm-windows for installing node.
It seems like this might happen if the download of electron-prebuilt fails the first time.

np then. let's close it! Thanks for the help!


Add code to do multi band repeatability plots and color composites

I have code I hacked together to make figures and images.  The code needs to be cleaned up and added to the repo (yes, yes, should have been doing this as I went along).

@SimonKrughoff I think I unwittingly just called this out in #111 ! Can I suggest that you add your validation code to the repo first, and then clean it up? If you put it in a folder called "validation" that will make it easier for @jchiang87 to reorganize (as in #105). I guess we'll want to import * from desc.twinkles.validation or somesuch. Thanks!

All the code for the plots I made lives in the repository.  I agree that we'll want to make it all easier to use.  I haven't had time to reorganize the repo.  @jchiang87 feel free to take that on if you are eager to do it.

Great! Glad to have you home, safe.
On Fri, Feb 12, 2016 at 1:49 PM, SimonKrughoff<EMAIL_ADDRESS>wrote:

Closed #81
https://github.com/DarkEnergyScienceCollaboration/Twinkles/issues/81.
—
Reply to this email directly or view it on GitHub
https://github.com/DarkEnergyScienceCollaboration/Twinkles/issues/81#event-549110763
.


[Help] site_name ignored?

Hi!
I have succesfully installed two different instances of PowerDNS-Admin on one web server.
It's awesome! But now I cant' recognize one from the another one without lookin at URL...
I've changed "site_name" in configuration, but nothing happen.
I'm missing a step?
I'm doing right?
How can I modify site name to recognize them easily?
Thanks!

site_name isn't ignored per se.  It is used in page titles rather than allowing you to change the branding on the login page and base page.
You could either replace it with the name you'd like or have or pull site_name to be something you can easily change.

Thank you!
I haven't noticed page title because we usually have a lot of tabs opened... Our fault!
How can I modify login page and base page to be sure that they don't be overwrited when we update?

You can adjust the SITE_NAME value in Settings > Basic > site_name. It is used for the page title. If you wish to have a different name in the login page, feel free to edit the source code here :-)


Counter example tests

Coverage increased (+0.004%) to 32.847% when pulling 39872fdb77bd90fe961eb570b2d7ce5df793c8b2 on counter_example_tests into 3e0060b7cfabf6f7c29eb7ecc7bb29accf2ee70d on master.

Coverage increased (+0.004%) to 32.847% when pulling 39872fdb77bd90fe961eb570b2d7ce5df793c8b2 on counter_example_tests into 3e0060b7cfabf6f7c29eb7ecc7bb29accf2ee70d on master.


Streamer Media Overlays, SMIL files with interspersed par and seq elements

@HadrienGardeur
https://github.com/readium/r2-streamer-go/blob/master/parser/epub/smil.go#L7
type Body struct {
	TextRef string `xml:"textref,attr"`
	Seq     []Seq  `xml:"seq"`
	Par     []Par  `xml:"par"`
}

type Seq struct {
	TextRef string `xml:"textref,attr"`
	Par     []Par  `xml:"par"`
	Seq     []Seq  `xml:"seq"`
}

This breaks with SMIL files such as (the order of interspersed seq and par elements is important):
<smil>
    <body>
        <seq>
            <par>
           ....
            </par>
            <seq>
            ....
            </seq>
            <par>
           ....
            </par>
        </seq>
        <par>
        ....
        </par>
    </body>
</smil>

I addressed this in the NodeJS implementation by making sure body extends seq, and seq has a sorted children list, which items are either seq (recursive) or par:
https://github.com/edrlab/r2-streamer-js/blob/develop/src/parser/epub/smil-body.ts#L14
https://github.com/edrlab/r2-streamer-js/blob/develop/src/parser/epub/smil-seq.ts#L16
https://github.com/edrlab/r2-streamer-js/blob/develop/src/parser/epub/smil-par.ts#L18

Thanks, I understand better now, I thought that you were talking about the model that we use for the Web Publication Manifest.
cc @banux

Can we close this issue? cc @danielweck


Quotation marks

Please for the sanity of all of us allow all these quotation marks
-”
-"
-'
I spend lot of time checking why my definition of class is not working! The problem was I'm using different quotation marks...

JSON.parse only accept double quotes ". So only those character can be used for class field definition. I'll add that in the documentation

What is the difference between (”) and (") ?

In dataview you can use any of these. Why you just not add cleanup function to replace quotation marks to correct JSON format?


refactor: updated german translations

Description
Updated German translations. Key points:

Sort json object (equivalent to en.json)
Added all missing translations
Replaced Plex with {mediaServer} where applicable
Removed some deprecated key where newer apply

Screenshot (if UI-related)
To-Dos

[ ] Successful build yarn build
[ ] Translation keys yarn i18n:extract
[ ] Database migration (if required)

Issues Fixed or Closed

@all-contributors please add @Bretterteig for translation

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

v1.9.0
GitHub release

Your semantic-release bot :package::rocket:


fix Regression in RoyalMail

Discovered in Fedora CI for laminas/barcode
https://koschei.fedoraproject.org/package/php-laminas-barcode
There was 1 error:
1) LaminasTest\Barcode\Object\RoyalmailTest::testProvidedChecksum
Laminas\Barcode\Object\Exception\BarcodeValidationException: The input failed checksum validation
/builddir/build/BUILDROOT/php-laminas-barcode-2.9.0-2.fc35.noarch/usr/share/php/Laminas/Barcode/Object/AbstractObject.php:1313
/builddir/build/BUILDROOT/php-laminas-barcode-2.9.0-2.fc35.noarch/usr/share/php/Laminas/Barcode/Object/AbstractObject.php:1284
/builddir/build/BUILDROOT/php-laminas-barcode-2.9.0-2.fc35.noarch/usr/share/php/Laminas/Barcode/Object/AbstractObject.php:959
/builddir/build/BUILDROOT/php-laminas-barcode-2.9.0-2.fc35.noarch/usr/share/php/Laminas/Barcode/Object/AbstractObject.php:633
/builddir/build/BUILD/laminas-barcode-74034895076d2305ce6716f93dd8d821f3c1e4fc/test/Object/RoyalmailTest.php:151

Since https://github.com/laminas/laminas-validator/commit/641382417563a3995809788bf9c462f8e6963b75
-        if ($chkvalue == $checksum) {
+        if ($chkvalue === $checksum) {

As $checksum is a string (result of substr), $chkvalue   must also be a string.

Could you please add a test for this so we wont run into this again?
I'd prefer having the same case as the mentioned CI ran in.

I am not really convinced about the fact that an array is casted to string.

It is not an array, but an integer.

I really would love to see a failing test to see how to improve the code accordingly.

Added

I have absolutely no idea why psalm is crying (and honestly, I don't care), the new test is mostly a copy/paste of a previous one.

I've adjusted the failing test: the reason why psalm complains is that we use a baseline to "freeze" pre-existing errors, and prevent repeating our past mistakes.
The test you copied had one of those errors: I removed the usage of laminas-config in c6b1e42
Meanwhile, pushing this and releasing the fix, thanks for the patch!


⚠ Bofrost.de has degraded performance

In a3091de, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1484 ms

Resolved: Bofrost.de performance has improved in 4d01f80 after 39 minutes.


fix: un-awaited method

This un-awaited method will prohibit the uploading of large files.
Related issues: https://github.com/iterative/dvc/issues/10643

Can you please try fixing the formatting please? Try running pre-commit run --all.


malloc error when using mpi+metis

Dear all,
First of all, thank you to make this code available :+1:
I get an error with test Euler/UniformFlow when executing it with mpi and metis partitionner, see log
log.txt
This test works with openmp or mpi + sfc partionning.
I am using gcc version 7.5.0 on ubuntu 18.04. mpich 3.2.1 and metis 5.1.0 compiled with default options except IDXTYPEWIDTH=64 and REALTYPEWIDTH=64.
Horses3D compiled with default options except COMM=PARALLEL and MODE=DEBUG
Ask me if you need any other info
Thanks again !
François

Hi @francoisfraysse,
Sorry for the long time to reply. I completely forgot about this issue. When you wrote the comment we had less experience with compiling horses with different compilers and versions of the libraries, so I didn't know what to do to try and solve the problem.
What I have seen that works is use newer versions. For example, gcc 11 or gcc 12 work well.
About metis, I use 5.1.0, just as you. I am not so sure about the MPI.
If you still have any kind of interest about this, I'll try to solve any question faster than last time :D
Gonzalo


Add Resilio.SyncHome <IP_ADDRESS>0

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

Microsoft Reviewers: Open in CodeFlow

it appears to install but there is an installation error


38 deep transform locals hash

Enhancement
Description
Uses Hash#deep_transform_values to swap models in a hash for GIDs
Fixes #38
Checklist

[X] My code follows the style guidelines of this project
[X] Checks (StandardRB & Prettier-Standard) are passing

I haven't had a chance to load this up, but I'm really pleased to see that you ran with the idea of shimming deep_transform_values. What will be the minimum version of Rails, now? (I know I found it, but I've forgotten.)
Anyhow, FWIW the PR looks structurally sound and sane based on what I expected to find. You introduced appraisal which added a lot of boilerplate noise to this PR, but I kept scrolling and somehow survived. 😁

Thanks and sorry again! I will give this some extra attention next week


Not matching e-mail address even if still exists

Laravel Version: 8
Adldap2-Laravel Version: 6.1
PHP Version: 7.4
I have users table.
I'm trying to make their authentication with LDAP.
But when user login, It tries to recreate It's record if 'username' and 'objecthuid' columns are empty. But there's still a user with same e-mail address, so that singularity validation returns error "there's still a record with that email address."
I want to work with LDAP like;
"If email exist, then login and update. If not exist so create new record."

Hi @tcagkansokmen,
LdapRecord supports this feature out of the box. I’d recommend moving to it:
https://ldaprecord.com/docs/laravel/v2/auth/database/configuration/#sync-existing-records


Getting message :serenity.callToStageFor(...) has been deprecated. Please use actorCalled(name) and actorInTheSpotlight() functions from @serenity-js/core to access the actors instead.

Hi @jan-molak ,
We have started using SerenityJs 2.0 in our Projects. I am getting "serenity.callToStageFor(...) has been deprecated. Please use actorCalled(name) and actorInTheSpotlight() functions from @serenity-js/core to access the actors instead."
Our team has followed all the steps given in https://serenity-js.org/modules/core/class/src/stage/WithStage.ts~WithStage.html while stage implemetation .
-------------------setup.ts file-------------------------
import { serenity, WithStage } from '@serenity-js/core';
import { setWorldConstructor } from 'cucumber';
import { Actors } from './stage';

setWorldConstructor(function (this: WithStage, { parameters }) {
  this.stage = serenity.callToStageFor(new Actors());
});

-----------------stage.ts file---------------------------------------
export class Actors implements DressingRoom {

    prepare(actor: Actor): Actor {
        switch (actor.name) {
            case 'Actor1':
                return Actor.named(actor.name).whoCan(
                    BrowseTheWeb.using(protractor.browser));
                                  }
                    }
}

------------------------Stepdef file---------------
Given(/that (.*) is an authorized user of the scheduling application/, function (
  this: WithStage,
  actorName: string
) {
  return this.stage
    .theActorCalled(actorName)
    .attemptsTo(Be.onSomeApplication());
});

Is there something that I am doing wrong?
Let me know if you need some more information

Manish

Hi @Manishrn! I'm actually working on the Serenity/JS 1.x to 2.x migration guide (#377) as we speak.
Serenity/JS 2.x has introduced a number of features designed to make your life easier, for example, the (deprecated) WithStage interface is no longer needed and you could implement the code from your examples as follows:
// setup.ts

import { serenity, WithStage } from '@serenity-js/core';
import { setWorldConstructor } from 'cucumber';
import { Actors } from './stage';

// instead of:
// setWorldConstructor(function (this: WithStage, { parameters }) {
//  this.stage = serenity.callToStageFor(new Actors());
// });

// use:
Before(() => engage(new Actors());

In your actors file:
// stage.ts

// instead of:
export class Actors implements DressingRoom {

    prepare(actor: Actor): Actor {
        switch (actor.name) {
            case 'Actor1':
                return Actor.named(actor.name).whoCan(
                    BrowseTheWeb.using(protractor.browser),
                )
        }
}

//use:

export class Actors implements DressingRoom {

    prepare(actor: Actor): Actor {
        switch (actor.name) {
            case 'Actor1':
                return actor.whoCan(
                    BrowseTheWeb.using(protractor.browser));
                                  }
                    }
}

And in the step refs file:

// instead of:
Given(/that (.*) is an authorized user of the scheduling application/, function (
  this: WithStage,
  actorName: string
) {
  return this.stage
    .theActorCalled(actorName)
    .attemptsTo(Be.onSomeApplication());
});

// use:
Given(/that (.*) is an authorized user of the scheduling application/, (actorName: string) =>
    actorCalled(actorName).attemptsTo(
        Be.onSomeApplication()
    ));

When(/.../, () =>
    actorInTheSpotlight().attemptsTo(
        // ...
    ));

Does this help?

Hi @jan-molak
The issue that I had mentioned is fixed by the implementation you suggested.
Thank you for your help 👍

You're welcome, thanks for letting me know 👍 I've expanding the soon-to-be-published migration guide to help other engineers who might have encounter the issues you came across.


WebAssembly module for protobuf encoding / decoding

Are there any plans on wasm implementation of protobufs?

This basically works with C++ protos and communicate via sending typed arrays back and forth.
We should document how to do this.

I am also interested to know which implementation is faster. protobuf.js, this repo, or the WASM build.


Enhance the MII sample pod wait script for 4.0

Changes:

Rely on a domain reaching its 'Complete' condition instead of requiring user to pass in the expected pod count
Check that the domain generation and observedGeneration match
Check that the domain api version is at least v9
Fail fast by default if a 'Failed' condition is detected.

NOTE: This is converted to 'draft' as there are issues in the new 4.0 operator conditions that in turn make this change unstable (as this change depends on conditions). I have reproduced using the latest Operator version and have filed a corresponding internal JIRA task.

NOTE: This is converted to 'draft' as there seem to be issues in the new 4.0 operator conditions that may be making this change unstable. I am trying to reproduce using the latest Operator version and will file Operator tasks if needed.

Minor side comments about potential further minor improvements (lower priority - may be added as time permits):

Simplify: Replace use of temporary files by populating internal results into an env var instead.
Clarify: Update usage to clearly mention that the script checks generation vs. observedGeneration.
Simplify: Create log helper function for replacing 'echo timestamp ...' style lines.
Fix: Usage should get defaults from default constants - it is currently using vars that might change when the command line is processed.

Closing - moved to different branch/pull OWLS-88344-new


Wireframe out a Parks & Rec Scholarship Application

In general we want to take PDF applications and wireframe them out into question-by-question mobile app. Example of this can be found here. Visual assets for wireframes can be downloaded here.
We recommend starting with the Parks & Rec Scholarship Application PDF.

Take PDF applications and wireframe them out into question-by-question mobile app. Example of this can be found here. Visual assets for wireframes can be downloaded here.

Someone had a question about this in Slack; here's my response....

The plan is for all forms to lead off with a few basic questions like the person's name, phone number, and email. (I think that's it but @HansEngebretsen can confirm.) Specific forms will have to implement all the other questions like your address and such. But once the user fills out one form, they'll be able to skip all those questions in future forms, since their data will be auto-filled.

I think I am tackling this one, and welcome anyone interested to be part of this.

@baipai happy to work on this with you. Let me know how I can help.


Remove backspace character

Xcode 7 keeps crashing when we are editing code that use MTSMotisObject, because there is invalid backspace character in comment block.
Application Specific Information:
ProductBuildVersion: 7B1005
ASSERTION FAILURE in /Library/Caches/com.apple.xbs/Sources/IDEPlugins/IDEPlugins-9057/IDEQuickHelp/Controllers/IDEQuickHelpController.m:360
Details:  Error creating XML document from clang-parsed comment block: Error Domain=NSXMLParserErrorDomain Code=9 "Line 2: PCDATA invalid Char value 8
" UserInfo={NSLocalizedDescription=Line 2: PCDATA invalid Char value 8
}
Object:   <IDEQuickHelpInspectorController: 0x7fe7187a7e50>
Method:   -generateHTMLForSymbol:fromQueryDictionary:inExpressionSource:context:
Thread:   <NSThread: 0x7fe712d19bc0>{number = 1, name = main}
Hints:   None
Backtrace:
  0  0x0000000106780d56 -[IDEAssertionHandler handleFailureInMethod:object:fileName:lineNumber:assertionSignature:messageFormat:arguments:] (in IDEKit)

Thanks for the fixing!

Changes available in CocoaPods under version 1.4.2

Thank you :)

You're welcome :smile:


Rework to be able to build against system fcl and master openrave

When FCL is installed from source (e.g. from the upstream https://github.com/flexible-collision-library/fcl), it does not install a cmake file, but does install a pkg-config file.  This PR tries both methods for finding FCL.
Also, some internals of the OpenRAVE::CollisionReport structure have changed recently.  This PR contains configuration-time check to determine which version of OpenRAVE you're building against.  It also uses the built-in CollisionReport::Reset method to initialize the report instead of manually resetting each member.

Note that this PR replaces #6.

Did you verify that this works in our fork of OpenRAVE?

Yes, builds on ava!


Fix anchor tag text to Azure Just-in-Time page.

@scovetta : Thanks for your contribution! The author, @TomShinder, has been notified to review your proposed change.

Verified that the correction matches the link article title; merging the PR.

@scovetta Thanks!


#19148 add configuration parameters to PG role DDL

Please check the parameters with and without IN DATABASE part
Please check other PG forks (few, on your choice)
https://www.postgresql.org/docs/current/sql-alterrole.html

verified


Example: AMM / Liquidity Pool

@jonjove it seems you are essentially do this as part of your work?

Yes, I'm already working on it.

Blocked by #179


TypeError: conv2d(): argument 'input' (position 1) must be Tensor, not tuple

any answer appreciated!

Harry-KIT could you be more explicit? conv2d() takes as arguments (c, output_feature, strides, padding). Could you print the shape of your input?


Update Stylelint to the latest version

Trying to update Stylelint to version 15.10.1, and beyond, broke linting. Looking at the changes the issue appears to be that the bin/stylelint.js file was replaced with bin/stylelint.mjs instead, which our gulp lint runner wasn't able to automatically find; see https://github.com/stylelint/stylelint/compare/15.10.0...15.10.1

/botio lint

Looks good!


Stopping use deprecates.

The Data SDK for Typescript will have removed in the next release the
deprecated constructor for the VersionedLayerClient and also
methods getVersion() and withVersion() for the DataRequest.
This CR adapts the code according to future changes.
Signed-off-by: Oleksii Zubko<EMAIL_ADDRESS>

@harpgl-bot Retest this please

@harpgl-bot Retest please


Chp_14 potentially incorrect covariance functions

I could be wrong but I believe in Chp_14.ipynb cell 72, the covariance function
SIGMA = etasq * pm.gp.cov.Exponential(input_dim=1, ls=rhosq)
is not correct since it only takes the first column of Dmat_ord I think a correct covariance function would be something like below:
    def __init__(self, etasq, rhosq):
        self.etasq = etasq
        self.rhosq = rhosq

    def __call__(self, X):
        return self.etasq * tt.exp(-self.rhosq * X) + tt.diag(np.ones(len(X)) * .01)

a similar problem is also present in cell 55.

Even if taking it from another angle, I have been fighting with this covariance function for a few days now, and indeed what you proposed here solved my problem (as far as I understand).
I have been working on the exercises and ex. 14M5 asks to reimplement the same model. If I redo the implementation proposed in cell 72 of Chp_14.ipynb it works fine, but then I tried to implement the model without writing a custom class for the mean, but simply using the same form as for the previous models and just change the covariance matrix. Here is what I came up with:
m_14m5b = pm.Model()

with m_14m5b:

    a = pm.Normal('a', mu=0, sigma=1)
    bB = pm.Normal('bB', mu=0, sigma=0.5)
    bM = pm.Normal('bM', mu=0, sigma=0.5)

    eta_sq = pm.Exponential('eta_sq', lam=1)
    rho_sq = pm.Normal('rho_sq', mu=3, sigma=0.25)

    mu = a + bB * B + bM * M
    SIGMA = eta_sq * pm.gp.cov.Exponential(input_dim=1, ls=rho_sq)(Dmat_ord)

    G = pm.MvNormal('G', mu=mu, cov=SIGMA, observed=G_)

    m_14m5b_trace = pm.sample(return_inferencedata=True)

With this model I kept getting sampling errors:
SamplingError: Initial evaluation of model at starting point failed!
Starting values:
{'a': array(0.), 'bB': array(0.), 'bM': array(0.), 'eta_sq_log__': array(-0.36651292), 'rho_sq': array(3.)}

Initial evaluation results:
a              -0.92
bB             -0.23
bM             -0.23
eta_sq_log__   -1.06
rho_sq          0.47
G               -inf
Name: Log-probability of test_point, dtype: float64

and I indeed suspected that it was due to the lack of a noise term in the covariance since I got the same error with the original model when I set noise=0 in gp.marginal_likelihood.
I then appended tt.diag(np.ones(len(X)) * .01) to the definition of SIGMA in the model above and it worked!

Hi, I also got stuck on this. It seems like the notebook changed at some point in the last two years because the current code looks quite a bit different. But the bug seems to have stuck around. It looks like the simplest fix, in terms of keeping as much of the pymc code as possible is gp.cov.ExpQuad(input_dim = 10) instead of input_dim = 1.  I think this arises because McElreath uses a 10x10 matrix to represent distances whereas most pymc GP examples use a 1x100 column vector to represent distances. I tried the column vector version too and it looks like it would force us to rewrite a lot of plotting code. Making this change also improves how well our estimates match the reported estimates in the 2nd edition of statistical rethinking. I'll make a pull request to fix this.
with pm.Model() as m14_8:
    a = pm.Exponential("a", 1.0)
    b = pm.Exponential("b", 1.0)
    g = pm.Exponential("g", 1.0)

    etasq = pm.Exponential("etasq", 2.0)
    ls_inv = pm.HalfNormal("ls_inv", 2.0)
    rhosq = pm.Deterministic("rhosq", 0.5 * ls_inv**2)

    # Implementation with PyMC's GP module:
    cov = etasq * pm.gp.cov.ExpQuad(input_dim=10, ls_inv=ls_inv) # change input_dim=10
    gp = pm.gp.Latent(cov_func=cov)
    k = gp.prior("k", X=Dmatsq)

    lam = (a * P**b / g) * tt.exp(k[society])

    T = pm.Poisson("total_tools", lam, observed=tools)

    trace_14_8 = pm.sample(4000, tune=2000, target_accept=0.99, random_seed=RANDOM_SEED)


ali50m.AddCurrentPath version 1.3.4

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.2 schema?

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

Microsoft Reviewers: Open in CodeFlow

This app does not seem to work correctly yet.


[BUG] Auth policy won't be the last one when there is a custom policy added in the clientOptions

Describe the bug
The auth policy is expected to be the last policy in the pipeline array before the transport adapter.
See:
https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/storage/Azure.Storage.Common/src/Shared/StorageClientOptions.cs#L112
Hoever, the HttpPipelineBuilder will add any policy from the clientOptions per-retry list to be after the auth policy (which is at the end of the per-retry policies from the client SDK)
https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/core/Azure.Core/src/Pipeline/HttpPipelineBuilder.cs#L70
Expected behavior
What is the expected behavior?
Actual behavior (include Exception or Stack Trace)
What is the actual behavior?
To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

Environment:

Name and version of the Library package used: [e.g. Azure.Storage.Blobs 12.2.0]
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): [e.g. Azure AppService or Windows 10 .NET Framework 4.8]
IDE and version : [e.g. Visual Studio 16.3]

Is this issue blocking some scenario? I'm slightly reluctant to reorder policies without a strong need now that we have customers that might rely on existing order.

There have been no customer reports of this behavior causing issues in the two years since this was filed.  As Pavel mentioned, it is potentially risky to change the ordering of policies.   I'm going to close this out and we can reconsider a course of action if we see a customer report for real-world impact.


Missing ending parentheses in AddAuthorization example code

Fixes missing ending parentheses in AddAuthorization example code.

@ferrx,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@ferrx  thanks!


3090003 nodejs push_transaction.How to get 'packed_trx'?

How to pack transaction without cleos?
So the new API only accepts a packed_trx but there is no RPC call to pack a signed transaction.
How can it be done on system without cleos installed? Is there any implementation in pure NodeJS ?

See https://github.com/EOSIO/eosjs or https://github.com/EOSIO/eosjs2 for implementations.


Add "Trading APY" to our overall APY per vault. ($500 UI + $2000 API Bounty)

This is a feature that I've been meaning to add for some time, and I just saw that Alpha Homora has it implemented.
They display the trading APY of holding an LP and add it to the total as well. This is relevant because a lot of the time, you're comparing the profitability of farming with an LP as opposed to holding an asset like BNB. In this sense, the trading profits are part of the equation, and it would be good to be explicit about it.

Blocked by API task: https://github.com/beefyfinance/beefy-api/issues/111

Since pancake info data is outdated, we should have a warning, like Alpha has: "Trading fees APR shown has only synced to block number 5640759/6706829. The data shown may be delayed and not reflect the actual APR." Or we simply wait until pancake data is fixed to show their trading data, since it can be misleading.

Hi, Can you assign me this issue to me?

Hi, Can you assign me this issue to me?

Of course 😄

@sirbeefalot - Are you available for a short video call, and explain the ticket in detail?


[Question] Is there a way to limit the number of playlist/channel/API pages downloaded?

For channels with a very large number of videos (1000+) the number of pages pullexd gets huge and becomes a big slow spot during an update of a channel archive. It would be handy to have a way to stop fetching beyond the first few pages of videos. For example, Styxhexenhammer666 makes several videos a day on BitChute (5599 so far as of this post) and the program downloads 225 channel pages before archiving the latest 4-5 videos that are only on the first channel page anyway.
I didn't see any option in the docs that does this already, but there are a heap of options, so I may have missed it. Edit: I should also note that I archive several hundred channels on multiple video platforms and this is far from the only channel that has this issue.

$ yt-dlp https://www.bitchute.com/channel/Styxhexenham/
[download] Downloading playlist: Styxhexenham
[BitChuteChannel] Styxhexenham: Downloading channel page 1
[BitChuteChannel] Styxhexenham: Downloading channel page 2
[BitChuteChannel] Styxhexenham: Downloading channel page 3
[BitChuteChannel] Styxhexenham: Downloading channel page 4
[BitChuteChannel] Styxhexenham: Downloading channel page 5
[BitChuteChannel] Styxhexenham: Downloading channel page 6
[BitChuteChannel] Styxhexenham: Downloading channel page 7
[BitChuteChannel] Styxhexenham: Downloading channel page 8
[BitChuteChannel] Styxhexenham: Downloading channel page 9
[BitChuteChannel] Styxhexenham: Downloading channel page 10
[BitChuteChannel] Styxhexenham: Downloading channel page 11
[BitChuteChannel] Styxhexenham: Downloading channel page 12
[BitChuteChannel] Styxhexenham: Downloading channel page 13
[BitChuteChannel] Styxhexenham: Downloading channel page 14

ERROR: Interrupted by user
Terminate batch job (Y/N)?
^C
$ yt-dlp https://www.bitchute.com/channel/Styxhexenham/ --playlist-end 100
[download] Downloading playlist: Styxhexenham
[BitChuteChannel] Styxhexenham: Downloading channel page 1
[BitChuteChannel] Styxhexenham: Downloading channel page 2
[BitChuteChannel] Styxhexenham: Downloading channel page 3
[BitChuteChannel] Styxhexenham: Downloading channel page 4
[BitChuteChannel] playlist Styxhexenham: Downloading 100 videos
[download] Downloading video 1 of 100
[BitChute] 3OgrZvZOJxCs: Downloading webpage
[BitChute] 3OgrZvZOJxCs: Checking video URL

ERROR: Interrupted by user

Thanks!


Support different database systems

Hello!
We've come across this solution today and got really interested in using it. However, we are curious if it would be possible to allow using other database solutions instead of MySQL. For our case, we would be looking into using PostgreSQL.
Looks like the solution is using Knex.js, which seems to support other databases. However, I'm not that familiar with Knex.js to say how complex it would be to do that.
We are also interested in using a different message broker solution than Kafka, but that might be a topic for a different issue.
Apologies if this is a duplicate question/request. I've tried searching for similar questions but couldn't find anything.

hey @vs-gtadeu, this was raised in https://github.com/tot-ra/graphql-schema-registry/discussions/144 as well. If this gains traction from the community, I'm not against it, just needs sufficient support by the developers and/or automated testing.
I do use Postgre, but not with schema-registry :)


[BUG] Image logo not working if position is right

Be sure to read the FAQ before submitting a new issue.

General description of bug:

What happened:
Image doesn't show up on right. Instead distro ascii logo does. The image does however shows up if i use left or top position.

What should happen:
image should show up on the right.

Fastfetch version used:

fastfetch 2.19.1 (x86_64)

Did it work in an older version:
I have not used any fastfetch versions prior to this.

Where did you get the binary:
Github releases

Does this issue still occur in the latest dev build?
Yes

Often helpful information:
Screenshot:
Using with "position": "left":

Using with "position": "right":

The content of the configuration file you use (if any):
    "logo": {
        "source": "~/.config/fastfetch/logo.png",
        "type": "kitty-direct",
        "position": "right"
    },

In order to print logo at the right side of terminal, fastfetch must move the cursor to right side first. In addition, fastfetch must leave a space for the image, so we need to know the image width
You need to specify the image width and height for now. After the next version is released, you  need to specify the image width only.


[Feature] centralised texts

Hello,
it would be very nice if you could store building blocks cetraly and open and use them by several people at the same time.
Alternatively, maybe set up an auto-reimport when starting the application.
It is very good if you use the software in a company or department and everybody should use the identical texts.

I've been add plan for sharing/sync. That a rather large project and unfortunately I no not have as much time for Beeftext as a would like to at the moment.
But it will be coming to a future version.

Closing as a duplicate of #157.


Using HTTP handlers as closures

Closes #81.
Please,@MaxFedotov review this: I guess this could be related to your changes

@prometherion Thanks a lot for great work!
What I thought is maybe we can use a single hook for both,networking.k8s.io and extensions groups, as it seems like they had the same logic. We can use a switch on req.Resource.Group and find out in which object (either &networkingv1beta1.Ingress{} or &extensionsv1beta1.Ingress{} to decode it

I'll definitely take a look to this option and prepare an additional commit to achieve this, thanks!

@MaxFedotov as you suggested, I just implemented with 2f40a35 the single webhook.
If it looks good to you, @bsctl needs to update Capsule Charts too since the Validation Webhook manifest has changed

LGTM, thanks a lot 👍


Show map in filters [WIP]

Raising PR for Heroku review app.

orphaned as no trello card either close or follow up


Added LeNexus

https://polkadot.subscan.io/extrinsic/22678896-2
https://kusama.subscan.io/extrinsic/25061479-2


fixing tutorial errors master branch

as described in http://code.opencv.org/issues/4154

Weird. If I look at documentation
http://docs.opencv.org/trunk/doc/tutorials/imgproc/histograms/template_matching/template_matching.html#template-matching
Then you can see that the code at glance contains the error. But if i look at the file that should be loaded (samples/cpp/tutorial_code/Histograms_Matching/MatchTemplate_Demo.cpp) than the fix is already applied there. Someone has an idea how this is possible?

@StevenPuttemans, old sphinx documentation (last generated on 30 Dec 2014) contained copy of the source file in the rst file: like this.
The new doxygen documentation takes same source block from original .cpp file, thus fixing the source (earlier) has fixed the docs.
It is better to use snippet command to include small marked code blocks from same source file if possible.
Some details:

OpenCV documenting guide
Doxygen command reference

So I should change this to use the snippet command or was it just an explanation on your side?

No, additional changes are not necessary, your fix is good too. :smile:

:+1:

:+1:


Fix issue with passing case_file_name argument to launch_fluent() throwing TypeError

Fixes https://github.com/ansys/pyfluent/issues/2310 by replacing the call to root.read_case() with root.file.read() using keyword arguments so that they are passed in correctly.

I think this reveals another issue with the source code. The docstring for launch_fluent() states the following for the lightweight_mode parameter:
lightweight_mode : bool, optional
        Whether to run in lightweight mode. In lightweight mode, the lightweight settings are read into the
        current Fluent solver session. The mesh is read into a background Fluent solver session which will
        replace the current Fluent solver session once the mesh read is complete and the lightweight settings
        made by the user in the current Fluent solver session have been applied in the background Fluent
        solver session. This is all orchestrated by PyFluent and requires no special usage.
        This parameter is used only when ``case_file_name`` is provided. The default is ``False``.

However, the existing code in standalone_launcher.py carries this out by making the following call:
session.file.read_case(self.case_file_name, self.lightweight_mode)

Which passes the value of lightweight_mode to the lightweight_setup parameter of read_case(), which does not exist. If the intention is to pass the argument to lightweight_setup, this will not actually load the mesh in the background, conflicting with the documentation.
Does the code or the documentation for pyfluent.launch_fluent() need to be updated?

I think this reveals another issue with the source code. The docstring for launch_fluent() states the following for the lightweight_mode parameter:
lightweight_mode : bool, optional
        Whether to run in lightweight mode. In lightweight mode, the lightweight settings are read into the
        current Fluent solver session. The mesh is read into a background Fluent solver session which will
        replace the current Fluent solver session once the mesh read is complete and the lightweight settings
        made by the user in the current Fluent solver session have been applied in the background Fluent
        solver session. This is all orchestrated by PyFluent and requires no special usage.
        This parameter is used only when ``case_file_name`` is provided. The default is ``False``.

However, the existing code in standalone_launcher.py carries this out by making the following call:
session.file.read_case(self.case_file_name, self.lightweight_mode)

Which passes the value of self.lightweight_mode to the lightweight_setup parameter of read_case() (which does not exist). If the intention is to pass the argument to lightweight_setup, this will not actually load the mesh in the background, conflicting with the documentation.
Does the code or the documentation for pyfluent.launch_fluent() need to be updated?

I think this reveals another issue with the source code. The docstring for launch_fluent() states the following for the lightweight_mode parameter:
lightweight_mode : bool, optional
        Whether to run in lightweight mode. In lightweight mode, the lightweight settings are read into the
        current Fluent solver session. The mesh is read into a background Fluent solver session which will
        replace the current Fluent solver session once the mesh read is complete and the lightweight settings
        made by the user in the current Fluent solver session have been applied in the background Fluent
        solver session. This is all orchestrated by PyFluent and requires no special usage.
        This parameter is used only when ``case_file_name`` is provided. The default is ``False``.

However, the existing code in standalone_launcher.py carries this out by making the following call:
session.file.read_case(self.case_file_name, self.lightweight_mode)

Which passes the value of self.lightweight_mode to the lightweight_setup parameter of read_case() (which does not exist). If the intention is to pass the argument to lightweight_setup, this will not actually load the mesh in the background, conflicting with the documentation.
Does the code or the documentation for pyfluent.launch_fluent() need to be updated?

@ansjsia It seems that this must have got broken with the recent PR that added standalone_launcher.py etc. The documentation states the intent correctly but the recent code changes are diverging from that.

@raph_luc @seanpearsonuk The latest push restructures the code to match the samples you gave and adds unit tests for reading case/data files into Fluent at launch. However, I'm running into an issue with the root.read_case_lightweight() method where it throws an error about not being able to read the mesh (see https://github.com/ansys/pyfluent/issues/2313 ).
It doesn't seem to affect reading the case settings though, as the new unit tests I put in are still passing.

where it throws an error about not being able to read the mesh

@ansjsia (cc: @raph-luc)
Please could you try to test doing a lightweight read of the same case directly in Fluent. I believe that it is possible in the TUI - not in the GUI last I saw,

@seanpearsonuk I've been able to do a settings-only read using the following methods:

In the launcher GUI
After launch in the GUI
In the TUI by calling file/read-case-settings
In PyFluent by calling root.file.read(..., lightweight_setup=True)

Since root.read_case_lightweight() calls root.file.read() under the hood, the issue seems to be with creating a session to load the mesh in the background.


git actions: coverage

Coverage remained the same at 91.257% when pulling 03f381e19eafaf668ccf4ce101b2413b1116e692 on dissoupov into 10217a66e606a04c9139e2482134e96486522cdd on main.


Refactor row level security to support ampq fanout

When a row in the database is updated, I would like to send notifications to connected clients (e.g. web browsers). Of course, they should only receive information they are authorized to receive---Alice shouldn't get a push notification of Bob's new or changed assignment_grade.
Right now, this is hard to do because I can't easily answer the the following question: given these changed rows, who has read permission and should therefore be notified? I think I could answer this question and at the same time simplify my row level security by building views like permissions.assignment_grade and permissions.assignment_grade_submission. Then I would have queries like EXISTS(SELECT * FROM permissions.assignment_grade_submission where accessor_id=5 AND can_read=TRUE). I could have similar permissions for write.
If I had views like that, my RLS policies would be cleaner---I'd effectively be moving the policies into those views. Also, I could easily support some kind of fan out on change because I could do SELECT accessor_id FROM permissions.assignment_grade_submission JOIN updated_rows.... etc.
Then, RabbitMQ could have topics like user_id:5.meetings, or something. That is, each user gets their own topic and they only have permission to subscribe to their own topic.
With this in place, real-time updates to various clients would be easy.
I think the flow would look like this
First, set up an "AFTER STATEMENT" trigger for INSERT/UPDATE/DELETE. This trigger will get a table containing all the changed rows: a transition table. We want to do this because we don't want to call a trigger for each individual row. We need to do some joins for the permission/auth (to see who has read access) and there is no reason to do that query one time for each rows. We should just do it for all affected rows at once. See this example and this example.
Second, in that trigger, we aggregate the user_ids that should be able to see an update, based on the read permissions (RLS policy). Usually, we'll be doing something like affected_rows CROSS JOIN user..., etc.
Third, publish a single message for each row to an AMPQ exchange with a BCC header. This exchange should use the BCC field to fan out the message to the queues of all connected users. (Unclear how I should set this up. When a user joins, is a queue created? Do I have to create them beforehand?)
Fourth, messages gets pushed to user clients using webstomp or mqtt over websockets.
Fifth, clients update their local data using local logic.

I'm thinking now that this won't work. The main reason I think so is that some of the RLS policies are doing extensive joins or have time-dependent rules. E.g. an RLS policy like that on data.assignment_field_submission is doing joins onto data.assignment_submission in order to determine if a user can read it. Imagine that both these tables change, e.g. an assignment submission is deleted and that delete cascades to the assignment field submissions. Then, when a statement-level after trigger fires for assignment_field_submission, if I joined this to the current assignment_submission table, I would an inaccurate reflection of which users ought to be notified about this change because the assignment submission would have already been deleted.
Where does this leave me? Likely with the status quo: clients subscribe to alerts of table changes and then they refresh their data. Clearly, the refresh takes care of the RLS permissions.


command failed (exitstatus=1)

I am getting following error
RuntimeError (command failed (exitstatus=1): /var/www/apps/sophyapp-staging/shared/bundle/ruby/2.1.0/bin/wkhtmltopdf --quiet --page-size letter --margin-top 0.01in --margin-right 0.01in --margin-bottom 0.01in --margin-left 0.01in --encoding UTF-8 --header-spacing 0 - -):
I am using pdfkit-0.6.2.

You can configure pdfkit to not use --quiet so you can get more details as described in the documentation. This is not a bug. For support questions please use StackOverflow.


implement identical variant of cards for list and category view at marketplace.

Current Behaviour
implement identical variant of cards for List and category view at marketplace.
Screenshot:

Expected Behaviour
identical variants should be same at List and Category view.
Screenshot

Steps To Reproduce
1.Login to catena-X portal.
2.Mouseover to Marketplace.
3.Click on App marketplace.
4.Observe the List and Category App overview.

Hi @MaximilianHauer
I tested this issue it's working fine. Can you close this ticket.
Attached Screenshots for references:


[Animated] Use native animations in TouchableBounce and TouchableOpacity

Now that native animations for opacity and springs have landed in both iOS and Android, we can enable native animations both for TouchableBounce and TouchableOpacity.
Test Plan: Open the UIExplorer's Touchable demo and tap the "Enabled TouchableOpacity" component. Then change it to be TouchableBounce and reload the app and verify that works too. Do this on iOS and Android.

Talked to @ide about it and it has been used for a few months already in exponent. It will be one of the first major use of native animations in RN core.
@facebook-github-bot shipit

The internal system does not appear to have actually started importing. I'll retry

@ide Got reverted, something I thought about is that the native module for native animations is not included by default so this is probably the issue. We could check if it is present and set useNativeDriver accordingly.

@janicduplessis Good catch and that would be simple to fix.

@lacker Could you look into why D4099819 was reverted? If the problem has to do with "NativeAnimatedModule" not defined, we have a simple fix for that issue.

This gist has a patch with the possible fix. Could someone who's interested in getting this landed patch it and send a PR (feel free to change the author to yourself)? https://gist.github.com/ide/1802afe1146e53602e3b22f306c25909

This broke Instagram and a few other apps because they didn't have the native animated module added to them yet.

@fkgozali Is going to add the native animated module to fb apps and then we can merge this again.
Sorry about that!

We could check if it is present and set useNativeDriver accordingly.

@ide, @janicduplessis That sounds like a good thing to do in any case.

@janicduplessis, @ide Could you ping us on Slack before landing PRs that touch something very core? There are people in the west coast timezone on Slack so hopefully you'll get a reply quickly.

@mkonicek Well to be fair I imported this one. What else should I have done to check for problems here?

I've relanded this in the commit: https://github.com/facebook/react-native/commit/8bb707b6869624e4e133f9db7d29d627d9a18290
If there's anything breaking internally again, we'll fix forward. Thanks for your patience.

We'll probably want to either check if the NativeModule is available or mention this in the next release notes so we don't break OSS apps.

@mkonicek sorry for breaking your app :(
@janicduplessis I would prefer we check if the module is available -- that feels safest to me.

I have a pending change to warn instead of redboxing (and fallback to JS animated) - that's probably the temporary migration path

I've relanded this in the commit: 8bb707b

Thanks so much Kevin!

added fallback to JS with warning instead of redboxing: https://github.com/facebook/react-native/commit/3237ade34f4eae07771f9471f14bec5486fdf1d2

I'm seeing an issue with this PR that I described on this issue:  https://github.com/facebook/react-native/issues/11094#issuecomment-263240420.  But this thread may be the more appropriate place.
As I indicated (with more detail) on that issue, I'm getting the red box below only when I include and link RCTAnimation in my app (which I've been building/releasing/upgrading since RN 0.14.0... but is now on 0.38.0).  However, if I remove the useNativeDriver from TouchableOpacity I both avoid the crash below and don't get the fallback to JS warning that's included in 0.38.0.
Any ideas where I've gone wrong?  If I remove RCTAnimation but leave in this PR, no crash but I get the fallback to JS warning.  When I add and link RCTAnimation the warning goes away but my app crashes with this PR.  If I remove this PR I don't get the fallback warning and the app doesn't crash.

@janicduplessis, @fkgozali - It seems that any place in my app that has useNativeDriver: true causes the red screen above.  Does that mean RCTAnimated is not properly installed, even though I'm no longer getting the warning?  Or is something else going on?

Hmm I've never seen this error before, the setter is set here so I don't see why this happens https://github.com/facebook/react-native/blob/master/Libraries/NativeAnimation/RCTNativeAnimatedModule.m#L33.
Maybe try cleaning and removing the ~/Library/Developer/Xcode/DerivedData dir.

@janicduplessis - Wow... clearing the derived data fixed it.  Thank you so much... been fighting for this for a week.

 What things don't make sense deleting that folder is often the solution lol.


Allow override of properties on the command line

I'd like the ability to override stanzas on the command line.  Combine this with Issue #6 to provide a better eventgen debug environment:
i.e.
python eventgen.py default/eventgen.conf --stanza sp-iisupload.csv --property outputMode=file --property fileName=/tmp/output.log

I allowed some of hte properties without going full bore.  You can check out the current dev branch and see if overriding outputMode and a few others is sufficient.


Hello, just a question. I'm running into problems with the Service Worker Registration. Is this expected behavior?

Above is the Screenshot, the project is live @ https://portfolio.arturgvieira.com
I am running one initial HTTP REST request to an Express Server. I think the Service Worker does not like that. When the app fails to register the Worker, in the mobile device I'm testing the app on, the screen goes blank. Any information would be highly appreciated.
Thanks in advance!
Local Environment:
Linux / nvm installed NodeJS 9+ / React + Express + Firebase
Git repo:
https://github.com/arturgvieira/portfolio

When I visit the site it registered for me, haha [Chrome]. I'm not sure what's going on here / why Firefox doesn't like it.
Maybe @jeffposnick would have an idea.

Thanks for the response!

Ditto, the site registers the service worker successfully in both Chrome and Firefox when I visit it.
Requesting any resource within the service worker using http: (rather than https:) would lead to the error you saw. Did you recently change things to use https: instead?
In any case, can you try reproducing this in a Chrome Incognito window locally? If you can't reproduce it, then it's likely due to something you've since changed in your code.

Ooh, maybe a firefox extension is injecting a script into the page which contains a http call. Does this reproduce on a Firefox private/incognito window?

On a private/incognito window, the service worker does not fail to register.

I'm at a loss for what could have led to that specific failure that you're seeing, since we can't reproduce it ourselves and since there are protections in place in the c-r-a codebase to prevent you from, e.g., registering a service worker from a cross-origin URL, or registering a service worker via a http: URL when your main site is accessed via https:.
You talk about the REST request being an issue—are you seeing proper behavior when you locally test it without the REST call? And the failure only occurs when you have the call?
Also, I noticed that you have HTTP caching (of 1 hour) enabled for your service-worker.js file. Chrome respects that HTTP caching (Firefox does not), and that can make it frustrating to test changes to your site. I'd recommend disabling HTTP caching of service-worker.js in the meantime to eliminate one other source of inconsistency.

Thanks for the response, and the comments. I am currently also reviewing whether to keep the current features used of this and another website. Trying to balance feature usage and writing standards compliant websites. It's the current challenge and I think this is part of a bigger standards review I need to do.

@jeffposnick I was unable to reproduce in Chrome, I have changed the code, but in Firefox I am still unable to register the worker due to failure during registration. The REST calls, which currently there are a few small content calls. Fail during service worker registration.
I wanted to ask you about the caching config, I don't remember setting it up for 1 hour, maybe it was set as the default.

I removed the REST calls and the service worker still does not register.

@arturgvieira I believe it's one of your extensions in Firefox; try disabling them one by one to find the culprit.

I removed all the extensions, no luck. I did some more testing, I found a call to www.example.com/null by the service worker, returns a 404. Is that supposed to happen?

It's not supposed to happen (presumably), but at the same time, it's not necessarily a call that originates from the service worker. It might be another part of your web app that's inadvertently requesting that URL—once the SW is installed, any web traffic that the SW responds to will be from "(from ServiceWorker)" even if it didn't originate from it.

Oh ok. Thank you.


Revert "[CH-123] Support short/byte/binary/decimal/array/map/struct"

Reverts Kyligence/ClickHouse#128

Can one of the admins verify this patch?


test/es6/module-namespace.js: Random CI failure (release/1.7)

Saw in CI test build. It doesn't repro.. every time.
... -WERExceptionSupport -ExtendedErrorStackForTestHost -BaselineMode -maxInterpretCount:1 -maxSimpleJitRunCount:1 -bgjit- -dynamicprofilecache:profile.dpl.2160 -ES6Module -Es6ToStringTag -args summary -endargs test/es6/module-namespace.js
...
... Output:
... ----------------------------
... *** Async result of test #8 (7): Issue3246: namespace property names are sorted
... Test threw exception: assert.areEqual failed: expected: $ actual: undefined: ModuleNamespace
...
... FAILED
... Summary of tests: total executed: 8; passed: 7; failed: 1
...
... ----------------------------
... exit code: 0

Not sure to whom this should be assigned?

Related test code;
                import * as ns from 'ValidExportStatements2.js';
                var p = new Proxy(ns, {});
                var names = ["sym0","default","$","$$","_","\u03bb","aa","A","a","zz","z","\u03bc","Z","za","__","az","\u03c0"].sort();

                var verifyNamespaceOwnProperty = function(obj, objKind) {
                    assert.areEqual(names, Object.getOwnPropertyNames(obj), objKind+" getOwnPropertyNames()");

                    var propDesc = Object.getOwnPropertyDescriptors(obj);
                    assert.areEqual('{"value":"Module","writable":false,"enumerable":false,"configurable":false}', JSON.stringify(propDesc[Symbol.toStringTag]),
                        objKind+" getOwnPropertyDescriptors() @@toStringTag");
                    assert.areEqual(names, Object.keys(propDesc), "ModuleNamespace", objKind+" getOwnPropertyDescriptors()");
                };

                verifyNamespaceOwnProperty(ns, "ModuleNamespace");
                verifyNamespaceOwnProperty(p, "Proxy");

                var propEn = [];
                for (var k in ns) { propEn.push(k); }
                assert.areEqual(names, propEn, "ModuleNamespace enumerator");
                ;

It complains about ns is being undefined ?
 import * as ns from 'ValidExportStatements2.js';

This is the expected variable list;
var names = ["sym0","default","$","$$",.....
Below; what it should read;
var x;
export { x as \u03c0 }; // u03c0
export { x as az };
export { x as __ };
export { x as za };
export { x as Z };
export { x as \u03bc };
export { x as z };
export { x as zz };
export { x as a };
export { x as A };
export { x as aa }
export { x as \u03bb }; // u03bb
export { x as _ };
export { x as $$ };
export { x as $ };
export default null;
export var sym0 = Symbol('abc');

So, for some reason, instead of reading $, reads undefined ?

@akroshg assigning to you to triage

Make sure to re-enable the test once fixed, see #3808

Related OS#12970055

@Cellule @leirocks ?

This doesn't repro anymore.
Also reported as OS#12970055
Created a batch file on Windows, in order to try repro'ing there too. No success.
@echo off
IF EXIST module-namespace.js (
    FOR /L %%A IN (1,1,200) DO (
        ..\..\..\Build\VcBuild\bin\x64_debug\jshost.exe -bvt -BaselineMode  -maxInterpretCount:1 -maxSimpleJitRunCount:1 -bgjit- -dynamicprofilecache:profile.dpl.UnnamedTest1785  -NoNative -ES6Module -Es6ToStringTag -args summary -endargs module-namespace.js
        IF %ERRORLEVEL% NEQ 0 (
            echo "FAILED!"  %ERRORLEVEL%
            goto :eof
        )
    )
) ELSE (
    echo "Run this script under test/es6 OR core/test/es6 (under full)"
)

(if you want to give it a try on chakracore, replace jshost to ch and remove first -bvt)
Closing this for now.

If we think this really doesn't repro anymore, then we have to reenable the test before closing this issue

If we think this really doesn't repro anymore, then we have to reenable the test before closing this issue

Agree, I was about to open this issue and let PR to close this. Saw you did that already.