Can't pass array through attribute (to tag options)

Hello, guys.
I have list of items (image-list). Each item (image-item) have palette that should render as list of colors.
Have a problem with pass palette to image-item tag. I have one base array called items:
self.items = [
    { src: 'images/1.jpg', palette: [{ color: 'red', name: 'Red' }, { color: 'blue', name: 'Blue' }, { color: 'yellow', name: 'Yellow' }] },
    { src: 'images/1.jpg', palette: [{ color: 'red', name: 'Red' }, { color: 'blue', name: 'Blue' }, { color: 'yellow', name: 'Yellow' }] },
    { src: 'images/1.jpg', palette: [{ color: 'red', name: 'Red' }, { color: 'blue', name: 'Blue' }, { color: 'yellow', name: 'Yellow' }] }
  ];

Then I pass this array to image-list tag:
<image-list items={ items }></image-list>

image-list tag:
<image-list>
  <image-item each={ items } src={ src } palette={ palette }></image-item>

</image-list>

And image-item tag:
<image-item>

{ name }

</image-item>

You see that I added console.log to image-item tag. In console I have weird log:
t {riot-src: "images/1.jpg", palette: Array[3]}
t {riot-src: "images/1.jpg", palette: Array[3]}
t {riot-src: "images/1.jpg", palette: Array[3]}
t {src: "images/1.jpg"}
t {src: "images/1.jpg"}
t {src: "images/1.jpg"}

Looks like image-item called twice (why?) and when it called first time options have palette (but wrong riot-src). Finally I have options without palette array (why?).

What is the riot version? 2.0.15 was just released today, there were several fixes to extra calls, maybe fixed with this #620/ #624 or #473

