Rewriting documentation

[x] Home
[x] Getting started

[x] Requirements
[x] Guidelines
[x] Level: First Steps
[x] Skin: First Steps

[x] Tools

[x] Content packing tools

[x] Fast XNB Builder
[x] XNBCLI
[x] TConvert
[x] Monogame Pipeline
[x] PNG to XNB

[x] Level helping tools

[x] Hitbox2Screens
[x] JumpKingManager

[x] Level Making

[x] Hitboxes
[x] Blocks
[x] Screens
[x] Props

[x] World Items
[x] Hidden Walls
[x] NPCs
[x] Merchants
[x] Ravens
[x] Messages

[x] King
[x] GUI
[x] Audio
[x] Particles
[x] Tips

[x] Skin Making
[x] Publishing
[x] Updating
[x] Credits
[x] Porting content from JK+ [external]

Notes:

Getting started: Maybe add "and use" after creation in the text under "What is the Workshop?"
Getting started has a link to "Sample custom level" that currently leads nowhere
Rules and guidelines: Remove the "don't" under the header, pretty sure it is the opposite of the intended message right now
Guidelines: Mentioned before I'm pretty sure but "won't endorse" -> "is unlikely to endorse" and "won't" -> "do not" under the "Polished levels" section
"Tips" is not in the table of contents
Publishing: Same change as with guidelines with phrasing
Publishing: Looks like there's some WIP content, is the documentation actually done then for the section?
Porting content: Link does not work

Done.

Improvements:

[ ] Level Making: Add a "File Types" subpage under "Level Making" listing all file types used for level making with explanation of what they do. Explain XML and XNB on an entry level
[ ] ALL: Clean up/cut text that has personal flavour in them (only informative text should remain)
[ ] Hitboxes: Swap out the picture in "Hitboxes" with a similar but exclude "fake block" in the picture. Add image text explaining that block types will be explained in the "Blocks" subpage


Join the mailing list to engage in conversations and get faster support at<EMAIL_ADDRESS>

If you have triaged this as a bug, then file an issue directly.

Describe the problem you faced
We are running Glue streaming with Hudi 0.12.1 to write data from Kafka. The table is MOR for quick ingestion and commit method is inline as we don't want to run another job to perform compaction.
Initially each commit took 1.5 minutes. But after one day, it became slow to 3-4 minutes. After a few days, it became more than 10 minutes.
We noticed the slowness is due to Hudi was reading archived commits files after each commit. Because there were tons of commits files under .hoodie/archive directory on S3, it really took time. After I deleted those files, the speed became quick. But it would come down later.
I did some research and I found currently Hudi does not support purge these archived files automatically. There is a feature to merge small archived files. I tried it and the situation became better. But it was not fast enough.
Further research shows that might be related to sync hive meta. But I do need to sync. Is there any way to speed up this procedure? Thank you.
A clear and concise description of the problem.
To Reproduce
Steps to reproduce the behavior:

Set up a Kafka

Write pyspark to read stream from Kafka and then sink to hudi on S3.

Hudi config:
 'className' : 'org.apache.hudi',
 'hoodie.datasource.hive_sync.use_jdbc':'false',
 'hoodie.datasource.write.partitionpath.field': partitionKey,
 'hoodie.datasource.write.recordkey.field': primaryKey,
 'hoodie.datasource.write.precombine.field': timestamp,
 'hoodie.datasource.write.operation': 'insert',
 'hoodie.table.name': targetTableName,
 'hoodie.consistency.check.enabled': 'true',
 'hoodie.datasource.write.hive_style_partitioning':'true',
 'hoodie.datasource.write.table.type': 'MERGE_ON_READ',
 'hoodie.datasource.hive_sync.database': targetDBName,
 'hoodie.datasource.hive_sync.table': targetTableName,
 'hoodie.datasource.hive_sync.enable': 'true',
 'hoodie.datasource.hive_sync.partition_fields': partitionKey,
 'hoodie.datasource.hive_sync.partition_extractor_class': 'org.apache.hudi.hive.MultiPartKeysValueExtractor',
 'hoodie.datasource.hive_sync.support_timestamp':'true',
 'hoodie.datasource.write.keygenerator.class': 'org.apache.hudi.keygen.SimpleKeyGenerator',
 'hoodie.archive.merge.enable':'true',
 'hoodie.index.type':'BLOOM',  #SIMPLE
 'hoodie.compact.inline': 'true',

Expected behavior
A clear and concise description of what you expected to happen.
Environment Description

Hudi version : 0.12.1

Spark version : 3.1 (AWS Glue 3.0)

Hive version :

Hadoop version :

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) :

Additional context
Add any other context about the problem here.
Stacktrace
Add the stacktrace of the error.

Just happened to find the PR https://github.com/apache/hudi/pull/7561. It seems be helpful. Is that true? Thank you.

yes, we already fixed it and will be part of 0.13.0
thanks for reporting.
will go ahead and close out the issue since its already fixed.


test(bridge-withdrawer): add black box tests

Summary
This adds black box tests for the bridge withdrawer.
Background
The bridge withdrawer needs tests.
Changes

test helpers for the submitter
sanity check that batches are submitted properly
tests to verify that failures during submissions causes the submitter to halt

Testing

[x] helpers

[x] setup
[x] response mocks

[x] genesis
[x] get_nonce
[x] broadcast_tx_commit

[x] failure mocks

[x] get_nonce
[x] broadcast_tx_commit

[x] tests

[x] submit sucess
[x] submit check_tx failure
[x] submit deliver_tx failure

From what I can see these are not running the bridge as a blackbox, at least in the Rust put-it-under-tests/-sense.
Can you move the tests from inside the crate to outside, as for the other services?


Self Service Onboarding

Self-Service Onboarding spec:
https://docs.google.com/document/d/1un2ALk3dH2AGo0KzpHZiCe8PA-vhXOWm3ayTECGlgwA/edit?pli=1#
V0 - Publish a page off the current simplereport.gov site, with the current docs and videos (Starting Sprint 8, into 9)
V1 - Refine and design the information architecture (Sprint 9 Feb. 1-13)
V2 - Dev & HHS Tool Integration (Sprint 10 Feb. 15-29)
V3 -
V4 -

v0 via Slack:

Launching the pages with the:

intro content Willys created
survey link
videos
guides we currently have

Can you send me a link to each of these?
Do you have an idea of the IA you want?
What do we want for the basic page layout (e.g. USWDS docs template)?

@willusds


Cleanup the consumer interfaces and legacy code

Deprecated:

PartitionLevelConsumer: Use PartitionGroupConsumer instead
In PartitionGroupConsumer:

Remove endOffset from fetchMessages() because offset might not be monotonically increasing for some consumers (e.g. Kinesis), and we should not use end offset to filter messages

In MessageBatch:

getMessageAtIndex(): Use getStreamMessage() instead
getMessageBytesAtIndex(): Use getStreamMessage() instead
getMessageLengthAtIndex(): Use getStreamMessage() instead
getMessageOffsetAtIndex(): Use getStreamMessage() instead, where offset info is embedded in the StreamMessageMetadata
getMetadataAtIndex(): Use getStreamMessage() instead
getNextStreamMessageOffsetAtIndex(): Use getOffsetOfNextBatch() instead
getNextStreamPartitionMsgOffsetAtIndex(): Use getOffsetOfNextBatch() instead

In StreamPartitionMsgOffset:

fromString(): Should be a static method

Removed Metadata Extractor interfaces because they are not well designed and not really pluggable (plug at consumer level should be good enough)
Add offset info into RowMetadata:

StreamPartitionMsgOffset getOffset()
StreamPartitionMsgOffset getNextOffset()

Bugfix:

With offset info properly attached to message metadata, we can correctly track the offset for each message. Currently we are using the next offset for each message

Clean up the code to adapt the above changes.

Codecov Report
Attention: Patch coverage is 42.55814% with 247 lines in your changes are missing coverage. Please review.

Project coverage is 27.85%. Comparing base (59551e4) to head (26b5023).
Report is 156 commits behind head on master.

Files
Patch %
Lines

...apache/pinot/plugin/stream/pulsar/PulsarUtils.java
52.22%
38 Missing and 5 partials :warning:

...lugin/stream/kinesis/KinesisConnectionHandler.java
5.88%
32 Missing :warning:

...e/pinot/plugin/stream/kinesis/KinesisConsumer.java
53.96%
20 Missing and 9 partials :warning:

...a/manager/realtime/RealtimeSegmentDataManager.java
0.00%
22 Missing :warning:

...apache/pinot/spi/stream/StreamMessageMetadata.java
0.00%
21 Missing :warning:

...java/org/apache/pinot/spi/stream/MessageBatch.java
0.00%
20 Missing :warning:

...in/stream/kafka20/KafkaPartitionLevelConsumer.java
64.51%
9 Missing and 2 partials :warning:

...ava/org/apache/pinot/spi/stream/StreamMessage.java
0.00%
11 Missing :warning:

...in/stream/kinesis/KinesisPartitionGroupOffset.java
33.33%
10 Missing :warning:

...in/stream/pulsar/PulsarPartitionLevelConsumer.java
61.53%
5 Missing and 5 partials :warning:

... and 13 more

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12697       +/-   ##
=============================================
- Coverage     61.75%   27.85%   -33.90%
+ Complexity      207      192       -15
=============================================
  Files          2436     2453       +17
  Lines        133233   134237     +1004
  Branches      20636    20792      +156
=============================================
- Hits          82274    37398    -44876
- Misses        44911    93993    +49082
+ Partials       6048     2846     -3202

Flag
Coverage Δ

custom-integration1
?

integration
?

integration1
?

integration2
?

java-11
?

java-21
27.85% <42.55%> (-33.77%)
:arrow_down:

skip-bytebuffers-false
27.85% <42.55%> (-33.89%)
:arrow_down:

skip-bytebuffers-true
?

temurin
27.85% <42.55%> (-33.90%)
:arrow_down:

unittests
27.85% <42.55%> (-33.89%)
:arrow_down:

unittests1
?

unittests2
27.85% <42.55%> (+0.12%)
:arrow_up:

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

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

I've reverted the change of removing separate executor for Kinesis and Pulsar to make this PR only focus on the interface change

Thanks @Jackie-Jiang for cleaning up and fixing these interfaces. It helps address issues like #https://github.com/apache/pinot/pull/12602


Add poweron architecture ppc64le to travis build

This is part of the Ubuntu distribution for ppc64le. This helps us simplify testing later when distributions are re-building and re-releasing,For more info tag @gerrith3.
We typically build applications for customers and ISVs, and while we don't use this package directly, we do count on all of the packages in debian/ubuntu to build other packages. So we more likely have this as a second or third level dependency and couldn't tell you explicitly which features we use or our usage model.

Pls help to review and approve

any update on this pls

any update on this pls


Issue with Route Grouping

