tests-pytest and doc-build actions fail for pyansys-advanced with poetry template

https://github.com/ansys/pyfluent-reader/actions/runs/6469381862/job/17563451958#step:2:451
/home/runner/work/_temp/98be34b6-060e-4d3a-9fac-879cfefcd336.sh: line 1: pytest: command not found

https://github.com/ansys/pyfluent-reader/actions/runs/6469381862/job/17563344927#step:2:1063
/bin/sh: 1: sphinx-build: not found

Hi. I was looking at this recently and had issues pytest: command not found.
And it seemed to be because poetry is running in default mode and creating a virtualenv which is not activated in the github actions. I created an environment variable to configure poetry not to do this in the CI/CD so poetry should be found.
POETRY_VIRTUALENVS_CREATE = false

Hi @philipjusher thanks for commenting and using our actions! Could you share a Code snippet of the changes you performed so that we understand your issues and solution? Really appreciated! A URL also works =)

Hi @philipjusher - the way we have it right now is that poetry (in case of tests) tries to install a group called tests. See https://github.com/ansys/actions/blob/main/tests-pytest/action.yml#L139
If you have pytest defined inside your group, this should solve your issues. You should also define your test dependencies inside it.

Something like this for example:

I checked that first

However poetry creates in a python virtualenv and there is nothing that activates that before you run the pytest command on line  148. The solution I had was to make sure it didnt create the virtualenv and installed to the system python.  Our code is here for now https://github.com/ansys-internal/pyconceptev-core

Gotcha! Thanks for the explanation. I will add this right away directly inside the various poetry actions. Thanks a bunch!


seed some questions

Some questions to start us off.  Feel free to delete, adjust, add, and of course answer!  I'll start adding some info this weekend if nobody beats me to it.
I was debating whether we need to add a disclosure section but decided to leave it off for now...

Added some names I recognized from the list as reviewers.  Just to get some extra attention.  Everyone welcome of course to edit!


what does 'duration' mean in downsample?

Describe your question
From this document, I see there's a duration (after with), and the document says it must be smaller than the duration of the corresponding retention policy.
But I cannot understand its meaning, does it mean the down sampled data will only be kept to that duration?
If yes, and if raw data was deleted after down sample, because this duration is smaller than the duration of the retention policy, then history data will always be deleted earlier before it reach the duarion of the retention policy. I don't think this make sense.
If raw data was not deleted after down sample and will be kept until the duration of the retention policy, then the down sample result becomes  an extra copy of the data, that's down sample can never reduce storage requirements.
BTW1: the document says there' should be a watermark in the syntax, but after I checked 'sql.y', I think it should not be there.
BTW2: the document says nothing about where the down sample result will be, while in my experiment, the raw data was deleted but no down sample data was generated.

you may reference to the downsample instroduction

reference to the downsample instroduction

Thanks, but the URL is a local one, its host is 'localhost'.


question: What is the purpose of the SSH & GPG keys, and the Connected Accounts areas in settings

Having published both SSH and GPG keys to my localdb I can't see how these can be used.  There's no mention in the docs and I couldn't find much in the code.
Can you give some examples of how these can be used please?
I also can't see how an account can be connected into the Connected Accounts area.

@jinnko, I use them in my projects. For example, compile a list of public SSH keys for server access. GPG keys are used for the encryption of files sent between users. These capabilities are not a part of the portal, but they are there for that reason.

@greenpau - one more question: is there an endpoint that exposes these keys publicly or privately, particularly the SSH keys?  (I'm already using GnuPG WKD for GPG)?

@jinnko ,  there is no endpoint at the moment. It probably deserves one.
The reason it is not there is that I can read from db file itself.
my plans include adding different database types backed by cloud based KV services, e.g. dynamodb, etcd, etc

I already started the process, but my work life made it difficult to contribute
https://github.com/greenpau/go-authcrunch-ids-dynamodb

I already started the process, but my work life made it difficult to contribute
https://github.com/greenpau/go-authcrunch-ids-dynamodb

Cool - I know the feeling.
I'll try to make some time myself to raise a PR against the docs to include the info shared in this issue.

there is no endpoint at the moment. It probably deserves one.

Have you considered an implementation to expose SSH keys similar to Github/Gitlab?  For example SSH keys are exposed publicly at github.com/username.keys and similarly on gitlab.com/username.keys.  If you follow the pattern you could expose the user keys on the authentication host.
For GPG the WKD standard has been good enough for me so I don't have another suggestion there.
Just some ideas.
Thanks for answering my questions and thanks for putting together this awesome project!


Notifications: keepOpen is not working for choices

/cc @alexandrudima you were the only consumer

@roblourens this needs the "Extension host is unresponsive" notification to show up, which I guess is hard to trigger unless you modify some sources. But you can run a simple notification to get similar behaviour:

write an action that uses INotificationService.prompt()
add a primary action that sets keepOpen: true
trigger the notification and click on the button
verify the notification does not close as long as keepOpen: true is set


rfc: A new tool to implement memory stress

Signed-off-by: FingerLeader<EMAIL_ADDRESS>

There is my implementation, PTAL at memStress.


Feature request: option - on scroll end, prevent outer page from scrolling

I have a feature request for configuration - "scroll_end" with options "native" or "stop_on_end". Thank you!

You can use https://github.com/rstacruz/jquery-stuff/blob/master/preventoverscroll/jquery.prevent_overscroll.js in conjunction with NanoScroller.

Isn't it a big enough feature to include it?


chore(LinearAlgebra): format/add dates to remaining deprecations

aim for the deprecation date on the same line as the attribute (this is easier to search).
dates were found using mathlib-changelog

Thanks!
bors r+


More Pipeline Dropdowns

I really like the Pipeline drop down.  It makes it very easy to select a pipeline without creating typos:

It would be nice to be able to do that in other parts of the project as well.  Unless there is a design choice for having to enter these pipelines manually?

If there is a design reason, maybe there can be a drop-down arrow inside the textbox that can be selected in order to present the user with a list of options that they can select to prefill the textbox?

Yup, this is doable. I initially used the input line, since faster to do. In a later pass I added the drop down. Add it there too is definitely a good idea and it's definitely doable.

Moving the proposal to discussions to keep the issue just for things to fix.


Optimisation at times exceeds p_nom for deferrable loads

Describe the bug
At times the optimisation allocates power levels which are above p_nom for deferrable loads.

To Reproduce

Expected behavior
p_deferrable should not exceed p_nom, so in this case it should not exceed 600 W for the 17:00 timesteps.
Then a few minutes later, optimisation is delivering the expected behaviour.

Home Assistant installation type

Home Assistant Supervised

Your hardware

OS: Linux
Architecture: amd64

EMHASS installation type

Add-on

Additional context
deferrable4 (my hot water heat pump) has an additional constraint to complete by sunset, which occurs at the 17:00 timestep.
I suspect the optimisation is allocating additional power to deferrable4 at this point, because it is constrained..
Another set of sub optimal results 5 minutes later:

Another user with the same issue.
https://community.home-assistant.io/t/emhass-an-energy-management-for-home-assistant/338126/2528?u=markpurcell
@GeoDerp , can you assist I think there is a logic error in the stop intervals code that it tries to squeeze in all the power required in the time remaining before the stop interval without consideration of the p_nom restriction.

Hey @purcell-lab , Unfortunately im not verse in that area of the EMHASS code.
Out of curiosity, What happens if 'def_start_penalty': [] in runtime ?

Maybe related to this issues I am see power allocated to deferrable loads when the def_total_hours runtime param contains zero for that particular def load.


Add new "hll" bucket type

This is needed for unit testing the new hyperloglog functionality in riak-erlang-client.

+1 f8ef138


Allow for RolesAllowed annotation on classes

Currently, the AuthDynamicFeature checks @RolesAllowed annotation only on method level.
This prevents creating class level annotations on resources.
Expected behavior:
Check also for the annotation on resource class and register the authentication filter in case the annotation is found
Proposed code:
public class AuthDynamicFeature implements DynamicFeature {
    private ContainerRequestFilter authFilter;
    public AuthDynamicFeature(ContainerRequestFilter authFilter) {
        this.authFilter = authFilter;
    }

    @Override
    public void configure(ResourceInfo resourceInfo, FeatureContext context) {
        //get the RolesAllowed on the class level
        RolesAllowed ra = resourceInfo.getResourceClass().getAnnotation(RolesAllowed.class);

        AnnotatedMethod am = new AnnotatedMethod(resourceInfo.getResourceMethod());
        if(ra != null || am.isAnnotationPresent(RolesAllowed.class) || am.isAnnotationPresent(DenyAll.class)) {
            context.register(authFilter);
        }
    }
}

+1 definitely a show stopper for us

Would you mind submitting a pull request to implement this improvement?

Surely I can submit a PR. Give me a few days please.

I have submitted a PR #1437 regarding this

@jplock I believe that this has been addressed in: https://github.com/dropwizard/dropwizard/pull/1437 and it can be closed.


Fixes so data loading code works on Django 1.10

This set of changes fixes the data loading routines to be compatible with Django 1.10.
Unfortunately, my earlier testing with Django 1.10 seems to have worked because I didn't fully flush the docker container and rebuild it with docker build . or docker-compose build. So these additional gremlins popped up today.
There were a number of deprecations to fix jumping from Django 1.6.
The DJANGO_SETTINGS_MODULE needs to be set pretty much all the time for scripts to work. reset_db.sh and init_docker.sh updated accordingly.
As part of the shift from syncdb to migrations, ./manage.py sqlclear is replaced by ./manage.py sqlflush in reset_db.sh
Also in reset_db.sh, ./manage.py syncdb is replaced with ./manage.py migrate.
So now we must ./manage.py makemigrations thresher whenever we update models.py and those migrations need to be committed to the repository.
In standalone scripts like load_data.py, we now have to call django.setup().
Finally, I removed the --reset-db option from load_data.sh, as it has been replaced by reset_db.sh. Aside from being superseded, it depended on the undocumentd (as far as I can tell) django.core.management.sql.sql_delete which is gone.

I updated this PR with the updates to pagination settings needed for Django-Rest-Framework 3.4.5.
Prior pagination settings were pending deprecation in DRF 3.1 deprecated in DRF 3.2 and removed in DRF 3.3
I also updated the README.md to clarify when containers need to be rebuilt.
I merged the pending PR's into the branch pre-master so that people can test and verify that the pending PRs result in a working build. You should see Steven's new TopicPicker working in this branch. You should be able to use Redux DevTools successfully to inspect internal state.


Gene referenced in the database that did not show up after computing ExtractEdges.py

Hello,
First of all, I would like to thank you for developing NATMI, It is really a great tool!!
Nevertheless, I have an issue that I do not understand. I have five mouse datasets and in each one of them the gene Nrg1 is expressed so this gene is present in my Normalized Expression Data that I give as an input of the ExtractEdges.py script. Although Nrg1 is present in the database and in your homogene.data file,  in the result table, the Nrg1 gene does not show up in any of my five datasets. Do you have any idea what can be the issue?
As an example,here is the command line that I used :
python ExtractEdges.py --interSpecies expandp --interDB LRDB_NATMI --emFile P56wt_counts_norm.txt --annFile P56wt_metadata.txt --species mouse --coreNum 10 --out P56_wt

Thank you very much in advance.
Have a good evening.

Resolved through emails.


[#2115] fix(jdbc-catalog):  Can't create a table that exist in other database.

What changes were proposed in this pull request?
Add check logic about schema name when loading table meta from driver.
Why are the changes needed?
Some drivers , such as PG drivers, contain schema name information, we need to filter it. Some drivers like MySQL don't like it, we do not need to check it.
check it.
Fix: #2115
Does this PR introduce any user-facing change?
N/A.
How was this patch tested?
Add ITs: testCreateSameTableInDifferentSchema

@Clearvive
Can you please help review it or raise a PR about these related issues here? Please follow these issues and other related ones.

Should you continue this work @yuqi1129 ?

Should you continue this work @yuqi1129 ?

Yeah, it's currently under review.

@diqiu50 @FANNG1
Could you kindly review it for me?

@FANNG1
Please help me review this PR if you are free.

@FANNG1
Code updated, please take a look again, thanks.

MySLQ is OK with a wildcard in the db name.   assuming there are three database name db10, db1_, db12, and each database contains a table name t,  then when I query the table t in database db1_,  I only get 1 result not 3.


Generate Report of MSFAA File Errors

#2377 is a supplementary way of handling this issue.
Description of Task:
Generate a daily report documenting all SABC students listed on the MSFAA Received file containing a processing error.
Context:
The federal service provider (NSLSC/Finastra) sends a daily MSFAA Feedback file which sometimes contains error codes for particular applications. These error codes indicate there was at least one error in the list of validations which exist on the NSLSC side. For e.g., the address listed on a student's application is not recognized, or a student is missing a first name.
Some of the errors do not block the disbursement from being issued, but others do. For any errors which block disbursement, the student's application and disbursements are now completely in limbo and will not get resolved unless someone within SABC takes an action to resolve. However, without knowledge of which students have errors, there is no way for SABC staff to know which applications need attention.
What's Included on Each Report

A new entry for each student which contained at least one error
First Name, Last Name, DOB, SIN associated with each student
A list of all error codes associated with each student, and the written description of what each error code means Need the list of MSFAA error codes which might be encountered - Hugh to ask NSLSC for these
Aggregate details in the report header: 1) Date, 2) Number of students with errors

