[BUG] - Probably serialisation issue between 1.35.7 and 8.* (git HEAD) causing a segfault.

Internal
Area
ChainDB and chain sync protocl
Summary
Almost certainly an issue in the serialization code somewhere. Did the following:

Upgraded node to git HEAD (81b7ebe6bf5a383cc79b88c0cc37d25ad7238c46) and ran it.
Kept my existing db-sync binary and connected it to the node. Its synced and then as soon as it started following thenode it segfault. Retried this a couple of time and each time syncing was fine as as soon as it started following it segfault.
Checked out the last release of node (tag 1.35.7) built it and ran it.
As soon as it finished reading the non-volatile DB It segfaulted.
Restarted it and then it decided to invalidate the non-volatile DB.
Once it revalidated all on-disk databases it started following the chain.
I then restarted db-sync and it replayed its internal ledger state from genesis.

A Haskell process just segfaulting like this is REALLY BAD.
There is probably some minor change in the serialization code somewhere that caused this, but we should track down why and make sure this does not happen again. This will be very confusing to anyone switch back and forth between the latest current release and the new version.
Versions:
cardano-node 1.35.7 - linux-x86_64 - ghc-8.10
git rev f0b4ac897dcbefba9fa0d247b204a24543cf55f6

cardano-db-sync <IP_ADDRESS> - linux-x86_64 - ghc-8.10
git revision 162523bf099b792ce7f5fc0510ab3af6b152d9c4

While my db-sync was replaying its ledger state, it suffered a bus error. The node had not restarted,but when I restarted db-sync it could no longer file the node's local domain socket. I had to restart the node to fix this.

I keep on restarting db-sync and I keep on getting bus errors and segfaults.
I put new (more) RAM in this machine today and maybe that's the issue.

Ok, this is not good:
> sudo /usr/sbin/memtester 32G
[sudo] password for erikd:
memtester version 4.6.0 (64-bit)
Copyright (C) 2001-2020 Charles Cazabon.
Licensed under the GNU General Public License version 2 (only).

pagesize is 4096
pagesizemask is 0xfffffffffffff000
want 32768MB<PHONE_NUMBER>8 bytes)
got  32768MB<PHONE_NUMBER>8 bytes), trying mlock ...locked.
Loop 1:
  Stuck Address       : testing   4FAILURE: possible bad address line at offset 0x00000001e29e1ac0.
Skipping to next test...
  Random Value        : ok
  Compare XOR         : ok
  Compare SUB         : ok
  Compare MUL         : ok
FAILURE: 0x000000000000000c != 0x000000000000008c at offset 0x00000001eed36f20.
  Compare DIV         :   Compare OR          : ok
  Compare AND         : ok
  Sequential Increment: ok
  Solid Bits          : ok
  Block Sequential    : testing   1FAILURE: 0x2101010101010101 != 0x0101010101010101 at offset 0x0000000257860ac0.
  Checkerboard        : testing   1FAILURE: 0x8aaaaaaaaaaaaaaa != 0xaaaaaaaaaaaaaaaa at offset 0x0000000187e82340.
  Bit Spread          : testing   6FAILURE: 0x00000000000001c0 != 0x0000000000000140 at offset 0x00000003517cef30.
  Bit Flip            : testing  23FAILURE: 0x2000000000000004 != 0x0000000000000004 at offset 0x000000028496bac0.
  Walking Ones        : testing  51FAILURE: 0xeff7ffffffffffff != 0xfff7ffffffffffff at offset 0x000000014b587340.
  Walking Zeroes      : testing   5FAILURE: 0x2000000000000020 != 0x0000000000000020 at offset 0x000000015b7f5bc0.
  8-bit Writes        : ok
  16-bit Writes       : /^C

Popped the cover, removed the new RAM, ran the air duster across the slots, cleaned the RAM contacts, reinserted them and retried. This reduced the error rate but did not eliminate them. Did the same again, still errors.
Ended up putting the original RAM back in and no more errors.
Closing this.


Character padding issue

[x] I am on the latest Hyper.app version

[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Windows 10

Hyper.app version: 3.0.2

Link of a Gist with the contents of your .hyper.js: https://gist.github.com/ERENKYI/930a606fd9222156a99079a7fbf16cd3

Issue
As you can see from the screenshot below, the arrow characters have some sort of space/padding issue. The window below belongs to wsl.exe which doesn't have that problem with the same font.

Duplicate of #3617


Connection leak when there is a master/slave node fail over

I encountered an issue when there is a change in topology
Setup: 20 master nodes, each with 2 slaves. Running 4.1.1.Final. ReadFrom is read from slaves
Sequence of events:

M1 went down; redis process got killed completely
S1 got promoted to be new master
S2 now starts syncing from S1, the new master

During the time window when S2 is syncing from S1, there are many connections opened from a host to S2, eventually causing Redis to hit its client count limit and reject other connections. I see a bunch of IOException Connection reset by peer in the log.
Looking into the code (https://github.com/mp911de/lettuce/blob/4.2.x/src/main/java/com/lambdaworks/redis/cluster/PooledClusterConnectionProvider.java#L520), here is what i think is happening:

Cluster topology refresh happens
Requests for reading against S2 come in; new connection is initialized
Before this new connection is passed back to the LoadingCache, readonly() is invoked on the sync interface.
This command fails with error "Loading redis is loading dataset in memory"
The exception got propagated back to the caller, resulting in a failure. The connection created in second step is now orphaned and not accessible thru the cache
The loading process takes a few minutes and with enough TPS, S2 is completely saturated with connections.

Is the fix here to close the connection when readonly() fails?
Would be nice if Redis "cluster nodes" command provides a flag for this state when Redis is loading in RAM so we can exclude from topology. An alternative is to somehow trigger an event when this happens (first time seeing Redis loading error on a connection) and have the topology refresher exclude the node from the topology. These are just wish-list  items though. Would really appreciate if you can provide feedback on the bug and the proposed fix.
Note that when S2 is saturated, we're also seeing very weird behavior in another set of servers where they purely perform writes which should never establish any connection to slaves except for topology refresh. These hosts would have ConnectionWatchdog throwing exceptions like NullPointerException, cannot initialize channel ... Unfortunately, we don't have debug/trace log enabled so cant really debug more that that. I am assuming this issue is related to https://github.com/mp911de/lettuce/issues/278 ? After a while, both the writer hosts and reader hosts got into a state where this exception is seen everywhere
21 Aug 2016 08:59:17,579 [WARN]  (lettuce-nioEventLoop-3-152) com.lambdaworks.redis.protocol.CommandHandler: [/<IP_ADDRESS>:49672 -> /<IP_ADDRESS>:6379] Unexpected exception during request: io.netty.util.internal.OutOfDirectMemoryError: failed to allocate 16777216 byte(s) of direct memory (used:<PHONE_NUMBER>, max:<PHONE_NUMBER>)
io.netty.util.internal.OutOfDirectMemoryError: failed to allocate 16777216 byte(s) of direct memory (used:<PHONE_NUMBER>, max:<PHONE_NUMBER>)
	at io.netty.util.internal.PlatformDependent.incrementMemoryCounter(PlatformDependent.java:613)
	at io.netty.util.internal.PlatformDependent.allocateDirectNoCleaner(PlatformDependent.java:567)
	at io.netty.buffer.PoolArena$DirectArena.allocateDirect(PoolArena.java:699)
	at io.netty.buffer.PoolArena$DirectArena.newChunk(PoolArena.java:688)
	at io.netty.buffer.PoolArena.allocateNormal(PoolArena.java:237)
	at io.netty.buffer.PoolArena.allocate(PoolArena.java:213)
	at io.netty.buffer.PoolArena.allocate(PoolArena.java:141)
	at io.netty.buffer.PooledByteBufAllocator.newDirectBuffer(PooledByteBufAllocator.java:262)
	at io.netty.buffer.AbstractByteBufAllocator.directBuffer(AbstractByteBufAllocator.java:179)
	at io.netty.buffer.AbstractByteBufAllocator.directBuffer(AbstractByteBufAllocator.java:170)
	at io.netty.buffer.AbstractByteBufAllocator.ioBuffer(AbstractByteBufAllocator.java:131)
	at io.netty.channel.DefaultMaxMessagesRecvByteBufAllocator$MaxMessageHandle.allocate(DefaultMaxMessagesRecvByteBufAllocator.java:73)
	at io.netty.channel.nio.AbstractNioByteChannel$NioByteUnsafe.read(AbstractNioByteChannel.java:111)
	at io.netty.channel.nio.NioEventLoop.processSelectedKey(NioEventLoop.java:571)
	at io.netty.channel.nio.NioEventLoop.processSelectedKeysOptimized(NioEventLoop.java:512)
	at io.netty.channel.nio.NioEventLoop.processSelectedKeys(NioEventLoop.java:426)
	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:398)
	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:805)
	at io.netty.util.concurrent.DefaultThreadFactory$DefaultRunnableDecorator.run(DefaultThreadFactory.java:145)
	at java.lang.Thread.run(Thread.java:745)

Thanks,

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You specify the lettuce version and environment so it's obvious which version is affected
[ ] You provide a reproducible test case (either descriptive of as JUnit test) if it's a bug or the expected behavior differs from the actual behavior.

You're tackling a few issues here. Let's untangle these and see what we have here:

Clients exceed server capacity: If a node goes down and the remaining nodes are not able to deal with the capacity, then you might have a capacity issue. If you don't strictly require to read from slaves only, then you can either switch to ReadFrom.NEAREST or you provide an own implementation for ReadFrom.
Orphaned Connections: That's a bug that requires a fix, I created #343 to fix that issue.
Flag for a slave that synchronizes: I'm not aware of any flags that could be used for that. The output of CLUSTER NODES is limited to nodeId/endpoint/ping/health/slot details and does not provide any details about lag. Synchronization is an ongoing async process so it's not possible to determine with a high accuracy why a node is syncing and what should be considered a faulty state.
ConnectionWatchdog throws NullPointerException: That's a bug that was fixed with lettuce 4.2.2.Finalwith #335 and #315. Sometimes the reconnect is faster than the disconnect notification. That leads to the mentioned behavior. Please upgrade to 4.2.2.Final and create a ticket if the issue if the problem persists.

We already have our own ReadFrom implementation which basically prefer slave over master
Thanks for the fix. Is there any other place where this could potentially be an issue?
in "info" command output, you can have tell whether the node is doing a full sync by looking at the flag "master_sync_in_progress". If only this flag got exposed as part of "cluster nodes", we will be able to exclude this node. Alternatively, we can call "info" on each node but that seems excessive. Would be nice to have Lettuce detect this error "LOADING" the first time and have topology reacts to this event and automatically exclude this node from further requests until this is done. This can be achieved with a background process to detect when the node is done syncing by checking this flag from "info" command once every 10 seconds or something
I thought so too. Upgraded our library to 4.2.2 and will report back if there is any issue

