Is re-frame for Clojure reactive programming?

Hello, thanks again for that great project :-)
I'm using it server-side to take part of cljc isomorophism (run same code in server and client) and for the most part it's quite awesome but after reading some code and documentation, I wonder whether it's an intended usage.

re-frame is a pattern for writing SPAs in ClojureScript, using Reagent.
— From the README

…there isn't often much point firing a timed event in a test.
— From Clojure code

From Clojure code:
(defonce ^:private executor (Executors/newSingleThreadExecutor))

Hence, I have a couple of questions:

Why a single thread?
Would you accept a PR to use multiple threads in the JVM?
If any strong reasons not to use it server-side, would you know of any other libraries?

Hi @piotr-yuxuan, we added CLJ support for re-frame to support testing and development use-cases. This lets you simulate re-frame handlers calling to in-process 'server-side' handlers, run unit tests from a Clojure REPL, and similar dev-time activities. I've also seen re-frame used server-side to render an HTML page which is then hydrated with Reagent when it is sent over to the client.
I guess you could use re-frame for creating a server, but I'm not really sure how it would work. In re-frame the results of your events don't directly create a response, they only update a reactive (in ClojureScript) database which can then trigger subscriptions. You would need some way of replying to a client with the changes from an event. It doesn't seem like a terribly good fit to me, but I'm interested in hearing more about how you would do it.
To your questions:

Why a single thread?

Because re-frame was built to work in a single-threaded browser environment, and we wanted the Clojure version to match that.

Would you accept a PR to use multiple threads in the JVM?

Probably not, I don't think it's a path we're very interested in, and we've already had some concurrency issues on the JVM which we've had to fix.

If any strong reasons not to use it server-side, would you know of any other libraries?

I'd recommend looking at CQRS and event sourcing patterns. There is CMDR which might give you some ideas.
In summary, building server applications isn't re-frame's intended usage (currently), and you might find other solutions which are a better fit.

@danielcompton,
Thanks a lot for your detailed, precisely worded reply! Now my previous message looks blur, and I like to give it more context.
I've started a pet project which displays a list of LineageOS-compatible phones. I wrote it with re-frame. At first the phone list was constructed on each page rendering but page load was slow so UX wasn't great. I chose to fetch and insert the list at compile-time so it would faster the first rendering and still update it client-side to make sure it's up to date. In order to do so and minimize logic duplication, I ported my existing cljs re-frame code to cljc. Now I'm figuring out that some other data sources can't be fetched from a client runtime (because of CORS), only from a Clojure runtime (e.g a REPL) so this pet project is going to be executed more and more from a Clojure REPL.
Right now I'm standing here: I like the structured event-based programming style but I'm wondering whether re-frame is still the right tool for this job, hence the title of this issue. In my humble point of view, here are the pros and cons, in no particular order:

👍 re-frame avoids code duplication between clj and cljs, but duplication is better than bad abstraction
👍 re-frame code is about how we mutate data, has structured event-based programming style with strong conceptual model, and strong state management.
❌ re-frame is single-threaded both in cljs (of course) and in clj (by choice)
❌ clj and cljs fx differ, I'm maintaining a common API but is it a good thing?
❌ re-frame in clj sometimes has different behaviour (like set-timeout!)

If re-frame isn't a good fit for that use case, would anybody mind suggest some structured event-based programming libraries? Manifold, RxClojure, or quasar-pulsar? I've had a brief look at them and they all seem to miss the "data-oriented, state management" part.

Further to what @danielcompton has written, I'd  recommend recomend two FAQs to you:

https://github.com/Day8/re-frame/blob/master/docs/FAQs/Why-CLJC.md
https://github.com/Day8/re-frame/blob/master/docs/FAQs/FullStackReframe.md

We'd like re-frame to be excellent for browser-based SPAs.  Beyond that, we have no ambitions.

Follow-up: use the right tool for the right job. re-frame is the right tool for graphical FRP. I've switched to core.async + minimal plumbing for computation-oriented FRP. Thanks all of you for your precise replies 🙂


CI: split Nix cache as separate save and restore cache

Current behavior

Closes #6522
Proposed changes

As described in the issue the Nix cache needs to be split into separate save and restore cache.
Checks

[ ] All commits in this Pull Request are signed and Verified by Github.
[ ] All commits in this Pull Request follow the Ockam commit message convention.
[ ] There are no Merge commits in this Pull Request. Ockam repo maintains a linear commit history. We merge Pull Requests by rebasing them onto the develop branch. Rebasing to the latest develop branch and force pushing to your Pull Request branch is okay.
[ ] I have read and accept the Ockam Community Code of Conduct.
[ ] I have read and accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam repository. The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a separate Pull Request, this will mark the commit as verified.

@metaclips, I have made the requested changes.

@metaclips @mrinalwadhwa I have added my name to CONTRIBUTORS.md through #6616


Show last-saved and last-trained classifier dates on source main page

A simple solution for part of issue #159, suggested by Veronica Radice on the Google Group:

Perhaps the wording on the source home screen could be revised from "Last classifier trained" to "Last classifier saved" to reflect how it works? An additional date/time stamp for the true "Last classifier trained" category would be great, so users are aware of the training as it occurs.

She's referring to the first gray box on this page - which I completely forgot had this timestamp, to be honest. If that seems easily missable, we could instead consider placing this info between the "Automated Annotation" header and the line plot.

@beijbom What do you think of this?

Sorry for chiming in late. I love this idea. Simple yet powerful. @StephenChan : u want to take this one? Otherwise, I can do it.
(I had also forgotten we show that piece of information!).

@beijbom If you've got time this week or the coming weekend, then feel free to do it, otherwise I can take it.

Sounds good. I'll see if I can squeeze it in.
On Tue, Aug 18, 2020 at 12:36 PM StephenChan<EMAIL_ADDRESS>wrote:

@beijbom https://github.com/beijbom If you've got time this week or the
coming weekend, then feel free to do it, otherwise I can take it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/beijbom/coralnet/issues/341#issuecomment-675672940,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAITTF2CKWAD43QHH7BWUXDSBLJ3JANCNFSM4PGBSMIA
.

This is working as far as I can tell.


Upgrading wallet > 1.2

Having this on console, how to?
[info] Generating new keys
[error] <ClientError:UPGRADE_NEEDED BWC clients < 1.2 are no longer supported.>
{ code: 'UPGRADE_NEEDED',
message: 'BWC clients < 1.2 are no longer supported.' }
Thanks!

Same problem here.
Any hints for this error?

Found a solution (I was using the 'wallet' command installed by 'bitcore-wallet'):

Find globally installed version of 'bitocre-wallet'.

I used 'which wallet', whose output was '/usr/bin/wallet'.
By typing 'ls -l /usr/bin/wallet' I found that it was pointing to '/usr/lib/node_modules/bitcore-wallet/'.

Remove the whole directory (/usr/lib/node_modules/bitcore-wallet/)
Download (or update by git pull) the git repo of bitcore-wallet
(as sudo) npm install -g

Hope this helps

Found a much easier solution:
'nvm use 6'
It turned out that nvm was using version 4, required by the bitcore-wallet-service, but the wallet app needs version 6

I post a comment in other issue page,maybe can help you.
584#issuecomment-267301718

Thanks @esseak


Panic when send tx and set fees without typing unit.

Summary of Bug
celestia-appd tx distribution withdraw-rewards celestiavaloper1zdrz4w2pwwffdvmpum0626vycel9caay9n3pll --from W3 --commission --fees 21500
It's panic because '21500' but not '21500utia'.
Version
1.3.0 on mainnet.
Steps to Reproduce
just as the summary.
For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Hey again @HoytRen, I believe this is intented behavior from the cosmos-sdk. If you think this should be changed, then I would reccomend moving this upstream. There might be a way to pick a default denomination for fees, but I'm not sure we would adopt such a feature without it also being adopted upstream.
If this is a bug or something else, please elaborate further and reopen this issue, :slightly_smiling_face: thanks!

Hi, @evan-forbes don't see you for a while. I believe it just need to be wrapped with a error prompt. I may have time to look into the code on thursday or friday.

I checked the code and need more time to work out a solution because the sdk doesn't throw out errors by design. Hardcoding the error prompt could be a solution, but can not be pushed back to upstream, then I need to think about a better solution so that we don't need to maintain the modification over and over again.

Maybe, I should close this issue, and try a PR to Cosmos SDK, and wait until Celestia imports the new version of SDK?

I checked the latest version of Cosmos SDK, this problem is fixed already, then, let me close this issue and simply wait for bump up of the sdk.

Thanks for investigating this! Out of curiosity which Cosmos SDK version fixes this? celestia-app currently uses a fork of cosmos-sdk that we regularly pull upstream changes from.
Ref: https://github.com/celestiaorg/cosmos-sdk/releases/tag/v1.20.0-sdk-v0.46.16 which is based on Cosmos SDK v0.46.16


Minor tense correction for deprecated docs

This addresses https://github.com/docker/docker/pull/22574#issuecomment-217926098.

Thanks! LGTM


Remove output min storage deposit validation for existing outputs

Description
Remove output min storage deposit/rent cost validation for existing outputs, only for new outputs on the output side of transaction it must still be checked
Motivation
Current outputs will be converted to new outputs which have require more storage deposit (added Mana field for example), so if the rent cost is not reduced, this will make many outputs have less amount than the min required rent cost would be in the upgraded network
Requirements

Don't return an error if output from the node has an amount smaller than the min rent
Don't return an error if output with smaller amount is used in the input selection

Are you planning to do it yourself in a pull request?
No.

Checked and in .get_output()/finish() output we don't check the minimum anywhere, so this issue can be closed


bump(ocaml-format): v0.26.2

Describe your changes
Update v0.26.1 to v0.26.2
See Release Notes
Issue ticket number and link

Checklist before requesting a review

[x] I have successfully tested installation of the package.
[x] I have successfully tested the package after installation.

Screenshots

I'm not sure why https://github.com/mason-org/mason-registry/actions/runs/10099963761/job/27930285167?pr=6028 fails @williamboman I assume it's due to it expecting OCaml 5.2. Is there a way to configure this with out full manual build?

@eratio08 I believe it's related to the fact that GitHub changed their macOS-latest runner from Intel to Apple chips which screws up the test matrix a bit. Should hopefully be fixed in https://github.com/mason-org/actions/pull/7


Perf: Settings editor renders all tree elements at once

I had seen this before many times, but only managed to repro now:

Open a text file (pinned)
Open settings (pinned)
Open the file again
Hide the sidebar
Open settings again

You'll notice a hang (300ms on my fast machine) and a quick flash of a lot of text rendered in the editor:

PR: https://github.com/microsoft/vscode/pull/78507


"Unsupported operand types" should not leads to runtime TypeError

<?php

declare(strict_types=1);

use Typhoon\Reflection\TyphoonReflector;

require __DIR__ . '/vendor/autoload.php';

