Maybe in Tomcat?

Pardon for using GitHub, I didn't find any other means :)
https://bz.apache.org/bugzilla/show_bug.cgi?id=65770
Anyway, I'm using your library together with Letsencrypt's certbot.  I'm still waiting for the first renewal...
Regards,
Anders

Nice idea, thanks for letting me know!
I actually never considered pushing my implementaion upstream. I'm convinced there are better ways of implemeting it for someone who is more experienced with tomcat internals.
Still makes a lot of sense to include a feature like this into tomcat core IMO. Newer servers from non-java technological ecosystem often include ACME/LE by default. With Tomcat we're stuck with an anti-intuitive technology from the 90s like JMX to implement cert reloading manually 🤔
Keeping my fingers crossed for your initiative on the Apache BZ 👍
Interesting to read that you use this in conjunction with certbot. My showcase example uses a bash-only cert reloader (dehydrated) https://github.com/schnatterer/letsencrypt-tomcat
Did you automate your setup and would you mind sharing it? Maybe as a gist? I'd gladly add a link to the README as another example.

Nice idea, thanks for letting me know!
I actually never considered pushing my implementaion upstream. I'm convinced there are better ways of implemeting it for someone who is more experienced with tomcat internals.
Still makes a lot of sense to include a feature like this into tomcat core IMO. Newer servers from non-java technological ecosystem often include ACME/LE by default. With Tomcat we're stuck with an anti-intuitive technology from the 90s like JMX to implement cert reloading manually 🤔
Keeping my fingers crossed for your initiative on the Apache BZ 👍
Interesting to read that you use this in conjunction with certbot. My showcase example uses a bash-only cert reloader (dehydrated) https://github.com/schnatterer/letsencrypt-tomcat
Did you automate your setup and would you mind sharing it? Maybe as a gist? I'd gladly add a link to the README as another example.

Hi @schnatterer, you have done a lot more than I!
I only installed https://certbot.eff.org/, put your reloading-connector in Tomcat's lib folder, and updated the Tomcat SSL configuration in server.xml to point to the file created by certbot.  I have a whole bunch of different host names which made certbot a good choice:
  <Service name="Catalina">

    <Connector port="80" protocol="org.apache.coyote.http11.Http11AprProtocol"
               connectionTimeout="20000"
               redirectPort="443" />

    <Connector port="443" protocol="info.schnatterer.tomcat.ReloadingHttp11AprProtocol"
               maxThreads="150" SSLEnabled="true" defaultSSLHostConfigName="webpki.org">
        <SSLHostConfig hostName="webpki.org">
            <Certificate certificateKeyFile="/etc/letsencrypt/live/webpki.org/privkey.pem"
                         certificateFile="/etc/letsencrypt/live/webpki.org/cert.pem"
                         certificateChainFile="/etc/letsencrypt/live/webpki.org/chain.pem"
                         type="RSA" />
        </SSLHostConfig>
    </Connector>

    <Engine name="Catalina" defaultHost="webpki.org">

      <Realm className="org.apache.catalina.realm.LockOutRealm">
        <Realm className="org.apache.catalina.realm.UserDatabaseRealm"
               resourceName="UserDatabase"/>
      </Realm>

      <Host name="webpki.org" appBase="webapps"
            unpackWARs="true" autoDeploy="true">
      </Host>
      <Host name="spaceshop.webpki.org" appBase="spaceshop.webpki"
            unpackWARs="true" autoDeploy="true">
      </Host>
      <Host name="test.webpki.org" appBase="test.webpki"
            unpackWARs="true" autoDeploy="true">
      </Host>
      <Host name="mybank.webpki.org" appBase="mybank.webpki"
            unpackWARs="true" autoDeploy="true">
      </Host>

      <Valve className="org.apache.catalina.valves.AccessLogValve" directory="logs"
             prefix="localhost_access_log" suffix=".txt"
             pattern="%h %l %u %t "%r" %s %b"/>
    </Engine>
  </Service>

This was done very recently so I have yet to see it renew.  I have though verified that reloading works  👍

Cool, thanks for sharing, @cyberphone !


Added bullet section for Powershell scripts

Added new section for PowerShell scripts included with the Security Compliance Toolkit

Scripts
-Baseline-ADImport.ps1
-Baseline-LocalInstall.ps1
-Remove-EPBaselineSettings.ps1
-MapGuidsToGpoNames.ps1

