noduleCADEvaluationLUNA16compare error

Hello Mr. @wentaozhu, on the ReadMe you say "You can directly run noduleCADEvaluationLUNA16.py, and get the performance in the paper." but when I try to directly run the .py, there's an error and I have not idea why it is, could you help me? Am I doing something wrong?
This is the error:
\DeepLung>python evaluationScript\noduleCADEvaluationLUNA16compare.py
Anotations file path:
C:\Users\aleja\Documents\DeepLung\evaluationScript\annotations\newannotations.csv
C:\Users\aleja\Documents\DeepLung\evaluationScript\annotations\newannotations_excluded.csv
C:\Users\aleja\Documents\DeepLung\evaluationScript\annotations\newseriesuids.csv
Traceback (most recent call last):
File "evaluationScript\noduleCADEvaluationLUNA16compare.py", line 530, in
(noduleCADEvaluation(annotations_filename,annotations_excluded_filename,seriesuids_filename,results_filename,'./', vis=True, delimiter='\t', clr = 'r', cap='3DDPN26FasterR-CNN'))
File "evaluationScript\noduleCADEvaluationLUNA16compare.py", line 509, in noduleCADEvaluation
(allNodules, seriesUIDs) = collect(annotations_filename, annotations_excluded_filename, seriesuids_filename)
File "evaluationScript\noduleCADEvaluationLUNA16compare.py", line 484, in collect
annotations          = csvTools.readCSV(annotations_filename)
File "C:\xxxx\xxxx\xxxx\DeepLung\evaluationScript\tools\csvTools.py", line 12, in readCSV
for line in csvreader:
_csv.Error: iterator should return strings, not bytes (did you open the file in text mode?)
What is the './' on the function "noduleCADEvaluation" for?
Thank you in advantage, I'll wait for your answer

Thank you very much for your help. In fact, I tried to read the CSV in a normal way, without binary files, just "r". But i have another question if you're okay with it.
When I'm trying to run "noduleCADEvaluationLUNA16.py", there's a part on computeFROC def, when you make a for and indexed a local FROCGT or FROCProb if there's a false on the "exclude list"
for i in range(len(excludeList)):
if excludeList[i] == False:
FROCGTList_local.append(FROCGTList[i])
FROCProbList_local.append(FROCProbList[i])
But when I try it, it just has True responses, so there's no way on how to calculate the roc_curve and this is the error:
Traceback (most recent call last):
File "evaluationScript\noduleCADEvaluationLUNA16.py", line 531, in
noduleCADEvaluation(annotations_filename,annotations_excluded_filename,seriesuids_filename,results_filename,'./')
File "evaluationScript\noduleCADEvaluationLUNA16.py", line 523, in noduleCADEvaluation
numberOfBootstrapSamples=bNumberOfBootstrapSamples, confidence=bConfidence)
File "evaluationScript\noduleCADEvaluationLUNA16.py", line 359, in evaluateCAD
fps, sens, thresholds = computeFROC(FROCGTList,FROCProbList,len(seriesUIDs),excludeList)
File "evaluationScript\noduleCADEvaluationLUNA16.py", line 144, in computeFROC
fpr, tpr, thresholds = skl_metrics.roc_curve(FROCGTList_local, FROCProbList_local)
File "C:\Users\aleja\anaconda3\envs\pytorch_exp\lib\site-packages\sklearn\utils\validation.py", line 63, in inner_f
return f(*args, **kwargs)
File "C:\Users\aleja\anaconda3\envs\pytorch_exp\lib\site-packages\sklearn\metrics_ranking.py", line 914, in roc_curve
y_true, y_score, pos_label=pos_label, sample_weight=sample_weight)
File "C:\Users\aleja\anaconda3\envs\pytorch_exp\lib\site-packages\sklearn\metrics_ranking.py", line 702, in _binary_clf_curve
pos_label = _check_pos_label_consistency(pos_label, y_true)
File "C:\Users\aleja\anaconda3\envs\pytorch_exp\lib\site-packages\sklearn\metrics_base.py", line 244, in _check_pos_label_consistency
f"y_true takes value in {{{classes_repr}}} and pos_label is not "
ValueError: y_true takes value in {} and pos_label is not specified: either make y_true take value in {0, 1} or {-1, 1} or pass pos_label explicitly.
Could you help me about it?

Traceback (most recent call last):
File "evaluationScript/noduleCADEvaluationLUNA16.py", line 522, in
noduleCADEvaluation(annotations_filename,annotations_excluded_filename,seriesuids_filename,results_filename,'./')
File "evaluationScript/noduleCADEvaluationLUNA16.py", line 514, in noduleCADEvaluation
numberOfBootstrapSamples=bNumberOfBootstrapSamples, confidence=bConfidence)
File "evaluationScript/noduleCADEvaluationLUNA16.py", line 350, in evaluateCAD
fps, sens, thresholds = computeFROC(FROCGTList,FROCProbList,len(seriesUIDs),excludeList)
File "evaluationScript/noduleCADEvaluationLUNA16.py", line 137, in computeFROC
fpr, tpr, thresholds = skl_metrics.roc_curve(FROCGTList_local, FROCProbList_local)
File "/home/arumota/anaconda3/envs/pytorch_exp/lib/python3.7/site-packages/sklearn/utils/validation.py", line 63, in inner_f
return f(*args, **kwargs)
File "/home/arumota/anaconda3/envs/pytorch_exp/lib/python3.7/site-packages/sklearn/metrics/_ranking.py", line 914, in roc_curve
y_true, y_score, pos_label=pos_label, sample_weight=sample_weight)
File "/home/arumota/anaconda3/envs/pytorch_exp/lib/python3.7/site-packages/sklearn/metrics/_ranking.py", line 702, in _binary_clf_curve
pos_label = _check_pos_label_consistency(pos_label, y_true)
File "/home/arumota/anaconda3/envs/pytorch_exp/lib/python3.7/site-packages/sklearn/metrics/_base.py", line 244, in _check_pos_label_consistency
f"y_true takes value in {{{classes_repr}}} and pos_label is not "
ValueError: y_true takes value in {} and pos_label is not specified: either make y_true take value in {0, 1} or {-1, 1} or pass pos_label explicitly.
In fact, I also tried on a Linux distro, maybe thinking it was about the lecture of "evaluationScript/annotations/newannotations.csv", but the same error happened

Yes. Try the linux and re-download all the files.

Thank you very much for your help. In fact, I tried to read the CSV in a normal way, without binary files, just "r". But i have another question if you're okay with it.
When I'm trying to run "noduleCADEvaluationLUNA16.py", there's a part on computeFROC def, when you make a for and indexed a local FROCGT or FROCProb if there's a false on the "exclude list"
for i in range(len(excludeList)): if excludeList[i] == False: FROCGTList_local.append(FROCGTList[i]) FROCProbList_local.append(FROCProbList[i])
But when I try it, it just has True responses, so there's no way on how to calculate the roc_curve and this is the error:
Traceback (most recent call last): File "evaluationScript\noduleCADEvaluationLUNA16.py", line 531, in noduleCADEvaluation(annotations_filename,annotations_excluded_filename,seriesuids_filename,results_filename,'./') File "evaluationScript\noduleCADEvaluationLUNA16.py", line 523, in noduleCADEvaluation numberOfBootstrapSamples=bNumberOfBootstrapSamples, confidence=bConfidence) File "evaluationScript\noduleCADEvaluationLUNA16.py", line 359, in evaluateCAD fps, sens, thresholds = computeFROC(FROCGTList,FROCProbList,len(seriesUIDs),excludeList) File "evaluationScript\noduleCADEvaluationLUNA16.py", line 144, in computeFROC fpr, tpr, thresholds = skl_metrics.roc_curve(FROCGTList_local, FROCProbList_local) File "C:\Users\aleja\anaconda3\envs\pytorch_exp\lib\site-packages\sklearn\utils\validation.py", line 63, in inner_f return f(*args, **kwargs) File "C:\Users\aleja\anaconda3\envs\pytorch_exp\lib\site-packages\sklearn\metrics_ranking.py", line 914, in roc_curve y_true, y_score, pos_label=pos_label, sample_weight=sample_weight) File "C:\Users\aleja\anaconda3\envs\pytorch_exp\lib\site-packages\sklearn\metrics_ranking.py", line 702, in _binary_clf_curve pos_label = _check_pos_label_consistency(pos_label, y_true) File "C:\Users\aleja\anaconda3\envs\pytorch_exp\lib\site-packages\sklearn\metrics_base.py", line 244, in _check_pos_label_consistency f"y_true takes value in {{{classes_repr}}} and pos_label is not " ValueError: y_true takes value in {} and pos_label is not specified: either make y_true take value in {0, 1} or {-1, 1} or pass pos_label explicitly.
Could you help me about it?

could you tell me how to solve this problem? thank you


[WIP] Issue-48: Allow users to set an onTimeout callback, overriding the de…