This doesn't exclude the route grouping folder(s) in Nextjs.
ex. Folder structure (users), (marketing), (auth), etc.
This adds the route grouping folder/route to the route
ex. /(auth)/login, /(patientFacing)/appointments/[appointmentId], /(user)/settings
instead of /login, /appointments/[appointmentId], /settings.

The code is there to handle route groups. Please provide a repro case.


the quantnized model can not run with error, when set inference batch_size=4、8、16

Thanks for your method for the quantized of llama, and when i try this code, i have face some problem. when i modify the code to support batch generate, such as batch_size=4
python3 llama_inference.py /data1/llama-7b/ --wbits 4 --load llama7b-4bit.pt --text "My name is Julien and I like to make" --max_length 128

the llama_inference.py was changed as follows:
tokenizer = AutoTokenizer.from_pretrained(args.model)

    text = [args.text] * 16

    for idx, bs in enumerate([4]):
        t0 = time.time()
        input_ids = tokenizer.batch_encode_plus(text[ :bs], return_tensors="pt").input_ids.to(DEV)
        # print(input_ids)
        with torch.no_grad():
            generated_ids = model.generate(
                input_ids,
                do_sample=True,
                min_length=args.min_length,
                max_length=args.max_length,
                top_p=args.top_p,
                temperature=args.temperature,
            )
        torch.cuda.synchronize()
        time_consume = time.time() - t0
        print(f"consume  time: {time_consume*1000:.2f} ms")

        print(tokenizer.batch_decode(generated_ids))
        torch.cuda.empty_cache()
        gc.collect()

i have face the error

Changed to use Triton instead of CUDA now. It seems that this problem won't happen.

Changed to use Triton instead of CUDA now. It seems that this problem won't happen.

After i run the new code from your repo, the error still occurs occasionally。 And the speed is slower 10x than pytorch in fp16

Your measurement seems wrong.
Change your code to the following and test.
    model.to(DEV)
    tokenizer = AutoTokenizer.from_pretrained(args.model)
    text = [args.text] * 64

    for idx, bs in enumerate([4,4,4,4]):
        t0 = time.time()
        input_ids = tokenizer.batch_encode_plus(text[ :bs], return_tensors="pt").input_ids.to(DEV)
        # print(input_ids)
        with torch.no_grad():
            generated_ids = model.generate(
                input_ids,
                do_sample=True,
                min_length=args.min_length,
                max_length=args.max_length,
                top_p=args.top_p,
                temperature=args.temperature,
            )
        torch.cuda.synchronize()
        time_consume = time.time() - t0
        print(f"consume  time: {time_consume*1000:.2f} ms")

        print(tokenizer.batch_decode(generated_ids))
        torch.cuda.empty_cache()
        gc.collect()

Since the first batch consumes a lot of time due to triton, it is correct to compare from the second batch.
batch 4 result: 50 token / ms
cuda: 1688.47ms
triton: 2437.59ms

Your measurement seems wrong. Change your code to the following and test.
    model.to(DEV)
    tokenizer = AutoTokenizer.from_pretrained(args.model)
    text = [args.text] * 64

    for idx, bs in enumerate([4,4,4,4]):
        t0 = time.time()
        input_ids = tokenizer.batch_encode_plus(text[ :bs], return_tensors="pt").input_ids.to(DEV)
        # print(input_ids)
        with torch.no_grad():
            generated_ids = model.generate(
                input_ids,
                do_sample=True,
                min_length=args.min_length,
                max_length=args.max_length,
                top_p=args.top_p,
                temperature=args.temperature,
            )
        torch.cuda.synchronize()
        time_consume = time.time() - t0
        print(f"consume  time: {time_consume*1000:.2f} ms")

        print(tokenizer.batch_decode(generated_ids))
        torch.cuda.empty_cache()
        gc.collect()

Since the first batch consumes a lot of time due to triton, it is correct to compare from the second batch.

batch 4 result: 50 token / ms cuda: 1688.47ms triton: 2437.59ms

Thanks for your reply !
I change my code , my GPU is 3090Ti, max_length=128
and my result:
triton: 5927.86 ms


Brightness and Contrast

Bug
Brightness and Contrast in the LBRY client are incorrect. This is effecting both the light and dark modes. The videos, thumbnails and everything else are dim.
Screenshots
In the following screen shots I compare the LBRY client to lbry.tv in Firefox.

System Configuration
Ubuntu 19.10
LBRY 0.37.2
Additional context
This issue may have occured as a result of upgrading from Ubuntu 18.04 to 19.10. I tried changing the themes in gnome tweaks but it had no effect.

Thanks for reaching out! We recently pushed out a redesign on lbry.tv but it hasn't made it into a desktop app update - that's probably the discrepancy you are seeing. It should be out later this week.

Thanks for reaching out! We recently pushed out a redesign on lbry.tv but it hasn't made it into a desktop app update - that's probably the discrepancy you are seeing. It should be out later this week.

That would account for the different colors, being a electron app the CSS for lbry.tv might be different.
How ever, I do not think that would account for the dark videos and thumbnails.
The issue with the videos and thumbnails has gone away, it might be because I restarted my computer.

The CSS is the same. @tuxfoo it looks like you're an active dev, if you want to contribute to LBRY it's pretty easy to get started: https://lbry.tech/contribute

This issue seems to be reoccurring.

Launching lbry with the following argument fixes the issue. The colours will then be correct and the videos are no longer dim.
-force-color-profile

The bug appears to be in electron.
https://github.com/electron/electron/issues/10732


CommonIdea

We currently have that a NamedIdea may have an abbreviation.  This is good.  But there are times when there are ideas around which we insist must have an abbreviation.  Cueing from the definition

Any convenient short form used as a substitution for an understood or inferred whole.

from https://en.wiktionary.org/wiki/abbreviation, I think we should introduce a sub-class of NamedIdea which assuredly has an abbreviation.  [Note that reading about the generic concept of abbreviation at https://en.wikipedia.org/wiki/Abbreviation is rather informative].  We make heavy use of "must have" abbreviations in our referencing system, for example.
One of the combinators we should have (on NamedIdea!) is short, which will return the abbreviation (if it exists), otherwise the term.  We can then use short while constructing "templates" for the more informal sentences that make up the SRS.

Sounds good. I take it we'll also need to create a new data structure for Concepts which are an instance of this new sub-class as well?
On another note, what should we name the sub-class? My first thought was "Short"/"ShortForm", though that doesn't sound very descriptive, and other names like "AbbreviatedIdea" don't sound great to me.

I meant to use CommonIdea for that class.  An idea becomes 'common' when it has a short form.
And yes, it will need a new data-structure.

This issue has been resolved as of the inclusion of Language.Drasil.Chunk.CommonIdea


Practice App: Creating Models and Serializers

What's up?
We have designed our ER Diagrams as documented in #94. Now corresponding models and serializers must be created.
To Do
Following actions must be taken:

[x] Create models:

[x] Create user model that extends the default User model provided by Django.
[x] Create comment model.
[x] Create tag model.
[x] Create art item model.
[x] Create follow model.

[x] Make migrations and apply these migrations.
[x] Register the models within the admin.py. Login as an admin and check if tables are listed properly.
[ ] Create serializers:

[ ] Create user serializer.
[ ] Create comment serializer.
[ ] Create tag serializer.
[ ] Create art item serializer.
[ ] Create follow serializer.

Deadline
Deadline: 11.05.2022 @22.00
Additional Information
Contributors:

@BElifb
@KarahanS

Reviewers
No response

Reviewed pull request #93 and merged it do the dev branch. Serializers.py will be updated as we develop our APIs.

Created models, serializers, updated some things and migrated. More details at #92

Went over pull request #97 changes and issues, here's the related comment.

Thanks for the review. Responded to your comment here.

Updated structure image fields.
Added default images.
Removed media from .gitignore to keep images.
commit
Tested the default image.
Tested the fact that django in fact changes image file names in case of contradiction. (You may see it in media folder)

Added a username() method to the myUserSerializer in order to fetch the id of a user in JSON format.
Added created_at field to the Follow model.
Please refer to #99 for details.

The models.py file suffices the tasks and requirements of our application.For milestone 2 it seems it is finalized. If further development is needed. one can open this issue. Closing the issue


Multi-tenancy support

We would like to add multi-tenancy support. There is a partially implemented version of this called Teams.

There are some notes on this in [this doc](https://docs.google.com/document/d/1ELpv0TNkEjJfmxepHqThEJbHyFMzxXNeZWBLdhMw7Ws/edit#_


Using 'v-for in array.sort()'  in dynamic components causes infinite update loop

Version
2.6.14
Reproduction link
codesandbox.io
Steps to reproduce
in dynamic components, use Array.sort() in v-for
What is expected?
run without any warning.
What is actually happening?
In console,
[Vue warn]: You may have an infinite update loop in a component render function.

sort mutates the array. That's a reactive operation. Reactive operations during render result in an infinite update loop, that's expected.
Possible Solutions include makeing a copy of the array befor sorting, i.e. Array.from(testData).sort( ....)


Singularization Not Working Correctly for Certain Field Names

Issue Description
The singularization process in the config generator is not working correctly in certain cases. For example, the field name orderStatus is being incorrectly converted to OrderStatu instead of OrderStatus.
Example:
Given the following configuration:
  - curl:
      src: "https://example.com/api/v2/order/597ad6e9-ee1a-406a-842f-c9c62564d24e/status/"
      fieldName: "orderStatus"
      headers: *headers

Current Behavior:
The generated GraphQL query incorrectly singularizes the type name:
orderStatus(id: String!): OrderStatu @http(path: "/api/v2/order/{{.args.id}}/status/")

Expected Behavior:
The singularization should correctly handle the field name:
orderStatus(id: String!): OrderStatus @http(path: "/api/v2/order/{{.args.id}}/status/")

/bounty 25$

/attempt #2564

/attempt #2564


feat: support new request context

Which issue does this PR close?
Closes #
Rationale for this change
Adopt with latest protobuf

What changes are included in this PR?

Extract schema from database field in request context

Are there any user-facing changes?
Yes, SDK will pass request context

How does this change test
CI

Codecov Report

Merging #636 (14185f8) into main (2819d5e) will increase coverage by 0.02%.
The diff coverage is 13.63%.

:exclamation: Current head 14185f8 differs from pull request most recent head 8d01ab7. Consider uploading reports for the commit 8d01ab7 to get more accurate results

@@            Coverage Diff             @@
##             main     #636      +/-   ##
==========================================
+ Coverage   66.41%   66.43%   +0.02%
==========================================
  Files         283      283
  Lines       44257    44252       -5
==========================================
+ Hits        29393    29401       +8
+ Misses      14864    14851      -13

Impacted Files
Coverage Δ

router/src/lib.rs
50.00% <ø> (ø)

server/src/grpc/storage_service/mod.rs
42.95% <0.00%> (+0.75%)
:arrow_up:

server/src/grpc/storage_service/prom_query.rs
54.42% <ø> (ø)

server/src/grpc/storage_service/sql_query.rs
0.00% <ø> (ø)

server/src/grpc/storage_service/write.rs
38.72% <0.00%> (-0.07%)
:arrow_down:

server/src/handlers/prom.rs
68.44% <0.00%> (+0.33%)
:arrow_up:

sql/src/frontend.rs
0.00% <ø> (ø)

server/src/grpc/forward.rs
79.92% <100.00%> (+0.44%)
:arrow_up:

analytic_engine/src/manifest/details.rs
97.51% <0.00%> (+0.19%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Update 9p mount syscall so mmap is rw

By default the mmap command for 9p is read only
Reason: generic_file_readonly_mmap is used by 9pfs Kernel side client
code, in case there is no caching.
For the guest OS, we need to ensure:

CONFIG_9P_FSCACHE is enabled in the guest kernel
v9f2_file_mmap is used instead of generic_file_readonly_mmap

For 2, we want cache=fscache to enable writable mmaps. This patch
addresses this requirement.
Fixes #92.
Signed-off-by: Eric Ernst<EMAIL_ADDRESS>

depends on https://github.com/clearcontainers/linux/pull/3

LGTM

LGTM

Does this need https://github.com/clearcontainers/linux/pull/3
What will happen if we have an older kernel.

As I look at this closer, I noticed a couple things:

we could use cache=mmap or
cache=fscache

In either case, I actually saw the failure go away in either case (seemingly the mmap resolves as rw), regardless of the guest kernel's config update. I'm asking Arindam for clarity on this next.
I'm hoping to take a look @ the performance numbers for this PR -- @grahamwhaley - can you help point me in the right direction (or help with a run if you aren't tracking this repo w/ localci?)

@egernst
Reading @rarindam's info on #92, the statement there is that the mmap will be readonly with un-cached 9p mounts, and thus it would seem either mmap or fscache caching would enable the rw mmap.
The relevant code I think will be these fs ops tables in the kernel 9p:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/fs/9p/vfs_file.c#n669
Thus, I'm not convinced the kernel config update is necessary (but I have not dug into it). I'll let @rarindam comment on that.
For performance, sure, we can run the fs metrics. I will note here also, this msize setting improvement in the same area never did land in the hyperstart repo, and I believe it is not in our local agent either - that is also stuck on getting 'metrics'. We should combine the efforts:
https://github.com/clearcontainers/hyperstart/pull/25

From my measurements, mmap is slightly more performant relative to fscache and meets our RW needs.  Will update the PR to use this cache mode instead.

Hi Eric, Graham,
Okay first things first:
From the 9pfs mount options, 2000L and acl are not required. These were part of my commandline, and I just pasted them, which I see have created a lot of confusion. So you can remove them, they are not relevant. Only thing required to be passed for 9p is cache=mmap OR fscache.
Second: It was from reading the code as Graham pointed out, that I discovered that by default, only READ only mmap is enabled in 9p. Hence when testpmd tries to acces a RW mmap area for rte config, it fails. Note: mmap call with RW will not fail. But trying to acces it through write, will fail.
The RW mmap is enabled (as part of different function table altogether) when one uses the mmap option while mounting.
Third: Its always good to have cache enabled to mmap or fscache, because no one knows which code will be using mmap in RW mode, which is actually a pretty common occurence.
Finally, if this issue is not being seen in 3.0 then there has been some change which I am not aware of, certainly for the better.
Arindam
On Aug 29, 2017, at 7:24 AM, Eric Ernst <notifications<EMAIL_ADDRESS>wrote:
From my measurements, mmap is slightly more performant relative to fscache and meets our RW needs. Will update the PR to use this cache mode instead.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/clearcontainers/agent/pull/93#issuecomment-325680405, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AbyHQ1VtPvL06Oe8k9Fu4NB-6jPAtbh_ks5sdB8DgaJpZM4PE_7Q.

Thanks @rarindam
The issue is observed in 3.0 -- this pull request is being made to address and fix it.  with cache=mmap, my issue is resolved.


Relation between Connection and IncomingStreams

On a successful connection in either direction (i.e., either actively via connect or passively via listen) I get two things eventually - one Connection and one IncomingStreams. As a user i expect these to be related at some level since they denote a connection to the peer. However it seems that closing the connection does nothing for the IncomingStreams.
Details:

I get a Connection and IncomingStreams from a peer while I'm listening
Something happens and I decide I don't want this peer any more
I get the Connection object and do Connection::close(...).
Nothing happens to the IncomingStreams - it continue to stay indefinitely with tokio event loop - I would have expected it to resolve to completion/failure at this point and destroy itself.
Remote peer still tries to send something to us on their connection to us (Which is still alive for them) by trying to open a new stream to us. The peer keeps getting "ConnectionAborted - closed by remote peer" error which is fine and expected.

So the IncomingStreams stream (and possibly any other stream obtained from it which the remote hasn't shutdown/closed/destroyed yet - though I haven't checked this part) uselessly remains unresolved with tokio. I have to now keep extra knowledge about closing these streams when I close the Connection to the peer.
Wouldn't the better/expected design be that when I close Connection (or drop/destroy it) all the related stuff resolve into an error (or anything, but resolve) to gracefully collect all resources ?

Yup, that seems like an oversight we should fix. Do you want to try a PR? We probably want a dropped flag in ConnectionInner similar to the one in EndpointInner; if IncomingStreams is polled and dropped is true, it should probably return Ready(None).

Is the IncomingStreams definitely notified when the drop flag is set ? Otherwise could be quite a while between the drop flag being set and when the stream is polled (which will probably be when the remote tries opening one to us making us be at the mercy of the remote peer). If that's true, maybe have IncomingStreams pass through a select (some stream or future selector) first which selects  IncomingStreams or Blah (whatever we name it) with Blah being a type that gets notified when something fires on drop and then resolves to Ready(None) or Err or whatever. E.g, Blah could be a oneshot or mpsc rx or something custom atop it or whatever makes sense

When the dropped flag is set, we should likely also notify the pending.incoming_streams_reader.
Note that a lot of this code will be overhauled soon, to the effect that ConnectionInners will no longer hold a reference to the EndpointInner, but will instead own a quinn::Connection and have some channels to facilitate communication between endpoint and connection.

This (and #252) is intended to be handled by the connection emitting ConnectionLost once it has been closed, causing Pending::fail to be called, which wakes up all blocked streams. However, on review, we only seem to do this when the peer explicitly acknowledges the close, which is optional and something we don't even do ourselves. I'll draft a fix tonight.

This, #251, and #252 should be fixed as of #271, if not earlier. Can you confirm?

hmm i tested #251 with commit https://github.com/djc/quinn/commit/e9b9a3861b9c26417d93757a673354cf6de11b9d in master and dropping connection still doesn't close it. I tried to see what the dtor of ConnectionRef (as that's probably going to be called when I drop the Connection) does and this:
https://github.com/djc/quinn/blob/aae9edecd48e5bc0e3b5ff1e8d328cd42b55a6ad/quinn/src/connection.rs#L272
happens to be 4 or something so probably since it's shared with IncomingStreams and other stuff, the ref count actually never reaches the correct value to trigger a close ?
However if i explicitly call close on it (which now seems to consume Connection): https://github.com/djc/quinn/blob/aae9edecd48e5bc0e3b5ff1e8d328cd42b55a6ad/quinn/src/connection.rs#L180
then yes, the IncomingStreams and all children streams waiting in read_to_end are resolved. So that issue is perhaps fixed now.
Also while testing i now see random panics from quinn when i explicit call close(). I cannot produce these in debug so my backtrace is probably useless but here's the relevant section:
 8: core::result::unwrap_failed
  9: <quinn::endpoint::Driver as futures::future::Future>::poll
 10: <futures::future::map_err::MapErr<A, F> as futures::future::Future>::poll

The panic is on some send-error:
panicked at 'called `Result::unwrap()` on an `Err` value: SendError("...")'
Just putting here for convenience but if you want i can open a separate issue.

The intention is that the connection will only be closed when both the Connection and the IncomingStreams are dropped -- otherwise you might still be using the connection through the other. And you might still have more references through any streams that were already open. In other words, if you intentionally want to close the connection, dropping is not enough -- but if all references that would allow you to use the connection go away, it should be implicitly closed.

In other words, if you intentionally want to close the connection, dropping is not enough

that contradicts the documentation a bit though:
https://github.com/djc/quinn/blob/aae9edecd48e5bc0e3b5ff1e8d328cd42b55a6ad/quinn/src/connection.rs#L121
Or at-least makes it a bit confusing in the context.
I think (but not surely ofc) there could be a better approach though. Is it because the design that we are constrained - perhaps because we are mismanaging ownership or using smart-pointers in-appropriately ? Just a braindump but could we have restricted the ownership to just say the Connection (since that's what the doc implies) and the rest have weak-ptrs instead of cloning strong-ptrs and making it ownership nightmare ? I find logic based on ref-counts to be fragile as that's what it's ("smart" ptrs) meant to abstract - otherwise we might have as well used raw-ptrs and done the bookkeeping.
/IncomingStreams is not something one is holding on to manually - it's spawned into the event loop and forgotten. The "handle" is the Connection and that is all that should count.

So the semantics seems (from reading the docs) similar to mpsc::Sender where if you drop the last copy of the Sender it causes the rx to exit (with error in most cases and with an Ok for tokio::..::watch)
Also the caveat on depending on ref-counts: https://doc.rust-lang.org/std/sync/struct.Arc.html#method.strong_count and in c++ too which is probably worse due to relaxed loading instead of seq-cst as in Rust:

the value returned by use_count is approximate (typical implementations use a memory_order_relaxed load)

So it seems to be a fragile design, but ofc this might be the best in the given context - just that managing logic depending on ref-count seems a little niche :)

Hmm, not sure. IncomingStreams is not spawned into the event loop, actually -- that's ConnectionDriver, and that's account for (it's why we check for reference count == 2). The semantics we were going for is that if you're holding any reference into the ConnectionInner that you can actually still use, we don't want to close the Connection implicitly. Conversely, we find it reasonable if you drop the Connection but hold on to the IncomingStreams, for example for server connections, and in that case clearly the connection should not be dropped.

So it seems to be a fragile design, but ofc this might be the best in the given context - just that managing logic depending on ref-count seems a little niche :)

This case seems to be a relatively safe. If the count == 2 inside a Drop, that means it's about to be reduced to 1 -- which should really always be the ConnectionDriver. Since we're returning a triple of ConnectionRef owners when the connection is established, that seems unlikely to happen in any other case unless the ConnectionDriver is not spawned correctly, which we should maybe more clearly document as user error.

not sure. IncomingStreams is not spawned into the event loop

By spawned into the event loop i mean it's consumed, converted into some leaf future and then spawned - so basically it's gone while the Connection remains as a handle. You can drop that to drop all incoming as well as outgoing stuff. Otherwise it's contradictory that Connection::close() then causes IncomingStreams and all it's children streams to exit.

You haven't convinced me yet that the current design is bad, although I would agree it needs better/clearer documentation!

You haven't convinced me yet that the current design is bad

np, i'm fine using it the way it is - to me it appears to have more maintenance hassle in future tracking ref-count and all - but that might be just me.

We really wanted to support the case of dropping the Connection but holding on to the IncomingStreams, for example for server connections

This part is confusing me a bit from usability perspective. Connection::close() does affect other things - if it affected only outgoing streams (e.g, write_to_end or open_bi/uin etc., resolved on its destruction/closure) then yes they are orthogonal and have to be managed separately. But on one hand we have something like a handle when we call close on, it is consumed and closes everything even the incoming stuff which it did not have to do anything with, on the other we are saying that we could drop the last copy of the Connection which also consuming it (well dropping is consuming) without it affecting everything, but just the outgoing.

Yes, that's exactly right:

Implicitly dropping things does not close the Connection until all the references into it are dropped, there is no explicit intent to close the connection
Explicitly closing will error out all references into it, because the user has communicated their intent to trigger that

OK will use it that way then.

mplicitly dropping things does not close the Connection until all the references into it are dropped, there is no explicit intent to close the connection

Bit strange to the user as they don't care about the "references into it" - they just care about how many copies of Connection they have since that's what they see.
Anyhoo, just so that the other issue with panic doesn't get lost in this discussion. Let me know if you want a separate issue for that one.

We think it would be strange if IncomingStreams would error out just because you dropped Connection, or alternatively, that it would be useful to be able to use IncomingStreams without holding on to Connection.
Probably good to file another issue for the panic!

The implicit close semantics are intended. In particular, it is necessary for this to work:
incoming_connections.for_each(|(driver, conn, streams)| {
    runtime.spawn(driver);
    runtime.spawn(streams.for_each(handle_request));
});

Note that the Connection handle is dropped immediately, but the user's intent is clearly to continue invoking handle_request on incoming streams until the peer closes the connection. By contrast, calling close expresses the intention that the entire connection should be immediately interrupted.

Also the caveat on depending on ref-counts: https://doc.rust-lang.org/std/sync/struct.Arc.html#method.strong_count and in c++ too which is probably worse due to relaxed loading instead of seq-cst as in Rust:

I think you've caught a subtle bug here, thanks! In particular, if two separate threads drop a ConnectionRef or EndpointRef simultaneously, and there are initially three references, then it is possible for neither to observe it reaching 2 references, so the implicit close will not take place. We can probably fix this by locking the connection's/endpoint's mutex before inspecting the count, thereby forcibly linearizing drops.

I'm happy to close this now as i can see all stuff regarding and spawning from this are resolved (at-least in my tests), or the admins can close it later if they think they need to work on this more somewhere.

We believe this to be solved, and if your tests are passing, that's good confirmation; thanks!


Chaining txs.

https://hackmd.io/G0uD0r52Qca89BCINuDByQ
- [x] Aggregate `pending_read_requests` in the private kernel circuits and verify them in the base rollup circuit.
- [ ] Include pending notes when selecting notes for the `get_notes` oracle call. If "read", add the note hash to `pending_read_requests`.
- [ ] Create a new PXE api to add pending note to the db.
- [ ] Add pending note data to the database when successfully submitting a tx.
- [ ] Monitor the tx and clear pending note data if the tx is dropped.
- [ ] Include chained txs in the rollup in the correct order.
- [ ] Discard a tx if one of its backward links is not a settled or pending note hash.

https://github.com/AztecProtocol/aztec-packages/pull/2869 addresses first task (minus verification, since we are about to migrate the cpp circuits to noir)


Updated Icon/Simple toggles

The Framer Motion Layout has been implemented correctly, based on my knowledge.
The current usage:

The parent component is a non-motion component with a non-animatable style property (in this case, Justify-content) that switches between 'end' and 'start'.
The child component is a motion component with a layout set to true.

Please note that animating x from 0 to 100% is more performant and reliable than manipulating the DOM. However, I updated the code based on @Ansub 's suggestion.
File modified:

src/showcase/components/toggle/IconToggle.tsx
src/showcase/components/toggle/SimpleToggle.tsx

@ITs-AIKO afaik Framer Motion animations are powered by JS and don't directly impact DOM performance in the same way that traditional CSS animations or transitions might do.
Am I missing out on something here?

CSS animations are usually faster and more efficient than animations made with JS, but they give us less control. We've been using Framer Motion for our animations, which is so optimized that we can't really tell the difference for small animations.
Let's say we want to animate a blur effect from 0 to 4px while also changing the x-scale of a div element. Using Framer Motion (with type: spring, bounce: 0, and ease: linear) will work great on most devices, but on some Android phones especially with battery saver mode on, it might feel a bit laggy. However, using CSS for the same animation will run way smoother. (without hardware acceleration)
But what I meant is that when animating (with JS) the translation of an element from point A to point B it is recommended to use the transform property (e.g., transform: translateX() which Framer Motion shortens to "x"). This is because the transform property is browser-optimized. But when using the Layout property, Framer Motion does all the calculations for us.

so instead of changing Margin, we should simply do transform-x?

so instead of changing Margin, we should simply do transform-x?

Yes! 'x'because we are using framer motion.

but i didn't get what the change is about in this PR then

but i didn't get what the change is about in this PR then

Used the layout property the right way.

The issue is that styling of this toggle is not consistent with the one in production, See this image:

whereas the toggle in your PR:

I prefer the uniform gap: where the gap in the side is the same as the gap on the top and bottom.
It the prod version, the gap is the same on all sides.
Can we modify the change to make this gap uniform? or was there a reason for the chance?


⚠ GitLab Server has degraded performance

In 56f0fc9, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 3303 ms

Resolved: GitLab Server performance has improved in 13eac73.


Use msys2 clang toolchain to build windows distribution

Closes #264.
Note: still a WIP, will undraft and ping for review after:

[x] Both x64/x86 builds are ready
[x] There's basic testing that msys2 dll dependencies don't leak. make check won't serve this purpose since it needs to be run in a msys2 shell, for now I believe clang --version and wasm-ld --version should suffice since they are the largest executables anyway.

@sunfishcode Ready for review.

Does the switch from windows-2019 to windows-latest (which is windows-2022) mean that we lose any support or older Windows versions in our distributed artifacts?

Does the switch from windows-2019 to windows-latest (which is windows-2022) mean that we lose any support or older Windows versions in our distributed artifacts?

The binaries will run fine on Windows 10 and later versions that ships ucrt in system32. If you wish I can also switch the build platform to windows-2019, though it doesn't provide any meaningful assurance in compatability, since the build process does not use systemwide visual studio installation at all.

I just started a Windows 7 SP1 x64 virtual machine, and the new artifacts work there as long as ucrt is installed. I believe that's a reasonable lowest bound on supported Windows versions. And this is not a regression from the old msvc-produced artifacts, those also require ucrt to be manually installed (and additionally msvcp140).

Great, thanks for investigating that! I think we can switch to this for our next release, which I'm hoping to put out soon.


How to update workflow variables from a simple task ?

Is it possible to update the value of a workflow variables from a simple task ? Without actually using a Set Variable task.

No. Workers can only update tasks.
https://netflix.github.io/conductor/labs/beginner/#return-response-add-logs
It is best to use only Set Variable tasks to update the value of variables.

Thanks


Downloading symbols Tests-ERM is not working. The remote server returned an error: (404) Not Found

Most of the symbols are downloaded by the Compile-AppInNavContainer.ps1 script (Test Runner, Tests-TestLibraries, etc.) but the scripts is trying to download an error appears  that the url http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=Tests-ERM&versionText=<IP_ADDRESS>&tenant=default is not found.
Using Symbols Folder: C:\Agent_work\1\s.packages
Downloading symbols: Microsoft_Test Runner_18.0.23013.23795.app
Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=Test Runner&versionText=<IP_ADDRESS>&tenant=default
Downloading symbols: Microsoft_Tests-TestLibraries_18.0.23013.23795.app
Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=Tests-TestLibraries&versionText=<IP_ADDRESS>&tenant=default
Downloading symbols: Microsoft_System Application Test Library_18.0.23013.23795.app
Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=System Application Test Library&versionText=<IP_ADDRESS>&tenant=default
Adding dependency to System Application from Microsoft
Downloading symbols: Microsoft_Library Variable Storage_18.0.23013.23795.app
Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=Library Variable Storage&versionText=<IP_ADDRESS>&tenant=default
Downloading symbols: Microsoft_Any_18.0.23013.23795.app
Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=Any&versionText=<IP_ADDRESS>&tenant=default
Downloading symbols: Microsoft_Library Assert_18.0.23013.23795.app
Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=Library Assert&versionText=<IP_ADDRESS>&tenant=default
Downloading symbols: Microsoft_Tests-ERM_<IP_ADDRESS>.app
Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=Tests-ERM&versionText=<IP_ADDRESS>&tenant=default
ERROR The remote server returned an error: (404) Not Found.
Details of the error are:
##[error]The remote server returned an error: (404) Not Found. No published package matches the provided arguments.
(ms-correlation-x = )
At C:\Program
Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.15-preview449\AppHandling\Compile-AppInNavContainer.ps1:382
char:21

                throw (GetExtenedErrorMessage $_.Exception)

                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (The remote serv...rrelation-x = ):String) [], RuntimeException
FullyQualifiedErrorId : The remote server returned an error: (404) Not Found. No published package matches the p
rovided arguments. (ms-correlation-x = )
##[error]PowerShell exited with code '1'.

What could be the cause of this issue?

That is likely because Tests-ERM was not published.
All the other apps seems to be libraries you get when including only TestLibraries.
Tests-ERM are actual tests, which needs to be published as well.
If you include the script used to create the container and the full output, then it would be possible to see whether this is true.

Hi Freddy,
Sorry for the late reply. Hereby I send you the requested files. Hopefully you can help me. Thanks in advance
Output log.txt
_CICD.zip

In the latest version of the CI/CD samples, the settings file has a InstallTestApps property, which is designed for installing apps, which your test apps depend on.
Scripts in the scripts folder have been updated accordingly to use this property - you need to update the scripts.
Then you should be able to set that to ":C:\Applications\BaseApp\Test\Microsoft_Tests-ERM.app"(include the starting :, which means that the reference is to a file inside the container)
Note that you cannot run this pipeline on a cloud based environment, as Tests-ERM cannot be published to a cloud sandbox - only to containers.

Hi Freddy,
Thanks.  Thanks in advance

on Github:
PTE sample: https://github.com/BusinessCentralApps/HelloWorld
AppSource App sample: https://github.com/BusinessCentralApps/HelloWorld.AppSource
or on Azure DevOps (identical to the samples on Github)
PTE sample: https://dev.azure.com/businesscentralapps/HelloWorld
AppSource App sample: https://dev.azure.com/businesscentralapps/HelloWorld.AppSource

I was doing some testing today with the installTestApps parameter and receive the error below after publishing the Tests-ERM app in the Install Test Apps part of the Run-AIPipeline:
Extension compilation failed
2021-07-10T16:41:29.8857552Z ERMIntrastatJournal.Codeunit.al(1074,30): error AL0132: 'TestPage Intrastat Journal' does not contain a definition for 'CreateFile'
2021-07-10T16:41:29.8870296Z ERMIntrastatJournal.Codeunit.al(1144,30): error AL0132: 'TestPage Intrastat Journal' does not contain a definition for 'CreateFile'
2021-07-10T16:41:29.8882181Z ERMIntrastatJournal.Codeunit.al(1190,30): error AL0132: 'TestPage Intrastat Journal' does not contain a definition for 'CreateFile'
2021-07-10T16:41:29.8892923Z at , : line 40
When I add the below script manually in the Run-AIPipeline.ps1 script the publishing is going well. What do I wrong?
$testsERMApp = "C:\bcartifacts.cache\sandbox\18.0.23013.23795\nl\Applications.NL\Microsoft_Tests-ERM_18.0.23013.23795.app"
$Parameters = @{
"containerName" = $containerName
"tenant" = $tenant
"credential" = $credential
"appFile" = $testsERMApp
"skipVerification" = $true
"sync" = $true
"install" = $true
}
Invoke-Command -ScriptBlock $PublishBcContainerApp -ArgumentList $Parameters
Thanks in advance.
See the attachment with the total log.
Log.txt


Do we default to the uploader as creator for projects and works?

Should we allow entry of "Other" (non-vdc) people?
What does the creator really mean?

I think we need to find a way to differentiate between author and uploader, as they will not necessarily be the same person. And when it comes to citing this data, people will be confused if the true author is not easily identified.


[QUESTION] npm install via link

I was wondering. how can i install a particullar package via link using the require function.
For example. I have a private package, I usually install the package using npm install github_url_to_package.
there is a way to install a package that is installed via a url?. if not. there is a workaround ?

https://github.com/extremeheat/JSPyBridge/blob/master/docs/python.md#built-ins

You can install a package internally by using python3 -m javascript --install <npm package>. Internally, whatever you place after --update will be passed to npm install <...>. For example, use python3 -m javascript --install PrismarineJS/vec3 to install the vec3 package from git.****

if you have a custom registry you can use https://stackoverflow.com/a/35624145/11173996
if you want a local package, try npm install -g to install it globally

Thank you for your answer. That will help me alot


Page Always Reload When Clicked

I have problem with vue-pdf-app, it's always reload/refresh the page when i clicked zoom in/zoom out also it will refresh if i click other toolbar button
Here i meet same issue with this code
https://codesandbox.io/s/bold-darkness-tfbpp?file=/src/components/ModalPDFReader.vue:2993-3004

A temporary workaround is to stop the button click event propagation. Suppose you installed vue-pdf-app in a div called "vuePdfApp", you can add the following JavaScript code to stop event propagation:
$(document).on('click', '#vuePdfApp button', evt => {     evt.stopPropagation();     return false; });


add Commonly Proposed Changes section

https://lists.swift.org/pipermail/swift-evolution/Week-of-Mon-20151214/003799.html

Awesome, please move this to the very bottom of the page though (under "Accepted proposals for Swift 2.2") and I'll merge it.  Thanks again.

Thanks for working on this! I'd rather this content be in a separate document, because it's going to grow long, and moving the content to the bottom will be less discoverable. Instead, let's have a link to the separate document (e.g., commonly_proposed.md) in the first bulleted list of README.md.

Ah, that makes sense to me, thanks Doug.

moved to its own file and added a couple more entries. thanks!

Cool, please squash these and I'll merge it.  Thanks again!

Closing this in favor of https://github.com/apple/swift-evolution/pull/72


Changed how the prediction is printed

Prediction will now show
Class: The Class
Confidence Score: (The confidence score e.g. 0.99989)

/gcbrun


Allow the static javascripts and css to be bundled with search.html

meaning?

Currently we have a ability to pass a custom search.html but there is no way to bundle static assets with it so I want to allow user to pass static folder that would contain all the static assets which will be later added to javascript and css folder.


Decouple option values of regex literal modifiers

The compiler currently passes the integer value of the compiler's version of Regex::Options when expanding a RegexLiteral.
For example /cat/i expands to Regex.new("cat", Regex::Options.new(1)) (with Regex::Options::CASELESS = 0x00000001).
This constitutes a strong dependency between the compiler and Regex implementation because they need to agree on what value represents what. With such a tight coupling it's impossible to change the stdlib implementation (as discussed in #13152).
The compiler should use logical names to represent the modifier options instead of integer values.
I expect this can be done in a backwards-compatible way so that the compiler would also be able to build an older stdlib.
Extracted from https://github.com/crystal-lang/crystal/issues/13152#issuecomment-1482615941

@bcardiff mentioned in https://github.com/crystal-lang/crystal/issues/13152#issuecomment-1482760232

For the regex options I would suggest to add a known method by the compiler. Regex.make_options that would receive boolean named arguments.
What type is returned will only need to match the constructor.
/cat/i becomes Regex.new("cat", Regex.make_options(i: true))
The compiler can check the presence of make_options method. If not defined the current logic/values are used. The new convention would be supported by both engines and decouple the compiler from it going forward.

@brian's API proposal should work for that. But we can go a step further to a single method: Regex.literal. This reduces the interface between compiler and stdlib and thus gives more freedom to the implementation.
Regex.literal("cat", i: true)

The .literal method has been discussed before (I don't recall where exactly though) and I think this would be a perfect use case for it.

Why not use an array instead?
Regex.literal("cat", [:i])

👍  on Regex.literal("cat", i: true). Seems like a nice and clear API.


ADS1256 SPI analog addon

This PR is for a new addon for the ADS1256, a low noise 24-bit 8-channel Analog-to-Digital converter that communicates over SPI. This addon is modeled after the existing ADS1219 addon, so extra features like deadzone config and autocalibration are not present. This work includes:

Implemented SPI driver in PicoPeripherals library, including support for DMA (not used for ADS1256, but there when needed)
Ported this ADS1256 library into GP2040-CE and uses the PicoPeripherals SPI driver
Fixed support for FLOAT and DOUBLE in the Protobuf implementation. The macros needed to support those data types were not present in config_utils.cpp.
Added ADS1256 section to the Addons page

For the addons page, SPI pulls from the configured peripherals and the addon will not allow configuration if at least one SPI block is not configured, similar to the ADS1219 addon. It will also present a dropdown to select the CS pin using a combination of the selected SPI block's HW CS pin and other available pins.

Nice, I'll do a once over but this is looking clean to me


Added examples to read me documentation

I use the library at my company and added some very useful examples that I think others can find useful.

Thanks @mtpjr88. The examples are definitely useful but I would revise the proposal a bit to make it lighter (and specific to node-hubspot)

I showed examples of exactly how hubspot expects your payloads. Do you have specifics so I can revise it?

would you be able to merge with the changes I have?

awesome thank you!


I got gm_key_64.gz from the site

When I am on the license.download website to download the software and key. The key download as gm_key_64.gz and I can’t seem to unzip this file. Finder suggested the file should be gm_key_64.tar.gz

Hi @angus821,
Thank you for your interest in finder. That's fine, as long as you have the key in whichever compressed form is fine.
Thanks.

thanks @sagnikbanerjee15 for the amazing tool. I just zipped the file used:gunzip gm_key_64.gz
is that all right?
thanks again.

Hi @BenAawf ,
Sorry for the late reply. Yes, that should work.
Thank you.


Fix close button not using the tab's text color (#8046)

Summary of the Pull Request
This sets the tab close button color to match the tab text color.

References
#8046

PR Checklist

[x] Closes #8046
[x] CLA signed. If not, go over here and sign the CLA
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #8046

Detailed Description of the Pull Request / Additional comments
This sets the tab close button color to match the tab text color.

Validation Steps Performed
Test light theme white tab mouse hover effect:

Test dark theme white tab mouse hover effect:

@zadjii-msft , @dcheatha - I guess this one is blocked on the conflict - the name of the Tab was changed to TerminalTab

@dcheatha can you take care of the conflict? It shouldn't be too bad considering it's really just a renamed file that git can't follow 🤷♂

@zadjii-msft Hi, I rebased the commit off of main. I hope that's okay!


[FEATURE] Highlight email input if its wrong.

Description
Highlight email input if its invalid (e.g. name@example. ). Also display a message below email input.
(very simple but I would be happy to contribute :) )

demo:

Looks good! Thank you


clustered management - fix missing thread delegation when receiving management call requests

When using the client communicator, the execution should be delegated to another thread.
So this PR is to use the ehcache execution service to create an executor specialized in executing the management call requests in another thread.

ok, I think I got it :

when a new DefaultClusteringManagementService is constructed, pass it a configuration
when the DefaultClusteringManagementService starts, we retrieve the executor to be used, using the alias from the configuration (if null, it will use the default pool alias)
when the DefaultClusteringManagementService switches to AVAILABLE, wrap the executor retrieved previously in a LoggingExecutor that will log RuntimeExceptions

LGTM, nice enhancement to the threads use in ehcache3 management calls
@lorban @ljacomet feel free to add your comments, otherwise I'll merge it tomorrow (if still based on current master)


Make token optional in ITokenThemeRule interface

Theme options should allow rules without token.
Example :
    /// https://microsoft.github.io/monaco-editor/playground.html#customizing-the-appearence-exposed-colors
    monaco.editor.defineTheme('myTheme', {
        rules: [{ background: 'EDF9FA' }],
    });

The monaco.d.ts file is a generated file. It is generated from VS Code sources. In this case, the truth of the interface is at https://github.com/microsoft/vscode/blob/da765520e8092fd39384744ae90147cd1bbcdefa/src/vs/editor/common/modes/supports/tokenization.ts#L9-L14


Tensor-Train Density Estimation

https://arxiv.org/pdf/2108.00089.pdf

Started on 1 - 4 - 2024 , should end on 4 4 2024


[service discovery] read inject process metadata

What does this PR do?
Match the decoding of injected process metadata of https://github.com/DataDog/auto_inject/pull/404/files.
Added a matcher to avoid licence checks with tinylib msgp generated files. Tinylib always sets the generated file comment on the 3rd line, after the package
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

merging to be with the agent freeze, can make tiny adjustments if required during the QA week (injector PR is being reviewed)

/merge


"No such file or directorytabase_1of2.zip" when installing ee

Hi!
I have tried to create the docker image for Database <IP_ADDRESS> EE in a vagrant oel 7.2 box and in an offficial oraclelinux docker container via
./buildDockerImage.sh -v <IP_ADDRESS> -e
I have cloned this repository and provided my windows directory via volumes into the vagrant box and the oraclelinux docker container and added the downloaded files from otn:
[vagrant@oraclelinux7 dockerfiles]$ pwd
/docker-images/OracleDatabase/dockerfiles
[vagrant@oraclelinux7 dockerfiles]$ ls -la <IP_ADDRESS>/linuxamd64_12102_database_*
-rwxrwxrwx 1 vagrant vagrant<PHONE_NUMBER> Jul 23 05:13 <IP_ADDRESS>/linuxamd64_12102_database_1of2.zip
-rwxrwxrwx 1 vagrant vagrant<PHONE_NUMBER> Jul 23 05:07 <IP_ADDRESS>/linuxamd64_12102_database_2of2.zip
In both cases I get following error:
[vagrant@oraclelinux7 dockerfiles]$ ./buildDockerImage.sh -v <IP_ADDRESS> -e
Checking if required packages are present and valid...
: No such file or directorytabase_1of2.zip
: FAILED open or readbase_1of2.zip
linuxamd64_12102_database_2of2.zip: OK
md5sum: WARNING: 1 listed file could not be read
MD5 for required packages to build this image did not match!
Make sure to download missing files in folder <IP_ADDRESS>.
Any idea whats wrong with this?
Kind regards
Torsten

Similar error with xe version:
[vagrant@oraclelinux7 dockerfiles]$ ls -la <IP_ADDRESS>/oracle-xe-11.2.0-1.0.x86_64.rpm.zip
-rwxrwxrwx 1 vagrant vagrant 315891481 Jul 23 05:51 <IP_ADDRESS>/oracle-xe-11.2.0-1.0.x86_64.rpm.zip
[vagrant@oraclelinux7 dockerfiles]$ ./buildDockerImage.sh -v <IP_ADDRESS> -x -p
./buildDockerImage.sh: option requires an argument -- p
Unknown error while processing options inside buildDockerImage.sh
Checking if required packages are present and valid...
: No such file or directory0.x86_64.rpm.zip
: FAILED open or readx86_64.rpm.zip
md5sum: WARNING: 1 listed file could not be read
MD5 for required packages to build this image did not match!

You have to download Oracle DB binaries. See README.

Please see above, that I have download this:
vagrant@oraclelinux7 dockerfiles]$ ls -la <IP_ADDRESS>/linuxamd64_12102_database_*
-rwxrwxrwx 1 vagrant vagrant<PHONE_NUMBER> Jul 23 05:13 <IP_ADDRESS>/linuxamd64_12102_database_1of2.zip
-rwxrwxrwx 1 vagrant vagrant<PHONE_NUMBER> Jul 23 05:07 <IP_ADDRESS>/linuxamd64_12102_database_2of2.zip
This seems a problem of the mdsum, when I use the -i switch I have now start this, but it is not ready at the moment. Could it be, that checksum does not work when the folders / files are provided via a vagrant/docker volume mapping?

This is a weird problem. On your first try it did find the second zip file:
[vagrant@oraclelinux7 dockerfiles]$ ./buildDockerImage.sh -v <IP_ADDRESS> -e
Checking if required packages are present and valid...
: No such file or directorytabase_1of2.zip
: FAILED open or readbase_1of2.zip
linuxamd64_12102_database_2of2.zip: OK
md5sum: WARNING: 1 listed file could not be read
MD5 for required packages to build this image did not match!
Make sure to download missing files in folder <IP_ADDRESS>.
We heard before that the comments in the Checksum file can give some issues on some platforms. Can you try and delete all "#" marked lines and just leave the checksums themselves in the files.

No this does not change the behaviour - only skipping via switch -i helps to prevent this error.

This works fine when running directly on the windows host.
This give the mentioned error when running on the vagrant oel 7.2 client on the shared folder from the host.
This gives the error too when running on the oel 7.2 docker container running in the oel 7.2 client mapping the volume from the vagrant oel 7.2 client.

Hm, can you do a ls -al on the first install zip file in the vagrant environment? It almost sounds like as if vagrant doesn't expose the file to the environment.

See several updates before - the file is there, again:
ls -al linuxamd64_12102_database_1of2.zip
-rwxrwxrwx 1 vagrant vagrant<PHONE_NUMBER> Jul 23 05:13 linuxamd64_12102_database_1of2.zip

Did you try to execute the checksum command? Does that work within the vagrant environment?

Again: See 2 uptates above!

@tkleiber skip the checksum. It is just an optional verification the buildDockerImage.sh shell script does. This script is only intended to help anyway, not required at all to be used.
You may just as well go into the 12..1.0.2 folder and call docker build <options>.

If the md5sum Unix command itself isn't working with the vagrant environment there is little we can do to fix that.

Have found a solution now.
It seem that the Checksum.*file is in dos/windows UTF 8 format, at least when cloning the github repository on windows 10.
If this is mapped into the vagrant box, md5sum does gives the error.
If I convert the Checksum.* files to unix ansi format then the md5sum call is successful.


cannot import name 'PromptTemplateConfig' from 'semantic_kernel.prompt_template' (unknown location)

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
import asyncio
from semantic_kernel import Kernel as sk
from semantic_kernel.connectors.ai.open_ai import OpenAIChatCompletion, AzureChatCompletion
from semantic_kernel.prompt_template import PromptTemplateConfig
from semantic_kernel.utils.settings import openai_settings_from_dot_env, azure_openai_settings_from_dot_env
Error Message and Stack Trace (if applicable)
Exception has occurred: ImportError
cannot import name 'PromptTemplateConfig' from 'semantic_kernel.prompt_template' (unknown location)
File "C:\Users\TINGLE\Downloads\1\import asyncio.py", line 4, in
from semantic_kernel.prompt_template import PromptTemplateConfig
ImportError: cannot import name 'PromptTemplateConfig' from 'semantic_kernel.prompt_template' (unknown location)
Description
Getting this error while trying to import the below libraries:
System Info
NA

wrong library


refactor and test voucher

@marfavi merged develop again. Can't Update Branch from here because merge conflicts


Update Dockerfile for new setup.py changes

Updated to include changes from: https://github.com/apache/beam/blob/master/sdks/python/setup.py
I will also CP this to the release branch. This is not technically a release blocker, but it would make the release branch consistent within itself.

Run python validatescontainer

Run Python Dataflow Validatescontainer

https://github.com/apache/beam/pull/5628 is the CP PR.


Homepage badges aren't clickable

The badges in the homepage such as JavaDocs and Maven Central aren't clickable. It would be nice if they could be made clickable so that it would be easy to access the corresponding resources from the website.
Screenshot to clarify the badges I'm referring to

Side plugs

In case the logo found in the website is indeed the one chosen for the library, consider updating the same in the GitHub org page too 🙂
Also, the website link could be updated in the repository description on GitHub. This might mean the wiki may get less discoverable. We could address that by possibly linking to the wiki from the website so that it isn't missed.

cc @wumpz

Greetings!

In case the logo found in the website is indeed the one chosen for the library, consider updating the same in the GitHub org page too

Please elaborate, since the logos are identical.

Please elaborate, since the logos are identical.

I suppose you're mentioning about the logo in the README and website. My suggestion is to upload the logo in the organization page. Right now it only shows the default logo.
)

Thank you for the clarification.
You are right of course but only @wumpz was able to do that and as far as I understand it, he does not have any spare time at the moment.

I will make the Badges clickable though.

fixed via https://github.com/JSQLParser/JSqlParser/commit/be55d269536ab7916a5b5b0550d1e53ad90698f8


On receiving new props, use defaultSort if sortBy is false

If columns have changed on receiving new props, this.state.currentSort
may reference a column which no longer exists. It should then be
updated to nextProps.defaultSort if it exists.

Can you add a test covering this please?

I have a test for this but it works using new code in my other PR: https://github.com/glittershark/reactable/pull/298 , so I'll wait for that to be merged. Thanks

I think this is ready :)

@glittershark Hi. I'm running this in production now and would love to get this merged upstream. Thanks!


⚠ Minecraft has degraded performance

In e6b0bf3, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1032 ms

Resolved: Minecraft performance has improved in 80cd91c.


Increase timeout to 15m for Multicluster e2e test

Multicluster e2e test was killed because the default timeout is not enough sometime, I have seen below errors in old patch releases, so increase it to 15m.
*** Test killed with quit: ran too long (11m0s).

/test-multicluster-e2e

/test-multicluster-e2e


1.3.1 Release

[ ] Tag issue being addressed
[ ] Add tests
[ ] Update docstrings/documentation, if applicable
[x] Update changelog
[x] All tests successfully pass

Codecov Report

Merging #143 (0896d1c) into master (377862a) will decrease coverage by 0.20%.
The diff coverage is 60.00%.


fix(disabled ui): loop safe check when disabled #105

Alaska Airlines Pull Request
This is a safe check so we do not enter an infinite loop when a radio group is disabled and we push key "up"
this is the current state when we push arrow "up" on a disabled auro-radio-group:

Fixes: #105
Summary:
This changes affect the disabled button and do not disrupt current functionality.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)
[X] bug

Checklist:

[X] My update follows the CONTRIBUTING guidelines of this project
[X] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 1.6.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Edit Rectangles Sizes/ Text Values in python

I think its self explainitory if i say if this would be possible?
https://www.figma.com/file/f6RWVRQx4TNfn8gh5bXjPb/Untitled?node-id=0%3A1
I would have to edit the size of the rectangles due to values...

I did not get what you meant but I checked the file and it had incorrect naming and groups.

Yes i just wanna know if i can edit the size of rectangles in python

Sorry for late response.
Yes you can edit the size.

ok thanks!


fix: handle code fences better

Markdown has two code fences syntaxes: multiple backticks and multiple tildes. It also supports nested code fences. For example
An admonition:

````ad-note

```git
+ print("hello")
- print("world")
```

~~~python
print("hello world")
~~~
````

This pull request makes the plugin support the more general Markdown code fences syntax.

Thanks!


🌐 Add Spanish translation for docs/es/docs/how-to/testing-database.md

🌐 Add Spanish translation for testing-database documentation
File: docs\es\docs\how-to\testing-database.md

Good job! 🙈

Cancel PR because contains three file translate.
I'm going to do three separate PR
Sorry


Dashboard code cleanup

Lighthouse is flagging some unused dashboard js which is interesting and could merit a generic code clean up to make sure t's are crossed and i's are dotted (so to say) given the updates we made this summer.
You can see this by running lighthouse perf on the dashboard before and after making the update. Remember that locally running lighthouse is not the same as running it on staging (our dev environment is wayyyyy slower than prod).

After some investigation, I believe that no changes are necessary now.
Running lighthouse in an incognito window on staging showed us a few javascript files that were unused on first load:

Ignoring chrome extensions, we have

wp-dashboard chunk
wp-admin load scripts (php)
vendors-wp-dashboard
lodash

wp-dashboard and vendors-wp-dashboard are already separated out into their own chunks so we do not need to do anything there.
The wp-admin load scripts aren't something that affect the dashboard either, and are also loaded in a different chunk.
Which leaves us with lodash. We don't want to load all of lodash in, but the app hardly uses it. Turns out that lodash is loaded in the react-colors package:

This is also loaded in apart from the main chunk. I believe for now we don't need to do anything about the files listed in the lighthouse audit.


Wrong syntax in bibtex (DOCS)

System information
Irrelevant
Describe the problem
The bibtex entry for citing the Tensorflow white paper has incorrect syntax: \'{\i} should be \'{i} in the first author name.
Source code / logs
See also: https://tex.stackexchange.com/questions/23817/biber-writes-incorrect-unicode-encoding-of-i

Looking into this.  Are you sure that shouldn't be {'{\i}} given the link you had?
Or would it make more sense to just make the fancy i in unicode and put it in that way?

It's definitely \'{i} and the unicode character would cause trouble in bibtex. Only biber is able to cope with unicode characters (for as far as I know).

Do you want to submit a pullrequest or should one of us fix it internally?  I'm not sure I can test it properly is why I hesitate to do it myself.

@dr4b There is little to test, I'm afraid. It's just a matter of using correct syntax. As I am still not quite ready to sign legal documents for the purpose of deleting a single character in a .bib entry, it would be nice if you could do that.

  How can I do that?

If you just want to see the \'{i} rendered, make a file test.tex with the following code:
\documentclass[border=1pt]{standalone}

\begin{document}
  Mart\'{i}n
\end{document}

Then compile this, e.g. using pdflatex with the command pdflatex test.tex. This will generate a PDF, which contains the rendered result.
If you have no system with LaTeX installed, try out this online LaTeX previewer.

It has been 14 days with no activity and this issue has an assignee.Please update the label and/or status accordingly.


Do you have pre-trained model ready for use?

Hi,
I am currently looking into your great work. I download the data and the code but find it will take too much time even with GPUs to train the model. So I am wondering whether a pre-trained model (RECON, or RECON_EAC, etc.) can be provided? Many thanks.
Dennis

Hi,
I am currently looking into your great work. I download the data and the code but find it will take too much time even with GPUs to train the model. So I am wondering whether a pre-trained model (RECON, or RECON_EAC, etc.) can be provided? Many thanks.
Dennis
Hello, how many results did you reproduce in wikidata

Hello,
unfortunately, due to compute issues, we dont have pre-trained models. However, errors have been removed from the code, and we pushed latest changes. Please try to execute latest code. thank you.


v6.6.0-rc1 fails to build with --with-numa option

Original author: Nicolas Bock (@nicolasbock)
Original issue: https://charm.cs.illinois.edu/redmine/issues/357

When building with
./build charm++ net-linux-amd64 --build-shared --enable-charmdebug --enable-tracing --enable-tracing-commthread --with-numa -j1 -O0 -g -pipe
The build fails at:
../bin/charmc  -O0 -g -pipe -build-shared -c -I. memoryaffinity.c
memoryaffinity.c: In function ‘CmiInitMemAffinity’:
memoryaffinity.c:205:9: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement]
cc1: some warnings being treated as errors
Fatal Error by charmc in directory /var/tmp/portage/sys-cluster/charm-6.6.0_rc1/work/charm-6.6.0_rc1/net-linux-amd64/tmp
   Command x86_64-pc-linux-gnu-gcc -Werror=declaration-after-statement -m64 -DCMK_GFORTRAN -I../bin/../include -D__CHARMC__=1 -I. -O0 -g -pipe -fno-stack-protector -fPIC -c memoryaffinity.c -o memoryaffinity.o returned error code 1
charmc exiting...
gmake: *** [memoryaffinity.o] Error 1
-------------------------------------------------
Charm++ NOT BUILT. Either cd into net-linux-amd64/tmp and try
to resolve the problems yourself, visit
http://charm.cs.illinois.edu/
for more information. Otherwise, email the developers at charm`cs.illinois.edu

Without the --with-numa option, it works.

Original date: 2013-11-19 23:28:52

Fix at http://charm.cs.uiuc.edu/gerrit/#/c/11/ awaiting review.

Original date: 2013-11-19 23:32:36

Reviewed, looks good. Merged into charm.

Original author: Nicolas Bock (@nicolasbock)
Original date: 2013-11-20 04:45:34

Thanks! That was fast! The patch works for me as well.


chore: Execution trace control buttons disabled state

Add a disabled style to the execution trace control buttons. All buttons are disabled if there is no compiled program. The step back button is disabled on the first step. The step next and continue execution buttons are disabled on the last step.

i can take this

Hey @melnikga, assigned to you 👍


Github actions Go Unit tests fail sometimes (TestGetStats)

I'm submitting a ...

bug report

Traffic Control components affected ...

Traffic Control Client
Traffic Monitor
Traffic Ops

Current behavior:

When you make a commit or create a PR, Github actions will run a set of tests on your code. The Golang unit tests task sometimes fails due to a couple of known culprits. Look here: https://github.com/srijeet0406/trafficcontrol/runs/1150200903
I haven't been able to reproduce this locally, but maybe if you run the test multiple times, it will eventually fail (because it seems like a race condition atm).
Expected / new behavior:

All github action tasks pass consistently.
Minimal reproduction of the problem with instructions:

This happens intermittently. This is one example: https://github.com/srijeet0406/trafficcontrol/runs/1150200903
Anything else:

The failing test is:
=== RUN TestGetStats TestGetStats: datareq_test.go:248: expected getStats TotalMemoryMB > 0, actual: '0' --- FAIL: TestGetStats (0.01s)

The line where that's failing is checking the actual memory of the machine running the test using runtime, so for small virtual machines dedicated to running Go unit tests and nothing else it's not unreasonable that they'd be using less than a megabyte of memory. So I think that particular expectation of the test should be removed.

So I think that particular expectation of the test should be removed.

what do u think @srijeet0406 ?

I agree, that test expectation can be unreasonable at times. Also, I haven't seen this issue happen for a while now, not sure if anything changed from our side.

I'm going to close. please reopen if we see this again.


AnnotationRandomizerRegistry should cache and reuse randomizers to improve performance

The current implementation of AnnotationRandomizerRegistry creates a new randomizer for each random object, which is not ideal for perfomance.
Randomizer randomizer = (Randomizer) field.getAnnotation(Randomizer.class);
Class<?> type = randomizer.value();
RandomizerArgument[] arguments = randomizer.args();
return ReflectionUtils.newInstance(type, arguments);

@benas would it be a good solution to add a Map<Field, Randomizer<?>> similar to the one in CustomRandomizerRegistry and populate the map for every new encountered field? I'd like to try my luck with a PR :)

This is awesome! Thank you for reporting the case and for the PR 👍  Closing this after merging #366 .


Custom OAuth clientRedirectUri handling

As seen in #27, I did not find a way to pass a custom redirect uri (my app have many) to Botkit. This small PR add this capability.

Damn. Did not see #28 and made a duplicate :(

Yep, this is a duplicate of #28. Seems like naming is the biggest difference. Do you have any strong opinions about if we should merge this one or #28?


Menu of StatusNotifier in wrong place with multiple monitors

Video with the bug: https://youtu.be/jZkR-MZvbZE
I have two monitors, one horizontal, one vertical, when using StatusNotifier I get menu in wrong place
notifier_defaults = dict(
    menu_background=colours[0],
    menu_font='FiraCode Nerd Font',
    menu_fontsize=font_size,
    menu_foreground=colours[7],
    opacity=opacity,
    separator_colour=colours[8],
    highlight_colour=colours[12],
)

notifier = widget.StatusNotifier(**notifier_defaults)

If I use Mirror on notifier widget, then one monitor have menu in correct place, but another not
If i define StatusNotifier for each bar then on both monitors i have menu in wrong place
Tested on git version of qtile and qtile-extras

Ah, yes. I think I can see the error in the code.
If I submit a patch later today, are you able to test?
Thanks for reporting.

No problem with it ;)

I think #89 should fix this. Can you test and let me know?

Now… it always display under one of the trays, but when i restart qtile and remove __pycaches__ It can change under which monitor it display…
Globalmenu widget work fine for me, but not StatusNotifier. And both have quite similiar display_menu function currently

Are you using separate instances or a Mirror?

Actually, I think I know what the issue is here. It's a bit more complicated. Let me try something hacky first...

I tried with following widgets in bars:

```python
notifier_defaults = dict(
    menu_background=colours[0],
    menu_font='FiraCode Nerd Font',
    menu_fontsize=font_size,
    menu_foreground=colours[7],
    opacity=opacity,
    separator_colour=colours[8],
    highlight_colour=colours[12],
)
notifier = widget.StatusNotifier(**notifier_defaults)

widget_main_list = [
    widget.CurrentLayoutIcon(),
    widget.GroupBox(**groupbox_defaults),
    widget.CurrentScreen(**current_screen_defaults),
    prompt,
    widget.GlobalMenu(**globmenu_defaults),
    widget.TaskList(**tasklist_defaults),
    widget.Mpd2(
        **decorations,
        status_format='{play_status} {artist}/{title}',
        play_states={
            'pause': '', 'play': '喇', 'stop': 'ﭥ'
        },
        max_chars=60,
        idle_format='{idle_message}'
    ),
    widget.WidgetBox(
        widgets=[
            widget.CPU(**decorations, format="龍 {load_percent: >4g}%"),
            widget.Memory(**decorations, format=" {MemPercent: >4g}%"),
            widget.Net(**decorations, format='{down}李{up}'),  # notifier,
        ],
        text_closed="",
        text_open=""
        ),
    widget.StatusNotifier(**notifier_defaults),
    # notifier,
    widget.Clock(format='%Y-%m-%d %H:%M', mouse_callbacks={'Button1': display_cal}),
]
widget_2nd_list = [
    widget.CurrentLayoutIcon(),
    widget.GroupBox(**groupbox_defaults),
    widget.CurrentScreen(**current_screen_defaults),
    prompt,
    widget.GlobalMenu(**globmenu_defaults),
    widget.TaskList(**tasklist_defaults),
    # widget.WidgetBox(widgets=[widget.StatsNotifier(**notifier_defaults)], text_closed="", text_open=""),
    widget.StatusNotifier(**notifier_defaults),
    # notifier,
    # widget.Systray(),
    widget.Clock(format='%Y-%m-%d %a %H:%M')
]

And i was switching between version with notifier in the code and widget.StatusNotifier(**notifier_defaults),
globalmenu work fine if i have seperate instances, so i mainly was testing with seperate instances for notifier too.

Yeah, the issue is that the widget wasn't designed to have multiple instances (an oversight by me rather than any specific limitation). I'll need to submit a fix in the main qtile version first and then do an update in qtile-extras.
That may take a bit of time as I'm really busy at work at the moment.
Sorry for the inconvenience.

The updated fix in #88 should work once https://github.com/qtile/qtile/pull/3749 is merged.

Thank you very much. I highly appreciate your work ;)
Installed qtile and qtile_extras with corresponding PR and now it's working fine.


spirv-fuzz: Do not outline regions that end with a loop header