(Ref. #6784 )

(Ref. #7073 )


OCPBUGS-26466: Disable PSA in non-previews 4.16

What
Disable PSA in non-previews. Keep enabled in TechPreviewNoUpgrade and DevPreviewNoUpgrade
Why
We didn't meet the threshold to enable it yet.

PTAL @deads2k

to pass verify, you must modify https://github.com/openshift/api/blob/master/features/features.go#L78 and regenerate.

/jira refresh

/retest-required

/jira refresh

/retest-required

/retest-required

/lgtm
/approve


Create configmaps during deploy

Wdyt about removing the checked in configmaps and just generate and upload in one go?

In general I agree with you. The idea behind generating the configmaps is to end up with a set of manifests that we could hopefully squash into one manifest at some point and make it available as a helm package. As far as I understand it helm doesn't handle creating configmaps directly from files.

Yea, Helm and KPM packages are the eventual goal for this repository.

What about generating them automatically for now until the packaging story comes up (the generation could still happen during package release)? Feel free to close if you don't want to change anything with the current setup.


CalDAV-support

This issue is currently deferred
Due to low priority and current lack of resources, this issue was tentatively put aside.
See also info about deferred issues.

as mentioned on the old google-code-ticket, CalDav-Support would be great (even though it's probably something that won't really happen)
**What new or enhanced feature are you proposing?**
Extending wsgidav with caldav

**What goal would this enhancement help you achieve?**
Allowing enhanced calendar support for a collaborative site i'm working on

Due to low priority and current lack of resources, this issue was tentatively put aside.
See also info about deferred issues.


speedtest vs speedtest-cli

The setup script installs two scripts, speedtest and speedtest-cli. Both do the same. Is there a reason for the two commands?

Fun

So one version will be removed? Great!

Since they do the same as I got it, it is my opinion as well that one of them should be removed.


Perf regression in VSCode extension with bicepconfig.json configuration file

The Bicep 0.4.1 language server lags behind significantly when running on OSX with a bicepconfig.json file.
With dotnet profiling, looks like the issue is that the FileSystem.Watcher class is invoked a lot via the ConfigHelper class, which has known performance issues on non-Windows platforms (https://github.com/dotnet/runtime/issues/42036):

Thanks @miqm for noticing and reporting this!

Temporary fix: remove bicepconfig.json from folder tree.


Atualizando branch principal

Atualizando branch principal, em acordo com a release

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

40 of 49 (81.63%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+54.5%) to 84.483%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/cloudinary/cloudinary.ts
15
24
62.5%

Totals

Change from base Build<PHONE_NUMBER>:
54.5%

Covered Lines:
45

Relevant Lines:
54

💛  - Coveralls


compute world matrix of children

@deltakosh Double check that this looks like the right fix. It seems as though the controller mesh is not part of scene.meshes. Do you expect that the recursive world matrix logic should already exist somewhere else or would you expect scene.meshes contain the controller?

Unfortunately this is not good as: it will compute wm on all nodes even if you have an octree. Also this is part of performance issue we identified (double tree traversal)
Why the mesh.computrWorldMatrix() not enough?

If you can repro the issue in a PG I will make sure to fix it

OK let's do that :)


sentence_dedup.py: "struct.error: ushort format requires 0 <= number <= (0x7fff * 2 + 1)"

  File "/weka/home-griffin/datatrove/src/datatrove/pipeline/dedup/sentence_dedup.py", line 69, in save_hashes
    f.write(struct.pack("<H", hs.sent_id))
    │ │     │      │          │  └ 312748
    │ │     │      │          └ HashSig(hash_value=6388858470426, doc_id=5179808, file_id=None, sent_id=312748)
    │ │     │      └ <built-in function pack>
    │ │     └ <module 'struct' from '/usr/lib/python3.10/struct.py'>
    │ └ <function LocalFileOpener.write at 0x7f020cfcd990>
    └ <fsspec.implementations.local.LocalFileOpener object at 0x7ec05d247d00>

struct.error: ushort format requires 0 <= number <= (0x7fff * 2 + 1)

I'm processing a large dataset (30BN tokens) -- is this a numerical overflow issue?
Is there a quick fix? Thank you!

Indeed it is, you have an overflow on the sentence id. Currently we do not support a config to let you change the datatype for sentence id, so the quickest solution is to find your document with >= 312748 sentences and split it into smaller documents

Thanks - Yes - i just saw that it was on sent.id.  have done the filter and re-running thank you!


Add a new API that tags the result of a query

In gitlab by @MohamedHamouGisaia on Jan 4, 2018, 14:01

In gitlab by @MohamedHamouGisaia on Jan 4, 2018, 14:01
added ~2329926 label

Fixed in #243 PR


Feature Request: 'Detailed' UI view

Would be much more practical to view snippets in a Detailed view as opposed to exclusively Tiles. Also, the addition of columns for each category to sort by in this view to sort by would be great.

Support this


Made some UI improvements in the footer section

Fixes #77.
List of changes made:

Changed the icons of the gitter channel, the mailing list and the email address
Added padding to icons
Changed the font family of the labels to "Open Sans", "Helvetica Neue"
Changed the title from "Contact" to "Contact Us"
Changed the label of the Gitter channel from PSLab Channel to PSLab Gitter Channel
I've made the above-mentioned changes to the Privacy page as well.

Screenshots for the change:

@CloudyPadmal @mariobehling Please review :)

Thanks!


MPI not working with python3.6 in mac

 ~/t/b/b/ppo1   master ±  python run_atari.py                                                    (ppo_cont_check)
Traceback (most recent call last):
  File "run_atari.py", line 57, in <module>
    main()
  File "run_atari.py", line 54, in main
    train(args.env, num_frames=40e6, seed=args.seed)
  File "run_atari.py", line 17, in train
    from baselines.ppo1 import pposgd_simple, cnn_policy
  File "/Users/abhishek/test/baselines/baselines/ppo1/pposgd_simple.py", line 6, in <module>
    from baselines.common.mpi_adam import MpiAdam
  File "/Users/abhishek/test/baselines/baselines/common/mpi_adam.py", line 1, in <module>
    from mpi4py import MPI
ImportError: dlopen(/anaconda/envs/ppo_cont_check/lib/python3.6/site-packages/mpi4py/MPI.cpython-36m-darwin.so, 2): Library not loaded: @rpath/libmpi.1.dylib
  Referenced from: /anaconda/envs/ppo_cont_check/lib/python3.6/site-packages/mpi4py/MPI.cpython-36m-darwin.so
  Reason: image not found
 ~/t/b/b/ppo1   master ±  conda list | grep mpi                                                  (ppo_cont_check)
mpi4py                    2.0.0                     <pip>

Any ideas?

+1 to this issue

I think I fixed this problem with:
brew install mpicc
pip3 install mpi4py
but I was using virtualenv, not anaconda, but it might work in the same way

Right, the problem is python mpi is just a wrapper. You have to install library itself.

Hi guys, I have the same issue and I do not know how to fix it. I have little experience with python and I need to do a homework about it. This error is shown: ImportError: dlopen(/anaconda3/lib/python3.6/site-packages/mpi4py/MPI.cpython-36m-darwin.so, 2): Library not loaded: @rpath/libmpi.1.dylib
Referenced from: /anaconda3/lib/python3.6/site-packages/mpi4py/MPI.cpython-36m-darwin.so
Reason: image not found
Thank you so much if you could help me with this.

For me, I install mpich with this command conda install mpich, then it will work.


chore(main): release 0.1.1

:robot: I have created a release beep boop
0.1.1 (2024-07-07)
Bug Fixes

Handle annotations that don't include a digest (59140a0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/philipcristiano/docker-registry-cleaner/releases/tag/v0.1.1 :sunflower:


get contract sources from contract definition

Description
Fixes :
https://github.com/scaffold-eth/scaffold-eth-2/pull/564#issuecomment-1781833141
https://github.com/scaffold-eth/scaffold-eth-2/pull/564#issuecomment-1763430600

Summary of the solution :
If you look at sources which is generated by hardhat-deploy for contract E are :

 Contract `E` sources :

{
  'contracts/A.sol': {
    content: '//SPDX-License-Identifier: MIT\n' +
      'pragma solidity >=0.8.0 <0.9.0;\n' +
      '\n' +
      'contract A {\n' +
      '\tfunction foo() public pure virtual returns (string memory) {\n' +
      '\t\treturn "A";\n' +
      '\t}\n' +
      '}\n',
    keccak256: '0xf1fd7d3a46e81bfb4bb4c45788eb1f09c8991ebde86ce57716c2da7725713c69',
    license: 'MIT'
  },
  'contracts/B.sol': {
    content: '//SPDX-License-Identifier: MIT\n' +
      'pragma solidity >=0.8.0 <0.9.0;\n' +
      '\n' +
      'import "./A.sol";\n' +
      '\n' +
      "// Contracts inherit other contracts by using the keyword 'is'.\n" +
      'contract B is A {\n' +
      '\t// Override A.foo()\n' +
      '\tfunction foo() public pure virtual override returns (string memory) {\n' +
      '\t\treturn "B";\n' +
      '\t}\n' +
      '}\n',
    keccak256: '0x694df20286795fd4f09730c2157578d071f13b57fdca7940a4e4111b1c677583',
    license: 'MIT'
  },
  'contracts/C.sol': {
    content: '//SPDX-License-Identifier: MIT\n' +
      'pragma solidity >=0.8.0 <0.9.0;\n' +
      '\n' +
      'import "./A.sol";\n' +
      '\n' +
      'contract C is A {\n' +
      '\t// Override A.foo()\n' +
      '\tfunction foo() public pure virtual override returns (string memory) {\n' +
      '\t\treturn "C";\n' +
      '\t}\n' +
      '}\n',
    keccak256: '0x55497f031a5fee0c152747fa25bf6c8c79551ad61b455b209e652c1588126dbc',
    license: 'MIT'
  },
  'contracts/E.sol': {
    content: '//SPDX-License-Identifier: MIT\n' +
      'pragma solidity >=0.8.0 <0.9.0;\n' +
      '\n' +
      'import "./B.sol";\n' +
      'import "./C.sol";\n' +
      '\n' +
      'contract E is C, B {\n' +
      '\t// E.foo() returns "B"\n' +
      '\t// since B is the right most parent contract with function foo()\n' +
      '\tfunction foo() public pure override(C, B) returns (string memory) {\n' +
      '\t\treturn super.foo();\n' +
      '\t}\n' +
      '}\n',
    keccak256: '0x777c64b583bb7ae1eca9463b379a72811bbb31be28a590c61154a05395a4971f',
    license: 'MIT'
  }
}

In the previous solution we were just mapping over the above object keys using object.keys(sources) and as we can see in the above object contract/C.sol comes after contract/B.sol therefore contract/C.sol was overriding it.
In a new solution instead of relying on object keys of sources we instead derive actualSources from this :
"contract E is C, B {\n" that are [C, B] and hence always right most parent will override it
This also nicely solves https://github.com/scaffold-eth/scaffold-eth-2/pull/564#issuecomment-1763430600 since now we are getting "actual real sources" from contract definition directly.
Lol not that good explanation but if anyone wants to play around you can check Rinat's branch https://github.com/scaffold-eth/scaffold-eth-2/tree/inherited-edge-case (Here I have put helpful console.log to make sense) 🙌

Nice solution! Tested it and it works!
Left two small commments. Approve, thanks!

Thank you for your help @technophile-04 @rin-st . I had a look and it looks great. :+1:


Add spring node gravity and simple collision

Issue #58
Gravity sets a target bias in the -y direction.
Add support for a ground plane collider and a list of spherical
colliders. The ground plane prevents the spring from crossing below y=0.
Spherical colliders repel the springs from a point in space.
To create a geometrical UI for the collider, create a NURBS sphere, connect its
world space position to the input position, and its world space scale to the
collider radius.
Collision can be given a softness value to create soft or hard collisions.
Zero softness prevents springs from crossing the barriers entirely.

Hello @miabrahams  Thanks for the PR!
Can I ask you for a big favor?  Can you do the PR but instead of modifying the original spring node, create a new one? maybe we can call it "springGravityNode.cpp"
Later I will do that optional in the components as an option.
The main reason is that @hulud75 just has updated the Rumba contra part mGear solvers with the spring node, if we change it now we will break compatibility 😅  https://github.com/MercenariesEngineering/mgear_solvers_rumba
It is not possible for you to do it. I will do it by hand from your PR 😄
About the Node ID: don't worry  just add one random ID and I will update it later.
Thanks!!!

Hi Miguel, that's no problem. I also added a custom gravity direction and ground plane orientation because why not.
https://user-images.githubusercontent.com/10999400/152655179-50feb275-7b0f-4ef6-83e5-5c537ad17827.mp4

Thanks a lot! @miabrahams  😄

Hello @miabrahams Thanks for the PR! Can I ask you for a big favor? Can you do the PR but instead of modifying the original spring node, create a new one? maybe we can call it "springGravityNode.cpp" Later I will do that optional in the components as an option. The main reason is that @hulud75 just has updated the Rumba contra part mGear solvers with the spring node, if we change it now we will break compatibility sweat_smile https://github.com/MercenariesEngineering/mgear_solvers_rumba It is not possible for you to do it. I will do it by hand from your PR smile About the Node ID: don't worry just add one random ID and I will update it later. Thanks!!!

Hi ! Please do not worry about the Rumba port in your design choice. For sure, it's good for the mgear users if you do not break the compatibility with previous Maya versions, but regarding the Rumba port, we will simply follow your modifications, no worries.

@miabrahams  finally doing the merge! Thanks again for the PR!


Created New Docker Image for Protocol Server and Deployed it in Dev Environment

Description
We have created a new Docker image for the protocol server and successfully deployed it in the Dev environment.
Goals

Build a new Docker image for the protocol server.
Ensure the image includes the latest configurations and dependencies.
Deploy the new Docker image in the Dev environment.

Expected Outcome
A new, fully functional Docker image for the protocol server, deployed in the Dev environment, with all functionalities working as expected.
Acceptance Criteria

[ ] Build a new Docker image for the protocol server.
[ ] Ensure the Docker image includes the latest configurations and dependencies.
[ ] Deploy the Docker image in the Dev environment.

Mockups / Wireframes
N/A (This section can be added if there are specific UI requirements.)
Product Name
Beckn
Domain
Protocol Server
Tech Skills Needed

Docker
Kubernetes (for deployment)
CI/CD pipeline management
Troubleshooting and debugging

Complexity
Medium
Category
Infrastructure
Sub Category
Containerization

We have created the new docker image for the PS with the latest code changes and deployed it in dev env.


[Bug] [dolphinscheduler-ui] When editing the timing, the alarm group displays 0

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened

Version
2.0.1-prepare
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I will solve


fix(mission_planner): align goal pose to lanelet map

Signed-off-by: Xinyu Wang<EMAIL_ADDRESS>Description
Goal pose is aligned to lanelet map by calculating an average height of each segment from the closest lanelet component.

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

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

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

[ ] The PR follows the pull request guidelines.

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

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

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

I can confirm that the goal pose is on the map with this fix. In main commit it was ~50m under the map.

The code changes are minimal and on point. I've rebased it to main and once CI is finished, it should be merged.


Support vcpkg

Hi, could you please support vcpkg?
Then the integration process will as easy as vcpkg install optick.

I agree with you.
We need this feature.
I have personally researched how to support vcpkg.
I was able to confirm that it works by placing the attached sample in vcpkg/ports/optick.
If @bombomby feels there is no problem, I would like to see this reflected in https://github.com/microsoft/vcpkg.
{
    "name": "optick",
    "version-string": "1.4.00",
    "port-version": 1,
    "description": "Optick is a super-lightweight C++ profiler for Games.",
    "homepage": "https://github.com/bombomby/optick",
    "license": "MIT",
    "dependencies": [
      {
        "name": "vcpkg-cmake",
        "host": true
      },
      {
        "name": "vcpkg-cmake-config",
        "host": true
      }
    ]
  }

vcpkg_check_linkage(ONLY_DYNAMIC_LIBRARY)

vcpkg_from_github(
    OUT_SOURCE_PATH SOURCE_PATH
    REPO bombomby/optick
    REF <IP_ADDRESS>
    SHA512 9b835163aef7adaa7296c90eb104a742a32021a420f05ae7dd4025bceea0c4f624a1f447684a2c8a88ee8c43fd54ec83809e9434e6a786922b3b07b90f10c64e
    HEAD_REF master
)

vcpkg_cmake_configure(
    SOURCE_PATH "${SOURCE_PATH}"
    OPTIONS
        -DOPTICK_USE_VULKAN=OFF
)

vcpkg_cmake_install()

vcpkg_cmake_config_fixup(CONFIG_PATH lib/cmake/optick)

file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug/include")

vcpkg_copy_pdbs()

# Handle copyright
file(INSTALL "${SOURCE_PATH}/LICENSE" DESTINATION "${CURRENT_PACKAGES_DIR}/share/${PORT}" RENAME copyright)

I agree with you. We need this feature.
I have personally researched how to support vcpkg. I was able to confirm that it works by placing the attached sample in vcpkg/ports/optick. If @bombomby feels there is no problem, I would like to see this reflected in https://github.com/microsoft/vcpkg.
{
    "name": "optick",
    "version-string": "1.4.00",
    "port-version": 1,
    "description": "Optick is a super-lightweight C++ profiler for Games.",
    "homepage": "https://github.com/bombomby/optick",
    "license": "MIT",
    "dependencies": [
      {
        "name": "vcpkg-cmake",
        "host": true
      },
      {
        "name": "vcpkg-cmake-config",
        "host": true
      }
    ]
  }

vcpkg_check_linkage(ONLY_DYNAMIC_LIBRARY)

vcpkg_from_github(
    OUT_SOURCE_PATH SOURCE_PATH
    REPO bombomby/optick
    REF <IP_ADDRESS>
    SHA512 9b835163aef7adaa7296c90eb104a742a32021a420f05ae7dd4025bceea0c4f624a1f447684a2c8a88ee8c43fd54ec83809e9434e6a786922b3b07b90f10c64e
    HEAD_REF master
)

vcpkg_cmake_configure(
    SOURCE_PATH "${SOURCE_PATH}"
    OPTIONS
        -DOPTICK_USE_VULKAN=OFF
)

vcpkg_cmake_install()

vcpkg_cmake_config_fixup(CONFIG_PATH lib/cmake/optick)

file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug/include")

vcpkg_copy_pdbs()

# Handle copyright
file(INSTALL "${SOURCE_PATH}/LICENSE" DESTINATION "${CURRENT_PACKAGES_DIR}/share/${PORT}" RENAME copyright)

Worked perfectly!
Thanks!


Unable to release GPU memory after training a model (OOM)

Hi,
On a Google Colab notebook with keras(2.2.4) and tensorflow(1.13.1) as a backend, I am trying to tune a CNN, I use a simple and basic table of hyper-parameters and run my tests in a set of loops.
My problem is that I can't free the GPU memory after each iteration and Keras doesn't seem to be able to release GPU memory automatically. So every time I get a Ressource Exhausted : Out Of Memory (OOM)
I did some digging up and run into this function that reassembles different solutions that have been suggested to solve this problem (didn't work for me though) :
def reset_keras():
    sess = get_session()
    clear_session()
    sess.close()
    sess = get_session()

    try:
        del model # this is from global space - change this as you need
    except:
        pass

    print(gc.collect()) # if it's done something you should see a number being outputted

    # use the same config as you used to create the session
    config = tf.ConfigProto()
    config.gpu_options.per_process_gpu_memory_fraction = 1
    config.gpu_options.visible_device_list = "0"
    set_session(tf.Session(config=config))

The only thing that i didn't fully grasp is the "same config as you used to create your model " since with Keras we don't chose explicitly a certain configuration. I get by for one iteration, some times two, but I can't go beyond. I already tried to change the batch_size and for the moment I am unable to afford for a machine with higher performances.

Please fill the issue template here. Could you update them if they are relevant in your case, or leave them as N/A?  Thanks!

Done! follow the link

Thanks for filling the template in the new issue. I am closing this issue here as this is duplicate of https://github.com/keras-team/keras/issues/12929. Thanks


PostgREST Dockerfile

PostgRest Dockerfile with ENV parameter passthrough. This dockerfile builds an image that can be linked to the offical postgresql image and uses ENV vars for postgrest parameter passthrough.
build postgrest docker image
docker build -t pgrest .
start postgresql container
docker run --name pg  -p 5432:5432 -e POSTGRES_PASSWORD=sillypass -d postgres:9.5
start postgREST API server linked to the db
docker run --name pgrest --link pg:pg -p 3000:3000 \
	-e POSTGREST_ANONYMOUS=anon \
	-e POSTGREST_SCHEMA=public \
	-e POSTGREST_JWT_SECRET=thisisnotarealsecret \
	-e POSTGREST_MAX_ROWS=1000000 \
	-e POSTGREST_POOL=200 \
	-d pgrest

Really grateful for your contribution, docker support will be great.
I've got some questions (keep in mind I'm new to the docker world!)

Is this compatible with having docker-compose run and connect images for the db and postgrest?
Will this image allow me to do automated builds on Docker Hub? (I created a repo there.) It might be cool to have pre-built docker images for the latest code on master as well as specific tagged versions. Is that a common thing to do?
If for some reason the postgrest server died in the container would it be restarted?
Could you add to this pull request some info in the docs about running the docker image? The relevant sections are https://github.com/begriffs/postgrest/blob/master/docs/admin/deployment.md and possibly https://github.com/begriffs/postgrest/blob/master/docs/install/server.md

Finally @ptman, @ChristopherBiscardi or anyone else would you care to review?

@begriffs: Regarding the Docker Compose compatibility concern raised, I can confirm that using this pull request's Dockerfile in my own solution (https://github.com/danielxvu/docker-postgrest-with-postgresql) works as expected with the PostgreSQL image via Docker Compose. (On a slightly related note, I'd love to hand over my Docker Compose config to this repository or add a section in the wiki at least--is one of those options preferable over the other?)

Thanks for the encouragement @begriffs on my first pull request ever.
For 1) this docker file should be compatible with docker-compose. I can get an example of this and some other examples as well.
For 2) I've never setup an automated docker build but this dockerfile should be fine but will need to bump the postgrest version number and ubuntu version manually.
For 3) We can add --restart=always flag to the docker run command or to the docker-compose file for auto-restarts.
Finally, we could combine all the run commands into one command. There may be an advantage to keeping them separate though. From the dockerfile docs:

