feat(go/adbc/driver/snowflake): improve bulk ingestion speed

What

Replace Snowflake bulk ingestion with Parquet-based approach with higher throughput and better type support

Previously: INSERT bind parameters were uploaded to a CSV-based stage, once per record batch
Now: Parquet files written concurrently to stage independently of record batch size. Parquet logical types are used to infer schema on COPY.

Tests to validate type support and consistency through Arrow -> Parquet -> Snowflake -> Arrow roundtrip
Improved type mapping between Arrow <-> Snowflake timestamps. TIMESTAMP_LTZ is more consistent with Arrow timestamp semantics than TIMESTAMP_TZ, which can lead to lossy roundtrips.
Minor bugfix where Snowflake local timestamps with timezone set to UTC were being interpreted as non-local.

Why

Implements #1327, which comes from improvement request #1322
BindStream ingestion is significantly faster
Arrow type support is improved

Methodology
The general approach for ingestion is most clearly demonstrated by the path taken when stmt.Bind() for a single record is used:
IngestRecord
flowchart LR
    A(Record) --> B(Write Parquet)
    B --> C(Upload File)
    C --> D(Execute COPY)
    D --> E(Check Row Count)

The Arrow record is written to a Parquet file due to its logical type support, compressibility, and native Snowflake support. The file is then uploaded to a temporary Snowflake stage via PUT query, and then loaded into the target table via COPY query. Once the COPY has finished, one more query to check the resulting row count is dispatched to accurately return the number of rows affected. This is used instead of counting the Arrow rows written in case there are any undetected losses when importing the uploaded file into Snowflake.
A similar approach is taken when ingesting an arbitrarily large stream of records via stmt.BindStream(), but makes use of several opportunities to parallelize the work involved at different stages:
IngestStream
flowchart LR

    A(Read Records) --> B(Write Parquet)

    A --> C(Write Parquet)
    A --> D(Write Parquet)
    A --> E(Write Parquet)

    B --> J(Buffer Pool)
    C --> J
    D --> J
    E --> J

    J --> K(Upload File)
    J --> L(Upload File)

    K --> M(Finalize COPY)
    L --> M

    M --> N(Check Row Count)

    O(File Ready) --> P(Execute COPY)
    P --> O

The same steps are used, but the stream of records is now distributed among a pool of Parquet writers. This step is inherently CPU-bound, so it is desirable for it to scale independently with the availability of logical cores for writing/compression. These Parquet files are written to a buffer pool in memory to help decouple the upload stage from writing, and so that a writer can start working on the next file while the last file it wrote is being uploaded. Uploads from the buffer pool also benefit from parallelism, but more so to maximize network utilization by limiting idle time between uploads and amortizing potential slowdown in any one upload.
Technically, only a single COPY command is required after the last file is uploaded in order to load the Parquet files into the Snowflake table. However, on many warehouses this operation takes as long or even longer than the upload itself but can be made faster by paying for a larger warehouse. Given the batched approach taken and that the COPY command is idempotent, we can execute COPY repeatedly as files are uploaded to load them into the table on an ongoing basis. These COPY queries are executed asynchronously and listen for an upload-completed callback to ensure at least one file will be loaded by the query (otherwise it will no-op so this just prevents spamming Snowflake with a bunch of no-op COPYs).
Empirically, ingestion works reasonably well on an XS warehouse. COPY speed is no longer a bottleneck with an S warehouse with high-speed home internet, or on an M warehouse with same-region data center networking.
Performance
Running on GCP e2-medium (shared-core 1 vCPU, 4GB RAM)
Snowflake warehouse size M, same GCP region as Snowflake account
Default ingestion settings
Benchmarking TPC-H Lineitem @ SF1 (6M Rows):

Current: 11m50s
New: 14s

Benchmarking TPC-H Lineitem @ SF10 (60M Rows):

Current: Didn't attempt
New: 1m16s

This configuration is CPU bound, so I did another attempt with more cores available...
Now with GCP e2-standard-4 (4 vCPU, 16GB RAM)
Benchmarking TPC-H Lineitem @ SF1 (6M Rows):

Current: 11m17s
New: 9.5s

Benchmarking TPC-H Lineitem @ SF10 (60M Rows):

Current: 1h47m
New: 45s

Considerations

Snowflake guides indicate that ingestion via CSV is the fastest. Experimentally, it does appear to be true that a COPY query on staged CSV files executes much faster than for Parquet files. However by distributing the COPY workloads in parallel to the batched file uploads, overall performance is better with Parquet since it can be compressed much more efficiently allowing the upload to complete in less time and with fewer bytes transferred than with CSV. Type support is also much better.
Single-Record ingestion performance is slightly worse than the previous INSERT-bind approach. As a rough idea, a record that previously ingested in about 1.7s now ingests in about 2.5s. However, the new approach does come with expanded type support and better consistency with the streaming approach.
An ingestion run that fails part-way through may leave the table with partial results. Transaction semantics may be added in the future by overriding the CopyConcurrency parameter to be 0, in which case only the final COPY will execute.

Additional Work
Blocking

~Timestamps will roundtrip properly after Arrow GH-39466 is closed. A test is included but skipped for now.~
~Date64 will roundtrip properly after Arrow GH-39456 is closed. A test is included but skipped for now.~

Non-Blocking

Compression codec and level are included in ingestOptions but are not configurable using stmt.SetOption(). It is trivial to add this, but it would be nice to be able to use the currently internal CompressionCodecFromString method to automatically pick up support for any other codecs added in the future. Captured in #1473.
List and Map types have some issues on ingestion. Snowflake returns SQL execution internal error whenever repetition level is greater than 0. Still some more investigation to do here. This is non-blocking because neither type was previously supported for ingestion.
Context cancelation is supported for all goroutines and queries executed as part of ingestion, except for the PUT query (i.e. file uploads). This issue is being tracked in gosnowflake 1028. In practice, it likely takes just a few seconds for in-progress uploads to complete and properly conclude cancelation. Once this issue is fixed, the queries would be canceled in Snowflake, allowing the process to exit faster and reduce unnecessary work.
~The code previously meant to map Snowflake types to Go types is no longer used. It may still be useful for binding an Arrow record to an arbitrary Update query, but stmt.Prepare should be implemented first to follow ADBC spec for binding parameters.~

@joellubi this looks pretty good to me, do we know why that single test is failing?

@zeroshade How do you feel about the resolution to changes you requested, and about handling the row group bytes written as a separate issue? I rebased to bring in the v15 libraries and was able to un-skip the test that depended on the issues fixed upstream. All appears to be green now.

@joellubi Looks good to me and we can address the bytes written as a separate issue. Thanks a ton for this! Would you mind commenting with any updated benchmarks on this comparing the old version to this, just so we have them in the notes (so we can include them in the release notes when this gets pushed out)

@joellubi Looks good to me and we can address the bytes written as a separate issue. Thanks a ton for this! Would you mind commenting with any updated benchmarks on this comparing the old version to this, just so we have them in the notes (so we can include them in the release notes when this gets pushed out)

Thanks! I just re-ran the benchmarks on the current main branch in the same cloud environment as the existing benchmarks. The numbers came out basically the same so I would say the numbers in the PR description and in the documentation are accurate.


CSS broken

Laravel Version: 5.5

Voyager Version: 1.0

PHP Version:

Database Driver & Version:

Description:
In voyager admin panel > settings
here in the bellow, new setting option area/ section's css has broken
Steps To Reproduce:

All fields are required when submitting issues, with very few exceptions.  Additionally, since you're mentioning CSS, it would likely be helpful to submit screenshots of the issue.
When you've updated the issue, we can reopen it.

Dear Eric Fletcher, here the screenshots of the issue.
Regards
Md Khurrum Jha Murad (Sony)AdvocateSupreme Court of Bangladesh
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>Contact: +880 1737 484802 <+880%201737-484802>
Website: www.sonyscript.com http://www.sonyscript.com
On Wed, Oct 11, 2017 at 8:57 AM, Eric Fletcher<EMAIL_ADDRESS>wrote:

Closed #1978 https://github.com/the-control-group/voyager/issues/1978.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/the-control-group/voyager/issues/1978#event-1287301059,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANDHxiYGYjzTJfP7KfzH_MQPmBJnZWAcks5srC6ngaJpZM4P0yAU
.


discrepancy in selector item

Challenge http://www.freecodecamp.com/challenges/waypoint-target-the-parent-of-an-element-using-jquery has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
challenge says to make the "left-well" element red but only allows user to advance if you make the target1 element red.

@duanwalker thanks for posting this issue!
If you still need help with this see my comment.
@BerkeleyTrue this issue can be closed, thanks.

Please review the Guidelines for Contributing, thank you!.


What is the point of the register_length() method?

(More a feature removal request, actually)
Both second_q.hamiltonians.hamiltonian.Hamiltonian and second_q.operators.SparseLabelOp (and accordingly all their children) require a .register_length() method. However, this method does not seem to be used anywhere at the moment? What is the point of having this method?
From a logical point of view, it seems weird to associate a register length with an operator, as the size of a qubit register should only be clear after mapping?
The docstrings are rather cryptic, with SparseLabelOp.register_length() simply saying: "Returns the register length" and Hamiltonian.register_length() the not much more informative "The size of the operator generated by the second_q_op method." How does one measure the size of an operator? Especially since SparseLabelOp is supposed to be rather generic and not make any assumptions about the types of string labels it holds, I am not really sure what I should expect from this method as a return value?

The idea here is that each operator should be able to inform about the number of "informational bits" which it acts upon.
Since the SparseLabelOp is generic (as you say) we have opted for a name which is generic.
To be more concrete, in the case of a FermionicOp its register_length equals the number of fermionic orbitals (or spin orbitals in an electronic structure picture). In a BosonicOp the register_length equals the number of bosonic modes.
Other cases may be more complicated, for example the VibrationalOp differentiates the different modals within each of its vibrational modes. Its register_length equals the sum of all modals along its modes.
Since the register_length function has a @property decorator, its value is accessed like an attribute (i.e. without parentheses). You will see this used around the stack in various places, but the meaning of the returned value is very much context-dependant.
Hope this clarifies things. Documentation of this could probably be improved.


Nested data type support