Riot version is 2.0.15 (I use next link https://cdn.jsdelivr.net/g/riot@2.0.15(riot.min.js+compiler.min.js) for include it).

Here example of code: https://github.com/jmas/riot-issue-629-example

I have not yet tried your code, just looked at it and one thing I think is wrong is the mounting. If you try and change from:
(function() {

  riot.mount('image-item');
  riot.mount('image-list');
  riot.mount('app');

})();

to:
(function() {

  riot.mount('app');

})();

Prefect, thank you! But still can't understand why Riot convert src attribute to riot-src?

The reason is to prevent illegal server request for src="{src}" before tag expressions are evaluated.

@rsbondi Thank you for explain.
You, guys are very help me. I closed issue because all issues are solved.


Feature Migration

In this PR I did the following:

[ ] Initialize rails app
[ ] Create migrations for the app

Hi @yegonkimutai
Great job implementing the migration👍🏽
Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
STATUS: APPROVED 🟢
Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.


why is the quanti-aware training very slow?

why is the quanti-aware training very slow?(almost 2 times slower than float model training). Is there any way to speed up the quanti-aware training?

@brisker, in part it is so because every tensor that is quantized has an overhead of quantization simulation at every forward, and an overhead of calculating gradients for the trainable quantization parameters. This overhead is for each element of each quantized tensor, be it activations or weights. We already provide CUDA kernels for both forward and backward quantization computations so that the slowdown is alleviated somewhat:
https://github.com/openvinotoolkit/nncf_pytorch/tree/develop/nncf/extensions
You may try taking a look at the kernels and if you spot a lack of optimization somewhere, please let us know.


BIP32

Have a nice Swift interface to BIP32 functions in https://github.com/trezor/trezor-crypto/blob/master/bip32.h

Done with https://github.com/TrustWallet/trust-keystore/commit/98abb27bff5d04b144aa5e7633d137b502f5b490


Change solo control

Use shift + track keys to solo a track. This avoids entering solo on accident and will make the first step row available for other performance tasks.

To be released with 0.1.22.


[SPARK-35566][SS] Fix StateStoreRestoreExec output rows

What changes were proposed in this pull request?

This is a minor change to update how StateStoreRestoreExec computes its number of output rows. Previously we only count input rows, but the optionally restored rows are not counted in.
Why are the changes needed?

Currently the number of output rows of StateStoreRestoreExec only counts the each input row. But it actually outputs input rows + optional restored rows. We should provide correct number of output rows.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Existing tests.

cc @HeartSaVioR @cloud-fan

Looks good to me.

Test build #139087 has started for PR 32703 at commit a8135d8.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43608/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43608/

Test build #139087 has finished for PR 32703 at commit a8135d8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139087/

Merged to master, branch-3.1 and branch-3.0.

Thanks all!


Can’t get Opera Mini/Mobile to work…

Sorry too bother but no matter what combination I try, I can’t get Opera Mini in iOS 8 to work with the browser-specific-css plugin. Of the option below, the only one that does anything for me is the last one, but that only affects the desktop version of Opera (Mac).
.ios.operamobile .show-menu:after {
font-size: 1.625em;
content: "\25be";
}
.operamobile .show-menu:after {
font-size: 1.625em;
content: "\25be";
}
.ios.operamini .show-menu:after {
font-size: 1.625em;
content: "\25be";
}
.operamini .show-menu:after {
font-size: 1.625em;
content: "\25be";
}
.opera .show-menu:after {
font-size: 1.625em;
content: "\25be";
}

Any help would be greatly appreciated.

can you open this wesbite http://whatsmyuseragent.com/ on the device you are testing and post the user agent string you get here ?


What is the difference between 'DependencyContext' and 'basic-reference-assemblies'?

Hello.  I am using DependencyContext .Default .CompileLibraries to get references.  which namespace in Microsoft.Extensions.DependencyModel.
It works(However, there are some fields or attributes whose metadata cannot be found during dynamic compilation).
What is the difference between Microsoft.Extensions.DependencyModel and basic-reference-assemblies?

Can you link to any docs to that API? I'm unfamiliar with that so hard to say how it relates to this project.

@jaredpar  https://github.com/dotnet/runtime/tree/main/src/libraries/Microsoft.Extensions.DependencyModel
When I used dynamic compilation before, the roslyn error could not find the object reference. For dynamic compilation is a very troublesome thing, currently I use

foreach (var asm in DependencyContext
            .Default
            .CompileLibraries
            .SelectMany(cl => cl.ResolveReferencePaths()))
            {
                DomainManagement.Default.AddReferencesFromDllFile(asm);
            }

this code to solve the problem.

When I use the dll file provided by DependencyContext to reflect private fields, such as _size of List<int>. It will indicate that this field cannot be found. So I have to reload typeof (List<int>). Assenbly.Location into the reference to find _size;. I suspect DependencyContext can't provide metadata other than public level, or I'm not familiar with it.


Depend on Bijectors

Deletes utilities/transform.jl and depends on Bijectors.jl. Requires https://github.com/TuringLang/Bijectors.jl/pull/2 to be merged first, then the Bijectors.jl package should be registered. Closes https://github.com/TuringLang/Turing.jl/issues/540.

Registration PR for Bijectors:
https://github.com/JuliaLang/METADATA.jl/pull/18473
We can merge this PR once the registration is accepted.


[FLINK-23332] update gradle to 7.1

Also update the ' shadow' plugin accordingly.

After approving this, I also tried
$ gradle --no-build-cache test
Starting a Gradle Daemon (subsequent builds will be faster)

FAILURE: Build failed with an exception.

* Where:
Build file '/Users/david/stuff/flink-training/build.gradle' line: 32

* What went wrong:
A problem occurred evaluating root project 'flink-training'.
> Failed to apply plugin [class 'com.github.jengelman.gradle.plugins.shadow.ShadowBasePlugin']
   > This version of Shadow supports Gradle 7.0+ only. Please upgrade.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 14s

Not sure if I should be concerned.

You have to use the provided gradle wrapper script, not your system gradle version, otherwise you will see errors like this.
I have a commit in one of my branches which adds a few commands that will hopefully help, sounds like I should include this one or a note to use the gradle wrapper. (I'll create a pr after all these changes)

The problem I'm having testing this is that I don't know how to get ./gradlew to clear the cache, so I'm not succeeding in seeing this do very much work. I'm frustrated that ./gradlew clean doesn't seem to work.

I upgraded my system gradle 7.1.1, but the tests fail when I use it. They seem to pass when I use ./gradlew, but I'm not confident they are being run.


Remove WOODPECKER_DOCS config

I don't really see a reason why this should be configurable.
The docs link is mainly used to construct links to our docs so if you would change this, you'd need to have the same structure and pages as our docs have.

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

Comparison is base (f44aa8a) 34.10% compared to head (0751cba) 34.06%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2647      +/-   ##
==========================================
- Coverage   34.10%   34.06%   -0.04%
==========================================
  Files         205      205
  Lines       13126    13125       -1
==========================================
- Hits         4476     4471       -5
- Misses       8297     8301       +4
  Partials      353      353

Files
Coverage Δ

server/web/config.go
0.00% <ø> (ø)

... and 1 file with indirect coverage changes

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


looping "contact us" by creating a support request.

[Enter feedback here]

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

ID: 3f0af130-6b0a-0a7e-5fd9-e67810f66b45
Version Independent ID: 445b1c77-472a-e0f7-45fc-168f4a2454f3
Content: Self-service exchanges and refunds for Azure Reservations - Microsoft Cost Management
Content Source: articles/cost-management-billing/reservations/exchange-and-refund-azure-reservations.md
Service: cost-management-billing
Sub-service: reservations
GitHub Login: @bandersmsft
Microsoft Alias: banders

@a-c-b Hi there. Sorry about that issue. Try this:

In the top right corner of the Azure portal, select Support + Troubleshooting (a question mark symbol).
In the Support + troubleshooting window, in the Briefly describe the issue box, type a few words about the problem you need help with.
At the top of the page, select Create a support request.
On the New support page, complete all the required actions to create a new support request.

#please-close


Why does Pull Request Quantifier now show as deprecated on GitHub?

The name of the Pull Request Quantifier app in GitHub has become pull-request-quantifier-deprecated. I can't find any statement or explanation about the deprecation. Does anyone have context on this?

We don't offers the service, outside of Microsoft.


Unable to enter formulas into Dice Calbacks in the Statblock Layout

Check for existing bug reports before submitting.

[X] I searched for existing Bug Reports and found no similar reports.

Expected Behavior
I am trying to enter this formula return [{ text: "1d%!5>=96+" + monster.Stamina}]; into the Dice Callback section of the Property Line in the Statblock Layout. I should be able to add the formula.
Current behaviour
I am unable to type anything in the Dice Callback or the Callback section as those sections act like they are protected fields and do not allow me to enter anything.
Reproduction

Click on a layout for Statblock.
Then click on add Property line.
Click on the pencil symbol to edit the property line
Add the name of the monster property you want to add in the "Link Monster Property" box.
Scroll down to Dice Callbacks and try to enter anything in that box. It will not allow me to.
I clicked on Render as Markdown but that doesn't help.

Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[ ] macOS
[X] Windows

Obsidian Version Check
1.4.16
Plugin Version
3.16.1
Confirmation

[X] I have disabled all other plugins and the issue still persists.

Possible solution
No response

I can't recreate this. Are there any errors in console?

No errors pertaining to Dice Roller or Fantasy Statblocks. The errors I do
have are for missing links in Breadcrumbs and the map icons I added to
Leaflet (the icons do still work though).
Basically the block where you enter the formula for the dice callback will
not let me enter anything, like it's set to not allow entries.
On Tue, Dec 5, 2023 at 6:55 PM Jeremy Valentine @.***>
wrote:

I can't recreate this. Are there any errors in console?
—
Reply to this email directly, view it on GitHub
https://github.com/javalent/fantasy-statblocks/issues/314#issuecomment-1841898773,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADLKBO4OW6I6L2E4TXQC7HDYH67A5AVCNFSM6AAAAABADXWL6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBRHA4TQNZXGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Another thing is that the formulas in the Dice Callbacks I have from before show up at the bottom of the box instead of at the top (where they used to show up at).

Unfortunately, I still can't recreate this.


Add rust format

Could you add a readme on installing rustfmt with cargo and add it to the CI (see https://github.com/rust-lang-nursery/rustfmt#checking-style-on-a-ci-server) if you agree with #6

@Tristramg poke


Issue 843 new plans

related issue: https://github.com/EnMasseProject/enmasse/issues/843

Yes that's true, I should move this test into standard and rewrite it (little bit). I added this test into brokered because of time required for creating new address space. And original purpose of this test was verify that is possible to use dynamically created plan. I'll fix that.


[Hexagon] Remove uses of LLVM from simulator runtime

The TVM runtime is not linked with LLVM libraries, so using LLVM in it carries a risk of referencing undefined symbols. This may
work for objects defined in header files, but it then relies on LLVM keeping them there.
Replace uses of LLVM utilities in the Hexagon simulator runtime with simple alternatives.

Thanks @kparzysz-quic @jroesch and @tkonolige !


[Catalog] Client-oriented API

category of trees
support headcrumbs

@F1nZeR re-open if smth is wrong


All pretrained models not available

Hi,
Some of the models seem to be missing from the link shared, for instance pwcnet-lg-6-2-cyclic-chairsthingsmix. Can you please update?
Thanks for great work btw

Thanks for the kind words, @rajat95
The "cyclic" training schedule is here only for quick experimentation, should you want to test a quick modification of your own of the PWC-Net architecture. Ultimately, the only PWC-Net models you should be using are the ones trained using the "multisteps" schedule.
We've made available our multistep models, that are far superior to the models trained using the cyclic schedule. You can still train those models longer if you want to, or finetune them to your own dataset, using whatever training schedule you want but, at least, you will be starting with the best models we could train ourselves.
We are also generating two models (a small and a large PWC-Net) using mixed-precision training. These models (that use FP16 input images instead of FP32 images) will be available within two weeks.
Hope this clarifies everything.


run_model_bpa Running Forever and Not Completing for Certain Datasets

run_model_bpa is running from last many hours. Please check the scenarios as below.

Tried with both function run_model_bpa and run_model_bpa_bulk
published dadatset to diffrent workspace, same result
Dataset is very simple, It's size is 2 MB.  Large semantic model storage format is disabled. Sensitivity is internal. Total number of tables are 2 and few calculated tables.
I tried 7-8 times on different days same result.
It doesn't throw any error, just keeps running.
Same function in the same connection instance running fine.
I can download dataset, it looks fine.  I published it to different workspace with different name, but same issue.

Is the model processed? Can you run labs.get_model_calc_dependencies() against that semantic model? Can you connect using the Tom wrapper:
with connect_semantic_model(dataset='', workspace='') as tom:

From: Jai-Prakash-HU @.>
Sent: Tuesday, November 12, 2024 7:43:13 PM
To: microsoft/semantic-link-labs @.>
Cc: Subscribed @.***>
Subject: [microsoft/semantic-link-labs] run_model_bpa running too much time for few dataset. (Issue #274)
run_model_bpa is running from last many hours. Please check the scenarios as below.

Tried with both function run_model_bpa and run_model_bpa_bulk
published dadatset to diffrent workspace, same result
Dataset is very simple, It's size is 2 MB. Large semantic model storage format is disabled. Sensitivity is internal. Total number of tables are 2 and few calculated tables.
I tried 7-8 times on different days same result.
It doesn't throw any error, just keeps running.
Same function in the same connection instance running fine.
I can download dataset, it looks fine. I published it to different workspace with different name, but same issue.

image.png (view on web)https://github.com/user-attachments/assets/e18bcb42-577d-48be-ab48-85bd04da114d
image.png (view on web)https://github.com/user-attachments/assets/19d8f45e-f03e-4d72-a192-e4028e414402
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/semantic-link-labs/issues/274, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHBQBNWKSMYLYHBHVU64JYT2AI43DAVCNFSM6AAAAABRUTSHMKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGY2TEOBZG42DQNI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I think there is some problem with labs.get_model_calc_dependencies function itself.  It never returns anything and keeps running. Just to test I gave wrong workspace name, and it was still running.
is it like run_model_bpa is depended on get_model_calc_dependencies in some scenario? and in those scenarios run_model_bpa is running for forever?

Yes, run_model_bpa is dependent on get_model_calc_dependencies. Every time you call run_model_bpa, get_model_calc_dependencies is run in the background in order to satisfy specific rules. Can you run this DMV against that model in DAX studio (against the original model)?
SELECT * FROM $SYSTEM.DISCOVER_CALC_DEPENDENCY

Update: earlier it was running forever. Now it is throwing error.

SELECT * FROM $SYSTEM.DISCOVER_CALC_DEPENDENCY:  It is working fine. It gives result within seconds
labs.get_model_calc_dependencies(dataset = 'XXXXXXXXXXXXXXXXX',  workspace = 'XXXXXXXXXXXXXXXX') : Working fine It gives result within seconds
labs.run_model_bpa(dataset = 'XXXXXXXXXXXXXXXXXXXXX',  workspace = 'XXXXXXXXXXXXXXXXXXXX') : It is throwing error.

please check the error below:
816             "Naming Conventions",
817             ["Table", "Column", "Measure", "Partition", "Hierarchy"],
818             "Warning",
819             "Object names must not contain special characters",
820             lambda obj, tom: re.search(r"[\t\r\n]", obj.Name),
821             "Object names should not include tabs, line breaks, etc.",
822         ),
823     ],
824     columns=[
825         "Category",
826         "Scope",
827         "Severity",
828         "Rule Name",
829         "Expression",
830         "Description",
831         "URL",
832     ],
833 )
835 return rules
File ~/cluster-env/clonedenv/lib/python3.11/site-packages/sempy_labs/tom/_model.py:2281, in TOMWrapper.is_field_parameter(self, table_name)
2276 import Microsoft.AnalysisServices.Tabular as TOM
2278 t = self.model.Tables[table_name]
2280 return (
-> 2281     self.is_field_parameter(table_name=table_name)
2282     and t.Columns.Count == 4
2283     and any(
2284         "NAMEOF(" in p.Source.Expression.replace(" ", "") for p in t.Partitions
2285     )
2286     and all(
2287         "[Value" in c.SourceColumn
2288         for c in t.Columns
2289         if c.Type == TOM.ColumnType.Data
2290     )
2291     and any(
2292         ep.Name == "ParameterMetadata"
2293         for c in t.Columns
2294         for ep in c.ExtendedProperties
2295     )
2296 )
File ~/cluster-env/clonedenv/lib/python3.11/site-packages/sempy_labs/tom/_model.py:2281, in TOMWrapper.is_field_parameter(self, table_name)
2276 import Microsoft.AnalysisServices.Tabular as TOM
2278 t = self.model.Tables[table_name]
2280 return (
-> 2281     self.is_field_parameter(table_name=table_name)
2282     and t.Columns.Count == 4
2283     and any(
2284         "NAMEOF(" in p.Source.Expression.replace(" ", "") for p in t.Partitions
2285     )
2286     and all(
2287         "[Value" in c.SourceColumn
2288         for c in t.Columns
2289         if c.Type == TOM.ColumnType.Data
2290     )
2291     and any(
2292         ep.Name == "ParameterMetadata"
2293         for c in t.Columns
2294         for ep in c.ExtendedProperties
2295     )
2296 )
[... skipping similar frames: TOMWrapper.is_field_parameter at line 2281 (2967 times)]

File ~/cluster-env/clonedenv/lib/python3.11/site-packages/sempy_labs/tom/_model.py:2281, in TOMWrapper.is_field_parameter(self, table_name)
2276 import Microsoft.AnalysisServices.Tabular as TOM
2278 t = self.model.Tables[table_name]
2280 return (
-> 2281     self.is_field_parameter(table_name=table_name)
2282     and t.Columns.Count == 4
2283     and any(
2284         "NAMEOF(" in p.Source.Expression.replace(" ", "") for p in t.Partitions
2285     )
2286     and all(
2287         "[Value" in c.SourceColumn
2288         for c in t.Columns
2289         if c.Type == TOM.ColumnType.Data
2290     )
2291     and any(
2292         ep.Name == "ParameterMetadata"
2293         for c in t.Columns
2294         for ep in c.ExtendedProperties
2295     )
2296 )
RecursionError: maximum recursion depth exceeded

Ah, my mistake. I will fix this in a quick release. Thanks for raising the issue.

Fixed in 0.8.6.


Remove limitation for maintenance collateral ratio

As the new feature will be released next month, the limitation is not neccessary.
https://github.com/bitshares/bsips/blob/master/bsip-0030.md

Codecov Report

Merging #109 into develop will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #109      +/-   ##
===========================================
+ Coverage    59.36%   59.41%   +0.05%
===========================================
  Files           37       37
  Lines         3502     3499       -3
===========================================
  Hits          2079     2079
+ Misses        1423     1420       -3

Impacted Files
Coverage Δ

bitshares/dex.py
18.26% <ø> (+0.51%)
:arrow_up:

Continue to review full report at Codecov.

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


feat(just): add bazzite-arch shortcut

This effectively let's any downstream image "bolt-on" the bazzite gaming experience - awesome pattern!

2 minutes...

Any interest in exporting the applications it has automatically?


Support slightly different names for buttons and their dialogs: PRESENT for button, but PRESENTATION for dialog

In presentation.html the dialog's title can be changed to any translation string id


[REPO] Consider adding an "Issue Priority"

Is there an existing issue?

[X] I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

[X] This is not a support question.

Description of bug
The bug issues are not prioritized. This makes it difficult to focus on relevant issues - especially (but not only) for newcomers:
https://github.com/paritytech/substrate/issues?q=is%3Aopen+is%3Aissue+label%3AI3-bug+sort%3Aupdated-asc
Consider adding some kind of priority to the issues

the U2-some_time_soon and similar U-labels.

Steps to reproduce
Go through the bug issues, and try to identify the most important ones:
https://github.com/paritytech/substrate/issues?q=is%3Aopen+is%3Aissue+label%3AI3-bug+sort%3Aupdated-asc

Putting priorities on issues and having tasks for newcomers are two independent problems.
Newcomers can search for https://github.com/paritytech/substrate/labels/Z6-mentor issues.

?

not all newcomers need mentors
some newcomers can mentor the regulars (e.g in issue-tracking standard-procedures, refactoring, code-elegance etc. etc.)

As an example, I entered the go-algorand project, fixed bugs and reduced duplicated code - naturally without a mentor.
but! What is very difficult for a (High xp or not ) newcomer is to identify issue priorities.
In any way, if you (or this project) think that issue/bug priorities are not necessary, then stick to it and close this.
(though its really a must have)

Just noticed that it exists already, the U2-some_time_soon and similar U-labels. Updated text.

Yea there are priority labels. But they are not enforced and every team / dev uses them differently.
I am not in a position nor want to tell how other people should use labels.
We normally use the "mentor" label on free-standing issues that can be worked on asynchronously. Not necessarily that you need supervision.

Yea there are priority labels. But they are not enforced and every team / dev uses them differently.

This is standard-procedure, really not much space for things like "individuality" etc.

I am not in a position

Who is? (usually project lead or CTO)

We normally use the "mentor" label on free-standing issues that can be worked on asynchronously. Not necessarily that you need supervision.

You sure? This sounds very(!) confusing.
In any way, I start to get exhausted here, asking to get what should be already there.
To speed things up: Is there any chance to get priorities on the bug issues?

If bugs in general are high priority they are taken on directly,

Thus all open bugs have a low priority. Ok.

start requesting these things because you know them from other projects

I know them because they are standard-procedure.
IT pros usually do not discuss those "things".

issues with the mentor label for external people that don't require that much help from someone working on the project full time.

Mentor issues are for juniors and dummies.
Me neither of those.
Closing, as I'll loose my cool if I see one more time the word "mentor"...


[AUDIT][SPARK-41219] IntegralDivide use decimal(1, 0) to represent 0

Is your feature request related to a problem? Please describe.
Spark 3.5 changes the definition of decimal zero from DecimalType.bounded(intDig, 0) to DecimalType.bounded(if (intDig == 0) 1 else intDig, 0).
Describe the solution you'd like
We have the same code in org.apache.spark.sql.rapids.GpuIntegralDecimalDivide#resultDecimalType so presumably we need to update this for 3.5 for consistency.
Describe alternatives you've considered
Additional context
https://github.com/apache/spark/commit/a056f69438

We don't need to do this as we are not relying on resultDecimalType for calculating the return type. We directly call a kernel to do the division

We don't need to do this as we are not relying on resultDecimalType for calculating the return type. We directly call a kernel to do the division

So is GpuIntegralDecimalDivide.resultDecimalType even used? Should we remove it?

So is GpuIntegralDecimalDivide.resultDecimalType even used?

Just to clarify a bit resultDecimalType is called by CudfBinaryArithmetic.dataTypeInteral which becomes the dataType for the expression, except GpuIntegralDivide overrides dataType to be a long, so we really don't

Should we remove it?

We can, but if it ever is called we would get an exception. I think that is a risk I am willing to take, but it would be good to have a comment explaining it.

So is GpuIntegralDecimalDivide.resultDecimalType even used?

Just to clarify a bit resultDecimalType is called by CudfBinaryArithmetic.dataTypeInteral which becomes the dataType for the expression, except GpuIntegralDivide overrides dataType to be a long, so we really don't

Should we remove it?

We can, but if it ever is called we would get an exception. I think that is a risk I am willing to take, but it would be good to have a comment explaining it.

I think getting an Exception is better than the alternative. I will remove it.


Area2D can not override space at all.

v2.0.2 stable
win 10 x64
As it said here:
http://docs.godotengine.org/en/latest/classes/class_area2d.html#class-area2d
Area2D can override space parameters.
But:

Whatever I change Gravity Vector and Gravity to, nothing changes, the acceleration is the same.

You need to give your Area2D (and other CollisionObjects) shapes with which it collides. Easiest way is to use CollisionShape2D nodes for this: http://docs.godotengine.org/en/latest/tutorials/2d/physics_introduction.html#collisionshape2d

@eska014 Thank you, solved. But is there a way to set the default gravity?

You can change it in Scene→Projects Settings in the category Physics 2d.


Suggestion :: Design Choice :: Central Server Infastructure Elimination

With 1 little extra step, we can eliminate the need for a central server. We can easily send our public ip address using email,sms,phone,etc using hexadecimal digits, like this: <IP_ADDRESS> -> C0A80101. It's well within humans ability to remember 8 chars, if for example you own a static IP, you can memorize your IP, like you do with your passwords. Using base-62(26x2+10) encoding we can go down to 6 chars transmitted. The library should be able to identify computer's public ip, encode it properly and echo it to user. For example user executes "wormhole send thisfile*" , app prints "AzS9sF", receiver executes "wormhole receive AzS9sF"

Interesting idea.. thanks!
You'll still need the secret words though (to protect against network adversaries, or the server being compromised), so the code would be at least wormhole receive AzS9sF-purple-sausages. And we'd have to either require the server to run on a well-known port, or include the port number too (wormhole receive AzS9Sf:4000-purple-sausages). And I'd like this protocol to survive the transition to IPv6 (wormhole receive 2600:3c01::e8f3:9145:fed4:59ed:4000-purple-sausages), which makes the server addresses much to long to feasibly transcribe.
Also, base62 is maximal for things that need to be typed, but since the codes need to be verbally transcribable too, I don't think we can afford to have upper-vs-lowercase. So base32 or base36 is probably the densest we can do, which makes the encoded IP addresses slightly larger. A big part of the value of magic-wormhole is how easy the codes are to speak and to write, which constrains what we can do with the server address.

Protocols use std ports since ever. Unless security is a concern user should be permitted to either use a standard port and reduce typing load or use a random port and transmit it. SSH faced similar problems, when in fact stakes where much higher, a rooted machine vs 1 file.
Base36 is indeed easier to transmit verbally. Again user could choose what to use. At library side, library could calculated both. At receive side typing either, can be easily detected.
Maybe sender/receiver could opt for 1-time passwords, or a pre-shared key/secret knock
ipv6 is a tough nut. 128 bits ~ 25 chars/base36. Static Ip & caching could help.  Maybe static-ip are issued by default in a ipv6 world. Cheers!

couldn't you use a DHT to find each other?

@bj0 yeah, probably, but we need to allocate unique channel IDs, which is a lot easier with a centralized service.. it can probably be done reliably-enough in a DHT, but would take more work and more time, increasing latency. Let's use #72 to brainstorm about DHT options.
@FrantzelasG I forgot to mention NAT.. most of the time, when I'm sending a file to someone else, neither of us has a public IP address, so we need something that we can both connect to, just to bootstrap the hole-punching process. And I've found that most of my machines that do have a publically-reachable IP address aren't really personal machines, they're servers, and they've got stable public hostnames too, and I just remember the hostname.
I agree that using standard ports is fine, so we needn't worry about fitting those extra characters into the string.
Using base36 (or 62 or whatever) loses a lot of the error-checking options for the code: any typo will prevent the connection from being made. (In the current 4-purple-sausages scheme, the "4" is critical and has no error-checking, but the "purple" and "sausages" come from a short wordlist and so the receiving client could use the normal spell-checking techniques to find the closest word and tolerate typos). So I'd like to make the non-error-checked portion as short as possible.
The purple-sausages part of the existing scheme is basically a one-time password (although the PAKE crypto we use makes it safe even if someone guesses the code afterwards). Using a pre-shared key or some out-of-band thing like port-knocking means you have to deliver that secret along with the rest of the code, making it bigger and more error-prone (e.g. "send me a file, by typing in IP address X.Y.Z.Q, and also make a TCP connection to port 1948 just before you run the program, and also pre-encrypt the file with key abc..256..bits..later..xyz", vs "4-purple-sausages"). And, if it's optional, then people will prefer to not use it, and then this isn't a secure file-transfer tool anymore.
My other concern with transcribing an IP address is that the direction of the file transfer might not align with the direction of the information transfer. If I'm sending a file to you, and you've got a public IP address (but I don't), then first you have to tell me your IP address, and then I send you the file.  They might line up if I happen to have a public IP (then I tell you "please contact this IP and ask for a file") but there's no guarantee that it'll work out that way.
And yeah, v6 makes all manual-adress-handling things harder. I'm glad there are folks doing things like automatic-local-service-discovery (MDNS-type stuff), that's going to make things a lot better.
thanks!

We don't need to transfer the IP address of the user.
Instead, we can have a network of rendez-vous servers, that know about each other (hold a list of other servers, join new servers to this list).
A client bootstraps into this network and downloads a list of servers. Then, it picks one at random, performs the usual rendez-vous stuff. It gets its one-time password, then compresses the IP of the server, and the resulting string looks something like AzS9Sf4-purple-sausages.
There's still an infrastructure, yes, but it's peer-to-peer, it's self-configuring, it costs peanuts to run.


parseDuration: return an error on bad values

There was a typo in parseDuration: it returned nil even if strconv.ParseDuration returned an error. I also added a test for it.

Thanks, but I'm confused; you fixed parseDuration yet added a test for an invalid bool config. The test should be for an invalid duration right ?

Right, sorry I was working on something using ParseBool at the same time thus the confusion.

Thanks !


Dative bond support

RDKit supports dative bonds in SMILES but it's not clear to me that this is generally supported by other toolkits. For instance, the OEChem TK documentation doesn't mention them.
Should this affect our recommendations for e.g. representing coordination complexes in reactions?
(Credit to @brilee for the question.)

Closing this; we decided that it's fine to require RDKit for this case, and we can always prepare Molfiles or non-dative SMILES if needed.


Do not expose runtime.internal imports in runtime.api package

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

$title
Fixes #38077
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Created https://github.com/ballerina-platform/ballerina-lang/issues/40337 to change Runtime, Future, Environment APIs to interfaces.

CI Full Build Ubuntu / Build Stdlib Level (6) is failing due to https://github.com/ballerina-platform/ballerina-standard-library/issues/4405. Will fix that.


[Android] Server starts listening but doesn't respond to connections

Hi,
Copy-pasted it from your  readme example. The server even connects (according to the client) but it looks like it doesn't enter to the onOpen() function to get the connection data.
I'm using Ionic.
Thanks,

check origins and protocols options.

They're both commented out.

What do the clients say exactly?


errno: 'ETIMEOUT'

I have a problem. My problem is I typed npx shadcn-ui@latest init and Enter, it didn't load install it show the error like the picture below. Can someone help me.

have same problem for weeks

i had same problem for weeks
first run this npm i shadcn-ui
then run npx shadcn-ui init

I tried running npm i shadcn-ui and it installed normally, but when I typed the npx shadcn-ui init command, the error 'ETIMEDOUT' still appears.

Make sure you have deactivated your ad blocker.

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Fix Oxygen project creation

Description
Due to the way Hydrogen projects are bootstrapped in the admin, we need to temporarily continue storing templates in the deprecated create-hydrogen-app package.
This PR revives a prepack script that was removed in https://github.com/Shopify/hydrogen/pull/1475. This script copies the demo-store template into this package where it is later used by the Oxygen admin to seed new projects.
In the future, the Oxygen admin should use our templates directly from this github repo in the stackblitz branch to seed new projects. (cc @gustavo-sanchez @Shopify/oxygen-admin)
Before submitting the PR, please make sure you do the following:

[ ] Read the Contributing Guidelines
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123)
[ ] Update docs in this repository according to your change
[ ] Run yarn changeset add if this PR cause a version bump based on Keep a Changelog and adheres to Semantic Versioning

e2e test failure is known and unrelated


feat(creds): Add delete credentials command

What does this change
Add command to delete credentials
What issue does it fix
Closes #854
Checklist

[x] Unit Tests
[ ] Documentation

[ ] Documentation Not Impacted

/azp run porter

This looks great! 👍
I just manually kicked the build because of a hiccup in our build system over the weekend. Going forward builds of porter should automatically kick off, and porter-integration is something that we would manually trigger only if it is affected by the PR.


[Suggestion] sharable/linkable group containers.

Not sure if this is the right place for this suggestion, but is it possible to make a container item appear on every player character's sheet when one person has it equipped? Or to make it so that an item can be linked to a container like actor so that everyone with the item on their sheet can access said container actor by clicking on the item on their sheet so it opens up a container window?
This would be really useful for containers that each party member accesses frequently, like a Bag of Holding, a carriage's cargo or a Keeps Storage etc? Because it's a pain for my players to constantly having to ask who has the bag of holding right now, is x item in the bag atm etc. especially since dragging and dropping items always only dupes them, never moves them like the containers do in dnd 5e now.

This kind of special item would be the purview of a specialized module, I think.
However, consider this idea: some have suggested using a Group Actor's inventory as the party / camp inventory. This is a great way to provide loot, coinage, and shared resources. It should also support containers like a bag of holding.

Ah ok good to know. Hope someone makes such a module some day. Because using the Group Actor for this is a similar problematic with items just duplicating on switch back and forth unlike containers that actually move things.

Containers moving things is unfortunately an illusion of how they are coded. They give the impression that they are moving items from container to actor and vice versa. The truth is, dnd5e is just putting on or taking off a container ID for the dragged item. It's not actually moving data. It does feel right to move rather than copy items, and at a system level, it would be good if they coded it so items actually transferred when dragged between actors. Maybe someday. I'm sure someone has requested it on the system repo.


Code Action: Allow enabling/disabling and improve ranking of explain

Description

Group the fix and explain actions
Allow enabling/disabling the actions all together

Enabled:

Disabled:

Interface:

Test plan
Tested locally, enabling and disabling through interface and manually in VS Code settings

@abeatrix Thought about it but I think it's OK to just keep the basic implementation for now. Doesn't feel like users will often enable/disable inline chat, and the extra overhead of having to register/deregister the action to ensure we don't have duplicate code actions isn't worth it IMO. We can adjust if we get feedback from users that we need this


[CI] Restore Publish, Danger on Circle CI 2.0

Test Plan
Run on Circle.

Trying again with #16342


Update physics API and example

What problem does this solve or what need does it fill?
Rapier physics integration is very valuable, but the current API and examples do not use the latest API, and physics_dungeon is broken.
Describe the solution would you like?
Use the latest Rapier API including:

Using RigidBody and Collider components rather than RigidBodySet

Describe the alternative(s) you've considered?
Removing the physics example while non-functional?
Additional context
N/A

Yeah it was shelved for the time being as it required quite a large overhaul that was WIP.


SELinux support for oci create-image

The oci create-image command should label all the files using the default labels set in the policy from the container.
If a user wants to change a label for a file, they can modify the SELinux policy file mapping using semanage in a container layer and that will be taken into account by this.
To do that, we need to lookup all the files labels when we create the composefs blob.
Looking up a file label is done like that in C (from Allison's code):
#include <stdlib.h>
#include <unistd.h>
#include <selinux/label.h>
#include <selinux/selinux.h>
#include <sys/stat.h>
#include <err.h>
#include <stdio.h>

int
main (int argc, char **argv)
{
    struct selinux_opt opts[] = {
    };

  struct selabel_handle *hnd = selabel_open (SELABEL_CTX_FILE, opts, 0);

  for (int i = 1; i < argc; i++)
    {
      struct stat buf;

      if (lstat (argv[i], &buf) != 0)
        err(EXIT_FAILURE, "stat %s: ", argv[i]);

      char *ctx = NULL;
      if (selabel_lookup (hnd, &ctx, argv[i], buf.st_mode & S_IFMT) != 0)
        err(EXIT_FAILURE, "selabel_lookup('%s', %d): ", argv[i], buf.st_mode & S_IFMT);

      printf ("%s → %s\n", argv[i], ctx);
      freecon (ctx);
    }

  selabel_close (hnd);
}

As we need to use the SELinux policy from the container, we need to:

figure out which policy is going to be used (or maybe we just extract all the policy available?)
extract the SELinux file contexts mapping files from the container to a temporary path:

/etc/selinux/{SELINUXTYPE}/contexts/files/file_contexts
...
See https://man7.org/linux/man-pages/man5/selabel_file.5.html

point libselinux to the extracted files_contexts by setting the folllowing selinux_opt:
SELABEL_OPT_PATH -> /path/to/etc/selinux/{SELINUXTYPE}/contexts/files/file_contexts

Example custom SELinux context in https://gitlab.com/fedora/bootc/tests/container-fixtures/-/merge_requests/13.

This is going to go something like this (discussed with @travier in a super-productive call):

during cfsctl oci create-image after the layer merge but before calling mkcomposefs:

create a tempdir
use composefs::fs::write_to_path() to extract /etc/selinux/ of the merged container image to the tempdir
selabel_open() the policy from the tempdir
walk the filesystem tree doing a selinux query for each inode, adding xattrs as we go

call mkcomposefs

Risks: we're using libselabel from the host system which is yet another moving part in our must-be-super-deterministic pipeline.  That means that if something changes in a way that affects some image (some regexp engine bugfix or so) we're going to invalidate our composefs fs-verity digest.
Future: we should ask libselabel about an API that lets us feed the policy directly instead of needing to unpack to a tempdir.  Maybe such an API already exists, or maybe they'd be willing to add it for us.

Further future: we probably want to move as much of this as possible to the server/build side.  In the case that we're created a sealed UKI image then it definitely makes sense to do the entire computation there, producing new tar layers with the xattrs already baked in.  But we need to be able to work with these two situations:

the user derives a base image and adds some file (which needs to get labelled)
the user derives a base image and modifies the policy (which means the entire system needs to be potentially relabelled)

Those two cases mean that we need the labelling support already on the client side anyway, so we may as well go ahead and develop it.  We can do the optimization of moving it "build side" for performance reasons if/after we discover performance problems.

Risks: we're using libselabel from the host system which is yet another moving part in our must-be-super-deterministic pipeline.

Our examples should use the FROM oci-archive trick by default which links everything into a container build pipeline and ensures host independence. (We can still support being run outside of a containerfile too of course, but I think the "Containerfile is entrypoint" is the right default)


Fix typo in expr_opt doc comment.

This PR changes the doc for expr_res, not expr_opt (which was fixed in 87e7613 )


Improve test_version_format() error message for version mismatches

What Does This PR Address?
Enhances the test_version_format() Error Message for Version Mismatches.
Fixes #949
Description
This PR enhances the test_version_format() function in tests/test_version.py by updating the error message to be more informative. The new message provides clearer guidance on resolving version mismatch issues, including a suggestion to rerun make install if needed.

Sir I have resolved the issue carefully. Please review the updated.

Thank you for your feedback, @kevinjqliu . I have resolved the issue carefully . Please review the updated.

Sir, is there any new or further update?

I will investigate the issue and address the failing tests as soon as possible. I'll update once the problem is resolved.

Thanks @laksh-krishna-sharma
Looks like this is a linter issue
fix end of files.........................................................Failed
- hook id: end-of-file-fixer
- exit code: 1
- files were modified by this hook

Fixing tests/test_version.py

You can run make lint to auto-correct

Closed in favor of #1015


Bootstrap Glyphicon

Are you planning add Glyphicons from Bootstrap?

No. I'm not planning add Glyphicons or any other icons just now.
But you can use custom template with any icons you want
For example:


fix: Resize splash logo to appropriate size

Fixes #1591
Changes:

Resize the splash logo shown at app launch to appropriate size maintaining the aspect ratio of the image

Screenshots for the change:

@niranjan94 Please take a look


feat: support for jobs using the Pipeline Multibranch plugin

Description
Problem
Jenkins-source does not recognize jobs that use the Pipeline Multibranch plugin.
Analysis
Jobs using the Pipeline Multibranch plugin have the _class 'org.jenkinsci.plugins.workflow.multibranch.WorkflowMultiBranchProject' that are not handled
currently by jenkins-source.
Solution
Added the handling of the _class 'org.jenkinsci.plugins.workflow.multibranch.WorkflowMultiBranchProject' to recognize these jobs
currently by jenkins-source.
Type of change

[ ] Bug fix
[X] New feature
[ ] Breaking change

Superseded by https://github.com/faros-ai/airbyte-connectors/pull/721/files


Update shell completions

Current shell completions are generated by optparse-applicative. But they are not good.

This has already done by 13287e0c49d3e04cccc18cfa5869f392ad439b8c.


Fix HdrHeap unit test.

I think this was the interaction of two different PRs that affected similar areas and ran successfully independently but not not together (which was not detected because the tests had already passed).
At least for me, this doesn't compile on master and I can't see how it could, given the HDR_HEAP_DEFAULT_SIZE symbol simply doesn't exist anywhere else.

Closing in favor of  #5068


[New Block] Gen-AI-Row for Max homepage

For the carousel row block that can contain either CTAs or gen-ai input forms. Should be a benign PR since it's a new block and not used anywhere yet.
Authoring kitchen sink: https://gen-ai-cards--express--adobecom.hlx.page/docs/library/kitchen-sink/gen-ai-row?lighthouse=on
Resolves: https://jira.corp.adobe.com/browse/MWPW-136567
Test URLs:

Before: https://stage--express--adobecom.hlx.page/drafts/jingle/gen-ai-row?lighthouse=on
After: https://gen-ai-cards--express--adobecom.hlx.page/drafts/jingle/gen-ai-row?lighthouse=on

Codecov Report

Merging #272 (186a4f7) into stage (7dc3c77) will increase coverage by 0.64%.
Report is 12 commits behind head on stage.
The diff coverage is 80.09%.

@@            Coverage Diff             @@
##            stage     #272      +/-   ##
==========================================
+ Coverage   47.16%   47.80%   +0.64%
==========================================
  Files          29       30       +1
  Lines        8840     9071     +231
==========================================
+ Hits         4169     4336     +167
- Misses       4671     4735      +64

Files Changed
Coverage Δ

express/blocks/gen-ai-row/gen-ai-row.js
80.09% <80.09%> (ø)

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


oranda fails with undescriptive error

Given a config that only specifies usage of cargo-dist:
{
  "artifacts": {
    "cargo_dist": true
  }
}

oranda build fails with:
↪ >o_o< INFO: Running build...
↪ >o_o< INFO: Loading config at ./oranda.json
↪ >o_o< INFO: Detected Rust project...
✗ >o_o< ERROR: error decoding response body: expected value at line 1 column 1

When the config is empty / file not existing it builds fine.
The cargo-dist is not actually set up in the repo (however, there is a release existing). Doe sit query github releases and that fails? If so, the error should be a bit more descriptive :)

Hey!
Can you please let me know what is the repo you are using?

hey @spitfire05 - long time no talk :P
i'm running into this too, agree that there's a bunch of error handling work to do around how manage github requests. right now we are assuming success and just immediately trying to parse the body :) i'll put a PR for this later today