Thanks for your quick turnaround on this

1+3. I think these things are related to each other. Removing the node from the topology might fit for your case, but not for others. Adding further configuration possibilities comes at a complexity and I'm not convinced yet that this is a common requirement. RedisClusterClient publishes a topology changed event. You can listen to these and query the nodes yourself. By using the ReadFrom API you're able to implement a selector that ignores these nodes in the candidate selection.
2. I need to check MasterSlave to verify the bug does not occur there as well, thanks for the hint.

Is there anything I may assist you within the scope of this ticket or can I close it?


Make Gadget Height Adapt to Displayed Entries

The gadget height in fix. If you have 9 or more items, the gadget height should adapt to the number of entries.

Fixed w/ adding support for user-prefs (to filter the projects)


mriqc, Error, need help

Dear all,
I have validated my dataset with bids validator: http://incf.github.io/bids-validator/ (No errors accured)
and I am trying to analyse the data quality with mriqc. I receive the following Error:
Dmitriys-MacBook-Pro:~ dmitriydesser$ docker run -it --rm -v /Users/dmitriydesser/Desktop/studie/bids:/data:ro -v /Users/dmitriydesser/Desktop/studie/mriqc:/out poldracklab/mriqc:latest /data /out participant --participant_label 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18 19 20
2018-06-27 22:55:10,697 mriqc.cli:WARNING No T2w scans were found in /data
2018-06-27 22:55:10,698 mriqc.cli:WARNING Anonymized quality metrics will be submitted to MRIQC's metrics repository. Use --no-sub to disable submission.
2018-06-27 22:55:24,453 workflow:WARNING Some nodes exceed the total amount of memory available (1.76GB).
Traceback (most recent call last):
File "/usr/local/miniconda/bin/mriqc", line 11, in
load_entry_point('mriqc==0.11.0', 'console_scripts', 'mriqc')()

File "/usr/local/miniconda/lib/python3.6/site-packages/mriqc/bin/mriqc_run.py", line 316, in main
workflow.run(**plugin_settings)

File "/usr/local/miniconda/lib/python3.6/site-packages/nipype/pipeline/engine/workflows.py", line 595, in run
runner.run(execgraph, updatehash=updatehash, config=self.config)

File "/usr/local/miniconda/lib/python3.6/site-packages/nipype/pipeline/plugins/base.py", line 121, in run
self._prerun_check(graph)

File "/usr/local/miniconda/lib/python3.6/site-packages/nipype/pipeline/plugins/multiproc.py", line 211, in _prerun_check
raise RuntimeError('Insufficient resources available for job')

RuntimeError: Insufficient resources available for job
Can anybody help me?
Thanks a lot!
Best,
Dima

Hi Dima!
Thanks for the details, I think with what you've provided, I can slim down what the issue should be. It appears docker only has access to 2 gb of RAM, whereas mriqc typically takes 8 gb of RAM, so you can provision docker to have access to 8 gb of RAM if your machine has at least 10 gb of RAM (gotta give the operating system 2 gb to complete necessary tasks). If your machine does not have that much RAM, I can suggest trying to use the --mem_gb <however much RAM you can spare> option when calling mriqc, and limiting your call to one participant label to minimize memory consumption.

Thank you very much! It works! There are some Errors while preprocessing
step, but reports seem to be normal at the end.
Thanks a lot for your help!
Best,
Dima
James Kent<EMAIL_ADDRESS>schrieb am Do. 28. Juni 2018 um 04:57:

Hi Dima!
Thanks for the details, I think with what you've provided, I can slim down
what the issue should be. It appears docker only has access to 2 gb of
RAM, whereas mriqc typically takes 8 gb of RAM, so you can provision
docker to have access to 8 gb of RAM
https://docs.docker.com/docker-for-mac/#advanced if your machine has at
least 10 gb of RAM (gotta give the operating system 2 gb to complete
necessary tasks). If your machine does not have that much RAM, I can
suggest trying to use the --mem_gb
option when calling mriqc, and limiting your call to one participant label
to minimize memory consumption.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/mriqc/issues/723#issuecomment-400895243,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ajtq962fuSNACQ9AQ--Lq3NX6Z4T5w74ks5uBEYvgaJpZM4U6g0J
.

Awesome! If you want to detail your preprocessing errors, neurostars would be the best place to get answers.

Please reopen if you keep experiencing issues.


blog: 2024-04-04-in-person-install-instructions-session

Add based on the notes and recollection
I'm not sure how much detail to go into, since the changes have been
made.  Instead, I went into the motivation and what happened after
in more detail, and the effects.
This probably needs more of a close read to think about what it
should say exactly and if anything important is missing.

Thank you!


Merges to main don't get posted to main && it should be clearer how branches are deployed

fixes #319 - merges to main still don't get posted to /main , and #362 It should be clearer how branches are deployed.
Specifically:

Moves to using pull_request instead of push -- this is actually the correct event, as we want these flows to fire when opening and closing pull requests, not simply pushing. We can't push to master directly anyhow.
Also imposes the restriction that if the PR is closed, and merged, then we should update the main URL; otherwise, we should use the hash. Both of these need to be true; we could theoretically close without merging or merge without closing.
Only comment with the deployment URL when the PR is opened - when it has been closed, all PRs will go to /main.
Bot now adds comment to the PR itself with the URL of the preferred deployment
Job now echoes the full set of URLs to the console when running the flow.