A lot of big data has nested data types, including structs, arrays, and maps. These are specific to big data / Spark and are not  found in pandas, so we would need to design the APIs. It's best to write a short design doc on how to access and operate on the aforementioned three types.
Some initial thoughts after discussing with @thunterdb ...
We could reuse Spark's acessors, e.g. a.b means struct a's subfield b; a[0] means first element of array a. df['struct_field'] should return a DataFrame (so a DataFrame is internally  backed by a query plan along with a column expression).
Note that the precedence should be if there is a column named a.b, then df['a.b'] should return a Series representing that column. But if there is no column named a.b, then df['a.b'] should return a Series representing struct column a's sub-field b.

Oh hm.. should I close #419 for now?

I feel that's still valid though.

are you folks aware of: http://pandas.pydata.org/pandas-docs/stable/development/extending.html#extension-types
these are in-pandas support for arbitrary data types defined (internally or externally). These in fact were developed to allow for nested data types. Using a backing store of pyarrow nested types shouldn't be that hard to support.

Hey @jreback. Thanks for some input here :-). Sounds like user defined type which PySpark has as well (but similarly rather internal and experimental).
pyarrow has nested types, yes but supporting to map an arbitrary type out of the box would be a non-trivial work. For instance, if we convert pandas to pyarrow somehow and then read it in JVM sides, what types should it be :D ? We should probably somehow interact some metadata for it as well.

@jreback thanks for commenting. Is there any docs or examples on how that works with pyarrow from the users' perspective?

