Change function arguments to use pointers

AdcChannelConfig member functions don't use pointers. They could be changed from
 void InitMux(dsy_gpio_pin adc_pin,
             dsy_gpio_pin mux_0,
             dsy_gpio_pin mux_1,
             dsy_gpio_pin mux_2,
             size_t       channels);

to this:
void InitMux(const dsy_gpio_pin* adc_pin,
             const dsy_gpio_pin* mux_0,
             const dsy_gpio_pin* mux_1,
             const dsy_gpio_pin* mux_2,
             size_t       channels);

There's probably more places like this.

AdcChannelConfig is not problematic, dsy_gpio_pin is very small and its only used once during setup.
I tried to change it but too many other parts of libDaisy rely on the pass-by-value arguments so it doesn't make sense to change this.


For loop missing array on subsequent component instance

The data:
navigation = [{
  label: 'Writing',
  url: '/articles/'
}, {
  label: 'Speaking',
  url: '/presentations/'
}, {
  label: 'Projects',
  url: '/projects/'
}]

The parent component:
<navigation :@items="navigation"></navigation>
<navigation :@items="navigation"></navigation>

The navigation component:
<script webc:type="js">
  console.log('items', items);
</script>
<template webc:for="item of items">
  <script webc:type="js">
    console.log('item', item);
  </script>
</template>

The console output:
items [
  { label: 'Writing', url: '/articles/' },
  { label: 'Speaking', url: '/presentations/' },
  { label: 'Projects', url: '/projects/' }
]
item { label: 'Writing', url: '/articles/' }
item { label: 'Speaking', url: '/presentations/' }
item { label: 'Projects', url: '/projects/' }
items [
  { label: 'Writing', url: '/articles/' },
  { label: 'Speaking', url: '/presentations/' },
  { label: 'Projects', url: '/projects/' }
]
item undefined // <!---- Items clearly available, but not within `webc:for` loop

This occurs if the second instance of the component is using the same data object, or a different one. Just started kicking the tyres of WebC this evening, so maybe I'm missing something?

Fix will ship with 0.9.4—thank you!

No problem – glad to hear it was a bug and not my own ineptitude ☺

of course it was my fault Paul 😅


Move Cargo onto stable Rust

This commit moves Cargo onto 100% stable Rust now that the necessary fs features
have been stabilized. The tests were updated to understand when they're running
with a non-nightly compiler and disable all plugin-related tests. The only major
feature here is that the calls to fs::set_file_times were reimplemented
manually in the tests that Cargo has.
Cargo still requires a nightly compiler due to the stable features not having
made their way into the stable channel yet, but it will soon work on all of the
stable, beta, and nightly compilers once 1.1 is released!

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)

r=me with @chris-morgan's comment addressed.

@bors: r=brson dab6e56

:hourglass: Testing commit dab6e56 with merge 832d9f6...

:broken_heart: Test failed - cargo-win-gnu-64

@bors: r=brson 8453a90

:hourglass: Testing commit 8453a90 with merge 3bfcc7b...

:broken_heart: Test failed - cargo-win-gnu-32

@bors: r=brson
On Tue, Jun 16, 2015 at 10:16 PM, bors<EMAIL_ADDRESS>wrote:

[image: :broken_heart:] Test failed - cargo-win-gnu-32
http://buildbot.rust-lang.org/builders/cargo-win-gnu-32/builds/43
—
Reply to this email directly or view it on GitHub
https://github.com/rust-lang/cargo/pull/1712#issuecomment-112654027.

:pushpin: Commit 7c2d1a5 has been approved by brson

:hourglass: Testing commit 7c2d1a5 with merge 08199c0...

:broken_heart: Test failed - cargo-win-gnu-64

:pushpin: Commit 23762e4 has been approved by brson

@bors: r=brson
On Tue, Jun 16, 2015 at 10:30 PM, bors<EMAIL_ADDRESS>wrote:

[image: :broken_heart:] Test failed - cargo-win-gnu-64
http://buildbot.rust-lang.org/builders/cargo-win-gnu-64/builds/46
—
Reply to this email directly or view it on GitHub
https://github.com/rust-lang/cargo/pull/1712#issuecomment-112655276.

:hourglass: Testing commit 23762e4 with merge 35c05ae...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64


Catalog card arrangement

Describe the bug
Components that are ready to be installed comes as second priority after "Coming Soon" components.
To Reproduce
Steps to reproduce the behavior:

Open Control Center and click on Catalog.
See error that "Coming soon" components appear in sequence before other available components for install.

Expected behavior
Always showing components ready to Install first.
Screenshots

Environment

Browser: [Chrome]
Version: [Version 0.50.0-alpha]

Additional context
Add any other context about the problem here.

@M-Shorouk
we have "COMING SOON", "OPEN" and "INSTALL"
OPEN -> INSTALL -> COMING SOON would be my suggested order, what you think?


Host documentation

Host chpldoc-generated documentation, possibly utilizing https://pages.github.com/ in a docs/ directory.
Ideally, CI (Travis?) could build and update this documentation for every PR.

If using github pages, don't forget to add a .nojekyll file.

Thank you Ben. I will organize the doc because my personal website is hosted in github pages. I will make it as subfolder project.
I will notify you when things go on.

@ben-albrecht, I did not forget this issue. I am getting an error in the RST compiling. This is occurring because I forgot some comma in the comments.
When I port the ORM to MySQL and SQLite I will document it and close this issue.

I have hosted the documentation https://krishnadey30.github.io/cdo-docs/
@buddha314 @ben-albrecht

Looks great @krishnadey30 - How about I create a gh-pages branch on cdo repo and you can push the built docs to there?

I have pushed your docs to marcoscleison/cdo/gh-pages, but unfortunately I do not have admin permissions for the repo to flip the switch and enable github pages. @marcoscleison - would you be able to do that?


Make use of Server.Shutdown

See https://golang.org/pkg/net/http/#Server.Shutdown

PR :)?
On Thu, 20 Feb 2020 at 10:47 pm leizihui<EMAIL_ADDRESS>wrote:

When will this function be completed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jpillora/overseer/issues/16?email_source=notifications&email_token=AAE2X44UML7BGUS76NEKK6LRDZUUDA5CNFSM4DRO3YK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMNSDRA#issuecomment-588980676,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE2X42PHBUBZFE2WLSGL6TRDZUUDANCNFSM4DRO3YKQ
.


Add Pylance setting and prompt for install if Pylance extension is missing

For #13122

Add Pylance setting to python.languageServer

Check if Pylance is installed and prompt for install if it is missing

Related tests

Add few strings to localization

[X] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).

[X] Title summarizes what is changing.

[X] Has a news entry file (remember to thank yourself!).

[X] Appropriate comments and documentation strings in the code.

[X] Has sufficient logging.

-   [ ] Has telemetry for enhancements.

[X] Unit tests & system/integration tests are added/updated.