The RUN instruction will execute any commands in a new layer on top of the current image and commit the results. The resulting committed image will be used for the next step in the Dockerfile.
Layering RUN instructions and generating commits conforms to the core concepts of Docker where commits are cheap and containers can be created from any point in an image’s history, much like source control.

There are a few easy wins in terms of size, which turns the Dockerfile into:
FROM debian:jessie

ENV POSTGREST_VERSION <IP_ADDRESS>
ENV POSTGREST_SCHEMA public
ENV POSTGREST_ANONYMOUS postgres
ENV POSTGREST_JWT_SECRET thisisnotarealsecret
ENV POSTGREST_MAX_ROWS 1000000
ENV POSTGREST_POOL 200

RUN apt-get update && \
    apt-get install -y tar xz-utils wget libpq-dev && \
    apt-get clean && rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/*

RUN wget http://github.com/begriffs/postgrest/releases/download/v${POSTGREST_VERSION}/postgrest-${POSTGREST_VERSION}-ubuntu.tar.xz && \
    tar --xz -xvf postgrest-${POSTGREST_VERSION}-ubuntu.tar.xz && \
    mv postgrest /usr/local/bin/postgrest && \
    rm postgrest-${POSTGREST_VERSION}-ubuntu.tar.xz

CMD postgrest postgres://${PG_ENV_POSTGRES_USER}:${PG_ENV_POSTGRES_PASSWORD}@${PG_PORT_5432_TCP_ADDR}:${PG_PORT_5432_TCP_PORT}/${PG_ENV_POSTGRES_DB} \
              --port 3000 \
              --schema ${POSTGREST_SCHEMA} \
              --anonymous ${POSTGREST_ANONYMOUS} \
              --pool ${POSTGREST_POOL} \
              --jwt-secret ${POSTGREST_JWT_SECRET} \
              --max-rows ${POSTGREST_MAX_ROWS}

EXPOSE 3000

How
Ubuntu version (original Dockerfile)
ubuntu                   14.04               36248ae4a9ac          188 MB
pgt                      latest              0403d77f66fc          352.3 MB

removing the whiteouts by consolidating the apt commands (and cleanup) into a single layer reduces size by ~20mb
pgt-1                    latest              712dc71fff2a          330.8 MB

Using debian:jessie as a base image brings it down another ~80mb
debian                   jessie              9b607719a62a        125.1 MB
pgt-3                    latest              2aca79a0bd9d        253.8 MB

Merging the wget/tar/cleanup commands reduces another ~20mb
pgt-4                    latest              d02b67c0edb4        229.3 MB

@begriffs

Is this compatible with having docker-compose run and connect images of the db and postgrest?

Looks fine. I'll try to check it out in a real compose setup soon.

Will this image allow me to do automated builds on Docker Hub? (I created a repo there.) It might be cool to have pre-built docker images for the latest code on master as well as specific tagged versions. Is that a common thing to do?

You can set this up. Building master should work by default (and I think it also builds any other branches by default) although alternate schemes can be set up.

If for some reason the postgrest server died in the container would it be restarted?

I'd expect the container to restart in this scenario, not the server inside. You can set up a restart policy for that purpose.

@ChristopherBiscardi some time ago i was suggesting using your method to build a very small distribution (and package it in docker based on alpine).
https://github.com/begriffs/postgrest/issues/365
Would it be possible for you to try to build postgrest that way?
The only real dependency for postgrest is libpq and after a superficial search on google, it seems ppl have compiled postgresql agains musl so there should be a way to get libpq compiled agains musl.

@ruslantalpa
I haven't tried it with postgrest, but it seems likely. If there's interest I could do it. The bigger issue would likely be making sure whatever ghc-musl based build process (for building postgrest itself) is acceptable to the project.
We wouldn't need to compile libpq. The following yields a 4.969 MB image.
FROM alpine
RUN apk add libpq --update && rm -rf /var/cache/apk/*

Thanks @ChristopherBiscardi, sliming down docker images is always nice.
This is a good reference on building postgresql on alpine docker: http://michael.otacoo.com/postgresql-2/minimal-docker-container-postgres/
I think it would be a good idea to keep the interface as close as possible to the official postgresql image. FIY, a key Apline linux maintainer just got hired at Docker -it's possiable that the official images will start to be ported to Alpine this year: https://www.brianchristner.io/docker-is-moving-to-alpine-linux/
As for the docker-compose file, I would propose keeping that in a separate thread/pull-request. The compose format was just changed recently. Also worth considering what services should be included. E.g. I think it would be cool to include a Caddy web-server/load-balancer with auto SSL enabled in addition to postgresql + postgRest services. The new compose features would make it easy to scale-up the number of postgRest containers.

@ChristopherBiscardi there is no "build process standard" and even a guide on how to do it would be very welcome, and maybe later an automated process can be developed but for now a guide to manually build for alpine and have it in docker would be cool.

I've updated the Dockerfile with @ChristopherBiscardi suggestions.

Thanks again for your contribution!


More explicit error messages when vulkan calls return errors

Some changes made while debugging on Android that can be generally useful.

Thanks!


improve windows automatical

Vim bootstrap version(number in first line):
Operating system: win10
Vim/Nvim version: gvim8.1
Languages: en
automatical

Description
on windows, 32bit gvim has to use %windir%\Sysnative to call curl.(64bit has not)
I want to PR but not familiar with vim script.
hope you guys can help fix it, thanks!

I figured out the way.
do you need pr?

send PR

https://github.com/avelino/vim-bootstrap/pull/321

solved in #321


Clean up examples

Clean up the Hello World example which had broken targets and some trailing garbage code.
Fix the build of examples/sample_project that uses Kremlin to drive compilation and thus needs an extra -drop WasmSupport and including kremlin/internal/compat.h because of the use of Prims.string.
Add examples/dependencies that includes a template Makefile showing how to use --dep full to compute dependencies and do incremental and parallel builds of OCaml binaries.

@s-zanella Thank you for improving the hello example, but this broke the instructions for testing binary packages, both the human readable ones:
https://github.com/FStarLang/FStar/blob/master/INSTALL.md#testing-a-binary-package
and the automatic scripts that produce the binaries:
https://github.com/FStarLang/FStar/blob/master/.scripts/process_build.sh#L107
https://everestlogstorage.blob.core.windows.net/fstar-binaries-linux/buildlogfile_FStar-Binaries-Linux-20190603-1.html
Can you please update these as well? :)


Interpolation error output from forward model in gridded case

Hi all, when I run gridded 50x50 hyperspectral datacubes, I get the following error in forward mode:
Traceback (most recent call last):
  File "/Users/bob/anaconda3/lib/python3.7/site-packages/isofit-1.4.3-py3.7.egg/isofit/core/isofit.py", line 88, in __init__
    self.__call__()
  File "/Users/bob/anaconda3/lib/python3.7/site-packages/isofit-1.4.3-py3.7.egg/isofit/core/isofit.py", line 119, in __call__
    self.fm.reconfigure(*configs)
  File "/Users/bob/anaconda3/lib/python3.7/site-packages/isofit-1.4.3-py3.7.egg/isofit/core/forward.py", line 317, in reconfigure
    self.surface.reconfigure(config_surface)
  File "/Users/bob/anaconda3/lib/python3.7/site-packages/isofit-1.4.3-py3.7.egg/isofit/surface/surface.py", line 61, in reconfigure
    self.resample_reflectance()
  File "/Users/bob/anaconda3/lib/python3.7/site-packages/isofit-1.4.3-py3.7.egg/isofit/surface/surface.py", line 67, in resample_reflectance
    p = interp1d(self.rwl, self.rfl, fill_value='extrapolate')
  File "/Users/bob/anaconda3/lib/python3.7/site-packages/scipy/interpolate/interpolate.py", line 431, in __init__
    _Interpolator1D.__init__(self, x, y, axis=axis)
  File "/Users/bob/anaconda3/lib/python3.7/site-packages/scipy/interpolate/polyint.py", line 60, in __init__
    self._set_yi(yi, xi=xi, axis=axis)
  File "/Users/bob/anaconda3/lib/python3.7/site-packages/scipy/interpolate/polyint.py", line 125, in _set_yi
    raise ValueError("x and y arrays must be equal in length along "
ValueError: x and y arrays must be equal in length along interpolation axis.

It appears that the IO object is iterated over each pixel and fed into self.fm.reconfigure(*configs). While this runs file the first iteration, on the second iteration it appears to pull the reflectances from the reference_reflectance ENVI file and try to interpolate these with the sensor band positions from the data/wavelengths.txt file. These are different lengths and it errors (the reference data is 211 bands while the target is 213). Where this gets quite weird is that it works correctly for the standard 1x15 bare-earth reference_reflectance ENVI file, which has 425 bands.
I'm running a trace now to try to dig to the source of this problem. A major rewrite of the IO class is recommended for the future, as the way it selects the operation mode from the JSON configs is extremely opaque. I would recommend a standard argument like 'mode' that executes specialized subroutines conditional upon the value of this switch - then it will be easier to understand the call graph and where the data is mutated.

So, I modified the surface wavelength_file to match the surface reflectance bands and not the wavelength_file used in instrument and radiative_transfer, which are the sensor wavelengths I believe. I don't know if this is correct, but it works. Please be sure to correct me if I am doing this wrong, as it is not described in the wiki (link).

Stale issue, resolved through some of the systematic refactors a while back, but this got left behind.  Closing!


Automated Testing show errors

VSCode Version: 1.40.2
OS Version: macOS Catalina

Steps to Reproduce:

Get the latest source code and build succeeded as How to Contribute.
Run Automated Testing
$ ./scripts/test.sh > test_result
There are 12 same errors output on the terminal

[93167:1130/140149.646715:INFO:CONSOLE(736)] "_saveResultAndEncode: 1 --> 0-full(10), 10-full(10    ), 20-full(3)", source: file:///Users/lamont/vscode/out/vs/workbench/api/common/extHostLangua    geFeatures.js (736)

I have removed the console logs


Fix spelling mistake in kube scheduler

This PR aims to fix a spelling mistake in the Kubernetes Scheduler documentation.
An 's' was missing. I checked the rest of the file and did not find any other spelling mistake. No broken link found.
Link to doc: https://kubernetes.io/docs/concepts/scheduling-eviction/kube-scheduler/

Hi @alexis974, thanks for your contribution. We usually consider PRs like this one trivial edits. Could you go through the page to check if there are any other issues you can fix on this page?

please take another look @Princesso

Hello @kbhawkey, can you take a look ?

/lgtm
Great clean up - thanks!

/assign @reylejano


PromiseQueue not being processed

Sometimes a promise that was added to the promsie is queue is never resolved, resulting in no new instances getting initialized.
https://github.com/blutorange/primefaces-monaco/blob/master/src/main/js/widget.js

The promises added to the queue are resolved once the editor is rendered - and since the editor is a deferred widget that is rendered only when its container becomes visible, the queue waits for all initially queue editors to get ready.


Component Abstraction Layer

Create a component abstraction layer between the component and slave that allows you to easily specify modules based on available RP2040 peripherals and the component I/O. Modules will use functionality this layer provides in their own wrappers. I.E., separate out the lower-level details of the RP2040.
Ideas:

[ ] GPIO (Button might implement this in wrapper)
[ ] ADC (Joystick might implement this in wrapper)
[ ] I2C (OLED, Accelerometer might implement this in wrapper)
[ ] SPI
[ ] UART
[ ] PWM
[ ] PIO (WS2812 LEDs might implement this in wrapper)

Example: For GPIO, we might make an init function that initializes a pin for input with internal pull-up, and a read function that returns the pin status (HIGH or LOW). The button wrapper can utilize these functions on the specified pin to read in the pressed status for transmission to master.

We decided to abandon this approach as a team. The Pico SDK is high-level enough that we can reasonably expect users to extend the Prototroller with it in order to create custom modules.


Modal select opened from popup

Hi,
I got a new issue today while I was trying to open a modal select view from a popup.. The view stay behind the popup meaning that I can still interact with it but the view is greyed and I was hopping the view to be forward..
This css trick didn't really help
.popup-open .backdrop { z-index: 11; }

mmh, interesting.
should the popup be closed before opening the select?

No, it should stay open to fill one field in the popup...

Actually, changing the z-index of a modal to 13 is solving the issue.


Create trouble shooting setup guide

We get a lot of people who come to us with very similar setup problems. We should have a single page that says "If you have Nix + Cabal v2, be aware of ...". If your tests fail with ghcide then ... If it works in ghcide but not VS Code then perhaps....

Would it be possible to detect any of the typical problems automatically? For example, incompatibility of GHC version between ghcide and project.

@jacg haskell-ide-engine implements this by querying ghc --version to check that it matches. This isn't really perfect though because you might not use the ghc on your path to compile a project (for example using ob build or stack build)
@ndmitchell This is the kind of stuff I was thinking of putting in documentation for hie-bios as it is usually not anything to do with ghcide. https://github.com/mpickering/hie-bios/issues/58

@mpickering we really need one piece of documentation that covers everything from hie-bios to ghcide. We want to have the documentation refer to running ghcide and the output it gives you. I appreciate most of the docs will actually be about hie-bios though. Thoughts on how to square that circle?

See https://github.com/digital-asset/ghcide/issues/135 for one of the most common issues people run into.

@ndmitchell do you think this is acomplished by #129?

@jneira you just linked back to this ticket.
I suspect this should be a HLS thing rather than a Ghcide thing though.

Ops, sorry, i wanted to link your pr #142

#142 definitely started this task, but for Ghcide it was more a brain dump of issues, rather than a curated guide. There seems a lot of information in that guide that isn't in the HLS readme. Is that because its not necessary? Is everyone having such a smooth experience that we ran out of support requests? That seems unlikely to me, but at the same time, I haven't seen many questions about failing to set up HLS, so maybe everything got better?


Add OderBy output stage spill

Add a new type of spiller named kOrderByOutput.
It is used to spill data during the output processing
stage of OrderBy operator. The spiller uses a newly
added API that supports spill data by row pointers.

@xiaoxmeng Hi Xuan, I've made a small refactor and resolved all your comments. Could you please help take another look? Thanks.

@xiaoxmeng Hi Xuan, thanks for your review and guidance, I've resolved all the comments, could you please help to take a look?


New compiler warning

Whoops.
I have seen what I think is a new compiler warning:
gcc -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_MACVIM -Wall -Wno-unknown-pragmas -pipe  -DMACOS_X_UNIX  -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1        -o objects/ui.o ui.c
terminal.c:819:7: warning: implicit declaration of function
      'gui_mch_get_rgb_color' is invalid in C99
      [-Wimplicit-function-declaration]
        fg = gui_mch_get_rgb_color(cell->fg.red, cell->fg.green, cell->fg.blue);
             ^
1 warning generated.

I don't know if that's worth worrying about, but there it is.


Version 4.5.3; unreadable text color

Version 4.5.3, as of the writing of this issue, does not correctly render most text on a Wikipedia page.

Hi, thanks for submitting! Someone else reported this issue too and fixed it, please let me know if this helps you: https://github.com/hirschan/Dark-Mode-Wikipedia/issues/44

It does! Thank you so much.
For anyone in the future; uBlock might disallow certain JS scripts from running, so just whitelist Wikipedia.


api/token or api/tokens?

The readme makes reference to resp = requests.post('https://guacamole-server-domain/api/token', data={'token': jwtToken),  yet I'm seeing api/tokens as the relative path in the javascript sent by the guacs that I've seen.

Yes, I will update the readme doc.


Default frame bigger than content

Description

Steps to Reproduce

Run the app from the raw

Expected Behavior
There is no scrolls, all content fit into the app window
Actual Behavior
There is a scroll and empty line at the bottom
Environment
https://github.com/novasamatech/nova-spektr/commit/fadf2caf57bebc1727f648b053a8e2953204a5eb

Status verification: ✅


Group border remains on screen with wokspaces fade animation

Steps to reproduce:

Set this animation: animation=workspaces,1,9,default,fade
Set a border for groups.
Create a group.
Swap to an empty workspace.

You can see that the group border remains ~1 second on screen.
Thank you so much for your attention. Bless.

fixed in 7ed401e5e0779e851667c2e5d220c24e9a5b8737

Amazing, it is working fine. Thank you very much.


Protected branches: git push --force/--delete should be available only to administrators

Right now Gogs allows git push --force/--delete to anyone having write permissions.
Therefore Gogs should introduce idea of "protected branch" list repository setting that can be only deleted or forcefully pushed only by administrator. This should basically work the same as in GitHub or GitLab. I also propose to make "master" by default a protected branch when creating new repo.
WDYT?

Thanks, maybe you should follow on https://github.com/gogits/gogs/issues/776?

@Unknwon I think #776 mixes two concepts/ideas in one single issue (may be bad for Gogs project management). Code review is good idea, but it is somehow complicated, not well defined and "big" feature. This issue here is more simple and about security, and has nothing to do with #776 code review functionality.
I think the security concern about NOT letting to --force/--delete to anyone but administrators is really important, otherwise Gogs may be no go for enterprise.
I think I can try to implement that, but next week earliest. IMHO the implementation should include following atomic steps (commits):
Preparation

Gogs adds own pre-recive hooks, all existing pre-recive hooks should be renamed to pre-receive-local. Editing pre-receive hook should effectively edit pre-receive-local. The Gogs own pre-revice hook should simply check for existence of pre-recive-local and fire it, so administrators are able to customize/extend pre-receive hook.
(optional) We may do the same with update hook, currently in-editable. Introduce update-local fired if present by Gogs managed update.

Actual implementation

Introduce internal API that return true if the given user is allowed to push/delete given ref. We may start by assumption that master is protected and required administrator and anything else is no protected.
In pre-receive hook we check if the pushed stuff deletes or forcefully updates some ref, we ask Gogs via new API is it is allowed. If not, we return error in the hook.

Next having this basic implementation we may start implementing protected branch list and extending the API to actually check if ref belong to protected list rather than simply checking if it is master.

In my point of view, disable --force/--delete is part of the protected branch feature, so I still think you should follow #776.

ping @nanoant

Okay, closing this one in favor of #776.

:smiley:


Chapt 8: Curiosity Driven Deep Learning RAM Requirements

I have attempted to Chapt 8 code, as a python file,  on a 32Gb CPU RAM Ubuntu 18.04 rig with 16Gb NVidia 1800 GTi GPU card.  However my RAM Utilisation grows excessively as the training epochs run, exceeds 30 Gb when I hit 1800 epochs on the Super Mario Mario Curiosity Deep Training code.
The book suggested that this code would only take 30 minutes on a Mac Book Air (no GPU) So i don’t understand why the RAM use grows to excess as the training epochs grows.
Interested in any others experience on this, or why I would be experiencing such excessive and growing RAM utilisation.

After training for about 30 minutes on a MacBook Air you could get an agent that was at least able to progress through the most of the first level - just demonstrating that the algorithm is working. It won’t be able to do much more than that with such short training. I’ll review the code when I get a chance, it must be storing something that doesn’t need to be stored each loop iteration.

OK, I have attempted to run this in Windows 10 (I didn't think the Atari games would run in Windows  from previous AI Gym experience. )
I encountered no problems, the code runs well, and trains the networks, and trains a pretty useful Mario Agent Player.
So it must be something odd , with my own Ubuntu python environment.


Can't use the package on Web

Hello,
I am facing the following error when using the package on web. (I have no issues with the same code on mobile);
Please help
Here is my query:
 Widget paginateFirestoreTest() {

    return PaginateFirestore(

        itemsPerPage: 10,
        itemBuilderType: PaginateBuilderType.listView, // listview and gridview
        itemBuilder: (index, context, documentSnapshot) {

          return Container();
        },
        query: FirebaseFirestore.instance.collection("MyCollection").limit(2));

  }

And the error I got when running this:
Bad state: Source maps are not done loading.
    at Object.a (http://localhost:52288/stack_trace_mapper.js:298:3)
    at cG.kB (http://localhost:52288/stack_trace_mapper.js:2612:23)
    at Object.j2 (http://localhost:52288/stack_trace_mapper.js:250:69)
    at jU (http://localhost:52288/stack_trace_mapper.js:2631:10)
    at http://localhost:52288/stack_trace_mapper.js:2625:42
    at dart._StackTrace.new.toString (http://localhost:52288/dart_sdk.js:7783:19)
    at Object.str (http://localhost:52288/dart_sdk.js:5488:32)
at Function._applyStackFilter (http://localhost:52288/packages/flutter/src/foundation/debug.dart.lib.js:3766:32)
at new assertions.DiagnosticsStackTrace.new (http://localhost:52288/packages/flutter/src/foundation/debug.dart.lib.js:3779:135)
at assertions.FlutterErrorDetails.new.debugFillProperties (http://localhost:52288/packages/flutter/src/foundation/debug.dart.lib.js:3477:24)
at http://localhost:52288/packages/flutter/src/foundation/debug.dart.lib.js:2523:26
at assertions._FlutterErrorDetailsNode.new.get builder (http://localhost:52288/packages/flutter/src/foundation/debug.dart.lib.js:2526:30)
at assertions._FlutterErrorDetailsNode.new.get builder [as builder] (http://localhost:52288/packages/flutter/src/foundation/debug.dart.lib.js:3804:27)
at assertions._FlutterErrorDetailsNode.new.get emptyBodyDescription [as emptyBodyDescription] (http://localhost:52288/packages/flutter/src/foundation/debug.dart.lib.js:2535:58)

Here my yaml file
name: myApp
description: here is my description

# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1

environment:
  sdk: ">=2.12.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^1.0.2
  google_fonts: ^2.1.0
  auto_route: ^3.2.0
  responsive_builder: ^0.4.1
  auto_route_generator: ^3.2.0
  feather_icons: ^1.1.0-nullsafety.0
  font_awesome_flutter: ^9.0.0-nullsafety
  flutter_staggered_grid_view: ^0.4.0
  url_launcher: ^6.0.3
  layout: ^1.0.2
  carousel_slider: ^4.0.0
  dots_indicator: ^2.0.0
  animated_text_kit: ^4.2.1
  visibility_detector: ^0.2.0
  get: ^4.3.8
  webviewx: ^0.2.1
  cloud_firestore: ^3.1.5
  firebase_core: ^1.3.0
  awesome_dialog: ^2.1.0
  string_validator: ^0.3.0
  intl: ^0.17.0
  firebase_auth: ^2.0.0
  webview_flutter: ^2.0.13
  google_tag_manager: ^0.1.0
  seo_renderer: ^0.3.0
  shared_preferences: ^2.0.6
  paginate_firestore: ^1.0.3
  flutter_rating_bar: ^4.0.0
  cached_network_image: ^3.1.0
  universal_html: ^2.0.8

dev_dependencies:
  flutter_test:
    sdk: flutter

  build_runner:

flutter:
  uses-material-design: true

  # To add assets to your application, add an assets section, like this:
  assets:
    - assets/images/

And my flutter doctor
`
[✓] Flutter (Channel stable, 2.8.1, on macOS 11.6.1 20G224 darwin-x64, locale fr-FR)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.63.2)
[✓] Connected device (1 available)
• No issues found!
`

@ben55j Can you attempt to remove <script src="main.dart.js" type="application/javascript"></script> from web/index.html


iOS: Warn about slow main thread React methods

When code blocks the UI thread for too long, it's a bad sign because this can prevent the app from remaining responsive. This change helps detect such responsiveness issues by warning when a React method executes on the UI thread longer than some threshold.
Test Plan
Changed AppState's getCurrentAppState method to sleep for 500 ms and verified that a warning was emitted:
2017-08-17 19:45:29.479 [warn][tid:main][RCTModuleMethod.mm:527] mainThreadWatchdog: invocation of [RCTAppState getCurrentAppState:error:] took 501ms

Adam Comella
Microsoft Corp.

I very much like the idea.
Small considerations:

We have to wrap this with #ifdef RCT_DEV;
I don't think that NSDate is the most efficient way to get a time stamp; usually the function from CoreAnimation is a good choice;
The message can be more clear and human oriented;
The duration should be a named constant and (ideally) configurable some way.

@shergin Thanks for reviewing this so quickly. I have some questions/comments:

We have to wrap this with #ifdef RCT_DEV;

My team measures performance on release builds not debug builds so I don't think it'll work for our use case if we wrap it with #ifdef RCT_DEV. Thoughts?

I don't think that NSDate is the most efficient way to get a time stamp; usually the function from CoreAnimation is a good choice;

Specifically which time API should I use? I spent a few minutes searching but couldn't figure it out.

The message can be more clear and human oriented;

Okay, what do you think of this?
mainThreadWatchdog: Invocation of [RCTAppState getCurrentAppState:error:] blocked the main thread for 501ms. Consider spending less time on the main thread to keep the app's UI responsive.

The duration should be a named constant and (ideally) configurable some way.

Do you have a recommendation for an easy way to make it configurable? If not, I'll just hardcode a value in a named constant for the purposes of this PR.

Specifically which time API should I use? I spent a few minutes searching but couldn't figure it out.

CACurrentMediaTime from <QuartzCore/CAAnimation.h> probably

Thanks, @janicduplessis. For reference, I found this detailed article about benchmarking on iOS: http://nshipster.com/benchmarking/. It mentions some relevant details about why CACurrentMediaTime is a good fit for our problem:

Unlike NSDate or CFAbsoluteTimeGetCurrent() offsets, mach_absolute_time() and CACurrentMediaTime() are based on the internal host clock, a precise, monatomic measure, and not subject to changes in the external time reference, such as those caused by time zones, daylight savings, or leap seconds

@rigdern
If we will not wrap it in RCT_DEV, my only consern is performance. Can we (conceptually) cache [NSThread isMainThread] value somewhere in this class... or this method (invokeWithBridge:) can naturally be called from several threads?
Proposed warning text is awesome! And just using named constant is good enough for now.

cc @javache I feel we need your blessing here anyways. 😄

@javache can you comment on whether it's safe to cache isMainThread and what level would be a good place to cache it (e.g. at the module or method level). Also, do I need to extend this PR to RCTCxxMethod.mm in order to support the new iOS bridge?

I agree with @shergin that this needs to be behind a flag. You could maybe do something like
#ifndef ENABLE_MAIN_THREAD_WATCHDOG
#def ENABLE_MAIN_THREAD_WATCHDOG RCT_DEV
#endif

That way, it can be enabled for your builds by overriding the flag but kept RCT_DEV-only for everyone else.
Also +1 on the CACurrentMediaTime.

To have a way to tweet threshold duration via #define flag is also good idea.
I like this approach in general though, I believe it is helpful. We used something similar heavily at one of my previous job. (But we crashes the app in such cases. 😆 )

@javache, @shergin I addressed your PR feedback:

Introduced constant RCT_MAIN_THREAD_WATCH_DOG_THRESHOLD. Defaults to 20 ms in debug builds and to undefined in release builds. Undefined means the watchdog is disabled.
Switched from NSDate to CACurrentMediaTime.
Switched from NSThread.isMainThread to RCTIsMainThread.
Switched to [self methodName] for printing the module and method name.
Made the warning message friendlier.


Could cgroups manage AMD GPU devices?

Could cgroups bind AMD gpu devices to a certain process?

Hi @nanguanqi , can you refer to the following comment? Option #3 would help bind GPU devices to a certain docker instance using cgroup. Are you referring to any other use cases?


NH-55892: Package workflow

Description
Change exit status based on last statement

Would steps such as bundle install or gem build fail but still end in a success exit for this script? If so, maybe using set -e is an option.

If bundle install or gem build fail it will fail since there is no gem file to send over the github package; but I agree set -e is a good solution. I have changed it to both ruby and ruby-contrib


Cart Rule -> Category is not being retained even after we have selected and saved it.

Terms

[X] #9959

Bagisto Version(s) affected
Master
Issue Description
I have created a cart rule and then i have applied the rule for category. I have selected the Category and then saved the Rule, after checking and editing the rule I found that the selected category is not appearing as selected and also no console errors are appearing.
Preconditions

Admin must be logged in.

Steps to reproduce

Go to Admin -> Marketing -> Cart Rule.
Create a cart rule -> Select Rule for Category -> Select Category
Save the Rule -> Edit again and check for the selected category.
See error

Expected Result
The categories which are selected need to be saved. Even we edit we should be able to see the selected categories.
Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/d900d7f7-4fcd-4d8a-adca-f9d0d9d41501
Actual Result
Currently we can see that the selected categories are not appearing.
Screenshots
No response
Additional context
No response

Checked and found that the issue has been fixed and PR has been merged and hence I am closing the issue from here.
PR Link -> https://github.com/bagisto/bagisto/pull/9955


Menu buttons renaming when click on contact

When you click on "contacts" the menu buttons get renamed for some reason

the same happens in the footer:

Fixed with https://github.com/BuildTheEarth/website-frontend/commit/8102a95c3b64a8a96dd2684cc7615f397eab0659


Bind vs. symlink

Hey, thanks for this very useful snapcraft part!
The layout docs recommend using symlink over bind where possible for performance reasons. Is there a specific reason for using bind for this alsa part?
https://github.com/diddlesnaps/snapcraft-alsa/blob/edde1b15030ffc13efce46f03f929a336e214ea4/snapcraft.yaml#L29
In my testing with the veloren snap, using symlink seemed to work fine.

I found that defining a layout with symlinks is more brittle and prone to failure.

Okay, fair enough. Are there known problems with using symlink specifically with this ALSA part though?
As it seems to be working for veloren, I'm inclined to stick with using symlink over bind for veloren.

I can't recall any specific to the alsa part, no

Ok, thank you!


Release planner - SDK release app should show information why Next button is disabled

Next button on the pre-req page of SDK release app is disabled if Spec PR is not signed off or merged. This causes confusion when a user looks at the release plan. We should show a notification why Next button is disabled to help user understand the reason.
It can be a tool tip for now.

This is covered in the UI improvements designs created by @ccbarragan . Clossing for now


added navigation buttons to player headline on player stats page

im popping off bro

this feature is useful because the top search bar brings you to this page and if you wish to access these pages for a player you have to go to the homepage and then type in their name and then go to the desired page

Actually, you don't need to go back to the homepage. You can click on "Achievement Points", "Quests", "Friends", or "Guild" on the player's left-hand card to quickly navigate between different pages.

huh I didn't notice this, guess this PR is completely redundant. thanks!

Similarly, you could type "arena_1v1 q" in the top searchbar and it will bring you to the quests page for that player. Works all the same with "g" for guild, "f" for friends, etc. Maybe I should make that more clear on my site.

I think a great place to put it is in the tips on the frontpage.


Update cloudinit to 21.2

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

Updates cloud-init to 21.2. Refreshes patches on top of 21.2.
Change Log

Deleted CVE patches, as they have been merged upstream
Rebased remaining patches

Does this affect the toolchain?

NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

local build, vhd deployed and basic cloud init functionality validated, version update confirmed

sounds like the check tests are still passing; can you confirm?

sounds like the check tests are still passing; can you confirm?

If you are referring to this line:
Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
Then imho yes.
To validate, I ran: sudo make build-packages CONFIG_FILE= RUN_CHECK=y from the MarinerDermo repo that has been initialized based on the public instructions from github. Was that sufficient?

Can you please have a look at the linter failure and tell me what is wrong there?

sounds like the check tests are still passing; can you confirm?

If you are referring to this line:
Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
Then imho yes.
To validate, I ran: sudo make build-packages CONFIG_FILE= RUN_CHECK=y from the MarinerDermo repo that has been initialized based on the public instructions from github. Was that sufficient?

Yes that should be sufficient to run the tests. You can open the cloud-init build log (in build/logs/pkggen/rpmbuilding/cloud-init-*.log and you should see the cloud-init test output followed by:
"====== CHECK DONE cloud-init. EXIT STATUS 0"

sounds like the check tests are still passing; can you confirm?

If you are referring to this line:
Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
Then imho yes.
To validate, I ran: sudo make build-packages CONFIG_FILE= RUN_CHECK=y from the MarinerDermo repo that has been initialized based on the public instructions from github. Was that sufficient?

Yes that should be sufficient to run the tests. You can open the cloud-init build log (in build/logs/pkggen/rpmbuilding/cloud-init-*.log and you should see the cloud-init test output followed by:
"====== CHECK DONE cloud-init. EXIT STATUS 0"

Just reran the build to make sure, but I cannot find CHECK in that log. Any thoughts? Is it because I am building this in a derivative repo?

sounds like the check tests are still passing; can you confirm?

If you are referring to this line:
Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
Then imho yes.
To validate, I ran: sudo make build-packages CONFIG_FILE= RUN_CHECK=y from the MarinerDermo repo that has been initialized based on the public instructions from github. Was that sufficient?

Yes that should be sufficient to run the tests. You can open the cloud-init build log (in build/logs/pkggen/rpmbuilding/cloud-init-*.log and you should see the cloud-init test output followed by:
"====== CHECK DONE cloud-init. EXIT STATUS 0"

Just reran the build to make sure, but I cannot find CHECK in that log. Any thoughts? Is it because I am building this in a derivative repo?

In that case the tests must not be running. You might need to force a rebuild with sudo make build-packages CONFIG_FILE= RUN_CHECK=y PACKAGE_BUILD_LIST="cloud-init" PACKAGE_REBUILD_LIST="cloud-init". I tried the updated tests but am seeing some failures.

In that case the tests must not be running. You might need to force a rebuild with sudo make build-packages CONFIG_FILE= RUN_CHECK=y PACKAGE_BUILD_LIST="cloud-init" PACKAGE_REBUILD_LIST="cloud-init". I tried the updated tests but am seeing some failures.

Thanks Andrew, I can see the test run results now and the failures too. Will investigate further. Side question, should we add these notes into the Demo repo readme?

In that case the tests must not be running. You might need to force a rebuild with sudo make build-packages CONFIG_FILE= RUN_CHECK=y PACKAGE_BUILD_LIST="cloud-init" PACKAGE_REBUILD_LIST="cloud-init". I tried the updated tests but am seeing some failures.

Thanks Andrew, I can see the test run results now and the failures too. Will investigate further. Side question, should we add these notes into the Demo repo readme?

Good idea, I'll update the Demo repo readme with check test information.

Closing in favor of https://github.com/microsoft/CBL-Mariner/pull/1405


Rename public in API, so C++ compilers won't complain

Because public is a reserved keyword in C++, we can't use that as an identifier in our public API (because we want that to be include-able in C++ projects).
This series:

Renames the few places where public was used to public_key
Adds a simple test executable whose purpose is to be compiled by a C++ compiler, to check that this compilation works

Haha, I mean, we can go to pub and sec (I think that's what they were before the recent crypto changes?)

It doesn't bother me that much.


Data is not persisted on web via localStorage

Somehow using mmkv on electron does not persist the data. We are using react-native-web, creating a storage.web.ts file fixed this issue for us. Maybe it helps to find the issue...
/* global localStorage */

