Implement a test mode

Implement a test mode in which my username is always eligible for messages

It'll also be useful to allow users to clear their history here


Addressing FSM-ambiguity in wires

Issues with wire state
Wires are driven as a Finite State Machine (FSM). This is a one-dimensional value; internally a integer from 0 to 4.
type Wire<T> = {
  // ...
  /** FSM state: RESET|RUNNING|IDLE|PAUSED|STALE */
  state: WireFSM;
  // ...
};

type WireFSM =
  | FSM_RESET
  | FSM_RUNNING
  | FSM_WIRED_IDLE
  | FSM_WIRED_PAUSED
  | FSM_WIRED_STALE;

Where state is used today
The 0-4 states are used here:

In wire (aka createWire) during creation:

Sets RESET

In wire during a run:

Throws if RUNNING is already set (infinite loop check)
If PAUSED

Sets IDLE OR RESET depending if there were subscribers (sigRS)
Repeats this for all wire.inner children

Else

Sets RESET as part of wireReset
Sets RUNNING for wire.fn() call duration
Sets IDLE OR RESET depending if there were subscribers (sigRS)

In _runWires before calling wires:

Moving from PAUSED -> STALE if needed (i.e STALE means PAUSED_AND_STALE)
Calling a wire if it's NOT PAUSED AND NOT STALE (i.e PAUSED_AND_STALE)

In wirePause:

Sets PAUSED for this wire and all inner wires

This already has problems, seen right in the FSM names themselves: I'm overloading terms to imply, for instance, that a wire can never be idle and stale, and that a stale wire is a paused wire. This gets really messy with the introduction of computed-signals that also have a stale state (which is stale but never paused; oh no) and needs to be skipped somehow possibly via another state.

In signal (aka signalBase) during a read-subscribe:

If this is computed-signal whose wire is STALE or NOT_WIRED, run it

In signal during any read (pass or subscribe):

If this is computed-signal whose wire is STALE or NOT_WIRED, run it

In signal when writing a wire:

If this is computed-signal already

Set saved wire as RESET

Set incoming wire as STALE (so _runWires doesn't call it)

In _runWires:

First loop: If wire belongs to a computed-signal set to STALE
Second loop: If wire is PAUSED or STALE skip it

Issues with this state model
Note that I'm not interested in trying to prevent people from explicitly doing bad things like wire.state = xyz. I just don't want to introduce unexpected or accidental state transitions.
Here are some of those pitfalls:

If you call wirePause during the wire function run (wire.fn) you can move your own state from RUNNING into PAUSED skipping infinite loop detection (!).

Calling a wire to unpause is magic. Especially since it breaks the principal "Wires can always be run manually by calling them and this won't cause other side-effects within the engine."

Calling a wire of a computed-signal removes its STALE state but doesn't update the stored value of the computed-signal This breaks the signal until the wire is marked STALE again.

Computed-signal can be initialized with any wire including ones that are PAUSED or that have already had their function wrapped to support DOM patching. When the computed-signal is taken down (writing a value so it becomes a normal signal) the wire is set to RESET. This breaks the wire and prevents it from doing any other tasks such as DOM patching since all subscriptions have been removed. There's no way to know it had other tasks; they're lost in the wrapping.

Although computed-signals are set as STALE in _runWires, it's possible that they'll run somehow and end up as IDLE between the 1st and 2nd run loops. This means _runWires must check wire.cs and not rely only on state... Is belonging to a computed-signal a state now?

It doesn't make sense to pause a computed-signal but it's not good enough to just hope they don't try pausing it; it can happen accidentally. This happens in wire function wrapping (used to patch a single wire into multiple areas of the DOM; i.e chaining). If one of those chains was in a when() block, the wire would be re-adopted and when() would pause all patches, not only its child elements.

Issues with chaining
The issue above about chaining is very concerning. It also came up when I tried fixing computed-signal's to have their saved value updated when a wire is manually run - a small optimization but worth considering.
Why not just wrap the wire function like DOM patching does? The value will be updated via closure...
It'd be impossible to undo.  It's also not safe to restore to the previously seen function since the wire could have been wrapped again during the computed-signal's lifetime.
Fixes
This is how I'm trying to address all these issues...
Both pause and resume should be dedicated methods
Wires should always run when manually called since it's their expected behaviour. Don't have a magic "unpausing" call. This means a new wireResume next to wirePause.
It'll drop bundle size.
Replace chaining with a Set of post-run calls
It's not clear, explicit, and debuggable to have a dedicated wire.hooks Set that tracks all the post-run actions. During takedown of a computed-signals I can use wire.hooks.delete(...) via function reference.
Might add bundle size.
Split state into a bitmask
I've seen projects like Kairo do that. It could be smaller than adding more individual properties like wire.paused.
It'll add to bundle size.
Here's how I broke it down:
[IS_CW][IS_WIRED][IS_RUNNING][IS_PAUSED][IS_STALE]

I can drop IS_CW and use wire.cs instead. This is a two-way link between a signal and wire. It's there for principles of consistency and debugging, so I'm keeping it. I can drop IS_WIRED and use wire.sigRS as well.
[IS_RUNNING][IS_PAUSED][IS_STALE]

I'm still overloading the term "stale" which mostly means "skip" in _runWires context and "run later" in other contexts.
[IS_RUNNING][SKIP_ME][WAS_SKIPPED]

Lastly I need to convey that a skipped wire is overdue for a run. I'd also like wireReset to be able to indicate that it's uninitialized; wire.run doesn't work since wires can always be manually reset even after they're run.
[RUNNING][SKIP_RUN_QUEUE][NEEDS_RUN]

Here's how this works for the above cases:
Pause/Resume:

wirePause: Sets SKIP_RUN_QUEUE. Has no impact to computed-signals because they already have
SKIP_RUN_QUEUE.
_runWires: If SKIP_RUN_QUEUE is set then set NEEDS_RUN. This is the same for computed-signals
too.
wireResume: Clears SKIP_RUN_QUEUE only if it's NOT a computed-signal.

Computed-signals:

signal setup: Sets SKIP_RUN_QUEUE, wire.cs, and wire.hooks.add(...).
signal read-subscribe: If NEEDS_RUN, run.
signal any read (pass or subscribe): If NEEDS_RUN, run.
_runWires: If SKIP_RUN_QUEUE is set then set NEEDS_RUN. This is the same for paused wires too.
signal takedown: Uses wireReset (clears state; sets only NEEDS_RUN). Uses delete wire.cs
and wire.hooks.delete(...).

Reset via wireReset:

Clears all state and sets only NEEDS_RUN. Because it's not idling yet? This helps computed-signals know to auto-run the wire on read to initialize the wire's sigRS and propagate subscriptions (i.e sigIC).
TODO: Benchmark:

if (wire.sigIC.size) wire.sigIC = new Set() vs
wire.sigIC = new Set() vs
wire.sigIC.clear()

Scenarios that are fixed using this model:

Mark a wire as paused (sets SKIP_RUN_QUEUE), have a signal try to run it (sets NEEDS_RUN), and then manually run it - it'll clear NEEDS_RUN but leaves it as SKIP_RUN_QUEUE so it's still paused.

Resume a wire during it's wire run - it doesn't break the RUNNING state.

Manually run a computed-signal's wire that is in NEEDS_RUN - it'll clear NEEDS_RUN and update the signal's saved value using wire.hooks.

Create a computed-signal using an existing/used wire - it sets SKIP_RUN_QUEUE (has no effect on pausing/resuming since they respect wire.cs). If the wire has never run it'll be in NEEDS_RUN and I'll run it; else it'll be idle* and nothing happens. It's always safe to use sigRS.

Other considerations:

I don't warn or throw if the wire they pass has an empty sigRS. If they pass an unrunnable wire with no subscriptions it should be easy to debug outside of a computed-signal. Not my problem.

Regarding the *; it has to be idling (RUNNING=0) since if it was (RUNNING && NEEDS_RUN) they'd be initializing the computed-signal within the running wire and it'll throw a loop error.

Might add bundle size.
...
It'll add to bundle size.

Just my personal advice: forget about bundle size.
I've fallen into this trap one too many times in the past - obsessing about size/perf before actually reaching a complete, functional design with passing tests. These things are super important, but they really are a distraction at this stage - the problem matter itself is difficult enough without burdening yourself with optimizations for size and speed - concerns that tend to make the code harder to read and change, and often more sensitive to subtle bugs.

Make it work.
Make it fast.
Make it small.

Be kind to yourself. 😄


VMS: Redefine _XOPEN_SOURCE_EXTENDED with the value 1 in apps/ocsp.c

Some versions if the VMS C system header files seem to require this.
Fixes #24466 on release older than 3.3.

Just a question here: Should this not come from the VMS file in Configurations rather than in a source file? We had to do this same change for NonStop. See Configurations/50-nonstop.conf

Just a question here: Should this not come from the VMS file in Configurations rather than in a source file? We had to do this same change for NonStop. See Configurations/50-nonstop.conf

Both ways have been permissible, and they both come with pros and cons.  I've often opted for defining such things in individual source files where needed, when it was just a select few.
I did change my mind for master before 3.3 was branched off, due to an increase of possible spots, but that doesn't mean the previous way to do it is a bug.  Hence, we preserve the way that this was done in 3.2 and older.

@tmshort, if you wouldn't mind casting an eye on #24470 too?

24 hours has passed since 'approval: done' was set, but as this PR has been  updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.


projection calculation involving long should use double not float

https://github.com/opennars/opennars/blob/ConceptNet/src/main/java/org/opennars/inference/TruthFunctions.java#L492
and probably other places
also whats with the weird 100000000 constant. not all clocks run at the same rate or use the same time units

;ets just wait for Patrick cycles to process this


Scale

Hello, it would be cool to add a real scale of objects so that the proportions are 1:1.

@AndreiExtr if you want to contribute, you would be welcome, I have still an unofficial fork here: https://github.com/Luccid-ai/react-planner
See it in action there: beta.luccid.ai !


When the right mouse button opens, several vscode versions appear.

- VSCode Version: Code 1.14.2 (cb82febafda0c8c199b9201ad274e25d9a76874e, 2017-07-19T23:22:50.650Z)
- OS Version: Darwin x64 16.1.0
- Extensions:

Extension
Author (truncated)
Version

beautify
Hoo
1.1.1

html-snippets
abu
0.1.0

cssrem
cip
0.0.3

vscode-svgviewer
css
1.4.2

vscode-eslint
dba
1.2.11

vscode-html-css
ecm
0.1.7

auto-rename-tag
for
0.0.14

vue-snippets
hol
0.1.2

vue
jcb
0.1.5

vscode-fileheader
mik
0.0.2

debugger-for-chrome
msj
3.1.6

vscode-css-peek
pra
1.3.1

open-in-browser
tec
1.0.0

Steps to Reproduce:

Reproduces without extensions: Yes/No

You have them somewhere in your system. Locate them and delete them.


Update style.css

Changes in #loader before and after
Consolidated Common Styles:
The styles for :before and :after pseudo-elements are consolidated into a single block, reducing redundancy and improving readability.
Simplified Animation Declaration:
Animation properties (animation: spin 3s linear infinite;) are now directly applied in the common block for both pseudo-elements, avoiding duplication and enhancing maintainability

Which issue are you assigned to @pranjal-kotkar


[haskell-updates] haskellPackages.hnix: Build broken because neat-interpolate dep is to new for stack

This issue just tracks the upstream issue, so that no one else repeats the debugging.

Closed in #93105.


Longest pass of starting QB for week 2?

Sport
Football
Summary
Is it possible to grab the longest pass of the starting qb for week 2? Doesn't matter if it's a touchdown or not just the longest pass of week 2?

Hmm it doesn't look like the stats get down to that granular of individual passes and longest passes. Are you able to see this stat on ESPN's website?

I am able to see it via game log. Like Dal Prescotts longest pass in week 2 was 65 yards

I am looking at Dak's game log through my fantasy team and don't see it. Could you post a picture of it?

I'm looking through this link

I see, yeah this is outside of the fantasy API calls. Not sure what API's this view is using and the format data.


Cannot use same names for import, in different modules

Hello there,
I am building a simple react/TS app, and am encountering issues with TSLint right now.
In 2 different modules I "import * as React from 'react'" and my IDE keeps yelling at me even though everything builds up and run like a charm.
(here in GlobalForm.jsx and I do the same import in App.jsx.):

If I rename the "React" in my import, I am not bothered anymore.
Is it a bug or a feature I just don't get?
Here is my tslint.json :
{
  "extends": ["tslint:recommended", "tslint-react", "tslint-config-prettier"],
  "linterOptions": {
    "exclude": [
      "config/**/*.js",
      "node_modules/**/*.ts",
      "coverage/lcov-report/*.js"
    ]
  },
  "rules": {
    "ordered-imports": false
  }
}

Thanks!

@Copainbig that's unfortunate! Sorry to see you're having weird errors pop up. 🙁
The error you're seeing isn't from TSLint; it's TypeScript. Four numbers prefixed by "TS" indicate it's a TypeScript diagnostic.
Closing this issue as external to TSLint, but if you'd still like some help with this, we can keep talking here or on https://gitter.im/palantir/tslint. What IDE are you using, and what are the full contents of that file? Have you tried restarting the IDE? What does your tsconfig.json look like?


attribute((overloadable)) function has wrong name

Input C Header
void
my_function(int a)
    __attribute__((overloadable));

void
my_function(const char *a);

Bindgen Invocation
$ bindgen input.h

Actual Results
/* automatically generated by rust-bindgen */

extern "C" {
    #[link_name = "\u{1}_Z11my_functioni"]
    pub fn my_function(a: ::std::os::raw::c_int);
}
extern "C" {
    pub fn my_function1(a: *const ::std::os::raw::c_char);
}

Expected Results
Normally bindgen gets things correct with the overloadable attribute, but in this specific case (when overloadable comes first followed by a function without it), the second function has the wrong name, and the binding doesn't work.

Yeah, we do if mangled_name == function_name { mangled_name = None } before doing the times_seen mangling, so I can see this breaking.
Should be pretty easy to fix though, thanks for the report!

Hmm, fwiw, that testcase in some compilers it complains hard:

tests/headers/issue-1350-attribute-overloadable.h:2:6: error: overloaded function 'my_function' must have the 'overloadable' attribute, err: true


Update docs to remove Mongodb

We used to use Mongo in deployment and locally but not anymore. We switch the deployment from Open Shift to cloud foundry and as part of that we switch to Cloudant in production and Couch db for local development.
The services readme needs to be updated with this information but also a scrub of any other references to Mongo needs to be done.

@davidnixon Hey David, the issue title references updating docs, should we also remove any mongo code in the repo?

Thanks @hluko. @davidnixon has removed mongo from the code. Please feel free to remove any occurrences from the readmes. I see one here - https://github.com/Call-for-Code-for-Racial-Justice/Five-Fifths-Voter/blob/16faf534b811f77afbcbf670f21fd69e283f0c83/services/README.md
Create a PR when done.

@upkarlidder Is this actually closed?  Looks like Automation closed it.

This is still open. There is one occurrence of Mongo in the README.md file.
@hluko would you like to pick this up for HacktoberFest?

Hi @upkarlidder may I take up this issue?

hi @gibrankasif can you link your PR to this issue please, thanks!

Hi @gibrankasif your change was merged
@davidnixon agreement to close?


cmd/go2go: type parameters can omit any constraint

What version of Go are you using (go version)?

$ go version
go version devel +1964d56295 Wed Aug 26 01:59:01 2020 +0000 darwin/amd64

What did you do?
According to the commit message of https://github.com/golang/go/commit/1964d562957d94ebc105e09eabee12a5eec9c0fd:

all type parameters must have a constraint, can be "any"

However, given the following example:
// p.go2
package p

type I [T] interface {
	M() T
}

What did you expect to see?
Throw missing constraint error, e.g.
$ go tool go2go translate p.go2
p.go2:3:10: expected type constraint, found ']'

What did you see instead?
$ go tool go2go translate p.go2

Nothing thrown.

The experimental translation tool accepts a range of syntaxes.  It does not strictly enforce the rules required by the design draft.

@ianlancetaylor But where to find a document for all possibilities without knowing the implementation details? How can one classify bugs from expected behaviors?

go2go is an experimental tool, and the main purpose of the tool is to see if the generics design draft provides the semantics that people need and want.  The syntax is not the point of the tool.  Report bugs where the behavior is not what you expect, or where you can't do what you want.  Bugs about syntactic details are less interesting, because the syntax is much easier to change than the semantics.
That said, we will soon be updating the dev.go2go branch to default to only permitting the syntax described in the design draft.
Thanks.


Precedence tuning

This PR

fixes several corner cases, such as:

# empty list
[,,,]

# record with leading comment
# previously parsed as closure
{
# foo
echo : 1
}

generalizes general_body_rule to cover more rules within the same pattern
makes anonymous tokens more friendly for topiary to format

Thanks


DBZ-8137 Fix JMX signal and notification channel for multiple tasks connectors

closes: https://issues.redhat.com/browse/DBZ-8137

Test failure looks related
SignalsIT.jmxSignals:88->sendLogSignalWithJmx:103 » InstanceNotFound debezium.sql_server:type=management,context=signals,server=server1,task=0

@mfvitale Applied, thanks!


Added 'My rewards' component and solved issue #11

Added 'My rewards' component and solved issue #11

Great work. Thanks @JelenaDinic


GROOVY-8930, GROOVY-8944: Ant: allow setting targetBytecode 1.6, 1.7 and 1.8

Examples from GROOVY-8930 and GROOVY-8944 run successfully if compiled with Ant/Groovyc using targetBytecode="1.8"

Merged into GROOVY_2_4_X. Thanks. Feel free to close. The auto close feature only works for commits to master.


Page 1

On our first page we want...
an input bar where users can type in passwords
a table that displays what standards are passed and what standards are not

Closing this issue as it is too broad. Will create new issues to inherit contents from this one


find demo

Hi Mauro,
can you help me please? where I can find this exactly demo https://www.borahvandoorslaer.be/node_modules/hello-week/demos/demos.html

I don't have more this example, but what do you need, toggle to range or button today?

Hi Mauro,
actually yes, I need the toogle to range, can you help me please?

It's simple.
For example:
const calendar = new HelloWeek();
calendar. setRange(true);

Thanks for your help Mauro!


Find references with code lens enabled makes the peek view's editor jump

VSCode Version: Code - Insiders 1.16.0-insider (d0d00e882437092f521d2d7893cc56ee296d278f, 2017-08-29T06:58:13.897Z)
OS Version: Linux x64 4.10.0-32-generic
Extensions:

Extension
Author (truncated)
Version

EditorConfig
Edi
0.9.4

lorem-ipsum
Tyr
1.0.0

sort-lines
Tyr
1.3.0

vscode-svgviewer
css
1.4.2

tslint
eg2
0.17.0

git-project-manager
fel
1.4.0

md-navigate
jri
0.0.1

theme-monokai-pro-vscode
mon
1.0.1

vscode-scss
mrm
0.6.2

seti-icons
qin
0.1.3

Ruby
reb
0.15.0

vim
vsc
0.9.0

(1 theme extensions excluded)

#33299
Might not be related to smooth scrolling (I don't normally have code lens on). Maybe code lens should always be disabled in the peek view?

Fixed via e070afceabc88e57190697c9a79dc3ab3a98d3d5 and 8ce745843c581cff6d69ea68b2d9730ce76633f4


Addresses Processed download does not have street direction

I downloaded the addresses-processed.zip file. The CSV files have lat/lon, street number and street name/type. It does not have street direction!
That's a problem because 100 North 1st would be the same as 100 South 1st without street direction.

Stale - Feel free to create a new ticket if this is still an issue.


DinD image not working

Due to the changes in #253, the latest summerwind/actions-runner-dind images keeps crashing pods:
[Wed Jan 27 13:04:45 UTC 2021] [INFO] [/usr/local/bin/startup.sh] Starting supervisor
[Wed Jan 27 13:04:45 UTC 2021] [INFO] [/usr/local/bin/startup.sh] Waiting for processes to be running
[Wed Jan 27 13:04:45 UTC 2021] [INFO] [/usr/local/bin/startup.sh] Process dockerd is not running yet. Retrying in 1 seconds
[Wed Jan 27 13:04:45 UTC 2021] [INFO] [/usr/local/bin/startup.sh] Waited 0 seconds of 30 seconds
[Wed Jan 27 13:04:46 UTC 2021] [INFO] [/usr/local/bin/startup.sh] Process dockerd is not running yet. Retrying in 1 seconds
[Wed Jan 27 13:04:46 UTC 2021] [INFO] [/usr/local/bin/startup.sh] Waited 1 seconds of 30 seconds
[Wed Jan 27 13:04:47 UTC 2021] [INFO] [/usr/local/bin/startup.sh] dockerd is running
Github endpoint URL https://github.com/
/runner should be an emptyDir mount. Please fix the pod spec.

Setup:

actions-runner-controller: v0.16.1
summerwind/actions-runner-dind:v2.274.2 (OK)
summerwind/actions-runner-dind:v2.275.1 or latest (FAILS)

I ran into this as well.
It can be fixed/worked around by adding the following to each of your RunnerDeployment specs:
      volumeMounts:
       - mountPath: /runner
         name: runner
       volumes:
       - emptyDir: {}
         name: runner

I ran into this as well.
It can be fixed/worked around by adding the following to each of your RunnerDeployment specs:
      volumeMounts:
       - mountPath: /runner
         name: runner
       volumes:
       - emptyDir: {}
         name: runner

Sure, that worked for me too, but wondering why this isn't done by the controller.

Sure, that worked for me too, but wondering why this isn't done by the controller.

This is the working RunnerDeployment:
apiVersion: actions.summerwind.dev/v1alpha1
kind: RunnerDeployment
metadata:
  name: foobar-group-runnerdeploy
spec:
  replicas: 2
  template:
    spec:
      organization: foo
      group: bar
      image: summerwind/actions-runner-dind:v2.274.2
      dockerdWithinRunnerContainer: true
      resources:
        limits:
          cpu: "4000m"
          memory: "2Gi"
        requests:
          cpu: "200m"
          memory: "200Mi"

...but this workaround is needed for the latest tag:
apiVersion: actions.summerwind.dev/v1alpha1
kind: RunnerDeployment
metadata:
  name: foobar-group-runnerdeploy
spec:
  replicas: 2
  template:
    spec:
      organization: foo
      group: bar
      image: summerwind/actions-runner-dind:latest
      dockerdWithinRunnerContainer: true
      resources:
        limits:
          cpu: "4000m"
          memory: "2Gi"
        requests:
          cpu: "200m"
          memory: "200Mi"
      volumeMounts:
      - mountPath: /runner
        name: runner
      volumes:
      - name: runner
        emptyDir: {}

This is the working RunnerDeployment:
apiVersion: actions.summerwind.dev/v1alpha1
kind: RunnerDeployment
metadata:
  name: foobar-group-runnerdeploy
spec:
  replicas: 2
  template:
    spec:
      organization: foo
      group: bar
      image: summerwind/actions-runner-dind:v2.274.2
      dockerdWithinRunnerContainer: true
      resources:
        limits:
          cpu: "4000m"
          memory: "2Gi"
        requests:
          cpu: "200m"
          memory: "200Mi"

...but this workaround is needed for the latest tag:
apiVersion: actions.summerwind.dev/v1alpha1
kind: RunnerDeployment
metadata:
  name: foobar-group-runnerdeploy
spec:
  replicas: 2
  template:
    spec:
      organization: foo
      group: bar
      image: summerwind/actions-runner-dind:latest
      dockerdWithinRunnerContainer: true
      resources:
        limits:
          cpu: "4000m"
          memory: "2Gi"
        requests:
          cpu: "200m"
          memory: "200Mi"
      volumeMounts:
      - mountPath: /runner
        name: runner
      volumes:
      - name: runner
        emptyDir: {}

I got the same. Since #253 it is required to have a /runner directory mounted as emptyDir. The controller does not add these directories for dind images (or more specifically, when dockerdWithinRunnerContainer: true)
I think the emptyDir mount can be done regardless of when running in DinD mode or not. This is straightforward to add to the controller.
@mumoshu do you agree? (I don't understand all the details of the fix you made in #253)

Seems to be working with the current master now. I'm closing this one.


Add synchronized to the builder

This might have caused the NPE during the sync process seen by the community.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11036/
Test FAILed.

alluxio-bot, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11037/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11041/
Test PASSed.

@HelloHorizon PTAL

@yuzhu
Why does this cause an NPE? I don't think the synchronized is necessary, because in that part of the method, there is only 1 thread accessing that builder.

@gpang this is not the cause of the NPE. closing it now.


TextLayer does not render well on Windows/Chrome

Windows with Chrome 65. Nvidia Quadro M2000.
Actual Result

Expected Result
Proper rendering.
Reproduce Steps

Run layer browser
View TextLayer

To Do List

[ ] Add label and assign to milestone
[ ] Coding
[ ] Test

@georgios-uber After investigation the root cause is from chrome's inconsistent text alignment on windows and osx for some font faces, microsoft edge couldn't guarantee the correctness for each font face either. One possible solution in short term is that we can expose an API to specify a font face which you can choose to make sure it works for chrome 65. How do you think?

This issue still reproduces on 5.3.0-alpha.1


Rename Contract.wallet to Contract.account

Or maybe setSenderAddress and getSenderAddress in case we want to set it to a contract-based account?

Getter/Setter should be renamed too.
Refs: https://github.com/tasitlabs/TasitSDK/pull/357#discussion_r279087773

Good question. We can have these situations:

Same account as the signer and sender;
An account as sender and one as the signer;
An account as sender and many as the signers;

Good question. We can have these situations:
* Same account as the signer and sender;

* An account as sender and one as the signer;

* An account as sender and many as the signers;

Yep. Supporting and testing all of those situations is important (and maybe outside the scope of this issue based on its current name?)

For short-term we can just rename from wallet to `account and then split this issue to continue the discussion of how should be the Contract API to support the situations above. Make sense?

For short-term we can just rename from wallet to `account and then split this issue to continue the discussion of how should be the Contract API to support the situations above. Make sense?

Yep, perfect!

Discussion about API extracted to: https://github.com/tasitlabs/TasitSDK/issues/375


Removed hardcoded dependency on App\Controller\AppController

Should be perfectly backwards compatible (not tested though).
Usage, in app.php:
'OAuthServer' => [
    'appController' => 'MyApp\Controller\AppController'
]

PHPCS does not like the code in OAuthController, it can be moved to bootstrap.php, but that would require adding 'bootstrap' => true option while loading the plugin.

I don't think a hardcoded App\Controller\AppController is a bad thing. In fact it is a standard amongst all CakePHP plugins that have a controller. I would rather document how to create a class alias in the application code if you are not using the standard App namespace.

Hm, I never came across a plugin that required me to create a class alias. Can you maybe give me an example?

The CakeDC Users plugin is one such example: https://github.com/CakeDC/users/blob/master/src/Controller/AppController.php
It doesn't require you to create a class alias, rather you need to create one if you are not using the standard App/ namespace.

I see.
What would you do, if you wanted to use 2 such plugins in one project, and wanted each to extend different app controller? You can't alias App\Controller\AppController twice. But each plugin can have its own alias.
Users and OAuthServer might not be the best examples, because they both deal with users :-)
But for example we want to have oauth's app controller separate from the rest of the app (mainly because the main app controller does things that we don't want to do in oauth - like load shopping cart, etc.).

But for example we want to have oauth's app controller separate from the rest of the app (mainly because the main app controller does things that we don't want to do in oauth - like load shopping cart, etc.).

Our solution here is conditional loading of things in our app controller. At this stage I'm not entirely sure what the best solution is though :)


Allow get_data() function to accept URLs

-would need to ignore all other arguments if a URL is supplied

url should just be a GET request


[backport] Fix the docstring format of cupy.asarray

Backport of #2821

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 58568ee6057371ccf62c555636611322d5682d7e, target branch v6) succeeded!

LGTM.


Add support for simple values and rename "prim" to "simple"

C libraries: Rename prim/primitive to simple
Which is what it's called in RFC8949.
Includes exposing new APIs: zcbor_simple_(encode/put/decode/expect)
Add some tests for the new APIs.
Note: The use of "Prim" in zcbor.py is different and is not touched by
this change.

@hakonfam The use of "Prim" in zcbor.py is different and is not touched by this change.


Card tables need responsive treatment on narrower screens like deck tables

The colspan headers are going to be a problem here.

Basic version is not ideal.

Applying FooTable to /cards/ even in a cut-down version takes 10s. We may need a less intense solution.

I did this. I don't apply FooTable to tables >1000 entries, I just let them overflow-x: auto aka scroll-in-place. Smaller card tables get the full footable treatment.


admin CLI which will bootstrap new projects

Describe the problem feature solves
Right now when you want to run the project you have to install and setup everything on your own From the express setup to the database etc. But in some cases, you would like to bootstrap the entire project from scratch in an organised way, with the authentication, orm models etc.
Describe the solution you'd like
I would add a new CLI command admin new which will ask for:

language: typescript / javascript
orm/odm: mongoose, sequelize, typegoose, typeorm
plugin: hapi, express, nestjs, nextjs, firebase functions

Then it will create entire project, install packages and create scripts.
Acceptance criteria

add new or generate command to admin cli (cli.ts)
command will create admin.ts file which runs the admin panel
it will bootstrap folder structure
it will create models for authentication.
ideally there will be another command for adding resources.

CLI concept changed - closing this one


Unable to add new resource from local tile server (in local maputnik)

I am running maputnik in my localserver and I try to add a new source from my local tile server. I add a Vector XYZ url but nothing appears on the map:
http://localhost:8080/data/KingCo_2000_Census_BlockGroups/{x}/{y}/{z}.pbf
I also tried adding with this combination:
http://localhost:8080/data/KingCo_2000_Census_BlockGroups/{z}/{x}/{y}.pbf
I get no errors on the inspector.
Anyone can advise me how to make it work? I need to add the source to style the layer and get information about the source-layer (is there another way to get information about the source-layer?)

@karakostis did you check the console and network logs from the browsers inspector? The following thread has some debugging tips: https://github.com/maputnik/editor/issues/208

@pathmapper yes I have checked that thread.

Closing here - if this is still an issue, please let us know and we'll reopen the issue.

Hi,
I am having sort of the same problem:
I am able to display succesfully this group of geoserver layers in qgis with no problem, and through the Network activity panel in qgis I see all tiles served:
http://localhost:8080/geoserver/gwc/service/tms/1.0.0/burgos-callejero<EMAIL_ADDRESS>On the other hand, when adding it as a Vector (XYZ URL) in maputnik version Editor: 1.7.0; Desktop: 1.0.7, I don´t even get any signal of it in my browser network.
If I do http://localhost:8080/geoserver/gwc/service/tms/1.0.0/burgos-callejero<EMAIL_ADDRESS>or http://localhost:8080/geoserver/gwc/service/tms/1.0.0/burgos-callejero<EMAIL_ADDRESS>nothing happens also.
What I migh be missing?
Thx

Greetings,
I, too, am having the same sort of problem.
I logged an Issue #905 because I could not get Maputnik to see (or even make requests to) data served from a MapLibre Martin server on port 3000 in the form of ZXY.
http://localhost:3000/jlptiles/{z}/{x}/{y}
The Martin server which I built was validated with QGIS.
My Issue was converted into Discussion.
I launched my martin server and maputnik, both on a Debian server, and then tried accessing maputnik from my Window workstation using Firefox and 2 CORS errors appeared without my doing anything within the interface.  Here are the error messages copied form the Firefox console in full:

03:12:41.617 Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://localhost:8888/styles. (Reason: CORS request did not succeed). Status code: (null).
03:12:42.183 Failed to process sources for 'https://api.maptiler.com/tiles/v3-openmaptiles/tiles.json?key=get_your_own_OpIi9ZULNHzrESv6T2vL' SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data App.tsx:671:11


element_mask is removed. who know how to use *mask.reshape((mask.size, 1, 1, ..., 1)) ?

element_mask is removed. Please use src*mask.reshape((mask.size, 1, 1, ..., 1)) directly as binary ops now support broadcasting.
I update the mxnet version to 0.9.1 in the window. I play the lstm, but 'module' object has no attribute 'element_mask'.
please help me for a hint and code example in python to use *mask.reshape((mask.size, 1, 1, ..., 1)) , replace the element_mask.

I also want ask why the element_mask is removed? Then how can we mask the padding data to 0 in rnn model.

@ZiJianZhao yes, why remove the element_mask ? element_mask  is very good to use!

@whaozl you can use broadcast_mul as a substitution of the element_mask.

thank you very much for the hint!

how to use broadcast_mul,.... i tried , but wrong

You can follow the api instruction in the mxnet.io website. if you can't get it right, then maybe you mxnet version is not right or anything else. I'm not sure.


build: introduce buildhelper replacing git-semver