The outliner would outline regions ending with a loop header, making
the block containing the call to the outlined function serve as the
loop header.  This, however, is incorrect in general, since the whole
outlined function -- rather than just the exit block for the region --
would end up getting called every time the loop would iterate.
This change restricts the outliner so that the last block in a region
cannot be a loop header.

Maybe we should have one GitHub issue that lists several limitations of the outliner (and similarly for other groups of limitations). I suppose one issue per limitation can also work, but it might be nice to group them and then create individual issues only when we want to start or assign them.

Maybe.  However, we don't actually want to enhance the outliner to handle these situations.  Rather, we would like other transformations to get code into an outline-ready form.  I'm undecided as to whether we want to explicitly direct that process, or just have enough transformations in the mix that regions of code have a good chance of becoming outlinable.


Dataset not found page

Dear @lbc12345 ,
Thanks for your fantastic work.
When I tried to download the datasets provided in your paper, it showed me like this:

Bests regards,

Thanks, you can find the link from the project page data


Unable to use Babel Filter with Jinja Extension

I setup the Jinja extension and it works great. When I add "babel" to the filters argument, babel does not appear to be processing the assets. I believe Babel requires a preset to be provided otherwise it passes the code through untouched.
I do not see a way to provide the es2015 preset to babel using the Jinja asset tag.
{% assets filters='babel', output=output, filepath %}
  <script type="text/javascript" src="{{ ASSET_URL }}"></script>
{% endassets %}

The docs for the babel filter only show it being used with the bundle interface:

Example python bundle:
es2015 = get_filter('babel', presets='es2015')
bundle = Bundle('**/*.js', filters=es2015)

builtin_filters.html#webassets.filter.babel.Babel

I also tried setting the OS envitonment variable for BABEL_PRESETS to es2015, but it did not seem to make a difference.

Ok, I was able to get this working. The underlying issue was that the babel presets don't invalidate the cache... I though I had cleared the cache by wiping the .webassets-cache directory, but apparently I had to wipe my output directory also.
For future reference, the cleanest way to configure filters is using the asset config:
assets_environment = AssetsEnvironment(...)
assets_environment.config['BABEL_PRESETS'] = 'es2015'

@miracle2k I didn't find this info until I read the options section of Creating custom filters. Would you be interested in reviewing a PR that adds this to the Using the tag section of the Jinja2 docs? Is there any other place in the docs that you think this information would be useful?


[Bug] Planner throws an error in cases where we need to spend more than one note

If the balance on the account is split into several notes, the planner will give an error when trying to spend more than the smallest of the notes contains

We're getting an error here
https://github.com/penumbra-zone/web/blob/00fc1340ce0ed27dac643e056a779440638aa0d6/packages/wasm/crate/src/planner.rs#L412-L424

Current results of the bug investigation:

Most likely planner is working correctly
Part of my balances are on ephemeral addresses, and part on the main address.
It seems that the display of balances is also broken because of this. Two Account 0's are displayed, with the same address, and the entire UM balance is on one address

When sending a TransactionPlannerRequest we pass source - the account from which we want to spend the funds.
But source is AddressIndex, a normal address and an ephemeral address will have a different AddressIndex. more details here https://discord.com/channels/824484045370818580/1044457038886998137/1220143628404392006

The reason for the bug is that minifront thinks that the entire UM balance is on one account, but it sends to planner source with ephemeral AddressIndex, which has only one note with a small balance. Planner get notes by source, and obviously can't spend it to cover the entire balance of the transaction.


[EVENT] PaleoHackWeek 3

Summary
The PaleoHackWeek community is running another hackathon in about 10 days! This is similar to the previous hackathon's that they've run (though it will be 3 days instead of 2).
Note that this hub is already running at https://paleohack2021.hackathon.2i2c.cloud/, but we'll probably need to ready the hub for another scale-up event, so creating this issue so that we can track it.
In addition, @CommonClimate and @khider were hoping to do a trial run on March 9th where they'd ask the attendees to try logging in to make sure the scaling worked (even though they plan to start on March 10th).
Event Info

Community Representative: @commonclimate and @khider
Event begin: March 9th
Event end: March 11th
Active times: Roughly working hours on US/Pacific
Number of attendees: ~60
Hub Events Calendar

Hub info

Hub URL: https://paleohack2021.hackathon.2i2c.cloud/hub/login?next=%2Fhub%2F
Hub decommisioned after event?: No

Task List
Before the event

[x] Dates confirmed with the community representative and added to Hub Events Calendar

[x] One week before event Hub is running.

[ ] Confirm with Community Representative that their workflows function as expected.

👉Template message to send to community representative
```
Hey {{ COMMUNITY REPRESENTATIVE }}, the date of your event is getting close!
Could you please confirm that your hub environment is ready-to-go, and matches your hub's infrastructure setup, by ensuring the following things:

[ ] Confirm that the "Event Info" above is correct
[ ] On your hub: log-in and authentication works as-expected
[ ] nbgitpuller links you intend to use resolve properly
[ ] Your notebooks and content run as-expected

</details>

[ ] 1 day before event, either a separate nodegroup is provisioned for the event or the cluster is scaled up.

During and after event

[ ] Confirm event is finished.
[ ] Nodegroup created for the hub is decommissioned / cluster is scaled down.
[ ] Hub decommissioned (if needed).
[ ] Debrief with community representative.

👉Template debrief to send to community representative
Hey {{ COMMUNITY REPRESENTATIVE }}, your event appears to be over 🎉

We hope that your hub worked out well for you! We are trying to understand where we can improve our hub infrastructure and setup around events, and would love any feedback that you're willing to give. Would you mind answering the following questions? If not, just let us know and that is no problem!

- Did the infrastructure behave as expected?
- Anything that was confusing or could be improved?
- Any extra functionality you wish you would have had?
- Could you share a story about how you used the hub?

- Any other feedback that you'd like to share?

cc @khider and @commonclimate - it would be helpful if you could run a representative notebook from top to bottom a day or two before the event, just to confirm that the environment is correct and such!

Thanks! Yes, I'm hoping to release the latest version of the code today or tomorrow so I can rebuild the environment and see if everything works.
They will be busy with another platform on Wednesday March 9th (some may make it to the hub during the night between 9th and 10th).

Sounds good @khider !
Also just in case it'll take a bit of time to figure out stress-testing stuff, I opened an issue so that we can track it: https://github.com/2i2c-org/infrastructure/issues/1041

@choldgraf We updated all the notebooks and environment. The container built automatically and I grabbed the image from quay for the Configurator but this is the error I get when restarting:

Hmmm - that might be because you're just using the tag and not the fully-resolved name. I just tried setting it to:
2i2c/paleohack-2021:cb5757e7c9f8
try logging in and see if that works?

ah it needed to be the fully-resolved name:
quay.io/2i2c/paleohack-2021:cb5757e7c9f8

it should be working now

I opened up a PR to clarify the image tag prompt here: https://github.com/yuvipanda/jupyterhub-configurator/pull/11

Thanks! I got it to run but now I can't get it to sync with GitHub. It tells me that it did but actually didn't for mine. When opening a new user server, I don't get the notebooks at all.

What's your nbgitpuller link @khider ?

https://paleohack2021.hackathon.2i2c.cloud/hub/login?next=%2Fhub%2Fuser-redirect%2Fgit-pull%3Frepo%3Dhttps%253A%252F%252Fgithub.com%252FLinkedEarth%252FpaleoHackathon%26urlpath%3Dtree%252FpaleoHackathon%252F%26branch%3Dmain

@khider I just tried clicking that link and it worked for me ). Can you share a specific notebook that you expect to be there, but that is not there?
Here are the notebooks in my folder:

I just tried clicking that link and it worked for me

same here

For me, it seems to merge old/new versions of the notebooks.
This is the hub version:

But this is the version on GitHub

If this is just me (because I have been using the same account for 3 hackathons), then not a problem.

And the merge decision is somewhat random. I changed the preamble from the version we had and it got updated but the working with datasets didn't.

Hmmm - the language around the merging behavior is defined here: https://jupyterhub.github.io/nbgitpuller/topic/automatic-merging.html
Does that help disambiguate at all? If we can clarify that language let me know and we can update it.

Yes, that makes more sense now. I'll delete my files and retry.

Perfect!

wohoo! I am curious - where could advertise this link in the 2i2c docs so it is discoverable?

I have been building my own Google Doc with all the links.

Ah cool - well, maybe once this round is over you'd be willing to tell us all of the things that were confusing, and that we could improve in the docs? :-)

We are ready on the notebook side. Should I try to launch all the servers at the same time?

I believe that @yuvipanda has provisioned a dedicated set of nodes for the event (see https://github.com/2i2c-org/infrastructure/pull/1058) so there's no problem in giving it a shot if you like

Thank you! And we're all ready to go.

@khider sorry, it's not done yet, I'm doing that by the next hour - sorry.

@khider ok I've scaled up a lot of nodes (20), and enabled an additional performance optimization (pre-pulling the existing image).

BTW, everything is working great!

Hey @khider - just want to confirm that this event is over, and that you no longer need the extra nodes for the event!

I just set the minimum node size to 0.

Yes, we're done! Thank you; everything worked great!

Wohoo, thanks @khider ! If you like, it'd be helpful for any thoughts that you might have about how the event went:
If you like, could you answer any of the following questions?

Did the infrastructure behave as expected?
Anything that was confusing or could be improved?
Any extra functionality you wish you would have had?
Could you share a story about how you used the hub?
Any other feedback that you'd like to share?

I'll close this one since I don't think there are any more actions to take!

@choldgraf I need to put the hub back into the general pool (rather than to its own), will close when done.

Done in https://github.com/2i2c-org/infrastructure/pull/1123


Control maximum number of concurrent pages to be crawled

Is there an option to control the number of concurrent pages taken out of the scheduler? I tried the MaxConcurrentThreads but seems to have no effect.
I need to set it to 1 while debugging. Do you have any suggestions?

Hi MaxConcurrentThreads does exactly that.


Add whole screen mode

Like zen mode

CodeMirror supports fullScreen, it works with F11 key.
Need to know how to apply.

Having trouble with #content, it will be complex problem.


Use CMAKE_TOOLCHAIN_FILE environment variable earlier

Some decisions in build are made if CMAKE_TOOLCHAIN_FILE is defined, so we need to read it from the environment variables from the beginning.

Thanks!


Breaking down comment is little vague

https://github.com/aws-samples/amazon-codewhisperer-immersion-day/blob/41634aa4b5d12ba0b1cb788db28ecdb1480836ee/python/basics/example3.py#L2
In this example, i donno why code whispher assumes, there will be csv file with # in line starting. I feel its too much information and we need to troubleshoot the correctness of AI here.
Also its giving a pause there. Can it simply complete the function anyhow and we can correct it later ?

For asking AI recommendations with #, for single line it makes sense. but for multi line every time we come up with a new line and we are thinking to write the next line, its already giving the recommendation by then. Like my above example. I was just trying to break down each line.