export default {
  clearAll: () => localStorage.clear(),

  // setters
  setBoolean: (key: string, value: boolean) =>
    localStorage.setItem(key, JSON.stringify(value)),
  setNumber: (key: string, value: number) =>
    localStorage.setItem(key, JSON.stringify(value)),
  setString: (key: string, value: string) =>
    localStorage.setItem(key, JSON.stringify(value)),
  setObj: (key: string, value: object) =>
    localStorage.setItem(key, JSON.stringify(value)),

  // getters
  getAllKeys: () => Object.keys(localStorage),

  getBoolean: (key: string, defaultValue?: boolean) => {
    const value = localStorage.getItem(key);
    if (value == null) {
      return defaultValue;
    }
    return JSON.parse(value);
  },

  getNumber: (key: string, defaultValue?: number) => {
    const value = localStorage.getItem(key);
    return value === null ? defaultValue : JSON.parse(value);
  },

  getString: (key: string, defaultValue?: string) => {
    const value = localStorage.getItem(key);
    return value === null ? defaultValue : JSON.parse(value);
  },

  getObj: (key: string, defaultValue?: object) => {
    const value = localStorage.getItem(key);
    return value === null ? defaultValue : JSON.parse(value);
  },

  // delete
  delete: (key: string) => localStorage.delete(key),
  contains: (key: string) => localStorage.getItem(key) !== null,
};

