Feedback/Contact form and map

Description

Contact from added
map added
Navbar Updated

Related issue
#93
Proposed Changes

A feedback/contact form with the following details addded:

Name
Email
Contact
Message
Captcha (Remaining - will be added by @himanshujaidka )

Map added by using google api
Navbar updated in home, editor and contact page

Screenshot

Note : If any change is required let us know. Thanks :)

@smaranjitghose please merge this PR so that I am able to propose changes

Try using graphics from undraw.co
Use google maps for this https://developers.google.com/maps/documentation/javascript/examples/

Hey @anushbhatia sir Can you tell me specifically coz there are lots of options available in map...

@nooras While you make this the Contact Page, Don't forget to make the current page named as Contact as Team Page, and update the navbar as well

How is this right now ? @anushbhatia @smaranjitghose
Navbar(Desktop+Mobile) and contact page done
(Map UI is remaining)
(Desktop)

Navbar hover

(Mobile Version Navbar)

Map added :

Night mode map added
Animated marker is added

Note : But there is some issue in my apikey.... below image is stays only for 1-2 sec after that I will got error maybe due to billing info.

If any changes is required so let us know. Thanks @anushbhatia @smaranjitghose

Map added :

Night mode map added
Animated marker is added

Note : But there is some issue in my apikey.... below image is stays only for 1-2 sec after that I will got error maybe due to billing info.
If any changes is required so let us know. Thanks @anushbhatia @smaranjitghose

I still think the map doesn't look good.
Also, do we really need a map here? @anushbhatia  @smaranjitghose

New Map Api Mapbox added @anushbhatia @smaranjitghose
Have a look...

New Map Api Mapbox added @anushbhatia @smaranjitghose
Have a look...

@nooras The Feedback form is for a separate page
So the changes should be:

Add a separate contact.html, contact.css
Update the navbar for all pages

Map on right with a subheading of Made in India, and a contact form on the right
Refer this

I quit with this task...

What happen noora?

@nooras any updates?


Fix an issue so that it only pulls the status for the defined interval

Problem:
Currently it's continuously requesting the status from github ignoring the global interval of 30 seconds
Tasks:

[ ] Update so that it only requests updates for the set interval or default 30 seconds

hey @haroon-sheikh I would like to work on this issue

Hi @RacHiT101, please go ahead and let me know if you have any questions.

i got caught in a loop and exceeded the Api rate limit


C/C++ Intellisense is showing error for to_string command which was later added to c++ library

VSCode 1.53.0 (user setup)
OS Version:Version: Windows_NT x64 10.0.16299

Steps to Reproduce:
1.Go to the extension settings and enable error squiggles
2.Make sure that the intellisense engine is default and not tag parser
3. Use the the function to_string and the error occurs saying "indentifier to_string is undefined"

Does this issue occur when all extensions are disabled?: Yes/No
No
error.docx

Sorry it was my other settings problem. Was ble to solve it


Surveys

Implement surveys and a solution for app to listen to changes

Do we have a background for this. It looks ok but it's hard to understand why it's there :D

Actually something I hadn't considered yet, so will update this tomorrow.

Also update the migration numbers, I just had to add one


Fail with undefined 403 on windows-latest

On my windows-latest action of private repository, it shows:
Run softprops/action-gh-release@v1
👩🏭 Creating new GitHub release for tag CDTest0.0.3...
⚠ GitHub release failed with status: 403
undefined
retrying... (2 retries remaining)
👩🏭 Creating new GitHub release for tag CDTest0.0.3...
⚠ GitHub release failed with status: 403
undefined
retrying... (1 retries remaining)
👩🏭 Creating new GitHub release for tag CDTest0.0.3...
⚠ GitHub release failed with status: 403
undefined
retrying... (0 retries remaining)
❌ Too many retries. Aborting...
Error: Too many retries.

My config is:
name: MSBuild

on:
  push:
    tags:
      - '*'

env:
  # Path to the solution file relative to the root of the project.
  SOLUTION_FILE_PATH: .

  # Configuration type to build.
  # You can convert this to a build matrix if you need coverage of multiple configuration types.
  # https://docs.github.com/actions/learn-github-actions/managing-complex-workflows#using-a-build-matrix
  BUILD_CONFIGURATION: Debug

permissions:
  contents: read

jobs:
  build:
    runs-on: windows-latest

    steps:
    - uses: actions/checkout@v3

    - name: Add MSBuild to PATH
      uses<EMAIL_ADDRESS>
    - name: Restore NuGet packages
      working-directory: ${{env.GITHUB_WORKSPACE}}
      run: nuget restore ${{env.SOLUTION_FILE_PATH}}

    - name: Build
      working-directory: ${{env.GITHUB_WORKSPACE}}
      # Add additional options to the MSBuild command line here (like platform or verbosity level).
      # See https://docs.microsoft.com/visualstudio/msbuild/msbuild-command-line-reference
      run: msbuild /m /p:Configuration=${{env.BUILD_CONFIGURATION}} ${{env.SOLUTION_FILE_PATH}}
    - name: Package
      run: 7z a win10_debug_x64.zip x64
    - name: Release
      uses: softprops/action-gh-release@v1
      with:
        body: ${{ github.event.head_commit.message }}
        files: |
          win10_debug_x64.zip
      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Refer to previous issues: https://github.com/softprops/action-gh-release/issues/94#issue-854973442, which is led by the wrong permission setup, I have checked that it is configured correctly.

I found out the issue:
The default MSBuild template includes a
permissions:
  contents: read

Please remove it.


json search engine support

add json option to local search.
TODO:

search with tag and catalog
doc update

@flashlab can u please give some comments and examples how to use it?

In the local search field of config file, 'search.xml' will end with xml database generated while 'search.json' comes with the json. @ivan-nginx

@flashlab and? How to use json in search? Need something like documentation to test it for merge.

Update generator plugin to the latest version. @ivan-nginx

@flashlab there are 2 generators for search: hexo-generator-search and hexo-generator-searchdb. Both support json?

yes. but only the latter one is compatible with Next.

@flashlab need to update hexo-generator-search to latest version, which is automatically support json objects, right?

but only the latter one is compatible with Next
So, hexo-generator-searchdb for now is not compatible with Next, right?

No, I mean hexo-genrator-searchdb is compatible. the latest vesion(1.0.5) support json.

@flashlab ok, lets repeat it:

This pull is support compatible with json objects for Next theme.
To use this json compatible, need to setup hexo-generator-search-db instead of original hexo-generator-search.

It is right?

P.S. Also, can u, please, rename path from
https://github.com/flashlab/hexo-generator-search to
https://github.com/flashlab/hexo-generator-searchdb
to see in google what it's modification and not just original fork?

@ivan-nginx it's forked because it was modified from the original one to work better with Next. No need to initiate a new path I think.

@flashlab ok, what about last question?

It is right?

Absolutely right. And we may continue this conversation in chatter;)

And we may continue this conversation in gitter

Ok, check gitter, there is 1 bug.

There is some instructions for use hexo-generator-search-db with this pull request. Check your NodeJS and NPM versions.