@SaraVieira I was playing with oranda using https://github.com/spitfire05/portproxy
@ashleygwilliams hi! 😄 understood, hope better error handling comes soon, as this can be very misleading (I spent quite some time trying to figure out what the hell is wrong with.. the config I provide ;) ).

@spitfire05 #193 has several error handling improvements :) i'm going to merge and cut a release (along with cargo-dist), later today.
we haven't formally released/announced oranda yet largely due to these rough edges (very much appreciate early users giving us feedback!!!) please keep filing things you run into!

specifically: https://github.com/axodotdev/oranda/pull/193/commits/632cc53f98ff565640ddf6375b5741f7b05b1abd#diff-fefa9072271fc2a9538b49a84c25e3ec807581a2cd5a3d7de97703de07436826R50 should fix what you were running into (and we were too!)

this should be fixed as i cannot reproduce it. i'll cut 0.0.2 tomorrow. if that ends up not fixing this, please feel free to reopen @spitfire05 !


file read errors go uncaught

monofile crashes on a file read error due to such error not being caught. Probably should be fixed

does this still happen in api-v1?


[Breaking change] Run Tez UI in App Timeline Server

Changes Tez UI from http://clustername-m:9999 to
http://clustername-m:8188/tez.

This avoids needing to run a separate webserver for the Tez UI