tl;dr - we were using the wrong event and trigger; fixing this made it so we're setting the right variables to the right values and running at the right time.
(Interestingly because this fix targets ./client, it won't fire for this change itself, but I have confirmed this fix does indeed work with a local repository.

LGTM. Super excited for this change!
Interesting thing I found when researching this change: many project use labels like feature-flags to control the ci process. contains(github.event.pull_request.labels.*.name, 's:benchmark-required') type stuff.

Yeah! This is definitely an interesting opportunity; we used similar branch prefix-based jobs with jenkins at my last job as part of the git-flow model (more here, interesting side note that the creator now advocates for feature branches instead!) - that model has a lot more baggage (and was actually quite helpful for mobile development), but it does seem we could adopt some general branch prefixes for this project to do some cool stuff like pre-releases etc.


Support for valueless variants

In some examples, we need a variant that does not require a value. For instance, here is a type declaration, we could have in the two phase commit:
(*
 @typeAlias: MESSAGE = Commit | Abort | Prepared(RM);
 *)
TwoPhaseTyped_aliases = TRUE

The current implementation of variant types does not support such valueless variants.

There is a very simple way to deal with valueless invariants. We can call it an idiom. An implementation of this issue would not differ from this idiom:
(*
 @typeAlias: MESSAGE = Commit(NIL) | Abort(NIL) | Prepared(RM);
 *)
TwoPhaseTyped_aliases == TRUE

\* @type: MESSAGE;
MkCommit == Variant("Commit", "0_OF_NIL")

\* @type: MESSAGE;
MkAbort == Variant("Abort", "0_OF_NIL")

\* @type: RM => MESSAGE;
MkPrepared(rm) == Variant("Prepared", rm)


Add unit test to cover non built-in GenericAlias types

Adds a unit test to ensure that parameterized generic types that are not built-in are handled properly by type hinting code.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @damccorm

Portable Python pre-commit passed but failed to update the PR description


Add support for suppressing non-error output

Expected Behavior
A Gradle/Maven configuration option can be used to disable non-failure log output
Current Behavior
The validation report is always printed.
Context
When running gradle build nearly the entire output is detekt-related. I'd like to only see output when validation fails.

Hey thanks for the feedback.
Can you try turning off all console output with:
console-reports:
  active: true
  exclude:
    - 'ProjectStatisticsReport'
    - 'ComplexityReport'
    - 'NotificationReport'
    - 'FindingsReport'
  #  - 'BuildFailureReport'

inside the config file?
These lines are already in your config, you need just to remove the #.


TypeError: issubclass() arg 1 must be a class when having a field with a complex type

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from datetime import datetime
from typing import Optional, List, Tuple, Dict, Union
from sqlmodel import Field, SQLModel, create_engine

class SemanticSearch(SQLModel, table=True):
    id: Optional[int] = Field(default=None, primary_key=True)
    id_user: int
    date_time: datetime
    query: str
    clean_query: str
    semantic_search_result: List[Tuple[int, Dict[str, Union[str, int, float]]]]

## sqlite
sqlite_file_name = "database.db"
sqlite_url = f"sqlite:///{sqlite_file_name}"

engine = create_engine(sqlite_url, echo=True)

SQLModel.metadata.create_all(engine)

Description
I would like to create a semantic_search table in the database.db sqlite database with the 6 fields specified.
The last field semantic_search_result with List[Tuple[int, Dict[str, Union[str, int, float]]]] type seems to cause the error.
semantic_search_result looks like this for example:
[
   (0, { "acquisCode": str, "code": str, "level": int, "title": str, "similarity_score": float}),
   (1, { "acquisCode": str, "code": str, "level": int, "title": str, "similarity_score": float}),
   (2, { "acquisCode": str, "code": str, "level": int, "title": str, "similarity_score": float})
]

When running the code I got the following error:
Traceback (most recent call last):
  File "/home/matthieu/Code/Python/fastapi-graphql/test_SQLModel.py", line 6, in <module>
    class SemanticSearch(SQLModel, table=True):
  File "/home/matthieu/anaconda3/envs/sts-transformers-gpu-fresh/lib/python3.8/site-packages/sqlmodel/main.py", line 292, in __new__
    col = get_column_from_field(v)
  File "/home/matthieu/anaconda3/envs/sts-transformers-gpu-fresh/lib/python3.8/site-packages/sqlmodel/main.py", line 415, in get_column_from_field
    sa_type = get_sqlachemy_type(field)
  File "/home/matthieu/anaconda3/envs/sts-transformers-gpu-fresh/lib/python3.8/site-packages/sqlmodel/main.py", line 373, in get_sqlachemy_type
    if issubclass(field.type_, str):
TypeError: issubclass() arg 1 must be a class

I got the same error if I tried to define a subclass of the typing_extensions.TypedDict class:
class SemanticSearchDict(TypedDict):
    acquis_code: str
    code: str
    level: int
    title: str
    similarity_score: float

class SemanticSearch(SQLModel, table=True):
    id: Optional[int] = Field(default=None, primary_key=True)
    id_user: int
    date_time: datetime
    query: str
    clean_query: str
    semantic_search_result: List[Tuple[int, SemanticSearchDict]]

I couldn't find a solution but #57 seems to have the same problem since having the same error.
Operating System
Linux
Operating System Details
Ubuntu 18.04 LTS
SQLModel Version
0.0.4
Python Version
3.8.8
Additional Context
No response

🆙

Up!

same Problem

same problem here!

🆙
thank you to the maintainers & contributors of this awesome library!

Any update on this issue?

Same problem when doing int | None type. Workaround for this: Optional[int].

same happen using UUID4 for id

Its because of the get_sqlachemy_type.


allowFontScaling  is not working in webview for Android.

Environment
Environment:
OS: macOS High Sierra 10.13.5
Node: 6.9.4
Yarn: Not Found
npm: 3.10.10
Watchman: 4.9.0
Xcode: Xcode 9.4 Build version 9F1027a
Android Studio: 2.2 AI-145.3537739
Packages: (wanted => installed)
react: ^16.3.1 => 16.4.1
react-native: ^0.55.0 => 0.55.4
Description
allowFontScaling  is not working in webview for Android.
Reproducible Demo
Increase your Android device font size to the maximum and open url in webview you observe that font size of text in webview for Android is increases and when you put allowFontScaling value false then no effect for android in webview rather than ios is working fine.

Migrated to https://github.com/react-native-community/react-native-webview/issues/30 and this issue can be closed.


Allow Admins to add or remove a user from a User Group without notifying them

Request
Allow admins to be able to add or remove members from User Groups without notifying those members.
I'm unsure if Moderators should also have this ability. You may not want regular members to have this ability even if they are part of the user group they are changing.
Use case
In an organization I administer some members have become inactive and I want to remove them from certain Zulip user groups so they don't receive a mention notification when we mention that user group. However, I don't want to bother them with a notification that I removed them from the user group either.

Thanks! Closing as a duplicate of #25943.
I'm not sure about the notification part. This should be rare enough that a DM from Notification Bot wouldn't be too disruptive.

I found a way to do what I wanted by deactivating the user, removing them from the user group(s) and reactivating their account. Maybe not intended that you could do that, but it worked for my use case.


⚠ sandbox-api.piste.gouv.fr healthcheck has degraded performance

In 3d3bfd5, sandbox-api.piste.gouv.fr healthcheck (https://sandbox-api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 4907 ms

Resolved: sandbox-api.piste.gouv.fr healthcheck performance has improved in 6eb44e9.


Added beforeFallback hook & made facet management public methods virtual

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (ddd647f) 100.00% compared to head (52de4e2) 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##            master       #41   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           28        28
  Lines          860       862    +2
  Branches       123       123
=========================================
+ Hits           860       862    +2

Impacted Files
Coverage Δ

contracts/diamond/Diamond.sol
100.00% <100.00%> (ø)

.../diamond/presets/OwnableDiamond/OwnableDiamond.sol
100.00% <100.00%> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Default param injection

Expected Behavior
Parameters that aren't set manually but have a default value should use the default value in any cases.
Current Behavior
Currently Ziggy check the number of parameters to determine if it needs to inject default parameters. It will throw an error if a parameter doesn't get a value even if it should get it from the default values.
For bug reports only please provide
Currently installed Laravel version:
6.3.0
Currently installed Ziggy version
v0.8.0
What could be done
The default parameters handling in the function hydrateUrl() in route.js need to be rewritten.
I'd gladly submit a push request.

Here an example to illustrate my problem:
let say we have the route :
life/on/{planet} ->name('life')
and a default parameter :
"planet" => "mars"
if while using we use add a custom parameter to the ajax call for example :
route('life', {ziggy:"startdust"})
Ziggy will throw an error saying that the parameter "planet" is required.
How ever if you do
route('life', {ziggy:"stardust", goblin:"king"})
it will work and produce the route
/life/on/mars?ziggy=stardust&goblin=king

Hey Eric!
Thanks for this issue report. I've tried to replicate the issue and can't get it to work the same as you've reported.
If I do route('life', {ziggy:"stardust", goblin:"king"}) it fails with a missing parameter error, the same as route('life', {ziggy:"stardust"}).
Is there something else in your setup/application that might be causing the issue? What does your actual route definition look like for this route?

@eric-reichenbach can you update Ziggy to version 0.8.1 or higher and confirm if this is still an issue? I'm pretty sure it was fixed by https://github.com/tightenco/ziggy/commit/f3d08274629b17a2298279e9a51a1fec1833d3e1, which was released in 0.8.1 (2 days after you submitted this issue).

Sorry for the delay, I can confirm I don't have to problem anymore. Thanks a lot.


Write a C program to check leap year

Description
Write a C program to check leap year
Leap years are years that are divisible by 4, but not by 100, unless they are also divisible by 400. For example, 1996 is a leap year, but 1900 is not. Write a function that takes a year as a parameter and returns True if the year is a leap year, False otherwise.
Input  : 2016
Output : Leap Year

How to contribute

Save the solution in program/CheckLeapYear.c file
Add CheckLeapYear.c file in check-leap-year folder

!assign


Request to host package

This issue was originally filed by<EMAIL_ADDRESS>
https://github.com/prujohn/zart_web
This is an application, not a library, so should be an interesting exercise for pub?
See http://code.google.com/p/dart/issues/detail?id=5472 for the related library (Zart)

This issue has been moved to dart-lang/pub#162.


Not being able to load data that has been previously saved

Description
I am trying to load data in a Dataset after I have previously saved it, but I cannot get the data itself. I know that the data is being read because the feature names are being loaded, but the data is not.
Reproducible example
The following loads some data into a Dataset:
>>> from sklearn.datasets import load_iris
>>> import lightgbm as lgb
>>> data = load_iris()
>>> X = data.data
>>> y = data.target
>>> ds = lgb.Dataset(X, y)
>>> ds.save_binary("example.bin")

Now that I have my data saved, I want to load it:
>>> ds2 = lgb.Dataset("example.bin")
>>> ds2.data
'example.bin'
>>> ds2.get_data()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/vladimirvargas/micromamba/envs/real-estate/lib/python3.9/site-packages/lightgbm/basic.py", line 2768, in get_data
    raise Exception("Cannot get data before construct Dataset")
Exception: Cannot get data before construct Dataset

As expected, I cannot get_data, so I construct lazily the Dataset:
>>> ds2.construct()
[LightGBM] [Info] Load from binary file example.bin
<lightgbm.basic.Dataset object at 0x10061d6d0>
>>> ds2.data
>>> ds2.get_data()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/vladimirvargas/micromamba/envs/real-estate/lib/python3.9/site-packages/lightgbm/basic.py", line 2787, in get_data
    raise LightGBMError("Cannot call `get_data` after freed raw data, "
lightgbm.basic.LightGBMError: Cannot call `get_data` after freed raw data, set free_raw_data=False when construct Dataset to avoid this.

which is rather unexpected, as I don't know why the raw data was freed, initially. Let us follow the message advise, though:
>>> ds2 = lgb.Dataset("example.bin", free_raw_data=False)
>>> ds2.data
'example.bin'
>>> ds2.construct()
[LightGBM] [Info] Load from binary file example.bin
<lightgbm.basic.Dataset object at 0x144e820a0>
>>> ds2.data
'example.bin'
>>> ds2.get_data()
'example.bin'

Note that this is not what I expected, I expected to actually be able to access the data. How can I access it?
Environment info
LightGBM version or commit hash:
v4.0.0
Command(s) you used to install LightGBM
pip install lightgbm

in a virtual environment.

Also, I've noticed that loading the data and using it for something is not tested in https://github.com/microsoft/LightGBM/blob/v4.0.0/tests/python_package_test/test_basic.py#L247

Saving the binary doesn't save the raw data I believe.  In this case the "binary" data refers to the binned data, and the "raw" data refers to the data prior to binning (floats for instance).  What if you just don't call get_data() but do lgb.train({}, ds2)?  Do you also get an error?

import lightgbm as lgb
import numpy as np

np.random.seed(1)
X = np.random.randn(300, 3)
y = np.random.randn(300)

data = lgb.Dataset(X, label=y, init_score=np.zeros_like(y), free_raw_data=False).construct()
data.save_binary('data.bin')

data2 = lgb.Dataset('data.bin', free_raw_data=False).construct()
data2.get_data()

This returns just the path to the data fwiw

lgb.train({}, ds2)

Running this gets me
>>> lgb.train({}, ds2, num_boost_round=3)
[LightGBM] [Warning] Auto-choosing col-wise multi-threading, the overhead of testing was 0.000556 seconds.
You can set `force_col_wise=true` to remove the overhead.
[LightGBM] [Info] Total Bins 98
[LightGBM] [Info] Number of data points in the train set: 150, number of used features: 4
[LightGBM] [Info] Start training from score 1.000000
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
<lightgbm.basic.Booster object at 0x100a32160>

I don't think any training happened 🤔

Though, if I do it with the original dataset I get the same output:
>>> lgb.train({}, ds, num_boost_round=3)
[LightGBM] [Warning] Auto-choosing col-wise multi-threading, the overhead of testing was 0.000464 seconds.
You can set `force_col_wise=true` to remove the overhead.
[LightGBM] [Info] Total Bins 98
[LightGBM] [Info] Number of data points in the train set: 150, number of used features: 4
[LightGBM] [Info] Start training from score 1.000000
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
<lightgbm.basic.Booster object at 0x100a18490>

I would throw a warning when trying to get data and it is not possible because the Dataset has been loaded from a .bin file.


Setting sqlalchemy restrictions to be compatible with our version of pandas

This PR adds a runtime restriction on sqlalchemy<2.0 because version 2.0 is incompatible with pandas=1.3. See here for more information.
The runtime restriction was missing even though it was included in the conda dev dependencies.
Included in this PR are changes to remove Git LFS from the base image and only install it in later stages used to build the conda package.

/merge


Feature Request: Make it possible for transform to return issues

When it comes to custom validation or transformation, AFAICT, Valibot's  transform(), custom() and special() can either validate or transform something but not both.
When using zod, one pattern that I've been using sometimes is to use transform() right after a validation and perform custom validation and transformation.
One example is being able to parse and validate one JSON string (code below taken from a little library I wrote):
export function zjsonstring(schema) {
  const validateJson = z.string().transform((val, ctx) => {
    try {
      const parsed = JSON.parse(val);
      return parsed;
    } catch (e) {
      ctx.addIssue({
        code: z.ZodIssueCode.custom,
        message: e instanceof Error ? e.message : "Invalid JSON"
      });
      return z.NEVER;
    }
  });

So I can just use zjsonstring({ foo: z.string(), bar: z.number() }) more practically. But there are other cases in the wild, like Kent C Dodds use transform() to perform a login and return the session.
Given current valibot architecture, this would be a breaking change as the new API would need to return output or issues, and not output directly like today.

Given valibot's architecture, not sure if this would require a new API since the passed action would need to return either { issue } or { output } and the direct output like today.

Thank you for creating this issue. Basically you need the ability to add or return an issue in our transform method. You are right that if we stick to the current approach in our pipeline, we would need an object with output or issues, which would be a breaking change. Alternatively we could consider adding the ability to throw a ValiError in transform.

Thanks! I can take a shot at a PR today (actually already done it in a fork that I'm using and I can try to contribute it upstream).
Do you have any preferences between a breaking change vs. throwing a ValiError ?

Currently I think it is a better API to just be able to return the transformed data without return { outout: ... }. In the pipeline it is Ok, because here mostly the validation and transformation functions are used. What do you think about this?

Throwing a ValiError in transform seems like a good alternative and easy to add, I just don't like very much the lack of parity between pipeline and transformations (but maybe that's just me that needs some getting used to - they are already different in many senses)

Also maybe I wouldn't necessarily throw a ValiError since the transform would still need to catch to "merge" the issue with the info object, so I guess a throw getTransformIssue(transform, error, input?).
I get this isn't a little unorthodox (I personally use sometimes 😅) so I understand if you're not on board, just throwing it (pun not intended 😅) out there.

Another solution might be to validate your JSON in our pipeline (maybe with a new json validation function) and then transform it in transform. Since transform is only executed if there are no issues, this should work.

@fabian-hiller the problem with that approach is running the same code twice (once for validation, other for transformation). For the JSON case that would be fine (and a new type might even make more sense - but could also be done in userland).
For more complex cases (like the authentication I case wrote above), running the check twice wouldn't work.

Mhm... that's probably true. At the moment I don't know exactly how I would proceed. Every solution seems to be not optimal.

Perhaps a similar approach with a function passed as an argument as in Zod is a better compromise.

Do you mean, instead of:
.transform(value => newValue);

you're suggesting:
.transform((value, addIssue) => {
  if (something) {
    addIssue(issue);
    return
  }
  return newValue;
});

?

Yes for example.

I'm in the process of validating search parameters and my first assumption was that the transform function would actually accept an error message and handle thrown errors by default.
This is especially useful for Temporal.PlainTime/PlainDate.from() as they throw if the argument cannot be parsed correctly.
Throwing a ValiError would be really useful but also providing a default error message for other errors would awesome.
My current schema looks like this:
export const PlainDateStringSchema = transform(
  string([
    custom((input) => {
      try {
        PlainDate.from(input);
        return true;
      } catch (_) {
        return false;
      }
    }, "Must be a valid date."),
  ]),
  (input) => PlainDate.from(input),
);

But could be reduced to something like this if errors were handled automatically:
transform(
  string(),
  (input) => PlainDate.from(input),
  "Must be a valid date."
)

Thank you very much! I am currently rewriting Valibot based on discussion #463. I will take a look at your code once that is done. Feel free to ping me if I forget to answer.

I would very much like to be able to either throw a ValiError from inside a transform action, or have a zod-like addIssue method and be able to return a v.NEVER symbol.
I have a transform function which returns a branded string value or undefined (if the transformation fails) and I need to be able to error on the undefined case.
declare const _BrandedFooString__Brand: unique symbol;
type BrandedFooString = string & { [_BrandedFooString__Brand]: true };

declare function parseFoo(str: string): BrandedFooString | undefined;

const fooSchema = v.transform(v.string(), parseFoo);
//      ^? v.SchemaWithTransform<v.StringSchema<string>, BrandedFooString | undefined>

If I try to further validate the transformed value, to get rid of the undefined value, I lose the braned type, and the type still includes undefined
const fooSchema = v.transform(v.string(), parseFoo, v.string('Not valid foo'));
//      ^? v.SchemaWithTransform<v.StringSchema<string>, string | undefined>

The Type I want is v.SchemaWithTransform<v.StringSchema<string>, BrandedFooString>
The solution I'm left with at the moment is this borderline abomination:
const fooSchema = v.transform(
  v.transform(
    v.string(),
    parseFoo,
    v.string('Not valid foo')
  ),
  (v) => v as BrandedFooString  // ack, manual type assertion
);

What I'd love to be able to do is this:
const fooSchema = v.transform(
  v.string(),
  (value) => {
    const foo = parseFoo(value);
    if (foo === undefined) {
      throw new v.ValiTransformError('Not valid foo');
    }
    return foo;
  }
);

For now, I recommend using the pipe of string to validate, e.g. with regex, that parseFoo never returns undefined. But I will investigate the possibility of adding issues in transform after we merge the ongoing rewrite of the entire library in #502.

ok, if parseFoo performs minor cleanup/transformation on the input before branding and returning it
...how would you use the pipe for that?
Is there a page in the docs I might have missed?

Sorry, it's before my coffe o'clock here. I understand what you mean now.
Essentially this, right?
const fooSchema = v.transform(
  v.transform(
    v.string(),
    parseFoo,
    [v.regex(/./)]
  ),
  (v) => v as BrandedFooString  // ack, manual type assertion
);

Right now you have two options. The first is to validate the input before executing your transform function. The second is to validate the output of the transformation.
// Option 1
const Schema = v.transform(v.string([v.decimal()]), parseInt);

// Option 2
const Schema = v.transform(v.string(), parseInt, v.number());

I would like it if thrown errors are not automatically handled. I would like to be able to distinguish between issues I'm intentionally returning and errors that are problems in my schema code.
That also makes it a bit more natural to pass a message.
function toNumber(message = "Invalid number syntax") {
  return v.transform(input => {
    let trimmed = input.trim()
    if (trimmed === "") {
      return null
    }
    let parsed = Number(trimmed)
    if (!Number.isFinite(parsed)) {
      return v.issue(message)
    }
    return parsed
  })
}

let urlIntParam = v.nullish(
  v.pipe(
    v.string()
    toNumber("custom message"),
    v.nullable(v.pipe(v.number(), v.safeInteger()))
  ),
  null,
)

You may want to consider the experience adding types for these actions/messages though:
// v.action() would handle the types all internally for you
const toNumber = v.action((message = "Invalid number syntax") => {
  return v.transform(input => {
    // ...
    return v.issue(message)
  })
})

toNumber("custom message")

@jamiebuilds this is related to #597

This is now possible with our new rawTransform API.


Test failures from Radical pilot executor

Describe the bug
I noticed some instability in the radical pilot executor that caused failures in the CI tests for the mpi_experimental_3 branch. These failures are intermittent so rerunning failed tasks gets past these errors, but it would be good to have these tests or the cause of instability fixed.

Hey @yadudoc, thanks for raising this. It seems like the error source is from the job submission layer (saga/PSI-J).
I am on it and I will try to reproduce it if possible.

since some recent changes that hopefully fixed other CI failures, it looks like this error is now the dominant cause of CI test failures that I am experiencing.

(this has affected every PR I've tried to merge or review today)

@benclifford I am preparing a PR that fixes this in addition to several fixes by today.

@yadudoc , @benclifford, a PR to fix this issue and others is open now https://github.com/Parsl/parsl/pull/3060.

@AymenFJA 's fix PR #3060 didn't satisfy my "why is this broken / how does this fix it?" curiosity, so I dug in a bit today. I've identified these issues which interact here, I think, in no particular order:

radical.saga PTYProcess misbehaves when more than FD_SETSIZE (I think) are open: file descriptors returned by fork cannot always be used with select, resulting in base error message that is reported in this issue #3029: ValueError: filedescriptor out of range in select() means the given file descriptor is bigger than FD_SETSIZE despite otherwise being valid (I think).

some part of the pytest stack hangs at exit when the above error occurs during a test, rather than exiting cleanly. I haven't investigated what is hanging there - I think it should be reproducible by adding a raise call into radical.saga at the select line mentioned above.

parsl tests in --config local mode are accumulating around 10 file descriptors per test: so it's pretty easy for 1024 open FDs to be reached, if the radical test happens to run late in the randomised order. On my laptop I've seen other fd exhaustion errors, in a ulimit constrained environment, distinct from this #3029 radical hang. I've documented this more in issue #3077

I did some more hacking for the hang described about as "some part of the pytest stack hangs at exit": the stack is waiting for a process launched in radical.pilot/proxy.py to exit before the whole Python process will exit (as that is how multiprocessing processes work) -- however it looks like that process does not exit in this situation:
I modified my Python runtime so that it reports the name of processes that it waits for at exit, and I modified radical.pilot to add a name= on all processes, and I see this at the end of the hanging test when I'd expect pytest to exit.
        proc = mp.Process(target=self._worker, args=(sid, q, term, self._path), name="BENC: proxy")

and then:
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! stopping after 1 failures !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
================================================ 2 warnings, 1 error in 8.06s ================================================
calling join() for process BENC: proxy

If I set daemon=True on that proxy process, pytest exits without hanging, but I have not tried to understand if that is a safe thing for proxy.py to do - it was just to understand if that was the only hang.

none of the above gives any explanation for what works (or does not work) about Aymen's PR #3060 though... so I'll try to understand that.

Hi Ben,
thanks for deep-diving into this!
As Aymen said earlier, SAGA is on it's way out of RP and we'll likely not attempt to fix this on the SAGA layer.  We still use it in some places as PSI/J does not jet support data staging.  The PARSL tests triggering these kind of errors only add to the motivation to complete the transition though ;-)  Let me have a look over the next few days to see if I can do a hackish SAGA replacement for the code path relevant to this.
PS: Setting daemon on the RP processes will avoid the hangs, but at the same time may result in zombie processes.  Getting Python to clean up nicely when subprocesses and threads are mixed has been (an continues to be) a challenge unfortunately.


Variable Key : Value pairs not working for Athena data source

What happened?
We are trying to use the Athena data source to load data from a simple table consisting of metaname (key) and deviceid (value).
As described in below articles, it should be possible (and simple) to extract these for use in a Variable dropdown as key-value pairs:
https://github.com/grafana/grafana/issues/30012
https://grafana.com/blog/2019/07/17/ask-us-anything-how-to-alias-dashboard-variables-in-grafana-in-sql/
However, in practice this does not seem to work when using the Amazon Athena data source.
To showcase this, see below:
The query I am using
I am using below query:
SELECT
    deviceid AS __value,
    metaname AS __text
FROM
    tbl_aggregations_devicemeta

Using query in regular panel
As evident, the query returns the expected results as two separate columns:

Using query in Grafana variable
Below is the result when using this query in a Grafana variable. As evident, the query returns the values (instead of the keys) for the dropdown. Futher, only the values are returned when fetching the value of the dropdown as well - it's not possible either to explicitly extract the metaname keys by using the :text modifier.

This is a very basic requirement for dynamic Variable dropdowns, so it would be great to have a fix for this.
What did you expect to happen?
I expected the Grafana Variable query to create a dropdown showing the metaname keys and that when I reference this Variable in my queries, I would get the deviceid values.
Did this work before?
No, we reported this as a bug in an older issue, https://github.com/grafana/grafana/issues/30012, but that issue was closed.
How do we reproduce it?
It should be easy to reproduce by creating a simple key/value table and using Amazon Athena to query this within the Variable section.
Is the bug inside a dashboard panel?
No, it is inside the Variable section.
Environment (with versions)?
Grafana: latest cloud version
OS: Windows 10
Browser: Chrome
Grafana platform?
I use Grafana Cloud
Datasource(s)?
Athena

Hi @MatinF, the documentation you're referencing is a bit outdated, here's the current one: Athena readme  and Grafana Template Variable docs
As you can see the text and value fields are defined as just that, without the __ before it.
Does that work for you?

Thanks a ton, that made it work!


Backups with FTP upload

Hello,
it would be great to have the ability to create compressed backups of worlds that the user picked.
The backups should be stored either on the local harddrive or uploaded on an external FTP server.
And the backups should be automated created at a time set at an schedule.
Let me now what you think. ;)

I like this idea
Op ma 30 mrt. 2015 13:00 schreef Cometex<EMAIL_ADDRESS>
Hello,
it would be great to have the ability to create compressed backups of
worlds that the user picked.
The backups should be stored either on the local harddrive or uploaded on
an external FTP server.
And the backups should be automated created at a time set at an schedule.
Let me now what you think. ;)
—
Reply to this email directly or view it on GitHub
https://github.com/PufferPanel/PufferPanel/issues/473.

Any updates on this? It would be an awesome feature :)

This would actually need to be a Scales request, as all of that logic would run off Scales.

Yeah, it looks like this was created before Scales was available... should I recreate this issue over in the Scales repository then?

I am fine with this being here still since it's not exclusively for Scales. @computerwizjared for updates on our current development progress please see this post.

Thank you :)