NodeJS > 4.2 & NPM > 3: just follow this instructions.
NodeJS < 4.2 & NPM < 3: need to install separated patch with npm install<EMAIL_ADDRESS>command. You may see this patch in [old-nodejs](https://github.com/flashlab/hexo-generator-search/tree/old-nodejs) branch and follow same instructions.


[GLUTEN-1632][CH]Daily Update Clickhouse Version (20230531)

Auto commit by gluten daily build, please check the build status and merge it if it's green.

It's compatiable with current gluten


[Documentation]: Add release document

This should mention the process for releasing
Tasks are SDK team is responsible for and tasks that partner teams are responsible for a successful release

This will be part of the on-boarding documentation.


Whitespace not preserved between multiple primary associated types

In swift-format 0.50700.1, spaces are not preserved between primary associated types the same way they are preserved between generic parameters.
$ swift format -v
0.50700.1
$ echo "struct A<B, C> {}" | swift format format
struct A<B, C> {}
$ echo "protocol A<B, C> {}" | swift format format
protocol A<B,C> {}

That was fast! 🎉


Document all possible error reasons

We should document all possible error reasons in Mint.TransportError and Mint.HTTPError. For Mint.HTTPError, since each module defines its own error reasons, what we can do is have a @type error_reason() :: ... in each module with documentation for it and just link to those typespecs from Mint.HTTPError.

Closed by #141, #143, #144, #146.


Allowing Sequences to be maintained using incremental //// CHANGE scripts

I'm trying to add a Sequence to an Oracle database. Oracle's SQL dialect for Sequences is very much like tables: CREATE SEQUENCE, ALTER SEQUENCE. Without this functionality, I don't know how to manage Sequences with Obevo, which could be a show-stopper for using Obevo.
Expected Behavior
Manage Oracle Sequences like Tables with CHANGE sections to CREATE and ALTER incrementally.
Actual Behavior
If I try to set a //// CHANGE section, Obevo complains that: Unexpected sections found: [CHANGE]; only expecting: [METADATA, BODY, DROP_COMMAND]
Obevo Version where this issue was observed
6.6.0
Steps to reproduce the behavior

Initialize an Oracle database
Initialize Obevo
Create a SEQUENCE file with a CHANGE section
Try to add an ALTER section inside

We can add support to support sequences in an incremental manner. As of today, sequences are treated like other rerunnable objects like views/sps, which are "drop/create" or "create or replace"
It wouldn't be difficult to add this support, but in the interim, you can use the migration functionality - https://goldmansachs.github.io/obevo/db-project-structure.html#Ad-hoc_data_migrations
In the meantime, can you please send examples of the create/alter sequence operations that you do, for us to add to our test cases?

Hi there.
Thanks for getting back to me. I found that moving the SEQUENCE file to the table/ directory, Obevo thinks it’s a table and is creating it correctly. I’m not sure if I’ll keep it there (or how I'll migrate afterwards), but it is operating correctly for now.

Thanks!

Sounds good. This should suffice for you for now. You may run into issues if you need to drop this sequence permanently; I'll look to document the workaround for you in case that comes to pass (though I assume it would be less likely to happen)
Will keep this ticket open to account for 1) supporting sequences in an incremental fashion  2) publicly documenting the workaround to remove unneeded table files (your use case isn't the first time we've encountered this)

Something else I just tried is to add the SEQUENCE creation as part of the table it's for. Not my favorite because now we're mixing control of two objects into one file. But it's a little cleaner option than having sequences definition files mixed in with the table definition files in the tables/ directory.

I do like this option a bit better. Still gets tricky on drops, but better than having it as a separate table.
Another option - manage it as a migration; note the "dependencies" attribute usage in MYTABLE. Here, you can manage it as a separate file. Only downside is that migration scripts can be dropped, so this is slightly less safe, but you will get the incremental behavior that you want (and easier to move to the strategic solution of having this in the /sequence folder once that functionality is ready)
/migration/PERSON_IQ_SEQ.sql
//// CHANGE name=init
CREATE SEQUENCE "PERSON_ID_SEQ"  MINVALUE 1 MAXVALUE<PHONE_NUMBER>99999999999999999 INCREMENT BY 1
GO

//// CHANGE name=reset_increment
ALTER SEQUENCE PERSON_ID_SEQ INCREMENT BY 5
GO

/table/MYTABLE.sql
//// CHANGE name=init dependencies="PERSON_ID_SEQ.init"
CREATE TABLE MYTABLE ( ... using PERSON_ID_SEQ somehow ...)
GO

//// CHANGE name=reset_increment
ALTER TABLE MYTABLE ...
GO

To confirm the requirements on this:

We should allow sequences to be maintained in an incremental fashion, using  //// CHANGE entries

We should keep backwards-compatibility with existing deployments that already have sequence files defined

We should have an easy migration path for folks that want to convert their old rerunnable sequences to the incremental sequences, without having to run the INIT command or anything

It would be preferable to keep the incremental managed sequences in the same /sequences folder, but I can be flexible on this depending on the implementation difficulty. Note that keeping it in separate folders (akin to what we did with triggers) may not work as easily here


cleanup: Remove unneeded _index.md files from docs

This commit removes the doc/service/<name>/_index.md files for service that provide a README.md in their own directory.
The CI's docs pipeline generates and publishes (to the docs branch) the _index.md files from the READMEs. We don't need to have these files duplicated master branch of the git repo. They just cause confusion and are not kept in sync with the READMEs on the master branch.

@mmattel @kobergj After our chat earlier I think this should be safe (no content is lost). And also cleanup some confusion regarding README.md vs. _index.md. Please holler if you  disagree.

Imho we should set this temporarily on draft for the following reasons:

The cleanup is not complete
It is not only about _index.md files but includes, historically driven, if present, supporting files too, see the settings service as example. These files were included by the index file and can contain content that is linked. Without having the index file, we do not longer know how that was related.
We need to check what is in the docs branch as supporting files compared to these files here.
When we have fixed all that above, we could kill all content that is in docs/services/xxx/<all here> by extending this PR and do not longer have any orphans.

Finally when this PR is merged and one runs locally a make docs-generate, he gets clean data generated he MUST NOT push but can use for testing the automatism.

UPDATE: Post a discussion with @rhafer, following clarification was made:

Supporting files are added by the CI to the dev docs and appear via table of contents.
By merging this PR, we have now two locations for content which are:

servcies/<service_name>/README.md
docs/servcies/<service_name>/<additional_files>

Fixing the two location issue should be made in a follow up PR.
Approving this PR therefore.


Can't set rating

I'm unable to set rating through binding and through code.
Is this not supported?

Should be supported. Can you please give out more info? and, if possible,
will you be able to upload a demo that I can try?
2017-05-13 10:21 GMT-05:00 Hjord<EMAIL_ADDRESS>
I'm unable to set rating through binding and through code.
Is this not supported?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/messier16/Forms.Controls/issues/12, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABvJEpBDfncTvvSenKEs5nTdpT7tMIgXks5r5cp0gaJpZM4NaGer
.

--
Antonio Feregrino Bolaños |:)
I do programming stuff!
Blog: http://thatcsharpguy.com
A silent smile, a private kind of happiness*.*
Signature powered by
http://www.wisestamp.com/email-install?utm_source=extension&utm_medium=email&utm_campaign=footer
WiseStamp
http://www.wisestamp.com/email-install?utm_source=extension&utm_medium=email&utm_campaign=footer

It's not much code, so i just post it here if thats ok.
<controls:RatingBar
            x:Name="ratingBar"
            FillColor="Gold"
            HeightRequest="50"
            MaxRating="5" />
<Button
            Clicked="Button_OnClicked"
            Text="Set" />

private void Button_OnClicked(object sender, EventArgs e)
{
            ratingBar.Rating = 3;
}

And nothing happens, same if i do it in OnAppearing()
It works if i do it in the constructor, but in my use case i can't do that.
Am I missing something?

I've only seen this issue on Android, and it seems like it has been fixed in the latest version of the package. Please check it out and do let me know if the issue presists.

Just to confirm, version 0.5.1 fixed it for me.
Thank you.

:)


Add connection-based configuration option that toggles automatic CCSID manipulation

Is your feature request related to a problem? Please describe.
This request is related to an issue I encountered and reported in:
All files in stream file deployment path blindly set to CCSID 1208 after deploy #2126.
Basically, the feature delivered with PR: Set CCSID of deployed files to 1208 if needed #2015 causes the CCSID of every file in the deployment path (including subdirectories) to be changed to 1208.
I have a use case where some files in the deploy hierarchy must maintain specific CCSIDs.
Describe the solution you'd like
I would like to have a new connection-based configuration option that toggles the automatic CCSID manipulation.
Describe alternatives you've considered

Restructure legacy project to avoid issues (not possible)
Post-deployment re-manipulation of CCSIDs

Additional context
Steps to reproduce the issue are provided in the original issue:
All files in stream file deployment path blindly set to CCSID 1208 after deploy #2126

[ ] New option in connection settings to set CCSID automatically to UTF8 (enabled by default)
[ ] add logic to check if new option is enabled here: https://github.com/codefori/vscode-ibmi/blob/master/src/api/local/deployment.ts#L243


Fix test directories not cleared

Fix TestBuilders classes not cleared after a gql fragment change

@asimonigh: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

Good catch 👍

good catch from @martinbonnin ;)


disableInitialLoad option, register slot callback, and npm audit

This PR allows you to enable DFP's disableInitialLoad option [link]. Also added a a callback prop that allows you to fire an ad bidding function before refreshing the ad with DFPManager.refresh([options.slotId]).
The original purpose of this PR is to get react-dfp to integrate with ad bidding systems such as Amazon's Transparent Ad Marketplace via APS [link].

@jaanauati code review comments addressed. Can you review? Hoping to get this merged and on NPM before our new site launches :]

LGTM!,
given that we are adding a new feature I think we should set version=0.15.0, don´t you agree?


Hundreds of extra empty divs in .vjs-control-text

Hello and thank you for your amazing job.
I have a problematic issue you can reproduce on the simple example : while buffering any youtube video hundreds of empty divs are added here :

