refactor(typegen): path parameters required in output

This PR omits the optional on parameters if path parameters are used. This can be enabled using the CLI options disableOptionalPathParameters.
The reason for this PR is that path parameters are always required and it doesn't make sense that they should be optional.

I think this makes sense as a default option 👍


Can locales customize grouping strategy min2?

Currently, the spec text says:

[[UseGrouping]] is a Boolean or String value indicating whether the conditions under which a grouping separator should be used. The positions of grouping separators, and whether to display grouping separators for a given number, is implementation-defined. A value ["min2"] hints the implementation to display grouping separators ... if there are at least 2 digits in a group

But how about when locales want to restrict the grouping separators further?  For example, there are some locales whose "auto" grouping starts at 3 or 4 digits, or prefer not to use grouping at all.  Should those locales' behavior take precedence over "min2", or should "min2" apply the same behavior across all locales, more like "always" ?

Discussion 2021-01-13: https://github.com/tc39/ecma402/blob/master/meetings/notes-2022-01-13.md#can-locales-customize-grouping-strategy-min2-77
Conclusion: Close this issue.


Remove phpdocumentor specific styles.

@jaapio Please transfer them to your theme

I think this is a good move, but I keep this PR open for now, as I need to do some work on phpDocumentor core to get things working on that side.
Thanks!

merged manually. Thanks!


Configure synapse (1),timeseriesinsights (5),traffic (3),vpn (1),web (20)

External name configuration of 1 resources in the synapse group:

[ ] azurerm_synapse_workspace_vulnerability_assessment

External name configuration of 5 resources in the timeseriesinsights group:

[ ] azurerm_iot_time_series_insights_access_policy
[ ] azurerm_iot_time_series_insights_event_source_iothub
[ ] azurerm_iot_time_series_insights_gen2_environment
[ ] azurerm_iot_time_series_insights_reference_data_set
[ ] azurerm_iot_time_series_insights_standard_environment

External name configuration of 3 resources in the traffic group:

[ ] azurerm_traffic_manager_azure_endpoint
[ ] azurerm_traffic_manager_external_endpoint
[ ] azurerm_traffic_manager_nested_endpoint

External name configuration of 1 resources in the vpn group:

[ ] azurerm_vpn_gateway_nat_rule

External name configuration of 20 resources in the web group:

[ ] azurerm_app_service
[ ] azurerm_app_service_active_slot
[ ] azurerm_app_service_certificate
[ ] azurerm_app_service_certificate_binding
[ ] azurerm_app_service_custom_hostname_binding
[ ] azurerm_app_service_environment
[ ] azurerm_app_service_environment_v3
[ ] azurerm_app_service_hybrid_connection
[ ] azurerm_app_service_managed_certificate
[ ] azurerm_app_service_plan
[ ] azurerm_app_service_slot
[ ] azurerm_app_service_slot_virtual_network_swift_connection
[ ] azurerm_app_service_source_control
[ ] azurerm_app_service_source_control_token
[ ] azurerm_app_service_virtual_network_swift_connection
[ ] azurerm_function_app
[ ] azurerm_function_app_slot
[ ] azurerm_linux_web_app
[ ] azurerm_service_plan
[ ] azurerm_source_control_token

Ticket #35 combined with #48


Documentation for Appium support

Thank you for such a great too!
From one of the blog article  I read from your team, I would like to double check if there's any Appium support to allow parallel test with flank.
If not wondering do you have any alternative suggestions on how your team automatically run your Appium tests on real device at the moment?  Thank you!

hi, Firebase Test Lab doesn't support appium tests. Flank is a Firebase Test Lab runner, that means we're limited to what Firebase supports.
If Firebase adds Appium support then Flank will work with appium. I don't think that'll happen anytime soon though. Your best bet with appium is a provider such as bitbar, sauce labs test object, xamarin device farm, or similar.
In general I don't recommend appium as an automation technology. The native frameworks (Espresso, EarlGrey, XCUITest) are a better investment for performance and productivity.

hi, Firebase Test Lab doesn't support appium tests. Flank is a Firebase Test Lab runner, that means we're limited to what Firebase supports.

That makes sense!  That's what I thought also and wondering if there's an alternative alluded to by the blog article.

In general I don't recommend appium as an automation technology. The native frameworks (Espresso, EarlGrey, XCUITest) are a better investment for performance and productivity.

Thank you for the frank suggestion.  I would agree for performance wise native frameworks is way better.  As the article suggests there's a factor of reusability, and maintainability that we are exploring also.
Feel free to close this issue - thank you!
p.s. We are at the stage of exploring Appium as a potential candidate so if you have any references as to horror stories of Appium uses would love to hear!  Thank you.

I've spent some time exploring appium https://github.com/appium/appium/graphs/contributors
If you want developer involvement and the apps are native, then native is the way to go. If you have a react native app then detox will be a better fit. I'm not sure there's a great use case for appium today, maybe hybrid apps with webviews.

I couldn't find it explicitly mentioned anywhere, but @bootstraponline regarding

If you have a react native app then detox will be a better fit

Does detox currently work with Flank on Firebase test lab, even if only virtual devices?


Support for Custom Http Client

Hi @stanleynguyen i concern with your comment on https://github.com/xendit/xendit-php/issues/67#issuecomment-618150627
But why guzzle on composer.json strict to version 6?
I have other package that require guzzlehttp/guzzle version 7.
I can't install this package. Please Help. :pray:

Hi @rizalfauzie, we will consider upgrading our GuzzleClient to 7.
In the meantime, you can install guzzle 7 in your project and swap the http client with https://github.com/xendit/xendit-php#use-custom-http-client

i can't @stanleynguyen
  Problem 1
    - xendit/xendit-php 2.5.0 requires guzzlehttp/guzzle ~6.0 -> satisfiable by guzzlehttp/guzzle[6.5.x-dev].
    - xendit/xendit-php 2.5.1 requires guzzlehttp/guzzle ~6.0 -> satisfiable by guzzlehttp/guzzle[6.5.x-dev].
    - Can only install one of: guzzlehttp/guzzle[7.2.0, 6.5.x-dev].
    - Installation request for guzzlehttp/guzzle ^7.2 -> satisfiable by guzzlehttp/guzzle[7.2.0].
    - Installation request for xendit/xendit-php ^2.5 -> satisfiable by xendit/xendit-php[2.5.0, 2.5.1].

Why not just add version ^6.0|^7.0 on composer.json?
is there any backward compatibility within this package?

i can't @stanleynguyen
  Problem 1
    - xendit/xendit-php 2.5.0 requires guzzlehttp/guzzle ~6.0 -> satisfiable by guzzlehttp/guzzle[6.5.x-dev].
    - xendit/xendit-php 2.5.1 requires guzzlehttp/guzzle ~6.0 -> satisfiable by guzzlehttp/guzzle[6.5.x-dev].
    - Can only install one of: guzzlehttp/guzzle[7.2.0, 6.5.x-dev].
    - Installation request for guzzlehttp/guzzle ^7.2 -> satisfiable by guzzlehttp/guzzle[7.2.0].
    - Installation request for xendit/xendit-php ^2.5 -> satisfiable by xendit/xendit-php[2.5.0, 2.5.1].

Why not just add version ^6.0|^7.0 on composer.json?
is there any backward compatibility within this package?

Oh interesting. I didn't know the composer dependency model is this strict. Lemme put in a quick fix

Hey @rizalfauzie please try again with https://packagist.org/packages/xendit/xendit-php#2.5.2
I'm repurposing this issue as a request to update guzzle-client

thank you, it work.


Setup prettier github action

Probably this one?
https://github.com/marketplace/actions/prettier-action

Seems like bots having friendly fire https://github.com/kodadot/nft-gallery/pull/1881#issuecomment-1014517882
Turning off from deep source Prettier and keep watching https://github.com/kodadot/nft-gallery/commit/19f73ecc92436ce6a2ce505d8f04413e4c867402

we already have something on Prettie stuff iirc


UDF container crashing won't restart the pod causing the pipeline to stall

If the UDF container crashes, the pod continues to be in a running state. This is an irrecoverable state where the main processes keep trying to reach UDF, and it never succeeds. If the UDF container crashes, either we have to restart the UDF container or restart the pod.

@whynowy, we might have to prioritize this too.


New command: Restore dependencies for a PowerApps Component Framework solution project

Restores the dependencies of a PowerApps component framework solution
pa solution restore
Equivalent to the PowerApps CLI command.

Changed the verb to restore which better reflects the command's purpose

Your change didn't save, I think, because I still see my original submission.

Good catch. Thanks!

Closing due to lack of activity


chore(main): release dotbot-helm 0.5.0

:robot: I have created a release beep boop
0.5.0 (2024-01-28)
Features

Added some new feature (test) (4931aa1)
bump chart.yaml minor version for testing (2a291cd)
refactored CI, lets see how it goes (5cfeb41)
test minor semver increment on helm changes (39ca903)

Bug Fixes