Let's create a "first time only" tag for the newcomers

A lot of people are interested in contributing to this project, but they might not know how to start.
Let's create a "first time only" kind of a tag and let's also create a CONTRIBUTING.md file,
with somebody more experienced than I will describe the main ways of contributing to the project.
It also won't hurt to raise a few example issues tagged "first time only".

FWIW, there is also a link to a "Contributing" guide from the Paragon documentation website:

But I believe it does require you to have access to the Open edX Confluence.

Opting to close this issue as we have created a "good first issue" label, this PR will be merging shortly to add CONTRIBUTING.md file, and I split out a follow-on task of ensuring the "Contributing" link in the documentation website (shown above) links to a page on the documentation website itself rather than leading to Confluence, which requires authentication.


SteamOS (Arch) x86 linux. fusion helper "bind exeption"

console outputs:
[B]Bind exception: System.Net.Sockets.SocketException (98): Address already in use
   at System.Net.Sockets.Socket.UpdateStatusAfterSocketErrorAndThrowException(SocketError error, String callerName)
   at System.Net.Sockets.Socket.DoBind(EndPoint endPointSnapshot, SocketAddress socketAddress)
   at System.Net.Sockets.Socket.Bind(EndPoint localEP)
   at LiteNetLib.NetManager.BindSocket(Socket socket, IPEndPoint ep), errorCode: AddressAlreadyInUse