Hundreds of div are appened to this div as buffering goes and it drastically slows down the browser causing small freezes and performances issues.
Any idea ?

No idea why this is happening. Do you have the same issue with barebone videojs?

No problem with barebone videojs, you can reproduce this easily by checking it on your examples/simple.html file. I guess this has something to do with an event getting triggered on buffering that causes videojs to append this empty div (which is used to display the load progress bar).
I'll working on a fix for it, will tell you if i can find one.

I've been able to work around this issue by commenting Lines 5494 to 5507 in video.js
 // add child elements to represent the individual buffered time ranges
 /*for (var i = 0; i < buffered.length; i++) {
      var start = buffered.start(i);
      var end = buffered.end(i);
      var part = children[i];

      if (!part) {
        part = this.el_.appendChild(Dom.createEl());
      }

      // set the percent based on the width of the progress bar (bufferedEnd)
      part.style.left = percentify(start, bufferedEnd);
      part.style.width = percentify(end - start, bufferedEnd);
    }*/

Child elements are added (i counted up to 5.000+ empty div for a 10 min long youtube video) but not correctly removed. Not sure if this part of code is really used and should probably be removed from the main version of video.js as the only thing this method (which is LoadProgressBar.prototype.update) should do is update the progress bar's percentage.
Anyway this fix will do it for me :)

I'm guessing this is some weird interaction between videojs and youtube where we're adding a buffered indicator fo each buffered range.

I belive this is fixed by #411.

Available in 2.2.0.


lower max firestacks to 10, refactor flammable

About the PR

Lowers max firestacks from 20 > 10
Uncaps the damage limit of 5 fire stacks
Lowers heat damage from fire stacks to be the same as before at most

Less fire stacks will do less damage however

Why / Balance

fires don't last as long
passive fire reduction unchanged

Changelog
:cl: Whisper

tweak:

i think mass scaling is better to fix liltenhead ops

also damage was capped at 5 firestacks, any more than that just makes it last longer

also damage was capped at 5 firestacks, any more than that just makes it last longer

I see that now, and that's a little weird

still dont think this is needed:

if you are aware you are on fire you can stop drop and roll
if you are dead someone can splash water on you
if someone wants to burn you to a crisp a single 20 stack wont do it with most armour so theyll still have to go for seconds anyway

I'm going to uncap it since the cap is an arbitrary number and reduce fire damage

Ok so at the time of writing this comment, a max stack fire burns out at 1471 heat, which would mean armored people do not burn to ash but unarmored are nearly ashed.
Antagonists could ash people still by simply adding more fuel at least once


Fix inconsistent doc for Sequencer:remember()

Also I don't really understand why eval is recommended for RNNs and both is recommended for LSTMs.
I would think that the mode depends more on the application than the implementation (RNN, LSTM, GRU...).
If so, I think the recommandation is misleading.
If not, what am I missing ?

@vgire I just recommended that on my own experience. Doing both for RNNs would seem to yield very unstable rnns. But this may have changed since my tests (the library has improved).

@vgire both is the default mode right? So why make it less obvious by removing the (the default) mention from the docs?

Ah ok, I jut got it !
neither is the default mode of the Sequencer module while both is the default value for the
Sequencer:remember() method (which makes sens in the scope of the remember method).
Sorry for the confusion.


Error viewing replay with CityFlow (console.log showing roadnet is undefined)

i managed to use CityFlow to test the replay with the example at https://github.com/cityflow-project/data/tree/master/frontend/replay
however, i get the error message "roadnet is undefined" when trying with roadnet_6_6.json used by coLight https://github.com/wingsweihua/colight/tree/master/data/template_lsr/6_6
the content in both the json file looks kind of different.

The input roadnet and output roadnet of CityFlow are different. You should use the output roadnet of Cityflow in the replay website index.html.


Fixed 'show settings' issue #4

Fixes Issue #4
Description
Fixed the double click issue
Testing
Try to click on settings icon in any widget after changing settings (e.g. Ticker)
Risk
No breaking changes
Do you agree with those?

[+ ] I agree to follow the Contributing guidelines
[+ ] I agree to follow the terms of the Code of Conduct

Also, fixed typo in menu


Use the atom-text-editor tag instead of the editor class.

In styles/red-wavy-underline.less: Use the atom-text-editor tag instead of the editor class.

Thanks! Duplicate of #8.


isFileExists is on a loop after file is downloaded

Hi!
Thanks for this amazing plugin!
However, i am facing an issue. Will describe in brief whats going on.
There is a isFileExist method used in the plugin, which checks for the file at the download location but doesn't register or assert that the file is found.
I physically checked during runtime, File is actually present.
And more over, the isFileExist is in a loop.

Hey,
Thanks for the well-described issue.
The isFileExist function is pretty straightforward, it uses existsSync function and that's it. So, I believe that the problem is in the file name or path to the file. Moreover, as I can see in the screenshot there are some differences. It logs that 'Waiting for the report_12/07/2021.pdf file' but it tries to find report_12-08-2021.pdf.
- report_12/07/2021.pdf
+ report_12-08-2021.pdf

Hey, Thanks for pointing it out! And sorry for taking your time for such a small issue...
Thanks again!


removed auto play on AVAudioSessionRouteChangeReasonNewDeviceAvailable

fixes #423

@zmxv @schumannd - any reason this was merged in? It seems like it removed a feature because of an issue in an external library (that apparently already has issues, see #423)

The original feature wasn't properly implemented and should be at least guarded by a flag.


Styling appears broken on @nuxtjs/sanity docs website

Hi, I wanted to report that the styling on the @nuxtjs/sanity docs website appears to be broken. The website is functional but the styling is not being applied correctly, making the content difficult to read.
https://sanity.nuxtjs.org/
Thanks in advance!

thanks for flagging! should be resolved with a lockfile update + a hotfix in pinceau


[CPREQ-9284] k8s and 3rd party update

Description
New kubernetes version v1.28.12, v1.29.7 & v1.30.3 should be supported
New plugins versions should be adopted.
*
Solution
Adopt latest release of kubernetes, calico, local-path-provisioner and nginx-ingress-controller
*
How to apply
Provide steps how to apply on top previous Kubemarine version to execute on existing clusters

Install task/s
Upgrade procedure

Test Cases
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.
TestCase 1
Steps:

Kubemarine install

Results:

Before
After

Not applicable
Success

Kubemarine upgrade

Results:

Before
After

Not applicable
Success

Checklist

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] Integration CI passed
[ ] Unit tests. If Yes list of new/changed tests with brief description
[ ] There is no merge conflicts

Unit tests
Indicate new or changed unit tests and what they do, if any.

I'm not sure that we need Calico v3.28.0 original YAMLs

It's not needed, since it not used in the compatibility map


% should do format on (string,number)

Vyxal, 33 bytes
`Hi I'm Bob, I'm % years old` 5 %

Try it Online!
Should do "Hi I'm Bob, I'm 5 years old"

% is already supposed to do exactly this:
https://github.com/Vyxal/Vyxal/blob/29e1b81a91e8c1a127976c68aa366c7a42993ed5/vyxal/elements.py#L2139-L2144
https://github.com/Vyxal/Vyxal/blob/29e1b81a91e8c1a127976c68aa366c7a42993ed5/vyxal/helpers.py#L149
It must just not be working.

@Lyxal how is this not a bug?

 (num, str) -> (b split into a equal pieces)[-1]
 (str, num) -> (a split into b equal pieces)[-1]

@chunkybanana is suggesting to change the current overload to something more useful.

Oh I'm dumb, you're talking about formatting with numbers, not strings


How can I use gpu_ids=5,6,7 to do distributed training(not from the gpu=0 to do distributed training)?

I want to use CUDA_VISIBLE_DEVICES=5,6,7 to do distributed learning, however, I found it still run from the first gpu.

CUDA_VISIBLE_DEVICES=4, 5,6,7 GPUS=4 GPUS_PER_NODE=4 CPUS_PER_TASK=2 . tools/slurm_train.sh

@puhan123 You can refer to https://github.com/open-mmlab/mmdetection/blob/master/docs/en/1_exist_data_model.md#training-on-multiple-gpus

Thank you very much. This answer solves my confusion.


Support psr/http-message v2

There is an increasing number of libraries requiring v2, resulting in conflicts
I see there already is a MR
https://github.com/slimphp/Slim-Http/pull/237

Version 1.4.0 is now released.


Line numbers aren't propagating in errors

When using @chain, line numbers are given at the level of the @chain command, not sub-commands. This makes it hard to debug. Consider the following script
using Chain

