RFC: update tests to MathOptInterface

I started a new branch rather than rebasing the old moi.

@frapac want to run Knitro over this branch?

I will! Unfortunately, I do not have KNITRO on this computer and have to wait till Monday...

AmplNLWriter.jl is currently passing the nlp and nlp-cvx tests on this branch!
(With the exception of nlp_cvx_109_010.)

This MOI branch has been updated to the latest version of the tests on master (a v0.2 rc).
@frapac @odow @leethargo last chance to request to changes the cases before I tag v0.2, merge this branch and tag v0.3.  Will leave this open for discussion for about 48 hours.

Look good to me!
Knitro is able to pass almost all tests (I just have to set opttol=1e-8 to have a solution with enough precision for the new nlp_cvx/109_010).
However, if I solve nlp-mi/006_010 with Knitro, I get a different (and better) solution than those expected:
    check_objective(m, 1.69314718055995, tol = objective_tol)
    check_solution([x,y], [0.69314718055995, 1.], tol = primal_tol)

instead of
    check_objective(m, 1.8813786425753092, tol = objective_tol)
    check_solution([x,y], [0.7546057578960682, 1.1267728846792409], tol = primal_tol)

I am still unable to solve the following tests, but this is unrelated to this PR:

nlp_mi/005_011
nlp/005_011

Let me see if I can adjust nlp-mi/006_010 so that all solvers are finding OPT.  Are there any simple adjustments we can make to nlp_cvx/109_010 to migitage the tolerance issue, or is this a fundamental challenge of transitential functions.

The only thing that I do not understand is why the initial position is set equal to the optimal one on od/moi (see these lines) whereas this is not the case on master.
But I may have missed a point in the discussion!
Apart that particular point, nlp-cvx/109_010 looks good.
If you think that could be useful, I could add other tests in MINLPTests in a near future.

The only thing that I do not understand is why the initial position is set equal to the optimal one on

https://github.com/JuliaOpt/MINLPTests.jl/pull/8#discussion_r253249285

OK, so I tried running the tests again with SCIP.jl, but I find it tricky to update the branch, actually:
I copied the approach from AMPLWriter and set up a project for MINLPtests inside SCIP. There, when I now use ] dev MINLPTests, it will install Juniper and downgrade JuMP to v0.18.5, to the effect that with_optimizer no longer exists etc.
I also tried to ] dev Juniper but it seems that Juniper#master is not compatible with JuMP v0.19 either.
When I finally figure out the right Pkg incantations, I will propose to add these steps to the README here :wink:

@leethargo commit the Project.toml and Manifest.toml files, and then start Julia with julia --project=MINLPTests and everything should work.

@odow I already did that previously: SCIP.jl/MINLPTests
But would this old Project.toml not reference the old commit prior to @ccoffrin merging into this branch? That's why I wanted to do ] up while in the MINLPTests project.

If you think that could be useful, I could add other tests in MINLPTests in a near future.

More tests are surely welcome as long as they bring new value and keep in mind the current scope of this repo, which is testing JuMP / MOI API interfaces.
For testing more algorithmic properties we might make a new instance categories here or even another repo which focuses on that specifically.

@lethargo instead of dev MINLPTests, add MINLPTests#od/moi.

@lethargo instead of dev MINLPTests, add MINLPTests#od/moi.

Thanks. Yes, that's what I did initially. I did not think it would automatically pull the newest commits from that branch on later uses.
When I try that ] add now, in the project that already has MINLPTests, I still the version conflict through Juniper: gist
Anyway, that's kind of off-topic here, I guess.

Bump. @ccoffrin any timeline on merging this?

Totally lost track of this.  My bad.
Master is ahead of the last release by 6 commits.  I should have tagged v0.2 a while ago.  This would be simple to do now, except for the Julia package manager updates.
My proposal is,

update to master to the new Julia package manager
tag v0.2 (JuMP v0.18 compatible)
merge this branch
tag v0.3 (JuMP v0.19 compatible)

Sound good to everyone?

@odow, can you make sure this branch has the latest changes to the models from master?  I just want to make sure we are up to date with the model revisions between v0.1 and v0.2.

@ccoffrin the latest commit aligns the tests between master and this branch. Mainly reverting changes we had made here.


Old elo's pattern

Hey there, didn't find any guideline to open a issue in this repo, neither to open a pull request, but here i am haha!
So, my issue is with Elo's credit card number validation, some new numbers aren't being detected.
The fix is simple, just update the actual regex to the new one, based on this gist comment https://gist.github.com/erikhenrique/5931368#gistcomment-1683630
index bbc9246..a384543 100644
--- a/src/index.coffee
+++ b/src/index.coffee
@@ -93,7 +93,7 @@ cards = [
   }
   {
       type: 'elo'
-      pattern: /^(4011(78|79)|43(1274|8935)|45(1416|7393|763(1|2))|50(4175|6699|67[0-7][0-9]|9000)|627780|63(6297|6368)|650(03([^4])|04([0-9])|05(0|1)|4(0[5-9]|3[0-9]|8[5-9]|9[0-9])|5([0-2][0-9]|3[0-8])|9([2-6][0-9]|7[0-8])|541|700|720|901)|651652|655000|655021)/,
+      pattern: /^401178|^401179|^431274|^438935|^451416|^457393|^457631|^457632|^504175|^627780|^636297|^636369|^(506699|5067[0-6]\d|50677[0-8])|^(50900\d|5090[1-9]\d|509[1-9]\d{2})|^65003[1-3]|^(65003[5-9]|65004\d|65005[0-1])|^(65040[5-9]|6504[1-3]\d)|^(65048[5-9]|65049\d|6505[0-2]\d|65053[0-8])|^(65054[1-9]|6505[5-8]\d|65059[0-8])|^(65070\d|65071[0-8])|^65072[0-7]|^(65090[1-9]|65091\d|650920)|^(65165[2-9]|6516[6-7]\d)|^(65500\d|65501\d)|^(65502[1-9]|6550[3-4]\d|65505[0-8])/,
       format: defaultFormat
       length: [16]
       cvcLength: [3]

Thanks!

Hello there, I think this regex is already outdated :)
Here is a regex including 636368 and 650921 to 650978:
/^401178|^401179|^431274|^438935|^451416|^457393|^457631|^457632|^504175|^627780|^636297|^636369|^636368|^(506699|5067[0-6]\d|50677[0-8])|^(50900\d|5090[1-9]\d|509[1-9]\d{2})|^65003[1-3]|^(65003[5-9]|65004\d|65005[0-1])|^(65040[5-9]|6504[1-3]\d)|^(65048[5-9]|65049\d|6505[0-2]\d|65053[0-8])|^(65054[1-9]|6505[5-8]\d|65059[0-8])|^(65070\d|65071[0-8])|^65072[0-7]|^(65090[1-9]|65091\d|650920)|^(65165[2-9]|6516[6-7]\d)|^(65500\d|65501\d)|^(65502[1-9]|6550[3-4]\d|65505[0-8])|^(65092[1-9]|65097[0-8])/

ELO Sucks! :P

This was released in 2.4.0


RethinkDB Connector

This feature request is a central place to discuss the development of RethinkDB connector. Please contribute in the comments below.

Given RethinkDB's baked in and performant reactive data layer, I think this connector could be interesting for the first step(s) towards a live query implementation!

Where do you guys think I should start with this connector? I will focus on mutation, query, and synchronization between Prisma data storage and MongoDB collections

Any progress on this suggestion?

Bump

Any updates on this?

@QuantumlyTangled
Considering that even the MS SQL Connector (which a lot more people are waiting for) is still not ready either, I doubt the RethinkDB one will be dealt before the MS SQL one. Just guessing here...


merge package template from arlac77/npm-package-template

LICENSE

chore(license): add current year 2018

Codecov Report

Merging #350 into master will decrease coverage by 1.63%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #350      +/-   ##
==========================================
- Coverage   79.78%   78.14%   -1.64%
==========================================
  Files           3        3
  Lines         183      183
==========================================
- Hits          146      143       -3
- Misses         37       40       +3

Impacted Files
Coverage Δ

tests/service-test.js
95.12% <0%> (-2.44%)
:arrow_down:

src/service-consul.js
71.64% <0%> (-1.5%)
:arrow_down:

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 f720e31...154b101. Read the comment docs.

Coverage decreased (-1.3%) to 71.308% when pulling 154b101490c16e8a185fe04fb1cdc37abc7167c1 on template-sync-1 into f720e31e290c8a42fb12ae98fdf100d630f3c50e on master.

Coverage decreased (-1.3%) to 71.308% when pulling 154b101490c16e8a185fe04fb1cdc37abc7167c1 on template-sync-1 into f720e31e290c8a42fb12ae98fdf100d630f3c50e on master.


Add Objectives to sample and average April Tag pose

This PR adds new objectives to sample and average the pose estimation of an April Tag detector.
New objectives added:

Loop Detect April (used to visualize frames in Rviz, calibrate to the point_cloud, or experiment with apriltag_detection_config.yaml settings)
Sample April Tag (sub-tree that collects a specified number of pose samples and calculates the average)

I also updated the Pick Apriltag Labeled Object Objective to use the new Sample April Tag sub-tree.
To test this out you can run the Loop Detect April and Pick Apriltag Labeled Object from the Studio-UI.

Finally, these changes mean we have to update these docs, so plan to do that once these changes go in!

That whole tutorial is going to be re-written to show people how to teach grasp offsets from AprilTags once all of the required behaviors are added to core Studio.


Add support for SingleSignOnService/prepare_for_authentication with HTTP-POST binding

Warning, this changes the return type of prepare_for_authentication by including the chosen binding, and opens the door for supporting other SingleSignOnService bindings. If we want to maintain backwards compatibility though we could make a different function.

@rohe: Hello there. Do you have a suggestion for what to name this other method? How does prepare_for_negotiated_authentication sound? It's a bit wordy though.

@rohe: I pushed up a new version that adds prepare_for_negotiated_authentication, which returns a triplet, and kept the current behavior for prepare_for_authentication. Would it make sense then to deprecate the older method?

@rohe: I've updated the patch so it merges again. Does it look good to merge?

@rohe: Just checking in if you've had a chance to look it over :)

Thanks @rohe!


Add more TAC(s)

I'll add a few more in a couple of hours

I'd open another PR but that'll conflict -- might as well piggy back.. Could you add 35883293,SCV49 ?
Thanks!

@martined I added that line at the end and fixed an error i made while modifying the original content


MAINT: Use extern templates to improve sparsetools compile time

What does this implement/fix?
While trying to compile SciPy on a raspberry pi for gh-10463 I kept running into out of memory issues compiling sparsetools/bsr.cxx. Since its implementation relies on CSR for a bunch of functionality, it fully compiles the templates for both BSR and CSR for  all type combinations. So, I use the C++11 extern template syntax so the compiler doesn't generate code for non-inlined csr_ routines when compiling bsr.cxx.
A really minor other issue was that test_throw_error is unnecessarily templated and gets compiled twice just because the code generator calls the function as test_throw_error<npy_int32> or test_throw_error<npy_int64>. So, I fixed the generator to support non-templated functions.
These additions not only reduce the compiler memory footprint but also makes compilation faster.