Initialized UDP socket on port 0

Fusionhelper has the required firewall permissions
ive tried running sudo, ive tried disabling the read only rootfs
nothing has worked.
Im not sure if there is a dependency im not told about or nothing
steam is open and steamvr is in my library

the issue was fixed by installing microsoft .net for linux. This should be in the readme and not hidden in the files for someone to find.

Just wondering how did you got .net on the steam deck?

Just wondering how did you got .net on the steam deck?

digging through the files there is an SH file to install .net linux just make sure to use "sudo steamos-readonly disable" before running it. after run "sudo steamos-readonly enable".

Just wondering how did you got .net on the steam deck?

digging through the files there is an SH file to install .net linux just make sure to use "sudo steamos-readonly disable" before running it. after run "sudo steamos-readonly enable".

Can't seem to find the SH file but thanks for the help

Person fixed issue


No line breaks in multiline Java output after switching to EKS 1.21 and containerd.

Last week I've updated my EKS cluster to 1.21 and enabled containerd. Logging works (I had to set pathDest to /var/log/pods and logFormatType to cri), but I have one issue: my multiline Java stacks previously had line breaks inside, which caused the output was easy readable for humans. Now it's all in one line and it's hard to read. Is there something I could do in splunk-connect itself or it's just 'a feature, not a bug' in CRI and there is nothing I can do?
Here is my current regexp for multiline:
firstline: /^\d{4}-\d{2}-\d{2}\s\d{2}:\d{2}:\d{2}|^\d{4}-\d{2}-\d{2}\s\d{2}:\d{2}:\d{2}|^\d{1,3}.\d{1,3}.\d{1,3}.\d{1,3}\s-\s-/

and I also tried with it disabled, but still to not avail.
The bad sign (I suppose?) is I have no line breaks in raw log stream on the node itself. Is there anything I could adjust?

Maybe to be more precise, here is some output:
The 'before' state, where line breaks were respected:
2022-07-06 11:07:07.336 ERROR 12 --- [3 - domainevent] c.c.t.messaging.sqs.CompanyRouteBuilder      : [] Failed delivery for [....]

Message History (source location and message history is disabled)
---------------------------------------------------------------------------------------------------------------------------------------
Source                                   ID                             Processor                                          Elapsed (ms)
[...]

The 'after' state, where line breaks are totally omitted:
2022-07-26 09:33:34.284 ERROR 11 --- [7 - domainevent] c.c.t.messaging.sqs.CompanyRouteBuilder      : [] Failed delivery for [...]Message History (source location and message history is disabled)---------------------------------------------------------------------------------------------------------------------------------------Source                                   ID                             Processor                                          Elapsed (ms)[...]

(Of course there is a lot more in places I've put [...], but as it's sensitive/internal data, I had to make them redacted a bit.
The only change after switching to EKS 1.21 with containerd was adding the following block:
containers:
  pathDest: /var/log/pods
  logFormatType: cri

to my values.yaml file (as previously we had json from pure docker). I've also tested with /var/log/containers, but there is no difference anyway.

Hi @ftpd, sorry for the delayed response. Actually, it is expected behaviour. You can use provide a line separator as well. It is not documented, so need to add one.
Sample config:
logs:
    <name>:
      from:
        pod: <pod-name>
      multiline:
        firstline: /<regex>/
        separator: "\n"


Extent aggregate problem

I'm using the 0.8 version and there is a problem with ST_EXTENT:
@Override
protected void add(Geometry geometry) {
if (result == null) {
result = geometry;
} else {
if (geometry != null) {
result = result.union(geometry.getEnvelope());
}
}
}
If result is null you set it to geometry. The next geometry gets its envelope added with union. Now, this will work if the geometries are polygon. But when the geometry is for example LineString, the first union will create a geometry collection (linestring from the first iteration and the polygon from GetEnvelope...) which can not be unified (JTS union actually has a check: checkNotGeometryCollection(this);)
The solution is to set the result to geometry.getEnvelope in the first iteration.

Thanks for reporting the issue. It makes sense but when I tried to reproduce I could not. Any chance you can provide a sample of geometries that lead to the issue?

Here is how i can replicate the problem. Firstly i create a database using the geotools (so all the aliases get setup) then i connected the DBeaver to this database and executed the following lines: (also note that the same thing happens on purely geotools created tables)
CREATE TABLE PUBLIC.TEST_EXTENT (
ID INTEGER NOT NULL,
GEOM BLOB
);
insert into PUBLIC.TEST_EXTENT(ID, GEOM) VALUES(1, ST_GeomFromText('LINESTRING (401144.32 4886020.44, 401169.16 4886039.33)', 2170));
insert into PUBLIC.TEST_EXTENT(ID, GEOM) VALUES(2, ST_GeomFromText('LINESTRING (401169.16 4886039.33, 401169.48 4886039.56)', 2170));
insert into PUBLIC.TEST_EXTENT(ID, GEOM) VALUES(3, ST_GeomFromText('LINESTRING (401157.54 4886055.71, 401169.48 4886039.56)', 2170));
-- all fine
SELECT ST_ENVELOPE(geom) FROM TEST_EXTENT
-- exception: General error: java.lang.IllegalArgumentException: This method does not support GeometryCollection arguments; SQL statement:
SELECT ST_Extent(geom) FROM TEST_EXTENT


Add Link to .Net Upgrade Assistant

Would be handy so a user doesn't have to go search for the approved upgrade assistant.

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

ID: 3275e223-a9cc-1efb-07ec-01c8501d7a46
Version Independent ID: 538673ca-22c5-2a5c-97c0-1552ae9f7b79
Content: Migrate your app from Xamarin.Forms to .NET MAUI - .NET MAUI
Content Source: docs/get-started/migrate.md
Product: dotnet-mobile
Technology: dotnet-maui
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @savionlee
It'll be linked to once it's finished and works :)


[TODO] A more stable Guest Token retrival method.

During a long duration data retrival session when AWS proxies are being used, guest tokens could expire, leading to a token.refresh . However, since Twitter wouldn't send guest token to AWS IPs, all following requests made after the token expires would fail. There need to be a way to retrive guest token using viable IP (either your own, or another resident proxy route), in order to achieve long term stability.

Currently in the main branch, the Guest Token is always retrieved without a proxy. Such is that our IP is still exposed to Twitter, which could then detect the frequent guest token request when doing large scale scrapping. By creating a proxy for retrieving guest token, we would be able to hide our IP and retrieve guest token in a more stable fashion.
However, if our proxies are AWS IPs, then this might not actually work well. Maybe we should consider implimenting a seperate proxy method for this.

