Potential issue with geographic location of Ungazikmit

Cassie Sprules (@csprules), who is working with me on a database, identified a potential issue with the geographic location of the Binford and EA society "Ungazikmit". Most of the source materials seem to focus on a group that is located on St. Lawrence Island and the Chukchi Peninsula, and the glottolog on D-Place also places them in St. Lawrence Island. However, the geographic location listed on D-Place (and the Binford R package) is inland Siberia. The distance between these locations is about 700km.
We were wondering whether there might be an error in Binford's dataset that has been inherited in D-Place and the EA. The difference in location could be accounted for by someone (potentially Binford) using the geographic coordinates 64.00, 173.00 (inland Siberia), as opposed to 64.00, -173.00 (the edge of the Chukchi Peninsula). It seems like it would be an easy mistake to make and could be worth checking?

Well spotted, thanks. @kirbykat -- can one of your hiwi's track this down?

This may already be in Xinyu’s database, but either way I suspect @csprules’ hunch is correct. We have found a number of such errors in the datasets. Xinyu is currently working on collecting lat/longs of sites at which orginal ethnographic work was carried out (I.e., ethnographic work upon which coded data are based).

Sorry - I meant to add I will check with Xinyu and get back to you ASAP.

Ok cool, I'd be keen to hear about any updates on the lat/long coordinates for Binford's dataset as we are currently working on extending a subset of it. How will you be archiving the PDFs? We have PDFs for a fair few Binford cultures that I'm not sure where to store once coding is complete. I guess these kinds of things cant be made public due to copyright, but it could be cool if we had a private storage place that we could share materials.

Agreed! It would be great to have a shared archive. I’ve talked a bit with @xrotwang about this, and I think we should be able to store them on an MPI server for internal use. Let me try to get a list together of all the PDFs we have so far (currently in various google drives, which we’re happy to share). The “big” part of this is going to be scanning texts that are not available in PDF format...I’m not sure how far we’ll get with that, but did get some advice from Piers and his hiwi on using the “book eye” scanner in the library. So, more on that in the next month, I think.

I agree with the change, which has now been applied to both the EA and Binford coordinates for Ungazikmit.


Python micro version

In commit aeb4e40124551ca72e3f4ee2da3a93173b235360 you added the Python micro version as requirement.
I think these versions are even interpreter specific and I see absolute no reason, why you are requiring that.
Shouldn't that changed back?

Will be brought in line with the template with the next PR.


Extensions and suggestions stopped working

After the last update of VS Code, all of our extensions such as automatic import replacement and typescript suggestions aren't working anymore. Also VS Code runs really slow.

For what is worth, @ #71863.
This is happening with several extensions. I did test in Linux/Ubuntu VM and the problem doesn't "seem" to exist there. However, it was running slow but since I have not used VSC in a Virtual Machine before, the speed could have been from the VM. It was inserting the snippets properly though.

Is this the same us this issue you guys have encountered /Seen ?
https://github.com/Microsoft/vscode/issues/72037

No, that is a different issue

@lewisflude upgraded @types/styled-components to 4.1.14 and pointed VSCode to use TS 3.3.3333 and all issues are gone. Thanks all.

I can confirm that forcing VSCode (1.33.0) to use TypeScript 3.3.3333 has fixed the problems I was experiencing with the related issue https://github.com/dsznajder/vscode-es7-javascript-react-snippets/issues/61.

Sounds like it could be #71886 but the description is too vague to tell. @MarioSamison please describe exactly what you are doing, what the expected behavior is, and what actually happens
Also test using latest VS Code insiders build

@mjbvz changing the typescript version to the workspace typescript version fixed everything for me. It seems like the new typescript version in VS Code collides with the typescript versions in the projects we're working on.

@MarioSamison In my case, I did not have typescript in my project, the version supplied in VS 1.33 was causing issues. I hope this is investigated before the next release.

@MarioSamison Can you please check to see if the new VS Code 1.33.1 release also fixed the issue for you
@polishedwp Typescript tooling also powers our javascript support

@mjbvz I had the same issue as @MarioSamison in 1.33.0 release. Still have the same issue in 1.33.1

@mjbvz The problem is better, but not entirely fixed. It works but in significantly slower than previous versions. For now, I have downgraded Typescript used by VS.

Closing as duplicate of #71863 for original poster
@arilenz @polishedwp Please open new issues


add support for groups

Add a group table, and allow users to view issues based on a group. Each issue should be assigned to a group, and when user switches groups they should see the issues for that group.

Duplicate of #24


[Custom-rule] Add fixer to no-if-in-oneline

Overview

Add fixer function to no-if-in-oneline.

See: #28

Tasks

[ ] Implement fixer of no-if-in-oneline

English

I found that when I added the standard rules curly and brace-style, I could not write code such that only the no-if-in-oneline rule applied. Thus, we are considering to obsolete the no-if-in-oneline rule.

If no-if-in-oneline is obsoleted in the future, this issue will no longer be necessary.

日本語

標準ルールの curly と brace-style を追加すると、no-if-in-oneline ルールのみが適用される様なコードが書けない事がわかりました。なので、no-if-in-oneline ルールは廃止を検討しています。

もしも将来 no-if-in-oneline が廃止される場合は、此の issue も不要になります。

Example

Add below code.

          if (nextTokenStartPosition) { return '' }

Log

  160:39  error  Statement inside of curly braces should be on next line                                                        brace-style
  160:41  error  Forbid if statements to be written on a single line                                                            openreachtech/no-if-in-oneline
  160:41  error  This line has 2 statements. Maximum allowed is 1                                                               max-statements-per-line
  160:51  error  Closing curly brace should be on the same line as opening curly brace or on the line after the previous block  brace-style


Updated readme file

[x] Added live demo link to deployment section in the readme file.

@ShadyShawkat Thanks for the review. 😄


Previous input values should not be deleted for new test

If you do some changes on the DMN and start the test, then you have to enter the test input again.
The fields are also empty, if you click on the back button.
This can be taken from the previous test.
Maybe the older values can be listed in an combo box, so that it is easy to verify a previous test?

hi @berndvarga
thanks for the suggestions. I understand it and agree that this would be a nice thing to change/add. Moving it to the backlog.


Kernel errors experimental mode

Works well for me with iJulia, don't test well with other languages.
But maybe too verbose like with Julia0.5.
Give a try, please

If this is too verbose, we could throttle it and make any errors beyond the first in a window of time just say "More error messages received. Check the Developer Console for details." At least for now; at some point it would be cool to have a little panel where you can view all the messages from the kernel.

@willwhitney fixed.
It isn't verbose if kernel is not. Well Julia is a chatterbox. In python or coffee I get only kernel errors.

We should deal gracefully with noisy kernels like Julia as well; I think the Notebook does something pretty concise in that case, and it'll be really annoying if lots of messages pop down in front of our code.

Let's try. I as Julia user didn't see trouble in all this warning bloating. If someone find this annoying he can create a ticket and we'll discuss it again.
I thought error handling is kernel internal, so it should be managed there.

OK, sounds good. Merge away!
On Fri, Nov 20, 2015, 5:36 PM Slava<EMAIL_ADDRESS>wrote:

Let's try. I as Julia user didn't see trouble in all this warning
bloating. If someone find this annoying he can create a ticket and we'll
discuss it again.
I thought error handling is kernel internal, so it should be managed there.
—
Reply to this email directly or view it on GitHub
https://github.com/willwhitney/hydrogen/pull/116#issuecomment-158545208.

Just leave this here: https://github.com/JuliaLang/julia/issues/4895


SAM validation error

Hi, I'm trying to use nft-bam on some files where the read group does not exist in the header.
Would it be possible for nft-bam to ignore this error?
 htsjdk.samtools.SAMFormatException: SAM validation error: ERROR::READ_GROUP_NOT_FOUND:Record 1, Read name m84039_230117_233243_s1/163973658/ccs, RG ID on SAMRecord not found in header: 92af947f/0--0

Hi @fellen31, can you give me a file I can use to test this out? I'll see if I can somehow do this using HTSJDK :)

Sure, thanks!
https://www.dropbox.com/scl/fi/w9yinzj4uhbba65xdtz3f/HG002_Revio_A.bam?rlkey=2fgjfawm7h9swf5mgan7wjk01&st=4pw0ose7&dl=0

Thanks for the file! However I don't get the error you posted when trying this out myself. Do you have an example of the code you used?

Hm, I'm trying to snapshot the reads, or md5s.
https://github.com/genomic-medicine-sweden/nallo/blob/f5dd4ca8559b8d47c98cf2818b0e8cf12e8af402/tests/main.nf.test#L163-L174
The error I get is:
      -[genomic-medicine-sweden/nallo] Pipeline completed successfully
      Stop plugin<EMAIL_ADDRESS>
    htsjdk.samtools.SAMFormatException: SAM validation error: ERROR::READ_GROUP_NOT_FOUND:Record 1, Read name m84039_230117_233243_s1/163973658/ccs, RG ID on SAMRecord not found in header: 92af947f/0--0
FAILED (262.751s)

  Assertion failed:

  1 of 100 assertions failed

Assertion failed:

1 of 100 assertions failed

        at com.askimed.nf.test.lang.extensions.GlobalMethods.assertAll(GlobalMethods.java:48)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:568)
        at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:107)
        at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:323)
        at org.codehaus.groovy.runtime.callsite.StaticMetaMethodSite.invoke(StaticMetaMethodSite.java:44)
        at org.codehaus.groovy.runtime.callsite.StaticMetaMethodSite.callStatic(StaticMetaMethodSite.java:100)
        at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCallStatic(CallSiteArray.java:55)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callStatic(AbstractCallSite.java:217)
        at main_nf$_run_closure1$_closure3$_closure62.doCall(main.nf.test:164)
        at main_nf$_run_closure1$_closure3$_closure62.doCall(main.nf.test)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:568)
        at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:107)
        at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:323)
        at org.codehaus.groovy.runtime.metaclass.ClosureMetaClass.invokeMethod(ClosureMetaClass.java:274)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1030)
        at groovy.lang.Closure.call(Closure.java:427)
        at groovy.lang.Closure.call(Closure.java:406)
        at com.askimed.nf.test.lang.TestCode.execute(TestCode.java:16)
        at com.askimed.nf.test.lang.pipeline.PipelineTest.execute(PipelineTest.java:135)
        at com.askimed.nf.test.core.TestExecutionEngine.execute(TestExecutionEngine.java:214)
        at com.askimed.nf.test.commands.RunTestsCommand.execute(RunTestsCommand.java:184)
        at com.askimed.nf.test.commands.AbstractCommand.call(AbstractCommand.java:43)
        at com.askimed.nf.test.commands.AbstractCommand.call(AbstractCommand.java:18)
        at picocli.CommandLine.executeUserObject(CommandLine.java:1953)
        at picocli.CommandLine.access$1300(CommandLine.java:145)
        at picocli.CommandLine$RunLast.executeUserObjectOfLastSubcommandWithSameParent(CommandLine.java:2352)
        at picocli.CommandLine$RunLast.handle(CommandLine.java:2346)
        at picocli.CommandLine$RunLast.handle(CommandLine.java:2311)
        at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:2179)
        at picocli.CommandLine.execute(CommandLine.java:2078)
        at com.askimed.nf.test.App.run(App.java:44)
        at com.askimed.nf.test.App.main(App.java:51)
  Nextflow stdout:

  Nextflow stderr:

FAILURE: Executed 1 tests in 262.759s (1 failed)

Aha I tried it with .getHeader(). That seems to work, but .getReads() fails. I'll see what I can do about it

Although it does look like you are missing a readgroup ID in your header that's present in the reads. I'm not sure this is intended?

Yes, there is a readgroup present in the reads that's not in the header. I actually didn't know it was there before I tried no snapshot the reads. No downstream tools (including samtools) complains though, and the SAM format doesn't strictly require a RG line in the header.

Allright I'm currently testing out a way to set the stringency of the HTSJDK validation which should allow you to ignore these errors. I'll let you know once that works :)

Hi can you try it out with this new (unreleased) version?
nft-bam-0.2.1.jar.gz
Just add this option to the bam() function: bam(..., stringency: 'lenient') (You can also set it to 'silent' if you also don't want the warnings
Then run the new plugin using this command:
nf-test test --plugins nft-bam-0.2.1.jar <your other options>

@fellen31 any news on this? This new feature is ready to release once you can confirm that it works on your side :)

I’ll have a look today!

Looks good!

Awesome! I'll release this then :)

Thank you!

release v0.3.0 contains these changes


module 'tensorflow_core._api.v2.train' has no attribute 'Optimizer'

Traceback (most recent call last):
File "run_classifier.py", line 25, in
import optimization
File "D:\bertpro\bert\optimization.py", line 87, in
class AdamWeightDecayOptimizer(tf.train.Optimizer):
AttributeError: module 'tensorflow_core._api.v2.train' has no attribute 'Optimizer'

I met the same problem with tensorflow 1.15.0.

same issue here!

Same issue.
My tensorflow-gpu version is 2.0.0

same issue
My tensorflow version is 1.11.0

It became a warning instead with the 1.14 version.

Same issue with Tensorflow cpu 2.0.0 on mac !

Same issue.
My tensorflow cpu version is 2.0.0 .

change tensorflow version to 1.14

change
class AdamWeightDecayOptimizer(tf.train.Optimizer)
to
class AdamWeightDecayOptimizer(tf.compat.v1.train.Optimizer)

This is going to become more prevalent as more users switch to TensorFlow 2.  I would suggest that the developers start a new branch for TensorFlow 2 development.  As more people switch, that branch can eventually become master.  Might want to tag a final release based on TensorFlow 1 or something before cutting over.

I am experiencing this as I am trying to execute a model in a new environment using Tensorflow 2.0, and GPU.
Probably need to figure out other migration and upgrading approach!.

Right, this code needs TF 1.15. I was responding to girottoma's comment above. In https://github.com/google-research/bert/blob/master/README.md, there is a section "Fine-tuning with Cloud TPUs" that references the Cloud TPU BERT tutorial (there are now 2 of them) and a BERT colab at https://cloud.google.com/tpu/docs/tutorials/. The BERT 2.x tutorial uses TF 2.1 and the BERT 1.x tutorial uses TF 1.1.5 (this tutorial was broken, but will be fixed and republished next week).

Haha!  My fault.  I knew I was missing something.  🤤
Thanks for clarification.

There was a problem with the TF 1.15 BERT tutorial (https://cloud.google.com/tpu/docs/tutorials/bert) that was causing it to fail. It has been fixed and the BERT 1.x tutorial should run correctly now.

Hey guys,i think i got the same issue cause i'm using tensorflow-gpu 1.15.0 and tensorflow 1.15.0 but when i train into the dataset of SQUAD i got
INFO:tensorflow:impossible example
Anyone has a clue ?

same issue using google colab
any solution ????

I fixed the issue in google colab by installing tensorflow 1.15 instead of 2. I get a warning only.
!pip install tensorflow-gpu==1.15.0

same issue using google colab
any solution ????

Hi, Did you get a reply on how to solve this issue?

I am also facing the same issue --
AttributeError Traceback (most recent call last)
in
1 import bert
----> 2 from bert import run_classifier
3 from bert import optimization
4 from bert import tokenization
5 from tqdm import tqdm
~\bert\run_classifier.py in
23 import os
24 from bert import modeling
---> 25 from bert import optimization
26 from bert import tokenization
27 import tensorflow as tf
~\bert\optimization.py in
85
86
---> 87 class AdamWeightDecayOptimizer(tf.train.Optimizer):
88 """A basic Adam optimizer that includes "correct" L2 weight decay."""
89
AttributeError: module 'tensorflow._api.v2.train' has no attribute 'Optimizer'

This can be solved by changing tf.train.Optimizer to tf.compat.v1.train.AdamOptimizer()

just in case this helps someone: one thing that has helped me solve this issue is to ensure my python==2.x (note 3.x), tensorflow==1.15.0

I am also facing the same issue --
AttributeError Traceback (most recent call last)
in
1 import bert
----> 2 from bert import run_classifier
3 from bert import optimization
4 from bert import tokenization
5 from tqdm import tqdm
~\bert\run_classifier.py in
23 import os
24 from bert import modeling
---> 25 from bert import optimization
26 from bert import tokenization
27 import tensorflow as tf
~\bert\optimization.py in
85
86
---> 87 class AdamWeightDecayOptimizer(tf.train.Optimizer):
88 """A basic Adam optimizer that includes "correct" L2 weight decay."""
89
AttributeError: module 'tensorflow._api.v2.train' has no attribute 'Optimizer'

This can be solved by changing tf.train.Optimizer to tf.compat.v1.train.AdamOptimizer()

Got the error below after making the above changes. Any idea how to fix it? Thanks!
Traceback (most recent call last):
File "run_classifier.py", line 29, in
flags = tf.flags
AttributeError: module 'tensorflow' has no attribute 'flags'

change
class AdamWeightDecayOptimizer(tf.train.Optimizer)
to
class AdamWeightDecayOptimizer(tf.compat.v1.train.Optimizer)

this could work！！

I am also facing the same issue --
AttributeError Traceback (most recent call last)
in
1 import bert
----> 2 from bert import run_classifier
3 from bert import optimization
4 from bert import tokenization
5 from tqdm import tqdm
~\bert\run_classifier.py in
23 import os
24 from bert import modeling
---> 25 from bert import optimization
26 from bert import tokenization
27 import tensorflow as tf
~\bert\optimization.py in
85
86
---> 87 class AdamWeightDecayOptimizer(tf.train.Optimizer):
88 """A basic Adam optimizer that includes "correct" L2 weight decay."""
89
AttributeError: module 'tensorflow._api.v2.train' has no attribute 'Optimizer'

This can be solved by changing tf.train.Optimizer to tf.compat.v1.train.AdamOptimizer()

Got the error below after making the above changes.
Traceback (most recent call last):
File "run_classifier.py", line 29, in
flags = tf.flags
AttributeError: module 'tensorflow' has no attribute 'flags'
Triggered a bunch of other function errors after changing flags = tf.flags to tf.compat.v1.flags. So went ahead made the following global change, which is probably equivalent to setting tf version to 1.15.0.
tf = tensorflow.compat.v1
Then the following error occurred. (The same error happens when setting tf version to 1.15.)
AttributeError: module 'bert.tokenization' has no attribute 'validate_case_matches_checkpoint'
Seems bert.tokenization does have this function defined though. Any thoughts how to fix?

I have same problem

class AdamWeightDecayOptimizer(tf.compat.v1.train.Optimizer):

Just change the code from
optimizer=tf.train.Adamoptimizer()
to
optimizer='adam'

I've tried jumping through the hoops of trying to make things work with python3, but was ultimately unsuccessful.
So I opted to just use python2 (2.7.18) and updating the requirements.txt file to be
tensorflow == 1.15.0   # CPU Version of TensorFlow.
# tensorflow-gpu  >= 1.11.0  # GPU version of TensorFlow.

Despite seeing some warnings, I currently have the classifier running.

change class AdamWeightDecayOptimizer(tf.train.Optimizer) to class AdamWeightDecayOptimizer(tf.compat.v1.train.Optimizer)

problem solved! thank you


Support Union requirements, satiesfied by unique match among the options

Fixes #75.

How will it look like in the graph or html wrapper...?

Should it be also allowed to have Union as a return type...?
For example, if here is provider_a that returns Union[int, float]
and provider_b that needs Union[int, float, str] for an argument a,
it'll not find the provider_a as a provider of a,
But if there is provider_c that returns Union[int, float, str] it should be able to use provider_c as a provider of a.
Maybe I'm overthinking about it...

How will it look like in the graph or html wrapper...?

It will show the match.

Should it be also allowed to have Union as a return type...?
For example, if here is provider_a that returns Union[int, float] and provider_b that needs Union[int, float, str] for an argument a, it'll not find the provider_a as a provider of a, But if there is provider_c that returns Union[int, float, str] it should be able to use provider_c as a provider of a.
Maybe I'm overthinking about it...

I feel this may be a bad idea. I think we should not support it for now.

I think we also need a short description of how Union annotation works in the documentation.


Title problem with Select2, Multi-select, empty values

Hi,
I'm having this problem
when using Select2, Multi-select with empty values
data: [' '],
    append_data_to_table_data: 'before',
    filter_match_mode: 'exact',

I'm looking for a way to have a title display for the empty selection
because there is no name for empty selection and this is not visible to the user that he can select empty.
Once selected here is the result if empty is selected, there is no title

×

See the example and try to select empty in First column
http://jsfiddle.net/lenamtl/epagqe6s/7/
So it's need to be fixed at 2 places in the select dropdown
and into the selected tag.
Any solution is welcome, if there is something for this already please let me know.
Thanks
Update: I did some test with label almost ok
I have tried this
data: [ {value:' ', label:'Others' }],
https://jsfiddle.net/lenamtl/p1t8zu9z/7/
The only problem is that the tag for Others is not created /does not appear when selected.
But it's appear into the dropdown list.
When doing the same thing on my system the label does not appear into the dropdown list but it does in tag list.... I'm confused

Hi,
Since there seems to be some sort of issue with yadcf/select2 and empty string for filtering I can suggest the following solutions:

Use regex (see this jsfiddle) -

    data: [ {value:'^$', label:'Others' }],     filter_match_mode: 'regex',

Use Chosen plugin instead select2 (see this jsfiddle)

Use datatables HTML5 data-* attributes ,  (see this jsfiddle)


Spark version 1.2.1, error AttributeError: <class 'rdd.BlockRDD'> object has no attribute treeReduce

countvectorizer = SparkCountVectorizer(tokenizer=tokenize_pre_process)
count_vector
<class 'rdd.ArrayRDD'> from PythonRDD[22] at collect at rdd.py:168
sel_vt = SparkVarianceThreshold()
red_vt_vector = sel_vt.fit_transform(count_vector)
Traceback (most recent call last):
File "", line 1, in
File "base.py", line 63, in fit_transform
return self.fit(Z, **fit_params).transform(Z)
File "feature_selection.py", line 72, in fit
_, , self.variances = X.map(mapper).treeReduce(reducer)
File "rdd.py", line 179, in getattr
self.class, attr))
AttributeError: <class 'rdd.BlockRDD'> object has no attribute treeReduce