…fault behavior of throwing ConditionTimeoutException
Closes https://github.com/awaitility/awaitility/issues/48.
This is a work in progress, I want to get feedback on this before making more changes.
As discussed on the issue thread, I've added an onTimeout method to the Awaitility API that lets users replace the default timeout handling behavior. The method lets the user set a callback method, which receives a context object containing various useful properties for e.g. constructing an error message.
The default timeout handling has been moved to a default callback, so there should be no behavioral changes unless users call onTimeout. I've added a convenience method for users (me) that want to make Awaitility throw AssertionError instead of ConditionTimeoutException. I'm happy to remove it if it isn't generally useful, since it is easy for users to implement on their own.
The part that's missing would be adding a similar hook for the case where the statement finishes too soon (at https://github.com/awaitility/awaitility/blob/548bcc27326a2a91ffb30c2b192b2e285009f982/awaitility/src/main/java/org/awaitility/core/ConditionAwaiter.java#L146). I think the best way to support that branch is to create an extra onEarlyTermination or something, since I think it will be irritating for people to have to check in onTimeout whether the statement finishes too early or late. I'll add that hook to this PR once the direction of these changes are approved.
I took a look at some other issues (notably https://github.com/awaitility/awaitility/issues/106, but there are others that are also blocked pending a Java upgrade), and got the impression that upgrading to Java 8 might be a possibility. Java 6/7 have been EOL'd for years now. That said, if upgrading the Java version is not an option I'll try to rework this to work with older versions.

Just saw this work - this is awesome!  This would be a high impact change!  Is this still being worked on?


Moving the  feature from drafts

I reviewed the  element file. The baseline date seems recent, but looking through Chrome status, it was added in Chrome 62

@tropicadri thanks for moving this along! Can you rebase or merge to resolve the conflict in time.yml.dist? Whatever the conflict is doesn't really matter, just make sure features/draft/html-elements/time.yml.dist doesn't exist, and run npm run dist features/time.yml so that the proper dist files does.
Here's what it looks like for me locally:
status:
  baseline: high
  baseline_low_date: 2018-10-02
  baseline_high_date: 2021-04-02
  support:
    chrome: "62"
    chrome_android: "62"
    edge: "18"
    firefox: "22"
    firefox_android: "22"
    safari: "10"
    safari_ios: "10"
compat_features:
  # baseline: high
  # baseline_low_date: 2017-10-24
  # baseline_high_date: 2020-04-24
  # support:
  #   chrome: "62"
  #   chrome_android: "62"
  #   edge: "14"
  #   firefox: "22"
  #   firefox_android: "22"
  #   safari: "10"
  #   safari_ios: "10"
  - api.HTMLTimeElement
  - api.HTMLTimeElement.dateTime

  # baseline: high
  # baseline_low_date: 2018-10-02
  # baseline_high_date: 2021-04-02
  # support:
  #   chrome: "62"
  #   chrome_android: "62"
  #   edge: "18"
  #   firefox: "22"
  #   firefox_android: "22"
  #   safari: "7"
  #   safari_ios: "4"
  - html.elements.time
  - html.elements.time.datetime

It's surprising that the status of the API and element isn't the same, this is probably wrong somehow.
The Baseline low date 2018-10-02 is the release date of Edge 18 (found by searching in BCD) so the most important question is whether Edge 14 or 18 is correct for Edge.
I doesn't look like the Safari data will change the Baseline low date, but figuring out which is the correct data would still be good.

I've sent https://github.com/mdn/browser-compat-data/pull/23638 to align the Edge data.
https://bugs.webkit.org/show_bug.cgi?id=148840 also confirms that HTMLTimeElement came later than <time> in WebKit, so we can leave that data as-is I think. It's not going to define the Baseline low date anyway.

In order to not have to wait for the next BCD release, adding this to the source YAML would work:
# TODO: remove after https://github.com/mdn/browser-compat-data/pull/23638
status:
  compute_from: api.HTMLTimeElement

I added the comment and regenerated the dist file.


Defining functions doesn't work

Jade4j does not seem to support defining functions, even though the JEXL documentation linked from readme lists it as part of JEXL syntax.  Trying the example from documentation:
- var fun = function(x, y) { x + y }
causes
org.apache.commons.jexl2.JexlException$Parsing:
de.neuland.jade4j.expression.JexlExpressionHandler.evaluateExpression@1:31
parsing error near '... + y } ...'
	at org.apache.commons.jexl2.JexlEngine.parse(JexlEngine.java:1260)
	at org.apache.commons.jexl2.JexlEngine.createScript(JexlEngine.java:506)
	at org.apache.commons.jexl2.JexlEngine.createScript(JexlEngine.java:452)
	at de.neuland.jade4j.expression.JexlExpressionHandler.evaluateExpression(JexlExpressionHandler.java:46)
	... 31 more

Should work with https://github.com/neuland/pug4j
Because it now uses jexl3


Style issue with StackView component

Describe the bug
When we have nested stack-block, expanded state styles are not applied to stack-block-content component as shown in attached image.

How to reproduce
https://stackblitz.com/edit/clarity-v3-dark-theme-timeline-demo-fadjma
Steps to reproduce the behavior:

Expand label 3
Expand Label 3-1

Expected behavior
Styles should be consistent
Versions
Clarity V3

I was able to fix this by adding the following styles:
clr-stack-block.stack-block.stack-block-expanded:not(.stack-block-expandable) > .stack-block-label > .stack-block-content,
clr-stack-block.level-1-with-children.stack-block-expanded > .stack-block-label > .stack-block-content,
clr-stack-block.level-1-with-children.stack-block-expandable:hover:not(.stack-block-expanded) > .stack-block-label > .stack-block-content,
clr-stack-block.level-1-with-children.stack-block-expandable:not(.stack-block-expanded) > .stack-block-label > .stack-block-content {
    background-color: inherit;
}


onnxruntime: 1.15.1 -> 1.16.3

This is an attempt at fixing and updating onnxruntime. Unfortunately, some tests are failing:
onnxruntime> [----------] Global test environment tear-down
onnxruntime> [==========] 3790 tests from 269 test suites ran. (67205 ms total)
onnxruntime> [  PASSED  ] 3780 tests.
onnxruntime> [  FAILED  ] 10 tests, listed below:
onnxruntime> [  FAILED  ] TransposeOpTest.PermRankDoesNotMatchTensorRank
onnxruntime> [  FAILED  ] TfIdfVectorizerTest.Int32_TF_onlyBigrams_Skip0_Empty_Dim1Fail
onnxruntime> [  FAILED  ] TfIdfVectorizerTest.Int32_TF_onlyBigrams_Skip0_Empty_Dim2
onnxruntime> [  FAILED  ] TfIdfVectorizerTest.Int32_TF_onlyBigrams_Skip01_Empty_Dim2
onnxruntime> [  FAILED  ] ConvTest.Conv1D_Invalid_Input_Shape
onnxruntime> [  FAILED  ] ConvTest.Conv2D_Invalid_Input_Shape
onnxruntime> [  FAILED  ] InferenceSessionTests.TestStrictShapeInference
onnxruntime> [  FAILED  ] FunctionTest.AttrSaturateNan
onnxruntime> [  FAILED  ] GraphTransformationTests.GatherToSplitFusion
onnxruntime> [  FAILED  ] GraphTransformationTests.GatherToSplitFusion_NoSqueeze

I don't know if this is due to the fact that we use an outdated version of googletest compared to what they have upstream...
Changelog: https://github.com/microsoft/onnxruntime/releases/tag/v1.16.3
cc @jonringer  @puffnfresh @ck3d @cbourjau
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Here are some more detailed logs of the failing tests:

onnxruntime:GraphTransformationTests

2024-02-02 15:21:40.599033015 [W:onnxruntime:GraphTransformationTests, graph.cc:108 MergeShapeInfo] Error merging shape info for output. 'node' source:{-1} target:{2,3,3,3}. Falling>
/build/source/onnxruntime/test/optimizer/graph_transform_test.cc:6366: Failure
Value of: _tmp_status.IsOK()
  Actual: false
Expected: true
[ONNXRuntimeError] : 1 : FAIL : CountOpsInGraph(graph)["Gather"] == 0 is evaluated to false
[  FAILED  ] GraphTransformationTests.GatherToSplitFusion_NoSqueeze (0 ms)

TransposeOpTest.PermRankDoesNotMatchTensorRank

[ RUN      ] TransposeOpTest.PermRankDoesNotMatchTensorRank
/build/source/onnxruntime/test/providers/op_tester.cc:94: Failure
Value of: status.ErrorMessage()
Expected: has substring "Node:node1 Output:Y [ShapeInferenceError] Mismatch between number of source and target dimensions. Source=3 Target=4"
  Actual: "Node:node1 Output:Y [ShapeInferenceError] Mismatch between number of inferred and declared dimensions. inferred=3 declared=4"
[  FAILED  ] TransposeOpTest.PermRankDoesNotMatchTensorRank (0 ms)

TfIdfVectorizerTest.Int32_TF_onlyBigrams_Skip0_Empty_Dim2

[ RUN      ] TfIdfVectorizerTest.Int32_TF_onlyBigrams_Skip0_Empty_Dim2
/build/source/onnxruntime/test/providers/op_tester.cc:94: Failure
Value of: status.ErrorMessage()
Expected: has substring "Mismatch between number of source and target dimensions. Source=2 Target=1"
  Actual: "Node:node1 Output:Y [ShapeInferenceError] Mismatch between number of inferred and declared dimensions. inferred=2 declared=1"
[  FAILED  ] TfIdfVectorizerTest.Int32_TF_onlyBigrams_Skip0_Empty_Dim2 (0 ms)

ConvTest.Conv2D_Invalid_Input_Shape

[ RUN      ] ConvTest.Conv2D_Invalid_Input_Shape
/build/source/onnxruntime/test/providers/op_tester.cc:94: Failure
Value of: status.ErrorMessage()
Expected: has substring "Node:node1 Output:Y [ShapeInferenceError] Can't merge shape info. Both source and target dimension have values but they differ. Source=1 Target=2 Dimension=>
  Actual: "Node:node1 Output:Y [ShapeInferenceError] Can't merge shape info. Both inferred and declared dimension have values but they differ. Inferred=1 Declared=2 Dimension=0"
[  FAILED  ] ConvTest.Conv2D_Invalid_Input_Shape (0 ms)

See also https://github.com/NixOS/nixpkgs/pull/258392

Here are some more detailed logs of the failing tests:

I think these errors are due to building onnxruntime with onnx 1.15 rather than 1.14 as expected by upstream.

Duplicate of #258392


Allow configuration of SYNC_PASSWORD setting for SCIM integration

We are seeing passwords being synchronized via the Terraform configured SCIM integration via Okta; which presents a security concern.
The Snowflake SCIM Integration supports a SYNC_PASSWORD property to disable this functionality, but it cannot currently be set via the Terraform provider.
We would like the Terraform provider to be able to optionally configure the SYNC_PASSWORD property as part of the "snowflake_scim_integration" (especially if using Okta).
Currently, the only option to set this property is outside of Terraform.

This is an important setting for our setup.
We want to completely disable the user of password basic auth for any user that is managed with Okta.
We currently have to manually go in after the fact and alter this setting via SQL.

Hey @ehluke @bschwedler   👋
The new version v0.93.0 is adjusted and contains sync_password. Please follow the migration guide.


support custom grammared sequence types

Changes:

[ ] rename  skbio.sequence._iupac_sequence.IUPACSequence class to GrammaredSequence: it is a sequence that conforms to the scikit-bio sequence grammar (based on IUPAC's notion of degenerates, gaps, nondegenerates, etc. but not necessarily constrained to the IUPAC alphabet).

[ ] make GrammaredSequence publicly importable from skbio.sequence

[ ] document GrammaredSequence as a place for users to inherit from to create grammared sequences with custom alphabets (add a doctest example of this too).

[ ] for GrammaredSequence make a metaclass that performs validation of its alphabet (e.g., default_gap_char should be present in gap_chars; degenerates must expand into valid nondegenerates; etc.)

[ ] make it impossible to inherit from DNA, RNA, and Protein and document why

@jairideout @ebolyen What exactly is meant by inheriting from GrammaredSequence? For example, say we have:
class GrammaredSequence(type):
    def __new__(cls, name, parents, dct):
        if dct['default_gap_char'] not in dct['gap_chars']:
            raise Exception("...")
        return super(GrammaredSequence, cls).__new__(cls, name, parents, dct)

class CustomSequence(object, metaclass=GrammaredSequence):
    gap_chars = '-.'
    default_gap_char = '.'

That works fine. However, IUPACSequence currently also has inheritable behavior associated with it, for example the _validate method. So we need a way to pass on that functionality. Here's one way:
class GrammaredSequence(type):
    def __new__(cls, name, parents, dct):
        if dct['default_gap_char'] not in dct['gap_chars']:
            raise Exception("...")

        dct['_validate'] = cls._validate

        return super(GrammaredSequence, cls).__new__(cls, name, parents, dct)

    def _validate(cls):
        pass

That feels really hacky and I wouldn't be surprised if there are serious consequences I'm not aware of. Another solution would be to have a separate class for behavior:
class GrammaredSequenceMeta(type):
    def __new__(cls, name, parents, dct):
        if dct['default_gap_char'] not in dct['gap_chars']:
            raise Exception("...")
        return super(GrammaredSequenceMeta, cls).__new__(cls, name, parents, dct)

class GrammaredSequence(object):
    def _validate(cls):
        pass

class CustomSequence(GrammaredSequence, metaclass=GrammaredSequenceMeta):
    gap_chars = '-.'
    default_gap_char = '.'

I like this better but still not ideal. A third approach is sort of a combination of the first 2:
class GrammaredSequenceMeta(type):
    def __new__(cls, name, parents, dct):
        if dct['default_gap_char'] not in dct['gap_chars']:
            raise Exception("...")

        parents = tuple([GrammaredSequence])

        return super(GrammaredSequenceMeta, cls).__new__(cls, name, parents, dct)

class GrammaredSequence(object):
    def _validate(cls):
        pass

class CustomSequence(metaclass=GrammaredSequenceMeta):
    gap_chars = '-.'
    default_gap_char = '.'

With that the metaclass injects the parent which contains the behavior. I like this the most, but I still feel like there might be a better way. Ideally someone could simply inherit GrammaredSequence without ever knowing it was a metaclass. Do you guys know of anything?

Can you make a private metaclass (e.g. GrammaredSequenceMeta) that performs alphabet/grammar validation and make GrammaredSequence use that metaclass? Then users won't have to know/care about the metaclass (because it is private) and they can just inherit from GrammaredSequence as usual.

Sorry for the confusion, the issue description is unclear about GrammaredSequence having a metaclass that performs validation. I'll fix that.

It looks like that will work. The only caveat I see is that GrammaredSequence must have valid defaults defined for things to be validated, but that doesn't seem like too much of a problem.

Would it be possible to inherit from ABCMeta for GrammaredSequenceMeta and only do the checks after super is done? Then the ABC can complain, and when it's done we get to as well.

I was just looking at how to resolve the multiple metaclass problem. We can try chaining six.add_metaclass or maybe something like this.

Seems to work:
class Meta(type):
    def __new__(cls, name, parents, dct):
        print("meta called")

        return super(Meta, cls).__new__(cls, name, parents, dct)

class OtherMeta(type):
    def __new__(cls, name, parents, dct):
        print("other called")

        return super(OtherMeta, cls).__new__(cls, name, parents, dct)

@six.add_metaclass(OtherMeta)
@six.add_metaclass(Meta)
class MyType(object):
    pass

meta called
other called

Works with ABCMeta, too. Makes me wonder, why does ABCMeta raise exceptions at object instantiation time rather than class declaration time?

Cool! ABCMeta probably behaves that way because you could decorate properties onto the class-object prior to any instantiation, so the ABC might be fulfilled after class declaration (or perhaps even in response to failure).

Does it still make sense for lowercase characters to be invalid by default for any GrammaredSequence? Seems like not.

hmm, I think you are right, but some of our writers will call lowercase so a GrammaredSequence would need to have that method at least, maybe it just wouldn't do anything?

Fine with me. If this becomes an issue at some point we can deal with it then.

@anderspitman @jairideout pointed out that this shouldn't be an issue at all in practice, if a lowercase letter is in the alphabet then it will pass _validate. So just don't have the metaclass care about casing.

@jairideout @ebolyen The metaclasses don't seem to be playing nice with classproperty. When I try to access any of the properties from the metaclass within __new__ or  __init__ I'm getting:
TypeError: argument of type 'classproperty' is not iterable

Can we develop a more complete list of validations to perform? Other than default gap and degenerate expansion?

gaps, degenerates, and nondegenerates do not appear in each other's sets
I think that is probably it.

Will do.

Also, I think we need to make classproperty publicly importable in order for people to inherit from GrammaredSequence

Either that or automagically decorate the appropriate attributes from the subclass

I'm ok with that, but static properties work fine too.
class MySequence(GrammaredSequence):
    degenerate_map = {'X': set(['A', 'B', 'C')}
    nondegenerate_chars = set(['A', 'B', 'C'])

If the user is creating their own class, we can trust them to understand what mutation will do.
We use classproperty to avoid situations (which have happened) where the alphabet will be borrowed and then mutated for a different purpose without realising the original on the class has been modified.

The problem is that the error if they forget to make it static or classproperty is very vague: "argument of type 'function' is not iterable".
I'll make classproperty public and document to use that or static properties.

:+1:
On Wed, Dec 9, 2015 at 12:00 PM, Anders Pitman<EMAIL_ADDRESS>wrote:

The problem is that the error if they forget to make it static or
classproperty is very vague: "argument of type 'function' is not
iterable".
I'll make classproperty public and document to use that or static
properties.
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/scikit-bio/issues/1175#issuecomment-163358165
.

Here's my initial approach for disabling subclassing:
class DisableSubclassingMeta(type):
    def __new__(mcls, name, parents, dct):
        for parent in parents:
            if getattr(parent, '__subclassing_disabled__', False):
                raise Exception("Subclassing is disabled for class %s" % parent.__name__)
        cls = super(DisableSubclassingMeta, mcls).__new__(mcls, name, parents, dct)
        cls.__subclassing_disabled__ = True
        return cls

This works great on its own, but unfortunately applying that metaclass is a mess due to trying to combine ABCMeta, GrammaredSequenceMeta, and DisableSubclassingMeta all at once. It's running into MRO issues. Any suggestions?

Here is another example which doesn't use a dunder property:
http://stackoverflow.com/a/2826746/579416
As for mro would it be possible to just say with_metaclass(Final, GrammaredSequence)? Ideally each would only need this one metaclass and the rest is inheritance. I guess I don't know how you are applying this.

Using the current tip of #1219 the following doesn't work:
class MyClass(with_metaclass(Final, GrammaredSequence)):
    pass

First because Final isn't inheriting ABCMeta, GrammaredSequenceMeta, and type. Changing the declaration of Final to:

class Final(ABCMeta, GrammaredSequenceMeta, type):
    ...

Changes the error to MRO resolution. But I haven't been able to figure out a solution. Probably will end up needing to structure our metaclass inheritance very carefully. I'll keep working on it...

See the section on "Conflicting Metaclasses" here.

What if you used add_metaclass? It should reconstruct the classobj after GrammaredSequence has done it's thing?
If the MRO is still an issue, skip using super and call __new__ explicitly on the typeclass

Thanks for the link! I'll check it out in a few.

@anderspitman We're going to bump this to the next release, so there's a ton of time to figure this out now.

Sounds good.

@jairideout @ebolyen finally looked at this again. The MRO problem was easily fixed by inheriting only from GrammaredSequenceMeta. Currently all requested functionality has been implemented. I just need to add tests and docs for the last bit. Oh yeah and merge...
One last thing. I feel like I already asked this at one point but can't remember: why exactly are we not allowing subclassing of DNA, RNA, and Protein?

Awesome, thanks @anderspitman! We're not allowing subclassing of DNA, RNA, and Protein because some of their methods are hardcoded to use a specific alphabet (IUPAC), so subclasses that redefine the alphabet may not work correctly.

Fixed in #1273.


Fix jsx typo in map header

autocompleted here is misspelled and should be campelCase to conform with jsx property naming conventions.
This fixes a React warning in development.
autocompleted => autoComplete

I wouldn't mind giving this a go as a first timer!

Sorry looks like there are already two PR's

womp womp, oh well, I'll keep looking around. Thanks!

Looks like this issue is still open.
Please read the Contributing guidelines first. Then claim this issue. Any PR that does not conform to the guideline will be closed.

@BerkeleyTrue I'm not a first-timer, but I can take this up, since it's open, and no one seems to be on it.

I'll claim it as a first-timer, if it already hasn't been taking!

@vinhhv Please follow the guide to ensure speedy acceptance.


picture_settings only contains the last set values

I was trying to integrate the new picture_settings and ability to set brightness, contrast, etc in home assistant and this is what I observed.
Initially picture_settings contains all four brightness, contrast, backlight and color properties. But once I called set_current_picture_settings with a dict containing only backlight, the picture_settings now only contains the backlight property.
Is this expected? How to ensure that all the properties are correctly reported even after setting one of the properties optionally?

Hi! Can you write down the exact steps (methods) to reproduce it, along with the expected result? I can take a look at it then.

Here is the minimal script that I tried with the output.
import asyncio
from aiopylgtv import WebOsClient

async def on_state_change():
    print("State changed:", client.picture_settings)

async def runloop():
    global client
    client = await WebOsClient.create('<host>', '<key file>')

    await client.register_state_update_callback(on_state_change)

    await client.connect()

    print("Updating backlight")
    picture_settings = {'backlight': 100}
    await client.set_current_picture_settings(picture_settings)

    await asyncio.sleep(5)
    print("After sleep:", client.picture_settings)

    await client.disconnect()

asyncio.get_event_loop().run_until_complete(runloop())

Output:
State changed: {'contrast': 100, 'backlight': 50, 'brightness': 50, 'color': 70}
Updating backlight
State changed: {'backlight': 100}
After sleep: {'backlight': 100}
State changed: None

Expected:
After calling set_current_picture_settings, it should report all the properties as before.
Actual:
After calling set_current_picture_settings, it reports only the properties that were updated.

I've fixed it in my fork, no release yet, so you have to install it from source.
(I also added the ability to specify certain static states and state updates.)
What happened was: the server only sent back the modified value(s), and it has overwritten the original ones.
Solution: merge the sent value(s) with original.
I'm not sure whether other state updates are also affected or not, you can see the full list here.

I can imagine that what's happening is that the subscription is modified underneath, so I would assume that the other values will not update in this condition.
Not sure what the proper solution is, other than always calling the get api again with the full set of parameters.

This should work fine, since we update it with values from the server, so I don't think another call is needed.
Not sure about the rest.

I released it (along with couple of other changes).

@bendavid Any update on this? I guess merging the results with an earlier reaponse might work on client side as well, but it'd be much nicer if the case was already handled here the right way. (I'm too new to this to know if there's a better way as well)
If we know the correct solution I might also be able to help of needed. Or if some more investigations I could do.


Clarification on Mobius Controller

    @pettermahlen We are mostly using mobius loops with viewmodel across our app.

What is the use case for a mobius controller?
Can Mobius Controller be thought of an equivalent to VM with loop?
Could you give me actual example use cases for Using Mobius controller over VM with loop?
Originally posted by @rajdheepak in https://github.com/spotify/mobius/issues/157#issuecomment-1306633286

The controller it's just an utility to be able to interrupt and continue a loop from its last state, plus it helps set up the cyclic dependency between UI and business logic. The controller predates MobiusLoopViewModel, and if you use MLVM then there is no need to have a controller.


Generic: Support direct mapping of curl parameters if not explicitly configured - (Was: Follow redirect?)

Is it possible to follow a url redirect using pyresttest?

Not currently; the curl/pycurl FOLLOWLOCATION option doesn't have a binding, though it would be trivial to add one.

@davidkurkov I'm converting this to a more generic form, which would enable broader functionality and catch curl options that don't have defined configuration in pyresttest yet.
Options would be on tests/benchmarks, with format like so:
---
- test:
  - name: bob
  - curl_option_followlocation: true
  - curl_option_moremagic: true

Internally, every curl_option is capitalized and converted on parsing to a test/benchmark config setting.
'curloption_optionname' which becomes curl.setopt(curl.OPTIONNAME, value).
Some sort of inspection of legitimate curl options would be desirable at parsetime to avoid test runtime errors.  The following works (tested) but is a bit clunky
democurl = pycurl.Curl()  # Actually must have a concrete instance
optionname = 'curloption_followlocation'
curlopt = optionname.lstrip('curloption_').upper()
if getattr(democurl, curlopt):
  pass # Set config variable, which will use this with setopt
else :
  raise Error("Invalid curl option given:  "+curlopt)

Thank you @svanoort!

@davidkurkov By the way, it's slotted into the 1.6.0 release, which is slated to be out on PIP in a week or so:  https://github.com/svanoort/pyresttest/milestones/Release 1.6.0
Current in-progress code here: https://github.com/svanoort/pyresttest/pull/83
So... available for full use soon! :santa:

@davidkurkov Available now in a beta state, as of: https://github.com/svanoort/pyresttest/pull/88


Name entered during registration does not appear on profile.

[x] Add to an appropriate Epic.

Tester (if different from issue creator):
Site: MLA Commons profile
URL: https://mla.hcommons-dev.org/members/hctester10/
Test Account: hctester10
Area (CORE, sites, profiles, groups, general): profile
If issue discovered on dev, does it also exist on prod? Not sure
Describe the issue
The tester name I entered during registration does not appear on the profile. The name does appear in the upper right, beside the avatar in the header. The title and affiliation from the MLA database correctly populated on the profile.
Perhaps relatedly,

the MLA badge is missing (though the tester has access to join MLA Commons groups)
the account doesn't seem to appear in the member directory.
MLA doesnt appear under "Current memberships" on settings.
The first few times I clicked on "Settings" on the dropdown menu below the avatar I was redirected to the hc (dev) homepage, ie a page not found. When I clicked on "General" beside "settings" the page loaded. The page loads correctly on prod. And after getting the page to load, I can't replicate this.

To Reproduce
Steps to reproduce the behavior:

Register a tester with MLA membership.
Check the profile's name field.
Check for MLA badge.
Check Settings / Current memberships
Check /members

Expected behavior
The name entered during registration should appear on the profile.
Screenshots
If applicable, add screenshots to help explain your problem.
Optional details, if they seem relevant
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

The name (Barty Test) does appear in WP Admin, so it's not an enrollment issue.
Name is blank when you edit profile. When profile is edited, name in profile appears, and is also changed in WP Admin.

Ready for testing.
See: https://github.com/MESH-Research/humanities-commons/pull/49

Name was filled correctly on profile upon registration. This fixed worked! Thanks, @mikethicke .
I'll create another issue for the other issues remaining. (And I realized that not appearing in /members is not a problem--it doesn't show yourself in the list of members. Logged in with a different account i see the tester in the member directory.)

Member type is blank on enrollment. No "HC."


Re-enable prebuilt check in CI in master

With #1576, prebuilt checks were turned off until all repos could be included.  These should be turned on again once all repos are in.

All repos have been in for a while, and we're using the enabled prebuilt checks to burn down the prebuilt list: https://github.com/dotnet/source-build/issues/1745.


Manifest drag and drop

Seems like something we could support. See iiif/iiif.io#602

:+1: This seems easy to do.

We should support drag

Maybe better as a pom ticket?


[KYLIN-5002]Not allowed build new Segment while running optimize.

Proposed changes
Not allowed build new Segment while running optimize.
when cube run optimize cuboidBytesRecommend is not null.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

retest


Refs #25175 -- Removed postgresql_psycopg2 in django.db.utils.load_backend().

postgresql_psycopg2 package doesn't exist anymore.
Follow up to 944469939b9eb93fda0924f78faba5c0ffae2dff.

@carltongibson Thanks for checking :+1:

@carltongibson Thanks for checking :+1:


feat: add iroh doctor derp accept and iroh doctor derp connect commands for testing derp only data transfers

Description
Tried to make this derp-y as possible. And by that I mean, bypasses all of MagicSock and MagicEndpoint, using the iroh doctor accept and iroh doctor connect as a framework for the protocol and progress bars.
Notes & open questions
Some of the adjustments to shoe-horn the derp::http::Client into the already established ProgressBar work is goofy and wasteful.
We would probably be better off implementing AsyncWrite and AsyncRead over the derp::http::Client, but I was worried about adding too much potentially buggy read / write code here, when we are already dealing with read / write errors inside of derp itself. Could be a good future edition, tho.
Change checklist

[ ] Self-review.
[ ] Documentation updates if relevant.
[ ] Tests if relevant.

What's the status of this? Seems stale at this point?

closing for now


Documentation: singlePlayback bug 🐛

Describe the bug

There seems to be an issues with Single Playback example singlePlayback

The singlePlayback property will ensure that only one media provider is playing at any given moment. All providers that are registered with <MediaSync> and playing will be paused if a new provider begins to play

To Reproduce
Steps to reproduce the behavior:

Go to Single Playback
Click on first player
Click on second player
See error, both players are playing at the same time

Expected behavior

You'd expect to only have one player playing at a time, and if the second one is click while the first one is playing, the first one should pause.

The singlePlayback property will ensure that only one media provider is playing at any given moment. All providers that are registered with <MediaSync> and playing will be paused if a new provider begins to play

Screenshots

Additional context

Tbh I don't know if the bug is with the demo on the documentation site or the functionality itself. (I guessing probably the latter)
Happy to contribute with a PR with a bit of guidance on how to best tackle this.

Thanks for bringing this up. Interesting, this may be a browser/device issue? I just tested on Desktop Chrome and it's working fine. As mentioned in Discord, we're in the middle of a tooling upgrade but we can come right back to this after next week 👍

Sounds good, thanks! happy to test it again when ready (not urgent on my end)

Same issue here in Chrome version 108.0.5359.98 (Official Build) (arm64). In safari 15.6.1  it doesn't want to load at all (showing the html5 controls), but not playing on click.

Closing for now as this as <vds-media-sync> has temporarily been removed as we review it. If the issue persists with the new implementation we can open a new ticket :)


Add example section to patterns

Is your feature request related to a problem? Please describe.
There are quantum computing patterns that refer to or show examples on how these patterns can be used. This is not possible in the Pattern Atlas currently.
Describe the solution you'd like
Add an example section to the pattern views and pattern-candidate views.

Fixed in pattern-atlas-content by extending the QC pattern schema


Lovelace Resources with type: module Fail To Load via Fully Kiosk in Fire OS

Home Assistant release with the issue: -0.82.0
Last working Home Assistant release (if known): Unknown
Operating environment (Hass.io/Docker/Windows/etc.): Docker via Debian VM
Component/platform: Lovelace UI
Description of problem:
In Lovelace UI some custom elements display the error "Custom element doesn't exist: mini-media-player." when using FullyKiosk on a Fire Tablet with latest Fire OS.  This appears to be, in part, due to an outdated Android WebView (version 59 instead of version 70) on Fire OS.
Most of my custom cards in Lovelace appear to load without an issue, but I noticed any using 'type: module' instead of 'type: js' exhibit this error. Simply changing most over to 'type: js' fixes the error.

I don't know if this is a bug or the result of 'type: module' not being supported by older versions of Android WebView, but we should probably at least seek to update the Lovelace wiki, if nothing else.

You're correct, this is due to the outdated webview in FireOS. A PR to the docs is welcome to update the Lovelace docs to mention this.


[runtime/stream] Remove uint32 framing codec from runtime and move it to stream

fixes #185

nit: I'd recommend renaming this PR. I suggest "[runtime + stream] Migrate u32 stream framing from runtime to stream::placeholder"


X509 Certificate Authenticator Fails for Super Tenant Secondary Userstore Users when SearchAllUserStores config is Disabled

Describe the issue:
X509 certificate authenticator fails for Super Tenant secondary userstore users with SearchAllUserStores config disabled.
How to reproduce:

Setup X509 certificate authenticator in WSO2 IS [1].
Create a traditional web app for the super tenant.
Configure basic authentication as the first step and X509 certificate authenticator as the second step for the newly created application.
Add a secondary user store.
Create a new user in the secondary userstore.
Create a x509 certificate for the created user following the steps in [2]

Here add the CN with userstore domain appended (ex: SecondaryUS/johndoe) since we have not enabled SearchAllUserStores config.

Upload the certificate into the browser.
Try login into the application the created user.
It results in the following error.

[2024-03-04 13:11:25,791] [31c53181-d5af-4e2d-ab80-651aaa90ce96] ERROR {org.wso2.carbon.identity.application.authentication.framework.handler.step.impl.DefaultStepHandler} - Authentication failed exception! Couldn't find X509 certificate to this authenticated user: johndoe

Furthermore, the browser ends up in a 500 Internal Server Error due to a known another issue - https://github.com/wso2/product-is/issues/19760.

Expected behavior:
Successful authentication when a valid x509 certificate is passed for the super tenant secondary userstore user.
Environment information:

Product Version: IS 7.0.0
OS: macos
Database: default
Userstore: default

Related issues:

https://github.com/wso2-extensions/identity-outbound-auth-x509/pull/34
https://github.com/wso2/product-is/issues/19820

[1] - https://is.docs.wso2.com/en/7.0.0/guides/authentication/mfa/add-x509-login
[2] - https://is.docs.wso2.com/en/7.0.0/guides/authentication/mfa/add-x509-login/#configure-x509-certificate-authenticator

Root Cause Analysis

According to [1], when the SearchAllUserStores config is enabled, the certificate does not need to contain the CN with user store domain appended. Therefore, when the above config is NOT enabled (default behavior), certificate CN should contain the user store domain (ex: SecondaryUS/johndoe).
When username is validated in X509 certificate authenticator, it checks whether the CN of the certificate is equal to the username of the authenticated user. There, when extracting the username from the authenticated user, for super tenant users, it removes the @carbon.super from the username by calling [3]. This not only removes the tenant domain but removes the secondary userstore domain as well from the username. But when SearchAllUserStores config is disabled, the username should contain the usertsore domain.
Therefore, as a solution, removeDistinguishedName() method [4] can be used instead of [3], which will only remove tenant domain from the username for super tenant users.
Furthermore, when SearchAllUserStores config is enabled, the userstore domain will be removed from the username with the PR [5].

[1] - https://github.com/wso2-extensions/identity-outbound-auth-x509/pull/34#pullrequestreview-310331801
[2] - https://github.com/wso2-extensions/identity-outbound-auth-x509/blob/2f12dcfb92bb147379efff6cf491f7d2c5cb18f4/component/authenticator/src/main/java/org/wso2/carbon/identity/authenticator/x509Certificate/X509CertificateAuthenticator.java#L590-L591
[3] - https://github.com/wso2-extensions/identity-outbound-auth-x509/blob/2f12dcfb92bb147379efff6cf491f7d2c5cb18f4/component/authenticator/src/main/java/org/wso2/carbon/identity/authenticator/x509Certificate/X509CertificateAuthenticator.java#L613
[4] - https://github.com/wso2/carbon-kernel/blob/d74fce5177c60200211c064d732e04b942546612/core/org.wso2.carbon.user.core/src/main/java/org/wso2/carbon/user/core/util/UserCoreUtil.java#L565
[5] - https://github.com/wso2-extensions/identity-outbound-auth-x509/pull/83


Deploy Process Mapper to Unity-Test shared services account

Deploy Process Mapper to the Unity-Test shared services account.
Acceptance Criteria:

[x] Process Mapper deployed to Unity-Test shared service account
[x] DAPA API routes installed in Unity-Test API Gateway and linked to the underlying service through VPC Link and Load Balancer
[x] Test API with calls to SDAP time series

The steps to link from the API in the API Gateway to the underlying backend process mapper service are detailed in the acceptance criteria in the corresponding ticket for deployment to the Unity-Dev venue.  For details, see this ticket: https://github.com/orgs/unity-sds/projects/3/views/9?filterQuery=U-AS&pane=issue&itemId=50940434

Terraform script works for both Unity-Dev and Unity-Test environments.  Working on testing with an SDAP time series call.

Status Update

Fully automated except for working on an issue with the VPC Link configuration in API Gateway (VPC Link connects to the network load balancer)

Status:

Diagnosing issue causing Network Load Balancer to fail AWS health checks.  This is possibly caused by IAM role being used.  Asked U-CS (@galenatjpl ) to create Unity-CS_Service_Role in Unity-Test environment so we can use that one instead.

Fully automated (Terraform) deployment to Unity-Test completed, including API Gateway.
Terraform scripts at: https://github.com/unity-sds/unity-analytics-dapa/tree/devel/terraform.
Tested with SDAP time series call: curl -vvvvv --location 'https://58nbcawrvb.execute-api.us-west-2.amazonaws.com/beta/am-uds-dapa/collections/JPL_MUR25-JPL-L4-GLOB-v04.2/processes/time-series:area?bbox=-160,-15,-130,15&datetime=2022-01-01T00:00:00Z/2022-01-31T23:59:59Z' --header 'Authorization: Bearer YOUR_TOKEN'


Add workflow reading csv, cleaning, saving to parquet

This workflow reads the GDELT project csv (actually sep="\t") dataset which is on S3. The plan is to do some cleaning and save the transformed data to parquet.

Closing to base the PR off of the branch in #724


ui-router and jquery-mobile

Hello
I use angualar with ui-router on some web and it works very fine.
New I develop a mobile application and I have a problem with ui-router usage.
I have the following apps.js (part):
        $stateProvider
            .state('main', {
                url: '/',
                views: {
                    "" : {
                        templateUrl: "views/main.html"
                    },
                    "header@main" : {
                        templateUrl: "components/header.html"
                    },
                     "body@main" : {
                        templateUrl: "components/body.html"
                    }
                }
            })
            .state('main.menu', {
                url: '/menu',
                views: {
                    "body@main" : {
                        templateUrl : "components/menu.html"
                    }
                }
            })

Access to the site is http://monsite.com and the main.html is displayed with no problem (header and body are displayed).
My header page is very simple:

The problem is when I click the selector icon above, I have the message
Uncaught SecurityError: Failed to execute 'replaceState' on 'History': A history state object with URL 'http://menu/' cannot be created in a document with origin 'http://monsite.com'.
I use jquery 2.14, jquery-mobile 1.4.1, angular-js 1.3.15 and angular-ui-router v0.2.13-dev-2015-04-23 (in this declaration order)
Is there something wrong ? ui-router has really no problems with my others sites (but without jquery-mobile...).
Many thanks

Sorry the error was I have not coded
$locationProvider.html5Mode(true);

This can be closed then right? If not let me know.


doc: Clarify the limits of dumping/backfilling via OpenMetrics

This is about native histograms (not yet supported) and staleness markers (for which OpenMetrics support isn't even planned).
Also see #8281.
@darshanime @machine424 as discussed in the meeting today. (All the confusion was coming from the fact that I accidentally based my doc branch on a much older version.)

@darshanime @machine424 could you have a quick look? We discussed this already. It should be easy.

apparently without stale markers, increase() e.g. can return wrong (negative) values: https://github.com/prometheus/prometheus/issues/13747

That's weird. IIRC increase and friends are explicitly ignoring staleness markers. Maybe I'm missing something here, but I guess there is something else going on there causing this.


Add Extension:Scribunto

This extension provides Lua support in MediaWiki, but thus also requires Lua to be installed on the host machine.  It was already installed in my VM, but we should have Meza1 also install Lua if not already present on the machine.

Created issue #129 to track installing Lua interpreter separate from this.

Do you have a specific use case for scribunto? Last time we looked into it
(a long time ago) the only advantage was performance (lua modules are
faster than wikitext templates), at the cost of lua modules being less
intuitive than wikitext. Perhaps that has changed. Performance wasn't a
huge issue to us at the time, but it will increasingly be more important.
I don't think I'm going to mark this for a pre-v1.0 milestone (e.g. I won't
prioritize working on it), but by all means if you implement it I'll merge
it.
On Aug 5, 2015 6:19 PM, "Randy Eckman"<EMAIL_ADDRESS>wrote:

Created issue #129
https://github.com/enterprisemediawiki/Meza1/issues/129 to track
installing Lua interpreter separate from this.
—
Reply to this email directly or view it on GitHub
https://github.com/enterprisemediawiki/Meza1/issues/128#issuecomment-128181803
.

https://github.com/FodWikis/FodWikis/issues/16 mentions using Scribunto to unify banner implementations across all wikis.


Maximum number of teams that user can join need to be added.

Maximum number of teams that user can join need to be added. For Office365 group, this upper limit is 1000.
Is this the same for teams?
Office 365 group: https://docs.microsoft.com/en-us/office365/admin/create-groups/office-365-groups?view=o365-worldwide

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: 905e2c09-ef3e-07cc-0809-6ff171157c47
Version Independent ID: c0d33162-183d-fff8-32e9-a999f391ea51
Content: Microsoft Teams の制限事項と仕様
Content Source: Teams/limits-specifications-teams.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@moegitree Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@OfficeGlobal @samanthaw

@moegitree Thank you for reaching out, we're investigating this.

@moegitree Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Add Grafana-managed LLM as an option

Adding Grafana-managed OpenAI as an option.
This is where the user has access to OpenAI via an account Grafana manages. This involves all requests being proxied via the llm-gateway, which implements rate limiting and adds the Grafana OpenAI key to requests.
A key requirement is that this option must have an explicit Opt-In action performed by the user before it is enabled. The history of the opt-in actions will (later) be saved for future reference.
Changes to the backend:

extend Settings to include LLMGateway struct, which contains the URL to the llm-gateway (with fallback to env var), and an opt-in boolean. Both should be supplied by GCom.
like in machine learning, also get a Token from GCom that contains the tenant_id and api key for the llm-gateway. Use this info to authenticate with the llm-gateway.
grafanaOpenAIProxy is added which proxies requests to the llm-gateway.

Changes to the frontend:

a redesign of the OpenAI settings page to include the Grafana-managed OpenAI as an option, along with User-supplied key option plus an additional Disabled option. Visuals here.

Note: this PR does not include any logic to push updated settings to GCom.

CI passed but I think we've run out of github api requests, so drone cannot update the state here


Explorer: readonly files should not report drag and drop feedback for "move"

Noticed how readonly files indicate that move is possible, but I would have only expected drop feedback when doing copy.

PR: https://github.com/microsoft/vscode/pull/184301

@bpasero What are some verification steps for this issue?
On Windows, I made a file read-only, and I was able to drag and drop it without any issues.

@rzhao271 open the vscode repo and try to drag and drop a file of the node_modules folder.


Backport PR #2982 on branch 1.2.x (test: Fix hugging face create repo test which was failing)

Backport PR #2982: test: Fix hugging face create repo test which was failing

the private test (HF connection) did not pass for this backport due to AWS secret user restrictions


doc: add draft unified urn format rfc

Can we have an explicit definition of what is resource in ODPF means?


Support Kubernetes 1.18 (add conntrack binary)

Current situation
Flatcar currently does not support running Kubernetes 1.18. Main blocker is a missing user space tool, conntrack for managing netfilter connection tracking (nf_conntrack_* family of kernel modules).
Impact
Kubernetes 1.18, released March 31st 2020, fails to install on Flatcar (via kubeadm)
Ideal future situation
Flatcar fully supports Kubernetes 1.18.
Implementation options
Add the conntrack binary (and possibly other required tools).
Additional information
This was originally raised by Azeem on our Flatcar-Users mailing list as well as in bug #77 .

The original bug was about running kubeadm on Flatcar: it now requires the conntrack binary because the kubelet requires it and kubeadm runs the kubelet directly as a systemd service.
Distros that, for example, use the hyperkube container like Lokomotive or Typhoon include the conntrack binary in the hyperkube image so k8s 1.18 works there.

Flatcar Container Linux aims to be a minimal, container-centric distribution. Therefore, each addition to the distribution is put under adequate scrutiny to maintain a small OS image size as well as to minimise attack surface.

CON: We expect the conntrack binary to increase the OS image by a few hundred kilobytes (exact data pending).
PRO: We do not expect the attack surface to increase as conntrack uses kernel module APIs already present on Flatcar (nf_conntrack_* family of kmods).
PRO: adding conntrack will add support for installing Kubernetes 1.18 via kubeadm and will work towards compatibility with Cluster API.

Following the discussion above we will go ahead and aim to include conntrack with an upcoming Alpha release (then stabilise via Beta and Stable).

Adding conntrack would also benefit the kuberntes/kops project.
At the moment, Kops is adding adding this as a static binary, which is not so simple and would become even more complicated for ARM64 support. So, this is great news.
@justinsb


Vulnerability roundup 84: re2c-1.2.1: 1 advisory

search, files

[ ] CVE-2018-21232 CVSSv3=5.5 (nixos-20.03)

Scanned versions: nixos-20.03: 82b5f87fcc7. May contain false positives.

20.03 is Frozen and EOL.
As such, issue is closed.


Any way to easily name the socket files?

Your proposal
For our project we use a web process and a worker process (multiple of each in production, but as I'm in this repo, obviously only thinking development). With the current mechanism (as I understand it), each gets a socket created that is named after the pid of the running process, with no context of what process type it is and therefore in VS Code when it lists the sockets to connect to a) the most detailed part doesn't fit widthwise and is trimmed, and b) it then adds another step to go and look up the PID of the process to connect to. It would be nice if there was a way of naming the socket files and/or choosing their path so that users can recognise them more easily in VS Code.
Additional context
I tried running:
RUBY_DEBUG_OPEN=true RUBY_DEBUG_SOCK_PATH=./tmp/sockets/fc-worker-dbg.sock bundle exec sidekiq -t 10 -C config/sidekiq.yml

But the command that VS Code's extension uses to list the sockets doesn't find them:
rdbg --util=list-socks

Maybe this is currently achievable and I'm just missing how to configure it, but if that's the case I'm happy to raise a PR for the docs once someone educates me :-)

Could you add same envval RUBY_DEBUG_SOCK_PATH for VSCode?

I don't think this is necessary with #1017, it solves the same problem (being able to name the socket in some way recognisably). Combined with https://github.com/ruby/vscode-rdbg/pull/349 this is a fuller worker solution for us.
I'm happy for this #1016 to be closed.


audio normalization value changed (inference process problem)

I inferred with pretrained model but the result except lj dataset had low-pitch.
In mel2samp.py
def get_mel(self, audio):
        #audio_norm = audio / MAX_WAV_VALUE
        print(abs(audio).max().item())

So, first I checked the audio was correctly int16 sampled with abs(audio).max() item's max value is 32768.0 min 792.0. After divided by 32768.0, the normalization process, audio's range is not exceed 1.
but after infer process in inference.py -
audio = waveglow.infer(mel, sigma=sigma)
           if denoiser_strength > 0:
               audio = denoiser(audio, denoiser_strength)

           print(abs(audio).max().item())
           #audio = audio*abs(audio).max()
           #audio = audio * 32768.0

when I print nearly half of abs(audio).max() exceeds 1. After appending the max value to list, and check for their min max value... It turns out that min 0.484375 max 2.12109375
I think it's the cause of problem I faced with because when I changed audio = audio * 32768.0 / 2, The result wav pitch is normal but with little noise...(female voice more clearer, but male voice quiter) I wonder that 1.23 value can't be fixed. It is variable according to the dataset, I think.
Can I change this in config.json or fix this?
Reference :
https://github.com/NVIDIA/waveglow/issues/5#issuecomment-437592753

@rafaelvalle

Closing due to inactivity.


Inform the user if permission has already been granted

As of now, when a user chooses to obtain authentication in the main menu, an authentication URL is generated regardless of whether there already is a stored token and whether that token is still valid. The program should first check to see whether a token is stored and whether it is still valid before going ahead and generate a new authentication URL.

Done.


Method to list PeakGroups for a Study

FEATURE DESCRIPTION
Feature Inspiration
Getting a list of PeakGroups for a given Study will provide the researchers with a sensible way to start previewing the data and will let us explore methods of generating queries across multiple tables.
Feature Description
A method of the Study class that provides a list of PeakGroups suitable for consumption by a template.
Alternatives Considered
It would be useful to have reusable methods to generate querysets like this that relate things across multiple tables. Hopefully, in the process of developing this feature we'll discover methods of providing search results that we can use for later, more sophisticated search implementations.
Comment
This should perhaps be combined with #22

ISSUE OWNER SECTION
Assumptions

The search results template will be able to be combined later with a search query template as sub-templates using a master search template (see design & limitations below for context)

Requirements

Search results page with data mimicking the sample spreadsheet
Link from study detail page that shows the results relevant to the study displayed in the study detail page

Limitations

Only the fields currently existing in the database will be in the results (and not the collection of columns currently represented in the sample spreadsheet.
No pagination will be implemented at this time.  Currently, only a function to retrieve the query sets and a template to only show the consolidated results.
Field values and overall page appearance will not be customized to look pretty.  That can be done in a separate issue.
The format of the search criteria definition that is passed to the search function may change with advanced search to support more complexity (or's, and's, parentheticals, etc)

Affected/Changed Components

DataRepo/views.py
DataRepo/urls.py
templates/search_results.html
templates/study_detail.html

DESIGN
GUI Change description
There will be a button or link on the study detail page to view all related tissue data.  The resulting page will dsiplay a table that mimics the table in the sample provided by Caroline.
Code Change Description (Pseudocode optional)

Create a search function in views.py

inputs

list of search criteria (where each item indicates table, field, type of comparison (e.g. greater than), and value/term
A format parameter (that indicates the allotment of columns to display in the results)

outputs

render(request, "search_results.html", {"results": search_results, "columns": column_info})

The search results template will loop through the study table records and contain nested loops for any related table records
Since the study detail view is not yet merged from Fan's branch, I will create a very simple placeholder version
A search results view will be referenced in DataRepo/urls.py, similar to other urls defined there

Tests

Test a search results object is the same given a study ID exact search term

@lparsons @jcmatese @fkang-pu I just filled out the issue owner section.  Essentially it's the same as what we discussed in this morning's meeting, but if you would like to review the details, let me know if you have any issues before I get too deep into it.

This looks good to me. As I'm working on the computed values, I'm realizing that it might work if the results are, at their heart, a list of PeakGroups, perhaps with some prefetch relations. That way we could retrieve the computed values for each PeakGroup and the related fields in the template.

Are you saying that all the columns in the sample tissue file are from the PeakGroup table?

Are you saying that all the columns in the sample tissue file are from the PeakGroup table?

For sure not. There are a lot of related fields, thus the need to prefetch some of the related things. I guess I'm just saying that it would be very handy if in the template has an instance of the PeakGroup handy.

It there are PeakGroup fields in the sample sheet, those objects will be there.  If there aren't, it would be easy to add when the computed values are ready.

I believe the spreadsheet has one row per PeakGroup.

Well either way, the strategy should easily support adding data via objects if they aren't there on my first pass.  Sounds like they will be, but obviously I'm focussing on function the function as opposed to the particular data. And it sounds like, if I understand you correctly, adding the computed values will be very easy.

@hepcat72 Looks good to me, except I have a couple of comments:

in my "fkt2" branch, I have home.html, base*.html in templates/, but created templates for each model in /templates/DataRepo/ based on Django's default location and name convention for CBV.
example: templates/DataRepo/compound_list.html
for search_results.html, I would prefer a more specific name, just in case we need to add other templates for database search later on (e.g. search_peakdata_per_study.html or search_peakdata.html)

@hepcat72 Looks good to me, except I have a couple of comments:

in my "fkt2" branch, I have home.html, base*.html in templates/, but created templates for each model in /templates/DataRepo/ based on Django's default location and name convention for CBV.
example: templates/DataRepo/compound_list.html

Yes.  I wasn't planning on establishing locations for templates.  In fact, I'm pretty much copying the study template from your branch just to have something to navigate, although, I wasn't going to fully integrate it.  It will just be a placeholder.
Although, I did wonder about where to put the search results template.  That too can be easily changed though.  I just need something to look at.  We can move it to wherever.  My focus is only on the functionality.  I expect that how it looks and where the files go will definitely change.

for search_results.html, I would prefer a more specific name, just in case we need to add other templates for database search later on (e.g. search_peakdata_per_study.html or search_peakdata.html)

I did think about a different name, perhaps based on the format, although I might craft the search results template to be reusable for multiple search types.  So the name could be based on the structure of the resulting table, but in keeping with the core search functionality issue, it could handle multiple formats - and I just didn't decide which way to go yet.  For now, I'm going to keep it simple, but overall, I'm planning to apply what I learned in the generic template work.

After just performing some basic setup, I think I'll establish a different search results template for each results format.  I think that makes editing easier.  Personally, I prefer the generic/dynamic path, but  the specific/static one might develop faster.  I'll keep the interfaces the same though so that it would be easier to transition to a generic/dynamic strategy as things get bigger.

I went through the columns in Caroline's file and traced the way I'll need to access all the data, given a study record.  At the bottom, I worked out the select_related() arguments I would need to get all the nested objects.  I'm not sure how to obtain seruminfusate.  I'm skeptical it can be done in 1 query, so I'm going to focus on everything else first and tackle it after I get everything else worked out.
Here are my notes, with the queryset call at the bottom.  I'll then access the data for each column in a series of nested loops in the template to render the table (note, the access in the PRM doesn't use the id fields and I may need to add some related_names to the model relation definitions):
## Fields from Caroline's spreadsheet:

### Compound
	PeakGroup.name
	Study:Animal:Sample:MSRun:PeakGroup
	Study.id -> Animal.{study_id -> id} -> Sample.{animal_id -> id} -> MSRun.{sample_id -> id} -> PeakGroup.{ms_run_id -> PeakGroup.name}
### C_Label
	Animal.tracer_labeled_atom + Animal.tracer_labeled_count
	Study:Animal
	Study.animals -> Animal.{tracer_labeled_atom + tracer_labeled_count}
### Sample
	Sample.name
	Study:Animal:Sample
	Study.id -> Animal.{study_id -> sample_id} -> Sample.{id -> name}
### fraction #"normalized"
	PeakData.fraction() #cached_property
		self.corrected_abundance / self.peak_group.total_abundance
	Study:Animal:Sample:MSRun:PeakGroup:PeakData
	Study.id -> Animal.{study_id -> id} -> Sample.{animal_id -> id} -> MSRun.{sample_id -> id} -> PeakGroup.{ms_run_id -> PeakGroup.id} -> PeakData.{peak_group_id -> PeakData.fraction}
### tissue
	Tissue.name
	Study:Animal:Sample:Tissue
	Study.id -> Animal.{study_id -> id} -> Sample.{animal_id -> tissue_id} -> Tissue.{id -> name}
### mouse
	Animal.name #They included ID in the example table, but Lance said it should be name
	Study:Animal
	Study.id -> Animal.{study_id -> name}
### tic #"corrected"
	PeakGroup.total_abundance() #cached_property
	Study:Animal:Sample:MSRun:PeakGroup
	Study.id -> Animal.{study_id -> id} -> Sample.{animal_id -> id} -> MSRun.{sample_id -> id} -> PeakGroup.{ms_run_id -> total_abundance}
### seruminfusate
	PeakGroup.enrichment_fraction() #DOES NOT EXIST AS A CACHED PROPERTY.  IT WOULD HAVE TO BE CALCULATED ON THE FLY.  IT IS CALCULATED ON A SELECTION OF RECORDS ASSOCIATED WITH SERUM SAMPLES
		PeakGroup.objects.filter(ms_run__sample_id=final_serum_sample.id)
		    .filter(compounds__id=self.ms_run.sample.animal.tracer_compound.id)
		    .get().enrichment_fraction
	Study:Animal:Sample:MSRun:PeakGroup
	Study.id -> Animal.{study_id -> id} -> Sample.{animal_id -> id} -> MSRun.{sample_id -> id} -> PeakGroup.{ms_run_id -> enrichment_fraction()}
		Cannot use this straight.  Need to filter PeakGroups on Study, grab all the unique compounds,
### normfraction
	PeakGroup.normalized_labeling() #cached_property
	Study:Animal:Sample:MSRun:PeakGroup
	Study.id -> Animal.{study_id -> id} -> Sample.{animal_id -> id} -> MSRun.{sample_id -> id} -> PeakGroup.{ms_run_id -> normalized_labeling()}
### infusate
	Compound.name (via Animal.tracer_compound_id)
	Study:Animal:Compound
	Study.id -> Animal.{study_id -> tracer_compound_id} -> Compound.{id -> name}
### experiment
	Study.name
	Study
	Study.{id -> name}
### state
	Animal.feeding_status
	Study:Animal
	Study.id -> Animal.{study_id -> feeding_status}
### infusionrate
	Animal.tracer_infusion_rate
	Study:Animal
	Study.id -> Animal.{study_id -> tracer_infusion_rate}
### infusateconc
	Animal.tracer_infuser_concentration
	Study:Animal
	Study.id -> Animal.{study_id -> tracer_infuser_concentration}

So the select_related things I need are:

	Study
		Animal:Compound
		Animal:Sample:Tissue
		Animal:Sample:MSRun:PeakGroup:PeakData

	This should be able to be accomplished with:

		study_obj.objects.select_related('animal__compound','animal__sample__tissue','animal__sample__msrun__peakgroup__peakdata')

Dupe of #22


[CORE] Bug: Select parent as himself

Describe your problem
When the user is editing the page, the user can select the same page that is editing as a parent.
Expected behavior
The page does not need to appear on the list as a parent.
Steps to reproduce the problem
Manage > Content > Edit a page and select it as a parent in Option tab.
Bludit version
v3.0.0 rc-3

Hellow,
I used the HTTP_REFERER global to solve this issue (within a plugin), but this global may didn't get send by the browser or just lies. So I guess the best solution would be to just pass the current slug within the AJAX request like on this possible way (It also pass the current slug to the buildParentPages function itself to prevent unnecessary loops).
Happy new Year. :tada:
Sincerely,
Sam.


cmake: fix guessing where 'conf' is for other toolchains

When building with other toolchains (like ISSM), default to be able to
find 'conf' if ZEPHYR_SDK_INSTALL_DIR is available (even if we are
building with the other toolchain).
Signed-off-by: Inaky Perez-Gonzalez<EMAIL_ADDRESS>

Closing in favor of #4854.


slack-19.0: fix structured logging

Description
This PR fixes structure logging in v19 (introduced in https://github.com/slackhq/vitess/pull/449) that is not being enabled due to a bug, although we set --structured-logging on daemons
Before:
tim@Tims-MacBook-Pro vitess % git checkout slack-19.0
Already on 'slack-19.0'
Your branch is up to date with 'origin/slack-19.0'.
tim@Tims-MacBook-Pro vitess % make NOVTADMINBUILD=1 build
Sat Oct 12 03:33:56 CEST 2024: Building source tree
tim@Tims-MacBook-Pro vitess % ./bin/vttablet --tablet-path test-123456789 --structured-logging 2>&1 | head -3
W1012 19:22:23.163333   37461 log.go:39] Failed to read in config : Config File "vtconfig" Not Found in "[/Users/tim/github/slackhq/vitess]". This is optional, and can be ignored if you are not using config files. For a detailed explanation, see https://github.com/vitessio/vitess/blob/main/doc/viper/viper.md#config-files.
W1012 19:22:23.190138   37461 http.go:57] Beginning in v20, pprof-http will default to `false`; to continue enabling pprof endpoints, please manually set this flag before upgrading.
Error: mycnf read failed: open /vt/vt_0123456789/my.cnf: no such file or directory

After w/this PR:
tim@Tims-MacBook-Pro vitess % make NOVTADMINBUILD=1 build
Sat Oct 12 19:20:14 CEST 2024: Building source tree
tim@Tims-MacBook-Pro vitess % ./bin/vttablet --tablet-path test-123456789 --structured-logging 2>&1 | head -3
W1012 19:21:23.195690   37283 log.go:39] Failed to read in config : Config File "vtconfig" Not Found in "[/Users/tim/github/slackhq/vitess]". This is optional, and can be ignored if you are not using config files. For a detailed explanation, see https://github.com/vitessio/vitess/blob/main/doc/viper/viper.md#config-files.
{"level":"info","ts":"2024-10-12T19:21:23+02:00","caller":"servenv/servenv_unix.go:66","msg":"Version: 19.0.5 (Git revision 6ba49614d7e25fbbe4fd649113a01fe7b5b13810 branch 'fix-structured-logging_slack-19.0') built on Sat Oct 12 19:20:14 CEST 2024 by<EMAIL_ADDRESS>using go1.22.8 darwin/amd64"}
{"level":"warn","ts":"2024-10-12T19:21:23+02:00","caller":"servenv/http.go:57","msg":"Beginning in v20, pprof-http will default to `false`; to continue enabling pprof endpoints, please manually set this flag before upgrading."}

Related Issue(s)

https://github.com/slackhq/vitess/pull/449

Checklist

[x] "Backport to:" labels have been added if this change should be back-ported
[x] Tests were added or are not required
[x] Did the new or modified tests pass consistently locally and on the CI
[x] Documentation was added or is not required

Deployment Notes

Confirmed this on tablets ✅:
tvaillancourt@tablet-iad-staging-vtfun-00-80-5m9z:/var/log/vttablet$ tail vttablet.log|tail -1 |jq .
{
  "level": "warn",
  "ts": "2024-10-16T08:24:27-07:00",
  "caller": "discovery/healthcheck.go:557",
  "msg": "not marking healthy primary us_east_1e-0170333722 as Up for loadtest/80- because its PrimaryTermStartTime is smaller than the highest known timestamp from previous PRIMARYs us_east_1b-0170353516:<PHONE_NUMBER> <<PHONE_NUMBER> "
}


Question: How to check if user is fan of my page?

How could I check That signed in user is acctual fan of my webpage?

Try this.
https://gist.github.com/1172561


kubelet/container: Replace DockerCache with RuntimeCache.

RuntimeCache is a generate container cache, which only requires the underlying container runtime implementation to implement GetPods(bool)[]*kubecontainer.Pod.

LGTM


[SA-18359] Return normal AttribMaker if the attrib is deprecated but required

peerInterface in Appliance
displayName in Entitlements

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

you can add a test in test_entities for example. Just change one of the entities we have in the tests or create  a new one that has both options set and check that required wins always.


User Feedback Modal

The "X" cancel button doesn't work when a user has selected 'Read Feedback' from a listing rejected by an Org Steward.  The only option is to select 'Edit this listing', to exit.

@tdressel Is this still an issue?

Go to Listing Management
State | Needs Action
Choose Drop down in top right corner of app tile
Read Feedback
RESULTS - Feed Back pop up displays.
Edit this Listing button opened Edit Listing Form as expected.
Repeat 1-4
Choose X to cancel out of FeedBack pop up.
RESULTS - Exits out of Feedback pop up display.

Passed on Staging.


Lint error when building assets

When running gulp build or grunt build, trailing commas are added to config/assets/production.js resulting in:
...
Running "eslint:target" (eslint) task

/Directory/config/assets/production.js
  11:65  error  Unexpected trailing comma  comma-dangle
  28:60  error  Unexpected trailing comma  comma-dangle

✖ 2 problems (2 errors, 0 warnings)

Warning: Task "eslint:target" failed. Use --force to continue.

Edit: Only gulp build adds those commas, grunt buildworks fine.

Its because grunt build just use the default assets configurations and doesn't create one based on bower.json (No wiredep task. Porbably a bug :) ).
I have worked on removing trailing comma using gulp-trailing-comma. But the npm package is not up-to-date friskfly/gulp-trailing-comma#2. I will create a PR once it is updated.

Is there still work that needs to be done here?

I have created a pr for it. since there is no update on friskfly/gulp-trailing-comma#2 , i have created a package and plugins for gulp and grunt for removing trailing commas.

@lirantal can you take a look at the PR?

Re-opening this issue as it is still relevant, waiting for @shanavas786 PR.

I have started #1412 .
@lirantal Please have a look.

@shanavas786 @lirantal Moving to backlog since there hasn't been a successful PR that solves this issue.


Questions about docker-compose

Hi @udosson When you have a moment, could you let me know:

Can we move the application/ folder from utility-emissions-channel/docker-compose-setup/ to utility-emissions-channel/ so it is parallel to chaincode/ and docker-compose-setup?  It seems the rest api should work with any chain code set up.

Is there an explorer down with docker compose or how do we stop the explorer?

Does deployCC deploy updated version of chain code?  Or do we need to add a version parameter?

Hi @opentaps

yes. already done
explorer down was included in ./network.sh down. I also added a function to only shut down the explorer: ./network.sh stopBlockchainExplorer
deployCC requires the version of the chaincode as well as the sequence as a flag. I updated the readme file with steps to follow. It'd be similar to ./network.sh deployCC -ccv 2.0 -ccs 2

What is the difference between utility-emissions-channel/application and utility-emissions-channel/docker-compose-setup/application/
Thanks!
What is the -ccs flag for?

There wasn't any different. I removed the duplication at utility-emissions-channel/docker-compose-setup/application/
:)
-ccs is the chaincode sequence. It's a counter of the sequence of the chaincode deployment. It must increment by one each new version of the chaincode. Where -ccv, the chaincode version, could be of any format like 2 or 2.0.1, etc.