Currently in the main branch, the Guest Token is always retrieved without a proxy. Such is that our IP is still exposed to Twitter, which could then detect the frequent guest token request when doing large scale scrapping. By creating a proxy for retrieving guest token, we would be able to hide our IP and retrieve guest token in a more stable fashion.
However, if our proxies are AWS IPs, then this might not actually work well. Maybe we should consider implimenting a seperate proxy method for this.

A temporary fix is implimented which only support http proxy.

A temporary fix is implimented which only support http proxy.

Scratch that, this fix is not actually working.

Scratch that, this fix is not actually working.

A temporal solution is in place, for now. This significantly decrease the speed of scrapping, due to the need of using proxy for token.

A temporal solution is in place, for now. This significantly decrease the speed of scrapping, due to the need of using proxy for token.


JSON Encoding errors are not precise enough

Steps to Reproduce
While I ran into this error while working on my own serializer, and didn't think about JSON restrictions, this could anger a programmer to no end.
json.encode({250: 250, 251: 251});

This fails with this error:
I/flutter ( 7618): ══╡ EXCEPTION CAUGHT BY GESTURE ╞═══════════════════════════════════════════════════════════════════
I/flutter ( 7618): The following JsonUnsupportedObjectError was thrown while handling a gesture:
I/flutter ( 7618): Converting object to an encodable object failed: _LinkedHashMap len:2
I/flutter ( 7618):
I/flutter ( 7618): When the exception was thrown, this was the stack:
I/flutter ( 7618): #0      _JsonStringifier.writeObject (dart:convert/json.dart:703:7)
I/flutter ( 7618): #1      _JsonStringStringifier.printOn (dart:convert/json.dart:902:17)
I/flutter ( 7618): #2      _JsonStringStringifier.stringify (dart:convert/json.dart:884:5)
I/flutter ( 7618): #3      JsonEncoder.convert (dart:convert/json.dart:285:30)
I/flutter ( 7618): #4      JsonCodec.encode (dart:convert/json.dart:184:45)

The JSON encoding library probably should warn users about unsupported types on conversion, for example
JSON does not support having type int as a key.

Logs
[√] Flutter (Channel beta, v0.7.3, on Microsoft Windows [Version 10.0.17134.228], locale en-BE)
    • Flutter version 0.7.3 at C:\tools\flutter
    • Framework revision 3b309bda07 (11 days ago), 2018-08-28 12:39:24 -0700
    • Engine revision af42b6dc95
    • Dart version 2.1.0-dev.1.0.flutter-ccb16f7282