foo(x, y) = x * y

x = [1, 2]

@chain x begin
    identity
    identity
    identity
    foo([3, 4])
end

The stacktrace says the error occurs starting on line 7. This makes it hard to debug long @chains. We should try to ensure line number propogation within the chain block.

works for me in vscode:

Line number in stack trace points to foo(x, y) which is also correct

Huh. That's very interesting. I will try to explore this more. You were running from include?

No alt+enter in vscode


Do J2Objc support predefine

i have some shared code to support android and iOS, and  i add some annotation to the shared code as the following:
shared.java:
public class javaClass{
....
@ObjectiveCName(value="enbaled")
public boolean isEnable() {
.......
}
...
}
I must add a jar named  'com.google.j2objc:j2objc-annotations', I do not want to import it in android side.
How to resolve it?

You have to add it on the Android side. If necessary, the library is available independent of a j2objc release, using Maven. If your team builds the Android app with Gradle, you can specify that Maven dependency in your project.


bootstrap failed on Windows if checkout by tag v0.35.4

I am trying to checkout and build electron by tag v0.35.4 on Windows 10 with Python 2.7.10, Git 1.9.5, and VS2013 installed.
The whole process works fine if I am checking out HEAD.
C:\Users\Tom>git clone -b v0.35.4 https://github.com/atom/electron.git
Cloning into 'electron'...
remote: Counting objects: 42814, done.
remote: Compressing objects: 100% (108/108), done.
remote: Total 42814 (delta 52), reused 0 (delta 0), pack-reused 42705
Receiving objects: 100% (42814/42814), 12.04 MiB | 4.61 MiB/s, done.
Resolving deltas: 100% (30875/30875), done.
Checking connectivity... done.
Note: checking out 'e1d7ef7e24affd8f4ede3689c5354ef11036bf62'.

C:\Users\Tom>cd electron

C:\Users\Tom\electron>python script\bootstrap.py -v
Running in verbose mode
git submodule sync
git submodule update --init --recursive
...
Cloning into 'vendor/native_mate'...
remote: Counting objects: 421, done.
remote: Compressing objects: 100% (4/4), done.
Receiving objects:  84% (354/421)   ote: Total 421 (delta 0), reused 0 (delta 0), pack-reused 417
Receiving objects: 100% (421/421), 105.31 KiB | 0 bytes/s, done.
Resolving deltas: 100% (298/298), done.
Checking connectivity... done.
Submodule path 'vendor/native_mate': checked out '5e70868fd0c005dc2c43bea15ca6e93da0b68741'
Cloning into 'vendor/node'...
remote: Counting objects: 179050, done.
remote: Compressing objects: 100% (6403/6403), done.
remote: Total 179050 (delta 2889), reused 0 (delta 0), pack-reused 172486
Receiving objects: 100% (179050/179050), 126.28 MiB | 4.83 MiB/s, done.
Resolving deltas: 100% (138008/138008), done.
Checking connectivity... done.
fatal: reference is not a tree: 1445826ca73cc79bc57d503dd11d4ffaf695625c
Cloning into 'vendor/requests'...
remote: Counting objects: 16115, done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 16115 (delta 0), reused 0 (delta 0), pack-reused 16110
Receiving objects: 100% (16115/16115), 6.70 MiB | 4.78 MiB/s, done.
Resolving deltas: 100% (9139/9139), done.
Checking connectivity... done.
Submodule path 'vendor/requests': checked out 'e4d59bedfd3c7f4f254f4f5d036587bcd8152458'
Unable to checkout '1445826ca73cc79bc57d503dd11d4ffaf695625c' in submodule path 'vendor/node'
None
Traceback (most recent call last):
  File "script\bootstrap.py", line 210, in <module>
    sys.exit(main())
  File "script\bootstrap.py", line 39, in main
    update_submodules()
  File "script\bootstrap.py", line 95, in update_submodules
    execute_stdout(['git', 'submodule', 'update', '--init', '--recursive'])
  File "C:\Users\Tom\electron\script\lib\util.py", line 176, in execute_stdout
    raise e
subprocess.CalledProcessError: Command '['git', 'submodule', 'update', '--init', '--recursive']' returned non-zero exit
status 1

I also tried v0.35.2 and had the same issue.
There was a similar issue back in July, #2086

It happens on both mac and windows for fresh clones. It looks like the node commits used in Electron v0.34.* and v0.35.* got orphaned (1445826ca73cc79bc57d503dd11d4ffaf695625c and edfbc29d09425f2f387c52d77f6351b6ce101659). The situation described here: https://help.github.com/articles/commit-exists-on-github-but-not-in-my-local-clone/.
Could you please raise this in priority?

Sorry for the trouble, I forgot to tag the commit before resetting the tree, it should now.

Thanks.

Yes, thank you!  Those builds work for me now.


[sveltekit-pod] 17  Create component for search input and filter sort buttons

Create component for search input and filter / sort buttons
Background
This is intended to be a reusable component, wrapping around a few different items that will have their functionality added in another ticket. The focus of this ticket is to create the items themselves and a wrapper around them, so the section can be used in multiple pages. This component will contain an input field for search queries, and a series of dropdown buttons for filtering and sorting.
Acceptance

[x] component will contain the following items - for design of pieces only, do not need functionality yet

[x] search input with “find a repository..” placeholder
[x] type dropdown button (does not need to contain options yet)
[x] language dropdown button (does not need to contain options yet)
[x] sort dropdown button (does not need to contain options yet)
[x] divider line
[x] space for showing text relating to selected options
[ ] clear filter button with x icon (does not need functionality yet)

Notes

Is there a chance that state is aligned with the current url so it can be shared and it can be rendered on the server?
e.g
mypage?filter1=foo&filter2=bar&filter3=baz


[INLONG-650] Adjust .asf.yaml's label

Codecov Report

Merging #503 (ec5d4ff) into master (9802d01) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master    #503      +/-   ##
===========================================
- Coverage      7.72%   7.70%   -0.03%
  Complexity      563     563
===========================================
  Files           355     355
  Lines         31077   31077
  Branches       4970    4970
===========================================
- Hits           2400    2393       -7
- Misses        28187   28193       +6
- Partials        490     491       +1

Impacted Files
Coverage Δ

.../java/org/apache/flume/sink/tubemq/TubemqSink.java
51.42% <0.00%> (-4.00%)
:arrow_down:

Continue to review full report at Codecov.

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


Rename BalancesMap to BalanceSet

BalancesMap is only a map accidentally - it never uses the map directly, and only ever accesses the underlying set of notes via at for a single account at a time. I've always found it quite confusing, and ultimately concluded that the reason for my confusion was just that the module was confusing.
We should rename this to BalanceSet, i.e. a set (of notes) that represents a balance. This would mean putting the Map back inside the token contract and changing the APIs so that they no longer receive the owner param. With the recent change from https://github.com/AztecProtocol/aztec-packages/pull/8043 this should be simpler, as we'll now only explicitly pass an npk and not an account.

We don't really need to do this for all token contracts since we're trying to merge them, the basic token one is the main focus.

Closed in https://github.com/AztecProtocol/aztec-packages/pull/8127


[fix][test] Fix auto recovery test thread leak problem.

There are thread leak in the Auto Recovery test, you can see the pictures. It show that it create many threads and not destroyed.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

/pulsarbot rerun-failure-checks

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (8beac8b) 73.42% compared to head (f018b79) 73.40%.
Report is 2 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #21797      +/-   ##
============================================
- Coverage     73.42%   73.40%   -0.03%
- Complexity    32795    32798       +3
============================================
  Files          1897     1898       +1
  Lines        140656   140668      +12
  Branches      15491    15492       +1
============================================
- Hits         103282   103259      -23
- Misses        29297    29322      +25
- Partials       8077     8087      +10

Flag
Coverage Δ

inttests
24.21% <ø> (-0.05%)
:arrow_down:

systests
24.74% <ø> (-0.11%)
:arrow_down:

unittests
72.70% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 69 files with indirect coverage changes

Close via https://github.com/apache/bookkeeper/pull/4165


ManyToMany through table with condition

First of all, thanks for contributing an awesome library <3.  I am just moving to objection from sequelize. I'm very happy with objection now.
But problem is, I don't know how to add conditions to ManyToMany "through table".
I have deleted_at column on the through table. I want to add default condition for rejecting unrelated(actually updated(marked) deleted_at column) relation. something like this,
products: {
  relation: Base.ManyToManyRelation,
  modelClass: Product,
  join: {
    from: 'product_models.id',
    to: 'products.id',
    through: {
      from: 'product_product_models.product_model_id',
      to: 'product_product_models.product_id',
      filter: (builder) => builder.whereNull('ProductProductModel.deletedAt')
    }
  }
}

