Binding should munch exceptions (IMHO)

This is testing with 10.5 so apologies if treated already (didn't see an issue/PR).  If ng:show's value is bound to the result of a function, and that function throws an exception, then errors appear in JS console.
It seems that having exceptions tranformed into null, false, or empty string (depending on binding location) should be best behaviour.

eating exceptions is never a good idea, but if you want to do that, just override the $exceptionHandler class with noop in the configuration

closing.

Was mostly thinking of cases where it's bound to data produced by a for-loop, and that for-loop is iterating through an array that is still being fetched from server.  I can always pre-initialize arrays, so that's what I'll do if $exceptionHandler seems hacky in my code base.
Thanks anyway, guess my brain is still stuck on Flex a bit :).

ng:repeat over undefinden should not throw exceptions. so that should not
be a problem.
2011/12/10 ludicast <
<EMAIL_ADDRESS>

Was mostly thinking of cases where it's bound to data produced by a
for-loop, and that for-loop is iterating through an array that is still
being fetched from server.  I can always pre-initialize arrays, so that's
what I'll do if $exceptionHandler seems hacky in my code base.
Thanks anyway, guess my brain is still stuck on Flex a bit :).

Reply to this email directly or view it on GitHub:
https://github.com/angular/angular.js/issues/688#issuecomment-3091858

In this instance, just using a for loop in JS to provide summation data for a view (to determine whether something would be shown).  But was easy enough to do one of these: (this.myArray || []) so that's what I found up doing.

I hear you on the lazy part, but swallowing exceptions is dangerous.


EntityFramework package (not core), has bad frameworkassembly values

From @rrelyea on August 8, 2016 21:5
Steps to reproduce
originally filed as: https://github.com/NuGet/Home/issues/3279
Repro Steps:

Create a new project C#-> Windows 8->Windows Runtime Component (Windows 8.1).
Right-click the project node and click "Manage NuGet Packages…" menu to start package manager.
Install package “EntityFramework”, and verify it can be installed.

Expected Result:
The package “EntityFramework” can be installed correctly.
Actual Result:
The package “EntityFramework” can’t be installed as below screenshot.

Note:
It happened on Dev14 U3+NuGet Dev 1731(<IP_ADDRESS>1).
It repro on any version of package “EntityFramework”.
It also repro on Dev14 U3 (NuGet <IP_ADDRESS>1).
The issue
NuGet sees
<FrameworkAssembly ... TargetFramework="" />
TargetFramework should not be empty, but should be Net40 or Net45.
When you don't specify a framework, nuget thinks this works in all TFMs. Thus the package seemed to install on this Win8.1 project, while it should fail.
Further technical details
None.
Copied from original issue: aspnet/EntityFramework#6272

EF 6 is not supported on Windows Runtime (WinRT) Components, I believe....

Expected Result:
The package “EntityFramework” can be installed correctly.

This will never work for EF6, so the fix we can do is to make sure the package won't be listed for platforms that it doesn't support.

So I tried this before a long time ago, but I couldn't remember why it didn't work. Looking into it again, it's because we have content (for App.config). There is no way to have framework-specific content, so the package will always install on any framework no matter what restrictions we put on lib, tools, dependencies, or frameworkAssemblies.

So we can either

Fix the targetFramework attribute so the error goes away and it "successfully" installs (with nothing in your project) on any framework.
Do nothing so you'll continue to get an obscure error on other frameworks.

Plague or cholera, as we say in Danish

@bricelam does this help? https://github.com/NuGet/Home/issues/2969#issuecomment-225932621

Oh weird, I couldn't find that anywhere in the documentation. I'll give it a try.

It worked. And it's documented. 🐑


consul load balancer

Dapr registers with Consul. After the same appid, one service takes the place of the first service
Expected Behavior
load balancer
Actual Behavior
Dapr registers with Consul. After the same appid, one service takes the place of the first service  #1199
dapr version 1.5
return &consul.AgentServiceRegistration{
		Name:    appID,
		Address: host,
		Port:    appPortInt,
		Checks:  cfg.Checks,
		Tags:    cfg.Tags,
		Meta:    cfg.Meta,
	}, nil

Steps to Reproduce the Problem
Release Note

The same problem was found


[Question]: Stations and Devices no longer available

@bropat Any idea why my station and devices are no longer available? The container stopped reporting they were available yesterday without any obvious changes on my end.
Steps I've taken:

I've verified the stations and devices are working
I've restarted all stations and devices
I've logged into the website and mobile app under my main account and my DEV account that the container uses and verified stations/devices are available
I've tried bropat/eufy-security-ws versions 0.8.4, 0.9.2, 0.9.4, and latest
I replicated the login (/passport/login), profile (/passport/profile), and device list (/house/device_list) and they all returned expected responses (devices in the device list as well)

Appears the response is missing the "data".

UPDATE:
I did another test by deleting the persistent.json file:

Again, I took the request data from the screenshot, added my password, and successfully received a response that had the expected data (user_id, etc).

@bropat Just saw this closed issue: https://github.com/bropat/eufy-security-ws/issues/152
That fixed my problem. Thanks!


Update for latest icinga2 master brunch

Graphite schema changed with recent commit
https://github.com/Icinga/icinga2/commit/b10cb8a2e7acccc285377f72fef533ab277f2ad1#diff-fd8634adc3e7216fc4ecb345caba5cab
example config should be changed to something like this:
host_name_template = "$host.name$.host.$host.check_command$.perfdata"
service_name_template = "$host.name$.services.$service.name$.$service.check_command$.perfdata"
graphite_args_template = "&target=$target$.value&source=0&width=300&height=120&hideAxes=true&lineWidth=2&hideLegend=true&colorList=049BAF"
graphite_large_args_template = "&target=alias(color($target$.warn,'yellow'),'warning')&target=alias(color($target$.crit,'red'),'critical')&target=$target$.value&source=0&width=800&height=700&c
also, escape function should be updated too:
'-' not replaced now and '::' should be replaced with '_'

@olfway a pull request would be cool if you have the time,


this package work in laravel 5.6?

I would like to know if this package works on version 5.6 of laravel, if it doesn't work would you have any to recommend me?

No, this won't work with 5.6.


OutOfMemoryError: Requested array size exceeds VM limit

OrientDB version : 2.0-M3
JDK : Oracle JDK 1.7.0_71
OS : CentOS 6.5
The below Exception came only once while starting our server application after crash. So, OrientDB was not shutdown properly.
java.lang.OutOfMemoryError: Requested array size exceeds VM limit
Dumping heap to /cygnet/heapdump/ ...
Unable to create /cygnet/heapdump/: Is a directory
Exception in thread "pool-1-thread-36" java.lang.OutOfMemoryError: Requested array size exceeds VM limit
        at java.util.Arrays.copyOf(Arrays.java:2219)
        at java.util.ArrayList.grow(ArrayList.java:242)
        at java.util.ArrayList.ensureExplicitCapacity(ArrayList.java:216)
        at java.util.ArrayList.ensureCapacityInternal(ArrayList.java:208)
        at java.util.ArrayList.add(ArrayList.java:440)
        at com.orientechnologies.orient.core.index.sbtree.local.OSBTree.findBucket(OSBTree.java:1514)
        at com.orientechnologies.orient.core.index.sbtree.local.OSBTree.get(OSBTree.java:188)
        at com.orientechnologies.orient.core.index.engine.OSBTreeIndexEngine.get(OSBTreeIndexEngine.java:226)
        at com.orientechnologies.orient.core.index.OIndexOneValue.get(OIndexOneValue.java:57)
        at com.orientechnologies.orient.core.index.OIndexOneValue.get(OIndexOneValue.java:44)
        at com.orientechnologies.orient.core.index.OIndexAbstractDelegate.get(OIndexAbstractDelegate.java:59)
        at com.orientechnologies.orient.core.index.OIndexTxAwareOneValue.get(OIndexTxAwareOneValue.java:233)
        at com.orientechnologies.orient.core.index.OIndexTxAwareOneValue.get(OIndexTxAwareOneValue.java:41)
        at com.orientechnologies.orient.core.sql.operator.OQueryOperatorEquals.executeIndexQuery(OQueryOperatorEquals.java:129)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLSelect.searchForIndexes(OCommandExecutorSQLSelect.java:1448)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLSelect.searchInClasses(OCommandExecutorSQLSelect.java:713)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLResultsetAbstract.assignTarget(OCommandExecutorSQLResultsetAbstract.java:184)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLSelect.assignTarget(OCommandExecutorSQLSelect.java:414)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLSelect.executeSearch(OCommandExecutorSQLSelect.java:396)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLSelect.execute(OCommandExecutorSQLSelect.java:367)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLDelegate.execute(OCommandExecutorSQLDelegate.java:64)
        at com.orientechnologies.orient.core.storage.OStorageEmbedded.executeCommand(OStorageEmbedded.java:86)
        at com.orientechnologies.orient.core.storage.OStorageEmbedded.command(OStorageEmbedded.java:75)
        at com.orientechnologies.orient.core.sql.query.OSQLQuery.run(OSQLQuery.java:70)
        at com.orientechnologies.orient.core.sql.query.OSQLSynchQuery.run(OSQLSynchQuery.java:84)
        at com.orientechnologies.orient.core.query.OQueryAbstract.execute(OQueryAbstract.java:33)
        at com.orientechnologies.orient.core.metadata.security.OSecurityShared.getUser(OSecurityShared.java:447)
        at com.orientechnologies.orient.core.metadata.security.OSecurityShared.getUser(OSecurityShared.java:181)
        at com.orientechnologies.orient.core.metadata.security.OSecurityShared.authenticate(OSecurityShared.java:157)
        at com.orientechnologies.orient.core.metadata.security.OSecurityProxy.authenticate(OSecurityProxy.java:86)
        at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.open(ODatabaseDocumentTx.java:254)
        at com.orientechnologies.orient.core.db.OPartitionedDatabasePool.acquire(OPartitionedDatabasePool.java:192)

But, many times when OrientDB was not shutdown properly it used to rebuild the indices while restarting our application. This Exception has not come those times.
It occurred only once. So, I don't feel it is reproducible from my side.
Pls tell me If I have to give any other information

It is the same index corruption which we already discussed.

No. It occurred in a new server machine. Not anything that was reported earlier