The nodejs install script does not retry on apt-get update failures, which causes the init action to fail occasionally. This PR may also fix that problem, since we do retry apt-get update.
W: Failed to fetch https://deb.nodesource.com/node_8.x/dists/jessie/main/binary-amd64/Packages  Hash Sum mismatch

E: Some index files failed to download. They have been ignored, or old ones used instead.
Error executing command, exiting
+ err 'Failed to setup node 8.'
++ date +%Y-%m-%dT%H:%M:%S%z
+ echo '[2018-05-15T23:32:39+0000]: Failed to setup node 8.'
[2018-05-15T23:32:39+0000]: Failed to setup node 8.
+ return 1

For posterity, I ended up setting the web ui path to /tez-ui to match Tez in Dataproc 1.3 preview.


include macros

why don't you use the standard macros?
http://doc.rust-lang.org/std/macro.include_bytes!.html
http://doc.rust-lang.org/std/macro.include_str!.html

@flosse Thanks for the info didn't see it till now. The thing about standard macros is you need to know the file path. Whereas in my usecase it dynamically generates the list of files and also allows you to serve the files as a folder for a static web server. This is very common and used a lot in go web development.

Whereas in my usecase it dynamically generates the list of files and also allows you to serve the files as a folder for a static web server

You could mention this in the README to point the difference and avoid confusion :)