Dominant Piranha

Recent codeforces question  and solution in cpp

@Vishruth-S please review my pull request , I have added the recent Codeforces contest question with explanation and my solution.


[BUG] Selenium error

Hello, I have this error:
Error Log
Traceback (most recent call last):
  File "ms_rewards_farmer.py", line 771, in <module>
    browser = browserSetup(True, PC_USER_AGENT)
  File "ms_rewards_farmer.py", line 35, in browserSetup
    chrome_browser_obj = webdriver.Chrome(options=options)
  File "/home/flavien/.local/lib/python3.8/site-packages/selenium/webdriver/chrome/webdriver.py", line 73, in __init__
    self.service.start()
  File "/home/flavien/.local/lib/python3.8/site-packages/selenium/webdriver/common/service.py", line 98, in start
    self.assert_process_still_running()
  File "/home/flavien/.local/lib/python3.8/site-packages/selenium/webdriver/common/service.py", line 109, in assert_process_still_running
    raise WebDriverException(
selenium.common.exceptions.WebDriverException: Message: Service chromedriver unexpectedly exited. Status code was: 1

here you go: go to requirements.txt replace selenium with selenium==3.14.1 then do "pip uninstall -r requirements.txt" after everything uninstalls then do "pip install -r requirements.txt"

someone re created the bot https://github.com/MehdiRtal/MicrosoftRewardsBot.git

Fixed with v3.0 #188


[db-up] Fix issue #3301: az postgres up fails unpleasantly when database name contains dash

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

db-up

need to update release note


set forceRedirect to false in dev env

While the app is embedded in an iframe, we don’t have access to react and redux dev tools in Chrome.
This change sets the EASDK’s forceRedirect to false when the app is run in dev mode, which lets us load the app outside of Shopify’s iframe, thus giving us access to these tools.

This largely makes sense to me, the issues I see are around using the EmbeddedApp components, since they work via sending post messages to core afaik. I feel like this would work but your modals and things would be broken :/

I feel like this would work but your modals and things would be broken :/
yup :/
it’s definitely not ideal, but it’s a little better than nothing

Would the messages to Shopify (ie. for modals) not work at all or would they only break if you loaded the app in a random browser tab?

Either way, I think I'd be fine with shipping this since it improves the current developer experience in this app. But I would add a comment to the codebase to highlight this caveat of breaking post messages with a link to our docs perhaps.

Would the messages to Shopify (ie. for modals) not work at all or would they only break if you loaded the app in a random browser tab?

only if you load the app in a separate tab outside of the shopify chrome, which i think would have to be a pretty conscious choice.

only if you load the app in a separate tab outside of the shopify chrome, which i think would have to be a pretty conscious choice.

Sweet then we should ship this ASAP.

@AWaselnuk added a comment. if this looks good to you i'll 🚢

This looks super great. Please 🚢 !
I also want to take this opportunity to formally apologize for using the expression "ASAP". I don't know what I was thinking.


Fix Data-Correctness Bug in GTE Comparison in BinaryOperatorTransformFunction

fixes #9460
The BinaryOperatorTransformFunction::compare methods already call getIntResult, but the places where it's called calls getIntResult again.
https://github.com/apache/pinot/blob/master/pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/BinaryOperatorTransformFunction.java#L371
Say the comparison is: 10 >= 20
Then, the call chain is (bottom-up): Double.compare(10, 20) ==> -1 ==> getIntResult(-1) ==> 0 ==> getIntResult(0) ==> 1
Which means 10 >= 20 gets evaluated to true
cc: @yupeng9 @chenboat

Added a UT. Existing tests weren't able to catch this case since this only happens when the argument types are:

float, long
double, long
long, float
long, double

I also found a bug in my PR. I had only updated calls for one of the compare methods. Caught that bug via the UT I added 😓

Codecov Report

Merging #9461 (2bea20c) into master (96f4d1e) will decrease coverage by 2.74%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master    #9461      +/-   ##
============================================
- Coverage     69.88%   67.13%   -2.75%
+ Complexity     5127     4940     -187
============================================
  Files          1909     1413     -496
  Lines        101749    73833   -27916
  Branches      15440    11799    -3641
============================================
- Hits          71104    49570   -21534
+ Misses        25627    20667    -4960
+ Partials       5018     3596    -1422

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
67.13% <66.66%> (+0.01%)
:arrow_up:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...form/function/BinaryOperatorTransformFunction.java
45.36% <66.66%> (+5.49%)
:arrow_up:

...va/org/apache/pinot/core/routing/RoutingTable.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/apache/pinot/common/metrics/ControllerMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/BrokerQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/MinionQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ache/pinot/server/access/AccessControlFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/messages/SegmentReloadMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/messages/TableDeletionMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 742 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Campaign view hero icon hovers should give more hero details

For example, you'll get this hero, with these skills, stats, etc.

Do you mean this hero icon?

If so, I will try to add a tooltip with a shortened version of this info:

I think he means these tantalizing hero previews:


modify the  URL of windows inference library

modify the version and the web site

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


docs: Fix broken link to frontend installation

Summary of Changes
Fixes a broken link to https://www.amundsen.io/amundsen/frontend/docs/installation/ from https://www.amundsen.io/amundsen/developer_guide/.
The link is going to the local .md file, which gave 404 when clicked online: https://www.amundsen.io/frontend/docs/installation.md
Tests
Just a docs change
Documentation
Fixed link
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

You'll need to sign the commit @razzius, thanks!

Okay~~! Amended with signoff


Make UnresolvedType use unresolvedName

Right now, UnresolvedType holds a string for its name. We want it to use unresolvedName instead, so it parses the modules as well.
@kiljacken

Will take a look at it in approx 6 hours.
On Thu, Jul 16, 2015, 07:18 Liam<EMAIL_ADDRESS>wrote:

Right now, UnresolvedType holds a string for its name. We want it to use
unresolvedName instead, so it parses the modules as well.
@kiljacken https://github.com/kiljacken
—
Reply to this email directly or view it on GitHub
https://github.com/ark-lang/ark/issues/528.


Provide installation instructions

I didn't think I was a newbie but I am not able to install this plug-in.  This plug-in does not appear in the community plug-ins section in the app so I attempted to install it by dropping the source folder into the hidden plugins directory.  After doing this, I still was not able to find the plug anywhere in the plug ins listing.
If there is some step I am missing, perhaps an install guide would be helpful.  I read the readme and it did not offer any guidance.

Hello!  Currently its a beta. So you can only install it with BRAT

doh.  I have not used BRAT but it might help the next noob save some time and frustration by including that in the documentation.  If it's there I missed it. :-)

I will add a warning of BETA with a link to BRAT then. Hope Obsidian approves my petition of publish the plugin soon

sounds good.  Hope it's approved.  So I installed BRAT and enabled this plug in but it's not very obvious where to start.   Perhaps a youtube video going over how to use the plug in would be useful because using a canvas to drive the creation of notes is not something you see every day.  I'm eager to learn how to use it....

once you enable the plugin, you have a mini tutorial the first time you click on the plugin ribbon icon
I will work on a web with all the documentation

The plugin was already published !
The brat instructions will not be necessary

Congratulations!


The checkpoint generated by tensorflow2.0 training does not have a meta file. How should it be converted to a pb file?

When T1, the meta file is converted to a pb file, and then the pb file is converted to a tflite file. Now T2 does not generate a meta file, what should I do? Can anyone help me?

You can try TFLiteConverter.from_saved_model() in TF 2.
See https://www.tensorflow.org/lite/convert/python_api#converting_a_savedmodel_


exif extraction for dataset creation

Hi guys, thanks for sharing your code :)
I'm trying to train 'exif/image CLIP model' as it is not currently released.
The paper mentions that you use the YFC100M dataset and filter out images with less than 10 exif tags.
My question is with which software did you extract the exif information? Different commands return slightly different outputs.
Also, how should I format the exif info before feeding it to the encoder?
Thank you