Is it possible?

I solved my self, It was filter! thanks


How to post a json array in react native

I have a JSON array like this. I want to post it via fetch function. i didn't find any reference for this kind of problems
[
{
"id": 824233,
"project_id": 1457,
"issue_id": 123420,
"activity_id": 71,
"comments": "Testing",
},
{
"id": 824234,
"project_id": 1457,
"issue_id": 123420,
"activity_id": 188,
"comments": "test",
},
{
"id": 824235,
"project_id": 1457,
"issue_id": 123420,
"activity_id": 188,
"comments": "Test",
}
]
I want to post this array

https://facebook.github.io/react-native/docs/network.html outlines how to make fetch calls. All you have to do is stringify it (example in docs) and post it to whatever URL you want.

@facebook-github-bot answered

@rmevans9. thanks for the response ya i saw the document. but in document post the json array is not there i think


Can't get this to work

So the documentation for mocha makes this look crazy simple.
I am sure I am doing something stupid, I have never done coverage testing before at all.
I added the babel plugin and it all runs but I get the following results:
experiencing result
----------|----------|----------|----------|----------|----------------|
File      |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
----------|----------|----------|----------|----------|----------------|
All files |  Unknown |  Unknown |  Unknown |  Unknown |                |
----------|----------|----------|----------|----------|----------------|

.babelrc
{
  "presets": [
    "stage-0",
    "env"
  ],
  "plugins": [
    "transform-class-properties",
    "lodash"
  ],
  "env": {
    "test": {
      "plugins": [ "istanbul" ]
    }
  }
}

