Anal core/extended download links are equal

There currently seems to be a misplaced download link for the Anal extended set. Both "Download annotation file" buttons refer to the same page on sharedocs. Also, if you switch between core and extended in the main page, you can see that for Anal, the count of speakers and words changes, but not the count for texts, which stays at 23.

Correct (I'm not sure why this is listed as a problem)
Non-core speakers within core files never appear in the metadata. This is intended.
core_extended is a file-level category, i.e. if the file belongs to the core set, all values will be core, even those that belong to non-core speakers. The latter will not have any entries in the ph column though.

Ok, so the info that they should be excluded comes solely from the ph-column. Thanks for the clarification.
It's listed as a (small) problem because the main page says it has 10 speakers, not 12.

Speaker count numbers for Anal will be fixed with the next update of the database. Thanks for pointing this issue out to us.


spec.volumes[0]: Required value: must specify a volume type

$ kubectl apply -f \
        nginx-pod-secrets-store-inline-volume-secretproviderclass.yaml

The Pod "nginx-keyvault-inline" is invalid:
* spec.volumes[0]: Required value: must specify a volume type
* spec.containers[0].volumeMounts[0].name: Not found: "keyvault"

What did i miss? Not sure how to enable feature gates (--feature-gates=CSIInlineVolume=true) on the api-server in AKS, how is that handled?
k8s version: 1.15.5 (on AKS)

@snobu Currently there is no way to add --feature-gates on AKS. This feature gate is enabled by default in 1.16. Would it be possible for you to try out secrets-store-csi-driver with a 1.16 cluster on AKS?

@snobu Please try it out on a 1.16 cluster. If you still have issues with 1.16 cluster feel free to reopen the issue.


add port param for etherlime ide to load the ganache

Codecov Report

Merging #198 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #198      +/-   ##
==========================================
+ Coverage   98.56%   98.56%   +<.01%
==========================================
  Files          73       73
  Lines        2856     2859       +3
==========================================
+ Hits         2815     2818       +3
  Misses         41       41

Impacted Files
Coverage Δ

cli-commands/commands.js
99.43% <100%> (ø)
:arrow_up:

cli-commands/etherlime-ide/etherlime-ide.js
97.91% <100%> (+0.09%)
:arrow_up:

Continue to review full report at Codecov.

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


Error: Invalid characters, got value: #

Facing this issue when i import existing cron from crontab
Error: Invalid characters, got value: #     at Function._parseField (/usr/lib/node_modules/crontab-ui/node_modules/cron-parser/lib/expression.js:155:11)     at parse (/usr/lib/node_modules/crontab-ui/node_modules/cron-parser/lib/expression.js:644:36)     at Function.parse (/usr/lib/node_modules/crontab-ui/node_modules/cron-parser/lib/expression.js:655:10)     at Function.parseExpression (/usr/lib/node_modules/crontab-ui/node_modules/cron-parser/lib/parser.js:39:25)     at /usr/lib/node_modules/crontab-ui/crontab.js:54:32     at newArguments.(anonymous function) (/usr/lib/node_modules/crontab-ui/node_modules/nedb/lib/executor.js:29:17)     at Cursor.execFn (/usr/lib/node_modules/crontab-ui/node_modules/nedb/lib/datastore.js:484:12)     at callback (/usr/lib/node_modules/crontab-ui/node_modules/nedb/lib/cursor.js:126:19)     at /usr/lib/node_modules/crontab-ui/node_modules/nedb/lib/cursor.js:193:12     at /usr/lib/node_modules/crontab-ui/node_modules/nedb/lib/datastore.js:329:14

Can you give me the output of crontab -l ?
On Mon, Dec 5, 2016, 6:42 PM IMRAN HASSAN<EMAIL_ADDRESS>wrote:

Facing this issue when i import existing cron from crontab
Error: Invalid characters, got value: # at Function._parseField
(/usr/lib/node_modules/crontab-ui/node_modules/cron-parser/lib/expression.js:155:11)
at parse
(/usr/lib/node_modules/crontab-ui/node_modules/cron-parser/lib/expression.js:644:36)
at Function.parse
(/usr/lib/node_modules/crontab-ui/node_modules/cron-parser/lib/expression.js:655:10)
at Function.parseExpression
(/usr/lib/node_modules/crontab-ui/node_modules/cron-parser/lib/parser.js:39:25)
at /usr/lib/node_modules/crontab-ui/crontab.js:54:32 at
newArguments.(anonymous function)
(/usr/lib/node_modules/crontab-ui/node_modules/nedb/lib/executor.js:29:17)
at Cursor.execFn
(/usr/lib/node_modules/crontab-ui/node_modules/nedb/lib/datastore.js:484:12)
at callback
(/usr/lib/node_modules/crontab-ui/node_modules/nedb/lib/cursor.js:126:19)
at /usr/lib/node_modules/crontab-ui/node_modules/nedb/lib/cursor.js:193:12
at
/usr/lib/node_modules/crontab-ui/node_modules/nedb/lib/datastore.js:329:14
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alseambusher/crontab-ui/issues/30, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABL1rqP3DL5Oup_Iq4SkB8zU-OU-ChBXks5rFA3IgaJpZM4LENN8
.

--
Thanks,
Suresh Alse

crontab
`# Edit this file to introduce tasks to be run by cron.

Each task to run has to be defined through a single line
indicating with different fields when the task will be run
and what command to run for the task

To define the time you can provide concrete values for
minute (m), hour (h), day of month (dom), month (mon),
and day of week (dow) or use '*' in these fields (for 'any').#
Notice that tasks will be started based on the cron's system
daemon's notion of time and timezones.

Output of the crontab jobs (including errors) is sent through
email to the user the crontab file belongs to (unless redirected).

For example, you can run a backup of all your user accounts
at 5 a.m every week with:
0 5 * * 1 tar -zcf /var/backups/home.tgz /home/

For more information see the manual pages of crontab(5) and cron(8)
m h  dom mon dow   command
*/5 * * * * /usr/bin/pypy/bin/pypy some.py
`

Looks like we have a bug while parsing comments in the crontab file. For now, remove the comment using crontab -e and try importing it.

Thank you for replying ...
removed but still same error..

So your crontab file just has the following line?
*/5 * * * * /usr/bin/pypy/bin/pypy some.py

Thanks for replying
yes exactly same as it is.

Are you sure that you are running crontab and crontab-ui as same user?
On Wed, Dec 7, 2016, 12:06 PM IMRAN HASSAN<EMAIL_ADDRESS>wrote:

Thanks for replying
yes exactly same as it is.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/alseambusher/crontab-ui/issues/30#issuecomment-265368162,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABL1ruVrS611mjjB2HGvnnPQfHbjb99Uks5rFlP5gaJpZM4LENN8
.

--
Thanks,
Suresh Alse

yes same i have solved by remove this module and then reinstall now its working fine : )


How I can add several anaconda channels dependency (priority) in the CI ?

Hi, I look in the code that with the variable $ANACONDA_CHANNELS I can add a channel to conda (but only one).

How I can add several anaconda channels dependency (priority) in the CI

ANACONDA_CHANNELS variable should be in the documentation (README) ?

@pradal @pfernique

You should modify directly your .appveyor.yaml or .travis.yml with
commands such as
install:
  - git clone https://github.com/OpenAlea/appveyor-ci.git appveyor-ci
  - cd appveyor-ci
  - call install.bat
  - conda config --add channels openalea/label/unstable
  - conda config --add channels msys2
  - conda config --add channels conda-forge
Le 05/12/2017 à 11:14, artzet-s a écrit :

Hi, I look in the code that with the variable $ANACONDA_CHANNELS I can
add a channel to conda (but only one).

How I can add several anaconda channels dependency (priority) in
the CI

ANACONDA_CHANNELS variable should be in the documentation (README) ?

@pradal https://github.com/pradal @pfernique
https://github.com/pfernique
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openalea/travis-ci/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGQC6_P0ipg7k4ix3OClTut0XukLPzaeks5s9Rd4gaJpZM4Q2CuA.

--
Frédéric Boudon
CIRAD, INRIA/CIRAD/INRA project-team Virtual Plants,  UMR AGAP
Bureau 02/225, CC 05018
Campus St Priest - Bâtiment 5,
860, rue St Priest
34095 MONTPELLIER Cedex 5, France
https://team.inria.fr/virtualplants/frederic-boudon/
tel   : (33) 4 67 14 97 97
install:

git clone https://github.com/OpenAlea/appveyor-ci.git appveyor-ci
cd appveyor-ci
call install.bat
conda config --add channels openalea/label/unstable
conda config --add channels msys2
conda config --add channels conda-forge

Thanks @fredboudon, I will use that to debug temporary my problem. But the problem still persist globally for openalea (but maybe is not the same question):

should we include the same dependencies channel via travis-ci for all the openalea package ? (Otherwise, if each package depend of different channel , good luck to make compatible all the openalea package ?)

Indeed, a list of recommended third party packages should be formalized
and the different modules of openalea should stick to it. However, it is
unclear to me if it should be just formalized as guidelines written on a
webpage or more hard coded into predefined dependency modules.
Le 05/12/2017 à 12:19, artzet-s a écrit :

Thanks @fredboudon https://github.com/fredboudon, I will use that to
debug temporary my problem. But the problem still persist globally for
openalea (but maybe is not the same question):

should we include the same dependencies channel via travis-ci for
all the openalea package ? (Otherwise, if each package depend of
different channel , good luck to make compatible all the openalea
package ?)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openalea/travis-ci/issues/1#issuecomment-349275006,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGQC65c3pcVEothTF98UlD3oyF4OAqlpks5s9SbKgaJpZM4Q2CuA.

--
Frédéric Boudon
CIRAD, INRIA/CIRAD/INRA project-team Virtual Plants,  UMR AGAP
Bureau 02/225, CC 05018
Campus St Priest - Bâtiment 5,
860, rue St Priest
34095 MONTPELLIER Cedex 5, France
https://team.inria.fr/virtualplants/frederic-boudon/
tel   : (33) 4 67 14 97 97

For me, the only possible solution is to build all the packages with exactly the same set of dependencies (i.e. channel). Otherelse, we will never reach ABI compatibility.


Handle SubscriptionBusyException in resetCursor api

Motivation
In method PersistentSubscription.resetCursor,  SubscriptionFencedException is thrown in several place, but not handled in PersistentTopicBase.
It caused the error message not very clear.   This fix tries to export SubscriptionBusyException in PersistentTopicBase for resetCursor, to make the error in reset api more clear.
Modifications

handle SubscriptionBusyException in PersistentTopicBase for resetCursor.
add tests for it.

close for dup


Ticking Entity

Describe the bug
Ticking Entity seems to be caused from Entity Blaze
To Reproduce
Steps to reproduce the behavior:

Kill Blaze
Repeat till crash.

Expected behavior
No crash.
System (please complete the following information):

Occultism Version: 1.13.0
OS: [e.g. Windows] Windows 10
Minecraft Version: 1.16.5
Modpack Link and Version, or list of mods: https://cdn.discordapp.com/attachments/883907557868072971/889254182043926528/A_Simplex_Life-2.5.1.zip

Additional context
https://gist.github.com/Traverse-Joe/ce46c1518a976ab113fd696ced07a7b4

Ticking entity in that crash report is a wilden hunter from ars nouveau

do you happen to have the greed effect from a dragon familiar when this is happening? looks like something is borked there xD is new
https://github.com/klikli-dev/occultism/blob/01b9cd51b8eaf868f901ea50efebfaeaecae13b6/src/main/java/com/github/klikli_dev/occultism/handlers/LootEventHandler.java#L76

currently i haven't even started Occultism yet so i shouldn't have the Dragon_Greed

I think its because when a player dies they drop exp and its not checking for EntityPlayer / instance of EntityPlayer but not sure we downgraded for the time being because it was a recuring crash every time someone died

I think its because when a player dies they drop exp and its not checking for EntityPlayer / instance of EntityPlayer but not sure we downgraded for the time being because it was a recuring crash every time someone died

That was it exactly - I wasn't aware killing player could be null in this event.


⚠ ClamAV has degraded performance

In 321c58c, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 472 ms

Resolved: ClamAV performance has improved in 2ba2e4c.


"Close All" button not working

the action is sent through the reducer (I see REMOVE_ALL_NOTIFICATIONS in the redux store as the last action) but the notifications remain in the store (and thus displayed).  I can dismiss them manually.  canDismiss is set to true on all my notifications.  There is no difference in redux store state when clicking close all.  Anyone else encountered this?
notifications displayed:

notifications in redux store:

last actions sent through redux store:

a sample of one of my notifications:

...

Seems the code could handle a force parameter, but the param is not being sent in
https://github.com/deep-c/react-redux-notify/blob/ea0c29e5e8554b3c3a5d8e0d4feb0437b02385dc/src/components/Notification/index.js#L113

the reducer is returning the same state:
https://github.com/deep-c/react-redux-notify/blob/ea0c29e5e8554b3c3a5d8e0d4feb0437b02385dc/src/modules/Notifications.js#L47

using with chrome on mac os-x10, with<EMAIL_ADDRESS>

Hi, thanks for creating the issue. That is quite strange and i haven't had a chance to investigate yet. I have however just quickly added the force close option above. See the branch force-close, ill merge that into master ASAP. When i get the time ill try and replicate the problem.


Fixes #2899 - Not getting proper relation between parent and child gl…

…ossary-terms
Describe your changes :

See #2899 for details

Type of change :

[x] Bug fix

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Test failure below is unreleated:
2022-02-23T07:41:14.9537936Z [ERROR]   TeamResourceTest.patch_deleteUserAndDefaultRoleFromTeam_200:192->EntityResourceTest.getEntityByName:1305 » HttpResponse


Installed Node.js doesn't work

I typed node -v and got v6.7.0 and then in VS Code, the error message shows Command 'leetcode.refreshExplorer' not found.

Hi @xxks-kkk , thank you for the feedback.
Currently the explorer do have some bugs now. But I cannot repro this one: Command 'leetcode.refreshExplorer' not found
Could you please provide more information about this?
like the version of vs code, repro steps, etc. 😄

I'm using Viisual Studio Code Version 1.20.1 (1.20.1). The steps I take roughly like:

I install plugin directly from the VS Code.
Once, I installed, I was notified that Node.js was not found from PATH.
So, I installed the node.js and run node -v with output v6.7.0.
Then I hit the refresh button of the plugin and the error message reported shows up.

I'm using Mac OSX 10.10.5.
Hope this description helps!

@xxks-kkk I see.
Currently, the extension will check the node runtime at the startup.
If user installed Node.js, then it will register related commands, otherwise, it just give a hint and return.
Could you please try to restart the VS Code (or maybe restart your Mac) and see if this issue still occur?

Hi @xxks-kkk Does the problem still occurs?

@xxks-kkk please feel free to reopen this issue if it still blocks you.

我也遇到了这个问题， 我尝试把插件卸载重装，之后就好了！


Moved headers to a separate include subdirectory.

I made a first attempt to use this repo as a submodule in cpp-netlib, and realised that I needed to separate the interface headers from those in the implementation (because the implementation uses boost, which conflicts with those that are need for the rest of cpp-netlib).
I've moved the relevant headers to an "include" subdirectory.

LGTM
Even if we don't end up using submodules I think this is still a good idea.


Protecting files?

Applying security to content appears to be pretty straightforward so far, but what about files (such as a list of files added using apostrophe-files)? Seems like I can protect the list, but if I copy the URL to one of the files (that should be protected with the permissions of that list), and paste it into another browser (that's not logged in), it's accessible? Is there a solution to protect the files themselves?

Not at present. There are ways this could be implemented, but there's a
cost involved: you can't just allow nginx or another frontend proxy to
directly serve them, which is a whole lot faster. Nevertheless this is an
area where additional features are possible.
On Mon, Mar 27, 2017 at 6:28 PM, David Webb<EMAIL_ADDRESS>wrote:

Applying security to content appears to be pretty straightforward so far,
but what about files (such as a list of files added using
apostrophe-files)? Seems like I can protect the list, but if I copy the URL
to one of the files (that should be protected with the permissions of that
list), and paste it into another browser (that's not logged in), it's
accessible? Is there a solution to protect the files themselves?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/punkave/apostrophe/issues/853, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fUpNo9O8VNg8cUxqo5TZB94g98UVks5rqDglgaJpZM4Mq6AF
.

--
*THOMAS BOUTELL, *SUPPORT LEAD
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Hmm - performance aside, any ideas on a possible implementation? I assume this would be done with middleware?

Should have added - I'd be also open to any combination of app code and nginx configs to accomplish this. I'll have complete control over the environment, so if everytime I need to set up a secure "folder" that files might be categorized under I have to add a simple rewrite rule to the nginx config, that shouldn't be an issue.

The current design of apostrophe-attachments is tough to reconcile with
this. Attachments have their own unique ID which isn't based on that of the
thing they are attached to. That would make it tough to write effective
logic for securing them from people who already know the URL. (The URLs are
already not guessable, thanks to the cuid module, so the main danger here
is someone who had access once and now should not.)
The basic strategy would be to move them to data/attachments, where the
static middleware / webserver can't see them, and then override
apos.attachments.url to generate URLs of the form:
/apostrophe-attachments/serve/:docId/:attachmentId
The doc in this question usually being an apostrophe-image.
An Express route would check whether the doc's permissions allow the
current user to view it and whether that attachmentId truly exists as a
property of the doc, then send the file.
As you can see, I'm all about it, but time is an issue. So this is
something we're likely to pursue when we have a client of our own or an
enterprise support client who shares this requirement. Feel free to be that
client:
http://apostrophecms.org/support/enterprise-support.html
Of course PRs are also welcome, with discussion around how they fit into
the architecture and achieve backwards compatibility.
On Mon, Mar 27, 2017 at 7:15 PM, David Webb<EMAIL_ADDRESS>wrote:

Should have added - I'd be also open to any combination of app code and
nginx configs to accomplish this. I'll have complete control over the
environment, so if everytime I need to set up a secure "folder" that files
might be categorized under I have to add a simple rewrite rule to the nginx
config, that shouldn't be an issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/punkave/apostrophe/issues/853#issuecomment-289614213,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fQknAemVh4F8aUot8g1s9T8YlkpIks5rqEMZgaJpZM4Mq6AF
.

--
*THOMAS BOUTELL, *SUPPORT LEAD
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

How was this addressed? I have attachments I need to protect.

Closed by accident when we addressed the more basic step of changing permissions (or, if you configure uploadfs that way, renaming the file to a cryptographically unguessable path) when attachments are only part of documents in the trash.
This issue is actually a separate feature request, as I understand it: prevent users from accessing an attachment unless they can access the document it is a part of according to Apostrophe permissions, i.e. "login required," view by "certain people" only, etc. In short. prevent sharing of the URL of the attachment.
Of course nothing prevents someone from hitting "save as" if they do have access, or screenshotting or copying and pasting, etc.
See my notes earlier in the thread about possible implementation strategies.
This could also be a good candidate for an Apostrophe enterprise support project as it is not a common requirement for our clients. Please do reach out to<EMAIL_ADDRESS>if you'd like to talk to us about funding development of the feature.

Now that attachments keep track of the live docIds that contain them, this can be done without changing how URLs are generated. Instead a query for permissions must be done on those docIds by middleware that intercepts express.static. If you get through for at least one of them, you can view the attachment.

Been overthinking the details... the correct implementation is to enable permissions for apostrophe-files pieces (already a thing developers can do), and then make sure attachments honor the view permissions in the following way:

attachments get a secureId property, random and unrelated to their _id
Each time an apostrophe-file piece gets saved...
The attachment gets told about the piece it’s in, perhaps not the first one (we already track this)
The attachment says “hey, do any of my docs have ‘public’ view permissions?”
If nope, attachment marks itself secure
attachments.url() calls all see that flag, generates URLs like /modules/apostrophe-attachments/secure?secureId=xxxx 100% of the time
when the request comes in, we look up the attachment, then look to see if the current user can view any of its docs (we just fetch them with a tiny projection, see if we get any back)
if so send file
if not 404.

In this scheme we don't even have to move the media out of the main uploadfs web-accessible folder because its URL there is never publicized...
Well. It could be. The _id of the attachment could leak in a widget data attribute.
A typical file or image widget wouldn't leak it though because they are typically pieces widgets, or otherwise join to apostrophe-file or apostrophe-image, which means the id of the attachment itself will not be in their public JSON.
So a further enhancement would be to move the file to storage that is truly web-inaccessible, however this requires a separate instance of uploadfs and is probably out of scope for now. With typical widgets and/or responsible use of the playerData option to other widgets, the risk can be eliminated as needed.

Necessary changes:

The attachment must in fact be renamed in uploadfs. Otherwise existing URLs from before the file was locked down will still work!
This must also be done anytime the set of non-public docs associated with the file changes. Otherwise, see above.

This will have a performance impact, obviously. But hey, people don't have to secure files if they don't want to.

A simple implementation is now in this module, which is awaiting code review before I hit npm publish; may be out by the time you read this:
https://github.com/apostrophecms/apostrophe-secure-attachments
For this implementation, no attempt is made to serve the public files faster via static webservers outside of node, etc. This allows for fairly straightforward middleware to check the permissions before the files are served. It also turns out that express.sendFile is pretty fast, and humungousness of traffic + the need for an intranet are usually not correlated on the same site anyway.
To ease migration and avoid "but... but... I didn't read and now it's insecure!" scenarios, there is a new and distinct default URL path for uploads when this module is in play; it's very easy to move content from the old public/uploads folder to the new data/secure-uploads one, and both uploads and data are already persistent shared folders between deployments. So this is about as painless as it gets.
The use of this module implies a requirement to use local storage rather than s3/azure/etc.


Add missing tests for the small simple functions of the models

Codecov Report

Merging #112 into master will increase coverage by 2.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #112      +/-   ##
==========================================
+ Coverage   75.45%   77.63%   +2.18%
==========================================
  Files          23       23
  Lines        1055     1055
==========================================
+ Hits          796      819      +23
+ Misses        201      177      -24
- Partials       58       59       +1

Impacted Files
Coverage Δ

internal/model/resource.go
100.00% <0.00%> (+55.00%)
:arrow_up:

internal/model/report.go
80.00% <0.00%> (+80.00%)
:arrow_up:

internal/model/app.go
100.00% <0.00%> (+100.00%)
:arrow_up:

Continue to review full report at Codecov.

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


How can I return just one picture ?

Hi, my name is Luciano, and I try to use this class, but I need permit that user select just one picture. Is this possible ?
is possible configure theme to the FacebookPhotoPickerActivity ?
tnx advanced

Hi Luciano,
Thanks for your message. The current up-to-date version of the Facebook
Photo Picker is here:
https://github.com/OceanLabs/Android-Print-SDK/tree/master/FacebookPhotoPicker
There is also a library on Maven that you can use. The up-to-date
version allows you to specify the maximum number of selectable images:
FacebookPhotoPicker:
startPhotoPickerForResult( Fragment fragment, int maxImageCount, int
activityRequestCode )
You can set this to 1.
The theme is set in styles.xml:

And the image picker theme is as follows:

You can redefine the themes to suit your requirements.
Hope this helps
On 13/11/16 02:08, lucksjb wrote:

Hi, my name is Luciano, and I try to use this class, but I need permit
that user select just one picture. Is this possible ?
is possible configure theme to the FacebookPhotoPickerActivity ?
tnx advanced
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/OceanLabs/FacebookPhotoPicker-Android/issues/6,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMWb-lRjKeg5RYp4PFqcggeIRRTlGnRZks5q9nEOgaJpZM4KwlbQ.


Intl.NumberFormat does not model required properties when style is set to currency

⚙ Compilation target
ESNext
⚙ Library
ESNext
Missing / Incorrect Definition
According to the MDN docs, when style is set to currency, then the currency property must be provided.
This could be achieved through a discriminating union on the style property e.g.
interface NumberFormatParamsStyleDecimal {
    style: "decimal"
}

interface NumberFormatParamsStyleCurrency {
    style: "currency"
    currency: string
}

type NumberFormatParams = NumberFormatParamsStyleDecimal | NumberFormatParamsStyleCurrency;

const numberFormat = (params: NumberFormatParams) => {};

numberFormat({
    style: "decimal",
})

numberFormat({
    style: "currency",
    currency: "USD"
})

Sample Code
This fails at runtime, but does not fail compilation:

new Intl.NumberFormat("en", {
    style: "currency",
    currency: undefined
});

VM159:1 Uncaught TypeError: Currency code is required with currency style.
    at new NumberFormat (<anonymous>)
    at <anonymous>:1:1

Documentation Link
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/NumberFormat/NumberFormat#style

This isn't just true of currency. Passing { style: 'unit' }, for example, will raise a TypeError if the unit property is not also provided, and there may be analogous additions to the spec in the future.
It's difficult to see how such a union could be implemented that is straightforward to parse, doesn't involve significant duplication, and is extensible in the case of one or more "has-dependent-required-properties" style values being added in the future. The shape of the resolved options object also changes based on the style provided; should that also be typed variably for consistency?
I feel like simplicity is probably the best policy here. There are myriad other ways to raise TypeErrors from Intl constructors that don't violate the type definitions, but my personal feeling is that it's not the role of lib.d.ts to necessarily safeguard against them all.


Confusing instructions for creation of MessageBody Class

In Step 4 of Update the Module with Custom Code , the user is instructed to "Add the MessageBody class" and is then instructed to paste in code for a class named FileEvent.  This is a bit confusing, perhaps it could be worded a bit differently?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ded0d4cd-5b54-69ab-75fd-027b7c9ce424
Version Independent ID: 1f5aa92e-050a-74b9-7c77-a527e32ab7c4
Content: C# IoT Edge module for Azure Data Box Edge
Content Source: articles/databox-online/data-box-edge-create-iot-edge-module.md
Service: databox
Sub-service: edge
GitHub Login: @alkohli
Microsoft Alias: alkohli

@toolboc Thank you for the valuable feedback,we are investigating the issue.

@toolboc Thanks for your feedback! The text is reworded and the changes are merged.  The content should go live after 3 pm.
Please close this. #please-close


no valid aps-environment entitlement string found for application

Hello everyone, i have a problem with push notification on iOS 10... it throws on register device id this error "no valid aps-environment entitlement string found for application" ... I have project in visual studio on windows and use remotebuild on mac to build for iOS... i try everything regenerate all certificates, enabling push notifications disabling push notifications, auto signing, manual signing but still it throws this error.. can anoyne help with that please?. Thanks a lot for any kind of help or advice.

I have the same issue.
https://issues.apache.org/jira/browse/CB-11854

One of our users experiences the same on iOS 8.4.1
Looks close to the isssue
http://stackoverflow.com/questions/27324020/no-valid-aps-environment-entitlement-string-found-for-application-on-app-store
Notifications worked and worked pretty goof - except they never worke on iOs 10, but other iOs devices and Android were just fine. I thoght that's an issue with iOs 10, but now, when we have the same case with iOs 8 I\m trying to figure out what shoul I do tio update the app.
Any suggestions?

Hello, any update on this ?

It was fixed in newer versions of CLI. I've just compiled new app at build.phonegap.com and had no more troubles.

Not sure if this is the appropriate place for this comment, but I'll also point out that for these notifications to work on iOS, you need to run the project from XCode, not using the Ionic CLI. Ran into that problem myself.

It took me a while to figure out what causing no valid aps-environment entitlement error.
The problem was that there were two .entitlements files in XCode project. One is platform/ios/<PROJECT_NAME>/<PROJECT_NAME>.entitlements and another one is platform/ios/<PROJECT_NAME>/Resources/<PROJECT_NAME>.entitlements and the appropriate entitlements is present in the former file only while XCode is configured to use the latter. I updated the latter file manually and everything started to work.
Another option to fix this is to change which .entitlements is used by XCode. This can be done in  Build Settings --> Signing --> Code Signing Entitlements config param.
I'm not sure why there are two .entitlements files in XCode. Could it be that the push plugin created the second one? And what is the correct location for the .entitlements file?

It took me a while to figure out what causing no valid aps-environment entitlement error.
The problem was that there were two .entitlements files in XCode project. One is platform/ios/<PROJECT_NAME>/<PROJECT_NAME>.entitlements and another one is platform/ios/<PROJECT_NAME>/Resources/<PROJECT_NAME>.entitlements and the appropriate entitlements is present in the former file only while XCode is configured to use the latter. I updated the latter file manually and everything started to work.
Another option to fix this is to change which .entitlements is used by XCode. This can be done in Build Settings --> Signing --> Code Signing Entitlements config param.
I'm not sure why there are two .entitlements files in XCode and one of them has APS Environmentkye and the other one does not. Could it be that the push plugin created the second one? And what is the correct location for the .entitlements file?

Oh man!!!! This is gold right here. I did everything the right way, followed every tutorial, read every doc and could not get it done. This was the problem. I created a github account just for leaving this reply and to thank you for this!!

I encountered the same issue. In the RunnerProfile.entitlements file, there was aps-environment development, but it was missing in the Runner.entitlements. I removed it from the first and added it to the second, and it worked.


New package: ConstructionBaseExtras v0.1.0

Registering package: ConstructionBaseExtras
Repository: https://github.com/JuliaObjects/ConstructionBaseExtras.jl
Created by: @rafaqz
Version: v0.1.0
Commit: 51f29aff6fc2fb2fc7a47380d1342b3c413b7bec
Reviewed by: @rafaqz
Reference: https://github.com/JuliaObjects/ConstructionBaseExtras.jl/commit/51f29aff6fc2fb2fc7a47380d1342b3c413b7bec#commitcomment-50414492

The test error seems to be with SPIRV_LLVM_Translator_jll... I'm not sure what that means [noblock]


view of the arms analysis release 001 is missing

Clicking on https://data.arms-mbon.org/analysis_release_001/latest/#./ and navigating to the actual "dataset" (the repo contents listing) takes me to a blank page.

issue raised in wrong repo


Remove August 2013 limit for CCG analyse queries

The limit should have been April 2013 (as this is when CCGs were formed)
but April and August are easy to mix up!
We can remove the limit completeyl now as we only hold five years of
data, and we have now imported data from April 2018.
Fixes #920.

Wait for @richiecroker to respond to query on #920 before merging.

Nothing special about August 2013!  April 2013 on the other hand was the first month of CCGS.


Implement code generation from OpenAPI spec

Description
Brings back the old ApiGenerator and refactors it to instead base off an OpenAPI spec
Issues Resolved
Closes #193
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

What was changed in API generator? Can I skip first commit for reviewing?

What was changed in API generator? Can I skip first commit for reviewing?

First commit is just a direct restore, so can be skipped. Main changes in ApiGenerator were general cleanup and refactoring plus the changes to the api spec parsing. There'll need to be a little more change to make it download the spec from the correct place

Can see an example of re-generating the tasks namespace here: https://github.com/Xtansia/opensearch-net/compare/feat/code-generation...Xtansia:opensearch-net:generation/tasks

I've decided to break this into smaller PRs with fewer changes to make it easier to review. Starting with just the revert commit in #222


[SDP-1180]  Send Payments to Circle - 🚨 Draft Do Not Merge

👀 Reviewed ?

[x] 🛠 [refactor] Introduce Circle Service to simplify Circle-related operations
[x] 🛠 [refactor] separate TSS specific logic for submitting

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr331.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr331.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr331.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr331.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr331.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr331.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr331.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr331.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr331.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr331.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr331.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr331.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr331.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr331.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr331.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr331.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr331.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr331.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr331.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr331.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr331.previews.kube001.services.stellar-ops.com

Something went wrong with PR preview build please check

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr331.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr331.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr331.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr331.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr331.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr331.previews.kube001.services.stellar-ops.com


Lazy load lang defaults

rebased

rebased

I noticed that for me on macOS (at least in make servewallet) the native locale endpoint returns an empty string.
 % locale
LANG=""
LC_COLLATE="C"
LC_CTYPE="UTF-8"
LC_MESSAGES="C"
LC_MONETARY="C"
LC_NUMERIC="C"
LC_TIME="C"
LC_ALL=

% defaults read -g AppleLanguages
(
    "en-US",
    "de-CH"
)

% defaults read -g AppleLocale
en_US@rg=chzzzz

added a commit to "fix" the backend by trying defaults read -g AppleLocale or just falling back to en_US, @Beerosagos could you take a look at my last commit?

Wouldn't it be also good to have the frontend be able to handle invalid or missing locale info?

I does already to a certain degree, but I need to test a bit more, what happens if there is really weird invalid data I believe worst case it just fallsback to EN.
https://github.com/BitBoxSwiss/bitbox-wallet-app/blob/9a74ab410863f1a01ae8effc82a8040eab340b66/frontends/web/src/i18n/utils.ts#L17-L25

Not sure what the issue was exactly

If no userLang has been set the app tries to fallback to the navite lang from the OS, if that doesn't work it fallsback to 'en'.
I found and fixed another issue, that missing keys didn't fallback to en (fixed with last commit).
I'll test a bit more with invalid data later...


Upload UI-mockup of intended final product

PR related to issues #14 and #1.

Codecov Report

Merging #15 (dc525d2) into master (28e931d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #15   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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

Looks Good! I think it can be merged once the TA responds to the enquiry.


Add a route name field to each route in route.Route list (HTTP route)

Issue Template
Title:  Add a route name field to each route in route.Route list (HTTP route)
Description:
This is a feature request for adding a name field to the http routes. It would be enabled by default but if the name is not specified in the config, then it won't be enabled. This feature has helped us a lot with logging and debugging at eBay. One of the use cases is when there are 100s of routes and we want to check which route has been matched; the name makes it easier to identify. We wanted to see if this will be useful for the community. We have a PR that can be made for this case. If there is interest, I can also make a subsequent PR adding the route name field to access logs. Here is an example of how the config JSON for the route.Route object would look like:

@drao9 yes I think this would be very useful to a lot of people. (This has come up at Lyft also.) It could also be the basis for allowing optional per-route stats also.

I was out on vacation the past few weeks - I am working on this now and will have a PR in a couple of days.


Bump etcd to v3.5.7

There's a new version of etcd out that fixes some data corruption issues. We should use it.

Validated using commit id 027cc187ce9f21157b8d37d62e67ee1c42968b4b on master branch
$ k3s -v
k3s version v1.26.3+k3s-027cc187 (027cc187)
go version go1.19.7

{"level":"info","ts":"2023-04-07T19:34:17.186Z","caller":"embed/etcd.go:306","msg":"starting an etcd server","etcd-version":"3.5.7"...


Release Google.Cloud.TextToSpeech.V1Beta1 version 2.0.0-beta11

Changes in this release:
New features

Add multi-speaker markup, which allows generating dialogue between multiple speakers (commit 9c3527e)

Created release for Google.Cloud.TextToSpeech.V1Beta1-2.0.0-beta11


chore(main): release 0.4.3

:robot: I have created a release beep boop
0.4.3 (2022-06-18)
Bug Fixes

deps: update dependency colorette to v2.0.19 (281242c)

Chores

deps: update dependency @babel/core to v7.18.5 (601783a)
deps: update dependency @types/jest to v28.1.2 (c072752)
deps: update dependency @types/semver to v7.3.10 (841925a)
deps: update dependency lint-staged to v13.0.1 (7e3c509)
deps: update dependency lint-staged to v13.0.2 (e9c715e)
deps: update dependency prettier to v2.7.1 (1cc780d)
deps: update dependency typescript to v4.7.4 (d10a5d5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/smarlhens/npm-check-engines/releases/tag/v0.4.3 :sunflower:


[3pt]  - Add tests for items counter - Student A

Make sure that function for the items counter is covered with unit tests using Jest. Make sure that you have tests for all edge cases.

https://github.com/Sajeelzafar/JavaScript-Capstone/pull/36


add persistent chat example for js

What this PR does 📖
adds a js example for conversation with persistence enabled (minimal_basic). (I used this to help debug conversation desync issue on browser refresh.)
Which issue(s) this PR fixes 🔨
Special notes for reviewers 🗒
Additional comments 🎤

LGTM. Maybe we should add a note that the persistent side require indexeddb and/or localstorage and that those storages should be cleared if they wish for things to be refreshed

Good point 👍  I took the opportunity to add descriptions to all js examples


Added software from Macquarie University

Added two repositories we have been working on recently
(also fixed indentation error...)

Thanks!
Great to see your contributions to the BIDS ecosystem. One thing that might help to form collaborations is updating your GitHub profile with a real name and affiliation.


no CommonJS dependencies if run in browser

I'm wandering if it wouldn't be better to reduce CommonJS dependencies, i.e. require('xmlhttprequest') and require('underscore') when running in a browser.
I'm using this awesome library from within TiddlyWiki, which runs in a browser and also uses kind of CommonJS requires.
P.S. sorry for my first pull request - it was build against a different fork, which used fewer underscore methods.

I'm not sure of the benefit of this and I worry it might break things. Could you please rebase this? I'll then run the current test suite against it. Thanks!

This is irrelevant due to Underscore being removed. Closing as duplicate.


Serverless Framework v3: new design & how to integrate

Hi 👋 , Serverless Framework core team here!
We're working on the v3 beta version right now. The v3 will include a redesigned CLI output:

You can try the v3 beta with: npm i serverless@pre-3
To help plugins integrate with that new design (and build a great user experience), we've added new APIs for plugins:

Video introduction (Youtube)
Documentation of the new API

Any feedback is appreciated! If you want to update your plugin to use these new APIs, feel free to ping me if you have any questions!
If you don't have time to update the plugin, let me know, we can send a pull request!

I have the changes implemented but I won't release them until v3 is out.

@juanjoDiaz got it, thank you for the update.

Hello everyone!
@juanjoDiaz Do you have changes for the JSON schema of the plugin to adhere to the strict mode that was added in the AJV v7, as SLS v3 updated it from the v6 to v8?
If not, I will be glad to make a PR.

PRs are always welcome as long as they are clean, tested and maintain current test coverage.
The v3 branch is https://github.com/juanjoDiaz/serverless-plugin-warmup/tree/serverless_v3

Thank you for the reply! I am going to make a PR in a couple of nearest days.

Hey @batovpasha - are you planning to address it in the coming days? If not, I'm happy to pick it up as we're aiming for a v3 release of Framework very soon

@pgrzesik Hello! Yes, I will make a PR for today/tomorrow.

Awesome, thank you @batovpasha


Create Test.md

for testing purpose
to delete after labeler act or fail

For labeler testing purposes, it is successful.

Okay, thanks.


collapse on other dataset

firstly,it is a great job to do ddp(distributed_data_paralel) train on huge dataset(other code is on cifar10), i could get the result you published.
however ,when i train the simsiam on other dataset(not published) the simsiam model tend to collapse, i wonder whether it is the problem of not suitable lr or  not including the procedure of warm up, i  really hope Dr.tao could share your suggestions.

Hi, I may not be able to tell the problem by that. Could you briefly tell me about the dataset and what changes you made?


Putting try around get url for unreachables routes

Description
Adding a try /except around is_url_available to be able to catch when a url is unavailable and report it properly

Can one of the admins verify this patch?


[Feat}: The Navbar Toggle Icon Button is not changing in mobile devices

I have changed the Navbar toggle icon button in the mobile devices
Issue No:- #123
https://github.com/Akshatchaube01/TimeWarp/assets/121508628/f25e7346-e8c1-4006-92e6-36212841554f
If it's not good, please assign me again I will do it better
Please add labels in this issue if you like my work

Please pull the recent changes and then commit


Allow schema to be a callable

This would allow us to dynamically return the choices such as objects from the database.
This would also allow us to return a schema based on user permissions.

This feature has been added in v2.1.0.

I have a model like this
class Model(models.Model):         json_schema = models.JSONField(default=dict) # editable by the user     json_value = JSONField(schema='whatever is in json_schema') # django_jsonform's field
how do I pass in a callable to schema that can actually fetch the json_schema value of the same row? Is it possible? Essentially I need the entire schema to be user editable.
Sorry if this is a basic question, thank you in advance

You'll need to pass the current model instance to the callable.
It's documented here: Accessing model instance in callable schema.
It's a bit of a manual work as you'll have to override the admin's model form.
I would have loved to provide an automatic way to pass the current model instance, but that's not possible due to the way Django initializes model fields and widgets.
See also: #8.


Add configuration UI

Add config UI to the plugin to specify:

Priority of the track lookup method (external AppleScript/AppleScript ScriptingEngine)
Name of the template variable
Value pattern: Define how the @soundtrack content is built

Fixed.


update(.github): enable CI checks for Falco 0.35.1

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind feature

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area rules

/area registry

/area build

/area documentation

Proposed rule maturity level

Uncomment one (or more) /area <> lines (only for PRs that add or modify rules):

/area maturity-stable

/area maturity-incubating

/area maturity-sandbox

/area maturity-deprecated

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [LucaGuerra,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [LucaGuerra,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold
Let's wait until we complete the Falco 0.36 release (just to avoid any interference). Then, feel free to unblock this.
cc @LucaGuerra

Is there anything else needed for this one? @leogr @LucaGuerra

/hold cancel


xref issue: MONDO_0100096 (reported via email)

I notice that
https://www.ebi.ac.uk/ols/ontologies/efo/terms?short_form=MONDO_0100096
gives a cross-reference to MESH:C000657245 which was assigned by the NIH Office of Rare Diseases Research when it was an emerging disease.
Because this is, unfortunately, not a rare disease now, NLM has moved COVID-19 to this MeSH Term:
https://meshb.nlm.nih.gov/record/ui?ui=D000086382

It looks like this has been updated- MESH:C000657245  is equivalentObsolete.


fix DeprecationWarning of flask app.json_encoder

The json_encoder was set to deprecated with Flask 2.2 and removed with Flask 2.3. The new json_provider_class is introduced with Flask 2.2. This means with this commit the minimum used Flask version has to be 2.2.
This PR introduced this warning.
The minimum version of Flask of 2.2 could be a problem. As i found only invenio-base sets the minimal and maximal versions for flask. If we apply this PR we have to increase the minimal version there to Flask>=2.2.
NOTE:
The local JSONEncoder should derive from json.JSONEncoder to have the same behavior as existed before (e.g. the existence of the encode method)

For reviewers: this is ready but @utnapischtim prefers to apply changes to other modules first and test it.
@utnapischtim can you please notify when ready to be merged? Thanks!

My read is that this PR should be reviewed/merged first, and then https://github.com/inveniosoftware/invenio-accounts/pull/434,  https://github.com/inveniosoftware/flask-security-fork/pull/50, and https://github.com/inveniosoftware/invenio-theme/pull/344 can be tested. Is that accurate @utnapischtim?

My read is that this PR should be reviewed/merged first, and then inveniosoftware/invenio-accounts#434, inveniosoftware/flask-security-fork#50, and inveniosoftware/invenio-theme#344 can be tested. Is that accurate @utnapischtim?

Yes indeed, the last chat we had is that @utnapischtim wanted to test locally these changes with other modules before merging.

yes @tmorrell this is correct. there will be around 20 PR's which are handling the flask-babelex -> flask-babel migration. maybe a little bit more. I am going throw all packages to check if this PR needs more work. I will add to every PR which package needs a release before.


New Admin accounts with ADMIN authorization having trouble accessing forms.

I created a new Admin account and selected the admin authorizations. Then I logged in as this new user but I am unable to access most pages on the Admin dashboard. For example I am able to view Category List but not Product List.

The backend throwing the following exception.
org.h2.jdbc.JdbcSQLSyntaxErrorException: Table "MERCHANT_STORE" not found; SQL statement


Add statistics API.

Fixes #330 - Implements the statistics API as per https://developer.github.com/v3/repos/statistics/.
Adds the following to the GHRepository class:

getContributorStats()
getCommitActivity()
getCodeFrequency()
getParticipation()
getPunchCard()

@bitwiseman Thanks for your comment. I apologize for taking so long to respond. (I've updated my Github notification settings now.)
Do you mean unit tests?

@martinvanzijl
My turn to be slow to respond.  :)
Yes, I unit tests.  We now have the start of WireMock based framework where you can make sure the values from the server are being processed correctly.

@bitwiseman I've added some unit tests (and pushed a fix). So far the tests just check that the methods run successfully. I'll aim to add more detailed tests to see that the statistics returned are correct.
Should I be using WireMock for these?

@bitwiseman I've added more detail to the unit tests. Could you please review?
GitHub caches statistics, and if they are out of date it will return "202" while it refreshes them. I return "null" in that case, but I could instead throw an exception, or simply call sleep() and try again in five seconds within the method itself. What do you think?

@martinvanzijl
Please use the WireMock testing utility, similar to the GistTest or PullRequestTest. This will let us run tests in CI.
I've also add you as a maintainer on
https://github.com/github-api-test-org/github-api .   Please point your tests at that repository if possible, and if not let's discuss if there is anything else you need.

@martinvanzijl

GitHub caches statistics, and if they are out of date it will return "202" while it refreshes them. I return "null" in that case, but I could instead throw an exception, or simply call sleep() and try again in five seconds within the method itself. What do you think?

I'm not sure, I'm not familiar with this part of the code yet.

@bitwiseman Thanks, makes sense. I'll try to make the changes as suggested.

I've uploaded my progress so far. I moved the code to a new class GHRepositoryStatistics as suggested, and used the Wiremock framework.
There is more I would like to do:

getContributorStats() has the "wait-till-ready" loop in the method itself, which can be turned off with a boolean parameter. Do the same for the other "get..." methods.
Return classes with accessor methods (e.g. ContributorStats.getAuthor("name")) instead of just the PagedIterables or Lists.
Cache the statistics and add a "refresh()" method to force a reload of data.


Error in provided Colab file

Hello,
When I run the Colab file that you provided, I get this error:
The tokenizer class you load from this checkpoint is not the same type as the class this function is called from. It may result in unexpected tokenization.  The tokenizer class you load from this checkpoint is 'T5Tokenizer'.  The class this function is called from is 'VLT5TokenizerFast'.
TypeError**                                 Traceback (most recent call last) <ipython-input-30-03619949772e> in <module>() ----> 1 tokenizer = VLT5TokenizerFast.from_pretrained('t5-base')
4 frames /usr/local/lib/python3.7/dist-packages/transformers/tokenization_utils.py in __init__(self, **kwargs)
   338      339     def __init__(self, **kwargs): --> 340         super().__init__(**kwargs)     341      342         # Added tokens - We store this for both slow and fast tokenizers
TypeError: super(type, obj): obj must be an instance or subtype of type
For the line:

tokenizer = VLT5TokenizerFast.from_pretrained('t5-base')

How can I solve this? Thanks already!

I changed it to:

tokenizer = VLT5Tokenizer.from_pretrained('t5-base')

Now i get the error:
TypeError                                 Traceback (most recent call last) <ipython-input-35-c98008ad6f6c> in <module>()       1 for question in questions: ----> 2     input_ids = tokenizer(question, return_tensors='pt', padding=True).input_ids       3     batch = {}       4     batch['input_ids'] = input_ids       5     batch['vis_feats'] = features
TypeError: 'NoneType' object is not callable


fix: add ajamru

resolves #1470
Checklist for profile addition

[ ] Name the PR fix: add <your-username>.
[ ] You've added file named your-username.json in the profiles directory.
[ ] You've linked the PR to the correct issue.

Other additions/changes

[ ] Changes made in this PR are mentioned below
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.
[x] I've read the contributing guidelines

This PR adds/fixes

Notes

Changed as suggested please verify


Handle too many requests response

Fixes https://github.com/rauchg/slackin/issues/110.

@rauchg I didn't know that GH allows PR approvals without write access, sorry for the noise.

When this will be released? It is worth of minor version, I thing.

We just hit this today with our implementation. Is this being merged any time soon? I'd rather not have to fork the project. Thanks for your time.

This PR resolved the issue #110

@rauchg any ETA on when this will be released?

Bump. This is a basic issue that is not visible to users but potentially affects end-users.


CSG primitives do not work with complex meshes

The current CSG objects were developed assuming only two intersections per primitive. These object do no appear to work with meshes involving more intersections. Explore and fix.

Turns out I did make the CSG primitives work with infinite intersections... so the problem must lie somewhere else. Need to develop a test case.


🐛 [BUG] - Resources gets created when plugin is disabled

Priority
(Medium) I'm annoyed but I'll live
Description
The resources defined in the PluginDefinition helm chart gets created in the remote cluster even when the plugin.spec.disabled is set to true.
Reproduction steps
1. Add a new plugin with `spec.disabled = true`
2. Monitor if the resources are being created in the remote cluster
3. Ideally, the resources should not be created in the remote cluster

Manifests
apiVersion: greenhouse.sap/v1alpha1
kind: Plugin
metadata:
  creationTimestamp: "2024-06-21T13:19:34Z"
  finalizers:
    - greenhouse.sap/helm
  generation: 6
  labels:
    greenhouse.sap/cluster: obs-eu-de-1
    greenhouse.sap/plugindefinition: kube-monitoring
    greenhouse.sap/pluginpreset: kube-monitoring
  name: kube-monitoring-obs-eu-de-1
  namespace: test-org
spec:
  clusterName: remote-cluster-1-control-plane
  disabled: true
  displayName: kube monitoring obs eu de 1
  optionValues:
    - name: alerts.enabled
      value: false
    - name: kubeMonitoring.defaultRules.create
      value: false
    - name: kubeMonitoring.prometheus.ingress.enabled
      value: false
    - name: kubeMonitoring.prometheus.prometheusSpec.externalLabels
      value:
        cluster: "{{ .Values.global.greenhouse.clusterName }}"
        cluster_type: '{{ (split "-" .Values.global.greenhouse.clusterName)._0 }}'
        organization: ccloud
        region: "{{ .Values.global.greenhouse.clusterName | trunc -7 }}"
    - name: kubeMonitoring.prometheus.prometheusSpec.externalUrl
      value:
        https://kube-monitoring-{{ .Values.global.greenhouse.clusterName }}-prometheus--ccloud--{{
        .Values.global.greenhouse.clusterName }}.ccloud.greenhouse-qa.eu-nl-1.cloud.sap
    - name: kubeMonitoring.prometheus.prometheusSpec.retention
      value: 30d
    - name: kubeMonitoring.prometheus.prometheusSpec.storageSpec.volumeClaimTemplate.spec.resources.requests.storage
      value: 100Gi
    - name: kubeMonitoring.prometheus.service.additionalPorts
      value:
        - name: grpc
          port: 10901
          protocol: TCP
          targetPort: grpc
    - name: global.greenhouse.clusterNames
      value:
        - obs-eu-de-1
        - obs-eu-nl-1
    - name: global.greenhouse.teamNames
      value:
        - baremetal
        - compute-storage-api
        - containers
        - core
        - email
        - foundation
        - identity
        - image
        - infraautomation
        - network-api
        - observability
        - solutiondevelopment
        - storage
    - name: global.greenhouse.organizationName
      value: ccloud
    - name: global.greenhouse.clusterName
      value: obs-eu-de-1
  pluginDefinition: kube-monitoring
  releaseNamespace: kube-monitoring

Screenshots
![DESCRIPTION](LINK.png)

Good catch.
Was running in this too, when testing this scenario:

Disabling the plugin in the pluginpreset
Enabling them with clusterOverrides for certain clusters

Not sure if the latter is even possible, but the disabled plugin shows as disabled correctly, however all resources are deployed.

I found the original commit that introduced the disabled field almost 1.5 years ago. But the functionality behind the field was never implemented.
There are two parts to this:

Disabling the Helm Chart leads to helm uninstall and skip reconciliation
Skip the UI for this Plugin (if the PluginDefinition contains a uiApplication)

I am not sure we should actually implement this, or deprecate and remove the field. It looks like there was no real use-case for it since the field was added 1,5 years ago.
The scenario which @viennaa is describing sounds like a feature for the PluginPreset, as to use a 'clusterSelector' but also provide a blocklist to remove clusters by name?

I am not sure we should actually implement this, or deprecate and remove the field. It looks like there was no real use-case for it since the field was added 1,5 years ago.
The scenario which @viennaa is describing sounds like a feature for the PluginPreset, as to use a 'clusterSelector' but also provide a blocklist to remove clusters by name?

okay, cool. Shall we discuss this topic in the next Greenhouse weekly meeting?
@IvoGoman

This topic came up again during the UI workshop and it has been formalised as a ClusterSelector in #724. This should address the need to exclude resources for the PluginPreset.


[Example]Add XGBoost example

Coverage increased (+12.5%) to 63.445% when pulling 4b91f233e468af42bdf18e91ade8c457b4b8e3b3 on zhaonan/xgboost into 212e3edf9a1b561e787b02ddf85b4426e67b336d on master.

Coverage increased (+12.5%) to 63.445% when pulling 4b91f233e468af42bdf18e91ade8c457b4b8e3b3 on zhaonan/xgboost into 212e3edf9a1b561e787b02ddf85b4426e67b336d on master.

Coverage increased (+12.5%) to 63.445% when pulling 4b91f233e468af42bdf18e91ade8c457b4b8e3b3 on zhaonan/xgboost into 212e3edf9a1b561e787b02ddf85b4426e67b336d on master.


PingFed engine curl initcontainer needs resources block

The PingFederate engine curl initcontainer should have a resources block to avoid pod scheduling issues in clusters/namespaces that must require all pod resources be set

Good feedback.  Will take a look at this.

Closed with release 0.3.2


Check if localizedTitle and localizedDescription are not nil

Sometimes the localizedTitle and localizedDescription can be nil, so ignore products if they are nil.

This situation had been reported here:
http://stackoverflow.com/questions/23657046/itunnes-skproduct-localizedtitle-is-nil

@genki Thanks for the PR. Going by the answer on SO, seems like adding new language to each product solves problem.
Regardless, i feel it's best to let your app dictate the behavior in case of empty/nil localizedTitle etc rather than handling it in this module. You can check if product. localizedTitle == '' in your javascript code.


meetup: use standard test naming format

Fixes #271

Closed by commit https://github.com/exercism/xjavascript/commit/e8b168490b1f3610aaee1b0670622a2623d9532c


Build service image 'frameworklauncher' failed

Step to reproduce:
sudo docker build -t frameworklauncher ./src/frameworklauncher/
Error logs:

2018-06-12 08:10:31,352 [INFO] - paiLibrary.common.docker_handler : Begin to execute the command: docker build -t frameworklauncher ./src/frameworklauncher/
Sending build context to Docker daemon   1.31MB
Step 1/9 : FROM hadoop-run
---> 68ccfe23ba58
Step 2/9 : RUN apt-get -y update &&     apt-get -y install maven
---> Running in 752370bfa242
Get:1 http://security.ubuntu.com/ubuntu xenial-security InRelease [107 kB]
Hit:2 http://archive.ubuntu.com/ubuntu xenial InRelease
Get:3 http://archive.ubuntu.com/ubuntu xenial-updates InRelease [109 kB]
Get:4 http://archive.ubuntu.com/ubuntu xenial-backports InRelease [107 kB]
Get:5 http://security.ubuntu.com/ubuntu xenial-security/universe Sources [81.2 kB]
Get:6 http://archive.ubuntu.com/ubuntu xenial-updates/universe Sources [256 kB]
Get:7 http://archive.ubuntu.com/ubuntu xenial-updates/main amd64 Packages [1024 kB]
Get:8 http://security.ubuntu.com/ubuntu xenial-security/main amd64 Packages [650 kB]
Get:9 http://archive.ubuntu.com/ubuntu xenial-updates/restricted amd64 Packages [13.1 kB]
Get:10 http://archive.ubuntu.com/ubuntu xenial-updates/universe amd64 Packages [817 kB]
Get:11 http://archive.ubuntu.com/ubuntu xenial-updates/multiverse amd64 Packages [18.8 kB]
Get:12 http://archive.ubuntu.com/ubuntu xenial-backports/universe amd64 Packages [8068 B]
Get:13 http://security.ubuntu.com/ubuntu xenial-security/restricted amd64 Packages [12.7 kB]
Get:14 http://security.ubuntu.com/ubuntu xenial-security/universe amd64 Packages [448 kB]
Get:15 http://security.ubuntu.com/ubuntu xenial-security/multiverse amd64 Packages [3735 B]
Fetched 3656 kB in 2s (1644 kB/s)
Reading package lists...
W: Problem unlinking the file /var/lib/apt/lists/partial/.apt-acquire-privs-test.sl2hET - IsAccessibleBySandboxUser (13: Permission denied)
W: Problem unlinking the file /var/lib/apt/lists/partial/.apt-acquire-privs-test.yyysBg - IsAccessibleBySandboxUser (13: Permission denied)
W: Problem unlinking the file /var/lib/apt/lists/partial/.apt-acquire-privs-test.ovFDyD - IsAccessibleBySandboxUser (13: Permission denied)
W: Problem unlinking the file /var/lib/apt/lists/partial/.apt-acquire-privs-test.YFfPv0 - IsAccessibleBySandboxUser (13: Permission denied)
E: Failed to stat /var/lib/apt/lists/partial/archive.ubuntu.com_ubuntu_dists_xenial-updates_universe_source_Sources.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
E: Failed to stat /var/lib/apt/lists/partial/archive.ubuntu.com_ubuntu_dists_xenial-updates_main_binary-amd64_Packages.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
E: Failed to stat /var/lib/apt/lists/partial/archive.ubuntu.com_ubuntu_dists_xenial-updates_restricted_binary-amd64_Packages.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
E: Failed to stat /var/lib/apt/lists/partial/archive.ubuntu.com_ubuntu_dists_xenial-updates_universe_binary-amd64_Packages.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
E: Failed to stat /var/lib/apt/lists/partial/archive.ubuntu.com_ubuntu_dists_xenial-updates_multiverse_binary-amd64_Packages.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
E: Failed to stat /var/lib/apt/lists/partial/archive.ubuntu.com_ubuntu_dists_xenial-backports_universe_binary-amd64_Packages.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
E: Failed to stat /var/lib/apt/lists/partial/security.ubuntu.com_ubuntu_dists_xenial-security_universe_source_Sources.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
E: Failed to stat /var/lib/apt/lists/partial/security.ubuntu.com_ubuntu_dists_xenial-security_main_binary-amd64_Packages.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
E: Failed to stat /var/lib/apt/lists/partial/security.ubuntu.com_ubuntu_dists_xenial-security_restricted_binary-amd64_Packages.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
E: Failed to stat /var/lib/apt/lists/partial/security.ubuntu.com_ubuntu_dists_xenial-security_universe_binary-amd64_Packages.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
E: Failed to stat /var/lib/apt/lists/partial/security.ubuntu.com_ubuntu_dists_xenial-security_multiverse_binary-amd64_Packages.lz4 - pkgAcqTransactionItem::TransactionState-stat (2: No such file or directory)
The command '/bin/sh -c apt-get -y update &&     apt-get -y install maven' returned a non-zero code: 100
2018-06-12 08:10:35,767 [ERROR] - paiLibrary.common.linux_shell : An error occurs, when building your image [ frameworklauncher ]

Solution:
Update the dockerfile and add one line before 'apt-get update' solved this issue:
RUN rm -rf /var/lib/apt/lists
RUN apt-get -y update && \
    apt-get -y install maven


GH-378: @KafkaListener: Fix Group Id When No id

Fixes https://github.com/spring-projects/spring-kafka/issues/378
Don't use a generated container id as the group id if no id property provided.
Also fixes sendToTransaction() by always populating consumerGroupId regardless of source.
cherry-pick to 1.3.x

@garyrussell ,
LGTM, but I would like to understand your comment:

Also fixes sendToTransaction() by always populating consumerGroupId regardless of source.

Looks like it always was like:
.getProducer().sendOffsetsToTransaction(offsets,															ListenerConsumer.this.consumerGroupId);

The problem was that for listener with no groupId property (used the group if from the consumer configs), that field was null 😦 .

Are you cherry-picking? Or shall I?

Cherry-picked to 1.3.x as https://github.com/spring-projects/spring-kafka/commit/9573d3cda0298d1f76642bb8ab0823d7784ebaa2


fix readme typo

observeOn doesn't exist and it wasn't obvious that observeIn was what I was looking for

Current coverage is 60.61% (diff: 100%)

No coverage report found for master at ccbe0c1.

Powered by Codecov. Last update ccbe0c1...eea8052

/// Set the execution context used to dispatch events (i.e. to run the observers).
public func observeIn(_ context: @escaping ExecutionContext) -> Signal<Element, Error> {
  return Signal { observer in
    return self.observe { event in
      context {
        observer.on(event)
      }
    }
  }
}

/// Set the dispatch queue used to dispatch events (i.e. to run the observers).
public func observeOn(_ queue: DispatchQueue) -> Signal<Element, Error> {
  return observeIn(queue.context)
}

Both methods exist within SignalProtocol.swift. I'm pretty sure the migration is documented correctly?

I think observeOn was probably not there at the time of PR, but it was added later so docs are now correct. Thanks for the PR @feighter09 anyway!


check_gcp_environment.cc:60] BIOS data file cannot be opened.

Does this issue affect the google-cloud-cpp project?
yes
What component of google-cloud-cpp is this related to?
I am using pubusb but I think it's a general problem
Describe the bug
I am receiving the following error E0421 08:37:58.704957838      66 check_gcp_environment.cc:60] BIOS data file cannot be opened. when launching my application. gRPC is trying to open the file /sys/class/dmi/id/product_name which does not exist in my filesystem. I am running a docker container on GCE. It seems like it was changed to INFO here https://github.com/grpc/grpc/pull/24893/files however I believe that google-cloud-cpp is using an older version on gRPC?
I think that the error is linked to the fact that I am building it inside docker?
To Reproduce

Include google-cloud-cpp via vcpkg
build a docker image and compile the code
run your application

Expected behavior
Error should be info and not cause a crash
Operating system:
Docker with ubuntu 21.04
What version of google-cloud-cpp are you using?
installed package from vcpkg

Thanks for the detailed bug report.

Describe the bug
I am receiving the following error E0421 08:37:58.704957838 66 check_gcp_environment.cc:60] BIOS data file cannot be opened. when launching my application. gRPC is trying to open the file /sys/class/dmi/id/product_name which does not exist in my filesystem. I am running a docker container on GCE. It seems like it was changed to INFO here https://github.com/grpc/grpc/pull/24893/files

That was merged in 2020-12-03, and should appear in the 1.35.x, 1.36.x and 1.37.x series.

however I believe that google-cloud-cpp is using an older version on gRPC?

We test and link against multiple different versions.  I think this is a packaging problem for gRPC.

I think that the error is linked to the fact that I am building it inside docker?
What version of google-cloud-cpp are you using?
installed package from vcpkg

Okay, vcpkg has not updated gRPC since 1.33.1:
https://github.com/microsoft/vcpkg/blob/master/ports/grpc/portfile.cmake
There are a number of bugs and pending PRs to get that fixed:
https://github.com/microsoft/vcpkg/pull/15078
https://github.com/microsoft/vcpkg/pull/17151
https://github.com/microsoft/vcpkg/pull/17276
It seems like it is all tangled up with updating upb and updating the patches for UWP and other windows variants.
I can think of two ways to get you unblocked:

You can create a vcpkg overlay for a newer version of gRPC.  That might seem daunting, but for Linux I expect you could skip most of the patches.
Meanwhile, I will try to see if I can help move those gRPC PRs along, no promises though, all I can do is volunteer to help, but there are multiple other folks to decide.

Changing the type because I do not see this as a problem in google-cloud-cpp per-se.

The nice folks from vcpkg have updated gRPC:
https://github.com/microsoft/vcpkg/pull/17422
I think that should fix this issue, please feel free to reopen if it does not.


(feat) Add Korean translation link

I've translated angular.io to Korean.
Can you guys add the link?

@naomiblack what should we do with this? Now that we have a official effort on the korean translation?

@Foxandxss I did it before join the translation team. I close the PR and do a official effort for Korean translation.


ENH: Periodic extrapolation for PPoly and BPoly

As was discussed in #6181 an option to treat PPoly (or BPoly) as a periodic polynomial would be useful. A relatively nontrivial part was to implement definite integration with periodicity assumption. Note, that antiderivative remove the periodicity property.
I also changed extrapolate=None to True as the default in __init__, because I don't see any purpose to have None, which acts like True.

Referencing it here as well, so that it does not get lost: https://github.com/scipy/scipy/pull/6181#discussion_r65109304

@pv I tried to make a more robust implementation. Now the distance between a and b is guaranteed to always be correct. Check it out.

I introduced periodic extrapolation to CubicSpline. Not it is more or less ready.

The interaction between extrapolate and bc_type keywords in CubicSpline is a little strange.
In [36]: x = np.linspace(0, 2.*np.pi, 11)

In [37]: y = np.cos(x)

In [38]: s1 = CubicSpline(x, y, bc_type='periodic')

In [40]: s1(2), s1(2*np.pi + 2)
Out[40]: (array(-0.41600968649400255), array(-0.41600968649400255))

In [41]: s2 = CubicSpline(x, y, extrapolate='periodic')

In [42]: s2(2), s2(2*np.pi + 2)
Out[42]: (array(-0.4160655828853339), array(-0.4160655828853339))        # slightly different from above?

In [43]: s3 = CubicSpline(x, y, extrapolate=False, bc_type='periodic')

In [45]: s3(2), s3(2*np.pi + 2)
Out[45]: (array(-0.41600968649400255), array(nan))     # Oops, extrapolate=False won.

The new remainder calculations appears safe to me. (and the following test passes with it https://gist.github.com/pv/1ac1becd19b53e958d2225aa581ffe8a)

@ev-br I find it all rather logical:

[42] is different from [40] because bc_type='not-a-knot' is used as the default, if you want additionally periodic extrapolation - why not?
In [43] you get False extrapolation and 'periodic' boundary conditions as was asked. Why should boundary conditions overwrite extrapolation?

To be honest I don't think it would be good to complicate the logic here. Now it is the most straightforward: each parameter does what is expected from it and nothing more.
@pv, good to know.

@ev-br I see what you mean: let boundary conditions be determined by extrapolation by default. I think it's reasonable, except I don't like to see extrapolation=None, bc_type=None, because it's so obscure. What does it mean? No extrapolation and no boundary conditions? How it is going to work then? Also what should happen when both of them are None?
I would even change extrapolation=None to extrapolation='auto', but here I follow already established convention in PPoly and BPoly classes. All-in-all I'm sure the usability of CubicSpline will be good, because people will seldom change extrapolation and it will be determined from bc_type. In other words let only bc_type be the "decisive" parameter.

Thinking about it a bit more, I actually like the status quo where bc_type  only controls construction, and extrapolate only controls evaluation. Maybe it should be stressed in the Notes section of the docstring. Other than that, LGTM.
(and even this is not bocking).

Added a note and also swapped the order of bc_type and interpolate (from considerations of importance).

OK, apparently all comments are addressed, merging. Thanks Nikolay


mdx output should use omd instead of pandoc

This will help in the CI (as we require a very recent version of pandoc) and on windows.

see https://github.com/ocaml/omd/issues/175
@clecat, it's ok to use the trunk version of omd for this if it makes it easier; we can coordinate a release

You mean without the section feature ?
I'm not aware of the impact of this modification, it may have huge effects on the visual representation of the docs.
We will need to check it twice before doing any merge

I meant that the trunk version of omd is the one we will have to use in the long term, as it is a substantial rewrite and is much easier to maintain.  So testing and adding features to the trunk version is encouraged over the current release -- if we have to bug fix omd-dev to get this mdx feature done, then that's fine.

My work is going well for now, but I am not sure about the section id which was discussed in the omd issue:
When pandoc meets a header, it wraps it inside a section, and gives it an id derived from the content of the header. My question is about what we put inside this id:

pandoc uses this format, even using the specific tags if specified.
the other way to do is simply copying the html5 id format (an id must contain at least one character and may not contain any space characters), which is easier to do and not depandant of an apparently pandoc dependant decision. However, it will generate diffs in the already generated pages and it may broke some links (because we would ignore the tags)

I'm more attracted by the first solution, but it may not be relevant to reimplement this format, so I prefer to get another point of view

Additional information: Omd follows the CommonMark standard, which does not seems to have any behaviour of this kind for now (so no specs nor tests suite, see this issue), so this decision definitely has to be made in mdx.

Same idea with the (typography extension)[https://pandoc.org/MANUAL.html#typography].

Nice progress @clecat!  I think that following the pandoc algorithm for id resolution would be wise.  It covers quite a few corner cases, is reasonably predictable (e.g. editors might support it) and works for other input formats in the futur as well.  So if it's reasonably easy to reimplement, I'd vote for that direction rather than another scheme with HTML5 ids.
It might make sense to contribute this logic (and the typography ones) to omd as extensions, and then just simply use them here.  I can see us compiling omd to javascript as part of an OCaml documentation extension in the future, so centralising the Markdown logic in omd makes sense from that perspective.  Is your work using omd trunk for the moment?

Mmmmh okay I see, I will ask it on omd. And yes, I'm using the omd trunk (if by that you mean the unreleased version). I think I can try a pull request and see for the extensions afterwards.
This is the output of the tests:
-

+

Test0, -toto- and titi

toto

-

-
Test1, -toto- and titi!

+

+
Test1, -toto- and titi! {#test-toto-and-titi}

titi

-

+<section id="Test2, <toto> and titi!" class="level1">

Test2, <toto> and titi!

yo!

-

+<section id="Test3, toto and titi!" class="level2">

Test3, toto and titi!

haha!!

-

-
–Test4, ?toto! a.nd. I/O

+

+
--Test4, ?toto! a.nd. I/O

foo

type <record-name> =

The only broken things are the ids mentioned earlier, the typograhy thing ("--") and some other work done by pandoc which is not part of the commonmark specs.

Long time no see, I did the needed work on omd, and I now have a code using it and passing our tests. However, as pandoc is a really huge project, we may expect some unexpected problems. A good thing to do would be to check with my version if some projects using mdx are not broken (e.g. rwo).

Do you have a branch of omd+mdx somewhere that I could try?

He is my mdx branch: https://github.com/clecat/mdx/tree/omd_pandoc.
For omd, just use the master branch: https://github.com/ocaml/omd

Can you open a PR?

I'm closing this one since the output command has been removed in #195


Miss 'Custom Domain' for one blob container under Quick Access

Storage Explorer Version: 1.14.0
Build: 20200616.5
Branch: rel/1.14.0
Platform/OS: Windows 10/ macOS Catalina / Linux Ubuntu 18.04
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand one storage account -> Blob Containers.
Select one blob container -> Add the blob container to Quick Access.
Check its Properties panel under Quick Access.

Expect Experience:
Display 'Custom Domain'.
Actual Experience:
Miss 'Custom Domain'.

@v-limu the submenu only appears if there is a custom domain configured for the storage account. Is there a custom domain configured for the account you were using to test?

@v-limu  ?

Hi @MRayermannMSFT  Details as below:😊

'Custom Domain' property displays on 'Properties' panel after selecting one blob container.

No 'Custom Domain' property displays after adding the blob container to Quick Access.

Verified on build 20200731.3. This issue has been fixed. So close it.


Updating ADALiOS to 3.0.0

Hi,
can you guys create a branch for using ADALiOS 3.0.0?
We wanted to migrate our app for using the oAuth v2.0 and v2.0 Discovery API.

This repo is being archived as the discovery API that is needed for this to work is deprecated.


Convert to NeRF

Can we convert OMAF JSON file to NeRF transforms.json File??

Yes, both OMAF JSON and NeRF JSON are supported file formats:
cams = camorph.read_cameras("mpeg_omaf","path/to/your/file.json")
camorph.write_cameras("nerf","path/to/output.json",cams)


Gun Bolt Sight has a big text saying "Alzore Let me merge"

Remove it after we have a playtest cause its funny

Fixed in #280


Unstable

Please edit the title to something more descriptive.
Also, add a short description of what changes you made.

Actually, please resubmit the PR against the unstable branch.


I can't find my data! - page on use guide

Relate to users that data is still being onboarded to the service
What is the process for adding new data?

Discuss in UCD catchup

I've shared a proposal for this with Sean and Yvan. Waiting for feedback.
Part of the proposed changes in: https://miro.com/app/board/uXjVKuHRwjI=/?share_link_id=797463948769


b3sum: 1.1.0 -> 1.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/BLAKE3-team/BLAKE3/releases.
meta.description for b3sum is: "BLAKE3 cryptographic hash function"
meta.homepage for b3sum is: "https://github.com/BLAKE3-team/BLAKE3/"
meta.changelog for b3sum is: ""
Updates performed

Rust version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/s17pq3zjcyfh74bprajy20yzxvdrz6z5-b3sum-1.2.0/bin/b3sum passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 1.2.0 with grep in /nix/store/s17pq3zjcyfh74bprajy20yzxvdrz6z5-b3sum-1.2.0
found 1.2.0 in filename of file in /nix/store/s17pq3zjcyfh74bprajy20yzxvdrz6z5-b3sum-1.2.0

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
b3sum
docbookrx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s17pq3zjcyfh74bprajy20yzxvdrz6z5-b3sum-1.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A b3sum https://github.com/r-ryantm/nixpkgs/archive/c295396a7bbdd735328956ad8fc1f528cdcf7b1a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/s17pq3zjcyfh74bprajy20yzxvdrz6z5-b3sum-1.2.0
ls -la /nix/store/s17pq3zjcyfh74bprajy20yzxvdrz6z5-b3sum-1.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

b3sum

Maintainer pings
cc @fpletz @ivan for testing.

Result of nixpkgs-review pr 148839 run on x86_64-linux 1

1 package built:

b3sum


Allow passing our own train/test/split to deeprank-gnn-2 NeuralNet

If we want to create our own train/test/validation split, for example according to peptide clusters, starting from the complete .hdf5 files, how can we use NeuralNet class?

NeuralNet has the percent setting, which can be used to divide the dataset into train and validation: https://github.com/DeepRank/deeprank-gnn-2/blob/9f668f2aaa06e9c8fb8ff954ef5f4b5d21dc073d/deeprank_gnn/NeuralNet.py#L193
We could expand this to testing

We've seen that setting, but we would need to split the data according to peptide clusters (not randomly), and this complicates things a bit. I think that it would be ideal to have the possibility to implement our own splitting (also for a future user). As it is now, I think it's a bit too much hardcoded. Ideally, it would be nice to pass the customized DataSet to the NeuralNet class, making it less hardcoded and more customizable.

I am implementing this in the PR #85 (issue #83)

We want to add one more step: the split of the dataset should be performed out of the NeuralNet class, after the instantiation of the HDF5DataSet object.


I can’t connect My React App

What version of Wallet are you using?
0.21.1
What is happening?
I run a React App but unfortunately i can not connect to wallet. i have installed fuel wallet but when i wanna connect to fuel wallet it open a window to install wallet please give me a way to complete my mission
steps to reproduce
I run a React App but unfortunately i can not connect to wallet. i have installed fuel wallet but when i wanna connect to fuel wallet it open a window to install wallet please give me a way to complete my mission
How it should behave?
I run a React App but unfortunately i can not connect to wallet. i have installed fuel wallet but when i wanna connect to fuel wallet it open a window to install wallet please give me a way to complete my mission
What os are you seeing the problem on?
Windows Desktop
What browser?
None
What is your browser version?
brave

I had this same issue when running my react app.
After compiling the front end, we had two options to view the frontend; the local host and the local network.
On the local host, the wallets (fuel wallet and fuelet wallet) are recognized but on the local network, it doesn't recognize them and asks the wallet to be installed.
Hope it gets fixed soon.


Fix API endpoint for disabling users

Implementation done in commit e396ad323cb0da45d873bcd940059b8e7c904905 is not working correctly. This PR fixes ruby syntax and other aspects of the destroy method implementation.

Not sure how it's different from head :no_content, but if we get 204,
we're good. It'll probably throw 500 on error, we can live with that.
Does using the disabled key return API_KEY_DISABLED? If that one's also
tested this is good to go.
On Mon 20 Aug, 2018, 22:07 Álvaro Arranz<EMAIL_ADDRESS>wrote:

@aarranz commented on this pull request.
In src/api-umbrella/web-app/app/controllers/api/v1/users_controller.rb
https://github.com/apinf/api-umbrella/pull/1#discussion_r211328721:

 @api_user.enabled = false

<EMAIL_ADDRESS>
 head :no_content

else

 format.json { render(:json => errors_response(@api_user), :status => :unprocessable_entity) }

end

@api_user.save
respond_with(:api_v1, @api_user, :root => "api_user")

Also, I'm not a rails guy either ;-), I was looking to other parts of the
code and they were using this line. Anyway, I've tested it and it is
returning a 204 no content response :).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/apinf/api-umbrella/pull/1#discussion_r211328721, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABnL8F4x7jGDT0_5zqsqCI7edjf-BAqmks5uSuWtgaJpZM4WETHc
.

It returns a API_KEY_DISABLED error if the API key used for making the request (for example, using the X-API-Key header) is disabled. If the API key used for making the request is valid but the user disabled, it returns a 204 response. Take into account that the else in the previous code was meant to provide an :unprocessable_entity error response because it was copied from a method handling post data. A DELETE method does not have body.

Allow me to explain it better.
Api umbrella considers users and keys the same (a user has a single key and
vice versa. Email uniqueness is not enforced, and multiple "users" with
same email can be thought of as multiple keys for the same logical user).
Disabling a "user" here is the same as disabling the associated api key.
Now, if this disabled key is used in a request for any existing api, it
should error out with 403 and API_KEY_DISABLED. It looked like that's
what will happen, but I did not test it yet.
You are right, any errors in destroy must be from trying to save, and that
can not return 422. 500 would be appropriate.
On Mon 20 Aug, 2018, 22:26 Álvaro Arranz<EMAIL_ADDRESS>wrote:

It returns a API_KEY_DISABLED error if the API key used for making the
request (for example, using the X-API-Key header) is disabled. If the API
key used for making the request is valid but the user disabled, it returns
a 204 response. Take into account that the else in the previous code was
meant to provide an :unprocessable_entity error response because it was
copied from a method handling post data. A DELETE method does not have
body.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/apinf/api-umbrella/pull/1#issuecomment-414387685, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABnL8BfHZ7KoJEeepD3SC-GbP7qsbks-ks5uSuovgaJpZM4WETHc
.


NMS.GEN.FLW.001 - FLW Data Import

The NMS system shall provide means to upload FLW data into its internal database via a web interface via a CSV file (or web service).
Multiple upload attempts shall be supported whereby the complete FLW data may not be available in the beginning.
Errors during FLW data upload shall be logged.

Is this still the plan of action with the data service from MOH coming up? If yes, I think we still have a gap in how the csv is authored or brought to the ops people...a can that we've been kicking down the road for a few months now

Example of FLW data:
https://gist.github.com/llavoie/fbe2106baed00746ec8e

Fixed by @kamknitt


Decompilation into GLTF format. Short animations

Describe the bug
It seems that when models are decompiled into .glb or .gltf format, the animations in it seem to be accelerated. I'm seeing this in Blender by importing a .glb file. Animations have non-integer keyframes. For example, the AK-47 model, the firstperson_reload animation in ModelDoc has 74 frames, but in Blender the end of the frame is 60.200 and all animations start from frame 1, not from 0. For the animation to be 74 frames, I need to “slow down” the animation by a factor of 1.25. This works for all animations.
The frames of all animations are 0.800, 1.600, 2.400, etc.

Relevant log output or exception with stacktrace
No response
What game does this happen in
CS2
Reference file in a Source 2 game
weapons/models/ak47/weapon_rif_ak47.vmdl_c
Expected behavior
I expect animations in .glb format to have the same number of keyframes as in ModelDoc
What version of VRF are you using?
9.2.3481 3af705de937d1f4065354648f5a7b9b547500b92 on Microsoft Windows NT 10.0.22631.0

I've checked this with babylon sandbox, and animation playback seems to match the GUI. Verified that we are exporting frames as "a set of floating-point scalar values representing linear time in seconds", gltf doesn't have fps.
Are you sure this is not an issue with blender's gltf importer?


PHPUnit Tests  Classifier

Hello @stefanzweifel
This is my first pull request, please review it. This is PR from issue #80.
Can I ask why extend PHPUnit\Framework\TestCase not Illuminate\Foundation\Testing\TestCase ? I guest, this is package is specially for laravel?

Hi @NoorAdiana ,
Thanks for your PR. Looks great 👍 .
You're right. We could also check if the test extend Illuminate\Foundation\Testing\TestCase, but I think naming the component could get complicated. Laravel 5.5 currently supports 3 ways on how you can write tests:

"HTTP Tests" (extending Illuminate\Foundation\Testing\TestCase)
"Browser Tests" / Dusk (extending Laravel\Dusk\TestCase)
"BrowserKit Tests" (extending Laravel\BrowserKitTesting\TestCase)

I will take it into consideration for the next release 😉

Closed as #89 has been merged.


Adjust Orchestration API

[x] Abstract the items of etcd and zk to change Orchestration configuration way.
[x] To consider to use SPI to load other registries except for zk and etcd.

fixed at 3.0.0


I can't reach the website locally

Summary
After building the container using Docker Compose, everything seems fine. Auttokitteh reports it is available on port 9982, but nothing is displayed when accessing the port.
Steps to reproduce

git reset to latest commit 304c9ed45c45bdc385a923ef79093259ab9f10ed

docker compose build
docker compose up

Open website on http://localhost:9982/
Expected behavior
The website should be displayed.
Actual behavior
Nothing is displayed.

you should do make ak. this will fetch the latest webplatform and make it work. you probably did make bin?

you could also make webplatform before make bin.

oh, i see now you're using docker. looking into it.