No software is needed.  YFCC100M will provide you with a metadata file that contains the EXIF info (I remember it just called yfcc_exif). Don’t directly extract EXIF from the downloaded image, though, cuz Flickr sometimes remove the metadata of users’ photo manually.

Hi and thanks for the reply!
Any chance you could share this file? I can't seem to find it online, it doesn't seem to be included in the S3 bucket.
Also, do you format the exif string in any way? Many of the exif values use integers to represent specific attributes (such as 1=sRGB for Color Space) do you convert these to strings according to the specific attribute (i.e. convert 1 to sRGB), or leave them as is?

Hi,
One year ago when I download those data I use:
The metadata files (ver 1. + expansion pack) are accessible at:  s3://mmcommons
List files:s3cmd ls s3://mmcommons
Download files: s3cmd get --recursive s3://mmcommons
Not sure if these links are still valid. If they are invalid, let me know and I will try to find a way to share those big files to you.
About the EXIF information formatting, I don’t do any specific formatting. I think no matter how you format it, the entire EXIF string is still kind of out of distribution for the pretrained text encoder, so it should be good to let the network learn all the things about it. But I do spend some effort cleaning yfcc data, like filtering out those garbage values, since yfcc metadata is quite noisy and it will affect training greatly unless you train it in an extremely large scale.


