New hoogle for ghcWithHoogle

Motivation for this change
Convert the ghcWithHoogle expression to work with hoogle >= 5.0.4.
Things done

[X] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Huh, why did you close the PR?

Peter Simons<EMAIL_ADDRESS>writes:

Huh, why did you close the PR?

It was based on your haskell-updates branch, but I submitted it against
master, which means it included a couple of extra automated package
updates; didn't want to force those unnecessarily.


Add functions to see if a provider is available for use.

Public function OSSL_PROVIDER_available() takes a library context and
a provider name, and returns 1 if it's available for use, i.e. if it's
possible to fetch implementations from it, otherwise 0.
Internal function ossl_provider_activated() returns 1 if the given
OSSL_PROVIDER is activated, otherwise 0.

Checklist

[x] documentation is added or updated
[x] tests are added or updated

Note that this new functionality is tested in test/evp_test.c

Rebased

Thanks for the review!  Merged
36f5ec55e69716024f70df53074a2871e091a3e1 Add functions to see if a provider is available for use.
3b5d61f4721f91b5f31a8d3b935f9b3cf4c27644 test/evp_test.c: modify to use OSSL_PROVIDER_available()
e2f72313ccd168eb571b8a4c7cfaf0bf46bdcf9f test/recipes/30-test_evp.t: Modify to test with different providers


TDX: Enable TLB flush hypercall support

Now that we have the private pool we can finally fully wire this up.
This will not be backported, both because it's only needed for TDX Guest VSM support, and to give us enough time to assess the performance impact.
Still needs testing.

This works if I turn on the private pool through the command line, but it does not work by default. @chris-oo what's the plan for the private pool going forward? How should I ensure it exists for TDX?


Bugfix/TS 247 Update Assessor Info

Issue: User Raised Issue BR_000058 #247

Fix the assessment ID in the background function onAssessmentUpdate.
Update the information of the first/second assessor in the assessment document when the background function onApplicationUpdate is triggered.

On the application I was able to update the information (type, full name, email) of the first/second assessor.
and on Independent Assessments" the information of the assessor has been updated. No issues ..


Missing desktop.json

Clean Nix installation results in Awesome refusing to load. After creating the json folder as directed here it stops complaining over settings.json (which it creates) however Awesome then spits out an error over desktop.json (which does not exist and is not created). What is supposed to be in this file? Simply creating an empty json doesn't solve it.
I would show the full error if I could but as I'm not entirely sure how to get Zephyr to work on Nix and as the notification styling - which seem to limit the number of letters allowed - does load, all I can see is the first few lines:
/home/namish/.config/awesome/ui/desktop/init.lua: 394:.cache/awesome/json/desktop.j is all that's shown but presumably it is because the file is missing.

creating an empty json shouldnt solve it, it should be automatically created on startup if .cache/awesome/json is made

creating an empty json shouldnt solve it, it should be automatically created on startup if .cache/awesome/json is made

It is not created. Awesome creates the settings.json file after creating the folder but does not create desktop.json. I wiped the entire awesome config and re-ran home-manager switch several times but the desktop.json file is never created.

Yes but if you create it manually, does it solve your issue ?

I got the same error too

Yup also getting the same error.. doesn't seem to want to  generate settings.json :(

you need to make sure the folder .cache/awesome/json exists

Hmm I've done this -- but let me try from the beginning again and make sure all dependencies are installed as well

you need to make sure the folder .cache/awesome/json exists

You know what.. after i got this working I kept running into an issue with the ruled package on nixos.. apparently I have awesome4.3 with the ruled package but its not ffinding it?? not sure whats going on..
If you have a some steps or a guide on how to get this working on nixos I'd appreciate it :)

you need the git version of awesome not the stable version.

I've tried but it keeps failing to compile awesome :( I know this is not a problem with your dots but would appreciate some guidance I'm new to nix!

refer to this to help with compilation:
https://github.com/awesomeWM/awesome/actions/runs/11543245596/job/32127399471


Gravity ticket

Organization
TEC
Start Date. (D/M/Y)
July 2023
Input mechanism
Outreach by coordination team
Graviton in charge
@BiancaGadelha @JuankBell
Transformational action considered
Approaching parties, providing harm reduction and recognition of unwanted behavior.
Dispute continued?
Trauma didn't continue happening
Notes
follow up for harm reduction on the affected party
Additional actions
Talking about normalization of colonizer behavior on "X space"
Evidence
Discord, DMs

The conflict finished in August, and September had the last follow up. On October the case is ready to be closed.


Mirror set up to "learn" timescaledb

I always wanted to explore timescaledb but never had a good use case for it.
Now I noticed this add-on so I can link my home assistant resource to it but I was wondering if it is possible without moving away from the current database before making this big step?
Is it possible to continuously pull data from home assistant instead of ditching the database that I am currently using?

Awesome thanks! That works like a charm, btw is there a config option to allow network access to TimescaleDB as currently it is only possible through pgadmin within home assistant and I would like to see / test to program against it.

My pleasure!
You can open-up the addon to the outside world using the network panel within the addon-configuration screen:


Small improvements for Linux

This change adds Miniconda to the system PATH as well as deleting unneeded stuff from the installer.

Tested on Arch Linux  󰣇

Ty!


[feature request]fold2

I wrote a compare function in terms of fold2, which seems generally useful.  It would probably belong in owl_dense_matrix_generic.ml.
let fold2 f init m1 m2 =
  let rows, cols as dims = shape m1 in
  if dims <> shape m2 then failwith "fold2: matrices have different shapes"
  ;
  let last_col = cols - 1 in
  let apply_f acc i j =
    f acc (get m1 i j) (get m2 i j)
  in
  let rec loop acc i j =
    if i < rows
    then loop (apply_f acc i j) (i + 1) j
    else if j < last_col         (* don't start on next col if at final col *)
         then loop acc 0 (j + 1) (* start over on next col *)
         else acc
  in loop init 0 0

It's not pretty, and the style probably shows my inexperience, but it's tail recursive.  I didn't succeed in coming up with a more elegant purely functional definition.  However, I've noticed that existing fold functions in owl use imperative methods internally.  I assume that's more efficient, so perhaps you wouldn't want to a version of fold2 similar to this one.  I'm not sure what the tradeoffs are between different iter function variants (iteri is defined in owl_dense_matrix_generic.ml, but iter there is defined by the iter in owl_dense_ndarray_generic.ml, which I haven't sorted out fully).

Perhaps something like this would be useful, too.  (I use it for a special-purpose compare function that I wrote.)  It's like fold2, but will abort the loop if the accumulator comes to have a specified stop_val value.  Other than the extra parameter, this differs from my fold2 by the addition of a single line of code in the loop function.
(** Fold f over matrices m1 and m2 starting with initial value init,
    short-circuiting if stop_val is encountered:
    Folds f through all corresponding pairs of elements of matrices m1
    and m2 by repeatedly applying f acc element_from_m1 element_from_m2,
    where acc is the result of previous applications.  init is the
    initial value for acc.  If f ever returns stop_val, this function
    returns immediately. *)
let short_circuit_fold2 stop_val f init m1 m2 =
  let rows, cols as dims = shape m1 in
  if dims <> shape m2 then failwith "matrices have different shapes"
  ;
  let last_col = cols - 1 in
  let apply_f acc i j =
    f acc (get m1 i j) (get m2 i j)
  in
  let rec loop acc i j =
    if acc = stop_val then stop_val
    else if i < rows
    then loop (apply_f acc i j) (i + 1) j
    else if j < last_col         (* don't start on next col if at final col *)
         then loop acc 0 (j + 1) (* start over on next col *)
         else acc
  in loop init 0 0

Cool, Marshall, many thanks for the code, I will borrow it :)  👍


Race condition in PersistentTopic#addReplicationCluster

Describe the bug
In class org.apache.pulsar.broker.service.persistent.PersistentTopic, we have field replicators of class ConcurrentOpenHashMap. There is a race condiftion in method addReplicationCluster.
replicators.computeIfAbsent(remoteCluster, r -> {
    try {
        return new PersistentReplicator(PersistentTopic.this, cursor, localCluster,
                remoteCluster, brokerService, (PulsarClientImpl) replicationClient);
    } catch (PulsarServerException e) {
        log.error("[{}] Replicator startup failed {}", topic, remoteCluster, e);
    }
    return null;
});

// clean up replicator if startup is failed
if (replicators.containsKey(remoteCluster) && replicators.get(remoteCluster) == null) {
    replicators.remove(remoteCluster);
}

It's clear that there is a race condition if multi threads would run in this code. For example
Thread A is just about to execute replicators.remove, Thread B inserts a non-null PersistentReplicator. Then thread A will delete the PersistentReplicator which thread B just created.
And there is no other thread safe measures applied to these code.
Expected behavior
It should be thead safe.
Screenshots
Not applicable.
Desktop (please complete the following information):

OS: mac

Additional context
Nop

I found another interesting thing is that in org.apache.pulsar.common.util.collections.ConcurrentOpenHashMap, if we call map.put(key, null), it throws an NullPointerException. But we can use map.computeIfAbsent(key, k->null) to insert a null value into the map, which leads to a confusing situation that map.containsKey(key) is falsse, but there is actually a "key" in the map with null value.
Same behavior in java.util.concurrent.ConcurrentHashMap though.

I am working on a fix.


Is the site down?

https://www.textile.photos/ gives me "Unable to connect" on Firefox.

Currently working for me...? Tested on Firefox, Safari, and Chrome. Can you give us additional information about your setup that might help us troubleshoot?

OK. I am closing this issue. Looks like, for whatever reason, the .photos domain is probably blocked at work.
I can access this site from my phone.


Default storage format now Bonsai

Builds on #6530
Change default storage format to BONSAI (was FOREST)
Note - if you had previously used FOREST, this is a breaking change - at startup you will get an error indicating the mismatch -
Mismatch: DB at '/path' is BONSAI (Version 2) but config expects FOREST (Version 1). Please check your config.
Refs #5391

What happens if you are not specifying the format before and after the upgrade to this version, is the previous Forest format recognized or there is an format mismatch error?

What happens if you are not specifying the format before and after the upgrade to this version, is the previous Forest format recognized or there is an format mismatch error?

Good callout - Added to the description of the PR and the changelog

That's a fair comment on the tests @fab-10 - I'm not going to get that done today though - do you reckon that could be a separate PR?

That's a fair comment on the tests @fab-10 - I'm not going to get that done today though - do you reckon that could be a separate PR?

Yes. definitely it could be done in another PR

That's a fair comment on the tests @fab-10 - I'm not going to get that done today though - do you reckon that could be a separate PR?

Yes. definitely it could be done in another PR

created https://github.com/hyperledger/besu/issues/6583


ci(versionsupport): adding golang version for 1.8 - WIP (trying one more thing)

Added support to generate coverage profile for multiple packages on GO 1.8.x using https://github.com/go-playground/overalls since it is not natively supported by GO 1.8.x.
Changed coveralls report branch in Readme.md from go-alpha to master.

Lol how did coverage go down without us touching source files? Are we including more files/dirs than we should? Coverage should only care about the optimizely package. @msohailhussain can we make sure of that?

Lol how did coverage go down without us touching source files? Are we including more files/dirs than we should? Coverage should only care about the optimizely package. @msohailhussain can we make sure of that?

yes the problem is being caused because for some reason cmd/ is being considered for test coverage for go1.8.x, we are looking into it.

Closing this PR, since #127 is replacing it.


http: return response object for chaining #481

Fix for https://github.com/GoogleCloudPlatform/cloud-trace-nodejs/issues/481

Thank you for putting this together!


Permission issue in deploying artifactory/nginx with the current tempalte. seccurityContext is missing.

While deploying with the template on kubernetes with the mounted pvc, you will hit permission issue.
2018-10-18 20:22:12   [73 entrypoint-artifactory.sh] Current max open processes is unlimited
2018-10-18 20:22:12   [83 entrypoint-artifactory.sh] Checking if /var/opt/jfrog/artifactory is mounted
2018-10-18 20:22:12   [88 entrypoint-artifactory.sh] /var/opt/jfrog/artifactory is mounted
2018-10-18 20:22:12  [133 entrypoint-artifactory.sh] Testing directory (/opt/jfrog/artifactory) has read/write permissions
2018-10-18 20:22:12  [141 entrypoint-artifactory.sh] /opt/jfrog/artifactory has read/write permissions for artifactory
2018-10-18 20:22:12  [133 entrypoint-artifactory.sh] Testing directory (/var/opt/jfrog/artifactory) has read/write permissions
/entrypoint-artifactory.sh: line 138: /var/opt/jfrog/artifactory/test-permissions: Permission denied
2018-10-18 20:22:12  [154 entrypoint-artifactory.sh] ###########################################################
2018-10-18 20:22:12  [155 entrypoint-artifactory.sh] /var/opt/jfrog/artifactory is NOT writable!
2018-10-18 20:22:12  [156 entrypoint-artifactory.sh] Directory: /var/opt/jfrog/artifactory, permissions: 755, owner: root, group: root
2018-10-18 20:22:12  [157 entrypoint-artifactory.sh] Mounted directory must be writable by user 'artifactory' (id 1030)
2018-10-18 20:22:12  [158 entrypoint-artifactory.sh] ###########################################################
2018-10-18 20:22:12   [42 entrypoint-artifactory.sh] ERROR: Directory /var/opt/jfrog/artifactory has bad permissions for user 'artifactory'

It helped to add the following spec to the deployment template:
  spec:
      securityContext:
        runAsUser: {{ .Values.artifactory.uid }}
        fsGroup: {{ .Values.artifactory.uid }}

From the chart value, the uid for artifactory is 1030 and for nginx is 104
https://github.com/jfrog/charts/blob/master/stable/artifactory/values.yaml

@ypan887 - the mounted volumes must have permissions to allow Artifactory user to read/write to them.
Can you add more details on your deployment process? Tools, scripts etc.

Closing this as there is no response for a long time and this repo is under deprecation.


[osf] .sps file rendering in Chinese

Steps to reproduce:

Navigate to this page to try to render the .sps file https://osf.io/zvp3s/

Expected:

Displays contents of .sps in plain text

Actual:

Appears to be displaying it in Chinese or a related language. Note that this does not happen for all .sps files (this specific .sps file is to create a graph, other files in the same component display correctly)

This is not Chinese actually. It's related to code transition.
On Mon, Jun 8, 2015 at 4:58 PM, Richard Klein<EMAIL_ADDRESS>wrote:

Steps to reproduce:

Navigate to this page to try to render the .sps file
https://osf.io/zvp3s/

Expected:

Displays contents of .sps in plain text

Actual:

Appears to be displaying it in Chinese or a related language. Note that
this does not happen for all .sps files (this specific .sps file is to
create a graph, other files in the same component display correctly)
[image: screen shot 2015-06-08 at 4 57 40 pm]
https://cloud.githubusercontent.com/assets/8781936/8045158/84b6c3ee-0dff-11e5-9ab2-9327da1da9c0.png

—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/3014.

@chennan47  -- Do we know why this file displays in this way?

The encoding and decoding is not working correctly.
On Wed, Jun 10, 2015 at 9:16 AM, LarryB<EMAIL_ADDRESS>wrote:

@chennan47 https://github.com/chennan47 -- Do we know why this file
displays in this way?
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/3014#issuecomment-110746743
.

Our issue then, yes? Do you think an intern could tackle?

It's a MFR problem. Better have someone familiar with MFR to deal with it.
On Wed, Jun 10, 2015 at 9:20 AM, LarryB<EMAIL_ADDRESS>wrote:

Our issue then, yes? Do you think an intern could tackle?
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/3014#issuecomment-110748160
.

Are sps files binary files?

No it's a script file for the stats program SPSS. Can open these files in a
text editor but see below for an example:
"GET
FILE='C:\Users\Vianello\Documents\Dropbox\Manylabs\Results\effectsizes.graphdata1.sav'.
DATASET NAME esdata.
DATASET ACTIVATE esdata.
add overall variables

compute Overalld=0.
if Study='allowedforbiddend' Overalld=1.96.
if Study='Anchoring1d' Overalld=1.17.
if Study='Anchoring2d' Overalld=1.79.
if Study='Anchoring3d' Overalld=2.23.
if Study='Anchoring4d' Overalld=2.42.
if Study='contactd' Overalld=.13.
if Study='flagd' Overalld=.03.
if Study='gainlossd' Overalld=.60.
if Study='gambfald' Overalld=.61.
if Study='IATd' Overalld=.53.
if Study='moneyd' Overalld=-.02.
if Study='quoted' Overalld=.32.
if Study='reciprocityd' Overalld=.30.
if Study='scalesd' Overalld=.49.
if Study='sunkd' Overalld=.27.
EXECUTE."

On Mon, Jun 15, 2015 at 4:02 PM, Chris Seto<EMAIL_ADDRESS>wrote:

Are sps files binary files?
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/3014#issuecomment-112190743
.

@mfraezz, mind looking into this MFR issue? Thanks!

I'm not as much of an expert on MFR as I was before it became MFRAAS, but this is the only .sps in that project that doesn't render properly. Bash tells me the files are encoded differently:
$ file Downloads/Graph.sps
Downloads/Graph.sps: Non-ISO extended-ASCII text, with CRLF line terminators
$ file Downloads/Order.effects.moderations.sps
Downloads/Order.effects.moderations.sps: ASCII text, with CRLF line terminators
$ file Downloads/ICCs.sps
Downloads/ICCs.sps: ASCII text, with CRLF line terminators