Replace git-semver, which we were using in the build step,
with another tailor-made tool superset of it.
What we gain is: we can now report the git commit from which
the build is done, and we can inject the values from outside,
bypassing the autodetection, to be friendlier with isolated build
systems, which would otherwise fail the autodetection.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>

should be cherry-picked to release-4.10 and release-4.11

/cherry-ppick release-4.11

/cherry-pick release-4.11


move command line to TOPP tools?

Would it make sense to move:
https://greengypsy-staging-openms.readthedocs.io/en/final-touches/docs/openms-applications-and-tools/command-line-interface.html
somehow into:
https://greengypsy-staging-openms.readthedocs.io/en/final-touches/docs/openms-applications-and-tools/topp-tools/introduction-to-topp.html
because it is kind of a concept of the TOPP tools.

Read Existing Documentation
I have checked the following places for the error:

[ ] OpenMS Documentation Troubleshooting
[ ] OpenMS API Developer FAQs

Create OpenMS Bug Report
I have created the bug report in OpenMS for any pipeline errors.

[ ] OpenMS Open Issues

Description of the documentation bug

[ ] I have attached screenshots of the incorrect documentation

Expected behaviour

OpenMS Installation

Version:

Additional context

The User Quickstart guide contains a lot of specific information for TOPP tools, such as how to add an INI file or change parameters in CLI or TOPPAS.
Could the Quickstart be restructured to be an overview that links to the specific pages instead?

In the TOPP section, we could mention that you can call a TOPP tool in different ways like CLI, KNIME, Galaxy...

If we do that, then I think it should be called a Command Line Interface Quickstart guide.

The Quickstart guide should be very general such as:

What is OpenMS?
Background (the table that's already their)
Get familiar with TOPP tools (underline their central role)
Choose your way to use OpenMS (use pre-defined workflows, build your own workflow, scripting)
Installation on Windows, Linux, Mac buttons


Move code from TrainerOptimizersMixin into TrainingTypePlugin

Proposed refactor
Move all the code from this class to TTP:
https://github.com/PyTorchLightning/pytorch-lightning/blob/2036dfb5df2a48056d1c043ee311aed793187070/pytorch_lightning/trainer/optimizers.py#L28
Motivation
After #10596 the TTP will own the optimizer related logic previously owned by Accelerator. It makes sense for the TTP to also own the optimizer related code in TrainerOptimizersMixin. This would allow us to avoid this type of circuitous logic: https://github.com/PyTorchLightning/pytorch-lightning/blob/dcafc95f2b0fd3f176d425139ca99676ce943a12/pytorch_lightning/plugins/training_type/training_type_plugin.py#L244-L245
We in general want to reduce the number of mixins (#10417)
cc @justusschock @awaelchli @akihironitta

Thanks for the initiative.
I propose to only move the contents of the init_optimizers method to the TTP. In there, we would check if the model has a configure_optimizers method and call it.
The static methods could become functions and be part of core (LightningModule) because they are responsible for parsing the output of the LightningModule hook. They don't belong into the TTP imo.

@awaelchli Sounds good!!

and be part of core (LightningModule)

@awaelchli do you mean in lightning.py?

Yes, basically in core/lightning.py or core/optimizer.py or somewhere there where it's close to the hook that retrieves the optimizers and schedulers from the user (configure_optimizers).

@awaelchli is something like this what you had in mind?
class LightningModule(...):

    @abstractmethod
    def configure_optimizers(self):
        ....

    def init_optimizers_and_lr_schedulers(self) -> TypedDict[...]:
        output = self.configure_optimizers()
        # parse & validate output
        # return components

then we could also schematize/add typing for configure_optimizers so all the typechecking & validation is in one place, with the strategy calling init_optimizers as opposed to configure_optimizers

@ananthsub it sounds like what you are saying is to move init_optimizers to the LM as well, rather than putting it in TTP?

@ananthsub it sounds like what you are saying is to move init_optimizers to the LM as well, rather than putting it in TTP?

I think the caller of configure_optimizers should be the training type plugin. However, the definition of the parsing & validation logic from the output of configure_optimizers should be shared.


Support for native builds for bare metals instead of Docker container

For Multicore OCaml, we would like to be able to build and run benchmarks on bare metal machines. It will be good to create an OCurrent pipeline just for native builds, while, still using the PostgreSQL database and frontend containers.

Do you want this pipeline to be triggered on PRs or to run it on a branch or both?
I'll try to prototype this to see what it would look like in the current pipeline, but we wouldn't want repositories who enroll in current-bench to run their benchmarks outside of docker.

Do you want this pipeline to be triggered on PRs or to run it on a branch or both?

Both.

but we wouldn't want repositories who enroll in current-bench to run their benchmarks outside of docker.

The requirement here is to run on bare metal and not in Docker, as we don't want the overhead of the Docker layer.

With hyperthreading disabled, the Docker results are close to bare metal as observed in https://github.com/ocurrent/current-bench/issues/151. Hence, this is not a requirement anymore.


webapp: I was trying to update an application in python django for the first time.

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp up
Errors:
'NoneType' object has no attribute 'reserved'
Traceback (most recent call last):
Temp\pip-install-yf22vmg2\knack\knack\cli.py, ln 215, in invoke
azure\cli\core\commands\__init__.py, ln 654, in execute
azure\cli\core\commands\__init__.py, ln 718, in _run_jobs_serially
azure\cli\core\commands\__init__.py, ln 709, in _run_job
azure\cli\command_modules\appservice\commands.py, ln 55, in _polish_bad_errors
azure\cli\core\commands\__init__.py, ln 688, in _run_job
azure\cli\core\commands\__init__.py, ln 325, in __call__
site-packages\azure\cli\core\__init__.py, ln 545, in default_command_handler
azure\cli\command_modules\appservice\custom.py, ln 3237, in webapp_up
AttributeError: 'NoneType' object has no attribute 'reserved'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp up --resource-group {} --name {}

Expected Behavior
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.6
Installer: MSI

azure-cli 2.6.0

Additional Context

webapp up

@paviomuniz please try the latest version we had some regressions in this version that we have since fixed. Please re-open if you are still seeing this with the latest version of CLI


chore: add typeclasses to unify various add_top, add_eq_top, etc.

Add the four typeclasses IsTopAbsorbing, IsBotAbsorbing, NoTopSum, NoBotSum, as additive equivalents for MulZeroClass and NoZeroDivisors. Add instances of these for ENNReal, WithTop α, WithBot α, PUnit, EReal, PartENat, Measure and Filter.
Also split Algebra/Order/AddGroupWithTop to Algebra/Order/Group/WithTop and Algebra/Order/Monoid/WithTop

Previous usages of lemmas with quantified names like WithTop.add_top have to be changed to just add_top.
add_lt_top is @[simp], in accordance with ENNReal.add_lt_top being @[simp]. This affects WithTop.add_lt_top which previously hadn't been @[simp].

Sorry, I am not yet convinced these classes are worth having. Can you remind me:
1. What instances you want to have that involve these new typeclasses.

2. What lemmas you expect these classes to generalise.

There are the instances in this PR, for ENNReal, WithTop α, WithBot α, PUnit, EReal, PartENat, Measure and Filter. Additionally, every canonically ordered monoid is a NoBotSum, as is PNat, but I'm not sure if that's useful. (I now see I missed Interval in this PR I'll add it soon).
There are add_top/top_add/add_bot/bot_add, which currently exist for LinearOrderedAddCommMonoidWithTop, WithTop, PartENat, Measure, WithBot, Filter, Interval, EReal. There are add_eq_top/add_eq_bot, which exist for WithTop, PartENat, ENNReal, WithBot, Filter, EReal (and could exist for LinearOrderedAddCommMonoidWithTop but is missing), as well as add_eq_zero_iff for canonical monoids (although that should probably remain separate). Additionally, I think it might generalize Filter.bot_sub with EReal.bot_sub and Filter.sub_bot with Interval.sub_bot, although I'm not quite sure about that.

So no instance is parametric? no instance depends on the new typeclasses? (as in take in one of the new typeclasses as an argument, rather than as an output)
Why can't we just add the missing lemmas for the specific types? Is it that you are trying to avoid a name clash?

It's really important that we can justify every typeclass we add to mathlib because, once a typeclass exists, people feel encouraged to generalise lemmas to it even when it is not mathematically useful to do so, and this makes those typeclasses look crucial when they are not, and also hard to remove.
I worry that the typeclasses in the present PR generalise few lemmas (< 10) from few types (< 10) which are very different mathematically very different (in particular are not thought to be different incarnations of the same algebraic structure, the way eg ℤ and ℝ are rings), meaning that those typeclasses have no backing on paper. Further, the list of generalised lemmas seems very sparse among the lemmas about top in the API of the respective types, meaning that people will find themselves using top lemmas specific to their types, except sometimes where the lemma they want just happened to be generalisable to one of your new typeclasses. This is cognitive overload.

I think wanting to avoid a name clash is one of the main reasons for this, yes.

Additionally, looking at MulZeroClass and NoZeroDivisors many of the multiplicative lemmas for them have additive equivalences here, which I think is also part of my reason for this, although arguably many of them aren't useful.
Not direct equivalence:

MulZeroClass.negZeroClass is equivalent to stating that in a IsTopAbsorbing SubtractionMonoid we have -⊤= ⊤ (and the same for bottom).
WithZero.mulZeroClass, WithBot.instMulZeroClass, WithTop.instMulZeroClass, WithBot.instNoZeroDivisors, WithTop.instNoZeroDivisors: these are mostly the contents of this PR, except for WithZero preserving IsTopAbsorbing/IsBotAbsorbing/NoTopSum/NoBotSum
Directly equivalent:
mul_eq_zero_of_left, mul_eq_zero_of_right
eq_zero_of_mul_self_eq_zero
mul_self_eq_zero, mul_self_ne_zero
zero_eq_mul_self, zero_ne_mul_self
mul_eq_zero_comm, mul_ne_zero_comm
mul_eq_zero, mul_ne_zero, mul_ne_zero_iff
mul_zero_eq_const, zero_mul_eq_const
eq_zero_of_ne_zero_of_mul_left_eq_zero, eq_zero_of_ne_zero_of_mul_right_eq_zero
left_ne_zero_of_mul, right_ne_zero_of_mul
mul_eq_zero_of_ne_zero_imp_eq_zero
ne_zero_and_ne_zero_of_mul
ite_zero_mul, mul_ite_zero
ite_zero_mul_ite_zero
SemiconjBy.zero_left, SemiconjBy.zero_right
Commute.zero_left, Commute.zero_right
instMulZeroClassLex
instMulZeroClassOrderDual
IsLeftRegular.subsingleton, IsRegular.subsingleton, IsRightRegular.subsingleton
isLeftRegular_zero_iff_subsingleton, isRegular_iff_subsingleton, isRightRegular_zero_iff_subsingleton
more regular stuff
Pi.mulZeroClass
pow_eq_zero, pow_ne_zero
sq_eq_zero_iff
pow_eq_zero_iff, pow_ne_zero_iff, pow_eq_zero_iff'
List.prod_eq_zero_iff, List.prod_ne_zero
Multiset.prod_eq_zero_iff, Multiset.prod_ne_zero
Finsupp.prod_ne_zero_iff, Finsupp.prod_eq_zero_iff
isSquare_zero

Another use of this I just noticed is that Multisets are a NoBotSum

Another instance of this I just noticed is that Multisets are a NoBotSum

That's already covered by the fact that they are a canonically ordered monoid
Do you still need this PR now that your SubtractionMonoid instance is merged?

I don't need it, but I still think it's good


logzio-otel-k8s-metrics v0.0.2

@tamir-michaeli This PR refers to the gh pages. what did you change in the helm chart?

@yotamloe https://github.com/logzio/logzio-helm/pull/63


Save button not responding.

Describe the Bug with repro steps
when I make chagnes to logic app the save button doesn't work - I have to move back to GA designer to make changes
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Edge
Additional context
No response

Hi @parasdesaicims, thank you for the report, could you give some more details on the issue you're experiencing so we can better look into this?

This would likely be due to validation errors somewhere. This may be partially fixed in the designer update rolling out now, we should look into ways of upleveling errors even better though.


fix: dep update

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore(tests/falco): drop debug printlns.

/hold
in case i need to also fix some bugs.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: FedeDP
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/unhold

/hold wait for ci

/unhold
ci fails because Falco master CI was not able to push updated images (with the fix) because it got stuck at testing-packages of course.
Works locally against a Falco built from master:
./build/falco.test -falco-binary /home/federico/Work/falco/build/userspace/falco/falco -test.run TestFalco_Miscs_HotReload -test.v
=== RUN   TestFalco_Miscs_HotReload
{"deadline":180000000000,"level":"info","msg":"running falco with runner","time":"2024-05-09T14:44:32+02:00"}
{"cmd":"/home/federico/Work/falco/build/userspace/falco/falco -c /home/federico/Work/testing/hot_reload_enabled.yaml -r single_rule.yaml -M 5 -o engine.kind=nodriver -o log_level=debug -o log_stderr=true -o log_syslog=false -o stdout_output.enabled=true","level":"debug","msg":"executing command","time":"2024-05-09T14:44:32+02:00"}
--- PASS: TestFalco_Miscs_HotReload (7.66s)
PASS

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


BasicJC1 is not correctly implemented

BasicJC1 is not correctly implemented. It's broadcasting a context to a user channel and trying to listen for it coming back in the same app, yet the FDC3 1.2 Spec states:

However, a Desktop Agent should ensure that context messages broadcast to a channel by an application joined to it should not be delivered back to that same application.

Its a recommendation so most agents should follow it (if not all) and hence this test can't rely on it. You either need to use another app or remove the broadcast from the test (presumably this already covered by another test anyway).
https://github.com/finos/FDC3-conformance-framework/blob/main/src/test/v1.2/basic/fdc3.joinChannel.ts/#L45-L58

In Finsemble we do deliver the message back on joinChannel, if it was sent before the channel was joined (as current context on the channel does not retain who sent it). However, I think its probably best just to draw back to the original test definition and make sure the functions are present and callable and leave validating listen/broadcast behavior to the advanced tests

resolved


Not working, can't sign in or change signed-in account !

Dear Support,
Recently today, I faced this issue while trying to switch google accounts in a game, I tried to remove cache and app data of the game, also I did it for the play service, but still not working, I tried another game, the same thing happened, no action fires when I click on 'Sign In'.
Please help, waiting your reply
Thank you
Mohammad Al Asmar

I restarted my smartphone, and everything worked well, Sorry :D !


Systems are not removed from database

Issue
It appears that when the systems array is written to the database, items that are in positions greater than the current length of the systems array are not removed.

Reproduction Steps:

Tag a brew with all four of the systems in the Metadata Editor tab.
Save the brew.
Return to the Metadata Editor and untick the Pathfinder systems box
Save the brew.
Refresh the page.
Check the Metadata Editor - Pathfinder has returned.

Second Repro:

Tag a brew with one or more systems.
Save.
Untick all systems boxes.
Save.
Refresh.
All ticks have returned.

Third Repro:

Tag a brew with any two non-5e systems.
Save.
Tick 5e.
Save.
Untick one non-5e system.
Save.
Untick the other non-5e system.
Save.
Refresh - 5e is the only remaining system, but cannot be removed. Checking this brew in the database will show that '5e' is in all positions 0-2 in the systems array.

The bug is here:
https://github.com/naturalcrit/homebrewery/blob/d15bec08a3cb1c80fdb34a28f2bf50f50112a597/server/homebrew.api.js#L55
_.merge({arr:[1,2,3]}, {arr:[5,6]}) produces {arr: [5, 6, 3]} .. i.e. _.merge recursively merges objects and elements, so target.arr[0] is replaced with source.arr[0] (etc).
Thus:
_.merge({systems:[5e,4e,3.5e,PF]}, {systems:[4e,3.5e,PF]}) produces {systems:[4e,3.5e,PF,PF]}.
Additionally, this: https://github.com/naturalcrit/homebrewery/blob/d15bec08a3cb1c80fdb34a28f2bf50f50112a597/client/homebrew/editor/metadataEditor/metadataEditor.jsx#L32-L39
wasn't written with the possibility that the system being pushed into the array is already in the array, and so by particular sequences of updates, saves, and reloads we end up with systems:["5e","4e","3.5e","3.5e","4e","4e","4e"].

We can keep using _.merge() (to handle future fields, generally)m but line 55 in homebrew.api.jsx does need to change.
Either

save the pre-merge brew and after _.merge manually re-set systems (and also authors for the same reason, and also views because the browser shouldn't be setting the views count to whatever old value it got when the editpage was loaded); or
_.merge(object, source, callback, thisArg) is a thing and we can thus control which elements get merged, which get replaced, and which get ignored.

Lodash's _.union looks like a good drop-in replacement.] for _.merge which I think will fix part of this.

_.union only works on arrays, not objects.
_.union([5,3.5], [4]) ⇒ [5,3.5,4] .. which is the same as _.merge

_.merge(object, source, callback, thisArg) seems to be the way. The main problem with _.merge is it is a deep merge.
Wait ..

_.merge(object, [sources])
This method is like _.assign except that it recursively merges [...]

Ah, _.assign is a shallow merge. Which is what we want.
var object = {
  'title': 'string',
  'systems': [3.5,4,5]
  };

var other = {
  'title': 'new string',
  'systems': [5]
  };

_.merge(object, other); // => {title: "new string", systems: [5, 4, 5]}
_.assign(object, other); // => {title: "new string", systems: [5]}

Except, separate from #1221, we still want to deep merge for authors, and to ignore views i.e. maintain the server value of views. Which are separate issues (not yet logged into github).

Just tested, this is no longer an issue, closing.


Quick Open (Ctrl+P) should not show the current file at the top

Ctrl+P is useful to quckly open files, but it's not useful to show the current file at the top, since you are in that file already, so when you want to open an other recent file, like the previous recent file or one of the more recent files below it then you have to press an extra down arrow key in the list to get there.
Quick open should not show the current file on the top of the list, it should at most put it at the bottom, since the current file is the most unlikely file you want to "open".
Alternatively, if it is shown at the top then it should not be selected by default, the file below it should be the one selected by default when Quick Open is invoked.

You can bind Ctrl+P to workbench.action.quickOpenPreviousEditor to select the second entry by default in the file/editor picker.

Thanks, but I meant the built in default. Putting the current file to the top selected by default is pretty useless, isn't it?

@codecoll it allows a user to dismiss the picker by simply pressing Enter instead of reaching for Esc. Changing the default behaviour will break this.

ESC is the general dismiss key, why should Enter be used here for this? Selecting the previous file by default is much more useful.

If you care about selecting a previous entry quickly via quick pick, there is an alternative that I am using since day 1: it allows to operate quick pick without the input field and just by keybindings and in that mode the second item will be selected by default. It works a bit like Alt-Tab for switching applications and can be accessed when you press Ctrl+Tab:

@codecoll in situations like this where we have a very straightforward way of configuring a different behavior, we are wary to change so that we don't mess with the muscle memory of users.
Since there's a simple alternative in this case, I'm gonna close this as as-designed.


Timeout for one request makes the others time out too

First of all, thanks for creating Typhoeus. We (Bogrobotten) are using it to fetch hundreds of thousands of URLs every day. It really is a great library.
However, it seems like when we use Typhoeus::Hydra to fetch e.g. 50 URLs and one of them is taking a long time to respond, and we have a timeout of 10 seconds on each request, all of them time out, even though only one request was taking a long time to respond.
Is this the expected behavior, and could we do something to prevent a single slow request from affecting the other requests in the hydra?
Thanks in advance.

I got the same Problem - posted it on google-groups (bevor I've see this issue).

I got the same Problem - posted it on google-groups (bevor I've see this issue).

Thanks for providing more informations @zt2 @PascalTurbo.
@PascalTurbo I think your understanding of timeouts is correct and I will look into that problem.

We are seeing this problem as well. Tested with: 10 queued requests in Hydra to S3

+1 please
We are seeing this problem as well. Tested with a loop:

10 queued requests in Hydra to S3.
Timeout of 2 seconds.
Each request on average takes 150-250ms.

We see an exception thrown afte the test has been running for a short time. The strange thing is that it nearly always (95%) responds with a status_code = 0 response, as seen in this issue: https://github.com/typhoeus/typhoeus/issues/411.
We're running jruby-1.7.19.

This is really messing me up too, if one request times out the whole queue does to hell.

Does this still happen?  I'd be willing to take a look if someone writes a spec for it.

@jpmoral do you want a test file.rb that recreates the error?

@revett
Sure, that would be helpful.

@revett
Here's what I used:
https://gist.github.com/jpmoral/1765e88f6dc526397660
I was able to reproduce the issue last week, I can't seem to now.  I used @zt2's code and changed it a bit.    A spec would be much better, I'm still working on one.

@jpmoral It'd be easier just putting a #sleep call in the Sinatra handler.

@Zapotek
Yes, it would.  :)

you can pass a delay for each request to the test server: https://github.com/typhoeus/typhoeus/blob/master/spec/support/server.rb#L81.

Any progress on this? I was going to take a crack at it but I was happy to see work already underway.

@nathanhoel
I tried to write a spec but I haven't been able to reproduce the issue again.  Do you have specs or code that reliably demonstrates it?

No, I have not tried yet that is why I was asking about it to see if I should invest anytime in it.
I can see how this would be difficult to reproduce.

I'm getting it when running against remote servers but can't seem to reproduce it against a local sinatra test server. I'll keep trying.

This is pretty serious. Here is an easy way to reproduce: https://gist.github.com/att14/edfceea56e434642aa1f
Whats the priority on this?

@att14 thanks for reporting. This is the most serious bug I am aware of right now.

Any news on this? I looked at the code and it seems to all be in Ethon. Is there a related ticket? If someone could describe the problem to me I could take a stab at fixing it.

@att14 you are right, the problem is in Ethon. There is no related ticket yet. I didn't run into that myself. What do you want to know exactly?

Well, I don't know anything about Ethon. Could you point me at what you believe to be the offending code?

@att14 the hydra libcurl interaction mostly happens in https://github.com/typhoeus/ethon/blob/master/lib/ethon/multi/operations.rb.

I have done some testing on this issue today and think I have made some progress.
The first thing I did was write a test based on code provided by @att14 which shows the error occurring by creating 200 connections to the local web server one being a timeout. Surprise, surprise it started failing
$ rspec spec/ethon/multi/operations_spec.rb:189
Run options: include {:locations=>{"./spec/ethon/multi/operations_spec.rb"=>[189]}}

Ethon::Multi::Operations
  #perform
    recovers from timeout
      only the request that timed out returns an error (FAILED - 1)

Failures:

  1) Ethon::Multi::Operations#perform recovers from timeout only the request that timed out returns an error
     Failure/Error: expect(easies.map(&:response_code).select(&:zero?).count).to eql 1

       expected: 1
            got: 155

       (compared using eql?)
     # ./spec/ethon/multi/operations_spec.rb:205:in `block (4 levels) in <top (required)>'

Finished in 0.17516 seconds
1 example, 1 failure

Failed examples:

rspec ./spec/ethon/multi/operations_spec.rb:204 # Ethon::Multi::Operations#perform recovers from timeout only the request that timed out returns an error

So 155 URLs failed to be fetched
The next thing I tried was reducing the number of urls to see if the test passed. When run with a pool of 50 urls and the result.
$ rspec spec/ethon/multi/operations_spec.rb:189
Run options: include {:locations=>{"./spec/ethon/multi/operations_spec.rb"=>[189]}}

Ethon::Multi::Operations
  #perform
    recovers from timeout
      only the request that timed out returns an error

Finished in 5.17 seconds
1 example, 0 failures

So why does this work with 50 urls but not 200. The answer is in the inner workings of libcurl multi, it seems that curl multi does no internal throttling of connections if you give it 1000 urls it will go off an simultaneously try to fetch all those urls.
For each url in the queue we need a file descriptor available so lets see how many file descriptors i can use
$ ulimit -n
256

so what happens if we increase the value and rerun the test with 200 urls again
$ ulimit -n
256
$ ulimit -n 1024
$ ulimit -n
1024
$ rspec spec/ethon/multi/operations_spec.rb:189
Run options: include {:locations=>{"./spec/ethon/multi/operations_spec.rb"=>[189]}}

Ethon::Multi::Operations
  #perform
    recovers from timeout
      only the request that timed out returns an error

Finished in 1 minute 6.39 seconds
1 example, 0 failures

Voila!. So increasing the max file descriptors made the test pass. It appears as-is you need enough available file descriptors for the number of urls in the curl multi queue.
So whats the long term fix?
It appears from this thread on the curl mailing list we will need to implement our own connection throttling inside Typheous so this is my next job!
Can anyone else confirm these findings?

@ptrippett Awesome! I'm not able to fully confirm your findings, but it sounds reasonable. I did some testing a few days ago and from what I could tell the issue was inside of libcurl, but I was not brave enough to find out where.

@ptrippett: @att14's code triggers the issue for me, but I've got a ulimit -n of 1024.

@ptrippett thanks for you work!!

The answer is in the inner workings of libcurl multi, it seems that curl multi does no internal throttling of connections if you give it 1000 urls it will go off an simultaneously try to fetch all those urls.

Thats not exactly what happens I belive. According to http://curl.haxx.se/libcurl/c/CURLMOPT_MAXCONNECTS.html it only creates a connection for every 4th easy:

By default libcurl will enlarge the size for each added easy handle to make it fit 4 times the number of added easy handles.

It is possible that I am interpreting this wrong. And maybe the details are irrelevant if you can help it by increasing the limits. Maybe there is more than one FD for every easy.

Edit: Around this point I confused myself being in Ethon talking about queues forgetting Hydra is actually in Typheous!

Ya, I can see how that happened. The Hydra equivalent in Ethon is Multi.

Could it be that the select stuff need to track and adjust individual timeouts for connections (kind of like this) instead of what's going on now?

@Zapotek the example you found seems to be a timeout for checking a socket. In libcurl though the timeouts are more - like the connect_timeout. It is impossible for Ethon to do that manually.

@Zapotek Your right it does still fail. I had to modify the server slightly because of io_fillbuf errors in webrick which I dont think are part of this same problem so I replaced webrick with thin
require "sinatra"
require "thin"

set :server, 'thin'

get "/" do
  "Hello world!"
end

get "/timeout" do
  loop {}
end

I then started paying with the max_connections option to hydra.
require_relative "./lib/typhoeus" # I am requiring relative because I am debugging

# Setting (max_concurrency: 20) here depending on the FD's available
#
hydra = Typhoeus::Hydra.new(max_concurrency: 20)

requests = 1000.times.map { Typhoeus::Request.new('localhost:4567', method: :get, timeout: 5) }
requests << Typhoeus::Request.new('localhost:4567/timeout', method: :get, timeout: 5)

requests.each { |request| hydra.queue request }

hydra.run

requests.each do |request|
  if request.response.code != 200
    p "ERROR #{request.response.code}"
  end
end

I started incrementing the max_connections variable on each run and I seem to be ok up to ~100 max connections after that I start seeing odd logs coming from thin.
Successful request:
<IP_ADDRESS> - - [27/Sep/2015:13:35:30 +0200] "GET / HTTP/1.1" 200 12 2.3658

Unsuccessful request:
- - - [27/Sep/2015:13:35:31 +0200] "GET / HTTP/1.1" 200 12 0.7198

Notice theres no requesting host in the failed log line but but the servers returning a 200 status.

I was using thin in my test and failures do increase with the max_concurrency.
I was getting errors with a concurrency of 20, both in the test and in real world applications -- 10 seemed to do fine.

About the failing log line, the connection must have been closed after the request was received by the server, way before the timeout was reached though, since the response took only 0.7198.
I'm just guessing.

Is there any update on this?

So this basically leaves us with massive numbers of false responses, and there's been no update on this in over two years?

Nope.  Not that i’m aware of.  Still somewhat busted.
On Sat, Aug 27, 2016 at 1:59 AM, Daniel Miessler<EMAIL_ADDRESS>wrote:

So this basically leaves us with massive numbers of false responses, and
there's been no update on this in over two years?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/typhoeus/typhoeus/issues/417#issuecomment-242898630,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQE5TSL0JTnTrj-o1SVLqKbxCoC6t9Rks5qj9IngaJpZM4C_f6A
.

A new ethon was released with a bugfix for a similar problem, maybe it fixes this one as well: https://github.com/typhoeus/ethon/pull/125.

I believe this is resolved with recent versions of ethon.


Text format is different when opened using pdf.js, Acrobat Reader and Google Chrome

Attach (recommended) or Link to PDF file here:
pdf-file.pdf
Configuration:

Web browser and its version: Chrome 76
Operating system and its version: macOS High Sierra 10.13.6
PDF.js version: tested using https://mozilla.github.io/pdf.js/web/viewer.html on 190827
Is a browser extension: no

Steps to reproduce the problem:

Load the document
Open page 1, and look for text 'To solve this, we will need the EM algorithm, which stands for Expectation Maximization.'
Select text and right click.
Note that there is extra spacing between characters.

What is the expected behavior? (add screenshot)
Expect text to have no spacing like when it is opened using Acrobat Reader and Chrome browser.

What went wrong? (add screenshot)
Extra spacing between each character.

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

I want to work on it!  Please assign me.

It's yours!

Why only exact specific line in that pdf showing with extra spaces on selection ?

Why only exact specific line in that pdf showing with extra spaces on selection ?

The entire purpose of this issue is to figure out why that happens, and then once you've found the answer try to write a patch that fixes the problem (without breaking something else in the process).
It honestly seems somewhat strange to first ask to be assigned to the issue, implying that you intend to debug/fix it, and then essentially ask someone else to debug it for you!?
(Whether or not this is a good beginner bug probably depends a lot on prior familiarity with the PDF spec.)

I understand @Snuffleupagus I just wanted some more insight. I will fix it


More docs for the Core csproj reference

Expanded the csproj Core reference with more things:

Added the Sdk attribute
Added the section on implicit package refs
Added the section on default compile includes
Finished a lot of known NuGet input properties

/cc @rohit21agrawal for correctness of the NuGet packing inputs.

@rohit21agrawal thanks for the timely review, added the changes you suggested.

I'll review the topic now but as I glance through the topic on staging my feeling is that we should have a separate topic for the MSBuild reference for .NET Core projects that is separate from this topic (that is, IU'd move the additions section to this new topic).

What do you mean?

From: Maira Wenzel<EMAIL_ADDRESS>Sent: Wednesday, February 15, 2017 6:20:26 PM
To: dotnet/docs
Cc: Zlatko Knezevic; Author
Subject: Re: [dotnet/docs] More docs for the Core csproj reference (#1553)
I'll review the topic now but as I glance through the topic on staging my feeling is that we should have a separate topic for the MSBuild reference for .NET Core projects that is separate from this topic (that is, IU'd move the additions section to this new topic).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs%2Fpull%2F1553%23issuecomment-280209944&data=02|01|Zlatko.Knezevic%40microsoft.com|5f8d545c5f464952800208d456125ef2|72f988bf86f141af91ab2d7cd011db47|1|0|636228084281519770&sdata=1QQ8B79DxTX1Bn2kvd7PQrKZxH0TFf%2FegsKTfDy%2BXdI%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAC-H_MlCo4Ed6eeR0QRwBZeNVq7_9DGQks5rc7JqgaJpZM4MAEmT&data=02|01|Zlatko.Knezevic%40microsoft.com|5f8d545c5f464952800208d456125ef2|72f988bf86f141af91ab2d7cd011db47|1|0|636228084281519770&sdata=x5W%2BNDnm%2F6zZ0dBJDfMhyenwooiNhqJlmVfOFeS%2FlCc%3D&reserved=0.

@blackdwarf I've addressed most of my own comments, Please take a look.
In addition to the previous feedback I gave (that this topic should probably be split in two), I also don't see the behavior described in the "Migrating a project" section anymore. Should we remove that entire section?
Thanks!

@mairaw sure, remove that section.

@mairaw Doc looks good. The RuntimeIdentifiers element is still a bullet i.s.o. heading though.

Thanks @EdwinVW! Yep, I was reviewing those heading while I was making my changes. I think they should look good now.

I've removed the PackageTypes property for now. If I find out otherwise that we should keep it, I'll edit the topic later.


 Will need to make a modal or something

My branch


Is work_dirs/mv-3ddet/epoch_12.pth == https://download.openxlab.org.cn/models/wangtai/EmbodiedScan/weight/mv-3ddet.pth?

Branch
main branch  https://mmdetection3d.readthedocs.io/en/latest/
📚 The doc issue
Can't find information on which model is work_dirs/mv-3ddet/epoch_12.pth
Suggest a potential alternative/fix
No response

Please download the pretrained model weights here and change the path accordingly.


I want to translate to Korean

Hello sir.
I want to translate Daedalus program for Korean
But, I don't know how to translate in this way... Do I need to get admission to translate?

I believe if you want to translate to Korean you simply have to edit this file:
https://github.com/input-output-hk/daedalus/blob/master/app/i18n/locales/ko-KR.json
You can look at the files for other languages for examples.

Can I translate terms of use too?
I'm still working with new files

Hi, based on this, I also translated this into spanish if you want: https://github.com/input-output-hk/daedalus/pull/653


Unit testing for methods in recommendation_graphs.py

Now that they're broken out, it should be easy to test-wrap them.

[ ] project_biases
[ ] split_sparse_tensor_indices
[ ] bias_prediction_dense
[ ] bias_prediction_serial
[ ] gather_sampled_item_predictions
[ ] rank_predictions

https://github.com/jfkirk/tensorrec/pull/39


new org assigned to Gabriel Bobis: AI For Good Foundation

check https://github.com/riceissa/aiwatch/commit/29a81c4c0c9fa099770e3bbd4ef4b5c0eb6c4b22

The organization has changed the presentation of its team on the website over the years, and in doing so, I detected at least 5 different URLs (http://ai4good.org/about/committees/ - http://ai4good.org/about/ - http://ai4good.org/team/ - https://ai4good.org/about-us/our-team/, among others).
There was a temporal gap in 2019 and especially in 2020, where very few updates were registered. Although there are many files in 2023, there was very little change in the members. On the other hand, some members detailed the months and years they worked in the organization, and in the "wayback" updates, their names remained listed several months after their departure, which is why I prioritized LinkedIn.
I chose "AI safety" as the organization's "CAUSE AREA" due to the contributions they make and based on the organization's description.


[assistant] Context Codable issue

We have a Kitura app that talks to the Assistant Service and as part of restoring the conversation between calls we encode/decode the Context using standard Swift JSONEncoder/JSONDecoder. Running locally on a Development Mac this runs fine, but when deploying to the IBM Cloud, decoding fails.
We use the following extension on Context:
extension Context {
static func decode(string: String) throws -> Context? {
guard let data = string.data(using: .utf8) else { return nil }
Log.debug("Context.decode: decoding '(string)'")
do {
let context = try JSONDecoder().decode(Context.self, from: data)
Log.debug("Context.decode: Sucessful '(context)")
return context
} catch let error {
Log.error("Context.decode: Failed '(error)")
throw error
}
}
func encode() throws -> String? {
let data = try JSONEncoder().encode(self)
let string = String(data: data, encoding: .utf8)
Log.debug("Context.encode: Sucessful '(string ?? "")'")
return string
}
}
And decode fails with:
[DEBUG] [xxx.swift:44 decode(string:)] Context.decode: decoding '{"conversation_id":","system":{"_node_output_map":{"Welcome":{"0":[0]}},"dialog_request_counter":1,"dialog_stack":[{"dialog_node":"root"}],"branch_exited":true,"dialog_turn_counter":1,"branch_exited_reason":"completed"}}'
[ERROR] [xxx.swift:50 decode(string:)] Context.decode: Failed 'typeMismatch(Swift.Dictionary<Swift.String, Any>, Swift.DecodingError.Context(codingPath: [], debugDescription: "Expected to decode Dictionary<String, Any> but found an array instead.", underlyingError: nil))
We are using:
Swift-sdk: 0.30.0
Kitura: 2.4.1
We have tried with both 4.1.1 and 4.1.2 of Swift with the same result .

You don't have to decode the context value yourself -- the SDK already decodes it for you.  All the "dynamic" properties are be contained in the additionalProperties dictionary of the context object.

Not sure I follow. Since this a web app (Kitura) we assumed we needed to attach the context to the web session, so that a user can continue their conversation separate from other users. The encode/decode are part of session restoration. Thanks for responding! We will continue investigating.

Closing as there should be no need to encode/decode the Context when using the SDK.  Please open a new issue if there is some other problem here that needs to be addressed.


ci-gke: Set useDigest=false for Hubble Relay

As with all other *-ci images, we should not use check the digest when pulling the image, as it is different for CI builds. On main this has not been an issue, because useDigest already defaults to false, but on release branches (such as v1.14), we do check the digest by default. This caused failing workflows on the v1.14 branch which should be fixed by this commit.
This extends the previous PR which was missing this change for the GKE workflow.
Fixes: 98003d5fef7b ("ci/github: Set useDigest=false for Hubble Relay")

/ci-gke

the button is not green 😿

It can't be green, but it's still ready-to-merge in our hearts (but I will leave the privilege to @gandro).

/test


Serialize all models into cluster metadata

Description
This PR removes a check that only serialized models into cluster metadata when they were in the "created" state.  This means that any model in "training" or "failed" states could only be accessed through blocking transport requests.  This PR also minimizes the transport calls in TrainingJobClusterStateListener, instead accessing model metadata from the cluster metadata.
Check List

[X] New functionality includes testing.

[X] All tests pass

[X] New functionality has been documented.

[X] New functionality has javadoc added

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

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

@ryanbogan and check from here: https://github.com/opensearch-project/k-NN/blob/7a144b84f8af1e178054d34cd716ce139209f1b5/src/main/java/org/opensearch/knn/index/query/KNNWeight.java#L213-L221

and here: https://github.com/opensearch-project/k-NN/blob/7a144b84f8af1e178054d34cd716ce139209f1b5/src/main/java/org/opensearch/knn/index/IndexUtil.java#L25

@ryanbogan and check from here:
https://github.com/opensearch-project/k-NN/blob/7a144b84f8af1e178054d34cd716ce139209f1b5/src/main/java/org/opensearch/knn/index/query/KNNWeight.java#L213-L221

I'll add a check to get the state

Windows failures are unrelated to this PR


Where is the softcapping implementation?

Tanh soft logit capping (following Gemma 2).

I don't see it anywhere in the code, I believe the score_mod should be passed to flex_attention.
See https://github.com/pytorch-labs/attention-gym/blob/5e0d1b8053a19b339ddf7c8015f2c4e02bf5b92c/attn_gym/mods/softcapping.py#L13

Pardon, I mixed up logit softcapping and attention scores softcapping.
Am I understanding it correctly that the attention scores are not soft-capped because we do not try to model long sequences here?


Linux builder forced to host platform system instead of it's package variant

Describe the bug
A clear and concise description of what the bug is.
Steps To Reproduce
Steps to reproduce the behavior:

On Mac Silicon, Utilizing with nix-darwin, i've the folllowing :

{
    package = pkgs.darwin.linux-builder-x86_64;
}

I always get qemu-system-aarch64 as qemu binary and then an aarch64 linux builder

Expected behavior

uname -a should give x86-64_linux
Qemu binary program in (https://github.com/NixOS/nixpkgs/blob/master/nixos/lib/qemu-common.nix) should not be forced to hostPlatform system for this linux builder case

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers
@Enzime
Metadata
 - system: `"aarch64-darwin"`
 - host os: `Darwin 23.5.0, macOS 14.5`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.21.2`
 - nixpkgs: `/Users/loic/.nix-defexpr/channels/nixpkgs`

Did not find any issues related to this qemu program choice.

I think the qemu binary is set correctly in the builders, in this file: https://github.com/NixOS/nixpkgs/blob/7950c8d688ebb5c751a1314ca723df2a8e3ca536/nixos/modules/virtualisation/qemu-vm.nix#L685
I was also having this problem, and found this issue while looking into it.  But I realized the result is being caused by these lines in nix-darwin:
  builderWithOverrides = cfg.package.override {
    modules = [ cfg.config ];
  };

(https://github.com/LnL7/nix-darwin/blob/master/modules/nix/linux-builder.nix#L10-L12)
That looks fine, but the builder package we're trying to use is also created using override!
  linux-builder-x86_64 = self.linux-builder.override {
    modules = [ { nixpkgs.hostPlatform = "x86_64-linux"; } ];
  };

So the override in nix-darwin is overriding the override in nixpkgs, making the resulting package have the default hostPlatform.
You can work around this by setting that value in your nix darwin config, like this:
 nix.linux-builder = {
  enable = true;
  package = pkgs.darwin.linux-builder-x86_64; # This line currently doesn't matter! But see my linked PR.
  systems = ["x86_64-linux"]; # set this to make nix on your laptop send the right kinds of derivations to the builder
  config.nixpkgs.hostPlatform = "x86_64-linux"; # This is what nix-darwin is removing, and what you need to add back
};

I've opened a pull request to make this work the way we expected it to: https://github.com/LnL7/nix-darwin/pull/974
Thank you for this issue, it confirmed that I wasn't the only person having this problem, and led me to find the cause. 😄

Thanks a lot !


[CSApply] Attempt value-to-opaque-result abstraction only after canon…

…icalization
ExprRewriter::coerceToType should canonicalize contextual type before
attempting to use it for value abstraction, because sugared type could
have typealias references that hide their underlying opaque result types.
Resolves: rdar://98577451

@swift-ci please clean test

@swift-ci please test


Implement Recursive Feature Elimination (RFE) algorithm

Implement RFE. Match Scikit Learn's implementation.
These are parameters we should support:

estimator
n_features_to_select
step
verbose

I'd like to try this issue!


Storybook > Elements >Stat>Default>Dark theme Icon color is not change

Describe the bug

Stat>Default>Dark theme Icon color is not change.
Team Member - Two star spacing is require.

To Reproduce
Steps to reproduce the behavior:

Go to 'Elements'
Click on 'Stat'
Click on 'Default'

Screenshots

Desktop (please complete the following information):
-OS: [Windows 10 Enterprise]
-Browser [ edge]
-Version [ 125.0.2535.92]
Smartphone (please complete the following information):NA

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Completed-

@trivenikulkarni
Closed by Rameshwar in version 1.1.6, which is available in version 1.1.7

version 1.1.7


MODELIX-851 Add option to exclude modules by name in bulk-sync plugin

Aside from the new feature this PR also includes a functional test for the plugin. The already existing tests should be migrated in MODELIX-1002
To be verified by reviewers

[ ] Relevant public API members have been documented
[ ] Documentation related to this PR is complete

[ ] Boundary conditions are documented
[ ] Exceptions are documented
[ ] Nullability is documented if used

[ ] Touched existing code has been extended with documentation if missing
[ ] Code is readable
[ ] New features and fixed bugs are covered by tests

Could you fix all the detekt warnings by either adding a comment or supressing them when it makes more sense?

Could you fix all the detekt warnings by either adding a comment or supressing them when it makes more sense?

Done

Could you fix all the detekt warnings by either adding a comment or supressing them when it makes more sense?

Done

I meant supressing them with an annoation in the code.
By that they will not bother us while coding.

:tada: This PR is included in version 8.18.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to use on Delphi

Hello, is it possible to somehow make these libraries friends with Delphi?
Need to convert phrase to ETH address !

This Delphi?

This Delphi?

Yes Delphi XE,  RAD Studio or others pascal lang

Actual still looking for a way how to use for Pascal (Delphi)


Consider adding bitflags helpers

Most likely after bitflags v2 since it will release with a new BitFlags trait

This would require Specialization; it would be more practical to instead derive ShaderType for bitflags. And since bitflags are just a newtype wrapper, maybe we can just allow deriving ShaderType for all newtypes.


Improving performance on gcsfuse mounted partitions

Hi,
I'm trying to enable FTP/SFTP access to Google Cloud bucket, by using Pure-FTP server on Google instance. However I'm experiencing some weird issue when I'm trying to delete file trough FTP, you can see this issue in more details here.
I'm suspecting that the ftp server is failing to address the DELETE request properly due to some small delays in gcsfuse mounted partitions.
So my question is if it's there any way that I can solve this by tuning up gcsfuse performance. My current mount settings are:
backup /home/backup gcsfuse rw,auto,allow_other,uid=1001,gid=1001,file_mode=0777,dir_mode=0777
Thanks

This sounds like it belongs on GoogleCloudPlatform/gcsfuse, not here. Closing this; please open there if you've still got concerns after the responses on the linked issue.
(If you do reopen there, I will need more details about what you think is happening. It may be worth knowing that renaming files is supported in gcsfuse and should be atomic, and renaming directories is not supported.)


Add Travis and Codecov integration

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e9d9399). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   83.76%
=========================================
  Files             ?        5
  Lines             ?      191
  Branches          ?        0
=========================================
  Hits              ?      160
  Misses            ?       31
  Partials          ?        0

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 e9d9399...fa72625. Read the comment docs.


Cannot compile map! macro using hash function's$... in rhs callback when mode is set in symbol

First of all, the callback of both vim.api.nvim_set_keymap and vim.keymap.set will not be called with any arguments unlike nvim_create_autocmd.


Just what I wanted, but!

Just what I wanted to output IGC format files, excellent. One small point you don't say what units '*_alt" is in.  I guess in the IGC format spec it says but it would make life easier for the user of aerofiles if it was stated in the documentation.

Feel free to open a PR :wink:

So I did what I should have done and looked up the IGC format spec and the answer is "metres"; unsurprisingly!


CPU is missing mib-std2-pq-zr-toolbox

I downloaded mib-std2-pq-zr-toolbox And copied as per procedure on the SD card but when I put it in the stereo I do not see the CPU folder and I do not see the START button to update. How come? Do I need to change cpu folder with some other? In addition I wanted to know if the radio should ever crash what needs to be done to restore it. Thank you.

Send your actual firmware version

MST2_EU_SE_ZR_P0253T I only have one SD card slot

Seat Leon Year 2016

MST2_EU_SE_ZR_P0253T I only have one SD card slot

You just answered yourself. https://github.com/olli991/mib-std2-pq-zr-toolbox#faq-frequently-asked-questions

I didn't understand Should I use the latest toolbox I update the firmware?

MST2_EU_SE_ZR_P0253T I only have one SD card slot

Not possible

Do I need to update the firmware?

Just read again from the beginning, this solution is not suitable for your unit

So everyone with an SD card slot can't patch the radio to activate full link? There must be a way......

Yes there's is, not easy but possible, if you read from the beginning, need tools and computer and soldering skills

Yes, tell me what I have to do regarding the electrical part and weld? Weld you do it or even at work

Where can I find a tutorial for both cases?

My problem is just activating full link then the rest does not interest me.


Where is TotalEventCPU in Report.pkl (JobAccountant)

I have no time to look at it, so just throwing this error message that I have just seen in vocms053 for the JobAccountant component.
2015-02-26 21:10:34,900:INFO:AccountantWorker:Handling /data/srv/wmagent/v1.0.0.patch4/install/wmagent/JobCreator/JobCache/fabozzi_RVCMSSW_7_4_0_pre8RunHI2011__RelVal_hi2011_150225_173436_9069/RECOHID11/RECOHID11MergeALCARECOStreamTkAlMinBiasHI/JobCollection_38166_0/job_296904/Report.0.pkl
2015-02-26 21:10:35,000:ERROR:ChangeState:Error updating job in couch: 'TotalEventCPU'
2015-02-26 21:10:35,001:ERROR:ChangeState:Traceback (most recent call last):
  File "/data/srv/wmagent/v1.0.0.patch4/sw.pre/slc6_amd64_gcc481/cms/wmagent/1.0.0.patch4/lib/python2.6/site-packages/WMCore/JobStateMachine/ChangeState.py", line 156, in propagate
    self.recordInCouch(jobs, newstate, oldstate, updatesummary)
  File "/data/srv/wmagent/v1.0.0.patch4/sw.pre/slc6_amd64_gcc481/cms/wmagent/1.0.0.patch4/lib/python2.6/site-packages/WMCore/JobStateMachine/ChangeState.py", line 319, in recordInCouch
    updateSummaryDB(self.statsumdatabase, job)
  File "/data/srv/wmagent/v1.0.0.patch4/sw.pre/slc6_amd64_gcc481/cms/wmagent/1.0.0.patch4/lib/python2.6/site-packages/WMCore/JobStateMachine/SummaryDB.py", line 157, in updateSummaryDB
    sum_doc = fwjr_parser(document)
  File "/data/srv/wmagent/v1.0.0.patch4/sw.pre/slc6_amd64_gcc481/cms/wmagent/1.0.0.patch4/lib/python2.6/site-packages/WMCore/JobStateMachine/SummaryDB.py", line 84, in fwjr_parser
    pdict['totalEventCPU'] += float(perf['cpu']['TotalEventCPU'])
KeyError: 'TotalEventCPU'

Just in case, I kept the Report.pkl file in:
/afs/cern.ch/user/c/cmst1/public/forAlan/Report.0.pkl

Fixed by @hufnagel in the last couple of months.


Loading dependency graph...jest-haste-map: Haste module naming collision

System:
OS: Linux 4.15 Ubuntu 18.04.3 LTS (Bionic Beaver)
CPU: (4) x64 Intel(R) Core(TM) i5-3340 CPU @ 3.10GHz
Memory: 838.14 MB / 7.68 GB
Shell: 4.4.20 - /bin/bash
Binaries:
Node: 10.16.3 - /usr/local/bin/node
npm: 6.9.0 - /usr/local/bin/npm
npmPackages:
react: 16.8.6 => 16.8.6
react-native: 0.60.5 => 0.60.5
npmGlobalPackages:
react-native-cli: 2.0.1
Hello Dev's
m new to React Native
i m using react-native CLI not expo
When i hit the npm start on the terminal, am getting the error
Error Start From below
Loading dependency graph...jest-haste-map: Haste module naming collision: HelloWorld
The following files share their name; please adjust your hasteImpl:
* /package.json
* /node_modules/react-native/template/package.json
Failed to construct transformer:  { Error: Duplicated files or mocks. Please check the console for more info
at setModule (/home/deepak/ReactNativeProject/HelloWorld/node_modules/jest-haste-map/build/index.js:620:17)
at workerReply (/home/deepak/ReactNativeProject/HelloWorld/node_modules/jest-haste-map/build/index.js:691:9)
at process._tickCallback (internal/process/next_tick.js:68:7)
mockPath1: 'package.json',
mockPath2: 'node_modules/react-native/template/package.json' }
(node:30995) UnhandledPromiseRejectionWarning: Error: Duplicated files or mocks. Please check the console for more info
at setModule (/home/deepak/ReactNativeProject/HelloWorld/node_modules/jest-haste-map/build/index.js:620:17)
at workerReply (/home/deepak/ReactNativeProject/HelloWorld/node_modules/jest-haste-map/build/index.js:691:9)
at process._tickCallback (internal/process/next_tick.js:68:7)
(node:30995) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:30995) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Error End Here
i try many solutuion but didnt work for me
Please help me

I had this error, it was cause by bad Pod imports syntax. I fixed it by running a fresh new react-native project and copying the correct syntax from their clean Podfile for the pods I was using.

Where i found the Syntax and what i do


Make sure studies page table styles are good and make mobile friendly

Consider using 2fr vs 1 fr on column widths


Service loopback

@aanm Adapted to reserve an IPv4 address out of the alloc range for service loopback use. This addresses the clash concern mentioned in the previous commit message.
I think we can merge the IPv4 part first and see if this is sufficient and performant and then add the IPv6 part as we have some experience with it.


Server cleanup

A few somewhat unrelated things, but separate cleanups for ease of review.

I was careful not to change much of significance in server.h, shouldn't be too conflicty (and I don't expect you'll be doing too much in server.cc itself, other than maybe remove a method?)...

I see merge conflicts.

I'm looking at your changes in 291276eba06fd29c5f2b81e1bd8f13eb0ae2306d, and I see many different, distinct changes that would be worth their own PRs (or, at the very least, commits). :wink:
Like "moving the flags to server.cc", "add Server::ProvideDatabase", "replace CertificateHandler ctor parameters with Server*", etc...

I refer to the Hon. Gentleman to his PR entitled "A few somewhat unrelated
things".
On 4 November 2015 at 15:15, Pierre Phaneuf<EMAIL_ADDRESS>wrote:

I'm looking at your changes in 291276e
https://github.com/google/certificate-transparency/commit/291276eba06fd29c5f2b81e1bd8f13eb0ae2306d,
and I see many different, distinct changes that would be worth their own
PRs (or, at the very least, commits). [image: :wink:]
Like "moving the flags to server.cc", "add Server::ProvideDatabase",
"replace CertificateHandler ctor parameters with Server*", etc...
—
Reply to this email directly or view it on GitHub
https://github.com/google/certificate-transparency/pull/1028#issuecomment-153758407
.

(or, at the very least, commits)

:wink:

I haven't created a PR yet. I can still split it up.
On 4 November 2015 at 15:21, Pierre Phaneuf<EMAIL_ADDRESS>wrote:

(or, at the very least, commits)
[image: :wink:]
—
Reply to this email directly or view it on GitHub
https://github.com/google/certificate-transparency/pull/1028#issuecomment-153760610
.

Indeed, it's a mere question of "git rebase" (what could go wrong?). :laughing:


add a schema version on releases of NMDC schema

We discussed using semantic versioning or simply a time stamp on release versions.  Either could work, to avoid needing to define the semantics of semantic versioning (does minor or patch break ingest or only major?) we decided time stamp release versions was simplest.
With versions on the schema it becomes possible to ask what schema version the metadata store (MongoDB or Terminus) is at and what version the search application ingest and relationship schema support

@wdduncan I'm familiar with the following scheme (which I used for https://github.com/polyneme/nmdc-mongo-tools):
# among dev requirements
invoke
setuptools-scm
twine

# in /setup.py

from setuptools import setup, find_packages

setup(
    packages=find_packages(),
    use_scm_version=True,
...

# /task.py
from subprocess import run

from invoke import task

@task
def publish(c):
    completed = run(
        "git describe --tags $(git rev-list --tags --max-count=1)",
        shell=True,
        capture_output=True,
        encoding="utf-8",
    )
    vtag = completed.stdout.strip()
    major_minor, dot, patch = vtag.rpartition(".")
    vtag_new = major_minor + dot + str(int(patch) + 1)
    run(["git", "tag", vtag_new])
    run(["git", "push"])
    run(["git", "push", "--tags"])
    c.run("rm dist/*.*", warn=True)
    c.run("python setup.py sdist bdist_wheel")
    c.run("twine upload dist/*")

Here, invoke publish on the command line bumps the patch version via a git tag and pushes that to PyPI. The logic above can be refactored to use calver (e.g. YYYY.MM.DD).

The nmdc-schema is now versioned using pypi.


we should get standardese working with Kvasir

https://github.com/foonathan/standardese

DoxyPress is also another option.