final class cls
{
    public const VALUE = [] & 'str';
}

var_dump(
    TyphoonReflector::build()
        ->reflectClass(cls::class)
        ->constants()['VALUE']
        ->value(),
);

klimenko@workstantion:~/IdeaProjects/typhoon$ php ./main.php
PHP Fatal error:  Uncaught TypeError: Unsupported operand types: array & string in /home/klimenko/IdeaProjects/typhoon/src/Reflection/Internal/ConstantExpression/BinaryOperation.php:35
Stack trace:
#0 /home/klimenko/IdeaProjects/typhoon/src/Reflection/ClassConstantReflection.php(95): Typhoon\Reflection\Internal\ConstantExpression\BinaryOperation->evaluate()
#1 /home/klimenko/IdeaProjects/typhoon/main.php(18): Typhoon\Reflection\ClassConstantReflection->value()
#2 {main}
  thrown in /home/klimenko/IdeaProjects/typhoon/src/Reflection/Internal/ConstantExpression/BinaryOperation.php on line 35

Do you mean that it should throw a different exception?

I think, this is the way it should be.
https://3v4l.org/1vZ7H

https://3v4l.org/1vZ7H

is this really reflection behavior? exactly the same exception is thrown if you just create a class.
https://3v4l.org/HHHia
Is it possible to avoid throwing any exception?
Maybe return some container with or without a value.
I haven't tried to get inferred type of a constant, but I expect to get never. I don't expect any exception.
I understand when an exception is thrown if a file could not be read due to unix permissions. But a tool that claims to work in a static analysis environment should not panic with exceptions in such simple situations.

is this really reflection behavior?

The declaration of a class itself does not throw anything on the parsing level. TyphoonReflection and ReflectionClassConstant do not throw as well: https://3v4l.org/5KA9s until you request the value.

I haven't tried to get inferred type of a constant, but I expect to get never. I don't expect any exception in this case.

Not exactly, but no exception is thrown:
use Typhoon\Reflection\TypeKind;
use Typhoon\Reflection\TyphoonReflector;

final class C
{
    public const VALUE = [] & 'str';
}

$constant = TyphoonReflector::build()->reflectClass(C::class)->constants()['VALUE'];

var_dump($constant->type(TypeKind::Inferred)); // null
var_dump($constant->type(TypeKind::Resolved)); // types::mixed

But a tool that claims to work in a static analysis environment should not panic with exceptions in such simple situations

Static analysis tools must not use $constantReflection->value(), because it returns the actual runtime values an potentially autoloads smth. For instance, if you have
final class MyClass {}

function myFunction(MyClass $object = new MyClass()): void {}

then TyphoonReflector::build()->reflectFunction('myFunction')->parameters()['object']->defaultValue() will autoload MyClass, because the actual value is requested. There's nothing we can do about this.

Maybe return some container with or without a value.

$reflectionClassConstant->value(...) is your container. :)

To summarize:

I think that the current behavior is expected and correct.
Static analysis tools should not call methods, that evaluate smth. Theses methods exist for hydrators, DI containers and other tools that do work at runtime.


Eliminate the use of allprojects from gradle/gradle build

All three uses of allprojects in our build currently configure buildSrc subprojects. They can not be removed until buildSrc is able to include another build with convention plugins.


MPI  parallel_learning support csv ?

Hi ,I  have a problem  with MPI data format . could MPI train  support csv ?

If you can run it in single machines, it should work fine in MPI as well.


multiprocessing.pool.RemoteTraceback

General information:

emcee version: 3.02
platform: Centos7 cluster
installation method (pip/conda/source/other?): pip

Problem description:
Expected behavior: as good as before
Actual behavior: stoped when running
What have you tried so far?:
Minimal example:

import emcee

# sample code goes here...
{
with Pool() as pool:
    sampler = emcee.EnsembleSampler(nwalkers, ndim, log_posterior, pool=pool,
                                                                    moves=[(emcee.moves.DEMove(), 1),])
    sampler.run_mcmc(starting_guesses, nsteps)
}

error:

multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/public/data/yuebb/miniconda3/envs/ROOT/lib/python3.7/multiprocessing/pool.py", line    121, in worker
    result = (True, func(*args, **kwds))
  File "/public/data/yuebb/miniconda3/envs/ROOT/lib/python3.7/multiprocessing/pool.py", line    44, in mapstar
    return list(map(*args))
  File "/public/data/yuebb/miniconda3/envs/ROOT/lib/python3.7/site-packages/emcee/ensemble.py", line 545, in __call__
    return self.f(x, *self.args, **self.kwargs)
  File "DYB_simplified_STD-Pool.py", line 399, in log_posterior
    lp = log_likelihood(theta)
  File "DYB_simplified_STD-Pool.py", line 144, in log_likelihood
    print("chiSq : ", chiSq)
OSError: [Errno 116] Stale file handle
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "DYB_simplified_STD-Pool.py", line 446, in <module>
    sampler.run_mcmc(starting_guesses, nsteps)
  File "/public/data/yuebb/miniconda3/envs/ROOT/lib/python3.7/site-packages/emcee/ensemble.py", line 384, in run_mcmc
    for results in self.sample(initial_state, iterations=nsteps, **kwargs):
  File "/public/data/yuebb/miniconda3/envs/ROOT/lib/python3.7/site-packages/emcee/ensemble.py", line 343, in sample
    state, accepted = move.propose(model, state)
  File "/public/data/yuebb/miniconda3/envs/ROOT/lib/python3.7/site-packages/emcee/moves/        red_blue.py", line 93, in propose
    new_log_probs, new_blobs = model.compute_log_prob_fn(q)
  File "/public/data/yuebb/miniconda3/envs/ROOT/lib/python3.7/site-packages/emcee/ensemble.py", line 427, in compute_log_prob
    map_func(self.log_prob_fn, (p[i] for i in range(len(p))))
  File "/public/data/yuebb/miniconda3/envs/ROOT/lib/python3.7/multiprocessing/pool.py", line    268, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/public/data/yuebb/miniconda3/envs/ROOT/lib/python3.7/multiprocessing/pool.py", line    657, in get
    raise self._value
OSError: [Errno 116] Stale file handle

Since the error is being thrown when you print in your log likelihood function, this is not an emcee issue. You'll have to debug the Python/multiprocessing setup on your system.

Since the error is being thrown when you print in your log likelihood function, this is not an emcee issue. You'll have to debug the Python/multiprocessing setup on your system.


Updated shopkeeper npc to use different druid

For issue #22

Same issue with everyone touching the same map files here! We should chat about a strategy to coordinate this better in the dev meeting!


ArtemisObject requires properties to be marked with DynamicProperty even if their name is listed in the HasDynamicProperties attribute.

Basically, the following two pieces of code are equivalent:
[HasDynamicProperties]
public class Foo : ArtemisObject
{
    [DynamicProperty]
    public int A { get; set; }
}

[HasDynamicProperties("A")]
public class Foo : ArtemisObject
{
    [DynamicProperty]
    public int A { get; set; }
}

The issue is that it's redundant to have to add the DynamicProperty attribute to A even after it's listed as a dynamic property in the HasDynamicProperties attribute.

Closed with pull request #47.


Add benchmarks for Agda CEK machine

Don't look here yet.  I'm just checking that I haven't messed anything up ...

Just checking that I haven't messed anything up...

/benchmark validation

/benchmark marlowe

/benchmark nofib

/benchmark plutus-benchmark:validation

/benchmark plutus-benchmark:marlowe

/benchmark plutus-benchmark:nofib

/benchmark validation

/benchmark marlowe

/benchmark nofib

/benchmark validation

/benchmark marlowe

/benchmark nofib

The benchmarking results aren't very consistent, but it looks as if the refactoring here hasn't affected the results of the standard CEK benchmarks noticeably.


manually upload haddocks to hackage

Hackage is using a version of haddock that chokes on pattern synonyms, so the docs are not available.

done in 5e940476a1d7e82b8ad0f76a3eb1b4bbf232fa84


Update README.md

Change line # 138 GIBLAB to GITLAB.

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


chore: update Android Zendesk module to allow sending of tags

@zoofan Could you please resolve conflict?


Add Righter interfaces so Claims becomes mockable

Adds interface AppRighter, GatewayRighter, ComponentRighter interfaces for mocking.

@rvolosatovs I agree, however that's a breaking change (doesn't have to be if clients use rights correctly), out of the scope of this PR.


Node version dependency

Expected behaviour
Function normally at any node version
Actual behaviour
Lags and page becomes unresponsive when node 10.17 is used
Steps to reproduce the behaviour
Change node version to 10.17
Need input
Does picovoice have a specific node version dependency.

we don't have a node binding. how do you use porcupine with node?

Sorry for that , there was a misnomer in the team that picovoice uses node to bind the js files


Fastwarc reader

Can we add a new warc reader using the fastwarc?
It is said to be much more efficient than warcio

Including fastwarc would be nice.
However, in the current processing pipeline for fineweb, the warc reader is not a bottleneck (<5% of runtime on my machine, while trafilatura is 95% of runtime).
Of course, this might differ for other datasets.


ServiceLocator: Review AuthorizeFilter.OnAuthorize

            var httpContext = context.HttpContext;
            var authService = httpContext.RequestServices.GetRequiredService<IAuthorizationService>();

            // Note: Default Anonymous User is new ClaimsPrincipal(new ClaimsIdentity())
            if (httpContext.User == null ||
                !httpContext.User.Identities.Any(i => i.IsAuthenticated) ||
                !await authService.AuthorizeAsync(httpContext.User, context, Policy))
            {
                context.Result = new ChallengeResult(Policy.AuthenticationSchemes.ToArray());
            }

https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Core/Authorization/AuthorizeFilter.cs#L76 is the only place where we use DI, the filter doesn't do any logging which we might want to do? (for example unauthorized requests). In any case, the solution in this case would be to change the code to have a FilterFactory that creates an instance of the filter that is not reusable, as it requires a transient service (DefaultAuthorizationService). While this is more "correct" from a design perspective, it only introduces an extra layer without too much benefit (The filter factory receives an instance of the service provider, so is just moving the composition root one layer above). I'm inclined to leave this one as is unless we decide we want to do some logging here (in which case it'll be good to have a known composition root).
Thoughts @rynowak @Eilon

If the value is low, skip it.
Adding logging would be a different task unrelated to this.

Removing 3-Done because we didn't "do" it.

@Eilon does that make this one "won't fix"? More generally, is a bare "task" fine to close w/o further clarification?

On the point of logging. Is it even possible for a developer to log authorization failures using [Authorize]? It seems like you have to write your own Attribute and Filter if you want to get any detail other than 'is the result a 401/403?'

Hi, it looks like you are posting on a closed issue/PR/commit!
We're very likely to lose track of your bug/feedback/question unless you:

Open a new issue
Explain very clearly what you need help with
If you think you have found a bug, include detailed repro steps so that we can investigate the problem

Thanks!


Syncing Russian README with the original

The Russian README file is outdated, e.g. it lacks the up-to-date recommendation & instructions for running the website using a container. This PR brings the file in sync. It also improves a few existing phrasings and reorders the list of READMEs on different languages according to the Russian alphabet.

/language ru

/check-cla

I signed it

/lgtm

/cc @kubernetes/sig-docs-ru-reviews

/approve
echoing https://github.com/kubernetes/website/pull/25808#pullrequestreview-559064777


I got some project if you are interested

It's media project based on go let me know

Do you have more information about the project?

deals with mpd live ones if you interested contact @alienmaster9


Don't cache external dependencies.

Fix the collect() function so it won't collect external dependencies,
consequently breaking module-deps when it tries to read the file later:
https://github.com/substack/module-deps/blob/1592d483d0/index.js#L189

thanks @tarruda, but I fixed this bug upstream in module-deps https://github.com/substack/module-deps/pull/91. if you update browserify, it should work now.


time series

When I am trying to use time series, it is having a problem, I tried adding",", end of each object as well, it is not working for some reasons. Spyder is showing invalid syntax.

import mindsdb
import pandas as pd

mdb = mindsdb.Predictor(name="ES-price")

mdb.learn(
    from_data="ES.csv",
    to_predict='close',
    timeseries_settings = {
      order_by: ['timestamp']
      nr_predictions: 3
      use_previous_target: True
      window: 100
    }
)

nr_predictions: 3
^
SyntaxError: invalid syntax
How can I create a time series prediction with mindsdb?
If I use only order_by, it is saying order_by is not defined at all.
mdb.learn(
    from_data="ES.csv",
    to_predict='close',
    timeseries_settings = {
    order_by: ['timestamp']

    }
)

```

**order_by: ['timestamp']

NameError: name 'order_by' is not defined**

Answered in mindsdb. The problem was in the wrong syntax.


web demo正常启动之后测试对话就报错 The following model_kwargs are not used by the model: ['streamer']

Traceback (most recent call last):
File "/usr/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
self.run()
File "/usr/lib/python3.10/threading.py", line 953, in run
self._target(*self._args, **self._kwargs)
File "/home/xchl/.local/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/xchl/.local/lib/python3.10/site-packages/transformers/generation/utils.py", line 1213, in generate
self._validate_model_kwargs(model_kwargs.copy())
File "/home/xchl/.local/lib/python3.10/site-packages/transformers/generation/utils.py", line 1105, in _validate_model_kwargs
raise ValueError(
ValueError: The following model_kwargs are not used by the model: ['streamer'] (note: typos in the generate arguments will also show up in this list)

我用的模型是baichuan2-7B-chat的

cli_demo同样的错误，我用的模型是baichuan2-13B-chat

cli_demo同样的错误，我用的模型是baichuan2-13B-chat

To upgrade transformers can solve this issue for me.


Run commands from internal structure

Commands which were read from the config file should be run using their pattern.

#5 lets you configure a command. A feature to run multiple commands on their schedule will be added later.


MM-41745 refactor upload license flow

Summary
Change the UI flow for the upload license process. This PR removes a step from the UI flow for uploading a license. With these changes when you click the upload button, the OS file selector window is shown and when you select the file THEN the upload modal is shown with the file information.
Ticket Link
https://mattermost.atlassian.net/browse/MM-41745
Screenshots
https://user-images.githubusercontent.com/10082627/156860041-010a3be3-3e4c-44e6-85d5-75d2d066e617.mp4
Release Note
NONE

Not sure if I understood that ticket right, but I thought the process would be:

Click on the "Upload file" button from the page
User selects their license file
Show the user the "Successful upgrade" modal rather than the file selection modal.

Am I wrong?

hey @JulienTant the idea of the ticket is remove just one previous step where you got showed the selection file modal before to actually being able to select a file. So in the past the steps were:

Click upload button
modal select-file is shown
click Choose file
Select a file
-Get visual feedback about the file you want to upload

With this change the flow is:

Click upload button
Select a file
modal select-file is shown with visual feedback about the file you want to upload


Preference settings missing; mouse wheel behavior option

Description
My scroll wheel pans the map. I read issue #680 , and it seems I should have the option to change behavior in preferences.

Using Chrome on Windows 10 with wireless mouse.
I tried full browser refresh no no avail.
Version
2.0.0-beta.0
What browser are you seeing the problem on? What version are you running?
Chrome v110.0
The OS you're using
Windows 10 Pro
Steps to reproduce
No response
The browser URL at the time you encountered the bug
https://mapwith.ai/rapid-v2-beta#background=VBMPImagery_Most_Recent&datasets=msBuildings&map=17.00/38.12945/-79.06190&poweruser=true
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
No response

Thanks! the preference option was added in #680 - we have to push out a new beta that includes the fix.


Upgrade Boxcryptor.app to v2.1.465 (Build: 708)

Merged as ca7e38e40abe5c4639f94055095626deada8bd20.


Feature Request: Distinguish values in fields that are set in the XDM Object data element

Expected Behaviour
Make it very clear what values are set in an XDM Object data element.
Actual Behaviour
It's not possible to distinguish which values are set in this data element, vs those set by Alloy automatically. (hint, it's just 1 field in the first _atag object)

Solution:
Perhaps similar to the change to identify values changed in schemas via AEP Mix-Ins. They've recently made it more obvious using bold lines instead of gray checkboxes:
https://jira.corp.adobe.com/browse/PLAT-58299
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Platform and Version
Sample Code that illustrates the problem
Logs taken while reproducing problem

Jira ticket: https://jira.corp.adobe.com/browse/CORE-50488


Zero Copy View of CBoard type

Proposed new feature or change:
After profiling my first bot it seems that about 15% of total time comes from taking a botris._core.Game state and extracting the board as as a numpy array through np.array(gs.board.board, dtype=np.uint32).
From my understanding this first copies the underlying std::array to a python list and then copies a second time to produce an array in memory with the correct type (in this case uint32 to correctly represent the cells on the tetris board).
I thought that we might be able to use a feature from nanobind, the current binding library, that allows for zero copy exchange between the two languages.
I implemented this feature locally by adding the following to board.cpp in _core/:
        .def("numpy_view", [](Board &self) {
            size_t shape[1] = {Board::width};
            return nb::ndarray<nb::numpy, uint32_t, nb::shape<1>>(
                /* data = */ self.board.data(),
                /* ndim = */ 1,
                /* shape pointer = */ shape,
                /* owner = */ nb::handle());
        }, nb::rv_policy::reference_internal)

This closely follows the examples provided in the nanobind documentation. I have verified that the returned numpy array provides read/write access to the underlying memory used to store the board. However I am unsure if this is leading to any noticeable speedup in my own testing (I haven't been able to measure much of a difference).
If anybody else could reproduce my results that would be nice. Also any feedback on any potential mistakes I have made would be appreciated.

Thanks for the issue @sharora,
Currently, the reasoning for using a copy reference was to prevent any memory corruption or data invalidation that might occur from delete operations. Such, as to eliminate any overhead from pre-converting to an NDArray when unnecessary.
The solution you have provided was certainly considered but not seriously tested. I will do some similar local testing, however, in the meantime you can:

Create a pull request with the suggested changes and testing benchmark results.
Consider alternatives to creating a numpy array.

Thank you!

Initial testing seems to have proved little to no improvement, with suggestions for downgrading performance.
This seems to suggest that either perhaps a parameter or two is set incorrectly or that such optimizations are infeasible and inefficient.
For code referencing testing check 4aebbfaaba8696a8d239459da17f46393d171a06, note that while undocumented, both nb::rv_policy::reference and nb::rv_policy::reference_internal were tested. Of the two reference_internal, seemed to perform slightly better but with the cons of the dangerous nature as well.
Note that all testing was very simple, insignificant, and local to a single computer. Feel free to do more well based testing!
Thank you!


Fix typos and wrong indentation

This allows to pip install and run the examples from master.
Thanks for this, it is cool stuff.

Wow, awesome!  Thanks for the PR.


Added ping command

Added ping command which can be used to ping a role so there is no fear of it being maliciously pinged if forgotten to be made un-ping-able again.
This may or may not have happened today and I wanted to show I was apologetic :joy:

The is not the approach we wish to go with.


Broken link in GridView documentation

https://gtk-rs.org/gtk4-rs/stable/latest/docs/gtk4/struct.GridView.html
The fourth paragraph reads "To learn more about the list widget framework, see the overview.". The URL behind "overview" is broken and points to 404 Page not found.

See #519


Blocking: Share button no longer working

Nothing happens when I press it.

For me sharing does work, unless I disable sharing http headers:

Then I get this error:
Uncaught TypeError: Cannot assign to read only property 'headers' of object '#<Object>'
    at Playground.tsx:1480
    at lodash.js:13379
    at lodash.js:4944

Could that be the issue you are seeing?


Change namespace of react package

See #10249

why webstories using custom react dependencies? and why not peerDependencies?

@c0b41 great question! we might switch to peerDependencies once we publish these packages to npm. Right now they are all private, so it does not matter at all.


Is it/will it be possible to use the same handler for 2 different events?

When i try to do, deployment work but it i got this error on sync:
Subgraph instance failed to run: Failed to handle Ethereum event with handler "handleMock": Function: Module doesn't have export handleMock, code: SubgraphSyncingFailure, id: QmNqVUUkBXNu134iGiPYmHt4Zy8ftYQzYYtjNKPyD2113Y

export function handleMock<T>(event: T): void {
  saveTransaction(event, 'MOCK');
}

I'm assuming AssemblyScript exports handleMock<T> under a different name than handleMock. I could see us supporting this at some point but there are no plans for it right now.
The following should work and feels like an acceptable workaround:
function handleMock<T>(event: T): void {
  saveTransaction(event, 'MOCK')
}

export handleSomeEvent(event: SomeEvent): void   { return handleMock(event) }
export handleOtherEvent(event: OtherEvent): void { return handleMock(event) }


Save LLMs

Will add tests. Have not supported saving Manifest yet - use of dictionary a bit weird

Will add tests. Have not supported saving Manifest yet - use of dictionary a bit weird

imo fine not support all of them if hard

Will add tests. Have not supported saving Manifest yet - use of dictionary a bit weird

imo fine not support all of them if hard

would be a good starter todo :)


Updated the order in side bar and generic asset to send asset

I think we missed updating the side bar font awesome icons somewhere?
Would be good to do that here

I think we missed updating the side bar font awesome icons somewhere?
Would be good to do that here

sure.. will see what needs to be done for that.

I think we missed updating the side bar font awesome icons somewhere?
Would be good to do that here

sure.. will see what needs to be done for that.

LGTM

@KarishmaBothara  Could you share it with design team to have a design review please?

@KarishmaBothara Could you share it with design team to have a design review please?

sure...thanks

@KarishmaBothara Could you share it with design team to have a design review please?

sure...thanks

approved by Lillian..


Bro is just NOT spawning bruh

I have spent the past 1 in a half days trying to get the giant to spawn in general.
Me and my friend had tested out a collection of new mods I put on it, including Advanced Company, Expanded Company, and the Variety mod. Expanded company added a Christmas village map. We went in and the mansion map was loaded, with the rolling giant a room away, we scanned him and everything. Then we left. Unfortunately, that has been the last time I've seen him so far.
At first, I was trying to get him to spawn in on offense with 100/100 chances to spawn, and he wouldn't. Later, I ran out of patience and went into the config settings and set him to spawn outside only, still wouldn't spawn. The only time I could've SWORN that it could have been him is when I had seen a mere silhouette outside, once. I spent entire in game days waiting for bro to spawn in even once so I could see my new configuration of him that I thought would make him more threatening to us. I even installed the enemy scanner mod to see if he was even spawning at all, I didn't see him, though I assumed the scanner still thought he was a coilhead.
That's when I figured the mod was bugged.
I then uninstalled The Variety mod (Didn't work).
I then uninstalled The Expanded company mod (Didn't work).
I then uninstalled The Advanced Company mod (Didn't work).
I finally made a whole new mod profile with just him and set him to spawn with a 100% chance to spawn outside on Experimentation. I waited nearly a whole ingame day, on a fresh save/run. Buddy did not spawn.
I'm not sure if my main save is having compatibility issues, if it's with the mod itself, or if it's on my end
Im leaving a link to my default mod profile incase you or somone else wants to identify the problem themselves for whatever reason.
As of now, I am done tryna get dawg to spawn In and now ima just play with different mods or something. Happy new year.

Hello there, sorry you haven't been able to find the giant.
Firstly, the spawn rate isn't a percentage, it's a weight, as all enemies have a weight that gets chosen from over time. Higher numbers just means a higher chance of getting picked, but not guaranteed. You can also go past 100 (which is the game's default) if you wanted to, albeit I didn't document that. I'll be making a patch detailing that in the config soon. For reference, it uses the same weights as the Coil Head.
Secondly, I don't see a mod profile link in your message. I also don't see a version number that you tested on, nor your config being used. More information will be useful to work with, so if can provide that, that would be great.
Thanks.

I just tested with v2.1.0 four times on Experimentation with a weight of 6, and got no giants, which makes sense as other monsters have higher weights so it lost in the picks. Which is very vanilla in that aspect.
I also tested once on March, which has a weight of 10 by default and got one on my first visit, which is a very low chance. So the mod is definitely functional.

Also turns out that a soft dependency to another mod, that was guarded from breaking the mod loading, managed to break the mod loading if you didn't have LethalSettings...
I'll be pushed a hotfix asap, if this ends up being your issue, I apologize 🫡. The fix will also have a probability curve, higher default chances, and some extra descriptions for what the numbers can do.


LICENSE warning in FirebaseMessaging pod

Describe your environment

Xcode version: 10.1 (10B61)
Firebase SDK version: 5.19.0
Firebase Component: Messaging
Component version: 3.4.0

Describe the problem
Seems that LICENSE file isn't integrated properly
Steps to reproduce:

Integrate Firebase Messaging via Cocoapods
Receive warning: [!] Unable to read the license file LICENSE for the spec FirebaseMessaging (3.4.0)

@tfonfara I'm not able to reproduce.   The pod was just published so I wonder if you have something out of sync in your configuration.
Would you try pod cache clean FirebaseMessaging and retry?

@paulb777 that fixed it! 👍


Add py2/3 features, convert print statements, add future requirement

I think this covers most Py3 stuff for this project. I haven't actually tried to run it yet but give it a go and see what happens.

adds from __future__ import absolute_import to the top of every file to ensure Py2&3 import compatibility
adds from __future__ import division to any file with division so that the Python 3 division behaviour is used in both Py2&3
adds from __future__ import print_function to any file with print so that Py2 can use the Py3 print function
adds future to requirements
for urlparse the following is used to have a consistent namespace between Py2&3:

from future.standard_library import install_aliases
install_aliases()
from urllib.parse import urlparse

Thank you for your help!


Name conflict with Microsoft’s official Fluent UI

See https://developer.microsoft.com/en-us/microsoft-365/blogs/ui-fabric-is-evolving-into-fluent-ui/
I think this absolutely warrants a name change in order to prevent confusion.

See https://developer.microsoft.com/en-us/microsoft-365/blogs/ui-fabric-is-evolving-into-fluent-ui/
I think this absolutely warrants a name change in order to prevent confusion.

Thank you for your suggestion.
The name change is currently under consideration, and there is still some work to be done after the name change, it will take some time.


Fix checkbox vertical position

Checkboxes (- [ ]) are not aligned correctly. I see that there are competing css rules for their vertical-align. A quick fix seems to be to just add more specificity to the overwritten rule, adding .task-list-label to the middle:
.markdown-source-view.mod-cm6 .task-list-label .task-list-item-checkbox {
    vertical-align: baseline;
}

Hi, please create a PR with the fix. Thank you. :)

Yeah still no PR, will try to create it for that issue, need to learn how to test them first

Unfortunately that CSS does not work anymore @wobbba, at least I tried to use it as a CSS snippet. Will see if I can figure out the fix, thats a shame this theme is not maintained anymore, such a beautiful theme

Any fixes for this? :(

Alrighty, the original comment fix didn't work, I just commented out this within the CSS. Doing this fixes it (maybe):
.markdown-source-view.mod-cm6 .task-list-item-checkbox {
/*	vertical-align: baseline*/
}

Before:

After:


Adding Instructions on how to use uv and venv for Swell installation

I used @ashiklom's instructions and included them in the documentation. I think py_lmod_installer might be confusing for some users so hope this is useful!
Also added a few Github links, feel free to suggest more sources.

@ashiklom I'd like a chance to try this out before this gets in.  I don't understand why something that seemed to have worked in the past no long seems to work - I insulated a version yesterday - the traditional way of installing - following the direction in SWELL doc. Then, I close my windows, the next day when I came and reloaded the modules - nothing worked - swell is not in the path.
I will try th uv way to see if any better.

@rtodling Let me know if you run into trouble! There's a lot of ways these different things can go wrong, depending on exactly what's in your .bashrc, etc. If it doesn't work, I could hop on a Teams call to try to debug interactively.


Make time based features testable

We should have an easy way to use TimeSkipScheduler to use in tests but still be able to have computation scheduler in the application.

I think we are still waiting for a new RxAndroid release so we can override schedulers with plugins, rather than SchedulerProvider abstraction.

I was thinking of a combination of time() Scheduler done the similar way as the plugins. We do not need the change to RxAndroid


Hotbar actions can't be dragged to remove/rearrange

Description
For some reason I was unable to rearrange my hotbar actions, they would only get pressed instead, I asked others and they said they have the same bug. Reconnecting did not help.
Reproduction
Uh, just drag actions?
Screenshots

I wasn't able to drag actions from action menu on hotbar as borg, so deleting any action becomes permanent till /ghost.


ASSERTION FAILURE: Name not found: pure_return

F* kindly asked me to file a bug report with the following words:
(Error) ASSERTION FAILURE: Name not found: pure_return
F* may be in an inconsistent state.
Please file a bug report, ideally with a minimized version of the program that triggered the error.

I am attaching an example file that I minimized as far as I managed without understanding what is going on: Bug2.txt
(Extension is .txt because github wanted it so. It is an .fst file.)
I used the latest "weekly" build from April 22. (https://github.com/FStarLang/binaries/blob/master/weekly/fstar_202104220032_13e688dda1_Linux_x86_64.tar.gz)

I pushed a "fix" to the master. There were a couple of issues:

pure_return is marked unfold, meaning that it is expected to be reduced by the typechecker before sending the query to smt. In the case of tactics, this unfolding has to be done by the tactic. The error message now says that the symbol is marked unfold, so smt expected it to be reduced already. In your example, we can add a norm [delta_qualifier ["unfold"]] step in the tactic (l_to_r ...; norm [delta_qualifier ["unfold"]])
Then there was a bug in the typechecker due to which this still left instances of pure_return that appeared under a blocked match. I fixed the bug here: https://github.com/FStarLang/FStar/commit/36d6c1db7157769e10d6e7f7a677f6cb8dcb4e01. So now, your example works once we add the unfold step.

Thanks for the report!

I will try it out when it reaches the weekly builds.
One question: Do I understand right after running a tactic, there are two cases: no unfold-symbols are there, and then norm [delta_qualifier ["unfold"]] does nothing. Or there are unfold-symbols, and then there will always be an error without a final norm [delta_qualifier ["unfold"]]? So what is the rationale of not doing this automatically before sending the subgoals to SMT?


Support Expo modules config

Description of the feature
Add IntelliSense for expo-module.config.json files.
Motivation
See https://github.com/expo/expo/blob/main/packages/expo-modules-autolinking/src/types.ts#L72-L146
Additional context
We could try and generate the code using TypeDoc. But I think we can just quickly start by creating a manual schema for now.

Continuing the discussion outlined in the PR:
There is no automated workflow to manage this schema yet. I'll keep the original issue open because of that. We have a few options:

Try to infer the json schema through typedoc from<EMAIL_ADDRESS>Add a schema to expo-modules-autolinking that we update here, just like eas-metadata.json

The manual schema lives here: https://github.com/expo/vscode-expo/blob/schemas/schema/expo-module.json


Add heroku config files

Adds all the env vars we need to run the app
Adds a buildpack that should let us deploy from a subdirectory

Note: added two files since I'm not sure which one heroku will pick up.

Looks like this needs to be merged before I can tinker with PR deploys. Will remove the unneeded file once I can get something working.


Document use of back level derby with ranger plugin

Dependabot proposed
Bump derby from <IP_ADDRESS> to <IP_ADDRESS> #1753
However we need substantive work for gaian to work with 10.15 so we should document our
usage & rationale for 10.8

Another reason to move Gaian to its own git repository.  Your proposal is good enough for now but we need to keep this in mind as we move to the data virtualization release.

Note. will also skip any hadoop dependency updates such as https://github.com/odpi/egeria/pull/1866 as the current version of ranger being used depends on older hadoop


Proper way for creating custom UserControls with Stylet

Hello,
First of all thank you for creating Stylet. It really helped me to understand a lot about mvvm.
So far I managed to work smooth with my views and viewmodels in application, but now i stuck while creating reusable ListBox with some custom properties like "ListName" and different ItemTemplate depending on collection of objects i provide in MyCustomItemsSource property.
I can't figure out how can i use attached properties or dependency properties inside my UserControl while i deleted all code behind. Ideally i want to use localized string like:
<UserControls:EditableListBoxWithHeader ListName="{Translate:Loc MyLocalizedString}" s:View.Model="{Binding MyListOfObjects}" />
and be able to define custom ItemTemplate for ListBox inside EditableListBoxWithHeader user control.
There comes simplified user control i want to create:
`public partial class EditableListBoxWithHeader : UserControl
{
public string ListName
{
get { return (string)GetValue(ListNameProperty); }
set { SetValue(ListNameProperty, value); }
}
    public static DependencyProperty ListNameProperty =
        DependencyProperty.Register("ListName", typeof(string), typeof(EditableListBoxWithHeader), new PropertyMetadata("default-ListName-value"));

    public EditableListBoxWithHeader()
    {

    }
}`

and xaml for it, but it for sure does not work .
`

    </ListBox>
</Grid>

`
I will maybe have some more questions but I should move forward after passing some dependency or attached properties and list of objects to ListBox inside UserControl.

I can't figure out how can i use attached properties or dependency properties inside my UserControl while i deleted all code behind.

For UserControls,  I wouldn't delete the codebehind. UserControls aren't part of normal MVVM: they're pretty low-level, they don't have a ViewModel, and they often have custom stuff in the codebehind.


Changing entityIds string to long

I have found some special cases that we need to take care of

On pointer events
Avatar checks nulls
AvatarEntityReference
AttachToFirstpersonreference
Builder In World

PhysicsCast

Possible problems:
MessagingBus - The bus is expecting an entityID string but we only have the hash

Desktop should change also the implementation of entityId?

Protocol problems:

Engine interface:

DCLBuilderWebInterface:

New approach will probably will have problems with:
avatarReport
Builder
BuilderInWorld


mx_records - postal.yml

Hello
Can some one tell me if I need the "-" before the dns setting in the postal.yml file as indicated in the example.
dns:
mx_records:
- mail.yourdomain.com

The - is part of the YAML syntax and shouldn't be part of your DNS record.


[BUG] Unknown option --metadata-file

When I try to export a note obsidian-pandoc return an error:
Pandoc export failed: Unknown option --metadata-file. Try pandoc --help for more information.

Easy fix is to comment this lines in main.js but it's not ideal:
if (input.metadataFile)
        args.push('--metadata-file', input.metadataFile);

I work o Windows 10
pandoc --version output:
pandoc <IP_ADDRESS>
Compiled with pandoc-types <IP_ADDRESS>, texmath <IP_ADDRESS>, skylighting 0.7.2
Default user data directory: C:\Users\user\AppData\Roaming\pandoc
Copyright (C) 2006-2018 John MacFarlane
Web:  http://pandoc.org
This is free software; see the source for copying conditions.
There is no warranty, not even for merchantability or fitness
for a particular purpose.

I got the same error. The issue was resolved by changing the Pandoc Plugin setting "Export files from HTML or markdown?" from HTML to markdown.

Another way is to select all content in a note, copy it, and directly paste to a new Word document. The results appear good.


fuzzgen: Disable unimplemented riscv64 ops

👋 Hey,
This is a WIP PR to disable all the missing and broken instructions in fuzzgen for the riscv64 backend.

@uweigand, FYI: This PR illustrates what we need to start routinely fuzz-testing additional Cranelift backends. I'd love to start testing s390x on our OSS-Fuzz quota too. For non-x86, OSS-Fuzz can only test that codegen succeeds, not that the generated code runs correctly, but I expect that to be enough to catch many bugs. Of particular interest right now, this should help us catch non-SSA lowerings.
The steps are:

add the s390x feature to the cranelift-codegen dependency in fuzz/Cargo.toml (optionally, you can temporarily remove the other backends there to focus on that one)
run cargo +nightly fuzz run cranelift-icache on any host, and/or cargo +nightly fuzz run cranelift-fuzzgen on an s390x host—whichever you have more CPU cycles available on
as it finds bugs, add backend-specific exceptions for known issues with lowering particular instructions at particular types, as illustrated by this PR

Once the fuzzer runs for an hour without finding new bugs it's probably ready.
These fuzz targets don't yet generate all possible Cranelift instructions, so you won't have to write down exceptions for every unimplemented thing. It's currently limited to scalar types and arithmetic/load/store operations mostly, but we're incrementally expanding coverage.

@jameysharp s390x support PR now posted here: https://github.com/bytecodealliance/wasmtime/pull/5596
Thanks for pointing this out!


Arrangement of controls on high DPI Monitors are different.

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

Double click on a match.

Expected behavior
Arrangement of controls on high DPI Monitors are different.
Screenshots

Design should be made fix on 96 DPI, so ok.


Bar chart Log scale showing x-axis lines below origin line

I have created a bar chart using below code. Which shows one line below x-axis.
And also while using logarithmic y-axis shows axis line overlapping if chart is small.
How can I show axises in regular interval, Like for this chart [0,500,1000, 5000, 10000..etc].

@neway003 could you put this in a fiddle so that we can debug?

Below link to fiddle
https://jsfiddle.net/neway003/532zc37o/

Looks to be related to the min: 0 and beginAtZero: true settings

Thankyou, I have commented beginAtZero: true

and kept min: 0 because if one of the ata point with value 1 and other one is much bigger like 6000, data point with value 1 is not shown as bar.

Now its working, I have updated the fiddle.
Thank you.
And also could you please suggest me how to remove some of the y axis interval shown which are overlapping.


Please help with Neural Stack Machine

Hi
Thank you very much for your tutorials. Please could you roughly tell how we can implemented the augmented memory module i.e. the neural stack of the Neural Stack Machine in tensorflow. I have understood the theory and I know how CNNs/RNNs/Autoencoders work in tensorflow but I'm unable to translate the augmented memory aspect into tensorflow.
Please help
Thank you

Dear arsenious,
Nice to know about your interest. Here are some resources you can follow:

https://iamtrask.github.io/2016/02/25/deepminds-neural-stack-machine/
https://github.com/carpedm20/NTM-tensorflow

I am currently creating a tutorial on neural Turing machine and DNC. Please follow me on twitter or github to get the updates.


Avoid usage of JSI Specializations which may use RTTI

Summary:
getNativeState() and the version of getHostObject() not specialized to HostObject do a dynamic_cast inside of an assert() to do a type check.
Most of RN for Android is built without RTTI enabled, but hadn't run into issues with these because fbandroid sets NDEBUG even in debug builds, causing any raw assert() to be compiled out.
All these functions really do is a checked cast, where the check is only in debug, and already compiled out on Android. This changes them to do a static_cast themselves instead.
This solution is probably preferable to conditioning the existing asserts on RTTI availability within JSI, since some libraries do build with RTTI, and changing behavior dependent on it in a shared header seems like a recipe for ODR. Though I'm not sure why casting is the API responsibility of JSI in general.
Static asserts are added so that new usages of functions meant to use RTTI are not added to units compiled internally with -frtti.
Changelog: [Internal]
Differential Revision: D45814500

Closing as abandoned internally


⚠ Bofrost.at has degraded performance

In 31e0843, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1304 ms

Resolved: Bofrost.at performance has improved in a0d5182.


[WIP] Add policy to Container Images

@screeley44 - I've tried to apply your tagging for container entities (ac28209cf11b9583270dc87d1fe31b420fc08360) patch on ContainerImages, but nothing I do doesn't show the "Policy" button. Any idea what's missing?
cc @simon3z

hi @oschreib @simon3z ,
So I think you just have the singular and plural code reversed for your toolbars, try below suggestions and see what happens...
So "container_images_policy (plural) in the product/toolbars/container_images_center_tb.yaml should be this:

:name: container_image_policy
:items:

:buttonSelect: container_image_policy_choice
:image: policy
:title: Policy
:text: Policy
:items:

:button: container_image_tag
:image: tag
:text: 'Edit Tags'
:title: 'Edit Tags for this #{ui_lookup(:table=>"container_images")}'
:url_parms: 'main_div'
:enabled: 'false'
:onwhen: '1+'

and "container_image_policy"  (singlular) in the product/toolbars/container_image_center_tb.yaml should be this:

:name: container_image_policy
:items:

:buttonSelect: container_image_policy_choice
:image: policy
:title: Policy
:text: Policy
:items:

:button: container_image_tag
:image: tag
:text: 'Edit Tags'
:title: 'Edit Tags for this #{ui_lookup(:table=>"container_image")}'

@oschreib - sorry that inline code turned out ugly, forgot about markdown...this looks better
hi @oschreib @simon3z ,
So I think you just have the singular and plural code reversed for your toolbars, try below suggestions and see what happens...
So "container_images_policy (plural) in the product/toolbars/container_images_center_tb.yaml should be this:
- :name: container_image_policy
  :items:
  - :buttonSelect: container_image_policy_choice
    :image: policy
    :title: Policy
    :text: Policy
    :items:
    - :button: container_image_tag
      :image: tag
      :text: 'Edit Tags'
      :title: 'Edit Tags for this #{ui_lookup(:table=>"container_images")}'
      :url_parms: 'main_div'
      :enabled: 'false'
      :onwhen: '1+'

and "container_image_policy" (singlular) in the product/toolbars/container_image_center_tb.yaml should be this:
- :name: container_image_policy
  :items:
  - :buttonSelect: container_image_policy_choice
    :image: policy
    :title: Policy
    :text: Policy
    :items:
    - :button: container_image_tag
      :image: tag
      :text: 'Edit Tags'
      :title: 'Edit Tags for this #{ui_lookup(:table=>"container_image")}'

@screeley44 thanks for the help, but I figured it out few minutes ago, it seems that "container_image" was missing from toolbar_chooser.rb (line 394)

@oschreib -  ah, cool, glad you got it working!

#3776 Already covers this.

https://github.com/ManageIQ/manageiq/pull/3776 already covers this issue.


0.2.12 typo fix + doc link should be to docs/ directly + ...

0.2.12 doc link should be to docs/ directly + note about shell security + typo fix (for for) + seperate the monthly from the stable

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
Assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

Preview:

This is the common pattern we follow on all navigation pages. You can fully
navigate the site without ever using the left-hand sidebar, which is
essential for a good experience on mobile, where the left sidebar is hidden
and inconvenient to use.
On Wed, Mar 28, 2018 at 3:42 PM, Laurent Demailly<EMAIL_ADDRESS>wrote:

Preview:
[image: screen shot 2018-03-28 at 3 40 09 pm]
https://user-images.githubusercontent.com/3664595/38060204-9f8cdfb6-329e-11e8-8c1e-14c607528f92.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio.github.io/pull/1115#issuecomment-377062079,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVucHe5ST7abyHn8V6TeGuxtwgTEmVblks5tjBHdgaJpZM4S_Ugu
.

you can't actually reach the release notes through navigation:
the about link goes to https://istio.io/about/intro.html
which if you don't have the nav bar is a dead end
should that be fixed (in a separate PR)

same for blog
ps: I do agree the about should go to an actual page/content (blog is more arguable) so that's not what should be fixed, I think mobile can pop out the left bar when needed

lastly https://istio.io/about/contribute/
doesn't have a double TOC either

Merging this for now as it's incrementally better before the 0.7 branch gets cut. we can further refine navigation later

Man, you are sure stubborn.
The contribute link does have a TOC in it, I fixed that a little while ago and its on preliminary.istio.io. And the design intent is that navigation is possible without the left side nav. Even though I mention this, you proceed with checking this in even though it is not a necessary change in any way and is just your whim, ignoring requests and bypassing the approval process.


Raise error if updating a parameter that already exists with check_already_exists=False

This will force people to use check_already_exists=False sparingly, and help to catch bugs related to misnaming parameters (this is caught if check_already_exists=True, but not if it's False

Yes

I guess if you need to do both you will need to call the command twice, but it is not too bad (especially if it helps catching typos).

Hey! Can you let me know that what exactly is the issue. where does it needs to eb implemented?

Hi! It is not good practice to assign multiple issues to a new user at once. Please choose one issue, and after it has been merged we can assign you another one. This applies to many other issues where multiple users have ask to get them assigned, so please reply in the one you want to tackle first.

Hello @brosaplanella, Please assign this to me. I don't have any assigned task right now.

Agreed upon @brosaplanella else, it will be unfair for other contributors. I have seen the comments on other issues so it is suggested that you work on particular and then comment on other. I was about to work on this as I have also mentioned about issue to ask upon but that's okay. keep up the good work

Basically, the idea is to modify the update method in parameter_values.py to throw the error described above.

@brosaplanella Should I work on this?

hey @jaskiratsingh2000 , currently I am working on it.

@brosaplanella Should I work on this?

@jaskiratsingh2000 I assigned you #2695 earlier, are you happy tackling that one?

Hey Sure!

Happy to both

Hello, @brosaplanella,
https://github.com/pybamm-team/PyBaMM/blob/4f03df423bc63f9b4bd5f7e926c78a2ad1af1f87/pybamm/parameters/parameter_values.py#L313-L319
Is this error is raised for the same case or the other case?

I added a portion of this change in #3382. Although that PR only logs a warning. I did a partial change due to the number of tests and examples that would need to be updated by the change.
For making it an error to update parameters with check_already_exists=False, I think one of the following two options could be good:

Change check_already_exists to something along the lines of add_new_parameters. My reasoning is that check_already_exists=True sounds like "make sure you don't overwrite parameters" while check_already_exists=False sounds like "Don't check, just create or update". It would be less confusing if the variable name was closer to the intended usage once the error is in place.
Split the update() function into two different functions. There could be an update_parameter() and an add_new_parameter(). This would get rid of check_already_exists and avoid confusion.

Of those two options, I would lean towards (2) since each function would have a single purpose and there would be no ambiguity from the name of the variable.

I agree with what you suggest, and I also think option 2 is clearer.


Feat: Config Logic Enhancement

Signed-off-by: Daniel Hu<EMAIL_ADDRESS>Pre-Checklist
Note: please complete ALL items in the following checklist.

[ ] I have read through the CONTRIBUTING.md documentation.
[ ] My code has the necessary comments and documentation (if needed).
[ ] I have added relevant tests

Description

Related Issues

New Behavior (screenshots if needed)

I think the structure could be made clearer further. @daniel-hutao
For example, the current renderVariables(varFileName string, configFileBytes []byte) is fused based on var filename and toolFilesConfigBytes.
This results in forcing the specification of varFile to be standalone, which is hard to modify and expand.
Why don't we change the parameters to, varFileMap and tooConfigBytes? This would make it easy to generate var map in any way, such as from a separate file, or rather a small portion of an entire config file.
Even further, we could define the operations of getting var map, tool bytes as two interfaces. Then just implement a generic function that stitches together the final config from var map and tool bytes. Then write some implementation classes to read var map and tool bytes from a single file or multiple files.
Similarly, you can do the same for reading state.
Translated with www.DeepL.com/Translator (free version)

I think the structure could be made clearer further. @daniel-hutao
For example, renderVariables(varFileName string, configFileBytes []byte) is based on var filename and toolFilesConfigBytes.
This results in forcing the specification of varFile to be standalone, which is hard to modify and expand.
Why don't we change the parameters to, varMap and toolConfigBytes? This would make it easy to generate final var map in any way, such as from a separate file, or rather a small portion of an entire config file.
Even further, we could define the operations of getting var map, tool bytes as two interfaces. Then just implement a generic function that stitches together the final config from var map and tool bytes. Then write some implementation classes to read var map and tool bytes from a single file or multiple files.
Similarly, you can do the same for reading state.

@aFlyBird0 Please don't rush, it's still wip status; I'm just refactoring in steps, please wait until the final version to review

Okay, just enjoy your refactor tour and notice me when finished. @daniel-hutao

After more than an hour of git manipulation, I decided to abandon this pr and redevelop it based on the latest code because the wip time was too long and the mainline code had been changed too much and the conflict resolution was too costly.


Make sure the element is in DOM, before destroying it.

Description
Make sure the element is in DOM, before destroying it.
Related to:
https://github.com/angular/angular/issues/14252
Steps to reproduce

npm -g install ionic
npm install
ionic serve
Click button "Show/Hide Chart"

Result
An error occurs.
TypeError: Cannot read property 'removeEventListener' of undefined
Expected result
No error should pop up.
Attachments
zoombarTestCase.zip

For all who can't wait for bug fix here is a workaround, that works for me:
In IgControlBase.prototype.ngOnDestroy - function I inserted following:
if (this._widgetName!= "igZoombar") {  ... content of function .... }


Admin UI : Deployment tab does not show the deployed jobs

While deploying the Spring batch job, which has properties file reading functionality from environment variable with context:property-placeholder tag, I have observed below 2 weird things:

Deployment tab does not show the job deployed from Admin UI/Cmdline.
Execution tab show the currently executing job with DELETED mark.

This is a bit of urgent priority as our operations team is not able to use the Admin UI for deploying the XD jobs for that matter.
Thanks,
Pratik

While deploying the Spring batch job, which has properties file reading functionality from environment variable with context:property-placeholder tag

Is this happening only for this type of job or everything?
If everything, which browser are you using? (If you're seeing similar experience in the shell, then the browser question wouldn't apply - it is unclear from your description though).
Perhaps also share the error(s) from the browser console, too.

@sabbyanandan No this is not a browser issue for sure. This is happening for Spring batch type of jobs only and I guess that too when we use property placeholder tag.
And from shell also the deployment and execution list is not visible for this type of job.
Additionally, I have below constants which are must for XD to take the deployment of Spring Batch custom module:
## XD related constants
xd.job.name=fileDownloadSourceMatch
dateFormat=yyyy-MM-dd
numberFormat=
listeners=
makeUnique=false

If I don't include above constants, the job deployment fails with below exception:
missing ${xd.job.name}...
Thanks for your help!
-Pratik

Looks like others may also face the same issue
http://stackoverflow.com/questions/31596522/spring-batch-job-execution-does-not-show-in-executions-tab


Managing keys properly in the DEL method with Redis in cluster mode

Currently, the keys are not grouped by slots, which results in an error: "multi key command with different key slots are not allowed."

Hi @arielzach,
Please use the MDel helper.


Unable to load webauthn.js from another domain

Preflight checklist

[X] I could not find a solution in the existing issues, docs, nor discussions.
[X] I agree to follow this project's Code of Conduct.
[X] I have read and am following this repository's Contribution Guidelines.
[X] This issue affects my Ory Cloud project.
[ ] I have joined the Ory Community Slack.
[ ] I am signed up to the Ory Security Patch Newsletter.

Describe the bug
Using Ory cloud and custom domain.
Application is running at domain.com and custom domain is ory.domain.com
When using webauthn, the following error occurs when trying to load the webauthn.js via javascript:
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://ory.domain.com/api/kratos/public/.well-known/ory/webauthn.js. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing). Status code: 200.
The script is added to the html as:
<script async="" crossorigin="anonymous" integrity="sha512-E3ctShTQEYTkfWrjztRCbP77lN7L0jJC2IOd6j8vqUKslvqhX/Ho3QxlQJIeTI78krzAWUQlDXd9JQ0PZlKhzQ==" referrerpolicy="no-referrer" src="https://ory.domain.com/api/kratos/public/.well-known/ory/webauthn.js" type="text/javascript"></script>
From the documentation below:
https://developer.mozilla.org/en-US/docs/Web/Security/Subresource_Integrity
It looks like the Ory cloud server should be adding the Access-Control-Allow-Origin: * header or there should be a way to add CORS allowed domains.
Reproducing the bug
Try to load the webauthn.js script from a different domain.
Relevant log output
No response
Relevant configuration
No response
Version
0.0.1-alpha.76
On which operating system are you observing this issue?
Windows
In which environment are you deploying?
Docker Compose
Additional Context
No response

Thank you for the report! We will fix that issue

Hi,
Just checking if there is any update on this issue?
Best,

Sorry for the lack of update @Benehiko is working on it - could you give an estimate of when CORS config will be available?

Currently CORS is still being worked on. It should land on staging probably next week and if all is good will be released :)

Thank you for the update! Looking forward to see it released.

Hello, CORS has been released last week!
https://www.ory.sh/docs/guides/custom-domains

Tried it late last week and seems to be working. Thank you.


Adding Discard and pull changes section in Git Sync

@Pranay105 what exact steps do we need to follow for discard and pull?


Temporary move rememberComposableLambda from commonMain to jvmAndNativeMain

Description:
Currently, compose compiler plugin doesn't handle properly this function when targeting web.
We will move the function back to commonMain, after all compiler plugins support the new function in web targets.

Fixes CMP-1547 for desktop and k/native. Web is still affected by the issue.
Testing

Open :compose:mpp:demo on desktop
Go to Components -> Dialogs -> Dialog with text field
Close the dialog and Go back (press the back button)
No unexpected dialog is shown.

To reproduce the issue, comment rememberComposableLambda.
Repeat the steps above.
An unexpected dialog will be shown after back button is pressed.
Note: when we move the function back, it means that CM 1.7 projects won't be able to use kotlin 2.0 with web targets. Only kotlin 2.0.10 if the compiler fix is accepted.

The change needed in compose compiler was already merged to kotlin 2.0.10 and 2.0.20 branches.
Therefore, this PR became obsolete.
We'll update the compose compiler plugin for 1.9.2x too and then web should work the same as desktop and ios.


v4.2.1

Description

Fixed the test TestClearDatabaseView that was failing if run multiple times.
Fixed a validation crash when a nullable field doesn't pass validation.

Possible drawbacks
None.
Related issue(s)
None.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4666

Relevant Lines:
4712

💛  - Coveralls


Entity Test failed because of Unmatched Zone in ZonedDateTime between DB and ElasticSearch

After I install Entity-audit on Jhipster 3.4.0, All XXXResourceIntTest failed because lastModifiedDate and createdDate are not matched in following test. Also you can find the failure trace below. It seems that Entity has different time zones between DB and ElasticSearch.
FYI, if run ZonedDateTime.now(), I get London TimeZone which different from UTC from ElasticSearch.
`    @Test
@Transactional
public void createAddress() throws Exception {
int databaseSizeBeforeCreate = addressRepository.findAll().size();
    // Create the Address
    AddressDTO addressDTO = addressMapper.addressToAddressDTO(address);

    restAddressMockMvc
            .perform(post("/api/addresses").contentType(TestUtil.APPLICATION_JSON_UTF8)
                    .content(TestUtil.convertObjectToJsonBytes(addressDTO)))
            .andExpect(status().isCreated());

    // Validate the Address in the database
    List<Address> addresses = addressRepository.findAll();
    assertThat(addresses).hasSize(databaseSizeBeforeCreate + 1);
    Address testAddress = addresses.get(addresses.size() - 1);
    assertThat(testAddress.getName()).isEqualTo(DEFAULT_NAME);
    assertThat(testAddress.getAddress()).isEqualTo(DEFAULT_ADDRESS);

    // Validate the Address in ElasticSearch
    Address addressEs = addressSearchRepository.findOne(testAddress.getId());
    assertThat(addressEs).isEqualToComparingFieldByField(testAddress);
}`

The failure trace are:

java.lang.AssertionError:
Expecting values:
<[2016-06-23T09:30:33.072+01:00[Europe/London],
2016-06-23T09:30:33.072+01:00[Europe/London]]>
in fields:
<["createdDate", "lastModifiedDate"]>
but were:
<[2016-06-23T08:30:33.072Z[UTC], 2016-06-23T08:30:33.072Z[UTC]]>
in <Address{id=5, name='AAAAA', address='AAAAA'}>.
Comparison was performed on all fields
at com.espion.ems.web.rest.AddressResourceIntTest.createAddress(AddressResourceIntTest.java:115)

I might not have time to work on this could you try to find out the root cause and provide a PR if possible?

I got the same issue, it happens after I updated the code to support auditing entity. The two times are equivalent but the isEqualToComparingFieldByField method doesn't undestand timezones.
Comparison was performed on all fields
VenuesFeatureResourceIntTest.createVenuesFeature:109
Expecting values:
<[2016-07-04T21:42:01.473-03:00[America/Cayenne],
2016-07-04T21:42:01.473-03:00[America/Cayenne]]>
in fields:
<["createdDate", "lastModifiedDate"]>
but were:
<[2016-07-05T00:42:01.473Z[GMT], 2016-07-05T00:42:01.473Z[GMT]]>
in <VenuesFeature{id=6, name='AAAAA'}>.
Comparison was performed on all fields
VenuesFeatureResourceIntTest.updateVenuesFeature:194
Expecting values:
<["anonymous", 2016-07-04T21:42:01.329-03:00[America/Cayenne]]>
in fields:
<["lastModifiedBy", "lastModifiedDate"]>
but were:
<[null, 2016-07-05T00:42:01.066Z[GMT]]>
in <VenuesFeature{id=4, name='BBBBB'}>.
Comparison was performed on all fields

I think this is happening only for the posgresDB for MySQL the tests are passing @Jie-Yang and @mozartf what DB are you guys using? can you post your yo-rc.json
Im not much of a Posgres user hence would appreciate any help here

ok, the failure might be related to presence of elasticsearch as well

@hipster-labs/developers does anyone have an idea about this?

Hi, I faced this issue a while ago. But I can tell a few things I remember:

My database is MySql.
This is a timezone issue. The times in the test are off by three hours,
exactly my timezone difference to GMT.
The time timezone information in the database seemed to be correct at the
time; the problem was the time information stored in elasticsearch. The
search is affected by this misinformation.
At the time I was playing with the generator-jhipster-entity-audit, that
could have broken some tests.

I reverted the changes to a previous version of the project(previous
Jhipster). So I'm not able to give you any more information, or reproduce
the probles
Thanks for your help, any more questions that you have I'll try my best to
answer because I'm interested in giving an upgrade on the Jhipster version
of my project.
Regards,
Mozart
Em 30 de ago de 2016 15:16, "Deepu K Sasidharan"<EMAIL_ADDRESS>escreveu:

@hipster-labs/developers
https://github.com/orgs/hipster-labs/teams/developers does anyone have
an idea about this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hipster-labs/generator-jhipster-entity-audit/issues/37#issuecomment-243530623,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQbbm0Wa6ji3A3toi8nisVccIXAq3Ytoks5qlHNpgaJpZM4I8mIF
.

Please find my .yo-rc.yml file below. In my case, h2 is used to run tests.
{   "generator-jhipster": {     "jhipsterVersion": "3.4.0",     "baseName": "ABC",     "packageName": "com.es.abc",     "packageFolder": "com/es/abc",     "serverPort": "8080",     "authenticationType": "session",     "hibernateCache": "ehcache",     "clusteredHttpSession": "no",     "websocket": "no",     "databaseType": "sql",     "devDatabaseType": "h2Disk",     "prodDatabaseType": "postgresql",     "searchEngine": "elasticsearch",     "buildTool": "gradle",     "enableSocialSignIn": false,     "rememberMeKey": "e677704271bdae415612315ae582fd5ec994282",     "useSass": false,     "applicationType": "monolith",     "testFrameworks": [       "gatling"     ],     "jhiPrefix": "jhi",     "enableTranslation": false   },   "generator-jhipster-entity-audit": {     "auditFramework": "custom"   } }

Hi, I faced this issue and I think the problem is that JpaRepository.save(entity) does not return the updated Audit  fields (lastModifiedDate) after saving changes before doing the subsecuent myEntitySearchRepository.save(entity). So the changes on that fields do not reflect on Elasticsearch data.
See: http://stackoverflow.com/questions/20382586/spring-data-repository-save-not-returning-instance-with-updated-audit-fields
Also the timeZone problem when comparing datetimes is present, but that would be the smaller issue in this case.
I really don't have much more time to spend on this, but I spect this help somebody to address a solution.

@santiagofalces : can you have a look at this https://github.com/jhipster/generator-jhipster/issues/4373 too ? Maybe it's related, but I'm not sure

@pascalgrimaud For me, they are two separated issues, because in this case the DateTime values are different Instants. So this is not a DateTimeFormat error, it's storing an obsolete value for field "lastModifiedDate" when updating the search repository.

Hi,
I investigated a bit on this issue.
Into the service addressServiceImpl, the addressSearchRepository.save(address) return an address object with good values.
public AddressDTO save(AddressDTO addressDTO) {
    log.debug("Request to save Adress : {}", addressDTO);
    Address address = addressMapper.addressDTOToAddress(addressDTO);
    address = addressRepository.save(address);
    AddressDTO result = addressMapper.addressToAddressDTO(address);
    addressSearchRepository.save(address);
    return result;
}

But if you use addressSearchRepository.findOne(address.getId()) just before the return, the object has bad values.
The ZonedDateTime fields have 1 hour shift.
It looks like a time zone error during a conversion.

Hi, Looks like adding these lines in the generated ElasticSearchConfiguration.java working for me without any other changes in the JHipster generated code, please see if its the correct way to do. But I still see update entity tests failing.
objectMapper.configure(DeserializationFeature.ADJUST_DATES_TO_CONTEXT_TIME_ZONE, false);             objectMapper.configure(SerializationFeature.WRITE_DATES_WITH_ZONE_ID, true);
Full code:
`@Configuration
public class ElasticSearchConfiguration {
@Bean
public ElasticsearchTemplate elasticsearchTemplate(Client client, Jackson2ObjectMapperBuilder jackson2ObjectMapperBuilder) {
    return new ElasticsearchTemplate(client, new CustomEntityMapper(jackson2ObjectMapperBuilder.createXmlMapper(false).build()));
}

public class CustomEntityMapper implements EntityMapper {