Hm, weird. Could you set breakpoints there to see if that's being used at all: https://github.com/mrousavy/react-native-mmkv/blob/master/src/createMMKV.web.ts

Hey, related to this issue, interface MMKVConfiguration should be updated since these:

  if (config.encryptionKey != null) {
    throw new Error("MMKV: 'encryptionKey' is not supported on Web!");
  }
  if (config.path != null) {
    throw new Error("MMKV: 'path' is not supported on Web!");
  }

Are compared to null yet their types are only either string or undefined.

@bombillazo This is using !=, not !==. It is not a strict comparison, so it works either way.
See this example.

It logically works, but TypeScript will not like it because it is implicit.

To make it most flexible, how about using if (!config.path)? I think this is the most readable way...

It logically works, but TypeScript will not like it because it is implicit:

Yea if you assign it, yes. I only use null for the implicit comparison. To assign it, I'd use undefined.

I think this is the most readable way...

I disagree, to me that is only a boolean (isValid, !isValid) check, not a nullability check.
I think I just got used to it from C#, where == null or != null are the norm. It doesn't make a difference in code, but we can change it to == undefined or != undefined.

Yeah, I haven't tested it but I'm assuming if I assign them to undefined, I won't get the errors thrown. But there it would work on web as expected.

Hey! Sorry I cannot reproduce the issue here and it has been inactive for a while.
I am going over issues right now and want to keep the repo clean - so I'm closing this for now, if anyone can post a reproduceable example here I can re-open and take a look :)