Acceptance Criteria

[ ] Create a process which reads the daily MSFAA receipt file and detects any students with associated errors.
[ ] Create a report (.txt?) daily which summarizes all detected errors from that day's MSFAA receipt file. The report should contain info consistent with the above section
[ ] The report should contain FT and PT applications in one report
[ ] If there are no error codes detected on that day's MSFAA received file, do not configure a new report
[ ] When the report is configured, send it to<EMAIL_ADDRESS>with the subject line "MSFAA Receipt Error Codes Detected"

Closing - MSFAA has a single response file that is R or C (received or cancelled). No errors.


DC big update

milos parameters fixed
CTparams improved (no ghost considered option)
intermediate results (needed for pipeline comparisons)

Please uncommit the .o object and executable files, those should not be on the repositories
You also commited hrt_pipe and hrt_pipe_sub in the main folder - those files we don't need there anymore


RFE - a few minor bits

So thanks for putting this out... I'd like to suggest some small possible changes/RFEs (I actually hacked up your source to put these in, but am not 100% sure it wouldn't impact other things, given the various options that I didn't use, and the changes are really minor, code-size wise.)

CRC... really? ;)  The SHA256 makes the CRC worthless.  I suppose if you're ultra concerned about speed... but at least give it the option of not using both (see next!)
option: use size or hash, not all two/three
option: use full hash instead of first 1k (this would obviate crc/size, obv)
print the status/non-dup information to stderr, not stdout, so you can redirect output to a file and still see the status/results

Finally, for me, for all but trivial amounts of potential duplicates, the output would be more useful if it were programatically consumable... I changed it to output an ID (match #), a tab, and then the full path to each filename, like:
0\t/path/to/foo
0\t/path2/to/bar
1\t/path3/to/baz
[...]

(etc.)  Then I could (did!) write a simple awk/whatever slicer/dicer on the output and do the actual removal step. What I don't want to do is write a parser to figure out your output.
Of course this might be another option - user friendly vs pragmatic or whatever.
These are pretty trivial code changes, YMMV, but for me it'd make it more useful as a utility.  Tnx again!

Also... having just did a big run... a minimum file size would also be useful :) Lots of little files match each other... if trying to recover some disk space, they're not terribly useful. Or, alternately, print out the file size/SHA/etc along with the filename in the output.

@zenfish I couldn't locate your fork. Can you share your changes?
Can you clarify the comment about:

SHA256 makes the CRC worthless

from reading the code and running the CRC will catch files that are different in the first 1Mb, sha256 is only used when file size matches and first 1Mb crc hash.
Minimum file size filter is a great idea 👍


QA Environment

Hi Stephen,
This closes #65, #75, #82.
Instructions:

Get a new copy of the Keepass database.
Copy all the QA keys to your ~/.ssh/ directory and set them to read only with chmod 0400.
Set the WINDOWS_QA_ANSIBLE_USER_PASSWORD to the value in the Keepass database.
Run make env-jenkins-qa-aws.
SSH to the Bastion and run make provision-jenkins-qa-aws.
Provision the mac from your host with make provision-rust_slave-macos-mojave-x86_64-qa-aws
Kick start the connection to Jenkins with: ping <IP_ADDRESS>.
Login to Jenkins at https://jenkins-qa.maidsafe.net and verify that all slaves are connected.
Go back to the Bastion and run make provision-jenkins-qa-aws.
Go back into Jenkins.

Acceptance critiera:

The QA environment should be created successfully.
The QA environment should be provisioned successfully.
You should be able to log into the QA environment.
The Windows slaves should be connected after they are re-provisioned and restarted.
On the re-provision, the log should indicate the haproxy-ssl-config : generate certificates and haproxy-ssl-config : concatenate full chain and private keys tasks should be skipped.

Cheers,
Chris

Acceptance critiera:

[x] The QA environment should be created successfully.
[x] The QA environment should be provisioned successfully.
[x] You should be able to log into the QA environment.
[x] The Windows slaves should be connected after they are re-provisioned and restarted.
[x] On the re-provision, the log should indicate the haproxy-ssl-config : generate certificates and haproxy-ssl-config : concatenate full chain and private keys tasks should be skipped.

TASK [haproxy-ssl-config : generate certificates] ************************************************************************************************************
skipping: [haproxy]
TASK [haproxy-ssl-config : concatenate full chain and private keys] ******************************************************************************************
skipping: [haproxy]


example error? ^3.3.11

the following example in the readme throws an error in version ^3.3.11
 //decimal mask
   Inputmask("(.999){+|1},00", {
        positionCaretOnClick: "radixFocus",
        radixPoint: ",",
        _radixDance: true,
        numericInput: true,
        placeholder: "0",
        definitions: {
            "0": {
                validator: "[0-9\uFF11-\uFF19]"
            }
        }
   }).mask(selector);

@diego-lipinski-de-castro ,
Jit masking on quantifiers is introduced in version 4.0.0.  This is not supported in v3.3.11


Here is a working sonatype URL

Issue Type
[ X] Question
[ ] Bug
[ X] Improvement
[ X] Build system related
[ ] Performance
[ ] Documentation
Description and/or steps/code to reproduce the problem
I wanted to try 6.0.0-SNAPSHOT without using the aar. As it turned out, the sonatype URL you give, requires a login. However I found one, that is working for me:
https://oss.sonatype.org/content/repositories/snapshots/
or in full`, the following repository works for me:
repositories {
   maven{        url 'https://oss.sonatype.org/content/repositories/snapshots/'         name 'OSS-Sonatype'     } }`
You might want to change your page correspondingly?!
As  I have plenty of situtations where I have double/int issues now, I must decide next, if I really do the jump ...
Environment
If it's a bug, version(s) of android this affects:
All
Version of osmdroid the issue relates to:
6.0.0-SNAPSHOT

Thanks il get it updated shortly
On Dec 25, 2017 09:00, "InI4"<EMAIL_ADDRESS>wrote:

Issue Type
[ X] Question
[ ] Bug
[ X] Improvement
[ X] Build system related
[ ] Performance
[ ] Documentation
Description and/or steps/code to reproduce the problem
I wanted to try 6.0.0-SNAPSHOT without using the aar. As it turned out,
the sonatype URL you give, requires a login. However I found one, that is
working for me:
https://oss.sonatype.org/content/repositories/snapshots/
or in full`, the following repository works for me:
repositories {
maven{ url 'https://oss.sonatype.org/content/repositories/snapshots/'`
name 'OSS-Sonatype'
}
}
You might want to change your page correspondingly?!
As I have plenty of situtations where I have double/int issues now, I must
decide next, if I really do the jump ...
Environment If it's a bug, version(s) of android this affects:
All
Version of osmdroid the issue relates to:
6.0.0-SNAPSHOT
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/osmdroid/osmdroid/issues/824, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB4kGfZ2X8ibyF3oXALNUAct9lcDbMtZks5tD6p2gaJpZM4RMTh8
.

did you run into any issues other than deprecated method calls? FYI, I just pushed another snapshot

Looked good overall. Elsewhere I posted that zoom button stuff and mentioned the missing close() statements. I have a branch for 6.0.0-SNAPSHOT now and plan to revisit occasional.


Accept decimal strings in Intl.NumberFormat

Fixes #52124
Note: This is not a coercion like in other JS APIs that also "accept" strings by coercing them, it is specifically designed to handle arbitrary-length decimals without precision loss. This was added as an explicit feature in Intl.NumberFormat v3 (which is why I added it to es2023.intl, which also houses formatRange(), which also got added in v3): https://github.com/tc39/proposal-intl-numberformat-v3#interpret-strings-as-decimals-ecma-402-334
The format matches what ${number} accepts:

The general syntax for decimal strings, essentially #.#E#, is a widely understood format in computing. The specific version we use is the ECMA-262 StringNumericLiteral grammar, which also allows non-base-10 numbers like hexadecimal and binary.

It is very common to represent monetary numbers as strings in JSON APIs to avoid any precision loss and those can currently not be passed to the Intl.NumberFormat API without casting as any. The resulting type errors currently misleads developers into using parseFloat() to make the type error go away, which causes precision loss.
It is supported by all browsers: https://caniuse.com/mdn-javascript_builtins_intl_numberformat_format_number_parameter-string_decimal