another helm bug (test) (65cf9c4)
bump minor version for testing (497f222)
digest parsing in Helm chart (95999ec)
release-please-config instead of yaml config because of bug in GH action (2695528)
release-pr only runs on merges to main (4f0db71)
semantic plugin (4a33888)
test helm change (1cc75b9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Sillock-Inc/Dotbot/releases/tag/dotbot-helm-v0.5.0 :sunflower:


⚠ Bofrost.be has degraded performance

In 6d91864, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 962 ms

Resolved: Bofrost.be performance has improved in 10a7155.


Responsive

👷 Deploy Preview for effortless-choux-ed3c7f processing.

Name
Link

🔨 Latest commit
c805bdacb20938a8be687fdb12c507c30241103a

🔍 Latest deploy log
https://app.netlify.com/sites/effortless-choux-ed3c7f/deploys/66393c2691914f0008085b2d


Lua doTweenX help!!!

What is your question?
I tried to make an event in Lua that made a timer run then moved a sprite and faded it out, but only the fade worked as intended. The tweenX worked as soon as the event started, but the tweenAlpha worked after the timer was over, yet, they were in the SAME GOD DAMN FUNCTION. WHAT THE HELL!!!! Doctor! Help Me!!!

Where is the code

oh shit sorry
`function onCreate()
makeAnimatedLuaSprite('uwu', 'SUNKEEEEHHHHHHHHH/uwu', 315, 150)
addAnimationByPrefix('uwu', 'uwu', 'uwu', 24, false)
setObjectCamera('uwu', 'camHUD')
scaleObject('uwu', 0.75, 0.75)
setProperty('uwu.alpha', 1)
end
function onEvent(name, value1, value2)
if name == "uwu" then
runTimer('fadeOut', 5)
addLuaSprite('uwu', true)
doTweenAlpha('uwu_tweenAlpha', 'uwu', 0.5, 2.5, 'linear')
end
end
function onTimerCompleted(fadeOut)
doTweenAlpha('uwu_tweenAlpha2', 'uwu', 0, 1.5, 'linear')
doTweenX('uwu_tweenX', 'uwu', 415, 5, 'linear')
end`

It worked for about like 30 minutes but then it broke after editing the makeAnimatedLuaSprite x and y values

The only issue I see is that any timer will trigger the tweens, try adding a tag check

Thanks!


Suggestion: Ability to import videos directly into projects

you would be able to import them like how you can import sounds on the sounds tab, I was gonna save the videos locally to the project as a data url but thats wayyy too laggy

may fly

i want this it would be cool

oh my peak


e2e: Fix a permissions error

9a36b19 e2e: Fix a permissions error
commit 9a36b19a00e2ea74405c2a3b5610ff65a7438dc0
Author: Russell Bryant<EMAIL_ADDRESS>Date:   Thu Jun 27 18:50:32 2024 -0400
e2e: Fix a permissions error

This line giving some system config debug output used to work fine,
but started failing on a permissions error today. Slap some `sudo` on
it to get things running again.

Signed-off-by: Russell Bryant<EMAIL_ADDRESS>

This is super trivial and fixes a broken CI job that runs on multiple repos, so I'm just going to merge it.


License

I've decided to ship this project with the GPL3 license to start. The closest project to this is GNU Make, which is licensed with this same license and as far as I'm aware that's never been too much of a restriction for that project.
The extensions will remain MIT licensed though.
This license is quite rare in the JS ecosystem currently, so if there is interest in the project there may well be questions about this.
Creating this issue to handle any feedback. The licensing decision can be changed at any point in future. By starting with MIT the converse isn't true though.

Closing as there is currently no action-item, but discussion can be picked up at any time.


More time-efficient order in timeslice file creation

Changed the order of timeslice file import and creation of dataframes. Previously, a pandas dataframe was created for each timeslice, and then the dataframes were concatenated into a combined dataframe for 15-minute resampling. Now, only numpy arrays are imported, which are then combined into one large dataframe.
Additions

Removals

Changes

For reading individual timeslices, added a routine _read_timeslice_file_numpy in nhd_io that only returns numpy arrays
Rewrote get_obs_from_timeslices: streamlined creation of the observation_df in one step

Testing

Running any example that reads usgs timeslice files
If you want to benchmark the time, the pyinstruments profiler is suggested, which will create a report with 1 millisecond time resolution:

from pyinstrument import Profiler
profiler = Profiler()
profiler.start()
 [CODE YOU WANT TO BENCHMARK]

profiler.stop()
profiler.print()
Screenshots
For Lower Colorado example [test_AnA.yaml]: benchmark for get_obs_from_timeslices shows 2-fold speedup:
BEFORE:

AFTER:

Notes

Todos

Suggest considering to delete the "quality" dataframe, which will result in further speedup

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows project standards (link if applicable)
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Target Environment support

[ ] Windows
[ ] Linux
[ ] Browser

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

I test it on an example I had on vpu-17 and on this line timeslice_obs_df  = pd.concat(dfList, axis = 1)
It gives error that *** pandas.errors.InvalidIndexError: Reindexing only valid with uniquely valued Index objects
Probably because you try to concatenate dataframes that have non-unique index values. Also when I checked there was a difference in length between dfList[0] and dfList[1]  and others.

Archived. End of Project.


Remove need for version.txt during release

Motivation
Last release failed because #4637 failed to remove all usages of version.txt. This PR fixes the issue.
Changes

release job was still parsing version.txt to assign file names
Makefile was still parsing `version.txt

Test Plan
Test release?
TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[x] Update documentation as needed

DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

bors merge

bors merge

bors merge

Some unrelated tests known to be flaky have caused the last to attempts to merge to fail.


Golem Harvest Seals don't work well with pam's harvestcraft plants

These plants aren't designed to be broken, as a matter of fact most plants aren't designed to be broken anymore.  I think the harvest golem would work much better if it right clicked on the plants instead of breaking them.  Maybe an option for an advanced harvest seal?
In any case, when mature plants are broken they don't usually drop seeds they drop fruit.  You can convert some of the fruit into seeds.

Maybe an option for an advanced harvest seal?

I can't second this enough; in TC5 I couldn't automate a Harvestcraft farm due to this issue either.
It's not just limited to Harvestcraft either: Simply Corn also has the option to harvest without breaking the crop in its config file, and drops corn (not seeds). Natura also has barley and cotton among others, which are right-click-harvestable.
Seems to be a pretty standard mechanic in modded MC these days.
I suppose golems can use an autocrafting table to make seeds from the various crops if they're broken, since those are usually a 1:1 recipe. But that would afterwards require a seal that's able to tell the golems to plant a specific crop in a specific location (planting seal?) In TC5 I couldn't get golems to plant crops where I wanted them and in the number I needed, they just planted randomly it seems.

They are also having trouble with agricraft seeds and crop sticks, breaking them even when the plant hasn't matured. And some they don't even recognize, like watermelon seeds in cropsticks.

In my api I have a IMC function that allows mod authors to set their crops to be click-harvestable by golems. Pam's used to have that, but I guess they might have removed it at some point. Maybe go remind them that it is a thing :)


Adding foo.js to end of a url causes the controller to be exposed

Noticed this issue on a number of sites that are running silverstripe 3+
If you go to any subpage from the top level and add /foo.js (or anything else with 'css', 'js', 'jpg', 'jpeg', 'gif', 'png', 'bmp', 'ico' extension) on it, it doesn't return a 404, it returns the controller and an error message.
The source of this problem is in the class 'HTTPRequest' class:
public function isMedia() {
   return in_array($this->getExtension(), array('css', 'js', 'jpg', 'jpeg', 'gif', 'png', 'bmp', 'ico'));
}

Not sure what the best fix is here since yes it's good for images, but we shouldn't been exposing the controller names.
I can give some example urls if needed.

This was a deliberate design decision, although I can appreciate that maybe this wasn't the behaviour that you expected / desired.
The reason that we introduced this functionality was for performance. Typically, URLs with the extensions defined aren't opened directly by the user, but are referenced as dependencies of a page. You'll sometimes get a situation where a broken reference to a CSS file will cause a 404 on every page view.
We wanted to ensure that responding to these 404s was both low-bandwidth and low-processing-time. So, rather than send a full friendly 404 page, we've opted to send a much more abridged view.
A few thoughts:

Not everyone wants this behaviour, and so maybe it should be configurable. I still think th
Right now the 404 message will often be some gobbledegook rather than an abridged, friendly message explaining that the page isn't available. Something like this might be better (the text could be amended with a config option):Content-type: text/plain

Sorry, this page cannot be found.

I would suggest resolving this by adding a couple of config options:

ContentController.friendly_error_for_media_requests: Default to false, if set to true then the rendered error page should be shown even for media requests.
ContentController.error_message_for_media_requests: If friendly_error_for_media_requests is false then this map of messages will used to send plain text responses. A map of HTTP status code => message.

See my comment on https://github.com/silverstripe/silverstripe-cms/pull/1091

@sminnee I don't think there is a problem with the friendly/lower bandwidth messages, I would agree with the design here.
The problem is exposing the controller name, any identifying information and not sending a HTTP 404 status header.
Potentially there should be another asset .html file like what's currently used for standard 404 pages.

I think that in this case, even when the information is being exposed, the response code is till 404 correct?

It returns a 200 status code currently, e.g.
http://www.silverstripe.org/software/test.png

@ColmMcBarron I'm getting 404

@tractorcow ok, weird not sure where I was seeing the 200, retrying and you are correct getting a 404 status code.

Current message I get is Action 'foo' isn't available on this handler. which, whilst unfriendly, doesn't expose information about the actual controller...

From memory, we no longer expose controller name outside of dev mode. The initial issue raised is resolved, even though the behaviour may still be less than desirable.


add a Dockerfile for building a hosting tunneler image for Red Hat Linux

This PR makes me realize that I'm not clear on what all of the docker files are in the /docker directory. Is there a place for some descriptions in the readme?

Good point. I'll re-write the Docker README to cover both Dockerfiles and their respective container images.

@scareything Here's the rendered view of the updated Docker README in this PR's branch: https://github.com/openziti/ziti-tunnel-sdk-c/tree/issue-512-docker-build-redhat-ubi/docker#readme


Add support for adding an alias to multiple commands with a single directive

Currently, it's only possible to set aliases for a single command with a single directive. In cases where we're adding aliases for backwards compatibility that follow the same pattern, this task becomes repetitive.
For example, when generating KeyVault cmdlets, the cmdlets in Az.KeyVault are called *-*ManagedStorageAccount and the cmdlets generated are called *-*StorageAccount. Rather than writing a directive per command, it would be nice to do something like the following:
- where:
      subject: StorageAccount
  set:
      alias: ${verb}-${prefix}${subject-prefix}Managed${subject}

This would also help in other cases, such as DataLakeStore, where they have an alias to *-Adls* for each of their cmdlets (to replace *-AzDataLakeStore*). This alias could then be easily set across all commands with the following directive:
- where:
      verb: .*
  set:
      alias: ${verb}-Adls${subject}

Fixed


Use :latest tag for yq image consistently

Previously, some of the quay.io/konflux-ci/yq pullspecs included the tag and some did not. This made Renovate open two PRs to update yq instead of one.

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

Trying to convince Renovate that #1312 and #1311 don't need to be two PRs


Standardize npm

Coverage remained the same at 100.0% when pulling 206835504b74ea213ddace1b5d0f8a70dbf0ad57 on standardize_npm into 8929f5531db49f7364de7a5e3f9bb8dabee8896e on master.

@jthomerson I'm pulling this PR back. Will update once it is ready for review again.


Draft: Fix supported characters in Tags

Resolves https://github.com/cucumber-rs/cucumber/issues/174
Synopsis
Despite nowhere to be documented, it looks like . can be present in Tags.
Solution
Support any non-whitespace character, excluding @ and ), as seen here.
Checklist

Created PR:

[x] In draft mode
[x] Name contains Draft:  prefix
[x] Name contains issue reference
[x] Has assignee

[x] Documentation is updated (if required)
[x] Tests are updated (if required)
[x] Changes conform code style
[ ] CHANGELOG entry is added (if required)
[ ] FCM (final commit message) is posted

[ ] and approved

[ ] Review is completed and changes are approved
Before merge:

[ ] Milestone is set
[ ] PR's name and description are correct and up-to-date
[ ] Draft:  prefix is removed
[ ] All temporary labels are removed

FCM
Fix supported characters in `Tag`s

- fix allowed keywords in `Feature`s `Description`
- allow comments on the same line with `Tag`s


Images converted have major yellow boost

I am working on converting batches of cr2 files to jpeg and in using the example code in the rawpy documentation I have found that the converted jpeg images are converted with a lot more yellow in the image than the original raw files.
path = 'images/test_image.CR2' with rawpy.imread(path) as raw:     rgb = raw.postprocess() imageio.imsave('converted/test_image.jpeg', rgb)

Maybe the whitebalance is not picked or recorded properly. You can manually set your desired white balance, just like you can do with any RAW postprocessor. Note that rawpy is just a thin wrapper around libraw. I suggest you verify your issue with another software based on libraw, and if you see the same then you could create a thread in the libraw discussion forums or on the photo community on stackexchange.

@aaronleopold, we have experienced the same issue with CR2 images captured on an EOS 5d Mark III. For now our solution was to use user user_wb=[1, 0.5, 1, 0] upon image open. Hope this helps!


How to infer useScopedI18n type?

I made type like this below, the question is, am i correct?

then I got the correct types:

but if I scoped other than yup it does not give me type error

here is my en.ts file:

Could you share a reproduction with the code?

Closing as stale.


Mojave 10.14.6 VM on BigSur host - Initial run hangs on "Start RandomSeed"

To Reproduce
Steps to reproduce the behavior:

Install Virtual Box 6.1.22 r144080
Run make download, download Mojave 10.14.6.
Copy Installer app to /Applications
Run make all
Observe Virtual Box VM screen

Expected behavior
Installer starts
Additional context

log file that is accessible via Console.app - see runMacOSVirtualbox.log
screen shot

I found a solution:
Disable "Enable Nested Paging". You may first want to import the VM in the virtualbox GUI.

Thanks! Hope the change fixed the issue.


Log identify calls

Follow up to https://github.com/artsy/force/pull/6006, uses window.analytics over analytics for identify events, and adds identify to new logging.
Related to https://artsyproduct.atlassian.net/browse/PLATFORM-2282

Merging this because it is blocking to force deploys


Webpack error; Module parse failed: Unexpected token

Hi
I just installed Valibot in my Nuxt 2 project (vue 2), node 18 - and i'm getting a webpack error .
This is most likely my setup - but i'm a little lost in what the solution could be?
Any advice would be appreciated.
ERROR in /home/node/app/node_modules/valibot/dist/index.js 31:8
Module parse failed: Unexpected token (31:8)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
| // src/error/ValiError/ValiError.ts
| var ValiError = class extends Error {
>   issues;
|   /**
|    * Creates a Valibot error with useful information.

What is your current version of Valibot? Did the problem also occur in previous versions? At the moment it looks like a webpack problem. Please let us know if you find a solution. Maybe the problem is also partly related to #240.

First time installing - v.0.20.

If the info in #240 does not help you, I suggest you contact Webpack as this seems to be unrelated to Valibot. I am sorry that I cannot provide you with a proper solution.

Thanks for quick response -
For others, my solution was to transpile valibot in my nuxt.config.
{
  build: {
    ...
    transpile: ['valibot'],
    ...
  }
}

Thank you for sharing!


Revert "[RIP-15]Add Ipv6 support for RocketMQ"

Reverts apache/rocketmq#1352

Coverage decreased (-0.8%) to 50.564% when pulling eca166a5f7a5dd62998516c140f621a0cc8e312f on revert-1352-IPV6 into c7c9601d36d0179ae69592e541764b04130cf790 on IPV6.


Web USB api

Arduino

firefox and safari are not supported.

It may be slow, this feature seems omake


join operator in Azure Resource Graph queries

I am not able to use leftanti or leftsemi join operators in Azure Resource Graph queries. I get below error message.
Please provide below info when asking for support: timestamp = 2020-01-02T19:24:29.6405799Z, correlationId = 7031f89e-aa3f-4301-a93a-ff989008fc42. (Code: BadRequest) Query is invalid. Please refer to the documentation for the Azure Resource Graph service and fix the error before retrying. (Code: InvalidQuery) The join kind "LeftSemi" is not supported or not allowed. (Code: UnsupportedJoinFlavor)

@kavi99506 Thank you for your feedback. Since this channel is more for driving improvements towards MS Docs, could you provide us with a URL of the documentation that your feedback is most applicable to? That way, we're able to locate the right team that's better suited to address your feedback.

@mike-urnun-msft Here is the link for the documentation - https://docs.microsoft.com/en-us/azure/governance/resource-graph/samples/advanced?tabs=azure-cli#join

Thanks for providing additional info, @kavi99506! I have assigned an engineer who can help with the next steps.

#reassign:DCtheGeek

@kavi99506 Thanks for the question! While leftsemi and leftanti are part of KQL for the join operator, they aren't supported by Resource Graph. Only innerunique, inner, and leftouter are supported. You can see the full listed of KQL language elements supported by Resource Graph here: https://docs.microsoft.com/azure/governance/resource-graph/concepts/query-language#supported-kql-language-elements
If you have a business case for needing that flavor, please post it to the UserVoice. Thanks for the question!
@MicrosoftDocs/azure-cxp-triage #please-close


test: stabilize test_detect_deadlock_when_shuffle_region

Signed-off-by: youjiali1995<EMAIL_ADDRESS>
What have you changed?
Stabilize test_detect_deadlock_when_shuffle_region.
What is the type of the changes?

Bugfix (a change which fixes an issue)

How is the PR tested?

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)
Fix https://github.com/tikv/tikv/issues/6153

/test

/test

/test

/test

/test

/test

/test

/test

/test

/test

/test

/test

/test

/test

/test

/test

/test

/test

@MyonKeminta @sticnarf  PTAL

/test


Suggestion: Color names instead of danger, warning, success, info

For semanthics, my suggestion is to replace success, info, danger and warning for their respective colors.
For example, if Facebook used Bootstrap, it would use "label-danger" to display it's notifications, where it is not trully a danger. "label-red" would be better in that case.
So, instead of success, info, danger and warning, we would have green, blue, red and orange.
label-blue instead of label-info
label-orange instead of label-warning
label-red instead of label-danger
label-green instead of label-success
And so on...

No!
Here is some code from bootstrap's sass files
// Start with assigning color names to specific hex values.
$white:  #fff !default;
$black:  #000 !default;
$red:    #d9534f !default;
$orange: #f0ad4e !default;
$yellow: #ffd500 !default;
$green:  #5cb85c !default;
$blue:   #0275d8 !default;
$teal:   #5bc0de !default;
$pink:   #ff5b77 !default;
$purple: #613d7c !default;

// Create grayscale
$gray-dark:                 #292b2c !default;
$gray:                      #464a4c !default;
$gray-light:                #636c72 !default;
$gray-lighter:              #eceeef !default;
$gray-lightest:             #f7f7f9 !default;

// Reassign color vars to semantic color scheme
$brand-primary:             $blue !default;
$brand-success:             $green !default;
$brand-info:                $teal !default;
$brand-warning:             $orange !default;
$brand-danger:              $red !default;
$brand-inverse:             $gray-dark !default;

Hope you understand now that why its like this and not the vice versa
Rest on @mdo to decide!!

for eg,
You make a website in blue color as primary color .... now your boss says to change it in red.
Now what would be the best option of the two ? .....

$brand-primary

$red:    #d9534f !default;
$blue:   #0275d8 !default

$brand-primary: $red

$blue

$blue:  #d9534f !default;

#d9534f is a red color and $blue is surely not the best answer for it

In a similar way to how <strong> is prefered over <b>, users should use the .label-* that conveys the correct semantic meaning and override the corresponding $brand-* Sass variable. It is an unfortunate reality that some people are going to simply import bootstrap.min.css without modifications and use .label-danger incorrectly as you suggest, but we cannot really cater to these users. I'm closing this as a WontFix. Feel free to ask any further questions.

@bkdotcom i dont think ..... the author of this issue deserved a thumbs down ..... its normal for folks to not understand such semantics, specifically as a beginner level
But it may be just me

Also you can find Bootstrap 4 colors palette and export to SCSS/LESS variables file
https://colorswall.com/palette/3


Re-implement Arbitrary and rountrip RLP tests for PooledTransactionsElement

We had a bug in RLP encoding that would have been caught by roundtrip tests:
https://github.com/paradigmxyz/reth/issues/4961#issuecomment-1754350320
We should reimplement these as the TODO states:
https://github.com/paradigmxyz/reth/blob/1cccd097ebe460ec989719a9fde0834a15313cdd/crates/primitives/src/transaction/pooled.rs#L14

I'd consider this done with #4972 and #5125


Add offset to LinearKernel

What's the new feature?
The matrix cookbook includes an offset parameter for the linear kernel.
What value does this add?
Additional flexibility.
Is there an alternative you've considered?
No response
Additional context
No response

Not actually feasible as the offset parameter will depend on the dimensionality of the inputs, can be done manually by users anyway


Failed creating a mirror pod...connection refused

After installing using helm I get:
Failed creating a mirror pod
`

ip-172-20-114-144 /home/core # journalctl -efx
Failed creating a mirror pod for "kube-proxy-ip-172-20-114-144.eu-west-2.compute.internal_kube-system(be626a0cf52acb592e944834fa66ec21)": Internal error occurred: failed calling webhook "mutate.kyverno.svc-fail": Post "https://kyverno-svc.kyverno.svc:443/mutate/fail?timeout=10s": dial tcp <IP_ADDRESS>:443: connect: connection refused`

I am using kops v1.19.3 --> unsupported.  Do you possibly have any ideas to help?
Many Thanks

This project doesn't use Helm. Do you mean you installed Kyverno using Helm? If so, you might want to create an issue over in the Kyverno repo.
I don't see anything wrong specific to this project, so I'm going to close this. If there's more information that is specific to the quickfix, please re-open and post.


Add Purdue Global University

Please add Purdue Global University to list

@25GradPurdue Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Consider otel span to be a transaction of type=message when some message attribute is set

TranslateTransaction only considers events that have message_bus.destination or messaging.destination set to be a transaction of type messaging. However, messaging.system is a required field according to https://opentelemetry.io/docs/specs/semconv/messaging/messaging-spans/ and should also be considered for deciding the transactions type.
See https://github.com/elastic/apm-data/blob/main/input/otlp/traces.go#L411-L414

In TranslateSpan, we are currently storing messageSystem in event.Span.Subtype and messageOperation in event.Span.Action.
In TranslateTransaction, messaging.system and messaging.operation are currently set as labels under default in the switch statement.

Question:
We don't have subtype and action in Transaction. Should we still keep the info in a label, after using it to determine transaction type? In that case the change will look like
diff --git a/input/otlp/traces.go b/input/otlp/traces.go
index 94072c7..8b8da65 100644
--- a/input/otlp/traces.go
+++ b/input/otlp/traces.go
@@ -413,6 +413,12 @@ func TranslateTransaction(
                        case "message_bus.destination", semconv.AttributeMessagingDestination:
                                message.QueueName = stringval
                                isMessaging = true
+                       case semconv.AttributeMessagingSystem:
+                               isMessaging = true
+                               modelpb.Labels(event.Labels).Set(k, stringval)
+                       case semconv.AttributeMessagingOperation:
+                               isMessaging = true
+                               modelpb.Labels(event.Labels).Set(k, stringval)

                        // rpc.*
                        //

Yes let's keep the information set as labels. Eventually the information should be mapped to the messaging.* field, but that would be a larger undertaking as the current apm model for message differs quite a bit from otel semantic conventions for messaging, and the apm model doesn't have attributes for system and operations.

Closes via #204


Add myself to signing members list

@torgo would like me to act as his alternate for steering committee meetings.


Website isn't responsive

The website isn't responsive for mobile devices.

Do u want to work on it? i can assign that to u!

yes, why not

Don't forgot to add the Screenshots once done!

Review #18

Hi @KiranPolaki , please review the pull request #18

Great work!


Changed DumpedOscillationAnalysis to DampedOscillationAnalysis

I have changed DumpedOscillationAnalysis to DampedOscillationAnalysis as per Issue #823

Thanks, seems like you are doing it correctly. Approved with a minor comment. Could you please double check if you can still load a deprecated class with deprecation warning?

Is it coming from your end?

I double checked the code on my end. It works fine. However the branch has some syntax errors and I cannot merge. I created a chained PR to your PR branch. Feel free to merge it, or ignore it and do the same by yourself. From next time, you can check this with command line tox -eblack and tox -elint-incr.

I double checked the code on my end. It works fine. However the branch has some syntax errors and I cannot merge. I created a chained PR to your PR branch. Feel free to merge it, or ignore it and do the same by yourself. From next time, you can check this with command line tox -eblack and tox -elint-incr.

Sure, I'll keep that in mind!

Could you please merge this PR to fix linter error? https://github.com/bopardikarsoham/qiskit-experiments/pull/1

Could you please merge this PR to fix linter error? bopardikarsoham#1

Merged it


Add hub locations to sidebar

https://github.com/foss4g-oceania/foss4g-sotm-oceania-2020/blob/69f6d7d5612b07272e774971ae3e9c6c50fca694/_includes/about.html#L26
List of Hubs:

Apia, Samoa
Auckland, New Zealand [/hubs/auckland/]
Bairiki, Kiribati
Cairns, Australia
Canberra, Australia [/hubs/canberra/]
Christchurch, New Zealand
Hobart, Australia
Majuro, Marshall Islands
Melbourne, Australia
Perth, Australia [/hubs/perth/]
Rarotonga, Cook Islands
Suva, Fiji

solved by #53

Awesome Jonah..we just love FOSS :)
On Thu, 1 Oct 2020 at 20:02, Jonah Sullivan<EMAIL_ADDRESS>wrote:

solved by #53
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Custom Fragment

Hi,
is there any way how to add custom fragment (for example EPG) when I select something in left navigation menu ?
Thx for advice.

I believe you should be able to override the click listener in your custom header fragment to do a behavior on the selected ViewHolder.

Hello Friends, I have custom fragment inside recyclerview with customized button and other controller.
I have focus issue if im trying to press down key focus is working but if im trying to press up key focus not move to top. Please help me for the same. I have extend (fragment) with custom view.

This sample has been migrated to a new location and updates are happening there (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


chore: remove a couple extra quotes

Description of the change
Trivial removal of a couple stray quotes
Existing or Associated Issue(s)
N/A
Additional Information
N/A
Checklist
Probably not needed - ship with something else and more meaningful.

[ ] Chart version bumped in Chart.yaml according to semver.
[ ] Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
[ ] JSON Schema generated.
[ ] List tests pass for Chart using the Chart Testing tool and the ct lint command.

nice spot


Cached project UUID in localStorage breaks /pipeline route

When a user removes a project from a different browser client the original browser client can have a selected project UUID that no longer exists. This leads to errors, for example:
When you visit the application route: /pipeline it loads the first pipeline it finds in the currently selected project. It uses the selected project UUID based on localStorage to find this, which can contain a project UUID that no longer exists.
There are multiple ways to fix this issue, preferably we can use a simple and performant check on a selected project in localStorage to verify whether it exists and handle non-existence gracefully/sensibly.

Fixed in #349


Replace ButtonBar.bar method with ButtonBarTheme

Description
This PR implements a proposed change to deprecate ButtonBar.bar and replace it with a new ButtonBarTheme to configure ButtonBar widgets.
Summary of changes:

Adds new ButtonBarTheme inherited widget and ButtonBarThemeData configuration class.
Adds new ThemeData.buttonBarTheme field that can be used to set the defaults for all ButtonBar widgets in the app.
Updates ButtonBar to use the surrounding ButtonBarTheme to configure itself and override the surrounding ButtonTheme for buttons contained within itself.
Changes the default ButtonTextTheme to primary for buttons in an ButtonBar to match the Material Design specification.
Deprecates ButtonTheme.bar.

These will allow us to fix an accessibility problem with button text as well as providing a more flexible way to configure ButtonBar widgets across the entire application (including internal usages like Dialog).
Related Issues
Fixes: #37333
Tests
I added unit tests for everything in the new ButtonBarTheme classes, and updated the unit tests for ButtonBar.
Checklist

[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 updated/added relevant documentation (doc comments with ///).
[x] 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
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[x] Yes, this is a breaking change (announcement email)

How to update your code
With this proposed change, if you have code that is just wrapping a ButtonBar in ButtonTheme.bar:
ButtonTheme.bar(
  textTheme: ButtonTextTheme.accent,
  child: ButtonBar(...)
)

you can simply remove the ButtonTheme.bar wrapper and move the parameters to the ButtonBar constructor:
ButtonBar(
  buttonTextTheme: ButtonTextTheme.accent,
  ...
)

If you would like to restore the default ButtonTextTheme used by all ButtonBar widgets to ButtonTextTheme.accent, you can just add the following to your app's theme:
MaterialApp(
  theme: ThemeData(
    buttonBarTheme: ButtonBarThemeData(
      buttonTextTheme: ButtonTextTheme.accent,
    ),
  ),
  ...
)

@darrenaustin I just landed https://github.com/flutter/flutter/pull/36880, could you update your PR to include the changes for ButtonBar in MaterialBanner too, if its not too much trouble?

@darrenaustin I just landed #36880, could you update your PR to include the changes for ButtonBar in MaterialBanner too, if its not too much trouble?

No problem at all. In fact I need to anyway, as the two themes added to ThemeData collide.


Changed text for service context menu 'remove' item

Related to #711
==========
Changed text for services context menu.

Pull Request Test Coverage Report for Build 79

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 39.192%

Totals

Change from base Build 67:
0.0%

Covered Lines:
2541

Relevant Lines:
4818

💛  - Coveralls


Function World.gravityAcceleration should not be protected

Reported by fcasella on 10 Feb 2010 10:29 UTC
Models in the multibody libraries access the gravity acceleration function of the world object instance by dot notation, e.g.
g_0 = world.gravityAcceleration(frame_a.r_0 + frames.resolve1(frame_a.R, r_CM));

However, the function gravityAcceleration is declared as protected in the World model. According to the Modelica Specification 3.1, section 4.1, "Protected elements in classes cannot be accessed via dot notation".
The function should then be moved to the public section of the World model, otherwise the Body model of the multibody library is illegal.

Migrated-From: https://trac.modelica.org/Modelica/ticket/272

Comment by otter on 26 Jul 2010 16:48 UTC
Fixed in 6054d20627bc71d582121813ab0be0529986eabe.
Additionally, function gravityAcceleration is made replaceable, so that user-defined gravity fields can be defined.


fix: update "run tests" instructions to use local instead of global jest

Description of changes
Updates README "Run the tests!" section to run the locally installed jest instead of the globally installed jest
Pull request checklist

[x] If this PR addresses an existing issue, it is linked: Fixes #31
[x] yarn test passes in all affected samples
[x] Added any applicable tests

Should we also update the screenshot to show the command (yarn test instead of jest)?


Fix linter stdout/stderr assertions in the test suite

I noticed while working on a new linter that my test was passing even though the stdout/stderr was incorrect.
This seems to be fallout from this refactor:
https://github.com/samuelmeuli/lint-action/commit/73e9574651632c461b95746ebb53b6f117d99fb6
I had to update the timezone date regex to allow my timezone as well.
I also updated the gofmt linter, since it sends its output to stdout. I also fixed the tests for it.
I updated the prettier test to remove the fix assertion on stdout because, on my machine, it outputs something like:
    Expected: ""
    Received: ".prettierrc.json 26ms
    file1.js 18ms
    file2.css 45ms
    package.json 6ms"

to stdout.

The linter check doesn't seem to be checking out my branch properly (since it's on my fork, not on this repo):
  stderr: "error: pathspec 'fix-assertions' did not match any file(s) known to git\n"

The "Lint" workflow is currently failing because of #13, that's not an issue with your code :)

Thanks!


Update k8s.io/utils digest to b307cd5

This PR contains the following updates:

Package
Type
Update
Change

k8s.io/utils
require
digest
3b25d92 -> b307cd5

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

merged at https://github.com/cybozu-go/pod-security-admission/pull/105


Follow 1215

https://issues.dlang.org/show_bug.cgi?id=1215
https://github.com/D-Programming-Language/dmd/pull/4516

This was fixed (there was a follow up P.R.: https://github.com/D-Programming-Language/dmd/pull/4918 ).

Related<EMAIL_ADDRESS>

This works since a while:
template Foo(T...)
{
    alias MyType = T[0].TheType;
}

template P(T)
{
    alias TheType = T;
}

void main()
{
    Foo!(P!int).MyType i = 0;
}

And dparse parses the "indexer" since a few week


[AJ-1310] Hide new workspace option when importing protected Azure data

https://broadworkbench.atlassian.net/browse/AJ-1310
Protected data can only be imported into a protected workspace. And TDR snapshots can only be imported to workspaces on the same cloud platform. Thus, protected Azure snapshots can only be imported into protected Azure workspaces.
There is currently no way to create a protected Azure workspace via Terra UI (it requires a direct API call). Therefore, there is no reason to show the "Start with a new workspace" option when importing a protected Azure snapshot, because that option cannot create a workspace that satisfies the import's requirements.
This PR hides the "Start with a new workspace" option when importing a protected Azure snapshot.

Bit of a nit - the page says "Choose the option below that best suits your needs" - but there's only one option. Should we remove that line in this case?

https://github.com/DataBiosphere/terra-ui/pull/4349/commits/1037e31de2be27956bd583dfd8a8e6690a9ed6f7 makes it so that message is not shown when there's only one option.

Presumably at some point we are going to allow creation of protected azure workspaces via the UI - is there a plan to put this back at that point? Is there a ticket for it?

Yes. WOR-1033 looks like the relevant ticket for allowing creation of protected Azure workspaces. I don't know what the timeline for that is.


hot reload on save with the flutter run CLI tool

Use case
It's convenient to have hot reload on save and not have to switch back and forth between a terminal and an editor, but it seems like support for this is only provided via plugins for specific IDEs.
Why not incorporate watching input files and reloading on changes into the CLI? It seems to make more sense than to implement that separately for each "officially supported" editor/IDE or leave users to hack together their own shell scripts to the same end. While some web frameworks do this by default, why doesn't flutter have this option?
Proposal
flutter run should watch input files for changes and automatically hot reload when a change occurs. Maybe a commond-line switch would make sense to turn this on/off and/or this could be toggled via typing in some character to a running flutter run process, like which is already being done with e.g. i for toggling the inspector

duplicate of https://github.com/flutter/flutter/issues/62679

thanks for pointing me there, I somehow couldn't find that issue when searching


BottomNavigationBarItem.tooltip is not displayed when set

Steps to reproduce
Run the code and hover on the bottom destinations.
Expected results
Tooltip should be shown when hovering a destination.
Actual results
Tooltip isnt shown when hovering the destinations.
Code sample
Code sample
import 'package:flutter/cupertino.dart';

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

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      title: 'Cupertino Tab Bar with Tooltips',
      theme: CupertinoThemeData(
        primaryColor: CupertinoColors.systemBlue,
      ),
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _selectedIndex = 0;

  @override
  Widget build(BuildContext context) {
    return CupertinoTabScaffold(
      tabBar: CupertinoTabBar(
        items: [
          BottomNavigationBarItem(
            icon: Icon(CupertinoIcons.home),
            label: 'Home',
            tooltip: "I am not displayed",
          ),
          BottomNavigationBarItem(
            icon: Icon(CupertinoIcons.search),
            label: 'Search',
            tooltip: "I am not displayed",
          ),
          BottomNavigationBarItem(
            icon: Icon(CupertinoIcons.settings),
            label: 'Settings',
            tooltip: "I am not displayed",
          ),
        ],
        currentIndex: _selectedIndex,
        onTap: (index) {
          setState(() {
            _selectedIndex = index;
          });
        },
      ),
      tabBuilder: (context, index) {
        return CupertinoTabView(
          builder: (context) {
            return CupertinoPageScaffold(
              navigationBar: CupertinoNavigationBar(
                middle: Text('Tab ${index + 1}'),
              ),
              child: Center(
                child: Text(
                  'Selected Tab: ${index + 1}',
                  style: CupertinoTheme.of(context)
                      .textTheme
                      .textStyle
                      .copyWith(fontSize: 20),
                ),
              ),
            );
          },
        );
      },
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Running on dartpad: Based on Dart SDK 3.5.4 and Flutter SDK 3.24.4

@ndonkoHenri
The tooltip isn't shown is correct as mentioned in https://github.com/flutter/flutter/issues/107036#issuecomment-1178381935, so I think this is WAI and hence closing as not an issue. If you disagree, write in comments and I'll reopen it.

@darshankawar thanks for responding.
I think the issues are different.
In my case, I am setting the tooltip property and it isn't showing up, which doesn't correspond to the behavior mentioned in the documentation.
The issue you linked goes about the default text of the tooltip. That's not my concern.

change the default to just not display a tooltip instead of displaying the label text

To be brief, setting the tooltip property has NO effect. It is useless.
Let me know if I am wrong and that's the expected behavior.

Thanks for the feedback. I now see what you mean and after revisiting the linked issue and comment, it seem different from what I interpreted.
Using the code mentioned in this comment, it does show the tooltip as below. So please check the same to see if it helps in your case or not.

@darshankawar, yes, that code works.
But mine, though correct, doesn't work, that's the issue. In mine I make use of alot of cupertino components and I don't want to switch to material components.
Can you please give my code a try?

Thanks for the update. I see that using CupertinoApp or widgets, the reported behavior occurs, but not with MaterialApp as mentioned in this issue.
I will keep the issue open for team's tracking.
stable : 3.24.4
master : 3.27.0-1.0.pre.443


[HUDI-4636] Output preCombine&partition fields of delete records when changelog disabled

Change Logs
when changelog disabled, flink will output pk columns only,
https://github.com/apache/hudi/blob/master/hudi-flink-datasource/hudi-flink/src/main/java/org/apache/hudi/table/format/mor/MergeOnReadInputFormat.java#L364
We need the preCombine and partition fields also, so pull this request.
Impact
Describe any public API or user-facing feature change or any performance impact.
Risk level: none | low | medium | high
Choose one. If medium or high, explain what verification was done to mitigate the risks.
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

We need the preCombine and partition fields also, so pull this request.

Can you explain why we need this then, do you want to write to another hudi table using these records ?

We need the preCombine and partition fields also, so pull this request.

Can you explain why we need this then, do you want to write to another hudi table using these records ?

we want to write this delete records to a warehouse or database, like apache phoenix, and want the precombine&partition field

We need more discussion on this.


scale-up instructions missing information

Is this a request for help?:
Yes
Is this an ISSUE or FEATURE REQUEST? (choose one):
FEATURE REQUEST
What version of acs-engine?:
0.8.0

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
kubernetes
What happened:
I have a running cluster (created with acs-engine 0.7.0) with an agentpool with 3 nodes, I will like to scaleup creating a new agentpool (different VM size and attached disks).
I found this https://github.com/Azure/acs-engine/tree/master/examples/scale-up and a couple of closed issues but I cannot find which commands need to be run.
What you expected to happen:
a list of commands describing how to proceed
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:

Things I tried:

acs-engine upgrade : as for description this is to upgrade k8s not scaleup
acs-engine generate : trying to run generate in any of the files provided in scale-up dir throw a validation error

So previous described error is related to https://github.com/Azure/acs-engine/issues/1159 but there is not a workaround

using current master solves previous error with deploy, but now I get this error:
using az group deployment create
Deployment failed. Correlation ID: a10febc8-024b-4c2c-cc24-cdfe147c0355. {
  "error": {
    "code": "PropertyChangeNotAllowed",
    "target": "imageReference",
    "message": "Changing property 'imageReference' is not allowed."
  }
}

Hi @jalberto, is https://github.com/Azure/acs-engine/blob/master/docs/kubernetes/scale.md this what you are looking for? You can use the following command:
acs-engine scale --subscription-id 51ac25de-afdg-9201-d923-8d8e8e8e8e8e \     --resource-group mycluster  --location westus2 \     --deployment-dir _output/mycluster --new-node-count 5 \     --node-pool agentpool1 --master-FQDN mycluster.westus2.cloudapp.azure.com

Yes! thanks
On Sat, 4 Nov 2017 at 01:28 Cecile Robert-Michon<EMAIL_ADDRESS>wrote:

Hi @jalberto https://github.com/jalberto, is
https://github.com/Azure/acs-engine/blob/master/docs/kubernetes/scale.md
this what you are looking for? You can use the following command:
acs-engine scale --subscription-id 51ac25de-afdg-9201-d923-8d8e8e8e8e8e
--resource-group mycluster --location westus2 \ --deployment-dir
_output/mycluster --new-node-count 5 \ --node-pool agentpool1 --master-FQDN
mycluster.westus2.cloudapp.azure.com
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/acs-engine/issues/1615#issuecomment-341857753,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGGV1d1ZLIhjeHhkaDun8WlplgbGWkkks5sy6-SgaJpZM4P-YBW
.


Fixed (BE) #5849 -- Users can't publish page without translation

Fixes #5849

Coverage remained the same at 82.604% when pulling bdf2f083035b06dc35c5f731dccc01ff700c1332 on czpython:fixes/3.4.x/5849 into ec42adb3c7bf0aa96be33e78ff3f43b501ec0794 on divio:release/3.4.x.

Closing as this will be fixed by a different PR.


Interaction between unsigned and pointer/array

Using an operator[] on a type that takes size_t works great with unsigned types. They convert to primitives.
However operator[] on a pointer is built in and takes a signed ptrdiff_t. Converting from an unsigned value to signed ptrdiff_t could overflow and thus needs to check(). But we want implicit signed conversions to not compile.
There's no way I can think of to limit the conversions to only happening inside ptr[x]. If the compiler would convert that to *(ptr + x) then it would work fine, but it doesn't. It needs x to become ptrdiff_t.
Allowing conversion to ptrdiff_t also allows conversion to long (on linux) and int64_t which is much too broad.
What do?

A method on unsigned types to check+convert?
Expect code with pointer indexing to be converted to span/array/vector/etc first? This should be happening regardless but ordering dependencies are not good.
Rewrite it to *(ptr + x)?
Introduce -funsigned-pointer-index which adds a size_t overload???
???

TODO: Make the better problem description doc public, and bring some kind of proposal to Clang.


[GH-5951] Update Plugins > Mattermost Boards, Playbooks, User Satisfaction Surveys

Summary
This PR updates the Plugins > Mattermost Boards, Playbooks, and User Satisfaction Surveys settings to the table-based format.
Ticket Link
This PR partly addresses https://github.com/mattermost/docs/issues/5951.

Hello @matthew-src,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.


wip/modify spiceDB schema management to configure more flexible permission and roles

closes #169

@krtkvrm https://odpf.github.io/handbook/engineering/commits :)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 17 (100.0%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 61.652%

Files with Coverage Reduction
New Missed Lines
%

internal/api/v1beta1/role.go
1
77.19%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
4688

Relevant Lines:
7604

💛  - Coveralls


implemented new gp-autocomplete design

:rocket: deployment of register-with-a-gp-beta-web succeeded (http://register-with-a-gp-beta-web-pr-197.dev.beta.nhschoices.net)

:rocket: deployment of nhsuk/register-with-a-gp-beta-web succeeded (http://register-with-a-gp-beta-web-pr-197.dev.beta.nhschoices.net)


Surfacing events for pods created by config files.

Also add column Source to the output of 'kubectl get events' so that we know the events are from node.

Addressed comments. PTAL?

Travis issue is fixed by #3256 now.


Separate 64 bit builds from 32 bit builds of WIN

Signed-off-by: Damien Zammit<EMAIL_ADDRESS>

This is completely unnecessary.
Just add "-m32" to your CFLAGS, CXXFLAGS and LDFLAGS, or call the 32bit compiler.


Fixes 'Connection refused' issue when running the example docker file.

The example docker-compose file is failing to publish to elasticmq. The log output
shows the message 'Connection refused'. This is because the Caml aws component will
first list the queues to get their urls. Elasticmq has not been configured with a
different host name so it will return "http://localhost:9324/queue/queue1" instead of
"http://sqs:9324/queue/queue1".

Codecov Report

Merging #33 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #33   +/-   ##
=======================================
  Coverage   37.18%   37.18%
=======================================
  Files          21       21
  Lines         320      320
  Branches        8        8
=======================================
  Hits          119      119
  Misses        201      201

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 b121896...98ec505. Read the comment docs.

thank you, @sam-io!


Adjust AmgX to use JACOBI_L1

This PR also fixes a bug when using AmgX defaults as preconditioner - I was hitting errors based on how it was configured.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#2345
@artv3
@tzanio
@pazner + @dylan-copeland
06/22/21
⌛due 07/06/21
⌛due 07/13/21

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

Note that PR #2348 is necessary for testing with the latest version of hypre. In the meantime, for testing PRs like this, we have to use an older version of hypre.

Merged in next for testing...


Bugfix/fix enter

not really a "fix" more like a bandaid
newline seems to work fine and uses operations
newLi is way too complex to handle
added tests to document and confirm current behavior
one test breaks because the actual behavior is so far off from desired I can't let it pass in good conscience
Unsure if this is a regression or if the case was already broken
in any case the whole command needs a rewrite

Sidenote this adds a move operation, which works but is probably not useful yet, and is not tested properly
I left it in because the work to abstract the operation algorithms into their own utility class is useful


Test Issue

Bug Description
Some sample text.
No
@DarumaDockerDev
frpc Version
0.5.1
frps Version
0.2.5
System Architecture
linux/amd64
Configurations
No configurations.
Logs
No response
Steps to reproduce

...
Affected area

[ ] Docs
[ ] Installation
[ ] Performance and Scalability
[ ] Security
[ ] User Experience
[ ] Test and Release
[ ] Developer Infrastructure
[ ] Client Plugin
[ ] Server Plugin
[ ] Extensions
[ ] Others

Is there any problem @DarumaDockerDev ?


Import Categories and Concepts page - Validate headers

When admin user uploads an Information Architecture doc, system checks that file contains correctly formatted row headers for each tab
As a an F&E Admin user
I want to check that the IA doc I am uploading includes the correctly spelled column headers in order to ensure a complete IA doc is uploaded
Scenarios
Scenario 1 - Header presence
Given I am a admin user who is logged in
And I am on the Import Categories and Concepts page
And I have pointed the 'Choose file' option towards an Excel file that on the 'Demographics' tab has the column 'Category (L1) Description' missing
And all the tabs are correctly formatted
When I select upload
Then I receive a message that 'The file you are trying to upload appears to be missing one or more columns – please check the column exists and the title is correctly formatted: Demographics \ Category (L1) Description'
And I see a button that takes me back to the 'Import Categories and Concepts' page
Note 1: Headers to be checked are:
Category (L0)
Category (L0) Description
Category (L1)
Category (L1) Description
Sub-category (L2)
Category (L2) Description
Sub-category (L3)
Category (L3) Description
Concept level	Concept Description
NPD Alias 1
(record may have more aliases, but we only want to check for first)
Note 2: Display of reported missing headers should include the tab they are missing from as well, so format should be '<Header Name>'.
Note 3: If more than one header is missing, present as a list.

Tested and complete


Option to link multiple 'Pains'

I recently commented on the thread for adding triggers, which I second, and ways to improve the comments section as a substitute.
Please let me know if I should move any of those to a separate issue :)
Another thought I had, was linking multiple pain types.
Example being, you would start a timer, you choose the (primary) pain type from your 'Pains' list, and everything runs as normal for the main purposes of the app.
But in the diary where you would add description, medication etc,
there would be an option to add extra/secondary pains, symptoms- and potentially triggers etc, whatever the user has in the 'Pains' list and decides to choose.
This could free up the description, maybe reducing the need for multiple entries, standardise responses for multiple/fluctuating pain types, and importantly ease of use - which is hard with headaches and migraines :)
Good for doctor reports, good for us too!
Let me know what you think of any of these ideas, and if you'd be interested in implementing them in any way!

Thank you also for this comment!
That sounds good too. Maybe you could just make a list of unspecific texts. Then you could simply tag your diary entry with these terms.
As soon as I have more time, I will think about it and get back to you.

I really like this idea, that's why I came here :)
I was looking for something else than the description to save whether it was an attack with aura or not and whether I was photophobic or not.
I never did that much Android development, but If there's something easy I can help with, please let me know :)
Thank you for your great work!


PGA output file name override

Dear CRABS team,
Firstly, thank you for creating such a user-friendly resource. I have found CRABS almost worryingly easy to use and follow which is great. I am having a few issues though, particularly with the PGA step. The output appears to be saving as 'CRABS_pga.fasta' regardless of my output specification. I ignored this at first, but the file doesn't seem to overwrite when rerunning this step, and I'm having some other downstream issues (described below) with no discernible source, so I'm wondering if this is indicative of any other issues. Here's my input:
docker run --rm -it \
  -v "$(pwd)":/data \
  --workdir="/data" \
  quay.io/swordfish/crabs:0.1.4 \
  crabs pga --input arthropoda_coi.fasta --output Arthropoda_coi_merged_LCO230_pga.fasta --database Arthropoda_coi_merged_LCO230.fasta --fwd GGTCAACAAATCATAAAGAYATYGG --rev CTTATRTTRTTTATNCGNGGRAANGC --speed medium --percid 0.95 --coverage 0.95 --filter_method strict

I am using CRABS with DADA2 and getting a lot of very poorly resolved assignments or misassignments (ASVs identified as exotic decapods from my terrestrial insect samples). When I BLAST these same sequences, I get reasonable terrestrial insect species (I understand the issues with BLAST, but the results appear to be relatively unambiguous and for insect COI metabarcoding I would expect a fairly decent resolution/coverage), so I can only assume that the above issue could be causing this somehow, but any other suggestions on how to resolve this would be greatly appreciated.
This may simply be my naivety though, and perhaps a user error or misunderstanding!
Many thanks,
Jordan

Hello @jordanpcuff,
Thank you for using CRABS!
I wonder if your problem is associated with the docker installation, since I cannot recreate the issue from the GitHub and conda instalments. @hughcross, could you please see if you can recreate the above mentioned issue in docker?
Just FYI, if the PGA step is not working as intended, I'm not surprised you are currently seeing some "weird" taxonomy assignments for your insect data. With COI, there will be a lot of reference barcodes that have either used one or both of your primers, which will have been cut off and currently not present in your ref DB. Once we get the PGA step working, you could try and restrict your reference sequences to non-marine organisms to see if it improves the taxonomy assignment.
Best,
Gert-Jan


Imgui

Integrate the lib, make it work across dll boundaries, seems to be fully working. Next to actually make something with it

Only things left to do are actually probably a decent amount of effort, such as making a better platform abstraction layer for imgui and making multiple viewports working so we can drag imgui windows outside the main window


Generate status for CPU compute pressure

Support on Chrome desktop only is correct for now. Edge 125 has not been
released yet, and the feature doesn't work on Android.

@captainbrosset this doesn't fix the Edge data, but it does mean that the next release of BCD will.

Edge 125 is marked as "current" in BCD now and has been for a few days. So, yeah, I guess we just need to wait for the next BCD release. I thought BCD was released twice a week though. Last time was last week. Do you know if that's expected?

@captainbrosset this is fixed in https://github.com/web-platform-dx/web-features/pull/1148.


feat(ast): complete AccessorProperty todo in has_decorator

[!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

Current dependencies on/for this PR:

main

PR #2176

PR #2178   👈

This stack of pull requests is managed by Graphite.

Merge activity

Jan 26, 2:42 AM: @Boshen started a stack merge that includes this pull request via Graphite.


[web: canvaskit] support specifying CanvasKit URL in debug builds

A custom CanvasKit URL can be specified using the FLUTTER_WEB_CANVASKIT_URL environment variable. However, environment variables don't work in DDC, only in dart2js. This prevents users from loading CanvasKit from a custom URL in debug builds.
We should find a way to specify custom URLs that's compatible with debug builds. For example, we could provide a runtime JS API to set it, such as window.flutter.canvasKitUrl = 'http://...'; that could be called prior to initializing the engine.

They do work in ddc, but not in the engine binaries since these are prebuilt. if we could inject the value from either the generated entrypoint or somewhere else in the framework it would be able to work in debug mode

any update on this issue?

Any update on this?

I'd say this is already implemented.
window.flutterConfiguration = {
  canvasKitBaseUrl<EMAIL_ADDRESS>}

Seems to work when I do flutter run -d chrome?
I'm closing this issue as resolved, if it isn't, feel free to reopen!


⚠ PrivateHD has degraded performance

In b3960cd, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1597 ms

Resolved: PrivateHD performance has improved in 0cedc5d.


System default Min/Max/Close buttons

Since I applied this theme, Firefox now doesn't use my system's min/max/close buttons

They don't even match the theme's color.
I'm on Windows 10 and I am using the Arc theme, and the buttons look like

I've searched on reddit and a user said that it can be fixed by commenting out or deleting tags titlebar-min, titlebar-max and titlebar-close on the .css files, however, Material Fox doesn't seem to have any.
Is there anything that can solve this issue?

That's really odd. I don't have a custom theme installed to test at the moment but could you try:

Remove userChrome.css and restart Firefox
If the problem has gone, add userChrome.css back and
Remove _tabbar.css and restart Firefox and let me know what happens

I'm only modifying the titlebar buttons on macOS so not sure why your buttons would be affected. Doing the above will confirm that the problem is in my code and then I can look further into it.

Looks like it isn't a bug in this theme so I'll mark it as closed, but if you find a solution, just open another issue and I'll see if I can integrate it.


Overnigth tests are failing: Seems runners doesn't have enough memory

See original issue on GitLab
In GitLab by [Gitlab user @jjardon] on Jan 15, 2019, 17:06
Job #145714925 failed for 3768e3e9578d37bfab1dde4b154c6c8bee6cfb78:
When building vulkan:
virtual memory exhausted: Cannot allocate memory

In GitLab by [Gitlab user @jjardon] on Jan 18, 2019, 10:41
changed the description

In GitLab by [Gitlab user @jjardon] on Jan 18, 2019, 10:42
changed the description

In GitLab by [Gitlab user @jjardon] on Jan 18, 2019, 10:42
Upgrade the overnigth runners to 6 cores/ 16GB machines (It was 4 cores / 8GB) seems to have fixed this, but no-cache test are still failing: #874

In GitLab by [Gitlab user @jjardon] on Jan 18, 2019, 10:42
closed


HeavySelect2MultipleWidget & HeavySelect2Widget do not set default data-minimum-input-length

When using HeavySelect2Mixin together with Select2Mixin (For example HeavySelect2MultipleWidget & HeavySelect2Widget), default data-minimum-input-length will be 0 instead of 2. Due to Python's method resolution order, Select2Mixin.build_attrs is the first one to set data-minimum-input-length. Because it is already set, it won't be updated in HeavySelect2Mixin.build_attrs.
It seems that same is true also for Select2TagWidget, Select2TagMixin and Select2Mixin combination.

I remember someone was already working on this for other cases. I'd much appreciate a pull-request for your side. Thanks!

Closed: This issue has been inactive for an extended period of time. Please reopen the issue if it is still relevant.


DBZ-2788 Replace MySQL connector option with SMT for mitigating wrong op flag

https://issues.redhat.com/browse/DBZ-2788

Revert back DBZ-2775
Introduce ReadToInsertEvent SMT.

@jpechane up for final review. Thanks.

Rebased and applied; thx a lot, @ani-sha!


Wizard step validators added in canenter/canexit unaware of wz-previous or wz-next direction

If you have some inputs on one of the steps that requires validation, it would most certainly be done via canexit callback. But canexit is called regardless of wz-next or wz-previous is pressed.
The weird problem I faced in my application is that I cannot return to the previous step unless all fields are filled up correctly, which is not the desirable effect. It would be perfect if canexit callback was aware of direction we are jumping to and take a decision whether to run validation or not.
The way I found to work around this problem is simply by moving validation logic to canenter of the next page (this allows to use wz-previous without any constraints). Of course, I am not using steps indicator and the switching between the steps is done without jumping, straightforward next or previous. As you can see, this is not too much convenient.
Take a quick look if you want to fix it, it should be very easy: I guess a matter of few-lines fix.
Thanks.

Tiny patch introduced in pull request#75.

Tiny patch introduced in pull request#75.

@vtytar I made the changes and am pushing it live so you can go back in steps without validating.  I am closing this ticket and your pull request, see my comments on the PR.
Thanks for pointing this out and your contributions!

@vtytar I made the changes and am pushing it live so you can go back in steps without validating.  I am closing this ticket and your pull request, see my comments on the PR.
Thanks for pointing this out and your contributions!

Thanks.


Add Japanese Generation names

Japanese generation names are missing from generation_names.csv

This probably isn't the appropriate place to ask this, but do you guys have a standard for the use of latin numerals in Japanese text? Generation I, for example, could be written 第1世代 or 第一世代. For a title like this, 第一世代 is definitely more common, so in this case, I would say it's better. Are these typically being handled on a case by case basis like this? Or is there some more general rule being applied to decide which one will be stored?

Added in PR #253

This probably isn't the appropriate place to ask this, but do you guys have a standard for the use of latin numerals in Japanese text?

Most of the text is ripped from the games; i don't think we have any Japanese text that has been translated by hand. So the short answer is "no".
If you can find it in any official sources, go with what they use. Otherwise i'd use whatever spelling is more common in the fandom.
I will note that the games tend to use numerals -- eg ２番道路 (Route 2). Also, if we go with numerals they should probably be fullwidth -- 第１世代, not 第1世代 .

Thanks, all good information. I grew up in Japan and can speak/write at native level so I can probably help out with translations for fields that can't be ripped from games (such as the move learn methods). There is a really dedicated fandom for Pokemon in Japan, so looking on Japanese wikis or just using my own experience playing the games in Japanese will probably be enough to get by :) I will go ahead and use full width Arabic numerals whenever I find a place where Arabic numerals are more appropriate.

Oh nice, i'll trust your judgement then.


Let AnnotationHelper respect targetVersion

this should avoid the problem of using the wrong annotation when building on jdk 11 but targeting java 8

(in case you consider squash merging this, please use the my same email as on the commit, thanks)

Do I need to attach any additional libraries if I generate on Java 17 targeting for Java 8?

Superseded by #1482.


[BUG] Compile source code error for ROCM platform when using #include <hip/hip_bf16.h>

As mentioned in #https://github.com/vllm-project/vllm/issues/2646
Building for ROCM platform will cause  multiple definition error,
This is caused by The function in <hip/hip_bf16.h> being defined without static inline , which will cause multi definition error when using this header file in multi .cu/.hip files.
#https://github.com/ROCm/HIP/issues/3403
The error will be fixed in future releases of ROCM, but till ROCM-6.0.2, this has not been fixed yet, maybe we need to find a way to bypass this.
Maybe using hip_bfloat16 instead of __hip_bfloat16? #https://github.com/ROCm/ROCm/issues/2534, but needs some code modification.

Running into this as well

I am working on a fix for this issue now. Will update when I have a pull request

Fix for vllm ROCm 6.0 build is here: https://github.com/vllm-project/vllm/pull/2790


Update working-with-the-docker-registry.md

Why:
Closes [issue link]

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Atualizar working-with-the-docker-registry.md

Atualizar working-with-the-docker-registry.md

This doesn't appear to make any visible changes, so I'm going to close this now.
Take a look at the contributing.md to learn how to contribute in this repo. You can open a discussion if you have any questions or check out issues with the help wanted label. :sparkles:


Add definitions for FBbt-mapped terms

This PR adds definitions for a bunch of terms that are mapped to corresponding terms in FBbt but were lacking definitions (listed in #1929). When needed, some terms are re-classified so that their classification reflects their meaning.
Some terms are obsoleted rather than being defined:

neuroepidermoblast (unused term in the literature);
scolopidial sheath cell (merged with scolopale cell).

The PR does not address the case of follicle stem cell (which is the subject of a distinct issue: #1943) and of hemocyte and related cells (which requires more work than merely adding definitions – I intend to review the entire hemocyte hierarchy in CL).
closes #1929

@gouttegd, I believe the conflict has resulted from the refreshed imports and CL release yesterday. This may be resolved if you refresh your branch with the latest master/imports.

Yes, I had already merged the imports from master just before submitting to fix a prior conflict, but with the release happening in the meantime, that was to be expected.
I’ll refresh again just before merging, once the PR is approved.

Thank you for all the updates and your comments.
Once the conflicts are resolved, i will approve the PR.

Thanks @aleixpuigb . The conflict affecting merged_import.owl is fixed.


only lock updates around the actual update

This corrects some 20 second plus lock holds we were seeing in the letsencrypt/boulder that were causing timeouts and 500s returned to users.
It introduces a new fsLock that will lock around both the updates and the file system effects to allow code reading from the tries to make forward motion while preventing other filesystem modifications at the
same time.
It also moves the updateLock to updateLookupMap.
Depending on how heavy the trie operations are in updateLookupMap and how large the tries get, we may want to either a) clone the tries andthen pointer swap them in or b) make the locks occur only around each exact operation. For instance, in the PREFIX_4B_SZ, CHUNK_TYPE_SUB case, we could allow the Lookup.Delete to occur in a write lock, take a read lock to pull the data from the FullHashes iterator, and only then go back with a write lock around with a write lock around each individual FullHashes.Delete.
Since all of this happens under fsLock, we don't have worry about multiple SafeBrowsingList.load calls stomping on each other's toes.

Just wanted to let you know that this caused a 7.5x reduction in 500s coming out of Let's Encrypt. Thank you much for taking this patch!

thank you for contributing it!  I am delighted you are using the library in Let's Encrypt, really pleased I could make a small contribution to the cause.


trying to add Stargate Auth in aspdot net core api

https://github.com/stargate/api-extensions/issues/13#issue-864414763

https://github.com/stargate/api-extensions/issues/13#issuecomment-824551214
@dougwettlaufer  my company is using stargate for authorize api based on this proxy and namespace thing... i have updated proxy and name-space  sample file in issue details.


merge package template from Kronos-Tools/npm-package-template

LICENSE

fix: update LICENSE

Codecov Report

Merging #101 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #101   +/-   ##
=======================================
  Coverage   83.33%   83.33%
=======================================
  Files           1        1
  Lines          12       12
=======================================
  Hits           10       10
  Misses          2        2

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 efce1aa...b5aa6a6. Read the comment docs.


Antvis G6 in Nuxtjs

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

Reproduction link

Steps to reproduce
Install antvis G6
import G6 from '@antv/g6' in any Nuxt pages and components
=> Result in error document is not defined

Environment
Info

g6
3.7.1

System
Ubuntu 18.04

Browser
Chrome Firefox (latest)

Can you add detail import and usage of Antvis G6 in Nuxtjs and Vuejs?

AntV G6 in react：https://github.com/baizn/g6-in-react

Hi, dear user. Since there was no response for a long time, we cannot figure out the answer of the problem you described. If there is new progress, feel free to open a new issue.