Fixed


OOM issue by accumulating Aggregate executor object instances.

Description:
We identify the memory leak in GroupByAggregationAttributeExecutor, where the aggregate attribute executors added in [1] not completely removed by the fixes given in https://github.com/wso2/siddhi/pull/739.
[1] - https://github.com/wso2/siddhi/blob/3.1.x/modules/siddhi-core/src/main/java/org/wso2/siddhi/core/query/selector/attribute/aggregator/AttributeAggregator.java#L41
Suggested Labels:

Suggested Assignees:

Affected Product Version:
Data Analytics Server 3.1.0,
Complex Event Processor 4.2.0,
API Manager 2.1.0,
Identity Server 5.3.0,
IS as Key Manager 5.3.0
OS, DB, other environment details and versions:
Steps to reproduce:

Extract the attached artifacts into DAS server.
Execute the JMeter script.
Then you can monitor the JVM heap memory level increases, and please verify the issue with heap dump.
Archive.zip

Related Issues:
https://wso2.org/jira/browse/DAS-608

Fixed by PR: https://github.com/wso2/siddhi/pull/779


Remove halt() call in TestSimpleServer (part of TestStressNRTReplication

A follow-up to #13164 . I don't fully understand this code (I get the gist) so I didn't go for any revolutionary changes - just removed the halt() call in favor of an external process killer. I've re-ran the test 10 times. It is currently only run in nightly mode (takes ages) but you can run it in non-nightly mode with:
gradlew -p lucene\replicator test --tests TestStressNRTReplication  -Ptests.filter="@Nightly"

because the tests.filter doesn't affect the "nightly" system property (so test parameters are not scaled to nightly standards but still run).

Thanks for doing this. I was disappointed that java doesn't allow this:
jshell> ProcessHandle.current().destroyForcibly()
|  Exception java.lang.IllegalStateException: destroy of current process not allowed
|        at ProcessHandleImpl.destroyProcess (ProcessHandleImpl.java:369)
|        at ProcessHandleImpl.destroyForcibly (ProcessHandleImpl.java:392)
|        at (#2:1)

I think it's in the method's documentation (current()) that the returned handle can't be used to stop yourself. Indeed, I tried it too. ;)


updated publications view

updated publications under the author page ^.^

👷 Deploy Preview for rayko-research processing.

Name
Link

🔨 Latest commit
79475805c54d4f6b67b4fb4ba4e289f726e2315f

🔍 Latest deploy log
https://app.netlify.com/sites/rayko-research/deploys/63ddc62f28ae910009de985d


Relationship filter parameters unsupported

I haven't had time to test this too thoroughly, I will try and update this issue with more information when I do, but it doesn't appear that fractal's nested relationships are working correctly when used with this package. By that I mean url patterns like this:
example.com/books?include=author.stores
Getting the "author" relationship works, but getting the ".stores" portion of it does not.

Hey! What version of the package are you using?

I'm not on my home computer right now, but I ran composer update a day or 2 ago, and I believe I am on the latest version (1.2.7).

I know there has been an issue with nested relationships in the past, however, this should have been resolved in a prior patch.
I can't seem to replicate the issue. Given the following code:
$user = App\User::with('posts.comments')->first();

    return responder()->success($user);

The following JSON was outputted:
{
    "status": 200,
    "success": true,
    "data": {
        "id": 1,
        "name": "Claude Cole Jr.",
        "email"<EMAIL_ADDRESS>        "created_at": "2016-10-25 15:06:07",
        "updated_at": "2016-10-25 15:06:07",
        "posts": [
            {
                "id": 1,
                "content": "This is a post",
                "user_id": 1,
                "created_at": "2016-10-25 15:06:07",
                "updated_at": "2016-10-25 15:06:07",
                "comments": [
                    {
                         "id": 1,
                         "content": "This is a comment",
                         "post_id": 1,
                         "created_at": "2016-10-25 15:06:07",
                         "updated_at": "2016-10-25 15:06:07"
                    }
                ]
            }
        ]
    }
}

Hmm, I will try to look into this further when I get home tonight then. Do you know if your example works when done through the "includes" parameter in the url of a route, rather than hard coded? For instance, like this:
example.com/user?include=posts.comments
Obviously you'd also need a route, controller, and transformers for the users and posts.

How are you loading the relations? It should work just as fine doing something along the lines of:
$user = App\User::with(request()->include)->first();

return responder()->success($user);

In my past experience with using Fractal directly, I never had to load anything directly like that (passing the includes from the request object into the query). I just instantiated Fractal like this:
$fractal = new League\Fractal\Manager();
Then loaded the includes from the URL:
if (Input::get('include')) {        $fractal->parseIncludes(Input::get('include')); }
Then whatever I put into the URL would just be lazy loaded automatically. Same way it's done in the docs (http://fractal.thephpleague.com/transformers/)[http://fractal.thephpleague.com/transformers/].

Yeah - this package works slightly different to be more in line with how Eloquent works. Basically, you don't have to call parseIncludes(), but instead pass in your data with the relations loaded (like the example above). Then the package will automatically catch all relations and call parseIncludes() on Fractal behind the scenes. :)

Interesting! I apologize, as I see that this is in the documentation.
Honestly, what's really weird to me, is that in my current code this works:
example.com/user?include=posts
But this does not:
example.com/user?include=posts.comments
I don't use "with" to load the relationship in either case. But the second example does the same thing as the first, it returns the user with the posts, but NOT with the posts comments.
When I get back home, I'll try to pass the "include" property from the request to my query directly and see if that makes it work. Sorry, it's difficult for me to be more helpful when I'm not working on it hands-on.

Okay, after working with this more, I believe I now understand how the relationship includes work with this package.
I absolutely see what you were going for, it's a very interesting idea. However, I'm not sure it's quite flexible enough. For instance, with Fractal you can do some crazy stuff with parameters like so:
?include=comments:limit(5|1):order(created_at|desc).
With this package, it doesn't seem that's possible, since there's no $availableIncludes parameter on the transformers and thus, no respective functions for loading relationships where you can do some business logic with those parameters.
Am I missing something or am I correct in thinking that you forfeit some of that flexibility when using this package since it tries to load relationships "auto-magically"?

Sorry to spam you, but after playing around further, I realized I could get the relationships to work more like traditional Fractal's by changing like 142 in SuccessResponseBuilder.php from this:
$this->manager->parseIncludes(array_merge($transformer->getRelations(), $this->resolveNestedRelations($resource->getData())));
to this:
$this->manager->parseIncludes(Input::get('include'));
I think it would be really cool if you added an option to the config file for something like "smart-load-relationships". If it was set to true, it would work like it does currently. If it was set to false, it would work like my way above.
This package is by far the best fractal abstraction, awesome job on it. But I imagine I won't be the last person who needs some more flexibility when it comes to relationships. Having the option to switch your smart-loading on or off would be great for those situations.

Yeah, you're right, that is indeed a missing feature from Fractal. I still don't feel like Fractal's DSL (if I can even call it that) feels very Laravel-ish, and would optimally like to find a better solution for it. However, I'm all up for allowing an alternative syntax to load includes until I can find a better solution for it.
What do you think of the following syntax:
return responder()->transform($user)->include('posts,comments:limit(5|1)')->respond();

This could even work together with the existing solution. If you don't load any relations into $user, the package wont be doing any smart autoloading of relations.

I think the problem with that syntax is that most of the time, people are going to be appending those includes to the URL's and not including them hardcoded like that. So you'd basically have to do this:
return responder()->transform($user)->include(Input::get('include', []))->respond();
anywhere in your controllers that you output your a model that might have includes. Personally, that feels the most un-laravelish to me. Also, with that proposed syntax, where would the logic for the "limit(5|1)" part go? Trying to figure that out auto-magically on your end would be incredibly hard, and would fail if I wanted to do something custom instead like "comments:only(best)" or whatever.
As much as I like the auto-magic stuff (the way you automatically detect transformers is my favorite part of this package), I think due to the highly customized way people sometimes need to use URL parameters, sticking with how Fractal typically handles it is the best solution. Maybe you could make a "smart_load_relationships" config option that would default to true, and if it was true it would work like it current does, but if it was false it would work like Fractal normally does?

I personally also load relations from query parameters, but I'm usually using 'with' instead of 'include', so if the package should pick up on relations automatically, I think it might be a good idea to include a config option for specifying which GET parameter to load relations from.
'loadRelationsFromParameter' => 'include' // or false to disable

That might require a bit of reworking of how the package currently works though. Where would you usually place the "limit(5|1)" logic in Fractal and what would it look like?

Typically the parameter logic would be placed in the transformer, the same way you would normally load relationships via Fractal, like so:
`<?php
use League\Fractal\ParamBag;

private $availableIncludes = ['comments'];
private $validParams = ['limit', 'order'];

public function includeComments(Book $book, ParamBag $params = null)
{
    if ($params === null) {
        return $book->comments;
    }

    list($limit, $offset) = $params->get('limit');
    list($orderCol, $orderBy) = $params->get('order');

    $comments = $book->comments
        ->take($limit)
        ->skip($offset)
        ->orderBy($orderCol, $orderBy)
        ->get();

    return $this->collection($comments, new CommentTransformer);
}`

That was taken almost verbatim from the Transformers docs, btw: http://fractal.thephpleague.com/transformers/ Just ctrl+f "Include Parameters" to find the section.

Right, I see. I still suppose you end up with a lot of boilerplate code when you have to do the above for every relation?
I've been thinking a bit about an alternative method for solving the issue. I'm imagining a scenario where you can place a 'Filterable' trait of some sort to every model you want to allow filtering on. Then the package can parse the request parameters and do the filtering if the model uses the trait.
That way we get the best of both worlds, you can filter based on GET parameters and you wont need to make one method for every relation in your transformers. We can still allow $availableIncludes and $validParams to the transformers, which I think is important when just parsing the request object directly.
Would that be a suitable solution you think? :)
Thanks for spending time on the issue!

I think that's a decent solution, my concern again would be the flexibility. You're right that there is a bit of boilerplate code with Fractal's standard solution for this, but it also allows the most flexibility. For instance, with Fractal if I wanted to craft a URL like this:
example.com/user?include=posts.comments
I would get the relationships to work by putting one "include" function on the USER transformer to get the POSTS like this:
public function includePosts(User $user)
{
    return $this->collection($user->posts, new PostTransformer);
}

and one on the POSTS transformer to get the COMMENTS like this:
public function includeComments(Post $post)
{
    return $this->collection($post->comments, new CommentTransformer);
}

Is this a little boilerplate-y? Yeah, I guess so. But let's say users can also "like" posts on my site, and now I want to do a URL like this instead to get a user with their posts and each posts likes:
example.com/user?include=posts.likes
All I have to do is add another method on my POSTS transformer like this:
public function includeLikes(Post $post)
{
    return $this->collection($post->likes, new LikesTransformer);
}

And now that will work. It's boilerplate in a sense, but it also has a clear separation of concerns. Every single type of relationship you can request is defined in it's own place.
If you go the Filterable trait route, my fear is either that you'd be re-creating exactly what Fractal does by default above, which would be a waste of your time, or the solution would be less flexible. If we have a trait with a method that just gives me the "include" part of the URL and I have to then manually parse "posts.likes" or "post.comments" and load the relationship, it could get really messy.
And no problem! Happy to discuss this.

Hey again!
Sorry for the delay. I just pushed some changes to the package which should fix these problems.  First, I added a new configuration key: load_relations_from_parameter, which defaults to 'include' and can be set to null to disable automatic relation loading from the GET parameter. I've also added the alternative route of parsing includes manually:
return responder()->transform($user)->include('posts,comments:limit(5|1)')->respond();

Additionally, I've added a new property to the base transformer (and any new transformer created with php artisan make:transformer:
protected $relations = ['*'];

As you can see, it defaults to '*', which simply whitelists all relations. This means we have complete backwards compatibility, so I pushed the changes as a minor version 1.2.8.
Oh yeah, and now you can create include methods in your transformer to filter your relations. The only difference from Fractal is that you call on orders() instead of includeOrders().
I've updated the documentation accordingly as well!
Here's the commit if you're interested in the implementation: https://github.com/flugger/laravel-responder/commit/183e5eb19cbc84828553cd92011a79ba8a6b8589
Let me know what you think :)