Is it reproduced when disk cache according to settings consumes all available RAM ?

Happens also with 2.0 final?

We discussed the reason where data read issues happened, typically it is happened when setting are set to use borders of available ram so in reality we go into swap.

Is it still valid ?

We did not see this after the reported instance....!
It may be closed now. We will request reopening when we see it again

Cool, I'm closing it.


Setting files are not revealed in explorer

Azure Data Studio Version:
Version: 1.12.0-insider (user setup)
Commit: 2d9f6dcd862227825050eb3182553c3a176e0ce9
Date: 2019-09-26T07:52:29.030Z
VS Code: 1.38.0
Electron: 6.0.9
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.18362

Steps to Reproduce:

Open Keyboard Shortcuts (JSON)
Press F1
Search for Reveal in Explorer command
Execute (press Enter key)

Other files are correctly revealed in explorer, but the settings are not. Same thing happens for snippets.
IIRC, previous versions do allow this feature.

This issue didn't repro for me on the current release.  Please let us know if you're still impacted by this issue.


Improve Accuracy

Hello
How can we improve accuracy of object detection ?

https://github.com/ultralytics/yolov5/wiki/Tips-for-Best-Training-Results


Materials on ordnance geometry crash the game

I have a very strong hunch this is related to #34 as the stock version works fine, but yeah this is a thing nonetheless. A thermal detonator with SP materials always crashes the game just as it's going to explode, which makes me think lvl_pack (assuming it's the one in charge of wrapping it all up) is not packing the explosion classes in. As I believe #34 was caused by the animation banks not being included, it would stand to reason this is the same kind of problem but of course I'll let you be the judge of that.

Interesting. Do BFront2.log or shader patch.log hold any clues?

I didn't catch any at the time it happened, but now I think I've been trolled. I can't replicate the crash anymore, which is stupid because all I did was clean the munged files and take out the material from the sides folder, then munge without it - now I wanted to double check the logs so I re-munged the material, and apparently now there's nothing wrong with the weapon. It's not like I changed anything in the material since the beginning so I haven't the slightest clue why it didn't work the first time. Maybe BF2 just hates me.

Update. While this crash never resurfaced again, there's a little bit of an interesting thing that popped up in the log just now. As it seems, the material of the ordnance model was loaded, but after that there's a "Destroyed material" message regarding it. Is this supposed to happen? If it means the material is released because there was nothing using it, I guess it makes sense... but of course an ordnance model can't be there to use it all the time so this could be a logic issue, I suppose?

Material lifetime should be controlled by the game. Is it possible an .lvl or child lvl being loaded after that one is also loading "cool_ordnance_material" not as a material but as a texture? That could cause it to get seemingly mysteriously destroyed and replaced like this.

Oh. Well, yeah, that's definitely happening but I thought it was fine since the game's standard order of operations is load first instance of an asset -> discard the subsequent ones. I suppose there's one more test I can do to find out.

Usually that is what happens but textures (which is what materials are loaded as from the game's perspective) are different. For whatever reason they decided to do it the other way for them, discard the previously loaded texture and load the new one.

After a little bit of sorting, I've now successfully gotten the ordnance material working in game. I don't think this crash is going to happen anymore.


Fix wrong comment

Codecov Report
Patch and project coverage have no change.

Comparison is base (61ec852) 97.89% compared to head (0bcd9e7) 97.89%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage   97.89%   97.89%
=======================================
  Files           8        8
  Lines         428      428
=======================================
  Hits          419      419
  Misses          5        5
  Partials        4        4

Impacted Files
Coverage Δ

deepcopy.go
100.00% <ø> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Eslint: add rules that do not error

The following rules do not error on our code and are part of the AirBnB style guide. This PR turns them on.
prefer-const, no-const-assign, quote-props, no-new-object, no-array-constructor, template-curly-spacing, func-style, wrap-iife, no-new-func, prefer-spread
The commit also includes rules set to warning for GCI in https://github.com/zulip/zulip/pull/2568, I can split it up if necessary.

Automated message from Dropbox CLA bot
@arpith, it looks like you've already signed the Dropbox CLA. Thanks!

Thanks @showell!


Add clock functions

Does the build fail without CONFIG_POSIX_CLOCK?
If so, we could add a cargo feature. I'm doing this for userspace and RUST_ALLOC_POOL by converting from environment variables to cargo features in rust/zephyr-core/build.rs

Made the change.


dart2js -> closure compiler, Parse error. missing name after . operator

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

convert dart code to js using dart2js
run the closure compiler on resulting js file
deploy/App.js:29475: ERROR - Parse error. missing name after . operator
return this.default;

What is the expected output? What do you see instead?
This might be a closure bug, or something I'm doing wrong, I don't know.  The line causing the error is in:
html_dart2js.dart, _TrackElementImpl class, defaultValue method.
I'm not explicitly using a TrackElement in my project.
What version of the product are you using? On what operating system?
Dart SDK version 13387
Please provide any additional information below.

I believe this is now working properly - I haven't tried myself, but users have reported using jscompiler on the dart2js output without issues.


[BUG] Fairness constraints cause bogus warning about Spec not in canonical form

The example below has been fixed. For a follow up, see https://github.com/informalsystems/apalache/issues/468#issuecomment-853259723
markus@avocado:~/Desktop/ewd998$ apalache check --config=EWD998.cfg EWD998.tla
# Tool home: /home/markus/src/TLA/_community/apalache
# Package:   /home/markus/src/TLA/_community/apalache/mod-distribution/target/apalache-pkg-0.8.2-SNAPSHOT-full.jar
# JVM args: -Xmx4096m -DTLA-Library=/home/markus/src/TLA/_community/apalache/src/tla
#
# APALACHE version 0.8.2-SNAPSHOT build v0.7.2-163-gde4505d
#
# WARNING: This tool is in the experimental stage.
#          Please report bugs at: [https://github.com/informalsystems/apalache/issues]
#
# Usage statistics is ON. Thank you!
# If you have changed your mind, disable the statistics with config --enable-stats=false.

Checker options: filename=EWD998.tla, init=, next=, inv=          I@11:41:19.936
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.google.inject.internal.cglib.core.$ReflectUtils$1 (file:/home/markus/src/TLA/_community/apalache/mod-distribution/target/apalache-pkg-0.8.2-SNAPSHOT-full.jar) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain)
WARNING: Please consider reporting this to the maintainers of com.google.inject.internal.cglib.core.$ReflectUtils$1
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
PASS #0: SanyParser                                               I@11:41:20.958
Parsing file /home/markus/Desktop/ewd998/EWD998.tla
Parsing file /tmp/Integers.tla
Parsing file /tmp/FiniteSets.tla
Parsing file /tmp/Naturals.tla
Parsing file /tmp/Sequences.tla
PASS #1: ConfigurationPass                                        I@11:41:22.111
  > EWD998.cfg: Loading TLC configuration                         I@11:41:22.115
Operator Spec of 0 arguments is defined as: (Init() ∧ ☐([Next()]_vars())) ∧ (WF_vars()(System())) E@11:41:22.300
Configuration error (see the manual): EWD998.cfg: Expected Spec to be in the canonical form Init /\ [][Next]_vars /\ ... E@11:41:22.302
It took me 0 days  0 hours  0 min  2 sec                          I@11:41:22.304
Total time: 2.374 sec                                             I@11:41:22.304
EXITCODE: ERROR (99)

------------------------------- MODULE EWD998 -------------------------------
EXTENDS Integers, FiniteSets

CONSTANT N
ASSUME NAssumption == N \in Nat \ {0} \* At least one node.

Nodes == 0 .. N-1
Color == {"white", "black"}
Token == [pos : Nodes, q : Int, color : Color]

\* igor: this is the apalache idiom to deal with dynamic ranges
rng(a, b) == { i \in 0..N: a <= i /\ i <= b }

\* old apalache type annotations, will change soon
a <: b == a
\* a type specification for the recursive function reduced
reducedT == [Int -> Int]

MyReduce(op(_,_), fun, from, to, base) ==
  (**************************************************************************)
  (* Reduce the elements in the range from..to of the function's co-domain. *)
  (**************************************************************************)
  LET reduced[i \in rng(from, to)] ==
          IF i = from THEN op(base, fun[i])
          ELSE op((reduced <: reducedT)[i - 1], fun[i])
  IN reduced[to]

VARIABLES
 active,     \* activation status of nodes
 color,      \* color of nodes
 counter,    \* nb of sent messages - nb of rcvd messages per node
 pending,    \* nb of messages in transit to node
 token       \* token structure

vars == <<active, color, counter, pending, token>>

TypeOK ==
  /\ active \in [Nodes -> BOOLEAN]
  /\ color \in [Nodes -> Color]
  /\ counter \in [Nodes -> Int]
  /\ pending \in [Nodes -> Nat]
  \* igor: instead of a membership over an infinite set, use Skolemization
  /\ \E p \in Nodes, q \in Int, c \in Color:
    token = [pos |-> p, q |-> q, color |-> c]
  \*/\ token \in Token
------------------------------------------------------------------------------

Init ==
  (* EWD840 but nodes *)
  /\ active \in [Nodes -> BOOLEAN]
  /\ color \in [Nodes -> Color]
  (* Rule 0 *)
  /\ counter = [i \in Nodes |-> 0] \* c properly initialized
  /\ pending = [i \in Nodes |-> 0]
  /\ token = [pos |-> 0, q |-> 0, color |-> "black"]

InitiateProbe ==
  (* Rules 1 + 5 + 6 *)
  /\ token.pos = 0
  /\ \* previous round not conclusive if:
     \/ token.color = "black"
     \/ color[0] = "black"
     \/ counter[0] + token.q > 0
  /\ token' = [pos |-> N-1, q |-> 0, color |-> "white"]
  /\ color' = [ color EXCEPT ![0] = "white" ]
  \* The state of the nodes remains unchanged by token-related actions.
  /\ UNCHANGED <<active, counter, pending>>

PassToken(i) ==
  (* Rules 2 + 4 + 7 *)
  /\ ~ active[i] \* If machine i is active, keep the token.
  /\ token.pos = i
  /\ token' = [token EXCEPT !.pos = @ - 1,
                            !.q = @ + counter[i],
                            !.color = IF color[i] = "black" THEN "black" ELSE @]
  /\ color' = [ color EXCEPT ![i] = "white" ]
  \* The state of the nodes remains unchanged by token-related actions.
  /\ UNCHANGED <<active, counter, pending>>

