hasImage method not working with HEIC images

Environment

System:
    OS: macOS 12.3.1
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 225.18 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.14.2 - ~/.nvm/versions/node/v16.14.2/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 8.5.0 - ~/.nvm/versions/node/v16.14.2/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.11.2 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.4, iOS 15.4, macOS 12.3, tvOS 15.4, watchOS 8.5
    Android SDK:
      API Levels: 28, 29, 30
      Build Tools: 28.0.3, 29.0.2, 30.0.0, 30.0.2, 31.0.0
      System Images: android-26 | Google APIs Intel x86 Atom, android-29 | Google APIs Intel x86 Atom, android-29 | Google APIs Intel x86 Atom_64, android-29 | Google Play Intel x86 Atom, android-30 | Google APIs Intel x86 Atom, android-31 | Google APIs Intel x86 Atom_64
      Android NDK: Not Found
  IDEs:
    Android Studio: 4.1 AI-201.87<IP_ADDRESS>42882
    Xcode: 13.3.1/13E500a - /usr/bin/xcodebuild
  Languages:
    Java: 1.8.0_265 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.1 => 17.0.1
    react-native: 0.64.3 => 0.64.3
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Platforms

both

Hi @ahansson89 , I am not able to reproduce the bug on iOS platform as checking for image on clipboard and pasting HEIC formats work well on my end, concerning the android platform there's no "hasImage" interface for checking for content on the Clipboard.  Although the implementation of the "hasString" interface checks for any content in the clipboard regardless if string or file. I have gone ahead to open a PR for pasting other Image formats from the clipboard on android

Hey @ahansson89 👋🏻
Could you see if the problem persists in the latest version?

The issue was not what I thought it was with HEIC images. Closing this.


[BUG]: Image/s are deleted from the component after translate from default language

Describe the bug
Image is deleted from inside of the populated component right after it is translated from default locale into Spanish language.
To Reproduce
Steps to reproduce the behavior:

Create a default English record for pages (https://prnt.sc/mKVmCENY_JmO) and add data with populated component having image/s (https://prnt.sc/pMqVlaQWf5CJ).
Click on "Locale" dropdown from Action list and move to Spanish language (https://prnt.sc/uLQsxmvrqdtS). Then click on "Translate from another locale" from English and click "Yes, fill in" and save the record.
Scroll down to the SEO component and you will find image is copied in Spanish record (https://prnt.sc/O-54oV-LgqQx) but the image from the English record - SEO component  will be removed (https://prnt.sc/mdGtyZVwNb8S)

Another way to translate all data:
https://github.com/Fekide/strapi-plugin-translate/assets/104756792/199085ad-a8ca-4bab-8961-cf7355218454
Expected behavior
The image should be copied and not deleted from the translated record.
System (please complete the following information):

OS: Win 10
Browser: Chrome
Versions:

strapi: 4.9.0
strapi-plugin-translate: 1.0.4
provider<EMAIL_ADDRESS>

Plugin config
Please add your plugin config here, e.g.
module.exports = {
  translate: {
    enabled: true,
    config: {
      provider: 'deepl',
      providerOptions: {
        apiKey: '*************'
      },
      translatedFieldTypes: [
        'string',
        { type: 'text', format: 'plain' },
        { type: 'richtext', format: 'markdown' },
        'component',
        'dynamiczone',
      ],
      translateRelations: true,
    },
  },
}

@Shekhar-Zealous I don't seem to be able to reproduce this, can you please verify if this still happens with the latest version of the plugin?

@Shekhar-Zealous I don't seem to be able to reproduce this, can you please verify if this still happens with the latest version of the plugin?

Yes, I have just checked it by upgrading the package to latest version 1.1.6 and it is still happening.


Additional User Input Flags 2

As requested over in my last PR, here's a new one that adds the option to disable the Username value.  I think I held off on that initially because of the possibility of a trailing comma if certain combinations were disabled.  I've reworked it a bit to ensure that there won't be trailing comma in any combination of prompts.  You should now be able to enable or disable all of the ones on the Welcome page without breaking the next steps.
I also went ahead and made it so that if the Configuration prompt is disabled it also removes mention of configurations from the Welcome text.
I think this will be my last PR before the next official release assuming everything works as expected.

Thanks again, @rougegoat.
I'm now observing a CONFLICT (content): Merge conflict in Setup-Your-Mac-via-Dialog.bash when attempting to merge; I think we were both making updates.
Will you please resubmit?

@rougegoat: Actually, will you please give me a few minutes to add some local fixes?

@rougegoat: I just pushed 1.10.0-rc23; thanks so much.

I have rebuilt using rc-23, and that looks to have fixed the conflict.


Last Location not working

Expected Behavior
Last location should pull the location from data\last-location-username.json (this file exist for me and has the latest time stamp)
"location": "",
"gmapkey": "",
"max_steps": 4,
"mode": "all",
"walk": 3.86,
"debug": false,
"test": false,
"initial_transfer": false,
"location_cache": true,
"distance_unit": "km",
"item_filter": {},
"evolve_all": "NONE",
"use_lucky_egg": false,
"evolve_captured": false,

Actual Behavior
Running bot won't work, tells me to specify location
[21:51:49] PokemonGO Bot v1.0
[21:51:49] Configuration initialized
[21:51:49] [x] Parsing cached location...
No cached Location. Please specify initial location.
Steps to Reproduce
Run the bot with a set coordinate for a bit, stop bot, remove coordinate and restart bot
Other Information
Dev Branch
OS: Win10
Git Commit: 9d2535001b92772c6b797183b1cf4df768806af3

You need to add your location coordinates into the "location" parameter. Example shown below:
"location": "37.787994, -122.407440",

I thought last location would be used if I left that field blank?

This is a duplicate of #1031.

Duplicate of https://github.com/PokemonGoF/PokemonGo-Bot/issues/1031
Please go to that thread for solution.
/closed


Cannot config password

Description:

Cannot connect to Redis Server that require password.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Cenos7 amd64
Steps to reproduce:
Drag drop redis connector via EI tooling into sequence flow and try to set property password.
<redis.init>
localhost
6379
3000
</redis.init>
Related Issues:

hello sir , did you find the solution for this yet ?
it does not work for me although I used

Moved the issue to https://github.com/wso2/micro-integrator/issues/3486


Java: Improve performance in all dataflow queries.

The uses of dataflow in the two Serializability framework libraries are in scope as soon as dataflow is brought in scope, by the import chain VirtualDispatch.qll -> Guards.qll -> GuardsLogic.qll -> IntegerGuards.qll -> RangeAnalysis.qll -> Reflection.qll -> Serializability.qll and this makes the dataflow configuration used in those libraries a bit bigger than needed for subsequent uses, so we shouldn't use the default dataflow library.
This should be a performance improvement for all dataflow-based queries.

Testing java/potentially-weak-cryptographic-algorithm on jdk8 shows that the intial pruning step nodeCandFwd1 goes from 40s to 5s.


Part 2 of 2 - EAP-RADIUS ipsec/strongswan support

This pull request is split across the tools and core repositories, this is the core part.

commit improves readability of the generated strongswan.conf by fixing indentions, and ipsec.conf by adding some missing spaces.
commit implements the config file generation parts that is missing.
commit implements the ui part, adding two fields for entering the RADIUS server and secret.

I merged the tools part, but still waiting for a final merge in a 17.1.x and feedback from Ad on this. :)

Good :) It has been years since I have touched PHP, so it might need a few changes here and there.
I have attached the  part of my config here, for people to test: https://forum.opnsense.org/index.php?topic=4323.0

@GurliGebis I will handle this one next week.
I have one small question, does this patch also add support for accounting, or is there more work needed for that? Our pluggable auth framework provides support for radius authentication, but doesn't provide accounting.

@AdSchellevis Great :)
All this patch does is make sure the right thing is added to the strongswan.conf and ipsec.conf for EAP-RADIUS.
The only parts added to ipsec.conf is the plugins/eap-radius part, with the "server" and "secret".
From what I know, that should make EAP-RADIUS use it for both auth and acc, but I haven't tested, since I have never had the need for accounting.
As for the RADIUS part in the pluggable auth framework, I haven't touched that.
I think that later on, it might be an idea to replace the two text boxes in my patch with a dropdown, where you can select multiple RADIUS servers defined in the auth framework.
But my plan with this is to get this up and running first, and then replace it later with the dropdown selecting one or more radius servers from the auth framework.
Does it make sense to go that route, or should I rework this to use the auth framework for selecting RADIUS servers?

Just checked the strongswan documentation - my patch does not enable accounting.
https://wiki.strongswan.org/projects/strongswan/wiki/EAPRadius
I think it would be better to save that for when it is changed to use the auth framework, since it would be possible to define it on the server there, what should be used and what shouldn't.

Okay, I know how this should be done.
Please don't merge this pull request until I give an okay (in case the idea I have does not work).
I will get back with a new pull request :)

Just got confirmation back from strongswan people - it is not possible to define different servers for different connections.
https://wiki.strongswan.org/issues/2229
So I plan on doing it the other way around - so to use this, you have to add one or more RADIUS servers to the auth framework.
Then, when you enable EAP-RADIUS for a connection, it takes all the RADIUS servers that is defined in there.
Does that sound like a usefull idea?

I will be recreating my repo, since the patches should not have been commited to the master branch (and it cannot be rolled back now that I has been pushed to github).
I will change my implementation like mentioned above, and create a new pull request from the new repository

Done, I have reimplemented it using the servers in the auth framework.
Please see the new pull request here: https://github.com/opnsense/core/pull/1345
I'm closing this one, since this version should not be pulled.


Data generation, Chinese language support and other LLMs

I have been able to get the data generation working by patching up the code shared in #21 for tool-augmented SFT data generation and setting up a connection to OpenAI, in specific gpt-3.5-turbo. However, when I tried to adapt the process for a tool in simplified Chinese, I was met with decoding errors as such:

