text:
Move s3_dst config to top level config

Description of proposed changes
Nested configs for s3 URLs are challenging to override by the Snakemake --config option, as discussed in the following comment:
https://github.com/nextstrain/dengue/pull/13#discussion_r1361131210
This moves the url to the top level as a s3_dst config value, and propagate those changes to the relevant Snakefiles.
It's important to note that I haven't conducted thorough testing, and there may be nuances in the implementation meant by the comment. Suggestions or clarification welcome.
Additionally, I was unsure regarding whether the s3_dst config should be a required value or an optional default (Snakemake styleguide: config values).
Related issue(s)

https://github.com/nextstrain/dengue/pull/13#discussion_r1361131210

Checklist

[ ] Checks pass

Since we can use: --config upload='{"s3": {"dst": "NEW URL"}}'
https://github.com/nextstrain/dengue/pull/13#discussion_r1375081279
This PR is no longer necessary.

repo: nextstrain/monkeypox


text:
Updates to genbank_accession lines (5974 new, 1760 removed, seqs downloaded 2021-01-29)

Another bi-weekly update of genbank accessions matched by 1. sequences being identical (possibly except for runs of N's at beginning and/or end) and 2. similar country/isolate/year names.

removed 1760 for which sequences were either updated in GISAID but not GenBank or were withdrawn from GISAID
added 5794 from sequences downloaded 2021-01-29.

Thanks so much!  :)
Angie

Thank you as always for your tireless work on this @AngieHinrichs ! 👍

Thank you as always for your tireless work on this @AngieHinrichs ! 👍

@AngieHinrichs Thanks again for this curation work!

removed 1760 for which sequences were either updated in GISAID but not GenBank or were withdrawn from GISAID

Thoughts on ways to maintain some linkage here, even if the sequences are no longer identical?  For example, do you think it'd be reasonable in cases like this to revert to the non-versioned GenBank accession?

Good question @tsibley.  Since GISAID sequences aren't versioned, I think it would be best if the submitters of those sequences could be persuaded to update the sequences in GenBank too.  I will attempt to contact them.
In other cases, where GenBank was updated and not GISAID, the GenBank accession version numbers were incremented, and I have kept the mappings to older GenBank versions.  However, in this case, the submitters updated GISAID and not GenBank.  The sequences are no longer identical, and GISAID has a claim on whatever base differences are unique to the updated sequence in GISAID.
Version numbers can cause their own minor headaches, but for issues like this, and for reproduceability of analyses, it would be nice if GISAID versioned their sequences as GenBank.  I don't think that's going to happen, though.
@dmaccannell maintains https://github.com/CDCgov/SARS-CoV-2_Sequencing/tree/master/files/ with all versions of my mappings between EPI IDs and GenBank (and COG-UK sequences, which are separately available from COG-UK), so the removed mappings can still be found in older files -- e.g. the 1760 removed mappings are still in the file epiToPublicAndDate.2021-01-15.gz.

@AngieHinrichs Nod, that makes sense. Thanks for the explanation!
The pointer to the full mapping files are useful. We may want to use them as part of this repo in the future. :-)

source: gharchive/pull-request

authors: AngieHinrichs emmahodcroft tsibley

repo: nextstrain/ncov-ingest

license: MIT

license_type: permissive

license_source: github-api


text:
[BUG] - Tooltip wrapper element can not propagate the click event.

NextUI Version
2.1.13
Describe the bug
Tooltip wrapper element can not propagate the click event.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
https://codesandbox.io/s/gifted-maxwell-pvgg2w?file=/src/App.js
https://github.com/nextui-org/nextui/assets/41265413/bd1bfab0-ecbd-4349-b245-bdec363b3829
Expected behavior
click event should propagate
Screenshots or Videos
No response
Operating System Version
NA
Browser
Chrome

+1 I am having the same issue

source: gharchive/issue

authors: Innei cheestudio

repo: nextui-org/nextui


text:
[BUG] - Backdrop blur w/ ScrollShadow

NextUI Version
2.4.2
Describe the bug
When using backdrop-filter: blur(Xpx) on a div which is inside a ScrollShadow, the filter doesn't work.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

Create a ScrollShadow element
Put an element (div for simplicity) inside the ScrollShadow
Add backdrop-filter: blur(Xpx) to the div in css or backdrop-blur-{size} in the classnames if using tailwind

Expected behavior
Im expecting the filter to work (the background to be blurred)
Operating System Version
Linux - Ubuntu 22.04 LTS
Browser
Chrome

please try the latest version first
if it still fails, then please provide a reproducible example in sandbox

Here an example:
Example Sandbox
I noticed that the problem only occurs when there is an overflow.

@BCPTe  I have investigated your issue and and I think
mask-image css in overriding the backdrop-blur style.
Working on it, will solve soon :)

authors: BCPTe abhinandan-verma wingkwong


text:
WotLK Update

Updated:

[x] Professions
[x] Wowhead
[ ] Blizzard API

I believe the Wowhead scraping to be mostly adjusted now, at least the items so now it will be mainly waiting for the WotLK Blizzard API to come up

@Nakroma - Will the data/json/zones.json be updated later? I see you updated the scraping code but not the datafile itself. Humbly asking, thanks a lot for the data for classic and tbc classic!

@Nakroma - Will the data/json/zones.json be updated later? I see you updated the scraping code but not the datafile itself. Humbly asking, thanks a lot for the data for classic and tbc classic!

Hey, yeah I'll update the data as soon as all the scraping code is ready.

Hey @Nakroma, I believe the Blizzard API is now working for WoTLK following prepatch. The namespace static-classic-us returns WoTLK items now (using the namespace static-3.4.0_45264-classic-us under the hood).

    Change: 503 items missing.
    Change: 13700 items added.
    Change: 19684 items changed.

Looks good to me, did a few spot checks and it looks fine. Gonna go ahead and merge and release. Lemme know if anythings wrong!

authors: Andrioden Nakroma albri

repo: nexus-devs/wow-classic-items


text:
Fix #28. Missing default configuration for genome in standard profile

This will fix issue #28. But -profile standard,docker still required

Thanks @tiagochst! We need to be a little careful that this doesn't break the other config profiles below, as the order of config file import is important for igenomes.config. I realise now that I hit a similar problem on the RNA pipeline recently (see |this PR](https://github.com/nf-core/rnaseq/compare/master...ewels:config_refactor#diff-c79fe4336e72c04860afccd21f4ae1c5)). I haven't tested that enough yet, but a similar refactoring could work here too.
Phil

authors: ewels tiagochst

repo: nf-core/chipseq


text:
Re-enable MultiQC for NanoComp once MultiQC issue is solved

Description of feature
Related to https://github.com/ewels/MultiQC/issues/1630.
This MultiQC issue is currently in the milestone list for the next release. This scnanoseq issue is a reminder re-enable MultiQC on NanoComp outputs once it is available.

Nanocomp is now included in the MultiQC report (as of commit bed7b6ee7e9023357c9cb9d353033dbf14da6532)

repo: nf-core/scnanoseq


text:
Upgrade alpine image to 3.12.0

We're currently using Alpine 3.7.0, it's time time update.

DONE for v2.2.0

repo: nforgeio/neonKUBE


text:
Issue with const in strict mode

Issue

I'm a gentledev i:

[x] fully read the README recently
[x] searched for existing issues
[x] checked i'm up to date with the latest version of the project

Expected behavior
One of my npm dependency: gulp-ng-annotate throws an error, after bufferstreams was released to version 1.1.2. Issue:

Becasue of that I cannot deploy my application to server.
Thank you for all the help.

Looks like you are using $ node --version v0.10.48, This library is only tested against node 6+. You should try to revert it to a prior version but I strongly recommend you using a stable maintained NodeJS release.

Thank you for the quick answer.
I'll update my Node version on jenkins and chekc if that works.

Yes. Updating Node and npm solved the issue.
Thanks for your help.

authors: nfroidure thomas-kulka

repo: nfroidure/bufferstreams


text:
getComputedStyle is not defined in SSR for ng-primitives tooltip

Affected Primitive
Tooltip
Version
0.14.0
Bug Description
When running the development server with SSR enabled, the application crashes due to getComputedStyle not being defined in the server environment. This occurs in the NgpTooltip component's ngOnInit lifecycle hook.
Environment

Angular CLI: 19.0.2
Node.js: v20.18.0
ng-primitives: v0.14.0

Steps to Reproduce
Add the following to an element:
    [ngpTooltipTrigger]="template"
    [ngpTooltipTriggerPlacement]="'right'"
    [ngpTooltipTriggerOffset]="12"
    [ngpTooltipTriggerDisabled]="false"
    [ngpTooltipTriggerOpen]="true

Reproduction
No response

Thanks! Should be fixed in 0.15.

That was fast! Thank you!

authors: ashley-hunter benjaminforras

repo: ng-primitives/ng-primitives


text:
Date picker format on get

Yes when 2 way binding to a input text box. the date picker format works fine when choosing a date. My date format shows as follow.  03-03-2017. But when retrieving Data json value it does not reformat.
2017-03-02T00:00:00. Is there a way to force it to keep the format when requesting data.?

Duplicate to #143

authors: Sevensnake allenhwkim

license: mit

license_source: bigquery


text:
crontab, update & merge duplicate

hi, once again I thank you very much for your hard work, I really appreciate your dedication. but I still have a question, which is as below

how does set run in crontab so it runs fine
When the update option does the list merge the previous lists or delete the old one then update the new list
If it's a shame to use multiple lists according to what's intruded on in the readme, will it result in duplicate IPs & conditions. how to deal with this incident

sorry for my ignorance, hopefully in the future it will get better and keep going

how does set run in crontab so it runs fine

You first should execute the update-ip-blocklists.sh script once to create all ipsets for the blocklists you would like to use by supplying a -l "<Blocklist Name> <Blocklist URL>" argument for each blocklist. Example:
./update-ip-blocklists.sh -l "blocklist https://lists.blocklist.de/lists/all.txt" -l "spamhaus https://www.spamhaus.org/drop/drop.txt" -l "spamhaus6 https://www.spamhaus.org/drop/dropv6.txt"

You then add this command to the crontab of your system. How this is performed in detail depends on your particular system. Usually you can edit your crontab via crontab -e or by editing the file /etc/crontab. Otherwise Google will surely help with your specific system.
You need to execute the script daily. Example:
@daily /full/path/to/update-ip-blocklists.sh -l "blocklist https://lists.blocklist.de/lists/all.txt" -l "spamhaus https://www.spamhaus.org/drop/drop.txt" -l "spamhaus6 https://www.spamhaus.org/drop/dropv6.txt"

You need to replace the /full/path/to/update-ip-blocklists.sh with the path where you put the script on your machine.

When the update option does the list merge the previous lists or delete the old one then update the new list

As blocklists usually unblock IPs after a certain amount of time, these need to be cleared from our local lists as well. Therefore, lists will always be replaced.
The script handles this atomically by creating a new temporary ipset during the update cycle and replaces the existing ipset with the updated one. The old ipset is then deleted. This ensures, that the server is protected even during the update cycle :)

If it's a shame to use multiple lists according to what's intruded on in the readme, will it result in duplicate IPs & conditions. how to deal with this incident

If multiple lists block one IP address, all affected ipsets will include this IP. This is not problematic and directly handled by ipset and iptables. You don't need to worry about it. The IP will be blocked as long as it is listed on at least one active blocklist.

I wish you success in deploying the script on you machines :+1:

thx, appreciate . i will try to my machine.
once again I thank you very much for your hard work

authors: alsyundawy ngandrass


text:
update nginx template for TLS passthrough

Proposed changes
Problem: Nginx templates didn't support TLS passthrough inside stream block
Solution: I added nginx template for stream servers and also added a Layer4Server type to data plane
Testing: Tested the conversion from data plane type to nginx template
Please focus on (optional): If you any specific areas where you would like reviewers to focus their attention or provide
specific feedback, add them here.
Closes #ISSUE
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING doc
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked that all unit tests pass after adding my changes
[x] I have updated necessary documentation
[x] I have rebased my branch onto main
[x] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

I didn't add anything to data plane for upstreams yet, I was thinking of reusing the existing type BackendGroup and just adding an IsStream boolean

would it make sense to enable GitHub pipeline including linting for the branch feature/tls-passthrough ?

would it make sense to enable GitHub pipeline including linting for the branch feature/tls-passthrough ?

Yes, definitely, good catch.

authors: kate-osborn pleshakov sarthyparty

repo: nginxinc/nginx-gateway-fabric


text:
NGL 2.2.2 doesn't show bond

Hi
Please see this issue: https://github.com/nglviewer/nglview/issues/1008#issuecomment-1868855451
I am able to reproduce with NGL 2.2.2 too

File is here (need to change .txt to .cif): https://github.com/nglviewer/nglview/files/13765118/last_molecule_r_0-pos-1.txt

The core cif file parsing functions expects the field atom_site_type_symbol to properly infer the element. This cif file does not have it: it only contains the atom_site_label.
It seems that, in this case, we could default to infer element based on the later, in case atom_site_type_symbol is missing.

hi team,
I noted that the browser console warns with 'more than 500 atoms...', it's come from this code.
https://github.com/nglviewer/ngl/blob/master/src/structure/structure-utils.ts#L606-L607
it will work well for me when I change the check limit to 99999. can we have a bigger count limit or the count limit can be a configured parameter?

@jump2cn - That code is inferring bonds between atoms in the same residue, and the 500 is there to avoid an explosion in complexity - I guess it's kind of an arbitrary number but 99999 seems a little high - your file has ~1500 atoms? We could set it to 2500.
I was trying to find an example where it would behave more poorly - it's doing a nearest neighbour search using a kdtree then for each pair looking up to see if they would reasonably be bonded. In your case where there's no disorder that nearest neighbour search isn't going to be a problem even if every atom is queries, most will return only a few neighbours. For something like an NMR structure with 20 models of a large ligand all overlaid I guess it could get expensive, but I couldn't find an example quickly where it was a problem.

How about just make this threshold configurable and still use 500 as the default value, so that the user can decide the right value by theirselve.

Or can I specific bonds in the data file before feeding it to nglviewer, so that it's up to user to decide instead of having nglviewer to guess by distance?

repo: nglviewer/ngl


text:
fix: tailwind preset type declarations wiped out

This was a miss from #320, my b!
https://arethetypeswrong.github.io/?p=%40ngrok%2Fmantle%400.2.0

we don't want to use clean: true in tsup because it introduces a race condition :(
https://www.loom.com/share/554049cd8b484089b2bc6c989d2b287e

after publishing, types are bueno now
https://arethetypeswrong.github.io/?p=%40ngrok%2Fmantle%400.2.1

repo: ngrok-oss/mantle


text:
Conflicting changeValue

I have realized that it can happen that several V variables are changed at the same time, e.q. Vradar and Vfield because the new radar doesn't have the previous field. I'm not fully sure what would happen then, but I believe _update_plot() would be called twice, this would be wasted time to say the least.
I will present my thoughts bellow.

At first I thought to hide the alteration of field calling
self.Vfield.value = new_field

But that is really wrong, since other classes may be holding just the Vfield but not the Vradar, they how then not be informed of the change. I decided then to make two kinds of value change: a soft and a strong one and is responsibility of the Display to react to the right one. There are two ways of doing that:

Variable have two signal "ValueChanged" and "StrongValueChanged", normally both are emitted but in the case of a soft update just the first is emitted. Display on the other hand would just react to "StrongValueChange"

There is still just one signal, but V variable send and extra argument strong=True/False. Then display would do the following:
 def NewField(self, variable, value, strong):
     # do some thing
     # ...
     if strong:
        self._update_plot()

Comments?

I agree that calling python _update_plot() more than once is wasted time and/or resources.  I admit that I haven't checked the signals coming back into the Display instance.  I'm not sure I understand the difference between strong and not strong.
I just went back and looked at the code again.   In cases like say the python tilt.py, is it necessary to have Vradar called?  If you comment that section out, there is seemingly no affect on the Display.  Can this be safely removed then?
I assumed that the .change takes the change and updates the Variable instance in the Display.  Is this the case?  And if so, why not drop the Vradar altogether?
Finally, are we talking about two different things here?

Yes, we are probably talking about different things. So I will not comment your questions, since I'm not sure if I understood.
I'm talking about, suppose for safety menu.py around line 313 is updated to the following
radar = pyart.io.read(self.filename)
if self.Vfield.value not in radar.fields.keys():
    self.Vfield.change('reflectivity')
self.Vradar.change(radar)

this is just an example, menu don't have Vfield and this check should be done by Display, I believe. But any way, in this case, plot is update twice. I would like to do the following
radar = pyart.io.read(self.filename)
if self.Vfield.value not in radar.fields.keys():
    self.Vfield.change('reflectivity',strong=False)
self.Vradar.change(radar,strong=True)

in this case Display can choose not to update the plot when Vfield is changed.

closed by PR #22

authors: gamaanderson nguy

repo: nguy/artview


text:
🛑 Vnetwork is down

In 487d521, Vnetwork (https://vnetwork.vn) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Vnetwork is back up in 629d91d.


text:
How to add html in validation message

Currently we can only add string in validation messages.
I want to add <a> and other html tags in validation message.
How can i add custom html in validation message?

The validation is defined in formly-field wrapper, so the only solution is to define your own wrapper where you have more control.
You may check bootstrap example formly.dev/examples/bootstrap-specific/bootstrap-horizontal and to apply the layout globally use form-field as name:
    FormlyModule.forRoot({
      wrappers: [{
-        name: 'form-field-horizontal',
+        name: 'form-field',
         component: FormlyHorizontalWrapper
      }],
    }),

authors: aitboudad lohiaad

repo: ngx-formly/ngx-formly


text:
🛑 Core is down

In 69f4f4c, Core ($CORE_URL) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Core is back up in 9111408 after 6 minutes.

repo: nhs-work/test-upptime


text:
Python 3.8 is not working

Running the examples with python 3.8 fails.

#26 Fixes this issue.

repo: ni/niveristand-python


text:
🛑 Authinator is down

In a9711b9, Authinator (https://www.authinator.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Authinator is back up in d26ee13.

repo: nickadam/status


text:
20220703 Update

將資料更新為 20220703 版本。
Python 腳本更新為 Python 3。
糾正了 Python 腳本當中的源檔案網址為最新的 Mediawiki 資源網址（該網址曾被 2020 年 8 月之後的某個 commit 變更）。

Thank you for your PR! 👍

authors: ShikiSuen nickcheng

repo: nickcheng/NCChineseConverter


text:
Crashes on Load under VS 2015

Using release https://github.com/nickgravgaard/AlwaysAlignedVS/releases/tag/2017.0.0
I assume that VisualStudio 2015 isn't supported, but it's not mentioned in the docs.
This extension works like a charm on VS2017, but under 2015 it shows this error on start-up:
CreateInstance failed for package [AlwaysAlignedPackage]Source: 'mscorlib' Description: Could not load file or assembly 'Microsoft.VisualStudio.Shell.15.0, Version=15.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.
System.IO.FileNotFoundException: Could not load file or assembly 'Microsoft.VisualStudio.Shell.15.0, Version=15.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified
File name: 'Microsoft.VisualStudio.Shell.15.0, Version=15.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'
   at System.Reflection.RuntimeAssembly.GetType(RuntimeAssembly assembly, String name, Boolean throwOnError, Boolean ignoreCase, ObjectHandleOnStack type)
   at System.Reflection.RuntimeAssembly.GetType(String name, Boolean throwOnError, Boolean ignoreCase)
   at System.Activator.CreateInstanceFromInternal(String assemblyFile, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, Evidence securityInfo)
   at System.AppDomain.CreateInstanceFrom(String assemblyFile, String typeName)

I get exactly the same issue, and the same message shows up in my ActivityLog.xml.

According to the author, this release version was build for VS2017.

Well that would explain it. Maybe the extension should not be installable under VS2015.

Same here. Could be explained please.

Is there a way to convert it to VS2015?

authors: DanTalash EtaoinWu R2D221 i-e-b runkelz

repo: nickgravgaard/AlwaysAlignedVS


text:
Server startup failure

I have been testing the ldap_auth.py script and can't see a way of specifying the user in the run command. Hence when I try to run the container it appears to stall.
I tested the script itself (i.e. not in the context of the container) and it does work. However, in that case I use a positional argument:
./ldap_auth.py <username>

Actually I see the same results when using rstudio_auth:
singularity run /apps/software/containers/rstudio-server-3.6.2.sif --auth-none 0 --auth-pam-helper-path rstudio_auth --www-port 28787

It just hangs there, not asking for a username or a password.

Ignore this, it was working all along...

repo: nickjer/singularity-rstudio


text:
Adapted for multiple certs

The current one doesn't take into account if you have multiple certs. The cronjob gets overwritten every time your run the role with a new cert. Now one job is created for each cert. I also randomized the starting minute, so we don't hit a rate-limiter for some reason....

Isn't this still going to create 1 cronjob total? It's just picking off the first domain name from the list instead of having a hard coded file name.

I have my playbook like this:
- role: nginx-letsencrypt
      letsencrypt_domains: [example1.org, www.example1.org]
    - role: nginx-letsencrypt
      letsencrypt_domains: [example2.se, www.example2.se]
    - role: nginx-letsencrypt
      letsencrypt_domains: [rss.example3.org]

That creates three different cronjobs
#Ansible: renew SSL certificates for example1.org
24 0 1 * * /usr/local/acme-tiny/example1.org-renew_certificate.sh
#Ansible: renew SSL certificates for example2.se
12 0 1 * * /usr/local/acme-tiny/example2.se-renew_certificate.sh
#Ansible: renew SSL certificates for rss.example3.org
44 0 1 * * /usr/local/acme-tiny/rss.example3.org-renew_certificate.sh

I see.
That's not really adapting it for multiple certificates. That's running the role more than once with a unique file name.
I thought you were going to internally create a loop in this role so this role only needs to be ran once.

Welp, this is what I've got :)

The cron job ovewrite also happends when this script is used to install certificate in multiple installations and this patch fixes it, would be relley nice to have it merged, can I do something to move forward?

authors: ScuttleSE nickjj pinkavaj

repo: nickjj/ansible-letsencrypt


text:
feat: add send whisper endpoint

implement new send whisper endpoint
add unit test
https://dev.twitch.tv/docs/api/reference#send-whisper

Pull Request Test Coverage Report for Build 3637426802

9 of 9 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 92.964%

Totals

Change from base Build 3595662938:
0.04%

Covered Lines:
1361

Relevant Lines:
1464

💛  - Coveralls


text:
Advanced Mode

An advanced mode UI should be added that works as follows

The preview starts as plain 100% intensity
There are the basic fields (intensity, resolution, etc)
User can add "modifiers" that effect a given range of angles
There should be a stack of modifiers, that are applied in order
They should be modular, so that more modifiers can be added

There main modifiers are for example

Reduce or increase intensity
Add noise or smooth

done, chose to do a whole new interface instead of just another mode

repo: nickmcdonald/ies-generator


text:
Add flood (1.0-rc1) formula

Here’s a formula for a new small tool. Add it if you like it, otherwise no harm done.
Project: https://github.com/sjmulder/flood
Notes:

I’ve tagged a rc1 version just in case something comes up.
Not sure if having prebuilt binaries is worth it. It takes only an instant to build.
The Makefile’s install target seems to work well, given the right prefix.

Thanks for merging!
I've just tagged 1.0 so if you were planning on building binaries that might be a good time to switch tags too.

repo: nickolasburr/homebrew-pfa


text:
App not showing right on Ubuntu 22

Is this app looking right or broken?
Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Ubuntu 22.04.1 LTS
Browser: Chrome
Version 106

Same with latest Firefox.

@tamalchowdhury, it's a work in progress. That's how it currently looks. This was it initial pairing and debugging for the beginning of the app. It's not fully functional yet. #49 and #52 are open to fix up the UI.
This makes me think we should mention it's a work in progress in the ReadME.

Okay, then it's cool. I am going to drop some code to help it get working.

Closing the issue

authors: nickytonline tamalchowdhury

repo: nickytonline/connect-four


text:
adding color parameter to heatmap function

I was asked by a user to render a heatmap table using green instead of red! Because "red" is hard-coded, I was forced to create a new version of the heatmap function.

Thanks for the PR! I will likely expand it a bit before merging so that it uses the rendererOptions mechanism.

You can now specify a custom color scale function, see the example here: http://nicolas.kruchten.com/pivottable/examples/montreal_2014.html

authors: nicolaskruchten pkpapani

repo: nicolaskruchten/pivottable


text:
pandas.read_hdf

Hi Niels,
Many thanks for this image which is very useful.
pandas.read_hdf requires the tables module.  Would you be able to add
pip install --upgrade tables
to your docker file?
(I have raised this with jupyter as sorting it in the earliest image that imports pandas probably makes most sense.  It might be that they prefer not to add it and this extra module fits quite well with the long list of optional extras at the end of your docker file...
Also, your image does not seem to have a tag (other than latest).  Are you able to give it a tag - so that if you change it we know which image we are pulling?

Hello Alan,
Sorry for the late answer! I just had merged the pull request for the upgrade of "tables".
I tested it locally with this script and it worked fine:
import numpy as np
import pandas as pd

df = pd.DataFrame({
    'A': [1, 2, 3, 4, 5, 1, 2, 3, 4, 5, 1, 2],
    'B': [0, 0, 1, 1, 1, 0, 1, 0, 1, 0, 0, 1],
    'C': [34, 32, 35, 34, 31, 34, 29, 34, 12, 34, 32, 34],
    'D': [11, 15, 22, 15, 9, 15, 11, 15, 14, 15, 13, 15]})

df.to_hdf('/tmp/out.h5', 'results_table', mode='w', data_columns=['A'],
          format='table')

print(pd.read_hdf('/tmp/out.h5', 'results_table', where='A in [1,3,4]'))

Can you pull the latest image, try this script on your side and validate that this is the behavior you expected?

About your docker tag request :

I totally agree, it's a good idea to start tagging future versions of the image! ( Until now I hadn't felt the need because... I was the only one using this image ^^ )
As a result, I just pushed the tag v.0.0 which corresponds to the version before the upgrade of "tables" (your correponded issue), and tag this new version as latest.
So far, I've done it manually. But I think later this week I will push new things to this repository to add a minimal form of CI (Makefile, JenkinsFile ... )!

Besides, I noticed that you cloned this repo and made your own modifications: if you think there are features that could be useful, don't hesitate to fork and propose your own pull request ! ;)
Cheers

Hi Niels,
Thanks.
I’m teaching some students at a University and I had to get an image to them by start of this week - sorry! - otherwise I would have waited for your response.  At the time (last week) I was brand new to docker and I had no idea what to do and did not want to mess up anything in yours.
In my modifications, mainly I deleted quite a few of the extra packages that you load at the end - just to make the code that creates the image easier for me to read.  I also updated some packages (because the jupyter notebook on which it is based has an old version of one of them (pandas I think or sk-learn).  Finally I installed statsmods from master rather than pip because statsmods on pip does not run with the latest version of scripy (it is a known bug).  And actually I could not get tables to work!!  Each time I added it, the build failed!
When I get a chance I will add any useful mods I did to your images and leave it as a pull request.
Very many thanks for your everything.
Regards
Alan

On 4 Jun 2019, at 23:30, Niels Borie notifications@github.com wrote:
Hello Alan,
Sorry for the late answer! I just had merged the pull request for the upgrade of "tables".
I tested it locally with this script and it worked fine:
import numpy as np
import pandas as pd
df = pd.DataFrame({
'A': [1, 2, 3, 4, 5, 1, 2, 3, 4, 5, 1, 2],
'B': [0, 0, 1, 1, 1, 0, 1, 0, 1, 0, 0, 1],
'C': [34, 32, 35, 34, 31, 34, 29, 34, 12, 34, 32, 34],
'D': [11, 15, 22, 15, 9, 15, 11, 15, 14, 15, 13, 15]})
df.to_hdf('/tmp/out.h5', 'results_table', mode='w', data_columns=['A'],
format='table')
print(pd.read_hdf('/tmp/out.h5', 'results_table', where='A in [1,3,4]'))
Can you pull the latest image, try this script on your side and validate that this is the behavior you expected?
About your docker tag request :
I totally agree, it's a good idea to start tagging future versions of the image! ( Until now I hadn't felt the need because... I was the only one using this image ^^ )
As a result, I just pushed the tag v.0.0 which corresponds to the version before the upgrade of "tables" (your correponded issue), and tag this new version as latest.
So far, I've done it manually. But I think later this week I will push new things to this repository to add a minimal form of CI (Makefile, JenkinsFile ... )!
Besides, I noticed that you cloned this repo and made your own modifications: if you think there are features that could be useful, don't hesitate to fork and propose your own pull request ! ;)
Cheers
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/nielsborie/ml-docker/issues/1?email_source=notifications&email_token=ACQ7P32H3KZLGXCL6VAGMILPY3UH7A5CNFSM4HOQ7G62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW6BYSI#issuecomment-498867273, or mute the thread https://github.com/notifications/unsubscribe-auth/ACQ7P36VKXQAEXQNLCAL2ZLPY3UH7ANCNFSM4HOQ7G6Q.

Hi Alan,
You're welcome, what I did is not a big deal!
I'm going to close this issue, so maybe see soon during a possible pull request!
Regards
Niels

authors: alanchalk nielsborie

repo: nielsborie/ml-docker


text:
Problem bei der Installation

Hallo,
ich habe versucht das Modul mittels pip zu installieren (pip install pyfronius). Leider erhalte ich eine Fehlermeldung:
Could not install packages due to an EnvironmentError: [Errno 22] Invalid argument: 'C:\\Users\\xxxx\\AppData\\Local\\Temp\\pip-req-build-qa47je68\\pyfronius/tests/test_structu
re/solar_api/v1/GetInverterRealtimeData.cgi?Scope=Device&DeviceId=1&DataCollection=CommonInverterData'

Ich habe Python 3.7.4 und versuche die Installation auf Windows 10.
Vielleicht habt ihr ja eine Idee.

Hallo, es sieht so aus als wären die Namen der Dateien nicht mit den Windows Datei-Namen-Konventionen vereinbar. Das ist auf die schnelle nicht so einfach zu beheben... Aber wir können es mal für die Nachwelt stehen lassen.
ENGLISH VERSION:
It seems that there is a problem installing this package on windows because some of the filenames violate windows file naming conventions. This is not easily fixable but everyone interested is welcome to fix it.

repo: nielstron/pyfronius


text:
HtmlToFarHelp: увеличить интервал перед заголовками

Сейчас что до что после заголовка интервал одинаков - 1 строка.
Как правило, я каждый раз редактирую потом hlf вручную, чтобы между текстом, и последующим заголовком было две строки.
Возможно стоит сделать это настраиваемым.

Наверное, мы можем добавить пару новых настроек в опции
https://github.com/nightroman/FarNet/blob/main/HelpDown/HtmlToFarHelp/README.md#conversion-options
<!--HLF:
    LinesBeforeTopic = 1;
    LinesAfterTopic = 1;
-->

А можно пример того, что производится и что вы делаете потом? А то, может, я иначе понимаю запрос.

Полноценные топики (@Contents) особого смысла тонко настраивать нет, поскольку на отображение это никак не влияет.
Вместо этого предлагаю их разделять двумя строками безусловно, чтобы в редакторе было нагляднее где начинается новый топик.

Обычные заголовки, не образующие топиков, я лично хочу разделять так же. Сделать ли это настраиваемым - решать вам.

Пример такой.

Конвертер выдаёт сейчас так:
 GitHub renders this file as the usual Markdown and this is very good.
See also its raw Markdown for useful details including HTML comments.

 #Features#

   • §¦~Headings~@headings@
   • §¦~Emphasis~@emphasis@

К этому я добавляю ещё одну пустую строку перед заголовком, и получаю желаемое:
 GitHub renders this file as the usual Markdown and this is very good.
See also its raw Markdown for useful details including HTML comments.

 #Features#

   • §¦~Headings~@headings@
   • §¦~Emphasis~@emphasis@

<!--HLF:
    EmptyLinesAfterHeader = 1;
    EmptyLinesBeforeHeader = 2;
-->

Кстати есть и нюанс: название EmptyLinesBeforeHeader не совсем отражает желаемое поведение:
Дополнительный интервал не нужен в самом начале топика, если перед заголовком нет никакого текста.
Аналогично и в ситуации, где заголовок (например H2) непосредственно следует за другим (H1), т.е. между ними нет никакого другого текста.

Если вы решите реализовать детальную настройку, то хотелось иметь и другие возможности:

<!--HLF:
    EmptyLinesAfterList = 0;
    EmptyLinesBeforeList = 0;
    EmptyLinesBetweenParagraphs = 0;
-->

Все-таки примеры того, что вы предлагаете, нужны. В HLF, что сейчас, и что нужно, и какой настройкой. Ибо у вас одна терминология, у меня другая. И несколько лет я не трогал этот тул, все меня устраивает. Потому детали могу не помнить.
Не хочется мне по умолчанию делать 2 пустые строки. Никто не жалуется (вы первый), и мне больше с 1-й нравится. Но если многие поддержат 2, почему бы и нет.

Основной пример с заголовками - в моём сообщении под кнопкой Details.
Другие примеры тоже подготовлю, не проблема.

Кнопку я не заметил :)

v1.3.8 New options EmptyLinesBeforeTopic, EmptyLinesAfterHeading, EmptyLinesBeforeHeading
Пробуйте.

<!--HLF:
    EmptyLinesAfterHeading = 0;
    EmptyLinesBeforeHeading = 2;
-->
# Header 1

## Header 2

Text

Даёт
@Contents
$ #Header 1#

 #Header 2#

 Text

EmptyLinesAfterHeading = 0 не сработало
EmptyLinesBeforeHeading = 2; работает, но в моём примере граничный случай, когда заголовки идут подряд, причём первый это заголовок топика, который ещё и чертой отделяется:

В данном случае и одной пустой строки более чем достаточно. При том горизонтальная черта тоже разделитель.
В некоторых случаях я вообще убираю лишний интервал:
$ #Header 1#
 #Header 2#

Другие желательные настройки:
<!--HLF:
    EmptyLinesBetweenParagraphs = 0;
    EmptyLinesAfterList = 0;
    EmptyLinesBeforeList = 0;
-->
# Header 1

Line 1

Line 2

- item 1
- item 2

Line 3

Хотелось бы получить в результате такой компактный текст:
 Line 1
 Line 2
   • §¦item 1
   • §¦item 2
 Line 3

До кучи ещё несколько "шашечек", о которых я упоминал (совсем давно) на форуме.
Для списков:

опционально: пустая строка между элементами списка
возможность подсветки нумерации/буллета
#1.# ...
#•# ...
возможность указать другой символ буллета (например, для списка второго уровня)
возможность указать другой отступ слева (например 0)

Для definition lists:

возможность "табличного" рендеринга (с указанием в какой позиции начинается второй столбец)

EmptyLinesAfterHeading = 0 не сработало

Да, нули не поддерживаются. И пока не будут, больше переделок требуется для этого, а времени нет.

EmptyLinesBeforeHeading = 2; работает, но в моём примере граничный случай, когда заголовки идут подряд, причём первый это заголовок топика, который ещё и чертой отделяется:

Над этим подумаю. Над остальными шашечками тоже, но со временем пока туго.

Вообще, есть план переделать этот тул радикально. Он создавался во премена еще хилых процессоров и задуман/реализован был для максимальной производительности. В частности, чтобы прямо из редактора маркдауна запускать просмотр текущего топика в виде хелпа по F1, много раз по мере редактирования. Потому вся конвертация делается в одит проход, и это делает код не очень простым и пригодным для создания новых шашечек.

v1.3.9 Ignore EmptyLinesBeforeHeading for headings after headings.
Если стало получше, то это пока все с хедерами. Остальные пожелания принял к сведению.

Да, почти забыл. Дефолтные значения выбирались даже не согласно моим вкусам (хотя это трудно избежать), а так, чтобы HLF рендерирг в Фаре был как-то подобен HTML ренденингу в броузере. В первую очередь для сгенерированных HTML, конечно, без стилей или с минимальными стилями (как pandoc генерит. а не руками созданные страницы).

v1.3.10
New options HighlightListBullet, HighlightListNumber

v1.3.11
New option ListBullet

Дефолтные значения выбирались даже не согласно моим вкусам (хотя это трудно избежать), а так, чтобы HLF рендерирг в Фаре был как-то подобен HTML ренденингу в броузере.

Насколько я могу судить то в html, который выдаёт pandoc, интервал до и после заголовка заметно отличаются.
margin-block-end 19.92px
margin-block-start 33.6px

New options HighlightListBullet, HighlightListNumber

Спасибо!

New option ListBullet

Предлагаю дать возможность задавать разный символы для разных уровней.
  ListBullet *+-

Насколько я могу судить то в html, который выдаёт pandoc, интервал до и после заголовка заметно отличаются.

А если убрать стили? Предыдущий конвертер и вначале пандок не добавляли ничего от себя.

Предлагаю дать возможность задавать разный символы для разных уровней.

Да, планирую.
Но надо не так, а с разделителем, ибо любая строка может быть использована как буллет. Запятая сойдет?

Например, ListBullet = ⚠ ☕ даст такой результат:

v1.3.12
ListBullet supports one or more strings

А если убрать стили?

Тогда всё ровно.

Вообще, есть план переделать этот тул радикально.

Я как-то хотел это попробовать https://pandoc.org/custom-writers.html

v1.3.13
New option ListItemEmptyLine

На этом пока думаю закрыть тикет.

repo: nightroman/FarNet


text:
onEnter not called on client

From what I understood from the readme, onEnter should be isomorphic, but it doesn't seem to be called from the client. RRv4 doesn't call onEnter like v2.
Also, on the server side it seems that preload.js executes all of the onEnter methods.

I fixed the SSR part, however for the client side it's pretty difficult at the moment since the history object is not exposed by RRv4. But they are planning to, so as soon as they expose it, I will update the code.
In the meantime, you can call your static methods inside componentDidMount
class MyComponent extends Component {
  onEnter() {
    //your code here
  }
  componentDidMount() {
    MyComponent.onEnter();
  }
}

Ah, I see. The history object is currently available through context. v4 has a withRouter PR ready that would expose it for components, so I understand you might want to wait until it's merged.
btw. did you push the SSR fix?

I wanted to make withRouter work with what I have before pushing but looks it'll take longer than expected. I'll push what I have in a few hours.

Pushed the fix

I've only just checked it over the weekend and seems working fine, thanks!
Just a notice, there's still /core/helpers/onEnter which doesn't seem to be used anywhere.

@omaksi Thanks, just deleted that file 👍

Is this on the master branch? It doesn't seem to be called.

authors: alexmipego nightwolfz omaksi

repo: nightwolfz/mobx-starter


text:
Folder processing outputs black images

When I use it with folders as an input, like
cain-ncnn-vulkan.exe -v -i in -o out
only the first batch works fine. With default settings it's 2 images, with "-j 1:4:1" it's 4. After that every next output is a black image.
Example with 4 processing threads.

Any ideas why this might be happening?

Checked on both Windows and Linux

I got the same problem too.

I got the same problem too.

"Fixed" by switching to RIFE. It's just better
https://github.com/hzwer/arXiv2020-RIFE
https://github.com/nihui/rife-ncnn-vulkan

"Fixed" by switching to RIFE. It's just better
https://github.com/hzwer/arXiv2020-RIFE
https://github.com/nihui/rife-ncnn-vulkan

nihui, you can also reproduce it perfectly on google colab. so you might have a way to find out whats wrong :)

"Fixed" by switching to RIFE. It's just better
https://github.com/hzwer/arXiv2020-RIFE
https://github.com/nihui/rife-ncnn-vulkan

thanks a lot, I will try this.

RIFE is not in general better.
I don't understand why CAIN is broken now, this can't be the solution for this project.

This implementation is broken.
I guess we still have the option of using the original Pytorch code.

original is missing the model file, so harder to come by a working version

Have also done more testing. And it seems that some files work and some do not, without explanation or error. If the author can not reproduce, it may mean they need to test more files. I find it happens more on GIFs. A sample set of 5 had 2 work and the rest fail with a better success rate if converted to WEBP.

I solved this issue by trying different resolutions - divide able by 128.
And it needs a monitor on each gpu you want to run it.
it solved it for me on OSX and windows.

Happens in Chainner too, Windows and Linux on the same hardware. It's something with NCNN.
Sometimes it works fine for minute but then will start outputting all black images. Bummer cause it's a little faster on some stuff for me compared to Pytorch or ONNX with the same models. Quite a bit faster on Linux Mint with Intel Core i9 and integrated graphics.
ONNX is close on Linux and Pytorch just blows and is usually twice as slow for me, which may be due to the CPU usage showing lower but that could also be more efficient code and the issue is with something else.
Pytorch is the slowest on Windows as well in my case but not as much.
My system details
System:
  Kernel: 5.15.0-107-generic x86_64 bits: 64 compiler: gcc v: 11.4.0 Desktop: Cinnamon 5.8.4
    tk: GTK 3.24.33 wm: muffin dm: LightDM Distro: Linux Mint 21.2 Victoria base: Ubuntu 22.04 jammy
Machine:
  Type: Laptop System: ASUSTeK product: Zenbook UX5401ZAS_UX5401ZAS v: 1.0
    serial: <superuser required>
  Mobo: ASUSTeK model: UX5401ZAS v: 1.0 serial: <superuser required>
    UEFI: American Megatrends LLC. v: UX5401ZAS.311 date: 10/25/2023
Battery:
  ID-1: BAT0 charge: 52.7 Wh (99.2%) condition: 53.1/63.0 Wh (84.3%) volts: 13.0 min: 12.0
    model: ASUSTeK ASUS Battery serial: N/A status: N/A
  Device-1: hid-0018:04F3:4065.0001-battery model: ELAN9008:00 04F3:4065 serial: N/A charge: N/A
    status: N/A
CPU:
  Info: 14-core (6-mt/8-st) model: 12th Gen Intel Core i9-12900H bits: 64 type: MST AMCP
    arch: Alder Lake rev: 3 cache: L1: 1.2 MiB L2: 11.5 MiB L3: 24 MiB
  Speed (MHz): avg: 887 high: 1000 min/max: 400/4900:5000:3800 cores: 1: 898 2: 899 3: 1000
    4: 1000 5: 998 6: 1000 7: 995 8: 1000 9: 997 10: 998 11: 798 12: 873 13: 757 14: 793 15: 790
    16: 792 17: 793 18: 788 19: 794 20: 793 bogomips: 116736
  Flags: avx avx2 ht lm nx pae sse sse2 sse3 sse4_1 sse4_2 ssse3 vmx
Graphics:
  Device-1: Intel Alder Lake-P Integrated Graphics vendor: ASUSTeK driver: i915 v: kernel ports:
    active: eDP-1 empty: DP-1,DP-2,HDMI-A-1 bus-ID: 0000:00:02.0 chip-ID: 8086:46a6
  Device-2: IMC Networks USB2.0 HD UVC WebCam type: USB driver: uvcvideo bus-ID: 3-9:6
    chip-ID: 13d3:5460
  Display: x11 server: X.Org v: 1.21.1.4 driver: X: loaded: modesetting unloaded: fbdev,vesa
    gpu: i915 display-ID: :0 screens: 1
  Screen-1: 0 s-res: 4608x2880 s-dpi: 120
  Monitor-1: eDP-1 model: Samsung res: 4608x2880 dpi: 388 diag: 356mm (14")
  OpenGL: renderer: Mesa Intel Graphics (ADL GT2) v: 4.6 Mesa 23.2.1-1ubuntu3.1~22.04.2
    direct render: Yes
Audio:
  Device-1: Intel Alder Lake PCH-P High Definition Audio vendor: ASUSTeK
    driver: sof-audio-pci-intel-tgl bus-ID: 0000:00:1f.3 chip-ID: 8086:51c8
  Sound Server-1: ALSA v: k5.15.0-107-generic running: yes
  Sound Server-2: PulseAudio v: 15.99.1 running: yes
  Sound Server-3: PipeWire v: 0.3.48 running: yes
Network:
  Device-1: Intel Alder Lake-P PCH CNVi WiFi driver: iwlwifi v: kernel bus-ID: 0000:00:14.3
    chip-ID: 8086:51f0
  IF: wlo1 state: up mac: <filter>
Bluetooth:
  Device-1: Intel type: USB driver: btusb v: 0.8 bus-ID: 3-10:7 chip-ID: 8087:0033
  Report: hciconfig ID: hci0 rfk-id: 2 state: up address: <filter>
RAID:
  Hardware-1: Intel Volume Management Device NVMe RAID Controller driver: vmd v: 0.6
    bus-ID: 0000:00:0e.0 chip-ID: 8086:467f
Drives:
  Local Storage: total: 5.74 TiB used: 655.05 GiB (11.1%)
  ID-1: /dev/nvme0n1 vendor: Crucial model: CT2000T500SSD8 size: 1.82 TiB speed: 63.2 Gb/s
    lanes: 4 serial: <filter> temp: 37.9 C
  ID-2: /dev/sda type: USB vendor: Seagate model: ST3000DM 001-9YN166 size: 2.73 TiB
    serial: <filter>
  ID-3: /dev/sdb type: USB vendor: JMicron Tech model: Generic size: 238.47 GiB serial: <filter>
  ID-4: /dev/sdc type: USB model: DockCase DSWC1P size: 953.87 GiB serial: <filter>
  ID-5: /dev/sde type: USB vendor: Generic model: MassStorageClass size: 28.86 GiB
    serial: <filter>
Partition:
  ID-1: / size: 233.18 GiB used: 84.85 GiB (36.4%) fs: ext4 dev: /dev/sdb2
  ID-2: /boot/efi size: 511 MiB used: 6.1 MiB (1.2%) fs: vfat dev: /dev/sdb1
Swap:
  ID-1: swap-1 type: file size: 2 GiB used: 1.1 GiB (55.1%) priority: -2 file: /swapfile
Sensors:
  System Temperatures: cpu: 55.0 C mobo: N/A
  Fan Speeds (RPM): cpu: 3400
Repos:
  Packages: 3780 apt: 3759 flatpak: 21
  No active apt repos in: /etc/apt/sources.list
  Active apt repos in: /etc/apt/sources.list.d/lunarg-vulkan-1.3.283-focal.list
    1: deb [signed-by=/usr/share/keyrings/lunarg-archive-keyring.gpg] https: //packages.lunarg.com/vulkan/1.3.283 focal main
  Active apt repos in: /etc/apt/sources.list.d/official-package-repositories.list
    1: deb http: //packages.linuxmint.com victoria main upstream import backport
    2: deb http: //archive.ubuntu.com/ubuntu jammy main restricted universe multiverse
    3: deb http: //archive.ubuntu.com/ubuntu jammy-updates main restricted universe multiverse
    4: deb http: //archive.ubuntu.com/ubuntu jammy-backports main restricted universe multiverse
    5: deb http: //security.ubuntu.com/ubuntu/ jammy-security main restricted universe multiverse
  Active apt repos in: /etc/apt/sources.list.d/winehq-bullseye.sources
    1: deb [arch=amd64 i386] https: //dl.winehq.org/wine-builds/debian bullseye main
Info:
  Processes: 450 Uptime: 2h 50m Memory: 31.03 GiB used: 5.69 GiB (18.3%) Init: systemd v: 249
  runlevel: 5 Compilers: gcc: 11.4.0 alt: 11/12 Client: Cinnamon v: 5.8.4 inxi: 3.3.13

I manually set my tile sizes to 256 on the upscalers in Channner and it stopped the black output. I hope this helps and can by applied in some way here.

authors: 78Alpha BahzBeih Egazaga Hudaldadi OrloDavid Tsubajashi n00mkrad

repo: nihui/cain-ncnn-vulkan


text:
No image output and broken on AMD Adrenalin 22.5.2

AMD Adrenalin 22.5.2 brings in additional Vulkan extensions and that might've broken it.
In some weird way, it still works if I enable TTA and it outputs an image, but with it disabled, it seems to "succeed" but has no image output and no errors, the reason I think it is working fine is because it takes some time to process instead of failing immediately.
Is there some workaround for this? The only workaround I have for now is downgrading back to 22.5.1. Maybe it's not the fault of the project either but AMD, and could be fixed next update, but just submitting an issue in here just in case.

oh cool im not alone! same issue with pro driver 22.q2
except it returns 3221226356
not sure if video2x as gui front end matters

can confirm that TTA works on both waifu and realsr. i have no clue what that is but thanks for the temporary fix!

can confirm that TTA works on both waifu and realsr. i have no clue what that is but thanks for the temporary fix!

TTA averages outputs by upscaling it 8 times in different angles, which means it's 8x slower, so while it may be a fix, it's not really an ideal one. Read more about it in here

This issue has been fixed with 22.6.1 WHQL drivers.


text:
Library import 241224-0958

PR was created by rightlib sync workflow run

Check checks_integrated failed. All future check are suspended, please remove the rightlib-fail label to enable checks.

Unable to merge PR All future check are suspended, please remove the rightlib-fail label to enable checks.

The PR was successfully merged into main using workflow

repo: nikitka/ydb


text:
Repo dead?

Hi,
is this repo dead? Or when will the 5.8 integration be available?

Sorry for the delay, merged the pull request to support laravel 5.8.

authors: nikkanetiya sheriffmarley

repo: nikkanetiya/laravel-color-palette


text:
fix(tokens): limit value of context to defined keys

Which issue does this PR address?
Closes #15
Note: when we are more restrictive with the context, we now need to define the special name item explicitly in filter expressions

I guess we're fixing another issue with this, too: https://nikku.github.io/feel-playground/?e=[+1+].Mike's+age&c=~3YCAkIC%2BgICAgICAgIC9AgCXnLILCNLEWVvD4A7RTE6e5VQHnL67RAf8ffGpOTFExGjxp2lIHt%2Fmy8diAzMQSsxIFwmdegBxI7oRrX98i6CA&t=expression.
Could you test cover that one as well?

authors: marstamm nikku

repo: nikku/lezer-feel


text:
Bootstrap Easy issue fixed

Bootstap issue fixed :- Added bootstrap classes to make it look nice

Bootstap issue fixed :- Added bootstrap classes to make it look nice

Well done!

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!

repo: nikohoffren/fork-commit-merge


text:
Add Dark Mode

🚀 Feature
(A clear and concise description of what the feature is.)
Add a dark mode and light mode
Motivation
(Please outline the motivation for the proposal.)
it will look nice

Pitch
(Please explain why this feature should be implemented and how it would be used.)
Add relevant screenshot or video (if any)
(Add here)

The website is currently having dark mode @SanjeevKumar22

repo: nilisha-jais/Musicophilia


text:
Adding a CONTRIBUTING.md file.

This file details contributing guidelines for users that wish to submit
a pull request. The README has also been modified to reference this new
file.

 [Infrastructure] Add a "Contributing" section to our public repository READMEs. [0]

repo: nimbis/djangocms-vimeo


text:
Fix demos

Resolves #152.
Some demos are not salvagable and I did not add new ones. But at least it's something for additional testing.

Rebased and fixed export demo.

repo: nimiq/keyguard-next


text:
Sidebar colors

Provide a description of the new feature
The current version of Bloom theme doesn't exactly change the sidebar colors and keeps the standar color scheme rules(?), which means the sidebar buttons have a background that follows the "main" color scheme and not leaving the whole sidebar color solid like the old "Fluent" theme...
So i expect the sidebar doesn't have the "main"  color on his buttons backrounds and follows the "sidebar" color, just like the old "Fluent" theme.
Current --> What i expect:

Specifications:
Spotify v1.2.26.1187
Spicetify v2.29.1
Additional Information
No response

will be changed only in fluent-dark and fluent-light color schemes
screenshots are old and wip

will be changed only in fluent-dark and fluent-light color schemes
screenshots are old and wip

I see... So there's gonna be new color schemes like fluent-dark and fluent-light?
So, those screenshots are a WIP from a future "Bloom" update or something? Also... That "Mica" effect is just a wallpaper inside spotify app? Or did you used a version of Spotify where is compatible with MicaForEveryone? I ask because i'm curious those buttons of minimize, maximize and close buttons aren't black as normally is... Like this one:

@AndresF2016 yes! we were developing a way to simulate the mica feeling for our users! it didn't go quite well, as you know spotify is a chromium-based web-app. using this type of simulations might cause increases in recourse usage from our users' pcs.
We lost between the decision whether to make it beautiful or to protect the principle called Calm and Cleen! so like always, we had to surrender in front of fluent2 principles

authors: AndresF2016 SunsetTechuila nimsandu

repo: nimsandu/spicetify-bloom


text:
cowboy_req:req() typespec is too strict

As the doc has stated, cowboy_req:req() is meant to be expandable by user. However, matching an user field like this:
-spec match(cowboy_req:req()) -> ok.
match(#{user_field := UserField} = Req) -> ok.

would result in dialyzer complaining about wrong spec since user_field is not part of the spec.
What do you think about adding _ => _ to the spec to denote free extension? I have tested and it solves all dialyzer issues.

Yeah that's probably what's missing.

I've fixed this locally, corrected the example and added a test. Push soon. Thanks!

repo: ninenines/cowboy

license: isc


text:
small formatting mistake in docs

in http://ninenines.eu/docs/en/cowboy/2.0/guide/req/
where it says
 Read more about constraints

constraints is formatted as a exponent or something similar, I think you wanted to make it a link?

lol
Well that's an oversight from converting to asciidoc I guess. Should be an xref link.

That one is also fixed, thanks!

authors: essen marianoguerra


text:
Allow adjusting Type, Color, Size from Code

Hi,
just tried to use this from a Storyboard. Would be great to be able to set the above values in code before calling startAnimation() ;)
Creating a PR for it..
Cheers

PR #17
And yes, I saw the delegate. But that's a bit oversize if you just want to set the color before calling startAnimation().

Nice point :+1:. Merged.

You're welcome! Thanks for the component and merging :)

authors: ninjaprox winkelsdorf

repo: ninjaprox/NVActivityIndicatorView


text:
Added preserve_names option, fixed position_selector duplication button

Added preserve_names option to sort by DOM order while preserving form names
Fixed position_selector duplication button

Symfony normally processes forms in order where it's fields were defined in it's form type so the order of the fields in DOM is not significant. However, there is an open issue with a solution how to handle this case.
https://github.com/symfony/symfony/issues/21595
While i was testing it I encountered a few bugs with combination of position_selector option which I have also fixed because the solution is also preserves keys.
I guess it could be a little bit simplified but I did not want to touch your code too much.
Thanks for this repo.

Well. Actually the findFreeNumericKey is not needed since [] can be used instead. I will update it probably tomorrow.

Hello maryo,
Thank you for your PR.
What are the bugs you encountered with position_selector? It seems that the new option preserve_names you propose does the very same thing.
There were a few bugs in this option, fixed 2 days ago.
Did you have a new try since 2.1.13 release?
Do you reproduce your issues at https://symfony-collection.fuz.org/symfony3/basic/positionField ?
Have a good day.

It seems that the 2.1.13 release was broken though (I pushed the bumping onto another branch...), it may be related. Updated to 2.1.14 just in case.
Please have a new try.

The new option is similar but it supports any key like uuid string keys etc. Not only numeric ones. I was using the new version from master I cloned yesterday. Try the example you linked and duplicate "walk the dog". Then duplicate this new duplicate again. It is inserted at the end instead of third position. And I think it has the same duplicated name but I am on my phone without devtools now. That's what I fixed.

You are right, thank you for the clarification and the fix 👍
About "preserve_names", well, it's:

incompatible with "allow_up" and "allow_down" options if position_field is null because symfony will order using those names

useless if position_field is not null because it does the same thing

So I propose that we enforce "allow_up" and "allow_down" to false if "preserve_name" is set to true.

Sounds good, will merge, add the doc and do what I wrote above.
Thanks for your contribution, I've only a few time for this plugin so that's highly appreciated.
Cheers

About

Well. Actually the findFreeNumericKey is not needed since [] can probably be used instead. I will update it probably tomorrow.

I'll let you create a new PR if you feel it needed.

It's incompatible and useless by default. But check this issue https://github.com/symfony/symfony/issues/21595
and the RestoreInputOrderExtension. I use it. Reordering then works even without the position field. And perhaps this feature could be solved in future in the Form component.

I understand the purpose, don't worry.
It just required a bit of improvements to make it cohabit with other options.
I think now it works as expected (currently writing demos on the website).

@ninsuo Thank you for such a quick integration 👍 .
But it is compatible with "allow_up" and "allow_down" options as well when the mentioned reordering form type extension is enabled. I was having a limited time yesterday to explain it better. Maybe the options could be set to false by default because it needs the type extension but it should not enforce it so yt can set back to true.
// if preserve_names option is set, we should enforce many options to avoid
// having inconsistencies between the UI and the Symfony result
if (settings.preserve_names) {
    settings.allow_up = false;
    settings.allow_down = false;
    settings.drag_drop = false;
    settings.add_at_the_end = true;
 }

Well. Actually the findFreeNumericKey is not needed since [] can probably be used instead. I will update it probably tomorrow.

So actually I realized it is needed :-). [] can not be used because

All right, this backend ordering integration makes sense. I may document it as well.
Will remove the options enforcing i added then (but not today weather is great going to enjoy it 😊).

authors: maryo ninsuo

repo: ninsuo/symfony-collection


text:
Pare down top-level table-of-contents

Right now the top-level ToC is:

Installation
Usage Notes
Running fMRIPrep via Docker containers
Running fMRIPrep via Singularity containers
Processing pipeline details
Susceptibility Distortion Correction (SDC)
Outputs of fMRIPrep
Defining standard and nonstandard spaces where data will be resampled
FAQ, Tips, and Tricks
Contributing to FMRIPREP
Citing fMRIPrep
API
What’s new

This is a lot to look through, and a pretty serious mixing of broad categories and specific issues. This conversation began in #1802, which was an attempt to deal with a rash of permissions issues faced by Singularity users, mostly related to TemplateFlow. In https://github.com/poldracklab/fmriprep/pull/1802#issuecomment-538131273, it was decided to defer another round of reorganization until today.
I would propose the following top-level structure:

Installing and Running
Processing Pipeline Details
Inputs, Outputs and Templates
Contributing
Citing
FAQ
API

Specifically, in the previous issue I proposed the following:

Installing and Running

Installation (subpage)

pip install fmriprep
pip install fmriprep-docker
docker pull
docker build (might just link out to contributors)
Fetch singularity (I guess from docker hub)
Build singularity (singularity build and docker2singularity)

Running fMRIPrep (subpage - mostly just the current usage notes)
Running fMRIPrep via Docker containers (subpage)
Running fMRIPrep via Singularity containers (subpage)

Processing pipeline details
...

I can flesh this out further in comments, but I want to start this conversation now.

A full ToC:

Installing and Running

Installation (subpage)

pip install fmriprep
pip install fmriprep-docker
docker pull
docker build (might just link out to contributors)
Fetch singularity (I guess from docker hub)
Build singularity (singularity build and docker2singularity)

Running fMRIPrep (subpage - mostly just the current usage notes)
Running fMRIPrep via Docker containers (subpage)
Running fMRIPrep via Singularity containers (subpage)

Processing pipeline details

Anatomical preprocessing
BOLD preprocessing
Susceptibility distortion correction

Inputs, Outputs and Templates

Inputs: BIDS, Pre-run FreeSurfer, touch on TemplateFlow (subpage)
Outputs (subpage)
Spaces and Templates (subpage)

Contributing
Citing
FAQ
API

I agree with this overall structure. :+1:
Side Note:
I think API has a lot of great information for users (the doc strings give good conceptual overviews of the workflows), but I'm afraid most will not think to look there (API connotes highly technical to me).
The solution I see would be to reference the API in other sections where appropriate to guide people to know how a workflow is constructed.

I think with 21.x we should address this issue. After all, it doesn't really require generating new documentation, but rather reorganizing existing contents.
One large change w.r.t. when this was proposed is that many sections describing the execution of containers and community/contribution guidelines have been migrated to nipreps.org (cf. #2469).
I would propose the following, based on the latest draft from Chris:

About/Home (landing page, doesn't have an entry in the menu)

README import (as it is now)
Licensing (I'll add this into #2325)
Citing/boilerplate (will require creating some specific subsection about boilerplates at nipreps.org).

Installing and Running

Installation (subpage)
pip install fmriprep
pip install fmriprep-docker
docker pull
Other (links to nipreps.org, including docker build, fetch singularity and build singularity)
Running fMRIPrep (subpage - mostly just the current usage notes)
Running containers (subpage - mostly just like current docs after #2469 was merged)

Processing pipeline details

Anatomical preprocessing
BOLD preprocessing
Susceptibility distortion correction

Inputs, Outputs and Templates

Inputs: BIDS, Pre-run FreeSurfer, touch on TemplateFlow (subpage)
Outputs (subpage)
Spaces and Templates (subpage)

FAQ
Developers

Guidelines (link to nipreps, mostly just like current after #2469)
Dev environment  (link to nipreps, mostly just like current after #2469)
API

@oesteban Looks good. The homepage is linked to by the home button in the top left, so I don't know it's necessary to add a new link to the landing page:

That said, it is a somewhat sad looking corner. Maybe we need a logo?

The homepage is linked to by the home button in the top left

Yup, I just wanted to be explicit about the contents of the landing page.

Maybe we need a logo?

I've thought of this a thousand times. Maybe we can run a call for logos starting next roundup?

This is completed.

authors: effigies jdkent oesteban

repo: nipreps/fmriprep


text:
Unable to get grouped photos

I'm using NPOI version 2.5.5 and had a problem trying to get the photo attached to the sheet. Multiple photos are attached, and if you group them, you will not be able to get individual photos.
I'm getting photos as shown below, but I can't get grouped ones.
Is there a way to get photos that are grouped by sheet?
Or is there a way to get a file that is an image of grouped photos / figures?
Book1.xlsx
`		public static ListGetPictures(this ISheet me)
{
var pictures = new List();
if (me.Workbook is HSSFWorkbook)
{
foreach (var shape in ((HSSFPatriarch)me.DrawingPatriarch).Children)
{
if (shape is HSSFPicture picture)
{
var position = new CellPosition(picture.ClientAnchor.Row1, picture.ClientAnchor.Col1);
var pic = new Picture(position, picture.PictureData);
					pictures.Add(pic);
				}
				else if (shape is HSSFShapeGroup group)
				{
					pictures.AddRange(GetPictures(group));
				}
			}
		}
		else if (me.Workbook is XSSFWorkbook)
		{
			foreach (var shape in ((XSSFDrawing)me.DrawingPatriarch).GetShapes())
			{
				if (shape is XSSFPicture picture)
				{
					var position = new CellPosition(picture.ClientAnchor.Row1, picture.ClientAnchor.Col1);
					var pic = new Picture(position, picture.PictureData);

					pictures.Add(pic);
				}
			}
		}

		return pictures;
	}

	static private List<Picture> GetPictures(HSSFShapeGroup group)
	{
		var pictures = new List<Picture>();
		if (group.Anchor is HSSFClientAnchor anchor) {
			var position = new CellPosition(anchor.Row1, anchor.Col1);

			foreach (var shape in group.Children)
			{
				if (shape is HSSFPicture picture)
				{
					var pic = new Picture(position, picture.PictureData);
					pictures.Add(pic);
				}
				else if (shape is HSSFShapeGroup group2)
				{
					pictures.AddRange(GetPictures(group2));
				}
			}
		}
		return pictures;
	}`

You are the first one to ask this question. Grouped pictures are definitely a special case. You are showing me a xlsx file but you are using HSSFWorkbook in your code,which is for xls file. Can you clarify which file do you want to deal with?

I want to handle the 3 extensions xls, xlsx, xlsm.
Book1.xls

May I ask what's your business scenario of using NPOI? It looks you are a GIS guy and you want to handle some terrian data by NPOI.

The attached data is a sample of a group of photos and shapes, and the photos themselves have no meaning.
What I want to do is to take in the record created in Excel and keep it as electronic data.
The record includes visualized drawings and photographs, which are grouped into meaningful data.
I want to capture them.

#488

I'm using POIFS Explorer to figure out your grouped  shape cases. I'm afraid it will be very complicated. It's kind of space tree structure.  (similiar to BSP)

To understand what's going on, there is another way for you to understand how Office store this grouped structure.

Save Book1.xls as Book1.xlsx
Unzip the xlsx file to a folder.
Go to \xl\drawings\drawing1.xml
Using a xml formatting tool to format the xml tree and make it readable.

You will see a few xdr:grpSp pairs as below.

Thank you for your survey and response.
I understand that it is difficult to deal with this.
If you have similar requests in the future, please reconsider.

authors: FCHata tonyqus

repo: nissl-lab/npoi


text:
GLM on the cortical surface

People would like to run the GLM on the corticalsurface.
THis could be implemented easily by relying on recent nilearn surface projection functions.

@bthirion I would be glad to chip in on this one. Seems that the main point is to make a meta pipeline where you sample the surface from the image and then execute GLM (using nistat's GLM) ? Am I understanding your idea well?

Yes. I would do that as an example to start with and then see how to factor contributions into modules.

Addressed by #196

Closing, since this has been addressed

authors: bthirion demianw

repo: nistats/nistats


text:
Update CentOS and add Fedora

Use Fedora EPEL repository consistently rather 3rd-party packages, mention prominent CentOS downstreams (CentOS is moving from a rebuild to a pre-RHEL position and gets more an upstream), add instructions for Fedora.

Thanks!

authors: nitefood robert-scheck

repo: nitefood/asn


text:
entities and caption_entities does not support MessageEntity[]

The bug
the bug is described as follows in the title
Environment

package
version

puregram
2.2.0

TypeScript
4.5.3

currently generate-interfaces does support both MessageEntity[] and TelegramMessageEntity[], should work :shrug:

repo: nitreojs/puregram

license: WTFPL


text:
Improve unhandled dev server errors

Describe the feature
With recent ecosystem updates, several issues surfaced that fail dev server startup, but Nitro does not provide enough context of what error is,
Only simple message like [ERROR [worker reload] [worker init] ../.nuxt/dev/index.mjs failed](https://github.com/primefaces/primevue/issues/6712#top) is currently displayed.
One simple way to find root of issues currently is to do a production build (npm run build should show more hints)
A nice solution (https://github.com/primefaces/primevue/issues/6712) had been suggested by @web-dev-sam to attach error cause.
(I will be working on a PR but if anyone wanted to pick it earlier, please ping + consider helping to make a minimal reproduction to confirm our fix alongside)
Additional information

[X] Would you be willing to help implement this feature?

@pi0 I think I can handle it this week. Is that ok for you?

Got something like that:

Is it ok? 😊  The reproduction is quite simple for the Nuxt. Just add app.config.ts:
// app.config.ts

export default defineAppConfig({
  ready: ref(false),
});

ref is not defined here, so the worker will not initialize.

I've created 2 PR for that 🧑💻 for V2 and V3

@pi0 can u move that forward?

repo: nitrojs/nitro


text:
Add defaultText to extraBin.namd

fixes #335

Please squash merge

Done already if not mistaking

Do you want me to change the commit message?

repo: nix-community/NixOS-WSL


text:
IRI as object

Hi,
in our project we have mapping rule
rr:predicateObjectMap [
rr:predicate jup:seeAlso;
rr:objectMap [ rr:column "web_url" ; rr:termType rr:IRI; ];
];
so we expect something like this
[] jup:seeAlso <http://www.another-example.org> .
but the parser generates this :(
[] jup:seeAlso <http://lod.mapybezbarier.cz/resource/http%3A%2F%2Fwww.another-example.org> .
Can anybody help me please?
Thank you
Petr

Hi Petr,
Thanks for bringing this up. I added one more property, default.forceURI in r2rml.properties that configures the tool to treat objects as URIs (even if they are explicitly declared as Literals).
See latest commit (960b099) and please let me know if this solves this issue.
Best,
Nikolaos

Hi Nikolaos,
thank you very much. Is it possible to rebuild jar on GitHub, please?
Best regards,
Petr

Hi,
property default.forceURI in r2rml.properties works if there is a rr:template, not a rr:column.
In our case:
rr:predicateObjectMap [
rr:predicate jup:seeAlso;
rr:objectMap [ rr:template "{web_url}" ; ];
];
Thank you very much Nikos!


text:
added a method in LocalBlobStore.Tx to rename the file ext of a sym link...

Added method to set the mime type to be other than jp2 for master tif images.

Changes Unknown when pulling e68703f117f5ad3d1b16fdb7c491c2ad9021c980 on symlink-upd into ** on master**.

authors: coveralls shuangzhou

repo: nla/doss


text:
JobFinished Event Not Fired when Level Up

Hello,
I have found a bug where the job data is not reset after a profile upgrade/level up. The OnJob stays true and the JobFinished stays false and everything indicates that the hob is still in progress, even the trailerattached is still true. This testing was done in the quick job/not owning my own truck on a new profile. Can you fix this?

The onJob flag is reset once the trailerAttached value goes low. See https://github.com/nlhans/ets2-sdk-plugin/blob/master/ets2-telemetry/src/ets2-telemetry.cpp#L95
The trailer attached flag, as you indicated stuck high, is directly from ETS2 SDK: https://github.com/nlhans/ets2-sdk-plugin/blob/master/ets2-telemetry/src/ets2-telemetry.cpp#L382
This indicates a minor bug in ETS2 causing the failure of the "onJob" flag. Either because of indeed the level-up screen, or because of the quick job (I haven't done a quick job in ages).
Unfortunately I don't have got any other information to go by that the job has been completed, so I can't easily think of a fix.

authors: jammerxd nlhans


text:
Destructor is called when filling object through assignement

Bug Report

What is the issue you have?
Destructor being called in an unexpected place.

Please describe the steps to reproduce the issue. Can you provide a small but working code example?

#include <iostream>
#include "nlohmann/json.hpp"

using namespace std;
using json = nlohmann::json;

class State {
public:
    double x;
    ~State() {
        cout << "destroyed" << endl;
    }
};

void from_json(const json& j, State& p) {
    try {
        p.x = j.at("x").get<double>();
    } catch(json::type_error& e) {
        std::cerr << e.what() << std::endl;
    } catch(json::out_of_range& e) {
        std::cerr << e.what() << std::endl;
    }
}

int main() {
    json data;
    State s;
    data["x"] = 10.;
    s = data;
    cout << s.x << endl;
    return 0;
}

Output is:
destroyed
10
destroyed

What is the expected behavior?
do not call destructor during assignment

And what is the actual behavior instead?
destructor is called

Which compiler and operating system are you using? Is it a supported compiler?

g++ (Gentoo 6.4.0-r1 p1.3) 6.4.0
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Did you use a released version of the library or the version from the develop branch?
checked for both: master and develop branch.

Could you please describe, whether it is a feature or a bug :) By default assignment operator doesn't call destructor, that is why the behavior seems to me unexpected.
Thank you!

Thank you very much for immediate response!
Actually, I had a bit more complicated situation. I'm freeing memory for some properties in the destructor. So the way out is to implement own move-assignment operator, hence move-constructor as well. Is it right workaround?
#include <iostream>
#include <string>
#include "nlohmann/json.hpp"

using namespace std;
using json = nlohmann::json;

class State {
public:
    string* x;

    State() {
        x = new string("hello");
    }

    State(State &&from) {
        x = from.x;
        from.x = nullptr; // delete does nothing with nullptr starting from c++14
    }

    State& operator=(State &&from) {
        if (this == &from) {
        }
        x = from.x;
        from.x = nullptr;
        return *this;
    }

    ~State() {
        cout << "destroyed" << endl;
        delete x;
    }
};

void from_json(const json& j, State& p) {
    try {
        *p.x = j.at("x").get<string>();
    } catch(json::type_error& e) {
        std::cerr << e.what() << std::endl;
    } catch(json::out_of_range& e) {
        std::cerr << e.what() << std::endl;
    }
}

int main() {
    json data;
    State s;
    data["x"] = "world";
    s = data;
    cout << *s.x << endl;
    return 0;
}

Now it works. And output is:
destroyed
world
destroyed

Thank you again!

I think so. I'm not sure whether you'd need to call std::move or std::swap at some place - but basically this should be it. To be sure, I'd mark the copy constructor and the copy assignment as delete.

Yes. Sounds reasonable. Thank you for review :) I like the library very much!

repo: nlohmann/json


text:
Complete contributor list

I forgot to add some contributors from the 3.11.0 release.

Coverage remained the same at 100.0% when pulling 79ea26105f83e5f79b3eddbbe82dd3c4fe8b4e99 on complete_contributors into 2ac7d023d424fffa31f2150103b7851c998061f4 on develop.

authors: coveralls nlohmann


text:
Loading data into data node fails with example script

Problem description
I ran the examples/push_dataset.py program to load the example data set into our data node and it failed to load the data.
Steps to reproduce the issue
I first edited the push_dataset.py and changed the host  line to read :
host = "http://ourhost.mcw.edu:8080/api/v1"
Then ran the program :
python push_dataset.py
Deleting exist dataset: test-dataset
Creating dataset: test-dataset
Creating Fhir Store: evaluation
Creating Annotation Store: goldstandard
Creating patient: 110
Creating note (110-01) for patient (110)
Traceback (most recent call last):
  File "push_dataset.py", line 114, in <module>
    note_create_request=note
  File "/home/crdw/nlpsandbox-controller/env/lib64/python3.6/site-packages/datanode/api_client.py", line 764, in __call__
    return self.callable(self, *args, **kwargs)
  File "/home/crdw/nlpsandbox-controller/env/lib64/python3.6/site-packages/datanode/api/note_api.py", line 122, in __create_note
    return self.call_with_http_info(**kwargs)
  File "/home/crdw/nlpsandbox-controller/env/lib64/python3.6/site-packages/datanode/api_client.py", line 811, in call_with_http_info
    self.__validate_inputs(kwargs)
  File "/home/crdw/nlpsandbox-controller/env/lib64/python3.6/site-packages/datanode/api_client.py", line 711, in __validate_inputs
    configuration=self.api_client.configuration
  File "/home/crdw/nlpsandbox-controller/env/lib64/python3.6/site-packages/datanode/model_utils.py", line 1403, in validate_and_convert_types
    check_type=_check_type
  File "/home/crdw/nlpsandbox-controller/env/lib64/python3.6/site-packages/datanode/model_utils.py", line 1284, in attempt_convert_item
    configuration, spec_property_naming)
  File "/home/crdw/nlpsandbox-controller/env/lib64/python3.6/site-packages/datanode/model_utils.py", line 1202, in deserialize_model
    return model_class(**kw_args)
  File "/home/crdw/nlpsandbox-controller/env/lib64/python3.6/site-packages/datanode/model_utils.py", line 1564, in wrapped_init
    return fn(self, *args, **kwargs)
TypeError: __init__() missing 1 required positional argument: 'note_type'

Expected behavior

Data loaded into data node

Extra information
I edited the example-patient-bundles.json and put in values for "type" field  :
   "type": "pathology",

and received the same error.

@gkowalski , this also seems like an issue with the data node not being up to date.

I have done a git pull and have the latest code :
cd data-node
$git remote -v
origin	https://github.com/nlpsandbox/data-node.git (fetch)
origin	https://github.com/nlpsandbox/data-node.git (push)
$git branch

develop

$git log -1 --oneline
c6ef9d4 Revert python slim buster (#159)
Seems to be the latest from https://github.com/nlpsandbox/data-node

@gkowalski

File "/home/crdw/nlpsandbox-controller/env/lib64/python3.6/site-packages/datanode/model_utils.py", line 1564, in wrapped_init
return fn(self, *args, **kwargs)
TypeError: init() missing 1 required positional argument: 'note_type'

This message tells us that the issue is related to a Python property named note_type in the file datanode/model_utils.py. The first thing to check is: Is this property part of the Note schema, which is source of truth. The property noteType in the schemas - which is converted to lower kebab case in Python object (i.e. note_type) - does not exist (any more). So there is somewhere a reference to noteType (or note_type in Python) that should be renamed to type.
Here is a link to the file datanode/model_utils.py listed in the error message. This file is part of the nlpsandbox-client and seems to have been generated automatically. This file does not include a reference to note_type. However, the client has two files that incorrectly include note_type: https://github.com/nlpsandbox/nlpsandbox-client/search?q=note_type. Several occurences are commented but one that is used in the integration test. The Note model included in the client is up to date and reference type and not note_type. So aside the type in the tests, the client seems up to date with the latest schemas.
@gkowalski Can you tell me what version of the NLP Sandbox client for Python you are using?
@thomasyu888 There's one thing that is weird: the latest stable release that you created on GitHub is 1.0.0 on March 22. However, the current version in https://github.com/nlpsandbox/nlpsandbox-client/blob/develop/nlpsandboxclient/init.py is 0.4.1. There is a mismatch there. There is indeed a version 0.4.1 of the package on PyPi, but it has been pushed on Feb 27, 2021. Looking at the log of the CI workflow that run when you created the release 1.0.0, I see that the CI workflow failed because PyPi detected that the version 0.4.1 already existed.
https://github.com/nlpsandbox/nlpsandbox-client/runs/2172520539
Run pypa/gh-action-pypi-publish@master
/usr/bin/docker run --name e4b38ca0d820604564a3174612a22558b3_bbbe30 --label 5588e4 --workdir /github/workspace --rm -e docker_repository -e pythonLocation -e INPUT_PASSWORD -e INPUT_USER -e INPUT_REPOSITORY_URL -e INPUT_PACKAGES_DIR -e INPUT_VERIFY_METADATA -e INPUT_SKIP_EXISTING -e INPUT_VERBOSE -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_REPOSITORY_OWNER -e GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_RETENTION_DAYS -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_SERVER_URL -e GITHUB_API_URL -e GITHUB_GRAPHQL_URL -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e GITHUB_ACTION_REPOSITORY -e GITHUB_ACTION_REF -e GITHUB_PATH -e GITHUB_ENV -e RUNNER_OS -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/nlpsandbox-client/nlpsandbox-client":"/github/workspace" 5588e4:b38ca0d820604564a3174612a22558b3  "__token__" "***" "" "dist" "true" "false" "false"
Checking dist/nlpsandbox-client-0.4.1.tar.gz: PASSED
Uploading distributions to https://upload.pypi.org/legacy/
Uploading nlpsandbox-client-0.4.1.tar.gz

  0%|          | 0.00/122k [00:00<?, ?B/s]
  7%|▋         | 8.00k/122k [00:00<00:04, 28.2kB/s]
100%|██████████| 122k/122k [00:00<00:00, 180kB/s]
NOTE: Try --verbose to see response content.
HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/
File already exists. See https://pypi.org/help/#file-name-reuse for more information.

Proposed solution

Open a PR to update the version of this package to 1.0.1 (__init.py__)
Create a new release version 1.0.1
Ask George to use the client version 1.0.1

Following the above discussion, we have just released the NLP Sandbox client version 1.0.1 for Python on PyPi.
@gkowalski Can you state whether you are using the client installed from PyPi or from source? If it's from source, us releasing the new version of the client won't help because the new release does not solve any know issues.
Please use the example script push_dataset.py provided in this repository along with the sample of the i2b2 data (example-patient-bundles.json).

I always use the source code . I didn't know of the pypi version  .

Please update your client (both the source and the pypi package have been updated). The client no longer has reference to note_type so you should not long get the error that you share above.

authors: gkowalski thomasyu888 tschaffter

repo: nlpsandbox/nlpsandbox-client


text:
Update 607935-Tasks.md

I removed "reproducible" because I'm not sure it's an appropriate term for NLP tools in this context. I'm open to being corrected if there's a good explanation of what it's supposed to mean.
I'm not sure what:

(see Section OpenAPI TODO)

means.
A few key terms referring to groups/individuals are capitalized (Driver, Developer, Data Hosting Sites). This seems odd and I'm getting rid of the capitalization unless there's a precedent/good reason for it. I also tried changing the word "Driver" to "collaborator" and "Developer" to "participant".

Merged as part of #69

authors: cascadianblue tschaffter

repo: nlpsandbox/nlpsandbox-website-synapse


text:
Signed integer division

I have a question: I do not really understand how and why stage_3/valid/div_neg.c ((-12)/5;) should work. My code crashes with
30274 floating point exception (core dumped)  ./div_neg , whereas the result should be 254 (this is the result, when the programm is compiled with gdb).
Why should the output be 254?
                            s = $"{Generate(rootNode.Children[0])}" +
                                "push %rax\n" +
                                $"{Generate(rootNode.Children[1])}" +
                                "movl %eax, %ecx\n" + //move calculated divisor to %ecx
                                "pop %rax\n" + //pop divident do %eax
                                "cdq\n" +
                                "divl %ecx\n"; //eax contains the result, edx the rest

This is my code for the division. Any help would be appreciated!
This is my code: https://github.com/Clemens-Dautermann/lcc

It would be easier to help if you simply showed what your generated assembly looks like.
Regarding the 254 value: how the returned int is treated is an OS specific thing. On Unix, I think it is converted to an unsigned 8 bit value, which matches the 254 value you see. What OS do you use?

It would be easier to help if you simply showed what your generated assembly looks like.
Regarding the 254 value: how the returned int is treated is an OS specific thing. On Unix, I think it is converted to an unsigned 8 bit value, which matches the 254 value you see. What OS do you use?

Oh yeah sorry.
Here is the assembly:
´´´
.globl main
main:
movl $12, %eax
neg %eax
push %rax
movl $5, %eax
movl %eax, %ecx
pop %rax
cdq
divl %ecx
ret
´´´
I am on x86 Gnu/Linux (Arch).
Thank you!

Oh yeah sorry.
Here is the assembly:
´´´
.globl main
main:
movl $12, %eax
neg %eax
push %rax
movl $5, %eax
movl %eax, %ecx
pop %rax
cdq
divl %ecx
ret
´´´
I am on x86 Gnu/Linux (Arch).
Thank you!

The message "floating point exception" is a bit unfortunate, since it can mean other things than errors from floating point computations. Rather, it is a general arithmetic exception. (See SIGFPE here: https://www.gnu.org/software/libc/manual/html_node/Program-Error-Signals.html)
To see why, first note that you are using the unsigned integer division (div) rather than the signed integer division (idiv) that Nora suggests. Now, the div instruction can produce a number of exceptions (https://c9x.me/x86/html/file_module_x86_id_72.html). In particular, one of them is that if the result of the division doesn't fit in the output register, an overflow exception occurs.
In your case, when you do cdq, %eax is sign extended into %edx, which for -12 means %edx becomes all ones. Then you do unsigned division of the result in %rax, i.e. 18446744073709551604, by 5. This becomes 3689348814741910320, which is much too large to fit in a 32-bit register.
Basically, replace div by idiv.

The message "floating point exception" is a bit unfortunate, since it can mean other things than errors from floating point computations. Rather, it is a general arithmetic exception. (See SIGFPE here: https://www.gnu.org/software/libc/manual/html_node/Program-Error-Signals.html)
To see why, first note that you are using the unsigned integer division (div) rather than the signed integer division (idiv) that Nora suggests. Now, the div instruction can produce a number of exceptions (https://c9x.me/x86/html/file_module_x86_id_72.html). In particular, one of them is that if the result of the division doesn't fit in the output register, an overflow exception occurs.
In your case, when you do cdq, %eax is sign extended into %edx, which for -12 means %edx becomes all ones. Then you do unsigned division of the result in %rax, i.e. 18446744073709551604, by 5. This becomes 3689348814741910320, which is much too large to fit in a 32-bit register.
Basically, replace div by idiv.

Well that's a simple fix. Thank you for the detailed answer! It is from those mistakes and answers I personally learn the most!

Well that's a simple fix. Thank you for the detailed answer! It is from those mistakes and answers I personally learn the most!

Yeah, I also learned something from this :)

Yeah, I also learned something from this :)

authors: Clemens-Dautermann kalj


text:
Axe FreqDist ordering/comparisons?

This StackOverflow question made me aware that the FreqDist class implements ordering comparisons. As the question shows, the implementation is buggy so we should do something about it.
I propose we get rid of it altogether, considering that its parent class Counter doesn't support comparisons and that it's not entirely clear what it means for one frequency distribution to be "greater than" another.
Thoughts?

@copper-head: the motivation is given in section 4.1 of chapter 2 of the NLTK book.
The operation corresponds to subset, for multisets.
Since the book uses it, I would rather fix the implementation than break the book example.

Hey! Could I give this a shot?

@bdevnani3 Sure! But this might be a little challenging since it has to do with builtin Python class.
The code referred to in this PR is in the nltk.probability.FreqDist specifically https://github.com/nltk/nltk/blob/develop/nltk/probability.py#L379
And the motivation of having __le__, __ge__, __lt__ and __gt__ functions in FreqDist is to check through all items in the FreqDist and make sure that all items are in the FreqDist has lesser/greater value than what it's compared to.
From http://www.nltk.org/book/ch02.html :

The FreqDist comparison method [3] permits us to check that the frequency of each letter in the candidate word is less than or equal to the frequency of the corresponding letter in the puzzle.

>>> puzzle_letters = nltk.FreqDist('egivrvonl')
>>> obligatory = 'r'
>>> wordlist = nltk.corpus.words.words()
>>> [w for w in wordlist if len(w) >= 6
...                      and obligatory in w
...                      and nltk.FreqDist(w) <= puzzle_letters]

['glover', 'gorlin', 'govern', 'grovel', 'ignore', 'involver', 'lienor', 'linger', 'longer', 'lovering', 'noiler', 'overling', 'region', 'renvoi', 'revolving', 'ringle', 'roving', 'violer', 'virole']

One solution is to remove these lines:
    def __le__(self, other):
        if not isinstance(other, FreqDist):
            raise_unorderable_types("<=", self, other)
        return set(self).issubset(other) and all(self[key] <= other[key] for key in self)

    # @total_ordering doesn't work here, since the class inherits from a builtin class
    __ge__ = lambda self, other: not self <= other or self == other
    __lt__ = lambda self, other: self <= other and not self == other
    __gt__ = lambda self, other: not self <= other

And write a generic inequalities function and make duck-types of the individual inequality:
    from operator import lt, gt, le, ge
    def inequalities(self, other, _operator):
        operator2string = {lt:"<", gt:">", le:">", ge:">",}
        if not isinstance(other, FreqDist):
            raise_unorderable_types(operator2string[_operator], self, other)
        return all(_operator(self[key], other[key]) for key in self)

    __lt__ = lambda self, other: self.inequalities(other, lt)
    __gt__ = lambda self, other: self.inequalities(other, gt)
    __le__ = lambda self, other: self.inequalities(other, le)
    __ge__ = lambda self, other: self.inequalities(other, ge)

[out]:
>>> import nltk
>>> nltk.FreqDist('abc') > nltk.FreqDist('abd')
False
>>> nltk.FreqDist('abd') < nltk.FreqDist('abc')
False
>>> nltk.FreqDist('abcc') > nltk.FreqDist('abc')
False
>>> nltk.FreqDist('aabbcc') > nltk.FreqDist('abc')
True
>>> nltk.FreqDist('aabbcc') < nltk.FreqDist('abc')
False
>>> nltk.FreqDist('abc') < nltk.FreqDist('aabbcc')
True
>>> nltk.FreqDist('abc') <= nltk.FreqDist('abc')
True
>>> nltk.FreqDist('abc') >= nltk.FreqDist('abc')
True
>>> nltk.FreqDist('abc') >= 'abc'
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/liling.tan/git-stuff/nltk-alvas/nltk/nltk/probability.py", line 402, in <lambda>
    __ge__ = lambda self, other: self.inequalities(other, ge)
  File "/Users/liling.tan/git-stuff/nltk-alvas/nltk/nltk/probability.py", line 396, in inequalities
    raise_unorderable_types(operator2string[_operator], self, other)
  File "/Users/liling.tan/git-stuff/nltk-alvas/nltk/nltk/internals.py", line 982, in raise_unorderable_types
    raise TypeError("unorderable types: %s() %s %s()" % (type(a).__name__, ordering, type(b).__name__))
TypeError: unorderable types: FreqDist() > str()

I've given this some thought and actually I think we shouldn't change the implementation (it makes sense), but instead document it clearly.
This involves me updating my SO answer, I'm cool with that, so just let me know what the final verdict on this is!

Hi,
I just had a student ask me about how FreqDist less than and greater than works. After explaining a bit about the order relation for sets they showed me this example. The fdist1 is both bigger and smaller than fdist2. This does not make sense to me.
import nltk
liste1=["This","is","a","a","list"]
fdist1=nltk.FreqDist(liste1)
liste2=["This","This","is","a","list"]
fdist2=nltk.FreqDist(liste2)
print(nltk.__version__) # 3.4.5
print(fdist1>fdist2) # True
print(fdist2>fdist1) # True

authors: Copper-Head alvations bdevnani3 pyfisch stevenbird

repo: nltk/nltk


text:
Improve documentation

I plan to keep this issue open indefinitely - please leave any requests/comments about improving the documentation below.

A CHANGELOG.md for pub.dev would be nice, at least for future releases

A LICENCE.md would be good to have as well

A CHANGELOG.md for pub.dev would be nice, at least for future releases

Provided we stick to Conventional Commits, melos will auto-update the changelog for each package, so agree that this should be a commitment going forward.

A LICENCE.md would be good to have as well

All packages should be Apache-2.0 licensed (including the repository itself), the files are there but perhaps incorrectly named/not in the right location to get picked up by GitHub/pub.dev. Any help to fix it up is welcome :)

A LICENCE.md would be good to have as well

All packages should be Apache-2.0 licensed (including the repository itself), the files are there but perhaps incorrectly named/not in the right location to get picked up by GitHub/pub.dev. Any help to fix it up is welcome :)

Seems like it's just missing a LICENSE file at the root of the repository. #42 fixes that.

Should we add an example to thermion_flutter? Something like the one in the old flutter_filament?

Should we add an example to thermion_flutter? Something like the one in the old flutter_filament?

I moved all the example apps to https://github.com/nmfisher/thermion_examples (mainly to avoid storing binary sample assets in this repository).
The flutter folder contains both the old example project and the new quickstart project, are you referring to those?

I moved all the example apps to https://github.com/nmfisher/thermion_examples (mainly to avoid storing binary sample assets in this repository).

I see. The only downside is that pub.dev doesn't pick up on it, but as long as there is a way to find them that's probably fine. Perhaps we could add an example/README.md with a simple example, and a reference to the complete example in the other repo?

The flutter folder contains both the old example project and the new quickstart project, are you referring to those?

That's what I was referring to, yes.

Perhaps we could add an example/README.md with a simple example, and a reference to the complete example in the other repo?

I think this is a good idea

Perhaps we could add an example/README.md with a simple example, and a reference to the complete example in the other repo?

I think this is a good idea

I have some time in the next few days, so I can work on this if you'd like.

Perhaps we could add an example/README.md with a simple example, and a reference to the complete example in the other repo?

I think this is a good idea

I have some time in the next few days, so I can work on this if you'd like.

That would be great!

repo: nmfisher/thermion