[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at C:\Android\android-sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • ANDROID_HOME = C:\Android\android-sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
    • All Android licenses accepted.

[√] Android Studio (version 3.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 27.1.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)

[√] Connected devices (2 available)
    • Archos 101c Neon             • A014091600000B7C • android-arm • Android 4.4.2 (API 19)
    • Android SDK built for x86 64 • emulator-5554    • android-x64 • Android 7.0 (API 24) (emulator)

• No issues found!

I am assuming you are using the json converter from dart:convert? In that case a better place for your bug report may be the Dart repository: https://github.com/dart-lang/sdk/issues/new

The error is from dart:convert/json.dart:703:7
I think it should be reported in https://github.com/dart-lang/sdk/issues


Method not found: 'System.Threading.Tasks.Task`1<Discord.IUserMessage>

In general, all my logic works well and I see the correct result. But in the logs I see an error generated by external code.
Is it a problem on my part?
{05:37:05 Command     Discord.Commands.CommandException: Error occurred executing "commandname" for UseTag in UserTag.
---> System.MissingMethodException: Method not found: 'System.Threading.Tasks.Task1<Discord.IUserMessage> Discord.UserExtensions.SendMessageAsync(Discord.IUser, System.String, Boolean, Discord.Embed, Discord.RequestOptions, Discord.AllowedMentions)'.    at Infobot.Handlers.CommandHandlerBase.CommandExecutedAsync(Optional1 command, ICommandContext context, IResult result)
at Discord.EventExtensions.InvokeAsync[T1,T2,T3](AsyncEvent`1 eventHandler, T1 arg1, T2 arg2, T3 arg3)
at Discord.Commands.CommandInfo.ExecuteInternalAsync(ICommandContext context, Object[] args, IServiceProvider services)
--- End of inner exception stack trace ---}
Exception: {"Error occurred executing "kurator" for KP0H™#8837 in @KP0H™#8837."}
Message: null
Severity: Error
Source: "Command"
msg.Exception.InnerException
{"Method not found: 'System.Threading.Tasks.Task1<Discord.IUserMessage> Discord.UserExtensions.SendMessageAsync(Discord.IUser, System.String, Boolean, Discord.Embed, Discord.RequestOptions, Discord.AllowedMentions)'."}     ClassName: null     Data: {System.Collections.ListDictionaryInternal}     HResult: -2146233069     HelpLink: null     InnerException: null     MemberName: null     Message: "Method not found: 'System.Threading.Tasks.Task1<Discord.IUserMessage> Discord.UserExtensions.SendMessageAsync(Discord.IUser, System.String, Boolean, Discord.Embed, Discord.RequestOptions, Discord.AllowedMentions)'."
Signature: null
Source: "Infobot"
StackTrace: "   at Infobot.Handlers.CommandHandlerBase.CommandExecutedAsync(Optional1 command, ICommandContext context, IResult result) in *blablabla*\\Handlers\\CommandHandlerBase.cs:line 120\r\n   at Discord.EventExtensions.<InvokeAsync>d__33.MoveNext()\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at Discord.Commands.CommandInfo.d__45.MoveNext()"
TargetSite: {System.Threading.Tasks.Task CommandExecutedAsync(Discord.Optional`1[Discord.Commands.CommandInfo], Discord.Commands.ICommandContext, Discord.Commands.IResult)}

The error Method not found means somthing is looking for that method that doesn't exist anymore because it has changed. You must have a package installed looking for this method.

Could you send the function that line 120 is in?

83  public Task CommandExecutedAsync(Optional<CommandInfo> command, ICommandContext context, IResult result) 84 {   //logic ... 118  context.Channel.SendMessageAsync($"***").Wait(); 119  return Task.CompletedTask; 120 }
With regular Discord.Net this code base class work fine. Problem after update to Discord.Net Labs

That exception happens when the library can't find the method with that exact signature. Make sure you're using the package Discord.Net.Labs.Commands (not Discord.Net.Commands), and make sure you have all Discord.Net Labs packages up to date.


Add Calendar Widget to lala-bar

Emm. Maybe next time it will be better to switch to another branch and make pr...


[BUG] Trouble installing in Windows 10

Hi guys,
I received my OAK D Lite package last week and am trying to install it on a Windows machine using the provided installer.
When I run the installer (3.0.5 version) it seems to function normally without any errors, but when I run the program I get the following error:

I have python 3.9 installed on my computer (from the windows store).
Here is my system information:

When this didn't work I also uninstalled the depthai software and tried the manual method using chocolatey to install components and use venv.  This ultimately led me to the same error, although sometimes I got an addition, different error:

I noticed a couple of warning during the manual install and I'm not sure if these are relevant:

This is the error I get when I tried after manually installing:

Sorry about the trouble here.  We have fixed this in 3.0.6 and then with additional corner-case improvements in 3.0.7:
https://github.com/luxonis/depthai/releases
Would you mind giving those a shot?
Thanks again,
Brandon

It looks like it was a cable issue.  I noticed the error codes said something about low speed usb, so I switched to a different cable and it worked!


Add .Net Standard 2.0 target

According to Open-source library guidance it would be good to target .Net Standard 2.0 to benefit from having a smaller package graph.
I would appreciate to help with it.

All packages have been updated on the dev branch


feat(cli): add barrel template to new command

Overview
What is it?

[x] Feature / enhancement
[ ] Bug
[ ] Docs / tests / types / typos

Description
This PR aims to implement the barrel template as suggested in the issue https://github.com/BuilderIO/qwik/issues/4933.
Use cases and why
An example of usage is:
qwik new MyComponent --barrel

The output of this example would be the generation of the file MyComponent/index.tsx in the src/components folder.
Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Worked on it with @lorenzoceglia and @fra-malagisi. 🙃

Hi, thanks for your contribution 👍


Check when song is paused. Allow to skip error message parameter.

Hi,
I modified your script to make it suit my needs.
When -e and -p parameters are omitted, error and pause messages are empty.
There is, however, one problem with current version: you can't just set pause message, you have to set error message as well.

Hello, thanks for helping out!
I was also thinking about adding some play status functionality.

@sebastian-frysztak Check out my update and see if you can spot any bugs. I made it so you can set only the parameters you want and the order doesn't matter.


Readme: feature list & examples.md fix

@petervidos please check the readme change.

Only README and tool change.


Is this project abandoned?

I noticed there's a ton of un-acknowledged outstanding issues and PRs.

The project is not abandoned, but it is fair to say it is stable and mature.  It is heavily exercised internally at Facebook on a daily basis by at least a half dozen projects.  That said, I have personally been letting you folks down however and for that I sincerely apologize.  I typically try to go through and cleanup issues at least once per month but have been distracted by a number of personal and professional projects and for that I'm truly sorry.
Toward the end of the year I'll have time to go through and cleanup PRs.

Stable and mature, but there at 18 PRs open and some fixing KIs. I think it hurts the credibility of the project for external pitches when it gets into the state that it has the last ~6 months. Litho also recently deprecated their stetho support, which adds to the wonder about whether or not it's still maintained. Just my 2c. Looking forward to development picking up again

@hzsweers I agree, this isn't an appropriate state for a project still in heavy use within Facebook and externally.  I'll resolve to do better.
As for Litho, there are separate issues at play there.  Litho is a complex abstraction and chrome://inspect's rules don't quite fit as well as we'd hoped.  We don't have a clear solution for the community yet at this time, but it otherwise doesn't suggest Stetho is dead either.


the request module has a CVE

can you remove the request module in favor of a non deprecated module like axios? there is a CVE in the request module:
  └─┬<EMAIL_ADDRESS>    └─┬<EMAIL_ADDRESS>      └─┬<EMAIL_ADDRESS>        └─┬<EMAIL_ADDRESS>          └──<EMAIL_ADDRESS>
+-------------+------------------+          +-------------------+---------------+---------------------------------------+
| json-schema | CVE-2021-3918    |          | 0.2.3             | 0.4.0         | nodejs-json-schema: prototype         |
|             |                  |          |                   |               | pollution vulnerability               |
|             |                  |          |                   |               | -->avd.aquasec.com/nvd/cve-2021-3918  |
+-------------+------------------+----------+-------------------+---------------+---------------------------------------+

Would love to hear more of a plan for this one. fetch is now in later node versions so another option seems to be to drop a version that uses fetch.

closing this as dup of https://github.com/jsforce/jsforce/issues/999


Allow for short @section syntax

Allow the syntax php artisan make:view --sections="title:Hello World,foo" to generate the following HTML:
@section('title', 'Hello World')

@section('foo')

@endsection

This is included since version 1.2.3!


[DUBBO-2321]Provider "async = true" need to be remove from url when register to registry#2321

What is the purpose of the change
provider "async = true" need to be remove from url when register to registry
Brief changelog
Issue: https://github.com/apache/incubator-dubbo/issues/2321

@chickenlj

Codecov Report

Merging #2322 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #2322      +/-   ##
============================================
- Coverage     54.81%   54.77%   -0.05%
+ Complexity     5237     5235       -2
============================================
  Files           573      573
  Lines         25455    25462       +7
  Branches       4537     4539       +2
============================================
- Hits          13953    13946       -7
- Misses         9408     9420      +12
- Partials       2094     2096       +2

Impacted Files
Coverage Δ
Complexity Δ

...e/dubbo/registry/integration/RegistryProtocol.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ache/dubbo/remoting/p2p/support/AbstractGroup.java
45.45% <0%> (-11.37%)
7% <0%> (-1%)

...org/apache/dubbo/rpc/filter/ActiveLimitFilter.java
83.33% <0%> (-5.56%)
6% <0%> (ø)

...he/dubbo/remoting/transport/netty/NettyServer.java
67.85% <0%> (-3.58%)
8% <0%> (-1%)

...pache/dubbo/registry/support/AbstractRegistry.java
67.41% <0%> (-1.88%)
62% <0%> (-1%)

...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java
87.85% <0%> (-1.87%)
15% <0%> (ø)

...apache/dubbo/rpc/protocol/dubbo/DubboProtocol.java
59.16% <0%> (+0.83%)
31% <0%> (+1%)
:arrow_up:

...dubbo/rpc/protocol/dubbo/CallbackServiceCodec.java
78.67% <0%> (+1.47%)
29% <0%> (ø)
:arrow_down:

...bo/remoting/transport/netty/NettyCodecAdapter.java
54.68% <0%> (+1.56%)
3% <0%> (ø)
:arrow_down:

...ubbo/rpc/protocol/dubbo/ChannelWrappedInvoker.java
45.83% <0%> (+4.16%)
3% <0%> (ø)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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

I agree with what carryxyh says in #2321, we should fix this problem on both Provider side and Consumer side:

Provider side, please focus on dev-meta for the changes made by @cvictory.
Consumer side, #2434 already merged.

Please reopen if necessary.


remove quick fix actions on run

fix https://github.com/microsoft/vscode/issues/162180
fix #162216
To Do:

[ ] fix broken keybinding
[ ] on hover of decoration, the command decoration peaks through fix w css or defer to another issue

@Tyriar didn't notice until just now that you'd put #162216 in planning and now I see why. This was pretty involved 😅

@Tyriar yes good pt, that's the next part I need to do. What's done thus far in this PR is managing a map of decorations/quick fixes instead of just one such that we can remove actions as they get run and won't remove the decoration until the quick fix list is empty


Refactor report_to_list

Refactor permissions check into can_change_or_view
Make group-by on multiple display fields work
Clean-up code around display field paths and totals
Generate values_list in a compatible (with history) way
Refactor sorting to use defaults rather than try/catch
Refactor choice_lists so that missing choices don't error
Refactor display_format with formatter function
Apply formatting to totals and arrange columns correctly
Simplify sort_helper based on new default-argument

These changes should be entirely backwards compatible.

This pull request resolves Issue #6.
This pull request also resolve issue 20 in django-report-builder. I'll have a separate pull requests with tests coming shortly.

This is nice. Did you see the report_to_list branch I made? It might be better to focus efforts there instead. What do you think?

I pulled to report_to_list since these are good tests. Clearly the report_to_list can't be merged as the approach is so different.  The report_to_list branch is still far less code so I still think it's the best way forward.
If the new report_to_list is going to take too long to reach feature parity - I could just merge this as is.

I did see the report_to_list branch but I had a harder time following code. It looked like you moved a lot of  the code to methods on Report. I can see that making sense but I'm wrapping up a project for a client this week so I ran with master.
I don't have time to merge this yet with report_to_list but hopefully the refactoring is a big win. These changes should also be entirely backwards compatible. I can't verify that without tests but I strived not to change semantics at all.
If you study the code, you'll see that lines 287 through 359 are actually verbatim. The diff fails to highlight this only because the indent level changed. I'm not using Custom Field nor am I familiar with it so I left it as-is. Everything else is pretty much just refactoring and code-shuffle. I do a couple things differently (like sorting with a default) but the semantics are un-changed.

Let's merge as is then. Would you have any interest in helping maintain these two projects? Lots of pull requests all the sudden.

Yes, I think so.
I'm trying to close another contract that will use this shortly. Assuming that goes through, I will continue to have a vested interest in this project.

Added - go ahead and merge these. When you're ready to make a pypi release I'll test against my own stuff and push to pypi.
If you get that other contract - let me know what you think about the other branch. It seems like we can do the same stuff with far less code. A made report utils in an attempt to share more code between this and the not popular django-admin-export. But I ended up having report_to_list having a lot of report builder specific code which defeats the point. Never enough time for perfection.
I'll have a new client using it too - hoping to get user stories from them soon. Should let me spend more time on this.

Merged. Ready for PyPI release.


Component library support

Usecase:
We would like to add a library of components. The path would be:
/node_modules/component-library/node_modules/custom-component/lib/
If the rendered template is somewhere else then this path won't be discovered.
Suggestion:
Add a way to add paths to the discovery feature

How about the following?:
component-library/marko-taglib.json_
{
    "import-taglibs": ["./package.json"]
}

Behind the scenes, Marko would read the package.json file to find the dependencies object. It would then loop over all of the module dependencies and do the following:
var taglibPathToTry = require.resolve('<module_name>/marko-taglib.json');

if (taglibPathToTry) {
    // Import the current taglib
}

Sound good?

That would be great.

New versions published with enhancement:
<EMAIL_ADDRESS><EMAIL_ADDRESS>Test for new feature are passing, but documentation has not been updated. Sample usage:
https://github.com/raptorjs/marko/blob/5a918e85d0a6e38e0aae4c96f5db273e48e1080b/test/test-project/marko-taglib.json#L102
Let me know if you see any issues.


iOS17: whitebar permanently visible

MaterialSkin:  4.1.1
iOS: 17.2.1
It's me once more. Just wanted to document the issue. It's not critical.
The iOS whitebar remains visible all the time at the bottom of MS .
It is afaik supposed to fade out after a while or should not be visible at all.
In the past there were tries on different other web-apps out there  to get it done similar to this way:
  <meta content="black-translucent" id="status_bar" name="apple-mobile-web-app-status-bar-style" />
However in several cases that turned out to be  insufficient. I couldn't find any recent discussions about the subject.
I've seen that you use "black" and not "black-translucent" in index.html .  Not sure if that'd be a hook.
Anyhow. It'd be nice to get rid of that iOS whitebar if it is not too much effort.
Thx for your efforts.

No, sorry this is a STUPID iOS thing.  By all means try replacing Material's "black" with "black-translucent"  - if that works I'll make the change. Then no work-around with adding padding would be required. s.

I don’t think it is iOS 17 only.  People having similar issues for years. However. It is still a visible flaw on a UI.
My web design skills are quite limited. On top comes that LMS beast you better don‘t touch. I can’t be of help beyond testing and reporting.
BTW.   You can believe me that I am not happy to spent hours on MS myself. But.  In todays world minor UI issues get a huge weight on the users opinion about the product/service. No matter if it is commercial or foc.
I think over the last couple of days MS improved a lot.
The whitebar is actually the last thing that‘s bugging me at the moment.

Its a flaw with Safari, or iOS - I only mentioned iOS17 as you state that version, and I have no access to it. I have tried setting Material to fullscreen, setting black-translucent, all to no avail. Not sure what else you expect of me? I am not a web developer, this is my first and only Javascript project.
If the "UI sucks" simply because of this, then its Safari/iOS's fault. As stated, if you can find a fix I'll gladly merge.
I'm not an iOS user - can't stand the restrictions (only 1 browser engine, can't install apps from elsewhere, etc.). However, I've tried on my wife's iPhone and even here with plain Safari the bottom/home bar stays. I think there's an accessibility setting to help with this.
But I've closed this issue as I will not be spending time on it. The padding was added so that this bar does not get in the way. Why iOS can't at least just make it semi-transparent?

Look. Don't get me wrong. You've built an excellent application.
It puts all other LMS controller apps to shame and that includes iPeng.
And me and thousands other very happy users highly appreciate your efforts.
Happy holidays.

There is a (pretty lame) workaround. Under iOS you can turn Guided Access under Accessibility on.
If you then open MS via HomeScreen icon and start Guided Access with a tripple-right-side-button-click you basically lock the app and put the phone in single-use mode. Only then the whitebar disappears!

Well a work-around is better than nothing.
As stated I don't personally have any use for iOS, hence not willing to spend a massive amount of time on an issue which, IMVHO, is a Safari/Apple issue. But if you ever find a real solution I'm happy to merge it.

I today tried all kind of hacks related to index.html  that I found on the web. Nothing worked.
BTW. I am a die-hard fan of Linux for decades (professionally and privately).
However. After fiddling around with Android for a while, I have to say: Never again!
iOS simply works. Support is awesome. Interwork is awesome. No way I go back to Android.

I'm not trying to persuade anyone to use Android, or any other OS. I have my personal reasons why iOS just does not work for me - but this is purely personal. My wife loves her iPhone, and I have never tried to persuade her otherwise. I just prefer my freedom, and feel iOS does not respect that (privacy yes, freedom no). iOS interwork is great between other Apple products, as to be expected. Interwork with others, no. It's also a great OS, easy to use, etc. But, as stated I don't like being told what I can and cannot do with my own device. But that's just me. Not saying Android is perfect, far from it. But (IMHO) iOS being bad does not mean Android is good.
Working on this skin has made me realise how much Apple holds back on WebKit, how its more crippled than it needs to be. All so webapps are not as good as real apps, therefore more store purchases. Which is fine, Apple are there to make money after-all (and are very good at that). It's just not for me. Works for you, then great - that's your choice, and I respect that.


Deploy existing SDK for flutter

Is this possible to deploy existing available SDK for flutter?
Actually, I have a SDK platform from a company like as firebase and I want to deploy it for flutter without writing again all of codes :).

I want to deploy this  SDK for flutter

Create a plugin https://flutter.io/developing-packages/#plugin

Is this still an issue?

Actually yes.

I think this is what lots of plugins do.
Probably better to ask on StackOverflow.


Fixed mistake in data-factory-azure-documentdb-connector.md

A 'from' and 'to' got switched in the first paragraph causing the article to repeat itself.

Hi @efraimbart, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@efraimbart - Thanks for the contribution.
@spelluru - Please review this update in the public repo. If you decide to merge, you should run this command from the private repo to merge the changes to your branch, then resolve the merge conflict:
git pull https://github.com/efraimbart/azure-content.git patch-1
Then add, commit, push, and create a PR. If you do that, we'll then close this PR.

@efraimbart - thanks for the contribution. I have made the same change and sent the content for live today. You should see the change in a few minutes.
@tysonn - please close this PR because I have made the change as part of a PR that has already been merged. Thanks.

@efraimbart
Thank you for this contribution. The changes have been made to the article.


Package the repo

I think it would be great to make the repo a python package, so that we can use it more easily, e. g. use Generator or Discriminator from outside the repo.
Would you accept a PR for that ?

Yes, of course.


Use latest jslint for NodeJs 0.12

The mojito-shaker failed at old jslint
@reid

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:


Extraparams (additional variables in widget) in gridview?

How to use extraparams (additional variables in widget) in gridview?

I do not understand the question, please elaborate more.

additional variables means that are not part of dataProvider object,
i needs to pass some custom variables in gridview to process/show in DataColumns area

It's possible by using use in anonymous functions: http://php.net/manual/en/functions.anonymous.php#example-186

Thanks


Request to add chaunceyjiang as org member

Requesting to add myself in Cilium org members list, as I will continue to contribute to cilium community.
My contributions:
PRs - https://github.com/cilium/cilium/pulls?q=+is%3Apr+author%3Achaunceyjiang+
Issues - https://github.com/cilium/cilium/issues/created_by/chaunceyjiang
Issues -  https://github.com/cilium/cilium-cli/issues/created_by/chaunceyjiang

Thanks for your contributions!


add checking for wmarchiver upload status

@amaltaro, Alan please review. (I will include in the tag for the test but please review)

Include name change for WMArchiver -> WMArchive to match the application name


speed up hot reloading in development

use zeitwerk as autoloader
remove jets server --reload option
adjust default autoload path
fix shared functions docs

[ ] I've added tests (if it's a bug, feature or enhancement)
[ ] I've adjusted the documentation (if it's a feature or enhancement)
[x] The test suite passes (run bundle exec rspec to verify this)