System == \/ InitiateProbe
          \/ \E i \in Nodes \ {0} : PassToken(i)

-----------------------------------------------------------------------------

SendMsg(i) ==
  \* Only allowed to send msgs if node i is active.
  /\ active[i]
  (* Rule 0 *)
  /\ counter' = [counter EXCEPT ![i] = @ + 1]
  \* Non-deterministically choose a receiver node.
  /\ \E j \in Nodes \ {i} : pending' = [pending EXCEPT ![j] = @ + 1]
          \* Note that we don't blacken node i as in EWD840 if node i
          \* sends a message to node j with j > i
  /\ UNCHANGED <<active, color, token>>

RecvMsg(i) ==
  /\ pending[i] > 0
  /\ pending' = [pending EXCEPT ![i] = @ - 1]
  (* Rule 0 *)
  /\ counter' = [counter EXCEPT ![i] = @ - 1]
  (* Rule 3 *)
  /\ color' = [ color EXCEPT ![i] = "black" ]
  \* Receipt of a message activates i.
  /\ active' = [ active EXCEPT ![i] = TRUE ]
  /\ UNCHANGED <<token>>

Deactivate(i) ==
  /\ active[i]
  /\ active' = [active EXCEPT ![i] = FALSE]
  /\ UNCHANGED <<color, counter, pending, token>>

Environment == \E i \in Nodes : SendMsg(i) \/ RecvMsg(i) \/ Deactivate(i)

-----------------------------------------------------------------------------

Next ==
  System \/ Environment

Spec == Init /\ [][Next]_vars /\ WF_vars(System)  \* <== FAIRNESS
=======

SPECIFICATION Spec
CONSTANT N=3

Thanks! I will fix that.

Thanks! I will fix that.

Another example of the same problem:
apalache check crdt.tla

gives me
PASS #1: ConfigurationPass                                        I@16:59:52.550
  > crdt.cfg: Loading TLC configuration                           I@16:59:52.554
Operator Spec of 0 arguments is defined as: (Init() ∧ ☐([Next()]_vars())) ∧ (WF_vars()(DeliverOnNode())) E@16:59:52.663
Configuration error (see the manual): crdt.cfg: Expected Spec to be in the canonical form Init /\ [][Next]_vars /\ ... E@16:59:52.666

crdt.tla
---- MODULE crdt ----
EXTENDS TLC, FiniteSets, Naturals, Sequences
CONSTANTS MAX_TIMESTAMP, KEYS, VALUES, N_NODES
VARIABLES timestamp, values, deliverQueues

vars == <<timestamp, values, deliverQueues>>
nodeIds == 1..N_NODES

DeliverSet(n, t, k, v) ==
  LET previous == { <<tp, kp, vp>> \in values[n]: kp = k }  IN
  IF previous = {} \/ \A <<tp, kp, vp>> \in previous : tp < t THEN
    values' = [ values EXCEPT ![n] = (values[n] \ previous) \union {<<t, k, v>>} ]
  ELSE
    UNCHANGED values

DeliverDelete(n, t) ==
  values' = [values EXCEPT ![n] = {<<tp, k, v>> \in values[n] : tp /= t }]

Deliver(n, command, payload) ==
  \/ command = "set"
    /\ DeliverSet(n, payload[1], payload[2], payload[3])
  \/ command = "delete"
    /\ DeliverDelete(n, payload)

Broadcast(n, command, payload) ==
  /\ Deliver(n, command, payload)
  /\ deliverQueues' = [
      i \in nodeIds |->
        IF i = n THEN
          deliverQueues[i]
        ELSE
          Append(deliverQueues[i], <<command, payload>>)
      ]

RequestSet(n, k, v) ==
  /\ timestamp' = timestamp + 1
  /\ Broadcast(n, "set", <<timestamp, k, v>>)

RequestDelete(n, k) ==
  \E <<t, kp, v>> \in values[n] :
    /\ kp = k
    /\ Broadcast(n, "delete", t)

RequestSetOnNode ==
  /\ timestamp < MAX_TIMESTAMP
  /\ \E <<n, k, v>> \in nodeIds \X KEYS \X VALUES : RequestSet(n, k, v)

RequestDeleteOnNode ==
  /\ \E <<n, k>> \in nodeIds \X KEYS : RequestDelete(n, k)
  /\ UNCHANGED timestamp

DeliverOnNode ==
  \E n \in nodeIds :
    /\ Len(deliverQueues[n]) > 0
    /\ \E <<command, payload>> \in {Head(deliverQueues[n])} :
        Deliver(n, command, payload)
    /\ deliverQueues' = [deliverQueues EXCEPT ![n] = Tail(deliverQueues[n])]
  /\ UNCHANGED timestamp

DeliverQueuesIsEmpty ==
  \A n \in nodeIds: Len(deliverQueues[n]) = 0

Terminating ==
  /\ DeliverQueuesIsEmpty
  /\ UNCHANGED vars

Init ==
  /\ values = [i \in nodeIds |-> {}]
  /\ deliverQueues = [i \in nodeIds |-> <<>>]
  /\ timestamp = 1

Next ==
  \/ RequestSetOnNode
  \/ RequestDeleteOnNode
  \/ DeliverOnNode
  \/ Terminating

Spec == Init /\ [][Next]_vars /\ WF_vars(DeliverOnNode)

AllValuesEqual ==
  \A <<n1, n2>> \in nodeIds \X nodeIds :
    values[n1] = values[n2]
EventuallyConsistent == <>[]AllValuesEqual
====

crdt.cfg
SPECIFICATION Spec

CONSTANTS
    MAX_TIMESTAMP = 3
    KEYS = {key}
    VALUES = {value}
    N_NODES = 2

PROPERTIES
    EventuallyConsistent

Another example of the same problem:
apalache check crdt.tla

gives me
PASS #1: ConfigurationPass                                        I@16:59:52.550
  > crdt.cfg: Loading TLC configuration                           I@16:59:52.554
Operator Spec of 0 arguments is defined as: (Init() ∧ ☐([Next()]_vars())) ∧ (WF_vars()(DeliverOnNode())) E@16:59:52.663
Configuration error (see the manual): crdt.cfg: Expected Spec to be in the canonical form Init /\ [][Next]_vars /\ ... E@16:59:52.666

crdt.tla
---- MODULE crdt ----
EXTENDS TLC, FiniteSets, Naturals, Sequences
CONSTANTS MAX_TIMESTAMP, KEYS, VALUES, N_NODES
VARIABLES timestamp, values, deliverQueues

vars == <<timestamp, values, deliverQueues>>
nodeIds == 1..N_NODES

DeliverSet(n, t, k, v) ==
  LET previous == { <<tp, kp, vp>> \in values[n]: kp = k }  IN
  IF previous = {} \/ \A <<tp, kp, vp>> \in previous : tp < t THEN
    values' = [ values EXCEPT ![n] = (values[n] \ previous) \union {<<t, k, v>>} ]
  ELSE
    UNCHANGED values

DeliverDelete(n, t) ==
  values' = [values EXCEPT ![n] = {<<tp, k, v>> \in values[n] : tp /= t }]

Deliver(n, command, payload) ==
  \/ command = "set"
    /\ DeliverSet(n, payload[1], payload[2], payload[3])
  \/ command = "delete"
    /\ DeliverDelete(n, payload)

Broadcast(n, command, payload) ==
  /\ Deliver(n, command, payload)
  /\ deliverQueues' = [
      i \in nodeIds |->
        IF i = n THEN
          deliverQueues[i]
        ELSE
          Append(deliverQueues[i], <<command, payload>>)
      ]

RequestSet(n, k, v) ==
  /\ timestamp' = timestamp + 1
  /\ Broadcast(n, "set", <<timestamp, k, v>>)

RequestDelete(n, k) ==
  \E <<t, kp, v>> \in values[n] :
    /\ kp = k
    /\ Broadcast(n, "delete", t)

RequestSetOnNode ==
  /\ timestamp < MAX_TIMESTAMP
  /\ \E <<n, k, v>> \in nodeIds \X KEYS \X VALUES : RequestSet(n, k, v)

RequestDeleteOnNode ==
  /\ \E <<n, k>> \in nodeIds \X KEYS : RequestDelete(n, k)
  /\ UNCHANGED timestamp

DeliverOnNode ==
  \E n \in nodeIds :
    /\ Len(deliverQueues[n]) > 0
    /\ \E <<command, payload>> \in {Head(deliverQueues[n])} :
        Deliver(n, command, payload)
    /\ deliverQueues' = [deliverQueues EXCEPT ![n] = Tail(deliverQueues[n])]
  /\ UNCHANGED timestamp

DeliverQueuesIsEmpty ==
  \A n \in nodeIds: Len(deliverQueues[n]) = 0

Terminating ==
  /\ DeliverQueuesIsEmpty
  /\ UNCHANGED vars

Init ==
  /\ values = [i \in nodeIds |-> {}]
  /\ deliverQueues = [i \in nodeIds |-> <<>>]
  /\ timestamp = 1

Next ==
  \/ RequestSetOnNode
  \/ RequestDeleteOnNode
  \/ DeliverOnNode
  \/ Terminating

Spec == Init /\ [][Next]_vars /\ WF_vars(DeliverOnNode)

AllValuesEqual ==
  \A <<n1, n2>> \in nodeIds \X nodeIds :
    values[n1] = values[n2]
EventuallyConsistent == <>[]AllValuesEqual
====

crdt.cfg
SPECIFICATION Spec

CONSTANTS
    MAX_TIMESTAMP = 3
    KEYS = {key}
    VALUES = {value}
    N_NODES = 2

PROPERTIES
    EventuallyConsistent

We are going to fix several bugs for the specs mentioned here and cut a new release on Monday

We are going to fix several bugs for the specs mentioned here and cut a new release on Monday

This is fixed now and will be included in the release (today). @lemmy, @Alexander-N, if you have further problems with your specs, open an issue or we could chat on zulip.

This is fixed now and will be included in the release (today). @lemmy, @Alexander-N, if you have further problems with your specs, open an issue or we could chat on zulip.

The current fix is too narrow:
---- MODULE M ----
EXTENDS Integers