Thanks @peterbell10!


Pin / unpin on toggle features panel does not work

Steps to reproduce

Grab the latest 2.1 prerelease
Build to HL2
Press the pin / unpin button

Expected Behavior
When I press, the button should toggle and menu should follow me
Actual Behavior
The button does not toggle and follow me

Marking this is as release blocker since it is user visible defect for a feature we are advertising

On latest stabilization via git branch and in Unity 2018.6, the hand interaction examples scene pin works

Version of this not working is same as #6169
For some reason, Unity is not treating FormallySerailizedAs properly. Try updating to latest stabilization and trying

Duplicate of #6169


Removed the Azure::Core::Http::Request ctor overload that takes both a bodyStream and a bufferedDownload boolean since it is not used anywhere.

/azp run cpp - storage

FYI @vhvb1989, made the ctor private.

/azp run cpp - storage


bundle dynamic import (typescript)

some dependencies use top level await, which isn't supported in cloudflare worker, I with it can provide transpile support for it

Cloudflare supports some top level awaits (like crypto calls), but not all - you'll get 10021 Uncaught Error: Some functionality, such as asynchronous I/O, timeouts, and generating random values, can only be performed while handling a request.
Did you have a question?  What I do normally is a one-time init in the scope of a request

Cloudflare supports some top level awaits (like crypto calls), but not all - you'll get 10021 Uncaught Error: Some functionality, such as asynchronous I/O, timeouts, and generating random values, can only be performed while handling a request.
Did you have a question? What I do normally is a one-time init in the scope of a request

ok I found the problem is the code is trying to instantiate a wasm module, it is not support anyway (cf require to use binding), I modified the code to not use wasm, and the problem is solved


Update README.md

This PR is a follow-up to #3 and contains some small changes to the README:

Mention what is available and link to API docs at the top
Also put install command at the top, instead of after the longish plotting example
Remove the Progress table. It doesn't contain extra info compared to the API docs, and manually keeping the table in sync with the API docs is tedious / error-prone.

Thanks!


Join page - mobile adjustment

The join page does not seem to adjust to mobile narrower width on Android.

Example of what I see on Android Mobile for Join page, white boxes should go inline

Added some flexbox/responsiveness

Deployment link:
https://plebdevs-eriknewland.vercel.app/

Thanks for the PR, a few things. I don't know how difficult it would be to make this adjustment, so let me know if its doable or not

when viewing full browser size, it doesn't go back to side by side and the rectangles are of different sizes.
Its not quite obvious when I click on the 'Individual' or 'corporate' boxes that there is a form, sometimes, it my window is smaller. How can we address this better? either by

A) redirecting to a separate page with the form
B) pushing the form to the top and the links to the bottom
C) doing a modal where the form pops up over

Example of what i see in Firefox desktop browser

Interesting, I'll try and track down what's going on with the flex issue there. And as for number 2, I think a Modal would probably be best. I'll get back to you later today with some updates

this is good, close?


Is k8s with crio supported?

Hi!
Is k8s with crio supported? So could I just change socket from /var/run/docker/docker.sock to  /var/run/crio/crio.sock?

Hi :) Support for crio and containerd is planned but not ready yet. The protocol used over docker.sock differs from the others, so I don't think it'll just work if you change the path.
As for gProfiler itself - it will run alright on k8s no matter what the container runtime is, you will get profiles just the same, but they will not be correlated with their containers/pods/deployments.

You can use the local output mode as described here. It will write files locally and you can collect them. The Performance Studio itself is not available on air-gapped environments, sadly.

https://github.com/Granulate/gprofiler/pull/296 will work for crio-o as well :) Will be merged soon.

Hi @dudniksergei, I forgot to update you here. #296 was merged and released in 1.2.15, so you can just grab the latest release and it will have crio-o support out of the box - you don't need to change anything.
Closing this as the original request was fulfilled, feel free to open another issue for other questions.

Hi @dudniksergei, I forgot to update you here. #296 was merged and released in 1.2.15, so you can just grab the latest release and it will have crio-o support out of the box - you don't need to change anything.
Closing this as the original request was fulfilled, feel free to open another issue for other questions.

thank you!


[1.10.2 Bug] Storage scanner crash

Hi just leaving a feedback for a bug i found while setting up some automation.  When using the storage scanner to dump coal directly into a seeder from the Progressive Automation mod the game crashes and renders the map you did this on unusable until you either start over or mcedit the map getting rid of the scanner from the world entirely and starting the process over.  Loving the mod as it has a ton of useful things.  Keep up the great work.

Can you give me a bit more information? Like the actual crash and also version of the mods?

Will do this evening.  Just about to leave for work.  I will set up a small world to recreate the crash so i can get you the crash report.  Until then the mod file names are rftools-1.10-5.51 and ProgressiveAutomation-1.10.2-1.6.50  I apologize for the wait of the crash report but i wont have time to set things up and duplicate this before work.  Tonight for sure though.