    private ObjectMapper objectMapper;

    public CustomEntityMapper(ObjectMapper objectMapper) {
        this.objectMapper = objectMapper;
        objectMapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
        objectMapper.configure(DeserializationFeature.ACCEPT_SINGLE_VALUE_AS_ARRAY, true);
        objectMapper.configure(DeserializationFeature.ADJUST_DATES_TO_CONTEXT_TIME_ZONE, false);
        objectMapper.configure(SerializationFeature.WRITE_DATES_WITH_ZONE_ID, true);
    }

    @Override
    public String mapToString(Object object) throws IOException {
        return objectMapper.writeValueAsString(object);
    }

    @Override
    public <T> T mapToObject(String source, Class<T> clazz) throws IOException {
        return objectMapper.readValue(source, clazz);
    }
}

}`

could you submit this as a PR
Thanks & Regards,
Deepu
On Tue, Dec 13, 2016 at 11:32 AM, jeyakumarsm<EMAIL_ADDRESS>wrote:

Hi, Looks like adding these lines in the generated
ElasticSearchConfiguration.java working for me without any other changes
in the JHipster generated code, please see if its the correct way to do.
But I still see update entity tests failing.
objectMapper.configure(DeserializationFeature.ADJUST_DATES_TO_CONTEXT_TIME_ZONE,
false); objectMapper.configure(SerializationFeature.WRITE_DATES_WITH_ZONE_ID,
true);
Full code:
`@Configuration https://github.com/Configuration
public class ElasticSearchConfiguration {
@Bean
public ElasticsearchTemplate elasticsearchTemplate(Client client, Jackson2ObjectMapperBuilder jackson2ObjectMapperBuilder) {
return new ElasticsearchTemplate(client, new CustomEntityMapper(jackson2ObjectMapperBuilder.createXmlMapper(false).build()));
}
public class CustomEntityMapper implements EntityMapper {
private ObjectMapper objectMapper;

public CustomEntityMapper(ObjectMapper objectMapper) {
    this.objectMapper = objectMapper;
    objectMapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
    objectMapper.configure(DeserializationFeature.ACCEPT_SINGLE_VALUE_AS_ARRAY, true);
    objectMapper.configure(DeserializationFeature.ADJUST_DATES_TO_CONTEXT_TIME_ZONE, false);
    objectMapper.configure(SerializationFeature.WRITE_DATES_WITH_ZONE_ID, true);
}

@Override
public String mapToString(Object object) throws IOException {
    return objectMapper.writeValueAsString(object);
}

@Override
public <T> T mapToObject(String source, Class<T> clazz) throws IOException {
    return objectMapper.readValue(source, clazz);
}

}
}`
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/hipster-labs/generator-jhipster-entity-audit/issues/37#issuecomment-266703406,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABDlF9_DxP_dnlOcWymftaOzZDnYXa9Eks5rHnQpgaJpZM4I8mIF
.

I think this is just an issue with how you test the datetime value.
2016-07-04T21:42:01.473-03:00[America/Cayenne] and 2016-07-05T00:42:01.066Z[GMT] hold the same Instant but are not equal because ES doesn't preserve the timezone. So you can't compare with equals() you need to use ZonedDateTime::isEqual.

@cbornet could you please do a PR

Well, that would be a fix in the main entity generator. What I don't understand is that ZonedDateTime+ES+not UTC Timezone should already be tested by Travis.
cc @pascalgrimaud

Hi @deepu105  first of all I m not sure what I have done is correct or not, but things are just working. And I m so new to JHipster and Generators, I might not be able to PR any time soon; Someone else can help this time, I can try from next time onwards. BTW, I have found out the root cause for update tests failures: tests started working when I used saveAndFlush in service implementation. So I think we have two solutions: 1) use saveAndFlush() instead of just save() in service/resource classes. 2) Invoke em.flush() just before findAll() call in update entity tests.

Sorry, I take back the second solution, only saveAndFlush works on update entity tests.

Any update on this ?

Closing since Friday xed with latest version


[Feature Request]Please simplify the judgment of param's load status

When I want to determine whether an attribute is loaded or not, the current API has only two ways
boolean pageLoaded = ImmutableObjects.isLoaded(role, "page");
boolean pageLoaded = ImmutableObjects.isLoaded(role, 12);

Both of these are not very user-friendly, and both hard-coded strings and numbers are error-prone。
I think it can be simplified to
boolean pageLoaded = ImmutableObjects.isLoaded(role, role.page());

Even more simpliy like this
boolean pageLoaded = ImmutableObjects.isLoaded(role.page());

That API cannot work
If "page" is not loaded, you cannot access it.
The current usage has no problems, there are used by the framework itself or other upper level frameworks:
If user want to use it directly, do it like this.
Java
private static final NAME = ImmutableProps.get(TreeNodeTable.class, TreeNodeTable::name);

private static final CHILD_NODES = ImmutableProps.get(TreeNodeTableEx::class, TreeNodeTableEx::childNodes);

public void f(TreeNode treeNode) {
    boolean namedLoaded = ImmtableObjects.isLoaded(treeNode, NAME.getId());
    boolean childNodesLoaded = ImmutabeObjects.isLoaded(treeNode, CHILD_NODES.getId());
}

Kotlin
private val NAME = TreeNode::name.toImmutableProp()
private val CHILD_NODES = TreeNode::childNames.toImmutableProp()
fun f(treeNode: TreeNode) {
    val namedLoaded = ImmutableObjects.isLoaded(treeNode, NAME.getId());
    val childNodesLoaded = ImmutableObjects.isLoaded(treeNode, CHILD_NODES.getId());
}

Next, I will add the third method: isLoaded(Object, ImmutableProp), you can do it like this
ImmutableObjects.isLoaded(treeNode, NAME);

0.2.5 added third method


Playback in Docker container not want to works

When I try playback video downloaded I get unknown format error when I enable trans-coding I get this error in console.

node:events:306

      throw er; // Unhandled 'error' event

      ^

Error: Output stream closed

    at Timeout._onTimeout (/opt/youtube-dl-react-viewer/youtube-dl-express-backend/node_modules/fluent-ffmpeg/lib/processor.js:491:25)

    at listOnTimeout (node:internal/timers:556:17)

    at processTimers (node:internal/timers:499:7)

Emitted 'error' event on FfmpegCommand instance at:

    at emitEnd (/opt/youtube-dl-react-viewer/youtube-dl-express-backend/node_modules/fluent-ffmpeg/lib/processor.js:424:16)

    at Timeout._onTimeout (/opt/youtube-dl-react-viewer/youtube-dl-express-backend/node_modules/fluent-ffmpeg/lib/processor.js:491:17)

    at listOnTimeout (node:internal/timers:556:17)

    at processTimers (node:internal/timers:499:7)

Need some more info to try and figure out what is causing this issue.

Which version of youtube-dl-react-viewer is running in the container?
Are any other videos able to play with transcoding enabled?
Does the video play fine if downloaded to your local machine from the youtube-dl-react-viewer ui?

As a workaround, you may be able to avoid transcoding by changing the downloaded video container from MKV to MP4. This is doneby going to the job settings, and modifying the format code to bestvideo[ext=mp4]+bestaudio[ext=m4a]/mp4. Under the advanced options, you would also need to remove --merge-output-format mkv. Please note this might cause issues when downloading videos from a site which does not provide MP4 files, and that it only applies to new downloads

Version: 1.1.0
Only one in 240p one
Yes video plays without problem on local PC (server do not have GUI :) )

Are any of the videos 'flv' files? The utilities to transcode those videos aren't in the docker image by default.

If you still are running into transcoding issues since updating to version 1.1.1, would it be possible for you to add some extra logging to the transcoder? This involves modifying some of the code. This can be done with a one-liner in the shell, or manually if you prefer.
Both methods are provided below:
Option 1: Shell
Instructions:

Launch a shell into your docker container
Run the following script in the container shell:

sed -i '/    ffmpeg(videoPath)/a .on('\''stderr'\'', function(stderrLine) {console.log('\''FFMPEG stderr: '\'' + stderrLine);}).on('\''error'\'', function(err, stdout, stderr) {console.log('\''FFMPEG cannot process video: '\'' + err.message);}).on('\''end'\'', function(stdout, stderr) {console.log('\''FFMPEG transcoding completed.'\'');})' /opt/youtube-dl-react-viewer/youtube-dl-express-backend/index.js

Restart the container

OR
Option 2: Manually
Change the following lines in youtube-dl-express-backend/index.js:
https://github.com/graham-walker/youtube-dl-react-viewer/blob/3c51b6d86db53459fbd3b5411aebe68cf3696f9a/youtube-dl-express-backend/index.js#L59-L66
to this
// Transcode videos
app.use('/transcoded/videos', globalPasswordMiddleware, (req, res) => {
    res.contentType('webm');
    const videoPath = path.join(outputDirectory, 'videos', decodeURIComponent(req.path));
    ffmpeg(videoPath)
        .on('stderr', function(stderrLine) {console.log('Stderr output: ' + stderrLine);})
        .on('error', function(err, stdout, stderr) {console.log('Cannot process video: ' + err.message);})
        .on('end', function(stdout, stderr) {console.log('Transcoding succeeded !');})
        .format('webm')
        .pipe(res, { end: true });
});

Transcoding is no longer a supported feature, please use the open in VLC button or recoding option instead.


🐛 [firebase_app_check] After deleting App check, cloud firestore is not working.

After deleting App check, cloud firestore is not working.
I deleted all existing apps from the console and reinstalled them using CLI (web, ios, android).
App initialization was successful, but I couldn't create or read data.
There is no explanation on how to revert App check.
Note that cloud functions worked fine without any issues.
What should I do?
  match /requestsTest/{requestId} {
    allow read: if true;
    allow create: if isAuthUser();
    allow update, write, delete: if isMyData();
  }

CollectionReference<RequestData> requestsTestCollectionRef =
firebaseFirestore
    .collection('requestsTest')
    .withConverter<RequestData>(
    fromFirestore:
        (DocumentSnapshot<Map<String, dynamic>> snapshot, _) =>
        RequestData.fromJson(snapshot.data()!),
    toFirestore: (RequestData model, _) => model.toJson());

  Stream<QuerySnapshot<RequestData>> getRequestDataStream() {
   try {
   return requestsTestCollectionRef
    .orderBy('createdAt', descending: true)
    .limit(5)
    .snapshots();
  }
  } on FirebaseException catch (e){
      logger.warning(e.message, e);
      throw RepositoryException(
          message: e.message ?? 'An undefined error occurred');
    } catch (e) {
      logger.severe(unkownMessage, e);
      throw RepositoryException(message: e.toString());
    }

[✓] Flutter (Channel stable, 3.7.5, on macOS 13.0 22A380 darwin-arm64 (Rosetta), locale ko-KR)
• Flutter version 3.7.5 on channel stable at /Users/patrick386/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision c07f788888 (4 days ago), 2023-02-22 17:52:33 -0600
• Engine revision 0f359063c4
• Dart version 2.19.2
• DevTools version 2.20.1
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
• Android SDK at /Users/patrick386/Library/Android/sdk
• Platform android-32, build-tools 32.1.0-rc1
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 14C18
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
[✓] IntelliJ IDEA Ultimate Edition (version 2022.2.4)
• IntelliJ at /Applications/IntelliJ IDEA.app
• Flutter plugin version 72.1.3
• Dart plugin version 222.4533
[✓] VS Code (version 1.71.1)
• VS Code at /Users/patrick386/Downloads/Visual Studio Code.app/Contents
• Flutter extension version 3.58.0
[✓] VS Code (version 1.75.1)
• VS Code at /Users/patrick386/Visual Studio Code.app/Contents
• Flutter extension version 3.58.0
[✓] Connected device (2 available)
• macOS (desktop) • macos  • darwin-arm64   • macOS 13.0 22A380 darwin-arm64 (Rosetta)
• Chrome (web)    • chrome • web-javascript • Google Chrome 110.0.5481.177
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!

After deleting App check, cloud firestore is not working.

@Patrick386 Have you already made sure that all the setup / config is reset just like it was before enabling app_check ? You may confirm that from your firebase console.
What error are you getting while trying to read or write any data ?

After deleting App check, cloud firestore is not working.

@Patrick386 Have you already made sure that all the setup / config is reset just like it was before enabling app_check ? You may confirm that from your firebase console.
What error are you getting while trying to read or write any data ?

Even though I stopped the app_check service in the console, it seems like I can't go back to the state before activating app_check.
There are no error messages when I try to read or write data.

Thanks for the update. I am not completely sure of the expected behavior for this scenario, so will keep it open and label for team's thoughts.

I would reach out to Firebase support for this particular issue if you haven't already resolved it. It shouldn't block Firestore requests if you aren't using App Check anymore.


Additional Headers on Destination in CF not forwarded

Describe the bug
As per the cloud-sdk documentation we have maintained additional parameters in CF Destination to pass APIKey using parameter name as "URL.headers.APIKey" and in value we are passing api key value.
The external API expects "APIKey" to be passed in Header, when we are executing this from Postman we don't have any issue.
But when clicking on "Check connection" we get 401 unauthorized inside destination in CF. When we checked the network call we have observed no request header for API key is passed.
Link to Documetation
To Reproduce
Steps to reproduce the behavior:
Follow the documentation and add any external API from API Hub and test it by maintaining APIKey in destination under CF subaccount as per definition.
Expected behavior
The connection should be be success.
Impact / Priority
We rely on external API to access crucial information in our project. Also we are working on very critical project and would like to solve this immediately.
Affected development phase:  Development
Impact:  Blocked
Timeline: We have to solve this issue ASAP as this is time critical.

Hi @VikasNTTData ,
When you click on Check Connection,  it only verifies if a connection to the backend is possible from network perspective. It does not read the maintained credentials to actually login to the target system. Read more on the official SAP Help page.
Also, your issue pertains to BTP and not the SAP Cloud SDK, for further queries, please reach out to the correct channel.
However, if you have an application where you are using the SDK and face any issue, feel free to raise a ticket in this repository. I'll close this in the meanwhile.
Regards,
Deeksha