@HyukjinKwon from your comments: https://github.com/databricks/koalas/issues/420#issuecomment-498561529
pandas provides full fledged Extension Dtypes (which are used in for example Datetime w/tz, Intervals, Categorical, nullable Ints for example), that simply are 1-d array containers. These work very similarly to how for example pandas holds floats (or ints or whatever). These are completely abstracted away from the user.
pyarrow is merely a way to hold nested data, but it is not necessary for the holder of the DataFrame to even introspect / peek at this data (or care) about the implementation; except to use 'special' methods, e.g. imagine holding JSON (we have a trivial example here.
There is a project (will be public  soon, but not yet) to actually add this in a first class way using pyarrow as the backing store.

Ah, thanks for details. I look forward that project to be public.
One problem we face is that currently the data itself is purely transferred via pyarrow and becomes a Pandas instance back at another node. So, other information has to be transferred in some ways to that node.
For PySpark's UDT, those other information like implementation class is held as JSON - inside that, the class definition is pickled and unpickled via cloudpickle later in another node.
As far as I know pyarrow <> pandas conversion isn't supported for Extension Dtypes out of the box  in pyarrow (correct me if I am wrong). Probably we should combine such way of PySpark's UDT to leverage pyarrow as the backing store so that they can be used in other nodes everywhere.

pyarrow is soon going to be able to fully replicate a round-trip of a DataFrame with extension types (and serialize to parquet), see: https://issues.apache.org/jira/browse/ARROW-2428, https://issues.apache.org/jira/browse/ARROW-5271, https://issues.apache.org/jira/browse/ARROW-3829

That would be nice to have it! I wonder how the types will be handled when they are roundtriped in non-Python ends (JVM for instance because that's what we do). If that's supported out of the box, yea, job done.


Chromium crashes

[25810:25888:0129/040905.056301:ERROR:mime_util_xdg.cc(136)] Invalid mime.cache file does not contain null prior to ALIAS_LIST_OFFSET=44
Warning: disabling flag --expose_wasm due to conflicting flags
Fontconfig error: Cannot load default config file: No such file: (null)
[25810:25810:0129/040924.022603:ERROR:interface_endpoint_client.cc(702)] Message 0 rejected by interface blink.mojom.WidgetHost
Warning: disabling flag --expose_wasm due to conflicting flags
# then Wayland reported the connection was lost

Using kinoite-laptop-userns-hardened.
Chromium is constantly crashing when using Github. Installed Extensions: NoScript, UBlock Origin.
Also Tabs are crashing when trying to write something on GrapheneOS Discuss.
gdb chromium-browser
Didnt work.

unable to repro. are you setting any custom config?

I will try a vanilla profile

Closing for housekeeping reasons. Please reopen if you can repro on a vanilla config.


0.10.2: Cursor.enumerate & Cursor.enumerateBulks maxDocs parameter not used

It seems to me that the maxDocs parameter of both methods is not used. Setting it to '1' doesn't limit the documents that are enumerated.

Fixed in PR #296.


cr: get pid from criu notify when restore

when restore container from a checkpoint directory, we should get
pid from criu notify, since c.initProcess has not been created.
Signed-off-by: Ace-Tang<EMAIL_ADDRESS>we need this patch or criu test in https://github.com/alibaba/pouch/pull/2516 will fail

Using git cherry-pick not git am patch, so I force-push


Implementing a more flexibel data ingest for a_b()

This PR is solving #140.
a_b() previously was not flexible in regard of data type of input. Now xr.DataArray, np.array, float(for frequency) and str (for polarization) are allowed.  Tests are updated to also checkxr.DataArray as input (also with str vor polarization) and one new raise.
This also will help to slacken nearby_rain_retrival().
ToDo:

[ ] Check if example notebooks still run
[ ] Check tests

Thanks to the tests i found a strange behavior of np.full_like which can be fixed by determining the dtype of np.nan:

Good to know!
But it is actually not a "strange behavior" because the [docs of np.full_like] state

Return a full array with the same shape and type as a given array

So you have to specify dtype to override the one from the provided array.

thanks for this enhancement 👍


Visual shading representation of unavailable hours

New Issue Checklist

[x] Use the latest version of CalendarKit
[x] I've read the Contribution Guidelines
[x] I've searched for existing GitHub issues

Issue Description
Is there an option to have certain times within the Calendar Timeline view to have a grey overlay? Representing unavailable times, or perhaps before/after business hours as a common use case.
Code I'm using with CalendarKit

N/A - looking for an enhancement, but asking as a question first

Result I am trying to achieve

Similar to this type of style (average mockup) that represents unavailable times before 7am

StackOverflow Question link

Related to issue #34
Hi, this is currently unavailable functionality. Would you like to implement this yourself and share with the community? Or would you like to make a one-off case for your own project by forking the library? Both options are possible.
Let me know which version you'd like to pick and I could guide you with the implementation,


Issue reporter should ask before overwriting clipboard

Issue Type: Bug
Make sure there is some data in clipboard before trying this. Select some text on a webpage for example and copy it.
Create an issue with the issue reporter
Enter steps, check include system and enabled extensions.
Click Preview on GitHub
Expected: Prompt saying that there is too much data to send to GitHub, would you like to overwrite the clipboard with information that you can paste in the GitHub issue?
Actual: Data overwritten in clipboard with data from issue reporter without any prompting.
It's not clear if there is still a query running as there is no progress feedback.
VS Code version: Code - Insiders 1.24.0-insider (bfd155e6039f64d2795c6b514a029ebd01a33527, 2018-05-29T05:16:50.002Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-3687U CPU @ 2.10GHz (4 x 2594)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: unavailable_softwarewebgl: enabledwebgl2: enabled

Memory (System)
7.89GB (1.22GB free)

Process Argv
C:\Program Files\Microsoft VS Code Insiders\Code - Insiders.exe

Screen Reader
no

VM
0%

Extensions (13)

Extension
Author (truncated)
Version

html-snippets
abu
0.2.1

vscode-eslint
dba
1.4.10

vscode-great-icons
emm
2.1.32

vscode-gutter-preview
kis
0.12.2

vscode-azureappservice
ms-
0.8.1

vscode-cosmosdb
ms-
0.7.1

azure-account
ms-
0.4.0

csharp
ms-
1.15.2

vs-keybindings
ms-
0.2.0

team
ms-
1.136.0

debugger-for-chrome
msj
4.4.3

addDocComments
ste
0.0.8

cordova-tools
vsm
1.3.10

Found while testing #50491

It'd be nice if this issue could be resolved by not requiring the clipboard operation in the first place.   To do that though, GitHub would need an API that allowed submitting all the issue detail with a web request directly from VS Code, and then return a URL from which that temporarily stored issue could be retrieved by a browser.

@msftrncs Yeah, I'd love to not use the clipboard for this. The way to avoid that with the current GitHub API would be to actually create the issue instead of previewing it, but this would require an auth piece


Fix login

Fixes #2155, primarily by downgrading Firebase from 10.x to 9.x

Oh look AngularFire's docs even say to use Firebase v9


New integration release: zeek v1.0.0

What does this PR do?
PR for a new integration Zeek 1.0.0
###Other Notes:
-- Pipeline and Facet group created for this integration are available in our sandbox and would be shared separately with the required teams.
-- Samples for the pipeline review would also be shared separately with the required teams.
-- OOTB detection rules JSON would be shared separately with the required teams.
-- Since during the standard attribute remapping we are not preserving the source attributes as per suggested best practices, it would result in filters using these standard attributes populating the values of other integrations as well as per current datadog behaviour.
-- We would make necessary adjustments in the PR if the samples/information from corelight team differ from the expectation if needed.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

Added DOCS-6996 for the documentation editorial review.

👋 @bhargavnariyanicrest I think you used the tile template when you created the integration and not logs. For log integrations, we also ship an example of a config file to avoid customers copy/paste things from our documentation. You can see one example with tenable which is also a log integration. You can see we have a bit more files because we actually bundle the file with the agent. Could you do the same thing for this (and the next) log integrations?
Feel free to ping me if you have more questions!

Done, can you please check this concern https://github.com/DataDog/integrations-core/pull/16499#discussion_r1441653005

Thanks for creating the docs for this! Most of my comments are for formatting, please let me know if you have any questions on my recommendations!

--> Thanks for the review comments, we have addressed various comments except for the ones that were as expected.

@ DD PMs: I re-synced the tile and assets to staging

Hello @mohittilala-crest @bhargavnariyanicrest 👋 FYI I'm going to push a commit to this PR to fix something with the changelogs. I don't want to bother you with this, this is more an internal change we are doing.
We are going to update the template used by the ddev create command very soon so you won't have to worry about it in the future.


Overwrite default tag instead of remove in non-revisioned installation

Please provide a description of this PR:
It seems that removing default tag webhooks per every istioctl install without revision is not needed.
They will be overwritten later on: https://github.com/istio/istio/blob/d8e3f8cca4d97fd6e41d8d6ff640bb3638ca9fa3/operator/cmd/mesh/install.go#L225

See also: https://github.com/istio/istio/issues/40134

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@cnvergence: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a9f55f79281255a042dd5ec15277ae5d7483f6ca
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I could test it by running a fresh installation of Istio, with the default tag pointing to the revision test.
istioctl install --set revision=test
Then I could run istioctl install without a revision successfully and without mutating webhook overlap errors.

/test release-notes_istio

@cnvergence: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
a9f55f79281255a042dd5ec15277ae5d7483f6ca
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio/wg-environments-maintainers PTAL

Thanks, @Monkeyanator!
one question - can it also be included in the next 1.15 release?

In response to a cherrypick label: new pull request created: #41646


[prim_ram] When to return read data

Currently, prim_ram_1p and prim_ram_2p don't agree on semantics when to return data on rdata.

prim_ram_2p returns data always. https://github.com/lowRISC/opentitan/blob/09a5791b1daacf390aa1f31d7f5d5d069b50753b/hw/ip/prim_generic/rtl/prim_generic_ram_2p.sv#L39-L42
prim_ram_1p returns data only on a read, and nothing on a write https://github.com/lowRISC/opentitan/blob/09a5791b1daacf390aa1f31d7f5d5d069b50753b/hw/ip/prim_generic/rtl/prim_generic_ram_1p.sv#L44-L53

Which behavior do we prefer? It should at least be consistent.
@tomroberts-lowrisc @msfschaffner @eunchan @tjaychen

Unless we're actually relying on that behavior, I'd say the second is preferable - only certain RAMs support writing and reading in the same cycle

I agree with Tom. I'm worried that if we always return data the design may
begin to rely on it unknowingly.  Then when we run into a real memory macro
that does not behave the same, the error will be hard to track down.
On Tue, May 12, 2020 at 6:00 AM Tom Roberts<EMAIL_ADDRESS>wrote:

Unless we're actually relying on that behavior, I'd say the second is
preferable - only certain RAMs support writing and reading in the same cycle
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/2201#issuecomment-627327060,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSUDGBP3EPVKQOC5O6LRRFB6BANCNFSM4M6ZPAPQ
.

A side comment - some 2P memories will have issues (undefined or strange behavior) when one writes and reads the same address across the two ports.  Or we would have to manually implement the bypassing of write-to-read if needed/expected.
The generic prim seems to accept this condition, not sure if we want to leave that as-is?  (@tjaychen 's comment made me think about subtle errors that could come up and this seems potentially like one of those.)

the 2P design is copied from Xilinx recommended 2P design to make it recognized as DPSRAM. I think we can add assertions on top of it to check the collisions.

SGTM.


Keep selected tab when switching views in the editor window.

When switching between editor and xml-view the selected tab is not remembered and it will reset to the first tab each switch.

Addressed in 0.2.5

Fixed in 0.2.5 release


Upgraded project from SFW 1.9.13 to <IP_ADDRESS>; no logon screen

Hi JaiQ,
I have upgraded (tried to) my current SFW V1.9.13 based Project to the most current V <IP_ADDRESS> according to your guidance within the newest serenity dev guide.
The Migration so far worked. I mean by that that only 3 warnings from SlickRemoteview and no Errors are left within Visual Studio.
The Project (script and web) compiles and the web browser is started and points to the Login URL
but: The Login Screen is completely blank and the following 2 errors are shown in Chrome:

mscorlib.js?v=HVXVT1tVrBXEMnCn7bC0ww2:304 Uncaught TypeError: Cannot read property '__typeName' of undefined

Failed to load resource: the Server responded with a Status of 404 --> http://localhost:/favicon.ico

The second error I think we can ignore but the 1st one I don't have any idea where to search for.

**Regarding the warnings: **
Update: I have found the fixed Versions of PermissionCheckEditor.cs and TranslationGrid.cs under the Github/Serene Project and going back in history.

I ask for your help to get the upgraded project working again.
May I ask you to Point out what has changed from V 1.9.13 when upgrading such a Project to V<IP_ADDRESS> because as I understand you in the dev guide, an existing Project should more or less work after an upgrade and conversion of client side projekt to "server-side" TypeScript is not yet enforced.
Thank you for your help and
With kind regards,
John

I don't know. You might have have missed something. Put a breakpoint and see whats that cannot read typeName of undefined comes from.

Hi JaiQ,
thanks for your feedback. I will then just try to upgrade the versions in smaller steps. Up to 1.9.19 it already fully works and I will see when it brakes.
With kind regards,
John


Ignore 'W504 line break after binary operator' and some other linting

What is the context of this PR?
Linting now failing with "W504 line break after binary operator".
I avoided fixing the errors as it'd create an unnecessary divergence from the upstream project.
How to review
...
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

Just going to merge this because this and #28 are dependent on each other for their builds to pass.


Do not panic on Layers by definition are suffixes of names

We catch the error and pass back to user instead of panicking. Fixes #162.

Background on what causes this:
Trying lookup the IP for riga.the-wedding.ru
->  A riga.the-wedding.ru -> roots
<- authoritatives for .ru and additionals for those NS. ( :thumbsup::skin-tone-4: )
->  A riga.the-wedding.ru -> .ru NS
<- authoritatives for the-wedding.ru: ns1.reg.ru, ns2.reg.ru and additionals for those NS ( :thumbsup::skin-tone-4: )
-> A riaa.the-wedding.ru -> ns2.reg.ru (specifically, <IP_ADDRESS>)
Now I would have hoped for an A record at this point, but instead, you get…
<- ;; AUTHORITY SECTION:
*.the-wedding.ru.	86400	IN	NS	ns1.reg.ru.
*.the-wedding.ru.	86400	IN	NS	ns2.reg.ru.
(which is nonsensical)


TLC: Integrate Supplemental Instruction (SI)

Complete the integration of the Fall 2018 project for TLC SI.

Using logic from the fall2018 branch, add TLC SI information to applicable classes on the Classes card and list views.
@woznaut sending your way to review/provide UI

first attempt at v1 with minimal testing and use of mock data completed in 7edaa5a0af29d47b68cf99858092b577e551cc5f

complete with mock data : 046db92814debe921fc29543d7bcfd13137e8f28
Most UI and data handling on client side are complete.
Now waiting for real data to make necessary changes in data parsing.

Verified in QA. The remainder of this task is blocked by #451.

completed with endpoint that has a mixture of real data and mock data in commit: 8fa1af93cbe0d50d5dfe0c7185201853cc0d7089

Verified on branch feature/tlc-integration, merged into 6.5

Resolved an issue where the TLC SI instructor names were not an exact match with SIS - 17 out of 35 records were affected. These records have been fixed, tested, and are ready to deploy.

TLC SI instructor names fixed, deployed, & verified


support publishing H265 and AV1 tracks with Enhanced RTMP (#1393) (#1446) (#1621)

Fixes #1446
Fixes #1621
TODO:

[x] support publishing H265 with enhanced RTMP
[ ] support publishing AV1 with enhanced RTMP

Codecov Report

Merging #1756 (11f3b37) into main (22fe655) will decrease coverage by 0.45%.
The diff coverage is 60.21%.

: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
@@            Coverage Diff             @@
##             main    #1756      +/-   ##
==========================================
- Coverage   62.98%   62.54%   -0.45%
==========================================
  Files          91       99       +8
  Lines       10072    10252     +180
==========================================
+ Hits         6344     6412      +68
- Misses       3276     3389     +113
+ Partials      452      451       -1

Impacted Files
Coverage Δ

internal/conf/logdestination.go
0.00% <0.00%> (ø)

internal/core/pprof.go
0.00% <0.00%> (ø)

internal/core/rpicamera_source.go
0.00% <0.00%> (ø)

internal/core/source.go
100.00% <ø> (ø)

internal/core/source_redirect.go
0.00% <0.00%> (ø)

internal/core/udp_source.go
0.00% <0.00%> (ø)

internal/formatprocessor/generic.go
56.00% <ø> (ø)

internal/formatprocessor/mpeg2audio.go
0.00% <0.00%> (ø)

internal/formatprocessor/mpeg4audio.go
0.00% <0.00%> (ø)

internal/formatprocessor/opus.go
0.00% <0.00%> (ø)

... and 58 more

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


拖动条显示多了

我实际给了2个子元素，给我1个split-bar就可以了，但是给了2个，下面的那个bar不能控制，很难看。
另外，有办法自定义bar的icon吗？

<Split mode={'vertical'} style={{height: '300px', overflow: 'hidden'}}>

    By dragging 'draggable' surface you can change size of the first pane. The first pane keeps then its size while the second pane is resized by browser window. By default it is the left pane for 'vertical' SplitPane and the top pane for 'horizontal' SplitPane. If you want to keep size of the second pane and let the first pane to shrink or grow by browser window dimensions, set SplitPane prop primary to second. In case of 'horizontal' SplitPane the height of bottom pane remains the same.

    By dragging 'draggable' surface you can change size of the first pane. The first pane keeps then its size while the second pane is resized by browser window. By default it is the left pane for 'vertical' SplitPane and the top pane for 'horizontal' SplitPane. If you want to keep size of the second pane and let the first pane to shrink or grow by browser window dimensions, set SplitPane prop primary to second. In case of 'horizontal' SplitPane the height of bottom pane remains the same.

</Split>

@gamefriends 自定义这个玩意儿？
滚动条是否能给个示例看看，得空我看看

@gamefriends 自定义这个玩意儿？
滚动条是否能给个示例看看，得空我看看

现在滚动条icon我是用::before 的contnt url来给的图片，但是这样很有局限性，比如按钮小了，只能用zoom来缩放，而且，调整位置也比较麻烦。
示例就是这个。
<Split mode={'vertical'} style={{height: '300px', overflow: 'hidden'}}>

    By dragging 'draggable' surface you can change size of the first pane. The first pane keeps then its size while the second pane is resized by browser window. By default it is the left pane for 'vertical' SplitPane and the top pane for 'horizontal' SplitPane. If you want to keep size of the second pane and let the first pane to shrink or grow by browser window dimensions, set SplitPane prop primary to second. In case of 'horizontal' SplitPane the height of bottom pane remains the same.

    By dragging 'draggable' surface you can change size of the first pane. The first pane keeps then its size while the second pane is resized by browser window. By default it is the left pane for 'vertical' SplitPane and the top pane for 'horizontal' SplitPane. If you want to keep size of the second pane and let the first pane to shrink or grow by browser window dimensions, set SplitPane prop primary to second. In case of 'horizontal' SplitPane the height of bottom pane remains the same.

</Split>

可以用 https://codesandbox.io/ 给我展示个示例？
提供一个 renderBar 的 API？
@gamefriends

https://codesandbox.io/s/jolly-carlos-brfyo?file=/src/App.js
应该是把style标签页作为显示对象遍历了。

可以用 https://codesandbox.io/ 给我展示个示例？
提供一个 renderBar 的 API？
@gamefriends

这样最好，用户可以自己根据需要调整样式结构。
@jaywcjlove

@gamefriends Upgrade<EMAIL_ADDRESS>


About dataset

Hi, authors
Thanks for your excellent work,
I would like to know  where to download the pan_maps for coco, cityscapes and ade20k dataset, (/data/dataset_annotations/training/coco_panoptic/pan_maps/)
Thanks

Hi,
The panoptic maps have to be downloaded from the respective dataset websites e.g. for COCO download the 2017 training images + the 2017 panoptic annotations from here: https://cocodataset.org/#download
For cityscapes you'll have to register on the webiste to get the download links. For ADE20k you should download the earlier 2016 MIT Scene Parsing variant from here: http://sceneparsing.csail.mit.edu/
For cityscapes and ADE20k the folder containing the panoptic annotations may not be called "pan_maps" so just rename it to that.

Hi,
Thanks for your prompt reply! It really helps.
For ADE20k,  The downloaded file is ADEChallengeData2016, is that correct?
For Cityscapes, could you pls tell me the name of the panoptic annotations?  I saw a few panoptic annotations on the official website.
Thank you!

Yes the folder for ade20k is called ADEChallengeData2016
For cityscapes, you should download the 'gtFine' annotations and then convert them to the coco panoptic format by running the conversion script provided by the cityscapes authors: https://github.com/mcordts/cityscapesScripts/blob/master/cityscapesscripts/preparation/createPanopticImgs.py

It's actually good that you brought this up because I did the data setup quite some time back and hence, forgot to document it in the readme. Let me know if this works or not.

Hi,
Thanks for your detailed information, I think ade, cityscape and coco are clear for me, but to download Mapillary Vistas, I would like to know if you use 1.2 or 2.0 version, and if the panoptic maps are included in it?
mapillary-vistas-dataset_public_v1.2.zip
mapillary-vistas-dataset_public_v2.0.zip
Thank you again for the help!

We actually used v1.2. The panoptic maps are included in the dataset under mapillary/training/v1.2/panoptic. You just need to link this folder to ${TARVIS_WORKSPACE_DIR}/dataset_annotations/training/mapillary_panoptic/pan_maps.


mob-name: Add translations

This PR adds translations to mob names. Credits to Chris Foryst#0131.

Merging for now; we can always review back later


Test failures on main

Noticed CI is failing the Windows latest job:
---- tests::verification_mock::tests::stapled_revoked_dns stdout ----
thread 'tests::verification_mock::tests::stapled_revoked_dns' panicked at rustls-platform-verifier\src\tests\mod.rs:51:9:
assertion `left == right` failed
  left: Ok(())
 right: Err(InvalidCertificate(Revoked))
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- tests::verification_mock::tests::stapled_revoked_ipv4 stdout ----
thread 'tests::verification_mock::tests::stapled_revoked_ipv4' panicked at rustls-platform-verifier\src\tests\mod.rs:51:9:
assertion `left == right` failed
  left: Ok(())
 right: Err(InvalidCertificate(Revoked))

---- tests::verification_mock::tests::stapled_revoked_ipv6 stdout ----
thread 'tests::verification_mock::tests::stapled_revoked_ipv6' panicked at rustls-platform-verifier\src\tests\mod.rs:51:9:
assertion `left == right` failed
  left: Ok(())
 right: Err(InvalidCertificate(Revoked))

failures:
    tests::verification_mock::tests::stapled_revoked_dns
    tests::verification_mock::tests::stapled_revoked_ipv4
    tests::verification_mock::tests::stapled_revoked_ipv6

And the Android task:
07-30 18:14:51.040  3199  3229 I rustls_platform_verif..: verifying Err(InvalidCertificate(UnknownIssuer))
07-30 18:14:51.042  3199  3229 W rustls_platform_verif..: certificate was not trusted: java.
org.rustls.platformverifier.CertificateVerifierTests > runMockTestSuite[test(AVD) - 9] FAILED
	org.junit.ComparisonFailure: A test failed. Check the logs above for Rust panics. expected:<[success]> but was:<[test failed!]>
	at org.junit.Assert.assertEquals(Assert.java:115)
security.cert.CertPathValidatorException: Trust anchor for certification path not found.

I won't have time to look into this right away, so filing for a rainy day.

I think that the mock suite's root expired the other day:

This might explain the test failure on Android (as it couldn't find a valid certification path), but I'm not sure what's happening on the Windows side yet.

I'm not sure what's happening on the Windows side yet.

Interestingly regenerating the expired root/intermediate also fixed the Windows build task.
I notice all the failures are related to stapled OCSP with revoked status. Is it possible that the Windows verifier doesn't consider revocation in that circumstance when the certificate chain is otherwise invalid due to expiry? :thinking:

Interestingly regenerating the expired root/intermediate also fixed the Windows build task.

Yeah I considered just doing that but I'm slightly concerned about the root cause of seeing Ok(()) in the test output 😓.

Is it possible that the Windows verifier doesn't consider revocation in that circumstance when the certificate chain is otherwise invalid due to expiry?

That would match with some behavior I've seen before IIRC but I have no idea why the platform verifier was returning Ok(()). I would have expected to see Err(Untrusted (or something close enough to that).

I'm slightly concerned about the root cause of seeing Ok(()) in the test output 😓.

Agreed - my thought process here was that we should investigate this deeper but it won't be better having main broken in the meantime.
Maybe I should cut a new issue specifically for this mystery so it won't get lost if this ticket gets closed with the CI fix?

I would have expected to see Err(Untrusted (or something close enough to that).

Also agreed :fearful:

With main fixed, I updated the issue title to be more reflective that this is now for hunting down whatever Windows weirdness is happening.


Add Trace level

From Dietmar Aust:

TRACE is a common level in many logging frameworks (not related to oracle trace files ;), see this for a discussion on it: http://wiki.apache.org/logging-log4j/TraceDebate

We'd really need to think this one through before adding it. It's not a technically difficult thing, rather we need a convincing way to separate Trace and Debug levels in the guidelines. If the two use cases are very similar it could confuse developers and could result in developers interchanging Trace and Debug which would ruin the entire point of adding it.

After discussing with Dietmar, we'll add a trace level. Here's an example of when we'll using it:
...

logger.log('before loop');
  for x in 1..100 loop
    logger.trace('in ' || i || ' value is ...');
    ...
  end loop;
logger.log('after loop');
...


feat: init client

Project Initialization
Router
Internationalization
Theme Switching Integration

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

@lxfu1

CLA is not signed.
make a new directory such as osgraph-web to store front-end code, so backend code (eg. SOFA/Java) will not be confused with it.


hyper-v: Added 5 new test cases, updated 3 test cases

[Please merge after PR 172 to avoid conflicts]
Added the following TCs:
-Regresion - VM heartbeat status after resume from PauseCritical state
-Test VM with max CPU cores count
-CheckNuma
-CheckNuma_Maximum
-Reboot VM with Different Memory Size
Changed BVT-CORE-RELOAD-MODULES to 2 TCs:
-LIS modules reload UP (stress test - 1000 iterations)
-LIS modules reload SMP (BVT test - 100 iterations)
Updated LIS version BVT-CORE-VERIFY-LIS-MODULES-VERSION TC.
Test results:
VHD under test xenial_16.04.3_x64.vhd
ARM Image under test  :  :  :
Total Executed TestCases        7 (       7 Pass,        0 Fail,        0 Abort)
Total Execution Time(dd:hh:mm) 0:0:30
XML file: TestConfiguration
BVT-CORE-RELOAD-MODULES-SMP Execution Time: 8.27 minutes
BVT-CORE-RELOAD-MODULES-SMP : PASS
Max-vCPU Execution Time: 5.53 minutes
Max-vCPU : PASS
Paused-Critical-Heartbeat Execution Time: 4.23 minutes
Paused-Critical-Heartbeat : PASS
Check-Numa Execution Time: 1.64 minutes
Check-Numa : PASS
Check-Numa-Maximum Execution Time: 1.44 minutes
Check-Numa-Maximum : PASS
Reboot-Different-Memory-Settings Execution Time: 3.48 minutes
Reboot-Different-Memory-Settings : PASS
STRESSTEST-RELOAD-MODULES-UP Execution Time: 6.19 minutes
STRESSTEST-RELOAD-MODULES-UP : PASS
VHD under test Centos_7.5_x64.vhd
ARM Image under test  :  :  :
Total Executed TestCases        7 (       7 Pass,        0 Fail,        0 Abort)
Total Execution Time(dd:hh:mm) 0:0:32
XML file: TestConfiguration
BVT-CORE-RELOAD-MODULES-SMP Execution Time: 8.96 minutes
BVT-CORE-RELOAD-MODULES-SMP : PASS
Max-vCPU Execution Time: 5.75 minutes
Max-vCPU : PASS
Paused-Critical-Heartbeat Execution Time: 4.3 minutes
Paused-Critical-Heartbeat : PASS
Check-Numa Execution Time: 1.58 minutes
Check-Numa : PASS
Check-Numa-Maximum Execution Time: 1.52 minutes
Check-Numa-Maximum : PASS
Reboot-Different-Memory-Settings Execution Time: 3.41 minutes
Reboot-Different-Memory-Settings : PASS
STRESSTEST-RELOAD-MODULES-UP Execution Time: 6.73 minutes
STRESSTEST-RELOAD-MODULES-UP : PASS

I have run PS Script Analyzer & made the changes suggested by it.

@adriansuhov Approving these changes.
Going forward -

Please don't use TC-COVERED values to avoid further cleanup. I've sent an email to Chris and Ju about this.
Make sure that GuestOnly functions are universal (meaning they can be used for Azure/HyperV VMs) and should not be restricted to HyperV only.
Regards,
Shital


I have also a problem with it even if it moves forward and creates the cluster, I have a problem regarding the internal registry.   Checking if kind cluster exists...
Creating cluster "kind" ...
DEBUG: docker/images.go:58] Image: kindest/node:v1.26.3@sha256:61b92f38dff6ccc29969e7aa154d34e38b89443af1a2c14e6cfbd2df6419c66f present locally
 ✓ Ensuring node image (kindest/node:v1.26.3) 🖼
 ✓ Preparing nodes 📦
 ✓ Writing configuration 📜
 ✓ Starting control-plane 🕹
 ✓ Installing CNI 🔌
 ✓ Installing StorageClass 💾
Set kubectl context to "kind-kind"
You can now use your cluster with:

kubectl cluster-info --context kind-kind

Thanks for using kind! 😊
configmap/local-registry-hosting created
716710aa3615603f3c3c7a5147013d833d0eecc8f846e5f8a8a88612c56c0ed1
docker: Error response from daemon: Ports are not available: exposing port TCP <IP_ADDRESS>:5000 -> <IP_ADDRESS>:0: listen tcp <IP_ADDRESS>:5000: bind: address already in use.

The problem is that it is not getting to expose the ports, so the registry container is created but not running:


Masking?

Hello. My sequences are of varying length so many of them are 0 padded. From what I understand the correct way to treat 0 padded sequences is with mask_zero=True on an embedding layer.
text_model = Sequential()
text_model.add(Embedding(max_features, 64, mask_zero=True))
text_model.add(BiDirectionLSTM(64, 64, return_sequences=True))
The above code raises the following exception:
"Exception: Cannot connect non-masking layer to layer with masked output"
Do you have any plans to support masking? It seems like an important feature for dealing with sequences.

that's definitely an important feature, I will look into it and see if i can add that in or if you are interested you can also try to look into adding that feature.

When you use mask_zero = True, you must use it on ALL your next layers.
BiDirectionLSTM(64, 64, return_sequences=True, mask_zero=True)
should work!


[SEC-1362] Allow user to set their cert IP instead of detecting

In some cases (like tethering on AT&T as of the past few days), the external IP used for http traffic is different from the external IP used for ssh. So allow the user to set their IP to a known value using the BLESSFIXEDIP environment variable.

+1

🚀


highlight SpecialKey and NonText

Steps to reproduce

rm -rf ~/.vimrc
gvim testfile   // tesfile should contains tab,eol
set list  lcs=tab:→\ ,space:·,nbsp:␣,eol:¶
highlight SpecialKey guifg=red gui=NONE
highlight NonText guifg=green gui=NONE

Expected behaviour
tab and eol should be with guifg=red
but tab with guifg=red, eol with guifg=green

Version of Vim
with the latest vim downloaed via choco install vim
Environment
gvim under Windows
Another things I found that highlight SpecialKey and NonText behavior different between vim and nvim
vim

:help SpecialKey

							*hl-SpecialKey*
SpecialKey	Meta and special keys listed with ":map", also for text used
		to show unprintable characters in the text, 'listchars'.
		Generally: Text that is displayed differently from what it
		really is.

:help NonText

							*hl-NonText*
NonText		'@' at the end of the window, "<<<" at the start of the window
		for 'smoothscroll', characters from 'showbreak' and other
		characters that do not really exist in the text, such as the
		">" displayed when a double-wide character doesn't fit at the
		end of the line.

nvim

:help SpecialKey

							*hl-SpecialKey*
SpecialKey	Unprintable characters: Text displayed differently from what
		it really is. But not 'listchars' whitespace. |hl-Whitespace|

:help NonText

							*hl-NonText*
NonText		'@' at the end of the window, characters from 'showbreak'
		and other characters that do not really exist in the text
		(e.g., ">" displayed when a double-wide character doesn't
		fit at the end of the line). See also |hl-EndOfBuffer|.

vim treats both keys listed with :map and listchars as SpecialKey. Sometimes we want color of listchars lighter (since tab,eol in source code should be lighter), but with keys listed with :map with darker color.
Personally I prefer nvim's solution to SpecialKey (without listchars) and NonText

I might be missing something, but Vim is exactly behaving like what is documented, isn't it? I am sure, if we change it, people will also complain, so that may not be the best idea.

yes, but there is a difference between unprintable characters in the text and characters that do not really exist in the text. So in this case it makes sense to distinguish between NonText and SpecialKey, because the eol char is not really there in the text.

In case someone has interest in this.
Just add a link to nvim's implementation about adding new higlight ground Whitespace for listchars
[RDY] new highlight group Whitespace for 'listchars'
nvim

:help hl-Whitespace

							*hl-Whitespace*
Whitespace	"nbsp", "space", "tab", "multispace", "lead" and "trail"
		in 'listchars'.

This sounds like a partial solution.  Soon someone wants one of the
items to be highlighted differently.  E.g. to make "trail" look more
like an error.  For a better solution a more drastic change is needed,
that also takes care of characters displayed when 'list' is not set.
The 'listchars' option clearly has outgrown it's original purpose.
--
I AM THANKFUL...
...for the mess to clean after a party because it means I have
been surrounded by friends.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


can serialize a complete graph model

go line by line through the tsv
create instances of subjects and sessions edit: we will add sessions to the graph when we model phenotypic variables at the session level
combine into a nb:Dataset instance
give the dataset a name. needs to be user specified.
serialize to json

We currently have two conflicting goals:

For the current release we plan to still store phenotypic metadata at the level of the subject
We have also decided to handle session information in the bagel pheno command

Typically session information is only available if there is more than one session. If that is the case, we would have to pick one session and apply the associated phenotypic information to the subject. Not ideal, but probably still useful. I'm leaning towards this solution.
The alternative would be that we need to update the API to expect these metadata to be attached to the session instance from now on. @alyssadai: do you have a sense of how much work that would be? I know we had said we'd tackle that in a future release.

Another note here: because the bagel pheno subcommand only expects a participants.tsv and a participants.json, we need to ask the user to provide the name of the dataset manually. If this were a BIDS directory, we could read the dataset name from the dataset_description.json file. But we can't expect this file to exist. We can also not expect the filename itself to reflect the dataset name (because if the files come from a BIDS dataset, their filenames would be generic).
So a mandatory user supplied option for dataset name seems to be the only option here. On the plus side: we can use that as a sanity check when bagel bids gets implemented to check that we are merging data for the same dataset.

@surchs in terms of the API expecting the metadata to be attached to the session instance, off the top of my head I think this would mainly entail rewriting the create_query function in utility.py, which shouldn't be too big of a challenge. That being said, would we expect phenotypic metadata (age, sex, etc.) to change from session to session? If I remember correctly our group decision to keep it at the subject level was because a reasonable assumption is "no", in which case your proposal of

pick one session and apply the associated phenotypic information to the subject

would make sense in the vast majority of cases, I think.

shouldn't be too big of a challenge

Thanks, that's what I thought as well. We'll still tackle it in a second round I'd say.

would we expect phenotypic metadata (age, sex, etc.) to change from session to session

Yes, that is indeed the reason. What reasonably changes and how often will obviously vary between variables, but age and the available assessment tools changing would be expected. We'll think it through more closely when we implement this.

Sounds good. Good point about the age + assessment tools, I forgot that with longitudinal data age is collected multiple times. 🤦♀


Implement full HMR in snowpack build --watch mode

Original Discussion: https://github.com/snowpackjs/snowpack/discussions/1459 https://github.com/snowpackjs/snowpack/pull/1008
/cc @joshnuss @francislavoie
snowpack build --watch. --hmr is working BUT it looks like it only supports basic browser-sync functionality. On change, the browser is automatically refreshed but import.meta.accept() is never called and full HMR never kicks in.  You can see why here: https://github.com/snowpackjs/snowpack/pull/1008#discussion_r484654704
The HMR server needs to understand your application to properly bundle HMR events to the correct accept() handler. In dev, we scan files as we serve them. In build, we should scan them when we build them instead.
This ticket should be resolved by copying the dev HMR management logic into build.ts. There should be lots of reuse here, and the PR should refactor so that most of this logic can be shared between dev.ts and build.ts.

is never called and full HMR never kicks in. You can see why here: #1008 (comment)

is the link correct? it points to a thumb up emoji

lol I meant to link to the line of code that the comment is referring to.

I've tried to do this while testing in react-typescript project, but I found the react-refresh plugin seems not to be work in build(production?) mode.
From this issue I saw this:

Don't forget that both Babel transform and this wrapping must only occur in development mode.

Looks like the react internal not support fast-refresh in production mode, correct me if I'm wrong.

I am seriously considering taking a stab at this work. Any advice for me, or is anyone else currently on this? I'm pretty new to snowpack - I've spent some hours attempting to convert an existing express app to snowpack specifically to use HMR, and also played around with some demo apps. So I have some familiarity with the docs, but totally new to the codebase.
I've spent an hour looking at build.ts and dev.ts.

I am seriously considering taking a stab at this work. Any advice for me, or is anyone else currently on this? I'm pretty new to snowpack - I've spent some hours attempting to convert an existing express app to snowpack specifically to use HMR, and also played around with some demo apps. So I have some familiarity with the docs, but totally new to the codebase.
I've spent an hour looking at build.ts and dev.ts.

I'd really love this features. I tried switching from webpack to snowpack but have a special use-case where I'm developing modules for a pre-existing software (in this case FoundryVTT, a virtual tabletop system).
As this software uses a closed source server and only allows me to load existing scripts from folders (instead of from an external URL) I can't use the dev-server as the software will look in my dist folder which is simply empty at the time and refuse to load anything.
I got it working without any hitches using the build --watch mode, but as HMR in this case always reloads the whole page I can't really use it to speed up my development process.

I'd really love this features. I tried switching from webpack to snowpack but have a special use-case where I'm developing modules for a pre-existing software (in this case FoundryVTT, a virtual tabletop system).
As this software uses a closed source server and only allows me to load existing scripts from folders (instead of from an external URL) I can't use the dev-server as the software will look in my dist folder which is simply empty at the time and refuse to load anything.
I got it working without any hitches using the build --watch mode, but as HMR in this case always reloads the whole page I can't really use it to speed up my development process.

@joehenry087 would obviously love your help! The basic premise is to copy the behaviors from dev.ts related to HMR and managing the scanned import graph:

whenever a JS file is built, scan & resolve its imports and call hmrEngine.setEntry(filePath, resolvedImports, ...
copy over the updateOrBubble() function from dev into build.ts. Or, even better, a place that both could share.

The other thing to figure out is how to rewrite the imports that point to the HMR-updated modules.In dev.ts you can see how we transform some imports into ${imp}?${reqUrlHmrParam}.

@joehenry087 would obviously love your help! The basic premise is to copy the behaviors from dev.ts related to HMR and managing the scanned import graph:

whenever a JS file is built, scan & resolve its imports and call hmrEngine.setEntry(filePath, resolvedImports, ...
copy over the updateOrBubble() function from dev into build.ts. Or, even better, a place that both could share.

The other thing to figure out is how to rewrite the imports that point to the HMR-updated modules.In dev.ts you can see how we transform some imports into ${imp}?${reqUrlHmrParam}.

This is definitely tricky though, give it a try but no worries if it ends up being too much

This is definitely tricky though, give it a try but no worries if it ends up being too much

any progress on this?, I also would like to help.

Spent a bunch of time looking at it and got busy with other things 😂

@rizkyramadhan I'm beginning work on this now through tomorrow. I'll be on discord JoebiWanKanobi#5940

#2707 ends up rewriting the build pipeline entirely in a way that gives us build HMR for free! I'd love your help testing the PR if anyone is able to check out the branch or run with npm install snowpack@next

Can confirm that snowpack build --watch --hmr --no-bundle is working for me.
I did notice that sometimes my server is still reading the old version of the file at the time it gets the refresh request. Not sure if this is something on my side, or if the updated files need to be fsync'ed or something before the refresh request is sent. Saving a second time picks it up, or I added a 50ms delay in my server between when it gets a request and when it reads the file in development mode, and that works too. 🤣


Optimize responsive

Hi, do you plan to optimize responsive? Jssor it's not usable on mobile devices and this is a limit. I think that scaling the slider it's not the best choice, because everything gets smaller.

For now the solution is to use big titles just like titles in video.


soc: arm: st_stm32: stm32u5: fix pm_power_state_set

Some PM cases were not handled correctly (missing default statement).
The error was caught by CI while doing other PM related work.

included as part of https://github.com/zephyrproject-rtos/zephyr/pull/40693


Retry the build-gh-pages workflow stage when previous attempt failed

Due to #2, the build-gh-pages workflow stage sometime fails (e.g. 22850)
In next scheduled workflow runs (e.g. 22851 and 22852), the build-gh-pages stage is skipped where it should run again due to the previous failure.

This is an interesting problem and I'm not sure what the best solution would be. As this workflow only triggers the GitHub Pages deploy once for each new update, it's more of an issue than other workflows, as that update is just skipped, and so reducing the fails/handling fails in that deploy job would help keep the Pacman repo more up-to-date.
Fetching the status of the previous run isn't great and I wouldn't want to add anything like that unless GitHub Actions added syntax for that directly in the workflow instead of calling the API yourself. Adding an extra step to the job that's just a copy-paste of the previous step feels like too much of a workaround and makes the workflow messier. I think my favourite approach would be adding something like a LATEST_VERSION file to the gh-pages branch and then comparing that file instead of using an if for the whole job. This would also avoid race conditions from having another workflow that periodically makes sure the main branch and the Pacman repo are in sync.
Another thing to keep in mind is the added complexity. The current solution is the more "correct" one and manages to completely avoid the deploy job as the two branches are always expected to be in sync and any issue is likely an infrastructure issue/outage (in this case random GitHub hiccups). If this was something updated much more occasionally, for example an app with ~monthly releases, the deploy job succeeding would be much more important, but with how often there are new Insiders updates, the impact of a single fail is fairly minimal. In the case of less regular deploys, I think I would prefer to manually retry the job.
This workflow isn't updated very often and fortunately "just works" as I tried to make it stable, so the added complexity of adding a check like this doesn't have much of an impact on code maintainability, but does have an impact on the workflow's efficiency (even if GitHub Actions usage on public repos is unlimited, I write my workflows as if they weren't and try to optimise them) and quality (if someone was to look at the current workflow so they can base theirs on it, I would be recommending the current solution of knowing if it should need changing, instead of adding a workaround for GitHub hiccups, which if considered would require most workflows to have a lot of added complexity and is a bit of a slippery slope for something that I consider to be a safe assumption).
I think for now I'm going to stick with manually retrying failed jobs and look at how often they occur, and if it's often enough, find a more automated solution. If you want to purely rely on updating Insiders from the Pacman repo, I would also recommend disabling VSCode's automatic updates, which would avoid seeing the in-app notification as described in #2.

This has been implemented with 6ecd85dad6ee9b4405014646aac8046ab9284956.

Great thanks !


Be dev feature isavmtests

This is a first attempt to verify the functionality of the Virtual Memory behavior. The riscv vm tests have been modified to consider the things that have not been implemented in our hardware yet (like S mode csr emulations and VM fence instructions).
Also for simulation, I'm relying on self-verification of the isa programs instead of trace replay because generating traces that will match with the actual hardware is going to be difficult if we take into account that some parts of the hardware are going to be emulated.

@dpetrisko @zaazad @spikemar
There are some changes in the FE here that have been made in order to pas the isa-v tests(mostly in pcgen). I think we should consider these changes when we are doing our rework of those FE modules.

Looks fine to me.  Small thing, but change RV64_TESTS and RV64_V_TESTS to be RV64_P_TESTS and RV64_V_TESTS

Yep, you’re right. I realized that right after commenting
Sent from my iPhone

On May 6, 2019, at 8:50 PM, Farzam Gilani<EMAIL_ADDRESS>wrote:
@farzamgl commented on this pull request.
In bp_be/src/v/bp_be_checker/bp_be_director.v:

@@ -222,7 +223,7 @@ bsg_dff_reset_en
redirect_pending_reg
(.clk_i(clk_i)
,.reset_i(reset_i)

,.en_i(calc_status.ex1_instr_v | trap_v_i)
There was a small change to the FE queue's dequeue signal which made this bug visible.
Before that change, we actually had extra elements in the queue.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Issue-2555: add placeholders

[x] Breaking change? (if so, please describe the impact and migration path for existing application instances) No

What changes did you make? Make placeholders visible in inputs on Topic Creation page when they are empty
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

[ ] No need to
[x] Manually (please, describe, if necessary)
[ ] Unit checks
[ ] Integration checks
[ ] Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[x] My changes generate no new warnings (e.g. Sonar is happy)
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged

Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)

@Haarolean I've checked react-hook-form api and didn't found fast way to implement the change for the following screen

maybe you can suggest the way how to achieve it or propose different change?

@Haarolean I've checked react-hook-form api and didn't found fast way to implement the change for the following screen  maybe you can suggest the way how to achieve it or propose different change?

@provectus/kafka-frontend please help

@Haarolean I've checked react-hook-form api and didn't found fast way to implement the change for the following screen  maybe you can suggest the way how to achieve it or propose different change?

@provectus/kafka-frontend please help

@vitalymatyushik
https://stackoverflow.com/questions/43964528/append-uneditable-text-to-end-of-input .
I think we have to think of a more css like solution , we can create a component for this or even do a custom code for this one where you can actually set the (7 days) etc in the input  or div attribute and with a pseudo element , and we can get the that attribute value with css and put it with pseudo element as mentioned above.

@vitalymatyushik and this one?


New SEM Page - Why Shred-it Missing CTA

Why Shred-it section is missing the CTA Button. This applies to all new SEM pages that use this template.
Example: https://main--shredit--stericycle.aem.live/en-us/marketing/bulk-paper-shredding

Fixed

@jindaliiita looks to be fixed on most. Still appears to be missing on - https://main--shredit--stericycle.aem.live/en-us/marketing/shred-it-pricing

@colinvlasak Can you do a hard refresh for this page. Because it's working fine for me.

@jindaliiita must have been a caching issue. I see it now. Apologies for the back and forth on this.


Latest improvements to APIM 2.5.0 under update 2.0

Describe your problem(s)
This issue was created to track the following git issues.

https://github.com/wso2/product-apim/issues/9489
https://github.com/wso2/product-apim/issues/9399
https://github.com/wso2/product-apim/issues/9468

Closing the issue since the updates are created.


How to uninstall Warden?

I was doing some tests, but I want to uninstall it completely from my machine for now.
How could I uninstall Warden completely?

@rafaelstz Are you on Linux or Mac OS? The basics will be the same, but there are a couple slight differences relating to DNS.
You'll want to run warden env down -v in each project you had up and then warden svc down -v to cleanup containers, associated volumes, etc for everything Warden ran for projects and for global services. Once this is done, there are files in ~/.warden which may be cleaned up (this is where Warden stashes generated certificates and misc errata relating to global services such as Traefik).
On Mac OS, there is a file at /etc/resolver/test which is responsible for causing *.test DNS queries to route to the dnsmasq container. On Linux it's a little more complicated. Some systems will be auto-configured per the logic here (https://github.com/davidalger/warden/blob/develop/commands/install.cmd#L58-L87) but some it simply fails (I'm actually planning to remove this and direct Linux users to a docs page on how to configure their DNS resolver since it's near impossible to cover all Linux configurations, and the references to systemd throw warnings on WSL2 since it's not available).
On Mac OS, there you'll find a Warden issued root CA has been added to the Keychain in order to have trusted SSL certificates for *.test sites. It can be removed without causing any harm. On Linux this same root CA will have been added to either /etc/pki/ca-trust/source/anchors or /usr/local/share/ca-certificates depending on the flavor of Linux you're running.
Lastly there is (an easily identified) block of configuration added to /etc/ssh/ssh_config for the connections to tunnel.warden.test which can be removed.
This is all written up off-hand, so I can't promise I won't have missed anything, but I believe this covers everything. My goal here has been to leave as few "artifacts" outside of the Docker containers as possible.


feat: add dnsmasq role to provide dhcp service

This commit adds support for a new kubeinit_dnsmasq role which will
provide dhcp service for the cluster provisioning network.  Other
changes include:

Added environment vars to set OCP params
Moved services containers startup to common path
Updated flow of coreos downloads into okd paths
Updated OKD version to the latest 4.7 release

I dont see anything wrong with this PR LGTM, we should merge and I can iterate over to investigate the RKE issue that might be related to the CI


updating jsons, web submodule and disabling social_handler to broker.…

updating jsons, web submodule and disabling social_handler to broker.pikabot.org

oh, I've found a bug in pokemon.json. fixing.

fixed json and tests. build passed. Awaiting approval and merge.

Closes #5934

👍


[Feature] Use standardized Rust crate defintions for better interoparability with other Rust crates

Is there an existing issue for this?

[X] I have searched the existing issues

Description
For example, instead of using custom http types, you could use the types defined in the http crate and use traits defined in num_traits. This would massively reduce the amount of glue code required for most integrations

This is being worked on in PRs like this:
https://github.com/cloudflare/workers-rs/pull/286


Delete task not working

Delete task not updating the cache.
 deleteTask(id: ID!): ID
should be replaced by
 deleteTask(id: ID!): Task
then fix deleteTask resolver

💯 Good catch! I think our cache helpers are configured to work with both cases, but we might just have bug somewhere in the app.

This bug should be fixed in https://github.com/aerogear/offix/pull/48, specifically: https://github.com/aerogear/offix/pull/48/files#diff-54f1fc59153563ec58bc4ff6964945f1.
We haven't done full releases yet. Once we do and all dependencies are updated this should go away. There will also be some small updates to the app.

Fixed


added max-of-n

I'm going to merge this now so we can build on it, code review can be fixed in another PR


fixed three bug

Does not support non-english characters;
Optimize indent (e.g. output error when  there is no h1['#'] );
Active text editor has not update when text editor switch

Hello @jruif,
great PR, thanks. I can't merge and test your improvements because previous PRs cause conflicts. Can you solve the conflicts please?
Happy coding,
Darius


[Spark] Set up Python Protobuf codegen for Delta Connect

Which Delta project/connector is this regarding?

[X] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[ ] Other (fill in here)

Description
This is the first PR for Delta Connect, which adds support for Delta's DeltaTable interface to Spark Connect. This is needed to support Delta table operations outside of SQL directly on Spark Connect clients.
This PR sets up the Python code generation for the Protobufs of Delta Connect. For this I created a new Buf workspace and I added a few initial Protobuf messages to confirm that everything works.

How was this patch tested?
Manually ran dev/connect-gen-protos.sh to confirm that it correctly generates the Python code for the Protobufs.

Does this PR introduce any user-facing changes?

No.

Failed tests are in org.apache.spark.sql.delta.perf.OptimizeGeneratedColumnSuite, seems unrelated to this PR.


bulk ebook import

Multiple EPUB import (Chrome app ebook library), with error handling and cancellable sequence. Works with drag-and-drop as well as the file picker dialog.

CC @brettins

Note that the per-EPUB progress bar is now a proper modal dialog (Twitter Bootstrap), whereas before it could be discarded by clicking outside the dialog (even though the WebWorker / background thread was continuing to import the ebook into the library).
Also note that there is a cancel button to stop the sequence of imported EPUBs, but a currently-importing EPUB still cannot be interrupted (i.e. no functional / behaviour change compared to code in the develop branch).

Another note: the selected list of files to import into the ebooks library may contain non-EPUB files (e.g. drag-and-drop of a bunch of random files onto the Chrome apps' window, or erroneous user selection from the system file picker). This is handled appropriately by this PR implementation: the culprit non-EPUB files are ignored.
Equally, there's graceful degradation if an error occurs during the loading of a given EPUB: the ebook is skipped, and the import queue moves to the next available EPUB file.

@danielweck I have tested the feature branch on both OSX and Windows 10 (under Parallels).  Tested with a variety of files, both valid and invalid, large and small.  Looks good!  Will merge.


Improve process streams

This makes a number of pretty significant and breaking changes to process streams, in the hopes of making them more useful, more consistent and easier to understand.
While we have found a pretty intuitive API for working with the processes themselves, their stdout and stderr streams have been a bit neglected. We are already foreseeing some things that we want to do in bigtest that are difficult to do with the current API.
First of all we create a solid foundation by making the std streams over Buffers. This is what node exposes by default, and since we would pay a performance penalty in cases where we unncessarily convert to strings, we should just expose this.
However, working with raw buffers is tedious, so we expose two smart mapping functions on iostreams text() and lines(). text() just converts the buffers into strings. lines is a bit fancier and creates a new stream which emits once for each printed line (waiting until the line is complete).
Additionally we improve the buffer function on Stream by making it return a resource, and also taking an optional limit, so we can create a bounded buffer using a RingBuffer behind the scenes.
In bigtest we have previously wanted to cache the previous X number of lines. With this change, something like the following becomes possible, which was previously pretty challenging:
let app = yield exec(options.appCommand);
let lines = yield app.stdout.lines.buffer(1000);

yield lines.filter((l) => l.includes('ready')).expect();

I have run into a use case a handful of times now where it would be great to see the output from an exec() with the stdout and stderr laced together as you might see in a terminal. You can accomplish that "manually", but it would be convenient to add something to join()/expect() to allow one to easily do this.
This is sort of functionally equivalent to stdio: 'inherit' which is quite popular in child_process usage, but we don't really want to let the user muck with stdio as it becomes a rabbit hole and 'inherit' blitzes any ability to inspect the output.

What parameters could we add to join/expect?

Another option would be to extend the Exec and Daemon builder apis:
yield exec(`npm install`).stdio('inherit').expect();

An alternate design here is to add the text() and lines() methods to all streams. But I'm not sure how I feel about this. It would remove some complexity, but doing something like someNumberStream.lines() is obviously pretty weird.

There is some danger to claiming that our implementation works the same as using inherit, because we're emulating its behaviour, but we can't actually guarantee it works exactly the same. I think flushing and the like could actually present issues where we behave slightly differently.
Like @jbolda noted, if we actually pass down inherit, we lose the ability to read data from the stdout and stderr streams, which is inconvenient. It's pretty easy now to emulate this yourself, using:
yield spawn(myProcess.stdout.forEach((c) => process.stdout.write(c)));

But if we add something like this, then IMO we shouldn't call this inherit.
I think a good solution might be to actually support inherit for where it is useful. In that case the stdout and stderr streams would be empty. If someone wants both the ability to read the streams and have them be printed to the console, then implementing that yourself is pretty easy.

But if we add something like this, then IMO we shouldn't call this inherit. And I think it's challenging to find an API here which is intuitive and useful.

Yeah, I was hesitant about calling it inherit because that isn't what it is. However, I've very rarely not had to have this line
yield spawn(myProcess.stdout.forEach((c) => process.stdout.write(c)));

While it's easy, it's also tedious, especially when it's the 90 in the 90/10
I think it is an annoyance in our API, but I do prefer to kick the can down the road rather than committing to an alternative that turns out to not be good.

An alternate design here is to add the text() and lines() methods to all streams. But I'm not sure how I feel about this. It would remove some complexity, but doing something like someNumberStream.lines() is obviously pretty weird.

We could have a split() method that split on a separator which would convert any stream of T into a stream of T[] and we be applicable to all streams.

We could have a split() method that split on a separator which would convert any stream of T into a stream of T[] and we be applicable to all streams.

Note that I still think there should be an IOStream with text() and lines() even if we add a split() method. It's just too handy, and there's no sense it trying to squeeze it into the super-interface. Maybe TextStream is a better name though since IO implies that it's read/write and I can imagine this being very useful for any stream unrelated to stderr and stdout ?

Much nicer API 👍🏻
I'm assuming the changes we talked about managing the process streams via stdio options will be in a follow-on?

@cowboyd I think this is ready for another look. I will post a follow up PR for the changes we worked on yesterday to improve stdio handling, since it really is a separate feature.


Error when inputting live weather location

Whenever I try to put in my location for the live weather function of this extension, I can an error message that simply reads "an unknown error has occurred."

Have you tried entering any other location? Are you able to provide the details of what you're entering that causes the error?

I had the same problem, when I try a zip code in US works... but when I try a zip code in CL (Chile), it shows "an unknown error has occurred."

I'm the Cumbria in the UK and any postcodes I try around my area throw up an error message

Failing for Argentina (1426, AR) as well. If this is going to work for the US only, it should probably be mentioned.


Provide a way to suppress some warnings

From #103: it could be annoying to deal with e.g. missing inventory warnings when a package and its documentation is out of your reach. But having warnings is still beneficial. I can see a couple of options that we could consider:

Provide a directive for suppressing warnings in a specific code example: this would mask other warnings in that code example though.
Provide configuration to disable extra warnings on intersphinx packages: this will surely mask lots of other warnings as well, and doesn't address the fact that you might not want to define a ref for each local module that can be imported.

Any other options? Personally I don't much care for either, but if I had to pick one it would be suppressing specific examples. At least that way people know that they might have problems and have more manual control over it.

A list of modules or prefixes for inventory items could work. So specifying library.foo would disable inventory warnings for either just that entry or alternatively everything under it.

Still not sure if it's worth to do this, but if we do, we might as well let the items be regexes so that people can disable exactly what they want and the logic on our end is simple.


Make soc_reset_last_reached persistent

Is your feature request related to a problem? Please describe.
It would be appreciated if soc_reset_last_reached could be saved to survive a restart of the GX device or a driver restart, as commented https://github.com/Louisvdw/dbus-serialbattery/blob/dev/etc/dbus-serialbattery/battery.py#L496
At the moment, this operation is performed way too many times.
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

What do you mean exactly with?

At the moment, this operation is performed way too many times.

This is already implemented. I only forgot to remove the comment.
https://github.com/Louisvdw/dbus-serialbattery/blob/9779b2e1773aa668afca9d1f647636c773f11bcd/etc/dbus-serialbattery/dbushelper.py#L1147-L1166


certain 'plot methods' yield "Error: Not a graph object"

including random, circle, and sphere.
maybe more.

Maybe fixed now.
On Sun, Jun 7, 2015 at 11:27 AM, joshua vogelstein<EMAIL_ADDRESS>
wrote:

Assigned #1 https://github.com/openconnectome/FlashR/issues/1 to
@dmarchette https://github.com/dmarchette.
—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/FlashR/issues/1#event-324686069.

yup!
On Sun, Jun 7, 2015 at 1:41 PM, David Marchette<EMAIL_ADDRESS>wrote:

Maybe fixed now.
On Sun, Jun 7, 2015 at 11:27 AM, joshua vogelstein <
<EMAIL_ADDRESS>
wrote:

Assigned #1 https://github.com/openconnectome/FlashR/issues/1 to
@dmarchette https://github.com/dmarchette.
—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/FlashR/issues/1#event-324686069.

—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/FlashR/issues/1#issuecomment-109781244
.

--
the glass is all full:  half water, half air.
openconnecto.me, jovo.me, office hours
https://www.google.com/calendar/embed?src=e2ktu4lrgul8anp8hclrcminp8%40group.calendar.google.com&ctz=America/New_York


feat(Combinatorics/SimpleGraph): add definitions for Hamiltonian paths and cycles

Co-authored-by: Bhavik Mehta, Linus Sommer

@linus-md We definitely don't want the Dirac file to be in the branch!

@b-mehta I See, I was wondering what happened.

I just pushed a few sorries for you to prove. Do you mind trying them?

Yeah, I cleaned up already once, so the sorries are the most blocking part. Please also merge master and ensure CI passes.
This PR was written during the Bonn workshop last September and I believe the original authors have gone to do other things, so feel free to take over!

It seems I don't have push access to this branch. Could that be right? Could I get the access rights?

It would be good to coordinate with @linus-md, @rishicomplex, and @b-mehta to be sure that they would appreciate this PR being dusted off and continued (I assume they would, but it's more polite to check rather than assume!)
For getting push access @LodeVermeulen, please go to Zulip and request write access.

I would be very happy if this gets finished! @LodeVermeulen @kmill

You seem to have missed https://github.com/leanprover-community/mathlib4/pull/7102#discussion_r1630267822


charts/librechat/templates: add an option to override deployment update strategy

Ability to overwrite update strategy is useful when using persistence for images.
Setting strategy.type: Recreate is needed in this case in order to recreate pod with the same PVC attached if PVC type is set to ReadWriteOnce.
Default value corresponds to default option for deployments, so this does not change current behaviour of the chart.

thx for the PR :)


Active payments not showing on desktop

As the title says, active payment is not showing on desktop.
The little green counter does show up but it shows 0
Safe is here: sep:0x5ecB0Cb2002Db1EF86D92DcA9963839982cDEe15
I created Safe, role, and added payment on mobile (iOS, Safari, Coinbase Wallet)
Desktop is mac OS, Arc browser

Update: same behavior on a newly created Safe with newly created roles:
Trice Design DAO
0x11b9F3f68154C8E55A6b42a3f709807badA4E1Ef
Desktop shows 0 active payments while mobile shows accurately 1 payment for each role.
*Desktop did show active payments briefly while I was adding payment to 2nd role but reverted to showing 0


it couldn't work on javascript-rendered html dom.

it couldn't work on javascript-rendered html dom.
I write some code to test this problem, in here
in addition， I also found that if js render html on load page, the bootstrap-validator could work. But if I generate html after loading page , like through clicking button, it couldn't work.
Is it a bug?
if not, how can I deal with it.

You have to call .validator('update') anytime you alter the set of form fields.


Standardise files with files in sous-chefs/repo-management

Description
Describe what this change achieves
Issues Resolved
List any existing issues this PR resolves
Check List

[ ] A summary of changes made is included in the CHANGELOG under ## Unreleased
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable.

Released as: 3.2.2


modularize rails-admin

we want to use rails-admin, but it is too much of a black box and very monolithic. This makes it only usable for small, standard projects.
The most important way to tab into it would be to be able to overwrite the views and use the nice form helpers but customize the rest. So i stable api for the nice form helpers would help here plus the ability to overwrite views per resource and action.
A next step would be to overwrite controller actions, add new controller actions .. etc.
what do you think? Is this planned, i guess it is hard to got for a stable form api at this early stage.

I support this request.
We now have custom forms (many thanks!), but I don't know how to code a custom form for it.
Good form helpers for custom forms would be very helpful.


[stable/hazelcast] Adding support for pod affinity

Signed-off-by: Ryan Orth<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Allows hazelcast pods to run on the same/different nodes based on affinity settings
Which issue this PR fixes
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@leszko or @mesutcelik - could one of you take a look at this pull request when you get the chance? Thanks!

/assign @leszko

@leszko - Thanks for looking at my pull request. I have added an example to the values.yaml file. If anything else is needed, please let me know!

Thanks @ryaneorth for the PR.
What happens if number of nodes become equal to number of hazelcast pods? New pods are not just placed by scheduler or they are scheduled on a node despite of that antiaffinity config?
We are planning to implement hostname based hazelcast partition grouping on kubernetes. When that is done, pods can simply share the same node as their backups will be located in another nodes.

Hi @mesutcelik - in the example I have in the values.yaml where it uses requiredDuringSchedulingIgnoredDuringExecution if the number of k8s nodes is less than the number of hazelcast pods desired then you are correct - the pods will not be scheduled if they cannot be run on different k8s nodes. However, you can also use preferredDuringSchedulingIgnoredDuringExecution to instead "prefer" that the hazelcast pods run on different nodes, but not make it a hard requirement for pod scheduling purposes.
For more details, see: https://kubernetes.io/docs/concepts/configuration/assign-pod-node/
I hope this answers your question. If you have more questions/concerns, please let me know!

/approve

@ryaneorth Thanks for all the changes. Looks good from my end.
@mesutcelik , are you fine to merge it?

👍

/lgtm

/approve


Better CLI experience

It's yet another fabulous project made by @MrWillCom. Thank you a lot for having this idea, and creating such a handy tool. Notwithstanding, rewriting the main.js file may seem too superfluous. So I wish the function of running this program with a couple of parameters in the command line should be implemented ASAP. This will add to its convenience as a tool. Moreover, I hope a GUI would be added, though there would be arduous work to do. I may help if you decide to add such features.

And what about downloading multiple editions/subjects simultaneously? A simple but really time-saving feature to be added!


Bug with double doors in 1.20.1

Hi, not sure if this is where to post this issue, but I came across a bug where all doors from this mod won't open when they are connected double doors. Single doors open fine, as do double doors that face the same direction, but when attempting to open two doors the sound of opening occurs, but they are not opened nor can you go through. I'm using the 1.5.7 build.
It isn't game breaking and doesn't cause crashes, but if I had to guess it might be a compatibility issue with quark's double door opening (or maybe mods that add opening both doubles doors in general) because when quark was removed or the double door feature was disabled the doors would open fine.
Love the mod by the way, one of my favorites! Let me know if this isn't where to post and if I need to post elsewhere.

Hello, and thank you for the report !
Indeed, it seems to be incompabitily with Quark's code, I will need to check their code.
If you find anything else, do not hesitate to report it :)


TabBar ignores Theme's iconTheme.size

Steps to reproduce
Open dartpad.dev with the latest version of Dart & Flutter. For me, that was:

Dart 3.5.3
Flutter 3.24.3

Paste my code sample from below.
Expected results
The icon gets its size from the theme in the TabBar and in the TabBarView.
Actual results

For TabBarView, the icon gets its size from the theme, as expected.
For TabBar, the icon ignores the theme. (Bad.)

Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(
        iconTheme: IconThemeData(size: 64),
      ),
      home: const Scaffold(
        body: DefaultTabController(
          length: 2,
          child: Column(
            children: [
              TabBar(
                tabs: [
                  // The icon theme is ignored for both:
                  Icon(Icons.backpack_outlined),
                  Tab(icon: Icon(Icons.map_outlined)),
                ],
              ),
              Expanded(
                child: TabBarView(
                  children: [
                    // The icon theme is respected for both:
                    Icon(Icons.backpack_outlined),
                    Icon(Icons.map_outlined),
                  ],
                ),
              ),
            ],
          ),
        ),
      ),
    );
  }
}

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.24.1, on Microsoft Windows [Version 10.0.19045.4894], locale en-US)
    • Flutter version 3.24.1 on channel stable at C:\Users\Vinnie\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5874a72aa4 (5 weeks ago), 2024-08-20 16:46:00 -0500
    • Engine revision c9b9d5780d
    • Dart version 3.5.1
    • DevTools version 2.37.2

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at C:\Users\Vinnie\AppData\Local\Android\sdk
    • Platform android-35, build-tools 35.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.11+0--11852314)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.9.1)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Dart plugin version 242.22855.32