Ok here we go  First up is all the mod versions rftools 1.10-5.51  progressive automation 1.6.50 running with Forge <IP_ADDRESS>9  Below is the crash report.  When recreating the crash i noticed linking the storage scanner to the Diamond Planter does not cause the crash its only when any coal attempts to be pushed into it.  This may not even be a bug with your mod but could be one with the progressive automation mod instead i have no idea as i dont know how to make sense of the crash report.  Hope this helps
---- Minecraft Crash Report ----
WARNING: coremods are present:
LoadingPluginBuildingBricks (Building-Bricks-Mod-1.10.2.jar)
NWRTweak (Redstone-Paste-Mod-1.10.2.jar)
MalisisCorePlugin (malisiscore-1.9.4-4.0.2.jar)
LoadingPlugin (Random-Things-Mod-1.10.2.jar)
Contact their authors BEFORE contacting forge
// Shall we play a game?
Time: 12/13/16 1:28 AM
Description: Ticking block entity
java.lang.NullPointerException: Ticking block entity
at com.vanhal.progressiveautomation.entities.BaseTileEntity.func_180462_a(BaseTileEntity.java:530)
at mcjty.lib.container.InventoryHelper.mergeItemStackInternal(InventoryHelper.java:337)
at mcjty.lib.container.InventoryHelper.mergeItemStackSafe(InventoryHelper.java:312)
at mcjty.lib.container.InventoryHelper.insertItem(InventoryHelper.java:231)
at mcjty.rftools.blocks.storagemonitor.StorageScannerTileEntity.injectStackInternal(StorageScannerTileEntity.java:214)
at mcjty.rftools.blocks.storagemonitor.StorageScannerTileEntity.func_73660_a(StorageScannerTileEntity.java:160)
at net.minecraft.world.World.func_72939_s(World.java:1804)
at net.minecraft.world.WorldServer.func_72939_s(WorldServer.java:619)
at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:709)
at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:613)
at net.minecraft.server.integrated.IntegratedServer.func_71217_p(IntegratedServer.java:149)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:471)
at java.lang.Thread.run(Thread.java:745)
A detailed walkthrough of the error, its code path and all known details is as follows:
-- Head --
Thread: Server thread
Stacktrace:
at com.vanhal.progressiveautomation.entities.BaseTileEntity.func_180462_a(BaseTileEntity.java:530)
at mcjty.lib.container.InventoryHelper.mergeItemStackInternal(InventoryHelper.java:337)
at mcjty.lib.container.InventoryHelper.mergeItemStackSafe(InventoryHelper.java:312)
at mcjty.lib.container.InventoryHelper.insertItem(InventoryHelper.java:231)
at mcjty.rftools.blocks.storagemonitor.StorageScannerTileEntity.injectStackInternal(StorageScannerTileEntity.java:214)
at mcjty.rftools.blocks.storagemonitor.StorageScannerTileEntity.func_73660_a(StorageScannerTileEntity.java:160)
-- Block entity being ticked --
Details:
Name: rftools_storage_scanner // mcjty.rftools.blocks.storagemonitor.StorageScannerTileEntity
Block type: ID #1135 (tile.rftools.storage_scanner // mcjty.rftools.blocks.storagemonitor.StorageScannerBlock)
Block data value: 4 / 0x4 / 0b0100
Block location: World: (-94,7,-1100), Chunk: (at 2,0,4 in -6,-69; contains blocks -96,0,-1104 to -81,255,-1089), Region: (-1,-3; contains chunks -32,-96 to -1,-65, blocks -512,0,-1536 to -1,255,-1025)
Actual block type: ID #1135 (tile.rftools.storage_scanner // mcjty.rftools.blocks.storagemonitor.StorageScannerBlock)
Actual block data value: 4 / 0x4 / 0b0100
Stacktrace:
at net.minecraft.world.World.func_72939_s(World.java:1804)
at net.minecraft.world.WorldServer.func_72939_s(WorldServer.java:619)
-- Affected level --
Details:
Level name: Think Tank
All players: 1 total; [EntityPlayerMP['Wolfie2446'/215, l='Think Tank', x=-95.32, y=7.00, z=-1100.60]]
Chunk stats: ServerChunkCache: 569 Drop: 0
Level seed:<PHONE_NUMBER>243577296
Level generator: ID 01 - flat, ver 0. Features enabled: true
Level generator options:
Level spawn location: World: (-92,4,-1271), Chunk: (at 4,0,9 in -6,-80; contains blocks -96,0,-1280 to -81,255,-1265), Region: (-1,-3; contains chunks -32,-96 to -1,-65, blocks -512,0,-1536 to -1,255,-1025)
Level time: 598071 game time, 27711 day time
Level dimension: 0
Level storage version: 0x04ABD - Anvil
Level weather: Rain time: 48222 (now: false), thunder time: 73439 (now: false)
Level game mode: Game mode: creative (ID 1). Hardcore: false. Cheats: true
Stacktrace:
at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:709)
at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:613)
at net.minecraft.server.integrated.IntegratedServer.func_71217_p(IntegratedServer.java:149)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:471)
at java.lang.Thread.run(Thread.java:745)
-- System Details --
Details:
Minecraft Version: 1.10.2
Operating System: Windows 10 (amd64) version 10.0
Java Version: 1.8.0_25, Oracle Corporation
Java VM Version: Java HotSpot(TM) 64-Bit Server VM (mixed mode), Oracle Corporation
Memory:<PHONE_NUMBER> bytes (1798 MB) /<PHONE_NUMBER> bytes (3291 MB) up to<PHONE_NUMBER> bytes (8179 MB)
JVM Flags: 6 total; -XX:HeapDumpPath=MojangTricksIntelDriversForPerformance_javaw.exe_minecraft.exe.heapdump -Xmx8G -XX:+UseConcMarkSweepGC -XX:+CMSIncrementalMode -XX:-UseAdaptiveSizePolicy -Xmn128M
IntCache: cache: 0, tcache: 0, allocated: 0, tallocated: 0
FML: MCP 9.32 Powered by Forge <IP_ADDRESS>9 54 mods loaded, 54 mods active
States: 'U' = Unloaded 'L' = Loaded 'C' = Constructed 'H' = Pre-initialized 'I' = Initialized 'J' = Post-initialized 'A' = Available 'D' = Disabled 'E' = Errored
UCHIJAAAA	mcp{9.19} [Minecraft Coder Pack] (minecraft.jar)
UCHIJAAAA	FML{<IP_ADDRESS>} [Forge Mod Loader] (forge-1.10.2-<IP_ADDRESS>9.jar)
UCHIJAAAA	Forge{<IP_ADDRESS>9} [Minecraft Forge] (forge-1.10.2-<IP_ADDRESS>9.jar)
UCHIJAAAA	buildingbrickshooks{1.10.2-2.0.12} [Building Bricks Hooks] (minecraft.jar)
UCHIJAAAA	mercurius_updater{1.0} [mercurius_updater] (MercuriusUpdater-1.10.2.jar)
UCHIJAAAA	actuallyadditions{1.10.2-r62} [Actually Additions] (ActuallyAdditions-1.10.2-r62.jar)
UCHIJAAAA	mcmultipart{1.2.0} [MCMultiPart] (MCMultiPart-Mod-1.10.2.jar)
UCHIJAAAA	bagelsmore{1.0.0} [Bagelsmore: The Return] (Bagelsmore Mod 1.10.2.jar)
UCHIJAAAA	bdlib{<IP_ADDRESS>} [BD Lib] (bdlib-<IP_ADDRESS>-mc1.10.2.jar)
UCHIJAAAA	betterbuilderswands{0.6.5} [Better Builder's Wands] (Better-Builders-Wands-Mod-1.10.2.jar)
UCHIJAAAA	buildingbricks{1.10.2-2.0.12} [Building Bricks] (Building-Bricks-Mod-1.10.2.jar)
UCHIJAAAA	Chameleon{1.9.4-2.1.5} [Chameleon] (Chameleon-Library-1.10.2.jar)
UCHIJAAAA	chisel{MC1.9.4-<IP_ADDRESS>} [Chisel] (Chisel-Mod-1.10.2.jar)
UCHIJAAAA	cookingforblockheads{3.1.26} [Cooking for Blockheads] (Cooking-for-Blockheads-Mod-1.10.2.jar)
UCHIJAAAA	correlatedpotentialistics{1.1.3} [Correlated Potentialistics] (Correlated-Potentialistics-Mod-1.10.2.jar)
UCHIJAAAA	crossbowmod{1.3} [Crossbow Mod] (crossbowmod-1.3.jar)
UCHIJAAAA	ExtraUtils2{1.0} [ExtraUtils2] (Extra-Utilities-Mod-1.10.0.jar)
UCHIJAAAA	mantle{1.10.2-0.10.4.jenkins163} [Mantle] (Mantle-1.10.2-0.10.4.jar)
UCHIJAAAA	tconstruct{1.10-2.3.3a.jenkins271} [Tinkers' Construct] (TConstruct-1.10-2.3.3a.jar)
UCHIJAAAA	immersiveengineering{${version}} [Immersive Engineering] (ImmersiveEngineering-0.10-38.jar)
UCHIJAAAA	JEI{<IP_ADDRESS>} [Just Enough Items] (Just-Enough-Items-Mod-1.10.2.jar)
UCHIJAAAA	cyclicmagic{1.8.14} [Cyclic] (Cyclic-1.10.2-1.8.14.jar)
UCHIJAAAA	cyclopscore{0.7.7} [Cyclops Core] (CyclopsCore-1.9.4-0.7.7.jar)
UCHIJAAAA	EnderBags{2.0.1} [EnderBags] (EnderBags-1.9.4-2.0.0.jar)
UCHIJAAAA	enhancedvisuals{1.0} [Enhanced Visuals] (Enhanced-Visuals-Mod-1.10.2.jar)
UCHIJAAAA	everlastingabilities{1.1.0} [EverlastingAbilities] (Everlasting Abilities Mod 1.10.2.jar)
UCHIJAAAA	fastleafdecay{v10} [Fast Leaf Decay] (FastLeafDecay-Mod-1.10.2.jar)
UCHIJAAAA	harvestcraft{1.9.4-pre} [Pam's HarvestCraft] (HarvestCraft-Mod-1.10.2.jar)
UCHIJAAAA	hopperducts{1.4.7} [Hopper Ducts] (Hopper-Ducts-Mod-1.9.4-1.10-1.10.2.jar)
UCHIJAAAA	immcraft{1.1.7} [Immersive Craft] (immcraft-1.9.4-1.1.7.jar)
UCHIJAAAA	llibrary{1.5.1} [LLibrary] (llibrary-1.5.1-1.9.4.jar)
UCHIJAAAA	jurassicraft{2.0.3} [JurassiCraft] (JurassiCraft-2.0.3.jar)
UCHIJAAAA	lootbags{2.1.0} [Loot Bags] (LootBags-1.10.2-2.1.0.jar)
UCHIJAAAA	malisiscore{1.9.4-4.0.2} [MalisisCore] (malisiscore-1.9.4-4.0.2.jar)
UCHIJAAAA	malisisdoors{1.9.4-5.0.1} [MalisisDoors] (malisisdoors-1.9.4-5.0.1.jar)
UCHIJAAAA	mocreatures{10.0.0} [DrZhark's Mo'Creatures] (Mo-Creatures-Mod-1.10.2.jar)
UCHIJAAAA	mobdrops{<IP_ADDRESS>} [MobDrops] (MobDrops <IP_ADDRESS>.jar)
UCHIJAAAA	mysticalagriculture{1.2.2} [Mystical Agriculture] (mysticalagriculture-1.2.2.jar)
UCHIJAAAA	powerinventory{2.3.0} [Overpowered Inventory] (OverpoweredInventory-1.10.2-2.3.0.jar)
UCHIJAAAA	simplerecipes{1.9.4-1.10.2a} [Pam's Simple Recipes] (Pam's Simple Recipes 1.9.4-1.10.2a.jar)
UCHIJAAAA	progressiveautomation{1.6.50} [Progressive Automation] (ProgressiveAutomation-1.10.2-1.6.50.jar)
UCHIJAAAA	randomthings{3.7.4} [Random Things] (Random-Things-Mod-1.10.2.jar)
UCHIJAAAA	redstonepaste{1.7.4} [Redstone Paste] (Redstone-Paste-Mod-1.10.2.jar)
UCHIJAAAA	rftools{5.51} [RFTools] (rftools-1.10-5.51.jar)
UCHIJAAAA	rftoolsdim{4.45} [RFTools Dimensions] (rftoolsdim-1.10-4.45.jar)
UCHIJAAAA	simplebarrels{1.25g} [Simple Barrels] (simplebarrels-1.10.2-2002-1.25g.jar)
UCHIJAAAA	slabcraftmod{6.01} [Slabcraft] (Slabcraftmod v6.01 - MC1.10.2 Forge.jar)
UCHIJAAAA	StorageDrawers{1.9.4-3.1.11} [Storage Drawers] (Storage-Drawers-Mod-1.10.2-1.10.jar)
UCHIJAAAA	TinkersAddons{1.0.3} [Tinkers' Addons] (Tinkers' Addons-1.10.x-1.0.3.jar)
UCHIJAAAA	torohealthmod{1.10.2-1} [ToroHealth Damage Indicators] (torohealth-1.10.2-1.jar)
UCHIJAAAA	wallcraftmod{6.01} [Wallcraft] (Wallcraftmod v6.01 - MC1.10.2 Forge.jar)
UCHIJAAAA	WIT{1.0.20} [WIT] (WIT-1.9.4-1.0.20-38.jar)
UCHIJAAAA	XaeroMinimap{1.9.6} [Xaero's Minimap] (Xaeros-Minimap-Mod-1.10.2.jar)
UCHIJAAAA	XaeroWorldMap{1.2} [Xaero's World Map] (XaerosWorldMap_b1.3_Forge_1.10.2.jar)
Loaded coremods (and transformers):
LoadingPluginBuildingBricks (Building-Bricks-Mod-1.10.2.jar)
com.hea3ven.buildingbricks.core.load.ClassTransformerBuildingBricks
NWRTweak (Redstone-Paste-Mod-1.10.2.jar)
net.fybertech.nwr.NWRTransformer
MalisisCorePlugin (malisiscore-1.9.4-4.0.2.jar)
net.malisis.core.util.chunkcollision.ChunkCollisionTransformer
net.malisis.core.util.chunkblock.ChunkBlockTransformer
net.malisis.core.renderer.transformer.MalisisRendererTransformer
net.malisis.core.renderer.icon.asm.TextureMapTransformer
net.malisis.core.util.clientnotif.ClientNotifTransformer
LoadingPlugin (Random-Things-Mod-1.10.2.jar)
lumien.randomthings.asm.ClassTransformer
GL info: ERROR RuntimeException: No OpenGL context found in the current thread.
Pulsar/tconstruct loaded Pulses:
- TinkerCommons (Enabled/Forced)
- TinkerWorld (Enabled/Not Forced)
- TinkerTools (Enabled/Not Forced)
- TinkerSmeltery (Enabled/Not Forced)
- TinkerGadgets (Enabled/Not Forced)
- TinkerOredict (Enabled/Forced)
- TinkerIntegration (Enabled/Forced)
- TinkerFluids (Enabled/Forced)
- TinkerMaterials (Enabled/Forced)
List of loaded APIs:
	* actuallyadditionsapi (22) from ActuallyAdditions-1.10.2-r62.jar
	* ChiselAPI (0.0.1) from Chisel-Mod-1.10.2.jar
	* ChiselAPI|Carving (0.0.1) from Chisel-Mod-1.10.2.jar
	* CoFHAPI (1.8.9R1.2.0B1) from ProgressiveAutomation-1.10.2-1.6.50.jar
	* CoFHAPI|block (1.8.9R1.2.0B1) from ProgressiveAutomation-1.10.2-1.6.50.jar
	* CoFHAPI|energy (1.8.9R1.2.0B1) from Extra-Utilities-Mod-1.10.0.jar
	* CoFHAPI|item (1.8.9R1.2.0B1) from mcjtylib-1.10-2.1.4.jar
	* CoFHAPI|tileentity (1.8.9R1.2.0B1) from ProgressiveAutomation-1.10.2-1.6.50.jar
	* ImmersiveEngineering|API (1.0) from ImmersiveEngineering-0.10-38.jar
	* ImmersiveEngineering|ImmersiveFluxAPI (1.0) from ImmersiveEngineering-0.10-38.jar
	* JustEnoughItemsAPI (4.0.0) from Just-Enough-Items-Mod-1.10.2.jar
	* McJtyLib (1.10-2.1.4) from mcjtylib-1.10-2.1.4.jar
	* StorageDrawersAPI (1.7.10-1.2.0) from Storage-Drawers-Mod-1.10.2-1.10.jar
	* StorageDrawersAPI|config (1.7.10-1.2.0) from Storage-Drawers-Mod-1.10.2-1.10.jar
	* StorageDrawersAPI|event (1.7.10-1.2.0) from Storage-Drawers-Mod-1.10.2-1.10.jar
	* StorageDrawersAPI|inventory (1.7.10-1.2.0) from Storage-Drawers-Mod-1.10.2-1.10.jar
	* StorageDrawersAPI|pack (1.7.10-1.2.0) from Storage-Drawers-Mod-1.10.2-1.10.jar
	* StorageDrawersAPI|registry (1.7.10-1.2.0) from Storage-Drawers-Mod-1.10.2-1.10.jar
	* StorageDrawersAPI|render (1.7.10-1.2.0) from Storage-Drawers-Mod-1.10.2-1.10.jar
	* StorageDrawersAPI|storage (1.7.10-1.2.0) from Storage-Drawers-Mod-1.10.2-1.10.jar
	* StorageDrawersAPI|storage-attribute (1.7.10-1.2.0) from Storage-Drawers-Mod-1.10.2-1.10.jar
	* WIT-API (1) from WIT-1.9.4-1.0.20-38.jar
Profiler Position: N/A (disabled)
Player Count: 1 / 8; [EntityPlayerMP['Wolfie2446'/215, l='Think Tank', x=-95.32, y=7.00, z=-1100.60]]
Type: Integrated Server (map_client.txt)
Is Modded: Definitely; Client brand changed to 'fml,forge'

The crash is actually in progressiveautomation. So most likely not a bug in RFTools. If you make a report to them you can say that they apparently don't properly support 'null' facing in the IItemHandler. And that is required. With facing equal to null they should give access to internal slots. Many mods make mistakes with that.


spl-token create-token  No such file or directory (os error 2)

When I execute eh spl-token create-token code it gives me the error 2
I do not know what I the problem, because I have created two tokens already without a problem, but now it would not let me.
I tried to do it  over and over again,but does not work,
Here is what I do:
cd desktop
mkdir moon
solana-keygen new --outfile moon/my-keypair.json  (I have my pubkey at this point )
solana airdrop 1 <RECIPIENT_ACCOUNT_ADDRESS> --url https://api.devnet.solana.com (I got the 1 sol)
solana config get
solana config set --url https://api.devnet.solana.com
solana config set --keypair ${HOME}/new-keypair.json
spl-token create-token ( and then I get the No such file or directory (os error 2) )
Also I changed the code: /new-keypair.json by /my-keypair.json and it let me create the token
but after I tried to change it to the main net ( solana config set --url https://api.mainnet-beta.solana.com )
and use the:  spl-token create-token
do not let me and gave me the same error 2

The error is because it is not able to find the new-keypair.json file which you have replaced.
To rectify change the solana config set --keypair ${HOME}/new-keypair.json to solana config set --keypair ${HOME}/my-keypair.json and then try to run spl-token create-token

@fanbyprinciple thanks for answering.
@Rafinix questions like this usually get answered more quickly in the Solana Discord's #developer-support channel.


Add Polish translation

Thanks for the PR! Would you mind to add some of the new texts? Here's an updated example in English
{
  "Azimuth": "Azimuth",
  "Dawn": "Dawn",
  "Dusk": "Dusk",
  "Elevation": "Elevation",
  "Noon": "Solar noon",
  "Sunrise": "Sunrise",
  "Sunset": "Sunset",
  "errors": {
    "SunIntegrationNotFound": "Sun integration not found"
  }
}

Done

That was quick! Thank you so much!


Do map operations under mutex in Firebase auth

Previously, the map of public keys in the Firebase authentication code were being concurrently accessed. This PR places both reads and writes behind a single mutex.
I found conflicting information (depending on Go version) about whether it was safe to concurrently read from a map. To be safe, I put the reads behind the mutex in addition to the writes.

This won't work you need to wrap ALL reads to any value in that struct in RLock() and RUnlock() and ALL writes to that struct in a Lock() and UnLock(). Also don't break up the struct update operation with two lock and unlocks use just one around the entire update operation including the unmashall.

Okay, I used RLock and RUnlock for the reads. I wasn't sure what you meant with breaking up the struct update operation into two locks and unlocks. I only see one place where I update/write to the struct which is in the Unmarshall.


parser: Issue parsing assignment of newly created pointer value in if clause

Go version
go version 1.23
Output of go env in your module/workspace:
GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/home/smu/.cache/go-build'
GOENV='/home/smu/.config/go/env'
GOEXE=''
GOEXPERIMENT='fieldtrack,boringcrypto'
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/home/smu/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/home/smu/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/lib/google-golang'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/lib/google-golang/pkg/tool/linux_amd64'
GOVCS=''
GOVERSION='go1.23 [redacted] X:fieldtrack,boringcrypto'
GODEBUG=''
GOTELEMETRY='local'
GOTELEMETRYDIR='/home/smu/.config/go/telemetry'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='clang'
CXX='clang++'
CGO_ENABLED='1'
GOMOD='/dev/null'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build1689118444=/tmp/go-build -gno-record-gcc-switches'

What did you do?
I expected
if mErr := &MyError{}; errors.As(err1, &mErr) {
...
}

to be parse correctly. See https://go.dev/play/p/XHkampOkowJ for a reproduction case.
Actual result of that playground snippet was:
./prog.go:23:10: syntax error: cannot use mErr := &MyError as value
./prog.go:26:1: syntax error: non-declaration statement outside function body
Changing the line
if mErr := &MyError{}; errors.As(err1, &mErr) {

into
if mErr := &(MyError{}); errors.As(err1, &mErr) {

or into
mErr := &MyError{}
if errors.As(err1, &mErr) {

makes the code work, but I would expect the first variant to work just as well.
What did you see happen?
./prog.go:23:10: syntax error: cannot use mErr := &MyError as value
./prog.go:26:1: syntax error: non-declaration statement outside function body
What did you expect to see?
A compiled program

Related Issues

builtin/error: late assignment of custom error type cause nil checking of the error variable not working #53768 (closed)
everything: fail to check err == nil when assigning explicit type to `error` #56930 (closed)
x/tools/gopls: Warn UnusedVar for unused assign #56598 (closed)
Compiler reports an error when trying to assign values from expression that includes function call #66529 (closed)
Nil-ness of a pointer is lost when converted to an interface #42306 (closed)
Can not compile this: if err := T{}.F(); err != nil{} #37043 (closed)
Function nesting calls a function that returns a pointer that implements the error, the returned error is always not equal to nil #43767 (closed)
Compile error: if-statement with assignment #35932 (closed)
cmd/compile: named return values produce functionally different code #21049 (closed)
errors: As can be confusing to use when given a pointer to a type that does not implement error, but a pointer to that type does #37625 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Probably the same as https://github.com/golang/go/issues/37043#issuecomment-582410977 ?

This is well known; see https://golang.org/ref/spec#Composite_literals:
A parsing ambiguity arises when a composite literal using the TypeName form of the LiteralType appears as an operand between the keyword and the opening brace of the block of an "if", "for", or "switch" statement, and the composite literal is not enclosed in parentheses, square brackets, or curly braces. In this rare case, the opening brace of the literal is erroneously parsed as the one introducing the block of statements. To resolve the ambiguity, the composite literal must appear within parentheses.

closing this

Reopening because I think we can produce a better error message.
package p

type S struct {
	F int
}

func F() {
	if s := S{}; s.F != 0 {
	}
}

The gc compiler says:
foo.go:8:7: syntax error: cannot use s := S as value
foo.go:10:1: syntax error: non-declaration statement outside function body

The gccgo compiler says
foo.go:8:17: error: parentheses required around this composite literal to avoid parsing ambiguity
    8 |         if s := S{}; s.F != 0 {
      |                 ^

CC @griesemer


Update copy on About page

Asana ticket

@littleforest my first guess is that the tailwind version we are using is different from the docs you are reading

my first guess is that the tailwind version we are using is different from the docs you are reading

We're on the latest tailwindrails-css version and it is using Tailwind 3.4.1 :( https://github.com/rails/tailwindcss-rails/commit/f989541ccfd8029fd0fdc967bb50ee6d62e8b51c

@FerPerales @jenlu33 please review and I will make a follow-up ticket re the Tailwind issues.


SFM4300-20-P Negative value

Hi
Thank you for creating library, for very easy use.
I have configured SFM4300-20-P with Arduino mega 2560 in which facing of issue flow rate over 12 or 13 l/m value becomes negative. also after overlapping range it becomes 0 (request for support showing final range limit).

Hi,
thank you for reporting this issue. It is due to an overflow when converting from the raw values to physical flow. We fixed the bug on the master branch.
Best, Léonie

Thank you so much as its resolved effectively and application runs successful


Subtract error

mybad

Javascript give me as a result (AKA -Inifnity) as string representation
But PHP gmp implementation of big number give me a correct result
The value is the same, seems.
So where is the difference? I see only the minus sign for js result.


We Cant sign up

Subject: We Cant sign up
Description:
Everytime we Go to sign up there is a Short Text that says They Send us an email but we didnt Get one
Results:
Backend version: develop-948446a4
Frontend version:
User Agent: Mozilla/5.0 (iPhone; CPU iPhone OS 15_6_1 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.6.1 Mobile/15E148 Safari/604.1
Page: https://couchers.org/signup
User:

dupe #3520


Implement the option to save simtel_array histograms to hdf5 tables and export them

I would like to adapt SimtelHistograms from simtel_histograms.py to allow saving the histograms to hdf5 files, as it is already done in CorsikaHistograms from corsika_histograms.py. This will standardise histograms produced by corsika and simtel_array within simtools. The advantage is to be able to use the same tool later to compare the histograms among different simulations.

By the way, this functionality is already partially in ctapipe and it's HDF5 DLx data format. I say partially, as we only currently write out the simulated events histograms (the ones we need to compute sensitivity later).  But it could easily be extended.

Hi @kosack, thanks for this suggestion. For now I am keeping it simple in simtools, using write_table from ctapipe.io.astropy_helpers to append all the histogram tables to the hdf5 file. I will further look into this to see if we can focus on the simulated events histograms instead of all histograms, in which case I could change to use the ctapipe data model as you mentioned.


Support for tokens

Hey,
I have programs that want access to authed endpoints. It would be good if you could generate tokens which can be set as a header (or maybe a cookie but I think that is less standard).
The header is usually called "Authorization" and the value is usually "Bearer <token>".
Tokens for authentication usually consist of:

Name. You usually want to know what the token is for.
Access restrictions. For Authelia that would probably be:

No user access or group access (used for authorisation only. Might not be useful currently?)
User access but no group access (used for generic user access which doesn't belong to a group. Maybe this one isn't useful?)
No user access but selected group access (used for restricting to a single application)
User access and all groups (this is really just the above bullet but you don't need to type in the group names. This is basically an "admin" token for the user)

Expiry. Sometimes you want temporary tokens, sometimes permanent tokens. You'd probably want to make it selectable between:

Custom end date
Forever

Revocation. You need to be able to revoke compromised tokens.

Note: This would require a front-end "management" interface for Authelia.
Note: Expired/revoked tokens are usually left in the UI with the "status" of expired/revoked so it is easy to see why a token has stopped working (and for audit purposes).
Cheers,
Ben

Hello @SerialVelocity ,
Thank you for the proposal! Does this spec comply with any known RFC, draft or any public document? Can you give me an example of program which requires those tokens?
Thanks,
Clément.

Just from a quick google search, seems like this may be a good place to start for some basic information about tokens:
https://scotch.io/tutorials/the-ins-and-outs-of-token-based-authentication
They recommend JWT tokens at the end but the problem with those tokens is that you can end up with extremely long tokens (which could go over default limits on header length). Your signing key could also be compromised requiring you to invalidate all tokens. Benefits of JWT tokens are that they are stateless. The server only has to verify the token according to its signature and check the revocation list to make sure it hasn't been revoked. Another problem is trying to add it to the revocation list since you have to know the full token or at least some token id otherwise the tokens will have to be stored on in the DB.
Another approach is randomly generated token strings. These are then stored in the database instead of the client and the database contains all information. These are easily revoked and can be updated automatically when the application is updated too (database migrations).
Using the "Authorization" header that I mentioned in the first post is the standard way of doing it while how the back-end deals with the token is usually up to the implementation. I would suggest the randomly generated token strings though as it is usually easier to deal with when implementing things like revocation, and dealing with old tokens as your DB migrations will upgrade them.
A few related RFCs:
https://tools.ietf.org/html/rfc1945#section-11
https://tools.ietf.org/html/rfc6750
You technically aren't OAuth 2, but I think Bearer tokens makes it very easy for users.

Hello,
My opinion around JWT tokens is a bit mitigated. Here is another interesting post I found a few months ago: http://cryto.net/~joepie91/blog/2016/06/13/stop-using-jwt-for-sessions/. The guy says JWT tokens are for transactions and not for sessions. I'm currently using the implementation of sessions for express. Those are encrypted and not stored on client side. Authelia could then invalidate one if needed but I have not felt the need so far and thus this is not implemented yet.
I guess OAuth would solve the problem, right? I was thinking of implementing it anyway.

I agree with JWT tokens aren't the way to go but I thought I'd give you some info about them anyway (that's why I suggested you should go down the randomly generated token strings route :) ).
OAuth would solve the problem but it is also much more complex to implement. It also needs to be supported in every application which means most open source apps now have to have oauth2_proxy proxying the connections.
Unless you mean just the token generation part of OAuth? If that's the case, it's usually best just to have randomly generated strings as I don't think you gain anything by adding the OAuth complexity.

@SerialVelocity , to (partly) solve your issue, you can configure your backend to be protected by single factor only and use your base64 encoded username and password in the service trying to authenticate (-H "Proxy-Authorization: Basic XXXXX" with curl).
This is not ideal but it can be used until we implement OAuth or app passwords/tokens.

Hello @SerialVelocity , OAuth2 is now in my shortlist and will therefore make Authelia support Bearer tokens. I propose we switch to the following thread related to the topic: https://github.com/clems4ever/authelia/issues/189.


Check spelling errors

I could take a look at this. Are there any specific files you're looking to target for finding spelling errors?

Hi @JoyceZhu, cool!
Please, take a look in README.md and dist/index.html.
This is index.hmtl


NSTOR4

Is this driver compatible with nstor4? does it support schedulers and shared volumes within a cluster?

this driver is for NStor5.x only, not compatible for NS4.

Ok. thanks.


create a ts file for the header - not logged in

Created a web component to handle case logged in and logged out


Update all dependencies

removes some of the obsolete packages

Codecov Report

Merging #526 (570574a) into master (1a3d75e) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #526   +/-   ##
======================================
  Coverage    6.66%   6.66%
======================================
  Files          13      13
  Lines          60      60
======================================
  Hits            4       4
  Misses         56      56

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 1a3d75e...570574a. Read the comment docs.


[ENH] Reorganise mock estimators in utils

this PR centralises some code with similar logic and purpose used in testing to the new testing package.
It does the following:

Moves a function "construct_dispatch" from its own file to the single place it is used (_reduce.py)
Combines the mock forecaster base and implementation files into a single file, _mock_forecasters dont think they need to be separate
Creates a new package in testing called mock_estimators, moving mock_forecasters there
Moves the testing classes in classification base class testing into a file mock_classifiers in the same directory.
Removes the now empty utils.estimators package

@ltsaprounis what do you think?


html from text replace

Failing to replace entity:

    \n
<    <
>    >
	    \t

for example:
<textarea id="text" name="text">hi!
<b>ali</b></textarea>

Good catch!

Time to fix this bug?

@javansolo @mervick could you explain this bug further please? I need to know the impact of it, and possibly fix it myself, thanks

@carpii use this for fix:
$(".selector").emojioneArea();
var text = $(".selector").eq(0).data("emojioneArea").getText().replace(/
/g, '\n').replace(/</g, '<').replace(/>/g, '>').replace(/	/g, '\t');
console.log(text);

@mervick - Thanks. I understand the original problem now.

Hello, you know that in applications such as facebook, twitter, whatsapp, users write some expressions quickly and manually. As in the following examples;
Expression: ":)", ":-)", ":-D", ":-d", "(:", ":-o", ":-O", ":-(", ":("
I want to be: ":grinning:", ":grinning:", ":smiley:", ":smile:", ":grinning:", ":hushed:", ":hushed:", ":disappointed:", ":disappointed:"
How do I change the above examples?

@viperaykiri your suggestion does not belong on this ticket, but its duplicate of #61 anyway


Missing Data to Duplicate the Experiments

It looks like I can't duplicate the experiments here because I'm missing the data. I'm specifically interested in inverse_problems_science. Could you please point me to this data - or tell me how to format the data in the expected format?
Thanks!

Q1: did you download the LSUN_bedroom dataset？？
A2: the format of data is convert source color image from RGB to YCrCb， and using CrCb  to represent  color ， the condition is Y and output is CrCb channel

You are right, there is no dataset from our collaborators that did the medical example (the person responsible has since left the group)
I will try to ask around and get this sorted!
I am moving this issue to the paper's own repository (this is now only for the FrEIA itself, which can implement arbitrary reversible architectures and training schemes, not just our method).


Login page not redirect to destination page if user logged in the sys…

I have just login as admin to my concrete5 store, And able to see "index.php/dashboard" . Now without logout , when i go to login page "index.php/login" it still allow me to login which should not allow.

Anyone have any thoughts on this?

We need to allow this so that themed login pages can be edited.
On Mon, Mar 28, 2016, 1:59 PM Andrew Embler<EMAIL_ADDRESS>wrote:

Anyone have any thoughts on this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/concrete5/concrete5/pull/3631#issuecomment-202509625

@avdevs why did you close this? I think it's a good idea to have something like that in place.


Added Arabic Book Names

Info.json - Added arabic chapter names.
infofixer.js - StructuredClone function doesn't exist before node 17. Added a workaround

This change is

Thanks, this will breaking changes for people who have already implemented this API, I will have to look for a better way to do this.

Yeah, makes sense, let me know if you finalize the design. I can tweak this implementation.

Thanks, this will breaking changes for people who have already implemented this API, I will have to look for a better way to do this.

Maybe instead of changing "info.json", changing the individual Arabic json files in https://github.com/fawazahmed0/hadith-api/tree/1/editions is a better option. So its also possible to add Turkish section names in Turkish json files, Urdu section names in Urdu json files etc...

Or a v2 of api can be made


fix: don't throw errors in constructor

With this PR errors from getUrlFromSession are resolved in the initialized Promise. A consumer of the library can access the promise by calling await client.initialize() to access those errors.
Also it handles a callbackUrl the same way as a verifyOtp, signUp, signInWith* and removes an existing session from storage before continuing.

hey @pixtron, thanks for the contribution! we're gonna merge this into @alaister's PR and rename some of it there - feel free to leave your feedback in the PR


Metrics for builder API

Consider following metrics for sequencer:

metrics inserting

insert payload last insertion success from builder timestamp (keep track when inserted w/ success)
insert payload last insertion fail from builder timestamp (keep track when failed to insert ⇒ alarms)

metrics fetching

last fetch payload with success with timestamp (keep track http all good)
last fetch payload error with timestamp (keep track http errors)

more logging

log application error code
log http error code

addressed in #3


Refactor/cleanup sample code

Change the sample code for the template app to a very simple to do list.
Move the integrations to a very simple plugin model, to make it easy to add/remove. Also make them not so hacky, but rather illustrate a better/cleaner approach.

Cleaned up enough for now. Refactored into a very basic plugin system. Changed package to changenode. Added a light/dark toggle.


Record params of execution as well

Rosbag recorder only records topics, not params. Therefore, at the end of the test launch, we need to have a rosparam dump for each run.
However, this dump should be parameterized, as it needs to save a new file for each test run.

88973a0
we only want to save the parameters that belong to the test. So we need to define the namespace. However, they will be saved in separate files.

Closed by https://github.com/ipa320/araig_test_stack/pull/35


feat: For developer/demonstrator, include way to run browser+node tests (with timeout)

https://www.npmjs.com/package/opn-cli


how to parse custom log format that contains a pipe as delimiter

my log format as follow:
03/Sep/2019:10:04:08 +0800 | <IP_ADDRESS> | http | 304 | <IP_ADDRESS> | - | www.hello.com | - | - | GET / HTTP/1.1 | 0 | - | Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.110 Safari/547.36 | - | 0.017 | - | - | -
Please can you help me with the right command ?
Thanks in advance.
Regards

Looking into this. Dup of #1303

This has been fixed. Feel free to build from development and let me know how it goes. It will be deployed in the upcoming version.


Migrate Service Operator with all service instances to another k8s cluster

Hi experts,
is there a possibility to move all service instances of an btp-operator installed on cluster A to another btp-operator installed on cluster B?
There should explicitly no new instances be created on cluster B, since there are already customers depending on the old service instances (for subscriptions and the like)
Might be related to #171

@kerenlahav Can you help here?

Currently this is not supported, can you please elaborate on the use case?
why is it needed?
will the same instance be used in more than one cluster?

Hi @kerenlahav
no, one instance will only be used on one cluster at the same time. We have a multi-cluster setup and want to migrate instances that were created on cluster A (where they don't belong to) to cluster B. We can't just recreate the instances since customers are already depending on it. Currently our only option is to copy the secrets from cluster A to cluster B.
The same would be needed if we would want to say migrate our infrastructure from AWS to Azure or similar.

it is not possible to move instances from cluster to cluster but some services can be shared between clusters.
you can find more information here:
https://cp-control-client-uc2.cfapps.sap.hana.ondemand.com/index.html#/knowledge_center/articles/44a5ed5eb4c145a2a89f9e6e1e5811b0


Update process/docs/release planner for new initial Beta arch board Info Meeting requirement (Committed for dilithium-1H)

Requirement:  All service teams planning to release new initial Beta data plane SDKs (client libraries) are required to schedule and complete Introduction meeting with the Azure SDK Architecture Board prior to release of their initial Beta SDKs.
### Tasks
- [ ] Update docs on engineering hub to indicate that new initial beta SDKs for data plane require Intro meeting. We need to confirm that this is when the namespace review kicks off and approved before SDK release.  This is what I have been told previously but Mariana has been participating in a v-team so may have updated guidance.
- [ ] Update the release planner to indicate that Intro meeting is scheduled for new initial Beta SDKs.   We can hide this step once we change the logic to identify new Beta and new Stable SDKs when the release plans are created - https://github.com/Azure/azure-sdk-tools/issues/7492.
- [ ] We need to find out from DPG (Renhe) if there are any additional prerequisites that are not in validated in the API Spec GitHub PR toolchain or the release pipeline that need to be confirmed before the DPG team releases the SDKs.  I don't know if there is logic running in the GitHub repos for languages (like the API Spec).

@ronniegeraghty do you know if this has happened?

Update docs on engineering hub to indicate that new initial beta SDKs for data plane require Intro meeting. We need to confirm that this is when the namespace review kicks off and approved before SDK release. This is what I have been told previously but Mariana has been participating in a v-team so may have updated guidance.

@ladonnaq could you clarify what does this mean?

We need to find out from DPG (Renhe) if there are any additional prerequisites that are not in validated in the API Spec GitHub PR toolchain or the release pipeline that need to be confirmed before the DPG team releases the SDKs. I don't know if there is logic running in the GitHub repos for languages (like the API Spec).

@ronniegeraghty do you know if this has happened?

Update docs on engineering hub to indicate that new initial beta SDKs for data plane require Intro meeting. We need to confirm that this is when the namespace review kicks off and approved before SDK release. This is what I have been told previously but Mariana has been participating in a v-team so may have updated guidance.

Made changer here: https://dev.azure.com/azure-sdk/internal/_git/azure-sdk-docs-eng.ms/pullrequest/1077


Feat/device service commands

New: c8y devices services
Managed services related to a device. The services concept was introduced in Cumulocity 10.14. Checkout the documentation for more details.
Managed device services (introduced in 10.14)

Usage:
  c8y devices services [command]

Available Commands:
  create      Create service
  delete      Delete service
  find        Find services
  get         Get service
  list        Get device services collection
  update      Update service status

Merging as failed tests are due to a problem with the testing environment


Add friction to the ball and pawns

Also, remove the speed property because we don't need it anymore. In any case it should just be the length of the velocity vector.

Thanks!


Add unit to 'charging_level_hv' sensor

Description:
This PR adds a unit to the 'charging_level_hv' sensor, which allows the UI to graph the values as a line graph rather than a bar graph of individual values.
Related issue (if applicable): fixes # n/a
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io# n/a
Example entry for configuration.yaml (if applicable):
n/a
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

Hi @timmccor,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Thanks!


Output placeholder image as data URI

Hello again!
I know I said I'd make a PR for how file extensions are handled, but needed this first :)
This PR outputs a placeholder image as a data uri to save network requests, addressing issue https://github.com/herrstucki/responsive-loader/issues/4.
The size of the placeholder image can be configured with a placeholderWidth option, but defaults to 40px wide.
@herrstucki If you have a moment, I would appreciate some extra eyes to ensure things make sense please. In particular this line, where I check if the output width equals placeholder with, then don't write the file but output as data uri instead.
The potential problem with this is if someone wants to actually write out a file at the placeholder width size.
A solution could be to add a flag to resizeImage() specifying whether the file should be written or not.
To Do:

[x] Update README to mention new feature

Nice, thanks!

I don't think it's a common case that someone resizes an image to placeholder width but it would be weird if that file got omitted without notice. So, size=40&placeholder should emit a file and a placeholder data uri. I probably wouldn't add the placeholder width to widthsToGenerate, but use a separate function for the placeholder (e.g. q.defer(resizePlaceholder, placeholderWidth) because we don't need to worry about filenames etc. anyway.
Will the placeholder use the same quality/background settings as the other images or should that be separate too? Maybe make the placeholder option an object which gets merged with default settings?
Maybe use placeholderSize (or placeholder.size if we use an object) as an option name instead of placeholderWidth because of consistency?
Do we need to add height information to the loader output, so the not-yet-loaded image can be scaled properly (this doesn't only apply to the placeholder)?

I just tried the suggested solution in the updated README, adding a base64 encoded small image to src and the original full image as srcset, and as far as I can see Chrome (53)  never displays the small image (tried with network shaping to get it to load slowly). Do you have any reference on this method?
I don't know if it's out of scope for this plugin to recommend how to actually use the placeholders, but it would be nice :)

Sorry, I realize I've been lagging very behind. I'll finish this before the week is up.
@oscar-b As for the placeholder image, you're right and the README needs changing.
Using the placeholder as src doesn't work with browsers that support srcSet, as that tag is never displayed. Instead you have to set it as the background image, and set backgroundSize: cover.
React.render(
  <img
      src={responsiveImage.src}
      srcSet={responsiveImage.srcSet}
      style={{backgroundImage: responsiveImage.placeholder; backgroundSize: cover}}
  />, el
);

@alex-ketch That's a nice way of doing it, never seen that before! I'm trying to figure out if it's feasible to use with responsive images though, since the background-size: cover won't behave the same as a regular image (at least without object-fit) and the tag easily ends up being 0x0 in size before loading.

That's why I think we should add width and height information to the loader output, so you can calculate the proper aspect ratio and set the dimensions on the image.

Yes that would be very useful.

@herrstucki So this is what I'm thinking:

[ ] Separate placeholder generation into separate function, outside of widthsToGenerate
[ ] Change setting to placeholderSize from placeholderWidth
[ ] Add height/width output as part of the output object

[ ] Should we add a convenience feature to set height/width/background automatically? Goal would be to make usage a bit more concise. Would need React & regular CSS output.

Also regarding:

Will the placeholder use the same quality/background settings as the other images or should that be separate too? Maybe make the placeholder option an object which gets merged with default settings?

I don't think full featured settings are necessary for the placeholder images, but open to be convinced otherwise. As those images are ~1KB at 40px wide, but if settings shave off significant bytes, then sure.

@alex-ketch Yeah, let's go for it! 👍 Very exciting. I was thinking of compiling some links to articles about different techniques of using placeholders too.
Actually, @lnhrdt has already added height to the output in #19, so just make sure the placeholder uses that too.
I probably wouldn't add more "convenience" output to the loader, since people may have different use cases (e.g. use the placeholder in canvas or svg). And how would that work in CSS?

@alex-ketch Do you have any eta for this PR? Eagerly awaiting it :)

@oscar-b Sorry for the delays, but I believe this should be it!
Would appreciate extra 👀

@alex-ketch I'm currently on vacation and won't be able to review/publish for at least a week. I've made you a collaborator in case you want to move more quickly 👍

I will try to test it in my environment asap, but looks good to me.

@alex-ketch This seems quite solid, I haven't had any problems using it at all. Nice work!

And fwiw the following works for ES6 style imports:
import topImage from 'responsive?placeholder=true,sizes[]=500w,sizes[]=1000w!./images/about-us.jpg';

@oscar-b Awesome! Thanks for battle-testing this @oscar-b.
I'll merge this if you're off for a little longer @herrstucki, otherwise I'll leave it for you.

@alex-ketch thank you so much for the outstanding work and @oscar-b for reviewing and battle-testing! I added two more tiny comments but otherwise I think it's good to go. I'll leave the honor of merging to you 😄
If you'd like I can also give you npm publish rights too (I'll need your username though).

Just a quick heads up; I'm having difficulties displaying the base64 encoded placeholder image as a background-image in Safari. Don't know why yet. Found some info that Safari doesn't display progressive JPEGs correctly when used as backgrounds, not sure how Jimp handles that (if the original is progressive, will the base64 also be progressive?).

It wasn't the progressive part, it was using background-image on a  which Safari doesn't seem to like. I refactored it a bit to display the placeholder on the container  instead, works beautifully now!

Thanks for bring this to a close @herrstucki, I'll update the Readme today to update mention not setting placeholder image as a background on an img tag and we should be good to go I think.

@herrstucki All clear to merge this?

@alex-ketch done and released in v0.7.0 ✌ Thanks for all the hard work!


Delete existing Staging AWS resources

Summary
Clean out the Staging AWS account resources with our AWS Nuke workflow to get things ready for the new infrastructure pattern we'll be using:
graph LR;
    WAF-->CloudFront;
    CloudFront-->Lambda;
    Lambda-->DynamoDB;

Staging has been cleaned up:


fix(theme): add unit to avoid deprecation warning

What is the current behavior?
Build throws warnings.

DEPRECATION WARNING: $saturation: Passing a number without unit % (0) is deprecated.

Issue Number
none
What is the new behavior?
No warning.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Specific Instructions
none
Other information
none

Thanks! @Rotzbua


Support subreddit custom CSS stylesheets

Is this related to issue #69?

Not sure anymore
On Sun, Jun 16, 2019, 3:30 AM Abdul Diaz<EMAIL_ADDRESS>wrote:

Is this related to issue #69
https://github.com/BenjaminAdams/RedditJS/issues/69?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BenjaminAdams/RedditJS/issues/73?email_source=notifications&email_token=AAC5K3GDUIIFOD3DZULK2V3P2X2YVA5CNFSM4CODIZ4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXZIDGY#issuecomment-502432155,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC5K3GTAMBJYAEE5A7CUVTP2X2YVANCNFSM4CODIZ4A
.


add trust wallet extention

What is the purpose of the change

ClickUp Task
ClickUp Task URL
Brief Changelog

Testing and Verifying

Documentation and Release Note

Replaced by #2774


Help with code on Nucleo-F410RB

I'm trying to use your code with a Nucleo-F410RB (100MHz).  I just have one strip of 3 LEDs so far, to verify that things work.  I have the LEDs directly connected to PB3 and most everything ported fairly easily.  The LEDs are all white after init and while running.  I have an oscilloscope on the output pin and the frequency and duty cycle looks good.  The only place WS2812_RESET_PERIOD shows up is in HAL_TIM_PeriodElapsedCallback(), but it's commented.  Any ideas?
BTW, I tried reaching out on your website, github, and Twitter, but couldn't find an email address and Twitter DMs are off.

The code has some commented parts because it was developed over time for F1, F3, F0, L0.. see my other repos.
The macro is commented because the 50 us reset pulse is handled diferently in this F4 port.
When the last LED data is sent the timer is stopped, reconfigured to 50us and run again. Also this 50us period is calculated dynamically.
https://github.com/hubmartin/WS2812B_STM32F4/blob/master/Src/ws2812b/ws2812b.c#L359

@splinepd Hey, I am also trying to run this on my Nucleo-F410RB but couldn't seem to get it to work. I have previously ran this on F407 discovery board and Nucleo-F302R8 board without any problems. I am using CubeMX to configure my board. Not sure why it is not working. Do you think you can share your project with me? Any help would be much appreciated.

Sorry, I haven't had a chance to get back into this.


NestedClassesVisibility(False negative): Nested class doesn't inherit visibility from parent

From the comment,
NonCompliant
internal class NestedClassesVisibility {
   public class NestedPublicClass // should not be public
}

Compliant
 internal class NestedClassesVisibility {
     internal class NestedPublicClass
 }

In actuality, both should be Non Compliant.
This is because both are equivalent as the nested class inherits its visibility from its parent class.
So effectively, the public class in the initial example is also internal.
The way it's written, just marking an already "internal" class "internal" makes the violation go away.

Sorry for handling this issue so late. This fell completely under our radar.

This is because both are equivalent as the nested class inherits its visibility from its parent class.
So effectively, the public class in the initial example is also internal.

Could you please provide a source for this statement?
Please also keep in mind that there is a reason for having this rule in the style ruleset.

If I have following code in detekt-api:
internal class A {

    class B
}

and try to use it in detekt-cli like
fun main() {
    A.B()
}

the compiler won't let you, so the visibility is inherited.
So this rule ensures that something like
class A {

    internal class B
}

is used.

The main issue with this rule is that it forces the use of an additional unnecessary modifier.
internal class A {

    class B

}

is non-compliant with the current implementation of the rule. The visibility of class B is already internal. The false positive forces the use of an additional internal like so -
internal class A {

    internal class B

}

which is pretty cumbersome.
Also, because the inner classes inherit their visibility from the parent class, inner classes cannot have visibility greater than their parent class.
So, afaik -
internal class A {

	public class B

}

class B still has internal visibility.

Thanks for the explanation, now I got your point.
I agree we should not flag no modifier at the nested class. I would still flag the public modifier to not confuse the user, maybe even with a new message.

That would work for me. :)

Is this rule even useful?

Nested classes are often used to implement functionality local to the class it is nested in. Therefore it should not be public to other parts of the code. Prefer keeping nested classes private.

What I understand from that is that any inner class should be private. That's huge, and doesn't allow you to model your public API. For that reason I think that it's only applied to internal classes.
But even with the internal classes it allow nested internal classes. So what this rule forbids is the public classes inside intenal classes... But those don't exist because the visibility is inherited.
So, in my opinion we should:

Remove this rule.
Force that ANY class/object/enum/interface (except companion object) inside an internal class should be private.
Change the description (and probably the name) of the rule to point that it flags the missleading modifiers. And we should check the private classes too.


Allow other content types in a request with multipart/form-data

This change removes a restriction that prevents other request content types to be used when multipart/form-data is one of them.  After initial testing it seems that modelerfour produces the expected code model output that will enable a generator to produce operation methods for the different content types.

@ShivangiReja @pakrym If one of you have a moment, could you try the test build above in the previous comment to see if the Computer Vision spec works correctly for you now?

@daviwil Computer Vision spec works correctly using https://github.com/Azure/autorest.modelerfour/pull/351#issuecomment-696899710 build.


Drop Mixpanel

Feature Request
Is your feature request related to a problem? Please describe:
The default quota of Mixpanel can be easily exceeded in our usage scenario. We need to investigate and find other solutions.

Describe the feature you'd like:
Replace Mixpanel with other services. Maybe Google Analytics can be suitable.

Describe alternatives you've considered:
Pay for Mixpanel. Not a best option.
Teachability, Documentation, Adoption, Migration Strategy:

Dropped.


Decide how to handle posts with community labels

Posts with community labels are embedded/rendered as if there was no warning whatsoever. We could implement a default "spolier" for such posts (with an optional unroll parameter), or we could keep it as-is and let users decide whether to spoiler the link individually (in which case, they could just post the usual Tumblr link instead - it gets embedded as a profile with no post info).

Probably just going to keep it as-is for now.


Pretrained previous work models

Could you please make available all the models that were used to generate the results shown in the paper (Diat, IcGAN, Cycle-GAN)?

Sorry, but I have no plan to release the code for baseline models. You can use official implementations.


Cannot read property 'uri' of null fix

There is a case that throw TypeError: Cannot read property 'uri' of null when I try to construct and remove the monaco editor in short amount of time in an web app. The setModelMarkers is in call stack already while the model is already removed.

Hi @inspiredjw, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Jeong Woo Chang). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Coverage remained the same at 61.235% when pulling de65f6911d318361030cd719a84791ed44484f8c on inspiredjw:patch-1 into 61328fd66c7bff7aad4e61f37e18373569c19768 on Microsoft:master.

Thank you for the contribution!


Add Danny to group and create pull request for Helge

Add danny's initials to our group on Helge's repo

Pull Request made pull request


How to create materialized view?

There's the createView() method I can use in migrations, but there's no option to make it a materialized view. How can I create one inside a migration?

Seems like adding an option would be the simplest:
https://github.com/romeerez/orchid-orm/blob/c9a3f8adf970c964d7d7bd60022b6f60edf0cc06/packages/rake-db/src/migration/createView.ts#L48-L50

We probably also need a way to refresh the view in js code, without the need to use raw sql

I'll need to learn this topic and see what's needed besides the MATERIALIZED keyword.

We probably also need a way to refresh the view in js code

Right, I see there is a REFRESH SQL statement, it should be supported.

Some time has passed, and I changed my view (pun intended) on managing views in orchid.
Instead of writing a migration with createView() I'm using raw SQL in the migrations/recurrent directory and it works beautifully. I use CREATE OR REPLACE since it runs on every migration.
Such a great feature!

Cool, I finally got here to answer that I'm glad you found a way with recurring migrations and raw SQL.
When something is missing in the ORM, it's generally possible to bypass it with raw SQL, and eventually I'll cover what's missing with a TypeScript DSL.
In this case, it would be ideal to write your views (materialized or not) closer to the app logic, and to be able to rely on view types, and the migration generator should learn how to overwrite views in the recurring directory.


Type Script error creating networks

Creating edges is throwing a typescript error.
If I have something like this:
let edgesArray = [
 { from: 1, to: 3 },
 { from: 1, to: 2 },
 { from: 2, to: 4 },
 { from: 3, to: 5 },
 ]

Then do:
let data = {
  nodes: new vis.DataSet(nodesArray),
  edges: new vis.DataSet(edgesArray)
}

I get the following error:
No overload matches this call.
  Overload 1 of 2, '(options?: DataSetInitialOptions<"id">): DataSet<Partial<Record<"id", string | number>>, "id">', gave the following error.
    Type '{ from: number; to: number; } []' has no properties in common with type 'DataSetInitialOptions<"id">'.
  Overload 2 of 2, '(data: Partial<Record<"id", string | number>>[], options?: DataSetInitialOptions<"id">): DataSet<Partial<Record<"id", string | number>>, "id">', gave the following error.
    Argument of type '{ from: number; to: number; } []' is not assignable to parameter of type

The only way I seem to get around this is if I create my edge array using a loop.
If I repurpose your exampleUtil.js getScaleFreeNetwork function and change the return statement to use vis.DataSet(arrays) everything works fine.
I've also created a StackOverflow question for this error but it hasn't picked up traction
https://stackoverflow.com/questions/58015656/problem-setting-edges-in-vis-network-with-typescript
Is this a bug with the types or am I just doing something wrong? I'm going against the examples and I'm using vis-network version 5.2.4

There's no issue with Vis Network or Vis Data Set. I'll answer this on StackOverflow.


Update Controller to use ErrorConstants value from Core

Updated AndcultureCode.CSharp.Core -> 0.7.0, update namespace changes, swap ERROR_RESOURCE_NOT_FOUND with error constant from core
Closes #14

[x] Related GitHub issue(s) linked in PR description
[x] Destination branch merged, built and tested with your changes
[x] Code formatted and follows best practices and patterns
[x] Code builds cleanly (no additional warnings or errors)
[-] Manually tested
[x] Automated tests are passing
[x] No decreases in automated test coverage
[-] Documentation updated (readme, docs, comments, etc.)
[x] Localization: No hard-coded error messages in code files (minimally in string constants)

@all-contributors add @brandongregoryscott for code


[Robot] Adding unstable tag to fix and investigate as this test is going to f…

One test in manage households will fail due to element change for accessibility as per the PR- https://github.com/SalesforceFoundation/NPSP/pull/5782
Moving it unstable for now
Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

old one not needed any more


Test Harness: Security Scheme Validation

User Story
As a Prism CLI user, I want to secure my endpoints, so that I know the API is only used by authorized users.
ACs
add test cases to test harness
add at least the cases described in the examples section
if any of the test fails then mark that it is an invalid result (e.g. mark as skipped) and report a Jira bug
link that bug to this card
Make sure to add the actual tests to TestRails
Additional resources
https://stoplightio.atlassian.net/wiki/spaces/EN/pages/52887637/Prism#Prism-MockServerFeatures (please find relevant row)
When operations have security schemes, how do they respond when no authentication information is being sent?
Try and write tests for every single authentication scheme in OpenAPI 2/3, and write tests for every different location those schemes security information can be in.
API Key in header
API key in query
API key in cookie (ugh why)
HTTP in Header
HTTP in query…
etc
Try all OpenAPI v2 specs against Prism v2 to get a general idea for how it is meant to work, then try that stuff in Prism v3.
Acceptance Criteria and Scope
Prism should throw errors if security criteria are not matched
If Prism does not respond the way it should (based on v2 or common sense) create a story for investigation and move on
Example tests
Given a spec has no global security scheme
And an operation has no security requirements defined
When I send a request to this operation
Then I get a response with 200 status code
Given a spec has a global security scheme
And an operation has no security requirements defined
When I send a request to this operation
And I don't include any credentials in the request
Then I get back a response with 401 status code
And the response should have WWW-Authenticate header
Given a spec has multiple global security schemes
And an operation has no security requirements defined
When I send a request to this operation
And I don't include any credentials in the request
Then I get back a response with 401 status code
And the response should have WWW-Authenticate header
Given a spec has multiple global security schemes
And an operation has no security requirements defined
When I send a request to this operation
And I include valid credentials for one of the global security schemes
Then I get back a response with 200 status code
Given a spec has multiple global security schemes
And an operation has no security requirements defined
When I send a request to this operation
And I include invalid credentials for one of the global security schemes
Then I get back a response with 403 status code
Given a spec has one or multiple global security schemes
And an operation defines its security requirements by specifying one of the security schemes
And I send a request to this operation
And I include invalid credentials for this operation security requirements
Then I get back a response with 403 status code
Given a spec has one or multiple global security schemes
And an operation overrides the global security scheme by overwriting them (making the operation not secured, having its security requirements as [])
And I send a request to this operation
And I don't include any credentials in the request
Then I get back a response with 200 status code

@lag-of-death keep in mind Prism does not have these capabilities at all right now, so all these tests will fail.

Probably best to copy these into the actual story for security validation like I did with CORS so it can be done as part of the implementation. That way we don’t need two tickets for one feature. :)

This work is being done in #484.


[FLINK-26336] Call cancel on deletion & clean up configmaps as well

Seperating the original commit from https://github.com/apache/flink-kubernetes-operator/pull/26 to not interfere with the refactor discussion.

I discovered that we need to wait for cluster shutdown before deleting the configmaps otherwise they might be recreated. Adding this now

I have addressed the comments @wangyang0918 @tweise , let me know if there is anything else.
I had to improve the waitForClusterShutdown logic to be able to create a safe point where we can delete the configmaps without the jobmanager recreating them.

@gyfora I'm going to run through some of the scenarios manually to verify. Hopefully I can add the missing tests in the next days.


Implemented password hashing using a salt, following this code&guidel…

I implemented password hashing following the guidelines on this site: https://crackstation.net/hashing-security.htm
I tested it using MySQL and H2 and it works on both databases.
There are some things that might be worth reviewing (plus any other you can think of :) )

It is not the same idea the password that comes from a web request that the password (hash) that is stored in the database. Yet, given the current JSON conversions, I was a bit stucked in reading the input password directly from the User object (since that one is loaded from the web request).

That seems a bit inconsistent, but the request body reader cannot be reused to recreate another json to read the password. Maybe we can extend the current request to create the associated json object, and store it through the life of the request. In this way we could parse different objects from the same request if needed, or retrieve a particular property.

The Hashing class is no longer necessary but it does have a handy method to hash a String. Should be keep it just in case or erase it since it is no longer used?

Can we have another field hashedPassword or something like that to avoid passing password separately to data manager methods? Then setPassword method can set both plain text password and hashedPassword fields. In the database we would save only hashedPassword. Do you see any problems with this approach?

I don't like the PasswordHash class because it uses different license. It doesn't really have much logic. I think it would be good to re-implement hashing with salt in original Hashing helper class.

Everything else looks good.

I changed to work like that (I pushed the change so you can see it).
There two things to notice about it:
a) I changed the serialization so a property can be marked to be ignored in the serialization (for the hashed password and salt). There is the obvious problem that we don't want to transmit the hashed password + salt through the network (since all privacy and secrecy would be lost :) ), and because the frontend seem to post back those values too. And when that happens, it sometimes overrides the logic in the setPassword, since the setHashedPassword is executed after setPassword (messing up everything).

b) To answer your question, I would say that the disadvantage is that not always all actions matches directly to an object in the model. And perhaps you sometimes need to load different objects that might not be "persistent models".
In this case it seems to me that we are adding a password field to the model just to solve the a serialization issue. But since the getter has some logic I think is a corner case (that is way I implemented it :) ). So maybe we can wait to a more problematic case to make a bigger change.

I thought the licences were compatible so I took the code since it seemed to be tested and debugged already. But if that is a problem we can try to rewrite those parts. It might take a bit of time since opening the blackbox will require me to learn a bit more about hashing :)

OK, let's leave the class for now. Thanks for the pull request.


How about the license of radb project ?

Dear Author,
The radb project is very useful for me and I have a quest about the license. Is it a MIT license ?
Thanks

It's Apache 2 Licensed.
I'll add a license ASAP :)


$createUser uid is too simple - prevents using uid as a key in firebase

I noticed the uid for Facebook and Google are unique enough where it would be difficult to guess, but simplelogin:1 simplelogin:2 are very easy to guess even though they are unique.  I was hoping to use uid as a way to refer to users, but it introduces security issues if the uid's are too easy to guess.
I'm saving the authData returned to create a user record, but given the ease of guessing a uid, what is a better option?  Use $push and have firebase create keys that way?
Open to suggestions, but a bit bummed I may not be able to use uid directly.

Henry, thanks for reaching out. $createUser() simply wraps the SDK method, so this isn't a complaint with AngularFire, but rather with the Firebase SDK. I'll ping the auth team and add your thoughts here to the feature discussion.

I was hoping to use uid as a way to refer to users, but it introduces security issues if the uids are too easy to guess.

This is not true. There are no security issues with a guessable uid. The uids for Facebook, Twitter, etc. are also guessable since they are just <provider>:<providerId> where <providerId> is your public Facebook ID, Twitter ID, etc.
Firebase authentication will generate a Firebase authentication token on our servers which is signed with your secret. The payload of this token is what is put in the auth variable in your Security Rules. The only way to generate this token is to first authenticate with the proper credentials. There is no way to spoof the Firebase authentication token client-side even if you know the user's uid. In order to do that, you would need the Firebase secret. But if you have that, then you already have the keys to the kingdom.
What you need is our user-based security rules. You should actually probably give the whole security guide a read to understand how Firebase security works. For example, you can make it so that they any authenticated user can read other user's data but only write to their own user data with rules like these:
{
  "rules": {
    "users": {
      ".read": "auth !== null",
      "$uid": {
        ".write": "auth.uid === $uid",
      }
    }
  }
}

Ultimately, there is nothing to pass on to the auth team here since there are no security issues around this issue. When dealing with user sign-on, it is important to differentiate authentication vs identification. Just because it is easy to guess identification does not mean it is easy to spoof authentication. That is what is happening in this case.

Here's a potential scenario which wouldn't be all that uncommon.  Assume uid is used through an application to retrieve a user's profile.  If the uid is passed as a parameter to retrieve a user's detail - it's possible for another "authenticated" user to change a URL to retrieve another user's detail assuming a user can read another user's data.  The Firebase example shows saving authData as the user's data using the uid as the key in the collection which implies using that key (since it's unique) is quite acceptable.
I appreciate the tutorial on firebase security rules, but this is really about being able to simply increment the simplelogin:# uid into a valid uid for another user.  All keys are "guessable" but so are all passwords (eventually).

Security rules should be used to secure data access, not obscurity. If the user has read access to the data, then reading it should not be a problem. You can certainly prevent them from writing to another profile.

It's certainly a matter of opinion but I can think of many cases where obscurity is just fine - for example an order # from an online store or a tracking # from UPS.  Hey, why don't credit cards just increment their numbers if they have a security key on the back?
Having a "unique" key that's basically a prefix followed by an incremented number is poor design choice as a generated key when so many other options exist which are just as easy, yet much more random.  So an email authentication returns a uid, but it's simple generation makes it a poor match to use a user key for an application due it's simplicity.
Obscurity, like anything security related, is used with other mechanisms like 2-factor authentication or physical keys to make it less easy for a system to be compromised.
Anyways - I was offering it as an enhancement suggestion.  Ignore if you want.  It's poor design, but that's just one person's opinion :-)

Thanks again for the feedback. As I mentioned before, I've put your vote on the list for discussion with the team. We certainly appreciate your thoughts here. It's devs that take the time to offer feedback--like you!--who make the Firebase community so amazing and fun.


Move to extension-ci-tools, enable distribution of nightly versions

Include https://github.com/duckdb/substrait/pull/108, plus fixes to build system.
Note that while building there are a few warnings connected to targets like:
Makefile:20: warning: overriding commands for target `pull'
extension-ci-tools/makefiles/duckdb_extension.Makefile:122: warning: ignoring old commands for target `pull'
Makefile:43: warning: overriding commands for target `test_release'
extension-ci-tools/makefiles/duckdb_extension.Makefile:82: warning: ignoring old commands for target `test_release'
Makefile:46: warning: overriding commands for target `test_debug'
extension-ci-tools/makefiles/duckdb_extension.Makefile:85: warning: ignoring old commands for target `test_debug'
Makefile:63: warning: overriding commands for target `format'
extension-ci-tools/makefiles/duckdb_extension.Makefile:115: warning: ignoring old commands for target `format'
Makefile:69: warning: overriding commands for target `update'
extension-ci-tools/makefiles/duckdb_extension.Makefile:119: warning: ignoring old commands for target `update'

Those might need to double checked, relevant code is likely already in extension-ci-tools, but I am not completely sure.

I think this should sort of work, although it might need some minor touches.

Thanks for picking this up!
I think the SQL tests are not running?
No test cases matched '/Users/runner/work/substrait/substrait/test/*'

@pdet: nice catch, up to the next

LGTM! Thanks Carlo!!

@carlopi I just gave this a go and got
con.execute("FORCE INSTALL substrait FROM core_nightly")
duckdb.duckdb.HTTPException: HTTP Error: Failed to download extension "substrait" at URL "http://nightly-extensions.duckdb.org/v1.1.0/osx_arm64/substrait.duckdb_extension.gz" (HTTP 403)
Extension "substrait" is an existing extension.

Yes, I had seen it earlier, I think https://github.com/duckdb/substrait/pull/110 solves this.

I think it worked:
FORCE INSTALL substrait from core_nightly;
LOAD substrait;
SELECT extension_name, extension_version, install_mode, installed_from FROM duckdb_extensions() WHERE extension_name = 'substrait';

┌────────────────┬───────────────────┬──────────────┬────────────────┐
│ extension_name │ extension_version │ install_mode │ installed_from │
│    varchar     │      varchar      │   varchar    │    varchar     │
├────────────────┼───────────────────┼──────────────┼────────────────┤
│ substrait      │ d8ca65b           │ REPOSITORY   │ core_nightly   │
└────────────────┴───────────────────┴──────────────┴────────────────┘


unix,win: fix timeout calculation in repeating timers

While callback takes long time to run, the next timeout of repeating timer is calculated by adding interval to the time that callback ends, which leads to delayed timeout.
So we should add interval to triggering time instead.

It seems like my commit breaks user called uv_timer_again().
I didn't notice it coz I have not used this function in my program.
I'll try to fix it asap.

ok, fixed.
this fix also stripped dup calls to uv_timer_stop() in timer repeats

redo the patch to fix timer re-adding

@bnoordhuis this issue is really hard to explain, it occurs in my production environment while services have been running for a long time.
Let me explain it in some detailed way:

I needed a timer to run every exact hour(aka XX:00:00) so that I wrote a repeating uv_timer with 3600 seconds as repeat and (now/3600)*3600+3600-now as first interval
I found the timer is not triggered at the exact time as expected after had running for some days, it became triggered at XX:00:01 then later XX:00:02.
So I had to diagnostic into libuv to check what was wrong with repeating timers, and found the next timepoint of repeating timer is added from current time, not last trigger time, which caused tiny gaps sumed up to seconds for timer delays.
This commit tried to resolve this problem to ensure repeating timers are triggers in exact interval despite of tiny execution time of timer scheduling and uv loop codes.

@bnoordhuis fixed

I'm not sure I like this change. I also don't like repeating timers.
People have built applications based on this expectations, which we'd be changing.
If your delay in the callback exceeds the duration of the timer then the next timer will be delayed anyway, so there is no silver bullet here.
@libuv/collaborators thoughts on removing repeating timers on v2? I could write a LEP and the implementation.

@saghul I agree with you, this is a potential bug.
So remove it is okay, but leaving a funtion like uv_timer_again with new interval supplied to save cost of a new uv_timer would be better.
Keeping repeating timer and note the potential bug in the document is another choice.

This seems to have stalled out and now has conflicts. I'm going to close this out for now.


chore(main): release 5.6.8

:robot: I have created a release beep boop
5.6.8 (2023-12-09)
Minor changes and bug fixes

update mediaplex to latest (78b5d85)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mariusbegby/cadence-discord-bot/releases/tag/v5.6.8 :sunflower:


Migrate VCs storage in old contracts

Description
Fixes #202
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my change

Tested in testnet, everything ok :rocket: