mpir: Add --enable-fat build option

Add --enable-fat to configuration options of mpir so that CPU-features of the build-machine are not relied upon in the library which may cause issues on older CPUs where these features are not present.
Motivation for this changeNixOS/nixpkgs/compare/master...polygon:mpir?expand=1
Programs using mpir can crash on older CPUs because newer CPU features from the build host are baked into the library due to CPU feature detection during build. This is also a major issue for reproducible builds since the outcome depends on the CPU architecture of the build machine.
In my case, using 'gqrx' failed with 'Illegal Instruction' because a SHRX instruction was not available. See the following Github issues and Reddit threads where this was discussed:
https://github.com/NixOS/nixpkgs/issues/58950
https://www.reddit.com/r/NixOS/comments/msqvb1/illegal_instruction_with_binaries_from_nix_store/
The --enable-fat option builds a combined x86_64 and x86_32 binary with runtime CPU feature detection which should alleviate these issues. On other CPU architectures, this will not change the build (according to documentation, untested)
Things done
I've built the binaries on a newer build-host and transferred them using nix copy to an old machine. Without this patch, the binaries crash on the old machine. With the patch, the binaries work on the old machine.

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 119791 at a2760598 run on aarch64-linux 1

10 packages marked as broken and skipped:

gnuradio3_7
gnuradio3_7Packages.ais
gnuradio3_7Packages.gnuradio
gnuradio3_7Packages.gsm
gnuradio3_7Packages.limesdr
gnuradio3_7Packages.nacl
gnuradio3_7Packages.osmosdr
gnuradio3_7Packages.rds
libsForQt512.alkimia
polymake

16 packages skipped due to time constraints:

arb
gfan
gnss-sdr
gnuradio
gnuradio3_7Minimal
gnuradio3_8
gnuradio3_8Minimal
gnuradio3_8Packages.gnuradio
gnuradio3_8Packages.limesdr
gnuradio3_8Packages.osmosdr
...

10 packages built successfully:

flint
kmymoney
libsForQt5.alkimia (libsForQt515.alkimia ,plasma5Packages.alkimia)
libsForQt514.alkimia
mpir
pynac
python38Packages.symengine
python39Packages.symengine
singular
symengine

1 suggestion:

error: no-flags-spaces
configureFlags cannot contain spaces, please use configureFlagsArray in Bash.
Near pkgs/development/libraries/mpir/default.nix:14:3:
   |
14 |   configureFlags = [ "--enable-cxx --enable-fat" ];
   |   ^

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 119791 run on x86_64-linux 1

9 packages marked as broken and skipped:

gnuradio3_7
gnuradio3_7Packages.ais
gnuradio3_7Packages.gnuradio
gnuradio3_7Packages.gsm
gnuradio3_7Packages.limesdr
gnuradio3_7Packages.nacl
gnuradio3_7Packages.osmosdr
gnuradio3_7Packages.rds
libsForQt512.alkimia

2 packages failed to build and already failed to build on hydra master:

sage: log was empty
sageWithDoc: log was empty

27 packages built:

arb
flint
gfan
gnss-sdr
gnuradio
gnuradio3_7Minimal
gnuradio3_8
gnuradio3_8Minimal
gnuradio3_8Packages.gnuradio
gnuradio3_8Packages.limesdr
gnuradio3_8Packages.osmosdr
gnuradio3_8Packages.rds
gnuradioMinimal
gnuradioPackages.gnuradio
gqrx
inspectrum
kmymoney
libsForQt5.alkimia (libsForQt515.alkimia ,plasma5Packages.alkimia)
libsForQt514.alkimia
mpir
polymake
pynac
python38Packages.symengine
python39Packages.symengine
qradiolink
singular
symengine

Might as well. Checked https links working and added.


Improve the jerry port api documents.

Notable changes:

Remove the comments in port impl, that's easily getting to in-consistence
Sync the jerryscript-port.h and 05.PORT-API.md
Fixes the invalid comment in port codes

JerryScript-DCO-1.0-Signed-off-by: Yonggang Luo<EMAIL_ADDRESS>

@LaszloLango Any suggestion about this?


Some countries inconsistent and missing

Data retrieved from endpoints inconsistent, this can lead to conflicts and inability to retrieve data across endpoints. If a country is retrieved from /countries/positions, that same country should be valid to retrieve data from /countries/cities
For instance in the endpoint
/countries/positions
South Korea is referred to as Korea, Republic of
/countries
South Korea becomes Korea South
Same inconsistency problem with Ivory Coast: in /countries/capital it is Ivory Coast but in /countries/positions it is Cote d'Ivoire.
Both Ivory Coast and Cote d'Ivoire returns country not found when querying /countries/cities

Hi Henry, thanks for pointing this out, yes this is something we're making fixes for since we're aggregating data from multiple data sources to make sure we have as much data as possible.
would you like to look into creating a fix for this, or, can you help identify other countries with the same issue? and I'll raise a fix for this

Hi Martins, It would be great if you can look into creating a fix and I can be a second pair of eyes. Unfortunately I cannot work on a fix directly as I am tied with other developments for my company. But I can certainly be your second pair of eyes.
Also, I found 2 Botwanas in countriesAndCities.js.

Hi @henryhuangh, be rest assured that I'm working on a fix for as much as I can find.

@henryhuangh have a look at the linked MR, let me know if it looks good.


Core Types

Oops, I merged core type without this PR merge. @chippmann can you merge this and update godot-jvm master ?

Oops, I merged core type without this PR merge. @chippmann can you merge this and update godot-jvm master ?

No worries. Sure I'll do that.


Sometimes the server just closes, randomly.

Anyone else having this problem?
https://gyazo.com/260aa6bf86d54098aa02345db9d89542
When this happens it just tries to connect to the server.
here is the console, no errors.
https://gyazo.com/e114aba20bfab5bc9e941c54108365ab

The client might be having a bit of a bug.

Try entering the ip in the address bar.
If it's your client, you need to fix the connection if the address bar isn't working.
I'm not having any bugs as I use Codeanywhereupmyarse and there aren't any errors with clients or servers.

Does your server sometimes work? @CalvinDKHD

the server works on old versions of multiogar

@CalvinDKHD Changue Protocol 11 to 9.

Can you run me through on how to do that?

https://github.com/Megabyte918/MultiOgar-Edited/blob/master/src/PacketHandler.js#L38
Changue the number 11 to 9.

@Jhair2015
Don't think that will help.

@Jhair2015 That won't do anything.
@CalvinDKHD Check the server log.

Maybe it has to do with this? https://github.com/Megabyte918/MultiOgar-Edited/issues/474

Possible, let's just wait for Calvin to respond.

lol sorry :p

no problem xD

Alright, ill let it run over night. If the same problem occurs in the morning, ill tell ya guys.

It won't do anything. It'll just block agar.io connections to the server.

clien problem

Delete minion check at https://github.com/Megabyte918/MultiOgar-Edited/blob/master/src/GameServer.js

wat

@Luka967 So i left it open overnight, and it didnt crash :)

The server obviously won't crash. You're having is a connection problem. Can you actually play on the server?

@CalvinDKHD Use older versión of MultiOgar!

yeah, i can play on the server. But after some time it just closes.

@CalvinDKHD Are you using this branch or the Updates branch? This should have been fixed on the updates branch.

Close? @Megabyte918


Add 'dispatches to transfomer objects' tests for find* functions

~Question: Do you prefer one PR per fn or should I batch them all into a single PR?~
Note: I'm not completely familiar with the transformer logic, so I have modeled the tests on other existing tests. Please take a look and let me know if I'm testing incorrectly.
Edit: Upon closer inspection, I realized that there are already transducer-related tests for these functions. With that in mind, I have edited the pull request title to reflect the new scope of this PR.
First mentioned here: https://github.com/ramda/ramda/issues/2346

I updated the can act as a transducer test after realizing the first go wasn't testing what it claimed to be testing :S

I think it's fine that this one was by itself, but the others could certainly be batched together.
I have to get my head around transducers again before I can comment on the changes themselves.

Let's just do them all here in a single commit if it is all the same to you @kurtmilam

Please see edited title and opening comment.

Looks good, can you resolve the conflicts @kurtmilam ?

@kedashoe I believe this is all clear now.

:cow2:


How to lazy-log extras / lazy-evaluate interpolations

With classic logging you'd:
logger.debug("some var: %s", the_var)

so that a perhaps costly conversion or interpolation isn't payed if the log level isn't debug or trace.
How would we do the same with structured logging?
logger.debug("some var: %s", the_var, extras={})

could work for the message but what if I need it inside another field?
logger.debug({"key1": "%s", "key2": "%s"}, var1, var2)

This looks nasty.
Or is there already something in place I've missed?

Hi @black-snow, I suspect that there is something you've missed, missing features, but also some few for lazy conversion.
Lazy message interpolation
As you've correctly identified (and as recommended by the docs and various linters) it is recommended to use lazy interpolation when creating log messages.
This works by storing the message and the variables to be interpolated within the LogRecord instance. When a Formatter needs the real log message it calls record.getMessage which does the string interpolation.
The reason this is lazy is that the Formatter is only called by a Handler if it is enabled for the given log level and logger name.
This deferred interpolation only applies to the message and does not affect any other data that is generated and attached to the LogRecord as this needs to be done before the record is passed to the various loggers or handlers.
Generation of JSON
In short, all generation of JSON is lazy as it is generated in the same place as log record messages - in a Formatter(which is only called by the Handler if it enabled for the logger and log level).
For str messages, the same lazy interpolation still works as normal as the JSON formatters will use record.getMessage to generate the message field.
dict messages
Things change when using a dict for a message. In this case generation of the JSON will still be lazy based on the log level, but the dictionary itself will be created before the actual log call.
This isn't particularly different to the lazy interpolation of the message in that all values to be interpolated must be known before making the logging call.
Consider:
logger.debug("some message %s", expensive_to_compute())

This also applies to the extra={...} argument which is stored on the log record at time of creation.
In these situations your best bet is to a logger + isEnabledFor:
logger = getLogger()  # get root logger since examples are using root logger, but can be done on specific loggers

if logger.isEnabledFor("DEBUG"):
    logger.debug("some message %s", expensive_to_compute())
    logger.debug({"some_field": also_expensive()})
    logger.debug("another message", extra={"another_field": somehow_also_expensive()})

dict messages and string interpolation
If your specific use case is something like:
logging.debug({
    "a_field": "this is a message %s" % expensive_to_compute(),
    "b_field": "this is another message %s" % also_expensive()
})

Then there currently is no way to do lazy interpolation. It could be implemented but would require parsing the values of fields in the message dict (or extra argument), which I think is a pretty rare case.
In these situations I would recommend using isEnabledfor to protect known expensive to compute log messages.

Thanks for the quick & extensive answer.  Python will assemble the string and pass it to debug. logger.debug("whatevs %s", whatevs) is.
logger.debug("whatevs %s", expensive_to_compute()) ain't lazy. logger.debug("whatevs %s", expensive_to_compute) would be if debug called callables (haven't checked - this is how you do it w/ decent libs in Java, for example).
To make this work for values in the dict we could accept str | tuple[str | callable] where the first item is the message / template.
logger.debug({"key1": ("string with a %s", var), "key2": ("%s %s %s", perhaps, many, vars)})

Which gets funny with deeper nesting but it could, in theory, be applied recusively. If you ever need it.

logger.debug("whatevs %s", expensive_to_compute()) ain't lazy. logger.debug("whatevs %s", expensive_to_compute) would be if debug called callables (haven't checked - this is how you do it w/ decent libs in Java, for example).

Right, it sounds like what you're describing as lazy here is not the same as what the python logging documentation refers to when discussing lazy / deferring the interpolation.
It's not wrong, that's just not what I understood from your original question.
So if I understand correctly you want to defer string interpolation and the generation of values used in that interpolation.
If that's the case I am loathe to add anything that makes implicit assumptions about the data being passed. For example how do I know you want to lazy interpolate and aren't just passing tuple[str, ...]? See also: policy on breaking changes.
That said I think there might be a solution creating a class that we can then add to the various JSON encoder default methods. Like using a dictionary for the message this would only work with our JSON formatters.
class LazyStr:
    __init__(self, template, *args) -> None:
        self.template = template
        self.args = args

    __str__(self) -> str:
        values = [item() for item in self.args if callable(item) else item]
        return self.template % tuple(values)

    __call__(self) -> str:
        # Could be useful so that LazyStr can contain other LazyStr in it's args
        return self.__str__()

def json_default(o: Any) -> Any:
    # Do we even need this if we're making use of __str__? the json_default functions will end up calling str(o) anyway...
    # https://github.com/nhairs/python-json-logger/blob/main/src/pythonjsonlogger/defaults.py#L37
    if isinstance(o, LazyStr):
        return str(o)
    # ...

logger.debug({
    "key1": LazyStr("string with a %s", var),
    "key2": LazyStr("%s %s %s", perhaps, many, vars),
})

Per my embedded comments: given that the JSON default functions will end up str(lazy_str_obj) one could argue that this could/should be implemented as an external package as not specific to JSON logging and others may benefit from it.
^ I am open to other opinions on this
For reference the LazyStr scaffold above would work with the existing LogRecord objects (and thus formatters etc) as it also calls str()
# https://github.com/python/cpython/blob/main/Lib/logging/__init__.py#L391
    def getMessage(self):
        """
        Return the message for this LogRecord.

        Return the message for this LogRecord after merging any user-supplied
        arguments with the message.
        """
        msg = str(self.msg)
        if self.args:
            msg = msg % self.args
        return msg

Thanks again for your time and effort.
What would you expect to happen if I passed tuple[str, ...] as a value? There's no JSON representation for tuples, right? (Well, arguably lists - if one argues that you cannot "change" JSON anyway.)
I don't know what I'd expect to happen here: logger.debug({"key": ("a", "b", "c")}) ... a list and a warning? A list if I configured it to turn tuples into lists and an error otherwise? ...
But then again, lists are rather supposed to be homogeneous as opposed to tuples - so perhaps ["a", "b", "c"] is the obvious answer.
LazyStr doesn't look too bad, though. It's seven characters extra but you will not have it everywhere plus if you knew about the special tuple evaluation knowing LazyStr was no different.
The more I read it the more I like it.

There's no JSON representation for tuples, right?

All Python JSON libraries I've used will produce a list by default. I don't think there is a reason for them to produce anything different.

I don't know what I'd expect to happen here: logger.debug({"key": ("a", "b", "c")}) ... a list and a warning? A list if I configured it to turn tuples into lists and an error otherwise?

You should expect {"key": ["a", "b", "c"]}.

It's seven characters extra but you will not have it everywhere plus if you knew about the special tuple evaluation knowing LazyStr was no different.

It's either that or use if logger.isEnabledFor(...):

With other packages and existing solutions, it's starting to feel a lot like something to put into the documentation rather than add to the library

With an extra if statement in "my" code it feels like a detail of the logging is polluting it.
I could also just warp the logger or patch something onto it.
What would you put into the docs then?

With an extra if statement in "my" code it feels like a detail of the logging is polluting it.

:shrug: I don't know what to tell you, using an if statement or using lazy strings are both going to pollute your code when avoiding expensive log statements.
Given there are straight forward workarounds I don't intend to add support for implicitly doing lazy interpolation.

I could also just warp the logger or patch something onto it.

If you do want to add your own magic then you should look into modifying the logged data.

What would you put into the docs then?

I'll probably add something to the cookbook like so:
import logging
import time

from pythonjsonlogger.json import JsonFormatter

def expensive_to_compute():
    time.sleep(5)
    return "world"

## Setup
## -------------------------------------
logger = logging.getLogger()
handler = logging.StreamHandler()
formatter = JsonFormatter()
handler.setFormatter(formatter)
logger.addHandler(handler)
logger.setLevel(logging.INFO)

## Log Using isEnabledFor
## -------------------------------------
start = time.time()
if logger.isEnabledFor(logging.INFO):
    logger.info(
        {
            "data": "hello {}".format(expensive_to_compute())
        }
    )
print(f"Logging INFO using isEnabledFor took: {int(time.time() - start)}s")

start = time.time()
if logger.isEnabledFor(logging.DEBUG):
    logger.debug(
        {
            "data": "hello {}".format(expensive_to_compute())
        }
    )
print(f"Logging DEBUG using isEnabledFor took: {int(time.time() - start)}s")

## Log Using LazyString
## -------------------------------------
# pip install lazy-string
from lazy_string import LazyString as L

start = time.time()
logger.info(
    {
        "data": L("hello {}".format, L(expensive_to_compute))
    }
)
print(f"Logging INFO using LazyString took: {int(time.time() - start)}s")

start = time.time()
logger.debug(
    {
        "data": L("hello {}".format, L(expensive_to_compute))
    }
)
print(f"Logging DEBUG using LazyString took: {int(time.time() - start)}s")


Suggestion: add terminator

I think "terminator" is an essential tool for ROS developers🤖
Of course the user can install it later if necessary, but for hands-on use of this Docker environment, it is very convenient to have it installed from the beginning.
What do you think about adding this?

Good idea!

Added in https://github.com/Tiryoh/docker-ros2-desktop-vnc/pull/73.
Thanks @takasehideki :+1:

Thanks for adding!
I should have sent a PR, but I apologize for not getting around to it ;(

No problem at all! 😄


CSS/React Hot Reloading

I think one of the best parts of something like this is that typically complex things to setup like hot reloading just work out of the box. It brings those advanced workflows to everyone.
Hot reloading is such a crucial part of my dev experience, and this project could expose it to others. The only problem is that React hot reloading doesn't work all the time, so I don't know if that is too confusing to beginners.
I haven't looked at how this project actually works yet, but I can't think of any reason why this wouldn't work. Just automatically include webpack-hot-middleware in dev mode, add the right entry point, add the right babel transform, etc. It can be completely hidden from the user.

CSS hot reloading is there.
React will be there is it’s stable enough but it requires work both on webpack and my side.

Oh, it's great that webpack's hot reloading infrastructure is already set up.
Yeah, this is purely anecdotal but when I tried switching from v2 to v3 of react-hot-loader it seemed a bit slower to compile my files. I definitely would make sure that this doesn't add significant overhead to the standard workflow of incremental compilation + refresh.

The nice thing in this project is we get to control the complete experience. We can replace any part if there is a better solution. e.g. I’m hopeful about https://github.com/motion/pundle.
cc @steelbrain

Agreed. I think the compilation from babel+react-hot-loader was the slow part though, not webpack. Anyway, I love that this could turn into the de-facto place to benchmark this kind of stuff and establish a standard for the best way for things to work together.

I love that you love it.

@gaearon There's a 2x speed boost in https://github.com/motion/pundle/pull/43. It also adds support for requiring non-js files (css for example) and custom hot reloading wrappers. It'll be in the next major pundle version, which you can expect in about a week or two

Thanks! @steelbrain Would you be interested in sending a proof of concept PR switching this project to pundle? Doesn’t have to be fully feature complete, just to get a taste for it.

Sure @gaearon! I'll send in a PR soon

Closing for now. CSS hot reloading is there, React hot reloading may come one day. 😉

What? React hot-reloading is like the main reason I would want (and need) this tool :astonished:

@gaearon what's the real issue with react hot reloading? I think it's nice to integrate that in CRA. Also, when it's integrated, more people can see and help react-hot-loader smooth out its issue.

It seems it's actually very simple to add HMR for React to your app. See<EMAIL_ADDRESS>It's literally an extra 10 lines of code.
Before:
ReactDOM.render(
    (
        <Provider store={store}>
            <App />
        </Provider>
    ),
    document.getElementById('root')
);

After:
function render(Component) {
    ReactDOM.render(
        (
            <Provider store={store}>
                <Component />
            </Provider>
        ),
        document.getElementById('root')
    );
}

render(App);

if (module.hot) {
    module.hot.accept('./App', () => {
        const NextApp = require('./App').default;
        render(NextApp);
    });
}

@Scimonster When I use that solution it still refreshes the whole page, so my redux state still gets lost. Did I forget something?

@anomaly44 I'm having issues once I started using react-router. Similar situation?

@grantgeorge Yes I m using React-router v4, gonna try taking it out tonight if I can find some time and see if it works without.

@anomaly44 interesting. I had issues w/ react router but worked when I upgraded to v4 last night and did some reconfiguration for the upgrade.
In case this helps, here's my setup:
package.json
"react-router-dom": "^4.0.0-beta.6",
index.js
import React from 'react';
import ReactDOM from 'react-dom';

import Routes from './routes';

import './index.css';

const rootEl = document.getElementById('root');

ReactDOM.render(
  <Routes />,
  rootEl
);

if (module.hot) {
  module.hot.accept('./routes', () => {
    const NextApp = require('./routes').default;
    ReactDOM.render(
      <NextApp />,
      rootEl
    );
  });
}

routes.js
const Routes = () => (
  <Router>
    <div>
      <Route exact path="/" component={Home} />
      <Route path="/artists" component={Artists} />
      <Route path="/brands" component={Brands} />
      {/* <Route path="/*" component={NotFound} /> */}
    </div>
  </Router>
)

export default Routes

I did the same, the only difference being that my  in the non hot render is wrapped with a redux provider. It works, I dont get any errors, but the whole page refreshes, so my redux state still gets reset to the initial state.  Strange

https://gist.github.com/anomaly44/83c7d7aa0b92997481f50e25800e6f88 Can you spot anything wrong with this? my  component is in my App file. Everything works, except that i lose my redux state when making changes.

When you make changes in which files? And what is your configureStore like?

When I edit some of the default text in App.js
added my store.js to the gist: https://gist.github.com/anomaly44/83c7d7aa0b92997481f50e25800e6f88

If you look at the code, you are rendering two different things. Why?
// 1
ReactDOM.render(
        <Provider store={store}>
            <Component/>
        </Provider>,

// 2
        ReactDOM.render(
            <NextApp/>,
            document.getElementById('root')
        );

In one case you are rendering App inside a Provider, in other case you don't use Provider. This is why the state gets lost.
You also declared a render() function but you only use it once. Why did you declare it? The purpose of declaring it is so you could reuse it in both places:

const render = (Component) => {
    return ReactDOM.render(
        <Provider store={store}>
            <Component/>
        </Provider>,
        document.getElementById('root')
    );
};

render(App);

if (module.hot) {
    module.hot.accept('./App', () => {
        const NextApp = require('./App').default;
        render(NextApp);
    });
}

I would recommend to get more comfortable with HMR API before using it. You seem to be copy and pasting examples, so you are missing the intention of these lines (and thus missing why the code did not work).
I hope this helps!

sorry for the confusion, the inconsistencies you pointed out were caused by me  switching things around and quickly trying some stuff I found googling because I couldnt get it working. The code you posted above is exactly what I tried the first time to get it working. I should have pasted that code instead of the mangled stuff I ended up posting.
Anyway, I added a console.log in the arrow function in the module.hot.accept. When I save a change in App I see the message in my console for half a second before the page refreshes and the state gets lost. Any idea what might be going wrong?

Sorry, no idea! If you post a project I could take a look.

While cleaning out my CRA project with the intent to post the project for you. I found out what was causing the problem.
it was this code I used to implement material-ui:
// App.js
import injectTapEventPlugin from 'react-tap-event-plugin';
injectTapEventPlugin();

Moving this code to a separate file as described in this issue solved the problem.
Thanks for the effort guys, much appreciated.

Interesting, thanks!

Sorry, I'm confused. It's HMR available ? Should I do something extra to activate it?
Many thanks

HMR API is available so you can use module.hot the same way you would normally do with Webpack.
Something like React Hot Loader is not included so you won’t be able to keep the component local state between hot reloads.

What's the difference ? May be that React Hot Loader is able to keep the state and all the components and the regular HMR api does not? Because before using @Scimonster suggestion the entire page was being reload (like pressing F5 or command +R ) but now using the module.hot stuff at least the browser is not refreshed, which is an improvement.

May be that React Hot Loader is able to keep the state and all the components and the regular HMR api does not?

This is correct.


make overtuning_data example reproducible

commit d7160b1 added this data used in the final chunk of slides-nested-resampling, but not the script used to create it -- do you still have it, @juliambr?  (if so plz push directly to the rsrc folder in the refactoring-w1920 branch, preferably. if not, for shame.... :hankey:)

talking about  overtuning_data.rds

I found the R script, pushed it into the rsrc folder in the refactoring-w1920 branch.

closed in b9c75a5a


Allow Disabling Pickup Reach Bonus

I really like have extra reach when placing down buildings, but when picking up items off the ground, I would rather not have that because then I accidentally pick up a bunch of stuff I didn't want to.
Could it please be made so that the different types of reach bonuses can be turned off and on independently from one another?

I've added this in 1.0.16. Feel free to test it out.

Thank you!


Any specific Requirements for creating a new mdx file?

Hi guys! Sorry to disturb you during the holiday season and Today I just found out about this template and wanted to spin up a new blog using this template.
everything is fine and the blog is running fine but every time I try to create a new post on post/new-example.mdx it shows up on the local and production build but it's not responsive for some reason.
Note: All other original posts is still responsive only the new ones are being responsive.Even when I copy a previous post!
Is there any specific way of working this or there is some error regarding my mdx?

I too have the same issue. At the least, the cause of my issues is the length of code blocks.
This works fine and the page retains a responsive design on mobile.
git verify-pack -v .git/objects/pack/

Anything more than that doesn't work and the page loses the responsive design on mobile.
git verify-pack -v .git/objects/pack/<pack-name>.idx | sort -k 3 -n | tail -n 2

I wonder why the code block doesn't retain the size on mobile which sort of breaks the page layout.

@mehediasif I made one change in the layout which fixed the issues I had with the code blocks.
I removed the CSS class items-center in the components/Layout.js file.

to

P.S.: I have dabbled enough in HTML, CSS, and js to make changes and fix minor issues. I still don't have the overall picture of the tailwind CSS & the layouts used in this theme. So the above solution may not be the exact fix that's required.

@subbramanil thank you for your suggestion
I have taken your idea and changed the code according to the tailwind docs for media query
If you read about tailwind docs you would see that you can set multiple media queries on the same element


fix: use fixed playwright version

Broken by the PW upgrade - https://github.com/elastic/synthetics/pull/485, we have always used to fix the version of PW and reduce the breakage footprint that comes with new versions which might have new system dependencies.
Without this zip urls monitors that uses the latest version of the Synthetics package will install PW versions that are the latest version 1.23.0 that also requires other system dependencies and fails to invoke the Synthetics.

Produces validation errors like this.
 Host system is missing dependencies to run browsers

║ Please install them with the following command: ║
║ sudo npx playwright install-deps ║
║ Alternatively, use apt: ║
║ sudo apt-get install libwayland-client0 ║

I'm not sure that goes back far enough. I tested that version of playwright by running the current todos demo with a modified package.json shown below in a heartbeat:7.3.0-SNAPSHOT container
{
  "name": "todos",
  "private": true,
  "description": "This suite tests the examples that ship with the open source Vue.js project.",
  "scripts": {},
  "license": "MIT",
  "dependencies": {
    "@elastic/synthetics": "*",
    "playwright-core": "=1.20.1"
  },
  "devDependencies": {
    "node-static": "^0.7.11"
  }
}

I still got the error:

heartbeat@2bc283fe3c0f:/tmp/synthetics-demo/todos/synthetics-tests$ npx @elastic/synthetics .
browserType.launch:
╔══════════════════════════════════════════════════════╗
║ Host system is missing dependencies to run browsers. ║
║ Please install them with the following command:      ║
║                                                      ║
║     sudo npx playwright install-deps                 ║
║                                                      ║
║ Alternatively, use apt:                              ║
║     sudo apt-get install libwayland-client0          ║
║                                                      ║
║ <3 Playwright Team                                   ║
╚══════════════════════════════════════════════════════╝


RSA CipherSpi should check for short buffers

Cipher.doFinal() and friends state that Ciphers should throw ShortBufferException when given an output buffer that's too small to hold the output.  Other BC ciphers do this (eg, https://github.com/bcgit/bc-java/blob/master/prov/src/main/java/org/bouncycastle/jcajce/provider/symmetric/util/BaseBlockCipher.java#L1109), but RSA ciphers instead just try to fill in the output buffer and throw ArrayIndexOutOfBoundsException when the buffer runs out.  This also happens after bOut has been modified, so if the caller retries the call with a properly-sized buffer they'll get incorrect output.

Pull request #323 added. Issue now fixed.


Resets to initial state when there is Link with prefetch

Reproduction code: https://github.com/Arctomachine/usequerystate-bug
Only works in production (start) mode, not in dev

pnpm install
pnpm run build
pnpm run start
Got to index page /
Click checkbox - page should flicker and reset to default value
Click it again - should work as expected
Move cursor over any link in grid - it should reset again

To "fix" it go to ./app/Wrapper.tsx, at line 33 change prefetch={true} to prefetch={false}

Thanks for the reproduction. It looks like prefetch calls history.replaceState, which throws things off track, thinking it's an actual navigation. And since prefetch is a production-only feature, it explains why it works fine in development.
I'll see if that's a recent behaviour in the newest canaries, otherwise we might have to use a different strategy to differentiate external navigation from prefetches.

I have yet to trace the full path taken from dispatching the ACTION_PREFETCH action in Next.js to the call to the history API that causes the problem, but it looks like Next.js is keeping an internal record of the URL it's "supposed" to be at, rather than using the actual location object.
A proof of this is using { shallow: false } for the query filter, which uses the Next.js router rather than a call to the history API. This solves both the checkbox blinking and the prefetch resets, because the router was told about the intent to have a querystring in the URL.
I'm not sure if the Next.js team would consider that a bug, but there may be other libraries interfacing with the history API that have a similar issue (eg: alternative routers).

I can confirm this behaviour started to appear in 14.0.2-canary.7, specifically due to this PR: https://github.com/vercel/next.js/pull/56497.

Note: 14.0.2 has been released with the PR that broke shallow URL updates when combined with prefetch links. next-usequerystate is unfortunately not compatible with this release (and any subsequent until I have identified the root cause and they patched it on their end, PR coming).

@franky47 Could you tell me if you plan to maintain this repo for a long period of time? I plan to use it, but I don't want libraries that will "die" in a few months. Would really appreciate if you could give me some feedback about your plans with this repo :)
Thanks! :)

I'm actively using it across many production projects, it's not going anywhere. Ideally, I'd like to get some of the complexity pruned out by merging it into code Next.js (shallow routing in app router).
That being said, it's also dependent on external factors, like a stable revenue stream, which is not the case at the moment. Sponsors are welcome to help with maintenance.

Found the root cause of the issue. Let's follow the N whys:
The main effect is, as described by @Arctomachine: Query updates get reset when hovering (or mounting) a prefetch Link.

Why: history.replaceState is called with the internal canonicalUrl in the app router's HistoryUpdater component, which is not aware of the shallow URL update. (source)
Why: the HistoryUpdater re-ran its useInsertionEffect.
Why: the sync parameter function changed
Why: it was rebuilt because the reducer state had changed (source)

The impacting PR did not change how the prefetch reducer behaves. It did not change the app router state when running in canary.6 and below, and actually still doesn't change the internal state in subsequent releases.
What changed however, is that the reducer state is now based on Promises. For each action, a new Promise is created. Those are referencially different, even if the state they resolve to is the same. The sync function is being rebuilt at every action dispatched, because it's essentially a different reference.
Solution: drop this sync function entirely, as its purpose was to synchronize the Redux devtools store somehow, but I doubt it still serves its purpose with Promises.
Going to run this by the Next.js team, I'll update this comment with the PR number when it's out.

@Arctomachine Could you please try the following combination, and let me know if you are still experiencing this issue?

<EMAIL_ADDRESS><EMAIL_ADDRESS>Setting the experimental flag windowHistorySupport to true

It works now, tested on project where it first appeared - no problems so far.

Thanks a lot for your feedback, I have released 1.12.0. Fingers crossed that PR on the Next.js repo lands on 12.0.3 🤞

<EMAIL_ADDRESS>has been released with the prefetch fix. The experimental flag is not needed anymore from there on (although its use is supported).


Document 'how to cite'

https://oud.ictresearchmethods.nl/How_to_cite
Either in the README or on the website (/about)

Apparently it is already on this link https://ictresearchmethods.nl/about/ 🙈

Yes, but apparantly it needs a more prominent place on the website, because students keep referring to Jochem Vogel as only author. As much as I appreciate your work, this credit does not really fit reality ;-)


Codewars problems added along with GeeksforGeeks problems

Re-issued pull request after resolving conflicts!

Nice work :tada: :+1:


Incompatible with new parallel_tests gem due to version restriction

$ bundle install
Fetching gem metadata from https://rubygems.org/.........
Fetching version metadata from https://rubygems.org/..
Resolving dependencies...
Bundler could not find compatible versions for gem "parallel_tests":
  In snapshot (Gemfile.lock):
    parallel_tests (= 1.5.1)

  In Gemfile:
    sauce (~> 3.5.6) ruby depends on
      parallel_tests (<= 1.3.7, >= 1.1.1) ruby

    parallel_tests (~> 1.5.1) ruby

Running `bundle update` will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

$ cat Gemfile
source 'https://rubygems.org'

gem 'parallel_split_test', '~> 0.4.0'
gem 'parallel_tests', '~> 1.5.1'
gem 'rspec', '~> 3.3.0'
gem 'sauce', '~> 3.5.6'

It'd be nice to use the new parallel_tests gem.

+1

+1


Build fails with apt-get not found

Docker compose file contains:
    build:
      context: https://github.com/mdhiggins/radarr-sma.git#build
      args:
        - radarr_tag=nightly
        - ffmpeg_tag=snapshot-vaapi

Get the following error on docker-compose build
Step 11/17 : RUN   apt-get update &&   apt-get install -y     git     wget     python3     python3-pip     ${extra_packages} &&   mkdir ${SMA_PATH} &&   git clone https://github.com/mdhiggins/sickbeard_mp4_automator.git ${SMA_PATH} &&   python3 -m pip install --user --upgrade pip &&   python3 -m pip install --user virtualenv &&   python3 -m virtualenv ${SMA_PATH}/venv &&   ${SMA_PATH}/venv/bin/pip install -r ${SMA_PATH}/setup/requirements.txt &&   chgrp users /usr/local/bin/ffmpeg &&   chgrp users /usr/local/bin/ffprobe &&   chmod g+x /usr/local/bin/ffmpeg &&   chmod g+x /usr/local/bin/ffprobe &&   apt-get purge --auto-remove -y &&   apt-get clean &&   rm -rf     /tmp/*     /var/lib/apt/lists/*     /var/tmp/*
 ---> Running in 42f3bb45e964
/bin/sh: apt-get: not found

LinuxServer.io rebased the nightly tag to Alpine: https://info.linuxserver.io/issues/2022-02-03-radarr/
Can this be accounted for in your dockerfile?

I will probably need to make a separate build container for alpine, I did already update the nightly tag for alpine but the dockerfile can't handle the conditionals
I'll see if I can come up with something

Thank you. Looks like they are rebasing a few containers to Alpine: https://info.linuxserver.io/affected/notifications/
Not sure if this will eventually apply to other tags as well, or if it's just being done for space efficiency with the nightly builds.

Just pushed an update, I already had accounted for most of this ins the 90-sma-config file when I made the sma-mod so since the build dockerfile can be alpine or ubuntu I just removed most of the packages from that and will let 90-sma-config do the work
Only thing I wasn't sure of is if alpine has an equivalent package to libva-drm2 as I couldn't find one (libva2 -> libva and i965-va-driver -> libva-intel-driver)
Let me know if that works

Thanks again.  Build works now.  Will have to wait for the next grab to see how the tag/conversion goes.

Double check your docker logs when starting the container for the first time for any errors (from 90-sma-config) since I moved most of the container setup to occur on the first launch and they might be hiding in there and not immediately apparent

Here's what I found...
ERROR: unable to select packages:
[90-sma-config] Installing additional hardware acceleration packages
chown: changing ownership of '/usr/local/sma/config/autoProcess.ini.bak': Operation not permitted
<<--above error repeated for every file in the directory-->>
Traceback (most recent call last):
  File "/usr/local/sma/update.py", line 8, in <module>
    from resources.readsettings import ReadSettings
ModuleNotFoundError: No module named 'resources'

I think I've always had the chown errors but it's worked fine.  Not sure about the other errors

Looks like it isn't doing the initial git pull, script won't work
Made a tweak, try the latest update (the permissions error looks like you have a .bak file that it cannot access, you could probably fix the ownership on that .bak or move it out of your mounted directory to make that go away)
433e2b5aaf3bcb465c11c8577689d6a0baecdf7d

Now getting this:
ERROR: unable to select packages:
[90-sma-config] Installing additional hardware acceleration packages
[90-sma-config] Pulling SMA from github master
./90-sma-config: line 55: git: command not found
<<-- my perms errors -->>
Traceback (most recent call last):
  File "/usr/local/sma/update.py", line 8, in <module>
    from resources.readsettings import ReadSettings
ModuleNotFoundError: No module named 'resources'

Alright I spun up a test container and think I have everything sorted now, try one more time

Looking pretty good now...
[cont-init.d] 10-adduser: exited 0.
[cont-init.d] 30-config: executing...
[cont-init.d] 30-config: exited 0.
[cont-init.d] 50-gid-video: executing...
groupadd: 'videozb2Z7vzD' is not a valid group name
groupmod: group 'videozb2Z7vzD' does not exist
usermod: group 'videozb2Z7vzD' does not exist
groupadd: 'videoiYiboXEP' is not a valid group name
groupmod: group 'videoiYiboXEP' does not exist
usermod: group 'videoiYiboXEP' does not exist
[cont-init.d] 50-gid-video: exited 0.
[cont-init.d] 90-custom-folders: executing...
[cont-init.d] 90-custom-folders: exited 0.
[cont-init.d] 90-sma-config: executing...
[90-sma-config] Init SMA from github master
[90-sma-config] Creating python virtual environment and installing requirements
[cont-init.d] 90-sma-config: exited 0.
[cont-init.d] 99-custom-files: executing...
[custom-init] no custom files found exiting...
[cont-init.d] 99-custom-files: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[Info] Bootstrap: Starting Radarr - /app/radarr/bin/Radarr - Version <IP_ADDRESS>23

Sweet, alright the build container should be universal now alpine or ubuntu, thanks for the tests
Closing this out reopen if you have any issues