VARIABLE
    \* @type: Int;
    x

Init ==
    x = 0

Next ==
    x' \in 0..42

\* No problem if  F  is in-lined.
F ==
    WF_x(Next)

\* Fails with Configuration error (see the manual): M.cfg: Expected Spec to be in the canonical form Init /\ [][Next]_vars /\ ... E@10:51:44.946
SpecF1== Init /\ [][Next]_x /\ F   \* This is not uncommon
SpecF2 == x = 0 /\ [][Next]_x /\ WF_x(Next)
SpecF3 == Init /\ [][x' \in 0..42]_x /\ WF_x(Next)

\* Works
SpecW1 == Init /\ [][Next]_x /\ WF_x(Next)
SpecW2 == Init /\ [][Next]_x /\ WF_x(x' \in 0..42)  \* Compare SpecF3
====

Btw. it would be nice if --config=SomeConfigFile.cfg could be --config SomeConfigFile.cfg to make the shell's tab-completion work.

Yeah, you are right. We should fix the spec. As for the --config, we are using an option parser that works that way. Still looking for a good replacement :(

This specification
Spec == /\ Init /\ [][Next]_vars
        /\ \A self \in servers : WF_vars(server(self))

Produced by translaction of this PlusCal algorithm
-------------------------- MODULE mod --------------------------
CONSTANT
    \* @type: Int;
    defaultInitValue,
    \* @type: Set(SERVER);
    servers
VARIABLES
    \* @type: SERVER -> Str;
    pc,
    \* @type: Int;
    s

Inv == TRUE
(*--algorithm main
variables s;
fair process server \in servers
begin
    Start:
    s := 1;
end process;
end algorithm; *)
=====================================================================

Also gives the same error when checked with config mod.cfg
SPECIFICATION Spec
CONSTANT
defaultInitValue = 0
servers = { "s1_OF_SERVER", "s2_OF_SERVER" }
INVARIANT Inv

apalache-mc check --config=mod.cfg mod.tla


[flutter_tool] flutter_tools depends on devtools_shared 0.9.7+3 which doesn't match any versions

Steps to Reproduce

git clone https://github.com/flutter/flutter
cd flutter && git checkout -b reproduce 2c5c50948686fb698842317c3f5d2bf54c6090ca
bin/flutter doctor -v

Logs

Logs

bin/flutter doctor -v

Downloading Dart SDK from Flutter engine b793775d2a01e358f7cb3d5eebe2d5e329751b5b...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  201M  100  201M    0     0  9083k      0  0:00:22  0:00:22 --:--:--  9.9M
Building flutter tool...
Because flutter_tools depends on devtools_shared 0.9.7+3 which doesn't match any versions, version solving failed.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (9 tries left)
Because flutter_tools depends on devtools_shared 0.9.7+3 which doesn't match any versions, version solving failed.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (8 tries left)
Because flutter_tools depends on devtools_shared 0.9.7+3 which doesn't match any versions, version solving failed.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (7 tries left)
Because flutter_tools depends on devtools_shared 0.9.7+3 which doesn't match any versions, version solving failed.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (6 tries left)
Because flutter_tools depends on devtools_shared 0.9.7+3 which doesn't match any versions, version solving failed.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (5 tries left)
Because flutter_tools depends on devtools_shared 0.9.7+3 which doesn't match any versions, version solving failed.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (4 tries left)
Because flutter_tools depends on devtools_shared 0.9.7+3 which doesn't match any versions, version solving failed.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (3 tries left)
Because flutter_tools depends on devtools_shared 0.9.7+3 which doesn't match any versions, version solving failed.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (2 tries left)
Because flutter_tools depends on devtools_shared 0.9.7+3 which doesn't match any versions, version solving failed.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (1 tries left)
Command 'pub upgrade' still failed after 10 tries, giving up.

I also encounter the problem recently, it seem to be the issue with flutter-io.cn, if changed to the Tsinghua source, downloads should be fine.
export FLUTTER_STORAGE_BASE_URL="https://mirrors.tuna.tsinghua.edu.cn/flutter"
export PUB_HOSTED_URL="https://mirrors.tuna.tsinghua.edu.cn/dart-pub"

Confirmed. Both https://pub.dartlang.org and https://mirrors.tuna.tsinghua.edu.cn/dart-pub work.
Filed another issue https://github.com/dart-lang/pub-dev/issues/4540


Verify that s3fs works correctly if you delete the credentials file after starting the process

Can confirm that after mounting and deleting the credentials the mount remains unaffected


Support for defining several columns using the same source

Support for cases where several columns with different types can be created from the same source.
Example:
- create_date:
    :source: created_at
    :type:  DATE
- created_at:
    :source: created_at
    :type:  TIMESTAMP

Can I ask you to add a test case for this new behavior? Otherwise, it seems reasonable.

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

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

Never signed it, and do not have plans to...


Nyx-testing hangs after update  on latest kAFL kernel

After kAFL 6.0 upgrade https://github.com/IntelLabs/kAFL/pull/178, Nyx-testing hangs on our kAFL runner

Does this issue still occur with the latest commit of Nyx-Testing?


Update sp-adddistributor-transact-sql.md

Replace deprecated system view

@MashaMSFT

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks @0x7FFFFFFFFFFFFFFF ! #sign-off

#sign-off


Accidental breaking change? tests started failing when going from 1.16 to 1.17

https://github.com/NullVoxPopuli/emberclear/pull/660/checks?check_run_id=556759684
Looks like it relates to disabled state.

Ya, thanks for the report!
it was introduced by https://github.com/san650/ember-cli-page-object/commit/873e1cda4a12a7642439a943bc4d7d755aec3ae9#diff-0dc38de04449789b2e0f2463693cb62cR88
prior to the change we've used $(element).attr(attributeName), and now switched to element.getAttribute(. It turns out jQuery normalizes attributes. So we have to restore to use it for attribute.
Also I've noticed the multiple deprecation, which doesn't look legit. Just haven't had time to look into it yet.
sorry for inconveniences.


python312Packages.pandera: init at 0.20.4

Init python312Packages.pandera, a Python library for Pydantic-style typing and validation of dataframes.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

Tests fail on Darwin
=========================== short test summary info ============================
FAILED tests/geopandas/test_engine.py::test_engine_geometry_coerce_crs[data_container0] - pyproj.exceptions.ProjError: Error creating Transformer from CRS.: (Interna...
FAILED tests/geopandas/test_engine.py::test_engine_geometry_coerce_crs[data_container1] - pyproj.exceptions.ProjError: Error creating Transformer from CRS.: (Interna...
FAILED tests/geopandas/test_engine.py::test_engine_geometry_coerce_crs[data_container2] - pyproj.exceptions.ProjError: Error creating Transformer from CRS.: (Interna...
FAILED tests/geopandas/test_engine.py::test_engine_geometry_coerce_crs[data_container3] - pyproj.exceptions.ProjError: Error creating Transformer from CRS.: (Interna...
FAILED tests/geopandas/test_geopandas.py::test_schema_dtype_crs_with_coerce[gdf_args2-False] - pandera.errors.SchemaError: Error while coercing 'geometry' to type geometr...
= 5 failed, 3058 passed, 40 skipped, 6 xfailed, 6896 warnings in 152.70s (0:02:32) =

Please fix or disable these tests, if the error is not major.

Tests fail on Darwin
=========================== short test summary info ============================
FAILED tests/geopandas/test_engine.py::test_engine_geometry_coerce_crs[data_container0] - pyproj.exceptions.ProjError: Error creating Transformer from CRS.: (Interna...
FAILED tests/geopandas/test_engine.py::test_engine_geometry_coerce_crs[data_container1] - pyproj.exceptions.ProjError: Error creating Transformer from CRS.: (Interna...
FAILED tests/geopandas/test_engine.py::test_engine_geometry_coerce_crs[data_container2] - pyproj.exceptions.ProjError: Error creating Transformer from CRS.: (Interna...
FAILED tests/geopandas/test_engine.py::test_engine_geometry_coerce_crs[data_container3] - pyproj.exceptions.ProjError: Error creating Transformer from CRS.: (Interna...
FAILED tests/geopandas/test_geopandas.py::test_schema_dtype_crs_with_coerce[gdf_args2-False] - pandera.errors.SchemaError: Error while coercing 'geometry' to type geometr...
= 5 failed, 3058 passed, 40 skipped, 6 xfailed, 6896 warnings in 152.70s (0:02:32) =

Please fix or disable these tests, if the error is not major.

Thanks, disabled the tests.

Looks good to me. Unfortunately I can't test as the dependency triton-llvm doesn't build at the moment #363965


FEAT: Add Yi-9B

Fixes #1114
add Yi-9B

Please rebase to the latest main code to run the CI.

The deletion of gemma should be reverted also.


PathStyleExtension in lonboard

As noted in #473 my use case is plotting link volumes on NZ roads. One "feature" of the road geometries (which come from OpenStreetMap) is that lane geometries lie on top of each other. This doesn't look good in the visualisation so we want to split paths with some offset. One fix seems to be the deck.gl PathStyleExtension with the offset:true option. I was wondering if there is a way to include this in a lonboard map even though it is not available in the current set?
new deck.PathLayer({
      widthMinPixels: 1,
      capRounded: true,
      data: pathData,
      getPath: (d) => d.path,
      getColor: (d) => d.color,
      getWidth: (d) => d.width,
      getOffset: -0.5,
      extensions: [
        new deck.PathStyleExtension({
          offset: true
        })
      ],
      parameters: {
        depthTest: false
      }
    })

Yes it definitely makes sense to add that to lonboard.
Do you think you'd be able to try to implement it? It should largely be straightforward if you look at how the other extensions are implemented

Yep, I can definitely give it a go :-)

Closed by https://github.com/developmentseed/lonboard/pull/487


Add feature state, feature flag off login and captureSidebar

To toggle a feature back on temporarily, you can use Redux dev tools to fire off the appropriate toggle event:


Evaluate and reduce INFO logging where practical from the UI

There seems to be excessive INFO message writen to the EVM logs contributing
to logging related bugs.
An evaluation and reduction of INFO logging where practical needs to be done.

@miq-bot add_label ui "technical debt"

@miq-bot add_label ui

@jvlcek do you have a couple examples of log messages that clutter the logs?

I do have example logs that I can share privately. What I did notice is that within 1 second a lot of duplicate messages are being reported. For example:
file % grep 03T17:14:42 evm.log | wc
104    2753   34585
file % grep 03T17:14:41 evm.log | wc
64    1992   22098
It appears to be related to C&U on vmware.

@jvlcek need a bit more to go on here, if you can . . . Thx, Dan

@jvlcek is this issue still relevant. If so, please remove the "stale" label. Otherwise, please close.


cups-config: only CUPS libraries go in non-static cups-config --libs

We're including compression and crypto libraries in the linker flags
output, even when we're built as a shared library. This doesn't make
sense, because callers of cups-config are only looking for their direct
dependencies. If they plan to make any SSL or zlib calls of their own,
they should be managing those flags on their own.
This resolves problems seen in package management, where libcups users
end up inadvertently linking against (e.g.) gnutls directly, without the
package manager being aware. (And thus, it doesn't know to rebuild on
library upgrades.)

We've had problems with this in the past; will re-investigate but I know on some platforms doing this doesn't actually work... :/

Any tips on what platform(s)? I don't believe any other *-config tool (e.g., pkg-config) works the way cups-config currently does.

AIX, HP-UX, IRIX, macOS and some versions of Solaris for sure.

do those include static linking ?  i can understand on static-only systems needing the extra linkage.

No, those are all operating systems that support shared libraries. They just don't always support recursive linkage (i.e. just the symbols referenced in the shared library get bound) so the application linking to libcups that also needs one or more additional functions from a dependent library must also link to the dependent library.
Like I said, I'll re-investigate to see if we can move away from that, or at least make it conditional on a per-platform basis...

Required quite a bit more than the simple change you suggested, but I believe master now has what you wanted... :)
[master 3e391a8e7] Update build system and cups-config script to not add extra libraries to the "--libs" output (Issue #5261)


Fixed broken email header when hosting static files remotely

Fixes #5525
Thanks for contributing to Wagtail! 🎉
Before submitting, please review the contributor guidelines http://docs.wagtail.io/en/latest/contributing/index.html and check the following:

Do the tests still pass? (http://docs.wagtail.io/en/latest/contributing/developing.html#testing) Yes
Does the code comply with the style guide? (Run make lint from the Wagtail root) Yes
For Python changes: Have you added tests to cover the new/fixed behaviour? No -- didn't think it was required for such a small change, but if needed, I can write them
For front-end changes: Did you test on all of Wagtail’s supported browsers? Please list the exact versions you tested. N/A
For new features: Has the documentation been updated accordingly? N/A

Thanks @edu2004eu! Code here looks good.
It would be good to have a test for this, so that we've got a permanent record of the fix and to ensure it doesn't get broken again in any future refactor.

@gasman I don't really have much experience with unit testing, but I did my best. Let me know if that's OK.
I've also changed the way I get the BASE_URL setting, since tests failed if the settings file didn't define one.

Thanks again! Merged in b9cfc24b37979b9c8528fd0466e39758170f90c7, with some minor tweaks to the test to make it more specific.


How do I pass variables from one page to another?

This is more of a general streamlit question, not specific to st_pages, so you'll probably find better support on https://discuss.streamlit.io, like here: https://discuss.streamlit.io/t/pass-variable-from-one-page-to-another-after-introduction-of-multi-page-apps/31482
The short answer is, you use st.session_state to pass variables between pages


Update Lavaplayer version

Update Lavaplayer to 1.3.49 to fix issues with Youtube tracks.

https://github.com/Frederikam/Lavalink/commit/ddb5f73b218ca0fc5f494788b282856e3728c6cd


Docker build caching

Docker build caches each command, so an apt-get update, etc. makes no change, if it was ever run on the same image before.
In Travis this is not a problem, as it always starts with a clean environment, there is no cache there. But on real environments, I believe the cache can be any age. I don't know how are you publishing the images, or how frequently are docker build commands run, but Makefile might need --no-cache for consistent results.

In Travis this is not a problem, as it always starts with a clean environment, there is no cache there. But on real environments, I believe the cache can be any age. I don't know how are you publishing the images, or how frequently are docker build commands run, but Makefile might need --no-cache for consistent results.

Our public Docker images use automated trusted builds so we do not push any
images explicitly.
For local development we want the docker build in  Makefile to cache otherwise
it would be too slow for development. Users of osm2vectortiles can build it themselves but we recommend to use the prebuilt images at https://hub.docker.com/u/osm2vectortiles/.

Docker build caches each command, so an apt-get update, etc. makes no change, if it was ever run on the same image before.

I would actually like to fix the the versions of aptitude packages installed to guarantee more repeatable builds. I will use this issue ticket to check out whether the Dockerfile are all good other
we can improve something.
Btw. shameless self plug. I wrote a Dockerfile linter http://hadolint.lukasmartinelli.ch exactly for this purpose to help people to create best practice Dockerfiles.

Sorry, I forgot to mention that this issue was only relevant to developers building the containers, for normal users the pre-built images are perfectly fine I believe.
I don't think you can pin apt packages like you can with npm and pip packages, as apt repositories usually only serve the very latest version of each package.
What you can do on apt is to hold an existing (installed) version, but since you are building Docker files from scratch it won't be an option either.
But don't worry, as Ubuntu LTS and Debian has a really conservative policy for packages, so you are guaranteed to be on the same minor version of a package for the lifetime of the OS. (Python 2.7 is for example from 2013): http://packages.ubuntu.com/trusty/python

What is the concrete improvement we can do here? Otherwise I close this :wink:  Adding --no-cache to the Makefile would make it too slow to work with.
I actually never ran into a problem with our apt installs so far.

No improvement / bug here, just wanted to raise the point that local builds and Travis build can be on different versions, so we might need to manually rebuild with --no-cache from time to time. I was surprised when I first understood this, might not be clear for others as well.

No improvement / bug here, just wanted to raise the point that local builds and Travis build can be on different versions, so we might need to manually rebuild with --no-cache from time to time. I was surprised when I first understood this, might not be clear for others as well.

We should create kind of a contributing / developer docs where we can write stuff like this down.
Right now we don't have much information on GitHub as the docs are on the website. But contributor information like this could live in the GitHub README imho.


[Feature Request] Please notarize or developer-sign cli-kintone

Why
With macOS Ventura, the default security setting has increased security and no longer allows the "anywhere" option without a terminal command.
A macOS user of cli-kintone encountered this issue recently:
https://forum.kintone.dev/t/verification-problem-with-command-line-tools-on-macos/824/
This forces cli-kintone users to manually disable macOS Gatekeeper settings with a sudo command.
What
Please notarize or developer-sign the cli-kintone package.
Or Add Support for Homebrew · Issue #310 · kintone/cli-kintone. This way, the brew package manager can handle the installation process.

I would also like to see this feature. Otherwise, one has to downgrade one's system security to use the CLI.
Jason

@ahandsel
Thank you for posting your feedback!
We will consider adding codesign to the macOS binary.
Please wait for further information.
While we checking this issue, please tell us the following.

Couldn't you add cli-kintone to an exception of the security settings?

I found that we can save non-verified executables as an exception of the security settings without disabling macOS Gatekeeper.
Could you try the following steps?
"If you want to open an app that hasn’t been notarized or is from an unidentified developer" section of the following page:
https://support.apple.com/en-us/HT202491

About Homebrew

Let me confirm my understanding.
According to your issue about Homebrew, we need to add codesign manually even if we make a Formula of cli-kintone.
Is that correct?
https://github.com/kintone/cli-kintone/issues/310#issuecomment-1521148131


"Developer: Reload Window" causes issues for long running tasks

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.77.3
OS Version: MacOS 13.3.1

Steps to Reproduce:
Run the following commands
mkdir test
cd test
cat <<"EOF" > package.json
{
    "scripts": {
        "watch": "while true; do sleep 1; echo \"$$ $(date)\"; done"
    }
}
EOF
code .

Trust the workspace, then start the script from the "NPM Scripts" panel. The script runs, and the name of the script appears to the right of the output panel, together with a spinner showing that it's running.
Now run the script again. A popup window appears saying that it's already running, and do you want to terminate or restart it (just close it without doing either).
This is all expected behavior.
Now (with the task running) run Cmd-Shift-P "Developer: Reload Window". The window reloads, and the script continues to run (you can tell it's the same script because it's printing its pid, and it doesn't change). The name is still in the right hand pane, but now the spinner has gone, as if vscode no longer knows that it's running.
Now run the script again, and although it's already running, it starts a second copy. This is the bug. It's especially painful for "runOnOpen" tasks, because every time you reload the window (eg because an extension needs updating) they all continue to run AND a new copy of each one starts.
I thought there might be another bug in the recently added AbstractTaskService._inProgressTasks (since that was supposed to fix exactly this issue - #159640), and took a look at that. There does appear to be a pretty serious bug in that code:
https://github.com/microsoft/vscode/blob/e9c1114e6c62ae498fa3eefdf6ea29d85aa08b90/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L1835-L1858
It adds qualifiedLabel to the _inProgressTasks at line 1839, then starts the task at line 1853, and then removes qualifiedLabel at line 1855 without waiting for the task to finish. So in practice the list is almost always empty, and doesn't actually do anything. I think the delete at line 1855 should actually be in a .finally clause attached to the _handleExecuteResult(...) on line 1856, so that it doesn't run until the task completes.
But fixing that doesn't actually fix this issue, because the set isn't preserved (or reconstructed) across reloads. In addition it breaks the above behavior where trying to run an already running task asks what you want to do with it (because now that inProgressTasks is actually working, it aborts the attempt to re-run the task before it gets to the point where that dialog is presented). It also breaks tasks that are explicitly marked as allowing more than one instance.
So since it isn't currently doing anything, and fixing it breaks things, I would suggest removing all the associated code (@meganrogge)
I'm not sure what the actual fix should be though. There seems to be code to try to reconnect tasks to terminals:
https://github.com/microsoft/vscode/blob/e9c1114e6c62ae498fa3eefdf6ea29d85aa08b90/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L340-L341
but it doesn't seem to work (I've stepped through the code with my example, and it goes into _attemptTaskReconnection but doesn't actually find any tasks to reconnect). So maybe the bug is there. Or maybe the bug is that the tasks shouldn't be preserved across reload (many are not; a terminal that isn't running anything will be killed by reload, for example).
But I've reached the limits of what I can debug, unless anyone has suggestions for where to look.

thanks for looking into this. the reason it's not in a finally is this was meant to (and did) fix an issue that was super hard to reproduce where duplicate tasks would get started.
the only tasks that get reconnected to are those with isBackground: true and have problem matchers.

It's especially painful for "runOnOpen" tasks, because every time you reload the window
you can disable this with task.reconnection: false.

task reconnection works for me
https://user-images.githubusercontent.com/29464607/232880261-079e986a-98eb-462f-ba12-5d8ca8767f9a.mov

the reason it's not in a finally is this was meant to (and did) fix an issue

Ok. So there's a race that happens between deciding to launch the task and actually launching it? In that case, the solution seems kind of misleading.
Shouldn't it be something like _launchingTasks rather than _inProgressTasks? but also, instead of
 if (executeResult) {
 	this._inProgressTasks.delete(qualifiedLabel);
 	return this._handleExecuteResult(executeResult, runSource);
 }
 this._inProgressTasks.delete(qualifiedLabel);

why not just do
 this._inProgressTasks.delete(qualifiedLabel);
 if (executeResult) {
 	return this._handleExecuteResult(executeResult, runSource);
 }

And why do you need the delete in https://github.com/microsoft/vscode/blob/e9c1114e6c62ae498fa3eefdf6ea29d85aa08b90/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L1918 since it was already deleted right after it launched?
But all this is incidental; I was hoping this code could prevent the bug I reported, but clearly it can't.

the only tasks that get reconnected to are those with isBackground: true and have problem matchers

Ok... but the problem is that my task is left running across the reload, but it doesn't get reconnected (apparently intentionally), so attempting to start it again succeeds, even though it's supposed to only allow one instance.

task reconnection works for me
https://user-images.githubusercontent.com/29464607/232880261-079e986a-98eb-462f-ba12-5d8ca8767f9a.mov

I'm not sure what that .mov is supposed to be showing me; I can't see anything relating to tasks being reconnected (but I'm not sure what Im looking for). But anyway, I'm not claiming it's broken; just that it doesn't reconnect my task (or similar ones). And if, as you say, thats intentional, then the bug is elsewhere. But as far as I can tell, either my task shouldn't be left running, or it should be reconnected.

Another data point.
After reloading, the task is clearly running, but Cmd-Shift-P: "Tasks: Show Running Tasks" tells me "No running tasks"

you can disable this with task.reconnection: false

So when I set this to false, my task is killed on window reload, which does mitigate the problem (in fact, for my use case, I think this does solve the problem - although I don't know yet whether it introduces new ones).
But that still leaves us with the issue when task.reconnection is true. The task is left running, but not (properly) reconnected - so although it's there, and its output is going to.a terminal, the abstractTaskService, and terminalTaskSystem aren't aware of it (so that "Show Running Tasks" says there are none, and attempting to start a second instance succeeds).

I won't be looking into this any time soon just fyi. So if it bothers you, feel free to keep digging. Otherwise, just keep the setting disabled.

I run into this problem all the time too, which is very frustrating. I have to reload often due to SSH reconnection etc, this reattach issue together with ssh agent forwarding on mac os breaking, are really detrimental to the work flow.

Yeah @patroza that is unrelated to this feature as you mentioned.

@meganrogge as noted in the pull request, this doesn't fix everything, but it makes my workflow (with several auto-run tasks) much cleaner. Those tasks will be properly reconnected on reload, rather than an additional set being kicked off on each reload.
For tasks that don't auto-run on reload, the only change is that the first attempt to run a new instance of that task will simply attach to the existing process; and subsequent attempts will fail the same way they do before the reload.
Other than that, there should be no change in behavior, so I think this should be pretty safe.
I spent some time trying to figure out how to actually reconnect all the running tasks, rather than just marking them for reconnection, but I kept running into issues where abstractTaskService.ts needed to know about the internals of terminalTaskSystem.ts (IReconnectionTaskData in particular). Or I needed to extend the ITaskSystem api - which I'm reluctant to do without guidance.
I also tried a completely different approach - namely, including non-background tasks in _persistentTasks. That almost worked, but the problem was that task.getRecentlyUsedKey() gave a different result when the task is launched manually vs when it's run from _reconnectTasks (when run manually, its a ContributedTask, but when run from _reconnectTasks its a ConfiguringTask, which gets resolved to a ConfiguredTask). As a result, the tasks did get reconnected properly, the various commands (eg workbench.action.tasks.showTasks) know they're there and do the right thing; but now trying to run them manually uses a different key, and still starts a new copy...

We will now only reconnect tasks for which isBackground:true and not persist the terminals for the others.

To verify, run a non background task. Reload the window. Verify that you can run that task again


Add recipe for remote debugging with bridge to kubernetes and the UI dashboard

Not all solutions can be run locally. This can be due to using a cloud service that does not offer a local emulator or a local runtime environment. In these cases it may be necessary to deploy a service into a cloud environment to perform end-to-end testing. When this end-to-end testing is being done it can be very useful to be able to attach a debugger (as one would if the entire solution could be run locally).
This issue will create a article describing an approach for enabling remote debugging using the Bridge to Kubernetes tool and UI Dashboard.

I would love to work on this one - @tasdevani21 let me know if you want to collaborate? :)

@charleszipp @tasdevani21 per our discussion I think we can close this issue as will try to contribute directly to https://code.visualstudio.com/docs/containers/bridge-to-kubernetes, and we will look into Observability section possibly being a better home for the K8s dashboard.

Closing as we found a home for the content that will be more widely available to the public


Cleanup docker config for DJUI.

Summary
@shangyian please have a look at this PR and let us know what you think.
Whew... it started with a simple "hey Sam, can you help me out with my djui docker errors?"
djui          | yarn run v1.22.19
djui          | $ webpack-dev-server --open --host <IP_ADDRESS> --port 3000
djui          | /bin/sh: 1: webpack-dev-server: not found
djui          | error Command failed with exit code 127.
djui          | info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
djui exited with code 127

And we went from there:

we found one command to be not kosher on the latest yarn/npm version: s/yarn install/yarn
then we started debating what commands should run during building of djui container vs during compose up
and then @samredai suggested to delete the Dockerfile for djui and to define all the params in the main docker-compose

Our initial goal was to be able to support both cases (for a newbie):

quick demo from scratch
dev cycle for djui components
@samredai please add any other points I missed in this description.

@shangyian once we agree on the best configuration, perhaps I should fix some dev docs around this topic. No rush on this, since I am unblocked, we can chat about it as well.
Test Plan
Tried multiple combinations of docker compose up and docker compose build djui until we got to a perfect state.

[ ] PR has an associated issue: #
[ ] make check passes
[ ] make test shows 100% unit test coverage

Deployment Plan

Thanks @agorajek for opening this and summarizing our discussion. This got me taking a peek at the other Dockerfiles and I think all but the datajunction-server one can be replaced with adding a simple key or two to the docker compose. For example this is the datajunction-reflection dockerfile:
./datajunction-reflection/Dockerfile
FROM python:3.10
WORKDIR /code
COPY . /code
RUN pip install -e .

We already mount the code directory in the compose so that file could be replaced by this simple change to the docker compose:
  djrs-beat:
    container_name: djrs-beat
    profiles: ["demo"]
-   build:
-     context: ./datajunction-reflection
-     command: ["celery", "--app", "djrs.worker.app", "beat", "--loglevel", "INFO"]
+   image: python:3.10
+   working_dir: /code
+    command: sh -c "pip install -e . && celery --app djrs.worker.app beat --loglevel INFO"
    networks:
      - djrs-network
    depends_on:
      - djrs-redis
      - dj
    restart: "no"
    volumes:
      - ./datajunction-reflection:/code

Even though the pip/yarn installation will run on every docker compose up, after creating the container for the first time, subsequent startups should be a very fast cache-hit. Then there's an added performance boost from the fact that we're relying on cached python:3.10/node:19 images which should be much more re-usable than our slightly customized images.

@samredai I can see where you are going with the suggestion on ./datajunction-reflection/Dockerfile but I also see that this container is used twice inside the docker-compose.yaml and I am not sure how that change will play in both cases. I'll merge this now and I we can review the this pattern for other containers later.


setImageDrawable(viewId, drawable) not working

i tried populating different drawables to imageView but it seems not to work.
Note: it worked for textView.

private List<Model> mModelList = new ArayList<>();

public void onViewCreated(View view, Bundle savedInstanceState) {
    // ommitted super.onviewcreated

    recyclerView.setLayoutManager....
    adapter  = new QuickAdapter();
    recyclerView.setAdapter(adapter);

     prepareData();

}

// the adapter constructor
public QuickAdapter() {
    super(R.layout.layout_view, mModelList);
}

@Override
protected void convert(BaseViewHolder helper, Model data) {
    helper.setImageDrawable(R.id.icon_id, data.getDrawable);
}

// my model
public class Model {
    private Drawable drawable;

    public Model(Drawable draw) {
        this.drawable = draw;
    }

    public Drawable getDrawable() {
        return drawable;
    }
}

// Data
private void prepareData() {
    Model model = new Model(ContextCompat.getDrawable(getActivity(), R.drawable.ic_image_1));
    mModelList.add(model);

    model = new Model(ContextCompat.getDrawable(getActivity(), R.drawable.ic_image_2));
    mModelList.add(model)

    mAdapter.notifyDataSetChanged();
}

my bad... its actually working, the drawables and my background were the same color and I didn't notice.
thanks for the library anyway, its a lifesaver compared to writing recyclerView from scratch.


Update AddContactedInfoCommandParserTest

This commit resolves https://github.com/AY2122S2-CS2103T-T17-3/tp/issues/112.
New test cases have been implemented to check for potential bugs.

Codecov Report

Merging #113 (75cbb47) into master (9ed928c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #113   +/-   ##
=========================================
  Coverage     71.48%   71.48%
  Complexity      649      649
=========================================
  Files           110      110
  Lines          2104     2104
  Branches        216      216
=========================================
  Hits           1504     1504
  Misses          542      542
  Partials         58       58

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 9ed928c...75cbb47. Read the comment docs.


SwitchMap no longer cancels ajax requests for queries

Newest version of Apollo no longer cancel ajax requests when using switchMap.
I use apollo.query, did not test watchQuery.
Works fine when using the Angular HttpClient.
    this.searchField = new FormControl();
    this.searchField.valueChanges.pipe(
      switchMap(term => this.search(term)),
    ).subscribe(res => {
      this.results = res;
    });

  search(term: string): Observable<any> {
    const casesQuery: DocumentNode = gql`
        query cases($searchQuery: String) {
            someQuery(
                search: {query: $searchQuery},
            ) {
                items {
                    id
                }
            }
        }
    `;

    return this.apollo.query({
      query: casesQuery,
      variables: {
        searchQuery: term
      },
    });
  }

Using Angular 6.1.0, I am not able to cancel any subscription in-spite of using async or unsubscribe();
this.data = this._apollo.watchQuery({query: getJobCode_Q}).valueChanges;

CODE: {{entry.CODE}}

Re-browsing to this component or calling getData() doesn't re-fetch any data from GraphQL server.
Only way to re-fetch/get new data is to reload entire Angular App.

@crissi In apollo-angular .query() wraps a Promise with Observable, I don't know if it cancels a promise, need to check it. Maybe you can create a test for it? I can help you!
@AmitXShukla I added tests and https://github.com/apollographql/apollo-angular/pull/733/files#diff-be568156b5ced52591b3b4f42328bcd8 seems to work fine.

@crissi .query() doesn't cancel the request because unsubscribe() can't cancel a Promise since it's not cancellable, only with AbortController which ApolloClient doesn't support. I think you should open an issue in ApolloClient.


Dynamically create icon versions by script

Several icon versions are harder to maintain. Versions with shield should be removed and should be created by script. This would also allow to specify several other parameters.
Also halos are not wanted by everybody. These could be removed and also added by a script.

Done.


Split building and loading the model

This can be useful if you want to build the BERT model from given config, but don't want to load pretrained weights.

Pull Request Test Coverage Report for Build 97

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 96:
0.0%

Covered Lines:
340

Relevant Lines:
340

💛  - Coveralls

Thanks for the PR~ I've published a new version.

Great, that was fast, thanks!


Create consistent qtree pool names

When creating new pvc with ontap-nas-economy driver creates a path like below
/trident_qtree_pool_****_EHGZHGUVQD. When we delete all the claims and create more of them a day later qtree names changed.  This prevents us from implementing a reliable backup of these volumes.
Is there a way to solve this issue right now? If not we would like a solution to be implemented so we can take backups more reliably.

Hi @brlbil,
qTrees are named using the PVC UUID and are restricted to 64 characters in length. Trident itself isn't designed to manage backups. You may be interested in investigating NetApp's Project Astra as a potential backup solution.


Assorted issues for the share plugin

not options available inside the select box for "Registered memebers"
options clipped by the pinned box
Anyone not available for documents

Clipping:

Missing "Anyone" for documents:

@giohappy about point 3 "Anyone not available for documents"
I cannot replicate, in my case, I show it.
that component depends from "/permissions" api, maybe in your case, the api didn't return anonymous group


add instructions on how to run locally

I added the commands I used, am I missing some?
Just saw that this strips trailing spaces too, but that's probably not a big deal.

Because I'm getting SQLSTATE[HY000] [2002] No such file or directory when running php artisan migrate etc.

Whoops, forgot this was in the README


Add suite params to shed.yml

Add advanced params to shed.yml to classify tool as suite.

Thanks!


Add flake8

Add it as a CLI command. Run it during the Travis build.

pr here #37
fixed a few of the template tests also.

https://github.com/realpython/cookiecutter-flask-skeleton/pull/37


LeaderBoard: Setup Project

LeaderBoard: Setup Project
The leaderboard website displays scores submitted by different players. It also allows you to submit your score.
Implemented Features

[ ] Created an npm project with webpack.
[ ] Written plain HTML markup with minimum styling
[ ] Use ES6 modules, with import and export.

Hey @rloterh thank you so much for review feedback 🤝
I will take required changes into consideration and workon it.

Hey @emmiiorji Thank you so much for approving my Project 🤝
it motivated me, cheers 🍾


Add locks

This adds a couple of locks to solve data races.

This was meant to be on our local fork.


[1Password] Extend CLI authorization time to 12 hours

Extension
https://www.raycast.com/khasbilegt/1password
Description
The current CLI authorization times out after 10 minutes. Here is description
"Allow terminal session in "Raycast" to get full access to your account "ACCOUNT" using the 1Password CLI. Authorization expires after 10 minutes of inactivity or when 1Password locks."
This makes it so every 10 minutes the 1Password extension needs to re-authorize with 1Password.
https://developer.1password.com/docs/cli/app-integration-security
"Authorization expires after 10 minutes of inactivity in the terminal session. There's a hard limit of 12 hours, after which you must reauthorize."
It would be great if the extension could increase the timeout from 10 minutes to the max of 12 hours.
Who will benefit from this feature?
Everyone
Anything else?
No response

Hello, Currently I have this feature ready on the pull request #11605. I will update it here as soon it gets released.

#11605 Has been merged  🎉. Should be solved now. Just enable background refresh on "Auto Renew Authorization" on the extension config.

@raycastbot close this issue


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1153 ms

Resolved: Bofrost.de performance has improved in 3644349.


build: list the PackageLoading dependency

Sources/Workspace/UserToolchain.swift references PackageLoading but
the build did not list the dependency.  Add the missing dependency.

@swift-ci please smoke test


Add buttons to revert/drop commits

When you change something via the panel but don't want to push it you currently have to use the command line to drop the commit (e.g. via git reset --hard HEAD~1 to undo the last commit)

I think it would be a great addition to add either a "Restore from remote" button or better yet a "Drop commit" button for all commits that are not pushed yet.
I can provide a PR for that but I want to hear a little bit of feedback on this first

I actually planned to integrate something like this. Dropping a commit might need to make it more complicated as you need to check if you only revert commits that have not been pushed to remote yet or you need to allow force push to remote. So just reverting to the remote HEAD might be easier.
Nevertheless: PRs are very welcome! :)


textarea not rendering error styling

What needs to happen to get the textarea label and border to render error styling?
It is not working on the form I just built. Please advise.
View code here: http://www.svcommunityservices.org/volunteer-signup-form.html
Thanks!

Hey TakeFlightGraphics,
thanks for using the plugin and providing a testing link it made the whole debug process a lot easier.
The issue is that on the textarea you have the NOTEMPTY validation but between the opening and closing tag there is a couple whitespaces (probably caused by a line return inside your code)
I've patched the NOTEMPTY rule to contain at least 1 non-space character (will be released in the next version)
// Before
NOTEMPTY: /./
// After
NOTEMPTY: /\S/

meanwhile you can quickly patch your issue by putting the closing  tag next to the opening one to avoid white spaces

Thank you for checking out my issue. The above tweak fixed the label. It now turns red for errors. However, the border is still not rendering red for errors. Any ideas?
I did notice that this code:
div.error,
div.error-list,
label.error,
input.error,
select.error {
color: #e13369 !important;
border-color: #e13369 !important;
}
does not include a textarea selector. Could that be it?
Thanks!

Yea! good catch
I've also noticed you are using MIXED for first name and last name validation. I checked the regex on my side and it might not be appropriated since it's missing apostrophe character ' and allows entries for numeric characters. (ex: James O'Reily will not be valid)
I've added a Rule (will go inside 1.5.2)
NAME: /^['a-z\s-]+$/i

Perfect. I updated the selector list:
div.error,
div.error-list,
label.error,
input.error,
select.error,
textarea.error {
color: #e13369 !important;
border-color: #e13369 !important;
}
and it now works. I will check out the NAME rule tomorrow for first and last name fields. Working on taxes right now. Fun, wow!
Thanks!

Tom:
Where can I make a donation to you? I could swear I saw a donate button
somewhere.
Please let me know. Thanks!
Best regards,
Robin Fisher
Take Flight Graphics~
On 2/18/2015 5:18 PM, Tom Bertrand wrote:

Yea! good catch
I've also noticed you are using MIXED for first name and last name
validation. I checked the regex on my side and it might not be
appropriated since it's missing apostrophe character ' and allows
entries for numeric characters. (ex: James O'Reily will not be valid)
I've added a Rule (will go inside 1.5.2)
NAME:  /^['a-z\s-]+$/i
—
Reply to this email directly or view it on GitHub
https://github.com/running-coder/jquery-form-validation/issues/10#issuecomment-74983418.

Hey,
I have currently no donation button but I might add one in the near future because it indeed helps for supporting the fixes and new features.
Thanks for your support and feedback!
ps. I've modified the above NAME rule to match also punctuated roman letters to cover more possibilities (released in 1.5.2)
NAME: /^['a-zãàáäâẽèéëêìíïîõòóöôùúüûñç\s-]+$/i,


Update Dagger to 2.2

Probably it'll fix #78 idk
Changelog https://github.com/google/dagger/blob/master/CHANGELOG.md#version-22-2016-03-22

Current coverage is 81.42%

Merging #140 into master will not affect coverage as of 9608244

@@            master    #140   diff @@
======================================
  Files           37      37
  Stmts          549     549
  Branches        47      47
  Methods          0       0
======================================
  Hit            447     447
  Partial         29      29
  Missed          73      73

Review entire Coverage Diff as of 9608244

Powered by Codecov. Updated on successful CI builds.

:+1: But #78 is still happen.

:+1:

@artem-zinnatullin the issue #78 was accidentally closed since you wrote fix #78 and the probably it'll was not read :D

oh shi, my bad, reopened :D


swift: mark as broken on darwin

Description of changes
https://hydra.nixos.org/job/nixpkgs/trunk/swiftPackages.swift-unwrapped.x86_64-darwin
https://hydra.nixos.org/job/nixpkgs/trunk/swiftPackages.swift-unwrapped.aarch64-darwin
https://github.com/NixOS/nixpkgs/issues/320900#issuecomment-2188286123

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

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


Support Sequence Recognizer for PhoneNumber in Python

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


Package request: openfoam

OpenFOAM is the free, open source CFD software developed primarily by OpenCFD Ltd since 2004.
OpenFOAM has a large user base across most areas of engineering and science, from both commercial and academic organisations. OpenFOAM has an extensive range of features to solve anything from complex fluid flows involving chemical reactions, turbulence and heat transfer, to acoustics, solid mechanics and electromagnetics.
Metadata

homepage URL: https://www.openfoam.com/
source URL: https://develop.openfoam.com/Development/openfoam
license: gpl3
platforms: linux

Add a :+1: reaction to issues you find important.

This is already addressed as part of #131486 and a PR for this is awaiting review (since 2 months) in #329213. There is also the (not yet marked ready) PR #329527 with OpenFOAM components.


Add the ability to spawn an isolate in Dart 2 preview mode

Pub users want to be able to run their executables via pub run using Dart 2 preview mode (see https://github.com/dart-lang/pub/issues/1807 and https://github.com/flutter/flutter/pull/14728#discussion_r168903155). Because pub run runs executables in isolates, this means it'll need to be able to spawn an isolate that uses Dart 2 preview mode.
I suggest adding a bool previewDart2 flag to Isolate.spawnUri() that will become a deprecated no-op once Dart 2 releases.

I'd much rather have the Dart version embedded in the dill file that the VM uses to spawn the isolate.
I see no advantage in supporting running actual Dart 2 code from source since that will not work anyway (the VM doesn't do type inference, and Dart 2 without type inference doesn't really work).
Alternatively, make pub itself run with the flag set. Perhaps make the pub script accept the --preview-dart2 flag and run itself on a VM with that flag. The pub code should still work, and anything it spawns will inherit the flag. I assume that there would need to be a flag on pub run anyway..

@kevmoo @devoncarew How important is this for general travis testing with Dart 2?
Some options:

Add the isolate support as requested.  Not sure it's actually possible for the VM to have some isolates on the CFE and some not (@a-siva ).

Get pub itself running on Dart 2 as Lasse suggested.  Not clear how much work it'll be to run it and its dependences on Dart 2.  We haven't gotten most of our own tools running in 2 yet.

Rework pub to not use isolate (and, e.g., start a new Dart process instead).  @nex3 - is that feasible?

I'd much rather have the Dart version embedded in the dill file that the VM uses to spawn the isolate.
I see no advantage in supporting running actual Dart 2 code from source since that will not work anyway (the VM doesn't do type inference, and Dart 2 without type inference doesn't really work).

Hold on a sec--if we don't plan to support running Dart 2 code from source, that's news to me. It also has major implications for all sorts of tooling. My understanding was that the VM would embed the common front end to do the initial type checking. Isn't that how --preview-dart-2 works today?

Alternatively, make pub itself run with the flag set. Perhaps make the pub script accept the --preview-dart2 flag and run itself on a VM with that flag. The pub code should still work, and anything it spawns will inherit the flag. I assume that there would need to be a flag on pub run anyway..

#32188 is tracking this, but there are a lot of complexities involved. In addition to getting the transitive closure of pub (which includes the analyzer and dart2js) Dart-2-clean, we'd need to build both Dart 1 and Dart 2 snapshots of pub, bundle them with the SDK, and have pub's wrapper scripts parse arguments to figure out which one to invoke. It's not impossible, but it would be a lot of work.

Rework pub to not use isolate (and, e.g., start a new Dart process instead).  @nex3 - is that feasible?

We used to do this, and it caused all kinds of problems around performance, signal-handling, and standard IO piping. I don't want to regress that behavior, and I'm not keen on adding an alternate code path that has more behavioral differences than just the Dart 2 semantics.

@nex3 - The VM supports running Dart 2 code from source we don't plan on getting rid of that.
@vsmenon - about the ability for the VM to run some isolates in strong mode and others not - we don't currently have support for this when running from source but it should be possible to do this (currently we start the CFE isolate eagerly under the --preview-dart-2 flag but if we made that lazy then this should be possible).

@rmacnak-google (see offline email about a possible way to get this working quickly)

the right solution to this is just to make pub Dart 2 compliant

Landing https://github.com/dart-lang/pub/issues/1857 first will make this a lot easier, since the surface area of pub will be reduced substantially. ETA – next few days.

Please confirm my understanding:

It is not currently possible to use/run a package:test test which use dart2 (or tests for dart2 specific issues), correct?
It is possible to use/run a flutter test test in dart 2 (that's the only way) due to flutter_test's different test backend.

It is not currently possible to use/run a package:test test which use dart2 (or tests for dart2 specific issues), correct?

It is not currently possible to run a VM test with dart 2 semantics using pub run test
It is possible to run a DDC test with dart 2 semantics.
I think it should be possible to run a VM test with dart 2 semantics using a hacky workaround of invoking the package:test entrypoint directly (have you find it first) instead of going through pub run.
There may be dart 2 semantics issues in the package:test code which would also prevent this.

Just tried this. package:test is not Dart 2 clean. There are a few bugs I can fix, but then there is also one in package:front_end getting pull in through analyzer...

https://github.com/dart-lang/sdk/issues/32867

Closing since I don't think we're going to support running Isolates in a different mode from the rest of the VM.
For now the solution is DART_VM_OPTIONS=--preview-dart-2 and long term that will become the only mode.


when pushing to gitea use a loop to try multiple times on errors

Fixes #1332
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>

lgtm


OneDrive folders crash while opening

Files App is crashing when trying to open any folder from then app.
dont know if its the local files or synced files on the cloud

Thanks for the report. Please add the app version to the issue description and post the app's log files.


Restore purchase option

Restoring a purchase would be very helpful. Thank you for the easy to use plug-in.

You can use getPurchases to request what identifiers (skus) have already been purchased by the customer.  Billing. getPurchases


Linux nightly downloaded instead of FreeBSD

I just updated my nightly toolchain and uhh…
$ rustup run nightly rustc
/home/greg/.rustup/toolchains/nightly-x86_64-unknown-freebsd/bin/rustc: /lib64/libc.so.6: version `GLIBC_2.18' not found (required by
 /home/greg/.rustup/toolchains/nightly-x86_64-unknown-freebsd/lib/libstd-91d56e7ad05aa8e9.so)
/home/greg/.rustup/toolchains/nightly-x86_64-unknown-freebsd/bin/rustc: /lib64/libc.so.6: version `GLIBC_2.18' not found (required by
 /home/greg/.rustup/toolchains/nightly-x86_64-unknown-freebsd/lib/../lib/../lib/librustc_llvm-2d2118ce884c1c66.so)
$ ldd /home/greg/.rustup/toolchains/nightly-x86_64-unknown-freebsd/bin/rustc
ldd: /home/greg/.rustup/toolchains/nightly-x86_64-unknown-freebsd/bin/rustc:
not a FreeBSD ELF shared object
$ file /home/greg/.rustup/toolchains/nightly-x86_64-unknown-freebsd/bin/rustc
/home/greg/.rustup/toolchains/nightly-x86_64-unknown-freebsd/bin/rustc: ELF 64-bit LSB shared object,
x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2,
for GNU/Linux 2.6.32, BuildID[sha1]=6cd5b2aab6699d6026ad53d7e8456dbb86fe40a3, not stripped

that doesn't look like x86_64-unknown-freebsd
(uninstalling and reinstalling did not help)

I believe this was fixed, we accidentally shipped the wrong artifacts during a rewrite of the build system.

Thanks!


[] update to <1.85.0>

Library name
boost
New version number
1.85.0
Other information that may be useful (release notes, etc...)
No response

duplicate of #38201


[Modern] User list broken

Current master the user list give a white Page
react-dom.production.min.js:216 TypeError: t is not a function
    at ct (formatter.js:3)
    at UsersPage.js:54
    at Array.map (<anonymous>)
    at ur (UsersPage.js:45)
    at na (react-dom.production.min.js:157)
    at Ba (react-dom.production.min.js:180)
    at Vs (react-dom.production.min.js:269)
    at Eu (react-dom.production.min.js:250)
    at _u (react-dom.production.min.js:250)
    at xu (react-dom.production.min.js:250)
is @ react-dom.production.min.js:216
formatter.js:3 Uncaught (in promise) TypeError: t is not a function
    at ct (formatter.js:3)
    at UsersPage.js:54
    at Array.map (<anonymous>)
    at ur (UsersPage.js:45)
    at na (react-dom.production.min.js:157)
    at Ba (react-dom.production.min.js:180)
    at Vs (react-dom.production.min.js:269)
    at Eu (react-dom.production.min.js:250)
    at _u (react-dom.production.min.js:250)
    at xu (react-dom.production.min.js:250)

Are you using the official release?
Have you cleared your browser cache?
Can I have access to your server to take a look? You can send over email if you don't want to share.

Are you using the official release?
Have you cleared your browser cache?
Can I have access to your server to take a look? You can send over email if you don't want to share.

Hey @tananaev I compiled the master of the modern web UI with yarn install & yarn build with the latest LTS 16.9.x

In /admin/UsersPage.js
<TableCell>{formatBoolean(item, 'administrator', t)}</TableCell>
breaks the translation
This should fix it
<TableCell>{formatBoolean(item['administrator'], t)}


rewind ex-leader before joining again

Hello,
I'm no pg expert, but I'm a bit surprised you don't try to rewind an ex-leader trying to rejoin the cluster. That's why pg 9.5 introduced the pg_rewind command. Before that you basically had to use pg_basebackup.
The problem as I understood it is that when the leader is disconnected from the cluster but didn't had time to replicate the most recent pages, and secondary takes role of leader, the previous leader can't rejoin unless it "rewinds" its most recent pages because both ex and new leaders have diverged since last common point in time.
What's your take on this?
Thank you,
Laurent Debacker

This is a scenario this template chose not to address. pg_rewind is effectively deleting data.  Every HA deployment has a different best-outcome for the times when pg_rewind would be used.  Some want to save the data from the mis-fork, some are okay with overwriting the errors, some need STONITH more than pg_rewind.
In certain scenarios, I would argue that it's better for a database to be down than to have mis-forked data.

Thank you for you fast response!
Even if you STONITH the ex-leader, if you used async replication, you may still have several megabytes in the replication queue when the ex-leader is finally shot. You will have no issue to use the new leader, but when you want to "recycle" the ex leader and bring it back online, I have read that either you wipe clean its data directory or use pg_rewind, otherwise there would be inconsistencies.
I guess you don't need pg_rewind if you use sync replication, because you know the secondary server will never lag behind.