Non-ISO means it contains characters not in the normal ASCII range (31-128), which is likely the reason it's displaying incorrectly. Searching for characters outside of this range gives us one character
-- 0x92, or ’, on line 100 of the file (hexdump also can't render it, so it shows  .) :
$ hexdump -C Downloads/Graph.sps | ag '[89a-f][0-9a-f]'
[...]
00000c10  6f 2e 20 67 61 6d 62 6c  65 72 92 73 20 66 61 6c  |o. gambler.s fal|
[...]

In short, the file's author didn't escape a quote correctly, but we should probably still handle this sort of thing. It's rendered by the code_pygments extension, which deals with more filetypes than any other renderer -- the difficulty isn't in fixing this, but making sure the fix doesn't break anything else rendered by this extension. I'd say it's appropriate for an intern task, though it's very low-priority.

This is fixed. @GaryKriebel


Fix for Python 3.5 and 3.6 - add dist name to travis.yaml

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   73.43%   73.43%
=======================================
  Files           5        5
  Lines          64       64
  Branches        7        7
=======================================
  Hits           47       47
  Misses         15       15
  Partials        2        2

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 508ea50...2586025. Read the comment docs.


Client() not working

Description of the problem
Client not working properly. It shows that I'm logging in facebook from some browser on
my phone (I set it up so that it shows me messages when I'm logging in from somewhere) but it crashes in the command prompt.
Traceback
Traceback (most recent call last):
File "sitakscool.py", line 9, in
client = Client(username, getpass(), max_tries = 1)
File "C:*\Local\Programs\Python\Python38\lib\site-packages\fbchat_client.py", line 103, in init
self.login(email, password, max_tries, user_agent=user_agent)
File "C:*\Local\Programs\Python\Python38\lib\site-packages\fbchat_client.py", line 205, in login
self._state = State.login(
File "C:*\Local\Programs\Python\Python38\lib\site-packages\fbchat_state.py", line 149, in login
return cls.from_session(session=session)
File "C:*\Local\Programs\Python\Python38\lib\site-packages\fbchat_state.py", line 186, in from_session
fb_dtsg = FB_DTSG_REGEX.search(r.text).group(1)
AttributeError: 'NoneType' object has no attribute 'group'
Environment information

Python 3.8.3
fbchat 1.9.7

No code after client = Client(..) works.
Code
import os
from fbchat import Client
from fbchat.models import *
from getpass import getpass
from time import sleep

username = "MAIL"
client = Client(username, getpass(), max_tries = 1)

if client.isLoggedIn():
    name = "GROUP NAME"
    friends = client.searchForGroups(name)
    friend = friends[0]
    thread_type = ThreadType.GROUP

    if os.path.exists('fromrtou.txt'):
        with open('fromrtou.txt', 'r', encoding='utf-8') as data:
            for each_line in data:
                if each_line == '\n':
                    msg = ("######")
                else:
                    msg = (each_line)
                client.send(Message(text = msg), thread_id=friend.uid, thread_type=thread_type)
                sleep(2)
    else:
        print("Sellist faili ei eksisteeri!")

    client.logout()
else:
    print("Midagi laks perse ja ei saanudki sisse logida!")

Did you try removing the line: client.logout()


Join isn't working for me...

I can select a page and save it without a problem, but when trying to join several pages, this is where it simply stops... Anyone has experienced anything like this while joining?
Running in OSX Captain with the newest code...

@assadtony Also running into this. I am getting errors that the pdf file is unable to be opened.

@prescottprue
I ended up doing this, and worked for me.
Join is a function of the library, not of the object. So you need to call it from the actual scissors and not the instance.
Example
`// Put all the scissor pages in a pages array
var pages = [];
var stream = scissors.join.apply(null, pages).pdfStream().pipe(fs.createWriteStream(output));
stream.on('finish', function() {
// Return and download.
res.download(output);
})`

Can this be closed? Or do you think we need a join method on the Command instance?

No further updates, closing.


Backport "Fix composer update via HHVM"  in 2.3.X branch

Can this PR  #150  be backported in branch 2.3.X so that we can use it with HHVM , thank you

An answer would be nice...I've the same issues...

:+1:

See this solution - https://github.com/sensiolabs/SensioDistributionBundle/pull/150#issuecomment-57270246
It helped me. The problem no longer occurs.

Closing as 3.0 is now compatible with 2.3.


Add support for a default SSL certificate in development

When the ssl certificate specified is 'localhost' and
no certificate has been explicitly specified in the certificates
section, fallback to a certificate with 'CN=localhost' (ASP.NET Core SSL
developer certificate) in the current user store and if not found,
an ssl certificate with 'CN=localhost' on the current machine
(IIS Express developer certificate)

@javiercn,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

This needs to go hand in hand with my requested changes for @cesarbs to enhance certificate selection first to prefer localhost certificates which have a SAN entry for localhost, then if no valid ones are present, drop back to subject name.

The code looks fine but ideally it should be somewhere where it can be easily testable i.e. you should be able to write a test with a mock store loader and verify that it's trying to load the expected things from the expected places.


Multi-Browser Support

Even though this project is named PhantomCSS, it would be great if you gave the option of testing in different browsers.
Because as we all know, Chrome != Firefox != IE != ...

Up!
Any updates regarding other browsers like Opera, Safari and the evil IE?


[BUG] Exceptions logged as console errors aren't appearing in AppInsights

Description/Screenshot
Steps to Reproduce
Open a site and page that has an exception and error appears in console
Go to App Insights. Neither the exception nor the console log appears

OS/Browser: Chrome
SDK Version [e.g. 22]:
How you initialized the SDK:
HTML snippet

Expected behavior
All exceptions and console errors are automatically pushed to App Insights.
Additional context

@bernardoflynn Can you please supply some more information, as per requested in the template.
App Insights hooks the windows.onerror to capture and report exceptions, so depending on how you are using the SDK and when it gets initialized etc any exceptions that occur before the snippet (if you are using that) is run ( on the page) or when the SDK is initialized will not be captured.

Hi @MSNev thanks for replying. The snippet is literally the first thing that loads. However we never see any errors in AppInsights that are definitely showing up in the console as errors. In fact all we see are user counts and page visits, etc. No errors.
e.g. in the console in the browser we see these:
TypeError: Cannot read property '0' of undefined
at I (_app-80d35823368a6bf2ad4d.js:1)
at qi (framework.53cfa66f9846f4090096.js:1)
at yo (framework.53cfa66f9846f4090096.js:1)
at su (framework.53cfa66f9846f4090096.js:1)
at uu (framework.53cfa66f9846f4090096.js:1)
at eu (framework.53cfa66f9846f4090096.js:1)
at framework.53cfa66f9846f4090096.js:1
at t.unstable_runWithPriority (framework.53cfa66f9846f4090096.js:1)
at jl (framework.53cfa66f9846f4090096.js:1)
at Kl (framework.53cfa66f9846f4090096.js:1)
However there are no errors reported in App Insights.

This error sounds familiar.
Can you check which version of the snippet that you are using, we had to recently update it for a issue around exception handling for catching errors during pre-initialization. Current version is 5.
Look for the "sv" variable in the snippet content.
This is a sample of where to find the snippet version
m={initialize:!0,queue:[],sv:"5",version:2,config:d};

Closing, assuming this issue was because of a bug in prior versions of the snippet.
Please comment again if this assumption is incorrect.


Title: Improvising overall aesthetics of the budgeting app

Is your feature request related to a problem? Please describe.
The overall design doesn't look like it's inviting users. A container taking the inputs and giving the percentage is placed randomly with no center positions. The tag line seems like it's a mere project display and not an application meant for use. It's not eye catchy and visually pleasing.
Describe the solution you'd like.
Using my knowledge in CSS and Html, I intend to give a suitable vibe to the page. Retaining the dark green color scheme of the input and calculate options, I'll work on making everything look fit and placed in an appealing manner with a bit of change in the tagline or heading.
Describe alternatives you've considered.
I just have a couple of designs in mind and I'm looking forward to implementing them in the project!
Add any other context or screenshots about the feature request here.
No response

please assign the issue to me whenever possible!

The Issue Going to close soon if you do not send PR within 31ST OCT 12 PM IST.


Fix Github URLs

Update URLs to be for this repository, rather than wikidata-item-quality-evaluator.

@lydiapintscher I was just watching your talk about data quality, and thought I'd try out this tool. Looks great! :-) I got mighty confused for a moment though, after not looking very closely at the URL of repo I was cloning...

Haha sorry! Bad copy-paste from the other repo probably. Thanks a lot for fixing it!


dts: riscv: mpfs_icicle devicetree cpu updates

Add the full complement of cpus available on the mpfs_icicle platform and
remove the old microsemi vendor compatible which is not correct for this platform.

Can you update commit comment to describe what the change to clint is about as well.
Hi @galak, will do and will push tomorrow, thanks!

Can you update commit comment to describe what the change to clint is about as well.

Hi @galak,
I have updated the commit message to describe the changes with the clint.
Cheers!

Hi @pgielda, @kgugala,
Could I request a review on this Devicetree change please?
Kind Regards,
Conor


Config should be published in Statamic subfolder of config folder

https://github.com/HandmadeWeb/statamic-cloudflare/blob/976b9118e223ef83a88b3629a0c8e2c976fe6184/src/ServiceProvider.php#L53
Here is my suggestion, to improve the config naming in Artisan and the destination of the published config :
$this->mergeConfigFrom(__DIR__ . '/../config/config.php', 'cloudflare');

if ($this->app->runningInConsole()) {
    $this->publishes([
        __DIR__ . '/../config/config.php' => config_path('statamic/cloudflare.php'),
    ], 'cloudflare-config');
}

Hi @jimblue, thanks for the suggestion, I had initially considered nesting it under the Statamic folder but decided not to, at least under the idea that perhaps that folder should be reserved for Statamic core config files, I also had a brief look at how some other Statamic addon's are handling it and I think I'm going to leave it as is.

Hi @michaelr0,
You're welcome :)
About your decision, my 2 cents :
Most addons and even Statamic team addons config files are saved under config/statamic.
In my opinion this make more cents as it's Statamic addons, not Laravel addon.

Hi @jimblue,
Thanks, I had looked into it and based my decision around how some other add-on's had been handling it, now I understand it could go either way, however my decision was based on how the following have handled the location of the config file.
https://github.com/jonassiewertsen/statamic-butik
https://github.com/doublethreedigital/runway
https://github.com/doublethreedigital/simple-commerce
https://github.com/doublethreedigital/duplicator
https://github.com/WithCandour/statamic-aardvark-seo
At this stage, I don't want to change the location/name of the file, because it is likely to introduce issues for anyone that has already configured their settings and would be a breaking change, just to account for a naming preference.
Thanks :)

Sure totaly understand you point, no worries 😉
Thank you for taking the time to answer.
Peace ✌🏼

Not a problem, let us know should you find any issues :)


deserialization failure at command line, and from invoke().send() but not console test

This is likely a multifaceted issue, possibly stemming from my SAM Template, but I have 1 API Gateway fronted lambda which works perfectly.  In order to process something that takes over 29 seconds, I needed to create a second lambda which processses events asyncronously.
From the console test page, I can send JSON and dbg!() the event payload.
The type for the event is widely used in the working lambda so I'm pretty sure that isn't the problem.  Oddly when I invoke from another lambda with the following code:
        let sdk_config = aws_config::load_from_env().await;
        let lclient = aws_sdk_lambda::Client::new(&sdk_config);
        match lclient
            .invoke()
            .function_name("**function name**")
            .invocation_type(aws_sdk_lambda::types::InvocationType::Event)
            .payload(Blob::new(value_of_my_payload_type.to_json_string()))
            .send()
            .await

The caller logs no error (normal and good for an async call), but the receiver logs:
Error: lambda_runtime::deserializer::DeserializeError

  × failed to deserialize the incoming data into the function's payload type: invalid type: map, expected unit at line 1 column 0

The reproducible thing is that cargo lambda invoke --remote returns the same error (in fact that's where I got the pretty printed version above from).
cargo lambda invoke on the local function works fine.
This pretty much narrows it down to some sort of issue between the configuration and the code, because the invalid type log prints before my little  dbg!() at the top of the main function, not even the handler.
My SAM template (I know this is getting out of the cargo-lambda wheelhouse) is just:
  MyFunc: # why is there no Event Event... or Async Event type?
    Type: AWS::Serverless::Function
    Metadata:
      BuildMethod: rust-cargolambda
      BuildProperties:
        Binary: my_func_binary_name_lined_up_from_cargo.toml
    Properties:
      # environment vars and policies redacted
      CodeUri: ./my_func
      Handler: bootstrap
      Runtime: provided.al2

What's weird to me is the lack of an Events field, ie what is supposed to trigger this thing.  But then when I look in the list of triggers I don't see "async invoke" or even "plain invoke" so I'm assuming it's a baked in default.
So yea, did I write some code that expects () instead of json and it got cached somewhere? I can't figure out how this would work 2 ways (local and through the console test), but not 2 other ways.  The service_fn stuff looks like:
    let func = service_fn(move |event| async move { function_handler(event).await });
    lambda_runtime::run(func).await?;

Putting the move |event| closure seemed necessary to make this work, and indeed I may need to a pass a database connection to the function_handler as well once I get this figured out.
I realize this isn't likely a pure cargo-lambda issue, you're tooling has really made working with rust + aws super nice.  Thanks for that, I'm sure I'll figure this out but I'd appreciate any ideas.

Probably immaterial, but invoking it with:
aws lambda invoke of --function-name  **fn-arn** --invocation-type Event --cli-binary-format raw-in-base64-out --payload file://test.json

returns a 202 (good), but then fails with the map instead of expected unit issue in the logs, with no indication that main was ever executed.
Here is that full error from cloudwatch in case it's informative.  I found the CONTEXT code that produces the error, but it looks like a simple wrap around a normal deserialize meant to make debugging easier. Here it's just got me asking when did I make a function that wants only ().
ERROR Lambda runtime invoke{requestId="2279b362-e0cb-47ba-9947-b79ec52934c0" xrayTraceId="Root=1-65552da4-5cde585b44a5ffb3341b60b9;Parent=21b90da40ab1ecd9;Sampled=0;Lineage=2a5fc3bb:0"}: DeserializeError { inner: Error { path: Path { segments: }, original: Error("invalid type: map, expected unit", line: 1, column: 0) } }

For the record it was a configuration issue, the test on the aws console works against the function but fails in the same way as the other modalities against the alias of the function.  Not sure what that was, but the code was fine and serde still works. :D Thanks for your quick response. The problem is between me and how aliases work (or don't work).


Installation instructions with composer

Composer will figure out current stable version and set constraint to ^0.14.0 and not upgrade to 0.15, ever. Unless instructed.

I would prefer that both methods (editing the text file directly, as well as the proposed "require" command) be present in the docs rather than just the one.

Why is that, if I may ask?

Thanks for the edit and PR!

"A lot of people do it" doesn't mean it's the recommended way 😉


Initialization never completes

MAX Module Version
8.0.4
React Native Version
0.74.5
Device/Platform Info
Android 8.1
Current Behavior
Hey,
For the past days the AppLovinMAX.initialize never seems to complete it just hangs for some reason. Was working just fine, started suddenly.
Only way it runs now is upon first time installing the app or clearing cache/local storage of the app, but after restaring the app it just runs endlessly without returning anything,.
We even tried to add an additional run later in the app not just on mount, by checking: const isInitialized = await AppLovinMAX.isInitialized(); and if it's not we run AppLovinMAX.initialize again and here it returns conf as an empty object, ads never load.
Tried downgrading to applovin 7.1, even 6.3 but same behavior in both.
Expected Behavior
No response
How to Reproduce
Just run AppLovinMAX.initialize
Reproducible in the demo app?
Not Tested
Additional Info
No response

Hi @MamoshiSE, can you provide the Android SDK version your device is running?

I encountered this issue as well as the SDK version is not right. After upgrading the SDK version the issue was solved. Maybe you need to check the Applovin SDK version to make sure it's 13.0.0 and other adapter SDK versions if you have.
react-native-applovin-max: 8.0.4
com.applovin:applovin-sdk:13.0.0


Installation under nix builds ghc!

nix-build -I nixpkgs=https://github.com/NixOS/nixpkgs/archive/2d575e31b8d39bf694268b1e640751d6101fcc13.tar.gz ~/IHaskell/release-8.8.nix --arg packages "haskellPackages: [ haskellPackages.lens ]"

seems to rebuild ghc! What am I doing wrong?

release-8.8.nix bundles its own nixpkgs that uses https://github.com/hvr/head.hackage. You can use my binary cache at https://vaibhavsagar.cachix.org/ to avoid building GHC yourself.

Thanks - if only I were competent at nix - I might be brave and try - or perhaps I should fall back to 8.6.

I forgot that you might be running macOS, in which case my binary cache won't help you!


 This should hopefully resolve the issue of the persistent store not actually saving.

Need to do some testing to confirm still.

Yeah that doesn't seem to fix #233

Even trying to reduce tauri API version again does not resolve the issue T-T

Still not fixed btw

Fixed with 6b087b1448d3e86a04359a752e2cc6ae3b886cdd on Linux. Cannot test Windows atm.

So basic issue was not saving store after inserting new keys?
I don't understand how persistence worked somehow while we didn't call save?

So basic issue was not saving store after inserting new keys? I don't understand how persistence worked somehow while we didn't call save?

So basically if you don't call save() it only saves persistence when exiting the application which was how we did saving so far. For some reason this broke at some point for reason unknown to me. Calling save() explicitly seems to be enough to "fix" it.
A proper fix would require investigating the issue upstream which is something I don't have the time/knowledge to do ^^"


*: update time to fix potential segfault

What problem does this PR solve?
Addressing RUSTSEC-2020-0071.
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note

No release note.

/run-all-tests


Add support for mh lb_algo

Signed-off-by: Arthur Outhenin-Chalandre<EMAIL_ADDRESS>
Pull Request (PR) description
Add support for mh lb_algo, also add related mh_port and mh_fallback similarly to sh_port and sh_fallback.
This Pull Request (PR) fixes the following issues

Looks good as far as someone who has zero experience with keepalived is concerned.
The module lacks a lot of documentation, I suggest adding some for these new parameters. I copied these from the man page available at https://man.archlinux.org/man/keepalived.conf.5.en but fill free to rephrase for better description wink

Thanks for the suggestion I also added those for the sh options in a separated commit!


Roadmap

[x] Figure out how plugins can quit

https://github.com/KillTheMule/nvim-rs/issues/2

[ ] Rewrite the API to be type save, and hide rmpv::Value from the user

Can probably be done for outgoing requests by recursively creating values

Fixed args are easy, but something like nvim_call_function probably needs going through a trait

Incoming Responses can be done like fixed args
Incoming Notifications/Requests might be impossible.

Check out rmp-serde or so, so maybe the user has to just declare a struct

[ ] Consider making a difference between requests and notifications, and send out notifications with the proper message type, and don't wait for a response

Does neovim actually recognize this? Read the docs! Maybe everything's a requests anyways, then don't bother

It does, see https://neovim.io/doc/user/api.html#api-global-events. Maybe not so usefull since there seems little to do where you don't want direct error feedback

[x] Figure out what to do if the handler needs some cleanup logic

https://github.com/KillTheMule/nvim-rs/issues/3

[ ] Consider error handling

Maybe get away from Box<dyn Error> in favor of a small enum?
https://github.com/KillTheMule/nvim-rs/issues/6

[x] Can we merge Requester into Neovim? Could avoid doubling the API

[ ] Need more tests

[ ]  Test all connection types on all OSes
[ ] Test exttypes
[ ]  Check https://docs.rs/loom/0.2.14/loom/

[ ] We need Tokio right now. What can we do to stay "as generic as possible", so we can move to other executors when the possibility arrives?

Use stuff from futures as far as possible.
See what we really need to expose from crate::runtime

[ ] Consider public API.

Right now, everything's just the way it would compile

[ ] Check rust's API guidelines

[ ] Document stuff

[ ] More examples

[ ] Consider logging

Maybe tokio's tracing is an idea?

First, thank you for your effort of creating this project. I've also been discovering neovim rpc in Rust and neovim-lib was not satisfying enough.
For

Rewrite the API to be type save, and hide rmpv::Value from the user
You can rely on rmpv::Value with serde feature and deserialize Value (or ValueRef) into user-defined struct. Your lib could also pre-define popular structs as well. Here is an example of me using in my project:
https://github.com/unrealhoang/lspc/blob/master/src/neovim.rs#L301-L312

Ahh very cool, I was hoping for something like this. Seems like it's an achievable goal to hide Value from the user :) Thanks!


4.1.3 GCP Adapter - GcfJarLauncher: Failed to lookup function

Describe the bug
We have still bug on spring-cloud-function-adapter-gcp 4.1.3.
We using Spring Boot 3.3.1 with Spring Cloud Function. When the function is invoking, we have this error:

Failed to execute org.springframework.cloud.function.adapter.gcp.GcfJarLauncher
java.lang.IllegalArgumentException: Failed to lookup function to route based on the value of 'spring.cloud.function.definition' property 'generateAvatar'

It was working with Spring Boot 3.1.x
Sample
pom.xml:
    <properties>
        <java.version>17</java.version>
        <spring-boot.version>3.3.1</spring-boot.version>
        <spring-cloud.version>2023.0.3</spring-cloud.version>
        <function-maven-plugin.version>0.11.0</function-maven-plugin.version>
        <spring-boot-maven-plugin.version>3.3.1</spring-boot-maven-plugin.version>
        <spring-cloud-function-adapter-gcp.version>4.1.3</spring-cloud-function-adapter-gcp.version>
    </properties>

        <dependency>
            <groupId>org.springframework.cloud</groupId>
            <artifactId>spring-cloud-function-context</artifactId>
        </dependency>
        <dependency>
	    <groupId>org.springframework.boot</groupId>
	    <artifactId>spring-boot-starter</artifactId>
        </dependency>
        <!-- Adapteur GCP -->
        <dependency>
            <groupId>org.springframework.cloud</groupId>
            <artifactId>spring-cloud-function-adapter-gcp</artifactId>
        </dependency>
        <!-- Cloud Function Web HTTP-->
        <dependency>
            <groupId>org.springframework.cloud</groupId>
            <artifactId>spring-cloud-starter-function-web</artifactId>
        </dependency>

	<plugin>
                <groupId>org.springframework.boot</groupId>
                <artifactId>spring-boot-maven-plugin</artifactId>
                <version>${spring-boot.version}</version>
                <configuration>
                    <outputDirectory>target/deploy</outputDirectory>
                </configuration>
                <dependencies>
                    <dependency>
                        <groupId>org.springframework.cloud</groupId>
                        <artifactId>spring-cloud-function-adapter-gcp</artifactId>
                        <version>${spring-cloud-function-adapter-gcp.version}</version>
                    </dependency>
                </dependencies>
                <executions>
                    <execution>
                        <phase>package</phase>
                        <goals>
                            <goal>build-info</goal>
                            <goal>repackage</goal>
                        </goals>
                        <configuration>
                            <outputDirectory>${basedir}/package</outputDirectory>
                        </configuration>
                    </execution>
                </executions>
        </plugin>

The java Bean :
    @Bean
    Supplier<AvatarResponse> generateAvatar() {
        return avatarGenerator::generate;
    }

The properties:
spring:
  application:
    name: vod-profil-avatar
  cloud:
    function:
      definition: generateAvatar

I have two integration tests to validate the function, all passed:
    @Autowired
    private TestRestTemplate rest;
    @Autowired
    private Storage storage;
    @Value("${u-cf.bucket-name}")
    private String bucketName;
    @Autowired
    private FunctionCatalog functionCatalog;

    @DisplayName("Doit générer un avatar")
    @Test
    void should_generate_avatar() {
        //Given 🔨

        //When 👉
        ResponseEntity<AvatarResponse> avatarResponse = this.rest.getForEntity(URI.create("/"), AvatarResponse.class);

        //then ✅
        assertThat(avatarResponse).isNotNull()
                .satisfies(response -> response.getStatusCode().is2xxSuccessful())
                .extracting(ResponseEntity::getBody)
                .isNotNull()
                .satisfies(input -> {
                    var blob = storage.get(bucketName, input.blobId());
                    assertThat(blob).isNotNull();
                });
    }

    @DisplayName("Doit générer un avatar")
    @Test
    void should_generate_avatar_catalog() {
        //Given 🔨
        Supplier<AvatarResponse> generateAvatar = functionCatalog.lookup(Supplier.class, "generateAvatar");

        //When 👉
        AvatarResponse avatar = generateAvatar.get();

        //Then ✅
        assertThat(avatar)
                .isNotNull()
                .satisfies(input -> {
                    var blob = storage.get(bucketName, input.blobId());
                    assertThat(blob).isNotNull();
                });

    }

On invocation, the beans of our application aren't scanned. However, locally with the function plugin, we saw all beans.
To show this, the config of logback-spring.xml:
<logger name="org.springframework.boot.autoconfigure.logging.ConditionEvaluationReportLogger" level="DEBUG" additivity="false">
    <appender-ref ref="consoleAppender"/>
</logger>
<logger name="org.springframework.context.annotation.ClassPathBeanDefinitionScanner" level="DEBUG" additivity="false">
    <appender-ref ref="consoleAppender"/>
</logger>
<logger name="org.springframework.beans.factory.support.DefaultListableBeanFactory" level="DEBUG" additivity="false">
    <appender-ref ref="consoleAppender"/>
</logger>

I have the same problem, it looks like beans are not being created unless they are defined in main class (@SpringBootApplication).
I ended up defining all beans like this:
@SpringBootApplication
public class Application {
    public static void main(String[] args) {
        SpringApplication.run(JiraToolsApplication.class, args);
    }

    @Bean
    public WorklogService worklogService() {
        return new WorklogService();
    }

    @Bean
    public WorklogReportFunction worklogFunction() {
        return new WorklogReportFunction(worklogService());
    }
}

It works if you define a bean as functional interface or as a separate class, as I have it:
public class WorklogReportFunction implements Function<Message<BufferedReader>, Message<byte[]>> {
...

@ihoroshko-n-ix I confirm, this solution works fine.
So the original problem becomes from the scanning step treat by the Spring IoC to find the stereotyps.
We loose on simplicity to create beans but we gain on cold start 😅
Once again, thanks for your solution

@ihoroshko-n-ix & @gfourny-sfeir
Could you please provide a link to a GitHub repo with the code using spring boot 3.3.x and spring-cloud-function-adapter-gcp 4.1.3?
I'd like to test it and understand how it behaves, especially if the beans are not being scanned.
How could it still function in this case?

After hours of working on this issue, I found that the issue is with Spring boot newer version which does not have support yet and hence downgrade your Spring boot version to 3.1.x in my case tested with 3.1.10 version and it works fine. Look at my example in https://github.com/balaji1974/functions
And I solved this issue by following this thread on the same issue:
https://github.com/spring-cloud/spring-cloud-function/issues/1085

@ihoroshko-n-ix @gfourny-sfeir  Can you post the code in Github repo as I am not sure of how to inject the service into the Function interface

@rajakumare1 I couldn't share our repos because they're private.
You need to declare your Bean in the main class like this:
@SpringBootApplication
public class VodProfilAvatarApplication {

    public static void main(String[] args) {
        SpringApplication.run(VodProfilAvatarApplication.class, args);
    }

    @Bean
    RestClient restClient(RestClient.Builder builder, VodProfilAvatarProperties properties) {
        return builder.baseUrl(properties.avatarGeneratorBaseUrl()).build();
    }

    //Business service without @Service or @Component annotation
    @Bean
    AvatarWriter avatarWriter(Storage storage, VodProfilAvatarProperties properties) {
        return new AvatarWriter(storage, properties);
    }

    //Business service without @Service or @Component annotation
    @Bean
    AvatarClient avatarClient(RestClient restClient) {
        return new AvatarClient(restClient);
    }

    //Business service without @Service or @Component annotation
    @Bean
    AvatarGenerator avatarGenerator(AvatarClient avatarClient, AvatarWriter avatarWriter) {
        return new AvatarGenerator(avatarClient, avatarWriter);
    }

    /**
     * {@link Supplier} called by Cloud Function and refers in application.yaml
     *
     * @return {@link Supplier<String>}
     */
    @Bean
    Supplier<AvatarResponse> generateAvatar(AvatarGenerator avatarGenerator) {
        return avatarGenerator::generate;
    }
}

In your application.yaml:
spring:
  application:
    name: vod-profil-avatar
  cloud:
    function:
      definition: generateAvatar #Name of the bean

@olegz We suspect the bug comes from the FunctionInvoker class in the adapter code by these lines:
Thread.currentThread() // TODO: remove after upgrading to 1.0.0-alpha-2-rc5
			.setContextClassLoader(FunctionInvoker.class.getClassLoader());

❓ Why
Because the classLoader used to scan is provides these lines and cannot scan the package created by our applications

Happy to report that both the issue mentioned in https://github.com/spring-cloud/spring-cloud-function/issues/1085 and this one are resolved for me after upgrading to version 4.1.4 (tested with Spring Cloud Release Train 2023.0.4 and Spring Boot 3.3.6). My Spring Boot starts correctly again when deployed to GCP, and all beans are correctly scanned 👍.


Dark mode

Global dark mode:

Toggle in header (defaults to system, options for dark or light)
Dark presets in echarts theme

If this is available, I can begin working on it. Might need a few pointers on the overall design (colors, toggle button design etc) for dark mode.

@nidhi-kala, this one's not quite ready to go yet -- I think we should tackle this after #738

Sounds good. 👍

Closing in favor of https://github.com/evidence-dev/evidence/issues/2499


BigInt imul?

I know of no use case for imul accepting BigInts, but at least its mathematical behavior would be well defined and exact.
On the other hand, making imul polymorphic could make optimization for asm.js more difficult. See also the original BigInt proposal’s “Don’t break asm.js” section.
@ljharb has said—and I agree—that that BigInts and Numbers should always be interchangeable by default, unless that would cause silent precision loss – i.e., unless there's a strong reason they should not be (https://github.com/js-choi/proposal-bigint-math/issues/8#issuecomment-912126129).
Under this framework, it’d be good to know if a polymorphic imul would worsen asm.js optimization, which would count as a “strong reason [BigInts and Numbers] should not be [interchangeable]”. CC: @syg, @codehag.

While it'd be good to know regardless, I was under the impression that asm.js was no longer a priority in favor of WASM.

For BigInts, Math.imul would be a mystification of *, so I see no reason to support it.
I don't really agree with argument that BigInts and Numbers should be interchangeable by default. Why should they be interchangeable by default? Floating point and arbitrary precision ints are drastically different and choice of which to use should be thought through by the programmer. We can't have even most of Math work for BigInts, even if you would like them to be interchangeable by default, that intuition just won't hold up in practice, so why push for it?

Closing. There are no known use cases for BigInt imul, and it may interfere with asm.js.
See also https://github.com/tc39/proposal-bigint-math/issues/13#issuecomment-919472001 and https://github.com/tc39/proposal-bigint-math/issues/14#issuecomment-918663577.


Consider the rqrcode-with-patches gem

Just a tip :smile: We've rolled our own solution for supporting Google Authenticator at my employer before I came across your gem. We used https://github.com/bjornblomqvist/rqrcode because it offers a few improvements over the original (and unmaintained) rQRCode gem:

Rendering PNG in Ruby with chunky_png, so you don't need ImageMagick
Chunky_png has a to_data_url function, so you can inline the QR code in HTML and you don't need to store files
This workaround for IE7 for data URI's is worth mentioning: http://www.phpied.com/mhtml-when-you-need-data-uris-in-ie7-and-under/
QR code length can be calculated automatically, so you don't get errors such as https://github.com/whomwah/rqrcode/issues/15 if you don't set a fixed length

Patches accepted! I'm not currently using this in production myself. I like the to_data_url functionality.


Move to wagtail-nest

Hey folks,
We are moving third-party packages to https://github.com/wagtail-nest
Could someone with admin rights, initiate the move for this package?
Thank you

Ping @gasman @thibaudcolas @tomdyson 🙏

Done!

Thank you @gasman.
Something I did not anticipate though, is that I cannot seem to be able to merge PRs anymore, I imagine I need to be added to the wagtail-nest org or something? Maybe he same can be done for @Stormheg which often works on this repo.

uh-oh, forgot about that. Added you as an admin @loicteixeira, and @Stormheg as maintainer

Perfect thank you


Per category result for MVTec AD

Hi Nelson,
can oyu provide the results of EfficientAD-M per category on the MVTec AD dataset? For matching the results in the anomlib framework, I would like to have the single category data :)
Thanks!

Hello, I have provided the results in this repository in results/metrics.json. Hopefully they are helpful.

Hello, I have provided the results in this repository in results/mvtec_ad_medium.json. Hopefully they are helpful.

Thanks!


fix(Project/ClusterMemberEditor): fix find roleTemplates bug

Summary
Fixes #10894

Occurred changes and/or fixed issues

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[ ] The PR template has been filled out
[x] The PR has been self reviewed
[ ] The PR has a reviewer assigned
[ ] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[ ] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

@richard-cox @aalves08 This needs a review if we have time - not sure why the checks didn't run - maybe needs a rebase

Should backport to 2.9


docs: Add new blog post "API monetization using an API Management and a billing provider"

Fixes: #[Add issue number here]
Changes:
Added new blog post about "API monetization using an API Management and a billing provider".
Screenshots of the change:

Need to upload images to our CDN but it seems that I do not have a proper access.


Add Modernizr loader

I'm not sure which category fits best for the Modernizr loader, feel free to change it if necessary :)

Thanks!


Add -i to docker command, otherwise doesnt work

Description

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[x] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[ ] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[x] I updated the documentation where necessary.

Codecov Report

Merging #4864 into master will decrease coverage by 6.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4864      +/-   ##
==========================================
- Coverage   82.27%   76.10%   -6.18%
==========================================
  Files         200      200
  Lines        9294     9294
  Branches      372      372
==========================================
- Hits         7647     7073     -574
- Misses       1647     2221     +574

Impacted Files
Coverage Δ

...core/database/cosmosdb/RxObservableImplicits.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedConsumer.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0.00% <0.00%> (-96.23%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0.00% <0.00%> (-93.90%)
:arrow_down:

...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala
0.00% <0.00%> (-92.31%)
:arrow_down:

...e/database/cosmosdb/cache/KafkaEventProducer.scala
0.00% <0.00%> (-77.78%)
:arrow_down:

...enwhisk/connector/kafka/KamonMetricsReporter.scala
0.00% <0.00%> (-76.20%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0.00% <0.00%> (-74.08%)
:arrow_down:

...abase/cosmosdb/CosmosDBArtifactStoreProvider.scala
7.69% <0.00%> (-50.00%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


Add -v flag for verbose log to stdout

This implements #7.
The -v short flag was being used for printing version number so I took it away from --version and put it on --verbose because that seems more conventional with CLI tools..
Example output:
~/ > ./gopatch -v -p temp.patch ./tempdir/...
[gopatch] /Users/sungyoonwhang/go/src/gopatch/tempdir/test1.go : Patch successfully applied.
[gopatch] /Users/sungyoonwhang/go/src/gopatch/tempdir/test2.go : Couldn't patch. The file couldn't be parsed.
[gopatch] /Users/sungyoonwhang/go/src/gopatch/tempdir/test3.go : Couldn't patch. Nothing was matched.
[gopatch] /Users/sungyoonwhang/go/src/gopatch/tempdir/test4.go : Patch successfully applied.
could not parse "/Users/sungyoonwhang/go/src/gopatch/tempdir/test2.go": /Users/sungyoonwhang/go/src/gopatch/tempdir/test2.go:2:1: expected 'package', found 'import' (and 1 more errors)

Codecov Report

Merging #22 (028d3f9) into main (1592705) will decrease coverage by 0.09%.
The diff coverage is 73.33%.

@@            Coverage Diff             @@
##             main      #22      +/-   ##
==========================================
- Coverage   86.34%   86.25%   -0.10%
==========================================
  Files          38       38
  Lines        1985     2000      +15
==========================================
+ Hits         1714     1725      +11
- Misses        217      221       +4
  Partials       54       54

Impacted Files
Coverage Δ

main.go
63.69% <73.33%> (+1.01%)
: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 1592705...028d3f9. Read the comment docs.

Thanks for the feedback @abhinav! I've updated the PR to use standard log.Logger instead of implementing a new interface for it since I don't think adding a custom interface is necessary for now as you mentioned.

Since the comment got hidden because of the suggestion, copying it here:

Separately, do we want verbose output on stdout or stderr? Generally, if the output is meant to be machine processable, it should be stdout so that it's easy to pipe into other commands, but if it's largely for human consumption, it should be stderr. (See also this part of clig.dev.) In this case, I'm open to either.

Separately, do we want verbose output on stdout or stderr?

I was thinking we should go with stdout because I can imagine the output getting to messy to be human-parsable with large # of files, and I can imagine people piping it for filtering: gopatch -v -p my.patch project/... | grep "skipped" > skipped.log.


[FIX] sap.m.Dialog: control sometimes "jumps" when dragged

This was casause by the fact that the drag event would be trigerred on a descendant of the dialog header. The offset from the dialog would then be incorrectly calculated.

Now, the offset is calculated by iterating through all ancestors of the target DOM element until the dialog is reached.

A secondary bug was that when pressing exactly on the border of the dialog (0 offset), the control would "jump" because of a falsy check which was also fixed.

Fixes https://github.com/SAP/openui5/issues/1262

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Hi @serban-petrescu,
Thank you for your contribution. I have contacted the responsible team for checking the pull request.
Kind Regards,
Gery (gihub dispatcher)

Hi @serban-petrescu ,
The responsible team has recently made a fix for reported issues.
Kind Regards,
Gery (github dispatcher)


Change all platformio examples to flash to the micropython partition by default.

The platform IO examples will now by default be flashed to the micropython partition. This also means that by default, restarting the device will go back to the main menu. To change this, you can change default_envs to fri3dbadge2024_override_main_firmware in the platformio.ini file.

Which values are changed in sdkconfig.h compared to the original one? Are you sure the base sdkconfig.h is the correct one? It mentions IDF version 4.4.7, but all platformio examples use<EMAIL_ADDRESS>which is IDF 4.4.6 as far as I remember.

Is it possible to put the sdkconfig file in a subdirectory ? We will need it for different versions (tinyusb or not, idf versions, debuglevel, ...).


add Redis Cluster support

Hi there,
ioredis has support for Redis Cluster connections.
Currently I dont see a way to connect to Clusters with redis-modules-sdk.
may be all that has to be changed is:
/modules/module.base.ts line 24 from
    async connect(): Promise<void> {
        this.redis = new IORedis(this.redisOptions);
    }

to something like
    async connect(): Promise<void> {
        if (this.redisOptions.cluster) // flag tbd
           this.redis = new IORedis.Cluster(this.redisOptions);
        else
           this.redis = new IORedis(this.redisOptions);
    }

This should work with "default" Redis, I dont know if this will work with Redis modules and saidly cannot test it atm.

Hi, thanks for the issue.
After a short look, today we can't support this.
The reason is that the 'Cluster' type does not support send_command function I use to initiate a command to the database.
I will need to open an issue and wait for it before this is done.

@MR4online Hi, sorry for the delay. I am merging an addition ability to add support for ioredis cluster.
This was not yet tested in ongoing integration so I'd be happy if you could try it out and let me know if it works.
Thanks,


jami: 20230619.1 -> 20231201.0

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3067

Reviewed points
Result of nixpkgs-review pr 272231 run on x86_64-linux 1

5 packages built:

jami
opendht
opendht.dev
opendht.lib
opendht.man

[X] package name fits guidelines
[X] package version fits guidelines
[X] package builds on x86_64-linux
[X] executables tested on x86_64-linux
[X] all depending packages build
[X] patches have a comment describing either the upstream URL or a reason why the patch wasn't upstreamed
[X] patches that are remotely available are fetched rather than vendored

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1314


Use of missing aggregations

I want to do a composite aggregation and include a nullcheck. I have seen that there is .missing. However, I get no results back as soon as I add it. Without .missing my query works. Can someone tell me how to use missing correctly? Here is my query:
	public static Map<String, CompositeAggregationSource> buildTermsAggregation(String name, String field) { 		return Map.of(name, new CompositeAggregationSource.Builder().terms(t ->                    t.field(field).missing(FieldValue.of("missing"))).build()); 	}

Thank you for reaching out!
This issue appears to be more related to the OpenSearch functionality itself rather than the Quarkus OpenSearch extension. Specifically, it concerns the use of the .missing parameter in composite aggregations, which is a feature of OpenSearch.
I recommend posting this question on the OpenSearch GitHub repository or discussing it on the OpenSearch community forums, where the maintainers and contributors for OpenSearch can provide more accurate guidance.
Therefore, I will be closing this issue here. Please feel free to reach out if you have any concerns directly related to the Quarkus OpenSearch extension. 😊


Spike: Deploy Cloud Run using Terraform

What is the context of this PR?
This spike demonstrates the Cloud Run deployment using Terraform instead of gcloud.
How to review
Review using:

https://github.com/ONSdigital/eq-pipelines/pull/263

OR

Deploy directly using Terraform:

Ensure you have tf 1.0.0 install locally.
Run cp ci/terraform/terraform.tfvars.example ci/terraform/terraform.tfvars and update the variables in this file if desired.
If you use an existing staging env, make sure to delete the existing runner Cloud Run deployment first!. Otherwise, deploy the main infrastructure to a new project using https://github.com/ONSdigital/eq-terraform-gcp
Once the infrastructure is up, deploy the app using:PROJECT_ID=<project_id> \
TERRAFORM_STATE_BUCKET=<state-bucket-name> \
./ci/terraform/scripts/deploy.sh

Destroy the app using:PROJECT_ID=<project_id> \
TERRAFORM_STATE_BUCKET=<state-bucket-name> \
./ci/terraform/scripts/destroy.sh

TO DO when implementing:

Enforce type hints and add validation for the TF vars

Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

Work has been migrated to new repo.


Fix - casing typo in the ArenaVueUiStackbar filename

When installing the project locally on Linux (Ubuntu in my case, but it would be the same for any OS with a file system that is case-sensitive), there is an error in the console:
[vite] Internal server error: Failed to resolve import "../TestingArena/ArenaVueUiStackbar.vue" from "src/App.vue". Does the file exist?
So I simply renamed the file as ArenaVueUiStackbar (with a lower b to bar) seems to be the dominant choice in the code-base.

Side note:
I see a similar difference between the types and the README for  VueUiSparkStackBarConfig and VueUiSparkStackBarConfig. Sometimes it's upper case for Bar sometimes lowercase bar. I was going to include a second commit to fix it in this PR, but actually I don't know if changing a type is a breaking change or not. So I'll let you handle it if you think it's necessary.

Thanks @cyrilf :)
Basically the readme should reflect the types in .d.ts. Changing a type name there leads to breaking changes unfortunately.

Yes, ok. @graphieros
As you can see on this page the casing is not correct.
Even if it's a breaking change, I would favor renaming the type file to respect the repo naming convention, in which case I have a commit ready, let me know.
Otherwise, I let you handle the renaming in the README and instalation files?

Allright @cyrilf let's go for the type renaming :)


Addition of Cybersecurity Roadmap

@rishicds I would like to add Cybersecurity Roadmap as well . Kindly , assign it to me . I'll start working on it immediately....Thanks :)

Assigned


ÃƒÂ¼  (in body)

@elcopeland commented on Wed Jan 31 2018
http://munews.wpengine.com/111215_journalism-2/
Alberto IbargÃƒÂ¼en, president and CEO of Knight Foundation.
ÃƒÂ¼ == ü
sauce: https://knightfoundation.org/about/staff/alberto-ibarguen

@elcopeland commented on Wed Jan 31 2018
other case:

http://munews.wpengine.com/110512_macon/
http://munews.wpengine.com/110426commencement/

Fixed with herculean entity encoding effort.


Certification on PowerBI.com

Hi,
thanks for this great visual!
Is there a chance for you to certify this visual in the AppSource? My company actually blocks all uncertified visuals on PowerBI.com.
I don't know how much effort this is, but I would really appreciate.

Yes, that would be really great if you could certify this very useful visuals in Microsoft Power BI AppSource! Microsoft certification is also needed from my company. Thanks a lot!

I've not reviewed everything in the code yet, but I'm not seeing anything preventing this visual from getting certified per the guidelines published here. Does anyone know of any technical limitations with getting this visual certified?


No main.js

I'm trying to install the plugin manually following the instruction provided but there is not main.js in the zip I downloaded from github. Inside source there is main.ts but even if I rename it I got an error (Failed to load plugin obsidian-calibre-plugin).
I also tried to copy the full folder in my vault but I got the same issue.

I'm trying to install the plugin manually following the instruction provided but there is not main.js in the zip I downloaded from github. Inside source there is main.ts but even if I rename it I got an error (Failed to load plugin obsidian-calibre-plugin).
I also tried to copy the full folder in my vault but I got the same issue.

https://github.com/caronchen/obsidian-calibre-plugin/releases

Thanks :)


Prevent Same value for different contacts + support for subjects

This PR should fix https://github.com/Logicify/mautic-advanced-templates-bundle/issues/6 and add support for subjects (https://github.com/Logicify/mautic-advanced-templates-bundle/issues/2)

Merged this into master and master and created a new release. Thank you very much @kuzmany for your help!


GEODE-7345: Decouple TcpServer from Core's Socket-Related Classes

GEODE-7345
Eliminate these exceptions in TcpServerDependenciesTest:
              // TODO - TCP socket related classes
              .or(type(SocketCreator.class))
              .or(type(SSLConfigurationFactory.class))
              .or(type(SecurableCommunicationChannel.class))
              .or(type(SocketCreatorFactory.class))
              .or(type(SSLConfigurationFactory.class))

This work is being done in pursuit of GEODE-7343 which will break all dependencies from from the org.apache.geode.distributed.internal.tcpserver package to classes in geode-core. The end goal of that ticket is to move the tcpserver package into its own module, geode-tcp-server that will have no dependencies on geode-core. That new module will serve both geode-core and the eventual geode-membership module.
To eliminate the dependencies on SocketCreator, I defined a new interface, TcpSocketCreator and a factory method that turns a SocketCreator into one of those: TcpSocketCreator.asTcpSocketCreator(SocketCreator).
The new TcpSocketCreator interface is a vastly narrowed (5 methods vs 50 methods) version of the SocketCreator class used throughout geode-core. This new interface has only the methods needed to server the needs of classes in the tcserver package (eventually the geode-tcp-server module).
In places where we were creating SocketCreator and using that to construct a TcpClient or TcpServer, we're now using asTcpSocketCreator() to adapt to the new (narrow) interface.
The other dependencies e.g. SSLConfigiurationFactory were easier to eliminate. I simply extracted construction of those objects up out of TcpServer and TcpClient as constructor parameters. Dependency injection wins.
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Fixed failing distributed tests.

Closing this PR while I fix this failing stress test.


Code List Config same_as field not serialised to CSV-W

When the user specifies a same_as relation between a concept in their new code list and an existing concept defined elsewhere, the information is not serialised into the code list CSV-W.
We need to make sure that the same as relationship is serialised correctly into the CSV-W. This means adding the column to the CSV file as well as adding the owl:sameAs relationship to the column definition.
Refactor behave tests in cube feature.

@robons I've updated the effort in this ticket from 3 to 5 as this involved an additional task - refactoring behave tests in the cube feature. I have refactored all the tests in the cube feature.

This is an example of it not serialising the same_as provided in a concept scheme. https://github.com/GSS-Cogs/family-climate-change/tree/master/datasets/Forestry-Research-Forestry-Statistics-2022-Carbon

Scoring: Impact is high: the functionality is not accessible and there is no workaround. Probability is moderate: users who want mixed-concept schemes will be impacted, which account for large number of datasets but not the majority.
This makes this bug critical.


Use Path class to work with paths instead of using string operations

In various places (e.g., https://github.com/mkim219/kimchi-ssg/blob/main/kimchi-ssg/helpers.cs#L107) you are working with pathnames as regular strings. A better option is to use the Path class, which provides methods for creating and inspecting paths.  This is important when you move between operating systems, where paths are different.

I have fixed issue #7. Thank you David


ask for new version

now halo update its plugin form~ so it can not be used in the newest version.
it request a plugin.yaml, and need be packaged as a jar file

reference sitemap has been integrated.


Extended Notifications not working

IPN calls to my server are working fine - getting paid, confirmed, and complete notifications correctly. However, when I run this:
$invoice->setExtendedNotifications(true);
Before creating my invoice, I don't get any IPN calls. What's going on here?

Can you provide some invoice ids where you have extendedNotifications=true, and you expected IPNs that were not delivered?

sure, these two:
GHyD39MKcsSXCQAfJksnvn
4SRsdKR3m2jS18yzJgsECu

paid&confirmed&complete webhooks were sent, but your server responded with HTTP 500.

hmmmm, is there something different about these that might cause this?
Everything works fine when extendedNotifications is false.
On Mon, Aug 28, 2017 at 1:46 PM, Pieter Poorthuis<EMAIL_ADDRESS>wrote:

paid&confirmed&complete webhooks were sent, but your server responded with
HTTP 500.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bitpay/php-bitpay-client/issues/246#issuecomment-325424770,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJ_3m77TaK5QZXPs8kEConNGG9xFwuiks5scvzygaJpZM4PEvNA
.

Yes, the following is added when using extendedNotifications:

an event block is added to the beginning (containing code + name, indicating the type of event)
the payload is put in a data block
Example:
{ event: { code: 1006, name: 'invoice_completed' },
data:
{ id: 'GHyD39MKcsSXCQAfJksnvn',
url: 'https://test.bitpay.com/invoice?id=GHyD39MKcsSXCQAfJksnvn',
posData: '',
status: 'complete',
btcPrice: '0.005797',
price: 25,
currency: 'USD',
invoiceTime:<PHONE_NUMBER>168,
expirationTime:<PHONE_NUMBER>168,
currentTime:<PHONE_NUMBER>299,
btcPaid: '0.005797',
btcDue: '0.000000',
rate: 4312.3,
exceptionStatus: false,
buyerFields:
{ buyerName: '',
buyerAddress1: '',
buyerAddress2: '',
buyerCity: null,
buyerState: null,
buyerZip: null,
buyerCountry: null,
buyerEmail<EMAIL_ADDRESS>buyerPhone: null,
buyerNotify: null } } }

ahhh I see. So the first few lines of my IPN endpoint are as follows
$data = file_get_contents("php://input");         if ($data && ($ipn = json_decode($data))) {             if ($id = $ipn->id) {
This is likely causing the 500 error when I try to grab the "id", yes? How would you suggest parsing the new data format? I'm confused about what you mean by "the payload is put in a data block"

got it thanks!


Fix crash on anonymous type parameter to implicit class

Fix crash on anonymous type parameter to implicit class. There are a couple of TODO comments to resolve before merging this, but feel free to review (including suggestions for the TODO comments).
Closes #366

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Sameer Sundresh seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Implement Picture Component

Syiro should provide a Picture Component to act as a reliable alternative to the HTML5 picture Element, which is not supported in:

Newer versions of Internet Explorer (it is supported in Microsoft Edge, however)
iOS Safari 9.2 and below
Android KitKat and below (really what is cared about is 4.4.2+)

This Picture Component should have a similar feature set to HTML5 picture, in that you will be able to set multiple sources and automatically choose the best source depending on the resolution of the browser or DPI.
Where would this be used?

Navbar & Sidepane logos.
Media Player backgrounds:

Background for Audio-type Media Player
Initial "poster" for Video-type Media Player

General usage not directly integrated into Syiro (like dynamic "hero" images on websites).

Implemented Picture Component, implemented into Navbar and Sidepane for logos. Will evaluate whether it is necessary or not for Media Player in the future.


add type boolean logic for requiredIf and requiredNotIf

Love the library!
Ran into an issue where using the requiredIf and requiredIfNot would not properly validate a boolean on the field you are requiring based on. Example below:
yield this.validateBody({
    isGuest: 'required|boolean',
    guest: 'requiredIf:isGuest,true'
});

If you were posting something like the following, guest would not be required. My fix will even work for 0 and 1 because JSON.parse() will convert 0 or 1 to a boolean.
{
    "isGuest": true
}

Hey Josh,
There is an alternative way to validate fields. Check the following link.
https://koa-validation.readme.io/docs/required-rules
You could make a tweak to the above code and write it following way.
yield this.validateBody({
    isGuest: 'required|boolean',
    guest: { requiredIf: [ 'isGuest', true ] }
});

Ah cool I was looking for something of the sort but overlooked that. Would still be nice for the shorthand though

@srinivasiyer using your example i'm noticing a bug with defining the following:
yield this.validateBody({
    isGuest: 'required|boolean',
    guest: { requiredIf: ['isGuest', true] },
    user: { requiredIf: ['isGuest', false] }
});

This fails to properly validate when passing isGuest: false in POST. It passes validation for user when it should not.

Any update on this? I'm having issues finding how your solve is chain-able with other validators.

Hi Josh,
Please give me time till the weekend to analyse the problem. I shall have it sorted by then. I apologize for the delay. It has been a very busy time!!

Thanks for the reply! I totally understand

Hi Josh,
Sorry for all the delays. I just made some changes to fix the bug but the approach is different from the one you used. There was an error in my code which was creating unable to identify the false condition in the requiredIf validator correctly. You will not not face an issue with it.
However note that you would still need to use the
yield this.validateBody({
    isGuest: 'required|boolean',
    guest: { requiredIf: [ 'isGuest', false ] }
});

rather than
yield this.validateBody({
    isGuest: 'required|boolean',
    guest: 'requiredIf:isGuest,true'
});

Philosophically, I am a bit against recognizing a boolean within a string. I hope the patch helps. I am closing the pull request and hoping that this patch helps you solve your problems. Thanks again for spotting a rather hard to spot error. Appreciate the help.

@srinivasiyer Hey man, no worries. Thanks for the update!


Improving french and spanish translations

Fixes some translations in french and spanish and adds some new translations.
While going through french translations, I fixed mistakes that changed the original meaning of the string. For example, "noon" and "midnight" were inverted. Also the case for the totalMontlyExpense translation : it was translated as income. And finally, "confinement" was translated as childbirth.
I fixed these errors, but I don't know if it's really a good idea, because it can be really disruptive for the french speaking users. What do you think?
cc @HospitalRun/core-maintainers

I would like to work on improving the Spanish translations as a project for my Software practicum at College of Charleston.
As a native spanish speaker with experience in the health care system, I think I could contribute to HospitalRun. I would appreciate it greatly if you agree to let me do this part.

Hello @atochef
No problem, I understand : I can revert the spanish translations commit so you can do it if you want.
I was mainly focused on french translations, but as a spanish speaker, I also checked spanish translations to help me understand the string meaning when needed, and filled the empty translations as I was going.

Hi @stukalin, what's the current status of this PR?

@Raul6469 it was accepted. I can't merge PRs myself, so let's bother @tangollama by mentioning him here :stuck_out_tongue:


[move cli] support calling public(script) functions

Bringing the CLI up to date with new Move features. This turned out to be a fairly large task that required a
number of semi-related CLI changes:

Make move view support viewing scripts, and also support viewing modules outside of the storage dir
move run can now take either a script source file (which is compiled before running it) or a script bytecode file as input.

Now, the actual improvement: move run can also take a path to a module bytecode file (must be published under the storage dir) + a function name as input. The command grabs the published module, looks up the function name, and calls the relevant function if it exists + has script visibility. I chose this because the alternatives have some downsides:

A fully qualified function name (i.e. address, module, function name) as input, but that is verbose to type + is quite different from what move run requires for a script (a path to a valid file). I am not opposed to adding this in the future, but seemed like a bad starting point.
A module source file. The problem is that the source file needs to be compiled and published before running the script function (e.g., because it may create resource types declared by the module). However, when you attempt to move run a script that depends on unpublished modules, it will fail to compile or verify (i.e., not compile and publish the missing modules!). It seems like move run should behave in the same way when you pass it a module, which can't happen if we accept source files.

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/diem/diem/tree/main/developers.diem.com, and link to your PR here.)
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

This sounds very convenient, but enforcing the uniqueness sounds potentially hard (e.g., I recompile a script file to rename a script to foo, but some other script file I didn't recompile already contains a script named foo).

You would basically be calling move_compile on src/, reading over the compiled_units and bailing if there is more than one script with that name. That flow might be a tad easier than eagerly enforcing uniqueness

This is what the PR adds, minus the .move part + forces you to say --script-name . I'm not sure if you can configure structopt to let you drop the --script-name part unless every invocation of run requires it.

I think we are still slightly off here. I would not allow --script-name to point to a module's script fun. This would only be used for normal scripts.
Script functions would only be invoked via module run <addr>::<module>::<name>

In avoiding this getting a bit too crazy, and trying to remain consistent with the other commands, I would revise/cut down my previous number of options for the command, and alter it slightly to the following
move run <path_to_source_files> --name (<name>|<addr>::<module>::<name>)

If the source files contains a single script, you can leave off name
Otherwise it will publish <path_to_source_files> and run the appropriate script based on --name
Should simplify some of the parsing issues and reduce the number of different configs while still remaining fairly "magical"

In avoiding this getting a bit too crazy, and trying to remain consistent with the other commands, I would revise/cut down my previous number of options for the command, and alter it slightly to the following
move run <path_to_source_files> --name (<name>|<addr>::<module>::<name>)

If the source files contains a single script, you can leave off name
Otherwise it will publish <path_to_source_files> and run the appropriate script based on --name
Should simplify some of the parsing issues and reduce the number of different configs while still remaining fairly "magical"

I think this flow could make sense in addition to the proposed one. Note that supporting both does not change the number/type of configs; in either case, we need both a file to run and a function name. I think it's important to have good support for the common use-case of calling a function of something that's already published (as opposed to one that you're about to publish) + exploring, then calling using the move view workflow described above.
In addition, I don't want to add publishing via move run in this PR, since I want to make sure it adequately warns you about unpublished local changes (as we discussed offline) + this will be a big change.

Sorry for the late entry, I think I like Todd's suggestion of
move run <path_to_source_files> --name (<name>|<addr>::<module>::<name>)

this <path_to_source_files> does not have to be a single .move file, it can be a directory (or several) (e.g., the move_src) and
everything in the directory will be compiled and (re-)published before executing the script or script function.

In this way, we also solve the problem of what if I changed a module locally but forgot to publish it before running a script that depends on that module.
On the other hand, I also see that we probably don't need to implement it in this PR. This PR is to bring support for calling public(script) functions and I am fine with the changes. We can probably bring the refinement of move run in a follow-up PR?

Sorry for the late entry, I think I like Todd's suggestion of
move run <path_to_source_files> --name (<name>|<addr>::<module>::<name>)

this <path_to_source_files> does not have to be a single .move file, it can be a directory (or several) (e.g., the move_src) and
everything in the directory will be compiled and (re-)published before executing the script or script function.

In this way, we also solve the problem of what if I changed a module locally but forgot to publish it before running a script that depends on that module.
On the other hand, I also see that we probably don't need to implement it in this PR. This PR is to bring support for calling public(script) functions and I am fine with the changes. We can probably bring the refinement of move run in a follow-up PR?

Agreed on all counts! I like the suggestion too, but don't want to add it here.

/land


Add support for setting old save path

I had a need for manually validating the update prior to committing to an update, and the upstream project inconshreveable/go-update has that as an option but it isn't exposed. This exposes it so I can utilize it without customizing things by copying your code into my code base.

Yeah that's a handy addition to the configuration, thank you
I notice my AI code review assistant hasn't spammed you on this one 😆


graalvm-ce: 22.1.0 -> 22.2.0

Description of changes
Update graalvm
ping @thiagokokada

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Right, I forgot it, updated the PR

@ofborg build babashka clojure-lsp jet

@ofborg build graalvm17-ce

babashka is failing to build with the update.

Probably we need to wait for a new babashka release for now: https://github.com/babashka/babashka/pull/1325

Right, let's wait for a new babashka release, probably will be a short wait XD
If I'm not mistaken, we are building babashka with graalvm 17, should we change to 11, since it's the version used in the babashka build script?

Right, let's wait for a new babashka release, probably will be a short wait XD If I'm not mistaken, we are building babashka with graalvm 17, should we change to 11, since it's the version used in the babashka build script?

I don't think so. In all packages:
  buildGraalvmNativeImage = callPackage ../build-support/build-graalvm-native-image {
    graalvm = graalvm11-ce;
  };

And babashka doesn't override it: https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/interpreters/clojure/babashka.nix

thanks for the info, I missed that in all-packages.nix

@ofborg build babashka clojure-lsp jet graalvm17-ce


Decision Tree- Documentation

issue #54

@aishwaryachand Looks pretty clean and in line with suggested documentation method. @geekquad what do you think?


Test current builds on macOS Monterey

Fire up a developer or public build of macOS Monterey and make sure we can continue to run Chef Infra Client. Pay particular attention to the data returned by Ohai and core Infra resources.

This is done now


When Nans are stored in TBranch, VS code does not display the branch at all

When a TBranch has a value of Nan stored (due to failing fits, etc), VS code does not display the branch at all.

To get around this one has to apply a cut TRUEPT != TMath::QuiteNan().

I believe this is not an issue when plotting with plain ROOT TBrowser but when displaying within VS code.
Would appreciate it if this could be fixed.

Hi @abhijitm08 and thanks for opening the issue! Could you provide a sample ROOT File so I can reproduce this? Thanks!

@AlbertoPdRF Thank you for the quick response. The ROOT file is at the CERN box. I just noticed that the CERN box ROOT viewer has the same behaviour as VS code browser.
When I do: tree->Draw("Branch1"), I get:

When I open TBrowser t, I get:

In VS code and the CERN box ROOT viewer I get:

I can also replicate the issue online in https://jsroot.gsi.de/latest/api.htm#url_syntax_open_file, so I'm CC'ing @linev.

Yes, NaN values where not handled correctly with TTree::Draw.
Now fixed in dev and 7.4 branch.

Yes, NaN values where not handled correctly with TTree::Draw. Now fixed in dev and 7.4 branch.

Thank you for the reply. Would be awesome if this gets fixed in the new release of root-file-viewer.

Now available with https://github.com/root-project/jsroot/releases/tag/7.4.2 release

Awesome @linev! I'll publish a release with JSROOT v7.4.2 soon 🚀


Updated copyright dates.

Just some boring maintenance.
I started working on another PR, and needed to create a file ... so I copied the file header format from another file and noticed the old dates. Now here we are ;-)

The two failures on Travis are the ones we know about (they've been reported on other tickets). All the other versions are passing CI/CD.

@rvirding okay, I've rebased this on your latest switch to the AST-based LFE. Ready for review!

Another branch that had these changes in it was merged, so this can just be closed.

Oh,  guess not -- reopening ;-)
(I'll rebase to fix the conflicts ...)

Okay, I've fixed the conflicts.