The conversion is failing.  Here is the log file...
2022-02-16 14:24:24 - RadarrPostProcess - INFO - Radarr extra script post processing started.
2022-02-16 14:24:24 - resources.readsettings - INFO - /usr/local/sma/venv/bin/python3
2022-02-16 14:24:24 - resources.readsettings - INFO - Loading config file /usr/local/sma/config/
autoProcess.ini.
2022-02-16 14:24:24 - resources.mediaprocessor - ERROR - isValidSource unexpectedly threw an exc
eption, returning None.
Traceback (most recent call last):
  File "/usr/local/sma/resources/mediaprocessor.py", line 318, in isValidSource
    info = self.converter.probe(inputfile)
  File "/usr/local/sma/converter/__init__.py", line 336, in probe
    return self.ffmpeg.probe(fname, posters_as_video)
  File "/usr/local/sma/converter/ffmpeg.py", line 587, in probe
    stdout_data = self._get_stdout([
  File "/usr/local/sma/converter/ffmpeg.py", line 537, in _get_stdout
    p = self._spawn(cmds)
  File "/usr/local/sma/converter/ffmpeg.py", line 530, in _spawn
    return Popen(cmds, shell=False, stdin=PIPE, stdout=PIPE, stderr=PIPE,
  File "/usr/lib/python3.9/subprocess.py", line 951, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/usr/lib/python3.9/subprocess.py", line 1821, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: '/usr/local/bin/ffprobe'
2022-02-16 14:24:24 - resources.mediaprocessor - INFO - File /videos/Movies/SomeFile (2022)/SomeFile
 (2022).mkv is not valid
2022-02-16 14:24:24 - RadarrPostProcess - INFO - Processing returned False.
2022-02-16 14:24:24 - RadarrPostProcess - ERROR - Error processing file.
Traceback (most recent call last):
  File "/usr/local/sma/postRadarr.py", line 301, in <module>
    sys.exit(1)
SystemExit: 1

Yeah looks like the FFMPEG build from the ubuntu containers does not work on alpine and the jrottenberg container only goes up to alpine 313 (linuxserver radarr container on 315)
The ffmpeg APK included in alpine however appears to contain VAAPI so I added some new options and this appears to be working for me. You'll need to include your hw devices and volumes etc but that's the environment variables needed
  radarr:
    image: ghcr.io/mdhiggins/radarr-sma:nightly
    container_name: radarr
    environment:
      - PUID=${PUID}
      - PGIG=${PGID}
      - TZ=${TZ}
      - SMA_USE_REPO=true
      - SMA_HWACCEL=true

Thank you.  Rebuilt container and will test with a file later.

So the conversion and tagging worked, but the rename failed:
2022-02-19 07:54:22 - RadarrPostProcess - ERROR - Failed to trigger Radarr renam
e.
Traceback (most recent call last):
  File "/usr/local/sma/venv/lib/python3.9/site-packages/requests/models.py", lin
e 910, in json
    return complexjson.loads(self.text, **kwargs)
  File "/usr/lib/python3.9/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.9/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.9/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/sma/postRadarr.py", line 288, in <module>
    rename = renameRequest(baseURL, headers, movieinfo['movieFile']['id'], movie
id, log)
  File "/usr/local/sma/postRadarr.py", line 56, in renameRequest
    rstate = r.json()
  File "/usr/local/sma/venv/lib/python3.9/site-packages/requests/models.py", lin
e 917, in json
    raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: [Errno Expecting value] : 0

However, the file is named correctly

Hm it almost looks like the API just failed to return anything. Is it doing this for every file or was this a one off error?
If it's doing it for every file could you turn on debug logging and send a full log from postRadarr

Looks like every file.
Here's a debug log: https://pastebin.com/bx3xhZwk

Fixed in latest version of SMA


[BUG] Unable to send answers past a certain length

Describe the bug
If you attempt to send in an answer that overflows the text box the answer button does nothing
To Reproduce
Steps to reproduce the behavior:

Create a quiz
Attempt to answer a question but fill the textbox so it begins to overflow

Expected behavior
Either the answer should be put through or there should be some type of indication that the length of you answer exceeds the maximum
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version: 107.0.5304.107

@nateachino I would recommend adding screenshots also! That will help others to understand the problem better and easily.

@nateachino Actually it is done on purpose, here is the js code that is doing that -https://github.com/prakhartiwari0/Arito/blob/main/js/main.js#L147
The logic is, the biggest answer which possibly can come in the whole test is the product of 99,999 x 99,999 because they are the biggest number that can be generated by the code in case the user selects very hard in the difficult level. And we all know that the product of any two numbers will always be greater than their difference, sum, quotient, or remainder.
And therefore, if the user tries to put numbers unnecessarily too big, it doesn't let them submit it.
I hope I was able to explain, please ask if you have any more queries!


channels: 21.11 is stable, 21.05 is deprecated

Needs redeploy? :-) I still can't see 21.05 as EOL on https://status.nixos.org

Wait, "deprecated" now when the previous use "unmaintained"?

Oh, I suspect we just need to transition them from "deprecated" to "unmaintained" now.

Yep, indeed -- December 2021 is over, so we can mark 21.05 as unmaintained now :)
https://github.com/NixOS/nixpkgs/blob/67cc0b103ab45ca2babb97d96e26d50dd0061644/nixos/doc/manual/release-notes/rl-2105.section.md?plain=1#L3

https://github.com/NixOS/nixos-org-configurations/pull/201


Something missed in 0.9.4

single-node-asg
output asg_iam_role_name is used outside the repo. Now it is removed.

persistent-ebs
output iam_profile_policy_document is referencing non-exist resource aws_iam_role_policy.attach_ebs.

Please address these in separate commits (or PR), TY.


Suppress relationship name in a view

Feature request to add a "None" item to the "Text Position" selector on the Appearance tab of a Relationship. The idea is to be able to name a relationship, but not necessarily see the name on any given View.
http://forum.archimatetool.com/index.php?topic=392

Hi,
Personnally I also think it makes sens to have a way to hide relationship name. I'm not sure "Text Position" is the best place to do it though (is "nowhere" a position?)...

It would be easy enough to have a "visible" checkbox in Properties to show or hide it.

This is implemented in 4.6


Corrected quickstart launch command.

As it seems the quickstart command to launch traefik was missing the "=".

Hi @shochdoerfer,
We forgot to update the README.md since we migrated on our own flag package.
Unfortunately, the way to use flags changed a little bit ^^
According to the POSIX standardization there is neither space nor = between a short flag and its value (but it still needs on = for a classic flag).
I fixed the README.md issue on this PR : https://github.com/containous/traefik/pull/420/files#diff-04c6e90faac2675aa89e2176d2eec7d8
Thank you :)

Thanks @shochdoerfer, we fixed this in another PR too ;)


Rancher/RKE k8s installation cannot authenticate with Artifactory Docker Registry

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Push all rancher docker images to private registry in Artifactory
Specify private_registries in rancher config file and specify artifactory host, username, API Token(password)
rke up --config myconf.config

Result:
WARN[0001] Can't pull Docker image [ARTIFACTORY HOST/rancher/rke-tools:v0.1.50] on host [IP ADDR]: Error response from daemon: unauthorized: The client does not have permission for manifest
Other details that may be helpful:
A normal docker login and docker pull works just fine with artifactory.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): Rancher 2.3
Installation option (single install/HA): HA

My work around for now was to deploy an additional VM with https://docs.docker.com/registry/ and push the images there.

Hi @sogajeffrey
Do you put the following info in your rancher-cluster.yml?
Do you user any self-signed cert?
private_registries:
- url: my-url:443 # private registry url
  user: jiaqi
  password: "password"
  is_default: true

Hi @sogajeffrey
Do you put the following info in your rancher-cluster.yml?
Do you user any self-signed cert?
private_registries:
- url: my-url:443 # private registry url
  user: jiaqi
  password: "password"
  is_default: true

Yes I did.

@sogajeffrey is this for a cluster that rancher manages or the rancher HA cluster itself?
If its for a managed cluster, did you set the default registry option when you deployed rancher via helm?
I've seen an issue recently where we had this set and it was not honouring the private_registries config in the rke yaml for a managed cluster. However, the config worked fine for the rancher HA cluster.

Artifactory uses a non Rancher compatible authentication process. Rancher can only work with user and password, but artifactory requires a Auth string which is a hashed combination of username and password.
The generated secret should look something like this:
{ 	"auths": { 		"https://artprod.mycompany" : { 			"auth": "<USERNAME>:<PASSWORD> (converted to base 64)", 			"email"<EMAIL_ADDRESS>	} 	} }
but Rancher generates something like this:
{"auths":{"index.docker.io":{"password":"test","username":"test"}}}

I've seen an issue recently where we had this set and it was not honouring the private_registries config in the rke yaml for a managed cluster. However, the config worked fine for the rancher HA cluster.

@dan-evolvere I used both rancher server and RKE to deploy and both wern't working but @karaktaka explained the issue pretty well. Thank you!

This may be related to: https://github.com/containerd/containerd/issues/3556

We ran into a similar issue in a different setup.
rke version: v1.0.6 (running on a jump box)
OS: RHEL 7.8
Docker: 19.03 (installed directly using tarball, not via yum)
Capturing the packets on the docker socket, we found that X-Registry-Auth header is having an empty value when the image is being pulled via rke up command, but when we do a docker pull we see it set properly. Not sure where the difference is kicking in.
Thanks to @ansilh for digging up these commands:
// backup the socket
sudo mv /var/run/docker.sock /var/run/docker.sock.original
// use tcp port 8089 proxy the original socket
sudo socat TCP-LISTEN:8089,reuseaddr,fork UNIX-CONNECT:/var/run/docker.sock.original
// use the new socket to proxy the 8089 port
sudo socat UNIX-LISTEN:/var/run/docker.sock,fork TCP-CONNECT:<IP_ADDRESS>:8089
sudo tcpdump -i lo -netvv port 8089

Could also be caused by this: https://github.com/rancher/rancher/issues/27029

Can this be tested with 2.4.5 as it seems to be the root cause of #27029?

Tested with artifactory v7.6.3 in Rancher 2.4.3:
Created a custom cluster with private registry pointing to docker artifactory. In this case password was set to "passw~rd" [#27029]. And saw the issue reproduce. Authentication fails with:
1776 remote_runtime.go:105] RunPodSandbox from runtime service failed: rpc error: code = Unknown desc = failed pulling image "rancher.<ip>.xip.io:80/rancher/pause:3.1": Error response from daemon: Get http://rancher.<ip>.xip.io:80/v2/rancher/pause/manifests/3.1: unknown: Authentication is required