I am using spark 1.2.1, and I think rdd has the method treeReduce.
Would you have any idea why this error could be popping out of the ArrayRDD extendig BlockRDD

I've successfully reproduced the issue and pyspark.RDD doesn't have the method treeReduce, see https://github.com/apache/spark/blob/branch-1.2/python/pyspark/rdd.py
If You can't update spark, I suggest to change treeReduce to reduce.
Right now, the spark version support is an ongoing discussion. We'll update the reqiurements in the README.

Thanks for the solution. We can't update spark yet so will change the method to reduce on rdd


Same number in comment and test_organization

Some readability improvement as discussed here https://github.com/opengisch/qfieldcloud/pull/705#discussion_r1250774648

Yes good point :D  - I kinda wanted to stick to the numbers. But will just drop them. thanks

@why-not-try-calmer hope better maintainable now.


refine script fails with weird message if maven download fails

Apparently the current Maven version is 3.8.6 and when it fails to download 3.8.5 it gives the error:
tar: Error opening archive: Unrecognized archive format
Error: Error while expanding apache-maven-3.8.5-bin.tar.gz

when it feeds the 404 HTML page to tar
To Reproduce

Uninstall mvn
Run refine script

Current Results
$ ./refine test
Using refine.ini for configuration
-------------------------------------------------------------------------------------------------
You have 1352M of free memory.
Your current configuration is set to use 1024M of memory.
OpenRefine can run better when given more memory. Read our FAQ on how to allocate more memory here:
https://docs.openrefine.org/manual/installing#increasing-memory-allocation
-------------------------------------------------------------------------------------------------
tar: Error opening archive: Unrecognized archive format
Error: Error while expanding apache-maven-3.8.5-bin.tar.gz

Expected Behavior
Ideally, it figures out the current version. At a minimum, say that Maven is a requirement and tell the user to install it.
Versions

Operating System: Mac OS
OpenRefine: HEAD

I only ran into this because I accidentally deleted my system install of Maven. I'm happy to just have it declared as a dependency.

Sounds good to me! So then this issue is about cleaning up the refine / refine.bat scripts by removing support for this, and a separate PR should be opened to update the docs.

See another error report here: https://github.com/OpenRefine/OpenRefine/issues/4865


Fix issue 17858

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
https://github.com/yiisoft/yii2/pull/17858/files

Thks @samdark


Remove Code Coverage tab from screenshot

The screenshot may induce users to think that the code coverage results
are always browsable in a separate tab once they are uploaded, but this
is not the case: this feature is not available.

The feature is not available to anonymous users. It can be seen by users that are authenticated.
Additionally, the documentation does not talk about the code coverage tab at all. It only talks about the artifacts from the build summary page, which should be available for ALL users.

Thanks for the clarification! Would it be ok if I updated the documentation
with your explanation? I still find it a bit confusing that the screenshot
shows the tab but the documentation does not mention it. Thanks!

@PBoraMSFT I assigned this to you, please merge when it is ready, or let me know when it is ready and I can merge it.


WIP pa11ycrawler updates for AC-709

Purpose of this PR:
(AC-709). Allow pa11ycrawler to crawl a single url, and dump the response through the pipeline.

Update:
Ben, regarding your feedback I have made the following changes:

Added documentation inside the README to provide some information about this feature.
Included a new handle_error method that will provide a little more information when a single_url request goes wrong (such as a bad url).

Outside of the suggested feedback, I also cleaned up a new issue with fetching credentials. Previously, credentials were obtained via auto_auth any time there was a single_url argument passed into the spider. However there were a few problems with that.

If credentials are passed into the spider, there is no need to go through auto_auth
If the single_url did not use the default port 8000, there was be an error unless they passed the valid port number into the spider
If a url was outside of localhost (say, google.com) there obviously is no auto_auth support

The solution to this can be reviewed inside of the start_requests method.

@michaelyoungstrom before merging, you'll want to squash your commits and rebase. When merging, you'll also want to re-title the request so that "WIP" isn't going to appear in the commit logs. (There will be a merge commit when you merge.) Let us know if you want a hand with any of these; it could be 2 minutes of showing you how to do it vs grok-time on your own ;)


Fix clippy warning in tutorial 3

Writing &String instead of &str involves a new object where a  slice will do
See https://rust-lang.github.io/rust-clippy/master/#ptr_arg

Thanks! I thought I remember this not compiling (when giving it to a parameter expecting a Fn(&T) with T = String) but it seems to work! :tada:


Fix TypeVar instance bug

Add missing symmetric case in de typechecker unification.
Enforce correct instancing behaviour in TypeVar.
And some debugging improvements

Thanks!


GH1815: Log inner exceptions message

fixes #1815

It would seem roslyn now returns a AggregateException, with diagnostic verbosity you got the full stack strace, but with any verbosity you just got the message of the AggregateException, now it'll print the messages of the inner exceptions to if present.

OK will revert the 7.1 needs newer sdk.

So that dit it CI much happier, @cake-build/cake-team this one ready for review.

@devlead your changes have been merged. Thank you for your contribution.


Chalk.keyword is not function in version 5.1.2, how ?

All of this, and more, is available by checking the releases page.
Also, please next time add a bit of a description as to what you're after.


Feature Request: Add chat log option for new interact options [REPLACEMENT ISSUE]

The original issue
Id: 2082
Title: Feature Request: Add chat log option for new interact options

could not be created.
This is a dummy issue, replacing the original one.In case the gitlab repository still exists, visit the following link to see the original issue:
https://gitlab.com/hooking/foundry-vtt---pathfinder-2e/-/issues/2082

No idea what this issue is even referring to anymore.


Dataset storage

Resolves #837.

Store active dataset and pending dataset in non-volatile memory.

Retrieve active dataset and pending dataset from non-volatile memory after each reboot.

Leader will generate the active dataset if it is not initialized.

Some more notes:

Local Dataset will not be stored after each manual configuration, it will only be stored after coming into effect (For example: the device forms a network and becomes Leader);

All Network Dataset will be stored;

Dataset will be deleted after Clear() operation, pending dataset will be cleared after it replaces active dataset.

Current coverage is 59.17% (diff: 100%)

Merging #866 into master will decrease coverage by 14.59%

@@             master       #866   diff @@
==========================================
  Files           112         14      -98
  Lines         16407       2844   -13563
  Methods        2255        342    -1913
  Messages          0          0
  Branches       1981        351    -1630
==========================================
- Hits          12103       1683   -10420
+ Misses         3643       1028    -2615
+ Partials        661        133     -528

Powered by Codecov. Last update 275d22d...d7f1e01

There are some test cases failed during travis check, and they all could pass in second run. But I don't see any changes in this PR could cause the random failures...

Thanks for this PR!
On Linux Ubuntu however i have one "issue".
Since you have added otPlatFlashInit() function, it takes approximately 16 seconds to execute it on my machine, which then prevent ot-cli to work for this time.

@LuDuda, It shouldn't take that long time to run otPlatFlashInit() function, since it only create and initialize a pseudo flash file without any time block. It works normally on my machine and Github server for travis check. Is it possible that there are some file write/create access issue on your machine?

@chshu Okay i will check if there is any way to fix it locally. I verified with my colleague and he has to wait 22 seconds. We are using virtual machines but with quite powerful computers.

@chshu Indeed the problem we had was that we used a shared folder between linux and windows which causes enormous delay on file access. Thanks!


Drop eventhandler 'e.map is not a function' on Safari

The event handler for 'drop' doesn't work on this component. I looked around and it seems its Safari specific and related to dropzone itself.
https://github.com/enyo/dropzone/issues/1640
Its not clear whether the fix was merged in because the issue is still open. Any work around for Safari drops?

I have the same Issue on IE11

this was fixed in Dropzone 5.3, the project currently uses 5.2.
https://gitlab.com/meno/dropzone/blob/v5.3.0/CHANGELOG.md

@felixrieseberg can we see the latest drop zone being used? Thanks!

Temporary fix:
yarn add JSafaiyeh/React-Dropzone-Component
npm install --save JSafaiyeh/React-Dropzone-Component


Adds ACS port mapping instructions to address issue #235

This updates the readme with additional instructions for mapping the Azure App Service's port 80 to the containers port 8085. While testing this I noticed that it didn't appear to be necessary to specify a startup file.
If you want, I can also add the instructions for setting up the app using the Azure CLI:
Create the resource group
az group create --name <resource group name> --location "West US"
Create the app service plan
az appservice plan create --name <app service plan name> --resource-group <resource group name> --sku S1 --is-linux
Create the app service itself
az webapp create --resource-group <resource group name> --plan <app service plan name> --name <app service name> --deployment-container-image-name <container image name>
Map the app service's port 80 to the containers exposed port 8085
az webapp config appsettings set --resource-group <resource group name> --name <app service name> --settings WEBSITES_PORT=8085

Hugs


Simplify docs infra for Qiskit, Provider, and Runtime repos

Now that docs are hosted exclusively at ibm.com, there is no need to have all their docs pages. We only need the API docs. For runtime, we also need to keep the tutorials folder.
Note that we still need projects to deploy their docs to CI so that we can consume them via https://github.com/Qiskit/documentation/issues/311. We also need to continue deploying their objects.inv file until https://github.com/Qiskit/documentation/issues/236 is figured out.

Done in https://github.com/Qiskit/qiskit-ibm-provider/pull/773, https://github.com/Qiskit/qiskit-ibm-runtime/pull/1256, and https://github.com/Qiskit/qiskit/pull/11352.


Why NeRF-W does not support datasets in LLFF form?

Thanks!

I have the same problem. Have you solved it?
This question is very important to me.
Thank you very much! @lif314

I have the same problem. Have you solved it? This question is very important to me. Thank you very much! @lif314

I no longer pay attention to this issue, and I have not implemented this dataloader.


[Bug]: onConnectivityChanged returning old state not the state that it was changed to.

Platform
iPhone 12
Plugin
connectivity_plus
Version
2.3.6 and 2.3.5
Flutter SDK
2.10.4
Steps to reproduce
I have wifi on my emulator turned on; when the connectivity is initialized it turnings a ConnectivityResult.none, and when I turn off wifi the listener returns a ConnectivityResult.wifi. This cycle persists regardless of the changes or order. So listening to the change is only returning the previous value not the new connectivity state.
Code Sample

import 'dart:async';
import 'dart:developer';

import 'package:connectivity_plus/connectivity_plus.dart';
import 'package:flutter/services.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';

class ConnectivityNotifier extends StateNotifier<ConnectivityResult> {
  ConnectivityNotifier() : super(ConnectivityResult.none) {
    initConnectivity();
    _connectivitySubscription =
        _connectivity.onConnectivityChanged.listen((data) {
      _updateConnectionStatus(data);
    });
  }

  final Connectivity _connectivity = Connectivity();
  late StreamSubscription<ConnectivityResult> _connectivitySubscription;

  Future<void> initConnectivity() async {
    late ConnectivityResult result;
    print('ConnectivityNotifier initConnectivity');
    try {
      result = await _connectivity.checkConnectivity();
    } on PlatformException catch (e) {
      log('ConnectivityNotifier Couldn\'t check connectivity status', error: e);
      return;
    }

    // If the widget was removed from the tree while the asynchronous platform
    // message was in flight, we want to discard the reply rather than calling
    // setState to update our non-existent appearance.
    if (!mounted) {
      return Future.value(null);
    }

    return _updateConnectionStatus(result);
  }

  Future<void> _updateConnectionStatus(ConnectivityResult result) async {
    print('ConnectivityNotifier._updateConnectionStatus: $result');
    state = result;
  }

  @override
  void dispose() {
    print('ConnectivityNotifier.dispose');
    _connectivitySubscription.cancel();
    super.dispose();
  }

  bool isConnected() {
    return state != ConnectivityResult.none;
  }
}

Logs
[   +1 ms] flutter: State variable after : ConnectivityResult.wifi
[+3767048 ms] flutter: State variable after : ConnectivityResult.wifi
[        ] flutter: connectivity from listener ConnectivityResult.wifi
[        ] flutter: ConnectivityNotifier._updateConnectionStatus: ConnectivityResult.wifi
[        ] flutter: State variable after : ConnectivityResult.wifi
[+12397 ms] flutter: State variable after : ConnectivityResult.wifi
[   +3 ms] flutter: connectivity from listener ConnectivityResult.none
[        ] flutter: ConnectivityNotifier._updateConnectionStatus: ConnectivityResult.none
[   +5 ms] flutter: State variable after : ConnectivityResult.none
[   +1 ms] flutter: State variable after : ConnectivityResult.none
[        ] flutter: connectivity from listener ConnectivityResult.none
[   +1 ms] flutter: ConnectivityNotifier._updateConnectionStatus: ConnectivityResult.none
[   +1 ms] flutter: State variable after : ConnectivityResult.none

Flutter Doctor
flutter doctor -v
[✓] Flutter (Channel unknown, 2.10.4, on macOS 12.4 21F79 darwin-arm, locale en-US)
    • Flutter version 2.10.4 at /Users/aaronkennedy/Development/flutter
    • Upstream repository unknown
    • Framework revision c860cba910 (4 months ago), 2022-03-25 00:23:12 -0500
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

[✓] VS Code (version 1.69.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.44.0

[✓] Connected device (2 available)
    • iPhone 12 Pro (mobile) • DEDA0D01-91A3-47EB-A253-6C2F91DE0637 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 103.0.5060.114

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

To clarify I also have this issue with the example app.

This appears to only be an emulator issue

I also have this issue, any suggested solution/workaround for this?


Create a quick-start material for developers explaining the app architecture

Bumping this one

Should this be included in #171?

Need to explain how to check out the app, which IDE's to use, and some kind of QnA, going to update this issue while onboarding new devs


Oauth2 support?

Dear maintainers,
as meetup is deprecating many authentication methods (see this link, i would like to know if anybody plans to work on oauth2 support for this amazing library?
many thanks in advance,
best

You can try this: https://github.com/rdohms/meetup-api-client/pull/58


At exception page links to private methods are generated while these aren't in API docs

For example,
http://www.yiiframework.com/doc-2.0/yii-db-activequery.html#removeDuplicatedModels()-detail

@samdark, this is from 2014. Is it solved already?

No. Here it is: https://github.com/yiisoft/yii2/blob/master/framework/web/ErrorHandler.php#L209


Issue with caching 302 redirects

I am not sure what the solution is here so posting to see if anyone else has any idea. I am using toolbox.networkFirst
I have a secure area
/account/manage
If the user tries to load that page we do a Redirect("/account") in MVC - this causes a 302 Found with the correct redirect.
from /account the user successfully logs in and then we do the same redirect to /account/manage but the original 302 is then being served from Service worker causing a loop back to /account. Surely this should be attempting the Network again and getting a correct 200 response?
If I change the URL in the address bar and load up /account/manage then the page loads correctly
const OWN_DOMAINS = ["localhost:53585"];

// Default cache rule
toolbox.router.get('/(.*)', function (request, values, options) {
   // networkFirst will attempt to return a response from the network,
   // then attempt to return a response from the cache.
   return toolbox.networkFirst(request, values, options).catch(function (error) {
       // If both the network and the cache fail, then `.catch()` will be triggered,
       // and we get a chance to respond with our cached fallback page.
       // This would ideally check event.request.mode === 'navigate', but that isn't supported in
       // Chrome as of M48. See https://fetch.spec.whatwg.org/#concept-request-mode
       if (request.method === 'GET' && request.headers.get('accept').includes('text/html')) {
           return toolbox.cacheOnly(new Request('/Offline'), values, options);
       }
       throw error;
   });
}, {
   cache: {
       maxEntries: 100,
       maxAgeSeconds: 86400
   },
   origin: ORIGIN_RULE
});

Seems I am getting
TypeError: Failed to execute 'fetch' on 'ServiceWorkerGlobalScope': Request with GET/HEAD method cannot have body.

Hi @AaronLayton, thanks for reporting. I'll take a look.
As a workaround, the decision on which status codes should be cached is based on toolbox.options. successResponses, which is by default set to a Regex: /^0|([123]\d\d)|(40[14567])|410$/
302 matches the regex, so those will be cached. Overriding this value will let you tweak what happens.

@wibblymat thanks I tried that also and it didn't seem to work. It was just tripping its self up because of the redirect I think.
Interestingly, I have tried it in Chrome Canary and I don't get the issue.
I am using this at the moment aswell
toolbox.router.get('/account*', toolbox.networkOnly, { origin: ORIGIN_RULE });

I just tried to put together a super simple demo of this.
In Chrome it seems to work as you'd expect. Firefox has some issues. See: https://travis-ci.org/GoogleChrome/sw-toolbox/builds/111731164
Does this match the rough scenario you are performing? It's essentially a request for a url which redirects to a path that returns JSON but sets a cookie. The following request to the first URL then detects the cookie and returns a success piece of json.
https://github.com/GoogleChrome/sw-toolbox/blob/network-first/test/browser-tests/network-first/serviceworkers/redirects.js
The only thing I had to make sure of was that the cookies were being sent with the requests (this is largely due to the test set up - but may be application) - but perhaps the cookies will result in the cache not matching against the url?
Or is there something else that could be different?

@gauntface I have a test server where you can see this happening if you like?
drop me an email -<EMAIL_ADDRESS>It could end up being something I am doing, but without the service worker things seem to be OK

Getting the same error here. It happens when a POST form submission returns 302 redirects. The error was thrown when trying to fetchAndCache the GET request we got redirected to.

Yup that is what I am getting aswell - doesn't matter if I cache the response or use networkFirst - same thing happens. Although I was only getting it on Stable, if I tried this in Canary then it sempt to go through OK

I still get this even if I have told sw-toolbox to do a networkOnly. Is this something wrong with fetch in general not being able to follow a redirect? @gauntface

I think fetch at one point didn't follow redirects, but that was a while ago.
I added the test case for this but it sounds like I'm still not getting the right behavior. Can you give me a super cut down version of how to reproduce this? Just general instructions is fine (although an example snippet would be better ;) )

This is a known issue of Chrome: https://bugs.chromium.org/p/chromium/issues/detail?id=573937
and was already fixed in the beta channel (tested with v50.0.2661.75).

Thanks for the info @ktmud.
@AaronLayton if you get a chance to confirm this is fixed would love to know if your problem is resolved.

I am not seeing this exact issue anymore - happy to close


adding a note about ServiceMonitor definition

In the Adding additional namespaces to monitor section, I appended a note showing the need for ServiceMonitor when adding additional namespaces...
see: https://github.com/coreos/prometheus-operator/issues/2557#issuecomment-498996568

Great improvements!
Next to these changes, we should make it obvious that this is not the only way to add ServiceMonitors. People getting it that way, would miss on the big part of self-service monitoring.

Next to these changes, we should make it obvious that this is not the only way to add ServiceMonitors. People getting it that way, would miss on the big part of self-service monitoring.

Indeed! As suggested by @brancz , I added the following note:

Typically it is up to the users of a namespace to provision the ServiceMonitor resource, but in case you want to generate it with the same tooling as the rest of the cluster monitoring infrastructure, this is a guide on how to achieve this.

@metalmatze, Do you think we should be more explicit around this?

CI failure is not related to this PR and can be omitted. Fix is in https://github.com/coreos/kube-prometheus/pull/117

Could you rebase this? Then it’s good to go from my side.

rebase done!

Thanks!


"timeouts" is missing in maze2d environments

When I download a maze2d dataset with env.get_dataset(), the downloaded hdf5 file contains "timeouts" key:
>>> import d4rl, gym
>>> env = gym.make('maze2d-large-v1')
>>> dataset = env.get_dataset()
Downloading dataset: http://rail.eecs.berkeley.edu/datasets/offline_rl/maze2d/maze2d-large-sparse-v1.hdf5 to /root/.d4rl/datasets/maze2d-large-sparse-v1.hdf5
load datafile: 100% 8/8 [00:01<00:00,  6.41it/s]
>>> dataset.keys()
dict_keys(['actions', 'infos/goal', 'infos/qpos', 'infos/qvel', 'observations', 'rewards', 'terminals', 'timeouts'])

However, when I attempt to generate the same dataset locally, it doesn't contain "timeouts" key:
$ python scripts/generation/generate_maze2d_datasets.py --env_name maze2d-large-v1
$ python scripts/generation/relabel_maze2d_rewards.py --relabel_type sparse --maze large --filename /data/maze2d/maze2d-large-v1.hdf5
# ^ This generates maze2d-large-v1-sparse.hdf5
$ python
>>> import d4rl, gym
>>> env = gym.make('maze2d-large-v1')
>>> dataset = env.get_dataset('/data/maze2d/maze2d-large-sparse-v1.hdf5')  # Load from a local file
load datafile: 100% 7/7 [00:00<00:00, 21.77it/s]
>>> dataset.keys()
dict_keys(['actions', 'infos/goal', 'infos/qpos', 'infos/qvel', 'observations', 'rewards', 'terminals'])  # <-- This is missing "timeouts"!

I believe I followed the wiki page to generate the dataset.
Am I missing any steps?

When I download a maze2d dataset with env.get_dataset(), the downloaded hdf5 file contains "timeouts" key:
>>> import d4rl, gym
>>> env = gym.make('maze2d-large-v1')
>>> dataset = env.get_dataset()
Downloading dataset: http://rail.eecs.berkeley.edu/datasets/offline_rl/maze2d/maze2d-large-sparse-v1.hdf5 to /root/.d4rl/datasets/maze2d-large-sparse-v1.hdf5
load datafile: 100% 8/8 [00:01<00:00,  6.41it/s]
>>> dataset.keys()
dict_keys(['actions', 'infos/goal', 'infos/qpos', 'infos/qvel', 'observations', 'rewards', 'terminals', 'timeouts'])

However, when I attempt to generate the same dataset locally, it doesn't contain "timeouts" key:
$ python scripts/generation/generate_maze2d_datasets.py --env_name maze2d-large-v1
$ python scripts/generation/relabel_maze2d_rewards.py --relabel_type sparse --maze large --filename /data/maze2d/maze2d-large-v1.hdf5
# ^ This generates maze2d-large-v1-sparse.hdf5
$ python
>>> import d4rl, gym
>>> env = gym.make('maze2d-large-v1')
>>> dataset = env.get_dataset('/data/maze2d/maze2d-large-sparse-v1.hdf5')  # Load from a local file
load datafile: 100% 7/7 [00:00<00:00, 21.77it/s]
>>> dataset.keys()
dict_keys(['actions', 'infos/goal', 'infos/qpos', 'infos/qvel', 'observations', 'rewards', 'terminals'])  # <-- This is missing "timeouts"!

I believe I followed the wiki page to generate the dataset. Am I missing any steps?

Excuse me. I meet the same question. Have you solved this problem? Thank you.


[core-amqp] Handle case of ErrorEvent being passed as Error to rhea

Rhea expects an Error object
https://github.com/amqp/rhea/blob/2.0.8/lib/connection.js#L591 but some times it
gets an ErrorEvent instead from certain packages, ws for example
https://github.com/websockets/ws/blob/8.16.0/lib/event-target.js#L224
while rhea may be changed to unwrap the error from an ErrorEvent instance, this PR handles the ErrorEvent case in our code base since we have more control here.
Packages impacted by this PR
@azure/core-amqp
Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/issues/29294

API change check
API changes are not detected in this pull request.


Complex number support for pinv

Currently, cupy.linalg.pinv does not seem to work with complex numbers.
This code
u, s, vt = decomposition.svd(a, full_matrices=False)
cutoff = rcond * s.max()     # error
s1 = 1 / s
s1[s <= cutoff] = 0    # error
return core.dot(vt.T, s1[:, None] * u.T)

does not work since max() is not supported on complex arrays (see #794) and the same holds for comparison operators.

Looking forward to having this implemented. @themightyoarfish, I'll likely start testing your patch tomorrow. Please let me know if there is any other ways I can help.

I suppose this is closed by #1636.

This may have been resolved when #2581 was resolved.

@kmaehashi This can be closed.


Zoom not working in ionic 3

Hi everybody,
I'm using ionic 3, I want to slides images can zoom, but It's not working
Below is my code
<ion-slides zoom="true" initialSlide="0" class="center" pager="true" paginationType="bullets">
      <ion-slide *ngFor="let img of listImgs">

      </ion-slide>
</ion-slides>

I also try add class "swiper-zoom-container", but it only zoom center and will reset orginal when my finger release out mobile screen
<ion-slides zoom="true" initialSlide="0" class="center" pager="true" paginationType="bullets">
      <ion-slide *ngFor="let img of listImgs">

      </ion-slide>
</ion-slides>

Anybody can help me. I think it very simple but It's amazing that this function is not working.
Thanks,
Thanh Nguyen

Please fill the issue template

Hello all! As it seems it has been a while since there was any activity on this issue i will be closing it for now. Feel free to comment if you are still running into this issue. Thanks for using Ionic!

@jgw96 , @mhartington
I trace this issue from 2015 to 2018. Yes, 3 years, how many 3years we can have? It's a very easy and simple bug, what Ionic team did is just merge all user's issues to #5039 ([https://github.com/ionic-team/ionic/issues/5039]) . And finally point to an invalid url link, and CLOSED!! What? Really? YES, what ionic team did is to just CLOSE issues one by one but NOT SOLVE issues.
"Ionic Team's attitude is just close issues, not solve issues?"
If you don't believe , you can look around Ionic github issues , many issues,  they close issues just because no new comments added, "NOT" they solve the bug(s). People are waiting for solutions, they didn't write duplicated comments once and again. But, because the didn't produced trash comments, Ionic team close issues.
Further more, they use Ionic ROBOT to AUTOMATICALLY close issues with no new comments. Be Notice, to CLOSE,  not to SOLVE.
I dont know Ionic team has intension to develop Ionic or not. But from their attitude, I regret to select IONIC. If I have a new chance, I won't select IONIC again.
sorry for the long post.

@pipoa Please try to be respectful when posting issues. Support for this has been in the framework for some time.
<ion-slides zoom="true">
  <ion-slide>

    <ion-label>Woof</ion-label>
  </ion-slide>
  <ion-slide>

    <ion-label>Meow</ion-label>
  </ion-slide>
  <ion-slide>

    <ion-label>Just keep swimming</ion-label>
  </ion-slide>
</ion-slides>

As per the rest of your comments, as we've stated in our blog post, we have a bot to help us manage new issues and issues where there has been no comment from the reporter. This is to help manage the large amount of issues that get reported.


eta reduce suggestion --refactor

Running:
hlint test.hs --refactor

with test.hs containing:
f x = putStrLn x

doesn't refactor the eta reduce suggestion.

Running with --json (which is the stuff the refactoring grabs), I see:
[{"module":"Main","decl":"f","severity":"Warning"
,"hint":"Eta reduce","file":"Sample.hs","startLine":1,"startColumn":1
,"endLine":1,"endColumn":17,"from":"f x = putStrLn x","to":"f = putStrLn","note":[]
,"refactorings":"[]"}]

Which seems like there is no refactoring alternative for it. @mpickering - is that expected here? Just an unimplemented case? Something that can't be implemented?

It used to be implemented but was a bit tricky to get right so I reverted the patch. It could be implemented in theory but might require some changes to apply-refact.

I was mostly checking to see whether I'd stuffed something up.  No burning need at my end and happy to close.

I've tagged it refactoring, so let's leave it - at some point I want to take a pass through the refactoring stuff.


Difference in cobertura report between coverlet 2.4.0 and 2.5.0

Issue description
When I run tests using command:
dotnet test /p:CopyLocalLockFileAssemblies=true /p:Exclude=\"[System.*]*\" /p:CollectCoverage=true /p:CoverletOutputFormat=cobertura

in my sample project https://github.com/rwasala/CodeCoverage I get different output in reports.
For version 2.4.0 it looks like:
<?xml version="1.0" encoding="UTF-8"?>
<coverage line-rate="1" branch-rate="1" version="1.9" timestamp="1547558846" lines-covered="26" lines-valid="26" branches-covered="4" branches-valid="4">
  <sources>
    <source>/var/lib/jenkins/workspace/Build_CodeCoverage_master/SampleLib/</source>
  </sources>
  <packages>
    <package name="SampleLib" line-rate="1" branch-rate="1" complexity="10">
      <classes>
        <class name="SampleLib.Author" filename="Author.cs" line-rate="1" branch-rate="0" complexity="3">
          <methods>
            <method name="get_Id" signature="()" line-rate="1" branch-rate="0">

For version 2.5.0 it looks like:
<?xml version="1.0" encoding="UTF-8"?>
<coverage line-rate="1" branch-rate="1" version="1.9" timestamp="1547558132" lines-covered="26" lines-valid="26" branches-covered="4" branches-valid="4">
  <sources/>
  <packages>
    <package name="SampleLib" line-rate="1" branch-rate="1" complexity="10">
      <classes>
        <class name="SampleLib.Author" filename="/var/lib/jenkins/workspace/Build_CodeCoverage_master/SampleLib/Author.cs" line-rate="1" branch-rate="1" complexity="3">
          <methods>
            <method name="get_Id" signature="()" line-rate="1" branch-rate="1">

Basically the difference is that in version 2.5.0 node source is empty which causes problems with cobertura Jenkins plugin - it is not showing the source code with marked what is covered and what is not.
Is it possible to make report from coverlet.msbuild in version 2.5.0 the same like in 2.4.0?

This happens because I switched to using absolute paths. #356 re-includes the source element

We're using 2.6.0 and it appears there are still issues with this. Jenkins can't render the source and shows the Source code is unavailable.... message. The filename it attempts to render is var/lib/jenkins/workspace/.... (note missing leading slash).
It appears the cobertura Jenkins plugin strips the leading slash character and expects a relative path: https://github.com/jenkinsci/cobertura-plugin/blob/master/src/main/java/hudson/plugins/cobertura/CoberturaCoverageParser.java#L172-L176
A hacky fix (short of refactoring the SourceLink related changes) could be adding the slash character to the <source> element that was added back (i.e. <source>/</source>).

We're seeing this as well. I reverted back to coverlet.msbuild 2.4.0 as a workaround.

@tonerdo I noticed this same issue too with 2.6.0
`[Pipeline] cobertura
[Cobertura] Publishing Cobertura coverage report...
[Cobertura] No coverage results were found using the pattern '/TestResults/coverage.*.xml' relative to '/home/ec2-user/workspace/'.  Did you enter a pattern relative to the correct directory?  Did you generate the XML report(s) for Cobertura?
This is our jenkin pipeline step that is failing right now with 2.6.0:step([$class             : 'CoberturaPublisher',
autoUpdateHealth   : false,
autoUpdateStability: false,
coberturaReportFile: '/TestResults/coverage.*.xml',
failUnhealthy      : false,
failUnstable       : false,
maxNumberOfBuilds  : 0,
onlyStable         : false,
sourceEncoding     : 'ASCII',
zoomCoverageChart  : false])`
Same reporter works fine with 2.3.1 and we have been using it since last year.


updated viscosity (1.5.11)

Merged as e662fb1192f148acb165cbb5a56644f847cbf210.

I'm getting an error installing since this was merged.
Error: Cask 'viscosity' definition is invalid: Bad header line: parse failed

Any ideas on how to trouble shoot or fix?

@jnunemaker
This error is not a bug, it's caused by an update to how homebrew-cask works. Details can be found here.
To fix the issue, please run:
brew update; brew cleanup; brew cask cleanup
brew uninstall --force brew-cask; brew update

If you are still having problems, open an issue with the appropriate template

That worked.
Seems like there should be a more helpful error message when running the command that says something about an update being needed or something. I'm not sure that I would ever think look at the README of cask when I have a compile issue on an unrelated thing.

Also, thanks! Sorry I forgot to say that. Appreciate the help.

Seems like there should be a more helpful error message when running the command that says something about an update being needed or something.

https://github.com/caskroom/homebrew-cask/issues/16344#issuecomment-168836594.

Seems like there should be a more helpful error message when running the command that says something about an update being needed or something. I'm not sure that I would ever think look at the README of cask when I have a compile issue on an unrelated thing.

There is no way for us to go back in time and add a more detailed error message for an older version.

@vitorgalvao beat me to the punch!


Replace gulp with npm scripts

Additionally, removes the version property from the bower.json file as it's no longer required / honored by bower.
Fixes #74

@tp fyi


How can i add the requester IP address to logs?

There is a way to log the requester IP address to mailsac.log?

During an HTTP request to the UI, or during SMTP stuff?

The HTTP request. I just wanted to keep IP logs of people who accessed the UI for auditing in case something goes wrong or bad.

It is a standard express app using morgan for logging.
The middleware in mailsac is here:
https://github.com/ruffrey/mailsac/blob/master/app.js#L112
morgan logger formatting docs:
https://github.com/expressjs/morgan#predefined-formats


Page says ".NET Framework 4 or higher, which is required by the StackEdchange.Redis client." in requirements

But the title is "Quickstart: Use Azure Cache for Redis with a .NET Core app"
So is this really .NET Core on .NET Framework?

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

ID: 11d0f376-ce05-b7a6-8e31-d5a1bc0a0488
Version Independent ID: 28844087-a4e9-e95c-9583-3ea9c7b47a83
Content: Quickstart: Use Azure Cache for Redis with .NET Core apps
Content Source: articles/azure-cache-for-redis/cache-dotnet-core-quickstart.md
Service: cache
GitHub Login: @yegu-ms
Microsoft Alias: yegu

Thanks for the feedback @jmostella. We have a pending update to address this. StackExchange.Redis does target net461, net472 but they are conditional. I did confirm by building the project in WSL.


make dropdown 460px wide

also fixed the search button width:

Changed to only be the search filter UI


Add a new STATE column

Create a new column containing the full state name.
It can be done based on the state code.
For example:
'NY' - 'New York'
'OH' - 'Ohio'
'OK' - 'Oklahoma'
'OR' - 'Oregon'

Feature included in the commit: f98e913bffa65eb299e7ba8c0a0eeecdfc35e6f7


REQUEST: Would you concider creating a Riddle Intergration?

I have to say we love your integration, Thank You.
Would you consider a Riddle Integration?

Riddle ... I don't know riddle, can you elaborate?


Please invite me to the GitHub Community Organization

Name
Tommie
Discord Username (if applicable)
keller
Additional Context
keller

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Purview Lab - cannot create multiple instances

Lab/Demo: 22 - Use Microsoft Purview with Azure Synapse Analytics
Description of issue:
Microsoft Purview cannot be set up multiple times in the same tenant any longer.  When multiple students try to set up Purview in a hosted lab environment it doesn't work.
Therefore, I think this lab should either be flagged as demo only, or removed.

There's already a note to that effect.
Different ALHs may use different approaches to providing subscriptions, and the exercise is also used by self-paced learners using a BYO subscription approach - hence not retiring the lab altogether.


Possible to include screencasting as well?

I'm loving this demo, I'm wondering if theres an opportunity for a more 1 way type connection, where the phone would broadcast the camera, screencast the website, and audio. The viewer would only receive the phone camera, screencast, and audio and broadcast audio.

It should be possible to add screen capture and make that available as another type of video source. However, this has limited use on an iOS device as video capture is paused whenever you switch to another app.

Thats good to know! I think for our immediate use case the user would be confined to the app more or less, so they wouldn't switch to other apps. For context, we are working on a tool to screencast InVision, Marvel, and Framer prototypes to a user research dashboard and right now our app is somewhat limited on all other AV stuff, outside of screencasting a webview. So we were looking at a WebRTC option.

You are welcome to dig deeper in to this, but it's unfortunately not a use-case we will be able to prioritise near term.

I think the GStreamer camera source element (called avfvideosrc) we use on OS X and iOS supports screen capture. I've never tried that functionality though and I don't know what is needed to get it working on iOS. I would expect some kind of statement that one is going to capture the screen so that the user is aware that the application does that.


Fixed plug mappings for delete buffer and line marks commands

I noticed that the  mappings for the delete line and delete buffer marks didn't seem to work. Seems to have just been a missing underscore.

Looks good to me - thanks for the catch!


Change the licence to OpenCensus authors.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #539   +/-   ##
=========================================
  Coverage     90.02%   90.02%
  Complexity      565      565
=========================================
  Files           110      110
  Lines          2355     2355
  Branches        208      208
=========================================
  Hits           2120     2120
  Misses          180      180
  Partials         55       55

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/io/opencensus/trace/ScopedSpanHandle.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...o/opencensus/implcore/tags/SerializationUtils.java
87.27% <ø> (ø)
11 <0> (ø)
:arrow_down:

.../src/main/java/io/opencensus/common/Timestamp.java
100% <ø> (ø)
25 <0> (ø)
:arrow_down:

...n/java/io/opencensus/implcore/internal/VarInt.java
13.13% <ø> (ø)
4 <0> (ø)
:arrow_down:

...ava/io/opencensus/impl/tags/TagsComponentImpl.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

...src/main/java/io/opencensus/stats/ViewManager.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

...i/src/main/java/io/opencensus/trace/BlankSpan.java
100% <ø> (ø)
9 <0> (ø)
:arrow_down:

api/src/main/java/io/opencensus/common/Clock.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

...in/java/io/opencensus/trace/samplers/Samplers.java
83.33% <ø> (ø)
4 <0> (ø)
:arrow_down:

...src/main/java/io/opencensus/stats/Measurement.java
87.5% <ø> (ø)
1 <0> (ø)
:arrow_down:

... and 100 more

Continue to review full report at Codecov.

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

@tags07 Done.


Update onboard-spring-boot-enabler.md

Inform user how to enable debug if something went wrong at application registration.
Issue from api-layer project.
https://github.com/zowe/api-layer/issues/570

Is there a related issue for this PR?
Issue number:
Developer's Certificate of Origin (DCO)

@janan07 Hi Andrew, is this PR good to be merged? Thanks!


Missing option to configure "Strictly enforce SSL" option for Targetservers

Hi,
Among existing switches for apigeecli targetservers create I can not find one for configuring "Strictly enforce SSL". This option is present in the UI.
Can we add this?

fixed and released in v2.2.0-beta.2
Please try it and let us know. To install this version
export APIGEECLI_VERSION=v2.2.0-beta.2
curl -L https://raw.githubusercontent.com/apigee/apigeecli/main/downloadLatest.sh | sh -

Tried. Works as expected. Thanks a lot.

Works as expected, thanks a lot.


Adjust webassembly runtime about array.

This patch adjusts the WebAssembly runtimes regarding the array creation. Thanks for your review.

Hi, this PR helps a lot and solves most of the problems, thanks! However, there still seems to be a minor issue that a RangeError will be raised if the offset argument passed to new SomeArray is not aligned to size of one array element. This may happen when allocating a f64 array after an i32 array with odd size is allocated and prevents some normal program to allocate. Utilizing DataView could probably solve this.

@NKID00 Thanks for your suggestion. What about the new code?

lgtm, merging


wanted: google/fuzztest

Module location
https://github.com/google/fuzztest
Link to bzlmod issue in the module's repository
https://github.com/google/fuzztest/issues/950
Any other context to provide?
No response
Fund our work

[ ] Sponsor our community's open source work by donating a feature bounty

Duplicate of #1377?

Yeah, but some information in each that is missing from the other. Will leave up to you how you want to merge.

/cc fuzztest contributors @fniksic @xinhaoyuan

@fniksic @xinhaoyuan Is this in your radar? Do you have an idea when you could add fuzztest to BCR?


Create Copy Button for request URLs and JSON Highlighting on Try It page.

A new dependency "@microlink/react-json-view"(fork of original lib) @uiw/react-json-view for JSON tree construction.
Theme switching is not implemented yet Implemented now. Light mode uses GitHub Light theme and dark mode uses nord theme in JSON highlighting.

Reverted due to big memory issues.


Add fix_variables (fix multiple variables at once)

Application
For example, given a multiplication circuit CSP, I'd like to do this:
In [142]: bqm.fix_variables({'a0': 1, 'a1': 0, 'a2': 1, 'b0': 1, 'b1': 1, 'b2': 0})

Instead of this:
In [142]: bqm.fix_variable('a0', 1)
In [143]: bqm.fix_variable('a1', 0)
In [144]: bqm.fix_variable('a2', 1)
In [145]: bqm.fix_variable('b0', 1)
In [146]: bqm.fix_variable('b1', 1)
In [147]: bqm.fix_variable('b2', 0)

Proposed Solution
Enable fixing multiple variables in a single command.
Alternatives Considered
Using fix_variables multiple times.
Additional Context
I think there are lots of situation where users will fix multiple variables and would like to manipulate the values using dicts.

Thank you, sir!


Set Title when User Start Editing

ℹ Please fill out this template when filing an issue.
All lines beginning with an ℹ symbol instruct you with
what info we expect.
Please remove this line and all above before submitting.

Before you start, are you using the latest SkyFloatingLabelTextField release?
Issues are for feature requests, and bugs; questions should go to Stack Overflow with
the tag skyfloatinglabeltextfield.

Report
What did you do?
ℹ Please replace this with what you did.
What did you expect to happen?
ℹ Please replace this with what you expected to happen.
What happened instead?
ℹ Please replace this with of what happened instead.
Environment
ℹ Please fill in this section with information about your environment
Xcode Version:
SkyFloatingLabelTextField Version:
Deployment Target:
Base SDK:
Method of Integration: CocoaPods | Carthage | Manual
Project that demonstrates the issue
ℹ Please link to a project we can download that reproduces the issue if applicable.

Hi @kushaseum this looks like a question rather than a bug report/feature request. If it is one of those please fill out the template otherwise ask on Stackoverflow


[resolves #458] Add support for camel-xmlsecurity data format

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/348/
Test PASSed.


Syntax Missing on less ubiquitous languages

Syntax is missing in some languages like Gleam
SInce we are using this code editor that builds on top of this syntax highlighter, some languages are not handled yet.
Exercism uses the same highlighter, however they handle those languages a little differently
This is how they handle gleam.

Update: stuck on this because of some knowledge gaps(Javascript-wise 😅) working on it though. Hoping to have something next week.

Update: stuck on this because of some knowledge gaps(Javascript-wise 😅) working on it though. Hoping to have something next week.

Thanks for the update, @angiemugo! Let me know if you need help with anything else...

Aside from JavaScript(node) or including?

Aside from JavaScript(node) or including? 😅

eehh... Including?.. I guess...🤷🏽♂Two stuck heads are better than one 🤭🤭

Hey @angiemugo ,
Findings Summary:

The Highlightr Swift package we're using isn't maintained anymore. Forking it and adding missing pieces seems like the best approach.
Exercism handles Gleam as a self-hosted language. We should extract the JavaScript file defining Gleam and integrate it into our Highlightr package.
The CSS styles in Highlightr are outdated. Updating them based on the latest highlight.js repository is necessary.
To handle new languages, manually adding their definition JavaScript files directly to the project and utilizing our existing mechanism for language registration is recommended.
We also need to update our highlight.min.js file to mirror the lastest one

Hope this helps!

Hey @angiemugo ,
Findings Summary:

The Highlightr Swift package we're using isn't maintained anymore. Forking it and adding missing pieces seems like the best approach.
Exercism handles Gleam as a self-hosted language. We should extract the JavaScript file defining Gleam and integrate it into our Highlightr package.
The CSS styles in Highlightr are outdated. Updating them based on the latest highlight.js repository is necessary.
To handle new languages, manually adding their definition JavaScript files directly to the project and utilizing our existing mechanism for language registration is recommended.
We also need to update our highlight.min.js file to mirror the lastest one

Hope this helps!

great job @cedricbahirwe.

Missing languages

[ ] Ballerina
[ ] Abap
[ ] Gleam
[ ] Cobol
[ ] Unison
[ ] x-86 assembly
[ ] Zig
[ ] wren
[ ] visual Basic
[ ] V
[ ] Standard Ml
[ ] SQlite
[ ] Red
[ ] Raku
[ ] Racket
[ ] PureScript
[ ] Pharo
[ ] LFE
[ ] jq
[ ] euphoria
[ ] 8th

It seems some languages are not handled by Highlightjs at all. Those are handled using codeMirror. 🤕. I'll explore that path and see where it leads us

Found a highlightAll hack that fixes this.

During my escapades to update Highlightr, I discovered a simplification: there's no need to individually add the minified files to Highlightr.
Instead, I incorporated them into my local Highlight.js, and they were automatically included when I generated a minified file using tools/build.js.
However, a lingering issue persists: when the language provided to Highlight() isn't recognized, it returns undefined rather than nil, as expected in Swift.
Consequently, languages without Highlight.js extensions remain unhandled.
The objective now is to address this by conditionally handling such cases using highlightAuto.
I have squashed all the earlier commits in favor of the new work on the commit c4b3d0e and later

We have a PR here

The PR has been merged! However, I'll continue using the forked version since I added a small feature here. For languages not recognized by Highlight.js, we default to HighlightAuto.

The PR has been merged! However, I'll continue using the forked version since I added a small feature here. For languages not recognized by Highlight.js, we default to HighlightAuto. This issue has been fixed in this PR

Great job on that 🎉 🚀


fix: update AB marker script for a consistent look and change load order

Changes as proposed in https://github.com/ThaUnknown/releases-moe/pull/17#issuecomment-2294980379
Before

After

which matches the way it looks without the Mediainfo Improvements script by placing the tag at the end.
If there are multiple torrent links in one table row (which should only happen because of the Mediainfo Improvements script) I also change the separator from | to / to match.
Open to suggestions on how to improve this further.

Don't use links[links.length - 1] use .at(-1)

I did actually try that lol but .querySelectorAll returns a NodeList (and not an array) which doesn't have the .at() method. Could convert it to an array first but it seemed unnecessary and less performant (though probably negligible in that aspect).

Thank you!


Cargo update

Automated cargo update

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 8.46%. Comparing base (2cac560) to head (1410446).

Additional details and impacted files
@@           Coverage Diff            @@
##            main    #258      +/-   ##
========================================
- Coverage   8.49%   8.46%   -0.03%
========================================
  Files          7       7
  Lines       3602    3602
  Branches     803     803
========================================
- Hits         306     305       -1
- Misses      3183    3184       +1
  Partials     113     113

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Refine first-cut remake implementation

At this point, using the remake branch of vizlab (including this PR) you should be able to make and more or less build the example repo using
library(vizlab)
vizmake()

so this PR fixes #246 .
Things I did in this PR:

add job blocks so you can vizmake('Fetch'), vizmake('Visualize'), etc. to build just one phase at a time
add explicit dependencies of each item on its declared scripts (or functions within scripts). if there are scripts declared for an item, prepSources first combines and/or subsets those into a single item-specific script file, stored in vizlab/remake/scripts. then the viz item depends more specifically on that file. prepSources also removes comments and whitespace, so you can add those without forcing a rebuild of an item. all of this is necessary because of #292, and it's a follow-on to #294 where i took a good stab at fixing that problem but didn't succeed in re-coupling remake targets to their script dependencies.

Coverage decreased (-2.0%) to 66.806% when pulling 4d782c8bfea9555e4841444a6f9f90b149438f91 on aappling-usgs:remakerefine into 80c67c34381b63521b3805a51539c89927374898 on USGS-VIZLAB:remake.

Coverage increased (+6.0%) to 74.775% when pulling 07e49da3dc0f53102bdc027e3c43b68a8cadaac4 on aappling-usgs:remakerefine into 80c67c34381b63521b3805a51539c89927374898 on USGS-VIZLAB:remake.

Coverage increased (+7.3%) to 76.093% when pulling 4d11fca403e9fe02053256951026d2d02022f287 on aappling-usgs:remakerefine into 80c67c34381b63521b3805a51539c89927374898 on USGS-VIZLAB:remake.

very nice :+1: vizmake('Fetch')


Update version info for breaking change

Fixes #21952.

@gewarren The .NET SDK follows a different versioning scheme than the .NET Runtime: https://docs.microsoft.com/en-us/dotnet/core/versions/
This was introduced in .NET SDK 5.0.100, not 5.0.1 (which doesn't exist).

@dsplaisted Thanks, I'll update it. So 5.0.100 is the first version of the SDK that shipped with .NET 5.0, not 5.0.000?

@dsplaisted Thanks, I'll update it. So 5.0.100 is the first version of the SDK that shipped with .NET 5.0, not 5.0.000?

Yes, correct


[Feature] Replace old auth api with new privilege

What type of PR is this：

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Going to replace the use of api in old Auth module in new privilege framework,
this is the second PR, since the situation where we need to handle the repalcement
is complicated, we will split this action into 4-5 PRs.
Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

[FE PR Coverage Check]
:disappointed: fail : 13 / 33 (39.39%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/loadv2/LoadJob.java
0
12
00.00%
[570, 572, 573, 575, 577, 581, 582, 584, 586, 587, 591, 592]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadManager.java
2
5
40.00%
[305, 306, 307]

:large_blue_circle:
com/starrocks/qe/ShowExecutor.java
3
7
42.86%
[1675, 1679, 1680, 1682]

:large_blue_circle:
com/starrocks/sql/ast/SetVar.java
4
5
80.00%
[117]

:large_blue_circle:
com/starrocks/sql/ast/SetUserPropertyVar.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/authentication/AuthenticationManager.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/qe/SetExecutor.java
1
1
100.00%
[]

run starrocks_admit_test


struct 'os.windows.ws2_32.POLL' has no member named 'IN'

Hi, I'm trying to run the my code as Windows 11, but I got the error below:
PS D:\from_ubuntu\zig> zig run server.zig
websocket.zig\src\server.zig:298:20: error: struct 'os.windows.ws2_32.POLL' has no member named 'IN'
  .events = os.POLL.IN,
            ~~~~~~~^~~
C:\zig-windows-x86_64-0.11.0\zig\lib\std\os\windows\ws2_32.zig:844:18: note: struct declared here
pub const POLL = struct {
                 ^~~~~~

On the latest master, I replaced the use of os.poll with setsockopt.
I don't have access to windows, but with this latest change, I was able to cross compile a windows binary.
However, the websocket.Client code is still using os.poll, so you won't be able to use any of that code.

However, the websocket.Client code is still using os.poll

Can it be fixed?
The server part is fine now, I was able to coomunicate with it from JavaScript:
PS D:\ubuntu\zig> zig run server.zig
debug: Ping

But the client part looks need some more fixes, I got the below:
PS D:\ubuntu\zig> zig run ws.zig
websocket.zig\src\client.zig:261:22: error: struct 'os.windows.ws2_32.POLL' has no member named 'IN'
    .events = os.POLL.IN,
              ~~~~~~~^~~
C:\zig-windows-x86_64-0.11.0\zig\lib\std\os\windows\ws2_32.zig:844:18: note: struct declared here
pub const POLL = struct {
                 ^~~~~~
referenced by:
    connect: websocket.zig\src\client.zig:50:23
    connect: websocket.zig\src\websocket.zig:9:27
    remaining reference traces hidden; use '-freference-trace' to see all reference traces
C:\zig-windows-x86_64-0.11.0\zig\lib\std\c.zig:354:12: error: dependency on libc must be explicitly specified in the build command
pub extern "c" fn poll(fds: [*]c.pollfd, nfds: c.nfds_t, timeout: c_int) c_int;
           ^~~

The client compiles on windows for me now (Check out the latest master)

Thanks, it is working fine weith me now:

The codes used are:
//server.zig
const std=@import("std");
const websocket = @import("./websocket.zig/src/websocket.zig");
const Conn = websocket.Conn;
const Message = websocket.Message;
const Handshake = websocket.Handshake;

// Define a struct for "global" data passed into your websocket handler
const Context = struct {
};

pub fn main() !void {
    var general_purpose_allocator = std.heap.GeneralPurposeAllocator(.{}){};
    const allocator = general_purpose_allocator.allocator();

    // this is the instance of your "global" struct to pass into your handlers
    var context = Context{};

    try websocket.listen(Handler, allocator, &context, .{
        .port = 9223,
        .max_headers = 10,
        .address = "<IP_ADDRESS>",
    });
}

const Handler = struct {
    conn: *Conn,
    context: *Context,

    pub fn init(h: Handshake, conn: *Conn, context: *Context) !Handler {
        // `h` contains the initial websocket "handshake" request
        // It can be used to apply application-specific logic to verify / allow
        // the connection (e.g. valid url, query string parameters, or headers)

        _ = h; // we're not using this in our simple case

        return Handler{
            .conn = conn,
            .context = context,
        };
    }

    // optional hook that, if present, will be called after initialization is complete
    pub fn afterInit(self: *Handler) !void {
        _ = self;}

    pub fn handle(self: *Handler, message: Message) !void {
        const data = message.data;
        std.log.debug("{s}", .{data});
        try self.conn.write(data); // echo the message back
    }

    // called whenever the connection is closed, can do some cleanup in here
    pub fn close(_: *Handler) void {}
};

And
//ws.zig
const std = @import("std");
const websocket = @import("./websocket.zig/src/websocket.zig");

const Handler = struct {
    client: websocket.Client,

    pub fn init(allocator: std.mem.Allocator, host: []const u8, port: u16) !Handler {
        return .{
            .client = try websocket.connect(allocator, host, port, .{}),
        };
    }

    pub fn deinit(self: *Handler) void {
        self.client.deinit();
    }

    pub fn connect(self: *Handler, path: []const u8) !void {
        try self.client.handshake(path, .{.timeout_ms = 5000});
        const thread = try self.client.readLoopInNewThread(self);
        thread.detach();
    }

    pub fn handle(_: Handler, message: websocket.Message) !void {
        const data = message.data;
        std.debug.print("CLIENT GOT: {s}\n", .{data});
    }

    pub fn write(self: *Handler, data: []u8) !void {
        return self.client.write(data);
    }

    pub fn close(_: Handler) void {}

};

pub fn main() !void {
    var general_purpose_allocator = std.heap.GeneralPurposeAllocator(.{}){};
    const allocator = general_purpose_allocator.allocator();

    var handler = try Handler.init(allocator, "<IP_ADDRESS>", 9223);
    defer handler.deinit();
    // spins up a thread to listen to new messages
    try handler.connect("/");

    var data = try allocator.dupe(u8, "hello world");
    try handler.write(data);

    data = try allocator.dupe(u8, "hello Zig");
    try handler.write(data);
    data = try allocator.dupe(u8, "hello Hasan");
    try handler.write(data);

    // without this, we'll exit immediately without having time to receive the
    // echo'd message from the server
    std.time.sleep(std.time.ns_per_s);
}


Change RTM default topic to match UKF node launch file

Status
PRODUCTION / DEVELOPMENT
Description
Fixes #2141
Related PRs
#1625
Steps to Test or Reproduce
#2141

I reproduced #2141 error.
Confirmed that the above error is fixed by this PR.

well i tried with ros kinetic and autoware 1.12, the same issue exist. I even try implementing fix manually and it didn't work.
Data used: moriyama_2015-06-11_02_free_download.bag


Fix: Frontend enhancements - style, typography, app name updates (#1928)

Ticket #1928
Description
Front-end updates to Grant Finder, including:

changing the app name
typography updates
button style updates
updating color inconsistencies generally (including refactoring some inline styles)
new login screen

There are no functionality updates.
Screenshots / Demo Video
See ticket #1928 for screenshots and a demo of the login screen
Testing

Go through the app and note if anything looks incorrect, or if anything behaves incorrectly

Automated and Unit Tests

[x] Added Unit tests (updated to reflect new app name)

Manual tests for Reviewer

[x] Added steps to test feature/functionality manually

Checklist

[x] Provided ticket and description
[x] Provided screenshots/demo
[x] Provided testing information
[x] Provided adequate test coverage for all new code
[x] Added PR reviewers

Looks good! Does this the text need more space here in the Grant Details UI?

 Looks good! Does this the text need more space here in the Grant Details UI?

@adele-usdr yes - good catch! redid some of my grant details updates in the latest push:


The default Unrestricted PodSecurityProfile does not match the default from Kubernetes examples

Rancher Server Setup

Rancher version: 2.6
Installation option (Docker install/Helm Chart): Docker

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:

Information about the Cluster

Kubernetes version:
Cluster Type (Local/Downstream):

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions:

Admin
Describe the bug
When installing a new cluster on vmware, the unrestricted-psp does not include the seccomp allowed profile names annotation, so any pod trying to use these don't start - including the rancher provided vmware csi driver.
To Reproduce
Install new cluster on vmware, with CSI driver
Result
Failing CSI driver pods
Expected Result
CSI Driver pods starting
Screenshots

Additional context
seccomp.security.alpha.kubernetes.io/allowedProfileNames=* should be added to podsecuritypolicy/unrestricted-psp, as per the example policy here:  https://kubernetes.io/docs/concepts/policy/pod-security-policy/#example-policies
I am not even sure where to go looking to submit a PR to fix this, sorry.

And this is why I moved away from Rancher.


Required changes for Azure HealthBot swagger

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[x] SDK of Java
[x] SDK of Js
[x] SDK of Go
[ ] PowerShell
[x] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@lirenhe The breaking change doesn't affect external customers - the service is not publicly exposed yet and is still being tested.

the service is not publicly exposed yet and is still being tested

It is publicly exposed in the Pulumi Azure NextGen provider. Is this API marked as pre-production somewhere?

@mikhailshilkov It's not marked as pre-production. The service is going to be publicly available next week.


fix yarn install issues

Changes
There was an issue naming Snowpack as a dependency, where yarn wouldn't treat it as a workspace member and wouldn't it's dependencies properly. As a result, it broke when used (cosmicconfig wrong version installed).
Testing
Will be better covered once CSA templates land here.

@drwpow I think I'll need your help on this one, it all works fine locally and SHOULD work fine in lerna, but for some reason its no good in CI. Maybe because I killed the second yarn install you were running with --force?

@drwpow I think I'll need your help on this one, it all works fine locally and SHOULD work fine in lerna, but for some reason its no good in CI. Maybe because I killed the second yarn install you were running with --force?

Pretty much how I was testing it was:

kill all node_modules in the entire repo (rm -rf . && git reset --hard HEAD)
run yarn && yarn test

If that passes on your machine, that should pass in CI (also ⚠ on the rm command feel free to use another method if you’d like)


Fix: OAuth2 Authorization Request OPTIONAL parameters are required by bruno

Description
OAuth2 configuration should allow empty scope or Callback URL fields, as these are optional.
Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Fixes #1797

Merged!
Thank you @pietrygamat !


feature/state_path_argument

Changes Unknown when pulling c0231ddecd8928b0d82448381b2eb64f22a414f2 on feature/state_path_argument into ** on master**.

Changes Unknown when pulling c0231ddecd8928b0d82448381b2eb64f22a414f2 on feature/state_path_argument into ** on master**.


Add solution to POSAND problem Oct-challange

@Vishruth-S  i made a PR, kindy review the same.

Ah!! @Vishruth-S ,  Sure will be commenting the code..

@Zaidtech  Alright. Please leave a reply here once you make the changes so that I can review 😃

@Vishruth-S kindly review the changes !!
Moreover is this repo under HacktoberFest :smile:

Thanks a lot! :smiley:

@Zaidtech Welcome 😄 . Also do star this repo if you haven't already 🙃

@Vishruth-S, if this repo comes under Hactoberfest can you please mark the PR as hactoberfest-accepted

@Zaidtech  The hacktoberfest-accepted label is only needed if this repo is not having the hacktoberfest Topic or I am not approving/merging your PR.
Since your PR was approved and merged in a hacktoberfest Topic Repo, it will be counted, no worries 😃 .
Source: https://hacktoberfest.digitalocean.com/hacktoberfest-update

Ah!! but I read the terms and the pr are not counted until they are labeled as hactoberfest-accepted  !! even one of my colleague has made his 4- PR's fail this way!! :sweat_smile:

I think you have been misguided. The link I sent you is the official hacktoberfest website and the guidelines given there are original and the correct ones.🙃
And this PR will definitely be counted if you have registered there 😃

ohk!! thanks a lot  @Vishruth-S :smile:


Why Ruby and not Go?

What would be the benefit?

That you don't need to install Ruby. Well, there's the Traveling Ruby project, but a Go binary would be much smaller. Pretty much the entire Kubernetes ecosystem is focused on Go.

Standalone binaries for pharos are available.
Traveling Ruby only has Ruby 2.2 support.
Btw, kubectl binary is about 52 megabytes, pharos binary is only about 15 megabytes.


Don't allow *dop objects to be mutated

As discovered in gh-150, printing out a laplacian changes the effect of the operator!
In order to ensure immutability, this:

Changes the following from list to tuple:
Sdop.terms
Pdop.terms
Changes the following functions to return a new copy, rather than mutating in place and returning themselves:

Sdop.simplify
Pdop.simplify
Pdop.factor
Dop.consolidate_coefs

Changes the following undocumented and unused functions to return a new copy, rather than mutating in place and returning None:

Sdop.TSimplify
Pdop.TSimplify
Sdop.sort_terms, which has been made private and renamed to _with_sorted_terms

Crucially, this removes the use of copy.deepcopy which was duplicating sympy expressions in weird ways.

Hmm, can't work out how to restart CI here


Add handling for unterminated strings

This PR introduces two new tokens for unterminated strings.
Without these tokens, an unterminated string will result in a confusing error message of "Illegal character '"' ..."
Additionally, this would stop parsing/tokenizing altogether and swallow any errors it had already encountered. By eliminating the tokenizing error (illegal char) unterminated strings can be reported alongside other errors with better error reports.
Before:
>>> json5.loads("[0o123, 'foo]")
[tb omitted]
JSON5DecodeError: Illegal character "'" at index 8

Notice that the octal (which are illegal) goes completely unnoticed until the unterminated string is fixed.
After:
>>> json5.loads("[0o123, 'foo]")
[tb omitted]
JSON5DecodeError: There were multiple errors parsing the JSON5 document.
The primary error was:
        Invalid integer literal. Octals are not allowed in or near token OCTAL at: line 1 column 2 (char 1)
Additionally, the following errors were also detected:
        Syntax Error in or near token UNTERMINATED_SINGLE_QUOTE_STRING at: line 1 column 9 (char 8)
        Expecting value. Unexpected EOF at: line 1 column 14 (char 13)

Also resolves #26


Remove badge section in the README

It seems the ss-python badge has generated fot users by default. And it is not necessary to mention again.

Good point!