Can't resolve '../core/config/OperationConfig.json'

Upon forking, running npm i, and npm start, the following output occurs:
 ERROR in ./src/web/index.js     Module not found: Error: Can't resolve '../core/config/OperationConfig.json' in 'C:\Users\Username\Desktop\CyberChef\src\web'      @ ./src/web/index.js 18:0-66 44:35-50
This file doesn't exist in repo, unsure if I am doing something wrong.

Same issue here. What system are you running on.
Running Windows 10 also.

same
On Wed, Oct 2, 2019, 05:04 Kyle Parrish<EMAIL_ADDRESS>wrote:

Same issue here. What system are you running on.
Running Windows 10 also.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gchq/CyberChef/issues/645?email_source=notifications&email_token=AGDFEE66RPW6E6GBQIDF7FLQMSE4PA5CNFSM4IZNKOT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAEQDPI#issuecomment-537461181,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGDFEEYH62YW374TSYIKGCDQMSE4PANCNFSM4IZNKOTQ
.

This has been fixed in #657, thanks to @BjoernAkAManf!
I'll just reiterate what I wrote there:

I can't promise that we won't break this again in future though. The dev environments we are supporting are limited to Linux and Mac. If it works on Windows, great, but that's not the intended target and in my experience we will burn a lot of time if we try to support too many platforms (either for dev or the final product). For a project we're working on in our spare time, I'd rather we focus on features and bug fixes than broad platform support. In an ideal world it would work everywhere, but we don't have the resources for that so we have to choose what's more valuable.
I've added a note on the Getting started wiki page to make this more clear.
Thanks again!


Add a worker for the ingests indexer

This allows the ingests indexer to, well, actually index things. Read off an SQS queue and log; there's no onward message here.

Once this is merged and built by Travis, I can start wiring it up to Terraform. Exciting!


Consider adding libxmljs to this benchmark tool

Yes we all know most benchmarks are problematic. But it would be interesting to see how we compare here.
http://thlorenz.com/exmala/client/

/cc @thlorenz

:+1:

@defunctzombie PRs on examala are welcome :)

Any progress about benchmark?


failing include when the ansistrano_custom_tasks_path is not set

If the variable ansistrano_custom_tasks_path is not set at the playbook level, the include statement fails with:
ERROR: file could not read: /path/ansible/roles/carlosbuenosvinos.ansistrano-deploy/tasks/{{ ansistrano_custom_tasks_path }}/before-update-code.yml

I believe this is because the include: statement is run when the playbook is parsed - before the default variables are loaded - so the default value is never used.
Does it work for you?

I'm currently adding the conditional includes also (based on your approach, good stuff!). But because of the requirement to add the ansistrano_custom_tasks_path variable I've gone a slightly different route:
https://coderwall.com/p/arh7bq/including-a-tasks-file-in-ansible

Thanks for trying and reporting the issue. I'll try your include approach to check if everything works as expected. I've seen your project_deploy project and maybe it could be interesting to merge efforts. What do you think?

I'm all for merging efforts - but what would you suggest? I don't see the role changing a lot anymore, it does what we need from it. The "hooks" where just a final piece that I just could not fit in properly untill now. Inspiration finally came from your task includes, so thank you :-)
PS: Thanks a lot for referencing my blogpost on the role, it's good to know at least one person used it... grin

We could add new deployment strategies (S3, rpm or deb files), resolve opcache atomic deployment problem, write down some examples, etc.

It's fixed with the current approach of specific files

I have adapted my previous ansistrano deployment playbook to the new way and it is working
Are you ok with closing this issue @ramondelafuente ?

:+1:


ci: change how coverage is collected

@oliverchang @another-rex I'm not sure why codecov isn't reporting the actual coverage change here, but I think this is what we want  - compare https://app.codecov.io/gh/google/osv-scanner/tree/change-coverage vs https://app.codecov.io/gh/google/osv-scanner/tree/main/pkg, you'll see this branch has a coverage rating of 77% whereas main has 68%, and there's just a lot more coverage being reported through out the packages.
Note that at least the coverage for cmd has actually dropped, but that's because without this change only packages that actually have tests get included at all - so now the coverage report includes packages like cmd/osv-reporter which have a coverage of 0.
I suspect that codecov hasn't posted a comment because of how different the two reports are, and that it thinks something like the reporting format itself has changed meaning it can't/shouldn't compare what it currently has to this new report. I say we merge this and we can use #524 to check its working properly.

@another-rex @oliverchang yup this looks to be better - have a look over the report before our next catchup; previously this function was completely uncovered, but now its reported as mostly covered (and #524 has it fully covered)


SQL tests can make test execution flaky

Sometimes when running SQL tests, Andy may not find any tests even though there are tests.

Closing, duplicate of #114


simple question about ld2410

Category

[ ] Hardware / PCB
[x] Software / ESPHome

Whats your idea?
...
it seems that in ld2410 yaml, a substitution is declared for
ld2410_max_distance_move: 4.5m
ld2410_max_distance_still: 4.5m
but can not find anywhere it is used for. can i change this value to change the detection range?

Hi,
interesting. I've never noticed. I just removed the substitution and bump the configuration to the latest version (#118).
You can (now) configure ld2410_max_distance_move & ld2410_max_distance_still via HomeAssistant:

You may have to update your the LD2410 firmware first to be capable to use all features: https://www.youtube.com/watch?v=t8-JdVVVH34


No SUSPICIOUS-CONTRACT alert for malicious contracts

BOT ID - 0x9aaa5cd64000e8ba4fa2718a467b90055b70815d60351914cc1cbe89fe1c404c
Txs

https://etherscan.io/tx/0xbc863c288a5eaa6bd0bcc411b546828a2b89deb786063b45aaf32ba280a21501

https://bscscan.com/tx/0xb76f72341c4cac8ecf8b6a7cfd38cfea6ad48ede0d7a5d891ab62e5fc3a14b4e

@salort-forta , any insights yet?

when running the tx, the model doesnt raise any alert; still investigating why.

For the eth transaction:
What I see here is that the top 10 features of the model, only a couple of them are pushing the prediction towards malicious contract. Since most of the features push the prediction towards non-malicious, it is marked as such. My theory is that this is caused by this model having a different structure than the ones on the training dataset. Also, the importance of the features is quite small in both directions, whereas in attacker contracts the values are much higher.
I would recommend adding this sample for retraining of future iterations of the model.
for the bsc:
top features are benign, and have a much higher importance (as an example, the most important benign feature almost cancels the 3 most important negative). I think this model may have a different structure than the ones used in training.

can you elaborate on what the features are? Can you attach your visualization that shows the weights of the opcode sequences?

0xb76f72341c4cac8ecf8b6a7cfd38cfea6ad48ede0d7a5d891ab62e5fc3a14b4e

0xbc863c288a5eaa6bd0bcc411b546828a2b89deb786063b45aaf32ba280a21501


New release coming soon?

Was just hoping you could give some idea of when the next release will be available.  The long-inoperative Musixmatch choice is sorely missed.  Thank you!

In a month? Maybe two? I don't know. I don't work to a schedule, I just make little bits of progress whenever I feel motivated to work on openlyrics. As you can see from the next milestone page though, there isn't a whole lot left that's scheduled to be included in the next release. One of those bugs is already fixed too, I just haven't pushed the code so github hasn't updated the tickets.
In general, if you're looking for a particular fix then it would be helpful if you could link that fix. In this case I assume you're referring to #265 (which is indeed fixed and will be part of the next release). As mentioned in that issue, you can work around the issue by making sure you have the correct Album tags.


Generating website with javascript

Is it possible to generate a website with javascript(without using coffee script), if possible can you refer me a sample project.

There are plenty of info on the wiki and README


Bug 32 numpy conflicts

Purpose
closes issue #32
Implementation
Uses oldest supported numpy metapackage for building.  According to the numpy docs, this is the most consistent way to avoid conflicting numpy versions.
Remaining issues
Not a big fan about how dependent we are on a metapackage for dealing with numpy conflicts, but that's more of a personal gripe than anything serious.

Is there a reason that these old PRs keep added as changes to dev?
Do we need to merge main into dev separately?

last one should've been merged to main from dev, my b.  The pr->dev->main just makes sure I bump the version in setup.py and make it a tagged release

Right, but #24 and #28 are already merged into main, so why are they showing up in the change log here?

I think those where directly merged into main, but weren't merged into dev first. since this is being merged into dev first, they're showing up here in the change log, since those changes aren't currently in dev

Got it


testkit weird org discrepancy and version

I need to import those:
  "org.scanamo" %% "scanamo"         % "1.0.0-M10",
  "com.gu"      %% "scanamo-testkit" % "1.0.0-M8" % Test

The version and the organization are not matching.
  "org.scanamo" %% "scanamo-testkit"         % "1.0.0-M10",

does not seem to exist

Hello - hmm, this is weird, the package is clearly there https://search.maven.org/artifact/org.scanamo/scanamo-testkit_2.12/1.0.0-M10/jar

What about 2.11 ?

hmm its there too https://search.maven.org/artifact/org.scanamo/scanamo-testkit_2.11/1.0.0-M10/jar


Attach data to post for spam detection package.

The underscores look weird… is there a specific reason why the property names are not in camelCase?


Implementation of Music Module

This includes:

Implementation of test for music module
Implementation of the music module
Addition of songs data file for music module
Addition of albums data file for music module
Addition of artists data file for music module
Addition of genres data file for music module

+1


Updated the 'About Us' making it more descriptive and engaging

Related Issue
None
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #(issue no.)
#1483
Description
changed the content of the about us in the contact us page to let the viewers have a better understanding of the website and a have a better impression.

Type of PR
content enhancement
Screenshots / videos (if applicable)
before
https://1drv.ms/i/c/3eafba334c127cda/EcaAU7XAvTdDud03msQEkt0BO6MksSA8ZP9DBd2M6D-BAA?e=14XLBf
After
https://1drv.ms/i/c/3eafba334c127cda/Ee0VwzDZDNhMsZxFO8QasoEBNXzaHoUOVTzjqcA3OFnuvQ?e=NZDbYD

[ x] I have made this change from my own.
[ x] My code follows the style guidelines of this project.
[ x] I have performed a self-review of my own code.
[ x] My changes generate no new warnings.
[ x] I have tested the changes thoroughly before submitting this pull request.
[ x] I have provided relevant issue numbers and screenshots after making the changes.

@PreetiTamana  Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. Exclusive swag awaits for those who go the extra mile! Also Take a moment to ⭐ Star the repository if you haven't already.
Great contribution ! 🎉


wired-tabs with nested wired-elements not working properly

if you put some wired-element (tested with button and tabs) inside of wired-tabs then any element, located not on initially active tab won't be rendered properly (e.g. buttons won't have their borders, e.t.c.)
An example here: https://codesandbox.io/s/48l9592k9x