This is also the case when bringing a cluster up with rke up using rke v1.1.1
When the password does not include special characters, provisioning with private registry set to docker artifactory succeeds.
Tested with artifactory v7.6.3 in Rancher 2.4.5 and RKE v1.1.3:
Created a custom cluster with private registry pointing to docker artifactory. In this case password was set to "passw~rd" [#27029]. Cluster provisioned successfully.
Cluster also provisioned successfully when bringing a cluster up with rke up using rke v1.1.3

Hi @sogajeffrey, can you please test and see if this issue is fixed in 2.4.5 for you?

@maggieliu In @janeczku's case, the string was encoded incorrectly, but when we hit this problem, the X-Registry-Auth header had an empty value. So I think we had hit a different problem.

We did not hit the issue mentioned in this ticket when testing with a rancher server setup that had the system images served from docker artifactory (1v7.6.3`).
Was able to pull images successfully and get a custom cluster up and running when private_registries was set to docker  artifactory instance that was auth enabled.

An empty x-registry-auth header would also be a symptom of https://github.com/rancher/rancher/issues/27029.  They seem to be identical issues.


atifacts when runing demo_image_editing

hi, I run this demo , but got bad results as follows:

the inversion is good, but after editing, the result is bad

huh, I am not sure. could you try with different images? I would also play around with the prompts used during inversion. you can be more specific like "photo of a man with short hair, with white tank top" or you can try just "person". I am not really sure what would do the trick here. Because looks like the reconstruction is not as good (given one of the slider images with scale=0 also has artifact)

huh, I am not sure. could you try with different images? I would also play around with the prompts used during inversion. you can be more specific like "photo of a man with short hair, with white tank top" or you can try just "person". I am not really sure what would do the trick here. Because looks like the reconstruction is not as good (given one of the slider images with scale=0 also has artifact)

yes, I try with this image, the result is ok, but I change the prompt with the problematic image, the result is still bad:

And I test Xl slider inference with eyebrow lora, it influence other area besides eyebrow:

I see, this looks to be related to null inversion. Clearly the null inversion can invert the top image very good, but the bottom image without slider is still not properly reconstructed.
To reduce the effect on other aspects in image - you can reduce the start_noise parameter during inference.
Closing this issue. please feel free to reopen if the issue persists

And I test Xl slider inference with eyebrow lora, it influence other area besides eyebrow:

this is not about real image edit, when the scale is large, it seems will affect other attribute， when scale small, the effect is not obvious.


add: Oluwatimilehin

Description
Added my own profile
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Note to reviewers

@therealtmmy  Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and Star the repo to help us grow even further!


Request config.json via relative path for easier deployment behind a proxy

request ./config.json instead of /config.json in main.js so that it is trivial to route to the configuration behind a proxy:
https://github.com/timoschwarzer/gitlab-monitor/blob/ec4dff67cf9f0607fb3026fe9ea8d62e6fadc24a/src/main.js#L31

Maybe just an environment variable like CONFIG_PATH?


How can we help develop the addon?

How can the community help develop the addon?
My question is - are you interested in expanding the number of voiced-over NPCs and how can I help you?
Could you please provide an example of sql or some other type of data that would be useful for you to input and implement in future versions of the addon?
Also, if you need a second opinion on something in russian, would be honored to help.

Yes. I actually uploaded a plan yesterday for the community. I have someone started on Part 2 (scraping wow text). I'm working on Part 1. If youre looking to join the effort for Part 2 join the discord and @mrthinger. I can connect you with the other dev working on this and help answer clarifying questions.
That goes for anyone else that'd like to help too.
https://github.com/mrthinger/wow-voiceover/blob/master/RETAIL-PLAN.md
The plan is marked as retail, but includes everything we'd need for any expansion or language.


customizable parametric adaptation decorator

BasicParametricAdaptation and other kinds of ParametricAdaptation functions should be implemented as decorators for the DecoratedActivationFunction.

done


cannot find symbol when compile FE with JDK 12

Describe the bug
The following error will occur when compile FE with JDK 12:
[ERROR] /home/kangkaisen/palo/fe/src/main/java/org/apache/doris/catalog/EsTable.java:[43,22] package javax.xml.soap does not exist
[ERROR] /home/kangkaisen/palo/fe/src/main/java/org/apache/doris/task/LoadPendingTask.java:[41,24] cannot find symbol
  symbol:   class Debug
  location: package sun.security.ssl

I found the sun.security.ssl.Debug is unnecessary. So we could remove it.
Besides, I think we should add a maven-checkstyle-plugin to check FE code style.

Besides, I think we should add a maven-checkstyle-plugin to check FE code style.

It's a good idea. Can you help us to add this plugin?
And I think we should discuss about how to use it. For example, what should be checked first.
I'm afraid there will be too many problems if we enable this checker

OK. I could do this work, but I am busy recently.  I will do this work when I have time.
Before I do it, I will discuss the detail with you.


Not support battery percent remaining devices

Check all that apply
Type of Change

[ ] WWST Certification Request

If this is your first time contributing code:

[ ] I have reviewed the README.md file
[ ] I have reviewed the CODE_OF_CONDUCT.md file
[ ] I have signed the CLA

[ ] I plan on entering a WWST Certification Request or have entered a request through the WWST Certification console at developer.smartthings.com

[x] Bug fix
[ ] New feature
[ ] Refactor

Checklist

[ ] I have performed a self-review of my code
[ ] I have commented my code in hard-to-understand areas
[ ] I have verified my changes by testing with a device or have communicated a plan for testing
[ ] I am adding new behavior, such as adding a sub-driver, and have added and run new unit tests to cover the new behavior

Description of Change
Add battery device vid witch does not support battery percent remaining attribute.
Summary of Completed Tests
Tested with hue device witch also does not support battery percent remaining attribute.

 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.xjie.huang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update SimpleToken.sol

bugfix;

decimals() was always returning 0

Huh, is that so? The tests are written in such a way that may very well be possible. Great catch!
Anyway, Solidity 0.5.x disallows this sort of thing, and this has already been solved during the port (see here), so I'll close this issue to avoid having to also merge this into that branch.
Thanks a lot @sinasabet81!


Cut Copy Select doesn't work

Does Cut Copy Select work in Chrome and Safari?
Does It only support Internet Explorer?
document.execCommand("cut");
document.execCommand("copy");

menu.js.

As it says on the readme, this can't really be done at the moment:

Clipboard support is somewhat limited. You can copy with Ctrl+C, cut with Ctrl+X, and paste with Ctrl+V, but data copied from JS Paint can only be pasted into other instances of JS Paint. There's apparently no way for web apps to properly copy image data to the clipboard. "Support programmatical copying of images to clipboard" is currently the top starred issue of chromium. To use the clipboard menu items, you need to install the Chrome app.

The chrome app, or an electron app. Or possibly a browser extension to enable it in the web app.
Hopefully browsers implement this new asynchronous clipboard API sooner rather than later, and support images.
I could however add a notice of some kind, directing you to right click and copy an <img> in a dialog window.

This might be of some interest to you guys:
https://developers.google.com/web/updates/2018/03/clipboardapi

@oleteacher Yeah, I'm waiting for it. There's no image support yet..

@rraallvv I can't detect whether the cut/copy/paste is going to work (currently*), so I've gone the route of "playing it safe" and graying the options out in order to lessen the expectation there that they'll work. But a better approach may be to show the items as enabled, but show a dialog box that explains the limitations.

Selections you cut or copy from within JS Paint can't be pasted into other applications. (Web pages can't copy images to the clipboard in a standard way yet, however jspaint uses a textual representation so that you can paste into other instances of jspaint.)
Pasting means accessing the user's clipboard, and all the major browsers (Chrome, Firefox, and Edge) prevent web pages from triggering this, but you can use Ctrl+V to initiate a paste yourself

*There's queryCommandEnabled, but it doesn't work reliably. There may be a way get it to work reliably by selecting hidden text, even tho that's not what you'll actually copy.
(Also, execCommand returns a boolean, which is sadly not described as a success flag, but rather in terms of "supported" and "enabled", which is theoretically what we can already determine from queryCommandEnabled and queryCommandSupported.)

Just for heads up, the image support for clipboard copy/paste is now supported in Chrome, based on the posts at https://bugs.chromium.org/p/chromium/issues/detail?id=150835.

Oh, cool. Okay, I've implemented this today. I still need to:

Test it in different browsers, to make sure the old functionality still works
Test error messages to make sure there's not an error in showing an error (it might sound silly or obscure but it's very easy to mess this sort of thing up because you're generally focusing on the main functionality, and generally not getting errors (hopefully!) – you have to explicitly test your error handling, either by simply throwing a fake error to test how it propagates into an error dialog box, or where possible triggering real errors in the APIs you're using, like by giving it bad data)
Oh and there's the URL stuff: I made it so you can paste a URL, now both from Ctrl+V and from the menu – even tho there's already File > Load from URL – but I need to make it actually check that the text you paste is a URL because otherwise it shows a meaningless error message if you weren't trying to paste a URL but rather, incidentally copied some text in between copying and pasting an image, for instance.

Added URL validation, fixed & improved error messages, and got the old functionality working again.
Now released! Try it out: https://jspaint.app
(TODO: allow copy/cut/paste of text in edit fields from the menus, as works in mspaint.)

Copy to system clipboard does not work in Firefox v88, v89 or Safari 14.1 (macOS BigSur)
Safari at least pops an error message:

Firefox does not show any messages or hints about errors, although if there was an image on he clipboard prior trying to copy, it clears it:

cmd + ctrl + shift + 4 (select portion of the screen and copy it to the system clipboard)
paste to JS Paint (works)
play around on the canvas (e.g. "highlight" important bits to focus on)
select all (ctrl + a or cmd + a )
copy (ctrl + c or cmd + c)
try to paste it to any platform (cmd + v) (e.g. GitHub comments section)

Expected: modified image content should be uploaded
Actual: nothing happens (the clipboard is cleared because the previous snippet isn't pasted either)
(Inconvenient) Workaround:
After modifying the pasted snippet, the partial screen capture can be used to save the contents of the canvas to the clipboard again.
p.s. if this inconvenience would be solved, it would be the best easy-to-use Paint alternative for macOS/Linux that doesn't require intermediate file saves when documenting/providing quick support for clients/colleagues etc.

@forgedhallpass Unfortunately I don't have a lot of control over this. Firefox and Safari haven't enabled support for copying images to the clipboard yet. However, I could do better about detecting when it's not supported, e.g. NotAllowedError.
In Firefox 88, support is behind a flag, which you can enable by visiting about:config and toggling dom.events.asyncClipboard.clipboardItem to true (search for it and then double click to toggle).
Copying works, but pasting does not, and in fact it seems to throw undefined rather than an Error object.

In Firefox 88, support is behind a flag, which you can enable by visiting about:config and toggling dom.events.asyncClipboard.clipboardItem to true (search for it and then double click to toggle).

This solved the problem for me, thank you @1j01 !


Voice cloning in English

General Question

Hi
I want to ask does paddlespeech supports voice cloning in English. I have tried and configured english (am & voc and ge2e) models but unable to get voice cloning.Its generating english voice but in the same accent for any input.I am using voicecloning.py script for text to speech
Kindly answer the question/query.

Hello developers, thank you for paying attention to PaddleSpeech open source project, sorry for bringing you a bad development experience, the current open source project maintenance manpower is limited, you can try to modify the PaddleSpeech source code by yourself, or ask other developers in the open source community to help. Communication channel:：飞桨AI Studio星河社区-人工智能学习与实训社区


fpsim - local doc build failing (make html)

Closing, as this works fine with our RTD build and with Windows multiprocessing issue fixed.


Overall Project Spec in README.md

The following info should be provided:

Project motivations
What is Fact Finder? (and also what is not)
How Fact Finder works!
Project milstones and scopes
How to run the code
How to contribute or how to join the team

i can take this

Should we do these in Thai or English? What do u think?
@tee4cute @vtno

Should we do these in Thai or English? What do u think?

@chaintng English would be better (but can do later :)


[HOLD for payment 2024-07-10] [HOLD for payment 2024-07-02] [$250] Send invoice - Two unread markers when marking message unread in invoice thread

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.82-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to FAB > Send invoice
Enter amount, select participant and send the invoice
Go to invoice thread
Send a message
Right click on the message > Mark as unread

Expected Result:
There will be one unread marker above the message
Actual Result:
There are two unread markers above the message
If the invoice currency is foreign currency, three unread markers will show up - another one is below Total row
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/af715cb2-0c7d-42f7-a426-f8245a44fa5c
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0172ef4f418ef0c267
Upwork Job ID:<PHONE_NUMBER>966992916
Last Price Increase: 2024-06-14
Automatic offers:

suneox | Reviewer | 102768842

Issue OwnerCurrent Issue Owner: @twisterdotcom

@twisterdotcom  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Hello folks, sorry I was OOO last week. I have a bunch to do, I will get them all done today. Thanks for your patience.

Payment Summary:

@suneox paid $250 here (Reviewer)
@daledah paid $250 here (Contributor)

`


Spike rearchitect app context for deferred load of data per page

Overview
.
References and additional details
https://www.notion.so/shapeshift/Mobile-profiling-0321f42373a14e399db7e5255676a97c
Acceptance Criteria
.
Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

Some screens that are particularly laggy from willy wallet:

connecting my native wallet, after enterting password the UI gets frozen for 1-2 minutes.
after unlocking the wallet .
return to the app, the page is blank for a while, but eventually the UI loads and gets stuck trying to ask for the native
wallet password while the balances load in the background.
occasionally the page becomes unresponsive; for some reason it takes longer to load when returning to the app than when connecting the wallet from the dashboard.
-->wondering where this comes from in context<--


ALVR doesn't stream audio

Linux Mint 21.3
Kernel Version <IP_ADDRESS>-generic
Pipewire on newest version.
ALVR Version is 20.10.0
Whenever I connect ALVR Audio as my default streaming device, I am unable to load any spotify music, or youtube videos and the audio doesn't stream, I am a bit lost since it'd be nice to have audio for beat saber and other games.

Nvm, my pipewire install got fucked up after some updates, switched to linux mint 22 with a brand new install and it works great


How download to custom directory

For example
how download to f:\download ?
gdrive.exe download 0B3X9GlR6EmbnZ1NGS25FdEVlWEk -O f:\download or how ?
And, how save with another filename?

You can specify download path like this:
$ gdrive.exe download --path "c:\path\to\download\directory" <FILE ID>

Or you can redirect from stdout and can save with another name
$ gdrive.exe download <FILE_ID> --stdout > "C:\Path\to\Directory\Filename"


how to generate custom json types with prisma-json-types-generator?

Hi. I tried using prisma-json-types-generator but it looks the json type generated by prisma-kysely is still unknown even after declaring prisma-kysely right after prisma-json-types-generator

Hey there @waptik. We don't have any inter-compatibility with other generators, and it's not planned as of right now. Sorry. You could possibly use zod to achieve what you're looking for, or just perform a straight cast if you are confident that the type is correct.

@valtyr Would you accept PR's if I checked on how to make it compatible with other generators?

@pruxis I have a separate feature planned that should make this possible. We already support per-field type overrides using the /// @kyselyType(...) docstring. Here's the section in the README. What's missing is a way to add a preamble to the prisma-kysely output file with import statements. We'll be tracking the feature in issue #61.


Remove unnecessary set_env_class_* Makefile targets

What
Now that the makefile is structured so that we always do make <env> <action>, there's no need for the environment setup to be in separate targets, it can be in the env targets themselves.
How to review
Verify that the makefile still works (eg verify that make <env> showenv outputs the expected values etc).
Who can review
Anyone but myself.

works


gha-merge-up - Merge-up javascript

Follow up issue from https://github.com/silverstripeltd/product-issues/issues/753
gha-merge-up currently does not support js files, instead it will fail any build that includes client/ in the files diff
ACs

[ ] Diffs that include client/ filter are allowed
[ ] If changes in the client/ are detected, for non silverstripe/admin modules, silverstripe/admin is installed in a sibling directory and yarn install is run (see gha-update-js for how to do this)
[ ] nvm use <correct_version> is run where correct_version is correct for the major version being merged-into
[ ] yarn build is run in the module being merged-up
[ ] The build fails if there is any issue with yarn build
[ ] Any updated dist files are git added
[ ] Initially, only asset-admin will use this functionality

Follow up issues created

https://github.com/silverstripe/gha-generate-matrix/issues/59

PRs

https://github.com/silverstripe/gha-merge-up/pull/6
https://github.com/silverstripe/silverstripe-asset-admin/pull/1375

PRs merged, all ACs met.


[Frontend] Integration the Influencers Signup with Backend Fixed

Description
Frontend and backend changes are included and influencer signup is completed.
Issue
Issue Link resolve #10

@Kajol-Kumari  @RohitGupta1237  can you please check and merge it.

@Kajol-Kumari  updated.

the font is still looking inconsistent

 the font is still looking inconsistent

Here everything is like I the design only.

@Hemu21 lets take ui fix in a separate pr 'coz the current ui looks very inconsistent


Add WithOptionalCodeBlock trait and adopt it where there's a body

Introduce the new WithOptionalCodeBlock trait to cover entities that have optional code blocks, which includes all of the function-like syntax nodes (FunctionDeclSyntax, AccessorDeclSyntax, etc.).
Update SwiftSyntaxBuilder to use With(Optional)CodeBlock.

Make HasTrailingCodeBlock inherit from WithCodeBlock. It only applies to a subset of types that conform to WithCodeBlock.
Eliminate HasTrailingOptionalCodeBlock in favor of WithOptionalCodeBlock.

@swift-ci please test

Can you add a release note entry about the addition of WithOptionalCodeBlock?

Sure, added.

@swift-ci please test

@swift-ci please test Windows


Add "delete torrent with data" option for torrents

@harchani-ritik  I'd like to work on this

@pratikbaid3 Apology but I am already working on this and will soon make a PR for this issue.


bash interpolation inside shots.yml

Hi there! thanks so much for this awesome library. I've been using it to archive photos of my website every day to keep a history.
One thing I'm having trouble with is bash interpolation inside shots.yml for a multi-shot set-up. Basically, I want to configure the command to output to a folder with the current date as the file name. Here's an example of what that looks like for an item inside shots.yml
- url: https://spencerchang.me
  output: personal-website/$(date '+%Y-%m-%d' | sed 's/-0\([0-9]\)-/-\1-/').png
  ...

That output works fine inside my terminal but interprets it literally whenever I run shot-scraper multi. I've tried surrounding it in different variations of quotes and escaping without luck, so wanted to check to see if this is supported?

Okay, looking through the code and given #36 it seems this doesn't work, and python's standard subprocess.run doesn't support interpolation :(


It seems Slate is adding additional Index Sections on Compilation/Upload

Problem
Inside of my Slate-created theme, I have 2 or 3 Homepage Sections (with Presets so they show in the homepage theme customizer).
However, upon uploading the theme, it appears that the theme has 19 or so Homepage templates – many of which are not in my source files,
Including "Map", "Testimonials", "Gallery", etc, as you can see here:

If I download a ZIP of my theme from Shopify, these "new" sections are included in the Downloaded folder. But, again, they're not in my Slate files (or the Slate dist files).
So, I'm trying to find where additional sections are merged or added.
My main issue is that I have made a "Featured Products" section for the homepage. But Shopify seems to be adding it's own, so I now have 2 Featured Products sections.

How can I edit or delete these auto-added sections?

Ah. Of course...
Slate doesn't upload the settings_data.json by default, so that would explain why those sections weren't showing locally. I updated from within the Admin theme editor to fix.
Thanks, @huguestennier


[tool_crash] FileSystemException: Cannot open file, path = 'C:\Users\ARPIT\Desktop\NV main\source.flutter-plugins' (OS Error: Access is denied., errno = 5)

Command
flutter pub get

Logs
  FileSystemException: FileSystemException: Cannot open file, path = 'C:\Users\ARPIT\Desktop\NV main\source\.flutter-plugins' (OS Error: Access is denied.
, errno = 5)

  #0      _File.throwIfError (dart:io/file_impl.dart:645:7)
#1      _File.openSync (dart:io/file_impl.dart:489:5)
#2      _File.writeAsBytesSync (dart:io/file_impl.dart:614:31)
#3      _File.writeAsStringSync (dart:io/file_impl.dart:638:5)
#4      ForwardingFile.writeAsStringSync (package:file/src/forwarding/forwarding_file.dart:154:16)
#5      _writeFlutterPluginsList (package:flutter_tools/src/plugins.dart:339:17)
#6      refreshPluginsList (package:flutter_tools/src/plugins.dart:784:24)
#7      FlutterProject.ensureReadyForPlatformSpecificTooling (package:flutter_tools/src/project.dart:210:5)
#8      PackagesGetCommand.runCommand (package:flutter_tools/src/commands/packages.dart:125:23)
<asynchronous suspension>
#9      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:626:18)
#10     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:75:64)
#11     _rootRunUnary (dart:async/zone.dart:1134:38)
#12     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#13     _FutureListener.handleValue (dart:async/future_impl.dart:140:18)
#14     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:682:45)
#15     Future._propagateToListeners (dart:async/future_impl.dart:711:32)
#16     Future._completeWithValue (dart:async/future_impl.dart:526:5)
#17     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:556:7)
#18     _rootRun (dart:async/zone.dart:1126:13)

  [✓] Flutter (Channel beta, v1.13.6, on Microsoft Windows [Version 10.0.18363.592], locale en-US)
    • Flutter version 1.13.6 at C:\src\flutter
    • Framework revision 659dc8129d (5 weeks ago), 2019-12-30 09:24:47 -0800
    • Engine revision bdc9708d23
    • Dart version 2.8.0 (build 2.8.0-dev.0.0 c547f5d933)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at C:\Users\ARPIT\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: C:\Program Files\Android\Android Studio1\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    ✗ Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[!] Android Studio (version 3.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    ✗ Unable to determine bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio1
    • Flutter plugin version 42.1.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[✓] VS Code (version 1.41.1)
    • VS Code at C:\Users\ARPIT\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.8.0

[✓] Connected device (3 available)
    • SM J701F   • 52005384e28e645b • android-arm    • Android 9 (API 28)
    • Chrome     • chrome           • web-javascript • Google Chrome 79.0.3945.130
    • Web Server • web-server       • web-javascript • Flutter Tools

! Doctor found issues in 2 categories.

Flutter Application Metadata
Version: 1.4.4+2
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Plugins
cloud_firestoreconnectivitydevice_appsfirebase_analyticsfirebase_authfirebase_corefirebase_messagingfirebase_storageflutter_account_kitflutter_facebook_loginflutter_webview_pluginfluttertoastgoogle_maps_flutterimage_pickerintercom_flutterlocationnotification_permissionspath_providerrate_my_appshareshared_preferencessqfliteuni_linksurl_launchervideo_playerwebview_flutter

HI @arpitpro47
does this solution work for you ?
thank you

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so it's always better to open new issues and reference related issues.


fix: merged ASAR does not unpack when there is only one unpacked file

Pattern {file1,file2} only works with multiple files. minimatch does not recognize pattern {file1} so if user only has one unpacked file, asar will not unpack it.
Resolve https://github.com/electron/universal/issues/54

@quanglam2807 please sign in to app.circleci.com and re-push your commit so that CI runs

@MarshallOfSound Job was rejected because resource class medium, image xcode:12.2.0 is not a valid resource class

@MarshallOfSound Did I miss anything? I couldn't make CircleCI run :(

@quanglam2807 rebase on latest master it should be good now 👍


Known to work version

[skip ci]

I may have spoken too soon.
Yes, no segfaults. And tests pass. But most I tried to translate doesn't work. I realize there are limitations.
I'm looking at this file: https://github.com/wavexx/bond-drivers/blob/a412983da1bb4d0eb3c6155193c80c227cf2627d/Python/stage2.py
and even trying tiny peices of it, at most so far I've seen work is:
py2jl"""
       def __BOND_repl():
           SENTINEL = 1
       """

E.g. the next line "while True:" or a little more doesn't work. Should it? I tried to change the condition. Maybe there's another version of Julia that works better and/or an older of your package?

Oh, about this. Not all python constructs are supported now, and some of them are not tested yet. Please report the issues and for this sort of problems I can promise fast fixes.


Listen to image loadings over time using streams

Hello,
See this kind of related issue:
https://github.com/smidwap/wysihtml5_size_matters/issues/6
As you can see I'm using some code to resize an editor:
      Wysihtml5SizeMatters.prototype.handleImagesLoading = function() {
        var _this = this;
        this.$body.imagesLoaded()
            .progress( function( instance, image ) {
              _this.adjustHeight();
            })
            .always(function( instance ) {
              _this.adjustHeight();
            });
      };

It would be cool if it was possible to, instead of using a deferred JQuery object, be able to listen to image loadings over time under a particular dom node.
Basically this would do:

Listen to DOM changes under a given DOM element
On new image tag added, emit the regular deferred in a stream

The idea is that it could be nice if your lib were able to detect the add/remove of img tags under a given DOM node, so that we could subscribe to the images loading over time.
Maybe it's out of the scope of this library but I think my need is quite common.
I've had to use some code found on SO to do that:
// see http://stackoverflow.com/a/14570614/82609
      Wysihtml5SizeMatters.prototype.detectDOMStructureChanges = (function(){
          var MutationObserver = window.MutationObserver || window.WebKitMutationObserver,
              eventListenerSupported = window.addEventListener;

          return function(obj, callback){
              if( MutationObserver ){
                  // define a new observer
                  var obs = new MutationObserver(function(mutations, observer){
                      if( mutations[0].addedNodes.length || mutations[0].removedNodes.length )
                          callback();
                  });
                  // have the observer observe foo for changes in children
                  obs.observe( obj, { childList:true, subtree:true });
              }
              else if( eventListenerSupported ){
                  obj.addEventListener('DOMNodeInserted', callback, false);
                  obj.addEventListener('DOMNodeRemoved', callback, false);
              }
          }
      })();

Thanks for this feature request! Looks like this issue didn't get much interest over time. Closing!


Figure out inline issues

I'm running DBSCAN benchmark with 100K points using gcc/7.5.0 on the host (Intel Xeon CPU E5-2620 v4 2.10GHz).
The current master flamegraph (flame.svg.txt) has stacks containing ArborX::Point::operator[], which was expected to be inlined:

If I change all KOKKOS_INLINE_FUNCTION to KOKKOS_FORCEINLINE_FUNCTION in Point files, it disappears (flame-forceinline.svg.txt)

and the traversal is sped up by 15-20%.

This does not seem to be the case for CUDA, though. Introducing force inline (at least, in Point) did not seem to change runtime. And I don't have a flame graph for it.
We need to better understand the situation, and how to fix it.
P.S. To produce flame graph, execute this:
perf record -g ./ArborX_DBSCAN.exe --cluster-min-size=100 --binary --eps 0.042 --filename hacc_points.dat --core-min-size 1 --print-dbscan-timers
perf script | $FLAMEGRAPH_DIR/stackcollapse-perf.pl | $FLAMEGRAPH_DIR/flamegraph.pl > flame.svg

assuming you downloaded FlameGraph into $FLAMEGRAPH_DIR. The resulting file, flame.svg, can be opened in Firefox for examination.

We need to better understand the situation, and how to fix it.

Seems to me that this is a valid case to use KOKKOS_FORCEINLINE_FUNCTION. We probably want to check on other benchmarks to verify this result but I don't see what we need to understand better.

We need to better understand the situation, and how to fix it.

Seems to me that this is a valid case to use KOKKOS_FORCEINLINE_FUNCTION. We probably want to check on other benchmarks to verify this result but I don't see what we need to understand better.

User error, wrong CMake build type (CMake silently ignored it), which resulted in it building unoptimized code. With proper one, flame graph is actually much better, and everything in the tree traversal is inlined:
flame-opt.svg.txt:

User error, wrong CMake build type (CMake silently ignored it), which resulted in it building unoptimized code. With proper one, flame graph is actually much better, and everything in the tree traversal is inlined:
flame-opt.svg.txt:


isSelectable() and ignored toggleActivation

There is a method FlexibleAdapter.isSelectable() with default implementation delegating the call to an item. This is OK.
This method is also used in the body of FlexibleViewHolder.toggleActivation() where if the result is false, the view is NOT activated (still correct). However, this ignored activation request is not propagated to upper layers which matters for example in FlexibleViewHolder.onLongClick which has a return value true/false and even in this case of ignored activation returns 'true' (line 143).
I suggest that toggleActivation() has a boolean return value and consider this return value at all places where invoked.
Maybe I am missing something. What I try to achieve is that only certain items in the list can trigger multi-select mode activation. For these items I am returning FlexibleAdapter.isSelectable() = false. The item does not get selected (the multi-select does not get activated). However still the phone vibrates as if the multi select gets activated. I am now using this workaround in my ViewHolder:
    public boolean onLongClick(View view) {
        // hack to workaround bug in FlexibleAdapter - selectable = false
        FlexibleAdapter.OnItemLongClickListener oldL = mAdapter.mItemLongClickListener;
        mAdapter.mItemLongClickListener = null;
        super.onLongClick(view);
        mAdapter.mItemLongClickListener = oldL;
        // ignore
        return false;
    }

This does not make the phone vibrate, the long click gets properly ignored.

@pavelkryl thanks for the explanation!
If I understood well, there's haptic feedback still going on when item is NOT selectable because the long click implementation returns always true.
Indeed by looking at the code, the long click is always executed but the check is done too late in toggleActivation().
I propose to fix this by checking if item is selectable, before calling the user implementation. This should resolve also the execution of that code that indeed should not occur. If not selectable the onLongClick() will return false.
However, I'm not sure that by returning false is enough to not vibrate the phone.
The object View has however options for haptic feedback, please check the Android SDK.

As I suspected, returning false is not ok. the haptic feedback is still there. So, there are 2 ways: remove the listener as you do OR you disable the haptic with the setter when item is unselectable. The latter is better but the client can do it himself, in its LongClick implementation in the method override (but also in the binding method should be fine).
As it is now, there is its own equilibrium between the click, long click, drag and activation events. I tried several solutions, also the one you suggested, but it got worst.
Watch out that, by returning false, the click event is also triggered.

If you don't have any update, I will close this issue.


resolves #403 advance table to next page if rowspan in first row does not fit in space remaining on current page

@rillbert FYI, this is the patch you proposed in https://github.com/prawnpdf/prawn-table/issues/106.


Feature request: support submissionType and consensus

I'm trying to update a kramdown-written document to match what the RFC Editor has done to it.  I'd like to set the attributes submissionType="IETF" and consensus="yes" in the  element.

1.0.41 now has these additional attributes in  as top level YAML variables (and provides convenient aliases):
+                  "consensus", "submissionType=submissiontype", "xml:lang=lang",

Note that "yes" means true in YAML, so you have to quote it:
consensus: 'yes'