👍 (this is the relevant feature in the spec – strings are now converted to mathematical values, rather than coerced to numbers as in previous versions)
FYI, the ECMA-402 v10 NumberFormat definitions were moved to es2023.intl since your branch, so this will need updating.

Rebased. The new structure is nice!

@felixfbecker updated tests: afc13e9e53731792be58f6b50b83c87f1db06afe

This makes sense to me. I'll bring the issue up at a design meeting.

We agreed that this makes sense. @felixfbecker would you mind bringing the PR up to date with main?

@sandersn Awesome, I rebased onto main! Build is passing.


health endpoint

Introduces health endpoint in a form StatusBoard expects.
Closes #281

Release notes:

Introduces health endpoint in a form StatusBoard expects


Accidentally made the download path in appdata and not in a folder when migrating it migrates the whole appdata..

I already deleted any trace of bloxstrap roblox and most of the new files

oh my bad i deleted the registry its done


Ava/test custom workflow

Making some changes to the way the library autogenerates:

query now looks at all jhu repos with added page limit of 500.
pulls out the homepage
checks for anvil within topics tagged in the repo description, and removes any that don't have it
pastes topics for easy viewing in a rendered table
removes any repos without a homepage
includes a github link when the table is rendered

Resolves #116 !
Will work on syncing stuff in a bit.


ISPN-9333 Delete destination file before rename

On windows File.renameTo() fails if destination file exists. Quick fix provided.
https://issues.jboss.org/browse/ISPN-9333

@danberindei @rigazilla should we use move() instead? https://docs.oracle.com/javase/8/docs/api/java/nio/file/Files.html#move

@pruivo yeah I picked up a very old solution (let me run a test with move).
@danberindei ok? It seems that you prefers a different approach, but I'm not sure I understand.

btw the renameTempFile is called only twice in the code, so the solution proposed by @danberindei can be also easily implemented. Or maybe we can just rename the method to suggest that an overwrite is ongoing

-1
I think the operation should be atomic (or closest to that). Also, the file is updated under the a lock. you shouldn't remove it before invoke the method.
I prefer the move() approach.

@pruivo -1 to failing the move if the target exists?
BTW, I'm not sure how a move could not be atomic: it either succeeds or it fails, it can't move half the file.

@danberindei yes. the target is always there (except in the first run). Every time a cache-configuration or a counter-configuration is defined, the file is updated.
Also, I don't like the idea of "the caller delete the file before rename it"

Integrated, thanks Vittorio!


DTSPO-22369 - fix help bot set up documentation

Jira link
https://tools.hmcts.net/jira/browse/DTSPO-22369
Change description

The slack help bot documentation for creating a new slack bot is outdated after legacy workflow steps have been deprecated.

Instead of using the now deprecated “Step from App”, it is recommended that you re-implement your step as a custom function.

This update to the set up documentation should allow you to easily create a slack help bot from scratch using the new methods.

Checklist

[x] commit messages are meaningful and follow good commit message guidelines
[x] README and other documentation has been updated / added (if needed)
[x] tests have been updated / new tests has been added (if needed)
[ ] Does this PR introduce a breaking change

Plan Result (cftptl - Job)
No changes. Your infrastructure matches the configuration.


Malformed SVG when displaying in Inkscape

I am running svgo v1.0.3 over this file:
https://gist.github.com/iampeter/6aaa951a1b20499e827f16a09877bf95
When opening in Inkscape afterwards, it looks like this:

The original looks good:

@iampeter can you try with v2.1.0?

Eva is alive in the last v1 and v2. Thanks!


lib/domains/sn/univ-thies.txt

Demande de tirage!

@camarakassa What is the email address pattern in this university<EMAIL_ADDRESS>If yes, please rename univ-thies.txt to lib/domains/sn/univ-thies.txt. Thank you!

@camarakassa File path is incorrect in your request, thus I manually added a correct file to the repository. Pull request closed. Please wait at least two hours after this message is sent, before start requesting the licenses at https://jetbrains.com/student.

@camarakassa File path is incorrect in your request, thus I manually added a correct file to the repository. Pull request closed. Please wait at least two hours after this message is sent, before start requesting the licenses at https://jetbrains.com/student.


Ripple not working when primaryColor is white

Hi, this is my implementation:
<com.ashokvarma.bottomnavigation.BottomNavigationBar android:layout_alignParentBottom="true" android:id="@+id/bottom_navigation_bar" android:layout_width="match_parent" android:layout_height="wrap_content" app:bnbBackgroundStyle="background_style_static" app:bnbBackgroundColor="@color/colorPrimary" app:bnbMode="mode_fixed"/>
The problem is that no ripple is shown when I select a tab. Only when setting the bnbBackgroundColor to something different, the animation is shown.

@Florianisme I will look in to the issue and if it is a bug i will try to fix it as soon as possible.
If you can share your code snippet for the bottomTabs setup it would be really helpful.
Thankyou for using the library

It's the setBottomNavigationBar method at the end of this file https://github.com/Florianisme/Aperture/blob/master/app/src/main/java/com/florianisme/aperture/MainActivity.java

Which ripple due you need exactly ??
https://github.com/Ashok-Varma/BottomNavigation#bottomnavigationbar-customizations
you need to apply app:bnbBackgroundStyle and app:bnbMode according to your requirment

I just need the normal ripple as if you selected something in android when using the static mode.

I think I'm seeing the same issue.  In the sample app if you click on MODE_FIXED and BS_STATIC it shows a ripple when you click on an item (as expected).  In my app, I have the following set and don't see a ripple when I select an item.  Same thing if I click the selected item - in the sample app it shows another ripple as feedback, but in mine no feedback is shown.
<com.ashokvarma.bottomnavigation.BottomNavigationBar
		android:id="@+id/bottom_navigation_bar"
		android:layout_alignParentBottom="true"
		android:layout_gravity="bottom"
		android:layout_width="match_parent"
		android:layout_height="wrap_content"
		app:bnbBackgroundColor="@color/default_white"
		app:bnbActiveColor="@color/primary_pink"
		app:bnbInactiveColor="@color/kindof_grey"
		app:bnbMode="mode_fixed"/>

@jt-gilkeson @Florianisme i was able to reproduce this issue.

This is only happing when i use xml attributes for mode and background_style . Some code issue with xml attributes. Thanks for reporting.

You guys can try setting the background style and mode in code for now. I will look in to this weird issue

Using no xml attributes and setting the mode to fixed in code didn't seem to work for me either :(  I still get no ripples.

Mode and background_style also remove from XML. it worked.But I will reverify and let you know

Shouldn't the bottom_navigation_bar_overLay have the following set default in bottom_navigation_bar_container.xml?
android:visibility="gone"
Making that change fixed the issue for me.

both are almost same

@Florianisme @jt-gilkeson 1.2.2 is out with the fix.


If we want to remove "Create Site" command only

If we remove the "Create Site" links, will the users still be able to create Teams and Groups ?

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

ID: b29978b1-a400-bacb-1330-55ed3d652053
Version Independent ID: eae9fd16-57fc-1a80-3ce9-9b34b68eb1e0
Content: Manage site creation in SharePoint Online
Content Source: SharePoint/SharePointOnline/manage-site-creation.md
Service: sharepoint-online
GitHub Login: @kaarins
Microsoft Alias: kaarins

@AndreaBarr : Sorry for this late response ... did not see the notification earlier ...
Yes, for sure. Thanks a lot :-)


Update to new attriubutes API and use pyright

Closes https://github.com/DiamondLightSource/fastcs-odin/issues/40
Uses FastCS from github main currently until a release is made for the new API.

Github glitched out my comments so they weren't applied to the threads on Gary's review :upside_down_face:


Return IHtmlContent from GetHtml()

part of aspnet/Mvc#3123
also ignore launchSettings.json files

:up::date: and changed title

:shipit:

78face4 for .gitignore update
6a9b38d


Replacebcftools functionality with R package

Currently LD.filter_vcf requires bcftools CLI to be installed.
Better to replace with something like VariantAnnotation package

This is now done all in one efficient step during VCF querying:
https://github.com/RajLabMSSM/echoLD/blob/main/R/query_vcf.R


Subtitle control

Will there be a way to add subtitle settings like you have for audio think that would be great the only thing is missing.

Yes, you should be able to view subtitles in the Fire stick FCast player if you enable them for the video on Grayjay before casting to the Fire stick. However seems like the video does not load after enabling subtitles if you do this, so will look into fixing this bug in a future update.

I got excited today cuz I just saw there was an update. It looks like on
the fire stick the app uploaded to version 41 from version 40. So I got all
excited thinking that. Oh my God did they fix it already but unfortunately
it's not done. I took that screenshot and I circled so you could see where
you can pick from the different audio formats to play back. But as you can
see, the only thing you can do other than picking the audio is to adjust
the speed of the audio. It would be so great if you could just add an
option right down in the same spot for the subtitles. Okay? Cuz I can't get
subtitles any other way I'm trying to use grayjay to push the subtitles but
having some issues so I just wanted to send it. Sorry this email is so long.
On Fri, Dec 6, 2024, 4:56 PM Michael Hollister @.***>
wrote:

Yes, you should be able to view subtitles in the Fire stick FCast player
if you enable them for the video on Grayjay before casting to the Fire
stick. However seems like the video does not load after enabling subtitles
if you do this, so will look into fixing this bug in a future update.
—
Reply to this email directly, view it on GitHub
https://github.com/futo-org/fcast/issues/13#issuecomment-2524717567, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BIPAMY6DM22G224XPS6R7A32EJBSDAVCNFSM6AAAAABTDSCFUCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRUG4YTONJWG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Minimal project descriptor

Readable
See also #25

Updated based on feedback from 12/5/19 WG meeting

Is the idea here that platforms would convert into the project descriptor from whatever they support for metadata files?
Would unimplemented attributes raise an error? So a specific platform might extend the descriptor internally, for instance for extending os package support.

Is the idea here that platforms would convert into the project descriptor from whatever they support for metadata files?

We haven't defined how a platform will support this file. We hope that eventually it will make it's way into the spec. If it does, we generally believe that lifecycle should in some way honor it (when possible). How that manifests itself is undecided (a new /lifecycle/describer or inline with each binary maybe?).

Would unimplemented attributes raise an error?

No. The only error condition we define is when exclude and include are both present.

So a specific platform might extend the descriptor internally, for instance for extending os package support.

Could this fit within the arbitrary [metadata] table?

Is the [project] property source-url intended to duplicate the property ("io.buildpacks.app.source".metadata.url) in RFC 0013?
Is it possible to include the metadata properties described in RFC 0013 within the project.toml file proposed in this RFC?

@iainsproat i'm not sure if RFC0013 was ever implemented, but we should probably align this with it. However, I think the app should be replaced with project ("app" has become too overloaded)

@jkutner It's true that RFC0013 has not been implemented yet. It has already been agreed upon and we have a pending PR into the specification. We could replace app with project though I'm not sure of the process required to update a previously accepted RFC. Does this require a new RFC?

@nebhale i've addressed your feedback, and loosened the requirements on license type and uri.


Switching of pages at server-side paging with filter doesn't work or works from the second time.

I'm submitting a ...  (check one with "x")
[x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here

Current behavior
Switching of pages at server-side paging with filter doesn't work or works from the second time. Showing empty area without default 'empty' text (No data to display):
Expected behavior
The pager must switched to next page and show new fetched rows
Reproduction of the problem
the component code
http://pastebin.com/pA4bYezn
the template code
http://pastebin.com/ncWhjvPX
screenshots
before search:
http://dl1.joxi.net/drive/2016/11/08/0004/1376/279904/04/32943b796c.jpg
on first trying search:
http://dl2.joxi.net/drive/2016/11/08/0004/1376/279904/04/ed5eee6e99.jpg
on second trying search:
http://dl2.joxi.net/drive/2016/11/08/0004/1376/279904/04/e5ebbeec88.jpg
on change page
http://dl1.joxi.net/drive/2016/11/08/0004/1376/279904/04/ad71a0fc9d.jpg
What is the motivation / use case for changing the behavior?
I think what it is bug
Please tell us about your environment:
Windows10
WebStorm
npm 3.10.8
node v6.8.1

Table version:
1.3.x

Angular version:
2.0.x

Browser:
all

Language: [all | TypeScript X.X | ES6/7 | ES5]
all

Can you make a demo plunkr?

it is difficult (
I can as that differently to help to reproduce a problem?

I cant create the demo plunkr, but can take more info about this bug.
The bug i show an in the demo server-side pager.
So... source code fully from the demo server side pager...
http://pastebin.com/KnfYuu8Y
Current behavior:
ctrl+R (chrome) in component with table(only):
http://dl2.joxi.net/drive/2016/11/10/0004/1376/279904/04/8e0d47c75d.jpg
In markup datatable directive I see it: http://dl1.joxi.net/drive/2016/11/10/0004/1376/279904/04/832046c217.jpg
...
but after resize browser window all working all right: http://dl2.joxi.net/drive/2016/11/10/0004/1376/279904/04/46299f6c6b.jpg
Fix pls it.

Please retest on latest.

Im use this solution
https://github.com/swimlane/angular2-data-table/issues/261

I don't know but maybe this is also related to #486

This is a duplicate of #515. See that issue and associated PR for details.


Rate limiting Rest api using spring cloud gateway not working

I am trying to rate limit my rest api's using spring cloud gateway ,and I am using the below dependencies:
org.springframework.cloud
spring-cloud-starter-gateway
Finchley.RELEASE
org.springframework.boot
spring-boot-starter-data-redis
I am using spring boot version 2.0.0.RELEASE .
my application.yml file :
spring:
 cloud:
    gateway:
      routes:
      - id: greeting-service
        uri: http://localhost:8091
        predicates:
        - Path=/greeting/helloworld
        filters:
        - name: RequestRateLimiter
          args:
            key-resolver: "#{@userRemoteAddressResolver}"
            redis-rate-limiter.replenishRate: 1
            redis-rate-limiter.burstCapacity: 1

@Bean(name = "userRemoteAddressResolver")
    public KeyResolver userKeyResolver() {
        return exchange -> {
            return Mono.just(exchange.getRequest().getRemoteAddress().getAddress().getHostAddress());
        };
    }

and an application startup and controller class.
But the rate limiting is not working.
I'm able to send multiple requests and the excess requests are not getting blocked.
All requests return 200 OK.
Am I missing something here?

The Finchley Release train is going to end of life in August and Boot 2.0.x is no longer supported.  Can you try the latest Boot 2.1.x release with Greenwich.SR3.

@ryanjbaxter:
Thanks for your reply.
I tried using spring boot version of 2.1.6.RELEASE and spring-cloud.version Greenwich.SR2.
And still I get 200 response for any number of requests.

getting a 200 response doesn't mean it is not working. How are you testing it? How many requests a second are you sending?

I've gives the rate as redis-rate-limiter.replenishRate: 1 and redis-rate-limiter.burstCapacity: 1 ,in the application.yml file
I am testing this by postman client.
Since I am getting 200 response for any number of requests, how will I be sure that it sends 429 error when the rate exceeds?

I also tried to test this by writing test case using springRunner.
(just called the endpoint in a loop of 10 in the test class)
these requests were fired within a second,but the tests also sent me 200 response for all the 10 requests,without a single request failure

Can you provide a complete, minimal, verifiable sample that reproduces the problem? It should be available as a GitHub (or similar) project or attached to this issue as a zip file.

You can have a look at it here:
https://github.com/Priyanka-khadke/testRepo

Spring mvc and tomcat are not compatible with gateway, please remove them

I tried removing the webmvc and tomcat dependency.
When i remove this I'll not be able to run the test class.
I still tried to test endpoint using postman client.
And it behaves the same manner as before.

There are a couple of reasons why this is not working. First and foremost is that Spring Cloud Gateway is build on a reactive stack. That's why you shouldn't use spring-webmvc and tomcat. You should also use the reactive starter for Redis: spring-boot-starter-data-redis-reactive.
The second problem is that you have a RestController where the getSimple method is being exposed on the exact same path as your route definition, and because of that the incoming request will always be sent to the rest controller.
If you change the path in the application.yml file to /helloworld and your uri to http://localhost:8091/greeting this will work if you go to http://localhost:8091/helloworld.
That will trigger your gateway route and will send it downstream to http://localhost:8091/greeting/helloworld.


Make strings const in funcs.rs

Replaced hard-coded strings with constants.
#266
@mtshiba

Thanks!


Only use es2015 if supported

Move iteration support to helpers module.
Only add it when it is supported.
This would allow to avoid having to include babel runtime
See #386

Hello @kirjs. The method is the correct one. However, I feel that moving this part into the helpers isn't really a good thing since the helper will only be used once since the code is really relative to the cursor file and that it would be better to keep this logic inside it. What do you think?

I wanted to have it in a function to simplify testing, but then I couldn't see any reason for cursors to expose it, so I moved it to helpers.
Also if we're planning to have map (#383) for baobab instances, it would probably make sense to make it iterable too.
I don't have super strong feelings about the placements though, let me know if you still want me to move it into the cursor file.

The tree's methods are bootstrapped from the root cursor's prototype so I guess the helper wouldn't be needed. If you have some time, I am currently fixing the lib's linter. Let me commit this so you can have a clean base on which to work on because I fear you may have merging issues otherwise.

Done! I've updated the linter etc.

Any update @kirjs?

Will do this week10:14 AM, December 1, 2015, Guillaume Plique notifications@github.com:Any update @kirjs? —Reply to this email directly or view it on GitHub.     Sent from Yandex.Mail for mobile: http://m.ya.ru/ymail

Hi Guillaume,
I got rid of the helper code, and just added the check in the cursor module instead.
I tried running npm lint but got a bunch of errors in other files, ptal;
kirjs$ npm run lint

><EMAIL_ADDRESS>lint /Users/kirjs/code/baobab
> eslint ./*.js && eslint ./src

/Users/kirjs/code/baobab/benchmark.js
   1:1   error  Unexpected var, use let or const instead                no-var
   1:1   error  Split 'var' declarations into multiple statements       one-var
   2:5   error  Expected indentation of 2 space characters but found 4  indent
   4:1   error  Unexpected var, use let or const instead                no-var
   6:1   error  Unexpected var, use let or const instead                no-var
   9:1   error  Unexpected var, use let or const instead                no-var
  12:22  error  Missing function expression name                        func-names
  15:22  error  Missing function expression name                        func-names
  18:24  error  Missing function expression name                        func-names
  21:29  error  Missing function expression name                        func-names
  22:17  error  There should be no space after '{'                      object-curly-spacing
  22:22  error  There should be no space after '{'                      object-curly-spacing
  22:27  error  There should be no space after '{'                      object-curly-spacing
  22:32  error  There should be no space after '{'                      object-curly-spacing
  22:39  error  There should be no space before '}'                     object-curly-spacing
  22:41  error  There should be no space before '}'                     object-curly-spacing
  22:43  error  There should be no space before '}'                     object-curly-spacing
  22:45  error  There should be no space before '}'                     object-curly-spacing
  25:31  error  Missing function expression name                        func-names
  28:30  error  Missing function expression name                        func-names
  31:33  error  Missing function expression name                        func-names
  34:32  error  Missing function expression name                        func-names
  37:16  error  Missing function expression name                        func-names

/Users/kirjs/code/baobab/gulpfile.js
   1:1   error    Split 'var' declarations into multiple statements       one-var
   1:1   error    Unexpected var, use let or const instead                no-var
   2:5   error    Expected indentation of 2 space characters but found 4  indent
   3:5   error    Expected indentation of 2 space characters but found 4  indent
   4:5   error    Expected indentation of 2 space characters but found 4  indent
   5:5   error    Expected indentation of 2 space characters but found 4  indent
   6:5   error    Expected indentation of 2 space characters but found 4  indent
   7:5   error    Expected indentation of 2 space characters but found 4  indent
   8:5   error    Expected indentation of 2 space characters but found 4  indent
   9:5   error    Expected indentation of 2 space characters but found 4  indent
  10:5   error    Expected indentation of 2 space characters but found 4  indent
  13:1   error    Unexpected var, use let or const instead                no-var
  13:1   error    Split 'var' declarations into multiple statements       one-var
  14:1   warning  Line 14 exceeds the maximum line length of 80           max-len
  14:5   error    Expected indentation of 2 space characters but found 4  indent
  17:23  error    Missing function expression name                        func-names
  24:34  error    Missing function expression name                        func-names
  28:21  error    Missing trailing comma                                  comma-dangle

✖ 41 problems (40 errors, 1 warning)

npm ERR! Darwin 15.0.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "lint"
npm ERR! node v5.1.0
npm ERR! npm  v3.3.12
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>lint: `eslint ./*.js && eslint ./src`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>lint script 'eslint ./*.js && eslint ./src'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the baobab package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     eslint ./*.js && eslint ./src
npm ERR! You can get their info via:
npm ERR!     npm owner ls baobab
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/kirjs/code/baobab/npm-debug.log

Hello @kirjs. Thanks for the update! Let me pull this and fix the merging issues so you don't have to deal with those :).

Done @kirjs! Thanks.

Do you need a new release on npm soon?

No rush here, I'm using a pre-build version.
Thanks a lot!

Just released 2.2.0 @kirjs. Thanks for your help!


Revert "Revert: "build(ci): workflow review" (#9414)"

This reverts commit b2d1c70c5d9d3995403ba897fc5b9e727c6dbfd5.
This PR resolves []
Brings back the work flow review from before.
Description

PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against 37a9399a2ee92a0ba9644db9c6062c10b9c38b53


fix: filter invalid chars for gradle wrapper

Some special chars in the app name can cause conflicts when you try to add a gradle wrapper. In this pull request I filter all invalid chars before generating settings.gradle
Here is an example app containing invalid chars: https://who-becomes-rich-български.apk.gold/
Steps to reproduce:

save as gradle project
run 'gradle wrapper'

FAILURE: Build failed with an exception.

* Where:
Settings file 'settings.gradle' line: 2

* What went wrong:
A problem occurred evaluating settings.
> The project name 'Who Becomes Rich?' must not contain any of the following characters: [/, \, :, <, >, ", ?, *, |]. Set the 'rootProject.name' or adjust the 'include' statement (see https://docs.gradle.org/7.6/dsl/org.gradle.api.initialization.Settings.html#org.gradle.api.initialization.Settings:include(java.lang.String[]) for more details).

This issue can also easily be reproduced with new Android Studio project. Android Studio does not allow any special chars in the application name when the app is generated, but AndroidManifest.xml can be edited later like in my unit test.

@nitram84 thanks! Nice fix.


Cucumber-style code generator from feature files

The GherkinSyntax capability of Spectrum makes it a substitute for

runtime parsing of feature files
"glue code" which binds the Gherkin syntax to runtime code

In Spectrum, both of the above are expressed in the initialisation of a test class in a single place.
In CucumberJVM, the "glue code" (e.g. @Given("there are cukes") public void there_are_cukes() {}) is not hand-written, it's generated. This is how you bind code to the feature files and this is how it's easy for testers or BA's to update feature files and for them to be used in tests.
While Spectrum can in a lot of situations be simply easier and neater than using Cucumber, there's one big difference in the process. With Spectrum as it stands a developer has to manually convert a Gherkin spec into the Java code and if the tester/BA were to change the original Spec, nobody would necessarily notice and it would be a manual conversion if they were.
Proposal: provide a means of scanning a source of .feature files and mapping them to the respective test classes and spec definitions within those classes in Spectrum. This would allow:

start with feature file and generate skeleton of test
scan that the tests match the feature files
provide diffs between tests and feature files, proposing the code change to put the skeleton in

In Cucumber, this check runs before every test, and can be set to make the test fail, or just be a warning. The output of cucumber might be:
Missing steps. Try adding:

@Given("there is a cuke")
public void there_is_a_cuke() {
    throws new PendingException();
}

We could do something similar in Spectrum. There could be a feature file scanner which might output:
Warning: missing test class for feature file "MyFeature.feature" with:

    Feature: My feature
    Scenario: My scenario

    Given a cuke
    When eat cukes
    Then no cukes

Try adding:

@RunWith(Spectrum.class)
public class MyFeature {{
    feature("My feature", () -> {
        scenario("My scenario", () -> {
            given("a cuke", () -> {
                pending();
            });
            when("eat", () -> {
                pending();
            });
            then("no cukes", () -> {
                pending();
            });
        });
    });
}}

Or maybe it could output diffs
Warning class "SomeFeatureTest" has extra:

    feature("some deleted feature" ...)

I think this would not be difficult to produce, and would make Spectrum a serious contender for a CucumberJVM replacement.

Very interesting idea. Can you describe more detail about the workflow the Cucumber provides, for those of us who don't use it often? For example, what's the flow when adding a new feature to the system? (Create a .feature file?, then run program X?, then .... ?)

In Cucumber you normally:

Write a feature file

This might re-use the text of existing test steps you'd written previously and that's part of the idea of Cucumber, to build a specification language
It might contain new Given, When or Then descriptions

Execute the test (with either fail on pending or not)

Any pending steps - i.e. ones which have no glue code implementation for the Gherkin steps mentioned - will be flagged
For each pending step, the skeleton of the glue code is output to the console for you to paste into your step definitions classes

When you define a step, you annotate a function in a glue code class to match what's in feature files, then you make that function execute the step

Functions can be parameterless, or can take values or objects implied by the text of the step
Where there are values in the text, these need to be selected for by regexes in the @Given/@When etc annotation on the function
Cucumber will assemble pojos from multiple values if you wish - it can also assemble tables

After each round of test definition (feature file modification) and glue code construction, you re-run the tests to see what's now working/not/missing


Gene name text overflow shows ellipsis

General information
Associated issues: #53
Checklist
Author:

[x] One or more reviewers have been assigned.
[ ] Automated tests have been included in this pull request, if possible, for the new feature(s) or bug fix.
[x] The associated GitHub issues are included (above).
[x] Notes have been included (below).

Reviewers:

[x] All automated checks are passing (green check next to latest commit).
[ ] At least one reviewer has signed off on the pull request.  Reviewers have two business days to review the pull request, after which the author may merge in the pull request unilaterally.

Notes
Pull request adds...

Long gene names in the gene panel are abbreviated with an ellipsis
The full text of the gene name is shown when the gene is expanded. This only happens for gene names that are abbreviated.


Split Microsoft.Owin.Security.Cookies.Interop into 2 packages

We'd like to use the ticket serializer and the data protector adapter in the OWIN/Katana version of ASOS, but having a dependency on the cookies middleware is something we'd like to avoid. Separating the cookies extension from the rest of the package would be ideal.
https://github.com/aspnet/Security/pull/408#discussion_r38157266
https://github.com/aspnet-contrib/AspNet.Security.OpenIdConnect.Server/issues/188#issuecomment-163291304
If you agreed with the general idea, I (or @MonkeyJamboree) would be happy to submit a PR. I guess simply creating a new Microsoft.Owin.Security.Interop package containing the ticket serializer and the data protector would be the best way to go.
/cc @Eilon @HaoK @Tratcher

We are not planning on doing this; we recommend just copying the code as needed.

TBH, this doesn't seem like a good idea, as the ticket serializer adapter must exactly match the one used by ASP.NET 5, which is an easy task when using an official package but rather fragile when rolling your own version.

@Eilon if you don't like the idea of having a new package, maybe we could simply make the new Microsoft.Owin.Security.Interop I guess a sources package?

@Eilon out of curiosity, why is copying the code recommended over actually separating out Microsoft.Owin.Security.Interop into its own package? It seems to me since the namespace and code is assumed to be for interoperability, it should be consumable by others easily. If not, is there a package that's explicitly for this purpose instead? Creating unneeded dependencies in code or duplicating code are usually are not good things.

This package was built specifically for interop with Microsoft.Owin.Security.Cookies.
One way to pull the  src is via git submodule.

This package was built specifically for interop with Microsoft.Owin.Security.Cookies.

True, but the ticket serializer and the data protector shim are not specific to the cookies middleware. Sounds like a bad layering IMHO.

The ticket serializer was for Forms auth. Which became cookie auth, Arguably it is for the cookie middleware.
Data Protection has a 4.5.2 implementation which you can bring in as a normal nuget package.

The ticket serializer was for Forms auth. Which became cookie auth, Arguably it is for the cookie middleware.

That would be a valid argument if the ticket serializer was not HEAVILY used by Katana's OAuth2 middleware...

It's not any more though is it? And the interop package itself wasn't for OAuth either. It was designed for forms authentication, nothing more.

It's not any more though is it?

Of course, since there's no OAuth2 middleware in vNext :trollface:
But people (like me) might want to use the new ticket serialization format and the new data protection stack with Katana's authorization server.
There's absolutely no difference between cookies and access tokens as they use the same exact serialization format (which explains why the OAuth2 authorization server/bearer middleware and the cookies middleware share the same TicketDataFormat). You're just introducing an artificial distinction.

So, wouldn't a new Microsoft.Owin.Security.Interop sources package be a good compromise?

+1

+1
Yeah, that'd be great. And I really don't see any logical arguments why not... I understand you don't want to support multiple packages, but essentially it's splitting same code that you have. Not much of a difference for you, a whole lot benefits for us.

No Christmas gift for us? :smile:

Re-opening, moving to Backlog, and marking as up-for-grabs. If someone wants to take a crack at this then we can publish the package with the code.
However, be warned: source packages have very few guarantees around them: stuff can change at any time. This is a use-at-your-own-risk type of scenario.

Ahhhhhhhhhh, thanks! :gift: :dancers:

Re-opening, moving to Backlog, and marking as up-for-grabs. If someone wants to take a crack at this then we can publish the package with the code.

I'll prepare a PR and send it ASAP :smile:

However, be warned: source packages have very few guarantees around them: stuff can change at any time. This is a use-at-your-own-risk type of scenario.

Would you prefer a standard package, with just the ticket serializer and the data protector?

I think the audience for this package is super tiny, so I'd rather do something very un-fancy, if possible :smile:

As long as this package is managed by the ASP.NET team and published on NuGet, I'm fine :smile:
Question: the Interop package doesn't have its own PropertiesSerializer and relies on the properties serializer developed for Katana, Currently, it's not really a big deal as Katana's PropertiesSerializer and vNext's PropertiesSerializer are identical, but it might cause weird issues if vNext's PropertiesSerializer had to be updated in the future. Is this something you plan to fix? /cc @HaoK

Would need @HaoK and/or @Tratcher to answer that. They might both be on vacation, though.

We don't have any plans to fix it yet because it's not broken :-). If at some point it needs a similar interop approach then we could use a similar solution.


Windows inline ps1 when exec privilege restricted

In restricted windows machines, it is forbidden
to specify a powershell execution policy and
it is usually forbidden to execute an unsigned script.
Inlining the script as an encoded command avoids that problem.
Note that the encoded command cannot be passed a parameter:
so we need to place the value of the image path inside the script
and generate it dynamically on every invocation.

If one trusts things downloaded from the internet, here is my build of the plugin with this patch: https://github.com/hmalphettes/vscode-paste-image/releases/tag/1.0.4.inlinedps
The safer thing to do is of course to build it yourself or apply the patch on the installed plugin sources.


Making blocks_2d for SD U-net are more configurable

This change provide more options to control hidden channels in different blocks

please also wait for @yichunk 's LGTM, thanks!


Allow WMarker term as textblock

Uses the term as a textblock so that words in a word marker are rendered out when getting textblocks of a higher marker such as VMarker.

Blocked until USFMToolsSharp #142 is resolved.


T63475: Imported GLTF files with keyframes creates strange graphs until manually updated

See issue on blender tracker:
https://developer.blender.org/T63475

Probably linked to tangent of CUBISPLINE, not implemented yet

Fixed by 1ea44df8a71d20972f33b219a2d61aa6cbb826d9
Note that tangents are still not managed from glTF file, but at least, there is now correct animation in Blender


Routing problem /sth-abc->App:sth/foo (ok)    and   /sth/abc->App:sth/foo (nok)

I tried to use normal strings instead of parameters ($1) in controller calling.
$routes->get('/sth-abc',                       'App::sth/foo',  ['subdomain'=>'']);
$routes->get('/sth/abc',                       'App::sth/foo',  ['subdomain'=>'']);
In the  case route is defined as: '/sth-abc',  everything works as expected - www.host.com/sth-abc shows page returned by App::sth with foo as first parameter. In the second case when route is '/sth/abc'  www.host.com/sth/abc returns 404.
I'm not sure if it is bug, because there is no example like this in docs but it can be a useful feature.

I thought we had fixed it, but there might be a bug still in the routing that doesn't like have a leading slash in the "from" portion of the route. Try rewriting that more like this:
$routes->get('sth/abc', 'App::sth/foo', ['subdomain'=>'']);


chainermn.links.create_mnbn_model raises an assertion error for functions in chainer.Sequential

Chainer: 6.0.0
The reproduction code is below:
import chainer
import chainermn
import chainer.functions as F
import chainer.links as L

comm = chainermn.create_communicator('naive')
net = chainer.Sequential(F.relu)

net = chainermn.links.create_mnbn_model(net, comm)

Thank you for the issue. I will check it.


Linq Extension Methods Returning Empty Result

The Linq extension methods are returning an empty result.
This is due to the type discriminator change from __type to _type.
Update the CosmosDatabase.ToAsyncEnumerable(...) method to filter on _type.

Closed per #99.


🚐 Road map

At the end of this road map, we will have a responsive and customizable atomic CSS framework.
To avoid reinventing the wheel, we would follow the naming structure of Stacks.
Checklist

[x] Display
[x] Spacing vertical
[ ] Spacing horizontal
[ ] Font size
[x] Positioning
[x] Typography
[x] Flex
[x] Background
[x] Object fit
[x] Transitions
[x] Color

[ ] Cursors: https://stackoverflow.design/product/base/cursors/

@vuquangpham Can you help me with this one?

[ ] Line-height: https://stackoverflow.design/product/base/typography/#line-height

Allow updating line-height list on config file.

[ ] color:inherit;

@vuquangpham
Responsive for .flex-grid-template
.flex-grid-template.grid-col-xs-1 {}
.flex-grid-template.grid-col-xs-2 {}

.flex-grid-template.grid-col-sm-3 {}
.flex-grid-template.grid-col-sm-2 {}

.flex-grid-template.grid-col-md-2 {}
.flex-grid-template.grid-col-md-3 {}

.flex-grid-template.grid-col-lg-3 {}
.flex-grid-template.grid-col-lg-4 {}


Repair ingress tests

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

/retest

/retest

/retest

/retest


Store the default setting value for the app.

Persist the default user name for the gitlabci app.

Do we still need this PR?

Actually not. The setting has been removed from the app.


exception for php version 7.4

Bug report
Title
bagisto version 0.2.2 throwing an exception for php version 7.4
Issue Description
bagisto version 0.2.2 throwing an exception for php version 7.4
Preconditions
1. framework Version- 0.2.2

Steps to reproduce
1. step1- create new product/ search product throw this error

Actual result

this issue has been fixed in higher version https://github.com/bagisto/bagisto/issues/2021


Style differences

Synopsis
We've noticed that some of the styles differ between the plugin test page and the initial example. Should those styles be explicitly set or not?
Details
These differences exist because some of the styles are being inherited from the theme, and aren't explicitly set within the plugin.
Since this is a plugin which will get activated on lots of different sites with lots of different styles, there may be some styles should be explicitly set, and therefore override the theme styles (such as vertical alignment), and some styles which would be good to inherit from the theme (such as font choice).
Here are some possible places where styles maybe should be explicitly set within the plugin:

Vertical spacing between the blog titles
Vertical alignment within the lines (the arrows vs. the titles not being vertically centered)
Possibly more

Screenshot

I did start working on this and was able to fix one of the problems, but I think the fonts aren't getting loaded properly, is that something you can check on? That would be where I'd start. I think that will straighten out some of the alignment problems.

Indeed, the fonts aren't being loaded from the plugin. If those fonts are going to be baked into the feeds, then we should be able to just load them directly from Google without too much overhead.
I tested this locally by enqueueing the fonts directly from Google:
wp_enqueue_style(
    'feedland-google-fonts-ubuntu',
    '//fonts.googleapis.com/css?family=Ubuntu:400,500i,700',
);

wp_enqueue_style(
    'feedland-google-fonts-rancho',
    '//fonts.googleapis.com/css?family=Rancho',
);

It works just fine, and cleans up the styling differences (screenshot):

Fernando, would you be able to spend some time checking if that works on the public test site?

It looks a lot better but there are still some tweaks to be made, on my end, in the styles.

This is, for me, the last big thing to do before we can use this with other people.
We have to make the blogroll look excellent without dependencies on the CSS environment it's running in.
Here are two screen shots of where we're at, in the blogroll as displayed on blogroll.social and on the test page, as they appear now.

So where are we at font-wise?
I think I heard that Ubuntu isn't loaded?
That might account for some of the extra spacing between lines?
The text in the right example is too big, again might be due to not having Ubuntu loaded.
Also I'm going to try to see about getting the border around the table. That's a big missing piece, back in a minute.

@scripting The font not loading there is almost certainly causing the internal style differences. I can help tackle that today while Fernando focuses on server issue elsewhere. Should be able to get it resolved by EoD today.

@scripting I believe it's because the div has an ID, not a class of divBlogrollContainer
For your styles to apply, it should be #divBlogrollContainer instead of .divBlogrollContainer.
I think this should solve it.


deprecated function interpreter is still called

Repl call an deprecated function interpreter
    val i = {
      val l:HackSparkILoop = loop.asInstanceOf[HackSparkILoop]
      l.interpreter
    }

the spark code:
  private[repl] var intp: SparkIMain = _
  @deprecated("Use `intp` instead.", "2.9.0") def interpreter = intp
  @deprecated("Use `intp` instead.", "2.9.0") def interpreter_= (i: SparkIMain): Unit = intp = i

SparkILoop change intp  as a private variable and the deprecated function interpreter maybe removed at any time.
If that happen, how will spark-notebook do?

recently i have an Chinese article about spark repl: 图解spark repl
We need to interact with spark cluster gracefully.  I think spark-repl must support 3th repl call without trick


MicroPCF doesn't start without Internet connectivity

As requested by @davidwadden in #17 here it is a new issue:
Environment:
MicroPCF 0.5.0
Vagrant 1.8.1
VBox 5.0.14
Windows 7
When I start MicroPCF with Internet connectivity, after 5-6 minutes MicroPCF starts successfully.
If I try to start it without Internet connectivity, it gets stuck at:
==> default: Waiting for services to start...
After 12-14 minutes it still at that point. And as soon as I plug the ethernet (Internet connectivity) it finishes successfully. If I don't attach the ethernet which not always is possible it never finishes.
Don't know if this is an issue or not. I understand I need Internet connectivity to get a new buildpack or a docker image from the community, but if I just want to test my own things locally with no external interaction, should it work?

Hi @debu66er,
If you start MicroPCF without internet access, the *.local.micropcf.io CNAME will not resolve. This prevents MicroPCF from booting. It's possible to shim out the CNAME with dnsmasq on the host. See here.
We have a story in our backlog to address the startup issue by introducing dnsmasq inside the VM.

I successfully followed this guide on OSX https://passingcuriosity.com/2013/dnsmasq-dev-osx/

Hi , I have same issue, But I have internet connectivity through proxy. I am able to download vagrant micropcf files through proxy but after that it hangs at**_ ==> default: Waiting for services to start..._** for long time. Can you help me. Many thanks!!!!

+1 not proxy friendly.  Everything fine up to waiting for services to start.

+1 on the proxy sticking at "Waiting for services to start."

@benlaplanche I tried to follow the guide that you mentioned, but it hangs on "Waiting for services to start...".  If I put vagrant in UI mode, it brings up a login prompt at that point.  Any suggestions?

@sclevine What does the story around this look like?
We are looking at PCF for our enterprise.  This looked promising for improving the dev experience with distributed cloud architectures however its seems impossible to get this to work behind our corporate proxies.  I am not sure if I am missing a step or if that is a use case you do not plan on supporting.
Thanks,

Nvm now I see @davidwadden posted a deprecation notice.  Lattice- > micropcf -> Pivotal Network... I am getting dizzy.
I will reach out to our account manager and get the closed beta I guess.  I assume it has feature parity with what this offers..

Hi @wgorder,
Happy to hear that you've found MicroPCF useful! We're working on an early customer access group for the PCF Dev closed beta on Pivnet. We also plan to resume our Github presence very soon. Unlike the product transition from Lattice to MicroPCF, the PCF Dev name change is more of a re-branding with a greater focus on providing PCF functionality locally.
The latest closed beta release of PCF Dev should start behind a proxy, and we have work in flight for more complete support.
Please let us know if you run into any other issues,
Stephen


Rename to simpla-paths, inline HTML dist

Just made a couple of tweaks - hope that's ok. There was a bug in the gulpfile and just a few files missed in the renaming.
Other than the comment I've made above, all good 👍

Looooks good, I'll merge. Feel free to change xfb thing if you want


make omit_nwis_sites a target

Currently the omit_nwis_sites is a character vector that is defined in _targets.R. The problem with this is that if/when we update this vector, targets doesn't recognize that a "dependency" has changed. That is because it's not a targets::tar_target. So I think it would be best to define it as it's own target. This is how Jake & Co. approached similar situations in the temp forecasting pipelines.
cc @jds485  - we'll need to think about this for the salinity pipeline too

I didn't realize that, thanks for pointing this out, @jsadler2. We're currently using omit_nwis_sites in DO and inland-salinity to omit tidal sites that are coded as "ST" in NWIS.

Are you sure? I just tested what changing that vector of values would do and it looks like it's picked up as a change

hmm. I'm not sure. I thought that's the behavior I noticed, but let me double check.
Always keeping us honest, @jds485 ! 😄

haha, that's what reviewers are for!

I stand corrected! Good catch, @jds485


Datatrail

Purpose/implementation Section
What changes are being implemented in this Pull Request?
adding style files to create a DataTrail branded template
What was your approach?
added files to the asset dir and created *_DataTrail files for the index and output files
What GitHub issue does your pull request address?
None
Tell potential reviewers what kind of feedback you are soliciting.
Look good? still waiting for the link to modify the feedback button

Should we not wait on this for Release 1.2? This doesn't really have implications for the current downstream repos anyway.

Should we not wait on this for Release 1.2? This doesn't really have implications for the current downstream repos anyway.

Yeah I think waiting on it makes sense... should hopefully get that link soon

These changes got added by @cansavvy  after she merged them into her PR to rearrange files for style.  See #334


DatabaseConnector or CDMConnector in suggests

At the moment TreatmentPatterns supports either DatabaseConnector or CDMConnector, but with DatabaseConnector in imports while CDMConnector is in suggests. But one problem I had with this is that when I run the analysis with CDMConnector, deploying an app with TreatmentPatterns, even just to call createSunburstPlot, requires the shiny server to have java installed (which is problematic for ours at the moment).
@mvankessel-EMC What about both DatabaseConnector and CDMConnector to be in suggests? Then if someone goes down the CDMConnector route they wouldn't need the heavy dependencies.

I agree, done one dev d8265f862160d7004bfb6ec4c6ea07bb0266b230

@mvankessel-EMC great, I think SQL render would also need to be in suggests to remove the java dependency for a shiny app?

Yes, done 62061d26e64906a13765d5997a7d5ba6bf137d85.


Fix doc for Dictionary.erase

The advice "Do not erase..." was changed to descriptive "Does not erase..." by mistake.
This PR reverts the change and documents a possible solution.

Thanks!

Original change at https://github.com/godotengine/godot/commit/f7f6115f7627df24a08a9a0882b2f573cc838eb1#diff-330b5efa394cd22fb33eb40c8e745680e3b70e819bf18f7cdbc86386937c82eeR50

Cherry-picked for 3.4.

Cherry-picked for 3.3.4.


⚠ Turnos 24x7 has degraded performance

In 8fd65b3, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 939 ms

Resolved: Turnos 24x7 performance has improved in f8efe6b.


[SUPPORT] Column used as Record key has to be added to hoodie.metadata.index.column.stats.column.list in order to have  column_stats index used for range pruning during upsert operations

I'm using Hudi 0.13.1 and using a COW table, BLOOM filter and meta table enabled, in specific enabling metatable indexes for bloom_filters,column_stats and files. In addition I'm using hoodie.bloom.index.use.metadata to enable range pruning through the metadata column_stats.
What I noticed is that with hoodie.metadata.index.column.stats.enable=true if we specify values for hoodie.metadata.index.column.stats.column.list the record key column is not included inside the column_stats index. As a result column_stats index cannot be used for range pruning during upsert operations.
In specific here sample scenarios I tested with the related results:
Scenario 1
inputDF = spark.createDataFrame(
    [
        ("W01D000", "53.3572085", "202001150800","2020","01"),
        ("W01D001", "53.3572085", "202001150800","2020","01"),
        ("W01D002", "53.3572085", "202001150800","2020","01"),
        ("W01D003", "53.3572085", "202001150800","2020","01")
    ],
    ["device_id", "value", "timestamp","year","month"]
)

inputDFNew=inputDF.withColumn("id",concat(inputDF.timestamp,inputDF.device_id))

RECORD_KEY="id"
PARTITION_FIELD="year,month"
PRECOMBINE_FIELD="timestamp"

DATABASE="default"

TABLE_STATS="table_1"
TABLE_STATS_LOCATION="s3://mybucket/datasets/table_1/"

hudi_options_with_statistics = {
    "hoodie.table.name": TABLE_STATS,
    "hoodie.table.type": "COPY_ON_WRITE",
    "hoodie.datasource.write.recordkey.field": RECORD_KEY,
    "hoodie.datasource.write.partitionpath.field": PARTITION_FIELD,
    "hoodie.datasource.write.hive_style_partitioning": "true",
    "hoodie.datasource.write.precombine.field": PRECOMBINE_FIELD,
    "hoodie.datasource.hive_sync.enable": "true",
    "hoodie.datasource.hive_sync.table": TABLE_STATS,
    "hoodie.datasource.hive_sync.database": DATABASE,
    "hoodie.datasource.hive_sync.partition_fields": PARTITION_FIELD,
    "hoodie.datasource.hive_sync.partition_extractor_class": "org.apache.hudi.hive.MultiPartKeysValueExtractor",
    "hoodie.datasource.hive_sync.use_jdbc": "false",
    "hoodie.datasource.hive_sync.mode":"hms",
    "hoodie.metadata.index.bloom.filter.enable": "true",
    "hoodie.bloom.index.use.metadata": "true",
    "hoodie.metadata.index.column.stats.enable": "true",
    "hoodie.metadata.enable": "true",
    "hoodie.index.type": "BLOOM"
}

inputDFNew.write.format("org.apache.hudi").option(
    "hoodie.datasource.write.operation", "upsert"
).options(**hudi_options_with_statistics).mode("append").save(TABLE_STATS_LOCATION)

inputDFNew.write.format("org.apache.hudi").option(
    "hoodie.datasource.write.operation", "upsert"
).options(**hudi_options_with_statistics).mode("append").save(TABLE_STATS_LOCATION)

The second upsert uses the index for range pruning. This because the hoodie.metadata.index.column.stats.column.list property is left empty and so all the columns are added inside the column_stats.

Scenario 2
inputDF = spark.createDataFrame(
    [
        ("W01D000", "53.3572085", "202001150800","2020","01"),
        ("W01D001", "53.3572085", "202001150800","2020","01"),
        ("W01D002", "53.3572085", "202001150800","2020","01"),
        ("W01D003", "53.3572085", "202001150800","2020","01")
    ],
    ["device_id", "value", "timestamp","year","month"]
)

inputDFNew=inputDF.withColumn("id",concat(inputDF.timestamp,inputDF.device_id))

RECORD_KEY="id"
PARTITION_FIELD="year,month"
PRECOMBINE_FIELD="timestamp"

DATABASE="default"

TABLE_STATS="table_2"
TABLE_STATS_LOCATION="s3://mybucket/datasets/table_2/"

hudi_options_with_statistics = {
    "hoodie.table.name": TABLE_STATS,
    "hoodie.table.type": "COPY_ON_WRITE",
    "hoodie.datasource.write.recordkey.field": RECORD_KEY,
    "hoodie.datasource.write.partitionpath.field": PARTITION_FIELD,
    "hoodie.datasource.write.hive_style_partitioning": "true",
    "hoodie.datasource.write.precombine.field": PRECOMBINE_FIELD,
    "hoodie.datasource.hive_sync.enable": "true",
    "hoodie.datasource.hive_sync.table": TABLE_STATS,
    "hoodie.datasource.hive_sync.database": DATABASE,
    "hoodie.datasource.hive_sync.partition_fields": PARTITION_FIELD,
    "hoodie.datasource.hive_sync.partition_extractor_class": "org.apache.hudi.hive.MultiPartKeysValueExtractor",
    "hoodie.datasource.hive_sync.use_jdbc": "false",
    "hoodie.datasource.hive_sync.mode":"hms",
    "hoodie.metadata.index.bloom.filter.enable": "true",
    "hoodie.bloom.index.use.metadata": "true",
    "hoodie.metadata.index.column.stats.enable": "true",
    "hoodie.metadata.index.column.stats.column.list": "year",
    "hoodie.metadata.enable": "true",
    "hoodie.index.type": "BLOOM"
}

inputDFNew.write.format("org.apache.hudi").option(
    "hoodie.datasource.write.operation", "upsert"
).options(**hudi_options_with_statistics).mode("append").save(TABLE_STATS_LOCATION)

inputDFNew.write.format("org.apache.hudi").option(
    "hoodie.datasource.write.operation", "upsert"
).options(**hudi_options_with_statistics).mode("append").save(TABLE_STATS_LOCATION)

The second upsert tried to use the index for range pruning but ended up loading column ranges from files. This because the hoodie.metadata.index.column.stats.column.list contained the value year and so only that column is added inside the column_stats.

Scenario 3
inputDF = spark.createDataFrame(
    [
        ("W01D000", "53.3572085", "202001150800","2020","01"),
        ("W01D001", "53.3572085", "202001150800","2020","01"),
        ("W01D002", "53.3572085", "202001150800","2020","01"),
        ("W01D003", "53.3572085", "202001150800","2020","01")
    ],
    ["device_id", "value", "timestamp","year","month"]
)

inputDFNew=inputDF.withColumn("id",concat(inputDF.timestamp,inputDF.device_id))

RECORD_KEY="id"
PARTITION_FIELD="year,month"
PRECOMBINE_FIELD="timestamp"

DATABASE="default"

TABLE_STATS="table_3"
TABLE_STATS_LOCATION="s3://mybucket/datasets/table_3/"

hudi_options_with_statistics = {
    "hoodie.table.name": TABLE_STATS,
    "hoodie.table.type": "COPY_ON_WRITE",
    "hoodie.datasource.write.recordkey.field": RECORD_KEY,
    "hoodie.datasource.write.partitionpath.field": PARTITION_FIELD,
    "hoodie.datasource.write.hive_style_partitioning": "true",
    "hoodie.datasource.write.precombine.field": PRECOMBINE_FIELD,
    "hoodie.datasource.hive_sync.enable": "true",
    "hoodie.datasource.hive_sync.table": TABLE_STATS,
    "hoodie.datasource.hive_sync.database": DATABASE,
    "hoodie.datasource.hive_sync.partition_fields": PARTITION_FIELD,
    "hoodie.datasource.hive_sync.partition_extractor_class": "org.apache.hudi.hive.MultiPartKeysValueExtractor",
    "hoodie.datasource.hive_sync.use_jdbc": "false",
    "hoodie.datasource.hive_sync.mode":"hms",
    "hoodie.metadata.index.bloom.filter.enable": "true",
    "hoodie.bloom.index.use.metadata": "true",
    "hoodie.metadata.index.column.stats.enable": "true",
    "hoodie.metadata.index.column.stats.column.list": "year,id",
    "hoodie.metadata.enable": "true",
    "hoodie.index.type": "BLOOM"
}

inputDFNew.write.format("org.apache.hudi").option(
    "hoodie.datasource.write.operation", "upsert"
).options(**hudi_options_with_statistics).mode("append").save(TABLE_STATS_LOCATION)

inputDFNew.write.format("org.apache.hudi").option(
    "hoodie.datasource.write.operation", "upsert"
).options(**hudi_options_with_statistics).mode("append").save(TABLE_STATS_LOCATION)

The second upsert uses the index for range pruning. This because the hoodie.metadata.index.column.stats.column.list property contains the id column which is used as hoodie.datasource.write.recordkey.field.

Scenario 4
inputDF = spark.createDataFrame(
    [
        ("W01D000", "53.3572085", "202001150800","2020","01"),
        ("W01D001", "53.3572085", "202001150800","2020","01"),
        ("W01D002", "53.3572085", "202001150800","2020","01"),
        ("W01D003", "53.3572085", "202001150800","2020","01")
    ],
    ["device_id", "value", "timestamp","year","month"]
)

inputDFNew=inputDF.withColumn("id",concat(inputDF.timestamp,inputDF.device_id))

RECORD_KEY="id"
PARTITION_FIELD="year,month"
PRECOMBINE_FIELD="timestamp"

DATABASE="default"

TABLE_STATS="table_4"
TABLE_STATS_LOCATION="s3://mybucket/datasets/table_4/"

hudi_options_with_statistics = {
    "hoodie.table.name": TABLE_STATS,
    "hoodie.table.type": "COPY_ON_WRITE",
    "hoodie.datasource.write.recordkey.field": RECORD_KEY,
    "hoodie.datasource.write.partitionpath.field": PARTITION_FIELD,
    "hoodie.datasource.write.hive_style_partitioning": "true",
    "hoodie.datasource.write.precombine.field": PRECOMBINE_FIELD,
    "hoodie.datasource.hive_sync.enable": "true",
    "hoodie.datasource.hive_sync.table": TABLE_STATS,
    "hoodie.datasource.hive_sync.database": DATABASE,
    "hoodie.datasource.hive_sync.partition_fields": PARTITION_FIELD,
    "hoodie.datasource.hive_sync.partition_extractor_class": "org.apache.hudi.hive.MultiPartKeysValueExtractor",
    "hoodie.datasource.hive_sync.use_jdbc": "false",
    "hoodie.datasource.hive_sync.mode":"hms",
    "hoodie.metadata.index.bloom.filter.enable": "true",
    "hoodie.bloom.index.use.metadata": "true",
    "hoodie.metadata.index.column.stats.enable": "true",
    "hoodie.metadata.index.column.stats.column.list": "year,_hoodie_record_key",
    "hoodie.metadata.enable": "true",
    "hoodie.index.type": "BLOOM"
}

inputDFNew.write.format("org.apache.hudi").option(
    "hoodie.datasource.write.operation", "upsert"
).options(**hudi_options_with_statistics).mode("append").save(TABLE_STATS_LOCATION)

inputDFNew.write.format("org.apache.hudi").option(
    "hoodie.datasource.write.operation", "upsert"
).options(**hudi_options_with_statistics).mode("append").save(TABLE_STATS_LOCATION)

The second upsert tried to use the index for range pruning but ended up loading column ranges from files. So it seems adding inside hoodie.metadata.index.column.stats.column.list the _hoodie_record_key colum, is adding that inside the column_stats but that is not used during range pruning.

Conclusions

In order to leverage column_stats for range pruning during upsert operations we have to include the column used as record key (i.e., the id column in my case) on the hoodie.metadata.index.column.stats.column.list list.

This should be added inside the Hudi doc for clarity or (better) record key should be always added in the column_stats and the hoodie.metadata.index.column.stats.column.list list should specify additional fields.

When using a Composite key (i.e., RECORD_KEY="timestamp,device_id") I see that I'm never able to leverage the column_stats for range pruning during upsert operations. Neither with "hoodie.metadata.index.column.stats.column.list": "year,_hoodie_record_key" nor with "hoodie.metadata.index.column.stats.column.list": "year,timestamp,device_id"  nor without specifying "hoodie.metadata.index.column.stats.column.list". In all cases I see the second upsert tried to use the index for range pruning but ended up loading column ranges from files.

This should reinforce what mentioned in point 1, record key should be always added in the column_stats and the hoodie.metadata.index.column.stats.column.list list should specify additional fields.
Anything I'm missing here?

Thanks for raising this with such a detailed explanation. Looking into this. @ssandona

@ssandona Yes, I checked and you were correct. Range pruning doesn't work with composite key, We will make those doc updates. Thanks.

@ssandona With this PR - https://github.com/apache/hudi/pull/9886, I confirmed that in scenario 4 and also with composite key range pruning started working also,

Fixed via: https://github.com/apache/hudi/pull/9886


Hi I have changed html and I fixed errors

Good Morning.

Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

This contains potentially hazardous tags that need manual review.

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

097f1fc2f0113e1605e55f3811fe19647f2e3145


Example snippets in readme throws

error: Uncaught TypeError: Cannot read property 'send' of undefined
    return this.webSocket!.send(message);
                           ^
    at WebSocket.send (websocket.ts:96:28)
    at client.ts:10:4

Removing the send line:
error: Uncaught Error: ws: server didn't accept handshake: version=HTTP/1.1, statusCode=302
    throw new Error(
          ^
    at handshake (mod.ts:495:11)
    at async connectWebSocket (mod.ts:539:5)
    at async WebSocket.createSocket (websocket.ts:57:23)

Let me know about your environment.
Deno version, OS, ...

@ryo-ma It was the k3s service-lb. It appears kubernetes is not notifying the underlying host (ubuntu bionic) that it is using port (8080 in this case). So Deno was happy to bind to that port.
I'm not sure if there is any way to fix that. So the second error above is fine (it's unexpected).
I'm going to leave this open though because the program shouldn't be throwing a TypeError.
The issue is that https://github.com/ryo-ma/deno-websocket/blob/master/lib/websocket.ts#L17 is async.
The code needs to wait for that to resolve before it accepts messages.

Thank you for your reply.
OK. I think too the program shouldn't be throwing a TypeError.
I will be throwing the other error when never finished initializing the connection.

I have fixed the error type and error message.
https://github.com/ryo-ma/deno-websocket/blob/10a1954ac81c3d09f4bb2269c1b03edf500c5c09/lib/websocket.ts#L114


port table designer fixes

original PR: #19020
This PR fixes #19006
This PR fixes #18999
This PR fixes #18997
This PR fixes #18987
This PR fixes #18985

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.329%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
27590

Relevant Lines:
60925

💛  - Coveralls

@kburtram the PR validation builds are completed now, could you please merge the PR? thanks!


小小影视在线观看免费完整组件

关于小小影视在线观看免费完整组件
>>>>>>点/此/进/入/最/新/观/看/入/口<<<<<<
愉悦小道的周边无秩序的生长着几株艳丽的野花红的粉的紫的虽混乱无章却给这幅美景更增添一份性感夹杂着一份纯洁的妖一

你可以将小小影视在线观看免费完整组件	18、感冒怎么比看到心爱的人出轨还要难受。

一个小电视看免费的在线完整的组件
Their home thing I know is not much, inconvenience, just thought, that more than seventy - year - old man, I don't know how difficult: would like to see q, but can't own.

请参考下方链接：
https://github.com/fowerv/qzdva/issues/1
https://github.com/makedete/mccflf/issues/1
https://github.com/downdesa/iual/issues/3
https://github.com/tooere/mwdpfj/issues/4
https://github.com/makedete/awjiuvt
https://github.com/aoyouzg/ganh/issues/7
https://github.com/goodiders/jsgvbgs/issues/2
https://github.com/tooere/fmofxdv/issues/3
https://github.com/makedete/prnuo
https://github.com/tooere/dkurp/issues/2
https://github.com/powercvt/uzarvzb/issues/5
https://github.com/planshow/hgkfot/issues/3
https://github.com/fowerv/tgtrw/blob/main/README.md
https://github.com/powercvt/bflrl/issues/1
https://github.com/tuderw/ooc-oocmq
https://github.com/planshow/ofnl/issues/3
https://github.com/planshow/kufrqk/issues/2
https://github.com/tooere/ecggvo/issues/3
https://github.com/powercvt/iymuu/issues/1
https://github.com/tooere/jpkqdvs/issues/5


[BUG] Querying SF cluster does not handle empty cluster certificate thumbprints

Library name and version
Azure.ResourceManager.ServiceFabric 1.0.0
Describe the bug
Calling
await armClient.GetServiceFabricClusterResource(sfClusterResourceIdentifier).GetAsync(cancellationToken); on a SF cluster with cluster certificate without Thumbpint pinning
throws
    System.ArgumentNullException: Value cannot be null.
    Parameter name: certificateIssuerThumbprint

  Stack Trace:
    ClusterServerCertificateCommonName.ctor(String certificateCommonName, BinaryData certificateIssuerThumbprint)
    ClusterServerCertificateCommonName.DeserializeClusterServerCertificateCommonName(JsonElement element)
    ClusterServerCertificateCommonNames.DeserializeClusterServerCertificateCommonNames(JsonElement element)
    ServiceFabricClusterData.DeserializeServiceFabricClusterData(JsonElement element)

We don't pin cluster certificate to thumbprints. This is a regression compared to Microsoft.Azure.Management.ServiceFabric nuget.
Expected behavior
Cluster model property deserializes
Actual behavior
Desiarization logic doesn't handle empty cluster certificate thumbprints
Reproduction Steps
See in bug description.
Environment
Windows 10 .NET Framework 462

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

I am also getting this error in the following scenario:

I set up a cluster with SF reverse proxy cert using common name
I cannot specify an issuer thumbprint for SF reverse proxy because the RP says it doesn't support it yet
When I use the SDK to read the SF cluster resource, the certificate constructor throws the error saying the issuer thumbprint is not set

@Yao725 Any update on this?

@Yao725 @jsquire Any update on this issue?

Unfortunately, I do not have insight.
@ArthurMa1978, @Yao725 : Would you please share an update on status for this issue?

@ArthurMa1978 @Yao725 Anyone looking into this?

@ArthurMa1978 @Yao725 Any update on this?


No insert_id and other critical properties?

At https://developers.amplitude.com/docs/http-api-v2 it's clearly stated that you highly recommend always passing in a insert_id, yet the sdk does not support this property? It is not part of the typescript typings.
Similarly, lots of properties are missing, e.g. os_name, app_version, etc.

@tommedema,
Thanks for raising this issue. You're totally correct that those types are missing - feel free to open a pr to @amplitude/types or we can look into resolving this issue this week; we see this as a great first issue.

Seems like there are a lot of properties missing, and this is more than just a typings issue? We'll be reverting to using the HTTP API again

@tommedema Hi! Could you explain more by what you mean by this being more than just a typings issue - have you run into issues using the Node SDK itself because of the event shape (adding insert id, os name, etc.), outside of type compilation issue?

I haven't been able to try and add more properties since typescript won't compile, so didn't realize you allow for passing in arbitrary event properties. Still, we'd like to deploy today so cannot wait for the typings to be updated.

We'd expect all properties at https://developers.amplitude.com/docs/http-api-v2 to be supported

Right, sorry about the lack of typings - why not cast the event sent into logEvent as any for the time being until this is resolved?
I think the call to use the HTTP API is also valid if the Node SDK is not providing any value here - we wanted to help remove the need to do your own HTTP + Batching + Retrying logic but those can be replicated

Ok -- we will be reverting back to the SDK then. We've never had to use any in our codebase so hopefully this can be resolved soon.

@tommedema,
Hi there, we want to give you an update about this issue.
The issue has been resolved in our most resent node v1.5.0. It includes the types of insert_id and other properties.
Again, thanks for raise the issue up.


chore(main): release 1.0.1

:robot: I have created a release beep boop
1.0.1 (2024-08-20)
Bug Fixes

typings: support generic type parameter to parse method (#7) (fea1d02)

Performance Improvements

improve toJSON performance (#5) (e23bc16)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wellwelwel/jsonc.min/releases/tag/v1.0.1 :sunflower:


config: upgrade envoy.ip_tagging to v3alpha.

This is a manual pilot of the v3 API type upgade inside of Envoy. The IP
tagging filter was picked as its API dependencies are relatively
confined in scope, allowing a self-contained PoC to be written.
This will be followed by automated upgrade via the API type database for
the bulk of Envoy's internals.
Part of #8082
Risk level: Low
Testing: Additional unit and integration tests added
(version_integration_test).
Signed-off-by: Harvey Tuch<EMAIL_ADDRESS>

@lizan @mattklein123 this should act as a litmus test for how the automated updates are going to look.