Thanks for the update! Seems like a step in the right direction. I haven't dug too deeply into this but did try it out quickly last night. One question, it seems like whatever I add the the $relations variable on the transformer gets auto-loaded every single time I return that model.
For instance, if I have a UserTransformer and protected $relations = [''posts'];, every time I return a user I also get their posts. Is this intended functionality? What I was expecting was for the posts to only get returned when I add "posts" to the "include" param in the URL. If the relationship runs every single time, then I'm running an extra query every time I load a user even when I don't need their posts.

Ah, I see - that was indeed not intentional. I'll look into it

Hi again, pushed up a fix (1.2.9), let me know if you have any more issues with it :)

I am having issues with the relations property as well, (using v1.2.9). It doesn't seem like it is applying the relations array properly. Here's the response I get:
{
    "status": 200,
    "success": true,
    "data": [
        {
            "id": 1,
            "posts": [
                {
                    "id": 1
                }
            ],
            "comments": [
                {
                    "id": 1
                }
            ]
        }
    ]
}

Here is my code:
Route::get('/api', function () {
    $users = User::with([ 'posts', 'comments' ]);

    return responder()->success($users);
});

<?php

namespace App;

use App\Transformers\UserTransformer;
use Flugg\Responder\Contracts\Transformable;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable;

class User extends Authenticatable implements Transformable
{

    use Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name',
        'email',
        'password',
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password',
        'remember_token',
    ];

    /**
     * The path to the transformer class.
     *
     * @return string
     */
    public static function transformer()
    {
        return UserTransformer::class;
    }

    public function posts()
    {
        return $this->hasMany(Post::class);
    }

    public function comments()
    {
        return $this->hasMany(Comment::class);
    }
}

<?php

namespace App\Transformers;

use App\User;
use Flugg\Responder\Transformer;

class UserTransformer extends Transformer
{

    /**
     * A list of all available relations.
     *
     * @var array
     */
    protected $relations = [ 'posts' ];

    /**
     * Transform the model data into a generic array.
     *
     * @param  User $user
     *
     * @return array
     */
    public function transform(User $user)
    {
        return [
            'id' => (int) $user->id
        ];
    }
}

Sorry, that was indeed a small mistake from my part. 1.2.10 should be good.

1.2.10 Seems to be working for me

2 questions regarding 1.2.10:

Am I correct in thinking that this is still different from Fractal in the sense that you need to manually pass your "include" parameter to your model in your controller?

I'm not sure if this is just my code, but when I load relationships via a model that has been route model binded, all of a sudden I get a list of all the models. For instance this:

    public function show(Request $request, User $user)
    {
        $user = $user->with($request->input('include', []));
        return responder()->transform($user);
    }

I get a list of ALL my users instead of just this user.

Hey again!

No, it should be even simpler than that now - the package's service provider will now inject includes from the GET parameter "include" into the response builder automatically. You shouldn't need to do anything :)

When calling with() on a model you are turning it into a query builder. If the responder receives a query builder it will call get() on it, which returns all users. You should instead use load().