entire output
bradynapier:~/workspace/reducer-generator-object-map (master) $ yarn run test
yarn run v0.24.5
$ cross-env NODE_ENV=test nyc mocha --recursive --require ./test/setup.js ./test/tests/*.js

  - Generate & Test Reducer
    Generate Object Map Reducer
      ✓ should generate a reducer function when called
      ✓ should initialize initialState when called without state
    [MATCH] | Test Action Dispatched
      ✓ should include the initialState when called and matched
      ✓ should include the changed values when called and matched
      ✓ should include both merged initialState and new values when matched
      ✓ should pass the initialArgs as arguments to the reducing function
      ✓ should pass the reducerArgs as arguments to the reducing function after initialArgs
      ✓ should return a new object when called and matched
      ✓ should be the same deep-equal object when called more than once with the same action
    [NO MATCH] | Test Action Dispatched
      ✓ should include the initialState when called and not matched
      ✓ should equal the initialState when called and not matched

  11 passing (24ms)

----------|----------|----------|----------|----------|----------------|
File      |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
----------|----------|----------|----------|----------|----------------|
All files |  Unknown |  Unknown |  Unknown |  Unknown |                |
----------|----------|----------|----------|----------|----------------|

Hmm, odd -- updating cross-env seems to have fixed this...

@bradennapier odd, not quite sure what might have been happening to you; but glad things are running well now ... and I didn't even have to do anything \o/


I can not find your paper

You can download the paper from this repository. The URL is https://github.com/NetEase-GameAI/Face2FaceRHO/blob/master/paper.pdf.


MERC-999 update feature product style

column-count breaks layout to top to bottom arrangement rather than left to right; use pattern-lab columns
@bc-ong

@bc-ong one more go 💃

:+1:


feat: set config

Currently errors out.  I am only passing one value.  Trying to figure out how little I can pass for it to accept.

This now works!! Next to create a data object of some form that can be passed in.  I only want this method to update the items we tell it to.  It however needs flexibility to do any of the dozens of config items!


Add support for 'min-height' parameter for buildings

Needs to be supported in CartoCSS, MapnikVT and VT renderer.

In carto.streets schema https://openmaptiles.org/schema/#building tag name is render_min_height

Implemented, relevant CartoCSS parameter: building-min-height


Cleanup Checkstyle errors

We need to go through the code and make sure there are no Checkstyle errors so that the build is cleaner.  This information can be seen currently as warnings after executing gradle build test.

hehehe i hate to be that guy but checkstyle was passing for me (on the psk and oer-fixes branch). I'm a few commits behind on the dev branch, let me work through it to try figure out whats going on.
If you know of a way to make checkstyle output more visible from circle-ci, shout!

The good news:
looks like we can get circle-ci to correctly fail the build if checkstyle fails (thanks to Adrians superior google-fu) - see #39
The bad news:
the checkstyle tests passed at build 40 which is for commit 66d2681. They start failing at build 41, which is for your commit 235d3df :D
(the checkstyle report is kept as an artifact).
If i find some time i'll try clean it up, might only be end of next week though.

Oh interesting - I thought checkstyle was set to only warn on the build, not fail.  I can look at this too.

We changed it to fail on a warning otherwise CircleCI passes the build even if there are CheckStyle issues.


[v1.15-backport] helm: loadBalancerClass for Cluster Mesh APIserver

[ ] https://github.com/cilium/cilium/pull/33033

Can a maintainer please remove backport-pending/1.15 from #33033 and add backport/author? Thanks

/test

/test-backport-1.15

Thanks, Nick, for triggering the tests. Maybe I'm wrong, but to me, it looks like the two failed CI tests aren't related to this change:

https://github.com/cilium/cilium/actions/runs/9656740711/job/26634809608#step:15:294
https://github.com/cilium/cilium/actions/runs/9643081112/job/26592822913?pr=33342#step:3:242


"notebooks/" --> "content/notebooks/" in all the Binder buttons

The path is wrong in all the Binder rocket buttons.

Removed binder button for now


Not able to use rancher-nfs with out having "insecure" option in nfs mounts.

Server version - Build from master
rancher/storage-nfs:v0.2.0
Steps to reproduce the problem:
When I did not have "insecure" option for my mount point in nfs server , containers having volume drivers set to rancher-nfs failed to start with error message -
Error (Error response from daemon: VolumeDriver.Mount: Failed mount <IP_ADDRESS>:/var/nfs/test11_foo1_1f77e /var/lib/rancher/volumes/rancher-nfs/test11_foo1_1f77e: Running in chroot, ignoring request. mount.nfs: rpc.statd is not running but is required for remote locking. mount.nfs: Either use '-o nolock' to keep locks local, or start statd. mount.nfs: mounting <IP_ADDRESS>:/var/nfs/test11_foo1_1f77e failed, reason given by server: No such file or directory)
When i added "insecure" option for my mount point in nfs server , I was able to start couple of instances successfully.
Expected Behavior:
We should be able to work with out having to add "insecure" option.

Image used should be rancher/storage-nfs:v0.4.0

Tested with rancher/storage-nfs:v0.4.0. Able to use rancher-nfs with out having "insecure" option in nfs mounts.


git-lfs doesn't work

bundles are committing as-is.

Oops, nevermind.


Upgrade to SQLalchemy >2.0

Detailed Description
This can be done using https://github.com/openclimatefix/pv-site-datamodel/issues/88
and pv-site-datamodel==1.0.17
Context

sqlalmacy upgrades should make things faster

Possible Implementation
upgrade requirements and run tests

@peterdudfield sir, can you please assign me this issue I have just created a PR for a similar issue.

@peterdudfield  sir, I am getting 2 errors while running make all : Error : # Make sure the date of .venvis afterpyproject.tomlprocess_begin: CreateProcess(NULL, # Make sure the date of.venvis afterpyproject.toml, ...) failed.
and while running make all -j 8 : Error : => `ERROR [11/13] RUN poetry install --without dev --no-root                                                     2.0s

[11/13] RUN poetry install --without dev --no-root:
1.779 The currently activated Python version 3.10.14 is not supported by the project (^3.11).
1.779 Trying to find and use a compatible version.
1.886
1.886 Poetry was unable to find a compatible version. If you have one, you can explicitly use it via the "env use" command.`

I made the changes based on the poetry lock and poetry errors. Please see this PR #80  for changes.


Syntax error with mariadb

Hi,
I am trying to install latest airflow by doing git clone + sudo pip install -e '<airflow repo path>'
I have a mariadb installation locally and sql_alchemy_conn as
mysql+pymysql://root:password@host:port/airflow
I created database by name airflow, set load_examples = False and did airflow initdb it works fine and tables are created with no dags.
Then I add tutorial.py in dags folder and run airflow scheduler, it gives me error which I can trace up to line -
...
  File "/home/centos/git/airflow/airflow/models.py", line 272, in deactivate_inactive_dags
    DagModel).filter(~DagModel.dag_id.in_(active_dag_ids)).all():
...

The sql alchemy exception is as follows -
sqlalchemy.exc.InvalidRequestError: This Session's transaction has been rolled back due to a previous exception during flush. To begin a new transaction with this Session, first issue Session.rollback(). Original exception was: (ProgrammingError) (1064, u"You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ''', 'airflow')' at line 1") u'INSERT INTO dag (dag_id, is_paused, is_subdag, is_active, last_scheduler_run, last_pickled, last_expired, scheduler_lock, pickle_id, fileloc, owners) VALUES (%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)' ('tutorial', 0, 0, 1, None, None, None, None, None, '/home/centos/airflow/dags/tutorial.py', 'airflow')

Interestingly, if I use only sqlite, things work perfectly fine.
Any suggestions ?
Server version: 5.5.44-MariaDB MariaDB Server
CentOS Linux release 7.0.1406 (Core)

Full logs for reference -
2015-09-29 18:21:52,706 - root - INFO - Importing /home/centos/airflow/dags/tutorial.py
Traceback (most recent call last):
  File "/usr/bin/airflow", line 8, in <module>
    execfile(__file__)
  File "/home/centos/git/airflow/airflow/bin/airflow", line 10, in <module>
    args.func(args)
  File "/home/centos/git/airflow/airflow/bin/cli.py", line 278, in scheduler
    job.run()
  File "/home/centos/git/airflow/airflow/jobs.py", line 159, in run
    self._execute()
  File "/home/centos/git/airflow/airflow/jobs.py", line 490, in _execute
    dagbag = models.DagBag(self.subdir, sync_to_db=True)
  File "/home/centos/git/airflow/airflow/models.py", line 121, in __init__
    self.deactivate_inactive_dags()
  File "/home/centos/git/airflow/airflow/models.py", line 272, in deactivate_inactive_dags
    DagModel).filter(~DagModel.dag_id.in_(active_dag_ids)).all():
  File "/usr/lib64/python2.7/site-packages/sqlalchemy/orm/query.py", line 2307, in all
    return list(self)
  File "/usr/lib64/python2.7/site-packages/sqlalchemy/orm/query.py", line 2425, in __iter__
    return self._execute_and_instances(context)
  File "/usr/lib64/python2.7/site-packages/sqlalchemy/orm/query.py", line 2438, in _execute_and_instances
    close_with_result=True)
  File "/usr/lib64/python2.7/site-packages/sqlalchemy/orm/query.py", line 2429, in _connection_from_session
    **kw)
  File "/usr/lib64/python2.7/site-packages/sqlalchemy/orm/session.py", line 881, in connection
    execution_options=execution_options)
  File "/usr/lib64/python2.7/site-packages/sqlalchemy/orm/session.py", line 886, in _connection_for_bind
    engine, execution_options)
  File "/usr/lib64/python2.7/site-packages/sqlalchemy/orm/session.py", line 306, in _connection_for_bind
    self._assert_active()
  File "/usr/lib64/python2.7/site-packages/sqlalchemy/orm/session.py", line 214, in _assert_active
    % self._rollback_exception
sqlalchemy.exc.InvalidRequestError: This Session's transaction has been rolled back due to a previous exception during flush. To begin a new transaction with this Session, first issue Session.rollback(). Original exception was: (ProgrammingError) (1064, u"You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ''', 'airflow')' at line 1") u'INSERT INTO dag (dag_id, is_paused, is_subdag, is_active, last_scheduler_run, last_pickled, last_expired, scheduler_lock, pickle_id, fileloc, owners) VALUES (%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)' ('tutorial', 0, 0, 1, None, None, None, None, None, '/home/centos/airflow/dags/tutorial.py', 'airflow')

I'd reproduced this error on 3 fresh installations atleast.
Not observing it any more,
sql_alchemy_conn is mysql://root:aether@<IP_ADDRESS>:3306/airflow
I installed mariadb-devel explicitly this time, probably that helped.


fix: update alpine version to try to get away from docker version caching

We're seeing issues with a character being added when jx-release-version is outputted to a file via the command line.

/approve


Change the Button component background colour

Expands the public API to include a new $govuk-button-background-colour scss variable.
Allows users who brand GOV.UK to do so without brittle selector overrides
which can break in future updates.
This variable naming matches other similar variables such as
$govuk-body-background-colour and $govuk-canvas-background-colour.
At the moment I have to do this sort of thing:
@import "node_modules/govuk-frontend/govuk/base";
@import "node_modules/govuk-frontend/govuk/components/button/index";

$app-button-colour: $app-primary-color;
$app-button-hover-colour: govuk-shade($app-button-colour, 20%);
$app-button-shadow-colour: govuk-shade($app-button-colour, 60%);
.govuk-button:not(.govuk-button--secondary):not(.govuk-button--warning) {
    background-color: $app-button-colour;
    box-shadow: 0 $govuk-border-width-form-element 0 $app-button-shadow-colour;
}

.govuk-button:not(.govuk-button--secondary):not(.govuk-button--warning):hover {
    background-color: $app-button-hover-colour;
}

After:
@import "node_modules/govuk-frontend/govuk/base";

$govuk-button-background-colour: govuk-colour("blue");
@import "node_modules/govuk-frontend/govuk/components/button/index";

Is it possible to get this considered for triage please?
@36degrees @vanitabarrett

Thank you @vanitabarrett ! Appreciate it.

@querkmachine pushed an update addressing your feedback.

A commit from your accordion focus colour PR seems to have snuck in and caused a merge conflict. If you can fix that up, I'm happy to approve!

Ah yeah rouge rebase gone wrong, will sort that thanks.

Hi @NickColley,
Sorry! This has ended up overlapping with some other work we're doing and we've inadvertently introduced a new merge conflict to your PR — the file at tasks/gulp/sassdoc.js has been moved to tasks/sassdoc.js on the main branch now.
If you're okay to take a look at it, please do. We're currently preparing a bugfix release of Frontend however, so it may be a few more days until this is approved and merged.

@querkmachine okay sorted that conflict thanks for the heads up. 👍🏻

@NickColley I've just rebased your branch and moved the changelog entry to the right place now that the 4.3.1 release has gone out. Will approve once the tests have ran. Thanks for putting up with the faff!

Thanks @querkmachine @owenatgov for your help getting this one in 👍🏻


Add a script to easily generate the required ebook structure from a corpus of repos

I guess you’ve got some Github API integration to rebuild the library for every release, but without it I’m finding it difficult (even with the README documentation on expected formats) to build the expected heirarchy of ebooks and ebook data. Would it be possible to add a script that, given a folder containing a set of cloned SE repos could build the expected tree of data and potentially copy it into place?

Could I have the script too, please? I've managed to get a server set up, but I'm lost on how to set up the ebooks directory. I've tried to place one of the repos in www/ebooks with the built epubs in the dist folder, but <IP_ADDRESS>:8080/ebooks/ just displays the 404 page.

If <IP_ADDRESS>:8080/ebooks/ is a 404, are you certain it's actually hitting <IP_ADDRESS>:8080/ebooks/index.php? Perhaps your server is misconfigured.
You can set up an example ebook in the website like so:
mkdir -p /path/to/se/root/www/ebooks/g-k-chesterton/the-man-who-was-thursday/dist
cd /path/to/se/root/www/ebooks/g-k-chesterton/the-man-who-was-thursday/
git clone https://github.com/standardebooks/g-k-chesterton_the-man-who-was-thursday

I think that should be it.

<IP_ADDRESS>:8080/ebooks/index.php is also a 404, even with the directory tree set up like that. It definitely executes ebooks.php, but I assume it triggers this exception.
This is what I have right now. The server starts up, php files get served correctly, but the ebook listing doesn't work. I'm probably doing something wrong, but I can't see what that would be.

Unfortunately I'm not familiar with Vagrant. I suggest removing the catch block in index.php to see exactly what the exception is that's happening. However if you follow the bash script I included above, I don't see why it wouldn't work. (I haven't tested it though.)

Removing the try/catch gives me this error:
Fatal error: Uncaught InvalidEbookException: Invalid repo filesystem path: /standardebooks.org/ebooks/g-k-chesterton_the-man-who-was-thursday_g-k-chesterton_the-man-who-was-thursday in /vagrant/lib/Ebook.php:66 Stack trace: #0 /vagrant/lib/Library.php(65): Ebook->__construct('/standardebooks...') #1 /vagrant/lib/Library.php(33): Library::GetEbooks() #2 /vagrant/www/ebooks/index.php(58): Library::GetEbooks('newest') #3 {main} thrown in /vagrant/lib/Ebook.php on line 66
It seems to be looking for the bare git repo but without the .git suffix? I'm not sure.

Right. You did not follow my shell script above. Try running it and it will create a nested directory hierarchy, not the flat one that you created.

Okay, I made a mistake when setting the target directory for the git clone. I apparently ran git clone https://github.com/standardebooks/g-k-chesterton_the-man-who-was-thursday g-k-chesterton_the-man-who-was-thursday_g-k-chesterton_the-man-who-was-thursday for whatever reason. Sorry for wasting your time. After re-running the lines correctly and doing a --bare clone of the book repo in /standardebooks.org/ebooks/, it works like it should. It's only missing the images, which are easy enough to get from the website, but is there a nice way to generate them from the book repositories?

Not at the moment, it is automated but the code that does it is rough and part of a larger system. For now you can copy them from the website, or tweak the website code to hard-code a single image for all the books.

Could you take a look at this before I do a pull request?
https://github.com/PyroLagus/standardebooks-web/commit/8b685a65b54d5a8bbf4f7885196b0490e0f75fe1

Right now I don't think I want to script pulling the compiled SE corpus from the SE web server. People are sometimes not responsible, and bandwidth and server resources are a concern.
Instead, for the person interested in building the entire SE corpus on their local machine, the process should mimic what occurs on the server:

The raw SE ebook sources are stored in /standardebooks.org/ebooks/ (or wherever). These can be pulled from Github in some scripted fashion.

A script is then run on each raw source to generate the correct cover images, build the sources using the toolset, and create the correct directory hierarchy. This script exists on the server but it is not yet in a state that is ready for public consumption.

This is more time consuming for the local user, but less resource consuming for SE. And, most people won't be doing this anyway. Even for web development it's not necessary to have the entire corpus on the local host, maybe just 2 or 3 pages of books.
So the script to write would be one that pulls the raw corpus from Github onto the local machine. I would also make sure to be polite about the timing of sequential requests, maybe space them out by one or two seconds. There are only three non-ebook repos in the SE account: tools, manual, and web.

Well, the script only pulls the cover and banner images from standardebooks.org (I actually know how to generate the cover images from the sources, but the "hero"/banner images are a mystery to me), not the ebook files. It gets the ebook sources from GitHub (or any other git repository including local), so the website shouldn't have any issues. It doesn't actually add compiled ebooks (epub, mobi, etc.) since I didn't want to assume how the user uses SE tools; whether it's in the path, from the cloned repo, used via venv, etc.
I thought I made that clear in the readme and the shell script. Is there anything I could do to clarify further?

OK, I've cleaned up the script I use to deploy an ebook to the website: https://github.com/standardebooks/web/blob/master/scripts/deploy-ebook-to-www
That script is what you should use to take a raw ebook source from /standardebooks.org/ebooks/ and deploy it to the local website. It builds all the necessary images and also the distributable files, and it also updates the OPDS feed and new releases RSS feed.
So, what we need your script to do is merely scrape Github for SE ebooks and clone them all to /standardebooks.org/ebooks. Then, we can add a section in the readme for how to initialize the SE website locally. It would basically be two steps: 1. Run the script to clone all ebook sources locally, and 2. run deploy-ebook-to-www /standardebooks.org/ebooks/* to deploy everything locally.
I prefer this approach because it does not depend on the remote SE server. If it blows up in the future, or is for whatever reason inaccessible, then we can rebuild everything locally.

I haven't tested them yet, but something like this? https://github.com/PyroLagus/standardebooks-web/commit/71ef62ba9bfd73801b13f087068c97177707b85d

pull_ebooks.sh clones a list of repositories into /standardebooks.org/ebooks/ (should probably be called clone_ebooks.sh instead)

update_ebooks.sh does a 'git pull' for all repositories in /standardebooks.org/ebooks/

all_ebook_urls.sh prints a list of all ebook repositories to stdout that you can use with pull_ebooks.sh

I think pull_ebooks.sh and update_ebooks.sh can be merged in to one script, sync-ebooks. This script would accept one parameter, the directory where SE ebook sources are stored (i.e. /standardebooks.org/ebooks. Then, it would first iterate over all direct children of that folder and do git pull --rebase, and subsequently it would download any new ebooks from GitHub.
Later, we can add an option to sync-ebooks to only pull, or only download new ebooks. But we can put that aside for now.
I don't think all_ebook_urls.sh is strictly necessary. You can simply subsume that functionality into sync-ebooks.
I see you're using jq to parse GitHub output, but that's not a core utility. Since the use case here is simple can you use sed or grep instead, so that we don't add another dependency to the toolset?

Sure, no problem. That said, it would still be nice to offer a way to clone only a few books, but I suppose that could just be mentioned in the readme as a one-liner (or nicely formatted three-liner) since that's just a 'while read...' loop.

I guess, but if the user just wants to clone a few then they can just do a few git clones... :)

How's this? https://github.com/PyroLagus/standardebooks-web/commit/40e5658455ac6641d9f85f2f6535a2dda27138a3

Yes, that looks good. Take a look at deploy-ebook-to-www for an idea of Bash code style to apply. In particular you can copy and paste the top boilerplate for the script's help text.
In general we want to follow the Unix philosophy of "quit unless there are problems." So by default the script should output nothing unless there are errors (and then to stderr). We can add a --verbose flag later if the user wants more details as to what 's happening.
Also, we should not use environmental variables to pass parameters to the script. Instead we should read them as flags. deploy-ebook-to-www has an example of basic flag parsing and you can adapt that to handle multiple different flags.
I'm curious as to why you trap ctrl + c. Doesn't ctrl + c just exit the script anyway? Why trap it, only to do what it already does by default?

When there's a sub-process running in the foreground, ctrl+c will just kill that process and continue, which is particularly annoying in a loop. Trapping assures that the script ends.
About the flags, getopts should work fine if we don't need POSIX compatibility anyways, but then the directory to be operated on would also have to be passed via an option argument, but I suppose that's fine.
Do you really not want any progress output by default? The script can take quite a while to finish.

Is this fine? https://github.com/PyroLagus/standardebooks-web/commit/bd0befaaa0f6a0518c70060bd199877b12947dfc


Can't join vanilla server through packet_inspector proxy.

How to reproduce
start a vanilla mc server
docker run -e EULA=TRUE -e ONLINE_MODE=false -d -p 25565:25565 --name mc itzg/minecraft-server

run packet inspector
cargo r -r -p packet_inspector -- <IP_ADDRESS>:25566 <IP_ADDRESS>:25565

Open mc and connect to localhost:25566
See that you never load into the world
Notes
The disconnect occurs after the server sends the first ChunkData packet. The full error is
Error: Err(
    Error {
        context: "decoding packet after decompressing",
        source: Error {
            context: "failed to read field `block_light_arrays` from struct `ChunkData`",
            source: Error {
                kind: UnexpectedEof,
                message: "failed to fill whole buffer",
            },
        },
    },
)

This is not displayed by the packet inspector logs currently, because UnexpectedEof errors are explicitly ignored with this code.
if let Some(e) = e.downcast_ref::<io::Error>() {
    if e.kind() == ErrorKind::UnexpectedEof {
        return Ok(());
    }
}

Removing that snippet makes this problem go away. The packet still fails to decode, but it doesn't disconnect the client. This is because the return makes the loop stop.


Update Protractor.js

This will fix the issue of the Protractor helper always sending Firefox as the browser to Browserstack. Before this even if you said browserName: 'Chrome' it would startup Firefox in Browserstack. With this change it will boot the right browser.

Thank you!

However, your fix breaks tests. Could you look into them and fix? https://travis-ci.org/Codeception/CodeceptJS/jobs/201278629
browser name should be converted to string

Ah yes I see! Let me check tomorrow at work 👍 For now this works form me, but let's make it work for real.


Unable to activate azure-0.6.4, because mime-types-2.5 conflicts with mime-types (~> 1.0)

Im trying this cookbook to install de azure gem, but im stuck with this error
Compiling Cookbooks...
[2015-05-05T17:45:05+00:00] WARN: chef_gem[macaddr] chef_gem compile_time installation is deprecated
[2015-05-05T17:45:05+00:00] WARN: chef_gem[macaddr] Please set `compile_time false` on the resource to use the new behavior.
[2015-05-05T17:45:05+00:00] WARN: chef_gem[macaddr] or set `compile_time true` on the resource if compile_time behavior is required.
Recipe: microsoft_azure::default
  * chef_gem[macaddr] action removeWARN: Unresolved specs during Gem::Specification.reset:
      nokogiri (>= 1.4.0, ~> 1.5)
WARN: Clearing out unresolved specs.
Please report a bug if this causes problems.

    - remove  package macaddr
[2015-05-05T17:45:10+00:00] WARN: Cloning resource attributes for chef_gem[macaddr] from prior resource (CHEF-3694)
[2015-05-05T17:45:10+00:00] WARN: Previous chef_gem[macaddr]: /var/chef/cache/cookbooks/microsoft_azure/recipes/default.rb:18:in `from_file'
[2015-05-05T17:45:10+00:00] WARN: Current  chef_gem[macaddr]: /var/chef/cache/cookbooks/microsoft_azure/recipes/default.rb:22:in `from_file'
[2015-05-05T17:45:10+00:00] WARN: chef_gem[macaddr] chef_gem compile_time installation is deprecated
[2015-05-05T17:45:10+00:00] WARN: chef_gem[macaddr] Please set `compile_time false` on the resource to use the new behavior.
[2015-05-05T17:45:10+00:00] WARN: chef_gem[macaddr] or set `compile_time true` on the resource if compile_time behavior is required.
  * chef_gem[macaddr] action install
    - install version 1.6.1 of package macaddr
[2015-05-05T17:45:22+00:00] WARN: chef_gem[azure] chef_gem compile_time installation is deprecated
[2015-05-05T17:45:22+00:00] WARN: chef_gem[azure] Please set `compile_time false` on the resource to use the new behavior.
[2015-05-05T17:45:22+00:00] WARN: chef_gem[azure] or set `compile_time true` on the resource if compile_time behavior is required.
  * chef_gem[azure] action install (up to date)

  ================================================================================
  Recipe Compile Error in /var/chef/cache/cookbooks/microsoft_azure/recipes/default.rb
  ================================================================================

  Gem::ConflictError
  ------------------
  Unable to activate azure-0.6.4, because mime-types-2.5 conflicts with mime-types (~> 1.0)

  Cookbook Trace:
  ---------------
    /var/chef/cache/cookbooks/microsoft_azure/recipes/default.rb:34:in `from_file'

  Relevant File Content:
  ----------------------
  /var/chef/cache/cookbooks/microsoft_azure/recipes/default.rb:

   27:
   28:  chef_gem 'azure' do
   29:    version '0.6.0' #FIXME systemu gem conflict workaround
   30:    #version node['microsoft_azure']['azure_gem_version']
   31:    action :install
   32:  end
   33:
   34>> require 'azure'
   35:

  Running handlers:
[2015-05-05T17:45:22+00:00] ERROR: Running exception handlers
  Running handlers complete
[2015-05-05T17:45:22+00:00] ERROR: Exception handlers complete
[2015-05-05T17:45:22+00:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
  Chef Client failed. 2 resources updated in 30.8459925 seconds
[2015-05-05T17:45:22+00:00] ERROR: Unable to activate azure-0.6.4, because mime-types-2.5 conflicts with mime-types (~> 1.0)
[2015-05-05T17:45:23+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

Any idea why? Is there any workarround?

Apparently this conflicts when running chef 12.0, but it works if chef version is 11.8.2


لودي نت :(Source request)

Source name
لودي نت
Source link
https://lodynet.cloud/
Source language
Arabic
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a title with source name.
[X] I have checked that the extension does not already exist on the website extensions list or the app.
[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I will fill out all of the requested information in this form.

Duplicate of #405

@Secozzi Link to the old site .


Add eviction mechanism to RegExp cache

Implement an eviction mechanism for RegExp cache, if the cache is full throw out an element and add the new bytecode into the cache. May be a FIFO algorithm would cause the less performance regression.

Solved by #947 and 6f536c79423ecca4c45897d6be555e08739ed71b


If no hash algorithm is selected, don't compare

If no hash algorithm is selected, nothing should be checked here. The comparison result shall be reset to neutral.
Doing this update, #30 becomes obsolete.

Also if file is not found, no comparison shall be done. We can say: Only compare of a valid hash was generated


fix: hesai_decoder replace mktime with timegm

PR Type

Bug Fix

Description
Hesai Decoder. Replace std::mktime with timegm to take into consideration non UTC times.
Review Procedure

Play a rosbag with pandar_packets msg
Launch the hesai_decoder corresponding to the packets being played.
Confirm the decoded timestamp matches the current Time Zone.

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[ ] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR: Required to pass before the merge.

LGTM


Vectors, rectangles and other types

I propose to add these types:

Vectors (Vector2, Vector3)
Color
Rect
Size

See implementations in freezedev/flockn or elysion-powered/elyssa for reference.
I feel though that it should be possible to choose not use these types and there needs to be a way to extend these types to custom needs.

This is done in 5e32bc8d8fe9fa3d3fc010142b97563c3c908eae


i want to add page with meditation songs.

Description
Is your feature request related to a problem? Please describe.
A clear and concise description of the problem or limitation that the requested feature would address.
Describe the solution you'd like
A clear and concise description of the feature or enhancement you would like to see implemented.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional Information
Context
People while get relax after listening to the song.
Potential Implementation (optional)
If you have any ideas or suggestions on how the feature could be implemented, please share them here.
Screenshots or Mockups (optional)
If applicable, include any screenshots, mockups, or visual representations of the requested feature.
Priority (optional)
Please indicate the priority level of this feature request (e.g., High, Medium, Low).
Additional Notes (optional)
Add any other notes or comments about the feature request here.

Hello @akshitagupta15june Ma'am,
I would like to add page with meditation songs. This will help to enhance the user experience.
Please assign me this issue. I would like to work on this issue under GSSOC'23. Please let me fix this issue #694.

on AR Meditate page there are songs for meditating. Pl check.


TOPIC: Parsing PDFs table contents for RAG supporting notebook

This notebook is supporting article: https://github.com/elastic/search-labs-elastic-co/pull/1270

LGTM. @justincastilla do you mind giving this a once over for me?

On it!

@carlyrichmond  can this be merged and closed?


Run Prettier Formatter and add it to CI

@bjornharrtell I was going to maybe commit this earlier so I don't have it as a confusing diff in https://github.com/flatgeobuf/flatgeobuf/pull/159
How do you feel about running this formatter step in CI? Is that going to be useful or annoying for people? Having it not formatted made it a little challenging for me since my editor kept trying to apply the prettier config on save, but it then of course made big diffs on the branch I was working on.

Hm i guess there's not actually a CI on this? I don't see CircleCI reporting status on these PRs like I imagined

@worace yeah I seem to have a CI config that does not build PRs. I'll try to get that fixed ASAP.
I think it makes sense to verify the lint/formatting on CI.

I hope to have been able to enable CI to build PRs.


fix a little word jumble

Super quick fix

Can't believe this was missed this long, thanks for the catch!


fix: helm downgrade mongodb

Description
Migration from MongoDB 10.X.X to 13.X.X was causing the issue with accessing pvc. Therefore, we change MongoDB version to 12.1.X
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix

How Has This Been Tested?
Integration tests, manual tests
Checklist

[ ] My commit message is conventional
[ ] I have run pre-commit on all files before creating the PR
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have checked my code and corrected any misspellings

Codecov Report

Merging #668 (3bd0d76) into develop (906b49e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #668   +/-   ##
========================================
  Coverage    86.83%   86.83%
========================================
  Files           25       25
  Lines         1603     1603
========================================
  Hits          1392     1392
  Misses         211      211

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

:tada: This PR is included in version 1.8.5-beta.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Allow delegation to existing JWKs

Updates the sdk to enable delegation to existing externally generated JWKs, corresponding to kepler pr 126.

jwk needs to actually be merged into the config here I think:
https://github.com/spruceid/kepler-sdk/blob/main/src/authenticator.ts#L13
parents too from the previous pr.


Document :usec option to Timestamp macro

This is for #31. Added documentation both in the README and at the code level so that elixir docs know about it too.

Thanks so much!


[GraphQL] Gql bump version prod to 2024.4.4

Description
Bump prod version, just in case we need this branch again.
Test plan
How did you test the new or updated feature?

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:
[ ] REST API:

What about crates/sui-graphql-e2e-tests/tests/call/simple.exp ?

Thanks for the stamp. Will fix and then merge.


Updated for new version support of MariaDB and MySQL

https://aws.amazon.com/about-aws/whats-new/2018/10/amazon-rds-now-supports-mariadb-10_3/
https://aws.amazon.com/about-aws/whats-new/2018/10/amazon-rds-now-supports-mysql-8/

Perfect. You're on the ball!