Add additional country - Finland

Is your feature request related to a problem? Please describe.
We are currently working on including Finnish data in the model. We would be happy to contribute this to your repo. How should we do that in the best way?
Describe the solution you'd like
I would need information on what files (csv) needs to be updated and how.  ECDC data already have death counts for Finland, so that is already handled.
I guess the following files need to be updated:

interventions.csv when interventions have been done.
popt_ifr.csv: I'm not totally sure about this. Is this computed based on age groups outside the repo, since the file contain one value per country?
ages.csv. This is not superclear to me. Is that the age groups sizes in 1000s?

With kind regards
Måns

I added Finland to the version 1 of the model in a quick and dirty way last week. Based on that:

Yes. The report and the web page can be helpful for deciding what constitutes as which intervention. We're still debating what counts as the lockdown date for Finland though.

The model uses on IFR per country. The IFR is calculated using data that is not in the repository. There's some documentation on how the calculation is done in the report: It's not empirical IFR from the countries, but rather based on a study estimating true IFR of the disease, weighted by both age distribution and some sort of a contact matrix. Last week I used Swedish IFR on my version 1 calculations as a placeholder, because I doubt the Finnish IFR is heavily different than that of other Nordics.

It is in thousands as far as I know.

For version 1, there were also some minor code changes required for it to support 12 countries.
I'll send a mail to your assumed work e-mail address from my work e-mail!

Do so! See mansmagnusson.com for contact info.

Hi  @MansMeg  and @elmokki, good job. We are currently not in a position to independently add countries. For countries to be added we need to make sure all of our assumptions and data match. I am happy for you guys to work independently but work for a new country can't be just added because we also want to make sure IFR and age distribution we use are correct and verified.

Ok, so how do we help you to do this verification. We are working with this from Finland so we could do the verification you need, I suppose?

It is mainly working with the people who have better understanding of estimating IFR from contact matrices and other is how much data is really there to estimate with confidence.

Would you happen to have a reference for estimating IFR from contact matrices?

Hi @elmokki  as it is said in repository we are using verity.et.al as our reference for IFR and contact matrices. Just that the report doesn't really have it for those countries, which means we then talk to them and work with them to get it correct.

@s-mishra thanks! I seem to have missed that reference. I'll look into it!

I do think I can actually do that estimation, if you are interested. Or are you in contact with others in Finland currently?
Although, I have no intention to push this if you are not interested. It was just that it was explicitly stated that you welcomed additional countries as contributions.

Sorry, I missed the last comment. So if I understand it correctly, the popt_ifr file contains values of ifr from the verity et al paper. If they dont have the estimate, they need to supply it to you for you to be able to include another country?

Hi @MansMeg, sorry if I sounded rude. I am just saying even if you do the update we might not be able to get it in the main branch without going through it in detail. However, if everything works we will add. I just need to see Finland data to be sure we can act on it.

No problem, I fully understand that you would not blindly accept a PR from random people.
So, as mentioned, we are multiple people that want to include this for Finland. I suggest we do that and open a PR. Then you can look at it when you have time.
For us to add Finland for our sake, we need to know:

popt_ifr.csv: This is the ifr from the verity.et.al paper? You are in contact with authors to get these values computed for all countries? Hence you do not want us to supply these data in a PR?
ages.csv. This is age groups sizes in 1000s?

In the PR:

Is there anything you need to simplify checking of our data?


Search engine frontend now live.

LGTM. Thanks!

Very cool @dokterbob!
Where is the source code for the frontend located?

For now it is at https://github.com/dokterbob/ipfs-search-front/
Should be integrated with the main repo soon, or split out and decently organised. Don’t know yet. :)


Error while runnig Oryx  batch

Hello,
I'm using Oryx 2.5 on CHD 5.12 , when i tried to run oryx-run.sh batch i got the following error :
spark2-submit --master yarn --deploy-mode client      --name OryxBatchLayer-ALSExample --class com.cloudera.oryx.batch.Main  --files oryx.conf      --driver-memory 1g --driver-java-options "-Dconfig.file=oryx.conf"      --executor-memory 1g --executor-cores 2      --conf spark.executor.extraJavaOptions="-Dconfig.file=oryx.conf" --conf spark.ui.port=4040 --conf spark.io.compression.codec=lzf --conf spark.logConf=true --conf spark.serializer=org.apache.spark.serializer.KryoSerializer --conf spark.speculation=true --conf spark.ui.showConsoleProgress=false --num-executors=2 oryx-batch-2.5.0.jar
bash: /usr/bin/spark2-submit: cannot execute binary file

Can you spark2-submit anything? that seems to say the script itself isn't executable. It also isn't where the script would live CDH. Did you actually manually install it?


on basics/structs