-   [ ] Test plan is updated as appropriate.
-   [ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
-   [ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #13123 into master will decrease coverage by 0.00%.
The diff coverage is 96.55%.

@@            Coverage Diff             @@
##           master   #13123      +/-   ##
==========================================
- Coverage   59.79%   59.79%   -0.01%
==========================================
  Files         671      671
  Lines       36681    36707      +26
  Branches     5158     5163       +5
==========================================
+ Hits        21933    21948      +15
- Misses      13655    13662       +7
- Partials     1093     1097       +4

Impacted Files
Coverage Δ

src/client/activation/common/activatorBase.ts
13.22% <ø> (-9.00%)
:arrow_down:

src/client/activation/activationService.ts
84.24% <83.33%> (+0.09%)
:arrow_up:

src/client/activation/languageServer/activator.ts
93.33% <93.33%> (-0.61%)
:arrow_down:

src/client/activation/extensionSurvey.ts
98.38% <100.00%> (ø)

src/client/activation/node/activator.ts
71.05% <100.00%> (+27.05%)
:arrow_up:

src/client/common/utils/localize.ts
96.19% <100.00%> (+0.05%)
:arrow_up:

...nt/languageServices/proposeLanguageServerBanner.ts
93.47% <100.00%> (ø)

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/datascience/crossProcessLock.ts
79.41% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

... and 6 more

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 6c99859...32d3626. Read the comment docs.

One other thing that would be good to do:
Right now, for people without Pylance installed, it's going to be two reloads because the process is like:

Change in the settings to Pylance. Extension prompts "LS changed, please reload".
New Pylance code runs, and Pylance extension isn't installed. Extension prompts "go here to get Pylance".

Can the first step be modified to say "if changing to Pylance and Pylance not installed, load pylance banner instead"? That way you only have the one reload to actually finish the whole process.

Basically activator and change settings tracker should be calling the same code that would check if Pylance is installed. So the code needs to move from activator into some shared class.

Some of it, anyway. Three lines to do if (this.extensions.getExtension(PYLANCE_EXTENSION_ID) { ... } is probably not worth the abstraction, but the rest of the banner yeah.


[x] Add allowlist for direct dep duplicates

Duplicates are occasionally desired when doing a long upgrade of a
dependency. When such upgrades land over multiple PRs, the dependency will be
temporarily duplicated.

/land


[NETBEANS-228] Attributing content of anonymous classes whose superty…

…pe is erroneous.
Consider code like:
new Unknown() { public void run() { } };
Vanilla javac (i.e. not nb-javac) won't dive inside the anonymous class to attribute it, which then leads to various other problems, including the exception from the bugreport, and also features like code completion not working well inside such anonymous class.
The attempt here is it attribute the inside of the anonymous class if it hasn't be attributed. nb-javac is doing something with a similar effect, but differently.

Thanks, merging.


Change Hashing to JWT Token

Change password hashing to JWT Token

https://techarena51.com/blog/json-web-token-authentication-with-flask-and-angularjs/


Missing 7 in the reputation.score_id enum values

The number 7 is missing in reputation.score_id enum values. Should we define it?

@rroupski we should have a 7, how about a caption of Suspicious and a description of Indicators of suspicious behavior?


Fix variables in IP2-type update

Summary
I just remove unused variables.

Codecov Report

Merging #98 (5fa92af) into develop (d7ba32a) will increase coverage by 0.69%.
The diff coverage is 100.00%.

:exclamation: Current head 5fa92af differs from pull request most recent head c5c6ed2. Consider uploading reports for the commit c5c6ed2 to get more accurate results

@@             Coverage Diff             @@
##           develop      #98      +/-   ##
===========================================
+ Coverage    93.71%   94.41%   +0.69%
===========================================
  Files           18       18
  Lines         2388     2702     +314
===========================================
+ Hits          2238     2551     +313
- Misses         150      151       +1

Impacted Files
Coverage Δ

ssspy/bss/fdica.py
91.71% <100.00%> (-0.07%)
:arrow_down:

ssspy/bss/ilrma.py
97.85% <100.00%> (+0.70%)
:arrow_up:

ssspy/bss/iva.py
94.54% <100.00%> (-0.06%)
: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 b9d513f...c5c6ed2. Read the comment docs.


filetype detection

I would love simple "magic number" detection using 1st 4 bytes, instead of using file-extensions, in pntr/pntr_app. I made this that can detect a bunch of different file-types (and even directory, in the wrapper functions) and it helps in situations where the extension is unreliable or not available. In my usecase, the user could have referenced a wasm file, or a zip, and in other things, media files. I had the idea to embed carts inside png files, too (similar to PICO8) so being able to handle whatever the user throws at me, without relying on how it's named is a bit important.
I think all of the common media/archive formats use just 4 bytes, so although some detection needs more (PDFs need lots more, I think) it would probly cover most of our usecases to use the first int. This works in any filesystem where you can read the first 4 bytes, and in my example, I have "real" and physfs wrappers that look at it.
The main meat of the function looks like this:
// these are the supported filetypes we can detect
// see fs_parse_magic_bytes()
typedef enum {
    FILE_TYPE_UNKNOWN,
    FILE_TYPE_ZIP,
    FILE_TYPE_WASM,
    FILE_TYPE_PNG,
    FILE_TYPE_JPEG,
    FILE_TYPE_WAV,
    FILE_TYPE_OGG,
    FILE_TYPE_MP3,
    FILE_TYPE_DIR
} DetectFileType;

// just detect filetype from first 4 bytes
DetectFileType fs_parse_magic_bytes(uint32_t magic_number) {
  switch (magic_number) {
    case 0x4034b50:  // ZIP
        return FILE_TYPE_ZIP;

    case 0x6d736100:  // WASM
        return FILE_TYPE_WASM;

    case 0x89504E47:  // PNG
        return FILE_TYPE_PNG;

    // JPEG variants
    case 0xe0ffd8ff:  // JPEG/JFIF
    case 0xe1ffd8ff:  // JPEG/Exif
    case 0xeeffd8ff:  // JPEG/SPIFF
    case 0xfeffd8ff:  // JPEG/COM
        return FILE_TYPE_JPEG;

    case 0x46464952:  // WAV ("RIFF")
        return FILE_TYPE_WAV;

    case 0x5367674f:  // OGG ("OggS")
        return FILE_TYPE_OGG;

    // MP3 variants
    case 0x03334449:  // MP3 with ID3v2.3
    case 0x02334449:  // MP3s with ID3v2.2
    case 0x04334449:  // MP3 with ID3v2.4
        return FILE_TYPE_MP3;

    default:
        return FILE_TYPE_UNKNOWN;
  }
}

and covers all the formats I need in my engine, but more could be added (anything supported in pntr, and maybe a few more like zip and wasm.)
It could also be broken out into just the filetypes supported by each engine, like if pntr has jpeg/png/etc support, detect those, and supported sound-files in another function in pntr_app.

That'd be cool! Would definitely simplify some of the loading methods.


Reorganize a couple of the legacy helpers [npm node jenkinsplugin]

Ref #2832

Warnings

:warning:

This PR modified helper functions in lib/ but not accompanying tests. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for jenkins but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for npm but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

  Generated by :no_entry_sign: dangerJS against 7e345553dd1083e7ddb01c7032aac744193c4855

DynamicXML failures were unrelated.


[FR] Allow users to pass through additional HTML arguments in FrontEnd.py

Description of Requested Feature
Allow users to specify width and/or class in FrontEnd.dropdown_from_dataframe
Context
Currently, width is fixed at 400px for even if this isn't ideal for the webpage.
Overriding this using CSS can only be done using IDs, which isn't ideal if you're using multiple dropdowns that you want to format the same.
Current workaround is like this:
// python:
d1 = dropdown_from_dataframe("dropdown_a", df, "col_a")
d2 = dropdown_from_dataframe("dropdown_b", df, "col_b")
[...]
dn = dropdown_from_dataframe("dropdown_n", df, "col_n")

// CSS:
#dropdown_a, #dropdown_b, [...] #dropdown_n {
    width:  200px;
}

Ideal behavior might look something like this:
// python:
d1 = dropdown_from_dataframe("dropdown_a", df, "col_a", class="dropdown")
d2 = dropdown_from_dataframe("dropdown_b", df, "col_b", class="dropdown")
[...]
dn = dropdown_from_dataframe("dropdown_n", df, "col_n", class="dropdown")

// CSS:
.dropdown {
    width: 200px;
}

Priority
Low pri
Resources

^^ @benhummel I added some options, let me know if that matches what you were hoping for!

Looks great, thanks @kristiewirth!


HTTPRequest parameter matching: "extension" is stripped off the final value?

SS_HTTPRequest's match function apparently strips off so-called "extensions" off the final parameter in the URL, which is quite unexpected in some cases, for example when an email address is in the URL.
class UnsubscribePage_Controller extends Page_Controller {
    private static $url_handlers = [
        '$Hash/$Email/$Confirm' => 'unsub'
    ];
    private static $allowed_actions = [
        'unsub'
    ];
    public function unsub() {
        var_export($this->request->allParams());
        die();
    }
}

Result:
<EMAIL_ADDRESS>
array (
  'URLSegment' => 'uitschrijven',
  'Action' => 'myHash',
  'ID' => 'email@example',
  'OtherID' => NULL,
  'Hash' => 'myHash',
  'Email' => 'email@example',
  'Confirm' => NULL,
)

As you can see, the .com is stripped off the email address. This only happens if $Email is the final parameter; if the $Confirm parameter is added, the .com  is kept.
I went looking in SS_HTTPRequest::match which mentions: "The pattern can contain a number of segments, separated by / (and an extension indicated by a .)". That's the only explanation I can find about it, and it's between brackets as if it's an afterthought, but this is making it impossible to distinguish between very different URLS:

<EMAIL_ADDRESS>vs<EMAIL_ADDRESS>/download/file.pdf and /download/file.docx
etc

Can the stripping of extensions be turned off? Should it even be enabled by default?.. It's very unexpected!
I am using SS 3.3.1.

The use-case it was designed for was URLs like /api1/v1/SiteTree/5.json vs /api1/v1/SiteTree/5.xml. It made sense to process extensions explicitly in such URLs, but not for the ones you point out. Note that SS_HTTPRequest::getExtension() returns the extension.
It might be a little hard to fix in SS3 without breaking backward compatibility, but maybe with remove or significantly alter this functionality in SS4?
For SS4, I would suggest that we have something in the url_handler system that lets you mark a URL has optionally having an extension, parsed as a separate variable. We can use this in places like restful APIs.

If you're looking for a quick-fix for your own project you might want to do something like:
if ($extension = $this->getExtension()) $params['Email'] .= ".$extension";

A bit ugly but it should do the job.

I worked around it in my own project by making sure the email address is not the final parameter. I don't mind that it keeps working like this in SS3, but would indeed suggest changing it in SS4 because it's just not intuitive if you don't know about it. Perhaps a twin method could be added alongside getExtension to retrieve the part without the extension, so that the 'raw' parameter can be left alone.

Could you use a GET parameter instead? Also perhaps the method stripping off the extension might do it only if it detects one it recognises e.g. .json or .xml

I think we should close this, since the API is behaving as designed. @sminnee's work around solves the problem in your case, right @JorisDebonnet ?

It would be an enhancement ticket, if we were to keep it open, which would be okay if someone puts up their hand to work on the feature in the next 3-6 months.
Alternatively someone could just raise a PR when they're done.
My suggested fix would be to alter extension processing to be ah optional feature of the URL matcher, disabled by default and enabled on APIs and other places where it is of use.


Add 2016-monitor

@FirmlyReality
I think the redirect lines added in the code file could be push to master branch.
Thanks !


add next links to docs

closes #1752

Coverage remained the same at 78.835% when pulling cd09c3a4d74122e9bcbdcd1d306e2b6537499ac3 on abemassry:docs-next-links into 167ab9e9e41aaea3cbac524fe96c792f99a4163a on linode:master.


Only default values displayed or something, not actual values

Example: I have found a unique shield three times. Each of them has slightly different values. TQVaultAE displays the same values for all three, but those are not the actual values of any of the three shields in-game.
Would be a huge help with picking / sorting stuff if TQVaultAE displayed the actual in-game values.

Unfortunately, we have cracked that one yet. See the F.A.Q..


Toggle menu coming up on Large screens

Toggle menu overlay coming up while clicking menu link in large large screen.

i will check later

fixed


Fixed Navbar and Hero Section

Closes #80

I have fixed the hero section and have made it responsive
Changed the opacity of behind image and added it in the responsive layout too

As Instructed, I have fixed the hamburger menu, it had a lot of bugs and was not visible at all in my system
Now it looks like this after my changes

@SujalSamai Please resolve the branch conflicts.

.deploy

@SujalSamai Please open the PR after fixing the branch conflicts and link #80 to that PR.


[SPARK-49966][SQL] Use Invoke to implement JsonToStructs(from_json)

What changes were proposed in this pull request?
The pr aims to use Invoke to implement JsonToStructs(from_json).
Why are the changes needed?
Based on @cloud-fan's suggestion, I believe that implementing JsonToStructs(from_json) with Invoke can greatly simplify the code.
https://github.com/apache/spark/pull/48466#discussion_r1802533505
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Update existed UT.
Was this patch authored or co-authored using generative AI tooling?
No.

+1, LGTM. Merging to master.
Thank you, @panbingkun and @cloud-fan for review.


Remove Gistlyn (Dead Link)

The Gistlyn (https://gistlyn.com/) site is dead. Seems the domain has been sold.
* [Gistlyn](https://gistlyn.com/) - Create, run and share your executable C# GitHub Gists.

Alternatively, it might make sense to direct to their GitHub? https://github.com/ServiceStack/Gistlyn

@John-Paul-R I think the correct action is to remove it. Seems like the website is one of the key features


Fixed dependencies

:+1:

Brace yourself, the "Support 5.2 " issues are coming :( ;) Haha :)

I'll send a PR to allow 5.2.x usage, but I recommend you only merge that at least after symfony 3 has been tagged.

Brace yourself, the "Support 5.2 " issues are coming :( ;) Haha :)

I doubt it. If someone asks composer for ~2.0 of this package, composer will resolve v2.0.9 because you marked it as compatible with all 5.x. That's going to be a pain in the future, for people thinking this works on Laravel 5.4, or whatever.


Fixed documentation warning

It's normal POST data, not a JSON blob. This is incorrect.

im using right now a json object to authenticate and it works. Are you sure that that recording is up to date?

That was over a year ago, but the library works fine. They're running a Ruby (maybe Rails, probably Sinatra) app on their end and those tend to be ambivalent about different input formats (I wouldn't be surprised if they accept XML or YAML too). Given that POST is more standardized than JSON, I'd rather leave this as an HTTP standard interface, rather than adding another format layer on top.

Have a look:
Request: POST https://owner-api.teslamotors.com/oauth/token
Request Body:
 {
  "grant_type" : "password",
  "client_id" : "e4a9949fcfa04068f59abb5a658f2bac0a3428e4652315490b659d5ab3f35a9e",
  "client_secret" : "c75f14bbadc8bee3a7594412c31416f8300256d7668ea7e6e7f06727bfb9d220",
  "email" : "MY_REAL_EMAIL",
  "password" : "MY_REAL_PASS"
}

Result: {"access_token":"A_TOKEN","token_type":"bearer","expires_in":7776000,"created_at":1457465651}

Ok. There is only the warnings issue then.

Finally fixed that. They should have been Attributes.

Cool :)


Safari Extension

Would it be possible to bring this addon to Safari? Apple has the "Safari web extension converter" which is able to convert Chrome addons into a Safari extension using Xcode.

I'm currently unable to support the Safari browser.
Because Apple requires me to pay $100 per year to be an Apple developer so that I can publish the extension to the store. And I would need to buy an Apple device to run XCode.
But, I believe you can convert the extension with XCode on your own Apple device and run it locally in your Safari browser.

Thanks for the reply, I understand it is unrealistic to buy and Apple device in addition to the $100 per year just to port the extension. As for converting the extension myself I ran into issues with archiving then distributing the extension to myself for personal use, I am completely unfamiliar with anything related to coding which has led me to ask if porting the extension officially is possible. Thanks again

No problem.
Let me know when Apple changes their requirements.
Or when you get it working. Then we could write a guide for other Safari users.


Vaccine reservation system in CPP

Describe the project you want to add with tech stack

I'd add a vaccine reserving system, using which the users can apply for the vaccine service for themselves as well as their well knowns.
Features:

Dedicated user login system
User can add their record into the system
User can view all aviliable records in the system
User can view vaccine statistics and apply for the same.
There would be a CLI interface for the same.

Expected behavior

User logins/signs up after executing the program run
Post login, they select appropreate option to choose.
If choosen add records, they add their record through proiding relevant inputs
If choosen view records, they view all records aviliable
If choosen book vaccine, they can book vaccine or view avaliability.

Screenshots (optional)

@pranjay-poddar @TusharKesarwani @MohitGupta121 please assign me this!


context for code snippets

should be optional (set 0-3 lines in options menu)

either needs to be in the JSON emitted by rustc, or (better) we need to do this in the background, since loading context from files will take time.


Include theme folder in plugins scan

@silend to create a new theme, just copy the /theme/default.css into /plugins/myplugin/theme/mypugintheme.css and select it in the user editor.


Automate installing prereqs for local development

To set up your local development environment:

Clone the git repo: $ git clone git@github.com:brainix/bloom-filter.git
cd into the clone: $ cd bloom-filter
Install the software prereqs: $ make install
Run Memcached: $ make services

🐟


Improve changelogs