Test tags with iOS

see https://github.com/participedia/usersnaps/issues/27

We seem to be hitting something like https://github.com/callemall/material-ui/issues/3638

This appears to have been fixed upstream in the material-ui library, will attempt to integrate those changes. Other multi-select inputs also appear to be broken on Safari mobile (related cases, related methods, related organizations), but that may be a different issue (if so, I will file it), since they don't appear to be triggering API calls.

This is still happening, and upgrading the underlying libs doesn't seem to fix it. We can make the CSS changes ourselves, but as of today's check-in meeting with Jesi and Amber we decided we can go ahead with user testing before it is fixed.


NP-317: Refactor to use enum and set dialect version to scala 3

LGTM. So the Test wasn't Scala3 yet? That's a surprise

Just the scala fmt was running in Scala 2.13, so there was a failure with something using Scala 3 dialect


Document using String.percent_encode() with OS.shell_open() (3.x)

3.x version of https://github.com/godotengine/godot/pull/74283.
See https://github.com/godotengine/godot/issues/74279.

Thanks!

Cherry-picked for 3.5.2.


engine: show all files that triggered a Tiltfile build

Now when you change a a config file that causes a manifest's definition to change, and thus be re-deployed, that file will show up as an edited file in both the Tiltfile and the manifest that changed.
Here's an example: I changed deploy/snack.yaml in Servantes and both snack and (Tiltfile) are marked as edited, but hypothesizer is not.

TODO: write a unit test (will do before I merge)

@nicks Now it's attached to the manifest state and consumed (reset) when those files are appended to the build record in handleBuildStarted


Adding Bitcoin Checks

Hey,
I am the founder for the Bitcoin Checks project. Please let me know if I have missed anything.
Cheers.

Added


Add layer.isLoaded

For #4091
Change List

Layer.isLoaded
CompositeLayer.isLoaded
Rename IconManager.loaded to IconManager.isLoaded
Tests
Documentation

Coverage increased (+0.003%) to 80.865% when pulling e58d24220d88cf7cb256999fb7e56814afc3f0fe on x/layer-is-loaded into 3b45d2387f3e0ff6152cfe03c618e5f161676d1e on master.