maybe example from here are more clear (https://developer.apple.com/library/ios/documentation/Swift/Conceptual/Swift_Programming_Language/ClassesAndStructures.html)
(reported by @claudiug)

This issue was moved to dlang-tour/english#40


Respect num_eval_samples in predict method

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-103/1 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-103/1/index.html

Job PR-103/2 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-103/2/index.html


added PartiallySaturatedBC

Description
closes #160
Checklist

[x] This pull request is associated to an issue
[x] This PR contains a description
[ ] Did you add a new method? If so, you need to

[x] add method description
[x] maybe mention class in the corresponding __init__
[x] add an example using the method in examples/advanced_flows/ or examples/simple_flows/
[ ] add a test in tests/

[ ] Add someone else as reviewer and wait for approval before merging.

Hello @PhiSpel! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file examples/advanced_flows/PartiallySaturatedObstacle.py:

Line 20:1: E302 expected 2 blank lines, found 0
Line 58:1: E302 expected 2 blank lines, found 0

In the file lettuce/ext/_boundary/partially_saturated_boundary.py:

Line 7:1: E302 expected 2 blank lines, found 1


[Suggestion] Don't close the grid if the stack is empty but the ravenmind isn't

I often start staff-casted hexes by putting something in the ravenmind, only for the grid to close, followed by a facepalm, then drawing the same thing again but preceded by Nullary Reflection this time. It'd be nice if the auto-grid-close could respect the ravenmind as well.

In more general terms, it should only close and clear the grid if the entire "program state" of the hex is empty. This is doubly true if the ignored Ravenmind Iota then gets cleared BECAUSE it was ignored.

I think @macks2008 that should be a different issue. But yeah it is a good point.


"ResourceWarning: Unclosed socket" while stopping generator iteration

I would like to exit the get channel generator after I have reached the video I want. However, it seems there is no safe exit, a socket is left open. Below is the code:
import scrapetube

channel_videos_generator = scrapetube.get_channel(channel_id=channel_id, sort_by="newest")

for video in channel_videos_generator:
    if video["videoId"] != since_video_id:
        yield video["videoId"]
    else:
        channel_videos_generator.close()

When I run the above code, I get the warning below:
ResourceWarning: unclosed <ssl.SSLSocket fd=756, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=0, laddr=('<IP_ADDRESS>', 58222), raddr=('<IP_ADDRESS>', 443)>
  channel_videos_generator.close()
ResourceWarning: Enable tracemalloc to get the object allocation traceback

thanks for reporting!
i had to add this to reproduce, i guess you were running it with some sort of testing
import warnings

warnings.filterwarnings(action="always", message="unclosed", category=ResourceWarning)

It's true, I have incorporated testing in my codebase. I filtered the warning. I went through scrapetube's source code and it seems a session in a while loop might not be closed after exiting abruptly; or maybe the warning is only raised during testing. Thanks though, in case the problem is inherent, and you consider fixing it, I will appreciate.


Avoid upper_bound call in DelayedBind look-up with a workspace

Should be possible to store the index of the last query in the workspace, along with the index of the corresponding submatrix. Then the current query can check whether we're in the same segment, and if so, use the submatrix index.

Closed by d0dfd2a152f8182cc788b6f2b782d0808054cee0.


Merge specular_weight and specular_ior_level

We had much discussion on Slack about how the layering (e.g. of the specular layer on the diffuse base) should be implemented in code, and described physically in the spec. We eventually came up with a proposal for a modification that clarifies and simplifies the model: to reinterpret specular_weight as the existing specular_ior_level, thus merging the two parameters and removing the latter from the model. (Also, removing coat_ior_level).
We originally settled on retaining both specular_weight and specular_ior_level from the ADSK and Adobe models as a sort of compromise solution. It seems though that it would be an improvement to merge them, both from the point of view of simplifying the user experience, and clarifying what the parameters are supposed to correspond to physically for implementers.
I try to summarize the discussion below (the background from Slack, then the proposal).

Background
Currently in the spec, we say about specular_weight that it functions as follows:

The specular_weight and specular_color parameters modulate the Fresnel factor of fdielectric.  ... The light transmitted through the dielectric will be compensated accordingly to preserve the energy balance (thus generating a complementary color if specular_color is not white).

As discussed in https://github.com/AcademySoftwareFoundation/OpenPBR/issues/145, this intepretation of specular_color is problematic since it implies there will be a complementary color.  So we will need to modify this to say that the specular_color only affects the reflection Fresnel factor, not the transmission.
But we also need to clarify what specular_weight does (in code, and physically). It actually doesn't make sense for specular_weight to only multiply the reflection Fresnel factor, since this would mean there would be darkening of the base even when specular_weight  goes to zero. For example, in the case of the glossy-diffuse layer, this is supposed to be a layer of dielectric on top of a diffuse base. If specular_weight only modulates the reflection Fresnel factor, then dialing it to zero still generates darkening of the base due to the internal reflections in the layer, which are explicitly unaffected. In terms of the albedo-scaling approximation, specifying that the weight only multiplies the reflection Fresnel factor implies:
f_{\mathrm{glossy-diffuse}} \approx W \mathbf{C} f_{\mathrm{dielectric}} + \left(1 - E_{\mathrm{dielectric}}\right) \, f_{\mathrm{diffuse}}

where specular_weight $W$ and specular_color $\mathbf{C}$ multiply the first term. Then the $\left(1 - E_{\mathrm{dielectric}}\right)$ factor remains in the second term as specular_weight  $W \rightarrow 0$, which produces darkening.
In standard surface, instead what we had was a formula like:
f_{\mathrm{glossy-diffuse}} \approx W \mathbf{C} f_{\mathrm{dielectric}} + \left(1 - W \mathbf{C} E_{\mathrm{dielectric}}\right) \, f_{\mathrm{diffuse}}

which ensures that as  $W \rightarrow 0$, only the undarkened $f_{\mathrm{diffuse}}$ remains. Except, this generates the complementary color tint of the diffuse lobe.
It was proposed to alter this to (e.g. this is how MaterialX implements their dielectric layering):
f_{\mathrm{glossy-diffuse}} \approx W \mathbf{C} f_{\mathrm{dielectric}} + \left(1 - W E_{\mathrm{dielectric}}\right) \, f_{\mathrm{diffuse}}

Physically, this can be interpreted as meaning that the specular_weight is functioning as the presence weight of the dielectric layer (this interpretation leads to the formula above, in albedo-scaling approximation, as is easy to prove).
This makes some sense for the glossy-diffuse part of the specular lobe, but not really for the subsurface and transparent base, where the dielectric base is supposed to be the semi-infinite bulk. Putting this in a statistical superposition of present and absent is physically dubious (it also was for the glossy-diffuse layer really, as the dielectric was supposed to embed the diffuse medium, not just sit on top of it). So to use this presence interpretation of specular_weight we would have to specialize that to the glossy-diffuse case only, and say something different for the subsurface/transmission (e.g. that specular_weight reverts to being a non-physical multiplier of the Fresnel factor).

Simplification proposal
Rethinking the issue, Peter and I propose that we can achieve the desired the behaviour more simply just by having specular_weight function exactly as specular_ior_level does now, then omitting the latter parameter. That is, the specular_weight specifies a multiplier of the reflection Fresnel factor, achieved by modulating the IOR of the entire dielectric base.
The corresponding albedo scaling formula will be:
f_{\mathrm{glossy-diffuse}} \approx \mathbf{C} f_{\mathrm{dielectric}} + \left(1 - E_{\mathrm{dielectric}}\right) \, f_{\mathrm{diffuse}}

as now the specular_weight  $W$ functions by altering the IOR of $f_{\mathrm{dielectric}}$ (thus the Fresnel factor). As $W \rightarrow 0$,  the specular BRDF $f_{\mathrm{dielectric}} \rightarrow 0$ automatically, and thus its reflectance $E_{\mathrm{dielectric}} \rightarrow 0$.
We would retain specular_color $C$ as a non-physical tint of the reflection Fresnel factor.
This approach:

makes the reflection Fresnel factor go to zero as specular_weight $\rightarrow 0$, but does this in a more physically correct way, keeping the grazing angle highlight but having it narrow to zero width, rather than just killing the highlight.

makes the physical description clearer in the spec, and obvious how to implement. The dielectric base is now always present (embedding the media described by the transmission, subsurface, and diffuse slabs). The specular_weight is just modulating the IOR of this dielectric base.

removes a parameter which was practically redundant (as specular_weight and specular_ior_level had a very similar effect, apart from the former being less physically plausible with a damped highlight). The more obscure sounding specular_ior_level was likely to be a source of confusion to artists, especially given its extremely similar behaviour to specular_weight .

retains the ability to non-physically modulate the reflection lobe color/intensity, independently of the transmission, via specular_color. (So no functionality is actually lost).

It is optional whether we want to have the specular_weight increase the Fresnel factor at the top end of the range (e.g. default at 0.5, and max out at 1, where the Fresnel is doubled, as the current specular_ior_level works) or not. It would be reasonable to just omit this, and have the weight default to 1, and only decrease the Fresnel.
Additionally, we propose to remove coat_ior_level, as it is functionally equivalent to coat_weight (the presence weight of the coat) for the purposes of modulating the coat reflection strength. Removing this also simplifies implementation of the coat lobe.

An specific side effect of the new proposal would be that it would eliminate any nonphysical mismatch of the Fresnel reflection and the corresponding Fresnel transmission. Any misalignment can be particularly noticeable when going from high IOR to low IOR due to the visible discontinuity where total internal reflection begins.


cmdline_args failing to build when using C99

When adding to project.yml
:test_runner:
  :cmdline_args: true

to a newly created project, the project fails to build when using C99:
Test 'test_thing.c'
-------------------
Generating runner for test_thing.c...
Compiling test_thing_runner.c...
build/test/runners/test_thing_runner.c:77:22: error: implicit declaration of function 'UnityParseOptions' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
  int parse_status = UnityParseOptions(argc, argv);
                     ^
1 error generated.

The test running is generating:
/*=======MAIN=====*/
 int main(int argc, char** argv)
{
  int parse_status = UnityParseOptions(argc, argv);
  if (parse_status != 0)

However no declaration is being including in the runner source file, and the prototype for UnityParseOptions is not part of Unity.h

Hi @nscooling,
Can you share with me your project.yml and if it is this possible project, so I will be able to reproduce the issue, and fix it.
As well, can you tell me, on which commit/branch you are working on?

Hi @lukzeg - Here is a repo that causes the issue:
github link
Version detials
$ ceedling version
Welcome to Ceedling!
    Ceedling:: 0.31.1
       CMock:: 2.5.4
       Unity:: 2.5.4
  CException:: 1.3.3

macOS Ventura 13.2.1 (22D68)
$ gcc -v
Apple clang version 14.0.0 (clang-14<IP_ADDRESS>)
Target: x86_64-apple-darwin22.3.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

$ ruby -v
ruby 2.6.10p210 (2022-04-12 revision 67958) [universal.x86_64-darwin22]

Hi @nscooling ,
My apologies for a silence from my side. Today I tried to reproduce your issue, but to be honest, on my Ubuntu 24.04 I'm not able to reproduce it.
The project which you shared it works perfectly fine for me.
I as well checked example app under Unity submodule:
https://github.com/ThrowTheSwitch/Unity/tree/master/examples/example_1
where I updated makefile, to fit your needs:
https://github.com/ThrowTheSwitch/Unity/blob/master/examples/example_1/makefile#L30
by updatting -std=c89 to  -std=c99 and adding -DUNITY_USE_COMMAND_LINE_ARGS and everything works perfectly fine.
Can you check wherever your vendor/unity submodule is up to date?
Can you as well try to build with verbose logging option:
ceedling verbosity[4] logging test:all

And confirm wherever the -DUNITY_USE_COMMAND_LINE_ARGS is visible in your  compilation process? As the error mostly inform that this flag is missing.

I was able build this on my Ubuntu 22.04, where:

vendor/Unity  -> 5a36b197fb34c0a77ac891c355596cb5c25aaf5b
Ceedling -> db98f857cbebe80af55a8756720928c90a248c2b

and I'm using gcc -> gcc (Ubuntu 11.3.0-1ubuntu1~22.04) 11.3.0.

This has been fixed in the release candidate. It just hasn't been made official yet. To manually fix it yourself, add UNITY_USE_COMMAND_LINE_ARGS to your project.yml defines list. In the future, this will be automatically added when this feature is enabled.

I added
  :common: &common_defines      - UNITY_USE_COMMAND_LINE_ARGS
can confirm that does remove the error. I still have issues where it does not allow me to select a test case but it doesn't fit in this issue.
Thank you

We are nearing an official release of 0.32. I can confirm that :test_runner ↳ :cmdline_args works as expected and works without issue when -std=c99 is added to test executable compilation.


Gemini unary API fixes and natural conversation demo

Added direct setting of tools and tool_config to Gemini unary API.
Fixed token usage counting for Gemini unary API
Reworked Gemini native audio in natural conversation (smart turn detection) demo

LGTM!


[BEAM-13641][Playground] Add SCIO SDK support on the CI/CD step

[BEAM-13641]
Add SCIO SDK support on the CI/CD step

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @KhaninArtur @ilya-kozyrev

R: @pabloem

LGTM


error reading values after  on line 32 in

I'm using gulp-sass and gridle and using the following task
gulp.task('styles', function() {

    var dir = config.dist + '/css';

    return gulp.src(config.scss) // src/styles/main.scss
        .pipe(sass({
            sourceComments: dist ? 'none' : 'map',
            sourceMap: 'sass',
            outputStyle: dist ? 'compressed' : 'nested',
            includePaths: ['node_modules/gridle/sass']
        }))
        //.pipe(autoprefixer("last 2 versions", "> 1%", "ie 8"))
        .on('error', handleErrors)
        .pipe(gulp.dest(dir))
        //.pipe(gulpif(browserSync.active, browserSync.reload({ stream: true })))
    ;
});

if the option errLogToConsole: true i the a  error reading values after  on line 32 in node_modules/gridle/sass/gridle/_common-mixins.scss
but i dont understand why running the command sass src/styles/main.scss -Inode_modules/gridle/sass' works fine

I'm also getting this. I'll send a pull request with a failing test case—I have no idea how to fix it, sorry.
Broken:
'use strict';

var gulp = require('gulp');
var sass = require('gulp-sass');

gulp.task('default', function () {
  return gulp.src('./test/fixtures/sass/test.sass')
    .pipe(sass())
    .pipe(gulp.dest('output'));
});

Working:
'use strict';

var gulp = require('gulp');
var sass = require('node-sass');

gulp.task('default', function (done) {
  sass.render({
    file: './test/fixtures/sass/test.sass',
    success: function () { done() }
  });
});

The problem is in gulp-sass.

And test.sass contains only this:
$color: blue

body .div
  color: $color

As far as I can see, that is perfectly valid Sass.

Thanks for putting in the effort to write a failing test. I'll take a look at this a little later.

Just ran that test into the 2.x branch and that seems fixed, I added that example as a test in https://github.com/dlmanning/gulp-sass/pull/222


overlay: fix cleanup directory deletion race

Fixes a bug where a writable transaction may create or make changes to a directory while the cleanup is running, leading to removal of a directory which will be referenced.
fixes #2210

The test failure is still https://github.com/containerd/containerd/issues/1545

Can we merge this?

1 LGTM, you can make it 2 and hit merge :)

LGTM. :)


add dependency_usage field

Please add a top-level field called dependency_usage to the job entity. It's optional and of type string.
Context: https://github.com/eu-cdse/openeo-cdse-infra/issues/261
Thanks!

@bossie  - Can you also provide some insights if you want this field to be searchable?

Not searchable.


resolve_coref option for openie

Hi,
I would like to know if there's an option for coreference resolution in the python wrapper of OpenIE, similar to the Java implementation.
Thx!

Update: I was able to pass the exact list of annotators to the StanfordNLP instance like this:
    triples = client.annotate(corpus,                              properties={                             "annotators":"tokenize,ssplit,pos,lemma,depparse,ner,coref,mention,natlog,openie",                             "outputFormat": "json",                             "openie.format": "ollie",                             "openie.resolve_coref": "true",                                                           })