Summary

Change the autoloader to use zeitwerk instead of ActiveSupport. This allows us to use hot-reloading for development and web requests pretty easily.
Context

Related issues: #253 #240
Version Changes

Updated Jets to this branch, got the following error while running a jets g scaffold:
✦ ➜ g scaffold alert device_id origin destination date:datetime
Traceback (most recent call last):
        11: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/bin/ruby_executable_hooks:24:in `<main>'
        10: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/bin/ruby_executable_hooks:24:in `eval'
         9: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/bin/jets:23:in `<main>'
         8: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/bin/jets:23:in `load'
         7: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/bundler/gems/jets-1d2b10d2f0bb/exe/jets:13:in `<top (required)>'
         6: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/bundler/gems/jets-1d2b10d2f0bb/exe/jets:13:in `require'
         5: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/bundler/gems/jets-1d2b10d2f0bb/lib/jets.rb:67:in `<top (required)>'
         4: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:291:in `require'
         3: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:257:in `load_dependency'
         2: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:291:in `block in require'
         1: from /Users/sofianegargouri/.rvm/gems/ruby-2.6.1/gems/zeitwerk-2.1.6/lib/zeitwerk/kernel.rb:23:in `require'
/Users/sofianegargouri/.rvm/gems/ruby-2.6.1/gems/zeitwerk-2.1.6/lib/zeitwerk/kernel.rb:23:in `require': cannot load such file -- jets-gems (LoadError)

This PR has already been merged and released. For posterity though:

RE: cannot load such file -- jets-gems (LoadError)

Jets contains some git submodules. So you need to use something like this:
Gemfile
gem "jets", git: "https://github.com/myusername/jets", submodules: true, branch: "mybranch"

The submodules are noted here: http://rubyonjets.com/docs/contributing/


Add profit in autobuyer menu

To be perfect,just need to add the profit numers in autobuyer menu

Currently out of scope


Support for vhpiForGenerateK

If I want to access vhdl modules within a vhdl for generate statement, I need to explicitly use the __getattr__ function of the SimHandle object, since the name of the vhpiForGenerateK objects within the simulator contain parentheses.
For example, to dig down through a vhdl generate for loop, I would expect to use:
dut.vhdl_module_inst.GENERATE_FOR_LOOP(0).lower_inst.sig_name <= 1

This fails as python expects GENERATE_FOR_LOOP to be a function call when it sees (
It's currently achieveable with __getattr__ but a little clunky:
vhdl_module_handle = dut.vhdl_module_inst
gen_handle = vhdl_module_handle.__getattr__("GENERATE_FOR_LOOP(0)")
gen_handle.lower_inst.sig_name <= 1

@luked80 We'll implement this using slice indexing rather than parenthesis.  So your example would be:
dut.vhdl_module_inst.GENERATE_FOR_LOOP[0].lower_inst.sig_name <= 1

I assume that would work for you?


Implement the Solana Program Library (SPL)

Implement the following Solana Program Library on-chain programs:

[x] Token Program ( #76 )
[x] Token Swap Program ( #260 )
[ ] Token-Lending Program ( #254 )
[x] Associated Token Account Program ( #106 )
[x] Memo Program
[x] Name Service ( #101 )
[x] Shared memory Program ( #105 )
[ ] Stake Pool Program

Edit: After some thought we've decided not to include the Feature Proposal Program in our SPL implementation for v1
Edit2:
Discarded program implementations.

[ ] Token-Lending Program ( #254 )

Because the Token Lending Program isn't currently being used, rather, it's forks are, it will not be included in Solnet.Programs, this code has been moved to Solnet.Reference.Implementations which is a different repo that will not be published to nuget. This is merely so that the implementation can be used as reference, the majority of the relevant code will be published under Solnet.Solend.


Install steps for compose-on-kubernetes on Amazon EKS

Signed-off-by: Anca Iordache<EMAIL_ADDRESS>

Codecov Report

Merging #130 into master will decrease coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #130      +/-   ##
=========================================
- Coverage    64.7%   64.6%   -0.11%
=========================================
  Files          90      90
  Lines       10944   10944
=========================================
- Hits         7081    7070      -11
- Misses       3558    3564       +6
- Partials      305     310       +5

Impacted Files
Coverage Δ

internal/controller/resourceupdater.go
46.29% <0%> (-3.71%)
:arrow_down:

internal/controller/stackreconciler.go
70.21% <0%> (-3.2%)
:arrow_down:

internal/convert/service.go
97.65% <0%> (-1.57%)
:arrow_down:

internal/convert/pod.go
87.55% <0%> (-0.96%)
:arrow_down:

Continue to review full report at Codecov.

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


Change TransformState to NamedTuple

This is quite a large PR unfortunately but fixes #83 and also cleans up the TransformState code by moving from mutable dataclass to immutable NamedTuple which is also better encapsulation practice for functional code.
One thing that's nice is that NamedTuple is already added to the pytree registry for optree and torch (fixing #83), so we don't need to do that manually as before.
There was an issue with the aux handling as modifying aux is not possible to do in-place as we don't know the structure of aux before log_posterior is called, also aux is not guaranteed to be a TensorTree and could contain strings etc.
The proposed fix is to return a new state with all other attributes modified in-place (i.e. pointers to old state) but aux replaced.

Looks good to me, but in my opinion it would be good to check how this affects memory consumption and allocation - since you allocate a new state whenever you update it, which may be quite inefficient

This PR shouldn't affect memory consumption, it just changes the handling of the algorithm states to a better convention.
It would be good to have some numerics on memory consumption and the pros and cons of using inplace and even whether we should continue to support it.
There is also an element of horses-for-courses since for MCMC-style where you want to collect samples along a trajectory you need inplace=False whereas for optimization or deep ensemble-style where you only care about the final result you may prefer inplace=True if its faster (which I'm not sure on).

Looks good to me, but in my opinion it would be good to check how this affects memory consumption and allocation - since you allocate a new state whenever you update it, which may be quite inefficient

This PR shouldn't affect memory consumption, it just changes the handling of the algorithm states to a better convention.
It would be good to have some numerics on memory consumption and the pros and cons of using inplace and even whether we should continue to support it.
There is also an element of horses-for-courses since for MCMC-style where you want to collect samples along a trajectory you need inplace=False whereas for optimization or deep ensemble-style where you only care about the final result you may prefer inplace=True if its faster (which I'm not sure on).

I'm not sure about this: the previous inplace behaviour would change the elements of a previously allocated object. Now, you re-allocate a new object every time

Looks good to me, but in my opinion it would be good to check how this affects memory consumption and allocation - since you allocate a new state whenever you update it, which may be quite inefficient

This PR shouldn't affect memory consumption, it just changes the handling of the algorithm states to a better convention.
It would be good to have some numerics on memory consumption and the pros and cons of using inplace and even whether we should continue to support it.
There is also an element of horses-for-courses since for MCMC-style where you want to collect samples along a trajectory you need inplace=False whereas for optimization or deep ensemble-style where you only care about the final result you may prefer inplace=True if its faster (which I'm not sure on).

I'm not sure about this: the previous inplace behaviour would change the elements of a previously allocated object. Now, you re-allocate a new object every time

We define a new NamedTuple but if inplace=True all Tensors are pointers to the same memory as the previous ones (aside from for aux which is not guaranteed to be a TensorTree). This is also checked in the tests.


Logo idea

I made these in vector format if there's any interest in using them.

Nicely done! We could also investigate whether it's feasible to use the pieces of the Elm tangram to convey the React atom.

We could also investigate whether it's feasible to use the pieces of the Elm tangram to convey the React atom.

That's what I was going for :smile:

Well yeah.. :D
What I meant by that was "using only the 7 pieces of a single tangram puzzle".

I couldn't see an obvious way to make it work but can revisit.

@paulyoung Actually, just submit a PR with your first logo suggestion in place of the current one, and let's call it a day. It's really good-looking! :sparkling_heart:

I put this together for Dribbble, wanted to share,

👍