feat: reuseable workflow

Fixes https://github.com/xline-kv/Xline/issues/700
Please briefly answer these questions:

what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)
Added reuseable workflow and implement in benchmark.yml and validation.yml

what changes does this pull request make?

are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)

@Phoenix500526 @liangyuanpeng Please review and suggest for any changes and addition/fix in the script.

@Phoenix500526 Just pushed, Please check

This PR resolved this issue - https://github.com/xline-kv/Xline/issues/700

Update to Fixes #700, and then it will auto close the issue when this PR is merged.

requested a review from liangyuanpeng

LGTM and thanks, (I don`t have owners here.)

LGTM! Thanks for your work :)

Got it. Thanks! @bsbds Please Review.

@ChaudharyRaman
I think we still update here, do you want to continue it?
https://github.com/xline-kv/Xline/blob/eadf2d9a1473fa33e2c5017a5f6f1682742fa243/.github/workflows/pull_request.yml#L128-L151

@liangyuanpeng Yeah right, I would do that. Currently, I am working on GSOC proposal with dead-line so close, Will create a PR by tomorrow. Thank You.

@liangyuanpeng, Previously while implementing what i thought is the for actions/upload-artifact@v4 , we are using for upload Log and Benchmark, and since Upload logs depends on failure() || cancelled(). Its better to let it be there and so instead of creating new Upload, lets create a general Case for rest including Benchmark and _output/xline.tar. What do you say.


Style/Layout Revisions

@brianek Please review and merge this pull request. Thank you!

@DavidGinzberg A little help here on merging.

@ParentJA don't delete your branch just yet.

Summary of PR from @ParentJA: @davidginzberg: hey guys, I reorganized some of the JS to handle some issues I saw: took variables off the global scope and encapsulated them into an Angular service, named functions for easy debugging, removed unused Angular controllers, implemented ui-router which is recognized as a big improvement over the native Angular ngRoute
parentja [13:44]
@davidginzberg: I reorganized some of the files: removed unused JS libraries, created folders to hold images and data files respectively
parentja [13:45]
@davidginzberg: the major changes included removing the Angular Material library and incorporating Bootstrap components and CSS into every part of the application

@DavidGinzberg i'm confused by the last post.

@brianek are you in the Slack channel right now? My comment is just a snippet from that conversation that gives a little more detail on what was changed and why.

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! THANK YOU !!!!!!!!!!!!!!!!!!!!!!! @DavidGinzberg


RxJs 6 merged.subscribe is not a function

Documentation Related To Component:
merge
Please check those that apply

[ ] typo
[ ] documentation doesn't exist
[ x] documentation needs clarification
[ x] error(s) in example
[ ] needs example

Description Of The Issue
Hi 👋
I took an example from the docs: https://rxjs-dev.firebaseapp.com/api/index/function/merge#examples:
const {interval, operators: {take, merge}} = rxjs;
const timer1 = interval(1000).pipe(take(10));
const timer2 = interval(2000).pipe(take(6));
const timer3 = interval(500).pipe(take(10));
const concurrent = 2; // the argument
const merged = merge(timer1, timer2, timer3, concurrent);
merged.subscribe(x => console.log(x));

created a fiddle https://jsfiddle.net/evgeny_t/7je14aoq/9/, added https://cdnjs.cloudflare.com/ajax/libs/rxjs/6.3.2/rxjs.umd.min.js, but I get Uncaught TypeError: merged.subscribe is not a function in the console. Is there some discrepancy between the docs and the code? Am I doing it wrong?
Thanks!

So there's merge for creation, and one for operators.
merge(...) is being used to creation method, one in operators/merge won't work in those cases.
const {interval, merge, operators: {take}} = rxjs; //import creation method `merge` here

...same snippets below

for operators, you can use it for pipe operator like someobservable.pipe(merge(...));.


CI: Pin cc for MSRV build

The cc dependency just stopped working for MSRV build with error
error: package `cc v1.0.96` cannot be built because it requires
rustc 1.63 or newer, while the currently active rustc version is
1.56.1

Pin cc back to an earlier version for MSRV build.

Thanks for being on top of this!
And the new cargo error messages are nice. It's clear that it's a deliberate decision by upstream and it's clear what the issue is.


how to remove a local  jar form a library

Expecte Behavior
remove a jar form a library
myProject
compile cn.bmob.android:bmob-sdk:3.5.3 library(.aar I can't modify The source code)
----libs
---------okhttp-3.3.0.jar(it's a jar)
i watch the projtct depdences by gradle command
gradle dependencies

i use this code but  invalid, okhttp.jar still in bmob Library
compile ('cn.bmob.android:bmob-sdk:3.5.3',{
        exclude module: 'okhttp'
    })

please tell me how to exclude this local jar,thank you very much

Is there a method for inspecting/excluding modules from .jar dependencies not available on Maven? gradle dependencies seems to not report dependencies added with compile files('libs/example.jar')

The GitHub issues are reserved for feature request and bug. Please use the user forum for any question related to Gradle usage. In the event your question does turn out to be a valid GitHub issue, please follow the contribution guidelines.


feat: split_n on write pass

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#183
#170
#182  👈
#181
#180
#176
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rphmeier and the rest of your teammates on  Graphite

Merge activity

May 15, 8:21 AM EDT: @pepyakin started a stack merge that includes this pull request via Graphite.


PCMS 1.0.22 "Something went wrong installing sharp" when using this plugin

Hey there, first of great idea, I was just about to to do something like this myself!
Unfortunately with the latest version of Payload CMS I (1.0.22 at writing time) I am unable to use it since there seems to be a conflict with sharp. Please advise or update if possible :)

@Etheliene version 1.0.3 of the plugin appears to install just fine with Yarn and Payload 1.0.22, what exactly is the error you're experiencing?

@Etheliene hey, just checking whether this is still an issue for you, please let me know so that I know whether there's still work to do here :^)

Hey, thank you for checking in. I do not seem to have that issue anymore  - at the moment -


Code action?

Hi there! Thanks for this very promising project, I love anything that gets me out of an IDE and back into Emacs! Your critique of eclim is spot on; it is both slow and finicky about project setup. The biggest differentiator to me at least is the ability to fix issues automatically. In eclim it is clunky and takes a crazy amount of time to apply the change, but it's still faster than checking the flycheck error and manually applying a fix. I would be happy to take a shot at implementing the functionality but all of this lsp stuff is so new to me I'm not sure how to begin. Any pointers? Thanks again!

Hello. The two main actions applicable to code documents would be code refactorings and problem quickfixes. In LSP both can be represented using the CodeAction system. By the specification, the user can call codeAction on a region and get back an LSP CodeAction list, which contains a specific command to execute for that action. The CodeAction types could be things like Extract function/Organize imports for globally available refactors, or Import missing package/Create missing class for diagnostic fixes.
Refactorings
Some actions would be globally available regardless of which errors/warnings are in the file. The process for gathering the list of those available code actions would be something like:

User invokes the codeAction request for a region.
In the server, the availability for each supported code action is checked, and the ones available for the region are returned in the response as CodeAction[]. Each code action would return an LSP command identifier for use with the executeCommand request.
On the client, when one of the received code action commands is executed, start the dialog in IntelliJ for setting the options of the refactoring.
Make the command result return a list of document edits. (The return value of an LSP command can be any value, so a convention would be needed.)
On the client, make a request to apply the command's edits on the server, if any were returned.

The second part could be accomplished using part of the RefactoringActionHandler in the IntelliJ SDK. It could be used for determining which code actions to add and how to calculate the result of the corresponding LSP command associated with the action.
Quickfixes
The quickfix menu in IntelliJ seems to use a separate mechanism for applying fixes. A single problem can have multiple quickfixes, and the code actions available for a region depend on specific instances of diagnostics being available in that region. If a code action for one of the diagnostics is selected, the server would have to correlate which IntelliJ problem it is associated with, then return actions associated with the quickfixes for that problem. Also, the problems would have to be retrieved later if the quickfix code action is selected, so a caching or synchronization mechanism might be needed.
There is support for the codeAction request to add a list of diagnostics in a CodeActionContext object, so that is how the list of problems would be sent to the server for getting quickfix actions.
Here is where IntelliJ's quickfixes seem to be held. You would be able to call applyFix by providing the instance of the quickfix information that was generated by a diagnostics pass. To obtain the edits needed, something like differenceFromAction could be run to apply the quickfix inside a copy of the working document, then using the difference as the workspace edits to send to the client. It seems a specific diagnostic can hold multiple quickfixes at once, and may not be constrained to a single working document (which differenceFromAction expects), but that's the general idea.

There are a lot of subtleties and synchronization that would be needed to interface with IntelliJ's mechanisms for both refactoring and the quickfix feature, but it's probably doable with enough time. Also, it may be necessary to add support for certain features to clients nto completely compatible with the spec. I added some amount of extra code to the Emacs client for running certain commands and showing executable tests or main entry points.
Hopefully this is somewhat informative. Although I'm sort of occupied by other projects at the moment, if something isn't clear, please ask.


Routes with different variables on the same level do not work

Hi,
I noticed that when adding routes with dynamic and static fields on the same level the trie get's confused:
	router = Lennarb::RouteNode.new
	router.add_route(['foo', ':foo'], 'GET', proc { 'foo' })
	router.add_route(%w[foo special], 'GET', proc { 'special' })
	router.add_route(['foo', ':id', 'bar'], 'GET', proc { 'bar' })

	_, params = router.match_route(%w[foo 23], 'GET')
	assert_equal({ foo: '23' }, params) # Fails because params are {:id=>"23"} instead of {:foo=>"23"}

I have added a PR to illustrate this.
This is the same problem hanami-router has.
Thanks for publishing this gem. I think it's super nice.

Hi @ahx! Thank you for reporting this issue. I really appreciate your feedback and the PR you submitted to illustrate the problem. I'll take a look and try to fix the trie behavior. Any suggestions? 😄

Hi. I did not find a way to solve this so far. It’s not an easily solved problem. Others have struggled with this before and failed.

Hi @ahx,
I'm not entirely sure, but this https://github.com/aristotelesbr/lennarb/pull/54 might address the issue you reported here.
Let me know if you get a chance to test it! 🙃

Hey @aristotelesbr. Thanks for looking into that. The test is green, so I think you solved the issue. Nice work!

Something not related to this issue: The reason I am interested in all this is, because I am looking for a small, fast routing library to use in openapi_first. I wish we would have a de-facto default library/gem to use in Ruby where people can build their own micro/midi/macro framework on top off. Lennarb's route_node.rb looks like a good candidate for that.


Fix spelling of npm (and pnpm)

I'm part of the engineering team at npm and wanted to quickly contribute this to use the correct spelling of npm which is always all lowercase, compare also https://github.com/npm/cli#is-it-npm-or-npm-or-npm

I'm part of the engineering team at npm and wanted to quickly contribute this to use the correct spelling of npm which is always all lowercase, compare also https://github.com/npm/cli#is-it-npm-or-npm-or-npm

@lumaxis thanks for contributing this PR.
Our documentation standard is to spell words using conventional English capitalization.  We understand that some brands like to call attention to themselves with stylized lettering such as "μTorrent" or "Toys "Я" Us", however participating in a marketing effort is isn't our priority. We aspire to provide professionally written doc content, for example where every sentence can begin with a capital letter. 😉


wip: Crash in static init method locs

Motivation

The more test cases I find like this, the more I wonder if we should
have something akin to our lsp_test_runner fast path tests, but instead
of adding one \n for each character in the file and running the fast
path, start the file with that many \n characters and run a fast path
where we delete them all. (Unfortunately, I think this means running
another full slow path edit, and then another fast path edit after, for
each test, which could substantially slow down our tests. Maybe it's the
kind of thing that we do just once to find the problems, commit
individual tests where relevant, and then don't commit any changes to
the test harness? We could at least build it and see how much slower
things get.)
Test plan

See included automated tests.

Closing in favor of #7232


Apache Hive support #142

Resolves #142
All the tests in HiveMigrationTestCase pass.
The default HiveDriver throws UnsupportedOperationException all over the place.
I had to wrap the Driver and override some methods.
I cannot seem to run the HiveMigrationTestCase with maven without specifying the test explicitly with -Dtest
Could someone please review the pom.xml and let me know what command to run.

Hi there,
I really need this feature.
I will fix the conflicts and update the PR. What is the chance this will get merged?

@wilderrodrigues For this to be merged:

all conflicts must be resolved
all contributors must have signed the CLA
all versions of Hive this has been tested against must be specified
all Docker images representing those hive versions must be listed along with the exact settings used to start them to be able to test this

 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.

Thank you for your submission. We cannot merge this at this time as it would need re-implementing in the https://github.com/flyway/flyway-community-db-support repository.


Basic dynamic keys handling with -d param (fix #46)

I needed a basic dynamic key ignore option, so I added it with the -d param.
It is inspired by https://github.com/aldarund/vue-i18n-extract/commit/7fe5fe6d28c06e95f9a6743b2a603b3abf1fc267 and might fix #46 :)
Basically dynamic keys are the ones containing ${ caracters or ending by a dot ..
I don't have time now for doc updates, will try later if this PR suits you :)

Are there plans on merging this or should people try to figure out an alternative for handling dynamic keys?


Wrong 'from X stackers' shown

How can it say 465 sats in total but only one stacker zapped and I zapped 100? Am I just confused or does this really not look right? lol
https://github.com/stackernews/stacker.news/assets/27162016/27745a26-351c-4629-8c8e-37c117a8cdfd

Currently unsure how to reproduce. Will wait until we know more when this happens.

I think this might've been fixed when I implemented idempotent zapping. Reopen if encountered again.

Still happening:

It's this comment: https://stacker.news/items/378901
What do you see?

I wonder if this is because rewards add to the comment/post stack count.

I wonder if this is because rewards add to the comment/post sats.
So the X stackers is correct, but because you can't see how rewards contributed to the total it's confusing.

Ah, this would make sense.

This no longer happens


youtuve live video

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

Description

WRITE DESCRIPTION HERE

app does not resolve the youtube live video

Maybe it does, maybe not. Follow #30839; use the checklist (why do we have one?); supply the log.


Rename Windows base box from windows_2012_r2 to windows-2012r2

This change is intended to provide consistency with other projects
where we are running Windows under test-kitchen (e.g. omnibus_updater).

Good stuff!


Question: Random semantic embedding in SemanticTransformer?

When we already got the semantic token ids by HubertKmeans, the semantic embeddings are calculated using a randomly initialized embedding layer in SemanticTransformer. So why don't use the cluster centroids of pre-trained Hubert as the embedding?

The idea of the attention mechanism in the transformer network is to capture the relationship between token ids. Those semantic embeddings are randomly initialized but will be trained or will learn to capture the relationship between tokens in the training process.


Serverless with bref (v2)

Ref: https://github.com/PUGX/badge-poser/pull/332

New suggestion: what about leveraging bref Symfony bridge?


The output of volume rendering and rasterization on Neural_3D_Video Dataset

I tried the Neural_3D_Video dataset on a pretrained model, but the output of the rasterization was poor, while the volume rendering results look nice.
Do you have any ideas about this?
Dataset: https://github.com/facebookresearch/Neural_3D_Video
Volume rendering

Rasterization

Hi, thanks for your interest.
This is probably due to scale_factor. Since our model is trained on DTU data set (depth range is 425~905), when testing on other new datasets, we use scale_factor to adjust the depth of the new scenario to be close to the depth of the DTU dataset. For example, for the nerf synthetic dataset with a depth range of 2.5 to 5.5, we set scale_factor to 200, and for the LLFF dataset with a depth range of about 20 to 100, we set it to 12. You would try modifying scale_factor here.

Thank you for your quick response and excellent work!
It worked well. That was one reason; I also did not set zfar to 100 and znear to 0.01.


Make transaction type choosable

What context is your feature request related to?
Transactions and setting their types.
Right now CodableTransactions are type legacy by default. Using the integrated features this often leads to issues where users would rather choose newer types - instead of legacy - but can't really change the transaction type before sending it.
What solution would you like?
The solution is not clear yet - as this probably touches lots of different places in the library. This will need some more research. Ideas welcome!

I've leave my view here. Whilst I'd releasing 3.0.0 one, I've thought about to made type of CodableTransaction manageable my any internal method or something. But it have ended with the lack of time and tx in depts knowledge, so I left it as it is now: being able to be set only within initialisation.
The problem I've passed was the inability to appropriately manage internal types as EIP1559Envelope and so on, which one are actually stores the whole types within them, but it seems that it's in a much better state right now, like the data passing through CodableTransaction to them in appropriate way, but for now that's the one way street, when to as I see there's two way data passing required to implement such switches.
One more thing to think about is how to handle gas related properties between Legacy and EIP1559 types? Like they have directly opposite fields in itself, and the design should handle that somehow.

One more thing to think about is how to handle gas related properties between Legacy and EIP1559 types? Like they have directly opposite fields in itself, and the design should handle that somehow.

Good points in general. For the gas related properties specifically, I think the question would be what should CodableTransaction be. Should it be very powerful it what it can do or is it just an object to configure/prepare a transaction.
For now, I think I'd prefer the latter one. Keep it very simple and clean (and also more flexible that way). Checks e.g. for gas related properties based on types could probably be done on the individual services.

Should it be very powerful it what it can do or is it just an object to configure/prepare a transaction.

@janndriessen you've described the exact intent of mine that I've kept in mind when refactored it up to current state before.
It should not become to the only separate framework itself, but should be just the type that encapsulates all transaction values, that all other methods in framework should manage by their own.

🤝


Create a proper package

Hey,
I needed to use some of your modules in a project of mine and decided to create a proper python package instead of copying them into my project.
I have choosen the name 'steg_lsb' because I couldn't come up with a better one. This package is ready to be uploaded to PyPI if you want that.
Additionally, I have also moved all the command line stuff into a single module. The CLI is now
using 'click' for all it's magic.
PS: Not sure if these changes are of any use to you.. but they have been lingering around in my repository for quite a while already

Thanks!
I'm busy with some other projects at the moment, but will review these changes and try to merge them in a few weeks.

I fixed a few bugs I found in testing and tweaked the printed output slightly.
I also added an AUTHORS.md and credited you as "Peter Justin - Contributor" -- is that okay?
If so, I'm ready to merge the pull request and upload the package to PyPI.

Yeah sure, I am glad that my changes were of some use to you! :-)


Business Central SOAP Authorization for AAD Guest Accounts

Hello,
We have some users that have access to multiple Business Central tenants through a single AAD account that has been added as a Guest to AAD/Business Central. This works fine for the web client but SOAP services rejects them across the board with authentication/authorization errors. I don't see any known issues listed around this type of setup. Is this a use-case that is expected to be supported?
Thanks,
Zachary

Hello Zachery,
As long as the users have a valid license, this should work. I spoke with our development team who confirmed that this a supported scenario. So this might be a bug. We will investigate this in more detail. and provide a fix at some point as required.
Thanks for the feedback.

Hello Zachery,
The development team tested this functionality, and it worked for them. However, this might be an issue that with a specific scenario. If you could provide some more details about your scenario, I could pass the information on, and they could take another look.
Thanks
John


Missing guide for Hyper-V disaster recovery drill

When clicking on the link to run a disaster recovery drill for Hyper-V VMs, it shows the documentation for VMware VMs instead.

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

ID: d928a5ee-3c34-b56a-c797-d682e343c392
Version Independent ID: 6a95c94b-85eb-3b35-6e0b-c0a0172de7c4
Content: Run a disaster recovery drill for on-premises machines to Azure with Azure Site Recovery
Content Source: articles/site-recovery/tutorial-dr-drill-azure.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@daruizcorrea This guide is relevant for both Vmware and Hyper-V replication. I've added text to indicate this.
Thanks for your valuable input!
#please-close

hi @rayne-wiselman, thanks for the response. Unfortunately, it is still referring VMware on most of the paragraphs and the next steps link refers to the failover and failback for on-premises VMware VMs. It sounds like it should have its own guide so it does not mix details for different hypervisors.

@daruizcorrea We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Added support for recommendations requests.

Hi this adds the required classes and methods to support the Spotify recommendations api documented here: https://developer.spotify.com/web-api/get-recommendations/
I think this follows all of your practices. Let me know if you need anything revised or changed.

After submitted this PR I realized I'd forgotten to update some tests. These have been changed and should be a part of this PR now.

Bump?

Thanks for the tests!
I've added the invalid tag because this feature was merged into branch v2_requests, but not into the master branch yet. This way I have a better overview over all PRs.

Woah! It lives! 😉

;) see #72 for explanation.
Merry Christmas!


Option to concatenate RESORB file

This PR, built upon @scottstanie's PR #132, attempts to solve the issue that can take place when using RESORB file with ascending node crossing (ANX) time correction.
RESORB file, which will be the major orbit information source for forward processing for OPERA, has shorter time range than POEORB file. It is about as long as two times of the orbit period (T_orb = 1:38:44.57), with time shifting of about T_orb. That means, In rare case, this can cause that no single RESORB file can cover the time range of [sensing_start - T_orb, sensing_stop], which is necessary for ascending node crossing time correction.
This PR attempts to find the two RESORB files, and concatenate them into one. The two source RESORB files are:

One that covers [sensing_start, sensing_end] with padding (60 seconds)
The other covers [sensing_start - T_orb - padding, sensing_start - T_orb +padding] to ensure ANX before sensing time.

The concatenation takes place either when trying to find the orbit files from directory (get_orbit_file_from_dir()), or when downloading the orbit file (retrieve_orbit_file(), which was download_orbit())

Thank you @scottstanie and @gshiroma for taking a look at the PR and making a good points in the comment. Would you mind taking another look? Thanks.

Last question-
In get_orbit_dict, the arguments are  start_time, end_time, which each get padding added to them. We're doing other specific padding to the start/end elsewhere in the code depending on the orbit type. Should we get rid of this?
    # Add a 30 min margin to start_time and end_time
    pad_30_min = datetime.timedelta(hours=0.5)
    pad_start_time = start_time - pad_30_min
    pad_end_time = end_time + pad_30_min

based on my cartoon-based thought, 2 * 30 mins of padding would be okay for timing design of RESORB (duration, time shifting). However, shorter padding might help reducing the chance of the unnecessary use of RESORB pair. Based on the personal conversation with @vbrancat and time constrain we have, I prefer not to touch this part.

Sure not changing it now due to the time crunch is fine. I just wanted to make sure we know that we are padding the start once with T_ORBIT + 60 seconds and then again with 30 minutes in a different function

TODO: get rid of 30 min padding in get_orbit_dict()

@scottstanie @hfattahi @gshiroma Thank you for the good comments and feedback. I think this PR is ready for the another round of review.

@LiangJYu Thanks for the feedback. All good points. Please let me know if you have more suggestions, or if I have missed responding some of your comments. Thank you!

Thank you @scottstanie, @gshiroma, @hfattahi, and @LiangJYu for the review! I think the code is much better than the initial commit.


Swift 3.1 build updates

Description

Change .swift-version to 3.1
Add a swift n-1 linux build on travis
Add Package.pins with autoPin set to false
Disable swiftlint large tuple rule

Motivation and Context

How Has This Been Tested?

Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1046      +/-   ##
==========================================
- Coverage   89.92%   89.75%   -0.18%
==========================================
  Files          35       35
  Lines        1698     1698
==========================================
- Hits         1527     1524       -3
- Misses        171      174       +3

Flag
Coverage Δ

#Kitura
89.75% <ø> (-0.18%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/Kitura/RouterResponse.swift
91% <0%> (-1.5%)
:arrow_down:

Continue to review full report at Codecov.

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


Add TextTheme.of(context) (#65410)

Description
Theme.of(context).textTheme is quite a bit of boilerplate, having a TextTheme.of(context) would be a good addition.
Related Issues
Fixes #65410.
Tests
I added the following tests:
TextTheme retrieved from Theme.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

I wasn't able to run any tests as this is how my IDE looks like on current master:

So am trusting the automated tests here will do.
Are those errors intended as the tree isn't ready yet due to some change? Or is it my environment? (I already run flutter update-packages).

I'm going to close this. It's a nice idea but we've decided not to use it - https://github.com/flutter/flutter/issues/65410#issuecomment-689147821. Thanks again for the contribution!

Not a problem, I made it into a library if someone is interested.


Created oye-bobs-badge-submision.md

If you are submitting a change to index.md, you can go ahead and delete this template.
If you are submitting your badge completing file in !BADGE-SUBMISSIONS -- first, congratulations on completing your badge!
Make sure that you have met all the requirements before you submit:

[ ] Link to a Home Towns Pull Request
[ ] Link to main portfolio page

[ ] At least one heading
[ ] Some text
[ ] Links to other portfolio pages
[ ] At least one image

[ ] Link to a portfolio page that has a map generated with Python (this can be the main page again)
[ ] Link to climate change portfolio page

[ ] Instructions have been removed or replaced
[ ] Analysis of a different place (not Boulder)
[ ] Descriptive header
[ ] Site description
[ ] Analysis overview
[ ] Final figure
[ ] Headline for final figure

[ ] Link to vegetation portfolio page

[ ] Instructions have been removed or replaced
[ ] Analysis of a different event (not Cameron Peak fire)
[ ] Descriptive header
[ ] Site description
[ ] Analysis overview
[ ] Final figure
[ ] Headline for final figure

hey @oye-bobs - i saw your most recent pull request and your updated submission. Great job!! I'm going to close this one. Thanks! -NQ


Support Streaming in batch mode

What have you changed?
Please explain in detail what the changes are in this PR and why they are needed:
Support streaming in TiKV coprocessor. Here was 3 part of the change:

Support streaming in BatchDAGBuilder, and open the support for it in build_handler.
Implements handle_streaming_request in components/tidb_query/src/batch/runner.rs. And add a handle_internal_request to simplify the code.
Implements take_scanned_range for BatchExecutorRunner and add correspond integration tests for it (#5362 ).
Update testing in test_stream_batch_row_limit.

Don't assume reviewers understand the original issue.
What is the type of the changes?

New feature.

How is the PR tested?
Please select the tests that you ran to verify your changes:

Unit test
Integration test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No.
Does this PR affect tidb-ansible?
No.
Refer to a related PR or issue link (optional)

https://github.com/pingcap/tidb/pull/13062/files

Benchmark result if necessary (optional)
Any examples? (optional)

/run-all-tests

/run-all-tests

@breeswish @lonng PTAL

I'll take one day to see the new code here, and maybe I'll create a new pr.

 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.


fix(hooks):  fix issues with excessive number of pending callbacks

in usePlaybackState avoid possible race condition where setPlayerState could override the value received from the event listener by using callback version of setState by only setting if initial state is still missing
useProgress: the useProgress hook can overload slow systems because a setInterval loop is used without waiting for the results in between (see https://github.com/doublesymmetry/react-native-track-player/issues/1355) – this pr makes sure only one call is in the air at a time
useProgress: simplify the logic used to avoid updating state when progress hasn’t changed by using callback style of setState
replace isMountedRef usage with inline ‘mounted’ variables

@dcvz should be good to go now

@puckey it LGTM, if you could make the change @dcvz requested I'll do some testing tomorrow.

@jspizziri Thanks! I have answered the change request above

@jspizziri took care of the blocking review – so this should be ready for verification from you

@dcvz @puckey I tested this morning and everything seems good!


macros: make ParentType optional

Use glib::Object if not specified.
Fixes https://github.com/gtk-rs/gtk-rs-core/issues/350

Nice! thanks @pbor


LITE-27388 in formula transformation update decimal select with more …

Add selection of auto and 1 decimal in formula transformation

 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.Paulina Marcinowska seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Correct port mappings in proxy ingress

Currently, the pulsar proxy ingress exposes port 8001 with tcp at a path of /. While I'm not entirely sure what this port is user for, the service behind the ingress has quite a few other ports open (see below). Should the ingress be more in sync with the service? Maybe 6650 and 8080 be opened in the case of non tls and 6651 and 8443 in the case of tls.
The ports used in the proxy service:
  - name: https
    nodePort: 31200
    port: 8443
    protocol: TCP
    targetPort: 8443
  - name: pulsarssl
    nodePort: 30087
    port: 6651
    protocol: TCP
    targetPort: 6651
  - name: wss
    nodePort: 30535
    port: 8001
    protocol: TCP
    targetPort: 8001
  - name: http
    nodePort: 30284
    port: 8080
    protocol: TCP
    targetPort: 8080
  - name: pulsar
    nodePort: 30290
    port: 6650
    protocol: TCP
    targetPort: 6650
  - name: ws
    nodePort: 31025
    port: 8000
    protocol: TCP
    targetPort: 8000
  - name: burnell
    nodePort: 30168
    port: 8964
    protocol: TCP
    targetPort: 8964

Correction, I missed a few things in how the ingress works.


Add the options to not change existing models

If a user wants to pick only one of the tab completion or chat models, and keep using an existing one (eg. chat:granite, completion:deepseek-coder), it should be doable. Currently both models are overridden 100% of the time.

Hi @fbricon ,
I've a doubt. Is it possible to get current selected chat model from the continue? I tried, but couldn't found a way to do that.
But I got all downloaded chat models and showed inside the combo box and also added a checkbox for resetting the default value. I'm currently working on reset functionality.

Is it possible to get current selected chat model from the continue?

No you can't at the moment
I would like to keep it as simple as possible:

simply add : "Keep existing configuration" options to all 3 combos.
no new section with checkbox

That's it. And I'm not sold on displaying "Downloaded" instead of the size.

What if we do "Ollama list" cmd to get the list of already installed models and ignore those that are already installed?

I don't understand. You still need to be able to select any model we propose

Closed with #77


Proposed fix for optional highlight

Types of Changes
Prerequisites
Please make sure you can check the following two boxes:

[x] I have read the CONTRIBUTING document
[x] My code follows the code style of this project

Contribution Type
What types of changes does your code introduce? Put an x in all the boxes that apply:

[x] Bug fix (non-breaking change which fixes an issue, please reference the issue id)
[x] New feature (non-breaking change which adds functionality, make sure to open an associated issue first)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] All new and existing tests passed

Description
Implemented a fix for #280 - allowing highlight to be undefined. Also an additional type check has been added, to counter act other types for highlight.

Ping @klaidigorishti @boromo


Fix secondary indices write with adaptive granularity

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
For changelog. Remove if this is non-significant change.
Category (leave one):

Bug Fix

Short description (up to few sentences):
Fix bug with writing secondary indices marks with adaptive granularity.
Fixes #6121

Seems like tokenbf_v1 is not able to work with non-UTF-8/ASCII encoded strings.

Integration test failure is not connected with code.

Seems like tokenbf_v1 is not able to work with non-UTF-8/ASCII encoded strings.

True.


add Linux test in the pipeline

It should be possible to use the Linux build from the editor and have it ran here, just so we can catch things earlier (some platform difference that should be alright).
The pipeline is fairly small and quick here, so ideally, this is done in a way that doesn't make things slower - like adding just a new block that would run the Linux test, but keep the build and run in windows in the same block things are right now.

fixed by https://github.com/adventuregamestudio/ags-test-games/commit/e3c69dbe295050293b45eb5b67002d8825de06f0


Create a logout button on registration portal

On clicking the logout button user should be redirected to the login page,
Create logout button on home.html and formcorrection.html.

Create a logout button on home.html, whenever users click on log out a pop-up should appear "Are you sure you want to log out your changes have not been saved" then it takes the user back to the login page.
Also, try to add something like logged in Mr. XYZ. If you can. Else just create a logout button.

/assign

can u assign me this issue i have better ideas like i can create a prompt as well "do u want to logout"?

can u assign me this issue i have better ideas like i can create a prompt as well "do u want to logout"?

Already assigned to others.

Can anyone help me  I had creates a logout button but I cant redirect it to the login portal after clicking it can anyone help me.

Can anyone help me I had creates a logout button but I cant redirect it to the login portal after clicking it can anyone help me.

See other buttons like the help button how it is redirecting to other pages.

I would like to work on it please let me know if it is available

I would like to work on it please let me know if it is available

Others are already working on this issue.
Look for other issues.

@jatinjain001, @tanzzz-tech, and @sameer2403 any updates?

Can i try this?

Can i try this?

DCP or LGMSoC?

Can i try this?

DCP or LGMSoC?

dcp

Create a logout button on home.html, whenever users click on log out a pop-up should appear "Are you sure you want to log out your changes have not been saved" then it takes the user back to the login page.
Also, try to add something like logged in Mr. XYZ. If you can. Else just create a logout button.

i need to do only these changes right?

On clicking the logout button user should be redirected to the login page,
Create logout button on home.html and formcorrection.html.


Clarified language for windows users

This resulted from the product docs channel in the community slack: https://dbt-labs.slack.com/archives/C017GDLAF7D/p1664978917575649
Description & motivation

To-do before merge

Prerelease docs
If this change is related to functionality in a prerelease version of dbt (delete if not applicable):

[ ] I've added versioning components, as described in "Versioning Docs"
[ ] I've added a note to the prerelease version's Migration Guide

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects
[ ] Run link testing to update the links that point to the deleted page

thanks @b-per !


Added Pavement Coffee off Gainsborough in Boston

:coffee:
Same coffee shop as the one above, but a different location.

I love this list more and more :)


[WIP] Add react.config.js

A combination of webpack-chain and react.config.js would allow us to extend CRA without having to eject.
// react.config.js

module.exports = function() {
  return {
    webpackOverride(config) {
        // ... use webpack-chain API
    }
  }
}

I would love to have a conversation about this topic since we were asking for years about this and many people could take advantage of this feature.
Or
We could expose a webpack-chain config object as well as the existing plain object. The reasoning is that webpack-merge could do that much and webpack-chain configurations allow us to customize the config easier and more robust.
This will be useful for those who want to use CRA configuration since there is a lot of knowledge on the configuration but are forced to eject.

First of all, great work with the PR and taking your time!
This is a two edged sword kind of thing. CRA has always primary been the goto project for new comers to react. Meanwhile during the years many of us has started to build production apps with it and it works great. The upside is obvious, we get a community maintained stable webpack configuration without having to dig deep on our own.
The concern before has been that developers new to react and cra might copy and paste extended configuration from else where without really knowing whats going on and when it breaks it might cascade into reported issues here on github.
Over the years CRA has given in more and more to become a multi purpose build tool adding support for scss, typescript and more.
I think its valid to revisit this disucssion now that tools like webpack-chain
Custom templates are about to drop soon, maybe we should go a similar way to  provide an API for custom webpack configuration recipes?
One thing to keep in mind though, adding such tool makes CRA more difficult to follow webpack releases. Not that CRA needs to have the latest webpack on day 0 but just to put it out there, using for example webpack-chain would need CRA to track a version of webpack that also webpack-chain supports.

Meanwhile during the years many of us has started to build production apps with it and it works great.

I wouldn't propose this if it weren't the case that I am forced to eject.
Legacy software, with legacy infrastructure, not that easy to decide to use CRA.
I had done it and I had to keep track of every release simply because an organization with 14 years of development can't use CRA a the moment (we are almost there).
I wish I could import the config from CRA (just an everyday object), and that's all.
But once you get into implementation details, you would understand how webpack-chain API will be beneficial, sometimes you need the especial API to be able to configure some options of a plugin or particular loader.

This is a two edged sword kind of thing.
The upside is obvious, we get a community maintained stable webpack configuration without having to dig deep on our own.

I would ask for some understand that some people do understand Webpack and these configurations.

One thing to keep in mind, though, adding such a tool makes CRA more challenging to follow webpack releases. Not that CRA needs to have the latest webpack on day 0 but just to put it out there, using, for example, webpack-chain would need CRA to track a version of webpack that also webpack-chain supports.

I don't know what to do about this one, and I can move webpack-chain into CRA. 😄
This incentive good collaboration in the ecosystem, we may have an issue, or we may not 🤷♂ , in any case, all it matters is the support form multiple parties.
I am not sure, you are 100% right.

Custom templates are about to drop soon; maybe we should go a similar way to provide an API for custom webpack configuration recipes?

What do you mean? I am not sure what templates are.
P.S:
I hope we can discuss this topic, for such a long-time CRA had been in a strong position of rejecting any proposal, and I don't think that it is fair for people that do have to understand Webpack and underline tools and can't merely use CRA.
I do understand, however, that CRA doesn't want to take responsibility for those who screwed the configurations.
Still, I genuinely believe that those that are trying to mess around internationally with the config are willing to accept this.
Just asking for some sympathy for those that are forced to eject 😢 My life for the last year had been watching releases and making sure that I keep my code in sync, I know how valuable these configurations are.
Especially that I know that most people reject for simple things like adding support for styled-components or it used to be because of TypeScript and so on.
Maybe enabling the feature will allow people to have a better experience on their end without having to ask CRA to include everything that is cool out there.
Hopefully, we can have a conversation about it, I will try to help as much as I can, just want to be proactive.

What do you mean? I am not sure what templates are.
Templates is an upcoming feature to allow bootstraping new cra apps with custom templates to get some foundation to start build upon. Imagine there's going to be community maintained templates that includes design systems, router etc.
https://github.com/facebook/create-react-app/pull/7716

An alternative to your proposal could be to allow for custom packages. For examplecra-config-styled-components   or similar.

P.S:
I hope we can discuss this topic, for such a long-time CRA had been in a strong position of rejecting any proposal, and I don't think that it is fair for people that do have to understand Webpack and underline tools and can't merely use CRA.

Just to balance,  over 2700 PRs has been merged so its not fair to say "any propsal gets rejected". My reply was meant to start discussing the topic, because yeah custom webpack config has been proposed before so I just gave some background. Personally I like your proposal and my reply wasn't meant as criticism, more like initial neutral.

Hopefully, we can have a conversation about it, I will try to help as much as I can, just want to be proactive.

This is a conversation about it, just have patience and stick around to hear what other thinks about it. Again, good job with the initial work.

Oooh the way I see it, templates are not quite the same thou, but I am happy to see templates becoming part of CRA actually ❤

Just to balance, over 2700 PRs has been merged so its not fair to say "any propsal gets rejected".

Well I had been following this topic about extending CRA for years, and every single time the conversation came out, we get nowhere and gets rejected.
This is why I want to do the work and then talk about it 😄

Personally I like your proposal and my reply wasn't meant as criticism, more like initial neutral.

Oh kool, I have all good feelings about your replies so don't worry.

@andriijas I need your support,
At this point, we need to decide whatever direction we wanna take,

Add react.config.js with webpackOverride
Add webpack-chain to people can take advantage of the underline webpack config and be able to customize the configuration since webpack-merge will no cut it.
Reject either proposal

Hi @yordis, we like the idea of a config and have been talking about if for a while, but the concern is always the complexity it adds - for users, and for us as maintainers too.
The webpackOverride feature would be a much harder sell, it's something that has been rejected numerous times before...
We have also talked about the idea of allowing developers to set different scripts versions in templates too, but that also introduces risk.
It would be good to involve @iansu and @ianschmitz in this conversation.

Hi @yordis, we like the idea of a config and have been talking about if for a while, but the concern is always the complexity it adds - for users, and for us as maintainers too.

@mrmckeb I am more than happy to write an article explaining before and after, and videos for you if that is the case. I am not sure what to do about that one other than be here to help you as much as I can.

The webpackOverride feature would be a much harder sell, it's something that has been rejected numerous times before... @andriijas has summed all of this up nicely above.

I dream of the day where we remove // @remove-on-eject-begin and we make the tool 0Config always, but friendly to extend.
As I said before, I am more than happy to document every line, how it works for maintainers and make sure that I do my part on explaining every piece of the software, but you have the final say regardless.
But if at least I get the config I guess that is beyond enough personally speaking.

I am more than happy to write an article explaining before and after, and videos for you if that is the case.

This is not about the documentation of the feature. It's about the long-term impact. We already have trouble responding to all issues and PRs as it is, which is not great.

@yordis This PR and the direction will be discussed. You have done excellent work so far so just enjoy the weekend and have some patience  :) thanks!

I was pointed to this issue by the wonderful @ianschmitz: https://www.twitter.com/schmitz_ian/status/1189296593654165507
I just wanted to chime in and say how a mechanism along these lines would be delightful to me and I suspect many others. Let me explain my own situation.
CRA is 95% perfect for my needs. However, I eject for two reasons:

Because I want more control over workbox (that actually might get resolved in future by https://github.com/facebook/create-react-app/pull/5369 )

I want to swap babel-loader for ts-loader. It's worth noting that I maintain the ts-loader project.

I'd love it if a mechanism existed to allow to swap out the pieces of webpack config if you needed. I'd be more than happy if that was available on the same basis as eject is. That is to say: if you use it you can't raise support tickets around it; much as you can't with ejected configs. Buyer beware and all.

This is not to provoke, just need to get a better understanding - why isn't neutrinojs, razzle, next or gatsby an option for those rare occasions you need to eject?
@yordis  What do you think about rather than exposing and having the webpack extensible via config file,  rather refactor the internal webpack config in CRA to smaller chunks ("lego blocks"). Which can be independently tested and glued together to a complete config? And as a end user you could glue it together with your own additions (but probably still via a tool like react-app-rewire)

@yordis What do you think about rather than exposing and having the webpack extensible via config file, rather refactor the internal webpack config in CRA to smaller chunks ("lego blocks"). Which can be independently tested and glued together to a complete config?
This is what I actually have in some internal tool, I will recommend you to do so since testing chunks of configs is easier than a massive config.
Worth saying, you will trade-off discoverability since you don't see the full config, but you gain in maintainability.
But, I definitely recommend this, and I had done it and I know how we could extract the pieces today, just let me know if that is the direction that CRA wants.
I would create another package just for config related stuff, and react-scripts is the CLI + configs.

Related #3775

We are currently exploring ways to extend create-react-app with a config file rather than relying on environment variables only. We do appreciate you taking time for these efforts. Full webpack override is not something create-react-app is looking to support long term at this moment or in the near future. Sorry for any inconvenience!

@andriijas what about add new option in eject --config script that will only create react.config.js  project root after that you need to worry about maintainene because It's already mention as

If your project needs more customization, you can "eject" and customize it, but then you will need to maintain this configuration.

we need this future because we don't want all configrations files to add a webpack plugin or something and simply upgradeable.
sorry if anything wrong please advice me.
Thanks.

@pavinthan @yordis what would help a lot is to get a better understanding what kind of custom configuration is actually needed and we can maybe find another way.  There are great tools like Neutrino js, razzle and even next js allows custom webpack config.
Everyone acknowledge that there is a small percentage of all users that demands full webpack overrides. The cost (in terms of time) in maintaining is unfortunatly too high.

@andriijas Honestly, based on your response, I feel profoundly offended, and your answer feels like you ignored all I said, or you genuinely don't understand; I will take the second one.
I don't believe that Mob mentality like you said "small percentage of all users" is something that we should encourage.
And we are not a small percentage, this is far from reality in the first place, you don't know what you don't know.
History lesson:

TypeScript support took forever to add
You have to eject because of GraphQL support, still
Styled-Components and Babel had to work together for Babel macros concept so you didn't have to mess around with configs because the #1 community was forced to eject from a fantastic project, still
People keep asking for some configs because they are being forced to change, go and look up your existing and closed issues, and many of them could be simply solved without eject in their end.

I am sorry, you had missed history entirely on what happened regardless of this topic.
I hope you learn how to enable the community how the Vue community does it, as Angular does it, like almost every ecosystem out there that actually listen to their people.

@yordis, @andriijas is a respected member of our team, and the views he shares represent those of the entire team - there is no offense meant and none should be taken. Please speak to the team with respect. We're all volunteers, even though this is a Facebook project.
There's not a single member of the core team that doesn't want to enable more customisation and open up more configuration, and we will continue to do that progressively.
I'm not sure why you need to eject for GraphQL support, and I think the Babel macros solution works well. I agree that TypeScript took too long - but it is supported now, and we're continuing to improve support in that area.

@mrmckeb I didn't offend anyone.
I do respect his opinion, as I expressed previously in the thread I wanted to start the discussion knowing that you will reject it unless you want to ignore the full conversation.

I answered multiple times, included privately on Twitter what the use cases are.

His reply in support of Mob decisions is not something I will support. Maybe rephrase the sentence to be a more welcoming and inclusive community, instead of using "minority" as a way to win.

The fact is that he missed history on this topic and ignore all the issues opened and PR closed about it.

That being said, the work was done, and once again was rejected, I don't think anyone is getting anything out of the conversation.


Feature Request: Birthday Management

Feature Request
Birthday Management
Description:
Integrate a feature to manage birthdays within the application.
Objective:
Users can input their birth dates, registering them into a data file.
Implementation:

Create a function for users to input and delete their birth dates.
Implement functionality to store and retrieve these dates.

Birthday Greetings
Description:
Implement automated birthday greetings for users.
Objective:
Automatically send birthday messages to users at 8:00 AM on their respective birthdays.
Implementation:

Create a task in order to check if they is a birthday everyday at 8:00AM.
Create a custom message to ping and greet the user.

TO DO

[ ] Secure the data file

Add a command to display all the birthdays

New features / Changes:

[x] Sort date by order
[x] Changing response language to french
[x] Add descriptions to arguments
[x] Add README information to contribute

Fixes:

[x] Manage leap years and 29 february

Update:
Currently working on:

better formatting?
next birthday timestamp function for list command


cant locate a mislocalized nucleus term

I'm looking for the FYPO term when 2 nuclei end up in a single compartment. I can't find it anywhere by a number of searches and routes...any ideas
tried "nuclei"
mislocalized nucleus, children
compartment
binucleate vegetative cell and children

Its this one
septation following abnormal chromosome segregation, with anucleate cell formation
I am now wondering if chromsome segregation is strictly wrong here, its just that because the nucleus is in the wrong place ?

I'd just found the term myself. Let me know if any additional synonyms would help.
This term (FYPO:0003166) might also be a bit more specific than you want, precisely because it does specify abnormal chromosome segregation. It's named and defined that way because it's the sibling of cut.

yeah but I would probably have looked for a "binucleate monoseptate cell
with nuclei grouped" or something.
On Wed, Jul 8, 2015 at 3:06 PM, ValWood<EMAIL_ADDRESS>wrote:

Its this one
septation following abnormal chromosome segregation, with anucleate cell
formation
I am now wondering if chromsome segregation is strictly wrong here, its
just that because the nucleus is in the wrong place ?
—
Reply to this email directly or view it on GitHub
https://github.com/pombase/fypo/issues/2239#issuecomment-119572559.

--
Antonia Lock, PhD
PomBase Biocurator, http://www.pombase.org
Department of Genetics, Evolution and Environment,
The Darwin Building,
University College London
London WC1E 6BT, UK

let's have a new term also which does not specify a chromosome segregation problem
liberal synonyms, and should it be related to mislocalized nucleus?

note to self, update curs/c4669f01934c07d6

OK. I think has_part mislocalized nucleus might work.

So far I've added this (and made it an additional parent of FYPO:0003166):
[Term]
id: FYPO:0004699
name: monoseptate vegetative cell with binucleate and anucleate compartments
def: "A cell phenotype observed in the vegetative growth phase of the life cycle in which a cell has two nuclei and one septum, but both nuclei are in a single compartment on one side of the septum." [PomBase:mah]
synonym: "binucleate monoseptate cell with nuclei grouped" RELATED [PomBase:al]
synonym: "monoseptate cell with binucleate and anucleate compartments during vegetative growth" EXACT [PomBase:mah]
synonym: "monoseptate vegetative cell with binucleate and anucleate compartments during mitotic cell cycle" RELATED [PomBase:mah]
is_a: FYPO:0001225 ! monoseptate
relationship: has_part FYPO:0002071 ! mislocalized nucleus during vegetative growth
I can add any other synonyms that might help you find it (including related ones, if
something non-exact would be useful). Will close when you're happy with the synonym assortment.

done in 09c63e367ebd2ddf4ea2f613c214a0eebc6b5c69
(reopen or create new ticket if you think of any more synonyms)


dt and dd elements on Documents detail page

We have the goods and their categories that are listed on each Documents detail page in dt and dd elements, (which is probably semantically incorrect) and they need to be children of a dl element, which we're currently missing

I inserted the dl element. Do you want to put an (un)ordered list instead? The groceries could be ordered alphabetically anyways

Alphabetically ordering the groceries sounds like a good idea. About the unordered list, I don't know, personally I don't think the category counts as a description of the term, but then again with a simple unordered list we lose any (semantic) connection between good and category...


Same Board?

The board this is for, is it the same as this one? https://www.amazon.com/HiLetgo-Gyroscope-Acceleration-Accelerator-Magnetometer/dp/B01I1J0Z7Y/ref=sr_1_1_sspa?dchild=1&keywords=InvenSense+MPU-9250&qid=1617741587&sr=8-1-spons&psc=1&spLa=ZW5jcnlwdGVkUXVhbGlmaWVyPUExN0FCVzdSNklDVUVaJmVuY3J5cHRlZElkPUEwNzI4MTE4NExFVEFURzdYODNHJmVuY3J5cHRlZEFkSWQ9QTAxMDY3NDUzT01PT0ZYS0pUVDNTJndpZGdldE5hbWU9c3BfYXRmJmFjdGlvbj1jbGlja1JlZGlyZWN0JmRvTm90TG9nQ2xpY2s9dHJ1ZQ==

It's for the MPU-9250 sensor, not any particular breakout board. I would use caution and prefer to purchase a breakout board from a reputable vendor, like Sparkfun, there are a lot of fakes on the market.

It's for the MPU-9250 sensor, not any particular breakout board. I would use caution and prefer to purchase a breakout board from a reputable vendor, like Sparkfun, there are a lot of fakes on the market.

I don't really have any experience in this area. I'm just curious about these boards because I'm looking for a way to change the orientation of my monitor in software when I physically rotate it. Since I don't have any experience with this sort of thing, I don't really have any idea of reputable brands - aside from Sparkfun, which you mentioned.

Besides Sparkfun and Emsensor, I would also trust Pesky Products:
https://www.tindie.com/products/onehorse/ultimate-sensor-fusion-solution-mpu9250/
If Adafruit carried it, they would be on the trusted list as well.


[Networking] True async without Atomics

Amazing work on this project - it's no easy feat getting all of these layers to work together in a simple way.
I built a user space TCP/IP JS library (via WASM bindings to lwIP) for use cases like communicating with emulators via L2 frames and exposing a nice TCP interface to the developer (running in the browser). I originally built it in Go using gvisor's tcpip stack but later rewrote to use lwIP which is much smaller and avoids bundling the Go runtime in the WASM file.
I've been trying to find a way to connect this with c2w so that devs can communicate with the container via TCP/IP in the browser. So far I am:

using your wasiHackSocket() example to implement the socket APIs which bjorn3's shim doesn't have
using your wasiHack() example to implement poll_oneoff which (I think?) is necessary to use the socket APIs.

IIUC, poll_oneoff is expected to be a synchronous blocking call which your example accomplishes via SharedArrayBuffer + Atomic APIs within a worker. The trouble with these APIs is that they limit a large number of developers due to cross origin header security requirements which many developers don't have control over.
I'm trying to see if there is any possible way to implement this asynchronously from the JS side. My understanding is that this type of thing is usually accomplished by compiling the WASM using some form of asyncify. I noticed that you use Binaryen's asyncify pass in both Bochs and TinyEMU to implement setjmp and longjmp, but this is where my understanding ends. Can we somehow use jumps/trampolining/unwinding or similar to asyncify poll_oneoff? Or maybe something else entirely?
Hope this makes sense and looking forward to hearing your thoughts.

I guess your design might work though I haven't tried it. An example implementation is: poll_oneoff wrapper can start unwind (maybe with saving args to somewhere JS can easily access) instead of calling the imported function.  wasm_start needs to return to the caller JS so that JS code can do anything needed to handle poll_oneoff. Then JS code calls Bochs  again then it should rewind to poll_oneoff.

Thanks for the quick reply. This approach makes sense - I'll see if I can make it work.


Subresource integrity

Add subresource integrity checks — should (partially) help mitigate a certain class of MITM attacks. Also bumped to buffer-graph@4 which now uses sha512, which in turn we can use to set the integrity checks.
See Also

https://developer.mozilla.org/en-US/docs/Web/Security/Subresource_Integrity

Todo

[x] make sure the stores URLs in the service-worker globals are stores in hex
[x] store the hashes in bankai build are written as hex
[x]  fix tests that rely on hashes

Had to disable this for CSS; conflicted with preloading. See https://twitter.com/yoshuawuyts/status/920794607314759681 :(

LGTM otherwise! very nice :D

Implemented feedback. Merging! :D Yayayay


Step dependencies are paths to other values, which trigger the step when they are updated

Ensure that values updated by processes are not also updated by steps.
Other checks to support circularity without infinite loops.


Ignore prefer_const_constructors in generated code

This is necessary to avoid warnings in generated Dart code for Rust
enums that have variants with no fields.

Maybe run codegen (https://github.com/fzyzcjy/flutter_rust_bridge/blob/master/CONTRIBUTING.md#appendix-sample-commands-to-run-code-generator) to make CI happy

Done.


Role assignments should use Principal Id

https://github.com/Azure-Terraform/terraform-azurerm-kubernetes/blob/bfe925b1c7d918a27c375651b5000c09ec8b076d/main.tf#L14
This should instead reference the principal_id property:
principal_id         = (var.user_assigned_identity == null ? azurerm_user_assigned_identity.aks.0.principal_id :                           var.user_assigned_identity.principal_id)

Hi @appkins  - Can you review that PR please and let me know if it fixes the issue?
Thanks

@jhisc We actually need this changed in one other location. I saw the pull request was approved before I could catch these
https://github.com/Azure-Terraform/terraform-azurerm-kubernetes/blob/1f61f8ce8d2373a9c720ac5f79d70316d37c9e7b/main.tf#L24
This needs to reference the principal_id. I also found that there's an incorrect reference in the output here:
https://github.com/Azure-Terraform/terraform-azurerm-kubernetes/blob/9966c3653cd67e2b8c149343c87d945d3aa6303e/output.tf#L76

This issue is fixed. Thanks @jhisc!


slugify extension options should support importing function by path

Extensions like toc currently assume the slugify option specifies a callable.
In some cases, markdown.Markdown kwargs might be configured via text files (such as yaml). This prevents the slugify parameter from being specified as a string (e.g. 'mycodebase.toc_slugify').
Could python-markdown extensions that require a slugify callable, namely toc and headerid, accept a slugify option as a string and automatically import it?

While I'm not completely opposed to the idea, it most likely won't happen for some time. Python-Markdown it currently in "maintainance mode" as we work towards 3.0. In other words the 2.x line will only receive bug fixes going forward with no new features.
The cited example of using YAML config files is not a very strong case as the PyYaml Lib allows python objects to be assigned as values. You just need to explicitly define the type.
Additionally, while accepted strings (using dot notation) and then importing using that string is already in use in version 2.x (for extension names), that behavior will be removed in version 3.0 (we will be using entry_points instead). Therefore, I'm even more hesitant to support Python dot notation strings elsewhere in the code.
Additionally, part of the version 3.0 Roadmap includes breaking out all extensions from the main Markdown package into their own standalone packages. That being the case, each extension could grow on its own (and more easy be forked). So at that time it might be worth reconsidering.
However, at this time, for this package, I am going to close this as "wontfix" When the version 3 implementation of the TOC extension is broken out into its own package, you can raise the issue there.

With the 3.0 roadmap in mind, I see that as a completely reasonable approach :+1: . It could be a future enhancement once the official python-markdown extensions are broken into dedicated packages.


WAK 197: Yegna Article Upnext

@sewagodimo Please review.
Cleanup upnext article file. Customize for watch, listen categories.

wtih


Add --stacktrace to perf tests

So that we can see the stacktrace in build log.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Only display the assetlib Retry button if the download failed

The Retry button will be hidden at first and will only show if the download failed for any reason. It will hide again when clicking it.
Preview
https://user-images.githubusercontent.com/180032/108141425-fcf85980-70c3-11eb-861d-e12462b957b3.mp4

Thanks!

Cherry-picked for 3.2.4.


Enforce db_prefix.

Make this a repeated field in SourceRepository. No Datastore migrations are necessary as NDB seamlessly handles translating from a singular field in Datastore to a list in ndb.Model.

Remove old legacy code that constructed a Datastore key for Bug based on the source (<source_name>:<osv ID>) if we're importing an unseen record and the the db_prefix didn't match. This actually never got used, because all of our Bug queries check for db_id and bypass the Datastore key (https://github.com/google/osv.dev/blob/f182e67869f4963d1cfc406b75953edd46eb6338/osv/models.py#L337)
Existing keys in Datastore with the existing format (<source_name>:<osv ID>) will remain unmodified (again, because our queries key on db_id). In the future, we can further clean this up by removing db_id and just using the Datastore key instead (which will likely also bring some minor performance improvements).

For #2433.

You'll want to omit any changes to source.yaml until the code changes have hit Production, right? Perhaps also source_test.yaml until the code changes are live in Staging?

Yes, good call out, I'll split this out.

You'll want to omit any changes to source.yaml until the code changes have hit Production, right? Perhaps also source_test.yaml until the code changes are live in Staging?

Yes, good call out, I'll split this out.

Broke out https://github.com/google/osv.dev/pull/2446 for this.


when TS->Blocks conversation fails, suggest remove blocks file

Big red button + confirmation message that "main.blocks" should be removed.

so, we have separate files for blocks and TS?

Yes we do. Removing the .blocks file effectively makes it a TS project.
Sent from Outlook Mobilehttps://aka.ms/djubve

From: Tom Ball<EMAIL_ADDRESS>Sent: Sunday, August 7, 2016 1:23:20 AM
To: Microsoft/pxt
Cc: Peli de Halleux; Author
Subject: Re: [Microsoft/pxt] when TS->Blocks conversation fails, suggest remove blocks file (#189)
so, we have separate files for blocks and TS?

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/pxt/issues/189#issuecomment-238070264, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AD-4KXbriJd42U8OJ8k9ndlqsm2E4S9Tks5qdZX4gaJpZM4JcLj4.

Fixed in issue #189


Ballerina byte type needs to be refactored to use long registry and should be treated as int during compile time

Description:
According to the spec, "The byte type is an abbreviation for a union of the int values in the range 0 to 255 inclusive".
Hence, we can use the same long registry which is used to store Ballerina int values to store byte as well. Also, we can treat the byte values as int during the compile time as it would be the optimum way.

Fixed with #14209


Testing structure for app

As an app developer I would like to have a ready and efficient testing structure, which allows the easy testing of new application versions.
Acceptance criteria:

[ ] testing methodology ready and usable

Pollion reference:
https://github.com/pollion/sunflower-deployment/issues/23

Review issue description and move to "Ready for Development". @leagerndt @JuliaFreitagBGSt

I would suggest we move the testing structure to non-critical. Please review @leagerndt @JuliaFreitagBGSt

@nscharioth agreed. As discussed we will stick to human testing for now and develop a testing structure in the future
@leagerndt


What are the different permissions required to call SSIS packages from local and file systems?

@Albych commented on Wed Apr 22 2020
Hi
In the section Run SSIS packages in SSISDB by Azure SQL Managed Instance Agent step 6, the screenshot shows that we CANNOT change package source and server. That means we can only execute SSIS packages on the server. But in the next section Run SSIS packages in file system by Azure SQL managed instance agent step 4, the screenshot shows that we CAN change package source and server etc.
Now I'm in first situation, but I want to execute SSIS packages from file system or other SQL Server, I wonder what different permissions between them.
Thanks for your reply!

文档详细信息
⚠ 请勿编辑此部分。 docs.microsoft.com ➟ GitHub 问题链接必须具有此部分。

ID: af104174-8686-bd3a-7e58-f0563e9959b0
Version Independent ID: 6b4e1b1a-fa33-62a2-6f48-15d3174ffe57
Content: 按 Azure SQL 托管实例代理执行 SSIS 包
Content Source: articles/data-factory/how-to-invoke-ssis-package-managed-instance-agent.md
Service: data-factory
GitHub Login: @lle
Microsoft Alias: lle

@srvbpigh commented on Wed Apr 22 2020
Hello, @Albych
Thank you for your feedback.
We are actively reviewing your comments and will get back to you soon.
Kind regards,
Microsoft DOCS International Team

@lle commented on Wed Apr 22 2020
I am unsure why I am tagged in this. Please clarify

@CeciAc, @Albych Thanks for reaching out. We are looking into the query and will get back to you shortly.

@djpmsft Can you please look at this discrepancy?

#reassign @lle

lle is not a valid GitHub ID, or is not a collaborator on this repo.

Hi @CeciAc ，
If your SSIS packages are stored in SSISDB, the MI agent only can execute the package from the SSISDB in the same Managed Instance. So in our UI, you don't need to choose it as current Managed Instance is default.
But if your packages are stored in file system such as Azure Files, then you need to specify the location of your packages in MI agent UI.
Hope this answer your question.

#please-close


Make sessions more canon

Also, can we please resolve the unit tests in the tests/schemas/test_timetable.py file? They'll need to be updated as the Session model has changed.


CodeceptJS / Playwright / Getting intemittent 'go to' undefined issue on I.amOnPage(...)

What are you trying to achieve?
We're trying to understand what's causing the intermittent issue where we are getting 'undefined'
What do you get instead?
We’ve been having a intermittent issue where we are getting ‘go to undefined’ when calling ‘I.amOnPage(’/’)’ Does anyone has an idea how to resolve this? Appreciate the help in advance.
NOTE:

We are running in parallel using 5 bamboo agents but 1 browser instance each.
That ‘loginToDashboard’ custom function and used by all of our scripts.

loginToDashboard: function (username, password) {
      this.amOnPage('/');
      this.login(username, password);
    },

Feature('Sample')

Before(async ({ I }) => {
    I.loginToDashboard(Login.Email, Login.Password);
});

Details

CodeceptJS version: 3.2
NodeJS Version: v14.17.3
Operating System: Windows 10
Playwright
Configuration file:

It can implicitly mean that the instance of browser fails to start. goto just happens to be the first command on the browser, so it can seem not obvious what is happening. I have had the same issue, a CI agent in my case happened not to have enough memory and killed the browser. You should investigate/monitor resources consumption of the host where you run the test at the times of the failure.

It can implicitly mean that the instance of browser fails to start. goto just happens to be the first command on the browser, so it can seem not obvious what is happening. I have had the same issue, a CI agent in my case happened not to have enough memory and killed the browser. You should investigate/monitor resources consumption of the host where you run the test at the times of the failure.

Thank you Jan, We also noticed resources consumption when running parallel test (Multiple browser instance) in the server, it reached almost 90% CPU usage. We decided to move back into (1) single browser instance so we don't consume that much resources but still the same, intermittently getting this issue.
Is there anything else we can do about it or any known workaround? Appreciate the help.

Another case when my browser was failing to start with that error was when I passed bad params to the browser. I was preparing a plugin to Chrome that sometimes got malformed and went to Chrome corrupted, so Chrome could not accept it and failed to start. Do you do any preparations of that kind?

Another case when my browser was failing to start with that error was when I passed bad params to the browser. I was preparing a plugin to Chrome that sometimes got malformed and went to Chrome corrupted, so Chrome could not accept it and failed to start. Do you do any preparations of that kind?

Hmm, No, I don't think we have that special setup we pass before launching the browser aside from this argument :(

Hi someone here can help me out please, why we're getting this intermittent 'go to' issue when calling I.amOnPage(...) function...


Support asynchronous file I/O on Windows

Since #14255, it is now possible to open non-blocking Files on Windows, as previously their blocking mode could not be changed after initialization. This PR makes operations on those Files actually work.

@HertzDevil is this ready for review?

All the progress here has been split into individual PRs that work with the refactored IOCP interface; closing


UP-4901- added unit tests for package org.apereo.portal.api.portlet

…dded assert statement to check window state and portletmode

Checklist

[X] the individual contributor license agreement is signed
[X] commit message follows commit guidelines
[X] tests are included
[ ] documentation is changed or added
[ ] message properties have been updated with new phrases
[ ] view conforms with WCAG 2.0 AA

Description of change

Added unit test for org.apereo.portal.api.portlet  package
Jira ticket- https://issues.jasig.org/browse/UP-4901

:office: Per offline discussion ./gradlew :uPortal-api:googleFormatJava is not working in this case.
./gradlew :uPortal-api:uPortal-api-platform-impl:googleFormatJava may work as an alternative.

AppVeyor passed but is unable to report status back to GitHub https://ci.appveyor.com/project/drewwills/uportal/build/1.0.589


Add skip method

Change log

lib/it.each.js
Added the it.each.skip function which will allow you to skip a test that uses it.each.

test/it.each.js
Added test suite for the new it.each.skip function that skips 3 tests, runs 1. Then the count from the test suite is then verified.

Verification
npm run test

@jdavis61 this seems okay except for two things:

Can you please remove the package.json version bump?
Can you squash this PR? There are a whole bunch of commits for a very small delta.

@whitfin feedback implemented, please let me know if anything needs to be changed or updated.

@jdavis61 perfect, merging!


Scenecomposer's FilterExplorer is broken

So I wanted to add a water filter to my scene just to see how it would look.
Checked out this youtube video and he used the FilterExplorer tab next to the SceneExplorer. However in current version of sdk opening the FilterExplorer tab only displays an nameless tree node with a blank piece of paper icon and right clicking on it does nothing.
Having that back would be great, since it allows quick preview how the scene would look with a filter.

First off, this is an issue tracker, not a troubleshooting section, thats what the forum at hub.jmonkeyengine.org is for.
As for your problem, create a new filter file (j3f) in your assets via the new File menu and open it, thats when you can add new filters to it in the FilterExplorer.

Okay, sorry. I thought this was a bug.


Fix (Observable.fromEvent): event listener with options is not removed

removeEventListener must be called with the same options used in addEventListener

Description: This problem is noticeable in combination with Angular2. When having subscribed to and then unsubscribed from an observable created with Observable.fromEvent with options, each time the event is triggered it causes Angular's change detection.
Related issue (if exists):

Coverage remained the same at 97.734% when pulling 7922fbe9c933129d2c32fda95b3e81a8a1e59a53 on zy-serguei:patch-1 into 3bb624032fe600580605307e1586699982220f14 on ReactiveX:master.

Is there a way to change the commit message via the web interface? I couldn't find it. Please feel free to change it if it's not much trouble for you.

no, you have to change via local git git --commit-amend and update branch. Still, you may need to update commit to include test cases.

Really this just needs a test case and it's good. I can change the commit message if I have to.

This is already done in master.


Fix breadcrumbs logger

What problem does the pull request solve?
no logs appear in sentry
Checklist

[ ] I've used the pull request template
[ ] I've linked this PR to the relevant issue (if mission work)
[ ] I've written unit tests for these changes (if code change)
[ ] I've updated the documentation in (If any documentation requires updating)

[ ] README.md
[ ] Elsewhere (please link)

If anyone works on the runner before Monday, do merge this in to get the new version of ruby :ok_hand:


RHODS: minor updates

config: ansible.cfg: disable Inventory caching

See https://github.com/ansible/ansible/issues/78178#issuecomment-1175212485
This commit should avoid random crashes caused by running Ansible in
parallel, with a file-based cache plugin.

rhods_test_jupyterlab: capture only the ods-ci container logs

This commit removes the collection of the 'artifacts-exporter'
container of the test Pod. The logs of this file are not so useful,
and they are available in the artifacts collected by the gather-extra
script of the cluster destruction post test step.

testing: ods: generate_matrix-benchmarking: fix crashes

This commit fixes various code mistakes introduced by
https://github.com/openshift-psap/ci-artifacts/pull/373.
These two test commands are now functional:

/test ods-plot-jh-on-ocp plots the last results of /test ods-jh-on-ocp
/test ods-plot-jh-on-osd plots the last results of /test ods-jh-on-osd

Commits tested as part of https://github.com/openshift-psap/ci-artifacts/pull/404
/cc @fcami

/approve


Create an unsaved changes message when you make changes in user permissions or book settings without saving it

Done!


Added rate limit throttling info

Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

On hold pending exact numbers (100 active concurrent search jobs)

@JV0812, is this PR still on hold?

@JV0812, this PR is pretty old. Can it be published yet?


implement lwm2m deviceshifu and gateway

What this PR does / why we need it:
implement deviceshifu lwm2m and gateway lwm2m
Will this PR make the community happier?
yes
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
How is this PR tested

[ ] unit test
[ ] e2e test
[ ] other (please specify)

Special notes for your reviewer:
Release note:

@rhoninl you can close this PR now

finished in #1001 and #1002


Single quotes prevent variable substitution in shell commands in BUILD.md

When trying to execute the following shell commands from the BUILD.md file on Ubuntu, I encountered an error because the variable substitution did not occur due to the use of single quotes:
curl -s https://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add
curl -s https://apt.kitware.com/keys/kitware-archive-latest.asc | gpg --dearmor - | sudo tee /usr/share/keyrings/kitware-archive-keyring.gpg
echo 'deb http://apt.llvm.org/$(lsb_release -cs)/ llvm-toolchain-$(lsb_release -cs)-12 main' | sudo tee /etc/apt/sources.list.d/llvm.list
echo 'deb [signed-by=/usr/share/keyrings/kitware-archive-keyring.gpg] https://apt.kitware.com/ubuntu/ $(lsb_release -cs) main' | sudo tee /etc/apt/sources.list.d/kitware.list

As you can see, the commands use single quotes to enclose the string arguments that contain variables. This prevents the shell from expanding the variables, causing the commands to fail.
To fix the issue, the single quotes should be replaced with double quotes, like so:
echo "deb http://apt.llvm.org/$(lsb_release -cs)/ llvm-toolchain-$(lsb_release -cs)-12 main" | sudo tee /etc/apt/sources.list.d/llvm.list
echo "deb [signed-by=/usr/share/keyrings/kitware-archive-keyring.gpg] https://apt.kitware.com/ubuntu/ $(lsb_release -cs) main" | sudo tee /etc/apt/sources.list.d/kitware.list

This will allow the shell to expand the variables correctly.
I suggest updating the BUILD.md file to reflect this change and avoid confusion for other users who may encounter the same issue.

https://github.com/ytsaurus/ytsaurus/pull/10


Add social links below the hero section and footer

Description
Add a section below the hero container and footer with quick navigation icons to social links:

Github: https://github.com/Redot-Engine/redot-engine
Discord: https://discord.gg/redot
Twitter/X: https://x.com/redotengine
Reddit: https://www.reddit.com/r/RedotGameEngineMain/
Youtube: https://www.youtube.com/@ReDotEngine

Figma design:
URL: https://www.figma.com/design/47xbUZkEOEkDg7OBEkL5H9/ReDot-Website?node-id=2612-1499&t=HOzJzxzAfGFSYn8g-1
Hero section:

Footer:

This one I think we should ask the design guy and others and see what they think. It's good idea but I am not sure about separating it from the logo and download button.

Yeah this is in the designs. We currently don't have that as current state


add content to about.rst

Hi @bwang64, can this be merged asap so that we can check the rendered version on readthedocs? Thank you!


Bad source of random and bugs

I found these problems with the code:

Math.random is a bad source of random.
rawprng() can return NaN.
random.done() can cause use after free type bugs.
In random.random(), Number.MAX_VALUE is not what you think it is.
random(), random.range(), random.intBetween(), and random.random() are not evenly distributed.
This is clearly not a CSPRNG. You should remove the "UHEPRNG" code and replace it with Node's crypto.randomBytes and if a seed is given just do something like key = H(seed); counter = 0; for seeding, key = H(key + "-" + entropy) for adding entropy, and key = H(key + counter++); return H(key); for random data. H() being a cryptographic hash function like SHA256 or a KDF like HKDF. This will be vastly superior.

I decided to write this (https://github.com/Sc00bz/break-uheprng) to show how broken UHEPRNG is.

Thanks for taking the time to provide all this information!  I hope I didn't suggest this was a CSPRNG (as I wouldn't know enough to prove or disprove it). The repo you pointed to looks very informative.  It's concerning that I could receive a NaN at times.
I published this module years ago b/c I needed a seeded random generator in node, and I found https://www.grc.com/otg/uheprng.htm which wasn't yet published on npm (so figured I'd give people a way to easily install it).
I've only used this lib for mocking Math.random() in unit tests, and for generative art type projects.
Hopefully other people will find your link informative as well.
It might be useful someday to update the README of this project w/ a list of other seeded random number generators.

In the original UHEPRNG code it is stated as a "cryptographically strong PRNG".
Fixing the NaN problem is easy. Move the return line in mash() to the end of the function:
-			return (n >>> 0) * 2.3283064365386963e-10; // 2^-32
		} else n = 0xefc8249d;
+		return (n >>> 0) * 2.3283064365386963e-10; // 2^-32

Interesting. I did read the license when I originally published it (but had forgotten that line).  Now my concern is making changes that might break things for people who are solely relying on this for unit tests and the like.  I guess that's what semver is for though.
Again, I appreciate your time and info.
At the very least I need to update the readme with alternatives and more info about your findings.


Discover tool seems to not be discovering all values

Hi, when running the discover tool i get these two results from two of our environments (extract)
`
BIT
CoherenceClusterSystemResource:
    defaultCoherenceCluster:

        CoherenceResource:

            CoherenceClusterParams:

                ClusteringMode: unicast

                Transport: tmb

                TimeToLive: 4

                UnicastListenPort: 9991

                ClusterListenPort: 9991

                CoherenceClusterWellKnownAddress:

                    WKA1:

                        ListenAddress: GDFMW000026

                    WKA2:

                        ListenAddress: GDFMW000027

                    WKA3:

                        ListenAddress: GDFMW000028

                    WKA4:

                        ListenAddress: GDFMW000030

SPA
CoherenceClusterSystemResource:
            defaultCoherenceCluster:

                            Target: osb_server01,soa_server01,ess_server01,soa_server02,soa_server03,soa_server04,osb_server02,osb_server03,osb_server04,wsm_server01,wsm_server02,wsm_server03,wsm_server04,ess_server02,ess_server03,ess_server04

                            CoherenceResource:

                                            CoherenceClusterParams:

                                                            UnicastListenPort: 9991

                                                            ClusterListenPort: 9991

                                                            CoherenceClusterWellKnownAddress:

                                                                            WKA1:

                                                                                ListenAddress: GPFMW000007

                                                                            WKA2:

                                                                                ListenAddress: GPFMW000008

                                                                            WKA3:

                                                                                ListenAddress: GPFMW000009

                                                                            WKA4:

                                                                                ListenAddress: GPFMW000010

`
The values such as
ClusteringMode: unicast
Transport: tmb
TimeToLive: 4
are configured the same in both, but dont seem to be captured in the resulting SPA model.

Please provide the config.xml from both of the domains.

bit_config.txt
spa_config.txt
Ive renamed the files to txt so github will accept them.

@WhatDoor discoverDomain does not write attributes whose values are "not set" (e.g., not set to a value that is different than the default value).  It appears that you were using discoverDomain in online mode where the default value for ClusteringMode is unicast:
https://github.com/oracle/weblogic-deploy-tooling/blob/main/core/src/main/resources/oracle/weblogic/deploy/aliases/category_modules/CoherenceClusterSystemResource.json#L176

I see, thanks for that clarification Robert.
So I ran the discovery tool on BIT in offline mode, where the default is blank resulting in those values being written. And then I ran the discovery tool in SPA in online mode, where the values matched the default.
This results in the output I'm seeing. Is my understanding correct?

Hi,
I just ran the discovery again on the SPA environment in offline mode and it's still not showing those values.
Here is the command I ran
/u01/app/oracle/product/fmw1221//oracle_common/common/bin/wlst.sh /home/oracle/weblogic-deploy/lib/python/discover.py -oracle_home /u01/app/oracle/product/fmw1221/ -domain_home /u01/app/oracle/admin/ent_soa_spa_domain/aserver/ent_soa_spa_domain/ -archive_file /home/oracle/discoveredspa.zip -model_file /home/oracle/discoverspa.yaml -variable_file /home/oracle/discoveredspa.props

Could you zip up and provide the config directory for spa

Can you specify the files you want to see? So I can check them before sharing them.

@WhatDoor yes, your understanding is correct.  It is very unlikely that discoverDomain is not working in terms of which attributes are being written to the model.  What version of WDT are you using?
Try reading the domain with WLST offline and see what values those attributes have?

@WhatDoor when trying to replicate issues, it is not uncommon for us to ask for the entire $DOMAIN_HOME/config directory so that we can simulate your domain.  You should feel free to ask to send it directly to the person asking rather than posting it but rest assured, we treat your data with the utmost consideration for maintaining privacy and, without the domain encryption keys, we cannot read encrypted database passwords and such.
I am still unsure of the purpose of this issue.  Is there something you are trying to accomplish that isn't working or is this just you trying to understand the behavior of the tool?

<EMAIL_ADDRESS>

Offline discovered model:
resources:
    CoherenceClusterSystemResource:
        Coherence-0:
            CoherenceResource:
                CoherenceClusterParams:
                    ClusteringMode: unicast
                    Transport: udp
                    TimeToLive: 4
                    ClusterListenPort: 7574

online discovered model:
esources:
    CoherenceClusterSystemResource:
        Coherence-0:
            Target: Cluster-0-1,Cluster-0-2
            CoherenceResource:
                CoherenceClusterParams:
                    ClusterListenPort: 7574

I am unable to reproduce your issue.  Closing as works as designed.  Feel free to reopen the issue if you have a reproducible test case.