Currently there is no automation for release notes/changelog (see latest https://github.com/allegro/bigcache/releases/tag/v2.1.1 and https://github.com/allegro/bigcache/releases/tag/v1.2.0).
It will be good to have some automation for this, so changelog will be created automatically (it's fine to paste it manually, but the text should be done automatically).
PLEASE if you're going to start working on this issue, comment below, to discuss the solution, thanks in advance.

Hmm, what do you think the best way to generate the Changelog? Commit message?

@muzavan I think it will be better to add descriptions at Github releases page (see links in issue description). There should be automatic tools for that.

@cristaloleg this is what I was using to generate release information: https://github.com/github-changelog-generator/github-changelog-generator

@mxplusb can we automate this somehow? even makefile command will be enough, imo

Yeah, we definitely can. It's mostly around just automating against tag management, so it's pretty straightfoward.

I've implemented a github action using https://github.com/marketplace/actions/release-drafter.
It will create a release draft every time a pr is merged into the master branch. I can make a pr for this one if you'd like. I think this doesn't interfere with #175 integration tests.

@gnomeria sounds awesome, let's make it! +1 🎉

would be something like this

Good, exactly what good project should have.
Sadly current release will be huge, time to change this.
I'm completely for such change with automatic changelog, hopefully no one objections from others @muzavan @janisz


Allow community members to run the tests in a CI

Describe the bug
At the moment, pull requests created from the community fail to run the Github tests. This happened in two PRs in the last two days:

https://github.com/astro-projects/astro/pull/172
https://github.com/astro-projects/astro/pull/176

Version

Astro: 0.6.0

To Reproduce
Steps to reproduce the behavior:

Fork Astro-projects/astro
Create a PR from the fork into astro-projects/astro
See most tests failing, with messages complaining that credentials are not set, e.g.:

   File "/home/runner/work/astro/astro/.nox/test-3-8/lib/python3.8/site-packages/botocore/signers.py", line 93, in handler
    return self.sign(operation_name, request)
  File "/home/runner/work/astro/astro/.nox/test-3-8/lib/python3.8/site-packages/botocore/signers.py", line 165, in sign
    auth.add_auth(request)
  File "/home/runner/work/astro/astro/.nox/test-3-8/lib/python3.8/site-packages/botocore/auth.py", line 378, in add_auth
    raise NoCredentialsError()
botocore.exceptions.NoCredentialsError: Unable to locate credentials

https://github.com/astro-projects/astro/runs/5463617773?check_suite_focus=true
Expected behavior
Community team members should be able to fork and create PRs and see the tests passing with no secrets issues.

We are in the process of refactoring our integration tests - once this is accomplished, it will be much easier to address this.

There are 2 options here:

Only run unit tests (which is a larger work as currently not all integrations tests are marked as integration)
Committers (one with write access to repo) can add a certain label to the PR to mark the PR as "trusted" and hence allow the PR to run tests with secrets from the CI


pattern match fixes

Update logic for including/excluding patterns
Resolve comments of previous PR

looks much cleaner!!


Reworks InitializeInfectionsExponentialGrowth to work with numpyro.plate

@dylanhmorris @sbidari docs are not rendering. Probably worth a broader discussion about what components of PyRenew we expect to "just work" in plates before I attempt a fix.

@dylanhmorris @sbidari docs are not rendering. The problem is that the shape of the output of the initialization method for a non-plated context has changed from (x,) to (x,1) (in a plated context is will be (x, n_plates)). Probably worth discussing a general plan for handling this type of change throughout PyRenew before I attempt a fix.

Is there a reason not to use jnp.squeeze in contexts like this?

LGTM. Tests could probably be DRY-ified via pytest.mark.parametrize but that can be a separate PR.

I thought about that, but the logic to determine the manual computations seemed nearly as verbose as the fully manual solution, while being less readable.


Scrollspy doesn't respect "display:none".

For my use case, I have two separate "nav" classes that I use scrollspy with. I never use them simultaneously, and to switch between them i set the display of one to none and the other to visible.
However, scrollspy does not work in accordance with element visibility making it not work unless the element is actually removed from the DOM.

That's true. Why would you have two navs in page at once? I'm assuming they are almost identical and that the buttons are being activated on the hidden one...

at any rate - checking for style on scroll would kill performance just isn't something we can do. sorry :(

Ah, didn't see you responded here for some reason. What I ended up doing was simply modify scrollspy's jQuery selector to include a special class -- I called it 'activenav'. I would then have whichever navbar I wanted displayed set its link classes to 'activenav', refresh scrollspy, and it's all good. No performance hit, sans needing to refresh the scrollspy coordinate cache when switching navbars.
You asked about my use case -- it's this: view sorting. That is, the navbar might show dates when the app is in 'sort-by-date' mode, and categories when it's in 'sort-by-category' mode. Perhaps this wasn't the best strategy, but it seems to work for my use case.


Inaccurate error message when snapshotting container with existing snapshot name

When snapshotting container using existing snapshot name the error message displayed is inaccurate. Now the error message is "The container already exists" even though I am trying to create a snapshot, not a container. It is also possible to have snapshots named after containers (container foo with snapshot bar, see steps to reproduce), so for this reason too the error message is misleading.
I propose that the error message is modified to print "error: Snapshot 'wily/bar' already exists" or similar indicating the name of the container snapshotted and the name of the existing (duplicate) snapshot.
If above mentioned error message is deemed appropriate (or some other is more fitting), I can implement the modifications to lxd/container.go#L597 and create a PR.
Steps to reproduce

lxc launch ubuntu:wily foo
lxc launch ubuntu:wily bar
lxc snapshot foo bar
lxc snapshot foo bar

Sure, that sounds reasonable. A PR would be great, thanks!

Fixed in 58835f9


Dropdown for languages

As the list of languages grows, the buttons will get unwieldy. They are getting that way already.

I propose something @whitep4nth3r mentioned on stream yesterday (5th Apr 2021) which is a dropdown for languages. But by taking it one step further we can introduce a second dropdown purely for lols which is a list of esoteric languages such as LOLCODE, Brainfuck, Rockstar and the like.

I think adding more and more buttons just adds to the ridiculousness


Wallet Name

During set up it asks to name wallet. 'Wallet' is already filled in. Any reason not to use 'Wallet'? If not, suggest new user keep 'wallet'

Because you can make and restore multiple wallets on the same Bails USB and need a unique name to disambiguate them in Bitcoin Core.

After the identifier is changed to public masterkey fingerprint this question can be moved to the last step of Restore wallet before the info about encryption.
It still needs to be a pre-backup step of 'Create wallet'

Naming the wallet should come after the mentioned dialog. The flow should be as follows:
Show the dialog with the choices: "Create a new seed," "I already have a seed," and "Import descriptors."
Based on the user's choice in the dialog, proceed accordingly:
a. If the user selects "Create a new seed," continue to the next step to ask for the wallet name.
b. If the user selects "I already have a seed" or "Import descriptors," proceed with the respective actions.

Remove naming the wallet from "Create a new seed"
Because none of it has anything to do with a wallet and it doesn't matter what the wallet is named in that function block
The new wallet will get named after 'Restore wallet' has recovered seed and unique id
That way a user can name the wallet something meaningful based on the seed they created or restored.
And the default name can include the identifier if it needs to helping the user remember which shares are to their own wallet and which are to someone elses.
Cons to asking for wallet name first
it felt weird being asked to update the name due to collision, like minutes later after setting the name.
Grabbing the wallet name first for extra entropy in seed creation isn't that helpful, either they add enough entropy in the 'Add Randomness' step, or they don't really get enough from "name, threshold, locations, identifier" to be secure against a backdoor unless they rolled dice to make their passphrase and most people leave it at "Wallet".
Another reason it doesn't belong in 'Create a seed' is a custom wallet name cannot be recovered from the codex32 backup.
if you name it Purple Donkey, you can't have that name be automatically restored when you later choose "I have a seed"
meaning: wallet name is NOT part of "creating a seed backup"

Contextual decision: Users are more likely to make an informed decision about naming their wallet after understanding the options for seed creation. By providing them with the information about creating a new seed or using an existing one first, they can better comprehend the implications of their choices and how it may relate to the wallet name.

Reduced ambiguity: Naming a wallet without knowing whether it will be a new seed or an existing one can lead to confusion. Users might unintentionally give a name that doesn't match the type of seed they eventually use, which can cause confusion and make it harder to manage multiple wallets.

Streamlined flow: By first asking about seed creation, the flow becomes more streamlined and intuitive. Users will first decide on the core aspect of the wallet creation process (seed choice) and then move on to the secondary aspect (naming the wallet).

This has been resolved. It asks for the wallet name right after the seed is successfully restored. I'm not currently defaulting the identifier into the name unless there is a collision with the wallet name chosen. But I'm open to difference of opinion about this. Just open a new issue about it with your argument to make it default to say
"Wallet 7jsn" so you know exactly what shares belong to your wallet. That said, the information is in listdescriptors so it is unnecessary to to put it in the wallet name if that makes the shoulder surfing privacy better. Wallet names are not masked.


Activated all tests in test.contrib

Realized they weren't running.
Before: Ran 407 tests in 40.602s
After:  Ran 426 tests in 42.342s

This is also why coveralls doesn't display coverage for hive.py in #720


Test successful update two images

Add a test case for Update Two Images
Related to issue #261

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


Implement the Causal Cluster Variance Estimator following Abadie et al

Basic implementation seems to work now:
# load autoreload extension
%load_ext autoreload
%autoreload 2

from pyfixest.estimation import feols
from pyfixest.ccv import ccv
import pandas as pd
import numpy as np

df = pd.read_stata("C:/Users/alexa/Downloads/census2000_5pc.dta")
fit = feols("ln_earnings ~ college", vcov = {"CRV1": "state"}, data = df)
fit.ccv(treatment = "college", pk = 0.05)

while the Stata version reports
.


Feat/#115 enable add partners view

Description
Updated the socios view, now it shows a table where render all the socios and allows to change the active status and it has a edit icon to can edit the socio.
Some issues to take in count:

The color doesn't match with the design, the $success color variable in colors.scss file is different that in the design provided.
The icon doesn't match the design, that's because I couldn't find it online, even using Icons8, so I used another similar icon.
It doesn't update the user active status in the database because I couldn't find the way to update it, the UPDATE_USER query ask for a password field and I don't have that data.

fixes #115
Screenshots

Type of change

[x] New feature (non-breaking change which adds functionality)

How to test it
If applicable, describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

Go to 'socios'

Checklist:

[x] Doesn't break the current code.
[x] Passes linters and test, also is building.
[x] Doesn't have spelling or grammatical problems.
[x] Doesn't have unnecessary comments or debugging code.
[x] The branch is updated with the last dev branch changes.

the container needs to be centered, and responsive at least up to 1200px.
chrome_U8UUEelkxI.mp4

Should it cover all the screen or just be centered?

Cover all screen:

https://user-images.githubusercontent.com/77207702/188473561-9a7c088e-3273-437c-a01a-3bfb82b66cd2.mov

Centered:

https://user-images.githubusercontent.com/77207702/188473470-6ab8bada-85b9-4812-a1ed-021aaa14aa0c.mov

⚠ Please check the spacing and font size of the titles and the list, in Adobe XD there is a guide of the sizes it should have, please follow them.
Current

Expected Result
- Textos

- Titulos

you can also get the icon from there

Please check the spacing and font size of the titles and the list, in Adobe XD there is a guide of the sizes it should have, please follow them.

also change the color of active text and toggle button

Current

Expected Result
- Textos

- Titulos

you can also get the icon from there

Ok, I made some changes to try to match a little more the design provided.

I still need to fix the icon but I haven't found a way to get the icon from the design.

https://www.svgrepo.com/svg/351981/edit for the icon. Just add fill="color"

Done. I had to create an EditIcon component and use the svg from Font Awesome official site.

Changes made

Satisfactory change of images.
Change texts
Change colors

⚠ the only thing I see is that the horizontal scroll does not appear until the end of the list and that does not allow a good experience.

Like this?
I removed the "Crear nuevo socio" because it was the way that I made it work.
https://user-images.githubusercontent.com/77207702/188511390-f61b3dfa-eabc-43f4-a562-b7bd280d935f.mov

There must be something in the percentages that you are using to adjust the content to the space that it does not give you, I think that @victor-heliomar @Israel-Laguan @e-azocar can help you better than me in that.

There must be something in the percentages that you are using to adjust the content to the space that it does not give you, I think that @victor-heliomar @Israel-Laguan @e-azocar can help you better than me in that.

Ok, if someone can help with that, it would be excellent.
The unique idea that I have right now is set the ListHeader component height as a fixed height, and save it as a css variable, then use
height: calc(100vh - var(--header-height));
So it will cover available screen space.

The unique idea that I have right now is set the ListHeader component height as a fixed height, and save it as a css variable, then use height: calc(100vh - var(--header-height));
So the table will cover all the available screen space.

What about adding some padding/margin at that viewport size?

Well, I think it's done, I almost fixed the responsive to cover 1200px, there are some big limitations when working with tables that made it really hard to make changes like adding a css overflow-ellipsis.
And the table show the scrollbar on any screen.
https://user-images.githubusercontent.com/77207702/188523188-90a1287b-1391-4df4-8fb4-1cf6ebc1d6e9.mov

Well, this is gonna take more time. I made a rebase with dev and now the pagination is over the table.
https://user-images.githubusercontent.com/77207702/188685515-c55da267-e26c-4f81-a3ca-89296150514f.mov

Well, this is the result for now. The socios view now has the pagination and it's working properly.
https://user-images.githubusercontent.com/77207702/188695779-3ba37c55-9f2e-4769-9e2f-50cf53441597.mov


chore(component libraries): add vivid

What I did

added Vonage's vivid web components library to the componentLibraries list

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


'os' not defined, resolved by updating outdated Docker image

Thank you for #50 ! Ever since the update, attempting to run the program gives an 'os' not defined error. It seems like this has something to do with attempting to read environment variables in the config.py file.
Traceback (most recent call last):
  File "/usr/local/bin/uvicorn", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/uvicorn/main.py", line 410, in main
    run(
  File "/usr/local/lib/python3.12/dist-packages/uvicorn/main.py", line 577, in run
    server.run()
  File "/usr/local/lib/python3.12/dist-packages/uvicorn/server.py", line 65, in run
    return asyncio.run(self.serve(sockets=sockets))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/asyncio/runners.py", line 194, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "uvloop/loop.pyx", line 1517, in uvloop.loop.Loop.run_until_complete
  File "/usr/local/lib/python3.12/dist-packages/uvicorn/server.py", line 69, in serve
    await self._serve(sockets)
  File "/usr/local/lib/python3.12/dist-packages/uvicorn/server.py", line 76, in _serve
    config.load()
  File "/usr/local/lib/python3.12/dist-packages/uvicorn/config.py", line 434, in load
    self.loaded_app = import_from_string(self.app)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/uvicorn/importer.py", line 19, in import_from_string
    module = importlib.import_module(module_str)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 995, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/root/faster-whisper-server/faster_whisper_server/main.py", line 80, in <module>
    cors_origins = os.environ.get("CORS_ORIGINS", "").split(",")
                   ^^
NameError: name 'os' is not defined. Did you forget to import 'os'?

Was able to fix this by removing my docker image for latest-cpu with
docker rmi fedirz/faster-whisper-server:latest-cpu

and then reinstalling from source!
I didn't realize but it seems like Docker attempts to reuse cached images even when new updates are pushed. Is the fix in the future to be something like docker pull fedirz/faster-whisper-server:latest-cpu? Was wondering what would be best practice and if any of this should be incorporated into the README.
Thank you once again!

I think it was using the image you had previously built (for #47) rather than the one pulled from Dockerhub.


fix logging

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

LGTM


Refactor config to enable all-local dev URLs.

This will be in conflict with https://github.com/dart-lang/pub-dartlang-dart/pull/223, please merge only one of them, I'll rebase the other.


Fixing broken link: yourfirstpr is not working.

The "Your FIrst PR" link under "Open Source Websites" section is throwing an "Error:404" upon request.
I have searched for the relevant GitHub Pages and the repo, but it seems like the original author has deleted them.
Therefore, it is requested to remove this link permanently.

@sdmg15 I have also opened a PR for fixing the broken link, kindly merge it :)


Feat/276 Table

Resolves https://taiga.sovryn.app/project/sovryn-light/task/276
@t-z-u please review new table on /storybook.

Still waiting on input from @t-z-u.


tf.random.log_uniform_candidate_sampler gives undesired true class

From tensorflow's word2vec tutorial: https://www.tensorflow.org/tutorials/text/word2vec#define_loss_function_and_compile_model
Nagative sampling is conducted using tf.random.log_uniform_candidate_sampler. Given the context class (true class), the goal is to sample negative classes from the whole vocabulary list. To my understanding, The negative classes must differ from the given context class. However, I found that the context class may appear in the negative classes sampled by tf.random.log_uniform_candidate_sampler. Here is the code:
import tensorflow as tf
SEED = 42

# encode the words
sentence = "The wide road shimmered in the hot sun"
tokens = list(sentence.lower().split())
vocab, index = {}, 1 # start indexing from 1
vocab['<pad>'] = 0 # add a padding token
for token in tokens:
  if token not in vocab:
    vocab[token] = index
    index += 1
vocab_size = len(vocab)
print(vocab)
inverse_vocab = {index: token for token, index in vocab.items()}
print(inverse_vocab)

# make (hot, the) as a context pair
target_word, context_word = 6, 1
print("target: {}, context: {}".format(inverse_vocab[target_word], inverse_vocab[context_word]))

# negative sampling
# Set the number of negative samples per positive context.
num_ns = 4

context_class = tf.reshape(tf.constant(context_word, dtype="int64"), (1, 1))
negative_sampling_candidates, _, _ = tf.random.log_uniform_candidate_sampler(
    true_classes=context_class, # class that should be sampled as 'positive'
    num_true=1, # each positive skip-gram has 1 positive context class
    num_sampled=num_ns, # number of negative context words to sample
    unique=True, # all the negative samples should be unique
    range_max=vocab_size, # pick index of the samples from [0, vocab_size]
    seed=SEED, # seed for reproducibility
    name="negative_sampling" # name of this operation
)
print("negative samples\' index: ", negative_sampling_candidates.numpy())
print("negetive samples: ", [inverse_vocab[index.numpy()] for index in negative_sampling_candidates])
# "the" will show in negative samples, if not, run it several times.

The word "the" is the context class of word "hot", why it could show in the sampled negative classes? Moreover, the target word hot could also be sampled as negative class. Do I misunderstand something?

Have you already seen https://www.tensorflow.org/extras/candidate_sampling.pdf ?

Have you already seen https://www.tensorflow.org/extras/candidate_sampling.pdf ?

I've read that. If you mean

The random choice of S_i may or may not depend on x_i and/or T_i.

But if positive sample can show in negative samples, why the function have this true_classes argument? And what does the comment # class that should be sampled as 'positive' in the word2vec tutorial mean, what's the effect of sampled as positive?

Was able to reproduce the issue with TF v2.3 and TF-nightly. Please find the gist of it here. Thanks!

@Yu-Qing-Wang I am not deeply in this op but I think that true_class interpretation is in https://github.com/tensorflow/tensorflow/blob/v2.3.1/tensorflow/python/ops/candidate_sampling_ops.py#L120-L128

Was anyone able to fix this issue? I had the same question and would appreciate an explanation of why thr true_classes is included in the distribution returned.
Thanks!

@MarkDaoust Do you think that we could improve the documentation here?

Facing the same issue. Does using tf.nn.fixed_unigram_candidate_sampler instead of tf.random.log_uniform_candidate_sampler for negative sampling not cause this issue of generating undesired true class also?

Facing the same issue. Does using tf.nn.fixed_unigram_candidate_sampler instead of tf.random.log_uniform_candidate_sampler for negative sampling not cause this issue of generating undesired true class also?

From what I understand there's a probability attached to each label while sampling. So there is a chance that the true_label gets returned as a false label while generating the context classes here. I might be wrong here, but the PDFs and the code snippet referenced above point in that direction.

I think this is designed to be, but the line pass the context word as true class to exclude it from being sampled in the tutorial is really misguiding. I'm just learning this from the tutorial, so there maybe mistakes in my understanding.
In my test, true_classes has no effect on the sampled results, seems it's only used to calculate the second returned value true_expected_count:# Do sampling 1000 times using true_classes [0, 8]
sample_func = lambda ii: tf.random.log_uniform_candidate_sampler(true_classes=[[ii]], num_true=1, num_sampled=4, unique=True, range_max=8, seed=42)
dd = {ii : np.stack([sample_func(ii)[0].numpy() for jj in range(1000)]) for ii in range(8)}
# Calculate the distribution in each true_class
for ii in dd:
    print("true_class:", ii, ", negative value_counts:", pd.value_counts(dd[ii].flatten()).to_dict())
# true_class: 0 , negative value_counts: {0: 871, 1: 722, 2: 584, 3: 466, 4: 402, 5: 329, 7: 319, 6: 307}
# true_class: 1 , negative value_counts: {0: 867, 1: 695, 2: 571, 3: 485, 4: 411, 5: 380, 6: 316, 7: 275}
# true_class: 2 , negative value_counts: {0: 869, 1: 716, 2: 541, 3: 488, 4: 389, 5: 357, 6: 321, 7: 319}
# true_class: 3 , negative value_counts: {0: 877, 1: 715, 2: 582, 3: 482, 4: 394, 5: 355, 6: 318, 7: 277}
# true_class: 4 , negative value_counts: {0: 883, 1: 716, 2: 566, 3: 489, 4: 394, 5: 367, 6: 316, 7: 269}
# true_class: 5 , negative value_counts: {0: 862, 1: 717, 2: 583, 3: 496, 4: 376, 5: 357, 6: 315, 7: 294}
# true_class: 6 , negative value_counts: {0: 859, 1: 725, 2: 575, 3: 482, 4: 413, 5: 356, 6: 302, 7: 288}
# true_class: 7 , negative value_counts: {0: 880, 1: 724, 2: 555, 3: 488, 4: 425, 5: 324, 7: 302, 6: 302}

# Result of `true_expected_count`
print({ii : np.mean([sample_func(ii)[1].numpy() for jj in range(1000)]) for ii in range(8)})
# {0: 0.99967235, 1: 0.7245632, 2: 0.5737029, 3: 0.47004792, 4: 0.3987442, 5: 0.34728608, 6: 0.3084587, 7: 0.27554017}

Simmilar result with tf.nn.fixed_unigram_candidate_sampler or tf.keras.preprocessing.sequence.skipgrams with negative_samples=4:unigrams = [0.99967235, 0.7245632, 0.5737029, 0.47004792, 0.3987442, 0.34728608, 0.3084587, 0.27554017]
sample_func = lambda ii: tf.nn.fixed_unigram_candidate_sampler(true_classes=[[ii]], num_true=1, num_sampled=4, unique=True, range_max=8, unigrams=unigrams)
dd = {ii : np.stack([sample_func(ii)[0].numpy() for jj in range(1000)]) for ii in range(8)}
print("true_class:", 0, "value_counts:", pd.value_counts(dd[0].flatten()).to_dict())
print("true_class:", 7, ", negative value_counts:", pd.value_counts(dd[7].flatten()).to_dict())
# true_class: 0 , negative value_counts: {0: 793, 1: 666, 2: 546, 3: 475, 4: 426, 5: 382, 7: 356, 6: 356}
# true_class: 7 , negative value_counts: {0: 759, 1: 674, 2: 579, 3: 526, 4: 418, 5: 386, 6: 356, 7: 302}

That means, if 0 representing the word the and it's a context_word, which would be a high probability, it also will be selected as a negative sample with high probability no matter what the context_word is.
But in the original word2vec TrainModelThread I think the positive word is actually excluded in negatives. In my understanding, I think current behavior of log_uniform_candidate_sampler is better.
Actually in a real practice like the shakespeare dataset, the ratio selecting a positive as negative in the same sample is low, so this behavior may doesn't matter:# After generated `targets, contexts, labels` following the tutorial
rr_contexts = np.array(contexts)[:, :, 0]
rr = [rr_contexts[ii][0] in rr_contexts[ii][1:] for ii in range(rr_contexts.shape[0])]
print("Total negatives containing positive:", np.sum(rr), ", ratio:", np.sum(rr) / rr_contexts.shape[0])
# Total negatives containing positive: 2226 , ratio: 0.03430843685459758
print("Sample:", rr_contexts[np.array(rr)][:5].tolist())
# Sample: [[1, 3, 0, 73, 1], [1, 1, 2, 47, 625], [4, 9, 717, 11, 4], [8, 15, 37, 26, 8], [1, 97, 1, 4, 120]]

Moreover, it's a high ratio that a positive pair (target_word, context_word) would also be a negative sample pair:targets, contexts, labels = np.array(targets), np.array(contexts), np.array(labels)
dd = pd.DataFrame({"targets": targets.tolist(), "pos": contexts[:, 0, 0].tolist(), "neg": contexts[:, 1:, 0].tolist()})
cc = dd.groupby('targets').apply(lambda ii: np.sum([jj in ii['pos'].values for jj in np.concatenate(ii['neg'].values)]))
print("Total negative pairs containing positive pair:", cc.sum(), "ratio:", cc.sum() / dd.shape[0])
# Total negative pairs containing positive pair: 38660 ratio: 0.5953095887035925

So IMO, we need an update on the tutorial description, really misguiding.

According to the big table in https://www.tensorflow.org/extras/candidate_sampling.pdf , the phrase "negative samples" may mean different things in different algorithms. For example, in NCE "negative samples" means S_i (which is just the sampled classes) which may overlap with true classes, while in Sampled Logistic "negative samples" means S_i - T_i which excludes the true classes.
For tf.random.log_uniform_candidate_sampler, the return value sampled_candidates corresponds to S_i, not any specific definition of "negative samples" in any specific algorithm. It's the caller's responsibility to pick an algorithm and calculate the "negative samples" defined by that algorithm (e.g. S_i - T_i).
The true_classes argument is for calculating the true_expected_count output (as a by-product of the op's main calculation), which may be needed by some algorithms (according to https://www.tensorflow.org/extras/candidate_sampling.pdf). It's not for excluding true classes in the return value sampled_candidates (again that step is algorithm-specific and should be carried out by the caller).

@Wang-Yu-Qing
Is this still an issue

Even if Wang-Yu-Qing is fine, I think we need to update documentations to make clear the meaning of "negative samples".

Agreed. We'll track the documentation update in #49490.

In that case, I will close this issue and look at  #49490.

In that case can this be move to closed status.

Closing this issue and tracking the documentation update in #49490. Feel free to reopen.

According to the big table in https://www.tensorflow.org/extras/candidate_sampling.pdf , the phrase "negative samples" may mean different things in different algorithms. For example, in NCE, "negative samples" means S_i (which is just the sampled classes) which may overlap with true classes, while in Sampled Logistic "negative samples" means S_i - T_i which excludes the true classes.
For tf.random.log_uniform_candidate_sampler, the return value sampled_candidates corresponds to S_i, not any specific definition of "negative samples" in any specific algorithm. It's the caller's responsibility to pick an algorithm and calculate the "negative samples" defined by that algorithm (e.g. S_i - T_i).
The true_classes argument is for calculating the true_expected_count output (as a by-product of the op's main calculation), which may be needed by some algorithms (according to https://www.tensorflow.org/extras/candidate_sampling.pdf). It's not for excluding true classes in the return value sampled_candidates (again that step is algorithm-specific and should be carried out by the caller).

@wangpengmit
In this section the idea of negative sample is defined different from what you are highlighting in the comment.

A negative sample is defined as a (target_word, context_word) pair such that the context_word does not appear in the window_size neighborhood of the target_word. For the example sentence, these are a few potential negative samples (when window_size is 2).

Based on this do you think we should explicitly exclude the context class from negative sampling?


--disable-uncorrectable option does not insert todo comment as expected for multiline case

thank you for the fix to https://github.com/rubocop/rubocop/pull/11701
but, another issue similar to https://github.com/rubocop/rubocop/issues/11685 is there.
Expected behavior
Rubocop should process
ARRAY = %w[
  AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
].freeze

as
# rubocop:todo Layout/LineLength
ARRAY = %w[
  AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
].freeze
# rubocop:enable Layout/LineLength

for "--disable-uncorrectable" option
Actual behavior
it become infinite loop and I got
# rubocop:todo Metrics/CollectionLiteralLength
ARRAY = %w[
  # rubocop:todo Layout/LineLength
  # rubocop:enable Layout/LineLength
  # rubocop:enable Layout/LineLength
  ...
  # rubocop:todo Layout/LineLength
  # rubocop:todo Layout/LineLength
  # rubocop:enable Layout/LineLength
  # rubocop:todo Layout/LineLength
  AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
# rubocop:enable Layout/LineLength
  # rubocop:todo Layout/LineLength
# rubocop:enable Layout/LineLength
  ...
# rubocop:enable Layout/LineLength
  # rubocop:todo Layout/LineLength
  # rubocop:todo Layout/LineLength
  # rubocop:enable Layout/LineLength
].freeze
# rubocop:enable Metrics/CollectionLiteralLength

Steps to reproduce the problem

create test_case.rb
create.rubocop.yml
run bundle exec rubocop -A --disable-uncorrectable test_case.rb

test_case.rb
ARRAY = %w[
  AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
].freeze

.rubocop.yml
Layout/LineLength:
  Max: 30

RuboCop version
This happens with HEAD of master (4346f8ef4)
$ [bundle exec] rubocop -V
1.48.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.27.0, running on ruby 3.0.5) [x86_64-darwin22]
  - rubocop-performance 1.16.0
  - rubocop-rake 0.6.0
  - rubocop-rspec 2.19.0

Ahh I probably made a mistake.
I think I should have used overlaps? instead of crossing? here:  https://github.com/rubocop/rubocop/blob/4346f8ef496599c1dfc53332df5116de735e3382/lib/rubocop/cop/autocorrect_logic.rb#L85


Added Tech-Giants-Acquisitions Database which Includes Founders , Board Members, Acquisitions, Acquiring and Acquired Companies.

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Good one @airajena23
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow these steps.

Star this repository. ⭐
SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .
Go to the profile section from the top right-hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard!
And don’t forget to share your achievement with your friends.


multi: wait for rpc MW registration complete msg

In this commit, we update the RegisterRPCMiddleware function to wait for
the server to indicate that it has completed registering the middleware
interceptor before returning from the function. This allows clients to
register mw clients in a deterministic order.
Depends on https://github.com/lightningnetwork/lnd/pull/6754
Not familiar with how branches are organised in this repo. So not sure if this is opened against the correct branch?
Pull Request Checklist

[ ] PR is opened against correct version branch.
[ ] Version compatibility matrix in the README and minimal required version
in lnd_services.go are updated.

@ellemouton, remember to re-request review from reviewers when ready

updated to point to merged lnd pr :)


Repo sync for protected CLA branch

The pull request is created from master637366551832663930 to master to fix git push error for protected CLA branch

@ghogen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


[lbry] config: allow non-localhost connections with TLS disabled

close #67

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
13 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.008%) to 51.836%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

config.go
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

mempool/mempool.go
1
66.23%

integration/rpctest/blockgen.go
2
83.78%

peer/peer.go
10
76.31%

Totals

Change from base Build<PHONE_NUMBER>:
0.008%

Covered Lines:
22477

Relevant Lines:
43362

💛  - Coveralls


chore(kubernetes): update docs to reflect V2 as preferred provider

Closes https://github.com/spinnaker/spinnaker.github.io/issues/1582
- chore(kubernetes): remove kubernetes v1 provider codelab
We no longer want to steer new Spinnaker users to the Kubernetes V1 provider because it is in maintenance mode.
- chore(kubernetes): updates to setup to reflect v2 provider as preferred kubernetes provider
Re-orders the list of cloud providers in the Setup nav so that the V2 provider appears first. Adds text to v1 and v2 provider instructions making it clear that V2 is the recommended path, and V1 is in maintenance mode.
- chore(kubernetes): remove unused placeholder file
Removes unused file leftover from initial commit to add placeholder k8s docs.

hmm, RTM label doesn't work here.


Add failing tests for MissingReadOnlyPropertyAssignRule

Closes https://github.com/phpstan/phpstan/issues/7337
Looks like the problem is actually here in PropertiesExtension (which is now used at least). https://github.com/phpstan/phpstan-doctrine/blob/1.3.6/src/Rules/Doctrine/ORM/PropertiesExtension.php#L85 returns always false, because $metadata->isReadOnly (which marks if this is a Doctrine readonly entity?) is already false.
Does this need another early exit dealing with native readonly or smth like that? I gave it a shot, but don't know much about this here. Feel free to finish it :)

ok, that looks better I think. The isAlwaysWritten and isAlwaysRead are still not tested though. Any good test case ideas? Or anything else you need? I can add it here or in follow-ups, whatever is preferred.

Most of PropertiesExtension should be covered now, sorry for the many force-pushes 😅

based on your comments I found another simplification that I did. Not sure why it is there exactly, but I also took over that getIdentifierFieldNames exception handling.

@herndlm Because ClassMetadata can throw various exceptions and I don't want PHPStan crashing on that. There's a new bleedingEdge rule (https://github.com/phpstan/phpstan-doctrine/blob/1.3.x/src/Rules/Doctrine/ORM/EntityMappingExceptionRule.php) that's gonna tell you if Doctrine doesn't like your mappings.

Thank you very much :)


Expose workflow.labels and workflow.annotations as JSON string

Summary
It is possible to access individual parameters, labels, or annotations from within a workflow via

workflow.parameters.<NAME>,
workflow.labels.<NAME>,
and workflow.annotations.<NAME> respectively.

It is also possible to get all parameters as a JSON string:

workflow.parameters

It would be great if also labels and annotations could be exposed that way:

workflow.labels
workflow.annotations

Use Cases
I have for instance some default templates in the configuration of the controller which I would like to control via the labels/annotations on the particular workflows. Also it would be nice to introspect labels like submitted from ui or cron workflow for complex workflows that spawn other workflows (they generate names based on this and there are notifications in our system which are confusing if they all look the same which I would like to control via the labels).
It would be nice to have all labels exposed that way as this allows argo lint to still validate references to missing variables. Also good to just hook them into a python script step for instance and then deal with that.

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

@scravy it is a good enhancement. Do you. like to submit the PR for this enhancement?

@sarabala1979 Thank you for pointing out the appropriate place in the code. I'll give it a try over the weekend!

@scravy if you have too many on your plate. We have a new team member. He can help with this.
Please let me know it is ok to you

Sure, that's okay.

@dpadhiar Please take this issue


Support for big endianness missing

It looks like .sav files created in big endian OSes are failed to get parsed correctly. If I understand it correctly, LayoutCode is supposed to be used to detect endianness which should later be applied to all Int32 reads. If LayoutCode = 0x00000002 then no endianness change is needed [on Windows]. But if it's 0x02000000 then all integers have to be read as big-endian (currently not implemented).
I'll try to create a pull request to fix this.

Thanks!
Jonathan Pierce

From: Dmitry Gudkov<EMAIL_ADDRESS>Sent: Friday, November 10, 2017 4:14:25 PM
To: TNSGlobal/SPSS-.NET-Reader
Cc: Subscribed
Subject: [TNSGlobal/SPSS-.NET-Reader] Support for big endianness (#27)
It looks like .sav files created in big endian OSes are failed to get parsed correctly. If I understand it correctly, LayoutCode is supposed to be used to detect endianness which should later be applied to all Int32 reads. If LayoutCode = 0x00000002 then no endianness change is needed [on Windows]. But if it's 0x02000000 then all integers have to be read as big-endian (currently not implemented).
I'll try to create a pull request to fix this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/TNSGlobal/SPSS-.NET-Reader/issues/27, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMzcCoJ0_5qLczGjiOuuXKengMQQt74Bks5s1MrAgaJpZM4QaLgG.


Roadmap / plan

Hi,
Is there any roadmap / plan / idea about the future of this library?
We use it a lot in new applications and converted a few existing one's since we notice that the WebApi performs faster then the SOAP library (less data going over the wire), next to that the WebApi library doesn't carry the whole SuperOffice SDK and dependencies with it, causing a lot less conflicts.
Currently the last release, preview3, was 6 months ago. Any indication for a 'stable' release with the latest API additions?

Hi,
We at i-Centrum are also interested in the roadmap of future releases and when the 1.0.0 release is planned.
Regards

I believe this issue has been answered on the docs.
This library is meant as a preplacement for SuperOffice.NetServer.Services, and all future investment in web services API will be with SuperOffice.WebApi.


docs: use absolute url in api-service doc

This is breaking links on the website, and this way every time we sync we won't have to remember to fix this for the latest version. We only sync the Documentation folder, so linking to things outside of it breaks.

Unfortunate, but we'll deal with it for now.


ci: configure the protected branch

Configures CI for branch
Precondition: Merge #1241

Changing LTS to 1.19.x


Versioning of deleted/overwritten files

Hi!
Is there any way to set in the sync or backup mode that deleted/overwritten files get versioned to e. g. a hidden subfolder and it is possible to e. g. keep 5 versions?
That would be great, because I am synching from Azure to a local machine as backups. If someone would delete files accidentally on azure, then also the synched files would disappear. The same would be if I set it to "copy" and a new version of a file overwrites.

Hey @martinweihrauch ,
If you're using blob storage, I think versioning feature might be what you are looking for. which when enabled on your Azure storage account, can help restore an earlier version of a blob to recover your data if it is erroneously modified or deleted. After you enable blob versioning for a storage account, Azure Storage automatically maintains versions for every blob in the storage account.
In AzCopy, we support downloading/deleting given blob versions currently. If you overwrite an already existing blob, instead of deleting the previous version, a new version will be created.

This feature is still not supported in sync but we look forward to adding that in the future releases.

Unfortunately, we are not going to add this functionality. We are focused on documentation, testing, and quality, and this feature would represent a large increase in scope for the product.


Sync Java JET Template with generated code

The JET Templates should now generate code that is exactly the same as what was already existing in the repository.
Relates to issue #697, although not a complete solution.
Until that issue is fixed, will not be able to run the codegen.xml since will overwrite updated Java JET templates

I have verified that the students in the fall all worked on issues unrelated to Java generation, such as analysis, website, php generation. So I am pretty sure we are safe here now.


WIP: Minor edits and formatting fixes

fixed URL typo
Adding oio:consider to table
fixing list formatting. Fixes #233

Is this WIP, then use draft PR!


signOut not working

I'm using jToker alongside devise_token_auth, nothing but standard.
I can't logout using $.auth.signOut(). I got a 404 in the console, and when I curl the request I get
"errors":["User was not found or was not logged in."]
I found this SO post
http://stackoverflow.com/questions/32507637/why-am-i-unable-to-sign-out-using-devise-token-auth-and-curl
but I can't succeed with this method, and I think that jToker documentation is missing something or outdated.

I found the solution in @kunovski answer to this issue : https://github.com/lynndylanhurley/j-toker/issues/7
it was a problem of CORS configuration (server-side pb). I quote his answer:
You should do this:
Rails.application.config.middleware.insert_before 0, Rack::Cors do
  allow do
    origins 'example.com'

    resource '*',
      headers: :any,
      methods: [:get, :post, :put, :patch, :delete, :options, :head]
  end
end

Instead of:
Rails.application.config.middleware.insert_before 0, Rack::Cors do
  allow do
    origins 'example.com'
    resource '*',
      headers: :any,
      expose: ['access-token', 'token-type', 'client', 'expiry', 'uid'],
      methods: [:get, :post, :put, :patch, :delete, :options, :head]
  end
end

You still need expose: ['access-token', 'token-type', 'client', 'expiry', 'uid'] in there


Incorrect namespace for LDES

The currently configured namespace for LDES is http://w3id.org/ldes# but it should be https://w3id.org/ldes#

to avoid possible issues with existing apps, will add a new environment variable that will use the correct namespace if it's set to true. It will eventually become the default in a future major release.


Remove spurious prefixes

Sorry, it seems my branch was out of sync and the changes I wanted to proposed had already been done 🚀
keep up the nice work


Placholder text on image upload wrong in Arabic

'Enter' is in the wrong part of the message, probably because the text direction going from right to left has confused the way the string is formed

Done


kubelogin ENOENT

    name: Set secret
    uses: azure/k8s-create-secret@v4
    with:
      namespace: ${{ inputs.namespace }}
      secret-type: 'generic'
      secret-name: database-data
      string-data: '{"data": "Very secret data"}'

I get an error for this action
Run azure/k8s-create-secret@v4
Warning: Failed to delete secret with statusCode: undefined
Deleting secret:
undefined
Creating secret
***"errno":-2,"code":"ENOENT","syscall":"spawnSync kubelogin","path":"kubelogin","spawnargs":["get-token","--environment","AzurePublicCloud"

How to fix this?

@alxxyz can you attach the rest of the workflow? which action are you using to set your context before calling k8s-create-secret?

Hi @davidgamero, these are other actions
  - name: Authenticate to AKS
    uses: azure/login@v1
    with:
      creds: ${{ inputs.principal }}

  - name: Set AKS Context
    uses: azure/aks-set-context@v3
    with:
        cluster-name: ${{ inputs.cluster-name }}
        resource-group: ${{ inputs.resource-group }}
        subscription: ${{ inputs.subscription }}

are you using this as a non-admin user? you may need to add kubelogin
You can do this by adding use-kubelogin: 'true' to your aks-set-context "with" clause and adding this step before it:
- name: Set up kubelogin for non-interactive login
  uses: azure/use-kubelogin@v1
  with:
     kubelogin-version: 'v0.0.24'

It works, thank you!
I guess it will be good to add this info to documentation.

@alxxyz I'm glad it works for you!
this is already included in the documentation under Example > Kubelogin in the README


I cannot train the network with my own dataset

I am getting an error ::
when I run : experiments\SiamFC-3s-color-scratch.py
OutOfRangeError (see above for traceback): End of sequence
[[node train/IteratorGetNext (defined at experiments..\datasets\dataloader.py:102)  = IteratorGetNextoutput_shapes=[[?,127,127,3], [?,239,239,3]], output_types=[DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"]]

Please take a look of #40 and see if it helps.


Delta Stream overwrite possibility

I'm using spark stream to append data to delta table,but i need only the latest data(data of latest file recieved in each partition). As stream doesn't support overwrite, is there any work around? Can we keep only the latest files in each partition and vacuum rest.
Is there any way to use overwrite in spark stream?
Urgent help needed

Sorry for the delay. Spark streaming doesn't support overwrite mode explicitly. To be clear, you want to retrieve/save only the data from the latest committed file on a per-partition basis correct?
For overwriting on a per-partition basis, we are releasing support for dynamic partition overwrite in Delta 2.0 which allows you to selectively overwrite only the partitions with data being written into them. However, we do recommend using this cautiously and validating which partitions your data touches to avoid unintentional data loss.
You can then use ForeachBatch to perform the overwrite.


Adopt NEP 29

https://numpy.org/neps/nep-0029-deprecation_policy.html
This is a recommendation that  all projects across the Scientific Python ecosystem adopt a common “time window-based” policy for support of Python and NumPy versions.
This means, for example:
On next release, drop support for Python 3.5 (initially released on Sep 13, 2015)
On Jan 07, 2020 drop support for Numpy 1.14 (initially released on Jan 06, 2018)
On Jun 23, 2020 drop support for Python 3.6 (initially released on Dec 23, 2016)
On Jul 23, 2020 drop support for Numpy 1.15 (initially released on Jul 23, 2018)
On Jan 13, 2021 drop support for Numpy 1.16 (initially released on Jan 13, 2019)
On Jul 26, 2021 drop support for Numpy 1.17 (initially released on Jul 26, 2019)
What do you think? This could reduce the development burden by reducing the number of older versions we support. On the other hand, we would have to be more proactive about adding new versions to the CI test matrix.

On reflection, I agree that a 24 month window is rather short for scientific projects. On the other hand, it would be nice to have some clear guidelines, both for Neo developers and for downstream projects. NiBabel has gone for NEP 29 + 1 year (https://github.com/nipy/nibabel/issues/803)
Currently, Neo claims to support Python >= 3.5 and NumPy >= 1.10, however the CI tests only tes Python 3.5 and 3.6, and always install whatever NumPy pip gives, normally the most recent.
Adopting NEP 29 + 1 year would give:

On Jan 16, 2020 drop support for Numpy 1.12 (initially released on Jan 15, 2017)
On Mar 14, 2020 drop support for Python 3.5 (initially released on Sep 13, 2015)
On Jun 08, 2020 drop support for Numpy 1.13 (initially released on Jun 07, 2017)
On Jan 07, 2021 drop support for Numpy 1.14 (initially released on Jan 06, 2018)
On Jun 23, 2021 drop support for Python 3.6 (initially released on Dec 23, 2016)
On Jul 23, 2021 drop support for Numpy 1.15 (initially released on Jul 23, 2018)
On Jan 13, 2022 drop support for Numpy 1.16 (initially released on Jan 13, 2019)
On Jul 26, 2022 drop support for Numpy 1.17 (initially released on Jul 26, 2019)
On Dec 26, 2022 drop support for Python 3.7 (initially released on Jun 27, 2018)

This seems more reasonable. It means we should bump the minimum version of Python to 3.6, and of NumPy to 1.13. On the other hand, pypistats gives the usage for the last month as:

category
percent
downloads

3.7
49.19%
20,738

3.5
16.73%
7,054

3.6
14.97%
6,312

2.7
8.86%
3,737

3.8
8.68%
3,659

null
1.52%
640

3.3
0.02%
8

3.4
0.02%
8

Total

42,156

so one in six Neo downloads is still for Python 3.5.

Principle of NEP + 1 year accepted during meeting on 07/09/2020


Handle new fees types

Linear ticket: https://linear.app/swan/issue/FRONT-584/integrate-new-banking-fees
⚠ this PR depends on https://github.com/swan-io/lake/pull/98
This PR adds translations for new types of fees.
I also added InvalidTransferDate for transfer rejection reason.
And last detail: I removed exhaustive check for rejected transaction reason and fees description

ℹ Pipeline fails because of breaking change in QuickActions, once https://github.com/swan-io/swan-partner-frontend/pull/215 is merge, it should be good


Error due to dependency flutter chips input

after upgraded to flutter 3.I could not build app due to flutter_chips_input that flutter_form_builder_extras dependent on
`
Error: The non-abstract class 'ChipsInputState' is missing implementations for these members:
../…/src/chips_input.dart:87

TextInputClient.insertTextPlaceholder
TextInputClient.removeTextPlaceholder
TextInputClient.showToolbar
Try to either
provide an implementation,
inherit an implementation from a superclass or mixin,
mark the class as abstract, or
provide a 'noSuchMethod' implementation.

class ChipsInputState extends State<ChipsInput>
^^^^^^^^^^^^^^^
: Context: 'TextInputClient.insertTextPlaceholder' is defined here.
../…/services/text_input.dart:1124
void insertTextPlaceholder(Size size) {}
^^^^^^^^^^^^^^^^^^^^^`

same

same

fixed in version 8.0.0


Evaluate feasibility of deterministic indexical chunk mapping for state sync

Is there an efficient way we can map from integer indices to sub-key-spaces of a particular tree version, such that chunks would be approximately the same size or at least size-bounded, and individually provable (some metadata about top-level tree-structure can be transmitted up front)?
ref https://github.com/tendermint/tendermint/issues/828
cc @Liamsi @ebuchman

would you like see a new proposal here: https://github.com/tendermint/tendermint/issues/828#issuecomment-479859591

https://github.com/tendermint/tendermint/pull/3799#discussion_r303619323

Closing in favour of more up-to-date discussions.


Cannot connect to repo.mamba.pm server when creating a Quetz environment

When I try to create a quetz environment by running mamba env create -f environment.yml I get the following error:
RuntimeError: Download error (7) Couldn't connect to server [https://repo.mamba.pm/conda-forge/noarch/repodata.json]     Failed to connect to repo.mamba.pm port 443: Connection refused
https://repo.mamba.pm seems to be down. Is there a way to easily fix this?

yep, just change to conda-forge.

Yes that works. Thanks for the quick response!


Access to this.location: Property 'location' does not exist

The docs says that we can access to this.location to get the route parameters:
https://vaadin.github.io/vaadin-router/vaadin-router/#/classes/Router/demos/demo/index.html
And it works.
But when using TypeScript, I get an error:

Property 'location' does not exist

Looks like an interface or a mixin is missing that would make all router-added properties and callbacks known to TypeScript.
What do you think of a syntax like below?
class PageHome extends RouterView(LitElement) {
  ...
}

Makes sense!

Any news here? Thanks in advance! 😄


./config.sh: line 26: [{FILE}: No such file or directory

Hi,
I've found out that config.sh contains a bug while retrieving $REPLICATION_URL in order to update the database. Turns out, shell language was not respected on line 26 (spaces between [, ] and expressions). Starting the container with the pre-requisite parameters give you this warning / error message :
./config.sh: line 26: [https://download.geofabrik.de/europe/monaco-updates/: No such file or directory
Simple fix here.

The tests fail because the download of download.geofabrik.de/europe/monaco-220326.osm.pbf times out.
I will wait a bit and re-trigger later.

You can commit it to this PR.
But for me both http and https are timing out.

You can commit it to this PR.
I'm not sure how to modify this workflow file (new to this feature),and if I'm allowed to.

But for me both http and https are timing out.
https://download.geofabrik.de/europe/monaco-220326.osm.pbf times out for me too, maybe it is not a valid file on Geofabrik's server ? Where are you pulling this URI from ?

It's computed. We take a dump from a few days ago and then apply updates to it.

Looks like I've only edited line 19 but it seems to have the same issue on line 26. Shall create another commit + PR ?

It's computed. We take a dump from a few days ago and then apply updates to it.

Oh yes, I can see it now.

Restarting the tests a few times helped.
Thanks for the contribution!


Guide: how to run DDEV for a Drupal 7 subfolder setup

Writing a public summary of what we had to do to make DDEV work with a Drupal 7 site subfolder setup.
Subfolder
A Drupal site might be operated from a subfolder (subdirectory) of a domain. Example for such a URL:

No Subfolder frontpage: https://example.com
Subfolder frontpage: https://example.com/jobs

That means the Drupal site only lives in the /jobs part. A reverse proxy sits in front of the site and delivers requests either to the main page "/" or the Drupal site at "/jobs.
Simulating a subfolder setup in DDEV
In order to have a development environment that mimics the production site subfolder as much as possible we want to run DDEV also in a subfolder. We need to change a couple of things for that.
Nginx
Nginx needs 2 rewrite rules for the /jobs subfolder (replace /jobs with your subfolder name). Create .ddev/nginx-site.conf (see also https://ddev.readthedocs.io/en/stable/users/extend/customization-extendibility/#providing-custom-nginx-configuration )
# This file is overridden from DDEV because we need Nginx customizations for the
# /jobs subfolder setup.

# Set https to 'on' if x-forwarded-proto is https
map $http_x_forwarded_proto $fcgi_https {
    default off;
    https on;
}

server {
    listen 80; ## listen for ipv4; this line is default and implied

    # The WEBSERVER_DOCROOT variable is substituted with
    # its value when the container is started.
    root $WEBSERVER_DOCROOT;

    include /etc/nginx/monitoring.conf;

    # Rewrite the /jobs subfolder.
    # Special rule for the front page: we need to pass "/" explicitly here to
    # $_GET['q'] and PHP. This is not done by Nginx automatically and then
    # Drupal falls back to the full request URI which would include the
    # subfolder /jobs.
    rewrite ^/jobs/?$    /index.php?q=/&$args   last;
    rewrite ^/jobs/(.*)$   /$1   last;

    # The following part is copied and modified from
    # https://github.com/drud/ddev/blob/master/containers/ddev-webserver/files/etc/nginx/nginx_drupal7.conf
    # Replaced @rewrite location content with
    # rewrite ^ /index.php?q=$uri&$args;
    # MODIFIED COPY START

    index index.php index.htm index.html;

    # Make site accessible from http://localhost/
    server_name _;

    # Disable sendfile as per https://docs.vagrantup.com/v2/synced-folders/virtualbox.html
    sendfile off;
    error_log /dev/stdout info;
    access_log /var/log/nginx/access.log;

    location / {
        absolute_redirect off;

        # First attempt to serve request as file, then
        # as directory, then fall back to index.html
        try_files $uri $uri/ /index.php?q=$uri&$args;
    }

    location @rewrite {
        # For D7 and above:
        # Clean URLs are handled in drupal_environment_initialize().
        rewrite ^ /index.php?q=$uri&$args;
    }

    # Handle image styles for Drupal 7+
    location ~ ^/sites/.*/files/styles/ {
        try_files $uri @rewrite;
    }

    # pass the PHP scripts to FastCGI server listening on socket
    location ~ \.php$ {
        try_files $uri =404;
        fastcgi_split_path_info ^(.+\.php)(/.+)$;
        fastcgi_pass unix:/run/php-fpm.sock;
        fastcgi_buffers 16 16k;
        fastcgi_buffer_size 32k;
        fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
        fastcgi_param SCRIPT_NAME $fastcgi_script_name;
        fastcgi_index index.php;
        include fastcgi_params;
        fastcgi_intercept_errors off;
        # fastcgi_read_timeout should match max_execution_time in php.ini
        fastcgi_read_timeout 10m;
        fastcgi_param SERVER_NAME $host;
        fastcgi_param HTTPS $fcgi_https;
    }

    # Expire rules for static content
    # Feed
    location ~* \.(?:rss|atom)$ {
        expires 1h;
    }

    # Media: images, icons, video, audio, HTC
    location ~* \.(js|css|png|jpg|jpeg|gif|ico|svg)$ {
        try_files $uri @rewrite;
        expires max;
        log_not_found off;
    }

    # Prevent clients from accessing hidden files (starting with a dot)
    # This is particularly important if you store .htpasswd files in the site hierarchy
    # Access to `/.well-known/` is allowed.
    # https://www.mnot.net/blog/2010/04/07/well-known
    # https://tools.ietf.org/html/rfc5785
    location ~* /\.(?!well-known\/) {
        deny all;
    }

    # Prevent clients from accessing to backup/config/source files
    location ~* (?:\.(?:bak|conf|dist|fla|in[ci]|log|psd|sh|sql|sw[op])|~)$ {
        deny all;
    }

    ## Regular private file serving (i.e. handled by Drupal).
    location ^~ /system/files/ {
        ## For not signaling a 404 in the error log whenever the
        ## system/files directory is accessed add the line below.
        ## Note that the 404 is the intended behavior.
        log_not_found off;
        access_log off;
        expires 30d;
        try_files $uri @rewrite;
    }

    # MODIFIED COPY END

    include /mnt/ddev_config/nginx/*.conf;
}

server {
    listen 443 ssl;

    # The WEBSERVER_DOCROOT variable is substituted with
    # its value when the container is started.
    root $WEBSERVER_DOCROOT;

    ssl_certificate /etc/ssl/certs/master.crt;
    ssl_certificate_key /etc/ssl/certs/master.key;

    include /etc/nginx/monitoring.conf;

    # TODO It seems this server is unused even under HTTPS, why?

    include /etc/nginx/nginx_drupal7.conf;
    include /mnt/ddev_config/nginx/*.conf;
}

This could be improved to a simpler setup once we get #2267 committed.
Base URL
We want the site to be reachable at https://example.ddev.site/jobs . We need to set the base URL for Drupal so achieve that. Add a settings.php hook in .ddev/config.yaml:
hooks:
  pre-start:
  - exec-host: cp -f sites/default/settings.ddev-custom.php sites/default/settings.php

Create sites/default/settings.ddev-custom.php:
<?php

// Include ddev generated file.
$ddev_settings = dirname(__FILE__) . '/settings.ddev.php';
if (is_readable($ddev_settings)) {
  require $ddev_settings;
}

// This project is in a URL subfolder.
$subfolder = '/jobs';
// Behat tests run under HTTP, use that scheme if used in the request.
$http_scheme = 'http';
if (drupal_is_https() || PHP_SAPI === 'cli') {
  $http_scheme = 'https';
}
$base_url = "$http_scheme://" . getenv('VIRTUAL_HOST') . $subfolder;

// Return an empty page for the main site outside the subfolder. This helps
// detecting errors during subfolder development.
if (PHP_SAPI !== 'cli' && strpos(request_uri(), $subfolder) !== 0) {
  http_response_code(404);
  print "This is the main site, please go to the subfolder $base_url";
  exit;
}

That should be it. I will update this guide if we find a simpler solution or need to tweak other things.

Thanks, could you move this to a PR in https://github.com/drud/ddev-contrib?

Done! https://github.com/drud/ddev-contrib/pull/68


CSR extensionRequest support

It would be nice to provide some helper/shorthand to encode requested certificate extensions into a CSR

done in 2.0.0


Add support for nodejs16.x

https://aws.amazon.com/about-aws/whats-new/2022/05/aws-lambda-adds-support-node-js-16/

Related to #275

@alixaxel It would be awesome to get this one merged so that we are able to run on the latest runtime 🙏

We would be really grateful if this can be merged, it's blocking us from upgrading our infra on Lambda.

Please merge it, as it's crucial for to use it with Node.js v16.

We need this as well. It would be really great if @alixaxel could merge it. Thank you!

@alixaxel is this repo maintained? Any reason why so much delay with merging this update?

Works like a charm. I use patch-package with the following patch file as a workaround for now:
diff --git a/node_modules/chrome-aws-lambda/build/index.js b/node_modules/chrome-aws-lambda/build/index.js
index f78e183..599af55 100644
--- a/node_modules/chrome-aws-lambda/build/index.js
+++ b/node_modules/chrome-aws-lambda/build/index.js
@@ -7,7 +7,7 @@ const fs_1 = require("fs");
 const lambdafs_1 = __importDefault(require("lambdafs"));
 const path_1 = require("path");
 const url_1 = require("url");
-if (/^AWS_Lambda_nodejs(?:10|12|14)[.]x$/.test(process.env.AWS_EXECUTION_ENV) === true) {
+if (/^AWS_Lambda_nodejs(?:10|12|14|16|18)[.]x$/.test(process.env.AWS_EXECUTION_ENV) === true) {
     if (process.env.FONTCONFIG_PATH === undefined) {
         process.env.FONTCONFIG_PATH = '/tmp/aws';
     }
@@ -145,7 +145,7 @@ class Chromium {
             lambdafs_1.default.inflate(`${input}/chromium.br`),
             lambdafs_1.default.inflate(`${input}/swiftshader.tar.br`),
         ];
-        if (/^AWS_Lambda_nodejs(?:10|12|14)[.]x$/.test(process.env.AWS_EXECUTION_ENV) === true) {
+        if (/^AWS_Lambda_nodejs(?:10|12|14|16|18)[.]x$/.test(process.env.AWS_EXECUTION_ENV) === true) {
             promises.push(lambdafs_1.default.inflate(`${input}/aws.tar.br`));
         }
         return Promise.all(promises).then((result) => result.shift());

Save this file in your project as patches/chrome-aws-lambda+10.1.0.patch, install patch-package and add the following script to your package.json:
 "scripts": {
+  "postinstall": "patch-package"
 }

@Edweis are you going to maintain this project since I see you approved the PR? Can you merge it? I switched using https://github.com/Sparticuz/chrome-aws-lambda since @Sparticuz currently maintains it.

Works like a charm. I use patch-package with the following patch file as a workaround for now:
diff --git a/node_modules/chrome-aws-lambda/build/index.js b/node_modules/chrome-aws-lambda/build/index.js
index f78e183..599af55 100644
--- a/node_modules/chrome-aws-lambda/build/index.js
+++ b/node_modules/chrome-aws-lambda/build/index.js
@@ -7,7 +7,7 @@ const fs_1 = require("fs");
 const lambdafs_1 = __importDefault(require("lambdafs"));
 const path_1 = require("path");
 const url_1 = require("url");
-if (/^AWS_Lambda_nodejs(?:10|12|14)[.]x$/.test(process.env.AWS_EXECUTION_ENV) === true) {
+if (/^AWS_Lambda_nodejs(?:10|12|14|16|18)[.]x$/.test(process.env.AWS_EXECUTION_ENV) === true) {
     if (process.env.FONTCONFIG_PATH === undefined) {
         process.env.FONTCONFIG_PATH = '/tmp/aws';
     }
@@ -145,7 +145,7 @@ class Chromium {
             lambdafs_1.default.inflate(`${input}/chromium.br`),
             lambdafs_1.default.inflate(`${input}/swiftshader.tar.br`),
         ];
-        if (/^AWS_Lambda_nodejs(?:10|12|14)[.]x$/.test(process.env.AWS_EXECUTION_ENV) === true) {
+        if (/^AWS_Lambda_nodejs(?:10|12|14|16|18)[.]x$/.test(process.env.AWS_EXECUTION_ENV) === true) {
             promises.push(lambdafs_1.default.inflate(`${input}/aws.tar.br`));
         }
         return Promise.all(promises).then((result) => result.shift());

Save this file in your project as patches/chrome-aws-lambda+10.1.0.patch, install patch-package and add the following script to your package.json:
 "scripts": {
+  "postinstall": "patch-package"
 }

Hi @urish, looking to try this approach in the short term till this gets commited.  I get using patch-package, I'm a little confused on what constitutes "the following patch file" in your example - is it, literally, copying the code segment in your example to patches/chrome-aws-lambda+10.1.0.patch?
Thanks!

Would be great to merge this, is there anything blocking this PR? would be happy to help if there is

can we get this merged please?

@truuth-of-abhi I don't think the repository is maintained anymore, better to switch to this fork: https://github.com/Sparticuz/chrome-aws-lambda

The latest repository is https://www.npmjs.com/package/@sparticuz/chromium


Headers with multiple values should only be separated by a comma

Hi all
The PSR7-specification (see PR https://github.com/php-fig/fig-standards/pull/1194) as well as the RFC 2616 define a comma, not comma with white-space, as separator.
We should probably update the following method:
https://github.com/guzzle/psr7/blob/152fab1581e564743e14a85adb33c01e8773b254/src/MessageTrait.php#L65-L68

See https://github.com/Nyholm/psr7/issues/137 for further opinions and discussions.

Closing as per https://github.com/Nyholm/psr7/issues/137#issuecomment-629275400


some questions

Please tell me if this code only has the implementation of ecmp (there is only the call of ecmp in the p4 file), because after following the readme, what is drawn is what the ecmp looks like in the picture, which is not like qcmp.


Little improvements for right indexes in vocabulary dictionaries

Hi, @lyeoni !
You have written great tutorials. I really appreciate you)
We can improve a little bit with one  pretty line. Look, please)
Here, we fill first key-value items of stoi, itos by special tokens.
I suggest insert this line before cycle.
special_tokens = filter(lambda x: x is not None, [self.unk_token, self.bos_token, self.eos_token, self.pad_token])
If we don't set value for self.unk_token and set for self.bos_token, then index in dictionary become wrong. So, we need filter None values before.
Input
vocab = Vocab(body, bos_token='<bos>');                                                                               vocab.build(); vocab.stoi;
Wrong Output
'<bos>': 1  ' ': 1,  'hi': 2,  'bear': 3, ...

Hi @datason ! Sincerely thank you for giving me nice + kind comment :)
You're absolutely right, and i will fix the line that you mentioned (or if you leave pull request, I will merge). I think of that your comment makes this tutorial better.