Thanks for the reply! We're getting very close here.

The automatic injection you described doesn't work like that for me. For instance, when I call this URL:

example.com/users/33?include=posts
it works with this:
    public function show(Request $request, User $user)
    {
        $user = $user->load($request->input('include', []));
        return responder()->transform($user);
    }

but not with this:
    public function show(Request $request, User $user)
    {
        return responder()->transform($user);
    }

It seems like in my case, I need to do the "load" automatically.

Nested relationships don't seem to be working for me. For instance putting posts.comments in the "include" gives me the posts but not the comments.

Sorry about all the issues. I renamed the config name and forgot to change it in the service provider. Should all be good as of 1.2.13, crossing my fingers there wont be more problems! I'll get around to write some more test scenarios for the new stuff this week.
Your point 3) should work fine. You can create a method named whatever in your transformers, and that will be called if the method exists, else it will just call $model->relation.

Thanks man! No worries about the issues, I appreciate you being so open to suggestions. I will try the fix for #1 when I get home tonight.
Regarding #3, when I tried it last night I got an error (only for transformer relations that weren't also model relations). But let me try it again tonight to confirm. At the very least, I can get you the actual error message.

Well, after trying this out tonight, everything works! Thank you again for being so open to my suggestions and quick to respond. With these additions, I really believe this package is the cleanest and most powerful fractal abstraction. Phenomenal job.
If I could make one final suggestion, I think it would be helpful to add more detail to the documentation about custom transformer relationship methods. Currently, the documentation has an example like this:
public function roles(User $user, ParamBag $paramBag)
{
    //
}

Normally in Fractal, you would fill out this method someway like this:
public function roles(User $user, ParamBag $paramBag)
{
    return $this->collection($user->roles, new RoleTransformer);
}

With this package, you do this:
public function roles(User $user, ParamBag $paramBag)
{
    return responder()->transform($user->roles)->getResource();
}

The important part there being, that in transformer relationship methods, you have to return an instance of a Fractal resource, which you can obtain with the "getResource()" method on your package. That's something I think most users could easily get stuck on.
Thank you again.

No worries, glad to be of help!
I admittely rushed the documentation a bit and will definitely improve it, so its on par with the rest of the documentation.
Btw, another thing is, you dont actually have to return a resource from the roles method, you may in fact return just the data: return $user->roles; and the package will wrap it in a resource automatically.
Thanks again for spending so much time on the issue, really appreciated! I will close this as it seems like all issues are solved :)

Any ideas why the includes are not "automatically" working for me?
public function show(Request $request, $id)
    {
        $user = User::findOrFail($id);

        return Responder::success($user), 200); // doesnt include roles relationship

        return Responder::success($user::with('roles')), 200); // works
        return Responder::success($user::with($request->input('include', [])), 200); // works
    }

What version of the package are you on? And have you linked a transformer to your user model?

v1.2.17 and yes I have this on User Model
...implements Transformable

---

public static function transformer()
    {
        return UserTransformer::class;
    }

Hmm, and the load_relations_from_parameter configuration is left to the default 'include' value I suppose? The automatic including is performed in the service provider: https://github.com/flugger/laravel-responder/blob/master/src/ResponderServiceProvider.php#L123
Does the following work for you:
return Responder::transform($user)->include('roles')->respond(200);

This is basically how the automatic including works. You may also set a whitelist of which relations are allowed using the technique above using the $relations property in your transformer, however, that should default to ['*'], which whitelists all relations.

Yes, its default 'include'.
That line works. 👍
and actually its the only one of those I posted earlier which works. These lines includes the relationship, but for some reason they return all Users, not only the one with id=1
return Responder::success($user::with('roles')), 200);
return Responder::success($user::with($request->input('include', [])), 200);

This works perfectly, but doesnt look so nice:
return Responder::transform($user)->include($request->input('include', []))->respond(200);

When using with() you're getting a query builder, you want to load relations on a set model, so something like this should work:
return Responder::success($user->load('roles')), 200);

However, with the load_relations_from_parameter set, you shouldn't need to explicitly load relations. Could you try adding the following code on line 123 in Flugg\Responder\ResponderServiceProvider:
dd($this->app[Request::class]->input($parameter, []));

Does that output 'roles'?

It outputs []

Hmm, that's the problem then. Does dd($parameter); output 'include'? And what do you get when you do dd($this->app[Request::class]->all());?

dd($parameter) = "include"
and
dd($this->app[Request::class]->all()) = []


[WIP] skip gemm in bwd.

 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.


[SPARK-10540] Fixes flaky all-data-type test

This PR breaks the original test case into multiple ones (one test case for each data type). In this way, test failure output can be much more readable.
Within each test case, we build a table with two columns, one of them is for the data type to test, the other is an "index" column, which is used to sort the DataFrame and workaround [SPARK-10591] 1

Merged build triggered.

Merged build started.

Test build #42488 has started for   PR 8768 at commit 9fd75c2.

Test build #42488 has finished for   PR 8768 at commit 9fd75c2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42488/
Test PASSed.

Merged build finished. Test PASSed.

Can we let every test run more times (50 or 100 times) before we merge it?

@yhuai OK, I'll make a temporary commit to trigger such a Jenkins build and revert it later.

@yhuai BTW, I did try to run every test 100 times locally, and it works.

Merged build triggered.

Merged build started.

Test build #42624 has started for   PR 8768 at commit 9c4fee1.

oh, then it is fine. We can revert the temp commit and test it again. Once jenkins passes, we can merge it.

Test build #42624 has finished for   PR 8768 at commit 9c4fee1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42624/
Test PASSed.

Merged build triggered.

Merged build started.

LGTM. Once jenkins is good, I will merge it to master and branch 1.5.

Test build #42671 has started for   PR 8768 at commit 5e3361a.

Test build #42671 has finished for   PR 8768 at commit 5e3361a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42671/
Test PASSed.

Thanks! Merging.


Check if the name of the Project is valid

What this PR makes?
Update the bsvNew.py file to add a new function to check if the name is a legal name according to [1]
Summary of the Changes

create function is_legal_name. The rest of the cases refereed in [1] are cover by the condition of the Capital Letter
check the validity of the given name inside the function in the main

Rerefences
[1] Bluespec SystemVerilog Reference Guide, 2.2 Identifiers and keywords

This pull Request was made to avoid the next errors:
$ bsvNew ActionValue
Creating base directories.
Creating local BSV_TOOLS path file .bsv_tools (Add to your .gitignore)
Creating gitignore
Creating makefile
Creating main module

$ make
/usr/bin/mkdir -p build
/usr/bin/mkdir -p bs_axi4_lite/include
checking package dependencies
Error: "bs_axi4_lite/src/TestsMainTest.bsv", line 4, column 12: (P0005)
  Unexpected BSV 3.8 keyword 'ActionValue'; expected foreign module, foreign
  function, or import
make: *** [bluespec/BSVTools/scripts/rules.mk:126: compile] Error 1

$ bsvNew Axi4-Slave
Creating base directories.
Creating local BSV_TOOLS path file .bsv_tools (Add to your .gitignore)
Creating gitignore
Creating makefile
Creating main module

$ make
/usr/bin/mkdir -p build
/usr/bin/mkdir -p bs_axi4_lite/include
checking package dependencies
Error: "bs_axi4_lite/src/TestsMainTest.bsv", line 4, column 16: (P0005)
  Unexpected `-'; expected `::'
make: *** [bluespec/BSVTools/scripts/rules.mk:126: compile] Error 1


CSS needed for example code highlighting to work

Hi
your example needs the prism css for highlighting to work
ie add the line
import 'prismjs/themes/prism.css'

It's mentioned in the README that you might need to import some CSS https://github.com/satya164/react-simple-code-editor/blob/master/README.md#usage

@satya164 So show the whole working demo, with any syntax theme really.

@FateRiddle maybe you'd find the full ass demo in the example folder if you spent 2 minutes looking in the project instead of being rude on GitHub issues.

@satya164 I was rude because I find your answer to a legit question passively aggressive and not helpful.
People using this editor may not want to style it all by themselves but also prefer a wide range of theme choices. So a custom styled example might not be a perfect answer.

@FateRiddle how is linking to the docs passive aggressive? it's an open source project and I work on it on my free time. I don't have time to write detailed answer to every github issue. if my answer is not satisfying to you, feel free to send PR to improve the README or answer in issues. anyway, that doesn't mean you should be rude.
regardless, this library doesn't do syntax highlighting. it only renders a text area over your syntax highlighted code. you can use prism or any other library you want. how you make syntax highlighting work is outside the scope of this library. if you're setting up prism, you need to follow prism docs for that, or any docs of the lib you're using. this is already mentioned in README. but if it's not clear, then PR welcome to make it more clearer.


[ENG-390] Set up placeholder routes and waffles for submission

Duplicate of #707 for easier merging.
ENG-390

Pull Request Test Coverage Report for Build 3196