Since gpt-3.5-turbo is capable of generation in simplified Chinese, I assumed the issue was somewhere in the code you gave in #21, or somewhere else in the ToolBench environment.
I actually solved the problem while writing up this issue but I'm still going to include it here in case this helps later: you have to set the encoding for the python I/O to an encoding that accommodates for your language e.g. simplified Chinese. No encoding was specified for the open() that was called to read in the query files, and after specifying the utf-8 encoding everything worked.
Also, I read somewhere on the ToolBench repo that you are working on integration with other LLMs like CPM-Bee. When will this be available, and will there be relevant documentation on how one could use custom-generated SFT data to train these models?
Thanks in advance.

Thanks a lot for repporting the bug in our previous codes!!
For integration with other LLMs and the relevant documentation, we'll make it within this month :)

Sounds good, looking forward to it!


Add regression test for spawning entities during remove hooks

I have made a test for this, that fails at 0.14.2 but passes in current bevy main, so it should cover from this regression
#[test]
fn test_spawn_entities_during_on_remove_hook() {
    #[derive(Component)]
    struct MyComponent;

    App::new()
        .add_plugins(MinimalPlugins)
        .add_systems(Startup, |world: &mut World| {
            world.register_component_hooks::<MyComponent>().on_remove(
                |mut world, _entity, _component_id| {
                    world.commands().spawn_empty();
                },
            );
        })
        .add_systems(
            Update,
            |mut commands: Commands,
                my_component_q: Query<Entity, With<MyComponent>>,
                mut exit_event: EventWriter<AppExit>,
                mut after_first_frame: Local<bool>| {
                for entity in &my_component_q {
                    commands.entity(entity).despawn();
                }

                commands.spawn(MyComponent);

                if *after_first_frame {
                    exit_event.send(AppExit::Success);
                }

                *after_first_frame = true;
            },
        )
        .run();
}

It's a bit of an unusual test (I'm not even sure if MinimalPlugins can be used in a test like this). It's more an integration test rather than a unit test. I don't know if there is some place inside Bevy repo where more of these tests exist. If so, I can open a PR to add it.
Originally posted by @Maximetinu in https://github.com/bevyengine/bevy/issues/16219#issuecomment-2458452806

I think we can probably swap this over to a simpler unit test using World and run_system_once :)  Then it can go in the hooks part of bevy_ecs, which is much simpler.


disallowAnonymousFunctions: remove errant "s" from error message

Coverage remained the same at 97.69% when pulling 96f1cc4acaed698be910ee2bef9180db8d229ece on jcchingithub:disallow-anonymous-functions into 038e20ff262cea691e513c9439b948bbbd80c151 on jscs-dev:master.

Thanks for contributing @jcchingithub.

@mrjoelkemp You're welcome!


fix issue #397

Isn't this fix working https://github.com/NativeScript/android-runtime/pull/401

@slavchev

https://github.com/NativeScript/android-runtime/pull/401 works fine. I just went for a simpler solution. Yesterday I fixed https://github.com/NativeScript/nativescript-cli/pull/1640 so now a simple enableAgent() call is enough.

It seems to me #401 fixes the debugger statements and this one. That's why I suggest we move on with  #401 instead.

I prefer to keep it as simple as possible. That's why I fixed the bug the CLI and then there is no need for workarounds it the runtime code. Win-win situation.

Just to clarify. The #401 pull is related to the bug when a debugger; statement exists in code and when the app run it should not stop there if no debugger is connected. The #401 pull fixes this and includes the changes you did in this PR. To me it is irrelevant which PR gets in, we just need to make sure the debugger statements issue is fixed as well.

@slavchev Have you tested this with the {N} VS code extension. It seems to me the extension will not work when doing an Attach command.

run ci


requests shows lots of DeprecationWarning on Python 3.6

Running tests on my project with the -Wd option now displays lots of DeprecationWarnings coming from requests.
From the Python 3.6 Changelog (https://docs.python.org/3/whatsnew/3.6.html#deprecated-python-behavior):
A backslash-character pair that is not a valid escape sequence now generates a DeprecationWarning. Although this will eventually become a SyntaxError, that will not be for several Python releases. (Contributed by Emanuel Barry in bpo-27364.)
For reference, this is the commit that fixed those DeprecationWarnings in Django 1.11: https://github.com/django/django/commit/8119b679eb85cdc0ae3d321e54d06dd0200a1e82

Do those deprecation warnings happen to point at the location in the code where the problem arises? If so, it'd be useful to see that output.

They actually do:
/venv/lib/python3.6/site-packages/requests/models.py:847: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/api.py:67: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/api.py:80: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/api.py:93: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/api.py:108: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/api.py:122: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/api.py:136: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/api.py:148: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/sessions.py:498: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/sessions.py:509: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/sessions.py:520: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/sessions.py:533: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/sessions.py:544: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/sessions.py:555: DeprecationWarning: invalid escape sequence \*
  """
/venv/lib/python3.6/site-packages/requests/sessions.py:565: DeprecationWarning: invalid escape sequence \*
  """

Yup, so these almost all seem to be in docstrings where the backslashes should be being doubled-up. Anyone who likes to is welcome to submit a PR to remove these.

(By the way, that "anyone who likes to" includes yourself @rubendura! Just wanted to make sure that you didn't think I was excluding you.)

I am on it. I just forked the repo ;)

For some reason I can't get the deprecation warnings to show when running the tests against the requests test suite.
I tried running them as shown in http://docs.python-requests.org/en/latest/dev/todo/#development-dependencies
But doing that doesn't allow me (or I don't know how) to pass the -Wd option to python, so I tried running them as
~/.local/share/virtualenvs/requests-ThIWAgEP/bin/python -Wd setup.py test
That seems to run them and displays DeprecationWarnings in other packages, but not on requests.
I want to get them against master and not use the warnings from my project because there could (and will) be differences in the source code for requests. Also this will show me how to set up my env in case I wanted to contribute to requests in the future.
Some help would be appreciated.

The easiest thing to do is to note that py.test is a Python script, and so you can do python $(which py.test) which should get you closer.

Won't python -m pytest work?

Thanks. But that gets me the same output. No DeprecationWarnings for requests when there should be some.
Below the full output of the command (I activated the venv created by pipenv before running it):
(requests-ThIWAgEP)  ✘  ~/src/requests   master ●  python -Wd $(which py.test)
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/bin/../lib/python3.6/site.py:165: DeprecationWarning: 'U' mode is deprecated
  f = open(fullname, "rU")
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/_pytest/assertion/rewrite.py:7: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  import imp
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_httpbin/plugin.py:4: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import serve, certs
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/plugin.py:8: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import embed
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/plugin.py:9: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import engine
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/engine.py:11: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from .compat import StringIO
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/plugin.py:10: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import compat
============================================================================================================================= test session starts ==============================================================================================================================
platform darwin -- Python 3.6.1, pytest-3.0.7, py-1.4.33, pluggy-0.4.0
rootdir: /Users/ruben/src/requests, inifile:
plugins: mock-1.6.0, httpbin-0.0.7, cov-2.4.0
collected 483 items

tests/test_hooks.py ...
tests/test_lowlevel.py ............
tests/test_requests.py ............................................................................................................................................................X................X.......................................................................................
tests/test_structures.py ...........................
tests/test_testserver.py ...........
tests/test_utils.py ..s......................................................................................................................................................................

============================================================================================================== 480 passed, 1 skipped, 2 xpassed in 54.42 seconds ===============================================================================================================

As you can see requests DeprecationWarnings are not being shown, but I don't get why.

@sigmavirus24 Yup, that also works but gives the exact same result as it should be doing the same.

That's odd. Do you see them if you just import the code?

No luck
(requests-ThIWAgEP)  ~/src/requests   master ●  python -Wd
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/bin/../lib/python3.6/site.py:165: DeprecationWarning: 'U' mode is deprecated
  f = open(fullname, "rU")
Python 3.6.1 (default, Mar 23 2017, 16:49:06)
[GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-8<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import requests
>>>

Ok, so that suggests that the deprecation warning there is either not coming from Requests or not valid in the current master branch.

So I thought, but it's easy to find an example where that still happens. For example, in requests/models.py:862 there is a tripe-quoted doc string containing an invalid escape sequence "*".
Unless I'm getting something wrong, that should be flagged as a DeprecationWarning in Python 3.6

That's very interesting

Sure, but it isn't. ;) So...what is going on?

Funny, I had to escape \ in that comment before for GitHub not to escape \ 😅

I tried running the same with -Wa and adding a random line to the code containing a few escape squences and still no luck. Running out of ideas here :S

OH! I wonder if you have to remove all the *.pyc files in the repo and any __pycache__ directories. Want to try that?

No luck with that. I started to wonder if py.test does something with those and came across this:
https://docs.pytest.org/en/features/warnings.html#ensuring-function-triggers
I can't figure out if that has something to do with it or not. My guess is that the problem lies somewhere else because DeprecationWarnings were not seen when importing requests.

Yeah, my assumption is that py.test does not by default suppress these. But I have no idea why you aren't seeing them.

Ok, this is interesting. If I interrupt py.test with Ctrl+C DeprecationWarnings will be printed. Compare this two sessions.
Without Ctrl+C
(requests-ThIWAgEP)  ✘  ~/src/requests   master ●  python -Wall -m pytest
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/bin/../lib/python3.6/site.py:165: DeprecationWarning: 'U' mode is deprecated
  f = open(fullname, "rU")
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/_pytest/assertion/rewrite.py:7: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  import imp
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_httpbin/plugin.py:4: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import serve, certs
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/plugin.py:8: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import embed
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/plugin.py:9: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import engine
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/engine.py:11: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from .compat import StringIO
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/plugin.py:10: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import compat
============================================================================================================================= test session starts ==============================================================================================================================
platform darwin -- Python 3.6.1, pytest-3.0.7, py-1.4.33, pluggy-0.4.0
rootdir: /Users/ruben/src/requests, inifile:
plugins: mock-1.6.0, httpbin-0.0.7, cov-2.4.0
collected 483 items

tests/test_hooks.py ...
tests/test_lowlevel.py ............
tests/test_requests.py ............................................................................................................................................................X...............<IP_ADDRESS> - - [05/Apr/2017 15:28:27] "GET /stream/4 HTTP/1.1" 200 1080
.X.......................................................................................
tests/test_structures.py ...........................
tests/test_testserver.py ...........
tests/test_utils.py ..s......................................................................................................................................................................

============================================================================================================== 480 passed, 1 skipped, 2 xpassed in 54.95 seconds ===============================================================================================================

With Ctrl+C
(requests-ThIWAgEP)  ✘  ~/src/requests   master ●  python -Wall -m pytest
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/bin/../lib/python3.6/site.py:165: DeprecationWarning: 'U' mode is deprecated
  f = open(fullname, "rU")
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/_pytest/assertion/rewrite.py:7: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  import imp
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_httpbin/plugin.py:4: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import serve, certs
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/plugin.py:8: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import embed
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/plugin.py:9: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import engine
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/engine.py:11: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from .compat import StringIO
/Users/ruben/.local/share/virtualenvs/requests-ThIWAgEP/lib/python3.6/site-packages/pytest_cov/plugin.py:10: ImportWarning: can't resolve package from __spec__ or __package__, falling back on __name__ and __path__
  from . import compat
============================================================================================================================= test session starts ==============================================================================================================================
platform darwin -- Python 3.6.1, pytest-3.0.7, py-1.4.33, pluggy-0.4.0
rootdir: /Users/ruben/src/requests, inifile:
plugins: mock-1.6.0, httpbin-0.0.7, cov-2.4.0
collected 483 items

tests/test_hooks.py ...
tests/test_lowlevel.py .^Csource:69: DeprecationWarning: invalid escape sequence \*
source:82: DeprecationWarning: invalid escape sequence \*
source:95: DeprecationWarning: invalid escape sequence \*
source:110: DeprecationWarning: invalid escape sequence \*
source:124: DeprecationWarning: invalid escape sequence \*
source:138: DeprecationWarning: invalid escape sequence \*
source:150: DeprecationWarning: invalid escape sequence \*
source:508: DeprecationWarning: invalid escape sequence \*
source:519: DeprecationWarning: invalid escape sequence \*
source:530: DeprecationWarning: invalid escape sequence \*
source:543: DeprecationWarning: invalid escape sequence \*
source:554: DeprecationWarning: invalid escape sequence \*
source:565: DeprecationWarning: invalid escape sequence \*
source:575: DeprecationWarning: invalid escape sequence \*

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! KeyboardInterrupt !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
to show a full traceback on KeyboardInterrupt use --fulltrace
/usr/local/Cellar/python3/3.6.1/Frameworks/Python.framework/Versions/3.6/lib/python3.6/socket.py:586: KeyboardInterrupt
=========================================================================================================================== 4 passed in 2.06 seconds ===========================================================================================================================

I don't know what's going on there, but looks like py.test might be somehow interfering with it.

So...why does running python itself exhibit the same behaviour?

I have no clue and it's really triggering me. Still investigating.
Have you been able to reproduce it on your side?

Nope, I get the relevant output. Feel free to use it:
/Users/cory/Documents/Python/requests_org/requests/requests/models.py:866: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/api.py:69: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/api.py:82: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/api.py:95: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/api.py:110: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/api.py:124: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/api.py:138: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/api.py:150: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/sessions.py:508: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/sessions.py:519: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/sessions.py:530: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/sessions.py:543: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/sessions.py:554: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/sessions.py:565: DeprecationWarning: invalid escape sequence \*
  """
/Users/cory/Documents/Python/requests_org/requests/requests/sessions.py:575: DeprecationWarning: invalid escape sequence \*
  """

I managed to get the warnings on my system by deleting pycache as you suggested and importing the package. Not deleting pycache or running the test suite was hiding those messages from me.


Open collective

Note: We need at least 100 stars on the repository

Setup open-collective:

Sponsor link
Contributors, sponsors grid view.

Update: Collective created at https://opencollective.com/villain


auth is enabled : false throws 503 unavailable

Environment

Operating System: Linux
Node Version:     v18.15.0
Nuxt Version:     3.3.3
Nitro Version:    2.3.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      srcDir, modules, vite, i18n, nitro, schemaOrg, telemetry, algolia, head, auth, viewport, css, build, runtimeConfig
Runtime Modules<EMAIL_ADDRESS>genesis-design-system/nuxt<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
We have a web development in which we have implemented the login "isEnabled : true". Everything works correctly.
Now we want to develop another website with the same programming, but that does not have a login, so to do a proof of concept we have set the variable isEnabled : false, which throws us the following error

Describe the bug
We have a web development in which we have implemented the login "isEnabled : true". Everything works correctly.
Now we want to develop another website with the same programming, but that does not have a login, so to do a proof of concept we have set the variable isEnabled : false, which throws us the following error

Additional context
No response
Logs
No response

This is an interesting one and I know how this happens: While you did turn off the auth module you still had the endpoint that imports the nuxtAuthHandler. But, the module didn't set this up when you turned it off!
As nuxt auto-strips unused code, and this only affects the backend, I think we can pull adding the import before the isEnabled flag is checked in the module.ts setup (:

edit: I actually now think we cannot: if we make the server imports still available if the module is disabled, we also have to change it so that it e.g., throws a 404 error when it is hit with a request.
Doing this will then cause the app-side integration to break and we'd have to later that to also "mock" auth behavior. The scope of this is quite big and unclear (eg: how should the module behave if it is loaded onto an authenticated page while the module is turned off?).
I'd recommend that instead of turning off the module for your external developer you turn off authentication, e.g., by removing any auth-checks from the app-side code.
Thanks and let me know if there's anything else we can do here!

Ok, but... if i put isEnabled  false from process.env variable, not throws me an error...
I think that this variable doesnt admit process.env variables

.env

edit: I actually now think we cannot: if we make the server imports still available if the module is disabled, we also have to change it so that it e.g., throws a 404 error when it is hit with a request.
Doing this will then cause the app-side integration to break and we'd have to later that to also "mock" auth behavior. The scope of this is quite big and unclear (eg: how should the module behave if it is loaded onto an authenticated page while the module is turned off?).
I'd recommend that instead of turning off the module for your external developer you turn off authentication, e.g., by removing any auth-checks from the app-side code.
Thanks and let me know if there's anything else we can do here!

I'm going to have to hack a lot of the code to implement what you're telling me... surely there's no chance that when the module is disabled, if I use auth, it will launch everything as if I weren't logged in?

We have the same code for diferent projects, in some projects there is Login / register, and others hasn't login...
We need that "isEnabled" works!


Unable to publish to docker hosted kafka

Im using kafka 0.11.0
I have set kafka endpoints as below
KafkaBootstrapEndpoints = [{"<IP_ADDRESS>", 9092}].
11> ok = brod:produce_sync(client1, Topic, Partition, <<"key1">>, <<"value1">>).
Output:
=ERROR REPORT==== 29-Aug-2017::14:55:31 ===
client client1 failed to connect to <IP_ADDRESS>:9092
reason:{connection_failure,econnrefused}
=WARNING REPORT==== 29-Aug-2017::14:55:31 ===
Failed to (re)init socket, reason:
{connection_failure,econnrefused}
=ERROR REPORT==== 29-Aug-2017::14:55:32 ===
client1 (re)connect to <IP_ADDRESS>:9092 aborted, last failure reason:{connection_failure,
econnrefused}
=WARNING REPORT==== 29-Aug-2017::14:55:32 ===
Failed to (re)init socket, reason:
{connection_failure,econnrefused}
=ERROR REPORT==== 29-Aug-2017::14:55:32 ===
client client1 failed to connect to <IP_ADDRESS>:9092
reason:{connection_failure,econnrefused}
=WARNING REPORT==== 29-Aug-2017::14:55:32 ===
Failed to (re)init socket, reason:
{connection_failure,econnrefused}
=ERROR REPORT==== 29-Aug-2017::14:55:33 ===
client1 (re)connect to <IP_ADDRESS>:9092 aborted, last failure reason:{connection_failure,
econnrefused}

@vedavidhbudimuri try to run kafka with advertised.listeners=PLAINTEXT://<IP_ADDRESS>:9092 in server.properties.

Sorry, my bad
the issues is that I have set listener in server.properties for kafka node to <IP_ADDRESS>


Cannot roll less than zero dice error

When I'm trying to evaluate expression (5+(-10))d10. However with -5d10 it works correctly ([-3, -2, -3, -1, -1])
  File "/usr/local/lib/python3.6/site-packages/dice/__init__.py", line 24, in roll
    return _roll(string, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/dice/__init__.py", line 58, in _roll
    raise DiceBaseException.from_other(e)
dice.exceptions.DiceFatalException: Cannot roll less than zero dice! (Add(5, -10) evaluated to -5) (at char 0), (line:1, col:1)

I think (5+(-10))d10 gets evaluated to (-5)d(10) and errors, and -5d10 evaluates to -((5)d(10)), rolling 5 dice and making each result negative.

Yes, that's right.

The interesting thing is, maxroll gives output (0) with the same expression (-5)d5 and roll doesn't. Or is it just my parser?

Min/max rolls short-circuit some of the evaluation code and return [min_value] * amount or [max_value] * amount, which results in an empty list when amount < 1. It should return an error, though, so I'll fix it soon.
Sam is right about the other things; the d operator has a higher precedence than the unary minus sign, so -5d10 evaluates to: Negate(roll 5d10 -> [1, 5, 7, 5, 2]) -> [-1, -5, -7, -5, -2]
Oh, and if you're using discord.py, I have a dice cog for Red here that you can use as a base, if you want.

This should be fixed in 3ce27ecb3d0ef54fcac4bdf0ca7a81d45af79c45, can you verify?

Yes, error message shows up now.

Alright then, I'm closing this.


Really laggy gameplay

I just compiled a newer version of this engine and it's really laggy every time I start a song.
(there is a link cuz file size is too big)
https://youtu.be/c724AHvVRbE

hmmm, I’ve not encountered this issue before, so it might be a problem with your pc

rip


Updated validations.py python script.

Updated validations.py python script.
Fixed the behavior of validate_user funcation in validations.py
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

Updated validations.py python script


frontend Notifications/List.tsx: Fix and reformat styles

Enhancements to Notifications List View for Better UX
Solves issue #1667
Description
This PR focuses on improving the user experience of the Notifications List View. It involves utilizing the available horizontal space more effectively to display a larger portion of the notification message. Additionally, the cluster name in each notification will now be a clickable link, redirecting users directly to the associated cluster.
Changes

[ ] Expanded the message display area in the Notifications List View to make better use of horizontal space.
[ ] Converted cluster names in notifications into hyperlinks that lead to the respective clusters.

Verification

[ ] Ensured that more of the notification message is visible in the expanded view.
[ ] Tested the functionality of the new cluster links to confirm they correctly redirect to the appropriate cluster.

Screenshots
WIP
To do

[x] Create more space for messages to be read
[x] Make messages columns responsive and show more info
[x] Create a link off the button next to notification messages

Dev notes

This push is a solid base, I have tweaked the vw for each size to not clip or run into other columns (ignore the colors)

I am using the CellProps and a custom style map to put in custom style changes to achieve this

Currently using useEffect to rerender when any screen size changes, when it changes it grabs a new view size for the message column

The comps are responsive
Large

Medium

Small

Dev note

[x] Added context links to the notifications view


Allow dismissing a disconnected pipe w/

attach
do stuff
ctrl-d to disconnect.
move mouse over to close button and click close

Would be nicer to be able to dismiss disconnected pipe w/ a whack to esc

Wouldn't it be better (from the usability point of view) to implicitly disconnect on ctrl-d ?

Thats a fun thought!
But:

Not every disconnected pipe is a initiated by a ctrl-d. In these other cases we should leave the terminal open w/ an explanation rather than surprisingly closing it.
Possible to hack this but would rather not:
If a pipe is disconnected within 2s of a ctrl-d being sent consider that a cause and effect.

I think a quick and easy tap to  to close the window is a nice compromise for now.


failed to check proof: should have 2 roots

Some litenodes were updated in testnet yesterday and now we get this error on GetBlockShardsInfo():
v.1.8.9
failed to check proof: should have 2 roots

v1.9.0 return timeout
Could you please take a look?

UPD:
v1.9.0 error
failed to parse message: failed to parse field Data of adnl.MessageAnswer, err: failed to parse struct from bytes for interface {}, err: failed to parse field Proof of ton.AllShardsInfo, err: incorrect cells len in field []*cell.Cell

Hi, new liteserver version is not sends shard state proof in GetBlockShardsInfo in second root of proof anymore.
For now you can import lib like that go get -u github.com/xssnick/tonutils-go@41495ebfc0d81266892021ed9ba7ba3d01766c7d, so it will work in both new testnet and old mainnet.
A bit later i will release 1.9.1

Released v1.9.1

Thanks! All good now!


Add 'compression' feature for csv tables

The csv tables exported from RedCap contain large numbers of columns, i.e. in case of multiple choice questions each answer is represented as one column. This is fine for programmatic access of the data, but not ideal for human readability.
Proposal:
Add an option to compress the columns of a RedCap csv table, e.g. by merging all columns of a multiple choice question into one and listing the selected choices as value.

Solved via #35


[DO NOT MERGE][Azure Search] Add swagger specs for data operations

Adding the [DO NOT MERGE] tag since I plan to add a couple commits that cover other data operations.
First commit only covers search and index operations. I will add lookup and suggestions in subsequent commits.
@brjohnstmsft, @Yahnoosh, @natinimni
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[NOT YET] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

@mhko  What is the status of this PR? Do you want this to be reviewed and merged? @brjohnstmsft  has left some comments that have not been addressed yet. Could you address them?

@sarangan12, thanks for checking. Yes, I do plan to address comments from @brjohnstmsft in a couple days. Sorry, some of our live site investigations have been keeping me busy.

@mhko Checking in. Any updates?

@sarangan12 Nate is on leave, so I'll be taking over this PR. Please close it for now and I'll re-submit it when l have time to pick it up.

Automation for azure-sdk-for-js

@sarangan12 Could you please reopen this PR? I am back and would like to continue iterations on this.

Closing. brjohnstmsft will open a new PR.


Improve reconnect capability

Actual behavior
The SDK API client is able to reconnect correctly only under certain circumstances. Usually when the core application quits and starts again.
Other scenarios are not covered properly.

For some reason the blockchain events are subscribed multiple times. That results with multiple index updates with the same data. Aggregated results can become incorrect due to that issue.
Sometimes the indexing process does not response properly, i.e. it gets stuck and disconnects from the gateway as well. The leaves the application in an undefined state and it cannot deliver data to clients. Only manual restarts help.

The consequences of that issue is Lisk Service not able to retrieve any blockchain data (blocks, transactions, etc.).
It is also not possible to login to the wallet when this issue happens.
Expected behavior
The issues described above are mitigated, keeping the process work as designed.
Steps to reproduce

Run the app in Docker against mainnet.
Wait ~24hrs.
The process will run into the undefined state.

Which version(s) does this affect? (Environment, OS, etc...)
0.6.2

This has been resolved with #1009. Closing the issue.


Add documentation for outside release usage

The fin block image can be used outside of a fleet release.
For example for ad hoc or sporadic usage on FIN devices.

@Bucknalla I see.  I wanted to highlight that the block can be used outside of a release. To retrieve ad hoc information from a FIN that is already in production and deployed to the field.
Do you think I should just refine it in the doc or would you keep this out of this repo at all?

@fisehara I'd really like to re-write the block such that it can be used completely independent of balenaCloud. So perhaps that's the direction we move towards, where certain functionality is disabled if not using balenaCloud?
If you're thinking of using this as a utility tool, for example for support tickets, I think that also has merit but I think I would like package in a more user-friendly mechanism. I will be breaking the node modules into standalone packages at some point so this might make more sense when you can use them as node modules.

@Bucknalla In my case it was a user request having a FIN fleet and wanting to check which HW revision each device has. For this I liked the approach of just pulling an image. An image came very useful as it could be executed directly by the balena engine without installing, setting up or anything. Thus an image that has only the ad hoc functionality and the full features when deployed in a balena release would make sense.
I think splitting it down makes sense, but maybe ends up in a more complicated 'putting all modules together'. When having node modules that a user would have to put into a docker image and push to a registry that is accessible by a device it becomes more a development task than a tool.

@Bucknalla as it's just an addtional documentation for the fin block, do you see any blocker on just merging it?

dropping it for inactivity


add the first allocation tests for nio-http2

Motivation:
Allocation tests are good :).
Modifications:
Add two allocations tests and NIO's integration test framework.
Result:
Hopefully fewer allocations in the future. And better ways to check what
allocations are happening.

@swift-nio-bot test this please


DayPickerRangeController Week Header is misaligned with verticalScrolling orientation

react-dates version
e.g<EMAIL_ADDRESS>Describe the bug
Week header appears misaligned in verticalScrolling DayPickerRangeController in Chrome (86.0.4240.198)
Source code (including props configuration)
It is present in the Storybook without changes:
http://airbnb.io/react-dates/?path=/story/daypickerrangecontroller--vertical-scrollable
Screenshots/Gifs

Desktop (please complete the following information):

OS: OS X (10.15.7)
Browser: Chrome
Version: 86.0.4240.198

What exactly appears misaligned in your screenshot?

Sorry if I was unclear. The header with the two-letter abbreviation for day of week is not aligned with the table elements containing day of month. The misalignment is not too bad, and it's clear that, for instance, the 14th of November is a Saturday, but the label is off to the right of the correct column, and its further right than "Su" is above the 8th of November.

ah, thanks, i see it. It looks fine in safari, so we’ll need some special handling for chrome.


Image metadata could come from the gallery.json datafile too

I see there are image metadata both in _data/gallery.json and each gallery/image-one/index.md Mardown file.
With #1, you could also move all metadata in _data/gallery.json and generate image pages without needing any Markdown file at all, using Eleventy's pagination/iteration feature: https://www.11ty.dev/docs/pages-from-data/

I moved all the image metadata into _data/gallery.json and generated image pages with Eleventy's pagination/iteration feature! I'm glad to have gotten rid of those extra Markdown files. Thanks! @nhoizey

You're welcome!


No menu inside Cordova

I'm submitting a...

[x ] Bug report
[ ] Feature request
[ ] Documentation issue or request
Current behavior
When I generate a mobile app with this settings :
{   "generator-ngx-rocket": {     "version": "3.1.1",     "props": {       "appName": "lol",       "target": [         "cordova"       ],       "mobile": [         "android"       ],       "ui": "material",       "layout": "simple",       "auth": false,       "projectName": "lol",       "packageManager": "npm"     }   } }
then inside cordova app I can't see the menu. Its present but it has no style . Inside Chrome works correctly , also with mobile emulation

Environment
"version": "3.1.1"

:tada: This issue has been resolved in version 8.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Randomness if top_rate and/or other_rate too low

Hello,
I'm tuning a LightGBM model using its goss version for a multi-class classification problem. However, I'm running into an issue where I observe significant randomness in replication results if top_rate and/or other_rate hyperparameters are too low, even though I thought I had set all of the seeds. Specifically, I have explicitly set the seed for R's random number generator, as well as the bagging_seed and the feature_fraction_seed.
I do not see any other seeds listed in the documentation for lightgbm, but maybe I am missing something. Do you have any ideas if there is an additional seed which needs to be set?
Thanks!
Environment info
Operating System: UNIX
CPU/GPU model: AWS EC2 r5.24xlarge
C++/Python/R version: rstudio36
LightGBM version or commit hash: 2.2.3

Thanks for using LightGBM and for reporting this issue.

Can you please share the code you are running, ideally a reproducible example that shoows the issue?
Can you please share the output of this R command --> sessionInfo().
How did you install LightGBM's R package? Version 2.2.3 is more than a year old, I'm wondering if you can update to a more recent version.

@christopher-rojas
As you run the old version, the bagging will be affected by num_thread.
you may need to fix num_thread and try again.


Blank screen

I was playing around in the playgroud and it seems that when I paste the following, the page goes blank. This might happen after a second, or while switching tabs.
  <form>

      Name
      <input
        type="text"
        id="name"
        name="name"
        formControlName="name"
        required
      />

  </form>

In the console, I can see this error
react-dom.production.min.js:209 TypeError: t.querySelector is not a function
    at a (queryAdvise.js:15)
    at l (queryAdvise.js:43)
    at h (Preview.js:27)
    at ua (react-dom.production.min.js:153)
    at Xa (react-dom.production.min.js:175)
    at Ro (react-dom.production.min.js:263)
    at Lu (react-dom.production.min.js:246)
    at Ru (react-dom.production.min.js:246)
    at Cu (react-dom.production.min.js:239)
    at react-dom.production.min.js:123

I forgot that we have a shareable link.
I tried to reproduce it, but I wasn't able to.
Going to close this issue for now.


dataGen.py with OpenFOAM version 6.0 error : keyword freestreamValue is undefined in dictionary

Hi, I try to launch dataGen.py with OpenFOAM version 6.0 and the following error occurs :
--> FOAM FATAL IO ERROR:
keyword freestreamValue is undefined in dictionary "/home/pierre/EF/DataSet/data/OpenFOAM/0/p.boundaryField.inlet"

file: /home/pierre/EF/DataSet/data/OpenFOAM/0/p.boundaryField.inlet from line 25 to line 25.

    From function const Foam::entry& Foam::dictionary::lookupEntry(const Foam::word&, bool, bool) const
    in file db/dictionary/dictionary.C at line 566.

FOAM exiting

Traceback (most recent call last):
  File "./dataGen.py", line 172, in <module>
    outputProcessing(basename, fsX, fsY, imageIndex=n)
  File "./dataGen.py", line 93, in outputProcessing
    ar = np.loadtxt(pfile)
  File "/home/pierre/.conda/envs/PyTorch-learning/lib/python3.7/site-packages/numpy/lib/npyio.py", line 955, in loadtxt
    fh = np.lib._datasource.open(fname, 'rt', encoding=encoding)
  File "/home/pierre/.conda/envs/PyTorch-learning/lib/python3.7/site-packages/numpy/lib/_datasource.py", line 266, in open
    return ds.open(path, mode, encoding=encoding, newline=newline)
  File "/home/pierre/.conda/envs/PyTorch-learning/lib/python3.7/site-packages/numpy/lib/_datasource.py", line 624, in open
    raise IOError("%s not found." % path)

It seems that a freestream value is required for OpenFOAM simulations.

Hi, yes - sorry, at the moment, the code only supports OpenFoam5. With that version it should work!

Hi, just add the line below each "type freestreamPressure" in the file "OpenFOAM/0/p"
freestreamValue uniform ;

it doesn't recognize neither type freestreamPressure :( can you give me an example, please?


Hope to support Android

Hope to support Android

https://github.com/c-smile/sciter-sdk/tree/master/demos.android
https://sciter.com/windowless-sciter-android-and-iot-devices/
https://sciter.com/sciter-lite-is-published/
As for the Rust - I, personally, haven't tried it. But looks possible.


Can't run on Xcode 12.4 iOS 14.4 and Swift Version 5.3.2

Hi,
I installed "react-native-exponea-sdk": "^0.5.0"
on react-native: 0.61.5 project but when run on Xcode 12.4 I see some Swift Compiler Errors

Cannot convert value of type 'T??' to specified type 'NSDictionary?' and Generic parameter 'T' could not be inferred - in parsePushNotificationTracking func in ConfigurationParser.swift
 if let iosDictionary: NSDictionary = try? dictionary.getOptionalSafely(property: "ios") {          appGroup = try iosDictionary.getOptionalSafely(property: "appGroup") ?? appGroup      }
also here
        if let iosDictionary: NSDictionary = try? dictionary.getOptionalSafely(property: "ios") {          requirePushAuthorization = try iosDictionary.getOptionalSafely(property: "requirePushAuthorization") ?? true      }

'ExponeaAppDelegate' has different definitions in different modules; first difference is definition in module 'react_native_exponea_sdk' found super class with type 'UIResponder' - in ExponeaAppDelegate.h - fixed by adding  libExponeaSDK.a in Link Binary manually

React-native info:
System:
OS: macOS 11.1
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Memory: 72.05 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 12.16.2 - /usr/local/bin/node
Yarn: 1.22.10 - /usr/local/bin/yarn
npm: 6.14.4 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 14.4, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
IDEs:
Android Studio: 4.1 AI-201.87<IP_ADDRESS>53283
Xcode: 12.4/12D4e - /usr/bin/xcodebuild
npmPackages:
@react-native-community/cli: ^3.0.4 => 3.2.1
react: 16.9.0 => 16.9.0
react-native: 0.61.5 => 0.61.5

I just added libExponeaSDK.a in Link Binary manually.
This fixed the second error 'ExponeaAppDelegate' has different definitions...'
Just to Notice, first I run pod install to auto-link.

also to mention and swift version: 5.3.2

Hi @nikolovnikos,
thanks for the report! Sorry it took until now but this has been quite peculiar issue. I believe this could help you.
You actually managed to find 2 separate issues so let's go step by step.

'ExponeaAppDelegate' has different definitions in different modules; first difference is definition in module 'react_native_exponea_sdk' found super class with type 'UIResponder' - in ExponeaAppDelegate.h

This is the easier one. Based on our testing, it happens only with completely empty Derived Data folder (the folder where iOS stores all build-related files, .app folders, compiled files, etc.). After it fails it should be sufficient to just build the iOS project second time and the build succeeds (we tested it both by directly building in Xcode and also via yarn run ios in terminal). This is happening due to the same name of the app delegate class in both Exponea React Native SDK and the underlying Exponea iOS SDKs. We will change the name in future release of the Exponea React Native SDK. For now, the workaround is to just rebuild the iOS project second time if this error shows up until then.

Cannot convert value of type 'T??' to specified type 'NSDictionary?' and Generic parameter 'T' could not be inferred

This one was trickier. I believe this is just one of many issues that occurred when building the project. First of all, we were able to reproduce it on React Native 0.61.5 but it didn't occur in freshly created empty project based on newer React Native 0.63.2. We noticed in the build log the following warning:
ld: warning: Could not find or use auto-linked library 'swiftFoundation'
ld: warning: Could not find or use auto-linked library 'swiftCompatibility51'
ld: warning: Could not find or use auto-linked library 'swiftMetal'
ld: warning: Could not find or use auto-linked library 'swiftDarwin'
ld: warning: Could not find or use auto-linked library 'swiftCloudKit'
ld: warning: Could not find or use auto-linked library 'swiftUIKit'
ld: warning: Could not find or use auto-linked library 'swiftCoreData'
ld: warning: Could not find or use auto-linked library 'swiftCoreFoundation'
ld: warning: Could not find or use auto-linked library 'swiftCompatibility50'
ld: warning: Could not find or use auto-linked library 'swiftQuartzCore'
ld: warning: Could not find or use auto-linked library 'swiftCore'
ld: warning: Could not find or use auto-linked library 'swiftCoreGraphics'
ld: warning: Could not find or use auto-linked library 'swiftObjectiveC'
ld: warning: Could not find or use auto-linked library 'swiftDispatch'
ld: warning: Could not find or use auto-linked library 'swiftSwiftOnoneSupport'
ld: warning: Could not find or use auto-linked library 'swiftCoreImage'
ld: warning: Could not find or use auto-linked library 'swiftCompatibilityDynamicReplacements'
ld: warning: Could not find or use auto-linked library 'swiftCoreLocation'

It seems that Swift standard libraries are missing and that's why the Exponea SDK's Swift files produce various errors. Solution here is to add an empty Swift file into <project root dir>/ios, e.g. File.swift with empty content:
import Foundation

This needs to be done in Xcode (File -> New -> File -> Swift File). When Xcode asks you if you want to create bridging headers, it's not necessary to do so.
Clean the project, rebuild it and it should succeed. Please let me know if this worked.

Hi,
Thank you for the answer.

The first issue is no longer reproducible. ("ExponeaAppDelegate")

The second one still persists. I created an empty file (File -> New -> File -> Swift File) File.swift (with only import Foundation). Then clean the project and run through Xcode also npm run ios or yarn run ios. Same errors in ConfigurationParser.swift

Hi,
Could you please try deleting the whole content of Derived Data folder for your app (not just cleaning the project) and then rebuild via Xcode?
If that doesn't help, could you please post here the whole output from the building process (as a file, preferrably)?
Thanks

Hi,
I deleted the Derived Data folder for my project and then rebuild it. Same errors.
Here is a doc with exported Issue Navigator from Xcode: https://docs.google.com/document/d/18HrhnSc3AMC5rYkpF-HTgfm1GpCuCpO7UeWq1PAcTn4/edit?usp=sharing

This looks like a problem with the nested optional returned by try? dictionary.getOptionalSafely(property: "ios").
Unfortunately, I'm not able to reproduce this (RN 0.61.5, Swift 5.3.2, Xcode 12.4), clean project with installed SDK compiles correctly.
Could you please edit ConfigurationParser.swift in Development Pods and change try? to try on lines 68 and 81?

This works. Now the project is building with no swift errors.

Glad to hear that. Do you think you could try it on a clean project?
npx react-native init AwesomeProject --version 0.61.5
cd AwesomeProject
yarn add react-native-exponea-sdk
cd ios
pod install
Then try to build it. First time it should fail on the AppDelegate error and you may also need to add the empty swift file as I proposed earlier. After these steps the project compiles fine on my machine. Could you please verify this?

@nikolovnikos What build system do you use? (In Xcode it's File -> Workspace Settings)
I used the new build system:

@nikolovnikos What build system do you use? (In Xcode it's File -> Workspace Settings)
I used the new build system:

Legacy Build System, because of other SDK

Glad to hear that. Do you think you could try it on a clean project?
npx react-native init AwesomeProject --version 0.61.5
cd AwesomeProject
yarn add react-native-exponea-sdk
cd ios
pod install
Then try to build it. First time it should fail on the AppDelegate error and you may also need to add the empty swift file as I proposed earlier. After these steps the project compiles fine on my machine. Could you please verify this?

will try it

I was able to build it also after switching to legacy build system, so that's not it.

So I created a new RN project with 0.61.5. Where I have to add not only platform :ios, '10.3' but also ENV['SWIFT_VERSION'] = '5' at the top of the pod file. This fix errors in ConfigurationParser.swift file. No need for manual editing the code.

Thanks, we'll update the documentation to mention this.
I added SWIFT_VERSION = 5 to User Defined section in Xcode as without it, the pod didn't event want to install.
Thanks for your help!

@nikolovnikos One question. In your project, did you have the SWIFT_VERSION value set to a different value, or you didn't have it there at all?

My SWIFT_VERSION in Pods -> Build Settings is 5.0

I can confirm it - Without ENV['SWIFT_VERSION'] = '5' you can't install pods on RN 0.61.5. With this in the Podfile, everything is working with no errors - building and running

@nikolovnikos thank you.
We noticed one thing in the log file you sent earlier:
Conversion to Swift 5 is available
That indicates that you might have been using an older version of Swift which would explain the compilation errors and why we weren't able to reproduce this on a clean project.

Yes, I migrated to SWIFT 5.0 now all good.


Why you include appcompat as dependency

I removed appcompat as dependency and every works fine..why you include it?

It's only included in the example module, and I'm pretty sure that's not
included in the library from Maven Central.
On Mar 17, 2015 9:11 AM, "Colymore"<EMAIL_ADDRESS>wrote:

I removed appcompat as dependency and every works fine..why you include it?
—
Reply to this email directly or view it on GitHub
https://github.com/rahatarmanahmed/CircularProgressView/issues/3.

Fixed in 4faef9e878790b12a634803d93294b8c24e55e86


Consider adding a history api fallback out of the box

webpack-server doesn't ship with a history api fallback out of the box 😢.
Nearly everyone of our projects is going to need this so I think we should contemplate adding it in right out of the box. We will need to follow the example here: https://github.com/webpack-contrib/webpack-serve/blob/master/docs/addons/history-fallback.config.js

Yeah thats all good.
This issue is to make our dev server return the index.html for deep links.
They used to support it in webpack-dev-server https://webpack.js.org/configuration/dev-server/#devserver-historyapifallback
But now with webpack-serve you need to install it as a plugin https://github.com/bripkens/connect-history-api-fallback

I feel that this would be making some app architecture assumptions that are
too specific for this package. Not all projects built with this package are
single page apps with serverside rendering (or even single page apps where
the server serves index.html at other paths). Would it be ok to install the
plugin at the project level?
On Thu., 31 May 2018, 12:52 pm Jenna Salau<EMAIL_ADDRESS>wrote:

Yeah thats all good.
This issue is to make our dev server return the index.html for deep links.
They used to support it in webpack-dev-server
https://webpack.js.org/configuration/dev-server/#devserver-historyapifallback
But now with webpack-serve you need to install it as a plugin
https://github.com/bripkens/connect-history-api-fallback
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DeloitteDigitalAPAC/webpack-config/issues/30#issuecomment-393492911,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAfVU0Qp17sZzb-66i_lv6d3BbPM58Jwks5t38ttgaJpZM4UUsyy
.

I wasn't planning on blindly turning it on.
Just bringing back a feature like historyApiFallback: true and keeping a single configuration package since we do do many SPA's.
Perhaps, ill just add a guide on how to install it and set it up.

Hey Jenna, sorry I must have misunderstood. I think it would be fine to add
it if it's not enabled by default.
On Fri., 1 Jun. 2018, 1:00 am Jenna Salau<EMAIL_ADDRESS>wrote:

I wasn't planning on blindly turning it on.
Just bringing back a feature like historyApiFallback: true and keeping a
single configuration package since we do do many SPA's.
Perhaps, ill just add a guide on how to install it and set it up.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DeloitteDigitalAPAC/webpack-config/issues/30#issuecomment-393709050,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAfVU_sv3BsqFVXh4iQMuzL9EF8F8kiFks5t4HYKgaJpZM4UUsyy
.


Add note about docstring to 'How do I reduce the size of the binary modules' faq

Related to #4425

Looks like a reasonable suggestion to me. I don't suspect it's a huge saving for most people, but it's easy to do.

Some modules can be docstring heavy - e.g. see https://github.com/scipy/scipy/blob/e630ecd9cff5c1b5ba0e4e88de17068bfbaa4f51/scipy/spatial/transform/_rotation.pyx

if it accumulates I suppose it can have some impact but to be honest I did not do any kind of investigation so I don't have any numbers.

But what I see as bigger problem is that Option cannot be controlled easilly per file as directives (Maybe I am wrong). Hence, there can be a problem for projects which has part of .pyx files as internal and part of API which should hold their docstrings.
Anyway, I wll merge this PR if there is no opposition or improvements in grammar/wording.


[Activation] Fix BlockNotification class and BlockNotificationTest

This was broken during the Activation work.
It needs to be fixed.

issue resolved

The light wallet is currently broken because these classes have been commented out during the activation work.
This issue can be resolved when either these classes are adapted to the new way of doing things or the light node will take the shape of a pruned node instead.

Migrated to: https://dev.azure.com/Stratisplatformuk/StratisBitcoinFullNode/_workitems/edit/3055


Allow dynamic focus target

Describe the problem

Some visits don't need to reset focus to the main element, e.g. fragment visits like filter UIs
Ideally, the custom focus target could be overwritten per visit
The fragment plugin and users can then just set visit.focus = '#list' or similar

Describe the proposed solution

New key in visit object: visit.focus — or maybe nested/grouped somewhere?
New hook content:focus that can be replaced

How important is this feature to you?

[ ] Nice to have
[x] Would make my life a lot easier
[ ] I cannot use swup without it

Very nice idea!


JBPM-5029: Make it clear in Jobs perspective that "Job name" will be …

…used as a "businessKey" for that executor job

+1 to merge.


Prefer "minikube delete" over "rm -rf ~/.minikube"

Using rm -Rf ~/.minikube is not necessary in modern (v1.x) versions of minikube, and is highly discouraged as may can leave dangling VM's running, which minikube will fail to recover from.
Example issue: https://github.com/kubernetes/minikube/issues/4377

/lgtm


152 - Add .npmrc explanation in the README

What this PR does / why we need it:
This PR introduces an explanation in the README file on how to add an .npmrc file to the project and how to create a GitHub token. The change is necessary to facilitate the installation of the @iqss/dataverse-client-javascript package from the GitHub registry. This package is crucial for establishing a connection with the Dataverse API.
Which issue(s) this PR closes:

Closes #152

Special notes for your reviewer:
Please ensure that the explanation is clear enough for any user to understand the steps.
Suggestions on how to test this:

Follow the steps outlined in the README.
Verify that the application runs as expected.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No
Is there a release notes update needed for this change?:
No
Additional documentation:

coverage: 98.426%. remained the same
when pulling c45af148a31764dea6088d08b5113cb2024e8189 on feature/152-readme-should-explain-authentication-token
into 0576f50832211decf4cd49db32037fc659b35426 on develop.


Grouped column chart with one of the columns stacked

Hello!
Sorry to bother again with one more question :)
This time, I would like to check if having a grouped column chart with one (or more) of the columns being a stacked one is supported...
I know we can have either grouped or stacked columns, but I didn't find a way to have both in the same chart.
Is this something feasible?

Hello! I’m afraid this isn’t supported out of the box. You’d have to extend ColumnChart and make the required modifications.

I’m afraid that, at the moment, we don’t plan on adding such a feature.


Frogs clip through the floor and into the endless void

When placing a portal at the bottom of a world, a froggo can find itself in the endless void of eternal nothingness, acompanied only by pain, suffering and anguish - never to be seen or heard from again. The frog undergoes complete and utter inter-thermal damnation within the eternal void and its suffering will not be relieved even by means of a merciful death.
Save froggo from clipping into the barrier that seperates the comfortable life the froggo knows, loves and cherishes, from the everlasting void of nothingness and despair - for they are only to be greeted with eternal torment.

tl;dr blocks can touch the floor on the otherside of a portal, but a frog will keep going

Happy Froggo :D

Awesome! Also there are loooads of URL parameters that you can use to try the different experiments out. Such as https://www.timepond.cool/?experiment=gfling&menu=purple&portal=pastnow

can there be a list showing each type of experiment?

Hey, I added some presets to the readme. Here are some you can try:

Simple
Right
Fling
Fall
Bump

Teleport
Teleport + Simple
Teleport + Right
Teleport + Fling
Teleport + Fall
Teleport + Bump

Future
Future + Simple
Future + Right
Future + Fling
Future + Fall
Future + Bump

Past
Past + Simple
Past + Right
Past + Fling
Past + Fall
Past + Bump

Rewrite
Rewrite + Simple
Rewrite + Right
Rewrite + Fling
Rewrite + Fall
Rewrite + Bump

Branch
Branch + Simple
Branch + Right
Branch + Fling
Branch + Fall
Branch + Bump

I left some of the other presets in a broken state, (eg: Nexus) so I'll fix those at some point and add them too!


Add prototype THREDDS-to-tarfile script

Add prototype THREDDS-to-tarfile script to allow downloading ODC dataset yaml files from THREDDS endpoints and prepare them for indexing via index-from-tar.

It's possible to write directly to tar without going to FS first:
https://github.com/opendatacube/dea-proto/blob/master/dea/io/tar.py#L63

Now streams all files to memory, then outputs to tar (I found i couldn't stream directly to tar without corrupting the archive).
Reduced the number of required parameters as it should be possible to infer the THREDDS HTTP file server endpoint.

Yes tar.write is not thread-safe, options are (1) locking in app code (2) decoupling Read/Write threads via Queue (3) load all concurrently then write all serially
(2) is more likely to be performant and allows for cleaner "back-pressure" control.
Given that we now have three different small file datasources that all go to tar it's worth making tar output into a class. I might take out logic in s3-to-tar into a  thread-safe class at some point. For now this solution will do, so long as you are not reading too much at a time and it's a "prototype" just to get some data into db to play with.


Generate efficient FSMs for slow modules

When a module does not have an II=1, we should be able to generate smaller, counter-based FSMs. This is particularly important if we're interacting with modules of high-latency since we may need to a long time and the generated pipelined FSM can get quite large.

Here is a sketch for slow FSMs that I came up with. The high-level idea is that for a module that can retrigger every $d$ cycles and has $n$ states, we can instantiate $n/d$ counters that count out to $d$ before asserting done and chaining them together. Overall, this uses $n/d \times log(d)$ bits instead of $n$ bits.
The overhead for this process is that each counter gets an $n/d$-bit adder so it's not obvious that we should always do this but it is also hard to come up with an obvious heuristic to decide when to allocate it.
In general, the cases that actually matter are probably completely iterative modules where delay = latency or fully pipelined modules.
/// A module that counts up to 3 and resets.
module count_3 (
    input wire clk,
    input wire reset,
    input logic go,

    output logic done,
    output wire s0,
    output wire s1,
    output wire s2
);
    logic [1:0] count;

    assign s0 = go & count == 0;
    assign s1 = count == 1;
    assign s2 = count == 2;

    always @(posedge clk) begin
        if (reset) begin
            count <= 0;
        end else if (count == 2) begin
            count <= 0;
        end else if (go || count != 0) begin
            count <= count + 1;
        end
    end

    always @(posedge clk) begin
        if (count == 2) begin
            done <= 1;
        end else begin
            done <= 0;
        end
    end
endmodule

/// An FSM with 12 states that can only be triggered every 3 clock cycles.
module FSM (
    input wire clk,
    input wire reset,
    input logic go,

    output wire done,
    // Expose all 12 states
    output wire s0,
    output wire s1,
    output wire s2,
    output wire s3,
    output wire s4,
    output wire s5,
    output wire s6,
    output wire s7,
    output wire s8,
    output wire s9,
    output wire s10,
    output wire s11
);

wire c0_done, c1_done, c2_done, c3_done;

// FSM is done when the last counter is done.
assign done = c3_done;

// Instantiate 4 count_3 modules.
count_3 c0 (
    .clk(clk),
    .reset(reset),
    .go(go),
    .done(c0_done),
    .s0(s0),
    .s1(s1),
    .s2(s2)
);
count_3 c1 (
    .clk(clk),
    .reset(reset),
    .go(c0_done),
    .done(c1_done),
    .s0(s3),
    .s1(s4),
    .s2(s5)
);

count_3 c2 (
    .clk(clk),
    .reset(reset),
    .go(c1_done),
    .done(c2_done),
    .s0(s6),
    .s1(s7),
    .s2(s8)
);
count_3 c3 (
    .clk(clk),
    .reset(reset),
    .go(c2_done),
    .done(c3_done),
    .s0(s9),
    .s1(s10),
    .s2(s11)
);
endmodule

It might be cool to have "unchecked" modules in Filament because the above can easily be implemented in Filament but cannot be type checked.


Remove wildcard imports

I removed (I hope so) all wildcard imports I found.
For me on my pc this improved the build time:

for egui -5s
for eframe -12s

[x] I have followed the instructions in the PR template

Thanks!


Added additional parameters to mixing multiple LoRAs through SVD, added ability to mix LoRAs through concatenation

Hi!
This PR introduces the following changes:

Fixes an error during SVD decomposition when Conv2d layers are present in LoRAs (caused by the lack of fan_in_fan_out in Conv2d LoRALayer)
https://github.com/huggingface/peft/blob/412d7bc985f52225be8d9a7abc6a0cf2d71b2415/src/peft/tuners/lora.py#L659-L661
Adds an ability to combine multiple adapters with concatenation of their A and B matrices. It's super fast compared to SVD decomposition (e.g. on my GPU SVD takes about 30+secs, and concatenation takes less than 1sec), and is also mathematically equivalent to successive applying of multple LoRAs. When using this method, one should be aware that mixing LoRAs may result in target LoRA being too large, so in this case SVD decomposition should be preferred.
Adds an ability to provide the desired target SVD rank when performing SVD decomposition. So, one can find a balance between inference speed and loss in precision.
Adds an ability to provide additional parameters to torch.linalg.svd like driver to use during decomposition.
Adds an ability to disable clamping matrices values after SVD decomposition. From my experiments I see that clamping this values for Stable Diffusion LoRAs also influences results compared to Automatic1111 webui.

You can see my previous results here #643.
webui output:

SVD decomposed mixture of 2 LoRAs, clamped:

SVD decomposed mixture of 2 LoRAs, unclamped:

@pacman100 your review is kindly appreciated.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Adding (crude) pretty print functionality

I added a pretty: true option to encode() that will indent map and list values.
The indentation level is currently fixed to 2 spaces. Can ofc be a customizable option as well,
just want to know if this is even something that will be considered.
Also pretty sure that someone will know how to make this go faster.

Thanks for this @tbug. This is on the roadmap. Give me this weekend to evaluate the PR and make it faster.

Roger.
Feel free to delegate work o/

You can now control what chars are used for indentation.
Default is still two spaces, but if you want four, you can now:
Poison.encode(structure, pretty: %Poison.Style{indent: '    '})

@tbug Thanks for taking a stab at this. I've added what I believe to be a faster implementation to master.
You can use Poison.encode!(value, pretty: true) for pretty printing. Control indentation with the indent option or initial offset with the offset option: Poison.encode!(data, pretty: true, indent: 4, offset: 4).

Looks great.
Happy so see this hitting Poison :+1:


Add support for codecov

Codecov Report

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

@@            Coverage Diff             @@
##             develop      #37   +/-   ##
==========================================
  Coverage           ?   96.13%
==========================================
  Files              ?        6
  Lines              ?      207
  Branches           ?        0
==========================================
  Hits               ?      199
  Misses             ?        8
  Partials           ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d8311d...707d2c3. Read the comment docs.


Re-use Envision window over multiple experiments

Is your feature request related to a problem? Please describe.
When running experiments with Envision enabled, multiple tabs are opened when a single tab could be re-used
Describe the solution you'd like
Check to see if the envision tab is in use and re-use the instance instead of creating a new one
Describe alternatives you've considered
Alternatively try to automatically close the tabs after an experiment has concluded
Additional context
Add any other context or screenshots about the feature request here.

After investigation, webbrowser cannot open url's in the same tab. It also cannot close tabs or read url's which is what I was going to do to prevent opening new envision tabs. Selenium looks like it is able to accomplish this however I don't think it is worth to add the dependency for this quality of life enhancement.

The same tab can be reused by running scl envision start & first. A page refresh may be needed for each new run.


zstd-sys allows a vulnerable version of memoffset to be included in the build

> cargo generate-lockfile -Z minimal-versions && cargo audit
    Updating crates.io index
    Fetching advisory database from `https://github.com/RustSec/advisory-db.git`
      Loaded 59 security advisories (from /home/wim/.cargo/advisory-db)
    Scanning Cargo.lock for vulnerabilities (86 crate dependencies)
error: Vulnerable crates found!

ID:       RUSTSEC-2019-0011
Crate:    memoffset
Version:  0.2.0
Date:     2019-07-16
URL:      https://rustsec.org/advisories/RUSTSEC-2019-0011
Title:    Flaw in offset_of and span_of causes SIGILL, drops uninitialized memory of arbitrary type on panic in client code
Solution:  upgrade to >= 0.5.0
Dependency tree:
memoffset 0.2.0
└── crossbeam-epoch 0.3.0
    └── crossbeam-deque 0.2.0
        └── rayon-core 1.4.0
            └── rayon 1.0.0
                └── cc 1.0.28
                    ├── zstd-sys 1.4.13+zstd.1.4.3
                    │   └── zstd-safe 1.4.13+zstd.1.4.3
                    │       └── zstd 0.4.28+zstd.1.4.3
                    └── libloading 0.5.0
                        └── clang-sys 0.28.0
                            └── bindgen 0.51.0
                                └── zstd-sys 1.4.13+zstd.1.4.3

Upgrading the cc dependency to 1.0.45 drops rayon as a dependency and avoids this issue.

Thanks for the report!
Bumped the version of cc and made a new release.


Diff/patch may produce non-identical JSON

Is it expected that the reversing a diff/patch may produce non-binary equivalent JSON?
ex.
dozadiff a b > diff
dozapatch a diff > b2
dozadiff b b2 # [] empty
diff b b2 # non-empty

The file in question is quite large, it appears like it is effectively the same, but not exactly the same.
Does this affect composition of patches?
dozadiff a b > abdiff
dozadiff b c > bcdiff
dozapatch a abdiff > b2
dozapatch b2 bcdiff > c2
dozadiff c c2 # [] empty
diff c c2 # non-empty

Can this potentially cause problems, or as long as dozadiff thinks there is no diff then the patch should still work?

The tools dozadiff and dozapatch will not normalize the patch nor the outputted JSON. They should be semantically equivalent though. E.g. a could be{"name":"Bob","karma":100} and b could be {"karma":100,"name":"Bob"}. As long as dozadiff returns an empty array then all should be good.


version_info.major >= 3 and version_info.minor >= 5

This is used in three places in scipy, and I've seen this sort of thing in other repos too:
if sys.version_info.major >= 3 and sys.version_info.minor >= 5:
    from math import gcd
else:
    from fractions import gcd

The if is incorrectly False for 4.0 - 4.4, 5.0 - 5.4, etc.
To fix:
-if sys.version_info.major >= 3 and sys.version_info.minor >= 5:
+if sys.version_info >= (3, 5):
     from math import gcd
 else:
     from fractions import gcd

Can this easily be detected?

Yep, I'll have a go!


Should _get_line_syntax_highlighting(line) in SyntaxHighlighter be a const function while get_text_edit() is not?

Godot version
4.0
System information
Linux Mint
Issue description
_get_line_syntax_highlighting(line) is declared as a const function. But when using GDExtension, it needs to call get_text_edit() which is not a const function.
Link with: https://github.com/godotengine/godot-cpp/issues/1084
Steps to reproduce
Try to use get_text_edit() from _get_line_syntax_highlighting(line) in a GDExtension.
Minimal reproduction project
Not needed.

SyntaxHighlighter::get_text_edit() can also be made const, it doesn't have any side-effects. We don't have a good consistency when it comes to using const, so feel free to correct such oversights with a PR :)


[Feature Request] - Show emoji's name on hover

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

Problem:
If someone uses an emoji I don't know the name of, I have no way of using it too other than manually scrolling through the whole list.
Solution:
Show an emoji's name on hover.
Context/Examples/Screenshots/Notes:
Platforms:

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

[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Expensify/Expensify Issue URL:
Issue reported by: @iwiznia
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1673375710575859
View all open jobs on GitHub

This issue has not been updated in over 15 days.  eroding to Monthly issue.
P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

@kavimuru, this Monthly task hasn't been acted upon in 6 weeks; closing.
If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.


Adding support for query parameter in Swagger UI

Why make this change?

Closes #1572

This is necessary to allow REST calls for GET and GET ALL support query parameters.

What is this change?

Updating the OpenApi document to include the supported query parameters
Separate Logic for StoredProcedures and Table/Views.
$filter,$orderBy, $first, and $after will be available in swagger UI for GET/GET_ALL REST requests in Table/Views.
StoredProcedures will support input parameters for all GET requests, but for others request Body should be used.

How was this tested?

[X] Integration Tests
[X] Manual Tests

Screenshots
GET request will have all the query parameters

POST/PUT/PATCH/DELETE will not have the above query parameters

Input parameters for Stored procedure, depending on the stored procedure definition

/azp run


Exec format error on running Node.js app using Dockerfile + Raspberry Pi 3 + balenaCloud

Summary of Problem
(Please answer all 3)

What are you trying to do?

Use node-serialport as a dependency in my Node.js application for a Raspberry Pi 3.
My Dockerfile looks like this:
# Base image for node 14.15.1 on raspberry pi 3
FROM balenalib/raspberrypi3-alpine-node:14.15.1-run

# Install packages
RUN install_packages make gcc g++ python2 linux-headers udev

# Defines our working directory in container
WORKDIR /usr/src/app

# Copies the package.json first for better cache on later pushes
COPY package.json ./package.json

# This install npm dependencies on the balena build server,
# making sure to clean up the artifacts it creates in order to reduce the image size
RUN JOBS=MAX npm install --production --unsafe-perm --build-from-source \
  && npm prune --production \
  && npm cache clean --force \
  && rm -rf /tmp/*

# This will copy all files in our root to the working  directory in the container
COPY . ./

# Enable udevd so that plugged dynamic hardware devices show up in our container
ENV UDEV=1

# Run this when container starts up on the device
CMD ["npm", "start"]

What happens?

The Node.js app starts crashing as soon as npm start starts the application.
uncaughtException: Error loading shared library<EMAIL_ADDRESS>Exec format error
    Error: Error loading shared library<EMAIL_ADDRESS>Exec format error
        at Object.Module._extensions..node (internal/modules/cjs/loader.js:1122:18)
        at Module.load (internal/modules/cjs/loader.js:928:32)
        at Function.Module._load (internal/modules/cjs/loader.js:769:14)
        at Module.require (internal/modules/cjs/loader.js:952:19)
        at require (internal/modules/cjs/helpers.js:88:18)
        at bindings (/usr/src/app/node_modules/bindings/bindings.js:112:48)
        at Object.<anonymous> (/usr/src/app/node_modules/@serialport/bindings/lib/linux.js:2:36)
        at Module._compile (internal/modules/cjs/loader.js:1063:30)
        at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
        at Module.load (internal/modules/cjs/loader.js:928:32)

Versions, Operating System and Hardware

SerialPort@? 9.0.4
Node.js v? 14.15.1
Windows? Linux? Mac? Dockerfile using Alpine Node 14.15.1
Hardware and chipset? (Prolific/FTDI/Other) Raspberry Pi 3

It seems to be incompatible binary. I guess that the image was built on the desktop but you are trying to use it on the pi.
The images was built on the pi, however, I suspect some library (dependency) could be built on the different kernel. AFAIK, Alpine provides aarch64, armhf, armv7 dist for the pi.
How about try to build @serialport/bindings inside the docker environment in manually?

It seems to be incompatible binary. I guess that the image was built on the desktop but you are trying to use it on the pi.
The images was built on the pi, however, I suspect some library (dependency) could be built on the different kernel. AFAIK, Alpine provides aarch64, armhf, armv7 dist for the pi.
How about try to build @serialport/bindings inside the docker environment in manually?

Hi @dakshshah96 did rebuilding from source solve your issue?


Prettify output code (CSS, HTML, JS)

Run CSS and HTML (and maybe JS) through code prettying routines.

One could use prettier for this. This will greatly improve code readability and we could expose some prettier config options to the user. However adding prettier (or any other code beautifier) will further increase the worker bundle size and potentially increase loading times. I think it is a fair tradeoff but it remains to be seen by how much these values change.

With parsers for CSS, HTML and JS adding prettier increases the worker bundle size from 274kB to 1.4MB. This is a lot but might be acceptable. Unfortunately for some files (e.g. Tabs Aside css/sidebar.css) will cause prettier to throw errors in production mode but not in debug mode. I don't know why, it probably requires some webpack config tweaks to get it to work properly.


firewalld & flannel

I am looking for help with firewalld rules to allow flannel to work in my environment, sorry if this is the wrong place to post this.
I have flannel setup across 2 RHEL 7 hosts with firewalld enabled as it is a requirement for me.
I can ping a running container across hosts, but I cannot curl the same container; i get a "Failed connect to <IP_ADDRESS>:8080; No route to host"
Watching tcpdump on my main eth I see:
17:13:26.811485 IP <IP_ADDRESS>.46667 > <IP_ADDRESS>.webcache: Flags [S], seq<PHONE_NUMBER>, win 29200, options [mss 1460,sackOK,TS val 4898533 ecr 0,nop,wscale 7], length 0
The curl works once I stop firewalld.
I assume I need to allow the subnet range somehow?
Any help will be appreciated.

@robtaireeo Why this is closed? Have you figured it out yourself? Sorry for the delay. @eyakubovich is on a vacation.

@xiang90 I did not figure it out, but I believe my issue is not flannel, but really libreswan/ipsec, as I am trying to encrypt flannel traffic between the two hosts.

OK. Thank you.


historical orderbook // get trades

I am pretty sure this is not the right place for me to be asking this (sorry) but feel free to point me the way.
I would like two features that are currently not present within the API

fetch historical orderbook data
get trades starting on and after a certain timestamp (instead of before)

From what I saw in the API's code, it simply makes requests to urls.
Is there an open repository that define those urls and what data is retrieved from dydx blockchain?
Any good sources I can read about how to get started on it? I don't really have any experience on it, but I am a quick learner and I don't mind trying things out and pushing for changes.
Thx,

Found any solution yet ??


Lab03/letters

Fixes #
Changes proposed in this PR:
Implementing frequency of letters in ascending order

Please remove the fibonacci series related files.

Please fix broken build and Update branch

Codecov Report

Merging #74 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #74   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          10     11    +1
  Lines         159    175   +16
=====================================
+ Hits          159    175   +16

Impacted Files
Coverage Δ

03_letters/SandhyaSMani/03_letters.go
100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1262ce0...55ebddf. Read the comment docs.

Hi,
There has been  no activity in the last 20 days so I will close this PR.
Feel free to open it again if you have something to add.
Thanks!
Julia


Add data from openpolicedata.com

St. John data is part of our datasets already. It has been moved to openpolicedata so is not currently accessible.
https://www.openpolicedata.com/StJohnIN/
https://www.openpolicedata.com/GriffithIN/
https://www.openpolicedata.com/CedarLakeIN/

Closed in f8a8af0