[√] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 15 (API 35) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.19045.4894]
    • Chrome (web)                 • chrome        • web-javascript • Google Chrome 128.0.6613.138
    • Edge (web)                   • edge          • web-javascript • Microsoft Edge 128.0.2739.42

[√] Network resources
    • All expected network resources are available.

• No issues found!

Can you share the screenshots?

TabBar currently overrides the IconTheme size and color.
https://github.com/flutter/flutter/blob/4ca51a1de59772d6adc6f8fbcc627c79d975a564/packages/flutter/lib/src/material/tabs.dart#L313-L319
Since IconThemeData cannot be nullable when calling IconTheme.of(context) . Allowing inherited IconTheme isn't straight forward.
I tried to find other classes running into similar situation when calling IconTheme.of(context) and found a workaround in search_anchor.dart in  by @nate-thegrate.
https://github.com/flutter/flutter/blob/4ca51a1de59772d6adc6f8fbcc627c79d975a564/packages/flutter/lib/src/material/search_anchor.dart#L1444-L1452
I'm not sure if this workaround is worth copying in other classes.
cc: @QuncCccccc, as you worked on Icon related stuff.

@QuncCccccc
Could we make IconThemeData nullable when returned by IconTheme.of(context)  without breaking change?

Current behavior reported is as rendered below:

Stable : 3.24.3
Master: 3.26.0-1.0.pre.173

@QuncCccccc Could we make IconThemeData nullable when returned by IconTheme.of(context) without breaking change?

Probably no. Changing it to be nullable doesn't follow the convention that of method always return non-null values, and it may cause a lot of breaking changes. Maybe we can create a maybeOf() method for IconTheme and it only return a IconThemeData getting by context.dependOnInheritedWidgetOfExactType<IconTheme>(). So in TabBar, we can call this method to get the ambient icon theme and icon size if there's any. WDYT:)?

We cannot check for default colors inside IconTheme.maybeOf as it will require importing Material library inside widgets/icon_theme.dart, interesting problem.

Can we check it in TabBar?

We cannot check for default colors inside IconTheme.maybeOf as it will require importing Material library inside widgets/icon_theme.dart, interesting problem.

Can we check it in TabBar?

Currently we're doing it in the same search anchor. I'm just wondering if there is a better way that can be applied in multiple classes.