Ah yeah. Some background around this:
Wired elements need to know their size to draw sketchy lines around them.
When not visible the size is not available.
As a work around you can call requestUpdate() method on the buttons when tab is visible.
I know this is not a viable workaround - will resolve this properly soon.

Dev note: Perhaps add a trigger to parse down the tree to re-render

same issue when use them in react app created by create-react-app
I was trying to put a wired-link inside a wired-card
then the wired-link would not show at all
`import React from "react";
import "wired-card";
import "wired-link";
const Home = () => {
return (

askjdhakjshdkjash

this is wired card
this is wired link

);
};
export default Home;
`

same issue when use them in my react app created by create-react-app
I was trying to put a wired-link inside a wired-card
then the wired-link would not show at all

this is a wired card
this is a wired link

resolved in 2.1.2


Where is this book's source?

Nice book!
I mean it looks like this book is written by Latex, and where is the Latex source code of this book?
If someone has found an issue, he can simply make a pull request to fix the error in the book.

Yeah, this book was written using LaTeX.
However, we currently have reservations about releasing the source files due to concerns that certain individuals might recompile them, tamper with the copyright, or exploit our work for commercial purposes. Such actions could potentially bring some trouble to us.
If you find any issues, you can simply raise them on this page, and we will address them in the v2 version, where we will also include some new original content.


Adds Nek5000 air jacket tutorial

This PR adds the Nek5000 air jacket tutorial. This is a natural-convection based 2D RANS model. The documentation and input files are provided, and an initial condition file restart_airjacket.fld and the mesh airjacket.re2 have been added using Github LFS.
@aprilnovak @dshaver-ANL @eshemon please share your feedback whenever possible. Thanks.
EDIT: issue #264 can be closed once this PR is merged.

@anshchaube for the git lfs to work properly, you need to squash the commits. Otherwise the humongous file is still present in the git history, so it doesn't actually save any storage.

I was trying to get it to work first by removing the files, then re-adding with git lfs to address Guillaume's comment, and finally squashing last... are you saying I need to squash the commits first?

If you

Add the big files
Remove them

They still exist in the git history.

@aprilnovak thanks. I knew that squashing would be necessary as soon as I saw git lfs failed, my question was more about why git lfs wasn't working for me, and whether me not squashing right away was affecting that.
Anyway, that seems to have been fixed, and the commits have been squashed. @aprilnovak @GiudGiud please let me know if you have further feedback.

@GiudGiud PR feedback should be addressed now. Please let me know if you have further comments.

@GiudGiud feedback implemented, please let me know if you have further comments. thanks.

Job VTB Documentation on c849693 wanted to post the following:
View the site here
This comment will be updated on new commits.

Looks good thanks for the contribution!
Testing of Nek inputs is still WIP, I ll be in touch when we start doing that

Thanks @anshchaube @GiudGiud !


Markdown reporter

Is there an explicit markdown reporter? The text reporter is close, but has an intial row that doesn't get included in the table and it is missing the outside | and so doesn't quite work if you output the text in a Github PR comment.

Link to bug demonstration repository

N/A
Expected Behavior
| File                      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s              |
| ------------------------- | ------- | -------- | ------- | ------- | ------------------------------ |
| All files                 |    62.6 |    29.06 |      66 |   61.91 |                                |

GitHub output:

File
% Stmts
% Branch
% Funcs
% Lines
Uncovered Line #s

All files
62.6
29.06
66
61.91

Observed Behavior
--------------------------|---------|----------|---------|---------|--------------------------------
File                      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
--------------------------|---------|----------|---------|---------|--------------------------------
All files                 |    62.6 |    29.06 |      66 |   61.91 |

GitHub output:
--------------------------|---------|----------|---------|---------|--------------------------------

File
% Stmts
% Branch
% Funcs
% Lines
Uncovered Line #s

All files
62.6
29.06
66
61.91

Troubleshooting steps
The main proble is the initinal row. Even though the rest of the table isn't a strictly formatted table, it does get formatted.
Environment Information

N/A

Hmmm I'll take a crack at this! 🙂


Page errors

Those pages are containing errors.
Links:
https://hrpg.lscult.com/admin/view/tests   < some kind of database error.
https://hrpg.lscult.com/admin/view/attacks   < error, database i think.
https://hrpg.lscult.com/admin/view/errors    < ironically enough errors show a database error :))
https://hrpg.lscult.com/profile   <weird overlays?
https://hrpg.lscult.com/forum    <forums with "n" threads. Where threads?
https://hrpg.lscult.com/admin/view/shop   < 404
https://hrpg.lscult.com/admin/view/attacks/hacker/18150       < error with db, might be related to second link.
https://hrpg.lscult.com/admin/view/attacks/hacker/18150/logs/show      <same as above
https://hrpg.lscult.com/support   < where captcha?
https://hrpg.lscult.com/alpha_coins  < smol error.
https://hrpg.lscult.com/admin/view/manageQuest/qid/82   < weirdly formatted on mobile.
https://hrpg.lscult.com/social < 404
https://hrpg.lscult.com/faq/?p=14  <404
https://hrpg.lscult.com/skills///  < skills not increasing when spending points.

Will not deal with layout issues - some may be because that's how I originally thought it looked good and it doesn't haha
For faq that might be legacy code, removed the text that points to it: https://github.com/nenuadrian/Secret-Republic-Hacking-Browser-Game-V3/commit/c86f4f6cb4e364894a952553b07314f8ff7acfab
Frequently asked questions exists but not content for formatting text anymore: https://hrpg.lscult.com/frequently-asked-questions

For captcha, have you created and added your key here? https://github.com/nenuadrian/Secret-Republic-Hacking-Browser-Game-V3/blob/master/includes/constants/constants.php#L20

For forum threads, I have deleted all of them before putting the game as open-source. But I forgot to update some of the database details so the numbers are correct.
Fixed now in SQL: https://github.com/nenuadrian/Secret-Republic-Hacking-Browser-Game-V3/commit/dc123cca3504bd92a605740d5dc498bd2a618ff2

I have deleted the social integration to not support it, so deleting any references to it

There was a bug on the skills page! Fixed it on your host and in the repository, also formatter the file so you can't see exactly where but it was
$skillsData = $this->db->where('uid', $uid)->where('skill', $skill)
Should have been
    $skillsData = $this->db->where('uid', $uid)

If you make me an admin (cardinal), I can have a look at the admin pages.

You would also need to update the keys in public_html/includes/class/recaptchalib.php at the top for captcha to work i expect - might remove it entirely and leave it to those that want it to add it

Tests for admin are not a thing anymore - deleting it

Fixed the admin attacks page: https://github.com/nenuadrian/Secret-Republic-Hacking-Browser-Game-V3/commit/444dd5a54b9591637043f63326cac9c4358568c0

Fixed admin debug errors, debug_errors table is called debug_404_errors

https://hrpg.lscult.com/admin/view/shop < 404
This link should just not haven been there: https://github.com/nenuadrian/Secret-Republic-Hacking-Browser-Game-V3/commit/5be024a3c4fe7ed9ab57fabc6eca144b32ea80b6

So I believe I have addressed everything other than:
https://hrpg.lscult.com/alpha_coins < smol error.
If you can please open new issue with how to reproduce!


Document logging configuration options

We don't have documentation for logging-related configuration such as:

log.max-history
log.max-size
http-server.log.max-history
http-server.log.max-size

Prepared a doc PR but there doesnt seem to be any of this config in the presto code base .. is this part of the airlift config basically so I can look it up there?

There is lots more e.g. https://github.com/airlift/airlift/blob/418b0adea11560b81330bf750f4b42a173848d97/http-server/src/main/java/io/airlift/http/server/HttpServerConfig.java
Maybe we should see what sort of docs we create for airlift as well.. and then just refer to it.

Maybe we should see what sort of docs we create for airlift as well.. and then just refer to it.

@mosabua see https://github.com/airlift/airlift/issues/753, https://github.com/airlift/airlift/issues/417, and https://prestosql.slack.com/archives/CFLB9AMBN/p1567834707126800?thread_ts=1567726032.112300&cid=CFLB9AMBN.
Quoting @electrum

For user facing config like HTTP server, that should be part of Presto documentation. Users shouldn't have to look elsewhere.

Also l looked at https://github.com/airlift/airlift/blob/ca3cb480e1d052c960bf969dd0e90623d7ad0be0/log-manager/src/main/java/io/airlift/log/LoggingConfiguration.java


feat: when user clicks on any pandal search result - take them to /app/pandals and pan the map on that location marker

Is your feature request related to a problem? Please describe.
Currently, the map panning feature only works on /app/pandals page. We want the search-to-map feature to also work on other pages which have the searchbar
Describe the solution you'd like to see
Behaviour on "Home" and "Help" tab searchbars should be same as on "Pandals" tab
Describe alternate solutions
N/A
Additional information
No response

i m looking into this