41 of 47 (87.23%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 75.587%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/registries/addon/drafts/draft/form/route.ts
5
6
83.33%

lib/registries/addon/drafts/draft/metadata/route.ts
5
6
83.33%

lib/registries/addon/drafts/draft/review/route.ts
5
6
83.33%

lib/registries/addon/drafts/index/route.ts
5
6
83.33%

lib/registries/addon/forms/help/route.ts
5
6
83.33%

lib/registries/addon/forms/index/route.ts
5
6
83.33%

Totals

Change from base Build 3194:
-0.01%

Covered Lines:
6914

Relevant Lines:
8039

💛  - Coveralls


[SMALLFIX]Fixed the style of method names in UnderStorageSystemInterfaceIntegrationTest

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7693/
Test PASSed.

LGTM


[BUG] - getaddrinfo ENOTFOUND web.whatsapp.net when using downloadMediaMessage for stickerMessage

Describe the bug I'm encountering an issue where trying to download a sticker message using the downloadMediaMessage function results in an error. The error message indicates a failure to resolve the address web.whatsapp.net.
Steps to reproduce the behavior:
Receive a sticker message in the application.
Attempt to download the sticker using the following code:
buffer = await downloadMediaMessage(
    msg,
    'buffer',
    {}
)

getaddrinfo ENOTFOUND web.whatsapp.net

started to happen day 10/10

Same

same

Same

Same, the sticker message looks like this:
StickerMessage {
  url: 'https://web.whatsapp.net',
  fileSha256: Uint8Array(32) [],
  fileEncSha256: Uint8Array(32) [ ],
  mediaKey: Uint8Array(32) [ ],
  mimetype: 'image/webp',
  height: 125,
  width: 125,
  directPath: '/v/t62.15575-24/35735771_919333640142241_8735230311634381553_n.enc?ccb=11-4&oh=01_Q5AaIIfqFN8JajRBWxF8cEG7H_Um75d8gwXMH8qyNpW_Y9ls&oe=67594EB0&_nc_sid=5e03e0',
  fileLength: Long { low: 19276, high: 0, unsigned: true },
  mediaKeyTimestamp: Long { low: 515959760, high: 402, unsigned: false },
  stickerSentTs: Long { low: 452930615, high: 403, unsigned: false }
}

I noticed that the "url" is "https://web.whatsapp.net" instead of "https://mmg.whatsapp.net"+directpath.
So for stickers a i've manually set the parameters for downloadContentFromMessage( ):
    const directPath = message.directPath
    const mediaKey = message.mediaKey
    const url = "https://mmg.whatsapp.net"+directPath+"&mms3=true"
    await downloadContentFromMessage({directPath, mediaKey, url}, context);

Tested and its working, Hope that it will be fixed soon.


Use xip.io instead of modifying /etc/hosts

If you use http://xip.io/, no need to modify /etc/hosts (or system32 in windows ...).

Thank your for the tip! Do you have any hints to help us using it?

Hey,
for example, instead of using :

traefik.my-awesome-project.local use the domain traefik.my-project.<IP_ADDRESS>.xip.io will do the same, but no host modification needed.

So you just need to modify the domain in your config file i guess.

Great, I'll take a look and try it!


Create adcet

Please add my college in your list.

@vivek8097 Please rename lib/domains/edu/adcet to lib/domains/edu/adcet.txt to follow this repository rules. Thank you!

@vivek8097 Also, please indicate what is the official web site of the school, because it cannot be reached at adcet.edu.

to where i add my college website.??
On Mon, Apr 17, 2017 at 11:55 PM, Philip Torchinsky <notifications<EMAIL_ADDRESS>wrote:
@vivek8097https://github.com/vivek8097 Also, please indicate what is the official web site of the school, because it cannot be reached at adcet.eduhttp://adcet.edu.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/JetBrains/swot/pull/2938#issuecomment-294551263, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANgRZS463rJWReT807X7lzcwQMKllYr1ks5rw66AgaJpZM4M6UtV.

@vivek8097 You can answer this message by email (if you receive notification emails from GitHub) or in the comment field at the request page (https://github.com/JetBrains/swot/pull/2938). I found the college with the name which you have indicated, and the website does not have emails @adcet.edu at the contact page: http://www.adcet.in/ADCET/ContactUS.php. What is a domain part of your email - adcet.edu or adcet.in? If the latter, please rename the file path in your request from lib/domains/edu/adcet to lib/domains/in/adcet.txt. In any case, the file must have an extension .txt. Thank you!

@vivek8097 I am closing this request because it duplicates #2958.

Dear sir,
My previous pull request to add adcet.txt in lib/domain/edu  is wrong because my college email id end with adcet.inhttp://adcet.in so i need to add my adcet.txt inside lib/domain/in folder.Please sir remove adcet.txt from lib/domain/edu and approve my pull request for lib/domain/in.
Regards,
Vivek singh
On 19-Apr-2017 11:03 pm, "Philip Torchinsky" <notifications<EMAIL_ADDRESS>wrote:
@vivek8097https://github.com/vivek8097 I am closing this request because it duplicates #2958https://github.com/JetBrains/swot/pull/2958.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/JetBrains/swot/pull/2938#issuecomment-295357655, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANgRZYlW8QQO-mvoYAjbXEl0mP8TMi60ks5rxkUHgaJpZM4M6UtV.


Silent Known Folder Move

What should be the behavouir when the folders already exist on the Onedrive location and, there is data inside both online and local folders. Do the folders merge with the local folders silently? We use version 19.192.0926.0012.

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

ID: a3a0cebd-7e82-6631-bdee-031b3b84f74d
Version Independent ID: ee04db29-a6db-2874-2573-d80f0b415308
Content: Block syncing of specific file types
Content Source: OneDrive/block-file-types.md
Service: one-drive
GitHub Login: @kaarins
Microsoft Alias: kaarins

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

@BakkerJan The article contains a note now which describes the behavior:
"If users already have blocked file types in their OneDrive, the files will sync to their computer, but any changes they make on their computer won't be uploaded."

@BakkerJan 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.


feat: Add config option to ignore files based on Unix shell-style wildcard pattern matching

This adds the skip_file_globs config option to enable skipping files based on a Unix shell-style glob. This also updates the skip_files config option to treat file paths as OS-agnostic.
fixes #21
fixes: #20

@pawamoy let me know if there are any changes necessary in this PR.

Thanks for the PR!

You are welcome. Let me know if there are any other changes needed.

Thanks for the PR!

You are welcome. Let me know if there are any other changes needed.

@pawamoy, I think this is ready now.

Yep, thanks for the notification, I'll review this tomorrow :slightly_smiling_face:


[backport] Fix true_divide with dtype argument

Backport of #2076

[automatic post] Jenkins, test this please.

Force-pushed.
Please check the diffs and trigger Jenkins. >reviewer

Jenkins CI test (for commit 4626d6108bb9108b05a4935f5bb901cafbde45f7, target branch v7) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 14f7cd70c0da1502e017f009cefa4466d27d4b8f, target branch v7) succeeded!

LGTM.


Higher kinded monoid and semigroup for arrays

🚀 Feature request
Current Behavior
It has not been implemented. Or I have been unable to find it.
Desired Behavior
To have a way to concat Kind<M, A[]> types.
Posible use case:

Generically implement collect from Task<Either<E, A>>[] or IO<Either<E, A>>[]

Example:
export const collect = <M extends URIS>(m: Monad1<M>) => <A>(
  te: Kind<M, A>[],
): Kind<M, A[]> => {
  const monoid = getHKTArrayMonoid<M, A>(m); // this is the function I am proposing

  return pipe(
    te,
    foldMap(monoid)(a => m.map(a, v => [v])),
  );
};

NOTE: Implement this function generically is my primary interest.
Suggested Solution
export const getHKTArraySemigroup: <M extends URIS, A>(
  m: Monad1<M>,
) => Semigroup<Kind<M, A[]>> = m => ({
  concat: (x, y) => m.chain(x, a => m.map(y, b => concat(a)(b))),
});

export const getHKTArrayMonoid: <M extends URIS, A>(
  m: Monad1<M>,
) => Monoid<Kind<M, A[]>> = m => ({
  empty: m.of([] as any),
  concat: (x, y) => m.chain(x, a => m.map(y, b => concat(a)(b))),
});

NOTE: I am not sure those are the best names :sweat_smile:
Describe alternatives you've considered
I have tried to use direct monoid and semigroup implementations from Array but it is different from what I am proposing.
Your environment

Software
Version(s)

fp-ts
2.5.3

TypeScript
3.8.2

@frndmg collect is sequence (from Traversable) which only requires an Applicative instance
import * as A from 'fp-ts/lib/Array'
import * as IE from 'fp-ts/lib/IOEither'
import * as TE from 'fp-ts/lib/TaskEither'

declare const tes: Array<TE.TaskEither<string, number>>

// result1: TE.TaskEither<string, number[]>
const result1 = A.array.sequence(TE.taskEither)(tes)

declare const ies: Array<IE.IOEither<string, number>>

// result2: IE.IOEither<string, number[]>
const result2 = A.array.sequence(IE.ioEither)(ies)

@gcanti Sorry for my late reply.
The issue here is that I want the whole list of Eithers, not just the accumulation of the results because it brokes when an error occurs if I use sequence.
Example:
import * as E from 'fp-ts/lib/Either'
import * as T from 'fp-ts/lib/Task'
import * as TE from 'fp-ts/lib/TaskEither'

declare const tes: Array<TE.TaskEither<string, number>>

// result1: T.Task<Array<E.Either<string, number>>>
const result1 = collect(T.task)(tes)

@frndmg that's sequence again, configured with T.task this time
import * as A from 'fp-ts/lib/Array'
import * as T from 'fp-ts/lib/Task'
import * as TE from 'fp-ts/lib/TaskEither'

declare const tes: Array<TE.TaskEither<string, number>>

// result1: T.Task<Array<E.Either<string, number>>>
const result1 = A.array.sequence(T.task)(tes)

@gcanti sorry to hijack this issue to ask this but how would you sequenseS Options that would keep only the Somes and ignore the Nones...
Because currently and in the example in the code it will be none...
So instead of this
const adoOption = sequenceS(option)

    assert.deepStrictEqual(adoOption({ a: some(1), b: none }), none)

return this
    assert.deepStrictEqual(whatToDoHere({ a: some(1), b: none }), some({ a: 1 }))

@gkamperis Looks like https://github.com/gcanti/fp-ts/blob/6fe1808dedd879be1add109c27709ced2cb3c753/src/ReadonlyRecord.ts#L716-L726

@raveclassic thanks! Am using array.compact all the time... Need to read about compactables... :)


expand readme.md

Can you expand the readme to include examples of running things individually instead of needing ansible. Only mining farms will want to use ansible, the average user will want to run stand-alone.

Good point. We should probably have a discussion about power management and overclocking, would the lambda user want them enabled by default?

No, overclocking and power management should be a very specific per card and you don't want to have default settings that could potentially damage cards without having tested them. I would say you even want to put a disclaimer in here somewhere that states this.

Currently each card has a default (safe?) overclocking/undervolting configuration.
I have to think about how to configure the target and each GPU separately.


Layout Boxed bug when resize page

When you use layout-boxed much times it gives an error of jQuery, because the _fix function in $.AdminLTE.controlSidebar, delegate itself to the resize method of page when it's layout-boxed, so after some resizing it turn to like 2k calls of the same method.

I understand what you did here but unfortunately it still doesn't fix the issue. I tested your code and the boxed layout still jams after a few resizes. We are currently working on a fix for that.
Thanks!

Why not? Here it works :T

Issue resolved.
Thanks!


502 Bad Gateway

All the creation went successfully and service_url was logged.  Resources: 7 added, 0 changed, 0 destroyed.
Outputs:
artifact_bucket_id = <bucket_name>
mlflow_password =
mlflow_username =
port = 8080
service_url =
status = "RUNNING"

Update:
url dosen't work with postfix :8080

Hi, @SyedHamza0196 sorry for the late answer.
Have you fixed this issue?