Theming for css with primitives

There are a couple things to note in this implementation:

temporary dev dependency on primer/css
With primer/primitives@8 just around the corner, we are using primitives v8 in the component with a fallback on primitives v7
For v7, css variables for primitives are shipped in primer/css, which is why we have a devDependency on primer/css to test in storybook. This is only till primitives v8 are shipped in dotcom (~ 1-2 weeks), we can remove primer/css from our setup after that.

data attributes added to BaseStyles
color primitives change value color modes, this works by setting data attributes on the root of your application. See documentation.
With primer/react, this should be taken care by ThemeProvider, but because ThemeProvider does not render any wrapper elements, I have chosen to add them to BaseStyles instead, which is used along with ThemeProvider (including in dotcom)
This includes mapping the additional color modes for primer/react to primer/primitives (day → light, night → dark). We should look into deprecating these in the future!

@langermank Need to confirm if this is a breaking change or a bug,
In primer/css/color-modes.css, [data-color-mode="light"][data-light-theme="dark"] is a valid combination (same for all dark themes)
see code on unpkg

but in primer/primitives/tokens-next-private/css/, this isn't true:
screenshot:
see code on unpkg
screenshot:

@siddharthkp quick question, in the dark themes it seems like the border tokens are not being defined. Just wanted to check-in to see if this was intended with this setup or not 👀

It seems like potentially for these combos the custom properties are not being set at the theme provider level

@joshblack Yep! Noted that in https://github.com/primer/react/pull/3238#issuecomment-1534825375
Seems like a bug (or a intentional breaking change) in primitives, talking to @langermank about it.

In primer/css/color-modes.css, [data-color-mode="light"][data-light-theme="dark"] is a valid combination (same for all dark themes)

@siddharthkp this has been removed, as we figured that there is no way to get this combination in the wild. A few people checked it, but maybe we missed something. If we did, how can you get to this combination?

@siddharthkp this has been removed, as we figured that there is no way to get this combination in the wild.

That's perfect! Happy to drop support for it :)

For storybook, setting colorMode based on compatible scheme now. Looks much better:


[Nightly] Sent reactions don't show until app-refresh

I dont think this is still an issue. Closing.


Add data directory option to storage settings

This is a follow-up issue to this comment in the PR for adding data directory selection UI during onboarding.
Idea is that we also need to show the directory later during main application use.
Here's a quick mock up of what that could look like.

I'd like to clarify the user actions here. There's opening that directory in the native finder, and changing the directory. I am curious what's involved in changing the directory. Is already downloaded data moved over? And do we need to inform users about what is happening? If this is a more involved process, then we may need a dedicated page to handle it. Otherwise it could be just a simple interaction.

@GBKS thanks for starting this issue... It is addressed in here https://github.com/bitcoin-core/gui-qml/pull/397
I like having the address under the header will look into how to pull that off :)

I'd like to clarify the user actions here. There's opening that directory in the native finder, and changing the directory. I am curious what's involved in changing the directory. Is already downloaded data moved over? And do we need to inform users about what is happening? If this is a more involved process, then we may need a dedicated page to handle it. Otherwise it could be just a simple interaction.

If we want to allow users to change the datadir yeah, we would need to move the data over if the sync process already started and yeah we'll need to inform the user about all this process is about (perhap adding a new page as you said).
Also, when you start the app with a specific datadir from the command line (e.g.: bitcoin-qt datadir="/some/datadir"), we'd need to inform it there on the same "Data directory" field, maybe we could indicate that this was passed from the command line as well, which is not persisted (bypassing the datadir that has been configured during onboarding) and only "lasts" until the app shuts down, calling the bitcoin-qt again without passing -datadir will use the datadir saved in the settings.

Is it a good idea to notify people explicitly about what parameters they passed in and what the effect will be?
Here's a quick mock-up, which could probably be extended further.

Temporarily changing the data directory just for the lifetime of a single run sounds very messy. Would it move data to that directory during the run, and then the application would not find any data anymore during the next run and start from scratch?

Is it a good idea to notify people explicitly about what parameters they passed in and what the effect will be?

I'd like that, it's something currently QT doesn't have, but there is a request about the same on that repo.

Here's a quick mock-up, which could probably be extended further.

I'd also indicate in the "permanent" "Storage settings" that the datadir was passed, leaving the persisted/ configured/ onboarded datadir visible but with a mark/ link to this temporary settings section (I'm thinking loud about it, it makes sense to me, but let's see if we follow this path it makes it clearer for the user or not).

Temporarily changing the data directory just for the lifetime of a single run sounds very messy.

It's something that's used in both bitcoind and bitcoin-qt, sometimes a user would like to verify some stuff without changing current env setup (e.g. release testing).
Would it move data to that directory during the run, and then the application would not find any data anymore during the next run and start from scratch?
If I follow your question correctly, yes, using -datadir or not, if there are already blocks sync-ed on the directory (or default datadir) it would continue from there, if there's nothing, it would start from scratch. If you remove data during the run, depending what, you would make the app to crash and making the data to be in a inconsistent state.

Would it move data to that directory during the run, and then the application would not find any data anymore during the next run and start from scratch?

Also, adding a useful reference of the documentation about the files stored in the data directory.

Thanks for those details. It sounds like we have two different user needs here.

 Maybe their hard drive got full and they bought an external drive they want to start using. So they want to permanently move data to this new place.
A developer needs to regularly switch between environments. An environment might be using testnet and storing blocks in the blocks_testnet folder, or using mainnet and using the blocks_mainnet folder. They manage multiple data sets and want control.

A direction here could be that the GUI is optimized for the first use case, and we rely on devs using the command line.
So if the user changes the directory in the GUI, the existing files are moved. If the directory is passed in from the command line, no files are moved.
I'll try to mock something up about the temporary overrides.

Thanks for those details. It sounds like we have two different user needs here.
1.  Maybe their hard drive got full and they bought an external drive they want to start using. So they want to permanently move data to this new place.

If a user wants to move the data from where it is to another folder or drive, at the moment, it has to do a manual process, I'm not aware of such tool or feature.

2. A developer needs to regularly switch between environments. An environment might be using `testnet` and storing blocks in the `blocks_testnet` folder, or using `mainnet` and using the `blocks_mainnet` folder. They manage multiple data sets and want control.

Yes, the different chains have their own subdir within the datadir (default or custom). Please check in the datadir location doc, if there's something unclear please let me know so we can update it.

A direction here could be that the GUI is optimized for the first use case, and we rely on devs using the command line.

Not sure if it's something other bitcoin core devs want (to support?), maybe it's worth it to ask on the IRC channel #bitcoin-core-gui.

So if the user changes the directory in the GUI, the existing files are moved.

If we allow them to do it, again, not sure if that's what we want. (If the following is confusing, please ignore it, sorry for the unnecessary tech details) This path it's also specify in the config file Bitcoin-Qt.conf (in current QT you can see it in its contents something like strDataDir=/home/pablo/.bitcoin). (For reference, PR https://github.com/bitcoin-core/gui-qml/pull/398 it's for us to use a different file from QT).

If the directory is passed in from the command line, no files are moved.

If a user passes -datadir="..." from the command line, the above config will be bypassed so the block files and all datadir files will be different.

Does that make any sense? I fell like I'm overthinking or forgetting something.

Yes, better to clarify everything in advance.

Based on this, I thought we could/would move data for the user, but from your other statement it sounds like that is not the case (or not possible). Can you please clarify?

It is possible, unless there's something I'm not aware of, just saying we need to warn the user and make the user aware of the steps, and from the dev side be prepare to handle issues related with the procedure.

I think what I am trying to resolve is how the user flow of changing the directory later would work. Let's say I synced a good amount of data, but now want to move, and I only use the app (not config or command line).
If I start the application, go to settings, and change the directory, what happens? Do I need to restart?

I'd need to test this once the wiring (settings saved/ persisted in the config file) is done, but as in current QT (not QML) I imagine, if you change the settings file manually or thru the app in order to the changes being taken into account you'd need to restart, but if you restart and there are no files there, it will start the sync from scratch.

Do I need to manually move the data, or can the application do it for me?

Currently, you'd need to move the data manually but you need to make sure the app is not running otherwise you can brake it. Currently the the app doesn't do it for you, I think it's doable to add the feature, not sure if it's convenient at this stage.

What do you think is best based on how the code/logic works at the moment?

Until we define if we add the migration/ moving datadir feature (scope, approach), better not to let the user change the datadir after onboarding (current QT doesn't allow it).

Thanks. Let me try to summarize an approach.
V1 (of this feature)

Users can set the directory during onboarding
Settings shows the directory and points user to manually editing bitcoin.conf for further changes (close app, move files, edit bitcoin.conf, start app)

V2

Users can also change the directory in settings
App automatically stops the node, moves files over, and restarts the node (no restart required)

Does that sound right?

V1 (of this feature)
* Users can set the directory during onboarding

* Settings shows the directory and points user to manually editing `bitcoin.conf` for further changes (close app, move files, edit bitcoin.conf, start app)

Yeah, for V1 we should only display the datadir selected during onboarding on the settings making it not editable/ read-only. Not sure if there's an official bitcoin core documentation already on how to migrate/ move datadir with all the steps if the user needs to, but if it is (or once it is) I'd add a link to that doc next to the displayed datadir.

V2
* Users can also change the directory in settings

* App automatically stops the node, moves files over, and restarts the node (no restart required)

Yes, I'd think so.

Does that sound right?

Yes


feature: add gammaln function support.

Description
This PR implements gammaln function. It's the same as the log gamma function of torch.lgamma, mxnet.nd.gammaln.
It is used in the distributions package of torch and mxnet to calculate the probability density at the value, and this result can be used for the loss function calculation of the predicting probability distribution model. Here is its usage in pytorch and mxnet.
pytorch
def log_prob(self, value):
        if self._validate_args:
            self._validate_sample(value)

        log_unnormalized_prob = (self.total_count * F.logsigmoid(-self.logits) +
                                 value * F.logsigmoid(self.logits))

        log_normalization = (-torch.lgamma(self.total_count + value) + torch.lgamma(1. + value) +
                             torch.lgamma(self.total_count))
        log_normalization[self.total_count + value == 0.] = 0.

        return log_unnormalized_prob - log_normalization

mxnet
def log_prob(self, value):
        if self._validate_args:
            self._validate_samples(value)
        F = self.F
        lgamma = gammaln(F)
        binomal_coef = lgamma(value + self.n) - \
            lgamma(1 + value) - lgamma(self.n)
        # log(prob) may have numerical issue.
        unnormalized_log_prob = self.n * \
            F.np.log(self.prob) + value * F.np.log1p(-self.prob)
        return binomal_coef + unnormalized_log_prob

Codecov Report

Merging #1990 (d86b967) into master (bb5073f) will decrease coverage by 2.31%.
The diff coverage is 67.70%.

@@             Coverage Diff              @@
##             master    #1990      +/-   ##
============================================
- Coverage     72.08%   69.77%   -2.32%
- Complexity     5126     5902     +776
============================================
  Files           473      584     +111
  Lines         21970    26163    +4193
  Branches       2351     2824     +473
============================================
+ Hits          15838    18256    +2418
- Misses         4925     6532    +1607
- Partials       1207     1375     +168

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

.../cv/translator/InstanceSegmentationTranslator.java
0.00% <0.00%> (-86.59%)
:arrow_down:

...nslator/InstanceSegmentationTranslatorFactory.java
7.14% <0.00%> (-11.04%)
:arrow_down:

.../cv/translator/SemanticSegmentationTranslator.java
0.00% <0.00%> (ø)

... and 498 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Sales Tax Calculator

Hey , I would like to work on this issue under JWOC.
The Sales Tax Calculator would compute , input as  before-tax price, sale tax rate, and output as final, or after-tax price.

@neokd go for it.

Thanks

I want to work on this issue under JWOC

can we use cdn for tailwind css?


CFA with optional chaining

@rbuckton

TypeScript Version:  master

Search Terms:
Code
declare const o: { x: 0 } | undefined;
if (o?.x === 0) {
  o.x;
}
declare const a: [0] | undefined;
if (a?.[0] === 0) {
  a[0];
}

Expected behavior:
pass
Actual behavior:
$ node built/local/tsc --noEmit --strict index.ts
index.ts:3:3 - error TS2532: Object is possibly 'undefined'.

3   o.x;
    ~

index.ts:7:3 - error TS2532: Object is possibly 'undefined'.

7   a[0];
    ~

Found 2 errors.

Playground Link:
Related Issues:

This looks to be tied to https://github.com/microsoft/TypeScript/issues/33736.

The same idea is commented there but that issue doesn't aim it.


Error: vagrant ssh requires a specific VM name to target in a multi-VM environment.

In Read the Docs / Setting up the Virtual Machine vagrant ssh fails with error: "This command requires a specific VM name to target in a multi-VM environment.". Should be vagrant ssh ansible instead?
Vagrant 1.8.5

Thanks for the report. I've made ansible the primary VM which should fix this issue.


HttpLib#request has strange argument names

If you look at how Context#request calls HttpLib#request it looks like this:
    return self.http.request(
        self.url(path), {
            'method': message.get("method", "GET"),
            'headers': message.get("headers", []) + self._headers(),
            'body': message.get("body", "")})

But if you look at the signature for HttpLib#request:
def request(self, url, headers=None, **kwargs):

And in turn when that calls the Handler#request the signature is:
def request(url, message, **kwargs):

Why is the second argument of HttpLib#request named headers when it's clearly the message argument later?

Good catch, that seems clearly wrong.

Name fixed. We were passing the message through the incorrectly named headers param. Name was a leftover from a previous design iteration.


Memory leak

There is no garbage collection. Every time the project is recompiled, the Python backend adds another 1 GB of memory.

The language server is already optimized to prevent various memory leaks and over-extensive memory consumption.
The described behavior is caused by incremental compilation. To prevent the need of re-compiling whole project again on a single change, the language server keeps some data from previous compilation and pairs them with latest compilation information.
This results in compilation size increasing on re-compilation. However, the newly consumed memory should be limited by total number of files in the workspace and how (and in what order) were recompiled.
Will be happy to look into your specific scenario if you can provide a link to your project and brief description of how to achieve the described memory consumption.

The language server is already optimized to prevent various memory leaks and over-extensive memory consumption.
The described behavior is caused by incremental compilation. To prevent the need of re-compiling whole project again on a single change, the language server keeps some data from previous compilation and pairs them with latest compilation information.
This results in compilation size increasing on re-compilation. However, the newly consumed memory should be limited by total number of files in the workspace and how (and in what order) were recompiled.
Will be happy to look into your specific scenario if you can provide a link to your project and brief description of how to achieve the described memory consumption.

Thanks. I'm working on river-build/river which has hundreds of contracts. To reproduce, simply change the formatting and recompile a couple times.


Add limited android test support

We could provide android test support by launching an android emulator and running tests in the emulator (like how it is done in libc and rust), but there is licenses issues in redistributing the sdk (see 3.4 of https://developer.android.com/studio/terms.html). Off course, I maybe be wrong about this.
So we download and build bionic libc from source, which I think allows we to distribute the binaries. Having the android libc allows running native binaries using qemu, which can be enough for projects like, nix, ripgrep, etc.
Related https://github.com/japaric/cross/issues/82

I'm wondering if we can workaround this potential redistribution issue by downloading the SDK on user machine (and cache it somehow) instead of distributing it with our docker images.

An alternative is to build the docker image in the user machine. In a more general away, we can pack all Docker files inside the cross binary and give the user the option of downloading the pre-build docker image or build it locally. We can also let the user specify a Docker file.

I submitted the qemu patch upstream.
Running ripgrep tests in an i686 android emulator works as expected. So I think std is correct. Anyway, the error seems only to happen when Command is used.
@japaric Anything else? Can we test this?

Sorry for the delay @malbarbo. Could you please add:

A note about testing support being limited for these target. Noting that Command doesn't work, for example, would be great.

Add the _RUNNER variable to the Dockerfiles so that these targets work without binfmt-misc support (which requires sudo). You can find an example of that variable here.

After that you can r+ this.
@homunkulus delegate+

:v: @malbarbo can now approve this pull request

Thanks @japaric!
@homunkulus +r

Ops, let's try again.
@homunkulus r+

:pushpin: Commit 140aff3 has been approved by malbarbo

:hourglass: Testing commit 140aff3c01298f5ce3abf37159787bfdc957d527 with merge 3de557cb889fde2c239697f829709122387d93cd...

:sunny: Test successful - status-travis
Approved by: malbarbo
Pushing 3de557cb889fde2c239697f829709122387d93cd to master...


Categories Icon

Describe the requested icon(s)
Categories icon please, it's an option for mobile apps.
References

Have seen this before, and it's probably something we can add. I'm guessing shapes and circles-four don't work for you?


z.record returns partial type when using enum with a 2nd schema

Love this library, and while I was updating a site to the latest version, I realized something had changed on z.record. I had been using record  with two schemas as described in the changelog, since I realized it was allowed and the readme documentation seemed behind on fancy functionality:
const schema = z.record(z.enum(["Tuna", "Trout"]), z.boolean());
type schema = z.infer<typeof schema>; // Record<"Tuna" | "Trout", boolean>

This was working fine until around v3.13.2. Now the result seems to be Partial<Record<"Tuna" | "Trout", boolean>>

https://github.com/colinhacks/zod/issues/992


made artwork 1 : Flower

fixed issue #2

@aswinzz
live link - https://codepen.io/dhairyabaweja/full/KJPbJG

@dhairyabaweja Please resolve the merge conflicts.


[pmo] Eliminate non-ownership code/tests.

NFCI.

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - 36aad6db46d4b743159979879bd2c6439af83570

Build failed
Swift Test Linux Platform
Git Sha - 36aad6db46d4b743159979879bd2c6439af83570

Hmmm... I think that we probably need to not run pmo on canonical sil.

Can one of the admins verify this patch?


Added sv translation

Added Swedish translations. Was not sure if _missing_translations.json had to be updated, so I did just in case.

Thank you! The _missing_translations.json file is not needed.


Feature/improve tests

Coverage decreased (-0.9%) to 97.724% when pulling 31e58f47a9a689f01bbddb72b0239f6ca8aa037c on feature/improve_tests into fefc430ae747a6b37322632a1cd87bef6636224d on master.


[ARTEMIS-2487] Updated to org.jctools:jctools-core:2.1.2

To resolve wrong import range due to wrong declaration in
jctools-core:2.1.1

Good job!


docker-machine ls/env errors when machines are on multiple providers

Docker-machine depends on the environment variables to contact the provider (rather than caching them) and can never work correctly if you have many machines across multiple providers.
I have been using environment variables which I set (via source) in the shell before issuing docker-machine command.
If I set the variables to point to my amazonec2 account, then create a machine, docker-machine ls works correctly.
If I change those variables to point to another EC2 account or an on-site vSphere account, and create a new machine, I can never run docker-machine ls without errors.
Due to its dependency on environment variables, docker-machine either fails to talk to amazonec2, or fails to talk to my on-site vSphere instance.

As an example of the problem;

Set the required environment variables including MACHINE_DRIVER to amazonec2 and create a machine.
Reset environment to local vSphere, create another machine.

From another session without any of the environment variables set, docker-machine ls returns the following;
NAME            ACTIVE   DRIVER          STATE     URL                       SWARM   DOCKER        ERRORS
fmsprod         -        vmwarevsphere   Running   tcp://<IP_ADDRESS>:2376           v17.03.2-ce
zbdev                    amazonec2       Timeout

With the amazonec2 environment variables set, I see

NAME            ACTIVE   DRIVER          STATE     URL                         SWARM   DOCKER        ERRORS
fmsprod         -        vmwarevsphere   Running   tcp://<IP_ADDRESS>:2376           v17.03.2-ce
zbdev           -        amazonec2       Running   tcp://<IP_ADDRESS>:2376             v17.05.0-ce

All the ip addresses and session keys etc are visible and correct in the zbdev/config.json file, but it can't connect without the local environment variables being set.
Cheers


No main manifest atrribute

I have tried to build and run the project on both my Mac and Windows and have gotten the same errors on both computers. I followed the steps in the Readme and I can get "mvn package" to run. But after that I can't get anything to work.
target>java -jar lasttry-1.0.0-SNAPSHOT.jar
no main manifest attribute, in lasttry-1.0.0-SNAPSHOT.jar
I have tried several variations of this including being in the target folder, out of it and using an absolute path, quotes, even trying to run the main class using the jar as the classpath:
target>java -cp lasttry-1.0.0-SNAPSHOT.jar org.egordorichev.lasttry.LastTry
Error: Could not find or load main class org.egordorichev.lasttry.LastTry
The only warnings I get while running maven is that it's using platform encoding, could this be it?
Where is the main manifest? Or how do I get it to generate properly?

Thank you! I will fix this today!

Added main class to the manifest. First part of issue is fixed, but the main class is still not found, don't know why.

I tried adding a new class with a main method in the same package as LastTry.java and the new file was able to be run. In the new main file I called the main() method of LastTry and got the following:
Exception in thread "main" java.lang.NoClassDefFoundError: org/newdawn/slick/state/StateBasedGame
  at java.lang.ClassLoader.defineClass1(Native Method)
  ...
  at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
  at org.egordorichev.lasttry.Main.main(Main.java:6)

Caused by: java.lang.ClassNotFoundException: org.newdawn.slick.state.StateBasedGame
  at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
  at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
  at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331)
  at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
  ... 13 more

So it seems like StateBasedGame cannot be found but not sure why it would build successfully if this was the case. I was able to solve this by using the maven shaded plugin which puts all the dependencies in the one jar, and then the new error is:
Exception in thread "main" java.lang.UnsatisfiedLinkError: no lwjgl in java.library.path
  at java.lang.ClassLoader.loadLibrary(ClassLoader.java:1867)
  at java.lang.Runtime.loadLibrary0(Runtime.java:870)
  at java.lang.System.loadLibrary(System.java:1122)
  at org.lwjgl.Sys$1.run(Sys.java:72)
  at java.security.AccessController.doPrivileged(Native Method)
  at org.lwjgl.Sys.doLoadLibrary(Sys.java:66)
  at org.lwjgl.Sys.loadLibrary(Sys.java:96)
  at org.lwjgl.Sys.<clinit>(Sys.java:117)
  at org.lwjgl.opengl.Display.<clinit>(Display.java:135)
  at org.newdawn.slick.AppGameContainer$1.run(AppGameContainer.java:39)
  at java.security.AccessController.doPrivileged(Native Method)
  at org.newdawn.slick.AppGameContainer.<clinit>(AppGameContainer.java:36)
  at org.egordorichev.lasttry.LastTry.main(LastTry.java:177)
  at org.egordorichev.lasttry.Main.main(Main.java:6)

Not sure why it compiles with dependencies but can't find them when I run it.

Are the dependencies included in the built jar file? If not, are you adding them to the class path?

I think the maven file is completely broken. I will try to generate a new one.

Ok, i've figured out the first part of the error. The LastTry class, witch has the main() method, is extended from StateBasedGame. Moving main() to another class fixed it.
But now it looks like it doen't pack the dependencies right.

Sorry for this error, we are building using IDE and installed libs. Trying to figure out, why it does not pack the dependencies.

Finally got it. Added "-Djava.library.path=lib/macosx/" to the command line when running and it worked.

Oh, great! Thank you! Will add that to the README.


Created profile for gcp e2e test run excluding billing metrics test c…

Created profile for gcp e2e test run excluding billing metrics test classes

/retest

/retest


dnu restore does not work on Ubuntu 15.10

dnu is not working correctly on Ubuntu 15.10. I installed DNVM, DNX prerequisites, .net core clr and
then cloned the repository at https://github.com/aspnet/Home. On doing a dnu restore, it doesn't work on Ubuntu 15.10 whereas trying it on Ubuntu 14.04 works

I have this issue too :-\
happens with both runtimes
$ dnvm list

Active Version              Runtime Architecture OperatingSystem Alias
------ -------              ------- ------------ --------------- -----
       1.0.0-rc1-final      coreclr x64          linux
  *    1.0.0-rc1-final      mono                 linux/osx       default

https://github.com/aspnet/dnx/blob/ed98b4c687cf0785650d59e7b7a65a371bc5332f/src/Microsoft.Dnx.Tooling/Restore/ImplicitPackagesWalkProvider.cs#L96

well, until they make it compatible, here is how you downgrade
wget http://security.ubuntu.com/ubuntu/pool/main/i/icu/libicu52_52.1-8ubuntu0.2_amd64.deb
sudo dpkg -i libicu52_52.1-8ubuntu0.2_amd64.deb

I have the same problem on Ubuntu 15.10.   Just tried a fresh install of all .Net components based on various instructions on Microsoft/ASP.Net sites.
dnu restore runs, but with not output at the terminal at all.
I have tried the most simple app based on this article: https://docs.asp.net/en/latest/dnx/console.html and in VS Code - most of the code is underlined with errors -eg the "using System;" line says:
The type or namespace name 'System' could not be found (are you missing a using directive or an assembly reference?) [dnxcore50]
Unnecessary using directive. [dnxcore50]
System
I was evaluating .Net on ubuntu as a legit platform for web development.  Obviously a long way from ready.  Shame.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


how does the http-parser to support the multi threads

hi, I have a sence that client send the big data request, then server will use multi thread to process the request, these threads might invoke parser parallel, I'm not sure hao do I support the multi threads via http-parser.

http-parser is oblivious to threads. As long as there's only one thread at a time operating on the struct http_parser instance and provided you issue the proper memory barrier when passing the struct between threads, everything is fine.


Impossible to scan wallet address from Metamask

They have add a prefix network as ethereum:0x578657858765 so the scan does not work

@ThierryThevenet We'd be happy to look into this for you. kindly file a support ticket below
https://forms.gle/Wmpm6NLzYm9uXYuU9


chore(behavior_path_planner): move scene module utils

Description
move following module's utils

lane change
side shift

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Thank you!
ok for me in the side shift module part!

resolve conflicts


Fails to build on Android

This fails on Xamarin Test Cloud with 66/66 devices failed.
I downloaded it, loaded it with VS and I got 160+ errors when I tried to run this app on my phone.

I get these errors in build for the Android version:
6>G:\Devel\Tests\app-evolve\src\XamarinEvolve.Android\Resources\Resource.designer.cs(151,132,151,158): error CS0117: 'Resource.Layout' does not contain a definition for 'zxingscanneractivitylayout' 6>G:\Devel\Tests\app-evolve\src\XamarinEvolve.Android\Resources\Resource.designer.cs(152,132,152,158): error CS0117: 'Resource.Layout' does not contain a definition for 'zxingscannerfragmentlayout' 6>G:\Devel\Tests\app-evolve\src\XamarinEvolve.Android\Resources\Resource.designer.cs(154,92,154,104): error CS0117: 'Resource.Id' does not contain a definition for 'contentFrame' 6>G:\Devel\Tests\app-evolve\src\XamarinEvolve.Android\Resources\Resource.designer.cs(155,114,155,140): error CS0117: 'Resource.Layout' does not contain a definition for 'zxingscanneractivitylayout' 6>G:\Devel\Tests\app-evolve\src\XamarinEvolve.Android\Resources\Resource.designer.cs(156,114,156,140): error CS0117: 'Resource.Layout' does not contain a definition for 'zxingscannerfragmentlayout'
The UWP version doesn't build either because it says it can't finish restoring packages.
I don't have a MAC nearby so I couldn't try that one...
Regards,
Andrés

Can you do a full clean and re-build of the project. I have had no issues at all on multiple machines.


Widget not working in custom page

I have created new Custom page for google analytics widget but it's not showing up .

ps: I am still learning filament

i think you need to enable that from the config


concat: accept ActiveSupport::Multibyte::Chars

This is a revival of https://github.com/Faveod/arel-extensions/pull/27.
Currently if we try to pass the result of val.mb_chars to concat, it ends with
ActiveSupport::Multibyte::Chars can not be converted to CONCAT arg
.../gems/2.5.0/gems/arel_extensions-1.2.12/lib/arel_extensions/nodes/function.rb:77:in `convert_to_node'

Thanks, merged!


Add CONTRIBUTING.md guidelines; note GitHub ToS

Closes #147.
Per mbland/cla#2, I've opted to note the GitHub Terms of Service regarding contribution licensing rather than using my own Contributor Licensing Agreement for now.
Also adds a code of conduct and a pull request template.

Coverage remained the same at 98.089% when pulling 510ff91457312d43607cb6af3807ccb5d2cafe3c on cla-#147 into e1e6a656ca4e64842200472e964f9054e1df8115 on master.

Coverage remained the same at 98.089% when pulling 510ff91457312d43607cb6af3807ccb5d2cafe3c on cla-#147 into e1e6a656ca4e64842200472e964f9054e1df8115 on master.


Not all namespaces and helm charts show up

Description
We have deployed this onto a large cluster with over 300 helm charts however only 256 helm charts appear. We can see that in the request to get all namespaces, all namespaces are returned from the cluster (355 namespaces). However, some of these are not displayed as there are apparently no helm charts found which does not seem to be the case.
Is there a limit on the number of helm charts it can show or some known issue?
Screenshots
No response
Additional information
No response

Adding pagination might help here as in an instance only a specific set will be queried and loaded

The 256 releases is a Helm default limit that we'll need to address either with higher limit or with pagination. It needs a fix in the code.
Infinite number of releases might be a wrong strategy, we should detect when we are hitting some limit and offer to narrow display down with namespace filter. TBD
The place where it originates from: https://github.com/helm/helm/blob/main/cmd/helm/list.go#L55

Thank you so much for the quick response! I think pagination would work well from a user experience point of view. Also, I assume one way to get around the limit is to show all namespaces and when you click on a namespace it would then find the charts for the given namespace?

I'm still in doubts. IMO pagination is wrong, 'cause you usually have idea about what you are looking for. It's either string pattern, or a namespace.
We can detect when there is more charts than we display, and suggest narrowing down the search. In this way, we'll have also to make filtering on backend, as opposed to current frontend way.
It definitely would need to wait for React


CRB-2374 & CB-20459

See detailed description in the commit message.

please in the future  follow the 1 commit/pr rule

DatabaseServerSslCertificateConfigIntegrationTest.java

please in the future follow the 1 commit/pr rule

I dont think we have such rule but let me do it

Repost: #14121


Dynamic content in locale file is loaded in SSR then reloaded on the client during hydration

Version
v6.8.1
Reproduction link
https://none
Steps to reproduce

Enable lazy: true in nuxt.config.js.
Create an /locales/en.js
In the en.js file, add a message: Math.random() translation
In a template, display {{ $t('message') }}
Reload the browser
Observe the page loads with the translations already applied in SSR, showing a random number
Observe that a few moments later, hydration happens and the random number is replaced with a new number

What is expected ?
Only the SSR version of the translations would be applied and hydration would not actually update the translations.
What is actually happening?
Hydration is updating the translations, changing the content from what SSR generates.
Additional comments?
The reason this came up is we're using a lorem ipsum module to quickly mock up fake content in our translations. Having the content change suddenly after first load is jarring.

This bug report is available on Nuxt community (#c436)

It sounds like a good idea and might be a good base for fixing some other issues.
This is something that makes it non-trivial - language files can export function in which case we have to pass context object to them and we can't do that if we are gonna process those when module's code run (which we would have to do to fix this).
We could potentially treat those two differently but not sure...
Are you exporting plain object or function from your language files?

I am exporting a plain object but I populate some of it using function calls. For example
      description: lorem.generateSentences(2),

The issue is/was that the lorem-ipsum module generates new random text each time it is called. My work around was to write a custom randomization function and pass that to the lorem object when it's constructed so the text is always generated on the same seed.
But this just solves the visual problem of the text changing, it doesn't alter the fact that the hydration step still re-runs the translation file. I could see more elaborate translation setups being rather costly to re-run.

The fix would work for that (object) case, but for functions, we would need to handle them as we do now - processing them separately for client/server.
I think I'm fine with that. Just might be difficult to explain that difference in the docs for people to understand it really.

Couldn't a flag in the store be set indicating that the function had already been called server-side and not to call it again client-side?

Current functionality passes Nuxt context and that one is different for server and client. We can't really make any assumptions on how people are using that functionality so running it only once (on the server) would potentially break someone's use case (unlikely but still possible).
Also, your suggestion to have a store state for that:
a) doesn't solve the issue above
b) is not gonna work when store is disabled and this package doesn't rely on store for core functionality

I'm experiencing a similar issue (without lazy: true though). I have a non-trivial setup when I actually load and setup localization from the legacy backend. This is being done inside the fetch phase of SSR:
    app.i18n.setLocaleMessage(lang, renderData.messages)
    app.i18n.setLocale(lang)

When everything loads, I see localized messages for a first moment, and then client-side hydration is performed and all localizations are lost. I'm sort of puzzled - is there any way to avoid this issue and keep localization messages from the SSR?

@xfyre your issue is not the same. If you are programmatically adding translations then they won't be preserved on the client. Please open a new issue (or a "question") for your problem.

@rchl Thanks. I've created a question - #694.


Metadata schema additions for sc/snRNAseq (Parse)

Select the target metadata schema name (can be multiple), or choose 'Other' if not listed.
snRNAseq, scRNAseq
If 'Other' is selected, please enter the schema name below.
No response
Describe the change request (use a numbered list if many).
add barcode_offset = '10,48,78'  NOTE: This may impact pipeline processing and/or soft assay decision rules engine; investigating
add library_preparation_kits

10X Genomics; Chromium Next GEM Single Cell 3' Kit v3.1, 16 rxns; PN 1000268
Attached (CG000315.pdf, pg. 7)
10X Genomics; Chromium Next GEM Single Cell 3' Kit v3.1, 4 rxns; PN 1000269
Attached (CG000315.pdf, pg. 8)
10X Genomics; Chromium Next GEM Single Cell 5' Kit v2, 16 rxns; PN 1000263 Attached (CG000331.pdf, pg. 7)
10X Genomics; Chromium Next GEM Single Cell 5' Kit v2, 4 rxns; PN 1000265
Attached (CG000331.pdf, pg. 7)
Parse Biosciences; Evercode WT v2 Kit (48 reactions); PNs WP100, WB100, WC100, WX100, WA100, WA200
Attached (Evercode_WT_v2.0.1.pdf, pg. 6-9)
Parse Biosciences; Evercode WT Mini v2 Kit (12 reactions); PNs SP100, SB100, SC100, SX100, WA100, SA200
Attached (Evercode_WT_Mini_v2.2.1.pdf, pg. 6-9)
Illumina; TruSeq Stranded mRNA Library Prep (48 samples); PN 20020594
TruSeq Reagents link
Illumina; TruSeq Stranded mRNA Library Prep (96 samples); PN 20020595
TruSeq Reagents link

add sample_indexing_kits

Parse Biosciences; Fragmentation Reagents; PN WX100 Attached (Evercode_WT_v2.0.1.pdf, pg. 8)
Parse Biosciences; UDI Plate - WT; PN UDI1001 Attached (Evercode_WT_Mini_v2.2.1.pdf. pg 10)
Illumina; TruSeq RNA Single Indexes Set A (12 Indexes, 48 Samples); PN 20020492 TruSeq Reagents link
Illumina; TruSeq RNA Single Indexes Set B (12 Indexes, 48 Samples); PN 20020493 TruSeq Reagents link
Illumina; TruSeq RNA CD Index Plate (96 Indexes, 96 Samples); PN 20019792
TruSeq Reagents link
Illumina; IDT for Illumina - TruSeq RNA UD Indexes v2 (96 Indexes, 96 Samples); PN 20040871 TruSeq Reagents link
Integrated DNA Technologies: Custom DNA Oligos
Our Genomics team custom orders indexing oligos through Integrated DNA Technologies (IDT)

add sequencing_reagent_kits

Illumina; NovaSeq 6000 S4 Reagent v1.5 Kit (300 cycles); PN 20028312 [NovaSeq 6000 Reagents link (https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.illumina.com%2Fproducts%2Fby-type%2Fsequencing-kits%2Fcluster-gen-sequencing-reagents%2Fnovaseq-reagent-kits.html&data=05|02|RSM66%40pitt.edu|895f6c1f379d44058b2108dc44630812|9ef9f489e0a04eeb87cc3a526112fd0d|1|0|638460439342819400|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|0|||&sdata=5X7t9HDz0AGpstDl%2BEACai20W5gDwbgFNSFG90Ckh0M%3D&reserved=0)
Illumina; NextSeq 1000/2000 P2 Reagent v3 Kit (300 Cycles); PN 20046813 NextSeq Reagents link
Illumina; NextSeq 1000/2000 P2 Reagent v3 Kit (200 Cycles); PN 20046812  https://www.illumina.com/products/by-type/sequencing-kits/cluster-gen-sequencing-reagents/nextseq-1000-2000-reagents.html
Illumina; NextSeq 2000 P3 Reagent Kit (300 Cycles); PN 20040561 https://www.illumina.com/products/by-type/sequencing-kits/cluster-gen-sequencing-reagents/nextseq-1000-2000-reagents.html

Evercode_WT_Mini_v2.2.1.pdf
Evercode_WT_v2.0.1.pdf
CG000331.pdf
CG000315.pdf

I checked the Evercode manual and correct the product number for the 2 kits from Parse Biosciences for the library_preparation_kit:

Parse Biosciences; Evercode WT v2 Kit, 48 rxns; PN ECW02030
Parse Biosciences; Evercode WT Mini v2 Kit, 12 rxns; PN ECW02010

The product numbers (SKU) are found at the left bottom of the manual's front page. Please verify.

Kits confirmed. Go ahead with the add.

Closes with https://github.com/hubmapconsortium/dataset-metadata-spreadsheet/commit/84ac08d6a19352efa34a880f2a13936f2954f7bd


fix: created btc dust amount constant and renaming

Description
The temporary hotfix for dust btc amount is in zetaclient. The fix needs to be done in zetacore to prevent dust CCTX from registration (https://github.com/zeta-chain/node/issues/2326). The PR only ported the constant and remaining.

Renamed BtcDustOffset -> BtcNonceMarkOffset and created BTCWithdrawalDustAmount constant for zetacore usage.

Closes:
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Include instructions and any relevant details so others can reproduce.

[ ] Tested CCTX in localnet
[ ] Tested in development environment
[ ] Go unit tests
[ ] Go integration tests
[ ] Tested via GitHub Actions

Checklist:

[ ] I have added unit tests that prove my fix feature works

It doesn't seem the change modify any logic in ZetaClient did we forget to do
2000+BTCWithdrawalDustAmount

?

The 2nd commit: https://github.com/zeta-chain/node/commit/b40b511cf1856cc24e4fcfdc8b68b4206a988c08
The 2nd commit according to Panruo's comment: https://github.com/zeta-chain/node/commit/8a34fbe7d602865451fb41eb0eaa9564bba8a849

Does this fix https://github.com/zeta-chain/node/issues/2326?


Add Geometrical Dominator - Piano Cover by Waterflame

Song Name
Geometrical Dominator - Piano Cover
Artist Name
Waterflame
Source
Youtube
Youtube Link
vBnDz3d63JI
Direct File Link
No response
Song ID
18
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 11707005
Downloaded: Yes

I worked hard to find this so pls accept

what does that mean


Build postEvent Lambda

What changed
Build postEvent Lambda
AUTH_IPV_AUTHORISATION_REQUESTED

F2F_YOTI_START

IPV_F2F_CRI_VC_CONSUMED

AUTH_DELETE_ACCOUNT

AUTH_IPV_AUTHORISATION_REQUESTED AFTER MARKING RECORD FOR DELETION

Pls add link to Jira ticket and on the PR title


Feature/lol/image metadata

Connects #330
This re-enables the image metadata panel and populates it with the exif data pulled from the tiff.
Retrieving the metadata is done at two places: first, during image validation and second during thumbnailing. During validation we have access to gdal's opened copy of the image. At this point we retrieve the geodata from the tif. During thumbnailing, since we have the entire image already on disc, we read it and grab the exif data at that time. During layer thumbnail generation we calculate the layer's bounding box since we have all the images already verified and thus they should all have bounding boxes.
To test:

Upload layers
Once complete, view the layer details and click on the images tab.
Click the "image metadata" link on each image in the list to see the metadata.
Check the database to see that the bounding boxes have been added to the layer and layer images.

+1

@azavea-bot rebuild

@azavea-bot rebuild


Error with Visualization

Hello Max, I try running your code with the grid model (with installing all dependencies), unfortunately, ended with the following error: FileNotFoundError: [Errno 2] No such file or directory: 'models/saved_models/grid_1-1.pt'. There is no folder saved_models, so I removed that in the vis_agent.py, but still I face the same issue. Is there any solution.

Hello Max, I try running your code with the grid model (with installing all dependencies), unfortunately, ended with the following error: FileNotFoundError: [Errno 2] No such file or directory: 'models/saved_models/grid_1-1.pt'. There is no folder saved_models, so I removed that in the vis_agent.py, but still I face the same issue. Is there any solution.

Hello. could you show me how to run the code?
I only know "python main.py"


react-mixin

react-mixin/react-mixin.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/react-mixin
http://bower.io/search/?q=react-mixin
others?

[ ] has a test file? (react-mixin/react-mixin-tests.ts or others)
[ ] pass the Travis-CI test?

wow, please wait.
related https://github.com/DefinitelyTyped/definition-tester/issues/11

@vvakame Oh, sorry I see.  I'll wait until the issue above has been fixed.

thanks mate! now DefinitelyTyped support .tsx

Thanks for your quick response!


GM camera ACC: vision-only ACC behind toggle

Here are all the brake thresholds we'll need to implement to make this PR work without any faults:

openpilotLongitudinal:

Standstill:

Enabling: 20
Cancelling: 11

Moving: Both 11

not openpilotLongitudinal:

Standstill: 20 (camera handles cancelling)
Moving: 20 (camera handles cancelling)


With columns: (auto, auto) does not respect document with columns(2, document)

Using auto, auto makes the table wider than the linewidth for a 2-column layout.

If I change one to a 1fr, it fills the space nicely:

This is certainly odd. Will investigate when I can.

Fixed with 77ed90054867570b3a5d15ea371ad29bf161b5b8; please confirm later if you're still getting this, and feel free to re-open the issue if so. Thanks for reporting!
As a bonus, here's a video of my discovery of the cause of this bug... you wouldn't believe my face when I found it out, haha!
https://github.com/PgBiel/typst-tablex/assets/9021226/9292724e-5785-4f69-b588-fa77c1ddef7d


Support preview for layoutlib

To solve this problem it is necessary to gain access to assets directory in edit mode. There is a bug in Android.

Would love to see this!

Waiting for release of issue


Current Logged in User's access token is stored inside the web browsers local Storage

Core Library
MSAL.js (@azure/msal-browser)
Wrapper Library
MSAL Angular (@azure/msal-angular)
Public or Confidential Client?
Public
Documentation Location
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/dev/lib/msal-browser
Description
Logged in User's access token is stored inside the web browsers HTML localStorage, this has been flagged by our Security partners as vulnerable, please suggest on this issue?
Please refer the below Screen Shot:

Cache location is configurable and can be set to memoryStorage if you have concerns with local or session storage. Please note however that there are drawbacks with this option, namely loss of persistence across page refresh. We do not believe there are inherent security risks with localStorage as long as you are not vulnerable to XSS based attacks. Please see this doc for more information about storage configurations and recommendations.


Add slack and --parallel flag

Oh man, I wanted to review this! Gotta give me more time @franklinkim

sorry, bad habit of being the only one in these repos


Please remove the themes folder in exampleSite

Hello @dt801ts,
it's correct that exampleSite mirrors the root folder of a Hugo website. But there's no need to create a themes directory in exampleSite. exampleSite/themes/sublime-hugo is a symbolic link that creates an infnite loop. This causes an error when someone tries to clone the hugoThemes directory.
Hence, please remove the themes folder in exampleSite.

Hi @digitalcraftsman ,
I apologize for the late reply. Somehow I'm just seeing this now. I removed the themes directory from exampleSite. Hopefully this resolves the problem, and hopefully my theme can now be put back on Hugo's theme gallery page ?
Thanks

Hi @dt801ts.
it looks good now. I've (re)added your theme. It should appear soon on the theme site.
Best,
Digitalcraftsman


Set up Travis hook

I've tested the formula locally with the following:
$ brew install ./Formula/neomutt.rb --with-sidebar-patch --HEAD

Everything goes well.
We should set up the Travis hook to get nice green badge.  There's a complaint from brew audit ./Formula/neomutt.rb at the moment, which would cause the Travis build to fail. I'll push a fix after the hook is set up to test it.
/cc @flatcap

@alexpearce
I set my travis account to build on each commit and I see one failed build.
https://travis-ci.org/neomutt/homebrew-neomutt/builds/122265802

I don't see any way to share the travis account, so let me know if anything needs changing.

Hurrah!  I now see "build passing".

🎉 thanks!


Missing Password parameter in Set-LabVMDefaults function

Set-LabVMDefaults -Password 'Some3P@ssw0rdString'
Set-LabVMDefaults : A parameter cannot be found that matches parameter name 'Password'.
At line:1 char:19

Set-LabVMDefaults -Password 'Some3P@ssw0rdString'

              ~~~~~~~~~

CategoryInfo          : InvalidArgument: (:) [Set-LabVMDefaults], ParameterBindingException
FullyQualifiedErrorId : NamedParameterNotFound,Set-LabVMDefaults

Thanks! I was aware that this parameter was missing, but also conscious that we really need to implement a SecureString or the equivalent.
When this module eventually gets pushed to the gallery, anything with a -Password or -Credential parameter that is using a String will get rejected. Do you have any thoughts on how you would like to see this implemented?
In the interim, I'm happy to review the pull request and merge :)

Thanks Iain.
Don't really have any thoughts on that at this point to be honest. Just needed it to work for some testing I started today. Will see if I can come of with a clever idea, but don't have much experience in that area at this point (in other words, don't hold your breath).

OK - no worries - thanks for giving it a spin! I'll merge the pull request when I get a chance; just be aware that there may be a breaking change in the near future.
If you have any other suggestions or problems, just log an issue/enhancement :D


goto_*() arguments no longer work

Having this line (as instructed in readme):
vim.keymap.set({'n', 'v', 'o'}, 'J', require('tree-climber').goto_next({ highlight = true }), keyopts)

raises:
Error detected while processing /home/filip/.config/nvim/init.vim:
line  160:
E5108: Error executing lua vim/keymap.lua:0: rhs: expected string|function, got nil
stack traceback:
        [C]: in function 'error'
        vim/shared.lua: in function 'validate'
        vim/keymap.lua: in function 'set'
        [string ":lua"]:4: in main chunk
E325: ATTENTION

Seems to be related to https://github.com/neovim/neovim/issues/20532

I think I know the reason for this error and it is a bit simpler. In the readme the line is:
vim.keymap.set({'n', 'v', 'o'}, 'J', require('tree-climber').goto_next, keyopts)

, which passes a function require('tree-climber').goto_next into the keymap.set.
In your case you pass the result of function call require('tree-climber').goto_next() (note the brackets after the goto_next) and this result is nil. The easiest solution is to wrap it in the anonymous function:
vim.keymap.set({ 'n', 'v', 'o' }, 'J', function()
  climber.goto_next({ highlight = true })
end, keyopts)

Hope this helps (and if yes please close the issue).


Add operator-tensor  multiplication and type hints

also add more tests for operator and linearoperator

Stack from ghstack (oldest at bottom):

(to be filled)

Finally. This should now work.
Sorry for the notification spam.
These changes were already reviewed by @ckolbPTB once as part of #249 and #183 .


Fix integration test "FreeText Editor FreeText (delete existing) must delete an existing annotation"

The most recent failure is in http://<IP_ADDRESS>:8877/018387cde149e38/output.txt with the following traceback:
Failures:
1) FreeText Editor FreeText (delete existing) must delete an existing annotation
  Message:
    ProtocolError: Waiting for selector `#pdfjs_internal_editor_3.selectedEditor` failed: script.callFunction timed out. Increase the 'protocolTimeout' setting in launch/connect calls for a higher timeout if needed.
  Stack:
        at <instance_members_initializer> (file:///C:/pdfjs/botio-files-pdfjs/private/018387cde149e38/node_modules/puppeteer-core/lib/esm/puppeteer/common/CallbackRegistry.js:89:14)
        at new Callback (file:///C:/pdfjs/botio-files-pdfjs/private/018387cde149e38/node_modules/puppeteer-core/lib/esm/puppeteer/common/CallbackRegistry.js:93:16)
        at CallbackRegistry.create (file:///C:/pdfjs/botio-files-pdfjs/private/018387cde149e38/node_modules/puppeteer-core/lib/esm/puppeteer/common/CallbackRegistry.js:19:26)
        at BidiConnection.send (file:///C:/pdfjs/botio-files-pdfjs/private/018387cde149e38/node_modules/puppeteer-core/lib/esm/puppeteer/bidi/Connection.js:51:32)
        at Session.send (file:///C:/pdfjs/botio-files-pdfjs/private/018387cde149e38/node_modules/puppeteer-core/lib/esm/puppeteer/bidi/core/Session.js:154:42)
        at Session.<anonymous> (file:///C:/pdfjs/botio-files-pdfjs/private/018387cde149e38/node_modules/puppeteer-core/lib/esm/puppeteer/util/decorators.js:94:27)
        at WindowRealm.callFunction (file:///C:/pdfjs/botio-files-pdfjs/private/018387cde149e38/node_modules/puppeteer-core/lib/esm/puppeteer/bidi/core/Realm.js:92:51)
        at WindowRealm.<anonymous> (file:///C:/pdfjs/botio-files-pdfjs/private/018387cde149e38/node_modules/puppeteer-core/lib/esm/puppeteer/util/decorators.js:94:27)
        at #evaluate (file:///C:/pdfjs/botio-files-pdfjs/private/018387cde149e38/node_modules/puppeteer-core/lib/esm/puppeteer/bidi/Realm.js:127:42)
        at async BidiFrameRealm.evaluateHandle (file:///C:/pdfjs/botio-files-pdfjs/private/018387cde149e38/node_modules/puppeteer-core/lib/esm/puppeteer/bidi/Realm.js:94:16)

The source code can be found here: https://github.com/mozilla/pdf.js/blob/master/test/integration/freetext_editor_spec.mjs#L1213-L1263
We should find out what causes this as it's one of the last intermittents we have left.

Maybe I fixed it in https://github.com/mozilla/pdf.js/pull/18366.
The rect is empty because the editor isn't rendered and then clicking on "it" does nothing, consequently the editor isn't selected.

Unfortunately I don't think that fixed this particular issue because this intermittent happened in #18367 and that one was rebased onto the current master branch where that patch is already active.

It's indeed a bit problematic because I can't spot anything wrong in the code, especially after the extra hardening that you applied to getRect, and I can't reproduce it locally too. For now let's just use this issue to track that it happened and see if it happens again, and in the meantime I'll think about (and have some ideas for) how to get more debugging information if such exceptions happen.

For later reference, it just happened again in #18369, see http://<IP_ADDRESS>:8877/fc78ad56a1a8b11/output.txt, which also contains the getRect hardening, so it does appear that there is a different issue with this particular test only.

This one might be fixed by #18374 but because it's a bit hard to tell due to its intermittent nature we'll leave this tracking issue open for a couple of days to make sure it doesn't reappear.

Closing since we haven't seen this reappear so far. If it does, we'll track it in #18396.


Add V8 cross-thread profiling patches

V8's profiler does not expect isolates to be locked from different threads. Pull request #710 moved inspector protocol handling to a separate thread, meaning profiling was started and stopped on the inspector thread, not the Worker's request connection thread. If I remember right, the result of this is that the profiler ends up sampling the inspector thread, which doesn't normally run JavaScript, and causes a bunch of "(program)" placeholder spans to show up in the profile.
We previously discovered this issue with our internal runtime, which has always been heavily multi-threaded, and patched around the problem. This commit copies the two relevant patches from our internal repo.
Fixes #1754.

This seems a bit surprising -- the inspector protocol seems explicitly designed to run from a different thread, in order to support breakpoint debugging (which suspends the execution thread?).
(But hey if it works, ship it.)

I guess our patches are Linux-specific:
Mac:
external/v8/src/libsampler/sampler.cc:604:11: error: use of undeclared identifier 'SYS_tgkill'
  syscall(SYS_tgkill, platform_data()->tgid(), lockedTid, SIGPROF);
          ^

Windows failed for a separate, probably spuriuos reason about an unexpected clang version, which I think @jasnell mentioned in chat. Presumably it won't appreciate SYS_tgkill, too, though.
C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Tools\MSVC\14.40.33807\include\yvals_core.h(898,1): error: static assertion failed: error STL1000: Unexpected compiler version, expected Clang 17.0.0 or newer.

This seems a bit surprising -- the inspector protocol seems explicitly designed to run from a different thread, in order to support breakpoint debugging (which suspends the execution thread?).

It looks like it suspends execution by calling this runMessageLoopOnPause() function, which then explicitly pumps the inspector message loop until V8 tells it to stop:
https://github.com/cloudflare/workerd/blob/6a57e25fd87f8ce498e4cf293cf5074959ea30bb/src/workerd/io/worker.c%2B%2B#L406-L411

Could we perhaps actually deliver the profiler start/stop messages over to the isolate thread, rather than trying to start the profiler cross-thread?

Could we perhaps actually deliver the profiler start/stop messages over to the isolate thread, rather than trying to start the profiler cross-thread?

This is done in #2497.


Script mixer_codegen.sh does not work as per Istio mixer custom adapter dev guide

Bug description

Istio mixer custom adapter guide uses mixer_codegen.sh script which invokes protoc with argument "docs_out=xxxx",  there's no protoc-gen-docs plugin anywhere on the internet. Is it a Google internal plugin? Or is protoc-gen-docs migrated to protoc-gen-doc while the istio mixer custom adapter guide hasn't update accordingly?

Error message:
"
--docs_out: protoc-gen-docs: Plugin failed with status code 1.
Attempt 1/3 to run protoc failed with exit code 1
protoc-gen-docs: program not found or is not executable
Please specify a program using absolute path or make sure the program is available in your PATH system variable
"

protoc.sh invoked by mixer_codegen.sh uses variable REPO_ROOT while it seems to be null at run time. Supposingly,  $REPO_ROOT=$GOPATH/src/istio.io/istio.

Error message:
"
/common-protos: warning: directory does not exist.
google/protobuf/descriptor.proto: File not found.
mixer/adapter/aifabric/extensions.proto:21:1: Import "google/protobuf/descriptor.proto" was not found or had errors.
mixer/adapter/aifabric/extensions.proto:44:8: "google.protobuf.FileOptions" is not defined.
mixer/adapter/aifabric/extensions.proto: "google.protobuf.FileOptions" is not defined.
mixer/adapter/aifabric/template.proto:3:1: Import "mixer/adapter/aifabric/extensions.proto" was not found or had errors.
Attempt 1/3 to run protoc failed with exit code 1
"
Expected behavior
mixer_codegen.sh to run smoothly when following Istio mixer out of process adapter dev guide at https://github.com/istio/istio/wiki/Out-of-process-Attribute-Generating-Adapter-Developer-Walkthrough
Steps to reproduce the bug
Follow the guide.
Version (include the output of istioctl version --remote and kubectl version)
istio release 1.4 (actually 1.3 and 1.2 too)
How was Istio installed?
Git clone from github istio repo
Environment where bug was observed (cloud vendor, OS, etc)
Local machine. Linux Ubuntu 18.04.

Those docs are sufficiently out-of-date, unfortunately. The codegen stuff was all moved over to be docker-ized. We use BUILD_WITH_CONTAINER=1 make gen from the root dir to generate any artifacts that are needed.
@geeknoid can explain the codegen migration process.

Will that be possible to create a page explaining how to create the stubs and yaml files without mixer_codegen script ?
This way we could avoid to use the Istio codebase in order to build our own adapters and use other languages for out-of-process adapters than Go.

Those docs are sufficiently out-of-date, unfortunately. The codegen stuff was all moved over to be docker-ized. We use BUILD_WITH_CONTAINER=1 make gen from the root dir to generate any artifacts that are needed.
@geeknoid can explain the codegen migration process.

Hi Douglas,
Thank you for the clarification.
Hi Martin,
Would you please kindly update the document to share more about the correct way for custom mixer out of process adapter development? I managed to get custom adapter yaml generated and then got into troubles connecting custom adapter with mixer as there isn't much detailed info on instance definition.
Looking forward to hear back from you.

A follow up, after resolving the connection reset error.
S2I build fails around half the time with error:
"ReadTimeoutError("HTTPSConnectionPool(host='pypi.org', port=443): Read timed out. (read timeout=15)"
One of the suggestion is that set the pip install time out to 100s instead of default 15s.  https://stackoverflow.com/questions/43298872/how-to-solve-readtimeouterror-httpsconnectionpoolhost-pypi-python-org-port
Is there a way to set the pip install timeout during invoking s2i build command?

Hi all,
Is there any update on this issue, or another document where is explained how to build an adapter? Thank yoi


fixed dockerfile lnd inside docker folder

What does this fix? Should have a description.
Also, might conflict with #2259

Looks like it can actually replace #2259. This PR will allow us to no longer assume the source code is present within the current directory.

Works, in conjunction w/ #2334 , on Ubuntu 16.04

i'd suggest the method proposed in my PR #2334 instead.
However, in #2334 one needs to build the containers from the root of lnd as such, but its better than cloning the git repo :
host@~/lnd/$ sudo docker-compose -f docker/docker-compose.yml build lnd
but i think this is more useful than cloning the repo, because this builds against whatever is currently checked out in the root of the lnd directory (e.g. a feature branch)

After writing this command in the guide
docker-compose run -d --name alice lnd_btc
I got this error:
ERROR: Service 'lnd_btc' failed to build: The command '/bin/sh -c apk add --no-cache     git     make &&  cd /go/src/github.com/lightningnetwork/lnd &&  make &&  make install' returned a non-zero code: 2 (edited)

Looks like it can actually replace #2259. This PR will allow us to no longer assume the source code is present within the current directory.

@wpaulino yes, but now to build you add a dependacy of needing http and more importantly GitHub.  I think this actually makes the build more brittle.

@wpaulino yeah Github is used other places to pull files into the container.  Either options work, I just think it adds extra complexity since the state of github may be different than the state locally, and also github could go down which does happen from time to time.  Also, build time is slightly faster if not relying on an external service to pull int files that could otherwise be provided locally.  The net result is the same as a way to copy files into a docker container.


New package: GenieSessionFileSession v1.0.0

Registering package: GenieSessionFileSession
Repository: https://github.com/GenieFramework/GenieSessionFileSession.jl
Created by: @essenciary
Version: v1.0.0
Commit: 2ef7ba66cabe3b407347d0c20ae228d68b9e7cdd
Reviewed by: @essenciary
Reference: https://github.com/GenieFramework/GenieSessionFileSession.jl/commit/2ef7ba66cabe3b407347d0c20ae228d68b9e7cdd#commitcomment-79349069
Description: File system based session adapter for GenieSession.jl

Would it be possible to skip the 3 days wait time for this package please? We have a lot of dependent updates waiting for this one and we'd like to have all the packages registered for JuliaCon. Thanks!

[noblock]


Add support for Buckaroo's Brq_culture parameter

The parameter "Brq_culture" contains a localestring like "en-US" or "nl-NL".
Usage
$request->setCulture('nl-NL');

The parameter "Brq_culture" contains a localestring like "en-US" or "nl-NL".
Usage
$request->setCulture('nl-NL');

Merged. Can someone please confirm that this works in production then I will tag for release.

Just implemented it in production.
works well.

This has been checked and works in production here.


Fresh project with npm install - npm ERR! ERSEOLVE unable to resolve dependency tree

Issue summary
In VS Code (Windows) pulled latest code from repository to start a fresh project. Ran npm install to build module directory. npm produces ERR! ERSEOLVE unable to resolve dependency tree. Having such an error is a blocker for new projects with this repository.
Expected behavior
Starting a fresh project from repository and running npm install should not have any dependency issues.
Actual behavior
In VS Code (Windows) pulled latest code from repository to start a fresh project. Ran npm install to build module directory. npm produces ERR! ERSEOLVE unable to resolve dependency tree. Full error below. Having such an error is a blocker for new projects with this repository.
npm notice
npm notice New minor version of npm available! 8.1.2 -> 8.12.1
npm notice Changelog: https://github.com/npm/cli/releases/tag/v8.12.1
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/eslint
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@shopify/eslint-plugin
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
Steps to reproduce the problem

Have npm installed.
In VS Code (Windows) with a new workspace, pull down latest repository.
Enter npm install in Terminal window.

Checklist

[x] I have described this issue in a way that is actionable (if possible)

sorry, my mistake. Posted to wrong repository. Please ignore.


unify build dir for Python and Java

What do these changes do?
As Ray supports Java API now, the build dir may need to be unified, so we can just build Ray once and support Python and Java API simultaneously.
Now, Python's build dir is “ray/python/ray/core”, while Java's build dir is "ray/build", so I change Python's build dir to "ray/build" too. When building for Python, just move the generated libraries and executables of Ray to Python's original dir, so it will make no difference.

@raulchen @imzhenyu @eric-jj please help to review.

LGTM

@raulchen I changed python's build dir to "ray/build", after building for python  completed, then I try to build for Java and come across this error. It seems some of the ray source code should be built again, so it can produce the jars which java api depended on.

@songqing that's true. Python and Java do depend on different sets of binaries.
I forgot to mention that, we need to make sure that if the user only want to build python version, we don't use anything related to JDK.

LGTM, feel free to file this RP to ray whenever you think it's ready.


[tuify] Add tests for existing functionality

@nazmulidris
i am new to base source, could you please address which are functionality that need to add test.
i mean i should start somewhere, to get familiar with the source code
willing to help

@0xkelvin Thank you. Here is a good place to get started to get a handle on terminals and TUIs: https://github.com/r3bl-org/onboarding
In the current tuify codebase we have tests in the following files:

https://github.com/r3bl-org/r3bl_rs_utils/blob/main/tuify/src/components/select_component.rs#L213
https://github.com/r3bl-org/r3bl_rs_utils/blob/main/tuify/src/scroll.rs#L128

There need to more tests to check things that a user would perform, such as try and select a single item. There's the happy path. Where they have a few items and enough display viewport size to see them all without scrolling. Then they make their selection or not, and we are done.

In these cases the tests can check whether moving up and down w/ the cursor is working.
Whether things are rendered correctly as the focus state changes.
And finally if the selection actually selects the right thing.

You can see in some tests the following trait is mocked https://github.com/r3bl-org/r3bl_rs_utils/blob/main/tuify/src/components/select_component.rs#L26 so that you can check what its contents are (very similar to ReactJS snapshot tests). In these cases, it is important to override the color support, since we run all tests in CI/CD and they don't all support color output. This is done in one of the tests listed above.
Here are all the paths that I've been manually testing:
https://github.com/r3bl-org/r3bl_rs_utils/blob/main/tuify/examples/main_interactive.rs#L53
It would be great to automate those testing paths.
PLMK if you have any questions 👍🏽 . And we have a discord server if you would like to chat (more sync than async like issues) about the issue or PR.


Use bulma CSS

Simplify Makefile
Removed tools (for now)
Added bulma to assets
Moved htmx to assets
Added default dirs to assets
Added index route with bulma and htmx
Included vendored templ package

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Replace packr with native Go embed

How to categorize this issue?

/area os
/kind enhancement
/priority 3
/os suse-chost
What would you like to be added:
Replace packr with native Go embed
Why is this needed:
Less external dependencies.

I think this was done some time ago with https://github.com/gardener/gardener-extension-os-suse-chost/pull/39.
Is it still used somewhere else in this repo?

You're right, I guess the occurrences of packr which I saw were due to an outdated local state of the extension.
/close


"Add" is not equivalent to "Set'

The tool works, but where you say in the script file that the new coins amount will be "added" this is not true. It actually SETS the coins value instead of adding to it.

You're right, it adds to the lifetime coins amount but sets the main coins amount, so improved the description :+1:
Thanks!


Alter return type class name to enable static analysis

Issue 1
After installing this library into a project, larastan was no longer able to analyse the project. It threw the following error for each of the classes which used the Obfuscatable trait:
Internal error: Class EvoMark\LaravelIdObfuscator\Traits\EvoMark\LaravelIdObfuscator\Eloquent\Builder|static(App\Models\Example)
was not found while trying to analyse it - discovering symbols is probably not configured properly.

Solution
Adding the missing preceding backslash to the classname EvoMark\LaravelIdObfuscator\Eloquent\Builder in the PHPDoc fixed the issue.

Issue 2
Internal error: Class EvoMark\LaravelIdObfuscator\Eloquent\Builder|static(App\Models\Example) was not found while trying to analyse it - discovering symbols is probably not configured properly.

Same error message as before, but at least the pathname to the Builder class looks to be right now
Solution
Removing the static return type from the PHPDoc got Larastan back up and running. What was the intention with including static in the PHPDoc? I'm likely missing something! 🤦
Thank you again for this excellent project 😊

Thanks for this. Not sure what the 'static' thing was about.


Calling Close in polling mode causes NullReferenceException

The currentStream is null in polling mode, causing this to throw a null ref.
https://github.com/harness/ff-dotnet-server-sdk/blob/0528d4296dc7c40dd625db0fcb3712dbf71a85c2/client/connector/HarnessConnector.cs#L224

Fixed in 1.3.x, closing


[Feature request] Scroll to specified plugin in community plugins tab

When I click the arrow, I expect to be taken to the plugin's settings, or at least have it highlighted in the community plugins page. As it stands, it just takes me to the top of the community plugins page and gets me to search again from there.
In my view this is a bug even though I submitted it as a feature request. Any plans on fixing this?

@huachuman
It is not quite clear what steps you are taking and when. Are you able to write it out start to finish, or record a gif/video to explain exactly what it is you are requesting?

Clicking the arrow on the right should take me to the plugin's card, or similar, but instead, it just takes me to the community plugins search box.

I expect that when I click the arrow on the right it should scroll to the plugin's card and highlight it, or something along those lines, but instead, it just takes me to the community plugins search box.
Here's an example result from a search I did with settings-search
Here's where clicking the arrow takes me
That plugin also has a settings icon in the community plugins page but it's not showing up in the settings-search results (see first image) where it should, beside the arrow:

That is odd behavior indeed. I am not sure why you are brought back to the community page unless you deliberately ask for it, as I show in my gif below in the dataview example.

As far as Emoji Shortcodes settings not showing up in Setting Search, that would be something that the author may need to enable. I'll check to see if they have dynamic searching enabled, and if not, make a FR over there.


Sometimes last frame is missed on mobile

Let's say we need to animate value from 0 to 2.5 during 600ms.
Sometimes on iphone 5se (Chrome) last frame is missed, so value stops on 2.46 (for example) instead of 2.5. Layout will be updated with correct value after some user interaction, for example select text with value.
settings:
<AnimatedNumber
    value={value}
    stepPrecision={2}
    formatValue={(number) => number.toFixed(3).slice(0, -1)}
    duration={600}
/>

I've added shouldComponentUpdate to make sure another updates don't impact this, but this strange behavior still occurs sometimes on mobile.
Have you noticed something like this?

I got a similar problem, the number "stuck" at the end of animation
This is the screen cap when no missing frame

This is the screen cap when the bug appear
The component's state currentValue stuck in "50" where currentTime updating

I got similar problem, some times it's not right, not every times.
How to fix it~~~


Workspace Conda, Poetry, Virtual Envs

Note: Ignore hatch for now

Done

what needs to be done to support Hatch? (I'm the maintainer)

@ofek You can add hatch support here, Don can provide the details. When the extension API for this becomes available, you will likely want to use that as it will allow you to do more than just discovery.
FYI, this is an intermediate bridging tool, with the intent that this will allow us to break out the tightly coupled environment code in the python extension. On the python extension side I will be opening up a set of APIs that allow environment managers to ship support in their own extensions.
For env types supported here like pyenv, poetry, conda, etc, we expect those to be removed (from here) as and when extension for those env managers will ship. The reason we made this tool was to set a baseline expectation for discovery in python extension. So when env managers ship their own extensions there is some benchmark for expected performance. The TS discovery code was outdated and overdue a refresh, hence this re-write.

@ofek
You can have a look at the pet-poetry crate (please ignore the file environment_locations_spawn.rs
I am yet to document the code and API (actually will be working on that today/tomorrow).
The file lib.rs has the implementation for the trait Locator

try_from needs to return a PythonEnvironment if it is a hatch environment.
find will return all of the known hatch environments.

Next update the crates/pet/src/locators.rs file to add an entry to the hatch locator just above or below the Poetry locator (this line locators.push(Arc::new(Poetry::from(&environment)));.
Bulk of the code in Poetry is used to locate the directory where Poetry environments are located.
I had to get that from the Poetry code, with Hatch you will know very well where they are located, and then for each environment you will need to determine the project it belongs to, thats where the project field needs to be populated.
E.g. for hatch once you find environments, (I think), you will then go through the file ..lib/python3.9/site-packages/_hatch_demo.pth and find the project this belongs to and use that as the value for project field (from what I can tell, i think it needs to be the parent of src folder).
Hope this helps.
Will update the docs with contribution guides this week.


using notary to import keys and certs for use with docker content trust

It's not clear to me how to do this. Ideally,
somewhere> create key/cert
builder> acquire key, build image, push image using cert
runner> acquire cert, pull image using cert, be confident the builder had the key
It seems like the notary CLI is the correct entrypoint to accomplish this, but it's not clear how.
My first thought was to use '$ notary key import --gun docker.io/skelterjohn/img key.pem', but the --gun flag doesn't appear to be used. At least, '$ notary key list' does not show my image as the cert's gun.
Even if that worked, it's not clear to me how I can use notary to inform the runner about the cert, though I have been able to place the cert manually in ~/.docker/trust to get the desired effect. But, much nicer to use the provided CLI than to mess with a directory structure I don't fully understand.

I had a good conversation with programmerq in #docker-trust today. I'll summarize the key points here.

DCT is a trust-on-first-use model. This goes for both the registry trusting pushes, and for the client trusting pulls. That is, once you push with a particular key, no pushes can be done in the future without that key. Same with pulls. If the key is lost for pushes, you have to email docker and ask them to forget your key. If the key changes, clients will get nasty warnings that they can choose to ignore.

Transport of the public key is "properly" done through the registry using the trust-on-first-use model, rather than a side channel. There may a way to send a client a cert directly and for them to only trust things that use that cert.

The proper way for private key distribution is to accept a delegate of the root key and use that for pushing images from a CI system. The customer should hold onto the root key, perhaps using a hardware token, but it's something the customer should manage on their own.

It is not clear, from experimentation or documentation, how to accept a delegate of a root key for use with pushing images. Hopefully '$ notary key import/export'?

Or how to create a delegate in the first place. '$ notary init docker.io/skelterjohn/img' seems like a good start, but does not succeed because my "client is offline".

Hi @skelterjohn, I just saw your conversation in IRC. We're currently working on the best way to bootstrap trust for a repo. It will most likely be a configuration file that whitelists Key IDs to trust. We have yet to decide on the UI that will enable adding a key to the whitelist, it may need the cert, or it could simply take the ID, or possibly options to do both.
Using notary with docker requires some extra configuration as it was built to work as a stand-alone tool. The -d and -s flags must be set to your local .docker/trust directory, and https://notary.docker.io respectively.
Just to ensure our terminology is the same, I think your definition of delegations doesn't quite line up with the notary/TUF definition.. If you haven't done so already, I highly recommend reading The Update Framework's spec to get an overview of how notary operates. We're not 100% faithful to the spec, but have discussed the changes we've made with the authors and are converging on the overall design.

I see, thanks. I did know to point it at ~.docker/trust, but I did not
know about https://notary.docker.io. Should I be able to run something
locally using notary-server?
On Mon, Feb 22, 2016 at 2:56 PM, David Lawrence<EMAIL_ADDRESS>wrote:

Hi @skelterjohn https://github.com/skelterjohn, I just saw your
conversation in IRC. We're currently working on the best way to bootstrap
trust for a repo. It will most likely be a configuration file that
whitelists Key IDs to trust. We have yet to decide on the UI that will
enable adding a key to the whitelist, it may need the cert, or it could
simply take the ID, or possibly options to do both.
Using notary with docker requires some extra configuration as it was built
to work as a stand-alone tool. The -d and -s flags must be set to your
local .docker/trust directory, and https://notary.docker.io respectively.
Just to ensure our terminology is the same, I think your definition of
delegations doesn't quite line up with the notary/TUF definition.. If you
haven't done so already, I highly recommend reading The Update
Framework's spec
https://github.com/theupdateframework/tuf/blob/develop/docs/tuf-spec.txt
to get an overview of how notary operates. We're not 100% faithful to the
spec, but have discussed the changes we've made with the authors and are
converging on the overall design.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/notary/issues/583#issuecomment-187354368.

Yes. If you're running your own notary server locally, the notary CLI will attempt to contact "https://notary-server:4443" by default. However, if you're attempting to push existing data to that server, it will error as the local server will not know about the timestamp key.
The easiest way to run locally is to use our docker-compose file which creates instances of the server, signer, and a database that uses a volume to persist metadata and keys across restarts. Currently that also requires adding the fixtures/root-ca.crt to your system certs, but we're looking at a way to accept it explicitly via configuration instead.

@skelterjohn Closing this issue for now, please file a new issue if you still need assistance


fix: issue node 19

This help for the people as me who use node 19

Thanks, I just bumped it to 3.4 and it should be covered.


Minor grammatical fixes in Headless section of Templating Guide.

Whilst reading the Templating Guide, I noticed a few minor grammatical issues that could be easily fixed and make the document a little easier to read.

Thanks @stevemason for the update.


Cannot query field "renderNavigation" on type "Query".

Although I followed the procedure described in the FAQ, I keep getting the following error: Cannot query field \"renderNavigation\" on type \"Query\".
This is my config/plguin.js file:
module.exports = {
  'navigation': { enabled: true },
  'drag-drop-content-types': {
    enabled: true,
  },
  'graphql': { enabled: true },
}

the file should have been called plugins.js


enable amSkipLocalTriedResource == true by default

Coverage decreased (-7.7%) to 62.183% when pulling f79e043302c25665bba2768017c9e7230860d384 on changelauncherconfig into c530af240692067e4d7c0ad4b0e9eec06f8e500b on master.


fix zh language keyword

Chinese language keyword is zh-CN and zh-TW, which is not 2-character-keyword. So I define a function to adapt it.
BTW there are some other language keywords are more than 2 characters, such as mni-Mtei.
But input the long keyword manually is inconvenient. I have no idea how to improve it. Maybe user should set a default language keyword?

Solved in other way #6, an update will be reasy soon. Thanks for your help


further improve vertical density

compare beaker advanced mode to ipy3:

from top to bottom the ipython cell is 55px tall, we are at 57px,
will the drop by 2px be ok?

i measured ipython at 83 pixels vs beaker at 150...


Changing the color box button's width and height programmatically.

Hi,
The available color box sizes are high and I don't want to change this through the CSS file. Maybe you can offer an option. For example, it can be 16x16, 24x24, 32x32 and 64x64.
Can you add this feature?
Thanks.

No, there are already way enough options and doing this just for one of thousand use-cases is not reasonable. It's literally just 4 lines of css and the classes (such as pcr-button which is used for the button) are not mangled - therefore it's easy to style pickr as you want.
Adding such features implies adding options for all other elements in case someone wants to change anything with js - that just doesn't work out.
There has to be a balance between available options given by a library and ease-of-access changing stuff, which is in this case quite easy as I already mentioned.
And if you want to do it programmatically, you can always use classList.add or .style on the button element which can get, among all other elements, accessed by pickr.getRoot() (See methods).


Add debug npm command.

Running npm run debug initializes server-side debugging via node-inspector with settings-development.json accessible to the Meteor instance.

This one is good to go. See a couple nitpicks on the other commit. Once these are set I'll merge and bump to 3.2.0. Thanks for putting these together :)

I agree that it's really handy. I think in order for meteor folks to know what to do it should be documented somewhere. Since the README of base isn't use for that and Base's website is, I'll leave it up to you.

Going to rope this into some other changes later. Will merge this then :)

Going to close this out for now as I'm not seeing much demand for it (or in Slack when we chatted about it).

Fair enough.


Turn up arrows green

It is more intuitive if when you vote up, the arrow turns green, and down is red.

closed by commit.


Custom system prompt?

Where is this system prompt coming from and can it be customized?
Given a user prompt, you will return fully valid JSON based on the following description and schema.
You will return no other prose. You will take into account any descriptions or required parameters within the schema
and return a valid and fully escaped JSON object that matches the schema and those instructions.

description:
json schema:

System prompt is first message in chat from system:
const UserSchema = z.object({
  // Description will be used in the prompt
  age: z.number().min(0).max(120).int().describe('The age of the user'),
  firstName: z
    .string()
    .describe(
      'The first name of the user, lowercase with capital first letter'
    ),
  surname: z
    .string()
    .describe('The surname of the user, lowercase with capital first letter'),
  sex: z
    .enum(['M', 'F'])
    .describe('The sex of the user, guess if not provided'),
})

// User will be of type z.infer<typeof UserSchema>
const user = await client.chat.completions.create({
  messages: [
    {
      role: 'system',
      content:
        'You are a world class extractor. You always respond in JSON. Current date is ' +
        new Date().toISOString(),
    },
    {
      role: 'user',
      content: 'John Doe born in 1988',
    },
  ],
  model: 'llama3-70b-8192',
  temperature: 0.0,
  max_retries: 3,
  response_model: { schema: UserSchema, name: 'UserSchema' },
})

The system prompt I mentioned is added to the array of messages before your "You are a world class extractor..." message. So there are two system prompts with your example stacked.

This system prompt is only used when using the "MD_JSON" mode - it gets added in the params resolver  that we use here:
https://github.com/hack-dance/island-ai/blob/main/public-packages/zod-stream/src/oai/params.ts#L77
I have not had any requests to be able to customize, but it would be relatively straightforward to add an option - I've mostly just tried to avoid adding too many options but maybe a straight up pass-through to zod-stream would work?

this issue is now urgent with the o1 models erroring when a system prompt is given
BadRequestError: 400 litellm.BadRequestError: OpenAIException - Error code: 400 - {'error': {'message': "Unsupported value: 'messages[0].role' does not support 'system' with this model.", 'type': 'invalid_request_error', 'param': 'messages[0].role', 'code': 'unsupported_value'}}


Update heapster, fixup mqautoscaler

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/571/
Test PASSed.


Implement ball holding rule

According to the rule book :

A direct free kick is also awarded to the opposing team if a player commits the following offense:

holds the ball for more than 1 second in a way that the ball cannot be removed from the player (a goal keeper may hold the ball up to 6 seconds on the ground or 10 seconds lifted up with one or both hands, a player performing a throw-in may lift the ball up with one or both hands for up to 10 seconds). More than half of the ball's volume must be outside the convex hull of the player, projected to the ground, for the ball to be considered removable. If the ball enters the convex hull repeatedly, it must be removable in between for the majority of the time. If more than one player of a team is in the vicinity of the ball, the convex hull is taken around all the player of a team, which prevent removal of the ball. Ball holding offenses always occurs at the location of the ball.

If an offense did not happen within a radius of approx. 1 m around the current ball position, or if the ball is not in play, the direct free kick is replaced by a removal penalty. Ball holding leads to a free kick independently of the distance between the robots and the ball.

To speed-up the execution of the referee, we simplified the computation of the convex hull and collision with the ball:
The convex hull will be computed as a quad for every player. The quad will correspond to the projection on the field of the center of the solid parts of the robot marked 'foot' and 'hand'. Each robot should have exactly two solid nodes marked 'foot' and two solid nodes marked 'hand' as the suffix of the Solid.model field, e.g.:
    Solid {
      model "left hand"
    }
    ...
    Solid {
      model "right hand"
    }
    ...
    Solid {
      model "left foot"
    }
    ...
    Solid {
      model "right foot"
    }

If this convex hull collides with the ball, the robot is holding the ball.
If the convex hull of several robots collide with the ball, the robots are collectively holding the ball.

When running the ball_holding/multi_robot_kid test, I am getting the following crash:
Traceback (most recent call last):
  File "test_supervisor.py", line 76, in <module>
    status.update(time.time() - system_start, simulated_time,
  File "D:\msys64\home\Olivier\webots\projects\samples\contests\robocup\controllers\test_supervisor\rc_testing.py", line 68, in update
    self._updateGC(system_time, simulated_time, gc_status)
  File "D:\msys64\home\Olivier\webots\projects\samples\contests\robocup\controllers\test_supervisor\rc_testing.py", line 153, in _updateGC
    if phase > len(self._sec_state_starts[sec_state]):
KeyError: 'DIRECT'
[0015.038|0071.344] PREMATURELY EXITING TESTING

Can you check it @medrimonia?

I fixed it in the experimental branch, if you merge it, it should work properly now.

I believe the ball holding rule is now fully implemented. The test is passing. However, we should probably develop more tests, including one for the goal keeper who is allowed to hold the ball for 6 seconds.


In the feedback form, when there is no text, a user should not be able to click submit

By default, the submit button on the feedback form is disabled when a user has not started typing. Right now, a user can start typing, then backspace and delete everything they typed, and the submit button is still enabled, so they can then submit an empty feedback form.

[ ] Please make it such that the submit button is disabled on the feedback form if the form is blank, even if a user types and then backspaces to make it empty.

@angielt just checking , have you started working on this issue? If not, I can take it over and work on it ??

@angielt just checking ,have you started working on this issue? If not, I can take it over and work on it ??

It's already done but there are plenty more issues you can take a look at!


Misleading documentation

The title says: "Applies to: Azure Stack integrated systems and Azure Stack Development Kit" ... and the note later in the page says "Metrics and diagnostic logs are not available for the Azure Stack Development Kit."
This is misleading documentation. Can we have this fixed please.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: be873ec6-e85f-3bc2-ee23-8ff6b849ac70
Version Independent ID: b2597ba2-cfde-b4ac-558d-dc9ef002c8dd
Content: Azure Monitor on Azure Stack
Content Source: articles/azure-stack/user/azure-stack-metrics-azure-data.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@dipito31 Thanks for pointing this out. I've updated the tag at the top of the page. The content for Azure Monitor on Azure Stack should be updated with our next doc build. #please-close

@dipito31 We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Typo correction: passowrds

Simple 1 word typo correction, passowrds -> passwords
Thanks to @snotty for reporting the typo. :)
Closes #4897

#sign-off