However, no matter what kind of annotators I add, I get the following error:
AnnotationException: java.lang.IllegalArgumentException: annotator "openie" requires annotation "CorefChainAnnotation". The usual requirements for this annotator are: tokenize,ssplit,pos,lemma,depparse,natlog
I can extract triples without issues if I put the "openie.resolve_coref":  to false.
There must be a connection with this issue: https://github.com/stanfordnlp/CoreNLP/issues/789
Has anyone figured out how to overcome this problem?

@lighteternal yep that's the correct way to use it. But it seems like you're trying to do something that the lib does not really allow. Maybe it will be available with the 4.0.0.
You'll have to change the version here: https://github.com/philipperemy/Stanford-OpenIE-Python/blob/f01a302878ee584870276a63fcac4aff925de087/openie/openie.py#L13
Let me know :)

Thanks for the feedback! Closing this and will update when v4 becomes available.

Thank you!

any update on this

Yup, stumbled upon this too... Any news?


touch, date: Replace humantime_to_duration with fundu

Like I mentioned in another issue a while ago, fundu since version 1.0.0 is capable of parsing gnu relative times as described in the gnu documentation. If you're interested you (or I) could replace the usages of humantime_to_duration::from_str in touch and date with fundu.

sorry but what would be the gain?

To be honest, I am more thinking about removing the usage of fundu in coreutils to use our implementation :(

sorry but what would be the gain?

fundu is definitely a lot faster than humantime_to_duration and it's well tested. Please also see the README. Maybe you could also spare yourself the maintenance of another crate.

Was here already something done? When I try to update coreutils I get this error:
Updating coreutils     Updating crates.io index   Installing coreutils v0.0.19     Updating crates.io index error: failed to compile coreutils v0.0.19, intermediate artifacts can be found at C:\Users\xxxxxxx\AppData\Local\Temp\cargo-installawI5rs`
Caused by:
no matching package named humantime_to_duration found
location searched: registry crates-io
required by package uu_date v0.0.19
... which satisfies dependency date = "^0.0.19" of package coreutils v0.0.19
Updated 0 packages.
Failed to update coreutils.`

No I haven't done anything yet. I'm waiting on a response. Even after the rename of the crate the gnu relative time parsing to a Duration could be done by fundu.

Maybe you could also spare yourself the maintenance of another crate.

Well, sorry to be blunt, but given that you are the only contributor to fundu, that it is published on your own github and don't have any other users than the rust coreutils yet, this isn't appealing to me :(

Should this really be done so extreme?
Because I want to make sure that nobody uses it.

What happens if one wants to install an older version?
I unyanked the version version needed by 0.0.19

Hi @Joining7943!
We've discussed among the maintainers and we think we've found a good solution.
Let me first explain why switching to fundu is tricky. First, we need to support a very specific format with 100% compatibility, this makes it very likely that we need to tweak the parsing, which might be hard to do if the changes have to go through you. We'll probably also get the bug reports if something is wrong since people might notice the differences with GNU. Therefore, we'd like to have a crate under the uutils organisation, which is parse_datetime (previously humantime_to_duration). We also do not need the flexibility that fundu provides.
Therefore, we propose that we hard fork fundu and tailor it to our needs under the uutils organisation. We'll credit you of course and keep your git history. That way, you can keep the fundu crate and the flexibility it provides, because it looks very good for many other applications!
We'll probably also use some parsing library, but we'll have to see how that works out. Does that sound good to you?

@tertsdiepraam Looks like we've commented simultaneously

Let me first explain why switching to fundu is tricky. We need to support a very specific format with 100% compatibility. This makes it very likely that we need to tweak the parsing, which might be hard to do if the changes have to go through you. We'll probably also get the bug reports if something is wrong since people might notice the differences with GNU.

I actually checked compatibility with gnu (relative time parsing) and haven't found any issues. If there are any, I would be interested, too :) Maybe have a look at the gnu example in the examples folder. However, maybe you want to wait for v1.1.0 (it should be ready soon), I'm in the process (currently on the develop branch) to expose the core parser in a separate feature, so you don't have to use the standard, custom feature and on the other side have influence on the parsing of time units. This'll make the creation of the parser possible at compile time as const removing the initialisation time which is not const from the parsing time, makes the parsing of the time units slightly faster and also lowers the final binary size a little bit, if that's a concern. Like I said in the comment before if you have ideas, feature requests, questions, compatibility issues etc. feel free to create an issue on fundu's github page.

Therefore, we propose that we hard fork fundu and tailor it to our needs under the uutils organisation. We'll credit you of course and keep your git history.

Honestly, I'm not sure which parts you want to tailor to make it more compatible with gnu but feel free to fork it.

Alright! Thank you! We'll wait for v1.1 and try to fork from there.

Or would you be ok to move fundu in uutils and maybe we can play with rust features to have it only what we need for it?

Thank's for your offer, but I would rather keep fundu as it is for now. I created an issue https://github.com/Joining7943/fundu/issues/22 for you. I'm currently restructuring the project anyways and I'll create a package fundu-core together with fundu's main package v1.1.0 release. I'm not sure but I guess fundu-core is what you wanted, although I still think it's easier and also have 100% compatibility with gnu if you would use one of the facades from the custom or the (currently in development) base feature, which I described earlier in the comment above. Like I said before, feel free to join the discussion at fundu or create an own issue if you have any questions, feature requests etc.

Thank's for your offer, but I would rather keep fundu as it is for now.

Sure I understand why you want this.
But given you are the only maintainer/contributor, it adds risks for coreutils that I am not sure I want to take currently.
Sorry

I definitely won't stop anyone from contributing. Although, due to the restructuring of the project it's not the best time to do so. But, as soon as the big restructuring is completed (v1.1.0) it's a great to time to make contributions. Also, if these are just tests to guarantee a specific property. For example, I wrote tests to check against the gnu format but I haven't marked them specifically, so they are included in the rest of the tests ecosystem (which by the way have 100% coverage). Fundu's also tested on critical systems and I'm running fuzzy tests pretty much from the beginning.

Should this really be done so extreme?

Because I want to make sure that nobody uses it.

What happens if one wants to install an older version?

I unyanked the version version needed by 0.0.19

That action fixed it:
   Installed package `coreutils v0.0.19` (executable `coreutils.exe`)

well said @piotrkwiecinski

@piotrkwiecinski Can't whoever who wants to continue development just fork it in any of those scenario? I don't see the issue as long as the current version is well maintained.

Using fundu's parse_with_date would fix #4811.

We have made the decision to expand our own parse_datetime crate instead. We would happily accept fixes for #4811 there. Thank you all for your input. I'll close this issue now.


Add amount formatting to TECurrency

Due to the nature of the Economy Service API, Currency#format(BigDecimal) should format the given number into a Text object used to display the amount. This means that it should also include the currency symbol (which I have added in this PR).
I've also added a method call that will format the amount using commas. For example, BigDecimal.valueOf(65536D) will be formatted as "65,536" and BigDecimal.valueOf(4392283.37D) will be formatted as "4,392,283.37". This makes the number easier to read when displayed.
I used the English locale as other languages may display numbers differently. For example, the French locale would format the number BigDecimal.valueOf(4392283.37D) as "4.392.283,37". You could go so far as adding a locale option or using the server locale, but that's up to you.

I love this! Thanks for the help. Will merge it sometime today or tomorrow.


reload everything after everytime i used inpainting

thank you for you work, this one works terrific!!! much better than the old workflow by use SAM
The problem I fond is that every time after i used inpainting, the whole webui reload everything again, even though it doesn't take me too much, but still a bit annoying~ is this my problem or this plugin has some other settings i don't know?

The Inpainting, Cleaner and Segment Anything functions do not utilize the Stable Diffusion checkpoint, located in the upper left of the Web UI, and as such, it is intentionally unloaded and reloaded in the program. Therefore, this is a standard log.

okay,i see, thank you~

Unloading and reloading the model is performed intentionally to free up VRAM. Therefore, this issue will be closed.


rowHeight larger than 70dp does not work well

First of all, great library!
There seem to be a problem with rowHeight larger than 70dp. The indices seem to be misplaced and laggy, or "jumpy"

also, why is it demanding minSDK 22? I downloaded the project and it works fine with lower versions (checked 15)

Hi @gilgil28, I'm glad that this library is somehow usefull to you. The problem with the height, I believe, is being caused due the way I am recoverying the dp value and setting it to the height (I might need to use ´´´TypedValue.COMPLEX_UNIT_PX´´´ when setting the new layout height). The min sdk version is due my creation process in AS, I likely forgot to change it before uploading the file. I took note of our comments and once I have sometime (perhaps today yet) I'll try to fix it and make a new release. Thank you for your comments

Hi @gilgil28 , I believe your bug is fixed in the verion 1.2.0. Can you please try it out and tell me the result?

thanks for the update! can you make the minimum SDK version lower tha 22? maybe 15?

I tried it and the sticky letters are all gone, appear only sometimes. probably a problem with the alpha parameter

Hello @gilgil28 , I am sorry for the trouble but all my tests are passing now. You were right about the alpha, however it wasn't easy to spot a solution for it - I hope now it works fine. I released a new version with this bug fix 1.2.1. Just tell me if it works now. Also, not that I've added the new attribute app:stickyWidth that allows you to control the distante between the text and the parent corner (details in the read-me file).


Remove string array for shortName alias

The upgrade to the new SDK is still blocked. Reopening #325

FYI @ryanbrandenburg I will merge once Appveyor/Travis run

@jbagga : This passed on Travis/Linux, but failed on Travis/Mac and AppVeyor.  Think it's safe to merge to unblock the Templating repo?

@mikeharder Actually @ryanbrandenburg unblocked the SDK upgrade from Update Universe which means I do not need to merge this in at all. With the new SDK there should be no failrues. Right now I am just waiting on approval for https://github.com/aspnet/templating/pull/474 which should unblock Templating.

Closing as the new SDK is now being consumed by our builds and there is no need to revert the use of string array in the template.json


RFTOOLS Spawner cannot handle VINDICATOR

mcjtylib-1.1x-2.3.7.jar,rftools-1.1x-5.83.jar,1.11.2-forge1.11.2-<IP_ADDRESS>6
Shown in Spawner (gui)  nullRF ,nothing else
Please fix, they will drop emeralds
regards

The spawner only supports mods that have been added to the config. You need to add several entries to the config for this

something like this

S:"minecraft:vindicator.spawnamount.0" <
I
minecraft:emerald
0
0.1
>
S:"minecraft:vindicator.spawnamount.1" <
B
minecraft:log2
0
0.2
>

regards

Should be fixed now I think


Create and manage a pool without any VIP request association

Today, when we allocate a VIP requisition, it creates one server pool for each group of servers that attend the same VIP port.
We need to be able to allocate these server pools "request" apart from VIP request. We need similar form, but to be able to allocate a server pool by itself, not connected to any VIP request. We also need the server pool to have this healthcheck configuration by itself, today the healthcheck configuration is a feature of the VIP request (one per VIP).
We need the same kind of functionalities that we have in the VIP request:

We need to be able to deploy(create)/delete this server pool in the LB. If the server pool is associated with any VIP request, the API should deny the deletion.
We need to be able to add, delete, enable and disable a member (server:port) in the server pool
We need to be able to check the status of a member (server:port) in a server pool. This status should be checked in the load balancer by the external plugin (scripts).

Available in 2.4.0


Check zswag compatibility with OpenAPI YAML

Current State: Currently, the zswag server doesn't check if a present OpenAPI YAML is usable because of version compatibility.
Desired Behavior: zswag check if a present OpenAPI YAML is usable (for example zswag could put the version that has been used to generate the OpenAPI spec directly in the YAML file) - zswag should output a message if the OpenAPI spec is not compatible.

The zswag OpenAPI parser and the server now generate very detailed error messages when an incompatible OpenAPI feature is used. Does that cover the requirement?


portal-bridge: Add support for beacon light client data

What was wrong?
We want to seed light client data into the beacon network. Current portal-bridge functionalities do not support beacon data.
Closes #820
How was it fixed?

Add a  BeaconBridge instance with functionalities related to the beacon network.
Based on a network cli flag, the bridge launch beacon or/and history bridge instances on separate tokio threads.
Support only the latest mode for now. Test mode and peertest scenario will be added in another PR because it requires some additional refactorings which will extend the scope of this PR (#848 ).
Add support for the following light client data: LightClientUpdatesByRange, LightClientOptimisticUpdate, and LightClientFInalityUpdate.  Support for LightClientBootstrap will be added in a following PR (#849) because it requires some more research to find the most feasible way to deliver the latest bootstrap available.

Known issues

https://github.com/ethereum/trin/issues/850

To-Do

[ ] Add entry to the release notes (may forgo for trivial changes)
[ ] Clean up commit history

From a high level, the main thing I noticed was a lack of tests. Is there really nothing that makes sense to test here?

I don't have great ideas other than adding an integration test in ethportal-peertest #848 . Most of the data is based on an internal clock and requests to pandaops. I guess we can mock the ConsensusApi but then I'm not sure what benefits we gain in regard to testing.

🌉 I'm not sure what your plan is for getting this live. Do you want to use this feature in our live network bridge nodes? Or keep it experimental & just use locally until it's a bit more battle-tested?

My plan is to first test it locally for a bit and then push it to the live bridge nodes.


Add Gemfile and Gemfile.lock

We don't need to do this here. The CI does all these. We shouldn't be putting clide info in public Gemfile.


Fix slow loading of larger conversations.

Initializing GPT3Tokenizer is pretty slow and was done for each message.
Edit: While it works when run with npm dev, if fails to compile: TypeError: GPT3Tokenizer is not a constructor. Maybe someone with more experience in TypeScript knows how to fix it.

Thank you for your contributions, @Schroedi. The TS error is really weird. Imo this is a misinterpretation by the compiler because it makes no sense to me that it works if enclosed in a function. Since we can't fix the export default class in the tokenizer, the best workaround is probably what you already did in your branch: still call the ctor in the function but only if it hasn't been initialized yet. I did that on your PR branch and also added some additional comments and meta stuff (changelog, version).


Fix CDPX build

https://github.com/microsoft/PowerToys/commit/212ea2de30d69dbbfebf86c9d3604008d2272346 renamed settings UI projects paths, but didn't update hardcoded paths in localization script

Since it's not a user facing fix for a released bug, we can close it now.


Pomodoro is not working in any way

Pomodoro is enabled in the "general" tab, in settings. However, its icon does not appear and, when clicking on it, in the panel, there are no option to set or switch to turn it on. It does not matter whether I choose to join menus into a single icon into panel, or if I leave them separate.
$ gnome-shell --version
GNOME Shell 3.28.4
$ cat /etc/issue
Ubuntu 18.04.2 LTS \n \l
$ uname -srm
Linux 4.18.0-25-generic x86_64

What version of timepp are you using?

Version 145. This is what the metadata file looks like:
{
  "_generated": "Generated by SweetTooth, do not edit",
  "cache-file-format-version": {
    "alarms": 3,
    "pomodoro": 3,
    "stopwatch": 4,
    "timer": 3,
    "todo": 10
  },
  "description": "A todo.txt manager, time tracker, timer, stopwatch, pomodoro, and alarm clock",
  "gettext-domain": "timepp",
  "issues_url": "https://github.com/zagortenay333/timepp__gnome/issues",
  "name": "Time ++",
  "original-author": "https://github.com/zagortenay333",
  "shell-version": [
    "3.24",
    "3.26",
    "3.28",
    "3.30"
  ],
  "translations_url": "https://github.com/zagortenay333/timepp__gnome/tree/master/data/po_files",
  "url": "https://github.com/zagortenay333/timepp__gnome",
  "uuid": "timepp@zagortenay333",
  "version": 145
}

Hmm what is that "generated by sweettooth" thing? Did you download from the gnoem site? Try using the version from github: https://github.com/zagortenay333/timepp__gnome/releases
I can't really help you out since I don't use gnome at the moment.

Thanks, it is working flawlessly now! Yes, I had downloaded it from extensions.gnome.org


Error naming convention

Problem
There should be a naming convention to remove ambiguity between errors.
Solution
Use a context-based naming convention like
HuakError::PackageInvalidVersion,
HuakError::PackageInvalidOperator,
HuakError::PackageInvalidName,
HuakError::PackageMissingVersion,
HuakError::PackageMissingOperator,
HuakError::PackageMissingName,

But for everything, not just for Package.

Good enough from #398


Excluding items with 'read' feedback from popular/latest recommendations

Is your feature request related to a problem? Please describe.
I'm building a feed of recommendations (e.g. TikTok) where each item is recommended once only until 'read' feedback. Currently, the popular or latest recommenders don't exclude items with 'read' feedback. Also, when increasing the 'explore_recommend' values, the User-based Similarity Recommender includes the same latest or popular items every time, showing duplicates to user.
Describe the solution you'd like
Config to exclude latest or popular items with 'read' feedback in recommendations.
Describe alternatives you've considered
Tried scrolling through result lists with the cursor option but not ideal because fresh recommendations should be based on user feedbacks as the user consumes the content.

Would be cool to be able to do this by passing the user_id to the endpoint.
For what i reremember popular and latest indexes are not connected to a  specific user.

A query parameter has been added to removed read items for a user. You could preview it in the nightly version or v0.4.10.


Player ghosts in portals instead of actual player

Here is my friend's ghost after he entered the nether 1 min ago:

And here is his screen at the same moment:

You can also see that we cannot see each other at the actual positions.

Oh and as I see right now, the name of him is clipped by the portal. Maybe you know where that comes from and can fix it 👼

Fixed in 0.19


System OutOfMemory Exception thrown in GroupsController

When I try to use the getmembers method, I get a outofmemory exception on the line users.Add((User) directoryObject); I'm not sure what caused this as I had plenty of RAM available

Are you able to reproduce this issue on other machine with same request?

Don't worry. I managed to resolve this issue.


feat: add "Hyper Torture" mode

❗Hyper 🤓 Torture mode active❗
"Infinite loop of despair activated."
"Hyper Torture" mode is like the ultimate test for your brain while browsing the web. Turn it on, and every new website you visit brings up a random LeetCode problem.  And guess what? Open another webpage, and boom, a new problem appears. This loop of challenges won’t stop until you turn off Hyper Torture mode.
But there's more! It counts your winning streak but be careful. It is in Sudden Death Mode. You make a mistake - your streak goes back to zero. It's all about keeping you on your toes and making sure you think hard before you code. Turn it on and off in the settings as you like. Are you ready for the challenge? #HyperTortureChallenge #BrainInPain
How to Toggle On/Off Hyper Torture Mode:

To Turn On:

Go to the settings menu of the extension.
Find and click on the "Hyper Torture" mode option to activate it.

To Turn Off:

If you feel like tapping out, head back to the settings menu.
Click on the "Hyper Torture" mode option again to deactivate it.

What is Hyper Torture:

Task on New URL:

Every time a new URL is opened, a random LeetCode problem pops up.
You cannot navigate the web freely until you solve the presented task.

Dedicated Streak Counter:

The Hyper Torture mode has its own streak counter.
The streak updates only when the Hyper Torture mode is enabled.
Your streak begins when you toggle on the mode and ends when you disable it from the settings.

This feature adds extra pressure and encourages careful and thoughtful coding.

haha this is awesome, go crazy and have fun with it


New cloudfront bahaviour

Hi @danielcondemarin,
I would like to add a custom cloudfront behaviour that would allow me to restrict user access for a given path (trustedSigners: self). Is it possible to do from the component itself? Do you have any recommendation?
Thank you in advance for your help.
Cheers

@bredele Part of the limitation is that the underlying code from @serverless/aws-cloudfront doesn't support customizing TrustedSigners. I opened an issue here:
https://github.com/serverless-components/aws-cloudfront/issues/17
After that's complete, serverless-next.js still needs to allow customizing more options in the distribution and cache behaviors.

In case anyone needs to patch this as a quick fix, the relevant code is
https://github.com/serverless-components/aws-cloudfront/blob/913c4fdbc90c5c8041bafdedcb7e21acbf6b63c2/lib/getDefaultCacheBehavior.js#L18
https://github.com/serverless-components/aws-cloudfront/blob/913c4fdbc90c5c8041bafdedcb7e21acbf6b63c2/lib/getCacheBehavior.js#L21
Then, you'd want to use the local Serverless component code. In serverless.yml:
component: ./node_modules/serverless-next.js


Brian profile

Here is my participant page, let me know if there are any issues.

@infosecdad thx merged
can you add some info to the '(add info, links, blogs, twitter, bio, what would like to see at the summit)' section
thx


[BUG] Package google_api_headers:windows references google_api_headers:windows as the default plugin, but it does not provide an inline implementation.

Steps to Reproduce
Try to compile for iOS
Expected behaviour
No warning message
Actual behaviour
Package google_api_headers:windows references google_api_headers:windows as the default plugin, but it does not provide an inline implementation.
Ask the maintainers of google_api_headers to either avoid referencing a default implementation via platforms: windows: default_package: google_api_headers or add an inline implementation to google_api_headers via platforms: windows: pluginClass or dartPluginClass.
Flutter doctor output
[✓] Flutter (Channel beta, 3.25.0-0.1.pre, on macOS 14.0 23A344 darwin-arm64,
locale en-US)
• Flutter version 3.25.0-0.1.pre on channel beta at
/Users/x/development/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision d3733fcb0e (2 weeks ago), 2024-08-21 10:27:04 -0500
• Engine revision 062b3a72fc
• Dart version 3.6.0 (build 3.6.0-149.3.beta)
• DevTools version 2.38.0
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
• Android SDK at /Users/x/Library/Android/sdk
• Platform android-34, build-tools 34.0.0
• Java binary at: /Applications/Android
Studio.app/Contents/jbr/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build
17.0.6+0-17.0.6b829.9-10027231)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 15F31d
• CocoaPods version 1.15.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2022.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build
17.0.6+0-17.0.6b829.9-10027231)
[✓] VS Code (version 1.92.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.94.0
[✓] Connected device (4 available)
[✓] Network resources
• All expected network resources are available.
• No issues found!
Additional Context
Plugin used by another package (flutter map)

Related to #469

Duplicate of #461


Feature:  Password Generator using python

Hello! @keshavsingh4522, I would like to contribute a password generator using python.

hi @keshavsingh4522  request for assigning this pull. I have a GUI version of this proj

hi i want to contribute in this issue .
please assign this to me


WIP - Fix bugs

This is an attempt at fixing:
https://github.com/silbinarywolf/silverstripe-components/issues/14
https://github.com/silbinarywolf/silverstripe-components/issues/15
https://github.com/silbinarywolf/silverstripe-components/issues/16

This needs more effort:

Tests to ensure $Message works with escaped quotes.

Closed in favour of:
https://github.com/silbinarywolf/silverstripe-components/pull/23
(Other PRs covered off the invalid property issues too)


IE support

Webida does not work on IE (version of supporting html5, over IE10).
It's very critical issue for supporting web standard.
Many people still use IE, especially B2B market.

@bd1978 We have a plan to support cross browsing.

@kyungmi Please put on this issue to top priority. We must support IE until November.

@bd1978 We're discussing about this issue now. I'll let you know the result soon.

Related with #539

@kyungmi Thanks a lot :)


Enabling PECOS for aarch64(arm64) CPU Architecture

Issue #, if available:
N/A
Description of changes:
This PR aims at enabling PECOS for aarch64(arm64) CPU architecture, including:

aarch64 version SIMD functions
Related GitHub Actions CI/CD for:

auto build/publish aarch64 wheels
auto pytest

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Current checked-in branch is a working version that uses third-party header file SSE2NEON to auto-convert x86 SSE intrinsics to Arm/Aarch64 NEON intrinsics.
Please review the high-level refactored C++ header files to make sure this modularization is desirable.
Next step will add:

More versions of SIMD functions for aarch64 intrinsics (e.g. SVE) that provide better efficiency
Pytest for aarch64 architecture

Current checked-in branch is a working version that uses third-party header file SSE2NEON to auto-convert x86 SSE intrinsics to Arm/Aarch64 NEON intrinsics.
Please review the high-level refactored C++ header files to make sure this modularization is desirable.

This part will be moved to a new PR

Next step will add:
* More versions of SIMD functions for aarch64 intrinsics (e.g. SVE) that provide better efficiency

This part will also be moved to a new PR.

* Pytest for aarch64 architecture

Done.


docs(test only): grady's test PR

What I did

Testing Instructions

View DP

Notes to Reviewers

Closing because this was just a test


API retrieved project retired_subjects_count giving wrong value?

Not sure if I'm doing this correctly - the retired subject count returned by the api doesn't match what actually exists in the data
I have 521 subjects that have more than 20 classifications and could be retired out of  8436
I'm getting retired_subjects_count:3903 when I call the api
wget -qO - --header="Accept: application/vnd.api+json; version=1"  https://panoptes.zooniverse.org/api/projects/17  |  jshon -e projects -e 0  -e retired_subjects_count
3903
And in the annotations  csv all the retired states are false.
Is it just retired_subjects count means something different than I think it does?

@mschwamb that larger number seems to have been a remnant of the prev subject sets. I've reset the retired count on that worklow so it should match what you are expecting. For reference the project level retired_subjects_count will summate all your workflows, so you can use the workflow end point to get more specific details.

@camallen  Thanks. I'll try and figure out how to access that part with the api.
~Meg


[FEATREQ] [bash] Add support for colored output

Is your feature request related to a problem? Please describe.
Rather an improvement
Describe the solution you'd like
Add support for ANSI color escape codes on bash script's output

Make it opt-out either via env vars NOCOLOR, NO_COLOR, TERM or --color flag

NOCOLOR, NO_COLOR: See no-color.org
TERM: See gnu.org
--color: implement manually

Describe alternatives you've considered
Remain as-is
Additional context
None

Rust ANSI color output is still pending

Closing this issue and opening one for rust here: https://github.com/Diomeh/dsu/issues/33


Translations main

Merge translations into master
Translations has been reviewed in separate parts.
This only merges into the master branch after fixing the merge conflicts.
TODO: Add the English translations for the Privacy Policy, Terms of Participation and Cookie information

Looks good!


[Specification Update] Observing 404 for forwarding contruct and forwarding-construct-port APIs.

See #26
This issue is for applying the required changes to the specification.

has already been resolved with #38


Starts a contrib module to capture LLM DAGs

This one specifically brings in the text summarization example code and makes it more specific, that given some text, you can summarize it.
The next extension would be to connect to a vector store to retrieve docs/look up which ones to summarize...
Usage example
This would mirror what one does with Langchain:
from langchain ... import NAME_OF_CHAIN

llm = ChatOpenAI(temperature=0, model="gpt-3.5-turbo-0613")

chain = NAME_OF_CHAIN(llm)

result = chain.run(question=question, context=context)

This is what it would look like to an end user to use:
from hamilton.contrib.... import NAME_OF_FLOW

dr = driver.Driver({"temperature":0, "model":"gpt-3.5-turbo-0613"}, NAME_OF_FLOW)

result = dr.execute(["facts"], inputs={"question": question, "context": context})

Changes

adds contrib module
adds text summarization flow

How I tested this

locally

Notes / Things to think about

Rather than having a new repo, I thought putting it into Hamilton would be fine for now.
naming -- can we standardize so these flows can connect? or is that too hard?

Checklist

[ ] PR has an informative and human-readable title (this will be pulled into the release notes)
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future TODOs are captured in comments
[ ] Project documentation has been updated if adding/changing functionality.

TODOs:

make this run in a fastapi.
make this run on spark.

also probably want to figure out a way to evolve the package/changes/complexity of implementation?
Randomly this also made me think about making our examples pip installable too...

This has been superseded -- see contrib/ folder.


Request animationframe tracking

Performance before:

Performance after:

New way definetly uses more performance, but it does not seem like a huge impact, so the stability the new method provides seems worth it


Use Int overflow polyfill for Swift <3.2, not just Swift <4.0

The extension on Int has a Swift version check which assumes version 4.0, however, we did not anticipate version 3.2. To compile Freddy without upgrading to 4.0 and keeping with Swift 3.2 we need to make sure version check on the Int takes into account 3.2.
This project builds successfully with Xcode 9 and Xcode 8.3.

Thanks! Looks like it passed.

Oops, got the direction we were filling in backwards. inequalities--


FR: pngquant

Hey,
Wondering if you will be looking into integrating pngquant for lossy png compression? I presume this is what tinypng uses, but, don't want to use them as I believe they add their own meta data & you need to pay if you use over 500 images per month.
Many thanks,
Joe

I'll add it to my todo-list. :)

Woop, thanks dude!

👍

This has been implemented and will be in the next release.

thanks so much!
On 29 December 2016 at 22:04, André Elvan<EMAIL_ADDRESS>wrote:

This has been implemented and will be in the next release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aelvan/Imager-Craft/issues/55#issuecomment-269700741,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACOsdo3P9avTmuPpe4-cQIzIOzix-UTrks5rNC54gaJpZM4Kd-kP
.


Cherry-pick ep_adminpads3 changes

@MShultz, what do you think?
See #5.

@MShultz: I tried these changes on my test Etherpad and the "Last edited" column is blank again.

@MShultz: I tried these changes on my test Etherpad and the "Last edited" column is blank again.

It's working in Firefox but not Chromium.  Hmm...

@MShultz: I tried these changes on my test Etherpad and the "Last edited" column is blank again.

It's working in Firefox but not Chromium. Hmm...

@rhansen That was an issue we were running into as well. It worked on my local dev build of etherpad (tested on chrome), but pushing it up and installing it on our live version of etherpads caused the column to disappear again. Been working the last few days on trying to figure out what the issue is. I can get consoles to output for the date formatting, but haven't had the time to look into anything after that point.

@MShultz: I don't want to merge this until a fix is in place for the blank "Last edited" column bug. Can you take a look?

@MShultz: I don't want to merge this until the blank "Last edited" column bug has been fixed. Can you take a look?

Looking into this tonight and this weekend. If I can't fix it with the added feature, I'll be removing the feature this weekend just so the "Last edited" column is fixed.

@rhansen: Having an issue getting good debugging. The current published version of the plugin errors out with the following error whenever we go to the admin section. It's something that persists outside of these changes, so I'm trying to figure out what might have caused that. If you've got any suggestions, let me know.

I get this error both locally and on our production version. Running on Chrome.

@rhansen I believe that this is ready to be merged.
I fixed our issues that weren't involved in this PR that I mentioned earlier and retested thoroughly. Have had success on Edge, Chrome, and FireFox both locally and on our production build. The issue was you need to clear you cache before you see the updated changes (Ctrl + F5 on most). I don't want to merge until you verify.
I will also bump the version for these changes later to 2.1.0 since it adds a new feature. There are also some standards things I'll be updating (single ticks instead of quotes, etc.) that I'll be updating.
This allows the date to be in US Format by using the following setting added to the settings.json file.
"ep_adminpads2": {     "formatDateUS": true   }
You can see that here.

Whoops, I didn't see that you had pushed updates before I force-pushed a rebase. Would you mind re-pushing your branch (with --force)?

I haven't pushed updates recently. Testing with what was there (probably about 45 minutes ago now, I'm hoping nothing changed) the feature works correctly. It was a caching issue we were seeing, but I'd like you to verify.  Clearing the cache shows the last edited and adds the setting.

Testing with what was there (probably about 45 minutes ago now, I'm hoping nothing changed)

I merged some cleanups into master and rebased this branch onto it ~15m ago. The changes I added to master should not affect the functionality here.

Closing because #11 accomplishes the same goal in a more general way.


GD-454: Tests skipped when type is precised in the test function

The used GdUnit4 version
4.2.5 (Latest Release)
The used Godot version
V4.2.2 stable official
Operating System
Sonoma 14.4.1
Describe the bug
Tests are skipped in parameterized tests, if the parameters of the test function are typed and the parameterized data contains objects or null
Steps to Reproduce
func test_good_owner(p_resource: RPGResource, test_parameters: Array = [
						[RPGMap.new()],
						[RPGEquipment.new()],
											]) -> void:

pass

RPGMap and RPGEquipment are child of RPGResource
the error message is :
This test is skipped!
Reason: '
The parameter set at index [0] does not match the expected input parameters!
The value '<Resource#-9223371933289019745>' does not match the required input parameter <p_resource:Variant>.'
if I remove the type of p_resource, it works.
I got the same error, if  null is inside the test_parameter .
Minimal reproduction project
No response

The parameterized parsing do only supports concrete types, it must match with the input parameters.
You can use Variant instead.

I tried but I got the same error
func test_good_owner(p_resource:Variant, test_parameters: Array = [
						[RPGItem.new()],
						[RPGEquipment.new()],
											]) -> void:

and the error is :
line 95: This test is skipped!
Reason: '
The parameter set at index [0] does not match the expected input parameters!
The value '<Resource#-9223371932466936163>' does not match the required input parameter <p_resource:Variant>.'

The only way to run the test is to remove the type of p_resource

ok I will check this tomorrow.

You need to upgrade to GdUnit4 v4.3 where is not official yet released


Explore Manning N as a matter of landcover from datasets in literature

From Greg Cocks:
Based on a literature search, 25 correlations have been located and summarised in a crosswalk.
Task: Hydrologists in conjunction with the modelers to review these literature search results – and develop (a) modeling crosswalk(s)
It seems likely that a series of crosswalks will be needed, based on regional hydrologic, geologic, soil, topographic and/or other morphologies.
Further, this will almost certainly be an iterative process, with the crosswalk(s) being refined as model runs are undertaken - and the assignments adjusted as part of the overall FIM parameter sensitivity analyses and testing (i.e. based on real world benchmarks, etc)
NLCD_Mannings_working_GJC_Apr21.xlsx

For rating curves, before exploring further, let's wait to receive the draft NWM v3.0 routeLink which already has N values for the floodplain derived from land use/land cover.

Sounds good. I will mark as low priority until that time.

NLCD_Mannings_working_GJC_Apr21 onward.xlsx
This Jun. '21 version includes another column derived from the latest HEC-RAS manual


Integrate settings with plugin

Task/Issue URL: https://app.asana.com/0/488551667048375/1205758454523232/f
Description
See task
Steps to test this PR
See task

Current dependencies on/for this PR:

develop

PR #3545

PR #3565

PR #3633

PR #3718   👈

This comment was auto-generated by Graphite.


Weapon restrictions not working

Today they decided to randomly stop working. I have an example below
Giving ☣☢Bryantdl7☢☣ a m9k_davy_crockett
"m9k_davy_crockett" is a restricted swep from your rank.
Yet I still get the swep.
Here is my restrictions.txt
http://darkstorm.site.nfoservers.com/hosted/restrictions.txt

I will have to look at this later tonight.  Would you mind giving me a link to the weapon addon(s) you are using.

It is not specifically M9k that is unrestricted, here is my entire addon pack in case you see anything known for conflicting
http://steamcommunity.com//sharedfiles/filedetails/?id=147909284

The Ulysses team rewrote their hook priority system and it broke URS.  A quick fix has just been pushed.
Confirm fixed?


Fr_CD

I add fr_cd for DRC

Codecov Report

Merging #1980 into master will increase coverage by 0.31%.
The diff coverage is 23.52%.

@@             Coverage Diff              @@
##             master    #1980      +/-   ##
============================================
+ Coverage     55.90%   56.22%   +0.31%
- Complexity     2068     2074       +6
============================================
  Files           306      308       +2
  Lines          4851     4868      +17
============================================
+ Hits           2712     2737      +25
+ Misses         2139     2131       -8

Impacted Files
Coverage Δ
Complexity Δ

src/Faker/Provider/fr_CD/PhoneNumber.php
0.00% <0.00%> (ø)
1.00 <1.00> (?)

src/Faker/Provider/fr_CD/Address.php
33.33% <33.33%> (ø)
5.00 <5.00> (?)

src/Faker/Provider/de_AT/Person.php
0.00% <0.00%> (-100.00%)
1.00% <0.00%> (ø%)

src/Faker/Provider/en_IN/Person.php
0.00% <0.00%> (-100.00%)
1.00% <0.00%> (ø%)

src/Faker/Provider/hr_HR/Company.php
0.00% <0.00%> (-100.00%)
1.00% <0.00%> (ø%)

src/Faker/Provider/es_PE/Person.php
50.00% <0.00%> (-50.00%)
2.00% <0.00%> (ø%)

src/Faker/Provider/ne_NP/Person.php
0.00% <0.00%> (-50.00%)
2.00% <0.00%> (ø%)

src/Faker/Provider/es_VE/Company.php
50.00% <0.00%> (-50.00%)
2.00% <0.00%> (ø%)

src/Faker/Provider/fi_FI/Address.php
66.66% <0.00%> (-33.34%)
3.00% <0.00%> (ø%)

src/Faker/Provider/es_ES/Person.php
75.00% <0.00%> (-25.00%)
3.00% <0.00%> (ø%)

... and 36 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 1fcc343...31e282c. Read the comment docs.


Create UI for Showtimes page

To have UI for Showtimes page:

Correct UI
Correct convention

@HuyenNgo, please help me fix all issues!


show percentage in place count number in review in velocity theme

Bug report
Title
show percentage in place count number in review
Preconditions
1. framework Version. Master

Expected result:
https://prnt.sc/tr5epl
Actual result
https://prnt.sc/tr5a9k

fixed in master


[Feature][api] Integration with existing platforms(SSO)

Integration with existing platforms.
Whenever you commit a task to Dolphinscheduler via the API from a third-party platform, you have to log in to dolphinscheduler to view the task log and status. This can be very cumbersome.
Expect dolphinScheduler to provide SSO to facilitate seamless integration with existing platforms.

Hi,
I personally think this is a good feature to implement sso, it is a very common need in a lot of companies, but at present, if we want to implement this feature, ds's file privilege, datasource privilege, and tenant mode will have a big changes, so if anyone want to design this feature, welcome to leave a message or send a mail to mailing list.

Related issue: #4531

I would like to integrate Casdoor which is a powerful SSO platform into DolphinSchduler :)
As mentioned above, integration may make breaking changes. So is there any advice for this work?

Can we consider integrating with knox sso

close by https://github.com/apache/dolphinscheduler/pull/8706


Quit button in the pause menu crashes the game

Explicit, debugging reports a stack overflow

I tried debugging this but hit a wall because I don't know much.
The actual error happens on line 279 of CollisionManager.cs, after the Quit button is pressed:
277    public void RemovePoint(CollidablePoint point)
278            {
279                if (_points.Contains(point))
280                    _points.Remove(point);
281            }

The game is in the process of deleting entities. Using the default (server-generated) map, the first thing that triggers this code is a door, but I don't think that matters.
I think the error on 279 is that the Contains() function only works on strings (unless I read MSDN wrong, or unless this isn't the standard Contains() function, or really anything else). When I change it to this:
if (_points.ToString().Contains(point.ToString()))

line 279 works just fine.
However, then the stack overflow just happens one line later, on 280. I don't know how to fix that, or even if my "fix" to 279 did anything.
Regardless, I'm fairly certain line 279 is what's throwing the exception. Past that is anyone's guess really.

There's a PR up that intends to fix that at #171

This actually happens because of a legit stackoverflow.
The number of times we do IocManager.Resolve<Thing>.Shutdown
Which does
foreach(var comp in components)
    comp.Shutdown()

which then does fucking
foreach(var comp in components)
    comp.Shutdown()

Is causing the stack to just grow to large, we've instead moved to dispose them all a little better (there already was the functions). but we found that its not working 100% as it should and its breaking in the collision manager right now.
I've found that some entities are not being cleaned up or losing an owner when the system reconnects and so when you move the AABB collision detection fails.
If you want to provide any help, #171 has a bunch of changes most them don't actually help, as it was all just me testing. Ill go through today and keep only the stuff that actually helps so that you can use it if you want to help with debugging @h3half.

#171 fixes this and uncovers new bugs as well. Those should probably get their own issue.


Add contract spark session API

SAS-3261

To review, check out the preliminary docs: https://deploy-preview-768--jovial-piroshki-65ff4d.netlify.app/


Proxy implementation of appearance doesn't work

Unfortunately I can't compile a Project which includes following line:
[UINavigationBar appearance].tintColor = [UIColor whiteColor];
I got following error:
"error : property 'tintColor' not found on object of type 'id'"

Have you tried this?
[[UINavigationBar appearance] setTintColor:[UIColor whiteColor]];

You could also try casting the result of [UINavigationBar appearance] to type UINavigationBar *.

@wjk Thanks for your support, works perfect.

I would say no, as Xcode's compiler has the exact same problem with dot-notation access to properties on type id.

@wjk Ok I'm going to close my ticket, your hint has worked for me.

Hey guys, whenever I do
[[UINavigationBar appearance] setTintColor:[UIColor whiteColor]];

I get the next Exception

Plaform::NotImplemented Exception ^.
Selector +[appearenceWhenContainedIn:forUIClass] is not implemented for class UIApperanceSetter on object 0x1754840!

Any help will be welcome, also is there a way to change the blue color of the nav bar?


Improve data table template example to mention case-sensitivity

Description
https://discourse.nodered.org/t/working-with-v-data-table-in-dashboard-2/87783/3
Epic/Story
No response
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

Detailed as a vuetify bug report here: https://github.com/vuetifyjs/vuetify/issues/10002


Debug - The Electric Demon

Bill's dialogue box comes up too quickly after you defeat him in battle, causing the F5 glitch.

Simon, despite being a questgiver, lacks the label

Delay has been added, person who does the tags has been notified


Expose formatting function for JSON objects from webhooks

I'm using the Square API in Node.js. JSON response objects are formatted to camelcase. However, responses received from webhooks are formatted in underscore notation. Here is an example.
    {
    	"amount_money" : {
    		"amount" : 3052,
    		"currency" : "USD"
    	},
    	"id" : "luSGCASUgrETw5zbTbxqoMcCNYFZY_TlymxTGdkvplzO5M9Ux6cJItmPhzDQy3iYBHoJqBoTF",
    	"order_id" : "jLi6ahIO2pVmcJp70s3gp1IQGRFZY",
    	"updated_at" : "2021-01-10T18:32:04.974Z",
    	"payment_id" : "luSGCASUgrETw5zbTbxqoMcCNYFZY",
    	"version" : 5,
    	"status" : "PENDING",
    	"created_at" : "2021-01-10T18:32:04.966Z",
    	"location_id" : "LJ8GN0VT498M1"
    }

Please add a function that would expose this formatted as a separate function. This way we could format objects from a webhook and parse them in the same way as an object returned directly by the API.

As far as I know, the sdk doesn't support webhooks. You may pass the webhook request obj to this camelCase function to get desired format. @PomegranateApps

As far as I know, the sdk doesn't support webhooks. You may pass the webhook request obj to this camelCase function to get desired format. @PomegranateApps

@luis-jetson is correct.  The SDK does not currently support webhooks.


Add tilt, incline

Please add angle level meter, angle of slope

@Ygarr  Hello, this tool already exists, you are looking for the Inclinometer (under the tools tab), or the bubble level
(inclinometer is designed for slope/height measurement while the bubble level is meant to tell if a surface is flat)


Add pre-commit to shell.nix

so that in the nix shell we have nix's pre-commit, not the one in your .local/bin installed from pip, which for some reason doesn't work in the nix shell.

@michaelpj this makes nix's pre-commit appear earlier in PATH in the nix shell. Previously pre-commit was at the end of PATH, after $HOME/.local/bin, which means the wrong pre-commit is used if you have one in .local/bin.

Hmm, but that can't be true for all the stuff that goes in the shell? otherwise everyone's tools would be overriding the stuff in the shell. So why is pre-commit at the end?

I see: it's the shellHook from pre-commit-hooks.nix which appends it to the PATH: https://github.com/cachix/pre-commit-hooks.nix/blob/c3b4f94350b0e59c2546fa85890cc70d03616b9c/modules/pre-commit.nix#L322
Could you open an upstream issue and add a comment explaining why we need this?

Done, nice diagnosis!


A couple more stack files tweaks + update CI

Follow up to #1368.

Thanks again!

No problem! Could I get you to look at #1367 @vaibhavsagar? It should be easy to do, and would unblock IHaskell on Nixpkgs for GHC 9.2.

Hey, I've been putting it off because I have to bump the version numbers and upload everything correctly, a tedious yet detail-oriented task. I will get to it in the next day or two, please feel free to ping me again if that doesn't happen

Sounds good, thanks for looking into it!
Not sure I totally understand the difficulty -- ghc-parser is a standalone package and it doesn't seem like its version number is synced up with anything else (like ihaskell proper or any of the display packages). Isn't the whole task just to bump the version number to <IP_ADDRESS> (I don't think a major bump is even needed) and upload to Hackage? No pressure of course, just excited to get this working :)

It's not just ghc-parser, ipython-kernel and ihaskell will also need new uploads. My workflow for this could perhaps use improvement but since I do it so infrequently I don't really see the point in making it faster.


How to get image size from cache ?

Hi !
I need to know the size of an image to resize it by width and preserve ratio.
I've tried to use react-native-auto-height-image, but it doesn't support source prop,
the displayed image it gets from cache and the size from URL... It's not the behavior that I expect.
So, How can I get image size from cache with react-native-img-cache ?
thank you in advance 😃

@ldubos, you can simply use getSize() method on the Image component https://facebook.github.io/react-native/docs/image.html#methods to get the width and height of an image. However, if you are trying to get the size of an image already in cache you will need to get a path of the image in cache using the ImageCache on method https://github.com/wcandillon/react-native-img-cache#onuri-observer-immutable.

Ok thank

How to turn ffthe ImageCache.get.on()
I am trying to do it with the dispose but it does not work


Line SDK login issue, unable to return to login screen

In the case when the Line app is open and logged in, calling Line SDK login from an Android app does not return to the app but stays in the Line app interface. How to solve this issue? Can you please help me translate this question into English so that I can ask the official support team?

i have same issue
when i login success
activity will jump to ask for permission page
and then i click allow btn
page is stay at ask for permission page
not return to my app
and interesting thing is
if i close page by click x at top left
it will retunt to app and you can recieve line loging success data and token


feat(ENG-1440): db layer for value storage

value storage for projects will use the regular db instead of redis. this supplies the db layer which a service will be built upon to allow the script to manipulate environment wide variables.

please add tests.

done, black box style.


The database is empty, the next day or later.

The database is empty, the next day or later.
These are simply no available data in the database, but new data is recorded and stored by a couple of days.
Version of Realm and Tooling

Realm framework version: 3.4.0
Xcode version: 9.3
iOS/OSX version: 11.3
Dependency manager + version: Swinject

@BarredEwe I'm sorry, but it's not enough information to help you in any way. You have to be much more explicit about your problem and why you think there should be a bug in Realm that causes your issue.

@bmunkholm In the appendix there are no methods for database cleanup. But the database is completely cleared in a few days.
init() {
        let fileURL = FileManager.default
            .containerURL(forSecurityApplicationGroupIdentifier: "*************")!
            .appendingPathComponent("Library/Caches/default.realm")
        Realm.Configuration.defaultConfiguration.fileURL = fileURL
    self.realm = try! Realm()
}

@nirinchev Precisely, thanks, I think the problem was solved.


English Please

What is the problem with Chinese README's
Firstly, we congratulate you for getting so much star by sharing this repository with humanity.
But it is very disappointing for non-Chinese speakers when one couldn't understand what a trending repository is about.
When we see such a repo on trending, our minds are blurring like Gollum's.

There is a way you can help to solve this disappointment which I believe is experienced by many people who want to know more about your valuable work and appreciate it.
What we want:

Please add English translation of your README so you are sharing your work and knowledge with more people.

How this will help you:

More feedback to fix and improve your project.
New ideas about your project.
Greater fame.

“Sharing knowledge is the most fundamental act of friendship. Because it is a way you can give something without loosing something.”
— Richard Stallman
Thank you!

The developers of repo will not be held responsible for problems that arise in this script.

This issue generated by us/english-please script. Please open issue on any error. Thank you!

Calma aí, tem um robô que sai por aí criando issues em repositórios que ele identifica que o README não tá em inglês? Kkkk
Pior que ele não checa se é em chines, ele só checa se não é inglês mesmo: https://github.com/us/english-please/blob/master/english_please.py#L49

O mais engraçado é quando tem um repo em chinês e não tem tradução para inglês.

Lamentavel


Zend_Service_Twitter validInteger is really wrong and makes Twitter calls in error

Simple way to test this : call the valideInteger fonction with parameter ANPM1, the function will return ANPM1
But as you can see ANPM1 is not an integer.
So to fix this, the preg_match function must be like this :
    protected function validInteger($int)
    {
        if (preg_match("/(^\d+$)/", $int)) {
            return $int;
        }
        return 0;
    }

I have add a ^ before the \d to test the beginning of the string to test is a digit
and add $ after the \d+ to test all the line.
And now the call with ANPM1 returns 0.
regards,

@footcow
Can you write an unit test for this problem?

I can replicate this issue too, for this username [Kirsty_Maj0r] the function will return true.
From my terminal:
php > var_dump( preg_match("/(\d+)/", "Kirsty_Maj0r") );
int(1)


🐛 [BUG]: Uncaught TypeError: Cannot read properties of undefined (reading 'toLowerCase')

Is there an existing issue for this?

[X] I have searched the existing issues and this is a new bug.

Current Behavior
I haven't updated for a while, so I did and noticed I have a bunch of errors when I select the node.
The come from this function
function isKeyMatch(pt, _t, xt, kt) {
    const Ct = _t.split("+").map(Et=>Et.trim().toLowerCase());
    return Ct.length === 1 ? pt.toLowerCase() === _t.toLowerCase() : (kt ? xt.delete(pt.toLowerCase()) : xt.add(pt.toLowerCase()),
    Ct.every((Et,$t)=>xt.has(Et) && Array.from(xt.values())[$t] === Ct[$t]))
}

Expected Behavior
No errors
Steps To Reproduce
It looks like this error is specific to my setup. If a nature of the error is completely not clear I'll try to provide some access to that page online. Is that going to help?
Relevant log output
No response
Anything else?

When does this issue happen exactly? Does it actually happen when you do some keypress combo?

If you're using some keypress combo, what does it look like? Can you show me some of the code you use? :)

@bcakmakoglu just clicking on element, no keys pressed, that's very odd

Selecting as in clicking a node or as in creating a selection-box? Just clicking a node doesn't seem to trigger the isKeyMatch function 🤔
Which version are you on right now?
Any more infos in the stacktrace? Like a hint where the error started from so that it ended up in the isKeyMatch section?

Does this happen across all browsers or just specific ones?

ok, thanks for the tip, it happens only in Chrome. When I try incognito mode the error is not occurring even in chrome, so checking extensions

Any updates on this? Were you able to find the issue?

Any updates on this? Were you able to find the issue?

unfortunately I could not reproduce the issue, turning off extensions did not help

Are you, by any chance, using any key-filters for any key-codes that might be empty arrays?

I assume the problem is the same as #1255
So this should be fixed in 1.29.2

Fixed in 1.29.2

fixed indeed, thanks @bcakmakoglu


Error using from_xml with StructType for schema

I've got a Dataframe with a column that contains strings of XML formatted data. There's quite a bit of variability to the schema of any individual record, but I have an XSD file that should be valid for any of them. I'm trying to use the StructType that I parsed out of the XSD as the schema argument in from_xml but I'm getting an error. The actual schema I'm using is huge but I can reproduce the issue with a smaller example as below:
First I make a sample dataframe that looks kind of like what I'm actually trying to work with in that it has a string column containing XML:
b1xml = """
<book id="bk103">
      <author>Corets, Eva</author>
      <title>Maeve Ascendant</title>
</book>
"""
b2xml = """
<book id="bk104">
    <author>Corets, Eva</author>
    <title>Oberon's Legacy</title>
</book>
"""
bookslist = [
    ("bk103", b1xml),
    ("bk104", b2xml),
]

raw_df = spark.createDataFrame(bookslist, ["bookid", "bookxmlstr"])

Next I make sure I can parse it by inferring schema from a specific record:
strparsed_df = (
    raw_df
    .withColumn("parsedxml", from_xml(raw_df.bookxmlstr, schema_of_xml(b1xml)))
)
strparsed_df.show()

This works as expected
Finally, I try and pass in a StructType as the schema parameter for from_xml:
bookschema = StructType([
    StructField("_id", StringType(), True),
    StructField("author", StringType(), True),
    StructField("title", StringType(), True),
])

parsed_df = (
    raw_df
    .withColumn("parsedxml", from_xml(raw_df.bookxmlstr, bookschema))
)
parsed_df.show()

This fails with the following error:
AnalysisException: [PARSE_SYNTAX_ERROR] Syntax error at or near '{'. SQLSTATE: 42601

The StructType exactly matches what I see if I look at the schema of strparsed_df so I don't think it's an issue with how I'm describing the struct.
I've tried this on databricks runtime 14.3, 15.1, and 15.2 and get the same result across them

Update - this only seems to happen in databricks clusters using shared access mode. If I create a cluster in single user access mode then I do not encounter the error

If you're on Databricks, then spark-xml is already integrated into Spark there, and you do not need to install a library. That could in fact be the problem, if that's what you're doing. Shared is not going to allow libraries to call JVM code, unless they're whitelisted
In any event this wouldn't be an issue with this llibrary, but something related to Databricks and the port of this code into Spark.


no encoding declared, non-ASCII character

Is this written for python 2.x or 3.x?
The error is in dicts_init.py_
line 20:
SEP = ' ● '

Thank you, and yes, this happens when you run it in Python2.x, which needs encoding to be specified; however, ASCII Combat is written in Python3, since it is the latest version, and I am not intending to add Python2 compatibility, It runs perfectly fine in Python3.

Is there a library I need that's missing? I'm running main.py from a python3.6 virtenv and getting that error.

Can you send me the error details? What operating system are you using?

On Win10. In a py3.6 virtenv. The error is:
no encoding declared, non-ASCII character. The error is in dicts_init.py_ line 20: SEP = ' ● '.
I'll try again tonight. I am new to coding, so maybe something is wrong with my virtenv.
Check out my github, I have a very simple text game in python 2. I've updated it to py3 which I'll post. I'd love to help with your game. I'm learning a lot.

Screenshot the error, and try to run it without without using virtenv, just open a terminal inside ASCII Combat's directory and type in python main.py or python3 main.py if you have multiple versions installed. I ran the game on 5 different computers with different systems and it works fine.

Btw, I checked your game pythonRPG, looks pretty cool, but I noticed how you put everything in one file, This isn't good. If you want to follow professional coding practices, you should separate it into multiple distinctive files, with each file serving a specific function (Enemy class and its child in one file, one file for quotes, etc..) this way you can organize your project and make it more readable. Also I recommend having a look on this blog post with actually inspired me to start on ASCII Combat and gave me some solid ground for where to start from https://inventwithpython.com/blog/2014/12/11/making-a-text-adventure-game-with-the-cmd-and-textwrap-python-modules/

Thanks. Will work on all that this weekend. I'm new to contributing to other projects on GitHub, but look forward contributing to yours. Feel free to make a PR to my GitHub so I can see what that looks like if you want?

OK, works now. My computer was using python2 for some reason.

Created a PR to fix the Readme instructions. Please merge. Thanks!

I will close this issue now!


Support separate calls for particular information about the word

Is not actually useful as all the data is already accessible to a user


Add Clone support for ProofWithPublicInputsTarget

When I have done some recursive example, I found out the lack for Clone derive for ProofWithPublicInputsTarget.
Here example: https://gist.github.com/tikhono/6b3222ccaa2030280efcbc7038eeb373
Does the usage of clone correct? Or lack of this derive is intendant, so there are some reasonings for this?

Thanks for sending, this sees fine to me if it's needed. That said I think there are a couple other potential fixes,

It looks like our current recursion examples call set_proof_with_pis_target before verify_proof. It does seem a bit odd to require that ordering though.
We could change verify_proof to make proof_with_pis a reference. I think we usually take references to "target" structs (besides small ones which impl Copy); AFAIK there wasn't a particular reason we took a value in this case.

Do you think the reference change would address the issue for you?

Yes, I think reference should work for me.
Thanks for fast response!

Sounds good! I made #828 to change that, it might just be a bit delayed as our build is broken at the moment (https://github.com/rust-lang/rust/issues/105037)


Comments on an application do not appear in the order they were posted

See
https://recruit.karmafleet.org/recruits/application/93/
The last 2 comments, mine was posted first but is shown last.

Orders now done by date, in dev, push tonight


Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

#2423 838e58da Thanks @chentsulin! - Fix peer dependency declaration by using || instead of | to link multiple major versions

Updated dependencies [838e58da]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

#2423 838e58da Thanks @chentsulin! - Fix peer dependency declaration by using || instead of | to link multiple major versions

The latest changes of this PR are not available as canary, since there are no linked changesets for this PR.


Buzzer Subsystems - Background and other functions

[x] Understand and explain part in more depth

[x] safe connections

[x] test more resister combinations and see current draw and what each resistor does

[x] other types of driving circuits
[x] MIDI tones

[x] play different frequencies

[x] lots of more instructions

https://github.com/rb2468/tray-a-way/issues/59
Talked to Rachel about starting up pigiod from boot

Good work!

New Part - Buzzer - Tommy Gao (2).pdf


how to do condtional rendering

function MyProjects () {
  const { data: user, error } = useSWR('/api/user?training_id=router.query')
  const { data: projects,error } = useSWRInfinite(index => '/api/projects?uid=' + user.id&page=index+1)
  if (!projects) return 'loading...'
  return 'You have ' + projects.length + ' projects' + user.name + 'user'
}

first, get the user using useSWR(which depends on training_id) than get his projects using useSWRInfinite. I want to implement this

and also for both, I need error const
@shuding,  please help me in this
thanks in advance

@varunkumargande Please don't open issues for questions (it's for tracking bugs and feature requests, and this is the documentation site). You can always go to https://github.com/vercel/swr/discussions to ask questions (feel free to @ me or DM me directly on Twitter https://twitter.com/shuding_).

Regarding the question, you can return null if user is not ready (so SWR won't fetch):
  const { data: user, error } = useSWR('/api/user?training_id=router.query')
  const { data: projects, error } = useSWRInfinite(index => user ? '/api/projects?uid=' + user.id&page=index+1 : null)

ok will use discussion forum @shuding

No worries, happy to help!


Moved hit Tolerance parameter to own example.

This PR separates the select interaction and hit tolerance examples.

Thanks @simonseyok.


Need Responsive Menu Navigation mobile submenu to expand on single click and go to a URL on double click

Is there any way to make a Responsive Menu Navigation mobile submenu to expand on single click and go to a URL on double click?  The hover on full desktop view is fine as the node can be a URL and also have a submenu.  Just trying to get a better mobile navigation user experience.  I tried the new Submenu Toggle feature which works great, but I think this behavior would be the ultimate!  Is there any way to get it?
For an example, you can see what I mean at http://www.kayalortho.com. Click on Therapies & Treatments and then Orthopaedic Services & Surgery to see what I mean.

This gets us closer.
1st click - Open the accordion
2nd click  - Open the link
<script>
    jQuery('.is-accordion-submenu-parent > a').click(function(){
        jQuery(this).unbind('click');
    });
</script>

I modified code found on http://foundation.zurb.com/forum/posts/37943-f6-accordion-menu---enable-parent-links

@kball has this all worked out https://github.com/zurb/foundation-sites/pull/10616/files?diff=unified


Deprectation warning for loaderUtils.parseQuery() from loader-utils on ng build

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

Windows 8.1 Pro
Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:

@angular/cli: 1.0.0-beta.32.3
node: 7.4.0
os: win32 x64
@angular/common: 2.4.8
@angular/compiler: 2.4.8
@angular/core: 2.4.8
@angular/forms: 2.4.8
@angular/http: 2.4.8
@angular/platform-browser: 2.4.8
@angular/platform-browser-dynamic: 2.4.8
@angular/router: 3.4.8
@angular/cli: 1.0.0-beta.32.3
@angular/compiler-cli: 2.4.8
Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.

Brand new app created using the CLI.
Run ng new myproject then ng build.
No changes to any code.
The log given by the failure.

Normally this include a stack trace and some more information.

λ ng build
10% building modules 4/4 modules 0 active(node:10396) DeprecationWarning: loaderUtils.parseQuery() received a non-string value which can be problematic, see https://github.com/webpack/loader-utils/issues/56
parseQuery() will be replaced with getOptions() in the next major version of loader-utils.
Hash: 4a28dd932abdbd64b02a
Time: 6930ms
chunk    {0} polyfills.bundle.js, polyfills.bundle.js.map (polyfills) 153 kB {4} [initial] [rendered]
chunk    {1} main.bundle.js, main.bundle.js.map (main) 4.23 kB {3} [initial] [rendered]
chunk    {2} styles.bundle.js, styles.bundle.js.map (styles) 10.1 kB {4} [initial] [rendered]
chunk    {3} vendor.bundle.js, vendor.bundle.js.map (vendor) 2.39 MB [initial] [rendered]
chunk    {4} inline.bundle.js, inline.bundle.js.map (inline) 0 bytes [entry] [rendered]
Mention any other details that might be useful.
Related to loader-utils issue 56:  https://github.com/webpack/loader-utils/issues/56

The deprecation warning is caused by ExtractTextWebpackPlugin.
Issue: https://github.com/webpack-contrib/extract-text-webpack-plugin/issues/421


Doesn't work with adrenaline + other issues

Hello gave this a try but seems if you play a game on adrenaline or even try to launch adrenaline the ps vita will just hang. I removed this plugin to make sure it wasn't something else but when I removed it, adrenaline started working again.
Also I didn't really notice and performance increase. I tested it with a few games like minecraft and borderlands 2.
I think minecraft it made a slight difference but with borderlands 2 I swear it actually made the fps worse until I put the oc back to the game default or default.
Another thing does the overclock drain the battery? I have the old 1000 phat ps vita

I too have noticed the issue with adrenaline, whenever i load shoutcast radio, the vita will freeze and shutdown after 20-40 seconds.
it will only be a minor increase in frame rate, i have noticed an increase in fps on every game i have played though, including borderlands 2.
it will decrease battery life.